feature: show survey card on Home page

Description
[In draft, pending real copy for the card and an actual URL to link the survey to.]
This adds the logic to show a notification on the home screen prompting the user to take a survey. This also adds some new notification types and a fairly generic way to track a user's usage of various features inside the browser's own local storage, which will be useful for other possible business cases (like extending the conditions surveys pop up in, or cosmetic things like Achievements).
Task Item
refs #8029
Screenshots

Sorry for the late response to this, but can we use a banner instead of a notifications card? Notifications are more interruptive and seem better for time-based messages that directly help the user (like when a resource has been provisioned).
@garypretty @kaerm @joanjasak thoughts?

Sorry for the late response to this, but can we use a banner instead of a notifications card? Notifications are more interruptive and seem better for time-based messages that directly help the user (like when a resource has been provisioned).
@garypretty @kaerm @joanjasak thoughts?

If you can get me designs for exactly what that should look like and you're okay with it taking a little more time, I'm fine with it. I figured the notification I've been showing people and getting thumbs-ups was good enough, since it will only get shown usually once per session (right now I have it showing up when the home page loads).

I would recommend we stick with notifications - this is the approach we have previously discussed and that has been tried and tested by other teams (i.e. VS Code). It should not be too disruptive since it should not be shown often to the users.
@beyackle you mentioned it is currently implemented as once per session but it should really be once per user after x sessions. Did I understand what you mentioned correctly?

@beyackle you mentioned it is currently implemented as once per session but it should really be once per user after x sessions. Did I understand what you mentioned correctly?

I meant "at most once per session" - I've implemented the checks for at least 5 different calendar dates, 90 days since the last survey if any, and a 15% chance on top of that. Because of the 90 days thing, if a user takes the survey, it will set a flag so they won't see the notification again for 90 days.

Great, that sounds right

If the user selects the X to close the notification (but does not select opt out), will it reappear in the next session?

@mewa1024 it believe it will not appear in the next session - @kaerm can correct me if I am wrong.
How does this appear in ABS?
For R14, here is my take on priority.

@beyackle to continue with implementation with the current visual notification and prioritize getting the feature end to end with the correct trigger and URL, with all of the required params (meeting with Kate tomorrow to unblock).

If we can get the feature end to end and we have time, then I am happy for us to look to update the visual, if we feel like time is tight, we go with the notification for R14 and iterate in R15 (we know we have more work to do on this in future milestones, this is just iteration 1).

@mewa1024 maybe you can sync with @kaerm to understand how HaTS is appearing in other products (you may already have background here) and we go from there.  I think in the future this could be triggered during the use of the application based on other arbitrary triggers, so I think notifications is what I have seen used elsewhere?

Sorry, I missed @kaerm's comment above about sticking with notifications and looks like my recall was right about this being tried and tested elsewhere.  @beyackle  let's stick with the notification approach - thanks.

If the user selects the X to close the notification (but does not select opt out), will it reappear in the next session?

That's effectively the user saying "no, but I don't want to opt out". The next session will have the same 15% chance to show it as it did before.

@beyackle here is the body copy. Melanie reviewed and provided edits.
Copy for survey notification
H1: Let us know how we're doing.
Body: We read each and every comment and will use your feedback to improve.
L1: Yes, I'll take the survey!
L2: No, I'd like to opt out.
Copy for opt out confirmation
H1: Opt-out success!
Body: Thanks, you will no longer receive requests for this survey.
@mewa1024 @kaerm

Opt out success sounds a little odd to me - can we confirm the copy being used in ABS / other apps when the user opts out please?  If they are using similar wording, no problem, but I'd like us to review please. @kaerm do you have any data that can help here?

We do not have an equivalent anywhere that I know off. VSCode uses this verbiage for the initial buttons but once you click on one I believe it just closes the notification without a message.
('takeSurvey', "Take Survey"),
('remindLater', "Remind Me later"),
('neverAgain', "Don't Show Again"),

In Azure Portal there is no opt-out option.

We use "Don't show me again" for modals that ask users to confirm a destructive action. What about plain old "No, thanks" ?

@kaerm @beyackle and I synced yesterday on some of the technical pieces for this work item and discussed the copy at the same time. We agreed that the survey will not show on the home page, but rather it will appear the first time a customer lands on the designer page in that session. This is to try and avoid getting in the way of the customer opening their bot and we are hoping this will lead to a higher uptake. We are going to add telemetry to measure the level of uptake and interaction with this feature too.
Copy
Regarding copy I think we should take the copy from VS Code (given we don't have our own data here and they will have undoubtedly tested their copy), but amend the 'Don't show me again' to 'No thanks' as per @mewa1024's comment above. They also don't have a sub text, so I have taken the one suggested by @joanjasak and amended slightly to match the title. We will also show no follow up notifications, so if the user chooses to dismiss / reject, the card just disappears.
So copy would be.
H1: Do you mind taking a quick feedback survey?
Body: We read every response and will use your feedback to improve Composer.
L1: Take survey
L2: Remind me later
L3: No thanks
If anyone has any issue with the above then reach out to me and we can discuss. @beyackle please proceed with the above for now.
Survey trigger
Survey will now be triggered after 2 sessions with a 30% chance of it being seen - we have amended this based on our current telemetry data to hit a large enough segment of users.
Telemetry
@beyackle We should collect the following telemetry items. @kaerm please review and jump in if you think I missed something or we need to amend (we will automatically have the usual global properties included with the below. I considered adding a custom property for 'trigger', but I think we add this in the future as we add additional triggers).
Event Name: HATSSurveyOffered
Fired when the notification card is shown.
Event Name: HATSSurveyDismissed
Fired when the user clicks 'Remind me later' or closes the card using the 'x'.
Event Name: HATSSurveyRejected
Fired when the user clicks 'No thanks'.
Event Name: HATSSurveyAccepted
Fired when the user clicks 'Take survey'.

Thanks for the summary. @beyackle can you share screen shot when it is ready? We don't normally have 3 link buttons in notification, so I just want to check that everything fits okay.

@mewa1024

Minor change to layout (group primary and second actions together and aligning left, push third action to aligning right):

Hey there - I had not seen this work-item or design until today.
Please include @mewa1024  on all design requests. Melanie and I are doing planning and prioritization of design & design backlog items so we want to make sure we have an accurate sense of what's in the works & track status.
Please assign UI copy items to me - I am working with techwriters and Azure Doc writers to ensure all copy work is tracked and captured (in an xls while we're working on a more mature system.
Regarding the control: Following the FLUENT guidelines, we'll need to us a Message Bar here. The current notifications are custom designs that are in the process of being reconciled as part of the 'Book of Composer' work that Melanie is working on (page templates and style guides).
The bar can have up to 2 buttons, so please make the following updates to match Fluent gudelines:
H1: Do you mind taking a quick feedback survey?
REMOVE body copy
L1: Take survey
L2: Remind me later
The 'x' at the top is 'close' (= no thank you)


[Cleanup] Remove unused algorithm classes.

cc @XTrimmer

@Wraith2 this was legacy code, exists only in NetFx, copied over as-is, but it's unused code.


rush: look for rush.rc in /etc

Description of changes

Before this change, rush would look for its configuration file in the
Nix store. That made it difficult to declaratively configure rush.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/declarative-configuration-for-gnu-rush/21455/5

Is there anything blocking this PR, or did it just fly under the radar?
I'd like to be able to configure rush via /etc, so getting this fix into nixpkgs would be great.

Hi all, is there anyone who can help move this forward? GNU Rush doesn't seem to work on NixOS without these changes, and the package is currently not usable in its present state. It would be great to get this resolved so others can benefit from the improvements. Thanks!

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/2044

I just pushed a new version of this PR that fixes the merge conflicts.


fix [GitHubLerna] service test

Fixes #6106
Babel has stopped using Lerna, apparently. The test with the branch was still passing because Babel have switched from master to main already, and their older master branch still had the lerna.json file. Changing the target for both tests though to prevent future breakage, as there's no telling how long Babel may keep the old branch around

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @calebcartwright!

  Generated by :no_entry_sign: dangerJS against d9db458c6d67ed54c480cb03c4d319ce80556abc

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @calebcartwright!

  Generated by :no_entry_sign: dangerJS against d9db458c6d67ed54c480cb03c4d319ce80556abc


Typescript cannot find declaration file (.d.ts) when used via npm

Issue
Issue name
Typescript cannot find declaration file (.d.ts) when used via npm
Issue description
When one tries to import es6-tween in TypeScript, they get:
ERROR in [at-loader] ./lib/graphics.ts:7:24
    TS7016: Could not find a declaration file for module 'es6-tween'. '[...]/node_modules/es6-tween/full/Tween.js' implicitly has an 'any' type.
  Try `npm install @types/es6-tween` if it exists or add a new declaration (.d.ts) file containing `declare module 'es6-tween';`

Related
This issue is related to

[X] package.json

What you excepted?
To fix, you simply need to add a "types" field in package.json. See http://www.typescriptlang.org/docs/handbook/declaration-files/publishing.html.
I'll send a pull request.

I'm seeing this issue again. Looks like there used to be TS definitions here, but they no longer exist.

PR is welcome =)


Fixing top section of Readme

Fixing svg links in top section of Readme
Closing #138

@universalmind303 Can we please merge this minor PR? Thx


PID commentary

https://github.com/Ben-PH/SimpleFOC-rs/blob/47ea9ed03db2f3749ca7d6a038bf721271c66459/src/common/pid.rs#L110
Similar to the LPF, it'd be useful to support fixed timestep operation.
Additionally, some sensors can return velocity (derivative of position) directly, so if the derivative term is used it may occasionally be useful to pass the value for the derivative in, instead of calculating it here via finite difference. This should be considered relatively low priority though.

I like it.
One thought i have is around the idea of not limiting the order if derivatives. If someone wants to go down to the nth differential, that should only be a matter of setting a type parameter.
By the sfoc spirit, the Library, by default, would be the simple and uncomplicated version, easily digestible for those that are just stepping into this. When they see that deeper differentials are available, that should prompt them to investigate, make discoveries, and learn.
Or in other words: at the surface, this library is a toy to make motors go brrrr in satisfying ways. Just beneath, is a rabit hole of discovery, revealed when you go into the generated documentation


Why can't the parameters in the step names map to the parameter names in their implementation

I have a step that creates a specific data structure with multiple values. Most of these values are default though, and I only need to manipulate a specific value at a time.
e.g.

create structure with power "100" kW
create structure with voltage "100" volts

@step(["create structure with power  kW",
"create structure with voltage  volts"
"create structure with power  kW and voltage  volts"])
def create_struct(power=0,voltage=0):
struct.power = power
struct.voltage = voltage
The parameters instead depend on their location in the step when being passed.
Gauge version: 1.0.4
Commit Hash: 3a9a647
Plugins
html-report (4.0.7)
java (0.7.1)
json-report (0.3.0)
python (0.3.6.nightly-2019-03-07)
screenshot (0.0.1)
spectacle (0.1.3)
xml-report (0.2.1)

Gauge does not support named parameters, mostly because it is not a common feature across all languages that gauge works with. Languages that do not support named parameters require positional arguments to match the signature of the method implementation.
In your particular case, I suggest you try using concepts to abstract out parameters. ex:
Step:
@step("create structure with <power> kW and <voltage> volts")
def create_struct(power,voltage):
struct.power = power
struct.voltage = voltage

And have concepts like these:
# create structure with <power> kW
* create structure with <power> kW and "0" volts

# create structure with <voltage> volts
* create structure with "0" kW and <voltage> volts

Does this help?

I guess... I've been doing it by checking the step name for keywords which would make the specs less cluttered (but the step implementation more complex).
It's a trade-off. But it would be really cool if there was a way to handle optional parameters.
Thanks for your help.


Bugfix/issue 569

Fixes # 569
This PR is ready for review.
Risk
This PR makes no API changes.
Testing Plan
Test RouterServiceValidator.validate() using app context and running service router from different packages; also test using app context and running service router from the same package.
Summary
During app's router service validation, if the running router service is created by the app, validation is good by default. This will let the app bind to the service it created regardless of its Multiplex security setting.

Codecov Report

Merging #571 into develop will increase coverage by 0.07%.
The diff coverage is 100%.

@@              Coverage Diff              @@
##             develop     #571      +/-   ##
=============================================
+ Coverage      39.81%   39.88%   +0.07%
- Complexity      2253     2257       +4
=============================================
  Files            310      310
  Lines          13799    13801       +2
  Branches        1431     1432       +1
=============================================
+ Hits            5494     5505      +11
+ Misses          8106     8096      -10
- Partials         199      200       +1

Impacted Files
Coverage Δ
Complexity Δ

...rtdevicelink/transport/RouterServiceValidator.java
55.42% <100%> (+3.86%)
46 <0> (+4)
: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 9295bf5...060a3ac. Read the comment docs.


Read boot time on musl

Boot time lookup on musl does not work as musl does not support "wtmp" which is needed by last. As a consequence, svlog with --boot or --boot-offset will throw an error.

Partially fixed in 0.5.3 by reomving the dependency on wtmp for --boot and --boot-offset 0. wtmp is still needed for --boot-offset <N> for N > 0.

--boot-offset removed for musl in v0.6.0.


minor changes to publishing

@olafurpg We need this internally at Twitter, so please take a quick look on Monday. Otherwise, I'll just merge, and we'll figure out the details later.


[26703] - Add Optional Attempts for CopyItemMatching.

Ticket

[26703]

Github Issue

ISSUE-

Description

Add Optional attempt for get queries on SecureStorage & AuthenticatedSecureStorage.

Commit message

Add Optional Attempts for CopyItemMatching.

Screenshot

Testing

[ ] Added unit tests
[x] Ran unit tests successfully
[ ] Added documentation for public APIs and/or Wiki

          4 Warnings

:warning:
TwilioVerifySDK/TwilioSecurity/Sources/Storage/AuthenticatedSecureStorage.swift#L23 - Prefer at least one space after slashes for comments.
comment_spacing AuthenticatedSecureStorage.swift:23

:warning:
TwilioVerifySDK/TwilioSecurity/Sources/Storage/AuthenticatedSecureStorage.swift#L25 - Prefer at least one space after slashes for comments.
comment_spacing AuthenticatedSecureStorage.swift:25

:warning:
TwilioVerifySDK/TwilioSecurity/Sources/Storage/AuthenticatedSecureStorage.swift#L36 - Prefer at least one space after slashes for comments.
comment_spacing AuthenticatedSecureStorage.swift:36

:warning:
TwilioVerifySDK/TwilioSecurity/Sources/Storage/SecureStorage.swift#L31 - Prefer at least one space after slashes for comments.
comment_spacing SecureStorage.swift:31

Code coverage
Total coverage: 93.08%

File
Coverage

TwilioVerifySDK/TwilioSecurity/Sources/Keychain/Keychain.swift
87.57%

TwilioVerifySDK/TwilioSecurity/Sources/Storage/AuthenticatedSecureStorage.swift
92.64%

TwilioVerifySDK/TwilioSecurity/Sources/Storage/SecureStorage.swift
85.87%

Powered by Slather

  Generated by :no_entry_sign: Danger

Code coverage
Total coverage: 93.07%

File
Coverage

TwilioVerifySDK/TwilioSecurity/Sources/Keychain/Keychain.swift
87.57%

TwilioVerifySDK/TwilioSecurity/Sources/Storage/AuthenticatedSecureStorage.swift
92.76%

Powered by Slather

  Generated by :no_entry_sign: Danger

Code coverage
Total coverage: 93.14%

File
Coverage

TwilioVerifySDK/TwilioSecurity/Sources/Keychain/Keychain.swift
87.63%

TwilioVerifySDK/TwilioSecurity/Sources/Storage/AuthenticatedSecureStorage.swift
92.76%

Powered by Slather

  Generated by :no_entry_sign: Danger


scratch-blocks, scratch-render build

Resolves
What Github issue does this resolve (please include link)?

Resolves #

Proposed Changes
Describe what this Pull Request does
Reason for Changes
Explain why these changes should be made
Test Coverage
Please show how you have added tests to cover your changes
Browser Coverage
Check the OS/browser combinations tested (At least 2)
Mac

[ ] Chrome
[ ] Firefox
[ ] Safari

Windows

[ ] Chrome
[ ] Firefox
[ ] Edge

Chromebook

[ ] Chrome

iPad

[ ] Safari

Android Tablet

[ ] Chrome

1


VSIX dependency on NuGet.BuildTools

Fixes #3852 by ensuring that NuGet's tasks and targets are present
wherever MSBuild itself is, since common.targets now depends on NuGet.
Closes NuGet/Home#7390.

Duplicate of #4146, but targeting the right release branch.


Dataset is Incomplete

Downloaded tiles in folder new_ncpcc is imcomplete

So sorry for the late response to this long standing issue, didn't get email notifications, added by the latest commit.


Refactor file handing for self-update.

What does this PR change? What problem does it solve?
This PR refactors file handling in self-update to better support existing failure modes.

New file payloads are written to a temp file before touching the original binary. Minimizes the possibility of failing mid-write and corrupting the binary.
The original binary is moved out to a temp directory rather than removing it, to avoid the binary being locked on some operating systems.

Was the change previously discussed in an issue or on the forum?
Closes #2248
Checklist

[X ] I have read the contribution guidelines.
[X ] I have enabled maintainer edits.
[ ] I have added tests for all code changes.
[X ] I have added documentation for relevant changes (in the manual).
[X ] There's a new file in changelog/unreleased/ that describes the changes for our users (see template).
[X ] I have run gofmt on the code in all commits.
[X ] All commit messages are formatted in the same style as the other commits in the repo.
[X ] I'm done! This pull request is ready for review.

FYI, if you rebase on the master branch now, you don't have to be compatible to Go 1.14 any more (so you can use the new tempfile method). I've merged #3680 yesterday which raises the minimum version of Go to 1.15. :)

Behavior is now split by Windows and non-Windows as requested. The functions are still probably the right ones despite the removal of 1.14, as it looks like CreateTemp didn't come around until 1.16.

I've fixed the build failure and slightly edited to commit message to no longer mention a temp directory but rather a temp file.


Add support for a new line representing a full week (top or bottom)

I tried, it's not that convincing visually and not sure if it's useful (for now at least)


Request: proxy support

https://create-react-app.dev/docs/proxying-api-requests-in-development/

@North101 may I ask what exact use-case you have for proxying?

@North101 may I ask what exact use-case you have for proxying?

Closing because of inactivity.


Added Messaging App Website using basic PHP

Hello Sir..I have made my comment/message website(App) using basics of php and html,in which I stored data in comments text file which will store old comments as well as append new comments...
My interface looks like this..

Please check and confirm it under hacktoberfest 2022

Hello !!!please tell me why my repo was excluded in hactoberfest


[Master] release/v1.0.1 QA Sign-off

This PR is automatically created by CI to release a new official version of caver-java-ext-kas.
When this PR is approved by QA team, a new version will be released.

Package
Tests
Failures
Ignored
Duration
Success rate

xyz.groundx.caver_ext_kas.kas.anchor
25
0
0
0.811s
100%

xyz.groundx.caver_ext_kas.kas.tokenhistory
57
0
0
1.707s
100%

xyz.groundx.caver_ext_kas.kas.utils
2
0
0
0.020s
100%

xyz.groundx.caver_ext_kas.kas.wallet
106
0
0
36.438s
100%

xyz.groundx.caver_ext_kas.node
46
0
0
1.152s
100%

@sirano11 Can you write down newly added TC count from next time?


Components

In this PR I have made the following changes:

Create a design for the calculator
Add basic components to display the calculator
Use props to pass data from child to parent

**STATUS: APPROVED  🟢 **

Hi @Lusindiso ,
Great work on making the changes requested by a previous reviewer 👏🏻
Your project is complete! . There is nothing else to say other than... it's time to merge it 🍾🚢 :shipit: . Congratulations!  🎉💯🌟

Cheers and Happy coding! :clap: :clap: :clap:
Feel free to leave any questions or comments in the PR thread by mentioning my Github username if something is not 100% clear.

As described in the Code reviews limits policy you have  limited number of  reviews per this project. If you think that the code review was not fair, you can request a second opinion using this form.


useOrientation does not work for iPads

Description of the bug
The useScreenOrientation hook does not work on iPads.
To Reproduce
Just copy the example code on an iPad (real device or simulator) and render the results.
Expected behavior
It does correctly return PORTRAIT or LANDSCAPE.
Actual behavior
It always outputs "UNKNOWN" for orientation.
Additional context
Tested with Expo SDK 36

@StefanSchwartze same here

I think it's related to this issue, https://github.com/expo/expo/issues/5693. There is nothing much we can do here because we are dependent on Expo for this. Let's keep this issue open until it is resolved. Thanks for reporting btw! ❤

This should now be fixed with the new v2 release! 🎊  See this comment for more info.


Unskip ndimage tests.

Unskip ndimage tests and skip tests those fail on rocm 5.4.

cupy_full_tests_04_18.log
Getting errors in NCCL tests on my machine. Not sure if this is OMPI version issue
cc: @pnunna93 @AdrianAbeyta


For Edge/Chrome, use the Network tab to create a HAR file, not the Console tab

Screenshot in the documentation shows console tab with red outline, indicating that's the tab to create a HAR file from. Screenshot should be updated to reflect correct place, along with correct corresponding text.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 13f9ad7f-cf04-3030-9f71-511e5d603713
Version Independent ID: e43fa6b7-536d-1bca-b2ad-4223e2a18101
Content: Capture a browser trace for troubleshooting - Azure portal
Content Source: articles/azure-portal/capture-browser-trace.md
Service: azure-portal
GitHub Login: @JnHs
Microsoft Alias: jenhayes

@efav
Thanks for your feedback! We will investigate and update as appropriate.

Thanks @efav for mentioning this. We are in the process of reviewing this topic and making sure it has the current info, I'll close this item once that work has been completed and this is addressed.

Hi @efav, thank you for your patience. After reviewing the content, I see that the Edge/Chrome steps already do state/show the Export HAR function is on the Network tab.
I'm going to close this issue now as it appears to be represented correctly, but please let me know if I've missed something and you're still seeing wrong information. If so can you please tell us specifically where this appears. Thank you for taking the time to share your feedback and help us improve the docs.
#please-close

#please-close


Add M3 processors

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

Sources:

https://github.com/OpenMathLib/OpenBLAS/pull/4315/files
https://github.com/Tencent/ncnn/blob/master/src/cpu.cpp

This results in a dunno value when running on M3. I've opened a fix here: https://github.com/Homebrew/brew/pull/16393


Don't require to implement optional abstract properties

Search Terms
abstract optional override
Suggestion
If in abstract class abstract property is marked as optional, allow child classes not to implement it.
So I suggest to remove an error for property x of class D in the following code:
https://www.typescriptlang.org/play?ts=4.0.2#code/IYIwzgLgTsDGEAJYBthjAgggg3gKAUIQAcoB7CAU3koBMFRIZ4EpLhayA7ZATwQAeAfgBcCLgFcAtiEpQCRUhWpV6jaHERsO3Pgl5jJMuQgA+CCV1qUAZgEsudPAsLEJIZHdgJOAZSkQABYAFACUuC5EhLDcYGTIlAB0yGQA5sFBdmCJAgA0CJnZvKGRAL545XgoaBgAIgiUAlRWGNj4UUpUNPTanDz8-AC8CACMAAwVzo4A7gi1YYl+ASGhQA
abstract class A {
    protected abstract readonly x?: number
    protected abstract readonly y: number | undefined

    public doSmth() {
        console.log(this.x, this.y)
    }
}

class D extends A {
    protected readonly y = 10
}

new D().doSmth()

But property y still must be implemented.
Note that we already can do some sort of it
https://www.typescriptlang.org/play?ts=4.0.2#code/JYOwLgpgTgZghgYwgAgJLIN4Chm+QBygHt9owBPAYThAGUBrYfAfgC5kQBXAWwCNoceQiTLkAspwDOYVN3wAbCNwjh2XPtGQAfZJxAATCDFAR9g3OeTKwACyL7qdRiwAUASnYA3IsDN4rELb2EtKyCkoqYO5ePmYAvlhYcLzSUIhgyAjycJKSyACCmJbCpFAUjgxMbBw8-FCWyanpBMSlFCEycorKqjUaUNq6BkYmfniNYGkIGSWiHWHdkQAqRETV6nWJ-tZ2DjSVrh7I3r4NKZPNO8FSneE9UUcnY7gTUxlX+vNdEeAra9HHWJYBJYLI5PKUQoQAAekAMeUK2AA9AAqSwAOSIIAAtK9mmDcsgAOSQonIfRECB5EBEDLAb73ZCgGzQYCQfTIPHTAL9YkfL53SJkmDEbiZbKEon5IkAOhcACYAKwARkVbgxWNx5ze4vBxNJ5Mp1NpTIZkSZIBZUDZpk52suSjqfMCuwFi1+q2Fot1kulcqVqvV-kxOK5GQJeRJ0sNVI4JvpgvAFqtNo5YZ5TqJszK4huCx+YC9RDFEeJfoVKrVGtD9u5pajZIpsZpdLNSeZrPZdqa3OUvKzrTmebbYD+RZLEsj5YDVZRSOBiVLlHQCfdYDy6GRaP8lEn+tQZNACCIUCgEGm8nIpsT64tkFgiBQRIPMssS3IpH3h7y3GAuVAADmyC2CgMBEPI8hEAA7oBLQiGUwCxiKxbAR+T4Huw2btMON4ADQBEEnw4WuCoACzygADOqc7AkAA
interface I {
    propertyCanSkip?: number
    propertyMustImplement: number | undefined

    methodCanSkip?(): void
    methodMustImplement(): void
}

abstract class A {
    propertyCanSkip?: number
    abstract propertyMustImplement: number | undefined
    abstract propertyMustImplementToo?: number

    methodCanSkip?(): void
    abstract methodMustImplement(): void
    abstract methodMustImplementToo?(): void
}

class CA extends A {
/*
    Non-abstract class 'CA' does not implement inherited abstract member 'methodMustImplement' from class 'A'.(2515)
    Non-abstract class 'CA' does not implement inherited abstract member 'methodMustImplementToo' from class 'A'.(2515)
    Non-abstract class 'CA' does not implement inherited abstract member 'propertyMustImplement' from class 'A'.(2515)
    Non-abstract class 'CA' does not implement inherited abstract member 'propertyMustImplementToo' from class 'A'.(2515)
*/
}

class CI implements I {
/*
    Class 'CI' incorrectly implements interface 'I'.
    Type 'CI' is missing the following properties from type 'I': propertyMustImplement, methodMustImplement(2420)
*/
}

But there is a set of problems for properties
We have two ways to override property (via property declaration or via getter and setter). And now (in TS4) the limitation have changed. For nonabstract property base class always defines how it should be implemented in children.
Let's look what implementations are possible (don't forget about useDefineForClassFields compiler flag that makes it more important):

Code
Property can be omitted
Child can implement as property
Child can implement as get/set

propertyCanSkip?: number
Yes
Yes
No

abstract propertyMustImplement: number | undefined
No
Yes
Yes (except https://github.com/microsoft/TypeScript/issues/40632)

abstract propertyMustImplementToo?: number
No
Yes
Yes (except https://github.com/microsoft/TypeScript/issues/40632)

get getter?(): number
N/A
N/A
N/A

abstract get getterMustImplement(): number | undefined
No
No
Yes

abstract getterToo?(): number
N/A
N/A
N/A

It's easy to see, that if the property should really be optional, there is only one way to make it such which will not allow to implement it as getter and setter. But we have 2 absolutely identical lines with optional and nonoptional abstract property. I see no sense for them to be synonyms as ? in the 3rd line definitely says that the property is optional, but doesn't give me ability to make so in further code.
So I propose to change this table in following way:

Code
Property can be omitted
Child can implement as property
Child can implement as get/set

propertyCanSkip?: number
Yes
Yes
No

abstract propertyMustImplement: number | undefined
No
Yes
Yes

abstract propertyCanSkipToo?: number
Yes
Yes
Yes

get getter?(): number
N/A
N/A
N/A

abstract get getterMustImplement(): number | undefined
No
No
Yes

abstract get getterCanSkipToo?(): number
Yes
No
Yes

Abstract getter is NOT a part of this feature request, just shown for consistency.
Use Cases
Provide ability to list and use for reading an optional property in abstract class without limiting a way of its implementation in child classes. Such problem occured in a real project because of migration from TS3 to TS4. Before that it was possible, but because of breaking changes of TS4 it's not anymore.
https://www.typescriptlang.org/play?ts=4.0.2#code/IYIwzgLgTsDGEAJYBthjAgggg3gKAUIQAcoB7CAU3koBMEpLhayA7ZATwTAFsIALAPwAuBKwCuPEJSh4CRYuJDIAlrAQAzABQBKXPKJFYbMGWSUAdMjIBzLQJVgLvAToMIAvni94UaDABCCJQAHlSstBjY+IYu-AgAvAgAjAAM3nJ+6AgAwsFhlBFR+oY2lIhxurgM5eJQrAgAssACFjARZDxVXj5ZGAAi+eGRWPo+rJQA7ggBuhbabhPTOXMLeEsI-au6QA
abstract class A {
    protected readonly smth?: number

    public f() {
        console.log(this.smth)
    }
}

class B extends A {
    smth = 10
}

class C extends A {
    get smth() { return Math.random() }
}

class D extends A {
}

new B().f()
new C().f()
new D().f()

Examples
See above.
Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

Why it's not a breaking change?
It changes behavior of existing construction, but it's not a breaking change in terms of code.
If you had your code working and you have
abstract propertyMustImplementToo?: number

in it, that means that you implemented this property in all child classes. So after its meaning changes all you code keeps being valid and compiles into absolutely the same javascript code as before. Nothing changed.
At the same time, for further development you have to decide whether you want to allow child classes to skip the property or not. If yes, or you don't care - keep it with ? as it is. If no then update it to
abstract propertyMustImplementToo: number | undefined

without any other changes needed.
Related Issues:
https://github.com/microsoft/TypeScript/issues/6413
https://github.com/microsoft/TypeScript/issues/22939

Discussed this with @sandersn a bit and we think this is reasonable, but also that the current behavior is also very defensible. In general we're stare decisis for longstanding behavior that hasn't received any other feedback, so would prefer to leave this alone unless there's strong evidence that the majority of the people using this particular set of modifiers feels the same way.

Another point from our discussion; if #40632 is fixed, these two programs emit different code with useDefineForClassFields: true:
abstract class C { p?: number }

emits a defineProperty.
abstract class C { abstract p?: number }

would not emit a defineProperty.
Right now these two programs both emit the [[Define]], and they both check the same way, so you can get the desired effect today by leaving off abstract.

Typescript doesn't distinguish between p?: number and p: number | undefined in properties. The first is a shortcut for the second.

@sandersn
Does that mean that with a type like:
interface Foo {
	 bar: string;
	 baz: string | undefined;
}

It’s valid to do:
const foo: Foo = { bar: "bar" };

By omitting baz even with strictNullChecks?

No, I am wrong. p?: number is equivalent to p?: number | undefined BUT is different from  p: number | undefined.

I said
abstract class C { p?: number }
abstract class K { abstract p?: number }

are checked the same today (and emitted, but that's bug #40699)
class D extends C { } // doesn't need to be abstract, p is optional
class B extends K { } // currently: B must be abstract

This issue requests that B be valid code, and not required to be abstract. But that's the same as D.

@sandersn I don't understand you still. Here is code for TS3 - how do you propose to update class A without changing B, C and D so that the code will be valid for TS4? Full example, please.

B vs C is a separate issue. For B and D, you don't need to update A if you want smth to be optional. If you want smth to be required, make it abstract.

@sandersn I want optional field and all 3 classes compatible with A. For me it seems impossible now, isn't it?

@sandersn It doesn't seem that you addressed the problem @Qwertiy described in his above comment.
The problem is, no matter which way is chosen (you're describing the two ways we can choose), there is a type error regardless.
How do you make that example work with and without abstract? A solution is needed.

I think this is the only way to implement conditionally optional implementation requirement.
playground
abstract class A<T extends string = ""> {
  abstract f: "" extends T ? (T | undefined) : T;
}

// ERROR: Non-abstract class 'B' does not implement inherited abstract member 'f' from class 'A<"">'.(2515)
class B extends A {
  // now, it is needed to implement f even if it can be undefined
  // f = undefined;
}

class C extends A<"a" | "b" | "c"> {
  // implementation is needed
  f = "a" as const;
}

I have no idea about necessity.

I’m having something like the code below (simplified) in our project, which we’d like to run with  :
abstract class BaseClass {
    abstract readonly requiredProperty: string;
    abstract readonly optionalProperty?: string;
}

The idea is that sub-classes must specify how the properties are defined (including if they are based on getters or values), and once they do that they can’t be changed, but the optionalProperty either is a string or it’s not present at all. Note that exactOptionalPropertyTypes makes the distinction between “a property is not present” and “a property has the undefined value” explicit.
In apparent contradiction with some of the comments above, it seems impossible to actually define a subclass of BaseClass whose members don’t have the optionalProperty at all.
(Well, I can define it by adding a constructor that explicitly deletes the property, but that’s not what I mean.)

FWIW this is also functionality we need. Same case.

Same here, could be great to have this (similar to virtual in .net)

I hit a requirement that's impossible with current typescript:
A class with a method signature that subclasses may or may not implement.

If the method is abstract, then the subclass can't skip its implementation.
If the method is not abstract, then the subclass is required to add override under noImplicitOverride, which is misleading because it's not actually overriding anything.

These kinds of problems are encountered only by api designers, so the "majority of people" will never be affected by this. If that's what's expected, then there's no hope for changes like this to happen.


pull request for e2e test at 01/27/2017 02:15:12

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add timestamp to nodes in verbose checks 🙏

Currently verbose reports only show the start time and end time, the nodes themselves don't have a timestamp. I think it would be useful to show timestamps on nodes in the web viewer.

Don't rely on it being present though ^^ :wink:


Remove LRN reuse section

@drnaylor-oc - I am content to put this PR on hold until nearer to 11 October. However, a question for you: do we want to keep this subsection of the service guide after 11 October? https://developer.service.hmrc.gov.uk/guides/ctc-traders-phase5-service-guide/documentation/developer-setup.html#duplicate-lrn-error-notification

No, let's remove it because it shouldn't happen in this way any more (it'll be an IE056 with the details in, just like P4)

OK. Understood. Will remove section.


Change Tenant<>VMT relationship from 1:n to m:n

It's a beautiful disaster.
Views will need to be tested manually. Note that VM instantiation logic will remain deficient until such time as it becomes possible to identify the current User/Tenant context (i.e. we need something like @current_user.get_current_tenant, or perhaps simply @current_tenant, or maybe @appliance.appliance_set.user.current_tenant in VM-related services). Please make sure to review the appropriate TODOs that have been left in the code.

I hate that dog. :anger:

I was trying to start it, unfortunately following exception is thrown in the worker:
08:47:01 worker.1 | 2015-07-06T06:47:01.437Z 11527 TID-orovldatg WARN: PG::UndefinedColumn: ERROR:  column atmosphere_virtual_machine_templates.tenant does not exist
08:47:01 worker.1 | LINE 1: ..._virtual_machine_templates"."id_at_site" = $1 AND "atmospher...
08:47:01 worker.1 | : SELECT  "atmosphere_virtual_machine_templates".* FROM "atmosphere_virtual_machine_templates" WHERE "atmosphere_virtual_machine_templates"."id_at_site" = $1 AND "atmosphere_virtual_machine_templates"."tenant" = 1 LIMIT 1
...
08:47:01 worker.1 | /home/marek/git/atmosphere/app/services/atmosphere/vm_updater.rb:120:in `source_template'

Merge on hold pending resolution of VMTMonitoringWorker issues.

OK, I think I may have caught the problem - VMTUpdater was erroneously duplicating VMT records. Fixed and added spec to prevent this from happening in the future. @mkasztelnik - before you run any more tests you'll need to wipe your local DB (it will have been left in an inconsistent state by the faulty updater).
Btw., caught you pushing a failing build! ;)

Please let me know when it will be ready for retests

@mkasztelnik You think I can merge it now or are there outstanding issues?


update for numba and pandas

I think there are other places pointing to old version:

reuirements.txt contains numba=0.51.2
README contains old versions of numba and pandas.

@pcheremu, @xaleryb This seems to be duplicate of PRs: #939 and #959. Also I would prefer to merge to master first, stabilize development builds there (some are still red, specifically Windows + python 3.7, see here and here) and only after that merge from master to gold/2021 (as per agreed branching strategy).

Closed as duplicate.


Revert "Change the interface of PriamScheduler (#1049)"

This reverts commit b0b5a432ac44b66ba9eece1d776ae16b0b075330.

Revert due to accidentally merged the PR before resolving the comments.


Change ReactDom.render to createRoot (React 18 render)

Fixes React 18 console warning
Warning: ReactDOM.render is no longer supported in React 18. Use createRoot instead.
Until you switch to the new API, your app will behave as if it's running React 17.
Learn more: https://reactjs.org/link/switch-to-createroot

Let's wait until Slinky supports this.


Initial tutorial for envoy golang extensions

Not yet actually compiled in terms of restructure text
Need to decide where to link this into the broader docs

Summary of changes:
Fixes: #issue-number
<!-- Enter the release note text here if needed -->

How to test (optional):
e.g. Run make tests, make runtime-tests and/or some more specific way to
exercise the new behavior(s).
Note: The runtime testsuite will modify the host it runs on.

This change is

Great comments @jrajahalme , thanks!  I think I have addressed all comments.
I'm still hoping to add two diagrams to the doc, but I'm thinking that what we have here is better than no docs already, so we can consider merging if we don't see any other issues.  Thanks!


added slim/psr7 support

Slim 4 now no longer has it's PSR-7 implementation bundled.
It's now at https://github.com/slimphp/Slim-Psr7.
But slim/psr7 0.6 requires php ^7.1 though, which causes PHP 7.0 tests failed.

Thanks, but there's already a pull request to release a new major version (v3.0) with support for PSR-17 (factories) that includes Slim-Psr7: https://github.com/middlewares/utils/pull/11

Ah, cant believe I wasn't aware of that! :+1:


Account Filter Recognition Failure

I am attempting to nuke every resource I can in every region except one IAM user. I have followed the account filter doucmentation as best as I could.
regions:
  - all

accounts:
  ############:
    filters:
      IAMUser:
        - "******************"

Note: The account ID and IAM user is filled in correctly when ran. I have triple checked.
Here is the result I get every time:
Configuration Details

Account ID:       ############
Resource Types:   505 (total)
      Included:   505
      Excluded:   0
Filter Presets:   0
Resource Filters: 0

Note: use --with-filtered to see resources with filters defined
Note: use --with-included to see included resource types that will be nuked
Note: use --with-excluded to see excluded resource types

Are there any fixes or work arounds for this? It seems to recognize the file and the regions just not the account filter I have in place.

@JL7105 the explain-config is about the resource types and not the resources. You will need to run the full nuke which will due a dry run and show you what is filtered vs what will be removed. Once you've reviewed the output, then you can run with --no-dry-run which will prompt you a second time and actually do the removal of resources.
I'll work on the verbiage in the explain-config as I can see how it's a bit ambiguous.


no "SMS" or iMessage can be received or sent

Device
iPhone X
Version
iOS 16.7.8
Passcode?

[ ] Passcode is enabled.

Start going into detail..
i just updated my iPhone X to iOS 16.7.8, and jailbreak it using "v2.0.0 beta 9 Update." Ever since, i can't receive SMS messages, nor able to send any. For example, i don't get the verification code (for any 2-step verification website/app). What have changed? or what should i do? because i can't log-in to important apps/websites. Please be informed that i use the "rootfull" version. However, as soon i have switched to "rootless" the problem is gone!
Familiarity with palera1n

[X] I am familiar with palera1n and how to use it!

Familiarity with CLI

[X] I am familiar with the command-line!

hm so my rootless suggestion worked, but I'll take a wild guess it's because of the iOS update that broke your rootful partition.

try using derootifier to install those rootful tweaks on rootless

may not always work on every tweak but worth a shot

respectfully, my main reason of creating this post, is if you ever going to issue a new version/update for "palera1n - rootfull," please consider a fix for this problem. thank you in advance.

I can't imagine it's recommended to not only jailbreak your primary phone with rootless, but then to also upgrade it.  Apple patches the bugs that allow jailbreaks.  I would highly recommend not updating your phone if you manage to get everything working how you want it.


ci: Add PHP 8.2 support to build matrix

Coverage: 99.778%. Remained the same when pulling 4386fea0e2989dc552a369af9c2f6981fe002137 on php-82-to-build-matrix into 7596bea180d8b1e5fbd9874b80d224f36b6d7296 on develop.


[event-hubs] Updated @azure/eslint-plugin-azure-sdk

Fixes #8347
ESLint keeps causing errors, since it's currently not required due to the || exit 0 at the end of the lint command.
The current list of errors and warnings can be seen here: https://gist.github.com/sadasant/d1c4818d8fd947723f70b7771bf2825c
So far, I have only fixed what could be automatically fixed with lint:fix. Going through all of the errors will take considerable more time. Depending on how much time I have left this week, I could add some other fixes in this PR.

/azp run js - eventhub - ci

/check-enforcer reset

/check-enforcer override


[Bug] [Knowledge] can not delete knowledge space

Search before asking

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

Operating system information
Linux
Python version information
3.10
DB-GPT version
latest release
Related scenes

[ ] Chat Data
[ ] Chat Excel
[ ] Chat DB
[X] Chat Knowledge
[ ] Model Management
[ ] Dashboard
[ ] Plugins

Installation Information

[X] Installation From Source

[ ] Docker Installation

[ ] Docker Compose Installation

[ ] Cluster Installation

[ ] AutoDL Image

[ ] Other

Device information
Device : GPU
GPU Count : 3
GPU Memory : 24G per GPU
Models information
LLM : zhipu_proxyllm
Embedding Model : text2vec-large-chinese
What happened
无法正常删除知识库。删除的时候显示文件繁忙，通过终端查看对应的文件正在被DB-GPT服务占用。

What you expected to happen
对应的文件通过终端查看显示正在被 python dbgpt/app/dbgpt_server.py 占用。
How to reproduce

创建知识库
添加知识源
删除刚刚创建的知识库

Additional context
No response
Are you willing to submit PR?

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

check is there are another dbgpt process?
ps -ef | grep dbgpt

检查是否有其他 DBGPT 进程？ ps -ef | grep dbgpt

我也是这个问题，目前进程只有当前运行这个。我从pilot/data中删掉xxx.vectordb知识库和文件后。页面点击删除按钮，无法删除。并且在pilot/data中删掉的xxx.vectordb知识库又复现了。

restart your server? and create another space and check if there still have that question.

重新启动服务器？并创建另一个空间并检查是否还有该问题。
我重新拉了一下代码 并且创建的都是引文的知识库空间 删除后还是有这个问题，


fix broken link

Fixes #5430

Realized that the same broken link was also used in two other topics.


Minimum required numpy version (1.16.6) has security vulnerability

Describe the bug, including details regarding any error messages, version, and platform.
We currently use version 11.0.0 of pyarrow. We run Snyk checks to find vulnerabilities in our third party dependencies
In the version we use, but also on main, the setup.py file states numpy >= 1.16.6 is required (https://github.com/apache/arrow/blob/main/python/setup.py#L451)
This Python version is affected by the following vulnerability: https://security.snyk.io/vuln/SNYK-PYTHON-NUMPY-2321964
To solve the issue, version 1.22.2 or higher should be required
Component(s)
Python

Hey @diegohavenstein, thanks for bringing this up. While I would definitely encourage users to install a numpy version w/o known vulnerabilities, I don't think we want to enforce it in this case given that numpy 1.22.2 was released just a little over a year ago (Feb 3, 2022)[1]. Snyk also reports the known vulnerabilities as low severity, so it's probably best to allow users additional time to upgrade their numpy version.
[1]https://pypi.org/project/numpy/1.22.2/

Indeed, it is up to applications and end users to ensure they use a newer numpy version in case those security reports are relevant for them (for many users scripting locally, it is not relevant at all), and not for libraries starting to limit allowed versions.
https://github.com/numpy/numpy/issues/19038 is also an interesting read, and essentially disputes the vulnerability, quoting:

Not a meaningful vulnerability because triggering the issue seems only plausible if the malicious party already has the privilege to run NumPy commands. Thus, while a bug, it does not present an escalation of privilege.


Fail in last PR regarding the use of json module

There is a typo in my last PR regarding the use of json module
I used json.loads with a s instead of json.load that support de serializing python object and not string
Every use of workspace_service failed in the plugin because of that

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.317%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

mergin/client.py
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
3275

Relevant Lines:
4129

💛  - Coveralls


merge from arlac77/npm-package-template

package.json

chore(scripts): cover@#overwrite c8 -x 'tests/**/*.mjs' --temp-directory build/tmp ava && c8 report -r lcov -o build/coverage --temp-directory build/tmp
chore(engines<EMAIL_ADDRESS>docs(package): remove keyword null
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/*-test.mjs' as in template
chore(package): set $.ava.extensions='mjs' as in template

.travis.yml

chore(travis): remove node versions 10.16.2,12.8.0
chore(travis): add node versions -10,-11,-12,10.16.3,12.8.1

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Errors in the Dec 2020 edition

p.7: C++ stack diagram should be Car1, Car2
p. 33: copy function copies 64 Point structs, should copy 64 bytes

p,169 find_slow() is faster than find_fast(). It looks like the times are swapped.
p.339 cout is NOT entirely thread safe (for instance when padding is used)
p.468 the close paren after Fpar on the bottom right is subscripted, it should be at the level of the whole 1 - Fpar expression


January Meetup

[x] Speaker: @bmeck
[ ] Sponsor:
[ ] Confirm venue (PoC is @bartronx7)
[ ] Meetup post
[ ] Blog post
[ ] Tweet
[ ] Order pizza
[ ] Record stats
[ ] Invoice sponsor

From Language Design to TC39  (The JS language standards group)

This is a reflection on a lot of discussions that come up when talking about language design and what it means to design for a future that is always bigger than the past. Topics from the past present and future such as ASI, ESM, decorators, and private fields are things that are important to get right if we are stuck with them forever. We can use discussion examples of each of these topics to get a better understanding of why it is so difficult to make things "just work". This will discuss ongoing themes of discussion within TC39 and efforts that are changing how TC39 is viewed.

pizza
eastside pies<PHONE_NUMBER>
2 pepperoni
1 margherita
1 bell pepper, mushroom, black olive, and red onion
double cut; ask for plates and napkins; deliver at 7:15
spredfast @ 200 w cesar chavez #300


"ITEM isn't a valid reinforcement block" disables reinforcing to another group

Today I passed my house to another player. While I was changing the control group to the player's own group I kept hitting some blocks I haven't reinforced before. The "Air isn't a valid reinforcement block" message would appear as normal, but I wasn't able to change groups until I /ctr again, but was able to reinforce non-reinforced blocks with stone.
It appears the error message disables passing a reinforcement to another group.

Do we actually support group chagning like this? I thought you had to transfer whole groups at a time.

It might not be well documented, but any error while in CTR mode exits CTR mode. This is expected behavior. You had nothing selected in your hotbar, hence the air message, and the removal from CTR mode. When transferring in the future, just keep a stack of stone selected and you won't have this issue.

Before, you could change both blocks of a double chest to another group, with /ctr and empty hand. Can you still do this if you must have the same reinf material in hand to change it?

You can do it, if and if the chests was previously reinforced.
Here's the success conditions to change a group using /ctr:

The target block must be reinforced
You must be MOD on both the original block's group and the new group

If (1) is violated, and you have an empty slot selected in your hotbar, you get an "Air isn't a valid reinforcement block" message and /ctr mode ends (without any additional message).
If (2) is violated, you get a Don't have permission message. I believe /ctr mode ends as well.
If (1) is violated but you have a valid reinforcement material selected in your hotbar, the block is reinforced to the new group using one of the valid reinforcement materials.
Hope that clears things up!

so probably a simple fix when we take player out of ctr mode maybe we should notify them? Sounds pretty convoluted for all the ways you can be removed from ctr

Yeah probably we could just modify whatever handler returns the "invalid" message and append a "REINFORCEMENT mode ended." message to that same player message.
Easy fix, anyone up for it?

This issue should have been in citadel pretty sure the case causing problems is here
https://github.com/Civcraft/Citadel/blob/master/src/vg/civcraft/mc/citadel/listener/BlockListener.java#L522

Yeah, definitely Citadel, good catch. That whole piece needs some cleanup, I see a lot of dangling error handling and edge cases where there is no user feedback, simply quiet failure. That's not good. I don't know what you're working on, but mind taking a closer look at that?

I can take a look at this now, I have a little spare time right now

Thanks muchly. Mind if I close this, and you can CopyPasta OP's report into a Citadel issue?
Thanks again @mclemente for raising this, hopefully we can get some clearer messaging.


Branch docs user guide

Codecov Report

Merging #210 (5f09857) into master (f913a96) will increase coverage by 0.12%.
The diff coverage is 82.35%.

@@             Coverage Diff              @@
##             master     #210      +/-   ##
============================================
+ Coverage     63.85%   63.98%   +0.12%
- Complexity      855      856       +1
============================================
  Files           126      126
  Lines          3027     3043      +16
  Branches        394      394
============================================
+ Hits           1933     1947      +14
- Misses          960      962       +2
  Partials        134      134

Impacted Files
Coverage Δ
Complexity Δ

...va/seedu/tr4cker/logic/commands/CommandResult.java
78.43% <81.25%> (+1.68%)
19.00 <1.00> (+1.00)

...java/seedu/tr4cker/logic/commands/HelpCommand.java
100.00% <100.00%> (ø)
2.00 <1.00> (ø)

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 f913a96...5f09857. Read the comment docs.


fix(TimeSeriesCard): don't filter null values

Closes #1528
Summary

When null values are passed to TimeSeriesCard, we should display a gap in the line charts where the data is missing to give the user an indication of the null values

Related Carbon Charts PR: https://github.com/carbon-design-system/carbon-charts/pull/464/files
Change List (commits, features, bugs, etc)

Remove null filtering of data values
Add additional conditional to timestamp to make sure the undefined === undefined race condition does not occur

Acceptance Test (how to verify the PR)

Go to any TimeSeriesCard story and pass it a mix of null and valid values
You should see a gap in the line where the null values were passed

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Infinite loop, direction change when you reach the end.

The new "infinite" prop for  introduced in version 1.23.0 is a very nice feature, but I was wondering if it would be possible to change the direction when the user has reached the end of the slides. So instead of going left all the way to the beginning, you would go to the right and end up at the beginning.
Like in the examples shown here: "https://kenwheeler.github.io/slick/"

Hi @danielbotn we’re gonna go ahead and close this issue and ask that you re-open with one of the provided issue templates. That helps us manage and track our issues internally.
To save you some time when you do re-open, since this is a feature request, we’ll likely ask that you implement the suggested feature by making a PR.
Thanks!


open in place possible?

I'm used to using vim netrw, and would like to browse for file, and open it in place, closing the file browser. I couldn't find in the doc a shortcut to do this. Is it possible? Thanks.

https://github.com/aziz/SublimeFileBrowser#close-filebrowser-when-files-have-been-opened


Is it possible to refresh or re initialize

i got error when calling again the function
Error: SkedType plugin has been initialized yet

I have the same issue. Did you find a workaround?

@sienedo I have the same issue. Did you find a workaround?

@emas00 for now i only re initialize by calling the script with ajax.

Fixed in v1.4.6.
Also you may destroy an instance with $sked.skedTape('destroy').

well if i destroy it i might initialize it again without ajax right?

Yes, that's correct.

This error still occurs in 2.4.6 - Can you provide a working example? I have tried many different combinations and result in the same


checkpoint varopt, no union code yet

Coverage increased (+0.04%) to 98.707% when pulling fadfc70961bc4a716b6741d27ad21cf6ff5718ab on varopt_nounion into eb2d7945d1331941e2e1e9e58adcb6bfa4f52c41 on master.


Avoid "form movements" in the login screen

I have Avoided "dynamism/movements" of username and password fields in the login screen by stopping the soft keyboard from pushing my view up.
Fixes #1107

Codecov Report

Merging #2618 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #2618      +/-   ##
=========================================
- Coverage    2.69%   2.69%   -0.01%
=========================================
  Files         258     258
  Lines       12260   12261       +1
  Branches     1107    1107
=========================================
  Hits          331     331
- Misses      11903   11904       +1
  Partials       26      26

Impacted Files
Coverage Δ

...n/java/fr/free/nrw/commons/auth/LoginActivity.java
0% <0%> (ø)
: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 7cb87f3...e35f13b. Read the comment docs.

I can click the password field after entering my username.

Thanks dear but with the new implementation you can still  click the password field after entering my username. The keyboard can still support you.

Dismissing my previous review on grounds of lacking appropriate issue, as I've now tagged #1107.

I can't click the password field after entering my username.

Thanks dear but with the new implementation you can still click the password field after entering my username. The keyboard can still support you.

I don't seem to be able to with my emulator config:
Name: Galaxy_Nexus_API_28
CPU/ABI: Google APIs Intel Atom (x86)
Path: /home/adam/.android/avd/Galaxy_Nexus_API_28.avd
Target: google_apis [Google APIs] (API level 28)
Skin: 720x1280
SD Card: 512 MB
fastboot.chosenSnapshotFile:
runtime.network.speed: full
hw.accelerometer: yes
hw.device.name: Galaxy Nexus
hw.lcd.width: 720
hw.initialOrientation: Portrait
image.androidVersion.api: 28
tag.id: google_apis
hw.mainKeys: no
hw.camera.front: emulated
avd.ini.displayname: Galaxy Nexus API 28
hw.gpu.mode: auto
hw.ramSize: 1024
PlayStore.enabled: false
fastboot.forceColdBoot: no
hw.cpu.ncore: 4
hw.keyboard: yes
hw.sensors.proximity: yes
hw.dPad: no
hw.lcd.height: 1280
vm.heapSize: 80
skin.dynamic: yes
hw.device.manufacturer: Google
hw.gps: yes
skin.path.backup: _no_skin
hw.audioInput: yes
image.sysdir.1: system-images/android-28/google_apis/x86/
showDeviceFrame: no
hw.camera.back: virtualscene
AvdId: Galaxy_Nexus_API_28
hw.lcd.density: 320
hw.arc: false
hw.device.hash2: MD5:fcb1c44f60d9172bb27b44fdd821eed1
fastboot.forceChosenSnapshotBoot: no
fastboot.forceFastBoot: yes
hw.trackBall: no
hw.battery: yes
hw.sdCard: yes
tag.display: Google APIs
runtime.network.latency: none
disk.dataPartition.size: 2G
hw.sensors.orientation: yes
avd.ini.encoding: UTF-8
hw.gpu.enabled: yes

Screenshots

Before
With changes

I can't click the password field after entering my username.

Thanks dear but with the new implementation you can still click the password field after entering my username. The keyboard can still support you.

I don't seem to be able to with my emulator config:
Name: Galaxy_Nexus_API_28
CPU/ABI: Google APIs Intel Atom (x86)
Path: /home/adam/.android/avd/Galaxy_Nexus_API_28.avd
Target: google_apis [Google APIs] (API level 28)
Skin: 720x1280
SD Card: 512 MB
fastboot.chosenSnapshotFile:
runtime.network.speed: full
hw.accelerometer: yes
hw.device.name: Galaxy Nexus
hw.lcd.width: 720
hw.initialOrientation: Portrait
image.androidVersion.api: 28
tag.id: google_apis
hw.mainKeys: no
hw.camera.front: emulated
avd.ini.displayname: Galaxy Nexus API 28
hw.gpu.mode: auto
hw.ramSize: 1024
PlayStore.enabled: false
fastboot.forceColdBoot: no
hw.cpu.ncore: 4
hw.keyboard: yes
hw.sensors.proximity: yes
hw.dPad: no
hw.lcd.height: 1280
vm.heapSize: 80
skin.dynamic: yes
hw.device.manufacturer: Google
hw.gps: yes
skin.path.backup: _no_skin
hw.audioInput: yes
image.sysdir.1: system-images/android-28/google_apis/x86/
showDeviceFrame: no
hw.camera.back: virtualscene
AvdId: Galaxy_Nexus_API_28
hw.lcd.density: 320
hw.arc: false
hw.device.hash2: MD5:fcb1c44f60d9172bb27b44fdd821eed1
fastboot.forceChosenSnapshotBoot: no
fastboot.forceFastBoot: yes
hw.trackBall: no
hw.battery: yes
hw.sdCard: yes
tag.display: Google APIs
runtime.network.latency: none
disk.dataPartition.size: 2G
hw.sensors.orientation: yes
avd.ini.encoding: UTF-8
hw.gpu.enabled: yes

Screenshots
Before	With changes

Can you try it out on a real device . Am using a real device to test

Real device screenshot:

It is better, however I'm concerned it won't work on devices with lower resolutions / different aspect ratios. It's also still annoying in the emulator as it makes it harder to copy and paste passwords which I do a lot when logging in etc. to test PRs.

Can i create a new issue for adding{ detect  paste event in password and username  editext  box}

I can't click the password field after entering my username.

Thanks dear but with the new implementation you can still click the password field after entering my username. The keyboard can still support you.

I don't seem to be able to with my emulator config:
Name: Galaxy_Nexus_API_28
CPU/ABI: Google APIs Intel Atom (x86)
Path: /home/adam/.android/avd/Galaxy_Nexus_API_28.avd
Target: google_apis [Google APIs] (API level 28)
Skin: 720x1280
SD Card: 512 MB
fastboot.chosenSnapshotFile:
runtime.network.speed: full
hw.accelerometer: yes
hw.device.name: Galaxy Nexus
hw.lcd.width: 720
hw.initialOrientation: Portrait
image.androidVersion.api: 28
tag.id: google_apis
hw.mainKeys: no
hw.camera.front: emulated
avd.ini.displayname: Galaxy Nexus API 28
hw.gpu.mode: auto
hw.ramSize: 1024
PlayStore.enabled: false
fastboot.forceColdBoot: no
hw.cpu.ncore: 4
hw.keyboard: yes
hw.sensors.proximity: yes
hw.dPad: no
hw.lcd.height: 1280
vm.heapSize: 80
skin.dynamic: yes
hw.device.manufacturer: Google
hw.gps: yes
skin.path.backup: _no_skin
hw.audioInput: yes
image.sysdir.1: system-images/android-28/google_apis/x86/
showDeviceFrame: no
hw.camera.back: virtualscene
AvdId: Galaxy_Nexus_API_28
hw.lcd.density: 320
hw.arc: false
hw.device.hash2: MD5:fcb1c44f60d9172bb27b44fdd821eed1
fastboot.forceChosenSnapshotBoot: no
fastboot.forceFastBoot: yes
hw.trackBall: no
hw.battery: yes
hw.sdCard: yes
tag.display: Google APIs
runtime.network.latency: none
disk.dataPartition.size: 2G
hw.sensors.orientation: yes
avd.ini.encoding: UTF-8
hw.gpu.enabled: yes

Screenshots
Before	With changes

Can you try it out on a real device . Am using a real device to test

Can i create a new issue for adding{ detect paste event in password and username editext box}

however I'm concerned it won't work on devices with lower resolutions / different aspect ratios.

I think you're correct. The screenshot of the "With changes" really doesn't seem to work well low resolution devices (FWIW, I own one).
That said, I'm pretty much not sure what kind of solution I had in mind when I opened #1107. Maybe one that

however I'm concerned it won't work on devices with lower resolutions / different aspect ratios.

I think you're correct. The screenshot of the "With changes" really doesn't seem to work well low resolution devices (FWIW, I own one).
That said, I'm pretty much not sure what kind of solution I had in mind when I opened #1107. It's been a long time since I opened it. Maybe I was thinking of a solution that would show you both the username and password fields (even in small resolution devices, to an extent of course ;-) but wouldn't adjust the position of the fields when the user switches between fields. I couldn't come up with a real life example that exactly matches what I'm trying to say but the "Wikipedia" android app's login screen could be a good example.

Can I slightly shift the form on top so as to avoid overlaping on vary small devices

however I'm concerned it won't work on devices with lower resolutions / different aspect ratios.

I think you're correct. The screenshot of the "With changes" really doesn't seem to work well low resolution devices (FWIW, I own one).
That said, I'm pretty much not sure what kind of solution I had in mind when I opened #1107. It's been a long time since I opened it. Maybe I was thinking of a solution that would show you both the username and password fields (even in small resolution devices, to an extent of course ;-) but wouldn't adjust the position of the fields when the user switches between fields. I couldn't come up with a real life example that exactly matches what I'm trying to say but the "Wikipedia" android app's login screen could be a good example.

Can I slightly shift the form on top so as to avoid overlapping on vary small devices

Can I slightly shift the form on top so as to avoid overlapping on vary small devices

Before that, I have a doubt. When I tried to reproduce the issue in #1107 using my device I couldn't. I tried using two keyboards and couldn't reproduce using either of them. I doubt if #1107 is an issue at all.
Can anyone else reproduce the issue stated in #1107?

Can I slightly shift the form on top so as to avoid overlapping on vary small devices

Before that, I have a doubt. When I tried to reproduce the issue in #1107 using my device I couldn't. I tried using two keyboards and couldn't reproduce using either of them. I doubt if #1107 is an issue at all.
Can anyone else reproduce the issue stated in #1107?

Yes i can reproduce the issue on my device. when i remove the new code the soft keyboard from starts pushing the form view up and down hence introducing  "dynamism/movements" of username and password form.  .

@noahwalugembe What keybooard do you use? Also, can you please post screenshots demonstrating the issue (maybe a video)?
I just wanted to know whether we're on the right page about our understanding of #1107.

@noahwalugembe What keybooard do you use? Also, can you please post screenshots demonstrating the issue (maybe a video)?
I just wanted to know whether we're on the right page about our understanding of #1107.

Thanks dear i have shared the gif's blow.  They are big and i could not embed them in here
Before:
Soft keyboard  push form view up on login screen
https://drive.google.com/open?id=1xeEMbRd3qHuYTsiK41rLPsK1amMgUqsw
After :
Stoped oft keyboard  from push form view up on login screen
https://drive.google.com/open?id=1aaB3ujvKKQ3El9jkRfc3eKQs3o801qsZ
Soft keyboard from pushing my view up.

Only judging from the GIFs in the comment above, the biggest "form movement" that I can see in the "Before" GIF seems to not be present in the "After" video :-)

Only judging from the GIFs in the comment above, the biggest "form movement" that I can see in the "Before" GIF seems to not be present in the "After" video :-)

Is this some thing we can regard as a user experience improvement?

I am not familiar with the topic but personally I would say so.
Does it work well in landscape mode, on small and big screens?

I am not familiar with the topic but personally I would say so.
Does it work well in landscape mode, on small and big screens?

Yes it can now work well on both small and big screens

Would you mind posting a screenshot showing how the balance looks on big screens? Thanks! :-)

Would you mind posting a screenshot showing how the balance looks on big screens? Thanks! :-)

Currently am having issues with running an emulator on my PC i have only tried it on two small phones.
Does your pc support emulators

Would you mind posting a screenshot showing how the balance looks on big screens? Thanks! :-)
Thanks @nicolas-raoul for your response. I have tried to tried to get a big screen and blow are the images

Does it work well in landscape mode, on small and big screens?

I suspect this would not work well on small screen devices. Particularly in landscape mode. I haven't tested it but I do have a small screen device. I'll let you know when I manage to build and test this one. If someone gives me an APK I it would be easier for me to test :-)

I have tried to tried to get a big screen and blow are the images

I have a doubt with the images you posted. I suspect your changes aren't kicking in for the landscape mode. If they do, then the form shouldn't have moved but it seems to be moving when the keyboard opens. May be you have to make changes to the corresponding landscape mode layout file? Or is it intentionally left untouched?

it intentionally left untouched?
Yes  it intentionally left untouched on medium and small devices landscape mode layout  so as to avoid scrolling after fixing the view in-static position.

@noahwalugembe I think you should be creating a separate PR for #2660.

Deletionnomination #2682

Hi @noahwalugembe , this pull request needs some more work for fixing conflicts, removing unneeded changes and deciding to the best strategy for the UI change we want. Sending a fresh new PR would only require the last step, so will be easier. That's why I am closing this one, please feel free to reopen it if needed.


Investigate removal of curl and directly running the scripts from the checkout repo

Made changes to the script to run the script where we have the latest version of the script i.e. unstable. We are fetching the script from unstable when we are on master.


[Device Support Request] Moes ZHT-3000-GB-SS-MS thermostat floor temperature sensor

Problem description
I have connected this thermostat to Home Assistant using Zigbee Home Automation and the SkyConnect dongle. The basic features work: the current temperature, set temperature, on/off, and it appears as a functional climate device in HA. The thermostat is for electric underfloor heating and so has a room temperature sensor (which I can read) and a floor temperature sensor which I cannot read.
I previously opened issue #2479 which was closed by a bot as I didn't respond quickly enough. The info below is up to date.
Solution description
I would like to be able to read the floor temperature over Zigbee.
Screenshots/Video
Screenshots/Video
[Paste/upload your media here]

Device signature
Device signature
{
  "node_descriptor": "NodeDescriptor(logical_type=<LogicalType.Router: 1>, complex_descriptor_available=0, user_descriptor_available=0, reserved=0, aps_flags=0, frequency_band=<FrequencyBand.Freq2400MHz: 8>, mac_capability_flags=<MACCapabilityFlags.FullFunctionDevice|MainsPowered|RxOnWhenIdle|AllocateAddress: 142>, manufacturer_code=4098, maximum_buffer_size=82, maximum_incoming_transfer_size=82, server_mask=11264, maximum_outgoing_transfer_size=82, descriptor_capability_field=<DescriptorCapability.NONE: 0>, *allocate_address=True, *is_alternate_pan_coordinator=False, *is_coordinator=False, *is_end_device=False, *is_full_function_device=True, *is_mains_powered=True, *is_receiver_on_when_idle=True, *is_router=True, *is_security_capable=False)",
  "endpoints": {
    "1": {
      "profile_id": "0x0104",
      "device_type": "0x0301",
      "input_clusters": [
        "0x0000",
        "0x0004",
        "0x0005",
        "0x0201",
        "0x0204",
        "0xef00"
      ],
      "output_clusters": [
        "0x000a",
        "0x0019"
      ]
    }
  },
  "manufacturer": "_TZE200_aoclfnxz",
  "model": "TS0601",
  "class": "zhaquirks.tuya.ts0601_electric_heating.MoesBHT"
}

Diagnostic information
Diagnostic information
{
  "home_assistant": {
    "installation_type": "Home Assistant OS",
    "version": "2024.3.0",
    "dev": false,
    "hassio": true,
    "virtualenv": false,
    "python_version": "3.12.2",
    "docker": true,
    "arch": "x86_64",
    "timezone": "Europe/London",
    "os_name": "Linux",
    "os_version": "6.6.16-haos",
    "supervisor": "2024.03.0",
    "host_os": "Home Assistant OS 12.0",
    "docker_version": "24.0.7",
    "chassis": "vm",
    "run_as_root": true
  },
  "custom_components": {
    "hacs": {
      "version": "1.34.0",
      "requirements": [
        "aiogithubapi>=22.10.1"
      ]
    },
    "octopus_energy": {
      "version": "10.1.4",
      "requirements": []
    },
    "ytube_music_player": {
      "version": "20240131.01",
      "requirements": [
        "ytmusicapi==1.4.2",
        "pytube==15.0.0",
        "integrationhelper==0.2.2"
      ]
    },
    "tapo": {
      "version": "3.0.0",
      "requirements": [
        "plugp100==4.0.3"
      ]
    },
    "better_thermostat": {
      "version": "1.4.0",
      "requirements": []
    },
    "scheduler": {
      "version": "v0.0.0",
      "requirements": []
    }
  },
  "integration_manifest": {
    "domain": "zha",
    "name": "Zigbee Home Automation",
    "after_dependencies": [
      "onboarding",
      "usb"
    ],
    "codeowners": [
      "@dmulcahey",
      "@adminiuga",
      "@puddly",
      "@TheJulianJES"
    ],
    "config_flow": true,
    "dependencies": [
      "file_upload"
    ],
    "documentation": "https://www.home-assistant.io/integrations/zha",
    "import_executor": true,
    "iot_class": "local_polling",
    "loggers": [
      "aiosqlite",
      "bellows",
      "crccheck",
      "pure_pcapy3",
      "zhaquirks",
      "zigpy",
      "zigpy_deconz",
      "zigpy_xbee",
      "zigpy_zigate",
      "zigpy_znp",
      "universal_silabs_flasher"
    ],
    "requirements": [
      "bellows==0.38.1",
      "pyserial==3.5",
      "pyserial-asyncio==0.6",
      "zha-quirks==0.0.112",
      "zigpy-deconz==0.23.1",
      "zigpy==0.63.4",
      "zigpy-xbee==0.20.1",
      "zigpy-zigate==0.12.0",
      "zigpy-znp==0.12.1",
      "universal-silabs-flasher==0.0.18",
      "pyserial-asyncio-fast==0.11"
    ],
    "usb": [
      {
        "vid": "10C4",
        "pid": "EA60",
        "description": "*2652*",
        "known_devices": [
          "slae.sh cc2652rb stick"
        ]
      },
      {
        "vid": "10C4",
        "pid": "EA60",
        "description": "*slzb-07*",
        "known_devices": [
          "smlight slzb-07"
        ]
      },
      {
        "vid": "1A86",
        "pid": "55D4",
        "description": "*sonoff*plus*",
        "known_devices": [
          "sonoff zigbee dongle plus v2"
        ]
      },
      {
        "vid": "10C4",
        "pid": "EA60",
        "description": "*sonoff*plus*",
        "known_devices": [
          "sonoff zigbee dongle plus"
        ]
      },
      {
        "vid": "10C4",
        "pid": "EA60",
        "description": "*tubeszb*",
        "known_devices": [
          "TubesZB Coordinator"
        ]
      },
      {
        "vid": "1A86",
        "pid": "7523",
        "description": "*tubeszb*",
        "known_devices": [
          "TubesZB Coordinator"
        ]
      },
      {
        "vid": "1A86",
        "pid": "7523",
        "description": "*zigstar*",
        "known_devices": [
          "ZigStar Coordinators"
        ]
      },
      {
        "vid": "1CF1",
        "pid": "0030",
        "description": "*conbee*",
        "known_devices": [
          "Conbee II"
        ]
      },
      {
        "vid": "0403",
        "pid": "6015",
        "description": "*conbee*",
        "known_devices": [
          "Conbee III"
        ]
      },
      {
        "vid": "10C4",
        "pid": "8A2A",
        "description": "*zigbee*",
        "known_devices": [
          "Nortek HUSBZB-1"
        ]
      },
      {
        "vid": "0403",
        "pid": "6015",
        "description": "*zigate*",
        "known_devices": [
          "ZiGate+"
        ]
      },
      {
        "vid": "10C4",
        "pid": "EA60",
        "description": "*zigate*",
        "known_devices": [
          "ZiGate"
        ]
      },
      {
        "vid": "10C4",
        "pid": "8B34",
        "description": "*bv 2010/10*",
        "known_devices": [
          "Bitron Video AV2010/10"
        ]
      }
    ],
    "zeroconf": [
      {
        "type": "_esphomelib._tcp.local.",
        "name": "tube*"
      },
      {
        "type": "_zigate-zigbee-gateway._tcp.local.",
        "name": "*zigate*"
      },
      {
        "type": "_zigstar_gw._tcp.local.",
        "name": "*zigstar*"
      },
      {
        "type": "_uzg-01._tcp.local.",
        "name": "uzg-01*"
      },
      {
        "type": "_slzb-06._tcp.local.",
        "name": "slzb-06*"
      }
    ],
    "is_built_in": true
  },
  "data": {
    "ieee": "**REDACTED**",
    "nwk": 31246,
    "manufacturer": "_TZE200_aoclfnxz",
    "model": "TS0601",
    "name": "_TZE200_aoclfnxz TS0601",
    "quirk_applied": true,
    "quirk_class": "zhaquirks.tuya.ts0601_electric_heating.MoesBHT",
    "quirk_id": null,
    "manufacturer_code": 4098,
    "power_source": "Mains",
    "lqi": 172,
    "rssi": -57,
    "last_seen": "2024-03-15T17:50:46",
    "available": true,
    "device_type": "Router",
    "signature": {
      "node_descriptor": "NodeDescriptor(logical_type=<LogicalType.Router: 1>, complex_descriptor_available=0, user_descriptor_available=0, reserved=0, aps_flags=0, frequency_band=<FrequencyBand.Freq2400MHz: 8>, mac_capability_flags=<MACCapabilityFlags.FullFunctionDevice|MainsPowered|RxOnWhenIdle|AllocateAddress: 142>, manufacturer_code=4098, maximum_buffer_size=82, maximum_incoming_transfer_size=82, server_mask=11264, maximum_outgoing_transfer_size=82, descriptor_capability_field=<DescriptorCapability.NONE: 0>, *allocate_address=True, *is_alternate_pan_coordinator=False, *is_coordinator=False, *is_end_device=False, *is_full_function_device=True, *is_mains_powered=True, *is_receiver_on_when_idle=True, *is_router=True, *is_security_capable=False)",
      "endpoints": {
        "1": {
          "profile_id": "0x0104",
          "device_type": "0x0301",
          "input_clusters": [
            "0x0000",
            "0x0004",
            "0x0005",
            "0x0201",
            "0x0204",
            "0xef00"
          ],
          "output_clusters": [
            "0x000a",
            "0x0019"
          ]
        }
      },
      "manufacturer": "_TZE200_aoclfnxz",
      "model": "TS0601"
    },
    "active_coordinator": false,
    "entities": [
      {
        "entity_id": "climate.bathroom_thermostat",
        "name": "_TZE200_aoclfnxz TS0601"
      },
      {
        "entity_id": "select.bathroom_thermostat_keypad_lockout",
        "name": "_TZE200_aoclfnxz TS0601"
      },
      {
        "entity_id": "sensor.bathroom_thermostat_hvac_action",
        "name": "_TZE200_aoclfnxz TS0601"
      },
      {
        "entity_id": "update.bathroom_thermostat_firmware",
        "name": "_TZE200_aoclfnxz TS0601"
      }
    ],
    "neighbors": [
      {
        "device_type": "Router",
        "rx_on_when_idle": "On",
        "relationship": "Sibling",
        "extended_pan_id": "**REDACTED**",
        "ieee": "**REDACTED**",
        "nwk": "0x4898",
        "permit_joining": "Unknown",
        "depth": "15",
        "lqi": "52"
      },
      {
        "device_type": "Router",
        "rx_on_when_idle": "On",
        "relationship": "Sibling",
        "extended_pan_id": "**REDACTED**",
        "ieee": "**REDACTED**",
        "nwk": "0xB25A",
        "permit_joining": "Unknown",
        "depth": "15",
        "lqi": "123"
      },
      {
        "device_type": "Router",
        "rx_on_when_idle": "On",
        "relationship": "Sibling",
        "extended_pan_id": "**REDACTED**",
        "ieee": "**REDACTED**",
        "nwk": "0xF56C",
        "permit_joining": "Unknown",
        "depth": "15",
        "lqi": "52"
      },
      {
        "device_type": "EndDevice",
        "rx_on_when_idle": "Off",
        "relationship": "Child",
        "extended_pan_id": "**REDACTED**",
        "ieee": "**REDACTED**",
        "nwk": "0x87DF",
        "permit_joining": "NotAccepting",
        "depth": "2",
        "lqi": "151"
      },
      {
        "device_type": "EndDevice",
        "rx_on_when_idle": "Off",
        "relationship": "Child",
        "extended_pan_id": "**REDACTED**",
        "ieee": "**REDACTED**",
        "nwk": "0x6960",
        "permit_joining": "NotAccepting",
        "depth": "2",
        "lqi": "120"
      }
    ],
    "routes": [
      {
        "dest_nwk": "0x0000",
        "route_status": "Active",
        "memory_constrained": false,
        "many_to_one": false,
        "route_record_required": false,
        "next_hop": "0xB25A"
      },
      {
        "dest_nwk": "0x4898",
        "route_status": "Active",
        "memory_constrained": false,
        "many_to_one": false,
        "route_record_required": false,
        "next_hop": "0xB25A"
      },
      {
        "dest_nwk": "0x082B",
        "route_status": "Active",
        "memory_constrained": false,
        "many_to_one": false,
        "route_record_required": false,
        "next_hop": "0x082B"
      },
      {
        "dest_nwk": "0x93D0",
        "route_status": "Active",
        "memory_constrained": false,
        "many_to_one": false,
        "route_record_required": false,
        "next_hop": "0xB25A"
      }
    ],
    "endpoint_names": [
      {
        "name": "THERMOSTAT"
      }
    ],
    "user_given_name": "Bathroom thermostat",
    "device_reg_id": "e46bdea69f3400293a20ecdc6533e5e9",
    "area_id": "bathroom",
    "cluster_details": {
      "1": {
        "device_type": {
          "name": "THERMOSTAT",
          "id": 769
        },
        "profile_id": 260,
        "in_clusters": {
          "0x0000": {
            "endpoint_attribute": "basic",
            "attributes": {
              "0x0001": {
                "attribute_name": "app_version",
                "value": 72
              },
              "0x0004": {
                "attribute_name": "manufacturer",
                "value": "_TZE200_aoclfnxz"
              },
              "0x0005": {
                "attribute_name": "model",
                "value": "TS0601"
              }
            },
            "unsupported_attributes": {}
          },
          "0x0004": {
            "endpoint_attribute": "groups",
            "attributes": {},
            "unsupported_attributes": {}
          },
          "0x0005": {
            "endpoint_attribute": "scenes",
            "attributes": {},
            "unsupported_attributes": {}
          },
          "0xef00": {
            "endpoint_attribute": "tuya_manufacturer",
            "attributes": {
              "0x0128": {
                "attribute_name": "child_lock",
                "value": 1
              },
              "0x0101": {
                "attribute_name": "enabled",
                "value": 1
              },
              "0x0402": {
                "attribute_name": "manual_mode",
                "value": 1
              },
              "0x0424": {
                "attribute_name": "running_mode",
                "value": 1
              },
              "0x0403": {
                "attribute_name": "schedule_mode",
                "value": 1
              },
              "0x0210": {
                "attribute_name": "target_temperature",
                "value": 20
              },
              "0x0218": {
                "attribute_name": "temperature",
                "value": 210
              }
            },
            "unsupported_attributes": {}
          },
          "0x0201": {
            "endpoint_attribute": "thermostat",
            "attributes": {
              "0x001b": {
                "attribute_name": "ctrl_sequence_of_oper",
                "value": 2
              },
              "0x0000": {
                "attribute_name": "local_temperature",
                "value": 2100
              },
              "0x0012": {
                "attribute_name": "occupied_heating_setpoint",
                "value": 2000
              },
              "0x0025": {
                "attribute_name": "programing_oper_mode",
                "value": 1
              },
              "0x001e": {
                "attribute_name": "running_mode",
                "value": 0
              },
              "0x0029": {
                "attribute_name": "running_state",
                "value": 0
              },
              "0x001c": {
                "attribute_name": "system_mode",
                "value": 4
              }
            },
            "unsupported_attributes": {
              "0x0010": {
                "attribute_name": "local_temperature_calibration"
              },
              "0x0002": {
                "attribute_name": "occupancy"
              },
              "0x0003": {
                "attribute_name": "abs_min_heat_setpoint_limit"
              },
              "0x0004": {
                "attribute_name": "abs_max_heat_setpoint_limit"
              },
              "0x0005": {
                "attribute_name": "abs_min_cool_setpoint_limit"
              },
              "0x0006": {
                "attribute_name": "abs_max_cool_setpoint_limit"
              },
              "0x0011": {
                "attribute_name": "occupied_cooling_setpoint"
              },
              "0x0007": {
                "attribute_name": "pi_cooling_demand"
              },
              "0x0008": {
                "attribute_name": "pi_heating_demand"
              },
              "0x0013": {
                "attribute_name": "unoccupied_cooling_setpoint"
              },
              "0x0014": {
                "attribute_name": "unoccupied_heating_setpoint"
              },
              "0x0015": {
                "attribute_name": "min_heat_setpoint_limit"
              },
              "0x0016": {
                "attribute_name": "max_heat_setpoint_limit"
              },
              "0x0017": {
                "attribute_name": "min_cool_setpoint_limit"
              },
              "0x0018": {
                "attribute_name": "max_cool_setpoint_limit"
              },
              "0x0030": {
                "attribute_name": "setpoint_change_source"
              }
            }
          },
          "0x0204": {
            "endpoint_attribute": "thermostat_ui",
            "attributes": {
              "0x0001": {
                "attribute_name": "keypad_lockout",
                "value": 1
              }
            },
            "unsupported_attributes": {
              "0x0002": {
                "attribute_name": "schedule_programming_visibility"
              },
              "0x0000": {
                "attribute_name": "temperature_display_mode"
              }
            }
          }
        },
        "out_clusters": {
          "0x000a": {
            "endpoint_attribute": "time",
            "attributes": {},
            "unsupported_attributes": {
              "0x0007": {
                "attribute_name": "local_time"
              },
              "0x0000": {
                "attribute_name": "time"
              }
            }
          },
          "0x0019": {
            "endpoint_attribute": "ota",
            "attributes": {
              "0x0002": {
                "attribute_name": "current_file_version",
                "value": 72
              }
            },
            "unsupported_attributes": {
              "0x0002": {
                "attribute_name": "current_file_version"
              }
            }
          }
        }
      }
    }
  }
}

Logs
Logs
[Paste the logs here]

Custom quirk
Custom quirk
[Paste your custom quirk here]

Additional information
No response

I got the similar issue and want to upvote!!!

Homeassistant
SkyConnect
Electric floor heating
Tuya Thermostat
All Works, but cant read floor temperature which results in max floor temperature of 45 °C

With the default set-up I can read the room temperature from the device and control the floor heating that way. As the thermostat unit controls the underfloor heating in the bathroom but is not actually inside the bathroom, this is not ideal. For what it's worth, I am currently pairing the Moes device with a separate zigbee temperature sensor with the Better Thermostat add-on so that the underfloor heating reacts to the bathroom temperature. I'd still like to read the floor temperature as well for better control.

Do you have any updates on reading the floor temperature of the Zigbee Thermostat?
I'm also looking parrallely for another thermostat that may provide all values (including the floor temperaure) via Zigbee (ZHA) but haven't found any.

Do you have any updates on reading the floor temperature of the Zigbee Thermostat? I'm also looking parrallely for another thermostat that may provide all values (including the floor temperaure) via Zigbee (ZHA) but haven't found any.

None from me, but I have not done any further investigation myself.

I can confirm that with the latest version of everything the issue persists.


Shahar Yair - Learning JavaFX

same as @zahimizrahi, since we both work on the driverGUI.
I too shall master the art of JavaFX, with these general steps:

[x] Step 1: Creating Basic Window

[x] Step 2: Handle User Events

[x] Step 3:  Anonymous Inner Class & Lambda Expressions

[x] Step 4:  Switching Scenes

[x] Step 5:  Creating Alert Boxes

[x] Step 6: Communicating Between Windows

[x] Step 7:  Closing the program properly

[x] Step 8: Embedding Layouts

[x] Step 9: GridPane

[x] Step 10:  Extract and Validate Input

[x] Step 11:  CheckBox

[x] Step 12: ChoiceBox (Drop Down Menu)

[x] Step 13:  Listening for Selection Changes

[x] Step 14: ComboBox

[x] Step 15: ListView

[x] Step 16:  TreeView

[x] Step 17: TableView

[x] Step 18: Making Menus

[x] Step 19: Handling Menu Clicks

[x] Step 20: CheckMenuItem

[x] Step 21: RadioMenuitem

[x] Step 22: CSS

[x] Step 23:  FXML

[x] Step 24:  SceneBuilder

I have covered all the above by watching youtube tutorials. Doesn't mean I'm fluent in all of them, but I got the important subjects, mainly focused on the first subjects an css, fxml. Closing issue.


Make Info_Page responsive

[ ] Info page doesn't look right on small phones or android phones 👍
IOS:

Android:

Looks good on all iphone, need to fix android


Add css and formatting

Added

.css for row colours (init commit)
date formatter
changed row height

#12


Team Drive support

Hi, it doesn't look like this connector supports Team Drives, only the earlier My Drive version of Gdrive. Am I missing something?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2a6ad0f2-376b-0092-71bb-224a95b717a5
Version Independent ID: c6e520d2-dd4d-2516-b3fa-51d4b77c50a6
Content: Connect to Google Drive - Azure Logic Apps
Content Source: articles/connectors/connectors-create-api-googledrive.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

Hi @numeratus Thank you for your feedback! We will investigate and get back to you with our findings.

@numeratus Since there is no update needed for this doc, we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Question: why is the MemorySize for DDBStreamsFunction 3072 ?

Hi, again thanks for putting this sample together, it is much appreciated.
I'm wondering why, unlike the other lambda functions that have 128MB as default, DDBStreamsFunction has MemorySize: 3072. I couldn't find it in the commit history, and it seems to use 24MB or so (which is great!) when running the tests.

Hi @mostalive !
My original reasoning was that, since DDBStreamsFunction will process batches of up to 1000 events, I should allocate more memory to the function to also get more CPU and network resources to speed up the transformation. So this was not driven by memory usage, but for CPU/networking resources.
In reality, the function spends a lot of time waiting for the EventBridge PutRecords API anyway. I just did a test at 128MB and 3072MB under heavy load to compare. The functions takes on average 700ms at 128MB, versus 200ms at 3072MB. As this function is not latency-sensitive, it's best to optimize on costs. I'll make a change lowering the memory to 128MB.

Hi @nmoutschen thanks for the explanation and the change!


[create] don't scaffold demo folder when scaffolding storybook

When the user has selected to use storybook, we shouldn't scaffold the demo files. The user can always add them himself, but this will reduce the amount of files we create.
We will need to change npm/yarn start to start the storybook.

After looking into this, the problem wasn't the generated demo files for the project but rather the generated demo files inside the components folder of an app. Closing this issue.


WIP: Added Optional File Paths for TLS Certs

This PR Is related to issue #6355 and #5582. The idea is that the user can choose to add their own certs if they would like to, in favour of the defaults provided from the image.
This has not been tested properly yet, as I am struggling to get the argo server to run in secure mode from the makefile with make start to a local k3d cluster. Therefore, it would be great if I could get some help with this :)
Checklist:

[ ] My organization is added to USERS.md.

Tips:

Your PR needs to pass the required checks before it can be approved. If the check is not required (e.g. E2E tests) it does not need to pass
Sign-off your commits to pass the DCO check: git commit --signoff.
Run make pre-commit -B to fix codegen or lint problems.
Say how how you tested your changes. If you changed the UI, attach screenshots.

Can be closed as superceeded?

While #5582 / #7621 / #9789 et al did add --tls-certificate-secret-name, using file paths is a better option IMO, as it does not require Argo to have access to the Secret.
Instead, a user can mount the Secret as a volume, and that way Argo does not need to have RBAC to get that Secret.


WebSocketClient can't recive message when WebSocket service message too large

websocketClinet can't recive message when  webSocket service message to large
This tool have some setting about this  to recive large message

Hello @littleBoyGoing
Please use the Bug Report template provided at https://raw.githubusercontent.com/TooTallNate/Java-WebSocket/master/.github/ISSUE_TEMPLATE.md.
Without a full bug report this will be closed.
Best regards,
marci4

No feedback!
Closing dead issue.
Best regards,
marci4


[feat] add kafka streaming for event sourcing/cqrs

Processed events will be automatically streamed to kafka.

@Tochemey just wondering whether do we want to support other message brokers (eg. NATS, RabbitMQ) in the future? if that's the case, perhaps we can use the same plugin mechanism as the plugin store.

@sdil That is a valid point. I need to extract this interface into a more generic way and then we can use the contrib repo to add them.