[Android] Configure download url of crosswalk library apk

When crosswalk app running on shared mode, there's a built-in updater to help to download and update crosswalk library apk. By default, the updater will get library apk from Google Play Store.
By this feature, the developer of crosswalk app could self-deploy library apk according to their own requirements, and specify the download url of apk in the app. The updater will fetch the library apk using android download manager.
Setup Method 1: define a meta-data item with the name of "xwalk_apk_url" under application tag in AndroidManifest.xml. For example,
<application android:name="org.xwalk.core.XWalkApplication">
    <meta-data android:name="xwalk_apk_url"
               android:value="http://host/XWalkRuntimeLib.apk" />
    ...
</application>

Setup Method 2: use --xwalk-apk-url option for make_apk.py. For example,
make_apk.py --xwalk-apk-url=http://host/XWalkRuntimeLib.apk ...

@halton @fujunwei @axinging

LGTM

Does the downloaded APK need to be signed with the Crosswalk store key, or can it be anything? How do we ensure that no malicious crosswalk is loaded by an application installed from the store?

In the security check implementation, there's an option "verify_xwalk_apk" for building. If this option is enabled, the app will check the signature of package before loading crosswalk library.
In this case, the apk need to be signed with the crosswalk store key as well. We verify the download apk with the same way of verifying crosswalk from play store, and don't differentiate where the apk came from.

@rakuco

lgtm

@lincsoon ok, thanks for the explanation. @terriko you may want to have a look too.

Because this patch is based on shared mode implementation, and the PR of shared mode need to be reverted due to a packing problem, please hold the merging of this PR.

Please note you need to rebase your branch, as 4ac738e shouldn't be part of this PR.

This patch is not ready to merge, still need refine and rebase

@rakuco I added manifest support. Now this patch is ready.

lgtm, thanks


Use this knowledge of Camisole to try to write "bull-proof multi-language script launcher" for my teaching activites

I want to write a wrapper script like run-camisoled, that can read a file in Python/OCaml/C, and safely pass it to Camisole VM, and pretty-print its JSON results!

For my teaching next year this would be veryyy useful!.

Overview of the goal

Imagine you gave a Python exam to a large group of student (15-400).
You don't have any automated tests, you'll basically have to run every file, and then open it to add some comments, and finally grade it.
Students are expected to include examples and tests in their files, to save their figures to local image files, etc.
Oh, and at least one student has written a very evil program that will read your entire home folder, compress it and send it over the network to his Dropbox folder, then will delete and burn your laptop to ashes. :bomb: one of your student is a secret Nazi hacker!

So the setup is like this:
$ ls TP1_etudiants/
Etudiant1.py Etudiant2.py ...

To avoid such catastrophy, the solution would be
$ run-camisoled EtudiantHackeur.py
print output of this script, but ran from a FULLY secure isolated environment!

I want to have this "multi-language" launcher script that can execute Python3, OCaml 4.05+, C11 (and more) code, in a safe and secure environment.
A few reasons for that:

I should assume that every file submitted by a student is a very dangerous virus, and any attempt to execute/interpret/compile it locally will make my computer explode ;
Every student's file can generate a few figures, or temporary files, and I don't want to have ~139 figures and ~153 temporary files after having executed 50 Python scripts in the same folder!

one  easy solution is to move each file to its sub-folder!

I don't want to think about which languages it is written in, because for the same exam some students will have the choice between C, OCaml and Python!
I want to keep a local database, in every folder like this, for each student at each time I run their script, showing what their script shows.

So my solution should:

[ ] be open-source, easily installable (of course) ;

[ ] launch the Camisole VM if not already available ;

[ ] execute the code based on its MIME type or file extension (.py -> "python", .ml -> "ocaml", .c -> "c", etc) ;

[ ] pretty print the results, giving lots of information: compilation & execution time, stdout & stderr output (pretty printed accurately?)

[ ] possibly (but not by default) copy back all files generated by the script, from the VM storage to the local folder? or at least allow to view/read them?

[ ] store the results in a local file? (a local database? a text file? Markdown or org-mode file?)

[ ] Maybe it's not important that this script can handle multi-files, this is easy to do with a for loop outside the script:

$ ls TP1_etudiants/
Etudiant1.ml Etudiant2.ml ... Etudiant49.ml
$ for i in TP1_etudiants/*.ml; do
    clear; echo $i; hr
    run-camisoled $i
    # this will print a lot of information about student $i, and you have a break to read them, maybe open the file in VSCode, to add annotation etc
    read;
done

[ ] BUT I would LOVE that I could give multiple files and it just executes them, like if I provided a library.ml file (prior to exam) to each student, some student wrote etudiant37.ml and then I want to test their performance using tests.ml

$ ocamlopt library.ml etudiant37.ml tests.ml

Bonus idea?

[ ] run a linter on each file, but from a safe environment?

[ ] Can someone hack you if you just lint their file?

[ ] run the file maybe 100 times and get mean+-std execution time, maybe to allow an automated tool to identify the slowest codes and notify the students that something is wrong ?
[ ] interface it with awesome testing like on ocaml-learn-public, to have coverage test for each student like

Etudiant1: 10/10
Etudiant2: 4/10
...
Etudiant1: 10/10

Features for just OCaml

[ ] what about using ocamlprof to add execution counter everywhere in the file?

Features for just Python

[ ] there are a lot of Python linters, pylint, pyflakes etc: which one to choose?
[ ] automatic line/memory profiling of each code??

Features for just C

[ ] I don't know much, but I also guess there are a lot of C linters
[ ] automatic line/memory profiling of each code??

Tools to use?

Camisole or Docker for safe execution?
Mega-linter for unified linting?

See https://stackoverflow.com/questions/66287065/how-unsafe-can-it-be-to-read-ot-to-use-a-linter-of-an-untrusted-piece-of-code

... I just discovered that python3 (and pypy3) command line have this option:
-I     : isolate Python from the user's environment (implies -E and -s)


Missing Hash for _GET_ENTRY_POSITION_OF_DOOR

Hash: 0xC0572928C0ABFDA3
Output: Vector3 (in world space)
Parameters: Vehicle vehicle, int doorIndex

Was fixed in db4dd10e6b88afca32f51ea98f0ce527b59b2d78.


[feature] Reduce import dependencies (e.g. zustand)

Hi, just running through all the package deps, noticed that zustand was being added?

@tldraw/core also comes with react-use-gesture

Would be nice to slowly reduce the dependencies here — thanks!

Good catch! I've removed those dependencies from the package. (It's back to zero deps). The root package.json will show dependencies from the dev server, which is used for the example site; but the deps in the packages/perfect-freehand/package.json are now clean.

Closed in https://github.com/steveruizok/perfect-freehand/commit/1f9b57500bf9750f0fe04d8ab97460c2b65d96da


Developer poll seems hard coded

From @fbricon on May 2, 2017 16:50
Developer poll on https://openshift.io/_home seems hard coded. It resets after the page is reloaded. Seems like a left over mockup piece. Either make it functional or remove it, since the website is now live.
Copied from original issue: openshiftio/openshift.io#139

It was a space filler and there are UX plans to replace it.

@catrobson what do you want here instead?


Break Out PSL Parser

Break it out of the psl-cli project and replace the current psl-parser project with the cli one.

PR is up for this: https://github.com/linqs/psl/pull/15

Complete in this fork.
Will be merged into PSL formal.


titlebar is dark and title is illegible

Snapshot 152 seems to have introduced a bug where the titlebar is dark (not matching the macOS default titlebar color) and the title on it is also dark, resulting in an illegible title.
I Googled this and found that someone posted about it here (there's a screenshot for reference): https://superuser.com/questions/1382016/application-macvim-menubar-went-dark-how-can-i-fix-it
I saw this on macOS 10.12.6 (Sierra) so I don't have the new "dark mode" and the dark titlebar here does not match other windows' titlebars (or the color of the tabs in MacVim itself). I downgraded to snapshot 150 for now to work around this.

Duplicate of #799. Fix in snapshot 153.


Statewise charts

Preview:

Scale modes not working. Either Deactivate "uniform" switch like the other done or should able to switch to a uniform scale

Can any one check this please, or else please close the pull request

This is covered in deep dive section.


Adds docker py regression test.

Adds test to validate podman image endpoints.
Adds readme on how to run python tests
Adds contants file.
Signed-off-by: Sujil02<EMAIL_ADDRESS>

Couple nits, otherwise LGTM.  Nice work @sujil02 !
Would like a head nod from @jwhonce

@sujil02 did you take @jwhonce suggestions?

@sujil02 did you take @jwhonce suggestions?

Yes all fixes have been added.
@jwhonce please take a look again.

/lgtm
/approve


Negative index passed in substring method for log streaming

Expected Behavior
Dbt cloud plugin should handle properly log streaming.
It's totally my fault, I will provide a PR
Actual Behaviour

Steps To Reproduce
No response
Environment Information

Kestra Version: 0.12.0
Plugin version:  latest (develop-full)
Operating System (OS / Docker / Kubernetes): K8S
Java Version (If not docker):

Example flow
No response

Haha @tchiotludo 😅


[cmd] Parse print review status

Closes #2596

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 2

Clones removed
==============
+ web/server/tests/unit/test_source_code_comment.py  -2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 2

Clones removed
==============
+ web/server/tests/unit/test_source_code_comment.py  -2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 2

Clones removed
==============
+ web/server/tests/unit/test_source_code_comment.py  -2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 2

Clones removed
==============
+ web/server/tests/unit/test_source_code_comment.py  -2

See the complete overview on Codacy

@gyorb I fixed it. I have one more question. If there is not source code comment for a bug, do we want to show the default Unreviewed review status on parse?
E.g:
[HIGH] source_code_comments.cpp:7:3: suspicious usage of 'sizeof(K)'; did you mean 'K'? [bugprone-sizeof-expression] [Unreviewed]

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

Clones added
============
- analyzer/codechecker_analyzer/cmd/parse.py  1
- analyzer/codechecker_analyzer/cmd/check.py  1

Clones removed
==============
+ web/server/tests/unit/test_source_code_comment.py  -2

See the complete overview on Codacy

That is a good question. Showing the unreviewed state would be more in sync with the database, but probably it would add a lot of noise to the output. By default if there is no review comment the definition is that the report is unreviewed.
Probably I would not add it first only if it is requested, but it should be shown up in the json output.
@bruntib @gamesh411  what do you think about printing the Unreviewed status for each report without any review comment, would that be too much noise in the output?

@gyorb As we discussed it today we do not want to show the Unreviewed status for reports without any review comment.

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

Clones added
============
- analyzer/codechecker_analyzer/cmd/parse.py  1
- analyzer/codechecker_analyzer/cmd/check.py  1

Clones removed
==============
+ web/server/tests/unit/test_source_code_comment.py  -2

See the complete overview on Codacy

@gyorb On the ui by default we set this filter to unreviewed and confirmed. To be more consistent with this I set this filter on the command line also to show only unreviewed and confirmed bugs.
I fixed your other comments and I added a couple of new test cases (empty review-status filter and show only confirmed bug).

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

Clones added
============
- analyzer/codechecker_analyzer/cmd/parse.py  1
- analyzer/codechecker_analyzer/cmd/check.py  1

Clones removed
==============
+ web/server/tests/unit/test_source_code_comment.py  -2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

Clones added
============
- analyzer/codechecker_analyzer/cmd/check.py  1
- analyzer/codechecker_analyzer/cmd/parse.py  1

Clones removed
==============
+ web/server/tests/unit/test_source_code_comment.py  -2

See the complete overview on Codacy


∆ human activities, - infauna

From Jenn Brown:
In the meeting at the ICNMS office, we received feedback to create two different groupings related to humans activities

Human Activity Levels - when we have data related to the amount of the activity itself such as vessel traffic, visitation rates
Impacts from Human Activities - for data on noise levels, contaminants levels, etc.

So with that in mind, then the three graphs from SCCWRP  on the BRI would fit in the category of Impacts from Human Activities - Contaminants
It also our only measure of the infaunal community so I suggest we could also link these three graphs to Infaunal community.
Let's remove Infauna as that is redundant with Infaunal community.

Previously from Chris Caldow:
Thoughts on what needs to be done to link those? Do we differentiate in the CR between "infauna" and "infaunal invertebrates"? Can "infaunal invertebrates" be linked to figure App. #. 11.9, 11.10 and 11.11?

Previously from @bbest:
Here's a list of empty modals (ie popup windows) that we don't yet have assigned content to in image_priorities - Google Sheets unless I missed something:

habitat
title

Deep Seafloor
Infauna

Deep Seafloor
Infaunal invertebrates

Deep Seafloor
Mobile crustaceans

Deep Seafloor
Mobile invertebrates

Kelp Forest
Nesting birds

Kelp Forest
Non-indigenous species

Kelp Forest
YOY rockfish

Pelagic
Midwater assemblage

Pelagic
Nesting birds

Rocky Shore
Invertebrates

Rocky Shore
Nesting birds

Sandy Beach
Non-indigenous species

Sandy Beach
Talitrid amphipods (Megalorchestia spp.)

Sandy Seafloor
Infaunal invertebrates

And here are empty modals not yet linked in the svg_links_cinms.csv:

link
size

modals/sea-otters.html
0

modals/shrimp.html
0

Jenn, did you have static images in mind for these?

So now Human Activities modal contains two tabs with content split according to 2019-12-06 Feedback at CINMS in notes | nms-web - Google Docs:

Human Activities

Fishing
Oil & Gas
Ship Traffic
Visitors
[Research: deep-sea mapping]

Human Impacts

Contaminants
Marine Debris
Noise
Seabird disturbance
Whale entanglement

So now looks like this:


KuCoin l2snapshot and l2update don't include sequenceId so it's not possible to ensure correct data

Exchange
KuCoin
Subscription type
Level2 orderbook updates
Describe the bug
The KuCoin REST API for the orderbook snapshot, and the websocket feed for l2 updates, both include sequence info that ccxws isn't propagating. Often when subscribing via ccxws, you'll get a couple l2update events before the l2snapshot event, and it's not possible to know when to apply the l2updates so you can't be sure that the data is correct when maintaining a local order book.
To Reproduce
Steps to reproduce the behavior:

Subscribe to Level 2 updates on KuCoin market, for example ETH-BTC
You'll get a few l2update events
You'll get a l2snapshot shortly after
You don't know if the first few l2updates before the l2snapshot should be ignored, or applied to the l2snapshot

Expected behavior
Sequence ID is included in the l2update and l2snapshot event payloads from ccxws, since they are included in the KuCoin API payloads.

Thanks for reporting the issue!
After a quick look of the documentation: https://docs.kucoin.com/#level-2-market-data, it looks like updates have similar behavior to Binance.  To resolve this, the following changes will need to be made:

update _processL2Update

add sequenceId, sequenceIdLast properties from the WS update stream
remove timestampMs since it no longer provides value

update _requestLevel2Snapshot

include sequenceId

update tests suite to indicate it no longer supports timestampMs but now supports sequenceId

This has been released in v0.35.0, thanks!

thanks for the quick fix :+1:

@bmancini55 this may have introduced a bug, after upgrading from 0.34.1 to 0.35.0 I see this error now
 TypeError: this._handle.unref is not a function

      at Throttle._schedule (node_modules/ccxws/src/flowcontrol/throttle.js:27:18)
      at Throttle._process (node_modules/ccxws/src/flowcontrol/throttle.js:35:12)
      at Throttle.add (node_modules/ccxws/src/flowcontrol/throttle.js:12:29)
      at _sem.take (node_modules/ccxws/src/exchanges/kucoin-client.js:171:12)
      at item.task (node_modules/semaphore/lib/semaphore.js:40:29)

at this line in node_modules/ccxws/src/flowcontrol/throttle.js
 _schedule() {
    this._handle = setTimeout(this._process.bind(this), this.delayMs);
    this._handle.unref();   // <--- here
  }

Thanks, will add a conditional check and get an update out shortly.
unref was added in 0.9.1 (https://nodejs.org/api/timers.html#timers_timeout_unref). What version of node did this happen with?

node v8.11.4

Cool, thanks!  Just wanted to make sure there wasn’t a bigger issue

@evan-coygo should be fixed in v0.35.1.  Let me know if you encounter any other problems!


Include undetermined and unreachable in the verification summary

Description of changes:
With #737 and #798, in addition to "SUCCESS" and "FAILURE", a check's result may also be "UNDETERMINED" or "UNREACHABLE". This PR updates the summary line to include the number of unreachable/undetermined checks if any.
Examples:

If there are 8 undetermined checks, the summary will change from:

** 2 of 10 failed

to
** 2 of 10 failed (8 undetermined)

If there are 5 unreachable checks, the summary will change from:

 ** 1 of 9 failed

to:
  ** 1 of 9 failed (5 unreachable)

If there are no undetermined or unreachable checks, the summary will remain as is:

 ** 1 of 9 failed

Resolved issues:
Call-outs:

Testing:

How is this change tested? Added the summary lines to a few expected tests.

Is this a refactor change? No

Checklist

[X] Each commit message has a non-empty body, explaining why the change was made
[X] Methods or procedures are documented
[X] Regression or unit tests are included, or existing tests cover the modified code
[X] My PR is restricted to a single feature or bugfix

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

Thanks!


Active record linking via table

I have found it inconvenient to explicitly declare column values when using the link method involving a linking table, especially as these values are defined within the model for the linking table as defaults within the model rules.
If I ever change these rules, I then have to review all instances of the "link" method. Not very efficient.
I propose to pass an object of the link table model so that these defaults can be applied.  I have a way to do this that would be backwards compatible,  and not interfere with existing setups that don't use the link table model.
If you agree, I will complete a pull request with details of my proposed change.

Pull request made:
https://github.com/yiisoft/yii2/pull/6812

Thanks for you usecase. Its is becoming clear taht the problem with the current link method is that there are too many usecases thinkable. The more arguments a method gets, the less manageable the code becomes and the less OO we will get.
Linking 2 models seems to be a complete architecture on itself. There are things involved: models, attributes, relations, junctiontables, existing records, non existing records. Its too complex to cover with a method only and too error prone. A solution must be found from a complete architectural perspective.
Thanks for helping us to get to this understanding!


[EXPERIMENTAL] Add systemd support for deb packages

ref: #317
This will add new deb package for systemd-based distributions.
This p-r introduces:

add tenporary make task make deb-systemd to build deb package for systemd

I tested on Debian 8 and Ubuntu 16

This p-r does not introduce:

Uploading the systemd supported-deb package
systemd-supported deb package for stage and kcps

I'll do them in another p-r.

LGTM!

Thanks!


[native_assets_cli] Add JarAssets

[ ] Add a new JarAsset in the API.
[ ] Consume that asset type in flutter_tools.
[ ] Consume that asset type in Dart standalone. (optional)

This would solve:

https://github.com/dart-lang/native/issues/576

We didn't have a tracking bug for this yet, just some loose references in various places (for example https://github.com/dart-lang/native/issues/878#issuecomment-2222506593).

We also need a BuildConfig.targetJavaVersion that can be passed to javac as -target $version. (Note this should be a Version or String in the API, as versions such as 1.6 occur.)

We also need a BuildConfig.targetJavaVersion that can be passed to javac as -target $version. (Note this should be a Version or String in the API, as versions such as 1.6 occur.)

And a minimumAndroidSdk.
This also allows us to evaluate android sdk version comparisons at compile time and tree shake unused code away similar to how Platform.isFoo does it. We had this discussion with @mkustermann before. One question is where to put "Android.minimumSdkVersion". For now it's only accessible from Flutter but we could decide to have a specific android target in mind when compiling Dart standalone as well which we can pass in pubspec.yaml.

minimumAndroidSdk

https://pub.dev/documentation/native_assets_cli/latest/native_assets_cli/HookConfig/targetAndroidNdkApi.html

minimumAndroidSdk

https://pub.dev/documentation/native_assets_cli/latest/native_assets_cli/HookConfig/targetAndroidNdkApi.html

That seems like a confusing name though because we also have a targetSdkVersion.

minimumAndroidSdk

https://pub.dev/documentation/native_assets_cli/latest/native_assets_cli/HookConfig/targetAndroidNdkApi.html

That seems like a confusing name though because we also have a targetSdkVersion.

It seems that the reviewer on https://github.com/dart-lang/native/pull/49 should have done more bike shedding! 🤔

It seems that the reviewer on #49 should have done more bike shedding! 🤔


ATV14

Since homatics updated his box to ATV14 your wonderful app doesn't work anymore
Tried a reset and fresh install but can't activate in accessibility sei804 overlay ..(I set switch to on but return immediately to off)
Thank you

Hey,
This App is EOL.
Doesn't work anymore on ATV14 and will never be.
I have tried to get it to work, but the getprop values are missing in ATV14.


[Feature Request] Bank Place holder prices

Hello,
I think a cool thing to add is to make the price check possible for bank place holders.
That would make it easier for you to keep track of the price of the items that you once had and for some reason sold or lost.
Lets say you got a Twisted bow in a friends raid and had to split it between them, you added the bow to your bank and now once you examine it you can see the current price.
Thank you :)

Implemented in #5894

Thats awesome! Thank you Tomas! :)

De: Tomas Slusny<EMAIL_ADDRESS>Enviado: sábado, 29 de setembro de 2018 05:04
Para: runelite/runelite
Cc: Joshwat; Author
Assunto: Re: [runelite/runelite] [Feature Request] Bank Place holder prices (#2900)
Implemented in #5894
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/runelite/runelite/issues/2900#issuecomment-425625881, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AlmLHicGx8P1NaCg32bBxdwggYXWcuQ5ks5ufymMgaJpZM4UFfe0.


Better architecture to let operations rely on credentials for external services

Is your feature request related to a problem or area of OpenRefine? Please describe.
Some of OpenRefine's operations interact with web services, which can potentially require authentication:

Uploading edits to Wikidata
Add column by fetching URL
Reconciliation and data extension, if we add support for authentication in the API (#2015)

At the moment we do not have good workflows to handle the credentials required by these operations:

For Wikidata upload, these credentials are stored in the preferences, and accessed by the operation via a ConnectionManager which holds these credentials centrally. This makes it hard for multiple users to use the same OpenRefine instance with different credentials, even when working on different projects;
For URL fetching, the Authentication header optionally supplied by the user is stored as one of the parameters of the operation, therefore serialized in JSON in the undo/redo tab. This means that if users share their workflows, they will also share the API keys that they use, which is not ideal.
For reconciliation, the feature does not exist yet, so the mechanism would need to be designed.

Describe the solution you'd like
We need to think about the user experience we would like to aim for here. When the user works on a project by applying operations interactively in the UI, it is easy for us to adapt the UI to require authentication when appropriate.
When sharing a JSON workflow, how do we want to proceed? I would say we should avoid including credentials in the operations' JSON serialization. Should these credentials be included somewhere else in the workflow (for instance before the list of operations, as a well-identified header that users can easily censor?), or not at all? In this case, when applying a workflow that contains some operations requiring authentication, how do we detect that those operations do need credentials, and how do we prompt the user for them?
The way these credentials are acquired can vary greatly depending on the operation (OAuth for Wikidata, API key for URL fetching, perhaps even something else for reconciliation), so coming up with an architecture that lets operation describe those required credentials in a flexible way seems difficult (but not impossible).
Additional context
Some discussion about this is in #2661.

When sharing a JSON workflow, it would be awesome if it could prompt for the credentials needed.  That's a bit how some other tools work that I am used to.
Perhaps we can have a prefix on any credential key?

When exported those credential keys have "blank" values.
Then before actually applying we just scan the JSON first and see if there are any special "cred:" prefixed keys like "cred:WikidataAuthPass" and lookup in the preferences if we have values or not and if not...

build up a prompted list to show the user?  or if too complex...
just warn them "xyz key not found. Please go to your Preferences and set this key and then re-apply your operations JSON".

@afkbrb I have an afterthought: it might look like I have just pushed a big project with very complicated design decisions into the scope of your GSoC project. That is not my intention! The issue message above is blue sky thinking, just trying to imagine what would be the ideal system we should strive towards.
It totally makes sense to do something less ambitious initially. Your idea to just let operations extract credentials from HTTP headers is a pragmatic one, even if it does not lead to good UX when the credentials are missing. That being said, I would try to keep the core datamodel classes (such as AbstractOperation) independent from the web framework: executing OpenRefine workflows should be doable in isolation, outside the context of a web app (so with no dependency to any servlet library). So there should be some class(es) to interface that: extract the credentials from an HTTP request at the Command level, store that in a class we control, and pass that on to the operation, if you get the idea.
Introducing such interfaces is a good first step towards something genuinely generic, where these credentials can not only be extracted from an HTTP request but also prompted to the user if they are absent. So it is good to have the blue sky thinking in mind when designing those classes.


Add a * to clues that have notes

* is coloured based on the user_attr
for the user who most recently added a note.

Placement of * is relative to terminal width!

for possible later iterations, what do you think about placing the * closer to the right-hand side of the clue box, maybe at 10% of the distance between the clue box and the right-hand side of the terminal? this might just be me (it happens a lot :)) but i think i'd miss the connection between the *s and clues when my terminal is full-screened.
another option that looks good to me in a quick trial is putting the * to the left of the dirnum and just adding a space before dirnums without notes.


Calculate right hashes

We should calculate hash only from canonnical json.

@patriotyk Is this ready from your perspective?

@patrickvacek  Yes it is ready


lunchd should only restart upon crashes

Currently, launchd will automatically restart yabai when the process is not running, regardless of why it is not running.
It seems to me that it would be more appropriate to make it only restart yabai if it stops running unexpectedly. i.e. the process terminates because of a crash (bug).
This has some implications, some good and some that might be unfavourable:
Good:
When yabai is installed through brew and the user has not yet given the process accessibility access, the current version will spam the user with a popup repeatedly until the user successfully grants permission and restarts the service.
With this change yabai is launched, prompts the user for accessibility permission, and exists. launchd will not restart yabai automatically, because it didn't crash -- it was an expected exit. The user has to start the service again by invoking the correct command once more.
Bad:
Yabai has other requirements such as display has separate spaces. There are some up-front memory allocations that take place during startup which yabai uses for its entire lifetime which upon failure will cause the process to exit (expectedly), which is not a crash, and will not prompt an automatic restart by launchd.
--
I guess the best solution is to do a bit of both. First separate the various stages of failure that can occur into categories.
Then have each category handle termination approptiately.

Requirements for correct system state (AX API, Display has separate spaces, etc.).
Requirement for yabai to initialize correctly (memory, socket, etc.)
Unexpected termination

We want launchd to restart yabai in case of category 2 and 3.
Category 1 requires an action from the user and should not prompt an automatic restart.

Somewhat related to #703


fix(chore): bumping dependencies and resolving npm audit warnings

🧰 Changes
This bumps the following deps:

[ ] @babel/core
[ ] @babel/eslint-parser
[ ] @babel/plugin-proposal-object-rest-spread
[ ] @babel/preset-env
[ ] @readme/eslint-config
[ ] @readme/variable
[ ] babel-jest
[ ] codemirror
[ ] eslint
[ ] husky
[ ] jest
[ ] prettier
[ ] terser-webpack-plugin
[ ] webpack
[ ] webpack-cli
[ ] webpack-dev-server

I've also updated our CI workflow to stop running tests against Node 12 because we aren't using Node 12 in production.
Finally I ran npm audit fix to resolve a number of dependency security warnings. There's still some remaining but they'll require a bump of remark-parse and I'm scared to touch the Remark deps:
$ npm audit
# npm audit report

trim  <0.0.3
Severity: high
Regular Expression Denial of Service in trim - https://npmjs.com/advisories/1700
fix available via `npm audit fix --force`
Will install<EMAIL_ADDRESS>which is a breaking change
node_modules/trim
  remark-parse  <=8.0.3
  Depends on vulnerable versions of trim
  node_modules/remark-parse

2 high severity vulnerabilities

To address all issues (including breaking changes), run:
  npm audit fix --force

🧬 QA & Testing

[ ] Tests pass?
[ ] npm build work?

There's still some remaining but they'll require a bump of remark-parse and I'm scared to touch the Remark deps:

Yea, that's going to be a big change. To update any of the deps, we're going to have to rewrite all the parsers.
https://linear.app/readme-io/issue/RM-52/update-remark-parse-in-the-rdmd-engine

I've also updated our CI workflow to stop running tests against Node 12 because we aren't using Node 12 in production.

Completely forgot to do that, thanks!

@kellyjosephprice Oh cool looks like you fixed the devServer problems. This should be good to go now!


chore: bump most dated deps

Summary
Now that we've dropped node 6, we can upgrade a bunch of dependencies. None of these should be breaking changes for our consumers, which is why it's a pretty big one.
~Only deps held back are jsdom and micromatch, both of which I'll send separate PRs for.~
I've also held source-map back as it's sync only (https://github.com/mozilla/source-map/issues/331)
~EDIT: Need to hold back ansi-escapes as well due to hoisting issues~

Test plan
Green CI. Should probably land #8411 first though

Getting closer. Only failure left are some weird snapshot failures.

They all complain about Compared values have no visual difference., not sure why

code on Circus is undefined, likely some more adjustments to execa output are needed

Codecov Report

Merging #8850 into next will decrease coverage by 0.01%.
The diff coverage is 10%.

@@            Coverage Diff            @@
##            next    #8850      +/-   ##
=========================================
- Coverage   63.8%   63.79%   -0.02%
=========================================
  Files        274      274
  Lines      11576    11589      +13
  Branches    2841     2842       +1
=========================================
+ Hits        7386     7393       +7
- Misses      3560     3566       +6
  Partials     630      630

Impacted Files
Coverage Δ

packages/jest-changed-files/src/git.ts
0% <0%> (ø)
:arrow_up:

packages/jest-changed-files/src/hg.ts
0% <0%> (ø)
:arrow_up:

packages/jest-haste-map/src/lib/FSEventsWatcher.ts
12.5% <50%> (+1.59%)
:arrow_up:

packages/jest-test-sequencer/src/index.ts
91.66% <0%> (+0.17%)
:arrow_up:

packages/jest-core/src/SearchSource.ts
64.44% <0%> (+0.39%)
:arrow_up:

packages/jest-watcher/src/lib/Prompt.ts
73.46% <0%> (+0.55%)
:arrow_up:

...t-core/src/plugins/update_snapshots_interactive.ts
53.12% <0%> (+4.84%)
:arrow_up:

Continue to review full report at Codecov.

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

Yay for mac CI, I actually broke watch mode by upgrading fsevents :P I read https://github.com/fsevents/fsevents/releases/tag/v2.0.0, without thinking about the deprecated title - the API is different in v2. I think just fsevents.watch(root, handler) is enough, but I'm not sure. I'll leave it at v1 for now, seeing as they've released a version that works on node 12

@SimenB Yes, your diagnosis is correct that custom serializer can contradict an original assumption that the value is enclosed in matching punctuation, like double quote marks for strings.
After const pass = expected === receivedSerialized; failed and before call to diff results:
    expected = (expected || '').trim();
    actual = (actual || '').trim();

To make sure that I understand the situation, some snapshots of CLI output had a blank line but after bump, the received values did not? After you updated them, diff told you what was the problem?

To make sure that I understand the situation, some snapshots of CLI output had a blank line but after bump, the received values did not?

Correct

After you updated them, diff told you what was the problem?

Yeah, I ran the test locally to try to figure out what was wrong, and the reporter told me to update the snapshot. So it's somewhat an issue with just the silent reporter not printing the snapshot thing (https://github.com/rickhanlonii/jest-silent-reporter/issues/15), but I don't think I should be told that the values "have no visual differences" when they do. And yes, updating the snapshots and doing git diff made it obvious what the change was 🙂

I don't know what the correct solution is. Maybe look for whitespace differences if strings are being compared? Not sure

Oh, a candidate solution comes back to me! Instead of trim it needs to do the opposite of:
// Extra line breaks at the beginning and at the end of the snapshot are useful
// to make the content of the snapshot easier to read
const addExtraLineBreaks = (string: string): string =>
  string.includes('\n') ? `\n${string}\n` : string;

If a snapshot:

has at least 3 newlines
starts and ends with newline

then removeExtraLineBreaks slices away the first and last newline. What do you think?

Yeah, that sounds reasonable!


[Bug] tun模式下auto-route为false时仍会添加ipv6路由和规则

Verify steps

[X] I have read the documentation and understand the meaning of all configuration items I have written, avoiding a large number of seemingly useful options or default values.
[X] I have not reviewed the documentation and resolve this issue.
[X] I have not searched the Issue Tracker  for the problem I am going to raise.
[X] I have tested with the latest Alpha branch version, and the issue still persists.
[X] I have provided server and client configuration files and processes that can reproduce the issue locally, rather than a desensitized complex client configuration file.
[X] I have provided the simplest configuration that can reproduce the error I reported, rather than relying on remote servers, TUN, graphical client interfaces, or other closed-source software.
[X] I have provided complete configuration files and logs, rather than providing only parts that I believe are useful due to confidence in my own intelligence.

Operating System
Linux
System Version
OpenWrt 23.05.3
Mihomo Version
v1.18.4
Configuration File
tun:
  enable: true
  stack: system
  auto-route: false
  auto-detect-interface: false

Description
TUN模式下，设置auto-route为false时，mihomo仍会添加ipv6路由和规则。
root@openwrt:~# ip -6 rule
0:      from all lookup local
32765:  from all lookup 2022
32766:  from all lookup main

root@openwrt:~# ip -6 route
default dev utun  metric 1024

Reproduction Steps
Linux下开启tun模式即可复现
Logs
No response

该行为来自 https://github.com/SagerNet/sing-tun/blob/fb6e917a2cd7a5e6a0c45cc5291f2cbe9c8e8bcb/tun_rules.go#L106-L179 ，似乎是为了避免有 Inet6Address 但没有 ipv6 路由时，ipv6 绕过 tun 直连发出。

好像这个逻辑造成一个问题就是clash dns的解析不会返回ipv6解析

应该是写错了


chore: promote test15 to version 0.0.16

chore: promote test15 to version 0.0.16
this commit will trigger a pipeline to generate the actual kubernetes resources to perform the promotion which will create a second commit on this Pull Request before it can merge

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign q1323829945
You can assign the PR to them by writing /assign @q1323829945 in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@q1323829945: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

verify
6b7fcf416694f3f855b97977077f441de4a828bd
link
/test verify

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


Switched to gradle and created different source sets

I added an api source set for other plugins to ship. When building, a jar for api will be created. Moreover, I added travis ci integration file and applied sponge code style to all files. Also, I added utility method to get api instance from the service manager, and prevented unwanted instance creation.

Thanks... You can just run ./gradlew in the future to build. The jar built is in build\libs folder, and there will be an api jar for other plugin makers to include as a dependency and the production plugin jar for server owners to use.


Add support for BARPA py18 data store

This is my first attempt at adding an NCI ESM Datastore to the catalog.
I have started playing with the translators which, I think, could be refactored.

I'll run through this later this afternoon - I'm not overly familiar with the process of adding datastores yet so will probably take me a couple hours - I'm planning to use it to understand the process properly

Thanks @charles-turner-1

Closes #211

Closes #197


Improve +fci|disable-when-company-activates

Hello, and sorry for my poor English...
I think this change avoids the error caused when the variable fci-mode is void.
Thanks.

Thanks for your help!


Code Coverage Report is empty

🐞 Bug report
Command (mark with an x)

[ ] new
[ ] build
[ ] serve
[X] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?

 Yes, the previous version in which this bug was not present was 8.3.21
Description
 After updating npm package from version 8.3.21 to 8.3.22, When I use the Angular CLI to generate a code coverage report locally the index.html inside the "coverage" folder does not show any file names or code coverage % statistics.

No. This is not a browser specific issue.

No.

No longer able to reproduce issue. Might have been a local issue.


Update tslint-config-typings

Fix linting

did someone just publish a breaking change as a patch version for tslint-config-typings?

Nope. It's a regression on tslint

https://github.com/palantir/tslint/issues/1106


Fixing #1716

Resolves #1716.
import fiftyone.utils.yolo as fouy

print(fouy._get_yolo_v5_labels_path("/a/b/images/000001.jpg"))
# "/a/b/labels/000001.jpg"

# Previously raised an error, now works
print(fouy._get_yolo_v5_labels_path("images/000001.jpg"))
# "labels/000001.jpg"

@ehofesmann okay good catch. The discrepancy was that, ordinarily, relative paths are interpreted w.r.t. your current working directory. However, with YOLOv5, I believe you are correct that, when relative paths are used in a split.txt file, they are intended to be interpreted relative to the dataset YAML file.


Cannot copy a picture from an album to a subalbum

I try to copy pictures from an album to a subalbum but I'm not able to.
When I click the move button the only albums I see are the "mainalbums" and not the subalbums"
Output of the diagnostics (Settings => Diagnostics)
Diagnostics
-----------
Warning: Dropbox import not working. dropbox_key is empty.
Warning: You may experience problems when uploading a large amount of photos. Take a look in the FAQ for details.

System Information
------------------
Lychee-front Version:      -
Lychee Version (git):      No git data found. Probably installed from release or could not read .git
DB Version:                040000
System:                    Linux
PHP Version:               7.3
MySQL Version:             5.7.24

Lychee total space:        287.45 MB
Upload folder space:       0.00 B
System total space:        1.20 TB
System free space:         137.88 GB (11%)

Imagick:                   1
Imagick Active:            1
Imagick Version:           1684
GD Version:                bundled (2.1.0 compatible)

Config Information
------------------
version:                   040000
check_for_updates:         1
sorting_Photos_col:        takestamp
sorting_Photos_order:      ASC
sorting_Albums_col:        description
sorting_Albums_order:      DESC
imagick:                   1
skip_duplicates:           0
small_max_width:           0
small_max_height:          360
medium_max_width:          1920
medium_max_height:         1080
lang:                      en
layout:                    1
image_overlay:             1
image_overlay_type:        desc
default_license:           none
compression_quality:       90
full_photo:                1
delete_imported:           1
Mod_Frame:                 0
Mod_Frame_refresh:         30000
landing_page_enable:       0
landing_owner:             PhotoLychee Team
landing_title:             PhotoLychee Team
landing_subtitle:          just a testwebsite
landing_facebook:
landing_flickr:
landing_twitter:
landing_instagram:
landing_youtube:
landing_background:        dist/cat.jpg
thumb_2x:                  1
small_2x:                  0
medium_2x:                 0
site_title:                Lychee v4
site_copyright_enable:     1
site_copyright_begin:      2019
site_copyright_end:        2019
public_search:             0
additional_footer_text:
display_social_in_gallery: 0
public_recent:             0
recent_age:                1
public_starred:            0
downloadable:              0
photos_wraparound:         1

Browser and system
This happens with Chrome and Edge.
My site is installed on a shared host: www.photolychee.nl

It's a subtle bug that may be triggering in your case because of the peculiarities of your shared hosting environment as we've established earlier, although I'll check in a regular 32-bit environment later if I can see the same behavior there, just to be sure.
The bug is a side effect of a behavior of Lychee v4 that I don't particularly like, namely that we sometimes -- but not always -- send IDs as strings and other times as numbers. I believe we should consistently send data to the front end as strings, which would avoid any such issues, whatever their source.
I just prototyped such a fix at your site; the changes are to app/Album.php and app/Photo.php. It seems to fix the particular problem you reported, but can you play around to see if it breaks anything else in the process?

Hi Kamil,
Thank you very much for looking into this.
I tested a few foto's and did not notice any strange behavoviour or other side effects.
I copied the 2 files to another installation and also there it is working great.
I will test it further and will report in case of errors.

I tested in a 32-bit environment and I could not reproduce it, so it must be again a peculiarity of the MySQL installation at your shared hosting. Still, as I had said, we were being unnecessarily sloppy on the server side, so I'm fixing that. The PR that I submitted for merging is more comprehensive than the version I prototyped in your environment but I wouldn't worry about that if what you have works for you.

Thanks for keeping me updated!
I  have a few other shared accounts hosted by other companies and I will also try it there as well.
Maybe it's something only happenig by this host.


[Beacon] more descriptive content for incorrect Carbon for IBM.com version

The dds-version-check-audit audit checks the page to see if the Carbon for IBM.com package version is present in the Digital Data Object, and if so, if it is the latest version. If the version being used is older than the latest version of Carbon for IBM.com, the audit will alert the user. This copy can be more informative than it currently is. Users should be aware that by using older versions, other audits could also fail.
Also, the Learn more link currently points to the Digital Data Object page, which for package version, is not very helpful at all. Comparing releases might be a bit too granular – perhaps more general copy informing users of the disadvantages/issues of using older versions?

@kennylam Thanks for describing the requirements for the message. That helped a lot. Here is a first pass at new content. I wanted to see if we could reorder the content at the top and make the warning sentence the red sentence. Some other things:

I removed the word stable. Is it really necessary?
I also shortened the two sentences referencing the version. They may need to include Carbon for IBM.com but maybe not if they are always paired with the warning sentence above.
I wrote two paragraphs. You could skip the second one if you think this is getting too long. I wrote it so that we could end on a positive.

Anyway, let me know what you think. Happy to adjust any way we need to!


move detail functions out of public headers

url/decode.hpp has detail function declarations that need to be moved to url/detail/decode.hpp

done


chore(ui): Update TypeScript 5.1.5 devDependencies

Description
https://github.com/Microsoft/TypeScript/issues?utf8=✓&q=milestone%3A"TypeScript+5.1.5"+is%3Aclosed+
Fix a few regressions which would break the build if we hit them:

stack overflow
heap out of memory
compiled code can contain JSX

Checklist

[ ] Investigated and inspected CI test results
[x] ~Unit test and regression tests added~

Testing Performed

yarn build in ui
yarn start in ui

/test ocp-4-10-ui-e2e-tests


Allow full customization of AssumeRoleInput for the AssumeRole credential provider

Describe the feature
From discussion https://github.com/awslabs/smithy-rs/discussions/1309
Customers should be able to set every field on the AssumeRole input without code changes to expose those fields so that if STS adds new fields to the operation, they're immediately accessible after the STS model is updated.
Use Case
A new field is added to AssumeRole and I want to use it when directly constructing an AssumeRoleProvider.
Proposed Solution
Expose the AssumeRoleInput in the AssumeRoleProviderBuilder.
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request
[X] This feature might incur a breaking change

A note for the community

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue, please leave a comment

Seems like customers should also be able to configure the underlying STS client when they create the AssumeRoleProvider.


Add annotation about ResourceQuota for HRQ implementation to some documents

What type of PR is this?:
/kind documentation
/kind cleanup
What this PR does / why we need it:
As discussed https://github.com/kubernetes-sigs/hierarchical-namespaces/pull/301#issuecomment-1738301338, I've added annotation about ResourceQuota for HRQ internal implementation as Note section.
In addition, the expression of the Note section was different depending on each the document, so I've unified them.
Tested:
None(Only document fix)
Related:
#275
https://github.com/kubernetes-sigs/hierarchical-namespaces/pull/301#issuecomment-1738301338

/lgtm
/approve


feat(cdk/dialog, cdk/overlay): support open and close animations for overlay and backdrop

Fixes #28878

Will fix and add new tests once I know if this can even be merged.

Hmm, that looks to me like a bad idea to be honest. But if thats the way, ok.

To elaborate on why we want to move away from it long-term:

Our animations are fairly basic so we don't have much out of the module in the first place.
We don't want to force our users to install the module.
The animations DSL can be a bit hard to follow which makes the animations harder to change.
Over the years there have been issues with memory leaks coming specifically from the animations module.

Thanks for the insight!

Our animations are fairly basic so we don't have much out of the module in the first place.

Makes sense to me for the material ui as it is opinionated and it is expected that people do not want to change it, but that does not really work for CDK in my opinion. The CDK is described in the README in the following way:

@angular/cdk: Library that helps you author custom UI components with common interaction patterns

The animations DSL can be a bit hard to follow which makes the animations harder to change.

Subjective, but I see where you are coming from. For me personally it is much easier and flexible than defining it in CSS.

2 and 4, yeah, valid... 4 could be fixed though but probably unrealistic.


Replace firstNonWhitespaceCharacterIndex with rangeTrimmed

Fixes #2519
Checklist

[/] I have added tests
[/] I have updated the docs and cheatsheet
[/] I have not broken the cheatsheet

Updated with undefined if the range is empty or whitespace.


ROX-21034: qa-e2e test for GCS backup integration

Description
Add qa-e2e test for GCS external backups using both a service account key and a workload identity.
Checklist

[x] Investigated and inspected CI test results
[x] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
See tests.
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

[!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

Current dependencies on/for this PR:

master

PR #9868

PR #9913

PR #9914   👈

This stack of pull requests is managed by Graphite.

/test all

tests are failing currently because the bucket in acs-san-stackroxci does not exist yet.

Need to create the required resources via terraform first.

/test ocp-4-11-qa-e2e-tests ocp-4-14-qa-e2e-tests

/test all

/test ocp-4-11-qa-e2e-tests ocp-4-14-qa-e2e-tests

/retest

/test ocp-4-11-qa-e2e-tests ocp-4-14-qa-e2e-tests

/test ocp-4-11-qa-e2e-tests ocp-4-14-qa-e2e-tests

/retest

/test ocp-4-11-qa-e2e-tests ocp-4-14-qa-e2e-tests ocp-4-15-qa-e2e-tests


it's navigate to phone's settings not in health app settings in iOS 18

any update?

any update?

not yet...

Hi! I'm currently working on it. I commented with more details in #23.


Wear OS App?

Hi,
it would be nice if this app works also on Wear OS like this:
Press a Watch Button, while press the button, the App streams the Audio to Rhasspy base and do there the processing of the command. So we can use the Watch as satelitte and can speak to Rhasspy base without the need of an satelitte in the room.

This could be done but since I don't have access to a physical device to test on, development is not easy. if you have a smartwatch with wear OS I could initially try the development thanks to the emulator and then release a beta so you can try the app and if it doesn't work properly you could open an issue.

Thanks for your reply. This time i have only a Pebble Time Steel :-) . But if there will be an App i will buy a Wear OS Watch.
This App would be great, always a Satellite on the Wrist. If you realize this App, it would be amazing if the Wake Up of the App is triggered by the Movement from the Watch to the Mouth by the accelerometer of the Watch.
You should really think about an Donate Page, i think many People will donate for such a App (me too :-) ).
I can post a Link to this Issue in the Rhasspy Forum. Perhaps one of the Users has a Watch with Wear OS. I will look this Days online for a Watch ond order one.

I could add a donation page to keep myself even more motivated to the development of the app and perhaps in the future, I will be able to run the app on Wear OS. Anyway, nice idea of using the accelerometer to wake up the app, when I manage to make it work on a smartwatch I’ll add this functionality.

Is it for the Development much work to create a Tizen App for the Galaxy watch? Is there a Cross Platform developement possible? Or Wear OS is near by Android so that a Wacth with Wear OS is better?

The application was developed thanks to the frawemork flutter this allows the application to run on various platforms such as Android, iOS , Web, Windows, Linux and macOS but does not officially support Tizen Os this leads to having to do a lot of work in case you want to support it.

Have now a Wear OS Watch.
I am ready for testing if you want.
Greetings

Have now a Wear OS Watch.
I am ready for testing if you want.
Greetings

Okay, I don’t think I can produce anything this week, but by the next week, I should be able to share an experimental version. Could you tell me what model of smartwatch you have?

Okay, I don’t think I can produce anything this week, but by the next week, I should be able to share an experimental version. Could you tell me what model of smartwatch you have?

Don´t rush :-). My Watch is a Ticwatch Pro3.
Had installed the Flutter Framework an do a look. Perhaps i can help.
But this time im Program only with Perl, PHP, Python...
Thanks a lot.

Don´t rush :-). My Watch is a Ticwatch Pro3.
Had installed the Flutter Framework an do a look. Perhaps i can help.
But this time im Program only with Perl, PHP, Python...
Thanks a lot.

I have created a new branch where it is the new code for the wear os app. I have tried with the emulator and it seems to work pretty well obviously the UI doesn't fit well on a small screen but this can be fixed later.
from what I read on the internet to run the app on a smartwatch you need to set up ADB wireless and then you can do flutter run. Thanks in advance for your time

I have created a new branch where it is the new code for the wear os app. I have tried with the emulator and it seems to work pretty well obviously the UI doesn't fit well on a small screen but this can be fixed later.
from what I read on the internet to run the app on a smartwatch you need to set up ADB wireless and then you can do flutter run. Thanks in advance for your time

Hi, many thanks for Your time 😊
Got it running on the watch, but cant test it because dont reach the setting to insert ip adress etc.
Is there a remote tool in the Android studio to insert the settings?
Von: razzo04<EMAIL_ADDRESS>Gesendet: Donnerstag, 14. Januar 2021 15:35
An: razzo04/rhasspy-mobile-app<EMAIL_ADDRESS>Cc: Jochen Entenmann<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [razzo04/rhasspy-mobile-app] Wear OS App? (#4)
I have created a new branchwhere it is the new code for the wear os app. I have tried with the emulator and it seems to work pretty well obviously the UI doesn't fit well on a small screen but this can be fixed later.
from what I read on the internet to run the app on a smartwatch you need to set up ADB wireless and then you can do flutter run. Thanks in advance for your time
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi, many thanks for Your time 😊
Got it running on the watch, but cant test it because dont reach the setting to insert ip adress etc.
Is there a remote tool in the Android studio to insert the settings?
Von: razzo04<EMAIL_ADDRESS>Gesendet: Donnerstag, 14. Januar 2021 15:35
An: razzo04/rhasspy-mobile-app<EMAIL_ADDRESS>Cc: Jochen Entenmann<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [razzo04/rhasspy-mobile-app] Wear OS App? (#4)
I have created a new branchwhere it is the new code for the wear os app. I have tried with the emulator and it seems to work pretty well obviously the UI doesn't fit well on a small screen but this can be fixed later.
from what I read on the internet to run the app on a smartwatch you need to set up ADB wireless and then you can do flutter run. Thanks in advance for your time
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

As I except there difference between the emulator and real devices, on the emulator I could still click to navigate between pages. However, with the last update, a swipe is enough to navigate between the pages.

As I except there difference between the emulator and real devices, on the emulator I could still click to navigate between pages. However, with the last update, a swipe is enough to navigate between the pages.

Hi, sorry cant swipe. Only below the blue bar in can scroll down. But dont reach the settings.
Von: razzo04<EMAIL_ADDRESS>Gesendet: Freitag, 15. Januar 2021 21:06
An: razzo04/rhasspy-mobile-app<EMAIL_ADDRESS>Cc: Jochen Entenmann<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [razzo04/rhasspy-mobile-app] Wear OS App? (#4)
As I except there difference between the emulator and real devices, on the emulator I could still click to navigate between pages. However, with the last update, a swipe is enough to navigate between the pages.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi, sorry cant swipe. Only below the blue bar in can scroll down. But dont reach the settings.
Von: razzo04<EMAIL_ADDRESS>Gesendet: Freitag, 15. Januar 2021 21:06
An: razzo04/rhasspy-mobile-app<EMAIL_ADDRESS>Cc: Jochen Entenmann<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [razzo04/rhasspy-mobile-app] Wear OS App? (#4)
As I except there difference between the emulator and real devices, on the emulator I could still click to navigate between pages. However, with the last update, a swipe is enough to navigate between the pages.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

In the latest version, I removed the blue bar because it didn’t fit right on the screen. Did you download the latest version?

In the latest version, I removed the blue bar because it didn’t fit right on the screen. Did you download the latest version?

Have still the blue bar. Uninstalled App on Watch.  Deleted "C:\src\rhasspy-mobile-app". In git changed branch to "wear_os_app".  Then copied Link and in src Dir "git clone https://github.com/razzo04/rhasspy-mobile-app.git".
Started Android studio with the project, connected watch with adb and run.
Anything wrong? :-)

Have still the blue bar. Uninstalled App on Watch.  Deleted "C:\src\rhasspy-mobile-app". In git changed branch to "wear_os_app".  Then copied Link and in src Dir "git clone https://github.com/razzo04/rhasspy-mobile-app.git".
Started Android studio with the project, connected watch with adb and run.
Anything wrong? :-)

I guess git isn’t taking into account the branch change so you could try to pass on git clone the branch name.
git clone -b wear_os_app https://github.com/razzo04/rhasspy-mobile-app.git

Then if you want to run the app you can use android studio or flutter run.
cd rhasspy-mobile-app
flutter run

I guess git isn’t taking into account the branch change so you could try to pass on git clone the branch name.
git clone -b wear_os_app https://github.com/razzo04/rhasspy-mobile-app.git

Then if you want to run the app you can use android studio or flutter run.
cd rhasspy-mobile-app
flutter run

Hi. Thanks This did the Trick.
I can go to the settings and insert values. But it is very hard to insert the values.
After that it works. Sadly very, very, very slow. But it works :-)

Hi. Thanks This did the Trick.
I can go to the settings and insert values. But it is very hard to insert the values.
After that it works. Sadly very, very, very slow. But it works :-)

I’m glad it works. What problem did you encounter when entering values? What are your tips for improving the UI? At least for future people who will try to set-up the application, the process will be easier.

I’m glad it works. What problem did you encounter when entering values? What are your tips for improving the UI? At least for future people who will try to set-up the application, the process will be easier.

Hi, sorry fort he late answer. Entering Values on the watch is only possible with write with the fingers on the screen. Using oft he on screen keyboard ist not really usable cause you dont see what you are enter.
I send you some screenshots. I think for display on the watch the ui should be 50 percent from the normal ui.
What i see when i use the app:

Startup is slow. Needs about 30 seconds till i can see the ui
From touch the microphone until speak are about 5 seconds
When the watch is connected with bluetooth tot he phone the app does not work

The app can only work over WLAN. I had to disconnect bluetooth on the phone to get the app working.
Think the better way would be to have a companion app to the smartphone app. But this is sure to much work.
Another way perhaps is to have a MQTT Audio streamer that only streams the audio to the server over a companion app over bluetooth.
Greetings and thanks for your Work!
Von: razzo04<EMAIL_ADDRESS>Gesendet: Donnerstag, 21. Januar 2021 18:02
An: razzo04/rhasspy-mobile-app<EMAIL_ADDRESS>razzo04/rhasspy-mobile-app<EMAIL_ADDRESS>Cc: Jochen Entenmann<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Jochen Entenmann<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [razzo04/rhasspy-mobile-app] Wear OS App? (#4)
I’m glad it works. What problem did you encounter when entering values? What are your tips for improving the UI? At least for future people who will try to set-up the application, the process will be easier.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi, sorry fort he late answer. Entering Values on the watch is only possible with write with the fingers on the screen. Using oft he on screen keyboard ist not really usable cause you dont see what you are enter.
I send you some screenshots. I think for display on the watch the ui should be 50 percent from the normal ui.
What i see when i use the app:

Startup is slow. Needs about 30 seconds till i can see the ui
From touch the microphone until speak are about 5 seconds
When the watch is connected with bluetooth tot he phone the app does not work

The app can only work over WLAN. I had to disconnect bluetooth on the phone to get the app working.
Think the better way would be to have a companion app to the smartphone app. But this is sure to much work.
Another way perhaps is to have a MQTT Audio streamer that only streams the audio to the server over a companion app over bluetooth.
Greetings and thanks for your Work!
Von: razzo04<EMAIL_ADDRESS>Gesendet: Donnerstag, 21. Januar 2021 18:02
An: razzo04/rhasspy-mobile-app<EMAIL_ADDRESS>razzo04/rhasspy-mobile-app<EMAIL_ADDRESS>Cc: Jochen Entenmann<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Jochen Entenmann<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Betreff: Re: [razzo04/rhasspy-mobile-app] Wear OS App? (#4)
I’m glad it works. What problem did you encounter when entering values? What are your tips for improving the UI? At least for future people who will try to set-up the application, the process will be easier.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Thank you for sharing your experience.
Most likely the app is slow because it was compiled in debug mode and not release and so you could try to run the command "flutter run --release" or change the settings on Android studio.
I will try to resize the UI again to make visible what you are typing. Where did you send the images? I couldn’t find them.
I think that is too much work to develop any kind of companion app even for streaming only the audio. I will investigate why the app doesn't work if Bluetooth is enabled.
Thanks again for your time.

Thank you for sharing your experience.
Most likely the app is slow because it was compiled in debug mode and not release and so you could try to run the command "flutter run --release" or change the settings on Android studio.
I will try to resize the UI again to make visible what you are typing. Where did you send the images? I couldn’t find them.
I think that is too much work to develop any kind of companion app even for streaming only the audio. I will investigate why the app doesn't work if Bluetooth is enabled.
Thanks again for your time.

Hi, had to set up my pc new. Actual no flutter installed. Can youd compile the apk?
With the mails it seem there are no images treansfered to github.
I load it up here, hope it helps :-)

Hi, had to set up my pc new. Actual no flutter installed. Can youd compile the apk?
With the mails it seem there are no images treansfered to github.
I load it up here, hope it helps :-)

Thank you for sending the images, I will try to improve the UI.
I compiled the app in release mode and I realized that its size is 48 Mb so I guess I’ll have to do some optimizations but in the meantime, if you want to try it.
app-release.zip

Hi. Had installed the apk and it is mutch more quick.
But cant connect to Rhasspy. IP and Port are right and the watch has connected to the WLAN.
What a shame, it would have been a great solution to always have Rhaspy on your wrist.
thank you anyway for your effort

Sorry that it doesn’t work. in this period I’m very busy with various other projects which precludes me from being able to work on the app. As soon as I have some time I will try to improve it and apply the Bloc pattern.

Unfortunately, I have been very busy, I haven’t been able to work on it, and I haven’t used Flutter in a while, now for rhasspy I am developing a skill system to allow more easily to create and execute skills when I will finish, I will continue developing the app.

Hello razzo04, amazing! Thanks so much for your feedback! Let me know once we can do or test anything. Thanks, red

Hi razzo04, thanks for the feedback. It would be great if this will working some day :-) . I have searched also for a "Mic App".  With Rhasspy we only need a "remote mic" that streams the Audio with to the Server. So we dont had to speak loud across the room to the installed Rhasspy.

Sooner or later, I’ll be able to get back to it. However, I think maintaining two different code bases one for the phone version and another for Wear OS would take longer. I’ll see if I can bring back most of the features using the same code base if this doesn’t work I think I’ll separate the code of the two apps.

In the meantime Samsung released their Galaxy Watch4 which also runs with WearOS. Maybe there will be more interested people asking for this rhasspy satellite app.
Are there any news about this topic?
Red

In the meantime Samsung released their Galaxy Watch4 which also runs with WearOS. Maybe there will be more interested people asking for this rhasspy satellite app.
Are there any news about this topic?
Red

Gw4 user here, I'd definitely be interested in this functionality.

Same, also just got a Gw4 and this would be a perfect use for it.
Maybe a bounty on this issue ?

A Tasker plugin might do it. That way going through autowear -> tasker -> rhasspy on the phone would do the trick.
In the meantime I think it should be doable to re-setup phrases in AutoWear and have Tasker call homeassistant directly, bypassing rhasspy. It's more work but it's a way to get this working today, at least if your intents are going to HA.

Unless I'm mistaken, I don't think it's currently possible to give voice commands to AutoWear/Tasker on the GW4 since the Google Assistant is still MIA and Bixby isn't compatible.  The AutoWear Voice Screen just brings up "Please wait" and does nothing.

Well there is a way to get GA on the GW4 but I didn't realize AutoWear depended on it, you're right that's not a great solution. Guess for now I'll just have the watch trigger rhasspy on the phone

Actually, I hadn't bothered trying since @joeduck reported it not working, but it does.
I just installed the apk in the zip published above, it's a bit tough to configure as the keyboard seems to be really bugged with this app but after multiple tries I got the correct values in the fields and it works pretty well.
A secondary app that would automatically go in record mode would be a great addition, that way we could bind the double home button to it and we'd be all set ! I'm sure I can get AutoWear to launch the app and simulate a button tap for now.

Well there is a way to get GA on the GW4

The only current solution is a paid third party app that routes your authentication and all commands through a third party server.  And it doesn't work with AutoWear anyway because it isn't the native implementation of the assistant.

Before continuing to improve the compatibility with wear os I have to migrate the all code base to null safety and refactor most of the code ( This is what happens when using a new and rapidly developing framework ) this will require some time to do it.  Anyway, it is difficult to develop without a physical device so in any case will be a slow development.
I don’t know if the standards android widget is a thing in wear os  but if it this the app include a widget that when clicked it will start immediately recording.

I don't believe widgets exist, the equivalent would be either a complication (little circle you can display on some watch faces for things like step counts) or a tile. I know tiles weren't open to all developers until very recently so that's probably not the easiest route.
But I don't think either of these would be that useful for rhasspy, the ideal IMHO would be a secondary app in the menu, like tasker or autowear have.
That secondary app could then be bound to a physical button in the watch settings, so that pressing that button would bring up rhasspy in recording mode immediately.
Still, I'm using it now and pressing the microphone is fine, perfectly usable. I had to tick the checkmark to use an external dialogue manager to get it to be a bit more reliable. With the local manager the app randomly stops understanding any commands, with the remote manager sometimes pressing the microphone doesn't do anything until the app restarts. Behavior is essentially the same but with the remote manager it's easy to see when the app needs to be restarted since the microphone doesn't turn red.
I'm guessing that's probably a network issue, not sure, but it doesn't happen that often.
Thanks for the app anyway, very handy on the watch I have to say !

I’m glad that works.
Shouldn’t be difficult to create a secondary app that as soon as you start the app it begins listening, when I have some free time I will try to make the necessary changes.
it’s very probably a network issue as you said because with the local manager it doesn’t have to wait for a response from rhasspy to start listening, in fact, everything is handled by the app but with the external manager, I will start recording if rhasspy send a message to do so. Still, it is strange that the app didn’t notice that it lost connection to rhasspy. Have you been able to see the app logs?

Has there been any updates on this? I'm very interested in getting this on my watch. Where am I able to get the latest verison?

Im also very interested. Got the new Galaxy Watch 5 and a WearOS app would be really handy to control Home Assistant, without placing speakers all over my house.

@razzo04, is this project dead in the meaintime (last update v1.7.0 Dec 31,2020) or is there any further plans?

The project isn't dead but needs to be redesigned from scratch because when I initially started I haven't developed with maintainability in mind but it was more like a side project. I don't have much free time but if there will be enough requests I can start working on it again and try to make a stable wear os app, now should be simpler because I have a Galaxy Watch 4.

looking forward to seeing one.

https://github.com/BigBoot/GW4Remap will be a bit useful once you get it working.

I don't have a WearOS watch (yet) but something like this would be a motivating factor in deciding to buy one.  As to the comment about the UI not fitting on the watch.  Maybe make it so that the watch UI consists of nothing more than a microphone icon that the user can press (and maybe a couple smaller buttons to mute the microphone and speaker in case you happen to need to silence it temporarily).
Configuration could be done on the user's phone.


Feature: Export createEmptyEditorState and other LexicalEditorState functions

By default EditorState manipulation functions like createEmptyEditorState are not exported. It'd be great to have these also exported by default so it's easier to create and manage EditorState.

We don’t want people manually constructing EditorStates as it won’t work properly outside an update cycle and it won’t work with the reconciler properly. So it’s unlikely we’ll ever expose this function as it’s an implementation detail.

@trueadm how are we supposed to supply the initialEditorState argument to LexicalComposer if there is no way to construct an editor state...?


feat: Enrich addr info with remote addr info

It is able to get full 5-tuple info when hooking connect and accept syscalls.
$ sudo ./bin/ecapture tls -d
...
2024-12-08T16:44:24Z DBG AddConn success fd=5 pid=113767 tuple=<IP_ADDRESS>:37736-<IP_ADDRESS>:443
...

It shows the local addr info and remote addr info when curl https://<IP_ADDRESS>.
See #682 to get more context.
See eBPF Talk: 在 bpf 里将 fd 和 sock 关联起来 for design ideas and technical implementation.

	rport, lport := binary.BigEndian.Uint16(ce.Ports[0:2]), binary.LittleEndian.Uint16(ce.Ports[2:4])
	raddr, laddr := net.IP(ce.Addrs[0:4]), net.IP(ce.Addrs[4:8])
	ce.Tuple = fmt.Sprintf("%s:%d-%s:%d", laddr, lport, raddr, rport)

Compared with remote/local ip/port, the expression of src/dest ip/port is more in line with the usage scenario requirements. Can the information output be designed according to the src_ip:src_port-dest_ip:dest_port format?

Can the information output be designed according to the src_ip:src_port-dest_ip:dest_port format?

Can. But unnecessary, because of performance consideration.
As struct connect_event_t is already compact enough, it must swap positions of local&remote addr info in bpf when it's passive connection, in order to avoid adding more field to struct connect_event_t.
@cfc4n WDYT?

Can the information output be designed according to the src_ip:src_port-dest_ip:dest_port format?

as log 2024-12-08T16:44:24Z DBG AddConn success fd=5 pid=113767 tuple=<IP_ADDRESS>:37736-<IP_ADDRESS>:443 ,Isn't this the format you expected?

Yes, it is the expected format, but compared with local-remote, the source -destination format is more suitable for most business scenarios. The current format can also meet the requirements, we will consider the business layer and make the difference between source and destination. Thanks very much for the contribution.

Can the information output be designed according to the src_ip:src_port-dest_ip:dest_port format?

Can. But unnecessary, because of performance consideration.
As struct connect_event_t is already compact enough, it must swap positions of local&remote addr info in bpf when it's passive connection, in order to avoid adding more field to struct connect_event_t.
@cfc4n WDYT?

Add a direction field to indicate whether the session is initiated by the local . For example, 0: accept, indicating passive acceptance of the connection, and 1: connect, indicating that the host initiates the request. Whether this solution can be accepted?
AddConn success fd=5 pid=113767 direction=accept/connect tuple=<IP_ADDRESS>:37736-<IP_ADDRESS>:443

AddConn success fd=5 pid=113767 direction=accept/connect tuple=<IP_ADDRESS>:37736-<IP_ADDRESS>:443

It's unacceptable for me. However, it's easier to provide saddr:sport-daddr:dport than direction.


[stdlib] Delete unused gyb variable

This variable is no longer referenced anywhere.

@swift-ci test


feat(properties): support time as a property type converting to rfc3339

This adds support for time.Time as a property type when building from a struct. Removes the need to format it ahead of time and pass in as a string.

:tada: This issue has been resolved in version 0.18.0 :tada:
The release is available on GitHub release :rocket:


Migrate to 2021 edition and add examples

[x] I agree to follow the project's code of conduct.
[x] I added an entry to CHANGES.md if knowledge of this change could be valuable to users.

We could go right to 2021, I think?

Oh, I was just being conservative because nothing else in georust is 2021 yet

Might be a good place to start, since wkt isn't tested on older compilers :smile:.

We need something like:
    - name: Install Rust toolchain
      uses: actions-rs/toolchain@v1
      with:
        toolchain: stable
        profile: minimal
        override: true
        components: rustfmt, rust-src

Two last things I'd like to nail down:

a one-line doc comment for the Wkt struct (what's it for?)
a one-line doc comment for the Geometry enum (what's it for?)

@urschrei should this be merged?  There are two approvals, but no merge, so high chance of bitrot

Oh, yeah. I think I was just leaving it for a few days.

bors r+


Frontend of register page

Currently there is no register page, i can design and code register page for the site.

@kashnx This issue  already exists