Support QueryMode=PROFILE a.k.a EXPLAIN ANALYZE

Currently, EXPLAIN statement work as QueryMode=PLAN.
I want to see the result of QueryMode=PROFILE.
It may be similar to EXPLAIN ANALYZE of PostgreSQL.
https://www.postgresql.org/docs/12/sql-explain.html
Considerations:

What runtime information is displayed?

I think we can show same type of information as the Spanner shows in the console.
Console output

Example output of spanner-cli
spanner> explain analyze select * from Singers@{FORCE_INDEX=SingersByLastName};
+------------------------------------------------------------------------------------------------------+---------------+------------+---------+
| Query_Execution_Plan                                                                                | Rows_Returned | Executions | Latency |
+------------------------------------------------------------------------------------------------------+---------------+------------+---------+
| .                                                                                |               |            |         |
| └── Distributed Union (subquery_cluster_node: 1)                                                     | 2             | 1          | 8.14 ms |
|     └── Distributed Cross Apply (subquery_cluster_node: 11)                                          | 2             | 1          | 8.12 ms |
|         ├── Create Batch                                                                             | -             | -          | -       |
|         │   └── Distributed Union (call_type: Local, subquery_cluster_node: 4)                       | 2             | 1          | 7.93 ms |
|         │       └── Compute Struct                                                                   | 2             | 1          | 7.92 ms |
|         │           └── Scan (Full scan: true, scan_target: SingersByLastName, scan_type: IndexScan) | 2             | 1          | 7.91 ms |
|         └── Serialize Result                                                                         | 2             | 1          | 0.12 ms |
|             └── Cross Apply                                                                          | 2             | 1          | 0.12 ms |
|                 ├── Scan (scan_target: $v2, scan_type: BatchScan)                                    | 2             | 1          | 0 ms    |
|                 └── Distributed Union (call_type: Local, subquery_cluster_node: 17)                  | 2             | 2          | 0.05 ms |
|                     └── FilterScan                                                                   | 2             | 2          | 0.04 ms |
|                         └── Scan (scan_type: TableScan, scan_target: Singers)                        | 2             | 2          | 0.04 ms |
+------------------------------------------------------------------------------------------------------+---------------+------------+---------+
13 rows in set (0.40 sec)
timestamp: 2020-05-25T18:44:03.049295+09:00
cpu:       4.08 msecs
scanned:   4 rows
optimizer: 2

Considerations

How to show the summary data (cpu time, scanned rows). The above example shows them as if it run with verbose option (-v).
Some Map operations has distribution values for the latency. How should we show them properly?

How to show the summary data (cpu time, scanned rows). The above example shows them as if it run with verbose option (-v).

I expect "EXPLAIN ANALYZE" to emit summary lines but
there seems to be no strong reason to change the format from -v.

Some Map operations has distribution values for the latency. How should we show them properly?

Let me organize.
The displayed values in the console seems:

heading
value

Rows returned
rows.total

Executions
execution_summary.num_executions

Latency
latency.mean

And values in tooltip seems:

heading
value

Total executions
execution_summary.num_executions

Mean time per execution
latency.mean

Standard deviation
latency.std_deviation

Total latency
latency.total

So five values are needed to cover all information in the console.
Below is just idea(seems long).
+------------------------------------------------------------------------------------------------------+---------------+------------+------------------------+
|                                                                                |               |            |        Latency         |
|                                                                                |               |            +---------+--------------+
| Query_Execution_Plan                                                                                | Rows_Returned | Executions | Total   | Mean±StdDev  |
+------------------------------------------------------------------------------------------------------+---------------+------------+---------+--------------+
|             └── Cross Apply                                                                          | 2             | 1          | 0.12 ms |              |
|                 ├── Scan (scan_target: $v2, scan_type: BatchScan)                                    | 2             | 1          | 0 ms    |              |
|                 └── Distributed Union (call_type: Local, subquery_cluster_node: 17)                  | 2             | 2          | 0.11 ms | 0.05±0.04 ms |
+------------------------------------------------------------------------------------------------------+---------------+------------+---------+--------------+

Thank you for summarizing that.
Considering we have \G separator, multi-line header would be not natural.
If we show all of items in a single line, it would look like this.
+------------------------------------------------------------------------------------------------------+---------------+------------+---------------+---------------------+
| Query_Execution_Plan                                                                                | Rows_Returned | Executions | Latency_total | Latency_mean_stddev |
+------------------------------------------------------------------------------------------------------+---------------+------------+---------------+---------------------+
|             └── Cross Apply                                                                          | 2             | 1          | 0.12 ms       |                     |
|                 ├── Scan (scan_target: $v2, scan_type: BatchScan)                                    | 2             | 1          | 0 ms          |                     |
|                 └── Distributed Union (call_type: Local, subquery_cluster_node: 17)                  | 2             | 2          | 0.11 ms       | 0.05±0.04 ms        |
+------------------------------------------------------------------------------------------------------+---------------+------------+-------------------------------------+

I'm not quite sure how latency.mean and latency.std_deviation is useful for users to investigate the query.

I am wondering whether supporting \G is required in all statements.
Either way, terminal width is limited so displayed column should be minimal.
Example
IMHO, total latency is intuitive rather than mean latency.(Σ is known as ambiguous width character unfortunately.)
+------------------------------------------------------------------------------------------------------+----------+-------+----------+
| Query_Execution_Plan                                                                                | Returned | Exec. | ΣLatency |
+------------------------------------------------------------------------------------------------------+----------+-------+----------+
|             └── Cross Apply                                                                          |   2 rows | 1     |  0.12 ms |
|                 ├── Scan (scan_target: $v2, scan_type: BatchScan)                                    |   2 rows | 1     |     0 ms |
|                 └── Distributed Union (call_type: Local, subquery_cluster_node: 17)                  |   2 rows | 2     |  0.11 ms |
+------------------------------------------------------------------------------------------------------+----------+-------+----------+

It seems no visualizer fits all use case.
I am trying to write external formatter which receive jsonpb representation of ResultSet from gcloud spanner databases execute-sql --query-mode=PROFILE --format=json to achieve my own use case.

This is trivial thing, but I don't think it's effective to shorten the length of the column names as the most width-dominant column could be Query_Execution_Plan in the most cases. Rather than abbreviating some column names, I prefer to choosing accurate, descriptive names.

IMHO, total latency is intuitive rather than mean latency.

I agree. This is not consistent with console UI, but I believe total latency is more useful to investigate the query.

I have a different opinion.
Wrapped table layout is hard to read and width of terminal is limited in some cases(screenshots, large font in presentation, tiling window).
Because 1 depth needs 4 characters, reducing 12 character allows to display 3 levels deeper plan.

Examples of long width table in markdown, text and image.
https://gist.github.com/apstndb/beea6021d5209a30f25b218c89b9fc32

What I wanted to say is that the Query_Execution_Plan column still account for the most part of the width and even if we shorten the length of the other columns, I doubt that the entire result can be shown in the limited screen.
So if we want to care about the width of the result, I think we also need to consider if we can do to shorten the Query_Execution_Plan column as well. For example, we can wrap long texts with tablewriter.
Anyway, let's implement this and see how it looks like.

@apstndb Can you take both #57 and #58? I can take either of tasks, so please feel free to ask me.

I agree that we need to consider wrap text to fit terminal width in tablewriter.
There are another idea to reduce Query_Execution_Plan value width by composing metadata.

scan_target: SingersByLastName, scan_type: IndexScan can be shorten to Index: SingerByLastName.
call_type can be composed to node name, e.g.

Distributed Union (call_type: Local) can be shorten to Local Distributed Union
Sort Limit (call_type: Global) can be shorten to Global Sort Limit

This notation is consistent with Query execution operators
 and Query execution plans

Can you take both #57 and #58? I can take either of tasks, so please feel free to ask me.
#57 is already concrete so I'll do it.

There are another idea to reduce Query_Execution_Plan value width by combining metadata entries.

Yeah sounds goods to me 👍


Add auto merge

@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/atomman-feedstock#8.


LEGO: Pull request from lego/hb_a12a4630-4852-4e7d-9cbc-c0e1117da1ab_20210506194628507 to main

LEGO: Pull request from lego/hb_a12a4630-4852-4e7d-9cbc-c0e1117da1ab_20210506194628507 to main with localized lcls

closing since there are no changes left


How to do prediction?

Hi
Thanks for your great work.
Could you please explain how to detect and localize  anomaly or generate a heatmap for the test image with the .pt file generated?

Hey. I recommend using the custom dataset runner. Just put at least one image in custom/test/classX/normal, custom/test/classX/anomalous, and custom/train/classX/normal. You can also put the same image in all folders in case you want to use just one. Then, run python runners/run_custom.py --supervise-mode unsupervised --it 1 -e 0 --load MY_SNAPSHOT.pt.  You can find the generated heatmaps in the log directory.
If you haven't used the VGG net, you need to change the architecture with -net. Note that, if your architecture expects inputs of different shape than (3, 224, 224), you need to modify the transforms here. In fact, for the best results you could change the transforms to use the mean and std of your training data.
How did you train your model? You should get heatmaps for images of the test set after having trained the model. Isn't that sufficient already?

Another comment on this. If you want the raw pixel values of the heatmaps that are generated, you can find these in the "tims" directory and load them via torch.load("heatmaps_paper_global_lbl0.pth").
If you want a script that loads the snapshot of a model and generates pixel-wise anomaly score tensors, I added such an implementation here.
I hope this helps.

Another comment on this. If you want the raw pixel values of the heatmaps that are generated, you can find these in the "tims" directory and load them via torch.load("heatmaps_paper_global_lbl0.pth"). If you want a script that loads the snapshot of a model and generates pixel-wise anomaly score tensors, I added such an implementation here. I hope this helps.

I have run run_prediction_with_snapshot.py,  and get the all_anomaly_scores tensor，but how to judge whether it is abnormal through pixel-wise anomaly score tensors?  and how to get the heat map through the run_prediction_with_snapshot.py method?
@liznerski

another question：
I only put one picture,and the path is shown in the figure:

then I ran python runners/run_custom.py --supervise-mode unsupervised --it 1 -e 0 --load ../../data/test/model/snapshot.pt --net FCDD_CNN224_VGG_F
Get the following result：

heatmaps_global.png as shown:

heatmaps_paper_global_lbl0.png:

heatmaps_paper_global_lbl1.png:

heatmaps_paper_local_lbl0.png:

heatmaps_paper_local_lbl1.png:

train_heatmaps_global.png:

so, why does run_custom.py try to process all the classes? For example, I did not place the cable picture, but it still generates normal_1? I just want to perform anomaly detection on the bottle image I set, and get its classification result and heat map. How can I do it?

Commented in #22.


Why divergent sequences cluster together using mmseqs2 easy-cluster?

Dear,
I am using mmseqs2 to remove redundant sequences and isoforms from eukaryotic proteomes. However, we obtained some unexpected and undesired clusters and we would like to understand what is going on. We would like to know if this result is actually the expected one (which I'm not sure beucase both clustered seqs are divergent), or if this is a bug, or something that we do not consider in the command line.
We used the proteome of Trichoplax adhaerens (https://www.uniprot.org/proteomes/UP000009022).
## Expected Behavior
We have focused on two sequences as example that should be in different clusters, B3RQY5 and B3S420.
The sequence similarity of both sequences is the following (aligned with needle):

Length: 643
Identity:     146/643 (22.7%)
Similarity:   199/643 (30.9%)
Gaps:         358/643 (55.7%)
Score: 682.5

B3S420
DHVFANENDNSDVYQKVASPIVTAAMEGFNGTIFAYGQTSSGKTHTMMGNHNDPGVIPLA
VNEIFRYINQKPNREFLLRVSYMEIYNEVITDLLNPSNTNLKIHENQKKEVYVGSLTENI
VNSPSQILTIMTQGETHRHTGGTNMNERSSRSHTIFRMIIESREQNQDQNEADQDTAVKV
SALNLVDLAGSERVSQTGSEGIRLKEGGFINKSLLTLGSVIAKLSEGEGNFIPFRDSKLT
RILQSSLGGNALTAIICTVTPVSLDETSSTLKFASRAKKIKNKPEVNEVVDDE
B3RQY5
MNSEDACNIRVVCRVRPLNSAETHAGSEFIPKFPNEGQIVLSGKSFSFDHVLNSSTNQQS
MYDIAAKPIVKDVLAGYNGTIFAYGQTSSGKTHTMEGVIGDPEWQGIIPRIIGDIFAYIY
TMDENLEFHIKVSYFEIYMDKIRDLLDVTKTNLAVHEDKNRIPYVKNITERFVSSPEEVF
EIIDEGKSNRHVAVTNMNEHSSRSHSIFLIHIKQENVETHKSVHGKLYLVDLAGSEKVSK
TGAEGMVLDEAKNINKSLSALGNVISALSEATKSHVPYRDSKLTRILQESLGGNARTTII
ICCSPSSINESETKTTLQFGARAKTIKNSVKVNEELPAEEWKRRYEKEREKSSRIKRVLE
NYELELKKWRDGENVPVNEQAGGKDEGKLTSNHSTSKINIADALGESERVQFDEERNRLY
EQIDEKDDELNNRNTLIEQLRRQLEDKDEEFHLIKNESTRRQAQINALEDELQDSKDEVK
EVLNALEELYVNFDEKSRQLEVKSQEYEKNLEELMGIKSKLSNMEENFEETKDTEKRYKR
RVTESIKNLLQDMHEIGDVLQDEELKTAIAKDSEKVSDEELTLARLHFGKIKGELKILVS
RNHTIESERAELEKKLNVSEANLSENQLLLTEACF

## Current Behavior
Both sequences are in the same cluster using easy-cluster.
## Steps to Reproduce (for bugs)
We used this command to obtain such result:
mmseqs easy-cluster TAHD.fasta. TAHD tmp --cluster-mode 2 --cov-mode 1 -c 1
## Context
We have tried to make the clustering with the option easy-linclust, and works 'fine', i.e. both sequences are in different clusters.
## Your Environment
MMseqs2 Version: 113e3212c137d026e297c7540e1fcd039f6812b1
Using mmseq binary from Eggnog mapper
In a HPC cluster.
(the same result is replicated in other systems)
Thank you very much in advance,
Carlos Santana Molina

This was also a while ago, however for clustering you should pretty much always supply a sequence identity threshold with --min-seq-id.
The cascaded clustering of MMseqs2 can still put together sequence outside the given thresholds. We have a separate parameter --cluster-reassign 1 to "fix" the clustering after the cascaded clustering.


Updates for Cacti 1.x compatibility

This is a large update to the Weathermap functionality that brings full support to Cacti 1.x, a few security issues and a rash of updated functionality.  See the CHANGELOG.md for details.  Consider this a BETA pull request.

Okay, did a few updates to capture some of @thurban comments, remove some remaining legacy getopt calls and on check and added some, but not all display_version() and display_help() functions for CLI's.

When attempting to delete a link, the dialog says Node on the button

If you had a custom image in the base image directory, it wants to move it, and fails when it doesn't have permissions.  I would expect these errors to show in the log instead of in the editor.

Okay, Mark.  Got them all addressed.


feat(gh_112) - BaseVisualCamelEntity for Pipe

relates to - https://github.com/KaotoIO/kaoto-next/issues/112

fixes #112


Getting Error in latest source of Quick start project with ASP.NET Core 3.1

I'm getting error from latest source code, the assembly IdentityServer4.Validation is not loading from the source code after integrating with my project using powershell script

Not enough into.

Hey @brockallen ,
I think @gowthamece says that the latest commit for Quickstart repo makes ~25 compilation errors.
For instance, the VS 2019 says it can't find ParsedScopeValue and ApiScope classes in DeviceController.cs.
Also, there it can't access to ValidatedResources property of request ( DeviceFlowAuthorizationRequest type ) that is received from here -> https://github.com/IdentityServer/IdentityServer4.Quickstart.UI/blob/master/Quickstart/Device/DeviceController.cs#L148

Thanks!

Hey @gowthamece,
I did manage to avoid the above errors by checkouting at Dec 20 date for Quickstart project.
PS: Exactly this commit - https://github.com/IdentityServer/IdentityServer4.Quickstart.UI/tree/0ae3fa9f5f5bf5cd1223b3f27882b431adc4b1d9

cc: @brockallen

Hi @KostetskyiYaroslav ,
Exactly this is what the issue I'm facing with VS 2019, let me try https://github.com/IdentityServer/IdentityServer4.Quickstart.UI/tree/0ae3fa9f5f5bf5cd1223b3f27882b431adc4b1d9

@KostetskyiYaroslav @gowthamece did you manage to solve the issue? Thanks!

@KostetskyiYaroslav @gowthamece did you manage to solve the issue? Thanks!

Yes, thanks for the support 👍


Use an existing method for validating params in ParamsTask

While I was debugging some changes for #1116, I noticed that if anything is entered in text_integer_attr2, we get a 500 error.

Yes I just noticed it wasn't related to the other PR, so I thought I'd fix it separately, but if we're changing it there then there's no need for this.


☂ iOS 17 CI issues

Tracking issue to keep track of iOS 17 CI issues
Critical:

[x] https://github.com/flutter/flutter/issues/141383

Workaround added to recipes

[x] https://github.com/flutter/flutter/issues/142917

Test temporarily put in staging to prevent failing of tree: https://github.com/flutter/flutter/pull/142918
Recipes fix added

Non-critical:

[ ] https://github.com/flutter/flutter/issues/142448

Does not seem to happen very often - have not seen in long time

[x] https://github.com/flutter/flutter/issues/141624

Happened very rarely, preventative measures added

[x] https://github.com/flutter/flutter/issues/142307

Issue seems to be specific to a particular bot

[x] https://github.com/flutter/flutter/issues/141787

Only happened once, has not happened again since

[x] Benchmarks negatively impacted by upgrading to iOS 17:

Not actionable
https://github.com/flutter/flutter/issues/142989
https://github.com/flutter/flutter/issues/143945

[x] https://github.com/flutter/flutter/issues/142125
[ ] https://github.com/flutter/flutter/issues/144146

Only seen once

All issues are either resolved or have not occurred in a long time. Closing.


Dev: Restrictions on trait "ResponseTrait"

About files : /system/API/ResponseTrait.php

For issues where description only supports string in ResponseTrait trait
I checked the following issues
#2176
#1702
#420
Especially "failValidationError"
In the case of functions, aren't there many cases where the description is an array?
Arrays are supported in the "fail" function,
Wouldn't it be nice if it was improved to allow returning arrays from other functions in trait?

Have a nice day
Thank you

It might be nice, but the string-only version is published and would require a major version because updating the signature would break any existing extensions: https://3v4l.org/s2L1T

If you feel strongly about this them raise some discussion on the forums and we can add it as a request for the version 5 brainstorm: https://github.com/codeigniter4/CodeIgniter4/projects/6


"Create a permission level" section is inaccurate

Step #2 is incorrect ("On the Site Settings page, in the Users and Permissions section, choose Site permissions.").  The new navigation path in SharePoint Online seems to be: Gear icon > Site Permissions > Advanced Site Permissions. Thanks

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

ID: db58d185-0ee0-84a4-1569-3ff03555f387
Version Independent ID: 95bd3846-4772-6130-a552-0f31df69053d
Content: How to create and edit permission levels
Content Source: SharePoint/SharePointOnline/how-to-create-and-edit-permission-levels.md
Service: sharepoint-online
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@chrisweis It will depend on if you're using Classic or Modern sites. Yes, the steps you describe is correct for Modern. We'll have to create a split in content, here.
@AndreaBarr can you assign to me, please?

@chrisweis We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

@officedocsbot close

Why was this closed?  The comment above from tseward acknowledged the issue, no?

@chrisweis it was closed as there was a fix implemented (that's what the purple Merged icon means when you read the thread). You can see the changes here: https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/pull/849


Implement distance of time in words

Symfony 1.4 date helper has a nice function to display strings  like "less than a minute" when given at least a datetime. It would be nice to have in Twig.

nice, but not in core

https://github.com/knplabs/KnpTimeBundle :)

@stof: nice! Thanks! I won't close the issue though because I don't know whether you plan to merge this bundle into Twig.

I don't think this is someone we want to add to the core. even the nl2br or the truncate filters are in the official extension repo instead of the core.
Thus, this bundle uses the Sf2 translator so it cannot be merged into Twig without removing the i18n feature

this can also be solved client-side by reusing Github's jQuery library relatize-date.
https://github.com/github/jquery-relatize_date
Take a look at how github formats date tags also for how to use it.  Works very well, and may be a better idea when considering how spiders crawl - they get to see an absolute date/time, and the average user gets a relative date/time within a certain window.

That's not something that belongs to the core mainly because there is no "standard" and also because it needs to be internationalized.

@damianb: good idea! Another advantage of doing this client-side is that you can still cache the page.


Make sure that everything works well for the 1920x1200

Looks like this is the one it's actually going to be used for.

To add onto this:
@willangley: (Classes - Access modifiers)
This slide doesn't fit on my MacBook's screen, and there's little indication I need to scroll it.
When navigating with arrow keys sometimes causes it to position slide from bottom up 😰

Also seemingly(??? still haven't been able to nail this down and don't know the cause) based on the size of the window when the index is loaded subsequent or previous slides are sized funny and/or have weird features like this one I just discovered (background is scrollable)


Update vscode to 1.49.0

This PR was automatically generated by nixpkgs-upkeep.
Internal tag: asdfasdf

FYI, there's already a PR updated vscode: https://github.com/NixOS/nixpkgs/pull/97950.


Any dependencies on AMD IOMMUv2?

Some annoying memory mapping failures, curious if anything in ROCM or AOMP expects to be running on a device with an AMD IOMMU.

It used to be the opposite - ROCM expected to be running with IOMMU disabled. AOMP will inherit this requirement. That may have changed in the last couple of months.


Download Link Incorrect for QGIS 3.16.7 LTR

It appears the download link for QGIS 3.16.7 LTR for MacOS still references the previous 3.16.6 LTR version to be downloaded.
(See https://qgis.org/en/site/forusers/download.html) and the resulting module that is downloaded still says 3.16.6-Hannover when after installation (Dated April 28, 2021, and is 3.2GB size).
Please provide the updated QGIS 3.16.7 LTR module for MacOS. Thanks!

all available versions can be listed here https://qgis.org/downloads/macos/ltr
the last version is
https://qgis.org/downloads/macos/ltr/qgis_ltr_final-3_16_8_20210619_044746.dmg


Fix unused parameters in tscore/ink_cap

Use /* name ATS_UNUSED */ for fully unused parameters
Use [[maybe_unused]] name for conditionally unused parameters

Tried building with -DENABLE_POSIX_CAP=ON and -DENABLE_POSIX_CAP=OFF for the OSX preset.
Both builds did not report any errors. I guess, this doesn't have the desired effect because I'm building on Linux.
I don't have access to OSX.
Fixed these from the CI report here

Cherry-picked to v10.0.x


Update GlslOut.hx

specify the shader version.
1.3 being the lower version which allow precision qualifier  (glsl defaut version being 1.1)

Could you check if it runs with WebGL ?
http://stackoverflow.com/questions/21103484/is-it-possible-to-run-version-120-shaders-with-webgl

cleaned a bit since pull request didn't came from the repo I expected (personal and not motiontwin)
Concerning the precision qualifier on WebGL, it is not supported indeed.
I'll make an other change and test it.


delete dweet

Deleted dweet results in blank page with no info

I think "remix of [deleted]" could also be included in this issue:

What should happen to the remix of a deleted dweet?
On main-dwitter remix of [deleted] is shown, on beta-dwitter it still shows the link to the deleted dweet


Introduce a new --dbname-filter flag

Allows to blend out any activities from DBs that are not of interest - DB size, TPS, running / waiting / blocking queries. To understand when a regex filter is specified I've added a star (*) to the DATABASE column...hopefully this is enough. Looks like that:

Hi,
I was thinking about adding more filters, it's great you made this patch !
I will try to have a stab a it this week or next week.
Thx a lot for the patch.

Just tested it; works as expected.
Maybe we should add that the regexp are documented in the following link in a FAQ entry and/or in the man page ?
https://www.postgresql.org/docs/current/functions-matching.html#POSIX-SYNTAX-DETAILS

If we want to add more filters, maybe it'd be nicer to have a single --filter option that would take a key/value to indicate the object we filter on; e.g. --filter=dbname:<regex> --filter=username:<regex>.

@blogh Thanks for verifying that it works! But about adding the link to regex docs...probably just overkill...as the people using this tool, especially the more exotic flags, are already very technical people and surely know what a regex means / looks like :)

@dlax Sure, not against the idea...but then the architecture for such generic filters probably needs to come from your side.
The hardest part will probably be keeping the SQL templates more or less sane in case of dynamic input...

@kmoppel I am not sure about that, but we can wait about the questions and add the FAQ entry then.

Added the generic --filter flag; can you give it a try?

I like the changes :)

I've cleaned up the git history.
This looks good to me now so we can merge this I think.
@blogh, what do you think?
(I'll fix the lint issue, due to mypy new version, in the meantime.)

Merged.
Thank you @kmoppel for the patch and @blogh for the review!


Animation not showing up.

I build an Animation with After Effect and am trying to render it in my react-native app.
It doesn't crash or anything, but the screen stays black. I also had to manually edit the JSON file to fix this issue: https://github.com/airbnb/lottie-react-native/issues/118
The example animations are working fine, so I don't think it can be an implementation issue. Also, the files are loading properly on the web version.
Here is my configuration for the creation of the bodymovin files:
AE: <IP_ADDRESS>
bodymovin: 4.6.7

Here is my json file:
https://gist.githubusercontent.com/NicolasBonduel/69c880c7250c31d9fd3d29a6c24c0d59/raw/1c16f1ea5e2fe8fcfcc0c9788ceb0c9049bfab6b/bodymovin

Should your animation look like to this?

Yes, I got it to work following this: https://github.com/airbnb/lottie-react-native/issues/78#issuecomment-286532808
It is rendered properly now.

Ok nice!
Weird, I don't have this constraint. I can import my animation from anywhere and play it with the Animation component.
import Animation from 'lottie-react-native';
import anim from '../../assets/anims/MyAnim.json';

// ...

render() {
  return <Animation source={anim} />;
}

ohhhh, I was importing it with require(), is it in any way different?
import Animation from 'lottie-react-native';
const anim = require('../../../assets/animation.json');

// ...

render() {
    return <Animation source={anim} />;
}

Having the same problem on android.

Having the same problem too, i fixed the frame rate in the JSON, but now it is showing a black screen without anything, and without errors

Guys ! i found a workaround !
for lottie-react-native, it is possible that the JSON contains "mistakes".
when you export your animation with bodymovin in After Effects, you have to check your json to be sure it always have a "right" number for the framerate and for the "op" section. (not sure of what it is)
For example, if your JSON contains a 29.97 value, you should replace it with "29" or "30".
This work for me

That issue is caused by this: https://github.com/airbnb/lottie-react-native/issues/128 and will be fixed soon.

is the issue solved


Create andressa.html

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


Generate .designlintrc file from Lona workspace

Use like this:
node src/main.bs.js workspace designlint PATH_TO_WORKSPACE .designlintrc
Feel free to leave the PR open until we release designlint

@jongold will designlint ever be open sourced? 😄

I'm not working on it anymore, not sure. I could ask if I can give you the
code
On Tue, Nov 6, 2018, 9:20 AM Devin Abbott<EMAIL_ADDRESS>wrote:

Closed #166 https://github.com/airbnb/Lona/pull/166.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/airbnb/Lona/pull/166#event-1949584247, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAkHG8ELXMqNyRsw4UzToi2N_nB0QZZHks5uscTegaJpZM4U44yH
.im

I'd love to have a look :)


Update books to current state

Deploy preview for openbuildservice ready!
Built with commit 4e8a8d407a992dc1a23b56afab4042ceea4634e3
https://deploy-preview-211--openbuildservice.netlify.com


Fix code scanning alert - Use of password hash with insufficient computational effort

Switch hashing algorithm to bcrypt?
https://security.stackexchange.com/questions/133239/what-is-the-specific-reason-to-prefer-bcrypt-or-pbkdf2-over-sha256-crypt-in-pass
Tracking issue for:

[ ] https://github.com/EffnerApp/EffnerApp/security/code-scanning/1

outdated version, moving issue to new repo


Add registration setup option to avoidResolutionScopeTracking and expose the respective attribute in MEF

Related to #320

I have checked it on the current version and it is hard to add without breaking changes and without the pains for the latter improvements, so postponing for v5.


Fix non-Windows dotnet build

cc @victorhurdugaci @bricelam
KoreBuild tries to restore/build our test UWP app...does not work on xplat for obivous reasons.

Would renaming the project.json to Microsoft.Data.Sqlite.FunctionalTests.UWP.project.json work?

Yup, let's do that. I didn't know VS tooling would still recognize <project name>.project.json.

Oh what knowledge you can gain from spelunking MSBuild scripts! :wink:

:shipit:

Depends on https://github.com/aspnet/Universe/pull/365

This will also be fixable when this goes in https://github.com/aspnet/Universe/issues/360

But that should be okay as long as your build system understands different names other than project.json

In this case, we want CI build system to skip the UWP test project but VS to still work.


Added a better example using --dest-folder

This is a better more explicit example for --dest-folder.

:shipit:


Feat(exporter-tokens): Sort typography tokens and remove unused param…

…eters

Description

Additional context

Issue reference

TODO increase exporter.json version


fix configuration option for zip with php 7.4

I added the --with-zip for the 7.4
#28

I just rebased to trigger CI.

Thanks @baldarn for your contribution!


Deployment failed for WLS cluster with application gateway and self-signed certificate

Problem
When deploying a WebLogic cluster with application gateway and TLS/SSL certificate option Generate a self-signed certificate using Oracle WebLogic Server Cluster on Azure VMs offer, the deployment failed with the following raw error message:
{
  "code": "DeploymentFailed",
  "target": "/subscriptions/260524c9-7a4d-4483-8d85-de54f9c40ae8/resourceGroups/jiangmatestwls/providers/Microsoft.Resources/deployments/keyVaultLinkedTemplate",
  "message": "At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.",
  "details": [
    {
      "code": "ResourceDeploymentFailure",
      "target": "/subscriptions/260524c9-7a4d-4483-8d85-de54f9c40ae8/resourceGroups/jiangmatestwls/providers/Microsoft.Resources/deployments/keyVaultwithSelfSignedAppGatewaySSLCert",
      "message": "The resource write operation failed to complete successfully, because it reached terminal provisioning state 'Failed'.",
      "details": [
        {
          "code": "DeploymentFailed",
          "target": "/subscriptions/260524c9-7a4d-4483-8d85-de54f9c40ae8/resourceGroups/jiangmatestwls/providers/Microsoft.Resources/deployments/keyVaultwithSelfSignedAppGatewaySSLCert",
          "message": "At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.",
          "details": [
            {
              "code": "ResourceDeploymentFailure",
              "target": "/subscriptions/260524c9-7a4d-4483-8d85-de54f9c40ae8/resourceGroups/jiangmatestwls/providers/Microsoft.Resources/deploymentScripts/script-createAddCertificate",
              "message": "The resource write operation failed to complete successfully, because it reached terminal provisioning state 'failed'.",
              "details": [
                {
                  "code": "DeploymentScriptError",
                  "message": "The provided script failed with multiple errors. First error:\r\nMicrosoft.PowerShell.Commands.WriteErrorException: This module requires Az.Accounts version 2.13.2. An earlier version of Az.Accounts is imported in the current PowerShell session. Please open a new session before importing this module. This error could indicate that multiple incompatible versions of the Azure PowerShell cmdlets are installed on your system. Please see https://aka.ms/azps-version-error for troubleshooting information.\r\nat <ScriptBlock>, /root/.local/share/powershell/Modules/Az.ManagedServiceIdentity/1.2.0/Az.ManagedServiceIdentity.psm1: line 93\r\nat <ScriptBlock>, /mnt/azscripts/azscriptinput/userscript.ps1: line 55\r\nat <ScriptBlock>, <No file>: line 1\r\nat <ScriptBlock>, /mnt/azscripts/azscriptinput/DeploymentScript.ps1: line 295. Please refer to https://aka.ms/DeploymentScriptsTroubleshoot for more deployment script information.",
                  "details": [
                    {
                      "code": "DeploymentScriptError",
                      "message": "Microsoft.PowerShell.Commands.WriteErrorException: This module requires Az.Accounts version 2.13.2. An earlier version of Az.Accounts is imported in the current PowerShell session. Please open a new session before importing this module. This error could indicate that multiple incompatible versions of the Azure PowerShell cmdlets are installed on your system. Please see https://aka.ms/azps-version-error for troubleshooting information.\r\nat <ScriptBlock>, /root/.local/share/powershell/Modules/Az.ManagedServiceIdentity/1.2.0/Az.ManagedServiceIdentity.psm1: line 93\r\nat <ScriptBlock>, /mnt/azscripts/azscriptinput/userscript.ps1: line 55\r\nat <ScriptBlock>, <No file>: line 1\r\nat <ScriptBlock>, /mnt/azscripts/azscriptinput/DeploymentScript.ps1: line 295"
                    },
                    {
                      "code": "DeploymentScriptError",
                      "message": "System.Management.Automation.CommandNotFoundException: The 'Get-AzUserAssignedIdentity' command was found in the module 'Az.ManagedServiceIdentity', but the module could not be loaded. For more information, run 'Import-Module Az.ManagedServiceIdentity'.\n   at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext, Exception exception)\n   at System.Management.Automation.Interpreter.ActionCallInstruction`2.Run(InterpretedFrame frame)\n   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)\n   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)\n   at System.Management.Automation.Interpreter.Interpreter.Run(InterpretedFrame frame)\n   at System.Management.Automation.Interpreter.LightLambda.RunVoid1[T0](T0 arg0)\n   at System.Management.Automation.PSScriptCmdlet.RunClause(Action`1 clause, Object dollarUnderbar, Object inputToProcess)\n   at System.Management.Automation.PSScriptCmdlet.DoEndProcessing()\n   at System.Management.Automation.CommandProcessorBase.Complete()\r\nat <ScriptBlock>, /mnt/azscripts/azscriptinput/userscript.ps1: line 55\r\nat <ScriptBlock>, <No file>: line 1\r\nat <ScriptBlock>, /mnt/azscripts/azscriptinput/DeploymentScript.ps1: line 295"
                    }
                  ]
                }
              ]
            }
          ]
        }
      ]
    }
  ]
}

How to reproduce

Open Oracle WebLogic Server Cluster on Azure VMs.
In Basics pane, fill in required fields. Select Next.
In TLS/SSL Configuration pane, select Next.
In Azure Application Gateway pane, select Yes for Connect to Azure Application Gateway?. Select Generate a self-signed certificate for Select desired TLS/SSL certificate option. Select Next.
In Networking pane, select Review + Create.
Select Create to initialize the deployment.
The deployment failed at sub-deployment keyVaultLinkedTemplate:

Root cause see https://github.com/Azure/azure-powershell/issues/8555

Debug info:
$PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.0.13
PSEdition                      Core
GitCommitId                    7.0.13
OS                             Linux <IP_ADDRESS>-microsoft-standard #1 SMP Mon Mar 7 17:36:34 UTC 2022
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Root cause: the PowerShell runtime is 7.0.13 even the template specified version 5.0. Az.ManagedServiceIdentity module has been loaded to runtime by default in PowerShell 7.0.13. No need to import the module in userscript.
Solution: upgrade AzurePowerShell version from 5.0 to 8.3. Do not import Az.ManagedServiceIdentity module.


Feature/49 create home and sidebar

https://github.com/Furlough-Friends/rota-flex-101/issues/49

Hey Guveer, mind writing a fuller description for this pull request! It helps me and Iain to properly review it!

Last thing I would change is the buttons in the sidebar could be <button /> elements instead of <div />'s as this implies they are clickable and tabbable

Okay so a couple of questions. Firstly regarding the getcards or as Ive called it getButtons functions, if I define its type of parameter where should that be stored? (right now its just a any[].
Secondly, is there a proper way to bulk remove the button style added by the browser?

Okay so a couple of questions. Firstly regarding the getcards or as Ive called it getButtons functions, if I define its type of parameter where should that be stored? (right now its just a any[].
Secondly, is there a proper way to bulk remove the button style added by the browser?

So for the any[] thing you should type it with a Typescript type you have defined. Probably in somewhere close to where you use that object.
For the style are you talking about removing the highlighted style?

I think if you have a rerender the css classes get reapplied? If these buttons took you to different routes in the app I suspect that would cause a rerender and might solve the problem. To give other options there is this https://reacttraining.com/reach-ui/tabs which is part of a ui library that can do tabs. Pretty customisible. I think its worth continuing down the path of trying to get the right css to apply though.

if I define its type of parameter where should that be stored? (right now its just a any[].

So, it's an array of objects, which have a custom shape. Sounds like we're gonna need to make an interface here. Perhaps inside the sidebarOptions.js file. Something like this:
interface SidebarOption { name: string }
You should also add this type to the sidebarOptions array that's in that file, like this:
const sidebarOptions: SidebarOption[] = [

so for style, I mean the fact that is a button, has style. Do i have to reply properties individually or is there a catch all to that stuff

Looks good to me! If there are specific button styles we want to unset I might use something like border: unset in the css file. You can do all: initial or all: unset to wipe out all inherited styles, too. Are there particular parts which are causing problems just now?

Looks good to me! If there are specific button styles we want to unset I might use something like border: unset in the css file. You can do all: initial or all: unset to wipe out all inherited styles, too. Are there particular parts which are causing problems just now?

Good to know, thanks. The tickets says not to worry about all the styles just yet so I will leave it out of this PR


Enabling LSP server based on file extension

Language server
No response
Requested feature
Is there a way to enable lsp servers based on the file extension?
That way we could enable cssls for files ended in style.js or the html server for blade.php for example
It would give more control over when to enable which servers
Other clients which have this feature
No response

you can set your file extension with a filetype which in server settings filetypes.

you can custom it with LspAttatch event if you are in 0.8 if file match style.js do smoething .

Could you show an example?

···lua
vim.api.nvim_create_autocmd('LspAttach', {
callback = function()
local file_name = vim.api.nvim_buf_get_name(0)
if not ifle_name:find('style.ks') then
return
end
local clients = vim.lsp.get_active_clients({buffer = 0 })
do something ..
end
})
``


Kindly provide your existing Mutual Fund Investment Platform

Hello,
Please provide link for  your existing mutual fund investment platform

@JigneshSavaliya89 Are you a distributor? I am building a mutual fund investing platform. You can reach out to me on email. Email ID is available on my Github profile page.

Hi @subodh-malgonde MF platform created?

@subodh-malgonde, Please send details on<EMAIL_ADDRESS>email.


install: Add cri-o paths

Add host and mount path for cri-o
Fixes: #192

/test

@bpradipt @jensfr @snir911 - I think this change is helpful to unify the ccruntime in the base coco operator and peer pods, but you will know better than me.

Seems right to me, thanks, cc @wainersm also

@arronwy @fidencio - if you could check TDX that would be great


clfft-client - added a feature to select specific devices

In our environment we got several GPU and CPU devices in parallel and we should be able to profile them indepedently for FFT performance. clfft-client for now only uses the last reported OpenCL device in a given category as a default.
So this pull request is a patch that allows all OpenCL devices in a system to be enumerated (results printed out via "-i") and selected specifically with their id in clfft-client. The selection is possible via a new command-line parameter "--device [dev-id]". Without the parameter clfft-client defaults to its current behaviour, selecting the last reported OpenCL device.
Afaik, this pull would also be a fix for issue #20.
It's just an idea and sort of a "best-guess" implementation, but maybe it's of some use for the lib.

Thank you for this PR; this sounds like a very nice feature addition.  After my team tests this a bit on our side, we will merge it in.

Merged after several corrections; thanks

Great to see this in the client, you're very welcome. If you prefer a different enumeration scheme for the platforms / devices, that's fine.

As said the numbering scheme with platform / device is fine, but then as a final correction openCL.misc.cpp:226 should read
... device [ " << n << " ]:" ...
instead of
... device [ " << devNo << " ]: ..." . Variable devNo can be dismissed totally in this case.
Otherwise a globally ascending device id will be print out by --clInfo, as the global id was used to explicitly select the device in the old numbering scheme.

Overlooked that; it has been fixed now


Play movie located on server and stream it to the user?

+1 Give this fake Netflix some actual use maybe?

Yup I agree


CI is failing on new color_grading example

https://github.com/bevyengine/bevy/actions/runs/8924250188/job/24510192976#step:6:475
Just follow the suggestion to fix this.

Tackling this now.


use Travis CI

To validate this pull request against other sbt modules, please visit this link.

Travis CI is slower, but at least it's working, so I'll merge this for now.


Correction for raw affiliation Brain and Mind Institute, Western University

Correction needed for raw affiliation Brain and Mind Institute, Western University
raw_affiliation_name: Brain and Mind Institute, Western University
new_rors: 02grkyz14
previous_rors: 00we1gw23;05cgtjz78
works_examples: W4366083753;W4389078325;W4372286277;W4386249051
contact: 960eba78228db409cac4bc34b12058d3:f69d3e2abf5f96cf @ uwo.ca

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


Connection lost on SQL Server

When running most queries using the SQL Server driver, I get failures that simply return:
Connection lost - read ECONNRESET

No other information is in the console. If I run a simple query like selecting a few rows from a simple table, it returns results as expected. Anything with more structure seems to fail. Note that this occurs with working queries that run without error with other clients (e.g. Azure Data Studio), so there is nothing inherently wrong with the query or the connection. However, the lack of information makes it difficult for me to debug this myself.
Desktop:

SQLTools Version 0.23.0
VSCode Version: 1.58.2
OS: MacOS Big Sur
Driver: MSSQL/Azure
Database version: SQL Server 13

Are you still able to reproduce this with latest VS Code and extension versions?

At the risk of being flamed for posting on a closed ticket (due to lack of response by OP) but not wanting to create another ticket with the same issue.
I have the same issue when trying to execute long queries e.g. long update queries where I have generated the update SQL from an Excel sheet and paste many updates that should run in sequence.
It appears to break if the query is >4000 characters
Also the SQL Console only seems to show the result of the last query executed.
Yes I know I can solve this in many different ways, I usually just open and paste into SSMS instead.

@jermwormza I will reopen the issue but don't currently have capacity to investigate.

ECONNRESET is coming from the tedious driver. There is an open issue which seems to be related to this: https://github.com/tediousjs/tedious/issues/1483.
Unfortunately at this point you can only wait for it to be fixed in the tedious library. I have a fork of the mssql driver that has msnodesqlv8 driver alongside tedious but the former is only used for windows authentication at this point.


Update license for a new year

@3lvis Thank you, but we don't add patches to our branch of LLVM unless strictly necessary.  Please submit the patch to the upstream llvm.org project.


added Do notation

I added Do notation to mirror that in the new fp-ts release plus a unit test similar to that in fp-ts

Thanks @kylegoetz

Thanks @kylegoetz


implement the way to determine the probability of test failure caused by bug

It'd be cool to create algorithm (using AI as an option), which let us calculate the probability of case when test failure has been caused by bug.
Consequently add sorting based on this parameter in order to provide user advanced assistance in decision-making.

Not planned to be implemented.


about 12-Hand Gesture Recognition_v2

when we use this application,
in the case of "if(hand_count == 0 && thresh_score > 70)",
this message is shown ,
" with low confidence"
what does it mean?

The sample app is designed to show good accuracy during the demos.
This app has 2 modes :

Normal mode (Number of hands =1).
Low confidence mode. (Number of hands =0).

Resnet-18 based hand gesture classifier operates on the full frame and not on the cropped image.
TinyYoloV3 model detects number of hands visible in the frame.  It is used as an error handling mechanism at application level.

If number of hands are more than 1 , an error is shown on the screen- "Multiple hands detected".
If number of hands ==1 , Then it is called as normal mode.  Bounding box around detected hand is shown along with hand gesture text.
In case, TinyYoloV3 could not detect a hand, but Resnet18 based hand gesture classifier shows good gesture detection confidence (more than 70% confidence), then the result of Resnet-18 based hand gesture classifier is displayed on the screen along with "Low confidence" message.

thank you for your response.
I run this application, "Low confidence" message is often displayed.
the case that TinyYoloV3 could not detect a hand but Resnet-18 could classify handgestures,
often occured.
"Low confidence" make confuse the user,therefore the message does not need to be displayed.

This minor update in output message is completed. Commit is made on main branch.

Thank you for your reply.
I confirmed to delete "Low confidence".


Update LICENSE template

Fix condition check in LICENSE file, which prevent cookiecutter to generate this file.

@romanosipenko thanks for catch this. :ship:


More cli/node/config refactoring

Eliminate dependencies on Cardano.Node.* modules outside of the node itself.
Make cardano-cli not depend on cardano-node

bors r+

bors r+


defineComponent is no longer tree-shakeable without /* @__PURE__ */ with rollup since 3.0.0-rc.3

Vue version
3.3.0-beta.4
Link to minimal reproduction
Rollup REPL
Steps to reproduce

Open the rollup REPL I linking above.
See the output includes defineComponent code.
Toggle codes in the defineComponent function in utils.js.
"re 3.0.0-rc.3 version" and "my suggestion version" works fine but "pre 3.3.0-alpha.6 version" and "current main branch version" doesn't

What is expected?
The defineComponent function is detected as side-effect free by Rollup as described in the document.
What is actually happening?
The defineComponent function is not detected as side-effect free by Rollup.
It seems the extend call and the property access (options.name) is treated as side-effectful.
related commits: https://github.com/vuejs/core/commit/304830a764cd9f28098cfb0ac0e520e1bb2f57c7, https://github.com/vuejs/core/commit/d77557c4038f88a676903b379505b280a88cc774
System Info
No response
Any additional comments?
Original issue: https://github.com/vitejs/vite/issues/13087

For a temporary workaround, you might consider this rollup plugin.


Fix instance Semigroup ByteString64?

This instance looks very wonky. e.g. base64 encoding can wind up using fractions of a char, this semigroup doesn't line up with that behavior.

The internal bytestring is not base64 encoded. It's encoded and decoded in e.g. aeson / http-api-data instances, getEncodedByteString64 etc.

For example:
*Data.ByteString.Base64.Lazy.Type> "ab" :: ByteString64
mkBS64 "ab"
*Data.ByteString.Base64.Lazy.Type> "cd" :: ByteString64
mkBS64 "cd"
*Data.ByteString.Base64.Lazy.Type> "ab" <> "cd" :: ByteString64
mkBS64 "abcd"
*Data.ByteString.Base64.Lazy.Type> getEncodedByteString64 "ab"
"YWI="
*Data.ByteString.Base64.Lazy.Type> getEncodedByteString64 "cd"
"Y2Q="
*Data.ByteString.Base64.Lazy.Type> getEncodedByteString64 ("ab" <> "cd")
"YWJjZA=="

Fair. I guess I completely misunderstood what the type was for.


Fix window.pid return value

PID (or 0) instead of Success/Error.
The reason the test is failing (so I still haven't activated it) is that xlogo is apparently a magic window:
$ xdotool search --name xlogo getwindowpid
> window 117440513 has no pid associated with it.

I have verified the functionality manually instead.

Hmm, that's interesting. What happens if you run xeyes or another builtin Xorg tool instead of xlogo?

(Changes LGTM, but I'd like to get the test working with xeyes or another application, if possible.)

(Changes LGTM, but I'd like to get the test working with xeyes or another application, if possible.)

I tried many and those that were available on my official repos are xlogo xeyes xclock xbill xcalc xload xmessage xev and out of those only xclock has a PID. But with xclock, there are some BadWindow errors but only sometimes (??) idk yet


[BUG]

setuptools version
61.0.0
Python version
3.9
OS
Debian Unstable
Additional environment information
No response
Description
kiwisolver 1.4.0 (I haven't checked earlier versions, but kiwisolver 1.4.0 did modernise their build, so issues with earlier versions will likely be to different code) can no build from source. I filed https://github.com/nucleic/kiwi/issues/130 with the kiwisolver developers, but it appears that setuptools 60.10.0 does work.  Based on looking at https://github.com/pypa/setuptools/compare/v60.10.0...v61.0.0 and the error message that appears (see output), I think this is related to the new(?) package discovery mechanism. I'm not a kiwisolver developer, so I can't answer questions about their build system (other than link you to the repository), but I'm filing this as it appears to be a regression (give setuptools 61 came out 10 hours ago when I wrote this).
Expected behavior
I expect pip install kiwisolver would build a wheel and install it.
How to Reproduce

Run pip install kiwisolver (given kiwisolver has wheels, you may need to disable using wheels to not download a pre-built wheel).

Output
      /tmp/pip-build-env-nwrt4o9m/overlay/lib/python3.9/site-packages/setuptools/config/pyprojecttoml.py:100: _ExperimentalProjectMetadata: Support for project metadata in `pyproject.toml` is still experimental and may be removed (or change) in future releases.
        warnings.warn(msg, _ExperimentalProjectMetadata)
      Install ``trove-classifiers`` to ensure proper validation. Meanwhile a list of classifiers will be downloaded from PyPI.
      error: Multiple top-level packages discovered in a flat-layout: ['py', 'kiwi'].

      To avoid accidental inclusion of unwanted files or directories,
      setuptools will not proceed with this build.

      If you are trying to create a single distribution with multiple packages
      on purpose, you should not rely on automatic discovery.
      Instead, consider the following options:

      1. set up custom discovery (`find` directive with `include` or `exclude`)
      2. use a `src-layout`
      3. explicitly set `py_modules` or `packages` with a list of names

      To find more information, look for "package discovery" on setuptools docs.

I am a maintainer of kiwisolver. I guess that setuptools is confused because kiwisolver is a pure C extensions without any python source. Currently it does not specify a value for py_modules nor for packages. I would appreciate any advice on how to keep things working with the latest setuptools.

Thank you very much for reporting this.
My understanding is that unfortunately there is a clash of 2 behaviours:
To make these changes as backwards compatible as possible, I added a condition that only works for packages not using pyproject.toml that will check if ext_modules is given and then skip auto-discovery.
But, going forward my assumption was that it is very common for packages to mix both regular Python code with extensions. So if a project is using pyproject.toml for metadata, auto-discovery is always on.
I did not foresee that kiwisolver was providing metadata in both pyproject.toml and setup.py already before the latest change. Sorry for that.
@MatthieuDartiailh, one way to disable auto discovery is to explicitly set py_modules=[] in setup.py or py-modules = [] in pyproject.toml [tool.setuptools]. Would that work for you?
If that is a viable alternative, it would help a lot! I was hopping to keep the auto-discovery always on for packages using pyproject.toml...

It would work, but actually my setup is broken since in its current state it does not distribute type annotations which I recently added. So I will need to rework the overall structure.
I had hoped to get rid of everything but the C extension building in setup.py however this warning is a bit frightening. Do you really think you may roll back support for PEP 621 ?

Do you really think you may roll back support for PEP 621?

I hope not, but it is very early to say. It was just released yesterday, and if it break to many projects I might have to roll back before re-working on it again.
Given that we are not forced to roll back, usage the [project] table should change, after all it is defined in a PEP, but the shape of [tool.setuptools] might differ a bit in the future...

my setup is broken since in its current state it does not distribute type annotations which I recently added.

Is this also caused by the latest release?

No I added those in the last release and apparently I did not properly test apparently.

Do you really think you may roll back support for PEP 621?

I hope not, but it is very early to say. It was just released yesterday, and if it break too many projects I might have to roll back before re-working on it again.
Given that we are not forced to roll back, usage the [project] table should change, after all it is defined in a PEP, but the shape of [tool.setuptools] might differ a bit in the future...

Here you mean "[project] table should not change", right ?

my setup is broken since in its current state it does not distribute type annotations which I recently added.

Is this also caused by the latest release?

We observe a similar error (but with different warnings) with scikit-fmm==2022.02.02 on python 3.10.2:
Collecting scikit-fmm==2022.02.02
  Downloading scikit-fmm-2022.2.2.tar.gz (434 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 434.5/434.5 KB 87.6 MB/s eta 0:00:00
  Installing build dependencies: started
  Installing build dependencies: finished with status 'done'
  Getting requirements to build wheel: started
  Getting requirements to build wheel: finished with status 'error'
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [18 lines of output]
      /tmp/pip-build-env-qhobckeh/overlay/lib/python3.10/site-packages/setuptools/_distutils/dist.py:275: UserWarning: Unknown distribution option: 'configuration'
        warnings.warn(msg)
      /tmp/pip-build-env-qhobckeh/overlay/lib/python3.10/site-packages/setuptools/dist.py:511: UserWarning: Normalizing '2022.02.02' to '2022.2.2'
        warnings.warn(tmpl.format(**locals()))
      error: Multiple top-level packages discovered in a flat-layout: ['skfmm', 'profile'].

      To avoid accidental inclusion of unwanted files or directories,
      setuptools will not proceed with this build.

      If you are trying to create a single distribution with multiple packages
      on purpose, you should not rely on automatic discovery.
      Instead, consider the following options:

      1. set up custom discovery (`find` directive with `include` or `exclude`)
      2. use a `src-layout`
      3. explicitly set `py_modules` or `packages` with a list of names

      To find more information, look for "package discovery" on setuptools docs.
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

Hi @tobni, thank you very much for reporting this.
Am I correct in assume that scikit-fmm does not specify any of py_modules, packages or ext_modules keywords?

Here you mean "[project] table should not change", right ?

Yes sorry @MatthieuDartiailh, I typed that too fast without checking 😅

@abravalheri Not in the pyproject.toml, no. I don't know exactly where to look in the setup.py file, the scikit-fmm package uses numpy.distutils.misc_util import Configuration which is hard to grok.

My question was more in the direction of setup.py. I will have a look on the numpy.distutils.misc_util.Configuration. Thank you very much @tobni.

Experiment a bit I found another undesirable behavior.
Kiwisolver uses setuptools-scm and after modifying the setup.py to include the following directives (which silences the previously reported error for setuptools 61) the installed package in site-packages includes all the C source files and the tests even though they are not requested. With Setuptools <= 60, the install contains just an __init__.py (added to allow accessing the C extension content), a .pyi file, py.typed and the built extension package.
    package_dir={"kiwisolver": "py"},
    packages=["kiwisolver"],
    install_requires=["typing_extensions;python_version<'3.8'"],
    ext_modules=ext_modules,
    cmdclass={"build_ext": CppyBuildExt},
    package_data={"kiwisolver": ["py.typed", "*.pyi"]},

Thanks @MatthieuDartiailh, I will have a look on that too.
This is likely related to the fact that the presence of [project] in pyproject.toml will automatically flip the default value of include_package_data (in setup.cfg/setup.py-only projects it defaults to False, while with pyproject.toml [project] it defaults to True.
This should have been added to the CHANGELOG. Sorry I forgot that.

It appears that through some re-organization of kiwi I managed to circumvent the issue (see the PR). But it may not apply to other packages out there.

Not sure if this is the same issue, but building scipy with --no-build-isolation fails as well since the setuptools 61.0 update.
With build isolation their pin to setuptools<60 in pyproject.toml saves the day.

Note that even though it does not break the build the issue of unwanted files being included can also be seen in https://github.com/nucleic/atom

Basically all C source files found under atom/src end being installed. What I find confusing with the include_package_data explanation is that even if I set it to False manually in setup.py I still see the same weird behavior. Do you expect that ?

if I set it to False manually in setup.py I still see the same weird behavior. Do you expect that ?

I will work on that. The current code expects include-package-data to be set to false in pyproject.toml [tool.setuptools], but I completely agree it should work also when it is set in setup.py.

I can confirm that settings include-package-data in pyproject.toml does indeed work.

I'm encountering this issue on packages that act solely as a meta package for distributing a collection of related packages via dependencies. The package does not have any source packages or modules. I can confirm that explicitly setting packages=[] resolves the issue.
I do still find it strange that what you call an experimental feature in the docs is enabled by default.

To be fair, I see this behavior is mentioned in your changelog under "breaking changes". That's my bad for not checking there in the first place.

Another package that fails to build now is cartopy:
#20 731.2 Collecting cartopy>=0.20.0
#20 731.2   Downloading Cartopy-0.20.2.tar.gz (10.8 MB)
#20 731.5      ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 10.8/10.8 MB 34.0 MB/s eta 0:00:00
#20 731.8   Installing build dependencies: started
#20 794.2   Installing build dependencies: still running...
#20 858.3   Installing build dependencies: still running...
#20 939.5   Installing build dependencies: still running...
#20 1007.0   Installing build dependencies: still running...
#20 1079.0   Installing build dependencies: still running...
#20 1080.7   Installing build dependencies: finished with status 'done'
#20 1080.7   Getting requirements to build wheel: started
#20 1080.8   Getting requirements to build wheel: finished with status 'error'
#20 1080.8   error: subprocess-exited-with-error
#20 1080.8
#20 1080.8   × Getting requirements to build wheel did not run successfully.
#20 1080.8   │ exit code: 1
#20 1080.8   ╰─> [16 lines of output]
#20 1080.8       Traceback (most recent call last):
#20 1080.8         File "/usr/local/pyenv/versions/3.10.4/lib/python3.10/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 363, in <module>
#20 1080.8           main()
#20 1080.8         File "/usr/local/pyenv/versions/3.10.4/lib/python3.10/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 345, in main
#20 1080.8           json_out['return_val'] = hook(**hook_input['kwargs'])
#20 1080.8         File "/usr/local/pyenv/versions/3.10.4/lib/python3.10/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 130, in get_requires_for_build_wheel
#20 1080.8           return hook(config_settings)
#20 1080.8         File "/tmp/pip-build-env-vso68_9c/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 177, in get_requires_for_build_wheel
#20 1080.8           return self._get_build_requires(
#20 1080.8         File "/tmp/pip-build-env-vso68_9c/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 159, in _get_build_requires
#20 1080.8           self.run_setup()
#20 1080.8         File "/tmp/pip-build-env-vso68_9c/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 174, in run_setup
#20 1080.8           exec(compile(code, __file__, 'exec'), locals())
#20 1080.8         File "setup.py", line 35, in <module>
#20 1080.8           from setuptools import Command, Extension, convert_path, setup
#20 1080.8       ImportError: cannot import name 'convert_path' from 'setuptools' (/tmp/pip-build-env-vso68_9c/overlay/lib/python3.10/site-packages/setuptools/__init__.py)
#20 1080.8       [end of output]

Looks unrelated on first glance, but it did start breaking with setuptools 61, so maybe it's not?

Not in the pyproject.toml, no. I don't know exactly where to look in the setup.py file, the scikit-fmm package uses numpy.distutils.misc_util.Configuration which is hard to grok.

Hi @tobni, the fix that I am considering for this problem is to disable automatic discovery when the configuration=... parameter is set in setup(). Would that work for you?
If no packages, py_modules, ext_modules and configuration are set, then for setuptools it will look to setuptools like it is asking for the auto-discovery to quick-in.
I understand that there are situations where the dev wants to have an empty distribution, but that is a breaking change covered in https://setuptools.pypa.io/en/latest/history.html#breaking-changes. The fix for that is to add a packages=[].

Fixing it, in setup.py
setuptools.setup(..,packages=[],..)

or
setuptools.setup(..,py_modules=[],..)

i have had the same problem in undertime. suddenly, setuptools starting failing my CI while everything worked fine literally an hour earlier. this is quite strange because the dependency list didn't change, the container image didn't change, and the commit (on my side) which introduced the problem (according to git bisect) happened months ago.
so i don't understand why this started failing all of  sudden.
i also don't understand the fix: #3211 mentions the configuration option, but I can't find documentation on that setting anywhere (and it's kind of hard to search for configuration in the documentation, because there's a lot of false positives). it's not documented here or here or here.
in the end, I had to add this line to my setup.cfg:
[option]
packages =

... to, basically, disable autodiscovery. and yes, i have a weird-ish package (it only ships scripts) but I don't think it's a particularly strange one.
a little care about backwards compatibility would have been appreciated here. ;)
i hope this hack will be useful for others!

Hi @anarcat, sorry to hear you are facing problems with your package.

3211 mentions the configuration option

configuration is something provided by numpy.distutils, that is why you cannot find it in setuptools.
If you are not using it, the fix to explicitly set either packages or py_modules. What you have done works well for a package with no Python code on it.
# setup.cfg
packages = # empty on purpose
# OR setup.py
packages=[]

Question: since you have an undertime.py wouldn't it be the case that instead of packages = # nothing you want to use py_modules = undertime?

Question: since you have an undertime.py wouldn't it be the case that instead of packages = # nothing you want to use py_modules = undertime?

because i want to keep things simple and not use an entry_point because those are traditionally slow.
it's just one script darn it! :)
so i ship it with scripts =

Ok, no problems, I just curious.

Hit this in numpy as well:
#9 529.2 Using pip 22.0.4 from /usr/local/lib/python3.9/dist-packages/pip (python 3.9)
#9 529.3 Looking in indexes: https://mirrors.aliyun.com/pypi/simple
#9 529.3 Processing ./mirrored-pip-GRCQUK/numpy
#9 529.5   Installing build dependencies: started
#9 529.5   Running command pip subprocess to install build dependencies
#9 531.1   Looking in indexes: https://mirrors.aliyun.com/pypi/simple
#9 531.1   Ignoring packaging: markers 'platform_machine == "arm64"' don't match your environment
#9 531.7   Collecting setuptools==59.2.0
#9 531.8     Downloading https://mirrors.aliyun.com/pypi/packages/18/ad/ec41343a49a0371ea40daf37b1ba2c11333cdd121cb378161635d14b9750/setuptools-59.2.0-py3-none-any.whl (952 kB)
#9 533.1        ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 952.0/952.0 KB 734.9 kB/s eta 0:00:00
#9 533.2   Collecting wheel==0.37.0
#9 533.2     Downloading https://mirrors.aliyun.com/pypi/packages/04/80/cad93b40262f5d09f6de82adbee452fd43cdff60830b56a74c5930f7e277/wheel-0.37.0-py2.py3-none-any.whl (35 kB)
#9 534.0   Collecting Cython<3.0,>=0.29.24
#9 534.0     Downloading https://mirrors.aliyun.com/pypi/packages/9a/26/d2b6bc4cb7d716c82ebc89690cbd5ba0f547db364809cd42dad34d593182/Cython-0.29.28-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.manylinux_2_24_x86_64.whl (1.9 MB)
#9 536.6        ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.9/1.9 MB 761.3 kB/s eta 0:00:00
#9 536.9   Installing collected packages: wheel, setuptools, Cython
#9 537.8   Successfully installed Cython-0.29.28 setuptools-59.2.0 wheel-0.37.0
#9 537.8   WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
#9 537.9   Installing build dependencies: finished with status 'done'
#9 537.9   Getting requirements to build wheel: started
#9 537.9   Running command Getting requirements to build wheel
#9 538.3   Running from numpy source directory.
#9 539.2   error: Multiple top-level packages discovered in a flat-layout: ['numpy', 'branding'].
#9 539.2
#9 539.2   To avoid accidental inclusion of unwanted files or directories,
#9 539.2   setuptools will not proceed with this build.
#9 539.2
#9 539.2   If you are trying to create a single distribution with multiple packages
#9 539.2   on purpose, you should not rely on automatic discovery.
#9 539.2   Instead, consider the following options:
#9 539.2
#9 539.2   1. set up custom discovery (`find` directive with `include` or `exclude`)
#9 539.2   2. use a `src-layout`
#9 539.2   3. explicitly set `py_modules` or `packages` with a list of names
#9 539.2
#9 539.2   To find more information, look for "package discovery" on setuptools docs.
#9 539.3   error: subprocess-exited-with-error
#9 539.3
#9 539.3   × Getting requirements to build wheel did not run successfully.
#9 539.3   │ exit code: 1
#9 539.3   ╰─> See above for output.
#9 539.3
#9 539.3   note: This error originates from a subprocess, and is likely not a problem with pip.
#9 539.3   full command: /usr/bin/python3 /usr/local/lib/python3.9/dist-packages/pip/_vendor/pep517/in_process/_in_process.py get_requires_for_build_wheel /tmp/tmpis1s8vl8
#9 539.3   cwd: /tmp/scratch/mirrored-pip-GRCQUK/numpy
#9 539.3   Getting requirements to build wheel: finished with status 'error'
#9 539.3 error: subprocess-exited-with-error
#9 539.3
#9 539.3 × Getting requirements to build wheel did not run successfully.
#9 539.3 │ exit code: 1
#9 539.3 ╰─> See above for output.
#9 539.3
#9 539.3 note: This error originates from a subprocess, and is likely not a problem with pip.

OS has setuptools 61.3.1 installed.
Numpy askes for setuptools 59.2.0 inside its build environment but that's probably too late.

Hi @xkszltl, thanks for reporting this.
What are the commands you are running? Do you have minimal reproduction for that?

@abravalheri
See details here:
https://github.com/numpy/numpy/issues/21288

@xkszltl, I don't seem to be able to replicate this problem inside of a container.
I posted my comments in https://github.com/numpy/numpy/issues/21288#issuecomment-1086866367.

Not that I know what I am doing but I got bitten by this albeit my package is pure python in the attempt of adopting a pyproject.toml workflow. https://github.com/ilayn/harold/pull/80 . It started reading into my notebooks folder and getting confused though I don't quite get why it would do that. Should I pin setuptools to something earlier?
I have to say I am quite lost in this packaging systems and thanks to people mentioning issue numbers in commit messages it is not very easy to follow the discussion above.

Hi @ilayn, I think what is happening is that you used to have packages=['harold'] in your setup.py, but when updating to pyproject.toml, that information got left out[^1]. Setuptools can automatically infer for some scenarios what are the packages, but the notebooks folder is confusing for the auto-discovery. Please see more details in https://setuptools.pypa.io/en/latest/userguide/package_discovery.html#flat-layout.
I think there is no need to pin setuptools, only add packages to your config. Does that work for you?
[^1]: I am seeing that your project has a harold/__init__.py file not a harold.py, so packages is more appropriate than py-modules.

@abravalheri Thank you for your swift response. You are probably right in all that. Let me experiment a bit more.
I am seriously struggling with understanding all these gotchas. While Python is probably the easiest tool to code, when it comes to packaging every other recommendation online is saying something else about using toml or cfg or setup.py including the official docs. This section for example is telling a lot but really not saying anything https://packaging.python.org/en/latest/tutorials/packaging-projects/#configuring-metadata
My point is apologies if I came to this issue before I did everything else right and bothering you. Either this stuff requires a much cleaner docs or this tool is getting complicated beyond regular user's understanding.

@ilayn no problems :)
Packaging in Python is a bit more difficult. I think the easiest for beginners is to use a package generator such as https://pyscaffold.org, https://github.com/ionelmc/cookiecutter-pylibrary or any other generator that you find suitable for your needs[^1].
[^1]: Sure, these generators might be a bit behind in terms of the [project] table inside pyproject.toml, but you don't need to rush to implement it right now...

one way to disable auto discovery is to explicitly set py_modules=[] in setup.py or py-modules = [] in pyproject.toml [tool.setuptools].

I may have misunderstood that workaround - @abravalheri, do you mean to literally write an empty list for py-modules? I tried it out, and while packages using that workaround are successfully installed, I cannot seem to import their modules (unless my working directory is the project directory, in which case installation is not required anyway).
Or was your recommended workaround to populate py-modules with the actual list of module names?

@bersbersbers the context for that is a distribution that does not contain any Python packages or modules.
If your project contains those, probably the best advice is to follow one of the suggestions in the error message.


DatePicker doesn't work when inside the FormControl element

Environment

Tech
Version

material-ui-pickers
^1.0.0-rc.5

material-ui
^1.0.0-beta.40

React
16.2.0

Browser
Chrome Version 65.0.3325.181 (Build officiel) (32 bits)

Platform

Steps to reproduce

Embed DatePicker inside FormControl.
Open the picker
Select a date different that is selected by default
Make the focus is lost from the selected day by clicking for example on the white background of the popup (clicking on buttons has the same effect but you cannot see what happens as the popup closes)
See that the selections goes back to the default selection

Expected behavior

Step 5, the new selection should be kept.
Actual behavior

Step 4, makes the selection beeing lost to the default value selected when popups opens
Live example

https://codesandbox.io/s/741wynl36j

And you cant change month when DatePicker is inside FormControl!

See https://github.com/dmtrKovalenko/material-ui-pickers/issues/287#issuecomment-374694268


Does have a way to load the model only once and use it for multi audio file in the same time?

I want to process multi audio files in the same time instead process them one by one.
I tried making the WhisperContext as a Singleton, but when got an error:
Assertion failed: (sum > 0.0f), function ggml_compute_forward_flash_attn_f16, file ggml.c, line 6299.

Maybe it's a problem of whisper.cpp.

whisper.cpp does not support this, you must load a new model for each thread.

whisper.cpp version 1.3.0 was released two days ago, and should support this now. I'll be working on adding the features into whisper-rs tonight.


Error in distfun(dT, i, seq(i + 1, V)) :  Not compatible with requested type: [type=character; target=double

I am getting this error while trying to run UMAP on my dataset. Below is the code that I used:
library(umap)
head(scenic_umap,3)
scenic.data <- scenic_umap[, grep("X|Y", colnames(scenic_umap))]
scenic.labels <- scenic_umap[, "X.1"]
scenic.umap <- umap(scenic.data)
Error in distfun(dT, i, seq(i + 1, V)) :
Not compatible with requested type: [type=character; target=double].
In addition: Warning message:
In umap.prep.input(d, config) : NAs introduced by coercion
Here is the data set:

Thank you !

Hi @Su870. It seems you are using R and this issue actually pertains to a different package here.
In any case, the dataset contains column 'X.1' with character values, and that is not supported. Try to remove that column and run only on numeric data.
Also, there is a warning message that suggests your dataset may have NA values. You will likely need to handle that somehow. Two common strategies are to remove rows with missing values or to impute the missing information in some way that suits your application.

Yes! @tkonopka it worked but then when I finish my run the umap is empty.
I renamed some of my variables from the code above, here is the updated one:
data <- Finalpyscenic_ucec_umap_edited
datapoint <- data[, -1]
labels <- data[, "Patient"]
ucec.umap <- umap(datapoint)
plot.scenic(ucec.umap, labels)
The reference code for plot.scenic is below:
plot.scenic <- function(x, labels,
main="A UMAP visualization of FH_UCEC",
colors=c("#ff7f00", "#e377c2", "#17becf"),
pad=0.1, cex=0.6, pch=19, add=FALSE, legend.suffix="",
cex.main=1, cex.legend=0.85) {
layout <- x
if (is(x, "umap")) {
layout <- x$layout
}
xylim <- range(layout)
xylim <- xylim + ((xylim[2]-xylim[1])*pad)*c(-0.5, 0.5)
if (!add) {
par(mar=c(0.2,0.7,1.2,0.7), ps=10)
plot(xylim, xylim, type="n", axes=F, frame=F)
rect(xylim[1], xylim[1], xylim[2], xylim[2], border="#aaaaaa", lwd=0.25)
}
points(layout[,1], layout[,2], col=colors[as.integer(labels)],
cex=cex, pch=pch)
mtext(side=3, main, cex=cex.main)
labels.u <- unique(labels)
legend.pos <- "topleft"
legend.text <- as.character(labels.u)
if (add) {
legend.pos <- "bottomleft"
legend.text <- paste(as.character(labels.u), legend.suffix)
}
legend(legend.pos, legend=legend.text, inset=0.03,
col=colors[as.integer(labels.u)],
bty="n", pch=pch, cex=cex.legend)
}
plot.scenic(scenic.umap, scenic.labels)

Glad the embedding worked. For the visualization, it is hard to see/guess what is happening. In such cases, start with previewing the matrix of coordinates, then make a basic plot with default settings, and add the visualization extras gradually.


chore: enable chainsaw integration tests

This PR adds chainsaw e2e tests:

allowed namespaces
scheduling gates

Those tests are equivalent to kuttl ones, they have been converted using the chainsaw migration command and adapted manually to use declarative syntax instead of relying on bash scripts.

plz also move https://github.com/keptn/lifecycle-toolkit/tree/main/test/testanalysis/analysis-controller-multiple-providers links to https://github.com/keptn/lifecycle-toolkit/tree/main/test/kuttl/testanalysis/analysis-controller-multiple-providers


Deployment is not valid according to the validation procedure.

Describe the issue:
Deploy failed when we try to run the file: "test-templates".
Repro Steps:
After az login , run bash ./test-templates.sh -t Azure-Samples/templates -c false.
Error Message:

Environment:

todo-nodejs-mongo-aca: Windows desktop, Linux Desktop, MacOS desktop, WSL, DevContainer in VS Code, Codespaces
todo-nodejs-mongo-swa-func: DevContainer in VS Code
todo-python-mongo-aca: Windows desktop, Linux Desktop, MacOS desktop, WSL, DevContainer in VS Code, Codespaces
todo-python-mongo-swa-func: WSL, Codespaces

The above is the error occurred in the template and the corresponding environment.
Branch: staging
Expected behavior:
Create resource with the command: bash ./test-templates.sh -t Azure-Samples/templates -c false can success.
@jongio for notification.

@wbreza @zedy-wj
FYI, Not seeing deployment failures for these templates in the pipeline runs.

Internal error is the following:
Inner Errors:
{"code": "MaxNumberOfRegionalEnvironmentsInSubExceeded", "message": "The subscription 'faa080af-c1d8-40ad-9cce-e1a450ca5b57' cannot have more than 10 environments in East US 2."}
We've exceeded the number of container app environments per subscription in this region.  I will go ahead and remove some of the old test resource groups that have not been cleaned up.


wrong beautification on SCSS concatenation in background url property

Hi,
I noticed an issue after beautifying my SCSS containing this kind of concatenation :

it produces this (it adds a double quote before my var), which is not correct :

Maybe I did something wrong regarding my SCSS syntax, but at first sight it looks more an error from the beautifyer.
Thank you.

https://github.com/prettydiff/prettydiff/issues/63
This is my bug. SCSS support is provided by Pretty Diff, so I need to fix this in my library.

ok. thanks in advance :)

QA version up for your review:
Up for review:
prettydiff.com/ignore/testlocation/prettydiff.com.xhtml

Now fixed in v1.11.3

@Glavin001 Is that fix has been embedded in the latest versions ?

Not yet. I am in the middle of exams, however once I can take a break from studying I'll make this fix and publish it. Thank you for your patients.
Also thanks to @prettydiff for being awesome and resolving these so quickly!

I updated to the atom-beautify latest version and it seems to work fine. I can't reproduce the bug. Maybe you embedded prettydiff 1.11.3 in a previous release :)
Anyway, good luck for your exams ;)

Ooh that may be the case: see https://github.com/Glavin001/atom-beautify/blob/master/package.json#L65
Thanks to semver, Atom likely updated the version of Prettydiff to latest automatically when you updated. Sweet! Thanks for letting me know.

Anyway, good luck for your exams ;)

Thank you :smiley:


Clear the System Managed Components Node Selector field?

Longhorn 1.7.2. I am unable to clear the following field. It is possible to change its value easily but not to blank it out completely:

Hi @turowicz,
Does any error appear on the UI when emptying the setting?
Could you provide the support bundle for the investigation?

This is what I've found:
We have deployed longhorn with a helm setting
--set defaultSettings.systemManagedComponentsNodeSelector=surveily/purpose:general
When I change the setting in the GUI to surveily/all:all it gets changed to surveily/all:all
When I change the setting to blank it goes back to surveily/purpose:general
We tried upgrading the helm installation by removing the set value, but the behaviour remains.
Some transient errors appear but don't impact te behaviour:

Support bundle:
https://surveily-my.sharepoint.com/:u:/p/wojciech_turowicz/ESVQMAE5eQRJuB8qJzBMxp4BckswMbkwnesiiAEz3vuzfg?e=SupfB7
I need a way of blanking out the value of that setting permanently.


API versioning and basepath

It does not seem possible to support API versioning in utoapi at present. There are a number of approaches that I am aware of, including generating multiple OpenAPI schemas and supplying a basepath to OpenAPI.
I don't see support for the OpenAPI basepath in utoapi and so being able to say, "all my endpoints are under X basepath" is not currently possible.
I also don't see any way of generating multiple schemas using utoapi, although this is in any case not ideal as they would mount up over time.
What is the recommended approach to managing API versions using utoapi at present? For instance, if I have both /api/v1/foo and /api/v2/foo` then what is the best way to organise this so that utoapi produces clear OpenAPI documentation?
Are these things on a roadmap somewhere for future support?

Currently there are 2 ways to actually solve this. The #[utoipa::path(...)] macro has context_path attribute. that can be used to prepend the path. Though this is currently bit cumbersome because it need to be given for each path.
Another way is to use the Server property in OpenAPI spec. E.g. creating couple of servers one for /api/v1 and another /api/v2. The servers can be put globally to the API or to the PathItem itself but there is no macro support for setting up servers to the path.
This would need some improvement, but there is no any convenient way to group paths together e.g. scoping because also the requirements for such are different for each framework.

I'm interested in this too.. Is this being worked on?

I would also be interested in this feature, at first glance rust seems to be a possible rising star for APIs and swagger is usefull for documentation and testing these. API versioning seems like a must have to me and workarounds never feel great.
+1 for the proposal of @ctron

In master, ultimately landing in 5.0.0 release, there will be nesting functionality to nest other APIs to one.
https://github.com/juhaku/utoipa/blob/f6d1c3d059e4b42ebcbedfbd98241a51f7991014/examples/axum-utoipa-nesting-vendored/src/main.rs#L13-L20

Defining multiple swagger endpoints might feel weird, but by omitting the "use_base_layout()" method and providing multiple swagger json urls it looks like the swagger UI creates a nice drop down to select the desired api version.
   let swagger_config = Arc::new(utoipa_swagger_ui::Config::new([
       "/v1/swagger.json",
       "/v2/swagger.json",
   ])

@R3dByt3 Yes that is basic functionality from Swagger UI. As demonstrated in example below you can also directly tell Swagger UI to use multiple different OpenAPI docs. Then it is just matter of defining what ToSchema implementations gets added to what openapi. In addition shown above nesting with nest(...) can be also used to control the different versions of schemas and paths.
https://github.com/juhaku/utoipa/blob/576f6c129010b4f6e6445c06393cfcf2cfee30df/examples/actix-web-multiple-api-docs-with-scopes/src/main.rs#L27C13-L36C16
To these words I would conclude this task as solved. If there are more matters to consider, this can be reopened.


Use Puppet-Datatype Sensitive for Passwords

let Function mongodb_password() accept Parameter password of Datatype Sensitive
add Parameter "sensitive" to Function mongodb_password to decide if its Returnvalue should be of Datatype Sensitive
let defined Type mongodb::db accept Datatype Sensitive for $password_hash and $password
let Class mongodb::server accept Datatype Sensitive for $key and $admin_password

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

Looks like we need some help with #602 to get those CentOS tests to pass.

@kenyon

Looks like we need some help with #602 to get those CentOS tests to pass.

#1396.5 failed
"message":"toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit"

I guess, there is not much I can help with…

#602 changes tests to use GitHub Actions instead of Travis, where we don't have the Docker Hub rate limit problem.

Hello @cocker-cc, can you rebase with  our master branch? This will allow the CI to run.

Hello @cocker-cc, can you rebase with our master branch? This will allow the CI to run.

Done.


Visual Designer

One of the core features of Winforms that made it so popular and easy to use was its drag and drop visual designer.  Modern.Forms will need one to provide the same level of ease and convenience.

I'm back!
Better and better. Greate!

DataGrid planned already;
WebView/WebControl/WebBrowser,
I find that avalonia has CEF porting, https://github.com/CefNet/CefNet.
Or something as mono/etc.  using Webkit or System native webControl.

Referencing a comment I left over in the template project...
https://github.com/modern-forms/Modern.Forms-Templates/issues/2

this may be a dumb question, but would the WinForms OOP Designer be a viable option to port to for use with Modern.Forms?

Porting the WinForms designer is definitely a viable option.  The Winforms designer is relatively loved and battle tested. There's a very early prototype of that here:
https://github.com/modern-forms/Modern.Forms/tree/designer-poc/samples/Designer
However the port isn't "simple".  It is an extremely complex piece of software that requires extensive porting to fit into a Modern.Forms world.

Can we have the designer made in vscode so it's cross platform and not only available to windows or people that can afford rider?

exciting if it happens


Request for FreeCAD Files

Hi @the-thought-emporium ,
I hope this message finds you well. I'm really interested in your project and was hoping to dive into the design files you've shared on GitHub. However, I noticed that the repository contains .step files instead of the FreeCAD source files.
If possible, could you share the FreeCAD files for this project? It would be extremely helpful, as I'd like to explore the original designs and make modifications within FreeCAD.
Thanks in advance for your help, and I look forward to hearing from you soon.
Best regards,
Alec Weiß

Hello the .step files are Universal files and usable with freeCAD/fusion360 or any other software you use


Update the changelog with old PRs and github issue ids (USDU-138)

Purpose of this PR
Ticket/Jira #: https://jira.unity3d.com/browse/USDU-138
The changelog needed an update to include PRs previously merged in dev.

I didn't cross-check all user-submitted PRs but I remembered this one that changes the default export FPS from 30 to 60 #170
Done.


Suggest enhancing the implementation of enumeration type documents

Current implementation

Although enumerations can be written using dataDictionaries, it can be cumbersome when dealing with a large number of enumerations. It is recommended to add smart doc tags to describe enumerations

Cloud submit a PR support it, Add a tag named @enumDictionary  using on Class

#977 A lot of changes have been made to the enumeration,  If the enumeration is not configured with a data dictionary, the first parameter of the enumeration is used as enumValue by default, and the comment of the enumeration is used as the enumDesc,and you can see if it meets your needs. @Hans-Wu-cn


Tolerate Zarr V3 module path name change

The encode_fill_value function was moved into a new module in the latest version of zarr, and this causes a problem for packages that depend on both zarr==3.0.0 and kerchunk, such as VirtualiZarr. I don't think this is necessarily related to https://github.com/fsspec/kerchunk/pull/292 , but that PR is free to cherry-pick this commit because they probably should be a single pull request.

Thank you @martindurant!


chore: rework Icon stories to fix missing story icons

Changes included (only affects the icon stories):

Split icons.js file into a basic and brands file for use in separate stories
Trim amount of icons used as examples (previously had 600+) and linked to searchable Font Awesome 6 list in stories
Create separate stories for basic and brand icons to prevent some icons from not displaying due to certain iconStyle values
Remove 'brands' iconStyle option from Available Basic Icons story to prevent issue where the icon would not render if passed as a prop in storybook
Remove iconStyles other than 'brands' for Available Brand Icons in story to prevent issue where the icon wont render if passed a different iconStyle

 Investigate react-gears storybook missing icons

 Investigate react-gears storybook missing icons


2D base equation options are unneeded, confusing, and should be removed

There are options to treat the base equation as 2D. These are just pointless to have, the 2D noise cache takes care of any performance issues anyways. These options should be removed.

Removed in dddf644c34b01af23b6bdbb4d6747db999e2ab0a

Removed in dddf644c34b01af23b6bdbb4d6747db999e2ab0a


Need option to opt-out dev container features

Right now the extension always builds vsc-workspace-id-features image on top of the images specified in devcontainer.json, even when no dev container feature is used.
If user specifies --pull always in runArgs, this behavior causes problems: Docker will try to pull the image from docker hub (and would't find it), despite it was just built locally by the extension, causing the workspace to not load. It also breaks pulling policy specified by user as now the local image is used so there's nothing to pull unless a rebuild is manually triggered on updated image.
I can think of two fixes of this for now: disable this local feature image creation if there's no dev container feature in used, or add an option to allow users explicitly opt-out from this local image creation.
Thanks for helping out!

VSCode Version: 1.72.2
Reproduces in: Remote - Containers

Steps to Reproduce:

create an devcontainer.json with "image": and runArgs: ["--pull", "always"]
open the workspace with Remote - Containers.
the workspace failed to load with docker error "...requested access to the resource is denied...", as it tries to pull a non-exists image from hub.

Sorry posted this in wrong repo. Moved to https://github.com/microsoft/vscode-remote-release/issues/7378