Add AsyncWriteExt::write_all_vectored utility

Since vectored I/O has been stabilised I was missing this. Two things to note;

this takes &mut [IoSlice], where the call in io::Read takes &[IoSlice]. This is needed to advance the buffer inside the IoSlice.
this uses some unsafe code, I couldn't get around without it.

TODO:

[x] Docs.

@Nemo157 I've created {IoSlice, IoSliceMut}::advance which does the most heavy lifting of this change in https://github.com/rust-lang/rust/pull/62987. However it's an unstable feature, now that Future is also available on stable how are nightly features supposed to be handled?

@Nemo157 sorry about the delay. I've rebased this on master, updated the code to use the newly created io_slice_advance Rust feature and added documentation.

Let me know if you want me to rebase this on the master branch.

@Nemo157 this pr has fallen out of date, but can I get an OK/not OK before spending more time on this?

@cramertj do you have time to review this?

Sure, sorry for the delay! Thanks for the ping.

@cramertj I've rebased this on master and changed the docs to match io::Write::write_all_vectored in the std lib (as merged in #70612).

An unrelated tests failed select_on_non_unpin_size on the CI. @cramertj should I just increase the expected size in test? Or is this being tracked elsewhere?

Fixed in #2119, though that's still failing because of a nightly bug in the rustdoc check.

Now that #2119 is merged I've rebased this.

Friendly ping @cramertj, I don't want to have to rebase this again :)

Thanks for the ping, and for the changes! Sorry for the delay on my end.


feat: add support for PWA

https://issues.jboss.org/browse/INTLY-3566
This PR fixes 2 things:

Add support for PWA. Turns out we don't have it yet and the app won't work if the device is set to offline.
Remove the warning about circular dependencies between the auth service module and the sync service module.

To verify:

Run the app using ionic serve --prod
Login with test/123 and try view and create some tasks (You can do this on a mobile device as well).
Use the network tab to turn offline (or put the app into airplane mode).
You should still be able to use the app and continue creating and viewing tasks.
Bring the device online again and you should see the data is get synced.

An amazing change! Reviewing now

ionic build --prod is required to generate service worker
ionic serve is not affected and works as it was before
auth works fine

@wtrocki thanks, I will made the changes you recommended.

@wtrocki I addressed your comments. I added a new command to make it simple to run the app as PWA app. Also I used serve instead of ionic serve. I am not sure how to get ionic serve to just serving the content from the www directory.


Feature Request: Change /help to /nextron help

Added here https://github.com/0PandaDEV/Nextron/commit/9473c0a404045831286792e17bae07c2bfb35285


fix: mesgdef should clear array pool directly

Since we use append on field array pool, it may grow more than the size of the field array pool. New underlying array will be created, leave behind field array pool data untouched when doing clear.
There is also bug on clearing field array pool on creating unknown messages, if we end up creating new slice, clear(arr[:len(unknownFields)]) may panic as the len of unknownFields is more than arr.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (b85b163) to head (8f7976e).

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #479   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           42        42
  Lines         3810      3810
=========================================
  Hits          3810      3810

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


Remove the lustre chart

Downstream equivalent of https://github.com/openshift-psap/special-resource-operator/pull/251.
/cc @bthurber @dagrayvid

/lgtm


Datasets.jl tests failure

Tests are failing after changing the packaged dataset to be gzipped but the CSV read call hasn't been updated to reflect that.

Relevant PR

Relevant PR

Fixed.

Fixed.


HDDS-3665. Container info command should print uuid of datanode

What changes were proposed in this pull request?
Display the uuid of the datanode to distinguish the datanodes in a same nodes.
With this PR, we can get the following result when info container, in fact i have 5 datanode in the same machine.
Container id: 1
Pipeline id: f47e37ff-7f7e-453b-9bcc-a58e8117b66f
Container State: CLOSED
Datanodes: [localhost, localhost, localhost]

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-3665
How was this patch tested?
bin/ozone admin container info 1
Container id: 1
Pipeline id: f47e37ff-7f7e-453b-9bcc-a58e8117b66f
Container State: CLOSED
Datanodes: [c4dcebcf-3997-4ebd-a501-8d7c3053de61/localhost,
5d7d5f77-57c6-4c90-8edf-eb0f872a0166/localhost,
aa01cb2c-6742-4938-a719-1dc75f76e88f/localhost]

@dineshchitlangia Thank you for telling me this skill, i got it and it is useful for me, Thank you for your quick review.


Remove NEXT_PUBLIC_MACI_START_BLOCK

Replace this argument with start blocks of deployed MACI contract, stored in the args in the localStorage.

[x] remove this arg in .env.example, env.js, and config.ts
[x] return this with maciAddress in the MaciService.getMaciAddress
[x] update getMaciAddress test
[x] update related documents (MaciContext)

close by PR #121


Added proxy support

This allows us to use this within an environment requiring proxy support.

Coverage increased (+3.6%) to 78.571% when pulling d7ca30278943a21c2c7445f0d391f0f9260260fb on markfalk:master into d1a1a6c4271dfc28684c6e1a2ad41dd52c77c463 on jonkemp:master.

Coverage increased (+3.6%) to 78.571% when pulling d7ca30278943a21c2c7445f0d391f0f9260260fb on markfalk:master into d1a1a6c4271dfc28684c6e1a2ad41dd52c77c463 on jonkemp:master.

Coverage increased (+3.6%) to 78.571% when pulling d7ca30278943a21c2c7445f0d391f0f9260260fb on markfalk:master into d1a1a6c4271dfc28684c6e1a2ad41dd52c77c463 on jonkemp:master.

Would the proxy support be considered a new feature or a patch?

I’d consider it a patch. It ensures that it will work in an environment which requires proxies. There are other libraries which do this by default. It broke our workflow until I identified the issue.
From: Jonathan Kemp<EMAIL_ADDRESS>Reply-To: jonkemp/remote-content<EMAIL_ADDRESS>Date: Monday, June 4, 2018 at 7:33 AM
To: jonkemp/remote-content<EMAIL_ADDRESS>Cc: Mark Falk<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [jonkemp/remote-content] Added proxy support (#2)
Would the proxy support be considered a new feature or a patch?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjonkemp%2Fremote-content%2Fpull%2F2%23issuecomment-394376288&data=02|01|mfalk%40dotloop.com|9b554b76c1fd46cfb8de08d5ca281e5c|033464830d1840e7a5883784ac50e16f|0|0|636637196015038828&sdata=4HgEPAi2WhBRb1aiN5pV48ynphnyja61R2uxgGWxypA%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAG95bI7UEfFY5mGDyfQA9slIgYrt65JSks5t5UUvgaJpZM4UOo5O&data=02|01|mfalk%40dotloop.com|9b554b76c1fd46cfb8de08d5ca281e5c|033464830d1840e7a5883784ac50e16f|0|0|636637196015038828&sdata=6kIKD53rMIJCUV9fpARKXnTpP895bdW3zDrSmqsg5%2Bs%3D&reserved=0.


On big image - crash

when upload  image file 8mb, webpage crash...

Reproducable crash of Chrome Version 70.0.3538.102 (Offizieller Build) (64-Bit), Windows 10 Pro 64 bit - November 13, 2018—KB4467708 (OS Build 17763.134), Memory 16 GB, Nvidia GTX 680, latest Driver
with big jpg-image of 73,4 MB.
File cannot be attached: "Yowza, that's a big file. Try again with a smaller file than 10 MB."
Would be helpful to give this information on the Homepage.

Closing this issue since it's been open for long time. consolidate to large photo issue to #298


problem with spawning scene with smoothing

i noticed that when spawning scenes that has this addon, at _ready it will always interpolate from the origin to the point you want the scene to spawn. i looked in the code and i replaced at line 36 this:
`var _m_Trans_curr: Transform2D = Transform2D()
var _m_Trans_prev: Transform2D = Transform2D()`
with this:
`@onready var _m_Trans_curr: Transform2D = get_parent().transform
@onready var _m_Trans_prev: Transform2D = get_parent().transform`
and so far it worked like a charm. for sure it's probably not the best way to go about it (probably), but i wonder if you could implement the fix in a way or another.
:)

Quoting from the docs:

Teleporting
There is one special case when using smoothing nodes - the case when you want a target to instantaneously move from one location to another (for instance respawning a player, or at level start) where you do not want interpolation from the previous position. In this special case, you should move the target node (by setting the translation or position), and then call the 'teleport' function in the Smoothing node. This ensures that interpolation will be switched off temporarily for the move. Make sure to call teleport AFTER moving the target node, rather than before.

Depending on your use case in your game, you can call teleport in _ready() or do what you have done (which is functionally the same). 2D core interpolation actually does this now on entering the scene tree, but core interpolation 3D does not (I think from memory there were some situations where it wasn't wanted, but I wrote this a while ago now).

ah, my bad then, i swear it completely went past my eyes when reading the readme. so sorry

No problem, it's something I may revisit eventually. :+1:

Depending on your use case in your game, you can call teleport in _ready() or do what you have done (which is functionally the same). 2D core interpolation actually does this now on entering the scene tree, but core interpolation 3D does not (I think from memory there were some situations where it wasn't wanted, but I wrote this a while ago now).

I just downloaded 4.x and teleport not happening "on entering the scene tree"

I just downloaded 4.x and teleport not happening "on entering the scene tree"

That is something to report on godot github, I was referring to 3.x (at the time of my post there was no core interpolation in 4.x). rburing is maintaining 4.x (although admittedly based on forward port of my 3.x version).

Sorry, didn't know about that, but is rburing maintaining the 4.x branch on this Repo, or should I use something else? Got a little bit confused now 😅

It is confusing I'll admit.
Godot 3.x
Has built in 3D physics interpolation from 3.5.
Has built in 2D physics interpolation from 3.6.
Godot 4.x
Has built in 2D physics interpolation from 4.3 (afaik). Although it is not feature complete, particles are not yet implemented.
That means there is no longer a need to use the smoothing addon in 3.x (provided you use up to date engine), and soon there will be no need to use it for 2D in 4.x.


[2.2]Install cluster monitoring with system account

Backport for https://github.com/rancher/rancher/pull/19644
Problem:
We are using cluster creator id as cluster monitoring creator. There
will be issue if the cluster creator is disabled or deleted.
Solution:
We should use system account to install cluster monitoring.
Related issue: https://github.com/rancher/rancher/issues/18787

Closing this one because we have cherry-picked the origin PR to v2.2.


[QUESTION] VQGAN Low quality prediction for some audios

Recently I found VQGAN seems not very stable on the prediction quality. It will give obviously low quality prediction on many cases.
Here is one typical bad case for example:
sample-6-gt-and-pred.zip
Is this a weakness of the VQGAN model, or it can be workarounded by having higher sampling rate?
I have seen that in vqgan_pretrain.yaml we have sample_rate: 22050, while in vqgan_pretrain_v2.yaml we have sample_rate: 44100.

We are working on a larger and more robust version of VQGAN v2.


Fixing location uri for DB entities in QuickStartV2

With the present code, on running quick_start.py, locationUri is not flowing in for DB entities. This fix will help associate the locationUri tag to the DB entities.

With the present code, on running quick_start.py, locationUri is not flowing in for DB entities. This fix will help associate the locationUri tag to the DB entities.

Started precommit for this PR - https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/603/console

+1 for the PR , thanks @shrutimantri , thanks @ashutoshm review


GUI not startting

I run Windows 7 x64 and there is no GUI, the program would not start. Can you tell me if I need some special version of .NET or whatever you used?
Event log:
Application: RDAExplorerGUI.exe
Framework Version: v4.0.30319
Description: The process was terminated due to an unhandled exception.
Exception Info: System.TypeInitializationException
Stack:
I tried the updated .NET 4.6.1 and downloaded/installed 4.0 re-distributable, bot the same issue

The application targets .NET Framework 4.5, so I'd expect that running it with 4.6.1 should work fine.
After some googling I have a suspicion... Can you please copy this file into the same directory as the .exe and try launching again: https://www.dropbox.com/s/kdl9bjx9notouex/RDAExplorerGUI.exe.config

I did a quick test at home this morning and the same happens with the config in the folder. I also disconnected my second monitor and the same happens.
No error and no GUI, the process pops up in the task manager and closes shortly after, leaving the event log.
The current .NET version is now 4.5.2 after I uninstalled all of it, then deactivated 3.5.1, reinstalled 4.0 and let Windows patch itself.
Here at work it tells me I need .NET 4.5 to run the application, the machine is not updated regulary and still uses pure 4.0.

Mh, I'm actually not entirely convinced. But as long as I can't reproduce this issue or get more information for debugging, there's not much I can do. In case there was more information in the event log, I would still like to see it, if you find the time to dig this up.
Also, does your work machine use Windows 7 too? Or could there be a difference here?

I am good. In the meantime I changed to Windows 10 and it works fine here.
Really sorry, this was a problem with my system.
http://imgur.com/U5lqTkm

Alright, thanks for clearing that up.


Search text box and advanced search text box.

This pull request adds the simple search text box and advanced search text box.

Hi Ivan,
Updated， Please look again.


problem with jest matchers

Describe the bug
I  tried to generate a new project using yarn create react-app my-project --template typescript. But I have a jest matcher errors right away.
Environment
local dev
Steps to reproduce
yarn create react-app my-project --template typescript
cd my-project
yarn start

Thanks!

This issue is often related to how different versions of TypeScript and Jest interact, especially when type definitions are involved. You might want to check your TypeScript configuration:
In your tsconfig.json file, ensure this option is set:
{
  "compilerOptions": {
    "jsx": "react-jsx"
  }
}

If that doesn't work, try explicitly importing the expect function and any matchers you're using from Jest:

import { expect, test, describe } from '@jest/globals';

test('my test', () => {
  expect(true).toBe(true);
});

I hope this helps!

this issue is often related to how diffrent versions of Typescript and Jest interact, especially when type definitions are involved. You might want to check your Typescript configurations:
In your tsconfig.json file, ensure this option is set:
{
 "compilerOptions": {
     "jsx": "react-jsx"
  }
}

If this doesn't work, try explicitly importing the expect function and any matchers you're using from Jest:
import { expect, test, describe } from '@jest/globals';

test('my test', () => {
  expect(true).toBe(true);
});

I hope this helps!


Ariston Nuos Split

Hello,
I wanted to understand if there was a way to configure with this model, loading it reads the name on the site (as I called the system) but on the integration page I get this error:

when I look on the log page I find this:
2022-11-16 14:47:01.557 ERROR (MainThread) [custom_components.ariston] Your device (4) is currently not supported. Contact with the developer. Your fallback device is Velis Evo maybe working
2022-11-16 14:47:03.890 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry Casa for ariston

Thanks

Duplicate of: https://github.com/fustom/ariston-remotethermo-home-assistant-v3/issues/58


Sending transaction to expired address receive the status "failed"

Description:
After sending transaction to expired address  through 720 blocks transaction receive the status "failed".Expected status "expired".

Logs attached.
logs (5).zip

checked


Broken old behaviour of put method

Our tests was broken due to this change: https://github.com/json-path/JsonPath/pull/775
Before that change this code was working
getJsonFromResource(
                                "/json/BillingDiscountTemplatesControllerUpdateIT/request_empty.json").
                                put("$.validFor", "startDateTime", "2020-08-19T17:47:39.000").jsonString()

request_empty.json is empty {}, thus validFor is absent in json.
I get PathNotFoundException.
I expect an option to get the past behaviour when the absent path is created to add a key.
Now it is not created, and just null or empty list is returned.
If there was an option to handle absent path and create those nodes as in version 2.4.0, that would be cool.

Imo adding extra options to enable exceptions is not a good idea. It's opinionated, but to me an exception should rise when the system has reached an inconsistent state. Ignoring inconsistencies is not something you should be able to toggle at will.
My concern here was not with the change itself, but introducing it in a minor version, not major. I agree that setting a value in a path that does not exist is inconsistent and should fail.

@abelsromero okay okay, so what should we do to make it right? Cancel that pr and move it to a new release?


Reverse proxy not worked in 2.2.3

I have many alluxio clusters,   I  used nginx Reverse proxy in case the master changed，and I use route to distinguish different cluster.for example, http://ip:port/a redirect to alluxio cluster a,http://ip:port/b redirect to alluxio cluster b.
It works in version 1.8.2,but not work in 2.2.3.
In 1.8.2,the network is this.

the network will redirect to http://ip:port/a/api

the network will redirect to http://ip:port/api, the route b is not found.In thi picture,it works because I redirect http://ip:port to cluster b,and I have only one cluster 2.2.3  and others are 1.8.2 now ,but I will updata them all to 2.2.3  in futrue.
It seems use publickPath in webpack will resolve this problem,I'm not sure if it can help.
the config is like this
upstream alluxioa {
    server <IP_ADDRESS>:19999 max_fails=1;
    server <IP_ADDRESS>:19999 backup ;
}
upstream alluxiob {
    server <IP_ADDRESS>:19999 max_fails=1;
    server <IP_ADDRESS>:19999 backup ;
}
server {
    listen 8888 ;
    location /a/ {
        proxy_pass http://alluxioa/;
    }
    location /b/ {
        proxy_pass http://alluxiob/;
    }

}

`

Will close it for now, feel free to reopen it and contact us if this is still valid.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/296 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target.git

Docs Build status updates of commit cfee2de:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-01-14-07-41-33 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-01-14-07-41-33

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.


optimal li instruction

li	a4,4
gets assembled into
lui      x14,0x0
addi     x14, x14, 4
instead of
addi x14, zero,4
So it should check for signed 12bit range and omit the lui if not required.

Notes from our recent discussion of this issue:
Currently, the assembler doesn't support passes that would alter label locations.
Examples of this would be the LI, CALL, and TAIL pseudo-insts (grow) or compressed inst swaps (shrink).

There is currently a chicken and the egg problem:
labels depend on inst sizes
immediates depend on labels
inst sizes depend on immediates

labels -> sizes
imms   -> labels
sizes  -> imms

How can this be avoided?
How do traditional assemblers / linkers solve this?

Just finished up my first crack at this tonight. Go ahead and give LI another try. I've still got a bit more to do on CALL, TAIL, and the compressed instructions. But overall, I think I found a design change that's going to work!

great job on li !
without a single change in the source file:
2072 Bytes
vs
1856 Bytes
there is still one thing missed, this time 'the other way round'
li t3, 16384 becomes:
67c: 00004e37 lui x28 0x4
680: 000e0e13 addi x28 x28 0
so a redundant add of 0.
I have a few of such cases. But that looks like a reasonably simply fix fortunately.

Good call! I've got this implemented now, too. Go ahead and see if everything looks correct on your end.


Haniffa - Covid19PBMC E-MTAB-10026

Primary Wrangler:
Wei Kheng Teh
Secondary Wrangler:
Marion

Associated files:
Pre-print: https://www.medrxiv.org/content/10.1101/2<IP_ADDRESS>49725v1.full
Google Drive: https://drive.google.com/drive/folders/1uoTWDLj57LTM9ZT2ceUQU8kF5QeWJmTM
Data available in the Covid19 Portal: https://www.covid19cellatlas.org/index.patient.html
Processed data available at E-MTAB-10026: https://www.ebi.ac.uk/arrayexpress/experiments/E-MTAB-10026/
Key Events

[x] Send initial instructions about contributing, including questionnaire and T&Cs
[x] Receive project questionnaire
[x] Assign the Primary Wrangler
[ ] Receive T&C form filled out by at least 1 representative of group/project
[ ] Send custom metadata template spreadsheet, how-to guide and directions for submitting data, matrix and cell annotation files.
[x] Receive spreadsheet (to a sufficient standard to submit dataset)
[x] Receive data, matrix and cell annotation files (to a sufficient standard to submit dataset)
[x] Curate metadata spreadsheet with ontologies
[x] Upload sheet to validate metadata
[x] Run graph validator & Check linking
[x] Validate data files
[x] Ask the Secondary Wrangler for an end-to-end review of the project. Ask the Expertise Wrangler to review specific tabs if needed
[ ] Get final approval of submission from the contributor including release date, metadata spreadsheet and data files
[x] Submit to production and inform ingest-devs the project is ready for archiving
[ ] Convert project data to SCEA format following the SCEA conversion SOP if appropriate

Please track the below as well as the key events:

Track date first spreadsheet received and final spreadsheet sent by editing ticket to include
date next to event.
Track spreadsheet iterations by placing asterisks next to receive spreadsheet event.
Track any metadata issues/tickets made for dataset with a bulleted list of links under
received spreadsheet event. Links should be to the ticket in the metadata repo.

The project accession is now public, and will begin wrangling.

This project is now ready for secondary review.
Project is here: https://staging.contribute.data.humancellatlas.org/submissions/detail?id=606c4377ed3471456300379a&project=037673f8-323f-416e-92ef-c76c8e215b2f
There is an Ingest-graph-validator issue:
21-04-06 12:14:39 [ingest_graph_validator.actions.test_action] - ERROR: test [file_donthave_outward_links.adoc] failed: non-empty result. 21-04-06 12:14:39 [ingest_graph_validator.actions.test_action] - ERROR: result: [{'f': (_381:analysis_file:file {describedBy: 'https://schema.humancellatlas.org/type/file/6.2.0/analysis_file', file_core.content_description: "[{'text': 'h5ad file  for BCR enriched libraries, TCR enriched lirbaries, cell surface proteins, and gene expression data', 'ontology': 'data:2082', 'ontology_label': 'Matrix'}]", file_core.file_name: 'covid_portal.h5ad', file_core.format: 'h5ad', id: 'covid_portal.h5ad', schema_type: 'file'})}, {'f': (_382:analysis_file:file {describedBy: 'https://schema.humancellatlas.org/type/file/6.2.0/analysis_file', file_core.content_description: "[{'text': 'csv file with barcodes and annotated  cell types and cell surface proteins', 'ontology': 'data:2655', 'ontology_label': 'Cell type identifier'}]", file_core.file_name: 'annotation_200112.csv', file_core.format: 'csv', id: 'annotation_200112.csv', schema_type: 'file'})}]
Looking at the test itself (https://github.com/ebi-ait/ingest-graph-validator/blob/master/graph_test_set/file_donthave_outward_links.adoc) and looking at the specific files in the ingest-graph-validator, there are no outward links from the analysis_file that link outwards, so I'm not sure what the problem is unless it's taking the 'Project' link and failing because of that.

Updating to say that this error 'file_donthave_outward_links' is also found in many other projects, and is also caused by sequence files which do not have outward links (via a process) to other entities. Branching off this issue to a separate ticket to look at it, but it's not essential at the moment (unless secondary reviewer thinks otherwise!)

Wei and I had a discussion about some queries I had when secondary reviewing. Overall it was really good and he has contacted the contributor for a few extra details.

Just noticed this is now published here: https://www.nature.com/articles/s41591-021-01329-2

The EGA submission was successful with accession : EGAS00001005465
The following updates need to be made:

add the EGA accession to the project field
update the analysis protocol ontology with the new ontologies

Added EGA Accession and analysis protocol ontology (processed_matrix_generation) just need to export

The analysis protocol ontologies are not exposed anywhere in the DCP so it's virtually impossible to check
The EGA accession has been added successfully and the number of files seems normal.
Moving it to verified, let me know if you'd like to move it back


debug.internalConsoleOptions not honored

Copied from #160265. We don't believe this is caused by the makefile tools extension.
Using GDB as the debugger, compiling via the extension Makefile tools with GCC as the compiler, as provided by MSYS2+MinGW64 on Windows 10.
Debugging is provided by the command makefile.outline.launchDebug
Every time I start debugging, it switches to the debug console. This is fairly annoying, since I want to see the actual console with its outputs and local variables in the side bar. The debug console taking priority and stealing focus from the output terminal is a small bump each time I debug.
According to https://github.com/microsoft/vscode/issues/11372 you are supposed to set "debug.internalConsoleOptions": "neverOpen", but that setting is not honored. I have set this in launch.json, settings.json and in the GUI. It is simply ignored in the case of the GDB debug console.
Does this issue occur when all extensions are disabled?: Yes
Version Info
Version: 1.71.0 (user setup)
Commit: 784b0177c56c607789f9638da7b6bf3230d47a8c
Date: 2022-09-01T07:36:10.600Z
Electron: 19.0.12
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19044
Sandboxed: No
Steps to Reproduce:
Set "debug.internalConsoleOptions": "neverOpen"
Launch with GDB as the debugger

That version of vscode is a year and a half old, can you check that it's still an issue on the latest?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Installed via Manager (and StableSwarm) - Nodes Show up but do not work

Hello,
I'm using ComfyUI through StableSwarm. I installed the Anywhere Everywhere nodes via the Manager. I get the new nodes, but when I connect them, their status/name doesn't change, and they remain inactive. I also do not get any new options when right clicking on the empty canvas. I tried all the troubleshooting steps from the Readme file. Nothing fixes it. Now, I am definitely showing some use_everywhere errors on the JavaScript console, and here's the log taken from an empty canvas:
**main.js?vary=<IP_ADDRESS>:988 Load page...
site.js?vary=<IP_ADDRESS>:156 Session started.
main.js?vary=<IP_ADDRESS>:992 First session loaded - prepping page.
scripts/app.js:1
   Failed to load resource: the server responded with a status of 404 (Not Found)

logging.js:315 Error loading extension /extensions/use_everywhere/use_everywhere.js TypeError: Failed to fetch dynamically imported module: http://localhost:7801/ComfyBackendDirect/extensions/use_everywhere/use_everywhere.js
console. @ logging.js:315
logging.js:315 Error loading extension /extensions/use_everywhere/use_everywhere_classes.js TypeError: Failed to fetch dynamically imported module: http://localhost:7801/ComfyBackendDirect/extensions/use_everywhere/use_everywhere_classes.js
console. @ logging.js:315
logging.js:315 Error loading extension /extensions/use_everywhere/use_everywhere_autocreate.js TypeError: Failed to fetch dynamically imported module: http://localhost:7801/ComfyBackendDirect/extensions/use_everywhere/use_everywhere_autocreate.js
console. @ logging.js:315
logging.js:315 Error loading extension /extensions/Davemane42/MultiAreaConditioning.js TypeError: Failed to fetch dynamically imported module: http://localhost:7801/ComfyBackendDirect/extensions/Davemane42/MultiAreaConditioning.js
console. @ logging.js:315
logging.js:315 Error loading extension /extensions/Davemane42/MultiLatentComposite.js TypeError: Failed to fetch dynamically imported module: http://localhost:7801/ComfyBackendDirect/extensions/Davemane42/MultiLatentComposite.js
console. @ logging.js:315
logging.js:315 Error loading extension /extensions/use_everywhere/use_everywhere_ui.js TypeError: Failed to fetch dynamically imported module: http://localhost:7801/ComfyBackendDirect/extensions/use_everywhere/use_everywhere_ui.js
console. @ logging.js:315
scripts/api.js:1
   Failed to load resource: the server responded with a status of 404 (Not Found)

logging.js:315 Error loading extension /extensions/ComfyUI-Custom-Scripts/js/autocompleter.js TypeError: Failed to fetch dynamically imported module: http://localhost:7801/ComfyBackendDirect/extensions/ComfyUI-Custom-Scripts/js/autocompleter.js
console. @ logging.js:315
scripts/widgets.js:1
   Failed to load resource: the server responded with a status of 404 (Not Found)

scripts/ui.js:1
   Failed to load resource: the server responded with a status of 404 (Not Found)

logging.js:315 [BilboX] Installing bilbox.css...
logging.js:315 [🐍 pysssss] Skipping LinkRenderMode as core extension found
Text2Image:1 Uncaught (in promise) Error: A listener indicated an asynchronous response by returning true, but the message channel closed before a response was received**
Any hint on to how to fix these errors is much appreciated, tks!

Looks like the javascript files aren't being deployed (for this or for extensions/Davemane42). I don't know anything about StableSwarm, unfortunately.

Raised a question with the StableSwarm repo https://github.com/Stability-AI/StableSwarmUI/issues/180

Thanks for your reply!

Should be fixed when I do #64

I'll have to be in later tonight when I get back home ~ I'll keep you posted, thanks again!

Sorry, not much time available to check ~ quick test after update of Swarm/Comfy/Nodes shows that it still doesn't work. I hope to have more time in the next few days to give you more details but as of now, got to go.

Just now pushed another update which might be relevant, so make sure you have the latest before testing again

Hi Chris,
Thanks. I won't have access to my computer for the next 12-15 hours. Will
check as soon as I can and will keep you posted.
Thanks again for working on it. Cheers ~
On Sat, Dec 2, 2023, 09:19 chrisgoringe @.***> wrote:

Just now pushed another update which might be relevant, so make sure you
have the latest before testing again
—
Reply to this email directly, view it on GitHub
https://github.com/chrisgoringe/cg-use-everywhere/issues/58#issuecomment-1836997855,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFOGI6G5IL7RXTG7GVGQGVTYHKF3VAVCNFSM6AAAAAA7ROJW22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZWHE4TOOBVGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


bugfix: fix getConfig cache value is 'null'

Ⅰ. Describe what this PR did
bugfix: fix getConfig cache value is 'null'
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #2904 into develop will decrease coverage by 0.14%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             develop    #2904      +/-   ##
=============================================
- Coverage      50.09%   49.95%   -0.15%
+ Complexity      3015     3012       -3
=============================================
  Files            597      597
  Lines          19291    19267      -24
  Branches        2347     2304      -43
=============================================
- Hits            9664     9624      -40
+ Misses          8667     8657      -10
- Partials         960      986      +26

Impacted Files
Coverage Δ
Complexity Δ

.../main/java/io/seata/config/ConfigurationCache.java
42.10% <0.00%> (-2.64%)
9.00 <0.00> (ø)

...obuf/convertor/BranchRegisterRequestConvertor.java
90.47% <0.00%> (-9.53%)
3.00% <0.00%> (ø%)

...otobuf/convertor/BranchReportRequestConvertor.java
90.90% <0.00%> (-9.10%)
3.00% <0.00%> (ø%)

...obuf/convertor/BranchRollbackRequestConvertor.java
92.00% <0.00%> (-8.00%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalBeginResponseConvertor.java
92.59% <0.00%> (-7.41%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalCommitRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalStatusRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...obuf/convertor/GlobalRollbackRequestConvertor.java
93.75% <0.00%> (-6.25%)
3.00% <0.00%> (ø%)

...protobuf/convertor/RegisterTMRequestConvertor.java
94.73% <0.00%> (-5.27%)
3.00% <0.00%> (ø%)

...protobuf/convertor/RegisterRMRequestConvertor.java
90.47% <0.00%> (-4.77%)
3.00% <0.00%> (ø%)

... and 25 more


Could not open library 'libsecp256k1'

Followed the README docs,  after run bundle exec pry -r ./lib/ckb/wallet.rb, I got an error on MacOS with ruby 2.3.7p456:
bundler: failed to load command: pry (/usr/local/bin/pry)
LoadError: Could not open library 'libsecp256k1': dlopen(libsecp256k1, 0x0005): dlopen(): file not found: libsecp256k1.
Could not open library 'libsecp256k1.dylib': dlopen(libsecp256k1.dylib, 0x0005): dlopen(): file not found: libsecp256k1.dylib
  /Library/Ruby/Gems/2.3.0/gems/ffi-1.9.25/lib/ffi/library.rb:145:in `block in ffi_lib'
  /Library/Ruby/Gems/2.3.0/gems/ffi-1.9.25/lib/ffi/library.rb:99:in `map'
  /Library/Ruby/Gems/2.3.0/gems/ffi-1.9.25/lib/ffi/library.rb:99:in `ffi_lib'
  /Library/Ruby/Gems/2.3.0/gems/bitcoin-secp256k1-0.4.0/lib/secp256k1/c.rb:9:in `<module:C>'
  /Library/Ruby/Gems/2.3.0/gems/bitcoin-secp256k1-0.4.0/lib/secp256k1/c.rb:6:in `<module:Secp256k1>'
  /Library/Ruby/Gems/2.3.0/gems/bitcoin-secp256k1-0.4.0/lib/secp256k1/c.rb:5:in `<top (required)>'
  /Library/Ruby/Gems/2.3.0/gems/bitcoin-secp256k1-0.4.0/lib/secp256k1.rb:3:in `require'
  /Library/Ruby/Gems/2.3.0/gems/bitcoin-secp256k1-0.4.0/lib/secp256k1.rb:3:in `<top (required)>'
  /Users/xxx/projects/ckb-demo-ruby-sdk/lib/ckb/utils.rb:1:in `require'
  /Users/xxx/projects/ckb-demo-ruby-sdk/lib/ckb/utils.rb:1:in `<top (required)>'
  /Users/xxx/projects/ckb-demo-ruby-sdk/lib/ckb/api.rb:1:in `require_relative'
  /Users/xxx/projects/ckb-demo-ruby-sdk/lib/ckb/api.rb:1:in `<top (required)>'
  /Users/xxx/projects/ckb-demo-ruby-sdk/lib/ckb/wallet.rb:1:in `require_relative'
  /Users/xxx/projects/ckb-demo-ruby-sdk/lib/ckb/wallet.rb:1:in `<top (required)>'
  /Library/Ruby/Gems/2.3.0/gems/pry-0.12.2/lib/pry/pry_class.rb:113:in `require'
  /Library/Ruby/Gems/2.3.0/gems/pry-0.12.2/lib/pry/pry_class.rb:113:in `block in load_requires'
  /Library/Ruby/Gems/2.3.0/gems/pry-0.12.2/lib/pry/pry_class.rb:112:in `each'
  /Library/Ruby/Gems/2.3.0/gems/pry-0.12.2/lib/pry/pry_class.rb:112:in `load_requires'
  /Library/Ruby/Gems/2.3.0/gems/pry-0.12.2/lib/pry/pry_class.rb:156:in `final_session_setup'
  /Library/Ruby/Gems/2.3.0/gems/pry-0.12.2/lib/pry/cli.rb:79:in `parse_options'
  /Library/Ruby/Gems/2.3.0/gems/pry-0.12.2/bin/pry:7:in `<top (required)>'
  /usr/local/bin/pry:22:in `load'
  /usr/local/bin/pry:22:in `<top (required)>'

No experience in ruby, how should I do?

Solution
https://github.com/nervosnetwork/ckb-demo-ruby-sdk/issues/13#issuecomment-447706499

[ ] Mention that secp256k1 should be installed manually in README

This is actually a dependency in the gem bitcoin-secp256k1, you can follow the prerequiste part in the gem to install secp256k1 library locally.

@ashchan I'm also thinking maybe just switching a different gem which doesn't require dynamic linking, it might ease the installation step a bit, thoughts?

This is actually a dependency in the gem bitcoin-secp256k1, you can follow the prerequiste part in the gem to install secp256k1 library locally.

It works, thanks! But i got another problem, see #14

@xxuejie Will add note to the README.

@xxuejie Will add note to the README.

@ashchan will or @xxuejie will?

@doitian I'll do that.


[OSD-19291] update service quota documentation

What is being added?
Updated documentation
Ref OSD-19291

Codecov Report

Merging #789 (f5396fd) into master (fd2734d) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #789   +/-   ##
=======================================
  Coverage   42.96%   42.96%
=======================================
  Files          43       43
  Lines        5942     5942
=======================================
  Hits         2553     2553
  Misses       3148     3148
  Partials      241      241

/approve


jsdom | matchMedia not present, legacy browsers require a polyfill

Hi
I am using jsdom with karma for react testing and following is my test file.
The module PDP gallery is importing react-slick.
import React from 'react';
import ReactDOM from 'react-dom';
import ReactTestUtils from 'react-addons-test-utils';
import expect from 'expect';
import PDPGallery from 'pages/pdp/components/pdp-gallery';
import configureStore from 'redux-mock-store';
import { Provider } from 'react-redux';
import thunk from 'redux-thunk';
describe('Image Gallery Test', () => {
const store = mockStore(initialState);
const navThumbnailClick = (abc) => {
console.log('quick View Handler');
};
before(()=>{
});
it('Test if pdp gallery gets displayed',() => {
  expect(gallery_list).toExist();
});

});
As soon as my react component is loaded the following error comes
matchMedia not present, legacy browsers require a polyfill
Found this solution :
https://github.com/WickyNilliams/enquire.js/issues/82
but not sure where to put this stub -
window.matchMedia = window.matchMedia || function() {
return {
matches : false,
addListener : function() {},
removeListener: function() {}
};
};
Please help

Check solution provided in this issue
https://github.com/akiran/react-slick/issues/472


MIGraphX Inference Examples and Deployment

General goals are highlighted below.

[ ] Model optimizer
[ ] Inference examples
[ ] Out-of-box experience improvements

Added #776 for BERT-based question answering example

PR'ed #776 #789 for release #766

Closing this for 1.2; and will reopen separate tickets for 1.3.


Pull fails after commiting inside docker container.

Bug
Difficult to fetch from the repository after running git commit.

To Reproduce
Steps to reproduce the behavior:
When following Getting Started,
presumably at make setup-linters , pre-commit run --all-files, git commit inside the docker, permissions under .git change for the following folders from user to root
-rw-r--r--  1 root  root     73 Dec  7 11:46 COMMIT_EDITMSG
-rw-r--r--  1 root  root      0 Dec  7 12:34 FETCH_HEAD
-rw-r--r--  1 root  root     23 Dec  7 12:28 HEAD
-rw-r--r--  1 root  root    322 Dec  7 12:30 config
-rw-r--r--  1 root  root  39355 Dec  7 12:28 index

This prevents git operations outside the docker like
~/dgp$ git pull
error: cannot open .git/FETCH_HEAD: Permission denied

and inside the docker like
/home/dgp# git pull
Bad owner or permissions on /root/.ssh/config
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

For shared development devices where users do not have root permissions, it is impossible to change the permissions affected by the container.

Expected behavior

Being able to pull either from inside or outside the container. With above constraint (no root permissions), both workflows are blocked unless setting up repo from scratch again.
The best way would probably be for above folder permissions to not change to root in the first place 🤔
Additional context

Thanks for reporting this!
I just inspected the Docker configuration. make docker-start-interactive drops you into a Docker container as root.   So, pre-commit should only be placing files inside the context of your container. That the error you see is in particular is with permissions under .git, I suspect this is related to running git operations only. This will affect your host as the codebase is bind-mounted from your host into the container.
As for the git issues inside the container, git does not seem to be provided in the image fetched by docker pull ghcr.io/tri-ml/dgp:master. I'm guessing you installed it locally.

Bad owner or permissions on /root/.ssh/config

Since you are operating as root, I believe what is happening here is that git is refusing to use your SSH identity because your active user is root but the uid/gid on your ssh files is brought in from your host environment and is not literally root.

BTW I've never actually used the Docker container for development myself, only the virtual environment. I'm guessing the folks who set up this workflow only intend for users to make code changes inside the container and not perform git operations. We'd still hit permissions issues if people created even source files inside the container, though.

I'm guessing you installed it locally.

Right, the GETTING STARTED also provides instructions on this
To set up linters wherever you plan on using git from (either on your host or inside the container), run

# If in Docker, run the following; otherwise skip this block.
dgp$ apt update
dgp$ apt install -y git
dgp$ git config --global --add safe.directory /home/dgp

# Set up the linters.
dgp$ make setup-linters

BTW I've never actually used the Docker container for development myself, only the virtual environment. I'm guessing the folks who set up this workflow only intend for users to make code changes inside the container and not perform git operations.

Hm understood. So I should run pre-commit separately inside container and always commit with --no-verify outside container...

Btw from the docs, it seems the workflow allows for users to use git inside the container (just not on devices where user doesn't have root permissions I guess...)

To set up linters wherever you plan on using git from (either on your host or inside the container)

Thanks; maintainers should probably change the docs to avoid guiding users into the situation you found yourself in. I have to delegate this at this time, though.

Maybe an opportunity for @ryo-takahashi-1206 and company to support?

Thank you for sharing. I will do the followings within 1 week:

Try bringing the host UID to the container

if no one is affected, make this the default behavior
Otherwise, invent some option to bring the host UID

Or, show alarts in make docker-start-interactive and the relvant docs

Thanks so much Takahashi-san; SGTM :)

Thank you Tyler and Takahashi-san 🙇♀

Linking the fix for the issue: https://github.com/TRI-ML/dgp/pull/138

@nehalmamgain
Please close this issue if you become happy with https://github.com/TRI-ML/dgp/pull/138 🙇

Thank you for the fix!


Alert menu wrong position

Windows 10
Firefox 68.0.1
when clicking on the alert bell icon, the alert menu appears off-screen to the right instead of below the icon

@KevinFarrow You find this bug?

@patrykKlimczak Not sure what you mean?

@ClintOxx what is the status on this fix, please tell me, if you need any help, please mark it as help wanted so that we can try to fix it.

@anselal I am on it.

@anselal @KevinFarrow @ClintOxx  There you have it. This pull request fixes the bug.
Please see this:

will check it out, @MiKr13

@MiKr13 I meant to say thanks for the pull request

@MiKr13 I meant to say thanks for the pull request

My pleasure man, keep up the awesome work!


[Settings]: 'Profile' text is not properly visible in High Contrast Desert Theme.

Windows Terminal version (or Windows build number)
1.10.2383.0
Other Software
Test Environment:
OS: Windows 11 Version 21H2 (OS Build 22000.168)
Screen Reader: Narrator
Tool: Accessibility Insight for Windows
Steps to reproduce
Prerequisites:

Open Settings app using 'Windows + i' Key.
Navigate to 'Accessibility' list item and activate it.
Then Navigate to 'Contrast themes' tab and activate it.
Apply Desert theme.

Repro Steps:
Open Windows Terminal.
Open Settings page using "Ctrl + ',' ".
Verify that 'Profile' text is properly visible in High Contrast Desert Theme or not.
User Experience:
User who user High contrast theme to use apps will be impacted here as they will not unable to see text if it is not visible properly.
Guideline Reference:
EN 301 549 V3.2.1 - <IP_ADDRESS> No disruption of accessibility features
Attachments:
Profile text is not visible in High Contrast Desert Theme Which is visible in Normal theme and High Contrast Aquatic theme..zip
Expected Behavior
Profile text should visible properly in High Contrast Desert Theme Which is visible in Normal theme and High Contrast Aquatic theme..
Actual Behavior
Profile text is not visible in High Contrast Desert Theme Which is visible in Normal theme and High Contrast Aquatic theme..

This sure looks like something that should be fixed upstream in WinUI - we're not providing any custom styling for the NavigationView for Desert Theme, which would suggest that any apps that are using the NavigationView in WinUI would similarly face this issue. @v-rpundir can we move this to an upstream issue on https://github.com/microsoft/microsoft-ui-xaml?

@zadjii-msft Yup you can move this issue to https://github.com/microsoft/microsoft-ui-xaml

Closing the bug as issue has been fixed.


3DX New Sheet

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <3DX>  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is:  (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <3DX> (i.e. Dungeons & Dragons, FATE)

[x] I have followed the Character Sheets Standards when building this sheet.

[x] This sheet has been made on behalf of, or by, the game's publisher.
[x] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: https://www.sistema3dx.com.br
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Character Sheet Info Roll20 Internal Use only.

*The publisher of this game is: Victor Ballogh

Fala @Vitones91, muito organizado o código da ficha. Show!

Fala @Vitones91, muito organizado o código da ficha. Show!

Opaa, muito obrigado @mperes! Espero que esteja tudo certinho para a aprovação.

Sim, acabei de publicar.

Sim, acabei de publicar.

Não tinha visto, show de bola, valeu!


⚠ Samsung has degraded performance

In 897c14f, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2679 ms

Resolved: Samsung performance has improved in 59817f8.


No error UI for handling "unregistered JSX" in MDX.

Description
I added a collection that manages MDX pages. When navigating to this collection in the admin, I get a blank page.

The issue is likely because I have yet to define my MDX templates in the schema:
import { defineSchema } from "@tinacms/cli";

export default defineSchema({
  collections: [
    {
      label: "Slides",
      name: "slides",
      format: "mdx",
      path: "content/slides",
      fields: [
        {
          label: "Body",
          name: "body",
          isBody: true,
          type: "rich-text",
          templates: [],
        },
      ],
    },
  ],
});

The console shows: "  Error: Found unregistered JSX or HTML: . Please ensure all structured elements have been registered with your schema.". Can we show an error for this in the UI?
Tools
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>

This is happening on the collection list page, so if you have any collection-item using a missing template, it breaks the whole collection

The admin is showing a toast error message for this now!


Is this project still under active development?

I noticed that there haven't been any recent updates to this project. Is it still being maintained?

Hi @archyly ,
I think it needs someone to champion it.  The builds are not working well for a while now.  The only thing happening is PRs from renovate so yes its dormant and if I'm not mistaken spring changed direction some for major upgrade so it needs some work.  If you or anyone else that wants to use this has some time to provide some effort getting it cleaned up, we will progress it.  I cannot speak for all but usually if there isn't direct usage, things tend to die on the vine.  I don't need this currently and think at best any personal usage is years off.  So it really needs others that feel its important to move it forwards.  We will help in any way in doing so.  Thanks for checking!

Hi @archyly ,
I think it needs someone to champion it. The builds are not working well for a while now. The only thing happening is PRs from renovate so yes its dormant and if I'm not mistaken spring changed direction some for major upgrade so it needs some work. If you or anyone else that wants to use this has some time to provide some effort getting it cleaned up, we will progress it. I cannot speak for all but usually if there isn't direct usage, things tend to die on the vine. I don't need this currently and think at best any personal usage is years off. So it really needs others that feel its important to move it forwards. We will help in any way in doing so. Thanks for checking!

Thank you for your reply.


test(mempool): tx added to the mempool are forwarded to the p2p propagator client

This change is


Layout Flags and Mobile settings views

So @tibetsprague thank you for reviewing that Mobile Settings Layout PR on Evo last week!
Unfortunately I should have flagged it as “draft” as it was still in fact a WIP.
Most of what you saw at the time you reviewed, the basic things related to the layout remains unchanged, but since you reviewed I’ve done substantial work on that branch on the Group Settings UI.
It’s ready for another review now. I'm hoping to get this release to coordinate with a Mobile release I'd like to start to QA against production sometime before Sprint Review.
Flagging @thomasgwatson as well in case you have time and a 2nd pass review might be for more benefit to you since Tibet already looked at an earlier version 🤷🏼♂
Thank you!

Thanks for getting right on that @tibetsprague ! Updates and notes made 👍🏼


update novnc

Codecov Report

Merging #107 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #107   +/-   ##
=======================================
  Coverage   57.95%   57.95%
=======================================
  Files           2        2
  Lines          88       88
=======================================
  Hits           51       51
  Misses         32       32
  Partials        5        5

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 29fd8c3...e453d6c. Read the comment docs.

@lanwen you can merge if OK.


Add configuration to disable forwarding DNS queries for external domains by NodeLocalDNS.

How to categorize this PR?

/area networking
/area usability
/kind enhancement
What this PR does / why we need it:
Custom DNS configuration for external domains is not working  in conjunction with NodeLocalDNS, as DNS requests for external domains are directly forwarded to an upstream DNS server.
This PR adds an additional option to shoot spec, to disable forwarding to upstream DNS. Instead of that, DNS requests for external domains are forwarded towards coreDNS, where custom config will be evaluated.
Drawback of this solution is, that there will be more load on cluster coreDNS. However, as queries for external domains are cached by node-local-dns the increase should not be significant.
For clusters, that don't use custom DNS config, it is still recommended not to disable the forwarding to upstream DNS.
Which issue(s) this PR fixes:
Fixes #6852
Special notes for your reviewer:
Release note:

Forwarding DNS queries to upstream DNS from node local DNS can be disabled, to use custom DNS config in conjunction with NodeLocalDNS.

/label do-not-merge/work-in-progress

/test all

cc @DockToFuture @ScheererJ

/test all

/test all

/test all

/test pull-gardener-integration


Add flowchart for client and container creation

Fixes https://github.com/containers/conmon-rs/issues/345

Codecov Report

Merging #397 (756638b) into main (db4ba32) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 756638b differs from pull request most recent head bf75c9c. Consider uploading reports for the commit bf75c9c to get more accurate results

@@          Coverage Diff          @@
##            main    #397   +/-   ##
=====================================
  Coverage   6.89%   6.89%
=====================================
  Files          2       2
  Lines         29      29
  Branches      17      17
=====================================
  Hits           2       2
  Misses        27      27


Improve examples

Using this example, it should be easier for everyone to log their state in a human readable way, no matter whether they use combineReducers or not.

Codecov Report

Merging #299 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #299   +/-   ##
=====================================
  Coverage      82%    82%
=====================================
  Files           5      5
  Lines         150    150
=====================================
  Hits          123    123
  Misses         27     27

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 3ca9f2c...6bac491. Read the comment docs.


Uncaught Error: Main (:[R]#1:[0]App): Return value for _getFocused must be an attached descendant component but its ':[R]#1:[0]App:[1]Main'

I am following this tutorial and it is complaining about Uncaught Error: Main (:[R]#1:[0]App): Return value for _getFocused must be an attached descendant component but its ':[R]#1:[0]App:[1]Main'  https://rdkcentral.github.io/Lightning/docs/gettingStarted/start-development
here is my repo. https://github.com/shorif2000/newsfeed
The documentation does not look up to date and is missing key information

Issue happening to me, as well


Update the semaphore impl in tower-batch, to avoid hangs

Is your feature request related to a problem? Please describe.
tower-batch is missing a semaphore fix that wakes waiting clones on close: tower-rs/tower#480
Describe the solution you'd like

[x] Copy the version of semaphore.rs included in our pinned version of tower, to tower-batch/src/: https://github.com/tower-rs/tower/blob/d4d1c67c6a0e4213a52abcc2b9df6cc58276ee39/tower/src/semaphore.rs

Describe alternatives you've considered
Do nothing: tower-buffer users might hang.
Additional context
This issue was discovered during the review in #1593.

This issue actually causes hangs, so I've added it to the next sprint.

This fix was incomplete, it also needs some changes in tower_buffer::Buffer (and maybe other files).
We should follow tower-rs/tower#480 almost exactly.


⚠ ClamAV has degraded performance

In 0c68d65, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 262 ms

Resolved: ClamAV performance has improved in 975c501.


Error receiving events from partition New receiver 'nil' with higher epoch of '0' is created hence current receiver 'nil' with epoch '0' is getting disconnected.

Query/Question
We are consuming events from event hub using azure cloud stream binder application and sometimes getting the error "Error receiving events from partition New receiver 'nil' with higher epoch of '0' is created hence current receiver 'nil' with epoch '0' is getting disconnected"
When we are checking the output data we are seeing some missing events. Please confirm if this error can cause some event to be missed out from processing in the consumer instances.
However if we sent the events again then the output count matches and this error is also not there..
Setup :

OS: Linux (AKS pods)
Event hub Partition: 25
Consumers: 25
IDE: Running as a POD in AKS
Library/Libraries: [e.g. com.azure.spring:azure-spring-cloud-stream-binder-eventhubs:2.8.0]

Information Checklist
Error details:
com.azure.core.amqp.implementation.ExceptionUtil.toException(ExceptionUtil.java:85)
datautil-5b9bc95dbf-snwfc/Console.log:com.azure.core.amqp.exception.AmqpException: New receiver 'nil' with higher epoch of '0' is created hence current receiver 'nil' with epoch '0' is getting disconnected. If you are recreating the receiver, make sure a higher epoch is used. TrackingId:8efac11d00006a00003e72bb61e07a3c_G21_B26, SystemTracker:****-eastus2-eventhub-01:eventhub:xxxxxxx_eh~22286|$default, Timestamp:2022-01-13T19:15:26, errorContext[NAMESPACE: ****-eastus2-eventhub-01.servicebus.windows.net. ERROR CONTEXT: N/A, PATH: xxxxxx_valids_eh/ConsumerGroups/$default/Partitions/16, REFERENCE_ID: 16_cf65be_1642101307869, LINK_CREDIT: 500]

[* ] Query Added
[* ] Setup information Added

Similar issue https://github.com/Azure/azure-sdk-for-js/issues/15893, @conniey could you please link the SinkEmissionException issue we logged?

In the JS issue linked https://github.com/Azure/azure-sdk-for-js/issues/15893#issuecomment-866361212, describes why you see this error. It is a normal part of load balancing between different instances of EventProcessorClient.

When we are checking the output data we are seeing some missing events. Please confirm if this error can cause some event to be missed out from processing in the consumer instances.

This error shouldn't cause any events to be missed. When we start reading from your Event Hub, we query the Storage Blob for the last checkpoint and start from there. The only time checkpoints are persisted is when you call updateCheckpoint() in the processEvent\Events callback.

Hi Team, if we are saying that with this epoch error we will not loose any data then we are fine for now to close this issue.

@conniey We got the same error when using checkpointstore SDK to consume message from EventHub. And we found that the entity which occur the same issue is an empty entity.  We didn't send any messages to this entity during the past one month, could you please help clarify if the empty entity would cause this issue? Thanks so much!


Adapter.toggleGroup(group) doesnt call collapse or expand animation in groupViewholder

calling.toggleGroup(group) doesnt call the collapse or expand method in the groups Viewholder

Line 36 of ExpandActivity in the sample app is intentionally disabling the animation to the GenreViewHolder. Here is the full code from that line with the comment explaining why:
// RecyclerView has some built in animations to it, using the DefaultItemAnimator.
// Specifically when you call notifyItemChanged() it does a fade animation for the changing
// of the data in the ViewHolder. If you would like to disable this you can use the following:
RecyclerView.ItemAnimator animator = recyclerView.getItemAnimator();
if (animator instanceof DefaultItemAnimator) {
  ((DefaultItemAnimator) animator).setSupportsChangeAnimations(false);
}

If you delete the setSupportsChangeAnimations line the rotation animation will work.
When callingadapter.toggle() the adapter passes that call through to the ExpandCollapseController who in turns calls through to the ExpandCollapseListener which the ExpandableRecyclerViewAdapter implements. If you look at theonGroupCollapsed() implementation in ExpandableRecyclerViewAdapter you can see that it calls notifyItemChanged on the header - which forces a call to onBindViewHolder which handles calling either expand or collapse on the GroupViewHolder. It's admittedly a little confusing to follow but this solution ensures both click events and programatic toggles will lead to the expand() and collapse() functions to be called.

I have implemented it for expanding one section at a time and on expanding one section other one will collapse. if a section is expanded then on back button press i collapse the section. functionality wise this setup is working except rotation animation.
I have not implemented default animator. I have noticed same behaviour If i expand different section then previous section gets collapsed but rotation animation does not work until I scroll list.
I will be glad if you can put more light on how can I achieve this.


Fix documentation

The docs are currently failing to build due to a change in read the docs yaml spec: https://blog.readthedocs.com/use-build-os-config/. This PR is an attempt to fix that along with other formatting issues in the documentation.

@curieux Can you take a look at this? This PR fixes the broken doc build process, adds a new notebook for using persistence landscapes and fixes broken badges in the readme.


Standard color name returns non-standard color code.

Godot version
3.4.3 stable
System information
it does not matter
Issue description
Standard color name (brown) returns non-standard color code. The standard brown color code is #a52a2a Color.brown.to_html() gives #a62929.  I just tried the brown. I haven't checked other color constants. I'm sure they are also problematic.

extends Node

func _ready():
	var c := Color.brown.to_html(false) # -> a62929
	# Standard X11/HTML Brown -------------> a52a2a - rgb(165, 42, 42)
	print(c)
	#a62929

Godot Documentation
https://docs.godotengine.org/en/latest/classes/class_color.html#color

You can also create a color from standardized color names by using the string constructor or directly using the color constants defined here. The standardized color set is based on the X11 color names.

Additional resource;
https://www.w3.org/TR/css-color-3/#ref-X11COLORS
Steps to reproduce
gd.tumeo.space playground example
Minimal reproduction project
gd.tumeo.space playground example

I can reproduce this on master fb2802578 as well.

Just checked - I think this bug extends to many other colors, not only Brown. ALICE_BLUE and ANTIQUE_WHITE also seem a little bit off.


cmake: Get unit tests passing CI again

The following commit:
commit 407b49b35c111042ce474785bb88760322b26f32 (refs/bisect/bad)
Author: Torsten Rasmussen<EMAIL_ADDRESS>Date:   Wed Feb 12 15:00:46 2020 +0100
cmake: use find_package to locate Zephyr

breaks as we don't find the ZephyrUnittest package.  For now revert to
the old means until a proper fix can be made.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

Issue can be see via:
./scripts/sanitycheck -T tests/unit/math_extras/
Seems like ZephyrUnittests is not found.

Interresting, if I run this test, i'll get:
> git log --oneline -n1
29b554849a (HEAD -> zephyr_base_improvement, tejlmand-zephyr/zephyr_base_improvement) doc: Update documentation to describe use of find_package(Zephyr)
tora@ubuntu /p/g/n/zephyr> ./scripts/sanitycheck  -T tests/unit/math_extras/
Renaming output directory to /projects/github/ncs/zephyr/sanity-out.2
INFO    - JOBS: 4
INFO    - Selecting default platforms per test case
INFO    - Building initial testcase list...
INFO    - 1 test configurations selected, 0 configurations discarded due to filters.
INFO    - Adding tasks to the queue...
INFO    - Total complete:    1/   1  100%  skipped:    0, failed:    0
INFO    - 1 of 1 tests passed (100.00%), 0 failed, 0 skipped with 0 warnings in 1.94 seconds
INFO    - In total 1 test cases were executed on 1 out of total 239 platforms (0.42%)
tora@ubuntu /p/g/n/zephyr>

@tejlmand maybe some local files not checked in?

@tejlmand maybe some local files not checked in?

No, related to the fact I had the ZephyrUnittest exported.
But it should still work, so investigating why the backward compatibility is not working correctly for the ZephyrUnittest, when it is working for Zephyr package correctly.


UI for empty roster

Need a UI for rosters that are empty.

This is handled by showing the roster with it's Dropbox-relative path and the number of entries in parenthesis.


samAssembler Part 5: Scoring output alignments

Checklist before requesting a review

[x] I performed a self-review of my code
[x] If it this a core feature, I have added thorough tests
[x] The command go fmt was used on all files included

Testing

None
In response to Dan's comment, I cut up samAssembler into helper functions, and used structs to organize the many variables. This cut the size of the main program in half.
This PR adds the 'score' mode to samAssembler, which takes in a 5 way alignment, representing the haploid reference sequence, a 'true' diploid sequence, and the diploid sequence reconstructed from samAssembler. This mode then constructs confusion matrices based on the alignment, which will provide the basis for evaluating the precision and recall of the model.
A confusion matrix is a 2d table like the one below, in which each column represents an actual genotype from our generative model of molecular evolution, and each row represents the corresponding predicted genotype. In other words, this matrix allows us to identify false positive and false negative variant calls, but allows us to further identify which genotype was incorrectly predicted from each pileup.
Values along the diagonal represent correct base calls, and values off diagonal represent misclassification. Values below the diagonal represent situations where the model predicted a genotype that is more rare than the true mutation (a false positive).
Tv1 and Tv2 refer to the two possible transversion mutations (for reference base A, this would be C or T). Tv1 and Tv2 are distinguished alphabetically, so for reference base "A", "C" would be Tv1.

Values above the diagonal represent when the model predicts a genotype that is more common that the true genotype (false negatives, in which the model is too conservative). For our use case, false positives are particularly bad, so when the model is uncertain, we expect that the diagonal line should decay up and to the right.
This is indeed what we see on a series of simulated tests, in which we allow coverage to decay, or the mean read length to drop. Note on the far right how the predictions are all along the top row, homoRef. In other words, if the model does not have sufficient data to make a variant call, it will guess that the genotype is homozygous reference. In other words, this is a conservative, reference-biased variant caller.

We can compare our informative prior to a flat prior, in which all genotypes are equally weighted. This is the same as variant calling by likelihood ratio test. Notice how as errors accumulate, they do not show the same pattern of accumulating above the diagonal. In other words, the likelihood ratio test produces as many false positives as false negatives.

Finally, I ran this program on a forward evolution simulation on hg38 chr21 and chr22 to conduct a larger test. I've displayed proportions as well as the raw counts.

I'll mark this as ready for review, but keep in mind another part will be on the way to allow us to score INDELs. These confusion matrices score only diploid base genotypes.

I also push a few lint formating lines of code. I didn't change the logic in any way, but be sure to do a git pull before you start anything else


Proxy for MySQL ports?

Is it possible to setup nginx-proxy to do for multiple MySQL installations on a single server that it can do for multiple web server installations on a single server?
Eg: mydomain.com:3306 redirects to mydomain.com:3306, anotherdomain.com:3306 redirects to anotherdomain.com:3307

Have look at #318


(GH-2140) Document escalating privilege in powershell scripts on Windows

The WinRM transport does not - and will not - support something akin to
a run-as command. This is because the way WinRM works is by connecting
with the user that has permissions to perform the actions, not by
elevating or changing to another user after connection. Put another way,
WinRM operates with a session level token and has no functionality to
change to another user after authentication.
Instead of escalating the user once connected, users can pass
credentials to scripts or plans running on the targets and use them
directly in commands. There are a number of ways to do this, each ideal
for different use cases. This adds a section to the 'Escalating
Privilege' document that describes the different ways to pass
credentials securely to either a script or plan to be used for
escalating on Windows. This includes using sensitive task parameters,
and describes setting and retrieving variables on Target objects
securely using Hiera data, Inventory plugins, and Remote tasks.
Closes #2140
!no-release-notes

I wasn't entirely sure of what examples to provide and where, since most of these things aren't specific to windows and the workflows are so distinct. I don't think writing an example for each possible workflow would be a good use of time, and there are already smaller (platform-agnostic) examples of each element of the workflow on their respective pages. I tested each workflow on a Windows machine to verify it worked as expected, so happy to include examples anywhere they would fit!

Booooo updated

CLA signed by all contributors.

Ok, I rewrote this to be more clear and tried to make it more clear how to choose which solution was best. I removed the Hiera bit because it's a whole other can of worms and I think this solution is simpler and will work for most users, but using Hiera is really powerful and kind of cool.
I feel like I said the same thing like 3 times, could definitely use a good editor!

Hi Lucy, I was having trouble editing this in the GUI, so I pushed up a PR to your fork instead: https://github.com/lucywyman/bolt/pull/11

Merged and made some small edits


App Bar not taking default value if collapsedHeight or expandedHeight is empty

Describe the bug
App Bar not taking default value if collapsedHeight or expandedHeight is empty.
What is the expected behavior?
App Bar should tak default value if collapsedHeight or expandedHeight is empty.
What are the steps to reproduce?

Go to Dev docs - /components/app-bar/playground
remove number from  collapsedHeight prop i.e pass empty string
remove number from  expandedHeight prop i.e pass empty string

The app bar starts misbehaving (Check below recording)
Screenshots / Screen recording
https://user-images.githubusercontent.com/8997218/194061258-0762d624-a980-4e8e-9c56-e43cda9406ad.mov
Code snippet / Link to minimum reproduction example

[CODE HERE](https://github.com/brightlayer-ui/react-component-library/blob/dev/components/src/core/AppBar/AppBar.tsx#L85)
https://github.com/brightlayer-ui/react-component-library/blob/dev/components/src/core/AppBar/AppBar.tsx#L92

Your environment information

All
Suggested fix

Anything else to add?

This issue has been fixed


[SPARK-35719][SQL] Support type conversion between timestamp and timestamp without time zone type

What changes were proposed in this pull request?

Extend the Cast expression and support TimestampType in casting to TimestampWithoutTZType.
There was a mistake in casting TimestampWithoutTZType as TimestampType in https://github.com/apache/spark/pull/32864. The target value should be sourceValue - timeZoneOffset instead of being the same value.

Why are the changes needed?

To conform the ANSI SQL standard which requires to support such casting.
Does this PR introduce any user-facing change?

No, the new timestamp type is not released yet.
How was this patch tested?

Unit test

Test build #139696 has started for PR 32878 at commit 443ca9c.

I will fix timestamp without time zone => timestamp in this PR, too.

Test build #139696 has finished for PR 32878 at commit 443ca9c.

This patch fails from timeout after a configured wait of 500m.
This patch merges cleanly.
This patch adds no public classes.

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

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

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

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

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

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

Test build #139731 has started for PR 32878 at commit e0e4596.

@MaxGekk @cloud-fan this is ready for review again. The conversions is a bit tricky. PTAL.

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

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

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

Test build #139731 has finished for PR 32878 at commit e0e4596.

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

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


@conda-forge-admin,please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/bokeh-root-cmd-feedstock#4.


FLINK-2523: making the task cancellation interval configurable.

FLINK-2523: Makes the task cancellation interval configurable.

Thank you for this patch. The idea is good, but I think it needs a bit of improvement:

The value is pulled in a fairly complicated way though all the layers and abstractions. That is not really necessary, as soon as it is in the ExecutionConfig, the Task can access the values. No need to change the TaskDeploymentDescriptior, JobGraph, ExecutionGraph, etc. This probably needs changes in 2 files (otherwise, our abstractions are messed up).

taskCancellationDelay is not a very intuitive name, in my opinion. It suggests that cancellation is delayed, which is not the case. Tasks are actually canceled a single time, the blocked threads are only repeatedly interrupted.

We should not try to show off such parameters in the examples. The examples need to stay simple, or we confuse people with parameters they need usually not worry about.

The JobManagerITCase is completely commented out. I guess this is a testing artifact.

Hi @StephanEwen ,
Could you please elaborate more on how you think that the ExecutionConfig could be accessed differently by the Task?
Thanks!

Ah, true, the ExecutionConfig is currently grabed in a pretty non-transparent way from the JobConfig.
I think it would be a good refactoring to pass the ExecutionConfig directly via the TaskDeploymentDescriptor. Then the canceling interval can be passed simply through there.

This is going into a good direction. Two suggestions:

Since you already have the ExecutionConfig in the Task, I would put it into the Environment from there. Then we could also get rid of the strange way of reading the ExecutionConfig in the AbstractInvokable

The ExecutionConfig should be directly attached to the JobGraph, rather than written to the job config and read from there.


New Keyword is optional now.

The Flutter Codelab 1 says:

If you didn't change Center to "new Center", the type system will warn that the Center object should not be typed as a constant (const), because its child Text object is no longer a constant. Therefore, both Center and Text must create instances using new.

As mentioned here: https://codelabs.developers.google.com/codelabs/first-flutter-app-pt1/#3
As of Dartlang 2.0, new keyword is optional.
Please fix.

Duplicate of flutter/website#1936


fix: typo where there was a conditional else block that would never b…

…e used

Summary
I want to fix a typo in the documentation
Set of changes
I want to remove a conditional else block in the willAuthError. Based on the logic it will never be used.

Closing as stale, feel free to reopen if you think my comment is word addressing


Bug in reporting ID of A-OK AC-114 compatible blinds

To control and monitor blinds using RFXTRX, it is necessary to configure a transmitting device and a receiving device (referred to as BLINDS REMOTE).
In respect of A-OK AC-144 blinds, a work-around is necessary to configure a transmitting device. (It also not possible to configure a receiving device- this may be the same underlying issue but is the subject of a separate issue report).
In order to create a device for controlling A-OK AC-144 blinds (or compatible brands eg Motolux) it is necessary to pick up a house code and unit number from the Indigo log on a remote control button-press and enter that data in the device config for an A-OK_AC-144 device. The house code is reported as a decimal, however there appears to be an error in the conversion from hex. The unit code appears to be a fixed value regardless of the actually unit code.
It is possible to successfully configure a transmitting device using raw hex values derived from RFXmgr or the Indigo logs when debugging reports are switched on.
The plugin needs a tweak to report the correct house and Unicode values required for configuring the device.

Try:
Have spent some time sorting this out - and works far better than could have guessed.  Added new device type to cover all Blind types - Turn On/Turn off works and device state is updated by Remote if shares same details:
Use the new BlindsT1234 device for most now.
https://github.com/Ghawken/rfxcom-plugin
2.2.1

Add new device type BlindsT1234 for all type=25 Blinds
(replacing A-OK AC114, Brel, and others, these others left alone so can update as wanted)
Adds Support for Dooya Blinds and Remotes
Correctly shows Housecode, unitcode and subtype for ALL these Blinds Remotes
Correctly updates On/Off state of device if command sent via Indigo, OR if Remote button pressed
Add subtype to device Def (should allow other blinds to be controlled)
Add last Command received to device State, lastupdated and type
Preferred Usage:

Install, setup Physical Blind, setup working Remote for Blind.

Update RFXCOM Plugin (this Plugin) to latest version on github

Turn on support for Blinds T1234 in Config

Turn on log unknown sensors as error in Plugin Config

Press Remote Button on your Blind Remote

Log Should show an Unknown Device Error Msg: (like below)
RFXCOM Error unknown device detected (type = 25). Select a BlindsT1234 Device from the list of devices  RFXCOM Error HouseCode (hex)=293301 ,subtype (int)=6 ,unitCode (int)=145

Create a RFXCOM BlindsT1234 Device Adding these Details  (demo image = not the same details!)

####Functions:
Turn On and Turn Off - will open close Blinds (and update Device State)
Remote will likewise turn on/turn off and the State will be updated in Indigo
Also will show last command/last time updated in device states:

Closed in release 3.0.2.