Enhancement Request: Web view of linked references

Idea:
Being able to Web View the referenced links to take out prints or save them as pdf. Your web viewer in Roam42 makes it a lot easier to create pdfs and take out prints but can the functionality be expanded to REFERENCED LINKS too?  One can select a few referenced links and see them all as pdf.
Note: If my earlier enhancement request #211 is considered, it will partially resolve this issue as one would be able to bilaterally drag and drop the linked references (blocks and pages) between the main page and linked references.

Problems faced/ why is this important?

Quick web view of any of the selected pages/ blocks on the main page and linked references.
One pdf page can accommodate as much linked references as possible.
More data on a single page, save the hassle of going through various half-filled pages. Efficient use of resources.

More illustrations to explain this concept are uploaded to this folder: https://photos.app.goo.gl/tiTJV1xKyDSmYQQ48
I don't know whether these suggestions are technically viable to implement or not but I thought the ideas might resonate with you. Thank you for your great work and efforts.

Roam42 Issue Log Audit - Will readdress if user interest resurfaces


[SPARK-35069][SQL] TRANSFORM forbidden  DISTICNT and ALL, also make the error clear

What changes were proposed in this pull request?
According to https://github.com/apache/spark/pull/29087#discussion_r612267050,  add UT in transform.sql
It seems that distinct is not recognized as a reserved word here
-- !query
explain extended SELECT TRANSFORM(distinct b, a, c)
                   USING 'cat' AS (a, b, c)
                 FROM script_trans
                 WHERE a <= 4
-- !query schema
struct<plan:string>
-- !query output
== Parsed Logical Plan ==
'ScriptTransformation [*], cat, [a#x, b#x, c#x], ScriptInputOutputSchema(List(),List(),None,None,List(),List(),None,None,false)
+- 'Project ['distinct AS b#x, 'a, 'c]
   +- 'Filter ('a <= 4)
      +- 'UnresolvedRelation [script_trans], [], false

== Analyzed Logical Plan ==
org.apache.spark.sql.AnalysisException: cannot resolve 'distinct' given input columns: [script_trans.a, script_trans.b, script_trans.c]; line 1 pos 34;
'ScriptTransformation [*], cat, [a#x, b#x, c#x], ScriptInputOutputSchema(List(),List(),None,None,List(),List(),None,None,false)
+- 'Project ['distinct AS b#x, a#x, c#x]
   +- Filter (a#x <= 4)
      +- SubqueryAlias script_trans
         +- View (`script_trans`, [a#x,b#x,c#x])
            +- Project [cast(a#x as int) AS a#x, cast(b#x as int) AS b#x, cast(c#x as int) AS c#x]
               +- Project [a#x, b#x, c#x]
                  +- SubqueryAlias script_trans
                     +- LocalRelation [a#x, b#x, c#x]

Hive's error

Why are the changes needed?
Does this PR introduce any user-facing change?
No
How was this patch tested?
Added Ut

Test build #137289 has started for PR 32149 at commit 118f242.

FYI @cloud-fan

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41869/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41869/

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

Test build #137289 has finished for PR 32149 at commit 118f242.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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

Test build #137340 has started for PR 32149 at commit 542aa46.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41919/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41919/

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

Test build #137340 has finished for PR 32149 at commit 542aa46.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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

thanks, merging to master!


Fix settings variable 'PIPELINE' to 'SOCIAL_AUTH_PIPELINE'

Changed variable name to be more specific.
'PIPELINE' variable is also a settings variable used in django-pipeline, which causes an error when used together. #838

You can already just use SOCIAL_AUTH_PIPELINE and it should work...

@webjunkie Thanks for proving my point.
The default namespace PSA uses is PIPELINE which causes an error when used with django-pipeline, (django-pipeline also uses PIPELINE) and as you said if you change it to SOCIAL_AUTH_PIPELINE for PSA there is no conflict between these two packages.
Thats the reason for this PR, I changed the default PIPELINE to SOCIAL_AUTH_PIPELINE.

@paulsoh You did not get me right. There is no use for this PR as the default is already SOCIAL_AUTH_PIPELINE. The self.setting automatically prefixes everything with SOCIAL_AUTH_ and at least for me, I just use SOCIAL_AUTH_WHATEVER in my settings and it works without conflicts.

python-social-auth will try several setting names in the following order:

SOCIAL_AUTH_<BACKEND_NAME>_PIPELINE
SOCIAL_AUTH_PIPELINE
PIPELINE

The utility setting_name in utils.py adds the setting prefix SOCIAL_AUTH_ to the names.

:+1:


Add Magic Secateurs plugin

The plugin adds a notification when the magic secateurs are in inventory and not equipped. Acts as a reminder to equip them before harvesting crops on farm runs to maximize yield and XP.

its highly recommended to not set enabledByDefault = false on external plugins; it confuses people because they install the plugin and it does nothing by defaultt

Fwiw, you can use the inventory changed event to see if secateurs are in the inventory instead of checking every render.

You can also utilize ItemId to grab the item’s id instead of hardcoding the value.

I'll echo what the others have said as well. We don't mind much the quality of the code in general, but removing enabledByDefault = false and adding a LICENSE to the repo should be done before we merge it.


Add screen_view analytics events

#463
#462
#461  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Function for creating a vanity url

I am checking out the UrlRedirect project and indeed it does a redirect.
But how to create the vanity url?  In other words, I have a long url, and I want to create the short URL so that I can use it with the redirect.  I don't see a function for that in the project. (UrlShortner.cs)
Am I missing something?

@gregoryagu @sergebsv This has been fixed when we merge the TinyBlazorAdmin repo with this one. Today the Azure function can create, list, redirect, delete.


update OWNERS file

Update owners on the EST personnel.

/cc @kashifest @dtantsur @elfosardo @zaneb
All of the active repositories in metal3-io organization need to be maintained or archived. We had discussion that this repository is not getting archived yet, so we need to update our personnel in OWNERS file. Let me know what you think.

/approve

/lgtm

Tide, wake up. Other PRs are held, put this in. Maybe fake trigger helps... :)
/test all

/hold

https://github.com/metal3-io/project-infra/pull/491 is needed to fix tide, there is non-existing check which it is waiting.

/test-ubuntu-integration-main

/unhold
Jenkins was the actual culprit, https://github.com/metal3-io/project-infra/pull/491 not needed.


feature request: ability to link directly to bot source

would be neat to be able to link directly to my bot's source, say in a post or from it's profile, so that people viewing the bot on bluesky can click through to see, rather than going to the blue bots site and trying to find via scrolling.

Agreed.

TODO:

[ ] Add react router to the project
[ ] Add backend endpoint (based on backend/list.php?) that returns info and source for a single bot
[ ] Create single bot page component in React
[ ] Set up route to single bot page
[ ] Create an "All bots" index page to display all bots, not just the last ten / most popular ten, with links to single bot pages


My PR commit 1 in branch PR2

Please review

Can one of the admins verify this patch?

ok to test

ok to test

ok to test


fix(consensus): remove the Clone trait bound from StreamData

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#2746  👈 (View in Graphite)
#2745
#2730
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


How can I update multiple columns in a row

let alice = users.filter(id == 1)
do {
if try db.run(alice.update(email <- "alice@me.com")) > 0 {
print("updated alice")
} else {
print("alice not found")
}
} catch {
print("update failed: (error)")
}
How should I do in order to update many field in a record
Example: I would like to update (email, name, age) of an user with id = 1, But I just found we only can update one field at once
Please Help. Thank you guys so much.

You can separate each column setter with a comma.
For example:
let alice = users.filter(id == 1)

try db.run(alice.update(
        email <-<EMAIL_ADDRESS>        name <- "Alice",
        age <- 24))

Not a bug, closing.

@anthonyjosephfarina This throws an "Extra call in argument" error for me, this resolved the error:
let alice = users.filter(id == 1)

try db.run(alice.update(
        [email <-<EMAIL_ADDRESS>        name <- "Alice",
        age <- 24]))

@robert-cronin
Thanks for the information!
the [] is SUPER important and saved my day!
I did not notice the difference at first.


138 bug single transaction view incomplete

Pull Request Template
Description
Transaction view info and tabs with action, traces and raw view.
Fixes #138
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

@DonaldPeat this is done now if you want to take a look.


UTM origin coordinate precision loss

When a final map is saved with utm: false parameter, then UTM origin is substracted from all points and it is saved to accompanying .utm file. However precision of serialized coordinates is limited and it can cause a few meters displacement.
In my case, value "5572094.155411" was saved as "5.57209e+06".

Thanks for reporting the issue. I fixed the text format so the precision is preserved. It'll be merged into the main branch soon.

I just noticed that there are two separate functions affected by this formatting issue. I've create PR https://github.com/koide3/hdl_graph_slam/pull/185 with your fix applied to the second function.


print max memory, spilled items.

This PR is to investigate the test failure.

Didn't reproduce. Trying again.

Didn't reproduce.
From the stats output, it seemed that the assertion initially failed due to disk spilling - which is weird because those small instructions shouldn't take more than 100ms to execute. I blame it on the cloud system and close this PR for now.


move grouped dates on work edit page into fieldsets

Why was this change made?
Fixes #1827 - move input controls that are related to the same field into a single fieldset (just wrap the existing div) and ensure they have a legend.
How was this change tested?
Localhost browser
Which documentation and/or configurations were updated?

This diff is really odd, since all I did was wrap the existing divs in a <fieldset> and add a legend.


Cant use neural strategy

Sorry for my english. Always when i use this strategy i got the same errors and cant work. I used neural recomended configuration, and i try with differents % sell / buy, and i have enough money ( 1ETH In this case ). Bot make a lot orders and cant work. When i tried on sim i have better results than the others strategies. Thanks for your help

An error occurred { Error: binance POST https://api.binance.com/api/v3/order 400 Bad Request {"code":-2010,"msg":"Account has insufficient balance for requested action."} (possible reasons: invalid API keys, bad or old nonce, exchange is down or offline, on maintenance, DDoS protection, rate-limiting)
at binance.defaultErrorHandler (/home/Xt/Documents/code/zenbot/node_modules/ccxt/js/base/Exchange.js:400:15)
at response.text.then.text (/home/X/Documents/code/zenbot/node_modules/ccxt/js/base/Exchange.js:413:25)
at
at process._tickCallback (internal/process/next_tick.js:188:7) constructor: [Function: ExchangeNotAvailable] }

Binance API is down! unable to call buy, retrying in 20s
[ { price: '0.008271',
size: '2.06935609',
orig_size: '2.04582361',
remaining_size: '2.04582361',
orig_price: '0.008270',
cancel_after: 'day',
product_id: 'KMD-ETH',
post_only: true,
type: 'limit',
side: 'buy' } ]
An error occurred { Error: binance order cost = amount * price should be > 0.001 BTC {"code":-1013,"msg":"Filter failure: MIN_NOTIONAL"}
at binance.handleErrors (/home/X/zenbotm/node_modules/ccxt/js/binance.js:627:19)
at response.text.then.text (/home/X/zenbotm/node_modules/ccxt/js/base/Exchange.js:412:18)
at
at process._tickCallback (internal/process/next_tick.js:188:7) constructor: [Function: InvalidOrder] }

Binance API is down! unable to call cancelOrder, retrying in 20s
{ Error: binance cancelOrder() error: {"code":-2011,"msg":"UNKNOWN_ORDER"}
at binance.cancelOrder (/home/X/zenbotm/node_modules/ccxt/js/binance.js:531:23)
at
at process._tickCallback (internal/process/next_tick.js:188:7) constructor: [Function: OrderNotFound] }
[ { order_id: '1956329', product_id: 'KMD-ETH' } ]
An error occurred { Error: binance order cost = amount * price should be > 0.001 BTC {"code":-1013,"msg":"Filter failure: MIN_NOTIONAL"}
at binance.handleErrors (/home/X/zenbotm/node_modules/ccxt/js/binance.js:627:19)
at response.text.then.text (/home/X/zenbotm/node_modules/ccxt/js/base/Exchange.js:412:18)
at
at process._tickCallback (internal/process/next_tick.js:188:7) constructor: [Function: InvalidOrder] }

`

Dear issue reporter,
We have slightly changed our github issue policy and would now kindly request folks that have questions that they ask them in our zenbot subreddit.
You can find it here: https://reddit.com/r/zenbot
This issue will be closed, but if you disagree with your ticket being marked as a question feel free to leave a comment defending your case.
Thanks for contributing time and effort!
Greetings,
🤖


No method fro appending new metadata

I cannot find a method for appending new metadata like FileName.

Hi @antman1p I'm not sure exactly what you mean by appending new metadata? the parser interface for example returns a dictionary content parsed text content and metadata. You can do what you want with that returned struct? So that should be enough, right?

I got it to work in my script with this line:
parsed['metadata'].update({"FullFileName": str(file)}


[apiv2] fails with 'schema: error converting value for "follow"'

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description
Podman API returns an error when using the follow=1 parameter (or another parameter) parameter before the stderr or stdout streams when retrieving logs on a running container.

Steps to reproduce the issue:

Run a container

podman run -d --name hello-world hello-world

Use cURL to retreiving logs with the follow=1 parameter on the running container

curl --unix-socket /run/podman/podman.sock http://docker/v1.24/containers/hello-world/logs?follow=1%26stderr=1

Describe the results you received:
Results in the following error
{
  "cause": "schema: error converting value for \"follow\"",
  "message": "Failed to parse parameters for /v1.24/containers/hello-world/logs?follow=1%26stderr=1: schema: error converting value for \"follow\"",
  "response": 400
}

Describe the results you expected:
HTTP/200 OK
Additional information you deem important (e.g. issue happens only occasionally):
It appears to also break when both stderr and stdout are specified at same time. Perhaps it is having issues parsing multiple arguments.
Output of podman version:
Version:      2.0.0-dev
API Version:  1
Go Version:   go1.14.3
Git Commit:   d857275901e8c1ea7515360631e5894018e17f30
Built:        Fri Jun  5 17:00:00 2020
OS/Arch:      linux/amd64

Output of podman info --debug:
host:
  arch: amd64
  buildahVersion: 1.15.0-dev
  cgroupVersion: v2
  conmon:
    package: conmon-2.0.18-0.6.dev.git50aeae4.fc33.x86_64
    path: /usr/bin/conmon
    version: 'conmon version 2.0.18-dev, commit: 51e91bbc42aaf0676bb4023fb86f00460bf7a0a2'
  cpus: 4
  distribution:
    distribution: fedora
    version: "33"
  eventLogger: file
  hostname: gallucci-fedora.devlnk.net
  idMappings:
    gidmap: null
    uidmap: null
  kernel: 5.7.0-1.fc33.x86_64
  linkmode: dynamic
  memFree:<PHONE_NUMBER>8
  memTotal:<PHONE_NUMBER>2
  ociRuntime:
    name: crun
    package: crun-0.13-2.fc33.x86_64
    path: /usr/bin/crun
    version: |-
      crun version 0.13
      commit: e79e4de4ac16da0ce48777afb72c6241de870525
      spec: 1.0.0
      +SYSTEMD +SELINUX +APPARMOR +CAP +SECCOMP +EBPF +YAJL
  os: linux
  remoteSocket:
    exists: true
    path: /run/podman/podman.sock
  rootless: false
  slirp4netns:
    executable: ""
    package: ""
    version: ""
  swapFree:<PHONE_NUMBER>
  swapTotal:<PHONE_NUMBER>
  uptime: 19h 56m 58.63s (Approximately 0.79 days)
registries:
  search:
  - registry.fedoraproject.org
  - registry.access.redhat.com
  - registry.centos.org
  - docker.io
store:
  configFile: /etc/containers/storage.conf
  containerStore:
    number: 6
    paused: 0
    running: 4
    stopped: 2
  graphDriverName: overlay
  graphOptions:
    overlay.mountopt: nodev,metacopy=on
  graphRoot: /var/lib/containers/storage
  graphStatus:
    Backing Filesystem: xfs
    Native Overlay Diff: "false"
    Supports d_type: "true"
    Using metacopy: "true"
  imageStore:
    number: 3
  runRoot: /var/run/containers/storage
  volumePath: /var/lib/containers/storage/volumes
version:
  APIVersion: 1
  Built:<PHONE_NUMBER>
  BuiltTime: Fri Jun  5 17:00:00 2020
  GitCommit: d857275901e8c1ea7515360631e5894018e17f30
  GoVersion: go1.14.3
  OsArch: linux/amd64
  Version: 2.0.0-dev

Package info (e.g. output of rpm -q podman or apt list podman):
podman-2.0.0-0.121.dev.git1fcb678.fc33.x86_64

Additional environment details (AWS, VirtualBox, physical, etc.):
Fedora 33 (Rawhide Prerelease)

@jwhonce PTAL

@mheon @jwhonce I just confirmed the issue is not exactly how I described it. There is a problem, just different. What I realized is the API does not like the %26 encoding. Even the Docker API has the same issue. To get past the first error I had to use ampersands(&) with single quotes (') as follows
curl --unix-socket /run/podman/podman.sock 'http://docker/v1.24/containers/hello-world/logs?follow=1&stderr=1'

The REAL problem is this command never returns the expected HTTP/1.1 200 OK as it does with the Docker API. The Podman API seems to hang indefinitely.
Please let me know if you want me to file a new issue or modify this one to reflect the real issue.

@mheon @baude This appears to be a regression of Issue https://github.com/containers/libpod/issues/435 and related pull request https://github.com/containers/libpod/pull/437 addressed 2 years ago. I'll file a new ticket.

/Close
I am going to re-open this issue to better describe the issue. The true issue is following a log with the API hangs indefinitely just like it currently does with the CLI but fixed recently.


allow docker to bypass CORS w/env var DISABLE_CORS

It isn't documented in the README as it won't work on existing catalog of images.  It is only for new images.
Signed-off-by: Jesse Andrews<EMAIL_ADDRESS>

I haven't looked at an automated test yet

Thinking about this further it seems like having a command like:
cog serve r8.im/zeke/haiku-standard@sha256:78458ba6976fb7d159f9f3a336754d9e8467110c0fb7e8a2c86d93ccdb524f03

in cog serve you can address issues not directly related to the model:

handle CORS #713
static files #687
file handling #496 - and additionally where we can upload to GCS/S3/R2 or store in a local filesystem
make api consistent between requests against a local docker deploy and replicate deployment

Why not do this in python cog.server.http

baking non-model specific code into models couple / complicate improvements to serving experience / existing cog'd model will always have older versions of "serve" (docker images)
move file access/uploading outside of docker images to focus solely on running the model, not integrations beyond model

I haven't looked at the go code beyond determining how the python code is injected during model build time. Perhaps this is a poor idea that I'll see once I try to look at prototyping this.

see POC in #738

The name DISABLE_CORS is a bit ambiguous, as it could be taken to mean "do not allow cross-origin requests" or "allow any cross-origin request". Maybe something more explicit like BYPASS_CORS or CORS_ALLOW_ALL?

Blocked by this as well. Generally running Cog-powered models on production seem to be very restricted due to the fact that the http server is not modifiable. Allowing to modify the underlying FastAPI server would be greatly helpful.

I am patching and using it myself. I hope this PR will be incorporated into production soon.

Is this getting in any time soon? Or is there an alternative way to change CORS settings on the server?


plugins: update asyncio and aiohttp interfaces on custom plugins

Python modules were updated and some changes were necessary regarding asyncio and
aiohttp modules on custom plugins in order to make them work again.

thanks @julianolf !


Add files rolling to CSV reporter

In production it is better to have a set of bounded size csv files instead of unbounded one.

Bummer. Would have loved to see this supported as well. Our application is installed on customer servers (Which can be Windows or Linux by the way) so asking them to configure logrotate is a bit too much for this.


Fix editing from embedded graphs

This is happening from a preview of the blog post in ghost (figured be nicer to embed the example file than just link out to it). Not sure if its a Ghost issue or an embed player issue.

Also do we really need to have play buttons on this ? pretty clunky having the two successive panels for it. (I definitely need to design a V2 of player UI that is much more discreet)

two issues that I can see

embed player issue.
no title and author.

p.s. i'd love a more discreet interface

yep, once the blog & videos titles are done, this is what i'm onto next (player with UI optimised for desktop & for mobile, and smaller). Be good if we can also display thumbnails as poster frames by default. And maybe have an option for autoplay or not – can think of plenty of cases where you'd want autoplay (like in my blog post or on a discovery site when you go to the full player page)


Regulatory Compliance policies are now enabled by a single click

Contact Details
No response
What happened?
Task: 01
Steps: 10-15
Description of issue: There is a new way to enable Regulatory Compliance
New steps:

Search for ISO 27001
Click the toggle under Status to enable

or -

Search for ISO 27001
Select the ellipsis (...) and the View Standard Definition
Click Assign and select your subscription

Lab
Lab 03 Exercise 02 Mitigate threats using Microsoft Defender for Cloud
Relevant screenshots

Do you want to help us? 👏

[ ] Create a Pull Requestt by following this project's Contribution Guide

I'll do an update, but I think this whole Learning Path on Defender for Cloud is going away.


Improve Vulnerability Checks Dev3.0.0

There should be a way to reduce the amount of code that we have here. Might be able to pass an array of Build Numbers as well as CVENames to the Get-VulnerabilitiesByBuildNumbersForDisplay

I'll take care of this. Please assign to me.

We are going to be handling this in the Dev3.0.0 branch in this commit: b9fed60086dfbb2629bcc7f5b7d0ccdf892f2c96


Dashboard fixed bugs

What this PR does / why we need it:

fixed Statistics bug
fixed i18n bugs

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
@leonwanghui @xing-yang
Release note:

Coverage remained the same at 43.911% when pulling 019e4bbb4cd26517b71eb49dcb728cb957115623 on jimccfun:development into d77e6d2544ef53b1f2fe254d75ec9427e0a31cc3 on opensds:development.


For testing only. Evaluate the new pr-title-cc action when the PR is created from a forked repository

DISCLAIMER: this PR is not intended to be merged
It serves as test for implementation of #92 when the Pull Request is created from a forked repsitory.

#92 has been merged, so closing.


Delete the 'formats' block from the info.json file after writing it for a YouTube video

It's full of temporary URLs that aren't actually worth saving, and a significant contribution to the size of the JSON.
Also, let's prettify the JSON while we're in there.

Done!


[CONSOLE] Add support for Base Goerli testnet

Added with: https://github.com/superfluid-finance/superfluid-console/pull/178


Allow to add multiple co-authors at once

Description

Screencast
https://github.com/raycast/extensions/assets/19756164/189efbc3-cf14-4df5-9c72-df0d24de4e52

Checklist

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

@pernielsentikaer should I update the changelog?

@mateusjunges yes please 🙂
@nickradford do you mind checking this?

@pernielsentikaer done 🙂

Thanks @pernielsentikaer!


⚠ Itaca Gallery has degraded performance

In 0c87168, Itaca Gallery ($SITE_TCGLR) experienced degraded performance:

HTTP code: 200
Response time: 6199 ms

Resolved: Itaca Gallery performance has improved in ba38e1b after 5 minutes.


fix(custom-element): fix parent, observer and exposed issues when remove element and append it back after fully unmounted

close https://github.com/vuejs/core/issues/12412
changes:

extract observer code
observe again and re-mount if it's resolved and no instance, new parent will be set on new mount
exposed properties should be configurable, and we should delete them if element is fully unmounted, so that we can expose again in new mount

Thank you for your contribution. I have some suggestions and questions.

Click "remove el2" and "append el2"

I think the reason for the error here is that _parent and _resolved were not reset when disconnected. Therefore, the correct fix should be:
// in disconnectedCallback
this._parent = undefined
this._resolved = false

and then click "change el2 attr", its attribute is changed, but prop is not updated

The fix to this part are generally correct. However, I don't understand why this line needs to be added.

extract observer code

I think this change is no longer necessary if reset _parent and _resolved in disconnectedCallback

@edison1105 Thanks for your suggestions! It's a good fix for current issues.
However I found another issue regarding moving element. If we move a child element directly into other element(not waiting two nextTick), its parent context is not updated
I will check more on this and update issue and commit this tonight

Actually setting _resolved to false can cause some duplicate executions, like _applyStyles

Actually setting _resolved to false can cause some duplicate executions, like _applyStyles

re-mount should also re-apply the style, right ?

The reason work with Teleport (shadowRoot: false) test case fails is that: my-y is defined first, my-p is defined later, and it removes my-p and causes unmounting.
I think this is not code issue, it's test case issue. We should always define outer element first, especially when SSR, otherwise inner element can't find correct parent, so I updated this test case

Updated, thanks for the info
Also I was correct, this._resolved = false will lead to applyStyle twice. unmount doesn't remove style nodes so we don't need to applyStyle again. I have added that in test case
So we just need to mount again instead of resolveDef again


Remove the need to call layout and layoutOptions in mwc-select

Is your feature request related to a problem? Please describe.
There have been several requests to not require calling layout or layoutOptions on several of our components like MWC select. It's generally bad DX
Describe the solution you'd like
Change or rewrite these components so that the user doesn't have to manually call a layout function
Describe alternatives you've considered
Asking users to write mutation or resize observers
Additional context
Dan working on pitching core to enable slotchanged in wit

Fixed in M3


[Bug] [Node.js] skywalking s3 upload error

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache SkyWalking Component
NodeJS Server Side Agent (apache/skywalking-nodejs)
What happened
Hi
I am using the skywalking skywalking-backend-js library, when uploading file (size> 1Mb) to s3 using AWS nodeJs SDK.
the upload is failing with an error. I see skywalking files in the stack trace.
nom page  versions
"skywalking-backend-js": "^0.4.1"
"aws-sdk": "^2.418.0",
agent config:
agent.start({
  serviceName: 's3 upload',
  collectorAddress: 'localhost:11800',
});

complete stack trace:
{
  error: Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
      at ClientRequest.setHeader (_http_outgoing.js:518:11)
      at /Users/demo-apps/node_modules/skywalking-backend-js/lib/plugins/HttpPlugin.js:101:76
      at Array.forEach (<anonymous>)
      at Object.module.request (/Users/demo-apps/node_modules/skywalking-backend-js/lib/plugins/HttpPlugin.js:101:37)
      at features.constructor.handleRequest (/Users/demo-apps/node_modules/aws-sdk/lib/http/node.js:42:23)
      at executeSend (/Users/demo-apps/node_modules/aws-sdk/lib/event_listeners.js:334:29)
      at Request.SEND (/Users/demo-apps/node_modules/aws-sdk/lib/event_listeners.js:348:9)
      at Request.callListeners (/Users/demo-apps/node_modules/aws-sdk/lib/sequential_executor.js:102:18)
      at Request.emit (/Users/demo-apps/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
      at Request.emit (/Users/demo-apps/node_modules/aws-sdk/lib/request.js:683:14) {
    code: 'NetworkingError',
    region: 'us-east-1',
    hostname: 'localhost',
    retryable: true,
    time: 2022-04-06T09:17:22.133Z
  }
}

What you expected to happen
the application should upload file without any failures.
How to reproduce
use simple setup to reproduce the bug using Localstack
index.js
const agent = require('skywalking-backend-js').default;
const express = require('express');
const uploadFile = require('./upload')

agent.start({
  serviceName: 's3 upload',
  collectorAddress: 'localhost:11800',
});

const app = express();

app.get('/upload', (req, res) => {
    uploadFile('file-greater-than-1mb.pdf', 'hello1').then(val => {
        console.log({val})
        res.send("uploaded")
     }).catch((error) => {
        console.error({error})
        res.send("error")
     })
});

app.listen(8081, () => {
    console.log('Started on port', 8081);
});

upload.js
const fs = require('fs')
const AWS = require('aws-sdk')
require('dotenv').config()

const credentials = {
    accessKeyId: process.env.AWS_ACCESS_KEY_ID,
    secretAccessKey: process.env.AWS_SECRET_KEY,
}
const bucketName = process.env.AWS_BUCKET_NAME
AWS.config.update({ region: 'us-east-1' });

const s3 = new AWS.S3({
    credentials,
    endpoint:  'http://localhost:4566' ,
    s3ForcePathStyle: true,
 })

const uploadFile = async (sourceFilePath, fileName) => {
    // read file from file system
    const fileContent = fs.readFileSync(sourceFilePath);
    console.log({fileContent})

    const params = {
        Bucket: bucketName,
        Key: fileName,
        Body: fileContent,
    };

    try {
        await s3.putObject(params).promise();
    } catch (error1) {
        console.error({error1})
        throw error1
    }
}

module.exports = uploadFile

Anything else
docker-compose.yml to setup Localstack (local s3)
version: '3.2'
services:
  localstack:
    image: localstack/localstack:latest
    container_name: localstack_demo
    ports:
      - '4563-4599:4563-4599'
      - '8055:8080'
    environment:
      - SERVICES=s3
      - DEBUG=1
      - DATA_DIR=/tmp/localstack/data
    volumes:
      - './.localstack:/tmp/localstack'

Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

This issue title is not well writen. Please fix.

not familiar with the repository, if anyone can help, I can submit a PR

@someone-1 did you confirm that it works if you remove skywalking agent from your codes?

@kezhenxu94 application works (successful uploads file) if I remove the skywalking agent code

I tried to fixed this problem, please review. https://github.com/apache/skywalking-nodejs/pull/82
This problem seems to be related to the request header Expect.
In aws-sdk/lib/http/node.js
var stream = http.request(options, function (httpResp) {...}
...
httpRequest.stream = stream; // attach stream to httpRequest

var expect = httpRequest.headers.Expect || httpRequest.headers.expect;
if (expect === '100-continue') {
  stream.once('continue', function() {
    self.writeBody(stream, httpRequest);
  });
} else {
  this.writeBody(stream, httpRequest);
}

@someone-1 thank you for reporting this and providing the complete reproduce steps and code snippets, and thanks @wuwen5 for fixing this 🙇


Port #7837 from core strategy to plugin

Ports #7837 from core strategies to parent_select plugin.
Adds unavailable_server_retries and markdown to strategies.

Cherry-picked to v9.2.x


502s after node draining

Hi there. We have a NodePort service with alb.ingress.kubernetes.io/target-type: instance. When we drain one of the nodes this service is running pods on we get 502s once the EC2 instance is terminated. Our workaround for this is to manually delete the TargetGroup for the drained node. Is there a way to decommission a node in a HA manner? I thought that draining a node might update the target groups in the ALB?

/close


TensorFlow Hub hosted Mv1/Mv2 fails execution in tflite runtime

The mv1 and mv2 quantized models hosted here
(mv1-224 0.25,0.5,0.75,1.0 and mv2-224)
(https://www.tensorflow.org/lite/guide/hosted_models) fails execution on the tflite runtime on my custom app modified from TensorFlow's demo app, running on a Google Pixel 3a mobile.
The error was,
java.lang.IllegalArgumentException: Cannot convert between a TensorFlowLite buffer with 150528 bytes and a Java Buffer with 602112 bytes.
It seems like there's an int8 vs float32 mismatch somewhere. However, if I manually take an mv2 and use the TFLite Converter (with quantization flag) to convert to a quantized model, it runs perfectly (both the models have the same size (hosted vs converted) hence ruling any issues of Converter not quantizing the model).
Can you please let me know why the hosted model could be erring out?

Also, I am unable to generate/convert models for Mv3-224 (small and large) and the existing model fails similarly.
However, the hosted Mnas models work perfectly well

@vinodganesan Can you please explain what you meant by the following? Give us more details on what you have done taking the hosted models.

The mv1 and mv2 quantized models hosted here
(mv1-224 0.25,0.5,0.75,1.0 and mv2-224)
(https://www.tensorflow.org/lite/guide/hosted_models) fails execution on the tflite runtime on my custom app modified from TensorFlow's demo app, running on a Google Pixel 3a mobile.

My questions are

from the hosted models, did you download saved model or tflite quantized model?
which model you ported into mobile? What are the steps you followed?
what do you mean by "Also, I am unable to generate/convert (to tflite) models for Mv3-224 (small and large) and the existing model fails similarly."

Let's take one at a time and understand the issue so that we can resolve faster. Thanks!

Hi,
Please find my answer to your queries below

I downloaded the tflite quantized model,

I ported Mobilenet_V1_1.0_224_quant and Mobilenet_V2_1.0_224_quant, using an app modified from the existing tensorflow demo app.. It more or less uses the steps mentioned in this link (https://codelabs.developers.google.com/codelabs/recognize-flowers-with-tensorflow-on-android/)

For Mv3, I took the tflite quantized model provided in https://github.com/tensorflow/models/tree/master/research/slim/nets/mobilenet and tried running it, it failed too.

However, if I take the float models provided and use TFLiteConverter with OPTIMIZE_FOR_SIZE flag, I get a quantized model with similar size as of that of hosted. However, I was successfully able to run these quantized models that I converted on the app.
Let me know if that clarifies.
Thanks,
Vinod

@vinodganesan Can you please create a standalone code to reproduce the issue? what are the sizes of your model (that you converted) and the hosted model? Are they both full integer quantization or float quantization models? Thanks!


[Backport v2.5] Bump rke for fix where addon jobs are stuck in removing

Backport of #36076
Related Issue: https://github.com/rancher/rancher/issues/36037
This PR bumps the version of rke to one that includes the fix for clusters reporting as active after an upgrade when removing old addons had failed. Now, a cluster will correctly transition to the error state.

@jakefhyde , I've updated issue description to link to 2.5 backport issue and not the original one as it's how it's supposed to be.


python310Packages.pytest-testmon: 1.4.0 -> 1.4.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.pytest-testmon is: "Pytest plug-in which automatically selects and re-executes only tests affected by recent changes"
meta.homepage for python310Packages.pytest-testmon is: "https://github.com/tarpas/pytest-testmon/"
meta.changelog for python310Packages.pytest-testmon is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.4.1 with grep in /nix/store/5j2yg1la5cxinf2mwnpxzspw87faff90-python3.10-pytest-testmon-1.4.1
found 1.4.1 in filename of file in /nix/store/5j2yg1la5cxinf2mwnpxzspw87faff90-python3.10-pytest-testmon-1.4.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.pytest-testmon
python39Packages.pytest-testmon

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5j2yg1la5cxinf2mwnpxzspw87faff90-python3.10-pytest-testmon-1.4.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.pytest-testmon https://github.com/r-ryantm/nixpkgs/archive/63419f7bbaff4276299a4c8322da0a80b6125838.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5j2yg1la5cxinf2mwnpxzspw87faff90-python3.10-pytest-testmon-1.4.1
ls -la /nix/store/5j2yg1la5cxinf2mwnpxzspw87faff90-python3.10-pytest-testmon-1.4.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python310Packages.pytest-testmon
python39Packages.pytest-testmon

Maintainer pings
cc @dmvianna for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.pytest-testmon is: "Pytest plug-in which automatically selects and re-executes only tests affected by recent changes"
meta.homepage for python310Packages.pytest-testmon is: "https://github.com/tarpas/pytest-testmon/"
meta.changelog for python310Packages.pytest-testmon is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.4.2 with grep in /nix/store/w3xq5cr9gl9j6qmn4zy4c8ir5m04jfpw-python3.10-pytest-testmon-1.4.2
found 1.4.2 in filename of file in /nix/store/w3xq5cr9gl9j6qmn4zy4c8ir5m04jfpw-python3.10-pytest-testmon-1.4.2

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.pytest-testmon
python39Packages.pytest-testmon

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/w3xq5cr9gl9j6qmn4zy4c8ir5m04jfpw-python3.10-pytest-testmon-1.4.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.pytest-testmon https://github.com/r-ryantm/nixpkgs/archive/710c2a8ce7fe7259ed94f319b16f9cbd4d314710.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/w3xq5cr9gl9j6qmn4zy4c8ir5m04jfpw-python3.10-pytest-testmon-1.4.2
ls -la /nix/store/w3xq5cr9gl9j6qmn4zy4c8ir5m04jfpw-python3.10-pytest-testmon-1.4.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python310Packages.pytest-testmon
python39Packages.pytest-testmon

Maintainer pings
cc @dmvianna for testing.


added entry point to package

the main entry point in package.json allows you to install the package locally with "npm i ../flowbite-svelte"

Thanks for the catch. I appreciate your help.

Merged using the following:
git remote add fork git@github.com:gsuhm/flowbite-svelte.git
git fetch fork
git merge fork/master
git push


Upgrade to arrow 22

Which issue does this PR close?
Closes #3362.
Rationale for this change
We should keep DataFusion up to date with Arrow.
What changes are included in this PR?

precision is always u8 now
builders are in the middle of being refactored so some have new() and others with_capacity()
pyo3 needs to be upgraded to 0.17

Are there any user-facing changes?
No

Codecov Report

Merging #3363 (bb54c53) into master (4c948a3) will decrease coverage by 0.00%.
The diff coverage is 84.87%.

@@            Coverage Diff             @@
##           master    #3363      +/-   ##
==========================================
- Coverage   85.51%   85.50%   -0.01%
==========================================
  Files         294      294
  Lines       54120    54115       -5
==========================================
- Hits        46279    46272       -7
- Misses       7841     7843       +2

Impacted Files
Coverage Δ

...usion/core/src/avro_to_arrow/arrow_array_reader.rs
0.00% <0.00%> (ø)

datafusion/core/src/physical_optimizer/pruning.rs
94.75% <ø> (ø)

datafusion/core/src/test/mod.rs
83.48% <0.00%> (ø)

...sion/physical-expr/src/aggregate/count_distinct.rs
95.00% <ø> (ø)

datafusion/physical-expr/src/array_expressions.rs
50.00% <ø> (ø)

datafusion/proto/src/from_proto.rs
34.26% <20.00%> (ø)

...sion/core/src/physical_plan/file_format/parquet.rs
94.56% <33.33%> (ø)

datafusion/physical-expr/src/aggregate/median.rs
73.49% <50.00%> (ø)

datafusion/physical-expr/src/aggregate/sum.rs
69.79% <75.00%> (ø)

datafusion/common/src/scalar.rs
84.62% <77.77%> (-0.17%)
:arrow_down:

... and 29 more

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

@alamb thanks for releasing arrow 22! Hopefully this helps release the next version of DataFusion :)

CI seems broken. @alamb or @andygrove appreciate it if you could kick it please.

Okay, ready for merge. No idea why github is taking so long today...

Thanks @avantgardnerio!


cli: idl close to directly an idl address

On upgrading the IDL account
anchor --provider.cluster devnet idl \
  --provider.wallet <fee-payer> upgrade <program-id> -f ./target/idl/program.json

It creates the IDL buffer and then it writes the buffer to the program IDL account.
The IDL buffer is still opened. It's not possible to use anchor idl close with specific IDL address (e.g., address of the buffer) to close it and get back the rent for the account.
Adding the optional argument of providing the IDL address directly makes possible to close the buffer account as well.

I wonder why upgrade command doesn't close the buffer account by default, perhaps to version the IDLs? ...

To me the suggestion makes perfect sense. I would expect it should work this way to follow how the solana program cli does.
Do you thing adding the idl_close call at id_upgrade makes this working as you proposed?
https://github.com/coral-xyz/anchor/pull/2760/files#diff-c1f8f7498da827a634bddc8a7559198bc99b296e9d9e8b91a70b503662995b8cR2264

@acheroncrypto Sure. So it's a new update here in PR. Let's see.


can I get image path?

This is so nice uploader and I like it
onChange function didnt return image path but only image name
How can I get image path and image name?
Thank you.

same, I also need the image path

Well, I did this in my onDrop function:
const onDrop = (picture) => {
    // use the FileReader to decode de picture file that comes
    let reader = new FileReader();
    let url = reader.readAsDataURL(picture[0]); // for the first picture in this case

    reader.onloadend = function (e) {
      // the reader.result variable will have your url, in this case I set the pictureUrl state to that
      setPictureUrl(reader.result);
    };
  };

and I can show the image in some preview in the page like this:
<img style={{ height: "50px", width: "50px" }} src={pictureUrl} />

** I have the following hooks, just in case it was not clear:
const [pictureUrl, setPictureUrl] = useState("");


[docs] Redirection issue

The problem
All requests to resources under _next when navigating to https://mui.com/toolpad/ fail, because of – I imagine – this conflicting rewrite rule:

Ideas on solutions

Setting a custom build directory instead of .next might be a possible way to fix this => doesn't work it has no impact on the URLs.
Configuring the _next prefix => not possible https://github.com/vercel/next.js/issues/31718
Using assetPrefix => OK-ish, not awesome because it would break the standalone previews
Using basePath => doesn't work because it breaks the links. Links we can't change because not coming from this repository.

This rule is surprising to me:
/_next/* https://material-ui-x.netlify.app/_next/:splat 200

How come it doesn't conflict with the core docs? The core docs need assets from /_next/* as well, but don't they all get routed to the X docs assets according to that rule?

How come it doesn't conflict with the core docs?

@Janpot Netlify will first resolve the files that are on the filesystem:

By default, you can’t shadow a URL that actually exists within the site.

https://docs.netlify.com/routing/redirects/rewrites-proxies/#shadowing
It's only if we where doing this that it would be broken:
/_next/* https://material-ui-x.netlify.app/_next/:splat 200!

@bharatkashyap Can you put that line in https://github.com/mui/mui-x/blob/master/docs/public/_redirects?


App tries to submit invalid trips

The server is complaining that some trips have unordered or overlapping station uses. Investigate how this came to be and fix.
EDIT: this also applies to trips that are being submitted by my Zopo 910 so I can probably reproduce the problem...

Other users are also apparently managing to generate the same trip UUID multiple times. Most likely, the synced status is not being saved properly on those devices.

This should be fixed with the combination of 55a4f8c93c8b66959de3f1c0e89f4d2ff31bcbb7 and https://github.com/gbl08ma/disturbancesmlx/commit/01b7f4fa963b43606f6f67d3fe131e9d809eb6df


<EMAIL_ADDRESS>hash check failed

Installing 'groovy' (2.5.2) [64bit]
apache-groovy-binary-2.5.2.zip (28.2 MB) [==================================================================> ]  98%
Checking hash of apache-groovy-binary-2.5.2.zip ... ERROR Hash check failed!
App:         groovy
URL:         https://dl.bintray.com/groovy/maven/apache-groovy-binary-2.5.2.zip
First bytes: 50 4B 03 04 0A 00 00 08
Expected:    495f96f1be35ef838abb5f1c10fc6f642460cacd94c7095eb3a9f1fbf62b282e
Actual:      e1f2951e7d3f0be9cbde325e40000fb487ea4283d19b11696c07776cad8762dd

495f96f1be35ef838abb5f1c10fc6f642460cacd94c7095eb3a9f1fbf62b282e is correct. Please try again.

From that download screen you can see that file wasn't fully downloaded (98%) Something interupted downloading, which resulted into corrupted file (different hash)

Indeed, I tried a couple times but didn't notice that error. Sorry.
Yes, it works fine now. Thanks.


Add external research page

More here in the doc from today's meeting: https://docs.google.com/document/d/1j23Jxp62wC1qMEL0R-Osp505mIflU2pOBfCRYRa6Su4/edit

@jwal1024 could you also check for evidence linking the outcomes in those papers to longer-term ones like adult earnings and mortality?


[SPARK-18816] [Web UI] Executors Logs column only ran visibility check on initial table load

What changes were proposed in this pull request?
When I added a visibility check for the logs column on the executors page in #14382 the method I used only ran the check on the initial DataTable creation and not subsequent page loads. I moved the check out of the table definition and instead it runs on each page load. The jQuery DataTable functionality used is the same.
How was this patch tested?
Tested Manually
No visible UI changes to screenshot.

@srowen and @tgravescs could you take a quick look at this fix for a bug @yhuai found

Test build #70039 has started for PR 16256 at commit 0f5e470.

Also in case the others are on vacation, @vanzin want to take a look too?

Test build #70039 has finished for PR 16256 at commit 0f5e470.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test PASSed.

Merged to master


replace token when kratos-session-token header is present

when this PR will be merged, the JWT will no longer be the preferred way of authenticated in the app.
request to the backend will return a new kratos-session-token header when a JWT is been sent in the request. Here is the code that is doing the update.
the mobile app should then replace the JWT to the kratos-session-token as its way to authenticate further requests to the app.
From the mobile point of view, nothing else should change at this point.
During the migrations period that should last some weeks, both the JWT and the kratos-session-token will be valid.
Eventually, the JWT will not longer be valid, and user will have to relogin if they:

have not used the app while the migrations period was ongoing
have not had a recent enough version last version of the mobile app that contain the code for upgrading

this has been implemented


[vue-router] Uncaught error during transition while opening comments

Opening comments does not work anymore. Tried it in Chrome and FF. The page just reloads and in the console there is:
"[vue-router] Uncaught error during transition: "

That was caused by some Firebase API errors. Seems to be good now.


Sorting by text doesn't seem to work anymore

Hello. Is time sorting working for you??

@Medullitus , yeah, this is working fine for me.
My date/day formatting is different though, you can try the same.


Text is incorrectly offset when zoomed in

Draw a rect
Pinch to zoom
Double click its center to edit text

Expected: text is edited in the center
Actual: text is edited with an offset and then jumps to center

I can reproduce, looking into it right now


nixos/mautrix-telegram: add documentation for setting arbitrary secrets

Motivation for this change
services.mautrix-telegram.environmentFile can be used to set arbitrary options and is not limited
to the five tokens listed in the original documentation.
In my case, I used it to set bridge.login_shared_secret (as a secret, it should not be added to the store), which maps to the environment value MAUTRIX_TELEGRAM_BRIDGE_LOGIN_SHARED_SECRET.
This PR adds documentation on how to override arbitrary other settings.
This is possible because of the changes here: tulir/mautrix-telegram#332
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

@pacien might want to have a look at this, too.

@CRTified is there an upstream documentation page that explains this behavior to which we can add a link instead of giving an incomplete list of environment variables?

@doronbehar I was not able to find any documentation in the official docs, probably because this was integrated only due to the PR that I referenced when opening this PR (and that contribution came from @pacien). This would suggest that setting these options with other means is not the intended method of configuration.
There's also this "long standing" issue that some configuration options are not accessible from the environment anymore: https://github.com/mautrix/telegram/issues/584


⚠ Secret Website 2 has degraded performance

In 0a90cd1, Secret Website 2 ($SECRET_WEBSITE_2) experienced degraded performance:

HTTP code: 200
Response time: 4489 ms

Resolved: Secret Website 2 performance has improved in 0caeb8b.


Add pin configuration based on selected board

Selection a board in the Arduino IDE sets a specific define for the compiler. This should be used to select the pin configuration accordingly.
e.g. selecting "TTGO LoRa32-OLED" will set -DARDUINO_TTGO_LoRa32_V1

https://github.com/matthias-bs/BresserWeatherSensorReceiver/commit/0f13f59e9dac3d3da2f0096401c7e7b9e80fe7dc


Create a clone of multisite

Hi,
I want to create a preprod environment of my network. (clone of porduction environment)
Is it possible with your plugin or we can just export one to one the subsites ?
Regards

@alex8276 if you want to clone the whole network you don't need this plugin, you can clone the network as is and do a search-replace.
However, if you want to clone just a subset of the sites then yes, this plugin is for you.

@alex8276 Correct, move folders and export and import the database.


Removed reference to 'internal' use

Context
We state that use is intended for 'internal' use by the GDS and government community. It's unclear what we mean by this, making it potentially confusing.
Changes proposed in this pull request
Removing reference to 'internal' in the introduction.

I don't think this is really worthwhile


Question. We can use df- to default values to input fields can we do the same with div or span?

Hi,
Is it possible to default a value into a span or a div as opposed to a field?
Thanks

Hi @SensorPro
It's possible with contenteditable elements.
View https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/contenteditable
Jero

Is there a way to do it without the contenteditable attribute?
I would like my data to be readonly but it seems that setting contenteditable to false makes it not render my data.

Hi @The-Ali-Reda
If it is only to show there is this option.
editor.editor_mode = 'view'

if you can't use textarea or inputs disabled.

Hi @jerosoler ,
I meant that I wanted the data to be view only but the node itself editable.
It seems that my current option is to do something like:
var id = document.getElementById("drawflow");
const editor = new Drawflow(id);
editor.start();
var html = `

df-name

`;
var data = { "name": 'TestName' };
html = html.replace("df-name", data.name); //focus on this part
editor.addNode('github', 0, 1, 150, 300, 'github', {}, html);

I was wondering if there is a way to do it in the library by defining specific attributes instead of doing it manually as this can be tedious, inefficient (from a performance standpoint because replace would be called multiple times unless we use regexp), and error prone depending on the number of fields in the database.

Hi @The-Ali-Reda
The simplest option I can think of is, modify the contents from true to false.
    editor.on('nodeCreated', () => {
        var editable_elements = document.querySelectorAll("[contenteditable=true]");
        for(var i=0; i<editable_elements.length; i++) {
            editable_elements[i].setAttribute("contenteditable", false);
        }
    })

    editor.addNode('aaa', 0, 1, 100, 300, 'aaa', { name: 'test'}, `<div contenteditable="true" df-name>` );

Hi @jerosoler ,
That worked. Thank you!
I was wondering is there any particular reason why the library requires contenteditable to be true during add time? (line 1249)

Yes with conteneditable require true for df- attributes synchronize.


Can't create one archive per album

I can't figure out how to get it working?

nvm figured it out


[Issue] Azure.yaml - terraform is not a valid value

[X] Make sure you've installed the latest version using instructions in the wiki

Output from azd version
azd version 0.8.0-beta.1 (commit 08209da7c22d28c6a123ef4e88d433132066b800)
Describe the bug
The JSON schema backing the azure.yaml (https://raw.githubusercontent.com/Azure/azure-dev/main/schemas/v1.0/azure.yaml.json)was changed with version 0.8.0 of azd restricting the allowed values for the infra.provider to bicep.
Although the provisoning via Terraform became a alpha feature that needs to be explicitly enabled, the JSON schema should either allow this value or a new version of the JSON schema should be created that is shipped with the templates.
To Reproduce

Execute azd init -t Azure-Samples/todo-nodejs-mongo-terraform
Open the cloned project in VSCode
Open the azure.ymal file

=> you will find a red curly line underlining the provider value terraform
Expected behavior
As mentioned above:

Either the Terraform provider is a valid value again for the enum in the infra section of the azure.yaml
A new version of the JSON schema is released that is not containing Terraform. However the original version should still contain the Terraform as a valid value and should then be referenced by the "old" Terraform templates

Environment
n/a
Additional context
n/a

@savannahostrowski @ellismg
Since being in active development and is prone to change in future releases - https://learn.microsoft.com/en-us/azure/developer/azure-developer-cli/feature-versioning
Similar to springapps which isn't also called out as part of azure.yaml. We will be following the same for terraform too? Also, we don't have a alpha/dev yaml published today. There could be implications in other areas of the system w.r.t to a new alpha/dev yaml?
Is this a GA blocker? Can we handle post GA?

I think the best thing for us to do now is to create another copy of this schema which tracks all the features supported when all the alpha flags are on.  We can use this as a place to document the alpha features without worrying about confusing the existing schema consumers.  I opened #2251 to do this.
Folks that want intellisense for the alpha features can update the URL to point at the alpha schema after we check it in.


refactor: improve CI workflow and package configuration

Description

Review

[ ] I self-reviewed this PR

Testing

[ ] I added/updated tests
[ ] I manually tested

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


WebDriverConnect.initialize_web_driver and AppiumServer.running? do not support Appium 2 default server base path

WebDriverConnect.initialize_web_driver and AppiumServer.running? use hard-coded Appium version 1.x default server base path with the /wd/hub base path legacy convention. Both methods should be refactored to support Appium version 2.x default server base path without the /wd/hub base path convention.

Fixed


Added tool requirements

Added a list of requirements that need to be installed before the build script will work.
Closes #73

Merged, thanks @peterbarrow.


[5.5] username() to identity().

Rename function 'identity()' instead of username().
Best regards.

is this purely semantic? if so 👎


When a soft error occurs continue onto the next dep file.

Moves the code from a try block to a separate function to make
expressing this cleaner, as early exits in try blocks exit the outer
function.
Also converts another try block to a smaller & cleaner style.

(See #64 for discussion on try blocks; I was just examining that when I noticed the early exit bug - at least I consider it a bug : soft errors should not invalidate other work, and this particular soft error path results in throwing away all the deps, not just the one missing file.

Are you suggesting to remove all try expressions, or all unstable features?

Are you suggesting to remove all try expressions, or all unstable features?

The discussion in that issue seemed to say that removing all unstable features was doable, but that among them were ~30 try blocks that would need removing, so I thought to start somewhere.

(See https://github.com/facebook/buck2/issues/64 for discussion on try blocks; I was just examining that when I noticed the early exit bug - at least I consider it a bug : soft errors should not invalidate other work, and this particular soft error path results in throwing away all the deps, not just the one missing file.

If you're missing one dep file, you can't just assume it's empty (which is what the proposed code does) You'll end up missing dependencies that were listed in this dep file.
The early exit is not a bug, it's the only correct thing to do besides failing (which is what we'd actually want to do, and which the OSS build of Buck2 does, but which due to interaction with other tools we haven't been able to do).

Thanks @krallin, I'll close this, and keep the existing behaviour when I get a bit more time to poke at the try block stuff.


Transient workflow failures from javax.net.ssl.SSLHandshakeException: PKIX path validation failed

Started noticing CI failures from https://github.com/googleapis/gapic-generator-java/pull/1557 presubmits that do not look related to the changes made. Opening this issue to track and investigate.
Failing workflow run: https://github.com/googleapis/gapic-generator-java/actions/runs/4515219935/attempts/2
Latest successful workflow run: https://github.com/googleapis/gapic-generator-java/actions/runs/4514393539
[INFO] --- download-maven-plugin:1.6.8:wget (download-metadata-proto) @ gapic-generator-java ---
Warning:  Could not get content
javax.net.ssl.SSLHandshakeException: PKIX path validation failed: java.security.cert.CertPathValidatorException: validity check failed
    at sun.security.ssl.Alert.createSSLException (Alert.java:131)
    at sun.security.ssl.TransportContext.fatal (TransportContext.java:353)
...
Error:  Failed to execute goal com.googlecode.maven-download-plugin:download-maven-plugin:1.6.8:wget (download-metadata-proto) on project gapic-generator-java: IO Error: Could not get content -> [Help 1]

Looks like this might have been a transient error - keeping this issue open to track for a bit, we can close if it doesn't continue to show up.

@emmileaf One of the step prints Maven version via "-V" option. Are they different between success and failure?

Oh, I completely missed this - thank you for the pointer @suztomo! No, there is no difference in the maven versions.
From the previously failed run:
> Run mvn -V -B -ntp clean install -DskipTests
Apache Maven 3.[8](https://github.com/googleapis/gapic-generator-java/actions/runs/4515219935/jobs/7952334215#step:4:9).8 (4c87b05d[9](https://github.com/googleapis/gapic-generator-java/actions/runs/4515219935/jobs/7952334215#step:4:10)aedce574290d1acc98575ed5eb6cd39)
Maven home: /usr/share/apache-maven-3.8.8
Java version: 11.0.18, vendor: Eclipse Adoptium, runtime: /usr/lib/jvm/temurin-11-jdk-amd64
Default locale: en, platform encoding: UTF-8
OS name: "linux", version: "5.15.0-[10](https://github.com/googleapis/gapic-generator-java/actions/runs/4515219935/jobs/7952334215#step:4:11)34-azure", arch: "amd64", family: "unix"

Thank you for checking. It's a nice mystery.

By Monday other people may face similar errors if it's widespread problem. Google the error message.
Create a minimum reproducible job
See whether you can get the error message in local computer. Diego recently found a nice tool to run GitHub Actions.

Closing since this transient issue was resolved and no longer reproducible.


Not possible to browse into some folders

When browsing some folders in filemin i.e. subfolders within  home/files$ I get the error:  home/files$/home/files$/stuff No such file or directory
I presume it is the $ causing the issue when drilling down to subfolders within the folder? All other folders access are fine. The $ is used to hide the share on windows systems and although I could just map these differently in Samba I prefer to have the folder structures the same.
Regards
Dave

Yep, $ sign is usually used in variables names in Perl, I think you're right and it causes the trouble. I'll look into how can this be avoided, so you can have $ in file/folder names.

I think I see the cause of this - it was due to incorrect escaping of file paths. I've just checked in a fix.

I think I see the cause of this - it was due to incorrect escaping of file paths. I've just checked in a fix.

Thanks, Jamie :)

On 01/01/2016 01:32, Jamie Cameron wrote:

I think I see the cause of this - it was due to incorrect escaping of
file paths. I've just checked in a fix.
—
Reply to this email directly or view it on GitHub
https://github.com/Real-Gecko/filemin/issues/82#issuecomment-168273246.

Hi Jamie
I saw the update 1.780 and now I can browse into a folder with a $
character in it but further browsing into sub folders the link is
broken, it shows as:
https://<IP_ADDRESS>:10000/filemin/index.cgi?path=/home/apps2$/home/apps2$/utils
whereas the link path should be
https://<IP_ADDRESS>:10000/filemin/index.cgi?path=/home/apps2$/utils
Regards
Dave
--
..........................
Dave Edwards (G7RAU)
email:
<EMAIL_ADDRESS>Web:
http://g7rau.demon.co.uk/
..........................

I just tested this with Webmin 1.780, and I could browse into subfolders just fine.

Fixed.

On 10/01/2016 18:10, Jamie Cameron wrote:

I just tested this with Webmin 1.780, and I could browse into
subfolders just fine.
—
Reply to this email directly or view it on GitHub
https://github.com/Real-Gecko/filemin/issues/82#issuecomment-170376018.

Hi Jamie
Rebooted the server here after making sure everything was up to date and
still the same issue, a duplication of the path in the link querystring.
I have changed my folder names to remove the $ now as I can still have
the $ in the samba config folder name but maybe some others will still
have this issue. I have left one folder in case you want any screenshots
for debugging etc.
Cheers
Dave
--
..........................
Dave Edwards (G7RAU)
email:
<EMAIL_ADDRESS>Web:
http://g7rau.demon.co.uk/
..........................

Can you post the screenshot?

On 17/01/2016 16:31, Real-Gecko wrote:

Can you post the screenshot?
—
Reply to this email directly or view it on GitHub
https://github.com/Real-Gecko/filemin/issues/82#issuecomment-172347223.

Screenshot attached.
--
..........................
Dave Edwards (G7RAU)
email:
<EMAIL_ADDRESS>Web:
http://g7rau.demon.co.uk/
..........................

I see none.

On 17/01/2016 16:50, Real-Gecko wrote:

I see none.
—
Reply to this email directly or view it on GitHub
https://github.com/Real-Gecko/filemin/issues/82#issuecomment-172349655.

OK, must have been removed. file link:
http://g7rau.demon.co.uk/temp/filemin.jpg
The mouse not in the screenshot but the link path on the bottom showing
the duplication.
Cheers
Dave
--
..........................
Dave Edwards (G7RAU)
email:
<EMAIL_ADDRESS>Web:
http://g7rau.demon.co.uk/
..........................

Version 1.0.0 is out. Now you can put "thousands of $$" into you file names :D Enjoy.

On 25/01/2016 07:10, Real-Gecko wrote:

Version 1.0.0 is out. Now you can put "thousands of $$" into you file
names :D Enjoy.
2
https://cloud.githubusercontent.com/assets/2231969/12544506/f2d37064-c364-11e5-9554-e2cadf57faa9.png
—
Reply to this email directly or view it on GitHub
https://github.com/Real-Gecko/filemin/issues/82#issuecomment-174422078.

Many thanks. I have left my spare drive mounted as /home/games$ to test.
FYI, when browsing around other folders I noticed another char, the hash
"#" is removed from the path and therefore the link fails. For me this
is not an issue but might be for others. I never created this folder, it
was created in a c# conversion option for some projects in a csharp
application so maybe it might be reported in the future by others.
Cheerio
Dave
--
..........................
Dave Edwards (G7RAU)
email:
<EMAIL_ADDRESS>Web:
http://g7rau.demon.co.uk/
..........................

OK, keep in mind issue about '#' sign and reopen issue to fix it in next release.

Fixed in 1.1.0


Reorder vim arguments in launchEditor so --remote works

This allows you to set REACT_EDITOR to 'vim --remote', so the file can
be opened in an already running vim process. When using vim without
remote, it works in the same way as before.
When launching vim without --remote, the order of the line and path
arguments doesn't matter. However, when using --remote the line argument
has to precede the path. This happens to be the same as joe and emacs
uses, so the vim cases were just moved there.
I haven't tested this with mvim, but the documentation says the same as
the vim documentation, so I assume it works the same.

Thanks! I haven't tested but trust you this works.

Hi there! This change is out in<EMAIL_ADDRESS>please give it a go! Thanks.


Prevent scrolling background body

Fixes pinch-zoom and iframe/inline scrolling on iDevices
Creates more width on desktop
Prevents background scrolling

Thanks!


Throw if useAuthUser is used without withAuthUser

Is your feature request related to a problem? Please describe.
If a developer accidentally uses useAuthUser without withAuthUser, it won't work: useAuthUser will always return an unauthed user. This is confusing: #150
Describe the solution you'd like and how you'd implement it
Throw an error if useAuthUser is called without withAuthUser above it in the component tree.
Implementation: don't set a default AuthUser value when defining the context, and in the useAuthUser function, throw if the value is undefined.
Is this a breaking change?
Yes, somewhat. If a developer is using this incorrectly, upgrading would throw a new error.
Describe alternatives you've considered

Leave as-is, though I don't see the value of useAuthUser without a context provider
Warn rather than throw

I have a question about this:
I have a header component which renders different UI whether or not AuthUser is defined or not. For example if AuthUser exists, the header will have a logout button with the handling logic within that Header component. This header component is imported in every page which needs it.
Now in this case, I want to use UseAuthUser directly in the Header component, however since I want to use the Header component also on pages without the withAuthUser HOC (for example the welcome page), this is a case where I actually use useAuthUser in a component (the header) without the withAuthUser HOC for the page which imports this component. And the useAuthUser in this case returning undefined would be intended, because I would render a different UI for that case...

@lazlothemonkey Good question. I'd recommend one of the following approaches for your header scenario, depending on what you need:

Use withAuthUser on all pages. The only overhead here is loading the Firebase JS SDK.
Pass the user info as a prop to your header component from each page that has withAuthUser, like the example app does. (There's no real reason the example app does it this way. I just decided it would be a presentational component.)
Create separate header components for pages with and without authed user info. You already know in advance which pages use which, because they're the ones using withAuthUser.

Thanks, I think I'll go with option 3, so that I don't have one huge component with an if else block containing two different UIs

Closed in #395.

@kmjennison What about static hoisting?
https://github.com/gladly-team/next-firebase-auth/pull/237

@adithep Could you clarify what you mean? If you believe there's a bug, please open a new bug issue with a reproduction. Thanks!


Initialize ChequeForm and ChequePreview content region at ChequeModule

Separate two content region for two view (ChequeFormView and ChequePreviewView)
Move the related code in ChequeView to their respective views
The purpose for this changes are making responsive view by constructing their layout in a user control.

Pending for moving the respective codes:

Form control -> ChequeFormView
Canva control -> ChequePreviewView

I have to discuss with @Yawhou for these changes.


Bug 4927 Value Error

Description

After looking into the problem it became clear that the issue was not caused by pandas but by the data given to it. For the specific issue linked to this PR (#4927), in the set of indicator data points being converted to a DataFrame there was a property called PercentB for the BollingerBand indicator, this property had repeated index values after the date 4/18 which caused the Value Error.
This property uses CompositeIndicator which was found to not fill forward properly and update the properties time. With these changes, if the conditions to update the CompositeIndicator are not met, it now creates a clone of the last point with the latest time. This way the indicator current.Time is always up to date.
Related Issue

Closes #4927
Motivation and Context

Example research was breaking, ended up being unrelated to research but still required fixing.
Requires Documentation Change

N/A
How Has This Been Tested?

All tests ran and passed.
Research notebook is working again.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] Refactor (non-breaking change which improves implementation)
[ ] Performance (non-breaking change which improves performance. Please add associated performance test and results)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Non-functional change (xml comments/documentation/etc)

Checklist:

[x] My code follows the code style of this project.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] My branch follows the naming convention bug-<issue#>-<description> or feature-<issue#>-<description>

@C-SELLERS I have merged your changes to my own fork and am still seeing the same error as #4927 in my notebook. I gather this would be because the base image isn't updated, if I want to spin my own image is this as simple as building from this Dockerfile with the updated code?
Thanks

Hey Athon! Yeah you will have to build the new image for it to work, you'll have to do two actually because the research image builds off of the Lean image.
So first build the Linux version of Lean; you can use "/p:ForceLinuxBuild=true" if on windows; then build the Lean Docker image with this DockerFile, then finally build the research image with this DockerFile but be sure to the FROM statement to the new Lean image you created in the previous step.
Then you should be good to go! If using the launch scripts make sure you enter your custom image name!

Closing for now, see https://github.com/QuantConnect/Lean/pull/4944#discussion_r530461527


Cannot access local variable 'duplicate_msg' where it is not associated with a value (RC1)

Hi Gary, good afternoon. Noticed this error today. I upgraded earlier today to RC1.
This error originated from a custom integration.
Logger: icloud3
Source: custom_components/icloud3/support/start_ic3.py:1797
Integration: iCloud3 v3: iDevice Tracker (documentation, issues)
First occurred: 16:15:36 (2 occurrences)
Last logged: 16:15:37
cannot access local variable 'duplicate_msg' where it is not associated with a value
Traceback (most recent call last):
File "/config/custom_components/icloud3/support/start_ic3_control.py", line 213, in stage_4_setup_data_sources
start_ic3.setup_tracked_devices_for_iosapp()
File "/config/custom_components/icloud3/support/start_ic3.py", line 1797, in setup_tracked_devices_for_iosapp
f"{CRLF_INDENT}{device_info_by_iosapp_devicename[iosapp_devicename]}{duplicate_msg}")
^^^^^^^^^^^^^
UnboundLocalError: cannot access local variable 'duplicate_msg' where it is not associated with a value

Thanks. Already fixed. It from a routine that displays an error if you have duplicate iOS app devices. I thought I push an update today but want to get the HA service call support done first.
————————
Gary Cobb
On Aug 7, 2023 at 3:22 PM -0400, marceloaf @.***>, wrote:

Hi Gary, good afternoon. Noticed this error today. I upgraded earlier today to RC1.
This error originated from a custom integration.
Logger: icloud3
Source: custom_components/icloud3/support/start_ic3.py:1797
Integration: iCloud3 v3: iDevice Tracker (documentation, issues)
First occurred: 16:15:36 (2 occurrences)
Last logged: 16:15:37
cannot access local variable 'duplicate_msg' where it is not associated with a value
Traceback (most recent call last):
File "/config/custom_components/icloud3/support/start_ic3_control.py", line 213, in stage_4_setup_data_sources
start_ic3.setup_tracked_devices_for_iosapp()
File "/config/custom_components/icloud3/support/start_ic3.py", line 1797, in setup_tracked_devices_for_iosapp
f"{CRLF_INDENT}{device_info_by_iosapp_devicename[iosapp_devicename]}{duplicate_msg}")
^^^^^^^^^^^^^
UnboundLocalError: cannot access local variable 'duplicate_msg' where it is not associated with a value
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thanks. Already fixed. It from a routine that displays an error if you have duplicate iOS app devices. I thought I push an update today but want to get the HA service call support done first. ———————— Gary Cobb
…
On Aug 7, 2023 at 3:22 PM -0400, marceloaf @.>, wrote: Hi Gary, good afternoon. Noticed this error today. I upgraded earlier today to RC1. This error originated from a custom integration. Logger: icloud3 Source: custom_components/icloud3/support/start_ic3.py:1797 Integration: iCloud3 v3: iDevice Tracker (documentation, issues) First occurred: 16:15:36 (2 occurrences) Last logged: 16:15:37 cannot access local variable 'duplicate_msg' where it is not associated with a value Traceback (most recent call last): File "/config/custom_components/icloud3/support/start_ic3_control.py", line 213, in stage_4_setup_data_sources start_ic3.setup_tracked_devices_for_iosapp() File "/config/custom_components/icloud3/support/start_ic3.py", line 1797, in setup_tracked_devices_for_iosapp f"{CRLF_INDENT}{device_info_by_iosapp_devicename[iosapp_devicename]}{duplicate_msg}") ^^^^^^^^^^^^^ UnboundLocalError: cannot access local variable 'duplicate_msg' where it is not associated with a value — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: @.>

@gcobb321 I have the same issue.... I assume it not have any impact on the functionality of the integration?

This error occurs during startup up when it is displaying a message about the iosapp devices.  I will be checking one thing dealing with short interval location updates over the next few hours and probably post an update after I check it out. The update will include support for service calls using the Developer Tools > Services.


Change HEADER["FROM" "TO"]

I need to change the "FROM" and "TO" strings on top of DatePicker, such that I am using multilanguage app. Which string I have to define in strings.xml file in order to change this headers.

I think you are referring to this library:
https://github.com/borax12/MaterialDateRangePicker


JavaScript code generation for methods

Todos

[x] Make it work on basic examples.
[x] Translate nullary methods to get property. Because we didn't implement calling nullary methods so far.
[ ] Need more tests.
[x] Inheritance.
[x] Method override.

Examples
Input
class Box[T]: { inner: T }
  method Map f = Box { inner = f this.inner }

def Box value = Box { inner = value }

def box1 = Box 1
def box2 = box1.Map (fun x -> add x 1)

Output
val Box: 'a -> box & {inner: 'a} = (value) => new Box({ inner: value })
val box1: box & {inner: 1} = Box { "inner": 1 }
val box2: box & {inner: int} = Box { "inner": 2 }

Generated
class Box {
  constructor(fields) {
    this.inner = fields.inner;
  }
  Map(f) {
    return new Box({ inner: f(this.inner) });
  }
}
const Box0 = (value) => new Box({ inner: value });
const box1 = Box0(1);
const box2 = box1.Map((x) => x + 1);

Cool! Now try it with inheritance and overriding :^D
Tip: you can just ignore method signatures in the source (those declarations of the form Mtd: <some type>).

Technical note: for this one, let's wait until https://github.com/hkust-taco/mlscript/pull/22 is merged and squashed (Cc @fo5for). Then we can merge directly into the main mlscript branch. But the PR is fine as it is right now.

Test Cases
Translate Nullary Methods into Getters
Input
class Box[T]: { inner: T }
  method Map f = Box { inner = f this.inner }
  method Get = this.inner

def Box value = Box { inner = value }

def boxOne = Box 1
def one = boxOne.Get
def boxTwo = boxOne.Map (fun x -> add x 1)
def two = boxTwo.Get

Output
val Box: 'a -> box & {inner: 'a} = (value) => new Box({ inner: value })
val boxOne: box & {inner: 1} = Box { "inner": 1 }
val one: 1 = 1
val boxTwo: box & {inner: int} = Box { "inner": 2 }
val two: int = 2

Inheritance
Input
class Box[T]: { inner: T }
  method Map f = Box { inner = f this.inner }
  method Get = this.inner

class DecoratedBox[T]: Box[T] & { motif: string }
  method GetBox: Box[T]
  method GetBox = this
  method GetMotif = this.motif
  method Decorate motif = DecoratedBox {
    inner = this.inner;
    motif
  }

def flowerBox = DecoratedBox { inner = 42; motif = "flower" }
def grassBox = flowerBox.Decorate "grass"

def FLOWER = flowerBox.GetMotif
def GRASS = grassBox.GetMotif

Output
val flowerBox: decoratedBox & {inner: 42, motif: "flower"} = DecoratedBox { "inner": 42, "motif": "flower" }
val grassBox: decoratedBox & {inner: 42, motif: "grass"} = DecoratedBox { "inner": 42, "motif": "grass" }
val FLOWER: string = "flower"
val GRASS: string = "grass"

Hmm, I should implement error in generated code. Maybe in another PR?

Great, looking good!

I should implement error in generated code. Maybe in another PR?

Yeah, it should probably be implemented as throwing an exception. Let's do it as part of this PR.

Also, let's merge this first, and soon we'll add proper JS code-gen testing and revisit the tests you described above.

@chengluyu  This branch needs to be rebased on mlscript and remove all commits before 7ad32d4 (these commits were squashed in the previous PR). Let me know if you need help doing that. You can look into git rebase -i (interactive rebase).

@chengluyu Actually since there are many conflicts, it would be easier to hard-reset this branch to the current HEAD of mlscript and then git-cherrypick your commits back one by one.

[...] it would be easier to hard-reset this branch to the current HEAD of mlscript and then git-cherrypick your commits back one by one. Then force-push the new history. [...]

I proposed a lot of changes to JSBackend.scala in PR #29. How about do this after when that PR was merged?

BTW, I have an idea of merging multiple function defs into one. For example, we can translate
def f { x = 0 } = 1
def f { x = 1 } = 0

into
const f = (x) => {
  if (x === 0) { return 1; }
  else if (x === 1) { return 0; }
  else { /* unreachable */ }
}

I have an idea of merging multiple function defs into one.

But this is not in line with MLscript semantics, which uses shadowing. In MLscript, f as defined above will always return value 0.

To be more specific, what would you do in this case, for instance?
def f 0 = "oops"
def f x = x

f 0 + 1 // valid in MLscript


Our gem for datatables-rails is dead

Looks like this gem: https://github.com/rweng/jquery-datatables-rails/tree/master/app/assets/javascripts/dataTables
is not updated for Foundation 5. So it doesn't use the XY grid.
Some alternatives:

Start using something like Yarn or NPM. Will this work on Heroku? Is this a pain?
Start using their CDN instead. Will this slow things down?

I'm leaning toward the CDN instead of Yarn or NPM.

In particular, we need to convert datatables to a more responsive model, and the latest doesn't really have this.
I spent some time on this, but I could use some more effort before we make a decisions.

Ok I got somewhere just now. Yarn will be a good choice I think. We'll need to port dataTables to the latest version and whatnot, but we can get there. Check out the yarn branch for my work on this.


attach files to email without saving content to local file

I have some binary content and would like to attach to email
the simple way is:
  content = <<binary-content>>
  File.write("/tmp/myfile.pdf", content )
  data = new_email()
      |> to(email)
      |> from( "email.com")
      |> subject("subject")
      |> put_attachment("/tmp/myfile.pdf")
      |> put_header("Reply-To", "email.com")
      |> html_body(html_body)
  File.rm("/tmp/myfile.pdf", content )

is there a way attach files without saving them locally?
something like:
  content = <<binary-content>>
  data = new_email()
      |> to(email)
      |> from( "email.com")
      |> subject("subject")
      |> put_attachment(content)
      |> put_header("Reply-To", "email.com")
      |> html_body(html_body)

I found the ability to do this via digging in the source, would people be cool with it if I updated the source docs to show the %Bamboo.Attachment{} struct ability to do this, and add a blurb on it to the started README?

That'd be wonderful!

This should work:
  content = <<binary-content>>
  data = new_email()
      |> to(email)
      |> from( "email.com")
      |> subject("subject")
      |> put_attachment(%Bamboo.Attachment{filename: "filename", data: content})
      |> put_header("Reply-To", "email.com")
      |> html_body(html_body)

@joeyrosztoczy Are you still interested in providing some documentation around this suggestion? If for, can you open a pull request with your recommended changes?

This seems to be documented on https://hexdocs.pm/bamboo/Bamboo.Email.html#put_attachment/2. I'm not sure if that got added before or after this issue. But I'm going to close this since it seems resolved.
If I misunderstood and we should reopen this, please let me know! I'd be happy to do it.


@vite/plugin-legacy bug with buildPolyfillChunk and const type variable

Describe the bug
I try build my project with vite 3.
I use @vitejs/plugin-legacy for support old browsers.
my vite.config.js file is
import { resolve } from 'path';
import { defineConfig } from 'vite';
import vue from '@vitejs/plugin-vue';
import legacy from '@vitejs/plugin-legacy';

export default defineConfig({
  base: '/',
  build: {
    rollupOptions: {
      input: {
        main: resolve(__dirname, 'pages/index.html'),
        about: resolve(__dirname, 'pages/about/index.html'),
      },
    },
  },
  server: {
    port: 3000,
    host: '<IP_ADDRESS>',
    hrm: true,
  },
  plugins: [
    vue(),
    legacy({
      targets: ['defaults', 'not IE 11'],
    }),
  ],
});

I run script yarn build but i have a problem
System Info
error during build:
Error: Transform failed with 1 error:
assets/polyfills-legacy.a7649e24.js:2104:3312: ERROR: Transforming const to the configured target environment ("es5" + 2 overrides) is not supported yet
    at failureErrorWithLog (/Users/aleksandreremeev/1bit/bitrix-vite-template/node_modules/esbuild/lib/main.js:1624:15)
    at /Users/aleksandreremeev/1bit/bitrix-vite-template/node_modules/esbuild/lib/main.js:1413:29
    at /Users/aleksandreremeev/1bit/bitrix-vite-template/node_modules/esbuild/lib/main.js:678:9
    at handleIncomingPacket (/Users/aleksandreremeev/1bit/bitrix-vite-template/node_modules/esbuild/lib/main.js:775:9)
    at Socket.readFromStdout (/Users/aleksandreremeev/1bit/bitrix-vite-template/node_modules/esbuild/lib/main.js:644:7)
    at Socket.emit (events.js:400:28)
    at addChunk (internal/streams/readable.js:293:12)
    at readableAddChunk (internal/streams/readable.js:267:9)
    at Socket.Readable.push (internal/streams/readable.js:206:10)
    at Pipe.onStreamRead (internal/stream_base_commons.js:188:23)

Used Package Manager
yarn
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I also encountered this problem, my project failed to build since yesterday. After looking into it I find that maybe cause by systemjs's 6.12.2 release. It include a "const" statement in "dist/s.min.js" which imported by @vite/plugin-legacy.
So we may need wait for its fix or use "resolution" or "overrides" field of package manager as a temporary workaround. Maybe @vite/plugin-legacy should pin his dependencies version?

i use<EMAIL_ADDRESS>it's has same problem

use  yarn create vite my-app --template react to create  a new empty  project that can reproduce the problem
just by default  option
// vite.config.js
import legacy from '@vitejs/plugin-legacy'

export default {
  plugins: [
    legacy({
      targets: ['defaults', 'not IE 11']
    })
  ]
}

I also encountered this problem, my project failed to build since yesterday. After looking into it I find that maybe cause by systemjs's 6.12.2 release. It include a "const" statement in "dist/s.min.js" which imported by @vite/plugin-legacy.
So we may need wait for its fix or use "resolution" or "overrides" field of package manager as a temporary workaround. Maybe @vite/plugin-legacy should pin his dependencies version?

i use<EMAIL_ADDRESS>it's has same problem

I set externalSystemJS to true resolved this problem

Can be reproduced by pnpm create vue@2 vite-issue-9618 --default

@patak-dev @bluwy Thanks for quick fix! But can you check if this fix will break the compatibility of es2015 unsupported browser. Since it skip the transform of  polyfills chunk, it will left es2015 syntax to output


update akka

After updating to the latest PlayFramework I was getting a strongly worded message that akka-remote was lagging behind.  Turns out it is included from my use of akka-kyro-serialization.
[warn] a.u.ManifestInfo - Detected possible incompatible versions on the classpath. Please note that a given Akka version MUST be the same across all modules of Akka that you are using, e.g. if you use [2.5.17] all other modules that are released together MUST be of the same version. Make sure you're using a compatible set of libraries.Possibly confl
icting versions [2.5.17, 2.4.12] in libraries [akka-protobuf:2.5.17, akka-actor:2.5.17, akka-slf4j:2.5.17, akka-remote:2.4.12, akka-stream:2.5.17]

If anyone happens to just need an akka update workaround:
"com.github.enalmada" %% "akka-kryo-serialization" % "0.5.3"

has been updated meanwhile


Dispatch truncates jest output

Hey Guys,
I admit that I might be using this wrong. However, I have a quick question,
Is there any way to get the plugin to show error outputs when it fails?
Sometimes Jest fails because of a syntax error or typescript error, and all I get in the output is this:

When running in the terminal by using jest , I get this:

As you can see, the terminal output is much more helpful. I'm sure it's a setting I'm missing somewhere. Can someone shed a little light on this?

Hi Raphael
I'm not too familiar with jest but I get failure output whenever I run rspec (ruby test framework) or cypress.
Here's my configuration for neovim:
-- vim-test

vim.api.nvim_set_var(
    "test#strategy",
    {
        nearest = "basic",
        file = "dispatch",
        suite = "dispatch_background"
    }
)

vim.api.nvim_set_var("test#ruby#rspec#executable", "bundle exec rspec")

vim.api.nvim_set_keymap("n", "<Leader>x", ":TestNearest<CR>", default_map_opts)
vim.api.nvim_set_keymap("n", "<Leader>t", ":TestFile<CR>", default_map_opts)
vim.api.nvim_set_keymap("n", "<Leader>r", ":TestSuite<CR>", default_map_opts)
vim.api.nvim_set_keymap("n", "<Leader>e", ":TestLast<CR>", default_map_opts)
vim.api.nvim_set_keymap("n", "<Leader>l", ":TestVisit<CR>", default_map_opts)

Looking at the executable that gets run https://github.com/vim-test/vim-test/blob/master/autoload/test/javascript/jest.vim#L39-L45 it should just write to stdout in the same way you're expecting in the terminal
So assuming you have node_modules/.bin/jest in the current repository you're running tests in, I'd expect to see vim-test running with node_modules/.bin/jest
I'd also pay attention to your test strategy (where your tests run) and see if you get the correct output trying a different strategy

I think the problem is the dispatch test strategy, I really want the quickfix so that I can navigate the test output with vim itself, however I think that dispatch attempts to parse the output into a quickfix navigatable list. How can I set it to just post the output?

I haven't used the Dispatch strategy myself however in order to get it to populate the quickfix properly you will need a compiler for your test runner. I'd recommend taking a look at the solutions in #331

Thanks guys, I found a solution.
I just bound terminal testing to a a chord.
nnoremap <leader>td exec '! npx jest '.expand("%")

if the test fails to run I run the 'test debug' binding for a more verbose output.


config: fix unmarshalling of secret URLs

Closes #1659

Sure, this is done.

Cool, thanks for the quick follow up. Leaving any last comments to @stuartnelson3.


fix(Error): use setPrototoypeOf if available for Error.

i abandoned this PR since it did not solve all of the issues.