apply fails with SerializationError

Description
Ever so often the terraform apply would fail with the error below. I've seen it with both a 200 and a 400 response code from AWS... so don't rule out my internet connection is partly to blame.
Versions
>terraform -version
Terraform v1.1.6
on windows_amd64

>terraform providers -version
Terraform v1.1.6
on windows_amd64

Reproduction

This is not easily reproducible at will, but I've had 2 TF scripts fail twice (each) today owing to this error.
Expected behavior
A retry in case terraform couldn't understand the response of a DescribeAPI.
Actual behavior
Terraform crashes and leaves the apply half-way abandoned.
Terminal Output Screenshot(s)
module.replica1.module.db_instance.aws_db_instance.this[0]: Still creating... [12m41s elapsed]
module.replica1.module.db_instance.aws_db_instance.this[0]: Still creating... [12m51s elapsed]
╷
│ Error: Error retrieving DB Instances: SerializationError: failed decoding Query response
│       status code: 200, request id: xxxxxx
│ caused by: read tcp <IP_ADDRESS>:58440-><IP_ADDRESS>:443: wsarecv: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.
│
│   with module.master2.module.db_instance.aws_db_instance.this[0],
│   on ..\..\modules\db_instance\main.tf line 27, in resource "aws_db_instance" "this":
│   27: resource "aws_db_instance" "this" {
│

Additional context
What is important to note is that I am having a bad day in terms of packet-loss on my home network (~5% packet loss), so unsure if this is related to that.
Googling, this appears to be more of a Windows + Golang specific error and then the request from this bug-report is to see if terraform could be more robust to catch such cases and retry if possible (at least for read-only API requests).

Hi @robins - I would file this with the upstream Terraform project https://github.com/hashicorp/terraform since its not related to this module


Combine site content forms

SiteAddContentForm and SiteEditContentForm are implemented separately right now. This may have been to support the multi-column layout from the mockups in the "add" view, and the single column layout in the right panel "edit" view. All the same, the functionality of these forms is largely identical, and we should not be in the practice of implementing add/edit forms more than once unless there is a significant functional difference.

I believe also that the multi-column view isn't set in stone (iirc @abdulkdawson said that a single column would work fine)

I agree, the add/edit form should always be the same.

See https://github.com/mitodl/ocw-studio/issues/191


google Map Symfony2

Good evening,
there is a video to explain how I used egeloen / ivoryMapBundle because I have tried many but no display.

I added egeloen / google-map-bundle: ~ 2.1 in my composer.json,
and after I add new Ivory \ GoogleMapBundle \ IvoryGoogleMapBundle () in my app / AppKernel.php
, After I did a simple test but nothing appears, I'm still trying and I created a new bundle name Ivory \ IvoryGoogleMapBundle, and also does not display anything

This bundle is no longer maintained. You can use it though, but it won't work for Symfony3+.

Bonjour,
ui je veux l'utiliser pour le symfony2, mais je n'ai pas connais comment ?
s'il vous plais m'aide.
2016-03-02 11:50 GMT+01:00 Konrad Mski<EMAIL_ADDRESS>
This bundle is no longer maintained. You can use it though, but it won't
work for Symfony3+.
—
Reply to this email directly or view it on GitHub
https://github.com/egeloen/IvoryGoogleMapBundle/issues/152#issuecomment-191184585
.

https://github.com/egeloen/IvoryGoogleMapBundle/blob/master/Resources/doc/usage.md
Install Bundle, then in action in controller:

$map = $this->get('ivory_google_map.map');
and send it to the template (.html.twig).

In template use:
{{ google_map(map) }}
And that's it. Open given web page and make sure:

there is no 404/js errors in console
rendered code is visible (might be css fix related)

Ok un grand merci MSR , je vais essayer une autre fois .Merci
2016-03-03 10:32 GMT+01:00 Konrad Mski<EMAIL_ADDRESS>
https://github.com/egeloen/IvoryGoogleMapBundle/blob/master/Resources/doc/usage.md
Install Bundle, then in action in controller:
$map = $this->get('ivory_google_map.map');
and send it to the template (.html.twig).
In template use:
{{ google_map(map) }}
And that's it. Open given web page and make sure:

there is no 404/js errors in console
rendered code is visible (might be css fix related)

—
Reply to this email directly or view it on GitHub
https://github.com/egeloen/IvoryGoogleMapBundle/issues/152#issuecomment-191677705
.

Good evening, I did as I say but that the result is this-I have to do a nv
Bundle src above and copy all the google-map-bundle file? or what ?
[image: Images intégrées 1]
2016-03-03 13:23 GMT+01:00 riahi rim<EMAIL_ADDRESS>
Ok un grand merci MSR , je vais essayer une autre fois .Merci
2016-03-03 10:32 GMT+01:00 Konrad Mski<EMAIL_ADDRESS>
https://github.com/egeloen/IvoryGoogleMapBundle/blob/master/Resources/doc/usage.md
Install Bundle, then in action in controller:
$map = $this->get('ivory_google_map.map');
and send it to the template (.html.twig).
In template use:
{{ google_map(map) }}
And that's it. Open given web page and make sure:

there is no 404/js errors in console
rendered code is visible (might be css fix related)

—
Reply to this email directly or view it on GitHub
https://github.com/egeloen/IvoryGoogleMapBundle/issues/152#issuecomment-191677705
.

Good evening, I like to tell me but the result was a simple map without action is this-I have to do a nv Bundle src above and copy all the google-map-bundle file? or what ?

Good evening, I like to tell me but the result was a simple map without
action is this-I have to do a nv Bundle src above and copy all the
google-map-bundle file? or what ?
2016-03-03 20:23 GMT+01:00 riahi rim<EMAIL_ADDRESS>
Good evening, I did as I say but that the result is this-I have to do a nv
Bundle src above and copy all the google-map-bundle file? or what ?
[image: Images intégrées 1]
2016-03-03 13:23 GMT+01:00 riahi rim<EMAIL_ADDRESS>
Ok un grand merci MSR , je vais essayer une autre fois .Merci
2016-03-03 10:32 GMT+01:00 Konrad Mski<EMAIL_ADDRESS>
https://github.com/egeloen/IvoryGoogleMapBundle/blob/master/Resources/doc/usage.md
Install Bundle, then in action in controller:
$map = $this->get('ivory_google_map.map');
and send it to the template (.html.twig).
In template use:
{{ google_map(map) }}
And that's it. Open given web page and make sure:

there is no 404/js errors in console
rendered code is visible (might be css fix related)

—
Reply to this email directly or view it on GitHub
https://github.com/egeloen/IvoryGoogleMapBundle/issues/152#issuecomment-191677705
.

Hi @rimariahiriri
You need to add event to the map ? I mean, you want to add event on map when you click or something like that ? I need to do it but i do not know how it work :/ Do you have find a solution ?

hi, KevinArtus
yes I tried and I found a simple solution

No. I asked myself even a video to explain how the map in symfony

Hi @rimariahiriri
Could you, please, share it with me ? I need to add onClick event on my map but i do not understand this line (https://github.com/egeloen/ivory-google-map/blob/master/doc/usage/events.md) :
Event name
Each object has his own event name which are described in the google map documentation.
If you have find a way to do that, i need your help =)
Thank you.

@KevinArtus @rimariahiriri
Hi guys. Sorry for delay. I will dig up some old project and paste some snippets in here later today.
Remember that this Bundle is not maintained anymore.


Impl From<&[A]> for Array1

At the moment, Array1 has
impl From<Vec> for Array1

but it does not have
impl From<&[A]> for Array1

Is there a reason for this?  The latter implementation seems like it would be more general (especially given that Vec can easily be converted to &[A]).  The only thing perhaps is that &[A] will probably require to A to be clonable.
If there is no reason for the lack of implementation, I'm happy to create a new pull request for this.

Ownership semantics matter, not just ownership of single elements but of a region of memory and thus From<Vec> has an important difference to the slice variant -- the from vec operation computes in O(1) time with no copying or reallocation.
I'm not 100% sure we should add this -- ArrayView::from(&[A]) already exists (also computes in O(1) time). Users could figure out the other details and the copying needed. Before I've mostly liked to use From for simple transformations of equivalent types like this -- but I guess From is getting wide use for all kinds of transformations in the Rust ecosystem in general?

The computation overhead part is especially important when it comes to an implementation of From for any type since it is used so widely in generic code via Into<T> that it could significantly degrade performance oriented generic programming in Rust if too many types do high-overhead computations on From impls.
For this we can still have explicit methods that (not only via doc) tell users about their costs instead of promoting further impliciteness as it is done in todays C++ code with very intransparent results.

Ok, I think we've decided to not do this for the time being, motivation is that we want to not obscure the cost of copying; the same operation is easily reachable by composing two methods or functions instead.


Segmentation fault of coclustering on debian 12

Description
Khiops Coclustering fails with a segmentation fault on LearningTest/TestCoclustering/Standard/Iris/
:warning:  It is a non-reproductible bug
Context

Khiops version 10.2.0
Debian 12

An example:
https://github.com/KhiopsML/khiops/actions/runs/7987638491/job/21810655602#step:10:1

Seems like @marcboulle 's domain.


[Registry Preview] Enables custom caption bar to allow better theme support

Summary of the Pull Request
Enables custom caption bar to allow better theme support

PR Checklist

[X] Closes: #25321
[X] Localization: All end user facing strings can be localized
[X] Dev docs: No changes required

Detailed Description of the Pull Request / Additional comments
Spent way too much time trying to do this using "simple" title bar customization; after discovering that it would be "iffy" and unsupported on Windows 10, I opted to extend the app canvas and enable a fully customized title bar.  This required a new string, updating the code to update the new text block as well as continuing to update the "traditional but hidden" title bar, since that's what shows up in the TaskBar.
After several hours of screaming "WHYYYYYYY?!" in my head, I finally found out why my light and dark theme colors were getting muted with grey, so I added two new brush "overrides" in the App.xaml file to avoid this unexpected yet default behavior.
Validation Steps Performed
Tested light and dark modes, before the app was running and during, on local and remote sessions.

/azp run


findRandomPointAroundCircle always return false when circle is small and poly is large

findRandomPointAroundCircle   use dtRandomPointInConvexPoly to generate a random point in poly, then check the random point is in the circle. But when the poly is large poly and the circle is a small circle ,  like circle radius is 1 while poly is an 10 * 10 grid, the  possibility that the random point generated by dtRandomPointInConvexPoly lies in the small circle  is so low that even hundreds of iteration cant return a valid position.

Whilst I would prefer it actually return a point in the circle, this is the behaviour defined within
https://github.com/recastnavigation/recastnavigation/blob/c5cbd53024c8a9d8d097a4371215e3342d2fdc87/Detour/Include/DetourNavMeshQuery.h#L463

I narrow down the problem to generate a random point in the intersection area of triangle and circle with in same plane. But so many cases to handle.  9 cases of intersection of triangle and circle

I narrow down the problem to generate a random point in the intersection area of triangle and circle with in same plane. But so many cases to handle.

As you have found it is very non-trivial (and slow) to clip the nav mesh to the input circle before generating the points, hence the function only uses the input circle to restrict which polygons can be selected. So indeed, it is by design that the point returned can lie outside the circle.

FYI: I have implemented such a functionality in recast4j by approximating a circle with a dodecagon and using the convex-convex intersection algorithm by O'Rourke (https://github.com/ppiastucki/recast4j/commit/44e4e33562441d25e69e4876f9bffe10b5b3225c). The version which clips polys is roughly 30% slower which is acceptable in my opinion.

FYI: I have implemented such a functionality in recast4j by approximating a circle with a dodecagon and using the convex-convex intersection algorithm by O'Rourke (ppiastucki/recast4j@44e4e33). The version which clips polys is roughly 30% slower which is acceptable in my opinion.

That looks interesting! For me performance isn't essential as im wanting to do a one off call to find spawning points around a boss but don't want them spawning in walls or too far away.

FYI: I have implemented such a functionality in recast4j by approximating a circle with a dodecagon and using the convex-convex intersection algorithm by O'Rourke (ppiastucki/recast4j@44e4e33). The version which clips polys is roughly 30% slower which is acceptable in my opinion.

for a low requency function, I would value the correct answer than a quick one. Thank you for the solution, I would look into it later.

Very cool! 30% is certainly not bad. If someone were to contribute this functionality I would happily accept it.

That looks interesting! For me performance isn't essential as im wanting to do a one off call to find spawning points around a boss but don't want them spawning in walls or too far away.

It's pretty common that the circle is large in which case rejection sampling should be ok. Is that not the case here?

Adapting the new query method addition to #638


Add new delegate to validate tokens being replayed that takes a token.

https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/blob/b45a5b4cf4e918ca2b2c12ff1951f1be7b592bac/src/Microsoft.IdentityModel.Tokens/TokenValidationParameters.cs#L99
This delegate takes the securitytoken as a string.
This requires the users to do some work to determine the token type and logic as different token types { JWT, SAML, SAML2, ...} may require special processing.
We have already hydrated the token into a object so pass it as a pararmeter.

Not a priority.

I think this was a good idea. For Saml2 tokens it makes sense: There is already a token-unique Id that can be used to detect replay without having to store the entire token. As Saml2 tokens are quite large it is much more efficient to just store the id.

@AndersAbel could you write a feature request? Would be great to have it on the issue board from the perspective of a user of the library.


[PRODUCT/STOCK] create a pop with a programmed task to display the stocks below the minimum QT

create a pop on dashboard or other thing with a programmed task to display the stocks below the minimum QT (without taking into account current orders)

@Circlegent

what is the best ?

Support of information ?

mail
page
pop up

What information in communication support ?

stock status of all items in the base
only the stock status of products linked to the tasks on order


Setup automated dependency checking

What needs to be done

[ ] Lets discuss setting up automated PRs on dependency updates to make @OS2agora/maintainers life easier.

I suggest we take advantage of the built-in dependabot feature for the sake of simplicity


Memory file broken, no metadata

Ever since the change that split the metadata from the embeddings, it can't be loaded from disk anymore, as only the embeddings get saved an not the metadata.
This makes everything work just fine when using it from Ram, but completely breaks as soon as you try to load from storage.
Not sure what is a clean solution here, as saving two files would also be a hassle...
We could make a single dictionary outa the two of them when saving and splitting again once loaded?

I see what you mean. Splitting them made a lot of sense from resource usage/performance standpoint but this was overlooked.
Two dicts make sense, want to give it a shot?

I see what you mean. Splitting them made a lot of sense from resource usage/performance standpoint but this was overlooked.
Two dicts make sense, want to give it a shot?

ye sure if that works out for you.


Make code compile and lint cleanly under all relevant versions of rust

We need to have more reliable CI for all relevant versions of the rust toolchain. Is there a way to make this happen? I'd prefer to not to commit Cargo.lock.
In our GH CI, this is what runs with the default Rust version (is it maybe the latest stable?)
https://github.com/Qiskit/openqasm3_parser/blob/5305ac349f69db705f0bd8e9f422bfd2c238f792/.github/workflows/main.yml#L26-L29
But for the MSRV, we do not run clippy in CI, and the command for running tests is slightly different.
https://github.com/Qiskit/openqasm3_parser/blob/5305ac349f69db705f0bd8e9f422bfd2c238f792/.github/workflows/main.yml#L41-L44
I found that occasionally upon pushing a PR, CI would fail because some component or dep is upgraded.
I tried some conditional compilation. For example here:
https://github.com/Qiskit/openqasm3_parser/blob/5305ac349f69db705f0bd8e9f422bfd2c238f792/crates/oq3_syntax/src/ptr.rs#L34-L49
I think it helped, but I don't recall why I was not able to solve the problem completely.

Followings are good references to resolve this issue:
clone_copy issue reported
[Roadmap] Lint groups
Thanks


Marker (in LayerGroup in Pane) not placed in the expected order

Checklist

[X] I've looked at the documentation to make sure the behavior isn't documented and expected.
[X] I'm sure this is an issue with Leaflet, not with my app or other dependencies (Angular, Cordova, React, etc.).
[X] I've searched through the current issues to make sure this hasn't been reported yet.
[X] I agree to follow the Code of Conduct that this project adheres to.

Steps to reproduce

Place one circle in a Pane with index 821
Place another circle in a Layergroup which was added to a different pane with index 850

Expected behavior
Expected the orange circle to be on top of the red circle because the orange circle belongs to the LayerGroup orangeGroup which is in the Pane orange which has a higher z-index than Pane red with the red circle.
Current behavior
The red circle is on top of the orange circle.
I'm not 100% sure if this is not expected but I didn't find a hint in the docs that it should work like this.
Minimal example reproducing the issue
https://plnkr.co/edit/MN6JUsDJfhwW3htm?preview
Environment

Leaflet version: 1.9.4
Browser (with version): Firefox 113.0.1
OS/Platform (with version): Zorin OS 16

Duplicate of https://github.com/Leaflet/Leaflet/issues/8958
L.Paths (including L.Circles) are added to a L.Renderer, not to a map pane.


Several links on the right nav bar don't work

Several of the in-document anchor links in the right side navigation panel don't actually jump the document to the expected heading.
(repro at least in Edge Chromium)

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

ID: 1e062363-94e9-f776-650f-a0723ab713a3
Version Independent ID: 9181db12-cbdf-6f95-27f1-833930d6d323
Content: Visual Studio Build Tools workload and component IDs
Content Source: docs/install/workload-component-id-vs-build-tools.md
Product: visual-studio-windows
Technology: vs-installation
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@AArnott Thanks for letting us know!
FYI Terry, I asked about this on Docs Support channel.

Hey folks, this has been resolved. Thanks for reporting it @AArnott (FYI @gewarren )


Glui label sizing is horribly stupid

However, this issue is a bit tricky to fix, because the label would need to know parent's align direction to know what direction to expand in (horizontally for hspace, vertically for vspace). Because the idea is a node shouldn't know its parent, it's not really possible to make this work, unless the parent passed this value to the child — and that's a terrible option.
Here are other options:

Re-introduce textWrap in style — Could work, but since this is layout-heavy, and the value is expected to be the same between styles, it would be better off being set in layout.
Split label to vlabel and hlabel — This absolutely would work, and could give more control over text wrap to the user. Current label could be aliased to vlabel and marked as deprecated.
Introduce inline mode (which is a thing that should eventually be done), where nodes can define a secondary draw call which would take 3 vectors (cursorPosition, lineEnd, lineStart), so text rendering would be managed by the frame — the issue would remain for regular vspace and hspace though. This would probably be the best replacement for a richTextLabel, though.

Currently I'm most supportive of the second option, but I'm not sure how it'd work with said inline mode. Probably the direction would be ignored. And what about text rendering outside of labels? Wrap is managed by Style after all…

btw it looks like Style moves multiple spaces to the end of the line… interesting.

btw it looks like Style moves multiple spaces to the end of the line… interesting.

Actually, this seems to be the reason for this to happen. The error message in the screenshot aligns properly as long as it doesn't feature multiple spaces.


Can you provide an example of using JSON plus IFormFile?

If a POST API has a model with strings, files, and IFormFile then it is not a straight forward way to implement the solution.

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

ID: 9a5cfa38-0e6c-4346-dafc-84ea5c513fcf
Version Independent ID: c6787f0f-3e9a-6efb-0e6e-5ccb957d3f4a
Content: Upload files in ASP.NET Core
Content Source: aspnetcore/mvc/models/file-uploads.md
Product: aspnet-core
Technology: aspnetcore-mvc
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Moved to File Uploads topic enhancements and master issue #12285


Add download task

Context
Explain context and other details for this pull request.
Checklist

[ ] ./gradlew build compiles code correctly
[ ] Added new tests where applicable
[ ] ./gradlew test passes all tests
[ ] Extended README or added javadocs where applicable

Can you update the description about what this is for?


Error in documentation - section Roles property values

The documentation specifies following values for the roles property: read, write, owner
The value of owner is not correct.  Should be fullcontrol

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

ID: 1191235c-879e-dffc-82eb-4e8f7edfd29c
Version Independent ID: 322f9be4-7921-41d8-78c2-2bdc62f916a3
Content: Permission - Microsoft Graph v1.0
Content Source: api-reference/v1.0/resources/permission.md
Product: sharepoint
Technology: microsoft-graph
GitHub Login: @spgraph-docs-team
Microsoft Alias: MSGraphDocsVteam

Closing as this will be tracked here: https://github.com/microsoftgraph/microsoft-graph-docs-contrib/issues/8945


Creating or dropping database with pgm.sql fails with cannot run inside a transaction block error

PostreSQL version: 10.9
node-pg-migrate version: 3.21.1
Below is extract from migration file and migration log resp. Any pointers round this are welcome. Thanks.

exports.up = (pgm) => {
	pgm.sql( "CREATE DATABASE boda");
	pgm.createRole( "bodastage", {
		"password": "password",
		"login": true
	} );
};

$ DATABASE_URL=postgres://postgres:postgres@localhost:5432/postgres yarn run migrate up -m db/migrations/
yarn run v1.15.2
$ ./node_modules/.bin/node-pg-migrate up -m db/migrations/
> Migrating files:
> -<PHONE_NUMBER>302_create-boda-database
### MIGRATION<PHONE_NUMBER>302_create-boda-database (UP) ###
CREATE DATABASE boda;
CREATE ROLE "bodastage" WITH NOSUPERUSER NOCREATEDB NOCREATEROLE INHERIT LOGIN NOREPLICATION ENCRYPTED PASSWORD $pg1$password$pg1$;
INSERT INTO "public"."pgmigrations" (name, run_on) VALUES ('1564257260302_create-boda-database', NOW());

Error executing:
CREATE DATABASE boda;
error: CREATE DATABASE cannot run inside a transaction block

I should have read the docs thoroughly first! I found the solution is to use pgm.noTransaction();


Instructions for using AngularDart v5 w/ popular IDEs

... basically, IntelliJ/WebStorm and VS Code, at least.

@matanlurey What about emacs 😛 ? Just kidding.

This isn't blocking or very actionable.


Feedback from Ralph

@allclark,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


[Enhancement] - A 'Sort by Name' sorting option

Because I primarily run SD off of a cloud service, I have the server zip my output folder so I can download it. However, when I unzip it, the "date created" is changed to the unzip date. This makes the images that are part of a large batch of zipped files nearly impossible to organize when they number in the hundreds or more because they all have the exact same creation date.
Since I instruct SD to name my files with the [datetime] signature, they organize themselves perfectly on my home computer when I sort them by name. I would greatly appreciate it if this fantastic Toolkit could implement a simple 'sort by name' feature as well! It would make my organization life a lot easier and be a huge boon in sorting & sifting through images.
While I've tried to understand how the cloud service (paperspace) zips its files so that I can try to instruct it to maintain the original creation date of the files upon zipping, I'm still relatively new at learning much about cloud services and how a Jupyter notebook functions.
Thanks so much, and thank you as well for creating and maintaining such a fantastic free tool for us all to use!!!

Hi,
Please try this build.
Diffusion.Toolkit.v1.5.2beta.zip

hi, Rupert,
i have download and tried the 1.5.2 and it is crashing constantly. even opening it triggered the windows installation safety warning.
plz advise.

Please try this one
Diffusion.Toolkit.v1.6 beta.zip


Does DuckDB work with single writer, multiple readers?

I have a use-case where I have one process continually writing to a DB and multiple readers who are reading from the DB.
Does DuckDB support for this use-case? Docs say multireader is ok, but not sure if that also allows for one writer.

These are separate processes, right? This works fine if you are talking about threads in a single process, but if multiple processes access a single DuckDB database file at the same time, they all need to be in read-only mode.

These are separate processes, right? This works fine if you are talking about threads in a single process, but if multiple processes access a single DuckDB database file at the same time, they all need to be in read-only mode.

May I suggest making this more prominent in the documentation?
Nothing on the home page suggests that you can't have multiple processes except in read-only mode. It ought to be in the "When not to use" column.

@tlbtlbtlb I have added "Writing to a single database from multiple concurrent processes" to the "When not to use DuckDB" section. Thanks for the suggestion. Whats the use case you were thinking about there? This restriction is mostly due to the lack of a database server that would be otherwise holding and synchronising transaction states. You can write to a duckdb file from concurrent threads within one and the same process though, is that not an option for you?

It's possible I can make it work. The single-process restriction just caught me by surprise because I'd built a previous version with sqlite3 and separate binary blob files. I was hoping to rewrite with all the data in DuckDB, because managing the blob files is error-prone.
My usage is mostly read-only queries, but I also store small bits of state in the database, like the last-used visualization settings for that data. I may be able to separate out the mutable state, and keep that in sqlite3. But I am a bit reluctant to have 2 similar-but-different database APIs in the same program.
I will think more about the best way to organize the data.


Payment countdown

A purchase quote has no countdown, which means the user may pay 30 minutes later, and the price could be significantly different.

The order essentially expires in 10 minutes if no transaction is submitted to the Bitcoin network (doesn't need to be in the block, just submitted). Handling of the unfunded order still needs work, which will probably require its own dedicated issue.

@drwasho
The order essentially expires in 10 minutes if no transaction is submitted to the Bitcoin network

How is this communicated to the user? Would it help to have a timer?


Access control for nomad and consul APIs

[x] Nomad
[ ] Consul

Consul 1.4 has a new ACL system.
Support in Vault is only available in 1.0 (see https://github.com/hashicorp/vault/commit/53ad302a315b0c3e399ec04d31183370a21b4569)


Add quick access to default navigation to waypoints in waypoint list in cache details view

Actually i missed quick access to default navigation for waypoints. As the click event is already in use for the listed waypoints you always have to open the menu. Also the second preferred navigation tool is not supported from the menu.
This change adds the navigation icon at the right border of the waypoints inside the waypoint list. The icon starts the first default navigation tool on click and the second preferred navigation tool on long click. It's the same behaviour as already used for default navigation for the cache.
From my point of view it is simple to understand, will improve the caching experience a lot and is only a small layout change to add access to already approved functions.

I'm against this change. Navigation is possible with the context menu, that's enough.


rotate, shear, contrast and other transformations

Is there any built-in or easily added way to increase the number of training images for object training? In particular, I need a way to increase my examples which are symbols on a flat plane where my symbol is distorted by viewing angle, scale, focus blur,  perspective (keystone?), lighting angle, etc. For example, think of trying to recognize the suit and value of a playing card using object detection. I have about a hundred examples from various distances, angles and rotations, but it has a poor recognition rate probably because I probably need to have a lot more examples that fill in the gaps. The biggest issue is creating the training data with the ground truth rectangles which would be better if I could tween (create images in-between key images) missing data.

@turbogeek  have u explored one shot object detection: https://apple.github.io/turicreate/docs/userguide/one_shot_object_detection/ ?

Hi, sorry for the very late reply. I just tried one shot, but having a few issues. Part of the problem is that I am getting very tiny images that would be vastly inappropriate to the task. Is there a way to control the relative size of the image to the target? FYI otherwise, this might work for me. My application is marked items like the attached image (easy to see here). My problem is a bit odd. Imagine looking for a particular dalmation in a gallery. So it is a dog, but a particular dog. I have many of these dogs (52, not 101), all slightly different but distinct.

Imagine this on a table where the image at most takes up about 1/5th - 1/8th or so of the image.
I have tried using a small amount of the image, that comprises the catagorizable part of the image, without success.

Hi, sorry for the very late reply. I just tried one shot, but having a few issues. Part of the problem is that I am getting very tiny images that would be vastly inappropriate to the task. Is there a way to control the relative size of the image to the target? FYI otherwise, this might work for me. My application is marked items like the attached image (easy to see here). My problem is a bit odd. Imagine looking for a particular dalmation in a gallery. So it is a dog, but a particular dog. I have many of these dogs (52, not 101), all slightly different but distinct.

Imagine this on a table where the image at most takes up about 1/5th - 1/8th or so of the image.
I have tried using a small amount of the image, that comprises the catagorizable part of the image, without success.

Just to add a little more, the one shot needs precise controls.  Need to have control over scaling, rotate, tilt, shear etc. Would be great to either replicate or just use the https://github.com/mdbloice/Augmentor capabilities that allow the user to create a pipeline that creates images and data files on disk.
Seems like the user needs to be able to tune images such that a human could confirm the configuration of augmentation and images are good enough for training (if I ran't recognize it, the machine can't). Also need an easier way to add my own images by pointing to a directory.
Also need to now what the proper sizes are that would be best for success.

Just to add a little more, the one shot needs precise controls.  Need to have control over scaling, rotate, tilt, shear etc. Would be great to either replicate or just use the https://github.com/mdbloice/Augmentor capabilities that allow the user to create a pipeline that creates images and data files on disk.
Seems like the user needs to be able to tune images such that a human could confirm the configuration of augmentation and images are good enough for training (if I ran't recognize it, the machine can't). Also need an easier way to add my own images by pointing to a directory.
Also need to now what the proper sizes are that would be best for success.


Render custom legend items with icons

@piensaenpixel need some help to make this look 👌 .
If you open examples/v4/leaflet-layergroup-legends.html in your browser, you'll see that I added one item with icon to the existing custom legend.
The template is src/geo/ui/legends/custom/legend-template.tpl, and I added a Legend-categoryIcon class to the span for the icon.
cc: @xavijam @nobuti @gfiorav

👏  thanks for fixing it @alonsogarciapablo 💋

cc @gfiorav and @javisantana

Fixed @alonsogarciapablo for me it's chachi piruli

Thanks @piensaenpixel! (and @nobuti for the pixel perfect centering 😛  ofc)


[JS] Only assign img alt text if it's supplied

Related Issue
Fixes #2235
Description
When a card author fails to supply alt text, we actually create an img element with the alt text set to "undefined" rather than not set the property in the first place.
How Verified
How you verified the fix, including one or all of the following:

New unit test
Local repro and inspection with F12 tools

Microsoft Reviewers: Open in CodeFlow

/azp run

Something we should consider backporting to release/1.2 and potentially prep for an AdaptiveCards v1.2.9 patch.. Thoughts?

Should be easy enough (though it'll likely be a manual port rather than a cherry-pick).

/azp run


Support batch split region command and empty batch command (#5470)

What have you changed?
Cherry-pick #5001 #5268 #5466
What is the type of the changes?

Improvement (a change which is an improvement to an existing feature)

How is the PR tested?
unit-tests
Does this PR affect documentation (docs) or should it be mentioned in the release notes?
NOTE: Does not support new TiDB run with old TiKV. For example, TiDB v3.0.4 + TiKV v3.0.0.
Does this PR affect tidb-ansible?
If there is a configuration or metrics change, please file a PR in tidb-ansible, and add the PR number here.
Refer to a related PR or issue link (optional)
#5001 #5268 #5466
Benchmark result if necessary (optional)
None
Any examples? (optional)
None

/run-all-tests

/run-all-tests

/run-integration-common-test

/run-unit-test

/run-integration-tests


Add Andromeda.toml as pre-genesis validator for v0.20.0

Description
If this is an UPDATE for a previous genesis validator
Checks are made against your net_address. If this has changed since the previous testnet, make sure you provide links of previous prs merged from your previous git username.
Thanks in advance!
Checklist before merging to draft

[ ] Only one toml is added in this PR
[ ] The file being added is indeed a .toml file
[ ] The toml being added is to the correct folder, and only to the correct folder
[ ] The eth_hot_key and eth_cold_key are present

I am a new validator and would like to participate in the testnet


WIP: Switch to new opencitations API

Initial port to new opencitations API.
TO DO:

[x] Add back citation counts (not part of the metadata in this api)
[ ] Cleanup
[ ] Make sure tests work
[ ] Formatting

Testing this branch, I find some DOI tests don't pass:
14: @test julia.year == 2017
fails as there is no year field  in the new format it seems.
19: show(io, MIME("text/html"), emjulia)
fails, not sure why.

Testing this branch (main times out), I find some DOI tests don't pass:
14: @test julia.year == 2017
fails as there is no year field in the new format it seems.

Thanks for testing. I think the main functionality is now in place, need to do some cleanup and make sure the tests pass still.

Tried running test as  well for commit https://github.com/hellemo/ShortCodes.jl/pull/12/commits/42dbc89c8d8cf48771ca59e07520eb0884b56bfe and as with @AshtonSBradley testing I'm seeing test fail but not due to 404 issue but due to 500 issues.
   Test threw exception
   Expression: length(split(julia.author, ";")) == 4
   HTTP.Exceptions.StatusError(500,"GET","/index/api/v1/metadata/10.1137/141000671",
...
...

Tests pass for me now


Feature : changing the Logo of Twitter with its new logo X

Is your feature request related to a problem? Please describe
Changing the old logo of twitter ( bird ) to its new logo ( X ) in the footer section of the blog page
Describe the solution you'd like
I would like to solve this issue by replacing the old logo ( Bird ) with the new logo ( X ) of the twitter
Describe alternatives you've considered
No response
Additional context
ScreenShot

Are you working on this?
Yes

@VaibhavUpreti @vedant-jain03 @tanmoysrt  please review this issue and assign it to me .

@DhairyaMajmudar ig it's on the blog website?
If yes, please raise the issue in blog's repository
Check here https://github.com/Circuitverse/blog

@DhairyaMajmudar ig it's on the blog website?
If yes, please raise the issue in blog's repository Check here https://github.com/Circuitverse/blog

Yes , its on a different website

@DhairyaMajmudar ig it's on the blog website?
If yes, please raise the issue in blog's repository Check here https://github.com/Circuitverse/blog

Yes , its on a different website

Open this issue in this repo https://github.com/Circuitverse/blog


BUG Fix false values for many_many_ExtraFields not being saved

Fixes #4067
This is because the $extraData was extracted directly from the posted data, when the correct way to retrieve data is to rely on the result of Form::saveInto, as this is how fields such as CheckboxField are able to overwrite values with false if no data is posted.

Fixed based on your feedback @assertchris

@dhensby want to look at this while you're reviewing stuff for 3.2?

Ta :D

This bug still appears to exist in 3.1.15.  I've just into it, Checkboxes are only being saved if their value is true.

Yep. :D Users will need to upgrade to 3.2.0 to get this fix.

I tried to work around this using a PickerField module, which turned out to have the same issue.  However it was possible to fix it there, and this offers a workaround for 3.1 users https://github.com/briceburg/silverstripe-pickerfield/pull/14#issuecomment-152885148


Migration

From VS Feedback
When we change something in more than one domain and try to add migration together for all the domain changes its not taking the changes in database  so we forced to  delete the all added migrations  and  to create the migration as initial

EF Team Triage: This issue is lacking enough information for us to be able to effectively triage it. In particular, it is missing the following information requested in the new issue template. Can you please provide this information?

Steps to reproduce

Ideally include a complete code listing that we can run to reproduce the issue.
Alternatively, you can provide a project/solution that we can run.
BTW we're not just doing this to be mean :smile:... we get a lot traffic on this project and it takes time to attempt to reproduce an issue based on fragments of information. In addition, our attempt is often unsuccessful as the exact conditions required to hit the issue are often not explicitly included in the code provided. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we ask that folks give us a self-contained way to reproduce an issue.
For a guide on submitting good bug reports, read Painless Bug Tracking.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.

Student1.zip
StudentDb.zip
when trying to make a property in a domain to be null ,migration is getting generated, but after updating database that field in the table kept unchanged
Above we attaching the project solution zip and database MySQL zip

@neethupvijayan This project does not build, so I cannot run the code. It references "..\Student.Domain\Student.Domain.csproj" which is not included.

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


Move TCL test unit/type/list-2 to Go case

Hi, I would like to work on this issue :)

@suica assigned. Go ahead!


Add context and token methods to allow accessing tokens on user-defined channels

Add context and token methods:
Context

start
stop

Token

channel
token_index

Also it needs to_ruby<Token*> to be moved before ContextProxy.
This is the first step to allow accessing tokens on user-defined channels.

For now, I access all tokens with a simple ParserProxy method returning an array of all tokens.
Array getTokens() {
  Array a;

  std::vector<Token*> tokens = this -> tokens -> getTokens();

  for (auto &token : tokens) {
    a.push(token);
  }

  return a;
}

I'm leaving it here for reference only, no PR, because proper implementation should expose TokenStream proxy and methods instead.

I'm ok with this change, but I don't really understand what your goal is. Why do you need access to all the tokens?

To be able to read tokens from custom channels. Usually this is done with BufferedTokenStream methods such as getHiddenTokensToRight, which takes channel id as the second argument.

@zakjan ok interesting. What do custom channels do? I've never used them before.

It's for secondary tokens, which can be processed separately from the main grammar. This SO question roughly illustrates my use case - processing doc comments which can appear anywhere in the input. https://stackoverflow.com/questions/12485132/catching-and-keeping-all-comments-with-antlr The first accepted answer is not a good solution, see others.
The Definitive ANTLR 4 Reference, section 12.1 Broadcasting Tokens on Different Channels contains intro to the topic.

Super interesting, thanks!


Remove table privelages checks from SystemAccessControl

Remove table privelages checks from SystemAccessControl
Presto do not manage table privileges on system level, but only on
connector level hence it should not control if one can GRANT or REVOKE
table privileges.
Notice that SystemAccessControl do not control access for GRANT or REVOKE role
already which is exactly the same case as table privileges.

I definitely think we should remove this for the file-based access controls as they check for table OWNERSHIP which is just wrong.  They should be checking for with grant option but the only supported with grant option is for select.
I'm not sure about removing these for all system level access control.  I think it might be possible for someone to add these to their custom implementation.  I would say the most useful version with be to block any grants in an installation to force users to use a different tool for grant management.


[ci] workaround gha mounting a new home dir.

Motivation
GHA mounts the home dir, so working around it by optionally installing tools from dev_setup.sh in /opt/ dir.   Should save a minute per job, and make CI less fragile.
This saves a minute from every rust job (tooling reinstall time), makes the CI more robust (dotnet downloads tend to fail when azure has a partial outage), and will be a good introduction to Todd, and deim-dev-infra team to what goes in to maintaining this fairly intricate part of CI that exists to make our builds more stable (no download failures), portable (the same dev-setup.sh was used to in circleci, and by some devs in move team), and to work around the limitations of gha (we can't build a docker image and use it in a pr securely - gha can't pull images from gha artifacts) so we have to keep dev-setup.sh idempotent on dev desktops and on CI systems.

Tested running in the gha-test-2 branch, by building the full base image, and running a pr against that branch here:
https://github.com/diem/diem/pull/9233/checks?check_run_id=3669290167.   It did show that boogie was always reinstalling so correcting that, first corrected on that test pr, then pushed back to this pr, then pushed directly to gha-test-2.
Related PRs
None
If targeting a release branch, please fill the below out as well

Justification and breaking nature (who does it affect? validators, full nodes, tooling, operators, AOS, etc.)
Comprehensive test results that demonstrate the fix working and not breaking existing workflows.
Why we must have it for V1 launch.
What workarounds and alternative we have if we do not push the PR.

/land will add documentation in a follow up pr - don't want to run full tests for small doc changes.

:broken_heart: Test Failed - ci-test

well, that's new, looks like forge uses the dev-setup.sh script now as well -- not sure that's a great idea.   Regardless, looks like I need to make a fix.    These docker image builds should have occurred with the update of the dev-setup.sh script in a pr, as well as just for the running of forge/cluster-test.    Good times.

/canary

:broken_heart: Test Failed - ci-test

/canary

:broken_heart: Test Failed - ci-test

/canary

:broken_heart: Test Failed - ci-test

/canary

/canary

This PR is already being canaried

:sunny: Canary successful

Pushed this to gha-test-2 and tests passed there as well, with a successful canary to test forge's build with the dev-setup.sh script, this seems ready.     The dev-setup.sh script should be assuming the tools it's installing will be immediately available on the $PATH.

/land


build: fix make clean

Signed-off-by: Michael Adam<EMAIL_ADDRESS>make clean was broken because the rm command pointed to a non-existing file location.
This fixes this by using the correct path.

@travisn  wrote:

The change looks good. In the commit message, please just move the line with Signed-off-by after the commit description.

Done, thanks!


Reduce dependencies on .NET5.0

This PR reduces the number of transient dependencies when targeting .NET5.0.
All SDK Contribution checklist:
This checklist is used to make sure that common guidelines for a pull request are followed.

[x] I have read the contribution guidelines.
[x] The pull request does not introduce breaking changes.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

We tried this change and it didn't work. See https://github.com/Azure/azure-sdk-for-net/issues/10326#issuecomment-839149950 for details.

@pakrym -- Good to have the context. Perhaps adding a .NET6.0 target to all the ns2.0 only libraries is an option in the November timeframe?

Closing this PR for now. Thanks again for taking a peek @pakrym.


Context menu in polygon

Hi everyone
I've tried to use context menu plugin on a polygon but it seems it doesn't work.
Is it any kind of bug or it's a feature that is not included? It would be nice to have it.
Thansk!

It works for polygons. Can you show me the code you're using?


Export to CDK: LogicalId missing for new layer version

Issue exporting to CDK. Logical id is missing, however <shared-layer>-cloud-formation-template.json includes two resources:
LambdaLayerVersionXXXXXXXX and LambdaLayerPermissionPrivateXXXXXXXX
I've removed all previous layers and updated the latest layer with the latest CLI.
amplify export --out <export-cdk-project> throws an error
Error: LogicalId missing for new layer version: <layer-name>
node: 14.8.3
amplify: 7.6.13
os: WSL Ubuntu

As an alternative, I removed all the dependencies of the shared layer and removed it completely.
amplify export --out <export-cdk-project> worked after that.
Still not a solution...

Hey @IsaacTrevino :wave: thanks for raising this! I was able to recreate this issue using the following steps:

create a new project amplify init -y
add a Lambda Layer with amplify add function
push with amplify push -y (note: this shouldn't be necessary but due to the expected error the export process never finishes)
amplify export --out tmp
observe error

LogicalId missing for new layer version: 9636layer84e8481c.
Error: LogicalId missing for new layer version: 9636layer84e8481c.
    at packageLayer (/Users/josef/.fnm/node-versions/v16.13.1/installation/pnpm-global/5/node_modules/.pnpm/amplify-category-function@3.4.1/node_modules/amplify-category-function/src/provider-utils/awscloudformation/utils/packageLayer.ts:83:11)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at /Users/josef/.fnm/node-versions/v16.13.1/installation/pnpm-global/5/node_modules/.pnpm/amplify-provider-awscloudformation@5.9.1/node_modules/amplify-provider-awscloudformation/src/resource-package/resource-packager.ts:156:40
    at async Promise.all (index 1)
    at ResourceExport.packageResources (/Users/josef/.fnm/node-versions/v16.13.1/installation/pnpm-global/5/node_modules/.pnpm/amplify-provider-awscloudformation@5.9.1/node_modules/amplify-provider-awscloudformation/src/resource-package/resource-packager.ts:151:12)
    at ResourceExport.packageBuildWriteResources (/Users/josef/.fnm/node-versions/v16.13.1/installation/pnpm-global/5/node_modules/.pnpm/amplify-provider-awscloudformation@5.9.1/node_modules/amplify-provider-awscloudformation/src/resource-package/resource-export.ts:72:31)
    at Object.run (/Users/josef/.fnm/node-versions/v16.13.1/installation/pnpm-global/5/node_modules/.pnpm/amplify-provider-awscloudformation@5.9.1/node_modules/amplify-provider-awscloudformation/src/export-resources.ts:40:31)
    at exportBackend (/Users/josef/.fnm/node-versions/v16.13.1/installation/pnpm-global/5/node_modules/.pnpm/@aws-amplify+cli@7.6.15/node_modules/@aws-amplify/cli/src/commands/export.ts:74:5)
    at Object.run (/Users/josef/.fnm/node-versions/v16.13.1/installation/pnpm-global/5/node_modules/.pnpm/@aws-amplify+cli@7.6.15/node_modules/@aws-amplify/cli/src/commands/export.ts:62:5)
    at Object.executeAmplifyCommand (/Users/josef/.fnm/node-versions/v16.13.1/installation/pnpm-global/5/node_modules/.pnpm/@aws-amplify+cli@7.6.15/node_modules/@aws-amplify/cli/src/index.ts:373:5)
    at executePluginModuleCommand (/Users/josef/.fnm/node-versions/v16.13.1/installation/pnpm-global/5/node_modules/.pnpm/@aws-amplify+cli@7.6.15/node_modules/@aws-amplify/cli/src/execution-manager.ts:178:3)
    at executeCommand (/Users/josef/.fnm/node-versions/v16.13.1/installation/pnpm-global/5/node_modules/.pnpm/@aws-amplify+cli@7.6.15/node_modules/@aws-amplify/cli/src/execution-manager.ts:30:5)
    at Object.run (/Users/josef/.fnm/node-versions/v16.13.1/installation/pnpm-global/5/node_modules/.pnpm/@aws-amplify+cli@7.6.15/node_modules/@aws-amplify/cli/src/index.ts:205:5)

Marking as a bug 🙂


Error moving file to output dir (mkvtomp4 & shutil)

I recently posted issue #887 and now I'm now trying to avoid it by combining all applications into one VM instead of separately (as the benefit of getting this to work outweighs the benefits of containerizing).
However I'm getting some permission errors when triggering delugePostProcess.py via deluge which I thought was related to #850 (because I'm running each app as a separate user) so I tried to run it via manual.py and received this error...
Processing file /mnt/download/completed/Movies/Mission Impossible (1996) [1080p] [YTS.AG]/Mission.Impossible.1996.1080p.BluRay.x264-[YTS.AG].mp4
Matched movie title as: Mission: Impossible 1996-05-22
Processing Mission: Impossible
Error moving file to output directory.
Traceback (most recent call last):
  File "/opt/SB_MP4_Auto/mkvtomp4.py", line 213, in process
    shutil.copy(inputfile, outputfile)
  File "/usr/lib/python2.7/shutil.py", line 120, in copy
    copymode(src, dst)
  File "/usr/lib/python2.7/shutil.py", line 91, in copymode
    os.chmod(dst, mode)
OSError: [Errno 1] Operation not permitted: '/mnt/download/transcoding/Mission.Impossible.1996.1080p.BluRay.x264-[YTS.AG].mp4'

My hope was to take moives from /completed ... process them in /transcoding ... move them into a /postProcess folder for final use. As I alluded earlier, I'm running these apps with 4 separate users deluge, sonarr, radarr, ffmpeg and each of them are members of the ffmpeg group so they can rwx your script.

ok scratch that issue...
I was confused why it worked on my old VM and not this one. turns out there were three changes I made:

I built ffmpeg from source
I was using python 2.7 instead of 3.5.2
I forgot quotes around the input directory in the manual.py command

I'm very sure it all had to do with the last item but I didn't realize it until I had reinstalled ffmpeg and upgraded python. I might go back to ffmpeg from source later, but i'm more concerned about getting this process to work for me before I start tinkering.


Invalid sender responses

It seems that MailHog accepts only single form of sender in MAIL FROM command, which is:
MAIL<EMAIL_ADDRESS>
putting space after colon or omitting angle brackets like
MAIL FROM<EMAIL_ADDRESS>
MAIL<EMAIL_ADDRESS>
MAIL FROM<EMAIL_ADDRESS>
leads to "550 Invalid sender" however most MTAs accepts those as valid forms.

its fixed ? for me MailHog dont work with swiftmailer 5.4.1 i dont know why... "Exception occurred while flushing email queue: Connection could not be established with host....." any idea please ?


Support memory quota for backup and import to avoid OOM and unnecessary IO

Development Task
Backup generates SST file in memory and then writes to external storage, it causes OOM when a host machine has restrict, e.g., 2GB.
Restoring a backup sst file, TiKV needs to write the file twice,

Download the file and save to disk (write no.1) https://github.com/tikv/tikv/blob/ac0f3b707be8ab055eeb5ccfea243703f609340b/components/sst_importer/src/sst_importer.rs#L205-L225
Rewrite keys in file (write no.2) https://github.com/tikv/tikv/blob/ac0f3b707be8ab055eeb5ccfea243703f609340b/components/sst_importer/src/sst_importer.rs#L353-L399

We can save no.1 by buffering in memory.
For both backup and import we should add a memory quota to prevent OOM and unnecessary IO.

For backup after #9283, the predictable memory usage by backup should be no more than (backup.num-threads × backup.sst-max-size), initialized to (CPU count × 0.75 × 144 MiB) = (CPU count × 108 MiB). So I think a memory quota for backup is not needed.

For backup after #9283, the predictable memory usage by backup should be no more than (backup.num-threads × backup.sst-max-size), initialized to (CPU count × 0.75 × 144 MiB) = (CPU count × 108 MiB). So I think a memory quota for backup is not needed.


System.NotSupportedException: 'Unable to broadcast already broadcasted shape.'

Here is the code
var shape = Z.shape;
var (u, v) = np.meshgrid(np.arange(shape[1]), np.arange(shape[0]));
var x = (u - Cu) * Z / f; <--- error raised here
var y = (v - Cv) * Z / f;
return np.vstack(x.ravel(), y.ravel(), Z.ravel());

Any ideas ?

We do not support broadcasting an already broadcasted NDArray.
My guess is the meshgrid uses broadcasting. If you'll clone the NDArray (NDArray.Clone()) that'll produce a linear/contagious stored NDArray which then can be broadcasted again.

My solution currently is to use np.full like this
var shape = Z.shape;
var (u, v) = np.meshgrid(np.arange(shape[1]), np.arange(shape[0]));
var CU = np.full(Cu, shape[0], shape[1]);
var CV = np.full(Cv, shape[0], shape[1]);
var x = (u - CU) * Z / f;
var y = (v - CV) * Z / f;
return np.vstack(x.ravel(), y.ravel(), Z.ravel());


I can`t build and run the project. Many building errors.

Sorry, not this place

In case of any error while using this plugin please share your code and the error.


winch(arm64): and, or, xor, shifts

Follow up to #8321.
Adds support to binary and, binary or, binary xor (eor), shl, shrS, shrU, rotr, rotl.

I'll take a look at this one.

Wonderful thanks for pushing this to the finish line!! ❤❤


Add material theme

Add material theme.
Default victory-label styles to match the grayscale theme.
Remove existing grayscale theme since it is the default for all the components, see https://github.com/FormidableLabs/victory-chart/pull/286
/cc @ebrillhart @boygirl @kylecesmat

Looks great!

looks good to me pending approval from @boygirl!

@paulathevalley Approved!


Add 'triggerCharacter' option for completions requests

Fixes #21012
@mjbvz I don't think it should be necessary for the editor to send us the trigger character? Shouldn't that always be the character immediately preceding position?
@amcasey You may be interested in using this in visual studio too.

@andy-ms Yes I would expect that it should be but I don't know for sure that this is correct 100% of the time

If it isn't correct, the code I added here will have an assertion failure, so I guess we'd find out..

I don't think it should be necessary for the editor to send us the trigger character? Shouldn't that always be the character immediately preceding position?

We need to distinguish between the cases where the user explicitly asks for completion (ctrl + space) vs when the editor triggers it automatically. the user could trigger completion using ctrl+space after a closing " for example. We could just say we will not support that. i.e. both cases get no completions..

Presently, VS does not use quotes as trigger characters so it has no need to disambiguate start from end quotes when triggering completion.  However, it's nice to know that this functionality could be lit up if someone asked for it.


sendImages

The main page shows an API call for generating images using DALL-E but when I run the example given:
openAI.sendImages(with: "A 3d render of a rocket ship", numImages: 1, size: .size1024) { result in // Result<OpenAI, OpenAIError>
    switch result {
    case .success(let success):
        print(success.data.first?.url ?? "")
    case .failure(let failure):
        print(failure.localizedDescription)
    }
}

it says OpenAISwift has no member 'sendImages'. Maybe I'm doing something wrong?

This works for me. I'm using the build at commit hash: 72f1d8997f4a4a830570bb0487ccd2edacb7da23
func requestDallEImageWithString(_ message: String) {
     let openAI = OpenAISwift(authToken: viewModel.apiToken)
     openAI.sendImages(with: message, numImages: 1, size: .size1024) { result in // Result<OpenAI, OpenAIError>
         switch result {
         case .success(let success):

             if let imageResponseArray = success.data {
                 let responseUrlString = imageResponseArray.first?.url ?? ""
                 // do what you want with the url string
             }

         case .failure(let failure):
             print(failure.localizedDescription)
         }
     }
 }

Yeah, created a new release tag 1.3.0 - please update to use this version 👍🏼


新增request中间件设计

Codecov Report

Merging #236 into master will decrease coverage by 0.07%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master     #236      +/-   ##
==========================================
- Coverage   92.61%   92.53%   -0.08%
==========================================
  Files          41       41
  Lines        1922     1930       +8
==========================================
+ Hits         1780     1786       +6
- Misses         96       97       +1
- Partials       46       47       +1

Impacted Files
Coverage Δ

dataflow/req.go
83.77% <50.00%> (-0.61%)
:arrow_down:

dataflow/dataflow.go
94.21% <100.00%> (+0.12%)
:arrow_up:

Continue to review full report at Codecov.

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


Is there a way to run the Body Tracking Viewer in the other Sensor Orientation modes?

Basically the title says it all: Is there a way to run the Body Tracking Viewer in other sensor orientation modes?

The body tracking algorithm can only be used with Azure Kinect. However, the K4aWrapper implementation for Orbbec Femto Mega and Femto Bolt provides the same interface as Azure Kinect, allowing the use of the Azure Kinect body tracking algorithm.


Issue with computer or this exe file?

I downloaded the right version of the latest release .exe file. I then ran it as administrator, but nothing really happened. Is there supposed to be a pop-up? Or does it just install files elsewhere.

same for me

You have to run fanbox-dl from a command line interpreter.
Please see updated readme https://github.com/hareku/fanbox-dl#usage


QF-869 back link bug

Looks OK. The pipeline has highlighted packages with high and critical vulnerabilities being used in quite a few places, and it looks like one of the steps has failed on that basis


Add license

Please provide a license for this library.
Why it's important:

If you find software that doesn’t have a license, that generally means you have no permission from the creators of the software to use, modify, or share the software. Although a code host such as GitHub may allow you to view and fork the code, this does not imply that you are permitted to use, modify, or share the software for any purpose.
~Source

My favorite ones (simple and permissive):
Unlicense - Public domain, no conditions
MIT - License and copyright notice
More licenses: https://choosealicense.com/

https://github.com/iamcco/async-await.lua/commit/2dfd9531e080a61c2b4f7c8b50c6a5498ab6da79


RancherOS configuration via rancher.yml

It will be great to have a RancherOS vagrant plugin which allow to configure RancherOS via a rancher.yml.

:+1:

We're no longer supporting our vagrant repo and have moved to supporting docker-machine to start up RancherOS. As a result, we will be closing the issues tab in the repo.
Learn how easy it is to use RancherOS with docker-machine.


Slice Renderers look different after reloading session file

Steps: (Updated)

Open VAPOR, load CDF compliant data: /little_glade/shaomeng/cm1.nc
Create a slice renderer. Set the Z location to 3.7 in the Geometry tab. Enable this renderer.
Repeat step 2, with a different Z location value: 6.
Repeat step 2, with a different Z location value: 10.
Repeat step 2, with a different Z location value: 16.
Rotate the scene and observe the 4 planes that are rendered.
Save the current state into a session file.
Close VAPOR, and then re-open VAPOR, and load the saved session file.
Rotate the scene and observe the 4 planes that are rendered. They look different from step 6.

Platform: cisl-vapor
Version: Official 3.2 release.
Before:

After:

@shaomeng  will provide more details on how to reproduce it.

Re-assigned to @StasJ


Integration tests for local slam.

Done with the corrections, only waiting to use TimedPointCloudData from PR #736.


Combine webfont weights

Fixes #95

Were you able to reproduce the bug that render the font incorrectly at the first place (i'm not)?

Looks good! Can you just make the counter (green arrow) with semi bold weight and the label (red arrow) with medium weight:

This should be same as in snapshot.page currently


Move component props into separate files

Some components are reused frequently (hint, error message etc). Move their props out to a separate file so they can be included in other components' prop lists easily without having to redefine all the props. Consider doing this for all components to improve experience for consumer apps that may want to wrap other components

I'm not sure this is possible with how docs are generated at the moment.
We use vue-docgen to parse props from components, which does static analysis and doesn't follow imports etc. It would be great to be able to do this in the future but it would probably require an enhancement to vue-docgen


The Python SDK should never raise an exception

We used to try very hard to make sure the Python SDK never raised an exception to user code but some of this got lost in the move to the new api.
We should:

Catch all exceptions that happen inside Rerun objects and methods so that using Rerun never causes your program to crash.
Instead always do the best we can with the data (usually just dropping it) and logging a Warning.
We need to take special care to handle exceptions in a reasonable way inside constructors of archetypes, components, and datatypes.

The right thing will usually be to just construct it as empty and log a warning

We need to take special care to handle exceptions in a reasonable way inside constructors of archetypes, components, and datatypes.

The right thing will usually be to just construct it as empty and log a warning

Poisoning seems like the most natural approach for this.
E.g. if every archetype constructor returns a AsComponents, we can introduce a PoisonedAsComponents() object that does the right thing when it gets logged: i.e. don't actually log the data but log a warning instead.

Poisoning seems like the most natural approach for this.
E.g. if every archetype constructor returns a AsComponents, we can introduce a PoisonedAsComponents() object that does the right thing when it gets logged: i.e. don't actually log the data but log a warning instead.

Not sure __init__  methods can return objects of a different type

Some additional notes from discussion:

This handling will apply to all Archetypes (and maybe all archetype-like things re-exported to the top-level rerun namespace).
Components and Datatypes will still raise exceptions so they can be used in python object-centric code (e.g. as part of user-custom archetypes) without needing to globally opt out of strict mode.
Where possible we will coerce to a RerunValueError(ValueError).
Archetype constructors will also be extended to include a strict kwarg to override the default behavior in one-off situations.
We won't catch the case of using invalid kwargs and let the interpreter raise the standard "unexpected keyword argument" - anything else will make migration and API changes too painful.

Any component that produces an exception when logging will result in an empty array of that component being logged instead.  Later we could potentially introduce a way of tracking "bad data" all the way through the data pipeline.


refactor: :bug: user not lockout with null date

:x: Build TheIdServer 3.0.1-fix-identity-store-issues1-0096 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/054e01af6b by @aguacongas)

:white_check_mark: Build TheIdServer 3.0.1-fix-identity-store-issues.1+97.build.1442 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/8bd0f9d798 by @aguacongas)


Skip flaky tests for #613

The metrics endpoint tests are flaky because there are no guarantees
that the metrics pipeline has processed events before the metrics
endpoint is read. This can cause CI to fail spuriously.
Disable these tests from running in CI until #613 is resolved.

@olix0r, it looks like these tests are not actually being skipped. I'm trying to figure out what's up with that.


feat: Added Elestio as one-click deploy option

This PR introduces new one-click deployment option Elestio
Fixes: #31408

Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Hmm, attempting to use the "deploy" button seems to result in an infinite loading spinner:

Or at least it's been like that for 10+ minutes. This makes me worry if this platform is too beta to recommend.

Hmm, attempting to use the "deploy" button seems to result in an infinite loading spinner:

Or at least it's been like that for 10+ minutes. This makes me worry if this platform is too beta to recommend.

So sorry you had to wait that long. We are not in beta, we are in prod since 2022 and have several thousands customers. Maybe it was a temporary issue and you could try again? Let me know and we can check into specifics of the issue :)

Seems like the step that hung indefinitely completed immediately this time (it was quite early in the flow). Going to finish setting up an instance so I can verify it is not busted.
Can you confirm whether your images are correctly generated unique secrets for each instance? scripts/setup/generate_secrets.py generates them; see also scripts/setup/generate-self-signed-cert, and you should be able to check /etc/zulip/ between two instances and make sure there's no values in comment in zulip-secrets.conf.
That's the most common mistake setting up pre-installed images like this.

Hey @timabbott, Glad to hear you had better experience deploying the instance this time. To answer you question, Yes we're using the zulip official image and properly passing the required env to generate this env var. You can check it out on our repo https://github.com/elestio-examples/zulip and we're using zulip's official Docker compose. A sample ss for your reference.

Happy to help with any additional information you require :)

So https://github.com/elestio-examples/zulip/blob/main/elestio.yml seems to have the word random_password as a lot of secrets; is that actually the password being used, or are those being substituted with different values somehow?
Also the screenshot on https://github.com/elestio-examples/zulip is a few years old; can you update that?

Hey we have updated the screenshot 🙌  and about random_password It's a kind of variable, when the user has this variable in the value then our frontend will generate a random password for the key.


[Suggestion] Also add sound for compiler errors

Hey there, first of really cool tool!
I very much agree on the Unity Editor lacking a bit on sounds notifications especially for failing/succeeding builds you let run in the background and that take a while. And your solution is even also custom adjustable for each local user 🤩
Just a small idea for an extension on top - I did this modification for me locally:

Also add sounds for a failing compilation. For example like
public static class CompilerNotifications
{
    [InitializeOnLoadMethod]
    private static void Init()
    {
        CompilationPipeline.assemblyCompilationFinished -= ProcessBatchModeCompileFinish;
        CompilationPipeline.assemblyCompilationFinished += ProcessBatchModeCompileFinish;
    }

    private static void ProcessBatchModeCompileFinish(string s, CompilerMessage[] compilerMessages)
    {
        if (compilerMessages.Any(m => m.type == CompilerMessageType.Error))
        {
            EditorAudioUtility.PlayCompilerNotificationSound(EditorNotificationSound.Error);
        }
    }
}

[optional] And then optionally I also added according enabled flag - so two different flags (either bools or enum flag). I therefore also split out the PlayNotificationSound into PlayBuildNotificationSound and PlayCompilerNotificationSound. Might be fine as well though to just stick to a central single one.

Heya @jerome-poenisch ,
Very nice addition. I added your suggestion to version 1.2.0 - With success and fail sounds for compilation and builds.


NDRS-549: include casper-node-macros in the casper-updater tool

This PR updates the casper-updater tool to also update files affected by version changes in the new casper-node-macros crate.

bors r+


chore: [Idle detection] Deal with users on systems that don't handle emoji

As discussed on chat, at least some Windows 7 users can't display emoji. This PR deals with this fact by providing a textual fallback for 💤 ("idle") and ⚫ ("away"). The default state, 🟢, is ignored on purpose.

TS is complaining :)


Move imports in waze_travel_time component

Breaking Change:
None
Description:
Moved import from function to top-level as requested in #27284
Related issue (if applicable): fixes #27284
Checklist:

[ ] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

Whoops, duplicates #27384

ok, closing this one then :)


invalid authentication

I am trying to use the Tuya v2 home assistant integration
However, I am unable to login. I keep getting "Invalid authentication" error message
I checked and rechecked that the Access ID & Access Secret are correct (copied from a new smart home cloud project in iot.tuya.com), and my account and password are the same used to login to iot.tuya.com (as well as the smart life app)
I think I had a similar problem with the older integration, but I don't recall how I managed to login

Checked every step?
https://github.com/tuya/tuya-home-assistant/wiki/Tuya-IoT-Platform-Configuration-Guide-Using-Smart-Home-PaaS?_source=27c468b2abbb019eb4983fc8cea49b90

yes
I am trying to use the same cloud app created to connect to the TuyaV1 integration
The access Id and secret are pretty clear
I am using "Smart Life" for the mobile app field
I am in Israel (and my smart life app displays Israel as the account location), so I am entering the country code (972). I see that my tuya cloud app availability zones is set to: "China, Europe, India, America, Western Europe, Eastern America". Do I need to change that? I could not find any information regarding Israel availability zones in "Correspondence of regions and availability zones"
Lastly, the account and password are the same as those used to successfully login to the tuya cloud
Btw, the cloud app is linked to the smart life app (under "account and security"/"cloud development project"). should I unlink it?

Can you create a ticket and provide your account info through it? Then we can track it.
https://service.console.tuya.com/8/2/create?source=content_feedback

thanks
I have submitted a ticket (# T20210831000388)
btw, is it possible to view the settings used for login of the TuyaV1 integration in home assitant?

Hi, I have the same problem. Is any information how to solve it?

Followed a youtube video describing the cloud project creation
There is a step of adding device notification service to the list of api's supported by the project
Then you have to link with your app (smart life or tuya) by scanning a qr code
Finally, use the user name and password associated with your app when logging in to the integration

I solved a problem another way. I had a project created about a year ago for my tuya v1 original HA integration. I read all manual about connection tuya v2 and add to my old project needed services, but had an error on connection stage. Then i create a new project and integration setup in a few moments. The problem in my opinion is that old projects have not some information, that needed for new integration.

There is a step of adding device notification service to the list of api's supported by the project
Then you have to link with your app (smart life or tuya) by scann

Can you share this video

https://youtu.be/Amc_fmYMQEo


Merge to master

This change is

Reviewed 80 of 80 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Flutter fails at Building flutter tool... on ubuntu 20.04

Steps to Reproduce

I used the git repo to install flutter, instead of using the premade files provided in the documentation page.
This seems to be happening on Ubuntu 20.04, however i am using a DigitalOcean Droplet (a new one). It is hosted in India.
I tried on two different new droplets as well with no luck!
I cloned using this
git clone https://github.com/flutter/flutter.git -b stable --depth 1

I tried it on my macbook, it seems to be working. Could it be on linux only?
Logs

Logs:
/home/code/.development-tools/sdk/flutter/bin/internal/shared.sh: line 153:  8783 Killed                  "$DART" --disable-dart-dev $FLUTTER_TOOL_ARGS --snapshot="$SNAPSHOT_PATH" --packages="$FLUTTER_TOOLS_DIR/.packages" --no-enable-mirrors "$SCRIPT_PATH"

flutter doctor -v output:
Building flutter tool...

Note: Tried the same on beta, dev and master as well.
stable and beta same results
dev  and master  seems to be stuck indefinitely
Any help on this is much appreciated.
Thanks!

Hi @FirzenYogesh
Having you tried on local machine with Ubuntu 20.04? this could be virtualisation issue, not sure if Flutter linux is supported on linux virtualisation

Hi @TahaTesser
I only have a macbook, so couldn't check.
Also I want it on a remote machine because I take my iPad when I am on the move and I code in that. That's why I am trying to install on a remote machine, also if I install using the premade packs from the documentation site, it works only for that build. When I switch to different channel and run flutter upgrade I get the same error.

It seems, I had insufficient RAM during the building process on my server, I put up a swapfile to solve it.
I am closing this.

It seems, I had insufficient RAM during the building process on my server, I put up a swapfile to solve it.
I am closing this.

Could you explain how you put up a swapfile to solve this in details?


nvm list available fails to show 7* versions

While "nvm list available" shows the latest LTS version (6.10.0), no 7.* version is listed - 6.8.1 is latest "current".

There is a bug processing the JSON for v7. I've fixed it and will release in the next update.

OK, this is fixed and shippied in v1.1.2.


Merge upstream

There's quite a bit f new failing tests here for ubuntu. The fix would be to install some fonts on ubuntu image before running tests


Feature Request:  Split's "Join delimiter" deletes after "Split delimiter"

To allow for quick parsing of senders from a domain in an email list, can a delete action be added to the Split Operation's "Split delimiter" where any characters trailing the "Join delimiter" are removed?  Currently, this is being performed by a bulkier Excel "Text to Columns" delimited on "@".
For example (Note: ";" used in place of '\n\r'):
Join delimiter - @
Split delimiter - Deletes trailing
Input - sender1@domain.com;sender2<EMAIL_ADDRESS>Output - sender1;sender2;sender3
Thanks for the consideration.

Hi @ghsdpolley
This can be accomplished by chaining the find and replace recipe onto the split recipe.
Find attached the example:
Regex @(.*) will match everything after the @ symbol
https://gchq.github.io/CyberChef/?recipe=[{"op"%3A"Split"%2C"args"%3A["%3B"%2C"Line feed"]}%2C{"op"%3A"Find %2F Replace"%2C"args"%3A[{"option"%3A"Regex"%2C"string"%3A"%40(.*)"}%2C""%2Ctrue%2Cfalse%2Cfalse]}]&input=c2VuZGVyMUBkb21haW4uY29tO3NlbmRlcjJAZG9tYWluLmNvbTtzZW5kZXIzQGRvbWFpbi5jb20

Thanks @WrathZA !
The source list has each address on a new line, so I was able to just parse through it with the RegEx addition alone.  Processed quickly and I was able to save this as a Recipe for future use.  Appreciate the fast response and help.  RegEx is the next 'language' on my list to learn...  Will close this out, but it would be cool to see a future feature allowing a default option too.  Thanks again!


Add Heights endpoints

Make search like search?height=8848.68
GET /api/v1/search?height_gt=1230.4&height_lt=4000
GET /api/v1/search?height_gt=1230.4
GET /api/v1/search?height_st=1230.4

Hey, I would like to add this heights endpoint feature! Could you assign this issue to me?

Hi I have assigned it to you @Indrajeet-S

Thanks but I can't see in assignees nvm I will start working on this issue!

My bad! Just assigned now

@RishavOsaurus Can I work on this too?

@RishavOsaurus Can I work on this too?

yes you can but better you two guys coooperate and do work distribution so that you dont duplicate the work already done

@RishavOsaurus Can you review this PR. I have tested it out locally with the added search parameters.


Init scrypt-shim repository

The following code got copied from the web3-eth-accounts module. This repository will be used as a dependency in version 1.x and 2.x.

Nice work!
web3-eth-accounts strings should be replaced with scrypt-shim, I think.


Detect Ubuntu releases after 16.04 LTS

Detects Ubuntu releases of 16.04 and beyond (certainly detects 16.10 at any rate!)

php7.0-xml is not a typo. Actually the error was not checking for the JSON and xml PHP packages for both pre and post 16.04. Also there should have been a check where depending on if the user selected to run the lite version where the XML package is installed and if not skipped.
I will be merging this request today and making the necessary changes.

I had to change things up a bit being etc/lsb-release does not exist in Raspbian. Check issue #314 for additional details. commit 793b144f23682ab3cddfe3a97d65f7e5921f1841 addresses this issue.


Remove msrestazure

What does this PR do?
This PR aims to remove the dependency on msrestazure. The bulk of the changes involves replacing the usage of msrestazure.azure_exceptions.CloudError with azure.core.exceptions import HttpResponseError.
Note:
There are still two instances where msrest functionality is required for retrieving cloud metadata. The specific locations where msrest is still used are:

src/saltext/azurerm/clouds/azurerm.py
src/saltext/azurerm/utils/azurerm.py

Codecov Report
Patch coverage: 80.00% and project coverage change: -0.15 :warning:

Comparison is base (a057fec) 51.80% compared to head (0f04b7a) 51.66%.

:exclamation: Current head 0f04b7a differs from pull request most recent head d7e9394. Consider uploading reports for the commit d7e9394 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
- Coverage   51.80%   51.66%   -0.15%
==========================================
  Files          17       17
  Lines         886      871      -15
  Branches       61       55       -6
==========================================
- Hits          459      450       -9
+ Misses        414      411       -3
+ Partials       13       10       -3

Flag
Coverage Δ

Linux
51.66% <80.00%> (-0.15%)
:arrow_down:

project
71.08% <75.00%> (+2.47%)
:arrow_up:

py37
18.33% <4.00%> (+0.47%)
:arrow_up:

py38
51.72% <80.00%> (-0.15%)
:arrow_down:

py39
51.72% <80.00%> (-0.15%)
:arrow_down:

salt_3005_1
51.66% <80.00%> (-0.15%)
:arrow_down:

tests
47.09% <100.00%> (-0.67%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/saltext/azurerm/utils/azurerm.py
70.18% <75.00%> (+2.52%)
:arrow_up:

tests/unit/utils/test_azurerm.py
95.61% <100.00%> (-0.33%)
:arrow_down:

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


DO NOT MERGE -- SPI testing

Remove SPI-like things from this repository to the wildfly-swarm-spi.git repository (and org.wildfly.swarm:spi, :spi-api, and :spi-runtime GAVs).
Refactor this codebase for the different GAV for these previously-mentioned
SPI-like things, along with the packages that changed.
Primarily interfaces:

Fraction
ServerConfiguration
DefaultDeploymentFactory

Additionally, ArtifactManager is now exposed (and slightly hackily)
as ArtifactLookup in the SPI.
Various common bits of ShrinkWrap (DependenciesContainer) have also
moved upstream to the SPI.
Additionally, normalize some of the maven projects within this repository,
so that:

foo/pom.xml
foo-parent aggregator

foo/foo/pom.xml
user-facing org.wildfly.swarm:foo dependency, which more cleanly gathers
together foo-api, foo-runtime and foo-modules with appropriate Maven scopes
to help with intra-repository build/test ordering, and also provides
simple a better handle to the entirety of a fraction.
It is this POM where inter-fraction runtime dependencies are specified.
For instance, jaxrs requires undertow, right? So
jaxrs/jaxrs/pom.xml -> org.wildfly.swarm:undertow

foo/api/pom.xml
Previously was :foo, and now is :foo-api
Has removed reference to :container, since it should only be using stuff
from :spi-api.
In the cases where one API builds upon another, :foo-api may have a dependency
upon :bar-api (not :bar).
If jaxrs-api needed some classes from the Undertow API, it would be
jaxrs/api/pom.xml -> org.wildfly.swarm:undertow-api

foo/runtime/pom.xml
Still foo-runtime, unchanged, aside from depending on :foo-api now
instead of simply :foo

foo/modules/**
unchanged, other than modules/.../api/module.xml now references
the foo-api.jar, as you might hope and expect from a module with
a slot="api"

Ultimately, this change brings about a 33% increase (or so) of
the number of pom.xml's in the project.  You're welcome, Toby.
It also radically reduces the size of each pom.xml. I believe we
had been accumulating dependencies through copy/pasting without
actually realizing if they were truly needed or not.  Those have
been removed where noticed.

We don't need this anymore right?

Kill it!
--
Sent from my iPhone
On Mar 11, 2016, at 10:41 PM, Ken Finnigan<EMAIL_ADDRESS>wrote:
We don't need this anymore right?
—
Reply to this email directly or view it on GitHub
https://github.com/wildfly-swarm/wildfly-swarm-core/pull/12#issuecomment-195651483
.

This has been dealt with


utils not installed with the module

When I installed the package by calling
pip install matplotlib-inline-labels

and ran
from inline_labels import add_inline_labels

I got
---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
Cell In[2], line 1
----> 1 from inline_labels import add_inline_labels

File C:\Python\Python312\Lib\site-packages\inline_labels.py:1
----> 1 from utils import (
      2     # timer,
      3     esc,
      4     Timer,
      5 )
      6 from datatypes import (
      7     Line_Chunk_Geometries,
      8     Label_Rotation_Estimates_Dict,
   (...)
     11     Labels_PRcs,
     12 )
     13 from drawing import (
     14     get_dbg_axes,
     15     retrieve_lines_and_labels,
   (...)
     21     add_dbg_noninlined_labels_legend,
     22 )

ModuleNotFoundError: No module named 'utils'

... there is probably a problem somewhere (possibly on my side, I am not an expert).
My python installation is
> py -V
Python 3.12.3
> pip -V
pip 24.0 from C:\Python\Python312\Lib\site-packages\pip (python 3.12)

Thank you for raising the issue.
It has appeared when I splitted the code into several module (without using subpackages). I’m currently diving into it.
I have a functional package on testpypi.
You can try it with:
pip install -i https://test.pypi.org/simple/ matplotlib-inline-labels==v<IP_ADDRESS>

I still have to solve some side issues with testing github action before releasing a patch.

@Martexiwell it should be fixed now in v0.2.2, available on standard PyPI server


Fix SetName, SetDisplayName bug

Fix bug which forces you to pass force=true to SetName, SetDisplayName and SetDescription.  After fix, you only need to pass foce=true if the stirng value you are passing is null or empty.

Thanks mate, makes sense merging it now and will push a new release as well :)


config file default location

Please, change line with config-file

config_file = ('--config', 'tslint.json')

to

config_file = ('--config', 'tslint.json', '~')

The file will be searched in the project directory. Otherwise the plugin can't find the file by default path and crashes without any message.

Done. Thanks.


Jira7.1 redirect to https

I opened this pull request to start the discussion.
At the moment, I set redirect https only if:
you set the version variable to 7.1.0 and up
and
if you set both tomcat_redirect_https_port and tomcat_native_ssl
It's safe to say that 7.1.0 accept this config and I dont' want to mess the web.xml file if you don't want to redirect. But I would like to know how we determine we need to redirect. I'm pretty sure that you might want to redirect even without $tomcat_redirect_https_port?
Is only tomcat_native_ssl is enough?

Hey @ericlaflamme . I don't see why you're referencing web.xml, it is not the config you are looking for.

Hi @sacres , I'm not sure I follow your comment. I want to make redirect to https work on Jira 7.1 and web.xml is the config file you need to modified to make it work. It came from the official documentation of atlassian:
https://confluence.atlassian.com/adminjiraserver071/running-jira-applications-over-ssl-or-https-802593051.html#RunningJIRAapplicationsoverSSLorHTTPS-Advancedconfiguration

Hi @ericlaflamme . Hmm, yes indeed, all 8277 lines of it 😲
I think we should manage this config file differently. From WEB-INF/web.xml:
<!--
All session-config, mime-mapping, welcome-file-list, error-page, taglib,
resource-ref, security-constraint, login-config, security-role,
env-entry, and ejb-ref elements should follow this fragment.
-->

**FOO BLOCK HERE**

</web-app>

Would you be able to work out something which inserts the required block @ FOO BLOCK HERE  and if so,  also amend the README.md to reflect this.
Thanks!

Hi @sacres, I agree with you that 8000+ lines is obviously not the best thing but what do you suggest? The thing is if we use something like Augeas, we will end up with an additionnal dependancy for injecting 8 lines.
Puppet isn't really good at injecting text.. Puppet is base by the fact that we manage the whole file or not at all.. And then some modules want to solved that but isn't that obvious.
Also, I'm not sure what to modified in the readme since it fixed an already working feature that isn't working anymore on jira 7.1. So it basically make the module work again :)

I really don't want another template in the module that needs to be kept up-to-date with upstream.
This is an understandable feature request, but the price seems way too steep when you could just put a proxy in front of JIRA :/

Given that this PR is over 3 years old and I'm just about to drop support for pre-8.0 JIRA from the module, I don't think there's value in keeping it open.
If people want to manage server.xml, they are able to do it outside this module. We should focus on making common production setups easy rather than trying to support everything.


test for missing calcChain part behavior

This test verifies the fix for #1281.
Question: should this test be broken into two tests, one for the desired exception and one for the non-exception case?

Do you mean add a test with IgnoreExceptionOnCalcChainPartMissing = false?

No, both tests (true and false) are in the one function. I was asking if we should put them in separate test functions.

They should probably be in separate functions. Makes it easier to diagnose failures

btw - I'm not sold on the API design with IgnoreExceptionOnCalcChainPartMissing. I have some ideas I'll noodle on, but may main concern is that (1) it doesn't allow for any additional behavior such as  if we want to remove it, modify it, etc and (2) it is spreadsheet focused. We don't have a better way of handling this at the moment, but would like to see a better design by the time we ship.


chore(main): release 2.0.0

:robot: I have created a release beep boop
2.0.0 (2024-09-24)
⚠ BREAKING CHANGES

add insert query (#42)

Features

add insert query (#42) (2bc1433)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v2.0.0
:sunflower:


Message received and readed events [question]

Is there a way to get events when a message was received or were readed  (double mark, blue mark)?
subscribe_new_messages() WAPI event only fires when message arrives.
Thanks!

You could do this by our own reading the XPATH of the blue double mark.
But for this you'll should create an watcher of every message received i guess.
For example:
When subscribe_new_messages receive one message try for each X seconds read the XPATH of blue mark
This is something that requires to be checked every X times because the person who received your message probally will not see, read and respond to your instantly


ValueError: Shape of variable bert/pooler/dense/bias:0 ((128,)) doesn't match with shape of tensor bert/pooler/dense/bias ([768]) from checkpoint reader.

I have pretrained an albert using my own data(chinese language), then I wanted to use this pretained model into downstream tasks, but I got this error when I load my pre trained Albert file. I guess it's because of some config params issue, but I don't know how to fix it.
my pre-trained Albert-config is:
{
"attention_probs_dropout_prob": 0.1,
"directionality": "bidi",
"hidden_act": "gelu",
"hidden_dropout_prob": 0.1,
"hidden_size": 768,
"initializer_range": 0.02,
"intermediate_size": 3072,
"max_position_embeddings": 512,
"num_attention_heads": 12,
"num_hidden_layers": 12,
"pooler_fc_size": 768,
"pooler_num_attention_heads": 12,
"pooler_num_fc_layers": 3,
"pooler_size_per_head": 128,
"pooler_type": "first_token_transform",
"type_vocab_size": 2,
"vocab_size": 20974
}
I have tried to change "hidden size" and "pooler_fc_size" etc., but it didn't work.
how can I fix this issue?

What was this error? Can you post a stacktrace?

Thanks for replying. I recently tried to train an ancient Chinese language
Albert with my own corpus using google-research Albert, the error occurred
when I used it into downstream tasks with another repo(
https://github.com/brightmart/albert_zh). I have emailed the owner of this
repo, the reason seems to be that there are some code changes in albert_zh,
so google’s pre-trained albert can not be used in albert_zh. Now the
albert_zh training is in progress, I would try again. I will keep u
updated, good day sir.
Sebastian Goodman<EMAIL_ADDRESS>周五02:04写道：

What was this error? Can you post a stacktrace?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google-research/ALBERT/issues/142?email_source=notifications&email_token=ACYRJIG5RPXL6XC57WDX7C3RBRGL3A5CNFSM4KPXTXK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELAGKSY#issuecomment-583034187,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACYRJIDMUVXRV5RN5IXEK23RBRGL3ANCNFSM4KPXTXKQ
.

--
Sent from my iPhone

Thanks for replying. I recently tried to train an ancient Chinese language Albert with my own corpus using google-research Albert, the error occurred when I used it into downstream tasks with another repo( https://github.com/brightmart/albert_zh). I have emailed the owner of this repo, the reason seems to be that there are some code changes in albert_zh, so google’s pre-trained albert can not be used in albert_zh. Now the albert_zh training is in progress, I would try again. I will keep u updated, good day sir. Sebastian Goodman<EMAIL_ADDRESS>周五02:04写道：
What was this error? Can you post a stacktrace? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#142?email_source=notifications&email_token=ACYRJIG5RPXL6XC57WDX7C3RBRGL3A5CNFSM4KPXTXK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELAGKSY#issuecomment-583034187>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACYRJIDMUVXRV5RN5IXEK23RBRGL3ANCNFSM4KPXTXKQ .
-- Sent from my iPhone
I can help u
我就 用中文吧。
你可以对比这两版albert的代码，你就发现他们做embedding的过程是不一样的，brightman在embedding的时候就从128映射到768了，而google是在tramsformer里做的映射。


[sdg] Use unDataLabel for place names

do this for map, scatter, line, and bar

thanks for the reviews!


removed ref to video on azure.microsoft.com site

update as the link to the kudu video (on azure.microsoft.com) was dead (resource removed)
made the resulting text clean again though :)

@timschps : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off