Update bundle manifests to include metrics-server

Sorry if I missed this before. I was not using the OperatorHub to deploy them. Thanks @gkurz for spotting this.
There is one commit that I'm not sure we should include now or later. Its prefixed with RFC.

RFC: 1.8 bump
Not sure how we are going to deal with this, since this image is not
published yet, only after the release. But we don't want to have the
manifest pointing to 1.7.

This is no different from other images. It is available at brew.registry.redhat.io/rh-osbs/openshift-sandboxed-containers-operator-monitor:1.8.0 and we can apply an ImageTagMirrorSet such as https://github.com/bpradipt/coco-install/blob/main/aro-coco/image_mirroring.yaml to pull it. The bumping is thus perfectly fine.

Thanks @gkurz , please take a look if is ok now.

@gkurz is this non-required test, safe to ignore?

imho, while the test is non-required, it should still pass or at least we should know the reason behind the failure.

Agreed. How can one access the logs from the test interface?

You can click the "Details" link on the failing job, which gives the most useful logs.
If you need more details, on the upper-right corner you have an "Artifacts" link that shows all the files generated by the job. You can download those locally for easier grep-ing (the command to do that is shown in the web page).

Agreed. How can one access the logs from the test interface?

You can click the "Details" link on the failing job, which gives the most useful logs. If you need more details, on the upper-right corner you have an "Artifacts" link that shows all the files generated by the job. You can download those locally for easier grep-ing (the command to do that is shown in the web page).

Your comment crossed the update to my comment. What I seem to be missing is the specific csv-sandboxed-containers-operator.v1.8.0.yaml. Can't find it through the web interface, but gsutil gave it to me.

@gkurz is this non-required test, safe to ignore?

imho, while the test is non-required, it should still pass or at least we should know the reason behind the failure.

Yes I agree. We invested quite some efforts into having an always green CI. We should understand what is happening as this might happen to a customer.

Error seems to be the OSC controller failing to come up :
[2024-11-19T19:14:34.466Z] ClusterServiceVersion sandboxed-containers-operator.v1.8.0 was created but never became ready
[2024-11-19T19:14:34.468Z] Dumping ClusterServiceVersion sandboxed-containers-operator.v1.8.0 as /logs/artifacts/csv-sandboxed-containers-operator.v1.8.0.yaml
[2024-11-19T19:14:34.982Z] ClusterServiceVersion sandboxed-containers-operator.v1.8.0 status phase: Failed
[2024-11-19T19:14:35.261Z] ClusterServiceVersion sandboxed-containers-operator.v1.8.0 status message: install failed: deployment controller-manager not ready before timeout: deployment "controller-manager" exceeded its progress deadline
[2024-11-19T19:14:35.519Z] ClusterServiceVersion sandboxed-containers-operator.v1.8.0 status reason: InstallCheckFailed

This should definitely be investigated.

 Thanks for digging this out of the hay stack :wink:
I still don't understand why this is happening though... I'll try to compare with previous CI runs.

This seems super relevant @c3d ! Thanks for digging this out of the hay stack 😉
I still don't understand why this is happening though... I'll try to compare with previous CI runs.

It seems to be a red herring as I'm also seeing that error in https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/test-platform-results/pr-logs/pull/openshift_sandboxed-containers-operator/472/pull-ci-openshift-sandboxed-containers-operator-devel-sandboxed-containers-operator-e2e/1854495614100836352/artifacts/sandboxed-containers-operator-e2e/gather-extra/artifacts/oc_cmds/events but the controller eventually starts.

According to https://quay.io/repository/openshift_sandboxed_containers/openshift-sandboxed-containers-operator?tab=history , the v1.8.0 tag was created this morning at 10:16 AM CET... maybe it wasn't there when the pipeline ran ?
I'll restart the test in case it is some transient error...

/test sandboxed-containers-operator-e2e

This seems to be related to the metrics-server, indeed:
[2024-11-20T13:23:01.419Z] ClusterServiceVersion sandboxed-containers-operator.v1.8.0 status message: install failed: deployment operator-metrics-server not ready before timeout: deployment "operator-metrics-server" exceeded its progress deadline

This seems to be related to the metrics-server, indeed:
[2024-11-20T13:23:01.419Z] ClusterServiceVersion sandboxed-containers-operator.v1.8.0 status message: install failed: deployment operator-metrics-server not ready before timeout: deployment "operator-metrics-server" exceeded its progress deadline

Good catch !

And it might be because: registry.redhat.io/openshift-sandboxed-containers/osc-monitor-rhel9:1.8.0 does not have the metrics-server image yet.

It is very likely indeeed.

As we cpaas-midstream/osc-operator/-/merge_requests/630/ has the missing bits.

So in order to pass upstream CI, we need to merge midstream first... this looks terrible but I guess this is not new. We're just discovering or re-discovering this now that we care for the e2e check again :wink:

This seems to be related to the metrics-server, indeed:
[2024-11-20T13:23:01.419Z] ClusterServiceVersion sandboxed-containers-operator.v1.8.0 status message: install failed: deployment operator-metrics-server not ready before timeout: deployment "operator-metrics-server" exceeded its progress deadline

Good catch !

And it might be because: registry.redhat.io/openshift-sandboxed-containers/osc-monitor-rhel9:1.8.0 does not have the metrics-server image yet.

It is very likely indeeed.

As we cpaas-midstream/osc-operator/-/merge_requests/630/ has the missing bits.

So in order to pass upstream CI, we need to merge midstream first... this looks terrible but I guess this is not new. We're just discovering or re-discovering this now that we care for the e2e check again 😉

But I'm not confident that merging that will solve the issue entirely, because the image is a public image, that will be only available after the release button is pressed.

On second thought, since we exactly know why CI is failing (no image with the metrics-server usable by CI), no need to hold this PR any longer.


removing deprecations

Description

removing deprecations

Overview of Changes

removed @Grid postButtonUrl
removed @Grid datakey
removed @GlobalNavMenu

Type of Change

[ ] Refactor/Technical Debt

Test Details

N/A
Additional Notes

N/A

You did it @lopezton-anthem!
Thank you for signing the Anthem Opensource Software CLA.


md-button takes a while to make a table responsive

I've got table with 100 cells in my table.
Table example:
<md-table-row v-for="(row, index) in filter.rows" :key="index" :md-item="row">     <md-table-cell class="center-content">          <md-switch v-model="row.status" class="md-warn"></md-switch>          <md-button v-on:click="removeRow(index)" class="dark-gray"><md-icon class="dark-gray">delete_forever</md-icon></md-button>      </md-table-cell>  </md-table-row>
Remove row method:
removeRow: function (index) { this.filter.rows.splice(index, 1); }
When I use md-button, table loading is slow. Table is not responsive when I hover the rows, it takes approximately 4-5 sec until I can see that colors are changing when I hover them. But when I use a regular button instead, it takes less then 1 sec.

Can you provide an Codepen example?

http://codepen.io/anon/pen/zNLJwv
Here it is.

I couldn't reproduce this behaviour in your code. I even change the list to 200 items and the performance was acceptable.

I'm closing this issue. Probably the bug was fixed on later release. Thank you @mzupancic19 !
Feel free to reopen this issue if you like.


Add call subcommand

Closes #1
It can be used like this:
$ extism call mycode.wasm function_name --input data

or
$ echo data | extism call mycode.wasm function_name

getting this error after pip3 install . from the root and running the call cmd:
extism call --log-level debug ../extism/wasm/code.wasm count_vowels --input testing123
Traceback (most recent call last):
  File "/opt/homebrew/bin/extism", line 8, in <module>
    sys.exit(main())
  File "/opt/homebrew/lib/python3.10/site-packages/extism_cli/__init__.py", line 513, in main
    extism.set_log_level("-", args.log_level)
AttributeError: module 'extism' has no attribute 'set_log_level'

tried --log-level in different positions too.. I must be doing something wrong

@nilslice - thanks, fix it - should have been set_log_file


Running falco-no-driver container with least privilege after falco-driver-loader fails to create process list

Receive the following when running falco-no-driver container on Ubuntu 18.04 LTS host:
$ docker run --rm -i -t --cap-add SYS_PTRACE $(ls /dev/falco* | xargs -I {} echo --device {}) --pid=host -v /var/run/docker.sock:/var/run/docker.sock falcosecurity/falco-no-driver:latest

2020-06-10T10:02:14+0000: Falco initialized with configuration file /etc/falco/falco.yaml
2020-06-10T10:02:14+0000: Loading rules from file /etc/falco/falco_rules.yaml:
2020-06-10T10:02:15+0000: Loading rules from file /etc/falco/falco_rules.local.yaml:
2020-06-10T10:02:15+0000: Loading rules from file /etc/falco/k8s_audit_rules.yaml:
2020-06-10T10:02:15+0000: Unable to load the driver. Exiting.
2020-06-10T10:02:15+0000: Runtime error: error creating the process list. Make sure you have root credentials.. Exiting.

After running the falco-driver-loader image as described in the Falco documentation under "Least Privilege" I get the following which indicates the kernel driver successfully installed:
$ docker run --rm -i -t \
>     --privileged \
>     -v /root/.falco:/root/.falco \
>     -v /proc:/host/proc:ro \
>     -v /boot:/host/boot:ro \
>     -v /lib/modules:/host/lib/modules:ro \
>     -v /usr:/host/usr:ro \
>     -v /etc:/host/etc:ro \
>     falcosecurity/falco-driver-loader:latest
* Setting up /usr/src links from host
* Running falco-driver-loader with: driver=module, compile=yes, download=yes
* Unloading falco module, if present
* Trying to dkms install falco module

Kernel preparation unnecessary for this kernel.  Skipping...

Building module:
cleaning build area...
make -j1 KERNELRELEASE=5.3.0-59-generic -C /lib/modules/5.3.0-59-generic/build M=/var/lib/dkms/falco/96bd9bc560f67742738eb7255aeb4d03046b8045/build....
cleaning build area...

DKMS: build completed.

falco.ko:
Running module version sanity check.
 - Original module
   - No original module exists within this kernel
 - Installation
   - Installing to /lib/modules/5.3.0-59-generic/kernel/extra/

depmod....

DKMS: install completed.
* falco module installed in dkms, trying to insmod
* Success: falco module found and loaded in dkms

I then run the no-driver variant and get the error described at the top of this post.
In short I followed the instructions here:
https://falco.org/docs/running/

Falco version: 0.23.0
System info: See below
Cloud provider or hardware configuration: Virtualbox 6.1, 1 CPU, 2GB RAM, 30GB disk space
OS:

NAME="Ubuntu"
VERSION="18.04.4 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 18.04.4 LTS"
VERSION_ID="18.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic

Kernel:

Linux tester-VirtualBox 5.3.0-59-generic #53~18.04.1-Ubuntu SMP Thu Jun 4 14:58:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Installation method:
Official Docker containers

I'm testing things out at the moment but I'd like to run this in Kubernetes with least privilege. I've tried running htop in a container with the --pid=host flag and it works fine. If I remove this flag in the command to run Falco no-driver it works. Adding the flag back results in the error returning regarding the process list.

/cc @fntlnz

Same problem for us:
falco --version: 0.20.0+d77080a
uname -a: Linux 4.19.0-9-cloud-amd64 #1 SMP Debian 4.19.118-2+deb10u1 (2020-06-07) x86_64 GNU/Linux
2020-07-01T20:26:20+0000: Falco initialized with configuration file /etc/falco/falco.yaml
2020-07-01T20:26:20+0000: Loading rules from file /etc/falco/falco_rules.yaml:
2020-07-01T20:26:20+0000: Loading rules from file /etc/falco/falco_rules.local.yaml:
2020-07-01T20:26:20+0000: Loading rules from file /etc/falco/k8s_audit_rules.yaml:
2020-07-01T20:26:21+0000: Unable to load the driver. Exiting.
2020-07-01T20:26:21+0000: Runtime error: error creating the process list. Make sure you have root credentials.. Exiting.

Same problem for us:
falco --version: 0.20.0+d77080a
uname -a: Linux 4.19.0-9-cloud-amd64 #1 SMP Debian 4.19.118-2+deb10u1 (2020-06-07) x86_64 GNU/Linux
2020-07-01T20:26:20+0000: Falco initialized with configuration file /etc/falco/falco.yaml
2020-07-01T20:26:20+0000: Loading rules from file /etc/falco/falco_rules.yaml:
2020-07-01T20:26:20+0000: Loading rules from file /etc/falco/falco_rules.local.yaml:
2020-07-01T20:26:20+0000: Loading rules from file /etc/falco/k8s_audit_rules.yaml:
2020-07-01T20:26:21+0000: Unable to load the driver. Exiting.
2020-07-01T20:26:21+0000: Runtime error: error creating the process list. Make sure you have root credentials.. Exiting.

IIRC, falco-no-driver was introduced since 0.23.0, so I'm not sure how you installed the driver to work with 0.20.0+d77080a. Could you share more details?

@leogr Thanks for pointing that out - indeed, we had to install Falco through the official installation script on Debian, which resulted in this version being installed.
We had to that because at first, we tried to load it through falco-driver-loader image, but that's failing for another reason (https://github.com/falcosecurity/falco/issues/1239).
So I'm not sure how we can proceed with falco-no-driver if we can't get falco-driver-loader to work first..

Hi @dinvlad
You need both falco-driver-loader and falco-no-driver to be the version 0.23.0 (IIRC the driver version changed from 0.20.0 to 0.23.0). You can find the .deb of the 0.23.0 :point_right:  here (it's the official package).
Regarding the need of --pid=host, AFAIK it may depend on the system, so in your case might be correct to remove --pid=host (as same as @DrkDigglr did, could you confirm that everything works without it?). Please, note that I'm not 100% sure about that and I haven't tested this case deeply, so do not consider my suggestion as "official" :)

@DrkDigglr that error usually happens when Falco can't traverse and read the proc filesystem mounted at /proc/.
That is why we need to do --cap-add SYS_PTRACE in the least privileged scenario.
I think that there's something going wrong with the command or we need to figure out if there is anything specific in your system preventing this.
First things first, can you try this command?
docker run --cap-add SYS_PTRACE   --pid=host -it alpine cat /proc/1/environ

It should output something like
TERM=linux/
``

Interestingly, the command I posted above does not work for me on Ubuntu 18.04
docker run --cap-add SYS_PTRACE   --pid=host -it alpine cat /proc/1/environ
cat: can't open '/proc/1/environ': Permission denied

First suspect, apparmor:
[ 2055.977977] audit: type=1400 audit(1593780613.624:163): apparmor="DENIED" operation="ptrace" profile="docker-default" pid=10951 comm="cat" requested_mask="trace" denied_mask="trace" peer="unconfined"

Yes it is apparmor
docker run --security-opt apparmor:unconfined  --cap-add SYS_PTRACE   --pid=host -it ubuntu cat /proc/1/environ

HOME=/init=/sbin/initNETWORK_SKIP_ENSLAVED=recovery=TERM=linuxdrop_caps=BOOT_IMAGE=/boot/vmlinuz-4.15.0-101-genericPATH=/sbin:/usr/sbin:/bin:/usr/binPWD=/rootmnt=/root

So in the case of Ubuntu, there are two ways:

We write a specific apparmor profile to allow the read of /proc and then pass it to the Falco least privileged container
We suggest to use --security-opt apparmor:unconfined

I confirm that by using apparmor unconfined I can get the least privileged Falco to work without giving full privileges.
docker run --security-opt apparmor:unconfined --rm -i -t --cap-add SYS_PTRACE $(ls /dev/falco* | xargs -I {} echo --device {}) --pid=host -v /var/run/docker.sock:/var/run/docker.sock falcosecurity/falco-no-driver:latest
2020-07-03T12:56:54+0000: Falco initialized with configuration file /etc/falco/falco.yaml
2020-07-03T12:56:54+0000: Loading rules from file /etc/falco/falco_rules.yaml:
2020-07-03T12:56:54+0000: Loading rules from file /etc/falco/falco_rules.local.yaml:
2020-07-03T12:56:54+0000: Loading rules from file /etc/falco/k8s_audit_rules.yaml:
2020-07-03T12:56:55+0000: Starting internal webserver, listening on port 8765

Hey,
Has this issue been fixed by https://github.com/falcosecurity/falco-website/pull/225 ?
Could anyone confirm?


Add cache argument to zn.metrics and zn.plots

You can now use cache to define if the results should be tracked via GIT or with DVC (cache=True).
These would be the default values:
class MyNode(Node):
   plots = zn.plots(cache=True)
   metrics = zn.metrics(cache=False)

Can be merged after on #411. Include in #422


fix: unit test cases added

User Story : https://dev.azure.com/CSACTOSOL/CSA Solutioning/_workitems/edit/7772
As part of this story, we have configured jest

QuestionInput
Layout
SpeechToText

The test coverage report as follows:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

What to Check
Verify that the following are valid

...

Other Information

closing it


In the main files, dynamically append the path so that we can use the same file locally as well as on the ATOS QLM.

Problem:

At the moment, we can only run code when we are on top-level of the repo.

Solution:

We just append to the filepath.

Alternative:

Build and install before each run.

Benefit:

We do not have to duplicate main files.

Notes:

This turned out to be far more complex than anticipated.

Solution:

What we can do instead is to make pulse_opt a package.
Then one can install it and use it as library without thinking about the paths.


[Bug Report] Images higher than 64x64 cause an exception

Describe the bug
Images larger than 64x64 cause an exception at start up
To Reproduce
Steps to reproduce the behavior:

Place an image larger than 64x64 in the "icons" folder
Use it in an MOTD
Restart the server
See error

Expected behavior
The image should either be scaled or not loaded and an error or warning should be logged
Additionally a note in the configuration file should be added to state that images larger than 64x64 are not supported
Server Software and Version:

Vendor: BungeeCord
Minecraft Version: N/A
Output of /version command: This server is running BungeeCord version git:BungeeCord-Bootstrap:1.19-R0.1-SNAPSHOT:02548c4:1663 by md_5

Unable to reproduce, I get [11:50:45 WARN] [MiniMOTD]: Could not load sample-png-images-3.png: image must be 64x64px.


Milestone #3 Feedback

GitHub actions workflow for continuous integration for the Python project: Mechanics
Comments
ci and cd are working but the readme bagde is missing
GitHub actions workflow for continuous deployment for the Python project: Mechanics
Comments
ci and cd are working but the readme bagde is missing
Python package documentation: Reasoning
Comments
Readthedocs.org documentation is not present or it is not possible to know so because the link it was not provided clearly in the readme or as a link
Specific expectations for this milestone: Mechanics
Comments
Song Yang only created 1 commit in Python repo

Thank you for this Feedback!


Add support for text tracks

This creates a 'player technology' (tech) that extends the default HTML5 tech and then replaces the HTML5 tech. By extending the HTML5 tech, we can have it properly emit events that trigger text track functionality in VideoJS.
closes #36

Wow, this is huge, thanks, I will take a look!

You are welcome. Thank you for merging!

@mfairchild365 I read your extensive commit messages, and you're basically saying most of the logic should move to a tech? I'm not so familiar with those (and this plugin started out as a 'hack' to re-use the video.js controls with wavesurfer.js tech) but if that's the recommended way to implement stuff we should look at that.

and is it possible to enable the captions at startup? the example now starts before you have time to click CC btn.

Yep, thats how I understand it. The 'tech' should handle the playback, while the plugin should change the interface/controls. I think it would be best to move most of the functionality into the tech rather than the plugin. Unfortunately, I didn't have time do that much work, so I made it work with a minimal number of changes. Perhaps it could be revisited in the future. Full disclosure: I'm honestly not very familiar with videoJS (this is my first time working with it).
To auto-show the captions, you can add the default attribute to the caption track that you want to display.
<track kind="captions" src="sample.vtt" srclang="en-US" default />

When this plugin was created (around video.js 4.x) audio was not supported by video.js so I took this approach. Now that video.js does support audio this plugin could be refactored a little. As long as we prevent that the audio file has to be loaded twice, once for video.js and once for wavesurfer.js.


added a text file

@alasalle

Just a note for futureassignments - a good title format for your PRs: Project Name | Your Name.


Debug RWS-HWS Model

[ ] Fix Errors:

Sluizen Panheel, target-level Kanaal Wessem-Nederweert = 28.65
Noordervaart; reverse flow-direction
Fix target of Sluis Panheel (kanaal Wessem-Nederweert, not Meuse)

[x] To Outlet:

Sluis 6 @ Zuid-Willemsvaart (min_level = 15, capacity = 18 m3/s (factsheet operational water management))
Sluis 1 @ Wilhelminakanaal (min_level = 15 (has to be 5.15 after including Sluis IV!), capacity = 40 m3/s (factsheet operationeel water management))
Sluis Empel @ Máximakanaal (min_level = 2), capacity = 18 m3/s (factsheet operational water management))
Sluis Eefde @ Twentekanaal (min_level = 10), capacity 30 m3/s (schatting op basis factsheet owm)

Required functions

[x] clip model by polygon
[ ] update state by polygon
[ ] remove node
[x] change node type
[ ] volume-courant
[x] reverse flow direction

https://github.com/Deltares/Ribasim/pull/1393 may be relevant for DiscreteControl related issues.


sorted by timestamp in directory lookup can cause faulty ranges

Opening data in an intermagnet format of a certain observatory by choosing a directory proposed in our cleaned dataset results almost always in an incorrect timeframe.
The reason for this is that in the directory the cleaned files didn't have timstamps that followed the order of the data that they contained. Looking into the code
https://github.com/geomagpy/magpy/blob/ae91462f1644ebf89fbe570ce1cf67d1d89eb926/magpy/gui/magpy_gui.py#L2234
The key is the timestamp of the file which is not ok for us. If we eliminate the key the order will be alphabetic order of the filenames. Because most ( all ?) of the intermagnet/iaga standard naming of files will be ok with the alphabetic order it seems to be more appropriate to do so.

Done in 9d719f8b75f1b868898398c04e315058213c0823


replace GNU gsl by lapack

We are just using qr solve, which also available via lapack.
And lapack is also available on HPC machines.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 7 Dec 2010 at 3:44

Blocked on: #115

@symashayak : Are we using any gsl calls in csg_re?

yes, it uses gsl for
votca::tools::linalg_cholesky_solve()
On Tue, Sep 27, 2016 at 8:46 AM, Christoph Junghans <
<EMAIL_ADDRESS>wrote:

@symashayak https://github.com/symashayak : Are we using any gsl calls
in csg_re?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/votca/csg/issues/32#issuecomment-249869378, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADu1NvXXvf-xIWjx_s3YuUHXKgkOMeaeks5quR4ugaJpZM4KHr0g
.

Yeah,
but we can use a direct LAPACK call there, in tools al the gsl calls would
have to be changed.
2016-09-27 18:39 GMT+02:00 Sikandar Y. Mashayak<EMAIL_ADDRESS>
yes, it uses gsl for
votca::tools::linalg_cholesky_solve()
On Tue, Sep 27, 2016 at 8:46 AM, Christoph Junghans <
<EMAIL_ADDRESS>wrote:

@symashayak https://github.com/symashayak : Are we using any gsl calls
in csg_re?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/votca/csg/issues/32#issuecomment-249869378, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ADu1NvXXvf-xIWjx_
s3YuUHXKgkOMeaeks5quR4ugaJpZM4KHr0g>
.

—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/votca/csg/issues/32#issuecomment-249922273, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ALk6afYqg8HFaKH3hnrqj8u8fJDf5s5sks5quUangaJpZM4KHr0g
.

Switching to Eigen


Client functionality of bind all events

What kind of change does this PR introduce?
A proposal on reducing traffic twice by implementing the "bind all events" only on the client. No need more send events with "*", after that fix, you could remove RealtimeWeb.Endpoint.broadcast_from!(self(), topic, "*", txn) in the RealtimeWeb.RealtimeChannel.handle_realtime_transaction
What is the current behavior?
In current behavior, the server always sends two messages, were different only in event name. On the client-side simple check matching by event name.
What is the new behavior?
The new behavior can bind all events where the event name equals type in the payload.
Example of current event
{
  "event": "INSERT",
  "payload": {
    "columns": [],
    "commit_timestamp": "2021-02-12T16:03:31Z",
    "record":{},
    "schema": "public",
    "table": "users",
    "type": "INSERT"
  },
  "ref": null,
  "topic": "realtime:*"
}

Additional context
https://github.com/supabase/realtime/issues/119#issuecomment-777783037

Hi @abc3

No need more send events with "*"

I'm not entirely sure this will work, but perhaps I am misunderstanding.
The "*" is like a "room" which the client can join to listen to "hear events". The server will "shout events" into a room, and any client in that room with hear it. If we remove this, then the client will have to join every room to hear the all events, rather than one room to hear all events.
FOr example, if I have 2 tables in my public schema, products and prices, and then I wanted to know what is happening anywhere in my database, the client would have to connect to:

Room 1: realtime:public:products
Room 2: realtime:public:prices

Either the developer or this library would have to know about all the tables that exist in the database so that it can connect to all the "rooms"
With the * implementation they just connect to a single room, and even if new tables are connected to their database while they are connected they will still receive the events

Now I don't understand :smile:
You probably speak about * in addChannel(). Right?
I mean * in on(). During one event, the server always sends two similar messages where different only event names. It's possible to send it once and handle * on the client side.
For example, if now inserted a row in the users table:

{
  "event": "*",
  "payload": {
    "columns": [
      {
        "flags": [
          "key"
        ],
        "name": "id",
        "type": "int8",
        "type_modifier":<PHONE_NUMBER>
      },
      {
        "flags": [],
        "name": "name",
        "type": "text",
        "type_modifier":<PHONE_NUMBER>
      }
    ],
    "commit_timestamp": "2021-02-13T06:16:42Z",
    "record": {
      "id": "108",
      "name": "John Smith"
    },
    "schema": "public",
    "table": "users",
    "type": "INSERT"
  },
  "ref": null,
  "topic": "realtime:public:users"
}

{
  "event": "INSERT",
  "payload": {
    "columns": [
      {
        "flags": [
          "key"
        ],
        "name": "id",
        "type": "int8",
        "type_modifier":<PHONE_NUMBER>
      },
      {
        "flags": [],
        "name": "name",
        "type": "text",
        "type_modifier":<PHONE_NUMBER>
      }
    ],
    "commit_timestamp": "2021-02-13T06:16:42Z",
    "record": {
      "id": "108",
      "name": "John Smith"
    },
    "schema": "public",
    "table": "users",
    "type": "INSERT"
  },
  "ref": null,
  "topic": "realtime:public:users"
}

The presence of the channel.on('*', msg => {}) in the code does not influence behavior, the server will always send two events. Traffic is increasing twice with each event.

Ah I see - this is not joining channels, but the events themselves. So instead of receiving events for *, INSERT, UPDATE and DELETE it will only receive INSERT, UPDATE and DELETE, then the * is handled client-side.
It think that makes sense. @soedirgo @w3b6x9 - this seems fine right? Shouldn't cause any breaking changes if we roll out these client side changes before removing this line
https://github.com/supabase/realtime/blob/af6344c7746e8a8af6a11a9b498721c1f97e339b/server/lib/realtime_web/channels/realtime_channel.ex#L22

+1, I've always felt like we can omit the * notification from the server. Shouldn't be a breaking change since * events are just ignored in this case.

@kiwicopple yep, this looks fine!
I think we'll have to merge this in, version bump realtime-js in supabase-js, and wait a bit for users to upgrade their clients before removing the broadcast_from! in realtime right?
I'll go and make a draft PR in realtime so we don't forget to do it later on.

:tada: This PR is included in version 1.0.7 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Building shared tensorflow_cc

I'm trying to build shared lib but i get this error:
cp: cannot copy a directory, '/tensorflow_cc/tensorflow_cc/build/tensorflow/tensorflow/contrib/makefile/gen/protobuf', into itself, './tensorflow/contrib/makefile/gen/protobuf/protobuf
CMakeFiles/tensorflow_shared.dir/build.make:106: recipe for target 'tensorflow-stamp/tensorflow_shared-configure' failed
make[2]: *** [tensorflow-stamp/tensorflow_shared-configure] Error 1
CMakeFiles/Makefile2:104: recipe for target 'CMakeFiles/tensorflow_shared.dir/all' failed
make[1]: *** [CMakeFiles/tensorflow_shared.dir/all] Error 2
Makefile:127: recipe for target 'all' failed
make: *** [all] Error 2

Hi, it seems that the library is trying to be installed to the current working directory. Have you used only cmake .. or have you used something as cmake -DCMAKE_INSTALL_PREFIX=... ?

Hi, i only used cmake -DTENSORFLOW_STATIC=OFF -DTENSORFLOW_SHARED=ON ..

Strange, I am not really sure where this could happen. Can you tell me your operating system version (uname -a) and the exact sequence of commands you have done?

Output of uname -a : 4.4.0-96-generic #119-Ubuntu SMP Tue Sep 12 14:59:54 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
Commande sequence: cmake -DTENSORFLOW_STATIC=OFF -DTENSORFLOW_SHARED=ON  ..
make && sudo make install

Also have this bug

I still do not know from where does the issue originate. :-( Can you try whether it also happens in the case of  building the static library?
(just cmake ..)
I will rebuild the Dockerfile tests to find out whether it is not because of a new version of some package.

I have made some improvements to the library. Can you please make a clean clone of the project and try over? Beware that the list of requirements has slightly changed, now g++-6 is used instead of g++-5, see the README.md.
In the new version, all tests are green and TensorFlow 1.4.0-rc0 is used.

I'm encountering this error after a clean clone and working through the issues in #27. I actually get a build failure and need to manually move a line in cuda/cuda_dnn.cc in order to get tensorflow to compile through bazel, but perhaps completing the build with an existing build folder is the issue here? I'll try editing one of your scripts to get the change to happen automatically.

So the issue I have with cuda/cuda_dnn.cc is resolved by applying this fix https://github.com/tensorflow/tensorflow/pull/13255.
I added the line
	    COMMAND printf %s\\n 235m247 w q | ed -s tensorflow/stream_executor/cuda/cuda_dnn.cc

After the other COMMAND lines in tensorflow_cc/cmake/TensorflowBase.cmake.

Hi, have you solved the issue? currently I also met with the issue as below:
Target //tensorflow:libtensorflow_cc.so up-to-date:
  bazel-bin/tensorflow/libtensorflow_cc.so
INFO: Elapsed time: 14.921s, Critical Path: 5.78s
cp: cannot copy a directory, '/tmp/repo/tensorflow_cc/tensorflow_cc/build/tensorflow/tensorflow/contrib/makefile/gen/protobuf', into itself, './tensorflow/contrib/makefile/gen/protobuf/protobuf'
make[2]: *** [CMakeFiles/tensorflow_shared.dir/build.make:112: tensorflow-stamp/tensorflow_shared-configure] Error 1
make[1]: *** [CMakeFiles/Makefile2:68: CMakeFiles/tensorflow_shared.dir/all] Error 2

Hi, I have not been able to reproduce the issue yet. It looks as if the CMake install prefix is set to the current working directory instead of /usr/local. Can you please try to configure the project using
cmake -DTENSORFLOW_STATIC=OFF -DTENSORFLOW_SHARED=ON -DCMAKE_INSTALL_PREFIX=/usr/local ..

That is, explicity setting the install prefix? Thanks.

@FloopCZ  Explicitly adding CMAKE_INSTALL_PREFIX solved the the "cannot copy a directory" on Ubuntu 16 for me.

@hyperdrive Thank you for the information, I will have to put this information to the README.md. I still wonder how is it possible that for some installations, the default cmake install prefix is set to the current workdir, though.

I have finally found and fixed the real source of the issue in the latest master.


Feature/#6: deck logs

This is the  basic implementation of the command deck logs. Howerver I'm not sure how to implement the "follow" option on a deck level. It is possible to follow logs of one pod by using unikube app logs -f.
Is the follow option required for decks?

That is a nice feature.


[Suggestion] More defined recipe removal for Squeezer

Same as the Fermenter, the recipe removal is rather unprecise at it uses the OuputFluid.
The same suggestion would go for the Fermenter, however only this would be sufficient for recipe removal in the Squeezer:
InputStack

InputStack indeed.

not possible as the squezzer uses an array

Then InputArray! :'D

not possible, sorry


Build app as webapp and deploy as GitHub pages

Flutter is able to build the app as web application (https://flutter.dev/docs/get-started/web). We can deploy that one automatically as GitHub page for the repository using CI.

Tried it, but it did not work properly just yet. Maybe we should try it later again.


feat: template pulling with npm config

test_output.tar.gz
Description
Use npm config values in arborist in order to inherit form the global config when pulling a template.
This allow to pass config like private registry and proxy-auth.
Related issue(s)
https://github.com/asyncapi/generator/issues/538

Hello,
The indent was not to make a dupplicate of https://github.com/asyncapi/generator/pull/1061 , I was scripting the generator at work this week and been trying to work around the limits of the generator (our ci has no direct acces to the web and to ways : private repository, http proxy with auth). Both are ok with npm because we can config it but not with the generator.
Maybe this PR can help to find a convenient way of using the generator as easily that npm by reusing the same source of configuration.

@asyncapi/bounty_team

@pierrick-boule as I wrote before. This https://github.com/asyncapi/generator/pull/1061 is already merged, so please update your PR with latest private npm support

Hello @derberg, i updated this PR, was able to test localy :
I put "<IP_ADDRESS> registry.npmjs.org" in /etc/hosts to make it unavailable (to remove later)
set a private registry :
npm config set registry https://private.registry.org/
Run few test OK
used a docker proxy https://hub.docker.com/r/thelebster/docker-squid-simple-proxy/
npm config set proxy http://username:password@localhost:3128
Run few test OK

Hello, i fixed the code style.
The backwards compatibility with nodeJS 12 is not that easy to get, i will need more time to check the old npm code :-( .

Hello,
Yes, I tried few implementation of npm config of npm 12 and it's very different of this PR, it will be verbose to implement a retrocompatible version and not very clear/maintainable, i was also thinking about depreciating it, and maybe introduce the new LTS of node (20) if possible.
I will now focus on this way (taking your proposals).

@pierrick-boule we should support what Node community supports, so minimum is Node 18 that is in maintenance until April next year -> https://nodejs.org/en/about/previous-releases
since we agreed we will run a major release with this PR:

please update title to feat!
lets refresh node support so make sure package.json and docs talk about Node 18 as minimum
make sure testing is also done Node 18 and 20
make sure package-lock is using lockfileVersion set to 3

@pierrick-boule whenever you're ready for another review round please rerequest review as I do not get notifications that you pushed all required changes.
also please solve conflicts with package-lock.json

Hello, @derberg it should be ok now.

/rtm

@all-contributors please add @pierrick-boule for code,test,doc


Separate the code into modules

This is a follow-up to #27 which separates the functionality into modules.
Aside from modularization benefits, it will enable us to import types and other functionality for testing.

Actually I'm going to skip this PR and include the change in the next one


PyDruid is not actually optional

When creating a Docker image from setup.py, pydruid does not end up being installed.
This is legitimate as it is an optional dependency as stated in setup.py but also the code  in superset/connectors/druid/models.py seems to be making the effort to allow the interpreter to load the module even if druid is not installed.
Expected results
Apache Superset can be installed without PyDruid installed as hinted by setup.py
Actual results
Apache Superset fails to start.
Screenshots
Truncated superset entrypoint logs:
2019-10-22 12:18:58,192:DEBUG:asyncio:Using selector: EpollSelector
2019-10-22 12:18:58,212:DEBUG:asyncio:Using selector: SelectSelector
2019-10-22 12:18:58,341:DEBUG:asyncio:Using selector: EpollSelector
2019-10-22 12:18:58,395:DEBUG:asyncio:Using selector: EpollSelector
[2019-10-22 12:18:59 +0000] [32] [ERROR] Exception in worker process
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/gunicorn/arbiter.py", line 583, in spawn_worker
    worker.init_process()
  File "/usr/local/lib/python3.6/site-packages/gunicorn/workers/base.py", line 129, in init_process
    self.load_wsgi()
  File "/usr/local/lib/python3.6/site-packages/gunicorn/workers/base.py", line 138, in load_wsgi
    self.wsgi = self.app.wsgi()
  File "/usr/local/lib/python3.6/site-packages/gunicorn/app/base.py", line 67, in wsgi
    self.callable = self.load()
  File "/usr/local/lib/python3.6/site-packages/gunicorn/app/wsgiapp.py", line 52, in load
    return self.load_wsgiapp()
  File "/usr/local/lib/python3.6/site-packages/gunicorn/app/wsgiapp.py", line 41, in load_wsgiapp
    return util.import_app(self.app_uri)
  File "/usr/local/lib/python3.6/site-packages/gunicorn/util.py", line 350, in import_app
    __import__(module)
  File "/home/superset/superset/__init__.py", line 241, in <module>
    ConnectorRegistry.register_sources(module_datasource_map)
  File "/home/superset/superset/connectors/connector_registry.py", line 37, in register_sources
    module_obj = __import__(module_name, fromlist=class_names)
  File "/home/superset/superset/connectors/druid/__init__.py", line 17, in <module>
    from . import models  # noqa
  File "/home/superset/superset/connectors/druid/models.py", line 121, in <module>
    class DruidCluster(Model, AuditMixinNullable, ImportMixin):
  File "/home/superset/superset/connectors/druid/models.py", line 173, in DruidCluster
    def get_pydruid_client(self) -> PyDruid:
NameError: name 'PyDruid' is not defined
[2019-10-22 12:18:59 +0000] [32] [INFO] Worker exiting (pid: 32)

How to reproduce the bug

Build a Docker image using:

RUN pip install --no-cache-dir \
	-r requirements.txt \
	celery[redis,gevent] \
	statsd==3.3.0 \
	-e .[hive,postgres]

instead of the current line in repository.
2. Try to run the image
3. See error with docker logs
Environment

superset version: Image built from git commit 03b35b3c1121bb452a7938ed92c6bede657377dd
python version: Python 3.6.9
node.js version: v10.16.3
npm version: 6.9.0

Checklist
Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Hi @EvaSDK,
Thank you for reporting this!
This is already fixed on master and should be included on superset's next release 0.35.1


Addressing issue #300

Fixes #300 locally.
(We should have a test for the correct creation of the html pages.)

The tests pass, so hopefully this has fixed the issues.

Well, nice that it works, but we also need to pin the rest. Otherwise quite soon something else will screw up because of incompatibility with that particular jinja2 version :)

Well, nice that it works, but we also need to pin the rest.  I have created an issue already. https://github.com/oscovida/oscovida/issues/301


Do not alter recipient substitution data keys

With this fix, both recipient-specific substitution data  and template substitution data (which was fixed with #52) should both be working as expected.

This should resolve #54


Add a gradio demo for the project

Thanks, we have updated the gradio demo; please check it out here: https://huggingface.co/spaces/toshas/marigold


Runtime error when casting a 'any' to 'string' in class scope

Description:
$title
Steps to reproduce:
public class Counter {

    private int n;
    any x = 12;
    int k = <int> x; // error: casting is not supported from 'string' to 'int'
    public function init(int n = 0) {
        self.n = n;
    }
}

public function main() {
    Counter count = new(10);
}

error: error: casting is not supported from 'string' to 'int'
Following program compiles without an error
public class Counter {

    private int n;
    any x = function() {};
    string k = <string> x;
    public function init(int n = 0) {
        self.n = n;
    }
}

public function main() {
    Counter count = new(10);
}

Affected Versions:
Ballerina Swan Lake Beta 6

Depends on https://github.com/ballerina-platform/ballerina-lang/issues/22192. Both should result in compilation errors anyway.


Allows type guards with switch statement (need help..)

Hi. This PR resolves issue #2214 allowing the following code to work
function logNumber(v: number) { console.log("number:", v); }
function logString(v: string) { console.log("string:", v); }

function foo1(v: number|string) {
    switch (typeof v) {
        case 'number':
            logNumber(v);
            break;
        case 'string':
            logString(v);
            break;
        default:
            throw new Error("unsupported type");
    }
}

I need some help (I'm kinda new to this..)

TypeOfExpression doesn't have a member "text" but the Javascript equivalent does. What can I cast TypeOfExpression to to get access to "text"?

I'm not sure where I should be writing tests.. (Or what a .types file is)

This might be too permissive. In the previous example, if I were to have a case 'Watermelon' it would still work. Should I check to see that the type is a subset of v's type?

Thank you!

Hi @BSteephenson, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;


Plans for extending the features?

Thanks for this project Ryan!
I was wondering if you are planning to extend the feature set or are open to consider feature requests?
With that I mean features available in the Playground example of lexical.
Thanks!

Hi! I have no plans to extend this but I'm happy to review Pull Requests.


Different versions of axiom Q8, and lemmas 31.22-23

In chapter 29 (931aeb0 debug version page 415), Q8 of system Q (Robinson arithmetic) is listed as:
(Q8-A) ∀x∀y(x < y ↔ ∃z(x+z' = y))
But in chapter 32, same version page 440
(Q8-B) ∀x∀y(x < y ↔ ∃z(z'+x) = y)
The difference seems trivial, I know,  but since Robinson arithmetic does not have a general commutativity of addition principle, it affects proofs.
In particular, I'm confused about Lemmas 31.22 and 31.23 on page 452. For part 2 of 31.22, only the base step of the induction is included in the text, and I cannot for the life of me figure out how to get the induction step to work using (Q8-B). I can get it work with (Q8-A), but if I use (Q8-A), the proof given for 31.23 does not work, and I cannot figure out how to recover it, at least not in weak Q. (In PA, it's not a problem.)
For the project, I'd suggest making the two definitions consistent, and then either filling out the proof of 31.22 so it's complete or revising the proof of 31.23, and indeed, I'd be very grateful if someone could point me towards an alternative text that actually carries out these proofs, as at the moment, I have to admit defeat. (All the textbooks I've looked at either use a different axiomatization of Q, or leave them as an exercise!)
[EDIT: After a brief rest, I figured out the induction step for 31.22 using Q8-B, though it's far from trivial and I still think it should be added. I'm still completely in the dark of how one would get these lemmas using Q8-A, if it's even possible. I haven't had a chance to familiarize myself with your TeX files; when I get a chance, maybe I'll do a pull request.]

Thanks for this! I fixed the typo in ch 29 and will add a sketch of the induction step for Lemma 31.22(2).
Jeremy Avigad's notes follow Epstein & Carnielli, Computability. The lemma in question is Lemma 15 on p. 195 of that book, with a proof on the following page.

Ok, I think I fixed it. I split proposition 22 into two. The relevant changes are on p. 12 & 13.
In the process, I found a much simpler proof. Proposition 21 is no longer required and can be deleted. Can you have a look?
representability-in-q.pdf
(I also changed Q3 to a disjunction to make it work intuitionistically.)

Thanks again. The proof of req.23, or (what is now) 33.23 looks good to me now. I gave a longer proof of the biconditional version in my lecture notes, page 43, but for this direction it accords I believe.
There are some minor changes elsewhere that are needed. Referring to the latest clean version of the whole text, the wrong version of Q₈ is still given in Problems 24.3 and 24.4. And even more minor, the 8 in "Q8" is not subscripted where it appears near the end of section 24.4.


Queries hang in WebApi actions

Issue raised through NuGet e-mail.
I tried the following, and it works without problem:
public async Task<IEnumerable<string>> Get()
{
    var scanner = new Scanner(ConfigurationManager.AppSettings["apiKey"]);
    var fromPlace = (await scanner.QueryLocation("London")).First();
    var toPlace = (await scanner.QueryLocation("New York")).First();

    //Query flights
    var itineraries = await scanner.QueryFlight(
      new FlightQuerySettings(
        new FlightRequestSettings(
          fromPlace, toPlace,
          new LocalDate(2016, 06, 19), new LocalDate(2016, 06, 25)),
        new FlightResponseSettings(SortType.Price, SortOrder.Ascending)));

    var itinerary = itineraries.First();
    var estimatedPrice = itinerary.PricingOptions.Min(option => option.Price);

    //Query booking
    var booking = await scanner.QueryBooking(itinerary);
    var actualPrice = booking.BookingOptions
      .Select(option => option.BookingItems.Sum(item => item.Price))
      .Min();

    ...
}

I obviously haven't seen the original e-mail, but this sounds like a deadlock caused by the client code blocking on one of the Tasks returned by a SkyScanner API method. This will occur as you're not using ConfigureAwait(false) on any of the awaits in the current code base.
I.e. new Scanner(ConfigurationManager.AppSettings["apiKey"]).QueryFlight(flightSettings).Result will deadlock in Web API.
You won't reproduce this in a console or test project as they run with SynchronizationContext.Current == null.

Thanks @KirillShlenskiy. I tried to repro this in a WebAPI project and couldn't. But the  .ConfigureAwait(false) makes sense.

Hello,
this code in a console application hangs at line Dim itineraries = queryFlightTask.Result
Best regards,
Filippo Bottega.
Imports NodaTime
Imports SkyScanner.Services
Imports SkyScanner.Settings

Module Module1

  Sub Main()
    Dim scanner = New Scanner("APIKey"))
    Dim queryLocationTask = scanner.QueryLocation("Venezia")
    queryLocationTask.ConfigureAwait(False)
    Dim fromPlace = (queryLocationTask.Result).First()
    Console.WriteLine($"{NameOf(fromPlace)}: {fromPlace.ToString}")
    queryLocationTask = scanner.QueryLocation("London")
    queryLocationTask.ConfigureAwait(False)
    Dim toPlace = (queryLocationTask.Result).First()
    Console.WriteLine($"{NameOf(toPlace)}: {toPlace.ToString}")

    'Query flights
    Dim queryFlightTask = scanner.QueryFlight(New FlightQuerySettings(New FlightRequestSettings(fromPlace, toPlace, New LocalDate(2016, 12, 23), New LocalDate(2016, 12, 31)), New FlightResponseSettings(SortType.Price, SortOrder.Ascending)))
    queryFlightTask.ConfigureAwait(False)
    **Dim itineraries = queryFlightTask.Result**

    Dim itinerary = itineraries.First()
    Console.WriteLine($"{NameOf(itinerary)}: {itinerary.ToString}")

    Dim estimatedPrice = itinerary.PricingOptions.Min(Function([option]) [option].Price)
    Console.WriteLine($"{NameOf(estimatedPrice)}: {estimatedPrice.ToString}")

    'Query booking
    Dim queryBookingTask = scanner.QueryBooking(itinerary)
    queryBookingTask.ConfigureAwait(False)
    Dim booking = queryBookingTask.Result
    Dim actualPrice = booking.BookingOptions.[Select](Function([option]) [option].BookingItems.Sum(Function(item) item.Price)).Min()
    Console.WriteLine($"{NameOf(actualPrice)}: {actualPrice.ToString}")
  End Sub

End Module

I found this:
Which services can I access and what are the rate limits?
Please note access to our Hotels API and Flights Live Pricing service are currently suspended. Apologies, but we are unable to give a date for these services to be re-launched, but please do check back regularly for updates.
If I use my API key I get:
{
  "status": 429,
  "statusText": "OK",
  "debugInformation": {
    "ValidationErrors": [
      {
        "Message": "Rate limit has been exceeded: 0 PerMinute for PricingSession"
      }
    ]
  }
}

Maybe this is the reason of the application's hang ...
Filippo.

Thanks for investigating this further. You're right, I haven't tested this scenario. It's a shame that SkyScanner has shut down the service. I hope it's only temporary. I'll try to contact them to get more information.


Kickback Podcasts Grant

Kickback Podcasts
Grant category
Please select one or more of:

[ ] Open source maintenance
[ ✅ ] Developer tools / services
[ ] Educational material

Description
Kickback is a podcast platform that leverages cryptocurrency and blockchain technology. Through a decentralized, user-owned, crypto-based network, Kickback is revolutionizing the way podcasted are shared and monetized.
At Kickback, we want to revolutionize what podcasting platforms could look like for hosts and listeners alike by adding a plethora of revolutionary features such as dynamic/auto-paid out advertising, NFT fan clubs powered by NFTs, donations, and a weekly lottery for consistent listeners.
Problem statement
For the last three months, I've been leading the monetization team for the BanklessDAO Podcast Hatchery. While leading this team, we thought a lot about how we could create Web3 monetization options like NFT-powered fan clubs for our incubated podcasts. After creating the smart contracts and code for this, we realized just how powerful the technology was and how it could be used across the podcast industry to help fix the devastating monetization short comings - podcasts have over 500 million global listeners (on par with music), yet the industry makes 95% less than music.
Proposed solution
As we described in our description, we believe that many of the podcast industry's monetization shortcomings are due to companies not looking at what makes podcasts unique. Podcasting a widely unique form on content in which hosts are able to create massive fan bases that have people hanging on to their every word. Yet traditional platforms have focused on advertising solely to fund creators. We believe that podcast hosts can turn their passions into a full time career by tapping into their 1000 true fans by creating fan clubs/subscriptions powered by NFT technology.
Impact
While we believe NFT fan clubs for podcasts are massive. We can also see how the technology we're building could be used for many different things, such as sports NFT fan clubs, movie NFT fan clubs etc. Since our code is open source, we believe our technology can create NFT fan clubs across the whole ecosystem, no matter the genre.
Milestones and funding
Month 1 - Audit the smart contracts and refine the website ($10,000)
Month 2 - Launch the NFT fan clubs with our partner DAOs ($5,000)
Month 3 - Advertise the platform and continue to build out the private fan club features ($10,000)
Team

Name
Role
Bio
Contact

Alexander
Full-Stack Engineer, Designer, & Founder
5+ years in the audio industry, WashU CS + Business, Atlanta Inno 25 under 25, Head of monetization at the BanklessDAO Podcast Hatchery, Author of the History of Hip-Hop

<EMAIL_ADDRESS>

Grant
Product Manager
PM experience at Tinder, Inroads Scholar, WashU CS + Business

<EMAIL_ADDRESS>

Shruti
Full-Stack Engineer
Full Stack & Web3 Experience, Open Source Developer Certified through Google

https://twitter.com/ShrutiGavali2

Aimes
Full-Stack Engineer
Full Stack & Web3 Experience, Front End Web Developer for Varscon group

https://twitter.com/aimes_js

thanks for the submission, @AlexanderBZ. Is this enabling developers in some form? This grant program is designed to support developers directly by providing services/tools or education. Based on the initial read, I think there's a mismatch. Correct me if I'm wrong though.


Cannot read property of 'getAttribute' of null

Occurs in version 1.4.3 in drop.js, line 178
Uncaught TypeError: Cannot read property 'getAttribute' of null [drop.js:178]
The stack trace on my page starts with jQuery, into drop, and then DropInstance.
I did not think this was dependent on jQuery.
It is also failing on your demo page, when I try to "run" the first to blocks that have a run button.

So i dug into this a bit further which was not too hard, and realized I probably provided a bad selector for the target option.
However, the error handling for this is not firing.
if (typeof this.target === 'undefined') {
throw new Error('Drop Error: You must provide a target.');
}
This is in the latest version of google chrome.

you are likely have a Array of HTMLElements in the selector, not one.

Can this be closed?


suppress warning on Windows

Follow up of #13. Same as ros2/rmw_opensplice#250.

Since the patch is matching what has been applied and approved for clang before I will go ahead and merge this. Please feel free to comment after the fact with any feedback.


navigation templates and layouts

Guidelines and wireframes for application and site navigation

On hold.

Work moved to different cards. See: common navigation work.


BasePathMapping

Hello, I'm trying to use a custom domain but I have the following error:
I'm using example: basic-starter-api-gateway-v2-typescript/
CREATE_FAILED                         AWS::ApiGateway::BasePathMapping      MyApiBasePathMapping

Resource handler returned message:
Mixing of REST APIs and HTTP APIs
on the same domain name can only be
accomplished through API Gateway's
V2 DomainName interface. Currently,
WebSocket APIs can only be attached
to a domain name with other
WebSocket APIs. This must also
occur through API Gateway's V2
DomainName interface. (Service:
ApiGateway, Status Code: 400,
Request ID: 5187005f-3413-47bb-b777
-6c98784001d0

Hi, I have the same problem.
I'm using my own domain providers and therefore I commented the Route53 services.

Sounds like we need to upgrade to https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-apigatewayv2-domainname.html


Add geom3.toTriangles()

Expected Behavior
This functionality is duplicated in several of the IO modules, and would be best as part of the 3D geometry (geom3)
Actual Behavior
Each IO module has a slightly different version of toTriangles()
Steps to Reproduce the Problem

No issues.

Specifications

Version: V2
Platform: all
Environment: all

A general solution might make sense, but even more important would be the removal of T-junctions.

The latest release of V2 JSCAD contains the new generalize() function, which includes converting geom3 polygons to triangles.


fix: correct parsing of measured value in test

topic:fix-correct-parsing-of-measured-value-in-test

#
head
base
diff
date
summary

0
07befe26
286558ce
diff
Dec 27 21:18 PM
1 file changed, 1 insertion(+), 1 deletion(-)

Reviews in this chain:
└https://github.com/kaihowl/git-perf/pull/116 fix: correct parsing of measured value in test


Support for a horizontal rule in Markdown.

Adding the support for Markdown Feature.
The Markdown --- should get converted to an  tag

link

closed by 26a647d


Review table 3 column example HTML markup

What
Review and revise the 3 column table header example, to make use of table headings and scope. To create a proper relationship between table cells and their column and row headers.
Design System table component.

Why
To improve the accessibility of the tables for screen-reader users.

Other information to support your proposal
Highlighted in this comment in the gov.uk design system: https://github.com/alphagov/govuk-design-system-backlog/issues/61#issuecomment-1340840905
Further reading on using headers and scope: https://developer.mozilla.org/en-US/docs/Learn/HTML/Tables/Advanced.
Related to: https://github.com/nhsuk/nhsuk-service-manual-community-backlog/issues/26

Added in release: 6.5.0
Column headers used in two examples. Bug raised for responsive tables: https://github.com/nhsuk/nhsuk-frontend/issues/985


Added backup restoration support

See #75

Codecov Report

Merging #76 (124f654) into main (65e863b) will increase coverage by 0.13%.
The diff coverage is 83.33%.

@@             Coverage Diff              @@
##               main      #76      +/-   ##
============================================
+ Coverage     85.53%   85.66%   +0.13%
- Complexity      423      433      +10
============================================
  Files            62       62
  Lines          1514     1556      +42
============================================
+ Hits           1295     1333      +38
- Misses          219      223       +4

Impacted Files
Coverage Δ

src/Controller/Admin/BackupCrudController.php
71.92% <53.33%> (-6.65%)
:arrow_down:

src/Helper/FlysystemHelper.php
100.00% <100.00%> (ø)

src/Service/BackupService.php
78.65% <100.00%> (+7.43%)
:arrow_up:

src/Entity/Database.php
100.00% <0.00%> (+2.70%)
:arrow_up:

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


Update login.js

Corrected tiny grammar omission. Came across this when I ran
$ ionic upload

@mturkson23 Thanks for the heads up. We've manually merged this. Thanks!


Fix the ordering of bytes in pitch bend commands

I found your page while looking for a description on pitch bending for MIDI while I was writing a control script for my Launchpad X to be used with Bitwig Studio, and found this article really helpful, but spent quite a while figuring out why it was working in an unexpected way. As it turns out, the order of the significance of the bytes were reversed. :upside_down_face:
(References: https://www.midi.org/specifications-old/item/table-1-summary-of-midi-message and https://www.cs.cmu.edu/~music/cmsip/readings/davids-midi-spec.htm)
Thank you very much for sharing such educational content, I hope that you'll manage to merge my fix to avoid confusing others that might stumble upon this. :wink:

Thanks!


Feature/plato

Description
Add in Plato reporting on the quality of the code in the project, uses the Eslint config used elsewhere.
Works in a similar way to npm run cover you use npm runreport
Todos

Make sure you’ve completed these:

[x] Does this feature run on all supported platforms?
[ ] Are there tests around this feature?
[x] Is there documentation for this feature?

Current coverage is 95.34% (diff: 100%)

Merging #73 into develop will not change coverage

@@            develop        #73   diff @@
==========================================
  Files             2          2
  Lines            43         43
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits             41         41
  Misses            2          2
  Partials          0          0

Powered by Codecov. Last update 76d2130...1d80a28

@bmds Looks good, happy to pull this in. Are you able to look at the AppVeyor build?


use variable instead of hardcoded DB name

Signed-off-by: Bob Callaway<EMAIL_ADDRESS>

This reminds me that we should like stop carrying storage.sql and instead point users upstream to trillian, as they might make changes to schema and we fall out of sync.


Update update_deps.yml LCA

Auto-Full-URL=
https://support.google.com/code/contact/project_quota_increase?

It seems to me this is a mistake, so I'm closing this PR.
Could you explain what are you trying to achieve here. If reasonable, we can reopen.


base_url return server ip instead of server host

In CI 3.0.3 base_url return IP server instead of domain name. It cause broken link on shared hosting, or multiple domain in the some IP server.

Yes.

Is is correct behavior?

Yes it is, you better fill $config['base_url'] with your domain name.

Well you ever fix this? and restore the good easy way to not set the $config['base_url'] manually


Add nginx to docusaurus

commit docusaurus build result for reducing build time
add nginx container for static file server

deployed to dev and confirmed working.
lgtm


Link to TF Command line tool not working

I posted this over at Riders bug tracking first, but they sent me here.
When trying out Rider a few days ago, I needed to checkout a project from TFS using TFVC. This gave me a helpful popup that told me I needed to install the TF command line tool (se attached image). The message includes a link to https://docs.microsoft.com/en-us/vsts/java/Docs/tools/intelliJ#_tfvc-support that returns a 404.

I am facing the same issue in android studio 3.1.1 while connecting to TFS 2018 U1.

@jeschu1 Fixed in #136

Closing as fixed.


[import/no-duplicates] (prefer-inline) Does not detect across type and regular imports

I misread the no-duplicates doc to mean that when using the "prefer-inline" option, the following would be detected as an error:
import type { Type } from "mod";
import { value } from "mod";

And would fix to:
import { type Type, value } from "mod";

I realize now that this is not what is stated in the doc, but I still want this behavior. It is possible to implement, and from my perspective is probably what those using prefer-inline actually want. But if not, perhaps could be implemented with a new option, e.g. prefer-combined-type-imports

https://github.com/import-js/eslint-plugin-import/pull/2835

This took me by surprise too. The docs say:

In order to support fixing to an inline type import when duplicate imports are detected, prefer-inline can be set to true.

I find it hard to read this in any way other than how @benkrejci also interpreted it. Earlier in the same document a duplicate is defined as "have to point to the same module on the filesystem". Surely a 2-line import as posted above should classify as a "duplicate import" using this definition?
If I'm also reading this incorrectly, an edit to the documentation to clarify would be greatly appreciated. Thanks!

I definitely agree that when prefer-inline is set, only one from 'mod' should remain.

Since there seems to be some agreement here, I added tests and an example to the doc: https://github.com/import-js/eslint-plugin-import/pull/2835

@ljharb hey any idea when the next release will be? (Or maybe we could get an -alpha release 🙏). Happy to help if I can, thanks.

I don't do prereleases, and I don't have planned release dates, but I am hoping to get one done soon.


Bond 5.0

Does Bond 5.0 replace ReactiveUIKit?

@iandundas Right.

Binding extensions provided by ReactiveUIKit framework are now provided by Bond. Just import Bond instead of ReactiveUIKit and change extension prefixes from r to bnd

in Readme.

Thanks

Ps I did not see that change coming


about LSTM + BatchNormalization

Hi guys,
I am trying to add BatchNormalization layer on top of the lstm layer. However I am always get the error as follows:
Build graph of the model...
Traceback (most recent call last):
File "trans_bilstm_asr.py", line 124, in
model.add_node(BatchNormalization(), name='batch_norm_forward', input='embedding')
File "keras/layers/containers.py", line 440, in add_node
layer.set_previous(self.nodes[input])
File "keras/layers/core.py", line 144, in set_previous
assert self.supports_masked_input(), 'Cannot connect non-masking layer to layer with masked output.'
AssertionError: Cannot connect non-masking layer to layer with masked output.
the snippet of my code
 model.add_node(LSTM(output_dim=256), name='forward', input='embedding') model.add_node(BatchNormalization(), name='batch_norm_forward', input='embedding')
Thanks.
Please make sure that the boxes below are checked before you submit your issue. Thank you!

[x] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps
[x] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps
[ ] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

model.add_node(BatchNormalization(), name='batch_norm_forward', input='embedding')
-->
model.add_node(BatchNormalization(), name='batch_norm_forward', input='forward')
maybe?

I've just come across this as well.
@fchollet  --- there's no reason batch normalization can't support masking right by just passing the mask forward?

Yes, that should not be an issue. It's actually just a matter of adding a flag in __init__.

I think I have the same problem with Sequential Model. When I add BatchNormalization after Masking+LSTM, I get the error:
Exception: Layer batchnormalization_1 does not support masking, but was passed an input_mask: Any{2}.0
Thanks

Hi @fchollet -- I downloaded the latest version and found out that this issue is still not resolved. It's just one small change in the __init__ as you mentioned. Should I do PR ? is it worth it ? Thanks

I'm having the same issue.

I'm also having this issue.
 model = Sequential( [             Masking(mask_value=0.0, batch_input_shape=(model_config.batch_size,model_config.TIME_STEPS, model_config.features_num ) ),             BatchNormalization(),             LSTM(model_config.num_units, stateful=True, return_sequences=False,                  recurrent_dropout = model_config.recurrent_dropout,                  dropout = model_config.dropout) ,             Dense(model_config.num_output, activation='sigmoid') #fully connected layer with 1 hidden units     ] )


Improve tick acceleration for campaign right after split

Currently the impl will accelerate ticks for the leader right after it applies a region split action to let campaign fast. A follower use randomized election timeout, whose value is of the range [election_timeout, 2 * election_timeout - 1), to trigger a new campaign. However the config of tick acceleration above if election_timeout - 1. So a leader which did spliting may have to wait election_timeout to start a new campaign in the worst case. This could be improved further.

I think we should reduce [election_timeout, 2 * election_timeout - 1) to [election_timeout, election_timeout + 3 seconds ), so we can elect leader faster.

@ngaut I am not sure whether it's a good idea to modify the randomization of this range for triggering a new campaign. This timeout value is designed and measured in the current range for the best activity and performance, although I do remember where these details are recorded.
We would achieve this improvement simply by adjusting the ticks acceleration number to randomized_election_timeout - 1, or just 2 * election_timeout - 1.

This improvement will fix this issue https://github.com/pingcap/tidb/issues/2695


feat: add "pro" variant for LocalStack image

PR adds a "pro" variant of the existing LocalStack image.

Yes, I guess MSRV was bumped as part of path/minor version of the dependency.
We can either stick it to particular version, or consider MSRV update, but as a breaking change.
Using specific version of dependency may be a pain for our users, if they already updated it
I think we can consider update of MSRV in separate PR

Yes, I guess MSRV was bumped as part of path/minor version of the dependency.
We can either stick it to particular version, or consider MSRV update, but as a breaking change.
Using specific version of dependency may be a pain for our users, if they already updated it I think we can consider update of MSRV in separate PR

What's that mean for this PR? Does it need to wait until CI can be fixed?


[outreachy] Registry: Improve visibility of teaser text

Outreachy task for @MercyCherop
Based on your user experience report, please provide the following suggested improvement via a PR:

The text just above the search bar seems to be important information and though it has been highlighted it still feels a bit hard to find. [...] I feel the text should be moved elsewhere the is more visible and have its font size slightly increased.

Please provide a suggestion how the teaser text can be made more visible, appealing and easier to find. Feel free to be creative using not only text improvements, but any kind of visual improvements that come to your mind
Note: This is an assigned issue for a participant that has completed the user experience report, if you are an outreachy participant as well, find instructions here: #5290

Thank you for participating in the outreachy participation phase, @mercycherop! As announced in the channel and in the meta issue   I closed the project for new contributors. Since you have not registered and also not provided a PR for this issue, I will close this now

[!IMPORTANT]
The deadline to register for the project via the outreachy platform has been extended until Oct 23 7am UTC. If you still want to participate, please make sure that you record at least one contribution to the "OpenTelemetry Registry" project. When you are registered I will reopen this issue.


added description

issue #3292

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@MaratMussabekov Please follow up with copy editing suggestions. Thanks.

Dear @JohanFreelancer9, @illfated,
your recommendations were applied,
thank you!

@Dansimp This PR introduces a description and an XML for Windows Event Log Event 5155 and has passed copy review. Please review and merge as the time permits.
Issue reference: #3292
Thanks

@Dansimp Following up for updates. Thanks.

@Dansimp Following up for updates.


chore(build): remove motion_utils.hpp and tier4_autoware_utils.hpp control/

Description
https://github.com/autowarefoundation/autoware.universe/issues/4821
Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[X] I've confirmed the contribution guidelines.
[X] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[X] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[X] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@TakaHoribe @kyoichi-sugahara Can you approve this ?


Java Program to check if a singly linked list is a palindrome or not.

Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Yes, I have.
Description
(Write your answer here.)
This is a Java Program to check if a singly linked list is a palindrome or not. I have included proper comments throughout the code, whenever found necessary.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)

I have made the required changes. However, Could you please tell me how to make markdown updates...? And, I don't get the one failing check - Java(beta).

@HarshCasper I have made the required changes. And, I don't get the one failing check - Java(beta)...

No you have not. Make the README (Markdown) changes.

I made the README changes. I am sorry. I forgot to, earlier.

@HarshCasper I have made the changes 2 days ago. Could you please check it?

Do some minor suggested changes and resolve the conflict so that I could merge the PR.

Did! @atarax665

I made the changes 3 days ago. Please review it @HarshCasper @atarax665.

I can still see the merge conflict, please do resolve it.


IAR - Inconsistent behavior when deploying apps that do not satisfy IAR.

acorn version - v0.8.0-rc4+b6a95e8a
Steps to reproduce the problem:

Enable IAR feature (acorn install --features image-allow-rules=true)
Add IAR that allows all images that are signed by a key say key1 - (in my santest3)

kubectl describe imageallowrules -n mm
Name:         rule1
Namespace:    mm
Labels:       <none>
Annotations:  <none>
API Version:  api.acorn.io/v1
Images:
  **
Kind:  ImageAllowRule
Metadata:
  Creation Timestamp:  2023-08-14T22:54:47Z
  Generation:          1
  Resource Version:    2002078
  UID:                 4682edc8-4971-49b1-8866-edb59fdfffc9-p
Signatures:
  Rules:
    Annotations:
      Match:
        Qatag:  ok
    Signed By:
      All Of:
        -----BEGIN PUBLIC KEY-----
******
-----END PUBLIC KEY-----

Events:  <none>

Deploy app using image that is not signed .
User is presented to following error message:

 % acorn run -n test4 ghcr.io/sangee2004/mynewcopytest:v0.0.1
  ✗  ERROR:  App.api.acorn.io "test4" is invalid: spec.image: Forbidden: ghcr.io/sangee2004/mynewcopytest:v0.0.1 not allowed to run: signature verification failed: expected signature artifact ghcr.io/sangee2004/mynewcopytest:sha256-***.sig not found

Sign the app with a key different that key1 ( santest3 in my case)

% acorn image sign ghcr.io/sangee2004/mynewcopytest:v0.0.1 --key ~/santest2.key -a qatag=ok
  •  Signing Image ghcr.io/sangee2004/mynew
copytest:v0.0.1 (digest: ghcr.io/sangee2004/mynewcopytest@sha256:ea92031f6bba821d5ce23b5608f898296113ac027ebd8fb1d83bc7d630fbde29) using key /Users/sangeethahariharan/santest2.key
? Enter password for private key /Users/sangeethahariharan/santest2.key: ********
  ✔  Done: Pushed signature sha256:****

Deploy app using this image.

% acorn run -n test4 ghcr.io/sangee2004/mynewcopytest:v0.0.1
  •  WARNING:  This application would like to use the image 'ghcr.io/sangee2004/mynewcopytest:v0.0.1'.
                 This image is not trusted by any image allow rules in this project.
                 This could be VERY DANGEROUS to the cluster if you do not trust this
                 application. If you are unsure say no.

? Do you want to allow this app to use this (POTENTIALLY DANGEROUS) image?  [Use arrows to move, type to filter]
> NO
  yes (this tag only)
  repository (all images in this repository)
  registry (all images in this registry)
  all (all images out there)
  ✗  ERROR:  interrupt: image <ghcr.io/sangee2004/mynewcopytest:v0.0.1> is not allowed by any ImageAllowRule in this project

This time we are prompted for allowing the image.
Why are the app deployment failures in step 3 and 5 being handled differently? It is confusing to see 2 different behaviors.

Tested with acorn version - v0.8.0-rc4-47-g80dd7169+80dd7169
Same behavior is seen when app deployment is not allowed because of being restricted by IAR in case of image not being signed at all or when image is signed with a key not satisfied by IAR.
%acorn run -n test4 ghcr.io/sangee2004/testlocal:v0.0.1
  •  WARNING:  This application would like to use the image 'ghcr.io/sangee2004/testlocal:v0.0.1'.
                 This image is not trusted by any image allow rules in this project.
                 This could be VERY DANGEROUS to the cluster if you do not trust this
                 application. If you are unsure say no.

? Do you want to allow this app to use this (POTENTIALLY DANGEROUS) image? NO
  ✗  ERROR:  image <ghcr.io/sangee2004/testlocal:v0.0.1> is not allowed by any ImageAllowRule in this project

% acorn image sign  ghcr.io/sangee2004/testlocal:v0.0.1 --key ~/santest2.key -a qatag=ok
  •  Signing Image ghcr.io/sangee2004/testlocal:v0.0.1 (digest: ghcr.io/sangee2004/testlocal@sha256:543ca38d7f6e7d57ce1fe5f79485b909988b4107a52c5af137af3d46d61b78b6) using key /Users/sangeethahariharan/santest2.key
? Enter password for private key /Users/sangeethahariharan/santest2.key: ********
  ✔  Created signature sha256:xxxxx

 % acorn run -n test4 ghcr.io/sangee2004/testlocal:v0.0.1
  •  WARNING:  This application would like to use the image 'ghcr.io/sangee2004/testlocal:v0.0.1'.
                 This image is not trusted by any image allow rules in this project.
                 This could be VERY DANGEROUS to the cluster if you do not trust this
                 application. If you are unsure say no.

? Do you want to allow this app to use this (POTENTIALLY DANGEROUS) image? NO
  ✗  ERROR:  image <ghcr.io/sangee2004/testlocal:v0.0.1> is not allowed by any ImageAllowRule in this project


Show all cities even if infect/epidemic chance are both 0.00

In the 8/10 game I noticed that it seemed like not all cities were being displayed at all times. I assume this is because they had a 0.00 infection and epidemic chance. But I think we should still show these cities in this case so we can confirm that they’re safe.

I think is more that they are not in the player deck anymore. I guess I can add everything in the discard too.


npm test appears broken out of the box.

I'm submitting a ...  (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, instead see use [gitter](https://gitter.im/mgechev/angular2-seed) or [stackoverflow](https://stackoverflow.com/questions/tagged/angular2)

Current behavior
npm test fails right out of the box with a just-pulled and -built angular-seed.
Expected behavior
Test suite runs, or at least reports no tests found.
Minimal reproduction of the problem with instructions
npm test
What is the motivation / use case for changing the behavior?
I'd like to run tests under angular-seed.
Please tell us about your environment:
MacOS 10.13.6, WebStorm (current)

Angular Seed Version: aaaaf75
5d931132e50c66bb9389c9182909621f90da2a12

Node: node --version =
node 8.10.0
npm 5.6.0

START:
08 08 2018 17:03:02.338:WARN [watcher]: Pattern "/Users/chris/Documents/...../testing/angular-seed/node_modules/.tmp/Rx.min.js" does not match any file.
[17:03:03] Finished 'karma.run' after 1.48 s
[17:03:03] Finished 'test' after 8.92 s
08 08 2018 17:03:04.504:INFO [karma]: Karma v2.0.4 server started at http://<IP_ADDRESS>:9877/
08 08 2018 17:03:04.504:INFO [launcher]: Launching browser Chrome with unlimited concurrency
08 08 2018 17:03:04.509:INFO [launcher]: Starting browser Chrome
08 08 2018 17:03:05.122:INFO [Chrome 67.0.3396 (Mac OS X 10.13.6)]: Connected on socket AXGA99pOMW9FC-8XAAAA with id 91392717
08 08 2018 17:03:05.485:WARN [web-server]: 404: /base/node_modules/.tmp/Rx.min.js
Chrome 67.0.3396 (Mac OS X 10.13.6) ERROR: 'Error: Fetch error: 404 Not Found
    at http://localhost:9877/base/node_modules/systemjs/dist/system.src.js?0bdee8ac056cf56be016bc7a2bb873d171693e42:1512:15
    at ZoneDelegate.invoke (http://localhost:9877/base/node_modules/zone.js/dist/zone.js?e0fce327c58f761bb85f158317f06e54b0e339c0:388:26)
    at Zone.run (http://localhost:9877/base/node_modules/zone.js/dist/zone.js?e0fce327c58f761bb85f158317f06e54b0e339c0:138:43)
    at http://localhost:9877/base/node_modules/zone.js/dist/zone.js?e0fce327c58f761bb85f158317f06e54b0e339c0:872:34
    at ZoneDelegate.invokeTask (http://localhost:9877/base/node_modules/zone.js/dist/zone.js?e0fce327c58f761bb85f158317f06e54b0e339c0:421:31)
    at Zone.runTask (http://localhost:9877/base/node_modules/zone.js/dist/zone.js?e0fce327c58f761bb85f158317f06e54b0e339c0:188:47)
    at drainMicroTaskQueue (http://localhost:9877/base/node_modules/zone.js/dist/zone.js?e0fce327c58f761bb85f158317f06e54b0e339c0:595:35)'
Chrome 67.0.3396 (Mac OS X 10.13.6) ERROR: 'Error: Fetch error: 404 Not Found
    at http://localhost:9877/base/node_modules/systemjs/dist/system.src.js?0bdee8ac056cf56be016bc7a2bb873d171693e42:1512:15
    at ZoneDelegate.invoke (http://localhost:9877/base/node_modules/zone.js/dist/zone.js?e0fce327c58f761bb85f158317f06e54b0e339c0:388:26)
    at Zone.run (http://localhost:9877/base/node_modules/zone.js/dist/zone.js?e0fce327c58f761bb85f158317f06e54b0e339c0:138:43)
    at http://localhost:9877/base/node_modules/zone.js/dist/zone.js?e0fce327c58f761bb85f158317f06e54b0e339c0:872:34
    at ZoneDelegate.invokeTask (http://localhost:9877/base/node_modules/zone.js/dist/zone.js?e0fce327c58f761bb85f158317f06e54b0e339c0:421:31)
    at Zone.runTask (http://localhost:9877/base/node_modules/zone.js/dist/zone.js?e0fce327c58f761bb85f158317f06e54b0e339c0:188:47)
    at drainMicroTaskQueue (http://localhost:9877/base/node_modules/zone.js/dist/zone.js?e0fce327c58f761bb85f158317f06e54b0e339c0:595:35)'

Finished in 0.001 secs / 0 secs @ 17:03:05 GMT-0700 (MST)

SUMMARY:
✔ 0 tests completed
npm ERR! Test failed.  See above for more details.

I am not able to reproduce the issue. This may be an obvious question, but have you run npm install?

Oh goodness. I can't even.


Using engine ciphers regression

Fetching ciphers from engine is broken since
9f2058611f7aec733d4a476f4f28c895d9e5667b is the first bad commit
commit 9f2058611f7aec733d4a476f4f28c895d9e5667b
Author: Shane Lontis<EMAIL_ADDRESS>Date:   Tue May 5 16:51:32 2020 +1000
Remove cipher table lookup from EVP_CipherInit_ex

Reviewed-by: Richard Levitte<EMAIL_ADDRESS>(Merged from https://github.com/openssl/openssl/pull/11731)

gost-engine grasshopper tests are failing with diagnostics
Encryption test for grasshopper ecb
00:52:A4:00:2D:7F:00:00:error:digital envelope routines:evp_generic_fetch:fetch failed:crypto/evp/evp_fetch.c:307:Default library context, Algorithm (kuznyechik-ecb), Properties ()
00:52:A4:00:2D:7F:00:00:error:digital envelope routines:EVP_CipherInit_ex:initialization error:crypto/evp/evp_enc.c:176:
/home/beldmit/upstream/engine/test_context.c:69: OpenSSL internal error: EVP_CipherInit_ex(ctx, type, NULL, K, iv, enc)

@slontis

@beldmit - Do you have any idea how this algorithm gets set up. The code I removed basically did a big switch statement on all the ciphers it knew about, if it wasnt one of these then it ran the legacy path. So somehow I need to detect this case now.

This algorithm is registered via EVP_add_cipher.

Oh, no, sorry. EVP_add_cipher is bypassed in this test.
The EVP_CIPHER object used in this test is created in this way:
https://github.com/gost-engine/engine/blob/b9d014538053c2d0717a0029dee88694cc4e4ed7/gost_grasshopper_cipher.c#L987-L1005

We've reimplemented the test. Now we don't just obtain the EVP_CIPHER object to run tests, but load the engine (that invokes EVP_add_cipher) and obtain it via the engine.
Here is the link to a previous (broken) state.
https://github.com/gost-engine/engine/tree/b6181aefc8d1b76652c12b6da3c15321b069257d
If we think that direct creating the EVP_CIPHER without EVP_add_cipher is bad practice, the issue can be closed. Many thanks!


Provide a client application that handles cancel option from Identity Server/HomeController.Login

I would like to see a running sample of how a client application handles the cancel options after a user clicks on the cancel option on the login page of the HomeController.
Client Application: Asp.Net Core MVC
-This should not introduce a breaking change.
This was the issue I followed:
https://github.com/IdentityServer/IdentityServer4/issues/2556

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

Sorry, we don't accept PRs on master.
As for what should happen if the user clicks 'cancel', the client should get an access_denied error passed back to the redirect_uri.


sensor.pirateweather_precip

Not an issue just want to know with this sensor it returns precipitation type and id like to know what all the types are, rain, etc. And what it will return if there is no precipitation. Null, nothing? Thank you very much

The precipTypes are sleet, snow, rain and if no precipitation then the value will be none.

@alexander0042 Since this is related I was wondering if Freezing Rain is classified as sleet or rain. The DarkSky docs have it classified it as sleet but since it's rain that freezes when touching the ground shouldn't it be classified as rain?

GREAT question! I really love these sorts of nitty gritty weather data questions.
Currently, hail, sleet, and freezing rain are all classified as "sleet". This definitely isn't ideal, but it's the best I can do within the Dark Sky framework. I have a longer term plan to add a flag that would let me separate these, since they are different, but for now, sleet is pretty much everything that isn't rain or snow.

Ok, thanks! I know DarkSky classified sleet as Mixed Precipitation/Sleet/Freezing rain in their docs but I wasn't sure how it worked for PW. As far as I remember it only ever returned Rain or Snow as conditions and it never seemed to use sleet outside of the daily summary.
I don't think I saw anything in their translation module for Hail but I did see something for Thunderstorms so maybe they never set that up before being bought by Apple?

I know with WeatherKit that they've implemented Thunderstorms but I don't think they've added anything else in terms of icons/precipTypes. With the current setup freezing rain would fit under rain but I'm not sure where exactly hail would fit. I was thinking sleet since they're similar but it might not make sense that way so maybe that's where the flag comes in?
If there was an additional flag it would be for showing additional precipTypes or would it add something else? Maybe once the translations are done it could be used to have an expanded iconset for those that want it? I know that Visual Crossing has a parameter for more icons instead of their default ones.


3rd edition of russion texts and one fix.

When the story was launched through incognito mode in the browser, nothing happened - just a white screen was shown.

merge

merge


Make 1.1.9 release

Want to finish that next week or so.

Sorry, for delay, still have plans to do above

1.1.10 was done, 1.1.11 and 1.2.1 will be soon


Archive by year

Should these be archived by year? Even if there is redundancy from year to year, this seems more organized and sustainable.

@lucywyman I think if we start organizing the website by year this might be a good idea. I'd ask for a second opinion though.

@jordane @borcean @tschuy


Real-time speaker diarization doesn't seem to work for me

When testing in the Speech Studio I am seeing exactly the behavior that I am trying to get, as in the picture. Real time speaking into the microphone gets continually transcribed and chunks of text get real-time diarized.
I tried to follow the quickstart guide on real-time diarization, but if I run it what happens is that I only get the "TRANSCRIBED" logs after the whole audio is processed and I don't get the intermediate results. If I add a listener on .transcribing this successfully logs text as it is processed, but without attributing it to a specific speakerId (speakerId is undefined). See image from my terminal. You can also see that I get all the "Transcribing" events first and the "Transcribed" events only at the end of the whole audio.

What I desire to see instead is what was shown in this announcement of real-time diarization where the transcribing events are interleaved with transcribed events once an utterance is identified.
I have searched the documentation and samples here, but seem to be unable to find anything on this. I am not sure if this is a bug or if I am missing some crucial piece of information. I would greatly appreciate any help! (I am using JavaScript)

Okay, I figured things out. I just needed to do processing in the browser and use ConversationTranscriber with fromDefaultMicrophoneInput(), also used the authorization flow described here: https://github.com/Azure-Samples/cognitive-services-speech-sdk/blob/master/samples/js/browser/README.md#token-exchange-process
Example:
const audioConfig = SpeechSDK.AudioConfig.fromDefaultMicrophoneInput();
const speechConfig = SpeechSDK.SpeechConfig.fromAuthorizationToken(authorizationToken, region);
const transcriber = new SpeechSDK.ConversationTranscriber(speechConfig, audioConfig);
transcriber.transcribing = (sender, transcriptionEventArgs) => console.log("transcribing", transcriptionEventArgs.result)
transcriber.transcribed = (sender, transcriptionEventArgs) => console.log("transcribed", transcriptionEventArgs.result)
transcriber.startTranscribingAsync();

Closed as resolved based on the latest comments.


Compile error in status_server on Mac

Bug Report

What version of TiKV are you using?

Master
What operating system and CPU are you using?

Steps to reproduce

run
What did you expect?
Everything goes OK
What did happened?
error[E0061]: this function takes 6 arguments but 5 arguments were supplied
    --> src/server/status_server/mod.rs:1255:33
     |
1255 |         let mut status_server = StatusServer::new(
     |                                 ^^^^^^^^^^^^^^^^^ expected 6 arguments
1256 |             1,
     |             -
1257 |             None,
     |             ----
1258 |             ConfigController::default(),
     |             ---------------------------
1259 |             Arc::new(SecurityConfig::default()),
     |             -----------------------------------
1260 |             MockRouter,
     |             ---------- supplied 5 arguments
     |
note: associated function defined here
    --> src/server/status_server/mod.rs:94:12
     |
94   |     pub fn new(
     |            ^^^
95   |         status_thread_pool_size: usize,
     |         ------------------------------
96   |         pd_client: Option<Arc<RpcClient>>,
     |         ---------------------------------
97   |         cfg_controller: ConfigController,
     |         --------------------------------
98   |         security_config: Arc<SecurityConfig>,
     |         ------------------------------------
99   |         router: R,
     |         ---------
100  |         store_path: PathBuf,
     |         -------------------

This Problem seems to introduced by #11266

/assign


performance regression in contam_visibility.visibilityPA.using_gtvt

We have noticed a performance regression in more recent versions of exoctk in the contam_visibility.visibilityPA.using_gtvt function (used by exo-mast).
Here's a summary of my findings:
Name (time in s)   Min       Max       Mean     StdDev   Median     IQR   Outliers  OPS      Rounds   Iterations
-----------------------------------------------------------------------------------------------------------------------
develop (py38)     1.0909    1.2363    1.1308   0.0604   1.1036    0.0579   1;0     0.8843   5        3
develop (py36)     1.0345    1.0621    1.0476   0.0115   1.0438    0.0188   2;0     0.9546   5        3
master (py38)      1.0745    1.2483    1.1147   0.0751   1.0823    0.0565   1;1     0.8971   5        3
master (py36)      1.0260    1.0499    1.0359   0.0107   1.0333    0.0193   1;0     0.9653   5        3
0.4.0 tag (py38)   0.6111    0.6397    0.6204   0.0120   0.6119    0.0182   1;0     1.6126   5        3
0.4.0 tag (py36)   0.6066    0.6654    0.6231   0.0240   0.6128    0.0197   1;1     1.6047   5        3
-----------------------------------------------------------------------------------------------------------------------

In order to reproduce:
I used pytest-benchmark. I created a separate conda environment for python 3.6 and 3.8. I did not do a fresh install in the environments for each branch, so they share the same dependencies within each environment.  However, I do not see differences between environments, only between branches, so I don't think this is the cause of the performance difference.
This is my test file:
from exoctk.contam_visibility import visibilityPA

def call_using_gtvt():
    instrument = 'NIRISS'
    ra = "-66"
    dec = "44"
    return visibilityPA.using_gtvt(ra, dec, instrument)

def test_benchmark_call_using_gtvt(benchmark):
    benchmark.pedantic(call_using_gtvt, iterations=3, rounds=5)

I benchmarked this after finding that exo-mast targetvis plots were taking around 500 ms longer... so this seems to correspond.
Was this already known? py-spy may be helpful here if we don't know what's causing the slow down.
Thanks for any additional information.

Edited the description above to provide more benchmark results across different commits and environments.

Edited the description above to provide more benchmark results across different commits and environments.

I suspect the slowdown might be related to this section of code: https://github.com/ExoCTK/exoctk/blob/master/exoctk/contam_visibility/visibilityPA.py#L388-L401
if this is necessary, perhaps there's a way to vectorize it to avoid the for loops
added in this commit

I suspect the slowdown might be related to this section of code: https://github.com/ExoCTK/exoctk/blob/master/exoctk/contam_visibility/visibilityPA.py#L388-L401
if this is necessary, perhaps there's a way to vectorize it to avoid the for loops
added in this commit

Hi @falkben,
Thanks a ton for this comprehensive report! We'll put it in the agenda for our next ExoCTK team meeting (Jan 19th), and will get back to this issue to see how we can handle it!
N.

Hi @falkben,
Thanks a ton for this comprehensive report! We'll put it in the agenda for our next ExoCTK team meeting (Jan 19th), and will get back to this issue to see how we can handle it!
N.

(Sorry! I hit the wrong button while reviewing your great input!)

(Sorry! I hit the wrong button while reviewing your great input!)

It looks like there's a lot of opportunity to vectorize the operations inside the using_gtvt function. My PR, linked above, addresses the section of code that is filtering badPAs against paNomnan but there's likely other areas to do as well. My testing indicates that vectorizing the code speeds things up by about 200ms, so that doesn't fully recover the performance difference reported in this ticket

It looks like there's a lot of opportunity to vectorize the operations inside the using_gtvt function. My PR, linked above, addresses the section of code that is filtering badPAs against paNomnan but there's likely other areas to do as well. My testing indicates that vectorizing the code speeds things up by about 200ms, so that doesn't fully recover the performance difference reported in this ticket

the results from the same benchmark, on develop branch, with my patch above applied, for comparison with this issue. Note, milliseconds instead of seconds.
Name (time in ms)                       Min       Max      Mean   StdDev    Median      IQR  Outliers     OPS  Rounds  Iterations
---------------------------------------------------------------------------------------------------------------------------------
test_benchmark_call_using_gtvt     689.7068  720.1474  701.7793  15.5694  691.0521  27.7793       2;0  1.4249       5           3

the results from the same benchmark, on develop branch, with my patch above applied, for comparison with this issue. Note, milliseconds instead of seconds.
Name (time in ms)                       Min       Max      Mean   StdDev    Median      IQR  Outliers     OPS  Rounds  Iterations
---------------------------------------------------------------------------------------------------------------------------------
test_benchmark_call_using_gtvt     689.7068  720.1474  701.7793  15.5694  691.0521  27.7793       2;0  1.4249       5           3

Hi, Just checking in to see if there's been any opportunity to take a look at this? Thanks!

Hi, Just checking in to see if there's been any opportunity to take a look at this? Thanks!

Hi @falkben,
Sorry for the very slow response on this. You caught us on the middle of defining our issue prioritization process within ExoCTK :-). I'll have a chat with @jaymedina and @hover2pi who are the tool champions to get into this ASAP.
Thanks!
Néstor

Hi @falkben,
Sorry for the very slow response on this. You caught us on the middle of defining our issue prioritization process within ExoCTK :-). I'll have a chat with @jaymedina and @hover2pi who are the tool champions to get into this ASAP.
Thanks!
Néstor