panicked at 'attempt to subtract with overflow'

Hi,
I ran into:
thread 'main' panicked at 'attempt to subtract with overflow', /home/m/.cargo/registry/src/github.com-1ecc6299db9ec823/cursive-0.4.1/src/views/list_view.rs:229
stack backtrace:
   1:     0x7fc581953f6c - std::sys::imp::backtrace::tracing::imp::write::hdc68b25dedcd1db8
   2:     0x7fc581952a7e - std::panicking::default_hook::{{closure}}::h5edd4f46e38f8310
   3:     0x7fc581952763 - std::panicking::default_hook::hda32855a02198554
   4:     0x7fc581952f0b - std::panicking::rust_panic_with_hook::hdd2e2d2d6c308562
   5:     0x7fc581952d54 - std::panicking::begin_panic::h48ad6352979e67f1
   6:     0x7fc581952cc9 - std::panicking::begin_panic_fmt::h2ded1b6830a36623
   7:     0x7fc581952c57 - rust_begin_unwind
   8:     0x7fc5819b1d6d - core::panicking::panic_fmt::h2aae0852c2562c9f
   9:     0x7fc5819b1ca4 - core::panicking::panic::h7f49eacfb0b4c1fc
  10:     0x55ec0df23aa8 - <cursive::views::list_view::ListView as cursive::view::View>::layout::h68ec5254d2a965f7
                        at /home/m/.cargo/registry/src/github.com-1ecc6299db9ec823/cursive-0.4.1/src/views/list_view.rs:229
  11:     0x55ec0de98ccc - cursive::view::view_wrapper::ViewWrapper::wrap_layout::h9e9d7de1055dca60
                        at /home/m/.cargo/registry/src/github.com-1ecc6299db9ec823/cursive-0.4.1/src/view/view_wrapper.rs:45
  12:     0x55ec0de9aa7f - cursive::view::view_wrapper::<impl cursive::view::View for T>::layout::h0437efaaaa52fd53
                        at /home/m/.cargo/registry/src/github.com-1ecc6299db9ec823/cursive-0.4.1/src/view/view_wrapper.rs:78
  13:     0x55ec0de98dac - cursive::view::view_wrapper::ViewWrapper::wrap_layout::hdeb15779bd802288
                        at /home/m/.cargo/registry/src/github.com-1ecc6299db9ec823/cursive-0.4.1/src/view/view_wrapper.rs:45
  14:     0x55ec0de9abff - cursive::view::view_wrapper::<impl cursive::view::View for T>::layout::had410b533505ca6d
                        at /home/m/.cargo/registry/src/github.com-1ecc6299db9ec823/cursive-0.4.1/src/view/view_wrapper.rs:78
  15:     0x55ec0df208ce - <cursive::views::linear_layout::LinearLayout as cursive::view::View>::layout::he430049eeb1c48a1
                        at /home/m/.cargo/registry/src/github.com-1ecc6299db9ec823/cursive-0.4.1/src/views/linear_layout.rs:201
  16:     0x55ec0de98bec - cursive::view::view_wrapper::ViewWrapper::wrap_layout::h5b2fd4995ac90378
                        at /home/m/.cargo/registry/src/github.com-1ecc6299db9ec823/cursive-0.4.1/src/view/view_wrapper.rs:45
  17:     0x55ec0de9acbf - cursive::view::view_wrapper::<impl cursive::view::View for T>::layout::hc245bd8e062c3d60
                        at /home/m/.cargo/registry/src/github.com-1ecc6299db9ec823/cursive-0.4.1/src/view/view_wrapper.rs:78
  18:     0x55ec0de64ad4 - <cursive::views::shadow_view::ShadowView<T> as cursive::view::view_wrapper::ViewWrapper>::wrap_layout::he941d882bcdabd4d
                        at /home/m/.cargo/registry/src/github.com-1ecc6299db9ec823/cursive-0.4.1/src/views/shadow_view.rs:58
  19:     0x55ec0de9ab9f - cursive::view::view_wrapper::<impl cursive::view::View for T>::layout::h7bb1fefbf58cdf4a
                        at /home/m/.cargo/registry/src/github.com-1ecc6299db9ec823/cursive-0.4.1/src/view/view_wrapper.rs:78
  20:     0x55ec0df29ed9 - <cursive::views::stack_view::StackView as cursive::view::View>::layout::h7de8e74d223fe846
                        at /home/m/.cargo/registry/src/github.com-1ecc6299db9ec823/cursive-0.4.1/src/views/stack_view.rs:154
  21:     0x55ec0df356ba - cursive::Cursive::layout::h593ae75221da9ba7
                        at /home/m/.cargo/registry/src/github.com-1ecc6299db9ec823/cursive-0.4.1/src/lib.rs:494
  22:     0x55ec0df35df5 - cursive::Cursive::step::hc789d40ea1a7d171
                        at /home/m/.cargo/registry/src/github.com-1ecc6299db9ec823/cursive-0.4.1/src/lib.rs:570
  23:     0x55ec0df35c8f - cursive::Cursive::run::ha463ea6606b82047
                        at /home/m/.cargo/registry/src/github.com-1ecc6299db9ec823/cursive-0.4.1/src/lib.rs:552
  24:     0x55ec0deae19f - rikki::frontend::curses::start::h0dcd081ef2d42d16
                        at /home/m/archive/code/rust/rikki/src/frontend/curses/mod.rs:38
  25:     0x55ec0deae761 - rikki::main::heea4d07080c68991
                        at /home/m/archive/code/rust/rikki/src/main.rs:84
  26:     0x7fc581961567 - __rust_maybe_catch_panic
  27:     0x7fc58195b667 - std::rt::lang_start::h667ad30118861e80
  28:     0x55ec0deb3a12 - main
  29:     0x7fc57fa2752f - __libc_start_main
  30:     0x55ec0de62759 - _start
                        at ../sysdeps/x86_64/start.S:120
  31:                0x0 - <unknown>

Anything I can do about it?

This happens (as it seems) if the terminal window is too small ...

Indeed, ListView wasn't properly defensive with regard to small viewports. Working on a fix.
Thanks for the report!

Should be fixed in 7d9cb03ffbfeeeac2e02adce2b4139a62fc5d31c.

For what it's worth, I'm also seeing an "attempt to subtract with overflow" on the latest master commit 5f5fb4e when I try to wrap a SelectView in a Panel (in a BoxView in a LinearLayout).  It looks like it's happening where Panel is trying to make room for a border in Panel.wrap_required_size().  (But it's a fair likelihood that I'm just misunderstanding something about view arrangement.)
extern crate cursive;
use cursive::Cursive;
use cursive::view::*;
use cursive::views::*;
fn main() {
    let mut siv = Cursive::new();
    let mut select_view = SelectView::new();
    select_view.add_item("Item 1", 1);
    select_view.add_item("Item 2", 2);
    let body = BoxView::new(
        SizeConstraint::Full,
        SizeConstraint::Full,
        Panel::new(select_view)
    );
    let status = BoxView::new(
        SizeConstraint::Full,
        SizeConstraint::Fixed(1),
        TextView::new("status line")
    );
    let layout = LinearLayout::vertical()
        .child(body)
        .child(status);
    siv.add_layer(layout);
    siv.run();
}

Indeed, that was a bug in Panel - thanks! Should be fixed in 87676295a0d79d57a660ebbc9490abcb74f04c4c.

@matthiasbeyer Can you confirm your bug is fixed in latest master?

I'll assume this is fixed - feel free to open it again if I'm wrong!

Currently, I get this error exactly like above stack trace with 0.9.1.
Didn't understand what caused this but if lines are long it bails out at this point.

@vertexclique Interesting! Can you post a sample code to reproduce the issue?

PanicInfo { payload: Any, message: Some(attempt to subtract with overflow), location: Location { file: "/Users/m.bulut/.cargo/registry/src/github.com-1ecc6299db9ec823/cursive-0.9.1/src/views/list_view.rs", line: 412, col: 43 } }

Indeed, there was an unchecked subtraction there. Should be fixed on 9b3b16f64dbcce1920f18252daa19b07ba3ad14d.

Oh, you're faster than my stack trace unwinding code :D cool… When I can use the new version?

Just published 0.9.2 which should include the fix.

Like A Boss!


Add a guild cd command

It would be really useful to have a guild cd <run_id> command (that goes to a specific run directory).
It would save me a lot of time!

Guild supports the --shell option for the open command, which does something similar. The command opens a new shell in the run directory. Alternatively use --shell-cmd <shell> to use a specific shell (default is bash).
This has the benefit of letting you return to your previous working directory at any time by running exit.

Have you had a chance to try the --shell option to open? Does that accomplish the sort of thing you're looking for?
The issue I see with cd is that it's hard to get back to the starting directory. I don't think I'd use that myself but rather the shell technique, where I can type exit to get back.
I suppose we could implement a --pop option and use pushd and popd to implement this.
Any thoughts?


Clarification of Points earned?

I assume that you will still get points even if the contact you make is to someone NOT participating in the roundabout (so and valid contact with another Ham using that repeater is valid).  That makes it a little less daunting - so we can just call CQ on the repeater and make any contact with a monitoring station to score.
Should the home page make this clear?

Done.


MKL Breaking Existing Functionality

Building against MKL-DNN breaks certain existing functionality. For example: the Activation symbol with "relu" crashes during a forward pass whenever the rank of the input is not 4.
Are there any other areas where linking against MKL-DNN breaks existing functionality? And is this expected behaviour, or is this a bug?
@piiswrong, @glingyan, @taliesinb

Hi  Sebastian:
Young Jin will support.
What MKL-DNN, which library and what kind of case?
From: Sebastian Bodenstein<EMAIL_ADDRESS>Sent: Thursday, June 8, 2017 10:47 PM
To: dmlc/mxnet<EMAIL_ADDRESS>Cc: Guo, Lingyan<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [dmlc/mxnet] MKL Breaking Existing Functionality (#6615)
Building against MKL-DNN breaks certain existing functionality. For example: the Activation symbol with "relu" crashes during a forward pass whenever the rank of the input is not 4.
Are there any other areas where linking against MKL-DNN breaks existing functionality? And is this expected behaviour, or is this a bug?
@piiswronghttps://github.com/piiswrong, @glingyanhttps://github.com/glingyan, @taliesinbhttps://github.com/taliesinb
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dmlc/mxnet/issues/6615, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AVcrhYf09Q5c_QDv5iHgQtHCN1XwyTE7ks5sCAlHgaJpZM4N0LZs.

Hi @sbodenstein ,
Are you trying to use this?
https://github.com/01org/mkl-dnn
If so, it is under active development and will be available some time later.
If not, could you please let me know the problem more details?
Thanks.

@sbodenstein when I try to use padding for convolution if mxnet is linked against mkl it crashes. just one more example of some buggy mkl related area.

@sbodenstein Could you provide some more details? Are you referring to the dimension of the tensor? If so, in what dimension does that crash?

@ykim362:
a = mx.sym.Activation(act_type="relu", data = mx.sym.Variable('a'))
e = a.bind(mx.cpu(), {'a': mx.nd.array([1,2])})
y = e.forward()

produces:
[18:26:50] /mxnet/dmlc-core/include/dmlc/logging.h:304: [18:26:50] /mxnet/nnvm/include/nnvm/tuple.h:446: Check failed: dim == static_cast<int>(ndim()) (4 vs. 1) dimension do not match target dimension 4 vs 1

@sergeykolychev: could you give an explicit example that fails for you?

Thanks, @sbodenstein !
I will investigate the case.

@sbodenstein ,
Currently, MKL relu is only enabled for 2D, 3D and 4D tensors.
Even though 1D tensor can be supported by MKL, it hasn't been enabled, yet. Above example is 1D tensor case.
I will fix two and make a pull request.
(1) Correctly support 1D tensor.
(2) In the case of non-supported dimensions (higher than 4D), make use of the default 3rd party library.
I will also look into the way how to support higher than 4-d tensors.
I am sorry for the inconvenience.
Thanks.

(1) is merged. (2) will be integrated when we are switching to MKL-DNN.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


fix: Remove Hashbang from URL

Type of modification

[ ] Breaking change
[ ] New Feature
[X] Bug Fix
[ ] Chore (refactor, documentation, tests... all the changes with no impact on ARA functionalities.)

Changes description
Remove the hasbang from the url
close #23
Technical description
**Switch to history mode in main.js *
PR CheckList
Please make sure your PullRequest respect all those items :

[X] Your PR's title has the prefix : feat:, fix: or chore:
[X] Unless your PR is related to a Hotfix (and approved by a ARA maintener), the targeted branch for it is the branch version/X.Y.Z and not master
[X] You have asked a review from one of the ARA maintainer in your PR.
[X] If your PR is related to an issue, add the issue's number in it.
[X] All the code you added is documented.
[X] All the code you added is tested and the  tests are in success.
[X] You already signed the Contributor License Agreement and give us the document

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


[fix][cli] Fix set topic retention policy failed

Fixes #xyz

Main Issue: #xyz

PIP: #xyz

Motivation
When using this command bin/pulsar-admin topics set-retention -s 20g -t 200d persistent://public/default/test_v1 to set a retention policy for a topic, it failed with following exception
PicocliException: Could not set value for field private java.lang.Integer org.apache.pulsar.admin.cli.CmdTopics$SetRetention.retentionTimeInSec to 17280000 while processing argument at or before arg[5] '200d' in [topics, set-retention, -s, 20g, -t, 200d, persistent://public/default/test_v1]: picocli.CommandLine$PicocliException: Could not set value for field private java.lang.Integer org.apache.pulsar.admin.cli.CmdTopics$SetRetention.retentionTimeInSec to 17280000
Usage: pulsar-admin topics set-retention [-hv] -s=<sizeLimit>
       -t=<retentionTimeInSec> <topicName>
Set the retention policy for a topic
      <topicName>          persistent://tenant/namespace/topic
  -h, --help               Show this help message and exit.
  -s, --size=<sizeLimit>   Retention size limit with optional size unit suffix.
                             For example, 4096, 10M, 16G, 3T.  The size unit
                             suffix character can be k/K, m/M, g/G, or t/T.  If
                             the size unit suffix is not specified, the default
                             unit is bytes. 0 or less than 1MB means no
                             retention and -1 means infinite size retention
  -t, --time=<retentionTimeInSec>
                           Retention time with optional time unit suffix. For
                             example, 100m, 3h, 2d, 5w. If the time unit is not
                             specified, the default unit is seconds. For
                             example, -t 120 will set retention to 2 minutes. 0
                             means no retention and -1 means infinite time
                             retention.
  -v, --version            Print version information and exit.

It is caused by cast Long to Integer failed
https://github.com/apache/pulsar/blob/28e47fa99dcb080385cfa567b844e74ab0cd85b1/pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdTopics.java#L1854-L1855
https://github.com/apache/pulsar/blob/28e47fa99dcb080385cfa567b844e74ab0cd85b1/pulsar-cli-utils/src/main/java/org/apache/pulsar/cli/converters/picocli/TimeUnitToSecondsConverter.java#L28-L35
Modifications
Change the type of retentionTimeInSec from Integer to Long
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

@hangc0276 Good catch! There seems to also be a problem with sizeLimit. It's an integer and could overflow. Instead of ByteUnitToIntegerConverter, the ByteUnitToLongConverter should be used.

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

Project coverage is 74.38%. Comparing base (bbc6224) to head (ca31316).
Report is 777 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #23688      +/-   ##
============================================
+ Coverage     73.57%   74.38%   +0.80%
- Complexity    32624    35064    +2440
============================================
  Files          1877     1945      +68
  Lines        139502   147448    +7946
  Branches      15299    16270     +971
============================================
+ Hits         102638   109678    +7040
- Misses        28908    29287     +379
- Partials       7956     8483     +527

Flag
Coverage Δ

inttests
27.26% <ø> (+2.67%)
:arrow_up:

systests
24.37% <ø> (+0.05%)
:arrow_up:

unittests
73.76% <ø> (+0.91%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines
Coverage Δ

...in/java/org/apache/pulsar/admin/cli/CmdTopics.java
79.98% <ø> (-1.20%)
:arrow_down:

... and 667 files with indirect coverage changes

@hangc0276 Good catch! There seems to also be a problem with sizeLimit. It's an integer and could overflow. Instead of ByteUnitToIntegerConverter, the ByteUnitToLongConverter should be used.

Addressed in #23689. It also covers changes to retentionTimeInSec overflow fix made in this PR. @hangc0276 please review #23689

It seems that this PR applies only to branch-3.3 and branch-3.4. The bug was most likely introduced in PIP-280 and PIP-343 changes.


Inconsistency in Parsing: SequenceExprSyntax vs InfixOperatorExprSyntax in SwiftSyntax and Compile

Issue Kind
I Don’t Know
Source Code
func testMacroWithInfixOperatorExpr() {
  final class Visitor: SyntaxRewriter {
    override func visit(
      _ node: InfixOperatorExprSyntax
    ) -> ExprSyntax {
      ExprSyntax(node.with(\.rightOperand, "2"))
    }
  }
  struct Macro: ExpressionMacro {
    static func expansion(
      of node: some FreestandingMacroExpansionSyntax,
      in context: some MacroExpansionContext
    ) throws -> ExprSyntax {
      let visitor = Visitor()
      let rewritedNode = visitor.rewrite(node)

      return rewritedNode.asProtocol(FreestandingMacroExpansionSyntax.self)?.argumentList.first?.expression ?? ""
    }
  }

  assertMacroExpansion(
    """
    #MyMacro(x + 1)
    """,
    expandedSource: """
    x + 2
    """,
    macros: ["MyMacro": Macro.self]
  )
}

Description
If I'm not mistaken the expression x + 1 is parsed differently by SwiftSyntax and the Swift compiler. In SwiftSyntax, it is represented as a SequenceExprSyntax, while the compiler parses it as an InfixOperatorExprSyntax.
Here is how the compiler expands the above macro into the following Swift expression:
x + 1

However, when attempting to test this macro using SwiftSyntax's MacroSystem, it fails to expand in the same manner as the compiler.

Tracked in Apple’s issue tracker as rdar://114963932

This is a duplicate of https://github.com/apple/swift-syntax/issues/2128. Closing so we are only tracking the issue once.

Not a problem at all, please file issues. I’ve got the better overview of which other open issues there are and am always happy to close ones that we are already tracking.


Season overlays have issues when a series does not have specified season posters

Version Number
1.18.1
What branch are you on?
master
Describe the Bug
As a preface, there are some shows that do not have a season poster for whatever reason. Plex defaults by using the main show poster for the seasons when this is the case. The problem is, after applying overlays, the seasons now have all the overlays that were placed on the main show poster. This is undesirable in cases where people do not want any overlays on season posters, or where it varies from the main show. Not sure if there is a check that can be built into PMM so that if a show has seasons which don't have their own posters, a copy of the original main poster will be placed in there, and then the normal process would proceed.
Relevant Collection/Overlay/Playlist Definition
No response
Logs
No response

fallback to the show poster has been added to nightly


GitHub actions never fail

The tool works really well & I'd love to adopt it across the board, however, it doesn't matter what I put into the:
soft_fail:
setting it never fails & never feeds back to the pull request. The snip of the output of the tool shows this:
"ARM/" "" "CKV_AZURE_17" "true" "false" "arm" "" "" "github_failed_only" "true" "WARNING"
input_soft_fail:false
running checkov on directory: ARM/
Which does indicate that the entrypoint.sh is getting the setting correctly, but it just doesn't feedback a failure.

I have a similar issue: the output shows many errors, I have soft_fail: false but still both the job and the step where the action is used show as successful.
Is there a way to force the action to make the job/step fail for failures found?

Another tip to make it more friendly: The tool should also generate and upload the SARIF file. This way the security vulnerabilities are captured in the securities tab.

@dhishan The SARIF upload functionality was added since your comment as I'm using this successfully to get the alerts in my GitHub repo's Security tab.

@HariSekhon Can you point any doc that show how to to do the upload?

support for SARIF output was added quite a while ago and it works correctly for a while.


Add CI/CD

Does Gitub Actions support Erlang yet? If so, great. If not, set up for Travis CI ...

Github Actions does support Erlang; testing it out now ...

Verified as running successfully.


docker-compose does not start the app

Steps to reproduce:

clone the repo (fresh clone)
follow README - run docker-compose up. The output is as below:

Traceback (most recent call last):
  File "urllib3/connectionpool.py", line 670, in urlopen
  File "urllib3/connectionpool.py", line 392, in _make_request
  File "http/client.py", line 1255, in request
  File "http/client.py", line 1301, in _send_request
  File "http/client.py", line 1250, in endheaders
  File "http/client.py", line 1010, in _send_output
  File "http/client.py", line 950, in send
  File "docker/transport/unixconn.py", line 43, in connect
FileNotFoundError: [Errno 2] No such file or directory

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "requests/adapters.py", line 439, in send
  File "urllib3/connectionpool.py", line 726, in urlopen
  File "urllib3/util/retry.py", line 410, in increment
  File "urllib3/packages/six.py", line 734, in reraise
  File "urllib3/connectionpool.py", line 670, in urlopen
  File "urllib3/connectionpool.py", line 392, in _make_request
  File "http/client.py", line 1255, in request
  File "http/client.py", line 1301, in _send_request
  File "http/client.py", line 1250, in endheaders
  File "http/client.py", line 1010, in _send_output
  File "http/client.py", line 950, in send
  File "docker/transport/unixconn.py", line 43, in connect
urllib3.exceptions.ProtocolError: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "docker/api/client.py", line 214, in _retrieve_server_version
  File "docker/api/daemon.py", line 181, in version
  File "docker/utils/decorators.py", line 46, in inner
  File "docker/api/client.py", line 237, in _get
  File "requests/sessions.py", line 543, in get
  File "requests/sessions.py", line 530, in request
  File "requests/sessions.py", line 643, in send
  File "requests/adapters.py", line 498, in send
requests.exceptions.ConnectionError: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "docker-compose", line 3, in <module>
  File "compose/cli/main.py", line 81, in main
  File "compose/cli/main.py", line 200, in perform_command
  File "compose/cli/command.py", line 60, in project_from_options
  File "compose/cli/command.py", line 152, in get_project
  File "compose/cli/docker_client.py", line 41, in get_client
  File "compose/cli/docker_client.py", line 170, in docker_client
  File "docker/api/client.py", line 197, in __init__
  File "docker/api/client.py", line 221, in _retrieve_server_version
docker.errors.DockerException: Error while fetching server API version: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))
[7840] Failed to execute script docker-compose

docker-compose version: docker-compose version 1.29.2, build 5becea4c
docker version: Docker version 20.10.14, build a224086

I'm an idiot, docker was not running. Closing.


 should not accept the first suggestion

Did you check the docs?

[X] I have read all the LazyVim docs

Is your feature request related to a problem? Please describe.
I tried this with docker, setting the syntax to ruby.

If at this point you type <Enter> to start writing the content new method foo, foobar gets selected from the suggestions. and you end up with def foobar
Describe the solution you'd like
I think it is not possible to always know if the user wants to accept the first suggestion or type <Enter>. Even if it may not happen a lot, for me personally it is not acceptable to (sometimes) add unexpected content.
I would purpose that for the user to select the suggestion, the user has to type something, same as if you'd like to select the second suggestion. Maybe it could even be a configurable option?
Describe alternatives you've considered
Typing <space> removes the suggestion, but leaves you with a space. I haven't seen if there's another way to cancel the suggestions, but I would still prefer that <Enter> works normally.
Additional context
Thanks for the amazing work you're doing with LazyVim!

You could press Ctrl-e which would close the selection menu if you're happy with what you've typed so far. Another way would be to disable the preselection, but then you would have to use C-n/C-p to select an item. Depends how you want to do it.

Why is this a bug? Change it however you want in nvim-cmp

No one said it's a bug, it's tagged as an enhancement. I think it's a better default that content doesn't get inserted unexpectedly and that if you want to type enter you don't need to dismiss the completion

No one said it's a bug, it's tagged as an enhancement. I think it's a better default that content doesn't get inserted unexpectedly.
The configuration for the mappings is done in LazyVim config, so I filed the issue here. I'll give it a try to configure it myself then.

Lazyvim does not change nvim-cmp default mappings, it just adds some more for selecting next/previous item. C-e is default mapping for aborting selection, so you can use that as soon as you've typed whatever you want without accepting the selection nvim-cmp is suggesting. The other way would be to disable nvim-cmp preselection with the drawback i mentioned in my previous post. You can achieve that as is done here. I personally prefer to disable preselection.


project environment

using  pip install -r requirement.txt，got this error
Could not find a version that satisfies the requirement torch>=1.1 (from pytorch-lightning->-r requirements.txt (line 1)) (from ver sions: 0.1.2, 0.1.2.post1, 0.1.2.post2) No matching distribution found for torch>=1.1 (from pytorch-lightning->-r requirements.txt (line 1))

Which Python version are you using?
BTW I will be cleaning up the repo folder further this week ; )

I think you might be using Windows -- is this issue related? https://github.com/pytorch/pytorch/issues/18384

I think you might be using Windows -- is this issue related? pytorch/pytorch#18384

yeah，windows10, so ur system version?Ubuntu 16.04 or other

Yes. You might want to try running on Linux or follow the official instructions to install PyTorch on Windows.

`# download the checkpoint
wget
to report CCC on the validation set
python3 eval.py --test_on_val --checkpoint m3t_mtl-vox2.pt
python3 get_smoothed_ccc predictions_val.pt
to generate test set predictions
python3 eval.py --checkpoint m3t_mtl-vox2.pt`
the weget link？


Remove duplicate context

Don't need response data type. Context can be returned as a list of chunks.
https://github.com/trustwiseai/trustwise/blob/c1573943929780893d143e4a5667a53fcc0ade0e/trustwise/request.py#L15-L25

Removed and Refactored the data models


Add function to create a new HMC (or NUTS) state

In order to initialize the HMC state we need to compute the potential energy at the initial position and the gradient of the potential energy at the initial position so we need to write the following every time we  use the HMC kernel:
q = at.vector('q')
potential_energy = -logprob_fn(q)
potential_energy_grad = aesara.grad(potential_energy, q)

Here we add a short helper function hmc.new_state that computes the initial state given the initial postiion and logprob function. Closes #22.
Changes

Update hmc.py to add a hmc.new_state function
Update nuts.py to add an alias to hmc.new_state, nuts.new_state
Update HMC.ipynb to be compatible with aeppl's latest version
Update HMC.ipynb to use the new hmc.new_state function
Update test_hmc.py to use the new hmc.new_state function
Update the README to use the new hmc.new_state function

I get a warning in the example notebook; Will mark the PR as ready for review when I understand where this comes from.

I get the same warning with the following code:
import aesara
from aesara import tensor as at
from aesara.tensor.random.utils import RandomStream

from aeppl import joint_logprob

from aehmc import hmc

# A simple normal distribution
Y_rv = at.random.normal(0, 1)

def logprob_fn(y):
  logprob = joint_logprob({Y_rv: y})
  return logprob

# Build the transition kernel
srng = RandomStream(seed=0)
kernel = hmc.kernel(
    srng,
    logprob_fn,
    step_size=1e-4,
    inverse_mass_matrix=at.as_tensor(1.0),
    num_integration_steps=10,
)

# Compile a function that updates the chain
y_vv = Y_rv.clone()
pe = -logprob_fn(y_vv)
peg = aesara.grad(pe, y_vv)

trajectory, updates = aesara.scan(
    fn=kernel,
    outputs_info=[
        {"initial": y_vv},
        {"initial": pe},
        {"initial": peg},
    ],
    n_steps=100
)
generator = aesara.function((y_vv,), trajectory, updates=updates)
print(generator(0.0))

As suspected at the beginning the warning comes from the changes in #25. Indeed if I revert this line from shape = shape_tuple(inverse_mass_matrix)[:1] back to shape = shape_tuple(inverse_mass_matrix)[0] the warning disappears.
The issue is that shape = shape_tuple(inverse_mass_matrix)[0] raises an exception when the inverse mass matrix, as it should. I thus updated the code to use shape = () when the inverse mass matrix is a scalar, shape = shape_tuple(inverse_mass_matrix)[0] otherwise.
Now the infer_shape warning is only returned for scalar inverse mass matrices! @brandonwillard any clue what I am doing wrong there?

Now the infer_shape warning is only returned for scalar inverse mass matrices! @brandonwillard any clue what I am doing wrong there?

Which test produces this warning?  I'm not seeing it when I run the test suite.

The following code produces the warning on my machine:
import aesara
from aesara import tensor as at
from aesara.tensor.random.utils import RandomStream

from aeppl import joint_logprob

from aehmc import hmc
from aehmc.metrics import gaussian_metric

# A simple normal distribution
Y_rv = at.random.normal(0, 1, size=2, name="Y")

def logprob_fn(y):
  logprob = joint_logprob({Y_rv: y})
  return logprob

# Build the transition kernel
srng = RandomStream(seed=0)
kernel = hmc.kernel(
    srng,
    logprob_fn,
    step_size=1e-4,
    inverse_mass_matrix=at.ones(2),
    num_integration_steps=10,
)

# Compile a function that updates the chain
y_vv = Y_rv.clone()
y_vv.name = "y_vv"

pe = -logprob_fn(y_vv)
pe.name="potential energy"

peg = aesara.grad(pe, y_vv)
peg.name="potential energy grad"

trajectory, updates = aesara.scan(
    fn=kernel,
    outputs_info=[
        {"initial": y_vv},
        {"initial": pe},
        {"initial": peg},
    ],
    n_steps=100
)

generator = aesara.function((y_vv,), trajectory, updates=updates)

as well as if I set Y_rv = at.random.normal(0,1) and inverse_mass_matrix = at.as_tensor(1.) instead.

Reproduce the warning
The following code produces the warning on my machine:
import aesara
from aesara import tensor as at
from aesara.tensor.random.utils import RandomStream

from aeppl import joint_logprob

from aehmc import hmc
from aehmc.metrics import gaussian_metric

# A simple normal distribution
Y_rv = at.random.normal(0, 1, size=2, name="Y")

def logprob_fn(y):
  logprob = joint_logprob({Y_rv: y})
  return logprob

# Build the transition kernel
srng = RandomStream(seed=0)
kernel = hmc.kernel(
    srng,
    logprob_fn,
    step_size=1e-4,
    inverse_mass_matrix=at.ones(2),
    num_integration_steps=10,
)

# Compile a function that updates the chain
y_vv = Y_rv.clone()
y_vv.name = "y_vv"

pe = -logprob_fn(y_vv)
pe.name="potential energy"

peg = aesara.grad(pe, y_vv)
peg.name="potential energy grad"

trajectory, updates = aesara.scan(
    fn=kernel,
    outputs_info=[
        {"initial": y_vv},
        {"initial": pe},
        {"initial": peg},
    ],
    n_steps=100
)

generator = aesara.function((y_vv,), trajectory, updates=updates)

as well as if I set Y_rv = at.random.normal(0,1) and inverse_mass_matrix = at.as_tensor(1.) instead. Note that before your commit the scalar version produced a warning while the above snippet did not.
Error
If I set aesara.config.on_shape_error='raise' I get the following stack trace:
Stack trace
and if I dprint the trajectory variable:
trajectory graph

Looks like we need some coverage for this in the tests.

Perhaps we could merge this (since the warning is not introduced by the new code) and open a new PR where we set the flag on_shape_error to raise?


Erro ao instalar o composer.

Acessei atraves do putty meu site. e cheguei ate a pasta do mapos, rodei o composer install porem gerou o erro, aguem poderia me ajudar ????? resumindo preciso rodar o composer dentro da pasta mapos em minha hospedagem, obs já esta liberado o acesso ssh na hospedagem.
Problem 1
- Installation request for mpdf/mpdf v8.0.5 -> satisfiable by mpdf/mpdf[v8.0.5].
- mpdf/mpdf v8.0.5 requires ext-gd * -> the requested PHP extension gd is missing from your system.
To enable extensions, verify that they are enabled in your .ini files:
- /opt/alt/php73/etc/php.ini
- /opt/alt/php73/link/conf/alt_php.ini
You can also run php --ini inside terminal to see which files are used by PHP in CLI mode.

Acessei atraves do putty meu site. e cheguei ate a pasta do mapos, rodei o composer install porem gerou o erro, aguem poderia me ajudar ????? resumindo preciso rodar o composer dentro da pasta mapos em minha hospedagem, obs já esta liberado o acesso ssh na hospedagem.
Problem 1

Installation request for mpdf/mpdf v8.0.5 -> satisfiable by mpdf/mpdf[v8.0.5].
mpdf/mpdf v8.0.5 requires ext-gd * -> the requested PHP extension gd is missing from your system.

To enable extensions, verify that they are enabled in your .ini files:

/opt/alt/php73/etc/php.ini
/opt/alt/php73/link/conf/alt_php.ini
You can also run php --ini inside terminal to see which files are used by PHP in CLI mode.

resolvido era a versao do php


Why is there onEnter* fired on state machine initialization?

Can that be disabled?
If I am setting existing state, machine is already in, why is there event fired?
Can that behaviour be disabled?
I can see I got there from "none", but, I do not want to have if(lifecycle.from == "none) return; in each event handler...

When the machine is loaded it will do a 'init'-transition and enter the state defined as 'init' in the constructor. Maybe what you are looking for is the goto to set a state without transition?

When the machine is loaded it will do a 'init'-transition and enter the state defined as 'init' in the constructor. Maybe what you are looking for is the goto to set a state without transition?

The way I use it, it is persistant state machine. I do save it to DB after each change, and load it from DB, every time I want to apply transition, so it is not great to have to check if it is init transition, on every state change and if it is, ignore it

The way I use it, it is persistant state machine. I do save it to DB after each change, and load it from DB, every time I want to apply transition, so it is not great to have to check if it is init transition, on every state change and if it is, ignore it


Perhaps a more correct calculation sizeof for memcpy()

@matttbe, @mjmartineau,
am I correct in assuming that in this code, memcpy has different size and it possible underflow or overflow?
I don't know code perfectly, so I'm asking you. If I make a mistake, cancel PR changes.

Thanks for PR, too!  It's always good to have more eyes on the code.  Much appreciated!

@GermanAizek thank you for this PR.
@ossama-othman thank you for the complete reply! I agree with you, we cannot replace the sizeof() for IPv6.
I guess we can then close this PR.


Docker Options Ignored

Example portion of docker-compose file for kafka-ui
  kafka-ui:
    image: provectuslabs/kafka-ui
    restart: unless-stopped
    environment:
      KAFKA_CLUSTERS_0_NAME: local
      KAFKA_CLUSTERS_0_BOOTSTRAPSERVERS: kafka:9092
      KAFKA_CLUSTERS_0_ZOOKEEPER: zookeeper:2181

docker-compose logs kafka-ui
app-kafka-ui-1  | Caused by: org.apache.kafka.common.errors.TimeoutException: Timed out waiting for a node assignment. Call: fetchMetadata
app-kafka-ui-1  | 2022-10-01 22:45:00,048 WARN  [kafka-admin-client-thread | adminclient-2] o.a.k.c.NetworkClient: [AdminClient clientId=adminclient-2] Connection to node 1 (/<IP_ADDRESS>:9092) could not be established. Broker may not be available.
app-kafka-ui-1  | 2022-10-01 22:45:00,951 WARN  [kafka-admin-client-thread | adminclient-2] o.a.k.c.NetworkClient: [AdminClient clientId=adminclient-2] Connection to node 1 (/<IP_ADDRESS>:9092) could not be established. Broker may not be available.
app-kafka-ui-1  | 2022-10-01 22:45:02,054 WARN  [kafka-admin-client-thread | adminclient-2] o.a.k.c.NetworkClient: [AdminClient clientId=adminclient-2] Connection to node 1 (/<IP_ADDRESS>:9092) could not be established. Broker may not be available.
app-kafka-ui-1  | 2022-10-01 22:45:02,956 WARN  [kafka-admin-client-thread | adminclient-2] o.a.k.c.NetworkClient: [AdminClient clientId=adminclient-2] Connection to node 1 (/<IP_ADDRESS>:9092) could not be established. Broker may not be available.
app-kafka-ui-1  | 2022-10-01 22:45:03,959 WARN  [kafka-admin-client-thread | adminclient-2] o.a.k.c.NetworkClient: [AdminClient clientId=adminclient-2] Connection to node 1 (/<IP_ADDRESS>:9092) could not be established. Broker may not be available.
app-kafka-ui-1  | 2022-10-01 22:45:04,861 WARN  [kafka-admin-client-thread | adminclient-2] o.a.k.c.NetworkClient: [AdminClient clientId=adminclient-2] Connection to node 1 (/<IP_ADDRESS>:9092) could not be established. Broker may not be available.
app-kafka-ui-1  | 2022-10-01 22:45:06,064 WARN  [kafka-admin-client-thread | adminclient-2] o.a.k.c.NetworkClient: [AdminClient clientId=adminclient-2] Connection to node 1 (/<IP_ADDRESS>:9092) could not be established. Broker may not be available.
app-kafka-ui-1  | 2022-10-01 22:45:07,268 WARN  [kafka-admin-client-thread | adminclient-2] o.a.k.c.NetworkClient: [AdminClient clientId=adminclient-2] Connection to node 1 (/<IP_ADDRESS>:9092) could not be established. Broker may not be available.
app-kafka-ui-1  | 2022-10-01 22:45:08,270 WARN  [kafka-admin-client-thread | adminclient-2] o.a.k.c.NetworkClient: [AdminClient clientId=adminclient-2] Connection to node 1 (/<IP_ADDRESS>:9092) could not be established. Broker may not be available.
app-kafka-ui-1  | 2022-10-01 22:45:09,373 WARN  [kafka-admin-client-thread | adminclient-2] o.a.k.c.NetworkClient: [AdminClient clientId=adminclient-2] Connection to node 1 (/<IP_ADDRESS>:9092) could not be established. Broker may not be available.
app-kafka-ui-1  | 2022-10-01 22:45:10,376 WARN  [kafka-admin-client-thread | adminclient-2] o.a.k.c.NetworkClient: [AdminClient clientId=adminclient-2] Connection to node 1 (/<IP_ADDRESS>:9092) could not be established. Broker may not be available.
Based on the documentation from my understanding it should be trying to connect to kafka:9092 not  <IP_ADDRESS>:9092.
It's like it completely ignores options for
KAFKA_CLUSTERS_0_NAME: local unless It is just trying to connect to slot 1 as well as zero and something else isn't working properly.

Hi, please share the full compose file with kafka settings in particular.

Hello good morning same problem
https://github.com/jSierraB3991/dotfiles/blob/main/Docker/kafka-compoe-with-ui.yml
`version: '3'
services:
zookeeper:
image: wurstmeister/zookeeper
container_name: zookeeper
ports:
- 2181:2181
kafka:
image: wurstmeister/kafka
container_name: kafka
depends_on:
- zookeeper
ports:
- 9092:9092
environment:
KAFKA_ADVERTISED_HOST_NAME: localhost
KAFKA_ZOOKEEPER_CONNECT: zookeeper:2181
kafka-ui:
image: provectuslabs/kafka-ui
container_name:  kafka-ui
ports:
- 9094:8080
depends_on:
- zookeeper
- kafka
restart: always
environment:
KAFKA_CLUSTERS_0_NAME: local
KAFKA_CLUSTERS_0_NAME: kafka
KAFKA_CLUSTERS_0_BOOTSTRAPSERVERS: kafka:9092
KAFKA_CLUSTERS_0_ZOOKEEPER: zookeeper:2181`

Hello good morning same problem
https://github.com/jSierraB3991/dotfiles/blob/main/Docker/kafka-compoe-with-ui.yml

Work
https://github.com/jSierraB3991/scripts/blob/main/bash/run_images_docker_host.sh#L152

@jSierraB3991 that won't work because your advertised host name is "localhost", which is invalid because you're connecting from another docker container.
Please take a look at our examples, there are plenty, e.g. with kafka in kraft mode. KAFKA_ADVERTISED_LISTENERS: PLAINTEXT://kafka0:29092,PLAINTEXT_HOST://localhost:9092


@conda-forge-admin, please update circle

Hi, I updated the circle-ci deploy key.

@isuruf this has not helped as it seems :(

@isuruf ping!


Api to get bank recent transaction

I am interested in this task. Can you please assign it to me ?


'Ring' cannot authenticate passwords

On v8-beta1, ring.go had 'depreciated features' removed. One of them was that password was transferred in the clientOptions() method from the RingOptions to the Options struct. As a result, when using a ring, an AUTH command is never sent even when a password was provided on the Ring declaration.
go-redis/cache seems to require the use of a Ring, but I can't use Cache at all now because I require a password on my redis server.
I can confirm that adding "Password: opt.Password," as line 124 in ring.go fixes the issue.
Why was this removed? What is the alternative?

That is an overlook - thanks for fixing.


page.pages[0].evaluate is not a function

page.pages[0].evaluate need to act similar as page.evaluate
Actual behavior
it doesnot recognise the function evaluate inside page.pages[0]
Steps to reproduce the behavior

create a pages inside page
inject js inside one tab
then inject another js inside another tab

OS: mac and ubuntu
Node version (node -v): 6.9.5
package version: 4.0.1

pages isn't even a property that exists. The functionality you are looking for must be added via a PR.

Thanks @elias-winberg. This a duplicate of #614. I am not sure what was the purpose of opening a duplicate ticket. As others have suggested, and I have suggested in #614, this functionality is not implemented right now. It would be very hard for page.pages[0] to work exactly as is because all calls are proxied to another process.
Please send a PR if you like to see this implemented. I can help you get it tested, approved and deployed. Most likely something likely, it would need to be page.pages(0) instead of [0] because node doesn't have access to inner array of phantom.


v1.1.0 release notes

release under discussion

Should the release notes have an extra note regarding https://github.com/apache/pekko-connectors/pull/314, as it is a breaking change?
At least from the wording in the changelog (Remove duplicate scopes value in reference.conf) it was not entirely obvious to me that this could break applications that were loading the scope from the service-account.scopes.

@mdedetrich what do you think about this being a breaking change?
I guess we can move this into its own 'Potentially Breaking Changes' section and try to provide more details about who might see issues.


WebUSB should auto-connect

Right now we have to manually reconnect every time we refresh the page. Device should reconnect on page reloads ...

Solved by https://github.com/arduino-bitcoin/simple_hardware_wallet/pull/28


on shutdown, TCPSite gets unregistered twice and the second time throws an exception

From @lyoshenka  - said he'd work on it.
I also see this on Jack's branch torba reconnect, not sure if related:
2019-07-08 10:59:56,125 INFO     lbry.blob_exchange.server:124: Blob server listening on TCP <IP_ADDRESS>:3333
Unhandled error in exception handler
context: {'message': 'Task was destroyed but it is pending!', 'task': <Task pending coro=<Event.wait() running at C:\Users\thoma\AppData\Local\Programs\Python\Python37-32\lib\asyncio\locks.py:293> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x06DFAD90>()]> cb=[_wait.<locals>._on_completion() at C:\Users\thoma\AppData\Local\Programs\Python\Python37-32\lib\asyncio\tasks.py:440]>}
Traceback (most recent call last):
  File "C:\Users\thoma\AppData\Local\Programs\Python\Python37-32\lib\asyncio\base_events.py", line 1643, in call_exception_handler
    self._exception_handler(self, context)
  File "c:\users\thoma\documents\lbry\torba\torba\client\basenetwork.py", line 58, in protocol_exception_handler
    exception = context['exception']
KeyError: 'exception'

fixed in #2297


build process packages up old versions of MistServer somehow

Check out this build. It's for commit cc9d1881159057f782222e725d03a8f3dd047634, right? Here's manifest.yaml for that commit, showing MistServer version DDVTECH/mistserver@2c3331192168d1c68863aef7320b60cd920d8baa.
However, the build itself downloads DDVTECH/mistserver@52e1cb9c35555ca67ab215d3f80919b344fb0c32. This shows up in the logs when it does verification:
#24 120.5 I0815 18:17:07.185310     474 downloader.go:67] Doing SHA checksum verification for service=mistserver
#24 120.7 I0815 18:17:07.375632     474 shasum.go:13] Verifying shasum with file=52e1cb9c35555ca67ab215d3f80919b344fb0c32_checksums.txt

I'm guessing the offending error is this, occurring right at the git clone step?
fatal: Not a valid object name cc9d1881159057f782222e725d03a8f3dd047634^{commit}

My initial thought was that it might be timing issue, where the mist builds had not yet succeeded and catalyst build was triggered. However, we push the updated manifest only after the binaries have been successfully compressed and uploaded to bucket; which in-turn created the auto-commit.
The other alternative "could" be that github action fetched the old copy of manifest (is there cloudflare caching on build.livepeer.live domain?)
The 3rd possibility i can think of is that the staging branch was merged pretty quick which led to the builder (which uses github URL for context) wasn't able to fetch information about it? This, however, shouldn't affect the make build pipeline at all.
Creating a PR with more logging context. I'll be observing the builds from auto-commits and their merges closely.

This seems to have been a caching issue. A fix was deployed to cloudflare yesterday; and the worker will be moved to wrangler with this open PR: livepeer/livepeer-worker#6

@hjpotter92 If the related PR is merged, can we close this issue?


Color parsing and display should be based on filetype and directory

Describe the bug
In TypeScript and JavaScript files, as well as several other file types, the expected color format would be #RRGGBBAA (or even #RGBA for shorthand notation, but this is more rare). This package seems to assume Android color format, which is #AARRGGBB. The result of this is that colors opaque colors highlight well in all contexts, and alpha colors in the Android dev context, but colors with alpha don't really make sense in other contexts.
This obviously isn't ever going to be perfect. Kotlin and Java files (and related XML) can be defined with colors that are designed to interface with w/ other environments like web, Node, or other languages. However, it should be reasonable to infer the default context based on filetype and location.
I would suggest the default should be #RRGGBBAA. This way it's Android's way and everything else. You could then parse as #AARRGGBB if you're in a non TS/JS file that's within android/src/main.
You could also do the inverse, and parse #RRGGBBAA only if you're in a JS/TS file, but this is less versatile because anything defined in JSON, XML, YAML in a web / node or other project wouldn't parse correctly.
To Reproduce

Create a JS/TS file
const RED_50 = '#FF000080'

Should look like full red at 50% opacity. Instead looks like dark blue at full opacity.
Expected Behavior
Android context colors parse with alpha parse differently based on context. JS/TS and other files that define HEXA should parse and display correctly.

Environment
IntelliJ IDEA 2023.1.3 (Ultimate Edition)
Build #IU-231.9161.38, built on June 19, 2023
Runtime version: 17.0.7+10-b829.16 x86_64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
macOS 13.5
GC: G1 Young Generation, G1 Old Generation
Memory: 2048M
Cores: 16
Metal Rendering is ON
Registry:
    editor.injected.highlighting.enabled=false
    debugger.new.tool.window.layout=true
    editor.distraction.free.mode=true
    ide.tooltip.initialDelay=504
    ide.experimental.ui=true
    ide.balloon.shadow.size=0
    ide.images.show.chessboard=true

Non-Bundled Plugins:
    monokai-pro (1.10)
    String Manipulation (9.9.0)
    com.mallowigi.colorHighlighter (15.0.0)
    izhangzhihao.rainbow.brackets (2023.2.11)

Kotlin: 231-1.8.21-IJ9161.38

Screenshots
50% red as defined using the "Android" style

50% red defined using HEXA

You know that you can switch the format in the settings, right? However I like the idea of checking the context. I don't know yet if this is possible in the current implementation, but this is a clue I'll try to investigate

Going back to this discussion. After consideration, I will leave the setting decide whether to use argb or rgba, but will change the default to rgba.
The reason is that it's not necessarily the case. Unlike Color(r,g,b,a) or NSColor or rgba(r,g,b,a) that are of course straightforward, hex colors are agnostic. True, CSS uses #rgba (or #rrggbbaa) and android uses #argb, but JS, TS, Python, Markdown or others are depending on the implementation.
What if you use a library that parses argb instead of rgba? What if my HTML elements use a data attribute that expects a argb color string? etc etc.


test: Modify Fatal to Fatalf as we have a specifier

Signed-off-by: Mrunal Patel<EMAIL_ADDRESS>

LGTM

/test integration

No reason to run E2E since this is only used in integration.  Merging.


feat: add a pre-commit hook

This pull request introduces a pre-commit hook for validating Browserslist configurations. The most important changes include the addition of a configuration file for the pre-commit hooks. There are also relevant additions to the README.md.
Pre-commit hook addition:

pre-commit-hooks.yaml: Defined a new hook, browserslist-lint, to validate Browserslist configurations using npx browserslist-linter. This hook is triggered by changes to .browserslistrc or package.json.

Documentation updates:

README.md: Added a section on the pre-commit hook, including instructions on how to add it to the pre-commit configuration.

This yaml implementation is a part of the pre-commit configuration. pre-commit is an open source framework that lets people deploy different packages for checks before they commit something toa. git repository. It is widely adopted and is practically an industry standard.
You can see the following pre-commit enabled repositories:

https://github.com/pre-commit/mirrors-eslint
https://github.com/rbubley/mirrors-prettier

The pre-commit-hooks.yaml file allows a pre-commit user to include browserslist-linting as a part of their workflow. It does not change anything in the package itself. An end user, such as myself, can then include this as part of their suite of pre-commit checks by the two lines as follows:
repos:
	repo: https://github.com/browserslist
	hooks:
		- id: browserslist-lint


Operator & CRD upgrade test harness

Problem statement
Currently operator has 1 E2E tests which tests the upgrade path https://github.com/Kong/gateway-operator/blob/ec5a314a0d2766f660c200562bf068129b335a50/test/e2e/upgrade_test.go#L30-L40.
This is not comprehensive enough since it only tests "latest tag" to current version upgrade path and it doesn't test CRD upgrades (is installing new version of CRDs safe and non-breaking?).
This issue tracks the effort of adding a new test suite which will test operator upgrades.
For example it could test upgrades from a known, predefined list of tags to a a known, predefined list of tags. Additionally this suite would also test installing new CRD versions so that those do not bring breakage.
Acceptance criteria

[x] New testing harness is added which tests upgrades from known operators tags to known operator tags, using a way of deployment closest to the one used by the end users.
[x] As a developer I can be sure that the changes I introduce will not break during upgrades from an older version to the version I'm developping.

https://github.com/Kong/gateway-operator/pull/256 introduces samples testing which applies and deletes sample manifests against a kind cluster on CI
https://github.com/Kong/gateway-operator/pull/257 adds mention of adding a test case if the released version if a new minor/major
https://github.com/Kong/gateway-operator/blob/9f33d27ab875b91e50d7e750b45a293c1395da2d/test/e2e/test_upgrade.go already contains tests cases against the latest release tags, against latest and nightly and against latest tag vs tip of the tested branch

Merging of those PRs will conclude this issue. Further development of upgrade tests using helm can be tracked in separate issues and PRs.


How to replace aadClientSecret?

Out current aadClientSecret expired and we have created a new one. However after updating cloud config file and restarting kubelets we still an error:
Failed to provision volume with StorageClass "standard": azure.BearerAuthorizer#WithAuthorization: Failed to refresh the Token for request to https://xxxx/disks/kubernetes-dynamic-pvc-....?api-version=2017-03-30: StatusCode=401 -- Original Error: adal: Refresh request failed. Status Code = '401'. Response body: {"error":"invalid_client","error_description":"AADSTS7000215: Invalid client secret is provided....}

Manually logging with credentials from cloud config works fine. It there any kind of cache that needs to be cleared?

Manage to workaround this issue by resetting key to same value it was before and extending expire time.


finetune chat_gpt

closes #132

Traceback (most recent call last):
  File "C:\Users\Abhigya Sodani\AppData\Local\Programs\Python\Python311\Lib\threading.py", line 1038, in _bootstrap_inner
    self.run()
  File "C:\Users\Abhigya Sodani\AppData\Local\Programs\Python\Python311\Lib\threading.py", line 975, in run
    self._target(*self._args, **self._kwargs)
  File "C:\Users\Abhigya Sodani\AppData\Local\Programs\Python\Python311\Lib\site-packages\llm_vm\completion\optimize.py", line 42, in new_thread
    t[0] = foo()
           ^^^^^
  File "C:\Users\Abhigya Sodani\AppData\Local\Programs\Python\Python311\Lib\site-packages\llm_vm\onsite_llm.py", line 474, in start
    fine_tuning_job = openai.FineTuningJob.create(training_file=upload_response.id, model='gpt-3.5-turbo')
                      ^^^^^^^^^^^^^^^^^^^^
AttributeError: module 'openai' has no attribute 'FineTuningJob'

got that error when testing

@abhigya-sodani It was working in my codespace after pip install -e .. I think your openai library does not have updates after improvements made to legacy API.
You may need to update openai package.

@abhigya-sodani just checked openai's release history, will need to change version in toml file. Will make the required changes, thanks!

Have you had a chance to change the TOML file

Hi, sorry for the late reply. I won't be able to work on it due to some
other commitments.. it's just a change in toml file to latest version.
Hopefully someone can take over.
Sorry for inconveniences.
Best,
Rakesh
On Fri, Sep 1, 2023, 12:42 AM abhigya-sodani @.***>
wrote:

Have you had a chance to change the TOML file
—
Reply to this email directly, view it on GitHub
https://github.com/anarchy-ai/LLM-VM/pull/168#issuecomment-1701617101,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK4GSXMXWTC7I4OL3CDZKHTXYDOZBANCNFSM6AAAAAA4BSYSHY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Feature :: Improve forms forms diplay

This PR mainly consists in aesthetics:

Clean duplicated CSS
Improve styling: text sizing of labels, placeholders & buttons, text color, title alignment and  autocomplete/multiselect items highlight color
Improve naming: labels & title

Before:

After:

Before:

After:

Codecov Report

Merging #222 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #222      +/-   ##
==========================================
- Coverage   99.28%   99.27%   -0.01%
==========================================
  Files          50       50
  Lines         696      692       -4
  Branches      113      111       -2
==========================================
- Hits          691      687       -4
  Misses          5        5

Impacted Files
Coverage Δ

src/components/stepforms/StepFormTitle.vue
100% <ø> (ø)
:arrow_up:

src/components/stepforms/StepForm.vue
100% <ø> (ø)
:arrow_up:

src/components/stepforms/WidgetMultiselect.vue
100% <ø> (ø)
:arrow_up:

src/components/stepforms/WidgetAutocomplete.vue
100% <ø> (ø)
:arrow_up:

src/components/stepforms/FillnaStepForm.vue
100% <100%> (ø)
:arrow_up:

src/components/stepforms/AggregateStepForm.vue
100% <100%> (ø)
:arrow_up:

src/components/stepforms/FilterStepForm.vue
100% <100%> (ø)
:arrow_up:

src/components/stepforms/WidgetAggregation.vue
100% <100%> (ø)
:arrow_up:

src/components/stepforms/ArgmaxStepForm.vue
100% <100%> (ø)
:arrow_up:

src/components/stepforms/PercentageStepForm.vue
100% <100%> (ø)
:arrow_up:

... and 6 more

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 0da76f4...a6a7b21. Read the comment docs.

One comment plus to store #a7a7a7 hex code in var and we're good, thanks for that :)

Done thanks !

Seems really good to me !
Just one thing I'm not sure if the color of the placeholder text has enough contrast for accessibility :

Update : I Check it on a web accessibility tool (source https://webaim.org/resources/contrastchecker/).
Results : it's not ok

Seems really good to me !
Just one thing I'm not sure if the color of the placeholder text has enough contrast for accessibility
Update : I Check it on a web accessibility tool

@zegonz thanks I replaced this color with a much darker grey !


Update cost of provisional driving

We list the fee for applying for a provisional as £34. This is the fee to apply online; the fee to apply by post is £43.
https://trello.com/c/H77MJZRD/477-change-fee-for-apply-for-a-provisional-licence

🚅


Cluster never comes available after moving to 9.0.0

After moving to 9.0.0, the cluster availability check in null_resource.wait_for_cluster fails to detect when the cluster becomes available.  The result is the is that it spins forever waiting.  This appears to be related to  750
Workaround is to override the wait_for_cluster_cmd and use the default value prior to 750  e.g.
wait_for_cluster_cmd  = "until curl -k -s $ENDPOINT/healthz >/dev/null; do sleep 4; done"
I'm submitting a...

[ x] bug report
[ ] feature request
[ ] support request - read the FAQ first!
[ ] kudos, thank you, warm fuzzy

What is the current behavior?
module.eks.module.cluster.null_resource.wait_for_cluster[0]: Still creating... [20s elapsed]
module.eks.module.cluster.null_resource.wait_for_cluster[0]: Still creating... [30s elapsed]
...
module.eks.module.cluster.null_resource.wait_for_cluster[0]: Still creating... [1h0m42s elapsed]
module.eks.module.cluster.null_resource.wait_for_cluster[0]: Still creating... [1h0m52s elapsed]
eventually times out
If this is a bug, how to reproduce? Please include a code sample if relevant.
module "eks" {
source          = "terraform-aws-modules/eks/aws"
version         = "9.0.0"
cluster_name    = local.eks_cluster_name
cluster_version = var.eks_k8s_version
subnets         = var.private_subnet_ids
vpc_id          = var.vpc_id
enable_irsa     = true
tags            = merge(var.eks_tags,local.env_tags)
cluster_enabled_log_types     = var.cluster_enabled_log_types
cluster_log_retention_in_days = var.cluster_log_retention_in_days
workers_additional_policies   = concat(["${aws_iam_policy.alb_ingress_node_policy.id}","arn:aws:iam::aws:policy/CloudWatchAgentServerPolicy","arn:aws:iam::aws:policy/AmazonSSMManagedInstanceCore"],var.workers_additional_policies)
workers_group_defaults = var.workers_group_defaults
worker_groups_launch_template = var.worker_groups_launch_template
node_groups_defaults = var.node_groups_defaults
node_groups = var.node_groups
manage_aws_auth                   = true
map_roles = [
{
rolearn = data.aws_iam_role.sso_admin.arn
username = "sso-admin"
groups   = ["system:masters"]
},
{
rolearn = data.aws_iam_role.sso_pu.arn
username = "sso-pu"
groups   = ["system:masters"]
},
{
rolearn = data.aws_iam_role.sso_ro.arn
username = "sso-ro"
groups   = ["system:authenticated"]
},
]
}
What's the expected behavior?
Are you able to fix this problem and submit a PR? Link here if you have already.
Environment details

Affected module version: 9.0.0
OS: os-x
Terraform version: 0.12.21

Any other relevant info

wget  --no-check-certificate -O -   $ENDPOINT/healthz
--2020-02-29 11:20:14--  https://bf589asd1d6166cb4d8cc7243625e4e.gr7.us-east-1.eks.amazonaws.com/healthz
Resolving bf589asd1d6166cb4d8cc7243625e4e.gr7.us-east-1.eks.amazonaws.com (bf589dec1d6166cb4d8cc770df625e4e.gr7.us-east-1.eks.amazonaws.com)... <IP_ADDRESS>, <IP_ADDRESS>
Connecting to bf589asd1d6166cb4d8cc7243625e4e.gr7.us-east-1.eks.amazonaws.com (bf589dec1d6166cb4d8cc770df625e4e.gr7.us-east-1.eks.amazonaws.com)|<IP_ADDRESS>|:443... connected.
OpenSSL: error:1407742E:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert protocol version
Unable to establish SSL connection.

What is your wget version ? can you please share your wget output in debug mode --debug?

wget --debug --no-check-certificate -O - https://2157EAD4C8AB1C95957XXXXXXXXXXXXX.gr7.us-east-1.eks.amazonaws.com/healthz
Setting --check-certificate (checkcertificate) to 0
Setting --output-document (outputdocument) to -
DEBUG output created by Wget 1.17.1 on darwin14.5.0.
Reading HSTS entries from /Users/geiner/.wget-hsts
URI encoding = ‘UTF-8’
--2020-02-29 17:12:50--  https://2157EAD4C8AB1C95957XXXXXXXXXXXXX.gr7.us-east-1.eks.amazonaws.com/healthz
Resolving 2157EAD4C8AB1C95957XXXXXXXXXXXXX.gr7.us-east-1.eks.amazonaws.com (2157EAD4C8AB1C95957XXXXXXXXXXXXX.gr7.us-east-1.eks.amazonaws.com)... 3.225.XXX.XXX, 52.23.XXX.XXX
Caching 2157EAD4C8AB1C95957XXXXXXXXXXXXX.gr7.us-east-1.eks.amazonaws.com => 3.225.XXX.XXX 52.23.XXX.XXX
Connecting to 2157EAD4C8AB1C95957XXXXXXXXXXXXX.gr7.us-east-1.eks.amazonaws.com (2157EAD4C8AB1C95957XXXXXXXXXXXXX.gr7.us-east-1.eks.amazonaws.com)|3.225.XXX.XXX|:443... connected.
Created socket 6.
Releasing 0x00007fccd7001940 (new refcount 1).
Initiating SSL handshake.
SSL handshake failed.
OpenSSL: error:1407742E:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert protocol version
Closed fd 6
Unable to establish SSL connection.
Saving HSTS entries to /Users/geiner/.wget-hsts

It looks that AWS API endpoint accepts only TLS 1.2 protocol which is case in
Your Wget version looks to be 1.17.1 which is from 2015, so quite old. I have checked GNU Wget 1.18 and newer which are working as expected.
So you should  update wget or you can return to previous method via overwriting default value for wait_for_cluster_cmd.

Yes as @daroga0002 mentioned, you're trying to do TLS 1.0 instead of TLS 1.2. Upgrade your wget or  use curl please.
@robgeiner Closing this. Feel free to reopen this issue if https://github.com/terraform-aws-modules/terraform-aws-eks/issues/757#issuecomment-593883841 doesn't help.

@robgeiner I am a terraform noob here. I am following this -
https://learn.hashicorp.com/terraform/kubernetes/provision-eks-cluster
And hitting the wget issue. Where should I add this
wait_for_cluster_cmd = "until curl -k -s $ENDPOINT/healthz >/dev/null; do sleep 4; done"

to make it work without failing for wget. Thanks

For example into this line of example:
https://github.com/terraform-aws-modules/terraform-aws-eks/blob/7afecf637905dc6aeb58519abaf4d09231e8c9a6/examples/basic/main.tf#L129

Yep, what @daroga0002 said.  For example,
module "eks" {
  source          = "terraform-aws-modules/eks/aws"
  version         = "11.0.0"
  cluster_name    = local.eks_cluster_name
  wait_for_cluster_cmd          = "until curl -k -s $ENDPOINT/healthz >/dev/null; do sleep 4; done"
  ...
}

Thanks it worked.


How to implement?

Is there an example explaining how to use your framework? How can I replace the Minkowski Engine?Thanks.

Hi @kai0416s, while I am still working on a concrete minmized example to provide a match between Minuet and MinkowskiEngine. You can either check the previous issue (#1), or the develop branch to see how to use Minuet, or our system artifacts to see how to match results with prior frameworks.
Please note that currently only Minuet is designed only for inference. While I am still working on implementing the backward operators, due to short of my bandwidth it might take a while to do so. Please stay tuned.


[BREAKING] Drop support for node 16

This PR drops support for Node 16. Only node versions higher than 18.0.0 are supported.

Coverage: 94.608%. Remained the same when pulling 055e98448bf91921f7a1d3503c9d1bdaf38c226e on drop-node-16-support into 125003d1aa28cae28b0b2386ef72bf1dd4666a5c on main.

Decided to support Node.js version 16.18.0 and above


Assignment 3: is this where i have to opt out of using tables? "numpy.ufunc has the wrong size, try recompiling. Expected 192, got 216"

probably too late for us but a few of us are getting this cryptic error:
Traceback (most recent call last):
  File "run_experiment.py", line 234, in <module>
    timings,
  File "run_experiment.py", line 41, in run_experiment
    exp.perform_cluster(dim)
  File "/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/experiments/RF.py", line 85, in perform_cluster
    hdf_path = self.dump_for_clustering(lambda x: filtr.fit_transform(x.get_details().ds.training_x,
  File "/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/experiments/base.py", line 105, in dump_for_clustering
    ds_2.to_hdf(hdf_path, self._details.ds_name, complib='blosc', complevel=9)
  File "/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/py3/lib/python3.7/site-packages/pandas/core/generic.py", line 1996, in to_hdf
    return pytables.to_hdf(path_or_buf, key, self, **kwargs)
  File "/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/py3/lib/python3.7/site-packages/pandas/io/pytables.py", line 278, in to_hdf
    complib=complib) as store:
  File "/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/py3/lib/python3.7/site-packages/pandas/io/pytables.py", line 469, in __init__
    import tables  # noqa
  File "/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/py3/lib/python3.7/site-packages/tables/__init__.py", line 93, in <module>
    from .utilsextension import (
  File "__init__.pxd", line 872, in init tables.utilsextension
ValueError: numpy.ufunc has the wrong size, try recompiling. Expected 192, got 216
/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/py3/lib/python3.7/site-packages/sklearn/externals/joblib/externals/cloudpickle/cloudpickle.py:47: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
  import imp

this sounds like a situation where tables dont work? or is there a way to resolve this?

after uninstalling and reinstalling numpy, i now get this different table related error:
/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/py3/lib/python3.7/site-packages/scipy/linalg/basic.py:1321: RuntimeWarning: internal gelsd driver lwork query error, required iwork dimension not returned. This is likely the result of LAPACK bug 0038, fixed in LAPACK 3.2.2 (released July 21, 2010). Falling back to 'gelss' driver.
  x, resids, rank, s = lstsq(a, b, cond=cond, check_finite=False)
Traceback (most recent call last):
  File "run_experiment.py", line 207, in <module>
    timings,
  File "run_experiment.py", line 41, in run_experiment
    exp.perform_cluster(dim)
  File "/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/experiments/ICA.py", line 80, in perform_cluster
    preprocess=False)
  File "/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/data/loader.py", line 226, in reload_from_hdf
    df = pd.read_hdf(hdf_path, hdf_ds_name)
  File "/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/py3/lib/python3.7/site-packages/pandas/io/pytables.py", line 394, in read_hdf
    return store.select(key, auto_close=auto_close, **kwargs)
  File "/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/py3/lib/python3.7/site-packages/pandas/io/pytables.py", line 741, in select
    return it.get_result()
  File "/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/py3/lib/python3.7/site-packages/pandas/io/pytables.py", line 1483, in get_result
    results = self.func(self.start, self.stop, where)
  File "/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/py3/lib/python3.7/site-packages/pandas/io/pytables.py", line 734, in func
    columns=columns)
  File "/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/py3/lib/python3.7/site-packages/pandas/io/pytables.py", line 2928, in read
    ax = self.read_index('axis%d' % i, start=_start, stop=_stop)
  File "/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/py3/lib/python3.7/site-packages/pandas/io/pytables.py", line 2523, in read_index
    _, index = self.read_index_node(getattr(self.group, key), **kwargs)
  File "/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/py3/lib/python3.7/site-packages/pandas/io/pytables.py", line 2621, in read_index_node
    data = node[start:stop]
  File "/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/py3/lib/python3.7/site-packages/tables/vlarray.py", line 685, in __getitem__
    return self.read(start, stop, step)
  File "/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/py3/lib/python3.7/site-packages/tables/vlarray.py", line 821, in read
    listarr = self._read_array(start, stop, step)
  File "tables/hdf5extension.pyx", line 2155, in tables.hdf5extension.VLArray._read_array
ValueError: cannot set WRITEABLE flag to True of this array
/Users/swyx/Desktop/OMSCS/ML/ASSIGNMENT3/swyx-a3/py3/lib/python3.7/site-packages/sklearn/externals/joblib/externals/cloudpickle/cloudpickle.py:47: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
  import imp

ok so it seems the version numbers conflict a lot especiallywhen it comes to numpy. you have 1.15 but a lot of things assume 1.16 (the current version at time of writing).
so you have to uninstall and reinstall everything again: https://stackoverflow.com/questions/11248073/what-is-the-easiest-way-to-remove-all-packages-installed-by-pip
my requirements from pip freeze:
cycler==0.10.0
kiwisolver==1.0.1
kneed==0.2.4
matplotlib==3.0.3
mock==2.0.0
numexpr==2.6.9
numpy==1.16.2
pandas==0.24.2
pbr==5.1.3
pyparsing==2.3.1
python-dateutil==2.8.0
pytz==2018.9
scikit-learn==0.20.3
scikit-optimize==0.5.2
scipy==1.2.1
seaborn==0.9.0
six==1.12.0
tables==3.5.1
xlrd==1.2.0


zeus generates empty coverage results with simplecov

I'm using zeus with rspec and simplecov (for ruby 1.9) in a rails 3 project, and executing
zeus rspec spec

generates empty testing coverage results in coverage/index.html. It is OK when running the tests without zeus.

@Johaned suggestion worked form me using Rails 4, Zeus 0.15.4, ruby 2.2.
Is it ok if I edit the Wiki page to reflect this solution when using Rails >= 4

hey @guapolo
sure, no problem :+1:

This is not working for me.
I get :

You tried to define an enum named "available_roles" on the model "User", but this will generate a instance method "admin?", which is already defined by another enum. (ArgumentError)

@xero88 that's a Rails enum thing. You defined two enums on your model with an element named :admin. You can't repeat enum element names on the same model.

@guapolo no I have not. When I change the element, I get :
You tried to define an enum named "available_roles" on the model "User", but this will generate a instance method "adminaa?", which is already defined by another enum. (ArgumentError)


Giftbox script function? bug

Before opening an issue:

[✓] I tested the latest release
[✓] I looked at other issues (even the closed ones)
[✓] I read the Troubleshooting Guide
[✓] I'll fill this template properly

Your issue will get closed/locked by the bot in case proper info isn't provided
If you open unnecessary/dumb issues, you'll be rewarded with appropriate tags.

Describe the bug
Currently the Giftbox script works even outside the giftbox screen
It will work with any screen (I accidentally clicked back and clicked a singularity and it will infinitely scroll there)
And screen with scrollbars (like the singularity list, roulette's prices)
Screenshots
No screenshot available
Video
No video available
Device Info (please fill at-least 'Device model'):

Device model: Xiaomi Redmi Note 7
Screen size: 1080 x 2340
Android Version: Android 9
RAM: 3GB

The Lottery and FP scripts also work anywhere :-P

Should be fixed when #483 is merged

Thanks


google-protobuf 3.3.0 causes errors in Datastore

Last Friday google-protobuf 3.3.0 was released. One of the changes made was:

Fixed Message#to_h for messages with map fields.

Unfortunately, this breaks API compatibility and caused failures on the Datastore CI build. I released google-cloud-datastore 1.0.1 over the weekend to pin the dependency to ~> 3.2.0. But we should update Datastore to work with the new behavior.

But we should update Datastore to work with the new behavior.

@blowmage Has this already been done?

No, not yet. I'll work on it this week.


Add cron jobs configuration option

As discussed in #129, please add a new configuration option to run cron jobs. Thanks!

Adding a new cron.yml task file, using the following configuration in config.yml to set up a cron job:
# Cron jobs are added to the root user's crontab. Keys include name (required),
# minute, hour, day, weekday, month, job (required), and state.
drupalvm_cron_jobs:
  - {
    name: "Drupal Cron",
    minute: "*/30",
    job: "drush -r {{ drupal_core_path }} core-cron"
  }

Multiple cron jobs can be added, and any of the time options (e.g. minute, hour, day, weekday, month) default to * if not configured. state defaults to present but you can set it to absent to remove an old cron job.


[RESOLVED] High ICMP ping response time

Problem started at 10:51:14 on 2023.08.05
Problem name: High ICMP ping response time
Host: MF-FG
Severity: Warning
Operational data: Value: 433ms
Original problem ID: 34622
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23132&eventid=34622

Problem has been resolved in 6m 0s at 10:57:14 on 2023.08.05
Problem name: High ICMP ping response time
Host: MF-FG
Severity: Warning
Original problem ID: 34622


Bug in configuring target audience NEWS

map is only shown after resize
hovering the map does not show the name of the city (show not re-color when not editable)
Budget show unlimited number of views
shop question

question should be fixed by the PR as well, it was just a visual bug and the item was never published regionally


Remove the Enhance API.

The Samsung Enhance API has been added in Cocos2D-x in 2015 to allow Samsung to tweak the games for their devices. It's a set of functions for which we do not have the implementation, consequently we can't maintain it.
This PR removes this API.

I'm not sure do we need remove Enhance API. some questions:

it's unused?
it's break changes?

I can't tell if it is used or not because it was a closed-source module. For what it's worth it seems to have been removed from Cocos Engine. It adds nothing to Axmol except code to maintain, and we can't even tell if it works, so IMHO it should go :)


Feature request: SQS token support IAM assume role in Jenkins builder and use AWS profile from the builder

Hi,
I would like to have a feature request that the SQS token supports the IAM role and use the default AWS profile. Currently, our CI builder also running on the AWS cloud with roles attached. Would be great if the SQS builder plugin supports such configuration, so we don't need to create an IAM user accessKey and secretKey
Thanks
--Guang

The plugin actually works without the accessKey and secretKey


docs: fix typo (Javasript and Typescript)

It fixes one typo: Javasript -> Javascript
and makes Typescript, TypeScript to match what Microsoft style of writing TS.

See comment here https://github.com/kulshekhar/ts-jest/pull/313#issuecomment-326525758

@tkrotoff OK, We can wait to verify that first.


Cast to decimal (resolves issue #430)

This issue resolves issue #430, which requests that the cast function handle values being casted to decimals.

Codecov Report

Merging #527 (b218e85) into main (1213d82) will decrease coverage by 0.19%.
The diff coverage is 4.00%.

:exclamation: Current head b218e85 differs from pull request most recent head 6a7a759. Consider uploading reports for the commit 6a7a759 to get more accurate results

@@            Coverage Diff             @@
##             main     #527      +/-   ##
==========================================
- Coverage   92.98%   92.78%   -0.20%
==========================================
  Files         175      175
  Lines       11182    11206      +24
==========================================
  Hits        10398    10398
- Misses        784      808      +24

Impacted Files
Coverage Δ

core/src/data/value/literal.rs
91.87% <0.00%> (-3.02%)
:arrow_down:

core/src/data/value/mod.rs
95.09% <0.00%> (-0.19%)
:arrow_down:

core/src/data/value/into.rs
93.08% <6.66%> (-4.29%)
:arrow_down:

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 1213d82...6a7a759. Read the comment docs.


Missing RFCs

https://why-cant-we-have-nice-things.mwl.be/users/kelunik doesn't show https://wiki.php.net/rfc/parameter-no-type-variance and https://wiki.php.net/rfc/improved-tls-constants, maybe others and I don't get why. Could you help how they're connected?

Guess it just hasn't updated its data.

Tracking in #105


Mention Windows Subsystem for Linux (WSL) GUI in "Unix-like development on Windows" section

As I mentioned on Fortran Discourse, Microsoft has recently upgraded WSL 2 so that you can run Linux GUI programs, including text editors. Currently fortran-lang says "All of the above approaches provide access to common shells such as bash and development tools including GNU coreutils, Make, CMake, autotools, git, grep, sed, awk, ssh, etc."
In the description of WSL I suggest adding the text "With Windows Subsystem for Linux GUI one can run text editors and other graphical programs."
I could submit a PR but my last trivial one is still pending ...

I could submit a PR but my last trivial one is still pending ...

Just took care of that one.
You can submit multiple PRs from your fork, by creating feature branches for each of your contributions. Just click on the branch dropdown menu and create a new one there:

This will also make it easier to sync your fork again, if someone (probably me) squashes small PRs on merge, which are done from the default branch of a fork.


Error Displayed Twice for 'This Field is Required' on Button Type proerty

Describe the bug
When the button type is removed from a button group component, there is a problem where the error message "This Field is Required" is displayed twice instead of once. This issue occurs specifically when the button type is removed.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://function-adminportal-test.shesha.dev/dynamic/Boxfusion.SheshaFunctionalTests.Common/twenty-two'
Go to the form designer
Navigate to Button Group Configuration >> Customize Button Group
Click any item
Remove the Button Type
Actual Behaviour: The Error "This Field is Required" display twice.

Expected behavior
The error message "This Field is Required" should be displayed only once

This issue is also observed when you clear the Modal Form name within the Entity Picker Component.


fix ''should not push route with duplicated key" issue

i think this makes sense. do you mind adding tests for this as well? you can run gulp watch and add a test for card stack reducer: https://github.com/bakery/react-native-navigation-redux-helpers/blob/master/test/unit/reducers.js#L94

i have added the test, but I not sure whether it is the correct way to write test spec >_<

released in 0.5.0. thank you for your contribution 🙌

thanks for merging !! it's my first time sending pull request in git >_<


Attribute max doesn't work

I have two fields and the value max in the second field it's the value of the first field:
<md-input-container md-no-float >
      This is first field
      <input name="field1" ng-model="field1"  required type="text" ui-money-mask="0" >  </input>
     </md-input-container>
   <md-input-container md-no-float >
       This is second field
       <input name="field2" ng-model="field2" required type="text" ui-money-mask="0" max="{{field1}}">      </input>

Field2 cannot be greater than field1

 </md-input-container>

But max doesn't work.
It work only if I set max static (ex: max="100").

Try <input name="field2" ng-model="field2" required type="text" ui-money-mask="0" max="field1">

Thank you!


bevy_app::App cannot be sent between threads

Bevy version
bevy 0.14
What you did
bevy 0.14 doesn't allow bevy_app::App to be sent between threads anymore.
What went wrong
Looks like RunnerFn:
type RunnerFn = Box<dyn FnOnce(App) -> AppExit>;

introduces this limitation.
Compiler error:
error[E0277]: `(dyn FnOnce(App) -> AppExit + 'static)` cannot be sent between threads safely
   --> base-init/src/lib.rs:133:28
    |
133 |                     .spawn(move || manual_game_loop(game_app_handle))
    |                      ----- ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ `(dyn FnOnce(App) -> AppExit + 'static)` cannot be sent between threads safely
    |                      |
    |                      required by a bound introduced by this call
    |
    = help: the trait `Send` is not implemented for `(dyn FnOnce(App) -> AppExit + 'static)`, which is required by `{closure@base-init/src/lib.rs:133:28: 133:35}: Send`
    = note: required for `std::ptr::Unique<(dyn FnOnce(App) -> AppExit + 'static)>` to implement `Send`
note: required because it appears within the type `Box<(dyn FnOnce(App) -> AppExit + 'static)>`
   --> .rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/alloc/src/boxed.rs:238:12
    |
238 | pub struct Box<
    |            ^^^
note: required because it appears within the type `App`
   --> .cargo/registry/src/index.crates.io-6f17d22bba15001f/bevy_app-0.14.0/src/app.rs:70:12
    |
70  | pub struct App {
    |            ^^^
    = note: required for `Mutex<App>` to implement `Sync`
    = note: 1 redundant requirement hidden
    = note: required for `Arc<Mutex<App>>` to implement `Send`

Additional information
bevy_app::App / Plugins are a very great abstractions that can be used in wide variety of situations and this limits possible use.

This was an intentional change. It's also documented in the migration guide. https://bevyengine.org/learn/migration-guides/0-13-to-0-14/#app-changes

App is not Send anymore, but SubApp still is.

See also the PR that made the change https://github.com/bevyengine/bevy/pull/9202

Moving !Send resource ownership to App will make it unambiguously !Send

and the fundamental issue that lead to this change https://github.com/bevyengine/bevy/discussions/6552


PowerToys Stops working in administrative

ℹ Computer information

PowerToys version: 0.21.1
PowerToy Utility:
Running PowerToys as Admin: attempting
Windows build number: [run "winver"] 18363.1082

📝 Provide detailed reproduction steps (if any)

… Clicked "restart as administrator" in general
… powertoys closed and wouldn't open from powertoysrun, file explorer, or win + s
… restarted computer, wasn't running as administrator, attempted again and the same crash happened

✔ Expected result
What is the expected result of the above steps?
❌ Actual result
What is the actual result of the above steps?
📷 Screenshots
Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form

@Raidan3 does this still happen in 0.27?  we adjusted a few things here.


Support for interceptors

Hi,
What do I need to modify in the source code to add support for something like this:
Upload.upload({
        url: my_url_here,
        data: {
          file: file_here
        },
        headers: {
         // add headers here
        },
        interceptor: MY_INTERCEPTOR_HERE
      })
      .then(success, error, percentageStep);

where MY_INTERCEPTOR_HERE should look like this:
(function() {
  'use strict';

  angular
    .module('app', [])
    .factory('MY_INTERCEPTOR_HERE', MY_INTERCEPTOR_HERE);

  /** @ngInject */

  function MY_INTERCEPTOR_HERE() {
      return {
      response: function(response) {
        // modify response here

        return response;
      }
    };
  }
})();

Thanks in advance!

One way would be to globally add the desired interceptor inside the config step:
$httpProvider.interceptors.push('RestServiceInterceptor');

but I'm curious if we could decorate the Upload service to add support for this functionality. Thx!

Use regular $http() syntax:
Upload.upload{..., transformResponse: function() {....}}


Improved shrinking for expressions

Pull request
Description
This PR simplifies outcome of QuickCheck tests, i.e., when a test fails, with these enhancements the detected failure is smaller and therefore easier to understand.
Related
No related issues
Checklist
This code only affects test code

[ *] The code is tested (ran 10k tests for both affected properties

Performance
This change only affects testing

This is really helpful, thank you! Sadly before we can merge it you have to go through the DCO dance. It's a git feature about signing the commits, it can be done at the time with the -s flag or after the fact, the DCO bot that's complaining above has some easy and copy&pastable commands for that


Npm install fails

npm WARN ERESOLVE overriding peer dependency
npm WARN While resolving<EMAIL_ADDRESS>npm WARN Found: peer<EMAIL_ADDRESS>from the root project
npm WARN
npm WARN Could not resolve dependency:
npm WARN peer<EMAIL_ADDRESS>from the root project
npm ERR! code ETARGET
npm ERR! notarget No matching version found for<EMAIL_ADDRESS>npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.

Hi @netdown,
Can you send me more information about you problem?

What is your environment (node version / npm version)?
What did you do to have this error?
What version of @ts2doc are you trying to use?

Tank you!

Bug appears on version 0.3.0, it has been fixed on this revision: https://github.com/jpiazzal/ts2doc/commit/1652edff3c43ea5358616639f634592417e8a8a0
Version 0.3.2 fix this issue


About the feature format

Market data is ts data, it seems current model is:
(ti, fi, fi, fi, ...) -> (li)
i is [1,N], t is timestamp, f is feature, l is label
ti maybe contain past window, e.g ma30.
li maybe contain future window,e.g performance in next 10 days.
My question is:
Should we add ti as feaure too if the feature has contained some past time window information.
This would effect the data of the model:
(ti, fi(current data), fi, fi, ...) -> (li)
(ti, fi(current data), fi, fi, ...) -> (li)
====== more samles
(ti, fi(window data), fi, fi, ...) -> (li)
======less samples, we has computed the data from past windows.
Which format is better?
Is there some custom hanlder to define it ?

Should we add ti as feaure too if the feature has contained some past time window information.
It depends on what information you want to expose to your model.

Besides, creating features with a past time window of information will not necessarily make the samples less.
e.g.   the window for the feature may be 10, but the steps to create features can be 1 (adjacent samples may have some information overlap)
A lot of features in Qlib's datasets contains some past time window information.  (e.g. https://github.com/microsoft/qlib/blob/103f8579c10f18219b729cd27837b3d119f9762c/qlib/contrib/data/handler.py#L103)

@you-n-g Thanks for your reply.
Yes, It depends the model design.
So the model may be built on many non independent features.
Anyway it's market domain knowledge.
I'm thinking about whether we could add some mechanism to handle this.
Your information is very helpful.Thanks again!


Add Remark Command

Codecov Report
Base: 72.18% // Head: 71.62% // Decreases project coverage by -0.55% :warning:

Coverage data is based on head (5881dca) compared to base (c004081).
Patch coverage: 67.90% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master       #6      +/-   ##
============================================
- Coverage     72.18%   71.62%   -0.56%
- Complexity      399      413      +14
============================================
  Files            70       73       +3
  Lines          1233     1304      +71
  Branches        125      135      +10
============================================
+ Hits            890      934      +44
- Misses          311      332      +21
- Partials         32       38       +6

Impacted Files
Coverage Δ

...main/java/seedu/address/commons/core/Messages.java
0.00% <ø> (ø)

.../java/seedu/address/logic/commands/AddCommand.java
100.00% <ø> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
88.88% <0.00%> (-11.12%)
:arrow_down:

...eedu/address/logic/parser/RemarkCommandParser.java
0.00% <0.00%> (ø)

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

...in/java/seedu/address/logic/parser/ParserUtil.java
92.68% <60.00%> (-4.54%)
:arrow_down:

...c/main/java/seedu/address/model/person/Remark.java
66.66% <66.66%> (ø)

...va/seedu/address/logic/commands/RemarkCommand.java
70.83% <70.83%> (ø)

.../java/seedu/address/storage/JsonAdaptedPerson.java
96.07% <77.77%> (-3.93%)
:arrow_down:

...java/seedu/address/logic/commands/EditCommand.java
95.89% <87.50%> (-1.13%)
:arrow_down:

... and 6 more

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

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


initialValue problem

Version 1.9.0
Here's what went wrong:

Put an initial array in prop value.
Next deselect an initial item gives an good onChange result. ( initial array minus 1 )
Reset
Put an initial array in prop value.
Select an new extra item gives a bad onChange result. ( only the selected item without initial array values )

Hi @floorswinkels
Thanks for raising this issue, though I'm unable to replicate.
Could you please create a CodeSandbox reproducing the issue.
You can fork this one if that makes it easier:
https://codesandbox.io/s/0p3oxq3150

I can't replicate it in codeSandbox,
but i think I found the problem.
The picky constructor is called before I have the initial data.
Could this cause the problem ?

Ahh great assessment. I can replicate now. I'll take a look!

@floorswinkels That should now be published.
I hopefully got the release right, semantic-release was playing up and I ended up publishing version 0.0.0 😁
Again, thanks for reporting the issue ⭐

thanks @Aidurber
It's working now with version 1.9.1 !

@floorswinkels Glad to hear it, thanks for letting me know!


west flash for nucleo_u575zi_q is failing

Hi,
I am following below document to build and flash hello_world sample code on nucleo_u575zi_q.
https://docs.zephyrproject.org/2.7.0/boards/arm/nucleo_u575zi_q/doc/nucleou575zi_q.html
Getting error while running west flash which uses stm32cubeprogrammer runner for flashing the code.
Error Output:

Commands used to build and flash:
west build -b nucleo_u575zi_q samples/hello_world
west flash

Able to run below command successfully but same is not working with west flash
STM32_Programmer_CLI.exe --connect port=swd reset=HWrst --erase all

OS: Windows
zephyr version: 2.7.99
STM32CubeProgrammer: 2.10.0

Did you update the Firmware of your nucleo board recently using the stm32CubeProgrammer ?That could help in any case


Gulp Auto Detection reporing errors in non gulp project

Issue Type: Bug
I have a project that is not Javascript, and does not use Gulp. It does have a tasks.json file to set up some build scripts. However every time I use the "Command-Shift-B" command, I get an error from the Gulp Auto Detection built in extension:
Auto detecting gulp for folder untitled folder failed with error: Error: Command failed: gulp --tasks-simple --no-color

I was complaining that Gulp was not installed, which it was not as I do not use Gulp. So I did install gulp and am now getting this error.
Running the command in the error directly in the terminal seems to complete without error.

VS Code version: Code - Exploration 1.53.0-exploration (0c9e2dadd986d0bc085f1c5352245531eef65e84, 2021-01-07T06:27:03.244Z)
OS version: Darwin x64 19.6.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz (12 x 2600)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
32.00GB (11.81GB free)

Process Argv

Screen Reader
no

VM
18%

Extensions (21)

Extension
Author (truncated)
Version

ng-template
Ang
11.1.1

path-intellisense
chr
2.3.0

vscode-eslint
dba
2.1.14

githistory
don
0.6.14

output-colorizer
IBM
0.1.2

mads
mir
0.0.5

csharp
ms-
1.23.8

cpptools
ms-
1.1.3

hexeditor
ms-
1.3.0

js-debug-companion
ms-
1.0.8

js-debug-nightly
ms-
2021.1.2617

vscode-typescript-next
ms-
4.2.20210128

vscode-typescript-tslint-plugin
ms-
1.3.3

debugger-for-chrome
msj
4.12.12

mvbasic
mve
2.1.2

ossaction
pga
1.0.0

material-icon-theme
PKi
4.5.0

open-in-browser
tec
2.0.0

msbuild-project-tools
tin
0.3.11

flatuscode
Vin
1.1.1

vscodeintellicode
Vis
1.2.10

(2 theme extensions excluded)

A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368cf:30146710
vsreu685:30147344
openlogontheside:30221882
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30208930
wsl2prompt:30219162
vstry244:30244315
pythonvsdeb440:30224570
unusedprompt:30219164
folderexplorer:30219166
openfilemenucf:30219169
pythonvsded773:30223139
pythonvspyt875:30251590
core-portspanel:30233467
coreuntitledfile:30249963

This is NOT a duplicate of existing issues, and referencing https://www.github.com/microsoft/vscode/issues/113982 I certainly would NOT want VS Code to automatically install Gulp for me.

This is NOT a duplicate of existing issues, and referencing https://www.github.com/microsoft/vscode/issues/113982 I certainly would NOT want VS Code to automatically install Gulp for me.

This should already be fixed by bebd06640734c37f6d5f1a82b13297ce1d297dd1

No, it is not, at least with this version:

Version: 1.53.0-exploration
Commit: 0c9e2dadd986d0bc085f1c5352245531eef65e84
Date: 2021-01-07T06:27:03.244Z
Electron: 11.1.0
Chrome: 87.0.4280.88
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 19.6.0

That build is quite old. Please try updating to a newer one!

There are no updates, at least using the build in "Check for Updates"

@bpasero it seems like there haven't been any exploration builds in a while. Is that intentional?
@jtsom I would recommend switching to VS Code Insiders if you aren't specifically testing changes that are only available in the exploration build!

@alexr00 yes they are suspended until we pick up with Electron 12.


Remove implicits from serialize

implicits should be avoided, rather, a single function on AST can call serialize functions for all FIRRTL graph nodes.
These serialize functions could be migrated to the actual traits and case classes in IR.scala. I originally didn't put them there in order to keep the IR "self documenting", but perhaps that's just better handled by the FIRRTL Spec and ScalaDocs.

Looks good to me

Merged in 0aa246385d1d2eabafce0e659d6438a38c3b6519


Change OID name and add clarification notes

This also closes issue #9. @dimakuv, @KB5201314 please review.