Memory Leak

@w-le could you please provide some context here, you had some screenshots or stats?
Possibly introduced in the most recent refactor

1.2205.1 will include GC snapshots as telemetry events. May help with this.

i just noticed it anedotally when running docker stats on placeos-dev instance and noticing that search-ingest container was using an unusually high amount of memory (gigs).  I restarted search-ingest container and it started to use a nominal amount of ram.
currently on placeos-dev with all containers on nightly there does not appear to be an issue
CONTAINER ID   NAME              CPU %     MEM USAGE / LIMIT     MEM %     NET I/O           BLOCK I/O         PIDS
53ac302fc43d   dispatch          0.00%     6.91MiB / 11.69GiB    0.06%     22.8kB / 192B     2.9MB / 0B        6
d872b6e112f4   nginx             0.33%     5.879MiB / 11.69GiB   0.05%     28.4MB / 46.8MB   2.59MB / 0B       3
82e2c15c6ad4   edge              0.37%     37.1MiB / 11.69GiB    0.31%     265kB / 263kB     0B / 0B           6
570a42e65365   staff-api         3.01%     201.3MiB / 11.69GiB   1.68%     40.7MB / 54.9MB   0B / 0B           6
d9bb67d6b018   rest-api          0.56%     89.79MiB / 11.69GiB   0.75%     26.8MB / 17.3MB   0B / 0B           6
6d986b024671   frontend-loader   0.42%     109.4MiB / 11.69GiB   0.91%     63MB / 1.04MB     5.72MB / 36.9MB   6
44252a48418f   search-ingest     0.47%     25.83MiB / 11.69GiB   0.22%     2.14MB / 3.57MB   717kB / 0B        6
5079dd74b0a2   triggers          0.46%     52.91MiB / 11.69GiB   0.44%     932kB / 1.74MB    0B / 0B           6
6f4b4d26113b   auth              0.05%     439.9MiB / 11.69GiB   3.68%     64.5MB / 52.9MB   7.82MB / 0B       193
77c98179af1d   core              0.56%     1.81GiB / 11.69GiB    15.49%    1.18GB / 1.34GB   64.8MB / 549kB    109
4d09529505fa   postgres          0.08%     23.07MiB / 11.69GiB   0.19%     15.1MB / 30.7MB   10MB / 632MB      8
1d45becde7ac   redis             0.67%     39.8MiB / 11.69GiB    0.33%     222MB / 223MB     64.8MB / 301MB    13
5fcf4241ba93   elastic           0.42%     1.271GiB / 11.69GiB   10.87%    66.7MB / 34.6MB   144MB / 157MB     78
a062b2d4b052   rethink           0.60%     166.2MiB / 11.69GiB   1.39%     1.38GB / 1.89GB   115MB / 2.16GB    75
776dc3399a9f   etcd              0.80%     76.97MiB / 11.69GiB   0.64%     1.6GB / 1.64GB    187MB / 168MB     15

So I think we can close this issue right?


ResponseHandler

Hi--
I'm using android studio 3.4.1 / gradle 3.4.2 / fuel-2.2.0.jar
In files FuelJson.kt and FuelSerialization.kts, the statement
import com.github.kittinunf.fuel.core.ResponseHandler   //("Unresolved reference")
... not finding the file / class ResponseHandler in any package. ResponseHandler doesn't seem to exist in the repository here either.  How to resolve these references?
Thanks,
--Marc
The following lines are in my app module level build.gradle
dependencies{
implementation 'com.github.kittinunf.fuel:fuel-android:1.13.0'
implementation 'com.github.kittinunf.fuel:fuel:1.13.0'
implementation 'com.github.kittinunf.fuel:fuel-gson:1.13.0'
implementation 'org.json:json:20180813'
}
and these are in top level build.gradle:
buildscript {
ext.kotlin_version = '1.3.41'
ext.kotlinx_io_version = '0.1.13'
ext.serialization_version = '0.11.1'
ext.coroutines_version = '1.3.0-RC2'
repositories {
    jcenter()
    google()
}
dependencies {
    classpath 'com.android.tools.build:gradle:3.4.2'
    classpath 'com.google.ar.sceneform:plugin:1.11.0'
    classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
    classpath "org.jetbrains.kotlinx:kotlinx-serialization-runtime:$serialization_version"
}

}

It's here: https://github.com/kittinunf/fuel/blob/master/fuel/src/main/kotlin/com/github/kittinunf/fuel/core/Handlers.kt

It looks like @marcspraragen using version 1.13.0 ? Maybe, you should upgrade to version 2?

I guess that should answer your question, I will close this issue for now. But feel free to create new one if you have other issues. Thanks!


Remove "This material may not be published, broadcast, rewritten or redistributed." from body

Fixed & covered by tests.


[1.16] Picking up Ore Meat Fluid voids nbt

Picking up the fluid in a bucket voids the nbt of that fluid..
industrial-foregoing-1.16.5-3.2.11-55cb112.jar

Fixed in https://github.com/InnovativeOnlineIndustries/Industrial-Foregoing/commit/a2f6f88182a97d3b8794e9e8ee87cab66e355344


Add pytest and pytest-benchmark dependencies for example world_count

I think tox automatically installs them :thinking:

Just find it.


Problems validating TLS CA Cert

Hello,
To start, I refreshed the TLS cert on a cluster of v20180320.0, as it was expiring.  When validating that the process was successful, we would see the following:
[me@<some host> ~]$ echo | openssl s_client -showcerts -servername status.<my.cluster.url> -connect status.<my.cluster.url>:443
CONNECTED(00000003)
depth=0 C = US, ST = Texas, L = Austin, O = "<My Company>, Inc.", OU = Flynn, CN = *.<my.cluster.url>
verify error:num=20:unable to get local issuer certificate
verify return:1
depth=0 C = US, ST = Texas, L = Austin, O = "<My Company>, Inc.", OU = Flynn, CN = *.<my.cluster.url>
verify error:num=21:unable to verify the first certificate
verify return:1
---
Certificate chain
 0 s:/C=US/ST=Texas/L=Austin/O=<My Company>, Inc./OU=Flynn/CN=*.<my.cluster.url>
   i:/C=US/O=DigiCert Inc/CN=DigiCert SHA2 Secure Server CA
-----BEGIN CERTIFICATE-----
< REDACTED FOR BREVITY>
-----END CERTIFICATE-----
---
Server certificate
subject=/C=US/ST=Texas/L=Austin/O=<My Company>, Inc./OU=Flynn/CN=*.<my.cluster.url>
issuer=/C=US/O=DigiCert Inc/CN=DigiCert SHA2 Secure Server CA
---
No client certificate CA names sent
Peer signing digest: SHA384
Server Temp Key: ECDH, P-256, 256 bits
---
SSL handshake has read 2298 bytes and written 475 bytes
---
New, TLSv1/SSLv3, Cipher is ECDHE-RSA-AES128-GCM-SHA256
Server public key is 2048 bit
Secure Renegotiation IS supported
Compression: NONE
Expansion: NONE
No ALPN negotiated
SSL-Session:
    Protocol  : TLSv1.2
    Cipher    : ECDHE-RSA-AES128-GCM-SHA256
    Session-ID: 1DF974B3F2E1CE8CE477C755F5B4D8D020CB87EF401836C337754376E8381E48
    Session-ID-ctx:
    Master-Key: <REDACTED>
    Key-Arg   : None
    Krb5 Principal: None
    PSK identity: None
    PSK identity hint: None
    TLS session ticket:
    0000 - d7 89 4e df e5 c5 1f 5d-9f c7 cc e3 2c af 1e e4   ..N....]....,...
    0010 - 61 be 5a 61 71 87 8c db-c3 2f 7f f2 ee c8 cf b7   a.Zaq..../......
    0020 - fa ef 76 0d 96 ac 4a eb-39 7c dd 51 f7 14 f9 b2   ..v...J.9|.Q....
    0030 - a3 d9 de c3 f1 03 ee 93-27 4a dd e7 97 74 ce 2e   ........'J...t..
    0040 - 66 ed 7d f6 02 84 ae ea-2c 43 79 80 5d 1e e5 d3   f.}.....,Cy.]...
    0050 - ee 54 09 e7 9c 9a d7 72-bd a9 bd b5 2c 73 b9 dd   .T.....r....,s..
    0060 - b5 82 73 2d e7 a3 80 c4-bc 72 61 b0 00 71 e3 5f   ..s-.....ra..q._
    0070 - e0 fe 4e 42 36 be c6 3d-                          ..NB6..=

    Start Time:<PHONE_NUMBER>
    Timeout   : 300 (sec)
    Verify return code: 21 (unable to verify the first certificate)
---
DONE

I published the credentials to our s3 bucket, and we started new clusters from scratch.  When trying to provision the new cluster, we got:
+ add_local_flynn_config
+ set +o xtrace
flynn cluster add my-cluster-cluster-timestamp my-cluster.mycompany.com $CONTROLLER_KEY
Error writing CA certificate: Get https://controller.my-cluster.mycompany.com/ca-cert: x509: certificate signed by unknown authority
+ sleep 60

We then created new clusters using v20170719.0, and using the same keys files (we store them in s3 and download them during provisioning), the clusters came up without issue.
Today, I tried to spin up clusters with v20180420.0, and I'm seeing the same issue I see when trying to create a cluster with v20180320.0.
+ add_local_flynn_config
+ set +o xtrace
flynn cluster add my-cluster-cluster-timestamp my-cluster.mycompany.com $CONTROLLER_KEY
Error writing CA certificate: Get https://controller.my-cluster.mycompany.com/ca-cert: x509: certificate signed by unknown authority
+ sleep 60

Note:

Curling https://controller.my-cluster.mycompany.com/ca-cert, and validating the cert using openssl x509 -in ./digicert.crt -text -noout is successful

 8 ➜  ~  openssl x509 -in ./digicert.crt -text -noout
Certificate:
    Data:
        Version: 3 (0x2)
        Serial Number:
            01:fd:a3:eb:6e:ca:75:c8:88:43:8b:72:4b:cf:bc:91
        Signature Algorithm: sha256WithRSAEncryption
        Issuer: C=US, O=DigiCert Inc, OU=www.digicert.com, CN=DigiCert Global Root CA
        Validity
            Not Before: Mar  8 12:00:00 2013 GMT
            Not After : Mar  8 12:00:00 2023 GMT
        Subject: C=US, O=DigiCert Inc, CN=DigiCert SHA2 Secure Server CA
        Subject Public Key Info:
            Public Key Algorithm: rsaEncryption
            RSA Public Key: (2048 bit)
                Modulus (2048 bit):
                    <REDACTED>
                Exponent: 65537 (0x10001)
        X509v3 extensions:
            X509v3 Basic Constraints: critical
                CA:TRUE, pathlen:0
            X509v3 Key Usage: critical
                Digital Signature, Certificate Sign, CRL Sign
            Authority Information Access:
                OCSP - URI:http://ocsp.digicert.com

            X509v3 CRL Distribution Points:
                URI:http://crl3.digicert.com/DigiCertGlobalRootCA.crl
                URI:http://crl4.digicert.com/DigiCertGlobalRootCA.crl

            X509v3 Certificate Policies:
                Policy: X509v3 Any Policy
                  CPS: https://www.digicert.com/CPS

            X509v3 Subject Key Identifier:
                0F:80:61:1C:82:31:61:D5:2F:28:E7:8D:46:38:B4:2C:E1:C6:D9:E2
            X509v3 Authority Key Identifier:
                keyid:03:DE:50:35:56:D1:4C:BB:66:F0:A3:E2:1B:1B:C3:97:B2:3D:D1:55

    Signature Algorithm: sha256WithRSAEncryption
        <REDACTED>
 8 ➜  ~

Cluster with v20170719.0 works, versions v20180320.0 and v20180420.0 do not work.  All clusters are using the same key material stored in an s3 bucket
Cert is issued by DigiCert, so we're using a known good CA

What, if anything has changed w.r.t. cert handling between v20170719.0 and v20180320.0?

Noticed some new nightly builds have been released, and tested v20180501.0, which seems to work.

Flynn is unmaintained and our infrastructure will shut down on June 1, 2021. See the README for details.


[BP-1.10][FLINK-17970] Increase default value of cluster.io-pool.size from #cores to 4 * #cores

Backport of #12360 to release-1.10.

Manually merged.


Fixes #976 - kube-proxy no longer supports command line flags.

Also, it would be nice if you could run the conformance tests in a cluster that contains this change; you could use Heptio Sonobuoy for that.

Codecov Report

Merging #986 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #986   +/-   ##
=======================================
  Coverage   34.84%   34.84%
=======================================
  Files          59       59
  Lines        4130     4130
=======================================
  Hits         1439     1439
  Misses       2532     2532
  Partials      159      159

Continue to review full report at Codecov.

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

https://scanner.heptio.com/0224690663e4b33302e3771e9252101b/  test results from sonobuoy for this PR.  What a neat tool!  thanks for recommending that, another test I can use on my clusters :+1:

/lgtm

@jeremyd Thanks as always for your contribution 👍


libwpe: 1.14.2 -> 1.16.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/libwpe/versions.
meta.description for libwpe is: General-purpose library for WPE WebKit
meta.homepage for libwpe is: https://wpewebkit.org
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.16.0 with grep in /nix/store/spx3wnif3agan6mvd1cwbjwjnj3rrw7v-libwpe-1.16.0
found 1.16.0 in filename of file in /nix/store/spx3wnif3agan6mvd1cwbjwjnj3rrw7v-libwpe-1.16.0

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

445 total rebuild path(s)

445 package rebuild(s)

First fifty rebuilds by attrpath
CHOWTapeModel
ChowKick
abracadabra
adapta-gtk-theme
aegisub
akkoma
alfis
almanah
am2rlauncher
amule
amule-daemon
amule-gui
amule-web
apache-directory-studio
apostrophe
archi
asc
asn1editor
astroid
audacity
aws-workspaces
ayatana-indicator-datetime
badwolf
balsa
bambu-studio
bespokesynth
bespokesynth-with-vst2
birdfont
bochs
boinc
bookworm
bossa
bossa-arduino
bubblemail
budgie.budgie-control-center
calls
cambalache
cargo-tauri
cemu
chatty
chirp
cinnamon.cinnamon-common
cinnamon.cinnamon-control-center
cinnamon.cinnamon-gsettings-overrides
cinnamon.cinnamon-screensaver
cinnamon.pix
cinnamon.xreader
cinny-desktop
citrix_workspace
citrix_workspace_23_09_0

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/spx3wnif3agan6mvd1cwbjwjnj3rrw7v-libwpe-1.16.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/9d4514f7a6fcdae7827a9bce6dc342f0c9c485f1#libwpe

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/spx3wnif3agan6mvd1cwbjwjnj3rrw7v-libwpe-1.16.0
ls -la /nix/store/spx3wnif3agan6mvd1cwbjwjnj3rrw7v-libwpe-1.16.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 45m and timed out

Maintainer pings
cc @bobby285271 @hedning @jtojnar @dasj19 @amaxine @matthewbauer for testing.

@ofborg eval

I guess we can drop webkitgtk.meta.maintainers now.


fixed overflow for unrender overlay

I fixed adding style to element body from overflow: scroll to overflow: auto when overlay is hidding. Now it doesn't add inactive scrollbars to pages with content fitted in window.
Second commit fixed colors in button accent-contrast style.

PR #470 addresses the same issue, but instead checks the previous value of property and re-adds it, or remove the property entirely if there wasn't a overflow to start with.
Hasn't been merged. So maybe @javivelasco wants to just got with a more straight forward solution as presented here.

Guys, thanks for the PRs. I thought I've mention a solution already but I forgot about pushing it. It here. It's solving the same thing so I'm closing!


Refactor JVM apps to use shared components

Left the processors as a separate parameter because I'll be adding extra ones for kafka to filter out existing unneeded metrics, so I'm just preemptively having that enhancement in the shared component.

The centos8 failure from ops agent integration test seems unrelated:
#100 [centos8-build 2/7] RUN set -x; yum -y update &&     dnf -y install 'dnf-command(config-manager)' &&     yum config-manager --set-enabled powertools &&     yum -y install git systemd     autoconf libtool libcurl-devel libtool-ltdl-devel openssl-devel yajl-devel     gcc gcc-c++ make cmake bison flex file systemd-devel zlib-devel gtest-devel rpm-build systemd-rpm-macros java-11-openjdk-devel     expect rpm-sign
#100 sha256:5ff878204840f6739beba3b9bc9b865f18d0f703c3ada36fe144f6a18fe7d190
#100 22.21 Package systemd-239-51.el8_5.3.x86_64 is already installed.
#100 22.22 Package systemd-239-51.el8_5.3.x86_64 is already installed.
#100 22.23 Error:
#100 22.23  Problem: cannot install the best candidate for the job
#100 22.23   - nothing provides rpm = 4.14.3-19.el8_5.2 needed by rpm-build-4.14.3-19.el8_5.2.x86_64
#100 22.23 (try to add '--skip-broken' to skip uninstallable packages or '--nobest' to use not only best candidate packages)


Daily reminders firing at wrong time

Firing at midnight, not at scheduled time

Actually, it's firing at midnight and at the desired time :-/

SMS messages are displaying the wrong time, too (not compensating for timezone). Emails are double-firing, but content is time-correct.


New actions: view next|prev

Add actions to view the next tag(s) to the right or the left side. Bind them to C-PageUp/PageDown, by default.
Examples:

If tag 1 is selected, view next shows all windows on tag 2 (same as view 2).
If tag 2 is selected, view next shows all windows on tag 3 (same as view 3).
If the last tag is selected, view next shows all windows on tag 1 (same as view 1).
If tags 2 and 4 are selected, view next shows all windows on tags 3 and 5 (same as view 3 + viewtoggle 5).
Analogous for view prev: If tag 2 is selected,  view prev shows all windows on tag 1 (same as view 1).

This sounds potentially useful, thanks! I'll think some more about it. Sorry I've been silent, busy with a couple of other things. I'll get to this!


FINERACT-1102: Swagger Codegen Client JAR should be available on a Ma…

Description
The following things need to be solved before the publishing can work:

It's probably best to create an account at https://bintray.com. They make publishing on Maven Central so much easier.
We need to upload a GPG signing key to Bintray once we have an account.
We will also need a user token from Sonatype to be able to publish to Maven Central. Details see here: https://stackoverflow.com/questions/38294001/how-to-sync-from-bintray-to-maven-central-without-knowing-maven
We have to decide if we want to list any developers in the generated pom.xml that gets published. There is a TODO in the root build.gradle
I've created the Maven publish configuration already in a way so that it can be applied to multiple modules without major edits. Note: next one to publish will be fineract-api.
There's a sample configuration to show how publishing on Github Packages would work. This just FYI and will be removed once we are done here. I think Maven Central is the best choice.

Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[x] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[x] Create/update unit or integration tests for verifying the changes made.

[x] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[x] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/api-docs/apiLive.htm with details of any API changes

[x] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

@vidakovic as in the FINERACT-1102 discussion we are now converging towards using ASF's Nexus, I would LGTM and merge this only if you remove the GitHub Packages and BinTray lines from the build.gradle. I fear it could create more confusion (Noob Contributor: "is this project deploying to GitHub Packages and BinTray? But how?") than add value to merge this as-is. And inline TODO FINERACT-1102: Actually use this to deploy to ASF (Apache Software Foundation) Nexus Maven Repository also couldn't hurt. But if this PR is "just" WIP from you that you don't want to see merged just yet, please mark it as Draft for now.

@vorburger I've removed any references to Github Packages and Bintray. Todo as you suggested placed instead. I'd say let's merge (it won't hurt anyone; this task is similar to release, just for a very small subset of Fineract devs will use it) and I'll circle back to this when I've published an artifact successfully. BTW: was able to login on Apache's Nexus admin app; as soon as I have something working I'll send a new PR with updates to the README.


GTFS indicates 0 if no more departure today

Home Assistant release (hass --version):
0.46 (Docker image)
Python release (python3 --version):
I don't know, the one bundled in the Docker image
Component/platform:
sensor.gtfs
Description of problem:
When the next departure at a given stop is scheduled for tomorrow, the GTFS sensor indicates 0. At midnight, it switches correctly to the delay until the next departure.
Expected:
If no more departures found today, the sensor should look at the next day(s).

This issue is still relevant.

The issue is still relevant.

Curious if this has been fixed.

Verify before closing

This is a year and a half old at this point. I'm going to close it. Please create a new issue if there is still an issue as of 0.84.


Dynamic mounted directory

Both Java and Go folks have shown that this would be useful for their platforms. Thinking a dynamic key in the platform object like mountDir():
mountDir: function(cwd) {
  return '/app/src'; // for Node
}

To be used instead of the magic dir here: https://github.com/heroku/heroku-docker/blob/7c6f7efb5d708b64ac6d1508983f8b93f6ed166d/lib/docker.js#L52
/cc @freeformz @jkutner

That works for me as I'd be able to lookup what I need in the Godeps/Godeps.json file and DTRT.


feat: enable tree shaking in rust by default

Description

Test Plan

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#501  👈
#497
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @hyf0 and the rest of your teammates on  Graphite

Merge activity

Mar 9, 5:40 AM EST: @hyf0 started a stack merge that includes this pull request via Graphite.


cell tip parentage

should cell tip be is_a cell pole? The comments don't suggest so but it looks like it would fit from the defs?

The commet does suggest it?
Cell tip
Comments
Note that this term differs from 'cell pole ; GO:0060187' in that it is applicable to the thin ends of elongated cells, such as the ends of axons or dendrites.
it sounds the same but narrower?

I'm not sure. Round cells like eggs can have 'poles'. AFAIK they are defined along an linear axis (like the north and south poles of the earth). I'm not sure tips follow this rule. I don't think anyone would consider the vegetal pole of an egg to be a tip.

OK, we should  just make sure we don't use pole/ We only used it once out of  232 annotations.

When I say we, it was I whoops, now fixed.


Add exolix

Description

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

@devexolix sorry this looks like it's rendering an iframe which is not allowed:
https://github.com/pillarwallet/x/pull/173/files#diff-5a58296effc18582ddc8fca3b692f072fbed94a29ab969790e6845fc4dae82dfR93

@devexolix sorry this looks like it's rendering an iframe which is not allowed: https://github.com/pillarwallet/x/pull/173/files#diff-5a58296effc18582ddc8fca3b692f072fbed94a29ab969790e6845fc4dae82dfR93

Yes, we use iframe, but it links to your repository in folder "/public/exolix-static" and you have all the source code and we haven't plugged in any outside code.

@IAmKio Yes, we pack build our site and add to app, and this is why we need use the iframe because the build version need render in HTML tag and have the own DOM.


SharePoint webparts embedding authenticated PowerBi reports combined with URL query parameters

Core Library
MSAL.js v1 (@azure/msal or msal)
Core Library Version
1.4.17
Wrapper Library
Not Applicable
Wrapper Library Version

Public or Confidential Client?
Public
Description
The Customer NGO SOS Chidlren's Villages has the requirement to send to coworkers link-URLs in Emails, with query parameters.
Those required query parameters are used to point in the developed single page app to a specific data display, showing data to collaborate on. The app is embedded in SharePoint Webparts and loads openStreetMap data, PNP Search Webparts - Search Results and PowerBi Reports.
The PowerBi Reports do require authentication.
I followed samples using package "msal" and PowerBi embedding .
The sample leads to UserAgentApplication  - getAccount() , after that call every I have is lost due a authentication redirect.
I am so far not able to preserve the original URL query parameter nor a react states/variables.
(on the other hand if start the app in SharePoint without of any query parameter in the hierarchical data top level and work with it, the PowerBi embedding works very fine.)
I do already cut away and memorize the initial URL query parameters in order that the redirection has no problems, when authenticated and returning to application page.
My question is how to get URL query parameters or a React state/variable preserved after a redirect?
UserAgentApplication  - getAccount()  is that a bad way.
(Is msal-node msal-react needed in typescript to achieve it?)
MSAL Configuration
scopes: string[] = ["https://analysis.windows.net/powerbi/api/Report.Read.All"];

Relevant Code Snippets
import { UserAgentApplication, AuthError, AuthResponse } from "msal";
---------------
    const msalInstance: UserAgentApplication = new UserAgentApplication( msalConfig );
---------------

    if ( msalInstance.getAccount()) {
      msalInstance
        .acquireTokenSilent(loginRequest)
        .then((response: AuthResponse) => {

Identity Provider
Azure AD / MSA
Source
External (Customer)

@Georg-Trixl I'm a little confused, the getAccount API does not invoke a redirect. Are you perhaps calling loginRedirect or acquireTokenRedirect somewhere? If so, you can preserve the query parameters through the redirect by providing them to the state parameter which will be returned to you in the AuthenticationResult object upon returning from the redirect and calling handleRedirectCallback

So i figured it out on my own, in SPFX the state and extraQueryParameters are also not usefull.
The in SPFX, PNPJS and also MSAL-React integrated authentication, things are not helping to embed a PowerBI-Report into a SharePoint - when a developer is required to put PowerBI into a SharePoint Webpart without of an pre generated embedding Link.


chore: add updated template for the railway

Changes

Replace the template with an updated version.

Template: https://railway.app/template/fcEVay

PR-Codex overview
The focus of this PR is to update the deployment link in the README.md file to point to a new Railway deployment option.
Detailed summary

Updated the deployment link in the README.md file to point to a new Railway deployment option.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

I removed the referral code as when you share the Railway markdown version the referral code is enabled by default.

Abort, will unpublish the template, the original creator wants to keep the template.

Hey, @arcoraven, can you review this?
As discussed here, the original template creator offers to unpublish the original template so we can use the template we made for the Railway. Our new template follows the standard and recommended deployment steps for the engine, ensuring every deployment succeeds.
Moving forward, I'll provide technical support to the railway community to ensure that any engine deployment will work as expected and maintain a 100% success rate.

Let's merge this @farhanW3 or @arcoraven so people can use the official community template.


Collect connection pool statistics periodically

This adds a new ability to ContextFactory objects that hooks them into
the server runtime metrics system and asks them to periodically produce
metrics.
The SQLAlchemy and Thrift context clients are now instrumented to
periodically report their connection pool statistics to give more
insight into what's going on in there.
For some simple context clients like this:
baseplate.add_to_context(
    "downstream", ThriftContextFactory(thrift_pool, BaseplateService.Client))
baseplate.add_to_context("db", SQLAlchemyEngineContextFactory(engine))

The following metrics might be sent:
example_server.runtime.hostname.PID13814.clients.downstream.size:10|g
example_server.runtime.hostname.PID13814.clients.downstream.checkedout:0|g
example_server.runtime.hostname.PID13814.clients.db.size:5|g
example_server.runtime.hostname.PID13814.clients.db.checkedin:0|g
example_server.runtime.hostname.PID13814.clients.db.overflow:0|g
example_server.runtime.hostname.PID13814.clients.db.checkedout:0|g

This starts to fix #150.

Related thought I had while looking at this: I wonder if it'd be possible to have a gauge metric which increments whenever we try to make a query and the pool is exhausted?

Related thought I had while looking at this, doesn't need to be solved here: I wonder if it'd be possible to have a counter metric which increments whenever we try to make a query and the pool is exhausted?

Yeah, I was thinking of putting a timer around the block for an empty slot but currently the pool is "below" that level and has no concept of request or trace context so it'd be hard to get at that. Will need to play with some reworkings for that to make sense.

:haircut_man:

One thought I had, would there be issues with this sending too many metrics or having too high a path cardinality given the ${hostname}.PID${pid} naming scheme?
CC: @ckwang8128

That concern already exists for the other runtime metrics. We're doing aggregation in telegraf before it gets to the stats backend.

Ah, I wasn't aware of those, great! 🐟


Remove ugly scrollbars from the page

Hi @abesto, I am new to contributing to open source, can I pick this issue?

@anuj98 All yours, thank you for giving this a go!

@anuj98 All yours, thank you for giving this a go!

I cloned the repo and have python and node installed. But couldn't run the following commands. I have a windows system. Could you please help me?

Please paste the full command and the full output. I don't develop on Windows, but let's see what we can find.

Not able to create YOUR_FAVOURITE_PROJECT variable on the command prompt/powershell

Try replacing $YOUR_FAVORITE_PROJECT with click
so the full command to run is
py-wtf index --project-name click www/public/_index

Ah yes. A=b is syntax that Linux/macOS shells support, but not PowerShell. As @zsol says, your life is likely to be easiest by just directly putting in the name of the project instead of using a variable here.

This is the next issue:

That looks like you missed (or ran in another shell?) the pipx run hatch shell command, which would set you up with an environment where py-wtf is available.
If that doesn't seem to work, you can pipx run hatch run py-wtf index etc etc

Thanks @zsol and @abesto it worked.

Hi,
I feel like I might still be missing something because I am not able to see any data? I know the fix but unless the inside components don't render won't be able to verify everything is working with that fix.

Yeah there should be a click.json file that's generated by py-wtf index --project-name click www/public/_index. Unless! You were already in the www directory? In which case you can run either:

py-wtf index --project-name click public/_index
py-wtf generate-test-index

@zsol Its not generating that file.
If I directly run py-wtf index --project-name click public/_index or py-wtf generate-test-index it gives the same error I got previously.

So I tried the solution pipx run hatch run py-wtf index.... then the command runs but don't see the file created.
I'll try to find a fix but cannot guarantee anything. Do let me know if I can try anything else to fix this issue.
Thanks

@anuj98 Well, unless you specifically care about understanding / debugging / working on the indexer, you could grab the live JSONs and plop them in the right place, e.g.: https://py.wtf/_index/click.json
For debugging: what's the output of pipx run hatch run ...? What does git status say?

you have py_wtf in your command line instead of py-wtf
You can also try running pipx run hatch run py-wtf generate-test-index

Raised a PR. Please take a look.


version 2

Hello!
Just asking if you (@flugger) have any estimates when the new version will be available? :)

Hey Hese! I've had my focus on the package for the past week and getting very close to a release. I ended up rewriting a few parts of the package again as I'm only satisfied when I reach perfection :) So, unfortunately, it has taken longer to finalize version 2 than first anticipated. However, I hope it's going to be worth the wait, because it's packed with new features and the performance of the package has drastically improved (in fact, in some cases, it's even faster than using Fractal alone due to a new caching layer for the transformers). Version 2 will also introduce a complete rewrite of the documentation and a changelog so the upgrade path will be easy.
I don't have an exact release date, but I'm keen on releasing it before July, so I can enjoy the summer. Thanks for being patient!

Awwwwwwesome!

Aaaaaaaaaand it's out! 🎉 Sorry for the delay - hopefully, you won't be disappointed though!


Not updating properly for device rotation

If you rotate the device while making the soup UI not updates properly.

@madnik Thank you for interest. Yes, it has such problem with rotation and we will give attention to fixing this bug.

The pull to refresh is too high and on iPhone 4, 5 it can't be triggered. That's the reason why we don't support landscape orientation


Remove NOT NULL constraints on non-id columns

There were NOT NULL constraints on columns that are either NULLable, or are often provided in imports without values.
The "processing-migrations" migrations don't need a separate migration to remove the constraint and can be edited in place because they are run on a fresh database for each import.
Pivotal bug 99134060

LGTM.


Needed to make two small changes that the article does not mention.

You need to change the line "public class SplashActivity : Activity" to "public class SplashActivity : AppCompatActivity" in the SplashActivity.cs file.
You need to add the line "<color name="splash_background">#4040FF</color>" in the colors.xml file. (For a nice blue color. You can pick your own.)
Other that that it worked great. Thanks Harry...

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

ID: 5823a3fa-87be-0b27-155b-b44130c67e84
Version Independent ID: c90633f5-22b9-f7dd-fd08-c9affc251dd4
Content: Splash Screen - Xamarin
Content Source: docs/android/user-interface/splash-screen.md
Product: xamarin
GitHub Login: @conceptdev
Microsoft Alias: crdun

This change appears to have been applied but this comment was not closed. Closing now.


Kiali unable to acquire traces

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

Deploy Rancher (I used 2.5.3)
Deploy Rancher Monitoring (I used 9.4.202)
Deploy Istio 1.7.301 <- Version required for tracing integration
Deploy Bookinfo
Open Kiali
Navigate to Services -> $SERVICE -> Tracing

Result:
Kiali will claim that it cannot collect traces.

Other details that may be helpful:
This happens because the jaeger configuration sets QUERY_BASE_PATH to /jaeger but the in_cluster_url is set to the root: 'http://tracing.istio-system.svc:16686'
Fixes:

Change in_cluster_url to http://tracing.istio-system.svc:16686/jaeger
Change QUERY_BASE_PATH to /

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): v2.5.3
Installation option (single install/HA): HA

Cluster information

Kubernetes version (use kubectl version): 1.19

Issue is seen on 2.5.4 as well:
Rancher | v2.5.4
User Interface | v2.5.4
Helm | v2.16.8-rancher1
Machine | v0.15.0-rancher49
Followed the same steps as mentioned above:
Deploy monitoring and istio with jaeger tracing enabled as standard user on single node installation and deploy the bookinfo workload.
In kiali --> services --> default namespace --> traces and inbound metrics have the same error:

Issue is seen on 2.5.4 as well:
Rancher | v2.5.4
User Interface | v2.5.4
Helm | v2.16.8-rancher1
Machine | v0.15.0-rancher49
Followed the same steps as mentioned above:
Deploy monitoring and istio with jaeger tracing enabled as standard user on single node installation and deploy the bookinfo workload.
In kiali --> services --> default namespace --> traces and inbound metrics have the same error:

I'm seeing the same on v2.5.3. Also seeing (possibly related) issue with collecting metrics from prometheus. I have the monitoring app installed in it's default location.
Can confirm the above changes to the in_cluster_url and QUERY_BASE_PATH fix the trace access from Kiali

I'm seeing the same on v2.5.3. Also seeing (possibly related) issue with collecting metrics from prometheus. I have the monitoring app installed in it's default location.
Can confirm the above changes to the in_cluster_url and QUERY_BASE_PATH fix the trace access from Kiali

rancher/rancher:v2.5.6-rc4
rancher-istio:v1.8.100-rc01

Install rancher-monitoring
Install Istio w/ Kiali enabled
Deploy demo bookservice
Kiali is installed and accessible
Traffic graph, metrics, and traces show correctly
Values yaml shows the correct values:

..
tracing:
      in_cluster_url: 'http://tracing.istio-system.svc:16686/jaeger'
..
tracing:
  contextPath: /jaeger


[QA] unprivileged users can promote themselves to admin

Setup via docker-compose-eos-test.yml at branch fix-yml-for-rc5 on localhost

eos space set default on for #862
Login user marie
Click the 9 dots menu, select accounts.
Note that marie is Role User.
Try '+Create new account' -> it fails. Good.
Switch Role from User to Admin.

Try '+Create new account' again.

Oops.
Expected behavior: normal user cannot.

We recently introduced a SelfManagement permission to the Userrole. As a result the accounts UI shows the logged in user. This made it possible here, to uncover that we in fact don't have any permission checks for assigning/unassigning roles. Needs to be implemented ASAP. Might make sense to move the role assignments to the account service in the same PR.

@kulmann, what is the reason for moving the role assignments to the accounts service?
I understand that right now the accounts UI is depending on the settings service but when we move the assignments to the accounts service then the accounts backend will still depend on the settings service.

@C0rby

it would save us a request in the account_resolve middleware in the proxy, because we could return the role assignments directly from the account service instead of having to make a second request to the settings services.
when creating users we assign a role. that could happen in the same service then, so it reduces latency.

Ah you mean we would even move the store to the accounts service?

Confirmed fixed in rc6
einstein can still change his role to admin in the web UI, but it has no effect. On a reload his role is shown as 'user' again. OK.


Latest release (4.4.1 version) of saml2-web-app-pickup-dispatch and saml2-web-app-pickup-manager webapps give 404 not found error when deployed

Describe the issue:

Latest release (4.4.1 version) of saml2-web-app-pickup-dispatch and saml2-web-app-pickup-manager webapps give 404 not found error when deployed.
How to reproduce:

Download the 4.4.1 version of saml2-web-app-pickup-dispatch and saml2-web-app-pickup-manager webapps.
Deploy the webapps and restart the Tomcat server.
Goto http://localhost.com:8080/saml2-web-app-pickup-manager.com/home.jsp or http://localhost.com:8080/saml2-web-app-pickup-dispatch.com/index.jsp

Expected behavior:

Users should be directed to the landing page of the sample application, but instead it gives 404 not found exception.

Environment information:

Product Version: IS 6.0.0, IS 5.11.0
OS: Mac
Tomcat Version: 8

It seems that the .com part has been removed from the folder name. @YogyaGamage can you try changing the webapp configs sso.properties file can be found in webapp accordingly? https://github.com/wso2/samples-is/blob/master/sso-samples/saml2-sso-sample/saml2-web-app-pickup-dispatch/src/main/resources/sso.properties#L35-L41

It seems that the .com part has been removed from the folder name. @YogyaGamage can you try changing the webapp configs sso.properties file can be found in webapp accordingly? https://github.com/wso2/samples-is/blob/master/sso-samples/saml2-sso-sample/saml2-web-app-pickup-dispatch/src/main/resources/sso.properties#L35-L41

It works when .com part is manually added to the folder name.


caffe and caffe2 not co-installable

Consider relocating caffe.proto to make caffe and caffe2 co-installable. Presently, it is in caffe/ which makes caffe2 clobber the caffe namespace with both its include directory and its python module. The symptom of this is https://github.com/caffe2/caffe2/issues/342, which has been closed but is still unresolved.
Relocating caffe.proto to either caffe2/proto or a new directory like caffe2/legacy would solve this problem. They still wouldn't be able to be used in the same process because of the protobuf conflict, but it would simplify being able to use both separately.
Happy to submit a PR for this if a location can be decided.

Resolved in https://github.com/pytorch/pytorch/pull/10380


Typo in demo

In the readme
Usage -> Query -> .catch(...) is missing an opening parentheses.

Thanks, we are correct it!


Thompson sampling punchline

Version: March 1, 2021, PDF page 286, book page 256.
Hey Kevin, Lihong:
In the short Thompson sampling section, you say you're using a single Monte Carlo sample as a very coarse approximation to the sampling distribution you want, and then sampling from it.  True, but I think that buries the lede.  The cool part is that sampling in this way gives you exactly the distribution you want.

Or maybe you want to leave that for an exercise...

yes, that's a good point. I've updated the text.
On Thu, Mar 4, 2021 at 7:26 PM David S. Rosenberg<EMAIL_ADDRESS>wrote:

Version: March 1, 2021, PDF page 286, book page 256.
Hey Kevin, Lihong:
In the short Thompson sampling section, you say you're using a single
Monte Carlo sample as a very coarse approximation to the sampling
distribution you want, and then sampling from it. True, but I think that
buries the lede. The cool part is that sampling in this way gives you
exactly the distribution you want.
[image: image]
https://user-images.githubusercontent.com/1561584/110062966-45587e00-7d38-11eb-9857-6a4c60c4a547.png
Or maybe you want to leave that for an exercise...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/probml/pml-book/issues/53, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDK6EE6MMTJZK4IG4X46CDTCBFMZANCNFSM4YUODQEA
.

(Notationally it's a bit unclear what happened to \tilde{theta}_t in line 2

perhaps the argument could be made more explicit by conditioning on  the
event theta=\tilde{theta}_t?)

On Sat, Mar 6, 2021 at 5:37 PM Kevin Murphy<EMAIL_ADDRESS>wrote:

yes, that's a good point. I've updated the text.
On Thu, Mar 4, 2021 at 7:26 PM David S. Rosenberg <
<EMAIL_ADDRESS>wrote:

Version: March 1, 2021, PDF page 286, book page 256.
Hey Kevin, Lihong:
In the short Thompson sampling section, you say you're using a single
Monte Carlo sample as a very coarse approximation to the sampling
distribution you want, and then sampling from it. True, but I think that
buries the lede. The cool part is that sampling in this way gives you
exactly the distribution you want.
[image: image]
https://user-images.githubusercontent.com/1561584/110062966-45587e00-7d38-11eb-9857-6a4c60c4a547.png
Or maybe you want to leave that for an exercise...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/probml/pml-book/issues/53, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDK6EE6MMTJZK4IG4X46CDTCBFMZANCNFSM4YUODQEA
.

What about making the variable of integration \tilde{theta}_t ?

Yes, I like the idea of making the variable of integration be \tilde{theta}_t . thanks for the suggestion.


docsite: make warnings failures

I mistyped paths:

first I used ../deploy/... in examples (relative path instead of absolute path),
then I misspelled a path (customize instead of kustomize):

/mnt/workspace/pmem-csi_PR-633/examples/memcached.md:44: WARNING: None:any reference target not found: /deploy/customize/memcached/ephemeral

It's easy to miss these warnings. Can we turn them into errors which case "make vhtml" to fail?
@intelkevinputnam

I use these switches to the sphinx build:  -W --keep-going
This will error on warnings, but it will also continue to build. That way we can catch all of them in one go if there are multiple warnings.


Use new tags slice for every service.

append in golang do not create new slice if appended entries will fit into slice capacity. This causes an issues when port tags were added and global tags count was power of two. Minimal example presenting this error is below:
package main

import (
	"fmt"
)

func a() {
	x := []int{0, 1}
	y := append(x, 2)
	z := append(x, 3)
	fmt.Println(y, z)

}

func b() {
	x := []int{0, 1}
	x = append(x, 2)
	y := append(x, 3)
	z := append(x, 4)
	fmt.Println(y, z)

}

func main() {
	a()
	b()
}

https://play.golang.org/p/qv520b4MnM

Coverage decreased (-0.006%) to 89.368% when pulling dcc00bce87cd852ecfd01a925fa8fe2a1dbdf067 on hotfix/slice_append_magic into e36ce2dc17344704e6afb217f5f103c83c585264 on master.

Here is a blog post with motivation for this pull request (if tests are not enoguht) http://allegro.tech/2017/07/golang-slices-gotcha.html

I come here thanks to the blog post. Above example would be easier to grapsh if we add 2 new lines, as:
package main

import (
	"fmt"
)

func a() {
	x := []int{0, 1}
	y := append(x, 2)
	z := append(x, 3)
	fmt.Println(y, z)

}

func b() {
	x := []int{0, 1}
	fmt.Println("cap(x) before:", cap(x))
	x = append(x, 2)
	fmt.Println("cap(x) after:", cap(x))
	y := append(x, 3)
	z := append(x, 4)
	fmt.Println(y, z)

}

func main() {
	a()
	b()
}

https://play.golang.org/p/RSEhh0COf8H


Cannot install node-sass 4.9.0 using latest node (v10.4.0)

NPM version (npm -v): 6.1.0
Node version (node -v): v10.4.0
Node Process (node -p process.versions):

{ http_parser: '2.8.0',
  node: '10.4.0',
  v8: '6.7.288.43-node.7',
  uv: '1.20.3',
  zlib: '1.2.11',
  ares: '1.14.0',
  modules: '64',
  nghttp2: '1.29.0',
  napi: '3',
  openssl: '1.1.0h',
  icu: '61.1',
  unicode: '10.0',
  cldr: '33.0',
  tz: '2018c' }

Node Platform (node -p process.platform): darwin
Node architecture (node -p process.arch): x64
node-sass version (node -p "require('node-sass').info"):

internal/modules/cjs/loader.js:596
    throw err;

npm node-sass versions (npm ls node-sass):

<EMAIL_ADDRESS>/Users/username/git/app-name
└── (empty)

After updating to node to v10.4.0 from 9 (using the latest OSX High Sierra) I got this error message
Error: Node Sass does not yet support your current environment: OS X 64-bit with Unsupported runtime (64)

So I decided to update node-sass to the latest version too in my project.
I first deleted completely the node_modules folder and running yarn cache clean then tried to install the latest version using yarn (v1.7.0)
yarn add<EMAIL_ADDRESS>--dev

Unfortunately I get a long output beginning like this
warning Error running install script for optional dependency: "/Users/username/git/app-name/node_modules/fsevents: Command failed.
Exit code: 1
Command: node install
Arguments:
Directory: /Users/username/git/app-name/node_modules/fsevents
Output:
node-pre-gyp http GET https://fsevents-binaries.s3-us-west-2.amazonaws.com/v1.1.3/fse-v1.1.3-node-v64-darwin-x64.tar.gz
node-pre-gyp http 404 https://fsevents-binaries.s3-us-west-2.amazonaws.com/v1.1.3/fse-v1.1.3-node-v64-darwin-x64.tar.gz
node-pre-gyp ERR! Tried to download(404): https://fsevents-binaries.s3-us-west-2.amazonaws.com/v1.1.3/fse-v1.1.3-node-v64-darwin-x64.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v64 ABI, unknown) (falling back to source compile with node-gyp)
node-pre-gyp http 404 status code downloading tarball https://fsevents-binaries.s3-us-west-2.amazonaws.com/v1.1.3/fse-v1.1.3-node-v64-darwin-x64.tar.gz
node-pre-gyp ERR! Tried to download(undefined): https://fsevents-binaries.s3-us-west-2.amazonaws.com/v1.1.3/fse-v1.1.3-node-v64-darwin-x64.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v64 ABI, unknown) (falling back to source compile with node-gyp)
node-pre-gyp http Connection closed while downloading tarball file
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR

I wasn't able to find any relevant issues already so thought I should raise it as an issue.
Any advice to resolve this much appreciated as without updating I'm unable to use node v10.x.
Many thanks

node-sass installed fine. The issue you're having is with fsevents..please open an issue there.


feat: Add toggle to allow fast type speed

Closes #18.
Within this PR:

A Toggle component was added to allow fast type speed;
The Nav component was refactored to be a functional component since we don't have language options yet;
The same "handle" function is used to update local state and localStorage when a Toggle changes;
A few styles were changed to better show the toggles;
The ESLint errors were fixed.

PS: I'm not satisfied with the labels Normal type speedand Fast type speed, but I don't know how to better name them. Suggestions are welcome. :sweat_smile:
Let me know if you want me to undo any change or do something else.

Oh, nice! This is so lovely! we really appreciate this PR.


Factor out enrichMissingMethodError

Motivation

Preparation for #7212, which will add types for certain super calls.
This code will all look a bit different when it's changed to support super,
because almost none of these autocorrects / suggestions apply to super.
With that in mind, it looks better if we can early exit from this method after
seeing that the error was caused by a call to super. I'd like to get this
refactor in now, so that the eventual diff is smaller.
Test plan

Existing tests.

Thought of a better way to factor this.


psd of NPM without javascript(dist)

https://unpkg.com/psd/

index.js

require('coffee-script/register');
module.exports = require('./lib/psd.coffee');

It's better that to release the javascript(built by coffee-script) version of psd.

And the browser's version should injected as window.PSD instead of require('psd')


Update .all-contributorsrc

Added myself as a contributor

yes, no problem then
[image: Mailtrack]
https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality11&
Sender
notified by
Mailtrack
https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality11&
03/23/22,
02:13:19 PM
On Wed, 23 Mar 2022 at 14:11, Franz Király @.***> wrote:

@.**** requested changes on this pull request.
This refers to #2285
https://github.com/alan-turing-institute/sktime/pull/2285, is that
correct?
Imo, that would be "doc", but not "design" (= sktime architecture) and
"ideas" (ideas on how to improve sktime or its operations).
—
Reply to this email directly, view it on GitHub
https://github.com/alan-turing-institute/sktime/pull/2286#pullrequestreview-918749168,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOYYPRJNX6WM7GX2VMUEFZ3VBMJ67ANCNFSM5RN67V4A
.
You are receiving this because you authored the thread.Message ID:
@.***>


Script fails when illegal character is entered for name

There is already a warning for slashes (/) which are illegal in filenames, however it will still accept a slash (/) in the input and the script will fail. There should either be a/an (in order of preference):

Allowance for special characters in the input and in the stream editor, but sanitize in the filename (optimal solution)
Allow for special characters in the input but sanitize the stream editor and the filename
Do not sanitize and prompt user for input again without illegal characters
Do not sanitize and display meaningful error message

#7 v2017.4 RC
is an implementation of option 2, option 3 still being prerfered


Cart bug

Changing product options always increments the cart total, as opposed to subtracting some from the cart if a cheaper option is selected.

Bigger pricing issues to handle


OpenAI API Key keeps getting disabled

Possible reasons:

Exposed in console when checking env vars
Github
API Key set as env in a client side container

Exclude tfplan from git


Feat/pupil 212/owa short answer

User Story
Attempt short answer questions and get feedback
Implementation details
Put as much information here as possible about how to do this task
refactor:

Move the useffect into callback with a switch statement in QuizRenderer
Create an onInitialChange callback and do the same

Acceptance criteria

[x]  refactor mcqs to remove useEffect (As per above)
[x]  short answer rendering
[x]  short answer grading
[x]  test coverage
[x]  documentation where relevant

to test
https://deploy-preview-2178--oak-web-application.netlify.thenational.academy/pupils/programmes/english-primary-ks2/units/crazy-about-cats-reading/lessons/analysing-use-of-language-in-crazy-about-cats#starter-quiz
https://deploy-preview-2178--owa-storybook.netlify.thenational.academy/?path=/docs/components-pupiljourneycomponents-quizshortanswer--docs

There is no visible label on the input field, as per the designs (this is a fill in the blank question, but the question type is the same in the data):

But we need further discussion on this issue, as well as an equivalent solution for MCQ question types, so I'm happy to approve this ticket as-is and tackle the problem in follow-up ticket(s).
Looks good, otherwise!

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


packages/exhibitor: bump to f6654479de96e898e7ed20a5fbef6b18ac88ff24

High Level Description
In https://github.com/dcos/exhibitor/pull/7 we introduced a thread pool with a corePoolSize of 10. This was too low and led to slow and broken responses as 8 of the 10 threads were pinned to the Acceptor threads, leaving only two other threads to do anything.
This issue was reproducible on a 5-master cluster on GCE.
We tested a thread pool bump from 10 to 20 and confirmed that it the issue is thus resolved.
We also bumped the request queue size to 4096 anticipating request spikes in a large cluster that boots up.
Related Issues

DCOS-14045 exhibitor requests hang/deadlock on 5-master cluster

Checklist for all PR's

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: no idea how to build a reproducible test for this. We don't even see it on all clusters. The fix was experimentally verified by hand on a 5-master cluster.
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): https://github.com/dcos/exhibitor/compare/fc44d4fe0e612381646a4738a1a898867b2086d8...f6654479de96e898e7ed20a5fbef6b18ac88ff24
[ ] Test Results: exhibitor has no CI
[ ] Code Coverage (if available): exhibitor has no Code Coverage report

@alberts @spahl @jgehrcke PTAL

@mesosphere-mergebot bump-ee

@gpaul , thanks for keeping the PR description useful. 👍  :)

@jdef PTAL

@mesosphere-mergebot bump-ee

This change was confirmed to fix the issue by launching a 1,000 node cluster and confirming that it becomes healthy. Without this change the maximum cluster size observed was in the low 10's of nodes.


Adding current build status to pipeline metrics

Would you consider adding another dimension to the pipeline metrics which is the current state of the master branch? eg CurrentBuildStatus = [0 | 1] Where 0 is success and 1 is failure. Or something along those lines?

Thanks for the q @pvdvreede! What's the problem / use case you're trying to solve here?

I would like to have metrics in cloudwatch for people to use on dashboards to see if they have any failing builds currently, this way people can know if they need to go and fix a build without having to look at buildkite.

So I've looked into this… I think for this one, until teams is official launched and there's REST APIs for it you're going to have to do this with the GraphQL API and posting your own CloudWatch metrics sorry!

Sorry, I've confused teams and this :)
I like the idea for this… I'd be open to adding LastDefaultBranchBuildStatus with 0/1.
Our API has the required endpoints, it's the org/pipeline/builds endpoint with the parameters ?branch=master&state=finished&per_page=1

I'm not sure how to achieve it and I'm pretty sure we're the definition of an edge case, but... Please support this for more than one default branch?
We maintain multiple concurrent long term (multi year) support branches.

what's the status of this?


e2e test: Added 'Integration tests'  test case

Fixes

https://issues.redhat.com/browse/HAC-3663
Description

Add Integration Test and verify.

Add Integration Test from Actions and verify.

Edit Integration Test and Verify.:

Integration test name input is marked as read-only and cannot be edited.
Save changes button only gets enabled if the edit form has new changes.

Verify that the latest changes are reflected on the IT list.

Delete Integration Test and verify.

Type of change

[ ] Feature
[ ] Bugfix
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[x] Other (please describe): Test case

Screen shots / Gifs for design review

How to test or reproduce?

Browser conformance:

[x] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

[test]

[test]

[test]

[test]

[test]

[test]

[test]

[test]

[test]

[test]

[test]

[test]

/lgtm
/approve

/retest

/retest

/retest


Add missing ASF headers and provide dev/check-license script

This PR adds missing ASF header in several files.
It also includes dev/check-license script to easily run RAT checks.

@msadoghi @cjcchen ^^


feat(payment): STRIPE-25 create a custom guest form for Stripe link

What? STRIPE-25

Create logic to render Stripeupe customer component
Create auto-step logic
Look and feel
logic to show only credit card in case StripeLink is activated and is an authenticated Stripe customer

Why?
As a Guest Shopper, I want to use the Stripe link experience.
Testing / Proof
HOW TO SETUP
https://drive.google.com/file/d/1Y6T-hohrbhelYQeupL754KRo6vLz7DPF/view?usp=sharing
PURCHASE USING STRIPE CUSTOMER
https://drive.google.com/file/d/1brzLC7kAXcC7YGz1BMrvA5BXUZTpHF7J/view?usp=sharing
WHAT HAPPENS IF A BC CUSTOMER TRY TO USE LINK  (spoiler, we are using BC components if It's a BC customer, Link experience is just for guest)
- AS A BC CUSTOMER AT THE BEGINNING
https://drive.google.com/file/d/14tUb6mmXuJWMzneNdhXM7tG8EX0_xQ5O/view?usp=sharing
- AS A BC GUEST AT THE BEGINNING BUT BC CUSTOMER ON THE CUSTOMER STEP
https://drive.google.com/file/d/1jG79FPKF4vNkUkSIO3DgYSDlFcK22Jph/view?usp=sharing
HOW TO CREATE A STRIPE CUSTOMER AND THEN PURCHASE AS A STRIPE CUSTOMER
https://drive.google.com/file/d/1ESENlAM5rIdWkTb4AY61acgjqZsBCVjr/view?usp=sharing
Depends on
https://github.com/bigcommerce/checkout-js/pull/1018
https://github.com/bigcommerce/checkout-sdk-js/pull/1616
@bigcommerce/checkout @bigcommerce/apex-integrations

💚  testing done !
https://bigcommercecloud.atlassian.net/jira/software/c/projects/STRIPE/boards/622?modal=detail&selectedIssue=STRIPE-52


Monitor subscription pool from publisher

Hi,
Is there a way to monitor the pool of subscribers from the publishing side? Ideally I want to cache messages if a subscriber drops and start publishing once a subscriber connects again.
Thanks.

You can use the NOT_CONNECTED return value from offer/tryClaim as a means to determine connectivity. However, if you need to know that a publisher is listening in a more detailed way, this is best handled as a protocol at the application layer.


⚠ Jenkins Plugins Artifactory has degraded performance

In 71837fd, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1225 ms

Resolved: Jenkins Plugins Artifactory performance has improved in d2dd6fc after 12 minutes.


[WIP] Update .env file to hide more sensitive information.

Add new variables to .env file to remove personal information from .yml files.
[#138940073]

Please add to Contributing.md to explain how to find these new values.

@jasonnoble I updated the Contributing.md file but I can't find the PIVOTAL_MEMBERSHIP_ID anywhere on Pivotal Tracker without making a request to the API. Since this was the variable that was giving trouble earlier, maybe it shouldn't be included in the .env file?


Add redis sentinel support

There are a few breaking changes made to support this.
Most notably, the an optional SentinelConfig struct
has been added to the RedisConfig object. It seems like
obfuscating the fields here might be a good idea, but
that's not how we've been rolling thus far with the
config objects.
Also, the RedisConnection trait has been redone
to take a RedisConfig instead of a DSN in order
to build the connection. Sentinel needs it owing
to the large number of potential config params needed
for Sentinel, and I think this is probably a better
approach anyway.
The base redis tests (well, most of them) have also
been updated to support testing against either
base Redis or Sentinel. The rstest crate has been
added in order to be able to parameterize the tests
sanely. We also use this crate for tests in redis-rs,
so I think it's a no-brainer to add here.

Is the breaking change only when sentinel is enabled?

Is the breaking change only when sentinel is enabled?

No. We could potentially add a field that's dependent on the feature, but I think a better approach is to stop making config fields public and instead switch to builders. This will cause some issues downstream with Bridge IIRC, but that's something that I'm guessing should be fixable.


Please invite me to the community

Name
Emmanuel Adom
Additional Context
I would love to participate and contribute to this incredible community of devs.

Invitation sent to join the GitHub Organization. Welcome to Infraform 🎉Please check your mail to accept the invitation sent by GitHub! 🤖About usInfraform is an open source solution for serverless platforms, it is designed for developing solutions and enhancing creators to create their infrastructure 💻 Great to have you in our community ✨ Looking forward to your valuable contributions! 🤓 You can also check our community ✨ Github ✨ LinkedIn 💜 You can also share our community with other folks who want to learn about development, DevOps, tech, and contribute together and with us 👉💜👈


Pruning

Paper: Pruning Convolutional Neural Networks for Resource Efficient Inference
Paper Link: https://arxiv.org/abs/1611.06440

Description
Pruning is the first step mostly taken to compress neural networks
Modifications
None as of now, might find some later on

Ok, so this comes inbuilt with pytorch now.  https://pytorch.org/tutorials/intermediate/pruning_tutorial.html#pruning-a-module
So I don't think we need a separate implementation.


Github actions and style checks

Motivation
Detect markdown issues and style issues on PR level by adding github actions capability.
This can be extended later to include extra styling for website etc.
This resolves first part of the #21

Actual style changes can be discussed and adjusted (to some extend) etc.

Rebased. Good to merge

Build is passing now and all styles were updated.
Thank you for review

@wtrocki thank you !


Uncaught TypeError: Failed to construct 'URL': Invalid URL (vite 3)

Uncaught TypeError: Failed to construct 'URL': Invalid URL
at index.2d20d94d.js:1:52013

See build: https://gist.github.com/yooouuri/b1d1706f7708865a359680bb3fdfbe0a#file-index-2d20d94d-js-L2745
It tries to generate a URL:
const nl = "" + new URL("electron.fe1dfbad.png", Wn.url).href

But,
const Wn = {};

There is no .url on Wn...
https://gist.github.com/yooouuri/b1d1706f7708865a359680bb3fdfbe0a#file-index-2d20d94d-js-L2
Happens when I upgrade vite to version 3.
How to reproduce:
git clone https://github.com/yooouuri/electron-vite-vue
cd electron-vite-vue
yarn install
yarn build

Open the binary in the release folder, for me its the .app (Mac OS).

Guess https://github.com/electron-vite/electron-vite-vue/issues/212 is related

Thanks for your feedback and reason analysis.
I have noticed this issue, I will try to fix it via a plugin or by submitting a PR to Vite.
At present, you can use<EMAIL_ADDRESS>to avoid this BUG.


report and stats

Produce a report at the end of the repertoire generation.
The report should include the plots and the stats of the moves.

[ ] stats means can be produced by applying the weight proportional to the move/line relative playing probability (move_freq)

[ ]  stats means can be produced by applying the weight proportional to the move/line relative playing probability (move_freq)

Each repertoire can be classified with several numbers:

the yaml settings of course
a weighted mean of the "leaf" of the repertoire that is the sum of the scores of the last moves of each lines weighted for the number of games in that position divided by the sum of weights

it can be done both for the db score (black or with depending on the repertoire side) and for the engine evaluations

[ ] Accumulate all the repertoire stats in a sqlite3 db


pipeline the escalation handshake logic

Would it be nice if the escalation handshake logic is pipelined to break the timing?

Thanks for the question, and apologies that this never got answered.
The escalation handshake logic has intentionally not been pipelined to keep the signalling latency (in terms of clock cycles) low.
There are only 4 escalation endpoints, and the frequency of the corresponding clock domain is pretty low (the peripheral clock domain runs at 24MHz).
Hence this is likely not going to create any timing issues.
Please feel free to reopen if you have any other questions related to this topic.


Icon and TextInput are not aligned inside InputGroup on Adnroid

@raminious Please check the latest of NativeBase and NativeBase-KitchenSink

the latest of NativeBase and NativeBase-KitchenSink, have the same problem :(

This will be fixed with the rewrite of NativeBase.
To be released very soon


Made ApiController non-abstract

This fixes php artisan route:list (controllers can't be abstract because then they're not instantiable)
I haven't tested the full API functionality, as I don't use this (yet). I don't think this will cause any problems though.

In my setup, I added a status endpoint to the API. This also broke due to the class not being instantiable :)

In my setup, I added a status endpoint to the API. This also broke due to the class not being instantiable :)

Hi @gertjankrol,
Thanks for offering this.
Just to confirm though, as far as I can see the error which this fixes would only occur due to an route being added that utilises this ApiController class; This does not fix a bug that exists in the release code. Is that correct?

Hi @gertjankrol,
Thanks for offering this.
Just to confirm though, as far as I can see the error which this fixes would only occur due to an route being added that utilises this ApiController class; This does not fix a bug that exists in the release code. Is that correct?

Hi, it also fixes artisan route:list, which is now failing due to the class not being instantiable

Hi, it also fixes artisan route:list, which is now failing due to the class not being instantiable

@gertjankrol But only when you add your own routes? When testing on my system artisan route:lists works fine.

@gertjankrol But only when you add your own routes? When testing on my system artisan route:lists works fine.

Oh, I'm sorry. Just confused it a bit, the problem is not in the release code :)

Oh, I'm sorry. Just confused it a bit, the problem is not in the release code :)

@gertjankrol Okay, Thanks for confirming. In that case, I'd probably prefer to leave this class abstract as I really intend for it to act as a base for all other API controllers.
You mentioned adding your own status endpoint, If you have any input to provide on https://github.com/BookStackApp/BookStack/issues/2467 that would be much appreciated, then we can get something officially built-in.

@gertjankrol Okay, Thanks for confirming. In that case, I'd probably prefer to leave this class abstract as I really intend for it to act as a base for all other API controllers.
You mentioned adding your own status endpoint, If you have any input to provide on https://github.com/BookStackApp/BookStack/issues/2467 that would be much appreciated, then we can get something officially built-in.

Sure, I can help out with that. I'll try to get a PR ready for review tonight

Sure, I can help out with that. I'll try to get a PR ready for review tonight


Password not being set on DB when running from Kubernetes

Hi,
I've been trying to set the password on the docker container through Kubernetes, but it's not locking it down.  Here's my code:
https://gist.github.com/codingwithoutcomments/1e22d041608cbe5e5c7a
Any ideas?

I figured out the problem.  You have to specify the POSTGRES_PASSWORD the first time.  If you don't, you cannot add it later.


Add integration tests for slurmdbd operator

Description
This pull request adds integration tests for the slurmdbd operator. The tests first build and deploy slurmdbd against the other slurm charms from latest/edge, checks that the slurmdbd and munge daemons are up, and verifies that slurmdbd is listening on port 6819.
How was the code tested?
I tested the code locally on my workstation
Related issues and/or tasks
This pull request is a part of the Charmed SLURM project.
Checklist

[x] I am the author of these changes, or I have the rights to submit them.
[x] I have added the relevant changes to the README and/or documentation.
[x] I have self reviewed my own code.
[ ] All requested changes and/or review comments have been resolved.

You will need to test my changes locally since the slurmdbd charm is still failing the lint tests. If they fail both unit and lint tests, I do not want to waste time running integration tests through GitHub Actions. Next pull request will be focusing on getting the code to pass lint rules (need to agree on a standard with Omnivector.)

Note: I have not merged this (but I have approved the code changes functionally). We shouldn't knowingly merge a failing test into gate - if this was standard practice, you'd never know when the gate was broken intentionally or not.
I know that style guide confirmation is pending, but I anticipate it aligning closely with the published charm style guide. There are generally 3 types of errors reported:

spelling mistakes
un-sorted import blocks
literal dict/list vs {},[]

There was a discussion around item 3, but its not worth holding off progress for that one pieces. My recommendation is to fix the spelling mistakes, run a tox -e fmt, commit the changes and call it a day.

I agree about not merging with failing tests; should be the standard going forward. That being said, we should merge this pull request to keep it on topic - I don't want to shoehorn in a commit that overalls the whole code base - and then the next pull request should be focused on fixing lint issues in the code base. That way @dvdgomez and I can take out all the linting issues in one fell swoop.


Parse error on the millionaire example

viaduct % sudo ./viaduct -v compile benchmarks/examples/Millionaires.via -o m-out.via
-- PARSE ERROR ---------------------------- benchmarks/examples/Millionaires.via
I ran into an issue while parsing this file.
11| host alice : {A & B<-}
^
I was expecting one of these:
EOF, FUNCTION, ASSUME

Instead, I found:
COLON

Thanks for reporting. We have updated the syntax (to make it easier to specify security policies), but seems like we forgot to update benchmarks.

Cool! I could help fix it if it's straightforward enough. Let me know how it should be written.

We should have it fixed pretty soon. I think @s-ren is working on it now.

I just updated benchmarks.
The updated syntax takes delegation declarations instead of principal parameters. For example:
host alice : {A & B<-}
host bob : {B & A<-}

becomes
host alice
host bob
assume for <- alice trusts bob
assume for <- bob trusts alice

Labels use host names directly.


ATLAN-2570 fixed the hasMoreVertices condition for terminating graph c…

Lineage Bug fix
Ticket: ATLAN-2570
https://linear.app/atlanproduct/issue/ATLAN-2570/find-hotel-20-or-[snowflake-lineage]-lineage-graph-not-showing-all

fixed the hasMoreVertices condition for terminating graph correctly

Type of change

[x] Bug fix (fixes an issue)
[ ] New feature (adds functionality)

Checklists
Development

[x] Lint rules pass locally
[x] Application changes have been tested thoroughly
[ ] Automated tests covering modified code pass

Security

[x] Security impact of change has been considered
[x] Code follows company security practices and guidelines

Code review

[ ] Pull request has a descriptive title and context useful to a reviewer. Screenshots or screencasts are attached as necessary
[ ] "Ready for review" label attached and reviewers assigned
[ ] Changes have been reviewed by at least one other contributor
[ ] Pull request linked to task tracker where applicable

LGTM


Event switchboard entries for Amazon EBS events.

This PR will configure entries in the event switchboard for AWS EBS events. The specific events are listed below and will trigger an EMS refresh to keep the AWS inventory up to date.
Links:
https://bugzilla.redhat.com/show_bug.cgi?id=1449235
https://www.pivotaltracker.com/story/show/146650341
Testing:

[ ]  Add an AWS provider
[ ]  Take note of the number of EBS cloud volumes
[ ]  Add a new EBS cloud volume through the AWS Console
[ ]  Ensure the new EBS cloud volume appears under: Storage / Block Storage / Volumes. This will indicate an event was received and went through the event switchboard where an  EMS refresh was kicked off.
[ ] Follow the same steps for:

DeleteVolume
AttachVolume
CreateSnapshot
DeleteSnapshot
CopySnapshot

@miq-bot assign @lfu
@miq-bot add_label fine/yes

@miq-bot add_label blocker, enhancement

The blocker label applies to the Fine release (5.8.1)

@bronaghs Modeling changes should be applied to the ManageIQ-content repo.

@bronaghs You only need to add those event instance yaml files.
Please export these instance yaml files from automate following steps here: http://talk.manageiq.org/t/creating-a-pull-request-for-enhancements-to-manageiq-automate-domain/1041
As @gmcculloug said, this PR should be moved to manageiq-content repo.

Closing. Superseded by https://github.com/ManageIQ/manageiq-content/pull/137 in the manageiq-content repo.


Can make build be made to run in parallel?

I'm running make -j 12 build on my beefy 8-core, 16-thread desktop, but the build is quite pokey, and the load reported by uptime is in the ballpark of 1.3.  I see a lot of runs of agda and html-minifier that look like they could be done in parallel.  Would it be possible to update the makefiles to make this so?

I’ve not put much work into this, and I’d be hesitant to accept a PR, given how inscrutable the build system is.

Closing due to lack of activity.


fmSetupAssistant: Security: Protect Layouts

The current security is too thin. The user account currently has full access, so Layouts can be edited, etc.

[x] Change PW to user.
[x] Ensure that fmSetupAssistant starts in the user account.
[x] Add a Re-Login button on the start page to swap between User and Admin

For now that should suffice.

The Re-Login button has been added to all layouts


Switch custom icon not clickable

Description
While implementing pull request 1511 (i.e. clickable area "None" for ui-switch), seems I overlooked the custom icons had a special part in the vue template.
When clickable area is "None", the custom icon switch should also be non-clickable.
You can use the example flow from another issue (see 1522) to test this: just change the clickable area to "None".
Related Issue(s)
1521
Checklist

[X] I have read the contribution guidelines
[ ] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

[ ] Upgrade instructions
[ ] Configuration details
[ ] Concepts

[ ] Changes flowforge.yml?

[ ] Issue/PR raised on FlowFuse/helm to update ConfigMap Template
[ ] Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production

Labels

[ ] Includes a DB migration? -> add the area:migration label

I've also added pointer-events: none to prevent any hover behaviour also being triggered when click events are set to "None". This ensure it's clear to the end-users that no interaction is available.


how to build a static bundle file for deployment if i was using webpack-dev-server?

i have been using webpack-dev-server for hot reload during development. but after my app is built i want to make a static bundle and i want to use it without running a development server. (without $ node server.js)
what is the best way of doing it?
webpack.config.js
var path = require('path')
var webpack = require('webpack')
var BundleTracker = require('webpack-bundle-tracker')

module.exports = {
	context: __dirname,
	entry: {
		'meet-simulator': [
			'webpack-dev-server/client?http://localhost:3000',
			'webpack/hot/only-dev-server',
			'./assets/js/index'
		]
	},
	output: {
		path: path.resolve('./assets/bundles/'),
		filename: '[name]-[hash].js',
		publicPath: 'http://localhost:3000/assets/bundles/',
	},
	plugins: [
		new webpack.HotModuleReplacementPlugin(),
		new webpack.NoEmitOnErrorsPlugin(),
		new BundleTracker({filename: './webpack-stats.json'}),

		new webpack.ProvidePlugin({
			$: 'jquery',
			jQuery: 'jquery',
			'window.jQuery': 'jquery'
		})
	],
	module: {
		loaders: [
			{
				test: /\.jsx?$/,
				exclude: /node_modules/,
				loaders: [
					'react-hot-loader',
					'babel-loader?presets[]=react',
				]
			}
		]
	},
	resolve: {
		modules: ['node_modules', 'bower_components'],
		extensions: ['.js', '.jsx']
	}
}

server.js
var webpack = require('webpack');
var WebpackDevServer = require('webpack-dev-server');
var config = require('./webpack.config');

new WebpackDevServer(webpack(config), {
	publicPath: config.output.publicPath,
	hot: true,
	inline: true,
	historyApiFallback: true
}).listen(3000, '<IP_ADDRESS>', function (err, result) {
	if (err){
		console.error(err);
	}

	console.log('Listening at <IP_ADDRESS>:3000');
});

Don't use webpack devserver. Don't invoke server.js but directly use webpack CLI command. A simple example: https://github.com/ezhome/django-webpack-loader/tree/master/examples/simple and webpack docs https://webpack.js.org/

Hello, I guess you wrote this article https://owais.lone.pw/blog/webpack-plus-reactjs-and-django/ @owais
So it's advised against using webpack-dev-server now?
we are using webpack4.


disable-bt and miniuart-bt device tree overlays are broken

Upgrading to 3.5 bricked my install on a Pi 3 B+ until I removed dtoverlay=miniuart-bt from boot/config.txt. While debugging the Pi, upon booting I was met with a blank screen and no other information.

https://github.com/agherzan/meta-raspberrypi/issues/374


Added 'max' operation to Merge layer

This allow to implement  convolutional maxout with two (or more) convoluion layers and one Merge (as it is done in caffe).
Example:
`
conv10a = Convolution2D(32, 3, 3, border_mode='same', init='orthogonal', W_regularizer=l2(0.0001),)(conv9)

conv10b = Convolution2D(32, 3, 3, border_mode='same', init='orthogonal', W_regularizer=l2(0.0001),)(conv9)

conv10 =  merge([conv10a, conv10b], mode='max', concat_axis = 1)

`
This can be lightweight alternative to https://github.com/fchollet/keras/pull/671
#3102

Please add it to the Merge layer tests.

@fchollet done

LGTM


assisted ztp: add disruptive label to operator tests

Add disruptive label to prevent flaky failures in operator suite

/cc @vkolodny

/lgtm


Change Event

I need to monitor for value changes on a multiple select field.  I can listen for the event on the actual select element, but that change event fires each time an item is selected.  However, this is not the behavior I need.
Are any events fired when the bootstrap-select is closed?  This would allow several selections to be made with a single event that can be used to trigger an action with the final value.  Possible?

Thanks @caseyjhol -- your suggestion works.
It'd be nice if bootstrap-select offered a cleaner way to get at the select element.
This feels like we're peeking behind the curtain:
.data('selectpicker').$newElement;

See commit 541d9a06af6fc69161156a78cdb081567f2a3b03

I'm a new Javascript/jQuery  user and I am facing this same issue at the moment.
@caseyjhol - https://github.com/silviomoreto/bootstrap-select/issues/216#issuecomment-33105746 does not work for me.
I want to trigger an event as soon as the the bootstrap-select is closed.
My Code snippet:
$('#multiselect').data('selectpicker').$newElement.on('hide.bs.dropdown', function() {
                alert('hide');
});

What is the current way to do this, after https://github.com/silviomoreto/bootstrap-select/commit/541d9a06af6fc69161156a78cdb081567f2a3b03 ?

@xennygrimmato the current way of doing this is:
$('#multiselect').on('hide.bs.select', () => alert('hide'));

Thanks @sebkoller !

My simple solution:
	$('[name^=selectP_fabricColor], [name^=selectP_fabricPattern]').parent('div').on('hidden.bs.dropdown', function(){ 		getOptions(currentA) 	});

  $('#selectpicker').on('shown.bs.select', function(){     console.log('its shown son!')   })
Works like a dream. Thanks @sebkoller, wish I had scrolled to the bottom instead of spending an hour googling :)

I was looking for a way to only submit data to the server once the multiselect has been closed but only if it was changed. The solution is fairly trivial so I leave it here for others.
Sure a 'changedhide.bs.select' event would be handy but for now:

var filterChanged = false;

$('.selectpickers').selectpicker({
    // selectpicker options
}).on('changed.bs.select', function (e) {
    filterChanged = true;
}).on('hide.bs.select', function (e) {
    if(filterChanged){
        filterChanged = false;
        // send data to server or what ever :)
    }
})

I am having a select picker in my html page as below which will  select multiple values in drop down.I am having difficult time in capturing the selected values in the page.viewmodel.js of knock out.Can some one help me out on how to capture multiple values

Name:

                            <select id="selName" class="someNameselectpicker input-small pull-left" data-live-search="true" multiple data-actions-box="true" data-bind="options: vm.demo.searchservicerequest.names(),
                                                    value: selectedRequester,
                                                    selectpicker: {  width: '350px' },
                                                    optionsValue: function(item) {
                                                        return item;
                                                    },
                                                    optionsText: function(item) {
                                                        return item.FirstName+', '+item.LastName;
                                                    },
                                                    optionsCaption: ui.selectCaption"></select>


Feature Request: Limiting users who can be messaged/found.

Description
In my case I'm using Laravel Spark, I only want a user to be able to message users within their team/teams. Regardless of Spark it would be nice to be able to "filter" the people they search/can have a conversation with.

Hey @Braunson, for a quick solution to have Mercurius working with Teams in a Laravel Spark app, you can try this steps. Was not tested but will give you an idea.

Add Route on routes\web.php

Route::post('/receivers', 'App\Controllers\ReceiversController@search');

Create ReceiversController

php artisan make:controller ReceiversController

Apply filter logic to ReceiversController

Open App\Http\Controllers\ReceiversController and code something like this:
namespace App\Http\Controllers;

use Illuminate\Http\Request;
use Illuminate\Http\Response;

class ReceiversController extends Controller
{
    /**
     * Create a new controller instance.
     *
     * @return void
     */
    public function __construct()
    {
        $this->middleware('auth');
    }

    /**
     * Search for receivers.
     *
     * @param Request        $request
     * @param UserRepository $userRepository
     *
     * @return \Illuminate\Http\Response|\Illuminate\Contracts\Routing\ResponseFactory
     */
    public function search(Request $request, UserRepository $userRepository): Response
    {
        if (($query = $request->input('q')) === null) {
            return response([
                'hits'  => [],
                'total' => 0,
            ]);
        }

        // Apply filter to the Users in Teams/Users
        // you will find this code on the UserRepository
        // ------------------------------------------------
        $limit = 6;
        $userFqcn = config('mercurius.models.user');
        $paginator = $userFqcn::where('name', 'LIKE', '%'.$query.'%')
            ->paginate($limit, [
                'id',
                'name',
                'avatar',
                'is_online',
            ]);
        // ------------------------------------------------

        $result = [
            'total' => $paginator->total(),
            'hits'  => $paginator->items(),
        ];

        return response($result);
    }
}

In a future release of Mercurius we will be able to load custom Controllers, using the config, and this configuration wont be required anymore.
Let us know the result. TY

Hey - random thought that might be easier....
As Laravel you can apply scope classes via service providers. Just make a scope class that adds a new condition to only show users who should be visible?
That way it’s as flexible as humanly possible as the scoping is 100% in the hands of the developer?

Great idea @OwenMelbz, thanks for the tip.


Add new storage backends

What is the context of this PR?

Removes Dynamo->SQL fallback
Adds storage backends for:

Redis
AWS S3
Google BigTable
Google Cloud Storage

This removes the RDS fallback code.  Also FTR most new datastores are not feature complete, usually because they don't support the overwrite flag properly.

Yeah I'm not sure about whether it's worth merging this. It's very much there for testing. Introducing unused stores would add unecessary code to maintain and unused dependencies.
Once we've decided what store we're going to use in future it might be worth just switching over to that.


Documentation for installation on Windows

Can you please include documentation on how to install Modin for Windows machines, since Ray is not supported on Windows?

Hi @waltonereed, thanks for your question! I will add this to the documentation, but for now here are the instructions:
I personally use Modin on Windows through the Windows Subsystem for Linux (WSL): https://docs.microsoft.com/en-us/windows/wsl/install-win10
After you install WSL, open the Linux distribution's shell you installed (Ubuntu in my case).
From here you can follow the installation instructions for Linux:
pip install modin

From here, you need to run jupyter or ipython or python from this terminal.
If you have any further questions, please let me know!

Closing as complete in #188

Just to update - It can now be installed on Windows using dask backned
pip install modin[dask]


Remove unnecessary qemu-img from vm images built with vm-builder

neonvm-runner does need qemu-img, but that's why the neonvm-runner Dockerfile does "apk add qemu-img". We don't need it in both images.

@hlinnaka
This is necessary. It is used to change the size of the root disk.
https://github.com/neondatabase/autoscaling/blob/main/neonvm/runner/main.go#L760
I believe this change will break staging and prod
https://github.com/neondatabase/cloud/blob/main/goapp/controlplane/internal/pkg/compute/provisioner/provisioner_k8s_vm.go#L502

@hlinnaka
This is necessary. It is used to change the size of the root disk. https://github.com/neondatabase/autoscaling/blob/main/neonvm/runner/main.go#L760

Right, but  the neondatabase/neonvm-runner image contains its own copy of qemu-img, see:
https://github.com/neondatabase/autoscaling/blob/f2c47b2d1bb5cb625adc74c85c75fb35612cb74b/neonvm/runner/Dockerfile#L35
The qemu-img in the neondatabase/vm-compute-node-v17 image is not even accessible to neonvm-runner.

I believe this change will break staging and prod https://github.com/neondatabase/cloud/blob/main/goapp/controlplane/internal/pkg/compute/provisioner/provisioner_k8s_vm.go#L502

To be honest, I haven't been able to set up an environment where to test this yet. I'm trying to set up a local environment now. How do others test changes to vm-builder / neonvm-runner?

Before merging, could you rename the PR to prefix the title with "vm-builder: " for consistency with other PRs?

Done

Also, seems like these lines should also be moved to the builder stage towards the very end, if you'd like to add that as well:
https://github.com/neondatabase/autoscaling/blob/f376b40c31516b701d87db8b97bf82b4fa19ea9c/neonvm/tools/vm-builder/files/Dockerfile.img#L45-L49
(we definitely don't need qemu-img inside the VM, and we already apk add e2fsprogs-extra inside the VM anyways)

Good catch! Moved.

Merged, thanks.

Our only usage of the final VM image is to copy the qcow2 file out of it (well, plus some other small bits), so removing qemu-img file should be ok.

This reminds me of another thing I tried: I figured it's silly for the VM image to contain anything other than the qcow2 file, so I tried to change the final part of Dockerfile.img to use FROM scratch instead of FROM alpine:3.19. That didn't work, because it's used as an initcontainer where we do this:
 "sh", "-c",
                                                "cp /disk.qcow2 /vm/images/rootdisk.qcow2 && " +
                                                        /* uid=36(qemu) gid=34(kvm) groups=34(kvm) */
                                                        "chown 36:34 /vm/images/rootdisk.qcow2 && " +
                                                        "sysctl -w net.ipv4.ip_forward=1",

So we do need 'sh', 'cp', 'chown', and 'sysctl' in the image.

Yeah -- technically we could avoid that by adding another init containers, but the slow down from another init containers probably isn't worth the benefits of a slightly reduced image size.


Anoosha-foodon

Updated terms- Changed the relations to 'has defining ingredient' and replaced the deprecated term 'obsolete: peach tree' with 'Prunus persica'.

Want some guidance: I can move few terms which are currently in import/foodon-edit.owl to import/foodon_product_import.owl. But firstly, that changes the file siren_augment_codes.owl which I think is due tocomment section.
Secondly, do we also move subclass of (anonymous ancestor)section? As that changes the file general import.owl?

You can actually keep the rdfs:comment annotation that is in the siren file by moving it alone back to the siren_augment_codes import file (via menu selection).
But about "general_import.owl" file - that is an auto-generated file, so one shouldn't be moving anything from or to there. It gets generated from general_ontofox.txt file sent to ontofox.

You can actually keep the rdfs:comment annotation that is in the siren file by moving it alone back to the siren_augment_codes import file (via menu selection).
But about "general_import.owl" file - that is an auto-generated file, so one shouldn't be moving anything from or to there. It gets generated from general_ontofox.txt file sent to ontofox.

Thanks..Took some time to understand that actually these terms are already present in foodon_product_import.owl, just that single axiom was missing. I've now moved the axiom too.


Ability to prevent header/footer on certain pages

This is simply a feature request (or if it's available already, a question on how I can accomplish this).
I've gotten the header and footer templates working but I'd like to prevent them from displaying on a specific page/section.
Is this possible or could it be added as a feature?

Hello @EvanAsbuilt, the Chrome dev protocol does not allow such behavior atm. Best option is to find a way to do it using JavaScript/CSS print libraries.


[Feature]: following timeline

Not sure if there is another way of doing this which is why its not already in this lib but adding support to fetch the following timeline.

Just seen the getFollowingTimeline in relationships.ts maybe that is the same thing, although its Followers vs HomeLatestTimeline 🤔🤷♀


The Manager isnt working

I dont know if i did something wrong but it puts my files in the normal downloads folder and not in the folder i put in.
http://prntscr.com/nvhj5k
Pls help

You can only use relative urls and they will be appended to your downloads folder, e.g. destination: "/audios" will put files into /downloads/audios


test: add jasmine testing

Hi @JayKan unfortunately I have no experience in testing Angular2 components but I will do my best to be useful in this issue. Apart from the repo you already suggested me, do you have other resources that could be useful to get me started?
Thanks


offsetparser + date timestructure in s3

Is it possible to use the system time of the upload to generate a s3 date structure instead of parsing messages to extract a timestamp there?

You can provide a custom MessageParser which will return the time partition
based on system time instead of looking inside the message.
On Wed, Dec 9, 2015 at 10:31 AM, Maximilian Weber<EMAIL_ADDRESS>wrote:

Is it possible to use the system time of the upload to generate a s3 date
structure instead of parsing messages to extract a timestamp there?
—
Reply to this email directly or view it on GitHub
https://github.com/pinterest/secor/issues/151.

It is possible (you can simply implement a custom message parser) but it is highly discouraged. If you use system time, you will break the consistency guarantees and will possibly introduce duplicate messages. Secor operates under the assumption that message->s3 path mapping is deterministic.


Uncaught (in promise) TypeError: Cannot read property 'includes' of undefined

Description
On hitting the play button on https://collab-project.github.io/videojs-wavesurfer/demo/live.html
it is showing below error on console.
Uncaught (in promise) TypeError: Cannot read property 'includes' of undefined     at MicrophonePlugin.fireEvent (observer.js:125)     at MicrophonePlugin.deviceError (microphone.js:348)     at microphone.js:153
Steps to reproduce
Go to https://collab-project.github.io/videojs-wavesurfer/demo/live.html then hit the play button,  then open console on the browser.
Results
Expected

No error
Download recorded audio.

Actual
Not able to download audio also error was thrown in the console
Error output
Uncaught (in promise) TypeError: Cannot read property 'includes' of undefined     at MicrophonePlugin.fireEvent (observer.js:125)     at MicrophonePlugin.deviceError (microphone.js:348)     at microphone.js:153
Versions
Make sure to include the following versions:
videojs/wavesurfer
what version of video.js, videojs-wavesurfer and wavesurfer.js does this occur with?

video.js 7.7.6
videojs-wavesurfer 3.2.0
wavesurfer.js 4.0.0

yea it's an issue with wavesurfer.js 4.0.0. Use wavesurfer.js 3.3.3 for now. See https://github.com/katspaugh/wavesurfer.js/issues/1975

Hey, @thijstriemstra thanks for a quick response.

I'll reopen this and close it once a new wavesurfer.js is released.

Use wavesurfer.js v4.0.1 or newer instead.


[FEATURE REQUEST] Set "Use system Chocolaty" by defualt

Before submitting the issue

[*] I have checked that the requested feature is not a duplicate or, if it is, it needs to be reopened
[* ] I have checked that I am running the latest version of WingetUI
[ *] I have checked the FAQ section
[* ] I have read the contributing guidelines and I agree with the Code of Conduct

Is your feature request related to a problem? Please describe.
Have the Chocolaty setting  "Use system Chocolaty" set by default by the installer if it detects Chocolaty is already installed.  I was very confused why my Chocolaty installed packages weren't showing up.  Turns out WingetUI was using it's own Chocolaty, something I had no idea it would do.
Describe the solution you'd like
During install, when it asks what sources you want to use and you select Chocolaty, detect if Chocolaty is already installed, and if so, automatically check a box to use installed Chocolaty.

Yes, it seems like a good idea

Done!
(This check will be done only on the welcome window, when setting up the package managers)