Laravel scout 9 support

I'm facing this issue using the package for scout v9
 matchish/laravel-scout-elasticsearch dev-php8 requires laravel/scout ^8.0 ->
 found laravel/scout[v8.0.0, ..., 8.x-dev] but it conflicts with your root composer.json require (^9.0)

https://github.com/matchish/laravel-scout-elasticsearch/issues/160
There is small braking changes in scout 8.6.1 the fix might simple but someone have to implement it and be sure that tests not broken. Fill free to send the PR

There is dev branch with scout 9 support https://github.com/matchish/laravel-scout-elasticsearch/tree/php8-test
Feedback are welcome

Are you going to support only PHP 8 with 5.x? Scout 9 supports minimal php 7.3, and this package bumping to 8.0 would not allow projects that use Elastic PHP APM for monitoring to update to it (there is no php 8.0 support for monitoring - I know it has nothing to do with the project, but monitoring php 8.0 is not realistic in the enterprise world yet).

What is your case to use scout 9 over 8?

Sorry, I don't understand your question, but as long as Scout 9.x is >=7.3, including 8.0, what is your reason to enforce people on php 8.0 that has poor monitoring support at this moment? If Scout 9.x will support, why don't you inline the package with it?

If you don't need php8 you can use 4x version. I don't see pros in scout 9. Main reason of 5x is support php8
Yes we can add support php7.3 to 5x but I don't have time for it now. So pr welcome. But be sure that tests are passes. And php7.3 added to travis config matrix for ci
Also scout 9 support can be added to 4x version as described in previous comment

Support in 4.0.7


Has no client.ConnectAsync

Please check, my AMWD.Protocols.Modbus v0.2.0 has no client.ConnectAsync function, so when i call client.ReadHoldingRegistersAsync then error orcur: System.NullReferenceException: 'Object reference not set to an instance of an object.'

Could you please try this pre-release?
https://nuget.am-wd.de/packages/amwd.protocols.modbus.tcp/0.2.1-7

Oh, i don't know that link. I tried and OK. Thanks for your support. you are really professional. I really like your Modbus library

The mentioned NuGet Feed will only have some pre-releases during development.
These can be deleted any time after a release is published ;-)
Issue fixed in Release v0.3.0


Second device login?

Hi -- first, thanks for doing devise-passwordless! It's a cool thing, and seems to work.
I have a use case that doesn't appear to work:

I'd like to bring up the page on device 1, on which I don't have email.
I'd like to put in my email, and get a link emailed to me.
I'd like to read the email, and click the link, on device 2 which does have email.
I'd like the session on device 1 to the be validated. It's fine if it's valid for device 2 as well.

Any clues appreciated -- Thanks!

That sounds like an interesting UX. Obviously the gem isn't built to accommodate that at the moment.
Off the cuff I think you could accomplish something like that by having the Rails server store some kind of nonce session value (think SecureRandom.base58(50)) on Device 1's cookie when Device 1 initiates the login request, and attach that nonce value to the link that gets generated and emailed. Device 1 would then busy poll an endpoint that will convert its cookie into a logged-in cookie (Devise's sign_in helper) when Device 2 visits the magic link.
Maybe there's an easier way I'm not thinking of but that's one way. You'd want to be careful your busy wait endpoint isn't vulnerable to timing attacks or things of that nature

Thanks @abevoelker! Good ideas. I think I may try implementing my own ideas; I don't need all the "weight" of Devise for this, as it's a (very) simple application. And, yeah, I think I'll just have it say, "Refresh this page once you've clicked the email link" to avoid a busy-wait.

For the record, this was inspired by a web exercise app, that let me log in to (for example) a smart-TV browser that I didn't have email set up on (nor would I want to). I also don't have email set up on my tablet.

@csterritt Got it, yeah that use case makes perfect sense then. If you don't want the full weight of Devise but want something a little more useful than session variables, you could write a Warden strategy (Devise is built on top of Warden if you didn't know). Devise requires Rails but Warden works with any Rack app. I found this RailsConf video to be a useful primer on Warden when writing this gem if you go that route.
Closing this for now since it doesn't pertain to the gem. Good luck to you on your project

Thanks Abe! I'll give that a look.
On Tue, Aug 10, 2021 at 10:33 AM Abe Voelker @.***>
wrote:

@csterritt https://github.com/csterritt Got it, yeah that use case
makes perfect sense then. If you don't want the full weight of Devise but
want something a little more useful than session variables, you could write
a Warden strategy
https://github.com/wardencommunity/warden/wiki/Strategies (Devise is
built on top of Warden if you didn't know). Devise requires Rails but
Warden works with any Rack app. I found this RailsConf video
https://www.youtube.com/watch?v=QBJ3G40fxHg to be a useful primer on
Warden when writing this gem if you go that route.
Closing this for now since it doesn't pertain to the gem. Good luck to you
on your project
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/abevoelker/devise-passwordless/issues/5#issuecomment-896082302,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAJIHMGDXJ6CCDS376352DT4E2E7ANCNFSM5BWZ5YQA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.


Updating PagesProject source repository in-place is not possible, but no replace is performed

What happened?
I have a Pulumi-managed PagesProject with source.type == "github". I changed source.config.owner and this is recognized as an update by pulumi preview / pulumi up, but after running those, the source repository didn't actually change on the Cloudflare side.
Looking at the same Pages project in the Cloudflare dashboard, there is no option to change the source repository either, which strongly suggests to me that this is impossible to change in-place. The provider should thus do a replace instead of an update.
Steps to reproduce

Deploy a PagesProject with source.type == "github".
Change source.config.owner or source.config.repoName and run pulumi up.

Expected Behavior

The preview shows a replace (as an in-place update is possible).
The (re-created) Pages project points to the new source repository.

Actual Behavior

The preview shows an in-place update (although it is not possible).
Nothing has changed after pulumi up finishes.
When running pulumi up again, it attempts to do the same impossible in-place update another time.

Output of pulumi about
CLI
Version      3.51.1
Go Version   go1.19.4
Go Compiler  gc

Plugins
NAME        VERSION
auth0       2.14.0
cloudflare  4.15.0
nodejs      unknown

Host
OS       darwin
Version  13.0.1
Arch     arm64

This project is written in nodejs: executable='/opt/homebrew/bin/node' version='v18.12.1'

Current Stack: dev

TYPE                                        URN
pulumi:pulumi:Stack                         urn:pulumi:dev::example::pulumi:pulumi:Stack::example-dev
pulumi:providers:cloudflare                 urn:pulumi:dev::example::pulumi:providers:cloudflare::default_4_15_0
pulumi:providers:auth0                      urn:pulumi:dev::example::pulumi:providers:auth0::default_2_14_0
auth0:index/client:Client                   urn:pulumi:dev::example::auth0:index/client:Client::auth0-client
auth0:index/resourceServer:ResourceServer   urn:pulumi:dev::example::auth0:index/resourceServer:ResourceServer::auth0-api
auth0:index/tenant:Tenant                   urn:pulumi:dev::example::auth0:index/tenant:Tenant::auth0-tenant
auth0:index/clientGrant:ClientGrant         urn:pulumi:dev::example::auth0:index/clientGrant:ClientGrant::auth0-client-management-api-grant
auth0:index/connection:Connection           urn:pulumi:dev::example::auth0:index/connection:Connection::auth0-passwordless-email
auth0:index/role:Role                       urn:pulumi:dev::example::auth0:index/role:Role::auth0-submitter-role
auth0:index/role:Role                       urn:pulumi:dev::example::auth0:index/role:Role::auth0-reviewer-role
cloudflare:index/pagesProject:PagesProject  urn:pulumi:dev::example::cloudflare:index/pagesProject:PagesProject::cloudflare-pages-components
cloudflare:index/pagesProject:PagesProject  urn:pulumi:dev::example::cloudflare:index/pagesProject:PagesProject::cloudflare-pages-spec
auth0:index/client:Client                   urn:pulumi:dev::example::auth0:index/client:Client::auth0-automation-client

Found no pending operations associated with dev

Backend
Name           Deniss-MacBook-Pro.fritz.box
URL            s3://pulumi?endpoint=fe43443d552eccfb1a94b1b6dfd66832.r2.cloudflarestorage.com&s3ForcePathStyle=true
User<EMAIL_ADDRESS>Organizations

Dependencies:
NAME                              VERSION
eslint                            8.30.0
@typescript-eslint/eslint-plugin  5.47.0
@typescript-eslint/parser         5.47.0
eslint-plugin-import              2.26.0
husky                             8.0.3
@pulumi/auth0                     2.14.0
@pulumi/cloudflare                4.15.0
@pulumi/pulumi                    3.48.0
@types/node                       14.18.34
prettier                          2.8.1

Pulumi locates its logs in /var/folders/hf/x89std3j1tsc_nbc6mhf0hm80000gp/T/ by default

Additional context
This might be an upstream issue with the Terraform Cloudflare provider, which I didn't test directly. I found no comparable bug report there.
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Thanks for reporting this! Terraform allows encoding that a resource property change would require replacing the resource, however this has not been done for the PagesProject as far as I can tell from the links below:
https://github.com/cloudflare/terraform-provider-cloudflare/blob/20222d155d457736d71eaa41a63bb87853cfaf29/internal/provider/schema_cloudflare_pages_project.go#L53-#L53
https://github.com/pulumi/terraform-provider-cloudflare/blob/b5da5489458fe4c2158224c19a5653f268b0cb33/internal/provider/schema_cloudflare_pages_project.go#L52-#L52
In this case the best way to get this fixed is to get github.com/cloudflare/terraform-provider-cloudflare to accept a change marking the properties as requiring replacement, and then bring pulumi/pulumi-cloudflare up-to-date with the latest version of upstream.

This field is now properly flagged to trigger a replace


IOU with segm masks and MAP for instance segment.

What does this PR do?
Fixes #821
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[X] Did you read the contributor guideline, Pull Request section?
[X] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

@gianscarpe mind resolve the conflict... :rabbit:

Any help with this error? It happens with ddp and I cannot figure out why @Borda
multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
File "/usr/lib/python3.8/multiprocessing/pool.py", line 125, in worker
result = (True, func(*args, **kwds))
File "/usr/lib/python3.8/multiprocessing/pool.py", line 51, in starmapstar
return list(itertools.starmap(args[0], args[1]))
File "/home/gianlucascarpellini/dev/unsupervised-sensor-network/third_parties/metrics/tests/helpers/testers.py", line 219, in _class_test
result = metric.compute()
File "/home/gianlucascarpellini/dev/unsupervised-sensor-network/third_parties/metrics/torchmetrics/metric.py", line 374, in wrapped_func
with self.sync_context(
File "/usr/lib/python3.8/contextlib.py", line 113, in enter
return next(self.gen)
File "/home/gianlucascarpellini/dev/unsupervised-sensor-network/third_parties/metrics/torchmetrics/metric.py", line 345, in sync_context
self.sync(
File "/home/gianlucascarpellini/dev/unsupervised-sensor-network/third_parties/metrics/torchmetrics/metric.py", line 297, in sync
self._sync_dist(dist_sync_fn, process_group=process_group)
File "/home/gianlucascarpellini/dev/unsupervised-sensor-network/third_parties/metrics/torchmetrics/metric.py", line 245, in _sync_dist                                                          if isinstance(output_dict[attr][0], Tensor):
IndexError: list index out of range
"""

Not an expert on the detection topic so just wanted to ask how many different common formats there exist?

@Borda  I'll fix my PR and submit next week :)

Hello guys, Let me add my 5 cents:
I just started to use this code, and it seems too complicated, maybe it would be better to create separate MAP for boxes and MAP for segments with common parent?
For example, I want to use segments, but I have to fill in boxes
If someone want to use both, he/she can create 2 metrics for boxes and for segments.
I maybe can contribute, If this idea seems fine

The idea seems interesting. Let's see what @justusschock  think about it! For the moment I have kept things as in the original pycoco implementation, where only one kind of iou_type

@gianscarpe mind checking the conflict colision? :rabbit:

@gianscarpe seems you miss one import... :)

@gianscarpe tried to resolve conflicts; mind checking if it is correct? :rabbit:

The idea seems interesting. Let's see what @justusschock think about it! For the moment I have kept things as in the original pycoco implementation, where only one kind of iou_type

@gianscarpe Sorry for the late reply - has been a busy time. I also think this is a good suggestion, but only if it is actually possible to make the code easier through this. If it is just for code sharing, we could also have helper functions outside the classes.

@gianscarpe, how is it going here? :rabbit:

@gianscarpe, how is it going here? rabbit

Hi @Borda! I computed the values with pycocotools to introduce the test regarding IOU segmentation type. Let me know if it's ok! :)

Update: test are failing because pycocotools is not in requirements. I suggest to add pycocotools only to test requirements, as are not necessary for the library but only for converting coco RLE mask to torch masks for evaluation

Is hashing the metric class mandatory? @Borda @SkafteNicki

@gianscarpe @SkafteNicki how is it going here? :rabbit:
we are considering a new feature release next week, do you think we can finish this addition by then so it will be included? :pray:

Hi @Borda, thanks for the interest! I fixed all tests for both segmentation and bbox IOU. For segmentation, I introduce pycocotools as there are currently no easy RLE encoding/decoding for pytorch. I believe it could be nice to introduce this in a future PR, maybe implementing in cuda. It follows that MAP with iou_type=="segm" does not currently work in GPU, and moves everything to CPU for most of the calculations. I produced the documentation and docstrings for the main functions, let me know if anything else is required.
I would take on the cuda implementation for RLE, preferably for another PR if that's fine :)

Update: I noticied that the docs check is failing because of pycocotools, but couldn't find how torchmetrics manages requirements in building the documentation. I would be happy to fix it, le met know how to handle it @Borda :)

Any updates from your side @Borda ? Thanks :)

Hi @Borda @SkafteNicki could you check the failing test?
        areas = compute_area(gt, iou_type=self.iou_type).to(self.device)

>       ignore_area = torch.logical_or(areas < area_range[0], areas > area_range[1])
E       RuntimeError: value cannot be converted to type int without overflow

torchmetrics\detection\mean_ap.py:583: RuntimeError

I'm trying to fix this but it happens only on windows. This should be a boolean value, so why overflow error? Any ideas?

Hi @gianscarpe,
I tried changing:
area = torch.tensor(mask_utils.area(input).astype("int"))

to
area = torch.tensor(mask_utils.area(input).astype("float"))

and hope that fixed the problem :]

if all is fine now, I ll move the data to S3

@Borda go ahead and do that, test are passing on windows now :]

@Borda go ahead and do that, test are passing on windows now :]

shall be all set now :)

Update: Seems like it's not installing pycocotools from detection requirements


[WIP][SPARK-31114][SQL] Constraints inferred from equality constraints with cast

What changes were proposed in this pull request?
This PR add support inferred morre constraints from equality constraints with cast, For e.g.,
spark.sql("CREATE TABLE SPARK_31114_1(a BIGINT)")
spark.sql("CREATE TABLE SPARK_31114_2(b DECIMAL(18, 0))")
spark.sql("SELECT t1.* FROM SPARK_31114_1 t1 JOIN SPARK_31114_2 t2 ON t1.a = t2.b AND t1.a = 1L").explain

Before this PR:
== Physical Plan ==
*(2) Project [a#0L]
+- *(2) BroadcastHashJoin [cast(a#0L as decimal(20,0))], [cast(b#1 as decimal(20,0))], Inner, BuildRight
   :- *(2) Project [a#0L]
   :  +- *(2) Filter (isnotnull(a#0L) AND (a#0L = 1))
   :     +- *(2) ColumnarToRow
   :        +- FileScan parquet default.spark_31114_1[a#0L]
   +- BroadcastExchange HashedRelationBroadcastMode(List(cast(input[0, decimal(18,0), true] as decimal(20,0)))), [id=#50]
      +- *(1) Project [b#1]
         +- *(1) Filter isnotnull(b#1)
            +- *(1) ColumnarToRow
               +- FileScan parquet default.spark_31114_2[b#1]

After this PR:
*(2) Project [a#218L]
+- *(2) BroadcastHashJoin [cast(a#218L as decimal(20,0))], [cast(b#219 as decimal(20,0))], Inner, BuildRight
   :- *(2) Project [a#218L]
   :  +- *(2) Filter (isnotnull(a#218L) AND (a#218L = 1))
   :     +- *(2) ColumnarToRow
   :        +- FileScan parquet default.spark_31114_1[a#218L]
   +- BroadcastExchange HashedRelationBroadcastMode(List(cast(input[0, decimal(18,0), true] as decimal(20,0)))), [id=#119]
      +- *(1) Project [b#219]
         +- *(1) Filter ((cast(b#219 as bigint) = 1) AND isnotnull(b#219))
            +- *(1) ColumnarToRow
               +- FileScan parquet default.spark_31114_2[b#219]

Why are the changes needed?
Improve query performance.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Unit test.

Test build #119661 has started for PR 27874 at commit 1e92854.

Merged build finished. Test PASSed.

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

Test build #119661 has finished for PR 27874 at commit 1e92854.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #119669 has started for PR 27874 at commit 1e92854.

Merged build finished. Test PASSed.

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

Test build #119669 has finished for PR 27874 at commit 1e92854.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

I closed it because this change cannot handle this case:
spark.sql("create table T1(a string)")
spark.sql("create table T2(b string)")
spark.sql("create table T3(c bigint)")
spark.sql("create table T4(d bigint)")

spark.sql(
  """
    |SELECT t1.a, t2.b, t4.d
    |FROM T1 t1 JOIN T2 t2
    |       ON (t1.a = t2.b)
    |     JOIN T3 t3
    |       ON (t1.a = t3.c)
    |     JOIN T4 t4
    |       ON (t3.c = t4.d)
    |""".stripMargin).explain()


enable mounting writable local paths

Allow Bacalhau jobs to have write access to allow-listed paths. A use case where this is helpful is when jobs need to checkpoint their progress so that future invocations/retries can resume instead of starting over, such as when processing local log files or local sensor data.

Any chance we could base the changes landing here onto https://github.com/bacalhau-project/bacalhau/pull/2492? #2492 is rewriting the entire storage spec to be specific to the implementation by defined independent structures  for each type of storage rather than continuing to append to the existing storage spec

@frrist I see your point. Though this PR introduces are other changes and exposes a new feature, and not only adding a field to the spec. I prefer that https://github.com/bacalhau-project/bacalhau/pull/2492 only has changes related to re-factoring the spec and not bundle this change with it. I can hold off from merging this PR and rebasing it after https://github.com/bacalhau-project/bacalhau/pull/2492 is merged if you think that will happen soon. Keep in mind there is a demo that is blocked on this PR

In that case merge it and I'll sort the conflicts in my branch when it's rebased onto master.

Thanks. I've just added you as reviewer whenever you have a chance :)


doc: content -> multi languages -> Chinese

Description
Manual description about content details for Chinese resumes, like translations, idioms, etc.
Todo list
NA

Done:


Close wiki unify content on repository, docs repository or on website

@ccordoba12 removed all the wiki pages. Can you disable the wiki?
The content can be found here if needed
spyder.wiki.zip
We should probably move the ROADMAP (beta, or normal) to either the spyder repo or the docs repo or the website.
@oscargus you can use the file I linked with the contents to update the contribution.md file. Thanks!

OK! I realized that the "Troubleshooting..." entry in the help menu is pointing to a Wiki page, now not reachable.

@oscargus I can restore that page if needed. But we should really unify everything under the website/or the docs.

removed all the wiki pages. Can you disable the wiki?

Why did you do that??!!!! We had important resources there, referenced in a lot of StackOverflow questions!! Please revert the wiki immediately.
I think I only agreed to remove the development pages, which I didn't know they existed (because I didn't create them), no to remove the entire wiki!

We should probably move the ROADMAP (beta, or normal) to either the spyder repo or the docs repo or the website.

And also the beta changelog, I mean, come on!! This is completely non-sensical!
You should have removed the wiki once all pages will be placed in other places, at the very least!

@goanpeca, please close this issue once you restore the wiki. This is not the way to go, because the wiki is nice to have for certain things.
The only bad thing is to have outdated pages that people is trying to use to get started with Spyder development.

@goanpeca, I'm very sorry for my outburst here. It's just that for a moment I thought we lost all the time and effort invested in several critical pages placed in our wiki.
I'm not against other developers taking the initiative and doing things on their own. But a decision as important as removing the wiki completely without moving its referenced pages to other places must be consulted first with the rest of the core team, or at least with me privately.

@ccordoba12 apology accepted.
From the conversation, we had over at the other issue I had understood that the wiki should be removed (not only the sections regarding contributing to the project), so my bad there. The wiki is a repository and I had the clone in my computer so restoring it was not really an issue.
However, this is not the first time such outbursts arise, as @CAM-Gerlach can unfortunately attest, and these kind of replies to contributors to the project are not acceptable, even more so taking into account the Code of Conduct you uploaded a few weeks ago.

However, this is not the first time such outbursts arise, as @CAM-Gerlach can unfortunately attest, and these kind of replies to contributors to the project are not acceptable, even more so taking into account the Code of Conduct you uploaded a few weeks ago.

Ok, I totally agree with you on this point. I'll open an issue in our private repo to discuss this situation with the core team.


feat: upgrade to angular v16

@jan-moxter a new PR to move to angular v16, rxjs v7 and typescript v5
(without this lib can't be used with v16 due to typescript version change)

I would prefer to name the release, my projects will be  on angular 15 for some time still and that would leave more space if something breaks for 15 or 16
v7.0.0 - support for Angular 15


Json package

Close #396.

Codecov Report

Merging #851 (e844882) into main (f40488c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #851   +/-   ##
=======================================
  Coverage   81.64%   81.64%
=======================================
  Files         362      362
  Lines       33888    33888
=======================================
  Hits        27669    27669
  Misses       6219     6219

Continue to review full report at Codecov.

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


Backport "Fix #19402: emit proper error in absence of using in given definitions" to LTS

Backports #19714 to the LTS branch.
PR submitted by the release tooling.
[skip ci]

No regressions detected in the community build up to lts-19080.
Reference


Coerce non-Errors thrown in JavaScript into Error objects

Currently, when throw is used with an object which isn't an Error, a warning is issue from Bugsnag.js, indicating that the potential report is being discarded. We should keep the warning but attempt to coerce the object into a RuntimeError, where the message or perhaps custom metadata includes the thrown object. The stacktrace may not be particularly useful in this case, but we have something appended by the React Native global handler, which is something to investigate. The report context should include that the coercion into an error was done automatically by the Bugsnag library, to reduce confusion.
Further discussion in #129

I worked around this by creating a helper function that allows me to send almost anything into my error logger:
export function logError(...args) {
  if (__DEV__) console.error(...args);

  if (args.length === 0) return;

  const errors = [];
  const nonErrors = [];
  args.forEach((a) => {
    if (a instanceof Error) errors.push(a);
    else nonErrors.push(a);
  });

  const error = errors.length > 0 ? errors[0] : new Error(String(nonErrors[0]));

  bugsnag.notify(error, (report) => {
    report.metadata = {
      args: nonErrors.map(nonError => String(nonError)),
    };
  });
}

@mifi Thank you for snipper.
I am improve your solution by add typings required for TypeScript projects
export function logError(...args: any) {
  if (__DEV__) console.error(...args)

  if (args.length === 0) return

  const errors: Error[] = []
  const nonErrors: any[] = []
  args.forEach((a: any) => {
    if (a instanceof Error) errors.push(a)
    else nonErrors.push(a)
  })

  const error = errors.length > 0 ? errors[0] : new Error(String(nonErrors[0]))
  bugsnag.notify(error, report => {
    report.metadata = {
      args: nonErrors.map(nonError => String(nonError)).join("\n"),
    }
  })
}

The latest release of the Bugsnag React native library includes support for this.
As of v7.3 the library itself has now become part of the bugsnag-js repository.
An upgrade guide can be found here with full documentation available here.


Anonymous Apex Error: compileProblem: Script too large

Summary
I am just trying to use MAS and anonymous apex to delete old log data.
I have a soql query that returns 10k records atm but the MAS doesn't run with 1 record.
I am unsure where the root cause lies.
Version of Mass Action Scheduler
2.400
Steps To Reproduce

SOQL to return old log records
select id from dca_mass_action__Mass_Action_Log__c where recordtype.developername='Parent_Log' and createdDate < Last_n_months:3
anonymous apex to delete the records
`void execute( List<Map<String, Object>> sourceRecordsBatch ) {
list deletable = new List();
for(map<string, Object> thisMap: sourceRecordsBatch){
deletable.add(JSON.deserialize(JSON.serialize(thisMap), Sobject.class));

}
delete deletable;
}3. go to Apex Jobs in setup and see the error First error: Anonymous Apex Error:
compileProblem: Script too large: void mas_initApexBlock() { List untypedRecords = (List) JSON.deserializeUntyped( ...
exceptionMessage:
exceptionStackTrace:
Invoke Action Error: System.HttpRespo...`
Expected Behavior
records are successfully deleted.
Actual Result
error in apex job
`First error: Anonymous Apex Error:
compileProblem: Script too large: void mas_initApexBlock() { List untypedRecords = (List) JSON.deserializeUntyped( ...
exceptionMessage:
exceptionStackTrace:
Invoke Action Error: System.HttpRespo...`
please note I don't seem to be able to get the rest of this error message
Source Type

[ ] Reports
[ ] List Views
[x ] SOQL Queries

Action Type

[ ] Process Builder & Flows
[ ] Workflow Rules
[ ] Email Alerts
[ ] Quick Actions
[x ] Apex

Does your configuration use Named Credentials?
Behaves the same with and without named credentials

Not sure what, if anything has changed. the following setup now works
select id from dca_mass_action__Mass_Action_Log__c where recordtype.developername='Parent_Log' and createdDate < Last_n_days:93
void execute( List<Map<String, Object>> sourceRecordsBatch ) {
list<ID> deletable = new List<ID>();
for(map<string, Object> thisMap: sourceRecordsBatch){
for(Object obj:thismap.values()){
system.debug(JSON.serializePretty(thismap));
deletable.add((ID)thismap.get('Id'));
}
}
database.delete(deletable);
}


Can't login

When trying to log in, the node reports "TypeError: Cannot read property '1' of null". I also tried importing the simple example flow but it does the same thing. My password is 21 characters, maybe that's the problem?

I have the same Problem. This issue is since VW changed the webinterface. I don't know if it is fixable.

I have the same problem = TypeError: Cannot read property '1' of null.
Anyone that found a solution?

Same problem with me.


Update Procedure.type

Right now Procedure.type is not being set in any subclasses.  We should either remove it or populate it everywhere.

the place we still have to figure out is the tissue prep - but as mentioned before, I think there needs to be a separate class for that.


Spinner not working on API 23

Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[] iOS
[x] Android
[] UWP
[] NET Standard 2.0

Version of OS(s) listed above with issue (if android - please include version of Xamarin.Android.Support.* you are using)
API 23
Resolution 1280 x 720
DPI 420
Version of ACR UserDialogs Library
7.0.4
Expected Behaviour
Spinner Showing (It works fine on other screen sizes and API)

Actual Behavior
I can only see the background of the spinner, but the spinner itself not showing in some cases I can only see tiny part of it

Please provide a small reproducible sample.  I'd encourage you to try the sample app on your device as well to see if it also has this issue.

Hi @aritchie
Thanks for your reply, I followed the sample code and it seems to be working perfectly now


Teaspoon fails intermittently on CircleCI

Teaspoon is failing on CircleCI intermittently, more often than not. I added the following to my circle.yml file, per the documentation:
test:
  post:
    - bundle exec teaspoon

When it fails, I get the following stack trace:
bundle exec teaspoon
Starting the Teaspoon server...
Teaspoon running default suite at http://<IP_ADDRESS>:52670/teaspoon/default
Error: ActionController::RoutingError: No route matches [GET] "/teaspoon/default"
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/actionpack-4.2.0/lib/action_dispatch/middleware/debug_exceptions.rb:21:in `call'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/actionpack-4.2.0/lib/action_dispatch/middleware/show_exceptions.rb:30:in `call'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/railties-4.2.0/lib/rails/rack/logger.rb:38:in `call_app'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/railties-4.2.0/lib/rails/rack/logger.rb:20:in `block in call'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/activesupport-4.2.0/lib/active_support/tagged_logging.rb:68:in `block in tagged'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/activesupport-4.2.0/lib/active_support/tagged_logging.rb:26:in `tagged'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/activesupport-4.2.0/lib/active_support/tagged_logging.rb:68:in `tagged'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/railties-4.2.0/lib/rails/rack/logger.rb:20:in `call'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/request_store-1.1.0/lib/request_store/middleware.rb:8:in `call'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/actionpack-4.2.0/lib/action_dispatch/middleware/request_id.rb:21:in `call'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/rack-1.6.0/lib/rack/methodoverride.rb:22:in `call'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/rack-1.6.0/lib/rack/runtime.rb:18:in `call'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/activesupport-4.2.0/lib/active_support/cache/strategy/local_cache_middleware.rb:28:in `call'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/actionpack-4.2.0/lib/action_dispatch/middleware/static.rb:113:in `call'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/rack-1.6.0/lib/rack/sendfile.rb:113:in `call'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/airbrake-4.1.0/lib/airbrake/user_informer.rb:16:in `_call'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/airbrake-4.1.0/lib/airbrake/user_informer.rb:12:in `call'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/rack-timeout-0.2.0/lib/rack/timeout.rb:108:in `call'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/railties-4.2.0/lib/rails/engine.rb:518:in `call'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/railties-4.2.0/lib/rails/application.rb:164:in `call'
/home/ubuntu/planit/vendor/bundle/ruby/2.2.0/gems/rack-1.6.0/lib/rack/handler/webrick.rb:89:in `service'
/home/ubuntu/.rvm/rubies/ruby-2.2.0/lib/ruby/2.2.0/webrick/httpserver.rb:138:in `service'
/home/ubuntu/.rvm/rubies/ruby-2.2.0/lib/ruby/2.2.0/webrick/httpserver.rb:94:in `run'
/home/ubuntu/.rvm/rubies/ruby-2.2.0/lib/ruby/2.2.0/webrick/server.rb:294:in `block in start_thread'
  # http://<IP_ADDRESS>:52670/teaspoon/default?reporter=Console:1

Failed to load: http://<IP_ADDRESS>:52670/teaspoon/default?reporter=Console bundle exec teaspoon returned exit code 1

I'm running Rails 4.2, with a very-close-to-vanilla setup of Teaspoon:
teaspoon_env.rb
# Set RAILS_ROOT and load the environment if it's not already loaded.
unless defined?(Rails)
  ENV["RAILS_ROOT"] = File.expand_path("../../", __FILE__)
  require File.expand_path("../../config/environment", __FILE__)
end

Teaspoon.configure do |config|

  config.suite do |suite|

    suite.use_framework :jasmine, "1.3.1"

  end

  config.coverage do |coverage|

  end

end

Not sure if it's related to this issue. Seems not.

I had to manually mount teaspoon route in routes.rb for a custom "/jasmine" route:
but not sure if that's necessary for rails 4. I'm still on rails 3.
And not sure if that's necessary for the default. You can give it a try though.
if Rails.env.test? || Rails.env.development?
  mount Teaspoon::Engine => "/jasmine"
end

teaspoon_env.rb:
Teaspoon.configure do |config|

  # Determines where the Teaspoon routes will be mounted. Changing this to "/jasmine" would allow you to browse to
  # `http://localhost:3000/jasmine` to run your tests.
  config.mount_at = "/jasmine"


services did not display in thread?

I use dubbo and code the method in a new Thread.
like this
new Thread(new Runnable() {
public void run{
remoteMethod();
}
}).start();
But I find no dubbo service display.

Hi @lujs678
So it looks like remoteMethod() is invoking dubbo client call to a remote dubbo server?
If this is the case, then yes, nothing will display on the client side because a) there's no entry point in which a trace is created and b) because even if there was an entry point which created a trace, it wouldn't have propagated over to the thread that executes remoteMethod().
You could instead define remoteMethod() as a custom entry point by setting the profiler.entrypoint option in pinpoint.config.


Automagically discover chrome extension ID for testing

From time to time the chrome extension ID is updated.  Currently, the integration tests have the chrome extension ID hard coded.  This happens infrequently enough that manually updating the ID works, but it is not ideal.
Since the testing framework is fairly intricate, this issue could be very challenging to debug for the wrong person.  We should find a way to determine the extension ID from the integration testing context, and use it to build the URL in the link above.

@irdan Not sure how the "chrome" variable would be exposed from within the run_all.rb file.
Any thoughts on this?

We talked about using Selenium to capture it after it is installed, probably from the first-run page since it is supposed to popup every time. This means it would not be a variable in run_all.rb.
@irdan Does that match with what we talked about?

I think we only need to run this on the first run page, from selenium: driver.execute_script("return document.location.href"), since that should give the path containing the ID. The trick is to always select the correct document/window to run this on.

Done


Fix for foreachstate required params

Signed-off-by: Tihomir Surdilovic<EMAIL_ADDRESS>Many thanks for submitting your Pull Request :heart:!
Please specify parts this PR updates:

[ ] Specification
[x ] Schema
[ ] Examples
[ ] Extensions
[ ] Roadmap
[ ] Use Cases
[ ] Community
[ ] TCK
[ ] Other

What this PR does / why we need it:
fixes error for foreach state required params
Special notes for reviewers:
Additional information (if needed):

small but important fix. no need to discuss in meet


[CALCITE-695] Ensure the SINGLE_VALUE function would not be added when t...

Passed the unit test; Add one test case;

I don't think it would do the right thing on this query:
select deptno
from EMP
where deptno > (select min(deptno) * 2 + 10 from EMP group by gender)

or this:
select deptno
from EMP
where deptno > (values 10)

In your javadoc can you explain what you mean by "scalar"?

Or this:
select deptno
from EMP
where deptno > (select deptno * 2 + 10 from EMP order by deptno limit 1)

First of all, in the javadoc, scalar refers to a single value. For example, 1 is while [1, 2, 3] is not.
For query 1, the subquery will return a non-scalar. So it is reasonable for the plan to contain a single value function like below, right? (notice because there is no "gender" this column in the table, I substituted it with "empno" in your original query.)
LogicalProject(DEPTNO=[$7])
LogicalFilter(condition=[>($7, $9)])
LogicalJoin(condition=[true], joinType=[left])
LogicalTableScan(table=[[CATALOG, SALES, EMP]])
LogicalAggregate(group=[{}], agg#0=[SINGLE_VALUE($0)])
LogicalProject(EXPR$0=[+(*($1, 2), 10)])
LogicalAggregate(group=[{0}], agg#0=[MIN($1)])
LogicalProject(EMPNO=[$0], DEPTNO=[$7])
LogicalTableScan(table=[[CATALOG, SALES, EMP]])

For query 2, you are right. The current patch is not able to respond properly. I already had a fix.
For query 3, it is the LIMIT which makes the subquery return a scalar. I am working on it now.

While you are working on the fix, please think carefully. Your first patch added a feature but introduced an obvious bug. If you are not 100% safe to remove the SINGLE_VALUE function in a particular query you should not do it.

Sorry to quibble, but a list with one element is not scalar. See http://en.wikipedia.org/wiki/Scalar


Add links to caniuse and a script to show mapping completion

This depends on https://github.com/web-platform-dx/feature-set/pull/21.


Feat/redhat plugin

Description

Integrated redhat plugin flow (which is disabled later) along with API integration.
Added clear list state for new releases.
Combined assetGroupObservable service and domainObervableService.
Formatted date in notification details
Removed redundant API calls which were made on applying filters.
Formatted date for notifications API when Created date filter is applied.
Made download API and data API payload in sync for user policy screen, asset list, violation list.

Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Chore (no code changes)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also
list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[ ] My code follows the style guidelines of this project
[ ] My commit message/PR follows the contribution guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Other information:
List any documentation updates that are needed for the Wiki

How is this related to RedHat? If it's not - rename the branch. What exactly happened so you need such huge PR? Don't see neither any description nor task connected.
I block this PR! Clarification is required! @plyubich do you have any insights on what's going on here?

@mkinchin-paladin This PR is for porting all my EE commits to CE. It started with commits related to redhat plugin but since there were many commits remaining and I was facing many conflicts, I pushed all the commits to this branch.
I will rename the branch.

PR after renaming branch: https://github.com/PaladinCloud/CE/pull/1709


ensure that all songs are being added to the tracklist upon start

More than that, it should probably rebuild the tracklist periodically so we don't need to restart in order to pick up new additions.


Maven and Ivy metadata implementations should map scopes/configurations to variants

At the moment the interfaces MutableModuleComponentResolveMetadata and ModuleComponentResolveMetadata (together with ComponentVariantResolveMetadata) mix the concepts of configurations (old concept) and variants (new concept). The new Gradle metadata, which is implemented as a specific behaviour in the (Mutable)MavenModuleResolveMetadata implementations, already offers variants instead of configurations. Ivy and POM-based maven metadata do not.
To make the implementation more consistent, we should move (Mutable)MavenModuleResolveMetadataand (Mutable)IvyModuleResolveMetadata  to variants. Ivy configurations and mvn scopes should be mapped to variants rather than Gradle configurations.
Also we should add attributes by default to the derived variants (ivy and pom metadata). E.g. introducing the java library variants. This can then be used to treat external dependencies as java libraries (#977 and related issues).
The goal here is to reduce the special cases in DependencyMetadata.selectConfigurations().
Note: this issue is not about changing the use of "configurations" inside the resolution engine. It can still work with ConfigurationMetadata but we will most likely make more use of VariantBackedConfigurationMetadata.

I have spiked this here: java-lib-published branch
Here is an example of a maven POM with runtime scope dependencies that resolves differently with the spike: Build Comparison for org.jasig.cas:cas-server-webapp:4.2.7
I experimented with two approaches:

Derive variants directly in the parser (the earliest possible moment)
Do it only when we request the variants (the latest possible moment)

I ended up with (2). The main reason why (1) does not really work are "dependency artifacts". In maven, I do not know which artifacts I have published. The dependency tells me that a certain thing should have been published (e.g. using a classifier).
So instead of creating "real" variants, we attach the Java library attributes to the DefaultConfigurationMetadata for selected configurations (runtime / compile) and use attribute matching, instead of the legacy behavior, under certain conditions.
The spike could be used as base for the implementation. TODOs for that:

Only use the new behavior with the experimental flag enabled
Add solid test coverage. This needs to be tested with different combinations of repositories. We should address the general topic of testing "Mixed repository setups" (see https://github.com/gradle/gradle/issues/3394#issuecomment-349272432) together with this.
Check if the identification of published libraries is smart enough and refine it if necessary. The test coverage should reveal this. Atm. we only look for the configurations with name runtime and compile and attribute them. (Maybe we also need to look at the artifacts: e.g. is there a jar file?)

We should only do this for maven.
In ivy this conflicts with the explicit configuration selection (conf->targetConf). And we can not make the assumptions in Ivy about how a Java library is structured as we can do for maven which only has the known scopes.
In ivy, we could rather improve publishing such that it publishes modules that reflect Java libraries.
Something like:
<ivy-module>
  <configurations>
    <conf name="compile" visibility="public"/>
    <conf name="default" visibility="public" extends="compile,runtime"/>
    <conf name="runtime" visibility="public"/>
  </configurations>
  <publications>
    <artifact name="dep" type="jar" ext="jar" conf="compile,runtime"/>
  </publications>
  <dependencies>
    <dependency org="commons-io" name="commons-io" rev="1.2" conf="compile->compile;runtime->runtime"/>
    <dependency org="org.apache.httpcomponents" name="httpcomponents-client" rev="4.5.1" conf="runtime->runtime"/>
  </dependencies>
</ivy-module>

The implementation itself and the test coverage is ready for review. But this is waiting for a decision and implementation about a service injection mechanism in the metadata implemetantion classes. This is also needed for #3159 (see: https://github.com/gradle/gradle/commit/f9fe0e1ff1ed7f979cd573bd3ec680349191bbb3#commitcomment-26356346). Then this can go into review.
We are working on a solution now. Once we have it, I will rebase #3826 and adjust it to that solution.

Follow up for the ivy case: #3854


ContentTypeID for The Masked Carnivale not included in content_type.ts

Description
The ContentTypeID for The Masked Carnivale is not included in /resources/content_type.ts
Additional information
/resources/content_type.ts is listed as an auto-generated file from /util/gen_zone_id_and_info.py.  It likely needs to be made aware of The Masked Carnivale (ContentType = 27) for inclusion in auto-generation.

Yeah, the table itself doesn't have an entry for it in the game data: https://xivapi.com/ContentType?pretty=true
The script itself would need to be updated to manually insert an entry.

I think also probably the script should be updated to rename it to a better id than the first carnivale fight too.


Remove Base64'ing of file contents

Updated example of reading secret values from a file so that it doesn't base64 the contents in the file before kubectl reads it.
Otherwise, if a user follows this guide they will end up with a Secret containing double Base64 encoded credentials, which means that when accessed from a Pod they will get the Base64 encoding of the credential they actually want.
This is also the case for one of the localised versions but I will PR for that separately.

/lgtm
/approve


Fix OSX 16-bit/8-bit signed properties.

fix bug that caused signed/unsigned conversion problems for 16-bit and 8-bit UVC properties.

Nice! 👍

That did the trick!


Docs: Recipe on how to create styles with dynamic values using scoped tokens

Showcase how to create styles that map to dynamic values using scoped tokens for cases mentioned here #721
If you have any examples you'd like us to cover please share them in the comments.

Hi @hadihallak,
I have a few questions.
We are implementing a huge E-Commerce platform interface on Stitches and what we need most is to send dynamic values to variants via props.
[I'm asking for clarity. "https://github.com/stitchesjs/stitches/pull/721#issue-966540684" seems to have merged the suggested codes made here. Will this proposition be released?](https://github.com/stitchesjs/stitches/pull/721#issue-966540684 Is there a way to do the proposition mentioned here right now?. Is the topic you opened here to share the ways to do this?)
In my review "https://github.com/stitchesjs/stitches/pull/721#issue-966540684" the PR of the proposition made here has been accepted and merged into the canary branch. In which version will this development be published?

Hi @hadihallak, I have been waiting for your answer to our questions for a long time. Your answers on this matter are important to us. Thanks.

@bbg sorry this got burred in my notifications.
In general and by design Stitches tries to re-use styles whenever is possible to optimize performance.
However, we can achieve something similar to dynamic variants by using css properties. will try to create an example/package for this in the following days and share it with you here

Thank you @hadihallak, I'm patiently waiting for your example/package.

@bbg Working on this right now. will share ASAP

@bbg
Take a look at the code here https://github.com/hadihallak/stitches-dynamic-variants
live demo https://codesandbox.io/s/damp-hill-m4gwwc?file=/src/App.tsx


update babel polyfill

dist/index.js is still requiring '6to5/polyfill'.  Should change to 'babel/polyfill'
also the dependency on tweenInterpolate is requiring '6to5/polyfill'

Thanks for the issue. I had to upgrade tween-interpolate as well. It should be fine now :)


Ex7: Override - check solution

After a class: the expected solution seems not to be "incremental" (the integer:true pattern from previous extension should be used?) Clarify that.

Should we remove the integer true for the runner?
https://github.com/senecajs/seneca-in-practice/blob/master/problems/override/seneca-override-executor-run.js#L9

Already done: https://github.com/senecajs/seneca-in-practice/commit/cecbaac223f3e4119a491f836dc0996e86058cb4
...but I think all the exercise should be reviewed

This was confusing for me as well. I then noticed in the solution that the interger:true option wasn't even present anymore. Some clarification here would be nice.


[ADDITION] Tyro

Category
Fintech
Software to be added
Tyro
Supporting Material
URL: https://www.tyro.com/
Description: Providing tailored payment solutions and value-adding business banking products to Australian businesses. (Provide EFTPOS machines)
Size:
HQ: Sydney
LinkedIn: https://www.linkedin.com/company/tyro-payments/
See Record on Airtable:
https://airtable.com/app0Ox7pXdrBUIn23/tblYbuZoILuVA0X3L/recVM0LrSLkFq94gL

Enough info in Airtable, adding to list


Reuse browser process to improve lambda performance?

Did anyone try to cache browser process and simply connect/disconnect with every lambda run? So far seem to work well.
const chromium = require('chrome-aws-lambda')

let browserWSEndpoint

module.exports.handler = async function (event) {
  let browser = null

  try {
    if (browserWSEndpoint) {
      browser = await chromium.puppeteer.connect({ browserWSEndpoint })
    }
    else if (!browser || !browser.isConnected()) {
      browser = await chromium.puppeteer.launch({
        args: chromium.args,
        defaultViewport: chromium.defaultViewport,
        executablePath: await chromium.executablePath,
        headless: chromium.headless,
      })

      browserWSEndpoint = browser.wsEndpoint()
    }

    // ...

  } finally {
      if (browser) browser.disconnect()
  }
}

After playing with this for few days, it looks like networkidle0 won't resolve because of simultaneous requests triggered by multiple lambdas.

@alixaxel I'm asking the same question. Why shouldn't we reuse the puppeteer.launch?
AWS lambda is containerized and Node.js is single-thread. I didn't see any reason and drawbacks to reuse puppeteer.launch between each call.

I tried to run Mihailoff's code snippet with Playwright (after adapting it for its API, ie. playwright.chromium.launchServer), but I was not able to reuse the browser process.
Playwright can launch the Chromium background process (server), then connect to it via WS endpoint just fine, and you can do basically anything as long as this is the first lambda invocation.
During second invocation, I can still connect to the WS endpoint, create a new browser context, but when I attempt to create a new page, I get Browser has been closed error.
During third invocation, it seems the server is no longer running because I am getting error connect ECONNREFUSED <IP_ADDRESS>:36131. Most likely ws endpoint is incorrect. My best guess at what's happening is that during second invocation, the server process crashes, so during third invocation it no longer exists (so connection is not successful).

I also attempted to not create a server at all, instead simply cache the Browser instance created via playwright.chromium.launch. This however produced the same experience as above:

everything works fine during the first invocation
second invocation gives me Browser has been closed error when I attempt to create a page (after I successfully create a context)
third invocation gives me Target page, context or browser has been closed error when I attempt to create a context (before attempting to create a page)

I found a workaround to get the caching of Browser instances work across invocations though. The first time you create a browser, also create a new empty page, and just leave it there. Then go on and do your things in a new context. On second invocation, again create a new context and go on with your stuff. The initial empty page is just sitting there the whole time, somehow keeping the Browser instance from either crashing or closing itself.
I do not know how brittle this workaround is or how long its gonna work. But it seems to be the only way to get chrome-aws-lambda work with Playwright while reusing browsers. Creating a new browser every time is very expensive and severely impacts responsiveness. Once the workaround stops working, the only way to keep your performance is using playwright-built Docker images and adding lambda runtime on top of it, which sadly means dropping chrome-aws-lambda out of your project.

@paya-cz thanks for your comment, the code below seems to work
const chromium = require('chrome-aws-lambda')

let browserWSEndpoint

module.exports.handler = async function (event) {
  let browser, page

  try {
    if (browserWSEndpoint) {
      browser = await chromium.puppeteer.connect({ browserWSEndpoint })
    }

    if (!browser || !browser.isConnected()) {
      browser = await chromium.puppeteer.launch({
        args: chromium.args,
        defaultViewport: chromium.defaultViewport,
        executablePath: await chromium.executablePath,
        headless: chromium.headless,
      })

      // Keep blank page open
      browser.newPage()

      browserWSEndpoint = browser.wsEndpoint()
    }

    page = browser.newPage()
    // ...

  } finally {
      if (page) page.close()
      if (browser) browser.disconnect()
  }
}

@Mihailoff curious to know if this reusability does actually improve the performance?
Since we're still opening a new page before every new execution

So far I haven't seen any major performance improvements (connect vs launch). I think the actual page load is far more expensive than the new tab or browser launch. I'll continue to experiement.

FYI your code is not showing a tangible improvement because you're disconnecting from the browser at the end of each lambda run (see browser.disconnect() in the handler) hence you're spinning up a new browser for each run.
If you remove that line you should see a dramatic performance improvement (roughly 3x).

Based on the @paya-cz comment, leaving one blank page will keep the browser instance alive. I can confirm that the subsequent run connects to the same instance. It takes about 100-500ms to connect compared to 2-3s for the first launch.
Perhaps caching a connected state will improve it even further. @squallstar do you have any metrics to share?

@Mihailoff by not closing the connection with the browser I'm getting roughly 4x-5x performance increase.
I have done around a thousand tests on sample webpages and the average end-to-end (round trip) decreased from 2s to just 500ms.

subsequent calls for me get
subsequent calls get
ERROR	Error: ErrorEvent {
  [Symbol(kTarget)]: WebSocket {
    _events: [Object: null prototype] { open: [Function], error: [Function] },
    _eventsCount: 2,
    _maxListeners: undefined,
    _binaryType: 'nodebuffer',
    _closeCode: 1006,
    _closeFrameReceived: false,
    _closeFrameSent: false,
    _closeMessage: <Buffer >,
    _closeTimer: null,
    _extensions: {},
    _paused: false,
    _protocol: '',
    _readyState: 3,
    _receiver: null,
    _sender: null,
    _socket: null,
    _bufferedAmount: 0,
    _isServer: false,
    _redirects: 0,
    _url: 'ws://<IP_ADDRESS>:34019/devtools/browser/102994c2-6b14-4a54-87ef-58c30e453b4d',
    _originalIpc: false,
    _originalSecure: false,
    _originalHostOrSocketPath: '<IP_ADDRESS>:34019',
    _req: null,
    [Symbol(kCapture)]: false
  },
  [Symbol(kType)]: 'error',
  [Symbol(kError)]: Error: connect ECONNREFUSED <IP_ADDRESS>:34019
      at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1278:16) {
    errno: -111,
    code: 'ECONNREFUSED',
    syscall: 'connect',
    address: '<IP_ADDRESS>',
    port: 34019
  },
  [Symbol(kMessage)]: 'connect ECONNREFUSED <IP_ADDRESS>:34019'
}

Did anyone figure out how to get this working?

Im using this: https://github.com/Sparticuz/chromium
which should be pretty much the same thing

Hi, solution above works for me, but for some small period of time, after which I suppose aws force deletes browser instance. Or is it something new in aws lambda behavior?


Some utilities are still broken after re-installing .NET 6

Microsoft PowerToys version
0.64.1
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Awake, ColorPicker, FancyZones Editor, Hosts File Editor, PowerToys Run, TextExtractor
Steps to reproduce

Uninstall .NET 6.
Install .NET 6 again.
See that an error pops up when using the above utilities.

PowerToysReport_2022-11-12-19-11-25.zip
✔ Expected Behavior
As I have installed .NET 6 again, the above marked utilities should work.
❌ Actual Behavior
The above marked utilities do not work.
Other Software
No response

I fixed the issue after reinstalling powertoys.

This should be no longer an issue with the self containment.

This was fixed with .66


adapter-node sends cache headers when static files are 404

Describe the bug
During gradual deployment of an app with multiple instances, it is possible to load the main page from one server and the second request for static content goes to a server that has the old code deployed on it.
The old server, with the old code, does not yet have the new file, therefore it returns 404. So far, so good.
The problem arises, that with the return of 404, it also sends cache-control header with a high max-age and immutable. The browser and intermediate CDNs cache this response as much as they can.
https://github.com/sveltejs/kit/blob/29ffc78560a99ce387b10a24f48b7f01205f51e7/packages/adapter-node/src/handler.js#L40-L42
Therefore even after the new server gets the new app, the CDNs and the browser need to force cache clear in order to be able to get the new static files.
It would make sense that unless the response is 2xx, the cache header is not sent
Reproduction
This can be reproduced easily by creating an empty project with adapter-node and requesting an asset that does not exist
Logs
No response
System Info
System:
    OS: Windows 10 10.0.22624
    CPU: (32) x64 AMD Ryzen 9 5950X 16-Core Processor
    Memory: 41.70 GB / 63.92 GB
  Binaries:
    Node: 19.0.1 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.10 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 8.19.2 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.22621.1391.0), Chromium (109.0.1518.78)
    Internet Explorer: 11.0.22621.1
  npmPackages:
    @sveltejs/adapter-node: ^1.2.0 => 1.2.0
    @sveltejs/kit: ^1.8.8 => 1.8.8
    svelte: ^3.55.1 => 3.55.1
    vite: ^4.1.4 => 4.1.4

Severity
serious, but I can work around it
Additional Information
No response

I think #9089 is related, but not exactly this issue. I think if this issue is fixed, instances of #9089 will happen less frequently


Allow overriding env variables in check-reaper

Describe the feature you would like and why you want it
Currently env variables used in check reaper template are hard-coded as in,
MAX_PODS_THRESHOLD - https://github.com/Comcast/kuberhealthy/blob/master/deploy/helm/kuberhealthy/templates/check-reaper.yaml#L32
JOB_DELETE_TIME_DURATION - https://github.com/Comcast/kuberhealthy/blob/master/deploy/helm/kuberhealthy/templates/check-reaper.yaml#L34
Some checks (like daemonset check) keeps a lot of pods in Completed state which basically eats IPs. If we could reduce(override) these values, it will be really useful.
Additional context
In general, I have observed some hard-coded values (which actually are set through variables in templates in most cases but) not exposed through values.yaml file which makes it a bit to difficult to customize things. I have opened few other issues like #766 , #760 etc. Allowing customizing the checks behaviors wherever possible will make them more usable IMHO. Thanks!

Raised https://github.com/Comcast/kuberhealthy/pull/769

Raised https://github.com/Comcast/kuberhealthy/pull/769

This makes sense, merged in #769! Closing this issue for now, but will keep your other issue #766! Feel free to submit prs as you see fit! Thank you!! :D

This makes sense, merged in #769! Closing this issue for now, but will keep your other issue #766! Feel free to submit prs as you see fit! Thank you!! :D


[nrf noup] drivers: entropy: Switch entropy driver in nRF54L20

nrf-squash! [nrf noup] entropy: Add fake entropy nRF PRNG driver
Change psa driver to nordic entropy. nRF54L20 needs it as well as nRF54H20.

Why would nrf54h20 use fake entropy?

This is nRF54L20, not nRF54H20. Also PSA entropy does not work on nRF54L20 yet IIRC. Without this, many samples we need for testing don't build.

Why would nrf54h20 use fake entropy?

This is nRF54L20, not nRF54H20. Also PSA entropy does not work on nRF54L20 yet IIRC. Without this, many samples we need for testing don't build.

got it :)


Adding custom user-agent, bypass Yahoo's block

It seems like Yahoo blocked python user agent. Changing the agent to "custom" seems to fix it.

This fixed worked for a couple of days
page = requests.get(url, headers={'User-Agent':'Custom'})
but now I'm getting the same error again, anyone else?
Traceback (most recent call last):
File "/Users/alex/PycharmProjects/Narus/scratch/scr8.py", line 35, in
yec_raw = pd.DataFrame(yec.earnings_between(date_from, date_to))
File "/Users/alex/PycharmProjects/Narus/venv/lib/python3.7/site-packages/yahoo_earnings_calendar/scraper.py", line 139, in earnings_between
earnings_data += self.earnings_on(current_date)
File "/Users/alex/PycharmProjects/Narus/venv/lib/python3.7/site-packages/yahoo_earnings_calendar/scraper.py", line 93, in earnings_on
page_data_dict = self._get_data_dict(dated_url)
File "/Users/alex/PycharmProjects/Narus/venv/lib/python3.7/site-packages/yahoo_earnings_calendar/scraper.py", line 39, in _get_data_dict
'\n') if row.startswith('root.App.main = ')][0][:-1]
IndexError: list index out of range

If you are relying on earnings dates to make actual money you should use a robust service like earningscalendar.net, else you'll be spending more time playing cat and mouse to avoid scraping blocks.


Fred/sol 21 support annotation management within

Changes are now stale, closing for now until functionality can be refactored


ListItemComments. Suggestions won't appear when rendering on FluentUi Pannel (Was working until now)

Category
[ ] Enhancement
[X] Bug
[ ] Question
Version
Please specify what version of the library you are using: [    3.5.0 ,   3.9.0   ] Happens on both
Observed Behavior
List item comments rendered in panel. Rendered suggestions are not visible (even though something happens because panel scrollbar sometimes appears depending on number of comments), probably hidden behind panel.
When rendering the same component somewhere else, suggestions appear without issues.
Steps to Reproduce
Render a list item comments component in a panel. Try to add someone with @XXX, suggestions won't appear.
Thanks!
Nothing changed by our side, and it was working fine a few days ago. Maybe it has something to do with the suggestion panel styles.

I noticed the div is being populated with data correctly when doing the @. But it isn't able to render on top of the panel.

Some more info:
I was able to freeze the DOM and manipulate the component styles while showing the suggestions. For some reason, it gets the "Left" css property wrongly and gets warped out of the page.
Once you deactivate this property, at least it's close to where it has to be.

Hi @Vertamin,
Did you have the chance to retry recently? Maybe with latest version (3.15.0)?


Example for jupyter-book

Hi
Could you add an example or clarify how to run with Jupiter-book? I tried as you suggested in the README to add
['html', 'zundler', 1]
to the _config.yml at
sphinx:
config:
nb_mime_priority_overrides: [
but it has no effect. Did I do it wrong or something?
Thanks in advance.

You're right, either something changed meanwhile or I forget to put --builder custom in there. For whatever reason you need to tell jupyter-book twice that you want a custom builder. I'll update the readme.

Works perfectly, thanks a lot!


Add ENOTSUP constant for uclibc

In the vein of other linux-like c libs, just define ENOTSUP as equal to EOPNOTSUPP.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @JohnTitor (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Actually I'm not sure that this is correct.  Seems like ENOTSUP is defined in libc/src/unix/uclibc/mips/mod.rs already.  I must not understand when something belongs in an arch-specific sub-module.
Some of this must have been generated programmatically.  If anyone knows how that was done, I'd love a heads up.


Add range expressions and types

Range expressions are inferred as RangeIterators while range type annotations are inferred as Range types which a new internal type.  The new type was necessary due to the complex rules dictacting how ranges unify with themselves and number literal types.

Codecov Report
Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (0587eb2) 80.80% compared to head (259a5b2) 81.07%.

Files
Patch %
Lines

src/Escalier.TypeChecker/Unify.fs
66.66%
4 Missing :warning:

src/Escalier.Data/Library.fs
0.00%
2 Missing :warning:

src/Escalier.TypeChecker/Infer.fs
90.47%
2 Missing :warning:

src/Escalier.Parser/Parser.fs
96.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #98      +/-   ##
==========================================
+ Coverage   80.80%   81.07%   +0.27%
==========================================
  Files          17       17
  Lines        5198     5262      +64
  Branches      637      646       +9
==========================================
+ Hits         4200     4266      +66
+ Misses        998      996       -2

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


Bump framework version

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2779245618

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2779245618
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2779245618

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2779245618
Status: failure


Symlink redis to /usr/bin

As far as I can tell, both redis-cli and redis-server aren't symlinked to /usr/bin during install. Is there any reasoning behind this?

@izolate Yes... this role was originally using the alternatives module to handle this, but was removed due to a regression discovered in https://github.com/DavidWittman/ansible-redis/issues/50.
I'm confident the issue has been addressed though and the alternatives should be brought back in.


Create Linux Rabbit MQ Recipe

Documentation:  https://docs.newrelic.com/docs/integrations/host-integrations/host-integrations-list/rabbitmq-monitoring-integration
Required Configuration :

Username: Prompt User, assume no default value is provided
Password: Prompt User, assume no default value is provided
Hostname: Allow user to confirm default of localhost
Port: Allow user to configure default of 15672
Configuration Path: Prompt user Please enter your RabbitMQ Configuration File Path:, assume no default is provided
Queues: Prompt User Please enter a comma separated list of the queue names you'd like to collect data from:
Exhanges: Prompt User: If desired, please enter a comma separated list of the exchanges you'd like to collect data from:
Vhosts: Prompt user: if desired, please enter a comma separated list of the Vhosts you'd like to collect data from:
SSL: Prompt: Would you like to configure SSL? Y/N IF YES set use_ssl to true then

Prompt: Please enter the location of the SSL certificate file or directory: Set ca_bundle_dir or ca_bundle_file accordingly.

For Logging: Ensure that common log file directories/patterns & process name for RabbitMQ are populated in the beginning of the recipe.
Aha! Link: https://newrelic.aha.io/features/VIRTUOSO-63

@rishimukhop1 hello, I think there's an error and the default port should be 5672 not 15672

@JakubKotkowiak The default port for RabbitMQ is 5672, but the integration connects to the RabbitMQ Management Plugin, which is on port 15672 by default.

Also, we should add a assert_pre_req/create_user_notification task that performs checks and displays some of the prereqs in this part of the documentation. Specifically:
Checks:

RabbitMQ Management Plugin is configured
RabbitMQ command line tool, rabbitmqctl, in the PATH of the root user.
Infrastructure Agent installed / running

The output message can look like this (I think these emojis work as is, but will need to confirm with our designer):

Note: In order for this integration to work, you must have:
✅ RabbitMQ Management Plugin configured
❌ RabbitMQ command line tool, rabbitmqctl, in the PATH of the root user.
✅ The New Relic Infrastructure Agent installed and newrelic-infra.service enabled.

I need to do some checking with the team to land on a standard pattern for us to do these checks/assertions to the user and figure out where that fits in the CLI UX. For now, I think adding the above messaging will suffice.


🌱 Uplift CAPI to v1.3.2

What this PR does / why we need it:
https://github.com/kubernetes-sigs/cluster-api/releases/tag/v1.3.2

/retest
/test-ubuntu-integration-release-1-3

/retest


dockerfile.getBuffer is not a function

[Error - 22:42:16] Request textDocument/formatting failed.
  Message: Request textDocument/formatting failed with message: Error while loading rule 'dprint-integration/dprint': dockerfile.getBuffer is not a function

maybe use ?. for it?
I'm not using docker.
I also log it
dockerfile { getPath: [Function: getPath] }

Get
[Error - 22:44:50] TypeError: Error while loading rule 'dprint-integration/dprint': WebAssembly.Module(): Argument 0 must be a buffer source

after fixing this


[SR-3936] Fix it for missing property type

Solve SR-3936 by providing a fix-it for the “Computed property must have an explicit type” diagnostic that insert a type placeholder.

Resolves SR-3936.

@swift-ci Please smoke test


Switch D and C berthing positions

Summary of changes
Asana Ticket: Handle Park Street WB berth changes
The berth switch is slated to start at the beginning of the Summer rating which is Sunday, June 19th. This PR adjusts the sign configurations for the sourcing of the data so that the text and audio will go to the new sign zones that will reflect the berth switch.
Formatting changes are also getting mixed into this PR which makes the count of changed lines quite high, but the only configs that were actually changed were for park_st_westbound_wall_track and park_st_westbound_fence_track.
Before switch (Screenshot from Signs UI dev):

After switch (Screenshot from running Realtime Signs branch and Signs UI locally):

Deployment
Deployment should happen either very late on June 18th after the end of service, or very early on June 19th at the start of service.

@cmaddox5 yeah sorry about all the fomatting stuff. I think because we upgraded the Elixir version recently, mix format might have been upgraded too and I have format on save set on vscode, but might as well apply the formatting standard here anyway.
And yep that's all that changed. We just swapped the stop_ids of the D and C lines so that C predictions now go to the sign zone park_st_westbound_fence_track and D predictions now go to the park_st_westbound_wall_track sign zone


handle case where open/save dialog is cancelled

If you selected the Save As or any of the Open menu options and then hit Cancel on the corresponding file dialog, I was still sending the request to NeoVim with a filename of undefined.  This caused Save As to create a file named undefined rather than cancelling the operation.  I now check to see if a filename was selected before sending the request.

Good catch! Thanks for the fix!


Fix: Dropdown source item set as disabled throws warning

fixes #1714

It's been some time, what is preventing this from being merged?


Liquid tags not rendered on main page when post has no excerpt

When a liquid tag is used in a post (like with plugins), it is not rendered on the home page, whether it is a base liquid tag (like {% if %}) or a plugin tag, but it is correctly rendered when viewing the post.
It can be easily reproduced by adding a simple {% if page.title %}{{page.title}}{% endif %} on the myfirstpost.md page in the test site.
However, when using the excerpt feature, the liquid tags are correctly rendered on the main page, even when the excerpt is equal to the content (when adding the excerpt separator at the end of the post).
So, on the home page, liquid tags are not rendered when using post.content but are rendered when using post.exceprt.
I will try to take a look at the code if I have time.
I am using the latest version (7a87a57).

I think this is the same bug as in #246.
I'll try to fix it during the following week.

Oh my bad, I searched if there already was an issue, didn't find it. I close this one since it is a duplicate.


Added new method to CastProxy called simpleCast()

Right now CastProxy.prototype.cast invokes this.sender_.cast to start casting and on success just unloads the local player calling this.localPlayer_.unload()
I needed a way to start casting without unloading the local player.
One possible solution was to pass an optional parameter to CastProxy.prototype.cast like this:
/**
 * @param {boolean=} opt_keepLocal True to not unload local player on succesfull casting.
 * @return {!Promise} Resolved when connected to a receiver.  Rejected if the
 *   connection fails or is canceled by the user.
 * @suppress {unnecessaryCasts} to cast Player to Object to access with []
 * @export
*/
shaka.cast.CastProxy.prototype.cast = function(opt_keepLocal) {
   var initState = this.getInitState_();

   // TODO: transfer manually-selected tracks?
   // TODO: transfer side-loaded text tracks?

  return this.sender_.cast(initState)
      if (!opt_keepLocal) {
        // Unload the local manifest when casting succeeds.
        return this.localPlayer_.unload();
      }
  }.bind(this));
}

and call it like this:
this.castProxy_.cast(true);

But I didn't like the boolean parameter, so I made another method and call it like this:
this.castProxy_.simpleCast();

I'm not happy with the name, but I don't have a better one for now.

Can you please explain the use-case for leaving the local player while casting?

I'm using videojs with shaka-player. When I cast, the video is unloaded and videojs controls are lost. I lose all control over the chromecast. I can't play/pause, change volume or change time. So I need the video to be loaded, but paused to still have access to controls.

Ah, I see.  Then isn't that a UI issue for the videojs tech to fix?
There are a lot of good reasons to unload the local video player.  For one, it's consistent with the cast experience of other applications.  For another, some protected content can have restrictions on the number of concurrent playbacks by the same user, so it's important to unload the content locally when playing remotely.

@joeyparrish You're right, it can be problematic with concurrent playback on protected content. Forget about this, I will see how I can come with a solution unloading the local video.


PAGENO in AL

On a report I'm using CurrReport.PAGENO and VSCode warns the following:
[AL] 'PageNo' is deprecated
Any idea how it should be done now?

CurrReport.PAGENO has been depreciated for years, in RDLC you can't use it anyway. Please use PageNumber and Totalpages in RDLC instead. AL is not aware of any pagenumbers in the layout, imho...


Auto Re-play when video reached end.

Hello ,
I like this lib very much and simple to use. But how can I replay same video when it reaches to end. I mean I want to play video in infinite loop (until user stops the video).

@dineshguptha you can use below code to play video infinitely.
Implement the delegate VGPlayerDelegate and implement below method.
func vgPlayer(_ player: VGPlayer, stateDidChange state: VGPlayerState) {
print("player State ",state)
if state == .playFinished {
        player.replaceVideo((player.contentURL)!)
        player.play()
    }
}


How to solve it  "ImportError: No module named readline"

I tried to run interactive.py on python 2.7.
But it had some of error like following.
I finished clone and update "git submodule init; git submodule update"
Please let me know how to solve it.
Thank you.
**>>> ================================ RESTART ================================

Traceback (most recent call last):
File "D:\ParrotDroneSDK\Python\ParrotPythonSDK\bybop\samples\interactive.py", line 7, in
import readline
ImportError: No module named readline

**

I guess you're on windows, and windows does not seem to have the readline module available.
As this module is only required for code completion in the interactive console of the sample, you can remove the lines 7, 8, 47 and 48 of the interactive.py sample (i.e all the lines referring to readline or rlcompleter)
Anyways, I have to say that I did never actually (and don't plan to) test this project on windows, so I can't really help more if it breaks in the internals.

Hey @hsy77101
I'm not sure if the change is done by me or is already there, but on windows machines you can simply use pyreadline instead of readline and it works fine.
@N-Bz : If you want you could ping me on windows related problems, I got it set up and running on windows 8 and 10 :)
I'm happy to help

I tried to pyreadline instead of readline. But it was same error.
I use windows7 with python 2.7.12.
Do I need to modify interactive.py more?
Please check the following procedure I did.

I just run Python GUI.
File>Open..>interative.py
Run>Run Module
But I got error.

After removing the lines 7, 8, 47 and 48 of the interactive.py sample, I got following errors.
Traceback (most recent call last):
File "C:\Python27\bybop\samples\interactive.py", line 12, in
from Bybop_Discovery import *
File "../src\Bybop_Discovery.py", line 3, in
from zeroconf import ServiceBrowser, Zeroconf
ImportError: No module named zeroconf

Have you installed the Zeroconf and pyreadline packages? If so, in which version?

I just installed python 2.7.12 on  windows7.
And then After cloning this repo, I initialized & updated the submodules: git submodule init; git submodule update
That's all I did.

The error message "No module named XY found" says that the package is missing on your system.
You have to install the required packages like pyreadline and zeroconf (e.g. by using pip). Be aware that there may be a bug in the current zeroconf package (see https://github.com/N-Bz/bybop/issues/3).

Try to stick with 32bit edition of python on windows. Had some issues getting the packages installed and running at first until I switched over.

Since @hsy77101 did not post here since 11 days, I'll assume that @Tala (btw, thanks for this !) answers solved his problem.

Sorry. I was on business trip.
I am still on looking for the solution.

I mainly use Anaconda python x64 and didn't have any problems installing the packages. If you have further information what are the problems @hsy77101 please let us know.

i have the same problem but in kali linux
( File "brutespray.py", line 4, in
import readline, glob
ImportError: No module named readline)
plz someone can help

I am facing this issue in my Sles-10 machine with 2.7.10 python version
[20180719-05:19:09|root@hgtest01-vm04 ~] python
Python 2.7.10 (default, Jul 18 2018, 01:02:37)
[GCC 4.1.2 20070115 (SUSE Linux)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
Traceback (most recent call last):
File "/etc/pythonstart", line 7, in
import readline
ImportError: No module named readline

Please help.

SLES 11 SP4
I had the same problem and was able to fix it by installing  libreadline5-32bit.
hostname: # python2
[GCC 4.3.4 [gcc-4_3-branch revision 152973]] on linux2
Type "help", "copyright", "credits" or "license" for more information.
Traceback (most recent call last):
File "/etc/pythonstart", line 7, in
import readline
ImportError: No module named readline
hostname:  # zypper in libreadline5-32bit
hostname: # python2
Python 2.6.9 (unknown, Apr  7 2015, 08:28:12)
[GCC 4.3.4 [gcc-4_3-branch revision 152973]] on linux2
Type "help", "copyright", "credits" or "license" for more information.

installlocation\lib\site-packages\vault\modules\autocomplete.py", line 2, in
import readline
ModuleNotFoundError: No module named 'readline'
you need to install pyreadline if you are using windows from CLI
pip install pyreadline
Collecting pyreadline
Downloading https://files.pythonhosted.org/packages/bc/7c/d724ef1ec3ab2125f38a1d53285745445ec4a8f19b9bb0761b4064316679/pyreadline-2.1.zip (109kB)
|████████████████████████████████| 112kB 113kB/s
Installing collected packages: pyreadline
Running setup.py install for pyreadline ... done
Successfully installed pyreadline-2.1
vault
__      __         _ _           .----.
\ \    / /        | | |         / /  \
\ \  / /_ _ _   | | | | |__| |
\ / / ` | | | | | _|    .' |   || '.
\  / (| | || | | |_     '..'
/ _,|_,||_|    |            |
'.______.'
Welcome to Vault. Please choose a secure secret key.

just install readline with:
pip install readline
KR ;)


Work on the functions section

(this can get a temp merge with no close while it's being developed)


Provide the schema for IDEs

It'd be really nice to have editor assistance, is it possible to have JSON/YAML schema files?

@Sacro Yeah indeed it is. Do you know any good tool to do this?

I think I'm going with https://github.com/invopop/jsonschema as I haven't find alternative for it, but it doesn't support yaml tag which is what I'm using but still not too bad to add it.

@Sacro I just created a PR for this feature, everything seems good so far so I might merge this tomorrow after more testing. See if there's anything you'd like to add. Thank you for the suggestions!

That looks great, I did try it myself but I had all the yaml tags converted to json for some reason, and I couldn't get the tests to pass.
One other next feature would be filing it with https://github.com/SchemaStore/schemastore/blob/master/CONTRIBUTING.md so it gets picked up more automatically!

Yeah yaml tag is needed both upstream Talos and here so I used Reflector.FieldNameTag = "yaml" instead.
I found a problem with nodes[].installDiskSelector.size because there's something being done internally by upstream to make a struct become string. I was thinking to ignore that field but it's not a solution so I just leave it and probably find a way to fix it in the future. Another thing is fields from upstream doesn't have any description/example because they're using comments and not tag.
I'll try to create a PR to SchemaStore and hopefully it can get merged. Thanks!

If you need to link to talos things, you can use $ref
They have a schema file at https://github.com/siderolabs/talos/blob/main/pkg/machinery/config/types/v1alpha1/schemas/v1alpha1_config.schema.json

@Sacro Thanks for the info! I see what I can do to it.
Btw the schemastore PR got merged :smile: https://github.com/SchemaStore/schemastore/pull/3352


Fix monolith statewide header link

Fixes #516.
There's now a JS implementation of the header's text ("Official website of the State of California"), which inserts that content directly into the DOM. I left the CSS implementation of that text in place too, just in case JS is disabled for some reason. We've established that it's messy and temporary, and I've made it a little messier here.
I don't know anything about how to test this on Flywheel. I'm not sure if I have access either. If my services are needed in that capacity, I'll need some help.
Also, let me know if I got the versioning of these monolith files incorrect. I didn't see any build tools or notes, so I just did it manually.

This looks good.
To test it: I added an account for you on api.cannabis.ca.gov.
Go to: https://api.cannabis.ca.gov/wp-admin/admin.php?page=caweb_options
Custom JS
Custom CSS
You can replace the relevant files to test your change fully integrated:
Here is what the CSS looks like.

Click the red X to remove just the file you don't want
Upload a new file
Save changes
At this point the file should update.

Every once it a while the cache may get stuck, in which case, you can add another point version to increment the file.

_monolith is our bucket. It contains code that Aaron & Konstantin worked on.
Your versioning guess is correct.

@xjensen Let me know if you are able to successfully test the code on api.cannabis.ca.gov & then I can approve this. Thank you!

When it's ready, I will upload it to production.

Here's a README for the files that I know we are using.
The others are archival in case we need to pull Aaron or Konstantin back in on the Design System style overlay.
https://github.com/cagov/cannabis.ca.gov/blob/main/_monolith/README.md

Thanks, @chachasikes, very helpful.
I've uploaded the new CSS and JS into api.cannabis.ca.gov. It's looking good and ready. The new version number for each is 1.0.4.
Under CAWeb Options, I needed to toggle on the Home Link (General Settings > Utility Header > Home Link). I'm assuming that's already activated on the production site.

Correct, it is enabled on production already. Thank you so much. Will upload later this afternoon.


Added which platforms are supported in README.md

Had to dig through the code to find which platforms were supported so added it to the readme.

Oh. Damn. I should've pulled this in but instead ended up doing it myself during the 0.4.0 release. Sorry about that, thanks for the PR tho!


Make S3File work as a context manager and add missing import 'datetime'

I also added a new unit test and made the unit tests work without a boto config file (using the passed in options)

Thanks much for this!


Add experimental support for an eZ80 toolchain using llvm and GNU binutils

Summary
Add experimental support for an eZ80 toolchain using llvm and GNU binutils.
An additional fix for the RTC driver to lock the RTC after setting values instead of unlocking it is included.
Impact
Allows a non-supported fork of llvm with Z80 support to be used to build ELF objects, and GNU binutils compiled with Z80 support enabled to be used as a linker. There are modifications to support 64-bit integers, and further modifications to allow a toolchain to support LONG LONG without supporting DOUBLE.
The assembly code for the eZ80 has been modified to support compilation using both ZDS-II and GNU assembler. This is partially accomplished with macros for the GNU assembler to partially implement the ZDS-II assembler directives.
Testing
I am using this toolchain to regularly build for the eZ80, with an additional (forthcoming) patch to add ELF support for the architecture. I have compiled the modified assembly using ZDS-II on a Linux VM using Wine, but not executed the result. I have validated boot code, interrupt handling, and drivers for SPI, Ethernet, and UART.
There is an error somewhere in the Ethernet driver that will occasionally lock the driver up if the receive buffer overflows. The error is not introduced by this patch.

@codebje could you enable z80 CI? it will avoid breaking z80 arch in the future. Only a few file need to change:
https://github.com/apache/incubator-nuttx/blob/master/tools/ci/docker/linux/Dockerfile
https://github.com/apache/incubator-nuttx/blob/master/tools/ci/testlist/other.dat

@codebje could you enable z80 CI? it will avoid breaking z80 arch in the future. Only a few file need to change:

I would like to do this, though there's a few roadblocks to sort out - the intrinsics mostly come from Zilog and aren't licensed for redistribution, and there's no prebuilt llvm for Z80+binutils yet. For CI builds I can make a prebuilt toolchain with stub intrinsics. The resulting binary won't run properly but the build will be tested without license issues.


[BEAM-13290] Cherry-pick https://github.com/apache/beam/pull/16018 to release branch.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @TheNeuralBit @dpcollins-google


Remove qscc concurrency limit

Signed-off-by: Wenjian Qiao<EMAIL_ADDRESS>Type of change

Improvement (improvement to code, performance, etc)

Description
Concurrency limit has been added to endorser and deliver grpc services
in PR https://github.com/hyperledger/fabric/pull/647.
Therefore, remove qscc concurrency limit since it is covered by endorser service.
Related issues
PR https://github.com/hyperledger/fabric/pull/647

/ci-run


platform.isAttached should return false if canvas is false-y

Expected behavior
isAttached() from platform.dom.js should only call _getParentNode() on valid a truth-y canvas value.
Current behavior
TypeError: Cannot read properties of null (reading 'parentNode')
  at _getParentNode(../../node_modules/chart.js/dist/chunks/helpers.segment.js:1820:24)
  at DomPlatform.isAttached(../../node_modules/chart.js/dist/chart.js:3434:23)
  at _a.bindResponsiveEvents(../../node_modules/chart.js/dist/chart.js:6266:18)
  at _a.bindEvents(../../node_modules/chart.js/dist/chart.js:6211:12)
  at _a._checkEventBindings(../../node_modules/chart.js/dist/chart.js:5907:12)
  at _a.update(../../node_modules/chart.js/dist/chart.js:5856:10)
  at this._doResize(../../node_modules/chart.js/dist/chart.js:5624:46)
  at sentryWrapped(../../node_modules/@sentry/browser/esm/helpers.js:37:17)

From sentry:

Reproducible sample
.
Optional extra steps/info to reproduce
No response
Context

While encountering the prior contribution (#11677) seemed to be frequent, this particular issue appears to be an even more unique edge case. Nonetheless, I've implemented a fix to enhance the stability of chart.js and the applications using it.
Alternatively, we could check for the canvas within _getParentNode itself. I will defer that decision to the @chartjs maintainers if they want this changed.

chart.js version
4.4.2
Browser name and version
Experienced on Chrome 122.0.0 according to my project's Sentry, but probably impacts others since this is missing a null check.
Link to your project
No response

Hello @etimberg / @LeeLenaleee 👋  Thanks again for the prompt review and merge.  Do you have an ETA for the release of 4.5.0 ?

@chartjs When should I expect this to land? Should I run a fork in production in the interim?


No unittests

I am following your youtube python tutorials for Python/Flask_Blog/ and it is fascinating.  Wondering if you are planning to put additional videos for unit testing.
Thank you!

Yes, I do plan on doing videos on testing once the Django series is finished.


Create a method for finalizing labels / deleting old deployments

From: Quick Rollbacks

We've decided not to do this.


Q: Is there any specifications on the image pixel values for the decoder?

Hi, do you know if there are any strict restrictions on the image input to the decoder? I remember it is mentioned to be [-1, +1] somewhere, but can we also used other values like [0, 1]? However, since we are adding random normal noise with mean 0 I guess not?

@xiankgx yea, this repository does it from -1 to +1, but i've seen some generative models out there (back when i trained a lot of GANs) that does it from 0 to 1. i don't actually know if i've ever read any papers that did a proper comparison between the two

We should be very careful when passing images to various places, for different CLIP implementations which expect different things, also as the prediction target or x_start for the decoder.

@xiankgx yes! we definitely need to keep an eye on normalization

@rom1504 yea, i think the issue is that the decoder will be trained images that are simply normalized to -1 to 1, but CLIP uses https://github.com/openai/CLIP/blob/main/clip/clip.py#L85 (but we can always just do this within the embed_image forward function for the CLIP adapter)

I don't understand.
Doesn't the decoder take as input (image, clip embedding) ?
If yes, then how is the normalization used in the clip forward related with the decoder training process?

@rom1504
basically, images usually start off normalized from a range of 0 to 1 (shrunk from 0 to 255)
for DDPMs, we normalize them to -1 to 1 using 2 * image - 1
for CLIP, if we do all the embedding processing externally, then there is no problem - however, for the decoder, we currently take in the image and do both DDPM and derive the CLIP image embedding. so I just have to make sure to unnormalize the image before renormalizing it with what CLIP was trained on, before passing it into the attention net. you can double check my work here! https://github.com/lucidrains/DALLE2-pytorch/blob/main/dalle2_pytorch/dalle2_pytorch.py#L228

ah I see, makes sense


Failure on main testing xarray.cross

What happened?
Failure on main testing xarray.cross
https://github.com/pydata/xarray/actions/runs/10298827771/job/28504816043#step:10:838
Possibly either a bug in apply_ufunc or an incorrect use of it.
What did you expect to happen?
No response
Minimal Complete Verifiable Example
No response
MVCE confirmation

[ ] Minimal example — the example is as focused as reasonably possible to demonstrate the underlying issue in xarray.
[ ] Complete example — the example is self-contained, including all data and the text of any traceback.
[ ] Verifiable example — the example copy & pastes into an IPython prompt or Binder notebook, returning the result.
[X] New issue — a search of GitHub Issues suggests this is not a duplicate.
[ ] Recent environment — the issue occurs with the latest version of xarray and its dependencies.

Relevant log output
No response
Anything else we need to know?
No response
Environment
main

It looks like this surfaced with new numpy 2.0.1 release.

So from what I can tell the align-path in below code returns should align a and b. But it returns multiple chunks instead of one chunk.
In the following elif-path there is special handling to rechunk. Hope this helps to figure out a solution.
https://github.com/pydata/xarray/blob/b22c4296f70ea9bf59cc1c8e6c34346489046338/xarray/core/computation.py#L1669-L1705