Merging Video and Audio Using youtube-dl

Checklist

[ ] I'm asking a question
[ ] I've looked through the README and FAQ for similar questions
[ ] I've searched the bugtracker for similar questions including closed ones

Question

I'm experiencing an issue when attempting to merge video and audio using youtube-dl. I have used the following command:
youtube-dl -g -e --get-thumbnail  -f 137+140 --merge-output-format mp4 --get-format URL

You're not asking a question.
First review #30839. Create a new issue following and completing (unlike above) the correct template, including a verbose log to show the software versions in use, what you were trying to do, etc.


Stricter declarations

Description
Restrict top level declarations and the declarations permitted as statements.
In doing so we also provide improved support for unnamed classes at the top level, and we clearly define the only valid locations for import and package declarations as outlined in the Java Language Specification.
Motivation
My motivation for this PR is that I am using tree-sitter both for syntax highlighting and for code validation. A key issue that I encountered was that various types of declaration could occur very widely within the grammar because they are permitted within statements.
I believe these changes are actually very congruent with the rest of the grammar, because only in statement were all declarations allowed. In other cases such as _class_body_declaration, annotation_type_body, interface_body we specify the exact types of permitted inner declarations. We now extend this idea to disallow any declaration inside a statement, and only permit class_declaration inside  a block.
Checklist:

[ ] All tests pass in CI.
[ ] There are sufficient tests for the new fix/feature.
[ ] Grammar rules have not been renamed unless absolutely necessary.
[ ] The conflicts section hasn't grown too much.
[ ] The parser size hasn't grown too much (check the value of STATE_COUNT in src/parser.c).

Please let me know your feedback on this direction and then I'll get to fixing the tests :)

Top level statements are unrestricted on purpose.


history var missing in externs ?

The History object is declared in https://github.com/google/closure-compiler/blob/master/externs/browser/html5.js#L2268
but not the history property of the window object in https://github.com/google/closure-compiler/blob/master/externs/browser/window.js .

It's here: https://github.com/google/closure-compiler/blob/master/externs/browser/ie_dom.js#L464.
Yeah our externs are kind of all over the place... :(

That's the constructor - the actual variable is: https://github.com/google/closure-compiler/blob/master/externs/browser/gecko_dom.js#L105-L110

@ChadKillingsworth Yeah thanks for the clarification.


Initial working code!

Still looking into the issue. I think I can autodoc_mock_imports circuitpython_typing and get it to pass, but something else is going on here. It's passing locally. circuitpython_typing is where WriteableBuffer is, the module comes from Adafruit-Blinka, and for some reason remote Sphinx isn't seeing it remotely. Locally, Sphinx passes on the current state of this PR. As well, this is apparently not the first library this has happened on - so it's worth actually finding the issue instead of simply doing a workaround. Working with @FoamyGuy to see if we can get it sorted out.

If that's the case, is there the chance that other Blinka modules won't need to be mocked, like digitalio?


Add database tools section to docs

Description
Added separate docs section for database tools on Windows 2016/2019, currently available tools are Azure CosmosDb Emulator, Mysql CLI tool, SQLPS module
Related issue: actions/virtual-environments-internal#1221
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run windows2016, windows2019


Multiple fixes

CORR: changing check for year 9999 and change from 2050 to 2100.
utils-json: checking for year 9999 and writing to 2100 instead to address problem with R and rendering issue for rectangles
uvhydrograph styles: adding same colored border to rectangles so that the little white gap doesn't appear when it was

Coverage increased (+0.1%) to 86.763% when pulling 2f23bcdc1f4ef127f5a1510f12099d33d19d930b on mhines-usgs:master into 4d5090d22c3b8d203bffb2527f4d4fa0cb6c3f84 on USGS-R:master.


Does Scully work without Angular Router ?

Hi There,
I wish you all a happy new year ! And congratulations for this great tool ! I'm working on Angular projects without the Router, and I would like to know if Scully can handle this kind of Angular project.
Thank you !
Benoît

@BenLune No, Scully does not work without the router.
If you really want, it can be done, but the out of the box experience requires the router.
When you want to go without the router, you need to manually set it up, and there is no guidance. It is possible, but there is no documentation or help available if you want to do so.

@BenLune No, Scully does not work without the router.
If you really want, it can be done, but the out of the box experience requires the router.
When you want to go without the router, you need to manually set it up, and there is no guidance. It is possible, but there is no documentation or help available if you want to do so.


Add more details about Exclude paths from requiring authentication

There are some scenarios that is tricky with the exclusion list:

How to specify more than one paths? Like '/a;/b'.
Is the path match case sensitive or not?
Is there a way to exclude the root like '/' but not other paths;

For example, what shall I use to exclude root '/' from requiring client cert but still requires it on other path like '/abc/' - is it possible?

Could you please update the documentation to explain the usage for those cases?

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

ID: eb440603-950a-c224-f6c9-222ca502d5a5
Version Independent ID: 0b5c4944-0327-91fc-2b0b-c19a1b873b64
Content: Configure TLS mutual authentication - Azure App Service
Content Source: articles/app-service/app-service-web-configure-tls-mutual-auth.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@xiaomi7732 Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.


[Sage-8] Tokens - Font Family Update to Circular

Description

Updates the Sage font family from Inter to Circular.
Note: Linting is failing because the fonts aren't being committed.
Screenshots

Before
After

Testing in sage-lib

View Sage Docs
Check that new font, Circular, is properly in use.

Testing in kajabi-products

(LOW) Updates the Sage font family from Inter to Circular. No effect on Kajabi Products.
Related

Sage-8

Regarding the failing lint we have options:

Merge into our epic branch and know we'll fix it shortly
Hold this PR for a bit as we work out the last bits of the CDN-based approach to the font paths.

Regarding the failing lint we have options:

Merge into our epic branch and know we'll fix it shortly
Hold this PR for a bit as we work out the last bits of the CDN-based approach to the font paths.

I can hold on to this PR and update when you're ready.

@anechol we're set up and ready with all the fonts under https://sage.kajabi-cdn.com/fonts/circular

I'm doing a separate PR on develop to handle this for the existing fonts FWIW.

For reference as needed https://github.com/Kajabi/sage-lib/pull/1183


feat: add Cosmos and Solana wallet and signer to SDK example

@RyukTheCoder Thanks for the PR. Please check why deployment has some errors?

@RyukTheCoder Thanks for the PR. Please check why preview deployment has some errors?

Fixed

When the Metamask session remains active and the wallet is not connected, initiating the 'Swap' action on an alternative chain triggers Metamask to connect, leading to the appearance of the network switch pop-up. Changing the network subsequently results in the presentation of this error :


[Feature] Operator to support alert manager

Apologies in advance if this is not the right place to open feature requests, please redirect me to the right place as I did not find any mention to that in the readme.
Is it possible for the Operator to support deployment of prometheus alert manager? This would be much more convenient instead of deploying it as standalone instance. Would you consider PRs open in that direction?

Already in flight! :-)
See https://github.com/GoogleCloudPlatform/prometheus-engine/pull/273

Amazing thanks!


Difficulty producing convincing results.

I got this result with:
python3 main.py --description "Blue Hair" --mode=edit

Do I need to change something in the invocation to get better results?

Hi @stolk ,
Thanks for your interest in our project ☺
As written in the paper, using optimization to edit images can be a bit tricky sometimes. Moreover, in the current version in the repo, the identity loss which can be helpful to obtain better results is not implemented.
We plan to update the optimization code, and upload the code of the mapper and the global directions (which are more stable) within the next few days. The waiting will be worth 😉
Until then, I would try to lower a bit '--l2_lambda'.

The other thing @stolk stolk is that FFHQ model has limited dataset diversity, both in terms of human diversity (less balance, hence when I found C-list movie stars yesterday with CLIP who are black/multiracial, often they look to be whiter in the retrieved latent) meaning that it's harder to find in the latents what the GAN has limited training data for (e.g. hence giving you that little slab of blue background).  By getting a broader training set and extending FFHQ in stylegan2 ADA and then using an improved model with StyleCLIP much, much more is possible.  (But obviously much more work unless you have one someone is willing to share with you an extended model.) Without going that far you can also try engineering the description phrase to emphasize the desired result (see original clip project on prompt engineering), but that's hit or miss.
I think I saw similar problem trying purple hair yesterday...e.g. got a purple blotch on the face...versus intent.  Haven't tried any hyperparameter tuning, so helpful tip, thanks @orpatashnik. Other CLIP+GAN projects I've seen, try to first select the closest matching start seed to a prompt before beginning to transverse the latents. However, adds to search time. I have created conditional GANs that give extra levers to control output as well, although I haven't seen as much of that in CLIP+GAN projects since theoretically it could be redundant (look at WikiArt if you want an example of how to adjust stylegan2 - great thing about conditioning is it helps to mix style categories and get closer in the latent neighborhood you're looking for.). cheers!

Until then, I would try to lower a bit --l2_lambda.

Thanks, @orpatashnik, with --l2_lambda=0.004 the results are much better!

@ModMorph - Thanks for your comprehensive comment. I agree that StyleGAN can limit the expressiveness of our method. However, we found that using the driving text it is possible to generate images that are not so common when generating an arbitrary image using StyleGAN. Some examples are the purple hair, and the unique expression shown in the paper.
@stolk Did you try our new GUI? 😎

Still wrestling with conda.
UnsatisfiableError: The following specifications were found
to be incompatible with the existing python installation in your
environment:
Specifications:

tensorflow-gpu==1.15 -> python[version='2.7.|3.6.|3.7.*']

Your python: python=3.8
On Sat, Apr 3, 2021 at 8:19 AM orpatashnik @.***> wrote:

@ModMorph https://github.com/ModMorph - Thanks for your comprehensive
comment. I agree that StyleGAN can limit the expressiveness of our method.
However, we found that using the driving text it is possible to generate
images that are not so common when generating an arbitrary image using
StyleGAN. Some examples are the purple hair, and the unique expression
shown in the paper.
@stolk https://github.com/stolk Did you try our new GUI? 😎
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/orpatashnik/StyleCLIP/issues/15#issuecomment-812879395,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADPE4CPBNIZRGF4KZQV64DTG4WYDANCNFSM42INY5KA
.

--
Owner/Director of Game Studio Abraham Stolk Inc.
Vancouver BC, Canada
@.***

Seems like your environment is python 3.8. Please try to create an environment with python 3.7

Can I close the issue? 😊

Thanks for the tips.


fix: fix instance type selection that does not match terway cni

What type of PR is this?
/kind bug

What this PR does / why we need it:
fix: fix instance type selection that does not match terway cni
Which issue(s) this PR fixes:

Fixes #none
Special notes for your reviewer:
https://help.aliyun.com/zh/ack/ack-managed-and-ack-dedicated/user-guide/container-network/?spm=a2c4g.11186623.help-menu-85222.d_2_4_3.6d501109uQI315&scm=20140722.H_195424._.OR_help-V_1
Does this PR introduce a user-facing change?

/lgtm


What should we use?

If we are not allowed to use this, then what should we use to get this information?

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

ID: 449e4700-7021-65da-8afa-29c6becc6363
Version Independent ID: 28344d71-84c1-96cb-636c-7996b62ac97b
Content: IOCTL_INTERNAL_USB_GET_PARENT_HUB_INFO
Content Source: wdk-ddi-src/content/usbioctl/ni-usbioctl-ioctl_internal_usb_get_parent_hub_info.md
Product: windows-hardware
GitHub Login: @windows-driver-content
Microsoft Alias: windowsdriverdev

Hi @BrianCatlin,
I’ve created an internal work item for myself to investigate this with the USB team, and I’ll make a note here when I complete the changes. In the meantime, I’m closing the GitHub issue while it’s tracked in the bug. Thanks again for helping to improve the docs.


Generate pod for scrutinize

This adds the logic to generate a dummy pod when a VSCR CR is created.
It does the following:

checks if the VerticaDB specified in Vscr exists
checks the server version and abort reconciliation if version is too old
creates a pod

For now the pod does not run scrutinize, the logic for that will be added on a follow-up PR. The pod composed of 2 containers, an init container (that will collect scrutinize container later) and a main container. Both run hard-coded image (busybox) and respectively sleep for 5s and for infinity.
We also set some fields(initContainers, Volume,...) with values from Vscr.

Can we have some e2e tests for the changes you made here?

Reading for another round.

I added the new status condition ScrutinizeReady. The lopgic(for now): If during the first reconciliation iteration vdb is not found/vdb does not have server version/vclusterops is disabled/version does not support scrutinize with vclusterops, ScutinizeReady is set to false and we won't ever try again. Otherwise it is set to true, we proceed and create the pod.
The idea I have about VerticaScrutinize is that it is a one time thing, tell me if I am wrong.


Dates get truncated inappropriately

Why does the library truncate dates? This is causing problems comparing timestamps we have that get generated in our back-end:
ie: "2016-04-22T13:45:53.1088331+00:00".toDate( .ISO8601Format( .Extended ))!
==> "2016-04-22 13:45:53 +0000"

Even if you pass in a custom format with the full precision it still truncates the last 4 digits:
ie: DateInRegion( absoluteTime: "2016-04-22T13:45:53.1088331+00:00".toDate( .Custom( "yyyy-MM-dd'T'HH:mm:ss.SSSSSSSZZZZZ" ))).toString( .Custom( "yyyy-MM-dd'T'HH:mm:ss.SSSSSSSZZZZZ" ))!
==> "2016-04-22T09:45:53.1080000-04:00"

Thanks for your report!

Using S....prints fractional seconds (truncated to the count of letters)
I think you should use A....to print Milliseconds in day up to 6 digits.
https://github.com/malcommac/SwiftDate/blob/develop/Documentation/1.CreateDates.md


Accidentally deleted Oceananigans compat

I thought that the compat change was on purpose, in the sense that code didn’t work now with pre-v0.89.2 versions of Oceananigans. Wasn’t it?


Problem running dsr_control_node2

Hi,
Every time I try to run the dsr_control_node2 with the launch files, I get this error :
I used ros2 launch dsr_control2 dsr_control2.launch.py, instead of the rviz launch to have only this error.
[INFO] [launch]: All log files can be found below /home/jtaveau/.ros/log/2021-06-22-11-03-49-470003-saluyt-247231
[INFO] [launch]: Default logging verbosity is set to INFO
/home/jtaveau/moveit2_ws/install/dsr_control2/share/dsr_control2/config/default.yaml
[INFO] [dsr_control_node2-1]: process started with pid [247240]
[dsr_control_node2-1] [INFO] [1624352629.578406090] [dsr_control_node2]: g_node = 0x0x7fff587a7410
[dsr_control_node2-1] [INFO] [1624352629.578591177] [dsr_control_node2]: rate is 100
[dsr_control_node2-1] [INFO] [1624352629.578703692] [dsr_control_node2]: controller_manager is updating!
[dsr_control_node2-1] [INFO] [1624352629.578949447] [dsr_hw_interface2]: name = dsr01
[dsr_control_node2-1] [INFO] [1624352629.578970939] [dsr_hw_interface2]: model = m1013
[dsr_control_node2-1] [INFO] [1624352629.578976680] [dsr_hw_interface2]: gripper = none
[dsr_control_node2-1] [INFO] [1624352629.578982193] [dsr_hw_interface2]: name_space is dsr01, m1013
[dsr_control_node2-1] [INFO] [1624352629.578986618] [dsr_hw_interface2]: constructed
[dsr_control_node2-1] [INFO] [1624352629.648201131] [dsr_hw_interface2]: [dsr_hw_interface2] init() ==> setup callback fucntion
[dsr_control_node2-1] [INFO] [1624352629.648250886] [dsr_hw_interface2]: joint_name = joint1
[dsr_control_node2-1] [INFO] [1624352629.648379565] [dsr_hw_interface2]: joint_name = joint2
[dsr_control_node2-1] [INFO] [1624352629.648409293] [dsr_hw_interface2]: joint_name = joint3
[dsr_control_node2-1] [INFO] [1624352629.648426622] [dsr_hw_interface2]: joint_name = joint4
[dsr_control_node2-1] [INFO] [1624352629.649017060] [dsr_hw_interface2]: joint_name = joint5
[dsr_control_node2-1] [INFO] [1624352629.649036958] [dsr_hw_interface2]: joint_name = joint6
[dsr_control_node2-1] [INFO] [1624352629.649049081] [dsr_hw_interface2]: INIT@@@@@@@@@@@@@@@@@@@@@@@@@
[dsr_control_node2-1] [INFO] [1624352629.649058139] [dsr_hw_interface2]: init() ==> arm is standby
[dsr_control_node2-1] [INFO] [1624352629.649122038] [dsr_hw_interface2]: host = <IP_ADDRESS>
[dsr_control_node2-1] [INFO] [1624352629.649129870] [dsr_hw_interface2]: port = 12345
[dsr_control_node2-1] [INFO] [1624352629.649134501] [dsr_hw_interface2]: command = 1
[dsr_control_node2-1] [INFO] [1624352629.649138682] [dsr_hw_interface2]: mode = virtual
[dsr_control_node2-1] [INFO] [1624352629.649142855] [dsr_hw_interface2]: host <IP_ADDRESS>, port=12345 bCommand: 1, mode: virtual
[dsr_control_node2-1] joint_name = joint1joint_name = joint2joint_name = joint3joint_name = joint4joint_name = joint5joint_name = joint6Connection refused
[dsr_control_node2-1] [ERROR] [1624352629.649930486] [dsr_hw_interface2]: DRHWInterface::init() DRCF connecting ERROR!!!
[dsr_control_node2-1] [ERROR] [1624352629.649953211] [dsr_control_node2]: Error initializing robot
[dsr_control_node2-1] [INFO] [1624352629.649986810] [dsr_hw_interface2]: DRHWInterface::~DRHWInterface()
[ERROR] [dsr_control_node2-1]: process has died [pid 247240, exit code -11, cmd '/home/jtaveau/moveit2_ws/install/dsr_control2/lib/dsr_control2/dsr_control_node2 --ros-args --params-file /tmp/launch_params_sw2nerns --params-file /tmp/launch_params_d_fnv78q --params-file /tmp/launch_params_tzx9c1k8 --params-file /tmp/launch_params_5imt5lon --params-file /tmp/launch_params_n9gdnyt7 --params-file /tmp/launch_params_5f_3wwyg --params-file /tmp/launch_params_q4vvj7g3 --params-file /tmp/launch_params_0twxt0ha --params-file /tmp/launch_params_b9kz3rvs --params-file /tmp/launch_params_v0x1_gxz'].
Could you help me, please ?
I'm using Foxy on Debian Bullseye.

Did the emulator run normally?
Try running the DRCF64 DRCF binary file in common2/bin/DRCF.
If it runs normally, install the lib-pocodev package and run it again.

When I run it, I get :
(env38) jtaveau@saluyt:~/doosan_ws$ ./src/doosan-robot2/common2/bin/DRCF/DRCF64

    Emulator of Doosan Robot Controller [ver M2.52]
              [service port = 12345]
              [robot model = m1013(config/config.xml)]

Does it mean it runs normally ?
I have already installed libpoco-dev.


Revamp Github Action settings for CI

Why are the changes needed?

Add missing asf header
Disable fail-fast to let jobs run individually
Group defaults maven builds with java & spark version matrix

so this PR brings fully tests with JDK 11 back

Group others(now cross-version check only) in include
replace actions/cache@v2 with actions/setup-java@v2 - cache: 'maven'

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Need to pass all CI jobs

Codecov Report

Merging #1947 (4ca4f23) into master (354bf83) will increase coverage by 0.07%.
The diff coverage is n/a.

:exclamation: Current head 4ca4f23 differs from pull request most recent head 20eeb7b. Consider uploading reports for the commit 20eeb7b to get more accurate results

@@             Coverage Diff              @@
##             master    #1947      +/-   ##
============================================
+ Coverage     60.89%   60.96%   +0.07%
  Complexity       69       69
============================================
  Files           302      302
  Lines         14785    14786       +1
  Branches       1914     1915       +1
============================================
+ Hits           9004     9015      +11
+ Misses         5011     5007       -4
+ Partials        770      764       -6

Impacted Files
Coverage Δ

.../org/apache/kyuubi/jdbc/hive/KyuubiConnection.java
4.28% <0.00%> (-0.01%)
:arrow_down:

...he/kyuubi/engine/spark/shim/CatalogShim_v2_4.scala
77.77% <0.00%> (+1.58%)
:arrow_up:

...n/scala/org/apache/kyuubi/engine/ProcBuilder.scala
85.32% <0.00%> (+1.83%)
:arrow_up:

...he/kyuubi/engine/spark/shim/SparkCatalogShim.scala
80.43% <0.00%> (+2.17%)
:arrow_up:

...rg/apache/kyuubi/engine/trino/TrinoStatement.scala
69.87% <0.00%> (+2.40%)
:arrow_up:

...ache/kyuubi/operation/KyuubiOperationManager.scala
95.00% <0.00%> (+2.50%)
:arrow_up:

...ache/kyuubi/engine/spark/schema/SchemaHelper.scala
98.79% <0.00%> (+4.81%)
:arrow_up:

Continue to review full report at Codecov.

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

thanks @pan3793 @ulysses-you @cfmcgrady for the help, merged to master


Calculate the pitch

How is the pitch calculated? Does anyone know the detailed process?

The pitch detector is similar to (not exactly the same as) the one described in https://www.ee.columbia.edu/~dpwe/papers/Talkin95-rapt.pdf

The pitch detector is similar to (not exactly the same as) the one described in https://www.ee.columbia.edu/~dpwe/papers/Talkin95-rapt.pdf

Thanks!

The pitch detector is similar to (not exactly the same as) the one described in https://www.ee.columbia.edu/~dpwe/papers/Talkin95-rapt.pdf

If the pitch detector gives out the wrong value, could LPCnet fix it ？or,  mitigate the adverse impact ？


Unable to download audiobooks

Expected behavior
As of october 2024 I was able to download audiobooks until november. I also tried the Ubuntu release and pip(x) install.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: Windows 10
Install method and python version: .exe from releases tab
Kobodl Version: 0.9.4

Additional context
"
Downloading fdc1a2d0-b13f-4da6-bcc3-af25044f3d6d to D:\Kobo3\Julie Caplin - De kleine theewinkel in Tokio
Traceback (most recent call last):
File "kobodl_main_.py", line 5, in
File "click\core.py", line 829, in call
File "click\core.py", line 782, in main
File "click\core.py", line 1259, in invoke
File "click\core.py", line 1259, in invoke
File "click\core.py", line 1066, in invoke
File "click\core.py", line 610, in invoke
File "click\decorators.py", line 33, in new_func
File "kobodl\commands\book.py", line 80, in get
File "kobodl\actions.py", line 231, in GetBookOrBooks
File "kobodl\kobo.py", line 345, in Download
File "kobodl\kobo.py", line 276, in __DownloadAudiobook
File "requests\models.py", line 1024, in raise_for_status
requests.exceptions.HTTPError: 403 Client Error: Forbidden for url: https://storedownloads.kobo.com/download?downloadToken=eyJ0eXAiOjIsInZlciI6IjEiLCJwdHlwIjoiQ2xpZW50QVBJRG93bmxvYWRUb2tlbiJ9.eKbJS0haCKyiqVxKq-Uc4Q.YO38PZtwqePRX-HdURiKl1EqfyasKxOrFCEZZwTkyXJH8ABy6P6LNZKVugMtnDjOwcU9tdh1yiXi75s5KOlRvMYNpS1fw8drozkcAlb8moVAW7ugQa-VqEvZcWOU-HFIXPbnr_ZNW7XV6AHOO_2iQybqsVPvJGqqb71gNAP1vyPfSqDPX96J9GznOxhOCLmHxEtlHl4Dv2iAjL8jQeG10GBNYNE8ClZovGMPOHpJHzsXHW59sgCiUEJuEWAOaVeMBlJfm-H2Rb1vGNu3QaHJuooEb1Zxk_JfHlILaOQHohJVQG9Tz-IBcDgNzcfT673JYK1pTtooXn9_aH8aS-I74ybD3snoLIcH2d_UBLBE3zMvTA1PFGxWY5HhQNbpGA2sLgJVY7JgZZc7gT_I-rcd3OSHpycHk-QnuPUnCaL_aeqKsGQkOvTazEQVEYI8ImPy6m2tfSMhXdauU8lcdPozEQEAQQqRx9JMO3ElXUNAOIKcRJk_CDHyIEHgnLCKALR1p9peL0KFhUR1tZRYaU4MWayylawcoI_SifNFKOd3R-4vQnoIMaXbng-co62IUfSvt2YJf8i8imOlBkx47RL_LSY4rOt2FELHXWKdKZxMVRti1W7HQbfBybYuxf-gA0ra7bcLMniu4hLg8QxS3PaTjKkEOsA13fmz_3UzWdLumgSsNMR-y7_6_wprfPILwxQwsX5qrcgmW1i_qiTEhmXsLA
[PYI-26336:ERROR] Failed to execute script 'main' due to unhandled exception!
"

Sorry, just found out it is known
(https://github.com/subdavis/kobo-book-downloader/pull/124)


feat: collect external_service_op perf object

Ready to review

Nice job @Hardikl - since the object collector is disabled by default, this dashboard needs a row/note saying it needs to be enabled similar to Workload dashboards

Couple of issues were identified as many more labels were included and started with plugin go way, then identified which labels would be sufficient for this. Then tried to solve same by moving them to template LabelAgent plugin, but these external service ops are across cluster, so just svm, service_name and operation won't be giving the exact result.
That's where I need to put changes in plugin go only with unique key as: datacenter, cluster, svm, service_name, operation.
Not a clean way of doing this, but this is what I found.


How to label obb data sets

Semantic Segmentation with Polygons？

I would think Object Detection with Bounding Boxes would be a more natural fit, but keep in mind these are just templates and you can customize the label config to meet your needs. Take a look at https://labelstud.io/templates/image_bbox also!


ProtectedVolumeReplicationGroupList API overwrites VRG metadata in case of failover

For fusion, in case of failover, VRG gets created on surviving site with label 'metrodr.isf.ibm.com/status=relocated'.Observed that after failed site is up and running again, VRG metadata having this 'relocated' label is being overwritten with VRG metadata stored on earlier failed site..Due to this, relocated label which is part of VRG metadata is not preserved leading to issues in sync mechanism of fusion.

@pdumbre Is the KubeObjectProtection feature enabled by the presence of said field in VRG.Spec ?

As far as I know, ProtectedVolumeReplicationGroupList controller only writes to s3 store not k8s API server.  However, I suppose it is possible fusion controller is reading from it and writing read VRG Kube object labels to k8s API server.  Is the failover "from" site's VRG fenced from writing to s3 store(s) prior to failover?

Closing ShioRamen issues.  Please reopen or if issue persists.


QUIC not working?

Any idea why cronet is using TCP even though I've defined to use quic in HttpClient constructor?
Tested on Windows 10 Home 21h1

any update on this issue?


Checked the other player housing dimensions, the above three appear to be the only houses that have an under ground room.

Yeah, it's a product of the import process of the house, as long as the houses are usable at this point it is not a priority, though I will note it for fixing at some point.

Fixed the first one, though decided to leave the other two, if there are gifts, you must work for them :D


Add target success 2

Coverage decreased (-1.02%) to 98.98% when pulling 8a6085268de5ee84a40e0117a36b48ac595707ca on add-target-success-2 into 4a3416b78f32dd29a02e6f07ab51a29f2774231a on master.

Coverage remained the same at 100.0% when pulling be37da5c990f2f458d19cd7fe7d135b65fb83c29 on add-target-success-2 into 4a3416b78f32dd29a02e6f07ab51a29f2774231a on master.


[RFE] local-gadget: Create integration tests

Current situation
There are no integration-tests for the local-gadget. Once we agree on how to implement the single call run https://github.com/kinvolk/inspektor-gadget/pull/730, we should start implementing the tests for them.
As said in https://github.com/kinvolk/inspektor-gadget/issues/719, these tests should test the JSON format and the transformation from JSON to columns will be done by unit-testing the GadgetParsers' method, e.g. for the socket gadget.

There are no integration tests, but there are tests for various gadgets in local-gadget with docker containers:
make local-gadget-tests

I wonder how to avoid test duplication. I think the tests in make local-gadget-tests should be kept because they run fast during development of new gadgets. Integration tests will be slower.


Generate load scripts on install abidding to new paket.dependencies option

new option generate_load_scripts: on|off|true|false in paket.dependencies
this option, if on/true, calls the loading script generation
Note that this does it in very eager way (if defined for any group, it will generate all), it also doesn't allow to specify framework or language like from command line, keeping it simple for now.
The option is off by default, because there is performance penality and it generates lots of small script files, there is also no proper logic for "garbage collection" process.
Implementation notes:

A new convenience method is also exposed on PublicAPI: Dependencies.GenerateIncludeScripts()
I had to migrate a good deal of stuff in a module I named Queries, mostly stuff which were raw in Program.fs
added a unit test for parser, testing all cases of the new option
added an integration checking calling install with option on results in script being generated without further invocation
Since both .dependencies and .lock files are used often in conjunction, I made a PaketFiles DU (in Queries module)
Since both group name and package name are used often in conjunction, I made a QualifiedPackageName single case DU, which helped me remove a bit of cruft related to default group

@forki (or other maintainers) I think few people were looking to have this available, could you have a quick look, I think the tests are conclusive enough for this to get merged unless you have code review comments?

@forki this is finally passing CI

very cool. /cc @dsyme
@smoothdeveloper can we have some docs as well?

@forki good point, I'll make a doc PR soon.


Hardcoded memory reset in CuDnn GRU implementation

Need to find some workaround.

@olegtarasov Testing Telegram comments.


Firewall: manage egress rules per interface

Currently the direction of the traffic can only be chosen in floating rules, but in some scenario's it's much easier to create outbound rules (only inbound is supported now).
When using a lot of interfaces, which should all be allowed to access devices on one specific interface, this would save quite some rules and is easier to track for the administrator.
This FR should add direction as on option and while already changing these pages, also allow to create "non quick" rules on interfaces.
Functionally the "regular" rules would be more aligned with the "floating" rules as we have now, with the exception that you can't add multiple interfaces in a normal rule due to the inability to reorder a single rule in multiple rulesets (rules are positional).
Considerations while adding this:

Validate if "policy based routing" is used for outbound rules, since this is not compatible
Mark outbound rules in the overview grid
Mark non-quick rules in the overview grid

So for a LAN rule you have a dropdown for outgoing interface and rule logic is still inbound? Outbound should be multi select for multi wan setups. Just a quick thought

@mimugmail no, this is just about the in/out part of the rule, pf doesn't support in / out interfaces per rule (as ipfw does)
Specifically, these will become available in per interface rules:

I would like to see in/out, but not any for the same policy routing fail reason. People won't understand this worst case.

This issue is based in the wishes from wowas in IRC, correct?

no, one of our clients. For traceability I made sure there's an issue to refer in future commits.

note to self: missed a rule in our overview while debugging this (https://github.com/opnsense/core/commit/f8d5c01a6f4ab2ef307324acd4711cd76e005827)


Verify macOS Binaries with spctl

I followed instructions on Verify Binaries page and it does not work for the latest release I downloaded.  I am running macOS Mojave version 10.14.6.
I entered command like this
spctl -a -t open --context context:primary-signature -v ~/myapps/IBM_Cloud_CLI_1.2.0_macos.tgz
And it comes back with
/Users/ricks/myapps/IBM_Cloud_CLI_1.2.0_macos.tgz: rejected
source=no usable signature

The tgz seems fine. I was able to expand it and use it.

@salmoidforwork .tgz is not a binary file. You need to de-compress this file, and run spctl against the ibmcloud binary.

@salmoidforwork .tgz is not a binary file. You need to de-compress this file, and run spctl against the ibmcloud binary.


How to create Recovery Service

Hi @indumathyn
Please be more specific in your issue. Thank you.

#please-close
Since we didn't get more specific information.

Working for me
Thanks


android: use row stride for Java impl too

Because the row stride (of the Y plane) is (on some devices) greater than the image width, I was not able to scan barcodes with the Java implementation on some of my test devices.

Thanks! (I only ever tested the code on a Pixel 3...)

Thanks again for merging 😄
Unfortunately, there are issues on Android that only occur on some devices but not on others. Part of the game, I guess 😉


General, unfiltered feedback

I took a quick glance at Sustainers.sol and took a few notes – I figured this was the easiest way to leave some feedback, which I time-boxed to ~15 minutes.
Worth noting I lack (almost) a complete understanding of smart contracts, particularly at the implementation level, so my comments are decently general.
Nomenclature
Some of naming in the codebase is a bit confusing, but not necessarily unbearable. Thoughts to follow.
 As I see it, this is the top-level, all-encompassing abstraction, which holds Purposes, Sustainers, Stewards, etc. Maybe it's too constricting, but I think something along the lines of "Market" or "Marketplace" feels a bit more apt here.
Steward
Maybe I get too hung up on naming, but I feel like there's something to be said about shooting for an elementary reading level when it comes to naming conventions – particularly when you're engaging with people that don't necessarily speak english. "Steward" has always been a weird word to me; I guess that's because it's pretty abstract and can mean many different things. Why not call this "creator", "owner", "admin", etc.?
I feel like these two changes could offer substantially better readability in the codebase.
Naming nits

"withdrawl" -> "withdraw", or "withdrawal"
Purpose.want – is "want" typical nomenclature for a token? If not, why not just "funding_token"?

Structure
One-to-one mappings / overall design
It seems like there are a few mappings that are currently one-to-one that could easily be one-to-many, namely address => Purpose.  I see that there is also a notion of nextPurposes, which is admittedly a bit confusing. Is there a reason that people are only allowed a single active purpose and a single next one?
I guess this leads to a larger question about the design of function sustain(address _steward, uint256 _amount). It seems the general principle here is that you're sustaining an individual, rather than a purpose. From a product perspective - is this a 'product'? – it's a bit misleading, but perhaps it's intentional. In the conventional fundraising world, funding is often directed towards the purpose itself and the stewards come secondary.
Perhaps my confusion comes from a misunderstanding of blockchain, but would it be at all possible to decouple the receiving address of the purpose from the steward itself? I.e., is it possible to generate a new address solely for each purpose, upon creation? This could offer more extensibility down the line if multiple stewards become owners of a purpose and have access to its funds.
I don't know, I'm rambling. Maybe this isn't the best forum for these kinds of questions.
Duplicated data
I see that currently sustained amount sustainment can essentially be derived from netSustainments. It may be worth extracting the former into a helper function to only hold one source of data for the currently sustained amount. If those get out of sync somehow, things could get ugly.

Okay, that's 15 minutes. I didn't really get through most of it, but let's catch up more on it soon. Great work, the idea here is really something unique and powerful.

thanks!


Support for conjunctive and adjunctive filters in TS.MRANGE?

When I pass just the filters id=(id1,id2) it works, and it also works if I pass one id with the other attributes, e.g.: id=id1 type=Type prop=Prop.  It doesn't seem like I can pass a combination of AND filters (type, prop) and OR (id1, id2)
A log of command.toWritable looks like:
command.toWritable(),  *5
$9
TS.MRANGE
$10
1676671745
$10
1676650507
$6
FILTER
$129
type=Type id=(id1, id2) prop=Prop

For what it's worth, I don't encounter this issue with TS.QUERYINDEX -- I can pass the conjunctive id filters and the adjunctive prop, type filters. command.ToWritable for that looks like:
command.toWritable(),  *4
$13
TS.QUERYINDEX
$9
prop=Prop
$102
id=(id1, id2)
$26
type=Type

If this functionality is not currently supported, would I be able to run the query I need with .sendCommand? Thank you

Maybe there is an issue parsing the parentheses? When I try TS.MRANGE<PHONE_NUMBER>000<PHONE_NUMBER>000 FILTER id=(id1) type=Type prop=Prop in CLI it works but this too doesn't seem to work through ioredis

I should mention -- I am using a library called redis-modules-sdk to issue MRANGE. The function uses ioredis.send_command(command, args), where command is TS.MRANGE and args is ['1676671745',   '1676650507',   'FILTER',   'type=Type id=(id1,id2) prop=Prop']

I believe the issue is with redis-modules-sdk not ioredis, apologies


Date picker(Calendar) is not open in webview

[x] I have read the Get Started - Installation section
[x] I have read and done the Get Started - Setup Android section
[x] I have read and done the Get Started - Setup iOS section
[x] I have already searched for the same problem

Environment

Technology
Version

Flutter version
2.5.1

Plugin version
5.3.2

Android version
31

iOS version
working

Xcode version
NA

Device information:
Description
Expected behavior:
The calendar should be open to pic date in webview.
Current behavior:
Currently, nothing happens on clicking of input button on webview.
Steps to reproduce

Open webview
Then there is a button from which we tap to select the date on the webview
Then nothing happens

Images
https://user-images.githubusercontent.com/12988908/168590911-68b2dabf-7daf-491c-9165-eb66debdfaeb.mp4
Stacktrace/Logcat
final InAppWebViewGroupOptions options = InAppWebViewGroupOptions(crossPlatform: InAppWebViewOptions(useShouldOverrideUrlLoading: true, supportZoom: false,javaScriptCanOpenWindowsAutomatically: true,javaScriptEnabled: true),   android: AndroidInAppWebViewOptions(     supportMultipleWindows: true,    useHybridComposition: true// crash If am I ussing this key   ));
IndexedStack(index: _position, children: <Widget>[                   Padding(                       padding: EdgeInsets.only(top: 56),                       child: InAppWebView(                         initialOptions: options,                         onWebViewCreated: (InAppWebViewController controller) {                           setCookie();                           controller.loadUrl(                               urlRequest:                                   URLRequest(url: Uri.parse(widget._url)));                         },                         shouldOverrideUrlLoading: navigationDelegate,                         onLoadStart: showLoader,                         onLoadStop: hideLoader,                         androidOnPermissionRequest: (controller, origin, resources) async {                           return PermissionRequestResponse(                               resources: resources,                               action: PermissionRequestResponseAction.GRANT);                         },                       ),                     ),                     Center(                         child: SizedBox(                       height: 98,                       width: 98,                       child: Image.asset("assets/images/loader.gif"),                     )),                   ])

I have the same issue


Add Simple Realism from Kerbal Stuff

This pull request was automatically generated by Kerbal Stuff on behalf of Krakenfour, to add Simple Realism to CKAN.
Please direct questions about this pull request to Krakenfour.

@plague006 i'd like to help with these. How do you annotate when you've contacted the person on kerbalstuff? Do you add the inprogress tag?

I usually just leave it in these PR comments. Ex. a link to a post I made in a mod's KSP forum thread and a description of what information I need(ed).

Perfect. Thanks @plague006

Closed by https://github.com/KSP-CKAN/NetKAN/pull/2465


Implement datachannel close

Works most of the time, but I've had instances where it doesn't seem to fire the event corresponding to SCTP_STREAM_RESET_OUTGOING_SSN flag and thus the final "on close" callback wouldn't be fired.

Requires #30 and #32 to be merged first in order to compile this error free.


Support negative numbers in Lucene40 codec

Lucene40 codec picks different methods of storing numeric values based on what data type is large enough to represent them. The issue was with the numbers that can fit in sbyte, they were being returned as byte due to our migration. Negative numbers -128 or greater were being returned as positive numbers.
This passes all remaining failing Lucene40 codec tests. Here is one example:
http://teamcity.codebetter.com/viewLog.html?buildId=180212&tab=buildResultsDiv&buildTypeId=LuceneNet_Core#testNameId-8934306368705483624
Test expects to get back -10 but is getting back 246 instead.

There's some more such places, I'm on it

FYI Lucene.Net.Index.BaseDocValuesFormatTestCase.TestTwoDocumentsMerged() still fails

@synhershko are you sure you are running fully compiled after merging my changes? In TC builds and several local builds that test case passes as expected. Make sure you did a full rebuild if you ran it locally after merging my changes, that has gotten me before.
The sbyte that you added will not make a difference because the header value is either 0 or 1 value, works in byte and sbyte format.

Yes, works now


Usability model

a notebook with logistic regression exploring devices, browser, and language with respect to the duration of the visit

thanks for the contribution 🙏

Yeah I tried to solved those conflicts in this MR.


Add configuration for readthedocs.org

Issue is #5.
Normally the defaults of readthedocs.org's doc builders (Python 3.7 and Sphinx) should work, but in order to be able to build mflib API docs, we need a little more than that: we need Python 3.9, and some extra packages (furo, myst-parser).  This configuration should take care of that.

Status of this PR:

I've created https://readthedocs.org/projects/fabrictestbed-mflib/. The docs, when they are successfully built, should appear at https://fabrictestbed-mflib.readthedocs.io/en/latest/.
The docs are currently being built (or: attempts are being made to build the docs) off my fork, off this specific PR branch.  (I'm using a fork because I can't push to this repo.  It is not strictly necessary that I should be able to push to this repo, I think.)
Currently there's an error when building the docs at RTD, because networkx and networkx-query (they are fim dependencies) do not like each other:

Installing collected packages: snowballstemmer, pytz, mock, commonmark, charset-normalizer, zipp, urllib3, sphinxcontrib-serializinghtml, sphinxcontrib-qthelp, sphinxcontrib-jsmath, sphinxcontrib-jquery, sphinxcontrib-htmlhelp, sphinxcontrib-devhelp, sphinxcontrib-applehelp, Pygments, pillow, packaging, MarkupSafe, imagesize, idna, docutils, certifi, babel, alabaster, requests, Jinja2, importlib-metadata, sphinx, readthedocs-sphinx-ext, sphinx-rtd-theme, recommonmark
Successfully installed Jinja2-3.1.2 MarkupSafe-2.1.2 Pygments-2.14.0 alabaster-0.7.13 babel-2.11.0 certifi-2022.12.7 charset-normalizer-3.0.1 commonmark-0.9.1 docutils-0.18.1 idna-3.4 imagesize-1.4.1 importlib-metadata-6.0.0 mock-1.0.1 packaging-23.0 pillow-9.4.0 pytz-2022.7.1 readthedocs-sphinx-ext-2.2.0 recommonmark-0.5.0 requests-2.28.2 snowballstemmer-2.2.0 sphinx-6.1.3 sphinx-rtd-theme-1.2.0 sphinxcontrib-applehelp-1.0.4 sphinxcontrib-devhelp-1.0.2 sphinxcontrib-htmlhelp-2.0.1 sphinxcontrib-jquery-2.0.0 sphinxcontrib-jsmath-1.0.1 sphinxcontrib-qthelp-1.0.3 sphinxcontrib-serializinghtml-1.1.5 urllib3-1.26.14 zipp-3.13.0

[rtd-command-info] start-time: 2023-02-13T17:00:50.096804Z, end-time: 2023-02-13T17:01:58.571072Z, duration: 68, exit-code: 1
python -m pip install --upgrade --upgrade-strategy eager --no-cache-dir .[docs]
Processing /home/docs/checkouts/readthedocs.org/user_builds/fabrictestbed-mflib/checkouts/latest
  Preparing metadata (setup.py): started
  Preparing metadata (setup.py): still running...
  Preparing metadata (setup.py): finished with status 'error'
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [14 lines of output]
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "/home/docs/checkouts/readthedocs.org/user_builds/fabrictestbed-mflib/checkouts/latest/setup.py", line 33, in <module>
          setup(
        File "/home/docs/checkouts/readthedocs.org/user_builds/fabrictestbed-mflib/envs/latest/lib/python3.9/site-packages/setuptools/__init__.py", line 152, in setup
          _install_setup_requires(attrs)
        File "/home/docs/checkouts/readthedocs.org/user_builds/fabrictestbed-mflib/envs/latest/lib/python3.9/site-packages/setuptools/__init__.py", line 147, in _install_setup_requires
          dist.fetch_build_eggs(dist.setup_requires)
        File "/home/docs/checkouts/readthedocs.org/user_builds/fabrictestbed-mflib/envs/latest/lib/python3.9/site-packages/setuptools/dist.py", line 806, in fetch_build_eggs
          resolved_dists = pkg_resources.working_set.resolve(
        File "/home/docs/checkouts/readthedocs.org/user_builds/fabrictestbed-mflib/envs/latest/lib/python3.9/site-packages/pkg_resources/__init__.py", line 777, in resolve
          raise VersionConflict(dist, req).with_context(dependent_req)
      pkg_resources.ContextualVersionConflict: (networkx 3.0 (/home/docs/checkouts/readthedocs.org/user_builds/fabrictestbed-mflib/checkouts/latest/.eggs/networkx-3.0-py3.9.egg), Requirement.parse('networkx<3.0,>=2.4'), {'networkx-query'})
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

Next steps:

Solve the above problem with building docs.  (It may need a fim release? I don't know for sure yet.)
Add @cscarp as a project maintainer at https://readthedocs.org/projects/fabrictestbed-mflib/.
When this PR is merged, switch repo at RTD to https://github.com/fabric-testbed/mflib, and default branch to main.
A webhook may be need to be set up once this PR is merged.

Turns out that the problem with dependencies, stated above, can be worked around by installing only the packages required to build docs.  Docs are now at https://fabrictestbed-mflib.readthedocs.io/en/latest/.

@cscarp Please review this and merge when ready!
After this PR is merged, there are a few more things needed to be done so that hosting docs on RTD can be considered "finished".  You should probably make an account on RTD (GitHub login will work) and join the project https://readthedocs.org/projects/fabrictestbed-mflib/, so that you can make the necessary changes.


【feat】kyuubi is need to be supported with spark.

https://github.com/apache/incubator-kyuubi

Hi,
We will have a look to the project and see if we can integrate it into our roadmap.

https://github.com/clemlabprojects/ambari/discussions/20

This project is really useful.
Thank you very much.


bash apollo.sh release_noproprietary

System information

OS Platform and Distribution Linux Ubuntu 14.04 without GPU
Apollo version 3.5

Steps to reproduce the issue:
I try to Build and Release your Docker Container. When I run this commend:
bash apollo.sh release_noproprietary I will confront the following error
Supporting materials (screenshots, command lines, code/script snippets):

Could you help me solve this problem?

@SJHBXShub do you have the necessary hardware to run Apollo?

I do not have hardware and GPU. I open a virtual machine on Google Cloud. Linux Ubuntu 14.04.
Here is my procedure:

install docker ( it is OK)
install LFT (It is OK)
bash docker/scripts/dev_start.sh( It is OK)
bash docker/scripts/dev_into.sh(It is OK)
bash apollo.sh release_noproprietary (It is Wrong!!!)
Do u think the procedure is right?

Hello, I follow the guide to run code. After I run commend as following:
install docker ( it is OK)
install LFT (It is OK)
$bash docker/scripts/dev_start.sh( It is OK)
$bash docker/scripts/dev_into.sh(It is OK)
$bash scripts/bootstrap.sh (It is OK)
Then type http://localhost:8888 on firefox. the image like this. Could you help me solve this problem. I really spend long time in solving this problem.

Can you check the response of the server? I.e. right click and choose "View page source". This looks like an error I had with git-lfs not being setup correctly.

@christian-lanius
hello, the response of server is ok. Could you tell me how to setup fit-lfs correctly?

I have solve this problem. The major problem is the firefox is not support WebGL. we can use the following link the check:
http://webglreport.com/?v=2


adding documentation details of function layer.get_weights(). Added missing word "biases".

layer.get_weights() returns list containing weights and biases of the layer. Documentation doesn't mention it explicitly. Though it is assumed, it should be stated explicitly for sake of clarity. For beginners it is confusing otherwise.

In Keras terminology, "weights" include both types of parameters, kernels/weights and biases. It's sufficiently explicit already.


Add on_existing_and_future option for grant resources where on_future is an option

Is your feature request related to a problem? Please describe.
For all grant resources that allow on_future, I would like some mechanism to guarantee that all existing objects (tables/views/etc.) will also be available to the users/roles listed in the grant resource. More details below.
When using a grant resource like snowflake_table_grant, I am able to specify on_future for the database or schema and assign that grant to a role or set of roles. To the best of my knowledge, the on_future parameter is only useful if I never change the role that has these grants.
Let's take a simple example, where I grant SELECT to all future tables in a new schema to "role_a" using the snowflake_table_grant.

The initial snowflake_table_grant specifies "role_a" and on_future = true. "role_a" can now SELECT all tables I create in this new schema
I create two tables - "table_1" and "table_2", which "role_a" can SELECT because of the on_future specified in the above resource.
I then adjust this snowflake_table_grant resource, and I replace "role_a" with "role_b".
"role_b" would not be able to SELECT on "table_1" and "table_2", because "role_b" has only been granted SELECT on future tables, not existing tables.

Describe the solution you'd like
It would be nice to have something like on_existing_and_future, which would conflict with on_future.
resource "snowflake_table_grant" "select" {
  privilege              = "SELECT"
  database_name          = "db"
  schema_name            = "schema"
  roles                  = ["role"]
  on_existing_and_future = true
  with_grant_option      = false
}

The above would ensure that a role change on the resource would not result in a loss of data accessibility for the new role.
One important concern of adding this feature is the resulting terraform id. I do not know the intricacies of this package, but I believe that the resulting terraform id for this grant resource is the combination of database_name|schema_name|table_name|privilege|on_future|with_grant_option. To implement this feature in such a way that it would not drastically affect the resulting terraform id, the |on_future| id slot - which currently can be nil, true, or false - could render as |oeaf| if on_existing_and_future = true.
Describe alternatives you've considered

In tandem with the snowflake_table_grant, use the snowflake_tables data source, issue per-table grants to the same role that is receiving the on_future grants. This would alleviate the issue but is does not scale desirably. This option all doesn't work when standing up infrastructure from scratch, so one would need to add this capability after the initial apply.
Slow roll schema ownership to the target role. First, create a brand new schema with the proper on_future grants and assign to the new role. Second, migrate the data from the existing schema into the new schema. Third, retire the old role/schema as needed. This alleviates the issue but requires much coordination and lacks a certain agility.

Additional context
I think this feature would really make grants work more desirably, and would prevent the need to slow roll toward role changes. This missing feature is the one reason I am apprehensive to terraform my Snowflake infrastructure.

yeah, at my previous job we forked this repo to build in that feature cause it was really usefull for migrating to terraform. I was not able to contribute it back but it was not hard to implement

@JohnCalhoun - any interest in pairing and PR'ing into the provider?

Is there any work being done on this?

Also very interested in this feature

I second this feature to be very useful. Not having this feature clashes a bit with the "desired state config" philosophy of terraform.

We are also very interested in this feature

Hey folks, the snowsql provider doesn't have the same guarantees as the snowflake provider but it does provide an escape hatch for the tricky to implement ALL FUTURE grants. Here is an example that you might find useful.

Also interested in this. It is a bit difficult because the resource can not ensure that all resources have the grant until it is executed again (unless the functionality is added in Snowflake itself, and is a configuration like the on future). Also, what happens with the resources that may not be managed by Terraform? Should it also be applied to those resources? I would say so 🤔.
In a module wrapping this provider, you could loop over all resources and generate a grant for each of them, reaching the same goal.


Delete Element By Dragging Off Page

Feature Description
As an editor I want items that I drag off the page and onto the workspace to be deleted
UX

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

When elements are dragged halfway off teh page ,the elements are all clipped in the workspace (see UX link)
Once elements are dragged off the work space and the user "let's go" the elements are deleted.

QA Instructions

FI: There's a duplicate: https://github.com/google/web-stories-wp/issues/445


Dragging over an iframe stops dragging when moving the mouse too fast

I found a solution which I'm sharing here.
I had something that looked like
...
return (
  <>

    <Draggable>

    </Draggable>
  </>
)

i.e. a draggable
 on top of an . When dragging too fast I would lose the drag.
My solution (inspired from this other post) was this:
...
const [isDragging, setIsDragging] = useState(false);

return (
  <>
    {isDragging && (
      <div style={{position: 'absolute', left: 0, right: 0, bottom: 0, top: 0, zIndex: 9999}}

    )}

    <Draggable
      onStart={() => setIsDragging(true)}
      onStop={() => setIsDragging(false)}
    >

    </Draggable>
  </>
)

This makes sure that when we are dragging, we are dragging on top of a big
 instead of the .

Amazing solution 👍
I had the same problem, thanks for sharing, it really helped!
I noticed that react-draggable library adds .react-draggable-transparent-selection class to body tag while an element is being dragged.
So here is an alternative solution, which is more like CSS approach and does not requires managing isDragging state.
.react-draggable-transparent-selection .draggable-iframe-cover {
    display: block;
    z-index: 999999;
}

.draggable-iframe-cover {
    position: absolute;
    display: none;
    top: 0;
    right: 0;
    bottom: 0;
    left: 0;
}

@Landish Nice! I switched to your simpler and cleaner solution! 👌


uv_cpu_info can crash the whole program instead of returning an error code

Version: 1.46.1-dev
Platform: Linux

This commit: https://github.com/libuv/libuv/commit/a939d643dd416e0705b56c0d551181859a56e4da has made it possible for
uv_cpu_info to abort instead of returning a proper error code. The documentation doesn't mention it, and I think it's not necessary here - it's not so critical error.

Check https://github.com/libuv/libuv/pull/4056#discussion_r1260158057 for reference why it was written this way. Specifically:

That aside, we're reading from a /proc pseudo-file where reads can't fail


One or more problems with zip files

Seems electrify is not downloading electron.
Setup project and export. The progress bar never moves when downloading electron. Have tried few different versions but always end up with error:
There were one or more errors with your zip/tar files. They were deleted. Please try export again.
Have tried few different project ending in same result. Not sure if Electrify issue or me screwing things up:)
Thanks Joey!

Cannot get working, seems links have changed to download electron. Will just download it and set things up manually to get project completed. You may want to check but then again may be just me spacing things out.


Laravel 5.1.* Update Problem

Hi everybody,
I'am using laravel 5.1.*. I was using Elasticquent in my project to this day.
But in today I run "composer update" and got this error.
Please Help Me.
Your requirements could not be resolved to an installable set of packages.
Problem 1
- Conclusion: remove laravel/framework v5.1.28
- Conclusion: don't install laravel/framework v5.1.28
- Conclusion: don't install laravel/framework v5.1.27
- Conclusion: don't install laravel/framework v5.1.26
- Conclusion: don't install laravel/framework v5.1.25
- Conclusion: don't install laravel/framework v5.1.24
- Conclusion: don't install laravel/framework v5.1.23
- Conclusion: don't install laravel/framework v5.1.22
- Conclusion: don't install laravel/framework v5.1.21
- Conclusion: don't install laravel/framework v5.1.20
- Conclusion: don't install laravel/framework v5.1.19
- Conclusion: don't install laravel/framework v5.1.18
- Conclusion: don't install laravel/framework v5.1.17
- Conclusion: don't install laravel/framework v5.1.16
- Conclusion: don't install laravel/framework v5.1.15
- Conclusion: don't install laravel/framework v5.1.14
- Conclusion: don't install laravel/framework v5.1.13
- Conclusion: don't install laravel/framework v5.1.12
- Conclusion: don't install laravel/framework v5.1.11
- Conclusion: don't install laravel/framework v5.1.10
- Conclusion: don't install laravel/framework v5.1.9
- Conclusion: don't install laravel/framework v5.1.8
- Conclusion: don't install laravel/framework v5.1.7
- Conclusion: don't install laravel/framework v5.1.6
- Conclusion: don't install laravel/framework v5.1.5
- Conclusion: don't install laravel/framework v5.1.4
- Conclusion: don't install laravel/framework v5.1.3
- Installation request for elasticquent/elasticquent 1.0.5 -> satisfiable by elasticquent/elasticquent[v1.0.5].
- Conclusion: don't install laravel/framework v5.1.2
- Conclusion: don't install laravel/framework v5.1.1
- elasticquent/elasticquent v1.0.5 requires illuminate/config ~4.2|~5.2 -> satisfiable by illuminate/config[v4.2.1, v4.2.12, v4.2.16, v4.2.17, v4.2.2, v4.2.3, v4.2.4, v4.2.5, v4.2.6, v4.2.7, v4.2.8, v4.2.9, v5.2.0, v5.2.6, v5.2.7].
- don't install illuminate/config v4.2.1|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.12|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.16|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.17|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.2|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.3|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.4|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.5|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.6|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.7|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.8|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.9|don't install laravel/framework v5.1.0
- don't install illuminate/config v5.2.0|don't install laravel/framework v5.1.0
- don't install illuminate/config v5.2.6|don't install laravel/framework v5.1.0
- don't install illuminate/config v5.2.7|don't install laravel/framework v5.1.0
- Installation request for laravel/framework 5.1.* -> satisfiable by laravel/framework[v5.1.0, v5.1.1, v5.1.10, v5.1.11, v5.1.12, v5.1.13, v5.1.14, v5.1.15, v5.1.16, v5.1.17, v5.1.18, v5.1.19, v5.1.2, v5.1.20, v5.1.21, v5.1.22, v5.1.23, v5.1.24, v5.1.25, v5.1.26, v5.1.27, v5.1.28, v5.1.3, v5.1.4, v5.1.5, v5.1.6, v5.1.7, v5.1.8, v5.1.9].

Thanks for reporting this. I will fix it as soon as I get to my desk!
	 ---- On Thu, 28 Jan 2016 02:17:56 +1100<EMAIL_ADDRESS>wrote ----Hi everybody,

I'am using laravel 5.1.. I was using Elasticquent in my project to this day.
But in today I run "composer update" and got this error.
Please Help Me.
Your requirements could not be resolved to an installable set of packages.
Problem 1
- Conclusion: remove laravel/framework v5.1.28
- Conclusion: don't install laravel/framework v5.1.28
- Conclusion: don't install laravel/framework v5.1.27
- Conclusion: don't install laravel/framework v5.1.26
- Conclusion: don't install laravel/framework v5.1.25
- Conclusion: don't install laravel/framework v5.1.24
- Conclusion: don't install laravel/framework v5.1.23
- Conclusion: don't install laravel/framework v5.1.22
- Conclusion: don't install laravel/framework v5.1.21
- Conclusion: don't install laravel/framework v5.1.20
- Conclusion: don't install laravel/framework v5.1.19
- Conclusion: don't install laravel/framework v5.1.18
- Conclusion: don't install laravel/framework v5.1.17
- Conclusion: don't install laravel/framework v5.1.16
- Conclusion: don't install laravel/framework v5.1.15
- Conclusion: don't install laravel/framework v5.1.14
- Conclusion: don't install laravel/framework v5.1.13
- Conclusion: don't install laravel/framework v5.1.12
- Conclusion: don't install laravel/framework v5.1.11
- Conclusion: don't install laravel/framework v5.1.10
- Conclusion: don't install laravel/framework v5.1.9
- Conclusion: don't install laravel/framework v5.1.8
- Conclusion: don't install laravel/framework v5.1.7
- Conclusion: don't install laravel/framework v5.1.6
- Conclusion: don't install laravel/framework v5.1.5
- Conclusion: don't install laravel/framework v5.1.4
- Conclusion: don't install laravel/framework v5.1.3
- Installation request for elasticquent/elasticquent 1.0.5 -> satisfiable by elasticquent/elasticquent[v1.0.5].
- Conclusion: don't install laravel/framework v5.1.2
- Conclusion: don't install laravel/framework v5.1.1
- elasticquent/elasticquent v1.0.5 requires illuminate/config ~4.2|~5.2 -> satisfiable by illuminate/config[v4.2.1, v4.2.12, v4.2.16, v4.2.17, v4.2.2, v4.2.3, v4.2.4, v4.2.5, v4.2.6, v4.2.7, v4.2.8, v4.2.9, v5.2.0, v5.2.6, v5.2.7].
- don't install illuminate/config v4.2.1|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.12|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.16|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.17|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.2|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.3|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.4|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.5|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.6|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.7|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.8|don't install laravel/framework v5.1.0
- don't install illuminate/config v4.2.9|don't install laravel/framework v5.1.0
- don't install illuminate/config v5.2.0|don't install laravel/framework v5.1.0
- don't install illuminate/config v5.2.6|don't install laravel/framework v5.1.0
- don't install illuminate/config v5.2.7|don't install laravel/framework v5.1.0
- Installation request for laravel/framework 5.1. -> satisfiable by laravel/framework[v5.1.0, v5.1.1, v5.1.10, v5.1.11, v5.1.12, v5.1.13, v5.1.14, v5.1.15, v5.1.16, v5.1.17, v5.1.18, v5.1.19, v5.1.2, v5.1.20, v5.1.21, v5.1.22, v5.1.23, v5.1.24, v5.1.25, v5.1.26, v5.1.27, v5.1.28, v5.1.3, v5.1.4, v5.1.5, v5.1.6, v5.1.7, v5.1.8, v5.1.9].
—Reply to this email directly or view it on GitHub.

Or not, as it turns out... GitHub was down!
I am currently testing this now, but it should be good to go!


Check a number exists in WhatsApp or not??

How will we check that a number is in WhatsApp or not ???

I don't think the library is still working (see #27).
Something that may relate to your question: https://github.com/p4kl0nc4t/wabf


feat: dont remove non pixi pypi installs

Check the INSTALLER when removing packages either through changing of dependencies or python version.
Fixes: #998

Before merging we need to re-think the logic a bit.

When we want to install a package that was not managed by us, but is managed by us now, we should still re-install.
Maybe add a warning when we find pypi packages that were not installed by us or something? So that people know that we've not touched those?

Two points:

when a package is installed (even by another package manager), but we want to install a different version of it, we should still remove it and install the pixi managed version (maybe with a warning?)
when users have an existing environment and update it might look ugly for the transition period. Maybe for one or two pixi releases, we should keep the uv handling as well?

Any progress on this? @wolfv @tdejager

Not yet, we should revive this. Although you can now use git dependencies if you want 😀

Awesome!


improvements

pass just the html code without and the website called without passing the url

This would require some sort of standardized html that always had the recipe data in the same place or the use of something more complex to "guess" where the data lives. If you look through the code for this module, each website is more or less uniquely scraped.

I just want to say you get html from for example yummly so you pass this html in a module function with the website name

I see, that would be doable. I'll consider adding that feature in a future release. Thanks for the input!

Yor welcome 👍


Admin section improvements

Some sub-pages allow you to go back to the main list (like add new org code) but the others don't.
Admin roles uses a back arrow icon for an adding action
Assign role to use button is messed up
Add new team tag button messed up (though on edit it looks nice)
Maybe get rid of reports since it is already on the header?
https://keas.azurewebsites.net/caesdo/TeamAdmin/AddMemberRole

@cydoval I think the "a btn-primary" and "a btn-danger" styles need to match "input btn-primary" and "input btn-danger" styles. Sometimes use inputs, but often just use links for the cancel parts of forms. Or let me know better classes to use for these?


Set up deployment using Travis CI

This removes Drone for building Docker images and deploying them.
This is now being done by Travis CI.
It also pushes the images to Docker Hub instead of Quay
and replaces the deprecated version of src-d/ci with v1.

@dpordomingo There something I don't understand. If we have different stages to build staging and production images (LANDING_URL is different), how come do we have just one type of images in https://quay.io/organization/srcd?

@rporres we have two kind of different images.
here is an example comparing srcd/landing:v4.17.11 against srcd/landing:commit-40cf53d.
$ docker run -it --rm --entrypoint /bin/sh quay.io/srcd/landing:v4.17.11 --
/srv # cat /var/www/public/index.html | grep '<base'
    <base href="//sourced.tech">
/srv # exit

$ docker run -it --rm --entrypoint /bin/sh quay.io/srcd/landing:commit-40cf53d --
/srv # cat /var/www/public/index.html | grep '<base'
    <base href="//landing-staging.srcd.run">
/srv # exit

Does this answer your question?
(also, we have srcd/landing-api images for prod and staging, using the same tags)

Here is what I wrote by slack, which could explain why both images are different:

Does this answer your question?

yes, thanks

This looks great. Let's wait for @vcoisne to give us green light to deploy

cc @znegrin who handles the technical aspect of our website deployment. LGTM

Thx, @vcoisne. Merging, then. This is a bit outside @znegrin realm, since it's related to how we deploy in our public clusters.
@znegrin Please ping me if you want further details on what this change means or anything about how/where we deploy.


yii2 core rules match not working with (not)

What steps will reproduce the problem?
$model->addRule(['keyword'],'match',['pattern'=>'/[-a-zA-Z0-9@:%.+~#=]{2,256}.[a-z]{2,6}\b([-a-zA-Z0-9@:%+.~#?&//=]*)/','message'=>'Enter Valid Keyword Not Url','not'=>true]);
What is the expected result?
it should be giving not url patter match found
What do you get instead?
preg_match(): Unknown modifier '/'
Additional info

Q
A

Yii version
2.0.?

PHP version

Operating system

You have a syntax error in your Regexp.
/[-a-zA-Z0-9@:%.+~#=]{2,256}.[a-z]{2,6}\b([-a-zA-Z0-9@:%+.~#?&\/=]*)/


does not support NSS

curl --version curl 7.29.0 (x86_64-redhat-linux-gnu) libcurl/7.29.0 NSS/3.53.1 zlib/1.2.7 libidn/1.28 libssh2/1.8.0
ldd /usr/bin/curl | grep ssl         libssl3.so => /usr/lib64/libssl3.so (0x00007f255286b000)         libssl.so.10 => /usr/lib64/libssl.so.10 (0x00007f254f9e4000)
libssl3.so is NSS , it does not contain SSL_read/SSL_write function

supported at v0.1.5 as nspr flag.


IAM: SageWorks AWSAccess Class

So there will probably be some complexity around AWS Accounts and Identity Management. There are several AWS services we will have to interact with

AWS IAM
AWS Identity Center (used to be called SSO)
Security Token Service (STS)
Others???

A Good place to start is what information is useful when running these commands
❯ aws iam get-account-summary
❯ aws sts get-caller-identity
❯ aws iam get-account-authorization-details

There's Python/boto3 equivalents of all these.

Okay.. so the aws_sageworks_role_manager has a lot of the IAM responsibilities.. seems to be working fairly well so far. We'll add functionality there if we need it. closing this ticket.


active and hovered checkbox button background colors should not be the same

If a v4 checkbox button is 'active' then clicking it with the cursor to uncheck it provides no visual feedback. This is because the 'active' and 'hover' states have the same background colours. You have to move the cursor away from the button to verify it is now unchecked.
Here, the middle checkbox is not checked/active, although it looks like it is:

hmmm...i'm fairly sure i had a PR in v3 that addressed this exact problem. wonder if it got left out in the transition to v4...
ah, i'm thinking of this https://github.com/twbs/bootstrap/pull/16154

*sigh* And this is why I said we should audit the recent v3 commits to ensure they had been ported and thus guard against such regressions...

sigh

There were bond to be oversights on my part :).

Addressed in 1f153b6.

<3


first Commit

Pitias Fessahaie
Thank you

It looks like you forgot to put @Test annotation on the top of your tests.

There is a missing Script. Please see the attachment file
Pitias Fessahaie
Automation Tester / QA Teter
<EMAIL_ADDRESS>(240)-423-7954
On Fri, Sep 18, 2020 at 12:58 AM hbahuguna2<EMAIL_ADDRESS>wrote:

It looks like you forgot to put @test https://github.com/test
annotation on the top of your tests.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Dotdashcom/take-home-webdriver-test/pull/48#issuecomment-694650727,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF3M4BM3UFWC2TFCXJ4MUULSGLSHNANCNFSM4RRGYHBQ
.

There is a file Called Script in a separate Package (take.homw.script )if you didn't get it. I have emailed it to you
I apologize for the misplacement
Thank you


Update line length docs to match the code

According to the code, the default line length is 79: https://github.com/timothycrosley/isort/blob/7e2fb3b21b4b588dc7a62ac65693781ae39e8a98/isort/settings.py#L52

Thanks!


Add Conda Publishing

Building w/ dependencies on our public channel!
Next step is to figure out the nightly publish of these packages.

How can we make sure the src file name in meta.yaml matches the built package?

@xiangyan99 I'll ping you when I can to discuss.


Fix volume replica parallelization within ec.encode.

What problem are we solving?
Make EC balancing logic topology-aware: https://github.com/seaweedfs/seaweedfs/discussions/6179 .
How are we solving the problem?
Fix volume replica parallelization for ec.encode. In particular, this PR makes the number of parallel tasks configurable.
See 826edd5d.
How is the PR tested?
No functional / test changes.
Checks

[x] I have added unit tests if possible.
[x] I will add related wiki document changes and link to this PR after merging.

@chrislusf , the weed shell is currently broken at head :disappointed:  This PR fixes it.


Doubts about reward

Thank you very much for such a great work. When I run gpt2-sentiment.py (https://github.com/lvwerra/trl/blob/main/examples/sentiment/scripts/gpt2-sentiment.py#L151), I have a question I would like to ask:
rewards = [torch.tensor(output[1]["score"]) for output in pipe_outputs]
What is the value range of the reward score here(I print to see that there are greater than 1, greater than 2, etc.), is it directly taken logits? And rewards can be any range?
For each input sentence, how is his reward obtained? Is it the sum of the logits of each token in the sentence?

Yes, we use the raw logits, we found that usually works better than e.g. softmax normalized outputs. And you get one reward per sequence from the classifier model so no need to aggregate them. In theory you could pass a reward per token but this is not implemented at the moment.

Thank you very much, I have another question, that is, the model I want to train with ppo is AutoModelForSeq2SeqLM, just refer to here(https://github.com/lvwerra/trl/blob/main/examples/sentiment/scripts/gpt2-sentiment.py#L104) and change it to
from transformers import  AutoModelForSeq2SeqLM model = AutoModelForSeq2SeqLM.from_pretrained(config.model_name, trust_remote_code=True) ref_model = AutoModelForSeq2SeqLM.from_pretrained(config.model_name, trust_remote_code=True)
But get the following error:
alueError: model must be a PreTrainedModelWrapper, got <class 'transformers_modules.local.modeling_glm.GLMForConditionalGeneration'> - supported architectures are: (<class 'trl.models.modeling_value_head.AutoModelForCausalLMWithValueHead'>, <class 'trl.models.modeling_value_head.AutoModelForSeq2SeqLMWithValueHead'>)
How can I adapt in the current trl framework so that I can use ppo to train my AutoModelForSeq2SeqLM model?
Thank you very much, looking forward to answer

@younesbelkada this might be an issue of a kwarg not being passed along to the original model class, wdyt?

Hello @Mryangkaitong,
Thanks for the issue!
Yes this is possible, however the script below is working on the main branch of trl:
from trl import AutoModelForSeq2SeqLMWithValueHead
model = AutoModelForSeq2SeqLMWithValueHead.from_pretrained("t5-small", trust_remote_code=True)

I believe you need to use the latest changes of the library, by installing trl from source:
pip install git+https://github.com/lvwerra/trl.git

Let us know if the problem still persists!

ok , I have solved it


Verify Account

[ ] Use account & password define in hook.json to control user access

close db7fa0d7eab0ede4a9ad54fe385bbdd5c918fd08


Fix spelling mistake

CC: @nuclearsandwich

Also cherry picking this onto noetic-devel.

Pushed to noetic as 9aa8c65

See also #184 and #185


enhance OpenSDS CLI function and add logging module

To solve Issues #10, after taking a reference with Kubernetes and Torus (by CoreOS), we found that Cobra, a library for creating powerful modern CLI applications, can help us enhance OpenSDS CLI function. So we did some change to the code to make CLI a better user experience. Besides, we add logging module in to debug the system.

Apologize for the wrong word "CNI", which is actually "CLI".


[FEATURE] Document compositing multiple applications and multi-file design

Is your feature request related to a problem? Please describe.
In the routing docs we need to explain how to bring in routes from different modules or applications into the main route.
Describe the solution you'd like
Create a clear, concise, interesting example. Reference FastHTML Gallery and other projects that leverage this technique.
Implementations

FastHTML Gallery

Problem solved
Explain what problem this feature would solve and how it would benefit users of the library:
Additional context
Add any other context or screenshots about the feature request here.
Confirmation
Please confirm the following:

[x] I have checked the existing issues and pull requests to ensure this feature hasn't been requested before.
[x] I have read the project's documentation to ensure this feature doesn't already exist.

fh-about is also a useful example.

In the latest version of fasthtml, There is APIRouter class which can be used for multi-file design to describe the routes. The routes explanation section is incomplete but there is link where you can find the explanation of the APIs of core module.
docs of core


Can we add preprocessor_config.json for llava-next-interleave-qwen-7b model on Huggingface?

Hi, I am using sglang to deploy llava-next-interleave-qwen-7b, but i found there is no preprocessor_config.json for  llava-next-interleave-qwen-7b model, could we add this to huggingface? Or do we have some other recommend and efficiency method to deploy it ? Thanks~

Currently llava-next-interleave didnt support sglang, if you are going to support it, you could refer to this file and create new siglip tokenizer for it. Inside it should have a preprocessor.json.
https://huggingface.co/lmms-lab/llavanext-qwen-tokenizer/tree/main


ci: update golangci-lint

Codecov Report

Merging #1586 (02173f3) into main (3a5350d) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head 02173f3 differs from pull request most recent head 31be772. Consider uploading reports for the commit 31be772 to get more accurate results

@@           Coverage Diff           @@
##             main    #1586   +/-   ##
=======================================
  Coverage   49.50%   49.50%
=======================================
  Files          42       42
  Lines        4965     4965
=======================================
  Hits         2458     2458
  Misses       2363     2363
  Partials      144      144

Files
Coverage Δ

cmd/gpu_plugin/gpu_plugin.go
86.74% <100.00%> (ø)

pkg/fpga/bitstream/bitstream.go
88.88% <100.00%> (ø)

:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


duplicate lines in docstrings of layout/init.py

layout/__init__.py:21
 16  Container classes::
 17
 18     - Container (Abstract base class)
 19        |- HSplit (Horizontal split)
 20        |- VSplit (Vertical split)
 21        |- VSplit (Vertical split)
 22        |- FloatContainer (Container which can also contain menus and other floats)
 23        `- Window (Container which contains one actual control

Fixed: https://github.com/jonathanslenders/python-prompt-toolkit/commit/950d9b0e81395febff2d02bbb4cb328255835151
Thank you!


Testing

Everything needs tests.

I just want to toss some notes here for the future, when we do start testing...
We can choose to go with the native unittest provided by Python, or pick up an external framework like py.test. I personally prefer the simplicity of py.test, but I'm not opposed to using other frameworks that may be better suited.
We are going to need to split tests up into those requiring hardware, and those which can be run without anything else. The mock library (integrated in Python 3.3+ as unittest.mock) will end up being useful for mocking the Edison and other external hardware.

Testing should be all set in the zorg-gpio module. Additional tests will be needed once the remaining drivers for #9 are created.

I'm going to close this one off because we have a good direction going for tests. Future issues for tests can be opened on a case-by-case basis.


Renamed tags in authenticator api to clashes across several files

Renamed tags in authenticator api to avoid generated classes to be overwritten.
Having same tag "Authentication" in both https://github.com/symphonyoss/symphony-api-spec/blob/master/authenticator/authenticator-api-public.yaml and https://github.com/symphonyoss/symphony-api-spec/blob/master/login/login-api-public.yaml resulted in one single class AuthenticationApi reflecting only login-api-public.yaml
This is needed in scope of APP-2845

Based on https://github.com/search?q=org%3ASymphonyOSF+symphony-api-spec&type=Code , I tested the following impacted repos were still compiling with these changes:

https://github.com/SymphonyOSF/symphony-agent
https://github.com/SymphonyOSF/symphony-sdk-java-v2
https://github.com/SymphonyOSF/symphony-testr
https://github.com/SymphonyOSF/pmp-integration-tests-poc
https://github.com/SymphonyOSF/symphony-bot-starter (was not even compiling without these changes)

@symphony-elias this change hasn't been reflected on SBE side. It caused an issue with latest spec updated made by @symphony-mariacristina (https://github.com/symphonyoss/symphony-api-spec/pull/81)


No output is visible in Console window

No output is visible in Console window:

option "Hide console window" is disabled
tried out with lame and qaac-lc encoders

The output is redirected to BatchEncoder stderr stream so it can parse output. I am planning to add logging support so you can check the console output.

That would be perfect, thanks.

@chros73 You can track progress here https://github.com/wieslawsoltes/BatchEncoder/issues/171.

Thanks, then I'm closing this issue.


FEAT: Add falco and multiqc.

I suggest to split the two blocks of groovy code off in two separate workflows, something like split_samples_and_validate and combine_samples.

I suggest to split the two blocks of groovy code off in two separate workflows, something like split_samples_and_validate and combine_samples.

Done! I also added multiqc in the meantime ⏩

Looks good!
My suggestion would be to put the spun off workflows in their separate 'component' like for instance https://viash-hub.com/data-intuitive/vsh-pipeline-operators/-/tree/main/src/split/vsh_flatten?ref_type=heads.

Looks good!
My suggestion would be to put the spun off workflows in their separate 'component' like for instance https://viash-hub.com/data-intuitive/vsh-pipeline-operators/-/tree/main/src/split/vsh_flatten?ref_type=heads.

I'm not sure that the current workflow got simpler by doing this. But once more functionality is added I think it will have been beneficial ✅

Looks good!


Remove urls, use service discovery

Overview

/describe

TODO:

Revert image tags before committing changes

Ready to be merged:
https://github.com/kubescape/helm-charts/actions/runs/6021436772


Add sql handling for contract interface events, and adjust database naming

This adds the sql handler for performing database operations on the "events" defined in a contract interface (or FFI).
It also adjusts the naming of "contractevents" and "contractmethods" to "contractinterface_events" and "contractinterface_methods" respectively (in the table names, file names, and method names). The primary reason for this is the need to differentiate between the format of an event that is part of a contract interface, and an actual event that has been emitted and recorded. But it also helps to make clear that these items are generally children of a contractinterface.
However, if this naming is too verbose, we could also consider using "ffi" in more places rather than "contractinterface".

Closing - meant to open this on the feature branch and not on main.


nothing shows up when i click Q

the menu shows up but its completely empty beside fron tools, also theres "search" on the top left corner

Not our bug.


Focus issue in Interactive window

Version: 1.60.2 (user setup)
Commit: 7f6ab5485bbc008386c4386d08766667e155244e
Date: 2021-09-22T12:00:31.514Z
Electron: 13.1.8
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.19043
Extension: v2021.8.2041215044

I'm having an issue with shortcuts, and they sometimes stop working in the bottom code entering field in the Interactive Window. I believe it happened before, but I couldn't find the issue that logged it. It might have something to do with "focus".
I can't find a consistent way to reproduce it now, and it has been here for a while. But when it happens, enter+shift will no longer execute the code I entered, and I must click somewhere in the Editor and then click the code entering field to get it back to work again.

I believe we've address this in the the insider version. Please could you install VS Code insiders and test this out.

@StevenLi-DS please reopen if insiders doesn't work for you. Thanks.

I'm not able to use it in insiders since the extension won't load for some reason.

@DonJayamanne @rchiodo Sorry for the late reply. After trying it out for a very short period of time, I'm not having this issue in Insiders now.


 The animation is in the wrong spot to: rather than being in the center of where the image should go, it is in the top left.
Here is a random product to try it with:
http://www.newegg.com/Product/Product.aspx?Item=N82E16824260148

Hmm, I was unable to reproduce the issue via the given url or the subsequent links. At the moment it seems to be working correctly.

Firefox Nightly 41.0a1 (2015-05-14)
Shumway v0.11.239


When the api key is wrong- the reponse from server is wrong and not useful

{"id":"forbidden","status":403,"title":"Forbidden","detail":"Issue with CSRF code","error":"http: named cookie not present","encrypted":false,"multires":null}
It would be more helpful if the error and detail informs that the apikey is wrong.

Ping! still the same issue

Thanks for the ping. We have plans to upgrade this package and improve usability. We will be addressing this then.


Resolve FailWith overload resolution when only constant-string is passed

(This is my first github pull request, so please bear with me)
I was unable to compile the develop branch in 26be9f6295d9b3ab6cbca8b4ee800fd9ca5674ae due to overload resolution on FailWith.

Thanks for the elaborate explanation. I noticed that VS2015 is more strict about ambigious overloads, but I didn't fully switch yet.


Code continuity update v2

-Fixed sizing issue with the share view controller
-Updated the test view controller to adhere to the UIComponets
-Fixed most minor error messages

@2020Nonprofit - Automation is now setup to prevent merging when someone adds lint warnings.  This is a common practice so I hope you understand, but you'll need to fix them all to be able to merge.  Please message me on Slack if you have any questions about that or want to discuss it.


Web - Pressing on keyboard enter key on notification option of a room takes back two steps

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.63-0
Reproducible in staging?: y
Reproducible in production?: no
If this was caught during regression testing, add the test name, ID and link from TestRail: EXP around https://expensify.testrail.io/index.php?/tests/view/4494233
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Action Performed:

Navigate to staging.new.expensify.com
Create a room
Open the room page
Click on header
Click on settings
Click on Notify me about new messages
Select a notification preference option by using the keyboard arrow keys
Press enter on one of the options

Expected Result:
App should navigate to Settings page
Actual Result:
App navigates to Room details page
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/43996225/6b79c3b3-8794-4848-8108-5a5b282cf797

View all open jobs on GitHub

@mountiny FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

We think this bug might be related to #vip-vsb

Since we have a better proposal here, we can close the other.

@suneox Were you able to figure out the PR which caused this issue?

@suneox can you prepare a PR with the fix and make sure you test properly all the keyboard selection shortcuts in various places of the app? Thanks

Please either @suneox or @allroundexperts try to raise a PR for this one when you get a chance, thanks

I'll be home in about 2 hours. Will raise one right away.

Started working on it.

@suneox you can test the PR as a C+ role

PR: https://github.com/Expensify/App/pull/40642

Checking 👀

Hi @allroundexperts could you please check my commit your PR was missing set shouldPreventEnterKeySubmit=true for the BaseSelectionList so the result still not fix

https://github.com/Expensify/App/blob/4dc0d91dbf2dbc41ee62032ae6b11b41b0c53d48/src/pages/settings/Report/NotificationPreferencePage.tsx#L41-L45
I'd like to explain for my commit change
At page NotificationPreferencePage SelectionList using ListItem={RadioListItem} so we have to forward shouldPreventEnterKeySubmit props inside RadioListItem
And we can use CONST.KEYBOARD_SHORTCUTS.ENTER.shortcutKey instead of string 'Enter'

Is this limited to RadioListItem only? I feel like we should do this for all list items.

@suneox Makes sense. Let's fix the radio item list first. Updated the PR.

Will update the screen recordings in the next few mins as well.

@suneox Updated and added screen recordings. Let's take this convo to the PR.

@suneox I believe you cannot review a you are not C+, but you can leave comments with the checklist and approve by comment

Payment summary:

@suneox $250
@allroundexperts $250 (NewDot)

$250 approved for @allroundexperts


Plugin setup

Update plugin package, name and vendor and other necessary setup tasks

@JChrist there's a really weird failure message here, in the GH action run...
Anything you might have come across before ?

after a closer look, it looks like something that was introduced in https://github.com/cytechmobile/radicle-jetbrains-plugin/commit/66641c515b421f54684f5bce376b329ab69d84c1 , as the GH action also failed with the same error then

I haven't really taken a look at the auto-generated tests and the code I added is mostly placeholder, so failures (even compilation ones) are to be expected at this time.
Merging this in.


Resource Groups are listed under disks category

Storage Explorer Version: 1.11.2
Build Number: 20191217.4
Platform/OS: Windows 10
Architecture: x64
Regression From:  N/a
Bug Description
When connected the Azure Storage explorer using Azure account; it shows all the resource groups under Disks section.
Steps to Reproduce

Open Azure Storage Explorer.
Right-Click on storage accounts.
Select Add an Azure account to list all Subscriptions.
Once subscription is selected, then it lists all the Resource Groups under Disks category.

Expected Experience
All the resource groups should appear under Resource Group category and only disks should appear under the Disks category.
Actual Experience

Additional Context

Hey @rahulsi386 , thank you for sharing this feedback!
What your're seeing is by design. When developing the disks feature set, we thought a lot about whether or not it made sense to have each disk on the left hand side tree view. What we ultimately decided was no, it didn't make sense. We didn't think it made sense because of the number of disks Azure customers have. If we just displayed them in a flat, unorganized list (which has a less than stellar search feature right now), it could be nearly impossible for you to find the disk you want! So we said, let's organize the disks by resource groups, and then list the disks on the right hand side. That also works out, because all the "operations" are now in the Disk Explorer toolbar, as opposed to hidden behind the tree view right click menu.
So, let me ask you:

Why do you want disks to appear on the left hand side?
If they did, would you expand any sort of experience on the right hand side of the application?
How many disks do you have in your subscriptions?
In general, what's the ideal way you'd like to work with your disks in Storage Explorer?

Please try to elaborate as much as possible in your reply! I'd love to hear your feedback. 😄
Thanks!

Thanks for clarification, Matthew. That makes sense.
On Thu, Dec 19, 2019 at 10:48 PM Matthew Rayermann<EMAIL_ADDRESS>wrote:

Hey @rahulsi386 https://github.com/rahulsi386 , thank you for sharing
this feedback!
What your're seeing is by design. When developing the disks feature set,
we thought a lot about whether or not it made sense to have each disk on
the left hand side tree view. What we ultimately decided was no, it didn't
make sense. We didn't think it made sense because of the number of disks
Azure customers have. If we just displayed them in a flat, unorganized list
(which has a less than stellar search feature right now), it could be
nearly impossible for you to find the disk you want! So we said, let's
organize the disks by resource groups, and then list the disks on the right
hand side. That also works out, because all the "operations" are now in the
Disk Explorer toolbar, as opposed to hidden behind the tree view right
click menu.
So, let me ask you:

Why do you want disks to appear on the left hand side?
If they did, would you expand any sort of experience on the right
hand side of the application?
How many disks do you have in your subscriptions?
In general, what's the ideal way you'd like to work with your disks
in Storage Explorer?

Please try to elaborate as much as possible in your reply! I'd love to
hear your feedback. 😄
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/AzureStorageExplorer/issues/2465?email_source=notifications&email_token=AENFMZSVNJXCECGVMLTFFZTQZOUG7A5CNFSM4J46QJW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHKJX6I#issuecomment-567581689,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AENFMZR2RCEDBA3HIOBYBW3QZOUG7ANCNFSM4J46QJWQ
.

--
Regards,
Rahul Prashant Singh
Mob: (+91)-9731019140

Glad to hear. Always feel free to reach out if you encounter any issues, or if you have an idea to make Storage Explorer better. We love to hear from y'all!


[MenuItem] - It changes the css when normalize.css is used

Problem Description

When I load normalize.css the MenuItem change your own view like this:

Versions

Material-UI: 0.14.4
React: 15.0.1
Browser: Versão 50.0.2661.86 (64-bit)

https://github.com/callemall/material-ui/issues/3764


fix: create VNI name based of the VSI name

Description

Release required?

[ ] No release
[x] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

/run pipeline

Upgrade test will fail due to this update-in-place. This is a non-disruptive change. Will skip the upgrade test

/run pipeline