Random Failure of LetsEncrypt renewal script

Just capturing this in an issue for posterity.
Output from command /opt/letsencrypt/letsencrypt-auto renew --rsa-key-size 4096 --renew-hook "/usr/bin/monit restart nginx":
Bootstrapping dependencies for Debian-based OSes... (you can skip this with --no-bootstrap)
Hit:1 http://ppa.launchpad.net/nginx/stable/ubuntu xenial InRelease
Get:2 http://security.ubuntu.com/ubuntu xenial-security InRelease [102 kB]
Hit:3 http://nyc2.mirrors.digitalocean.com/ubuntu xenial InRelease
Get:5 http://apt.postgresql.org/pub/repos/apt xenial-pgdg InRelease [41.5 kB]
Get:4 http://nyc2.mirrors.digitalocean.com/ubuntu xenial-updates InRelease [102 kB]
Get:6 http://nyc2.mirrors.digitalocean.com/ubuntu xenial-backports InRelease [102 kB]
Hit:7 https://deb.nodesource.com/node_6.x xenial InRelease
Get:8 http://nyc2.mirrors.digitalocean.com/ubuntu xenial-updates/main Sources [257 kB]
Get:9 http://nyc2.mirrors.digitalocean.com/ubuntu xenial-updates/universe Sources [163 kB]
Get:10 http://nyc2.mirrors.digitalocean.com/ubuntu xenial-updates/main amd64 Packages [572 kB]
Get:11 http://nyc2.mirrors.digitalocean.com/ubuntu xenial-updates/main Translation-en [232 kB]
Get:12 http://nyc2.mirrors.digitalocean.com/ubuntu xenial-updates/universe amd64 Packages [492 kB]
Get:13 http://security.ubuntu.com/ubuntu xenial-security/main Sources [77.8 kB]
Get:14 http://nyc2.mirrors.digitalocean.com/ubuntu xenial-updates/universe Translation-en [195 kB]
Get:15 http://security.ubuntu.com/ubuntu xenial-security/main amd64 Packages [297 kB]
Get:16 http://security.ubuntu.com/ubuntu xenial-security/universe amd64 Packages [141 kB]
Get:17 http://apt.postgresql.org/pub/repos/apt xenial-pgdg/main amd64 Packages [58.8 kB]
Fetched 2,834 kB in 1s (1,672 kB/s)
Reading package lists... Done
N: Ignoring file '50unattended-upgrades.ucf-dist' in directory '/etc/apt/apt.conf.d/' as it has an invalid filename extension
Reading package lists... Done
Building dependency tree
Reading state information... Done
ca-certificates is already the newest version (20160104ubuntu1).
gcc is already the newest version (4:5.3.1-1ubuntu1).
libffi-dev is already the newest version (3.2.1-4).
python is already the newest version (2.7.11-1).
The following packages were automatically installed and are no longer required:
  linux-headers-4.4.0-66 linux-headers-4.4.0-66-generic linux-headers-4.4.0-72 linux-headers-4.4.0-72-generic linux-headers-4.4.0-75
  linux-headers-4.4.0-75-generic linux-headers-4.4.0-77 linux-headers-4.4.0-77-generic linux-headers-4.4.0-79 linux-headers-4.4.0-79-generic
  linux-image-4.4.0-66-generic linux-image-4.4.0-72-generic linux-image-4.4.0-75-generic linux-image-4.4.0-77-generic linux-image-4.4.0-79-generic
Use 'apt autoremove' to remove them.
The following additional packages will be installed:
  libpython-dev libpython2.7-dev libssl1.0.0 python-pip-whl python2.7-dev python3-virtualenv
Suggested packages:
  augeas-doc augeas-tools
Recommended packages:
  libssl-doc
The following NEW packages will be installed:
  augeas-lenses libaugeas0 libpython-dev libpython2.7-dev python-dev python-pip-whl python-virtualenv python2.7-dev python3-virtualenv virtualenv
The following packages will be upgraded:
  libssl-dev libssl1.0.0 openssl
3 upgraded, 10 newly installed, 0 to remove and 32 not upgraded.
19 not fully installed or removed.
Need to get 0 B/32.6 MB of archives.
After this operation, 44.7 MB of additional disk space will be used.
N: Ignoring file '50unattended-upgrades.ucf-dist' in directory '/etc/apt/apt.conf.d/' as it has an invalid filename extension
Preconfiguring packages ...
(Reading database ... 269550 files and directories currently installed.)
Preparing to unpack .../libssl-dev_1.0.2g-1ubuntu4.8_amd64.deb ...
Unpacking libssl-dev:amd64 (1.0.2g-1ubuntu4.8) over (1.0.2g-1ubuntu4.6) ...
Preparing to unpack .../libssl1.0.0_1.0.2g-1ubuntu4.8_amd64.deb ...
Unpacking libssl1.0.0:amd64 (1.0.2g-1ubuntu4.8) over (1.0.2g-1ubuntu4.6) ...
Preparing to unpack .../openssl_1.0.2g-1ubuntu4.8_amd64.deb ...
Unpacking openssl (1.0.2g-1ubuntu4.8) over (1.0.2g-1ubuntu4.6) ...
Selecting previously unselected package augeas-lenses.
Preparing to unpack .../augeas-lenses_1.4.0-0ubuntu1_all.deb ...
Unpacking augeas-lenses (1.4.0-0ubuntu1) ...
Selecting previously unselected package libaugeas0.
Preparing to unpack .../libaugeas0_1.4.0-0ubuntu1_amd64.deb ...
Unpacking libaugeas0 (1.4.0-0ubuntu1) ...
Selecting previously unselected package libpython2.7-dev:amd64.
Preparing to unpack .../libpython2.7-dev_2.7.12-1ubuntu0~16.04.1_amd64.deb ...
Unpacking libpython2.7-dev:amd64 (2.7.12-1ubuntu0~16.04.1) ...
Selecting previously unselected package libpython-dev:amd64.
Preparing to unpack .../libpython-dev_2.7.11-1_amd64.deb ...
Unpacking libpython-dev:amd64 (2.7.11-1) ...
Selecting previously unselected package python2.7-dev.
Preparing to unpack .../python2.7-dev_2.7.12-1ubuntu0~16.04.1_amd64.deb ...
Unpacking python2.7-dev (2.7.12-1ubuntu0~16.04.1) ...
Selecting previously unselected package python-dev.
Preparing to unpack .../python-dev_2.7.11-1_amd64.deb ...
Unpacking python-dev (2.7.11-1) ...
Selecting previously unselected package python-pip-whl.
Preparing to unpack .../python-pip-whl_8.1.1-2ubuntu0.4_all.deb ...
Unpacking python-pip-whl (8.1.1-2ubuntu0.4) ...
Selecting previously unselected package python-virtualenv.
Preparing to unpack .../python-virtualenv_15.0.1+ds-3ubuntu1_all.deb ...
Unpacking python-virtualenv (15.0.1+ds-3ubuntu1) ...
Selecting previously unselected package python3-virtualenv.
Preparing to unpack .../python3-virtualenv_15.0.1+ds-3ubuntu1_all.deb ...
Unpacking python3-virtualenv (15.0.1+ds-3ubuntu1) ...
Selecting previously unselected package virtualenv.
Preparing to unpack .../virtualenv_15.0.1+ds-3ubuntu1_all.deb ...
Unpacking virtualenv (15.0.1+ds-3ubuntu1) ...
Processing triggers for libc-bin (2.23-0ubuntu9) ...
Processing triggers for man-db (2.7.5-1) ...
Setting up apport (2.20.1-0ubuntu2.6) ...
insserv: warning: script 'puma' missing LSB tags and overrides
insserv: There is a loop between service monit and puma if stopped
insserv:  loop involving service puma at depth 2
insserv:  loop involving service monit at depth 1
insserv: Stopping puma depends on monit and therefore on system facility `$all' which can not be true!
insserv: exiting now without changing boot order!
update-rc.d: error: insserv rejected the script header
dpkg: error processing package apport (--configure):
 subprocess installed post-installation script returned error exit status 1
Setting up nginx-common (1.12.0-1+xenial1) ...
insserv: warning: script 'puma' missing LSB tags and overrides
insserv: There is a loop between service monit and puma if stopped
insserv:  loop involving service puma at depth 2
insserv:  loop involving service monit at depth 1
insserv: Stopping puma depends on monit and therefore on system facility `$all' which can not be true!
insserv: exiting now without changing boot order!
update-rc.d: error: insserv rejected the script header
dpkg: error processing package nginx-common (--configure):
 subprocess installed post-installation script returned error exit status 1
dpkg: dependency problems prevent configuration of libnginx-mod-http-auth-pam:
 libnginx-mod-http-auth-pam depends on nginx-common (= 1.12.0-1+xenial1); however:
  Package nginx-common is not configured yet.

dpkg: error processing package libnginx-mod-http-auth-pam (--configure):
 dependency problems - leaving unconfigured
dpkg: dependency problems prevent configuration of libnginx-mod-http-dav-ext:
 libnginx-mod-http-dav-ext depends on nginx-common (= 1.12.0-1+xenial1); however:
  Package nginx-common is not configured yet.

dpkg: error processing package libnginx-mod-http-dav-ext (--configure):
 dependency problems - leaving unconfigured
dpkg: dependency problems prevent configuration of libnginx-mod-http-echo:
 libnginx-mod-http-echo depends on nginx-common (= 1.12.0-1+xenial1); however:
  Package nginx-common is not configured yet.

dpkg: error processing package libnginx-mod-http-echo (--configure):
 dependency problems - leaving unconfigured
dpkg: dependency problems prevent configuration of libnginx-mod-http-geoip:
 libnginx-mod-http-geoip depends on nginx-coNo apport report written because the error message indicates its a followup error from a previous failure.
                                                                                No apport report written because MaxReports is reached already
                                                      No apport report written because MaxReports is reached already
                                                                                mmon (= 1.12.0-1+xenial1); however:
  Package nginx-common is not configured yet.

dpkg: error processing package libnginx-mod-http-geoip (--configure):
 dependency problems - leaving unconfigured
dpkg: dependency problems prevent configuration of libnginx-mod-http-image-filter:
 libnginx-mod-http-image-filter depends on nginx-common (= 1.12.0-1+xenial1); however:
  Package nginx-common is not configured yet.

dpkg: error processing package libnginx-mod-http-image-filter (--configure):
No apport report written because MaxReports is reached already
                                                              No apport report written because MaxReports is reached already
                                                                                dependency problems - leaving unconfigured
dpkg: dependency problems prevent configuration of libnginx-mod-http-subs-filter:
 libnginx-mod-http-subs-filter depends on nginx-common (= 1.12.0-1+xenial1); however:
  Package nginx-common is not configured yet.

dpkg: error processing package libnginx-mod-http-subs-filter (--configure):
 dependency problems - leaving unconfigured
dpkg: dependency problems prevent configuration of libnginx-mod-http-upstream-fair:No apport report written because MaxReports is reached already

 libnginx-mod-http-upstream-fair depends on nginx-common (= 1.12.0-1+xenial1); however:
  Package nginx-common is not configured yet.

dpkg: error processing package libnginx-mod-http-upstream-fair (--configure):
 dependency problems - leaving unconfigured
dpkg: dependency problems prevent configuration of libnginx-mod-http-xslt-filter:No apport report written because MaxReports is reached already

 libnginx-mod-http-xslt-filter depends on nginx-common (= 1.12.0-1+xenial1); however:
  Package nginx-common is not configured yet.

dpkg: error processing package libnginx-mod-http-xslt-filter (--configure):
 dependency problems - leaving unconfigured
dpkg: dependency problems prevent configuration of libnginx-mod-mail:No apport report written because MaxReports is reached already

 libnginx-mod-mail depends on nginx-common (= 1.12.0-1+xenial1); however:
  Package nginx-common is not configured yet.

dpkg: error processing package libnginx-mod-mail (--configure):
 dependency problems - leaving unconfigured
dpkg: dependency problems prevent configuration of libnginx-mod-stream:No apport report written because MaxReports is reached already

 libnginx-mod-stream depends on nginx-common (= 1.12.0-1+xenial1); however:
  Package nginx-common is not configured yet.

dpkg: error processing package libnginx-mod-stream (--configure):
 dependency problems - leaving unconfigured
Setting up libssl1.0.0:amd64 (1.0.2g-1ubuntu4.8) ...No apport report written because MaxReports is reached already

Setting up libssl-dev:amd64 (1.0.2g-1ubuntu4.8) ...
dpkg: dependency problems prevent configuration of nginx-full:
 nginx-full depends on libnginx-mod-http-auth-pam (= 1.12.0-1+xenial1); however:
  Package libnginx-mod-http-auth-pam is not configured yet.
 nginx-full depends on libnginx-mod-http-dav-ext (= 1.12.0-1+xenial1); however:
  Package libnginx-mod-http-dav-ext is not configured yet.
 nginx-full depends on libnginx-mod-http-echo (= 1.12.0-1+xenial1); however:
  Package libnginx-mod-http-echo is not configured yet.
 nginx-full depends on libnginx-mod-http-geoip (= 1.12.0-1+xenial1); however:
  Package libnginx-mod-http-geoip is not configured yet.
 nginx-full depends on libnginx-mod-http-image-filter (= 1.12.0-1+xenial1); however:
  Package libnginx-mod-http-image-filter is not configured yet.
 nginx-full depends on libnginx-mod-http-subs-filter (= 1.12.0-1+xenial1); however:
  Package libnginx-mod-http-subs-filter is not configured yet.
 nginx-full depends on libnginx-mod-http-upstream-fair (= 1.12.0-1+xenial1); however:
  Package libnginx-mod-http-upstream
dpkg: error processing package nginx-full (--configure):
 dependency problems - leaving unconfigured
dpkg: dependency problems prevent configuration of nginx:No apport report written because MaxReports is reached already

 nginx depends on nginx-full (<< 1.12.0-1+xenial1.1~) | nginx-light (<< 1.12.0-1+xenial1.1~) | nginx-extras (<< 1.12.0-1+xenial1.1~); however:
  Package nginx-full is not configured yet.
  Package nginx-light is not installed.
  Package nginx-extras is not installed.
 nginx depends on nginx-full (>= 1.12.0-1+xenial1) | nginx-light (>= 1.12.0-1+xenial1) | nginx-extras (>= 1.12.0-1+xenial1); however:
  Package nginx-full is not configured yet.
  Package nginx-light is not installed.
  Package nginx-extras is not installed.

dpkg: error processing package nginx (--configure):
 dependency problems - leaving unconfigured
Setting up openssh-server (1:7.2p2-4ubuntu2.2) ...No apport report written because MaxReports is reached already

insserv: warning: script 'puma' missing LSB tags and overrides
insserv: There is a loop between service monit and puma if stopped
insserv:  loop involving service puma at depth 2
insserv:  loop involving service monit at depth 1
insserv: Stopping puma depends on monit and therefore on system facility `$all' which can not be true!
insserv: exiting now without changing boot order!
update-rc.d: error: insserv rejected the script header
dpkg: error processing package openssh-server (--configure):
 subprocess installed post-installation script returned error exit status 1
Setting up postgresql-common (182.pgdg16.04+1) ...No apport report written because MaxReports is reached already

insserv: warning: script 'puma' missing LSB tags and overrides
insserv: There is a loop between service monit and puma if stopped
insserv:  loop involving service puma at depth 2
insserv:  loop involving service monit at depth 1
insserv: Stopping puma depends on monit and therefore on system facility `$all' which can not be true!
insserv: exiting now without changing boot order!
update-rc.d: error: insserv rejected the script header
dpkg: error processing package postgresql-common (--configure):
 subprocess installed post-installation script returned error exit status 1
dpkg: dependency problems prevent configuration of postgresql-9.4:No apport report written because MaxReports is reached already

 postgresql-9.4 depends on postgresql-common (>= 142~); however:
  Package postgresql-common is not configured yet.

dpkg: error processing package postgresql-9.4 (--configure):
 dependency problems - leaving unconfigured
dpkg: dependency problems prevent configuration of postgresql-contrib-9.4:No apport report written because MaxReports is reached already

 postgresql-contrib-9.4 depends on postgresql-9.4 (= 9.4.12-1.pgdg16.04+1); however:
  Package postgresql-9.4 is not configured yet.

dpkg: error processing package postgresql-contrib-9.4 (--configure):
 dependency problems - leaving unconfigured
Setting up unattended-upgrades (0.90ubuntu0.6) ...No apport report written because MaxReports is reached already

insserv: warning: script 'puma' missing LSB tags and overrides
insserv: There is a loop between service monit and puma if stopped
insserv:  loop involving service puma at depth 2
insserv:  loop involving service monit at depth 1
insserv: Stopping puma depends on monit and therefore on system facility `$all' which can not be true!
insserv: exiting now without changing boot order!
update-rc.d: error: insserv rejected the script header
dpkg: error processing package unattended-upgrades (--configure):
 subprocess installed post-installation script returned error exit status 1
Setting up openssl (1.0.2g-1ubuntu4.8) ...No apport report written because MaxReports is reached already

Setting up augeas-lenses (1.4.0-0ubuntu1) ...
Setting up libaugeas0 (1.4.0-0ubuntu1) ...
Setting up libpython2.7-dev:amd64 (2.7.12-1ubuntu0~16.04.1) ...
Setting up libpython-dev:amd64 (2.7.11-1) ...
Setting up python2.7-dev (2.7.12-1ubuntu0~16.04.1) ...
Setting up python-dev (2.7.11-1) ...
Setting up python-pip-whl (8.1.1-2ubuntu0.4) ...
Setting up python-virtualenv (15.0.1+ds-3ubuntu1) ...
Setting up python3-virtualenv (15.0.1+ds-3ubuntu1) ...
Setting up virtualenv (15.0.1+ds-3ubuntu1) ...
Processing triggers for libc-bin (2.23-0ubuntu9) ...
Errors were encountered while processing:
 apport
 nginx-common
 libnginx-mod-http-auth-pam
 libnginx-mod-http-dav-ext
 libnginx-mod-http-echo
 libnginx-mod-http-geoip
 libnginx-mod-http-image-filter
 libnginx-mod-http-subs-filter
 libnginx-mod-http-upstream-fair
 libnginx-mod-http-xslt-filter
 libnginx-mod-mail
 libnginx-mod-stream
 nginx-full
 nginx
 openssh-server
 postgresql-common
 postgresql-9.4
 postgresql-contrib-9.4
 unattended-upgrades
N: Ignoring file '50unattended-upgrades.ucf-dist' in directory '/etc/apt/apt.conf.d/' as it has an invalid filename extension
E: Sub-process /usr/bin/dpkg returned an error code (1)

Fixed with #132


Direct link to the locally archive webpage (from the listing page)

Dear @sissbruecker,
Many thanks for your great app, especially the recent feature #672 !
As a huge fan of this new feature, it would be handy (at least for me) to have the ability to change the archive link (from the listing page) to use the locally archived version instead of the Web archive website (see the attached screenshot for the link I mention).
Indeed, having a direct link to the local archive version would speed up the usage of my local archive instead of clicking "view" and selecting the latest local archive. You can propose a toggle on the settings page to see if a web archive or local archive must be used for this link.
If you plan to address this suggestion, I highly suggest you to link the latest HTML archive of the local archive. This means that if one manually uploads an HTML archive (typically from an authenticated webpage), this link would point to that manually uploaded document.
Cheers!

Yes I would love this as well because if I disable url validation and put it in a local file directly even if you click it it will not open the html to the local archive.


Xcode 12 - beta - error opening realm file

I am just testing out a macOS app with Xcode 12 beta running on macOS 11 (10.16) beta and get a realm error when opening the realm. See error message below.
UPDATE:
I just tried a notarised version of the app and that works fine so I guess there is some permission change that no longer allows debug apps to access the file system - I assume there must be a setting somewhere to get around this.
FURTHER EDIT:
I had to add two privacy entries in the apps info.plist to get past the file permission error but now I have run into another realm error: "Key already used" - any idea why that is ?  Could it be a migration issue since the database being opened is a 4.6 database.
Ok I just tested this by trying to use Realm Swift 5.1.0 and get this error with Xcode 11.5/macOS10.15 - so I guess it must be a realm database upgrade error.  I assume 5.1.0 is supposed to be compatible with 4.4 databases and should be able to open them.
I have also just tried opening the realm file with v3.11.0 Realm Studio and get the same error message on the main Studio screen.
My guess is there must be a duplicate primary key in the database somewhere - although that does raise the question of how it got there in the first place...  So any suggestions on how to log the migration in order to find the duplicate key ?

@duncangroenewald
Could you post the messages of the errors you're mentioning. Please include the Realm and Xcode versions for each too.

That's it - that's the only error message "Key already used".
It comes from this function in realm-core cluster.cpp from what I can tell. I couldn't find that message anywhere else in the source code.
ref_type Cluster::insert(ObjKey k, const FieldValues& init_values, ClusterNode::State& state)
{
    int64_t current_key_value = -1;
    size_t sz;
    size_t ndx;

    if (m_keys.is_attached()) {
        sz = m_keys.size();
        ndx = m_keys.lower_bound(uint64_t(k.value));
        if (ndx < sz) {
            current_key_value = m_keys.get(ndx);
            if (k.value == current_key_value) {
                throw InvalidKey("Key already used");
            }
        }
    }
    else {
        sz = size_t(Array::get(s_key_ref_or_size_index)) >> 1; // Size is stored as tagged integer
        if (k.value < int(sz)) {
            throw InvalidKey("Key already used");
        }
        // Key value is bigger than all other values, should be put last
        ndx = sz;
        if (k.value > int(sz) && sz < cluster_node_size) {
            ensure_general_form();
        }
    }

Any suggestions on how to move forward on this ?

On 27 Jun 2020, at 1:47 am, ericjordanmossman<EMAIL_ADDRESS>wrote:
@duncangroenewald https://github.com/duncangroenewald
Could you post the messages of the errors you're mentioning. Please include the Realm and Xcode versions for each too.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/realm/realm-cocoa/issues/6604#issuecomment-650251125, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADJK5XHJNE4IWQ7SK7LNXB3RYS7HXANCNFSM4OI7NDAA.


Support pulling pull request from a specific head branch only

Currently, this resource allows to pull all pull requests, only, or all pull requests, which are created against a specific base branch. However, one use case is, that each branch in a git repository has it's own build pipeline in concourse. It would be great, if, in this case, this resource could be used to pull only pull requests which are created with this specific branch (head), and not all.

I am not sure I follow. If you want to limit the branch the PR was made against you can with base on the source, see the README for more information.

Sorry, if this wasn't clear. I know the base option already, however, I would want to trigger, when a PR is made from a specific branch. So, let's say, there's a master branch (which would be the base branch) and a feature branch feature/1. In this case, if I create a PR, master would be the base and feature/1 would be the head. My pipeline should only trigger, if the PR was made to get changes from feature/1 into master (or any other branch), but not, let's say, if someone creates a PR to pull changes from feature/2 into master.
Maybe it's even clearer when you take a look to my PR? :)

I'm not sure why you need to limit this. How many PRs are you making from a feature/1 branch?

Depends on, mostly one, however, afterwards the pipeline is being destroyed. That's probably not the best setup at all (there's one pipeline for each feature branch), however, it currently works pretty well, except for the pull request trigger (which would trigger each pipeline and not only the one for a specific branch).

I'm not sure I follow still.
You are creating a new pipeline for each feature branch. Why do you need to check the PRs on that feature branch then. Can't you just scope the git resource to the branch name of the feature branch.

Currently there're two kind of pipelines:

A feature branch pipeline (one for each feature branch) which is triggered by the git resource (scoped to one specific branch)

The pull request pipeline, which is triggered by each pull request. This is used to report a status of a test build back to the pull request and enable or disable merging.

Both pipeline types has the same tasks, they only differ based on the triggers and the puts. However, this means, there're two configuration files, which needs to be kept synchronous every time the pipeline needs to be changed. If the pull request resource could be limited to a specific head branch only, both pipelines could be merged into one, which would make things easier.

Why does the PR need to be tested here if you already have a pipeline for the feature branch? Isn't the PR just the feature branch?

The PR is built again to have the pull request marked as being mergeable (only, if the built and the tests are passing), or not-mergeable (if the built or tests are failing). Otherwise, it would be possible to merge a PR without having tests run.

I do not follow your explanation. What do you mean your PR is built again? The PR resource doesn't support anything like that.
I feel like I'm getting bits and pieces of the workflow. Not the full explaination of what problem you have and what this solves in that problem.

Ok, I'll try to explain it a bit wider, sorry if you feel, that the already provided information aren't enough :(
So, there's a github repo, where the code is hosted. The master branch of this repository is configured to "Require status checks to pass before merging", that means, that a concourse built needs to report back a "successful" build for a pull request before it can be merged, right?
Developers create a new feature branch whenever they start working on a new feature/story and push commits to this branch. These commits should trigger a pipeline in concourse to it get's checked (the pipeline will build the source code and execute tests on it). Once the development is finished, a pull request get's created, which asks for merging the changes from the feature branch into the master branch.
This pull request needs to have a successful status check (see my first point), before it is allowed to merge it (the "merge" button is disabled in the github UI). The pipeline, which is triggered by this pull request creation, is exactly the same (from a configuration point of view) as the one which gets triggered when a commit is pushed to a feature branch, with the only difference, that the pipeline is triggered by a pull request and not by a feature branch.
That means, that there're currently two pipelines, which are triggered by one branch, once by the git resource and once by the pull request. Instead it would be better to have one pipeline with different triggers only, instead of duplicating the pipeline.
Why there's a separate pipeline for each feature branch: This was the easiest way to get the branch name usable accross multiple resources, which does the built and deployment steps (deployment in a separate environment for each feature branch), that's why it's not that easily possible to use one pipeline which triggers for each git commit (no matter what branch it was made on).
So, having a filter for the head branch in the git pull request resource (like the barnch "filter" in the git resource) makes it possible to have each feature branch pipeline to trigger on a pull request or on a commit from the git resource. That's currently not useable, as each feature branch pipeline would be triggered, if one pull request would be created/updated, and not only the one for this particular feature branch.
I'm not sure, how I can explain it more clearly, but I hope that you got the problem now? :]

What is not clear to me, why do you have two different pipelines that do the same thing? It seems that you are you doing duplicate work. Testing the PR the exact same way as the feature branch. Why not one or the other? Adding this feature doesn't seem to solve the problem. As you'll still have a pipeline that tests the PR and the feature branch of the user, just at different times.

Glad you were able to work something out. I'm all for simplifying pipelines and workflows. 👍
That being said. The github pullrequest resource has a hidden feature. Whatever repo is given on path for the put the resource will set the status. It doesn't have to be a PR SHA.


EFCore in DotNetStandard2.0 Library contains not found dependencies

I have EFCore installed in a DotNetStandard2.0 CL and everything is working fine except I get a warning on Build regarding not found packages and resolution has to take place
This is the warning I get on Build
Severity	Code	Description	Project	File	Line	Suppression State
Warning	NU1603	Microsoft.Extensions.Primitives 2.0.0-preview1-final depends on
System.Runtime.CompilerServices.Unsafe (>= 4.4.0-preview1-25219-04)
but System.Runtime.CompilerServices.Unsafe 4.4.0-preview1-25219-04 was not found.
An approximate best match of System.Runtime.CompilerServices.Unsafe 4.4.0-preview1-25305-02
was resolved.

Installing System.Runtime.CompilerServices.Unsafe explicitly fixes warning
Install-Package System.Runtime.CompilerServices.Unsafe -Pre
Steps to reproduce
Create New DotNetStandard2.0 CL
Add Microsoft.EntityFrameworkCore.SqlServer
Run Build
Further technical details
EF Core version: 2.0.0-preview2-final
Database Provider: Sql Server
Operating system: Windows 10 Pro
IDE: VS 2017 Prevew 3

Can you share your csproj file and Nuget.config?

Certainly
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>netcoreapp2.0</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore.Identity.EntityFrameworkCore" Version="2.0.0-preview2-final" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.Design" Version="2.0.0-preview2-final" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Version="2.0.0-preview2-final" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer.Design" Version="2.0.0-preview1-final" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.Tools" Version="2.0.0-preview2-final" />
  </ItemGroup>

</Project>

Check out https://github.com/aspnet/EntityFramework/issues/8923#issuecomment-310431600
In preview2-final there is no SqlServer.Design package. Please remove that line and try again.

After removing the stale package restore works fine for me.

That did it, thanks


Extend scope k8s with deployment of prometheus exporter

The intention of this issue is to made easy as possible to deploy Prometheus Exporter https://github.com/criblio/appscope/pull/1413 with scope k8s command

Closing this manually, since it's been merged into release/1.4

Closing this manually, since it's been merged into release/1.4


Feature/sql script setup

Create database tables in the ansible script. Allows us to avoid running the sql script by hand.

approved!


Allow Registering Private Packages with Private Registry

I'm aware that the whole reason behind the private: true flag is to mark a package from being accidentally registered in bower registry. As is, the restriction applies to both public and private registry. I'm hoping there's an easy way to override this restriction if I were to register a private package WITH private registry.
Why? I've setup a private bower registry to host our internal packages (marked as private of course). Currently the only way to register these packages is by manually adding the necessary information directly into our registry. It would be nice if I could do this with bower client without throwing the error:
EPRIV The package you are trying to register is marked as private.

Maybe by executing a command, say, something like this:
bower register my-private-pkg the-private-pkg-url --private

I'm willing to create a PR for this, but I'm not sure if this kind of feature fits with the scope and philosophy of bower, or if there's already a better way to do it. What do you guys think? comment?
— Instead of all this, I know I could just remove the private: true flag on the private packages and be on my way, but that would defeat its original purpose.

not sure if this is the right place to ask question. So if i have my private component and i do not want to register it with bower what are the alternatives to install it withing various teams ?

Bower is deprecated and doesn't allow registering new packages anymore, so I'm closing this


Constexpr-based things in Any

@lisitsyn could you rebase and push... it should work now

@vigsterkr chido!

all equality tests failed 👻

all equality tests failed 👻

yay!

It is ready to be merged now.
@vigsterkr @karlnapf

:+1: squash and merge?

Deffs squash!


[142]

Summary:
alert('XSS')
Info
Reported by: Unknown<EMAIL_ADDRESS>- Open in Gleap
Location: IN
Type: 💬 INQUIRY
🔗 Open share link
🌍 Open ticket in Gleap

alert('XSS')


WorldMapOverlay: Clip to not draw over certain widgets

Also move WorldMapOverlay layer to ABOVE_WIDGETS to draw under the right-click menu properly.

Fixes #3456 & #3264

Needs rebase

What's the status of this PR? @MESLewis

I don't have time to rebase it, someone else is welcome to, it's not a big change. It's possible that widget ID's have changed.


chorn deugging code merge + ruby version bump

sorry i didn't want to merge this to master!  was trying to merge to my fork.


Clear clickable component value on rerun?

I have observed that, once a clickable component is clicked, the value of the component (now set to the id of the a that was clicked) is persisted indefinitely for the session. In other words, if I click a link inside clickable with id="event", then the value of clickable in the Python script will be event not just for the rerun following the click, but for every rerun afterwards as I interact with other components.
I assume this is the intended behavior, but it's an issue for my use case. I only want to trigger some behavior one time after the clickable is clicked (essentially I just want the same behavior as st.button). Is that possible with this component? I'm pretty new to Streamlit so maybe I'm missing something obvious.
I created a super simple streamlit app to reproduce the behavior and demonstrate my desired use case:
https://github.com/lansing/st-click-detector-issue
In the above app, I want to click one of the links, updating the session_state that the number box is bound to (changing the number to 0), then continue to increment the number box. But that's not possible because the clickable code block is getting executed on every subsequent rerun, resetting my number to 0.
I am currently using a hacky workaround that stores a last_clicked_event in the session_state and treats the clickable as un-clicked if its value is equal to st.session_state.last_clicked_event, but that obviously means you can't click the same event twice in a row.
Thanks, appreciate the contribution.

Hi @lansing. I'm not a Streamlit expert but I think that giving a specific key to a component leads to having this component's state persist from one session to another.
In your code, if you replace the static keys with different values for each session, you'll be able to increase the counter (I understand that's what you want). Here is one way to do it:
import random # <------
import streamlit as st
from st_click_detector import click_detector

html = """
    First Click me
    """

with st.sidebar:
    clicked_result = click_detector(html, key=str(random.randint(0, 1e12))) # <------
    if clicked_result != "":
        st.session_state.number = 0

with st.container():
    clicked_result_container = click_detector(html, key=str(random.randint(0, 1e12))) # <------
    if clicked_result_container != "":
        st.session_state.number = 0

    st.number_input("Then Increment me", value=1, key="number")
    st.write(f"st.session_state_number: {st.session_state.number}")

I'm having the same issue as OP, and I don't believe the above fixes the problem: making the key a random ID every time doesn't seem to register the click at all. What I'm trying to achieve (and I think OP) is:

link is clicked, variable is set
rest of streamlit app runs
variable is unset

FWIW I got around this by keeping track of what links were clicked (I believe similar to what OP is describing):
import streamlit as st
from st_click_detector import click_detector

if "links_clicked" not in st.session_state:
    st.session_state.links_clicked = {}

content = "My Link"
link_clicked = click_detector(content, key="key")

if link_clicked != "" and link_clicked not in st.session_state.links_clicked:
    # Do something only once here...
    st.session_state.links_clicked[link_clicked] = True

This let the click detector act as more of an event listener 😄

setComponentValue

Hey @nikodraca , you solution doesn't seem to register the click at all.

setComponentValue

Hey @nikodraca , you solution doesn't seem to register the click at all. Could you help out please?
My solution did not change any JS code, just the python snippet above (which has limitations of course)

curr_val

The code does not seem to work. The curr_val is always empty and the condition if (curr_val == links[i].id) is newer met. Could you create a branch with your code?

I believe it would be better if a feature can be added to detect single clicks, provided there are no technical issues. Using a single-click detector as a selector by tracking the last clicked element with session state is easier than using a selector to detect single clicks, as discussed previously.

I think we can simply reset the 'clicked' state by rerunning and using a different key. Here is my solution (modified from nikodraca's solution):
import streamlit as st
from st_click_detector import click_detector
import time

if "link_key1" not in st.session_state:
    st.session_state.link_key1 = 0
if "link_key2" not in st.session_state:
    st.session_state.link_key2 = 10000

content1 = "My Link1"
content2 = "My Link2"
link_clicked1 = click_detector(content1, key=st.session_state.link_key1)
link_clicked2 = click_detector(content2, key=st.session_state.link_key2)

if link_clicked1 != "":
    # Do something here...
    st.write('link1 clicked')
    st.session_state.link_key1 += 1
    time.sleep(1)
    st.rerun()
else:
    # Do something here...
    st.write('link1 not clicked')

if link_clicked2 != "":
    # Do something here...
    st.write('link2 clicked')
    st.session_state.link_key2 += 1
    time.sleep(1)
    st.rerun()
else:
    # Do something here...
    st.write('link2 not clicked')

It can support multiple click-detectors and won't listen to every event.

@mylank @PawelFaron
Did you find out how to create curr_val?


Setup a documentation repository

What needs to be done?

[ ] I suggest we clone https://github.com/OS2offdig/os2-docs-template

Any improvements or fixes before or after cloning should be contributed back to the template directory

[ ] Setup logos and initial readme and getting started docs
[ ] Make sure that people can understand what the source code does and how to use it.  Align with this guide to document the code

@kbp-di: Any progress on this? Do you need help or sparring for this task, then I'm available.


Support of other real types (Float32, auxiliary)

The first extra part of #1909, mainly addressing src/auxiliary.

Thanks for the review @ranocha ! I am thinking about how to test these PRs. Generally, I would treat these smaller PRs as preliminary improvements (as the general PR calling some basic functions here). So basically we could test and merge these smaller PRs first then merge and test the general PR. But what if some PRs are cross-referencing each other (for example, some PRs rely on the improvements from each other)?

Thanks for the review @ranocha ! I am thinking about how to test these PRs. Generally, I would treat these smaller PRs as preliminary improvements (as the general PR is calling some basic functions here). So basically we could test and merge these smaller PRs first then merge and test the general PR. But what if some PRs are cross-referencing each other (for example, some PRs rely on the improvements from each other)?
I am not sure whether the case I mentioned above is going to happen in these improvements.

In this case, the PRs providing functionality required by other changes should be merged first.

This depends on #1909 (or at least similar issues apply), right? If yes, please re-request my review here once changes in #1909 that are also pertinent for this PR here have been applied here as well 🙂

#1909 depends on this PR (and possibly more small PRs like this in the future).

Rerun CI please (if you like). I locally ran CI when these were running here - hitting the limit and thus too many CI failures here.


Using card as a connection state with red and green icons

Is it possible for me to add a colour when mapping from off to Disconnected and on to Connected?
It doesn't seem to be applying anything its just a black icon. Many Thanks
YAML configuration

type: 'custom:battery-state-card'
title: Connection Status
sort_by_level: desc
icon: 'hass:server-network'
color_thresholds:
  - value: 'off'
    color: red
  - value: 'on'
    color: green
state_map:
  - from: 'off'
    to: Disconnected
  - from: 'on'
    to: Connected
bulk_rename:
  - from: ' Connection State'
filter:
  include:
    - name: entity_id
      value: '/[a-z]{2}\d{10}_connection_state/'
      operator: matches

Screenshot

Version
1.6.2

No it is not possible because the threshold value has to be a number.
I must admit it is a really creative way of addopting this card for completely different scenario :) I hope that changing entity icon color will be supported i the nearest future natively in HA (I really don't understand why it's not there already)
You can take a look at this thread at community forum where people struggle with the same problem. Although the main solution from this thread is the CustomUI component which is pretty heavy - especially if the only thing you want to do is to change the entity icon color. The other suggestion from this topic is the lovelace-card-mod which sounds more reasonable.
I will keep this issue open as this wouldn't be a big effort to support such scenario - I will think about this more.

No it is not possible because the threshold value has to be a number.
I must admit it is a really creative way of addopting this card for completely different scenario :) I hope that changing entity icon color will be supported i the nearest future natively in HA (I really don't understand why it's not there already)
You can take a look at this thread at community forum where people struggle with the same problem. Although the main solution from this thread is the CustomUI component which is pretty heavy - especially if the only thing you want to do is to change the entity icon color. The other suggestion from this topic is the lovelace-card-mod which sounds more reasonable.
I will keep this issue open as this wouldn't be a big effort to support such scenario - I will think about this more.

Thank you for the reply. At least I'm not doing anything wrong.
Two ways I can think to implement this. Either support it direct or convert on and off to 0 and 100. This would give the colour and I think theres another request to be able to change the display value. Which sounds like it would kill two birds so to speak.
Thanks very much for your efforts this card is certainly very useful and flexible.

Thank you for the reply. At least I'm not doing anything wrong.
Two ways I can think to implement this. Either support it direct or convert on and off to 0 and 100. This would give the colour and I think theres another request to be able to change the display value. Which sounds like it would kill two birds so to speak.
Thanks very much for your efforts this card is certainly very useful and flexible.

It should be possible to achieve it in v3.
First in the state_map you need to convert off/on to numbers 0/1, in the same time you can specify display value as "Disconnected"/"Connected"
Second thing you use colors section to specify color per value (in your case 0 or 1)
Please take a look at the example of listing motion sensors:
https://github.com/maxwroc/battery-state-card#motion-sensors-sorted-by-state-and-last-changed-property
You can as well change the icon depending on the state which might be useful for this usecase too


refactor(ivc): simplify pp

Motivation
In the course of IVC implementation, the public parameters were significantly simplified
Overview

ck now provided as referense outside, it allows not to deal with management of such a large object internally and makes the system more flexible
Only table size instead of TableData, each step creates its own TableData
digest skips self.ck, details at #136
Instead of general serialization, the transformations needed for hashing became part of the digest method, which is much more correct (as it is not misleading)

could you explain this part: "Instead of general serialization, the transformations needed for hashing became part of the digest method, which is much more correct (as it is not misleading)"?
I don't quite get it yet.

could you explain this part: "Instead of general serialization, the transformations needed for hashing became part of the digest method, which is much more correct (as it is not misleading)"?

Serialization is still more about converting a data structure into another form, rather than an irreversible change. As soon as we skip some fields, make the transformation irreversible and only for the sake of hash - it looks counterintuitive.
That's why I removed the generic PP serialization, made the specification for digest

Although each step we synthesize it with different data, but the fixed columns and plonk_structure we are folding are the same. We have to make sure when folding two instances, they share the same plonk_structure e.g. same custom gates

The consistency of TableData is not based on reusing a single strcuture, but on the fact that we call a single configure between steps.
In general, we can migrate to configure TableData once in PublicParams and then clone it for each step to fix all parameters except witness.
Is this the kind of approach you had in mind?

I can bring back TableData and clone it for each step, but let's have a separate PR so we don't mess up the steamroller of PRs. Not a difficult change, bring back a bit of code.

I can bring back TableData and clone it for each step, but let's have a separate PR so we don't mess up the steamroller of PRs. Not a difficult change, bring back a bit of code.

That sounds good. Let's do it in future PR.


Type inference doesn't appear to be working on a generic

TypeScript Version:  2.5.3
Code
interface SocketHandler {
    <T>(
        locals: string,
        payload: T,
        callback: Function
    ): void;
}

interface SocketRouter {
    (
        eventName: string,
        handler: SocketHandler
    ): void;
}

interface StaffJSON {
    count: number;
    record: string;
}

class Staff {
    public count (): number {
        return 3; // would do some db interaction
    }
}

let router: SocketRouter = function ( name, handler ) {
    // Save the name and handler
}

router( 'staff-count', function<StaffJSON>(locals, payload, cb) {
    payload.record = 1;
    let staff = new Staff();
    cb( staff.count() );
} );

Expected behavior:
Expected this to result in an error since record is marked as a string, but a number is assigned.
Actual behavior:
The compiler doesn't complain at all. Moreover, the type inference displayed in VS Code shows that "payload" (and the other arguments of the router call's callback function) are of type "any". I expected the Generic to set the payload parameter to be StaffJSON type, while locals would be a string, etc.
My goal is to have router exposed from a module which can be included wherever requires - only the payload needs to be defined as a specific type (e.g. a different JSON object) for each call. Its somewhat similar to an Express router, hence my desire to keep router generic.
I'm guessing that this is a limitation in the type inference of the generics, and while I've read various other issues on this, I'm not really any closer to wrapping my head around it I'm afraid. My apologies if it has come up before and I haven't found it.
I presume there is a way to get this to work somehow - being able to type check what I'm using the payload for is why TypeScript is so attractive.

Try enabling --noImplicitAny and you'll get an error.
The problem is that function<StaffJSON>(locals, payload, cb) doesn't do what you think: it declares a function with a type parameter, and then doesn't use that type parameter. Since the function has a type parameter it doesn't get a contextual type and all of its parameters  are any.
Remove <StaffJSON> and you'll see that the types are locals: string, payload: T, cb: Function.
The problem is that SocketRouter declares to take a SocketHandler, and SocketHandler is a type of generic functions, which can only be satisfied by another generic function.
If you intended that a SocketHandler implementation can substitute anything for T, you might as well do away with the generics and just declare payload: any.

This feels like it should be a Stack Overflow question to me.   Maybe this is what you want to use, instead:
type SocketHandler<T> = (locals: string, payload: T, callback: Function) => void;

type SocketRouter<T> = (eventName: string, handler: SocketHandler<T>) => void;

interface StaffJSON {
  count: number;
  record: string;
}

declare const router: SocketRouter<StaffJSON>;

router('staff-count', function(locals, payload, cb) {
  payload.record = 1; // error as expected
});

If not, or you need more help, consider asking on Stack Overflow.  Good luck.

Thank you both for your replies - I really appreciate it! @andy-ms You are right, it isn't working the way I had anticipated - I thought since I had told it that the second parameter to a function with SocketRouter would be of type SocketHandler, it would match that signature and infer automatically. You've clarified that point for me - thank you :-).


Update expectations files for front-end tests

Many legacy and strong tests (and maybe others) have new expectations after https://dart-review.googlesource.com/c/sdk/+/99169

Fixed in https://github.com/dart-lang/sdk/commit/d3e00fd35e17bbbd90b477b62f648828a3a617bc.


[GPII-3507]: Monitor CouchDB snapshots with log-based metrics

This adds support of log-based metrics to our Stackdriver Ruby client, so we can create custom metric  counters for snapshot_created events and errors, and then alerting policy based on those metrics.
Known limitations / issues:

Some times it takes really long time for Stackdriver to digest newly created log-based metrics and make them available for use by alerting policies. During that time it keeps dying when trying to apply alert policies with The metric referenced by the provided filter is unknown. I am not sure of the solution to this at the moment.
Log-based metric that supposed to check for k8s-snapshots general errors is pretty dumb. Since k8s-snapshots does not separate logs by severity, and there are not much of error examples, I don't have better proposal atm.
Alert policy that based on log-based metrics fails in case any of those two conditions has not met:

Absense of k8s-snapshots errors.
Count of snapshot_created events for CouchDB is greater than 2 for 10 minutes.
This is does not exactly match reality (lowest replica count for CouchDB is 2 for dev clusters, with deltas of PT5M PT15M PT45M, which means that at least 4 new snapshots must be created in 10 minutes), but helps avoid issues with check alignment. I personally feel like it is good enough for the purpose, but we can always tweak this further if needed.

@mrtyler @stepanstipl Thanks for your reviews!
Summary of recent changes:

To workaround that nasty situation when log-based metric created, but not yet available for use by alerting policies, I had to move log-based metric creation into new module gcp-stackdriver-lbm and apply it at the same time with gcp-stackdriver-exclusion and gcp-stackdriver-export modules.
To avoid code duplication I decided to make our Stackdriver Ruby client more flexible to be called in the same way from new gcp-stackdriver-lbm module.
For the lack of better place, I put Stackdriver client ruby file into shared/rakefiles dir.
I changed alert policy configuration:

K8s-snapshots Errors Check is triggered when logging.googleapis.com/user/k8s_snapshots.error rate is higher than 1 for 5 minutes, which seemed to me like reasonable way to get notified in case there are persistent errors in k8s-snapshots logs.
K8s-snapshots Check for CouchDB is triggered when logging.googleapis.com/user/k8s_snapshots.couchdb.snapshot_created metric is absent for 5 minutes. Backup deltas for CouchDB starts with PT5M for all environments, meaning that there should be at least one successful backup event in 5 minutes, so this policy configuration seemed sufficient to trigger alerts in case snapshot creation events are missing in k8s-snapshots logs.

gcp-stackdriver-alerting module renamed into gcp-stackdriver-monitoring to make more sense, since it is dealing with other resources except alerting policies.

There are still couple of issues:

Issue with module dependencies: since gcp-stackdriver-lbm is now independent module, it would be handled one of the first on rake destroy as well, and would fail, trying to delete log-based metrics that are still in use by alerting policies of gcp-stackdriver-monitoring module. As workaround, I removed null resource that triggered on destroy from gcp-stackdriver-lbm module and now all log-based metrics are being deleted during gcp-stackdriver-monitoring module destruction.
Issue with new log-based metrics and long-living environments: for prd and stg, there can still be a situation, when there is not enough time passing between creation of newly added log-based metric and application of new alert policy based on that metric. This will result in The metric referenced by the provided filter is unknown errors, fixable by retry. I think we can live with that after adding troubleshooting note describing this situation into README.

LGTM
Can we solve both ordering issues by making monitoring depend on lbm via terraform.tfvars?

@mrtyler Unfortunately, even after adding dependency on gcp-stackdriver-lbm, both modules are being destroyed nearly at the same time. I returned destroy null resource back into gcp-stackdriver-lbm and added sleep to prevent race condition with gcp-stackdriver-monitoring module. I think this looks better now.

:(. This sounds like a Terragrunt bug.

@mrtyler Nevermind, looks like I somehow did not add lbm dependency to dev and prd environments.


Allow publish / unpublish per listing

Each listing should have its own publish / unpublish option on "edit listing" page.
Currently only all listings as a whole can be published / unpublished by activating / deactivating store on Settings page.

As OpenBazaar works today, a store that runs out of stock for an item must delete the listing to avoid there being any new orders placed for it.   So this issue reported affects so many OpenBazaar stores.
In the roadmap, under "short term" is "Inventory Management".
<EMAIL_ADDRESS>Perhaps this inventory management method would be how you would unpublish a listing.  Something like an inventory level of 0 would also "unpublish"?   Or perhaps a store would still want an out-of-stock item published -- just that with stock level of zero no buy order can be placed for it.

This is a duplicate of #531


enterFullScreen/exitFullScreen event handler

Hi,
I was wondering if this was something you were considering, currently have a scenario when using video in a overlay in chrome and hit fullscreen it will not expand to fullscreen only the width of the overlay, if we had the ability to trigger overlow:hidden on and off on the overlay based on whether we were in full screen mode or not this would be really useful.
Understand this is a minor but maybe it would be useful to someone else in the future also.  Really enjoying using this plugin,
Cheers,
Clare

Yes, there needs to be events throughout the player (like toolbar resize,
etc.). Please use github.com for feature requests and pulls.
JD
On Mon, Oct 24, 2011 at 6:15 AM, clarehyam <
<EMAIL_ADDRESS>
Hi,
I was wondering if this was something you were considering, currently have
a scenario when using video in a overlay in chrome and hit fullscreen it
will not expand to fullscreen only the width of the overlay, if we had the
ability to trigger overlow:hidden on and off on the overlay based on whether
we were in full screen mode or not this would be really useful.
Understand this is a minor but maybe it would be useful to someone else in
the future also.  Really enjoying using this plugin,
Cheers,
Clare
--
Reply to this email directly or view it on GitHub:
https://github.com/johndyer/mediaelement/issues/282

--
John Dyer - http://j.hn/

There are a few others that I need to build into the player

fullscreenchange
controlsresize
languageselected
srcchanged

Any others?


chore: update argocd to v2.5.3

Thank you for wanting to submit a Pull Request to the Civo Kubernetes Marketplace repository!
If your pull request is to submit a new application to the marketplace, please answer the following questions:

[ ] Have you followed the guidelines in our Contributing document in setting up the application?
[ ] Have you checked that the application your submission is proposing to add does not yet exist on the Marketplace, or as a pull request to be processed?
[ ] Have you tested your application on the Civo managed Kubernetes service? Please include a screenshot.
[ ] Can you confirm that the software you are submitting licensed for use in a publicly-available Kubernetes marketplace?
[ ] Have you included the specific version of the application to be installed?

If your pull request concerns an existing Marketplace application, please make sure you have:

[x] Notified the Maintainer of the application in this pull request so that they are aware of your proposal.
[x] Outlined the changes you are proposing, and the reasons these are required (e.g. stability, compatibility with new versions of Kubernetes implementations, etc).
[x] Tested that the application works with the proposed updates applied (including a screenshot).
[x] Updated the version number of the app if that has changed.

Thank you for the PR, sorry for the delay as I was in Japan and now travelling back.
Also, I am a heavy user of ArgoCD !


import chamfer: import error

Hello，when i run the eval_fig6.py in ubuntu,the import error happened in dist_chamfer.py"import chamfer",the error source is libtorch_cpu.so: cannot open shared object file.But i run the "python setup.py install" successfully and i run the "pip install chamfer" shows that chamfer has exist,so how can i do with the libtorch_cpu.so?

Hello，when i run the eval_fig6.py in ubuntu,the import error happened in dist_chamfer.py"import chamfer",the error source is libtorch_cpu.so: cannot open shared object file.But i run the "python setup.py install" successfully and i run the "pip install chamfer" shows that chamfer has exist,so how can i do with the libtorch_cpu.so?

请问一下你解决这个问题了吗


Unexpected result with single line dual variable declaration

Describe the bug
Unexpected result is observed when multiple variables are declared in single statement and Add final to variable action is triggered.
Source
String username = properties.getUsername(), password = properties.getPassword();

Actual Result
final String username = properties.getUsername();
final String password = properties.getPassword();

Expected Result
final String username = properties.getUsername(), password = properties.getPassword();

What triggered the plugin
CTRL+SHIFT+S.
Versions
IntelliJ IDEA 2019.3 EAP (Ultimate Edition)
Build #IU-193.5096.12, built on November 6, 2019
IntelliJ IDEA EAP User
Expiration date: December 6, 2019
Runtime version: 11.0.4+10-b520.5 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 10 10.0
GC: ParNew, ConcurrentMarkSweep
Memory: 4029M
Cores: 8
Registry:
Non-Bundled Plugins: Struts Assistant, com.codota.csp.intellij, com.dubreuia, com.intellij.ideolog, com.intellij.plugins.html.instantEditing, org.intellij.scala, org.jetbrains.plugins.vue

Hello @kedar-joshi, thank you for your bug report. I won't have time to fix this anytime soon, but PRs are welcomed. You can check the contributing guide in the README. Thanks

Moved to ideas https://github.com/dubreuia/intellij-plugin-save-actions/projects/2


Add uses of Tulipa to docs

Snapshot of changes when building docs locally:

Related issues

Closes #775
Checklist

[X] I am following the contributing guidelines

[X] Tests are passing

[X] Lint workflow is passing

[X] Docs were updated and workflow is passing

Open to feedback! :)

@clizbe thank you! I like it; it has the general overview that we lacked on the docs' landing page.
Although we generally speak as Tulipa, I suggest using TulipaEnergyModel in the documentation since there is another Julia package called Tulipa 🤷♂ so we avoid any confusion or potential quarrel.

Nah I think we can use TulipaEnergyModel for official stuff but can abbreviate to Tulipa in common parlance. And for this it makes it more approachable and easier to read.


Add draggable compatibility for Jazzmin

Added minimal compatibility for Jazzmin theme. This restores the draggable icon to the stacked inline header. There's still no support for collapsing inlines. I might return to this sometime down the road.

Great, thanks! I squashed all your changes and ran pre-commit once. I'm now preparing a release.


Support DiffEq ODE integrators

@ChrisRackauckas pointed out a paper (https://arxiv.org/abs/1912.03253) that uses the Calvo and Sanz-Sara methods in HMC. These integrators are already implemented in DiffEq (https://docs.juliadiffeq.org/latest/solvers/dynamical_solve/#Symplectic-Integrators-1). It would be nice to interface over them in AHMC.

Do you know if any of those correspond to the SHAKE/RATTLE symplectic integrators in the presence of geometric constraints?

They don't. Funnily enough, those are two I haven't gotten around to yet. Do they have a use case here? If they do I can get them done.

I may need them. I'm doing inference of protein structures. I expect due to collisions, my HMC trajectories will be littered with divergences that prevent accurate sampling. SHAKE/RATTLE are used in molecular dynamics to enforce fixed bond length constraints during sampling. I was planning to investigate whether they were appropriate for enforcing boundary constraints, in which case I could use them to sample my models with AdvancedHMC.

Let's call it done then. Also, implicit symplectic integrators would be able to do that too with a singular mass matrix. This would also be an algorithm that we would get for free here:
https://arxiv.org/abs/1911.05754
I'll see if there's any students who would find doing this research interesting. I know the set of benchmarks that one would get for free just by connecting these two libraries is essentially a free paper!

This should be a very big hint for anyone who wants to finish it: https://github.com/TuringLang/AdvancedHMC.jl/pull/146

Thanks so much @ChrisRackauckas!!

It's probably not complete. That PR was somewhat still WIP and needs tests, but it should be pretty close at least.

I will make another PR to add tests and stuff soon.

If there's one thing to check, it's whether I put the right one as the momentum and the state equations. I may have flipped them.

BTW, if you help make sure this is running, I may have a research student in February do some investigation of the different methods in this context.

It works now with #154

Probably can be closed now. Let's try to get a conference paper showcasing these methods 👍 . I might recruit a student to do some testing now. Thanks @IvanYashchuk for pushing it over the line! Let's get in touch over the Slack and get a channel going for paper efforts.

@xukai92 and myself are also looking at alternative integrators for solving Hamiltonian dynamics, particularly those parallelizable, might be interesting to discuss jointly.

I don't think there are very good parallel symplectic ODE solvers out there, but you can use RootedTrees.jl and a global optimizer kind of like I did in https://arxiv.org/abs/1804.04344 to derive one. I was going to do this to derive multi-GPU ODE solvers for first order ODEs, but if you want them, then hey, might be more fun to do this as a collab :).

Thanks for the reference @ChrisRackauckas. Let me create a channel to discuss this further.


Script fails on python 3.10.12

Describe the bug
Basic commands failed.

downloaded repo git clone https://github.com/kremilly/linkscraper.git
pip install as mentioned in readme.md ( pip install -r requirements.txt )
did basic command: python linkscraper -u https://example.com -a get-links
script failed

 _     _       _
| |   (_)_ __ | | _____  ___ _ __ __ _ _ __   ___ _ __
| |   | | '_ \| |/ / __|/ __| '__/ _` | '_ \ / _ \ '__|
| |___| | | | |   <\__ \ (__| | | (_| | |_) |  __/ |
|_____|_|_| |_|_|\_\___/\___|_|  \__,_| .__/ \___|_|
                                      |_|

------------------------------------------------------------
Homepage: https://github.com/Kremilly/linkscraper
------------------------------------------------------------
                v.2.5.1
------------------------------------------------------------
Target: https://example.com
Hostname: example.com
Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/home/dmann/workspace/linkscraper/linkscraper/__main__.py", line 52, in <module>
    Core.home(BASE_URL)
  File "/home/dmann/workspace/linkscraper/linkscraper/core/core.py", line 37, in home
    Layout.print("Scan:", cls.today_datetime(), "italic cyan")
TypeError: Core.today_datetime() takes 0 positional arguments but 1 was given

Screenshots

Desktop (please complete the following information):

windows with WSL ubuntu

Additional context
looks like a cool project, would like to use it to find expired domains outbound links on a website

I fixed it! Thanks for report bug

ok cool, was about to install go for https://github.com/twiny/spidy
but let me give your script another run... 1 sec

works! thanks!

any idea how to use proxies with this script? I guess I could just use proxychains
https://github.com/haad/proxychains

I'm considering adding proxy support, but it's a future consideration!

just noticed that the script does not follow any links?
I would like to scrape an entire domain for all outgoing links to domains that I can buy. ( who is check)
My guess is that i could use the domains for SEO Projects.
Still thanks for your great work and support!

Unfortunately, this script does not currently follow any links.
Thanks for use the script and report the bugs ❤

In the past I've used https://scrapy.org/ which is a python lib that allows you to scrape entire websites
it already has proxy support and many other things


Reconfiguration of vso-agent

Hello,
The agent pool that was assigned to our Mac has been deleted and I'd like to re-configure the agent. When I try and run it at the moment I get: 'Error starting the agent, agent was deleted'
How can I run the configuration process again or do I remove and start again?
Thanks.

delete the .agent file

Thanks!

ill doc today :)


Fixes for ROCM 6.3

retest Ubuntu-GPU-multi please

retest Ubuntu-GPU-single please

retest cpu-pycpp please


fix: dockerhub address

Signed-off-by: sh1luo<EMAIL_ADDRESS>

Looks like something is wrong, let me fix it :(


Error when port 4000 is not available

Description
When port 4000 is already in use I get an error that doesn't direct me to that fact.
Port 4000:
/home/eric/prog/src/ranviermud/bundles/ranvier-telnet/node_modules/ranvier-telnet/index.js:347
      this.emit('error', error);
           ^

TypeError: this.emit is not a function
    at Server.TelnetServer.netServer.on.error (/home/eric/prog/src/ranviermud/bundles/ranvier-telnet/node_modules/ranvier-telnet/index.js:347:12)

Port 4001, on the other hand tells me its in use:
events.js:183         throw er; // Unhandled 'error' event
      ^

Error: listen EADDRINUSE :::4001

Expected outcome
I would expect the telnet port to raise a similar error to the websocket port.
Actual outcome
See description above.
Steps to reproduce

Clone a fresh copy of Ranvier
Have something else up on port 4000 (another copy of ranvier maybe.)
./ranvier

Version of Ranvier
2.0.0 - a new clone of current staging

Looks like this was fixed in ranvier-telnet already: https://github.com/shawncplus/ranvier-telnet/pull/2
An update to the latest version of ranvier-telnet should be enough. Unfortunatly I am unable to update this dependency.


Can't Rename File within Modeling App - Gives Successful Toast

Description
This will happen every now and then, and I haven't figured out the cause just yet, but when I go to rename a file, it says "success" but doesn't rename it.
Version
v0.33.0
Recordings/Screenshots
https://github.com/user-attachments/assets/74a28aa0-706a-4024-8b16-e8bd6735e24b

Sorry @farskid, I fat fingered.


setBottomPadding support

Hi!
I've just added a call for setBottomPadding feature (very useful using tabs UI)
Thanks for the plugin! :)

Found a typo ...


Upload create breaks

Hi,
sorry for creating another issue today. I'm testing the develop branch at the moment. The first 8GB of files have been transferred just fine. Then an error occurs reproducible...

This is a normal jpg file like all the other ones in the same directory.
I have now temporarily removed this directory to see if it breaks again (still running).
Update: Yes, it breaks again without this thumb directory. I've run sftpclone with the parameters -f -t.
Best Regards,
Stefan

It is a Unicode related problem, I'll dig into it. What Python version?

I'm using Python 2.7.9.

I ran it again without -t and -f it broke again. I think the issue above is related to the -t switch as the error looks different now.
Switch -t -f: error above
Switch -t: error above
Switch -f: (currently running)
Switch none: error below

Thanks again for your report.
The last error should be already fixed here: 13ea41971f97e1a9d6fa16b6dff2cf8aaf84d4a9.
Generally speaking, on Python 2 that exception means that an IOError happened (and the FileNotFoundError is indeed aliased to IOError).
I'm investigating the previous one, I'll report back.
Does it consistently break on the same path? And if so, could you please paste it so that I can fake it locally?

Yes and no. Currently it is breaking somewhere else. But reproducable at the same path. Tonight a backup update was running, so the folders might have changed.
Today it breaks here:

You can find the information around the directory structure here:
http://srv.skrauth.de/files/sftpclone_err.txt
This is an archive of all files starting at the misc directory:
http://srv.skrauth.de/files/misc.tar.bz2
Thanks again & best regards,
Stefan

Hmm, I fixed the problem by hammering in utf-8 as default encoding.
import sys
reload(sys)
sys.setdefaultencoding("utf-8")

It seems to work in my case. No crash yet, but obviously a hakish solution.
Reference reading: http://stackoverflow.com/questions/3828723/why-we-need-sys-setdefaultencodingutf-8-in-a-py-script

Yes, I've seen that too. It's a worst case solution, but if I can I'd like to find a more elegant version.
Anyway, it should be noted that the problem disappears by itself in the Python 3 version.

This is fine for me too. Can I instruct setup.py to use python3?
Alternatively: Is it enough to change the shebang in /usr/local/bin/sftpclone to #!/usr/bin/python3?

Ah I figured, just calling it with python3 - very convinient. Maybe just fix it with a note in the readme...?

I almost always use virtualenvs, but for the setup.py file it should be sufficient to launch it with python3 setup.py.
The shebang trick should work too, but in many environments could select an unexpected python version (e.g. in a Python 2 virtualenv it would select the system v3 interpreter).
And yes, if I won't fix it I'll add it to the README.

Nope... python3:

(it's the original error again. I reverted the backup from tonight to verify this reproducability)

What about setting PYTHONIOENCODING=UTF-8 in the virtualenv?

Does PYTHONIOENCODING=UTF-8 fix the problem?
I'm having troubles reproducing it, I've added a few tests (ac9bae83faac5ec43838c63ccb16c0c43e9bd45c) to handle non-ascii characters and they pass.

LC_CTYPE is unset. LANGis currently set toC`.
I have played around with LANG and LC_ALL earlier. I've set both to C, de_DE.UTF-8 and en_US.UTF-8. No change in behavior.
PYTHONIOENCODING=UTF-8 did not fix the problem when I set it in my environment. However you mentioned virtualenv, so my assumption was, that my user environment is not used.

No behavior change with the latest verison.

To be sure about PYTHONIOENCODING=UTF-8, regardless of environment or virtualenv, it should be enough to launch sftpclone as follows:
PYTHONIOENCODING=UTF-8 sftpclone ...

Could you provide a few more info? Sorry about all this :)
$ python # inside the virtualenv and outside, i.e. for both 2 and 3 Python versions
>>> import sys
>>> print(sys.getfilesystemencoding())

I'm analyzing the previous path (root/BACKUP_OLDSERVER/mnt/docroot/http/caro-busch.www/wp-content/uploads/photo-gallery/beauty_fashion/thumb/angela-16.jpg). The error seems to happen on character #115 -.
I've dug through paramiko's files and it only tries to encode the received buffer, so there seems not to be anything on that side.
Are you sure to be using latest version, BTW?
Line #390 from latest commit seems to be different wrt the one indicated in the logs.
Another thing I'm noticing (from here):
        # sftpclone.py:L407
        local_path = join(self.local_path, relative_path, f)  # executed

        # ...

        # sftpclone.py:L407
        remote_path = join(self.remote_path, relative_path, f)  # fails, but `relative_path` and `f` didn't change

I guess your server path doesn't contain non-ascii characters, so it seems strange.

The screenshot was taken from the previous run. Sorry about that. But besides the line numbers, the error didn't change.
Okay, next one...  executing PYTHONIOENCODING=UTF-8 sftpclone -f -t

Well, maybe I should stop my backup. I would have to roll back the backup again to reach to the error from before yesterday.
I'm offline for the next 2 hours. If you want, I can set up a screen sharing session (teamviewer?) and you can look at it on my machine directly? If you can't reproduce it, this is probably easier this way.

For me it is reproducible with the misc.tar.bz2 file I've sent you the link to. I'll try to reproduce it again on another machine when I'm home.

Hi,
I've added print(sys.getfilesystemencoding()) under the imports. The output is:
ANSI_X3.4-1968

The output above helped me to find this article.
Indeed when I set LC_CTYPE to de_DE.UTF-8, sys.getfilesystemencoding() returns "UTF-8".
Unfortunately it still does not fix the error. It still breaks with the same UnicodeDecodeError: 'ascii' error.

OK, so what happens after merging #13?
All is well on Python 2, and what about Python 3?
I'm going to setup some more tests with a structure alike to the misc.tar.bz2 file posted here.

All well in python2. I transferred about 60GB of files with the fix. I'll do another batch of files with python3 later on.
For the test, you only need a subfolder and a file in it with an unicode character. The file creating trouble was "EBG_Elektronik_Sertifika_Hizmet_Sağlayıcısı.pem".

I have to test if the created files actually are okay and if the -t parameter did what it's supposed to do.
I'll report back as soon.

Python3 makes trouble with mixing byte strings and unicode (which works in python2). So it looks like it is more strikt in this regard. Also encoding the filename fixes this. I tried to do as less as possible in the patch. That's why I didn't do it before.

The weird thing is that I already had a test for creating unicode filenames (ac9bae83faac5ec43838c63ccb16c0c43e9bd45c) on the local/remote side and I had no problem. This led me thinking to the environment problem.
Anyway, can you try again after pulling this change? (4fa997316b5c37b19f57bebd94d7f88966bd6cfd)

Oh cool. Yes, this totally works!

Great, perfect. :+1:

Thank you very much! I'll report about my -t test in the other issue.


Rebuild last page

Hi,
first of all thank you very much for your great work on this package.
I'm using your package to improve the performance of my app by getting rid of the unnecessary build calls after several page switches.
Unfortunately, this also has the consequence that the page to which you switch when the back button is pressed is not reloaded.
E.g. if we got the following path: A --> B --> C. If the user goes back from C to B, B is not reloaded.
Is it possible to add a parameter in the configuration of Get() so that only the last page is reloaded if the user presses the back button?

Hi, this is not a side effect. In Flutter, returning from a route does not call initState again, because the previous route is not a new route, it is just a route that is below the current one. This behavior is expected, not only for Get but also for Flutter.
However, there is a lot you can do with Get to work around this.
If you want to give a setState when you return to route B, just put in Get.to(C()) on your page B an await and then give a setState:
onPressed:() async { // you need add async to your method
await Get.to(C());
setState ((){});
}

If you intend to call methods that are in initState, coming back from a route, this is impossible with default Flutter. The Flutter team has already manifested itself by the impossibility of the resource, and in the stackoverflow you will not find an answer to that.
Fortunately with Get's advanced API it makes this possible by using a MiddleWare to perform an action whenever a specific route is in the foreground, you can see an example of this in
https://github.com/jonataslaw/get/issues/31
Closing this because it is standard behavior, but if you have any questions, you can comment here.
If you find a bug, do not hesitate to open another issue.

@jonataslaw you said use setState and refresh that page but it is not working on classes that extends from GetView screens. I found Get.appForceUpdate() but it is refreshing all screens from starting of app.

@jonataslaw you said use setState and refresh that page but it is not working on classes that extends from GetView screens. I found Get.appForceUpdate() but it is refreshing all screens from starting of app.

This issue is from the time of Get 1.0 that had no state management.
Currently you should use an update() if you are using GetBuilder on your screen, or an Obx if the data is observable, and insert this into your controller, instead of your view.

Same issue here. When I go back to the previous page the widgets in that screen won't reload. Because of that If a widget i changed on the 2nd screen won't change when I go back to the previous screen

Hello
Is there a version of onPressed:() async { // you need add async to your method
await Get.to(C());
setState ((){});
} incase im routing within business logic


Implemented compound merge/sort/unique algorithm for optimization

New algorithm does (stable) sort, unique, and inplace merge at once. It may reduce number of scanning count of range.
TODO:

[x] Use other sort algorithm for smaller range instead of merge sort.
[ ] Performance benchmarking.
[ ] Reduce inplace_merge memory footprint (if possible)
[ ] Consider optimizations
[ ] Update documentation (mainly computation order)

It's not able to be faster than naive algorithm combination...


Remove noib3/nvim-compleet

Repo URL: https://github.com/noib3/nvim-completion
Repo is archived since 16th March 23
An issue reads like it is discontinued
Checklist:

[x] The plugin is specifically built for Neovim, or if it's a colorscheme, it supports treesitter syntax.
[x] The lines end with a .. This is to conform to awesome-list linting and requirements.
[x] The title of the pull request is Add/Update/Remove `username/repo`  (notice the backticks around `username/repo`) when adding a new plugin.
[x] The description doesn't mention that it's a Neovim plugin, it's obvious from the rest of the document. No mentions of the word plugin unless it's related to something else.
[x] The description doesn't contain emojis.
[x] Neovim is spelled as Neovim (not nvim, NeoVim or neovim), Vim is spelled as Vim (capitalized), Lua is spelled as Lua (capitalized), Tree-sitter is spelled as Tree-sitter.
[x] Acronyms should be fully capitalized, for example LSP, TS, YAML, etc.

Thanks for the PR!


Get markers by cluster

HI guys.
Is possible, get all markers by cluster?

It`s possible. Just do it:
let clusteresList: Array<any> = this.clusteres.getClusters();
for (let i = 0; i < clusteresList.length; i++) {
        let cluster = clusteresList[i];
        console.log(cluster.markers_)
}


Name only accepts private subnets

Not sure if it's intentional, but the Name fails if I try to add a subnet which does not fall into the <IP_ADDRESS>/8 or <IP_ADDRESS>/16 ranges

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

ID: 6c142eb5-bb83-78ca-ff87-f64cf1c0d99b
Version Independent ID: f91021d2-50f8-fa52-de1e-1f4f169788f8
Content: New-ADReplicationSubnet (activedirectory)
Content Source: docset/winserver2012-ps/activedirectory/New-ADReplicationSubnet.md
Product: w10
Technology: powershell
GitHub Login: @andreabarr
Microsoft Alias: v-anbarr

@audunmg Hello,
Thank you for your feedback.
I have just done a quick test and it seems to work for me.

Could you give the PowerShell command that did not work for you?

Closing the issue as the discussion hasn't developed towards a resolution.

Closing the issue as the discussion hasn't developed towards a resolution.


In responsive design mode (mobile) there is a gray box under the slider

Hi,
I installed and activated the theme.
I followed the instruction and add the folder and 1 image for the slider.
When I resize my window, or see it in the mobile, the div outer-wrapper height is not automatically adjusted, resulting in a gray degrade box below my image.
I know I can easily fix it with custom css, but it seems a bug to me.
Thanks,
Rafael

This only happen when the theme is installed as an addon product. On the other hand, installing the theme as a zip prevents the slider image from being resposnive.

For those experiencing the same problem, I fixed it by commenting the following height property in main.css. Please let me know id there is any problem in what I did.
.item,
.fh5co-owl-text-wrap {
/*  height: 800px;*/
display: table;
width: 100%;
}


Incompatible modules in windows

Steps to reproduce
When you compile in windows, If you use differents modules but in this modules has same variable name, I think cmake mixs variables.
I've opened a case with more details
https://github.com/ScerIO/packages.flutter/issues/490
Expected results
If I set variable in cmakelist.txt, the variable must be isolated
Actual results
When you launch a windows compile with a specific modules than shares variable name, the result will be indeterminate
Code sample
Code sample
Only copy two modules than shares variable name in your pubspec.yaml and launch a windows compilation.
For example:
printing: ^5.11.1
pdfx: ^2.6.0

Screenshots or Video

Screenshots / Video demonstration
Cmakelist.txt from pdfx module set PDFIUM_VERSION to 4634

Cmakelist.txt from printing module set PDFIUM_VERSION to 5200 but If I print value, It's 4634 when I compile it

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.16.9, on Microsoft Windows [Versi¢n 10.0.22621.3155], locale es-ES)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.4.1)
[√] Android Studio (version 2023.1)
[√] VS Code (version 1.86.2)
[√] Connected device (3 available)
[√] Network resources

• No issues found!

That's how CACHE works in CMake; if the package authors don't want the variable to be global and this have the behavior described here, they didn't be using CACHE. If they do want to declare a global variable, it's up to them to pick something unique (e.g., starting with the package name).
Closing as this is up to the audits of packages to resolve, so you'll need to report it to them.


Add youtube channels to the youtube integration

Currently the youtube integration only works for videos since #8 .
Add the preview of channels in js/templates/youtube.js
You should make it so that you find a way to consistantly identify channel links and return the type of a link being channel in the checkLinkType function
You should inject the HTML elements needed to preview a channel when the link is hovered in the bindToNode function (please do not use innerHTML and prefer DOM manipulation for firefox support)

I investigated this for a little bit.
The embedded channel will only show some playlist for a channel.
Also you would need to make an API call with the chanelId to get the channel username to embed  the channel

I looked into that, maybe we could use the page's metadata as it provides the channel description, the channel icon and the channel's name.

Hi
This API can be consumed for the channel name, image, description & subcribers.
_https://developers.google.com/youtube/v3/docs/channels/list?apix_params={"part"%3A"statistics"%2C"id"%3A"UCGMcVCg3tHKmy9oaUCIw8xw"}#usage_
The request parameters include:
part - brandingSettings, statistics
id- (channel id)
You need authorization & API key

Hello @lavina98 thanks for your suggestion, I think we've already discussed the use of the youtube API in #13 but it seem like a pretty bad solution as we would need to share the API Key, or have our own API, and we would have troubles scaling with the ratelimits.

#45


Edit access control

If user is not owner or manager, it should not be able to edit a records or change domain.

this is duplicated by #137

looks good to me 🔥


Add MigrationHandler

Proposed changes
Issue Number: close #10404
Problem Summary:
Add MigrationHandler in FE
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

It seems that the patch move cold data to s3 by creating new tablet on s3, then users can not insert new data to the cold tablet?
We implement the feature by just moving cold rowset to s3, then users can continue insert new data to all tablets.  IMHO, the later solution is more friendly to users. The pr is here https://github.com/apache/doris/pull/10280.
BTW, https://github.com/apache/doris/pull/10280 is now being tested, and our regression tests has passed. We also did performance testing, and I will add performance data later in the pr.


[vcpkg-ci-qt5-quickcontrol2] Add a test port for qt5

This port is just a test for qt5.

@dg0yt All your suggestions are applied. Thanks.
BTW, are you interested in solving #38930, which I once tried to solve without luck.


Added ngcomponentrouter to vendor list

Thanks for the contribution.
Unfortunately it's causing some build warnings:
Version: webpack 1.12.14
Time: 17677ms
                             Asset       Size  Chunks             Chunk Names
    vendor.9caddb72d2ab1a155e93.js    1.79 MB       0  [emitted]  vendor
       app.9caddb72d2ab1a155e93.js    96.1 kB       1  [emitted]  app
vendor.9caddb72d2ab1a155e93.js.map    2.08 MB       0  [emitted]  vendor
   app.9caddb72d2ab1a155e93.js.map     872 kB       1  [emitted]  app
                        index.html  497 bytes          [emitted]
   [0] multi vendor 136 bytes {0} [built]
    + 102 hidden modules

ERROR in ./src/containers/app.ts
(54,5): error TS2322: Type '{ controller: typeof RioApp; controllerAs: string; template: string; $routeConfig: ({ path: strin...' is not assignable to type 'IComponentOptions'.
  Object literal may only specify known properties, and '$routeConfig' does not exist in type 'IComponentOptions'.

ERROR in vendor.9caddb72d2ab1a155e93.js from UglifyJs
Unexpected token: punc ()) [./~/ngcomponentrouter/angular_1_router.js:3093,49]

ERROR in vendor.9caddb72d2ab1a155e93.js from UglifyJs
Unexpected token: punc ()) [./~/ngcomponentrouter/angular_1_router.js:3093,49]

I want to make sure I understand what's happening here before I merge.

On second thought, looks like we need this.  I'll sort out the minify issues separately.  Thanks for your contribution.

It was breaking the initial project when it wasn't there, so I figured it might be just a vendor inclusion. Thanks for merging :+1:


"Git: Unstage Selected Range" does not revert "Git: Stage Selected Range" on the same range

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.69.2
OS Version: Darwin x64 21.6.0
Git Version: 2.37.1

Steps to Reproduce:

Prepare a simple git repo with a file and start VSCode with no extensions

mkdir test
cd test
git init
echo -e "line 1" > file
git add file
git commit -m "commit"
echo -e "line 2\n\nline 4" >> file
code --disable-extensions .

Select line 2 like so

Run "Git: Stage Selected Ranges". Everything should look fine, the range is being staged, like so

With the same selection (pic), run "Git: Unstage Selected Ranges"

The selection is not being unstaged, everything stays as before. I would have expected that the selection is being unstaged

Additional Notes:
This is similar to #120928, but not exactly the same. Because in my case the "Stage Selected Ranges" does work outside of the diff view, just the "Unstaging" does not. Therefore I decided to open a separate issue.

@SimonTheLeg, thank you very much for the detailed repro steps. I have looked into this and while I do agree with you that the experience is not ideal, looking at the code, this seems to be the intended behaviour. The "Stage/Unstage Selected Ranges" commands are always present in the command palette but there are several conditions that have to be fulfilled in order for the command to execute (ex: there has to be an opened document, there has to be a selection, the selection has to contain changes, etc.). If any of these conditions are unmet, the command will not execute. In the case of the "Unstage Selected Ranges", the selection has to be made on the right side of the diff editor (Index), and the selection has to contain changes.
There are couple of things that we could to in order to avoid the confusion:

Hide the command for obvious cases (ex: there is no document opened, no selection)
Display an information notification for other cases (ex: selection does not contain changes)

@joaomoreno, what are your thoughts?

@Iszomorou those sound like massive limitations. I'm coming over from atom, since now it's sunset... :cry: and I could open a single combined diff view and unstage or stage any line I wanted by selecting it, right clicking and selecting "stage/unstage selected" it was a hugely useful workflow and not being able to undelete a section that has been removed because you can only stage/unstage on the "right" side is, to put it frankly, dumb. it's completely unintuitive to anybody coming from a another IDE where things "just work(tm)"

Closing as duplicate of #96850


Support custom naming styles

This is an attempt to support custom naming by loading a python module via a module path and class name. I'm happy to hear feedback and suggestions for alternate approaches. I added a test named namingstyle_custom, which was copied and modified from namingstyle to use fewer delimiters and more camel and pascal casing.
Passing in the styler (fourth commit)
I tried to make this more flexible by also passing a class name but it could be simplified by requiring a specific class name if that's preferred. I first tried to pass the arguments in as a tuple (nargs=2 in optparser.add_option) but that didn't seem to play nice when trying to run the tests or running it as a plugin; it kept getting split. I could wrap it in single quotes but then I'd have to strip those which felt wrong, so I switched to passing a single delimited string. A comma seemed appropriate as the delimiter but that didn't play nice in all cases either, so I switched to a dot since passing the option in the form MODULE.CLASS felt better.
The new --custom-style option is manually checked for being mutually exclusive with the -C/--c-style option. I see that arpparse supports this but optparse doesn't seem to. I could create a separate PR to migrate to using argparse if there is any interest.
The value is split via a callback so the parsing/validation can stay within optparser. It's then interpreted as being a file path, optionally with a ".py" extension. The module name is simply the basename without the extension.
The module is loaded using the approach from an example in the python docs and put into a local class so that NamingStyle is used as the last inherited class (or NamingStyleC if the -C/--c-style flag is also used), allowing the custom styler to only implement the methods they want to modify.
Modification to support more styling (all commits)
When using a custom styler I noticed a few places where I had issues with styling. Note that all tests still pass after these changes and I only saw one difference in the generated code (see below).
The reverse name mapping was affected when there was a package name that was stripped via mangling. It would generate mappings to things that were part way between being properly styled and canonical. To remedy this, I checked if the styled name was different and if the canonical name was different (to match a check that was being used when yielding the mapping definitions. I think this only affects custom stylings but it ends up with a mapping to a mapping. Here's an example from custom_naming_style_mangle.pb.h that is generated from the new test:
/* Mapping from canonical names (mangle_names or overridden package name) */
#define package_MY_ENUM2 MY_ENUM2
#define MY_ENUM2 MyEnum2

If I didn't to the double mapping, generation would fail elsewhere because it would fail to lookup enums from the dependencies.
43140b349631db814588aa8d12252cdc444aaed6 now simply uses the last part of the long names for each enum instead of trying to strip the type name prefix from it, which was assuming there was a single _ as the delimiter. This avoids that assumption.
af7ec95dd18021b8dee09e41ba4e28716c53845a removes a double-styling that was occurring for static bytes types. It was getting styled by calling var_name, which didn't feel quite right to me since it was a typedef, but the underlying ctype already gets styled by calling bytes_type(...) in the Field.__init__ method: https://github.com/nanopb/nanopb/blob/af7ec95dd18021b8dee09e41ba4e28716c53845a/generator/nanopb_generator.py#L746
This happened to work out because the styling was effectively the same, but would be a problem for a custom styler.
daa170bd9694bcf703afccb998bb3aa8b45d4717 styles the extension fields as variables. This only affects the generated output when using the -C/--c-style flag. You can see a diff below of naming_style.pb.c that gets generated from the namingstyle test:
 #define TEST_EXTENSION_TEST_EXTENSION_EXTMSG_CALLBACK NULL
 #define TEST_EXTENSION_TEST_EXTENSION_EXTMSG_DEFAULT NULL
 #define test_extension_test_extension_extmsg_t_test_extension_MSGTYPE test_extension_t
-pb_byte_t TestExtension_testExtension_extmsg_default[] = {0x00};
+pb_byte_t test_extension_test_extension_extmsg_default[] = {0x00};
 PB_BIND(TEST_EXTENSION_TEST_EXTENSION_EXTMSG, test_extension_test_extension_extmsg_t, 2)
 const pb_extension_type_t test_extension_test_extension = {
     NULL,

Thanks, excellent work!


WIP: converge cog and replicate HTTP APIs

Opening this PR solely for visibility at this point. This needs more work before it will be mergeable.

Once #827 is merged and we can rebase this on top of that, there's only one more test that's broken/one more tricky thing to fix before I think we should try and get this reviewed and merged.

This is the first step in making the Cog HTTP API more similar to Replicate's own API.

✨❤ ✨


467 collaborations again

Pull Request Overview

Only approved collaborations ca be requesetd to be upgraded to edit
Collaboration get response updated to reflect whatt is in codex MVP doc
Tests updated accordingly

**REST API Updates **
[GET] /api/v1/collaborations
{
    'guid': 'example', - already existing
    'user_guids': [first user guid, second user guid]
}

/rebase

/rebase

/rebase