Use os sdk as dependency

Use open source sdk
Description
Use open source sdk, @adobe/asset-compute-sdk

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Closing in favor of https://github.com/adobe/generator-aio-app/pull/55, that I'll merge into this fork after it's merged in upstream.


Handle 422 from email alert api

Email Alert API can return a 422 (unprocessable) error. This adds more error handling to finder-frontend for this case.
Co-Authored-by: @MahmudH
Trello: https://trello.com/c/so20SdvH/808

@barrucadu thanks for your comments! I made those changes.


[contract-deploy] Store bytecode hash for a contract in addition to verification key hashes

Today we use the verification key hash to identify each function in a contract. However, this means that unconstrained code isn't committed to, so an app could provide the bytecode for a contract with altered unconstrained code (eg the code that interacts with the oracle) and the user would have no way to know. Same applies to unconstrained functions in a contract, a dapp could inject whatever code they want in those, and potentially use it to exfiltrate private information. We should commit to the bytecode hash of a contract somewhere, in addition to the verification key.

Comment from @zac-williamson:

I think we would need both
In the Kernel circuit, the user provides the Verification Key  of the function circuit as a private input. We validate that the key is correct by checking its hash against the vk hash in the contracts tree.
We still need to be able to do this check - a bytecode hash or a hash of vk + bytecode won’t cut it here

This issue may already be implemented but currently disabled. See also https://github.com/AztecProtocol/aztec-packages/issues/5860.


Roll Flutter Engine from 0338d1c7d7fe to 252553199fcd (2 revisions)

https://github.com/flutter/engine/compare/0338d1c7d7fe...252553199fcd
2024-12-10<EMAIL_ADDRESS>Roll Skia from e11d70745224 to<PHONE_NUMBER>22 (1 revision) (flutter/engine#57084)
2024-12-10<EMAIL_ADDRESS>Roll Skia from 3a58be4817a8 to e11d70745224 (2 revisions) (flutter/engine#57082)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC bdero<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Add caching to the CI

What should be done?
When we have to trigger a new CI run, e.g. because of flakiness https://github.com/0xPolygonMiden/miden-base/issues/566, the time in the CI is dominated by the compilation of the test binaries. Adding caching will speed up re-runs and save us time.
How should it be done?
Use the GH's actions artifacts/caches to save rust/cargo's artifacts
When is this task done?
When the CI can be re-executed without code changes, and the worker does not have to re-compile the binary
Additional context
No response

Related: caching was introduced into miden-node in this pull request.

@Mirko-von-Leipzig - could you open a similar PR here when you get a chance?

@Mirko-von-Leipzig - could you open a similar PR here when you get a chance?

How similar? All of the same things?

Yep - I think all the things from that PR should apply here too (though, comment wrapping should be here already).

Closed by #818.


Issue/141/simplereadme addresses #141

Addresses issue #141
Remove a lot of the duplicated text from README.md and instead just point to the Read The Docs page. Also, add the references to specific codes for RAIL that should be cited if used to the citing.rst section of the read the docs.

I just noticed the links are in the wrong format on the installation.rst page, I'm just going to fix those in this PR before merging rather than create a separate issue, just need to switch from the copied .md to .rst syntax for links.


Update minimum sdk to 24 instead of 30.

This pr is supposed to fix issue #31

The app is not optimized nor tested for sdk lower than 30.


Standardize hr/ input for owe and pay command

Standardize on accepting either real value / 0.5

Ive changed pay to accept multiples of 0.5. I think @DennieDan dont need to change her owe command.


Re-add accidentally removed hacking-on-gitea.zh-cn.md (#23297)

Backport #23297
The documentation hacking-on-gitea.zh-cn.md removed in #21627 accidently. This takes it back and merged some changes from #23289

Where does this document come from? It looks like an dinosaur 🦕 (too old, lol) ......

There is no vendor or Govendor today
hence the go get approach is no longer recommended. by English version


[FeatureRequest] rust-analyzer to support loading output dirs

See setting for VSCode https://github.com/rust-analyzer/rust-analyzer/issues/1964#issuecomment-600530924 such that rust-analyzer will look in OUT_DIR.
Opening this issue if another finds the implementation specifics for rust-analyzer here for nvim-lspconfig. And whether it's something that would become part of the default, or just updated docs to indicate how to include add it if desired

How common is having this enabled, and are there common cases where it should not be enabled? It's trivial to add right now.
require'lspconfig'.rust_analyzer.setup{
  settings = {
    rust_analyzer = {
      cargoFeatures = {
        loadOutDirsFromCheck = true;
      }
    }
  }
}

How common is having this enabled, and are there common cases where it should not be enabled? It's trivial to add right now.
require'lspconfig'.rust_analyzer.setup{
  settings = {
    rust_analyzer = {
      cargoFeatures = {
        loadOutDirsFromCheck = true;
      }
    }
  }
}

Thanks @mjlbach I wasn't aware of the settings value there. I'll do that in my local config

Thanks @mjlbach I wasn't aware of the settings value there. I'll do that in my local config

@damienpontifex Glad to help, but my question is mostly should this be on by default? I'm a casual rust user and have never used this

@damienpontifex Glad to help, but my question is mostly should this be on by default? I'm a casual rust user and have never used this

Probably shouldn't be by default.
I was only looking to use the setting with rust-bindgen and OUT_DIR as described https://rust-lang.github.io/rust-bindgen/tutorial-3.html
I can't imagine this is a common enough scenario to be having it set as true by default

Probably shouldn't be by default.
I was only looking to use the setting with rust-bindgen and OUT_DIR as described https://rust-lang.github.io/rust-bindgen/tutorial-3.html
I can't imagine this is a common enough scenario to be having it set as true by default


[Support Request]: Unable to update to latest version

Attempted Debugging

[X] I have read the debugging page

Searched GitHub Issues

[X] I have searched GitHub for the issue.

Describe the Scenario
Hello,
I am running Bookstack 21.04.3 on Ubuntu 18.04.6 LTS, all patches up to date.
PHP ver. 8.0.14
When I run git pull origin release && composer install --no-dev && php artisan migrate, I get the following:
remote: Enumerating objects: 1074, done.
remote: Counting objects: 100% (979/979), done.
remote: Compressing objects: 100% (431/431), done.
remote: Total 1074 (delta 632), reused 769 (delta 545), pack-reused 95
Receiving objects: 100% (1074/1074), 1.13 MiB | 10.63 MiB/s, done.
Resolving deltas: 100% (661/661), completed with 117 local objects.
From https://github.com/BookStackApp/BookStack

branch              release    -> FETCH_HEAD
009212ab..53f3cca8  release    -> origin/release
Updating 2d8698a2..53f3cca8
error: Your local changes to the following files would be overwritten by merge:
.env.example
.env.example.complete
.github/ISSUE_TEMPLATE/api_request.md
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/language_request.md
.github/translators.txt
.github/workflows/phpunit.yml
.github/workflows/test-migrations.yml
.gitignore
LICENSE
app/Actions/Activity.php
app/Actions/ActivityService.php
app/Actions/ActivityType.php
app/Actions/Comment.php
app/Actions/CommentRepo.php
app/Actions/Tag.php
app/Actions/TagRepo.php
app/Actions/View.php
app/Actions/ViewService.php
app/Api/ApiDocsGenerator.php
app/Api/ApiToken.php
app/Api/ApiTokenGuard.php
app/Api/ListingResponseBuilder.php
app/Application.php
app/Auth/Access/EmailConfirmationService.php
app/Auth/Access/ExternalAuthService.php
app/Auth/Access/ExternalBaseUserProvider.php
app/Auth/Access/Guards/ExternalBaseSessionGuard.php
app/Auth/Access/Guards/LdapSessionGuard.php
app/Auth/Access/Guards/Saml2SessionGuard.php
app/Auth/Access/Ldap.php
app/Auth/Access/LdapService.php
app/Auth/Access/RegistrationService.php
app/Auth/Access/Saml2Service.php
app/Auth/Access/SocialAuthService.php
app/Auth/Access/UserInviteService.php
app/Auth/Access/UserTokenService.php
app/Auth/Permissions/EntityPermission.php
app/Auth/Permissions/JointPermission.php
app/Auth/Permissions/PermissionService.php
app/Auth/Permissions/PermissionsRepo.php
app/Auth/Permissions/RolePermission.php
app/Auth/Role.php
app/Auth/SocialAccount.php
app/Auth/User.php
app/Auth/UserRepo.php
app/Config/api.php
app/Config/auth.php
app/Config/broadcasting.php
app/Config/cache.php
app/Config/database.php
app/Config/debugbar.php
app/Config/dompdf.php
app/Config/filesystems.php
app/Config/hashing.php
app/Config/logging.php
app/Config/mail.php
app/Config/queue.php
app/Config/saml2.php
app/Config/services.php
app/Config/session.php
app/Config/setting-defaults.php
app/Config/snappy.php
app/Console/Commands/CleanupImages.php
app/Console/Commands/ClearViews.php
app/Console/Commands/CopyShelfPermissions.php
app/Console/Commands/CreateAdmin.php
app/Console/Commands/DeleteUsers.php
app/Console/Commands/RegenerateSearch.php
app/Console/Commands/UpdateUrl.php
app/Console/Commands/UpgradeDatabaseEncoding.php
app/Console/Kernel.php
app/Entities/BreadcrumbsViewComposer.php
app/Entities/EntityProvider.php
app/Entities/Models/Book.php
app/Entities/Models/BookChild.php
app/Entities/Models/Bookshelf.php
app/Entities/Models/Chapter.php
app/Entities/Models/Deletion.php
app/Entities/Models/Entity.php
app/Entities/Models/HasCoverImage.php
app/Entities/Models/Page.php
app/Entities/Models/PageRevision.php
app/Entities/Models/SearchTerm.php
app/Entities/Repos/BaseRepo.php
app/Entities/Repos/BookRepo.php
app/Entities/Repos/BookshelfRepo.php
app/Entities/Repos/ChapterRepo.php
app/Entities/Repos/PageRepo.php
app/Entities/Tools/BookContents.php
app/Entities/Tools/ExportFormatter.php
app/Entities/Tools/Markdown/CustomStrikeThroughExtension.php
app/Entities/Tools/Markdown/CustomStrikethroughRenderer.php
app/Entities/Tools/PageContent.php
app/Entities/Tools/PageEditActivity.php
app/Entities/Tools/PermissionsUpdater.php
app/Entities/Tools/SearchIndex.php
app/Entities/Tools/SearchOptions.php
app/Entities/Tools/SearchRunner.php
app/Entities/Tools/ShelfContext.php
app/Entities/Tools/SiblingFetcher.php
app/Entities/Tools/SlugGenerator.php
app/Entities/Tools/TrashCan.php
app/Exceptions/ApiAuthException.php
app/Exceptions/ConfirmationEmailException.php
app/Exceptions/FileUploadException.php
app/Exceptions/Handler.php
app/Exceptions/HttpFetchException.php
app/Exceptions/ImageUploadException.php
app/Exceptions/JsonDebugException.php
app/Exceptions/LdapException.php
app/Exceptions/LoginAttemptEmailNeededException.php
app/Exceptions/LoginAttemptException.php
app/Exceptions/MoveOperationException.php
app/Exceptions/NotFoundException.php
app/Exceptions/NotifyException.php
app/Exceptions/PermissionsException.php
app/Except
Aborting

I had read that this could be resolved by a git reset, which I have tried.  After running the reset and trying to update however, I get a wall of red highlighted text, and Bookstack (or Apache) stops and won't start again.  I end up having to restore from a snapshot.  I also read that this might have something to do with commiting changes prior to updating, but I'm uncertain on how to do that.  Any help would be greatly appreciated
Exact BookStack Version
21.04.3
Log Content
No response
PHP Version
8.0.14
Hosting Environment
Ubuntu 18.04.6 LTS, using the installation script.
No proxies
VM machine hosted in VSphere environment

Hi @plummerr,
This could likely be caused by permissions causing confusing in the version control system. If so, this can be easily solved but could you provide the output of the below command so we can verify that causation? Just run this from within your BookStack install folder:
git diff LICENSE

diff --git a/LICENSE b/LICENSE
old mode 100644
new mode 100755

@plummerr Thanks, Looks like this is permissions. You can run the following, from your bookstack install folder, to ensure permissions won't affect code control:
git config core.fileMode false

After that you can try updating again, hopefully now with success.

Well, making some progress...but now I get this
Deprecation Notice: Required parameter $path follows optional parameter $schema in /usr/share/php/JsonSchema/Constraints/UndefinedConstraint.php:62
Deprecation Notice: Required parameter $path follows optional parameter $schema in /usr/share/php/JsonSchema/Constraints/UndefinedConstraint.php:108
Deprecation Notice: Method ReflectionParameter::getClass() is deprecated in /usr/share/php/Composer/Repository/RepositoryManager.php:129
Deprecation Notice: Method ReflectionParameter::getClass() is deprecated in /usr/share/php/Composer/Repository/RepositoryManager.php:129
Cannot create cache directory /home/administrator/.cache/composer/repo/https---packagist.org/, or directory is not writable. Proceeding without cache
Cannot create cache directory /home/administrator/.cache/composer/files/, or directory is not writable. Proceeding without cache
In ArrayLoader.php line 44:
Unknown package has no name defined ([{"name":"aws/aws-sdk-php","version":"3.178.6","versi
on_normalized":"<IP_ADDRESS>","source":{"type":"git","url":"https://github.com/aws/aws-sdk
-php.git","reference":"0aa83b522d5ffa794c02e7411af87a0e241a3082"},"dist":{"type":"zip","url
":"https://api.github.com/repos/aws/aws-sdk-php/zipball/0aa83b522d5ffa794c02e7411af8
7a0e241a3082","reference":"0aa83b522d5ffa794c02e7411af87a0e241a3082","shasum":""},"require"
:{"ext-json":"","ext-pcre":"","ext-simplexml":"*","guzzlehttp/guzzle":"^5.3.3|^6.2.1|^7.
0","guzzlehttp/promises":"^1.4.0","guzzlehttp/psr7":"^1.7.0","mtdowling/jmespath.php":"^
This is just a snippet of it, and it's the red highlighted text that I mentioned earlier.

Hi @plummerr,
Is sounds like you might be using an out of date version of composer relative to your PHP version, possibly the composer package provided in Ubuntu. If that's in use I'd advise removing that (sudo apt remove composer). Then follow the composer site to setup the latest version: https://getcomposer.org/download/
Note the sudo mv composer.phar /usr/local/bin/composer command they show to globally install it.

I don't know how I screwed up the composer install the first time, so I started all over like you suggested.
We are good to go!  I appreciate all your help

Thank you for this helpful post, I had the same problem.
This has helped me.
Thank you very much.


openshit build failed.help

peerflix-server-1.log

From what I can tell it hasn't failed -- it's just stuck.


Don't export dart:async by the Google Maps plugin.

This was done in #463, not sure why, but I don't think we should export
it (and it makes Dartdoc complain).

Agreed.


[Bug] Expander causes exception in Embedded ContentPage

Description
Get System.InvalidOperationException thrown when using Expander in an Embedded ContentPage in a Xamarin NativeApp
Steps to Reproduce

Create Xamarin Android App
Add Embedded Content Page
Add Expander and enable flag Forms.SetFlags("Expander_Experimental");

Expected Behavior
Expander can be used in embedded page
Actual Behavior
System.InvalidOperationException: 'The class, property, or method you are attempting to use ('Expander') is part of Expander; to use it, you must opt-in by calling Forms.SetFlags("Expander_Experimental") before calling Forms.Init().'
Basic Information

Version with issue:
Last known good version:
NuGet Packages: Xamarin.Forms <IP_ADDRESS>1
Affected Devices:

Environment
Microsoft Visual Studio Enterprise 2019 Preview
Version 16.9.0 Preview 3.0
VisualStudio.16.Preview/16.9.0-pre.3.0+30914.41
Microsoft .NET Framework
Version 4.8.04084
Installed Version: Enterprise
Visual C++ 2019   00435-60000-00000-AA337
Microsoft Visual C++ 2019
ASP.NET and Web Tools 2019   16.9.525.50092
ASP.NET and Web Tools 2019
ASP.NET Core Blazor Language Services   16.0.19307.2
ASP.NET Core Razor Language Services   <IP_ADDRESS>5602+00b07fe95231ae5d6327076da72f1091405cfd5e
Provides languages services for ASP.NET Core Razor.
ASP.NET Web Frameworks and Tools 2019   16.9.525.50092
For additional information, visit https://www.asp.net/
Azure App Service Tools v3.0.0   16.9.525.50092
Azure App Service Tools v3.0.0
C# Tools   3.9.0-3.21056.4+e5eb71d6714749bccf91d9268ece78cd5d4d43ea
C# components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.
Common Azure Tools   1.10
Provides common services for use by Azure Mobile Services and Microsoft Azure Tools.
Extensibility Message Bus   1.2.6 (master@34d6af2)
Provides common messaging-based MEF services for loosely coupled Visual Studio extension components communication and integration.
IntelliCode Extension   1.0
IntelliCode Visual Studio Extension Detailed Info
Microsoft Continuous Delivery Tools for Visual Studio   0.4
Simplifying the configuration of Azure DevOps pipelines from within the Visual Studio IDE.
Microsoft JVM Debugger   1.0
Provides support for connecting the Visual Studio debugger to JDWP compatible Java Virtual Machines
Microsoft Library Manager   2.1.113+g422d40002e.RR
Install client-side libraries easily to any web project
Microsoft MI-Based Debugger   1.0
Provides support for connecting Visual Studio to MI compatible debuggers
Microsoft Visual C++ Wizards   1.0
Microsoft Visual C++ Wizards
Microsoft Visual Studio Tools for Containers   1.1
Develop, run, validate your ASP.NET Core applications in the target environment. F5 your application directly into a container with debugging, or CTRL + F5 to edit & refresh your app without having to rebuild the container.
Microsoft Visual Studio VC Package   1.0
Microsoft Visual Studio VC Package
Mono Debugging for Visual Studio   16.9.7 (df23ba6)
Support for debugging Mono processes with Visual Studio.
NuGet Package Manager   5.9.0
NuGet Package Manager in Visual Studio. For more information about NuGet, visit https://docs.nuget.org/
ProjectServicesPackage Extension   1.0
ProjectServicesPackage Visual Studio Extension Detailed Info
SQL Server Data Tools   16.0.62012.31170
Microsoft SQL Server Data Tools
TypeScript Tools   16.0.30106.2001
TypeScript Tools for Microsoft Visual Studio
Visual Basic Tools   3.9.0-3.21056.4+e5eb71d6714749bccf91d9268ece78cd5d4d43ea
Visual Basic components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.
Visual F# Tools   16.9.0-beta.21052.1+77c36dad3d5702c4eb3cf51f99d8643d4069a758
Microsoft Visual F# Tools
Visual Studio Code Debug Adapter Host Package   1.0
Interop layer for hosting Visual Studio Code debug adapters in Visual Studio
Visual Studio Container Tools Extensions   1.0
View, manage, and diagnose containers within Visual Studio.
Visual Studio Tools for Containers   1.0
Visual Studio Tools for Containers
VisualStudio.DeviceLog   1.0
Information about my package
VisualStudio.Foo   1.0
Information about my package
VisualStudio.Mac   1.0
Mac Extension for Visual Studio
Xamarin   <IP_ADDRESS> (d16-9@3bfa0b5)
Visual Studio extension to enable development for Xamarin.iOS and Xamarin.Android.
Xamarin Designer   <IP_ADDRESS>6 (remotes/origin/c4842c761b9b6a95407f72278ca7fb42f8f7fdf0@c4842c761)
Visual Studio extension to enable Xamarin Designer tools in Visual Studio.
Xamarin Templates   16.9.56 (96d1814)
Templates for building iOS, Android, and Windows apps with Xamarin and Xamarin.Forms.
Xamarin.Android SDK   <IP_ADDRESS> (d16-9/f908d16)
Xamarin.Android Reference Assemblies and MSBuild support.
Mono: 5e9cb6d
Java.Interop: xamarin/java.interop/d16-9@1d382be
ProGuard: Guardsquare/proguard/v7.0.1@912d149
SQLite: xamarin/sqlite/3.32.2@cfe06e0
Xamarin.Android Tools: xamarin/xamarin-android-tools/main@ad80a42
Xamarin.iOS and Xamarin.Mac SDK   <IP_ADDRESS> (f4c9327fa)
Xamarin.iOS and Xamarin.Mac Reference Assemblies and MSBuild support.
Show/Hide Visual Studio info

Build Logs

Screenshots

Reproduction Link

Workaround

System.InvalidOperationException: 'The class, property, or method you are attempting to use ('Expander') is part of Expander; to use it, you must opt-in by calling Forms.SetFlags("Expander_Experimental") before calling Forms.Init().'

This isn't needed anymore, this was a Forms thing. Chances are that this now works with the Xamarin Community Toolkit. Please update Forms to the latest version, install the latest stable XCT and try again. If it still not works please open a new issue.
Thanks!


build: Update RTSP server image and version

replace aler9/rtsp-simple-server with bluenviron/mediamtx, and upgrade to v1.8.2
closed #272

If your build fails  due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[x] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?)
[x] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions

New Dependency Instructions (If applicable)

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

Project coverage is 4.75%. Comparing base (068bc12) to head (655e03c).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@          Coverage Diff          @@
##            main    #334   +/-   ##
=====================================
  Coverage   4.75%   4.75%
=====================================
  Files          8       8
  Lines       1305    1305
=====================================
  Hits          62      62
  Misses      1243    1243

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


fix: crash in diagnostics consumer

Small bug introduced in 4fdce02df7046b7f8be8b112377d8c65593f2553 that broke the diagnostics consumer.


fix missing logic for multi-line-string

Description
I think this resolves #18005.
It was far easier than I thought. That said, I worry it was so easy.
I don't really understand how a DBOptionsAttributes::MultiLineString is any different from a DBOptionsAttributes::String. The public setter/getter methods are identical though somehow the underlying stored thing is a stringVector instead of a std::string. That said, I don't see any logic that somehow manages the contents of the option's value as a vector of strings. If you compare the logic for MultiLineString to, for example, Enum which also uses an underlying stringVector for storage, there are remarkable differences.
I think the goal of DBOptionsAttributes:MultiLineString was for a string-valued option in DBOptionsAttributes to have embedded new-lines (and maybe carriage returns) so you could have a string which when it got printed to the user or when the user typed it in to be digested by VisIt, it could look like some jasony or yamly looking thing...
{
   foo: 1,
   bar: "mark",
   coords: [0,1,2]
}

So that each line of the string would be captured as one of the entries in the underlyng stringVector associated with it. However, there is no logic in DBOptionsAtrributes.code that does any of that.
So, given how it behaves, what MultiLineString does is already possible in the cli using Python's """ operator. Next, I don't think there is any support in VisIt's GUI for rendering or editing such a string in the GUI widget that gets created for DBOptionsAttributes for either export or for read.
This is all making me think MultiLineString should be removed (or maybe tested and fixed if its not working) because its just duplicating other functionality we already have.

Type of change

[x] Bug fix~~
[ ] New feature
[ ] Documentation update
[ ] Other

How Has This Been Tested?
Manually on my macOS
Checklist:

[x] I have followed the style guidelines of this project.~~
[x] I have performed a self-review of my own code.~~
[x] I have commented my code where applicable.~~
[ ] I have updated the release notes.
[ ] I have made corresponding changes to the documentation.
[ ] I have added debugging support to my changes.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have confirmed new and existing unit tests pass locally with my changes.
[ ] I have added new baselines for any new tests to the repo.
[x] I have NOT made any changes to protocol or public interfaces in an RC branch.~~
[x] I have assigned reviewers (see VisIt's PR procedures for more information).~~

@biagas I just added you. If it appears to you that your work supercedes this, please feel free to close.

@markcmiller86  Hopefully to clarify confusion over MultiLineString, and why it is declared in DBOptionsAttributes as a stringVector, but is treated as a string. In fact, all of the types are stored in vectors, because a single DBOptionsAttributes class may contain multiple options of the same type:  there may be several String options, several int options, etc.
If there ever came a need for MultiLineString to be treated as a vector of strings, then the underlying type in DBOptionsAttributes would be changed to std::vector < std::vector < std::string > >.
I believe we came to the conclusion in the discussion comments for #18005, that using std::string as the underlying type for MultiLineString suffices as long as the newline chars that make it a 'multi-line string' are preserved throughout it's lifetime in VisIt,  Correct?

I believe we came to the conclusion in the discussion comments for #18005, that using std::string as the underlying type for MultiLineString suffices as long as the newline chars that make it a 'multi-line string' are preserved throughout it's lifetime in VisIt, Correct?

Ok, that sounds good though I think we want all whitespace to be preserved which includes newlines.

Fixed with #18085.


Can't Install

$make
curl -kLO https://github.com/skywind3000/ECDICT-ultimate/releases/download/1.0.0/ecdict-ultimate-csv.zip
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100   649  100   649    0     0    891      0 --:--:-- --:--:-- --:--:--   891
100 93.7M  100 93.7M    0     0  1763k      0  0:00:54  0:00:54 --:--:-- 2032k
unzip ecdict-ultimate-csv.zip && mv ultimate.csv ecdict.csv
Archive:  ecdict-ultimate-csv.zip
inflating: ultimate.csv
python ./ecdicts.py ./ecdict.csv ecdict.txt
dictfmt --utf8 --allchars -s ECDICT -u https://github.com/skywind3000/ECDICT-ultimate -j ecdict < ./ecdict.txt && dictzip ecdict.dict
/bin/sh: 1: dictfmt: not found
make: *** [Makefile:24: ecdict.index] Error 127

/bin/sh: 1: dictfmt: not found

dictd is both dependency and make dependency,  should be installed before building.

/bin/sh: 1: dictfmt: not found

dictd is both dependency and make dependency,  should be installed before building.


Consider adding a license?

Thanks for putting this together, @MazenAB!
I really like the content, and it's a fantastic way to get into coding with Glue.
I'd to contribute to some of the Python code examples in your instructions with some of the f-string practices to enhance readability as well as some formatting.
Would you consider adding a license to this repo so I can send you a contribution?

@djfurman, Thank you so much for your offer. I created this repo on a temporary basis and will have it private at the end of today. I can provide more info privately

Thank you!


plugin is not installed

Installation failed because the extension:  'JamesBirtles.svelte-vscode' was not found compatible with the current version of VS Code '1.19.3'
X64 architecture
OS Linux Mint

That version of vscode is almost a year old, I would suggest updating it


Compiling help: [Haxelib Problems / HScript Problems]

Hello, i'm getting these errors with compiling the last source code file branch main. How to fix?
[ x ] Windows
[ ] Linux
[ ] Mac

Update.bat crash after installing hxdiscord_rpc

Hello, i'm getting these errors with compiling the last source code file branch main. How to fix?
[ x ] Windows [ ] Linux [ ] Mac

For the very last one, go to MusicBeatState.hx, then line 124. Make sure to add a parenthesis symbol wherever needed.
As for the rest, it might have to do with the macros. I'm not exactly sure what, since I don't use macros usually, however.

Update.bat crash after installing hxdiscord_rpc

Are you sure you have git-scm installed on your system? It is a possibility that you might not have it.

yeah that actually happens cause you gotta update the hscript improved library

Update to 4.2.5

(closing this since no answers for a while, feel free to reopen if you need

Hello, sorry if i didn't answer.. I have latest haxe ver, latest hscript improved ver, and latest git ver, so it is not git


Support for version 1.1?

Hi, would it be possible to add support for JSON Feed 1.1?

Yes, I'd like to. Not sure I've got time at the moment, but I'll get to it when I can.

Do you mind my taking a shot and sending you a pull request ?

Not at all, I'd be glad!

I created a pull request for you


chore(Account Linking): fix for launch blocking account linking behaviour

The type of this PR is: Bugfix

This PR resolves []
Description

Fixes weird Account linking issues.

https://user-images.githubusercontent.com/18648835/158218345-e8c8d922-49e2-4d28-b8cd-48e88e7727a0.mp4

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Fix for account linking behaviour bug - kizito

This PR contains the following changes:

Dev changes (Fix for account linking behaviour bug - kizito)

  Generated by :no_entry_sign: dangerJS against 7329fd4eef72412da303f695685be073fb08a07b


Move to using raw json response from status with OCM

Once this merges, we should try out the change and move our implementation to support it. This will allow us to get more than one address back and also awkward translations between the work status and a gateway status
https://github.com/open-cluster-management-io/work/pull/202
We want to know if this works as expected and also when it will be available in a release of OCM

This issue is stale because it has been open for 30 days with no activity.


Make Draft consider the span tag as a valid block type

I m trying to convert HTML markup that contains span tags using convertFromHTML. The problem that span is not a Draft block type, for example Hello World will be considered as one block. I tried to set up a custom block rendering following the official documentation but in vain, blockStyleFn, blockRenderMap, BlockRendererFn are all implemented to make changes to valid block type.
I would appreciate your help

Hi @karim10 - I see that convertFromHTML has some undocumented parameters which you can pass to customize the blockMap. I believe this could be used to add support for custom blocks. I'll try to whip up an example to confirm this.
https://github.com/facebook/draft-js/blob/514490b1322e1c123524eae97d4aea25b4da16ce/src/model/encoding/convertFromHTMLToContentBlocks.js#L609-L613


exit status 1: Access is denied.

How To Reproduce:
Windows 10
An old nvm installation working correctly. (not sure of the version I had, 1.1.7 ?)
Close Visual Studio Code.
Try to upgrade this old nvm with nvm-update.zip
After apparently a successful upgrade occurs but now nothing works anymore with node and it complains of permissions errors.
Uninstall nvm from Windows using the Settings > Apps & features / Add or remove programs
Install 1.1.9 from nvm-setup.zip
Re-launch Visual Studio Code
nvm install 16.14.2
nvm use 16.14.2 <= this fails with access is denied
exit status 1: Access is denied.
Actual Result
exit status 1: Access is denied.
Expected Behavior:
No access is denied error

Ah I guess this is an expected change after 1.1.7
https://github.com/coreybutler/nvm-windows/wiki/Common-Issues#permissions-exit-1-exit-5-access-denied-exit-145
Doesn't running Windows in Developer mode presenting more security risks than before using 1.1.7 without it?

By popular demand, NVM4W 1.1.8 introduced support for developer mode. In other words, you won't receive a UAC prompt to elevate permissions if it's not explicitly necessary.
How come when I run "nvm use xyz" in Visual Studio Code I am not prompted for UAC and instead it fails. I think if it the command fails it was explicitly necessary, yet I was not prompted?

These symptoms all sound like they're versions of https://github.com/coreybutler/nvm-windows/issues/767

VS Code won't run anything as an admin unless it is opened with an account that has admin privileges. This is more of a problem of VS Code than with NVM4W. The UAC prompts were a workaround for VS Code, so use NVM v1.1.7 if you need this capability in VS Code. I'm planning to bring UAC prompts back in rt, so I'm tryin to avoid further changes in NVM4W.
I'm closing this, in part because it's a duplicate of #717 and in part because there isn't anything that can be done specifically for VS Code other than reverting back to UAC prompts.

I'm having this issue now after updating. The problem is I run this on a corporate-owned computer and even getting privileges to run anything as an admin is already a routine struggle on its own. Also, there is no "Developers" settings page in Windows 10 for me, so not sure what that is.
The UAC prompts were ideal because I could just run it in my existing terminal and wasn't forced to have to open a totally separate elevated cmd.exe prompt (which stinks for me since I strongly prefer Windows Terminal with bash). Is there a discussion/PR/issue for this change? I'd love to add my vote there to at least have UAC capability since the alternative doesn't appear to be viable for me.

Since at least the UAC dialog is far better than being rejected with a vague "access is denied" error 😕

same issue. i could bypass it by running PowerShell with admin privileges, but its not really ideal.

powershell with admin role also failed

@quantrpeter - that is not the same issue. That is a download failure. The npm repo for 6.14.17 has a bad link.

@coreybutler The thing is that it's still a problem. NVM 1.1.7 cannot install node versions above 16.13.2. And if upgrading NVM to 1.1.9, then UAC is broken.
Also I'm seeing new people getting introduced to Node and the first error they come across are permission denied without being prompted for elevated credentials (UAC dialog). It's not an edge case. Also the issue was closed partially because it's duplicated but both issues have been closed without resolution.
I'm wondering if this could be fixed in a 1.1.10 release please? Thanks

@coreybutler I just noticed you did exactly that in 1.1.10 and fixed UAC. So this ticket can be considered fixed and maybe the tag "edge case" removed considering the release notes mentioning "Many users have requested UAC prompts be reinstated."


Submodule menu can not be used!

GitLens Version:6.0.0
VSCode Version:1.18
OS Version:win10

Steps to Reproduce:
1.Select the submodule file in the file list and right-click on the Compare menu,not work!

The editor menu in the submodule file is no longer visible.
gitlens-5.7.1 is support!

@tiansin do you have a public repo with an example where I can try this out?

@eamodio

test.zip

@tiansin great, thank you. A fix for this will be in the next release -- sorry for the inconvenience. The refactoring for the multi-root workspace support affected a lot of things and this one slipped through the cracks unfortunately :(


WIP: Proposal of the v2 config syntax

See https://github.com/egnyte/gitlabform/issues/82 for more info.

The list of issues and PRs that require, or would benefit from the most, introducing backward incompatible changes to v1 got pretty long recently.
That’s why I plan to introduce only some changes to the syntax in v2.0 to release it faster and open the possibility to add more features based on it in v2.1+.


feat(cli,store): fetch table-specific logs

closes https://github.com/latticexyz/mud/issues/1658
needed in https://github.com/latticexyz/mud/pull/3171

lifted + improved test setup in https://github.com/latticexyz/mud/pull/3260, but otherwise this is unchanged from approval


add HPI SNR calculation and plotting

adds computation of SNR of the HPI coils.

still a draft because I don't have tests yet, but I think it's worth another quick look @larsoner.  Locally it passes existing tests and runs under Numba / jit, so right now I'm just looking for general feedback on whether the API makes sense / if you spot any obvious non-DRY parts of the code.

@larsoner this is still WIP because I want to add the new viz function to an example/tutorial. But meanwhile, there seems to be some numerical instability when dealing with gradiometers.  Here are plots of 4 SNR computations of the same data, run one after another in the same ipython session:

Is this likely to be a problem with gradiometers in general? Or something weird about the sample data?  Here's the snippet:
import os.path as op
import matplotlib.pyplot as plt
import mne

data_path = mne.datasets.testing.data_path(download=False)
chpi_fif_fname = op.join(data_path, 'SSS', 'test_move_anon_raw.fif')
raw = mne.io.read_raw_fif(chpi_fif_fname, allow_maxshield='yes')
for ix in range(1, 5):
    result = mne.chpi.compute_chpi_snr(raw)
    fig = mne.viz.plot_chpi_snr(result)
    fig.savefig(f'figure_{ix}.png')
plt.close('all')

cc @jjnurminen, as this implementation is based on your hpimon code.  Maybe you have some insight into the numerical instability seen above?

I ask

Is this likely to be a problem with gradiometers in general? Or something weird about the sample data?

and as usual, the answer is "neither, it is a bug in the code that I wrote" 😆

copy-paste error, was overwriting the mag results with grad results, so grad results remained as whatever np.empty initialized to.

OK I think this is ready for review @larsoner.  A couple questions to prime the pump:

for compatibility with Report, does the plotting function need to accept an axes parameter?
I'm worried that the tutorial is getting too computationally long:

computation time summary:
    - ../tutorials/preprocessing/59_head_positions.py:                        22.89 sec   237.9 MB

Do you think it makes sense to do the SNR demo in a short example and cross-ref it from the tutorial?

for compatibility with Report, does the plotting function need to accept an axes parameter?

I'm not sure about Report, but most of our plotting functions support this param so I'd add it in this PR anyway.

I'm worried that the tutorial is getting too computationally long:

20s isn't so long, but if you can make the same point by operating on a cropped segment of data (and I think you probably can?) then that's worth doing

CIs all green! Here's the new tutorial section: https://31423-1301584-gh.circle-artifacts.com/0/dev/auto_tutorials/preprocessing/59_head_positions.html#computing-snr-of-the-hpi-signal

LGTM other than needing latest.inc with your name and @jjnurminen's

found a bug in the plotting legend, which shows up in the tutorial.  Fixing now.

CIS are all green again, and the legend is fixed:  https://31437-1301584-gh.circle-artifacts.com/0/dev/auto_tutorials/preprocessing/59_head_positions.html#computing-snr-of-the-hpi-signal

Thanks @drammock !


Bug? if only enum is selected no TS file output?

HI @frhagn
I am having some unexpected results. I have a c# file with a nested enum in the class contained within the file. It turns out that I want to export just the nested enum but not the class that contains it:
public class DontExportClass {
    [ExportToTypescript]
    public enum EnumToExport {
    }

    public class OtherEmbeddedClass {
    }
}

If any class triggers output, then this enum will be processed. But if there is no class in the file that triggers output, then this enum does not trigger output and there is no file generated.
You can see my filters and functions over on #134. I even added a line that selects enums like I did with classes hoping that having the $Enum[] selector would spit it out, but that didn't work either.
Here is my class and enum selectors:
$Classes(c => ExportToTypeScriptFilter(c))[]
$Enums(e => ExportToTypeScriptFilter(e))[]
In this example, if I add the [ExportToTypescript] to the class DontExportClass or OtherEmbeddedClass, then I get a file spit out. But if I don't flag these classes for export with the attribute, then the file doesn't get created.
I see I added the enum processing after I posted in #134 so here is the ExportToTypeScriptFilter for enums:
    bool ExportToTypeScriptFilter(Enum e) {
        bool select = false;
        foreach (var attr in e.Attributes) {
            if (attr.Name == "ExportToTypescript") {
                select = true;
                break;
            }
        }
        return select;
    }

And the AllEnums function:
    IEnumerable<Enum> EnumsToExportToTypescript(File f)
    {
        List<Enum> enums = f.Enums.Where(ExportToTypeScriptFilter).ToList();
        foreach (var c in f.Classes) {
            enums = enums.Concat(c.NestedEnums.Where(ExportToTypeScriptFilter)).ToList();
        }
        return enums;
    }

These guys work as long as the class selection picks a class to output.  I have put in debugging and verified that the enum filter is picking up the correct attribute on the enum and returning true (I can get debug info when a class is output and a file is generated).
Is this a bug? Or something that can be enhanced?
Thanks
Greg

Ok I think I am wrapping my head around this.
I am now working under the assumption that the $Classes is the almighty selector. If it returns false for all classes, then no file will be output.
So I now do this:
    //////////////////////////////////////////////////////////////////////////////////
    // select the classes to convert to typescript
    bool ClassShouldBeExported(Class c) {
        // A class should be processed (only the relevant bits will be exported) if
        // 1: it contains the attribute export to typescript or export with knockout
        // 2: one of its nested classes (recurrsion) contains the same two attributes
        // 3: one of its nested enums contain the export to typescript attribute
        // we short circuit as soon as possible
        // This is used by the $Classes() filter to see if there are any classes within the file
        // that need processing. We then ignore that list and go onto the other filters with the
        // actual code. This is so we can write the typescript code snippets once
        if (ExportToTypeScriptFilter(c)) {
            return true;
        }
        foreach (var nc in c.NestedClasses) {
            if (ClassShouldBeExported(nc)) {
                return true;
            }
        }
        foreach (var ne in c.NestedEnums) {
            if (ExportToTypeScriptFilter(ne)) {
                return true;
            }
        }
        return false;
    }
    // These are the filters used to determine the relavent parts of the file to transform
    bool ExportToTypeScriptFilter(Class c) {
        bool select = false;
        foreach (var attr in c.Attributes) {
            if (attr.Name == "ExportToTypescript" || attr.Name == "ExportToTypescriptWithKnockout") {
                select = true;
                break;
            }
        }
        return select;
    }
    bool ExportToTypeScriptFilter(Enum e) {
        bool select = false;
        foreach (var attr in e.Attributes) {
            if (attr.Name == "ExportToTypescript") {
                select = true;
                break;
            }
        }
        return select;
    }

And then I have this line as the first line of the body in the .tst file:
    // Ok here we go. The first line determines if there is a class in this file worth outputing. If it
    // doesnt find any classes or bits of class to be exported, then a corresponding .TS file wont be output
    // this is why ClassShouldBeExported looks at all the bits and pieces (nested classes and enums)
}$Classes(c => ClassShouldBeExported(c))[]
$Enums(e => ExportToTypescript(e))[]

Notice that in ClassShouldBeExported, I look at the nested classes and nested enums. And I recurse on the nested classes.
So now with this, I can put the attribute [ExportToTypescript] on the top level class, or just one of the nested classes or on just one of the nested enums and the .tst file will pick that up and output a TS definition for that.
BTW @frhagn, I would be happy to supply the two .tst files that I am using if you want to put them up as examples of a more complex and complete solution in your documentation.


name and primary_name mismatch in service metadata for services with upper/mixed case.

Rancher Versions: v1.5.0-alpha1
Steps to reproduce the problem:
Create a service with name having mixed/upper case.
Fetch rancher metadata using curl rancher-metadata/latest/services/<id>/
name shows in all lower case
primary_service_name preserves the case.

@alena1108 , network policy manager relies on name and primary_service_name to be the same to determine that service is a primary/sidekick services.

test with metadata v0.8.5

Tested with build from master.
In service metadata , name and primary_name are in lowercase  for services with upper/mixed case.


_enableSnapshot loads upsert segments from snapshot even if _enablePreload is disabled

Recently, on one of our customer tables we added a comparison column where enableSnapshot was true but enablePreload was false. After adding the comparison column and restarting, the expected records for a key were still not returned. Upon further investigation, I discovered that with enableSnapshot set to true, the snapshot is read to load a segment. This behavior is counterintuitive since we already have a configuration enablePreload that handles this. Consequently, we had to disable enableSnapshot, restart, then re-enable enableSnapshot, and restart again to update one configuration (2 restarts).
IMO we should not perform a preload-type operation unless enablePreload is set explicitly. We can remove these lines during addSegment flow to achieve this: https://github.com/apache/pinot/blob/8df722a22f089d088bfb03e2b8aa49549bc4c38c/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/upsert/BasePartitionUpsertMetadataManager.java#L436-L442
cc @Jackie-Jiang @klsince what are your thoughts on this?

please consider the other scenarios when we need the snapshot.

Closing as the behaviour is fixed in #13579


removed docker ee section

alongside https://github.com/ondat/documentation/pull/57 I have removed the Docker EE section from the Best Practices document.

LGTM 👍


Add logging feature

As per title, this PR adds the text logging utility to the BAF library. In addition, it also adds CI build workflows.
The feature relies on the [bipedal-locomotion-framework](https://github.com/ami-iit/bipedal-locomotion-framework)'s one, just changing the name/prefix of the logged message to baf instead of blf.
More specifically, the feature relies on the following PR of blf which is merged but not yet released: https://github.com/ami-iit/bipedal-locomotion-framework/pull/582.
For this reason the tag used is the commit c1cb3fa3329cae7515de6d453bc099dde2cd0645 and the CI, which mostly relies on conda-based dependencies, installs blf from source.
Related TODOs will be removed once after the next version of blf is released.

Somehow I managed to find the cause of the failure on windows https://github.com/ami-iit/biomechanical-analysis-framework/actions/runs/4042357636/jobs/6949962545#step:11:78.
Turns out that Windows paths have a restriction of 260 chars (see here).
Now, the file that is being attempted to generate in the step above mentioned is: D:/a/biomechanical-analysis-framework/biomechanical-analysis-framework/bipedal-locomotion-framework/build/src/YarpUtilities/CMakeFiles/yarp_idl_to_dir/thrifts/BipedalLocomotion/YarpUtilities/vectorscollection//BipedalLocomotion/YarpUtilities/VectorsCollection.cpp, which has 263 chars.
In https://github.com/ami-iit/biomechanical-analysis-framework/pull/10/commits/6c943a7a886a83c50a26e178f995363725b0d15a I shortened the name of the directory where bipedal-locomotion-framework is cloned to blf to fix this issue.

Finally all checks are passing!

The logger and initial skeleton are there, for me we can merge.
I don't know if it makes any sense to add a line in the CHANGELOG such that at the first release of the library, we have a list of features.

Done @lrapetti. Thanks for the review, merging.

Super cool, first PR with implementation elements, well done!


Verbose mode should tell you downloader invoked.

Checklist

[X] I'm reporting a feature request
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2022.04.08 (update instructions) or later (specify commit)
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Description
Youtube-dl's verbose mode should tell you what downloader is being used to fetch the file.
Verbose log
When you run something like
yt-dlp "https://www.youtube.com/watch?v=NPBnphbYGFw" -v

I would like if rather than,
[debug] Invoking downloader on "https://rr1---sn-q4flrnle.googlevideo.com/videoplayback?expire=1652243655&ei=Z-h6YoX_EOnQlu8P282V6AI&ip=2603%3A300c%3A184d%3A1400%3A%3A4e0e&id=o-ALTncWiJhGCYpl4CiKyCOuKRRn2zpaN7ibt4i74YTg5X&itag=271&source=youtube&requiressl=yes&mh=MA&mm=31%2C26&mn=sn-q4flrnle%2Csn-5uaezn7e&ms=au%2Conr&mv=m&mvi=1&pl=38&initcwndbps=2057500&vprv=1&mime=video%2Fwebm&gir=yes&clen=305755282&dur=8073.332&lmt=1645227445100969&mt=1652221735&fvip=1&keepalive=yes&fexp=24001373%2C24007246&c=ANDROID&txp=5432434&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&sig=AOq0QJ8wRQIhAJs48aKe-GUU95hrtrrgSS855uoCG0hwNyadHViTu1uZAiBC56VQ6tlDLAdy6H4-kmLHGU1kbBiLcoJHUE2LVSYqZQ%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRQIhAKwETaT2ioVuN9UHfW-YrqJ_Jh8SMWe5HEWhTuP0KSx3AiAdhDMiOWZsHh6_QqFx4y6amBBA_kTcf4vwz-wl4W-cxg%3D%3D"

It said, WHAT downloader was invoked. This would help people new to the project to work on the downloaders.
Like this,

[debug] Invoking downloader [common.py] on

This will help trouble shoot the problem mentioned here. Thanks.

@pukkandan this still doesn't tell me why one is slower then the other. But it does tell me they both use the http downloader. Do you have any ideas of how this can be improved?
Run these two commands.
./yt-dlp.sh -v "https://www.youtube.com/watch?v=NPBnphbYGFw" -qiJ | jq -r '.formats[] | select(.format_id == "251").url' | xargs ./yt-dlp.sh -v -o out.webm

./yt-dlp.sh -v "https://www.youtube.com/watch?v=NPBnphbYGFw"

My guess from the above linked issue is that only one of them sets the download options http_chunk_size? Perhaps the downloader that receives the option, or the thing that sets the option should dump the options somewhere?
Ideally, I'm building up to a question of why doesn't targetting the url googlevideo.com set the download chunk size? If we know it's needed indirectly when you hit the viewer page, then shouldn't it be needed on direct download too?

My guess from the above linked issue is that only one of them sets the download options http_chunk_size? Perhaps the downloader that receives the option, or the thing that sets the option should dump the options somewhere?

See downloader_options and http_headers in the json. If you try to download the URL without these, ofc it will be slow!
PS: I don't get what is the point of your command anyway. If you can explain that (open another issue), perhaps I can help more

I think the point of that command is more of a question for @theofficialgman, I'll invite him to open the issue. Just to be clear, I get that those options need to be provided. I do not understand why they're implicit (provided in downloader_options or http_headers) in one command and not the other.

I do not understand why they're implicit (provided in downloader_options or http_headers) in one command and not the other.

https://github.com/ytdl-org/youtube-dl/issues/30568#issuecomment-1123192366

having a verbose mode that spit out all required information to facilitate fast downloads would be nice, yes.
so if that means setting http_chunk_size, setting the user agent, etc, it should all be output in the verbose output.

It still doesn't make sense to me to have that chunk information set in the thing that downloads the index page rather than the thing that downloads the video, which is what I understand is the architectural reason for this bheavior.
fn crawl_page(url) {
  fn download_video( url, options );
}

Seems less ideal than
fn get_default_options( url ) {
  // get global defaults override URL defaults
}

fn download_video( url ) {
  options = get_default_options(url);
}

fn crawl_page(url) {
  fn download_video( url, override_options );
}


Tabstops you can jump between?

One feature I noticed in the old snippets plugin, and a very common feature in code editors (I'm sure you're familiar) is to have highlighted replacement sections you can tab between and they are preselected so you can just start typing to replace them.
e.g here's an example from vscode:

Not sure how difficult/possible this is since you would need to temporarily take control of tab or some other key

I'm actually not familiar (I rarely use IDEs), and I'm no expert on Obsidian, but this could be possible. It would require a bit of machinery, so I can't say as to when the feature will be completed/worked on.
That said I haven't investigated either, so if it turns out that I need to use some kind of complicated API (e.g. CodeMirror...), that will make things considerably more difficult. Hopefully it will be easier than that.


Image does not load anymore when removed from cache

I discovered this issue by coincidence. I have cards in my app which can have multiple pictures. However, by default, only the first image is displayed (and thus loaded).
The images are loaded with KF and cached. The initial of download of all images works and behaves as expected. If i reload the app and thus the pictures, they are now fetched from the cache which also works as expected.
The problem is, if you do not click through the images and thus not load the images for a certain amount of time (i think the default is 7 days) they are removed from the cache. This also is intended and not the problem. However, the re-download of the images now does not work anymore. It keeps displaying an endless loading indicator.

It is the first time we receive this kind of report. Considering the amount of Kingfisher users, I guess it might be something in your use case only.
You can set the cache expiration duration by code like this:
KingfisherManager.shared.cache.memoryStorage.config.expiration = .seconds(10)
KingfisherManager.shared.cache.diskStorage.config.expiration = .seconds(20)

So it should be quite easy to "simulate" the expiration situation. I myself do not meet the endless loading with these expiration configurations. May I know if it is possible to have a good way to reproduce this issue? Thanks!


Fix broken jitpack publishing

Jitpack places all subprojects into single dependency. Not just it unnecessary increases size of users apk, but it can break compilation, as annotation processors cannot be placed in compile classpath in later versions of gradle. There's pom of moxy artifact.
That way, jitpack publishing should be either blocked or fixed.

We have at least one user with this bug. This behavior is documented on Jitpack: https://jitpack.io/docs/BUILDING/#multi-module-projects
Our problem is particularly tricky because Moxy's main module is called the same as repository, and Jitpack's all-in-one artifact name collides with our core module name: com.github.moxy-community:moxy:$moxyVersion.
Jitpack users should actually use different dependency declaration style: implementation 'com.github.moxy-community.Moxy:moxy:2.0.2' as shown on Jitpack itself: https://jitpack.io/#moxy-community/Moxy/2.0.2
Not sure if that can be fixed, we have yet to decide what to do.

If someone gets error like this — be aware that it may be caused by this bug. Spent several hours to figure this out.
* What went wrong:
Execution failed for task ':app:checkMirrorProductionDebugDuplicateClasses'.
> 1 exception was raised by workers:
  java.lang.RuntimeException: Duplicate class com.github.moxy_community.moxy.androidx.BuildConfig found in modules moxy-androidx-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-androidx:2.0.2) and moxy-androidx-2.0.2-runtime.jar (com.github.moxy-community:moxy-androidx:2.0.2)
  Duplicate class com.github.moxy_community.moxy.material.BuildConfig found in modules moxy-material-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-material:2.0.2) and moxy-material-2.0.2-runtime.jar (com.github.moxy-community:moxy-material:2.0.2)
  Duplicate class com.google.common.util.concurrent.ListenableFuture found in modules guava-23.0.jar (com.google.guava:guava:23.0) and listenablefuture-1.0.jar (com.google.guava:listenablefuture:1.0)
  Duplicate class moxy.MvpAppCompatActivity found in the following modules: jetified-moxy-app-compat-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-app-compat:2.0.2), moxy-androidx-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-androidx:2.0.2) and moxy-androidx-2.0.2-runtime.jar (com.github.moxy-community:moxy-androidx:2.0.2)
  Duplicate class moxy.MvpAppCompatDialogFragment found in the following modules: jetified-moxy-app-compat-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-app-compat:2.0.2), moxy-androidx-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-androidx:2.0.2) and moxy-androidx-2.0.2-runtime.jar (com.github.moxy-community:moxy-androidx:2.0.2)
  Duplicate class moxy.MvpAppCompatFragment found in the following modules: jetified-moxy-app-compat-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-app-compat:2.0.2), moxy-androidx-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-androidx:2.0.2) and moxy-androidx-2.0.2-runtime.jar (com.github.moxy-community:moxy-androidx:2.0.2)
  Duplicate class moxy.MvpBottomSheetDialogFragment found in modules moxy-material-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-material:2.0.2) and moxy-material-2.0.2-runtime.jar (com.github.moxy-community:moxy-material:2.0.2)
  Duplicate class moxy.ktx.DelegatesKt found in modules moxy-ktx-2.0.2.jar (com.github.moxy-community.moxy:moxy-ktx:2.0.2) and moxy-ktx-2.0.2.jar (com.github.moxy-community:moxy-ktx:2.0.2)
  Duplicate class moxy.ktx.MoxyKtxDelegate found in modules moxy-ktx-2.0.2.jar (com.github.moxy-community.moxy:moxy-ktx:2.0.2) and moxy-ktx-2.0.2.jar (com.github.moxy-community:moxy-ktx:2.0.2)
  Duplicate class moxy.ktx.MoxyKtxDelegate$field$1 found in modules moxy-ktx-2.0.2.jar (com.github.moxy-community.moxy:moxy-ktx:2.0.2) and moxy-ktx-2.0.2.jar (com.github.moxy-community:moxy-ktx:2.0.2)

  Go to the documentation to learn how to Fix dependency resolution errors.

Could you please explain your solution to that bug?

If someone gets error like this — be aware that it may be caused by this bug. Spent several hours to figure this out.
* What went wrong:
Execution failed for task ':app:checkMirrorProductionDebugDuplicateClasses'.
> 1 exception was raised by workers:
  java.lang.RuntimeException: Duplicate class com.github.moxy_community.moxy.androidx.BuildConfig found in modules moxy-androidx-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-androidx:2.0.2) and moxy-androidx-2.0.2-runtime.jar (com.github.moxy-community:moxy-androidx:2.0.2)
  Duplicate class com.github.moxy_community.moxy.material.BuildConfig found in modules moxy-material-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-material:2.0.2) and moxy-material-2.0.2-runtime.jar (com.github.moxy-community:moxy-material:2.0.2)
  Duplicate class com.google.common.util.concurrent.ListenableFuture found in modules guava-23.0.jar (com.google.guava:guava:23.0) and listenablefuture-1.0.jar (com.google.guava:listenablefuture:1.0)
  Duplicate class moxy.MvpAppCompatActivity found in the following modules: jetified-moxy-app-compat-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-app-compat:2.0.2), moxy-androidx-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-androidx:2.0.2) and moxy-androidx-2.0.2-runtime.jar (com.github.moxy-community:moxy-androidx:2.0.2)
  Duplicate class moxy.MvpAppCompatDialogFragment found in the following modules: jetified-moxy-app-compat-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-app-compat:2.0.2), moxy-androidx-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-androidx:2.0.2) and moxy-androidx-2.0.2-runtime.jar (com.github.moxy-community:moxy-androidx:2.0.2)
  Duplicate class moxy.MvpAppCompatFragment found in the following modules: jetified-moxy-app-compat-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-app-compat:2.0.2), moxy-androidx-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-androidx:2.0.2) and moxy-androidx-2.0.2-runtime.jar (com.github.moxy-community:moxy-androidx:2.0.2)
  Duplicate class moxy.MvpBottomSheetDialogFragment found in modules moxy-material-2.0.2-runtime.jar (com.github.moxy-community.moxy:moxy-material:2.0.2) and moxy-material-2.0.2-runtime.jar (com.github.moxy-community:moxy-material:2.0.2)
  Duplicate class moxy.ktx.DelegatesKt found in modules moxy-ktx-2.0.2.jar (com.github.moxy-community.moxy:moxy-ktx:2.0.2) and moxy-ktx-2.0.2.jar (com.github.moxy-community:moxy-ktx:2.0.2)
  Duplicate class moxy.ktx.MoxyKtxDelegate found in modules moxy-ktx-2.0.2.jar (com.github.moxy-community.moxy:moxy-ktx:2.0.2) and moxy-ktx-2.0.2.jar (com.github.moxy-community:moxy-ktx:2.0.2)
  Duplicate class moxy.ktx.MoxyKtxDelegate$field$1 found in modules moxy-ktx-2.0.2.jar (com.github.moxy-community.moxy:moxy-ktx:2.0.2) and moxy-ktx-2.0.2.jar (com.github.moxy-community:moxy-ktx:2.0.2)

  Go to the documentation to learn how to Fix dependency resolution errors.

Could you please explain your solution to that bug?

Could you please explain your solution to that bug?

As i could remember this message helped me https://github.com/moxy-community/Moxy/issues/63#issuecomment-570990776.
Could not remember details because i've removed Moxy from my project long time ago.

Could you please explain your solution to that bug?

As i could remember this message helped me https://github.com/moxy-community/Moxy/issues/63#issuecomment-570990776.
Could not remember details because i've removed Moxy from my project long time ago.

Could you please explain your solution to that bug?

As i could remember this message helped me #63 (comment).
Could not remember details because i've removed Moxy from my project long time ago.

Well, it didn't helped me. But I've successfully excluded the problematic package with this adding to app gradle:
configurations {
        all {
            exclude group: "com.github.moxy-community.Moxy", module: "moxy-androidx"
        }
    }

Could you please explain your solution to that bug?

As i could remember this message helped me #63 (comment).
Could not remember details because i've removed Moxy from my project long time ago.

Well, it didn't helped me. But I've successfully excluded the problematic package with this adding to app gradle:
configurations {
        all {
            exclude group: "com.github.moxy-community.Moxy", module: "moxy-androidx"
        }
    }

@arseniy899 Could you share your problematic Gradle config? The fix from https://github.com/moxy-community/Moxy/issues/63#issuecomment-570990776 should work, and if it's not, it's worth investigating your config for other problems.

@arseniy899 Could you share your problematic Gradle config? The fix from https://github.com/moxy-community/Moxy/issues/63#issuecomment-570990776 should work, and if it's not, it's worth investigating your config for other problems.


[SUPPORT] Metrics Prometheus pushgateway

I have discovered that you've added prometheus related changes
like here https://issues.apache.org/jira/browse/HUDI-210
But unfortunately there is no documentation related to pushing hudi metrics to Prometheus Push Gateway
https://hudi.apache.org/docs/metrics.html#hoodiemetrics
What parameters should be set in order to do that?
Environment Description

Hudi version : 0.6.0

Spark version : 2.4.7

Hadoop version : 2.7

Storage (HDFS/S3/GCS..) : hdfs

Running on Docker? (yes/no) : no

Here is an example config :
--hoodie-conf hoodie.metrics.on=true --hoodie-conf hoodie.metrics.reporter.type=PROMETHEUS_PUSHGATEWAY --hoodie-conf hoodie.metrics.pushgateway.host=<PUSH-GATEWAY-HOST> --hoodie-conf hoodie.metrics.pushgateway.port=9091 --hoodie-conf hoodie.metrics.pushgateway.job.name=<JOB_NAME> --hoodie-conf hoodie.metrics.pushgateway.random.job.name.suffix=false --hoodie-conf hoodie.metrics.pushgateway.delete.on.shutdown=false

Thanks a lot for the reply
I my case I am getting the following error:
1/04/26 13:29:52 WARN prometheus.PushGatewayReporter: Can't push monitoring information to pushGateway
java.net.UnknownHostException: https
at java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:184)
at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
at java.net.Socket.connect(Socket.java:607)
at sun.net.NetworkClient.doConnect(NetworkClient.java:175)
at sun.net.www.http.HttpClient.openServer(HttpClient.java:463)
at sun.net.www.http.HttpClient.openServer(HttpClient.java:558)
at sun.net.www.http.HttpClient.(HttpClient.java:242)
at sun.net.www.http.HttpClient.New(HttpClient.java:339)
at sun.net.www.http.HttpClient.New(HttpClient.java:357)
at sun.net.www.protocol.http.HttpURLConnection.getNewHttpClient(HttpURLConnection.java:1226)
at sun.net.www.protocol.http.HttpURLConnection.plainConnect0(HttpURLConnection.java:1162)
at sun.net.www.protocol.http.HttpURLConnection.plainConnect(HttpURLConnection.java:1056)
at sun.net.www.protocol.http.HttpURLConnection.connect(HttpURLConnection.java:990)
at io.prometheus.client.exporter.PushGateway.doRequest(PushGateway.java:305)
at io.prometheus.client.exporter.PushGateway.pushAdd(PushGateway.java:160)
at org.apache.hudi.metrics.prometheus.PushGatewayReporter.report(PushGatewayReporter.java:72)
at org.apache.hudi.metrics.prometheus.PushGatewayMetricsReporter.report(PushGatewayMetricsReporter.java:64)
at org.apache.hudi.metrics.Metrics.reportAndCloseReporter(Metrics.java:64)
at org.apache.hudi.metrics.Metrics.lambda$new$0(Metrics.java:57)
at java.lang.Thread.run(Thread.java:748)
my parameters are
--hoodie-conf hoodie.metrics.on=true
--hoodie-conf hoodie.metrics.reporter.type=PROMETHEUS_PUSHGATEWAY
--hoodie-conf hoodie.metrics.pushgateway.host=https://pushgateway_url.com/
--hoodie-conf hoodie.metrics.pushgateway.port=8081
--hoodie-conf hoodie.metrics.pushgateway.job.name=test_job_1
--hoodie-conf hoodie.metrics.pushgateway.random.job.name.suffix=false
--hoodie-conf hoodie.metrics.pushgateway.delete.on.shutdown=false \

@bvaradar : while you respond to author, can you add a faq entry(or some other documentation) on how to push hudi metrics to Prometheus Push Gateway.

@bvaradar Could you look into this please ?

@PavelPetukhov This looks like the push gateway URL has not been configured correctly. The hoodie-conf hoodie.metrics.pushgateway.host probably expects a hostname instead of a http URL. Can you see if this is the issue ?

I am experiencing the same issue using simpleclient with Spring boot for one of our cron jobs.
Caused by: java.net.UnknownHostException: https

Closing due to inactivity


loglocation and def.noc.sleep incompatibility

When sleeplog is specified and length(def.noc.sleep) != 1, this is not compatible and GGIR pretends to reset def.noc.sleep to its default value. However, I think the conditional test for this in check_params is wrong, so that GGIR resets def.noc.sleep even if sleeplog is not specified.

Just seen this was fixed in the master branch.


fix ee test in ci

Codecov Report
Base: 46.85% // Head: 46.85% // No change to project coverage :thumbsup:

Coverage data is based on head (2850c43) compared to base (13db2b3).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #431   +/-   ##
=======================================
  Coverage   46.85%   46.85%
=======================================
  Files          27       27
  Lines        2828     2828
=======================================
  Hits         1325     1325
  Misses       1404     1404
  Partials       99       99

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


agarplus map is working

How they got map working?

go look at the code. its in tampermonkey in the links


Noticeable delay in react-error-overlay showing up

Is this a bug report?
Yes (Originally logged here: https://github.com/smooth-code/error-overlay-webpack-plugin/issues/27)
Environment
System:
OS: Linux 3.10 Red Hat Enterprise Linux Server 7.6 (Maipo)
CPU: x64 Intel(R) Xeon(R) CPU E5-2690 v4 @ 2.60GHz
Binaries:
Node: 10.14.1
Yarn: 1.12.3
npm: 6.4.1
Browsers:
Firefox: support!
Mozilla Firefox 60.4.0
npmPackages:
react: ^16.7.0 => 16.7.0
react-dom: ^16.7.0 => 16.7.0
react-scripts: Not Found
npmGlobalPackages:
create-react-app: Not Found
Steps to Reproduce

I am using error-overlay-webpack-plugin to get react-error-overlay in my application
Whenever I trigger a runtime error, there is a noticeable delay in it showing up.
It seems that fetching the main bundle to create sourcemapped stacktrace takes some time.

Expected Behavior
It would be great if we show the compiled stacktrace until the sourcempped version is available so that the overlay is show immediately on error. Just like the package redbox-react that I was previously using.
Actual Behavior
It waits on the stacktrace sourcemapping which takes quite some time to complete.
Reproducible Demo
It can be done by triggering a runtime error on any CRA app. The bigger the app is, the longer is the delay in the error overlay showing up.

dupe of https://github.com/facebook/create-react-app/issues/6078, thanks!


Recursive definitions cause stack overflow

Something like:
struct F1: Codable {
let v: [F2]?
}
struct F2: Codable {
let v: [F1]?
}
This will bomb in the table.create().

Fixed.


Fix empty datacenter selection in vm provision

build_ci_hash_struct sets the class to the base_class, this means that a Datacenter has class EmsFolder.  This causes no Datacenters to be shown in the provision workflow.
Introduced when removing is_datacenter here https://github.com/ManageIQ/manageiq/pull/7070.

@miq-bot add-label bug

cc @gmcculloug

@agrare The Folder selection in the Provisioning workflow is also empty.  Can you confirm if this is related?

@gmcculloug I see the following,

@agrare Tested with the patch and I am seeing folders now.  :+1:

@agrare Please add tests.


Add ucred and socket related types to uclibc-mips32

Fix compilation of the tokio rust for MIPS architecture when using uClibc. Added missed types, which are defined in library headers:
https://github.com/kraj/uClibc/blob/ca1c74d67dd115d059a875150e10b8560a9c35a8/libc/sysdeps/linux/common/bits/socket.h#L320
https://github.com/kraj/uClibc/blob/ca1c74d67dd115d059a875150e10b8560a9c35a8/libc/sysdeps/linux/common/bits/in.h#L35-L36
https://github.com/kraj/uClibc/blob/ca1c74d67dd115d059a875150e10b8560a9c35a8/libc/sysdeps/linux/common/bits/in.h#L140-L143
Should also solve https://github.com/rust-lang-nursery/net2-rs/issues/88
Revisit of #1500

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @gnzlbg (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

All builds passed except one, which I believe is not related to this PR at all 😟

@bors: r+

:pushpin: Commit b911a0edc9ba2f84b1432ef6bcdc9e7a22c63a92 has been approved by gnzlbg

:hourglass: Testing commit b911a0edc9ba2f84b1432ef6bcdc9e7a22c63a92 with merge bc8aec2705971690cbea955a405a1d41a6f1ea3f...

:broken_heart: Test failed - status-azure

Same results, but there is one spontaneous failure due to unavailable resources 😟
I'm happy to fix one with unused constants, could you please point me to right direction what to do?

I'm happy to fix one with unused constants, could you please point me to right direction what to do?

Sure. Some of the constants that you added are "unused" which means that they are not exported by the library. This happens, for example, if a parent module e.g. .../mips/mod.rs already adds them, since these will shadow the ones that get imported from .../mips/mips32/mod.rs. So the best way to fix this is figure out if any of the parent modules exposes them, and if the values are correct, then just remove the ones that are duplicated. If the values are incorrect, then you need to move the constant from the parent module to the "upper" (more specific) modules for which they are correct.

@gnzlbg Thanks for the hint! Those were indeed duplicated. Actually, after I made more precise look, I realized that the ucred type is common for all architectures and is not mips32 specific. So altered PR to define type on common layer and make everybody happy 😎

@bors: r+

:pushpin: Commit 7335d1e4cda84ce60901da65934ae00bef47c56a has been approved by gnzlbg

@bors: retry

@zvirja: :key: Insufficient privileges: not in try users

Fair enough, but worth’s trying ;-)
@gnzlbg It looks like it stuck and didn’t run tests again to merge the PR :thinking:

:hourglass: Testing commit 7335d1e4cda84ce60901da65934ae00bef47c56a with merge cfd561c28987bed1f925e358d8ab03ad8cd9b085...

@zvirja there were other PRs in the queue, so bors couldn't start testing it immediately: https://buildbot2.rust-lang.org/homu/queue/libc

:sunny: Test successful - checks-cirrus-freebsd-10, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, status-azure
Approved by: gnzlbg
Pushing cfd561c28987bed1f925e358d8ab03ad8cd9b085 to master...

@gnzlbg Thanks for your help and review! 🤝
Do you plan to make a new patch release in a nearby future? So I could consume the changes via crate? 😊

There are a couple of PRs in the queue with good chances of merging as well. As soon as they merge in the next couple of hours I'll do a new release.


Mark flutter_gallery_instrumentation_test as flaky

See #18879.
Please merge for me if you're happy with this; it's getting late here so I'll be off v. soon.

Windows Cirrus failure is "Agent is not responding!", so I've restarted it.

This test doesn't seem flaky. I think we can unmark it.

You jinxed it :( Yellow 3 times in the last 6 runs.

https://flutter-dashboard.appspot.com/api/get-log?ownerKey=ahNzfmZsdXR0ZXItZGFzaGJvYXJkclgLEglDaGVja2xpc3QiOGZsdXR0ZXIvZmx1dHRlci9lZTM5NjI3MmQzMjYzZWVjNDZjOTRiYjFmYTAwY2QyY2E3MmRkMWU3DAsSBFRhc2sYgICAgICA1AgM
https://flutter-dashboard.appspot.com/api/get-log?ownerKey=ahNzfmZsdXR0ZXItZGFzaGJvYXJkclgLEglDaGVja2xpc3QiOGZsdXR0ZXIvZmx1dHRlci9hY2Y0YjZjMWFhNjMxZTVmYTYyOTliYTdlZjZkZmRmZTFmYTdkNmFiDAsSBFRhc2sYgICAgICAiAoM
https://flutter-dashboard.appspot.com/api/get-log?ownerKey=ahNzfmZsdXR0ZXItZGFzaGJvYXJkclgLEglDaGVja2xpc3QiOGZsdXR0ZXIvZmx1dHRlci81YjMwYjM5M2E4Y2Y3ZjAwOTJmMTcyNmM1YThmNzk4MzJiMGFkMGY4DAsSBFRhc2sYgICAgICAuAkM


Issue 137 - Split GitHub Actions build per-module

Resolves #137

I'm thinking of combining some of the modules together to make bigger chunks. We can have a lot fewer workflows and still keep them finishing within 5-10 minutes.


Paste image from clipboard

Is this possible? I'm comparing WYSIWYG editors and looking for this feature.

This project isn't maintained anymore. If you'd like to add a fix a PR is welcome otherwise you might want to fork it or look for another editor.


(PUP-9015) Append vendored modules to $LOAD_PATH

Append the vendored modules directory to the ruby $LOAD_PATH, which is effectively how things worked before we extracted them into modules. Note vendored_modules have least precedence so that it can be overridden by pluginsynced modules.

jenkins please test this on windows2012r2-64a

CLA signed by all contributors.

beaker tests passed in https://jenkins-platform.delivery.puppetlabs.net/job/platform_puppet-agent-extra_puppet-agent-integration-suite_pr/532/. Need to update spec test for windows.

To repro the bug:
$ git checkout master
$ bundle exec puppet module install puppetlabs-mount_core --target-dir ./modules --version 1.0.0
$ bundle exec puppet describe -l --vendormoduledir ./modules
Error: Could not autoload puppet/provider/mount/parsed: cannot load such file -- puppet/provider/mount
Error: Could not autoload puppet/type/mount: Could not autoload puppet/provider/mount/parsed: cannot load such file -- puppet/provider/mount
Error: Could not run: Could not autoload puppet/type/mount: Could not autoload puppet/provider/mount/parsed: cannot load such file -- puppet/provider/mount
$ git pr 6965
...
$ bundle exec puppet describe -l --vendormoduledir ./modules
These are the types known to puppet:
cron            - Installs and manages cron jobs
exec            - Executes external commands
file            - Manages files, including their content, owner ...
filebucket      - A repository for storing and retrieving file  ...
group           - Manage groups
mount           - Manages mounted filesystems, including puttin ...
notify          - .. no documentation ..
package         - Manage packages
resources       - This is a metatype that can manage other reso ...
schedule        - Define schedules for Puppet
service         - Manage running services
stage           - A resource type for creating new run stages
tidy            - Remove unwanted files based on specific crite ...
user            - Manage users
whit            - Whits are internal artifacts of Puppet's curr ...

jenkins please test this on windows2012r2-64a


Update README.md

Requires AdWords account, but incredibly useful in answering the question, "Does anyone care about your new product"

great resource, thanks!

Great job


mecanium test opmodes and XDrive refactor

Before issuing a pull request, please see the contributing page.

#3  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @minerkid08 and the rest of your teammates on  Graphite


Delete Retired .openpublishing.build.ps1 script of default branch

This PR is auto-generated by Build Dev team to delete retired .openpublishing.build.ps1 script of default branch. Please help review and merge this PR if not auto-merged. For any concern or questions, contact<EMAIL_ADDRESS>or Learn Content Publshing team.

@shujingMS Maybe you can close this PR and open a new one.  The issue still exists.

This docset publishes to MSDN site but not Learn site. Build fails as publishing pipeline does not support MSDN site now.
Per the docset name "MSDN.SkypeSDK-ops-redirect", it looks like redirection from MSDN site. As MSDN site has been retired entirely, I'm not sure if these redirections are in use today. If not, we can go offline and delete this repo.


Remove nightly instructions from setting-up-your-environment.md

Bend no longer requires nightly and now runs on stable Rust

Thank you for the PR. I will take a look and merge it.


Appendix for parameter estimates

Currently the values are the MLE values, need to make them posterior medians

Fixed it, commit number is b4519af.


[Refactor]: Folder structure improvement proposal

What feature would you like to see?
The current folder structure appears disorganized, with example projects and packages nested within each other. It is recommended to refactor the folder structure to provide better clarity and organization.
By restructuring the folder organization, users who only include this package in their projects will avoid downloading unused example files when executing flutter pub get. This change is expected to improve the performance of the command execution by reducing the amount of data that needs to be fetched.
See

Before
After

Thanks for the proposal. We need the example folder separately AFAIK to be able to show this page on pub dev. Otherwise examples folder looks good. I would also like to keep widgetbook inside coffee_maker_navigator_2 since it belongs to that specific project.
Other than than wolt_state_management and wolt_di also only used in coffee_maker_navigator_2. We should include them under it imo.

Package layout conventions - Examples
According to the explanation on this page, we can generate markdown files to add and redirect example project paths. For instance, the flutrlint project already includes a markdown file under the "Example" tab, which can serve this purpose.
We can re-add the Widgetbook project as you suggested.
I recommend moving the wolt_di and wolt_state_management packages to the packages folder for easier access in future projects. These packages are currently located in the root directory.

Also you can check this PR: [DRAFT] Refactor - Clean folder structure

@mukireus The draft PR looks good. We can merge this way. @TahaTesser what is your take?
@mukireus we will also need to update the ReadMe.

Looks good to me. This will make the samples navigation a lot better. I wanted to do this but couldn't get around it.

@ulusoyca Should we also move the wolt_modal_sheet package into the packages directory? I can open a separate PR for this, or we can pair on it together. We can close the issue if needed.

@mukireus Thanks for the great changes, and the PR! This repo is made for Wolt Modal Sheet, the other packages are planned to be released a packages during next year in separate repos, so let's keep them under the packages folder for now. I am closing this issue. Once more, thanks for the great work.


Error when view Terminal of Mysql Prject

Actual Behavior:

What is the issue? *

Version: 1.0.0-e4d31ca
1.Using Image Templates to deploy Mysql project.  Json file as default. --> Then the task state is always "failed".
2.Go to the service page, then go to the container page, click "Terminal" tag, Error occur as the screenshot.

What is the expected behavior?

1.Then the task state should not be "failed".
2."Terminal" tag should not show things like this.
CodePen (or steps to reproduce the issue): *

CodePen Demo which shows your issue:
Details:

Angular Versions: *

Angular Version:
Angular Material Version:

Additional Information:

Browser Type: *
Browser Version: *
OS: * Centos 7.2
Stack Traces:

this is regression bug, i found about 3+ bugs on related report.

resolved.


feat: make generic component server http

This change is

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

Project coverage is 83.03%. Comparing base (6ec0d90) to head (6c324f2).

Files
Patch %
Lines

crates/mempool_infra/src/component_server.rs
89.47%
1 Missing and 3 partials :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #337      +/-   ##
==========================================
+ Coverage   82.87%   83.03%   +0.16%
==========================================
  Files          31       31
  Lines        1489     1527      +38
  Branches     1489     1527      +38
==========================================
+ Hits         1234     1268      +34
- Misses        195      196       +1
- Partials       60       63       +3

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


Feature request: Choosing filetype and buf_local options

e.g. set ft to go or lua instead of markdown

E.g. set different conceallevel

You should write the issue description more clearly.
Why it is not enough that markdown's fenced language block.

Yep I should, sorry about that. I guess you are right about that. markdown's fenced language block is enough. If its possible, can you add a custom filetype on that buffer (extending markdown) so users can attach local option to it.
For example I dislike conceallevel while editing markdown, but I'd like to to temporary set it on the documentation window via filetype autocmd.
Thanks

Currently, nvim-compe's documentation window already set &conceallevel=2 as local to it.

Yep that doesn't work because I use tpope/vim-markdown plugin that offer an option to disable conceallevel
let g:markdown_syntax_conceal = 0

It means something like the following, right?

set &conceallevel=2 in nvim-compe.
Vim fires FileType autocmd.
tpope/vim-markdown set to &conceallevel as 2 by let g:markdown_syntax_conceal = 0

Hm... It is not reproduce on my side with install tpope/vim-markdown and set g:markdown_syntax_conceal = 0.

Thats wired, I'll double check

Yep it works okay with lua lisp, not sure why I still see the fenced language block . Maybe something todo with how I setup documentation content. I'll do further testing.

The fenced_language_block has a marker that is ts in this example ```ts content ```.
The marker is not real vim's filetype so vim's markdown syntax has a option for it.
You can add let g:markdown_fenced_languages = ['typescript=ts']

I've improved the error message for it.

@tami5 this issue can be closed, right?

@tami5 this issue can be closed, right?

Yep

Yep


Rename scripts to .sh (from .bsh); move to src/main/sh and src/test/sh

Use of .bsh as extension for shell scripts is bizarre
Shell scripts, by fairly universal convention, have the file extension .sh. The fact that you're writing scripts which depend on Bash features deliberately is not unusual, but appropriately guarded by your use of
#!/bin/bash

in the hash bang line.
Standardise src layout
In keeping with common layouts
Have scripts in src/main/sh
and tests in       src/test/sh

Agree - we also commented on the same internally - I haven't got around to correcting it. Happy to have  PR. Else it is on my list

#13 resolved it.


Identify parameters and their range for a parameter sweep

This relates to #29

@mwertsen @Dengxiao-Lang Can you please tell us where these variables can be found in the model?
For example, as we understand for the GateCapacity variable of farmer 1 and canal 10, one needs to change the values from 30 to desired value at the lines below. Is that correct?
https://github.com/mess-nlesc/model/blob/b0ece6ba725837d92c0fec5d0a742f3b243597e3/model/GateCapacity.nls#L95
https://github.com/mess-nlesc/model/blob/b0ece6ba725837d92c0fec5d0a742f3b243597e3/model/GateCapacity.nls#L101
https://github.com/mess-nlesc/model/blob/b0ece6ba725837d92c0fec5d0a742f3b243597e3/model/GateCapacity.nls#L106

@lyashevska just mentioned you run some experiments in the past which are also described in
the table on page 5 of Summary overview of IRABM3.docx. We would like to know what changes you made for each of these experiments.

Hi @fdiblen
These lines are for gate capacity adjusting. The logic of this process is:
In the beginning, all farmers have the same initial gate capacity (30<IGC< = 200).
Then, farmers' harvest situation will be evaluated every year:
If there is a poor harvest situation among these farmers, the gate capacity of upstream farmers and mid-stream farmers will be decreased; if there are poor harvest situations that last for many years and the gate capacity of upstream and mid-stream farmers is higher than 30 (like line 95 and 101), their gate capacity will be gradually decreased till to 30 (like line 106).
Thus, 30 is the minimum gate capacity for all farmers.
I hope this answered your question.
发自 Outlookhttp://aka.ms/weboutlook

发件人: fdiblen @.>
发送时间: 2024年3月19日 18:05
收件人: mess-nlesc/model @.>
抄送: Dengxiao Lang @.>; Mention @.>
主题: Re: [mess-nlesc/model] Identify parameters and their range for a parameter sweep (Issue #27)
@mwertsenhttps://github.com/mwertsen @Dengxiao-Langhttps://github.com/Dengxiao-Lang Can you please tell us where these variables can be found in the model?
For example, as we understand for the GateCapacity variable of farmer 1 and canal 10, one needs to change the values from 30 to desired value at the lines below. Is that correct?
https://github.com/mess-nlesc/model/blob/b0ece6ba725837d92c0fec5d0a742f3b243597e3/model/GateCapacity.nls#L95
https://github.com/mess-nlesc/model/blob/b0ece6ba725837d92c0fec5d0a742f3b243597e3/model/GateCapacity.nls#L101
https://github.com/mess-nlesc/model/blob/b0ece6ba725837d92c0fec5d0a742f3b243597e3/model/GateCapacity.nls#L106
―
Reply to this email directly, view it on GitHubhttps://github.com/mess-nlesc/model/issues/27#issuecomment-2007700819, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMMCLBQ2M3LMUSNV3Q2YUJTYZBV5DAVCNFSM6AAAAAA7WDIDIWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBXG4YDAOBRHE.
You are receiving this because you were mentioned.

Hi @Dengxiao-Lang,
Thank you, it helps to understand how gate capacity is calculated.
Can you please also help us with identifying where these variables are used in the code? We want to know what hard coded numbers need to change in order to run experiments with different values of:

RiverDischarge
GateCapacity
GC adjustment variation
Harvest memory
Available water memory

I am sorry I forgot to mention these variables in the earlier email.
The first three variables are presented as sliders (NetLogo function) in the NetLgo interface, the other two are calculated in this code:
Qin_average = RiverDischarge
QGateMaxFixed = GateCapacity (initial)
ComparisonTime = GC adjustment variation (please refer to the FarmersHarvestEvaluation.nls)
AverageHarvestBarley = Harvest memory (please refer to the HarvestMemory.nls)
AvailableWaterSupply = Available water memory (please refer to the AvailableWaterSupply.nls)
发自 Outlookhttp://aka.ms/weboutlook

发件人: fdiblen @.>
发送时间: 2024年3月20日 14:51
收件人: mess-nlesc/model @.>
抄送: Dengxiao Lang @.>; Mention @.>
主题: Re: [mess-nlesc/model] Identify parameters and their range for a parameter sweep (Issue #27)
Hi @Dengxiao-Langhttps://github.com/Dengxiao-Lang,
Thank you, it helps to understand how gate capacity is calculated.
Can you please also help us with identifying where these variables are used in the code? We want to know what hard coded numbers need to change in order to run experiments with different values of:

RiverDischarge
GateCapacity
GC adjustment variation
Harvest memory
Available water memory

―
Reply to this email directly, view it on GitHubhttps://github.com/mess-nlesc/model/issues/27#issuecomment-2009616149, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMMCLBRFLYVCVHI5YGFEWITYZGH4NAVCNFSM6AAAAAA7WDIDIWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBZGYYTMMJUHE.
You are receiving this because you were mentioned.Message ID: @.***>

Suppose we run the following:
./netlogo-headless.sh --model "model/main.nlogo" --experiment "RD40to200_GC30_CSPHY3__50years" --table output.csv
where:
--experiment "RD40to200_GC30_CSPHY3__50years" corresponds to  https://github.com/mess-nlesc/model/blob/5980b7e24080ff733598f4d55ef71e14ed4838a0/model/main.nlogo#L10570
If we further look into say Qin_Average (RiverDischarge) we can see it is controlled here:
https://github.com/mess-nlesc/model/blob/5980b7e24080ff733598f4d55ef71e14ed4838a0/model/main.nlogo#L10672
And QGateMaxFixed (GateCapacity) here:
https://github.com/mess-nlesc/model/blob/5980b7e24080ff733598f4d55ef71e14ed4838a0/model/main.nlogo#L10688
The <steppedValueSet> and <enumeratedValueSet> tags define the parameters. The <steppedValueSet> tag is used for parameters that should be varied in steps between a start and end value. The <enumeratedValueSet> tag is used for parameters that should take on a specific set of values. For example, the "Qin_average" parameter is varied from 40 to 200 in steps of 10, and the "QGateMaxFixed" parameter is set to 30.

This relates to #33


8.1.1476 broke make clean

After this patch, the output of make clean is full of "No such file or directory" warnings and  make produces errors.  See output below.  I suspect that an "-f" flag was left off of some "rm" calls.
Starting make in the src directory.
If there are problems, cd to the src directory and run make there
cd src && /Applications/Xcode.app/Contents/Developer/usr/bin/make clean
cd testdir; /Applications/Xcode.app/Contents/Developer/usr/bin/make -f Makefile clean
rm -rf *.out *.failed *.res *.rej *.orig
rm opt_test.vim test.log test_result.log messages
rm: opt_test.vim: No such file or directory
rm: test.log: No such file or directory
rm: test_result.log: No such file or directory
rm: messages: No such file or directory
make[2]: [clean] Error 1 (ignored)
rm test.out X* viminfo tiny.vim small.vim mbyte.vim mzscheme.vim test.ok benchmark.out
rm: test.out: No such file or directory
rm: X*: No such file or directory
rm: viminfo: No such file or directory
rm: tiny.vim: No such file or directory
rm: small.vim: No such file or directory
rm: mbyte.vim: No such file or directory
rm: mzscheme.vim: No such file or directory
rm: test.ok: No such file or directory
rm: benchmark.out: No such file or directory
make[2]: [clean] Error 1 (ignored)
rm valgrind.*
rm: valgrind.*: No such file or directory
make[2]: [clean] Error 1 (ignored)

Here is what the analogous portion of the output used to look like before the offending patch:
Starting make in the src directory.
If there are problems, cd to the src directory and run make there
cd src && /Applications/Xcode.app/Contents/Developer/usr/bin/make clean
cd testdir; /Applications/Xcode.app/Contents/Developer/usr/bin/make -f Makefile clean
rm -rf *.out *.failed *.res *.rej *.orig opt_test.vim test.log messages test.out X* viminfo tiny.vim small.vim mbyte.vim mzscheme.vim test.ok benchmark.out valgrind.*

Formerly there was one rm operation for all the things.

Running make clean once gave errors!


use default ALLOWED_STATUS_CODES

Thanks again @zvolsky, I've decided to not override allowed_status_codes in the constructor so we can distinguish if they've been provided by the user, see #26 for more details.


Additional tenants doesn't work

What is the use of Additional tenants and why doesn't it work. I tried testing it but it doesn't work at all (Child tenants can see still see catalogs even though its not checked).
Second issue is that if a catalog is created by child tenant, parent tenant user can see that they can select any tenants.

But if I click edit then it only shows the child tenants.

I wanted to see if Parent tenant can share the child tenant catalog to other tenants.

There is another issue with the checkboxes. We can't save the changes in one attempt. Instead we need to check the parent item (all tenant selection) and then undo it and then select the checkboxes we need. Only then it'll accept the changes.

Is this related to #21844 ?  If not, @kbrock can you take a look at this one?

If it is not the bug Fryguy was referencing, you may also want to take something into account.

If you look in https://github.com/ManageIQ/manageiq/blob/master/lib/rbac/filterer.rb#L129 you will get a better idea of how entities are available to just yourself, your parents, or your children.


Container image: save built image ID

Related to https://github.com/Azure/azure-dev/issues/2632
Suppose that a user or tool wants to run dotnet publish -p:PublishProfile=DefaultContainer to build the image locally, and then publish it in a subsequent step outside of dotnet publish. In this case, it'd be nice for there to be a way to reference the unique image ID produced. Currently, there are output MSBuild task variables present , but it'd be nice to have the reference available outside of being part of MSBuild process.
To support an external tool invocation, I could see something like dotnet publish -p:PublishProfile=DefaultContainer -p:IIDFile=<path> similar to docker build --iidfile useful.

The new MSBuild CLI Evaluation work might actually make this even easier - I think you'd be able to run a publish command and request specific properties/items from the build, and have that returned back to you in structured form, all without us needing to write any files.
>dotnet publish --os linux --arch x64 -p PublishProfile=DefaultContainer -getProperty GeneratedContainerManifest
{ .... }