Memory leak when using NetTopologySuite.Geometries.Point in query parameter

Swashbuckle version: 5.6.3
Reproduction:
Add query parameter of type NetTopologySuite.Geometries.Point from NetTopologySuite nuget (recommenced by MS for handling spatial data in .net).
Expected:
Swagger should work normally, some usage of custom filters may be needed to handle this type.
Actual:
Immediate usage of few GB's of RAM. Swagger fails to start. Custom OperationFilter is not triggered at all, custom SchemaFilter filter takes 50 seconds to fire. Adding custom MapType<Point>(...) doesn't help. Api itself works correctly.

This is an interesting one and technically NOT an issue with Swashbuckle per se (although it has highlighted a minor chnge that should be made to SB ... more on that later).
The root cause is that ApiExplorer, ASP.NET Core's built-in metadata component, automatically explodes query-bound, complex type parameters into individual API parameters - one for every leaf property in the object graph. It does this because this is exactly how model binding works. For example, if you have a query-bound parameter of the following type:
public class QueryParams
{
    public string Foo { get; set; }
    public string Bar { get; set; }
}

Then your API will accept a query string parameter called "QueryParams.Foo" and another called "QueryParams.Bar".
However, the object graph for NetTopologySuite.Geometries.Point turns out to be extremely complex, to the extent that the exploded set of parameters is massive, remarkably taking up GBs of RAM in ApiExplorer metadata, and ultimately bringing the Swagger processing to a hault.
Now, I'm assuming you have zero intention of having your API accept such a massive and complex set of query parameters. So, you're likely customizing the model binding process to accept these Point parameters as some kind of formatted string. You can do this with a custom model binder or a custom TypeConverter but unfortunately ApiExplorer only honors the latter approach. That is, if you use a custom model binder, it will still try to explode the object graph rather than treating it as a single query parameter. At this point, I'll hazard a guess that you went with a custom model binder 😉
To resolve your issue, I would recommend the following. Firstly, I would switch to a TypeConverter, which is the recommended approach from the ASP.NET Core docs when you have a string-to-complex-type conversion. Once you do this, you should be able to successfully apply the MapType setting to Swashbuckle. Actually, this setting shouldn't be neccessary because Swashbuckle has enough information to know the type should be described as a string. I'll add this in the next version. For now though, here's an example setup to support query parameters of type NetTopologySuite.Geometries.Point:
// Startup.cs
public void ConfigureServices(IServiceCollection services)
{
    ...

    // Register TypeConverter for NetTopologySuite.Geometries.Point so ASP.NET Core can accept it as a query parameter
    TypeDescriptor.AddAttributes(typeof(Point), new TypeConverterAttribute(typeof(PointTypeConverter)));

    services.AddSwaggerGen(c =>
    {
        c.MapType<Point>(() => new OpenApiSchema { Type = "string" });
    });
}

// PointTypeConverter.cs
public class PointTypeConverter : TypeConverter
{
    public override bool CanConvertFrom(ITypeDescriptorContext context, Type sourceType) =>
        sourceType == typeof(string);

    public override object ConvertFrom(ITypeDescriptorContext context, CultureInfo culture, object value) =>
        // Create the Point instance here from value.ToString()
}

Ok this is a workaround, but I would have left the issue open in order to track it and as a reminder while we are awaiting the response from aspnetcore on apiexplorer issue.

Ok this is a workaround, but I would have left the issue open in order to track it and as a reminder while we are awaiting the response from aspnetcore on apiexplorer issue.

@fededim I don't think this is a "workaround" - it's the recommended approach from the ASP.NET Core docs when you have a string-to-complex-type conversion.

@fededim I don't think this is a "workaround" - it's the recommended approach from the ASP.NET Core docs when you have a string-to-complex-type conversion.

It is a workaround since the NetTopologySuite.Geometries.Point type is the one advised by the same Microsoft to use when dealing with spatial data types in EF core and it shouldn't make the ApiExplorer (another component developed by Microsoft) go into a loop. Moreover using the above method the documentation output of Swagger UI maps the properties with type NetTopologySuite.Geometries.Point into a "string", whereas in my API I expect them to be an object with just 2 properties X and Y (I am using a custom JsonConverter for serialization/deserialization), so there is a mismatch between documentation and actual api.

It is a workaround since the NetTopologySuite.Geometries.Point type is the one advised by the same Microsoft to use when dealing with spatial data types in EF core and it shouldn't make the ApiExplorer (another component developed by Microsoft) go into a loop. Moreover using the above method the documentation output of Swagger UI maps the properties with type NetTopologySuite.Geometries.Point into a "string", whereas in my API I expect them to be an object with just 2 properties X and Y (I am using a custom JsonConverter for serialization/deserialization), so there is a mismatch between documentation and actual api.

@domaindrivendev just a question: isn't there a way to tell ApiExplorer not to consider certain properties from a complex type ?

@domaindrivendev just a question: isn't there a way to tell ApiExplorer not to consider certain properties from a complex type ?

@fededim yes. The approach depends on whether the property in question is bound to a request parameter (i.e. [FromRoute], [FromQuery], [FromHeader]) or if it belongs to an object that's bound to the request body (i.e. [FromBody]). For the former case, ASP.NET Core uses it's "model binding" subsystem to deserialize the values and if you want it to ignore certain properties simply decoreate the property with the [BindNever] attribute. For the latter, ASP.NET Core will delegate to the configured serializer (e.g. STJ or Newtonsoft) and therefore to ignore properties in that context, you simply decorate with the corresponding [JsonIgnore] attribute. For all of the above, Swashbuckle will follow suit and omit the property in question from the generated docs.

This works well for situations where you want GeoJson serialization and need to support different geometry types:
        public class GeometryTypeConverter : TypeConverter
        {
            public override bool CanConvertFrom(ITypeDescriptorContext context, Type sourceType) =>
                sourceType == typeof(string);

            public override object ConvertFrom(ITypeDescriptorContext context, CultureInfo culture, object value) =>
                new GeoJsonReader().Read<Geometry>(value.ToString());
        }


Unable to come up with d.ts file for this module

I require a module that I can new() up and I dont find a way in d.ts to express this.
Usage:
var mod = require("foo");
var foo = new mod();
I would come up with something like this in my d.ts file but I dont think its possible:
declare module "foo" {
new():ISomething
}

You can do
declare module "foo" {
 var newAble:{
    new():ISomething
 }
 export = newAble;
}

Yep, export = is the correct way to model this.

Thanks. In general I think d.ts creation needs better documentation. Is there somewhere else to go than the TypeScript documentation? Maybe a place with common patterns and snippets?

@bpasero https://github.com/Microsoft/TypeScript/wiki/Writing-Definition-Files#single-complex-object-in-external-modules ?

Looks good. It seems on that page another solution is being presented where I can export the module with a name "foo" and a constructor function. https://github.com/Microsoft/TypeScript/wiki/Writing-Definition-Files#typing-4

It seems on that page another solution is being presented where I can export the module with a name "foo" and a constructor function

Are you saying : that solution (https://github.com/Microsoft/TypeScript/wiki/Writing-Definition-Files#typing-4) is different from this (https://github.com/Microsoft/TypeScript/issues/1695#issuecomment-70282520)?
If so, then I disagree. The only real difference is that I stripped it down for your case, used things inline (which is actually naughty of me as I've locked it out of type extensiblity) and wrapped it up in a declare module "foo" (which is a different concept covered here : https://github.com/Microsoft/TypeScript/wiki/Writing-Definition-Files#typing-3)

Oh I see. In my case I need to be able to declare multiple modules, all of them potentially able to "new" them up. I think your variant should work nicely in this case.


Move section on possible installation errors from Install the Beeware Tools to Bootstrap a new project

Is your feature request related to a problem? Please describe.
Currently, pythonnet is not pip-installable with Python 3.9. There is a really good explanation of what can be done instead in the section called Install the Beeware Tools. However, this section relates to the briefcase new command, which installs briefcase-specific packages, and doesn't cause problems. The error arises during the next step, when you run briefcase dev and the pythonnet package is installed.
Describe the solution you'd like
I think it would be clearer for beginners if we move the section on installation problems to the Bootstrap a new project section.

👍 Agreed - the current location isn't the best option.

👍 Agreed - the current location isn't the best option.


Body Lenght is not the good value

// I put 14072 in the Header Content-Length. I used PostMan and in the raw body i pasted a big XML
// Used notepad++ to get the Lenght gives me 14072
int bodyLength = Int32.Parse(Request.Headers["Content-Length"].ToString());
// I use this
Request.HttpContext.Request.PeekBody().Length; // returns 14059
// If i copy the content of Request.HttpContext.Request.PeekBody.ToString() in notepad++ i get 14072 too. Why PeekBody.Length() dosent give me the same?

This issue can ben because of the different formatting and/or encoding of the notepad++ ,
please give a try to the new version with you can change the default encodings

The new version works very well i got the good length now. Very appreciated.


Update Java version in workflow.yml from 11 to 17

Quarkus has now a Java 17 baseline, see https://quarkus.io/blog/quarkus-3-7-released/.

See failure in https://github.com/asdf-community/asdf-quarkus/actions/runs/7866314142/job/21460434798?pr=31.


Open the code encryption POC repo to the public

We've done a proof-of-concept encryption plugin and we need to open it to the community so that it may be reviewed, tested and feedback about missing features may be collected.

[x] Verify the plugin works with the latest Runtime and modules
[x] Make the plugin easily installable
[x] Write more detailed instructions on how to install it and how it works
[ ] Write a blog post on the matter

This relates to https://github.com/NativeScript/NativeScript/issues/252.

Anything  ew on this one? Are there any instructions on how to use this?

Is the mentioned blog  this? https://www.nativescript.org/blog/how-secure-is-nativescript


made average and sum aggregates ignore null values

Number columns where a value can be null (unknown != 0) would give an error while calculating the average/sum of the column in the footer aggregate.
This fix ignores these null values when calculating

Changed to draft after discovering a bug after filtering

Thank you for your collaboration.
Please let me know when it is ready.
Good luck

Merged, Thank you 🙏🙏 @Software365-info

You're welcome. Can you please provide a new build on pub.dev, containing our 2 pull requests ?
Many thanks.


Make noise is GPR/SGPR constraints.positive by default

This PR makes a small change at GPR/SGPR models:

Previously: noise's constraint is constraints.greater_than(jitter). We only add noise to the diagonal of k(X, X).
Now: noise's constraint is constraints.positive. We will add noise+jitter to the diagonal of k(X, X).

This change does not affect much the inference and the term is added to k(X, X) is always guaranteed to greater than jitter.

It seems that travis is flaky?


Added selected font assignment

What's new?

Added selected font assignment

Closes #11

This is awesome! Thanks!


fix(icon): change import of icons from @carbon/icons

Import from '@carbon/icons' leads to import of all icons from this package.
When I changed import to direct imports of icons, the size of my application vendor bundle decreased:
Dev: -9MB
Prod: -6KB
Changelog
Changed

Use direct import of icons in icon.module.ts

I found these modules when I analyzed my app bundles using https://statoscope.tech/ , maybe this tool can help you to optimize library more.


Fix upstart script for Centos 6 (and Amazon Linux)

1. What does this change do, exactly?
Fixes upstart script so it actually works on Centos 6 and Amazon Linux. The script had comments that Centos6 upstart doesn't support some features, but it still used them.
2. Please link to the relevant issues.
https://github.com/mholt/caddy/pull/1189
3. Which documentation changes (if any) need to be made because of this PR?
None I can think of.
4. Checklist

[x] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I am willing to help maintain this change if there are issues with it later

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks -- but what about CentOS 7 users? These are kind of important (for example, not running Caddy as root).

Centos 7 uses Systemd, so it should be covered by https://github.com/mholt/caddy/blob/master/dist/init/linux-systemd/caddy.service

Okay. We can merge this but first can you make a comment or at least a note in the readme that systemd or a newer version of upstart should be used instead, to handle permissions properly?

Sorry, I don't get it. Centos 6 uses older upstart, and there's no way to update it. Amazon Linux is based on Centos 6 and uses the same upstart. This change just updates upstart script caddy.conf.centos-6 to do work with these upstart versions, doing the same things, and handling permissions correctly.
On the other hand, Centos7 uses systemd and doesn't even have upstart.
So if one uses Centos 6, they take linux-upstart/caddy.conf.centos-6.
If they use Centos 7, then take linux-systemd/caddy.service.

Ah, I see now. That's unfortunate. I guess this is the best that can be done (?). Thanks for the fix, then!


Add support for multimedia evaluation package

Tested (with https://github.com/xen-troops/meta-xt-rcar/pull/7) on CCPF-SK (h3ulcb-4x2g-ab build machine).
Video and audio playback works fine.

Acked-by is added.
Rebased, tested.
Ready to be merged.


Array(T) implementation

Uuh, it looks like the Array(T) implementation is ready and seems to even work :)

Very cool!
Before I review this: It looks like we have a bit of a git problem here. It appears as if you branched off your previous PR branch instead of master (perhaps while I was still reviewing the other PR). I then squashed the commits (to get rid of the fix-up commits and keep the history clean), but your branch still has them and then you merged and now we have all changes duplicated, which is a bit unfortunate.
I created a fresh branch feature/arrays in this repo that I branched off master and then cherry-picked over all your new array related commits, making the history clean again. It'd be great if you could re-create the PR using this branch, this should fix the duplication!


Deleting a directory in the Project panel doesn't close already open tabs with files from that directory

Assuming a project has dir/file.hx file, and that file is open,
Delete the entire dir directory using the Project panel.
The file.hx tab remains open. If you make any changes to this file and try to save it, an exception is thrown (since the directory doesn't exist anymore).

Should it just close the file or let you re-save correctly if you wish to do so? I would like to have the second to match the file operation.

Personally I think it should close the file.
When I delete a folder, there is already a confirmation dialog asking me if I'm sure.
I said yes I'm sure, and that should be it.
There's usually a recycle bin and/or source control to fix blunders.
Again, personally, I hate to have to confirm something more than once? Microsoft software and OS are really prone to this. Really frustrating when there are confirmations like this in a row: do you want to delete this folder? the folder is not empty, are you sure? one of the files is read-only / system / whatever, are you really really sure? Etc.

Very true, i think they are also trying move away from it. Maybe we could have a setting were user can have the confirms as they are now and then default to no multiple confirms.

Or not the confirms but the closing of the file when its or the containing folder is deleted.

@gene-pavlovsky  write to me in the skype

@SlavaRa I think I don't have your Skype, but I can ask Hector.
Gotta go to bed, I will message you tomorrow :)

@gene-pavlovsky skype: slava_er.a.


Fix: Correct the console logs, Update hardhatandfractal.md

Corrected the console logs that would be shown in the terminal after running :
npx hardhat run scripts/withdraw.js --network m1
There should be 6 console logs but only 2 are displayed in the tutorial documentation

Hi @Rahat-ch
open issue


How to install the riscv-gcc tool chain

I am trying to install riscv-gcc tool chain  by following below commands
1.git clone --recursive https://github.com/riscv/riscv-gnu-toolchain
2.sudo yum install autoconf automake libmpc-devel mpfr-devel gmp-devel gawk  bison flex texinfo patchutils gcc gcc-c++ zlib-devel expat-devel
3../configure --prefix=/opt/riscv
4. make linux
While following these above steps and after running step 4.i am getting compilation error
cc1: internal compiler error: Aborted.
Can you please tell correct steps for installing riscv-gcc toolchain
Can you also please tell after above toolchain installatiom. what should be the value of env variable RISCV_GCC.
Regards
Munish

Hi Munish,
Can you report the toolchain installation issue to the riscv-gnu-toolchain repo? I don't see any obvious issue from the steps you described. Maybe you need to provide more error info for them to debug.
RISCV_GCC should be set to something like  <your_riscv_gcc_path>/bin/riscv64-unknown-linux-gnu-gcc (or bin/riscv32-unknown-linux-gnu-gcc)


Please make a Light Theme

This is such a great theme for night coding.  Any chance you can make a daytime version also with a light colored background?

Duplicate of https://github.com/sdras/night-owl-vscode-theme/issues/59 - please upvote there! 😉


Support negative scale for decimal deserialization

:+1:
Also included a integration test.

Integration test lgtm!


Avoid random lookup to determine if attribute has undefined value for…

… a document.
Instead of comparing value against undefined value, compare enum reference.
@toregge PR

Will not pursue this one until further proof is found as to its cost.


Incorrect statement of "Microsoft Intune" column in feature area "Conditional access based on device compliance"

[Enter feedback here]
The statement of "Microsoft Intune" column in feature area "Conditional access based on device compliance" is incorrect. The statement should be "Yes", since Intune has the capability to support "Conditional access based on device compliance".

Feature area
Feature highlights
Basic Mobility and Security
Microsoft Intune

Conditional access based on device compliance
Prevent noncompliant devices from accessing corporate email and data from the cloud.
Not supported on Windows 10.Limited to controlling access to Exchange Online, SharePoint Online, and Outlook.
No

Reference:
Conditional Access is an Azure Active Directory capability that is included with an Azure Active Directory Premium license. Intune enhances this capability by adding mobile device compliance and mobile app management to the solution.
https://docs.microsoft.com/en-us/mem/intune/protect/conditional-access

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

ID: dbd65bdb-26f7-32ec-0e00-0cea0f1dc304
Version Independent ID: 0ffa2937-a9ed-f59d-d1d7-33a587872003
Content: Choose between Basic Mobility and Security and Intune - Microsoft 365 admin
Content Source: microsoft-365/admin/basic-mobility-security/choose-between-basic-mobility-and-security-and-intune.md
Service: o365-administration
GitHub Login: @KwekuA
Microsoft Alias: kwekua

@henrywang08 Thank you for the feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/3475
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @henrywang08 Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Games not visable

since the last version I can not see the games I'm launching from Discord. There is only a gray side on where down in the right corner of the Bandagebeterdiscordlogo emerges. before the update I was able to play with the Betterdiscord app.

Redownload the installer and run it again (BBD) there was an issue with one of the versions


Discontinuing MultiMC on CurseForge

Fabulously Optimized plans to discontinue distributing MultiMC versions on CurseForge, starting with v6.0.0.
FAQ
Why?

The new CurseForge site layout is making it significantly harder for MultiMC and vanilla users to download there
Due to #626, the version of MultiMC is already missing Sodium as of 4.8.1, and it is expected to miss more mods in the future
These were never "server packs" but some users got confused

On the other hand, only "server packs" made them visible in the first place

Removing that version simplifies the building and approval process on CurseForge
MultiMC has now fixed the Modrinth issues with its latest 0.7 release
Vanilla users are getting a #110 soon
Pretty sure mod devs were never getting compensated when posted there
Probably easier to make #462 work too
Recently some approvals were unpredictably pushed for manual approval due to a Fabric Language Kotlin update (which is already updated on CF itself...)

When?
MultiMC users are right now no longer encouraged to download from CurseForge.
The deprecation itself will happen on Minecraft 1.21.
Edit: now vanilla instructions have been updated.
What does it mean to users?
Awesome news! No, really:

MultiMC users will be able to download with a few clicks now
Vanilla users will be able to download with a few clicks as well
Mods and FO get the download counts they deserve on Modrinth
No confusion about "server packs" because there never were any

Really, no drawbacks at all?
Okay, there are some. But very minor:

Vanilla users will no longer be able to download FO manually. In terms of FO, that is actually still a good thing, as it means there will be less users who download and install just the mods, which is a very wrong thing to do. No longer the case
It may affect FO's download count and monetization on CurseForge a bit, but also maybe not at all
Pojav support may be slightly harder to do in the future, though not necessarily (if there is some kind of installer that can run on it) Pojav now has a modpack installer

While the new installer is not here yet, I've made a new page for vanilla user downloads, which downloads the pack and its mods directly from Modrinth and combines to a simple ZIP file.
The install instructions have been updated for 4.11.0+, with legacy ones available below.
CurseForge uploads for MMC Legacy will continue at least until FO 6.0.0-alpha.1.

6.0.0-alpha.1 removed the already deprecated MMC format. If everything goes as planned it won't be present in stable

Completed with 6.1.0.


capture console.log

I want to capture the console logs & JS exceptions for each visited pages.
I tried something like this:
$host = 'http://localhost:4444/wd/hub'; // this is the default
$capabilities = DesiredCapabilities::firefox();
$capabilities->setCapability('loggingPrefs', array('browser'=>'ALL') );

then
$logs = $driver->manage()->getLog("browser");
file_put_contents("logs.txt",print_r($logs,true));

I got a log of useless logs, but can't get the real console.log.
after several hours reading in the java world, I am  wondering now if the reason was not that there's actually some missing API in this driver?

Ok, I've found a hack outside  faceook/php-webdriver

Can you share the hack? I'm running into the same issue where I need to access console.log in the browser

@boussou and @NoodlesNZ got any solution for this? I'm getting empty log even though I specifically executed console.log for the purpose of trying to fetch it.

Hi @NoodlesNZ & @deleugpn sorry I missed the notification from Github.
I moved away from PHP, from what I remember, the trick was to inject some javascript wrapper via selenium.
I'll try to find a usable code to share.

I was able to note that console.log won't be caught by WebDriver, but any actual Javascript error will. So I called an undefined function to achieve the tests I was performing.

Sorry, I never found a solution for this.


Support for CUDA 11.1

Thanks for the great work! What can I do to help get support for CUDA 11.1? Would be great help to all users of new RTC 30 series GPUs.
https://news.developer.nvidia.com/cuda-11-1-introduces-support-rtx-30-series/

See https://github.com/conda-forge/cudatoolkit-feedstock/pull/26

Thanks for the pointer! Looks like others are already working on this. Will go ahead and close.


feat: Support export buildkit llb graph by envd

Description
Support export LLB graph

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

What format do you expect?

JSON I think, just like buildctl does. https://github.com/moby/buildkit/tree/master/cmd/buildctl

Reference: https://github.com/moby/buildkit/blob/c8d04dbee24d45013b3c36e9b1173bd737ca74b2/cmd/buildctl/debug/dumpllb.go#L83


Switch to the Adoptium api to get the latest nightly builds testimage

Currently we are still using the Adoptopenjdk api to get expected jdk testimage, which should switch to the Adoptium api.
Update
https://api.adoptopenjdk.net/v3/binary/latest/${JDK_VERSION}/${release_type}/${os}/${arch}/testimage/${JDK_IMPL}/${heap_size}/adoptopenjdk
to
https://api.adoptium.net.net/v3/binary/latest/${JDK_VERSION}/${release_type}/${os}/${arch}/testimage/${JDK_IMPL}/${heap_size}/adoptium?project=jdk
https://github.com/adoptium/aqa-tests/blob/master/get.sh#L230-L231

Similar to #https://github.com/adoptium/aqa-tests/issues/2909

Hi @sophia-guo, can I try to solve this issue?

Yes, please go ahead.


No longer works

The merge between Offline CheatBreaker and CheatBreaker 2 means that this no longer works, as the downloads are mostly closed off behind the OCB launcher's proprietary API.
However, you might be able to still get it to work. If you want to continue the project, you can message me on Discord, as I know some things to get it to work again. I would personally still like to use Prism Launcher for CB instead of the launcher, so I can use custom JVM arguments and overall have more features and customisation.


Update package.json version to mirror compiler 20150505 release.

@MatrixFrog Could you review this for the npm release?

lgtm. Someone will merge this in a few days (I know, the delay is annoying. Hopefully we'll be able to speed up the process soon.)

I'll go ahead and publish it before it's merged then.

This has been superceded.


Proposal: Simple custom text selection toolbars

It should be easier to create and modify custom text selection toolbars.
Use case
Users often want to just do something simple like add an extra button to the existing text selection toolbar.  That should look something like this:
TextField(
  textSelectionToolbar: TextSelectionToolbar(
    children: <Widget>[
      ...TextSelectionToolbar.defaultChildren,
      TextSelectionToolbarTextButton(
        onPressed: () { /* Do special stuff */ },
        child: const Text('My special button'),
      ),
    ],
  ),
)

Context
Recently, https://github.com/flutter/flutter/pull/69428 made it possible to customize the Material text selection toolbar and buttons, and https://github.com/flutter/flutter/pull/73578 does the same for Cupertino.  However, the current TextSelectionControls interface still requires a bunch of extra work in order to be able to use a custom toolbar.  Rather than the simple example above, it looks more like this:
TextField(
  textSelectionControls: MyTextSelectionControls(),
)

...

class MyTextSelectionControls extends TextSelectionControls {
  @override
  Widget buildToolbar(
    // Lots of confusing parameters that I don't care about when I just want to add a button:
    BuildContext context,
    Rect globalEditableRegion,
    double textLineHeight,
    Offset selectionMidpoint,
    List<TextSelectionPoint> endpoints,
    TextSelectionDelegate delegate,
    ClipboardStatusNotifier clipboardStatus,
  ) {
    /*
      ... Lots of confusing math that's duplicating what we do internally to calculate the anchors...
   */

    return TextSelectionToolbar(
      anchorAbove: anchorAbove,
      anchorBelow: anchorBelow,
      children: <Widget>[
        ...aRecreationOfTheDefaultButtons,
        TextSelectionToolbarTextButton(
          onPressed: () { /* Do special stuff */ },
          child: const Text('My special button'),
        ),
      ],
    );
  }
}

Proposal
I propose that we make a breaking change to the existing TextSelectionControls and its Material and Cupertino counterparts.  The new interface should allow something similar to the simplicity of the first example above.  It should be built around TextSelectionToolbar and should not hardcode a preference for the existing default buttons.
I plan to update this with a full design doc if I'm able to get around to it.
Update: Design doc

I've suggested some ideas about toolbar before. https://github.com/flutter/flutter/issues/36985#issuecomment-758596125
It started when I wanted to make a ux similar to GitHub at someone, but I had no control over not calling hideToolbar().
Currently, the toolbar's show and hide are still heavily dependent on internal properties, and there is no way to know externally when to trigger, or the editing region of the selection, so I suggest exposing these externally and refactoring the toolbar on top of that.
I believe more exposure will lead to more interesting design cases.

I've suggested some ideas about toolbar before. https://github.com/flutter/flutter/issues/36985#issuecomment-758596125
It started when I wanted to make a ux similar to GitHub at someone, but I had no control over not calling hideToolbar().
Currently, the toolbar's show and hide are still heavily dependent on internal properties, and there is no way to know externally when to trigger, or the editing region of the selection, so I suggest exposing these externally and refactoring the toolbar on top of that.
I believe more exposure will lead to more interesting design cases.

I was just working on a Mac desktop text selection toolbar (https://github.com/flutter/flutter/pull/73882), and I ran into the problems you mentioned in https://github.com/flutter/flutter/issues/36985#issuecomment-758596125, like wanting to show the menu in places other than EditableText.  As you mentioned here, the logic of when the toolbar should show/hide is really difficult to control.
I'm working on moving all of this text editing stuff to use the Shortcuts/Actions/Intents system (design doc).  This will make it possible to override this behavior anywhere in the widget tree using Shortcuts and Actions widgets.  It should also make the logic for hiding and showing the toolbar much simpler.
I think TextSelectionControls also needs to be rethought to fit into this system.  Hopefully we can come up with something that works for both the mobile text selection toolbars and the idea of menus that fits the desktop right-click menu better.

I was just working on a Mac desktop text selection toolbar (https://github.com/flutter/flutter/pull/73882), and I ran into the problems you mentioned in https://github.com/flutter/flutter/issues/36985#issuecomment-758596125, like wanting to show the menu in places other than EditableText.  As you mentioned here, the logic of when the toolbar should show/hide is really difficult to control.
I'm working on moving all of this text editing stuff to use the Shortcuts/Actions/Intents system (design doc).  This will make it possible to override this behavior anywhere in the widget tree using Shortcuts and Actions widgets.  It should also make the logic for hiding and showing the toolbar much simpler.
I think TextSelectionControls also needs to be rethought to fit into this system.  Hopefully we can come up with something that works for both the mobile text selection toolbars and the idea of menus that fits the desktop right-click menu better.

Whether the text selection control can span multiple SelectableText components, similar to web selection behavior

@yixiaco You're right, we do need some kind of global selection.  Some people have been thinking about this like https://github.com/flutter/flutter/pull/81849

Is it still not finished in version 2.5?

A year has already passed. Any news?

Update: You still need to implement your own TextSelectionControls if you want to customize the toolbar, which is still annoyingly difficult.  However, refactoring is underway that should unblock this issue.
I'm currently working on getting the context menu to work outside of EditableText (https://github.com/flutter/flutter/issues/98272).  I think this will include a way to specify the context menu in different parts of the tree as a widget.  I'll update this issue when I make progress on that.

This is a must-have feature. Thanks for your hard work.

Here's a design doc that I've written to make it easy to customize these menus on desktop and mobile.
It looks like I'll deprecate TextSelectionControls.buildToolbar, along with .canCopy and .handleCopy, etc.  Is anyone using those canCopy/handleCopy methods?  Maybe I'll provide a direct replacement.  Currently in the design doc, it's slightly more complicated to remove the Copy button or change what it does (you have to find and modify a buttonData in a list).

A few examples have been added to the repo related to this, this one for example.  Further examples will be added in the flutter/samples repo.


Index existing database?

Good day,
Happy to be putting whooshee through its paces. Quick question: is there a way to re/index data that existed prior to integrating whooshee?
Thanks in advance for any guidance.

Hi, there is a PR https://github.com/bkabrda/flask-whooshee/pull/5 from @xsuchy, which implements this. It looks good, but I wanted the submitter to implement a real test, which he didn't, yet... When that is done, I'll accept the PR and release new version.
@xsuchy, still interested in working on that PR?

Wow, I totally forgot about that. I will try to find some time to finish it. However if @cliquenoir enhance it and submit as new PR I would be happy.

This issue was fixed by #5, so I'm closing it. I'll release new version shortly.

Here's the new release: https://pypi.python.org/pypi/flask-whooshee/0.0.9
Enjoy!

Thanks guys! Look forward to testing this out. Cheers.


Memcached / APCu

RainLoop version 1.11.3 on Ubuntu 16.04 with nginx, php7 and php7-fpm
Hi,
I've tried changing caching from 'files' to 'APC' and then to 'memcached', but neither APC nor memcached show any data being stored in them, zero items
With APC, php-apcu 5.1.3 was installed and enabled, monitored with apc.php file from https://raw.githubusercontent.com/krakjoe/apcu/master/apc.php
With memcached, memcached 1.4.25 and php-memcached 2.2.0 were installed and monitored with "stats" in telnet connection to memcached.
Settings:
[cache]
enable = On
index = "v36" (+1 on every cache setting change)
fast_cache_driver = "memcache"
fast_cache_index = "v36" (+1 on every cache setting change)
http = On
http_expires = 3600
[labs]
cache_system_data = On
fast_cache_memcache_host = "<IP_ADDRESS>"
fast_cache_memcache_port = 11211
Am i doing anything wrong?

I'm running Debian Jessie with nginx and php5-apcu and for me it is working fine.
[cache]
enable = On
index = "RLv1"
fast_cache_driver = "APC"
fast_cache_index = "v1"
http = On
http_expires = 3600
server_uids = On

apc.php shows a lot of hits and in the user cache entries I see stuff related to RainLoop. In my nginx I see no specific configuration for apc. This is the php related config I have in nginx:
upstream php-handler {
        server <IP_ADDRESS>:9000;
        #server unix:/var/run/php5-fpm.sock;
}

server {
[...snip...]
        location ~ ^/rainloop/.*\.php$ {
                include snippets/fastcgi-php.conf;
                fastcgi_keep_conn on;
                fastcgi_pass php-handler;
        }
[...snip...]
}

snippets/fastcgi-php.conf:
fastcgi_split_path_info ^(.+\.php)(/.+)$;
try_files $fastcgi_script_name =404;
set $path_info $fastcgi_path_info;
fastcgi_param PATH_INFO $path_info;
fastcgi_index index.php;
include fastcgi.conf;

fastcgi.conf:
fastcgi_param  SCRIPT_FILENAME    $document_root$fastcgi_script_name;
fastcgi_param  QUERY_STRING       $query_string;
fastcgi_param  REQUEST_METHOD     $request_method;
fastcgi_param  CONTENT_TYPE       $content_type;
fastcgi_param  CONTENT_LENGTH     $content_length;

fastcgi_param  SCRIPT_NAME        $fastcgi_script_name;
fastcgi_param  REQUEST_URI        $request_uri;
fastcgi_param  DOCUMENT_URI       $document_uri;
fastcgi_param  DOCUMENT_ROOT      $document_root;
fastcgi_param  SERVER_PROTOCOL    $server_protocol;
fastcgi_param  HTTPS              $https if_not_empty;

fastcgi_param  GATEWAY_INTERFACE  CGI/1.1;
fastcgi_param  SERVER_SOFTWARE    nginx/$nginx_version;

fastcgi_param  REMOTE_ADDR        $remote_addr;
fastcgi_param  REMOTE_PORT        $remote_port;
fastcgi_param  SERVER_ADDR        $server_addr;
fastcgi_param  SERVER_PORT        $server_port;
fastcgi_param  SERVER_NAME        $server_name;

# PHP only, required if PHP was built with --enable-force-cgi-redirect
fastcgi_param  REDIRECT_STATUS    200;

The difference seems to be that you are using PHP5 whereas i am using PHP7.

Yes. I am planning to update to the latest Debian soon. If that's PHP7 then I'll know if this works for me or not. Perhaps something changes for PHP7 that breaks this, I don't know.

Sorry about the late reply.
Yes, APC is working after installing php-apcu-bc as described here: https://gist.github.com/philbot9/1e895303a37a1c18717e90eb7067df73
Thank you very much.


Ribbon click events stop working when screen set to 125%

As we want the top ribbon hidden after selecting something we add "Ribbon1.Minimized = true".   This way we maximize the amount of space on the screen.  What we have found is that when the screen resolution is set to 125% the click events for the ribbon no longer works to open any other screen????
Anyone know how to fix this?
Thanks
brendan
private void ribbonButton2_Click(object sender, EventArgs e)
{
foreach (Form f in MdiChildren)
{
if (f.GetType() == typeof(MDIChild1))
{
f.Activate();
return;
}
}
Form form = new MDIChild1
{
MdiParent = this
};
form.Show();
ribbon1.Minimized = true;
}

This problem has already fixed in new version.
You may try the Nuget version:
Guide: How to Add via Nuget:
https://github.com/RibbonWinForms/RibbonWinForms/wiki

Hi Adrian,
Thank you for letting me know it was updated.   I have added myself to the
GitHub notifications for new releases and I really appreciate the work that
you and the other contributors are doing!
Cheers,
Brendan Keyhoe
Kebrite Solutions Inc.
office:<PHONE_NUMBER> x 200
web: www.kebrite.com
links: support portal https://kebriteportal.axosoft.com/login &  file
upload https://spaces.hightail.com/uplink/kebrite
[image: e943ecc8-5be2-4768-a17e-1b829b908afe]
https://www.facebook.com/Kebrite[image:
541aa3ae-22f4-4ed9-bfd4-35cb8d8633b6]
https://twitter.com/KebriteSolution[image:
7b24d431-b33f-4266-8e00-6ec431907477]
https://www.linkedin.com/company/kebrite-solutions-inc-?
[image: 10Years]
CONFIDENTIALITY DISCLAIMER: The information contained in this transmission
may contain privileged and confidential information of Kebrite Solutions
Inc.   It is intended for review only by the person(s) named above.
Dissemination, distribution or duplication of this communication is
strictly prohibited by all recipients unless expressly authorized
otherwise.  If you are not the intended recipient, please contact the
sender by reply email and destroy all copies of the original message.
Kebrite® is a registered trademark of Kebrite Solutions Inc.
From: Adrian Voo<EMAIL_ADDRESS>Sent: Monday, August 19, 2019 8:24 AM
To: RibbonWinForms/RibbonWinForms<EMAIL_ADDRESS>Cc: Kebrite Solutions Inc<EMAIL_ADDRESS>Author <
<EMAIL_ADDRESS>Subject: Re: [RibbonWinForms/RibbonWinForms] Ribbon click events stop
working when screen set to 125% (#23)
This problem has already fixed in new version.
You may try the Nuget version:
Guide: How to Add via Nuget:
https://github.com/RibbonWinForms/RibbonWinForms/wiki
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/RibbonWinForms/RibbonWinForms/issues/23?email_source=notifications&email_token=ADLDJBYBNWUKQIGPNYADWGLQFK3KLA5CNFSM4GBQZAE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4TKOQI#issuecomment-522626881,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADLDJBZEND6QNOZZSUS3G53QFK3KLANCNFSM4GBQZAEQ
.


Prepare initial release

[x] Add a Changelog file
[x] Add a CONTRIBUTING.md file.
[x] Add ./tools/releasing.md with steps.
[x] Do an manual release of version 0.9.0
[x] Decide if releases need to be automated.

At this point, I'm not seeing much benefit of developing an automated release process. I would like to spend my time on some of the tbp issues.


Unable to Create dist with @vladmandic/face-api

Issue Description
Tried npm run build after having @vladmandic/face-api as dependency. Also tried using yarn to build. In both cases, building failed.
/  Building for production...
ERROR  Failed to compile with 1 error                      11:25:40 am
error
No bundles were parsed. Analyzer will show only original module sizes from stats
file.
Webpack Bundle Analyzer saved report to C:\Users\VedantM\recruitlive\recruitment-applicant-client\dist\report.html
ERROR  Build failed with errors.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: vue-cli-service build
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\VedantM\AppData\Roaming\npm-cache_logs\2022-08-25T05_55_41_199Z-debug.log
Steps to Reproduce
Expected Behavior
**Environment

Module version?
Built-in demo or custom code? Built-in demo
Type of module used (e.g. js, esm, esm-nobundle)? vue
Browser or NodeJS and version (e.g. NodeJS 14.15 or Chrome 89)?
npm version 6.14.15
node version v14.18.1

"vue-cli-plugin-webpack-bundle-analyzer": "^1.4.0"

OS and Hardware platform (e.g. Windows 10, Ubuntu Linux on x64, Android 10)?
Windows 10
Packager (if any) (e.g, webpack, rollup, parcel, esbuild, etc.)? webpack. Same issue when using vite

Additional

For installation or startup issues include your package.json
For usage issues, it is recommended to post your code as gist

I'll take a look next week as I'm currently traveling.
Most commonly this is related to webpack getting confused should it use node or browser version - try importing exact one instead (e.g. @vladmandic/face-api/dist/face-api.esm.js)

Happy journey and travel safe🏁

i'm taking a look and yes, it does sound like vue-cli-service which provides dev server and hot-module-reload is creating ugly internal webpack configuration so it totally gets it wrong if it should use nodejs or browser module.
now, i have never used vue but its a very similar problem to when using nextjs and solution there is to use dynamic imports.
take a look at https://github.com/vladmandic/human-next on how the basic project using next would look like
and especially dynamic import https://github.com/vladmandic/human-next/blob/6f6b9ee8f23f02a5753b20e7d8249c0d75334495/components/RunHuman.tsx#L30
this is pretty much all i can say as i'm really not a vue user.

any updates?

For the time being I'm using @mediapipe/face_recognition but I want to thank you for creating faceAPI and human and open-sourcing them. I've a lot to learn before getting to the bottom of solving this.


Removed second occurence of Juniper Networks

Updated the count to reflect the correct number of companies listed
Congratulations on making a pull request!
Please make sure you check all the items.

[x] The numbering of the company that you added starts with 1.
[x] You have increased the Total Companies Added count.

[x] #120


fix: avoid PodMonitor reconcile on no Prometheus

On PR https://github.com/cloudnative-pg/cloudnative-pg/pull/1213/
I over-fixed the logic, and can allow to proceed with the reconciliation
of PodMonitor, even if we know the Prometheus CRD are not installed
Signed-off-by: Jaime Silvela<EMAIL_ADDRESS>

/ok-to-merge


The focus style for the element template chooser list view has a too low color contrast

Describe the Bug
If you take a look at the screenshot on the projects description page (same is reproducible when actually using it)
https://github.com/bpmn-io/element-template-chooser/blob/master/resources/screenshot.png you notice that the grey focus style for the template selected has a background color of #EDEEED while the background has #FEFFFE which leads to a color contrast ratio of 1.2:1 but the goal should be >= 3:1(WCAG21 SC 1.4.11). That way the focus isn't visible or makes it had for people with lower vision  to determine which element among multiple elements has the keyboard focus (SC 2.4.7)
Environment

Library version: "@bpmn-io/element-template-chooser": "0.0.1"

cc @andreasgeier

@currandwyer

This is a tricky issue, because if the hover state background is too dark, the text above also becomes less legible. The current state of the design includes an additional icon. This icon links to external documentation, while helping to mark the hover state for a11y purposes.

indeed a tricky problem. i've used the element template chooser only in the context of ECA on Drupal. there we dont have for example the link to external documentation.
but taking a look at your screenshot it has a bit of modified styling compared to the screenshot from the description page i've linked in the initial page as well as the out of the box version of bpmn.io i've actively tested on. there the title and the description are both blackish in your linked exampled here you have a greyish description. that fails the SC 1.4.3. regular sized text has to have a ratio of 4,5:1 while large sized text aka 24px or bold with 18,5px has to have a ratio of 3:1.   so you have to keep SC 1.4.3 in mind as well unfortunately :/
and in regards of finding the appropriate focus color those two constraints have to be met for 1.4.11:

The area has a contrast ratio of at least 3:1 between the colors when the component is focused and it is not focused.
Where the area is adjacent to the component, it has a contrast ratio of at least 3:1 against the component or a thickness of at least 2 CSS pixels.
(means the background color for the focus has to have a color contrast of 3:1 to the one the regular background in the template chooser has. and the second constraint is automatically met in that case)

but one question about your suggested color. is that in the hsl format? g-225-10-95
but i suppose when an element in the list gets into focus its background-color, color of its title and description as well as the color of the external documentation icon has to be changed to get the desired results.

@rpkoller For a little bit of background on different screen showcased (https://github.com/bpmn-io/element-template-chooser/issues/7#issuecomment-1075042128): We'll ship some updates to our element template foundations as well as this library in the near future.
We'll end up with something like https://github.com/bpmn-io/element-template-chooser/issues/7#issuecomment-1075042128; you'll be able to

Provide (optional) icons with your element templates
Provide (optional) link to a documentation URL with your templates

Your input on A11y topics is really apprechiated, here and in other issues you've raised. We'll do our best to incorporate these hints as we move forward :).

Moving to backlog for the time being. We'll revise a11y feedback we get in the near future.

@nikku ah cool thanks for the headsup! that means the screenshot provided refers to a future state of the layout of the element chooser - its taken from the dev branch?
out of curiosity is the same icon used and applied for each element or are there different images for every element or a group of elements? in case it is the same icon for each element then it might be a good idea to mark those icons up as decorative with alt=""so they get ignored by assistive technologies. in case icons are for a group of elements each then that directly relates to https://github.com/bpmn-io/element-template-chooser/issues/5 - meaning if there is some sort of grouping and segmentation implemented based on issue 5 then the icons could be marked up as decorative as well imho otherwise it might be discussed if some sort of alt attribute value is needed. but that brings up again teh groundhog day problem if you have several elements with the same icon in a row https://github.com/bpmn-io/element-template-chooser/issues/5#issuecomment-1076292060 . and in case each element has its own icon i would tend to label it as decorative again. the icons are way too small to convey an individual meaning which isnt already communicated in the element title and or the description.
and about the optional link that is an excellent idea and improvement in general. unable to check right now in an installation to label the optional link with an aria-label that the user knows the purpose of the link and where it is heading. because just having and icon without and text is the similar case like the read more... text.

We're updating to modern popup menu foundations (https://github.com/bpmn-io/element-template-chooser/pull/14). Any a11y improvements now contribute directly to the core


Balancing API Backends

I don't undestand how I must configure two balanced Backends with the admin console. I can add multiple servers in Backend section, but in the next section, Host, I only can define one Backend Host, then all the request are consumed by this backed and the load balancing doesn't work.

For example:
I have two backend servers with the same API:
http://www.myapiserver1.com
http://www.myapiserver2.com
And I have de API-Umbrella in a server: www.myapimanager.com
In the API Backend configuration:
Servers:
http://www.myapiserver1.com
http://www.myapiserver2.com
Host:
Frontend Host: www.myapimanager.com
But, which is the Backend Host?

Very sorry for the delay in responding!
So the "Backend Host" field only dictates how the Host HTTP header is rewritten on the requests that your backend servers receive. It's a single value to standardize the requests and which Host the different backend servers must be listening to. If your backend servers are listening to all hosts, then the backend host field doesn't actually matter. However, if your backend servers are using name-based virtual hosts to respond differently to different domain names, then this "Backend Host" field must match whatever your backend servers are listening to.
For example, in this example case, you could also define the "Backend Host" to be "www.myapimanager.com", but then both your myapiserver1.com and myapiserver2.com servers must be setup to listen to for either all virtual host names, or "www.myapimanager.com" explicitly.
Does that make sense?

This is rare!
Then how is teh natural way to define API balanced?

In a typical setup, if your frontend host is www.myapimanager.com and the backend servers are http://www.myapiserver1.com and http://www.myapiserver2.com, then you'd typically also define your backend host to also be www.myapimanager.com. Then you'd just need to make sure that your actual backend servers are listening for that domain name. If they're listening to all hostnames, then you may not need to do anything else, but if they're using name-based virtual hosting at the Apache or nginx level, then you'd have to configure that there. Let me know if you still have any other questions or issues with this.

Thank you!
It's fine.


Threaded tests for expressions and update version numbers

Summary
This PR adds threaded expression tests and updates version number after the latest release.
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Rok Češnovar
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

Ready for review.
cc: @bbbales2


Wrong ParameterValidation(ERROR) when using VK_KHR_get_physical_device_properties2

If one uses VK_KHR_get_physical_device_properties2 and specifies one's features using VkPhysicalDeviceFeatures2KHR instead of VkPhysicalDeviceFeatures and sets 'features. samplerAnisotropy' to VK_TRUE, a 'ParameterValidation(ERROR)' is issued:
ParameterValidation(ERROR): object: 0x0 type: 0 location: 1497 msgCode: 8: vkCreateSampler(): The samplerAnisotropy feature was not enabled at device-creation time, so the maxAnisotropy member of the VkSamplerCreateInfo structure must be 1.0 but is 0.000000.
I suspect this might be true for any feature, not specifically 'samplerAnisotropy', but I might be mistaken as I did not test other cases.
I'm using VkPhysicalDeviceFeatures2KHR by setting VkDeviceCreateInfo 'pEnabledFeatures' to nullptr and 'pNext' to an instance of VkPhysicalDeviceFeatures2KHR, as specified in "4.2.1. Device Creation", "If the pNext chain includes a VkPhysicalDeviceFeatures2KHR structure, then pEnabledFeatures must be NULL".
Best regards,
Daniel

Merged in https://github.com/KhronosGroup/Vulkan-LoaderAndValidationLayers/pull/2160


Quality of life improvement using Makefile

Overview
This PR contains the following changes

Added some new make commands to run various scripts for all projects

make install = run npm install in api, app, and database
make lint = run npm lint in api and app
make lint-fix = run npm run lint:fix in api and app
make test = run npm test in api and app

This PR contains the following types of changes

[x] New feature (change which adds functionality)
[ ] Bug fix (change which fixes an issue)
[ ] Misc cleanup / Refactoring / Documentation
[ ] Version change

Checklist
A list of items that are good to consider when making any changes.
Note: this list is not exhaustive, and not all items are always applicable.
General

[ ] I have performed a self-review of my own code
[ ] My code follows the style guidelines of this project
[ ] New and existing unit tests pass locally with my changes

Code

[ ] New files/classes/functions have appropriately descriptive names and comment blocks to describe their use/behaviour
[ ] I have avoided duplicating code when possible, moving re-usable pieces into functions
[ ] I have avoided hard-coding values where possible and moved any re-usable constants to a constants file
[ ] My code is as flat as possible (avoids deeply nested if/else blocks, promise chains, etc)
[ ] My code changes account for null/undefined values and handle errors appropriately
[ ] My code uses types/interfaces to help describe values/parameters/etc and inform future developers

Documentation

[ ] I have commented my code sufficiently, such that an unfamiliar developer could understand my code
[ ] I have added/updated README's and related documentation

Tests

[ ] I have added/updated unit tests for any new code I've added/updated
[ ] I have added/updated Postman tests to account for any new/updated code

How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Screenshots
Please add any relevant UI screenshots, if applicable.

Note: I specifically cancelled the PR deployment as it would be a waste of resources (since none of the code actually changed)


Support multichannel audio output

Multichannel audio is currently being downsampled to stereo. I think it's because the channel layout is not being specified for the anullsrc filter.
log

Thanks for the report @asc3ns10n, will try to fix it for the version.

Thanks for the report @asc3ns10n, will try to fix it for the version.

Support added in 1.0.0


BIM-->PBIX Memory Failure

I'm following the TE2 documentation workaround to create a PBIX file from my BIM file now that I have enabled the xmla endpoint write capabilities. I'm following the steps and continuing to get a memory error when I open the PBIP file:

Furthermore, when I then save as a PBIX and refresh even one table I'm getting a memory error msg.
I'll note that I'm able to see the data tables when I select edit query from the PBIX file to display Power Query, but not from the actual data table view in PowerBI. Thanks for any advice or recommendation to ultimately be able to create a PBIX file since I'm unable to DL from PowerBI Online.

Which workaround did you use?

I don't believe it's possible to replace the .bim file within a .pbit (Power BI Template) file.
You should use PBIP (Power BI Project) folder format instead, as described here: https://docs.tabulareditor.com/te3/powerbi-xmla-pbix-workaround.html
More info about how to enable PBIP / Developer Mode: https://learn.microsoft.com/en-us/power-bi/developer/projects/projects-overview

I’m sorry I misspoke, that’s what I did with a pbip formatted file using that exact tutorial. Are there any other workarounds? Thanks again for your help.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Daniel Otykier @.>
Sent: Tuesday, July 30, 2024 7:10:40 AM
To: TabularEditor/TabularEditor @.>
Cc: Davis Stanley @.>; Author @.>
Subject: Re: [TabularEditor/TabularEditor] BIM-->PBIX Memory Failure (Issue #1210)
I don't believe it's possible to replace the .bim file within a .pbit (Power BI Template) file.
You should use PBIP (Power BI Project) folder format instead, as described here: https://docs.tabulareditor.com/te3/powerbi-xmla-pbix-workaround.html
More info about how to enable PBIP / Developer Mode: https://learn.microsoft.com/en-us/power-bi/developer/projects/projects-overview
—
Reply to this email directly, view it on GitHubhttps://github.com/TabularEditor/TabularEditor/issues/1210#issuecomment-2258198173, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BJV3NQAEHNMHJBAKPSZQGMLZO57EBAVCNFSM6AAAAABKNV4MN6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJYGE4TQMJXGM.
You are receiving this because you authored the thread.Message ID: @.***>

I see.
Once you replace the model.bim file, but before you open the PBIP folder in Power BI Desktop, could you try to delete the following files from the PBIP folder structure:
.pbi\localSettings.json
.pbi\editorSettings.json
.pbi\cache.abf
.pbi\unapplicedChanges.json
More information about these files here: https://learn.microsoft.com/en-us/power-bi/developer/projects/projects-dataset

Unfortunately, I’m continuing to get the below error when I load the pbip file. Otherwise, if I edit directly from tabular editor and update via the powerbi online interface everything is perfect…I just can’t get back to having a pbix copy, which I understand was a risk. Thanks for continuing to help with this, very much appreciated. This tool is fantastic.
@.***
From: Daniel Otykier @.>
Sent: Tuesday, July 30, 2024 8:22 AM
To: TabularEditor/TabularEditor @.>
Cc: Davis Stanley @.>; Author @.>
Subject: Re: [TabularEditor/TabularEditor] BIM-->PBIX Memory Failure (Issue #1210)
I see.
Once you replace the model.bim file, but before you open the PBIP folder in Power BI Desktop, could you try to delete the following files from the PBIP folder structure:
.pbi\localSettings.json
.pbi\editorSettings.json
.pbi\cache.abf
.pbi\unapplicedChanges.json
More information about these files here: https://learn.microsoft.com/en-us/power-bi/developer/projects/projects-dataset
—
Reply to this email directly, view it on GitHubhttps://github.com/TabularEditor/TabularEditor/issues/1210#issuecomment-2258339177, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BJV3NQGVUX3JFUOKHFX3GMTZO6HPHAVCNFSM6AAAAABKNV4MN6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJYGMZTSMJXG4.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Hmm, that's unfortunate. Even though Developer Mode (PBIP) is still in preview, making changes to the Model.bim file or replacing the Model.bim file as you did, should be fully supported.
@ruiromano, have you seen this issue before? Got any advice?

Yes, replacing the model.bim is supported. If the external changes applied respect the modelling authoring list: https://learn.microsoft.com/en-gb/power-bi/developer/projects/projects-overview#model-authoring
@dshsp you get the error when you open the PBIP? When you refresh? Do you have any report visuals? Can you try to open the PBIP with an empty report (just delete the report.json file)?
I also suggest you monitor PBI Desktop memory while opening using task manager.

Thanks for the reply. I get the error when I open the PBIP file after replacing the .bim file within the semantic structure. After the error message the file continues to open and it looks normal, with all of the data tables, measures, etc visible within the Data pane on the right, however, there is no data that has been loaded behind it (I can even click on an individual measure and see the DAX code). I can even see the relationships in the model view. It’s odd because when I click on an individual table from the Data pane and click edit query to view within the powerquery window, I can see all of the data for every table just fine, even after hitting refresh preview for all. However, after clicking close and apply the refresh in the file throws a memory error once again.
The report is blank right now, no visuals. I tried the deletion suggestion below for the report.json file and unfortunately didn’t even load the file as it said the report.json file was missing. Thank you for any follow ups….
From: Rui Romano @.>
Sent: Thursday, August 1, 2024 11:21 AM
To: TabularEditor/TabularEditor @.>
Cc: Davis Stanley @.>; Mention @.>
Subject: Re: [TabularEditor/TabularEditor] BIM-->PBIX Memory Failure (Issue #1210)
Yes, replacing the model.bim is supported. If the external changes applied respect the modelling authoring list: https://learn.microsoft.com/en-gb/power-bi/developer/projects/projects-overview#model-authoring
@dshsphttps://github.com/dshsp you get the error when you open the PBIP? When you refresh? Do you have any report visuals? Can you try to open the PBIP with an empty report (just delete the report.json file)?
I also suggest you monitor PBI Desktop memory while opening using task manager.
—
Reply to this email directly, view it on GitHubhttps://github.com/TabularEditor/TabularEditor/issues/1210#issuecomment-2263464555, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BJV3NQEQEYRJ5KRDUSQT3T3ZPJN6HAVCNFSM6AAAAABKNV4MN6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRTGQ3DINJVGU.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


Can not choose second gpu using wgpu

Describe the bug
When using the wgpu backend on windows (not yet tested on linux) I cannot set the wgpu backend
To Reproduce
Steps to reproduce the behaviour:

Download and setup whisper-burn
verify working on primary gpu
set WGPU_ADAPTER_NAME = 'intel' to secondary gpu
run and still uses primary
change let device = WgpuDevice::BestAvailable; to let device = WgpuDevice::DiscreteGpu(1); (also try DiscreteGpu(0) for completeness)
try again and still uses primary gpu

Expected behavior
Use the secondary gpu
Screenshots
Not applicable
Desktop (please complete the following information):

OS: Windows 11
Browser: Not Applicable
Version: burn git and burn-wgpu git

Additional context
I was able to successfully override by setting type Backend = WgpuBackend<burn_wgpu::Vulkan, f32, i32>; and using VK_ICD_FILENAME to forcibly load only the intel gpu. this is not a viable workaround however since it will not work with homogeneous gpu vendors. This may however work with setting primary gpu in windows settings, it's something I forgot to test.
original issue reported here: https://github.com/Gadersd/whisper-burn/issues/7

The intel GPU is probably an integrated gpu, can you try WgpuDevice::IntegratedGpu(0) instead?

That didn't work either, Though I am running an Arc A380, I did play with the windows could be reporting it as integrated (and I suppose it still  could be) It still runs on the primary AMD RX580 gpu instead of the Intel A380.


Init owner file in mappers-go

/cc @kubeedge/maintainers
/assign @chendave @kevin-wangzefeng @luogangyi @sailorvii

/lgtm
/approve


Added named formula class

This PR creates a class for Named Formulas. It also adds a constructor for Formula that takes an already parsed parse tree. This is so that when named formulas are parsed, their right-hand sides can be Formula objects without re-parsing.

Please add some tests parsing and constructing NamedFormulas

Please add some tests parsing and constructing NamedFormulas

Added


[Feature Request] Add Date to fileData

It's hard on Android to get the File DateTime.
I tried it with the string i get from google drive for example:
content://com.google.android.apps.docs.storage/document/acc%3D2%3Bdoc%3Dencoded%3DrighArrVG%2FbmPu3iP%2B8eAMUtEMOy%2BWopwMSW4Cu9GaGkVt1u6NpYWJXo7fNxKF%2BZNQ%3D%3D
I didn't find a way to get the correct Date from this FilePath and didn't find a way to do so.
It would be great if you could add the DateTime as seperate property of the FileData.
Kind Regards,
bluekuen

Do you have code to do this on Android? We could surely include this code in the FilePicker if you provide it. Thanks!

No sadly not, i was looking for a way to do it and opened another issue relevant to this but didn't get an useful answer. That's why i thought you maybe are more into it.

Closing this since this project will retire, please check out Xamarin.Essentials which now has a FilePicker based on this one.


Testing nixpkgs#350287

Testing https://github.com/NixOS/nixpkgs/pull/350287

@Zimmi48 @proux01 where you aware of this smtcoq failure?

No but it's not "uncommon" that the dependencies of smtcoq like cvc5 or Z3 break.
Maybe @vbgl heard of it.

No, I hadn’t head of smtcoq breaking. See: https://github.com/NixOS/nixpkgs/pull/348755

cvc5 & smtcoq have just been fixed in nixpkgs (and in coq-nix-toolbox).

nixpkgs PR merged, let's close


Not able to see the login page after running the command docker compose up -d successfully

Deployment Method

[ ] Installer
[ ] Docker Development
[x] Docker Production
[ ] Bare metal Development
[ ] Bare metal Production
[ ] Digital Ocean image
[ ] Other (please provide a link Steps to Reproduce

Describe the problem*

Steps to Reproduce
just follow the docker installation instructions.
Relevant log output

Site URL - update this to match your host
INVENTREE_SITE_URL="http://inventree.localhost"
COMPOSE_PROJECT_NAME=inventree

@nitume00 this is because you have an nginx service capturing all traffic to "localhost". The InvenTree docker setup uses Caddy as the proxy server, not nginx.
You should halt the nginx process running on your machine, and then the InvenTree server should be visible.

@SchrodingersGat  is Caddy  is the only option as proxy server or we can go with nginx also

@nitume00 you can change to nginx if you want - however our documentation and setup guides are based around Caddy.
Have a look in the docker-compose.yml file - you can see that there is a placeholder for a nginx proxy. You will need you substitute our example nginx configuration file too.
Please note though: If you are seeing a nginx page, that is not being served by our docker container. That is running somewhere else on your machine

@nitume00 I'm going to close this out, the issue here lies with your local machine having a separate nginx service running.


Port of PR #6 to branch 0.8.0

Port of PR #6 to branch 0.8.0
by the way, "upstream" pocketbase 0.8.0 got released a couple of days ago...

Thanks! By the way, I'll try to merge the 0.8.0-rc2 branch to master this weekend.


Added code for symbol pattern 28.

Code will print symbol pattern 28.
Related Issue - #714
Sample Outputs -

@aryashah2k Please review this PR.


Why Float is mapped to an Integer in the example given in readme.md

Could you please explain why weight: Float? is mapped to Integer in the example given in the readme.md. The generated schema will invalidate the payload when weight is passed as a float value.

 The example doesn't cover a floating number for the property weight. A fix is on it's way!


Review our use of impl Stream in e.g. Match queries

Currently our Match queries return impl Stream<Item = ConceptMap>.
This becomes a particular problem in the Concept API. We want to define methods such as get_instances in a trait, but trait methods cannot return impl Stream.
It might be more useful to users if they knew the exact type of Stream. Trouble is, that type is a super-complex type.
We should investigate the performance impact of using BoxStream. If not, we may need to not define these methods in traits. It may be worth consulting the Rust API guidelines.

This is expected and we no longer use this API


"pip install cookietemple" fails to complete successfully - conflicting dependencies

Associated Template/Command/Core

Just tried to install cookietemplate using the pip install as docuemnted in README file. Command fails.

To Reproduce
Steps to reproduce the behavior:

run the pip install command on cookietemple: pip install cookietemple

cookietemple remains uninstalled.
Expected behavior
Cookie to be installed succesfully, no errors
The error includes this:
To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict

but as a beginner to Python trying to come up to speed, I don't know how to apply this fix. I just expect cookietemple to install sucessfully.

The conflict is caused by:
    cookietemple 1.3.3 depends on questionary==1.9.0
    cookietemple 1.3.2 depends on questionary==1.9.0
    cookietemple 1.3.1 depends on questionary==1.9.0
    cookietemple 1.3.0 depends on questionary==1.9.0
    cookietemple 1.2.4 depends on questionary==1.8.1
    cookietemple 1.2.3 depends on questionary==1.8.1
    cookietemple 1.2.2 depends on questionary==1.8.1
    cookietemple 1.2.1 depends on questionary==1.8.1
    cookietemple 1.2.0 depends on questionary==1.8.1
    cookietemple 1.0.1 depends on questionary==1.7.0
    cookietemple 1.0.0 depends on questionary==1.7.0

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict

ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts
 ✘ plastikfan@Janus  ~/dev/tutorials/py/sandpit λ

Dear @plastikfan
I assume that you already have questionary or another tool that requires questionary to be installed in your global environment. There is nothing that we can do to resolve this conflict.
You could maybe try installing cookietemple with pipx?
pipx install cookietemple

Maybe this works? If not you need a clean environment for cookietemple, sorry.

Hi @Zethson, I just tried 'pipx install cookietemple' that also fails:
 plastikfan@Janus  ~/dev/tutorials/py/sandpit λ  pipx install cookietemple
Fatal error from pip prevented installation. Full pip output in file:
    /home/plastikfan/.local/pipx/logs/cmd_2022-02-28_13.39.36_pip_errors.log

pip seemed to fail to build package:
    cookietemple

Some possibly relevant errors from pip install:
    ERROR: Cannot install cookietemple==1.0.0, cookietemple==1.0.1, cookietemple==1.2.0, cookietemple==1.2.1, cookietemple==1.2.2, cookietemple==1.2.3, cookietemple==1.2.4, cookietemple==1.3.0, cookietemple==1.3.1, cookietemple==1.3.2 and cookietemple==1.3.3 because these package versions have conflicting dependencies.
    ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts

Error installing cookietemple.

I am just trying to get going with python and facing these types of issues is crippling. PLease give us a hint of what to do. What do you mean by a clean environment?

PS: what is questionary?

PS: what is questionary?

https://github.com/tmbo/questionary
Let's do this step by step.

Please install https://docs.conda.io/projects/conda/en/latest/user-guide/install/index.html
Create a new conda environment like conda create -n cookietemple python=3.8
Activate the conda environment -> conda activate cookietemple
pip install cookietemple

Does the virtual environment have to be done with conda or can it be done with poetry also?

Does the virtual environment have to be done with conda or can it be done with poetry also?

As long as it's clean you can also create it with poetry.

But something looks really weird in your error outputs because it tries to install several versions? I am confused. But let's try this first.

Ok thanks @Zethson, give us a sec. I will proceed with conda, then do it again afetrwards with poetry as I spent a few days learning poetry and I dont want to throw that away for conda which I know noth about.

Hi @Zethson, any chance you could tell me how to do this with poetry. Installing conda/anaconda seems to be overly complicated and unwieldly

Please read https://python-poetry.org/docs/managing-environments/
I am busy at the moment and cannot be more specific, sorry.

Ok thanks. I appreciate you trying to help, but I was not expecting such a simple and fundamental command pip install cooiketemple to fail. Cheers anyway

Again, that's not in our hands. One should NOT install packages globally.

Ah ok, but we got a chicken and egg situation going on here. If you don;t install globally, then how do you install cookietemple if you havent already got a repo to start off with. I hope you understand wheremy confusing is coming from. I thought the whole point of cookietemple was to bootstrap a new repo, but you cant install it without an existing repo. @!?

I don't think that we have a chicken and egg situation here.
You don't need a repository, you need a virtual environment. Those are two fundamentally different things.

Ok, but I thought there were the same thing

What issue did you run in for Python 3.10?

conflicting dependencies; same issue as above

In a new virtual environment?

... although when I installed conda, then created a new environment, the installation process completed more steps and looked like it was going to suceed, but failed at the last moment

In a new virtual environment?

Yes


feat: add beacon api schema

Motivation

see #5973

Description

simple machinery to download the beacon api openapi schema into our source tree

Closing this PR for now as we don't have plans to include the schema here.


Update problemlist files to refer to upstream issue

We prefer to reference the 'real' issue and close our placeholder issues when possible.  https://github.com/adoptium/aqa-tests/issues/3034 was a placeholder but now that the correct upstream issue is found, refer to it in ProblemList files instead.
https://bugs.openjdk.org/browse/JDK-8277361 appears to be fixed, which would imply we could reenable the test.  I want to make this update and then use this example to test our re-enablement automation (upcoming feature).
Closes #3034

Close this without merge, will come back to it under a fresh issue/PR.


colors!

That would be awesome if we could get colors in the usage description.

@fulmicoton Great suggestion! My opinion is that content decoration should be handled outside of yargs, so the API can stay focused on functionality. That's why I created yargonaut.
Originally it only supported decoration via figlet fonts (ASCII Art), but, based on your suggestion, I spent some time today adding support for chalk styles as well. With the latest version of yargonaut (1.1.0), you can now colorize the yargs-generated text for your CLI! (I can't believe I didn't think of this before! :wink:)
Please give yargonaut a try and let us know what you think, one way or another. Hopefully it gives you the flexibility you're looking for. :sunglasses:
Note 1: You'll probably want to focus on the helpStyle(), errorsStyle(), and style() methods.
Note 2: For best results (and full locale support), you should require('yargonaut') before you require('yargs').

@fulmicoton I'm with @nexdrew on this, colors are a great extension to yargs but I think we should keep it out of core. Seem reasonable?

Perhaps we could add a section to the README that has nifty related projects?

I thought of that but didn't want to shamelessly advertise my module or make it seem like yargonaut is officially endorsed by yargs. I'd be happy to add it though.
Personally I think it'd be cool to build support for plugins into yargs, but I haven't come up with an API for this yet. Was planning to bring it up when we talk about commands/V4.

Sorry for not replying earlier. Thank you so much @nexdrew for picking up on that so quickly. I will try that out later today.
About the discussion of whether to put that in yargs or yargonaut, whatever works for you guys :)

@nexdrew be as shameless as you see fit :)

@nexdrew Thank you! It works fine for me. I wondered if it is possible to apply a specific color to option (-f, --file) and commands.

@fulmicoton The official answer is no, because yargonaut can only customize the locale-specific strings baked into yargs. Option keys, command names, and descriptions are defined by the consumer module, and so are not subject to internal y18n lookup.
However, you can hack it via "hidden" commands and options, if you really want to. The code is pretty ugly, and you'll have to duplicate some yargs validation logic, but it would look something like this:
var yargonaut = require('yargonaut')
var chalk = yargonaut.chalk() // could also just require('chalk') yourself

var argv = require('yargs')
  .command(chalk.green('add'), 'Add a file')
  .command('add', false, function (yargs) {
    // need option validation here
    console.log('adding')
  })
  .command(chalk.red('rm'), 'Remove a file')
  .command('rm', false, function (yargs) {
    // need option validation here
    console.log('removing')
  })
  .demand(1)
  .option(chalk.yellow('f'), {
    alias: chalk.yellow('file'),
    describe: 'The file to add or remove',
    type: 'string'
  })
  .option('f', {
    alias: 'file',
    type: 'string'
  })
  .help('h').alias('h', 'help').wrap(null)
  .argv

Your best bet is probably to just color the descriptions instead of the options/commands themselves, like this:
var yargonaut = require('yargonaut')
var chalk = yargonaut.chalk()

var argv = require('yargs')
  .command('add', chalk.green('Add a file'), function (yargs) {
    console.log('adding')
  })
  .command('rm', chalk.red('Remove a file'), function (yargs) {
    console.log('removing')
  })
  .demand(1)
  .option('f', {
    alias: 'file',
    describe: chalk.yellow('The file to add or remove'),
    demand: true,
    type: 'string'
  })
  .help('h').alias('h', 'help').wrap(null)
  .argv

That's about all you can do for now.

It seems that yargonaut is applying some extra non printable characters? require('yargonaut').style('green');

How would you go about applying an rgb hex to the headgins alone (Commands, Options), and then a different styl to the command and a different one to the options?
Thanks.

I can't seem to find a usage: key?

Although if we have the two imports the opposite way around to what the docs specify, we get a yargs instance.

@binarymist Hi 👋 unfortunately I am no longer maintaining yargonaut and it may be incompatible with the latest version of yargs. Please read this for more info.

Ok, so basicallly the answer is to move to sywac. Although it's really anoying as I've just spent a couple of days on this, I do understand your reasoning and it makes sense. I've seen quite a few times project founders do this, and the second, third project in the same catagory inherits all the learnings (same mistakes not made) from the former projects. I have a few questions:

There are very few colaborators on sywac, in fact, you're essentially the only one. What do you think the reason for this is?
How long are you planning on maintaining sywac?
What are your thoughts for why the uptake has been so slow on sywac, I've also never heard of it before now, which is dissapointing
Does sywac resolve all the issues I've mentioned above?
In the comparison matirx  it states that Command Executiion is awaiting completion, does that mean it's not done, or does that mean awaits in the technical sense?
Does sywac allow for removing [boolean] for -v and -h?
Does sywac styling support everything in chalk, rgb/hex values, etc?

I'll create a branch and have a crack at sywac, the docs look good on first appearance, although a lot missing. Do you have a contibuting guide?

I agree that content & styles should be handled separately.
But yargs' API makes it almost impossible to style the output properly.
At least some API should be added to "help" enable colors.

Has sywac been abandoned? I don't see any movement on it for over 6 months and the doc is minimal.

@davidkjackson54 No, sywac has not been abandoned. It's at a relatively stable point and I use it for all my own CLIs. I just haven't had time to work on the docs or road-mapped features in a long time, and no one else has volunteered to help. I am still maintaining the project and will continue to do so for the foreseeable future.

Thanks that is great to know.  I have a need to be able to support positional parameters for example:
cli.js  jobs status --jname
where jobs and status are both positional parameters and jname is a required subparm.
I know yargs can handle this but I haven't been able to find any documented on how to accomplish this in sywac.  I also need to have the help auto generated for this.   So a user could enter
cli.js --help
cli.js jobs --help
cli,js jobs status --help
Are there any samples of how i can accomplish this available?

Are you asking for yargs or sywac? If you are asking how to do this in sywac, please open an issue there.

Done


Print out log file content when there is a CommandException

This allows us the contents of the log file to be printed when a command has failed, so that we don't need to manually find and inspect the log file.

If we are making this change, can we also change the build to run mvn with "-q" flag? Because the maven log will be very noisy in the jenkins console output.

No change is needed for -q, we already supports
--maven-option <comma-seperated-list-of-mvn-options>

@mayankgarg1990 @prithvip looking for an approval, thanks!


[C++] Unit tests failing on Mac

Description
Ticket to track unit tests failing on Mac
Environment info (Required)
Mac OSX 10.11.6
c++ --version
Apple LLVM version 8.0.0 (clang-8<IP_ADDRESS>)
Target: x86_64-apple-darwin15.6.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin
[  PASSED  ] 44 tests.
[  FAILED  ] 18 tests, listed below:
[  FAILED  ] COREOP_PERF.ExecuteBidirectional
[  FAILED  ] COREOP_PERF.TimingCPU
[  FAILED  ] CORE_OP_RUNNER.ExecuteBidirectionalSimpleUnaryList
[  FAILED  ] CORE_OP_RUNNER.ExecuteBidirectionalList
[  FAILED  ] CORE_OP_RUNNER.ExecuteBidirectionalDotProduct
[  FAILED  ] CORE_OP_RUNNER.ExecuteBidirectionalRunnerSimpleUnary
[  FAILED  ] CORE_OP_RUNNER.ExecuteBidirectionalRunner
[  FAILED  ] CORE_OP_RUNNER.ExecuteBidirectionalRunnerDotProduct
[  FAILED  ] CORE_OP_RUNNER.TimingCPUSimpleUnary
[  FAILED  ] CORE_OP_RUNNER.TimingCPUBinary
[  FAILED  ] CORE_OP_RUNNER.TimingCPUBinaryDotProduct
[  FAILED  ] OMP_TUNING.ExecuteBidirectional
[  FAILED  ] OMP_TUNING.EvaluateTuneTestFloat
[  FAILED  ] OMP_TUNING.EvaluateTuneTestDouble
[  FAILED  ] OMP_TUNING.EvaluateTuneTestFloat16
[  FAILED  ] OMP_TUNING.EvaluateTuneTestInt8
[  FAILED  ] OMP_TUNING.EvaluateTuneTestInt32
[  FAILED  ] OMP_TUNING.EvaluateTuneTestInt64

Steps to reproduce
#!/bin/bash
set -e
set -x

pushd .
mkdir -p build && cd build
#cmake -DUSE_CUDA=OFF -DUSE_OPENCV=OFF -DUSE_OPENMP=OFF -DCMAKE_BUILD_TYPE=Debug -DMKL_INCLUDE_DIR=/usr/local/include -DMKL_RT_LIBRARY=/usr/local/lib -GNinja ..  && ninja
export LIBRARY_PATH=/usr/local/opt/lapack/lib
cmake -DUSE_CUDA=OFF -DUSE_OPENCV=OFF -DUSE_OPENMP=OFF -DCMAKE_BUILD_TYPE=Debug -GNinja ..
ninja -v
popd
pushd .
cd python
pip3 install -e .
popd

build/tests/mxnet_unit_tests

With MxNet 1.3.1
[  PASSED  ] 71 tests.
[  FAILED  ] 2 tests, listed below:
[  FAILED  ] ACTIVATION_PERF.ExecuteBidirectional
[  FAILED  ] ACTIVATION_PERF.TimingCPU

@leleamol Something we can look at. 2 C++ tests failing out of 71.

I have fixed this in https://github.com/apache/incubator-mxnet/pull/13409


some special characters missing during conversion

reason: entries in the "direct_mapping" dictionary are no longer continuously numbered from key "0xe0" ("Ohm"). It's easily fixed, use character encoding from wikipedia page:
https://en.wikipedia.org/wiki/ISO/IEC_6937

Somebody else noticed the problem and made a pull request (which I missed). So after 3 years of (unintentionally) ignoring your issue, it's fixed...


Sometimes the button is drawn in the wrong place

I created a button all in  storyboard with height and width constraints (150 150) plus
align center x  = -30
align center y  = -125
the button is placed wrong but the touch is detected at the correct position (empty)
It seems that the align center constraints are ignored as when you drag a button to the view (storyboard) without setting anything
If the storyboard I remove the custom class the button (a square) is drawn in the correct position
if i remove addAnimationForView(self) from awakeFromNib the button is in correct position but not animate

i resolved this way
override public func awakeFromNib() {
    dispatch_async(dispatch_get_main_queue()) {
        self.addAnimationForView(self)
    }
//    dispatch_after(0, dispatch_get_main_queue()) {
//        self.addAnimationForView(self)
//    }
  }

the result is acceptable even with the commented code
but I believe that it is not the right solution

how did you even get it to build properly??


Discord

Description:
Send Discord messages using the notification service.
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#1767
Example entry for configuration.yaml (if applicable):
notify:
   - name: NOTIFIER_NAME
     platform: discord
     token: A1aB2b.C3cD4d-E5eF6f

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
[x] New dependencies are only imported inside functions that use them ([example][ex-import]).
[ x New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

I did not manage to get tox to run on my installation so I have not been able to test, but I tried to adhere to the coding guidelines as much as I could

Notify platform is not async ready at the moment. I wait until #5219 is merged for convert this component to async. I think that should be done befor next release is comming.

Thanks for letting me know. I'll keep an eye out for it. Is there anything I need to do once that happens?

Looks good! :dolphin:

Thanks for the hints and the merge

I need make some change on it for async notfiy component. But It looks good


Refactor github reporer to use common comments code

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

Project coverage is 94.67%. Comparing base (b9a4ba8) to head (db5ca1d).

Files
Patch %
Lines

internal/reporter/github.go
88.29%
5 Missing and 6 partials :warning:

cmd/pint/ci.go
57.14%
2 Missing and 1 partial :warning:

internal/reporter/gitlab.go
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1037      +/-   ##
==========================================
+ Coverage   94.58%   94.67%   +0.09%
==========================================
  Files          96       96
  Lines        9816     9814       -2
==========================================
+ Hits         9284     9291       +7
+ Misses        376      365      -11
- Partials      156      158       +2

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


Use Heroku for Deployment

Heroku is an almost completely free server-hosting solution. I think that browserl.ist should be hosted on Heroku. Custom domains are available in the free edition.

What about privacy? Do they have a guarantee that they do not track users?

The main problem with this website is not hosting. I have a solid and almost free solution, which I use for logux.io and postcss.org.
The problem is that I need to spend time to review pull requests and fix bugs. Because I am sending a few hours every day for PostCSS 8 I do not have time for Browserslist in next month,
For instance, recent PR of moving project to Vercel took a day with problems at my work.

Heroku has a git-based framework.

Every server I host, I work with Heroku.

Heroku has a git-based framework. You just need to click a button and run git push to that application. Also, it has a button for connecting to github.

Yeap. I have the same git-based system for logux.io and postcss.org. The same thing in Vercel.
But you will need a day to make it works correctly. For Vercel I send a day even if Vercel has that auto-git-deploy.

Yes, it does take some time to set up something like the vercel bot.


Using boxZoomEnd and destroying map instance

I have a special case...
When the boxzoomend event is fired, I do some things and create new instances the map (without reloading the old map). The problem with this is that once that the map is recreated the value of L.Draggable._disabled has the value of true.
The problem with this is that on the new map I can't drag.

Was this issue fixed by https://github.com/Leaflet/Leaflet/pull/3050?

Yep, seems like it!


Drag Drop: CdkDropList not adding class cdk-drop-list-dragging if ChangeDetectionStrategy.OnPush

Bug:
CdkDropList not adding class cdk-drop-list-dragging if ChangeDetectionStrategy.OnPush
What is the expected behavior?
CdkDropList should add  class cdk-drop-list-dragging while dragging
What is the current behavior?

Duplicate of https://github.com/angular/material2/issues/13680.


Upgrade to eslint 3.0

Probably good to bump the eslint-plugin-standard peer dep as well.  I tested this branch with v2 and it works fine:
     "eslint-plugin-standard": "^2.0.0"

Just needed to assign someone to get some traction. I don't see any reason to avoid upgrading at this point.


picocli class not found error reported when querying for liquibase version

hi,
Environment
Liquibase Version: 4.8.0
Liquibase Integration & Version: CLI - downloaded liquibase-4.8.0.tar.gz fie from https://github.com/liquibase/liquibase/releases/tag/v4.8.0
Liquibase Extension(s) & Version: N/A
Database Vendor & Version: postgres (PostgreSQL) 9.6.13
Operating System Type & Version: CentOS 7
Description
we are moving from liquibase 2.0.1 version to liquibase 4.8 version in CentOS VM. Liquibase untarred files from liquibase-4.8.0.tar.gz are put here:
ls -lrt /usr/local/lib/liqui*
-rwxr-xr-x 1 root root    2180 Jul 31 21:08 /usr/local/lib/liquibase
-rwxr-xr-x 1 root root 6688205 Jul 31 21:08 /usr/local/lib/liquibase.jar
It complains about picocli class not found error when queried for liquibase version.
So, we added the picocli-4.6.1.jar in path: /usr/local/lib and in /usr/local/jre/lib. This is the PATH env variable:
echo $PATH
/sbin:/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/jre/bin:/usr/local/jre/lib:/usr/local/lib
But still complains of below error:
$ /usr/local/lib/liquibase --version
Error: A JNI error has occurred, please check your installation and try again
Exception in thread "main" java.lang.NoClassDefFoundError: picocli/CommandLine$Model$ArgSpec
at java.lang.Class.getDeclaredMethods0(Native Method)
at java.lang.Class.privateGetDeclaredMethods(Class.java:2701)
at java.lang.Class.privateGetMethodRecursive(Class.java:3048)
at java.lang.Class.getMethod0(Class.java:3018)
at java.lang.Class.getMethod(Class.java:1784)
at sun.launcher.LauncherHelper.validateMainClass(LauncherHelper.java:650)
at sun.launcher.LauncherHelper.checkAndLoadMain(LauncherHelper.java:632)
Caused by: java.lang.ClassNotFoundException: picocli.CommandLine$Model$ArgSpec
at java.net.URLClassLoader.findClass(URLClassLoader.java:387)
at java.lang.ClassLoader.loadClass(ClassLoader.java:418)
at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:355)
at java.lang.ClassLoader.loadClass(ClassLoader.java:351)
... 7 more
Below is where jre is download and put in the VM:
$ ls -lrt /usr/local/jre
total 44
-rwxr-xr-x  1 root root  123 Jul 27 03:01 release
-rwxr-xr-x  1 root root  955 Jul 27 03:01 Welcome.html
-rwxr-xr-x  1 root root  190 Jul 27 03:01 THIRDPARTYLICENSEREADME.txt
-rwxr-xr-x  1 root root  190 Jul 27 03:01 THIRDPARTYLICENSEREADME-JAVAFX.txt
-rwxr-xr-x  1 root root   46 Jul 27 03:01 README
-rwxr-xr-x  1 root root   44 Jul 27 03:01 LICENSE
-rwxr-xr-x  1 root root 3244 Jul 27 03:01 COPYRIGHT
drwxr-xr-x  2 root root 4096 Jul 27 03:12 bin
drwxr-xr-x  4 root root 4096 Jul 27 03:12 legal
drwxr-xr-x  3 root root 4096 Jul 27 03:12 plugin
drwxr-xr-x 15 root root 4096 Aug  4 11:18 lib
JRE version is 1.8.0_321.
Could you please help what is wrong here?

@hemabs Can you start with a new install, untar the file and then leave things where they are?

Thanks @kataggart, the solution suggested above worked for us. Earlier we were just moving liquibase and liquibase.jar file selectively from the downloads.

WOO HOO!!! So glad @hemabs !


Use brighter words and branding

Naturally the docs and code is all a bit technical at the moment. Let's make everything look shiny, fun and clear before release.

Dan's done a great job with this!  Closing.


samples/drivers/led/pwm    regex matching the LOG output

Change the expected regex of the sample.yaml   to what the main.c is LOGGING
Fixes https://github.com/zephyrproject-rtos/zephyr/issues/80319

go with PR https://github.com/zephyrproject-rtos/zephyr/pull/80605


fix: remove debug_assertions check causing release builds to fail

Thanks! I managed to miss this yesterday.


[Explicit Module Builds] Use a stable hash to encode build arguments into PCM filenames

When generating build jobs for explicit module dependencies, we must distinguish multiple versions of the same Clang module/pcm by encoding the arguments used to build it (e.g. target triple) into the pcm filename.
So far, we have been using an unstable hash, which means that these filenames were not stable across invocations and were being needlessly re-built in some scenarios (e.g. many-target builds).
Switching to a stable hash (SHA256) ensures that the same Clang module, built with the same command line, will produce the same .pcm filename. This will allow the client build system (e.g. SwiftPM) to deduplicate identical compilation jobs because their output filenames will now also match.

@swift-ci please test

😬 Linux build failing with:
Fatal error: not supported on this platform: file TSCBasic/HashAlgorithms.swift, line 189

@swift-ci please test

@swift-ci please test

@swift-ci please test


Use settingKey for SettingKey definitions

I'm not sure exactly how SettingKey.apply changed such that these definitions result in "A named attribute key must start with a lowercase letter" errors on sbt 1.1.0-RC1, but using settingKey fixes the issue and is more consistent, anyway.

Thanks, Travis.

surely this is an sbt bug — is there a ticket on it?

at first glance I thought this must be an sbt bug, but now I think the old code here was just wrong and it's good that sbt now catches it.

Indeed, I see it as grey zone as well. sbt 1.1 shouldn't really cause an exception to be thrown, even if it's made strides to transition keys from the old "kebab-case" to camelCase, however the code here was also simply "wrong".


[ACS][CallAutomation] Adding RemoveParticipants Events

Adding RemoveParticipants Events for ACS CAll Automation

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-communication-callautomation


LayoutLMv2 for CORD dataset

Hi,
My custom dataset resembles the CORD dataset used in the paper , I am trying to fine tune it but can't find the exact code/approach used for fine tuning  , the paper doesn't mention any information in the supplementary section, I would appreciate your help if someone could shed some light on this
Thanks

@tanay2001 you may directly use the script for fine-tuning FUNSD


inurl default values in subpatchops

delay loading after values have been set from deserialize...?

seems to work fine, default value is not used at all