Documentation about the new Templated test suits

Hii,
I have seen the following in the release notes of the latest version v0.3.6
- Added templated Test Suites, to make re-usable tests possible (credits @hanseltime, resolves #57, resolves #186)

https://github.com/helm-unittest/helm-unittest/releases/tag/v0.3.6
However, I was not able to fully understand the changes in the code nor could I find examples or documentation about it.
Could you please shed the light on how to leverage this new feature
Thanks a lot in advance ^^

Hello @tewfik-ghariani,
See https://github.com/helm-unittest/helm-unittest#templated-test-suites for initial documentation and https://github.com/helm-unittest/helm-unittest/tree/main/test/data/v3/with-helm-tests as a example.
The feature is working as documented, but I did not get the change to fully test is outside of the helmcharts (and make the test-suite fully re-usable)
Greetings,
@quintush


FR: select all checkboxes command

First of all thanks for great plugin.
One thing I miss hear is ability to select all checkboxes before archiving.
Is there a chance to add such command to command palette?
Thanks

Hello, @mwoz123, please describe the workflow in more detail. I don't get what it means to "select all checkboxes".

plugin should go thought each line and does a change from
- [ ]
to
- [x]

This is already possible with simple 'search and replace' built into Obsidian:

right, but it's easier to have it in command palette (especially on mobile)...

Take a look at this plugin: https://github.com/jglev/obsidian-apply-patterns-plugin
It's a generic text replacement tool, and it does what you're describing right in the first demo in REAME 😺


ASP.NET Core

I'm following some ASP.NET Core introduction courses on the MVA, for instance "Introduction to ASP.NET Core with Visual Studio 2017" or "ASP.NET Core - Beginner". Are you going to move this courses to the new MICROSOFT-LEARN? Are you going to add ASP.NET and ASP.NET Core training features or paths?

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

ID: 26c7d914-e786-d705-977d-4f5fe050f420
Version Independent ID: 90e0d126-c3ff-00ec-b910-528c64a8c9db
Content: Introducing Microsoft.com/Learn
Content Source: teamblog/introducing-ms-learn.md
Product: team-blog
GitHub Login: @jeffsand
Microsoft Alias: jeffsand

I also studied ASP.NET Core 2.0 on MVA. I have completed the learning path and obtained the associated badge. When MVA will be retired, will this badge be available?

@davidArbeya @fabiocesarato Thank for both for your feedback on the blog.
I've re-assigned this issue to the Learn team to answer your questions.
Appreciate your contribution and involvement in our community.


[ci skip] README update.

@fordnox, can you directly merge this? I used the [skip ci] keyword to tell Travis-CI that this PR doesn't change anything in the code itself, and is just a documentation update, but GitHub expects Travis-CI to perform a test and report it's status back. As I used the keyword, Travis never will run a test and report it's result to GitHub, but GitHub requires the Travis test to be run before letting me merge the PR. So, you're the only one who can merge this now.

@evrifaessa I will try and merge this for you.

@evrifaessa Sorry it did not work.
MINGW64 ~/Documents/GitHub/boxbilling (evrifaessa-patch-1)
$ git merge master
Already up to date.
MINGW64 ~/Documents/GitHub/boxbilling (evrifaessa-patch-1)
$ git checkout master
Switched to branch 'master'
Your branch is up to date with 'origin/master'.
MINGW64 ~/Documents/GitHub/boxbilling (master)
$ git merge --no-ff evrifaessa-patch-1
Merge made by the 'recursive' strategy.
README.md | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
MINGW64 ~/Documents/GitHub/boxbilling (master)
$ git push origin master
Enumerating objects: 1, done.
Counting objects: 100% (1/1), done.
Writing objects: 100% (1/1), 242 bytes | 242.00 KiB/s, done.
Total 1 (delta 0), reused 0 (delta 0)
remote: error: GH006: Protected branch update failed for refs/heads/master.
remote: error: Required status check "continuous-integration/travis-ci" is expected.
To https://github.com/boxbilling/boxbilling.git
! [remote rejected]   master -> master (protected branch hook declined)
error: failed to push some refs to 'https://github.com/boxbilling/boxbilling.git'


Update contribution guide for merging PRs

This PR updates the contribution guide based on conversation with @LysandreJik that we try not to have the author merge the PR anymore.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


CoalesceBatchesExec reports no ordering

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
After https://github.com/apache/arrow-datafusion/pull/4122 some of our plans look like this:
+                "+---------------+-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+",
+                "| plan_type     | plan                                                                                |",
+                "+---------------+-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+",
+                "| logical_plan  | Sort: cpu.host ASC NULLS LAST, cpu.load ASC NULLS LAST, cpu.time ASC NULLS LAST                                                                                |",
+                "|               |   Projection: cpu.host, cpu.load, cpu.time                                                                                |",
+                "|               |     TableScan: cpu projection=[host, load, time]                                                                                |",
+                "| physical_plan | SortExec: [host@0 ASC NULLS LAST,load@1 ASC NULLS LAST,time@2 ASC NULLS LAST]                                                                                |",
+                "|               |   ProjectionExec: expr=[host@0 as host, load@1 as load, time@2 as time]                                                                                |",
+                "|               |     DeduplicateExec: [host@0 ASC,time@2 ASC]                                                                                |",
+                "|               |       SortPreservingMergeExec: [host@0 ASC,time@2 ASC]                                                                                |",
+                "|               |         SortExec: [host@0 ASC,time@2 ASC]                                                                                |",
+                "|               |           UnionExec                                                                                |",
+                "|               |             CoalesceBatchesExec: target_batch_size=4096                                                                                |",
+                "|               |               FilterExec: time@2 < -9223372036854775808 OR time@2 > -3600000000000                                                                                |",
+                "|               |                 ParquetExec: limit=None, partitions=[1/1/1/1/<uuid>.parquet], output_ordering=[host@0 ASC, time@2 ASC], projection=[host, load, time] |",
+                "|               |             CoalesceBatchesExec: target_batch_size=4096                                                                                |",
+                "|               |               FilterExec: time@2 < -9223372036854775808 OR time@2 > -3600000000000                                                                                |",
+                "|               |                 ParquetExec: limit=None, partitions=[1/1/1/1/<uuid>.parquet], output_ordering=[host@0 ASC, time@2 ASC], projection=[host, load, time] |",
+                "|               |                                                                                |",
+                "+---------------+-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+",

The CoalesceBatchesExec has been added after the FilterExec to make sure there are reasonable sized RecordBatches.
However  the BasicEnforcement rule has added an unnecessary SortExec
+                "|               |         SortExec: [host@0 ASC,time@2 ASC]                                                                                |",
+                "|               |           UnionExec                                                                                |",
+                "|               |             CoalesceBatchesExec: target_batch_size=4096                                                                                |",
+                "|               |               FilterExec: time@2 < -9223372036854775808 OR time@2 > -3600000000000                                                                                |",
+                "|               |                 ParquetExec: limit=None, partitions=[1/1/1/1/<uuid>.parquet], output_ordering=[host@0 ASC, time@2 ASC], projection=[host, load, time] |",

It is unnecessary because the data coming out of the parquet exec is already ordered (by host and time)
Describe the solution you'd like
The SortExec after CoalesceBatchesExec should not be present
I believe the issue is that CoalesceBatchesExec says its output is not sorted
https://github.com/apache/arrow-datafusion/blob/7c07e4d77aa2da60a83cc3558643eeac01fa98ce/datafusion/core/src/physical_plan/coalesce_batches.rs#L99-L101
Where the actual CoalesceBatchesExec preserves the ordering of its inputs, and thus should report ordering the same as its children
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
https://github.com/influxdata/influxdb_iox/pull/6160 which took us a while to work through ramifications of https://github.com/apache/arrow-datafusion/pull/4122

cc @ming535

LGTM

The same issue exists too with Repartition (hashing / round robin will maintain order)

I'm afraid Repartition can not maintain the ordering.
For example we have two ordered input streams [ 0, 10, 20, 25] , [ 5, 6, 7, 8]
after repartition (hashing), let's say the partition count is still 2
the two output streams might become
[ 0, 10, 20, 6, 8] ,[ 25, 5, 7]

Ah that's right. Was thinking about it maintaining order within the partition. Thinking whether there should be a way to still benefit from this (e.g. by keeping the partition number).


Very "easy" and "working" installation

C:\Program Files\obs-studio\OBS-Studio-Cursor-skin-2.3.0>mouse_skin_obs.py
Traceback (most recent call last):
File "C:\Program Files\obs-studio\OBS-Studio-Cursor-skin-2.3.0\mouse_skin_obs.py", line 1, in
import obspython as S  # studio
ModuleNotFoundError: No module named 'obspython'

C:\Program Files\obs-studio\OBS-Studio-Cursor-skin-2.3.0>python -m pip install obspython
Looking in indexes: https://pypi.org/simple, https://pypi.ngc.nvidia.com
ERROR: Could not find a version that satisfies the requirement obspython (from versions: none)
ERROR: No matching distribution found for obspython
C:\Program Files\obs-studio\OBS-Studio-Cursor-skin-2.3.0>py -0
Installed Pythons found by py Launcher for Windows
-3.10-64 *

You need to load this script into OBS Studio, it will not work as a standalone program - see first part of this tutorial.

There is no Cursor option in Sources window after load the script mouse_skin_obs.py in Tools-Scripts

"Cursor" is just a name for a source, e.g media source, image source. You have to add it yourself and select that source.

unbelievable ...


Skeleton for Cosmos DB OLTP Spark Connector

closing this PR as
feature/cosmos/spark30 merged to master here: https://github.com/Azure/azure-sdk-for-java/pull/20086


Limit EIDR event creation to admin users

Only admins can create and update events. Buttons associated with these tasks have been hidden for all other users.

I am going to hold off on merging this until we have a way to create admin users.

@mawaruDOLL Could you resolve the conflicts in this?

The conflicts are resolved now.

👍


Crash fix and code cleanup.

Adjusted some path handling to use equivalent system functions, eliminating some unnecessary OS assumptions.
Bit of namespace cleanup.
Fixed a couple build warnings.
Excluded the "Old" directory from the project. (Recommend deleting the files from git too; Use git history?)
Fixed a crash when clicking buttons in settings area before highlighting a preset in the list.

Looks good thank you :D


Issue with special character

When the folder name have this character ->  ©
Rclone cannot copy to amazon cloud drive, no error message is show, but the upload is marked as completed.

I uploaded a file called  ©.txt and viewed it in the web interface just fine.  I also downloaded it with rclone.
Can you send me a log of it going wrong? Of make me an example that I can reproduce?
Thanks
Nick
$ mkdir /tmp/z
$ echo "Hello" >/tmp/z/©.txt
$ cat /tmp/z/©.txt
Hello

$ rclone -v copy /tmp/z acd:test-copyright-symbol
2015/12/23 11:03:13 acd: Saving new token in config file
2015/12/23 11:03:14 test-copyright-symbol: Failed to read info: Node not found
2015/12/23 11:03:14 Amazon cloud drive root 'test-copyright-symbol': Modify window not supported
2015/12/23 11:03:15 Amazon cloud drive root 'test-copyright-symbol': Building file list
2015/12/23 11:03:15 Amazon cloud drive root 'test-copyright-symbol': Finished reading
2015/12/23 11:03:15 Amazon cloud drive root 'test-copyright-symbol': Waiting for checks to finish
2015/12/23 11:03:15 Amazon cloud drive root 'test-copyright-symbol': Waiting for transfers to finish
2015/12/23 11:03:16 ©.txt: Copied (new)

Transferred:            6 Bytes (   0.00 kByte/s)
Errors:                 0
Checks:                 0
Transferred:            1
Elapsed time:        3.2s

2015/12/23 11:03:16 Go routines at exit 16

$ rclone -v copy /tmp/z acd:test-copyright-symbol
2015/12/23 11:03:32 Amazon cloud drive root 'test-copyright-symbol': Modify window not supported
2015/12/23 11:03:32 Amazon cloud drive root 'test-copyright-symbol': Building file list
2015/12/23 11:03:32 Amazon cloud drive root 'test-copyright-symbol': Finished reading
2015/12/23 11:03:32 Amazon cloud drive root 'test-copyright-symbol': Waiting for checks to finish
2015/12/23 11:03:32 ©.txt: Sizes identical
2015/12/23 11:03:32 ©.txt: Unchanged skipping
2015/12/23 11:03:32 Amazon cloud drive root 'test-copyright-symbol': Waiting for transfers to finish

Transferred:            0 Bytes (   0.00 kByte/s)
Errors:                 0
Checks:                 1
Transferred:            0
Elapsed time:       600ms

2015/12/23 11:03:32 Go routines at exit 13

$ rclone -v copy acd:test-copyright-symbol /tmp/zz
2015/12/23 11:05:11 Amazon cloud drive root 'test-copyright-symbol': Modify window not supported
2015/12/23 11:05:11 Local file system at /tmp/zz: Building file list
2015/12/23 11:05:11 Local file system at /tmp/zz: Waiting for checks to finish
2015/12/23 11:05:11 Amazon cloud drive root 'test-copyright-symbol': Finished reading
2015/12/23 11:05:11 Local file system at /tmp/zz: Waiting for transfers to finish
2015/12/23 11:05:11 ©.txt: Copied (new)

Transferred:            6 Bytes (   0.01 kByte/s)
Errors:                 0
Checks:                 0
Transferred:            1
Elapsed time:          1s

2015/12/23 11:05:11 Go routines at exit 15

$ cat /tmp/zz/©.txt
Hello

Hello, I don´t know why the issue happens, but I´m having trouble with characters like this:
é ó í á ã õ ç è à ì
2015/12/23 19:43:56 A Hist▒ria da Geologia /A Hist▒ria da Geologia - Epis▒dio 1.mkv: Failed to open: open /root/toupload/A Hist�ria da Geologia /A Hist�ria da Geologia - Epis�dio 1.mkv: no such file or directory

My Locale:
LANG=en_US.UTF-8
LANGUAGE=
LC_CTYPE="en_US.UTF-8"
LC_NUMERIC="en_US.UTF-8"
LC_TIME="en_US.UTF-8"
LC_COLLATE="en_US.UTF-8"
LC_MONETARY="en_US.UTF-8"
LC_MESSAGES="en_US.UTF-8"
LC_PAPER="en_US.UTF-8"
LC_NAME="en_US.UTF-8"
LC_ADDRESS="en_US.UTF-8"
LC_TELEPHONE="en_US.UTF-8"
LC_MEASUREMENT="en_US.UTF-8"
LC_IDENTIFICATION="en_US.UTF-8"
LC_ALL=
Running rclone v1.25 for ARM on Ubuntu 14.04.2 LTS ->  GNU/Linux 3.2.34-30 armv7l - Marvell (Proprietary)

Ah, I suspect your file system is storing file names in latin1, not UTF-8.  rclone expects the local filing system to be in UTF-8 which all modern filing systems are.
You can see that because your terminal is set to UTF-8 but when you run ls you get ? in your file names instead of the accented characters.
I did a bit of searching - you might find this helpful: http://serverfault.com/questions/54911/debian-how-to-convert-filesystem-from-iso-8859-1-into-utf-8
If you do convert your filing system to UTF-8 then rclone should work fine!

You should probably check out the local file system docs too which has a section on encoding.

Thank you, I found a solution, just convert all files to utf-8 :)
convmv -f latin1 -t utf8 -r /pathto

Great - that is what I would have done :-)


Bump gems

Bump ecma-re-validator and regexp_parser to latest versions.

Thanks for the pr, @gciltaru! It looks like ecma-re-validator 0.4.0 isn't compatible with Ruby 2.5 or 2.4. I'm open to dropping support for those since they're EOL, but I've gotten some push back in the past: https://github.com/davishmcclurg/json_schemer/issues/76 Is there a reason you need these gem versions bumped?

Hi, The reason was upgrade to ruby3, but it seems that I was also relying on an older version that treated the v4 ids differently than in the latest version. I am closing this PR. Thank you!


bi-color scale

please make it possible to use two-color in the scale (ref #112 ). Thx.

Try: data('6375921d-af93-41a3-8204-83fc1fc708c1') x(DOW) y(Hour) color(#count:[white,purple]) sort(#count) tooltip(#count) style('symbol:rect; size:100%; stroke:none')
Is this what you are looking for?  If so, full docs are in the "Aesthetics" section in the online docs.

I added an example to #112

I'm afraid that I cannot make Brunel's Heatmap scale to bi-color. I'm tried many combinations, but just can't make it look anywhere close to bi-color scale.
The data I want to chart is here, which is aggregated from this.
This is the standard Seaborn Heatmap

From this Seaborn Heatmap we can clearly see where the "focuses" are. I can't make Brunel's Heatmap looks anywhere close to it.
Can somebody show how to create a bi-color scale Brunel's Heatmap which look like this please?
Thanks

Here is an example that is fairly close assuming the aggregations are averages.  Note, you will need to use the pre-aggregated data-- or restructure the aggregated data:
data('https://gist.githubusercontent.com/suntong/416bcaa0ae3156c5a15cbcc7018344d1/raw/6fd49a1be8ce7278bd7fcd411c3300c42cbbd0f5/HeatMap.csv') x(DOW) y(Hour) color(changes:[white, purple]) label(changes) tooltip(#all) mean(changes) style('symbol:rect; size:100%;) stroke:none')

Flipping the y-axis is addressed here:  https://github.com/Brunel-Visualization/Brunel/issues/110
The labels not appearing is addressed here:  https://github.com/Brunel-Visualization/Brunel/issues/109
Scales starting below zero is mentioned here:  https://github.com/Brunel-Visualization/Brunel/issues/104

Thanks dan,
This is how it looks (for data('https://gist.githubusercontent.com/suntong/416bcaa0ae3156c5a15cbcc7018344d1/raw/6fd49a1be8ce7278bd7fcd411c3300c42cbbd0f5/HeatMap.csv') x(DOW) y(Hour) color(changes:[white, purple]) label(changes) tooltip(#all) mean(changes) style('symbol:rect; size:100%;) stroke:none')),

The problem is, I'd called it single-color theme, as it looks like there are holes in the graph without data, but in fact, the bi-color theme can clearly shown all data are available:

Further specifying two color does not means we will get bi-color scale, because white is actually no-color on the normal web display. As soon as Ichange the "white, purple" to "beige, purple", it shows that two-color specification actually give three-colors:

Ref: here, two-color specification actually give three-colors&width=775&height=425&title=&description=&show_brunel=true)
So, please consider again to make the two-color specification means gradient fill from one color to another, just like the above Seaborn Heatmap, instead of giving three-colors.
Thanks

I am re-opening this.  We will add a way to force it to just 2 colors.

Thanks Dan.

In the 2.0 release (code now on master, release strongly expected in the next couple of days) we will add the option to specify the middle color as 'none', so the following will make a "true" 2 color scale:
color(field:[blue,none,red])
The reason for the default to three colors is that themes common use for a 2 color scale is for a divergent field, one where you want to emphasize the ends. For this, a neutral middle value is much better for interpretability. A color scale going from one hue to another is tricky for people to interpret unless the colors are chosen carefully, so we don't default to that and require an extra step of specifying none to make a true 2-color scale.
As a note, we are using HCL interpolation rather than RGB, which ensures that colors do not get mixed into "muddy" grayish values. If you need a more RGB-ish look, manually calculate the internal RBG values you want and make a 5-color scale and it should look pretty close.

:+1:


Not working with BandagedDiscord (April28th)

My discord updated on April 28th completely breaking BetterDiscord and not making it load whatsoever, I found out that most people are using 'bandaged better discord' (https://github.com/rauenzi/BetterDiscordApp) which indeed, made it work again.
But even tho it says it loads the quicksave plugin and I can see it in my settings, quicksave is not showing any of it 's save buttons on any images. Not in servers, not in private conversations.
This could absolutely be a problem on my end but I'd really love to find the root of the problem and fix it. I've tried using an older version of quicksave (2.3, currently using 2.5) and when starting up it says it loads them but their functions dont appear.
Made sure they are turned on and turned them off and back on, restarted discord, the buttons still aren't showing up on any images.
Thank you for your time and hard work !

Not working on my end either. Discord devs have been actively killing BetterDiscord plugins lately over the past couple weeks. Most of my plugins are broken now.

Discord renamed all of the classes again. Fixed.


Add support for local file provider

Registering models and metadata in the cloud is not always feasible.
To that effect we want to be able to add a provider to store these things on-premise

We probably want to start off with supporting SQL Server and a local file share. It would make sense that this was added to a separate package MLOps.NET.Local.SqlServer

@aslotte , do you mind if I'm considering adding SQLite? I think it will do just fine here, instead of SqlServer. But we can think of doing both, SqlServer and SQLite.

@dcostea we can certainly look at SQLite if you like. My thinking is that most organizations that want to start of locally will have some kind of SQL server set up, but that may be a misconception :)
Would love to discuss it tomorrow if you're joining the stream. I'm thinking we may want to spend a bit more time finding a general API we're happy with in Azure before working on the local solution.

@aslotte , sure, let's talk tomorrow.

I agree with @dcostea. For development purposes, using SQLite is easier to use instead of a full server setup. I completely agree that enterprises will want to use  connect with SQL server, but that can be implemented later.
Also, I'm unsure of the naming convention of MLOps.NET.Local._______. I am unsure of the 'Local' addition. Because if we used that, shouldn't we change the name up for MLOps.NET.Azure to MLOps.NET.RemoteStorage/Cloud.Azure. If we drop the '.Local", it will be more consistence with the current naming schema because the extension's name should be self-evident enough.

Great input @sammysemantics. To address some of your points:

I'm all good trying SQLite to start with
Maybe the name of the package should be MLOps.NET.SQLite?

Like that we can have additional packages for e.g. MLOps.NET.AWS if we open up for AWS as a storage provider.

It could also be called MLOps.NET.LocalStorage.SQLite and we could have MLOps.NET.LocalStorage.SqlServer` etc.

If we rename it MLOps.NET.LocalStorage.*, should we rename the Azure extension to MLOps.NET.RemoteStorage (or CloudStorage).Azure? Because I do see more extensions to this library that fits outside of storage and focuses on other operations in the MLOps lifecycle.

@sammysemantics - what do you think about re-naming it to MLOps.NET.Storage.Azure and MLOps.NET.Storage.SQLite etc.?

@sammysemantics - what do you think about re-naming it to MLOps.NET.Storage.Azure and MLOps.NET.Storage.SQLite etc.?

For me, it's better this way. Azure and SQLite are intrinsic local or remote, so there is no need to encode this in the namespace.

@dcostea just so I understand your last comment, do you think they should be called MLOps.NET.Storage.Azure or MLOps.NET.Azure for example?
I personally prefer MLOps.NET.Azure but I'm good with either and want to make sure it makes sense to you, @sammysemantics and others.

If we don't plan to have more modules related to Azure, then I prefer the shorter one, MLOps.NET.Azure. Similarly MLOps.NET.SQLite.

Agreed.

Perfect, that sounds great.


Resource Override doesn't work on Chrome 84 (workaround: Source > Filesystem)

On Chrome 84 Network and Sources, etc. shows Resource Override URL to URL doesn't have any affect, the extension is no longer working.
After more checking, I find a workaround by adding local path (that is also defined in Resource Override URL to URL) in Chrome DEV Tools > Source> Filesystem. By some reason Chrome DEV Tools > Source > Overrides doesn't workaround it, neither when only using Source> Filesystem.

Could you give more details please? I wasn't able to reproduce. I was using Chrome Version 84.0.4147.105 (Official Build) (64-bit) on Mac OSX 10.13.6 and was able to redirect one website to another.

I reproduced the issue on latest Chrome Windows 10. Used valid regex for URL to URL override.

Valid regex? The url match strings are not run through a regex engine. The "*" syntax is custom. Are you able to post the url strings?
Also just tested on Chrome Version 84.0.4147.105 (Official Build) (64-bit) on windows 10.

I am running into this on chrome 85.0.4183.102.  It is a bit more nuanced that the initial report.
I have the following override:
https://static.mycdn.com/*/my-app.min.js --> http://<IP_ADDRESS>:8081/my-app.js
It loads the non-minified version from localhost, as expected. This can be seen by the page content and the network tab.  The minified version has a 307 (internal redirect) in the network tab.
However, when I go to the sources tab in dev tools I cannot open the non-minified version. The only file available in the sources tab is https://static.mycdn.com/latest/my-app.min.js.
So in summary the plugin is still overriding the page content, but not the dev tools.


displayed alert on top

What type of PR is this?
/kind ui-change
What this PR does / why we need it:
To display alert on top
Which issue(s) this PR fixes:
Fixes #1092


Minor spelling fixes

Pull Request Test Coverage Report for Build 26

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.749%

Totals

Change from base Build 25:
0.0%

Covered Lines:
137

Relevant Lines:
139

💛  - Coveralls


List ancestors when shipping a child branch

@charlierudolph @allewun @ricmatsui
implements #562

:ship:


Missing armv7l (raspberry pi) binaries

Hi @Yeraze!
Thanks for the feature request, I'm curious about your use case for Savvy on a raspberry pi?

Could you also share the output of these commands from your raspberry pi:
os=$(uname -s | tr '[:upper:]' '[:lower:]')
arch=$(uname -m)
echo $os
echo $arch


Auto add license

Is your feature request related to a problem? Please describe.
if the license is invalid, by saying @zeobot add license, the license will be added to the files that do not contain a valid one.
We also happily accept pull requests.

Please use spotless


NIFI-1783: Load Authorizer

Addressing mistake in exmample XML.
Ensuring the configured Authorizer is loaded correctly.

Reviewing...

looks good, passes contrib-check, will push to master


Use poseidon with memory in the riscv machine

Depends on https://github.com/powdr-labs/powdr/pull/1655
This PR is based on https://github.com/powdr-labs/powdr/pull/1533 on top of registers in memory and the new links.

Did you measure how many more rows we need in the bootloader now?

Not yet.

We should assert that page 0 is never paged in, as this would override the scratch space.

Done in this PR.

We should also assert that the page size is at least 512 bytes (otherwise we'd have to add the above assertion for more pages)

This we can do in Rust code, no?

The current approach does break soundness for programs that access the scratch space in the regular program: The prover would not page-in this memory region, but could still access it, because it has been initialized by the bootloader. Maybe open an issue for this?

Done: https://github.com/powdr-labs/powdr/issues/1664

We should also assert that the page size is at least 512 bytes (otherwise we'd have to add the above assertion for more pages)

This we can do in Rust code, no?

Yes exactly, just an assertion that fails if anyone ever lowers the page size too much.

all comments addressed, ready for review again @georgwiese


Allow custom classes on filter input

This is similar to #30 and allows custom classes to be added to the filter text input.
<table-component
    ...
    filter-placeholder="Filter"
    filter-input-class="input"
    filter-no-results="No matches for your search criteria"
    ...
>

I figure a pretty common scenario is that a dev is using a framework that provides classes to style text inputs i.e. Bulma's .input selector.
This means we don't need to re-apply all these styles to the built in filter class .table-component__filter__field.

Thanks!

no, thank you!


Fix broken menu

Related Issues

Closes #509
Short Description and Why It's Useful

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: lukaszjedrasik:white_check_mark: Fifciu:x: FifciuuFifciuu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


GitHub Actions cache

Fixes #95

Stats:
Before the changes:
install 7-9 mins -> run build 1:15min -> 8-11 min
Now:
Get Cache 5 min -> install 50 secs -> run build 1:15min -> 7 min
Not as much as i thought but at least it's a bit faster.
Getting the cache now takes most of the time, I think this is a windows problem..
Unzipping the yarn cache takes about 1:45min on windows, vs. 7 seconds on ubuntu

The execution time improved significantly. We just have to clean everything up
Changed Timings (old -> new):

Frontend

Unit: 7:45 min -> 3:45 min
e2e: 10:45 -> 6:50
build: 14 -> 4

The servers were a bit slower than normal on the "old" test run, but still it's an improvement.
We just have to clean everything up.
Screenshot of the runs.

What has been done:

Removed cypress-io/github-action since the cache was not used efficiently in our use case
Added custom caches. These caches depend on the os, node-version, and the corresponding hashfiles for frontend or backend. The following caches are implemented:

Frontend: node_modules, cypress
Backend: node_modules

Added workflow build scripts in the build folder (see comment in build/workflows.js)

With that, workflows with reusable steps can be created
Reusable steps in this way via composite actions didn't work (see f62fec8, b173172)
The node script imports a shared action from build/workflows/actions into all workflows defined in build/workflows and writes the outputs to .github/workflows

The runtime is now reduced for every workflow (if the cache was set up, so if yarn.lock is unchanged. If it is modified, then it takes about the same time as before).
It's still relatively long on windows, but that's an windows problem (e.g. unzipping the cache files on windows (205MB) takes ~2:00-2:30min, on ubuntu (187MB) 7 seconds).


Update README usage section

Hi!
I found this gem in GitHub explore, it's nice one😁
Is your proposal related to a problem?
(Provide a clear and concise description of what the problem is.)
I think the RailsMiniProfiler::Engine should be mounted in the development environment only.  So I think It is a good idea to update the README that way as well.
Describe the solution you'd like
(Provide a clear and concise description of what you want to happen.)
How about updating usage section as follows?
# routes.rb
Rails.application.routes.draw do
  ...

  mount RailsMiniProfiler::Engine => '/rails_mini_profiler' if Rails.env.development?
end

like letter_opener_web
Describe any alternatives you've considered
(Let us know about other solutions you've tried or researched.)
Additional context
(Is there anything else you can add about the proposal? You might want to link to related issues here, if you haven't already.)

Thanks a lot for your request! Glad to hear you enjoy this gem :blush:
The thing about RMP is that is actually intended to be used both in production as well as your local development environment. Similar to rack mini profiler, profiling is available in production per default, but requests that you want to profile need to be whitelisted.
This is to avoid impacting performance for your end-users negatively, while still giving developers the option to perform some profiling in production. Profiling in Production should explain this in more detail I hope...
Do you have suggestions on how to improve the documentation to bring that across better? :thinking:

@hschne
Thank you for your thoughtful answer!

The thing about RMP is that is actually intended to be used both in production as well as your local development environment.

Sorry, I misunderstood about these things.
I was worried that anyone could access /rails_mini_profiler in the production environment by default, but I see that RMP is disabled in the production environment by the default configuration.
I had overlooked that 😅
I think the README is clear enough now!
It might be clearer if it's written that /rails_min_profiler is only accessible in the development environment by default.

@kenzo-tanaka Sooo, inspired by your request I did some testing and found some pretty severe bugs related to authentication/authorization in Production. Big thanks for pointing me that way! :grin:
I made some fixes and updated the Readme section about production profiling a bit. I believe it is a bit clearer now.

@hschne Thank you for comment! I'm glad I could help 😁


fix: docs indent

Change Summary
Doesn't need a change file

Related issue number
closes #2439

Checklist

[ ] Unit tests for the changes exist
[ ] Tests pass on CI and coverage remains at 100%
[ ] Documentation reflects the changes where applicable
[ ] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)

👍 👌


Do not stop on TypeError

Hi!
Thanks for a great lib!
For now there seems to be no clean way to prevent tcomb from throwing on first encountered TypeError, but throw all errors in the end of the check.
We need this to run tests in Jest and validate some models. Currently tcomb reports about first invalidation in struct and ejects, but we'd like it to run complete check.
I've tried to do something stupid. like this:
    let errors = []

    t.fail = (message) => { return errors = [...errors, message] }

    Schema(data)

    throw new TypeError(errors)

But it throws
    TypeError: Cannot read property 'length' of undefined

      at List (node_modules\tcomb\lib\list.js:40:32)
      at create (node_modules\tcomb\lib\create.js:9:104)
      at new Struct (node_modules\tcomb\lib\struct.js:82:19)
      at Struct (node_modules\tcomb\lib\struct.js:71:14)
      at Object.<anonymous> (tests\modules\other.test.js:36:5)
      at process._tickCallback (internal\process\next_tick.js:103:7)

Besides, it still feels like a hackable way rather than a solution to the issue.
Am I missing something?

We need this to run tests in Jest and validate some models

maybe https://github.com/gcanti/tcomb-validation ? its validate API returns a list of errors

Ouch, right! I saw tcom-validation, but I assumed that it's more for assertions like true/false. Missed that it also can return list of errors.
That exactly what we need, thanks!


Bootstrap v4 support?

Are there any plans for bootstrap 4 support?

It seems to run with BS4 beta-2 nicely for me, however it would be nice to know that it's officially supported.

Any news on this particular issue?

@rolandoisidoro  You're right,  I had a mistake, the repo I wanted to share is https://jaspero.co/resources/projects/ng-notifications

This should work with any version of bootstrap by simply changing the template to include the new class names.
I will look into this in more detail since I now use bootstrap 4 too.
Also sorry for the delay. I took a year-long break because I was working in a different field that used up way more of my free time than expected.


Make solium less opinionated by default as far as styling is concerned.

Hey,
I recently integrated solium to emacs solidity mode. It's great but also rather opinionated.
I know that these can be configured and turned off but it would be better if the user could avoid style suggestions unless requested. I find it annoying to have to disable all style rules. And for my styling rules there is no enforcement. For example I can't enforce snake case.
Examples of opinionated errors:
 warning    FunctionDeclaration must be succeeded by 1 blank line
error      Opening brace must be on a new line immediately after the last modifier.
 error      Opening brace must be on the line after returns declaration

And the last 2 should only have been warnings. There is no error involved. It's a style choice to have the brace like that, and a rather popular one called K&R.
Also this warning is rather off:
 warning    Function "ecrecover": in case of more than 3 arguments, drop each into its own line.

It is not the number of arguments that count but the number of columns they take. Most projects I have worked in always keep arguments in one line, unless the function call exceeds the XX column limit (with XX being either 80 or 100) in which case yes indeed each argument is split into its own line.
I suggest having style choices grouped into something like the different indentation styles seen here and allow the user to choose one and then fine-tune through specific options. But not only support the camelcase style that is supported now.
If you want to have a style linter make it one that can be used by all groups, not only one particular style.

Agreed, I believe the solidity style guide's rules are more suited as an optional ruleset (the current default ruleset is solium:all that you see in your .soliumrc.json which basically applies all the rules).
IMO here is how we can improve the linter:

Add rules for all the different types of styles (snake, camel, etc) - all disabled by default.
Allow user to enable/disable rules based on their own styles
Because the 2nd step can be annoying depending on how much customization you need to do, solium can provide some other rulesets (eg- solium:style-guide, solium:knr-gnu, etc or sharable configs for that matter) that one can switch to.
The default ruleset applied when running solium --init will be solium:recommended - this itself would contain only a small no. of style rules which would generally apply to all solidity code (so wouldn't complain about your variable naming style for example)

Feel free to give your input on this^, thanks for the detailed feedback!


repo: Ignore dot files by default (+some code check ignores)

Ignore dot files in .gitignore and .bazelignore, unless
explicitly included.
Also ignore dot files in code check script
Signed-off-by: Ryan Northey<EMAIL_ADDRESS>Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@adisuissa i have cleaned this up a bit - i think the remaining changes are pretty non-controversial
would be great to land, as atm im rebasing other PRs against it so i can test locally - without it a check that takes 2mins takes 20mins
apologies for rebase/force - i messed up locally while working on it

/retest


for usage with influxdb and grafana the json string needs to be extended

Is it possible to add the Name and the mac address to the json string which is published to mqtt so by making use of node red this fields also gets added to influxdb.
at the moment only the main values get written into influx but without name and/or mac address you cannot define respective dashboards for the plants.
as i am not so keen in your code this would help me to get further with my project having NodeRED-MQTT and your deamon pushing data into influx for usage with grafana !
Big Tnx
Alex

ok if ixed it now for me and for the mqtt_json reporting by changing the data generating section.
for param,_ in parameters.items():
if reporting_mode == 'mqtt-json':
data['name'] = flora['name_pretty']
data['location_clean'] = flora['location_clean']
data['location_pretty'] = flora['location_pretty']
data['mac'] = flora['mac']
data[param] = flora['poller'].parameter_value(param)
print_line('Result: {}'.format(json.dumps(data)))

I can't see any harm in this. Will include soon

Would be great, now I need to encode this in topic by using very fake names ;)


First version of the Travis build

Hi all,
I've build a simple CI to run the tests. This will run both OpenJDK 8 and 11, of which the OpenJDK 8 is required to pass. But I've also added OpenJDK11 for the future. We can also add more versions if required.
Cheer, Fokko

I've also added Java 9 and 10 because I was curious if it would pass :)


Use common labels instead of selectorLabels for pod labels

Signed-off-by: Tim Chaplin<EMAIL_ADDRESS>
What this PR does / why we need it:
By default, a newly generated Helm chart provides a labels macro and a selectorLabels macro. selectorLabels is currently used for the selector.matchLabels section of a Deployment, as well as template.metadata.labels. As a result, pods only get a subset of the common labels applied. I couldn't find any documented rationale for why you would only want to use selectorLabels for pods, so my guess is that it's a bug.
Special notes for your reviewer:
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

This is an interesting one. The labels include that it's managed by Helm and more. Looking back at the history it has long been what we now use as the selector labels. This is a bit philosophical on what should be there.
Any thoughts @helm/charts-maintainers ?

I completely agree with this. The Deployment's selector:
  selector:
    matchLabels:
      {{- include "<CHARTNAME>.selectorLabels" . | nindent 6 }}

so the pod's labels should also come from selectorLabels in order for the Deployment to select them.

@tim-chaplin-dd can you rebase this please?

@tim-chaplin-dd can you rebase this please?

Done, sorry for the delay!


UI mockup

Here's a quick drawing of how I imagine the UI currently. The elements could be floating, as with Fiji and in the drawing, or they could be docked in a grid, as with Spyder, Pycharm, RStudio, etc. I'm not set on either model.

There can be as many viewers as the user wants, and as many layers in a viewer as a user wants. We can think about the behaviour for new outputs. Lots of options, probably some combination of which makes sense:

new outputs go on the workspace, user has to drag to a viewer explicitly to view them
new outputs go on top of the input image if that image is already in a viewer
new outputs go on top of the current viewer if that viewer is selected when invoking the function
if we have a dialog to run a function, behaviour of outputs can be selected from a pulldown

Looking forward to everyone's comments!

PS the mockup was created with the Linea app on the iPad Pro, which I highly recommend. It's the only sketching app I've found that just works the way you want it to.

Very nice Juan, I like it.
With Kira we are focussing on the viewer for now,
Something to add to this mockup is histogram side-panel
with control over min, max and gamma. Also nice would be
the 'live-parameter' panel that avoids the 'let's-pop-up-a-blocking-window-
and-ask-for-parameters-that-nobody-ever-changes'

We should clarify the terminology: channels belong within numpy images, layers are numpy images or other types of ‘renderable’ or ‘non-renderable’ objects (lines, boxes, etc…).

:+1: Yes, very good, agree 100%, sorry for my sloppiness there. =)

I think the plugin’s function decorators should specify whether live update is the default or not,
moreover, the user should be able to override this globally, and interrupt a running calculation…

Sure. These are all great. But I think an MVP can forgo all this and ignore live updating, unless someone decides that live updating is trivial to implement. I'm not going to decide that. =P


Fix filter functional bugs

Closes #178, #171, #169
TODO:

[x] Fix #166

Change was made to include equality with lts/ and gts/ as without it, the filter command would cause an error with applicants who have un-rated interviews

Now closes #180


Cygwin Installation in windows 7

I am trying to install cygwin in windows 7 during the installation,while selecting the website starts with mirrors nothing is getting downloaded showing message nothing to download because of that i am not able to run commands

Cygwin is totally separate from WSL, you should start here:
https://cygwin.com/docs.html


Deprecate content fields of article table

Background
A follow-up task of article versioning. Keeping these legacy fields on article table could misguide our development.
Checklist

[ ] ipns-listen-repub is still relying on article table [0]

article table act as pointers that point to different versions in draft table. article table record the newest version and metadata, while draft table record different version of content.


STP - Enhancement - There should be option for user to remove the record from zstore

As we will want to give STP to user usage, we need to add option to remove the record from zstore
I dont think that user should do purge on zstore record
Saying that I suggest that there will be some option to do that
We might remove the zstore record automatically if agent is remove or give some other UI option to the user

@yanpom do you think its reasonable to use the zz API for the user to remove tokens if they want to reset?
Removal of agent is also a bit tricky. I fear that if the code to read config infotons fails we could end up deleting tokens and resetting agents etc... Should we not for now document that if the agent is removed you must remove the zstore entry to make sure the end user is really considering what they are doing? I am concerned at the unintended consequences.

I don't want the user to remove it from zstore. I want user to remove the agent itself, with an option to clear the zstore and then we will remove the zstore

@yanpom The problem is, at present, the way you remove the agent is through an API call to remove the config infoton. We can't make this call also to delete from store as the API is generic.
Do we want to stand back and consider if we want to develop the UI to manage agents ?
Only other option is for STP manager to check what agents are running and remove zstore records for all non running agents. This is dangerous however is the config infotons fail to be read.

We can pass a parameter with purge of agent infoton, which will remove the record from ZZ
Anyway, we need a solution for customer, we wont be able to always control what customer does with STP

After a discussion with @nicklester we suggest the following:
adding a new button to our beloved STP page which will let you remove a token and then will ask if you want to remove data from zstore as well.
Behind the scene this will be implemented as a API call.

@YoniMataraso - can I confirm that you means to "remove a token and then will ask if you want to remove data from zstore as well"

@nicklester exactly what I thought.

@yanpom @YoniMataraso  Quick thought - is it worth having a ui for zz generically so we can remove any object stored in zstore?

Pause/Resume/Reset tokens functionality works
But:

Only Root user can do it - we need to change the permissions to any privileged user
UI part - on each click another success:true window appears, we should stay on same window, because of the fact that it takes up to 30 sec for change to occur, we need to change immediately from Pause- Pausing, Resume - Resuming, Reset - Reseting and stay on same page while the change occurs.

Verified and closed


Adding support for api routes

Hello react-pwa team ... thanks for your hard work creating this repo:
I realize that you could make a fetch call inside the pages dir by adding a fetch call like        fetch('https://www.atyantik.com/wp-json/wp/v2/posts/?per_page=4&_fields[]=title&_fields[]=excerpt&_fields[]=jetpack_featured_media_url') .. however that is a third party api.
I have two following questions:

I might be missing a point here but is there a way to add api routes and backend logic to this repository.  It seems like not supported  since you have same RouteHandler for client and server.js. If it is not supported what is the recommended way of adding api?

How can I use a config variable lets say inside guest.js and reference  vars  like const apiHostUrl = process.env.npm_package_config_apiBaseUrl; So that I could make calls like         fetch(${apiHostUrl}wp-json/wp/v2/posts/?per_page=4&_fields[]=title&_fields[]=excerpt&_fields[]=jetpack_featured_media_url)

Hello @nodeEnthu, you can have a look at this repository as to how internal api's are made.

thanks

for my second question... is there a way to refer variables  inside the config file (pawconfig.json)  in the code .. depending upon which env it is..

@nodeEnthu I am not clear about your question completely.
But if you want to use different config for development and the other one for production. For Eg: SSR during development is not needed, in that case you can use it in a way mentioned in the examples. In examples we are using separate config file for production.
Repository: Example PawJS AntDesign
package.json showing how it can be used. You can see here that there are two different configs pawconfig.json( for development) and  prod.pawconfig.json(for production).


Fix adapters links

None of the adapters links were working, so I fixed them

@vfonic Thank you for submitting this PR! These gems are not available yet, so I prefer to merge #592 for now.

Sounds good to me! 👍


Use audited dependencies

@stablelib/ed25519, @stablelib/x25519, elliptic - replace with noble-curves
@stablelib/random, @stablelib/sha256, js-sha3 - replace with noble-hashes
bech32 - replace with scure-base
@stablelib/xchacha20poly1305, canonicalizedid-resolver, multiformats, uint8arrays will remain

Wanna me send a PR?

Not a maintainer, but I think it would be really nice to use audited packages.
I have one question though. Do these libs work on React Native?
As there is overlap in maintainers between Veramo and this package and one of the requirements of Veramo is that it also has to run on RN, this package also needs to run on RN.
If it does not it would break certain apps, like our wallet.

Yes they do

Cool
@mirceanis  What do you think. If you ask me it would be great to have audited packages for these core functionalities needed in this lib

@paulmillr a PR would be highly appreciated! 🙏
https://discord.com/channels/878293684620234752/933661591264702474/1081295858714296491

I'm in favor of a replacement.
I would have liked to move forward with #170, which aims to refactor this library to allow folks to bring their own crypto implementations instead of us bundling things in, but since I haven't had any time for that it makes more sense to do what you suggest.

I see you have ES256K, ES256 signers. How should this rewrite behave? Should I simply replace elliptic with noble in these files, or should I create new signers?

Great minds think alike: https://github.com/decentralized-identity/did-jwt/pull/280
Have just noticed this thread, otherwise would ping you pals first before doing any code.

Hey @ukstv , this is wonderful, thank you for doing this!
I was wondering if you're able to do the same for the did-provider-key lib, which still has the @transmute/... dependenceis and they marked as unstable releases. There have been discussions of replacing those as well. That would be amazing!

@inevolin The package you mentioned is not directly used by Ceramic ecosystem AFAIK, so I would like to politely refuse the invitation.
What I could propose instead, is maybe you could depend on key-did-provider-ed25519 or key-did-provider-secp256k1 which are cared for. As soon as  https://github.com/paulmillr/noble-curves/pull/32 gets merged, we could fully migrate to noble-crypto and leverage its speed and dependability there.

interesting! thank you


[DO NOT MERGE] Auto PR to update dependencies of istio

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
43cf72fb54cdc3dfe82a744e0be210cf99450ed3
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
43cf72fb54cdc3dfe82a744e0be210cf99450ed3
link
/test istio-unit-tests

prow/istio-pilot-e2e.sh
43cf72fb54cdc3dfe82a744e0be210cf99450ed3
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


⚠ 366.ru has degraded performance

In d39c4b7, 366.ru (https://366.ru) experienced degraded performance:

HTTP code: 200
Response time: 1399 ms

Resolved: 366.ru performance has improved in 14e7db1.


Bumps aioasuswrt

Description:
This adds libnacl to aioasuswrt
Related issue (if applicable): fixes #20361
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Checklist:

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

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

[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.

it is still error with asuswrt ssh in the 0.86.4...............

Please open an issue if you suspect a bug. If you need help please use our help channels:
https://home-assistant.io/help/#communication-channels
Merged PRs should not be used for support or bug reports. Thanks!


Fixing typos in Acrylic article

Fixes #1602

@mijacobs : The PR passed the acceptance checklist. Please evaluate further and approve/complete the PR if appropriate. Thanks!


Import internal version of six

Summary of changes
Fix for usage external six module, that was added at #1576 (v.40.6.0).
Closes  #1592
Pull Request Checklist

[ ] Changes have tests
[ ] News fragment added in changelog.d. See documentation for details

Can you add this test:
 setuptools/tests/test_virtualenv.py | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git i/setuptools/tests/test_virtualenv.py w/setuptools/tests/test_virtualenv.py
index e511c918..94768b27 100644
--- i/setuptools/tests/test_virtualenv.py
+++ w/setuptools/tests/test_virtualenv.py
@@ -1,3 +1,4 @@
+import distutils.command
 import glob
 import os
 import sys
@@ -134,3 +135,14 @@ def sdist(distname, version):
         'python setup.py test -s test',
     )).format(tmpdir=tmpdir))
     assert tmpdir.join('success').check()
+
+
+def test_no_missing_dependencies(bare_virtualenv):
+    """
+    Quick and dirty test to ensure all external dependencies are vendored.
+    """
+    for command in ('upload',):#sorted(distutils.command.__all__):
+        bare_virtualenv.run(' && '.join((
+            'cd {source}',
+            'python setup.py {command} -h',
+        )).format(command=command, source=SOURCE_DIR))

And a news entry.

Nevermind, I'll do it and release a bug fix version.

40.6.2 has been released.

Oops, my bad. For some reason I thought we did some magic to automatically resolve the vendored dependencies.
Thanks for taking care of it @benoit-pierre


How to configure windows to register a file associate?

I am build an app, to open .epub file.
but i have no idea how to configure electron forge to build installer to register .epub file associate with my app.
Build platform ( Windows 10 x86_64)
Any document show us  how to do ?
I can not find any useful document explain that.

Unfortunately, due to the limited amount of time of the maintainers, we cannot answer generic Electron questions in the issue tracker, which is reserved for bugs and feature requests specific to Electron Forge. Instead, consider using one of the community forums, including the Electron Discord server, as there are more people available to answer questions.


Allow for custom element anchors

What problem does this feature solve?
<router-link> will set a href attribute on its child only if this child has an <a> tag name (see findAnchor()). However, this won't work in case an anchor is wrapped in a custom element.
What does the proposed API look like?
Maybe something like this?
<router-link to="..." anchorTag="custom-anchor">
   <custom-anchor>Click Me!</custom-anchor>
</router-link>

See https://github.com/vuejs/rfcs/pull/34


feature: 新增常驻任务&APM预计算

 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.huginzhang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


document select

for #1343
note that we don't cover the case where symbol is constant, but we try to select it with select({symbol: "min"}); this should be red:

we don't cover the case where symbol is constant, but we try to select it

This is true in general; the types don’t validate that the channels referenced in the outputs actually exist. For example, you could declare title: "mean" as an output, but you wouldn’t get an error if you don’t declare the title channel as an input.


Swift: Support enums and unions as Clang types.

test with https://github.com/apple/swift/pull/22456
@swift-ci test


Add projet versionning

As suggested in our Slack community (join here), adding project versions would help a lot on collaboration.
We will need:

a version of the project created each time someone save a project
list version in UI
view a previous version in read only
rollback to a previous version (create a new version with the old one)
delete a version
name a version (with a description)

Technically we will:

add a project_versions table (id, project, name, description, created_by, created_at)
save project files for versions in /projects/$uuid/$iso-date.json
create a version whenever a project is saved
CRUD API for versions (list for a project, get, update, delete)

Thank you @loicknuchel! There's one part of this that we talked about in Slack that I don't see in your description, which is handling multiple collaborators editing the same project. When they save they should be warned if they made their changes to an old version, and in general they should notified when someone makes changes.
Without this feature collaborators will silently overwrite each other. This will lead to a frustrated user who finds their work has been overwritten, with their only recourse being to walk back through the version history to manually merge in their overwritten work.

Hi @imtibbet
Indeed, I did not put that here as most of it is already in place.
When you save, I fetch the current saved project and if it has changed, I try to apply your changes on top of it (like a git rebase).
If there is conflict I don't save and ask you to refresh the project (losing your changes), but at least not silent overrides.
What is still missing (and planned at some point) is a presence feature to see other users on the project at the same time and on which layout they are (helping to avoid conflicts).


Request to add NANO

Nano (NANO)
Formerly known as RaiBlocks (XRB)

link to logo: https://www.reddit.com/r/nanocurrency/comments/7wh5uz/nano_media_kit_logos_for_websites_apps_etc/ (Logos in Google Drive link)
official website: https://nano.org/en

Please, do it!


doublecmd 0.9 anyone?

Hi,
Could anyone update it here?
Or maybe point me to the to the docs where I can find out how to contribute?

sorry, wrong place, should be on extras


[WIP] Adopt new service catalog resource names

https://trello.com/c/y1lbFrbp
Instance -> ServiceInstance
Binding -> ServiceInstanceCredential
Requires:
https://github.com/openshift/origin-web-common/pull/182

mockData.service.ts references 'instances' to get mock service instances:
https://github.com/openshift/origin-web-catalog/blob/master/app/mockServices/mockData.service.ts#L233

Other instances:
https://github.com/openshift/origin-web-catalog/blob/master/src/components/create-from-builder/create-from-builder.controller.ts#L111
https://github.com/openshift/origin-web-catalog/blob/master/src/components/create-from-builder/create-from-builder.controller.ts#L296
https://github.com/openshift/origin-web-catalog/blob/master/src/components/order-service/order-service.controller.ts#L258

@jeff-phillips-18 Thanks for catching that. I think I had a typo in my search for instances

@jmontleon FYI, this PR is updated.
Building with @jpeeler's changes now to test

Cannot find any instances of:
resource: 'bindings'
case 'bindings'
kind: "bindings"
resource: "instances"
case 'instances'
kind: "Instance"

Searched in /app, /src and /test using both single and double quotes.

 Test failed.  See above for more details.
The command "hack/test-headless.sh test" exited with 1.

Yeah, it's the same problem in #428. Tests aren't running in Travis.

Ok.  More test fun :)


docs: fix yarn add command

I just fixed installation code as https://github.com/NozomuIkuta/h3-cors/pull/6.

Thanks!


Update what-are-lifecycle-workflows.md

Removing Azure AD from Lifecycle Workflows name for correct branding

Learn Build status updates of commit dd87b8f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/active-directory/governance/what-are-lifecycle-workflows.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

#sign-off


Startup issue in Enterprise Docker image 4.1.3-3

It seems the Docker image graylog/graylog-enterprise:4.1.3-3 (notably not graylog/graylog:4.1.3-3) has an issue:
graylog_1  | adding environment opts
graylog_1  | /docker-entrypoint.sh: line 109: /usr/local/openjdk-/bin/java: No such file or directory

Comparing the current OSS and Enterprise Dockerfiles, it seems the most relevant difference is the Enterprise version sets a default value for argument JAVA_VERSION_MAJOR (see image below).  Specifically, it seems to be due to #172 commit 7d8b6da2d9bbabe5bac9a294b4a2b8e20acb692b, which fixed this issue for OSS but not Enterprise.  I’ve tried passing in env vars JAVA_VERSION_MAJOR=8 and JAVA_HOME=/usr/local/openjdk-8 (individually and both; via docker-compose.yml with down+up), however the error above still occurs.
As a workaround I’ve reverted from graylog/graylog-enterprise:4.1.3-3 to graylog/graylog-enterprise:4.1.3-1, however that image has the unclean shutdown (journal corruption) issue previously reported (#174).
Ticket ID #554048406

That is embarrassing, and I am annoyed that I didn't catch it. I will fix it!

@coffee-squirrel Alright, I published 4.1.3-4. Can you pull it and see if it works okay for you now?

@juju2112  Thanks, that image fixes this issue.

@juju2112 Thanks, that image fixes this issue.

Thanks for pointing it out!!


use a minimum bounding box algorithm to recover the frame of a table

stops using getOrientedBoundingBox (vtkAnnotateOBBs). reuses code from continuous walking block segmentation. in particular gives more reliable standing frame. still needs some improvement but works better than previous approach

edgePoint is no longer required for segmentTableEdge, but haven't changed it in argument.
Before (can actually be much worse):

After:

by the way, I also re-used this code for a box picking demo I worked on.  I have not had a chance to clean up my code and commit it yet.  In my code, I removed the findFarRightCorner() function, I think it is overly complex.  For the DRC finals terrain task I developed some general purpose re-orientation code.  If I get around to it, when I clean up my code, I might incorporate it so that we can remove the findFarRightCorner dependency.

You're right - the look direction is enough and the head inversion shouldnt be a blocking issue. I'll clean this up further shortly.


Export all types used in public API

The problem
Some types referenced in the public API arenot exported.
This makes it difficult to write code that uses those types, as they cannot be referenced by name.
Specifically: MatchFilter, MatchFilters, PathMatch, Route and RouteMatch
The fix
Export those types

Thank you


Missing declared license

Type: Missing
Summary:
Missing declared license
Details:
Missing license
Resolution:
Apache 2.0: https://github.com/AzureAD/azure-activedirectory-library-for-dotnet/blob/2.19.208020213/LICENSE
Affected definitions:

Microsoft.IdentityModel.Clients.ActiveDirectory 2.19.208020213

@capfei - the curation guidelines have been updated to say that we would curate the license link on the package page, even if it points to the master branch vs. digging into the source.
See: https://github.com/clearlydefined/clearlydefined/blob/master/docs/curation-guidelines.md
"1. Target component metadata has license info indicating a license URL, but the link points to a volatile location.
Apply license found at the volatile location at the time checked."
So, we would curate MIT (over Apache in the source).
Agreed?

Updated PR to MIT


Token does not exist error

swagger-cli validate (v4.0.4) is reporting

Token "Resource" does not exist.

I'm not sure why.
Our OpenAPI 3.0.3 document is very long; I think the relevant sections are
...
  "components": {
    "schemas": {
      "Foo": {
        "title": "Foo",
        "description": "A foo.",
        "type": "object",
        "properties": {
          "entry": {
            "description": "An entry.",
            "type": "array",
            "items": {
              "type": "object",
              "properties": {
                "resource": {
                  "description": "The Resource for the entry.",
                  "$ref": "https://hl7.org/fhir/R4/fhir.schema.json#/definitions/Resource"
                },
...

Is an external URL $ref not supported, or is there something I'm doing wrong?
The error message doesn't include a line number, so it's difficult to be sure.
Thank you!

Yes, external URL $ref is supported.  In this case, your $ref also includes a JSON Pointer to #/definitions/Resource, and the error is saying that Resource doesn't exist in that file


Please add the option to capitalize SQL keywords

Is your feature request related to a problem? Please describe.
I saved the following statement in a query.sql file:
select * from wakanda where alpha = 8 and beta = 9 limit 10;
and then formatted it (btw, cmd+e cmd+b for OS X Mojave didn't do anything!) producing this result:
select
  *
from
  wakanda
where
  alpha = 8
  and beta = 9
limit
  10;

Describe the solution you'd like
I would very much have preferred the following result which I find more readable and compact:
SELECT *
FROM wakanda
WHERE alpha = 8
  AND beta = 9
LIMIT 10;

Describe alternatives you've considered
or at least capitalising the SQL keywords:
SELECT
  *
FROM
  wakanda
WHERE
  alpha = 8
  AND beta = 9
LIMIT
  10;

It would be really nice if you would add support for capitalisation, maybe as a setting.

Hi @odebroqueville .
Actually keywords already have the option to transform to upper case. Take a look here: https://github.com/mtxr/vscode-sqltools/blob/084759a15f5152a05fec4f12579fae60ab247578/.test-database/.vscode/settings.json#L13
The formatting you suggested is pretty cool. We already have a ticket for that in the formatter project. I'll link it here after work.
Closing since we already have a ticket for this.


Add eslint-plugin-redux-saga

Since it's now finally published, we should look into adding eslint-plugin-redux-saga! It lints for common mistakes when writing sagas, which is awesome!

Done by the amazing @jstockwin in #1060


Fixing a number of broken links/updating references

Hey @erquhart,
I've done some research around static site generators and found staticgen.com super useful! Thank you!
During the research I've found a bunch of outdated/broken links. If possible I've updated/replaced the links or if no source was to find/identify completely removed. Let me know if you got any questions!
Peter

Super helpful, thanks! You added a link to a blog post of yours, but it's broken (rather ironic lol) - it might be good to link somewhere that's very unlikely to change, eg. https://davidwalsh.name/introduction-static-site-generators

Hello @erquhart,
this is indeed hilarious. That's the blog article, I did my research for 😂 I want to share some information I collected myself and make the topic more accessible.
I've compared the articles and found the one by Eduardo Bouças on David Welsh' site is a bit dated (2015). It doesn't include nice things or doesn't highlight some more modern parts (solutions for workarounds with Netlify Forms & CMS for example). I hope to have this included well enough. I've also fixed the link 😂 I'm happy to update/include more, if you think it needs to 👍 Of course, I can also just swap it out (if you want).
Let me know what you think.
Cheers,
Peter

Fixed link works for me - thanks again for the updates!

No problem. Thanks for pointing it out!
On Thu, 9 May 2019 21:34 Shawn Erquhart<EMAIL_ADDRESS>wrote:

Merged #498 https://github.com/netlify/staticgen/pull/498 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/netlify/staticgen/pull/498#event-2330437513, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACAK7MYQTBJ2I6WZ67FJ5MLPUQY5ZANCNFSM4HHKDXZQ
.

I'll just point out here that @spekulatius is being pretty agressive with the link of his post including it in many other repositories https://github.com/Sirikon/pitch/pull/8#issuecomment-494561115
He's even quantifying the links he gets from different repositories, like here: https://github.com/seymoe/reactpress/pull/1/files#diff-04c6e90faac2675aa89e2176d2eec7d8R2 (Notice the utm_source and utm_medium).
I would suggest replacing it with https://davidwalsh.name/introduction-static-site-generators as pointed out initially by @erquhart , because Spekulatius' interests seem to be more personal than for the community's good, the Eduardo's post is better in terms of explanation and examples, and the part of highlighting modern solutions is already done by https://www.staticgen.com/.

Hello @Sirikon
valid critic and I accept it. The original intention was to fix the broken links to the same non-existing resource, but I got carried away. I've made a mistake by adding these links and I'll close the related PRs. Just on the side: I've got no advantages from the links - it's a personal blog, where I write about coding & tech 🙏
Peter


Suggested improvement for the documentation

First of all thank you so much for this new version of RSwitch it is a true gem for all R users on Mac 👍. I would like to make a suggestion to add a note in the documentation for users using the R (.pkg) installer. Currently the documentation says:

(You can also use the .pkg versions if you prefer a clicky-installer wizard.)

However, as mentioned on the CRAN documentation page:

The installer will remove any previous version of the R framework which it finds installed

So if one tries to successively install multiple R versions (e.g. 3.5.3 and 3.6.1 as shown below) from the .pkg, the previous version will no longer be accessible and be labelled incomplete:

The CRAN documentation page mentions that:

This can be avoided by using pkgutil --forget

So I ran pkgutil --pkgs | grep org.r-project in terminal to list out the R files, and then I used
sudo pkgutil --forget org.r-project.R.el-capitan.fw.pkg
sudo pkgutil --forget org.r-project.x86_64.tcltk.x11
sudo pkgutil --forget org.r-project.x86_64.texinfo
sudo pkgutil --forget org.r-project.R.el-capitan.GUI.pkg

After this installing a new version did not impact the pre-existing one.

This is an incredibly helpful suggestion! #y!!
Tis live now: https://rud.is/rswitch/guide/

@hrbrmstr thank you very much for the prompt implementation and the credit! Going one step further one could think about having a menu item in RSwitch running this "forget" script in the background. This may be helpful for folks who do not feel comfortable in terminal or with sudo.


treat stdgates uniformly with other DEFGATES (and more)

This commit does a good helping of refactoring, further clarifying
confusion over what a gate is, how a standard gate rears its head, and
how CONTROLLED and DAGGER gates get simplified.
Three big notable things in this PR:

Standard gates are read from a new file src/quil/stdgates.quil.

GATE-APPLICATION-GATE is not an accessor anymore, but a function
that computes and caches a gate object depending on the resolved
name.

Hooks to optimize the representation of a CONTROLLED gate and a
DAGGER gate exist with the generic functions CONTROL-GATE and
DAGGER-GATE, which are to be read as verbs.

This is intended to replace PR #217, and should NOT be merged only until @notmgsk's PR #150 is merged. (The reason to wait is because all of the standard gates are being boiled away to their Quil definitions, and thus will not be optimized any longer until that PR is merged.)
From Eric's PR:

[x] lookup-standard-gate should return a gate-definition of sorts, not a gate.
[x] resolve-applications should be modified so that we stash away that definition in the name-resolution field.
[x] We make generic functions control-gate and dagger-gate which are responsible for the kind of logic you wrote. This way the dispatch can be nicely specialized.
[x] To gate-application, we add the :writer called %set-gate-definition-gate.
[x] To gate-application, we add the :reader called %get-gate-definition-gate.
[x] To gate-application, we make the :initform to the gate form nil.
[x] We eliminate the :accessor gate-definition-gate, and instead make it a function that first tries to %get-gate-definition-gate, and if not, builds (but does not %set!) using the lifter.
[x] We modify operator-description-lifter to make use of control-gate and dagger-gate GFs. This should make that function much simpler, even simpler than before, since you're composing these GF's directly instead of making lambdas that are make-instanceing things all over.
[x] Change the documentation to operator-description-lifter to be clear that it could construct any manner of gate-like object, but will try to produce a reasonable, direct, and efficient representation.
[ ] Make sure tests pass. (They're not right now!)
[ ] Fix the QVM in any way that this breaks it.

Note that this PR will cause some performance regressions (in terms of time to compile) because it reconstructs gate objects much more frequently from the definitions, instead of having the definitions cached as they were before for standard gates.

This last commit fixed the performance regression suitably well. 🙌

This is a cool PR.

@ecpeterson done: https://github.com/rigetti/qvm/pull/84


feat: provide performance hints

at compile time we could analyze queries against a prod instance and provide useful insights to the user. like the need to create indexes.
more on the next comment

More information regarding the idea.
It would be to include an optimization check into the Compile Time check. My recommendation would be some modifications in the Type Generation to include additional metadata that can be discarded after the compile time check is done. I'd recommend leaving the scope to single column indexes while ironing it out, and then maybe target multi-column indexes. Single Column indexes are very common, so having the DB Library recommend them shouldn't be too bad.
Do note that  Postgres states there is no silver bullet for Indexes, in an ideal world we'd probably hook into a production DB, check if some variant of ANALYZE exists, and run it for all the queries. Instead this feature is to suggest good places for developers less familiar with DBA where to look.
A practical example should help showcase the idea. We'll have two tables to support a Customer/Order Relationship.
CREATE TABLE customer (
     id   serial   PRIMARY KEY,
     name text     NOT NULL,
     age  integer
);
CREATE TABLE order (
     id       serial      PRIMARY KEY,
     customer integer     NOT NULL,
     total    varchar(64) NOT NULL DEFAULT '0.00',
     currency varchar(3)  NOT NULL DEFAULT 'USD',
     FOREIGN KEY (customer)
         REFERENCES customer (id)
);

Now lets add in some queries to reference. (Apologizes for any bad syntax, I rarely just yolo SQL like this)
-- $1 would be a customer ID. Query in general is "Get total order count by customer Id"
SELECT COUNT(order.id) FROM order INNER JOIN customer on order.customer=customer.id WHERE customer.id = $1

-- $1 is a currency code, Query in general is "Get average customer age by order currency"
SELECT AVG(customer.id) FROM customer JOIN order on order.customer=customer.id WHERE order.currency = $1

Hopefully that is enough to work with, since I can't easily come up with more examples quickly.
In this case, ojotas would recommend an index on the order.customer column. We can see there is no index there, and both of the queries the application would run (in this small example) utilize a JOIN on that column. This points towards an area to review for "free" performance.
Naturally any hint system should have an opt-out feature, since if you do the review and find no reason to utilize the tip, you don't want to see it anymore. I'd recommend the following addition to the ojotasrc.json
{
  "hints": {
      "enabled": boolean,
      "ignore": [
          {
             "table": string,
             "column": string?,
             "hint_type": enum
          }
      ]
   }
}

Where if enabled is true, then ignore factors in. If hints is false, then no hint scanning is done.
When hints are enabled, than a declared table, or table/column pair can be excluded from specific hints (Assuming more hints get added.
As an example, lets display a configuration option to disable hint scanning on our order table, and then our order/customer Table Column Pair.
{
  "hints": {
      "enabled": true,
      "ignore": [
          {
             "table": "order",
             "column": null,
             "hint_type": null
          }
      ]
   }
}

{
  "hints": {
      "enabled": true,
      "ignore": [
          {
             "table": "order",
             "column": "customer",
             "hint_type": "INDEX"
          }
      ]
   }
}

column could be adjusted to columns if it is preferred to have a string[] instead.

Another thing we could do related to this is that if we are able to analyze al the sql queries, we could suggest the user to create composite indexes that could satisfy many queries instead of having many secondary indexes.

Also we could detect cases of index obfuscation

Maybe suggest to use union all for each branch

video on index obfuscation https://planetscale.com/learn/courses/mysql-for-developers/queries/redundant-and-approximate-conditions?autoplay=1 and how to mitigate it


Refactor command system

Lots of changes in this one!  I've refactored the command system in a way that I think makes it much more extensible and somewhat easier to use.  I also think it aligns better with the tModLoader style.
If everything looks good on this, I'm hoping to move forward and implement server commands and some utility functions such as the /help command.

Also, we were discussing earlier how multiple commands with the same name shouldn't thrown an error, but rather require the user to clarify which command from which mod to use with something like modname1:commandname modname2:commandname.
We could just pull this in and then change stuff though.

@JavidPack I agree! I didn't really know what should be the expected behavior for commands with the same name so I just had it throw an error for now.

We can add the conflict resolution. I'll merge this pending reversion to C# 4.0

@Chicken-Bones Just finished making the revisions you asked for!

Very nice changes! While you're at it, have you thought about automated argument parsing ?

Take a look at https://github.com/bluemagic123/tModLoader/commit/84d5d4b64653115271eec9b5cbaeeb62c0659c85


Schinagl.LinkShellextension version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

The installer requires VC++2015-2019 to be installed before running, needs to wait for dependency support

@wingetbot run

Waiting for external dependency support

Just undrafting things to run them through the validator, is dependency support out yet?

IIRC no, they hit some (a lot) speedbumps so they didn’t release any Dependency features yet, even in 1.3 its hidden behind the feature command. (Also pretty much no changes refering to Dependcies)

Ok, back to draft it goes


properly installing production EE server

using kobotoolbox/enketo-express unga-II branch
with all the stuffs: SSL cert, backups, firewall etc, etc.
The current server was very quickly put together. I don't remember what shortcuts I took. It might be good to install a new server from scratch.

Config:
"default theme" : "unga" (this will avoid style-sheet swapping and flashing)


Add support to Jakarta EE 9

Is your feature request related to a problem?
Already known issue - multiple APIs moved from package javax to jakarta in this version, meaning that our instrumentations will not be applied. This includes the sub-packages servlet, jms, ws, faces and ejb that we instrument at least.

Describe the solution you'd like
It is likely that the addition of the jakarta package to matchers will be sufficient in most or all cases, but it will require some investigation and a lot of testing.
Minimal requirements for resolving this issue

[ ] verify Servlet and FilterChain instrumentations work
[ ] verify JMS instrumentation works
[ ] verify JAX-RS instrumentation works
[ ] verify JSF instrumentation works
[ ] verify javax.ejb.Schedule instrumentation works
[ ] add Tomcat 10 to integration tests

Additional context
Our integration test on Tomcat 10 would fail due to this breaking change - see related comment.

Working on this might also be a good time to include https://github.com/elastic/apm-agent-java/issues/1294 as well to avoid duplication.


performance question for SQL query

OrientDB Version, operating system, or hardware.

v2.2.13-20161027.133116-9

Operating System

[x] Linux

I have a class "cdr_af_20161016" with about 70 STRING properties defined, plus a single DATETIME property named "PRT_DATE"; there is almost 21M documents in this class

a unique index "idx_cdr_af_20161016_0" is defined on [ENV,FILENAME,RECORD_NUMBER,PRT_DATE]

a LUCENE index "idx_cdr_af_20161016_2" with metadata {"allowLeadingWildcard": true} is defined on [CALLING_NUMBER,CALLED_NUMBER,CONNECTED_NUMBER]

When I run a counting query that only considers ENV and PRT_DATE, is goes very slow, using index "idx_cdr_af_20161016_0" :
SELECT count(*) AS count  FROM cdr_af_20161016 WHERE ENV='prod' AND PRT_DATE BETWEEN '2016-10-20 00:00:00.000' AND '2016-10-21 00:00:00.000'

-> 20941621 records, 1127 secs elapsed !!
Explain plan:
{
    "result": [
        {
            "@type": "d",
            "@version": 0,
            "documentReads": 20941621,
            "fullySortedByIndex": false,
            "documentAnalyzedCompatibleClass": 20941621,
            "recordReads": 20941621,
            "fetchingFromTargetElapsed": 1125809,
            "indexIsUsedInOrderBy": false,
            "compositeIndexUsed": 1,
            "current": "#840:3512803",
            "involvedIndexes": [
                "idx_cdr_af_20161016_0"
            ],
            "projectionElapsed": 270,
            "limit": -1,
            "evaluated": 20941621,
            "groupByElapsed": 0,
            "user": "#5:0",
            "elapsed": 1127001.1,
            "resultType": "collection",
            "resultSize": 1,
            "@fieldTypes": "documentReads=l,documentAnalyzedCompatibleClass=l,recordReads=l,fetchingFromTargetElapsed=l,compositeIndexUsed=l,current=x,involvedIndexes=e,projectionElapsed=l,evaluated=l,groupByElapsed=l,user=x,elapsed=f"
        }
    ],
    "notification": "Query executed in 1127.533 sec. Returned 1 record(s)"
}

Another query, targeting to benefit from Lucene index "idx_cdr_af_20161016_2", goes slow as well but less-slow than the previous query:
SELECT count(*) AS count  FROM cdr_af_20161016 WHERE (CALLING_NUMBER LUCENE '*123*') AND OUTPUTTYPE = '14'  AND ENV='prod' AND PRT_DATE BETWEEN '2016-10-20 00:00:00.000' AND '2016-10-21 00:00:00.000'

-> 305488  records, 262 secs elapsed !!
The explain plan of this latter query:
{
    "result": [
        {
            "@type": "d",
            "@version": 0,
            "documentReads": 305488,
            "fullySortedByIndex": false,
            "documentAnalyzedCompatibleClass": 305488,
            "recordReads": 305488,
            "fetchingFromTargetElapsed": 261022,
            "indexIsUsedInOrderBy": false,
            "score": 0.57735026,
            "current": "#838:3612316",
            "totalHits": 305488,
            "idx_cdr_af_20161016_2_totalHits": 305488,
            "_memoryIndex": "CALLING_NUMBER:\n\t'[33 38 38 39 31 35 33 38 65 65 65 65 65 65 65 65]':1: [(0)]\n\tterms=1, positions=1\ncdr_af_20161016.CALLING_NUMBER:\n\t'[33 38 38 39 31 35 33 38 65 65 65 65 65 65 65 65]':1: [(0)]\n\tterms=1, positions=1\n\nfields=2, terms=2, positions=2",
            "involvedIndexes": [
                "idx_cdr_af_20161016_2"
            ],
            "projectionElapsed": 5,
            "limit": -1,
            "evaluated": 305488,
            "groupByElapsed": 0,
            "user": "#5:0",
            "elapsed": 262209.66,
            "resultType": "collection",
            "resultSize": 1,
            "@fieldTypes": "documentReads=l,documentAnalyzedCompatibleClass=l,recordReads=l,fetchingFromTargetElapsed=l,score=f,current=x,involvedIndexes=e,projectionElapsed=l,evaluated=l,groupByElapsed=l,user=x,elapsed=f"
        }
    ],
    "notification": "Query executed in 262.711 sec. Returned 1 record(s)"
}

=> what am I doing wrong ? These kind of queries run in a traditional, relational database in a few seconds only ...
To stay positive : a query with Lucene index, avoid the leading wildcard, goes fast :
SELECT count(*) AS count  FROM cdr_af_20161016 WHERE (CALLING_NUMBER LUCENE '123*') AND OUTPUTTYPE = '14'  AND ENV='prod' AND PRT_DATE BETWEEN '2016-10-20 00:00:00.000' AND '2016-10-21 00:00:00.000'

-> 19843  records found, 3 secs elapsed

Hi @rdelangh
I think your query is using the index only for the first key (ENV). The index is returning 20941621 records that are scanned one by one, this is why the query is so slow.
I suggest you to define the index differently, eg.
[ENV,PRT_DATE,FILENAME,RECORD_NUMBER]
this way the engine will be able to use it for the complete range query on ENV,PRT_DATE.
As a general rule, partial match is allowed only on tree-based indexes on a prefix of the index key, eg. if you have an index on [A, B, C], you can use it for queries on [A], [A,B], [A,B,C], but not on [A, C].
The second query is faster because in that case the LUCENE index is used, and it returns a bit fewer records
I hope it helps
Thanks
Luigi

hi @luigidellaquila ,
I recreated that index as you suggested:
create index idx_cdr_af_20161016_0 on cdr_af_20161016(ENV,PRT_DATE,FILENAME,RECORD_NUMBER) UNIQUE

then re-ran my query:
SELECT count(*) AS count  FROM cdr_af_20161016 WHERE ENV='prod' AND PRT_DATE BETWEEN '2016-10-20 00:00:00.000' AND '2016-10-21 00:00:00.000'

-> 4502773  records counted, in 137 secs ...
Is this as good as it can be with OrientDB ? Over 2mins to count 4.5M records ...
Explain plan:
{
    "result": [
        {
            "@type": "d",
            "@version": 0,
            "documentReads": 4502773,
            "fullySortedByIndex": false,
            "documentAnalyzedCompatibleClass": 4502773,
            "recordReads": 4502773,
            "fetchingFromTargetElapsed": 136415,
            "indexIsUsedInOrderBy": false,
            "compositeIndexUsed": 1,
            "current": "#839:2400099",
            "involvedIndexes": [
                "idx_cdr_af_20161016_0"
            ],
            "projectionElapsed": 239,
            "limit": -1,
            "evaluated": 4502773,
            "groupByElapsed": 0,
            "user": "#5:0",
            "elapsed": 136663.89,
            "resultType": "collection",
            "resultSize": 1,
            "@fieldTypes": "documentReads=l,documentAnalyzedCompatibleClass=l,recordReads=l,fetchingFromTargetElapsed=l,compositeIndexUsed=l,current=x,involvedIndexes=e,projectionElapsed=l,evaluated=l,groupByElapsed=l,user=x,elapsed=f"
        }
    ],
    "notification": "Query executed in 137.232 sec. Returned 1 record(s)"
}

well, actually the count should be performed using the index only, not browsing the records.
Maybe there is a bug in the implementation of the BETWEEN operation?
Could you please try the following?
SELECT count(*) AS count  FROM cdr_af_20161016 WHERE ENV='prod' AND PRT_DATE >= '2016-10-20 00:00:00.000' AND PRT_DATE < '2016-10-21 00:00:00.000'

Thanks
Luigi

Or it's just a matter of type conversion together with indexes, it's worth trying to use the date() function do to explicit conversions:
http://orientdb.com/docs/2.2.x/SQL-Functions.html#date
Thanks
Luigi

@luigidellaquila
no change:
... WHERE ENV='prod' AND PRT_DATE>='2016-10-20 00:00:00.000' AND PRT_DATE<'2016-10-21 00:00:00.000'

returns the correct count after 122 secs (which is bad !)
... WHERE ENV='prod' AND PRT_DATE>=DATE('2016-10-20 00:00:00.000') AND PRT_DATE<DATE('2016-10-21 00:00:00.000')

returns the correct count after 106secs (which is bad !)
... WHERE ENV='prod' AND PRT_DATE BETWEEN DATE('2016-10-20 00:00:00.000') AND DATE('2016-10-21 00:00:00.000')

returns the correct count after 117 secs (which is bad !)

meanwhile, database cannot be opened anymore after running with 2.2.13* snapshots, see #6884
Waiting for that issue to be resolved first ...

@luigidellaquila
Database is usable again, issue #6884 is resolved.
-> can you please re-continue your investigations for these performance issues with dates in the WHERE clause?


Mandatory use of XSL transform is not being used

I read this documentation about PEM 111, https://www.nwsurreyccg.nhs.uk/stay-informed/documents/nhs-111-1/391-appendix-d-it-standards/file      see 11.7. it said a specific transformed XSL must be used against CDA. My understanding, this is not being currently being used looking from the payload returned, and if this transformed being used, most of the transformation may not be required, probably just need to get the patient, transformed html text, and attachment data (and mimetype) into FHIR response.

This looks like a requirement for 111 sending systems, NHS testing did not highlight this as a requirement. Closing with Status as WONT FIX.

Hi, My understanding working previously with MIM specs, the transform is not for the sending side, but for the recipient side.

@Kusnaditjung this section describes "CDA messages should be rendered using specific XML transforms" The requirement of  the 111 PEM adapter was to transform to JSON FHIR only. I think this could be describing a transformation at the HA side.

After the discussion with NHSD Architects (David Water and Adam Lee), the stylesheet is not a mandatory requirement for NHS 111. For UI rendering Parts, it will depend on the consumer system to render as it wishes, at the moment they don't seem to be aware of any requirements on the UI rendering parts. However, for KAINOS, there may need a check to see if they also translate Section 2 - Section 7. So, there may be more answer for KAINOS for this part.


Add spanish translation of 0.21.2

Attempts to exactly mimic #810 for spanish language,
meaning format issues in #810 directly apply here as well.
Untested at the time of submitting.

Last Ruby site I contributed a translation to,
had been a less bumpy contribution. 🤷♀

I force pushed, no content changes since 872544edfcc7,
I only reorganized the changes in 4 commits. 🤔

I forced push again, only to apply 451ad1a58893c43b4351152d1ed449ddeeabebf9 here.

Tested ACK 9b2ced7257d8dff08a57cce364606de819037a1a using Google Translate to verify text.  Thanks!
I did find the following reverse translation amusing (the actual translation looks good to me):

shoulda used "colaborador" :/
On Sat, Jan 8, 2022 at 4:35 PM David A. Harding @.***>
wrote:

Tested ACK 9b2ced7
https://github.com/bitcoin-core/bitcoincore.org/commit/9b2ced7257d8dff08a57cce364606de819037a1a
using Google Translate to verify text. Thanks!
I did find the following reverse translation amusing (the actual
translation looks good to me):
[image: latest]
https://user-images.githubusercontent.com/61096/148650182-5ff6e9a0-8451-4d92-b8dc-236fada91337.png
—
Reply to this email directly, view it on GitHub
https://github.com/bitcoin-core/bitcoincore.org/pull/814#issuecomment-1008028222,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRS4WYEKESJT2QXTT6FT5TUVBKVNANCNFSM5GFHORVQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


feat(simulators): added ability to capture Python APIs in simulator s…

…pecs

:tada: This PR is included in version 3.16.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Pygments lexer name correct?

https://github.com/pprzetacznik/IElixir/blob/master/lib/ielixir/socket/shell.ex#L54 sets the Pygments lexer name to its class name, but that seems to trigger an error in my version of Jupyter when triggering its use:
opt/anaconda3/lib/python3.6/site-packages/nbconvert/filters/highlight.py:131: UserWarning: No lexer found for language 'pygments.lexers.erlang.ElixirLexer'. Treating as plain text.
warn("No lexer found for language %r. Treating as plain text." % language)

When I manually edit the notebook to just specify "Elixir", the error doesn't appear. Interactive testing shows the same - Pygments' get_lexer_by_name seems to expect just "Elixir", not the full classname.
Maybe my version? I'm running Anaconda 4.3.0 on Linux with Python3, which includes Jupyter 4.2.1.

Hi @cdegroot, can you test it right now as I have updated packages and changed lexer in there? Thx in advance.