New styles for footer actions

Here are the styles of this specific component:

And more generally of all dropdowns like this: https://www.figma.com/file/h67EsVXdWsfu38WGGxWfpi/Wagtail-Design-System?node-id=1241%3A18660
Ideally we’d only get to this after @wagtail/gsoc-ux-unification has implemented their changes to the JS side of this.
Note for footer actions – there is much more than just the dropdown, we also need to make sure the new styles work with the bulk chooser’s components, and with our dirty form check messages.
As part of this, we could also add this component to the user creation screen to help with #5906. Ideally #7366, #6591, #7650 will have also been fixed.
Sub-tasks:

[x] Re-implement dropdown buttons with Tippy
[x] Re-implement bulk actions + button styles according to new designs
[x] Include styleguide improvements for warning buttons https://github.com/wagtail/wagtail/pull/10449#discussion_r1221309056

This should ideally involve fixing #6591.

Hi @thibaudcolas I was wondering if this is still open for anyone to work on?

Hi @ForJessicaSake, yep, I don’t think anyone else has worked on this.

Okay great,  I’ll work on it can you please assign it to me 😊 thank you!

@ForJessicaSake  we do not assign issues (outside of core team) so feel free to just start on this.

Okay noted, I’ll do just that

Hi @ForJessicaSake, can you let us know where you got to with this? Please let us know if you have any questions.

Hi @ForJessicaSake are you still working on this please?

Hi @thibaudcolas, is anyone working on this issue?

@Seremba feel free to give this a go as we have not heard back from @ForJessicaSake for some time.

Hi @thibaudcolas @lb- would you mind helping me locate the file for these changes?

@Seremba have a look at this PR - it touches the same area of the UI.
https://github.com/wagtail/wagtail/pull/9643
However, it's good to learn how to find these things yourself. The best way is to look at the CSS classes in the DOM and search through the code in your editor.

I’ve done quite a bit of the work on this in #11456 – the footer actions code is much simpler now, so should be easier for us to complete this ticket. There were a few snags:

The proposed widths for the sticky actions would mean a lot more wrapping of text to two lines for languages longer than english (so I stopped at 310px rather than the 250px in the designs)
The designs in Ben’s original screenshot here have too low contrast between the footer actions background and the page background in my opinion. Not sure this is an actual issue but I found it hard on the eye to not know exactly what part of the page was sticky and what was within. Something I’d like to discuss at least.

And more generally:

Those styles need to work for both footer actions on "forms", and bulk actions. Currently we only have designs for very simple "form" footer actions, while the bulk actions are more complex visually
We need to make sure the footer actions also support having text within, the "unsaved page data" warning and also meta text for bulk actions.

Another snag was that footer actions markup currently is copy-pasted all over the place, with lots of inconsistencies. So as part of completing this I think we need to refactor the code to have an implementation that’s easier to reuse. Particularly in the context of #9869.

Assigning back to @benenright for a review of how we can tweak the designs to complete this, based on my notes above. Here are specific problem scenarios that might warrant design changes.
Footer actions with warning message

Bulk actions

Bulk actions with "Select all" link

I think that we should keep the background indigo, as I don't think there is much value in trying to make a light background work behind these actions, and the potential usability issues outway the original intention...
The original intention was to reduce the visual weight of the form footer actions - being something that sticks on the browser bottom edge, it felt like this was worth trying. But in the context of the bulk actions, that only appear once triggered by selecting a checkbox, the dark background helps users to notice it, and if we move to a lower contrast background, it risks them not being noticed (some usability testing we did a while back showed that some users expected these bulk actions to appear at the top of the listing (as per Gmail for eg), but since the indigo bg was high contrast with the page bg, people did notice it at bottom).
I do think it would be worth revisiting how we ensure good contrast for this type of panel specifically in dark mode (as the low contrast means we likely have that usability issue where people don't notice it appear):

@benenright I forgot one other variation on footer actions, which is used when viewing a past version of a page:

This is testable on this past revision demo page. Could you give us design guidance for that as well?

re bulk actions, anecdotally we received related feedback from low-vision users (though it’s low on details so I’m following up):

Given the page layout it wasn’t immediately apparent that it was possible to affect the pages from the list view

Hi
In this view, I don't know the history as to why the button is orange, but to me it feels like the top banner is doing enough to alert people this is a previous version of this page. I feel like for that reason we could just revert the buttons to be teal here...
However, maybe it's likely that people who have become used to seeing the buttons orange will get confused if they go back to teal). So for that reason it's prob safer just to address the fact that the white text on orange is not high enough contrast... and we should change it to use the same text colour as in the alert banner.

While we're here. could we fix this UI glitch where the button loses it's top left and top right border radius after you open then close the options:
Before:

After:

👍 sounds good to me that we’d have the button consistently teal. As you say the banner is already doing a good job warning users, and the button text is different as well. Also agree re keeping the indigo background in the light theme.
Will leave it with @laymonage to figure out where to place the "Delete" buttons we occasionally have in forms, that could either go in a "footer actions" dropdown, or in a "slim header" falafel actions menu 👀


Add GHCR support

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

157 of 326 (48.16%)  changed or added relevant lines in 7 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage increased (+8.8%) to 14.59%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/app/build.go
0
1
0.0%

pkg/app/visibility.go
0
6
0.0%

pkg/app/rm.go
0
14
0.0%

pkg/app/images.go
0
17
0.0%

pkg/extended_registry/extended_registry.go
17
49
34.69%

pkg/extended_registry/aserto.go
53
88
60.23%

pkg/extended_registry/ghcr.go
87
151
57.62%

Files with Coverage Reduction
New Missed Lines
%

pkg/app/rm.go
1
0%

pkg/app/visibility.go
1
0%

Totals

Change from base Build<PHONE_NUMBER>:
8.8%

Covered Lines:
356

Relevant Lines:
2440

💛  - Coveralls


Release 3.1

Fixes #

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


Update to use uint64 APIs for priority

Use new apis for C client API
Updated comments for priority to indicate unsigned int parameter

Should we update the java client to change from int to long as well?
https://github.com/triton-inference-server/client/blob/99f101c31959569fc060b08366213167a0b63811/src/java/src/main/java/triton/client/InferenceServerClient.java#L350


node (vs. nodejs) is required when installing electron

If you're having an issue installing Electron, this is the place to report it.
If you're having an issue using Electron, please report it at https://github.com/electron/electron/issues

Electron version: 1.6.11
Operating system: Ubuntu 17.04

Trying to install electron with out nodejs-legacy, yields an error (node not found).
$ sudo npm install -g electron
After installing nodejs-legacy, installation succeeded.

GitHub issues are for feature requests and bug reports, questions about using Electron should be directed to the community or to the Slack Channel.
FWIW this behavior is expected, npm relies on node so it won't work without it

I suspect I'm misunderstood about the essence of this issue. This is not a question or a support issue. It is definitely a feature request, arguably a bug.

Node is installed on my machine.
On Debian and Ubuntu distribution, the node executable name is nodejs. See issue #4772 on NodeJS database (https://github.com/nodejs/node-v0.x-archive/issues/4772).
I had no problem workaround the issue. The canonical work around is to create a symbolic link from node to nodejs - and this way supporting packages that have 'node' hard coded, including electron.
I seriously suggest you reopen this issue. Most packages support it. Ubuntu is not a small distribution.

@uri247 I'll cc some people on this to discuss but as far as I am aware the nodejs package on linux is legacy and unsupported.  The version it installs 0.10 / 0.12 is actually end of life according to node's LTS schedule.  And all common node install techniques for linux (that aren't this legacy package) install the correct executable (node).
I am of the opinion that this is an issue with how node was installed in the legacy package and not an issue with electron calling node install.js.
What do you think @malept ?

Ubuntu 17.04 installs Node 4.x (has been since at least 16.04). That being said, it's really odd that nodejs-legacy exists (the package that provides the nodejs :arrow_right: node symlink), from a Node ecosystem point of view. (It makes slightly more sense from a Debian packaging POV...)
My opinion is that we should add to the docs "requires nodejs-legacy installed when using the system Node packages on Debian/Ubuntu."

My opinion is that we should add to the docs "requires nodejs-legacy installed when using the system Node packages on Debian/Ubuntu."

Sounds good to me 👍


Request: Prepare docker image on docker.hub

It would be really nice, to be able to run pyvisualcompare in docker (downloading from docker.hub)

Yes, good idea. Depending on the host system it might be a bit problematic due to the GUI - e.g. for systems without an X server. I'll have to try that out. What are you trying to run it on?

Mac

Just to check, do you intend to run the frontend on MacOS or the backend (with urlwatch)?

backend of course

I added a test image as nspohrer/pyvisualcompare, you're welcome to try it out.
Try something like the following (obviously you can change the URL and other parameters):
docker run -it -u qtuser nspohrer/pyvisualcompare /app/pyvisualcompare-md5.sh --crop-x 8 --crop-y 5 --crop-w 232 --crop-h 58 heise.de
For other people trying to run the frontend:
Under Linux (with X11), the following works for me to run the GUI:
docker run -it -v /tmp/.X11-unix:/tmp/.X11-unix -e DISPLAY=$DISPLAY -u qtuser nspohrer/pyvisualcompare python3 /app/main.py
Under MacOS you probably need to install XQuartz and play around a bit with the parameters, see e.g. here.

Closed as it's added to the README and no error was reported back


Enable execution of test cases - CI builds.

And we have our first successful CI execution on both platforms using the new CI folder from the testing credentials repo!
https://travis-ci.org/IBM-Swift/BluePic/builds/167389193
Note that Tests are only running on OS X. Opening new issue to track progress of tests on Linux.


Automated cherry pick of #3728: Fix identification of control-plane Node in e2e test

Cherry pick of #3728 on release-1.6.
#3728: Fix identification of control-plane Node in e2e test
For details on the cherry pick process, see the cherry pick requests page.

Codecov Report

Merging #3739 (cdcdc54) into release-1.6 (49d4253) will decrease coverage by 6.30%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           release-1.6    #3739      +/-   ##
===============================================
- Coverage        64.12%   57.82%   -6.31%
===============================================
  Files              278      278
  Lines            27839    27839
===============================================
- Hits             17853    16098    -1755
- Misses            8078     9954    +1876
+ Partials          1908     1787     -121

Flag
Coverage Δ

kind-e2e-tests
39.17% <ø> (-12.00%)
:arrow_down:

unit-tests
43.34% <ø> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

...g/agent/apiserver/handlers/featuregates/handler.go
0.00% <0.00%> (-82.36%)
:arrow_down:

pkg/agent/controller/networkpolicy/reject.go
0.00% <0.00%> (-82.09%)
:arrow_down:

pkg/agent/controller/networkpolicy/packetin.go
0.00% <0.00%> (-68.37%)
:arrow_down:

pkg/apis/controlplane/v1beta2/helper.go
33.33% <0.00%> (-66.67%)
:arrow_down:

pkg/controller/networkpolicy/mutate.go
0.00% <0.00%> (-62.07%)
:arrow_down:

pkg/controller/egress/store/egressgroup.go
1.38% <0.00%> (-58.34%)
:arrow_down:

...kg/apiserver/registry/system/supportbundle/rest.go
20.45% <0.00%> (-53.41%)
:arrow_down:

pkg/apiserver/handlers/webhook/mutation_crd.go
0.00% <0.00%> (-52.64%)
:arrow_down:

...ver/registry/controlplane/nodestatssummary/rest.go
50.00% <0.00%> (-50.00%)
:arrow_down:

pkg/support/dump.go
8.19% <0.00%> (-49.19%)
:arrow_down:

... and 70 more

/skip-networkpolicy
/skip-conformance
manually verified above tests.
/test-e2e


Decide common storage format

JSON flat file currently

Think leaving it an a flat file is fine for now. Maybe different directories for Majors and Courses. Also possible to split up by department, but may make searching more complicated.


topology definition not taking effect in local mode [tune #ackers and parallelism

When running in local mode [pyleus local  *.jar], the definition such as:
workers: 4
ackers: 4
in pyleus_topology.yaml file does not take effect. I need to change apache-storm-0.9.2-incubating/conf/storm.yaml in order to change the setup. Is the behaviar normal?
I have a storm only setup in local mode in a machine with 4 cores. My topology is running really slow. I am trying to tune the topology parallelism to see if I can speed things up in local mode. Is this even possible to do in local mode?
Thanks

Pyleus does not pass many options to local topologies because we considered local runs as a debug feature, not a "real" operating mode. And this is the feeling I got from discussions happened within the Storm community as well.
However, that should be "theoretically" possible to pass those options in local mode and you can do that easily modifying PyleusTopologyBuilder.java.  Feel free to submit a pull request after you tested this is doable and do not cause any harm to local runs, if this is a feature really important for you. I would also suggest to set a reasonable value for Config.TOPOLOGY_MAX_TASK_PARALLELISM, as mentioned in https://storm.apache.org/documentation/Local-mode.html

@n1epan were you able to speed thing up in local mode?


How to use on client side?

got an error when try running on windows nodejs.
c:\xampp\htdocs>node server.js
file:///c:/xampp/htdocs/server.js:3
import { Client } from "youtubei";
^^^^^^
SyntaxError: Identifier 'Client' has already been declared
←[90m    at Loader.moduleStrategy (internal/modules/esm/translators.js:145:18)←[39m
←[90m    at async link (internal/modules/esm/module_job.js:63:21)←[39m
c:\xampp\htdocs>node server.js
file:///c:/xampp/htdocs/server.js:1
const { Client } = require("youtubei");
^
ReferenceError: require is not defined in ES module scope, you can use import instead
This file is being treated as an ES module because it has a '.js' file extension and 'c:\xampp\htdocs\package.json' contains "type": "module". To treat it as a CommonJS script, rename it to use the '.cjs' file extension.
at file:///c:/xampp/htdocs/server.js:1:20
←[90m    at ModuleJob.run (internal/modules/esm/module_job.js:169:25)←[39m
←[90m    at async Loader.import (internal/modules/esm/loader.js:177:24)←[39m
←[90m    at async Object.loadESM (internal/process/esm_loader.js:68:5)←[39m
How can this actually being used in client server?

This library is made for Node only.
Read more here: https://github.com/SuspiciousLookingOwl/youtubei/issues/20#issuecomment-881393450


Why are dependencies linked to Github instead of NPM and semver?

Wondering why this library and most (all?) the libraries it depends on are pulling in Github dependencies?
They currently don't support semver versioning, and this leads to unpredictable results.
For instance this change from 2 days ago in a downstream dependency broke react-masonry-component for us. We had to shrinkwrap to enforce version numbers from within git dependencies.
https://github.com/desandro/matches-selector/commit/7582db4413229cdbe36bfe2b6228b210296b6d3d
What are the thoughts on moving dependencies over to npm?

Hi @afram - this whole module is a bit hacky as you have experienced.
The reason for the github deps is that the ones that originally were on npm (including masonry itself) didn't work with CommonJS.
To make the whole thing work I had to clone the whole dependency tree (and some forks that had implemented CommonJS/npm support) and make the necessary changes.
Also - because of mixed use of AMD/require.js (bower) and CommonJS (npm) it didn't work with Webpack (defaults to the first supported format, which when using the UMD boilerplate is AMD). If you look at the forks and the last commit(s) you'll see what i mean.
If you want to help clear things up and get a working version which uses only npm I'd be happy to merge it / make you a collaborator with access :-)
For more info check out these issues:

https://github.com/eiriklv/react-masonry-mixin/issues/1
https://github.com/eiriklv/react-masonry-mixin/issues/4

@eiriklv That solution might be the way to go - It's a few lines of webpack config (which sucks) but we get a lot more benefits from semver and maintained dependencies.
Thinking it'd be good to create 1x and 2x branches in case support is needed for legacy versions, and bump this to v3 - what do you think?

@eiriklv That solution might be the way to go - It's a few lines of webpack config (which sucks) but we get a lot more benefits from semver and maintained dependencies.
Thinking it'd be good to create 1x and 2x branches in case support is needed for legacy versions, and bump this to v3 - what do you think?

@afram As long as it's documented in the README and we branch off - then I'm good. Go for it if you have the capacity :-)

@afram As long as it's documented in the README and we branch off - then I'm good. Go for it if you have the capacity :-)

fixed in 3.0.0


Add external link for Explain how this works in JavaScript question

Hi @yangshun,
I just a small external link for question Explain how this works in JavaScript. But It has a very good explanation about this and also for apply , call, bind and use strict.

Sorry but I think this link is too lengthy. The codeburst link is concise and covers the same content IMO. Please reopen if you disagree.

Thanks @yangshu. It depends on your opinion. I respect that 😄


Add German translation

A promise made is a promise kept.
Don't be frightened, I'm done with translations now :)

Thank you!

@cbioley And the last task. Can you please localize /firebase messages? Thank you!

@steida Sure Daniel! Just give me some time, because I'm rather busy (like all of us :)) ATM.

@steida Here's what I've done so far: https://github.com/cbioley/este/commit/2213bb89b33ded8404ff4bcbea3f8a7cf126ea02
I localized all errors I found here. I initially wanted to put them out of Login component but it breaks because this is not what the babel extract plugin is expecting.
If the coding style is off, I'd be glad to get your inputs :)

Awesome! Btw, messages can be out of a component. Actually, we have it already https://github.com/este/este/tree/master/src/common/todos

@cbioley Can you do #811 too?

@cbioley Any chance on the German for #811?


Do not do unnecessary escapes

Apt does handle downloads correctly without these modifications on the "Packages" manifest.
Also, the usage of different methods on encode and decode lead to interesting problems with corrupted urls.
Fixes #11 .
Tested on Debian bookworm Docker container running in CircleCI. Generated:
Package: cutie-shell
Version: 0.0.3+git20221218211230.9147c27.bookworm
License: unknown
Vendor: none
Architecture: arm64
Maintainer: Erik Inkinen<EMAIL_ADDRESS>Installed-Size: 3077
Depends: libc6 (>= 2.34), libgcc-s1 (>= 3.0), libqt5core5a (>= 5.15.1), libqt5dbus5 (>= 5.0.2), libqt5gui5 (>= 5.0.2) | libqt5gui5-gles (>= 5.0.2), libqt5qml5 (>= 5.1.0), libstdc++6 (>= 4.1.1), cutie-settings-daemon, qml-module-qtquick2, qml-module-qtquick-controls, qml-module-qtquick-controls2, qml-module-qtsensors, qml-module-qtwayland-compositor
Section: libs
Priority: optional
Homepage: http://nourlgiven.example.com/
Filename: pool/bookworm/c/cu/cutie-shell_0.0.3+git20221218211230.9147c27.bookworm_arm64.deb
Size: 1864084
SHA1: e16c6b0c25a83acf7cbd3f21d5fabbae304cc80e
SHA256: e36f63739c99defc2d237eaaa954354173309f96c9dc15c3600a4238b3cce8c4
MD5sum: 8c3d38625153625c5e1ea7431f9c26ee
Description: This package contains Cutie Shell.
Multi-Arch: foreign

Installation shows fetch working:
droidian@droidian:~$ sudo apt install cutie-shell
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
The following NEW packages will be installed:
  cutie-shell
0 upgraded, 1 newly installed, 0 to remove and 0 not upgraded.
Need to get 1,864 kB of archives.
After this operation, 3,151 kB of additional disk space will be used.
Get:1 http://deb.cutie-shell.org/staging bookworm/main arm64 cutie-shell arm64 0.0.3+git20221218211230.9147c27.bookworm [1,864 kB]
Fetched 1,864 kB in 0s (4,586 kB/s)
debconf: unable to initialize frontend: Dialog
debconf: (No usable dialog-like program is installed, so the dialog based frontend cannot be used. at /usr/share/perl5/Debconf/FrontEnd/Dialog.pm line 78, <> line 1.)
debconf: falling back to frontend: Readline
Selecting previously unselected package cutie-shell.
(Reading database ... 97906 files and directories currently installed.)
Preparing to unpack .../cutie-shell_0.0.3+git20221218211230.9147c27.bookworm_arm64.deb ...
Unpacking cutie-shell (0.0.3+git20221218211230.9147c27.bookworm) ...

Thank you!


Upgrade Privacy Services Manager to v1.6.6

Thank you for the contribution. It was merged directly as 1ff2815b737025cf4226e4ce63c0141e5fc70ba0 to keep commit history cleaner. Your contribution is still credited to you.


DS18B20 API Improvements

The original DS18B20 API was functional, but not ideal for working with multiple sensors on the same bus.  In addition to a bunch of misc cleanup (use of open-drain mode, removing unnecessary critical sections, etc), this update adds several new API functions for identifying and accessing DS18B20 devices in more flexible (and much faster!) ways.

This looks very good to me. I also like that it moves away from function wrapper macros and has more typedefs for known types.
@hetii, @darksv  - you both used the older DS18B20 API. Any comments regarding merging this?

From my side all looks fine, especially when works faster ;)
BTW, thanks for info that new API is available.
Regards.


Accessor registration

Our project at https://github.com/intake/awkward-pandas is building integrations to dataframe libraries, allowing vectorised processing of nested, variable-length data structures via awkward, i.e., deeper data types than usually handles by dataframe APIs.
Awkward itself boasts, aside from vectorised kernels:

numpy-like API (e.g., slicing or aggregations on any index)
CPU and GPU kernels
numba integration
"behaviours" which give something like methods to records of known structures (think operations on vectors, IP addresses or polygons)

It's internal memory model is very close to arrow and interops nicely.
As the name suggests, we trialed awkward-pandas for pandas first, but now we have integrations of various completeness:

pandas
dask.dataframe
polars
cuDF
(and will therefore be looking for a new name)

These integrations follow the accessor pattern: df.ak.* or series.ak.* gives you the awkward namespace and slicing, e.g.:
df = ...
df["col1"].ak.sum(axis=2) # performs awkward-style sum on inner axis, returning series
df.ak[..., -1] # select last item in inner most list of every column

So, integrating with daft would be the first chance to bring this style of processing to Ray. As with dask.dataframe, making calls that work row-wise or partition-wise is easy; aggregations require first intra-partition operations, followed by some tree reduction; and inter-partition operations are simply hard. (The dask-awkward project shows we are working on this too, from the other direction. This would not be where to start any cooperation!)

Thanks @martindurant
I think exploring integrations on a per-row level for our List/FixedSizeList/Tensor types could be interesting. A slicing API for these types would be quite useful I think.
Does an integration require adoption of the awkward in-memory model though? That might be challenging because Daft has its own in-memory data representations as well.

As long as you can pass and consume [py]arrow, all is good! That's exactly what the polars integration does, which also uses arrow as the internal representation. And this would be per-row or per-partition operations.

I think exploring integrations on a per-row level for our List/FixedSizeList/Tensor

I should add, that the typical use case is for deeper nested things, containing lists and structs many levels down. You can already do some certain things with the existing list type. For example, a geometry object may be represented as a list of list of records array<[[x, y]]> or record of lists of lists array<[[x]], [[y]]>.


[Bug]: When trying to display model info, all models info is displayed

Is there an existing issue for this?

[X] I have searched the existing issues

Deploy type
Downstream version (eg. RHODS 1.29)
Version
RHODS 1.32.0
Current Behavior
When a User clicks on the number which represents the number of quantity of model which are deployed in a model server (in the project view), if the user has more than one model servers, the info of all deployed models (even in different servers) will be displayed.
The same happens if any of those servers has the Tokens enabled. All of them will open at the same time, showing the alert 'Tokens Disabled' in the servers which has the tokens disabled.

Expected Behavior
Clicking in one model info should show just the info of the model clicked.
Same should happen with the tokens.
Steps To Reproduce

login (as an admin) and go to "Data Science Projects" -> "Create new project". Fill the name of the project.
Inside the project config, go down to "Models and model servers"
Create one model server without Token
Create one model server with token
If the user clicks on the "1" displayed in the Tokens column (in the "Models and model servers" grid) all token info of both mode servers will be displayed
If the user clicks on one of the zeroes displayed on the "Deployed models" column (in the "Models and model servers" grid) , it will display the message "No deployed models" in both servers.
If we have deployed a model in one (or both) of the servers, then the info of the model will be displayed (e.g.: the endpoint) instead of the  "No deployed models" message

Workaround (if any)
No need, it's an UX bug which does not block any info or data, but it's a little bit unintuitive.
What browsers are you seeing the problem on?
Chrome
Anything else
No response

FWIW, this is a regression. The UI used to handle the row expansion properly.

Ah yes, this is totally a regression, I should have realized that cause we had this issue initially, add a fix by having the state inside the component and reverted back here https://github.com/opendatahub-io/odh-dashboard/pull/1646
@DaoDaoNoCode curious why we moved the state up in the component, was it related with the scroll behaviour?

@lucferbux It's because we want to expand the Deployed models tab when the user deploys a new model to indicate the changes in the model numbers. Refer to the code here https://github.com/opendatahub-io/odh-dashboard/pull/1646/files#diff-bc437fc442f3702adee24b7c009181e5fc76d0c6156bbe770c852f326088e1c2R93. That's why we lift it up. I can look into a solution here after this ticket is triaged.

@lucferbux It's because we want to expand the Deployed models tab when the user deploys a new model to indicate the changes in the model numbers. Refer to the code here https://github.com/opendatahub-io/odh-dashboard/pull/1646/files#diff-bc437fc442f3702adee24b7c009181e5fc76d0c6156bbe770c852f326088e1c2R93. That's why we lift it up. I can look into a solution here after this ticket is triaged.

Okk, that makes sense, perfect, we can check a solution after triage!


Add git & GitHub resources to README

Link to MozWOW resources.
Set expectations

Make sure to explain philosophy of "Just do it" rather than learning everything from first principles.

https://mozilla.github.io/open-leadership-training-series/articles/github-for-collaboration/

https://github.com/oliviaguest/git-and-github-cheat-sheet

http://rik.smith-unna.com/command_line_bootcamp


[BUG]Delete Setting ID

delete setting id.Make sure it's new every time
#12078

Codecov Report

Merging #12096 (9ee4232) into dev (08a4c79) will increase coverage by 0.02%.
The diff coverage is n/a.

:exclamation: Current head 9ee4232 differs from pull request most recent head 1931d5f. Consider uploading reports for the commit 1931d5f to get more accurate results

@@             Coverage Diff              @@
##                dev   #12096      +/-   ##
============================================
+ Coverage     38.71%   38.73%   +0.02%
- Complexity     4008     4011       +3
============================================
  Files          1001     1001
  Lines         37361    37360       -1
  Branches       4260     4260
============================================
+ Hits          14463    14471       +8
+ Misses        21253    21240      -13
- Partials       1645     1649       +4

Impacted Files
Coverage Δ

...api/service/impl/ProcessDefinitionServiceImpl.java
31.72% <ø> (+0.02%)
:arrow_up:

...erver/master/processor/queue/TaskEventService.java
75.00% <0.00%> (-5.36%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
52.85% <0.00%> (+1.42%)
:arrow_up:

...org/apache/dolphinscheduler/remote/utils/Host.java
44.68% <0.00%> (+2.12%)
:arrow_up:

...r/plugin/registry/zookeeper/ZookeeperRegistry.java
50.00% <0.00%> (+6.45%)
:arrow_up:

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

related pr: #11765


[WIP] Create Anchor Getting Started example docker-compose configuration.

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
[TODO: Short statement about what is changing.]
Why
[TODO: Why this change is being made. Include any context required to understand the why.]
Known limitations
[TODO or N/A]

Reference Server Preview is available here:https://anchor-ref-pr742.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr742.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr742.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr742.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr742.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr742.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr742.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr742.previews.kube001.services.stellar-ops.com/

Something went wrong with PR preview build please check

Reference Server Preview is available here:https://anchor-ref-pr742.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr742.previews.kube001.services.stellar-ops.com/


Screen cast mode - move the location of the displayed text

it may be helpful to move the location of where the text is displayed based on the screen, layout of the code, and other factors. I would love to see a setting for this at least giving a y coordinate.

I'd like to tackle this one if it's still available @joaomoreno

@joaomoreno Just submitted a PR for this one.  Sorry for the delay!

Vertical offset works fine -> verified.
Also feels like bloat.


What is the point of keeping the older Daedalus versions on disk?

I have noticed that several older versions of Daedalus are kept under the .local/share/Daedalus/mainnet/Downloads folder (Linux). Daedalus is already taking up a huge space on the disk. Especially considering that each version is like ~450Mb and an older version won't work anyway, can we not get rid of the olders once an upgrade to the latest version is successfully completed?

It has been agreed that these should be deleted on all platforms.
Can you confirm, @daniloprates?

@kedicesur Thank you for your report. The issue would be fixed in the upcoming release.

This has been fixed in the latest testnet release and coming in the next Daedalus mainnet release. The PR that solved it is: https://github.com/input-output-hk/daedalus/pull/2941


Button Option Enhancements

Describe the solution you'd like
Request to add more features for the Buttons:

Buttons with border off
Buttons with border on + circular shape + radius + border color + custom background on or off
Buttons with border on + rectangular shape + corner radius + border color +  custom background on or off

I think you have text color already, so that's covered for the buttons. The custom background can just be a custom color configuration.

Will now offer various level of controls over the buttons.

Set corner family and radius for negative and positive button in general
Set button style of negative and positive button (outlined button, text button, normal colored button)
Set corner family and radius of only positive and negative button, plus fine control for each corner of each button


Implementing dependabot

I just realized as well that in https://github.com/pyansys/pyfluent-parametric you do not have dependabot implemented. This is a good way of keeping up-to-date our requirements. And following https://github.com/pyansys/pyfluent/pull/703 it is also easy to keep up with GH Action  updates (they are also important to keep up to date).
If for any reason you do not want to set up dependabot in this repository just close this PR =)

Same here, I'd need another reviewer for this PR (when possible 😃)


Feature/bbss 4 d

Description
Implemented strong-strong 4D beam-beam with the pipeline following the 6D implementation
Closes # .
Checklist
Mandatory:

[ ] I have added tests to cover my changes
[ ] All the tests are passing, including my new ones
[ ] I described my changes in this PR description

Optional:

[ ] The code I wrote follows good style practices (see PEP 8 and PEP 20).
[ ] I have updated the docs in relation to my changes, if applicable
[ ] I have tested also GPU contexts

@xbuffat thanks for this. Could you please add a test in the test suite for this feature?

Thank you! We need to add a test to the suite before this can be merged

Done!

Once this is merged, issues https://github.com/xsuite/xsuite/issues/237 and https://github.com/xsuite/xsuite/issues/236 can be closed


[ISSUE #7164] Refactor Controller with asynchronous event handling architecture

Which Issue(s) This PR Fixes

Fixes #7164
Brief Description

How Did You Test This Change?

Codecov Report

:exclamation: No coverage uploaded for pull request base (refactor_controller@3a6ef04). Click here to learn what that means.
The diff coverage is n/a.

@@                  Coverage Diff                   @@
##             refactor_controller    #7165   +/-   ##
======================================================
  Coverage                       ?   42.56%
  Complexity                     ?     9398
======================================================
  Files                          ?     1163
  Lines                          ?    82185
  Branches                       ?    10671
======================================================
  Hits                           ?    34982
  Misses                         ?    42855
  Partials                       ?     4348

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


[python] Make TileDBCreateOptions suitable for human consumption

This updates the TileDBCreateOptions type to have attributes for each of the settings it provides.  This means that a user can construct one using normal-looking Python syntax, without having to build their own JSON-like structure to be re-parsed.  TileDBCreateOptions is now exported from tiledbsoma.options.
It also enables passing a TileDBCreateOptions directly as the platform_config value in creation operations, implementing the pattern that was recently legalized by the spec change on https://github.com/single-cell-data/SOMA/pull/165 and https://github.com/single-cell-data/SOMA/pull/166.

Python half of #1394.

Codecov Report
Patch coverage: 94.26% and project coverage change: +26.12 :tada:

Comparison is base (1169798) 65.30% compared to head (6ec591d) 91.43%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1408       +/-   ##
===========================================
+ Coverage   65.30%   91.43%   +26.12%
===========================================
  Files          98       30       -68
  Lines        7959     2486     -5473
===========================================
- Hits         5198     2273     -2925
+ Misses       2761      213     -2548

Flag
Coverage Δ

python
91.43% <94.26%> (+0.08%)
:arrow_up:

r
?

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

Impacted Files
Coverage Δ

apis/python/src/tiledbsoma/_collection.py
93.53% <ø> (-0.04%)
:arrow_down:

apis/python/src/tiledbsoma/_common_nd_array.py
97.82% <ø> (-0.05%)
:arrow_down:

apis/python/src/tiledbsoma/_dataframe.py
96.96% <ø> (-0.02%)
:arrow_down:

apis/python/src/tiledbsoma/_util.py
91.66% <ø> (+0.51%)
:arrow_up:

...n/src/tiledbsoma/options/_tiledb_create_options.py
92.91% <94.01%> (+1.55%)
:arrow_up:

apis/python/src/tiledbsoma/io/ingest.py
86.14% <100.00%> (-0.06%)
:arrow_down:

apis/python/src/tiledbsoma/options/__init__.py
100.00% <100.00%> (ø)

... and 68 files with indirect coverage changes

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


DataTable: add 'display' attribute on DataTableCell when using slots

I miss a feature when passing components to cell slots on DataTable.
When using DataTable you can create a header object with a 'display' attribute where you can place a function to format you data before showing. This features is very handy but ifyou pass a component to slot 'cell', the let:cell doesn't have this attribute. So you have to format it inside of the component you are passing. It would be nice if 'display' attribute be passed to let:cell.
I've done that by changing line 160 on DataTable.svelte file:
from:
  $: rows = rows.map((row) => ({     ...row,     cells: headerKeys.map((key) => ({ key, value: row[key] })),   }));
to:
$: rows = rows.map((row) => ({     ...row,     cells: headerKeys.map((key, index) => ({       key,       value: row[key],       display: headers[index].display,     })),   }));
Then on lines 368-369 and 381-382 I've replaced headers[j].display to cell.display. Also I've updated the types.ts and the jsdocs for the DataTableCell Object.
Is it fine to consider a PR?

Can you provide some sample code to better illustrate this?

Sure.
const formatDate = (date) => new Date(date).toLocaleString()
const headers = [
        { key: 'taskName', value: 'Task Name'},
        { key: 'creationDate', value: 'Creation Date', 'display': formatDate},
    ]
const rows = [
        { taskName: 'MyTask', creationDate:<PHONE_NUMBER>652 }
    ]

<DataTable {headers} {rows} />

<DataTable {headers} {rows}>

        {cell.value}
        {cell.display(cell.value)}

</DataTable>

Of course I could use the function directly on the component that I'm passing:
<DataTable {headers} {rows}>

        {formatDate(cell.value)}

</DataTable>

But since we may passed this value on the header object, it would be handy to have it in the cell object.

@rafadess Excellent – I understand completely and think this would be very convenient. Looking forward to the PR.


Open source project by Language

Description
Can we add a category called Languages and have an option to search project by language. Today if i am infested in python projects i need to open each and every project listed here and see what is the language that project is using which i feel is an overhead.

Screenshots
No response
Additional information
No response

@Praxkv1 This sounds like a good idea. However many projects use multiple languages. Instead, we could add what tech stack it uses in front of the project. How does that sound?

@siddhant-khisty Sounds good.

@Praxkv1 This sounds like a good idea. However many projects use multiple languages. Instead, we could add what tech stack it uses in front of the project. How does that sound?

@siddhant-khisty @Susmita-Dey
Yes this is a good catch. Is there a way we can add tags to the project? if so we can add programing language as a tag along with the percentage of that specific programing language being used in that project.
For example if
Project1 is built using 60% pythonand 40% Typescript
Project2 is built using 40% python and 60% Typescript
Then when some one search for project of type Python then you show them Project1 first then Project2 (result sorted by % of language used in descending order)

@Praxkv1 percent won't be a good idea as there is no accurate way I know which could derive the percent of language used in the project. Additionally, GitHub doesn't provide such feature (If you know about any please let us know). If you click on the project it will be redirected to the project repo where you are most likely to find languages used. Please feel free to open new issue if you come across any solution 😄


HSEARCH-4872 Rebuild only some of the modules to speedup the build

https://hibernate.atlassian.net/browse/HSEARCH-4872

Thanks.
We'll need to check that hibernate-search-integrationtests-backend-lucene actually gets executed even when you add those options; can you please check that?
Merging.

I've tried to run it locally:
./ci/list-dependent-integration-tests.sh hibernate-search-backend-lucene

has the hibernate-search-integrationtest-backend-lucene  in the list, and then I've just added -pl with the output of this script + another -pl for the list from additionalMavenArgs -- it even passed the tests 😃 but we probably would want to double-check it when it runs on CI. So I'll look at the results once we have a next run

sooo it seems that it worked 🎉 😄
[INFO] Hibernate Search ITs - Backend - Lucene ............ SUCCESS [01:37 min]

https://ci.hibernate.org/blue/organizations/jenkins/hibernate-search-dependency-update/detail/main/110/pipeline/444


SSH now functional

Still can't connect to 20forbeers.com:1338 :
Fatal: kex error : no match for method encryption client->server: server [aes128
-cbc,3des-cbc], client [chacha20-poly1305@openssh.com,aes256-gcm@openssh.com,aes
128-gcm@openssh.com,aes256-ctr,aes192-ctr,aes128-ctr,]
I reverted my ssh2 change - ssh2 seems unmaintained and can't connect to 20forbeers as well. But with a different error. However that doesn't matter.


Fix QuantileForecast.plot() to use DateTimeIndex

Issue #, if available: #2268
Description of changes: Changed plotting function from plt.plot() to Series.plot() to be consistent with the superclass and index to DateTimeIndex.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Please tag this pr with at least one of these labels to make our release process faster: BREAKING, new feature, bug fix, other change, dev setup

Thanks @abdulfatir! Would it make sense to add a test for this? There are some tests in https://github.com/awslabs/gluon-ts/blob/dev/test/model/test_forecast.py#L30-L40, I'm assuming that trying to plot the QuantileForecast given there would fail prior to this fix?


Programmatically config

Issue link
Refs: #184
Auto-
Types of changes
Bug fix (non-breaking change that fixes an issue)
New feature (non-breaking change that adds functionality)
Breaking change (fix or feature that would cause existing functionality not to work as expected)
Meta (a change that affects future changes to this repo)
Technical debt (a code change that doesn't fix a bug or add a feature but makes something clearer for devs)
Notes
After working a while trying to solve keeping config files (WIP: https://github.com/marcelomorgado/TasitSDK/tree/feature/config-cut-out) I've given up for now.
I've tried some modules available for react-native, seems that works fine on a single app project but they didn't work for our dependency tree:
tasit/demo (react-native) -> tasit-sdk -> tasit-action (should have module installed).
My solution, for now, is to expose the singleton ConfigLoader class that should receive a config object.
It's a very flexible API, I want to discuss how the best way for our users does a programmatically config. Since I think that even we solving a config file issue, we will keep programmatically way.

Interesting about the dependency tree affecting it.
Exposing the singleton for now makes sense and should work.
Agreed that we'll keep the programmatic way regardless. Sure, happy to discuss the best way to structure that API!

Other than that I think it's ready to merge


docs(readme): update usage steps

Make them clearer.
Change-Id: I63950a40dce7028fdbec88091480714750c86ca3
Depends-On: #210

This pull request is part of a stack:

docs(readme): update command options (#210)
docs(readme): update usage steps (#211)

@mergify refresh


[F764.5] Add firrtl.options package, but do not use

This adds a new package, firrtl.options, that provides a framework for working with options inside FIRRTL and in other frontends, backends, or toolflows. This treats all options, metadata, etc. as an Annotation. An AnnotationSeq then forms the ground truth for all options being passed to a toolflow.
This is the necessary framework for the "Options are Annotations" refactor, #764. However, this does not use any of the firrtl.options package! The existing ExecutionOptionsManager approach is left unchanged and continued to be used.
For this, the new Stage will replace the old Drivers. In parallel to this, I'm working on a compatibility layer that keeps the old Drivers intact.
Patch 5 for #764.
Merge strategy: @seldridge will merge via merge commit after a rebase

Ping @azidar. I need to write the compatibility layer, but this is good for a review (more from a software architecture perspective if this is the model that we want to move towards).

Thanks for the review, @chick.
Nah, is okay. I can take care of the style stuff.
For the deprecated things, I'll need to take a look. I think what was going on is that I intended to write a compatibility layer in firrtl.options and then realized that this could be entirely hoisted out. I'll check on that now.

@azidar: Thanks for the review!
@chick: Let me know if you're okay with this. I changed ExecutionOptionsManager to Shell. Short and sweet and also mostly correctly as this is what provides a user interface to a stage. No substantive changes otherwise, except to address feedback.

Ping @chick. Trying to see if all the FIRRTL stuff re: annotations/options can be merged before CCC. I can dismiss your review on this, but I'd prefer to get signoff on the Shell name.

Looks good to me.


libraw: 0.18.11 -> 0.18.12

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/libraw/versions.
These checks were done:

built on NixOS
/nix/store/a93d1dzw2ngcysj5sb3pj5ih6dcm8a1m-libraw-0.18.12/bin/raw-identify passed the binary check.
/nix/store/a93d1dzw2ngcysj5sb3pj5ih6dcm8a1m-libraw-0.18.12/bin/unprocessed_raw passed the binary check.
/nix/store/a93d1dzw2ngcysj5sb3pj5ih6dcm8a1m-libraw-0.18.12/bin/4channels passed the binary check.
/nix/store/a93d1dzw2ngcysj5sb3pj5ih6dcm8a1m-libraw-0.18.12/bin/simple_dcraw passed the binary check.
/nix/store/a93d1dzw2ngcysj5sb3pj5ih6dcm8a1m-libraw-0.18.12/bin/mem_image passed the binary check.
/nix/store/a93d1dzw2ngcysj5sb3pj5ih6dcm8a1m-libraw-0.18.12/bin/dcraw_half passed the binary check.
/nix/store/a93d1dzw2ngcysj5sb3pj5ih6dcm8a1m-libraw-0.18.12/bin/half_mt passed the binary check.
/nix/store/a93d1dzw2ngcysj5sb3pj5ih6dcm8a1m-libraw-0.18.12/bin/multirender_test passed the binary check.
/nix/store/a93d1dzw2ngcysj5sb3pj5ih6dcm8a1m-libraw-0.18.12/bin/postprocessing_benchmark passed the binary check.
/nix/store/a93d1dzw2ngcysj5sb3pj5ih6dcm8a1m-libraw-0.18.12/bin/dcraw_emu passed the binary check.
10 of 10 passed binary check by having a zero exit code.
0 of 10 passed binary check by having the new version present in output.
found 0.18.12 with grep in /nix/store/a93d1dzw2ngcysj5sb3pj5ih6dcm8a1m-libraw-0.18.12
directory tree listing: https://gist.github.com/20e1b0e52e1c94011da90df577fa84e2
du listing: https://gist.github.com/adb7f31a821d3bd4157e8003f58d47ce

meta.description for libraw is: '"Library for reading RAW files obtained from digital photo cameras (CRW/CR2, NEF, RAF, DNG, and others)"'.
Experimental: this build is cached with Cachix. To use the cache follow these experimental instructions:
One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout acfad81f9ead9199a710645727ab75835f6f45f5 && nix-shell --pure -I nixpkgs=. -p libraw

Success on x86_64-linux (full log)
Attempted: libraw
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/4qk8250h2h53lq4n7i4f88ag3041vwc6-libraw-0.18.12-dev
strip is /nix/store/21ymadblbmsbb2bk4q7gl4kjasp8zmgd-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/4qk8250h2h53lq4n7i4f88ag3041vwc6-libraw-0.18.12-dev/lib
patching script interpreter paths in /nix/store/4qk8250h2h53lq4n7i4f88ag3041vwc6-libraw-0.18.12-dev
checking for references to /build in /nix/store/4qk8250h2h53lq4n7i4f88ag3041vwc6-libraw-0.18.12-dev...
shrinking RPATHs of ELF executables and libraries in /nix/store/610nhp53srs65s231kgxgq9l0mxxa14x-libraw-0.18.12-doc
strip is /nix/store/21ymadblbmsbb2bk4q7gl4kjasp8zmgd-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/610nhp53srs65s231kgxgq9l0mxxa14x-libraw-0.18.12-doc
checking for references to /build in /nix/store/610nhp53srs65s231kgxgq9l0mxxa14x-libraw-0.18.12-doc...
/nix/store/a93d1dzw2ngcysj5sb3pj5ih6dcm8a1m-libraw-0.18.12

Success on x86_64-darwin (full log)
Attempted: libraw
Partial log (click to expand)
patching script interpreter paths in /nix/store/n9ffqr8cpnhvs45axkscd935jckmm2m2-libraw-0.18.12
strip is /nix/store/yyak5sjv68n9vdgnkd2cb5djk1l9sqj5-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/rfss63nfhw1akldxs5gm86153wnzn59i-libraw-0.18.12-lib/lib
patching script interpreter paths in /nix/store/rfss63nfhw1akldxs5gm86153wnzn59i-libraw-0.18.12-lib
strip is /nix/store/yyak5sjv68n9vdgnkd2cb5djk1l9sqj5-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/g8qviwp56z5gki0sqz31jq60517b4fxc-libraw-0.18.12-dev/lib
patching script interpreter paths in /nix/store/g8qviwp56z5gki0sqz31jq60517b4fxc-libraw-0.18.12-dev
strip is /nix/store/yyak5sjv68n9vdgnkd2cb5djk1l9sqj5-cctools-binutils-darwin/bin/strip
patching script interpreter paths in /nix/store/d50lri057il9h8b2vhbmdqy5nc432ckj-libraw-0.18.12-doc
/nix/store/n9ffqr8cpnhvs45axkscd935jckmm2m2-libraw-0.18.12

Success on aarch64-linux (full log)
Attempted: libraw
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/nkacak5slfwh2g58yli5jlkih547nw74-libraw-0.18.12-dev
strip is /nix/store/qg2agrqkf240s656d207zqhipl0bc2id-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/nkacak5slfwh2g58yli5jlkih547nw74-libraw-0.18.12-dev/lib
patching script interpreter paths in /nix/store/nkacak5slfwh2g58yli5jlkih547nw74-libraw-0.18.12-dev
checking for references to /build in /nix/store/nkacak5slfwh2g58yli5jlkih547nw74-libraw-0.18.12-dev...
shrinking RPATHs of ELF executables and libraries in /nix/store/j1jbpc95gz3wl1ibkagkxhbd47v0qi9h-libraw-0.18.12-doc
strip is /nix/store/qg2agrqkf240s656d207zqhipl0bc2id-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/j1jbpc95gz3wl1ibkagkxhbd47v0qi9h-libraw-0.18.12-doc
checking for references to /build in /nix/store/j1jbpc95gz3wl1ibkagkxhbd47v0qi9h-libraw-0.18.12-doc...
/nix/store/90ihzjfckfwlwzgcvjhvjy3q9jzjfh0c-libraw-0.18.12


Add link to prelude or other listing

Thanks, this is lots of fun - and instructive.
one of the issue's I've had, is not knowing sometimes which function is in the prelude or not,
maybe adding a link to hackage Prelude can help.
(or probably better, a page with the list of all the functions by name only.)
and maybe, after solving a link to the right function, like readFile :: FilePath -> IO String

Oh that's a great idea! I'll check how I can make it fit in the current design.

(adding a link to the Prelude module is the easy part. Adding links to each function is a bit harder, since the anchors have auto-generated ids)

This should be handled by https://github.com/divarvel/haskle/commit/e79e0434d78412c5ba492b348043945efc0f0e47


Add missing TrySetExceptions

closes #1155

Can we get a new release with this fix?

@rgroenewoudt v1.17.2 released!


Langauge settings does not work

The language settings does not change the language for a lot of the UI elements where they stay in Portuguese. I tried changing the language setting back and forth (English to Portuguese then back to English) and nothing changes, still mostly Portuguese with some English text. It seems to be the same when I change it to Korean too. The only difference on my side (since I don't see anyone else complaining) is that my Windows language is Chinese. Let me know if more information is needed. Thank you.

#47 helps a lot with this issue. If you know your way around git, github, and visual studio, you can check out that branch and compile with it to try it out.

#47 got merged in, just waiting for @sjohnson1021 @laetemn to push a new Release build

#47 got merged in, just waiting for @sjohnson1021 or @laetemn to push a new Release build

Build published with the new contributions. Thanks a ton

Is this issue considered closed since PR #47 was merged in?

I can confirm whether the new build fixes the problem I'm having tonight then close it.

Hello all, thanks very much for the new version, but it does not seem to work on my computer. I first tried installing the new version over the existing old installation but the language problem persists. Then I tried removing the install and then installing it, and it does not start at all. I tried uninstalling and installing multiple times with no success.

Hello all, thanks very much for the new version, but it does not seem to work on my computer. I first tried installing the new version over the existing old installation but the language problem persists. Then I tried removing the install and then installing it, and it does not start at all. I tried uninstalling and installing multiple times with no success.

For clarity's sake:
You

Uninstalled or deleted the program folder
Downloaded the latest setup/zip
Installed/Extracted

and the program simply refuses to launch?

Sorry for the late reply. I just tested it on my laptop running Windows 10 English. So the summary is that zip version works on both machines but installer version works on neither machines.
Regarding the language settings problem, it seems like the problem is still there. I basically opened the app, check the text, switch to English/Portuguese, restart the app. After doing this two times I can reproduce the mixed language issue on both machines. The only difference is that on my Chinese machine, choosing Portuguese is alright while choosing English will still have a lot of Portuguese text; on my English machine it's the other way around: English is alright but Portuguese has English text mixed in.

Sorry for the late reply. I just tested it on my laptop running Windows 10 English. So the summary is that zip version works on both machines but installer version works on neither machines.
Regarding the language settings problem, it seems like the problem is still there. I basically opened the app, check the text, switch to English/Portuguese, restart the app. After doing this two times I can reproduce the mixed language issue on both machines. The only difference is that on my Chinese machine, choosing Portuguese is alright while choosing English will still have a lot of Portuguese text; on my English machine it's the other way around: English is alright but Portuguese has English text mixed in.

Okay. That helps, the installer I believe includes my pre built config.ini file, which is why it won't launch . Someone else brought this up, and I fixed the zip file. Apparently I never got around to rebuilding the installer.
Thank you.
Regarding the mixed text.. I'm not seeing much if any mixed text, on English, but I'm aware the translation has been handled.. well, the best it could be at the time. But it needs attention for sure.

Sorry for the late reply. I just tested it on my laptop running Windows 10 English. So the summary is that zip version works on both machines but installer version works on neither machines.
Regarding the language settings problem, it seems like the problem is still there. I basically opened the app, check the text, switch to English/Portuguese, restart the app. After doing this two times I can reproduce the mixed language issue on both machines. The only difference is that on my Chinese machine, choosing Portuguese is alright while choosing English will still have a lot of Portuguese text; on my English machine it's the other way around: English is alright but Portuguese has English text mixed in.

When you get a chance, could you test the latest build? Ideally completely remove the previous installation just in case something is amiss with the config (Really no reason there should be). I will look into different methods of getting the language, but.. that really should only matter for the very first launch when it writes the config.ini. Once written, you can select which language you wish to use, and it should apply globally. All four languages currently work for me, on Windows 11 (English). I'm not sure if having the system language be in Chinese would matter or not, seeing as it's reading from a local resource file anyway, and the language is hard saved in the config on first launch, and when you update it from the options form.
Another possibility.. would be adding a Chinese translation, which.. I mean, yeah . eventually. I would only be able to estimate with Google translate, and sometimes.. that can be comical. other times. downright confusing, or potentially dangerous to files ('oops I didn't think that was going to delete that'., etc)
I could export the current resource file in an excel format, for translation, and add it if someone were willing to at least verify  that it makes sense and is coherent before publishing.

I tested both the installer and portable editions of the latest version on my Chinese machine, and everything works as expected: both editions start properly and I cannot reproduce the language switching issue anymore. Thank you!
The only two places where I still see Portuguese when English is selected is on the splash screen of first start (pretty minor I would say) and the Download GameTDB option (seems like a new feature so maybe it's just that the string is not there?)
Regarding Chinese translation, I am more than happy to do it! I can do translation for both Simplified and Traditional Chinese. I do know Git so I can most likely make a fork then a PR for the translation. You can open a new ticket and assign it to me if you want.

Are you comfortable using RESX manager? in VS? or should I export to an excel format?
The WiiTDB Download option... you are referring to the MessageBox that appears on launch if you haven't downloaded it and have an internet connection? .. Will look into adjusting exactly where the form is called to ensure the language is being detected and adjusted properly before rendering, but I believe It is working on my end.. I.. have made some further optimizations and trimmed excess with the branch I'm working on to improve transfers though.

I tested both the installer and portable editions of the latest version on my Chinese machine, and everything works as expected: both editions start properly and I cannot reproduce the language switching issue anymore. Thank you! The only two places where I still see Portuguese when English is selected is on the splash screen of first start (pretty minor I would say) and the Download GameTDB option (seems like a new feature so maybe it's just that the string is not there?)
Regarding Chinese translation, I am more than happy to do it! I can do translation for both Simplified and Traditional Chinese. I do know Git so I can most likely make a fork then a PR for the translation. You can open a new ticket and assign it to me if you want.

Are you comfortable using RESX manager? in VS? or should I export to an excel format?
The WiiTDB Download option... you are referring to the MessageBox that appears on launch if you haven't downloaded it and have an internet connection? .. Will look into adjusting exactly where the form is called to ensure the language is being detected and adjusted properly before rendering, but I believe It is working on my end.. I.. have made some further optimizations and trimmed excess with the branch I'm working on to improve transfers though.
Opened Ticket #89 @fei0316

Hi, I haven't used Resx manager but should be fun to play around with it! :)
For the download option I'm referring to this:

Thanks will fix that, I just fixed the splash screen. The program now detects and adjusts the language before even the splash screen shows. Which.. delays launch a tiny bit.. like 1 second? It bugs me but should be barely, if at all noticeable.

Hi, I haven't used Resx manager but should be fun to play around with it! :)
For the download option I'm referring to this:

I am unable to assign you to the ticket #89 until you either contribute to the project.. .... dumb, or comment there apparently.
This is ResX manager in VS2022:


Add layer for obstacles detected by vision/world model/...

Proposed changes

Add a nav2 cost map layer which adds incoming obstacles, balls, etc. to the cost map.

Related issues
#112

Closing this in favor of #176


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/cahek2202/ioBroker.yeelight

[ ] [E106] Title should not have word "adapter" in the name. It is clear, that this is adapter. Now: {"en":"Yeelight adapter","de":"Yeelight Adapter","ru":"Драйвер для Yeelight","pt":"Adaptador Yeelight","nl":"Yeelight-adapter","fr":"Yeelight adaptateur","it":"Adattatore Yeelight","es":"Adaptador Yeelight","pl":"Adapter światła"}
[ ] [E106] Title should not have word "adapter" in the name. It is clear, that this is adapter. Now: {"en":"Yeelight adapter","de":"Yeelight Adapter","ru":"Драйвер для Yeelight","pt":"Adaptador Yeelight","nl":"Yeelight-adapter","fr":"Yeelight adaptateur","it":"Adattatore Yeelight","es":"Adaptador Yeelight","pl":"Adapter światła"}
[ ] [E106] Title should not have word "adapter" in the name. It is clear, that this is adapter. Now: {"en":"Yeelight adapter","de":"Yeelight Adapter","ru":"Драйвер для Yeelight","pt":"Adaptador Yeelight","nl":"Yeelight-adapter","fr":"Yeelight adaptateur","it":"Adattatore Yeelight","es":"Adaptador Yeelight","pl":"Adapter światła"}
[ ] [E115] No license found in io-package.json
[ ] [E145] No news found for actual version 0.1.1
[ ] [E150] No common.connectionType found in io-package.json
[ ] [E152] No common.dataSource found in io-package.json
[ ] [E200] Not found on npm. Please publish
[ ] [E300] Not found on travis. Please setup travis
[ ] [E801] No repository about text found. Please go to "https://github.com/cahek2202/ioBroker.yeelight", press the settings button beside the about title and add the description.
[ ] [E802] No topics found in the repository. Please go to "https://github.com/cahek2202/ioBroker.yeelight", press the settings button beside the about title and add some topics.
[ ] [E604] No "## License" found in README.md

I have also found warnings that may be fixed if possible.

[ ] [W113] Adapter should support compact mode
[ ] [W400] Cannot find "yeelight" in latest repository

Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info

Thanks, that all bugs have been fixed.


MOVE FROM MAIN() TO WINMAIN()

Your entry point needs to be changes.
Also use #define _CRT_SECURE_NO_WARNINGS
Which Visual Studio version did you use for compiling?

Thank you.
i have never coded WINAPI C/C++ on Linux platform, though i do python and other programs via Linux.
Can you advice the best IDE on Linux to use with mingw installed? Or which IDE did you use on Linux?

Well it's not ideal to Code WinApi C/C++ on Linux. But whenever I do it's for testing only. I only use Visual Studio Code for all types of development. And if you want to have an IDE like feel and code WINAPI on Linux, Just install mingw-w64 and add include paths to VS Code. And of course you can Run the Executables using Wine.

Closing this due to no response from OP.


Swagger doc for properties that are dynamically generated and their types are also not the same.

I have a use case where the labels for a model's properties are dynamically generated from object to object and their types are also not the same. Any idea how to go about documenting this in Swagger2.0? Thanks!
e.g.: Object 1:
        form:
          properties:
            pl_ui:
              properties:
                environment:
                  properties:
                    BYPASS_SECURITY:
                      type: "string"
                    NODE_ENV:
                      type: "string"
                    ACCOUNT_MANAGEMENT_HOST:
                      type: "string"
                    ACCOUNT_MANAGEMENT_PORT:
                      type: "string"
                  type: "object"
              type: "object"
          type: "object"

Object 2:
        form:
          properties:
            es_search:
              properties:
                resources:
                  properties:
                    limits:
                      properties:
                        cpu:
                          type: "number"
                        memory:
                          type: "number"
                      type: "object"
                    requests:
                      properties:
                        cpu:
                          type: "number"
                        memory:
                          type: "number"
                      type: "object"
                  type: "object"
                config:
                  properties: {}
                  type: "object"
              type: "object"
          type: "object"

object 3:
        form:
          properties:
            work-ui:
              properties:
                resources:
                  properties:
                    limits:
                      properties:
                        memory:
                          type: "number"
                        cpu:
                          type: "number"
                      type: "object"
                    requests:
                      properties:
                        memory:
                          type: "number"
                        cpu:
                          type: "number"
                      type: "object"
                  type: "object"
                environment:
                  properties:
                    BYPASS_WSS_SEC:
                      type: "string"
                  type: "object"
              type: "object"

The only thing you can do is just define it as an object, with additionalProperties being 'open'. Something like:
type: object
additionalProperties: {}


'Drill to detail' and 'Driil by' - Error: {'per_page': ['Must be greater than or equal to 1 and less than or equal to 10.']}

When I try to use 'Drill to detail' or 'Drill by' features I receive an error message:
Error: {'per_page': ['Must be greater than or equal to 1 and less than or equal to 10.']}
I tried to use the feature in two deployments:

docker: docker-compose-non-dev, TAG=3.0.1
kebernestes: helm chart 0.10.14, TAG=3.0.1
and on different browsers (Firefox, Chrome, Edge).
The result on both deployments is the same:

superset_app          | <IP_ADDRESS> - - [08/Nov/2023:08:57:43 +0000] "POST /datasource/samples?force=false&datasource_type=table&datasource_id=18&per_page=50&page=1 HTTP/1.1" 400 95 "http://localhost:8088/superset/dashboard/10/?native_filters_key=rcqz_oA2Id3cQXghURt-88NtzMFENSNF136Fv8RGRkVUxoaroGF-eVdKWxOEgrqx" "Mozilla/5.0 (X11; Linux x86_64; rv:103.0) Gecko/20100101 Firefox/103.0"
superset_app          | <IP_ADDRESS> - - [08/Nov/2023:08:54:35 +0000] "POST /datasource/samples?force=false&datasource_type=table&datasource_id=18&per_page=50&page=1 HTTP/1.1" 400 95 "http://localhost:8088/superset/dashboard/10/?native_filters_key=tQ6U6Xb_PMEkzflgAQ5r8dlqiyTt-uw1da_9w6BimVVS6_7SjkIo7S12gwobW-gO" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/114.0.1823.58"
How to reproduce the bug

Go to any aggregated chart (can be one of the samples chart)
Click on 'Drill to detail' or 'Drill by'
See error: Error: {'per_page': ['Must be greater than or equal to 1 and less than or equal to 10.']}

Expected results
The window with details should occur.
Actual results
Error message occurs:
Error: {'per_page': ['Must be greater than or equal to 1 and less than or equal to 10.']}
Screenshots

Environment
(please complete the following information):

browser type and version: Mozilla/5.0 (but I also tried in Chrome and Edge)
superset version: 3.0.1 (helm chart  0.10.14, or docker-compose-non-dev, TAG=3.0.1)
python version:
node.js version:
any feature flags active:
"ALERT_REPORTS": True,
"EMBEDDED_SUPERSET": True,
"EMBEDDABLE_CHARTS": True,
"DRILL_TO_DETAIL": True,
"DRILL_BY": True,
"ENABLE_TEMPLATE_PROCESSING": True,
"DASHBOARD_CROSS_FILTERS": True,
"DYNAMIC_PLUGINS": True,
"VERSIONED_EXPORT": True

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[v] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[v] I have reproduced the issue with at least the latest released version of superset.
[v] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

@aszczepanska the results are limited by SAMPLES_ROW_LIMIT. You probably have set that to 10.


Sensors connected to ESP32

Hello,
I am using an ESP32 in the ArduiTouch Housing! It works perfectly! Thank you for this great piece of code.
Because my ArduiTouch is placed in the Bathroom to control the lights, I thought about measuring the temperature and humidity to know when I need to open the window!
Is there any opportunity to connect a sensor (e.g. DHT 22) directly to the ESP32 where Homepoint runs on. My idea would be to copy the needed library to the library folder, compile it by oneself, and then have the possibility to configure the library in the config.json file!
It would be great if something like this would be possible in the future!

Hey,
I think that's tricky, architecturally speaking.
Right now each sensor has to be a MQTT device, but I have vague plans for the future to have it ra bit more abstract so the things being shown on display could come from any source (MQTT, Hue, HomeKit or even custom sensor events).
Until then I'm afraid it's probably a bit tricky to add a custom one (I guess you could subclass one of the UIWidgets and hardcore-hack one in there).
But thanks for the suggestion! I'll keep this as a potential usecase in mind :)


🏗build (browser): rewrite build with a browser build

PR for #231.
Works in Browser!
ToDo:

[x] Rewrite build tools (use rollup and esbuild for js and tsc for d.ts files)
[x] Add Browser Build file
[x] Add Polyfills for Browser build
[x] Add Module file for Node (this works better if you use a bundler in your build chain)
[x] Add source map for better debugging (with just a tsc build this was not important but now with minified code this is a big benefit)
[x] change readme
[x] Remove Demo document and demo.html

Ok it is not working I have to "polyfill" the following internal node modules. Good for us browserify maintain some modules for this...
events -> events
stream -> stream-browserify
util -> util
vm -> vm-browserify
Buffer global -> buffer but has to get attacht to global
Build is now at 300K minified js code and 70K with brotli kompression and 85K gzip that is quite good.
Only thing missing is a readme update.
@jjhbw can you have a look? I added a demo.html and a demo docx file to test it out - should be removed before merge.

@jjhbw ready for review!

@jjhbw Done changed it back to tsc build. And I changed the rollup config.

Just as a information: I use this now in production and it is running good! It is not a big template but it uses nested if, for, etc. so if this runs all should be fine 👍

Looks great, thanks! I changed some minor things and merged it offline, see 3d6c1a8c5f33868c8f40e9996a2fa0db0889f86a for the end result.

LGTM 👍 Thanks!

@mathe42 small extra change:  527259eec95e84287ea04b8945b90c11f938fa52
LMK if I misunderstood.

Yes is correct missed that 👍


Rasterizer crash in Mac module_test_ios, testDualCold

The Mac module_test_ios, testDualCold crashes several times on rasterizer:
System Integrity Protection: enabled

Crashed Thread:        31  io.flutter.2.raster

Exception Type:        EXC_BAD_ACCESS (SIGSEGV)
Exception Codes:       KERN_INVALID_ADDRESS at 0x0000000000000000
Exception Note:        EXC_CORPSE_NOTIFY

Termination Signal:    Segmentation fault: 11
Termination Reason:    Namespace SIGNAL, Code 0xb
Terminating Process:   exc handler [31275]

It is unclear to me what caused the crash, by looking at the result file downloaded here, the crash seems happened pretty early during launching the flutter view.
The complete crash log log is here: https://github.com/flutter/flutter/issues/89175#issuecomment-1034268165
crash (1).txt

Labeling as P2 as this is causing https://github.com/flutter/flutter/issues/89175, which is a P1.

@cyanglaz Do you know if there's a reliable repro for this? Is this best way using the Mac module_test_ios test?
@bdero mind taking a look?

The segfault is happening during this non-threadsafe set insert.
The log appears to contain two raster threads, both of which are flushing an SkCanvas and are in the middle of allocating Metal buffers... Not sure what the story is with that yet.

@cyanglaz Do you know if there's a reliable repro for this? Is this best way using the Mac module_test_ios test?

Unfortunately, I don't have a reliable repro for this, I couldn't even repro Mac module_test_ios locally. All we have was the fails on CI and the logs :(

I think I know the bug. The DualFlutterViewController in the test creates two FlutterViewControllers, each of which are initializing their own engines with their own set of threads/task runners. However, the Skia event tracer is global state.
If we want to continue supporting this usage going forward, we need to make access to the state introduced in  https://github.com/flutter/engine/pull/25985 synchronized.

Nice detective work, @bdero. /cc @jason-simmons for advice.

I imagine that just syncing access and calling it a day would make the traces relatively unusable. A few ideas:

If we want to support running multiple engines with tracing enabled, we'd ideally write traces that can be differentiated based on the engine instance by mapping thread identities.
Refuse to launch a second engine when tracing is enabled with a clear error message.
Discard traces from every engine except one. This would also require mapping thread identities to engines, but is an easy stopgap on the way to solution 1.


NTR: rapid tRNA decay

Please provide as much information as you can:

Suggested term label:

rapid tRNA decay pathway
(and regulation term)

Definition (free text)

A cytoplasmic surveillance pathway that targets mature hypomodified tRNAs, catalyzed by 5’-3’ exonucleases Xrn1  (S. cerevisiae/human) and Rat1 (S. cerevisiae) XRN2 human, and regulated by adenosine 3’,5’ bisphosphate (pAp).

Reference, in format PMID:#######

PMID:35901126

Gene product name and ID to be annotated to this term

https://www.pombase.org/gene/SPAC26A3.12c

Parent term(s)

GO:0071026   cytoplasmic RNA surveillance
GO:0106354   tRNA surveillance

Any other information

PMID:35901126 also says there are
" two characterized decay pathways [13–15]. "
so this ticket and
https://github.com/geneontology/go-ontology/issues/27142
cover current knowledge.
interesting fact:
In addition, the nuclear surveillance pathway is also known to target about 50% of all tRNA transcripts, possi- bly due to stochastic errors during transcription, mis-folding, or natural competition between decay and processing [27].

+[Term]
+id: GO:0180037
+name: rapid tRNA decay
+namespace: biological_process
n) and Rat1 (S. cerevisiae) XRN2 human, and regulated by adenosine 3’,5’ bisphosphate (pAp)." [GOC:vw, PMID:35901126]
+is_a: GO:0106354 ! tRNA surveillance
+property_value: term_tracker_item "https://github.com/geneontology/go-ontology/issues/27141" xsd:string
+created_by: vw
+creation_date: 2024-02-26T13:45:16Z


Feature Request: Make Light Sensor Optional

Since it's not possible to move individual services into different rooms, the solar panel "light sensor" has to stay with the blind in the room - but as this light sensor isn't real it gives a false reading for actual light in the room that you cannot get rid of.
Making this optional via config would allow users to remove it.

Yeah, I'll probably add some config.json settings not to expose the light sensor and/or triggers.
I did consider exposing the Current Ambient Light Level and other light sensor characteristics on the Window Covering service, so HomeKit doesn't see it as a separate sensor.  Apple's Home app gets confused when both actuator and sensor services are exposed on the same accessory: it treats the services as grouped, and you cannot ungroup them.  You cannot control the device directly from the tile, as Home thinks it needs to popup the window to show the different services.   Of course, the sensor service doesn't show under the tile.
Alternatively, I could expose the light sensor as a separate accessory.  In that case, you can also assign it to a different room.  That actually makes sense, as the solar panel won't report the light level in the room, even when calibrated; it reports the outside light level.
The device actually reports the solar panel charging level, between 0 and 780 (max value I've seen so far).  The Smart Shades app exposes this both as charging level and light level.  I arbitrarily set Charging State to off when the value drops below 100.  Afaik there's no way to tell whether the solar panel is connected, but not receiving any light vs no solar panel connected.  Similarly between charger cable connected vs solar panel providing full charge level.
I understand from Wazombi Labs that there's no linear relationship between the reported value and actual light level.  I've placed one of my Hue motion sensors next to the solar panel, facing outwards, to compare the reported values.  Still need to write a script to log these for analysis.

The Smart Shades app exposes this both as charging level and light level. I arbitrarily set Charging State to off when the value drops below 100.

Where do you see light level? I only see charging level.

I understand from Wazombi Labs that there's no linear relationship between the reported value and actual light level. I've placed one of my Hue motion sensors next to the solar panel, facing outwards, to compare the reported values. Still need to write a script to log these for analysis.

That'd be useful - I'd be surprised if the panel light level is accurate enough to be useful, and if it isn't then you know it's probably not worth exposing as a separate accessory (just add it on the Window Covering service for reference).

Where do you see light level?

When you create or edit a trigger based on light level.

I'd be surprised if the panel light level is accurate enough to be useful

It should be useful enough for automations, based on thresholds for “dark” (which probably could also be achieved using the sunset and sunrise info), and “direct sunshine”.  I want to use the latter to tilt my venetian blinds slightly to keep the sunlight out of my room, so it won’t get too warm.  I have a bay window with one large window facing west, and two smaller windows facing south west and north west.  I’d like the blinds to adjust themselves based on the incoming sunlight per window.
Normally, I would use on-device triggers for that, but for more advanced use cases, I need the light level in HomeKit (or another home automation system).  E.g closing the blinds while watching TV, and opening them fully or slightly tilted after, depending on whether the sun shines onto the window.

Ah yes I see the light level reading in the app now, never noticed those triggers before.
Your use case makes sense, even if inaccurate, with a wide margin to determine "dark" or "direct sun" it can be useful to automate stuff based on that.

v0.0.14 no longer exposes a separate Light Sensor service.  The characteristics are exposed under the Windows Covering service.

v0.0.14 no longer exposes a separate Light Sensor service. The characteristics are exposed under the Windows Covering service.

Thank you :)


Invalid type coercion with == operator and strings

TypeScript Version<EMAIL_ADDRESS>2.9.2, playround

Search Terms:
== as typeguard
number typeguard
number type coercion
Code
function test(level: number | string):number {
    if (level == +level) {
        const q2 = level; //number
        return level;
    }
    return 0;
}
alert(test(5) + 1);
alert(test("5") + 1)

Expected behavior:
Compiler error
Actual behavior:
The code compiles.  As mentioned in the comment, typescript assumes that level inside the if block is a number; however in reality that condition passes also for number-like string ("5")
Playground Link:
http://www.typescriptlang.org/play/index.html#src=function test(level%3A number | string)%3A number {
    if (level %3D%3D %2Blevel) {
        const q2 %3D level%3B %2F%2Fnumber
        return level%3B
    }
    return 0%3B
}

alert(test(5) %2B 1)%3B%0D%0Aalert(test(%225%22)%20%2B%201)

seems narrow type in narrowTypeByEquality not handle the Relax Equal Operator
i would like to add a new type comparable  similar areTypesComparable,
that handle the Relax Equal Operator and narrow type
follow the rule: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Equality_comparisons_and_sameness