Workbook API: missing PATCH method for updating workbook range

Hi,
metadata doesn't contain method for updating workbook range. Without this method, we can't update cells. It make Workbook API still unusable. Metadata contains only GET method.
https://learn.microsoft.com/en-us/graph/api/range-update?view=graph-rest-1.0&tabs=http
@baywet @sebastienlevert @andrueastman It seems to me to be the last major blocker related to Workbook API.
I can try to fix missing PATCH, but I don't know what's the root cause.
Maybe another blocker is still absence of composable functions like https://learn.microsoft.com/en-us/graph/api/range-delete?view=graph-rest-1.0
<!-- Set IsComposable to false for all functions with a ReturnType of graph.workbookRange. -->
    <!-- This is to prevent overgeneration of composable functions in the OpenAPI. -->
    <xsl:template match="edm:Schema[@Namespace='microsoft.graph']/edm:Function[edm:ReturnType[@Type='graph.workbookRange']]/@IsComposable">
        <xsl:attribute name="IsComposable">false</xsl:attribute>
    </xsl:template>

Thanks for creating this.
For context this transform was added with #561 and is most likely the reason why the delete action operations are missing. We'd need to investigate the blast radius (in terms of expansion) if we removed that. @irvinesunday maybe there are other functions that are bound to workbook range, and are composable themselves, which lead to over expansion. And maybe pushing this transform "one level downwards" would help us enable those scenarios without blowing the expansion? What do you think?
As for the PATCH on the range function, I don't even think this means anything OData wise. Functions by nature do not describe a request body. It seems to me that the excel team decided to implement a PATCH operation support under that path even though it can't be represented with OData semantics.

Regarding to expanded paths. Do we need two ways to access the excel tables or names?
/drives/{drive-id}/items/{driveItem-id}/workbook/tables vs. /drives/{drive-id}/items/{driveItem-id}/workbook/worksheets/{workbookWorksheet-id}/tables
/drives/{drive-id}/items/{driveItem-id}/workbook/names vs. /drives/{drive-id}/items/{driveItem-id}/workbook/worksheets/{workbookWorksheet-id}/names
SDKs can generate code only for /drives/{drive-id}/items/{driveItem-id}/workbook/tables and /drives/{drive-id}/items/{driveItem-id}/workbook/names.
Or */microsoft.graph.columnsAfter() can be removed, because it's the same as */microsoft.graph.columnsAfter(count=n) where n=1
Same for */microsoft.graph.columnsBefore() and */microsoft.graph.columnsBefore(count=n) or */microsoft.graph.usedRange() vs. */microsoft.graph.usedRange(valuesOnly=bool).
https://learn.microsoft.com/en-us/graph/api/workbookrange-columnsafter?view=graph-rest-1.0
If the excel team could check newly generated paths in the diff mentioned in https://github.com/microsoftgraph/MSGraph-SDK-Code-Generator/pull/1141#issuecomment-1904988138, maybe they will find some paths which do not make sense, like:
/drives/{drive-id}/items/{driveItem-id}/workbook/tables/{workbookTable-id}/columns/{workbookTableColumn-id}/microsoft.graph.headerRowRange()/sort
Applying sort to headerRowRange doesn't make sense. Sort itself doesn't make sense (https://learn.microsoft.com/en-us/graph/api/resources/workbookrangesort?view=graph-rest-1.0), it should always be followed by apply (https://learn.microsoft.com/en-us/graph/api/rangesort-apply?view=graph-rest-1.0)
Regarding to the PATCH on the range function. Is there any workaround for this or it's something that requires changes in Workbook API?

Thanks for looking into it. This is a bit like airing our dirty laundry in public, but we've had multiple conversations with the Excel team about quality of the metadata in the past. Basically the API that's available in the graph is automatically generated from the Excel object model (in the product) to a large degree.
Both the Excel team and us don't have such a granular control over things, and there doesn't seem to be any priority/urgency in tweaking the generation process.
What we can control however are:

Whether functions are composable or not.
Whether an entity set/collection property is the "ContainsTarget" (canonical path) or not. (that triggers the functions expansion)
Additional annotations to say "yes, this is not supposed to be expanded, but please expand it anyway", or the opposite.

For the optional parameters, we need to provide both (with and without the parameters) since we don't want to force people to provide a value when they don't want/need to.
For the tables, I don't know that API very well, but I'm guessing that you can have tables outside of worksheets? If not, it's probably a metadata issue and we should kill those bindings to "save some space" as the APIs underneath most likely don't work anyway.

Since it's functionally equivalent to the other endpoint with worksheet, it'd be worth removing the binding from functions that return tables to workbooks then, so everyone uses the "canonical path" (through worksheet) to avoid over expansion.

Not sure why it was closed...

Please reopen

I'm looking into this

With regard to missing PATCH operations for workbook range...
The function range is a bound function
<Function Name="range" IsBound="true">
  <Parameter Name="bindparameter" Type="graph.workbookRangeView" />
  <ReturnType Type="graph.workbookRange" />
</Function>

According to the OData spec.: https://docs.oasis-open.org/odata/odata-openapi/v1.0/cn01/odata-openapi-v1.0-cn01.html#sec_InvokeBoundActionsandFunctionsonSing

We only generate get operations for bound functions. Do we want to start supporting patch operations for all bound functions?
cc: @darrelmiller

@irvinesunday Looks good, paths for all methods are generated.
I've noticed that workbook paths are generated also for File Storage Container (paths started at the line 8608) I'm not familiar with SharePoint Embedded, but if file storage container has relationship to drive and the drive itself can be accessed via /drives/{driveId} endpoint then no need to generate paths /storage/fileStorage/containers/{fileStorageContainer-id}/drive/items/{driveItem-id}/workbook. Similar to /sites/{siteId}/drive/* endpoints which are omitted from SDKs.

@irvinesunday thanks for looking into this.
It'd be interesting to see whether the paths we're loosing here are actually supported by the service or not?
Also the removal of the composable false in the XSLT here is a ~15% size increase. It's non-negligeable. It'd be interesting to look into the new paths that are added and see whether we're getting repetitions in the segments?
My hunch is that a couple of functions (not range) are marked as composable when in facts they are not, leading to a lot of "wasteful repetition". If we could identify them as part of this patch it'd probably result in a much better experience.

I have removed the containment for the drive navigation property of the entity fileStorageContainer and the resulting paths have significantly reduced. It seemed like we were having a lot of repetitive paths on the storage API:

Here's the new diff: https://www.diffchecker.com/U7BhT8uW/

As far as I understand the new file storage API, this is incorrect. When people deploy SharePoint embedded, they are effectively the equivalent of the Site/Web structure we have with other APIs. And the drive actually belongs to the container in the sense it can't be accessed any other way. Hence the containment.

@irvinesunday thanks for looking into this. It'd be interesting to see whether the paths we're loosing here are actually supported by the service or not? Also the removal of the composable false in the XSLT here is a ~15% size increase. It's non-negligeable. It'd be interesting to look into the new paths that are added and see whether we're getting repetitions in the segments? My hunch is that a couple of functions (not range) are marked as composable when in facts they are not, leading to a lot of "wasteful repetition". If we could identify them as part of this patch it'd probably result in a much better experience.

We are getting some repeated segments, for example:
/drives/{drive-id}/items/{driveItem-id}/workbook/worksheets/{workbookWorksheet-id}/microsoft.graph.usedRange()/microsoft.graph.usedRange() and
/storage/fileStorage/containers/{fileStorageContainer-id}/drive/items/{driveItem-id}/workbook/worksheets/{workbookWorksheet-id}/microsoft.graph.usedRange()/microsoft.graph.usedRange()
I've only picked those 2 instances for now.
Also, @baywet what do you mean when you say the file storage API is incorrect? That it is not supported by the service? I can see its reference doc. here: https://learn.microsoft.com/en-us/graph/api/resources/filestoragecontainer?view=graph-rest-1.0
Also, with regard to making the drive navigation property non-contained, can't we get to the same resource using the drives API? Example is given here: https://learn.microsoft.com/en-us/graph/api/filestoragecontainer-get-drive?view=graph-rest-1.0&tabs=http#http-request

I can't think of a valid scenario where the same function would be composed to itself directly (not other function in between). So this is probably a fix to implement for yoko.
What I meant was that I believe /storage/fileStorage/containers/{fileStorageContainer-id}/drive/items path and similar ones are valid and should be part of the conversion result.
For the drive that lives under the container (and drive items, etc...), there's no other path to access those.

I can't think of a valid scenario where the same function would be composed to itself directly (not other function in between). So this is probably a fix to implement for yoko.
What I meant was that I believe /storage/fileStorage/containers/{fileStorageContainer-id}/drive/items path and similar ones are valid and should be part of the conversion result. For the drive that lives under the container (and drive items, etc...), there's no other path to access those.

Got it. Let me make the necessary changes to Yoko

The Yoko lib PR to resolve this is ready for review: https://github.com/microsoft/OpenAPI.NET.OData/pull/552

Thanks for investigating this further @irvinesunday . This begs the question: if the API surfaces are the same, why does the container entity have a drive property? @darrelmiller do you have additional insights from the reviews here?

I just had a chat with Kevin Lam from ODSP. He confirmed that both path sets are valid on the service, but most people today are using the /drives set with SPE. This is because it was designed as a drop in replacement for OSDP and so ISVs don't have to update their code to support it. It'd be fine to patch the contains target to false and ask people to use the request builders with URLs they built themselves if they absolutely want to use that path over the canonical one.

Thanks for the follow-up @baywet


Viral Short Video Analysis

Pull Request for ML-Crate 💡
Issue Title:

Info about the related issue (Aim of the project) :
Name:
GitHub ID:
Idenitfy yourself:

Closes: #509
Describe the add-ons or changes you've made 📃
Creates an all sort of analyzed visuals, perform preprocessing in dataset and finally made a model.
Type of change ☑
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested? ⚙
Describe how it has been tested
Describe how have you verified the changes made
Checklist: ☑

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ ] I have added things that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

Please remove the unnecessary files and create a fresh PR with the proper project structure for this issue.
Project Folder
|- Dataset
   |- dataset.csv (dataset used for the particula project)
   |- README.md (brief about the dataset)
|- Images
   |- img1.png
   |- img2.png
   |- img3.png
|- Model
   |- project_folder.ipynb
   |- README.md
|- Web App
   |- templates
   |- static
   |- app.py
   |- demo.mp4
   |- README.md
|- requirements.txt


How to get a minified version for deployment?

This is probably wrong here, but I don't know where else to put it.
I don't quite get how to get or generate a minified version that can be used for deployment.
When I download the zip file or do "bower install ..." I get a lot of unnecessary stuff including examples and stuff from each package that I don't want to have lying around as assets. And a lot of the css and js aren't minified. Also, I don't need the source code versions of each package.
Is there a way to get a minified distribution?

You should use a build tool like Gulp or Grunt to handle all the required files (obviously - you won't need all SB Admin dependencies) so you concat all required styles and scripts using build tool and include one .css file and one .js file in your view template.
I highly recommend using Gulp because it's faster and more modern. Also I very like Laravel's approach to this subject - you can check it out at http://laravel.com/docs/5.1/elixir

Not what I have hoped for.
But thanks for the answer.

@sykano I think that a build tool is a good option. All of the code in this template is purposely set up in an expanded format just for ease of use. Typically, a built tool like Grunt or Gulp will allow you to concat and minify your JS, so I would also recommend looking into those sorts of tools. The main idea with this template is to act as a baseline starting point for some sort of web app back-end, but it isn't really production ready out of the box, it's more of a boilerplate of sorts with a few cool features added into it to allow you to rapidly prototype something.
Hopefully this explains why things aren't minified by default, and gives some insight into how to get to your minified goals!

Being an absolute beginner in terms of web front-end development i am trying to understand how i could use a build tool (Gulp) with sb-admin-2 as i would love to make a few simple change and have it setup so that i could use it on new projects really easily where it is more tailed towards what i would need.
Am i correct in understanding that i could setup gulp to do things like minify, compile less > css, create a dist folder for production purposes etc. but that none of this is included when i clone/fork sb-admin-2?
If that is correct i just need to figure out how to do that :) The laravel project seems to more or less do this but it is somewhat above my skill level to fully understand.
Thank you very much for building and maintaining this admin theme!

I've now released some Gulp tasks that minify the compiled CSS and JS and throws them into the dist directory. You can use the minified versions of the CSS and JS directly from the dist or you can use the LESS files and the newly added Gulp tasks to further develop the theme as you wish!


Loggers should never use f-strings

it's not enable by default by ruff, but we likely should.
https://docs.astral.sh/ruff/rules/logging-f-string/
See https://github.com/pydata/pydata-sphinx-theme/pull/2044#discussion_r1850241251_

Addressed already so closing


Text color not changing for table view cell drop down

Hello, I am having an issue of not being able to change the text color in drop down cells. However, the text color does change for the title cell which is a bit confusing.

did you get answer for that

did you get answer for that

I found a way to fix it in the code for the drop down file. If you just use control F to find all the spots where they change the text color apparently they didn’t allow customization of the drop down row text color so I just added a few lines in the file itself. I can send it to you if you would like and you can compare the two to find what code I changed because I forget the exact location.
Sent from my iPhone

On Jan 2, 2021, at 3:56 AM, khaled samer hassan elsedek<EMAIL_ADDRESS>wrote:

did you get answer for that
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I found a way to fix it in the code for the drop down file. If you just use control F to find all the spots where they change the text color apparently they didn’t allow customization of the drop down row text color so I just added a few lines in the file itself. I can send it to you if you would like and you can compare the two to find what code I changed because I forget the exact location.
Sent from my iPhone

On Jan 2, 2021, at 3:56 AM, khaled samer hassan elsedek<EMAIL_ADDRESS>wrote:

did you get answer for that
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


bump LanceDB [vectordb] to latest for stability reasons

0.1.8 looks to be broken and non-responsive to any client commands on Mac arm64. Will try in Docker container next to see if working on that platform


Return sorted keys stored in the map

In most languages, for instance C++ have keys ordered in sorted fashion.
Thus, I have added an option to receive keys in the map in sorted order.

Hi @orcaman
Can you review this PR?

Hi @deepak0004
Thank you very much for your time in preparing this PR.
I think it's important to distinguish between 2 different things:

Creating a sorted map, such that when iterating over the map, one could perform operations in some desired sequence
Getting a sorted string list of keys

While I can see why the former may be interesting, I don't think the latter is a need that should be suppored natively in the map object, just like the native map doesn't support this. Overmore, the implemnetation given here would suggest to the use that the former is being done while it isn't, which looks a bit confusing to me.
Anyways appreciate the time and effort on this.


add visibility declaration to __construct

Just cloned the repo and run CodeSniffer, found there was a __construct method without the visibility declaration.

:+1:

Thanks for the fix!

Not worries! it´s a pleasure to be able to help in such a great project! cheers


Playing local audio files

how would i go on about playing a local file?

Never tried this. Presumably, you would have to send the file path as the identifier. Bear in mind that the file is local to the lavalink server receiving the request.

oh so something like ?identifier=/path/to/file?

I'd assume so

i tried, got nothing :(

You need to URL encode the parameter.
/path/to/file becomes %2Fpath%2Fto%2Ffile

Also don't forget to enable the local audio source manager in application.yml

still didn't seem to work. My application has local: true and i did the URL encoded way and tried multiple paths and still didn't seem to work

You need to use it like this: .play file://Music/
Music would be the folder you added in the same folder where the fredboat.jar file is.
However there is a little bug. You can't have any spaces in your path or it won't work.

The audio file is on the server, right?

yes @Frederikam

So after testing, the file has to be in the same folder as the Lavalink.jar file, the URL to get the track is http://localhost:2333/loadtracks?identifier=TheFileName.extension

I'm not sure the file has to be in the same folder, I've had MP3 tracks in my music folder on Windows (no spaces, though), identifier isn't URL encoded (using lavalink.py V2) and it works

Also to clarify, paths with spaces are also working for me.

Thats werid, didnt work for me

What lavalink wrapper are you using?

Discord.js-lavalink (My own) but wrapper doesn't matter, its what you use to resolve the track, lol

Can also confirm it works for me and it doesn't have to be in the Lavalink folder.

Discord.js-lavalink (My own) but wrapper doesn't matter, its what you use to resolve the track, lol

If your wrapper doesn't percent encode the string it should fail. Eg you need %20 instead of a space.

@Frederikam The users of my wrapper are to handle resolving of the tracks themselves... it doesnt do anything on the rest side...

To sum up the given answers:

Make sure local files are enabled in your lavalink server config
Properly url encode the query parameters when resolving the tracks via /loadtracks
Ensure the audio files are actually present on the machine where lavalink is running

If this does not resolve the issue, please post logs of the server and/or show what happens when you curl the /loadtracks endpoint.


Dragable across X Axsis

Hi,
This is a great example, would be nice if we could also drag across the X axsis for lists that are horizontal.

Yeah, that might be a nice alternative/ extension. Must admit it has been a while since I work on the code (unfortunately). Sorry for the late reply.
But maybe I'll get an opportunity to do that. This simple example relies on a fixed height for each of the items in the vertical list (which is quite common). A simple example for horizontal version would require a fixed width for all items too, but then it would be relatively straightforward I guess.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/75682

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.


Client foraging

Summary

Added look at the previous 5 foraging history for deer hunters ( a lot of hunter past 5 turns mean low population thus chance to go deer hunting will decrease)
Halfed the amount for foraging if we get <0 RoI from initial (overcommitment? - waiting for the Infra to release the cost of living parameter for public use)
Touhed up gifting
Added opinions on gifting
Changed TH to percentages

Additional Information
I CHANGED THE PARMETERS AND DIDN'T CHANGE THEM BACK OOPS
Test Plan

Add information on how you tested your changes.

Changes made, but parameters for deer return and fish return still very high

Changes made, but parameters for deer return and fish return still very high


VTableHacks on Mac / Linux

Currently there are a few hacks in place to swap out vtable function entries to override native C++ functions without having USharp depend on engine modifications.
On Mac the vtable swap works but calling the original function results in a crash (this pointer is passed in via RDX when we call the original function, but it expects it to be in RDI (at least for AActor::BeginPlay()).
Is there something wrong with the way the original function is being called? If not then the vtable hacks cannot be used as they don't work as expected. Engine modifications would be the only alternative route.
typedef void (UObject::*ActorBeginPlayFunc)();
CSEXPORT void CSCONV Export_VTableHacks_CallOriginal_ActorBeginPlay(ActorBeginPlayFunc Func, UObject* Obj)
{
	(Obj->*Func)();// "Obj" is passed in via RDX, the target function expects RDI
}

EDIT:
After doing some tests in Compiler Explorer (https://godbolt.org/) with gcc/clang, they both seem to set up the registers as expected. Xcode's disassembly view where it crashes seems to chop off instructions prior to those in the stack trace, so I was assuming it was passing by RDX based on the register values from register read.
The following is some code which should work in UE4 based on the disassembly in the main 3 compilers (gcc, clang, msvc) which can be tested on https://godbolt.org/.
#include <iostream>
using namespace std;

class UObject
{
public:
    int Val1;
    int Val2;
};

class AActor : public UObject
{
public:
    virtual void BeginPlay()
    {
        cout << "AActor " << Val1 << " " << Val2 << "\n";
    }
};

class ASomeActor : public AActor
{
public:
    virtual void BeginPlay() override
    {
        cout << "ASomeActor " << Val1 << " " << Val2 << "\n";
    }
};

typedef void (AActor::*ActorBeginPlayFunc)();
void Export_VTableHacks_CallOriginal_ActorBeginPlay(ActorBeginPlayFunc Func, AActor* Obj)
{
    (Obj->*Func)();
}

int main()
{
    AActor obj = ASomeActor();
    obj.BeginPlay();

    Export_VTableHacks_CallOriginal_ActorBeginPlay(&AActor::BeginPlay, &obj);

    return 0;
}

Fixed in https://github.com/pixeltris/USharp/commit/19120d64b3a0be40ad51c7a29d16197c87153c97 (only tested Mac). The C++ invoker was moved to C#, using the ThisCall calling convention. I can't think of any logical reason why the C++ code doesn't work as expected. There will likely be other similar issues related to VTableHacks.... possibly look at submitting engine PRs (probably futile as they will likely get rejected due to not being useful for anything other than supporting USharp).

You should make a few PRs anyways. MHutch was actually able to get several of her PRs accepted even though she barely explained what their purpose was. I'd recommend testing it out first of course :joy:

Not sure if this fits here or not, but with 4.23
Engine/Extras/ThirdPartyNotUE/SDKs/HostLinux/Linux_x64/v15_clang-8.0.1-centos7/x86_64-unknown-linux-gnu/bin/x86_64-unknown-linux-gnu-ld: .config/USharp/Build/HostProject/Plugins/USharp/Intermediate/Build/Linux/B4D820EA/UE4Editor/Development/USharp/Module.USharp.cpp.o: in function FActorComponentTickFunction::FActorComponentTickFunction()': Engine/Source/Runtime/Engine/Classes/Engine/EngineBaseTypes.h:461: undefined reference to vtable for FActorComponentTickFunction'
Engine/Extras/ThirdPartyNotUE/SDKs/HostLinux/Linux_x64/v15_clang-8.0.1-centos7/x86_64-unknown-linux-gnu/bin/x86_64-unknown-linux-gnu-ld: .config/USharp/Build/HostProject/Plugins/USharp/Intermediate/Build/Linux/B4D820EA/UE4Editor/Development/USharp/Module.USharp.cpp.o: in function FActorTickFunction::FActorTickFunction()':  Engine/Source/Runtime/Engine/Classes/Engine/EngineBaseTypes.h:428: undefined reference to vtable for FActorTickFunction'


fix: remove static lifetime for name str parameter requirement for constant getter

remove static lifetime for name str parameter requirement for constant getter
Description
Hey!
I noticed get_constant_from_var_name has a different signature from other hint_utils helper functions, namely that it has a bound of static lifetime on the name parameter. I noticed it mainly comes from the error signature HintError::MissingConstant.
I changed the signature of MissingConstant to match other errors: from Box<'static &str> to Box using the same util you use around the hint error codebase: var_name.to_string().into_boxed_str().
In doing this, one no longer needs to have 'static strings for constant getting -> this will allow me in next PRs to have dynamic constant name getting, i.e., in garaga there are constant limbs of Prime P of the BN Curve.
And depending on N_LIMBS, we'll get P_i.
Let me know if this works!
Resolves: #1522
Description of the pull request changes and motivation.
Checklist

[x] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.
[ ] CHANGELOG has been updated.

Applying this patch should fix the linter:
diff --git a/vm/src/hint_processor/builtin_hint_processor/math_utils.rs b/vm/src/hint_processor/builtin_hint_processor/math_utils.rs
index 68cb30fc1..bbe18b5d7 100644
--- a/vm/src/hint_processor/builtin_hint_processor/math_utils.rs
+++ b/vm/src/hint_processor/builtin_hint_processor/math_utils.rs
@@ -2085,7 +2085,7 @@ mod tests {
         //Execute the hint
         assert_matches!(
             run_hint!(vm, ids_data, hint_code),
-            Err(HintError::MissingConstant(bx)) if *bx == ADDR_BOUND.to_string()
+            Err(HintError::MissingConstant(bx)) if &*bx == ADDR_BOUND
         );
     }

@@ -2309,7 +2309,7 @@ mod tests {
         //Execute the hint
         assert_matches!(
             run_hint!(vm, ids_data, hint_code),
-            Err(HintError::MissingConstant(x)) if (*x) == "MAX_HIGH".to_string()
+            Err(HintError::MissingConstant(x)) if &*x == "MAX_HIGH"
         );
     }

diff --git a/vm/src/hint_processor/builtin_hint_processor/uint384.rs b/vm/src/hint_processor/builtin_hint_processor/uint384.rs
index 4e2bb4412..9f6646dd3 100644
--- a/vm/src/hint_processor/builtin_hint_processor/uint384.rs
+++ b/vm/src/hint_processor/builtin_hint_processor/uint384.rs
@@ -535,7 +535,7 @@ mod tests {
         //Execute the hint
         assert_matches!(
             run_hint!(vm, ids_data, hint_code::ADD_NO_UINT384_CHECK),
-            Err(HintError::MissingConstant(bx)) if *bx == "SHIFT".to_string()
+            Err(HintError::MissingConstant(bx)) if &*bx == "SHIFT"
         );
     }


Fits don's include central value

One thing I noticed is that the x_values used to compute the function values don't include the central value (see derivative_calculator.py at lines 55–66). This isn't difficult to fix and may in fact not be necessary, but it is good to be aware of this.

One thing I noticed is that the x_values used to compute the function values don't include the central value (see derivative_calculator.py at lines 55–66). This isn't difficult to fix and may in fact not be necessary, but it is good to be aware of this.

yeah I noticed it, probs because I was adding the gaussian noise to it. can fix it when I do the upgrade with the broadcasting etc


Module Index

Is your feature request related to a problem? Please describe.
We have a list of modules and per module documentation. I would like to provide a module overview of sorts similar to a module index, e.g. a table with a list of modules and a short description.

This looks like a duplicate of #255

@khos2ow I think #255 is for a particular module (e.g. module A), while this issue is talking about a list of all available modules in directory modules. Does that distinction make sense?

Can you provide a sample table output of what you have in mind?

./modules/README.md <---- Module index
./modules/foo/README.md <---- Module readme
./modules/bar/..

I would like to see this as well - currently am doing it by hand: https://infra.dataops.tk/catalog/


Remove Seattle-specific applicant support page

I came across the UnconfirmedIdcsEmailBugView.java file and it's got hard coded text for people to contact Seattle with phone and email address. We probably want to make this configurable so it can be customized for other cities.
We should investigate to see if this can/should be generalized.

The file you found is a pretty Seattle specific issue that we may want to generalize as part of this bug.
It can only be triggered by Seattle users due how Seattle login team manages unconfirmed users.  So those specifics wouldn't translate to another deployment.

Oh that interesting. I'll leave this open so it can be investigated further.

Going to leave this for Seattle to clean up, changing the label to seattle-backlog

Here's the original Issue, the PR that added this, and the call site that triggers it.

@gwendolyngoetz to pull stats for review on those who would be affected

@gwendolyngoetz -- Are you working on this? Should we move this to "In Progress"? "Ready for Dev"? Still necessary?

@elisekalstad I added logging to monitor if tit was ever actually being called. Then I forgot about it. I need to review the logs to see if I can remove this.


IPv6 Support

Hi,
Are there any plans on adding IPv6 support?
It's hard to set up manually because wg-dashboard keeps overwriting the wg0.conf, but IPv6 does seem to work with some config changes.
Example IPv6 server config:
Address = <IP_ADDRESS>/24,fd10:37:37::1/64
PrivateKey = REDACTED
ListenPort = 58210
PostUp = iptables -A FORWARD -i %i -j ACCEPT; iptables -A FORWARD -o %i -j ACCEPT; iptables -t nat -A POSTROUTING -o ens3 -j MASQUERADE; ip6tables -A FORWARD -i %i -j ACCEPT; ip6tables -A FORWARD -o %i -j ACCEPT; ip6tables -t nat -A POSTROUTING -o ens3 -j MASQUERADE
PostDown = iptables -D FORWARD -i %i -j ACCEPT; iptables -D FORWARD -o %i -j ACCEPT; iptables -t nat -D POSTROUTING -o ens3 -j MASQUERADE; ip6tables -D FORWARD -i %i -j ACCEPT; ip6tables -D FORWARD -o %i -j ACCEPT; ip6tables -t nat -D POSTROUTING -o ens3 -j MASQUERADE
SaveConfig = false

[Peer]
        # client
        PublicKey = REDACTED
        AllowedIPs = <IP_ADDRESS>/32,fd10:37:37::2/128

Example IPv6 client config:
PrivateKey = REDACTED
Address = <IP_ADDRESS>/24, fd10:37:37::2/64
DNS = <IP_ADDRESS>

[Peer]
PublicKey = REDACTED
AllowedIPs = <IP_ADDRESS>/0, ::/0
Endpoint = REDACTED
PersistentKeepalive = 25

Also, net.ipv6.conf.all.forwarding=1 has to be set in the sysctl.conf

Hi,
adding IPv6 support is essential in our days of the internet. I will make sure to add this in the coming rewrite.

is there any estimated date to get ipv6 working?
thanks and very nice project! congratz!!

Can't wait for IPv6 implementation, and thanks for an already-great piece of software. :)


Screen Capture

Describe the bug
A clear and concise description of what the bug is.
When trying to record someone's screen we are getting "Failed to capture screen!"
Expected behavior
A clear and concise description of what you expected to happen.
Record the duration amount of screen reply of someone
Screenshots
If applicable, add screenshots to help explain your problem.
https://img.kaieagle.me/u/EF6rdCT.png

idk if this is still an issue but i am unable to reproduce this. What cluster is this on?

idk if this is still an issue but i am unable to reproduce this. What cluster is this on?

It's fixed now


client.getNonce() no longer works with SDK v0.39.0

Running getNonce produces the following error:
vue.runtime.esm.js?2b0e:1888 TypeError: str.toLowerCase is not a function
    at __decode (index.js?bf2b:78)
    at Object.decode (index.js?bf2b:116)
    at Function.decode (bech32.js?2ead:30)
    at Object.decodeBech32Pubkey (pubkey.js?904b:28)
    at SigningCosmosClient.getAccount (cosmosclient.js?b5a7:98)
    at async SigningCosmosClient.getNonce (cosmosclient.js?b5a7:78)

I'm thinking the problem is that the client API has changed http://localhost:1317/auth/accounts/cosmos12x4undlwc0...
v0.38.4:
{
  "height": "2",
  "result": {
    "type": "cosmos-sdk/Account",
    "value": {
      "address": "cosmos1c2ju5u3wx57zv8ceehpqcvh7nusc4vgt67dg3q",
      "coins": [
        {
          "denom": "token",
          "amount": "1000"
        }
      ],
      "public_key": "cosmospub1addwnpepqttcheas8x7p4f9km0kc9qgcguk9lkdery744tujnc2dshtjfqxh59l6gsa",
      "account_number": 2,
      "sequence": 1
    }
  }
}

v0.39.0:
{
  "height": "8",
  "result": {
    "type": "cosmos-sdk/Account",
    "value": {
      "address": "cosmos12x4up7dhsndlwc06t4p3jfn8zvz6x3t72fp30v",
      "coins": [
        {
          "denom": "token",
          "amount": "1000"
        }
      ],
      "public_key": {
        "type": "tendermint/PubKeySecp256k1",
        "value": "AvjV2kYc2DldNZ7XEqGzyoyv+sd6qJCyEps7APBE+ugV"
      },
      "account_number": "2",
      "sequence": "1"
    }
  }
}

Yup, the public_key has changed from a string to an object. Thanks for the heads up. We actually don't offer support for v0.39 right now, and I believe the plan is to jump straight to v0.40, but @webmaster128 can confirm.

We'll need to add support for 0.39/Launchpad. Ideally we have a tiny @cosmjs/sdk39 package that re-exports most of the functionality from @cosmjs/sdk38 and only adds the parts that changed.

@fadeev thank you for reproting this. Please have a look at https://github.com/CosmWasm/cosmjs/issues/306 for the higher level migration roadmap.

account_number/sequence also changed from number to string

@fadeev if you want to test or review a patch for this, see #314. That should allow you to connect to a launchpad backend. The wasmd/launchpad testing backend does not yet work.


hide an element on the mouse

how can I hide or remove a draggable element that follows the mouse? in the screenshot it is marked in red.
I want to keep ghost element.

https://github.com/SortableJS/Vue.Draggable/issues/340


Fix vagrant not use runc

minimum build (only builds nodejs)
do not use runc by default (readme instruction on how to get runc working)
build cli from source
cleaned up the readme a bit

Travis doesn't support vagrant, I tried linux and osx.

May be we can pin Docker version to one where runc works. As while working on Mac if I need to  try/test out any runc related aspect I rely on Vagrant image. So if it works with some older version we can pin Docker to  that and keep runc enabled

May be we can pin Docker version to one where runc works. As while working on Mac if I need to try/test out any runc related aspect I rely on Vagrant image. So if it works with some older version we can pin Docker to that and keep runc enabled

The problem vagrant is using ubuntu16 and installs latest stable, for example 18.03.0-ce, so the invoker image would need to be rebuild with the matching version.
If you are trying and testing runc related things, then is better to experiment with the latest stable release.
I added docs for this special case, for an user to experiment with runc.
The version we recommend and we currently test in Travis and have it pinned in the invoker is an older version 1.12

I was able to figure out how to increase the size of the disk with a vagrant plugin.
@mdeuser check if the plugin works on windows.

@mdeuser can you test this branch one more time.
I remove the disksize plugin since you reported problems running on Windows.
I just a got a clean deploy
    default: Swagger UI URL: https://<IP_ADDRESS>/api/v1/docs/index.html?url=/api/v1/api-docs
    default: ++ echo 'Swagger UI URL: https://<IP_ADDRESS>/api/v1/docs/index.html?url=/api/v1/api-docs'
    default: ++ wsk action invoke /whisk.system/utils/echo -p message hello --result
    default: {
    default:     "message": "hello"
    default: }
    default: +++ date
    default: ++ echo 'Wed Apr 18 01:21:10 UTC 2018: build-deploy-end'

skipping the runtimes I was able to not run out of disk space
vagrant@ubuntu-xenial:~$ df -h
Filesystem              Size  Used Avail Use% Mounted on
udev                    2.0G     0  2.0G   0% /dev
tmpfs                   396M  6.3M  389M   2% /run
/dev/sda1               9.7G  5.4G  4.3G  56% /
tmpfs                   2.0G     0  2.0G   0% /dev/shm
tmpfs                   5.0M     0  5.0M   0% /run/lock
tmpfs                   2.0G     0  2.0G   0% /sys/fs/cgroup
tmpfs                   396M     0  396M   0% /run/user/1000
vagrant                 466G  412G   55G  89% /vagrant
home_vagrant_openwhisk  466G  412G   55G  89% /home/vagrant/openwhisk

@csantanapr Are the steps to change box size similar to one posted by @neerajmangal here. In that case can we at least leave the config in Vagrantfile and just document steps on how to increase the size

@chetanmeh not the same procedure
At least in OSX I was able to use a vagrant plugin to which requires one line in Vagrant :-) tonset new size of disk like 50GB
But this didn’t work on Windows :-(  will need to dig with @mdeuser on why it doesn’t work
Another option is to create a second disk and configure docker to use it for images, but I have not figure out how easy to do that.
My intention is to merge this PR to unblock folks and deal with building all images later if required.

@mdeuser updated the readme with your comments.


Several commits seem to be reducing completion performance recently

I'm looking at this golem benchmark (das-flutter-completion-3): https://golem.corp.goog/Benchmark?repository=flutter-analyze&benchmark=das-flutter-completion-3
I see something like 3 points of reduced performance. I'll bisect and confirm and post here.

Good news, the first one can just be attributed to https://github.com/flutter/flutter/commit/4abe6fdafae3a2712487b33756b55fe1cfa76d3a#diff-7cab3683f480afefc47ecbb454f7e4c238a09cd639e45785dee2b923c22b63cb - in this commit, the test in which we perform completion grows by 236 lines. 🤷

After some analysis, I think these are each organic bumps, unrelated to analyzer commits.


@mobicycle do you have a question? It seems like you have pasted irrelevant ChatGPT output into the issue description.

          > @mobicycle do you have a question? It seems like you have pasted irrelevant ChatGPT output into the issue description.

As a paying customer, I expect a certain level of professionalism and courtesy, something neither @rozenmd nor @kodster28 have. This type of service appears to be typical of Cloudflare staff. I request to hear from a manager to file a formal complaint about the poor customer service outlined in the response above. If my concerns are not addressed satisfactorily, I will be compelled to cancel all my paid subscriptions.
Originally posted by @mobicycle in https://github.com/cloudflare/cloudflare-docs/issues/11911#issuecomment-1823266925

Can you elaborate on your question? Your previous issue did not specify any concern or question.
(I am a manager on the team, btw.)

Howdy @mobicyle, going to close this out as we haven't heard anything from you.
Feel free to re-open with more details.


need to support MPI_IN_PLACE in IALLTOALLW

This fails on the NULL.
    MPI_Ialltoallw(MPI_IN_PLACE, NULL, NULL, NULL, rbuf, rcounts, rdispls, types, comm, &req);
    MPI_Wait(&req, MPI_STATUS_IGNORE);

This is partially done.  If sendtypes is NULL, it isn't touched.  Not sufficient for perverse (adversarial?) inputs but fine for reasonable usage.
Need to do the cookie trick with buffers associated with requests to free on completion.

should be fixed except for the free on request completion part


Cohesion enhancement on Controllers, Services and utils

Codecov Report

Merging #1 (1558f49) into master (37ca45a) will increase coverage by 18.46%.
The diff coverage is 25.97%.

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


Subtitle generation is not working properly.

Which OS are you using?

OS: [e.g. iOS or Windows.. If you are using Google Colab, just Colab.]

windows 11
After a long-awaited update, I attempted to generate Chinese subtitles. As time goes on, I'm encountering an issue where subtitles are generated as numbers only from a certain point.

1286
00:25:55,660 --> 00:25:56,660
90
1287
00:25:56,660 --> 00:25:57,660
90
1288
00:25:57,660 --> 00:25:58,660
90
1289
00:25:58,660 --> 00:25:59,660
90
1290
00:25:59,660 --> 00:26:00,660
90

Hi, it seems like whisper hallucination.

Related discussion

https://github.com/openai/whisper/discussions/679

Many of possible solutions are discussed here.
You can try

Set condition_on_previous_text to False
Tune no_speech_threshold and log_probability_threshold values.

You can adjust these parameters in the "Advanced Parameters" tab of the WebUI.
Setting condition_on_previous_text to False would make texts less consistent about the context, but it will help to whisper to escape the "loop of failures" that you experienced.
no_speech_threshold and log_probability_threshold are the parameters that define how whisper will be "sensetive" to the small sounds. For example, in your case, this might happen because whisper is too sensitive to small sounds.
Lowering both no_speech_threshold and log_probability_threshold would make whisper insensitive to the small sounds.

Increasing temperature also solves this

I have also recently encountered the same hallucination issue in Korean. Even when using the vad_filter and adjusting the Advanced Parameters comprehensively, the same hallucination occurs after a certain point.
In my case, I found that changing the Model to large-v2 prevents hallucinations, although the text generation quality decreases.
Previously, there were no issues even when using large-v3, so I believe there is definitely a problem with whisper.

@jhj0517 Your efforts are always appreciated. Thank you for your feedback.

Silero VAD Filter is added in #153.
Open the "Advanced Parameters" tab in the WebUI, and check "Enable Silero VAD Filter". If the hallucination still occurs, uncheck "Condition On Previous Text".
If the hallucination still exists with the above methods, please let me know.

@jhj0517
The above method still causes problems. I don't have a problem with V2, but I have a problem with V3. I'm extracting Chinese subtitles.

@lgs777 Thanks for pointing this out, I think this is a pretty notable issue.
I'll just update the default model to large-v2 for now.

Thank you for all your help.
I was having problems with hallucination when exporting Japanese conversations, but changing to large-v2 greatly improved the problem.
I still had a little hallucination, but raising Temperrature to 0.2 eliminated it.

I just added BGM separation pre processing to reduce such hallucinations in #267.

It gave me better result on my test, please feel free to share the your result.

I was getting unusable translation results before turning on BGM separation and Silero VAD. It should be clarified that they are meant for this. A hint in the UI next to the translation toggle would go a VERY long way towards users understanding what these actually do.

@mark-wd Thanks for pointing that out. I updated some labels for clearer use of submodels in #308.
If anyone has suggestions for better clarification, I'd appreciate it.


Bug: Highlights - Query params are not being reset on resetting repository filters, URL not encoded right

Describe the bug

When we click on a filter again, the URL should should not contain the old filter.
The URL is not properly encoded. All query param slashes should be replaced with %2F% in the URL. This needs to be handled using URLSearchParams.

Steps to reproduce

Go to /feed
Click on one of the repository filters on the right hand side of the screen.
Observe the URL while clicking it again and again.
The query param does not actually clear and is not URL encoded.

Browsers
Chrome
Additional context (Is this in dev or production?)
dev and production
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

.take

Please let me know when I may start working on this issue. I have assigned to myself.


String Duration Support for context.sleep and context.waitForEvent timeout

Right now, the only way to pass durations is by number as below.
await context.sleep("wait-until-welcome-email", 60 * 60 * 24);

With this PR, we're introducing a Duration type, where interface allows to pass durations as 1d, 90m etc. as below
await context.sleep("wait-until-welcome-email", "90m");

can we also use string duration in here?
We can also add a similar test for waitForEvent and use a string duratino


Lodash assign for react native support

Love this library for use of Polyglot with web development. I'm working on a react native project with shared items to a react.js project and wanted to use this library for i18n. However, Object.assign in React-Native does not allow for enumerable keys to be prototypes on objects, and they are for Polygots.
The solution is to change the {... p, ...} (which is Object.assign) to use Lodash's assign instead.

I'll look into fixing the best tests... As of now, Jest isn't detecting that anything is found.

Changes made, passes all unit tests now. Still works in my app too.
Hope other React-Native users find use for this.

Nice work @rjerue, You are now part of redux-polyglot contributors.
Anyway, I just released<EMAIL_ADDRESS>on latest npm dist-tag.


fixed ms.author

Docs Build status updates of commit 8b539bf:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/case-study-representing-humans-in-mixed-reality.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[Android] New received images and other attachments intermittently do not show inside a conversation

Latest server and Android builds, issue occurs frequently but not every single time.
When the issue occurs:
A new image or attachment is received by BlueBubbles and a notification is received. Entering the conversation of the received attachment will not show the new attachment. Sliding left and holding to view message timestamps while in the conversation shows the correct timestamp of the attachment that should be showing below the last received message, but no attachment or message content is shown beside it. To get the attachment to show properly, the app must be fully closed in multitasking and reopened.

Fixed internally, typically only happens with heic images


Test manifest and helm charts

Inspektor Gadget provides a yaml manifest and helm charts to deploy it without using kubectl-gadget. Unfortunately they can get broken and go unnoticed as kubectl-gadget deploy continues to work, see #3389.
Implementation Options
It's not needed to run the integration tests, just checking that Inspektor Gadget can be deployed by using this two methods is enough.
Tasks

[ ] Helm
[ ] Yaml manifest

@andoriyaprashant done! Please let me know if you need any help with it. We currently lint/package the charts in CI so that can be a good starting point!


pkg/kubelet: move the capabilities related code to util.go

/cc @ArtfulCoder

Moved the capabilities related code to util.go. Other code is there.

dropped the kubelet receiver from canRunPod method.

LGTM

LGTM

LGTM


Setup Netlify CMS for blogging

Need to setup the Netlify CMS so we can do WYSIWYG based blog posts.
Docs:
https://www.netlifycms.org/docs/configuration-options/
https://www.netlifycms.org/docs/gatsby/

Install complete. Just need to wait for a team member to authorize the Netlify Auth stuff to our Github, then I can start more testing.

We now have a functional blog! I manually moved the last three blog posts into the new CMS and then built the blog index page and blog page.  I haven't tested every single thing, but it's done enough to close this issue.  As new items come up, we can open new specific issues.


errorMessage field too short

Hi,
Just wanted to say that I genuinely appreciate this package.
Regarding the errorMessage, it is critical for ordinary people entering their password to be advised on what should be included.  I was hoping to show the below in the errorMessage field but the length is too long and is only partially shown by errorMessage.  Is there a way to adjust the errorMessage field size to allow the entire text message below to appear?  Thanks in advance.
"Password should be 8 characters long with at least 1 Uppercase letter, 1 lowercase letter, 1 special character eg. . * @ # $, and 1 number."

Thanks for the issue yes that's possible we simply need to add a new  property that inputs to the errorMaxLines property of the original TextField ,I will update here about the progress

Hello @marcusaureliusbrutus I have updated the package passwordfield 0.0.7 is here  with two new feature errorStyle and errorMaxLines

I hope this resolves your issue I will be closing this issue for now assuming its resolved,feel free to reopen if you think it's not.


python3Packages.python-yate: init at 0.4.1

Description of changes
Release notes: https://github.com/eventphone/python-yate/releases/tag/v0.4.1

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 254410 run on x86_64-linux 1

4 packages built:

python310Packages.python-yate
python310Packages.python-yate.dist
python311Packages.python-yate
python311Packages.python-yate.dist


"az consumption usage list" command cannot be run in parallel

Related command

az consumption usage list
Describe the bug
The command can't be parallelized. You essentially have to run it serially (well, sometimes I can do 2, maybe 3 at once... woohoo!)  Combined with the very long execution time (https://github.com/Azure/azure-cli/issues/23204), this means I can't run daily reports.  All other az-cli commands support at least a tiny bit of parallelization, most easily handling  fifty or more jobs at a time.
Running this sub command in parallel (20 at a time) over our 1059 subscriptions returned:
63 successful
23 Returned no output (perhaps succeeded, perhaps not, haven't investigated)
973 failures
Failure breakdown according to the ERROR messages given by az-cli -
926 ERROR: (429) Too many requests. Please retry after 60 seconds.
 44 ERROR: (SubscriptionNotFound)
  2 ERROR: (401) Unauthorized
  1 ERROR: (400) Subscription scope usage is not supported for current api version.

The 44 that weren't found... a bit odd, since this was run immediately after running az account list
to get the subscriptions in the tenant.
On the bright side the failed run only took a bit over 4 minutes to complete.
To Reproduce
Here I'm using the rush command to parallelize shell commands, as I do with all the other az-cli sub commands  I use (rush is at https://github.com/shenwei356/rush); you can run from 1-N jobs in parallel at a time; when it finishes one job it starts another. Here rush is reading from standard input... the file "subs.txt", which has, one per line, the IDs of the > 1000 subscriptions we run in this tenant.)
First, the simple "costs.sh" shell script that will be run in parallel; it puts the result in /tmp -
start=$(gdate -d "last month" "+%Y-%m-01")
end=$(gdate -d "$(date +%Y/%m/01) - 1 day 00:00" +"%Y-%m-%d")
date_filter="-s $start -e $end"

az consumption usage list -a -m $date_filter --subscription "$1" > /tmp/$1.res

The output of running the above script under rush is attached (gitout.txt) the -j option below specifies the number of jobs to run in parallel (20 here.)
(see attachment)
gitout.txt
Expected behavior
The command should be able to be run in parallel from a single endpoint and return the results within a reasonable time frame. The cost web UI, which has the costs sliced and diced, actually loads pretty quickly.
Environment summary

Installed via brew on a macbook and iMac, currently running MacOS 10.16 and/or 10.15.7, but this has been slow for literally the last few years no matter what the OS or az-cli versions. Here's the macbook version, which was used for the above test and output -
`az version
{
"azure-cli": "2.23.0",
"azure-cli-core": "2.23.0",
"azure-cli-telemetry": "1.0.6",
"extensions": {
"azure-devops": "0.18.0",
"databricks": "0.7.2",
"kusto": "0.2.0",
"log-analytics-solution": "0.1.1",
"resource-graph": "1.1.0",
"storage-preview": "0.7.2"
}
}

**Additional context**

<!--- Add any other context about the problem here. --->

The `consumption` sub command has been in preview for years. Come on. From the consumptio:
"WARNING: Command group 'consumption' is in preview and under development."

I can send a full debug log of a run (about 500-800MB) if you want (I can point to a URI or whatever, if you mail me.)

route to CXP team

See: https://github.com/Azure/azure-cli/issues/11891.


driver: esp32: fix double promotions

Constants will get defined as doubles if missing the f at the end

These look fine. I note that there are more implicit double conversions in the i2c and in other temp sensor driver bits (for esp32c3 at least). There are also lots in the test and example code, but I'm not sure Zephyr can even build that?
Are the computations in components/esp_rom/patches relevant? components/bt/host/bluedroid/external/sbc/plc?
How can we make sure we've got all of this fixed? Seems like piles of code here and I doubt much of it gets tested with the Zephyr test suite...

@keith-packard, there are lots of unused code in there at this moment. We have been refactoring hal_espressif to bring latest esp-idf changes and also cleaning up unused stuff (by unused I mean in Zephyr level).


teyjus: actually update to 2.1

My automatic update of teyjus did not properly update it, as reported in https://github.com/NixOS/nixpkgs/pull/36702#issuecomment-373307935
Note, the binaries in this release still report 2.0-b2
https://github.com/teyjus/teyjus/issues/117 tracks asking upstream to
release versions with new version numbers.
cc @joachifm @vbgl @bcdarwin for review

@GrahamcOfBorg build teyjus

No attempt on x86_64-darwin
The following builds were skipped because they don't evaluate on x86_64-darwin: teyjus
No log is available.

Success on x86_64-linux (full log)
Attempted: teyjus
Partial log (click to expand)
shrinking /nix/store/2gs9lkp3h705cbn2vnqs4x0091zjyglz-teyjus-2.1/bin/tjcc
shrinking /nix/store/2gs9lkp3h705cbn2vnqs4x0091zjyglz-teyjus-2.1/bin/tjlink
shrinking /nix/store/2gs9lkp3h705cbn2vnqs4x0091zjyglz-teyjus-2.1/bin/tjdis
shrinking /nix/store/2gs9lkp3h705cbn2vnqs4x0091zjyglz-teyjus-2.1/bin/tjdepend
shrinking /nix/store/2gs9lkp3h705cbn2vnqs4x0091zjyglz-teyjus-2.1/bin/tjparse
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/2gs9lkp3h705cbn2vnqs4x0091zjyglz-teyjus-2.1/bin
patching script interpreter paths in /nix/store/2gs9lkp3h705cbn2vnqs4x0091zjyglz-teyjus-2.1
checking for references to /build in /nix/store/2gs9lkp3h705cbn2vnqs4x0091zjyglz-teyjus-2.1...
/nix/store/2gs9lkp3h705cbn2vnqs4x0091zjyglz-teyjus-2.1

Success on aarch64-linux (full log)
Attempted: teyjus
Partial log (click to expand)
shrinking /nix/store/6gfn181fdcdmmmzqb626r63ab76i14mp-teyjus-2.1/bin/tjparse
shrinking /nix/store/6gfn181fdcdmmmzqb626r63ab76i14mp-teyjus-2.1/bin/tjlink
shrinking /nix/store/6gfn181fdcdmmmzqb626r63ab76i14mp-teyjus-2.1/bin/tjdis
shrinking /nix/store/6gfn181fdcdmmmzqb626r63ab76i14mp-teyjus-2.1/bin/tjdepend
shrinking /nix/store/6gfn181fdcdmmmzqb626r63ab76i14mp-teyjus-2.1/bin/tjcc
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/6gfn181fdcdmmmzqb626r63ab76i14mp-teyjus-2.1/bin
patching script interpreter paths in /nix/store/6gfn181fdcdmmmzqb626r63ab76i14mp-teyjus-2.1
checking for references to /build in /nix/store/6gfn181fdcdmmmzqb626r63ab76i14mp-teyjus-2.1...
/nix/store/6gfn181fdcdmmmzqb626r63ab76i14mp-teyjus-2.1

No attempt on x86_64-darwin
The following builds were skipped because they don't evaluate on x86_64-darwin: teyjus
No log is available.

Success on x86_64-linux (full log)
Attempted: teyjus
Partial log (click to expand)
/nix/store/2gs9lkp3h705cbn2vnqs4x0091zjyglz-teyjus-2.1

Success on aarch64-linux (full log)
Attempted: teyjus
Partial log (click to expand)
/nix/store/6gfn181fdcdmmmzqb626r63ab76i14mp-teyjus-2.1


Animated FBX model that worked fine in v3.2.3 is broken in v3.3

Godot version: 3.3
OS/device including version: macOS Catalina
Issue description:
I'm trying to import an animated fbx model. The FBX file contains a simple rigged and animated character model with a walk cycle. The animation works fine in godot 3.2.3, but it is broken in 3.3 - the character's legs are imported incorrectly, with upper leg part missing, and lower legs and feet animated incorrectly.
Steps to reproduce:

Download the FBX File.
Import it to godot (set the Root Scale to 100 to fix the scaling issue).
Click on the AnimationPlayer node, playback the animation, look at the character's legs. It will work fine in Godot 3.2.3, but will be broken in Godot 3.3.

Can you guys help me out? How can I fix the issue? Is there any workaround to this?
Maybe there's some issue with the IK on the rig? I don't know, just a guess.
Edit:
Oh, and another problem - it looks like in 3.3, every node has a keyframe on it every frame. And in 3.2.3, the keyframes exist only on animated nodes (the way it should be).
Edit2:
The difference seems to be that some of the nodes in 3.3 (specifically left_upper_leg and right_upper_leg) are imported as "Skeleton" type nodes. In 3.2.3 these nodes are imported as "Spatial" nodes, that's why it works correctly.
Strangely enough, there are all kinds of bones in the character, but only the upper leg ones are imported as "Skeleton". I don't know what the difference is.
Is there any way that I can make godot import everything as "Spatial", maybe with an import script of some kind?
Edit 3:
Right clicking on the Skeleton nodes and doing Change Type > Spatial makes the character's upper legs reappear, but the animation is totally broken.

I think we ran into the same problem
Model animation is incorrect
Godot version:
This problem exists in both v3.3 and v4.0bate
operating system:
windows
Problem Description:
When playing FBX animation, all sides of the model are confused, which causes the animation to not play normally and the model displays abnormally.
Steps to reproduce:
Import FBX files with bones and animations;
Double-click spatial to view the content;
Try to click the AnimationPlayer node to play the animation;
---The model is displayed abnormally.
ps:
When importing FBX without bones and animation, everything is normal;
The problem seems to only appear when there are animations or poses that need to be presented;
So the problem probably lies in the process of how the bones of the model determine the changes in the model surface.
test.zip

Happens to me too. Model with animations that worked great in 3.x is scattered around in 4.x when no animation is playing. When I select one of the animations, it somewhat resembles the animation, but the end bones are messed up (in wrong positions)

If the model looks good in 3.3/3.4 and bad in master, it's an entirely different issue. Please open a separate issue for this.

This is probably fixed with the new FBX importer using FBX2glTF.
You can test it in Godot 4.0 alpha 6: https://godotengine.org/article/dev-snapshot-godot-4-0-alpha-6
This new importer might also be backported for the 3.x branch eventually.
If you still find issues with these models using the new importer, we suggest opening a new issue with details about the problems in Godot 4.0 alpha 6 or later.


Use 4 center pins in the IC for the GND pins

Couldn't you use the center 4 balls for GND and then put a via in the middle of them through to the ground plane? It would get you back another 4 pins....
See this diagram for example;

Center pins are purposely left out so that we and users can make use of the space for inductors on the redistribution layer.

Also, blind vias cost more money than your standard minimalist PCB.

@mithro : Okay, got it now, after out conversation.  I still suggest gauging the interest in the analog community for using the redistribution layer for inductors (although in the same comment I would point out that if the user does not touch the redistribution layer, then there can potentially be only one mask ever made for it, which would reduce the cost of the bump bonding in the long run).
Also:  Note that I have most ground returns on separate nets for purposes of noise isolation.  The three ground domains that I have connected together are already tied to a single bump.  So by adding four additional pads in the middle, you can only gain one additional pin for the user.

@mithro : Okay, got it now, after out conversation. I still suggest gauging the interest in the analog community for using the redistribution layer for inductors (although in the same comment I would point out that if the user does not touch the redistribution layer, then there can potentially be only one mask ever made for it, which would reduce the cost of the bump bonding in the long run).

This is expected. So basically, everyone would need to share that region.

@mithro :  This has been fixed in the documentation, and pushed.  I am waiting on some changes to the RTL from Mohamed Shalan's group before I add the additional user pin to the RTL.

User pin has been added to the RTL and all documentation (except for the provisional layout) has been updated, so I will close this ticket.


Fixes eerkunt/terraform-compliance#209

updated the Dockerfile to validate hash sum of downloaed terraform binary. added hashicorp public key to the repo. added passing of of hashicorp PGP public key during docker build to CI pipeline file

Pull Request Test Coverage Report for Build 782

0 of 0   changed or added relevant lines in 0 files are covered.
6 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.4%) to 67.899%

Files with Coverage Reduction
New Missed Lines
%

terraform_compliance/common/defaults.py
6
80.65%

Totals

Change from base Build 779:
0.4%

Covered Lines:
992

Relevant Lines:
1461

💛  - Coveralls

Great PR! Thanks a lot!


How to get list of metadata of class or properties or methods ?

Hello.
Could you provide a tutorial to access to metadata please ?
Thanks.

This test shows some examples: https://github.com/dart-lang/test_reflectable/blob/master/test/metadata_test.dart.

Thanks a lot


ARTEMIS-5074 Fix encoding of bytes properties as Binary in AMQPMessage

When a bytes property is added to an AMQPMessage and it is then reencoded it will fail without first wrapping the byte array in an AMQP binary as required by the ApplicationProperties section specification defined type allowances.

I've run these changes through CI a few times to validate no new failures


Use Proxy.getInvocationHandler instead of reflection

[ ] I have registered the PR changes.

Ⅰ. Describe what this PR did
Fix https://github.com/seata/seata/issues/5200
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

@a364176773 PTAL

Codecov Report

Merging #5201 (866f739) into develop (083c718) will not change coverage.
The diff coverage is 0.00%.

:exclamation: Current head 866f739 differs from pull request most recent head e752162. Consider uploading reports for the commit e752162 to get more accurate results

@@            Coverage Diff             @@
##             develop    #5201   +/-   ##
==========================================
  Coverage      48.47%   48.47%
- Complexity      4137     4138    +1
==========================================
  Files            743      743
  Lines          26517    26517
  Branches        3294     3294
==========================================
  Hits           12853    12853
- Misses         12263    12264    +1
+ Partials        1401     1400    -1

Impacted Files
Coverage Δ

...in/java/io/seata/spring/util/SpringProxyUtils.java
3.70% <0.00%> (ø)

...very/registry/zk/ZookeeperRegisterServiceImpl.java
61.02% <0.00%> (-0.74%)
:arrow_down:

...erver/storage/file/session/FileSessionManager.java
48.40% <0.00%> (+0.63%)
:arrow_up:


Remove uninstall delete directive from IntelliJ IDEA casks

Checklist

[x] The commit message includes the cask’s name and version.
[x] brew cask style --fix {{cask_file}} left no offenses.

None of Intellij IDEA product is linked into the /usr/local/bin during installation so they should be removed during uninstallation.

Move them to zap instead, then, rather than completely deleting them.

These links were never created by any of these casks, then You try to create them using

binary "#{appdir}/IntelliJ IDEA.app/Contents/MacOS/idea", target: 'idea'

the idea link will be broken and app will not start. You will get

2016-09-01 11:23:11.565 idea[58937:1245253] allVms required (null)
2016-09-01 11:23:11.570 idea[58937:1245253] Info.plist is corrupted, Absent JVMOptions key.

Moreover they are in conflict with each other because each of them is the same so potentially  uninstalling intellij-idea-eap will remove link from intellij-idea-ce.
That's why IMO it is better to delete them completely.

I'm not sure I understand: if, as you say, it's better to delete /usr/local/bin/idea, why would we remove that line from uninstall?

It is better to remove this line uninstall delete: '/usr/local/bin/idea' from all casks because:

none of this casks have ever create this link
if we try to create it, it will not work
each cask is removing the same link '/usr/local/bin/idea' so they are interfering with each other

Looks like these uninstall stanzas were added by @Jerzerak. The idea CLI also appears to be official, and may be automatically added through some setting in the IDE. @Jerzerak, is that the case? Does IntelliJ automatically link/copy the idea binary into /usr/local/bin?

IMO @Jerzerak added them by copy-and-paste.
The CLI is supported but idea does not automatically link/copy the idea binary into /usr/local/bin, there is some problem with luncher.
To test it, try to link idea manually by ln -s /Applications/IntelliJ\ IDEA.app/Contents/MacOS/idea /usr/local/bin/ and try to run it by idea, it will fail.

Funnily enough, I just found a /usr/local/bin/idea on my machine, and it wasn't a symlink. I've created a gist of it here: https://gist.github.com/jawshooah/0f24fabcc280c702a61be471557ebb10
I never manually installed this, so it must have been dropped there by the IDE. Let's leave the uninstall as it is.

@metyl @jawshooah I added that line to the uninstall stanza after monitoring what files were created upon installation and launch of several different IntelliJ IDEA versions and flavors (off the top of my head it was at least EAP, CE, & Ultimate). If we remove that line the applications eventually become confused as to where their config files are located and what version/flavor of the application they should be.
From what I could gather the file is created upon launch, if it is not already present, so there shouldn't be any downside to deleting it upon uninstallation as any other present IDEA versions/flavors should create a new /usr/local/bin/idea file automatically. I haven't tested this since I added the line so things might've changed but as long as things are working as intended I'd leave it be.

Thanks for the info, @Jerzerak. Looks like leaving the uninstall stanza alone is the right move.

I've check with current version of IntelliJ IDEA (2016.2.3) and latest EAP (2016.3 build 163.3512.7) and none of them is creating/copying anything as /usr/local/bin/idea. Can You guys check it? @jawshooah Yours /usr/local/bin/idea is for 2016.1.1 version with old location of --caskroom maybe this is the problem?

I've figured it out. You have to create this link manually in IntelliJ IDEA by Tools->Create Command-Line launcher... and specify the name of the script and the path where it should be created.


Correct intermittent fault in TLS test case

Depending on timing the connection state in testcase t_tls_1_3_cert_expired
is either to-be-closed or already closed.

Nice!
Isn't there a way to make this deterministic?

A sleep :)

Nice!
Isn't there a way to make this deterministic?

A sleep :)

Add a sleep then? :sleeping: In the makefile or in the test suite? timer:sleep(Milliseconds).

The error will arrive in the handle_info\3 for ssl_error, and this will trigger a disconnection. If there are queries outstanding these will be aborted using reply_all with {error, Reason} as code. If the query is a bit late it will get no_connection directly instead

The error will arrive in the handle_info\3 for ssl_error, and this will trigger a disconnection. If there are queries outstanding these will be aborted using reply_all with {error, Reason} as code. If the query is a bit late it will get no_connection directly instead

I understand. Let's keep it as it is now then. Let's merge! :100:

Nice!
Isn't there a way to make this deterministic?

A sleep :)

Add a sleep then?  In the makefile or in the test suite? timer:sleep(Milliseconds).

The sleep would be needed in the testcase, right before the query is triggered but after a connection has been setup. The testcase was created to test a query direct after the connect due to the different behaviour in TLS 1.3 compared to TLS 1.2. In 1.2 we get the error during the connect setup directly, and this testcase would expect the error to arrive just after the connection. Running locally this always triggered as in the testcase before, but I guess CI is a bit slower.


Feat/sol 342 enable custom provider Logos

Laying the base of theming the UI for providers:

documentation on how to create a brand theme in providers
adding logos for shopify and big commerce examples

This system is totally optional and the documentation makes it clear. this will be flexible enough to allow further theming systems coming from providers.

Logos implemented to test how well this would work (We can remove them, this is mainly for showing usage)

Shopify
bigcommerce

Todo:
fix the TS context types to work well

Will reopen not from my fork to simplify previews


Question : Filtering Audio with Miniaudio's built-in Filters?

Hi. I've been experimenting how can I implement 'Parametric EQ' or 'Peaking EQ' filter on flutter project.
After I found this package, I tried to make a parametric eq node but the filtered output audio was noticeably noisy.
I was then curious if I could utilize miniaudio's built-in peaking eq filter with 'coast_audio_miniaudio' package to go 'easier way'.
Is it possible to filter out audio with miniaudio's built-in filters?
P.S. The code below, is the 'Parametric EQ Node' I tried to make. I referenced several codes within this package, but I'm not sure whether it was done right.
I have tested the code with provided 'Music Player' Example, by connecting Parametric EQ Node before, and after FFT Node later.
The sound output was noticeably noisy on both Android Emulator and Physical Android Device with A12.
import 'dart:math';

import 'package:coast_audio/coast_audio.dart';

class ParametricEQNode extends AutoFormatSingleInoutNode with ProcessorNodeMixin, BypassNodeMixin {
  ParametricEQNode({
    required this.format,
    required this.centerFreq,
    required this.dbGain,
    required this.qFactor,
  });

  final AudioFormat format;
  double centerFreq;
  double dbGain;
  double qFactor;

  @override
  List<SampleFormat> get supportedSampleFormats => const [SampleFormat.float32];

  @override
  int process(AudioBuffer buffer) {

    final inputData = buffer.asFloat32ListView();

    // Following Coefficients and Transfer Functions are adapted from Audio-EQ-Cookbook.
    // more at https://www.w3.org/TR/audio-eq-cookbook/

    // Calculate filter coefficients
    double w0 = 2 * pi * centerFreq / format.sampleRate;
    double alpha = sin(w0) / (2 * qFactor);
    double A = pow(10.0, (dbGain / 40)).toDouble();

    double b0 = 1 + (alpha * A);
    double b1 = -2 * cos(w0);
    double b2 = 1 - (alpha * A);
    double a0 = 1 + (alpha / A);
    double a1 = b1;
    double a2 = 1 - (alpha / A);

    double x1 = 0;
    double x2 = 0;
    double y1 = 0;
    double y2 = 0;

    // Process each sample
    for(int frame = 0; frame < buffer.sizeInFrames; frame++) {
      for (var channel = 0; format.channels > channel; channel++) {
        final inputBufferIndex = (frame * format.channels) + channel;

        double x = inputData[inputBufferIndex];

        // Calculate output sample
        double y = (b0/a0)*x + (b1/a0)*x1 + (b2/a0)*x2 - (a1/a0)*y1 - (a2/a0)*y2;

        // Update the state variables
        x2 = x1;
        x1 = x;
        y2 = y1;
        y1 = y;

        // Update the buffer data
        inputData[inputBufferIndex] = y;
      }
    }

    return buffer.sizeInFrames;
  }
}

I think you should save your state variables(x1, x2, y1, y2) in your class's instance to reuse them like this.
import 'dart:math';

import 'package:coast_audio/coast_audio.dart';

class ParametricEQNode extends AutoFormatSingleInoutNode with ProcessorNodeMixin, BypassNodeMixin {
  ParametricEQNode({
    required this.format,
    required this.centerFreq,
    required this.dbGain,
    required this.qFactor,
  });

  final AudioFormat format;
  double centerFreq;
  double dbGain;
  double qFactor;

  double _x1 = 0;
  double _x2 = 0;
  double _y1 = 0;
  double _y2 = 0;

  @override
  List<SampleFormat> get supportedSampleFormats => const [SampleFormat.float32];

  @override
  int process(AudioBuffer buffer) {
    final inputData = buffer.asFloat32ListView();

    // Following Coefficients and Transfer Functions are adapted from Audio-EQ-Cookbook.
    // more at https://www.w3.org/TR/audio-eq-cookbook/

    // Calculate filter coefficients
    double w0 = 2 * pi * centerFreq / format.sampleRate;
    double alpha = sin(w0) / (2 * qFactor);
    double A = pow(10.0, (dbGain / 40)).toDouble();

    double b0 = 1 + (alpha * A);
    double b1 = -2 * cos(w0);
    double b2 = 1 - (alpha * A);
    double a0 = 1 + (alpha / A);
    double a1 = b1;
    double a2 = 1 - (alpha / A);

    // Process each sample
    for (int frame = 0; frame < buffer.sizeInFrames; frame++) {
      for (var channel = 0; format.channels > channel; channel++) {
        final inputBufferIndex = (frame * format.channels) + channel;

        double x = inputData[inputBufferIndex];

        // Calculate output sample
        double y = (b0 / a0) * x + (b1 / a0) * _x1 + (b2 / a0) * _x2 - (a1 / a0) * _y1 - (a2 / a0) * _y2;

        // Update the state variables
        _x2 = _x1;
        _x1 = x;
        _y2 = _y1;
        _y1 = y;

        // Update the buffer data
        inputData[inputBufferIndex] = y;
      }
    }

    return buffer.sizeInFrames;
  }
}

BTW

I was then curious if I could utilize miniaudio's built-in peaking eq filter with 'coast_audio_miniaudio' package to go 'easier way'.
Is it possible to filter out audio with miniaudio's built-in filters?

I haven't implemented miniaudio's filters so you need to fork this repo and implement them inside the mabridge.

Simply moving the state variables as you said, solved the noisy output!
I guess I can use this node without using miniaudio built-in filters.
Thanks for sharing your valuable insight!


Request for CSn version tables

File: cs-engine/1.12/index.md
File: cs-engine/1.13/index.md
For the offline install instructions, there are tables for the minor releases but there are not tables for the individual csn release versions.  Since CS can have multiple releases for a bugfix version number, it maybe worthwhile to have the individual versions available.
For example, there isn't a table to get the 1.13.1-cs1 version anymore now that 1.13-1-cs2 has been released.

@cpuguy83 WDYT? We decided to only have it be the latest, I thought.
This can't be for the hackathon as these directories are not browseable without access to the S3 buckets themselves.

Seems a bit excessive to me.

Can @andrewhsu work his magic and make the buckets browseable?

Looks like we decided again not to do this.


[Schema Registry] Pin dependency version

pin 1.0.0b1 in avroserializer

/azp run python - schemaregistry - tests


docs: add custom labels for DonutChart

Closes #2683
Description
Add documentation about custom labels for DonutChart.

Is this a breaking change (Yes/No):
No

@hirotomoyamada  Please translate into English.


Upstream CI failures

We have several failures right now in Upstream CI.  It's looks like they're coming from Matplotlib updates.
In particular the following examples are impacted:

NCL_xy_16.py
NCL_overlay_11b.py
NCL_hov_3.py
NCL_coneff_11.py

Log
Unexpected failing examples (4):
../Gallery/XY/NCL_xy_16.py failed leaving traceback:

Traceback (most recent call last):
  File "/home/runner/work/geocat-examples/geocat-examples/Gallery/XY/NCL_xy_16.py", line 102, in <module>
    plt.legend(handles,
  File "/home/runner/micromamba/envs/geocat-examples/lib/python3.11/site-packages/matplotlib/pyplot.py", line 3593, in legend
    return gca().legend(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/runner/micromamba/envs/geocat-examples/lib/python3.11/site-packages/matplotlib/axes/_axes.py", line 335, in legend
    self.legend_ = mlegend.Legend(self, handles, labels, **kwargs)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/runner/micromamba/envs/geocat-examples/lib/python3.11/site-packages/matplotlib/legend.py", line 462, in __init__
    if len(handles) < 2:
       ^^^^^^^^^^^^
TypeError: object of type 'list_reverseiterator' has no len()

../Gallery/Overlays/NCL_overlay_11b.py failed leaving traceback:

Traceback (most recent call last):
  File "/home/runner/work/geocat-examples/geocat-examples/Gallery/Overlays/NCL_overlay_11b.py", line 185, in <module>
    for col in cf.collections:
               ^^^^^^^^^^^^^^
AttributeError: 'GeoContourSet' object has no attribute 'collections'

../Gallery/Contours/NCL_hov_3.py failed leaving traceback:

Traceback (most recent call last):
  File "/home/runner/work/geocat-examples/geocat-examples/Gallery/Contours/NCL_hov_3.py", line 52, in <module>
    for i, collection in enumerate(cf.collections):
                                   ^^^^^^^^^^^^^^
AttributeError: 'QuadContourSet' object has no attribute 'collections'

../Gallery/Contours/NCL_coneff_11.py failed leaving traceback:

Traceback (most recent call last):
  File "/home/runner/work/geocat-examples/geocat-examples/Gallery/Contours/NCL_coneff_11.py", line 62, in <module>
    for i, collection in enumerate(p.collections):
                                   ^^^^^^^^^^^^^
AttributeError: 'QuadContourSet' object has no attribute 'collections'

I suspect it's because of these changes:

ContourSet moving to a single Collection - https://matplotlib.org/devdocs/api/next_api_changes/removals/28767-REC.html
A deprecation or something related to this feature- https://github.com/matplotlib/matplotlib/pull/24759

This is blocked while we wait for the Matplotlib fix for different colored hatching to be backported to v3.9.x.

It looks like like I missed the usage of collections in NCL_hov_3.py here:
https://github.com/NCAR/geocat-examples/blob/1431c482ce1d451c412c0799b957f07e6a781782/Gallery/Contours/NCL_hov_3.py#L77
That can get fixed along with the hatching and linewidth items below after the Matplotlib 3.9.3 release
https://github.com/NCAR/geocat-examples/blob/1431c482ce1d451c412c0799b957f07e6a781782/Gallery/Contours/NCL_coneff_11.py#L62-L64

Looks like the matplotlib 3.9.3 release is out 🎉 so we should be able to more easily / robustly fix the rest of this.


Document the use of operator <

@jashkenas I inserted a new "Notes" section between the links and the changelog. For now it only contains the note on operator <. Maybe that note is too long. Maybe the references are too short.
Implements #2846
Closes #2042
Closes #1834
Closes #1768

I love it, but it is a bit much 😉
Instead of such an excellent treatise on < and NaN behavior, perhaps try trimming it down to a version that simply briefly mentions that non-comparable JS values have undefined sort order behavior, perhaps with a link to good external docs on the matter, and then move on quickly to a more practical discussion about how to fix your sorting problems: By defining a custom comparator that allows you to specify a sub-key, or compute a value, by which to sort.

I do get carried away sometimes. 😅
Better now?

Looks great. The one thing I might consider adding would be what I consider the most common case of difficulties sorting in JS: sorting an array of objects or class instances. Generally, there’s a single property on the object, or a method to call, to produce the desired value for the sort iterator — but that’s not obvious to beginners!
It also makes for a good example snippet.

I agree there is a value in a general "sorting in Underscore for beginners" type of note. However, when writing this note I specifically set out to address issues like #1768, which I suspect people still run into after they have already figured out the property shorthand thing. After all, a property that is missing in some objects in a collection is a common way in which people may end up comparing undefined. The documentation of _.sortBy et al. already mentions the option to use an iteratee, anyway.
I think a general introduction to sorting for beginners would require a different approach. I'm not opposed to changing the note in order to suit this purpose, but I want to concentrate on other things right now. Given your approval, I'll just merge this for now. We can revise it at any time.
I have however added a link to the _.iteratee section to give a bit more of a hint of the available options.


Reorganize transpiler modules in API docs

Closes https://github.com/Qiskit/documentation/issues/1240 and partially addresses https://github.com/Qiskit/documentation/issues/942 for runtime.
Reduces transpiler modules
Before:

module
status

qiskit.transpiler
only links to passes

qiskit.transpiler.passes
only links to basis and scheduling

qiskit.transpiler.passes.basis
empty, should link to ConvertIdToDelay but doesn't

qiskit.transpiler.passes.scheduling
useful

Now:

module
status

qiskit.transpiler.passes
links to ConvertIdToDelay and scheduling

qiskit.transpiler.passes.scheduling
useful

This saves the user clicks. Now each page is useful enough to stand on its own.
This is what the index page now looks like:

This change assumes that we will not add new modules to qiskit.transpiler for some time, such as qiskit.transpiler.my_module. If we do eventually add a new module, then we can easily revert transpiler.rst to point to qiskit_ibm_runtime.transpiler again.
This change also assumes that we won't lots of new APIs to qiskit.transpiler.passes.basis. If we do, we would want to consider going back to qiskit.transpiler.passes.basis being its own standalone page. We can do that easily if necessary in the future.
Improves header hierarchy
Options
Before, we were using bold text to act like headers. Instead, we should be using proper headers for styling, SEO, and accessibility (screen readers).
qiskit_ibm_runtime.transpiler.passes.scheduling
This PR moves the classes list to be higher in the file and fixes some of the other headers to higher heading levels.
Removes unused docstrings
The utils module is not exposed, and Kevin said it should not be since it's internal. So I deleted its docstring to make this more clear.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

3 of 3 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.704%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6277

Relevant Lines:
7499

💛  - Coveralls


openntpd: fix constraints feature on NixOS

Motivation for this change
The OpenNTPD constraints feature requires a valid chain of SSL
certificates, but the default path in openntpd didn't match the one in
NixOS.
Unfortunately the configured certificate path becomes hardcoded into the
binary, so this feature will likely still fail on other
distributions/operating systems, unless the path coincides with the
NixOS path or the user sets up a symlink.
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

also backported to 17.09.


Remove :backoff dependency

Closes #19.
The backoff API is made slightly clearer (what is :normal?), a new backoff type has been added (:rand) and there are TODOs to remove backwards compatible code in 1.0.
However perhaps we should move this to Connection as it is likely to be useful there?

Wow that test failure, maybe we should remove +T 9

:shipit:


Render only top level categories in primary navigation menu

Support for multi-level dropdowns was removed from Bootstrap 4, with the justification being that these are difficult to use on mobile (and BS4 was designed with a mobile-first approach). Our primary navigation template got a bit mangled during the port to BS4 as a result - resulting in 1st and 2nd level categories all being rendered in a flat list.
This changes the primary navigation only to render the top level categories. I think this is fine given that the menu is already inside a dropdown, and that the full category tree is available on search/category pages.

In fact, support for the submenu dropdowns was removed in Bootstrap 3, not 4!


After upgrade to js-controller 3.3.x

Warnings in Log:

gartenbewaesserung.0
2021-04-30 12:29:29.017
warn
Read-only state "gartenbewaesserung.0.status.startAbendTimestamp" has been written without ack-flag with value "1619808776"

gartenbewaesserung.0
2021-04-30 12:29:29.016
warn
Read-only state "gartenbewaesserung.0.status.startAbend" has been written without ack-flag with value "Fri Apr 30 2021 20:52:56 GMT+0200"

gartenbewaesserung.0
2021-04-30 12:29:29.015
warn
Read-only state "gartenbewaesserung.0.status.startMorgenTimestamp" has been written without ack-flag with value "1619757309"

gartenbewaesserung.0
2021-04-30 12:29:29.014
warn
Read-only state "gartenbewaesserung.0.status.startMorgen" has been written without ack-flag with value "Fri Apr 30 2021 06:35:09 GMT+0200"

gartenbewaesserung.0
2021-04-30 12:29:29.012
warn
Read-only state "gartenbewaesserung.0.status.sonnenuntergang" has been written without ack-flag with value "20:51:56 GMT+0200 (GMT+02:00)"

gartenbewaesserung.0
2021-04-30 12:29:29.010
warn
Read-only state "gartenbewaesserung.0.status.sonnenaufgang" has been written without ack-flag with value "06:05:09 GMT+0200 (GMT+02:00)"

Trotz der letzten Version aus dem GIT 0.0.8 und löschen des Objekt Baum, nach wie vor noch State Meldungen nach einem Instanz Neustart und zwischendurch immer mal wieder "ack-flg" Fehler.
`

gartenbewaesserung.0
2021-07-18 08:56:57.873
warn
(1460) Read-only state "gartenbewaesserung.0.status.startAbendTimestamp" has been written without ack-flag with value "1626634052"

gartenbewaesserung.0
2021-07-18 08:56:57.824
info
(1460) Stop ventil4

gartenbewaesserung.0
2021-07-18 08:56:57.823
info
(1460) Stop ventil2

gartenbewaesserung.0
2021-07-18 08:56:57.823
info
(1460) Stop ventil1

gartenbewaesserung.0
2021-07-18 08:56:57.822
info
(1460) Bewässerung stop

gartenbewaesserung.0
2021-07-18 08:56:57.822
info
(1460) Stop ventil4

gartenbewaesserung.0
2021-07-18 08:56:57.821
info
(1460) Stop ventil2

gartenbewaesserung.0
2021-07-18 08:56:57.819
info
(1460) Stop ventil1

gartenbewaesserung.0
2021-07-18 08:56:57.656
info
(1460) State value to set for "gartenbewaesserung.0.config.ventil10.feuchtigkeit_tresh" has to be type "number" but received type "string"

gartenbewaesserung.0
2021-07-18 08:56:57.655
info
(1460) State value to set for "gartenbewaesserung.0.config.ventil9.feuchtigkeit_tresh" has to be type "number" but received type "string"

gartenbewaesserung.0
2021-07-18 08:56:57.654
info
(1460) State value to set for "gartenbewaesserung.0.config.ventil8.feuchtigkeit_tresh" has to be type "number" but received type "string"

gartenbewaesserung.0
2021-07-18 08:56:57.653
info
(1460) State value to set for "gartenbewaesserung.0.config.ventil7.feuchtigkeit_tresh" has to be type "number" but received type "string"

gartenbewaesserung.0
2021-07-18 08:56:57.575
info
(1460) State value to set for "gartenbewaesserung.0.config.ventil10.dauer_in_state_mult" has to be type "number" but received type "string"

gartenbewaesserung.0
2021-07-18 08:56:57.574
info
(1460) State value to set for "gartenbewaesserung.0.config.ventil9.dauer_in_state_mult" has to be type "number" but received type "string"

gartenbewaesserung.0
2021-07-18 08:56:57.573
info
(1460) State value to set for "gartenbewaesserung.0.config.ventil8.dauer_in_state_mult" has to be type "number" but received type "string"

gartenbewaesserung.0
2021-07-18 08:56:57.573
info
(1460) State value to set for "gartenbewaesserung.0.config.ventil7.dauer_in_state_mult" has to be type "number" but received type "string"

gartenbewaesserung.0
2021-07-18 08:56:57.493
info
(1460) State value to set for "gartenbewaesserung.0.config.ventil10.dauerstate_mult" has to be type "number" but received type "string"

gartenbewaesserung.0
2021-07-18 08:56:57.492
info
(1460) State value to set for "gartenbewaesserung.0.config.ventil9.dauerstate_mult" has to be type "number" but received type "string"

gartenbewaesserung.0
2021-07-18 08:56:57.491
info
(1460) State value to set for "gartenbewaesserung.0.config.ventil8.dauerstate_mult" has to be type "number" but received type "string"

gartenbewaesserung.0
2021-07-18 08:56:57.490
info
(1460) State value to set for "gartenbewaesserung.0.config.ventil7.dauerstate_mult" has to be type "number" but received type "string"

gartenbewaesserung.0
2021-07-18 08:56:57.443
info
(1460) State value to set for "gartenbewaesserung.0.config.ventil10.dauer" has to be type "number" but received type "string"

gartenbewaesserung.0
2021-07-18 08:56:57.442
info
(1460) State value to set for "gartenbewaesserung.0.config.ventil9.dauer" has to be type "number" but received type "string"

gartenbewaesserung.0
2021-07-18 08:56:57.441
info
(1460) State value to set for "gartenbewaesserung.0.config.ventil8.dauer" has to be type "number" but received type "string"

gartenbewaesserung.0
2021-07-18 08:56:57.440
info
(1460) State value to set for "gartenbewaesserung.0.config.ventil7.dauer" has to be type "number" but received type "string"

gartenbewaesserung.0
2021-07-18 08:56:57.437
info
(1460) State value to set for "gartenbewaesserung.0.config.ventil3.dauer" has to be type "number" but received type "string"

gartenbewaesserung.0
2021-07-18 08:56:57.182
info
(1460) starting. Version 0.0.8 in C:/iobroker/GLT/node_modules/iobroker.gartenbewaesserung, node: v14.16.0, js-controller: 3.3.14

host.GLT(GLT)
2021-07-18 08:56:54.946
info
instance system.adapter.gartenbewaesserung.0 started with pid 1460

`


CAMEL-11754 - add option to extract file name if ftp client directory

parser ends up with absolute paths

Thanks for the help guys.  It looks good.


Feature request: Allow overriding ollama's llama-cpp version

Would be ideal to have an 'escape-hatch' for using nixpkgs ollama, but with custom llama-cpp versions via .override. As it stands, you're forced to use an overlay, which can be bad ergonomics for many applications.
@dit7ya @elohmeier

You can override version and src via overrideAttrs, as long as you consistently use finalAttrs in the package expression: https://github.com/NixOS/nixpkgs/blob/bfbfb344598544058481e254b9f0f174d05b5e6a/pkgs/by-name/ll/llama-cpp/package.nix#L46

Sorry, my issue was confusingly named. I'm talking about overriding the package version of llama-cpp used here.

H'm. This can be passed in the .override, can't it? https://github.com/NixOS/nixpkgs/blob/d65bceaee0fb1e64363f7871bc43dc1c6ecad99f/pkgs/tools/misc/ollama/default.nix#L4

I... thought it could not, but now that you mention it, I should try that! :P
Sorry, I haven't worked with overrides much.

Thank you! Looks like it works perfectly :D


Test Todo List part 2

In this project, we added new test cases:
-In case of changing the description of a task.
-In case of change the state completed the task.
-In case of clear all the tasks.

@SaveryIV @MozamelJawad
Future section is blank, can you fill it asap?


DataSourceLoader only query against the result set from entity framework, does it mean it's doing 2nd query?

Query DB without filter
Query the result set from DB?

What's the best approach to improve the performance?

Depending on load options, DataSourceLoader can build one or multiple queries as well as perform some in-memory post-processing.
Using SQL Server Profiler, you can trace the actual SQL.
In general, we try to postpone query execution as long as possible.
For example, the code below doesn't necessary mean that the first Where is executed independently.
Usually, load options are appended to the existing quiery and later executed as a whole.
var subset = db.Set1.Where(...);
DataSourceLoader.Load(subset, ...);


benchmark print readability

To match the print format as shown in the readme: 1,000/1.000 for one thousand; 1,000,000/1.000.000 for one million etc.

I get this error error: ISO C does not support the ''' printf flag [-Werror=format=]
And this doesn't seem super important, so I'll leave it for now.


Integrate Nessie with Starrocks doesn't show databases

Steps to reproduce the behavior (Required)
I tried to integrate Nessie with Starrocks. but the databases don't show on Starrocks. But they do exists on Nessie
CREATE EXTERNAL CATALOG iceberg_nessie
PROPERTIES
(
"type" = "iceberg",
"iceberg.catalog.type" = "rest",
"iceberg.catalog.uri"="http://xxx.xxx.xxx.xxx:19121/iceberg/",
"iceberg.catalog.warehouse"="warehouse",
"aws.s3.enable_ssl" = "false",
"aws.s3.enable_path_style_access" = "true",
"aws.s3.endpoint" = "http://xxx.xxx.xxx.xxx:9000/",
"aws.s3.access_key" = "minioadmin",
"aws.s3.secret_key" = "minioadmin"
"client.factory"="com.starrocks.connector.iceberg.IcebergAwsClientFactory"
);
StarRocks > set catalog iceberg_nessie;
Query OK, 0 rows affected (0.00 sec)
StarRocks > create database uw2;
Query OK, 0 rows affected (0.09 sec)
StarRocks > show databases;
Empty set (0.03 sec)
the uw2 database does exists on Nessie. Perhaps there's something that I did wrong?
Expected behavior (Required)
Databases show on Starrocks
Real behavior (Required)
The databases don't show on Starrocks, but they're created on Nessie
StarRocks version (Required)
3.3.1-2b87854

@Valkyrie298 I am also facing same issue .

I think this is supposed to be fixed in the next patch release: #48844