Support youtube playlist

Heya, would you support having a youtube-plist directive?
.. youtube-plist:: XYZ

The embedding HTML is essentially the same, but using the playlist shortcode like: https://www.youtube.com/embed/videoseries?list=XYZ
If yes, then I can probably make a PR

Note, annoyingly I can't see a trivial way to get the thumbnail, without using the Google API (and thus needing an API key: https://stackoverflow.com/questions/10628760/how-to-get-a-youtube-playlist-thumbnail. This does work, though.
So then you would perhaps need to always provide two pieces of information:
.. youtube-plist:: XYZ
    :thumbnail: LMN

Sure, it seems like a natural thing to do. I'd prefer youtube-playlist because readability is more important than saving 3 charachters.
Thanks.


[BUG] currently does not support client body read timeout

From the code:
https://github.com/godwokenrises/godwoken/blob/2d8ad3510fcd528dfbfe76242b9572acdcf25084/crates/rpc-server/src/server.rs#L51
Seems godwoken also has a similar issue with ckb, https://github.com/nervosnetwork/ckb/issues/3880
I didn't verify it on godwoken, but from the code the rpc does not has a limit for a request, you can verify it with curl:
curl --location 'http://<IP_ADDRESS>:3000/rpc' \
-H 'content-length: 100' \
--header 'Content-Type: application/json' \
--data-raw '   {
        "id": 42,
        "jsonrpc": "2.0",
        "method": "@ping",
        "params": [
        ]
   }'

An possible fix is to add a timeout middleware: https://github.com/chenyukang/jsonrpc-utils/blob/810f6a36eec3ce5be8673c11311bf1dcc506c3f9/examples/server-timeout.rs#L39

Godwoken rpc is usually proxied by web3 / nginx etc., not exposed directly.
But adding a client body size limit / timeout is pretty easy, so I will add it just in case.

@chenyukang Just use TimeoutLayer from tower-http, no need for ServiceBuilder and HandleErrorLayer.

Got it, thanks.


Common data model

Functionality:
Common data model
Solution:
required changes done
WtId:
Dev617
Risk level:

[x] high
[ ] medium
[ ] low

How to test:
go to TeamDetails page and add github repo and then add gitlab repo its working using same service

The PR is made on old code and is not fit according to new architecture, not a good design


Monitor Storm via the REST API

What's this PR do?
This adds storm_rest_api.py (and tests), a check that crawls Storm 0.9's REST API interface, and reports metrics from that. The great thing about this one is that it includes metrics you don't see anywhere else (neither the built-in metrics consumers nor the Thrift interface report the capacity, for example).
What is left to do?

[x] Add a thing to add custom per-storm task ID tags from a json file
[x] fix any questionable python (-:
[x] see if we can turn down the check interval for this one - it takes 22s to retrieve the topology details.

@gphat - I think this is beginning to look REALLY good. I will try this out in QA tomorrow (without extra tags) & we can polish it some more then.

omg the tinyness of this in comparison to the toil involved in the java-side stuff…

I believe this is ready for review.
R? @gphat

Never mind the review request - I still have to find a way to whitelist per-executor stats (:

OK, I think this is ready for inclusion! (Provided we don't need to play games with caching the data locally)
R? @gphat

OK, this is FINALLY REALLY REALLY ready for reviewing.
R? @gphat

👍
I've tried to be thorough, but I still feel like I've not shaken this out as much as I could. I think since this is not a "inline" collector, it's safe to right ahead and accept the fact that we'll iterate. These are new metrics after all!
Nice work @antifuchs!

Squashed into one again! (:

Thanks! & YAY that we can merge this!!!


fix(client): fix Marker Settings flakey test

The embedded client's Marker Settings test suite flaked last evening (link) during the nightly smoke test run.
This should hopefully prevent that from happening again in future.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (3efb5f4) 80.96% compared to head (ea89d76) 80.96%.
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #420   +/-   ##
=======================================
  Coverage   80.96%   80.96%
=======================================
  Files          62       62
  Lines        4843     4843
=======================================
  Hits         3921     3921
  Misses        766      766
  Partials      156      156

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


Fix package references

Fix package reference to System.Collections.Immutable (was an assembly reference with hintpath)
Remove package reference to StackExchange.Redis (not needed).

The secured variables cannot be decrypted when the PR is created from an external fork... AppVeyor limitation. I think I'll skip SonarCloud if the pull request head repo is not this repository (APPVEYOR_PULL_REQUEST_HEAD_REPO_NAME.)
Can you please grant me permission for editing the PR?

Ah yeah no secrets on PRs...
You should already have edit rights.

This is checked:

Thanks @damianh!

You're welcome :)


Getting ERROR: Unable to obtain version info; Please try again later or Visit  https://github.com/yt-dlp/yt-dlp/releases/latest

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2022.10.04 (update instructions) or later (specify commit)
[X] I've searched the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
I'm downloading many files in parallel using the following configuration
ytvp () {
        TIMESTAMP=$( date +%y%m%d%H%M%S )
        NAME=$( sanitize_linux_filename "$1" )
        LOG_NAME="yt-dlp_${TIMESTAMP}_${NAME}.log"
        yt-dlp --format "(bv+(wa[abr>=64]/ba))/b" --format-sort res:720,tbr~2000 --continue --no-overwrites --no-post-overwrites --verbose --update --restrict-filenames --retry-sleep exp=2:64 --download-archive "downloaded.txt" --output "%(upload_date>%Y-%m-%d|)s%(upload_date& |)s%(title)s-%(id)s.%(ext)s" "$1" >> "/tmp/$LOG_NAME" 2>&1 &
}

I'm getting the process exit with status 1 meaning they have failed. But they have actually managed to download the video without problem. But they haven't been able to get the program version form the github API so they've given an error.
I don't want to see exit 1 when a process finishes unless it has actually failed to download the video.
I don't want to remove --update from the configuration since it is required to open an issue.
What can I do to fix this?
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

Nothing. An error code will be returned when there's any error - whether it be for update, download or anything else. That is exactly how error codes are supposed to work
I could change the updater error code from 1 to some other non-zero value. Would that help you?

I don't want to remove --update from the configuration since it is required to open an issue.

You can open issue without it, just make sure your it is actually up to date.
PS: if you cannot re-run the command with -vU and reproduce the problem, it's likely not a yt-dlp issue and you probably shouldn't be reporting it 🤷

I could change the updater error code from 1 to some other non-zero value. Would that help you
That would help.

I could change the updater error code from 1 to some other non-zero value. Would that help you?

Then what would be returned if both the updater and the download fail?

A common practice is to bitwise or them together. But yt-dlp does not do that for the other error codes.  So I'll just give priority to download error for now


Fix wrong search dir for libradiumCore

Actually FindRadium is also wrong on Windows for the libraries location. I will push my changes to this branch.

I've added to the PR a new behavior for the file Version.cpp.
It is now generated once to avoid systematic call to the linker each time cmake is called.
Since the file is produced by cmake, it is generated in the build dir, not in the source dir.
Consequently, removing the build dir will lead to the re-generation of this file.
Note that there is a dependency between this file and libRadiumCore, which allows to recompile the core library each time the file is modified.
Minor thing, the current git changeset is now added in Version.cpp and printed when starting Radium main app.

Regarding variables names, I think that RADIUM_INCLUDE_DIR should be renamed into RADIUM_INCLUDE_DIRS since there are multiple paths and they don't just apply to Radium but also to the 3rdPartyLibs.
Also, RADIUM_ROOT_DIR, RADIUM_INCLUDE_DIR and RADIUM_PLUGIN_OUTPUT_PATH are cited in the top-of-file comment, while other set variables are not (RADIUM_BUNDLE_DIRECTORY, EIGEN3_INCLUDE_DIR, ...). I think we should either unset them or "declare" them in the top-of-file comment.

I agree with @hoshiryu. When several directories are set in one variable, cmake guidelines recommend to name the variable XXX_INCLUDE_DIRS or XXX_LIBRARIES ...
I've modified this.
I also removed out some debug messages in the CMakeFiles and removed unnecesarry duplicate
For the last version of @nmellado, it is important that the file src/Core/Utils/Version.cpp is removed before compiling. When I checkout this PR, this file remained in my source directories and linker generates duplicate symbol errors.
I've commit some minor fixes and approved this PR.

For the last version of @nmellado, it is important that the file src/Core/Utils/Version.cpp is removed before compiling. When I checkout this PR, this file remained in my source directories and linker generates duplicate symbol errors.

Yes indeed.
Merging then.


Fix variable value changes not invalidating cache issue and etc.

Fix https://github.com/ansible-community/ansible-bender/issues/104. The idea here is doing the templating (variable substitution) for the result of task.get_ds() to get the correct fingerprint of the task. To do the templating we need some runtime information, which we are not able to obtain in Task object. The hacky way I use is using gc module to obtain the WorkerProcess obj that references Task obj.

Fix https://github.com/ansible-community/ansible-bender/issues/226. Let me explain why this issue happens: when a task failed, it is very likely (always when I tried it) is_failed here still returns False. It's a little bit complicated to explain why this happens behind the scene. Briefly speaking, the send_callback function does a clean_copy for task result that is gonna be sent to the hooks. And the clean_copy will wipe off the failed key in task result. The solution is to use status specific hooks instead of v2_on_any.

I tested my changes locally and two tests failed: integration/test_api.py::test_caching and integration/test_api.py::test_caching_mechanism but I think they failed for some reasons.
For test_caching, I am not sure I understand why it asserts that the task should not be cached (https://github.com/ansible-community/ansible-bender/blob/master/tests/integration/test_api.py#L43)
assert not b2.layers[4].cached

The corresponding task is a file action, but the source file does not change in between two builds. It should be cached.
- name: create a file
    copy:
      src: '{{ playbook_dir }}/a_bag_of_fun'
      dest: /fun

And this test passes before I made the changes due to a bug. Briefly speaking, in https://github.com/ansible-community/ansible-bender/blob/master/ansible_bender/callback_plugins/snapshoter.py, the task content is calculated twice (get_task_content called twice), before the task execution and after the task execution. For file-related task the result of two calculation will differ, causing wrong caching.
For test_caching_mechanism, it asserts this task will be cached
debug:
    msg: "{{ lookup('env','ANSIBLE_CONFIG', 'AB_BUILDER_NAME', 'AB_TARGET_IMAGE_NAME') }}"

If we do go down the variable substitution path, it makes more sense that this task does not get cached, because the ANSIBLE_CONFIG variable changes every run.
So we'll need to discuss about how we modify the tests, if I am not getting it wrong.
This is the first time I contribute to an open source project. Let me know if I am missing anything. Thanks!

It seems the test cases are wrong actually after reading your explanation. Please update them to what you feel like it's the right thing to do :)

Also sorry for replying so late, I was super-busy past few weeks, will try to provide replies in a timely manner so we can merge this next week.


Matches not being added to database

Describe the bug
After result of match, program will attempt to store entry in database, but will fail to do so, instead returning the following error: table SBMATCHES has no column named betOutcome. In addition, Matches following the failed attempt are also not stored/recorded in the database, but no additional errors are returned.
To Reproduce
Start program for first time (delete existing SaltDatabase.db if present). Wait for first valid match type (matchmaking or tourney). Upon result of match, program will attempt to store entry in database, then error will occur.
Expected behavior
Match expected to be stored in database after result obtained.
Log:
Player 1 chance to win: 50.0%
Current Balance is: $994,072
Bet placed: $1 on player2
Current Tier is: 2
Currently in Matchmaking with 100 matches remaining.  Game state is locked.
True Winstreaks are: (-1, 8)
Currently in Matchmaking with 99 matches remaining.  Game state is 2.
Player 2 wins!

Thank you for so much detail in the error, as well as reproduction and an attempted fix.
I've looked into this, and the error was originally caused by a bad rename on my behalf.  When this program runs for the first time, it creates a database if it doesn't find one already.
I accidentally renamed one of the values in the creation of the database to "bet_outcome" when renaming other things.  Whoops.
I'm renaming it back to "betOutcome", and will be pushing an update as soon as this comment is closed.
Thank you again for your research,
Friar


HIVE-28040 : Upgrade netty to 4.1.100 due to CVE-2023-44487

What changes were proposed in this pull request?
This PR upgrades Netty version to 4.1.100.Final.
Why are the changes needed?
The changes are required because Netty versions below 4.1.100 are impacted due to CVE-2023-44487. Hadoop and Tez are also currently at netty version 4.1.100.Final.
Does this PR introduce any user-facing change?
No
Is the change a dependency upgrade?
Yes
How was this patch tested?
The automated tests are successful.
OSS JIRA : HIVE-28040

The test failure seem flaky/unrelated and are passing locally.

Thank you for letting me know @aturoczy. I have filled out the details.

+1 (non-binding)


AsyncCall in RealCall sets the thread name to the url, exposing sensitive data

The AsyncCall runnable inside the RealCall class extends NamedRunnable, and sets the current thread name to the full url of the call.
This can expose sensitive information, for example when API's require a token in the url. I'm having the problem that I log the thread name with log4j2 and it now logs the full url.
I suggest changing it to toLoggableString().
Thanks

I thought we had all of these but we must have missed one. Can fix.

Closed with #2499.


Pigweed venv management doesn't respect requirements.txt

Problem
Cloning the latest CHIP repo and using with ESP32 leads to pyparsing version problems while compiling. This seems to be an actual error with the Pigweed env and is captured here: https://bugs.chromium.org/p/pigweed/issues/detail?id=390
Steps to recreate the issue:
$ cd /path/to/esp-idf
$ ./install.sh
$ . ./export.sh
$ cd /path/to/connectedhomeip  [newly cloned]
$ source scripts/bootstrap.sh
$ source scripts/activate.sh
$ cd examples/all-clusters-app/esp32
$ idf.py build

Output of idf.py build command:
The following Python requirements are not satisfied:
pyparsing>=2.0.3,<2.4.0
To install the missing packages, please run "/home/hrishi/open_source/esp-idf/install.sh"
Diagnostic information:
      <details>

Proposed Solution
Ideally pigweed env should honor the requirements.txt and the build should happen successfully.

The workaround for the above problem as of now is:
After getting error in idf.py build:
$ cd /path/to/esp-idf
$ pip install -r $IDF_PATH/requirements.txt
$ cd /path/to/connectedhomeip/examples/all-clusters-app/esp32
$ idf.py build

This might result in some issues with examples using Pigweed (not sure)

This has been fixed sometime back. For users who were/are encountering "Requirements not satisfied" issue with ESP32 and Mac/Darwin (the above workaround helps to solve that as well) can pull the latest CHIP code and try, it has been fixed recently https://github.com/project-chip/connectedhomeip/pull/10763


Refactoring

Coverage increased (+4.05%) to 92.233% when pulling 9a72908f625504e6d3ec3e7b3819fa342739d2c8 on andreas-schroeder:Refactoring into 7ac430d89f398eaa20ff84d06666de98ac33408f on AutoScout24:master.

Coverage increased (+4.05%) to 92.233% when pulling 2a8708ba0df38198a671705ba8fecfbb965056cf on andreas-schroeder:Refactoring into 7ac430d89f398eaa20ff84d06666de98ac33408f on AutoScout24:master.

Coverage increased (+4.05%) to 92.233% when pulling ac93293bbf4ec1dae95dca418866efb0999cb398 on andreas-schroeder:Refactoring into 7ac430d89f398eaa20ff84d06666de98ac33408f on AutoScout24:master.


Main thread hangs when interrupted, exit is required

proof https://github.com/mkn/mkn.kul/blob/master/os/nixish/inc/kul/src/signal/handler.cpp


Update branch code base

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 16.66667% with 5 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

src/main/java/seedu/address/ui/PersonCard.java
0.00%
5 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines
Coverage Δ
Complexity Δ

...c/main/java/seedu/address/model/person/Gender.java
93.33% <100.00%> (+0.47%)
8.00 <2.00> (+2.00)

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)


Confirmation please - mapping yml case sensitive?

I have a mapping where origin file has capitalised field name 'LATITUDE', destination 'Latitude'.
Is the mapping file case-sensitive?

case-sensitive testing (and testing for field/filenames in mapping match input files) on one-way transformation A-B:

when field names in mapping are lower case, as in input file, test runs successfully.
when field names in mapping are upper case, not as in input file, test fails:: Column name in mapping file must match column name in input file exactly.

In runtest.zip:
A2.csv - this was the file I adjusted to test above. When I restored values to lower case a, it errors - cannot convert string 'a' to float.
A.csv - I recreated a csv (UTF-8, as default in MS Excel) file from scratch in Excel, intended to match A2.csv. When I run the tool, error is "could not convert string to float: '\ufeffa'". Seems this is an invisible character created when saving the excel sheet to csv. Causes problems for the transformation. See files and logs attached.
runtest.zip
@OmegaDroid these seem like they could trip users up easily - please can we capture the error in tool, and can you advise how to resolve?
Still to test:

when input_format in mapping is lower case, not matching input file, test ....
when a command, e.g,. 'transformation:' is not all lower case, test ....

When an argument, e.g,. 'transformation:' is not all lower case, test fails with error 'got an unexpected keyword argument', e.g., got an unexpected keyword argument 'Transformation' (though does not return line number of mapping file)
When 'transform:' / 'forward:' / 'reverse:' are not spelt correctly (e.g., traNsform: instead of transform:), the test fails and the error is "No conversion path from A to B.". Better messaging to identify the issue would help users.
However, when 'types:' is not spelt correctly (e.g., type: or tpEs:), the test succeeds, no error is given.
When input_format in mapping is lower case, or otherwise not matching input file name, test fails and the error given is "No conversion path from A to B.".  Could additional information be provided to
@OmegaDroid Please can you consider these in revisions to error reporting, thank you


New Tab does not Display the Background Image

Windows Terminal version (or Windows build number)
Windows Build: 10.0.19042.906, Windows Terminal commit b310b1c built using Visual Studio 2019 16.9.3
Other Software
oh-my-posh 3.133.1
posh-git 1.0.0 (Latest)
PSReadLine 2.1.0
Ubuntu WSL 20.04
Steps to reproduce
Build Windows Terminal using visual studio. Release, x64, CascadiaPackage

Configure a profile to use a background.

Exit Terminal

Run Terminal and the background will be solid gray and not the image selected in step 1

Open Settings

Save Settings

The open tab background will show the background specified in settings

Start a new tab, and the background will be solid gray

The kind of shell doesn't seem to affect this. I've tried Powershell 7.1.3, Ubuntu 20.04, git bash using MINGW64.
Expected Behavior
Opening a new Windows Terminal tab shows the background image specified in the profile.
Actual Behavior
Opening a new Windows Terminal tab does not show the background image specified in the profile.
Included Settings.json
settings.zip

Merging dev/pabhoj/defaults_fix fixed this issue.


How to enable paging

Your demo site shows paging but I can not get it to work with the code from the repository. Is that something I need to enable?

Paging is spicify in the Request object:
SearchRequest searchRequest = new SearchRequest
            {

                Keywords = keyword,
                ItemPage = **page**,
                BrowseNodeId = browseNoseId,
                Resources = new[]
                                {
                                    "Images.Primary.Large",
                                    "ItemInfo.Title",
                                    "ItemInfo.Features",
                                    "ItemInfo.TradeInInfo",
                                    "ItemInfo.ByLineInfo",
                                    "ItemInfo.ManufactureInfo",
                                }
            };

So, for every request, you should set the page as you can see above.

https://github.com/nager/Nager.AmazonProductAdvertising/blob/9fdb77392138e8bf4b26f6f3813f6af453545d13/src/Nager.AmazonProductAdvertising/Model/SearchRequest.cs#L8


replace kinds with collections

Codecov Report

Merging #461 into master will increase coverage by 0.01%.
The diff coverage is 94.36%.

@@            Coverage Diff             @@
##           master     #461      +/-   ##
==========================================
+ Coverage   74.58%   74.59%   +0.01%
==========================================
  Files          54       54
  Lines        3946     3948       +2
==========================================
+ Hits         2943     2945       +2
  Misses       1003     1003

Impacted Files
Coverage Δ

flax/core/lift.py
75.15% <92.85%> (ø)

flax/core/scope.py
84.05% <94.54%> (+0.15%)
:arrow_up:

flax/core/__init__.py
100.00% <100.00%> (ø)

flax/linen/module.py
94.53% <100.00%> (ø)

Continue to review full report at Codecov.

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


added FAQ page to the website

Fix : (https://github.com/roc-lang/roc/issues/6260)

I have made the required changes, please check and inform me if there's any correction to be done

Done, please check and let me know, thanks

Can you merge this @rtfeldman?

Can you set up commit signing for future contributions @dotslashbit?

Ok I'll do that

There is currently a GitHub wiki entry for FAQ, but it returns 404. Either removing it from the GitHub wiki or adding a link to the website FAQ would be an improvement.

Thanks for alerting us @hoosierEE, I fixed it :)


Request mapper can have update entity

do you think someone would ever want an async variant of this new method as well?

Fair question, I don't know.  I saw that ToEntity and FromEntity have async versions, but wasn't sure when I would use them, so it didn't occur to me to add it.  I was just copying the implementation from the Mapper class, which does not having an async of UpdateEntity.
I suppose it wouldn't hurt to add to both Mapper and RequestMapper, to have parity with the To and From functions.

yeah, go ahead and add it.
will merge then.


Add dependencies on prometheus-operator-crd for quicker deployment

Introduce a dependency for apps that use a ServiceMonitor CR which is installed by prometheus-operator-crd.
Found by chance while checking in https://github.com/giantswarm/default-apps-azure/pull/202 why E2E tests failed. The dependency fix could improve deployment speed and therefore lead to fewer test errors, but that's only a guess. This also makes the dependencies more consistent with default-apps-aws.
Trigger e2e tests
/run cluster-test-suites

/run cluster-test-suites

/run cluster-test-suites

/run cluster-test-suites
come on

/run cluster-test-suites

/run cluster-test-suites

/run cluster-test-suites


can you add motherboard control functions?

can you add motherboard control functions?

I second that

@Vipeax

I "third" that :)

It's not easy because it requires creating a signed driver to access to SuperIO chip. Plus it's very difficult because each SIO chip has different way to be controlled. Check repo from nagasia/msi-led to read more about this. He reversed his own and made a tool for linux

Oh, i see. That tool looks good but sadly i'm still on windows.


Fix file listing for CP 9

Fixes https://github.com/adafruit/circuitpython/issues/8875. This fixes the connection issue.

Fair enough. I wanted to just get it fixed so it works in both CP 8 and 9 for the moment. I would like to make better use of the additional info in a future PR and I can have it check the version number at that point.


Sets norelativenumber

Category

[ ] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[ ] Completion
[x] Vim
[x] Neovim
[ ] Etc.

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[ ] bash
[ ] zsh
[ ] fish

Using fzf inside vim sets norelativenumber, globally.  I cannot figure out where this is happening, but :verbose set relativenumber? says it is set by this plugin, in fzf/plugin/fzf.vim.  Sure enough, the relativenumbersetting in my vim config is respected until I run fzf, after which point it is disabled.

UPDATE:  Only neovim has the issue.  Vim is unaffected.

I can't seem to reproduce. Can you reproduce the issue with other plugins disabled?
https://gist.githubusercontent.com/junegunn/6936bf79fedd3a079aeb1dd2f3c81ef5/raw

This happens only in terminal buffers. Your global setting isn't changed. If you want relativnumber in terminal buffers :
au TermOpen * set relativenumber

@justinmk You're right, it happens with a function I made to open Ranger in a terminal buffer as well.  Misdiagnosed this is as something wrong with fzf.  Thanks.
Also do you know why neovim does this?  It seems like a bug to me, and if you agree I'll report it.

@rosshadden It's intentional. In terminal-mode 'realtivenumber' is meaningless https://github.com/neovim/neovim/pull/6185/commits/857113ca8c76cd103b752ad4460ee1b091e0831a

@justinmk Sure, but I don't see why neovim doesn't restore the previous setting when you exit terminal mode.  Unless opening a file from within terminal mode (through shell, fzf, ranger, etc.) stays in it, I suppose.

It's just a local default. However, thinking on it some more, I don't think we need this default anymore, since terminal-buffer cursor position was fixed in 0.2 (in 0.1.7 and earlier, the normal-mode cursor always shifted to the bottom).
With https://github.com/neovim/neovim/pull/6796 terminal will no longer modify 'number' and 'relativenumber'.


Support nested modules

This one refers to #11. It turned out that support for nested modules was just provided. Elixir nested modules are separate beam files with the name Elixir.ParentModule.NestedModule, so they can be handled as normal modules. The tokens lookup is restricted to the location of the current specified function, so it is not possible to mistake tokens between modules.
This PR adds a test to ensure that everything works as it should. By the way, I move the logic that loads tokens to the file utils module and reused it in tests. Now it is possible to specify abstract forms even when tokens cannot be loaded.

Looks good :+1:


CI job on iron: updated branch and distribution names

Codecov Report

Merging #371 (5076a2e) into iron (96abb74) will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             iron     #371      +/-   ##
==========================================
- Coverage   69.56%   69.49%   -0.08%
==========================================
  Files          16       16
  Lines        2760     2760
  Branches      766      766
==========================================
- Hits         1920     1918       -2
- Misses        453      454       +1
- Partials      387      388       +1

see 1 file with indirect coverage changes


Introduce lazyLoading.includeRoutesInApplication option

Introduces a new lazyLoading.includeRoutesInApplication option which enables lazy engines to output their routes.js file to a stand-alone file rather than bundling it into the hosts vendor.js file.
This will enable engines to be discovered dynamically at runtime rather than requiring engines to be declared statically at build time.
Still needs some tests which I will do once others have reviewed.
Usage:
// Prevent routes being bundled into hosts vendor.js
lazyLoading: {
  enabled: true,
  includeRoutesInApplication: false
}

TODO:

[x] Tests

cc/ @rwjblue @trentmwillis

Thanks for the review @trentmwillis, next up is some tests, then this should be good to go.

Just pushed a test for this, I feel like the test could be simplified a bit but it serves the purpose for now.

Is there anything else needed to get this merged?


Size restriction

Hello there, my Question is if there is any way to upload a Image over the size of 4mb

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

ID: 867c13a4-259b-5686-3208-9c335cc504c1
Version Independent ID: dad41b97-f678-7807-0837-df9b426e9009
Content: Printed, handwritten text recognition - Computer Vision - Azure Cognitive Services
Content Source: articles/cognitive-services/Computer-vision/concept-recognizing-text.md
Service: cognitive-services
Sub-service: computer-vision
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

@KerimY Thanks for the question. We are investigating the issue and will update you shortly.

@ram-msft and?

@PatrickFarley Can you please help on this.

@ram-msft anything yet?

@KerimY
No, that's a limitation of the service.
#please-close


error with SilverStripe\Admin\LeftAndMain::getRecord()

Not sure what to make of this or if this is even an error.

[Emergency] Uncaught ArgumentCountError: Too few arguments to function SilverStripe\Admin\LeftAndMain::getRecord(), 0 passed in /var/www/ss4/mysite/vendor/lekoala/silverstripe-cms-actions/src/ActionsGridFieldItemRequest.php on line 145 and exactly 1 expected

i've merged your PR, does it fix your issue?

yes, it worked, but I am not sure if passing zero is the right thing to do.

i will have a deeper look at it. i'll probably add a bit of reflection to make sure it gets an argument or not

i've improved things a little bit here
https://github.com/lekoala/silverstripe-cms-actions/commit/7ea6559d52efbe739481cbdf49c885711b621dd4
should work the same for you, but avoiding passing a parameter when not needed


Hugepages test vm

Launch the test VM off a hugepages-enabled flavor, if requested so.
Also configure edpm node templates to use the matching configuration.
Depends-On: https://github.com/openstack-k8s-operators/install_yamls/pull/938
Jira: #OSPRH-10611

recheck

recheck

recheck

Created a test project to test adoption with hugepages https://review.rdoproject.org/r/c/testproject/+/55217

recheck

recheck

test VM failed to start, and there is no journal nor sos reports from standalone compute. So I resumed testproject https://review.rdoproject.org/r/c/testproject/+/55217

recheck

recheck

recheck stuck in zuul

recheck

recheck

recheck fixed config

a test VM was started OK, but next tasks failing:
TASK [tls_adoption : Create Certificate Issuer with cert and key from IPA] *****
...
"rc": 255, "start": "2024-11-21 10:21:44.085750",
"stderr": "+ IPA_SSH='ssh -i /home/zuul/.ssh/id_rsa<EMAIL_ADDRESS>podman exec -ti freeipa-server-container'
+ ssh -i /home/zuul/.ssh/id_rsa<EMAIL_ADDRESS>podman exec -ti freeipa-server-container pk12util -o /tmp/freeipa.p12 -n 'caSigningCert\\ cert-pki-ca' -d /etc/pki/pki-tomcat/alias -k /etc/pki/pki-tomcat/alias/pwdfile.txt -w /etc/pki/pki-tomcat/alias/pwdfile.txt
Error: can only create exec sessions on running containers: container state improper",
"stderr_lines": ["+ IPA_SSH='ssh -i /home/zuul/.ssh/id_rsa<EMAIL_ADDRESS>podman exec -ti freeipa-server-container'",
"+ ssh -i /home/zuul/.ssh/id_rsa<EMAIL_ADDRESS>podman exec -ti freeipa-server-container pk12util -o /tmp/freeipa.p12 -n 'caSigningCert\\ cert-pki-ca' -d /etc/pki/pki-tomcat/alias -k /etc/pki/pki-tomcat/alias/pwdfile.txt -w /etc/pki/pki-tomcat/alias/pwdfile.txt",
"Error: can only create exec sessions on running containers: container state improper"], }

recheck Indian Pale Ale

recheck IPA , NODE_FAILURE

the ipa command still fails as earlier

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

we need a testproject to test this in CI - 	the flag enablement https://review.rdoproject.org/r/c/rdo-jobs/+/55179/17/zuul.d/_data_plane_adoption.yaml#76
but it's not in the resulted vars.yaml https://logserver.rdoproject.org/74/674/84270c224bdb78d061a166606f6e1f5f4b376c7e/github-check/adoption-standalone-to-crc-no-ceph/c8d868f/controller/data-plane-adoption-tests-repo/data_plane_adoption/vars.yaml

recheck

recheck

recheck

recheck


fix a few casts

This PR

removes to(*args) fn from FusedExpertsNetwork

The layer imlp does not have self.fc1_weight, self.fc2_weight, self.fc1_bias, or self.fc2_bias.
This to(*args) fn is a bug.
At one point, this would have been correct, but it is no longer correct.

autocasts x in expert fn

p = list[experts.parameters()][0] will have dtype fp32 if using generic torch autocast; adding autocast to x if autocast enabled.

better cast x in gate

Nice fixes, Thanks so much!


cockroachdb: add benchmarks

This change adds a benchmark that spins up a local CRDB cluster and runs the kv workload against it.

This is looking really nice! I'll take another pass by EOW.


feat(link): adds default link style

Adds default link style

title: "type_of_change(scope_of_work): ticket title"
reviewers: "sds-eng, sds-design"
Summary
Shortcut ticket: sh-154877
Update Default Link Component
Checklist

[ ] Default Story in Storybook
[ ] LivePreview Story in Storybook
[ ] Test Story in Storybook
[ ] Tests written
[ ] Variables from defaultTheme.ts used wherever possible
[ ] If updating an existing component, depreciate flag has been used where necessary
[ ] Chromatic build verified by @sds-design

Woohoo! Thanks so much, Connor! Have an amazing weekend 🤩🎈


Inference problem: typeerror: sat.model.transformer.basetransformer () got multiple values for keyword argument' parallel _ output'

Hello, I encountered the following error when trying to load the model for reasoning: typeerror: sat.model.transformer.basetransformer () got multiple values for keyword argument' parallel _ output'.
I used the command line to load Python CLI_demo.py-from_retrained/root/autodl-tmp/transgpt-mm-v1/1 -prompt_zh. 图中的标志表示什么含义？
The following is my file structure:
1
|-- gitattributes
|-- latest
|-- model_config.json
|-- mp_rank_00_model_states.pt
`-- mp_rank_00_model_states.pt.lock
I modified "THUDM/chatglm-6b": tokenizer = auto tokenizer. from _ retrained (args. from_retrained, trust_remote_code = true).
Could you please provide more information? Thx !
The specific error information is:
[2024-07-25 23:05:06,615] [INFO] building FineTuneVisualGLMModel model  ...
[2024-07-25 23:05:06,619] [INFO] [RANK 0] > initializing model parallel with size 1
[2024-07-25 23:05:06,620] [INFO] [RANK 0] You didn't pass in LOCAL_WORLD_SIZE environment variable. We use the guessed LOCAL_WORLD_SIZE=1. If this is wrong, please pass the LOCAL_WORLD_SIZE manually.
[2024-07-25 23:05:06,620] [INFO] [RANK 0] You are using model-only mode.
For torch.distributed users or loading model parallel models, set environment variables RANK, WORLD_SIZE and LOCAL_RANK.
[rank0]: Traceback (most recent call last):
[rank0]:   File "/root/autodl-tmp/TransGPT-main/multi_modal/cli_demo.py", line 105, in
[rank0]:     main()
[rank0]:   File "/root/autodl-tmp/TransGPT-main/multi_modal/cli_demo.py", line 30, in main
[rank0]:     model, model_args = AutoModel.from_pretrained(
[rank0]:   File "/root/miniconda3/envs/transgpt/lib/python3.10/site-packages/sat/model/base_model.py", line 342, in from_pretrained
[rank0]:     return cls.from_pretrained_base(name, args=args, home_path=home_path, url=url, prefix=prefix, build_only=build_only, overwrite_args=overwrite_args, **kwargs)
[rank0]:   File "/root/miniconda3/envs/transgpt/lib/python3.10/site-packages/sat/model/base_model.py", line 334, in from_pretrained_base
[rank0]:     model = get_model(args, model_cls, **kwargs)
[rank0]:   File "/root/miniconda3/envs/transgpt/lib/python3.10/site-packages/sat/model/base_model.py", line 420, in get_model
[rank0]:     model = model_cls(args, params_dtype=params_dtype, **kwargs)
[rank0]:   File "/root/autodl-tmp/TransGPT-main/multi_modal/finetune_visualglm.py", line 14, in init
[rank0]:     super().init(args, transformer=transformer, **kw_args)
[rank0]:   File "/root/autodl-tmp/TransGPT-main/multi_modal/model/visualglm.py", line 34, in init
[rank0]:     self.add_mixin("eva", ImageMixin(args))
[rank0]:   File "/root/autodl-tmp/TransGPT-main/multi_modal/model/visualglm.py", line 18, in init
[rank0]:     self.model = BLIP2(args.eva_args, args.qformer_args)
[rank0]:   File "/root/autodl-tmp/TransGPT-main/multi_modal/model/blip2.py", line 56, in init
[rank0]:     self.vit = EVAViT(EVAViT.get_args(**eva_args))
[rank0]:   File "/root/autodl-tmp/TransGPT-main/multi_modal/model/blip2.py", line 21, in init
[rank0]:     super().init(args, transformer=transformer, parallel_output=parallel_output, **kwargs)
[rank0]:   File "/root/miniconda3/envs/transgpt/lib/python3.10/site-packages/sat/model/official/vit_model.py", line 111, in init
[rank0]:     super().init(args, transformer=transformer, **kwargs)
[rank0]:   File "/root/miniconda3/envs/transgpt/lib/python3.10/site-packages/sat/model/base_model.py", line 93, in init
[rank0]:     self.transformer = BaseTransformer(
[rank0]: TypeError: sat.model.transformer.BaseTransformer() got multiple values for keyword argument 'parallel_output'
[rank0]:[W725 23:05:10.281982969 ProcessGroupNCCL.cpp:1168]  Warning: WARNING: process group has NOT been destroyed before we destruct ProcessGroupNCCL. On normal program exit,  the application should call destroy_process_group to ensure that any pending NCCL operations have finished in this process. In rare cases this process  can exit before this point and block the progress of another member of the process group. This constraint has always been present,   but this warning has only been added since PyTorch 2.4 (function operator())

请问解决了吗？我也遇到了这错误，但是是最新的SwissArmyTransformer版本了


Can't run sample code

I ran
go get -u github.com/arriqaaq/skiplist
Copied the sample code from the readme and get errors when I try to go run the file
# command-line-arguments
./t.go:18:12: undefined: list
./t.go:19:2: undefined: assert
./t.go:19:15: undefined: t

This is the code I ran.. which is identical to readme
package main
import "github.com/arriqaaq/skiplist"
type kv struct{k,v string}
func main() {
// Set (accepts any value)
n := skiplist.New()
val := []byte("test_val")

n.Set("ec", val)
n.Set("dc", 123)
n.Set("ac", val)

// Get
node:= list.Get("ec")
assert.Equal(t, val, node.Value())

// Delete
n.Delete("dc")

}

There is a small typo, but apart from that you'd just have to import "github.com/stretchr/testify/assert"
Alternatively you can remove that assert line to avoid the import

updated

Trying latest code from readme
go get -u github.com/arriqaaq/skiplist
go get github.com/stretchr/testify/assert
go run sl.go
./sl.go:22:18: undefined: t

Also, this part doesn't seem like it is neede either
type kv struct{k,v string}

Yeah, avoiding the unnecessary assert, fixed it, thanks for the issue
package main

import (
	"fmt"

	"github.com/arriqaaq/skiplist"
)

func main() {

	// Set (accepts any value)
	val := "test_val"

	n := skiplist.New()
	n.Set("ec", val)
	n.Set("dc", 123)
	n.Set("ac", val)

	// Get
	node := n.Get("ec")
	fmt.Println("value: ", node.Value())

	// Delete
	n.Delete("dc")
}

Tested it

Latest code works. Thanks.


Enable log data removal during POSIX docker compose down

#8

Closing this pull request as there is no straightforward solution to create, share and remove volume during docker compose up and down.


Adding animations to the carousel qoutes

Adding animations to the carousal quotes will make it more eye-catching and enhance the user experience

hello , I am a participant of GSSOC'22 , can you assign me this project to work .

hello , I am a participant of GSSOC'22 , can you assign me this project to work .

@rashi1405 , this issue has already been resolved.


Error in ParseTree project under Visual Studio 2019

When I run ParseTree project I get error in DrawTree method of LithiumControl.cs in the following row:
p = new Point(graphAbstract.Root.X, graphAbstract.Root.Y);
because graphAbstract.Root == null. I get the following error message:
< System.NullReferenceException: 'Object reference not set to an instance of an object.'
Netron.Lithium.GraphAbstract.Root.get returned null.>
It is not happened when I run the project under Visual Studio 2015

I dig a bit of digging; there was an issue in the initialization of the ParseTree (see fix
I don't understand why we did not have this exception in the previous versions of Visual Studio.


Add description field to pipeline

TID-640

pipeline description
File: swaggerCalls.js
describe : "Store Pipelines"

Test
description

test the POST /store/pipelines
store pipeline with description


How to connect and get working EOS Community telegram group ?

Hi,
Would anyone who knows and uses telegram communicating with EOS Community explain how to setup EOS Community telegram on Ubuntu ?
I have tried before, but please see - no progress, can't connect to the EOS community groups on Telegram.
https://app.box.com/s/ljjbv8lw0iee8pmrjx12hzpkyswo8qi5
Please, this is very important, which version of telegram to install, how to find and connect to the EOS Community group(developers)?
Please explain.
Kind Regards.

Install the latest version of telegram (really doesn't matter) and connect on your smartphone - then you can link a telegram account via a browser (firefox / chrome work fine on ubuntu) at web.telegram.com - The developers chat is here - https://t.me/joinchat/EgOVjkPktgfUS3kt14FStw
This isn't really a question for the EOS GitHub
Telegram has a bunch of apis if that's your thing - https://core.telegram.org/


close page error

Description
import { connect } from 'puppeteer-real-browser'
const { browser, page } = await connect({
headless: false,
args: [],
customConfig: {},
turnstile: true,
connectOption: {},
disableXvfb: false,
ignoreAllFlags: false
})
let testPage = await browser.newPage()
testPage.on('response', async (response) => {
if (response.url().includes(?type=dealer&type=spclient)) {
testPage.close()
}
})
await testPage.goto(https://open.spotify.com/search/lady gaga)
Full steps to reproduce the issue
file:///Users/fengyu/Desktop/AI/express/node_modules/rebrowser-puppeteer-core/lib/esm/puppeteer/cdp/LifecycleWatcher.js:100
this.#terminationDeferred.resolve(new Error('Navigating frame was detached'));
^
Error: Navigating frame was detached
at #onFrameDetached (file:///Users/fengyu/Desktop/AI/express/node_modules/rebrowser-puppeteer-core/lib/esm/puppeteer/cdp/LifecycleWatcher.js:100:47)
at file:///Users/fengyu/Desktop/AI/express/node_modules/rebrowser-puppeteer-core/lib/esm/third_party/mitt/mitt.js:36:7
at Array.map ()
at Object.emit (file:///Users/fengyu/Desktop/AI/express/node_modules/rebrowser-puppeteer-core/lib/esm/third_party/mitt/mitt.js:35:20)
at CdpFrame.emit (file:///Users/fengyu/Desktop/AI/express/node_modules/rebrowser-puppeteer-core/lib/esm/puppeteer/common/EventEmitter.js:77:23)
at #removeFramesRecursively (file:///Users/fengyu/Desktop/AI/express/node_modules/rebrowser-puppeteer-core/lib/esm/puppeteer/cdp/FrameManager.js:514:15)
at #onClientDisconnect (file:///Users/fengyu/Desktop/AI/express/node_modules/rebrowser-puppeteer-core/lib/esm/puppeteer/cdp/FrameManager.js:91:42)
Node.js v20.17.0
[nodemon] app crashed - waiting for file changes before starting...
Issue Type
No response
Operating System
No response
Do you use Docker?
None

it seems like its run testPage.close() multiple times even after the page is already closed

it seems like its run testPage.close() multiple times even after the page is already closed

Solved, thank you


chore: fix close for ConnectionManager

fix close for ConnectionManager

I think this is an obvious bug, no need to add a specific test.

I think this is an obvious bug, no need to add a specific test.

I think this is an obvious bug, no need to add a specific test.

It is a bug, and a test will guarantee that if this code is changed it will not be created again. A simple test should be added here where a peer connects and than immediately after stopping, it should have no connections

I think this is an obvious bug, no need to add a specific test.

It is a bug, and a test will guarantee that if this code is changed it will not be created again. A simple test should be added here where a peer connects and than immediately after stopping, it should have no connections

The PR with master has been rebased, and tests have been added for ConnetionManager.

The PR with master has been rebased, and tests have been added for ConnetionManager.


Help wanted.. kernel 32.dll missing..

How can it solve..

The wine version you selected is not suitable. The wine staging version is only suitable for Kali arm64, not Debian Debian should choose wine devel or wine stable. You can try the release of box86 and box64 wine on arm64 Kali on August 12, 2022.

The wine version you selected is not suitable. The wine staging version is only suitable for Kali arm64, not Debian Debian should choose wine devel or wine stable. You can try the release of box86 and box64 wine on arm64 Kali on August 12, 2022.

Okay will try , thank you.


proposing a new release process

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Dear @falcosecurity/libs-maintainers
This PR proposes a new structured but still streamlined release process which uses release branches and reduces the code freeze period as much as possible.
This idea was initially proposed by @jasondellaluce , discussed in various community calls, and experimented it during the Falco 0.32.2 release with the help of @Andreagit97 .
This PR is going further, proposing precise rules and phases.
Now, the proposal is to keep this PR as a wip and test drive the new process during the next libs release. In this way, we can directly incorporate any feedback from an authentic release experience and eventually merge this once the release is done.
:smiley:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

cc'ing other folks I know might be interested
@araujof @terylt @incertum :hugs:

Nice job, @leogr! 👍
I think that the new release process will simplify release management and testing, without freezing the main dev branch. Thanks for working on this!

Hey Folks, nice work @leogr.  My only comment is that if you want to avoid a code freeze, what you could do is have two main branches:  main and dev.   All new checkins and PRs go into the dev branch.   When you are getting ready to cut a release, the dev branch gets merged into main and the main is tested, which acts as a code "freeze".  Bug fixes etc can be done in main and a release tag created.   Dev continues to be open to allow PRs to be added with no "freeze".   main basically holds your releases and bug fixes to those releases.  This takes away the urgency to unfreeze a branch and gives time to test.  We've used this approach in the past and it works well.  Just a thought.
Great work!

Hey Folks, nice work @leogr. My only comment is that if you want to avoid a code freeze, what you could do is have two main branches: main and dev. All new checkins and PRs go into the dev branch. When you are getting ready to cut a release, the dev branch gets merged into main and the main is tested, which acts as a code "freeze". Bug fixes etc can be done in main and a release tag created. Dev continues to be open to allow PRs to be added with no "freeze". main basically holds your releases and bug fixes to those releases. This takes away the urgency to unfreeze a branch and gives time to test. We've used this approach in the past and it works well. Just a thought.
Great work!

Hey @terylt
Thank you for your feedback :hugs:
Using the dev branch or most complex conventions like git-flow are alternatives we already evaluated but discarded because they looked more complicated and potentially required more cherry-picking in case of later discovered problems.
For example, if a significant issue is later discovered in the main branch, we would have needed to cherry-pick a complex patch back to the dev branch (which may have diverged significatively, meanwhile).
In our idea, the small freeze period should avoid the main branch, and the release branch would diverge too much. That should drastically reduce the likelihood of conflicts during cherry-pick operations.

Note: cherry-pick operations are a bit troublesome since our policy always requires protected branches and linear git history.
This policy is implicit and inherited from K8s automation - Prow, that we call @poiana - which ensures that ownership is consistently enforced thru PR reviews. Basically, any cherry-pick will require a PR, and no merge commits are allowed. We can't just cherry-pick and push to a branch.

That being said, we are still experimenting, so any alternative strategy is still to be taken into consideration :smiley_cat:

Hey Folks, nice work @leogr. My only comment is that if you want to avoid a code freeze, what you could do is have two main branches: main and dev. All new checkins and PRs go into the dev branch. When you are getting ready to cut a release, the dev branch gets merged into main and the main is tested, which acts as a code "freeze". Bug fixes etc can be done in main and a release tag created. Dev continues to be open to allow PRs to be added with no "freeze". main basically holds your releases and bug fixes to those releases. This takes away the urgency to unfreeze a branch and gives time to test. We've used this approach in the past and it works well. Just a thought.
Great work!

Hey @terylt
Thank you for your feedback 🤗
Using the dev branch or most complex conventions like git-flow are alternatives we already evaluated but discarded because they looked more complicated and potentially required more cherry-picking in case of later discovered problems.
For example, if a significant issue is later discovered in the main branch, we would have needed to cherry-pick a complex patch back to the dev branch (which may have diverged significatively, meanwhile).
In our idea, the small freeze period should avoid the main branch, and the release branch would diverge too much. That should drastically reduce the likelihood of conflicts during cherry-pick operations.

Note: cherry-pick operations are a bit troublesome since our policy always requires protected branches and linear git history.
This policy is implicit and inherited from K8s automation - Prow, that we call @poiana - which ensures that ownership is consistently enforced thru PR reviews. Basically, any cherry-pick will require a PR, and no merge commits are allowed. We can't just cherry-pick and push to a branch.

That being said, we are still experimenting, so any alternative strategy is still to be taken into consideration 😺

I'll just make one last comment on the dev branch model.   One of the really nice things about having a dev branch is that when you make a massive change to the code base,  you can push that change to dev, and have release candidates in there, and "edge" cuts of the project.  This allows the community at large to do more testing on major feature changes before they ever get into master.   In the current model, those changes are either in PR or they are in master.
Anyway, nice work!

Quick update: applied some reviews suggestions from @incertum
Also, :thinking: I wonder if we should add such as warning :point_down:

Hey @falcosecurity/libs-maintainers
I believe it's time to bump this.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Closing and reopening to trigger the CI
/close

@leogr: Closed this PR.

In response to this:

Closing and reopening to trigger the CI
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/reopen

@leogr: Reopened this PR.

In response to this:

/reopen

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, leogr, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,LucaGuerra,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Better error handling

NetflixOSS » eureka » eureka-pull-requests #372 SUCCESS
This pull request looks good


Backlinks are broken if there is a dot in filename

To reproduce,

Rename any zettel note that has non-zero backlinks

Rename to a filename that has dot in it (eg: Some foo.bar.md)

Go to the renamed zettel
The backlinks panel will now be empty

Hi, I'm unable to reproduce.
When you rename via "Rename Symbol" command and have a dot in the filename (or second extension) remember to add .md explicitly, otherwise, Memo will infer extension directly from the link, which is for instance useful in case of images and pdfs. Memo supports various extensions besides .md in the links.
In the following screencast I tried both ways via file rename and via "Rename Symbol" command and in both cases Backlinks Panel worked as it should:

I renamed to a text like "Foo v1.0 release".
I'll try the .md trick, and create a screencast of the bug if it still
happens again.
On Sun., Sep. 20, 2020, 10:18 a.m. Svyat Sobol<EMAIL_ADDRESS>wrote:

Hi, I'm unable to reproduce.
When you rename via "Rename Symbol" command and have a dot in the filename
(or second extension) remember to add .md explicitly, otherwise, Memo
will infer extension directly from the link, which is for instance useful
in case of images and pdfs. Memo supports various extensions
https://github.com/svsool/vscode-memo/blob/master/src/utils/utils.ts#L14-L44
besides .md in the links.
In the following screencast I tried both ways via file rename and via
"Rename Symbol" command and in both cases Backlinks Panel worked as it
should:
[image: issue #120]
https://user-images.githubusercontent.com/1452141/93713297-8599ba80-fb63-11ea-8d4e-face5c0e77e5.gif
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/svsool/vscode-memo/issues/120#issuecomment-695792566,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAA7HR4T3UNFHGE2YO2SOLSGYFLPANCNFSM4RTN5L6Q
.

Looks like Node's path.parse function considers 0 release as an extension 😅:
❯ node
Welcome to Node.js v14.4.0.
Type ".help" for more information.
> path.parse('Foo v1.0 release')
{
  root: '',
  dir: '',
  base: 'Foo v1.0 release',
  ext: '.0 release',
  name: 'Foo v1'
}

But anyway, please ensure that Foo v1.0 release has .md in the filename after the rename.

Even though it works as designed, I realized that it could be improved either by adding .md in the rename popup explicitly or by checking the filename's current extension and adding it automatically since it might not be evident that the user should add it.

It should work as intended in v0.2.1 (just released). Please remember to update Memo extension and reload VS Code.


Question: how to link rule with sheet option link:true?

Hi!
I'm trying to create a dynamic flex container with react-jss. Specifically, I'm trying to make a dynamic flex item gutter. When I added the dynamic flex item gutter on the flex items (ie. a positive margin on all childs), I got the warning: Warning: Rule is not linked. Missing sheet option "link: true".
My code looks like this:
const styles = {
    container: {
	display: 'flex',
	flexDirection: props => props.flexDirection,
	flexWrap: props => props.wrap,
	justifyContent: props => props.justifyContent,
	alignItems: props => props.alignItems,
	alignContent: props => props.alignContent,
	// Item spacing:
	margin: props => -(props..itemSpacing/2)
	'&>*': {
		// This is what causes the "rule not linked" warning
		margin: props => (props.itemSpacing/2),
	},
    },
}

export let Container = props => (
	<div className={props.classes.container}>
		{props.children}
	</div>
)

Container = injectSheet(styles)(Container)

My question is: how do I link this rule with sheet option link:true to make it work dynamically? Thanks!:)

what you posted should already work, link: true is integrated in react-jss, the last part with '&>*' will not work with function value due to https://github.com/cssinjs/jss/issues/682

Thanks. Do you know of a workaround, though, @kof , as the function value inside the child selector doesn't work?

The problem is your selector, see for an example: https://codesandbox.io/s/o43vmmlmp6
Your selector should be to get it working: & > *

Thanks, @HenriBeck!:) Why should & > * working, but not &>*? Does JSS try to do different things in these cases?

there is no difference between & > * and &>* for jss

Why does my codesandbox than work?

Need to check, there was a problem that we introduced in the recent versions.

Yeah, but that was only related to referencing rules I think.


Add Support For snowflake-connector-python>=3.0.0

Is your feature request related to a problem? Please describe.
Dependencies often found in the same projects as data-diff now require snowflake-connector-python>=3.0.0 but data-diff requires <3.0.0.
Describe the solution you'd like
Update the snowflake-connector-python dependency range to be >=2.7.2, <4.0.0 (or some upper bound above 3.0.0).
Describe alternatives you've considered
N/A
Additional context
There are vulnerabilities identified in snowflake-connector-python<2.8.2

We'll need to update https://github.com/datafold/sqeleton as well

Seeing another report of this. Here is the workaround for now:
# Installing data-diff for snowflake when you have another dependency that requires snowflake-connector-python>=3.0.0

pip install data-diff -U
pip install 'snowflake-connector-python>=3.0.0'

Alternatively in a requirements.txt, something like:
dbt-snowflake>=1.5
data-diff>=0.7.7
snowflake-connector-python>=3.0.0


Update Preferences.cpp

Follow my previous PR.
I apologize for that, but i'm not very skilled and Git is again a strange animal to me ;)

That's should be more "clean" ;)
My Git is laborious but i'm working on it :smile:

Please also update Format.csv (7th column) with the corresponding extensions, as there is now a test on the validity of the file extension, and your files are marked as with the wrong extension.

Please update the patch with spaces instead of tabs.

@JeromeMartinez Thank you ;)
Done in https://github.com/MediaArea/MediaInfoLib/pull/388


Possible to query date to present?

Is it possible to query files modified from a particular date onwards?
For example, I can do this for everything updated in the past week:
<!-- note-overview-plugin
search: updated:day-7
fields: title, updated_time
sort: title DESC
-->

But is it possible to do change that search for everything modified from 11/20/2023 onwards?
I tried a handful of queries trying to use the moments filter, but I couldn't get the results I was looking for.

The plugin use the Joplin search/search  filters for query data.
You can use a date like this: updated:20231120

Thank you! I thought that would only show files updated on that exact date, guess I should've tested more. Cheers!


is repo still maintained ?

At least a bug fixing

hi @djleonskennedy ,
Yes it is, I was just too busy on the last few weeks.


How to get internet gateways via neutronclient?

@nhatbkk57 : Please help me. I wonder what is internet gateways in network quota? And how to get them via python-neutronclient package?

Ok! an "internet gateway" means:
In OPS:

a router which connects to external network
limit: according admin allocate for user

In VPS:

a component or a resource that allows communication between instances in your VPC and the Internet
Default limit: 5 Internet gateways per region

How to get them in OPS:

count the numbers of router which have "external_gateway_info" being not null look like below dict
"external_gateway_info": {
"enable_snat": true,
"external_fixed_ips": [
{
"ip_address": "<IP_ADDRESS>",
"subnet_id": "b930d7f6-ceb7-40a0-8b81-a425dd994ccf"
},
],
"network_id": "ae34051f-aa6c-4c75-abf5-50dc9ac99ef3"
},

@daikk115 can you create a router connecting to the external network and an internal network and create another only connecting to internal network?? and then send me the json response???

@nhatbkk57 : please connect to bkcloud16 via ssh. I have some test code in /home/bkcloud16/test-neutronclient.(byobu)

@daikk115 ping, close this issue?


Typo "with a Azure DevOps"→"with an Azure DevOps"

https://learn.microsoft.com/en-us/azure/app-service/scripts/cli-continuous-deployment-vsts
https://github.com/MicrosoftDocs/azure-docs/blob/main/articles/app-service/scripts/cli-continuous-deployment-vsts.md
#PingMSFTDocs

Thanks, @hyoshioka0128!


/Patient//edit route on https://provider.medplum.com should default to React component with US Core profile.  This is required for a5

This is required for a5


[BUG] EventGridEvent incompatible with Asp.Net Core as it does not have a suitable constructor

Describe the bug
[HttpPost]  [Route("handle_event")]  public async Task<IActionResult> HandleEvent([FromBody] EventGridEvent[] eventGridEvents)
Expected behavior
Should deserialise JSON data as EventGridEvent[]
Actual behavior (include Exception or Stack Trace)
Throws deserialisation exception: No suitable constructor.
To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue)

Create Asp.Net Core controller method with signature as above
Expose running Asp.Net website to public Internet (I find ngrok useful for this)
Attempt to create an Event Subscription with Endpoint Type "Web Hook" pointing to your public website suffixed with /handle_event
Boom

Environment:

Name and version of the Library package used: Azure.Messaging.EventGrid 4.4.0
Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects):
.NET SDK (reflecting any global.json):
Version:   5.0.301
Commit:    ef17233f86

Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19041
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\5.0.301\
Host (useful for support):
Version: 5.0.7
Commit:  556582d964
.NET SDKs installed:
3.1.410 [C:\Program Files\dotnet\sdk]
5.0.104 [C:\Program Files\dotnet\sdk]
5.0.202 [C:\Program Files\dotnet\sdk]
5.0.204 [C:\Program Files\dotnet\sdk]
5.0.301 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.All 2.1.28 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.28 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.16 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.28 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.16 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.4 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.16 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 5.0.4 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 5.0.7 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

IDE and version : Visual Studio 16.10.3

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Can you include the exception stack? If the error is coming from System.Text.Json this should be fixed in our next release (in a couple weeks). You can test it out using the Nuget dev feed - https://github.com/Azure/azure-sdk-for-net/blob/main/CONTRIBUTING.md#nuget-package-dev-feed

Hi. Sorry for the late reply: I was on holiday last week and did not have access to my development environment.
Here is the exception and stack as requested. As you suggested, the exception is coming from System.Text.Json:
fail: Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware[1]
An unhandled exception has occurred while executing the request.
System.NotSupportedException: Deserialization of types without a parameterless constructor, a singular parameterized constructor, or a parameterized constructor annotated with 'JsonConstructorAttribute' is not supported. Type 'Azure.Messaging.EventGrid.EventGridEvent'. Path: $[0] | LineNumber: 0 | BytePositionInLine: 2.
---> System.NotSupportedException: Deserialization of types without a parameterless constructor, a singular parameterized constructor, or a parameterized constructor annotated with 'JsonConstructorAttribute' is not supported. Type 'Azure.Messaging.EventGrid.EventGridEvent'.
--- End of inner exception stack trace ---
at System.Text.Json.ThrowHelper.ThrowNotSupportedException(ReadStack& state, Utf8JsonReader& reader, NotSupportedException ex)
at System.Text.Json.ThrowHelper.ThrowNotSupportedException_DeserializeNoConstructor(Type type, Utf8JsonReader& reader, ReadStack& state)
at System.Text.Json.Serialization.Converters.ObjectDefaultConverter1.OnTryRead(Utf8JsonReader& reader, Type typeToConvert, JsonSerializerOptions options, ReadStack& state, T& value)          at System.Text.Json.Serialization.JsonConverter1.TryRead(Utf8JsonReader& reader, Type typeToConvert, JsonSerializerOptions options, ReadStack& state, T& value)
at System.Text.Json.Serialization.Converters.IEnumerableDefaultConverter2.OnTryRead(Utf8JsonReader& reader, Type typeToConvert, JsonSerializerOptions options, ReadStack& state, TCollection& value)          at System.Text.Json.Serialization.JsonConverter1.TryRead(Utf8JsonReader& reader, Type typeToConvert, JsonSerializerOptions options, ReadStack& state, T& value)
at System.Text.Json.Serialization.JsonConverter1.ReadCore(Utf8JsonReader& reader, JsonSerializerOptions options, ReadStack& state)          at System.Text.Json.Serialization.JsonConverter1.ReadCoreAsObject(Utf8JsonReader& reader, JsonSerializerOptions options, ReadStack& state)
at System.Text.Json.JsonSerializer.ReadCore[TValue](JsonConverter jsonConverter, Utf8JsonReader& reader, JsonSerializerOptions options, ReadStack& state)
at System.Text.Json.JsonSerializer.ReadCore[TValue](JsonReaderState& readerState, Boolean isFinalBlock, ReadOnlySpan`1 buffer, JsonSerializerOptions options, ReadStack& state, JsonConverter converterBase)
at System.Text.Json.JsonSerializer.ReadAsync[TValue](Stream utf8Json, Type returnType, JsonSerializerOptions options, CancellationToken cancellationToken)
at Microsoft.AspNetCore.Mvc.Formatters.SystemTextJsonInputFormatter.ReadRequestBodyAsync(InputFormatterContext context, Encoding encoding)
at Microsoft.AspNetCore.Mvc.Formatters.SystemTextJsonInputFormatter.ReadRequestBodyAsync(InputFormatterContext context, Encoding encoding)
at Microsoft.AspNetCore.Mvc.ModelBinding.Binders.BodyModelBinder.BindModelAsync(ModelBindingContext bindingContext)
at Microsoft.AspNetCore.Mvc.ModelBinding.ParameterBinder.BindModelAsync(ActionContext actionContext, IModelBinder modelBinder, IValueProvider valueProvider, ParameterDescriptor parameter, ModelMetadata metadata, Object value, Object container)
at Microsoft.AspNetCore.Mvc.Controllers.ControllerBinderDelegateProvider.<>c__DisplayClass0_0.<g__Bind|0>d.MoveNext()
--- End of stack trace from previous location ---
at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.g__Awaited|13_0(ControllerActionInvoker invoker, Task lastTask, State next, Scope scope, Object state, Boolean isCompleted)
at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.g__Awaited|19_0(ResourceInvoker invoker, Task lastTask, State next, Scope scope, Object state, Boolean isCompleted)
at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.g__Awaited|17_0(ResourceInvoker invoker, Task task, IDisposable scope)
at Microsoft.AspNetCore.Routing.EndpointMiddleware.g__AwaitRequestTask|6_0(Endpoint endpoint, Task requestTask, ILogger logger)
at Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context)
at Swashbuckle.AspNetCore.SwaggerUI.SwaggerUIMiddleware.Invoke(HttpContext httpContext)
at Swashbuckle.AspNetCore.Swagger.SwaggerMiddleware.Invoke(HttpContext httpContext, ISwaggerProvider swaggerProvider)
at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.Invoke(HttpContext context)

Thanks, that is as I expected - the issue is coming from System.Text.Json not being being able to deserialize. We are planning to ship a new version today that will have a fix for this.

This is fixed in https://www.nuget.org/packages/Azure.Messaging.EventGrid/4.5.0


Add python 3.8.6 constraint to whitesource

Supersedes #14
In theory, this change should pin the Python version to 3.8.6, which matches the version of Python Cloud Buildpacks uses. This should prevent automated PRs from whitesource like #14 trying to bump this dependency.

Thank you


Ignore computed read-only properties when inferring InputObjectType fields

Product: HotChocolate
Adds a simple check to ignore computed read-only properties when binding an InputObjectType from inferred fields.
~The check does not analyze the getter body and cannot distinguish between real computed properties and old-style read-only properties which simply return a read-only field initialized in a consstructor, but it should cover the overwhelming majority of use cases.~
Closes #5673

I have some further thoughts. Wouldn't it be better if we solved this particular problem with a source generator that creates the formatter and parser. We have started shifting some problems into this area? What are your thoughts?

As far as I understand, method IL is in memory anyway, and the cost of the check is trivial, just a couple of reflection calls.
AOP and code generation happen before the assembly loads. The check operates on the getters that will actually execute, so it should work regardless.
I think this is a much more serious concern. There may be quite a few such patterns users may want to handle, and someone is bound to be unhappy with any hard-coded set of patterns. I have since found that I can handle the issue that prompted this PR on my side by configuring my input object types, so I no longer think that the approach above is a good idea. I have two other suggestions instead:

Add a TypeContext parameter to ITypeInspector.GetMembers so that custom implementations can choose to omit members specifically when HC infers fields for input object types. This would enable the author of #5673 (and me) to deal with the issue with a few lines of code as long as they control the code base and know which property patterns are in use / allowed by code style.
Add something like IInputObjectTypeDescriptor.Field(MemberInfo) because right now I have to construct a LINQ expression around a property just so that InputObjectTypeDescriptor<T>.Field can call ExtractMember() on it to get the property back out.

AOP and code generation happen before the assembly loads. The check operates on the getters that will actually execute, so it should work regardless.

With AOP there should be no longer any IL code as I understand it ... there also os no longer a JIT. They for compatibillity allow expression compilation but they are interpreted. We have just started looking into, so I have to read up on a lot of stuff regarding AOP.

Add a TypeContext parameter to ITypeInspector.GetMembers so that custom implementations can choose to omit ...

That sounds like a good idea, we can do that. But since its a breaking change it will only become available with version 14.

With AOP there should be no longer any IL code as I understand it ... there also os no longer a JIT.

Perhaps you mean AOT (ahead of time) compilation? If I remember correctly, AOP stands for aspect-oriented programming. If you mean AOT then yes IL analysis may well stop working. I don't know how much metadata is retained in AOT's output.

That sounds like a good idea, we can do that. But since its a breaking change it will only become available with version 14.

I can live with Configure for now, but I can make a PR and try it out.

It should almost certainly be a separate PR, so I'll close this one.


it is not work :(

I try
not work I do all steps :(

ltr:pl-4

That is not nearly enough info for me to go on. What's your Tailwind config, for starters?

module.exports = {
purge: [
'./resources//*.blade.php',
'./resources//.js',
'./resources/**/.vue',
],
darkMode: false, // or 'media' or 'class'
theme: {
extend: {},
},
variants: {
extend: {
float: ['responsive', 'direction'],
margin: ['responsive', 'direction'],
padding: ['responsive', 'direction'],
},
},
plugins: [
require('tailwindcss-dir')(),
],
}

new one install not use any plugin

frist : i forget thank you for your reply :)

I use Laravel 8 and just install Tailwind and use your plugin thank you

it is work rtl and ltr but not this   ltr:pl-4 rtl:pr-4

it is work I don't how but work thank you :)


feat: CIP-1694

Epoch params and proposal params both missing pvt_p_p_security_group

Another new protocol param missing - min_fee_ref_script_cost_per_byte


Reading subsets of streams from files without iterating over everything

Hello,
I am investigating different toolboxes for PET listmode reconstruction and was trying out PyTomography, which supports PETSIRD as its input format, which in turn uses yardl for the model definition.
Specifically, I am looking into dynamic (frame by frame) reconstruction of the listmode data.
Furthermore, for testing PyTomography I tried to limit data input to a subset of a few seconds acquisition time, to try things out and not waste time loading and processing my full dataset (~10GB, ~1h+ acquistion time) before I know that things actually work.
PyTomography does currently not support specifying concrete time intervals to use from listmode data with a longer acquisition time. It allows to specify timeblock IDs, however, it still iterates over all the timeblocks, filtering out the ones with the specified IDs, which is very inefficient (see here).
Therefore, I dug a bit deeper and tried to figure out if I can adapt the toolbox in a way which allows me to read only certain timespans from the PETSIRD file. After seeing that the protocol readers use streams, I was analyzing the binary structure of the protocols to maybe find a way to calculate the size of the data and then seek the correct position in the file where I'd like to read data.
I found out that vector and array types store their length as the first part of the data in binary form. Therefore, AFAICT what I want to achieve isn't possible and I would have to iterate over all the timeblocks because I can't know the length of the event vectors stored inside them upfront.
Is there another way of achieving what I am trying to? Maybe I am missing/misunderstanding something also, please let me know if this should be the case :slightly_smiling_face:
If it really is not possible, IMO it would be a useful feature to consider. Being forced to read the full file when interested in part of it is cumbersome IMO. Also for dynamic reconstruction it could be beneficial, as it wouldn't be necessary to wait for loading of all frames before the first one can be processed.
However, I am not really far into the scope of this project, for transmitting the same stream of bytes over a network this would not make any sense I guess, so let me know if this just is not intended. :+1:
PETSIRD protocol definition can be found here.
Thanks in advance and best regards, Florian

Hi Florian,
You are correct that this is not yet possible in Yardl, however, this is one of the next features we plan to add.
Also, perhaps I misunderstand your meaning of "dynamic reconstruction", but you certainly don't have to wait to load a full dataset before processing it. Yardl enables processing your data as you read it from the stream (file, network, etc.). You can find examples of this for the MRD format here.

Thank you for your reply.
As mentioned above by "dynamic" I am referring to a frame by frame reconstruction. I am currently part of a neuroscience group which is using fPET. So we are reconstructing consecutive frames with a few seconds acquisition time to investigate if certain stimuli do cause a response in the PET signal.
I was looking for a quick solution to get PyTomography working because I think it has an interesting implementation approach for reconstruction which I didn't want to disregard only because of some minor problems.
I am aware that I don't have to load the full dataset, however, in my case there's no activity in the first frames so I would have wanted to jump somewhere into the middle of acquisition. Continuing the stream from where the last frame ended would help but require some major changes inside PyTomography and I was looking for a quick fix but disregarded it from my analysis for the time being.
For this kind of dynamic reconstruction it would be beneficial to jump somewhere into the stream to reconstruct single frames for debugging and evaluation but it is surely an edge case which does not come up in many applications.
BR, Florian


improve change memtable memory limiter policy

Proposed changes
#40912 has changed meaning of write_mem in memtable memory limiter.
This PR is a followup to change the active memtable flush policy accordingly.
It also changed:

The amount of memtable writers selected in one flush.
The memtable writers are selected in orders of its size.

run buildall


hysteriav2 tls: failed to verify certificate: x509: certificate signed by unknown authority when using CA cert

Describe the bug
I want to setup a hysteriav2 server with a self signed certificate, without using a domain name. I want the connection to be secure, not prone to MITM attacks. So I setup a self signed CA and server following https://v1.hysteria.network/docs/custom-ca/. I copied the CA cert to the client and tried to connect to the server, however I'm getting TLS error saying the certificate is signed by unknown authority.
To Reproduce
https://v1.hysteria.network/docs/custom-ca/ on hysteria2
Expected behavior
Not getting TLS errors, client being able to proxy through hysteriav2 server.
Logs
Nekoray's client side logs:
[Warning] [4156035736] app/proxyman/outbound: failed to process outbound traffic > proxy/socks: failed to establish connection to server > proxy/socks: server rejects request: 4 tcp:<IP_ADDRESS>:60440 accepted tcp:<IP_ADDRESS>:80 [socks-in -> proxy] ERROR	SOCKS5 TCP error	{"addr": "<IP_ADDRESS>:43416", "reqAddr": "<IP_ADDRESS>:80", "error": "connect error: CRYPTO_ERROR 0x12a (local): tls: failed to verify certificate: x509: certificate signed by unknown authority"}
Server logs: none
Device and Operating System
OpenBSD VPS running on VMD

Please add the following lines to your client configuration file
tls:
  insecure: true

If the problem persists, please provide your server & client config for further assistance.

@haruue
I did add that line, in which case it does work. But won't the certificate be ignored and the client would just connect to the server without verifying if I do? (I did test with a wrong certificate and the connection is still established). Sorry if this question seems stupid but I didn't find the docs that answer it.

You can specify your own CA file for TLS verification in the client configuration file:
tls:
  ca: custom_ca.crt

You can also pin the server certificate's SHA-256 fingerprint.
tls:
  pinSHA256: BA:88:45:17:A1:...

The fingerprint is generated byopenssl x509 -noout -fingerprint -sha256 -in server.crt

Please refer to our document for details:
https://hysteria.network/docs/advanced/Full-Client-Config/#tls


Restore "resources" layer

This layer is actually required for regular docker images (no optimized/AOT) since the jars which are included are not the regular jars.
Fixes #938

@melix remember to use GitHub Projects. I added it to 4.3.0.


Java Context API example uses the wrong context

What needs to be changed?
The Java Context API site has an example where the current context instead of the local one should be used.
What is the name + path of the page that needs changed?
content/en/docs/languages/java/api.md
Additional context:
Tiny fix, just not worth signing a CLA. See https://github.com/open-telemetry/opentelemetry.io/pull/5752

Thanks for the find. Code for the excerpt lives over here: https://github.com/open-telemetry/opentelemetry-java-examples/blob/main/doc-snippets/api/src/main/java/otel/ContextUsage.java#L39
I'll get to this soon enough, but just sharing in case anybody beats me to it.


tons of network traffic, but nothing is ever written to the pipe

$ ./qt428 -c 6 <IP_ADDRESS> > /tmp/S
tcpflow shows lots of data going over the network into ./qt428,
but nothing is ever written to the file.

I can see some images from the tcpflow files I made during the process,
mplayer -fps 1 -demuxer h264es /tmp/<IP_ADDRESS>.06036-<IP_ADDRESS>.43608

With this patch, clear_as_day.txt
I can now do
./qt428 -c 6 <IP_ADDRESS> | mplayer -fps 15 -demuxer h264es -
and now everything is clear as day, at least for me operating my neighbor's generic unknown DVR!
How did I get the idea to try it? Well https://github.com/mdrush/zmodopipe/ has
	char suppLoginBuf[] = {
	0x31, 0x31, 0x31, 0x31, 0x34, 0x00, 0x00, 0x00,
	0x01, 0x02, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
	0x01, 0x00, 0x00, 0x00, 0x24, 0x00, 0x00, 0x00,
	0x00, 0x00, 0x00, 0x00, 0x00/*high res*/, 0x00, 0x00, 0x00,
	0x00, 0x00, 0x00, 0x00, 0x00/*low res*/, 0x00, 0x00, 0x00,
	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
	0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
	0x00, 0x00, 0x00, 0x00 };

	if (channel <= 7 && channel >= 0) {
		if (highres) {
			suppLoginBuf[28] = channel+1;
		}
		else {
			suppLoginBuf[36] = channel+1;
		}
	}

so

Anyways, all I know is the https://play.google.com/store/apps/details?id=com.tvt.network  SuperLivePro app, right on the login screen, has the choice between "2G" and "3G" low and high resolution, and your fork doesn't :-(.


Implemented Linear Controller and added unit test.

The autoscaler are now supporting one more control pattern: linear mode. An example for the ConfigMap params as below:
data:
  linear: |-
    {
      "coresPerReplica": 2,
      "nodesPerReplica": 1,
      "min": 1,
      "max": 100
    }

Because serious performance evaluation of kube-dns might not be finished soon, I would like to use this simple pattern for the DNS horizontal scaling for now (will tune the specific params). Details for this new pattern are updated in README.
@bowei @thockin @bprashanth
cc @matchstick

this is net simpler, which is good.  I forget why Girish wanted the ladder mode.  Do we need to keep it?

I don't think he gave out the reason. But I think it does not harm to keep it.

Dead code is a liability - keeping it around means we support it and debug
it and compile it and test it.  If we really think linear is sufficient,
and I don't see why not, we should remove the ladder.  YAGNI.
https://dougseven.com/2014/04/17/knightmare-a-devops-cautionary-tale/
On Wed, Oct 19, 2016 at 5:46 PM, Zihong Zheng<EMAIL_ADDRESS>wrote:

I don't think he gave out the reason. But I think it does not harm to keep
it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-incubator/cluster-proportional-autoscaler/pull/5#issuecomment-254980801,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFVgVB0BYG-Kqe_GfKkt2l--DhsbKS81ks5q1rn4gaJpZM4KairZ
.

Your concern totally makes sense, but I vote for keeping the ladder mode:

I won't say ladder mode is completely useless. It is, in my opinion, another common and intuitive control pattern.
linear mode might be sufficient for kube-dns scaling. But I would like to see other applications also use this autoscaler and they may have different requirements.
ladder mode does something that linear mode can not do. It provides users more flexibility and customization. Before we have better substitute for this, I would like to support this pattern.

I would agree with keeping ladder as it is a way out for people to implement their own arbitrary scaling function

OK.
On Thu, Oct 20, 2016 at 3:24 PM, Bowei Du<EMAIL_ADDRESS>wrote:

I would agree with keeping ladder as it is a way out for people to
implement their own arbitrary scaling function
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-incubator/cluster-proportional-autoscaler/pull/5#issuecomment-255245171,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFVgVEkwsdukB4SD4I6bO_JQpg8lqtMWks5q1-omgaJpZM4KairZ
.

Thanks! Self merging.


Possible bug in maxShare and minShare

Don't know exactly is this a bug, but here:
https://github.com/graphhopper/jsprit/blob/master/jsprit-core/src/main/java/com/graphhopper/jsprit/core/algorithm/box/Jsprit.java#L220
in lines 220 and 221 you define maxShare_ and minShare_ integers, but they are equal:
int minShare_ = (int) Math.min(70, Math.max(5, vrp.getJobs().size() * 0.5));
int maxShare_ = (int) Math.min(70, Math.max(5, vrp.getJobs().size() * 0.5));

but in 212 and 213 you are declaring maxShare and minShare these way:
int minShare = (int) Math.min(20, Math.max(3, vrp.getJobs().size() * 0.05));
int maxShare = (int) Math.min(50, Math.max(5, vrp.getJobs().size() * 0.3));

The question is: should minShare_ and maxShare_ be equal?

Thanks @kschingiz. It is no bug. min and max is just equal in this configuration here.


list_relations_without_caching resulting in timeout

@WangSleep commented on Thu Nov 19 2020
When i excute "dbt docs generate" on CLI .it dosen't work .
it's error message :
PrestoQueryError(type=INTERNAL_ERROR, name=OPTIMIZER_TIMEOUT, message="The optimizer exhausted the time limit of 180000 ms", query_id=20201110_082631_11221_7vjyv)
i can excute "dbt deug" , "dbt test" ,"dbt run" on CLI . but "dbt docs generate" alone can't
Can you solve my problem?
thanks a lot ~~~😊

@WangSleep commented on Thu Nov 19 2020
I found out when I performed "dbt docs generate" , it will excute this sql
select       table_catalog as database,       table_name as name,       table_schema as schema,       case when table_type = 'BASE TABLE' then 'table'            when table_type = 'VIEW' then 'view'            else table_type       end as table_type     from "hive".INFORMATION_SCHEMA.tables     where regexp_like(table_schema, '(?i)\ADatebase\Z')
Too many resources are required, and execution time is too long to cause a timeout 。
Could you tell me what can be solved?  thanks a lot

@jtcohen6 commented on Thu Nov 19 2020
Hey @WangSleep, which database / adapter plugin are you using? It sounds like the catalog query—which dbt uses to grab metadata, populate catalog.json, and thereby feed information into the docs site—may be inefficient as currently implemented on this adapter.

@WangSleep commented on Thu Nov 19 2020

Hey @WangSleep, which database / adapter plugin are you using? It sounds like the catalog query—which dbt uses to grab metadata, populate catalog.json, and thereby feed information into the docs site—may be inefficient as currently implemented on this adapter.

yeh .
We query by configuring presto-python-client, which I know is for docs, but python_presto query "hive". INFORMATION_SCHEMA is slow. This causes a timeout. Later I found out that I used lore-zeppelin, which was based on jdbc, and I could query. It takes about 5 minutes.
thanks a lot ~

@jtcohen6 commented on Fri Nov 20 2020
I'm still a bit confused. Are you using dbt-presto? What's the output when you run dbt --version?

@WangSleep commented on Sun Nov 22 2020

I'm still a bit confused. Are you using dbt-presto? What's the output when you run dbt --version?

hi , bro !
yes .
I'm using it . dbt --version : 0.18.1

@jtcohen6 commented on Mon Nov 23 2020
We haven't released dbt-presto v0.18.1 yet (which is on us), and presto__get_catalog as implemented in that plugin looks different from the query snippet you pasted above. (In fact, your snippet doesn't look like any of the get_catalog queries familiar to me.)
So... I'm still confused as to which dbt plugin you're using :)

@WangSleep commented on Mon Nov 23 2020

We haven't released dbt-presto v0.18.1 yet (which is on us), and presto__get_catalog as implemented in that plugin looks different from the query snippet you pasted above. (In fact, your snippet doesn't look like any of the get_catalog queries familiar to me.)
So... I'm still confused as to which dbt plugin you're using :)

hi .  @jtcohen6
FYI
This is the version I installed  :)

@jtcohen6 commented on Tue Nov 24 2020
Ok, this makes much more sense now. Appreciate the clarification!
The snippet you included comes from presto__list_relations_without_caching. I'm not sure why it only times out on dbt docs generate, since that's a query that should run at the start of all invocations.
I'm going to transfer this issue to the dbt-presto repo, where we can discuss if there is a more performant way to write that metadata introspection. Out of curiosity, how many objects are in your "hive" catalog?

thanks ~
about 4000 - 5000 metadata table in hive and about 100 database information when i need to query :)

I'm pretty sure the culprit here is presto_ilike, which performs regex in the filter and likely requires scanning all tables in the metastore.
Several months ago, we made the decision to switch dbt-snowflake from performing ilike queries of information_schema.tables to instead use show terse objects in schema (https://github.com/fishtown-analytics/dbt/pull/2322).
Presto supports both information_schema queries and show for schemas, tables, and columns. It would be worth doing some performance benchmarking to check if one is faster than the other. The big change, that we should make regardless, is to remove the regex filter and replace it with an equality match, both in list_relations_without_caching and get_columns_in_relation.
As in dbt-snowflake, it will require a bit of extra handling to first figure out the right quoting and casing of available schemas.

I'm pretty sure the culprit here is presto_ilike, which performs regex in the filter and likely requires scanning all tables in the metastore.
Several months ago, we made the decision to switch dbt-snowflake from performing ilike queries of information_schema.tables to instead use show terse objects in schema (fishtown-analytics/dbt#2322).
Presto supports both information_schema queries and show for schemas, tables, and columns. It would be worth doing some performance benchmarking to check if one is faster than the other. The big change, that we should make regardless, is to remove the regex filter and replace it with an equality match, both in list_relations_without_caching and get_columns_in_relation.
As in dbt-snowflake, it will require a bit of extra handling to first figure out the right quoting and casing of available schemas.

I modified the adapters.sql and catalog.sql files。
adapters.sql : use '=' relpace 'ilike'
catalog.sql :  add two 'where' Conditions.
It now takes about 2 minutes to execute。so , thansk a lot .
For now, it's settled 。
thank you vary much .and finally ，snapshot materialization not implemented for presto。


Auto close

Why auto close in real device?

@cloudartos I think it's a problem with the material app because when I make the same design with cupertino it does not happen and second is the quality of the images is very high.


Update CONTRIBUTING.md with ruff

Closes #323

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (7bc338b) compared to base (383b824).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #324   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        10
  Lines          196       196
=========================================
  Hits           196       196

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Distribution auto-jumping bug fix

Distributions keep track of the number of jumps in a variable named ind. With auto-jumping, ind is increased by 1 each time the distribution is called. At the beginning of each time step, we were calling jump() with to=self.ti+1, which sets all distribution ind values to ti+1.
The major problem here is that with auto-jumping, a user could call a distribution multiple times per time step. The jumps counter (ind) might start at say 1, and subsequently increase to 2, 3, and 4 during a step (if a user calls rvs several times). But then at the next time step, step() was calling jump() setting ind to 2, causing repeated random numbers to be used.
The easy one-line fix is to remove the to= argument to jump inside the step loop.
NOTE: This science bug fix could change simulation results.

Let's discuss in the meeting tonight -- this solves the bug you describe, but introduces a new bug such that if there is ever a difference in the number of RNG calls, the numbers for that dist won't match for the rest of the sim. I think the solution is to increment +1 per call, but increment by a large number (larger than the possible number of calls) per timestep.

Should be fixed by #638

Going to include this after all as v1.0.2.