Incorrect minIntrincisWidth for Paragraph with placeholders

  testWidgets('foo RenderParagraph intrinsic width', (WidgetTester tester) async {
    await tester.pumpWidget(
      Directionality(
        textDirection: TextDirection.ltr,
        child: Center(
          child: RichText(
            text: const TextSpan(
              style: TextStyle(fontSize: 16, height: 1),
              children: <InlineSpan>[
                TextSpan(text: '1 '),
                WidgetSpan(
                  child: SizedBox(
                    height: 200,
                    width: 200,
                  ),
                ),
                TextSpan(text: ' 123'),
              ],
            ),
          ),
        ),
      ),
    );

    RenderParagraph paragraph = tester.renderObject<RenderParagraph>(find.byType(RichText));
    expect(paragraph.getMaxIntrinsicWidth(1000.0), 200 + 6 * 16.0);

    // This expect FAILS, it appears that the WidgetSpan
    // is not considered in the minIntrinsicWidth calculation.
    expect(paragraph.getMinIntrinsicWidth(1000.0), 200); // actual: 3 * 16
  });

I would have expected that in the example above the WidgetSpan is considered when calculating the minIntrinsicWidth (just like it is when the maxIntrinsicWidth is calculated). However, the minIntrinsicWidth appears to only be calculated based on the TextSpans.
A quick scan through the engine code that calculates the intrinsics for text seems to confirm that the placeholder spans are not considered:
https://github.com/flutter/engine/blob/069b3cf8f093d44ec4bae1319cbfdc4f8b4753b6/third_party/txt/src/txt/paragraph_txt.cc#L1165
My understanding is that  minIntrinsicWidth should return the smallest width at which the thing can still paint without clipping. However, if you force the Paragraph to the calculated minIntrinsicWidth, the WidgetSpan would get clipped. So, this appears to be a bug in libtxt?

/cc @jason-simmons @GaryQian @Hixie

Semantically, the min intrinsic width of a paragraph should be the widest word or the widest placeholder, whatever is wider.

This does seem wrong.  It looks like Libtxt is passing the object replacement character representing the WidgetSpan into Minikin, and Minikin is returning a glyph for that character.
Libtxt then proceeds to use the width of that glyph in the calculations that feed into minIntrinsicWidth.  It should substitute the dimensions of the WidgetSpan's placeholder.

Reproducible on latest stable.
flutter doctor -v
[✓] Flutter (Channel stable, 1.22.3, on Mac OS X 10.15.7 19H2, locale en)
    • Flutter version 1.22.3 at /Users/pedromassango/dev/SDKs/flutter_stable
    • Framework revision 8874f21e79 (13 days ago), 2020-10-29 14:14:35 -0700
    • Engine revision a1440ca392
    • Dart version 2.10.3

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    • CocoaPods version 1.9.3

[!] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] IntelliJ IDEA Community Edition (version 2020.2.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin installed
    • Dart plugin version 202.8070

[✓] VS Code (version 1.51.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[!] Connected device
    ! No devices available

! Doctor found issues in 2 categories.
Process finished with exit code 0


Restore PLO v2 format export to use one less num data columns

Resolves #2003

I downside with this change is that any previous plo v2 file exported that works with the HopsanCore plo parser will now stop working. Fixing the file manually is easy if you know what to do though. (Reduce number of columns with 1).
Maybe it would be possible to add a check after reading the number of columns (the column names) and compare with the specifed number of columns to give a warning message, but I did not try to implement that.

We can look at this again for PLO v3, I have a separate branch to finish that


Adding documentation links to MapLibre README

Added documentation links for MapLibre GL Native Markdown Book, Android API Reference, and iOS API Reference to README.

There is already a docs section in the readme. It can be improved, but this pr seems to be outdated.


--config does not work with recent version of angular/config.js and so on (and here is why)

Latest version of changelog-cli wants an exported object
if (flags.config) {
    config = require(resolve(process.cwd(), flags.config));
  } else {
    config = {};
  }
// [...]

var changelogStream = conventionalChangelog(options, templateContext, config.gitRawCommitsOpts, config.parserOpts, config.writerOpts)

while latest config version of conventional-changelog-angular exports a promise and other versions a callback.
That's why the conventional-changelog library works with these configuration files but not the CLI.

We're closing this issue out as this repository has been merged into our conventional-changelog mono-repository.
If you would like to see this change incorporated into conventional-changelog please consider re-opening the issue against the conventional-changelog mono-repository.


Ensure mobile screen respects viewport height on Safari / iOS.

The viewport height is currently set using the TailwindCSS h-screen attribute. This results in a scrollable portion of the page on some WebKit-based mobile browser engines and an unpleasant UX.

FYI: Will also use this PR to update the theme-color values for dark and light modes via the site's meta tags.


Composer install not complete

I have tried to install jPlayer with Composer, but it only installs the full JS file and not the SWF and the minified version. Is it possible to change that?

Same issue here.
Only JS in component-dir, is that intended behavior?

It looks like the file got removed on purpose from the components part of composer.json for whatever reason, so it's likely to be intendet. I don't know if this file is being included somewhere else, but either this part, or the composer.json need to be fixed. Does domeone know, why it got removed?

Hi
I've added "dist/jplayer/jquery.jplayer.swf" into composer.json
    ...
    "extra": {
        "component": {
            "scripts": [
                "dist/jplayer/jquery.jplayer.js",
                "dist/jplayer/jquery.jplayer.swf"
            ],
    ...

And run "composer update" but the swf file was not copied.
Don't know ...

Does no one use composer to get jPlayer? O.o
I've just tried and I'm missing the jquery.jplayer.swf file too.


add support for 1.6.3

LGTM!

@drdarshan @jmatthewsr-ms @brfealy Kevin Kuo is joining RStudio, just FYI in case you were wondering who is pushing new commits. I'll try to catch up with you by mail next week, really looking forward to the WINUTILS integration!


Improve PreCheckFunc function signature

How to categorize this PR?

/area ops-productivity
/kind enhancement
What this PR does / why we need it:
This PR improves the PreCheckFunc function signature. This is required so that in extension when a new health check is added, We can add PreCheckFunc that skips the health check untill the object get available.
Which issue(s) this PR fixes:
Fixes #
Release note:

NONE

/invite @rfranzke


Changed char limit from 140 to 160

Optimum summary length should be 160 characters, not 140.
https://trello.com/c/iBVy0LA1/56-guidance-on-summaries-in-writing-for-govuk
I've increased the count threshold to 160 and changed the warning message to reflect that.

@keithiopia do you want us to merge and deploy this?

@tuzz yes please - could you hold off deploying till Friday?


Improve Tx review (fees)

Improve the way transaction fees and amounts are represented when reviewing transaction.
Improve JSON parser to understand these fees/amounts/denominations.
Add a menu to allow selecting units are atoms or uatoms.

This has been included as part of:
https://github.com/LedgerHQ/ledger-app-cosmos/pull/8
https://github.com/LedgerHQ/ledger-app-cosmos/pull/9
The only feature that has not been included is the menu to select units. This could be problematic because the Cosmos app operates in a generic way so any possible unit can be used.
I will close this issue and reopen one to for the specific menu option (https://github.com/cosmos/ledger-cosmos/issues/158)


[Hub Generated] Review request for Microsoft.KeyVault to add version stable/2019-09-01

If you are a MSFT employee you can view your work branch via this link.
Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

The validation passed, I checked the two checkboxes in the issue description which retriggered the pipelines, then they failed (i.e. no code changes). @akning-ms can your team look into this?

/azp run automation - sdk

/azp run automation - sdk
Timeout in JavaScript SDK generation - the other ones that previous failed without code change passed.

/azp run automation - sdk


Add log file writing functions

Moving the writing of log files to this package, as we can then use a consistent file naming scheme across all the tests.

This could resolve #27


[META] Internationalization (i18n) Setup

Fixes

Fixes #941 by @brenoferreira
Fixes #942 by @brenoferreira
Fixes #943 by @brenoferreira
Fixes #944 by @brenoferreira
Fixes #487 by @brenoferreira
Description

Setup the vue-i18n plugin to enable the internationalization of the cc-search website.
Screenshots

Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the master branch of the repository.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Please merge this into a temporary branch instead of develop so that we can make releases from develop without merging in partial i18n work.

Okie dokie, all PRs have been merged! Please resolve the merge conflicts and do some last minute cleanup if it's needed, then @ me and @zackkrida so we can do final reviews before merging this :) @AyanChoudhary

@akmadian @zackkrida I am done with rebase and have fixed some formatting issues as well, this is ready for the final review.

@zackkrida thanks for pointing those out, even I was not sure about the placement of the locale-selector @panchovm maybe you can help with the 1st and 3rd points above. I might have missed credit the creator part, I will see to it today itself.

You are right @zackkrida, it makes sense to have it across all pages of a site. Users should be able to change the site's language whenever they are. We have not discussed this so thanks for bringing it. In general, we have not thought about responsive components. Let me see @AyanChoudhary  how we can face this because this has been a redundant issue.

As discussed, we will be redesigning the locale chooser to be a part of the global site header! Should be cool.

@AyanChoudhary, terribly sorry about the insane # of merge conflicts. I just merged in a PR to add code formatting via prettier, and felt this was the best time to do it (since there are so few pending PRs). I'll find a good way to fix these conflicts in bulk so you don't have to touch nearly every file in the codebase 😆

@AyanChoudhary think i got it, if anything looks broken just feel free to remove my last commits and we will figure it out. Thank you!

@zackkrida sorry for replying back a little late. The changes don't break anything. And thanks for integrating those.

@zackkrida was waiting for the confirmation, will do it as soon as possible.

@zackkrida looks like I broke a couple of things while fixing the large amount of merge conflicts 😥, the things which look to be broken are:

the alignment of elements in the footer
the hide filters button

Surprisingly everything else is intact. I will fix these two as well.

No problem, we will figure it out this week!

Fixed those as well 😁


bug with fuzzysearch ???

Hi
I have a list with these 2 items in it: "Austria" and "Australia".
If I search for "Austria", the 2 items appear
HTML
`

Australia
India
France
Austria

`
JS
var monkeyList = new List('test-list', {    valueNames: ['name'] });
check here : codepen
thanks

Hi! If you want just the exact match search result to appear, I might suggest using the default search option, which can be implemented by simply replacing the "fuzzy-search" class with "search".


Blackhole + EU2 =

https://paste.ubuntu.com/24846278/

I Have the Same Issue The server crashes when the Blackhole unit imput slot gets full and another item attempts to add another of the same item. https://pastebin.com/TZNW1Fyi
Happened with RS cables Tiny Progressions Cobblegens and Mekanism conduits.


Why you should use Celery with RabbitMQ

Why you should use Celery with RabbitMQ
Introduction paragraph (2-3 paragraphs):
In today's technology space, there is an increase in event-driven architectures as companies develop solutions that require asynchronous communication between their microservices. Ingesting these events faster in system architecture and processing them enables system architectures to be persistent, resilient and allows batch processing of data. Celery and RabbitMQ are some tools used in these architectures. Choosing the right combination of tools and viewing an example of these tools that goes beyond the "hello world" is what this article cover.
Key takeaways:

Differentiate between Message Queues and Task Brokers
Identify examples of message queues and task brokers
Understand why to choose The Celery & RabbitMQ Combo
Be able to set up Celery with RabbitMQ

References:
https://docs.celeryproject.org/en/stable/getting-started/brokers/rabbitmq.html
https://dog.ceo/

topic approved 👍 @edeediong


Argument type System.IServiceProvider [System.Private.CoreLib] is not assignable to parameter type System.IServiceProvider [System.ComponentModel]

I have an mvc core website (VS 2017, Core 1.1) with a Configure method in the Startup file like this:
public void Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory loggerFactory)
{

         if (env.IsDevelopment())
         {
              SeedData.Initialize(app.ApplicationServices);
              Configuration.Reload();
         }
}

And the SeedData.Initialize is like this:
public static void Initialize(IServiceProvider serviceProvider)
I get the following warning:

In order to avoid this warning I'm doing this in the configure method:
SeedData.Initialize((IServiceProvider)app.ApplicationServices);
Am'I doing something wrong and I get this warning? Is there a conflict in one of the packages I'm using?
How can I stop the unnecessary casting?
It's not really an issue because everything works fine, I just don't like to see the warning every time I go to the Startup code.
I have seen in another issue #https://github.com/dotnet/corefx/issues/12678 about some Type duplications between coreclr and corefx and that you are working towards removing the duplications. Is what I see falling under this cleanup or is completely irrelevant?

This is definitely strange... Are you doing anything exotic in your csproj? Did you manually reference System.ComponentModel?

Have the same.
I referenced a .dll from my test class library.

Almost seems like you're referencing a runtime binary (System.Private.CoreLib). Might help if you included your project files to see if there's an odd binary being added to your references.

I don't think i do anything exotic in my .csproj and I don't remember manually referencing the System.ComponentModel. I always tap Alt+Enter to add the System.ComponentModel.DataAnnotations reference to a .cs file if adding the [Required] attribute.
Here is my .csproj:
<Project Sdk="Microsoft.NET.Sdk.Web">

  <PropertyGroup>
    <TargetFramework>netcoreapp1.1</TargetFramework>
    <MvcRazorCompileOnPublish>false</MvcRazorCompileOnPublish>
    <PreserveCompilationContext>true</PreserveCompilationContext>
    <PackageTargetFallback>$(PackageTargetFallback);portable-net45+win8+wp8+wpa81;</PackageTargetFallback>
  </PropertyGroup>

  <ItemGroup>
    <None Update="wwwroot\**\*;Areas\**\*.cshtml">
      <CopyToPublishDirectory>PreserveNewest</CopyToPublishDirectory>
    </None>
  </ItemGroup>

  <ItemGroup>
    <PackageReference Include="AutoMapper" Version="6.0.2" />
    <PackageReference Include="AutoMapper.Extensions.Microsoft.DependencyInjection" Version="2.0.1" />
    <PackageReference Include="BuildBundlerMinifier" Version="2.4.337" />
    <PackageReference Include="Microsoft.ApplicationInsights.AspNetCore" Version="2.0.0" />
    <PackageReference Include="Microsoft.AspNetCore" Version="1.1.1" />
    <PackageReference Include="Microsoft.AspNetCore.Authentication.Cookies" Version="1.1.1" />
    <PackageReference Include="Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore" Version="1.1.1" />
    <PackageReference Include="Microsoft.AspNetCore.Mvc" Version="1.1.2" />
    <PackageReference Include="Microsoft.AspNetCore.Mvc.Razor.ViewCompilation" Version="1.1.0" />
    <PackageReference Include="Microsoft.AspNetCore.Rewrite" Version="1.0.1" />
    <PackageReference Include="Microsoft.AspNetCore.Session" Version="1.1.1" />
    <PackageReference Include="Microsoft.AspNetCore.StaticFiles" Version="1.1.1" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.Tools" Version="1.1.0" PrivateAssets="All" />
    <PackageReference Include="Microsoft.Extensions.Logging.Debug" Version="1.1.1" />
    <PackageReference Include="Microsoft.VisualStudio.Web.CodeGeneration.Design" Version="1.1.0" PrivateAssets="All" />
    <PackageReference Include="Microsoft.VisualStudio.Web.BrowserLink" Version="1.1.0" />
  </ItemGroup>

  <ItemGroup>
    <DotNetCliToolReference Include="Microsoft.EntityFrameworkCore.Tools.DotNet" Version="1.0.0" />
    <DotNetCliToolReference Include="Microsoft.VisualStudio.Web.CodeGeneration.Tools" Version="1.0.0" />
  </ItemGroup>

  <ItemGroup>
    <ProjectReference Include="..\V8.Api.Services.Models\V8.Api.Services.Models.csproj" />
    <ProjectReference Include="..\V8.Api.ViewModels\V8.Api.ViewModels.csproj" />
    <ProjectReference Include="..\V8.Data\V8.Data.csproj" />
  </ItemGroup>

</Project>

Also looked through the project dependencies and the nuget cache and can't see any reference to any package System.Private.CoreLib.

@flatproject - would you be able to share your application (e.g. in a GitHub repo) so we can take a look?

Apologies for the delay. I have put the project here https://github.com/flatproject/6237 stripped down to bare minimum. It compiles but not runs and the warning is still there.

I tried your application and it seems to work fine:

Would you mind clearing your global packages directory and package cache and see if that fixes your issue? Grab nuget.exe from https://dist.nuget.org/win-x86-commandline/latest/nuget.exe and run nuget.exe locals -Clear All and do a restore.

@pranavkm Thanks for looking into this. I had already tried the nuget global package reset and I tried it again but still the warning shows. The application is working fine, the piece of code giving me the warning is working fine so it must be something with my setup. I will look into moving my solution to a fresh one in the next few days and hopefully it will clear up the warning.

+1

Closing since this appears to be resolved

@flatproject  any news on this front.. having  the same issue

@MCGPPeters  I moved all the code to a new solution and manually recreated all projects and the same thing happens so nothing new to report! At this point i think the issue is with the Resharper extension as most of my false positives in the past were related to Resharper. Not really sure how to test if it is Resharper or Microsoft related issue.

same here but only when resharper is installed. I submitted a bug report to JetBrains.

Hello! Please try installing the latest ReSharper 2017.2 EAP3 (https://www.jetbrains.com/resharper/eap/) and check the issue once again. Apparently, the issue is fixed in this version.

@MPleskunina same issue still persists after updating today to the latest build 2017.1.3
I have cleared all nuget packages using nuget.exe locals -Clear All and also emptied all Resharper caches. Not sure what else i can try...

@fexdev @flatproject The problem should be solved in the latest EAP build: https://youtrack.jetbrains.com/issue/RSRP-464676#comment=27-2290098. You can download it here: https://www.jetbrains.com/resharper/eap/.

@MPleskunina, still having this in 2017.2 EAP 9:
Cannot convert expression type 'System.IServiceProvider [System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]' to return type 'System.IServiceProvider [System.ComponentModel, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a]`

@alexb5dh Please feel free to comment YouTrack ticket: https://youtrack.jetbrains.com/issue/RSRP-464676.
Any information to reproduce the issue will be highly appreciated (sample solution, repro steps etc).
Thank you!


Error: 'module' object has no attribute 'find_file'

I'm using SublimeLinter-contrib-rustc with ST3 on Windows. Whenever I open a file with the linter enabled, i get this error:
From a little  googling it looks like maybe SublimeLinter made a breaking change and not everyone has caught up yet? I'm not sure.
SublimeLinter: #1 rust render.rs ERROR:
=======================================

Linter crashed.

Traceback (most recent call last):
  File "lint.backend in C:\Users\Quinn Freedman\AppData\Roaming\Sublime Text 3\Installed Packages\SublimeLinter.sublime-package", line 100, in execute_lint_task
  File "lint.linter in C:\Users\Quinn Freedman\AppData\Roaming\Sublime Text 3\Installed Packages\SublimeLinter.sublime-package", line 761, in lint
  File "linter in C:\Users\Quinn Freedman\AppData\Roaming\Sublime Text 3\Installed Packages\SublimeLinter-contrib-rustc.sublime-package", line 70, in run
AttributeError: 'module' object has no attribute 'find_file'

This appears to be the same as #33. I am happy to accept a PR to fix it, but I think most users are better off using Rust Enhanced instead.


Add blob example

It took me a while to figure out that calling into() on a Vec<u8> in order to convert it into Bytes doesn't work as I expected.
impl<T: Into<Bytes> + Clone + Debug> From<Vec<T>> for Bytes exists but I'm not sure what it does exactly.

Changes Unknown when pulling aed115548f828af760019eb574d514e22cd88714 on dfaust:master into ** on AlexPikalov:master**.

Changes Unknown when pulling aed115548f828af760019eb574d514e22cd88714 on dfaust:master into ** on AlexPikalov:master**.

Changes Unknown when pulling aed115548f828af760019eb574d514e22cd88714 on dfaust:master into ** on AlexPikalov:master**.

Changes Unknown when pulling aed115548f828af760019eb574d514e22cd88714 on dfaust:master into ** on AlexPikalov:master**.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #106   +/-   ##
=======================================
  Coverage   95.56%   95.56%
=======================================
  Files          18       18
  Lines        1870     1870
=======================================
  Hits         1787     1787
  Misses         83       83

Continue to review full report at Codecov.

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

@dfaust impl<T: Into<Bytes> + Clone + Debug> From<Vec<T>> for Bytes converts a list of values into Bytes which is one of steps to convert into Value and then pass into query builder. For blob type which is already an array of bytes we can create a value directly Value::new_normal(blob). To be consistency with other type we may want to implement impl Into<Bytes> for Vec<u8> simply as
impl Into<Bytes> for Vec<u8> {
    fn into(self) -> Bytes {
        Bytes(self)
    }
}

This would be enough to be able to call vec![1,2,3,4].into()

as for
impl Into<Bytes> for Vec<u8> {
    fn into(self) -> Bytes {
        Bytes(self)
    }
}

I've just checked it raises a compilation error as a conflicting implementation

Yep, this needs specialization (https://github.com/rust-lang/rfcs/pull/1210/) which is still behind a feature flag on nightly rust.

And what I don't understand is, why impl<T: Into<Bytes> + Clone + Debug> From<Vec<T>> for Bytes doesn't work right now. If it was doing what I expected, it should convert the Vec<u8> into Bytes correctly but with a lot of unnecessary overhead. But it doesn't work, the data gets altered.

In case of blob or Vec<u8> in other words what we actually need is to pass an array of bytes into Value. Value is a 4-bytes length and array of bytes. What impl<T: Into<Bytes> + Clone + Debug> From<Vec<T>> for Bytes does is maps a list of separate values into Value one by one. Then concatenates bytes into a single byte array. Then basing of this array method builds non-empty value.
In case of blob (Vec<u8>) what we want to have is vec![1, 2] -> vec![0, 0, 0, 2, 1, 2]. And what impl<T: Into<Bytes> + Clone + Debug> From<Vec<T>> for Bytes does is vec![1, 2] -> vec![0, 0, 0, 1, 1, 0, 0, 0, 1, 2] -> vec![0, 0, 0, 10, 0, 0, 0, 1, 1, 0, 0, 0, 1, 2]

So there is an issue here. That for blob we should consider Vec<u8> as a single value, while in other cases ie. Vec<String> should be considered as a set of values (Cassandra's list or set type)

Yes, looking into mapping table I was thinking about the same. Probably Into<u8> is useless because of reasons you mentioned. So removing its implementation would allow to add without conflicts
impl Into<Bytes> for Vec<u8> {
    fn into(self) -> Bytes {
        Bytes(self)
    }
}


Greets and a few comments.

@jared-stilwell (and contributors) thanks for taking on maintaining escomplex. I'd certainly like to integrate it with ESDoc as a plugin in addition to using D3 for interactive graphs of the data generated. I've already done basic integration tests and here are my thoughts (hopefully this isn't a bit presumptuous).
I'd just like to know your intentions (I've read your blog posts) for escomplex and if you can see yourself and others working on it to get it up to production standards?  I'd be glad to help out in this regard or simply fork what you have as the skeleton framework is here. I'd prefer to collaborate / contribute if possible. The velocity is quite slow and unclear if there are intentions to make a production ready release on the horizon.
From one of your blog posts...

We'll continue using Esprima and set a goal to support any parser that complies with the EStree specification.

As it stands currently Esprima is a "surface" dependency (though not added in the package.json!) for escomplex I do recommend splitting off all of the AST processing to escomplex-core. And leave escomplex to just the index.js. Esprima is only used in index.js to generate the AST and is an unnecessary dependency for integration where parsing is already handled.
The reason being is that Espree is also a common ESTree based parser and is the one used in ESDoc and AST is already generated in ESDoc which can easily be sent to the core functionality of escomplex.
Here is the simple test integration I've made which outputs escomplex reports for every file processed by ESDoc (this is ES6 code too as espree is configured for ES6):
esdoc-plugin-escomplex/src/analyse.js
esdoc-plugin-escomplex/src/plugin.js
The main reason ES6 code is not parsed by escomplex right now is because incomplete options are being fed to esprima. If esprima is anything like espree too you have to vary the options depending on code being processed.
Of course the AST import directives are not being processed by escomplex and such. I can certainly help add the ES6 AST processing as I've been working a lot with ES6 AST recently.
Other small comments if you move forward I suggest creating a Github organization for escomplex and related repos. It makes it a lot easier to organize collaborators.
If there is a chance that you don't see yourself and others getting a production version together that supports ES6 anytime soon let me know. I'd be glad to setup a Github organization and just get it done as a production version shouldn't take more than 3 days effort if that. Again a bit presumptuous on my part.

@typhonrt , thanks for the feedback.
My intention is still to release 2.0.0 once it has a reasonable dependency structure and can support ES <= 6. However, as you pointed out, progress towards this end has been slow to the point of stalling.
Like yourself, I'd also like to see the escomplex project reach something like production-level quality/maturity. Towards that end, your recommendation to create an escomplex github organization seems correct. I'm happy to set this up and migrate all active projects to it.
Your suggestions all seem reasonable. I'd like to get a list of issues put together to capture them and perhaps carve out a milestone towards a 2.0 release. As long as the build stays clean and the changes are reasonable, I'm happy to review as many PRs as necessary to build traction.
Any help you can lend, presumptive or not, toward the end of getting escomplex to a better state is appreciated. Again, thanks for sharing your insight and offering to help out.

I'm really motoring with my ESDoc efforts and larger platform efforts that certainly benefit from thorough documentation and adding complexity reports along with everything else documentation related I'm doing is icing on the cake. I'll take a look at perhaps doing a separate refactor as a proof of concept later in the week and if that is deemed successful you'd be more than welcome to the code as ultimately I'm interested in folks continuing to work on accuracy of the reports generated as things progress and once a solid release is out there that seems quite possible. It's mostly all there for escomplex right now with just a few rough edges, so it'd be great to get thing working smoothly especially with direct outside integration without taking on extra dependencies.

@typhonrt Sounds good.


Provide Some for user on composingSemigroups

Removed cognitive distraction that Map.get returns Option[T]

Thanks for the contribution @metasim, the res0 parameter type in that function needs to be updated too since it's a user-provided result.

Dang... merge conflict. Never done a pull request on a pull request. Need to dig into it.

Thanks @RawToast and @metasim for your time!


Fix tests target in Makefile and add automatic running of tests

Add -I$(GTEST_INC) when generating .d files, otherwise make test fails on Line 8 of tests/cpp/unittest.mk on macOS Sierra.
$(CXX) -std=c++0x $(CFLAGS) -I$(GTEST_INC) -MM -MT tests/cpp/$* $< >tests/cpp/$*.d

Can one of the admins verify this patch?


Searching for packages returns duplicate results

Describe the bug
It seems that search results are not distinct, meaning that the same package can appear more than once in different pages of the results.
Expected behavior
Each package should appear only once in the search results.
To Reproduce
Here's a script that goes over the first 10 pages of search results for the term "relative", scrapes the package names from each page, and prints some of the duplicates:
import requests
import re
from collections import Counter

REGEXP = r'
\s*(.+?)'

c = Counter()
for page in range(1, 11):
    r = requests.get('https://pypi.org/search/?q=relative&page=%d' % page)
    for match in re.finditer(REGEXP, r.text):
        c.update([match.group(1)])

for entry in c.most_common(10):
    print(entry)

Sample output:
('bel', 2)
('compress_pyscripts', 2)
('attr_ns', 2)
('Axiom', 2)
('bakatrouble-pony-migrate', 2)
('chiru150', 2)
('codd', 2)
('collective.behavior.discount', 2)
('CreateMucRoom', 2)
('cs.csvutils', 2)

Interestingly, if the search results are ordered by "date last updated" instead of "relevance", the problem disappears. This probably means that "relevance" does not impose a concrete ordering over the results, so if you have two packages with the same relevance score, they may move up or down in the list of results relative to one another. If that is the case, simply ordering primarily by relevance and then by some unique id (or by the date) should resolve the issue.

Seems to me like the issue is that paginated URLs for the same search query are cached separately, so pages like:

https://pypi.org/search/?q=relative&page=2
https://pypi.org/search/?q=relative&page=3

might each represent different points in time and thus, the same package might appear in multiple pages.
The bigger problem is that this also means that a package could fall in between the cached pages and thus could be impossible to find (for a given query).

One way we could resolve this is by:
a) limiting the number of results returned for a term (e.g. to 100)
b) tagging every page for a given term with a surrogate key for every project
c) purging all pages with a given surrogate key when a project is updated
Some downsides to this:

would limit the total number of results
would need to precompute/query for all 100 results, even if only the first page is hit
would require adding a lot of surrogate keys to these pages

If we would pre-compute the 100 first results anyway, why paginate ? The HTML page is not going to be that much bigger if we send the 100 results. (And if we need really pagination for UX reasons, this could be done on JS side)


Fix no message handling

It made requests even when the queue was empty.

Current coverage is 93.60%

Merging #25 into master will increase coverage by +0.02% as of 4136e5c

@@            master    #25   diff @@
=====================================
  Files           10     10
  Stmts          265    266     +1
  Branches        12     13     +1
  Methods          0      0
=====================================
+ Hit            248    249     +1
  Partial          0      0
  Missed          17     17

Review entire Coverage Diff as of 4136e5c

Uncovered Suggestions

+1.88% via ...r/PusherClient.scala#160...164
+1.13% via ...er/PusherActor.scala#72...74
+1.13% via ...r/PusherClient.scala#145...147
See 6 more...

Powered by Codecov. Updated on successful CI builds.


State color bleeding over corners no matter the radius

[X ] I updated to the latest version available
[X ] I cleared the cache of my browser

Release with the issue:
1.10.9
Last working release (if known):
Browser and Operating System:
Chrome browser(Win 11) and on HASS android app
It happens at all times no matter the device I'm in.
Description of problem:

It appears the border radius on the right is about a pixel or so off from the slider overlay. I'm able to remove it manually in CSS on the browser by adjusting the right side border radius to a larger radius, but I can't add custom CSS with card mod to the card to get it to stay
Also I know I know small issue, I'm just picky and can't figure it out haha :)

I think this is not a specific problem of this Lovelace card but more a generic problem of the browser's rendering engine. There is a workaround possible as described in this GitHub ghist. Although it is related to the Safari browser, I can confirm that it also works on Chromium-based browsers (tested on Edge and Fully Kiosk).
I say "workaround" because it doesn't solve the problem 100% but at least to a degree that you can hardly see the issue afterwards (especially on displays with high pixel density). Additionally the fix can be permanently applied to the slider-button-card using card-mod (apply to ha-card > div > div.slider).


ENH Copy plots on Card save

Reference Issues/PRs

Fixes #300
What does this implement/fix? Explain your changes.
Implements copying files to the path of the repository.
Any other comments?

@BenjaminBossan @adrinjalali Here's the draft PR, there's not much because I wanted to make sure I was thinking of this correctly.
From what I understood from the thread on the issue, we're wanting to copy the files to the repo path whenever add_plot method is called. Is that understanding correct?
Also, I've been trying to think of how best to test this. Do y'all have any ideas?
Sorry, this PR is so small. I just wanted to make sure I was heading in the right direction before jumping into the deep end.

I'd be interested in what @BenjaminBossan thinks here too though.

@BenjaminBossan I've moved the logic into _generate_content and this is ready to review! Thanks! 👍🏽

I can't think of a use case, but maybe I miss something?

+1

@BenjaminBossan Thanks for the feedback! I refactored this to copy the files into the same path as the README. I also went ahead and added a second plot to the test.
(Side note it looks like there are 34 warnings when running pytest. Is that something that needs fixing? If so I can start working on a PR)

Sorry, I accidentally submitted "Approve" when I wanted to submit "Request changes", I don't know how to change it now -__-

@lazarust Thanks for working on the test, but it looks like something is broken now. Could you please check?

@BenjaminBossan Should be working now!

Thanks for fixing the test. The one failing CI job is unrelated, so please ignore it.
From the comments of my last review, not everything was addressed, right? Will you do that?

@BenjaminBossan Sorry about that, I've removed the copy_files parameter from everywhere other than Card.save()

@BenjaminBossan Sorry this took so long for me to get back to! This should be ready for you to take another look!


Clarification on Root Path Access and Integration with FastAPI Returning JSON String

Issue Description:
I'm exploring FastUI in combination with FastAPI and encountered a couple of issues that I would like some clarification on:
Accessing the Root Path (/) Without Explicit Definition:
In the FastUI README example, there is no explicit route defined for the root path /, yet the application is accessible at this path. How is this behavior achieved? Is there an implicit route handling for / that serves the FastUI frontend, or is there another mechanism at play?
Integration with FastAPI Returning JSON String Instead of Rendering UI:
When integrating FastUI with FastAPI, I noticed that accessing certain paths expected to render UI components instead returns a JSON string representation of those components. Here's a simplified example illustrating the issue:
`from fastapi import FastAPI, HTTPException
from fastapi.responses import HTMLResponse
from fastui import FastUI, AnyComponent, prebuilt_html, components as c
from pydantic import BaseModel, Field
app = FastAPI()
class User(BaseModel):
id: int
name: str
dob: date = Field(title='Date of Birth')
@app.get("/api/", response_model=FastUI, response_model_exclude_none=True)
def users_table() -> list[AnyComponent]:
# Implementation details...
@app.get('/{path:path}')
async def html_landing() -> HTMLResponse:
"""Expected to serve the FastUI frontend but returns JSON instead."""
return HTMLResponse(prebuilt_html(title='FastUI Demo'))`
I expected the /api/ endpoint to serve UI components, but it returns a JSON string. Similarly, the root path / handled by html_landing is expected to render the FastUI frontend but shows the JSON output. Is this the intended behavior, or am I missing a configuration step to render the UI properly?
Any guidance on these issues would be greatly appreciated. Also, I'm curious if there's a recommended way to integrate FastUI with FastAPI to ensure UI components are rendered as expected rather than returned as JSON strings.

my create env step:
micromamba create -n fastui_env -c conda-forge fastui python=3 nodejs yarn uvicorn pydantic pandas pyarrow pyyaml ipython jupyter ipykernel -y
mkdir fastui_project && cd fastui_project
yarn init -y
yarn add @pydantic/fastui @pydantic/fastui-bootstrap @pydantic/fastui-prebuilt


Add createAssetId function

Motivation
Creating an assetId manually is easy, but is also easy to mess up.  For new devs this function would be helpful.
Summary

Add new function createAssetId
Add documentation for createAssetId in Native Parameters -> Asset ID

Usage example
const assetId = createAssetId(contractId, subId)
Possible implementations
function createAssetId(contractId: string, subId: string) {
    const contractIdBytes = arrayify(contractId);
    const subIdBytes = arrayify(subId);
    const bits = sha256(concat([contractIdBytes, subIdBytes]));
    const assetId: AssetIdInput = {
        bits
    }
    return assetId
}

I agree with @Torres-ssf, there is already the getMintedAsset that returns pretty much the assetId and is the same function I had to use when writing the createAssetId. They are also located in the same file
Is there some special feature of the new createAssetId.


SW-1307 Improve performance of duplicate timeseries values

If an incoming timeseries values request has a lot of entries that are already in
the database, detecting the duplicates is currently pretty slow and leads to
request timeouts. Check for duplicates explicitly by querying for all of the
timestamps at once rather than relying on individual INSERT queries failing.
For a request whose values are entirely duplicates, this will result in the
server doing a single SELECT query rather than a ton of failing INSERTs.

Current dependencies on/for this PR:

main

PR #870   👈

This comment was auto-generated by Graphite.


Not storing failures to .respec_failures

Hi there,
I'm using Ubuntu 14.04 LTS, and when I run respec, it doesn't seem to save the failures to .respec_failures, which is the entire point of this gem. It works fine on a Mac.

I'm guessing you were running into an RSpec 2 vs rspec 3 issue. Try the latest version, which I've just fixed to work under RSpec 3.

I am experiencing this same issue using the latest, 0.8.1 - no failures are writing to .respec_failures. When I run my rspec tests I have one failed example and the file is not created. If I run respec fa message in the terminal says 'no failed tests', then the file .respec_failures is created, then all of the examples from my repo are run and recorded in .respec_failures.
Questions toward troubleshooting:

Is respec not compatible with certain versions of rspec? I tried  3.0.0 and 3.2.0
Is any certain formatting required? I tried specifying none and --format documentation.
I'm working on Mac OS 10.10.2.
Thanks for this gem and advice.

@carolmirakove
It's working for me with rspec 3.2.0 on Yosemite. Respec 0.8.1.
Demo shell session below. You shouldn't need to specify any formatter options. It should work fine with any that you specify (I run it with Fuubar regularly), but to narrow down on the problem, try without any.
If that doesn't help, you might need to dig in with a debugger to see why it doesn't create the failures file. Respec prints the rspec command that it invokes. You'll see it loads this file -- I'm curious what failures_path evaluates to.
g@woosh:~/tmp [m] $ cat test_spec.rb
describe 'test' do
  it 'fails' do
    expect(1).to eq(2)
  end
end
g@woosh:~/tmp [m] $ respec test_spec.rb
++ rspec test_spec.rb /Users/g/.rvm/gems/ruby-2.1.5/gems/respec-0.8.1/lib/respec/formatter.rb
F

Failures:

  1) test fails
     Failure/Error: expect(1).to eq(2)

       expected: 2
            got: 1

       (compared using ==)
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-expectations-3.2.0/lib/rspec/expectations/fail_with.rb:29:in `fail_with'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-expectations-3.2.0/lib/rspec/expectations/handler.rb:38:in `handle_failure'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-expectations-3.2.0/lib/rspec/expectations/handler.rb:50:in `block in handle_matcher'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-expectations-3.2.0/lib/rspec/expectations/handler.rb:27:in `with_matcher'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-expectations-3.2.0/lib/rspec/expectations/handler.rb:48:in `handle_matcher'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-expectations-3.2.0/lib/rspec/expectations/expectation_target.rb:54:in `to'
     # ./test_spec.rb:3:in `block (2 levels) in <top (required)>'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/example.rb:177:in `instance_exec'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/example.rb:177:in `block in run'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/example.rb:385:in `block in with_around_and_singleton_context_hooks'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/example.rb:343:in `block in with_around_example_hooks'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/hooks.rb:474:in `block in run'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/hooks.rb:612:in `run_around_example_hooks_for'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/hooks.rb:474:in `run'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/example.rb:343:in `with_around_example_hooks'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/example.rb:385:in `with_around_and_singleton_context_hooks'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/example.rb:174:in `run'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/example_group.rb:548:in `block in run_examples'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/example_group.rb:544:in `map'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/example_group.rb:544:in `run_examples'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/example_group.rb:512:in `run'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/runner.rb:110:in `block (3 levels) in run_specs'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/runner.rb:110:in `map'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/runner.rb:110:in `block (2 levels) in run_specs'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/configuration.rb:1526:in `with_suite_hooks'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/runner.rb:109:in `block in run_specs'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/reporter.rb:62:in `report'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/runner.rb:108:in `run_specs'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/runner.rb:86:in `run'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/runner.rb:70:in `run'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/lib/rspec/core/runner.rb:38:in `invoke'
     # /Users/g/.rvm/gems/ruby-2.1.5/gems/rspec-core-3.2.0/exe/rspec:4:in `<top (required)>'
     # /Users/g/.rvm/gems/ruby-2.1.5/bin/rspec:23:in `load'
     # /Users/g/.rvm/gems/ruby-2.1.5/bin/rspec:23:in `<main>'
     # /Users/g/.rvm/gems/ruby-2.1.5/bin/ruby_executable_hooks:15:in `eval'
     # /Users/g/.rvm/gems/ruby-2.1.5/bin/ruby_executable_hooks:15:in `<main>'

Finished in 0.00309 seconds (files took 0.23328 seconds to load)
1 example, 1 failure

Failed examples:

rspec ./test_spec.rb:2 # test fails

g@woosh:~/tmp [m] $ cat .respec_failures
test fails

@oggy I made a foolish usage mistake - I ran my original set of tests with rspec instead of respec. Your demo helped me realize my error. Works perfectly - thank you, this is a very useful gem!

@carolmirakove Awesome -- glad it's useful!


[BUG] README blog post link is not working

Description/Screenshot
The blog post link under (Alternative Setup Method) Include AI JS SDK script and initialize statically is directed to a broken link.

Steps to Reproduce
Click on the this blog post link.

OS/Browser:
SDK Version [e.g. 22]:
How you initialized the SDK:

Expected behavior
Additional context
Add any other context about the problem here.

Not sure why we are linking out to whatever that blog post was saying -- need to more to an Azure Docs link or add details to GitHub (this later is probably the better option -- perhaps raising and linking to an issue with the details).

@Karlie-777 can you please have a look at this when you get a chance

https://github.com/microsoft/ApplicationInsights-JS/pull/1771

v2.8.1 is now fully deployed

Rolled back CDN to v2.7.4 due to #1822

v2.8.3 is now fully deployed to the CDN


Sumatory of all inputs of the model.

Reported by alainc on 16 Jun 2009 15:37 UTC
I'm developing a partial class, and I need to do the sumatory of all inputs in the model
The problem is the input variables will be defined in the subclass (the class which inherit from this) of this class, and I don't know how much input variables will be, and neither their names. In the language I use to develop it was something like:
SumOfInputs= sum ( SelectByType(interface,InTerminal))
There is any way to do this in Modelica? How can I use in a partial class input variables from their subclases?
I would like doing the sumatory in the partial class.
Thanks in advance

Migrated-From: https://trac.modelica.org/Modelica/ticket/176

Comment by dietmarw on 28 Jul 2009 08:07 UTC
Dear Alain,
This tracker is used for tracking issues in the development of the Modelica language or the Modelica Standard Library. It's not for support questions. But you may try and post your question to the http://www.modelica-forum.com . I'm sure there are people that can help you with it.

Modified by dietmarw on 28 Jul 2009 08:12 UTC


Add client parameter to Histogram Widget

Description
Shortcut: (link)
Type of change

Feature

Let's have a meeting on Monday, before landing this. I've discussed with @Josmorsot some other potential approaches to this, to avoid using props on all widgets

Closed, we have change the approach. See #729


Clarify ANTs & PyTest

Hi,
This PR aims to fix issue #4 and #8.
Best,
Alex

Cool, thanks @alexandreroutier !
I'll merge to master.


Added OpenAI Pre-trained weights to CLIP Models

Summary:
Added pre-trained flag to all CLIP models. This will pull pre-trained weights for any model there are open weights for, otherwise it'll provide a warning and ignore the flag. The flag pulls the OpenAI weights from S3 which are saved as state_dict and loads those weights. All of the weights have been tested to output the same outputs as their corresponding models in the OpenAI CLIP repo.
CLIP models that were configured differently from the OpenAI repo were updated to match; this was the ViT-l/14 model and all the ResNet models. Additionally, a parameter was added to the CLIP text encoder to be able to be configured to match the OpenAI configs.
Test plan:
I added new unit tests that call each CLIP model with a random image and tokens and check the first 5 features of the text and image embeddings against an expected output. Running pytest tests/models/clip/test_clip.py -vv will run the tests. I removed the cached models on the server before running the test to verify the pre-trained weights were correctly pulled from S3 during the test.
Fixes https://github.com/facebookresearch/multimodal/issues/384

Codecov Report
Base: 63.52% // Head: 64.17% // Increases project coverage by +0.64% :tada:

Coverage data is based on head (793f32d) compared to base (403b53a).
Patch coverage: 94.59% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #397      +/-   ##
==========================================
+ Coverage   63.52%   64.17%   +0.64%
==========================================
  Files         143      143
  Lines        9605     9728     +123
==========================================
+ Hits         6102     6243     +141
+ Misses       3503     3485      -18

Impacted Files
Coverage Δ

torchmultimodal/models/clip/text_encoder.py
100.00% <ø> (ø)

torchmultimodal/models/clip/model.py
87.36% <86.20%> (+35.75%)
:arrow_up:

tests/models/clip/test_clip.py
100.00% <100.00%> (ø)

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.

Overall looks good to me. Are there any references for the updated ResNet parameters? Or for any the CLIP variations. Would be great to add some info on where they're from/where they're used in a docstring

The updated parameters are reverse engineered from the saved checkpoints from the CLIP library. I'm not sure where the original values came from.


Allow increasing executorPoolSize

A p4d.24xl offers 4x100Gbps of throughput. 25 threads will most likely not max out available bandwidth. Allowing configuration of executorPoolSize would allow for more threads, and faster s3 throughput.
I'd need to run a test with this library, but recently I saw 100Gibps of throughput to a m5n.24xl using ~90 threads downloading from s3, where-as with 25 threads downloading from s3 I got just 44.286Gibps of throughput.
Currently this library hard-codes 25 threads for s3 downloads:
https://github.com/aws/amazon-s3-plugin-for-pytorch/blob/38284c8a5e92be3bbf47b08e8c90d94be0cb79e7/awsio/csrc/io/s3/s3_io.cpp#L46

@cobookman
We're upstreaming the amazon-s3-plugin-for-pytorch into the torchdata package (https://github.com/pytorch/data/pull/165).
We're dropping support for this plugin.
The current s3 plugin doesn't have this feature, so do the new S3 IO datapipes. We'll backlog this feature request, and update the feature in the new S3 IO datapipes.


Feature request: Service editor improvements

This is a compiled list of improvements for service editor, which is still in beta stage.

[ ] Analytics Service Editor: warning for the mapping -> does not have class mapping for the target (needs some analytics here)
[ ] Handle the switch between single/multiple executions
[ ] Support explorer for multi execution: support addition/deletion
[ ] Allow better handling for setting parameters for test
[ ] Allow better support for parameters coming from URL
[ ] Upon creation, add current user as the owner of service (if initial getCurrentUser fails, do not retry or warn. Take it as an empty array of owners)
[ ] MAYBE Do validation on the number of owners (at least 1)
[ ] Restyle service execution editor to have a better visual distinction between panels
[ ] DISCUSS We should probably redesign this a little bit (it has a few panel being separated by thin border lines right now, perhaps, it's better to follow what we do in https://github.com/finos/legend-studio/pull/659 with query builder)

Close to file smaller issues

Split into
https://github.com/finos/legend-studio/issues/944
https://github.com/finos/legend-studio/issues/943
https://github.com/finos/legend-studio/issues/942


Update ddpg_pendulum.py

The environment Pendulum-v0 is out of date. Upgrading to version v1.

@fchollet It looks good in a new version and I added all the needed files


COLLECTIONS-791 - Simplify boolean expressions

Coverage remained the same at 90.142% when pulling 84fa7ecffb72f8d575e0372ecf6b8a8125bab0cb on arturobernalg:feature/COLLECTIONS-791 into e1f23b035d7caeab8333e8df6739eb143b64affb on apache:master.


First letter is repeated

https://github.com/billyeatcookies/mono/assets/95927277/b0c84629-1739-40a3-acce-25c0815952f1

The letter repetition is ANSI being removed?

@Moosems I think so. On <Return>, the text since last output is considered as new input and written to the stream, then that part is removed from the text widget. The next output read will contain the entered command as well (hence I had to remove it from editor, so that it won't be shown twice). See below:
https://github.com/billyeatcookies/mono/blob/2e89a388be49b26d501351bdde3f57b9e9142163/mono/terminal.py#L97-L107
You can try removing code for the removal of command and running that to see if there are any changes. As of now, I can't reproduce this error yet in Windows. Also is this an issue caused only with clear command?


Breaking Change: Fix RotateAboutGrabPoint and RotateAboutObjectCenter far interaction #6510

Describe the bug
After upgrading to 2.2.0, my MagicLeap control could grab and move, but the controller rotation would no longer effect the manipulated object. I tracked it to Issue #6510, and after reverting TwoHandMoveLogic.cs  and ManipulationHandler.cs to previous versions, my issue was solved and controller rotation effected the object as before.
I cannot test with WindowMR controllers, but this change may affect them as well.
To reproduce
Steps to reproduce the behavior:

Create a lumin project with MRTK 2.2.0
https://github.com/HoloLabInc/MRTKExtensionForMagicLeap add this to project
Run in zero iteration or build an mpk

Expected behavior
The controller's movement will work, but the rotation of the controller will not have any effect on the object.
Your setup (please complete the following information)

Unity Version 2019.2.7f2
MRTK Version v2.2

Target platform (please complete the following information)
MagicLeap
Additional context
tagging @MenelvagorMilsom

@MichealReed hey, I just got a chance to check this on Windows Mixed Reality 6dof controllers and it appears to be working fine. I imagine that this may be an issue with MRTKExtensionForMagicLeap. RotateAboutGrabPoint and RotateAboutObjectCenter use TryGetGripRotation in order to get the controller rotation. This is used to differentiate the pointer rotation and the hand pose rotation for articulated hands. However this should still provide the correct rotation for other controllers. Seeming as the this controller is implemented in the extension, I think it should be raised with them.


🎈 perf(spacing): remove rect to const

'rect' is never reassigned. Use 'const'

such pull requests so fun,hhh

such pull requests  i feel so fun,hh


Load Tweets

Sorry not an issue but a question, is it possible to load a single tweet in the lightbox? At the moment it supports entire url.
Thanks

Not out of the box, sorry.

No probs thanks.


How to evaluate the fcn-32 pascal caffemodel?

when I want to evaluate this given caffemodel : fcn-32s pascal, you gives a mean I/U 64.0. But I tried several times, I can not get the same value. I tried the follow ways:

resize the test image to 500 x 500, and resize the groundtruth to 500 x 500 too, then I got mean I/U 64.3
resize the test image to 500 x 500, and get the predict result with size 500 x 500, then I crop it to the size of original image, then got the mean I/U 64.5
keep the size of image, and input it directly(reshape the caffemodel's input layer), get the predict image of original size, then got the mean I/U 64.9
so my question is, how to process the image to get the right mean I/U 64.0, or is there code to get it by the author?
what's more, I test it on seg11valid.
can you help me, thank you.

Closing since the future branch is now deprecated; the relevant functionality has been merged to master.
Check out the fcn.berkeleyvision.org repo for master editions of the reference networks, weights, and code for learning, inference, and scoring.


Document and use consistent encoding expectations for redirect output property

Describe the problem
The current documentation for the redirect output property of the loading function just says it should be:

a string containing the location to which they should be redirected

From what I can tell, this string is being treated with different expectations around encoding status depending on the current runtime location:

The server will pass the redirect string into encodeURI(redirect)
The client passes the redirect string into new URL(redirect, current_url)

If the expectation is that the developer should return an already properly encoded URI string, then the server path results in double encoding.
If the expectation is that the developer should be returning an unencoded string, the client path is relying on the URL constructor to tolerate unencoded URI strings.  For the most part this works because the javascript engine bends over backwards, but can result in unintended results.
Example:

Say I have a /login page that accepts a redirect query parameter[1] that should contain the path of where to redirect the user after login.
Pages behind a login wall that need to redirect unauthenticated users to the /login page with an appropriate previously query parameter.
A user visits /app?foo=bar&fizz=buzz while unauthenticated, and the login wall wants to send them to the login page

If the login-wall treats the redirect string as pre-encoded:
return {
  status: 302,
  redirect: '/login?redirect=' + encodeURIComponenent(request.url.pathname + request.url.search)
}

Server side rendering will use a Location header of
encodeURI('/login?redirect=' + encodeURIComponent('/app?foo=bar&fizz=buzz'))
> "/login?redirect=%252Fapp%253Ffoo%253Dbar%2526fizz%253Dbuzz"  // double encoded query parameter

And the client will use
new URL('/login?redirect=' + encodeURIComponent('/app?foo=bar&fizz=buzz'), 'http://example.com')
> URL { href: "http://example.com/login?redirect=%2Fapp%3Ffoo%3Dbar%26fizz%3Dbuzz", origin: "http://example.com", protocol: "http:", username: "", password: "", host: "example.com", hostname: "example.com", port: "", pathname: "/login", search: "?redirect=%2Fapp%3Ffoo%3Dbar%26fizz%3Dbuzz" } // single encoded query parameters.

If the login-wall treats the redirect string as unencoded:
return {
  status: 302,
  redirect: '/login?redirect=' + request.url.pathname + request.url.search
}

Server side rendering will use a Location header of
encodeURI('/login?redirect=/app?foo=bar&fizz=buzz')
> "/login?redirect=/app?foo=bar&fizz=buzz"  // completely unencoded query parameter

And the client will use
new URL('/login?redirect=/app?foo=bar&fizz=buzz', 'http://example.com')
> URL { href: "http://example.com/login?redirect=/app?foo=bar&fizz=buzz", origin: "http://example.com", protocol: "http:", username: "", password: "", host: "example.com", hostname: "example.com", port: "", pathname: "/login", search: "?redirect=/app?foo=bar&fizz=buzz" }} // completely unencoded query parameters.

// Especially problematic because we effectively lose the `fizz` query param on the embedded redirect:
$_.searchParams.get("redirect")
> "/app?foo=bar"

[1] Yes, I'm familiar with the problems of open redirects.  This is a simplified example.
Describe the proposed solution
The redirect property should be treated as already encoded, and the documentation should reflect that.
Alternatives considered
No response
Importance
would make my life easier
Additional Information
No response

Possible duplicate of #2490

The encodeURI on the location header was added in PR #1273
Historically relevant issues:

#1228
#1238
#3126 (removed encodeURI from /kit/src/runtime/server/index.js respond())


chore: remove scripts/mk_all.sh

It has since been rewritten in Lean, and is unused.

CC @semorrison adomani

My main reason for wanting to keep this script was that Floris had problems using lake exe mk_all on windows.  However, I think that those issues were only marginally related to the lake script and they appear to have been solved.  I use the script regularly and CI uses is as well.  I would just like some confirmation from some MacOs user that it works as intended for them and then I'm happy to remove the shell script!

Worked for me on macos.
bors merge

I'm sad to see this - I'm still using it regularly

Rubén, why do you not use lake exe mk_all?


Add an example for "Set multiple variables in a single rule"

Issue in help/admin/admin/c-processing-rules/processing-rules-examples/processing-rules-examples.md
Because there is a limit of 150 rules, customers wonder how they can set 250 variables. Conditional statements allow you to do this: overwrite eVar2 with contextData.section if contextData.section is set.
You can have dozens of conditional statements in a single rule.
It would be good to show an example. This might also be mentioned in the tips and tricks section.

Thanks Bret! I've rewritten the following page to accommodate your recommendation: https://docs.adobe.com/content/help/en/analytics/admin/admin-tools/processing-rules/processing-rules-examples/processing-rules-copy-context-data.html
It should be published live by tomorrow.


build mochajs.org with Jekyll; closes #1640

Demo
Lots of stuff here, but the main things:

use Jekyll
convert index.md to actual GFM Markdown
a lot of edits of the documentation, including, but not limited to:

reorganization
grammar fixes
code style fixes
rewording confusing sentences
add a couple more examples
dynamically generate TOC
use CDNs for JS, though there's basically none

I've taken every effort to make the site look as close to the current version as it does now, and keep it compatible with old browsers.  However, I was not able to reuse the syntax highlighting scheme, as highlighting is now done by Pygments.

I'm not sure why it doesn't want to merge automatically.  This is rebased on top of 5e1fabd

somebody (not me) needs to look thru the demo site for problems.

ping @jbnicolai

I'll have a go ;)

ping! any updates here? :)

I've updated this PR:

Changed module which generates ToC to something I maintain, in case it needs fixing.
Create heading id's via redcarpet instead of JS
Fix Gemfile issue
Reduce depth of ToC, so it should match more closely with the original

The only stylistic difference that I can see is that the syntax highlighting isn't the same (imo, it's better).
Of course, there's all the changes to the actual documentation and code examples itself, too.
@rstacruz Sorry about this; one of us is going to have to throw out work.

cc @jbni

http://boneskull.com/mocha/#-w-watch seems broken in the TOC (should have two dashes between 'w' and 'watch)

To be honest, I'm not a big fan of the 2 level of nesting layout in the TOC. Especially on a smaller width is looks very confusing, eg:

How about we limit it to only show top-level entries?

The Gitter image now takes up more space, I think I prefer the old style.
New:

Old:

@jbnicolai that's odd... why would you see two levels of nesting?

@jbnicolai I know why.  because I didn't rebase into my gh-pages branch.

@jbnicolai you may wish to try to build the site yourself as well.  but anyway, I updated boneskull.com/mocha with the new code.

@boneskull I have, see http://blog.jbnicolai.com/mocha/
I suggest we go with #1777 though, as it includes both your branch, the competing branch and some additional changes :)

@jbnicolai Yes, let's go with your code anyway.


Update html() with notice on formatting

I have a MySQL JSON column that stores an array of uploaded files. This uploaded_files column is cast to array. The requirement is to display the links to each of those uploaded files.
Before I started using Filament, I display these records on a paginated table by iterating through Forms::all(). For the uploaded_files column, I iterate through each file (recall now cast to array) and dynamically generate links to the file:
<a href="{url($form->uploaded_files[$index])}"> {{$form->uploaded_files[$index]}} </a>

Switching to Filament, for the array-cast uploaded_files column, from reading the docs I expected to use formatStateUsing() to dynamically generate those links like above and use html() to prevent raw HTML tags from being displayed.
This caused Array to string exception because html() internally uses Str::of(...) whose argument expects a string, but my column is cast to array.
The way I currently handle this is to check the implementation of html(), and then remove call to html() and then make ->formatStateUsing() return Str::of(implode('', $urls))->toHtmlString()

We would need to add this to every function that uses ->formatStateUsing() under the hood then.

Alternatively, we can have a new section called Formatters and immediately under this section, we have this notice:
> All the methods in this section are state formatters. You should only use only one of these formatters at a time on the same TextColumn. This is because they internally use the same `formatStateUsing()` API, which may cause one to overwrite the other.

Thoughts?

Would be a good solution if people would read docs from top to bottom 😅 But I doubt it.

I think people should reach for view/custom columns if they want to render HTML inside links etc, like @damms005. I believe that you would open your app up to XSS attacks if you rendered the URL etc without proper sanitisation.


482 - arranged the table based on mission area and community partner …

…instead of mission type.

#482 QA
https://docs.google.com/spreadsheets/d/1zhNH7XGRlvwupBoZaclL82_snf7vLIs8lX3W21NOe9o/edit#gid=547627265


Use strong types to map lists instead of pure strings

The idea here is to implement enum's to map lists eg:

src/Meilisearch/Key.cs: public IEnumerable<string> Actions { get; set; }
src/Meilisearch/QueryParamaters/TasksQuery.cs: public List<string> Types { get; set; } and public List<string> Status { get; set; }
and others...

We may want to refactor this src/Meilisearch/TaskInfo.cs: TaskInfoType, TaskInfoStatus to a static class with public const fields.

Hi @brunoocasali,
Could you precise what you want to do here, it's not clear :

The idea here is to implement enum's to map lists eg:

You want enum at the begining

We may want to refactor this src/Meilisearch/TaskInfo.cs: TaskInfoType, TaskInfoStatus to a static class with public const fields.

Then you want a class with const fields.
    [JsonConverter(typeof(JsonStringEnumConverter))]
    public enum TaskInfoStatus
    {
        Enqueued,
        Processing,
        Succeeded,
        Failed
    }

    [JsonConverter(typeof(JsonStringEnumConverter))]
    public enum TaskInfoType
    {
        IndexCreation,
        IndexUpdate,
        IndexDeletion,
        DocumentAdditionOrUpdate,
        DocumentDeletion,
        SettingsUpdate,
        DumpCreation
    }

TaskInfoType and TaskInfoStatus are already enums mapping from Meilisearch response as string.
What would you change for this two types ?

Oh, sorry, let's make it clearer then:
In the Keys.cs we have the Actions field, which receives a list of Strings. That means I can add anything I want if they are a list of string like new string[] { "a", "b", "c" }, when I submit that to the server, I will receive an error and that is the thing I want to prevent.
Is there a way to accept a given set of values that the compiler will protect?
public IEnumerable<AvailableActions> Actions { get; set; } or something like that?
The second part is related to the way enums work in C#, I thought it could be the right way to solve the problem I pointed out above.

Ok, I see what you want !
This issue was something I started to solve with TaskInfoStatus and TaskInfoType to avoid magic strings.
I also don't like magic strings :)


add "remote.SSH.localServerDownload" setting

option to add wget/curl options

I think for this it would be better to add "remote.SSH.localServerDownload" setting as the official extension to download locally if there's some error downloading on remote
Originally posted by @jeanp413 in https://github.com/jeanp413/open-remote-ssh/issues/24#issuecomment-1299559557

Hi
I would love tihs settings too cause my remote server can't access internet.
In VSCode everything works great but with VSCodium i have : "could not resolve host github"

workaround: ssh into a remote where this works (could even be localhost) then scp the created vscodium-server directory under $HOME to your remote; but a setting to download via the local machine would definitely be useful!


remove Dimensions.get on currencies and countries

Uploading Screenrecorder-2021-06-01-16-29-34-393.mp4…


update video status to published when processing is finished

when we upload the video, we will set status pending
when we finish the full processing, then we set the status to published

@aninda052  you can do this task first if you are interested. 🙂

@aninda052 you can do this task first if you are interested. slightly_smiling_face

on it vai. :)

@foyzulkarim vai, we take visibility input from client. if we set pending as default value while uploading a video(we can do this from server side) , then i think we should remove visibility field from client. Is there any other  need for visibility filed? if yes, then we can use another filed called status and set pending as default value from the server side.


Add ability to apply the whitelist to input sockets and locators.

The UDP descriptors have been modified to support flags that
allow the interface white list to be applied to output sockets,
input sockets and/or locators.
If applied to input sockets, rather than binding to IP address
any (<IP_ADDRESS>), the input socket will bind to the multicast IP
if the locator is multicast or to specific allowed interface IPs.
If applied to locators, then the default unicast locator list
will be filtered against the allowed interface IPs.  It is not
applied in the case where the user has specified a default list.

Addresses https://github.com/eProsima/Fast-RTPS/issues/150

@richiware  Is there any interest in this PR? Due to the nature of the network in which we are working in, we are in need of something that allows us to isolate input channels as well as output.

@deng02  As this has been rebased on #231 and it is being discussed there, I will mark this as duplicate and close this PR

Closing as duplicate of #231


Fix for bitbucket local dashes in repo names

:sparkles: What kind of change does this PR introduce? (Bug fix, feature, docs update...)
Bug fix for BitBucket Local - by default, repos with spaces in are presented as dashes in the url. E.g.
"My Special Repo" in BitBucket Server is http://my-bitbucket/scm/myproject/my-special-repo
:arrow_heading_down: What is the current behavior?
C:\>nukeeper repo http://my-bitbucket/scm/myproject/my-special-project ACCESSTOKEN --maxpackageupdates 15 --consolidate --change Minor --include "Something.*" -v d
Matched uri 'http://my-bitbucket:7990/' to collaboration platform 'BitbucketLocal'
FindPushFork. Fork Mode is SingleRepositoryOnly
2020-02-08T18:03:25Z: Started
Getting from BitBucketLocal url rest/api/1.0/projects/myproject/repos?limit=999
Failed on repo my-special-project InvalidOperationException : Sequence contains no matching element
   at System.Linq.Enumerable.Single[TSource](IEnumerable`1 source, Func`2 predicate)
   at NuKeeper.BitBucketLocal.BitBucketLocalPlatform.GetUserRepository(String projectName, String repositoryName) in d:\a\r1\a\_NuKeeper PR Build\drop\Nukeeper.BitBucketLocal\BitBucketLocalPlatform.cs:line 91
   at NuKeeper.BitBucket.BitbucketForkFinder.IsPushableRepo(ForkData originFork) in d:\a\r1\a\_NuKeeper PR Build\drop\NuKeeper.BitBucket\BitbucketForkFinder.cs:line 56
   at NuKeeper.BitBucket.BitbucketForkFinder.FindUpstreamRepoOnly(ForkData pullFork) in d:\a\r1\a\_NuKeeper PR Build\drop\NuKeeper.BitBucket\BitbucketForkFinder.cs:line 38
   at NuKeeper.BitBucket.BitbucketForkFinder.FindPushFork(String userName, ForkData fallbackFork) in d:\a\r1\a\_NuKeeper PR Build\drop\NuKeeper.BitBucket\BitbucketForkFinder.cs:line 32
   at NuKeeper.Engine.GitRepositoryEngine.BuildGitRepositorySpec(RepositorySettings repository, String userName) in d:\a\r1\a\_NuKeeper PR Build\drop\NuKeeper\Engine\GitRepositoryEngine.cs:line 113
   at NuKeeper.Engine.GitRepositoryEngine.Run(RepositorySettings repository, GitUsernamePasswordCredentials credentials, SettingsContainer settings, User user) in d:\a\r1\a\_NuKeeper PR Build\drop\NuKeeper\Engine\GitRepositoryEngine.cs:line 43
Done at 2020-02-08T18:03:25Z

:boom: Does this PR introduce a breaking change?
No.
:bug: Recommendations for testing
:memo: Links to relevant issues/docs
:thinking: Checklist before submitting

[x] All projects build
[x] Relevant documentation was updated

What happens if there is a dash in the repo name - that will now be turned into a space and won't match? If we're comparing url to name, can we instead compare url to url? I mean on line 92, where there is x.Name.Equals(…),  is there a better field to use than x.Name ?

Or maybe it should be a string.Equals(Sanitised(x.Name), Sanitised(fullName), StringComparison.OrdinalIgnoreCase), where  a Sanitised function reduces them to the same form with respect to spaces and dashes?

I think we can do a reasonable Sanitised function here, reopen

Replaced by https://github.com/NuKeeperDotNet/NuKeeper/pull/991


Build system for tests & examples

Uses conan/cmake.

This works just fine with my AppleClang13. But, for strange reason it doesn't work with GCC-11 (although, it works when I import the P2300 code in concore). Also, it doesn't work with plain clang out of the box.

This works just fine with my AppleClang13. But, for strange reason it doesn't work with GCC-11 (although, it works when I import the P2300 code in concore). Also, it doesn't work with plain clang out of the box.

Build job is now checking tests & examples with clang-12 for every commit. I think I'm done with this PR. I'll focus next on adding unit tests.

This is AWESOME and a huge help. Thank you @lucteo.

My pleasure.
BTW. On my library, I also have the following checks that are not turned on here:

zero warnings for one compiler
clang-format doesn't find any differences (I find it hard to work without clang-format these days)
static checks (clang-tidy and cppcheck)
sanitizers

Does it make sense to have them here as well?

Hi @lucteo. I'm unfamiliar with this conan/cmake combo. I have been developing locally on Linux using clang++-12 with -std=c++20 -stdlib=libc++. I know how to specify the compiler and flags to cmake, but it seems that this command:
conan install .. --build=missing -s build_type=Release

is just using the system compiler and stdlib, which then causes cmake to error out due to a mismatch. Could you tell me how to make conan do my bidding?
Incidentally, problems like this is why I generally prefer using cmake itself to fetch dependencies. For instance, to get catch2:
Include(FetchContent)

FetchContent_Declare(
  Catch2
  GIT_REPOSITORY https://github.com/catchorg/Catch2.git
  GIT_TAG        v3.0.0-preview3
)

FetchContent_MakeAvailable(Catch2)

add_executable(tests test.cpp)
target_link_libraries(tests PRIVATE Catch2::Catch2WithMain)

(taken from https://github.com/catchorg/Catch2/blob/devel/docs/cmake-integration.md)

Hi
First question: does test-clang-12.sh script work for you?
Hopefully the answer is yes. If that's the case, then that's one good way to run everything.
The main advantage is that is using exactly the same build steps performed by the CI, so it can be used to fix strange CI failures.
The disadvantage is that it's using docker, so it's not native and it tends to consume a bit more resources.
Running test-clang-12.sh will show you the actual commands that can be used to build the solution. Here is a copy-paste from running the script:
export CC=/usr/bin/clang-12 CXX=/usr/bin/clang++-12 && conan install "/github/workspace" --build=missing -s build_type=Release  -s compiler=clang -s compiler.version=12 -s compiler.libcxx=libc++
export CC=/usr/bin/clang-12 CXX=/usr/bin/clang++-12 && cmake   -DCMAKE_C_FLAGS=" " -DCMAKE_CXX_FLAGS=" " "/github/workspace"
cmake --build . -v -j 4

The key here is to expose the compiler as environment variables before calling conan.
Instead of specifying the conan settings to be used, you can also define a conan profile. For example, on my mac, I've created ~/.conan/profiles/clang-12 with the following content:
[settings]
os=Macos
os_build=Macos
arch=x86_64
arch_build=x86_64
compiler=clang
compiler.version=12
compiler.libcxx=libc++
build_type=Release
[options]
[build_requires]
[env]
<EMAIL_ADDRESS><EMAIL_ADDRESS>CFLAGS=-L/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/lib
CXXFLAGS=-L/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/lib

With that, I can simply run: conan install <dir> --build=missing -pr clang-12
Please note that you still need to tell CMake what compiler you are using. Unfortunately, CMake doesn't pick up the compiler that conan uses. You can do that by using environment variables (like shown above), or by passing specifying the compiler as CMake parameters: cmake -DCMAKE_C_COMPILER=/usr/bin/clang -DCMAKE_CXX_COMPILER=/usr/bin/clang++ -G Ninja <dir>
On conan vs fetch_content
I'm not a build system expert, but my understanding is that CMake is not a package manager. I has some facilities that can be used when needing other packages (like fetch_content), but it's not complete. My understanding is that fetch_content always builds the dependent library along with the program; it can't use pre-built binaries.
A couple of years back, when I worked on my own programming language, I needed to depended on LLVM. You cannot simply build LLVM inside the CI. This is why I chose to go to conan in the first place.
Bottom line: I can't argue that this setup is the best. I just knows it's good and flexible enough to fit our purposes; and it was relatively easy for me to set it up.


Intermittent libxslt1-dev package install failures

Do NOT include any confidential information, this issue will be created on a public repo
This section is completed by the issue reporter
On which version(s) do you encounter this issue?
Please provide the output to 'git describe --all' for  all deployments where you have confirmed it occurs.
What type of issue is this? Please tick the appropriate box.

[x] bug - existing functionality that does not work for you as documented
[ ] feature - new functionality that you would like to see added

What is the expected behavior?
RPC deploys without failures on Xenial.
What is actually happening?
On Xenial builds, we're intermittently seeing:
TASK [os_cinder : Install apt packages] ****************************************
FAILED - RETRYING: TASK: os_cinder : Install apt packages (5 retries left).
FAILED - RETRYING: TASK: os_cinder : Install apt packages (5 retries left).
FAILED - RETRYING: TASK: os_cinder : Install apt packages (4 retries left).
FAILED - RETRYING: TASK: os_cinder : Install apt packages (4 retries left).
FAILED - RETRYING: TASK: os_cinder : Install apt packages (3 retries left).
FAILED - RETRYING: TASK: os_cinder : Install apt packages (3 retries left).
changed: [cinder2_cinder_scheduler_container-07cda304] => (item=[u'rpcbind', u'rsync', u'git', u'nfs-common', u'libpq-dev', u'libkmod-dev', u'libkmod2', u'libxslt1-dev', u'zlib1g', u'zlibc', u'libffi-dev', u'libssl-dev'])
changed: [cinder1_cinder_api_container-e616a8b8] => (item=[u'rpcbind', u'rsync', u'git', u'nfs-common', u'libpq-dev', u'libkmod-dev', u'libkmod2', u'libxslt1-dev', u'zlib1g', u'zlibc', u'libffi-dev', u'libssl-dev'])
changed: [cinder2_cinder_api_container-674f0d15] => (item=[u'rpcbind', u'rsync', u'git', u'nfs-common', u'libpq-dev', u'libkmod-dev', u'libkmod2', u'libxslt1-dev', u'zlib1g', u'zlibc', u'libffi-dev', u'libssl-dev'])
changed: [cinder1_cinder_scheduler_container-87e85c1a] => (item=[u'rpcbind', u'rsync', u'git', u'nfs-common', u'libpq-dev', u'libkmod-dev', u'libkmod2', u'libxslt1-dev', u'zlib1g', u'zlibc', u'libffi-dev', u'libssl-dev'])
FAILED - RETRYING: TASK: os_cinder : Install apt packages (2 retries left).
FAILED - RETRYING: TASK: os_cinder : Install apt packages (2 retries left).
FAILED - RETRYING: TASK: os_cinder : Install apt packages (1 retries left).
FAILED - RETRYING: TASK: os_cinder : Install apt packages (1 retries left).
failed: [cinder2] (item=[u'rpcbind', u'rsync', u'git', u'nfs-common', u'libpq-dev', u'libkmod-dev', u'libkmod2', u'libxslt1-dev', u'zlib1g', u'zlibc', u'libffi-dev', u'libssl-dev']) => {"cache_update_time": 0, "cache_updated": false, "failed": true, "item": ["rpcbind", "rsync", "git", "nfs-common", "libpq-dev", "libkmod-dev", "libkmod2", "libxslt1-dev", "zlib1g", "zlibc", "libffi-dev", "libssl-dev"], "msg": "'/usr/bin/apt-get -y -o \"Dpkg::Options::=--force-confdef\" -o \"Dpkg::Options::=--force-confold\"     install 'rpcbind' 'nfs-common' 'libpq-dev' 'libxslt1-dev' 'zlibc'' failed: E: Unable to correct problems, you have held broken packages.\n", "stderr": "E: Unable to correct problems, you have held broken packages.\n", "stdout": "Reading package lists...\nBuilding dependency tree...\nReading state information...\nSome packages could not be installed. This may mean that you have\nrequested an impossible situation or if you are using the unstable\ndistribution that some required packages have not yet been created\nor been moved out of Incoming.\nThe following information may help to resolve the situation:\n\nThe following packages have unmet dependencies:\n libxslt1-dev : Depends: libxml2-dev (>= 2.6.26) but it is not going to be installed\n", "stdout_lines": ["Reading package lists...", "Building dependency tree...", "Reading state information...", "Some packages could not be installed. This may mean that you have", "requested an impossible situation or if you are using the unstable", "distribution that some required packages have not yet been created", "or been moved out of Incoming.", "The following information may help to resolve the situation:", "", "The following packages have unmet dependencies:", " libxslt1-dev : Depends: libxml2-dev (>= 2.6.26) but it is not going to be installed"]}
failed: [cinder1] (item=[u'rpcbind', u'rsync', u'git', u'nfs-common', u'libpq-dev', u'libkmod-dev', u'libkmod2', u'libxslt1-dev', u'zlib1g', u'zlibc', u'libffi-dev', u'libssl-dev']) => {"cache_update_time": 0, "cache_updated": false, "failed": true, "item": ["rpcbind", "rsync", "git", "nfs-common", "libpq-dev", "libkmod-dev", "libkmod2", "libxslt1-dev", "zlib1g", "zlibc", "libffi-dev", "libssl-dev"], "msg": "'/usr/bin/apt-get -y -o \"Dpkg::Options::=--force-confdef\" -o \"Dpkg::Options::=--force-confold\"     install 'rpcbind' 'nfs-common' 'libpq-dev' 'libxslt1-dev' 'zlibc'' failed: E: Unable to correct problems, you have held broken packages.\n", "stderr": "E: Unable to correct problems, you have held broken packages.\n", "stdout": "Reading package lists...\nBuilding dependency tree...\nReading state information...\nSome packages could not be installed. This may mean that you have\nrequested an impossible situation or if you are using the unstable\ndistribution that some required packages have not yet been created\nor been moved out of Incoming.\nThe following information may help to resolve the situation:\n\nThe following packages have unmet dependencies:\n libxslt1-dev : Depends: libxml2-dev (>= 2.6.26) but it is not going to be installed\n", "stdout_lines": ["Reading package lists...", "Building dependency tree...", "Reading state information...", "Some packages could not be installed. This may mean that you have", "requested an impossible situation or if you are using the unstable", "distribution that some required packages have not yet been created", "or been moved out of Incoming.", "The following information may help to resolve the situation:", "", "The following packages have unmet dependencies:", " libxslt1-dev : Depends: libxml2-dev (>= 2.6.26) but it is not going to be installed"]}

We currently haven't seen this issue on any Trusty builds.
What were you attempting to do when you encountered this error (give as much detail as possible)?
Ran OnMetal multi-node AIO periodic builds. The earliest occurrence that we've seen of this error was March 13th.
https://rpc.jenkins.cit.rackspace.net/job/OnMetal_Multi_Node_AIO_newton-xenial/4/
https://rpc.jenkins.cit.rackspace.net/job/OnMetal_Multi_Node_AIO_newton-xenial/7/
https://rpc.jenkins.cit.rackspace.net/job/OnMetal_Multi_Node_AIO_master-xenial/24/
https://rpc.jenkins.cit.rackspace.net/job/OnMetal_Multi_Node_AIO_master-xenial/25/
How can we reproduce this issue?
Run the master and newton Xenial jobs, or manually deploy a mult-node AIO environment.
https://rpc.jenkins.cit.rackspace.net/job/OnMetal_Multi_Node_AIO_newton-xenial
https://rpc.jenkins.cit.rackspace.net/job/OnMetal_Multi_Node_AIO_master-xenial
How can we verify your issue is addressed?
https://rpc.jenkins.cit.rackspace.net/job/OnMetal_Multi_Node_AIO_newton-xenial
https://rpc.jenkins.cit.rackspace.net/job/OnMetal_Multi_Node_AIO_master-xenial
If you are using or know of a workaround please describe the necessary steps?
What is your evaluation on the importance of this issue, and why?

This section is completed during the issue resolution process. If you are reporting an issue, please do not modify the section below.
Which branches are impacted by this issue?
Remove branches that are not impacted.
For each pull request, update the branch line to reference it.

[x] master
[x] newton-14.0
[ ] mitaka-13.1
[ ] liberty-12.2

Upgrade impact. Consider the impact on moving between major, minor and patch releases.

[ ] This change does not impact upgrades
[ ] This change addresses a potential upgrade impact
[ ] The upgrade impact is being tracked by another issue.

The issue tracking the upgrade impact is:
Testing impact

[ ] An issue/pull-request has been create or exists to track the testing gap, and is referenced below
[ ] This change includes additional testing
[ ] This change is tested by the existing setup

Additional testing links:
Release note

[ ] This change includes a release note.
[ ] This change does not require a release note.

Doc Impact
Does your change require documentation?

[ ] Yes, and I submitted an issue to Docs issues
[ ] Yes, and I contributed a documentation patch to docs-rpc
[ ] No documentation required for this change

Additional documentation links (please add links if you have ticked yes):

I have hit this as well, outside of a gate. I did not investigate, however.

@cloader89 Any update on this?

As an update --- I have been trying to create a Multinode AIO to get the error for me to look around on. I have an OnMetal Xenial server, working towards an AIO. There are many differences between Trusty and Xenial, however there was no way to get a Multinode AIO in xenial on OnMetal without Dave's guide to Upgrade. I have a Xenial onmetal box now, Iam 1/5th through the AIO build on my own AIO. I will update when I fully complete the AIO. Xenial is complicated without being able to use the image that QE uses, so I will update you guys as I proceed.

P.S., I am using a good ol' upgrade from Trusty. as per @d34dh0r53 guide:
https://gist.github.com/d34dh0r53/95e0488a5588ffc2a71f837041098ea4

Update: Multinode AIO built, and got past the spot in the error, I will run another two tests tomorrow.  @mkam could you link me an active jenkins multinode run that has the error on it?

Moving this to "swimlane-rr" since it's targeting for the next Newton patch release(if any).

Update: Talked with @mkam and we are no longer seeing this as an issue. Closing.


Download bilbili videos in 1080p resolution?

Checklist

[x] I'm asking a question
[x ] I've looked through the README and FAQ for similar questions
[x ] I've searched the bugtracker for similar questions including closed ones

Question

How can I download bilbili videos in 1080p resolution? By default the videos get downloaded either in 720p or 480p resolution. I suppose that's because to access full HD resolution viewing, Bilibili requires login. What's the workaround?
I also tried using the cookies method by logging into bilbili and exporting my cookies.txt file. Then I used this command. But still the resolution of video was low.
youtube-dl https://www.bilibili.com/video/BV1W8411b7K5/?=8&share_source=copy_web&vd_source=14d801274def2dd112297a8c82660e8c res:1080--cookies cookies.txt
I get the message '[BiliBili] Format(s) 1080P 高清, 720P 高清 are missing; you have to login or become premium member to download them'

What yt-dl option is res:1080?
Where are the quotes around the URL?
Otherwise, duplicate of #31345.

The option is to download in 1080 format.
I've never used quotes around the URL before but I was downloading Youtube and vimeo videos for many years.

I now used quotes and removed the res:1080 line. Video got downloaded in 480p resolution.

Exactly. The issue of which this is a duplicate is about getting more formats. After the code changes to do that have been mad, the expectation is that 1080p will be available (when the page supports it) when you pass cookies from a logged-in browser session.
yt-dl options are all introduced by - or -- so an intended option res:1080 will actually be treated as an (unresolvable) URL. Actually, as there's no following space in your original command, both res:1080--cookies and cookies.txt will be treated as URLs.
The reason for quoting ("Windows", 'POSIX') URLs is that otherwise a special character in the URL confuse the command shell.
Considering your original command
youtube-dl https://www.bilibili.com/video/BV1W8411b7K5/?=8&share_source=copy_web&vd_source=14d801274def2dd112297a8c82660e8c res:1080--cookies cookies.txt

Windows cmd will see the &s and =s in the URL as separating a sequence of three commands:
youtube-dl https://www.bilibili.com/video/BV1W8411b7K5/?=8
share_source copy_web
vd_source 14d801274def2dd112297a8c82660e8c res:1080--cookies cookies.txt

while a POSIX shell will see the &s as job separators,
youtube-dl https://www.bilibili.com/video/BV1W8411b7K5/?=8
share_source=copy_web
vd_source=14d801274def2dd112297a8c82660e8c res:1080--cookies cookies.txt

asking to run the first two commands as background jobs (the second just setting and discarding the shell variable share_source), and in the foreground to run the res:1080--cookies command with environment variable vd_source set to 14d801274def2dd112297a8c82660e8c and argument cookies.txt.

From the evidence you provided in https://github.com/yt-dlp/yt-dlp/issues/5655, the "expectation" isn't met. I'll add a note in the original issue.

What exactly is the command that I must type to download 1080p resolution video?

There is no such command now.
Use -F/--list-formats to show the available formats and -f .../--format ... to specify one of the listed formats to download. When we've worked out how 1080p formats can be extracted (if they can), a yt-dl installation with the resulting future Bilibili extractor module will list 1080p formats if the page supports them; probably you'll need to provide cookies from a logged-in browser session as the current error message suggests.
What currently happens is that the extractor gets a data block (JSON) that contains a list of the known qualities (formats) and a separate list of the media links for various qualities. The error message reports all the known qualities that have no associated media links.
Meanwhile, just stand far enough away from the screen that you can't tell the difference between 480p and 1080p.

There is no such command now.
Use -F/--list-formats to show the available formats and -f .../--format ... to specify one of the listed formats to download. When we've worked out how 1080p formats can be extracted (if they can), a yt-dl installation with the resulting future Bilibili extractor module will list 1080p formats if the page supports them; probably you'll need to provide cookies from a logged-in browser session as the current error message suggests.
What currently happens is that the extractor gets a data block (JSON) that contains a list of the known qualities (formats) and a separate list of the media links for various qualities. The error message reports all the known qualities that have no associated media links.
Meanwhile, just stand far enough away from the screen that you can't tell the difference between 480p and 1080p.

I'm downloading software tutorials and the UI in those videos can't be seen when one stands far enough away. Had they been other kind of videos like short film, etc. , then it would have been okay.

Current final answer, pending back-port: https://github.com/ytdl-org/youtube-dl/issues/31345#issuecomment-1330543131