brew install @lts

A detailed description of the proposed feature
Ability to issue brew install node@lts that installs the latest stable release

The motivation for the feature
being able to consume the stable versions as opposed to specific versions

How the feature would be relevant to at least 90% of Homebrew users
because most users aspire to use lts versions of any software

What alternatives to the feature have been considered
manually identify the lts version and use it in the command line

reference: https://github.com/nodejs/help/issues/1599
Thanks in advance!

@Homebrew/core for thoughts on this for node specifically with an alias or formula (jenkins-lts is some prior art).

is it closed by mistake? just wondering about the issue life cycle - the above comment suggests that it is forwarded for further considerations, but where that will be tracked, that I can follow? thanks!

@gireeshpunathil In general we close those out we don't have specific plans to do.

ah! ok. I then mis-interpreted as forwarded for further consideration.
Any reason why this cannot be taken up? invalid requirement / not a common use case / anything else?

@gireeshpunathil A combination of not enough demand, existing solutions (being able to install by version number) perhaps being sufficient and there be no clear implementation in Homebrew for this solution.

Also it being pretty specific to NodeJS and a few other tools rather than most formulae.

Ability to issue brew install node@lts that installs the latest stable release

I’ve never seen LTS to mean “latest stable release”. To me it always meant Long Term Support, which is the latest stable only for a short while, then becoming an older and older version that still gets patches.

because most users aspire to use lts versions of any software

With that definition, I agree most users want the latest stable, but only a few (generally a company) want the LTS.

@vitorgalvao - I did not mean to expand lts into latest stable release. If the requested tag is confusing, please treat it as brew install xyz@lsr or something, you got the idea.
suggestion to install by specifying the version does not yield a good user experience, as it requires going through the software's website and reading deeper which, kind of defeats the premise of the tool's flexibility?

Then what’s the issue with brew install node? You don’t need to specify a version or any @. The default behaviour is to install the latest stable version, isn’t it?

default install installs current version (v12.x) , not the stable one (v10.x). ref: https://nodejs.org/en/

not the stable one (v10.x)

That’s the long term support version. They even state as much on the website.
You’re conflating things. “Latest Stable” is not the same as “Long Term Support”. What you want is a version for long term support. In that case, no, I don’t think this is “relevant to at least 90% of Homebrew users”.

I do not wish to pursue this any further, however the data points suggest usage of lts
https://nodejs.org/en/user-survey-report/#version


feat: Update Hugr lowering for lists

Closes #525

Implement None-swapping logic for linear list indexing similar to arrays
Lowering of __length__ is still todo until https://github.com/CQCL/hugr/issues/1543 is resolved

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

Project coverage is 91.09%. Comparing base (63a7502) to head (028ac62).

Files with missing lines
Patch %
Lines

guppylang/prelude/_internal/compiler/list.py
63.55%
39 Missing :warning:

guppylang/prelude/_internal/util.py
20.00%
4 Missing :warning:

Additional details and impacted files
@@                  Coverage Diff                   @@
##           chore/linst-is-ded     #534      +/-   ##
======================================================
- Coverage               91.54%   91.09%   -0.45%
======================================================
  Files                      59       59
  Lines                    6136     6264     +128
======================================================
+ Hits                     5617     5706      +89
- Misses                    519      558      +39

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


Added unit tests in pkg/es/config

Which problem is this PR solving?
partially fixes: #5068
Description of the changes

This commit adds tests for the pkg/es/config package.

How was this change tested?

make test

Checklist

[x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
[x] I have signed all commits
[x] I have added unit tests for the new functionality
[x] I have run lint and test steps successfully

for jaeger: make lint test
for jaeger-ui: yarn lint and yarn test

the DCO check is failing. You'd be able to fix it at the end by squashing commits, but it would become more difficult if you start merging main into PR. It's best to get into habit of  git commit -s.

the DCO check is failing. You'd be able to fix it at the end by squashing commits, but it would become more difficult if you start merging main into PR. It's best to get into habit of  git commit -s.

it passes now the DCO

I think you messed up the merge, you have irrelevant changes in the pr.

It's Now all fixed and the linter problem also

sorry I had to revert this, didn't notice that it has the UI changes. Please redo as a new PR, from clean branch. Also, try removing ./pkg/es/config/.nocover, as this is the whole point of the main ticket and it's the last remaining .nocover file.

whe I removed .nocover I got no goleak check in package  ./pkg/es/config/ is that ok
after searching I found that I should put this line of code func TestMain(m *testing.M) { 	goleak.VerifyTestMain(m) } and add it's pkg source but it also gave same error

please create a new PR - yes we should add a TestMain for goleak check.

please create a new PR - yes we should add a TestMain for goleak check.

the new pr: https://github.com/jaegertracing/jaeger/pull/5819


Fix calendarTextColor context key

The 'relevant context key' should be calendarTextColor, not buttonColor.

@af7 Thanks :+1:


name 'outfilename' is not defined

I got this when I tried to run the code.
name 'outfilename' is not defined
Something missing in the latest commit 7093fb1bf2ebae67e971aafbc30e8c8b35939bdc?
Great script btw! Thank you.

Fixed in the latest commit :-)


Added ReadOnlyDataTypeComponent constructors

These additional constructors were added to both simplify their usage and make it cleaner.

I'm closing this as I no longer see it as necessary. It it better to just use a helper for this.


Unclear statement - Creating a new circuit containing all of the prerendered components

Creating a new circuit containing all of the prerendered components that are part of the same page
How can a circuit contain prerendered components? As I understood, a circuit is a websocket connection, it is a mode of transportation.

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

ID: ce1440e7-febc-3fbc-09d7-7f47b740fd3b
Version Independent ID: 31071595-7c71-b6f7-3856-74f78c5c7cdf
Content: Handle errors in ASP.NET Core Blazor apps
Content Source: aspnetcore/blazor/handle-errors.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Full passage ...

Blazor components can be prerendered using Html.RenderComponentAsync so that their rendered HTML markup is returned as part of the user's initial HTTP request. This works by:

Creating a new circuit containing all of the prerendered components that are part of the same page.
Generating the initial HTML.
Treating the circuit as disconnected until the user's browser establishes a SignalR connection back to the same server to resume interactivity on the circuit.

I'm going to make a minor language change here to move away from "containing."


Flow Charting plugin Inspect Abnormal display

What happened: Flow Charting plugin Inspect Abnormal display
What you expected to happen: Grafana version 7.0.3 is displayed normally
As the picture shows:

Environment:

Grafana version: 7.3.0 or up
OS Grafana is installed on:Windows Server 2016
User OS & Browser:Windows Edge\Chrome
Grafana plugins:FlowChart plugin version:0.9.0

duplicate with
https://github.com/algenty/grafana-flowcharting/issues/209


Release v5.0.12

Changes since last release (https://github.com/lildude/linguist/compare/v5.0.12...v5.0.12)

Update MD5 digest for testing under Ruby 2.4 (#3643)
Support for Type Language (#3593)
Add recognition and classification of WebAssembly (#3650)
Improve running from cloned repo docs (#3669)
Fix spelling of Perl 6 (#3672)
Added syntax definition for Fantom language (#3660)
Update vendor.yml: skeleton.css (#3682)
Added Ring Language Support  (#3662)
Add new language for the Easybuild framework (#3466)
Add C sample (#3698)
Add misclassified C sample (#3652)
Register "cpanfile" as a Perl filename (#3704)
Add xspec as XML file extension (#3702)
Improve Pan language support (#3691)
Add NCSA to license whitelist (#3714)
fix Coq sample JsNumber.v (#3710)
improve .ms disambiguation (Unix Assembly / MAXScript) (#3707)

... and an update to the latest version of all grammars.

@lildude why this closed?

Because it's an experiment, along with the other identical and also closed PR 😁
I'm writing a script to make the whole release process a lot more automated and am doing it here rather than directly in the github/linguist repo to keep the noise down. Once I'm happy with it, I'll perform the actual release there.


Brew fails if flock is not implemented: -e:1:in `flock': Function not implemented (Errno::ENOSYS)

From @ningzhai on October 9, 2016 10:23
As problem issued in the title.
Following is how I installed the linuxbrew:
git clone https://github.com/Linuxbrew/brew.git linuxbrew
echo "#linuxbrew" >>~/.bash_profile
echo "export PATH=$PWD/linuxbrew/bin:\$PATH" >>~/.bash_profile
echo "export MANPATH=$PWD/linuxbrew/share/man:\$MANPATH" >>~/.bash_profile
echo "export INFOPATH=$PWD/linuxbrew/share/info:\$INFOPATH" >>~/.bash_profile
PATH=$(getconf PATH)
source ~/.bash_profile

When I want to install the first software through brew, that is "brew install hello", I got following error:
-e:1:in `flock': Function not implemented (Errno::ENOSYS)
        from -e:1
Error: Another active Homebrew process is already in progress.
Please wait for it to finish or terminate it to continue.
Error: Failed to install vendor Ruby.

Somebody can help me ?
Copied from original issue: Linuxbrew/legacy-linuxbrew#1110

@ningzhai Which operating system are you running? Please report the output of brew config

Hi,
I have also encountered this error. There is my configuration: $ brew config
HOMEBREW_VERSION: 1.0.7-8-gd5f4afad7
ORIGIN: https://github.com/Linuxbrew/brew.git
HEAD: d5f4afad76b31a61b9cf22e77da71be2a9ceb892
Last commit: 11 hours ago
Core tap ORIGIN: https://github.com/Linuxbrew/homebrew-core
Core tap HEAD: b7ca722c79156fb190df3daaf04ebf069a44ec18
Core tap last commit: 8 hours ago
HOMEBREW_PREFIX: /share/backup/zhujie/.linuxbrew
HOMEBREW_REPOSITORY: /share/backup/zhujie/.linuxbrew
HOMEBREW_CELLAR: /share/backup/zhujie/.linuxbrew/Cellar
HOMEBREW_BOTTLE_DOMAIN: https://linuxbrew.bintray.com
CPU: octa-core 64-bit dothan
Homebrew Ruby: 2.3.1 => /ifs1/ST_META/USER/wangzhifeng1/software/ruby/bin/ruby
Clang: N/A
Git: 2.10.0 => /home/zhujie/workdir/software/git/bin/git
Perl: /share/app/perl-5.22.0/bin/perl
Python: /share/app/python-2.7.10/bin/python
Ruby: /ifs1/ST_META/USER/wangzhifeng1/software/ruby/bin/ruby
Java: 1.8.0_102
Kernel: Linux 2.6.32-504.16.2.el6.x86_64 x86_64 GNU/Linux
OS: CentOS release 6.6 (Final)
OS glibc: 2.12
OS gcc: 4.4.7
Linuxbrew glibc: N/A
Linuxbrew gcc: N/A
Linuxbrew xorg: N/A
Please help me!
Thanks very much! :)

Do you know what type of file system you're on? It doesn't appear to support the flock operation.
-e:1:in `flock': Function not implemented (Errno::ENOSYS)

Please report the output of
mount | grep ifs1

@rwhogg Brew fails if flock is not implemented for that file system. It should fail with a better error message. brew doctor should check for this. It should fall back to an alternative locking system when flock is not available. (all wish list and notes to self)

$ mount | grep ifs1:
isilon1.sz.local:/ifs/data/ST_META on /ifs1/ST_META type nfs (rw,tcp,soft,intr,lookupcache=positive,vers=3,sloppy,addr=...)
It is nfs file system.

Your two options are

enable support for flock on your NFS server
hack up Linuxbrew to use an alternative lock other than flock and submit an PR

@LovelornPig @macmanes You can change the lock dir to a filesystem that supports flock like for example /tmp/$USER by hacking up Library/Homebrew/config.rb. There's no environment variable to configure this setting. 😕
See https://github.com/Linuxbrew/brew/blob/master/Library/Homebrew/config.rb#L27

@michaelmhoffman suggested that you make $(brew --prefix)/var/homebrew/locks a symlink to a file system that supports flock. 😎

Note that there are some packages that do not respect the specified locking dir and try to lock files in Cellar. An example of this is glibc-2.19 and Cellar/glibc/2.19/lib/locale.
Moving and linking this directory to a file system that supports locking and rerunning brew postinstall glibc solves this issue, but there may be other packages that do the same.

Thanks for the heads up, Michael. brew doctor should check that the FS of $(brew --prefix) and $(brew --cellar)supportsflock`. A PR would be welcome if you're interested.

The way my current computing cluster is set up is that there is only 1 GiB of storage available with flock support and more without. Cellar is too large to be moved to the first file system, so just checking for support wouldn't solve the issue for me.

Sorry, I don't follow, Michael. On which file system(s) is $(brew --prefix) and $(brew --cellar) installed, and do they support flock?

@sjackman sorry for the confusion, updated above

No worries. Thanks for the clarification. I understand now.

You can put $HOMEBREW_PREFIX and $HOMEBREW_CELLAR on different file systems, though it's not typical, so that $HOMEBREW_PREFIX/var/homebrew/locks is on your small 1 GiB file system that supports flock. I prefer your solution though, of making $HOMEBREW_PREFIX/var/homebrew/locks a symlink to a file system that supports flock.
brew doctor should check that $HOMEBREW_PREFIX/var/homebrew/locks supports flock.
We can't unfortunately ensure that the software installed by Linuxbrew doesn't use flock on files in that formula's prefix. That's entirely up to that software and out of our control.

I see that error in our centos6 docker image, running on my mac.
==> Pouring git-2.14.3.x86_64_linux.bottle.tar.gz
==> Caveats
Bash completion has been installed to:
  /home/linuxbrew/.linuxbrew/etc/bash_completion.d

Emacs Lisp files have been installed to:
  /home/linuxbrew/.linuxbrew/share/emacs/site-lisp/git
==> Summary
🍺  /home/linuxbrew/.linuxbrew/Cellar/git/2.14.3: 1,408 files, 43.2MB
which: no flock in (/home/linuxbrew/.linuxbrew/bin:/home/linuxbrew/.linuxbrew/sbin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin)
Initialized empty Git repository in /home/linuxbrew/.linuxbrew/Homebrew/.git/

[linuxbrew@93224d8bf376 ~]$ brew install systemd -svd
which: no flock in (/home/linuxbrew/.linuxbrew/bin:/home/linuxbrew/.linuxbrew/sbin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin)
/home/linuxbrew/.linuxbrew/Homebrew/Library/Homebrew/brew.rb (Formulary::FormulaLoader): loading /home/linuxbrew/.linuxbrew/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/systemd.rb
...

Is it still successful, despite the error message?
See https://github.com/Linuxbrew/brew/blob/master/Library/Homebrew/utils/lock.sh#L45

Yes it succeeds, and yes that error message needs to be suppressed. I opened a pull request upstream: https://github.com/Homebrew/brew/pull/3381

With brew 1.3.6 the first reported flock error should be fixed. With https://github.com/Homebrew/brew/pull/3381, the stderr message is suppressed, so this will be even better with brew 1.3.7 (which is not released for the moment).
All other flock errors related to specific formulae/packages should be reported upstream, because there is not much we can do about that.
@sjackman I would be in favour of closing this issue.

This issue has come up a few times (three or four times). A brew doctor check would be good to check that the $HOMEBREW_PREFIX/var/homebrew/locks directory implements flock, and if it does not, suggest symlinking this directory to a volume that does implement flock.


Unknown git revision in Email reports

Description:
The Git revision is shown as "Unknown" in the email reports.
Reason: The workspace is not set to TestPlan and so the default hard-coded value is taken as the outputPropetyFilePath.

Reopening since the issue exists yet.


feat(Condensed): light condensed objects

New foundations for analytic geometry based on light condensed sets are being developed by Clausen-Scholze (see this youtube playlist).
This PR defines light condensed objects.

[x] depends on: #8613
[x] depends on: #8643
[x] depends on: #8674
[x] depends on: #8676
[x] depends on: #8678
[ ] depends on: #9513

Sorry, messed up when trying to merge another branch into this


problems with font (introducing ListItem)

In my fork I pushed a first try at ListItem. It is a test.. and both worked and not worked...

As you can see, the font is "strange"... I think that Qt is stretching it a bit...
I did some more tests and noticed that I don't have Roboto font installed on my workstation. And that the library doesn't load them with ::addApplicationFont.
Also loading the font, I still have problems, the font is not used. If I ask for an installed font, it is used without problem.
Also trying to load another external font I get the default font....
Actually, wherever you have to use this code snippet:
   QFont font(db.font("Roboto", "Regular", 16));
   font.setPointSize(16); // it should not be necessary

means that font object has no size 16 and you have to force it....
I don't know what other test to do...
Do you have the font installed?

The font files should be bundled with the library as resources.
See components/resources.qrc:
<RCC>
    <qresource prefix="/fonts">
        <file>../fonts/Roboto/Roboto-Black.ttf</file>
        <file>../fonts/Roboto/Roboto-Bold.ttf</file>
        <file>../fonts/Roboto/Roboto-Medium.ttf</file>
        <file>../fonts/Roboto/Roboto-Regular.ttf</file>
        <file>../fonts/Roboto/Roboto-Light.ttf</file>
        <file>../fonts/Roboto/Roboto-Thin.ttf</file>
    </qresource>
...

These are then loaded by the style class (QtMaterialStyle). See qtmaterialstyle.cpp, line 23:
    QFontDatabase::addApplicationFont(":/material/fonts/Roboto/Roboto-Regular.ttf");
    QFontDatabase::addApplicationFont(":/material/fonts/Roboto/Roboto-Medium.ttf");
    QFontDatabase::addApplicationFont(":/material/fonts/Roboto/Roboto-Bold.ttf");

I don't have the font installed on my system, no.

I used the unix switch for grep on a windows workstation... no dir recursion... sorry about this.
This evening I will try on my home notebook, and will try this code snippet to see what happens with the fonts:
https://bugreports.qt.io/browse/QTBUG-61520
I still think that setPointSize call should not be necessary it the correct font is returned from the FontDatabase.

I confirm there is a problem with the font loading.

Which one is Roboto?
I think the problem is in the resource file... I'm not even sure you are using roboto font due to a mismatch in the path name...
QFontDatabase::addApplicationFont(":/material/fonts/Roboto/Roboto-Regular.ttf");
See the changes in the respource file:
https://github.com/fperillo/qt-material-widgets/commit/1bfb9990054121d8720df170d364282c958a14da
and where I add the fonts using the aliases

What if you put the fonts in the same directory as the icons? I mean, just copy the whole fonts directory into components/ and then change the path in the resource file from ../fonts/Roboto to fonts/Roboto.

The resource path is correct. It is generated by Qt Creator and it is not a one-to-one mapping to the file system path, because of how the resource system works in Qt. See http://doc.qt.io/qt-5/resources.html. (The IDE complains if I try to use an incorrect resource path.)

I'll try on a different machine also when I get the chance, to see if I can figure out what the problem is.

An alias should be used in the resource file, in this way:
../fonts/Roboto/Roboto-Medium.ttf
then load in this way:
QFontDatabase::addApplicationFont( ":/fonts/rmedium" );
This is an official way to load resources via aliases.
I then created font via:
QFont font("Roboto", 14, iFontWeight )
where f.e.
iFontWeight = QFont::Bold

Ok, I think I understand now. Does https://github.com/laserpants/qt-material-widgets/commit/2ee5d6dd5de94e8bade310330a6f8e651edbb60e work or?

It works now :-)
Thank you


fix: Use appName as package name in the apiResources.partOfPackage

Addressing the following issue given in the issue #38

"partOfPackage": "customer.capirebookstore:package:capire.bookshop:v1” → correct namespace prefix in package ref - use appName as packageName as a fallback.

@RamIndia @RoshniNaveenaS : Please let us know if we may merge this PR. Thanks!
cc: @zongqichen @ductaily @Fannon


[services.ddclient] IP discovery fails with usev4=ifv4

Describe the bug
The ddclient service will fail to discover an IPv4 address with the following error message:
Aug 20 14:27:10 nixrt ddclient[135978]: FAILED:   Unable to obtain default route information -- missing ip or ifconfig command
Aug 20 14:27:10 nixrt ddclient[135978]: WARNING:  mydomain.com: unable to determine IPv4 address with strategy usev4=ifv4

  services.ddclient = {
    enable = true;
    interval = "5min";

    # IP discovery
    usev4 = "ifv4";
    usev6 = ""; # disable IPv6 (AAAA) record
    extraConfig = ''
      ifv4=wan0
    '';

    # A record
    zone = "mydomain.com";
    domains = ["mydomain.com"];

    # Cloudflare API
    protocol = "cloudflare";
    username = "token";
    passwordFile = "${config.sops.secrets."cloudflare/api-token".path}";
  };

I believe the issue is that the systemd service path is not including pkgs.iproute2. My workaround is to add this:
  systemd.services.ddclient.path = [pkgs.iproute2];

We can see in the service derivation that this path is optional, but the condition is too restrictive.
https://github.com/NixOS/nixpkgs/blob/d2f3b99a6b7b74a6fecaac280d88ecb4b5b28c5a/nixos/modules/services/networking/ddclient.nix#L230
Steps To Reproduce

Add a configuration like the one above to your NixOS configuration, replacing wan0 with an existing WAN interface.
Deploy the configuration and check journalctl -u ddclient to see the failure.

Expected behavior
The service should succeed to discover and IPv4 address, like it does after I apply my workaround.
Notify maintainers
@bjornfor
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.46, NixOS, 24.11 (Vicuna), 24.11.20240818.8a33541`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.5`
 - channels(root): `"nixos-24.05"`
 - nixpkgs: `/nix/store/d9gbq853jvbccrz5g3y0irbwgc57w137-source`

Add a :+1: reaction to issues you find important.

@bonsairobo: Since you already found a fix, can you submit a PR?


Issue with SnapshotID parameter in 2-cluster.json

{
"AWSTemplateFormatVersion" : "2010-09-09",
"Description"              : "This CloudFormation stack creates a Microsoft HPC Pack 2012 R2 Cluster in an existing environment. WARNING This template creates two or more Amazon EC2 instances. You will be billed for the AWS resources used if you create a stack from this template.",
"Parameters"               : {
"KeyName" : {
"Description": "Name of an existing EC2 Key Pair",
"Type"       : "String",
"Default"    : ""
},
"SnapshotID" : {
"Description"          : "Snapshot Id of the HPC Installation data",
"Type"                 : "String",
"AllowedPattern"       : "snap-([0-9a-fA-F]{8})",
"ConstraintDescription": "must be a valid snapshot ID."
},
Under SnapshotID, the "AllowedPattern" parameter failed for me. Granted, I had to copy my snapshot between regions, and I noticed the replicated snapshotID was larger than the original. So, I changed line 13 of the json file to this:
"AllowedPattern"       : "snap-([0-9a-zA-Z]*)"
So the final version looks like this:
{
"AWSTemplateFormatVersion" : "2010-09-09",
"Description"              : "This CloudFormation stack creates a Microsoft HPC Pack 2012 R2 Cluster in an existing environment. WARNING This template creates two or more Amazon EC2 instances. You will be billed for the AWS resources used if you create a stack from this template.",
"Parameters"               : {
"KeyName" : {
"Description": "Name of an existing EC2 Key Pair",
"Type"       : "String",
"Default"    : ""
},
"SnapshotID" : {
"Description"          : "Snapshot Id of the HPC Installation data",
"Type"                 : "String",
"AllowedPattern"       : "snap-([0-9a-zA-Z]*)",
"ConstraintDescription": "must be a valid snapshot ID."
},

Hi @plbenpeterson, we've updated the solution to one that uses HPC Pack 2019 with different mechanisms for standing up the resources. The reported issue is no longer relevant. Thank you for reporting it, and please let us know if there are any issues with the updated template.


Add CMap classic (in iframe)

Many changes under maps/ to add CMap-classic within an iframe, and renaming the other CMap-js instances as cmapjs to distinguish from classic

@nathanweeks @jd-campbell @maxglycine - Because of the large number of files that changed in this branch (114), I would like to get this merged into main soon, to avoid merge conflicts in other branches. Can one of you review today and either request changes or approve and merge?

The CMap-classic links LGTM. To make CMap-js more similar to classic (where the backbones are in the middle & QTL on the either side), I'll suggest updating the position of the QTL tracks in the gm_gwas maps, changing "position": -1 to "position": 1 so the QTL tracks appear on the right-hand side of the backbone (and so the feature correspondence lines don't travel across the entire gm_gwas map area)

legacy links updated to use the "legacy" subdomain

Thanks, @jd-campbell and @nathanweeks. Those suggested changes have been made (thanks for changing the links to legacy.soybase.org, Nathan).


Do not install requirements as part of the action

Remove workaround requirements install from action, since this is now done here

Need to remove some of the test for this functionality.


edb export via SiWave execute GUI-less

Description of the feature
Exporting from EDB via SiWave would benefit from a flag that allows the operation to be performed without the GUI opening
Steps for implementing the feature
No response
Useful links and references
No response

Siwave doesn't have a proper way to run script in non graphical. BTW I'm adding a bool argument to export_maxwell named hidden which runs Siwave in embedding mode. It means you will see siwave icon but not the UI


Check if kube-go-client informer wouldn't be a better solution for cleanupJob

Is your feature request related to a problem? Please describe.
No direct problem at the moment but working with larger clusters the default list function of the kube client seems to put heavy load to the api-server and etcd. So it would be better to use the informer pattern.
Describe the solution you'd like
I want that for each control-loop an informer or maybe sharedInformer should be used to behave more "nicely" to the api-server and etcd. Possible areas in the code where the optimization should be noticeable are in
...
func readinessHandler
...
func cleanupJob
...

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

From my point of view not necessary anymore, because of the implementation of #294


Finalize defaulted type parameters

This RFC proposes finalizing the design of defaulted type parameters with two changes:

Using _ to explicitly use a default
Integrating defaults and inference

Rendered view.

After some fairly extensive discussion on this RFC, the core team is convinced that this feature will pose no serious problems for type inference around a future HKT extension. See this comment for details. Other than those concerns, this feature is a frequently-requested one, and a natural extension given integer fallback.
I have merged the RFC; the tracking issue is here.


No picture of result

What does user notes look like? Should have a picture of it

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

ID: 09d6ad8a-eae1-0e91-2730-342bf57cd6b2
Version Independent ID: 8404676c-831f-e067-da03-7b9d32aff48d
Content: User Notes (Hosted Control) in Unified Service Desk for Dynamics 365 for Customer Engagement apps
Content Source: ce/unified-service-desk/user-notes-hosted-control.md
Service: dynamics-365-customerservice
GitHub Login: @kabala123
Microsoft Alias: kabala

@NotEnoughCholula : Thanks for reaching out. You can see the image in the following topic.
https://docs.microsoft.com/en-us/dynamics365/customer-engagement/unified-service-desk/panels-panel-types-panel-layouts?view=dynamics-usd-4.1#panel-layouts-in-unified-service-desk
Hope this helps. Closing the issue for now.


Certification: Eigenvalue computation

Replaced eigenvalue computation with Scipy eigs.

overall looks good, but please check test error, also please re-run certify.py to make sure that it works on some data


Service logging

PR Template:
Describe your changes

Moves service logging to the application and data tiers and (where possible) away from biz
Uses error wrapping to build something like a stack trace with contextual data
Uses the Kratos request logging middleware to handle writing error and trace logs

Note: it currently returns detailed error messages to the client, which is not desirable. And considers validation errors to be errors, which may not be desirable.

Note: this logs request objects, however, Kratos defines a Redacter interface (that's basically a renamed Stringer) that's only called when logging, and we could implement that for request bodies that contain sensitive material.

Sample log output when a request completes normally:
INFO ts=2024-06-18T21:05:45-04:00 caller=logging/logging.go:40 service.id=wscalf-thinkpadp16vgen1.rmtussc.csb service.name= service.version=d47de56 trace.id= span.id= kind=server component=grpc operation=/kessel.relations.v0.KesselCheckService/Check args=resource:{type:{name:"group"} id:"omni"} relation:"member" subject:{subject:{type:{name:"user"} id:"foo"}} code=0 reason= stack= latency=0.111345474

Sample log output on a runtime error (connection refused by SpiceDB):
ERROR ts=2024-06-18T21:06:12-04:00 caller=logging/logging.go:40 service.id=wscalf-thinkpadp16vgen1.rmtussc.csb service.name= service.version=d47de56 trace.id= span.id= kind=server component=grpc operation=/kessel.relations.v0.KesselCheckService/Check args=resource:{type:{name:"group"} id:"omni"} relation:"member" subject:{subject:{type:{name:"user"} id:"foo"}} code=503 reason= stack=failed to perform check: error invoking CheckPermission in SpiceDB: rpc error: code = Unavailable desc = connection error: desc = "transport: Error while dialing: dial tcp <IP_ADDRESS>:50051: connect: connection refused" latency=0.000420355

Sample log output on a runtime error in a gRPC request that returns a stream of responses (connection refused by SpiceDB):
ERROR ts=2024-06-26T11:22:24-04:00 caller=server/logging.go:34 service.id=wscalf-thinkpadp16vgen1.rmtussc.csb service.name= service.version=845939c trace.id= span.id= kind=server component=server operation=/kessel.relations.v0.KesselLookupService/LookupSubjects args=resource:{type:{name:"group"} id:"omni"} relation:"member" subject_type:{name:"group"} subject_relation:"member" code=503 reason= stack=error retrieving subjects: error invoking LookupSubjects in SpiceDB: rpc error: code = Unavailable desc = connection error: desc = "transport: Error while dialing: dial tcp <IP_ADDRESS>:50051: connect: connection refused" latency=0.0001817

Though note that this does not (yet) obscure the error that gets sent back to the client, which looks like:
ERROR:
  Code: Unavailable
  Message: failed to perform check: error invoking CheckPermission in SpiceDB: rpc error: code = Unavailable desc = connection error: desc = "transport: Error while dialing: dial tcp <IP_ADDRESS>:50051: connect: connection refused"

Ticket reference (if applicable)
Fixes RH33199
Checklist

[ ] Are the agreed upon acceptance criteria fulfilled?

[ ] Was the 4-eye-principle applied? (async PR review, pairing, ensembling)

[ ] Do your changes have passing automated tests and sufficient observability?

[ ] Are the work steps you introduced repeatable by others, either through automation or documentation?

[ ] If automation is possible but not done due to other constraints, a ticket to the tech debt sprint is added
[ ] An SOP (Standard Operating Procedure) was created

[ ] The Changes were automatically built, tested, and  - if needed, behind a feature flag - deployed to our production environment. (Please check this when the new deployment is done and you could verify it.)

[ ] Are the agreed upon coding/architectural practices applied?

[ ] Are security needs fullfilled? (e.g. no internal URL)

[ ] Is the corresponding Ticket in the right state? (should be on "review" now, put to done when this change made it to production)

[ ] For changes to the public API / code dependencies: Was the whole team (or a sufficient amount of ppl) able to review?

/lgtm

@wscalf Any thoughts about contributing something to kratos re: streaming logging support?

Some. They responded to my ticket that it wasn't really implemented yet because there wasn't a good abstraction in place. Working through this, it does look like an abstraction that makes streaming first class and treats unary endpoints as a special case with 1 incoming and 1 outgoing message could be viable, but I haven't proposed it or anything.


Dependency problem in 2.0.5

I get the following error message in a project using pyethereum as a dependency. With a freeze to version 2.0.4 it works fine.
Could not find a version that satisfies the requirement viper (from ethereum->-r requirements.txt (line 7)) (from versions: ) No matching distribution found for viper (from ethereum->-r requirements.txt (line 7))

I feel your pain, I want to use the tester module, but you can't do that from pypi directly. If you install viper first (https://github.com/ethereum/viper), then run pip install ethereum==2.0.5 it will work.
#807 fixes this problem, however someone needs to re-upload ethereum 2.0.5 with that correction applied to pypi so the rest of us can use it no problem.

@vbuterin Could you help to re-upload ethereum 2.0.5 to pypi? Thanks!

@berndbohmeier You can close this, v2.1.0 was uploaded to pypi today!


Video Description from Kaltura  - displayed in a modal - contains inline markup

Detailed description

the modal rendering of a video-player contains a description, which contains inline markup - see https://ibmdotcom-web-components.mybluemix.net/?path=/story/components-content-block-cards--with-videos

Is this a feature request (new component, new icon), a bug, or a general
issue?

According to Jeff Chew comes the descriptions from the Kaltura API, which already contains HTML markup.

Is this issue related to a specific component?

components-content-block-cards--with-videos -- but in general a videoplayer within a modal

What did you expect to happen? What happened instead? What would you like to
see changed?

that the markup is stripped from the webcomponent

What browser are you working in?

Chrome

What version of Carbon for IBM.com are you using?

What offering/product do you work on? Any pressing ship or release dates we
should be aware of?

yes, PROD-Release of AEM MVE with 06.04
Steps to reproduce the issue

open https://prod-publish.roks.cms.cis.ibm.net/topic/supplychain#anchor-48438963 or just clicking on the tile for "Farmer Connect + IBM" in -
clicking on the tile for "Farmer Connect + IBM"
check the modal

Additional information

Screenshots or code
Notes

Jeff is meeting with Media Center team to remove the HTML from the source rather than by DDS.  We can make the update at a later date.

This is already fixed on our end.  Kaltura will also remove from the source.  Can close


Feature P5js integration

Hi, I see that there is blank space in between the Navbar section and the home div, I would like to add some cool p5 js models like:

.
Let me know if i should work on this, and you would like this in the portfolio or not.

Sure @Zaid-B24


Expand test coverage

And make sure tests are still passing in prep for a new release (0.6.1)

Immediate test failures:
py39-test: exit 2 (0.00 seconds) .tmp/py39-test> '!cov-!noopenfiles:' pytest --open-files --pyargs spectral_cube /home/runner/work/spectral-cube/spectral-cube/docs
.pkg: _exit> python /opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/pyproject_api/_backend.py True setuptools.build_meta
.pkg: exit None (0.00 seconds) /home/runner/work/spectral-cube/spectral-cube> python /opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/pyproject_api/_backend.py True setuptools.build_meta pid=1745
  py39-test: FAIL code 2 (25.98=setup[25.58]+cmd[0.39,0.00] seconds)
  evaluation failed :( (26.08 seconds)
Error: Process completed with exit code 2.

No idea what this means

Actually the !cov-!bla syntax now fails for astropy as well (astropy/astropy#14139), e.g.
https://github.com/astropy/astropy/actions/runs/3733576067/jobs/6334552564
so this seems to have been broken with a more recent tox update since 4.0.8...

https://github.com/radio-astro-tools/spectral-cube/actions/runs/4374807279/jobs/7654700456#step:5:1516 we have real failures in some tests because of unimplemented floor_divide?  I hope this just means we need to bump version numbers

Codecov Report
Patch and project coverage have no change.

Comparison is base (d004d79) 77.36% compared to head (edfd92d) 77.36%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #854   +/-   ##
=======================================
  Coverage   77.36%   77.36%
=======================================
  Files          24       24
  Lines        6027     6027
=======================================
  Hits         4663     4663
  Misses       1364     1364

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

There's at least one genuine bug here introduced by astropy upstream.  This test:
https://github.com/radio-astro-tools/spectral-cube/blob/master/spectral_cube/tests/test_projection.py#L137
fails on dev but not on older versions.  The sum p2 = p + p does not retain the beam from p

I don't see any sign of this breaking change in astropy's changelog.  The only way I can think to track it down is with bisection, which I'm really bad at and can't do on my personal computer.

Maybe this is our breaking change? https://github.com/astropy/astropy/commit/c6ff27ddcc8e0c899c5a5999ffc4c9d2be9bf4b8

No, that seems like it's not the problem; the default is to pass on info except for a few cases that don't apply to these tests.

I had to do some test refactoring to get tests to run locally at all.

right, so, it looks like noviz works, but 'all' doesn't.

OK, dev passes on its own, so it's not dev.  It's some interaction between the viz stuff and the dev stuff, but I'm not convinced the viz stuff works at all.  Split up the tests further.

Dev and viz independently cause failures.  The dev failures originate somewhere in quantity, but there's no difference in astropy quantity between 5.2.1 and 5.3.dev, so I'm stuck.

I can't test dev locally because:
      Traceback (most recent call last):
        File "/private/var/folders/k_/7qh4l0nn72b7qgq15pkd4hw40000gt/T/pip-build-env-1rq0a8qx/overlay/lib/python3.9/site-packages/setuptools/_distutils/unixccompiler.py", line 185, in _compile
          self.spawn(compiler_so + cc_args + [src, '-o', obj] + extra_postargs)
        File "/private/var/folders/k_/7qh4l0nn72b7qgq15pkd4hw40000gt/T/pip-build-env-1rq0a8qx/overlay/lib/python3.9/site-packages/setuptools/_distutils/ccompiler.py", line 1041, in spawn
          spawn(cmd, dry_run=self.dry_run, **kwargs)
        File "/private/var/folders/k_/7qh4l0nn72b7qgq15pkd4hw40000gt/T/pip-build-env-1rq0a8qx/overlay/lib/python3.9/site-packages/setuptools/_distutils/spawn.py", line 70, in spawn
          raise DistutilsExecError(
      distutils.errors.DistutilsExecError: command '/usr/bin/clang' failed with exit code 1
  ERROR: Failed building wheel for numcodecs
ERROR: Could not build wheels for numcodecs, which is required to install pyproject.toml-based projects

I'm at a loss with this; I've googled and found similar issues, but no fixes - the issues just end with 'oh ok we solved it'

the viz tests are amusing:
https://github.com/radio-astro-tools/spectral-cube/actions/runs/4384619998/jobs/7676312443#step:5:195
../../spectral_cube/tests/test_subcubes.py ..ssssssssssssssssssss        [ 98%]
Fatal Python error: Aborted

I think this is an indication that the tests crash completely as soon as test_visualization.py, the next test code alphabetically, is initiated.  I'm guessing the failure happens on the matplotlib import step.

https://github.com/radio-astro-tools/spectral-cube/actions/runs/4384619998/jobs/7676312293#step:5:1534  is pointing to an incorrect error being raised:
https://github.com/radio-astro-tools/spectral-cube/blob/d004d7967e4730e8d362712381014fcd9d8f49ec/spectral_cube/tests/test_spectral_cube.py#L366

I think we have major breaking changes in reproject, likely caused by https://github.com/astropy/reproject/pull/334/files.  They don't make sense to me yet.

  This is
            specifically added as an option to enable using the subtraction and
            addition operators without checking unit compatibility here because
            they _already_ enforce unit compatibility.

        Examples
        --------
        >>> newcube = cube.apply_everywhere(np.add, 0.5*u.Jy)
        """

        try:
            if check_units:
                test_result = function(np.ones([1,1,1])*self.unit, *args)
                new_unit = test_result.unit
            else:
                test_result = function(np.ones([1,1,1]), *args)
                new_unit = self.unit
            # First, check that function returns same # of dims?
            assert test_result.ndim == 3,"Output is not 3-dimensional"
        except Exception as ex:
            raise AssertionError("Function could not be applied to a simple "
                                 "cube.  The error was: {0}".format(ex))

        # We don't need to convert to a quantity here because the shape check
        data_in = self._get_filled_data(fill=self._fill_value)
>       data = function(data_in, *args)
E       TypeError: operand type(s) all returned NotImplemented from __array_ufunc__(<ufunc 'floor_divide'>, '__call__', dask.array<FilledArrayHandler 017ac2fc-5bf3-4d25-bf15, shape=(2, 3, 4), dtype=>f8, chunksize=(2, 3, 4), chunktype=numpy.ndarray>, <Quantity 0.5 K>): 'Array', 'Quantity'

All of the relevant tests have been stuck yellow for hours?  Did I overuse our minutes or something?

no... I tihnk I introduced an infinite loop 🤦

Got dev to go green, finally.    Everything is not fine.

OK so something changed in astropy-dev in the last 4 days that broke this.  I'll bisect this time.

Broken by https://github.com/astropy/astropy/pull/14508.  Can't stay ahead of astropy!

In the last commit, I was trying to replicate this line:
https://github.com/larrybradley/jdaviz/blob/2ad11c120a7317c01c0ee6957c19c494a54e7428/tox.ini#L62
what did I get wrong?


Fixes issue with properties format value

Coverage remained the same at 26.913% when pulling cfbe8c33907085733a031a338401fa1d1a05929a on theseyi:properties-format-value into 9dead1831e3928700d0d0eaf57fa833710835855 on linkedin:master.

Codecov Report

Merging #953 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #953      +/-   ##
==========================================
+ Coverage   23.23%   23.24%   +<.01%
==========================================
  Files         153      153
  Lines        4424     4423       -1
  Branches     1125     1126       +1
==========================================
  Hits         1028     1028
+ Misses       3396     3395       -1

Impacted Files
Coverage Δ

.../components/browser/containers/browser-viewport.js
12.5% <ø> (ø)
:arrow_up:

...pp/components/datasets/containers/datasets-list.js
20% <ø> (ø)
:arrow_up:

.../app/components/browser/containers/browser-rail.js
11.62% <ø> (ø)
:arrow_up:

.../app/components/browser/containers/browser-view.js
50% <ø> (ø)
:arrow_up:

wherehows-web/app/utils/api/datasets/properties.ts
26% <0%> (+0.5%)
:arrow_up:

Continue to review full report at Codecov.

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


replace kramdown with GFM

This swaps out the default kramdown renderer in Jekyll for GitHub pages' GFM renderer
There are pros and cons of this, the main benefits include that if it renders on GH when looking at a PR or commit, it will render the same on the site once it's deployed (something I noticed in the 2022-01 minutes where the code blocks in <details> rendered in GH, and to my knowledge would render as expected in jekyll, and then didn't)
Most downsides I'm aware of are covered by the code changes in this PR:

markdown tables MUST have headers
inline-html cannot be indented (and there's no {::nomarkdown} to get around it)
the anchor ID generation is slightly different leading to some links having to be updated

Currently breaking the {:toc} macro on the FAQ page. This is bad and needs to be solved


saving as HTML, or PDF not working

Describe the bug
saving as HTML, or PDF not working via atom plugin on Mac OS
To Reproduce
Steps to reproduce the behavior:

Go to elearn.js compatible markdown file
Click on Packages -> elearn.js -> "Save As HTML"
confirm default path for saving
nothing happens, no error message but as well no file is written, or overwritten

Expected behavior
HTML or PDF export file should be written on the selected path
Screenshots
not applicable
Desktop (please complete the following information):

OS: Mac OS Catalina
Editor Atom 1.57.0
Plugin Version 0.6.5

Additional context
I debugged the behavior to Util.js line 33
`
let filePath = await new Promise((res) => {
dialog.showSaveDialog({
defaultPath: defaultPath,
filters: filters,
}, res);
});
    return filePath;

`
The promise does not return a filePath, after the dialog is displayed and the path confirmed. The same behavior does not occur with VSCode and the vscode elearn.js plugin on the same system. I don't know about Electron but it seems it has something to do with how the save dialog is called.

Hello codeHorse87,
thanks for the bug report. I could verify the behaviour on MacOS Big Sur. On the long run we want to get rid of the Electron based workflow because of too many dependencies. The future will be an editor HTML-Page with export funktion. Currently we are reworking the UI and constructing he general architecture.
But for the intermediate period we like to implement your bugfix.
Michael

Hi @mheinecke
Thanks for the quick reply. Does it have something to do with LiaScript which is already using elearn.js?
To ease up the setup for our authors we're thinking about hosting a CodiMD with elearn.js embedded.
Let me know about the progress. We might be able to support the development, testing, or documentation.

@mheinecke I've created a pull request with a fix to the issue. I appreciate if ¥ou can apply the fix

Hi @SpaceCafe can I kindly ask you to run an apm publish for the atom package repository?
Thanks in advance!


Feature Request - Optional "Map Links by Zone" window

Could an option be created to optionally open the "Map Links by Zone" window when opening mparty, instead of it always opening? I find I never use this window, so it is just extra clutter on the screen.

Sure I can add that as an option.

This function is available under the option "Hide 'Map Links by Zone'" in /mpartyconfig


docs/preauth: privkey device install

To reduce the confusion about privkey device installation further, add a note for golden image editing, which can be used for automated builds that are basing their configuration on golden images.

sdimg's can be modified with the mender-artifact cp/install functionality ;)


Support for jsoniter-scala codecs

Is your feature request related to a problem? Please describe.
jsoniter-scala is the fastest json parser available in Scala. It's becoming quite popular, and several libraries already provide support for it (e.g., tapir, sttp, caliban (WIP)).
It would be good if Iron provided support for it same way that it does for circe / zioJson.
The implementation for jsoniter-scala is slightly less straightforward than circe / zioJson since it makes heavy use of macros for derivation and doesn't provide implicits for standard types. This could easily dissuade a user from using Iron if they can't figure the interop out easily / quickly.
Describe the solution you'd like
Very similar to the circe / zioJson approach, provide a given / method with which the user can create codecs for jsoniter-scala.
If you think that this is something that you would like to add to the library, I can make a PR for it (I already have a working example - just let me know and I'll open a PR)

Looks good to me and given the nature of the module (given instances like Circe/ZIO Json), it should not really increase the maintenance cost. 👍
I will have to be careful to keep the number of "official" modules reasonable. (i.e Scalacheck support)

@kyri-petrou do you want create the jsoniter module?

Don't worry for the time. I just wasn't sure if someone was already working on this issue or not 😛


Sqlite3 DB error after internet outage

With my google drive mounted I experienced an internet outage.  The folder was hung when I tried to access it, so I tried a fusermount -uz and a remount, which returned

Sqlite3 error: BUSY

I then rebooted the machine and now it just hangs when mounting.  I added the -debug command to get this output:
.gdfuse/default/gdfuse.log
.gdfuse/default/gdfuse.log

[0.000498] TID=0: Setting up default filesystem...
[0.000545] TID=0: BEGIN: Saving configuration in /home/jchang/.gdfuse/default/config
[0.000710] TID=0: END: Saving configuration in /home/jchang/.gdfuse/default/config
[0.000743] TID=0: Loading application state from /home/jchang/.gdfuse/default/state...done
Current version: 0.6.21
Setting up cache db...done
Setting up CURL...done
Refresh token already present.
[0.042962] TID=0: Starting filesystem /media/remote/gdrive/
[0.046214] TID=1: init_filesystem
[0.046276] TID=1: BEGIN: Getting root folder id (root folder=) from server
[0.046365] TID=1: END: Getting root folder id (id=root) from server
[0.046751] TID=3: getattr /.Trash
[0.046808] TID=3: BEGIN: Loading metadata from db
[0.047544] TID=3: BEGIN: Recalculating cache size (old value=33071276906)
[0.053535] TID=4: opendir / O_RDONLY,O_NONBLOCK
[19.941340] TID=3: END: Recalculating cache size (new value=33250315770)
[19.941408] TID=3: END: Getting metadata: Not valid
[19.941434] TID=3: BEGIN: Refreshing metadata
[23.578204] TID=3: END: Refreshing metadata
[23.578213] TID=3: BEGIN: Getting changes from server
[23.707675] TID=3: BEGIN: Updating timestamps
[24.495789] TID=3: Retrying (1/8) final_step after exception: Failure("Sqlite3 error: CORRUPT")
[25.999375] TID=3: Retrying (2/8) final_step after exception: Failure("Sqlite3 error: CORRUPT")
[29.117373] TID=3: Retrying (3/8) final_step after exception: Failure("Sqlite3 error: CORRUPT")
[33.819232] TID=3: Retrying (4/8) final_step after exception: Failure("Sqlite3 error: CORRUPT")
[42.290514] TID=3: Retrying (5/8) final_step after exception: Failure("Sqlite3 error: CORRUPT")
[42.915288] TID=5: getattr /
[58.821787] TID=3: Retrying (6/8) final_step after exception: Failure("Sqlite3 error: CORRUPT")
[92.013407] TID=3: Retrying (7/8) final_step after exception: Failure("Sqlite3 error: CORRUPT")
[156.884756] TID=3: Retrying (8/8) final_step after exception: Failure("Sqlite3 error: CORRUPT")
[168.183822] TID=6: getattr /
[285.715096] TID=3: Error during final_step after 8 attempts: Failure("Sqlite3 error: CORRUPT")
[285.715141] TID=3: Retrying (1/8) close_db after exception: Failure("close_db")
[286.960236] TID=3: Retrying (2/8) close_db after exception: Failure("close_db")
[289.767915] TID=3: Retrying (3/8) close_db after exception: Failure("close_db")
[294.709673] TID=3: Retrying (4/8) close_db after exception: Failure("close_db")
[303.689263] TID=3: Retrying (5/8) close_db after exception: Failure("close_db")
[320.395492] TID=3: Retrying (6/8) close_db after exception: Failure("close_db")
[350.777010] TID=7: getattr /
[352.964621] TID=3: Retrying (7/8) close_db after exception: Failure("close_db")
[417.615118] TID=3: Retrying (8/8) close_db after exception: Failure("close_db")
[546.205843] TID=3: Error during close_db after 8 attempts: Failure("close_db")
[546.205931] TID=3: Error in finally block:
Exception:Failure("close_db")
Backtrace:
[546.217588] TID=3: Error during request: Sqlite3 error: CORRUPT
[546.217697] TID=3: Giving up
[546.217746] TID=3: Exception:Failure("Sqlite3 error: CORRUPT")
Backtrace:
[546.217766] TID=4: BEGIN: Getting metadata from context
[546.217893] TID=4: END: Getting metadata: Not valid
[546.217995] TID=4: BEGIN: Refreshing metadata
[546.218089] TID=8: getattr /.xdg-volume-info
[546.720593] TID=4: END: Refreshing metadata
[546.720603] TID=4: BEGIN: Getting changes from server
[546.944319] TID=4: BEGIN: Updating timestamps
[551.948806] TID=4: Retrying (1/8) final_step after exception: Failure("Sqlite3 error: BUSY")
[558.600688] TID=4: Retrying (2/8) final_step after exception: Failure("Sqlite3 error: BUSY")
[565.765876] TID=4: Retrying (3/8) final_step after exception: Failure("Sqlite3 error: BUSY")
[574.874173] TID=4: Retrying (4/8) final_step after exception: Failure("Sqlite3 error: BUSY")
[588.712717] TID=4: Retrying (5/8) final_step after exception: Failure("Sqlite3 error: BUSY")
[609.862007] TID=4: Retrying (6/8) final_step after exception: Failure("Sqlite3 error: BUSY")
[647.296635] TID=4: Retrying (7/8) final_step after exception: Failure("Sqlite3 error: BUSY")

dmesg

[  967.643873] INFO: task pool:4300 blocked for more than 120 seconds.
[  967.643879]       Tainted: P         C O    4.10.0-33-generic #37~16.04.1-Ubuntu
[  967.643880] "echo 0 > /proc/sys/kernel/hung_task_timeout_secs" disables this message.
[  967.643882] pool            D    0  4300      1 0x00000000
[  967.643886] Call Trace:
[  967.643893]  __schedule+0x232/0x700
[  967.643896]  schedule+0x36/0x80
[  967.643898]  schedule_preempt_disabled+0xe/0x10
[  967.643900]  __mutex_lock_slowpath+0x193/0x290
[  967.643903]  mutex_lock+0x2f/0x40
[  967.643906]  fuse_lock_inode+0x2a/0x30
[  967.643909]  fuse_lookup+0x31/0x140
[  967.643913]  ? lookup_fast+0xe3/0x2f0
[  967.643915]  ? legitimize_path.isra.27+0x2e/0x60
[  967.643917]  lookup_slow+0xa5/0x160
[  967.643919]  walk_component+0x1bf/0x350
[  967.643921]  ? path_init+0x1d9/0x330
[  967.643923]  path_lookupat+0x52/0x110
[  967.643925]  filename_lookup+0xb1/0x180
[  967.643929]  ? __check_object_size+0x108/0x1e3
[  967.643933]  ? strncpy_from_user+0x4d/0x170
[  967.643935]  ? getname_flags+0x72/0x1f0
[  967.643938]  user_path_at_empty+0x36/0x40
[  967.643940]  vfs_fstatat+0x66/0xc0
[  967.643943]  SYSC_newlstat+0x31/0x60
[  967.643946]  ? SyS_futex+0x81/0x180
[  967.643949]  SyS_newlstat+0xe/0x10
[  967.643951]  entry_SYSCALL_64_fastpath+0x1e/0xad
[  967.643953] RIP: 0033:0x7f996e667c55
[  967.643954] RSP: 002b:00007f9969a69b08 EFLAGS: 00000246 ORIG_RAX:<PHONE_NUMBER>000006
[  967.643956] RAX: ffffffffffffffda RBX: 00007f9954000020 RCX: 00007f996e667c55
[  967.643957] RDX: 00007f9969a69b70 RSI: 00007f9969a69b70 RDI:<PHONE_NUMBER>db2a20
[  967.643959] RBP:<PHONE_NUMBER>000411 R08:<PHONE_NUMBER>100006 R09:<PHONE_NUMBER>d46df0
[  967.643960] R10:<PHONE_NUMBER>0001cc R11:<PHONE_NUMBER>000246 R12: 00007f9954000078
[  967.643960] R13: 00007f9954000078 R14:<PHONE_NUMBER>002709 R15: 00007f9954000078

Is this recoverable? I had some files that were uploading during this that I no longer have easy access to.   The .gdfuse/default/cache folder is 32 GB.
If it isn't, can I reset the state of the folder and database without re-setting up the entire folder and settings?

I had some files that were uploading during this that I no longer have easy access to. The .gdfuse/default/cache folder is 32 GB.

In the cache folder you can find all the files that were uploading. The name of each cached file is the Google Drive id of the file.

this recoverable?

No sorry, it looks like your sqllite db is corrupted (Sqlite3 error: CORRUPT).

If it isn't, can I reset the state of the folder and database without re-setting up the entire folder and settings?

After recovering your files, you can remove everything under .gdfuse/default/cache and you should be able to remount your drive.


feature: gotoLocationAndClose

Me like Grug. Me like to use Grug to find needle in haystack. When I find needle, I close Grug.
I would be nice to  on an entry, jump to it, and close Grug. Like the opposite of openLocation.
gotoLocation = { n = '<enter>' },
gotoLocationAndClose = { n = '<C-enter>' },

haha, that gave me a chuckle 😄
You can already achieve that like so:
vim.api.nvim_create_autocmd('FileType', {
  group = vim.api.nvim_create_augroup('grug-far-keybindings', { clear = true }),
  pattern = { 'grug-far' },
  callback = function()
    vim.api.nvim_buf_set_keymap(0, 'n', '<C-enter>', '<localleader>o<localleader>c', {})
  end,
})

I try as much as much as possible to just provide the building blocks in terms of actions as there are many combinations they can be used in.

added it to Cookbook in README

Love it! This works; thank you 🙏


Map CPU core ID to package ID

Is your feature request related to a problem? Please describe.
This is to follow https://github.com/sustainable-computing-io/kepler-model-server/issues/78
@sunya-ch

This is good since we can extend the current code to calculate the energy per socket instead on naively use the aggregation of all sockets.


Revamp wizards AI

Acutal wizards are getting too similar each other, and interaction on them is a bit messy. We should rethink this a bit...

@saleiva you mean the thumbnails section or the thumbnails itself

We talk about this on thursday :)


Customize CORS header handling at gateway

Currently, the gateway would eat all CORS header from upstream (gears, microservices, etc.), and replace them as specified in tenant config. In some situations, this may not be desirable. For example, we'd like to pass through CORS header from asset gears. We'll need some mechanism to customize CORS header handling at gateway.

Asset Gear: Upstream headers overwrite standalone mode headers
Gateway: Gateway respect upstream CORS headers.


Bugs in C++ tests or do I mistake somewhere?

Initial solution
using namespace std;
class Funij
{

  public:
  static unsigned long long sumin(unsigned long long n);
  static unsigned long long sumax(unsigned long long n);
  static unsigned long long sumsum(unsigned long long n);
};

@FlingWhale's solution
using namespace std;
class Funij
{

  public:
  static unsigned long long sumin(unsigned long long n)
  {
      unsigned long long sum = 0;
      for (int i = 1; i <= n; ++i)
      {
          int multiplier = (n - i) * 2 + 1;
          sum += i * multiplier;
      }

      return sum;
  }
  static unsigned long long sumax(unsigned long long n)
  {
      //I think I broke something
  }
  static unsigned long long sumsum(unsigned long long n)
  {
      return 1;
  }
};

Example Tests Cases
Describe(FunijTest)
{
    It(test1)
    {
        unsigned long long sol1 =<PHONE_NUMBER>0;
        unsigned long long ans1 = Funij::sumin(4000);
        Assert::That(ans1, Equals(sol1));
    }
    It(test2)
    {
        unsigned long long sol1 = 32484635;
        unsigned long long ans1 = Funij::sumax(365);
        Assert::That(ans1, Equals(sol1));
    }
    It(test3)
    {
        unsigned long long sol1 = 389549900;
        unsigned long long ans1 = Funij::sumsum(730);
        Assert::That(ans1, Equals(sol1));
    }
};

The first function sumin is correct, the second one sumax returns nothing but with the first correct one
the 2nd test pass or is not tested and the 3rd is not tested.!
Time: 15ms Passed: 1
Test Results:
  FunijTest
  test1
✔  Test Passed
  test2
You have passed all tests! :)

Same thing with "SUBMIT" tests.
It seems that something is wrong somewhere: the way the tests are managed or my test writing. Nevertheless if  sumax returns something false:
static unsigned long long sumax(unsigned long long n)
  {
      return 1;
  }

the tests don't pass.

I think the solution may have crashed so it became equivalent to the succeed&exit hack. Why it could crash is a different question. Although theoretically not returning in C++ results in undefined behavior, which is not necessarily just returning a random value but rather pretty much anything one can imagine, I wouldn't expect it to crash in a case like this.

The same kind of thing in JS:
String.prototype.toString = function() {
  return this.toString();
}

describe("t", () => {
  it("t1", () => {
    Test.assertEquals(true, true);
    Test.assertEquals('1'.toString(), '1');
  });
  it("t2", () => {
    Test.assertEquals(true, false);
  });
});

Closing because C++17 was added with improved feedback.


Trigger command failed in preview window sometimes

VSCode Version: 1.22.2
OS Version: MacOS 10.13.4

Steps to Reproduce:
I write a extension Super Replace which takes preview window as user interface. In the daily use, I observe that one out of 10 or 20 times, the command cannot trigger, and the console logs prevented webview navigation.
It's really hard to debug this, since it comes out randomly.
After a long time debugging, I suspect it a problem about adding the event listener handleInnerClick .
Here is the code (from webview-pre.js):
loadTimeout = setTimeout(function () {
    clearTimeout(loadTimeout);
    loadTimeout = undefined;
    onLoad(newFrame.contentDocument, newFrame.contentWindow);
}, 200);

newFrame.contentWindow.addEventListener('load', function (e) {
    if (loadTimeout) {
        clearTimeout(loadTimeout);
        loadTimeout = undefined;
        onLoad(e.target, this);
    }
});

onLoad is the function who adds the listener.
var onLoad = function (contentDocument, contentWindow) {
if (contentDocument.body) {
    // Workaround for https://github.com/Microsoft/vscode/issues/12865
    // check new scrollTop and reset if neccessary
    setInitialScrollPosition(contentDocument.body);

    // Bubble out link clicks
    contentDocument.body.addEventListener('click', handleInnerClick);
}
// ...
}

In most times, it works as expect. But if the webview was loaded in more than 0.2 second, we got trouble:

the setTimeout triggers, but the onLoad function cannot do it's job, because contentDocument.body is not ready.
the load event triggers, it cannot do the job either, because the loadTimeout has been set to undefined in last trigger.

So we can see, the handleInnerClick is not added in this case. This explains why the prevented webview navigation happens most in cold starts.

I get the same problem for my extension.
After open preview html, it will fail to execute command in the html with error '[Embedded Page] prevented webview navigation'. This happens most  when the first time to open the preview html.


Add template

You want to modify the script so that in the config.py file, users can add their own template IDs, and the bot will automatically use that template for the script. This way, it becomes easier for anyone to add their own templates manually. Did I understand your request correctly?

Yes


Document the include directory of the SDK

When you download a Dart SDK, the uncompressed dart-sdk directory contains an include directory. This directly isn't mentioned in the README file, or on dart.dev. We should document that this is, and what it can be used for.

Sort of a soft yes that it is required?  It's probable that SDK documentation is not generated directly except as a test that dartdoc works, or as part of the build system, which we do all the time.   It could be moved into lib easily in the bundled SDK.  It's a bit more work to move it somewhere else.

There is a use case I forgot to mention.
If it was removed entirely, developers changing documentation in the SDK would have to either follow special instructions to build dartdoc for the SDK to check for errors rather than simply dartdoc --sdk-docs as they do now, or risk that dartdoc would not catch the same set of errors that the build system does in documentation comments.  I'd really rather not cause that.  Moving to lib is quite possible though.

Fixed in https://dart-review.googlesource.com/c/sdk/+/215840


Virtual Libraries

[ ] Save advanced search as virtual library (#125)
[ ] Rename virtual libraries
[ ] Delete virtual libraries
[ ] Set virtual library as default
[ ] Use term 'All' (or something similar) as replacement for current 'Default' library. As allowing a library to be default, and having another named 'Default' would be confusing.

There are multiple ways in which libraries can be created; advanced search, SimpleLibraryViews plugin, etc. not sure library creation really belongs here. Fo rmy use case, a simple shell script and SimpleLibraryViews suffices. So, for now at least, I'm closing.

Having thought about this a little more, I might implement this as follows:

[ ] Fork SimpleLibraryViews
[ ] Add JSONRPC to SimpleLibraryViews
[ ] Allow creation of 'simple library views' libraries - by selecting albums and 'add to'
[ ] Allow deletion of 'simple library views' libraries
[ ] Allow renaming of 'simple library views' libraries

Re-closing. My desire for Virtual Libraries was to exclude tracks from DSTM/LastMix - but (AFAIK) this ignores Virtual Libraries, so not on my list of priorities. Plus, the user-experience would be lacking.


Product Drives -- expand export to be like Distribution etc

Summary
Although there is a product drive export, to be really useful, it needs  to have the items in it, like we do with the distribution and the donation exports
Why?
Increased utility.  Saving banks time.
Details

Export of product drive to be expanded to include the number of each item that were received from each product drive.
Have a column for each active item whether or not any of that item were received in the drive.
Item columns are in a stable order -- this will be used as input to other spreadsheets of the organization, so that is important
Similar to #3388 and #3430

Criteria for completion

[ ] Functionality as described above is available
[ ] tests to support this functionality

Hey @cielf, I can take this up, can you please assign me this issue?

@anosh96 You got it!

Yes,  but also inactive ones,  because they may be pulling a report across a significant period of time.
From the  way your phrased your question -- I want  to point out that the unfiltered export is across multiple product drives.

thanks for clarification @cielf, can you please review the PR linked to this issue?

Hi @anoosh96 I'm really busy early this week, but will take a look before the weekend if no one else beats me to it.

hi @cielf, just a warm reminder about the PR review.

great, thanks @cielf


Tag list feature for easier retrieval

Summarized Feedback from<EMAIL_ADDRESS>Users request a "tag" list feature for easier retrieval of notes, because they want to access all notes associated with a specific tag, similar to the functionality in Google Keep.

#869


Unable to change done button title  ?

When I select multiple images from galary then its Done button looks likes hanged/stucked.
Is there any way to change done button to loading indicator on pressing done button? like whatsapp doing.

The only solution is you have to confirm TLPhotosPickerLogDelegate and implement select/deselect method.
This method will be called before selectedAssets's count changes.


QuickstartDeployCLI with solutionType=FhirServerSqlServer fails

I was trying to use Sql Server in QuickstartDeployCLI.md. Encountered a few issues:

Examples reference the master branch- project has migrated to the main branch (not a blocker, just a heads up, bunch of issues like this in the repo).
Attempting to run the command with the addition of solutionType=FhirServerSqlServer results in:az deployment group create --resource-group fhir --template-uri https://raw.githubusercontent.com/Microsoft/fhir-server/main/samples/templates/default-azuredeploy.json --parameters serviceName=$servicename solutionType=FhirServerSqlServer
{"error":{"code":"InvalidTemplate","message":"Deployment template validation failed: 'The provided value '' for the template parameter 'sqlSchemaAutomaticUpdatesEnabled' at line '163' and column '29' is not valid. The parameter value is not part of the allowed value(s): 'auto,tool'.'.","additionalInfo":[{"type":"TemplateViolation","info":{"lineNumber":163,"linePosition":29,"path":"parameters.sqlSchemaAutomaticUpdatesEnabled.allowedValues"}}]}}

Adding sqlSchemaAutomaticUpdatesEnabled=auto results in:az deployment group create --resource-group fhir --template-uri https://raw.githubusercontent.com/Microsoft/fhir-server/main/samples/templates/default-azuredeploy.json --parameters serviceName=$servicename solutionType=FhirServerSqlServer sqlSchemaAutomaticUpdatesEnabled=auto
{"error":{"code":"InvalidTemplate","message":"Deployment template validation failed: 'The template resource 'Microsoft.Sql/servers/fhir/databases/FHIR' cannot reference itself. Please see https://aka.ms/arm-template-expressions/#reference for usage details.'.","additionalInfo":[{"type":"TemplateViolation","info":{"lineNumber":0,"linePosition":0,"path":""}}]}}

Is there a way to deploy the project using SQL server?
cc @jdanyow

Thanks for raising this, the documentation page is a little stale. The deployment worked for me when also providing the "sqlAdminPassword" parameter which is required for the SQL Server template.
On top of this I'd recommend using the "default-azuredeploy-docker.json" template.


[Bug - very minor] “File|Export|Model To Open Exchange File…” defaults to users home directory

“File|Export|Model To Open Exchange File…” defaults to users home directory (I presume an environment variable is being used?) – other options default to the models' current directory location which seems more sensible. (Note: On selecting “Choose” it defaults to the current directory that Archi was launched from.)
Version of Archi, Operating System
4.0.3, Windows 10
Expected Behaviour
Selecting “File|Export|Model To Open Exchange File…” defaults to current working directory
Actual Behaviour
“File|Export|Model To Open Exchange File…” defaults to users home directory
... but offer the current working directory on selecting "Choose".
Steps to Reproduce the Behaviour

Select “File|Export|Model To Open Exchange File…”

This is very minor, it just not what I would have expected.
Thanks
James.

This default happens only the very first time the Export dialog is opened. Thereafter, it remembers the last directory the user chose after the export is done.
It's a safe default for the very first time, and many other apps do the same. You'll never see it again once you've selected a different folder and clicked OK.
Won't change, so closing.


Settings not saving

Microsoft PowerToys version
0.59.0
Running as admin

[X] Yes

Area(s) with issue?
General, Awake
Steps to reproduce
I turned on "Always run as administrator" & "Run at startup" and turned off "Awake". Closed the window and the reopen it. Both are set to "Off" and Awake is now turned back on. PowerToysReport_2022-06-14-16-37-48.zip
✔ Expected Behavior
"Always run as administrator" to be turned on.
"Run at startup" to be turned on.

"Run at startup" still turned off.
"Awake" still turned on.
Other Software
No response

Having a similar problem here. Settings will save when running without elevation, but not when running as administrator. Seems to affect every setting.
In addition, "Check for updates" button doesn't work when running as administrator (just spins on "checking for updates..."), but works when running without elevation.

Is the administrative account different than the logged in user that's using PowerToys? /needinfo

Settings are saved in %localappdata%\Microsoft\PowerToys Access to that folder for PowerToys is needed in order to save the settings. Are the files read-only or hidden by any chance? /needinfo

If it's another admin user, the changes might be being saved to that specific user files.

If it's another admin user, the changes might be being saved to that specific user files.

So, what is the solution for users that are required to use a separate user profile for admin functions?


fastfetch causing midnight commander process to hang

Describe the bug
If you use midnight commander with the .bashrc provided by the script, you might notice that mc hangs for a few seconds before starting. The issue seems to be with mc and using fastfetch in .bashrc.
To Reproduce
Steps to reproduce the behavior:

type mc in console
wait for the process to start

Additional context
if you use mc with the -u or the --subshell flag, the process will start instantly.
Workaround
Add alias mc = "mc -u" to your .bashrc file.
Checklist

[ x] I checked for duplicate issues.
[ x] I checked already existing discussions.
[ x] This issue is not included in the roadmap.
[ x] This issue is present on both stable and development branches.

It has nothing to do with fastfetch. When using subshell it loads the .bashrc so it takes time for it to open. It has been an issue for a long period of time. The bash prompt setup is done via https://github.com/ChrisTitusTech/mybash you can open an issue there.

/label 'wontfix'
/close not planned


create talk.legalese.com to replace the lame readme.io discussion board and<EMAIL_ADDRESS>

Create a discourse.org website and make it work for talk.legalese.com / www.l4.chat

done.  http://talk.legalese.com exists
Next issue -- make a custom install of Discourse.org for www.l4.chat


powerai cudatoolkit 10.1 doesn't allow profiling

This is an issue affecting CUDA 10.1 defaulting to requiring administrator access to use the nvprof tool.[1,2]. Is it possible package a version of cudatool kit added to powerAI that will allow profiling?
Many thanks.
[1] https://developer.nvidia.com/nvidia-development-tools-solutions-err-nvgpuctrperm-NsightCompute
[2] https://devtalk.nvidia.com/default/topic/1049421/visual-profiler-and-nvprof/nvprof-warning-the-user-does-not-have-permission-to-profile-on-the-target-device-/

As you probably know from the links you posted, this was an intentional change by NVIDIA to address a security issue that allowed non-admin users to see data processed by the GPU by other non-admin users. https://nvidia.custhelp.com/app/answers/detail/a_id/4738
This NVIDIA document contains instructions on how to re-enable profiling for non-admin users, but in doing so you do expose the system to the security vulnerability that was fixed: https://developer.nvidia.com/nvidia-development-tools-solutions-ERR_NVGPUCTRPERM-permission-issue-performance-counters

Hi thanks, for looking into this. I don't really see any way to re-enable profiling for a non-admin as both sudo and CAP_SYS_ADMIN require admin privledges but I guess I need to take this up with my sysadmin. Thanks again.


A url that matches the priority problem

Thanks, that was pretty last minute change, previous it was the following, so never was an issue:
<Route path='/male' handler={CharacterList}>
       <Route path=':race' handler={CharacterList}>
         <Route path=':bloodline' handler={CharacterList} />
       </Route>
     </Route>
<Route path='/female' handler={CharacterList}>
       <Route path=':race' handler={CharacterList}>
         <Route path=':bloodline' handler={CharacterList} />
       </Route>
     </Route>

Multiple sections of the tutorial have been updated.


No such file or directory found while executing the application

Hey @evgenyigumnov,
I followed the same instruction that is present in the readme file and the execution part is present in the image below but it is giving me that no such file or directory found error.

It's now added

Cool. MarkDown parser do not work with images by http or https path
Could you fix it?


GlobalAveragePooling2D vs Flatten() on pretrained model

In keras pretrained model (keras applications page). tutorial uses GlobalAveragePooling2D() before feeding to customized top layers. What GlobalAveragePooling2D()  does? and why not using Flatten(), since these are going to be fed to FC layers? what would be the best practice in this matter if I use VGG16 or VGG19, or inception V3 with customized top layers?
Thanks.

Both Flatten and GlobalAveragePooling2D are valid options. So is GlobalMaxPooling2D.
Flatten will result in a larger Dense layer afterwards, which is more expensive and may result in worse overfitting. But if you have lots of data, it might also perform better.
As usual, it depends completely on your problem.


User registration fixes (name is optional)

Comment on STUDYBOX-166 suggests, that in requirements there are no "name" field required. But, however, we can predict, that this would be required in some time, so i didn't delete it, and just made it optional

Thanks!


stop css breaking in a popup BS3

replacement for request: https://github.com/vitalets/x-editable/pull/681

Hi, can you redo this PR to 'dev'? Thanks.


Python3

Test ran sucessfull on local machine with python 2.7 and 3.4.

Coverage increased (+0.4%) to 90.525% when pulling aa9f9b9d51a17972804eeb503c7442ea81d9557f on umeier:python3 into e0b2dbfa9ce2ad53cacf520b6ee111e349297f32 on loicgasser:master.

Great I ll have a look at it this week !

This is very good, thank you. I have only minor comments.
I would just like to keep the lazy behaviour in py2 as well.
I think this deserves a new version and will gladly deploy it when these comments are addressed.

I´m out of office next week, will address your comments asap.

Coverage increased (+0.4%) to 90.525% when pulling 21eb2a9f933d355865b1a6afcef38b8b84fd58a9 on umeier:python3 into e0b2dbfa9ce2ad53cacf520b6ee111e349297f32 on loicgasser:master.

Coverage increased (+0.4%) to 90.525% when pulling a5bfab8aad5b5f7981206e14da6eeea4a76f4f37 on umeier:python3 into e0b2dbfa9ce2ad53cacf520b6ee111e349297f32 on loicgasser:master.


Fixed typo (occured to occurred) in comments and resources.

I just fixed a typo in three files. This is my first attempt to collaborate in an open source project so I picked something simple to get the hang of it.

Hi @payini, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

Please go ahead and sign the CLA that the bot is requesting (takes just a few minutes) and we can go ahead and merge this. Thanks for sending the PR!

@payini, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

@payini thanks!
@dougbu can you review & merge?

@Eilon milestone?

Can wait for RC so we don't slow down any builds.

:shipit: will get this in shortly

f162f70

Awesome. My first contribution. Thank you.


readarr: <IP_ADDRESS>65 -> <IP_ADDRESS>87

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for readarr is: Usenet/BitTorrent ebook downloader
meta.homepage for readarr is: https://readarr.com
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS>78 with grep in /nix/store/5vzknam5x9ygqq9ic72vacp2nqpmxsj0-readarr-<IP_ADDRESS>78
found <IP_ADDRESS>78 in filename of file in /nix/store/5vzknam5x9ygqq9ic72vacp2nqpmxsj0-readarr-<IP_ADDRESS>78

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

readarr

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5vzknam5x9ygqq9ic72vacp2nqpmxsj0-readarr-<IP_ADDRESS>78 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A readarr https://github.com/r-ryantm/nixpkgs/archive/725ecd56cca277e5e032a2dbff4053451166243f.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/725ecd56cca277e5e032a2dbff4053451166243f#readarr

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5vzknam5x9ygqq9ic72vacp2nqpmxsj0-readarr-<IP_ADDRESS>78
ls -la /nix/store/5vzknam5x9ygqq9ic72vacp2nqpmxsj0-readarr-<IP_ADDRESS>78/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

readarr

Maintainer pings
cc @jocelynthode for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for readarr is: Usenet/BitTorrent ebook downloader
meta.homepage for readarr is: https://readarr.com
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS>87 with grep in /nix/store/a68cmb3fp911vvzcjir9qipdgcnylajc-readarr-<IP_ADDRESS>87
found <IP_ADDRESS>87 in filename of file in /nix/store/a68cmb3fp911vvzcjir9qipdgcnylajc-readarr-<IP_ADDRESS>87

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

readarr

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/a68cmb3fp911vvzcjir9qipdgcnylajc-readarr-<IP_ADDRESS>87 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A readarr https://github.com/r-ryantm/nixpkgs/archive/6258ca37e3eac87539c739afa5ace5802cc3561a.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/6258ca37e3eac87539c739afa5ace5802cc3561a#readarr

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/a68cmb3fp911vvzcjir9qipdgcnylajc-readarr-<IP_ADDRESS>87
ls -la /nix/store/a68cmb3fp911vvzcjir9qipdgcnylajc-readarr-<IP_ADDRESS>87/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

readarr

Maintainer pings
cc @jocelynthode for testing.


support a dispose func

hi, this is a great input wrapper plugin!
but l use it with turbolinks, can you support a dispose func to delete labelauty instande and remove relative dom element?

Hello,
First, thank you for using this plugin!
Currently, I am not maintaining this plugin since I dropped jQuery from my projects and started using reactive libraries.
Do you accept the challenge to create a pull request for this issue?
Otherwise, I will try to get some free time to implement this.


Various questions about implementation

Hey all, just reading through code and wondering why various decisions have been made. Oh and this list is unordered and just whatever thoughts appear while reading the code.

Why is validator Rc'd? Why not a Box?

Do App and Arg really need so many lifetimes? Can't they all just be bound to one? Why not just own Strings?

How many methods are there to create an App? Yaml, builder macro #217, usage string, manual builder. Am I missing any?

Why is there a default subcommand of help? Why not at least be opt-in via settings?

No FAQ? One very expected question; why not use docopt or getopts?

Does the Arg need to know what group it is in? Shouldn't the groups be handled outside of the Arg?

blacklist for conflicts? Why not just conflicts?

Any thoughts for using rustfmt to clean up the formatting of the code? The macros.rs for one looks like it could be cleaned up at least visually.

That'll do for now, but I'll likely update this issue with more questions as I get deeper into the code.

@james-darkfox Here's some of the answers, all great questions!

Why is validator Rc'd? Why not a Box?

Originally Box was the idea. But if an Arg is marked as global (meaning it's cloned down the child subcommands). I couldn't find a good solution that would satisfy sane lifetime requirements. Rc was simply the easiest method to accomplish this until I've got some time to revisit it...or someone smarter than me changes it ;)

Do App and Arg really need so many lifetimes? Can't they all just be bound to one? Why not just own Strings?

I wish they didn't. In the early days they were all bound to the 'static lifetime. But there times when you want pull a piece of information for an App or Arg at runtime, and therefore can't use 'static. There may be a way to specify that lifetimes of those particular settings are at least as long as the App struct itself and cut them down...but I haven't played it lately.
The reason for not using owned Strings was to try and keep clap competitive in the performance department. 99% of the settings and such use a 'static lifetime and don't need to be re-allocated in order to be used.

How many methods are there to create an App? Yaml, builder macro #217, usage string, manual builder. Am I missing any?

To build an App there are two (Yaml, and builder pattern, your macro would make three).
To build an Arg there are three (Builder, usage string, Yaml, and your macro would make four).

Why is there a default subcommand of help? Why not at least be opt-in via settings?

There is only a default subcommand of help when other subcommands have been defined manually. So it's opt-in(ish), being that you only get a help subcommand if you're actually using subcommands. Also, if the user defined a help subcommand themselves, the auto-generated one wouldn't be added (meaning it's only generated if the user hasn't defined one themselves). So the only missing thing I guess is allowing a user to not have a help subcommand at all. If there are users who want a way to opt-out of help I can look at putting that in, just no one has asked for it.

No FAQ? One very expected question; why not use docopt or getopts?

Adding a FAQ is a great idea! To answer that number one question I'd say it's two-fold:
Reasons to use clap instead of doctopt or getopt

You want to implement as little of the CLI experience as possible, i.e. just define your valid args and continue on with your application, yet still get all the nice CLI features users expect.
You need more performance than docopt provides
You have complex requirements/conflicts between your various valid args
You want to use subcommands
You want some sort of custom validation built into the argument parsing process and not as part of your application (which allows for earlier failures, better error messages, more cohesive experience, etc.)

Reasons to docopt instead clap:

You want automatic automatic serialization of your arguments into Rust types
You already have a docopt usage string already from another project or language

Reasons to use getopts instead of clap:

You need absolutely as few allocations as possible and don't mind implementing nearly everything yourself
You want to a portion of the arg parsing process to be very custom (i.e. again, implementing the details yourself)

Does the Arg need to know what group it is in? Shouldn't the groups be handled outside of the Arg?

I agree. I think this part is confusing and will probably be removed at 2.x. The idea though is that the Arg doesn't know what group its in, it's just a shorthand for specifying which group to add this Arg to. I.e. you've already set up a group, but didn't add this Arg to it yet.

blacklist for conflicts? Why not just conflicts?

That's just a legacy name that hasn't been refactored yet since it's internal and not public facing.

Any thoughts for using rustfmt to clean up the formatting of the code? The macros.rs for one looks like it could be cleaned up at least visually

Actually yes! :) I ran rustfmt against it a while back, but seeing all the news about rustfmt I think it's time to give it another run :+1:

Thanks! Answers are all good. One typo however:

"You want automatic automatic serialization of your arguments into Rust types" - Reasons for docopt

I have two possible additions to make clap even better:

Interactive CLI. An interactive CLI or prompt, shares the same parsing requirements as command line arguments and if it were as easy as a single macro call to generate such prompt, clap could quickly become a direct goto for developers who need some interactive ability to play with their code. (This would likely need to be feature gated)

Configurable template trait DisplayApp

I like the prompt/interactive cli idea, but yeah I would assume this is be feature gated. If you want to work something up we can see how it fits within the rest of clap!
For the template trait I'd need to see some examples of what you mean. I think I have an idea, but I could be way off too.

Docopt / posix getopt formats are two examples already featured.
Another example would be a straight list of subcommands where no arguments are even suitable.

It's somewhat possible to do already, albiet in a slightly more manual but more flexible fashion.
For example, users can override the usage string do display a more doctopt/getopts style usage string (even multi lined) using .usage(). They can also add additional details after the help message using .after_help(). There is also the option to combine what clap calls Options and Flags into a single "Options" sections (a la docopt/getopts) using AppSettings::UnifiedHelpMessage variants. Finally, the user can just simply override the entire help message if that's what they wish to do.
Do you mean something beyond changing those aspects, or something else entirely?

The wiki page FAQ has been started based on the questions listed here. Once we have some good additions/suggestions, etc. I'll put a link to it in the readme as well as create  FAQ.md

I mean like having a trait that iterates over the object to create the textual structure itself. Then you only need to pass the App through such implementations and you'll get your desired string.

All questions answered. Respective spawned issues created. This issue is tracking nothing anymore.


example app on recent version of Flutter SDK and Android Studio

I tried to run the example app in the repository on my Android device and encountered the following error.
$ git clone git@github.com:shaqian/flutter_tflite.git
$ cd flutter_tflite/example
$ flutter build appbundle # or flutter run
[!] Your app isn't using AndroidX.
    To avoid potential build failures, you can quickly migrate your app by following the steps on https://goo.gl/CP92wY.
Running Gradle task 'bundleRelease'...
Running Gradle task 'bundleRelease'... Done                         1.9s
Gradle build failed to produce an .aab file. It's likely that this file was generated under /Users/kstg/src/github.com/shaqian/flutter_tflite/example/build, but the tool couldn't find it.

My environment is the following version:
Flutter (Channel stable, v1.12.13+hotfix.8, on Mac OS X 10.15.3 19D76, locale en-JP)
Android Studio 3.6.1

After adapting the changes made by this pull request, the build succeeds
$ flutter build appbundle
Running Gradle task 'bundleRelease'...
Running Gradle task 'bundleRelease'... Done                         8.2s
✓ Built build/app/outputs/bundle/release/app-release.aab (83.4MB).

it runs flutter run and it works fine on the device.
Please check this pull request as a reference for resolution. thanks.
AndroidX Migration - Flutter
P.S.
I excluded the .ides/ directory to not include it in the repository because files are generated in the developer's environment.

Thank you!


Suggestions for new issue labels

I feel we should also add a few more labels, some organizations have them tagged as first-timers-only, low-hanging-fruits, easyfix.
While it is good to follow existing trends, it is also adventurous to set new ones.
Suggest some new ones to the existing list

Is it okay to have both area/maya and repo/maya  issue tags, how different it are they supposed to be?

hmmm.. @vharsh you have a point there. maya is really taking on different meanings depending on the context.

area/maya refers to the orchestration functionality
repo/maya refers to the code related to that functionality being checked in to repository/maya. However it is possible that the functionality could also be in other repositories like repo/kuberentes-incubator or repo/jiva. While the area/maya still holds good.


Support<EMAIL_ADDRESS>

New version of got was released recently.
https://github.com/sindresorhus/got/releases/tag/v12.0.0

Update: this will take a bit of time. Got has migrated over to using ES modules. Jest does not fully support ESM so I'll have to migrate the tests to a new runner.

I was kinda hoping I'd get by just enabling Jest's "experimental" ESM support but it just breaks nock interception mechanism :/
I've tried running a simple nock test in an ESM context (.mjs file extenstion + direct use of import statements in the test source) and it intercepted HTTP calls as expected just by running it with plain node.

Released as 5.0.0-next under the next channel on npm
npm install -g got-fetch@next

Officialy published as 5.0.0 🙌
npm install got-fetch@5


💫 [FeatureRequest] - Musical Alignment

Feature request
DatamuseAPI / GeniusAPI for rhyme and lyric retrieval in data pipeline, also slightly related to this:
https://github.com/Dartvauder/NeuroSandboxWebUI/issues/272
Motivation
Humans easily get aligned to belief structures through overexposure to music. I want to see if we can do something similar through self-alignment, and want to setup a training pipeline through this project. Particularly I tried running through git clone yesterday and it failed when start.bat was ran it just opened command spit something out then closed, nothing else. Now I'm reinstalling with the release 0.2.9 and will update this thread if I can get that working.
Your contribution
Break barrier to first issue and also prove whether or not it works on a Window 10/11 machine.
Related Links
https://www.datamuse.com/api/, https://lyricsgenius.readthedocs.io/en/master/reference/genius.html
Priority
Low
Expected Outcome
Available integration of DatamuseAPI/GeniusAPI as a tool or equivalent input/ouput device in synth data pipeline.

Okay I didnt enable auto-launch through the github option, but that being said, it's not very clear.