Add localized and localized(comment:) to String

Checklist

[x] I checked the Contributing Guidelines before creating this request.
[x] New extensions are written in Swift 4.
[x] New extensions support iOS 8.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+.
[ ] I have added tests for new extensions, and they passed.
[x] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[x] All extensions are declared as public.

I wasn't able to write tests for these extensions, because they require running with a different localization. As far as I know, it is currently not possible to set the 'Application Language' for a single test.

          2 Warnings

:warning:
Consider adding tests for new extensions or updating existing tests for a modified SwifterSwift extension

:warning:
The source files have been modified. Please consider adding a CHANGELOG entry if necessary.

          1 Message

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

  Generated by :no_entry_sign: Danger

Sorry for closing and reopening the pull request. Back to our conversation:
I frequently use localized strings without comments and would prefer "HelloWorld".localized to "HelloWorld".localized(). What do you think about it?

@Vyax I'd still prefer to have a single function :)
Lets ask other members here: @SD10, @LucianoPAlmeida @pvtmert what do you think guys?

Hey guys :) @omaralbeik, @Vyax , I like the localized to, but in my opinion I prefer a method with comment, because it's very useful when you work on teams to the other guys know what that localized key means. Maybe if you like use without comment you can define a default value like:
      func localized(comment: String = "")  {
            return NSLocalizedString(self, comment: comment)
      }

This way you can use both with or without comments :)

Okay, 2 vs 1. I changed it to a single function with an optional parameter.


Renaming a struct field does not catch all usages

In https://github.com/rust-lang/rust/blob/50d3c2a3cb96b6af2b5e3b9d08578a556ac70ede/src/librustdoc/clean/types.rs#L84, rename inner to kind. Most usages are updated, but not all.
    Checking rustdoc v0.0.0 (/home/joshua/rustc2/src/librustdoc)
error[E0609]: no field `inner` on type `&types::Item`
    --> src/librustdoc/html/render/mod.rs:3100:25
     |
3100 |                 match v.inner {
     |                         ^^^^^ unknown field
     |
     = note: available fields are: `source`, `name`, `attrs`, `kind`, `visibility` ... and 3 others

error[E0609]: no field `inner` on type `&types::Item`
    --> src/librustdoc/html/render/mod.rs:3150:58
     |
3150 |             if let clean::VariantItem(ref var) = variant.inner {
     |                                                          ^^^^^ unknown field
     |
     = note: available fields are: `source`, `name`, `attrs`, `kind`, `visibility` ... and 3 others

error[E0609]: no field `inner` on type `&types::Item`
    --> src/librustdoc/html/render/mod.rs:3182:60
     |
3182 |                     if let StructFieldItem(ref ty) = field.inner {
     |                                                            ^^^^^ unknown field
     |
     = note: available fields are: `source`, `name`, `attrs`, `kind`, `visibility` ... and 3 others

error[E0609]: no field `inner` on type `&types::Item`
    --> src/librustdoc/html/render/mod.rs:3592:70
     |
3592 |                         if let clean::TypedefItem(ref tydef, _) = it.inner {
     |                                                                      ^^^^^ unknown field
     |
     = note: available fields are: `source`, `name`, `attrs`, `kind`, `visibility` ... and 3 others

error[E0609]: no field `inner` on type `&types::Item`
    --> src/librustdoc/html/render/mod.rs:4183:80
     |
4183 |                     impl_.inner_impl().items.iter().find_map(|item| match item.inner {
     |                                                                                ^^^^^ unknown field
     |
     = note: available fields are: `source`, `name`, `attrs`, `kind`, `visibility` ... and 3 others

error[E0609]: no field `inner` on type `types::Item`
   --> src/librustdoc/passes/collect_intra_doc_links.rs:691:78
    |
691 | ...                   trace!("considering associated item {:?}", assoc.inner);
    |                                                                        ^^^^^ unknown field
    |
    = note: available fields are: `source`, `name`, `attrs`, `kind`, `visibility` ... and 3 others

error: aborting due to 6 previous errors

For more information about this error, try `rustc --explain E0609`.
error: could not compile `rustdoc`

Sorry not to provide an MCVE.

Maybe the issue is just that rust-analyzer can't infer the types from the closures?

Yeah, probably an inference problem.

I don't think there is much to gain from this issue anymore, since the cause was most likely an inference problem which will be fixed(and found again) eventually.


Minimap markers are slightly offset

Hi,
Like in the topic. I'm finding markers moved slightly on minimap:

Turned on packs (the newest versions available):

Module is in the newest version. GPU: 6900XT on 22.11.2 drivers. Tried to reset module settings. Any ideas what might cause the issue?

Please share a screenshot of your overlay settings scrolled to the bottom. This is almost always caused by your UI size being set independently of what the game is set to and we don't support it being non-synced.


nixos/minecraft-server: let server shutdown cleanly

Description of changes
Due to the nature of how minecraft servers run, they do not shut down correctly with any shutdown signal, there are a few like SIGTERM that trigger a proper shutdown, but this uses a different shutdown logic compared to the stop command. Peculiarities of this shutdown type is that mods may not be shut down correctly and that all output after the SIGTERM signal is non-existent, so any errors at shutdown will not be logged.
Many solve this issue by wrapping their services in tmux or similar software, but this is highly inelegant as it requires systemd to be put into forking and systemd can no longer manage the process as clearly. While options such as GuessMainPID can help this, it is by no means a clean solution.
This PR implements a systemd socket together with a shutdown script that will send a stop command to the minecraft-server's standard input, and watch the service properly shut down. This is the most compatible way to shut down a minecraft server, and does not depend on any modifications of servers to be put in place for a proper shutdown sequence. It also takes in no new dependencies and uses only systemd native options for this, which is a bonus on simplicity.
A bonus, is that while the server is running, you can run echo "command" > /run/minecraft-server.stdin and run commands on the server without logging in or enabling rcon. Which can be great for admins and setting up backup solutions with for example running save-off and save-all on the server.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Do you mind if I (attempt) to add a test for this functionality (within the existing NixOS test)?

I added a simple systemctl stop command to the end of the test, I dont think anything more complicated and it should catch any errors that the shutdown logic would have.

Yeah no problem, squashed 👍

So it crashing may not have been the best description of the issue that is being solved. Rather, currently the minecraft-server will get a SIGTERM signal to ask to shut down, this is not cleanly handled by the minecraft-server.jar binary and will error out with an 143 that systemd will complain about. Furthermore, this unclean shutdown procedure may cause your world to not get saved properly, or at worst get corrupted. This is made worse if you add mods, as some only hook into the /stop command, and not also the SIGTERM trap inside of the java binary.

I’m seeing this not automatically restart the server when it goes down, I think because the socket config makes it a socket-activated service? Systemd seems to make no attempt to restart it, presumably waiting for something to come in on the fifo before activating the service. I’m thinking of trying to remove the systemd socket and ise systemd tmpfiles to manage the fifo, see if that works 🤔


Tracing config support

There's new switch in the awscli for update-function-configuration:
       --tracing-config (structure)
          The parent object that contains your function's tracing settings.

       Shorthand Syntax:

          Mode=string

       JSON Syntax:

          {
            "Mode": "Active"|"PassThrough"
          }

It's AWS X-Ray related and I would like to add support for it. Proposal is:
:defaults {:tracing :active}

:tracing can be used in :defaults and per lambda function
:tracing values can be :active or :passthrough
If :tracing is not provided, it defaults to :passthrough

It defaults to :passthrough, because it's a default value for all AWS Lambda functions (checked via awscli).
Okay?

Yeah, that's fine.  If it's possible to not emit a CLI arg in the case that the config doesn't specify :tracing, that would relax the requirement that all users have to update their CLI.  Not the end of the world either way.

I already did it here: https://github.com/zrzka/cljs-lambda/tree/tracing
I understand your issue with not emitting --tracing-config when it's not specified, but on the other side, if it will create synchronization & comparison mess in local vs remote comparison -> fire or not fire update-function-configuration. I'd rather fire it always. This is the same issue as it was with VPC, devs were forced to update awscli to use VPC. I don't see it as an issue, but your project, decide ;-)


Port previous Vim plugins

Create a process (probably tied to init) by which plugins that are already installed can be ported to apt-vim's vim_config.json.  This will make on-boarding much more pleasant.

Fixed by 1bcc911


chore: remove element template integration

related to https://camunda.slack.com/archives/GP70M0J6M/p1680015255693349

As we're touching the description, let's consider if we can make it a little bit more meaningful:

Shall we provide a link to a documentation page, i.e. https://docs.camunda.io/docs/next/components/modeler/data-handling/ (keeping it short and simple, in the spirit of https://docs.camunda.io/docs/next/components/modeler/about-modeler/)?

Shall we refer to Task or to job - workers? Or shall we try to avoid that term at all? Here is a proposal to avoid it entirely:

A third proposal:

Shall we provide a link to a documentation page, i.e.

I would only add it once we have the documentation page ready. As we don't have it yet, let's not add (and ship) something that might end up at another URL.

Or shall we try to avoid that term at all? Here is a proposal to avoid it entirely:

I really like the second proposal, it clearly states what it does and that it is optional

Let's go with the second proposal then. Do you update the PR? Changes look good to me.

adjusted the wording: cddbd61


Cleanup linkList component

this pull request is based on comments  from @KyleW in #128.  It removes an 'if' function that didn't need to be there (replaced with ng-if) and cleans up some extra code that I shouldn't have pushed anyways

👍


Refactor luigi pipeline to use luigid

Currently we have a lot of indirection due to the use of pyinvoke and are due to the fact that we don't invoke the pipeline tasks using luigi, but instead wrap them in our own run() method that in turns created a luigi.Worker and then schedules them.
We should instead refactor the pipeline to call luigi directly and use the luigid scheduler.

This has been implemented and is now part of master.


Run tests on GPU and fix thus found errors

In tests files, I move models to GPU when it is available, and fix errors that are discovered in GPU tests. These are all done after pulling the branch remove_pooling, which is currently a PR in review.
test_plenoptic.py and test_model.py did not use GPU when available, so I made them do so. test_display.py partially does, and I did not increase coverage of GPU because I think it's not necessary for all display functions. test_eigendistortion.py, test_metamers.py, test_mad.py and test_steerable_pyr.py were already set to used GPU when available. Most of these files produce some errors when tested on GPU, and I believe I have fixed all of them.

Codecov Report

Merging #117 (b21dd2c) into master (2405bac) will decrease coverage by 32.84%.
The diff coverage is 45.20%.

@@             Coverage Diff             @@
##           master     #117       +/-   ##
===========================================
- Coverage   76.55%   43.70%   -32.85%
===========================================
  Files          36       33        -3
  Lines        3463     2631      -832
  Branches      741      528      -213
===========================================
- Hits         2651     1150     -1501
- Misses        620     1364      +744
+ Partials      192      117       -75

Impacted Files
Coverage Δ

...noptic/simulate/canonical_computations/__init__.py
100.00% <ø> (ø)

plenoptic/simulate/models/__init__.py
100.00% <ø> (ø)

plenoptic/tools/optim.py
79.31% <ø> (+9.92%)
:arrow_up:

plenoptic/tools/signal.py
40.44% <0.00%> (-8.98%)
:arrow_down:

plenoptic/synthesize/mad_competition.py
9.97% <13.33%> (-76.22%)
:arrow_down:

plenoptic/synthesize/synthesis.py
49.15% <63.63%> (-28.01%)
:arrow_down:

plenoptic/synthesize/metamer.py
78.00% <100.00%> (-6.91%)
:arrow_down:

plenoptic/tools/display.py
27.90% <100.00%> (-51.09%)
:arrow_down:

plenoptic/metric/model_metric.py
28.57% <0.00%> (-71.43%)
:arrow_down:

... and 10 more

Continue to review full report at Codecov.

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

Yes, I think I need to wait for #114 and #115.

This looks good to me, I just have a couple of questions I'd like clarity on before approving.
Bigger picture question: how do we know how many of our tests are running on GPU? Is there an easy way to check that? I assume the errors that you caught were where half the tensors involved ended up on GPU and half on the CPU, but what about those (like test_models.py), where nothing was happening on the GPU? Is there an easy way to check that we've caught them all (or, like much of test_display.py, don't care)?

I basically went through all tests and moved models and data to DEVICE if they were not already there. It is not impossible that I have missed something. Another option is to write at the beginning of the file:
if torch.cuda.is_availble(): torch.set_default_tensor_type(torch.cuda.FloatTensor)
This will create all tensors on GPU when GPU is available, removing the need of writing .to(DEVICE) or device=DEVICE all the time, and also making sure every test is being run on GPU when applicable. But this can also break things when you create tensors that should be on CPU. Also, writing .to(DEVICE) or device=DEVICE was the norm in this project.

Hmm, I do like that idea of setting the default tensor type at the top of the tests, which will force every test onto the gpu. then, if there are tests we don't want to run on the gpu, we'd have to make that explicit, which is nice (so we know what's supported on both and which is cpu-only). We probably want to support both types of usage (if it's not too hard), since people will do both.

Actually, what I initially considered was applying this to all files, not just test files. I'm not yet sure about the behavior of torch.set_default_tensor_type across multiple files. If I could restrict its effect for the test files, then it would be more manageable for this PR. However, such a change would not totally remove the need of using DEVICE, since we call many external functions (e.g. po.load_images) that would not be covered by torch.set_default_tensor_type. So this change still would not force every test to GPU without possibility of human error. For this PR, I think I will keep the current solution.

Does setting default tensor type have global effect? If so then I don't support it since I don't want our package messing with other people's default workflows.

Yeah, I agree with both of you. Let's avoid using set_default_tensor_type then. Still do wish there were an issue way to see what we're testing on GPUs, but I guess there's no easy fix.
Before approving, I would still like the two threads I haven't resolved addressed.


EOL- allow logger to work after EOL (EXPOSUREAPP-14849)

[x] Allow logger after EOL

Ticket
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-14849

Is this related to the error logging feature in the app, which the user can access? Or is "logger" another logger in this case? :D

Is this related to the error logging feature in the app, which the user can access? Or is "logger" another logger in this case? :D

No, it is for testing purposes


Use MONGODB_CONNSTRING when MONGO_HOST is not set

Resolves #153.  When MONGO_HOST is not set, try to use MONGODB_CONNSTRING (basically falls back to the old method of specifying the MongoDB URI), and fail when neither is set.
@italovalcy, does this work for you?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

12 of 18 (66.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 77.316%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sdx_lc/utils/db_utils.py
12
18
66.67%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
982

Relevant Lines:
1213

💛  - Coveralls

This PR is to provide a somewhat backward compatible way for @italovalcy and team to use their MongoDB deployment.  If it were up to me, SDX-LC (and SDX Controller) would still be using SQLite, both in development and in production. ;-)
As far as I have understood, both services simply needs to keep a small amount of state, mostly to restore the state between restarts, and SQLite should be able to do this job quite well. SQLite is simple, well-tested, and should be adequate for our purposes and we have not run into any limitations of SQLite.  I have not quite understood the case for MongoDB, except maybe that there will be a MongoDB deployment anyway because Kytos needs it?  But now everything hinges on that MongoDB deployment being up and running, and that seems to be less resilient than the original approach.
https://sqlite.org/whentouse.html says that "SQLite works great as the database engine for most low to medium traffic websites (which is to say, most websites)".  There are enthusiastic proponents of server-side SQLite: https://fly.io/blog/all-in-on-sqlite-litestream/, https://unixdigest.com/articles/sqlite-the-only-database-you-will-ever-need-in-most-cases.html, https://blog.wesleyac.com/posts/consider-sqlite.

Resolves #153. When MONGO_HOST is not set, try to use MONGODB_CONNSTRING (basically falls back to the old method of specifying the MongoDB URI), and fail when neither is set.
@italovalcy, does this work for you?

Hi @sajith thanks a lot for providing this alternative for the configuration! It sounds great to me, @sajith !

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

12 of 18 (66.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 77.964%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sdx_lc/utils/db_utils.py
12
18
66.67%

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
1214

Relevant Lines:
1493

💛  - Coveralls


removed extra line

Thanks for the PR @dmatrix, tests in master should be fixed now & I'll retrigger the build on this PR :)


Keyless verification command line flags.

Description
Adds the command line flags necessary to allow the users to verify the
keyless signatures. The flags allow the user to check signatures created
within Github actions and any other keyless signature.
Fix #175
This PR depends on #169. Once it's merged, I'll rebase this one on top of main.

@jvanz you allowed maintainers to write to this branch, I took advantage of that to perform the following actions:

Rebase against main - the PR you built on top of has been merged
Fix e2e test failures

I'll let the unit tests run, then I'll merge it as soon as everything is green

@jvanz you allowed maintainers to write to this branch, I took advantage of that to perform the following actions:
* Rebase against `main` - the PR you built on top of has been merged

* Fix e2e test failures

I'll let the unit tests run, then I'll merge it as soon as everything is green

Thanks!


Script fails to connect to the server for gmail emails and other domain ids.

Without modifying anything in the code, it was verifying gmail addresses till yesterday but now it does not and says connection to the server failed. What could possibly be the reason of all of a sudden failing of script? I am using Wamp to run the script.
Another question: It was not checking other domains emails like webart.pk emails. any suggested solution for that?
Thanks in Advance

I just checked my ip is listed in 2 blacklists. It might be the reason. But I am not sure when it was working, why it did not check other domain ids like of webart.pk

I believe you have some blacklisting issue, you have to have your mail server configured correctly in order to avoid being blocked. See http://curlybrac.es/2016/07/27/how-to-configure-a-mail-server/

@hbattat I have my personal mail server created using Mail-in-a-box. It has everything setup correctly.
So, if I use this method, to verify emails of gmail, yahoo or top providers, is there any chance I will get blacklisted? (I won't be checking more than 200/day)


Process: maintainer involvement in triaging issues

This is a re-hash of the initial discussion in #46759.
Following some offline discussion with @nashif, we agreed that it would be worthwhile to start sending requests for action from maintainers to periodically triage open bugs and issues in parallel to the ongoing broader discussions about release criteria that turned into https://github.com/zephyrproject-rtos/zephyr/issues/38566.
To resolve in particular:

Triaging of issues, especially low priority issues: what to use? "known issue" label? "milestone" metadata?
Any other triaging for initial effort?

Process WG:
For the initial triage, first experiment, not limiting future work to this:

Ask maintainers to use 'future' milestone for bugs they likely won't be able to address this release
We (the TSC) will exclude these low priority bugs from our release metrics
We (the release managers) will clear the 'future' milestone from every bug after each release, to enforce policy that we revisit this
Preserve 50 low priority bugs (without future milestone) as release criterion target for v3.2

AIs:

@mbolivar-nordic update my scripts to generate these emails
@stephanosio to work in the long term about setting a bot up
@mbolivar-nordic to send the initial emails

Process WG: moved to done; TSC has approved


Show correct subject in exception message

The Assert:path currenly shows:
The path "/" is not absolute.

when it should say
The path "foo/bar" is not absolute.

I couldn't find any tests for this class .. I can create one if you want.

The class is currently only tested indirectly through other classes. If you create some, that's perfect! :)

Added tests for the two cases that I fixed.

Thanks! :)


Get Runners by Race

Find or scrape a list of runners competing in a given race.
(closes #5)

Coverage decreased (-5.9%) to 92.92% when pulling 303d9f4dcbd88105c063021160d04a4f90762235 on get_runners into 8e63e9d4bac6b03917b87dbd50fffd4b6370fe68 on master.


Fix scoped modules

see https://github.com/facebook/react-native/issues/8623
Background:
Back in a days, one buggy patch sneaked into rnpm master. It was intended to fix react-native-fbsdk installation for version 0.1.0 (not required in version 0.3.0). Nowadays, we see that this patch break scoped modules @like/this, so this PR fixes it (basically, reverting the patch).
Test plan (required)

[x] Try to link unscoped react-native package
[x] Try to link scoped react-native package
Code formatting

Look around. Match the style of the rest of the codebase. See also the simple style guide.
For more info, see the "Pull Requests" section of our "Contributing" guidelines.

By analyzing the blame information on this pull request, we identified @grabbou to be a potential reviewer.

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review internal test results.


[video_player] Some videos play without audio on Android TV devices

flutter doctor -v
> [√] Flutter (Channel dev, 2.5.0-5.0.pre, on Microsoft Windows [Version 10.0.18363.1734], locale en-US)
>     • Flutter version 2.5.0-5.0.pre at C:\flutter
>     • Upstream repository https://github.com/flutter/flutter.git
>     • Framework revision 0f465e5b2a (3 weeks ago), 2021-08-04 18:59:50 -0400
>     • Engine revision a0d89b1a54
>     • Dart version 2.14.0 (build 2.14.0-377.0.dev)
>
> [√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
>     • Android SDK at C:\Users\diyar\AppData\Local\Android\Sdk
>     • Platform android-30, build-tools 30.0.3
>     • ANDROID_HOME = C:\Users\diyar\AppData\Local\Android\Sdk
>     • ANDROID_SDK_ROOT = C:\Users\diyar\AppData\Local\Android\Sdk
>     • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
>     • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
>     • All Android licenses accepted.
>
> [√] Chrome - develop for the web
>     • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
>
> [√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.10.1)
>     • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
>     • Visual Studio Community 2019 version 16.10.31402.337
>     • Windows 10 SDK version 10.0.19041.0
>
> [√] Android Studio (version 4.1)
>     • Android Studio at C:\Program Files\Android\Android Studio
>     • Flutter plugin can be installed from:
>        https://plugins.jetbrains.com/plugin/9212-flutter
>     • Dart plugin can be installed from:
>        https://plugins.jetbrains.com/plugin/6351-dart
>     • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
>
> [√] IntelliJ IDEA Community Edition (version 2021.1)
>     • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2021.1.2
>     • Flutter plugin can be installed from:
>        https://plugins.jetbrains.com/plugin/9212-flutter
>     • Dart plugin can be installed from:
>        https://plugins.jetbrains.com/plugin/6351-dart
>
> [√] VS Code (version 1.55.2)
>     • VS Code at C:\Users\diyar\AppData\Local\Programs\Microsoft VS Code
>     • Flutter extension version 3.22.0
>
> [√] Connected device (4 available)
>     • MiTV AESP0 (mobile) • <IP_ADDRESS>:5555 • android-arm    • Android 9 (API 28)
>     • Windows (desktop)   • windows            • windows-x64    • Microsoft Windows [Version 10.0.18363.1734]
>     • Chrome (web)        • chrome             • web-javascript • Google Chrome 92.0.4515.159
>     • Edge (web)          • edge               • web-javascript • Microsoft Edge 92.0.902.78
>
> • No issues found!

<
video metadatas

System:FileName |
-- | --
System:FileSize | 10145812
System:FileModifyDate | 2021:08:27 11:53:07+00:00
System:FileAccessDate | 2021:08:27 11:53:07+00:00
System:FileInodeChangeDate | 2021:08:27 11:53:07+00:00
System:FilePermissions | 100644
File:FileType | M2T
File:FileTypeExtension | M2T
File:MIMEType | video/mpeg
M2TS:AudioStreamType | 3
M2TS:Duration | 33.28
H264:ImageWidth | 720
H264:ImageHeight | 576
MPEG:MPEGAudioVersion | 3
MPEG:AudioLayer | 2
MPEG:AudioBitrate | 96000
MPEG:SampleRate | 1
MPEG:ChannelMode | 1
MPEG:ModeExtension | 0
MPEG:CopyrightFlag | 1
MPEG:OriginalMedia | 1
MPEG:Emphasis | 0
Composite:ImageSize | 720 576
Composite:Megapixels | 0.41472
Composite:Duration

example

minimal code sample
import 'package:flutter/cupertino.dart';
import 'package:flutter/material.dart';
import 'package:video_player/video_player.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      home: MyVideoPlayer(),
    );
  }
}

class MyVideoPlayer extends StatefulWidget {
  const MyVideoPlayer({Key? key}) : super(key: key);
  @override
  _MyVideoPlayerState createState() => _MyVideoPlayerState();
}

class _MyVideoPlayerState extends State<MyVideoPlayer> {
  late final VideoPlayerController controller;

  @override
  void initState() {
    initVideo();

    super.initState();
  }

  void initVideo() {
    controller = VideoPlayerController.network(
        "mpegts url here");
    controller.addListener(() {
      listener();
    });
    controller.initialize().then((value) {
      controller.play();
      controller.setLooping(false);
    });
  }

  void listener() {
    if (mounted) {
      setState(() {});
    }
  }

  @override
  void dispose() {
    controller.removeListener(() {
      listener();
    });
    controller.dispose();
    debugPrint("Video Disposed");
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      backgroundColor: Colors.black,
      body: AspectRatio(
          aspectRatio: 16 / 9,
          child: !controller.value.isInitialized
              ? const Center(child: CircularProgressIndicator())
              : VideoPlayer(
                  controller,
                )),
    );
  }
}

console

I/ExoPlayerImpl(15144): Release cf8c8c3 [ExoPlayerLib/2.14.1] [aquaman, MiTV-AESP0, Xiaomi, 28] [goog.exo.core]
4
E/BufferQueueProducer(15144): [SurfaceTexture-0-15144-2] cancelBuffer: BufferQueue has been abandoned
W/ExoPlayer:Playb(15144): type=1400 audit(0.0:486): avc: denied { read } for name="u:object_r:media_prop:s0" dev="tmpfs" ino=2335 scontext=u:r:untrusted_app:s0:c144,c256,c512,c768 tcontext=u:object_r:media_prop:s0 tclass=file permissive=0
D/SurfaceUtils(15144): disconnecting from surface 0xe196b008, reason disconnectFromSurface
E/libc    (15144): Access denied finding property "media.metrics.enabled"
I/ExoPlayerImpl(15144): Init e0981b1 [ExoPlayerLib/2.14.1] [aquaman, MiTV-AESP0, Xiaomi, 28]
I/OMXClient(15144): IOmx service obtained
D/SurfaceUtils(15144): connecting to surface 0xe196b008, reason connectToSurface
I/MediaCodec(15144): [OMX.amlogic.avc.decoder.awesome] setting surface generation to 15507460
D/SurfaceUtils(15144): disconnecting from surface 0xe196b008, reason connectToSurface(reconnect)
D/SurfaceUtils(15144): connecting to surface 0xe196b008, reason connectToSurface(reconnect)
I/AmAVUtils(15144): AmAVUtils::getComponentRole isEncoder :0 mime:video/avc

Hi @diyargulli
Is the issue only on Android TV devices? Can you please reproduce it on Android TV emulator?
Also please provide a sample video for this issue to be reproducible
    controller = VideoPlayerController.network(
        "mpegts url here");

the issue exist on most of android boxes and android tv devices also on some older android phones:
url: http://<IP_ADDRESS>:25461/line/C4@!a3a1@!w72A/61
you may need to add  android:usesCleartextTraffic="true" to your android manifest file
controller = VideoPlayerController.network(
"http://<IP_ADDRESS>:25461/line/C4@!a3a1@!w72A/61");

emulator debug result
E/libc    ( 7074): Access denied finding property "vendor.display-size"
W/m.kurdapps.iqtv( 7074): type=1400 audit(0.0:59): avc: denied { read } for name="u:object_r:vendor_default_prop:s0" dev="tmpfs" ino=6816 scontext=u:r:untrusted_app:s0:c51,c256,c512,c768 tcontext=u:object_r:vendor_default_prop:s0 tclass=file permissive=0
I/ExoPlayerImpl( 7074): Init dde37b0 [ExoPlayerLib/2.12.1] [generic_x86, sdk_google_atv_x86, Google, 29]
I/flutter ( 7074): disposed
I/ExoPlayerImpl( 7074): Release abb83b2 [ExoPlayerLib/2.12.1] [generic_x86, sdk_google_atv_x86, Google, 29] [goog.exo.core, goog.exo.hls]
D/SurfaceUtils( 7074): disconnecting from surface 0xc0786008, reason disconnectFromSurface
I/m.kurdapps.iqt( 7074): Background young concurrent copying GC freed 5003(296KB) AllocSpace objects, 2(104KB) LOS objects, 2% free, 12MB/12MB, paused 464us total 258.290ms
I/m.kurdapps.iqt( 7074): Background concurrent copying GC freed 13500(1303KB) AllocSpace objects, 129(8668KB) LOS objects, 49% free, 2623KB/5247KB, paused 388us total 284.550ms
I/flutter ( 7074): mounted: true
I/m.kurdapps.iqt( 7074): NativeAlloc concurrent copying GC freed 2704(186KB) AllocSpace objects, 5(244KB) LOS objects, 49% free, 2325KB/4650KB, paused 393us total 171.092ms
I/OMXClient( 7074): IOmx service obtained
D/SurfaceUtils( 7074): connecting to surface 0xc9e51808, reason connectToSurface
I/MediaCodec( 7074): [OMX.google.h264.decoder] setting surface generation to 7243780
D/SurfaceUtils( 7074): disconnecting from surface 0xc9e51808, reason connectToSurface(reconnect)
D/SurfaceUtils( 7074): connecting to surface 0xc9e51808, reason connectToSurface(reconnect)
E/ACodec  ( 7074): [OMX.google.h264.decoder] setPortMode on output to DynamicANWBuffer failed w/ err -1010
I/ACodec  ( 7074): codec does not support config priority (err -1010)
I/OMXClient( 7074): IOmx service obtained
I/ACodec  ( 7074): codec does not support config priority (err -2147483648)
I/ACodec  ( 7074): codec does not support config operating rate (err -2147483648)
D/MediaCodec( 7074): [OMX.google.h264.decoder] setting dataspace on output surface to #104
D/SoftwareRenderer( 7074): setting dataspace on output surface to #104

i noticed that other hls video has codec isssue too even if sound work so if my video link not workded you can use another url instead

Hi @diyargulli
The link you provided are local, can't access your local videos, if possible, can yoiu please drop test video in the comment?
Can you please provide more details of your emulator, can you hear the audio?

Hi @diyargulli
The link you provided are local, can't access your local videos, if possible, can yoiu please drop test video in the comment?

https://user-images.githubusercontent.com/68226757/131494956-cafab0c6-ad31-484b-af7b-b441095a5bad.mp4

Can you please provide more details of your emulator, can you hear the audio?

the link is online but may not work at your location
i am dropping video but please note that we have problem with hls not with locals
also i  drop another url wich has sound but still **I/ACodec  ( 7074): codec does not support config operating rate (err -2147483648)
** message appears;
url:
https://svs.itworkscdn.net/rudawlive/rudawlive.smil/rudawtv_chunks.m3u8
if you add Add factory for Extractors it will be well :
https://github.com/google/ExoPlayer/issues/2748

Hi @diyargulli

i am dropping video but please note that we have problem with hls not with locals
also i drop another url wich has sound but still **I/ACodec ( 7074): codec does not support config operating rate (err -2147483648)
** message appears

I can confirm this message in a video with working on audio on Android TV emualtor

logs
[   +1 ms] Installing build\app\outputs\flutter-apk\app.apk...
[        ] executing: C:\Users\Taha\Code\android-sdk\platform-tools\adb.exe -s emulator-5554 install -t -r
C:\Users\Taha\StudioProjects\plugins-clone\packages\video_player\video_player\example\build\app\outputs\flutter-apk\app.
apk
[ +717 ms] Performing Streamed Install
                    Success
[   +1 ms] Installing build\app\outputs\flutter-apk\app.apk... (completed in 718ms)
[   +1 ms] executing: C:\Users\Taha\Code\android-sdk\platform-tools\adb.exe -s emulator-5554 shell echo -n
c55bc2d4857926b4a189854cebc1140dc34db5c6 > /data/local/tmp/sky.io.flutter.plugins.videoplayerexample.sha1
[  +33 ms] executing: C:\Users\Taha\Code\android-sdk\platform-tools\adb.exe -s emulator-5554 shell -x logcat -v time -t
1
[  +60 ms] --------- beginning of main
           09-01 07:32:21.285 I/Finsky  ( 4871): [2] rhs.n(8): Invalidating cached PackageState for
           io.flutter.plugins.videoplayerexample
[   +6 ms] executing: C:\Users\Taha\Code\android-sdk\platform-tools\adb.exe -s emulator-5554 shell am start -a
android.intent.action.RUN -f 0x20000000 --ez enable-background-compilation true --ez enable-dart-profiling true --ez
enable-checked-mode true --ez verify-entry-points true
io.flutter.plugins.videoplayerexample/io.flutter.embedding.android.FlutterActivity
[  +90 ms] Starting: Intent { act=android.intent.action.RUN flg=0x20000000
cmp=io.flutter.plugins.videoplayerexample/io.flutter.embedding.android.FlutterActivity (has extras) }
[        ] Waiting for observatory port to be available...
[ +996 ms] Observatory URL on device: http://<IP_ADDRESS>:45223/5l-6UBAo_ug=/
[   +1 ms] executing: C:\Users\Taha\Code\android-sdk\platform-tools\adb.exe -s emulator-5554 forward tcp:0 tcp:45223
[  +25 ms] 65248
[        ] Forwarded host port 65248 to device port 45223 for Observatory
[   +6 ms] Caching compiled dill
[  +27 ms] Connecting to service protocol: http://<IP_ADDRESS>:65248/5l-6UBAo_ug=/
[ +261 ms] Launching a Dart Developer Service (DDS) instance at http://<IP_ADDRESS>:0, connecting to VM service at
http://<IP_ADDRESS>:65248/5l-6UBAo_ug=/.
[ +132 ms] DDS is listening at http://<IP_ADDRESS>:65251/IEAqTLtvT1Q=/.
[  +60 ms] Successfully connected to service protocol: http://<IP_ADDRESS>:65248/5l-6UBAo_ug=/
[  +47 ms] DevFS: Creating new filesystem on the device (null)
[  +22 ms] DevFS: Created new filesystem on the device
(file:///data/user/0/io.flutter.plugins.videoplayerexample/code_cache/exampleYKCLPE/example/)
[   +2 ms] Updating assets
[  +89 ms] Syncing files to device sdk google atv x86...
[   +2 ms] <- reset
[        ] Compiling dart to kernel with 0 updated files
[   +2 ms] <- recompile package:video_player_example/main.dart c71d7722-c595-4af6-a3bc-0aae91f25f87
[        ] <- c71d7722-c595-4af6-a3bc-0aae91f25f87
[  +73 ms] Updating files.
[        ] DevFS: Sync finished
[        ] Syncing files to device sdk google atv x86... (completed in 79ms)
[        ] Synced 0.0MB.
[   +1 ms] <- accept
[   +5 ms] Connected to _flutterView/0xd6a7c410.
[   +2 ms] Flutter run key commands.
[   +1 ms] r Hot reload. 🔥🔥🔥
[   +1 ms] R Hot restart.
[        ] h List all available interactive commands.
[   +1 ms] d Detach (terminate "flutter run" but leave application running).
[        ] c Clear the screen
[        ] q Quit (terminate the application on the device).
[        ] 💪 Running with sound null safety 💪
[   +1 ms] An Observatory debugger and profiler on sdk google atv x86 is available at:
http://<IP_ADDRESS>:65251/IEAqTLtvT1Q=/
[ +228 ms] D/EGL_emulation( 7581): eglMakeCurrent: 0xe1ed8d20: ver 2 0 (tinfo 0xd9db84d0)
[   +1 ms] E/libc    ( 7581): Access denied finding property "vendor.display-size"
[   +2 ms] W/eoplayerexample( 7581): type=1400 audit(0.0:51): avc: denied { read } for
name="u:object_r:vendor_default_prop:s0" dev="tmpfs" ino=8398 scontext=u:r:untrusted_app:s0:c51,c256,c512,c768
tcontext=u:object_r:vendor_default_prop:s0 tclass=file permissive=0
[  +88 ms] W/eoplayerexampl( 7581): Accessing hidden method Landroid/media/AudioTrack;->getLatency()I (greylist,
reflection, allowed)
[  +12 ms] DevTools activation throttled until 2021-09-01 22:30:35.710110.
[   +5 ms] I/ExoPlayerImpl( 7581): Init 7ee25d0 [ExoPlayerLib/2.14.1] [generic_x86, sdk_google_atv_x86, Google, 29]
[  +83 ms] D/NetworkSecurityConfig( 7581): Using Network Security Config from resource network_security_config
debugBuild: true
[ +205 ms] D/EGL_emulation( 7581): eglMakeCurrent: 0xe1e3ff80: ver 2 0 (tinfo 0xd6a81740)
[ +270 ms] I/VideoCapabilities( 7581): Unsupported profile 4 for video/mp4v-es
[  +77 ms] I/OMXClient( 7581): IOmx service obtained
[   +7 ms] D/SurfaceUtils( 7581): connecting to surface 0xc295d008, reason connectToSurface
[        ] I/MediaCodec( 7581): [OMX.google.h264.decoder] setting surface generation to 7762945
[        ] D/SurfaceUtils( 7581): disconnecting from surface 0xc295d008, reason connectToSurface(reconnect)
[        ] D/SurfaceUtils( 7581): connecting to surface 0xc295d008, reason connectToSurface(reconnect)
[        ] E/ACodec  ( 7581): [OMX.google.h264.decoder] setPortMode on output to DynamicANWBuffer failed w/ err -1010
[   +6 ms] I/ACodec  ( 7581): codec does not support config priority (err -1010)
[  +22 ms] I/OMXClient( 7581): IOmx service obtained
[  +10 ms] I/ACodec  ( 7581): codec does not support config priority (err -2147483648)
[        ] I/ACodec  ( 7581): codec does not support config operating rate (err -2147483648)
[ +227 ms] D/MediaCodec( 7581): [OMX.google.h264.decoder] setting dataspace on output surface to #104
[        ] W/Gralloc3( 7581): allocator 3.x is not supported
[        ] D/eglCodecCommon( 7581): allocate: Ask for block of size 0x2f8000
[        ] D/eglCodecCommon( 7581): allocate: ioctl allocate returned offset 0x3ff50e000 size 0x2f9000
[  +94 ms] D/SoftwareRenderer( 7581): setting dataspace on output surface to #104
[ +151 ms] D/HostConnection( 7581): HostConnection::get() New Host Connection established 0xd9d578d0, tid 7633
[   +4 ms] D/HostConnection( 7581): HostComposition ext ANDROID_EMU_CHECKSUM_HELPER_v1 ANDROID_EMU_dma_v1
ANDROID_EMU_direct_mem ANDROID_EMU_host_composition_v1 ANDROID_EMU_host_composition_v2 ANDROID_EMU_vulkan
ANDROID_EMU_deferred_vulkan_commands ANDROID_EMU_vulkan_null_optional_strings
ANDROID_EMU_vulkan_create_resources_with_requirements ANDROID_EMU_YUV_Cache ANDROID_EMU_async_unmap_buffer
ANDROID_EMU_vulkan_free_memory_sync ANDROID_EMU_vulkan_shader_float16_int8 ANDROID_EMU_vulkan_async_queue_submit
ANDROID_EMU_sync_buffer_data GL_OES_vertex_array_object GL_KHR_texture_compression_astc_ldr
ANDROID_EMU_host_side_tracing ANDROID_EMU_async_frame_commands ANDROID_EMU_gles_max_version_2
[+2110 ms] The Flutter DevTools debugger and profiler on sdk google atv x86 is available at:
                    http://<IP_ADDRESS>:9101?uri=http://<IP_ADDRESS>:65251/IEAqTLtvT1Q=/
[+6082 ms] D/EGL_emulation( 7581): eglMakeCurrent: 0xe1e3ff80: ver 2 0 (tinfo 0xd6a81740)

Since you cannot provide the source which has the actual audio issue, please confirm if the same has audio on a Android TV emulator? Also your video in https://github.com/flutter/flutter/issues/89038#issuecomment-909157772 fail to load

Hi @diyargulli

i am dropping video but please note that we have problem with hls not with locals
also i drop another url wich has sound but still **I/ACodec ( 7074): codec does not support config operating rate (err -2147483648)
** message appears

I can confirm this message in a video with working on audio on Android TV emualtor
logs
Since you cannot provide the source which has the actual audio issue, please confirm if the same has audio on a Android TV emulator? Also your video in #89038 (comment) fail to load

yes after i uploaded it i noticed that don't work because it was live stream i tried to download endless video then i paused may be interrupted.
it's been one year that my clients have this issue so how to fix :(

this issue can be solved by using ffmpeg extension of exoplayer in android.

this issue can be solved by using ffmpeg extension of exoplayer in android.

Right you solved the for me thanks

this issue can be solved by using ffmpeg extension of exoplayer in android.

how can you add this to flutter project?


Can we install this pm confluent platform. it also has kafka-connect to sink/source from the salesforce streaming API.

Can we install this pm confluent platform. it also has kafka-connect to sink/source from the salesforce streaming API.

Can we install this package on  confluent platform. it also has kafka-connect to sink/source from the salesforce streaming API. How to integrate?

@holgeradam Can you please help with this issue.? whenever you have time.if it works and I can build documents on the saleaforce integrate and contribute back to nodefluent( I understand this is not related to confluent at all)
Thanks.

Got you !!
Thanks . I really appreciate your tine and effort.


add note about importing from another validator

Sorry, I had missed that. Thanks

Appreciate it, thank you! Hopefully that makes it easier to discover for others.


Make icons larger (similar to Google Now Launcher's icon).

I think I'll only do this on tablets... I don't really like the huge icons in GNL.

One of the already merged commits already do this.


Thank you for your extension plug-in, but in this update on July 31, the search box cannot be used!

Unable to enter after opening, closed or display this search box, input content cannot be searched. I like this extension very much, it is very helpful to me, please help solve it. And the console didn't report any errors.

It's just a cache issue, you need to clear the cache or keep refreshing the page

It's just a cache issue, you need to clear the cache or keep refreshing the page

Thanks. I'll give it a try

clear the cache or refresh the page didn't work
only InPrivate window is fine

clear the cache or refresh the page didn't work only InPrivate window is fine

I just tried it, and yes, I can only go back to the July 21 version for now.

It's not working correctly for me either after the update. The Search box is persistent on my screen even after closing the sidebar. It also makes my workflow area unclickable in and around the area where the sidebar would expand to. I can't click to drag anywhere near or under the persistent Search bar. Definitely need to roll this one back, sir.

The same,weird, I've had the same problem since the latest update.

It's just a cache issue, you need to clear the cache or keep refreshing the page

This solution is effective, please clean up the browser cache you comfyui using, and it will be displayed normally.

It's just a cache issue, you need to clear the cache or keep refreshing the page

This solution is effective, please clean up the browser cache you comfyui using, and it will be displayed normally.

I've done that. It has not resolved my issue. I'm using Microsoft Edge.

clear the cache or refresh the page didn't work only InPrivate window is fine
which proves my point xD

You can do Ctrl+Shift+J  to open the developer console, then go to the Network tab and enable "Disable cache" to see if that helps solve the problem.

@Nuked88 I did as you suggested, restarted my server, and the issue still persists. My side bar is empty.

I guess if I can't get this resolved, I could always fall back to that new built in stuff that Comfy added to the base app. I doubt that it's as nice as your setup, though.

I guess if I can't get this resolved, I could always fall back to that new built in stuff that Comfy added to the base app. I doubt that it's as nice as your setup, though.

If it's working in incognito/inPrivate mode, then is a cache issue and sooner or later the problem will disappear itself (I really don't know why is not disappearing right away after clearing the cache though 😭)

I've pushed a new update (8270725) , if this doesn't fix the cache issue, I'm not sure what will :(

Shift+F5
Shift+F5
Shift+F5

Shift+F5 Shift+F5 Shift+F5

I did that about ten times. It made no difference.

I've pushed a new update (8270725) , if this doesn't fix the cache issue, I'm not sure what will :(

Sadly, even after updating today, the Side Bar is still broken for me. It was all working fine before that update. 🤷🏼

I've pushed a new update (8270725) , if this doesn't fix the cache issue, I'm not sure what will :(

Sadly, even after updating today, the Side Bar is still broken for me. It was all working fine before that update. 🤷🏼
But, did you tried with another browser? Or in incognito mode?

I've pushed a new update (8270725) , if this doesn't fix the cache issue, I'm not sure what will :(

Sadly, even after updating today, the Side Bar is still broken for me. It was all working fine before that update. 🤷🏼

It is suggested to try another browser, such as the native Chrome.

Still unable to retrieve, then switched to the built-in browser on the Mac, but the search issue was not resolved. After clicking the refresh button next to the search several times in a row, the computer froze directly.

Ctrl+Shift+J

First of all, thank you very much for providing this plugin. It can be said that I have been using this plugin since I came into contact with ComfyUI, so personally, I am one of my favorite plugins. However, I still haven't solved the problem of the search function malfunctioning, and I am still looking forward to its continued use.

I had the same problem and since you said it was a cache issue, I first deleted the cached images and files and it did not work. Then I deleted the cache of all 4 items in the attached image, closed all the comfyui tabs and restarted comfyui and it worked fine. (You will be signed out of the site you are signed in to because you will be deleting your cookies).
Also, this custom node is the best!


Mocking SMS/SQS

I was curious if you had any strategies for using applications without AWS access.  Perhaps we could mock sms/sqs with a simple but functional esb in the backend.  I realized there exists issue  https://github.com/meducation/propono/issues/45 which discusses simple mocking of propono methods for testing, but I am more curious about setting up application environments for user and development testing.
Has anyone ended up trying out ActiveJob or similar for the purpose?

Found this issue which discusses, closing.
https://github.com/meducation/propono/issues/45

There's a package somewhere that mocks AWS. Can't remember what it's called but maybe @ccare, @mmmmmrob or @malcyL can?
Personally, I tend to have two test sets. One fully stubbed for unit testing and then one that does integration testing against a development AWS account.
Give me a little more info about your usecase and we can brainstorm.

#102 allows passing of alternative endpoints to the SNS and SQS clients. This could be used to use a local AWS alternative such as GoAWS to mock out AWS for testing.
Example config for Propono to use GoAWS:
client = Propono::Client.new do |config|
  config.aws_options = {
    endpoint:            'http://localhost:4100'
  }
end


Automatic update and bundled tool versions

We're using v2.104.2 on Win10 with VSTS, and it is unclear how the agent is updated.
Do we need to do this manually, or does it happen automatically?
This page hints the following:

The agent automatically updates itself soon after we release a new version.

Note: If your agent version is older than 102.1, then you must download and configure the agent to get the newest version.

Does that note mean that we need to upgrade manually?
We also noticed that the latest version 2.105.2 comes with NuGet <IP_ADDRESS>, which is outdated as 3.4.4 is already available, fixing an issue we have with .NETStandard PCL's.
Since our version is still stuck at 2.104.2, does it mean automatic update doesn't happen?

In pools web admin UI there is s context menu item to update the agent.  It might take a little bit as it sends a message to the agent which downloads to the side and updates itself
We've had a few bugs in that area but it has been working for the most part.  Let us know how it goes

@bryanmacfarlane It was a bit hidden, but we spotted the context menu (right-click) on the Default and Hosted agent pools on the Agent Queues in the VSTS Control Panel.
After clicking, it updated the build agent to v2.105.2.
It created an alias for the original folders in the install directory, with version specific folders per version. We were not aware that the agent would do this, and it would be helpful to indicate this auto-update behavior in the installation notes.
However, even the latest v2.105.2 is still using outdated NuGet 3.3, which is not able to compile PCL assemblies targeting .NET Standard. For that we need NuGet 3.4. It would be better if the Build Agent could support this, rather than us needing to configure a custom NuGet path in all our build definitions.

I second the update to the shipped NuGet.  Would it not be better to always ship with the latest?
https://github.com/Microsoft/vsts-agent/blob/3c6145febf80de375e103c2a247bec2d4b732802/src/Misc/externals.sh#L140
You could just alter the URL to https://dist.nuget.org/win-x86-commandline/latest/nuget.exe

Have logged a new issue #582


⚠ Fetchers has degraded performance

In 0246aa6, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 268 ms

Resolved: Fetchers performance has improved in 43b5408.


"unexpected character at line 1 column 1 of the JSON data"

ChatGPT (Web) gives the following error + high CPU usage when waiting for response. Claude and Gemini work normally.

I have the same issue on a regular basis. Sometimes clicking the reload button helps, but not always.


[Proposal] Secret Auto-Injection

In what area(s)?
The Area is about Secret Management.

/area runtime
/area operator

I create this proposal following my discussion with the Dapr Team.
Context:

I started to look at the DB Bindings such as the PostreSQL and saw it asks the Dev. to write SQL statements whereas any Dev. would prefer to use an ORM framework

Dapr purpose is not to recreate a new ORM which I can understand so then I started to think about Dapr to support any existing ORM:

For Java: JPA, Hibernate, integration with Spring Boot / spring-data-jpa
For MicroProfile : it means Jakarta EE / JTA integration with Dapr
For MicroProfile / Quarkus : it would mean integration with Panache, Hibernate & spring-data-jpa
Entity Framework for .Net
Django for Python

The challenge is that any ORM configuration will end up with a DB password in clear text or as an environment variable in a property file.

The purpose of this proposal is to find a way for Dapr to automatically inject that DB password secret or more generally any kind of secrets (there are hundreds of use case : trust store, key store, SMTP gateway, publish/subscribe passwords, etc.) into the App property without any code change at App level.
Spring Cloud already provides an abstraction layer of each Cloud Provider
relevant examples for Secrets Managers :
SDK & https://github.com/Azure-Samples/azure-spring-boot-samples/tree/spring-cloud-azure_v4.4.1/keyvault/spring-cloud-azure-starter-keyvault-secrets/property-source
https://github.com/GoogleCloudPlatform/spring-cloud-gcp/tree/main/spring-cloud-gcp-samples/spring-cloud-gcp-secretmanager-sample
Apps portability drawbacks: need to update the Maven Lib dependencies, rebuild & redeploy the project, so reversibility is OK but this is not truly directly portable ==> This is where I see Dapr could bring value to any Java framework
Having said that, When you have an existing Spring Boot Apps using Spring Cloud which already provides an abstraction layer of each Cloud Provider, adding Dapr seems too add more useless complexity, no more value
With Jakarta EE / Micro Profile / Quarkus / Other Java frameworks
I guess the only implementation way would to be to use the Cloud Provider SDK, so definitely not a portable App
Here are some scenarios :

Deploy the App on AKS with Azure DB for PostgreSQL + Key-Vault
Deploy the App on ACA with Azure DB for PostgreSQL + Key-Vault
Deploy the App on GCP through Azure Arc with :
Azure Arc-enabled PostgreSQL
Azure Key-Vault
==> benefits : 100% portable App (even with Spring Cloud without Dapr), no need to update the Maven Lib dependencies. Drawback : network latency : ping/pong between Azure & GCP
Deploy the App on GCP without Azure Arc with :
GCP Cloud SQL for PostgreSQL
GCP Secret-Manager
==> no more ping/pong between Azure & GCP, 100% portable App with Dapr

see also :

https://github.com/dapr/dapr/issues/3354
Dapr integration with Spring Cloud
Dapr integration with Quarkus

@artursouza
Describe the proposal

Basic use case example with Java Hibernate ORM
hibernate.connection.url=jdbc:mysql://mydb42.mysql.database.azure.com:3306/myapp?useSSL=true
hibernate.connection.username=adm_db
hibernate.connection.password=ThisIsaHiddenSecret777!

The Integration with Dapr could look like something simple like this:
apiVersion: v1
kind: ConfigMap
metadata:
  name: daprsecretcm
data:
  dapr.secret.auto-injection: enabled
  dapr.secret.auto-injection.bulk: disabled
  dapr.secret.auto-injection.secret-store-name: mystore
  dapr.secret.auto-injection.secret-name: hibernate.connection.password

Dapr would check 1 ConfigMap to auto inject Secrets at runtime: Ex:  hibernate.connection.password is not set in the App properties file but Dapr know that he has to get a secret referenced by ‘dapr.secret.auto-injection.secret-name’ in the secret store referenced by ‘dapr.secret.auto-injection.store-name’

High-Level Design ideation for Python / Django ORM integration
Ref doc
see this tutorial
The Integration with Dapr could look like something simple like this :
django.db.dapr.enabled=true
django.db.dapr.secrets.bulk.enabled=false
django.db.dapr.secret-store-name=mystore
django.db.dapr.secret-name=mysecret

DATABASES = {
    'default': {
        'ENGINE': 'django.db.backends.postgresql',
        'NAME': 'mydatabase',
        'USER': 'mydatabaseuser',
        'PASSWORD': ==> reference the 'django.db.dapr.secret-name'
        'HOST': '<IP_ADDRESS>',
        'PORT': '5432',
    }
}

Then the ORM framework would have to call the Dapr API /v1.0/secrets/myvault/mysecret instead of just reading the DATABASES.PASSWORD setting
If django.db.dapr.secrets.bulk.enabled=true then all DATABASES settings could be read from the Secret Store
Beyond Secret Management & Data Access, what would be the other impact & requirements regarding Dapr ?
In which other Use Case (other than DB Password) the framework would need to use a Secret in the config (not calling an API):

SMTP Gateway user/pasword config ?
Configuration-stores ?
Bindings ? Kafka, RabbitMQ, Storage buckets, etc.
PubSub.? Kafka, RabbitMQ, Azure Service Bus, event Hub, GCP PubSub
If you have Python ORM experience please share your thoughts here.

High-Level Design ideation for .Net / Entity Framework integration

Ref doc
https://learn.microsoft.com/en-us/ef/core/miscellaneous/connection-strings
PgSQL

To be checked:  In which other Use Case (other than DB Password) the framework would need to use a Secret in the config (not calling an API)
Beyond Secret Management & Data Access, what would be the other impact & requirements regarding Dapr :

Configuration-stores ?
Bindings ? Kafka, RabbitMQ, Storage buckets, etc.
PubSub.? Kafka, RabbitMQ, Azure Service Bus, event Hub, GCP PubSub
SMTP Gateway user/pasword config ?

If you have .Net EF ORM experience please share your thoughts here.

High-Level Design ideation for Java SE Frameworks integration

JPA
Hibernate
Spring Boot / spring-data-jpa

Say you store the here under secrets in your Secret Store, Azure KV in this case , then spring.datasource.url, spring.datasource.username and spring.datasource.password will be automatically injected from KV secrets SPRING_DATASOURCE_URL, SPRING_DATASOURCE_USERNAME and SPRING_DATASOURCE_PASSWORD secrets:

Secret Name
Secret Value example

MYSQL_SERVER_NAME
daprrocks777

MYSQL_SERVER_FULL_NAME
daprrocks777.mysql.database.azure.com

SPRING_DATASOURCE_URL
jdbc:mysql://daprrocks777.mysql.database.azure.com:3306/petclinic?useSSL=true&requireSSL=true&enabledTLSProtocols=TLSv1.2&verifyServerCertificate=true

SPRING_DATASOURCE_USERNAME
adm_sql

SPRING_DATASOURCE_PASSWORD
NoSugarNoStar777!!!

SPRING_CLOUD_AZURE_KEY_VAULT_ENDPOINT
https://kv-daprrocks.vault.azure.net/

SPRING_CLOUD_AZURE_TENANT_ID
PUT YOUR AZURE TENANT ID HERE

Today without Dapr, the DB Connection using secrets in KV looks like this:
Spring application.yml
spring:
  sql:
  jpa:
    open-in-view: false
    hibernate:
      ddl-auto: none # create-drop or none
    # https://docs.spring.io/spring-boot/docs/current/reference/html/howto.html#howto.data-initialization.using-basic-sql-scripts
    # https://facingissuesonit.com/2019/07/15/org-hibernate-hibernateexception-access-to-dialectresolutioninfo-cannot-be-null-when-hibernate-dialect-not-set
    # database-platform: org.hibernate.dialect.MySQL5InnoDBDialect
    # database-platform: org.hibernate.dialect.MySQLDialect
    database: mysql

---
# https://microsoft.github.io/spring-cloud-azure/current/reference/html/index.html#configuration-17
spring:
  cloud:
    azure:
      keyvault:
        secret:
          enabled: true
          property-source-enabled: true
          property-sources:
            - name: kv-cfg-vets # KV Config for each App Vets-Service
              endpoint: ${SPRING_CLOUD_AZURE_KEY_VAULT_ENDPOINT}
              credential:
                managed-identity-enabled: false
                client-id: ${VETS_SVC_APP_SPN_CLIENT_ID}
                client-secret: ${AZURE_CLIENT_SECRET} # client-secret for SPN ONLY not for MI
              # profile:
              #  tenant-id: ${SPRING_CLOUD_AZURE_TENANT_ID}
            - name: kv-cfg-visits # KV Config for each App Visits-Service
              endpoint: ${SPRING_CLOUD_AZURE_KEY_VAULT_ENDPOINT}
              credential:
                managed-identity-enabled: true
                client-id: ${VISITS_SVC_APP_IDENTITY_CLIENT_ID}
            - name: kv-cfg-customers # KV Config for each App Customers-Service
              endpoint: ${SPRING_CLOUD_AZURE_KEY_VAULT_ENDPOINT}
              credential:
                managed-identity-enabled: true
                client-id: ${CUSTOMERS_SVC_APP_IDENTITY_CLIENT_ID}

The Integration with Dapr could look like something simple like this :
spring.data.jpa.dapr.enabled=true
spring.data.jpa.dapr.secrets.bulk.enabled=false
spring.data.jpa.dapr.secret-store-name=mystore
spring.data.jpa.dapr.secret-name=mysecret

If spring.data.jpa.dapr.secrets.bulk.enabled=true then all properties including  spring.datasource.url, spring.datasource.username and spring.datasource.password would be automatically injected from KV secrets SPRING_DATASOURCE_URL, SPRING_DATASOURCE_USERNAME and SPRING_DATASOURCE_PASSWORD secrets
Then the ORM framework would have to call the Dapr API /v1.0/secrets/myvault/mysecret instead of just reading the spring.datasource.password property
If the above was implemented that would enable a Spring Boot App to leverage Dapr to use a Secret store to fetch the DB Password but anyway the App would still not be fully directly portable without a rebuild & deploy when switching from 1 Cloud Provider to another if specific dependencies such as spring-cloud-azure-starter-keyvault-secrets or spring-cloud-gcp-starter-secretmanager are still required in the project. In other words the only way to get rid of such Cloud Provider dependency would be :

Spring community to create a new Project Spring Cloud for Dapr
Spring Cloud for Dapr would have a new dependency io.dapr.secret-stores-dependencies

Drawback : significative work on Spring side, this would turn also into a Governance issue as today the different Spring Cloud projects are maintained by each Cloud Provider (ex: Spring Cloud for Azure is managed by MS / Java DevDiv)
Benefits : At the end no more x Spring Cloud projects , 1 & 1 only one Spring Cloud for Dapr project to maintain and most important customers would get a true 100% portable App Write once, run anywhere certified
For Hibernate config, see:

https://docs.jboss.org/hibernate/orm/6.1/quickstart/html_single/#hibernate-gsg-tutorial-basic-config
https://docs.jboss.org/hibernate/orm/6.1/userguide/html_single/Hibernate_User_Guide.html#database

hibernate.connection.url
hibernate.connection.username
hibernate.connection.password

To be checked:  In which other Use Case (other than DB Password) the framework would need to use a Secret in the config (not calling an API) for Spring frameworks:

mail.spring.mail.password
data.spring.couchbase.env.ssl.key-store-password
data.spring.couchbase.password
data.spring.couchbase.password
data.spring.data.cassandra.password
data.mongodb.password
data.spring.datasource.password
data.spring.elasticsearch.password
data.spring.h2.console.settings.web-admin-password
data.spring.influx.password
data.spring.ldap.embedded.credential.password
data.spring.neo4j.authentication.password
data.spring.r2dbc.password
data.spring.redis.password
data.spring.redis.sentinel.password
data-migration.spring.flyway.password
data-migration.spring.liquibase.password
data-migration.spring.sql.init.password
integration.spring.activemq.password
integration.spring.artemis.embedded.cluster-password
integration.spring.artemis.password
integration.spring.kafka.admin.ssl.key-password
integration.spring.kafka.admin.ssl.trust-store-password
integration.spring.kafka.consumer.ssl.key-store-password
integration.spring.kafka.consumer.ssl.trust-store-password
integration.spring.kafka.producer.ssl.key-password
integration.spring.kafka.producer.ssl.key-store-password
integration.spring.kafka.producer.ssl.trust-store-password
integration.spring.kafka.ssl.key-password
integration.spring.kafka.ssl.key-store-password
integration.spring.kafka.ssl.trust-store-password
integration.spring.kafka.streams.ssl.key-password
integration.spring.kafka.streams.ssl.key-store-password
integration.spring.kafka.streams.ssl.trust-store-password
integration.spring.rabbitmq.password
integration.spring.rabbitmq.ssl.key-store-password
integration.spring.rabbitmq.ssl.trust-store-password
integration.spring.rabbitmq.stream.password
server.server.ssl.key-password
server.server.ssl.key-store-password
server.server.ssl.trust-store-password
security.spring.security.user.password
rsocket.spring.rsocket.server.ssl.key-password
rsocket.spring.rsocket.server.ssl.key-store-password
rsocket.spring.rsocket.server.ssl.trust-store-password
actuator.management.endpoint.configprops.keys-to-sanitize
actuator.management.metrics.export.elastic.api-key-credentials
actuator.management.metrics.export.elastic.password
actuator.management.metrics.export.influx.password
actuator.management.metrics.export.kairos.password
actuator.management.metrics.export.prometheus.pushgateway.password
actuator.management.server.ssl.key-password
actuator.management.server.ssl.key-store-password
actuator.management.server.ssl.trust-store-password

Then Each other Spring Framework requires some Passwords / Secrets, ex:

Spring Cloud for Kubernetes
Spring Cloud for AWS
Spring Cloud for Alibaba
Spring Cloud for Azure
Spring Cloud for GCP
Spring Cloud Config
Spring Cloud Vault

Beyond Secret Management & Data Access, what would be the other impact & requirements regarding Dapr :

Configuration-stores ?
Bindings ? Kafka, RabbitMQ, Storage buckets, etc.
PubSub.? Kafka, RabbitMQ, Azure Service Bus, event Hub, GCP PubSub

If you have Java ORM experience please share your thoughts here.

@ezYakaEagle442 can you please put this in the https://github.com/dapr/proposals repository using the provided template?
Thanks!

I think this is a great opportunity to have Dapr inject secrets automatically in apps without requiring any code change in the app itself, not even taking a dependency on Dapr's SDKs.

I think this is a great opportunity to have Dapr inject secrets automatically in apps without requiring any code change in the app itself, not even taking a dependency on Dapr's SDKs.

I am not sure how we'd do that, however, in practice. For secrets to be injected in the app, we would need the Injector service to be able to access secret stores, with the related auth issues. And that would only work on K8s.

@ezYakaEagle442 can you please put this in the https://github.com/dapr/proposals repository using the provided template?
Thanks!

Hi @yaron2 I created https://github.com/dapr/proposals/pull/15
PS: thank you @artursouza :)


7.0.x testers needed!

I'm happy to announce version 7 of my library finally released! I've put a lot of work into this library so if you've found any errors feel free to edit the documentation or create a pull request. To learn more about contributing please look at the contribution page on this wiki. Have a great day!

Just upgraded from v6 to v7.
the "Calling Unity methods from React" and the "Communication from React to Unity"  page in the WIki confused me a bit.
One suggested using this.unityContent.send,   and the other suggested using this.unityContent.sendMessage.
I tested with both  this.unityContent.send,  is the one that works.
When I use this.unityContent.sendMessage.   it will get into errors saying this.unityContent.sendMessage.    is not a function.

Thanks for noticing! It should have been send instead of sendMessage. I've updated the docs, thanks! @ray-su

@jeffreylanters  Happy to chip in too! We use the library a lot an would love to help out where possible. Planning to start testing next week.
Thanks for the work you're putting in! 🙌

@alanionita that's great to hear! Thank you so much. I'd love to hear your feedback and experience. If you're missing any features or see improvements feel free to contribute. Have a great weekend!


Fix count_time bug when count_time aggregation using with other aggregation

Description

Codecov Report

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

@@            Coverage Diff            @@
##             master   #11112   +/-   ##
=========================================
  Coverage          ?   43.90%
  Complexity        ?    23472
=========================================
  Files             ?     2781
  Lines             ?   168789
  Branches          ?    20111
=========================================
  Hits              ?    74115
  Misses            ?    88369
  Partials          ?     6305

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


chore: Add resource requests/limits and GOPROXY.

I've noticed that these builds can cause problems with CPU usage on
nodes, since they had no request or limit, and that they're slower
fetching the go dependencies to build the BDD tests. So let's add
those things.
Signed-off-by: Andrew Bayer<EMAIL_ADDRESS>

/test tiller

/test tekton

/test boot-vault

/lgtm


[SPARK-13899][SQL] Produce InternalRow instead of external Row at CSV data source

What changes were proposed in this pull request?
https://issues.apache.org/jira/browse/SPARK-13899
This PR makes CSV data source produce InternalRow instead of Row.
Basically, this resembles JSON data source. It uses the same codes for casting.
How was this patch tested?
Unit tests were used within IDE and code style was checked by ./dev/run_tests.

cc @rxin @falaki

Test build #53179 has started for PR 11717 at commit 11dcc3d.

Test build #53180 has started for PR 11717 at commit 798eceb.

Test build #53179 has finished for PR 11717 at commit 11dcc3d.

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

Merged build finished. Test PASSed.

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

Test build #53180 has finished for PR 11717 at commit 798eceb.

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

Merged build finished. Test PASSed.

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

Test build #53189 has started for PR 11717 at commit 06bc187.

This PR would allow to infer TimestampType more flexibly (e.g. includeing T and GMT) rather than just using Timestamp.valueOf().

Test build #53189 has finished for PR 11717 at commit 06bc187.

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

Merged build finished. Test PASSed.

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

Thanks - I'm going to merge this. There is a tiny comment. Can you remove that comment in one of your other pr?

Does that mean closing https://github.com/apache/spark/pull/11550 and not supporting custom date format?


collapse blocks of code from closing curly braces

Hello,
It is a very practical feature to collapse code blocks from closing curly brace like PHPStorm.

this is a very practical feature, since i dont always want to fold my blocks using "fold" key shortcuts. With this feature i dont need to scroll up the beginning of my block every time i need to collapse it.

Meanwhile you may like to try this extension I wrote.
https://marketplace.visualstudio.com/items?itemName=georgejames.thefoldingstuff

Meanwhile you may like to try this extension I wrote.
https://marketplace.visualstudio.com/items?itemName=georgejames.thefoldingstuff

thanks a lot but this still does not provide what we mentioned. jet brains already did this for their editors as you can see below .

Hi @gjsjohnmurray,
When the cursor is located at the end of the block and I fold the block, the parent block will be folded!

So you want a upwards facing arrow at the bottom of a folding range as well as the top?
what should happen if there are 2 folding ranges that end/start on the same line?
∨   if {
        // code
∧   }

∨   if {
        // code
∧   } else {    // what should happen here?
        // code
∧  }

Hi @RedCMD,
If 2 blocks start/end on the same line, the priority would be with the closing curly bracket, like below:
if {...} else {

}

When the cursor is located at the end of the block and I fold the block, the parent block will be folded!

@msh-shf I suggest you right-click one line higher up then.
@Jacobify it isn't possible for extensions to achieve what you want, but the extra context menu items are a workaround.

/duplicate of #60670
@aeschli that issue links to a PR that has been languishing for several years. Can it get some love?

as much as i find the context menu great to use , i still feel like adding an arrow at the end of each block won't be asking for overkill . the down-side arrow is there folding/unfolding blocks, all is need is a up-side arrow doing the same thing only located at the end of the block

@gjsjohnmurray Hello there . Sorry to mention you here again ,but I just wanted to see if this issue is getting any progress ? Thanks in advance for your time

Duplicate of https://github.com/microsoft/vscode/issues/60670


Confusion about the "resync" of informers

This is a request to reopen the existing issue for further discussion. The issue pertains to the need for periodic resync in the work process of informers.

The work process of infomers that i figured out:

List all the resources according to the options given at first, and then initialize the indexer (local cache)
A Watch Loop watch the events of ADD,UPDATE,DELETE , put (obj, event) into delta_fifo.
Pop the (obj, event) from delta_fifo, sync the indexer(local cache) by the event, and distribute event to listeners which is interested in the events of these resources.

But i see the delta type: sync;
when doing sync(or called resync method) operation, the delta fifo gets all objects from indexer(local cache), and then re put them into delat fifo(if the obj is not in the fifo currently) and then trigger an update event to listeners.
If there is a risk that the client will lost some events, why not just sync them from api server? because i think the data source of indexer (local cache) is just from the delta fifo, whatwill we benefit from the periodic resync method?

This becomes even more interesting when we examine the typical informer's handlers, where we check events by their resource version. As a result, resync may become redundant(e.g. traefik)

Hi @stillya,
From your issue I see that the main questions you're trying to answer are (correct me if I'm wrong):

Why not just sync events from the API server?
What are the benefits of the periodic resyncing?
resync may become redundant* (not a question, but it's still here)

All these questions come down to the conceptual difference between caching and direct accessing:

Why not just sync events from the API server?

While it's possible to obtain the events via the Kube API (in fact, from etcd) there are number of reasons to avoid this:

cache contains already deserialized and decompressed data
requesting from etcd might be very time-consuming, especially in a large clusters or when dealing with the high event frequency
this increasing Kube API load, which may affect the overall health of the cluster

What are the benefits of the periodic resyncing?

cache is a redundancy* layer - if API is unreachable or if there're other net issues, cache ensures you still can operate using the cached data
cache reduces the load on the API server, allowing you to avoid costly operations

* - the thing is that redundancy in case of caching is a good thing, because it effectively allows you to avoid data loss, repeated requests and improves the overal performance.

Let me rephrase Brown's quote:
«It's better to be prepared for an issue and not have one, than to have an issue and not be prepared.»

@stillya I think your question is more around "Relist" vs "Resync". If so, this link should provide you with more clarity https://hex108.gitbook.io/kubernetes-notes/fu-lu-rtfsc/informer

Resync will put all the data of the indexer back into FIFO and trigger an UPDTA.
My question is: if this logic is designed to worry about errors during the first update processing, then there may be execution errors in the update, and ADD and Delete may also have this problem. Why did ADD and Delete not callback again.

This is a request to reopen the existing issue for further discussion. The issue pertains to the need for periodic resync in the work process of informers.

The work process of infomers that i figured out:

List all the resources according to the options given at first, and then initialize the indexer (local cache)
A Watch Loop watch the events of ADD,UPDATE,DELETE , put (obj, event) into delta_fifo.
Pop the (obj, event) from delta_fifo, sync the indexer(local cache) by the event, and distribute event to listeners which is interested in the events of these resources.

But i see the delta type: sync;
when doing sync(or called resync method) operation, the delta fifo gets all objects from indexer(local cache), and then re put them into delat fifo(if the obj is not in the fifo currently) and then trigger an update event to listeners.
If there is a risk that the client will lost some events, why not just sync them from api server? because i think the data source of indexer (local cache) is just from the delta fifo, whatwill we benefit from the periodic resync method?

This becomes even more interesting when we examine the typical informer's handlers, where we check events by their resource version. As a result, resync may become redundant(e.g. traefik)

Resync will put all the data of the indexer back into FIFO and trigger an update.
My question is: if this logic is designed to worry about errors during the first update processing, then there may be execution errors in the update, and ADD and Delete may also have this problem. Why did ADD and Delete not callback again.


new: use .zshenv by default

References:
https://zsh.sourceforge.io/Intro/intro_3.html

`.zshenv' is sourced on all invocations of the shell, unless the -f option is set. It should contain commands to set the command search path, plus other important environment variables.

Implementation in rustup:
https://github.com/rust-lang/rustup/blob/f26d16b3b940bfb7b5e5aad7ee8f7121569f1d60/src/cli/self_update/shell.rs#L186-L191

Thanks!


improvement: make mutation arguments non-null

As discussed in #105 and #110, put this behind an opt-in configuration to avoid breaking existing code.
The ID in update mutations is always non-null if non-null mutation arguments are allowed, while input is non-null if it's allowed and there is at least a non-null field in the input.
Document the newly added config variable in the getting started guide.
Contributor checklist

[x] Features include unit/acceptance tests

🚀 Thank you for your contribution! 🚀


Updated ReadMe with Translations section

Explanation of Bugfix/Feature/Modification:
Included a translation section within the ReadME file with a direct link to the files to aid non-coders.

@Leblox the best way to help with translations is actually to go here: https://crowdin.com/project/toolkit-for-ynab make an account and start adding translations there.
We pull all translations down from that site every time we deploy!


Removed postcss-html-loader and added simple postcss-loader

I'm merging because I need this changes for further changes. Maybe we can get back here later.


istioctl analyze should allow specifying a subset of analyzers by name

istioctl analyze should allow specifying a subset of analyzers by name, for testing, scoped debugging, or automated analysis.
To make this easy to use, it should also allow listing analyzers by name. Available analyzers could be listed in the usage message, or if that gets unwieldy a separate subcommand could print the full list of available analyzers.
The analyzers README.md should also be updated to reflect this option for testing new analyzers.
UX aspects of this discussed [here].(https://docs.google.com/document/d/1KuxDRPPt283jx_cIs2C9sFHjoHvoskJt3IGqhPJVHtM/edit)
Affected product area (please put an X in all that apply)
[X] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Additional context

Another important part of this: the structured validation message should output the name of the analyzer it was generated by. This will make it easier for users to hone in on the subset of analyzers to include (or, in the case of noisy false positives, exclude).

Should we also support exclusions by name?

Should we output, at runtime, which analyzers are run? (Could do that nifty green checkmark thing).

Possibly relevant:
https://docs.google.com/document/d/1r1hOBIwkDesZ8vYaHMj2YThEcUxegB-fLMPBst_yXIU/edit#heading=h.xw1gqgyqs5b


Use list for *requires in setup.py

https://github.com/pypa/setuptools/commit/f012485e4767f3be81493c93ad534a02c6f14f14

@caffe2bot retest this please


Login issues when multi-factor authentication is enabled

Storage Explorer Version: 1.1.0
Platform: Windows
Architecture: ia32
Build Number: 20180507.1
Commit: bcc8990d
Steps to Reproduce:

Remove your Azure account from the storge explorer
Enable multi-factor authentication on your Azure account
Re-add your account in the storage explorer

Expected Experience:
I should be prompted for my credentials and auth code one time and the storage explorer should stay logged in if I close/re-open it.
Actual Experience:

You need to enter your credentials and auth code at least 2 or 3 times before your login is validated (the authentication popup closes / re-open and you have to type in everything again).
Each time you close the explorer, the authentication status is lost and you have to re-renter your credentials and auth code (again 2 or 3 times). The explorer does not even tell you that you are not logged in anymore, you just can't expand the subscriptions anymore, without any message explaining why.

I think I am experiencing the same issue. After changing the email address as well as resetting the multi-factor authentication on my account, I can no longer log in. When I add an account, I repeatedly get prompted for authentication. I gave up after the 5th try.

Temporary workaround that we use in the meantime: do not use Azure Account to manage your subscription, add the storage account using connection string (name / access key). It's a bit painful when you have multiple accounts, multiple users with different authorization scopes, and you want to reproduce the access levels for each of them, but it works.

Indeed you only have to add MFA to your existing Azure Account and experience this behaviour.
I'll try the workaround with access key.

I also have this issue, have to log in three times for my MFA account, which is really annoying since I need to do two things per attempt (Authenticator App + SMS code)

same here with MFA enabled.

This is still present in version 1.13.0. Evidentally we will have to wait another year before any triage gets assigned to it. Does no one who uses Azure use 2FA? Really?

Logging in using device code flow works for me with a 2FA enabled account.

device

Thanks. Doesn't work

I'm experiencing the same issue. I repeatedly get prompted for authentication with MFA enabled. Version: 1.14.0

I am unable to sign into my account using the ASE v1.16.0. MFA is enabled for our organization. I sometimes do get prompted for the Authenticator Code but not always. But in both scenarios I get the error Message "Unable to sign in: interaction_required: interaction_required":
{
"message": ""interaction_required: interaction_required""
}


BIGTOP-2547: Don't render ip-hostname-check in hdfs-site.xml if default

Closed by https://github.com/apache/bigtop/commit/f1f5619a3a34e539a7b719712a885a26cdd161c4

Thanks Kevin and Cory!


how to listen keyDown event?

how to setting Fontsize?

You can use the mathfieldConfig-property to provide configuration for the underlying mathfield, like this:
<MathfieldComponent
  initialLatex="f(x)"
  mathfieldConfig={{
    onKeystroke: (mathfield, keystroke, event) => { /* whatever */ },
    style: {
      fontSize: 15,
    },
  }}
/>

You can see a complete documentation of all the configuration options here
Hope that helps.

Thank you for your answer. I find problem that ’style‘ attribute doesn't work.

That would be an issue with mathlive then

The style attribute is not a valid MathLive option (but maybe it's a mathlive-react attribute?). With MathLive, the way you would control the size of the mathfield is by styling the element containing the mathfield (i.e. setting the CSS font-size attribute of that element). I think this should also work with mathlive-react, though.


Please port polymer.js 0.1.2 to polymer.dart

This issue was originally filed by<EMAIL_ADDRESS>
Now, polymer.js release 0.1.2

This issue has been moved to dart-lang/polymer-dart#109.


Ignores in config.yml not excluded from watch command

I have setup some ignores (i have tried both ignore_files and a link to another file as per the example through ignores), but the supposedly ignored files are still updated and uploaded when changed and with a watch command running.
Is this the correct functionality?

No this is not correct functionality, Can you please show me your ignores and have you checked the ignores documentation http://shopify.github.io/themekit/ignores/

I have read the documentation and i have to be honest, the use of the hyphen (-) has thrown me a little.
I have tried the following
development:
  password: ...
  theme_id: "..."
  store: ....myshopify.com
  ignore_files:
    - config/settings_data.json

development:
  password: ...
  theme_id: "..."
  store: ....myshopify.com
  ignore_files:
  - config/settings_data.json

development:
  password: ...
  theme_id: "..."
  store: ....myshopify.com
  ignore_files:
    config/settings_data.json

Both that top two would be valid YAML. What version of themekit are you using and what operating system because I cannot reproduce this.

I have installed today and prompted to update to 0.6.9
I am on a OS X El Capitan v 10.11.4

Can you give me the steps you take to reproduce this? Are these correct?

Set the ignore for settings_data
Run theme watch in your project directory
Save config/settings_data.json
console outputs [development] Received Update event on config/settings_data.json

The process is correct. Here is a complete log from the console;
15:46:19 [development] Watching for file changes on host ....myshopify.com
15:46:25 [development] Received Remove event on config/settings_data.json___jb_old___
15:46:25 [development] Received Update event on config/settings_data.json___jb_tmp___
15:46:26 [development] Successfully performed Update operation for file config/settings_data.json___jb_tmp___ to ....myshopify.com
15:46:26 [development] Successfully performed Remove operation for file config/settings_data.json___jb_old___ to ....myshopify.com

Ah those seem to be temp files from your editor. You should probably something like this to your config.
ignore_files:
- *___jb_old___
- *___jb_tmp___

You know as soon as i posted that last response i realised it could be the IDE stepping in. [face-palm].
Thanks for your help!

No problem happy to help.


Idea: Register SS as an event provider with Adobe.IO

Have you considered trying to enable SS to act as an event provider for Adobe.IO webhooks?
I'm not quite sure of a concrete use case just yet but it would be good to conform to the Adobe.IO event model.

I thought about adding a communication_module to trigger Runtime functions, but that wouldn't interact much with the event gateway. Do you have any ideas on how to enable SS as an event provider directly?

hey @maxakuru I was thinking we have a look at the event proxy implementation for AEM as it looks like it's using a similar mechanism.


Moved: JetBrains.Writerside.EAP version 232.10275

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @SpecterShell,
Did JetBrains re-release the 2023.2 version as this version?

Hi @SpecterShell,
Did JetBrains re-release the 2023.2 version as this version?

This should be the same as https://github.com/microsoft/winget-pkgs/pull/132813. I am moving it because it is an EAP version and they haven't released stable versions of this product yet.


Fix typo in Readme

This is a great doc. Thanks for open sourcing it.

@barefootford Thanks for finding and fixing that!


.NET Core Bindings: Different project file name, assembly name and default namespace causes confusion

When different project file name, assembly name and default namespace used in a .NET Core project, discovery reports a missing test assembly.

This issue was caused by a wrong assembly rename (I did not rename it in a config file). With proper rename it works.


Numpy ImportError: cannot import name 'sliding_window_view' from 'numpy.lib.stride_tricks'

Doing following -
!pip install -U git+https://github.com/twopirllc/pandas-ta
import pandas_ta

Gives following error -
Collecting git+https://github.com/twopirllc/pandas-ta
...
...
Successfully installed pandas-ta-0.3.2b0
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-1-74be42ea8840> in <module>
      1 #!pip install pandas_ta
      2 get_ipython().system('pip install -U git+https://github.com/twopirllc/pandas-ta' class="ansi-blue-fg">)
----> 3 import pandas_ta

/opt/conda/lib/python3.7/site-packages/pandas_ta/__init__.py in <module>
    114 }
    115
--> 116 from pandas_ta.core import *

/opt/conda/lib/python3.7/site-packages/pandas_ta/core.py in <module>
     12
     13 from pandas_ta import Category, Imports, version
---> 14 from pandas_ta.candles.cdl_pattern import ALL_PATTERNS
     15 from pandas_ta.candles import *
     16 from pandas_ta.cycles import *

/opt/conda/lib/python3.7/site-packages/pandas_ta/candles/__init__.py in <module>
      1 # -*- coding: utf-8 -*-
----> 2 from .cdl_doji import cdl_doji
      3 from .cdl_inside import cdl_inside
      4 from .cdl_pattern import cdl_pattern, cdl, ALL_PATTERNS as CDL_PATTERN_NAMES
      5 from .cdl_z import cdl_z

/opt/conda/lib/python3.7/site-packages/pandas_ta/candles/cdl_doji.py in <module>
      1 # -*- coding: utf-8 -*-
----> 2 from pandas_ta.overlap import sma
      3 from pandas_ta.utils import get_offset, high_low_range, is_percent
      4 from pandas_ta.utils import real_body, verify_series
      5

/opt/conda/lib/python3.7/site-packages/pandas_ta/overlap/__init__.py in <module>
      4 from .ema import ema
      5 from .fwma import fwma
----> 6 from .hilo import hilo
      7 from .hl2 import hl2
      8 from .hlc3 import hlc3

/opt/conda/lib/python3.7/site-packages/pandas_ta/overlap/hilo.py in <module>
      2 from numpy import nan as npNaN
      3 from pandas import DataFrame, Series
----> 4 from .ma import ma
      5 from pandas_ta.utils import get_offset, verify_series
      6

/opt/conda/lib/python3.7/site-packages/pandas_ta/overlap/ma.py in <module>
      6 from .fwma import fwma
      7 from .hma import hma
----> 8 from .linreg import linreg
      9 from .midpoint import midpoint
     10 from .pwma import pwma

/opt/conda/lib/python3.7/site-packages/pandas_ta/overlap/linreg.py in <module>
      4 from numpy import nan as npNaN
      5 from numpy import pi as npPi
----> 6 from numpy.lib.stride_tricks import sliding_window_view
      7 from pandas import Series
      8 from pandas_ta.utils import get_offset, verify_series

ImportError: cannot import name 'sliding_window_view' from 'numpy.lib.stride_tricks' (/opt/conda/lib/python3.7/site-packages/numpy/lib/stride_tricks.py)

I tried install older version which use to work for me using following command but I assume it couldn't find any earlier version.
pip install 'pandas_ta<0.3.02b0' --force-reinstall
ERROR: Could not find a version that satisfies the requirement pandas_ta<0.3.02b0
ERROR: No matching distribution found for pandas_ta<0.3.02b0

Is it a bug in newer 0.3.02b version?
Am I using correct command to install older version?

Hello @lakhanisagara,
Please see Issue #285.
Kind Regards,
KJ

Hello @lakhanisagara,
I assume by no response that the solution provided was sufficient. Thus I will be closing this issue in a few days.
Kind Regards,
KJ

This issue still persists on google colab, due to dependencies...
ERROR: tensorflow 2.5.0 has requirement numpy~=1.19.2, but you'll have numpy 1.21.0 which is incompatible.
ERROR: tensorflow 2.5.0 has requirement six~=1.15.0, but you'll have six 1.16.0 which is incompatible.
ERROR: google-colab 1.0.0 has requirement pandas~=1.1.0; python_version >= "3.0", but you'll have pandas 1.3.0 which is incompatible.
ERROR: google-colab 1.0.0 has requirement six~=1.15.0, but you'll have six 1.16.0 which is incompatible.
ERROR: datascience 0.10.6 has requirement folium==0.2.1, but you'll have folium 0.8.3 which is incompatible.
ERROR: albumentations 0.1.12 has requirement imgaug<0.2.7,>=0.2.5, but you'll have imgaug 0.2.9 which is incompatible.
Proceed (y/n)? y
ERROR: tensorflow 2.5.0 has requirement numpy~=1.19.2, but you'll have numpy 1.21.0 which is incompatible.
ERROR: tensorflow 2.5.0 has requirement six~=1.15.0, but you'll have six 1.16.0 which is incompatible.
ERROR: google-colab 1.0.0 has requirement pandas~=1.1.0; python_version >= "3.0", but you'll have pandas 1.3.0 which is incompatible.
ERROR: google-colab 1.0.0 has requirement six~=1.15.0, but you'll have six 1.16.0 which is incompatible.
ERROR: datascience 0.10.6 has requirement folium==0.2.1, but you'll have folium 0.8.3 which is incompatible.
ERROR: albumentations 0.1.12 has requirement imgaug<0.2.7,>=0.2.5, but you'll have imgaug 0.2.9 which is incompatible.


clientv3: support context cancellation on lease keep alives

LGTM. We probably should add testing around this change. (cancellation, multiple keepalive is called on one lease id).

I'm porting the recipes to use more of clientv3; I think it stresses all of those cases.

@heyitsanthony Awesome. Do not worry about the test then, I will add unit test for it.