nikolajlovenhardt/laravel-google-ads 1.2.3 requires googleads/googleads-php-lib ~31.0.0 -> satisfiable by googleads/googleads-php-lib[31.0.0] in Laravel 5.5

When running this command:
composer require nikolajlovenhardt/laravel-google-ads
In Laravel 5.5 and PHP 7.0 development

Hi @kennblvnp  Would you please include some more of the error message

@nikolajlovenhardt
Do not run Composer as root/super user! See https://getcomposer.org/root for details
Using version ^1.2 for nikolajlovenhardt/laravel-google-ads
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - nikolajlovenhardt/laravel-google-ads 1.2.3 requires googleads/googleads-php-lib ~31.0.0 -> satisfiable by googleads/googleads-php-lib[31.0.0].
    - nikolajlovenhardt/laravel-google-ads 1.2.4 requires googleads/googleads-php-lib ~31.0.0 -> satisfiable by googleads/googleads-php-lib[31.0.0].
    - Conclusion: don't install googleads/googleads-php-lib 31.0.0
    - nikolajlovenhardt/laravel-google-ads 1.2.2 requires googleads/googleads-php-lib ~30.0.0 -> satisfiable by googleads/googleads-php-lib[30.0.0].
    - Conclusion: don't install googleads/googleads-php-lib 30.0.0
    - nikolajlovenhardt/laravel-google-ads 1.2.1 requires googleads/googleads-php-lib ~29.0.0 -> satisfiable by googleads/googleads-php-lib[29.0.0].
    - Conclusion: don't install googleads/googleads-php-lib 29.0.0
    - nikolajlovenhardt/laravel-google-ads 1.2.0 requires googleads/googleads-php-lib ~28.0.0 -> satisfiable by googleads/googleads-php-lib[28.0.0].
    - Conclusion: remove phpdocumentor/reflection-docblock 4.2.0
    - Installation request for nikolajlovenhardt/laravel-google-ads ^1.2 -> satisfiable by nikolajlovenhardt/laravel-google-ads[1.2.0, 1.2.1, 1.2.2, 1.2.3, 1.2.4].
    - Conclusion: don't install phpdocumentor/reflection-docblock 4.2.0
    - googleads/googleads-php-lib 28.0.0 requires phpdocumentor/reflection-docblock ^3.0.3 -> satisfiable by phpdocumentor/reflection-docblock[3.0.3, 3.1.0, 3.1.1, 3.2.0, 3.2.1, 3.2.2, 3.2.3, 3.3.0, 3.3.2].
    - Can only install one of: phpdocumentor/reflection-docblock[3.0.3, 4.2.0].
    - Can only install one of: phpdocumentor/reflection-docblock[3.1.0, 4.2.0].
    - Can only install one of: phpdocumentor/reflection-docblock[3.1.1, 4.2.0].
    - Can only install one of: phpdocumentor/reflection-docblock[3.2.0, 4.2.0].
    - Can only install one of: phpdocumentor/reflection-docblock[3.2.1, 4.2.0].
    - Can only install one of: phpdocumentor/reflection-docblock[3.2.2, 4.2.0].
    - Can only install one of: phpdocumentor/reflection-docblock[3.2.3, 4.2.0].
    - Can only install one of: phpdocumentor/reflection-docblock[3.3.0, 4.2.0].
    - Can only install one of: phpdocumentor/reflection-docblock[3.3.2, 4.2.0].
    - Installation request for phpdocumentor/reflection-docblock (locked at 4.2.0) -> satisfiable by phpdocumentor/reflection-docblock[4.2.0].

Installation failed, reverting ./composer.json to its original content.

As you can read from the log - my package requires a specific version of reflection-docblock, but you already depend on another version of the same package.
"Can only install one of: phpdocumentor/reflection-docblock[3.3.2, 4.2.0]."
You need to fix this, so that both of the packages depend on the same version.

I had this problem also @kennblvnp
I got around it by installing 3.0.3 of phpdocumentor/reflection-docblock first
"phpdocumentor/reflection-docblock": "3.0.3",

and THEN
composer require nikolajlovenhardt/laravel-google-ads

What I ended up with was
"phpdocumentor/reflection-docblock": "3.0.3",
"nikolajlovenhardt/laravel-google-ads": "^1.2"

The composer.json of the googleads/googleads-php-lib package requires this
"phpdocumentor/reflection-docblock": "^4.0.0 || ^3.0.3",

Perhaps that will help you.

thanks @mmccaff !


gnuradio: remove volk cmake flags mistakenly added

Motivation for this change
see https://github.com/NixOS/nixpkgs/pull/118345#discussion_r606706207
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

The commit message should be IMO:
gnuradio: Remove volk cmake flags mistakenly added.

Done


Allow to specify the scope for a PIM activation

Welcome

[X] Yes, I've searched similar issues on GitHub and didn't find any.

Is your feature request related to a problem? Please describe
In my case in need to specify the scope of a PIM activation request.
Describe the solution you'd like
Allow to specify the scope as command line flag.
Describe alternatives you've considered

Additional context
No response

Please elaborate. What do you mean by scope? Are you thinking of the "Justification" (i.e. --reason in the CLI for az-pim-cli)? For the activation request itself (az-pim-cli activate), the following fields are available in the API: https://github.com/netr0m/az-pim-cli/blob/e6d96fc864f0b8cb5bad5d53cceb6fa094937cd7/pkg/pim/models.go#L170-L180

The scope is automatically selected, as this software (az-pim-cli) gets a list of roles and the subscription to which each belongs from the PIM API. The subscription is specified using either --name <SUBSCRIPTION_NAME> or --prefix <PREFIX_OF_SUBSCRIPTION_NAME>, and the role is specified using --role <ROLE_NAME>. The subscription (--name / --prefix) is always required. If --role is absent, the software will automatically select the first matching role in the list.
If you list roles, i.e. az-pim-cli list, you will get something like the following format:
$ az-pim-cli list
== SUBSCRIPTION_NAME_1 ==
         - ROLE_NAME_1
         - ROLE_NAME_2
== SUBSCRIPTION_NAME_2 ==
         - ROLE_NAME_1

for instance,
$ az-pim-cli list
== S100-Example-Subscription ==
         - Contributor
         - Owner

where you can activate the "Owner" role with az-pim-cli --name S100-Example-Subscription --role Owner, or the first match (here: "Contributor") with az-pim-cli --name S100-Example-Subscription.
As for the "Ticket system" and "Ticket number", those were recently implemented in #56 and is currently being released as part of v1.1.0
The "reason" is already supported by using the --reason <REASON> flag.

@netr0m Thanks for the explanation and the fix in #56 . I just tested it again and this time it worked as expected.


Create docs in GitHub repo

The docs should be placed in /docs/index.md in each version's branch.
Versions:

[x] Version 1
[x] Version 2
[x] Version 3

Closing this, in favor of #39


How to resize the calendar?

I couldnt find any function or setting to resize the calendar widget. I only have small space for the calendar and I need to resize it. Is there any way to do that? Thank you

Hello @ivder sorry for the late reply.. evocalendar is responsive but in your case, you can just customize the css file like adjust the sidebar's width and so.. :slightly_smiling_face:


Is License BSD or Apache 2

Hi,
In the pom.xml the license is specified as being a BSD license ( 3 clause ) but the LICENSE file is an Apache 2 license.
Which one did you choose ?

Apache, the file headers are correct, good eyes :-)
I have a PR out for the change (https://github.com/groupon/sparklint/pull/23).

cool thx, I'll update the build.sbt file I'm crafting


Scrollable Widget

Got the custom scroll view example from flutter website.
Played a little bit and (maybe) found a bug.
When I comment Silverlist (Top) out, everything works. But when I try to do it the other way round, the app crashes with an internal error.
Error:

Code:

If you pass center: centerKey to the CustomScrollView, it is an error not to actually include a child with that key: centerKey.

@KattlerJoe Can you please provide a completed and minimal reproducible code sample that doesn’t include 3rd party plugins or complex production code so that we may verify this? Also, please provide the output of flutter doctor -v as well. Thanks!


[CLIP ENCODER] Vision Transform for Clip encoder

Context
What is the purpose of this PR? Is it to

[x] add a new feature
[ ] fix a bug
[ ] update tests and/or documentation
[ ] other (please add here)

WIP
Changelog

LayerNorm
Vision Transform
CLIP specific positional embeddings that supports tiled images

Test plan
TBD

[ ] run pre-commit hooks and linters (make sure you've first installed via pre-commit install)
[ ] add unit tests for any new functionality
[ ] update docstrings for any new or updated methods or classes
[ ] run unit tests via pytest tests
[ ] run recipe tests via pytest tests -m integration_test
[ ] manually run any new or modified recipes with sufficient proof of correctness

[ ] include relevant commands and any other artifacts in this summary (pastes of loss curves, eval results, etc.)

Codecov Report
Attention: Patch coverage is 31.37255% with 210 lines in your changes missing coverage. Please review.

Project coverage is 26.18%. Comparing base (15c918d) to head (4a77d69).
Report is 6 commits behind head on main.

Files
Patch %
Lines

torchtune/modules/vision_transformer.py
16.48%
76 Missing :warning:

tests/torchtune/modules/test_vision_transformer.py
27.69%
47 Missing :warning:

torchtune/models/clip/_position_embeddings.py
27.27%
32 Missing :warning:

...orchtune/models/clip/test_positional_embeddings.py
32.50%
27 Missing :warning:

torchtune/models/clip/_component_builders.py
33.33%
14 Missing :warning:

tests/torchtune/modules/test_layernorm.py
66.66%
8 Missing :warning:

torchtune/modules/layer_norm.py
44.44%
5 Missing :warning:

torchtune/models/clip/_model_builders.py
83.33%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1127      +/-   ##
==========================================
- Coverage   26.74%   26.18%   -0.56%
==========================================
  Files         183      195      +12
  Lines        8362     9166     +804
==========================================
+ Hits         2236     2400     +164
- Misses       6126     6766     +640

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


Uncaught RemoteDisconnected Exception

We've got an application that uploads files to the GCS API, in our code we're catching a bunch of Google Cloud exceptions, however we're still seeing some failures.  It looks as though this library isn't catching exceptions coming from httplib2 and reraising them as Google Cloud exceptions.
A try except around the request here catching the base HTTP exception here seems like the best way to fix this?
//edit - Adding stacktrace.
RemoteDisconnected: Remote end closed connection without response
  File "fbq/main.py", line 255, in run_manage_file
    return self.manage_file(filename)
  File "fbq/main.py", line 270, in manage_file
    self.push_to_gcs(zipped_file)
  File "fbq/main.py", line 153, in push_to_gcs
    gcs_blob.upload_from_filename(file_name, content_type='application/gzip')
  File "opt/fbq-virtualenv/lib/python3.6/site-packages/google_cloud_storage-1.0.0-py3.6.egg/google/cloud/storage/blob.py", line 671, in upload_from_filename
    file_obj, content_type=content_type, client=client)
  File "opt/fbq-virtualenv/lib/python3.6/site-packages/google_cloud_storage-1.0.0-py3.6.egg/google/cloud/storage/blob.py", line 622, in upload_from_file
    http_response = upload.stream_file(use_chunks=True)
  File "opt/fbq-virtualenv/lib/python3.6/site-packages/google_cloud_core-0.24.1-py3.6.egg/google/cloud/streaming/transfer.py", line 1094, in stream_file
    response = send_func(self.stream.tell())
  File "opt/fbq-virtualenv/lib/python3.6/site-packages/google_cloud_core-0.24.1-py3.6.egg/google/cloud/streaming/transfer.py", line 1223, in _send_chunk
    return self._send_media_request(request, end)
  File "opt/fbq-virtualenv/lib/python3.6/site-packages/google_cloud_core-0.24.1-py3.6.egg/google/cloud/streaming/transfer.py", line 1133, in _send_media_request
    self.bytes_http, request, retries=self.num_retries)
  File "opt/fbq-virtualenv/lib/python3.6/site-packages/google_cloud_core-0.24.1-py3.6.egg/google/cloud/streaming/http_wrapper.py", line 384, in make_api_request
    redirections=redirections)
  File "opt/fbq-virtualenv/lib/python3.6/site-packages/google_cloud_core-0.24.1-py3.6.egg/google/cloud/streaming/http_wrapper.py", line 347, in _make_api_request_no_retry
    redirections=redirections, connection_type=connection_type)
  File "google_auth_httplib2-0.0.2-py3.6.egg/google_auth_httplib2.py", line 198, in request
    uri, method, body=body, headers=request_headers, **kwargs)
  File "httplib2/__init__.py", line 1322, in request
    (response, content) = self._request(conn, authority, uri, request_uri, method, body, headers, redirections, cachekey)
  File "httplib2/__init__.py", line 1072, in _request
    (response, content) = self._conn_request(conn, request_uri, method, body, headers)
  File "httplib2/__init__.py", line 1025, in _conn_request
    response = conn.getresponse()
  File "raven/breadcrumbs.py", line 353, in getresponse
    rv = real_getresponse(self, *args, **kwargs)
  File "http/client.py", line 1331, in getresponse
    response.begin()
  File "http/client.py", line 297, in begin
    version, status, reason = self._read_status()
  File "http/client.py", line 266, in _read_status
    raise RemoteDisconnected("Remote end closed connection without"

Apologies this is more https://github.com/httplib2/httplib2 that's leaking exceptions than this.  Just seen you're catching that base error.

I'm happy to accept a PR to re-map that error as well.

Yeah, it's Friday evening here so the pace is pretty slow.  PR done, should fix but really this is something for httplib2 to be catching.  I need to look at it with a fresh head on Monday as I'm totally spent today.
This should fix in the meanwhile though.


Solved the issue #585

I have solved the bug #585 and i have added the justification in the paragraph and after doing some more changes it's resolved thanks
Related Issue

Mention issue for which this pull request is a part of (please mention issue number # )

Proposed Changes

Change 1
Chnage 2

Additional Info

Any additional information or context

Checklist

[ ] ✅ My code follows the code style of this project.
[ ] 📝 My change requires a change to the documentation.
[ ] 🎀 I have updated the documentation accordingly.
[ ] 🌟 ed the repo

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Output Screenshots

Screenshot #1
Screenshot #2

Title goes here
Title goes here

Image goes here
Image goes here

check its now working properly


Multiple instances on one redis service

I found it to impossible to set cache prefix which allows for only one instance of Tus running on one server. The ability to modify cache key prefix would allow for using multiple Tus server instances with single redis service running

You can configure cache during server initialization.
$redis = new \TusPhp\Cache\RedisStore([
    // Connection parameters.
]);

$redis->setPrefix('cache_prefix_');

$server = new \TusPhp\Tus\Server($redis);

Nice,
how would you acomodate it to Laravel integration scheme?

I've tried changing the TusServiceProvider with:
$redis = new \TusPhp\Cache\RedisStore();
$redis->setPrefix(env('APP_NAME') . ":tus");
$server = new TusServer($redis);

But no luk there. Still it uses it's own key pattern.
I've noticed that second app running on the same server uses redis with index 2 tus2:className but i cant find why that's the case.

@MichalKrakow can you please let us know how can we reproduce this issue? Clear steps on what you did, what was expected and what you got instead would be helpful.


Request for reset stats and skill points buttons

I got extremely used to playing with PlugY mainly because of the extremely easy access to buttons to reset stats and skill points since I love experimenting with different builds as I level up in the game. Would love to have this feature in this editor.

Did you know that if you add "-enablerespec" in your D2R launch options that you get unlimited respec in game by simple doing and ALT-LeftClick on the strength plus (+) button in the character screen in game? This is a core feature that the Blizzard developers added in D2R for offline; no mod or hack required.


Controls between 2.6 and 2.9 of CIS v2.0.0 doesn't work with the GCP project configured as the instruction in the documantation.

Controls between 2.6 and 2.9 of CIS v2.0.0 report alert on the GCP projects even after they are configured as the Remediation section say.

https://hub.steampipe.io/mods/turbot/gcp_compliance/controls/control.cis_v200_2_6
https://hub.steampipe.io/mods/turbot/gcp_compliance/controls/control.cis_v200_2_7
https://hub.steampipe.io/mods/turbot/gcp_compliance/controls/control.cis_v200_2_8
https://hub.steampipe.io/mods/turbot/gcp_compliance/controls/control.cis_v200_2_9

The reason is that Logging Query Language written in their documentation has ( and ) but the queries used by their Control don't care them.
I think that this is a bug of Queries and the documentations look correct.

Hi @k-kawa Sorry for the delay, and thank you for raising this issue. We will investigate it and update you as soon as possible.

Resolved in https://github.com/turbot/steampipe-mod-gcp-compliance/pull/154


Autocomplete inputs?

I tried to use awesomplete with redux-form and ran into a lot of problems mainly in the fact the jquery setting the value of the field would not trigger a state change and so if you clicked away or did anything else the selected autocomplete value would not remain, does anyone have any experience with this?

Mixing React and jQuery is a recipe for trouble, in my experience. React really doesn't like it when you manipulate the DOM out from under it.

I have used react-select for autocomplete fields. It plays nicely with redux-form.

As Erik said, you should use a component to achieve this. The problem when using jQuery (or any DOM library) is that React does not operate directly on the DOM but on a virtual representation on the desired DOM tree. If something modify this tree then React won't be able to update the DOM.
react-select and react-selectize work both well with redux-form.

Hey sorry to bother you guys but I'm trying to use react-select with my form and it keeps getting cleared onBlur, can't really figure out why? Are there any examples of redux-form and react-select working together?

@erikras  do you have a sample you can post and share?

@ooflorent Do you have any sample? I'm working on a project and stucking at get react-select working with redux-form.


As a Bank Admin I would like more flexibility to change my default reminder and deadline days, I would also like the labels to be more clear

Summary

In the My Organization tab, when editing the My Reminder dates, I only have the ability to submit a date between 1-14. I would like the ability to change the date between 1-28. With the logic that the if the reminder day is later in the month, that the deadline day will be for the next next consecutive month. For example, if the bank admin selects a reminder day of the 25th, then the deadline day of the 5th would be for the next month.

Please also change Reminder day to say: Reminder day (Day an e-mail reminder is sent to partners to submit requests)
Please change Deadline day to say: Deadline day (Final day to submit requests)

Things to Consider

Mind if I grab this one?

@AngeVeza could I get a little clarification on this one?
Based on your specification, if we set the reminder to 27 and the deadline to 28, the deadline might be on January 27 whilst the deadline would be February 28; not what I would expect as a user. How about we simplify:

When deadline is 1 and reminder is 1: reject as invalid
When deadline is 2 and reminder is 1: send reminder on Jan 1 and deadline is on Jan 2
When deadline is 1 and reminder is 28: reminder is sent on Jan 1 and deadline is on Jan 28
When deadline is 28 and reminder is 1: reminder is sent on Feb 1 and deadline is Feb 28

All we then need is to ensure that both deadlines and reminders are actioned on the day of the month they're set to.

Having said all that, I've not yet had a chance to see what happens on a deadline (if anything?), so there might be other considerations.

@shkm
The reminder and deadline are used for the same email. How it works is that whatever number is entered for the reminder is the day of the month that all the partner organizations of a bank receive an email. In the email there is some text along the lines of:
This is a friendly reminder that if you haven't submitted your essentials requests they are due on [deadline number].
So, if the reminder day was 10 and the deadline day was 20. They would receive an email on Feb 10th saying:
This is a friendly reminder that if you haven't submitted your essentials requests they are due on February 20.
Our users did request that it could come on different months. For example if the reminder day was 25 and the deadline day was 5 we would like them to receive an email on Feb 25th saying:
This is a friendly reminder that if you haven't submitted your essentials requests they are due on March 5.
I am paraphrasing the language.

Hey @AngeVeza, thanks for the clarification! I think we're on the same page if I understand correctly.


Fix diagnostics when @warn_unused_result is used with other attrs

What's in this pull request?
After #2760, if a @warn_unused_result attribute was used along with other decl. attributes, it would result in a error. This PR fixes that issue.

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci Please test and merge


Chosen custom icon for a tab only visible once

Tab Modifier version
Browser: Chrome
Extension version: 0.22.0
Expected behavior
When the rule is applied on a pinned tab, I expect it to stay there all the time.
Actual behavior
The chosen icon is shown, than going to an other tab.
Then switching back, and then icon changes back to the page's original one.
Steps to reproduce the behavior

Pin a tab
Apply a rule based on the tab's url
Set a custom icon to the rule
Go to the tab you set the rule to
Refresh the page
See the new icon is visible
Leave the tab
Go back to the tab the rule set to
Tab icon changes back to it's original

Hello @zilahir
Is to possible to provide the website URL? It could be related to this website because some are changing the icon back when they detect another script has changed it.
Best

I have this same issue with https://web.whatsapp.com/
It works for other websites e.g. Gmail, but not with WhatsApp.
Is there anything to be done? Thank you!


Not change cursor when navigation is selected

When using a timeline graph and the toolbar is activated, to select the navigation, the cursor does not change to cursor move, as it is done in other graphs, the same happens with the selection area for the zoom
Explanation

What is the behavior you expect?

What is happening instead?

-the code that adds the style is in utils.js

Same issue for bar chart


VCPKG_TRACE_FIND_PACKAGE produces logs although CMAKE_DISABLE_FIND_PACKAGE_ is set

Repro
Using port sdl2:

Set -DCMAKE_DISABLE_FIND_PACKAGE_Git=ON and -DVCPKG_TRACE_FIND_PACKAGE=ON in OPTIONS
Notice that the configure logs still contain find_package(Git )

Expected behaviour
Don't see find_package(Git ) in the logs.
It would make it easier to see if finding a certain package is actually disabled or not - especially if we would add a post build check for optional calls to find_package.
/cc: @dg0yt Related: #27982

What is really needed is solving #18631. Ping @ras0219-msft for finishing #18961.

What is really needed is solving #18631. Ping @ras0219-msft for finishing #18961.

Is there something missing in this PR? It seems like #18961 can be merged if the conflicts are resolved?

There is also #27950 which could be extend to log (disabled by CMAKE_DISABLE_FIND_PACKAGE_<Pkg>=1).
@LilyWangLL There is no real bug here IMO. VCPKG_TRACE_FIND_PACKAGE logs something which does take place: a call to find_package(<Pkg>).

@LilyWangLL There is no real bug here IMO. VCPKG_TRACE_FIND_PACKAGE logs something which does take place: a call to find_package(<Pkg>).

Technically, you are right but IMO there is no reason to log calls to find_package if CMAKE_DISABLE_FIND_PACKAGE_ is set. Furthermore, there won't be a call to find_package if #18961 gets merged so this will indeed be a bug.

@LilyWangLL There is no real bug here IMO. VCPKG_TRACE_FIND_PACKAGE logs something which does take place: a call to find_package(<Pkg>).

Technically, you are right but IMO there is no reason to log calls to find_package if CMAKE_DISABLE_FIND_PACKAGE_ is set. Furthermore, there won't be a call to find_package if #18961 gets merged so this will indeed be a bug.

The job of VCPKG_TRACE_FIND_PACKAGE is printing every call to find_package, If https://github.com/microsoft/vcpkg/pull/18961 gets merged, find_package will not be called if CMAKE_DISABLE_FIND_PACKAGE_ is set, so VCPKG_TRACE_FIND_PACKAGE will not print log.
Should I labeled this issue to question?

There needs to be a decision if these calls shall be
logged as disabled (https://github.com/microsoft/vcpkg/issues/28552#issuecomment-1365955022)
or not logged at all (https://github.com/microsoft/vcpkg/issues/28552#issue-1510879545).

cc @vicroms

There needs to be a decision if these calls shall be logged as disabled (#28552 (comment)) or not logged at all (#28552 (comment)).

no stale

no stale

No stale

@vicroms This issue waits for a vcpkg team decision. https://github.com/microsoft/vcpkg/issues/28552#issuecomment-1366559796


BUG: Unable to remove user with existing second factor

Current behaviour
If you try to remove a user that has a second factor enabled, you will see a 500 Internal Server Error. This is due to the existing second factor:
<!-- Part of the logged Exception -->
Exception in line 182 of .../Packages/Libraries/doctrine/dbal/lib/Doctrine/DBAL/DBALException.php: An exception occurred while executing 'DELETE FROM neos_flow_security_account WHERE persistence_object_identifier = ?' with params ["46c79952-b120-4353-afcb-20179e4420f5"]:

SQLSTATE[23000]: Integrity constraint violation: 1451 Cannot delete or update a parent row: a foreign key constraint fails (`neos_database`.`sandstorm_neostwofactorauthentication_domain_model_secondfactor`, CONSTRAINT `FK_29EF8A7F7D3656A4` FOREIGN KEY (`account`) REFERENCES `neos_flow_security_account` (`persistence_ob)

This is especially a problem in cases where the second factor is enforced and can not be removed.
Expected behaviour
When removing a user with existing second factor(s), all second factors will be removed first. Then the user will be removed. So in the end, no warning or additional work for the administrator / user manager.
Steps to reproduce

Enforce Two-Factor Authentication (optional).
Create a new user.
Log in as that user and create a second factor.
Log out and re-login as Administrator / UserManager.
Try to remove the user with a second factor.

Environment

Flow: 8.3.3
Neos: 8.3.4
Sandstorm/NeosTwoFactorAuthentication: 1.1.2
PHP: 8.1.20

I think we need to do AOP on Neos\Neos\Domain\Service\UserService::deleteUser(User $user) for this.


adding buildenv

This buildenv helps buld libmd with all tests passing

@bamadhu are you going to look into this as part of the kafka effort?


NullReferenceException Thrown from ShardAgent when Exception Policy Handles Apply Exception

I am trying to write an exception policy that retries event application when a failure occurs. However, no matter what policy I write, it doesn't seem to actually save the batch, even when the retry policy successfully recovers from the exception. The exception being written to the logs is
fail: Marten.Events.Daemon.IProjectionDaemon[0]
      Failure while trying to process updates for event range Aggregate for Event range of 'Identity: UserIssueCounter:All', 0 to 500, 1 slices for projection shard 'Identity: UserIssueCounter:All'
      System.NullReferenceException: Object reference not set to an instance of an object.
         at Marten.Events.Daemon.ShardAgent.ExecuteBatch(ProjectionUpdateBatch batch)
         at Marten.Events.Daemon.ShardAgent.<>c__DisplayClass32_0.<<processRange>b__2>d.MoveNext()
      --- End of stack trace from previous location ---
         at Marten.Events.Daemon.ProjectionDaemon.TryAction(ActionParameters parameters)

I have been able to reproduce this in a simple top-level program. My projection throws an exception on the first few attempts, and then after 2 exceptions, it starts applying the events. Even though the whole batch is processed, the batch is never persisted to the database, and the above exception is logged.
Program.cs.txt

No clue. Can you try doing this in a failing test in a pull request? The NRE obviously isn't helpful at all in this case.

@jeremydmiller I did attach the reproducible example I've been using to the issue. I can try to convert this into a failing test in a PR. It's slightly challenging in that the exception is logged and swallowed, then the shard is stopped. So there's not a great failure condition for me to check on


Add memrise course to starting guide.

Description of Changes
Closes #

New Version Info
Author's Instructions

[ ] Derive a new MAJOR.MINOR.PATCH version number. Increment the:

MAJOR version when you make incompatible API changes
MINOR version when you add functionality in a backwards-compatible manner
PATCH version when you make backwards-compatible bug fixes

[ ] Update CHANGELOG.md, following the established pattern.

Collaborator's Instructions

[ ] Review CHANGELOG.md, suggesting a different version number if necessary.
[ ] After merging, tag the commit using these (Mac-compatible) bash commands:git checkout master
git pull
sed -n "$(grep -n -m2 '####' CHANGELOG.md | cut -f1 -d: | sed 'N;s/\n/,/')p" CHANGELOG.md | sed '$d'
git tag -a $(read -p "Tag Name: " tag;echo $tag) -m"$(git show --quiet --pretty=%s)";git push origin --tags

So… thinking user-experience, it's most beneficial to Vim users to get all the help they need from within Vim. This is why there is a high bar on Vim documentation. Having to context-switch into the browser to learn mappings whose docs are already available locally in the editor seems like quite the travel to me. I don't see why users would need this.

You are absolutely right. But Its not the point of this course. You will learn/memorize the Keybindings once and that's it. You won't need to travel even to docs! And that's fast. Try it.

Why not encourage people to learn the key bindings from the docs?

Ok Wow! This is getting weird! Its not about learning, its more about memorizing! You should learn them in docs. This Memrise thing is a tool that helps you to memorize things!


Validation messages doesn't work

Hi,
When i use '/api/register' with out 'Fields required', just can i see this

You need to set on Postman headers Accept application/json, Content-type application/json
with this Laravel would automatically identify the request response and return validation errors as json, instead of returning a redirect response.


Encode special characters

I have troubles while using space character in filename for webdav.PhysicalFileSystem((and mb others) with WinSCP(in Windows) or CloudDuck(Mac OS X).
feel free to contact me.

Hello,
Indeed, WinSCP has trouble with unencoded white spaces.
I fixed the problem (v2.3.8) and it should not happen with other special character anymore. I encoded the file name and the URL of the resources.
I could not try with CloudDuck on Mac OS X, so please, try with the patched version and tell me if the problem is still occurring.
I let you close the issue if you think that the problem is gone.
Thank you very much for your contribution.

Thanks, good work. WinSCP works fine. unicode chars & space work in Windows Explorer and Mac OS X Finder. But cyberduck 2 still doesn't display files with the white space on mac os x. I'm not sure, but possibly that is its problem. I will research for this behaviour later.
Also I have some strange Finder(OS X) behaviours, but I need to add more logging.

I cannot try on Mac OS X, because I don't have it.
I tried with Cyberduck 6.2.11 (Windows), WinSCP 5.11.2 (Windows), Nautilus (Ubuntu), Cadaver (Ubuntu), BitKinex 3.2.3 (Windows), WebDAV Redirector (Windows) GUI and command line (NET USE [...]). All seems to work with the new version 2.3.9 of webdav-server.
Nautlius was having some trouble with encoded names, but I fixed the issue I was encountering. Maybe that fixed the strange behaviour of Finder.
Can I have more information about the version of Cyberduck which causes trouble please?
Thank you for you help.

Great job!
I don't see any encodings problem with my WebDAV clients in the latest version!
And the last strange behavior of os x finder is duplicating entries in a file listing. This lasts for a second: folders and files are duplicated, and then they disappear. I have not seen this on any of the WebDAV clients. Windows Explorer is fine, too, this concerns only the finder.
I will try to add logging for _fastExistCheck() and _readDir() calls, hope it would help


Map-Chat / Crowd Sourced Data

This goes back to PR https://github.com/PokemonGoF/PokemonGo-Bot/pull/3400 wish should of been discussed a bit more as I don't think people realized the scope at which @solderzzc was talking of going with this is huge. Here is a working example (with his previous PR + help of course) of a crowd sourced map.
Basically the idea is to use a relay system (mqtt here) to recieve messages from the client (containing pokemon spawn coordinates in this example). Each client subscribes to the topic (in this case "chat" and pokemon coordinates), and this information is relayed across the network of all bots.
Think "torrent". In this way the bots are effectively able to communicate with eachother, and as demonstrated in our example, plot the pokemon spawns on a map using data gathered from all bots that are broadcasting. The source of the coordinates is stripped away from the message, so no one knows where it came from.
As it stands now we are using a test mqtt broker, which is perfect considering we are in development branch. The idea is that the community will host or create their own mqtt brokers (relay points or "hubs"), and transmit the data to those brokers and receive them the same way as in this example. This will allow us to display all kinds of information on the map such as pokemon spawn locations, high ban count zones, lured forts, ect.
In this working example the bot client transmits coordinates of catchable pokemon to the relay server, strips identifying data, and these coordinates are then readable by all other bot clients to work with.

To view the map and chat/communicate
cd map-chat
python -m SimpleHTTPServer 8080
And open <IP_ADDRESS>:8080 in your browser. To disable the feature as it's presented here, change "enable_social" in config

@alexyaoyang restored, sorry about that, branches were minutes apart

@BreezeRo All is good 👍

perfect , exactly what I have been waiting for , shared scanner data
now all that will be needed is an verification system so if multiple bots are is getting false spawns it can be removed from the swarm ? ( ie someone falsifying data )

@MasterCATZ absolutely, through use of the encounter_id + coordinates we can somewhat verify if the coordinate is legit, next step is to perhaps ensure that the long/lat is actually valid and not in the ocean before allowing it to be read. I'm sure some guys around here got some good ideas to input on this. Very good point you bring up.

@BreezeRo could you please reduce logging, cause people will turn off this feature and we will loose part of data, collected from enabled-by-default config #4602

I'm getting an error of:
Oops! Something went wrong.
This page didn't load Google Maps correctly. See the JavaScript console for technical details.
I check the console and see:
Google Maps API warning: NoApiKeys https://developers.google.com/maps/documentation/javascript/error-messages#no-api-keys
Is this working off the google maps api key in the config.json or should we be specifying this elsewhere?

Same as @TheCannings. Where do I put the Google Maps key?

yap ^

When this goes live, we need to modify the PokemonGo-Bot-Start batch to start this webserver too. Right now, it only starts the bot and the PokemonGo-Bot Map.

The key message is only a warning and can usually be ignored, let me see if I can reproduce it

FYI, the chat option doesnt work on Microsoft Edge. Maybe should add a warning about it. you can see the map and the dots, but the text boxes doesnt pop up.

Also, on Chrome all the messages are popping up as "undefined".

thats normal Daniels, please read the original OP, this is a work in progress ;)

I know, just providing some feedback.

Anyone with this error?
   self.mqttc.publish("pgomapgeo/"+geo_hash+"/"+str(data['pokemon_id']), str(data['latitude'])+","+str(data['longitude'])+","+str(data['encounter_id'])+","+str(data['pokemon_id'])+","+str(data['expiration_timestamp_ms'])+","+str(data['pokemon_name'])) KeyError: 'pokemon_name'

when creating something like this don't forget to update the readme and the other docs..

can this be used for snip pokemon ? without run a local map scanner, read other bot account same ip no good)


M3-4279: Remove duplicate text truncation helpers

Description
Combine both text truncation utilities into one file. The original intent was to get rid of truncateText.ts but I decided to merge the two instead since it breaks the text at the end of the word whereas the functions in truncate.ts breaks the text mid-word
How to test
Check everywhere that truncateText is be called to make sure there's no regression

@DevDW no idea but I just removed it 😂


encoding tickets with priority inferior not assigned via rpc

Reported by atze on 7 Jan 2014 23:22 UTC
see https://tracker.fem.tu-ilmenau.de/30c3/ticket/3688 which is currently in ready to encode but has priority inferior. there are 2 idle encoders but they don't get that ticket.

Comment by pegro on 8 Jan 2014 01:31 UTC
I doubt this has anything to do with priorities.
In the particular case the related encoding profile is "Audio only (DV) Mp3" which is configured to depend on another encoding profile "H.264 from DV (HQ)". The corresponding encoding ticket of this talk (same parent_id) is in state "material needed", so that the required dependecy is not fulfilled.
I have to admit that the complex mechanisms deciding which tickets are in a serviceable state are not very transparent. But that's a differnt construction side.
If this works with other priorities, than there are much more severe bugs within the code.


Doesn't bind to ::

As far as I can tell there's no way to bind sslip to :: or any other address for that matter. It always binds to <IP_ADDRESS> which only handles v4. I figured there should be a way to bind to v6 since the aws instance has it but I can't find any option for it.

Show me the output of sudo ss -lntu | grep 53
My output shows that it binds to IPv6 as well as IPv4:
udp   UNCONN 0      0                            <IP_ADDRESS>:53        <IP_ADDRESS>:*
udp   UNCONN 0      0                           <IP_ADDRESS>:53        <IP_ADDRESS>:*
udp   UNCONN 0      0                             <IP_ADDRESS>:53        <IP_ADDRESS>:*
udp   UNCONN 0      0                         <IP_ADDRESS>%lo:53        <IP_ADDRESS>:*
udp   UNCONN 0      0               [2600:1f18:aaf:6900::a]:53           [::]:*
udp   UNCONN 0      0                                 [::1]:53           [::]:*
tcp   LISTEN 0      4096                      <IP_ADDRESS>%lo:53        <IP_ADDRESS>:*

Huh??? That's weird... I'm running the following
sudo -u sslip ./sslip.io-dns-server-linux-amd64 -port 9593 -nameservers ns1.example.com -addresses ns1.example.com=2001:db8::
With my information in place and the only thing ss shows it binding to is <IP_ADDRESS>:9593. Is there some way to force it to bind to all addresses instead of ANYADDR? Maybe the problem is it only binds to v4 ANYADDR but will bind to v6 directly? I'm still not entirely sure how to stop it from using ANYADDR though.

Let's see the output of the following

ip a
sudo ss -lntu | grep 53

I have replaced my actual prefixes with 2001:db8 but left the rest unmodified including my ULA prefixes. Please note I am running IPv6 exclusively, there are no v4 addresses. This is a VM but no containerization is being done. Please note I'm running bind on 53 which is why I'm setting sslip to use 9553 but for completeness I have included my port 53 binds as requested as well as my 9553 binds. Debian 10 bookworm is the host
ip a:
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 state UNKNOWN qlen 1000
    inet6 fd2e:9fea:c7ed:6011::/64 scope global
       valid_lft forever preferred_lft forever
    inet6 fd2e:9fea:c7ed:53::/64 scope global
       valid_lft forever preferred_lft forever
    inet6 2001:db8:60:11::/64 scope global
       valid_lft forever preferred_lft forever
    inet6 2001:db8:53::/48 scope global
       valid_lft forever preferred_lft forever
    inet6 ::1/128 scope host
       valid_lft forever preferred_lft forever
3: eth0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 state UP qlen 1000
    inet6 2001:db8::1d1d/128 scope global
       valid_lft forever preferred_lft forever
    inet6 fe80::1d1d/64 scope link
       valid_lft forever preferred_lft forever
14: akashi@NONE: <POINTOPOINT,NOARP,UP,LOWER_UP> mtu 1444 state UNKNOWN qlen 1000
    inet6 fe80::2/64 scope link
       valid_lft forever preferred_lft forever
    inet6 fe80::303e:6ff:fe98:deb6/64 scope link
       valid_lft forever preferred_lft forever
15: raphtalia: <POINTOPOINT,MULTICAST,NOARP,UP,LOWER_UP> mtu 1500 state UNKNOWN qlen 500
    inet6 fe80::2/64 scope link
       valid_lft forever preferred_lft forever
    inet6 fe80::ed0:af23:8d67:2045/64 scope link stable-privacy
       valid_lft forever preferred_lft forever

ss -lntu | grep 53:
udp   UNCONN 0      0                                   <IP_ADDRESS>:9553       <IP_ADDRESS>:*
udp   UNCONN 0      0                   [fd2e:9fea:c7ed:6011::]:53            [::]:*
udp   UNCONN 0      0                   [fd2e:9fea:c7ed:6011::]:53            [::]:*
udp   UNCONN 0      0                     [fd2e:9fea:c7ed:53::]:53            [::]:*
udp   UNCONN 0      0                     [fd2e:9fea:c7ed:53::]:53            [::]:*
udp   UNCONN 0      0                        [2001:db8:60:11::]:53            [::]:*
udp   UNCONN 0      0                        [2001:db8:60:11::]:53            [::]:*
udp   UNCONN 0      0                           [2001:db8:53::]:53            [::]:*
udp   UNCONN 0      0                           [2001:db8:53::]:53            [::]:*
udp   UNCONN 0      0                                     [::1]:53            [::]:*
udp   UNCONN 0      0                                     [::1]:53            [::]:*
udp   UNCONN 0      0                          [2001:db8::1d1d]:53            [::]:*
udp   UNCONN 0      0                          [2001:db8::1d1d]:53            [::]:*
udp   UNCONN 0      0                         [fe80::1d1d]%eth0:53            [::]:*
udp   UNCONN 0      0                         [fe80::1d1d]%eth0:53            [::]:*
udp   UNCONN 0      0                          [fe80::2]%akashi:53            [::]:*
udp   UNCONN 0      0                          [fe80::2]%akashi:53            [::]:*
udp   UNCONN 0      0         [fe80::303e:6ff:fe98:deb6]%akashi:53            [::]:*
udp   UNCONN 0      0         [fe80::303e:6ff:fe98:deb6]%akashi:53            [::]:*
udp   UNCONN 0      0                       [fe80::2]%raphtalia:53            [::]:*
udp   UNCONN 0      0                       [fe80::2]%raphtalia:53            [::]:*
udp   UNCONN 0      0      [fe80::ed0:af23:8d67:2045]%raphtalia:53            [::]:*
udp   UNCONN 0      0      [fe80::ed0:af23:8d67:2045]%raphtalia:53            [::]:*
tcp   LISTEN 0      10                          [2001:db8:53::]:53            [::]:*
tcp   LISTEN 0      10                          [2001:db8:53::]:53            [::]:*
tcp   LISTEN 0      10                  [fd2e:9fea:c7ed:6011::]:53            [::]:*
tcp   LISTEN 0      10                  [fd2e:9fea:c7ed:6011::]:53            [::]:*
tcp   LISTEN 0      10        [fe80::303e:6ff:fe98:deb6]%akashi:53            [::]:*
tcp   LISTEN 0      10        [fe80::303e:6ff:fe98:deb6]%akashi:53            [::]:*
tcp   LISTEN 0      10                        [fe80::1d1d]%eth0:53            [::]:*
tcp   LISTEN 0      10                        [fe80::1d1d]%eth0:53            [::]:*
tcp   LISTEN 0      10                         [2001:db8::1d1d]:53            [::]:*
tcp   LISTEN 0      10                         [2001:db8::1d1d]:53            [::]:*
tcp   LISTEN 0      10                                    [::1]:53            [::]:*
tcp   LISTEN 0      10                                    [::1]:53            [::]:*
tcp   LISTEN 0      10     [fe80::ed0:af23:8d67:2045]%raphtalia:53            [::]:*
tcp   LISTEN 0      10     [fe80::ed0:af23:8d67:2045]%raphtalia:53            [::]:*
tcp   LISTEN 0      10                    [fd2e:9fea:c7ed:53::]:53            [::]:*
tcp   LISTEN 0      10                    [fd2e:9fea:c7ed:53::]:53            [::]:*
tcp   LISTEN 0      10                       [2001:db8:60:11::]:53            [::]:*
tcp   LISTEN 0      10                       [2001:db8:60:11::]:53            [::]:*
tcp   LISTEN 0      5                                     [::1]:953           [::]:*
tcp   LISTEN 0      5                                     [::1]:953           [::]:*
tcp   LISTEN 0      10                         [fe80::2]%akashi:53            [::]:*
tcp   LISTEN 0      10                         [fe80::2]%akashi:53            [::]:*
tcp   LISTEN 0      10                      [fe80::2]%raphtalia:53            [::]:*
tcp   LISTEN 0      10                      [fe80::2]%raphtalia:53            [::]:*

ss -lntu | grep 9553:
udp   UNCONN 0      0                                   <IP_ADDRESS>:9553       <IP_ADDRESS>:*

Did you build from source or download an executable?
When the DNS server starts up, does it emit a message similar to "I couldn't bind to "<IP_ADDRESS>:%d" (INADDR_ANY, all interfaces), so I'll try to bind to each address individually."?

I downloaded the latest binaries last night, I did not build from source. I do not get that message...I get this one
2023/07/11 18:22:19 Successfully bound to all IPs, port 9553.

Hmmm, maybe a bug in Golang 1.20.2. You a coder? Try building from source with Golang 1.20.6 (came out today) and see if that has the same problem.
I don't have any pure-IPv6 machines to test on, sorry.

Yeah I program. I was just being lazy trying to avoid building from source, only do that when I have to.  I'll build it and let you know what happens.

Let me know if building on your machine fixes it. Could you also test the one I built with Go 1.20.6? Here's the link: https://nono.io/sslip.io-dns-server-linux-amd64

On Go 1.20.5 which is what's in the debian sid repos it still only binds to v4. Same goes for your build...that being said I did figure out why the problem happens. It's a go "bug" triggered by a quirk of my very aggressive IPv6 only setup. For whatever reason go will only bind to :: if <IP_ADDRESS> is present on your loopback interface, I only have ::1. If I add <IP_ADDRESS> on my loopback it binds to :: ...but if I remove it as is my normal configuration it will only bind to <IP_ADDRESS>...which seems awfully backwards to me but that does make it not your bug. I tried adding loopback only because this isn't the first nor will it be the last time I find IPv4 assumptions in software and <IP_ADDRESS> is often a hardcoded assumption in many pieces of software causing wide ranging issues. I have patched it myself to work around the issue by doing
conn, err := net.ListenUDP("udp", &net.UDPAddr{IP: net.ParseIP("::"), Port: *bindPort})

Software breaks in weird ways without <IP_ADDRESS> if they don't use localhost, glad I thought of it. Here's the go issue for reference https://github.com/golang/go/issues/61310


web page opening but without format

After following the steps, I have successfully open the main page but I have no format (see printscreen)

thanks,

Solved it was necessary to have a certificate for https.

Thanks for your participation!

For newbies like me I also had the problem in parameter.yml you have to change https://cocorico.dev to yourwebsite.com


Allow ability to set default graph time period

Is your feature request related to a problem? Please describe.
At the moment, the graphs always default to 24hr view. This is annoying when you schedule infrequent speed tests and would be more interested in seeing the results over a longer period. e.g. 1 week.
Describe the solution you'd like
Allow ability to set default graph time view. At least for the public dashboard if enabled.
Additional context

Attaching to #448


Bump gov.uk frontend to 2.10.0

What
Bump gov.uk frontend to 2.10.0. Also adds the preventDoubleClick option to buttons, introduced in v2.8.0.
How to review

Code review

Who can review
Anyone

(stub doesn't expose billing for instance).

I've raised a story to address this


Add support for Dialog Select Datasource.

Add support for most recent changes to Dialog Select component.
I created an issue to keep track of this change:
Add support for Dynamic Select in Dialogs. #439

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Fernando Avalos seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@favalos Thanks for taking the time to report the issue and make this PR!
To sign our CLA, could you follow the instructions below?
git reset HEAD~1
git add . -v
git config user.email {your github email address}
git commit -m'{your commit message}'
git push origin master

Once you've signed the CLA, I'm happy to fix the above by adding more commits in addition to your changes for you.

@seratch I did the changes requested. There are only four files changes in the pull request now.
Thanks and let me know if I can help to add more test cases.

I've merged your change by https://github.com/slackapi/java-slack-sdk/commit/0bd925ff826a537739bd62808374683b7688e18e and https://github.com/slackapi/java-slack-sdk/commit/a14cb6303a6c040c2bf41092065b472b60779a0a
Thanks a lot for your contribution!


Can i use dynamic variables (generated in powershell by ##vso.setVariable command) in {{ each ... }} statement?

Can i use dynamic variables (generated in powershell by ##vso.setVariable command) in {{ each ... }} statement?
I want to generate some objects in powershell by ##vso.SetVariable command and iterate it by {{ each ....}} statement

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

ID: 6724abea-bbdc-bf66-ed5e-3214fa6c3e66
Version Independent ID: 4f8dab21-3f0e-da32-cc0e-1d85c13c0065
Content: Job and step templates - Azure Pipelines
Content Source: docs/pipelines/process/templates.md
Product: devops
Technology: devops-cicd
GitHub Login: @vtbassmatt
Microsoft Alias: macoope

You cannot. Template expressions (${{ }}) are fully expanded before the run even really starts.


is there a  build available  with -openssl

Most of the files on popular websites are all "https", is there a version of this build with -openssl option?

Joining to the question, how do we add a support for https links?

Joining to the question, how do we add a support for https links?

There is the same request from 2018 https://github.com/bravobit/FFmpeg-Android/issues/25 and still unfortunately not yet implemented.


Added commands to clean node_modules for Docker and CI Build

Description of changes

Added commands to clean node_modules in the Docker Image as well as the CI build.
Issue Resolved

Fixes #510

https://travis-ci.org/OperationCode/operationcode_frontend/builds/282029329#L494
@kylemh want to do the honors?

I think I should remove the make clean command first. That will delete the node_modules folder after build. Let me fix this real quick.

Ran all test suites.
console.error node_modules/react-dom/node_modules/fbjs/lib/warning.js:33
    Warning: React depends on requestAnimationFrame. Make sure that you load a polyfill in older browsers. http://fb.me/react-polyfills

https://travis-ci.org/OperationCode/operationcode_frontend/builds/282027886#L604

I believe this is good to go @kylemh


[UX] Incorrect footer note on carbon console

Description:
incorrect footer note on carbon console
Affected Product Version:
2.1.0
OS, DB, other environment details and versions:
Ubuntu 16.10
Steps to reproduce:

load carbon console
https://localhost:9443/carbon/admin/login.jsp

© 2005 - 2016 WSO2 Inc. All Rights Reserved.
year need to be updated

This is fixed for APIM 2.6.0


Include redirectUri in DispatchOutcome.VerifierResponse.Rejected

DispatchOutcome.VerifierResponse.Rejected must be updated to a data class that contains an nullable property redirectUri similar to DispatchOutcome.VerifierResponse.Accepted.
Even in the case that the user didn't give his consensus to present a credential, the verifier might want to be redirect to a certain uri to display extra information to the user.

@babisRoutis I was reading the spec again. I believe that redirect_uri is applicable only during the happy path.
In 6.2.Response Mode "direct_post", redirect_uri is defined as an optional property returned by the Verifier Endpoint once it has successfully processed the response provided by the Wallet.
Quoting:

If the Response Endpoint has successfully processed the request, it MUST respond with HTTPS status code 200. The following new parameter is defined for use in the response from the endpoint:
redirect_uri: OPTIONAL. When the redirect parameter is used the Wallet MUST send the User Agent to this redirect URI. The redirect URI  allows the Verifier to continue the interaction with the End-User on the device where the Wallet resides after the Wallet has sent the Authorization Response to the Response URI. It especially enables the Verifier to prevent session fixation (Section 12.2) attacks.

My initial concern still stands though. The Verifier might want to present some extra information to the User in case the response provided by the Wallet has not been successfully processed or has been rejected.
For this we can leverage the fact that OpenId4VP is an extension of OAuth2.
Checking 6.4. Error Response we read:

The error response follows the rules as defined in [RFC6749], with the following additional clarifications.

Checking RFC6749: 5.2 Error Response we read:

The authorization server responds with an HTTP 400 (Bad Request) status code (unless specified otherwise) and includes the following parameters with the response.

The parameter that stands out is error_uri which is defined as follows:

error_uri: OPTIONAL.  A URI identifying a human-readable web page with information about the error, used to provide the client developer with additional information about the error. Values for the "error_uri" parameter MUST conform to the URI-reference syntax and thus MUST NOT include characters outside the set %x21 / %x23-5B / %x5D-7E.

This seems to be a good candidate. What do you think?

@dzarras We can watch https://github.com/openid/OpenID4VP/issues/137


Chat channel filtering?

Hi,
First I want to say; Excellent work on this tool! Amazing features.
I am wondering if it's possible to either suppress certain types of chats (maybe with rules to match based on chat content?) or if I could at least only allow chats from human players? Im not seeing any documentation on it.
If not, I think this would be an excellent option to have. I like the idea of being able to help people while not being in the game myself.
One other feature that might be cool is the ability to @ people in the discord from game chat. Not sure how that would work though.

Spoke on discord, I will just close the issue, thank you!


Get a baseline performance for the IMEX-based path

Collect perf data for e2e OV execution via https://github.com/slyalin/openvino/blob/mlir/tools/mlir_bench/mlp_bench.sh.

Performance collected


Small glitches

(xhr) records appear for a split second when I expand a test
The "Turn on XHR visibility" button sometimes doesn't work when I click on it
When I click the "Turn on XHR visibility" button, (xhr) records get hidden after a couple of seconds automatically

You can see the above glitches here:
https://user-images.githubusercontent.com/8045060/228967568-3d5b15b1-0ee0-47e2-a2e1-469ac608cc39.mov

Cypress version: 12.9.0

Same here

happy to announce that these were all fixed with 1.2.1 release


hugin: move to openexr_3

OpenEXR 2 is EOL.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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

[x] 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/)
25.05 Release Notes (or backporting 24.11 and 25.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

[x] Fits CONTRIBUTING.md.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 2 packages built:

digikam
hugin

<!--
To help with the large amounts of pull requests, we would appreciate your
reviews of other pull requests, especially simple package updates. Just leave a
comment describing what you have tested in the relevant package/service.
Reviewing helps to reduce the average time-to-merge for everyone.
Thanks a lot if you do!
<p>List of open PRs: https://github.com/NixOS/nixpkgs/pulls
Reviewing guidelines: https://github.com/NixOS/nixpkgs/blob/master/pkgs/README.md#reviewing-contributions
-->

Add a :+1: reaction to pull requests you find important.

nixpkgs-reviewd workflow started

Extra args: ``
Free space: true
Who asked: lucasew

TARGETS

x86_64-linux: true
x86_64-darwin: false
aarch64-darwin: false

Status and logs: https://github.com/nixosbrasil/nixpkgs-reviewd/actions/runs/12447999511

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 367087

x86_64-linux

:white_check_mark: 2 packages built:

digikam
hugin


Upgraded influx module to version 4.0.1 to support influxdb 0.9

This addresses #11 by upgrading the influx module to version 4.0.1.
Though, this might be breaking on pre 0.9 versions of influxdb since the influx client method writePoint() used by this module has a new attribute, tag, in the 4.x version which I also compensate for here. Without compensating for this new attribute the collector will throw an exception and not write anything to influxdb.

Aha. I was not aware of numbat-influx to be honest.
Could one path be a removal of pre 0.9 support with a 2.0.0 release of this module? Then pre 2.x of this module is pre 0.9 influxdb and post 2.x of this module post 0.9 influxdb?
+1 for batching.


Add support for 'deprecated' tag

Compiling the 'deprecated' tag will result in a warning message in the console.
Note: The 'deprecated' tag was added PHP Twig 2.6, see: https://twig.symfony.com/doc/2.x/tags/deprecated.html.

+1 for using sinon in the test suite.

This is great! Thanks so much :+1:

This is great! Thanks so much

Thanks! Any idea when a new version will be released?

Anything in particular you think is holding it back? The changelog looks pretty solid.

No, I don't think so.


Problem with loss

I'm training a tiny yolo with 600 pic without loading pretrain weight. The problem is at the 1st epoch, the loss is very low (about 0.07) and the model was saved. At the 2nd epoch, the loss increase to 2 and after that it slowly descrease. The problem is the model was saved at the 1st epoch and after that it can't be saved anymore be cause the loss is alway higher than it is at the 1st epoch! please help
Epoch 1/50
38/39 [============================>.] - ETA: 20s - loss: 0.7124 Epoch 00000: val_loss improved from inf to 0.07960, saving model to best_weights.h5
39/39 [==============================] - 1103s - loss: 0.6963 - val_loss: 0.0796
Epoch 2/50
38/39 [============================>.] - ETA: 20s - loss: 0.9856 Epoch 00001: val_loss did not improve
39/39 [==============================] - 1093s - loss: 1.0072 - val_loss: 1.6463
Epoch 3/50
38/39 [============================>.] - ETA: 20s - loss: 1.4972 Epoch 00002: val_loss did not improve
39/39 [==============================] - 1096s - loss: 1.4875 - val_loss: 1.5270
Epoch 4/50
38/39 [============================>.] - ETA: 20s - loss: 1.2201 Epoch 00003: val_loss did not improve
39/39 [==============================] - 1100s - loss: 1.2121 - val_loss: 1.8843
Epoch 5/50
38/39 [============================>.] - ETA: 20s - loss: 1.0720 Epoch 00004: val_loss did not improve
39/39 [==============================] - 1091s - loss: 1.0613 - val_loss: 1.4738
Epoch 6/50
38/39 [============================>.] - ETA: 20s - loss: 0.9571 Epoch 00005: val_loss did not improve
39/39 [==============================] - 1085s - loss: 0.9489 - val_loss: 1.4657
Epoch 7/50
38/39 [============================>.] - ETA: 20s - loss: 0.8815 Epoch 00006: val_loss did not improve
39/39 [==============================] - 1083s - loss: 0.8774 - val_loss: 1.4764
Epoch 8/50
38/39 [============================>.] - ETA: 20s - loss: 0.7926 Epoch 00007: val_loss did not improve
39/39 [==============================] - 1085s - loss: 0.7873 -
PS: I'm using my train set as my test set to test the model, is that ok???

The loss starts low because of the warmup stage, which is defaulted to 100. It's advisable to leave it like that as warmup training is very important. Instead, you can start training the second time with the "pretrained_weights" in the config.json file set to "best_weights.h5". It will start from this weights.
It's not a good idea to use the train set as the test set. I think you should leave the test set empty. The code in train.py will automatically split the given train set to the train set and test set using the ration 0.8 if it cannot find the test set.


flake.nix add missing profiles

I noticed, that some of the profiles where missing from the flake.nix file.
I also ordered the profiles alphabetically.

Thanks!


adding longest increasing subsequence

C source code for longest increasing subsequence algorithm.
Utilized the binary search.

Adding C source code for coin change algorithm.
Used Greedy algorithm


Android 11+ Support is needed

Since Android 11 (API 30) is now the target API version for the Google Play store, some work is going to need to be done to allow saving images to the gallery on phones using Android 11+ ...
If I can find a fix, I'll of course send a merge request, but for now I'm still at a loss on how to implement it, but the best bets are using the MediaStore functions.

sorry for coming late. it's done.
let me know if the latest release works fine with your devices, so we can proceed with other fixes.
and thanks for the reminder.

Android 11 (API 30) camera can't save photo

Android 11 (API 30) camera can't save photo in real phone. But in emulator camera saves photo "image.jpg" to pictures

Are you making sure the camera permission is given to the app?

of course. in api 29 works fine but api 30 i cant solve the problem

of course. in api 29 works fine but api 30 i cant solve the problem

@ouygunol API 29 likely works fine because of requestLegacyExternalStorage
For the phone that can't save a photo in the phone, does any error show up? Did you make sure to give it a second test too? (I've seen a bug sometimes where first attempt to capture photo doesn't work)
I've yet to test this newest version on my own app, so I can't help too much.

Hi, for me its also not woking in Android 11 (API 30) on a real phone. Permissions are all set. Here is the Error Log when running the Os-FileUp App:
java.lang.RuntimeException: Failure delivering result ResultInfo{who=null, request=1, result=-1, data=Intent { act=inline-data (has extras) }} to activity {mgks.os.fileup/mgks.os.fileup.MainActivity}: java.lang.NullPointerException: uriString         at android.app.ActivityThread.deliverResults(ActivityThread.java:5016)         at android.app.ActivityThread.handleSendResult(ActivityThread.java:5057)         at android.app.servertransaction.ActivityResultItem.execute(ActivityResultItem.java:51)         at android.app.servertransaction.TransactionExecutor.executeCallbacks(TransactionExecutor.java:135)         at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:95)         at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2067)         at android.os.Handler.dispatchMessage(Handler.java:106)         at android.os.Looper.loop(Looper.java:223)         at android.app.ActivityThread.main(ActivityThread.java:7697)         at java.lang.reflect.Method.invoke(Native Method)         at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:592)         at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:952)      Caused by: java.lang.NullPointerException: uriString         at android.net.Uri$StringUri.<init>(Uri.java:496)         at android.net.Uri$StringUri.<init>(Uri.java:486)         at android.net.Uri.parse(Uri.java:458)         at mgks.os.fileup.MainActivity.onActivityResult(MainActivity.java:115)         at android.app.Activity.dispatchActivityResult(Activity.java:8310)         at android.app.ActivityThread.deliverResults(ActivityThread.java:5009)

any update on this issue?

The buttons are not responding in API 31 while it does work fine in lower API  versions.

will be fixing it by march end.


0.9.0

Signed-off-by: Stephen Curran<EMAIL_ADDRESS>

Added in a last PR that was deemed low risk.


http3: Handle concurrent Body.Close more gracefully

Re-apply https://github.com/quic-go/quic-go/pull/4638

What is "more graceful" about this? Please describe the bug this is supposed to fix (or even better: open an issue for that).
Please also add a test case.

What is "more graceful" about this? Please describe the bug this is supposed to fix (or even better: open an issue for that).

If the requestDone() is called concurrently, the program may panic, because "close of closed channel": https://github.com/XTLS/Xray-core/issues/4148

Please also add a test case.

I'm not sure if we can make real concurrent calls that must call "close(r.reqDone)" concurrently.
Please feel free to edit the code / the commit message directly.

And I'm not sure if if r.reqDone == nil { should be deleted.

I'm not sure if we can make real concurrent calls that must call "close(r.reqDone)" concurrently.

Sure, you can spawn 2 Go routines that both call Close. While that's not guaranteed to hit the race condition every time, at least from time to time it will.

I've added a line of comment, so that developers will handle these code carefully. I think it is more useful than a random test.

This comment doesn't add any value. Please remove it. And please add a test case, as I've requested for the third time now. It is useful, and this PR will not be merged without a test.

Well, I've add a test case, though "that's not guaranteed to hit the race condition every time". Please review it.

@marten-seemann It's ready.


Error with StrictMode option for _setDefaultFormat

Describe the bug
On the last version of the SunEditor 2.46.2 the StrictMode option is causing a input  managment error.
While the flag StrictMode is true the suneditor on the onKeyUp_wysiwyg function and more exactly in the _setDefaultFormat is throwing the error: Uncaught TypeError: format is undefined
To Reproduce
Steps to reproduce the behavior:

Go to 'SunEditor  with these configs:
{  "strictMode": true,  "value": "Ciao",  "mode": "classic",  "rtl": false,  "katex": "window.katex",  "resizingBar": false,  "showPathLabel": false,  "resizeEnable": false,  "imageResizing": false,  "imageHeightShow": false,  "imageAlignShow": false,  "imageFileInput": false,  "imageUrlInput": false,  "videoResizing": false,  "videoHeightShow": false,  "videoAlignShow": false,  "videoFileInput": false,  "videoUrlInput": false,  "videoRatioShow": false,  "audioUrlInput": false,  "tabDisable": false,  "shortcutsHint": false,  "mediaAutoSelect": false,  "buttonList": [  	[  		"undo",  		"redo",  		"font",  		"fontSize",  		"formatBlock",  		"paragraphStyle",  		"blockquote",  		"bold",  		"underline",  		"italic",  		"strike",  		"subscript",  		"superscript",  		"fontColor",  		"hiliteColor",  		"textStyle",  		"removeFormat",  		"outdent",  		"indent",  		"align",  		"horizontalRule",  		"list",  		"lineHeight",  		"table",  		"link",  		"image",  		"video",  		"audio",  		"math",  		"imageGallery",  		"fullScreen",  		"showBlocks",  		"codeView",  		"preview",  		"print",  		"save",  		"template"  	]  ],  "lang": "SUNEDITOR_LANG.it",  "lang(In nodejs)": "it" }

'
2. Click on 'With focus on the text value -> press "Enter then Ctrl + A, then backspace"
As you can see from the attach:

'
Expected behavior
To manage properly the input without strictMode active :)
Desktop:

OS: [Windows]
Browser [Firefox]
Version [126.0]

Desktop 2:

Device: [Desktop]
OS: [Windows]
Browser [Chrome]
Version [124.0.6367.210]

Had the same issue, wasn't able to solve. I'll follow the thread to see any updates on this, best of luck.

+1

Same issue here, I'll follow to see any updates on this.

Could it be a browser cache issue?
I cannot reproduce the issue.

Could it be a browser cache issue? I cannot reproduce the issue.

I tried it this way:

Open this link: http://suneditor.com/sample/html/examples.html
Selects the strictMode checkbox
Select the Value checkbox
Write in the Value textarea the following HTML code: Ciao
Click on the Create button
Click inside the editor, with the focus at the end of the word Ciao

Press Enter from keyboard to wrap

Press CTRL+A to select all the text (on Windows)

Press BackSpace from keyboard to delete


Attempt to invoke virtual method 'java.lang.String java.lang.String.toLowerCase()' on a null object reference

I am getting the issue after filtered , anybody please help me

Then you wrote false code.. Ask an answer at GitHub Home Page.. This error is not because of MaterialSearchView library.

Or Show me your code


Add voice recording function in the Android app

Is your feature request related to a problem?
May I ask if it is possible to add voice recording function in the Android app. The combination of voice and photography will be perfect
Describe the solution you'd like
Take a photo, and then the meaning of taking this photo can be explained by voice recording. Perfect!
Additional context
No response

Memos does not provide a Android app directly, you should contact the App's author you're currently used.


After adding the file as a link first, the file will not be added to the project.

Describe the bug

Steps to Reproduce

Create a Python Application.
Right click on the project and select Add > Existing Item …
In the prompts dialog, select the item you want to add.( PythonApplication10 )
Click the Add drop-down and select "Add As Link".
Right click on the project and select Add > Existing Item …
Select PythonApplication10 that you chose before, and select "Add". (You can repeat step 5 and step 6)

Expected behavior
After selecting an item "Add As Link" , the item PythonApplication10 should be "Add" in project.
Additional context and screenshots
More than one appeared in SE.( PythonApplication10 Status is "Add As Link" ) And the file PythonApplication10 is not in the directory.

Restart VS, you can see that there is only one. Then, repeat step 5 and step 6, it can repro this.

Output info
Info: Info: Pylance language server 2021.4.1 (pyright 6410a8a2) startingServer root directory: c:\program files\microsoft visual studio\2022\preview\common7\ide\extensions\microsoft\python\core\pylance\dist

Error: Warning: Info: Info: Info: Info: Error reading settings: TypeError: Cannot read property '0' of nullstubPath typings is not a valid directory.Assuming Python platform WindowsSearching for source filesNo source files found.Background analysis(1) root directory: c:\program files\microsoft visual studio\2022\preview\common7\ide\extensions\microsoft\python\core\pylance\dist

Info: Background analysis(1) started
Info: Info: Warning: Info: Info: Info: Info: Info: Info: No configuration file found.Setting pythonPath for service "PythonApplication12": "C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python39_64\python.exe"stubPath C:\Users\vting\source\repos\PythonApplication12\PythonApplication12\typings is not a valid directory.Assuming Python version 3.9Assuming Python platform WindowsSearching for source filesFound 1 source fileBackground analysis(2) root directory: c:\program files\microsoft visual studio\2022\preview\common7\ide\extensions\microsoft\python\core\pylance\distBackground analysis(2) started

In today's build 17.1.0 Preview 3.0 [32029.61.main], I can repro this.

@ttSpace Does this still repro? I can't add duplicate projects.

@StellaHuang95 I can repro in today's build 17.6.0 Preview 2.0 [33423.321.main], Python Package: 17.0.23047.1.

This will be fixed once the pr is finished here: https://devdiv.visualstudio.com/DevDiv/_git/VS/pullrequest/455205


RAM is out of memory during Training (Not GPU VRAM)

Checks

[X] This template is only for question, not feature requests or bug reports.
[X] I have thoroughly reviewed the project documentation and read the related paper(s).
[X] I have searched for existing issues, including closed ones, no similar questions.
[X] I confirm that I am using English to submit this report in order to facilitate communication.

Question details
Title: GPU Node VRAM Out of Memory (OOM) During Training
Description:
I'm encountering an Out of Memory (OOM) issue on my GPU node during training, which has 4 x A100 GPUs with a combined VRAM of 500 GB. The model I'm training is configured with a large batch size and gradient accumulation, but despite using mixed precision and optimizing memory usage, VRAM keeps running out.
Steps to Reproduce:

Train the model with the following configuration:

Batch size per GPU: 16,000
Gradient Accumulation Steps: 32
Mixed Precision: BF16

Dataset: GigaSpeech (11,000 hours, approximately 9.9 million text and audio pairs).
Running on 4 x A100 GPUs (40GB) on my GPU Node with a total of 500 GB VRAM.

Observed Behavior:
Training begins but fails after 3k steps (out of 56471 steps to be completed) due to VRAM limitations, resulting in an OOM error. See the attached, that available memory dropped fro 398GB to 111GB (in the meantime of writing this is 70GB available).
Attempts to lower the batch size and gradient accumulation steps partially alleviate the problem but limit effective batch size and training stability.
https://github.com/user-attachments/assets/68df2135-197c-4a92-be25-b26763395127

Hi @atlonxp , feel free to check https://github.com/SWivid/F5-TTS/issues?q=is%3Aissue+oom if would help, e.g. #233

@SWivid yes i did check that. My case is not GPU VRAM -- but instead it is machine RAM. The more steps complete the more RAM consumed.
I was also looking at PR about checkpointing but it is still a bug.

No idea then 😔
https://github.com/SWivid/F5-TTS/blob/3fcdbc70b4a9d4299e1ecd0b5a1c35209f23fd69/src/f5_tts/model/dataset.py#L248 would by default use in_memory=False

@SWivid Thanks for your prompt response. By the way, I'm working on Thai dataset above using gigaspeech. There are 9.9m data and I'm working on supercomputer (A100 40gb x 4) with Slurm. Do you have any recommendation for training hyperparameters?
it would help me a lot.

@SWivid I have added video recording -- just in case you would found the reason of memory leaks (or probably not)
@lpscr do you have any idea?

@SWivid Thank you for your response.
I have tested by turning persistent_workers=False as suggested. The issue still happens.
This guys also have the same issue: 57#discussioncomment-10981044
Here is my accelerate configurations
compute_environment: LOCAL_MACHINE
debug: false
distributed_type: MULTI_GPU
downcast_bf16: 'no'
enable_cpu_affinity: false
gpu_ids: all
machine_rank: 0
main_training_function: main
mixed_precision: bf16
num_machines: 1
num_processes: 4
rdzv_backend: static
same_network: true
tpu_env: []
tpu_use_cluster: false
tpu_use_sudo: false
use_cpu: false

Hyperparameters:
exp_name"F5TTS_Base"
learning_rate0.000075
batch_size_per_gpu16000
batch_size_type"frame"
max_samples64
grad_accumulation_steps78
max_grad_norm1
epochs100
num_warmup_updates20000
save_per_updates50000
last_per_steps5000
finetunetrue
file_checkpoint_train""
tokenizer_type"pinyin"
tokenizer_file""
mixed_precision"bf16"
logger"tensorboard"

will train.py have same problem? we haven't try with finetune scripts on large dataset yet

@SWivid let me try training via train.py (instead of finetune_gradio.py). I will let you know

@SWivid still OOM.
I think the problem might be this.
DataLoader num_workers > 0 causes CPU memory from parent process to be replicated in all worker processes
I'm investigating on how CustomDataset is being called, also found out that it use many Python List, which I believe that might be the cause.

Hi @atlonxp , agreed. I've heard about it long time ago from one of my colleagues, seems not totally fixed yet lol.
As we have got a mmap way (.arrow) of dataset organization, maybe

explicitly set torch.multiprocessing.set_start_method("spawn") as default
using num_workers=0

are possible ways out as I grasped from discussions in the long-standing issue?
correct me if I got something wrong, with hope of facilitating the implementation of current and future works, many thanks!

@SWivid I have tested all cases and found that we need to experiment by testing num_workers=1 and see how much RAM it consumed. For my cases, my batch_size is 18000 (for A100 with 40GB; this is around half of your batch_size for A100 with 80G). So I did test num_workers = 1, 2, ..., and lastly 4 as my free memory is left available around 40GB.
My assumption is that given the training configuration: batch_size = 18000, and grad_accumulation_steps = 1. One epoch takes around 100GB in RAM. I will test num_workers = 5 and onwards later.
Also, it might not be the case but I think it could probably cause the memory leak as well. It is __getitem__ function in CustomDataset using recursive calls, which believably could lead to Memory Leak.
My assumption is that If there are consecutive samples with invalid durations, this recursive call can lead to a deep recursion stack, consuming a significant amount of memory and possibly causing a stack overflow.
def __getitem__(self, index):
    row = self.data[index]
    audio_path = row["audio_path"]
    text = row["text"]
    duration = row["duration"]

    if self.preprocessed_mel:
        mel_spec = torch.tensor(row["mel_spec"])
    else:
        audio, source_sample_rate = torchaudio.load(audio_path)
        if audio.shape[0] > 1:
            audio = torch.mean(audio, dim=0, keepdim=True)

        if duration > 30 or duration < 0.3:
            return self.__getitem__((index + 1) % len(self.data))    # << -- recursive park

        if source_sample_rate != self.target_sample_rate:
            resampler = torchaudio.transforms.Resample(source_sample_rate, self.target_sample_rate)
            audio = resampler(audio)

        mel_spec = self.mel_spectrogram(audio)
        mel_spec = mel_spec.squeeze(0)  # '1 d t -> d t')

        return dict(
            mel_spec=mel_spec,
            text=text,
        )

So, given this assumption, I have fixed by this way
def __getitem__(self, index):
    while True:
        row = self.data[index]
        audio_path = row["audio_path"]
        text = row["text"]
        duration = row["duration"]

        # Check if the duration is within the acceptable range
        if 0.3 <= duration <= 30:
            break  # Valid sample found, exit the loop

        # Move to the next index and wrap around if necessary
        index = (index + 1) % len(self.data)

    if self.preprocessed_mel:
        mel_spec = torch.tensor(row["mel_spec"])
    else:
        audio, source_sample_rate = torchaudio.load(audio_path)

        # If the audio has multiple channels, convert it to mono
        if audio.shape[0] > 1:
            audio = torch.mean(audio, dim=0, keepdim=True)

        # Resample the audio if necessary
        if source_sample_rate != self.target_sample_rate:
            resampler = torchaudio.transforms.Resample(source_sample_rate, self.target_sample_rate)
            audio = resampler(audio)

        # Compute the mel spectrogram
        mel_spec = self.mel_spectrogram(audio)
        mel_spec = mel_spec.squeeze(0)  # Convert from (1, D, T) to (D, T)

        return {
            "mel_spec": mel_spec,
            "text": text,
        }

If you think this make sense, I will create PR

Hi @atlonxp , I'm quite sure the issue is not with __getitem__
cuz it's a common way for a dataset class in that way,
and if invalid samples with duration <0.3 encountered, will probably get a immediate cuda oom (since padding of many short samples to very possible long sample length) than a long term ram leak
we have experimented with batchsize 19200 on a machine with 250gb ram and with default setting of 16 workers, though cannot reproduce the error of mem leakage.
welcome to test if the modification takes effort in your case, it would be nice if will work

@SWivid Interesting -- now got me thinking what is the actual problem on my machine now. I will continue experiment further then. Let you know what I discover.

Stumbled across this from the link https://github.com/pytorch/pytorch/issues/13246, and just putting this here to try and help: if one of the machines you're trying to replicate on is using Windows or MacOS, the default torch multiprocessing method for those operating systems is 'spawn' and not 'fork', so if the mem leak is due to reference counting issues with 'fork' then it won't arise on that machine. You can also set 'shuffle=True' in the dataloader, and if the memory leak occurs more rapidly than with 'shuffle=False', then it points to the issue being the one discussed in the link above (which is solved by using objects that are reference counted once per total instance (ie numpy arrays, pandas dataframes etc.) and not using python native objects (where every entry is reference counted independently), or it is solved by using the multiprocessing method 'spawn' instead of fork).

@nicholas-greig This is very interesting suggestion. I will test it and let you know the results.


[BUG] node lifecycle controller in yurt-manager can not update status of node

What happened:
Node always stays with Ready status after stopping kubelet on it, even shutting down the node itself.
The bug causes the Pods can not be migrated to other nodes.
What you expected to happen:
The abnormal node should be updated into NotReady status.
How to reproduce it (as minimally and precisely as possible):
Stopping the kubelet on a node.
Anything else we need to know?:
Error log in yurt-manager's node lifecycle controller:
E0126 07:43:15.444074       1 node_lifecycle_controller.go:975] "Error updating node" err="nodes \"edge\" is forbidden: User \"system:serviceaccount:kube-system:yurt-manager\" cannot update resource \"nodes/status\" in API group \"\" at the cluster scope" node="edge"
E0126 07:43:15.452574       1 node_lifecycle_controller.go:715] "Update health of Node from Controller error, Skipping - no pods will be evicted" err="timed out waiting for the condition" node="edge"

nodes/status is a subresource, it should be added to the ClusterRole of yurt-manager also.
Environment:

OpenYurt version: v1.4
Kubernetes version (use kubectl version): v1.27.2

/kind bug

/assign @crazytaxii

It has been fixed in #1884.

The entire system:controller:node-controller ClusterRole for kube-controller-manager in Kubernetes cluster v1.27.2 is:
# ...
rules:
- apiGroups:
  - ""
  resources:
  - nodes
  verbs:
  - delete
  - get
  - list
  - patch
  - update
- apiGroups:
  - ""
  resources:
  - nodes/status
  verbs:
  - patch
  - update
- apiGroups:
  - ""
  resources:
  - pods/status
  verbs:
  - patch
  - update
- apiGroups:
  - ""
  resources:
  - pods
  verbs:
  - delete
  - list
- apiGroups:
  - networking.k8s.io
  resources:
  - clustercidrs
  verbs:
  - create
  - get
  - list
  - update
- apiGroups:
  - ""
  - events.k8s.io
  resources:
  - events
  verbs:
  - create
  - patch
  - update
- apiGroups:
  - ""
  resources:
  - pods
  verbs:
  - get

Compare to the ClusterRole of yurt-manager(v1.4):
# ...
- apiGroups:
  - ""
  resources:
  - nodes
  verbs:
#  - delete # missing one
  - get
  - list
  - patch
  - update
  - watch # extra one
# - apiGroups: # missing one
#  - ""
#  resources:
#  - nodes/status
#  verbs:
#  - patch
#  - update
- apiGroups:
  - ""
  resources:
  - pods
  verbs:
  - create # extra one
  - delete
  - get
  - list
  - patch # extra one
  - update # extra one
  - watch # extra one
- apiGroups:
  - ""
  resources:
  - pods/status
  verbs:
#  - patch # missing one
  - update
# - apiGroups: # missing one
#  - networking.k8s.io
#  resources:
#  - clustercidrs
#  verbs:
#  - create
#  - get
#  - list
#  - update
# - apiGroups: # missing one
#  - ""
#  - events.k8s.io
#  resources:
#  - events
#  verbs:
#  - create
#  - patch
#  - update
# ...

But the node lifecycle controller in yurt-manager differs a lot from the one in kube-controller-manager v1.27.2 definitely.

clustercidrs

@crazytaxii Except  networking.k8s.io/clustercidrs resource, other missed rbac settings should be added to yurt-manager.
because networking.k8s.io/clustercidrs is used by node ipam controller in kube-controller-manager, and it is not needed by nodelifecycle controller.


Slider does not fill parent width

I'm trying to use this library's slider, but I need it to fill the entire width. Cloning the source code and modifying some code, I was able to fix the slider "track" to match parent width, but the ball doesn't start and end at the very end. Can somebody help me with this?

I figured it out. Check this StackOverflow post: http://stackoverflow.com/questions/34279260/android-music-seekbar-match-parent/34289061#34289061


[mtl] sticky compute passes

Prevents rebinding of all the state and resources in case of consecutive compute dispatches.
Also, tries to make a list of commands for graphics pipeline setup and pass it as one big thing (instead of calling pre.issue for each individual one). This is piggy-backing on the issue_many function of the base commit here.
PR checklist:

[ ] make succeeds (on *nix)
[x] make reftests succeeds
[x] tested examples with the following backends: metal
[ ] rustfmt run on changed code

hmm, I detect a performance regression here, investigating the cause now...

Good news - the regression is solely caused by the extra commit here, so I dropped it. It's an interesting piece that's not obvious to me: so building one gigantic chain of commands and recording them in one go is slower than calling pre.issue() for each individual one, contrary to my intuition. The news is good because it allows us to improve the performance expectations and revise all the places where we are currently using the long command chains.

bors=grovesNL

bors r=grovesNL


Configure DEB & RPM docker-ce repository urls as variables

Configured DEB & RPM docker ce repository urls as variables to allow use of a local mirror of the official repository.
Updated readme with ànsible-role docgen` command.
Proposed changelog for v1.12.0
Updated travis-ci config with --become parameter as a replacement for deprecated --sudo parameter

Hi @franklinkim, could you have a look at this PR and tell me if something has to be changed ?

Hi @franklinkim, could you have a look at this PR and tell me if something has to be changed ?

THX

THX

Thanks for merging. Is v1.12 automatically pushed to ansible galaxy after some time or does it require manual action to make that new version available ?

Thanks for merging. Is v1.12 automatically pushed to ansible galaxy after some time or does it require manual action to make that new version available ?

Should have been automatically but I gave it a little push.

Should have been automatically but I gave it a little push.


Configure Payment App

Can you also provide an option to set the payment app option like google pay, phone pe,
Currently by default WhatsApp payment is getting opened on click of the link

https://user-images.githubusercontent.com/26687933/131973831-8beff657-13dd-4208-b9cd-2bafc7ef335f.mp4
In my set-up it's working fine


Fix tests and linter

On branch feature/fix-tests
Changes to be committed:
modified:   pyproject.toml
modified:   quinn/dataframe_validator.py
modified:   quinn/schema_helpers.py
modified:   quinn/split_columns.py
modified:   quinn/transformations.py
modified:   tests/test_split_columns.py
Close #146

@SemyonSinchenko changes look good besides the one pipe operator thing I noticed, I think it's ready to be merged

@SemyonSinchenko changes look good besides the one pipe operator thing I noticed, I think it's ready to be merged

Thank you! I fixed it!