Variant size selector does not create the up-to-date URL

Steps to reproduce:

Go to the product with variant, example:
http://localhost:4200/apparel-uk-spa/en/GBP/product/300738133/Asterisk SS youth washed orange L

Change the size

Generated URL does not match displayed variant (shows the previous one, see the picture below):

QA Steps:

Go to product linked in the description
Change the size
check if url match with product title - QA pass if it does

http://dev-com-17.accdemo.b2c.ydev.hybris.com:4200/apparel-uk-spa/en/GBP/product/300608386/I Love Tee SS Women turquoise L
used the product above, as well as others. The size changes and the url's product title matches.
✅
dev-com-17 got updated with latest develop branch, as well as including apparel-uk-spa


Adding a layout that always divide the current window

I tried i3-layouts today, and it seems awesome. Before this, I have been a happy user of autotiling, which automatically decide whether to use splith/splitv depending on the dimension of the currently focused window. Is it possible for i3-layouts to add a similar feature? Spiral seems close to what I want, but it always divide the smallest window, instead of the currently focused one.

Hey @yangsheng6810 ,
Thanks for trying i3-layouts.
autotilling sounds interesting. I'll look into it for its implementation in i3-layouts.

I've added  a layout called autosplit that should mimic the autotiling behavior. It's available in the last version (v0.12.0) if you want to try it:
set $i3l autosplit to workspace $ws2

Thanks again for your suggestion.

Thanks for the quick update. set $i3l autosplit to workspace $ws2 works like a charm. But it seems i3l autosplit in the commandline does not work. It prints {"success":true}, but i3-layouts crashes with the following output:
'layout_change_autosplit' is not a valid RebuildCause
Traceback (most recent call last):
  File "/home/yangsheng/.local/bin/i3-layouts", line 8, in <module>
    sys.exit(main())
  File "/home/yangsheng/.local/lib/python3.9/site-packages/i3l/cli.py", line 5, in main
    connect()
  File "/home/yangsheng/.local/lib/python3.9/site-packages/i3l/connect.py", line 39, in connect
    i3.main()
  File "/usr/lib/python3.9/site-packages/i3ipc/connection.py", line 514, in main
    raise loop_exception
  File "/usr/lib/python3.9/site-packages/i3ipc/connection.py", line 497, in main
    while not self._event_socket_poll():
  File "/usr/lib/python3.9/site-packages/i3ipc/connection.py", line 477, in _event_socket_poll
    raise e
  File "/usr/lib/python3.9/site-packages/i3ipc/connection.py", line 474, in _event_socket_poll
    self._pubsub.emit(event_name, event)
  File "/usr/lib/python3.9/site-packages/i3ipc/_private/pubsub.py", line 28, in emit
    s['handler'](self.conn, data)
  File "/home/yangsheng/.local/lib/python3.9/site-packages/i3l/handlers.py", line 50, in _on_tick
    handle_layout_tick(context, action_name, action_params)
  File "/home/yangsheng/.local/lib/python3.9/site-packages/i3l/handlers.py", line 31, in handle_layout_tick
    state.start_rebuild(RebuildCause.layout_change(action_name), context,
  File "/home/yangsheng/.local/lib/python3.9/site-packages/i3l/state.py", line 32, in layout_change
    return RebuildCause(f'layout_change_{layout_name}')
  File "/usr/lib/python3.9/enum.py", line 360, in __call__
    return cls.__new__(cls, value)
  File "/usr/lib/python3.9/enum.py", line 677, in __new__
    raise ve_exc
ValueError: 'layout_change_autosplit' is not a valid RebuildCause

Sorry for that. I have released a new version (0.12.1) to fix this issue.

That is FAST!!
BTW, you may forget to add autosplit to Configuration > Switching layout > Syntax.

Good catch ! It's fixed, thanks.


Update chisel3 to 3.4-20201209-SNAPSHOT

Updates edu.berkeley.cs:chisel3 from 3.4-SNAPSHOT to 3.4-20201209-SNAPSHOT.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "edu.berkeley.cs", artifactId = "chisel3" } ]

labels: library-update

This is not what we want to do.


Examples: fixes secure_auth_with_db

Using the first element of the returned list of users in get_token(),
because tinydb's db.search() always returns a list. This fixes
a "TypeError: list indices must be integers, not str".
Refactors the variable "user" to always contain a single user dict,
to be consistent.

Coverage remained the same at 99.541% when pulling c4f382826fe04a6b094eb7f43825cd82797ada74 on bernhardreiter:patch-2 into c2f04561e3d8b5847459dcc2ca2675269113678d on timothycrosley:develop.

Looks good! Thanks!


Fix NPE in server address

Fixed an NPE in GrpcServerAddress when the address is passed in unresolved.

@ggezer PTAL.

Merged build finished. Test FAILed.

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

Thanks for fixing this.
Do we always create InetSocketAddress with a host name?
There could be problems with Kerberos if there are places where we only specify the port and assume local address. (If Kerberos principals are configured to match reverse lookup name of hosts)

Thanks for fixing this.
Do we always create InetSocketAddress with a host name?
There could be problems with Kerberos if there are places where we only specify the port and assume local address. (If Kerberos principals are configured to match reverse lookup name of hosts)

From Javadoc InetSocketAddress.getHostName will return the original host name if given, or in the case of an ip address, a reverse lookup will be used to acquire the hostname.

Jenkins, test this please.

So the earlier behavior is that we take the reverse lookup name even if hostname was provided. Now always using the given hostname may have effects on certificate resolution. We may want to run some tests with this before checking in.

Merged build finished. Test PASSed.

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

LGTM.

alluxio-bot, merge this please


Update resource paths to new Google/AWS/Azure public hosted data

Since we'll end up with 3 versions (one on each of the major cloud providers) for each public file, we'll need to implement a new structure for specifying which cloud version to use

#373 added support for reading resources from the Google Cloud Public Datasets bucket.
See https://github.com/broadinstitute/gnomad_methods/pull/373/commits/f44ae96abcaf958800f41746efebae5703fcc0a5 for an example of how to add support for AWS and Azure.

It's not immediately clear what the URL scheme for files on AWS would be.
https://hadoop.apache.org/docs/stable/hadoop-aws/tools/hadoop-aws/index.html and https://spark.apache.org/docs/3.1.1/cloud-integration.html#recommended-settings-for-writing-to-object-stores suggest s3a://.
https://hadoop.apache.org/docs/stable/hadoop-aws/tools/hadoop-aws/index.html#Other_S3_Connectors mentions an s3:// client from Amazon EMR.

Need to update gnomad_methods resource code for AWS and Azure: https://github.com/broadinstitute/gnomad_methods/blob/master/gnomad/resources/resource_utils.py


Allow for resuming a workflow and mark a failed step as succeeded

Not replaying it.

This issue is a bit hard to implement as it requires meddling with the internals of durable functions.


[FEATURE] <A website for Practicing HTML,CSS & JS through mini projects>

Description
I want to add  FlorinPops iCodeThis website to the Frontend Projects section of Frontend Development.
Here's the Link: https://www.icodethis.com
Screenshots
No response
Additional information
No response

Looks good. Feel free to open a PR for the same

Yeah, I have opened a PR. Please review it.

I would really like to mentor this platform for practicing
frontend development , frontendmentor.io Please do check it out and let me
know about whether you like it or not ?
On Sun, Jan 22, 2023 at 11:12 AM Chayannn @.***> wrote:

Description
I want to add FlorinPops iCodeThis website to the Frontend Projects
section of Frontend Development.
Here's the Link: https://www.icodethis.com
Screenshots
No response
Additional information
No response
—
Reply to this email directly, view it on GitHub
https://github.com/WeMakeDevs/roadmaps/issues/372, or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3AEFS5TPFS4I5XFCCLNMN3WTTCC7ANCNFSM6AAAAAAUCZOK3A
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Should the order of the arguments of verify and verify_and_update be swapped?

Hello @frankie567,
First, thank you for this project! The fact that passlib is being unmaintained is clearly a bad sign... The fact that you decided to create an alternative is a great news!
I was integrating this lib as a PasswordHelper in fastapi-users. I noticed that the order of the hashed_password and plain_password are reversed when comparing against passlib. Is that something you would like to change to have a drop in replacement?
see:
https://github.com/frankie567/pwdlib/blob/2a847c1131a37eeffa1a47924eccf0dd4c3c3839/pwdlib/_hash.py#L91
and:
https://foss.heptapod.net/python-libs/passlib/-/blob/branch/stable/passlib/context.py?ref_type=heads#L2349

Here is the code of the PasswordHelper using pwdlib for fastapi-users: https://github.com/fastapi-users/fastapi-users/issues/1325#issuecomment-1979832994

Good catch! I was probably lead to that way when I wrapped the Argon2 API:
https://github.com/frankie567/pwdlib/blob/153a7346aaa8c791d032a244396eed93ed93f3a5/pwdlib/hashers/argon2.py#L55-L64
But yes, I think it's probably good to follow Passlib style :)


feat: now using modifiers for cassettes

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


add profile section and update README file guideline to setup

add profile section and update README file guideline to setup


[9.x] Remove dependencies from console commands constructors

It will be better to remove dependencies from constructor methods of the console commands because Laravel loading those dependencies not on command usage but on command registering.
For example in class Cog\Laravel\Love\Console\Commands\SetupReactable there are dependencies in constructor:
public function __construct(Filesystem $files, MigrationCreator $creator, Composer $composer)
{
    parent::__construct();

    $this->files = $files;
    $this->creator = $creator;
    $this->composer = $composer;
}

Resolved in #222


Minor TS Update

You already ninja'd the 2 open tickets :+1:  so just adding snapPoint and bumping the review date.

I probably ought to make snapPoint actually do something now :)


HLS Stream Transcription with timestamps

Hello!
First of all, amazing work on the library, I really appreciate what you're doing here!
I'm trying to understand how (and if it's even possible) to generate SRT with timestamps from HLS stream.
I have created a simple script:
from whisper_live.client import TranscriptionClient, Client

def test():
    client = Client(
    "localhost",
    9090,
    lang="en",
    translate=False,
    model="tiny.en",
    srt_file_path="test.srt",
)

    while not client.recording:
        if client.waiting or client.server_error:
            client.close_websocket()
            return

    client.process_hls_stream(
        "http://as-hls-ww-live.akamaized.net/pool_904/live/ww/bbc_1xtra/bbc_1xtra.isml/bbc_1xtra-audio%3d96000.norewind.m3u8"
    )

test()

but it doesn't work as I would expect. It sends stream to server but doesn't receive response, or hangs.
When using the original TranscriptionClient it works fine, but the text is just streaming without any timestamps etc.
Can this be arranged to work?
Thanks!

Hello @drajvver , thanks for using WhisperLive. The srt file is written at the end of the  session. So, the transcription that is streamed is actually without timestamps i.e. whatever audio has been already processed by the backend. If you want to use the timestamps associated with the subtitle, you'd have to use the start and end time associated with each segment returned as response by the backend append them to a srt file. This is something that would be a nice addition.

how to "end" a session properly? ctrl-c seems NOT a way, what I am doing now...
maybe also add some verboseiness to the console enabled by maybe like "--debug", to see what it is doing, like in the last version, right now it does not show anything in console but works fine. i think this was to reduce the clutter in the console, especially when working with multiple users at once. but still having it/back would be nice.  (may also with extended-verbose, showing the single transcriptions as cleartext in the console, too.  ( i am working single-user and would love that feature, to confirm and expand upon it)


[Doris On ES]Add simple explain for EsTable

related issue: https://github.com/apache/incubator-doris/issues/3306
Note: this PR just remove the es_scan_node_test.cpp which is useless
For the moment, just add a simple explain syntax for EsTable without translating the native predicates to ES queryDSL which is better to finished with moving the predicate translating  from  Doris BE to Doris FE,  the whole work is still WIP.
 explain select * from test;

partial output:

 explain select * from test where k2 > 2;

partial output:

For https://github.com/apache/incubator-doris/issues/3327


Prefer BUGSNAG_RELEASE_STAGE over RAILS_ENV for release_stage

We deploy our projects to multiple release stages, but they all have RAILS_ENV set to production. I think this is a good way to keep the environments in sync, Heroku also agree with me.
When using the bugsnag gem, it currently overrides BUGSNAG_RELEASE_STAGE set in the environment with the current RAILS_ENV value. I think it would be better that RAILS_ENV is only used if BUGSNAG_RELEASE_STAGE is not set.
I have a workaround for this, but I'd prefer not having to add this to all our projects.
Bugsnag.configure do |c|
  c.release_stage = ENV.fetch('BUGSNAG_RELEASE_STAGE') if ENV.key?('BUGSNAG_RELEASE_STAGE')
end

What do you think about changing the precedence of RAILS_ENV and BUGSNAG_RELEASE_STAGE?

I think that makes sense, thank you for the suggestion, @dziemba.

This has now been addressed on master and I will do a release soon!

Thank you very much for this!

@snmaynard this is really useful thanks -- could you add it to bugsnag official docs for ease of reference?
e.g. https://docs.bugsnag.com/platforms/ruby/rails/#automatically-captured-diagnostics

(add something like  will use environment variable BUGSNAG_RELEASE_STAGE or RAILS_ENV)


you can merge multiple array to one array. Then you can show result.

I really don't see the benefit of doing this. You should just merge the result when you receive it from remote API.

Yes, it is for remote API. My example is simulate API that have multiple array in object. Maybe developers will use this feature when they want to merge a few array.

What I'm trying to say is that you don't need to modify extractValue function. Just use your searchConcatData to merge the response.

This can be all done in remote-api-handler.


Check only main section for blog sub-entries

This PR improves the reliability and performance of the Cypress test cypress/e2e/hybrid/check_links.cy.js.
Condition before PR
The test cypress/e2e/hybrid/check_links.cy.js has been failing with a browser out-of-memory crash when it was attempting to check blog entries. This started to occur after the introduction of additional links to blog articles placed near the topic of https://www.coronawarn.app/en/blog/ and https://www.coronawarn.app/de/blog/ (see issue https://github.com/corona-warn-app/cwa-website/issues/3391).
About 50 new links were added, which is about a 50% increase in the number of links on the blog main page. These links, which duplicated links on the same page, cause their corresponding linked sub-entries to have their article contents checked a second time.
The headers and footers of the blog sub-entries are identical to each other and to the main blog page. Repeatedly checking these identical sections, which contain about 30 links each, means redundant checks were being made, and in some cases, websites such as Instagram or GitHub blocked the checks because of their rapidity and number, due to rate-limiting or security protection measures.
The four sub-entry tests /de/blog, /en/blog, /de/science and /en/science were carrying out 9817 link tests.
Condition after PR
This PR changes the test strategy of blog and science blog entries by restricting the link check to the main section of each blog (sub-)entry. The main section is the part which makes each blog entry unique. This means that the header and footer of a blog sub-entry are not checked, since they are the same as the main blog pages.
The header and footer continue to be checked on the main blog pages.
Comparison
The following table gives a comparison of before and after restricting the blog sub-entry test to the main section. The following parameters were measured on a Windows 11 system:

Item
time
number of tests

Total Test duration
~19:00 -> 10:06
39

/de/blog

4434 -> 794

/en/blog

4412 -> 772

/de/science

485 -> 317

/en/science

486 -> 318

It was difficult to get a complete performance baseline (before applying the PR) due to the test crashing. It took several attempts before the test ran without crashing.
Result of PR

Stops browser crash
Removes > 7000 unnecessary link checks
Cuts > 9 minutes off the total time of the test

Verification
On a local clone of cwa-website
npm ci
npm run test:open

Select check_links and confirm that the test runs successfully.
npm test

to run the complete test suite and check that this also runs successfully.

Internal Tracking ID: EXPOSUREAPP-14806

@larswmh

We appreciate the effort you put into this and the much shorter testing time. Unfortunately, we didn't notice that when implementing the blog index below the search bar.

I think this PR should be OK to merge. I found it working reliably when it is running locally, which is the way it has been used in the past.
There is a set of other problems trying to get the workflow to run on GitHub, which would need to be researched separately. That should not prevent this PR from being merged though, as otherwise it will be very difficult to test external links at all.
Although the README includes the text: "To minimize the occurrence of errors we would ask you to perform all tests when contributing to our repository." (see PR #2891) this advice has not always been followed. That is a manual process discipline issue. The fact that the full test takes such a long time to run has been a deterrent in the past. Hopefully the test time reduction here will help to reduce this barrier to quality.


go.mod: use latest stable build of tailscale

Signed-off-by: Keli Velazquez<EMAIL_ADDRESS>

I’ve been poking around at the Nix merge conflict and it seems like there’s a go version mismatch between flake.nix and go.mod. Based on previous commits, I'll need to update golink = pkgs.buildGo122Module (line 27) to a newer version, buildGo123Module. I don't see a way to call a point release explicitly, so I'll try this first to see if it clears up the conflict. cc @creachadair


Update enumeratum to 1.6.1

Updates com.beachape:enumeratum from 1.5.15 to 1.6.1.
GitHub Release Notes
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.beachape", artifactId = "enumeratum" } ]

labels: library-update, semver-minor

Superseded by #146.


Sum using iterator/reduce

I've opened  PR#95 several days ago, which uses reduce instead of iterator to implement sum function with fewer lines of code. However, a simple benchmark test shows iterator_sum is 3x ~ 5x times faster than reduce_sum.
According to https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/Array/reduce, reduce does have extra overhead compared to directly using iterator.
So the change of  PR#95 may be revoked and using origin sum implementation is better. Using reduce makes the code shorter, it harms efficiency. How do you think about it? @ecrmnn

Thanks for benchmarking and figuring this out @tenghuanhe. Implemented the old method now.


bump kafka_protocol to 4.1.3

Please bump kafka_protocol to version 4.1.3

Thank you for the PR.

released in 3.17.0


[diem-swarm] Add PrivateFullNode tests and Improve test framework for Full Node testing

Motivation
After spending many hours staring out the test framework, I found it was very validator centric, and hadn't been adjusted to a full node ecosystem.  Additionally, there were a lot of rough edges that make the average reader be confused, so hopefully that cleans up those.
Additionally, this makes a simple test to test out PrivateFullNodes and the ability to route traffic through Upstream nodes, and register Downstream nodes only to access them.
Have you read the Contributing Guidelines on pull requests?
Yeah
Test Plan
See CI
Related PRs
None

/canary

:broken_heart: Test Failed - ci-test-success

/canary Let's make sure I didn't break cluster test somehow

:broken_heart: Test Failed - ci-test-success

/land

:broken_heart: Test Failed - ci-test-success

Turns out it's really easy to not have a fallback for a VFN with 4 nodes, so I upped the nodes for the failing test to 7
/land

:broken_heart: Test Failed - ci-test-success

/land


Additional items needed in Netlist

[ ] - Indexing into arrays
[ ] - Bitvector slicing

Actually I think this can be done with the existing wiring


NDRS-940: Stop sync after very recent block.

We used to mark syncing as done if none of the peers gave us any new block (successor of the latest block we downloaded). This proves to be problematic when we have many peers and they don't respond to us in time, before the next block is created. This way we end up being stuck in a syncing loop.
This PR considers newly downloaded block to be the tip of a chain if it was created less than 5 seconds ago.

I'd even make it a minute (or rather: one minimum round length). Five seconds doesn't sound realistic: It might take that long just to execute it, so by the time the peers send it it would already be older than that.

I was thinking about using rounds here but this is highway-specific for now and LinearChainSync is generic. At least for now.

So should we just make it a minute for now?

So should we just make it a minute for now?

Already pushed.

bors merge


Indicate crawl alert

When a game is nearing it's end, tension rises when a player or team has not scored yet. Indicate which players are on 'crawl alert' in a game, starting from a 5+ score.
Rules. A player must crawl when:

it's an attacker and has not scored
it's a defender and the attacker has not scored

Notes:

own goals of the opposing team count as goals for the defender
there is no crawling after Friday 17:00h


net-online: fix ping test timeout

ping doesn't wait, if only one probe was requested.

Hi @DocMAX!
I made this pull request two years ago, and frankly don't remember why :)
Can you please help by describing your use-case?

Should be fixed by https://github.com/OpenRC/openrc/commit/e21b01b97e84f81589f52e087d2b2ac53f0bf144.


if UPDATE_GCC_BINUTILS is set, install gcc-4.8 and binutils 2.24

this is primarily needed for mirage-entropy-xen which depends on mnemonics only available in more recent compilers + as

Thanks!


Deprecation Cop: Check for duplicate issue before creating new

The Deprecation Cop encourages users to create issues on repositories when deprecated features are being used. This may lead to a lot of duplicate issues.

Duplicate of atom/deprecation-cop#21, please subscribe there for updates.


update officedown template

v 0.2.3 of officedown introduces some new chunk options that should probably be specified in the template:

support for knitr chunk options tab.topcaption and tab.lp.
support for knitr chunk options tab.cap.tnd, tab.cap.tns and tab.cap.fp_text that let define prefix for auto-number table captions. Same with fig.cap.tnd, fig.cap.tns and fig.cap.fp_text for plots.

closed with #12


Fix #394: Add ignore-case and ignore-whitespace options to react-a11y-anchors

Fixes #394.
Added toggleable ignore-case and ignore-whitespace options to react-a11y-anchors rule.
Also added unit tests and documentation for these options.

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

Hi @JoshuaKGoldberg, I've updated the pull request with the proposed changes. So now to use the new options, user can just update their tslint.json with this:
react-a11y-anchors: {
    true,
    "ignore-case",
    {
        "ignore-whitespace": "all"
    }
}

The Node 6 build timed out on an unrelated rule... re-running. Pretty sure it was just intermittent noise. https://travis-ci.org/Microsoft/tslint-microsoft-contrib/jobs/438391623


[XLA] Add half as supported type for TruncatedNormal op definition

Add the half type as a supported type for the truncated normal random op.  Backends can still decide on a per-backend basis whether they support that type or not.

Do you mind adding a test case to tensorflow/compiler/tests/random_ops_test.py ?

sure thing

Hi,
actually it already does test half types:
  def _random_types(self):
    return set(self.numeric_types) - set(
        self.complex_types) - {np.uint64, np.int64, np.uint8, np.int8}

Hi,
actually it already does test half types:

If there was a test it would have been failing. :)
I think you only need to change this line (and remove the TODO): https://github.com/tensorflow/tensorflow/blob/b1d7145be23670dc412659f36d62996d024ef709/tensorflow/compiler/tests/random_ops_test.py#L124

you are absolutely right.  I had checked the tests like:
  def testRandomNormalVariance(self):
    for dtype in self._random_types() & self.float_types:

but the testTruncatedNormalIsInRange is different.  will patch.

obviously i've only tested on the CPU and IPU backends, it could be that allowing all float types in that test will cause the GPU or TPU to get upset.  hopefully not.

i see. the 'truncated normal is not constant' doesn't test all types either. you are quite right - there were no tests at all.

done :)

@gbaned  I think I have sorted all of the requested changes here.

Hi,
i had to resolve an issue with this test changing slightly.  it seems that someone has added double support, so that is now in, but i have left in the checks for CPU/GPU w.r.t. halves.
i left the TODO(sanjoy) in, even though there is a separate TODO now for getting the test working with halves.

@DavidNorman Could you please check Ubuntu Sanity errors? Thanks!

@gbaned done :)

i've had to abandon the test changes.  there is someone inside google who is also working on getting half formats into the random test, and it keeps conflicting.
please consider merging this without the test changes.
this is the code: b/34339814

I've realised I can achieve the same result as this by overriding the TF2XLA implementation of truncated normal and specifying a backend-specific implementation of truncated normal.
closing this as no longer necessary.


feat: implement HistogramFold plan for prometheus histogram type

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
Part one of prometheus histogram. The next part is about planner and quantile.
HistogramFold
HistogramFold will fold the conventional (non-native) histogram (1) for later
computing. Specifically, it will transform the le and field column into a complex
type, and samples on other tag columns:

le will become a [ListArray] of [f64]. With each bucket bound parsed
field will become a [ListArray] of [f64]
other columns will be sampled every bucket_num element, but their types won't change.

Due to the folding or sampling, the output rows number will become input_rows / bucket_num.
le should be the same across the entire metric. Thus it can be a Dict<List<f64>> to reduce size. However this is not viable at present because I cannot construct such a dict array 🥲
Requirement

Input should be sorted on <tag list>, le ASC, ts.
The value set of le should be same. I.e., buckets of every series should be same.

Result
It's output looks like
+--------+---------------------------------+--------------------------------+
| host   | le                              | val                            |
+--------+---------------------------------+--------------------------------+
| host_1 | [0.001, 0.1, 10.0, 1000.0, inf] | [0.0, 1.0, 1.0, 5.0, 5.0]      |
| host_1 | [0.001, 0.1, 10.0, 1000.0, inf] | [0.0, 20.0, 60.0, 70.0, 100.0] |
| host_1 | [0.001, 0.1, 10.0, 1000.0, inf] | [1.0, 1.0, 1.0, 1.0, 1.0]      |
| host_2 | [0.001, 0.1, 10.0, 1000.0, inf] | [0.0, 0.0, 0.0, 0.0, 0.0]      |
| host_2 | [0.001, 0.1, 10.0, 1000.0, inf] | [0.0, 1.0, 2.0, 3.0, 4.0]      |
+--------+---------------------------------+--------------------------------+

Checklist

[x]  I have written the necessary rustdoc comments.
[x]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

related to #1042

Sorry, but there should be only one series each line? like it should be:
+--------+---------------------------------+--------------------------------+
| host   | le                              | val                            |
+--------+---------------------------------+--------------------------------+
| host_1 | [0.001, 0.1, 10.0, 1000.0, inf] | [0.0, 1.0, 1.0, 5.0, 5.0]      |
| host_2 | [0.001, 0.1, 10.0, 1000.0, inf] | [0.0, 1.0, 2.0, 3.0, 4.0]      |
+--------+---------------------------------+--------------------------------+


Tilt Requires Gems for Ignored Paths

Expected behavior and actual behavior
I have a directory that I want to be ignored during the build process, but Middleman still attempts to load the required gem for Tilt for any special file extensions in the ignored folder.
Steps to reproduce the problem (from a clean middleman installation)
Add a file with an extension that would require an added gem (e.g. LESS, TypeScript, LiveScript) and ignore the file. Then attempt to start a sever.
You'll get this error for whatever library tilt would require.
 `require': cannot load such file -- typescript-node (LoadError)
  from gems/tilt-2.0.7/lib/tilt/typescript.rb:2:in `<top (required)>'

https://github.com/fny/middleman_ignore_bug_example/commit/36267dcae6c6508f5b1e74fad152aa654686c998
Additional information

Ruby version: 2.4.0
Middleman version: 2.4.1
OS version: OS X 10.11

This behavior is also described in #2012


Reply-To, FromName - Not supported

Hi,
Would you please like to add support for reply-to and fromName ?

You can use the following criteria when doing names 'James Bond<EMAIL_ADDRESS>I have opened a new task for the reply-to option thanks for reporting

Is this resolved yet??? it's an important feature


CLI Command: "landscaper quickstart"

Description
The Landscaper CLI needs to provide an easy way to enable local development scenarios. This command should include the landscaper installation itself (into an already existing cluster, this is a prerequisite), and the installation of a locally running OCI Registry, which should be wired to the landscaper as default OCI Registry.
Done Criteria

[x] Concept for "landscaper init" created, discussed and agreed
[x] All needed Features implemented
[x] Code has been reviewed by other team members
[ ] Unit Tests created for new code
[x] Integration Test Suite updated (if applicable)
[x] Internal technical Documentation updated
[x] Enduser Documentation updated (if applicable)
[x] Successful demonstration in Review

Implementation is done.
Missing: Tests for the "landscaper init" command (which is actually called "landscaper quickstart"), which need to be added to the CLI Test Suite. Can only be done if #4 has been finished.

Implementation is done.
Missing: Tests for the "landscaper init" command (which is actually called "landscaper quickstart"), which need to be added to the CLI Test Suite. Can only be done if #4 has been finished.

Integration Tests for the landscaper quickstart command have now been added to the Integration Test Suite


Request for adding Portfolio Webite

I want to add my portfolio website.
Please have a look  - https://portfolio-ruby-seven-56.vercel.app/

Please assign it to me


Keep input when users refresh currency rates

Fixes #260
Description of the changes:

prevent UnitConverterViewModel to reset values when users click on update rates.
recompute UnitConverter's caches (m_ratioMap and m_categoryToUnits) once rates are updated (but check first if the user did/didn't change the category)

How changes were validated:

Manually tested with fake currency rates (HTTP responses modified on the fly via FiddlerCore)
Verified that it works no matter the selected field (From or To)
Verified that the currencies selected are kept after a refresh

@rudyhuyn, looks like UTs are failing.


Sometimes comments sheet is emphy

ie. it looks transparent - but it is probably just not showing.

Ie. it is empty for some entries, but not for others.


fix(pdb-minavailable-check): ensure correct handling of 0 replicas workloads

Related Issue(s)

Slack-thread: https://kubernetes.slack.com/archives/CLGR9BJU9/p1698916081905549
Description

This PR adds another precondition to pdb-minavailable-check policy - to ensure that 0 replicas workloads are not denied by the policy. We added this policy to our clusters and experienced a lot of false-positives on workloads with replicas: 0, and no PDBs in namespace. I think this happens because the context variable defaults to 0.
Checklist

[x] I have read the policy contribution guidelines.
[x] I have added test manifests and resources covering both positive and negative tests that prove this policy works as intended.
[x] I have added the artifacthub-pkg.yml file and have verified it is complete and correct.

Thanks for this! We do need DCO sign-off on all commits though.


[Question] Is there any way to read why the Discord client crashed?

Is there any way to read why the Discord client crashed?
The log file is empty and this message is useless.

Thanks
EDIT:
Solution for my particular crash issue: https://github.com/BetterDiscord/BetterDiscord/issues/994#issuecomment-901198533

Why do you think that is useless? Have you tried removing all of your plugins to see if it still crashes like the message tells you to do?

It doesn't crash, yes I've tried to remove plugins.
The problem is that even if I add only one libs it crashes, so I'd like to check what is the cause and possibly fix it myself.

Starting Install...
Creating required directories...
✅ Directory exists: C:\Users\mc201\AppData\Roaming\BetterDiscord
✅ Directory exists: C:\Users\mc201\AppData\Roaming\BetterDiscord\data
✅ Directory exists: C:\Users\mc201\AppData\Roaming\BetterDiscord\themes
✅ Directory exists: C:\Users\mc201\AppData\Roaming\BetterDiscord\plugins
✅ Directories created
Downloading asar file
❌ Failed to download package https://api.github.com/repos/rauenzi/BetterDiscordApp/releases
❌ Cannot read property 'assets' of undefined
The install seems to have failed. If this problem is recurring, join our discord community for support. https://discord.gg/0Tmfo5ZbORCRqbAd
Help Pls

@GhostzinRs That is unrelated to this issue

@LoneDev6 The cause of this particular crash is probably "0PluginLibrary.plugin.js". Just delete it and install any plugin that uses it. Then you will get a pop-up telling where you can click "install" to install the latest version of that library, which has the crash bug fixed. Many users had a crash today and this is what fixed it.
But I agree, it would be nice if there was a crash log directly telling you which plugin has caused the crash.

@LoneDev6 The cause of this particular crash is probably "0PluginLibrary.plugin.js". Just delete it and install any plugin that uses it. Then you will get a pop-up telling where you can click "install" to install the latest version of that library, which has the crash bug fixed. Many users had a crash today and this is what fixed it.
But I agree, it would be nice if there was a crash log directly telling you which plugin has caused the crash.

This fixed it for me. Thanks for the Help :)

The cause of this particular crash is probably "0PluginLibrary.plugin.js". Just delete it and install any plugin that uses it. Then you will get a pop-up telling where you can click "install" to install the latest version of that library, which has the crash bug fixed. Many users had a crash today and this is what fixed it.

Well, that's right. Hope this works for me! And for reference: https://github.com/rauenzi/BDPluginLibrary/pull/94

@LoneDev6 The cause of this particular crash is probably "0PluginLibrary.plugin.js". Just delete it and install any plugin that uses it. Then you will get a pop-up telling where you can click "install" to install the latest version of that library, which has the crash bug fixed. Many users had a crash today and this is what fixed it.
But I agree, it would be nice if there was a crash log directly telling you which plugin has caused the crash.

Related to the crash issue, ZeresPluginLibrary pops up with error log

Related to the crash issue, ZeresPluginLibrary pops up with error log
That's not actually an error log.

That's just what rauenzi made as an explanation for what was going on last time, in the form of a notification used for Discord Updates.

Starting Install...
Creating required directories...
✅ Directory exists: C:\Users\mc201\AppData\Roaming\BetterDiscord
✅ Directory exists: C:\Users\mc201\AppData\Roaming\BetterDiscord\data
✅ Directory exists: C:\Users\mc201\AppData\Roaming\BetterDiscord\themes
✅ Directory exists: C:\Users\mc201\AppData\Roaming\BetterDiscord\plugins
✅ Directories created
Downloading asar file
❌ Failed to download package https://api.github.com/repos/rauenzi/BetterDiscordApp/releases
❌ Cannot read property 'assets' of undefined
The install seems to have failed. If this problem is recurring, join our discord community for support. https://discord.gg/0Tmfo5ZbORCRqbAd
Help Pls

Uninstall and reinstall the installer. It worked for me

@LoneDev6 The cause of this particular crash is probably "0PluginLibrary.plugin.js". Just delete it and install any plugin that uses it. Then you will get a pop-up telling where you can click "install" to install the latest version of that library, which has the crash bug fixed. Many users had a crash today and this is what fixed it.
But I agree, it would be nice if there was a crash log directly telling you which plugin has caused the crash.

Thanks! Working perfectly :D

Why do you think that is useless? Have you tried removing all of your plugins to see if it still crashes like the message tells you to do?

...Because that is useless. It tells you only the most obvious thing: "something caused a crash, try removing everything so it doesn't crash".
It does not tell you why it crashed, it doesn't give you any info at all beyond stating the obvious you notice without the popup as soon as discord's window goes blank.
Additionally, you can't even restart with dev tools open, cause they close as well, and there are no actual logs that could help you, so you are actually stuck with "does this one crash? does this one?"
Now, i get that it might be difficult to create a log of a crash when it occurs in this case, but nonetheless the popup is quite useless in its current form.

I don't see why this has closed, this issue still exists today.

@josephworks same.
We need the exception and trace to figure out the plugin that's causing the issue. My Discord crashes about once per day and I'm not going to disable all plugins and then re-enable one every 2-3 days to make sure it's stable, lol. Just give us the error please.

@dav1312 could you reopen this issue?

+1

@josephworks same. We need the exception and trace to figure out the plugin that's causing the issue. My Discord crashes about once per day and I'm not going to disable all plugins and then re-enable one every 2-3 days to make sure it's stable, lol. Just give us the error please.

+1 to this.

Why is this closed? There should be the option for real logging and exceptions and traces to figure out stuff like this. My discord is also crashing every 6 or so hours and I don't want to go through and add one plugin at a time until I stop getting a crash...

Agree that this should probably be reopened, but just as a heads up these errors leave no traces. Renderer crashes are caused by v8 itself due to the context leak that BD uses and the error's origins aren't traceable.

Agree that this should probably be reopened, but just as a heads up these errors leave no traces. Renderer crashes are caused by v8 itself due to the context leak that BD uses and the error's origins aren't programatically traceable.

Is there any way at all then to tell what causes these crashes or are we just screwed

Would like to see more discussion on this topic, this is still a real issue

Yeah, why is this closed?

Just checked and I have asked previously for this to be reopened, but it doesn't appear that that has been heard.

Wow. Seriously? This has been going on for so long and no one at BetterDiscord's team cares at all to give us some proper information? I love BD but that is a bad joke.

There is no log generated in a full crash like that, there's not really much information I can provide. You can sometimes see the cause of the error if you run Discord through the command line, then errors will sometimes be shown in the terminal.

rauenzi said:
There is no log generated in a full crash like that, there's not really much information I can provide. You can sometimes see the cause of the error if you run Discord through the command line, then errors will sometimes be shown in the terminal.

Yikes... that's unfortunate. Guess only trial-and-error can solve it, which is a pain to do.

It's very unfortunate, it makes debugging these kinds of errors nearly impossible.


Fail to create custom dataset statistics for KID computation

Can I create custom dataset statistics to compute KID?
In my trial, after make_custom_stats, the invoking of compute_fid works fine but compute_kid tries to download the statistics:
>>> fid.compute_kid('sunglasses/0', dataset_name='sunglasses', mode='clean', dataset_res=256, dataset_split='custom')
compute KID of a folder with sunglasses statistics
downloading statistics to /home/brando/.local/lib/python3.7/site-packages/cleanfid/stats/sunglasses_clean_custom_na_kid.npz
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/brando/.local/lib/python3.7/site-packages/cleanfid/fid.py", line 335, in compute_kid
    mode=mode, seed=0, split=dataset_split, metric="KID")
  File "/home/brando/.local/lib/python3.7/site-packages/cleanfid/features.py", line 95, in get_reference_statistics
    fpath = check_download_url(local_folder=stats_folder, url=url)
  File "/home/brando/.local/lib/python3.7/site-packages/cleanfid/downloads_helper.py", line 27, in check_download_url
    with urllib.request.urlopen(url) as response, open(local_path, 'wb') as f:
  File "/usr/lib/python3.7/urllib/request.py", line 222, in urlopen
    return opener.open(url, data, timeout)
  File "/usr/lib/python3.7/urllib/request.py", line 531, in open
    response = meth(req, response)
  File "/usr/lib/python3.7/urllib/request.py", line 641, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python3.7/urllib/request.py", line 569, in error
    return self._call_chain(*args)
  File "/usr/lib/python3.7/urllib/request.py", line 503, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.7/urllib/request.py", line 649, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 404: Not Found

Detailed context can be found in https://github.com/GaParmar/clean-fid/issues/9.

I have the same problem for FID and get the same error.

For the custom stats error, it looks like your custom dataset statistics file was not created.
Can you check the files in the folder os.path.join(os.path.dirname(cleanfid.__file__), "stats")

Looking at this lines you do not need these normalization.
jit detector
If you look at the forward pass function defined within the torchscript inception model, the normalization applied is the same is the one in cleanfid's inceptionv3W class.


MagicMirror will not start

MagicMirror will not start. I have run MM for over 3 years. I ran the recent package; however it will not start. I checked and upgraded Node and NPM. Still no start. Following error was given for npm start:
pi@raspberrypi:~/MagicMirror $ npm start

<EMAIL_ADDRESS>start
DISPLAY="${DISPLAY:=:0}" ./node_modules/.bin/electron js/electron.js

/home/pi/MagicMirror/node_modules/electron/dist/electron: error while loading shared libraries: libxkbcommon.so.0: cannot open shared object file: No such file or directory
Any ideas?

you are missing a system library
sounds like you are not using the desktop

Actually; I am using the desktop? I have the full GUI up and running without any noted problems (I sent you the error message above via chrome from the Raspberry pi desk top). Where might I find the library to download and where does it need to be placed? Very odd as I have operated MM for several years and have downloaded numerous MM systems for various projects and have never had this problem before; albeit, it sounds like an error in the OS rather than the MM.

Google search.  did u load the 64 bit version?

Fixed it; indeed I was missing the library. Not sure why. I used sudo apt-get install libxkbcommon0, and the MM came up and started working. Probably something in the OS that was not downloaded. Thanks.

@ejmorgan55 can u close this now?


[Spanish] localize 'Database as a Service (DBaaS)'

Localize the content/en/database_as_a_service.md document to Spanish

Duplicated of https://github.com/cncf/glossary/issues/605
Also this term is marked as deprecated


Supporting APL

(Original issue 969 created by None on 2014-03-22T21:42:31.209000+00:00)
APL is quite active now, with Dyalog APL, several online versions of the free javascript NGN APL interpreter, and the very actively maintained GNU APL. Could Pygments support APL in the future? Regards.

(Original issue 969 last updated on 2014-03-24T07:23:44.994755+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by baruchel on 2014-03-23T13:12:18.024808+00:00)
Here is a working lexer:
#!python

class APLLexer(RegexLexer):
    """
    A simple APL lexer.
    Contributed by Thomas Baruchel<EMAIL_ADDRESS>    """
    name = 'APL'
    aliases = ['apl']
    filenames = ['*.apl']

    tokens = {
        'root': [
            # Whitespace
            # ==========
            (r'\s+', Text),
            #
            # Comment
            # =======
            # '⍝' is traditional; '#' is supported by GNU APL and NGN (but not Dyalog)
            (u'[⍝#].*$', Comment.Single),
            #
            # Strings
            # =======
            (r'\'((\'\')|[^\'])*\'', String.Single),
            (r'"(("")|[^"])*"', String.Double), # supported by NGN APL
            #
            # Punctuation
            # ===========
            # This token type is used for diamond and parenthesis
            # but not for bracket and ; (see below)
            (u'[⋄◇()]', Punctuation),
            #
            # Array indexing
            # ==============
            # Since this token type is very important in APL, it is not included in
            # the punctuation token type but rather in the following one
            (r'[\[\];]', String.Regex),
            #
            # Distinguished names
            # ===================
            # following IBM APL2 standard
            (u'⎕[A-Za-zΔ∆⍙][A-Za-zΔ∆⍙_¯0-9]*', Name.Function),
            #
            # Labels
            # ======
            # following IBM APL2 standard
            # (u'[A-Za-zΔ∆⍙][A-Za-zΔ∆⍙_¯0-9]*:', Name.Label),
            #
            # Variables
            # =========
            # following IBM APL2 standard
            (u'[A-Za-zΔ∆⍙][A-Za-zΔ∆⍙_¯0-9]*', Name.Variable),
            #
            # Numbers
            # =======
            (u'¯?(0[Xx][0-9A-Fa-f]+|[0-9]*\.?[0-9]+([Ee][+¯]?[0-9]+)?|¯|∞)([Jj]¯?(0[Xx][0-9A-Fa-f]+|[0-9]*\.?[0-9]+([Ee][+¯]?[0-9]+)?|¯|∞))?', Number),
            #
            # Operators
            # ==========
            (u'[\.\\\/⌿⍀¨⍣⍨⍠⍤∘]', Name.Attribute), # closest token type
            (u'[+\-×÷⌈⌊∣|⍳?*⍟○!⌹<≤=>≥≠≡≢∊⍷∪∩~∨∧⍱⍲⍴,⍪⌽⊖⍉↑↓⊂⊃⌷⍋⍒⊤⊥⍕⍎⊣⊢⍁⍂≈⌸⍯↗]', Operator),
            #
            # Constant
            # ========
            (u'⍬', Name.Constant),
            #
            # Quad symbol
            # ===========
            (u'[⎕⍞]', Name.Variable.Global),
            #
            # Arrows left/right
            # =================
            (u'[←→]', Keyword.Declaration),
            #
            # D-Fn
            # ====
            (u'[⍺⍵⍶⍹∇:]', Name.Builtin.Pseudo),
            (r'[{}]', Keyword.Type),
        ],
    }

(Original comment by birkenfeld on 2014-03-23T17:16:52.724276+00:00)
Thanks for the contribution. To finish this, could you turn it into a pull request complete with an example file for tests/examplefiles?

(Original comment by baruchel on 2014-03-23T21:03:05.762108+00:00)
@birkenfeld. I don't know how to do the pull request you are asking for. I can perfectly send all files to anyone by mail however.

(Original comment by birkenfeld on 2014-03-23T22:01:01.006280+00:00)
OK, no problem, just attach them to this issue then.

(Original comment by baruchel on 2014-03-23T22:12:22.676667+00:00)
@birkenfeld Here is an example below, and both the code and the example will be attached to the issue.
#!APL

∇ R←M COMBIN N;D;E;F;G;P
  ⍝ Returns a matrix of every possible
  ⍝ combination of M elements from the
  ⍝ vector ⍳N.  That is, returns a
  ⍝ matrix with M!N rows and N columns.
  ⍝
  E←(⍳P←N-R←M-1)-⎕IO
  D←R+⍳P
  R←(P,1)⍴D
  P←P⍴1
 L1:→(⎕IO>1↑D←D-1)⍴0
  P←+\P
  G←+\¯1↓0,F←⌽P
  E←F/E-G
  R←(F/D),R[E+⍳⍴E;]
  E←G
  →L1
∇

∇ R←M QUICKEXP N
  ⍝ Matrix exponentiation
  B ← ⌊ 1 + 2 ⍟ N
  V ← (B ⍴ 2) ⊤ N
  L ← ⊂ M
  R ← ⊃ +.× / V / L ⊣ { L ← (⊂ A +.× A ← ↑L) , L }¨ ⍳ B-1
∇

(Original comment by birkenfeld on 2014-03-24T07:23:20.881468+00:00)
Thanks!

(Original comment by birkenfeld on 2014-03-24T07:23:44.987081+00:00)
Closes #969: add APL lexer, by Thomas Baruchel.
→ <>


from_argparse_args on child classes doesn't pass around the correct arguments

🐛 Bug
For my research I made a hierarchy of classes where the parent BaseDataModule inherits from pl.LightningDataModule and all other child classes inherit from BaseDataModule. Initializing a child class using from_argparse_args takes the default values for attributes inherited from BaseDataModule and ignores values passed from the command line.
To Reproduce
In main.py:
import pytorch_lightning as pl
from argparse import ArgumentParser

class BaseDataModule(pl.LightningDataModule):
    def __init__(
            self,
            batch_size: int = 4,
    ):
        super(BaseDataModule, self).__init__()
        self.batch_size = batch_size
        # use batch_size later for dataloaders

class SpecialDataModule(BaseDataModule):

    def __init__(
            self,
            some_arg: int = 1,
            **kwargs
    ):
        super(SpecialDataModule, self).__init__(**kwargs)
        self.some_arg = some_arg

if __name__ == "__main__":
    parser = ArgumentParser()

    # for BaseDataModule
    parser.add_argument("--batch_size", type=int)
    # for SpecialDataModule
    parser.add_argument("--some_arg", type=int)

    args = parser.parse_args()

    print(args.batch_size)
    dm = SpecialDataModule.from_argparse_args(args)
    print(dm.batch_size)

Run using: python main.py --batch_size=10 --some_arg=20
Expected behavior
My datamodule should show a batch_size of 10 as passed in the command line arguments but takes the default value 4.
Environment

PyTorch Lightning Version (e.g., 1.5.0): 1.5.10
PyTorch Version (e.g., 1.10): 1.10.2+cu102
Python version (e.g., 3.9): 3.6.9
OS (e.g., Linux): Ubuntu 18.04
CUDA/cuDNN version: 11.4

Dependencies as given in my project's pyproject.toml:
[tool.poetry.dependencies]
python = "^3.6.2"
pytorch-lightning = "^1.5.9"
jupyterlab = "^3.2.8"
numpy = "<1.20.0"
sklearn = "^0.0"
poetry2setup = "^1.0.0"
nose = "^1.3.7"
Markdown = "3.3.4"
mlxtend = "^0.19.0"
wandb = "^0.12.10"
plotly = "^5.6.0"
differint = "^0.3.2"

[tool.poetry.dev-dependencies]
pytest = "^5.2"

[build-system]
requires = ["poetry-core>=1.0.0"]
build-backend = "poetry.core.masonry.api"

I looked at it, and it looks like it filters out arguments that don't belong to the selected class (parents are not considered) - so that only class-specific ones get called.
https://github.com/PyTorchLightning/pytorch-lightning/blob/7938293cd9bb27e5ad97aabc7261fb29027dc6b3/pytorch_lightning/utilities/argparse.py#L67-L70
It seems to me that this filtering is necessary and we cannot do without it, at the same time the use of lightning as described in the issue seems reasonable and should be allowed.
Possible solutions I see

If we want to support more complex cases (e.g. two levels of inheritance) we need to analyse all the 'available' arguments by going up the inheritance tree recursively.
Don't change anything - just give user warning if cls during filtering is not a direct child of LightningDataModule or LightningModule - this solution has a lot of cons (requires updates if something changes in other modules, users are not able to do their architecture flexible itp.)
Move logic of keeping allowed parameters list inside classes. For example, DataModule and the others modules will have some internal method that sets self.allowed_params += inspect.signature(self.__class__.__init__).parameters and we somehow force child class to run this method too, then in argparser we simply use:

 trainer_kwargs = {name: params[name] for name in valid_kwargs if name in cls.allowed_params}
 trainer_kwargs.update(**kwargs)

Remove filtering? But this will require from users to add **kwargs, and will be probably painful breaking change...

class BaseDataModule(pl.LightningDataModule):
    def __init__(
            self,
            batch_size: int = 4,
            **kwargs
    ):

I don't know, each of the solutions has downsides, maybe someone knows how to find all parameters that can be passed to function? I did not find something like that. @carmocca what do you think?

FYI @agarwalmanvi the code will work if you simply get rid of  **kwargs, but you have to do that in each SpecialDataModule

Hi! This is a problem that also impacts the CLI (see https://github.com/PyTorchLightning/pytorch-lightning/issues/11653).
The ideal solution would be (1) but it's complex to implement where an easy workaround already exists (unrolling **kwargs as already described). cc @mauvilsa in case you have thoughts on this.
I strongly suggest updating to use the LightningCLI over the old argparse logic if you can. Your example will work well with it:
import pytorch_lightning as pl

from pytorch_lightning.demos.boring_classes import BoringModel
from pytorch_lightning.utilities.cli import LightningCLI

class BaseDataModule(pl.LightningDataModule):
    def __init__(self, batch_size: int = 4):
        super(BaseDataModule, self).__init__()
        self.batch_size = batch_size
        # use batch_size later for dataloaders

class SpecialDataModule(BaseDataModule):
    def __init__(self, some_arg: int = 1, **kwargs):
        super(SpecialDataModule, self).__init__(**kwargs)
        self.some_arg = some_arg

cli = LightningCLI(BoringModel, SpecialDataModule, run=False)
print(cli.datamodule.batch_size)

python main.py --data.batch_size=10 --data.some_arg=20

Great, so we are waiting for @mauvilsa ;)

I would suggest the same as @carmocca, to use LightningCLI instead of from_argparse_args. The example above already works with LightningCLI because it assumes that **kwargs are forwarded with super().__init__ and thus finds the parameters of all parent classes. And I am working on improving this support so that the search for parameters is not just based on assumptions but based on the code itself.
The same support could be implemented for from_argparse_args, however, is this effort justified? LightningCLI does the same as from_argparse_args requiring less code to implement and providing many more features out of the box. Keeping from_argparse_args makes sense so that old code keeps working. But when implementing new things I don't know any good reason to not use LightningCLI.

So to sum up, we can mark this as "won't do", can't we?


Suggested edits to 'about us'

DIME is the World Bank's impact evaluation department. Part of DIME’s mission is to intensify the production of and access to public goods that improve the quantity and quality of global development research, while lowering the costs of doing IE for the entire research community. This Library is developed and maintained by DIME Analytics. DIME Analytics supports quality research processes across the DIME portfolio, offers public trainings, and develops tools for the global community of development researchers.

hyperlinks for DIME (https://www.worldbank.org/en/research/dime) and DIME Analytics (https://www.worldbank.org/en/research/dime/data-and-analytics)


V5: Consider adding form floating label color

Please consider adding a $form-floating-label-color variable to control the color of the floating label.

Closing as stale and a won't fix—going to keep the floating forms as close to default forms as possible, which includes inheriting their styles.

I understand that one wants to keep them as close to the default as possible however this comes with some problems. As the floating label is not a descendent of the <input> element it does not inherit it's color but instead to surrounding color (usually body). If surrounding element has a dark background and consequently a light font color the label inherits this and becomes illegible. A good example of this is a dark card.

Now this can obviously be solved by explicitly declaring a color on the label element however this can become quite repetitive.
A good example of this are dark themes which have to explicitly add a css selector for this case:
https://github.com/thomaspark/bootswatch/blob/d982c0baf3e425df284cbefa1a12620d6fbaf6e0/dist/darkly/_bootswatch.scss#L26-L28
https://github.com/thomaspark/bootswatch/blob/d982c0baf3e425df284cbefa1a12620d6fbaf6e0/dist/superhero/_bootswatch.scss#L72-L76
This is certainly a possibility though again comes with disadvantages (Bootstraps preliminary/WIP dark mode itself cleverly evaded this issue from the start by giving inputs a dark background). Ideally a theme should only require changing variables and nothing more. With bootstrap adopting more and more customizability I think this is a worthwhile change. A default value of currentColor (or $input-color) is also fine and does not hurt anyone.

@mdo I was just looking up this issue, currently (and maybe it's me, or the markup that I'm using) it's like @septatrix mentioned, the variables used to style regular labels do not apply to floating labels. I'm having to add some custom scss to apply the form label styles, right now in my project without it, it's using body text colour and font weight for example.
I'll share my markup in case it's relevant and anyone wants to look into it, because possibly that's what's wrong :-).

    <input id="df1c73b7-9a45-47d9-9a44-fcd357413122" placeholder="Name" name="_firstFormExampleModel.Name" class="form-control">
    Name


The characters on the screen do not move.

In the termin, I can see the character's actions, but they are not reflected in the screen:http://localhost:8000/simulator_home. And also the Replaying.   The demo can run perfectly.

have the same problem

same problem

same problem

So there are currently issues with the backend server you'll need to fix to get it working, but even a single step can take a good 60+ seconds with the 3 character base because openAI is used so much in a sequential manner. During this time (before a movement/N.json file is created) the character isn't going to do anything until eventually the step completes, the movement file is created for that step, and the frontend picks that up, which is confusing for sure. Once that completes though, the time will update and you will see the characters like so..

Please start the backend first. which is running without errors
Yeah, see the ticket for this.. basically you can't refresh the page or look at character details right now or the frontend will break for some reason.

@frankcarey Could you link to that ticket?

I found out why this problem was occurred. reverie.py didn't check and create movement folder in process. I fixed this problem by #55 .

Even with the movement folder issue fixed, the simulation completed successfully but replaying it does nothing.
However, when I try to replay one of the example simulations provided with the repo, it works perfectly.


[ADD] python courses of cisco, fix extra commas

Add new python courses and delete extra commas

also add in readme file


feat(operation-id-naming-convention): add new operation-id-naming-convention rule

PR summary
This commit introduces the new spectral-style
'operation-id-naming-convention' rule, which replaces
the legacy 'operation_id_naming_convention' rule.
PR Checklist
General checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)
[ ] Dependencies have been updated as needed
[ ] .secrets.baseline updated as needed?

Checklist for adding a new validation rule:

[x] Added new validation rule definition (packages/ruleset/src/rules/*.js, index.js)
[x] If necessary, added new validation rule implementation (packages/ruleset/src/functions/*.js, updated index.js)
[x] Added new rule to default configuration (packages/ruleset/src/ibm-oas.js)
[x] Added tests for new rule (packages/ruleset/test/*.test.js)
[x] Added docs for new rule (docs/ibm-cloud-rules.md)

Checklist for removing an old validation rule:

[x] Removed old rule implementation (packages/validator/src/plugins/validation/*.js)
[x] Removed or adjusted testcases (packages/validator/test)
[x] Updated default configuration to deprecate old rule (packages/validator/src/.defaultsForValidator.js)
[x] Removed docs of old rule (docs/ibm-legacy-rules.md)

:tada: This PR is included in version 0.14.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.65.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Send message to PM

Hi, if I ask the bot for help on a main server channel, how to I program it to send the help to my PM? Thanks, I can't seem to find the right function to do it.

That's not an issue, that's a question, this is the wrong place for it, go to discordapi's #js_discord.'s

@YorkAARGH Link Please?

Documentation relevant to your question:

https://discordjs.readthedocs.io/en/latest/docs_client.html#sendmessage-channel-content-options-callback
http://discordjs.readthedocs.io/en/indev/docs_resolvables.html#id6
A guide to using discord.js: https://eslachance.gitbooks.io/discord-js-bot-guide/content/
Help Channel: https://discord.gg/JcXmf


Some splinter functions are not working

Hi All,
I am using splinter to do some web testing on chrome , I have 2 different scenarios that I am trying to do but nothing happened.
This is the code i am using:
import splinter
browser = splinter.Browser('chrome')
browser.visit (‘http://URL/url.com’)
browser.choose('Period','Day')
browser.select('fromstatsday_sday', '03')

When choosing the radio by name nothing happened (See link)
https://drive.google.com/open?id=0B3gWFncJ_UJfM2ZyMjJjWUowWVE

2.When trying to select the checkbox and the "from day" "03" nothing happen (See link)
https://drive.google.com/open?id=0B3gWFncJ_UJfc3lzNlVBUVBBSmc
I guess i am doing something wrong, I am new with splinter.... So any suggestions ?

Can you paste the html structure?

This is an internal server

I need the html to try to reproduce, if you want you can paste a similar html data.

https://drive.google.com/folderview?id=0B3gWFncJ_UJfV19Pak42MGwwejA&usp=sharing
I hope this link will help

Any update on this? did the html helped to investigate this ?

Can someone suggest on this?

@andrewsmedina Given the age of this issue and nobody else raising it, I think it's best to kick it out of 1.0 and close.


MQTT access with the Traefik

Using the https://docs.teslamate.org/docs/guides/traefik instructions - how would one get access to the MQTT broker?

From an external application (e.g. a MQTT client app on a smartphone), use hostname:1883 where hostname is your Docker machine.

From another Docker container (e.g. Node-RED), use mosquitto:1883.


Keep snapPoints with dismissible

hey,
Does it's possible to keep snapoints enabled, but with dismissible={true}.
Regards

Added in #113

Does it's published on NPM?


ES6 : Angular 1.6 : $injector:modulerr

hi,
I have this error in my app :
Uncaught Error: [$injector:modulerr] Failed to instantiate module app due to: Error: [$injector:modulerr] Failed to instantiate module function Loki(filename, options) due to: Error: [$injector:unpr] Unknown provider: filename
I not found where module filname was injected in Loki !

filename is a variable (string).

I know, why angualr think is an unknow provider ?

The problem comme when inject Loki in angular.
I use webpack with babelJS to build my app.

no idea ?

I try  this change Loki declaration :
 //function Loki(filename, options) {     function Loki() {

Failed to instantiate module function Loki() due to:
ReferenceError: filename is not defined

If I can help you, and you me :(


Adds Gerty Extension

Codecov Report

Merging #1207 (53be745) into main (1d509fb) will decrease coverage by 1.11%.
The diff coverage is 26.47%.

:exclamation: Current head 53be745 differs from pull request most recent head e69f2be. Consider uploading reports for the commit e69f2be to get more accurate results

@@            Coverage Diff             @@
##             main    #1207      +/-   ##
==========================================
- Coverage   49.88%   48.77%   -1.12%
==========================================
  Files         255      263       +8
  Lines       14715    15376     +661
==========================================
+ Hits         7340     7499     +159
- Misses       7375     7877     +502

Impacted Files
Coverage Δ

lnbits/extensions/gerty/helpers.py
9.13% <9.13%> (ø)

lnbits/extensions/gerty/number_prefixer.py
18.18% <18.18%> (ø)

lnbits/extensions/gerty/crud.py
28.00% <28.00%> (ø)

lnbits/extensions/gerty/views_api.py
45.71% <45.71%> (ø)

lnbits/extensions/gerty/views.py
78.26% <78.26%> (ø)

lnbits/extensions/gerty/__init__.py
92.30% <92.30%> (ø)

lnbits/extensions/gerty/models.py
96.96% <96.96%> (ø)

lnbits/extensions/gerty/migrations.py
100.00% <100.00%> (ø)

lnbits/db.py
81.20% <0.00%> (-10.07%)
:arrow_down:

lnbits/commands.py
73.61% <0.00%> (-1.39%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Gerty! Please make format and fix mypy errors 🤙


[Grid] support custom columns with nested grid

in reference to issue https://github.com/mui-org/material-ui/issues/28554
A better representation of the above issue would be this
It can be clearly seen from the below image that the container inside the item uses the spacing props of the parent

The correct case should have been this

This can be fixed by just some boolean fixes
The prop had to be put before the value obtained for Gridcontext in the OR operation for it to be used first
The breakpoint unit 12 had to be set later so that it can be used as a default provided the earlier two parameters are null

[x] I have followed (at least) the PR section of the contributing guide.

I think a unit test is needed in this PR.

I think a unit test is needed in this PR.

Cool. Will try to implement the unit test for the change.

@Devesh21700Kumar Hi, thanks for the PR. You can add the following unit test. A codesandbox (e.g., https://codesandbox.io/s/determined-brook-j2bur?file=/src/App.js) or a test is necessary for the maintainers to confirm that the PR is indeed fixing the relevant issue.
diff --git a/packages/mui-material/src/Grid/Grid.test.js b/packages/mui-material/src/Grid/Grid.test.js
index 943f3bcaae..a6c1d6a74f 100644
--- a/packages/mui-material/src/Grid/Grid.test.js
+++ b/packages/mui-material/src/Grid/Grid.test.js
@@ -25,6 +25,24 @@ describe('<Grid />', () => {
       const { container } = render(<Grid container />);
       expect(container.firstChild).to.have.class(classes.container);
     });
+
+    it('should apply the correct number of columns for nested containers', () => {
+      const { getByTestId } = render(
+        <Grid container columns={16}>
+          <Grid item xs={8}>
+            <Grid container columns={8} data-testid="nested-container">
+              <Grid item xs={8} />
+            </Grid>
+          </Grid>
+        </Grid>,
+      );
+      const container = getByTestId('nested-container');
+      // `columns` of nested container should have a higher priority than that of root container
+      // otherwise, max-width would be 50% in this test
+      expect(container.firstChild).toHaveComputedStyle({ maxWidth: '100%' });
+    });
   });

Hey @hbjORbj , Thanks for the help. I've added the test and all the test suites and tests passed without showing any errors.

@Devesh21700Kumar Can you run yarn prettier:all and push again? test_static is failing.
@siriwatknp Hey, argos test is failing. Where do you think this comes from?

Hey, argos test is failing. Where do you think this comes from?

This PR is not updated with master branch, that's all. I have updated it, waiting for green CIs.


Add defaultNavigationOptions params to StackNavigator config

Hello,
I just add a new param, it might be cool in the future to have a NavigationOptions.t for each navigator (in order to avoid bad props).

I will when I came back from holidays 😉
Le ven. 1 nov. 2019 à 09:35, Christoph Knittel<EMAIL_ADDRESS>a
écrit :

Merged #12
https://github.com/reason-react-native/reason-react-navigation/pull/12
into master.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/reason-react-native/reason-react-navigation/pull/12?email_source=notifications&email_token=AAL2BBLIHJKQYHBIDZV4RLLQRPS4TA5CNFSM4JGNQMQKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUSUH5ZY#event-2762505959,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAL2BBP6CGEAAZSYF6FVHJTQRPS4TANCNFSM4JGNQMQA
.


[Linear] Preference to hide completed issues across all searches

Description
Preference to hide completed issues across all searches
Screencast
Preference

Before

After

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

closes: https://github.com/raycast/extensions/issues/11818

Thanks for the PR! Would it make sense to also hide canceled issues?


Add timestamps in cmds.log

Add timestamps when a command was executed so we can easily correlate with actions being performed in the CI.

Not relevant anymore


Support string interpolation in dict keys

This piece of code works:
b = 10
x = {"a": "{{b}}"}

It will print {"a": "10"}
However when using it in a key it will do something wrong:
b = 10
x = {"{{b}}": "a"}

std::print(x)

It will print @{'{{b}}': 'a'}
String interpolation should be supported in keys. If it is not possible, it should at least produce a compiler error.

This is not just a parser fix. Dict keys are expected to be literal strings. A string interpolation string is an expression that requires scheduling. Allowing this will require at least some and possibly major changes to how dictionaries are represented (and used) by the compiler. Knowing the scope of this ticket is not as small as it seemed, should this remain in the next column?
Detecting it and raising an exception or a warning would be straightforward.

detect this and raise an exception

Hey team! Please add your planning poker estimate with ZenHub @arnaudsjs @bartv @FloLey @Hugo-Inmanta @sanderr


ISA Json Export - Inconsistent ontologySourceReferences

The ontologySourceReferences gets populated upon ISA JSON file export, and it means the applied terms are not coming from the version necessarily.
Solution: Store ontologies information (name, file, version, description) when terms are getting populated

Important feature to be discussed and implemented with the entire SEEK team.


Updates on Reanalyse / Sample Efficiency (Re-executing MCTS, Parallelization, Stabilization with a target model, etc.)

This pull requests aims to provide the following additions :

The re-execution of the MCTS in Reanalyse to obtain updated/fresh policy targets (child_visits), which as stated in the MuZero paper is used for improved sample efficiency
The use of lagging parameters (target model) in Reanalyse to stabilize the bootstrapping of root values (as done in the MuZero paper).
The ability to parallelize multiple Reanalyse processes (especially useful when running MCTS is costly and/or the memory buffer is large).
An updated 'Reanalyse sampling' method to prioritize the update of episodes with 'older' targets (root values and child_visits).
The following settings are added to games files : 1) the parameters-update frequency of the target model used in Reanalyse, 2) the number of Reanalyse processes to be used, and 3) whether to update root values based on the target model alone (representation + value), or based on the updated MCTS root_values.
For PER, priorities are now updated in Reanalyse using the new MCTS predictions (in the same way they are initialized when saving an episode).
cartpole_sample_efficient.py with some of the settings used in the MuZero paper (PER=1, num_td_steps =5, num_unroll_steps=5), a train_steps/play_steps ratio of 20, parallelized Reanalyse, the use of a target model (high frequency/frequent update), and root_values updates based on the re-execution of the MCTS.

The number of games that get played drastically changes.
Your pull request -

Compared to the default cartpole -

But the results look good !

The way that git does its comparison, it's not easy to tell if you changed other parameters from the default that would influence the results but this method seems to be more productive if you can pay the higher computational cost.

The number of games that get played drastically changes.
Your pull request -

Compared to the default cartpole -

But the results look good !

Didn't compare your game parameters to the default that could influence the results and make this an imperfect expertiment but this method seems to be more productive if you can pay the higher computational cost.

Thanks for the feedback.
One of the changes influencing the number of played games is the ratio in the game file (20 in the sample_efficient version). I think you could try the same value (20) with the original version (not this pull request) to make the comparison fairer ! Also, about the computational cost, parallelization (using multiple reanalyze processes) can compensate for the higher 'reanalyze' cost.

Yeah, with parallelization it works out great. I didn't notice anything on my CPU.
However, I think your code is breaking tic tic toe and connect 4 now.
When I run other games (not cartpole or your cartpole sample)
Possible unhandled error from worker: ray::Reanlyze.reanalyze()
Then it freaks out at return.env.legal_actions() on line 167 of tic tac toe.
AttributeError: 'NoneType object has no attribute 'legal_actions"

The number of games that get played drastically changes.
Your pull request -

Compared to the default cartpole -

But the results look good !

Didn't compare your game parameters to the default that could influence the results and make this an imperfect expertiment but this method seems to be more productive if you can pay the higher computational cost.

Thanks for the feedback.
One of the changes influencing the number of played games is the ratio in the game file (20 in the sample_efficient version). I think you could try the same value (20) with the original version (not this pull request) to make the comparison fairer ! Also, about the computational cost, parallelization (using multiple reanalyze processes) can compensate for the higher 'reanalyze' cost.

So I revisited this PR because it did increase the efficiency of Muz.
I tracked down the culprit to this line of code in replay_buffer.py
self.game.env = None
When legal_actions was called by the MCTS, it had no game object. When I took out this line of code, it all runs without error and appears to work. Cartpole worked because it just returns a list of 0 and 1. But all the other games call the legal_actions on self. What was the intended purpose of this line?

The number of games that get played drastically changes.
Your pull request -

Compared to the default cartpole -

But the results look good !

Didn't compare your game parameters to the default that could influence the results and make this an imperfect expertiment but this method seems to be more productive if you can pay the higher computational cost.

Thanks for the feedback.
One of the changes influencing the number of played games is the ratio in the game file (20 in the sample_efficient version). I think you could try the same value (20) with the original version (not this pull request) to make the comparison fairer ! Also, about the computational cost, parallelization (using multiple reanalyze processes) can compensate for the higher 'reanalyze' cost.

So I revisited this PR because it did increase the efficiency of Muz.
I tracked down the culprit to this line of code in replay_buffer.py in the init() of Reanalyze.
self.game.env = None
When legal_actions was called by the MCTS, it had no game object. When I took out this line of code, it all runs without error and appears to work. Cartpole worked because it just returns a list of 0 and 1. But all the other games call the legal_actions on self. What was the intended purpose of this line?

When using cartpole, the "legal_actions" method is part of the Game object (self.game in Reanalyze). Since there can be many reanalyze processes, I wanted to save memory by removing the environment themselves (self.game.env in Reanalyze) in these game objects since they were not required to run the MCTS.
However, it appears that for other games (such as tic-tac-toe) the "legal_actions" method is obtained from the env object (self.game.env) itself, so it is required. I guess that the memory impact should not be too big if we just remove the line, as you suggested. Thanks for sharing this feedback !


Windows installer fixes

Fix issue with install scope always being set to "per-user".
Make sure that features added via ARP/Change are installed under the directory chosen during the initial installation.

r? @brson
(rust_highfive has picked a reviewer for you, use r? to override)


OBS - Failed to connect to server when running in docker

Hello,
I've been trying for a couple of hours to set up nodetube on my local machine (just to test it out) via docker-compose. I've added an extra port for the nodetube container "1935:1935" because I care more about the live streaming aspect of this project, Unfortunately, I couldn't connect to the live-stream via OBS. Every time I try to start the stream I get the message "Failed to connect to server".
I went to settings and made sure that I have the right link:
Server:  rtmp://<IP_ADDRESS>:1935/live
Stream Key:  /test?key=<my key>

I've also looked in the OBS logs and saw this:
14:33:55.946: ------------------------------------------------
14:33:58.007: [rtmp stream: 'simple_stream'] Connecting to RTMP URL rtmp://<IP_ADDRESS>:1935/live...
14:33:58.008: [rtmp stream: 'simple_stream'] Interface: Software Loopback Interface 1 (type 24, 1073 mbps)
14:33:58.013: RTMPSockBuf_Fill, remote host closed connection
14:33:58.013: RTMP_Connect1, handshake failed.
14:33:58.013: [rtmp stream: 'simple_stream'] Connection to rtmp://<IP_ADDRESS>:1935/live failed: -2
14:33:58.022: ==== Streaming Stop ================================================

I couldn't see anything in the nodetube container logs (nothing showed up other than the "normal" logs, no errors or warnings).
Could you point out what I'm doing wrong? Thank you very much!
Details about my system:
Windows 10 Pro
Server: Docker Engine - Community
Engine:
Version:          20.10.2

I looked into this and am stumped myself as well...
I cant seem to get this functionality running it from Docker or locally with the pre-reqs installed and running...
For whatever reason it just seems to be refusing to connect to the server.
The closest I got was with docker adding - "1935:1935" to the ports section of node. it seems to be timing out with lack of response... (that should work)
I'm poking around the backend to trace out where this listener should be started and why its not responding to RTMP traffic.

EDIT: Okay I think I see what's happening here... NGINX has configuration files written to work with nodetube, but is not actually installed or setup with the image... going to tinker with it and get a write up going...
In the build directory, see docs/ngnix/ in the project directory for what appears to be a few variations of a config. I have a similar setup so its only a mater of installing ngnix and copying a functional config to it...
furthermore, there is no mention of ngnix in the DockerFile, but that's an easy fix.

Do you think you can fix it? It would be nice to have a working docker version. Just docker-compose up -d and you have a whole video/streaming platform up and running.

I plan on it, its a little more complicated then adding a ngnix-rtmp docker container because nodetube expects the stream to be a readable file being created in uploads... Ill see what i can do to remedy that design.

Okay.
Got to the bottom of it, will submit a PR when I have a docker and non-docker build...
For docker, you can get the source from my fork for now and test it. https://github.com/l1ghtsword/nodetube.git
Had to catch up on some reading...
I was able to have this working in my docker build, should just be a matter of doing a docker-compose up on your side once you DL the source above. I made the config for nginx read from local every time so feel free to tweak it. I turned recording to ./recording off by default.

^ ignore that, PR'ed the wrong repo :x

Ok, so first, I'm sorry for the late reply, I've been really busy these days.  The problem with "failed to connect to server" is gone, it says that it's connected and also streaming, BUT it's not streaming... I've waited for longer than 10 seconds and still nothing, then I looked into the logs of the nodetube container and saw some 404 errors there whenever I've tried to connect to the stream. I don't know what the problem is this time (or if it's on my machine). To be sure I've also pulled up the good ol' WireShark and did a scan of the 1935 TCP port. You can download the PCAP file here if you want to check it out too.
Steps to reproduce:

docker-compose up -d
I've made an account "test" with the password set to "test"
Started live streaming with OBS

Logs from nodetube container:
><EMAIL_ADDRESS>start /app
> node --max-old-space-size=4096 app.js
BOOTING APP...
PLUS ENABLED: false
NODE_ENV: development
RUNNING WITH THIS MANY PROCESSES: 1
SAVE AND SERVE FILES DIRECTORY: ./uploads
THE MOST CONTROVERSIAL UPLOAD RATING ALLOWED ON THIS INSTANCE IS: SFW
CACHING IS ON
CACHING IS RUNNING AGAINST: mongodb://nodetube-mongo:27017/nodetube
CONNECTING TO REDIS, HOST: redis, PORT: 6379
CACHE RECENT INTERVAL IN MINUTES: 1
CACHE POPULAR, DAILY STATS AND INDEXES INTERVAL IN MINUTES: 5
(node:18) DeprecationWarning: The option `reconnectTries` is incompatible with the unified topology, please read more by visiting http://bit.ly/2D8WfT6
API key does not start with "SG.".
CONNECTING TO REDIS, HOST: redis, PORT: 6379
ffmpeg path: /app/node_modules/@ffmpeg-installer/linux-x64/ffmpeg
youtube-dl binary path: /app/node_modules/youtube-dl/bin/youtube-dl
DATABASE: mongodb://nodetube-mongo:27017/nodetube
CONNECTING TO REDIS, HOST: redis, PORT: 6379
UPLOADS ON: true
UPLOAD SERVER: /uploads
DOMAIN NAME AND TLD: https://nodetube.live
FRONTEND SERVER:
CACHING LOGS WILL NOT COME THROUGH
CONNECTED TO DATABASE AT: mongodb://nodetube-mongo:27017/nodetube
RUNNING AS FRONTEND
(node:25) DeprecationWarning: The option `reconnectTries` is incompatible with the unified topology, please read more by visiting http://bit.ly/2D8WfT6
NodeTube can be accessed on your local network at <IP_ADDRESS>:3000
Websockets server started over http on port 8443
✓ App is running at http://localhost:3000 in development mode
Press CTRL-C to stop
Access NodeTube on the public web via https://54f375c220a3.ngrok.io. This link will be changed if you restart the app, to
use Ngrok with a permanent subdomain please purchase a token and update the settings in .env.private (see runNgrok function in app.js)
ip: ::ffff:<IP_ADDRESS> req url: http://localhost:49161/ header: undefined
GET / 200 1166.902 ms - -
ip: ::ffff:<IP_ADDRESS> req url: http://localhost:49161/login header: http://localhost:49161/
GET /login 200 529.979 ms - -
ip: ::ffff:<IP_ADDRESS> req url: http://localhost:49161/signup header: http://localhost:49161/login
GET /signup 200 382.480 ms - -
ip: ::ffff:<IP_ADDRESS> req url: http://localhost:49161/signup header: http://localhost:49161/signup
test <--- inputted channelUrl forundefined
(node:25) DeprecationWarning: req.validationErrors() may be removed in a future version. Use req.getValidationResult() instead.
(node:25) DeprecationWarning: collection.count is deprecated, and will be removed in a future version.
The traffic from obs to nginx over 1935 is working, and the hls data is being forwarded to nodetube.
Do you get anything when you access the nodetube live page?
http://localhost:49161/live/test ???
Im not able to reproduce that on my end. Though, i am using my local ip and added a record to my host file for my dev domain in the env files.

Actually i can already see you didn't, or at least not enabling the live stream routes. RUNNING AS FRONTEND tells me its running the main routes but RUNNING AS LIVESTREAM or something of the sort would tell me you set that to true in your env.settings.
Some assembly required*
change env.settings
LIVESTREAM_APP=false to true (LIVESTREAM_APP=true)
And see if that alone is enough

Ohh, ok I'll try that. Where can I find env.settings?

so this is a little quirky, the dockerization of this app is not ideal (hence why i'm still working on it)...
The env.settings file is in the project source root, however... You will need to blow away the "nodetube", "bougyman/voidlinux" and "none" images then rebuild to edit them... not ideal - im putting together a more dev/ prod docker friendly environment.

Nvm, I've found it (typical me, first ask then look in the project...), anyway I've done what you've just said, but still no luck (I've changed the .env.settings.sample file tho then rebuild everything, I've also deleted the dangling images before building). I've got this in the logs now (302):
><EMAIL_ADDRESS>start /app
> node --max-old-space-size=4096 app.js
BOOTING APP...
PLUS ENABLED: false
NODE_ENV: development
RUNNING WITH THIS MANY PROCESSES: 1
SAVE AND SERVE FILES DIRECTORY: ./uploads
THE MOST CONTROVERSIAL UPLOAD RATING ALLOWED ON THIS INSTANCE IS: SFW
CACHING IS ON
CACHING IS RUNNING AGAINST: mongodb://nodetube-mongo:27017/nodetube
CONNECTING TO REDIS, HOST: redis, PORT: 6379
CACHE RECENT INTERVAL IN MINUTES: 1
CACHE POPULAR, DAILY STATS AND INDEXES INTERVAL IN MINUTES: 5
(node:17) DeprecationWarning: The option `reconnectTries` is incompatible with the unified topology, please read more by visiting http://bit.ly/2D8WfT6
API key does not start with "SG.".
CONNECTING TO REDIS, HOST: redis, PORT: 6379
ffmpeg path: /app/node_modules/@ffmpeg-installer/linux-x64/ffmpeg
youtube-dl binary path: /app/node_modules/youtube-dl/bin/youtube-dl
DATABASE: mongodb://nodetube-mongo:27017/nodetube
CONNECTING TO REDIS, HOST: redis, PORT: 6379
UPLOADS ON: true
UPLOAD SERVER: /uploads
DOMAIN NAME AND TLD: https://nodetube.live
FRONTEND SERVER:
CACHING LOGS WILL NOT COME THROUGH
CONNECTED TO DATABASE AT: mongodb://nodetube-mongo:27017/nodetube
Running as livestream app
(node:24) DeprecationWarning: The option `reconnectTries` is incompatible with the unified topology, please read more by visiting http://bit.ly/2D8WfT6
NodeTube can be accessed on your local network at <IP_ADDRESS>:3000
Websockets server started over http on port 8443
✓ App is running at http://localhost:3000 in development mode
Press CTRL-C to stop
Access NodeTube on the public web via https://5a0a0278fb47.ngrok.io. This link will be changed if you restart the app, to
use Ngrok with a permanent subdomain please purchase a token and update the settings in .env.private (see runNgrok function in app.js)
ip: ::ffff:<IP_ADDRESS> req url: http://localhost:49161/ header: undefined
GET / 200 1249.868 ms - -
ip: ::ffff:<IP_ADDRESS> req url: http://localhost:49161/account header: http://localhost:49161/
GET /account 200 1712.559 ms - -
ip: ::ffff:<IP_ADDRESS> req url: http://localhost:49161/account/livestreaming header: http://localhost:49161/account
::ffff:<IP_ADDRESS> 47428 ::ffff:<IP_ADDRESS> 3000
::ffff:<IP_ADDRESS>
3000
/account/livestreaming
rtmp://<IP_ADDRESS>:1935/live/test?key=Cp0Pk88bbvAnVSqSeKQJExr3Z http://localhost:49161/live/test
GET /account/livestreaming 200 999.673 ms - -
ip: ::ffff:<IP_ADDRESS> req url: http://nodetube/livestream/on-live-auth header: undefined
{ app: 'live',
flashver: 'FMLE/3.0 (compatible; FMSc/1.0)',
swfurl: 'rtmp://<IP_ADDRESS>:1935/live',
tcurl: 'rtmp://<IP_ADDRESS>:1935/live',
pageurl: '',
addr: '<IP_ADDRESS>',
clientid: '1',
call: 'publish',
name: '/test',
type: 'live',
key: 'Cp0Pk88bbvAnVSqSeKQJExr3Z' }
Cp0Pk88bbvAnVSqSeKQJExr3Z
authentication passed
found user: test
Access the livestream at /live/test
POST /livestream/on-live-auth 200 57.633 ms - 7
ip: ::ffff:<IP_ADDRESS> req url: http://localhost:49161/live/test header: http://localhost:49161/account/livestreaming
test
GET /live/test 200 3571.115 ms - -
ip: ::ffff:<IP_ADDRESS> req url: http://localhost:49161/test.m3u8 header: http://localhost:49161/live/test
GET /test.m3u8 302 238.003 ms - 61
ip: ::ffff:<IP_ADDRESS> req url: http://localhost:49161/test.m3u8 header: http://localhost:49161/live/test
GET /test.m3u8 302 73.945 ms - 61

Yeah I also came to that conclusion, just spun up the containers myself...
nope I'm wrong, it is adding the default sample files to the docker image, unedited...
Ill need to fix that as well. I was editing things manually in the container which is bad and i should know better. those changes wont be retained either...

Yeah, I've looked into the Dockerfile of the project, then I saw it immediately. I know how it is XD I work with docker almost every day.

maaaaaaaaan this is annoying, i had this working the other day, now it wont display anything...
Okay, leave it with me i need to figure out why its not publishing anything...

Okay, cardinal sin number 2, I made a last minute change without testing it. apologies my guy...
the hls folder is being monitored by nodetube directly, so it is a shared volume as well. I through it was forwarding the traffic however that does not appear to be the case...
Ill do more testing but this build /should/ work https://github.com/l1ghtsword/nodetube.git

Works like a charm, I still had to make some adjustments tho. The files .env.settings and .env.private couldn't be found so docker wouldn't start the container. I changed that to .env.settings.sample and .env.private.sample respectively in the volumes list of the node container then everything worked. I'll test it some more today and I'll let you know if I find something else. Thank you!
PS: I also have the modified .env.settings.sample with the variable LIVESTREAM_APP set to true

You would need to run it once to have it copy them to the local directory for you. a word of caution, i would advise against changing the env.sample files and instead simply copy and paste them into the root directory and remove the ".sample" at the end.
That way you have a clean reffernce in the event something breaks due to a setting change.
Also, env.sample files are subject to change as they are template files observed by git. .gitignore only ignores non sample files with the expectation api keys or private settings are in there...

New problem: I can't do anything other than seeing the "my account" page (click on the account icon top left then go to my account). If I try anything else I get redirected to something like this: https://https//nodetube.live/ and then the browser can't find an address for that (duh). This worked before the fix, but now it stopped working. I didn't know if I should start a separate issue or write it here since the PR hasn't been applied yet

That's an issue with lack of configuration not core functionality. for general questions not related to a specific software issue, you can ask them in the discord
Lets avoid adding out of scope issues to this issue #

@benny1611
Yeah you should create a separate issue for that.
In regards to .env.settings.sample and .env.private.sample, it looks like docker runs npm install, which also runs a postinstall script copying contents of the those files to .env.settings and .env.private respectively, that's why it worked (even though it's not advisable to do so). Hope you didn't do any public commits after changing those files, otherwise all your private keys will be for everyone to see.


kamel inspect is not providing dependencies specified in modeline

call kamel inspect MyRouteBuilderWithAdditionalDependencies.java with the specified file having this content:
// camel-k: dependency=mvn:com.github.camel-tooling/camel-lsp-server:1.1.0

import org.apache.camel.builder.RouteBuilder;

public class MyRouteBuilderWithAdditionalDependencies extends RouteBuilder{
	public void configure() {
    }
}

it returns:
mvn:org.apache.camel.k/camel-k-loader-java
the declared depdency in modeline is missing

Thanks for reporting this.
Solved by: https://github.com/apache/camel-k/pull/1924/files

Thanks for reporting this.
Solved by: https://github.com/apache/camel-k/pull/1924/files


PurchaseResponse.php: "Undefined index: messages"

It appears the error message structure has changed and is causing an exception at PurchaseResponse.php#L26 whenever BitPay returns an error:

Undefined index: messages

This is what was in $this->['data'] according to print_r:
Array ( [type] => limitExceeded [message] => Invoice not created due to account limits, please check your approval levels )

Noted. Any chance of a PR?

Yes. I'll prepare it today.

Merged, with thanks.