Error when using multiple guard authenticators

Hi! I'm using Symfony 3.1 and multiple guard authenticators. Here is my firewalls:
firewalls:
        dev:
            pattern: ^/(_(profiler|wdt)|css|images|js)/
            security: false
        main:
            anonymous: ~
            provider: app_user
            form_login:
                login_path: /auth/login
                check_path: /auth/login
                csrf_token_generator: security.csrf.token_manager
            logout:
                path:   /auth/logout
                target: /auth/login
            guard:
                authenticators:
                    - app.email_user_authenticator
                    - app.token_user_authenticator
                    - app.facebook_user_authenticator
                entry_point: app.email_user_authenticator

And after installing AsseticBundle I'm getting this error on cache:clear (or composer update):

[LogicException]
Unable to register extension "security" as extensions have already been initialized.

Please help me to figure out what is wrong. Thanks.

It may be the same issue as this: https://github.com/knpuniversity/oauth2-client-bundle/issues/30 in some way
@sudosu can you try 2 things:

If you remove the (Guard) authenticators key, does the error go away?
If so, put the authenticators key back. Now, if you only inject the Container into your authenticators (instead of injecting individual services), does the error go away?

Cheers!

@weaverryan
Hi! Thanks for reply.
Yes, I figured out. A problem was only in facebook authentificator with injecting oAuth facebook provider as service into constructor. So, I only inject the Container and it works fine now.
Thank you for advice. Cheers!)


Upgrade Spring 6.1.4 -> 6.1.5

This PR contains the following updates:

Package
Type
Update
Change

Spring
import
patch
6.1.4 -> 6.1.5

Release Notes

spring-projects/spring-framework (Spring)
v6.1.5
Compare Source
:star: New Features

Extract reusable method for URI validations #32440
Allow UriTemplate to be built with an empty template #32432
Optimize Coroutine invocations #32390
Improve exception message when Scheduled tasks that are using Virtual Threads fail on shutdown #32381
Add query parameters to MockMvc Kotlin DSL #32371
Optimize content type parsing in ServletServerHttpResponse #32361
Refine handling of disconnected client error in ExceptionHandlerExceptionResolver #32359
Refine \*HttpMessageConverter#getContentLength return value null safety #32325
Optimize HttpHeaders.getAcceptLanguageAsLocales #32318
SimpleAsyncTaskScheduler: consistently apply default error handling #32298
Use Spliterator of underlying collection in MutablePropertyValues / MutablePropertySources #32281

:lady_beetle: Bug Fixes

SpEL fails to compile expression indexing into boolean array #32400
WebClient Observations are missing error #32389
WebClient request sent twice instantly when using @Cacheable #32370
AopUtils.getMostSpecificMethod does not return original method for proxy-derived method anymore #32365
Fix ServletResponseHttpHeaders#get null handling #32362
Compiled SpEL expression fails if #root or #this is a non-public type #32356
Suspend handler methods fail on nullable value class parameters #32353
Better protect against concurrent error handling for async requests #32340
Performance degradation due to Kotlin value class checks #32334
With value referenced from property file in XML configuration, setter selection can be random in case of overloaded setter methods (e.g. on SimpleClientHttpRequestFactory) #32329
constructor-impl of Kotlin value class is not called #32324
ContentCachingResponseWrapper no longer honors Content-Type and Content-Length #32317
Hibernate native image support broken by HHH-17643 #32311
Revisit DatabaseMetadata#getProcedures to only apply fallback if original search yield more than one result #32295
Remove Observation.Context from WebClient request #32198

:notebook_with_decorative_cover: Documentation

Refine "Redirecting to a resource" section code sample #32423
Javadoc of ReactorNettyClientRequestFactory#setExchangeTimeout methods have wrong default value #32419
Fix order of 2 sections in the Validation chapter of the reference manual #32408
Fix Javadoc #32403
Document that SpEL expressions using array construction cannot be compiled #32401
Improve documentation for @Sql execution phases regarding lifecycle #32343
Omit parameter names in REST Clients section of reference docs #32335
Different transaction name when using  TransactionAspectSupport.currentTransactionStatus().transactionName #32310
Provide guidelines for using Kotlin properties with proxies (@RequestScope and similar use cases) #32287
Correct documentation for userDestinationPrefix in MessageBrokerRegistry #32272

:hammer: Dependency Upgrades

Upgrade to Micrometer 1.12.4 #32431
Upgrade to Reactor 2023.0.4 #32420

:heart: Contributors
Thank you to all the contributors who worked on this release:
@Xednar, @ZeroCyan, @bsgrd, @ddaaac, @kilink, @qeeqez, and @quaff

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade Spring 6.1.4 -> 6.1.5

See:
- https://github.com/spring-projects/spring-framework/releases/tag/v6.1.5
- https://github.com/spring-projects/spring-framework/compare/v6.1.4...v6.1.5


Java version using fork/join

This is a Java implementation using the Fork/Join functionality of 1.7+ (this particular version requires 1.8 or more).
This runs the test ten times to see the effect of a cold vs. warm JVM.  On my machine it goes from 250ms to 50ms per run.  (By comparison the .NET Core version takes 500ms.)  If I run it 1000s of times it gets down to the 10ms to 15ms range.

Updated to include some minor improvements.
Running the test 25 times I get:
Result:<PHONE_NUMBER>00
Took: 175.90ms
Result:<PHONE_NUMBER>00
Took: 118.05ms
Result:<PHONE_NUMBER>00
Took: 137.63ms
Result:<PHONE_NUMBER>00
Took: 139.81ms
Result:<PHONE_NUMBER>00
Took: 67.08ms
Result:<PHONE_NUMBER>00
Took: 22.92ms
Result:<PHONE_NUMBER>00
Took: 91.30ms
Result:<PHONE_NUMBER>00
Took: 25.64ms
Result:<PHONE_NUMBER>00
Took: 115.90ms
Result:<PHONE_NUMBER>00
Took: 27.16ms
Result:<PHONE_NUMBER>00
Took: 19.58ms
Result:<PHONE_NUMBER>00
Took: 22.93ms
Result:<PHONE_NUMBER>00
Took: 22.73ms
Result:<PHONE_NUMBER>00
Took: 18.59ms
Result:<PHONE_NUMBER>00
Took: 25.52ms
Result:<PHONE_NUMBER>00
Took: 25.53ms
Result:<PHONE_NUMBER>00
Took: 23.56ms
Result:<PHONE_NUMBER>00
Took: 13.28ms
Result:<PHONE_NUMBER>00
Took: 9.40ms
Result:<PHONE_NUMBER>00
Took: 10.39ms
Result:<PHONE_NUMBER>00
Took: 10.90ms
Result:<PHONE_NUMBER>00
Took: 16.43ms
Result:<PHONE_NUMBER>00
Took: 9.99ms
Result:<PHONE_NUMBER>00
Took: 11.90ms
Result:<PHONE_NUMBER>00
Took: 12.35ms

With all cores at 100%
As a comparison, I modified the .NET Core version to run 25 times also, and got the following:
499999500000
Async sec: 0.526
499999500000
Async sec: 0.393
499999500000
Async sec: 0.395
499999500000
Async sec: 0.386
499999500000
Async sec: 0.380
499999500000
Async sec: 0.382
499999500000
Async sec: 0.389
499999500000
Async sec: 0.394
499999500000
Async sec: 0.384
499999500000
Async sec: 0.397
499999500000
Async sec: 0.386
499999500000
Async sec: 0.393
499999500000
Async sec: 0.393
499999500000
Async sec: 0.385
499999500000
Async sec: 0.385
499999500000
Async sec: 0.387
499999500000
Async sec: 0.444
499999500000
Async sec: 0.402
499999500000
Async sec: 0.401
499999500000
Async sec: 0.387
499999500000
Async sec: 0.388
499999500000
Async sec: 0.396
499999500000
Async sec: 0.409
499999500000
Async sec: 0.389
499999500000
Async sec: 0.386

With all eight cores average 75%.
On the first run out of 25, the Java version appears to be approximately 3x faster.  At the last iteration, the Java version appears to be nearly 40x faster.

Well fork-join impl blew away go, scala and java benchmarks on my machine as well.


Update composer.json

Doing these all as I come around to updating for next release. Thank you for the heads up!


socat missing for CoreOS

preflight checks exit with:
core@ip-10-0-1-121 ~ $ sudo kubeadm init
Running pre-flight checks
preflight check errors:
	socat not found in system path

Yes, I know, but you can just ignore it, since kubelet runs inside an rkt fly container.
However, I've only gotten flannel to work as the pod network, on that side, CoreOS isn't stable unfortunately,


Run Lapce in browser that connects to remote proxy

For sometime when we'll have development bandwidth to think about other platforms such as running Lapce in browser.
Supported scenario would be running a Lapce client (in browser) that will connect to remote Lapce proxy (running as standalone process on remote host).

When implementing this, keep in mind to support mobile support. VS-Code for example doesn't support mobile web. Lapce could be the first real cross platform ide.

From #1878:

Our fork of druid would have to be modified to render to WebGL. Then there's probably various dependencies which need to be messed with in order to properly compile to webassembly.

It has been a year since that comment, so it might be worth mentioning here implementation status for WebGPU at the moment:

✅ Chromium Windows
✅ Chromium Mac
✅ Chromium ChromeOS
Chrome Android: Currently behind flag. Probably unflagged early 2024.
Chrome Linux: Currently behind flag. Probably unflagged some time in 2024.
Safari: Available in latest technical preview as of a few days ago.
Firefox: In development, not sure on timeline, but I'm guessing they'll beat Safari.

The WebAssembly comment still stands, though feature support is getting better by the month. Safari is lagging behind here though, as usual.
For Chromium, at least, it looks like this may become viable some time in 2024.

It'd be awesome if Lapce had a https://vscode.dev type web editor. This project seems particularly suited to that sort of thing given the use of WebGPU and WASI.

This is not going to be worked on due to other priorities, and with our current development speed I don't expec this to happen in next 2-3 years.


Repo sync for protected CLA branch

The pull request is created from master637550532964025874 to master to fix git push error for protected CLA branch

@huypub : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Tutorial: adding command

This branch was created to practice getting familiarised with the code base.

Codecov Report

Merging #36 (be73fe5) into master (3cb0d59) will decrease coverage by 0.66%.
The diff coverage is 63.88%.

@@             Coverage Diff              @@
##             master      #36      +/-   ##
============================================
- Coverage     72.15%   71.49%   -0.67%
- Complexity      399      410      +11
============================================
  Files            70       73       +3
  Lines          1232     1298      +66
  Branches        125      132       +7
============================================
+ Hits            889      928      +39
- Misses          311      333      +22
- Partials         32       37       +5

Impacted Files
Coverage Δ

.../seedu/address/logic/parser/AddressBookParser.java
88.88% <0.00%> (-11.12%)
:arrow_down:

...eedu/address/logic/parser/RemarkCommandParser.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)

...va/seedu/address/logic/commands/RemarkCommand.java
58.33% <58.33%> (ø)

...c/main/java/seedu/address/model/person/Remark.java
75.00% <75.00%> (ø)

...java/seedu/address/logic/commands/EditCommand.java
95.83% <83.33%> (-1.19%)
:arrow_down:

...c/main/java/seedu/address/model/person/Person.java
95.74% <85.71%> (-1.88%)
:arrow_down:

.../java/seedu/address/logic/commands/AddCommand.java
100.00% <100.00%> (ø)

...a/seedu/address/logic/parser/AddCommandParser.java
100.00% <100.00%> (ø)

...ain/java/seedu/address/logic/parser/CliSyntax.java
85.71% <100.00%> (+2.38%)
:arrow_up:

... and 3 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[storage-file-share] update changelog

#6302
Changelog and breakingchanges.md updated with changes that were made to file-share.

Hi @willmtemple, please help add label "Storage" and "Client", so our CRI system will track any PRs we need to follow in time. Thanks!


Filter for last X days won't work

Actual State
Filtering for the last X days doesn't work in both Monte Carlo Simulations. It picks X days but not the last days.
Line: "throughput[['Throughput']][: LAST_DAYS].reset_index(drop=True)"
Target State
Pick the last X days from now. Not by random.

Fixed with commit 384fdc0817617440fd0bdcec051c37df8e511f41


Cleanup tests by removing spurious errors and unnecessary noise

Remove diamond inheritance tests since these test a bug in Django,
not in django_polymorphic.
Fix spurious print message in test_primary_key_custom_field_problem(),
where uuid instances are for some reason being compared to uuid.UUID
classes. Replaced with a test that checks that the primary key values
are valid UUIDs.
Remove Django > 1.6 conditionals wrapping the multiple inheritance
tests and model definitions, in favor of using the definitions for
Django 1.7. Django 1.7 requires a custom primary key when using concrete
model inheritance because failing to do so is potentially ambiguous. Adding
the custom primary key does not break tests in older Django versions.

Thanks for looking into this, this definitely improves future maintenance!


[ENH] support for xarray DataArray & mtypes

Reference Issues/PRs

resolves #2655
What does this implement/fix? Explain your changes.

Does your contribution introduce a new dependency? If yes, which one?

What should a reviewer concentrate their feedback on?

Any other comments?

PR checklist

For all contributions

[x] I've added myself to the list of contributors.
[ ] Optionally, I've updated sktime's CODEOWNERS to receive notifications about future changes to these files.
[x] I've added unit tests and made sure they pass locally.
[x] The PR title starts with either [ENH], [MNT], [DOC], or [BUG] indicating whether the PR topic is related to enhancement, maintenance, documentation, or bug.

For new estimators

[ ] I've added the estimator to the online documentation.
[ ] I've updated the existing example notebooks or provided a new one to showcase how my estimator works.

#TODOs

[ ] make xarray to a softdependency (conditional import)
[ ] resolve TODOs in the code
[ ] Discuss if

DataSets should also be supported
Further scitypes should support xarray

for conditional execution (based on presence of package in the env), you can use:
if _check_soft_dependencies("xarray", severity="none"):
    import xarray
    etc

let me know if you think you are stuck - I can look at this is more detail, we can do pair programming, or I can give it a try.

let me know if you think you are stuck - I can look at this is more detail, we can do pair programming, or I can give it a try.

Thank you, I let you know if I am stuck

I have two questions:

I added the extend_conversion function from tables to series. (In panel, there exists something similar). Should I move this method to another file in the datatypes file, or should an extra PR do this?
Should I add xarray as a soft dependency somewhere that the automatic tests can install xarray?

I added the extend_conversion function from tables to series. (In panel, there exists something similar). Should I move this method to another file in the datatypes file, or should an extra PR do this?

Depends what you need it for. If needed in this PR, we can leave it in its location and refactor.
If you want it as a more general utility and not needed for the xarray topic, it is better putting in a separate PR.

Should I add xarray as a soft dependency somewhere that the automatic tests can install xarray?

Yes, pyproject.toml, all_extras dependency set.

I added the extend_conversion function from tables to series. (In panel, there exists something similar). Should I move this method to another file in the datatypes file, or should an extra PR do this?

Depends what you need it for. If needed in this PR, we can leave it in its location and refactor. If you want it as a more general utility and not needed for the xarray topic, it is better putting in a separate PR.

Ok, it is not needed for the xarray topic.

A solution for this will probably also solve also the problem with the tests in python 3.7 (it seems that in python 3.7 the index is a BaseIndex after the conversion, while for 3.8 and 3.9 it is a PeriodIndex)

Ouch... how odd. I suppose this is one of those non-obvious problems that occur...

👏 nice! Looks like it passes now!

👏 nice! Looks like it passes now!

Yeah 🥳
I think you can now take a more detailed look at this PR. I added already some comments, and I would appreciate your feedback there.
Should we add the support for other mtypes and the dataset in this PR or should I create additional PRs for the other?

Should we add the support for other mtypes and the dataset in this PR or should I create additional PRs for the other?

I would go with the "minimal modular change" principle and separate PR, also finalizing this one first.
Which other mtypes do you have in mind?

Update

I changed the check in deep_equals.
I manually create a dataframe instead of using to_pandas. This ensures that xr.DataArrays are always transformed to DataFrames and not to pd.Series if the xr.DataArray contains only one dimension. Note, to_dataframe does not work well, since it does some multi-indexing  stuff in case of two dimension which needs to be unstacked.

Works!
Do you want me to merge this, or wait for panel?

I would merge it and add the panel with a second PR if I need this. Currently, I can test and go on with the sktime-pyWATTS integration without implementing Panels. Perhaps Issue #2655 should stay open in that case.

@fkiraly Do we have to adapt utils.validation.series? In that file, something like VALID_DATA_TYPES is defined. Do I have to add xr.DataArray?

@fkiraly Do we have to adapt utils.validation.series? In that file, something like VALID_DATA_TYPES is defined. Do I have to add xr.DataArray?

I think that's used only in legacy code, so I would recommend to not touch it (who knows what will break).
The "critical touch point" is check_series, which is still linked to a few estimators that have not been fully refactored, such as ColumnwiseTransformer - but mostly decoupled from the vast majority of the code base.


Confusing "Local instalation" tutorial

I was going thru this http://pachyderm.readthedocs.io/en/latest/getting_started/local_installation.html and on the step pachctl port-forward I executed it. That command showed me Pachd port forwarded Dash websocket port forwarded Dash UI port forwarded, navigate to localhost:30080 Then I proceeded to http://localhost:30080/ and the dashboard crashed.
I've spent 60 minutes trying to figure out what is wrong. As it turns out, you don't get a dashboard during that local installation. You should add a warning about that into local installation or into the port forwarding message.

Thanks for reporting this, and I agree it is confusing. We are currently working on fixing this as tracked by https://github.com/pachyderm/pachyderm/issues/2444 and https://github.com/pachyderm/pachyderm/issues/1728. This should be fixed soon, but closing this as a duplicate.


Add building icon

The  building icon. (cf #78)

OK,  commited every changes 👍

@delnyn Awesome work!

Nice work!

Oops! Something is wrong, the icon is directly in the root of the repo 😓 sorry, I'm going to fix that...


Solix Smart Meter

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
For the HA Energy Board the House Consumption is needed. In my Case the Smart Meter measures it. But i can´t find it as a Device or Sensor in HA.
Describe the solution you'd like
A Integration for the Smart Meter as a Sensor
Describe alternatives you've considered
Installing a Shelly Meter parallel...but it´s alsready tight in my Fusebox.
Additional context
Let me know if you have any Ideas ;)

Did you check the 2.0.0 announcement #93 ?

Implemented with 2.0.1


'Download SSL Certificate' link in Azure PostgreSQL Flexible Server - Network blade directs to confusing article

In the Network blade of the Azure PostgreSQL Flexible Server resource, the 'Download SSL Certificate' button used to download a certificate file that could be used for SSL communication with the Azure Postgres Flexible Server. Recently, this button no longer downloads this file but instead opens a tab to https://learn.microsoft.com/en-us/azure/postgresql/flexible-server/concepts-networking-ssl-tls#downloading-root-ca-certificates-and-updating-application-clients-in-certificate-pinning-scenarios. This article makes a confusing attempt at explaining certificate pinning and when to download certificates in the context only of certificate pinning. However, it makes no mention of where to simply get the certificate for normal verify-ca scenarios.
This is extremely confusing and unintuitive now. I simply need the certificate required to setup an SSL connection with Postgres which I cannot find any longer. Mentioning of certificate pinning has sent me on a long goose chase and multiple meetings with Microsoft only to discover we are not in fact using cert pinning and none of this applies to us.

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

ID: 527dd8d1-89d8-098f-dcdf-031f222195f6
Version Independent ID: 99f7ef3a-a767-98b5-0a79-2ae0d85390c2
Content: Networking overview using SSL and TLS - Azure Database for PostgreSQL - Flexible Server
Content Source: articles/postgresql/flexible-server/concepts-networking-ssl-tls.md
Service: postgresql
Sub-service: flexible-server
GitHub Login: @GennadNY
Microsoft Alias: gennadyk

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

@rennerg
It looks like your issue is resolved since you closed this thread.
If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


trying to fix #176

Current coverage is 95.04% (diff: 96.42%)

Merging #179 into master will increase coverage by 0.13%

@@             master       #179   diff @@
==========================================
  Files            15         16     +1
  Lines           590        646    +56
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            560        614    +54
- Misses           30         32     +2
  Partials          0          0

Powered by Codecov. Last update 805f106...06a82a7

Sorry, I'm pretty delayed on feedback, due to traveling. Would you mind providing some more context for this PR? It appears you intend to detect if two states are equal based on a UUID that is maintained by reducers. Is this a pattern that you've used in your projects so far?
#176 Also mentioned that Redux has this feature out of the box. Do you happen to know how it's implemented there? I assume the JSON representation of the state is diffed? But haven't had a chance to check it out yet.

Hey, sorry for the late reply. My Idea is when we have 2+ substates and 2+ subscriptions in one screen. After user has changed some substate all subscribers get the notification. My solution is to use new subscriber protocol which keep the current substate uuid, if the new state uuid is different, only then call the newState().
All of the current code will work. To get the new functionality have to confirm to this 2 new protocols (EquatableState, EquatableStateStoreSubscriber) and change the uuid in the Reducer when you want to trigger the notification.

@sisomollov Would you mind taking a look at https://github.com/ReSwift/ReSwift/pull/190 ? It resolves the same issue as this PR, but in a different way. I'm interested to hear why you chose the UUID route vs general Equatable conformance.

@mjarvis For me the Equatable conformance is a bit annoying, especially if the substate has a lot of properties. That's why I'm using uuid as identifier.
I think your solution is better than mine, and I still can use my "uuid thing" with one general Equatable state protocol.

@sisomollov Thanks a lot for this contribution! I believe this issue should be solved with #203. I'll close this PR for now, but feel free to reopen it if you believe it's still relevant!


Polishing JakartaPublishingPlugin

fix Note: Recompile with -Xlint:unchecked for details.
return value of method addVariant is not used.
refine lambda.
remove unused import.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.

@gavinking Could you review?

Thanks!


WIP traffic stats

Will close #10
Current caveats:

Only works with one tunnel active
Only works with WgQuickBackend
Is not aware of current tunnel and thus shows in each tunnel's detail view

Test this at WireGuard-KT-76-2217586929ef5053d2e868ac6ee1d39e0fd1b94e.apk.
This is an automatically posted message from Travis-CI

Thought about all the current drawbacks and I feel a new fragment/activity with stats would look much better and provide easier ways to get rid of the issues we have currently.

Test this at WireGuard-KT-76-70208387efd77a3f309144370d4885bda444ef3f.apk.
This is an automatically posted message from Travis-CI

This should ideally be using the Statistics sub-class in Tunnel

wg show <interface/tunnel name> should resolve the main blockers here, still not sure how to implement statistics for GoBackend but all of this will have to end up waiting for when msf/upstream-sync is finished at any rate so guess I'll worry about it when the time comes.


updating the client

I have a problem in test of client. I can't see changes in the android and hbbtv client immediately by refreshing the pages after changing in the code. I am testing them by HbbTV viewer on Mozilla and (android phone and also chrome) for android.
I have to close browser, clear history to see the changes.  one of the problem is that after selecting the service list, client uses it and it seems it doesn't update its graphic . I saw in the code that hbbtv client update service list every minute .but not its html.??
now, I have to close browser. clear history and start both clients from the service list selection page to be able to see changes in the client.

Normally the application logic (HTML, JS, CSS) does not reload one running, however the application can read supporting data (such as the XML ML

No development action needed for this
Closing


Help

What is the prefix for the commands?

Bot using slash commands, make sure first to run slash.js file to register slash commands in your server


GraphQL 14 Support

Since GraphQL 0.14 is around the corner. Creating an issue to track its support.
I can take up the implementation part as I am maintaining the vscode-graphql extension and we have some issues regarding this change already.
Do any of the maintainers have any comments on this before I start to work on it? Thanks.

should be able to get this all released shortly! then we have some follow up work to deprecate support for <graphql 0.12

this should be resolved in the latest 2.1.0 releases! sorry it took so long.


add example eye tracking data

Description of the problem
Using synthetic data for most of the tests is fine, but when we show some tutorials we want to make sure to use real data.
Description of a solution
There are two different ways to do this:

We can add some basic data to the repository.
We can upload the data somewhere else and download it in the tutorials.

Minimum acceptance criteria

[ ] real world data usage with plots in at least one notebook

I thought about this a bit more and the way torchvision is doing is with its datasets module is the best way I think.
https://pytorch.org/vision/0.8/datasets.html
We would create a pymovements.datasets module where publicly available eye tracking datasets could me downloaded and parsed automatically.
This way we can foster the visibility of the aeye group in two ways:

by including a lot of datasets in the module, eye tracking researchers are drawn to the pymovements package as they get all this integration for free
by including all the datasets from our group, pymovement users can instantly integrate these into their own research

we added a small toy dataset in #221


Fix missing Dir#each to be an Enumerable

Dir is missing #each method to comply the Enumerable(String).
Part of #4811 renamed each to each_entry. This PR rollback that.
Probably most of the time Dir#each_child will be used. That method filters . and .. from the list of items.
Maybe the each should filter . and .. by default. But the proposed change keeps the behavior of 0.23.

issue: #5445
I'm not sure if a complete rollback is a good idea. Why not add it in as alias for the time being and discuss how Dir should look in the future? @RX14 earlier suggested to restructure the entire class (#5447 (line comment)).
Hopping back and forth with API changes is not great, especially if it's perhaps going to be entirely different anyway.

Thanks @straight-shoota to point relevant discussion/issue.
Note that this is not an entire rollback, but the minimum to have a consistent and working Dir.
Refactoring the Dir class, if done, shouldn't happen in 0.24.*

Let's keep the new name, but have each as an alias. That's the minimum fix, so I think that's what we should do for 0.24.2.

@RX14 I don't see why we would need an alias here. each_entry was introduced in 0.24 so probably wasn't broadly used yet.

@bcardiff it's a nitpick either way. If nobody else has any input might as well roll a dice...


Managed identity for WebApp has stopped working

We have been using Microsoft.Azure.KeyVault for some time now with success. Using the managed identity in our WebApps and an AD group to grant access to key vault. The app SP is added to the group and the group has access to key vault, I have tried giving the app direct access rights to KV also, but no help.
I have updated a couple of apps to use the Azure.Identity package and the .Net Framework app has continued to operate, but the .Net Core 3.1 app now does not seem to pick up the credentials.
If I add an explicit AZURE_CLIENT_ID, AZURE_CLIENT_SECRET, and AZURE_TENENT_ID that corresponds to an RBAC generated service principle, it all works fine. I don't want to have to do this though and would much prefer to use the managed identity (no config floating around).
These are the packages I am now referencing:
<PackageReference Include="Azure.Identity" Version="1.1.1" />
<PackageReference Include="Azure.Security.KeyVault.Secrets" Version="4.0.3" />

This is the construction code:
new SecretClient("name-of-vault", new DefaultAzureCredential());

This is the stack trace:
---> Azure.Identity.AuthenticationFailedException: DefaultAzureCredential authentication failed.
---> Azure.Identity.AuthenticationFailedException: Invalid response, the authentication response was not in the expected format.
at Azure.Identity.ManagedIdentityClient.Deserialize(JsonElement json)
at Azure.Identity.ManagedIdentityClient.DeserializeAsync(Stream content, CancellationToken cancellationToken)
at Azure.Identity.ManagedIdentityClient.AuthenticateAsync(String[] scopes, CancellationToken cancellationToken)
at Azure.Identity.ManagedIdentityCredential.GetTokenImplAsync(TokenRequestContext requestContext, CancellationToken cancellationToken)
--- End of inner exception stack trace ---
at Azure.Identity.DefaultAzureCredential.GetTokenAsync(Boolean isAsync, TokenRequestContext requestContext, CancellationToken cancellationToken)
at Azure.Identity.DefaultAzureCredential.GetTokenAsync(TokenRequestContext requestContext, CancellationToken cancellationToken)
at Azure.Security.KeyVault.ChallengeBasedAuthenticationPolicy.AuthenticateRequestAsync(HttpMessage message, Boolean async, AuthenticationChallenge challenge)
at Azure.Security.KeyVault.ChallengeBasedAuthenticationPolicy.ProcessCoreAsync(HttpMessage message, ReadOnlyMemory1 pipeline, Boolean async)    at Azure.Core.Pipeline.RetryPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline, Boolean async)
at Azure.Core.Pipeline.RetryPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline, Boolean async)    at Azure.Core.Pipeline.HttpPipelineSynchronousPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline)
at Azure.Core.Pipeline.HttpPipelineSynchronousPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline)    at Azure.Core.Pipeline.HttpPipeline.SendRequestAsync(Request request, CancellationToken cancellationToken)    at Azure.Security.KeyVault.KeyVaultPipeline.SendRequestAsync(Request request, CancellationToken cancellationToken)    at Azure.Security.KeyVault.KeyVaultPipeline.SendRequestAsync[TResult](RequestMethod method, Func1 resultFactory, CancellationToken cancellationToken, String[] path)
at Azure.Security.KeyVault.Secrets.SecretClient.GetSecretAsync(String name, String version, CancellationToken cancellationToken)
at Codat.Infrastructure.SecretsProvider.SecretsProvider.<>c__DisplayClass18_0.<g__TryGetKeyVaultValue|0>d.MoveNext()

Moved to SDK for .Net


[feat] I will change log facility from golang default fmt into glog

as the title says.

After some testing, found that there is confliction between glog and go-flags .
So not support this feature anymore, close #3

closes #3


Remove deduplication logic in evaluation history

I have run into a number of issues involving pq and sqlc's ability to deal with postgres timestamp arrays. At this point, rather than invest further effort, I have decided to remove the deduplication feature for now. Since this will only be useful at the point where we use reconciliation more extensively, I would prefer to reintroduce this functionality once we have more data to back up its utility.
Summary
Provide a brief overview of the changes and the issue being addressed.
Explain the rationale and any background necessary for understanding the changes.
List dependencies required by this change, if any.
Fixes #(related issue)
Change Type
Mark the type of change your PR introduces:

[x] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[x] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[x] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 52.73% (-0.04%) from 52.769%
when pulling 72851de204ba547b81c3a8c444b19bf57cc52662 on remove-dedupe-logic
into 3be7c67c2437de8da7512c214521463c4be2f709 on main.


run runner on deploy

I have been using whenever gem for cron job over aws elasticbeanstalk.
and i have been using
every 1.day do
  runner "SnapshotReport.add_report"
end

is there any way to fix this for ON DEPLOY as per what i need is to run this runner right i deploy my app. Right now its saying 1.day on which time of day it will be run if no time option is given?

I think its impossible? SO closing. right?


Feature: delete new drafts when open and close immediately

Problem
when you do cmd+N, we are creating a new draft and this creates a giant list of untitled drafts.
Solution
either:

we create the draft AFTER the first edit
we delete the new created draft IF the user closes the window before doing any changes to it.

wdyt @burdiyan @juligasa @GaboHBeaumont @ericvicenti @iskaktoltay

Crazy idea: cmd+n could open a new window without creating documents. I expect this behavior intuitively all the time, but I might be broken. It depends on whether we position ourselves as a "browser" or as a "word processor/note taker" I guess. Safari creates a new window, Apple Notes (and Pages) create new documents.
As a user, I keep getting surprised every time cmd+n creates a new document in Mintter, because it's pretty much never what I want :)

fixed!

Crazy idea: cmd+n could open a new window without creating documents. I expect this behavior intuitively all the time, but I might be broken. It depends on whether we position ourselves as a "browser" or as a "word processor/note taker" I guess. Safari creates a new window, Apple Notes (and Pages) create new documents.
As a user, I keep getting surprised every time cmd+n creates a new document in Mintter, because it's pretty much never what I want :)

This is what I wanted to fix with deleting the draft if there was no edits. This is an issue because we are creating a lot of drafts. I wil ltry to create an issue/project for it


fix: support SW usage

We stumbled across this issue in our PR switching the extension to manifest version 3 (MV3).
In MV3 the code is running within a service worker so no window exitsts. Not exactly sure where the document error is coming from but with thsi change it works.
What do you think? Would this work for all use cases?

Yeah globalThis seems like the way to go. Are you going to update to use that instead?

Done

Funny version :) Works! Thanks!


SmartThings information issues

Hi Ollo,
I follow all steps (several times) but can't connect with my Samsung Q55 when I try to use the SmartThing token.
I can connect properly if keep blank the token field. But I need to use SmartThings Keys...
When I try to connect with the TV it ask meif I want grant access to HA, fine. But later HA shows the message "Wrong SmartThings token or TV not registered in SmartThings." and the integration is never done.
But the file token-<IP_ADDRESS>.txt are created and have an 8 digits number. I delete this file each time that I try to connect with a new token.
I have my iPhone connected to the TV, so ST works. No?
The TV is registered in Samsung ST. I know it because I can see the events list of my TV in Samsung web (Turned ON at this time, switched to HDMI2...).
I tried deleting and generating new tokens, don't works.
No log to show, home-assistant.log file don't have anything about this connection.
Any ideas?

idem, but the TV does not ask me for confirmation

Hi Aiace,
Try to remove HA from the device list in Config\General\External devices in your TV
Maybe it will ask you for confirmation.

Sorry but is not clear how you generate ST token and seems that there is a little bit confunsion.
Token file generated in component dir have nothink to do with ST token. ST token must be generated connectig to ST web site and following the instruction provided in the main page. Did you follow this guide?

Hi Aiace,
Try to remove HA from the device list in Config\General\External devices in your TV
Maybe it will ask you for confirmation.

hi, are you referring to the television menu? there is only my phone between the devices

Sorry but is not clear how you generate ST token and seems that there is a little bit confunsion.
Token file generated in component dir have nothink to do with ST token. ST token must be generated connectig to ST web site and following the instruction provided in the main page. Did you follow this guide?

Hi, I follow step by step your guide in https://github.com/ollo69/ha-samsungtv-smart/blob/master/Smartthings.md and I have my token.

Because it didn't works for me, I tried to reconnect doing some changes like delete token file... Please, forget it.
When I reinstall your component, reboot HA, clear the device list in my TV, and follow all steps, I cant connect (but permission dialog is showed in the TV) and the message "Wrong SmartThings token or TV not registered in SmartThings" is showed.
I tried to make a manual connection editing the configuration.yaml but same result (dialog box in TV, no integration created) without any  log information.

Hi Aiace,
Try to remove HA from the device list in Config\General\External devices in your TV
Maybe it will ask you for confirmation.

hi, are you referring to the television menu? there is only my phone between the devices

Yes, on TV. This step works for me, even though the connection is not finished creating.

I am also having issues with this.

This is what the device looks like under SmartThings. I had to remove and then add it again to have it connected to my account. Now there is less information.

Problem is type. I'm filtering for "Samsung OCF TV". Or you change the type (with edit button) or I remove the filter but in this case I could associate a wrong device during setup,
Was this type associated automatically or you changed something manually?

Yes, I did use the app to associate it. How do you do it on the site?

Same as Sebastian. Associated by phone app and the Type result is "Placeholder".
I try to edit Type field to "Samsung OCF TV" and "Device Network Id" are required in this edition process.
Searching about ST "Device Network Id" I found:
Device Network Id – this must be unique, but otherwise doesn’t matter. I just used the name of my device, in all caps and one word.
So I  try this:

ha-samsungtv-smart can't connect.
The message showed is: "Unable to connect to this Samsung TV device"

I think that is not correctly registered. Remove it and register again using the phone app. I have a lot of information associated to my device that here is misssing.

This is my Device info after change to "Samsung OCF TV" Type

This are correct information! In this way should work also with the control

Do you think that my "Device Network Id" is right?
Or maybe i cant connect because "Device Network Id – this must be unique, but otherwise doesn’t matter. I just used the name of my device, in all caps and one word." is not correct?
When you setup your own TV, did you need to enter "Device Network Id"?

No, I don't use this, but I don't now how SmartThings use this. I think that registration should be handled by the SmartThings apps and not manually by you. I suggest to remove this device and try to register agin, than use the portal only to check that is correctly registerd. I have a UUID in the field network id for my TV.

OK, I tried this:

Remove device list in TV
Remove TV in my ST account
Connect my TV to ST using my phone app
The new TV in my ST account have "placeholder" in Type field and no "Device Network Id" is showed.

Edit device manually.
· Type -> "Samsung OCF TV"
· Device Network Id -> "samsung_qled_55"

Now Device Info are more detailed in the ST page
Delete and token and generate new one in Samsung account.
In HA, create new integration SamsungTV Smart
Grant access in TV side

Same result:  Wrong SmartThings token or TV not registered in SmartThings

I do not understand why your TV is created with type "placeholder", I think that something is wrong in this step. May be is a problem on the app, or may be you are not using the correct procedure. TV should be automatically detected by the phone APP, is this occuring? Is your phone connected to the same wi-fi when you register the TV?

Yes, I just find a TV and the app find it. TV and phone are in the same WiFi.

Maybe this is a clue:
On TV, when select "[Home Assistant] QLED 55" in remote device list -> show details, I see available services for this device and only shows one: play media from mobile device.
Perhaps must have a "remote control" or something similar, no?

i have the same problem with my q70, if i register in home assistant without the api it's ok, but if i use the api i have the same problem. my tv is type placeholder.

Maybe this is a clue:
On TV, when select "[Home Assistant] QLED 55" in remote device list -> show details, I see available services for this device and only shows one: play media from mobile device.
Perhaps must have a "remote control" or something similar, no?

What you see in the TV is only related to local connection and have nothing to do with SmartThings. I'm quite sure that the problem is how TV is registered in SmartThings. My TV show all information and type is correct without the need of manual change on the web site.

For me this integration was working.
Since yesterday I don’t get the states from smartthings. I tried to re add this integration and get the same error.

I think they are doing something on their server. I have the list of input with duplicated values, never see this before:

I just fixed the code in the component because with this broken list was creating wrong source list!!
Try to configuring manually on SmartThings the correct TV Type, but in "Device Network ID" put a generated uuid (you can generate here, maybe it work.
From my side SmartThings still working....

I have duplicated inputs too.
I generated uuid for Device Network Id and edited the configuration.yaml file for manual config. After reboot I have this:
Logger: custom_components.samsungtv_smart.config_flow
Source: custom_components/samsungtv_smart/config_flow.py:149
First occurred: 6:37:13 PM (1 occurrences)
Last logged: 6:37:13 PM
Error during setup of host <IP_ADDRESS> using configuration.yaml info. Reason: not_successful

I do not understand what you do in configuration.yaml. Anyway it is better to do first configuration using web interface, then you can use configuration.yaml for additional value (like app and source list)

This is my configuration.yaml
samsungtv_smart:

host: <IP_ADDRESS>
name: Samsung QLED
api_key:

And this is my config in ST:

And this is my config in ST:

but the device network id in my situation is not displayed...

Comment this and configure using web interface. After TV is configured you don't need to specify API_KEY or Name because are saved in the registry. You must only set the key (host) and the additional value (source_list, app_list, etc)
samsungtv_smart:
  - host: <IP_ADDRESS>
    source_list: <your source list, if you want>
    app_list: <your app list, if you want>
    ...

i remove my tv from my profile smartthings and i reconfigure from app android, but i dont see the network id....

Network ID and device type can be configured from web site: https://account.smartthings.com/login

Comment this and configure using web interface. After TV is configured you don't need to specify API_KEY or Name because are saved in the registry. You must only set the key (host) and the additional value (source_list, app_list, etc)
samsungtv_smart:
  - host: <IP_ADDRESS>
    source_list: <your source list, if you want>
    app_list: <your app list, if you want>
    ...

Sorry, I'm lost. Do you say to first include api_key, and later erase it?

Network ID and device type can be configured from web site: https://account.smartthings.com/login

Really? Great! Where can I get it? I don't see where

Network ID and device type can be configured from web site: https://account.smartthings.com/login

yes, i have modify the type in this site...but the network id is not present....
sorry, you are italian?

Sorry, I'm lost. Do you say to first include api_key, and later erase it?

I'm saying to use Home Assistant web interface to configure the component and not configuration.yaml!! Settings -> Integration and + button.
Configuration.yaml will be used after to add additional info.

Network ID and device type can be configured from web site: https://account.smartthings.com/login

yes, i have modify the type in this site...but the network id is not present....

sorry, you are italian?

Yes, I'm Italian, but why you say that "network id" is not present, is in your piture!!

no, i quote the post of theiamania.
this is my configuration.
i test to insert the token in the network id, i see the information of tv, but the configuration is not ok

p.s. sono italiano anch'io.

Sorry, I'm lost. Do you say to first include api_key, and later erase it?

I'm saying to use Home Assistant web interface to configure the component and not configuration.yaml!! Settings -> Integration and + button.
Configuration.yaml will be used after to add additional info.

Yes, like in the readme. Always I use this method, but I tried manual way too.
Thanks for add this featured!

Sorry, I prefer not use italian because this is a public post.
In the field "Device Network ID" you must put a generated UUID form here: https://www.uuidgenerator.net/ and not your SmartThings token. The SmartThings token you must use after, during component configuration.
I don't know if this work, but is the only suggestion that I can give.
I repeat that normally this values should be automatically generated during TV registration, so I think that something is not working on Samsung.

Sorry, I prefer not use italian because this is a public post.
In the field "Device Network ID" you must put a generated UUID form here: https://www.uuidgenerator.net/ and not your SmartThings token. The SmartThings token you must use after, during component configuration.
I don't know if this work, but is the only suggestion that I can give.
I repeat that normally this values should be automatically generated during TV registration, so I think that something is not working on Samsung side.

ok, i have generate the uuid from this site, i insert in the site samsung, and select type: Samsung OCF TV
now i have many information from the tv,  mac address, the channel, ecc....
i return in home assistant, insert the ip, name and token, but i have the same error.

when i generate the token, you select the all options?

thanks for the support!

No idea.
Just another tip: are sure that you set all correct permission to the generated token? Eventualy try to generate a new one a follow very well all the instruction
Later I will do some test reconfiguring my TV, but for me the problem is on samsung side. Network Id should be automatically generated and device type should assigned correctly during phone configuration.

Token name you can put any...

No idea.
Just another tip: are sure that you set all correct permission to the generated token? Eventualy try to generate a new one and follow very well all the instruction!!
Later I will do some test reconfiguring my TV, but for me the problem is on samsung side. Network Id should be automatically generated and device type should assigned correctly during phone configuration.

yes, i create a token here

and select all

Finally I can confirm that there are issues on SmartThings side.
The reason for the error during configuration is related to a call to SmartThings API that is not returning anymore the list of devices. The point is that this is a documented API, so this issue should not occur.
Also there are many states that are not updated anymore on the SmartThings server device.
I hope that this issue will be solved shortly, for now I put a warning at the beginning of the readme files.
Please on next days try again to configure your TV with ST and report here if it works again.

Thanks ollo, for your work, time and patience.
I will try to connect again in a few days and I will inform you when done.

I just create a new release with a work-around for this issue.
Please give me your feed-back.

I got the error Wrong SmartThings token with this new version. I also tried a new generated token.

I generated the new device id with the tool.
After update the device on the smartthings site with this id I have seen, what under "current states" the di and pi are different. But this id's are the right ones.

Not really clear what you do, but fine to hear that is working...

If you add a new device in the smartthings app, you got the entry "placeholder" as device. If you edit this device on the smartthings website and add the right tv model and the self generated uuid from the generating website, this network id is different from this too values.

Yes, but network id have nothing to do with configuration. I need Device ID that is the property "di".
Just another question: is your TV logged to SmartThings? Because my suspect is that is created as "placeholder" because you don't configure the connection on the TV!!!

It's work with the new instruction!

Just another question: is your TV logged to SmartThings? Because my suspect is that is created as "placeholder" because you don't configure the connection on the TV!!!

I think so the tv was showing the pin that you must enter in the smartthings app.

You should check on the TV menu. There is an aption to login the TV.

It's work for me too. Thank you ollo!

You should check on the TV menu. There is an aption to login the TV.

Do you mean the Samsung account login?

I should have found the cause of the original problem. With latest version deviceID should be automatically detected again. Issue was that response from SmartThings is changed!!!!

I should have found the cause of the original problem. With latest version deviceID should be automatically detected again. Issue was that response from SmartThings is changed!!!!

ok, you think it's better to delete everything and redo it, or if it works with the manual insertion procedure leave it and not touch?

Automatic dectection do the same thing that you did manually, the saved configuration in the registry at the end is the same.
You can update the component and leave the configuration as is.

ok, i update all.
now in logs i have:
2020-04-11 18:05:09 ERROR (MainThread) [custom_components.samsungtv_smart.media_player] SamsungTV Smart - Error refreshing from SmartThings
2020-04-11 18:42:39 WARNING (MainThread) [custom_components.samsungtv_smart.media_player] Retrieving applications list from TV. It is suggested to define a manual app_list (see component documentation)!!!
2020-04-11 18:42:49 ERROR (MainThread) [custom_components.samsungtv_smart.media_player] Error retrieving application list from TV. Method will be retried.
2020-04-11 18:42:49 WARNING (MainThread) [homeassistant.helpers.entity] Updating state for media_player.q70 (<class 'custom_components.samsungtv_smart.media_player.SamsungTVDevice'>) took 10.068 seconds. Please report it to the custom component author.

This logs I thing is related to network issue. This days network is overloaded. I didn't any change that can affect this error! I suppose they will dissapear when network works fine.
I also suppose that, error apart, the component is working, correct? For the update on the site, not always reflect the local changes...

hi, i update the component.
when i restart ha. i have this error:
2020-04-16 11:01:52 WARNING (MainThread) [custom_components.samsungtv_smart.media_player] Retrieving applications list from TV. It is suggested to define a manual app_list (see component documentation)!!!
2020-04-16 11:01:55 ERROR (MainThread) [custom_components.samsungtv_smart.media_player] Error retrieving application list from TV. Method will be retried.
and the time to reboot ha is too high.
at this time, the tv is off.
is it better to configure the list of apps by hand?

Yes, automatic generation of app list not always work and is time consuming. As you read in the warning you should define your app list with configuration.yaml. Read documentation for details.

ok, I'm doing, I don't understand from the documentation if I have to write all the various app and source lists below

i reboot ha with the configuration posted some minutes ago.
and now the entity is not avalaible and have this in the logs:

You don't write your app list properly.

You don't write your app list properly. Post it here so that I ca check.

ok, I fixed, there were no quotes.
now i have no errors, and it seems to change apps perfectly.
an error: if I turn on the TV the mediaplayer signals the channel correctly. but if I go to an app and then press on digitaltv, it no longer reports it.

You should check what happen on smartthings web site for your device.
Sometimes do not update the status.

You should check what happen on smartthings web site for your device.
Sometimes do not update the status.

in the smarthings site it's ok...

maybe it's a limit of the custom media player?
try to set up source lists too?

No, I don't think this is the problem.
But what happen on the device tvChannelName when you select an app?

No, I don't think this is the problem.
But what happen on the device tvChannelName when you select an app?

it's ok.
now in this situation, the change of channel not work, but if I turn the TV off and on again, the channels appear on the mediaplayer and the channel change works

update: in the site smarthings not update the channel...but in the app android yes.
remove the tv from smarthings?

If you want to clean-up the situation, do the following in this exact order:

remove TV from the APP
remove TV from HA
logout TV from smartThings (from TV menu)
delete TV device from smartThings web site (if still there)
login TV to smartThings (from TV menu)
add TV to your phone app (and set your pref as location, name etc)
generate a new SmartThings token
add TV to HomeAssistant with the new token
check SmartThings site

If you want to clean-up the situation, do the following in this exact order:

remove TV from the phone APP
remove TV from HA
logout TV from smartThings (from TV menu)
delete TV device from smartThings web site (if still there)
login TV to smartThings (from TV menu)
add TV to your phone app (and set your pref as location, name etc)
generate a new SmartThings token
add TV to HomeAssistant with the new token
check SmartThings site

ok, now from the site smarthings it's ok.
the problem is only for the hdmi and tv.
i configure     source_list: '{"TV": "KEY_TV", "SHIELD": "KEY_HDMI"}'
but not working...and not logs...
p.s. can you have telegram?

For source control you must use ST key, normal key do not work.
so use this: '{"TV": "ST_TV", "SHIELD": "ST_HDMI1"}'
Anyway I did some test, what I discover is that not always channel info in SmartThings is refreshed. To have the refresh of the channel working you have to open 1 time, after powering on the TV, the SmartThing apps and look for the current source.
This happen from the last update that also broke the TV discovery, so it seems that SmartThings API is in development and something else can still change...
I searched throught documentation but I found nothing to solve this issue.

If you want to clean-up the situation, do the following in this exact order:
* remove TV from the phone APP

* remove TV from HA

* **logout TV from smartThings (from TV menu)**

* delete TV device from smartThings web site (if still there)

* **login TV to smartThings (from TV menu)**

* add TV to your phone app (and set your pref as location, name etc)

* generate a new SmartThings token

* add TV to HomeAssistant with the new token

* check SmartThings site

I've been having exact same problem here with the new The Frame 2020 tv. I did those steps and tv still registers as "Placeholder" with blank network id. On top of it all it shows up as "inactive" in the web portal.

@ollo69 Maybe you could ask there https://community.smartthings.com

If you want to clean-up the situation, do the following in this exact order:
* remove TV from the phone APP

* remove TV from HA

* **logout TV from smartThings (from TV menu)**

* delete TV device from smartThings web site (if still there)

* **login TV to smartThings (from TV menu)**

* add TV to your phone app (and set your pref as location, name etc)

* generate a new SmartThings token

* add TV to HomeAssistant with the new token

* check SmartThings site

I've been having exact same problem here with the new The Frame 2020 tv. I did those steps and tv still registers as "Placeholder" with blank network id.

logged in on the tv, authenticated the app using pin etc.
I generate the network-id from your suggested link (i tried both v1 and v4 option).
Switching tv from placeholder to Samsung OCF TV populates all the fields and shows all details correctly

@ollo69
yes, the same with my q70 2019.
but now, a day later, the problem has come back on the smarthings website, which always reports as a YouTube channel. while the tv is on a tv channel.
Not update the status, yesterday.
I had solved it by removing it and putting it back in my smarthings profile ... but it is absurd to think of doing it continuously.

someone have this problem?

Keep in mind that this approach to SmartThings is not the official way to interact with device, have to be considered a work-aroud. In fact we are using the TV device  that create the SmartThings phone app, and from some time the device is not created anymore as TV but as "place-holder". Leaving aside the fact that Samsung itself has its problems managing TVs remotely (see Alexa skill), the correct approach would be to create an app which in turn creates a device with which to interact, but to do this HomeAssistant should be connected in Https and would greatly limit the use of the component. All I can do now is to eventually exclude the channel check via SmartThings with an option, everything else would require a more in-depth analysis of the whole SmartThings world. Any suggestions are still welcome...

The only problem I have:
SamsungTV Smart - Error refreshing from SmartThings
Althrough component works fine - I can mute/unmute change vol and call different apps, and well twice / hour I've got that error - I think it has to be that way.
@ollo69 does your HA shows that in logs too?

Yes, it happen, but this log is too aggressive and in many case unusefull. Just occur when network fail.
I have days when never occurs and days with many instance of the message,
I will manage in better way on next release

Same problems here.

I will manage in better way on next release

So have you already found a better way to do it? Are you developing it already?

idem, i have many logs.
[custom_components.samsungtv_smart.media_player] SamsungTV Smart - Error refreshing from SmartThings

Cool, glad to hear as I was chasing some problems today when HA became unresponsive on VM machine and looking into all logs now. Good work - cheers for that component!

So is there a definitive set up guide or things are still broken on samsung's end? :)

So is there a definitive set up guide or things are still broken on samsung's end? :)

What do you mean? Did you istall the component? Is it working for you?
The only current issue is that not always channel names / numbers are updated, but, as I know, the component is working.
The only alternative option SmartThings side is to implement comunication as is implemented for HA SmartThings component, creating a dedicated SmartThings app. But this require that HA is published in the internet with HTTPS protocol and will limit a lot usage of the component, so I don't consider this a valid alternative.

Looks like the issue was indeed with the Smarthings API. I tried it again today and everything worked right away from the first try. I checked the API and it looked like the tv entity was populated correctly on Samsung site this time as well. We can probably close this ticket as it was their fault... On another note ... for some reason, this particular new tv doesn't update state to Smartthings often. Hopefully, they will fix this at some point...
thanks again for your support!

I don't close this issue, because I'm really not covinced by the use of SmartThings and I need some feedback to better understand
I deleted and recreated my TV on SmartThings apps and now TVChannel are not updating anymore.
Many user are reporting the same, so in last component version I disabled the use of channel info (can be enabled again using the "option" button in integration page).
I'm not understanding if this is really a SmartThing issue or what really Samsung want to do.
May be different TV model have different behavior.
Pls report here your feed-back to identify the best way to use SmartThings

The good news is that a few days ago the TV channel info started to update and shows the current channel.

Closing this issue due to inactivity from long time.


Smesher screen - minor UI updates

Smesher Screen

[x] Started creating -> Started
[x] Finished creating -> Finished
[x] If setup was finished then hide the progress row - it is only relevant before setup has completed.
[x] There's redundant space around 'ONLINE' in the sub-title (below smesher title).
[x] Network name is wrong. Currently Network 10002 (Testnet) - it should be the current network friendly name, like the name displayed in the network screen.
[x] Change modify button label from MODIFY POST DATA to EDIT.
[x] Clicking on smesher id (blue line) should open the explorer in user's web browser with the smesher id perm url e.g https://explorer.spacemesh.io/smeshers/0x0065ec143ea822326cdd016039822b84f14664f5b48558e52d6f68b7e832f49621

I've verified all issues are fixed in the pr.


Test against timezone aware datetimes

Need to test also against timezone aware datetimes, as they're quite popular, especially in Django.

I've found that it works well with arrow in opsdroid. But explicit unit tests would be good.

Closed via #6


Add unit tests to cairo_runner

This PR adds more unit tests to cairo_runner.rs. Also, it removes the warning (warning: use of deprecated trait pyo3::PyIterProtocol: prefer #[pymethods] to #[pyproto]) that occurs when compiling them.

Codecov Report

Merging #166 (053a3ad) into main (b80cb28) will increase coverage by 2.71%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #166      +/-   ##
==========================================
+ Coverage   80.68%   83.40%   +2.71%
==========================================
  Files          12       12
  Lines         699      699
==========================================
+ Hits          564      583      +19
+ Misses        135      116      -19

Impacted Files
Coverage Δ

src/cairo_runner.rs
76.53% <ø> (+6.92%)
:arrow_up:

src/utils.rs
94.44% <0.00%> (+5.55%)
:arrow_up:

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


Numeric values in styles apply every responsive variant

See below for an example of a numeric value:
const Example = () => <Box styles={{ zIndex: 1 }} />

Using a numeric value will apply classNames for every responsive variant instead of just mobile. Changing to:
const Example = () => <Box styles={{ zIndex: "1" }} />

is a temporary workaround.
A likely culprit (not 100% sure) is here.
We probably need to check for 'number' as well as 'string':
https://github.com/WalltoWall/calico/blob/2c952efbb6c70a237c529b31f6bb4f2cb16732ec/src/utils.ts#L104-L111

This has been fixed.


Need guidance for developing extensions for the new Editor

Hi, this is related to the https://github.com/VsVim/VsVim addin, details below.

We're trying to port this add-in to Visual Studio for Mac's New Editor.
The documentation for how to do this is scarce, we've gathered what we could from the initial announcement, the brief summary of developing with the new editor, and various blog posts.
Could you please explain exactly how to register such an add-in with Visual Studio for Mac? We're creating a separate add-in for Mac, so we don't need it to be cross-platform with Windows; just the Mac instructions will suffice.
The functionality we need to expose is basically of listening for new text editor views and hooking into the keyboard pipeline.
We're blocked on this; hope you can help.
Thanks!


Provide ribbon xml fragments from extensions

With the addition of the extension loading machinery, we are in a good position to allow PyXLL extensions to be easily deployed and discovered with no manual editing of config files.
What's missing from this is a way to contribute to the ribbon. One can imagine the plugin loading code might also receive an xml fragment from each plugin.
I don't know what the ribbon manipulating code looks like right now, but in the worse case scenario, we could write these ribbon fragments out into a single file and then load it the way we already do. The best case would be something like responding to ribbon update events from the plugins and handling it dynamically.

Sounds good. Let's catch up after the 3.0 release and see what can be done on the pyxll side to make that work best.

Sure. Depending on how far away that is, I might not be available for a little while. At the very least, I'll leave an open PR here that implements my idea as best we can given what we already have.


Even if the values are the same when pasting, they are modified.

Description
When you copy data and paste it into a cell with the same value, the cell is marked as modified and is updated when you save it.
DBeaver Version
Community Edition <IP_ADDRESS>411171748
Operating System
Windows 10
Database and driver
Database: Postgresql
Steps to reproduce

open query results
move to result cell
ctrl + c and ctrl + v

Additional context
No response

Could you please explain why you need this feature?

Could you please explain why do you need this feature?

This is the behavior in previous versions. (Check Version <IP_ADDRESS>408041450)
Isn't it normal that if the values are the same, the update doesn't proceed? If you have a csv file with a size of 500 x 500 and only one cell has a different value, and you paste it into the editor, is it correct that 500 rows will be updated?


Unknown Type Name clang error when using NLOHMANN_DEFINE_TYPE_NON_INTRUSIVE

What is the issue you have?
Clang reports:
error: unknown type name 's'

with this code:
struct Foo {
	string s;
};

NLOHMANN_DEFINE_TYPE_NON_INTRUSIVE(Foo, s)

(in the same namespace)
more specifically:
main.cpp:24:37: error: unknown type name 's'
NLOHMANN_DEFINE_TYPE_NON_INTRUSIVE(Foo, s)
                                            ^  (the '^' appears under the 's')
main.cpp:26:1: error: expected function body after function declarator

Compiled as so:
clang++ -std=c++17 main.cpp -o build/main -O3
~/projects/jsontest clang --version
Apple clang version 11.0.3 (clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin19.5.0
Thread model: posix

Which version of the library did you use?
3.8.0

[ ] latest release version 3.7.3
[X] other release - please state the version: 3.8.0
[ ] the develop branch

If you experience a compilation error: can you compile and run the unit tests?

[X] yes
[ ] no - please copy/paste the error message below

Please try again with the latest develop branch. This issue should have been fixed lately.
The following code works with Apple clang version 12.0.0 (clang-1<IP_ADDRESS>).
#include <iostream>
#include <nlohmann/json.hpp>

using json = nlohmann::json;

struct Foo {
    std::string s;
};

NLOHMANN_DEFINE_TYPE_NON_INTRUSIVE(Foo, s)

int main()
{
    Foo foo;
    foo.s = "Hello, world!";

    json j = foo;
    std::cout << j << std::endl;
}

Output:
{"s":"Hello, world!"}

You are correct!! Thanks.


[Bug]: Nominal capacity : Parameter Change

PyBaMM Version
23.4.1
Python Version
3.11.5
Describe the bug
I'm currently simulating a Pouch-Cell with a nominal capacity of 65Ah. When I run the experiment using the predefined nominal cell capacity with OKane2022, everything works fine. However, when I adjust it to match the specific cell I'm working with, I encounter a debugging message.
Can you please indicate to me what may be the problem, and thank you for prividing the community with this very helpfull tool.
Steps to Reproduce
Action that created the error:
Parameters Change
parameter_values["Nominal cell capacity [A.h]"] = 65
Error:
The linesearch algorithm failed with too small a step.
At t = 50.7015 and h = 1.02133e-09, the corrector convergence failed repeatedly or with |h| = hmin.
The linesearch algorithm failed with too small a step.
The linesearch algorithm failed with too small a step.
The linesearch algorithm failed with too small a step.
The linesearch algorithm failed with too small a step.
2024-05-03 17:10:16.280 - [ERROR] callbacks.on_experiment_error(224): Simulation error: Maximum number of decreased steps occurred at t=0.0 (final SolverError: 'Error in Function::call for 'F' [IdasInterface] at .../casadi/core/function.cpp:1401:
Error in Function::call for 'F' [IdasInterface] at .../casadi/core/function.cpp:330:
.../casadi/interfaces/sundials/idas_interface.cpp:596: IDACalcIC returned "IDA_LINESEARCH_FAIL". Consult IDAS documentation.'). For a full solution try reducing dt_max (currently, dt_max=600) and/or reducing the size of the time steps or period of the experiment. Set return_solution_if_failed_early=True to return the solution object up to the point where failure occured.
Unexpected exception formatting exception. Falling back to standard exception
Traceback (most recent call last):
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\pybamm\solvers\casadi_solver.py", line 679, in _run_integrator
pybamm.logger.debug("Calling casadi integrator")
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\casadi\casadi.py", line 23390, in call
return self.call(kwargs)
^^^^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\casadi\casadi.py", line 20039, in call
return _casadi.Function_call(self, *args)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
RuntimeError: Error in Function::call for 'F' [IdasInterface] at .../casadi/core/function.cpp:1401:
Error in Function::call for 'F' [IdasInterface] at .../casadi/core/function.cpp:330:
.../casadi/interfaces/sundials/idas_interface.cpp:596: IDACalcIC returned "IDA_LINESEARCH_FAIL". Consult IDAS documentation.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\pybamm\solvers\casadi_solver.py", line 248, in _integrate
f"{t_window[0]:.2f} < t < {t_window[-1]:.2f}"
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\pybamm\solvers\casadi_solver.py", line 685, in _run_integrator
# If it doesn't work raise error
pybamm.expression_tree.exceptions.SolverError: Error in Function::call for 'F' [IdasInterface] at .../casadi/core/function.cpp:1401:
Error in Function::call for 'F' [IdasInterface] at .../casadi/core/function.cpp:330:
.../casadi/interfaces/sundials/idas_interface.cpp:596: IDACalcIC returned "IDA_LINESEARCH_FAIL". Consult IDAS documentation.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\IPython\core\interactiveshell.py", line 3526, in run_code
exec(code_obj, self.user_global_ns, self.user_ns)
File "C:\Users\OO000009\AppData\Local\Temp\ipykernel_11516\1774507600.py", line 51, in
solution = sim.solve(initial_soc=0.02)
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\pybamm\simulation.py", line 807, in solve
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\pybamm\simulation.py", line 785, in solve
if cycle_num == 1 and step_num == 1:
^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\pybamm\solvers\base_solver.py", line 1235, in step
final_event_values = {}
^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\pybamm\solvers\casadi_solver.py", line 283, in _integrate
else:
^
pybamm.expression_tree.exceptions.SolverError: Maximum number of decreased steps occurred at t=0.0 (final SolverError: 'Error in Function::call for 'F' [IdasInterface] at .../casadi/core/function.cpp:1401:
Error in Function::call for 'F' [IdasInterface] at .../casadi/core/function.cpp:330:
.../casadi/interfaces/sundials/idas_interface.cpp:596: IDACalcIC returned "IDA_LINESEARCH_FAIL". Consult IDAS documentation.'). For a full solution try reducing dt_max (currently, dt_max=600) and/or reducing the size of the time steps or period of the experiment. Set return_solution_if_failed_early=True to return the solution object up to the point where failure occured.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\IPython\core\interactiveshell.py", line 2120, in showtraceback
stb = self.InteractiveTB.structured_traceback(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\IPython\core\ultratb.py", line 1435, in structured_traceback
return FormattedTB.structured_traceback(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\IPython\core\ultratb.py", line 1326, in structured_traceback
return VerboseTB.structured_traceback(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\IPython\core\ultratb.py", line 1173, in structured_traceback
formatted_exception = self.format_exception_as_a_whole(etype, evalue, etb, number_of_lines_of_context,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\IPython\core\ultratb.py", line 1088, in format_exception_as_a_whole
frames.append(self.format_record(record))
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\IPython\core\ultratb.py", line 970, in format_record
frame_info.lines, Colors, self.has_colors, lvals
^^^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\IPython\core\ultratb.py", line 792, in lines
return self._sd.lines
^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\stack_data\utils.py", line 145, in cached_property_wrapper
value = obj.dict[self.func.name] = self.func(obj)
^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\stack_data\core.py", line 698, in lines
pieces = self.included_pieces
^^^^^^^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\stack_data\utils.py", line 145, in cached_property_wrapper
value = obj.dict[self.func.name] = self.func(obj)
^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\stack_data\core.py", line 649, in included_pieces
pos = scope_pieces.index(self.executing_piece)
^^^^^^^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\stack_data\utils.py", line 145, in cached_property_wrapper
value = obj.dict[self.func.name] = self.func(obj)
^^^^^^^^^^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\stack_data\core.py", line 628, in executing_piece
return only(
^^^^^
File "C:\Users\OO000009\AppData\Local\anaconda3\Lib\site-packages\executing\executing.py", line 164, in only
except (SyntaxError, ValueError):
executing.executing.NotOneValueFound: Expected one value, found 0
Relevant log output
No response

Hi @Not-slytherin, The nominal capacity is part of the parameterization of the cell model itself. Changing from the capacity of 5 Ah in the model to 65 Ah in your desired simulation would require changing more than just the capacity value. Things like the cell/electrode dimensions and lithium concentration would need to be changed as well.
PyBaMM does not go through and adjust all the other parameters to match the nominal capacity. Unfortunately this is not trivial to accomplish

Thank you so much for the clarification.
Best regards,

If you are interested in modeling larger batteries, you can look into liionpack which lets you turn a bunch of cells in PyBaMM into a single battery


argo watch does not update/refresh screen

Checklist:

[x] I've included the version.
[x] I've included reproduction steps.
[ ] I've included the workflow YAML.
[ ] I've included the logs.

What happened:
argo watch WORKFLOW_NAME most of the time does not refresh it's output
What you expected to happen:
I expected argo watch to refresh it's output every few seconds as was the case with at least v2.4
How to reproduce it (as minimally and precisely as possible):
Submit a workflow and either add the '--watch' flag, or submit a workflow and execute the argo watch WORKFLOW_NAME afterwards.
Anything else we need to know?:
Environment:

Argo version:

$ argo version
argo: v2.7.1
  BuildDate: 2020-04-07T16:40:24Z
  GitCommit: 2a3f59c10ae260a460b6ad97a0cadd8667d4b488
  GitTreeState: clean
  GitTag: v2.7.1
  GoVersion: go1.13.4
  Compiler: gc
  Platform: darwin/amd64

Kubernetes version :

$ kubectl version -o yaml
clientVersion:
  buildDate: "2019-10-15T23:41:55Z"
  compiler: gc
  gitCommit: c97fe5036ef3df2967d086711e6c0c405941e14b
  gitTreeState: clean
  gitVersion: v1.16.2
  goVersion: go1.12.10
  major: "1"
  minor: "16"
  platform: darwin/amd64
serverVersion:
  buildDate: "2020-01-31T20:00:26Z"
  compiler: gc
  gitCommit: c52f59bbba5fbf21fbb18e9a06f96e563fe4c20a
  gitTreeState: clean
  gitVersion: v1.14.8
  goVersion: go1.12.10
  major: "1"
  minor: "14"
  platform: linux/amd64

Message from the maintainers:
If you are impacted by this bug please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

This maybe fixed in v2.7.2 CLI- can you please try that version?

Just tried v2.7.2 CLI against v2.7.1 in cluster.
The first 8 seconds the screen refreshed every 1-2 seconds, after which it stopped refreshing until after 38 seconds. It refreshed a few more times every 1-2 seconds to finally "freeze" and stop refreshing the last time 52 seconds in. The workflow was still running nonetheless.
Tomorrow I'll give it another shot also updating the workflow-controller to v2.7.2

Looking at the most recent code I notice it only updates when the workflow changed. So you should see a change every time a step changes state (e.g becomes complete) or the workflow changes state (e.g. completes). You should not expect it to just "update every few seconds" without a state change.

Well, initially that's the impression that I had. However, it definitely does not refresh after a while, although the step changed state.
I just tested with a workflow that finished little over 15min, however the argo watch command stopped refreshing after 1m6s when it was performing the 3rd step out of 11.

Can you please provide an example of the workflow so I can try and repo?

I have a potential fix, but I'm not greatly confident in it.
I forgot to ask can you run argo --loglevel DEBUG watch ... and see what that says?

I was able to reproduce with the following workflow:
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: dag-nested-
spec:
  entrypoint: diamond
  templates:
  - name: echo
    inputs:
      parameters:
      - name: message
    container:
      image: alpine:3.7
      command: [sleep , 30]
  - name: diamond
    dag:
      tasks:
      - name: A
        template: nested-diamond
        arguments:
          parameters: [{name: message, value: A}]
      - name: B
        dependencies: [A]
        template: nested-diamond
        arguments:
          parameters: [{name: message, value: B}]
      - name: C
        dependencies: [A]
        template: nested-diamond
        arguments:
          parameters: [{name: message, value: C}]
      - name: D
        dependencies: [B, C]
        template: nested-diamond
        arguments:
          parameters: [{name: message, value: D}]
  - name: nested-diamond
    inputs:
      parameters:
      - name: message
    dag:
      tasks:
      - name: A
        template: echo
        arguments:
          parameters: [{name: message, value: "{{inputs.parameters.message}}A"}]
      - name: B
        dependencies: [A]
        template: echo
        arguments:
          parameters: [{name: message, value: "{{inputs.parameters.message}}B"}]
      - name: C
        dependencies: [A]
        template: echo
        arguments:
          parameters: [{name: message, value: "{{inputs.parameters.message}}C"}]
      - name: D
        dependencies: [B, C]
        template: echo
        arguments:
          parameters: [{name: message, value: "{{inputs.parameters.message}}D"}]

Below screenshot shows at the top the output of argo watch that "froze" after over a minute, while the bottom part shows the output of argo get after a few minutes.

Are you using argo server? Any kind of ingress?
Can I ask you to run with the --loglevel DEBUG command? Or did you already?

Yes, we are running argo-server in client auth mode.
We also have a few ingresses, however none that serve Argo (Argo-server is accessed through port-forward).
Please find the last lines of the debug output here
Namespace:           argo
ServiceAccount:      argo
Status:              Running
Created:             Thu Apr 16 23:45:03 +0200 (1 minute ago)
Started:             Thu Apr 16 23:45:03 +0200 (1 minute ago)
Duration:            1 minute 25 seconds

STEP                 TEMPLATE        PODNAME                      DURATION  MESSAGE
 ● dag-nested-jg6pz  diamond
 └-● A               nested-diamond
   ├-✔ A             echo            dag-nested-jg6pz-4063873041  46s
   ├-✔ B             echo            dag-nested-jg6pz-4013540184  35s
   ├-✔ C             echo            dag-nested-jg6pz-4030317803  34s
   └-◷ D             echo            dag-nested-jg6pz-4114205898  1s        ContainerCreating
DEBU[0095] Received event
DEBU[0095] Sending event                                 phase=Running type=MODIFIED workflow=dag-nested-jg6pz
Name:                dag-nested-jg6pz
Namespace:           argo
ServiceAccount:      argo
Status:              Running
Created:             Thu Apr 16 23:45:03 +0200 (1 minute ago)
Started:             Thu Apr 16 23:45:03 +0200 (1 minute ago)
Duration:            1 minute 29 seconds

STEP                 TEMPLATE        PODNAME                      DURATION  MESSAGE
 ● dag-nested-jg6pz  diamond
 └-● A               nested-diamond
   ├-✔ A             echo            dag-nested-jg6pz-4063873041  46s
   ├-✔ B             echo            dag-nested-jg6pz-4013540184  35s
   ├-✔ C             echo            dag-nested-jg6pz-4030317803  34s
   └-● D             echo            dag-nested-jg6pz-4114205898  5s
DEBU[0107] Result channel done


WQL is not working for 'marker' option

The WQL {"attr::Name::marker": "1"} is not working when we use with GET /credentials API. From below line of code (commented line), it seems 'marker' info is not getting stored wallet while storing Credentials.
https://github.com/hyperledger/aries-cloudagent-python/blob/main/aries_cloudagent/anoncreds/holder.py#L235

The marker tag is not needed on Askar because the $exist filter can be used to check for the presence of an attribute: https://github.com/hyperledger/aries-cloudagent-python/blob/cd4f1dc8fddc1194e0abc00ef4fb3d671745ad51/aries_cloudagent/anoncreds/holder.py#L339C23-L339C23

@andrewwhitehead : Could you give JSON WQL for to retrieve the all Credentials which have 'Name' as one of the attribute. I am planning to use this in GET /credentials and /present-proof/records/{pres_ex_id}/credentials. Hope, same WQL works in both the APIs.

I also get an error if I use the marker field on the /present-proof/send-request endpoint with acapy 0.7.5
"proof_request": {
"requested_attributes": {
"0_Participant": {
"value": {
"restrictions": {
"0": {
"attr::XXX::marker": {
"key": [
"String does not match expected pattern."

Per this comment: https://github.com/hyperledger/aries-cloudagent-python/issues/1257#issuecomment-1895873036
The answer is that in ACA-Py restrictions to use:

use value instead of marker
lower case the attribute name

Thanks @stockschlaeger for finding/noting that.  I'll get this added to the documentation at some point.

"At some point" meaning in the best place I can find to put it. I'll make the PR in the next day or two. :-)

The credit for @stockschlaeger is ok since we're working on the same project and it was his finding :)
However, the lower-case solution works only when using aca-py as verifier and the Lissi Wallet as holder. If you're using aca-py as holder as well then the attribute must be written exactly as in the schema definition.
The anoncred specification lacks a clear rule on that. For name and names it says it's case insensitive. For the restriction part it says nothing on that matter.


Use python dataclasses to streamline bodies.py

🐞 Problem
There is some redundant code in bodies.py. Use python dataclasses to remove the redundancies and streamline the code. Other solutions are also welcome. See #647 for a bit more discussion.
📋Contribution steps
Please consult the developer docs to setup your dev environment and get started. Also, feel free to communicate, if you hit a snag. You can do so in our Gitter or Matrix/Element chat rooms (recommended), or you can also reply here.

Comment below about what you've started working on.
Add, commit, push your changes
Submit a pull request and add this in comments - Addresses #649
Ask for a review in comments section of pull request
Celebrate your contribution to this project 🎉

Hi, I'd like to work on this

@VPraharsha03 Sure. Go ahead.

Hello, is this issue open or is it already taken?

@BorisHamadej @VPraharsha03 is working on this issue. Perhaps, look into #630? It has a list of modules that require documentation updates. Work on the documentation of any of those modules will count towards #630.


adb logcat output can't be filter with other tool(rg)

I always use rg to filter sth for adb logcat, it work fine in cmd, but it can't work with powershell.
Steps to reproduce
PS$ adb logcat | rg sth
output nothing.

use Select-String can be filter
PS$ adb logcat | Select-String sth
work fine.

use other command, it can be filter with rg.
PS$ dir | rg 2020
d-----         2020/11/3      9:54                test

use cmd.exe, it can be filter with rg.
cmd$ adb logcat | rg sth
work fine.

Expected behavior
display result

Actual behavior
nothing

Environment data
I test version 5.1(windows 10 version) and version 7.1(github last release), have same problem

Name                           Value
----                           -----
PSVersion                      5.1.19041.610
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.19041.610
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Name                           Value
----                           -----
PSVersion                      7.1.0
PSEdition                      Core
GitCommitId                    7.1.0
OS                             Microsoft Windows 10.0.19042
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Has ripgrep an debug output switch? This could help to find a root of the issue.

@iSazonov
in cmd, it can see rg's debug information.
cmd$ adb logcat | rg --debug ActivityManager
DEBUG|grep_regex::literal|crates\regex\src\literal.rs:58: literal prefixes detected: Literals { lits: [Complete(ActivityManager)], limit_size: 250, limit_class: 10 }
DEBUG|globset|crates\globset\src\lib.rs:431: built glob set; 0 literals, 0 basenames, 12 extensions, 0 prefixes, 0 suffixes, 0 required extensions, 0 regexes
11-14 10:43:51.481  1428  1428 I SystemServiceManager: Starting com.android.server.am.ActivityManagerService$Lifecycle
11-14 10:43:51.511  1428  1428 I ActivityManager: Memory class: 192

in ps, it output nothing, feel like no any data sent to rg
PS$ adb logcat | rg --debug ActivityManager

What is a result of:
$a = adb logcat
$a | rg ActivityManager

PS>$a=adb logcat
PS>echo $a
nothing
PS>$a | rg ActivityManager
nothing
PS>$a="ActivityManager"
PS>$a | rg ActivityManager
ActivityManager

@fcying Thanks!

PSVersion                      5.1.19041.610

The repository is for PowerShell 7 only, not for Windows PowerShell. Can you reproduce on latest PowerShell 7.1?

I try version 7.1.0, It have same problem.
Name                           Value
----                           -----
PSVersion                      7.1.0
PSEdition                      Core
GitCommitId                    7.1.0
OS                             Microsoft Windows 10.0.19042
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

I have same problem,  adb logcat |  find something  also does not work


feat: writefile magic func

https://github.com/janpfeifer/gonb/issues/103

Thanks @potoo0 !!
We were working at the same time on this, so let me take your changes and merge with mine -- I'll reorganize it a bit to separate the %%writefile (and also %%script that I'm implementing) to something that comes before Parse. It is also needed to disable  auto-complete in cells that may not contain Go.


Be able to find places talking about a paper through Arxie-Bot

It would be helpful to quickly jump to comment threads discussing a paper if there exist some. Something like:
@arXie-bot get me conversations about X
where X is the number of a paper in the current context.
Arxie-Bot would the return something like:
Here's some places people are talking about <PAPER-NAME>:

* Reddit r/machinelearning (21 Comments)
* Hackernews (1 Comment)
* Open Review (5 Comments)

Things to consider

Should limit the number of individual subreddits that can have their threads linked.
Open Review would be a particularly useful one to link
Is it possible (and useful) to link Twitter threads about a paper?

This could be implemented with Interactive Messages, where say a paper summary has buttons to "save" and "find conversations".

Turns out Interactive Message buttons can't link somewhere, so that idea is out.

This has been started for Reddit with #4


Python function project takes >250 Mb on disk

Create a new project with a simple Python Azure Function. Disk space is >250 Mb on Windows.
A good user experience might be something like:

Create simple function that can be tested locally without requiring a virtual env, or even do basic syntax check and allow option of not running locally.
Basic check for requirements.txt
Deploy to Azure

Azure Functions VS Code extension 0.14.0.
VS Code 1.31.1.
Windows 10.

@gbowerman the func cli requires a virtual environment to run locally. If you want that requirement removed, I suggest filing an issue here https://github.com/Azure/azure-functions-core-tools
However your second suggestion ("do basic syntax check and allow option of not running locally") might be possible from our extension's point of view. We could delay creating a virtual environment until the user tries to debug. This might help speed up project creation (https://github.com/Microsoft/vscode-azurefunctions/issues/1029), but it might also be confusing. @fiveisprime thoughts?
That being said, I'm not even sure deploy is supported without a virtual environment. @asavaritayal is func pack supported without a venv? It wasn't working for me, but that might just be my machine.

This is only because we're creating a virtual environment. Separate issue here for possibly removing venv: https://github.com/Microsoft/vscode-azurefunctions/issues/1092


do not perform lookups unless opted in

The passed provider may provide a way to do ENS lookups, but if it does not, no lookups should be performed unless the integrator opts in. This prevents data/privacy leaking.

@davatar/react makes calls to alchemy for ENS resolution on chainIDs besides 1, 3, 4, 5 (code).

I'm not sure I understand, what's the leak vector here? This library looks up ENS on mainnet (unless on a testnet with ENS deployed, where we assume we want to test with an ENS on that chain), which should be the correct behavior, even if the "connected" chain is a different one (say Polygon).