SIGSEGV calling NDList.encode()

Description
The following unit test crashes on my Linux box with a SIGSEGV when the NDList.encode() method is called.
Expected Behavior
No SIGSEGV.
Error Message
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00007c7568f29f52, pid=1108173, tid=1108182
#
# JRE version: OpenJDK Runtime Environment JBR-17.0.6+10-829.9-jcef (17.0.6+10) (build 17.0.6+10-b829.9)
# Java VM: OpenJDK 64-Bit Server VM JBR-17.0.6+10-829.9-jcef (17.0.6+10-b829.9, mixed mode, tiered, compressed oops, compressed class ptrs, g1 gc, linux-amd64)
# Problematic frame:
# C  [0.26.0-libdjl_torch.so+0x129f52]  c10::intrusive_ptr<c10::TensorImpl, c10::UndefinedTensorImpl>::operator->() const+0xc
#

hs_err_pid1108173.log
How to Reproduce?
    @Test
    public void testCrash()
    {
        try (NDManager ndManager = NDManager.newBaseManager())
        {
            HuggingFaceTokenizer tokenizer = HuggingFaceTokenizer.newInstance("openai/clip-vit-base-patch32");
            Encoding encoding = tokenizer.encode("fast blue car");
            NDArray attention = ndManager.create(encoding.getAttentionMask());
            NDArray inputIds = ndManager.create(encoding.getIds());
            NDArray placeholder = ndManager.create("");
            placeholder.setName("module_method:get_text_features");
            NDList ndList = new NDList(inputIds.expandDims(0), attention.expandDims(0), placeholder);
            ndList.encode();
        }
    }

Steps to reproduce
Run the above test class.  There is no GPU on my machine.
What have you tried to solve it?
I tried using a specific PYTORCH version, as I read the current version has issues elsewhere, using PYTORCH_VERSION=2.0.1 but it made no difference.

I just realised I reported this on djl-serving instead of djl.  I was attempting to use https://resources.djl.ai/demo/pytorch/clip.zip?translatorFactory=ai.djl.translate.NoopServingTranslatorFactory with djl-serving, and the above unit test code was in effect what I think is required to run over the query text, before sending to it djl-serving?

Closing this off since https://github.com/deepjavalibrary/djl/issues/3033 now exists.


Refactor

refactoring..

[ ] Add a .clang-format file
[ ] Extract descriptors/puara-button.h, descriptors/puara-jab.h from puara_gestures.cpp and puara_gestures_old.h
[ ] Moved Coord1D, Coord2D, Coord3D structs to puara-structs.h
[ ] .gitmodules: updated IMU_Sensor_Fusion submodule path

I just just reverted the -1 values back to what they were before, in this commit https://github.com/Puara/puara-gestures/pull/15/commits/be34c16cc068e948f1a803f68007f50b92a7bdb5

I just just reverted the -1 values back to what they were before, in this commit  https://github.com/Puara/puara-gestures/pull/15/commits/be34c16cc068e948f1a803f68007f50b92a7bdb5

I just just reverted the -1 values back to what they were before, in this commit  https://github.com/Puara/puara-gestures/pull/15/commits/be34c16cc068e948f1a803f68007f50b92a7bdb5


Text input focus border styling fixed

Text input focus style, numeric and error styling was buggy in different browsers and also at various zoom levels.
Normally this would not be an issue, but because we use round corners we can’t approach focus styles like this because outline does not support border radius which is causing the visual bugs. Box shadow however is fine on an element that uses border-radius.
Fixes: https://github.com/ONSdigital/design-system/issues/1475
Before / After

 Issue #1475: Text input focus border styling is inconsistent across devices/browsers...


How to save the output as TAP format

What is the command used to save the output of a test to a designated folder in a TAP format?

You have to specify the path of the TAP module in your command.
It should like that:
busted -o busted.outputHandlers.TAP
Then you can redirect the output to a file using standard redirection.
busted -o busted.outputHandlers.TAP > outputfile


WIP Browser compat fixes

Polyfills - https://reactjs.org/docs/javascript-environment-requirements.html
HTML5 shiv - https://stackoverflow.com/questions/29828561/react-js-not-working-with-internet-explorer-9
Additionally targeting contemporary document rendering mode for older IE - <meta http-equiv="X-UA-Compatible" content="IE=edge">
Shim/sham - https://github.com/es-shims/es5-shim
IE doesn't support includes
i18n uses methods on the console object that aren't supported in IE9 https://github.com/i18next/react-i18next/issues/455

Still testing complete workflow. Shooting for IE9 baseline.

Looking this more, it looks like https://www.npmjs.com/package/react-stepper-horizontal uses the Array method include, which fails in IE <= 11. Works in Edge.


makefile: add release.zip

Add make entry to create a release so it can be done outside of Jenkins
if required.

I am so very confused. Make wants to build targets, but they're already built.
I see this in Jenkins and on my local pc. I must have some error in the Makefile, but I can't find it.


dashArray on canvas uses ES5 functions (map)

See https://github.com/Leaflet/Leaflet/blob/master/src/layer/vector/Canvas.js#L79
map and other ES5 functions (forEach, reduce, filter, etc)  are not supported by IE8, which Leaflet still supports.

Not a critical issue, since IE8 does not support canvas, and will not run this code.
However, it seems like asking for trouble if we start using ES5 stuff in the code, while still supporting IE8. People might look at the code and figure its ok to use in other places, which will be executed on IE8.


docs: describe how to add to our CLI

This PR adds documentation that describes how to add to our CLI. It should be useful for any potential contributors to our codebase!
The new section is on the Contributing page of our docs.

converting to draft b/c we should also mention that they should create new tests and add proper documentation for new commands
and for new options, they should consider adding an example to the documentation for that command


Use vcpkg for managing libraries

It is not efficient to check the libraries whether they are outdated or not by myself.
Using vcpkg will do that automatically.

No longer needed.


Need to reselect libraries

When multiple outs provide the same library, a dialogue asking me to select which version to use appears the first time I'm running nix-alien. However, for subsequent runs there are no options to restart this process. Sometimes inappropriate choices render the program to crash, and I would like to reselect outs. I also wonder where does nix-alien stores my choices?

When multiple outs provide the same library, a dialogue asking me to select which version to use appears the first time I'm running nix-alien. However, for subsequent runs there are no options to restart this process. Sometimes inappropriate choices render the program to crash, and I would like to reselect outs.

See the README.md file, it describes the --recreate flag that you can use to force the recreation of the default.nix file. So running:
$ nix-alien --recreate <binary>

Should do the trick.

I also wonder where does nix-alien stores my choices?

You can use --print-destination flag and it will print the destination file with your choices and exit.

Closing this issue because it seems this question is solved.


Wishlist: check if Chrome is up to date

Add a threat that is active when Chrome is present but not up to date.

Closed by #57


rootless: add an entry to /etc/hosts when using slirp4netns

Closes: https://github.com/containers/libpod/issues/3405
Signed-off-by: Giuseppe Scrivano<EMAIL_ADDRESS>

Changes LGTM, just wondering if we should also add a trailing comment to the line too, something like "# used by rootless Podman"  or "# used by slirp4netns"  or perhaps just "# slirp4netns"

@giuseppe Yes please add a comment.  Is there a possibility for a conflict here?  If the hostname was added twice?  What happens?

@giuseppe Yes please add a comment. Is there a possibility for a conflict here? If the hostname was added twice? What happens?

comment added.  In that case the resolver will use all the specified addresses.
# printf "<IP_ADDRESS> foo\n<IP_ADDRESS> foo\n<IP_ADDRESS> foo\n" >> /etc/hosts
# python -c 'import socket; print(socket.gethostbyname_ex("foo"))'
('foo', [], ['<IP_ADDRESS>', '<IP_ADDRESS>', '<IP_ADDRESS>'])

/lgtm


Textarea for editing keys?

I find difficult to edit multiline translations due to the single line input texts in the translations.html.
What do you think of switching to a textarea with 2 or 3 lines?

This was added to 1.1.1. Please reopen if it isn't solved to your satisfaction :)


jdownloader cpu increase with container running time?

Hi for some reason my jdownloader2 container's idle CPU usage seems to increase over time, e.g. rises from 0-2% to 10% over 6 days.
Restarting the container fixes the issue, but it slowly increases over time again.
I couldn't see warnings/errors in the logs, and haven't had any issues with MakeMKV and handbrake (which i assume are quite similar)
Running jdownloader2 as a docker container on openmediavault (debian) on an i3-6100T
Let me know if there's any troubleshooting steps I can do.

same here. This docker container always has the highest idle cpu usage.  (process jdownloader.jar)

If the CPU usage is caused by jdownloader.jar, I think it's worth checking/asking on the JDownloader forum.  I'm not sure the behaviour you describe is caused by the fact that JD is running in a container.

Same behaviour for me, with very visible linear cpu-usage increase, see this netdata graph (drop is because of container restart). The container was mostly idle without any downloads.

Even if this is actually an upstream issue, I would vote to at least mention this issue in the readme and release-notes as a known limitation if nothing else.

As much work as @jlesage putted into the GUI, which is btw amazing...
I have to say that jdownloader headless is a more headachefree experience, im sorry but in headless mode i have not a single issue with the container, while i had some troubles in the first days in the GUI mode...

like suddenly filling up all memory of the LXC Container and using 100% cpu, which made the whole LXC Container unresponsive/Frozen almost. (I run Docker inside an LXC Container on Proxmox)
Crashing, etc...

Switched to headless some days ago and all my issues went away, so in my opinion, as nice as the GUI is, its not worth it.
And that has nothing with @jlesage todo, it's simply jdownloader itself, jdownloader is unreliable simply.
I mean it can be the case because of a virtual x11 desktop + novnc, but i doubt it.
I think jdownloader is simply buggy and written on a slow language anyway that i don't like (java)
Cheers


fix(material/snack-bar): ensure that the snack bar always runs inside the NgZone

Adds an extra call to ensure that the snack bar is inside the NgZone. This is something that has come up several times in internal tests where some API call is stubbed out, pulling the snack bar outside the zone and causing tests to fail when we remove a change detection somewhere.
Note: I had a hard time reproducing this in our own tests, presumably because the fixture ends up pulling it back into the zone.

This change doesn't work well with internal tests because they are so aggressive with flushing. This causes snackbar's setTimeout to finish before they can check expectations on the displayed snackbar.
Most tests can work around this by making sure they don't flush, but I wouldn't want to expect users to understand when and where they should feel safe flushing. See cl/437094895 for examples.
I'm worried this will make it difficult for users to easily write tests for the snackbar. Is there another way this can be implemented, or is there another fix we could try for these client tests that would be more intuitive?

The calls outside the NgZone seem to primarily break closing of the snack bar. I've reworked the PR to limit the impact only to the code paths related to closing.


Add node-rate-limiter package to avoid auth brute force

As talk in issue https://github.com/chrisleekr/binance-trading-bot/issues/281, https://github.com/animir/node-rate-limiter-flexible has been preferred compared to https://www.fail2ban.org/ implementation, in order to have a controlled ecosystem inside docker images.
Describe the solution you'd like
If authentication is activated, the auth handler will ban brute force attempts for a certain duration, if there too many requests in a given interval window.
For that, we'll use node-rate-limiter package and setup rules for banning bad attempters ips.
Alternatives

fail2ban

@caebwallace hey, I opened PR with changes. Could you take a look and let me know if you have any feedback? Otherwise, I will release today.


Add greta-formations.txt

@greta-formations Please provide us with a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.
You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

Hello,
Sorry, I ask this document to my hierarchy.
I am waiting for the answer.
Thank you for your understanding

HPSCAN_20220912145219741_2022-09-12_145307511.pdf
Hello,
sorry for the delay. I have the official request for adding the domain for educational licences.
Thanks you,
Best regards

Hello,
I am available if you need some informations.
Thanks
Best Regards

Hello @philipto
I add the requested document. The official website is : https://www.greta-lorraine.fr/reseau/greta-cfa-lorraine-nord/
We offer to our student a lot of IT formation and we would like to integrate the jetbrains tools in our program.
The domain greta-formations.fr is a technical domain used to map our e-mail services.
If you need something else, don't hesitate.
Thanks
Best regards

@greta-formations Thank you for your patience while waiting for a review and processing your request. Pull request merged.


QUIC: Uses unidirectional stream to send HTTP3 SETTINGS frames

Overall looks good to me.
I think this is just an internal implementation change but I want @masaori335 to take a look because I haven't seen H3 part for a long time.

Will look tonight, but it looks like this is underlay of HTTP/3.

@masaori335

but it looks like this is underlay of HTTP/3.

I wanted you to confirm:

Whether the stream type (unidirectional) is appropriate for SETTINGS frame
Whether QUICStreamManager's interface (not changed) is still reasonable for H3 (and QUIC apps)

Prior this change, QUICBidirectionalStream was used for uni stream. This change it QUICUnidirectionalStream correctly. So this should not have any impact for application.
Also introducing QUICStreamDirection looks reasonable.


Update KAINET/NANA.

Based upon feedback at
https://www.reddit.com/r/KerbalSpaceProgram/comments/38znqb/ckan_integration_to_kerbalx_and_ksp_mod_manager/crzdhr5

When the new one is released on KS I'm guessing we'll have to keep the identifier to avoid confusion of our systems, or implement a solution like the ones discussed in https://github.com/KSP-CKAN/CKAN/issues/904 which would solve these situations :D

So "near future" was a bit of a mis-estimate, but it's done.  New KS link: https://kerbalstuff.com/mod/1048/No Astronauts Need Apply
For future reference, I'll always publish to GitHub first (since that's where the source lives), and that is more likely to remain stable.


compile on scala 2.12.1, remove hdf5 from the core build, update test…

… dependencies

Travis has java version problems. "java.lang.UnsupportedClassVersionError: scala/tools/nsc/Main : Unsupported major.minor version 52.0"

@pityka gonna wanna run against the jdk8 on travis since I believe it's 7 by default. should be able to add something like https://docs.travis-ci.com/user/languages/java/#Testing-Against-Multiple-JDKs to test against only the openjdk8

Travis has oraclejdk8. Otherwise seems to be good now.

One final nit pick, can we change 2.10.5 to 2.10.6 that's the latest and I believe the final version of the 2.10 line
Sent from my iPhone

On Apr 9, 2017, at 11:47 PM, Istvan Bartha<EMAIL_ADDRESS>wrote:
Travis has oraclejdk8. Otherwise seems to be good now.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Awesome @pityka thanks!


Kong is unable to route the Istio urls

Summary
I am trying to explore how kong ingress controller works with Istio Service mesh. I followed the below url.
https://kubernetes.io/blog/2020/03/18/kong-ingress-controller-and-istio-service-mesh/
Not sure why my kong proxy url is not routing to istio bookinfo->product page url. I always get the following message.
$ curl $PROXY_URL/productpage
{"message":"no Route matched with those values"}

My istio version: 1.9.0, and the cluster is running on GKE.
Then I changed the istio version to 1.6.7 and tried the steps as per the following doc. I can see the same routing issue.
https://docs.konghq.com/kubernetes-ingress-controller/1.1.x/guides/getting-started-istio/
Tried hitting the url using both port-forward and external IP address but no success.
$ curl http://localhost:8080/productpage
{"message":"no Route matched with those values"}

$ echo $PROXY_IP
<IP_ADDRESS>:80
$ curl $PROXY_IP/productpage
{"message":"no Route matched with those values"}

Kubernetes version
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.6", GitCommit:"dff82dc0de47299ab66c83c626e08b245ab19037", GitTreeState:"clean", BuildDate:"2020-07-15T16:58:53Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"18+", GitVersion:"v1.18.12-gke.1210", GitCommit:"199a41188dc0ca5d6d95b1cc7e8ba96e05f9dd0a", GitTreeState:"clean", BuildDate:"2021-02-05T18:03:16Z", GoVersion:"go1.13.15b4", Compiler:"gc", Platform:"linux/amd64"}

$ istioctl version
client version: 1.6.7
control plane version: 1.6.7
data plane version: 1.6.7 (10 proxies)

$ kubectl logs -f example-kong-kong-6567cb7cdc-sqfm7 -c proxy -n kong-istio
2021/02/25 13:01:54 [notice] 1#0: using the "epoll" event method
2021/02/25 13:01:54 [notice] 1#0: openresty/<IP_ADDRESS>
2021/02/25 13:01:54 [notice] 1#0: built by gcc 9.3.0 (Alpine 9.3.0)
2021/02/25 13:01:54 [notice] 1#0: OS: Linux 5.4.49+
2021/02/25 13:01:54 [notice] 1#0: getrlimit(RLIMIT_NOFILE): 1048576:1048576
2021/02/25 13:01:54 [notice] 1#0: start worker processes
2021/02/25 13:01:54 [notice] 1#0: start worker process 21
2021/02/25 13:01:54 [notice] 1#0: start worker process 22
2021/02/25 13:01:54 [notice] 21#0: *1 [lua] cache.lua:374: purge(): [DB cache] purging (local) cache, context: init_worker_by_lua*
2021/02/25 13:01:54 [notice] 21#0: *1 [lua] cache.lua:374: purge(): [DB cache] purging (local) cache, context: init_worker_by_lua*
<IP_ADDRESS> - - [25/Feb/2021:13:01:55 +0000] "GET / HTTP/2.0" 200 8834 "-" "Go-http-client/2.0"
<IP_ADDRESS> - - [25/Feb/2021:13:01:55 +0000] "GET /tags HTTP/2.0" 200 23 "-" "Go-http-client/2.0"
2021/02/25 13:01:55 [notice] 22#0: *5 [lua] cache.lua:374: purge(): [DB cache] purging (local) cache, client: <IP_ADDRESS>, server: kong_admin, request: "POST /config?check_hash=1 HTTP/2.0", host: "localhost:8444"
2021/02/25 13:01:55 [notice] 22#0: *5 [lua] cache.lua:374: purge(): [DB cache] purging (local) cache, client: <IP_ADDRESS>, server: kong_admin, request: "POST /config?check_hash=1 HTTP/2.0", host: "localhost:8444"
<IP_ADDRESS> - - [25/Feb/2021:13:01:56 +0000] "POST /config?check_hash=1 HTTP/2.0" 201 2 "-" "Go-http-client/2.0"
<IP_ADDRESS> - - [25/Feb/2021:13:03:57 +0000] "GET / HTTP/1.1" 404 48 "-" "curl/7.54.0"
<IP_ADDRESS> - - [25/Feb/2021:13:05:59 +0000] "GET / HTTP/1.1" 404 48 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.182 Safari/537.36"
<IP_ADDRESS> - - [25/Feb/2021:13:06:04 +0000] "GET /productpage HTTP/1.1" 404 48 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.182 Safari/537.36"
<IP_ADDRESS> - - [25/Feb/2021:13:06:52 +0000] "GET /productpage HTTP/1.1" 404 48 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.182 Safari/537.36"
<IP_ADDRESS> - - [25/Feb/2021:13:06:54 +0000] "GET /productpage HTTP/1.1" 404 48 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.182 Safari/537.36"

$ kubectl logs -f example-kong-kong-6567cb7cdc-sqfm7 -c istio-proxy -n kong-istio
...
[2021-02-25T15:16:33.053Z] "- - -" 0 - "-" "-" 0 0 68781 - "-" "-" "-" "-" "<IP_ADDRESS>:8000" inbound|80|kong-proxy|example-kong-kong-proxy.kong-istio.svc.cluster.local <IP_ADDRESS>:47562 <IP_ADDRESS>:8000 <IP_ADDRESS>:58923 - -
[2021-02-25T15:17:41.834Z] "GET /productpage HTTP/1.1" 404 - "-" "-" 0 48 1 0 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.182 Safari/537.36" "e8df1d9c-5a8c-9eb2-b421-db608528dde4" "<IP_ADDRESS>" "<IP_ADDRESS>:8000" inbound|80|kong-proxy|example-kong-kong-proxy.kong-istio.svc.cluster.local <IP_ADDRESS>:47532 <IP_ADDRESS>:8000 <IP_ADDRESS>:43010 - default
[2021-02-25T15:17:49.196Z] "GET /productpage HTTP/1.1" 404 - "-" "-" 0 48 1 1 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.182 Safari/537.36" "05eb66e5-ed17-9c0a-95f4-687ee3125c79" "<IP_ADDRESS>" "<IP_ADDRESS>:8000" inbound|80|kong-proxy|example-kong-kong-proxy.kong-istio.svc.cluster.local <IP_ADDRESS>:47532 <IP_ADDRESS>:8000 <IP_ADDRESS>:43010 - default

$ kubectl logs -f example-kong-kong-6567cb7cdc-sqfm7 -c ingress-controller -n kong-istio
-------------------------------------------------------------------------------
Kong Ingress controller
  Release:    1.1.1
  Build:      ba4eea9
  Repository: git@github.com:Kong/kubernetes-ingress-controller.git
  Go:         go1.15.3
-------------------------------------------------------------------------------

W0225 13:01:52.312735       1 client_config.go:608] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
time="2021-02-25T13:01:52Z" level=warning msg="failed to fetch version of kubernetes api-server: Get \"https://<IP_ADDRESS>:443/version?timeout=32s\": dial tcp <IP_ADDRESS>:443: connect: connection refused" api-server-host="https://<IP_ADDRESS>:443" retry_count=0
time="2021-02-25T13:01:53Z" level=warning msg="failed to fetch version of kubernetes api-server: Get \"https://<IP_ADDRESS>:443/version?timeout=32s\": dial tcp <IP_ADDRESS>:443: connect: connection refused" api-server-host="https://<IP_ADDRESS>:443" retry_count=1
time="2021-02-25T13:01:55Z" level=info msg="version of kubernetes api-server: 1.18+" api-server-host="https://<IP_ADDRESS>:443" git_commit=199a41188dc0ca5d6d95b1cc7e8ba96e05f9dd0a git_tree_state=clean git_version=v1.18.12-gke.1210 major=1 minor=18+ platform=linux/amd64
time="2021-02-25T13:01:55Z" level=info msg="kong version: 2.2.1" kong_version=2.2.1
time="2021-02-25T13:01:55Z" level=info msg="datastore strategy for kong: off"
time="2021-02-25T13:01:55Z" level=info msg="chosen Ingress API version: networking.k8s.io/v1beta1"
time="2021-02-25T13:01:55Z" level=info msg="starting leader election" component=elector retry-count=1
I0225 13:01:55.624670       1 leaderelection.go:243] attempting to acquire leader lease  kong-istio/kong-ingress-controller-leader-kong-kong...
time="2021-02-25T13:01:55Z" level=info msg="syncing configuration" component=controller
I0225 13:01:55.636465       1 leaderelection.go:253] successfully acquired lease kong-istio/kong-ingress-controller-leader-kong-kong
time="2021-02-25T13:01:55Z" level=info msg="started leading" component=status-syncer
time="2021-02-25T13:01:55Z" level=info msg="leadership changed" component=status-syncer leader=example-kong-kong-6567cb7cdc-sqfm7
time="2021-02-25T13:01:56Z" level=info msg="successfully synced configuration to kong" component=controller
...
time="2021-02-25T15:21:58Z" level=info msg="syncing configuration" component=controller
time="2021-02-25T15:21:58Z" level=info msg="no configuration change, skipping sync to kong" component=controller

(additional context: originating thread on Kubernetes slack)

@shaneutt Any update on this ticket?

@vemahendran Is your ingress exposed on $PROXY_URL/productpage ?
Can you provide your ingress manifest?

@andrzej-jedrzejewski Here is the ingress manifest for product page?
$ kubectl get ingress productpage -n my-istio-app -o yaml
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  annotations:
    konghq.com/plugins: rate-limit
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"extensions/v1beta1","kind":"Ingress","metadata":{"annotations":{"konghq.com/plugins":"rate-limit"},"name":"productpage","namespace":"my-istio-app"},"spec":{"rules":[{"http":{"paths":[{"backend":{"serviceName":"productpage","servicePort":9080},"path":"/"}]}}]}}
  creationTimestamp: "2021-02-25T13:05:41Z"
  generation: 1
  managedFields:
  - apiVersion: extensions/v1beta1
    fieldsType: FieldsV1
    fieldsV1:
      f:metadata:
        f:annotations:
          .: {}
          f:konghq.com/plugins: {}
          f:kubectl.kubernetes.io/last-applied-configuration: {}
      f:spec:
        f:rules: {}
    manager: kubectl
    operation: Update
    time: "2021-02-25T13:05:41Z"
  name: productpage
  namespace: my-istio-app
  resourceVersion: "95469"
  selfLink: /apis/extensions/v1beta1/namespaces/my-istio-app/ingresses/productpage
  uid: 5e4b9394-59dc-41c7-a135-d14daa9f09dc
spec:
  rules:
  - http:
      paths:
      - backend:
          serviceName: productpage
          servicePort: 9080
        path: /
        pathType: ImplementationSpecific
status:
  loadBalancer: {}

looks ok to me. Can you also provide productpage service manifest and output from: istioctl proxy-status command, please?

@andrzejjedrzejejwski  here you go,
The productpage service manifest:
$ kubectl get service/productpage -n my-istio-app -o yaml
apiVersion: v1
kind: Service
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"v1","kind":"Service","metadata":{"annotations":{},"labels":{"app":"productpage","service":"productpage"},"name":"productpage","namespace":"my-istio-app"},"spec":{"ports":[{"name":"http","port":9080}],"selector":{"app":"productpage"}}}
  creationTimestamp: "2021-02-25T13:04:32Z"
  labels:
    app: productpage
    service: productpage
  managedFields:
  - apiVersion: v1
    fieldsType: FieldsV1
    fieldsV1:
      f:metadata:
        f:annotations:
          .: {}
          f:kubectl.kubernetes.io/last-applied-configuration: {}
        f:labels:
          .: {}
          f:app: {}
          f:service: {}
      f:spec:
        f:ports:
          .: {}
          k:{"port":9080,"protocol":"TCP"}:
            .: {}
            f:name: {}
            f:port: {}
            f:protocol: {}
            f:targetPort: {}
        f:selector:
          .: {}
          f:app: {}
        f:sessionAffinity: {}
        f:type: {}
    manager: kubectl
    operation: Update
    time: "2021-02-25T13:04:32Z"
  name: productpage
  namespace: my-istio-app
  resourceVersion: "94892"
  selfLink: /api/v1/namespaces/my-istio-app/services/productpage
  uid: 34a9a35c-4b42-4858-8906-08e49542dc10
spec:
  clusterIP: <IP_ADDRESS>
  ports:
  - name: http
    port: 9080
    protocol: TCP
    targetPort: 9080
  selector:
    app: productpage
  sessionAffinity: None
  type: ClusterIP
status:
  loadBalancer: {}

The proxy-status output,
$ istioctl proxy-status
NAME                                                   CDS        LDS        EDS        RDS          PILOT                     VERSION
details-v1-558b8b4b76-2579n.my-istio-app               SYNCED     SYNCED     SYNCED     SYNCED       istiod-c5f8dc9f-klqz6     1.6.7
example-kong-kong-6567cb7cdc-sqfm7.kong-istio          SYNCED     SYNCED     SYNCED     SYNCED       istiod-c5f8dc9f-klqz6     1.6.7
istio-egressgateway-85f9d8d8f6-cvxv4.istio-system      SYNCED     SYNCED     SYNCED     NOT SENT     istiod-c5f8dc9f-klqz6     1.6.7
istio-ingressgateway-84dd75f677-ltfgf.istio-system     SYNCED     SYNCED     SYNCED     NOT SENT     istiod-c5f8dc9f-klqz6     1.6.7
productpage-v1-6987489c74-chjp9.my-istio-app           SYNCED     SYNCED     SYNCED     SYNCED       istiod-c5f8dc9f-klqz6     1.6.7
prometheus-5ccccff68c-bprz5.istio-system               SYNCED     SYNCED     SYNCED     SYNCED       istiod-c5f8dc9f-klqz6     1.6.7
ratings-v1-7dc98c7588-2mvv9.my-istio-app               SYNCED     SYNCED     SYNCED     SYNCED       istiod-c5f8dc9f-klqz6     1.6.7
reviews-v1-7f99cc4496-hn4mf.my-istio-app               SYNCED     SYNCED     SYNCED     SYNCED       istiod-c5f8dc9f-klqz6     1.6.7
reviews-v2-7d79d5bd5d-t5nxk.my-istio-app               SYNCED     SYNCED     SYNCED     SYNCED       istiod-c5f8dc9f-klqz6     1.6.7
reviews-v3-7dbcdcbc56-rb4tg.my-istio-app               SYNCED     SYNCED     SYNCED     SYNCED       istiod-c5f8dc9f-klqz6     1.6.7

Regarding configuration.konghq.com: do-not-preserve-host text, do I need to manually add this in annotations?
here is KongPlugin manifest for rate-limit,
$ kubectl get KongPlugin -n my-istio-app -o yaml
apiVersion: v1
items:
- apiVersion: configuration.konghq.com/v1
  config:
    minute: 30
    policy: local
  kind: KongPlugin
  metadata:
    annotations:
      kubectl.kubernetes.io/last-applied-configuration: |
        {"apiVersion":"configuration.konghq.com/v1","config":{"minute":30,"policy":"local"},"kind":"KongPlugin","metadata":{"annotations":{},"name":"rate-limit","namespace":"my-istio-app"},"plugin":"rate-limiting"}
    creationTimestamp: "2021-02-25T13:05:29Z"
    generation: 1
    managedFields:
    - apiVersion: configuration.konghq.com/v1
      fieldsType: FieldsV1
      fieldsV1:
        f:config:
          .: {}
          f:minute: {}
          f:policy: {}
        f:metadata:
          f:annotations:
            .: {}
            f:kubectl.kubernetes.io/last-applied-configuration: {}
        f:plugin: {}
      manager: kubectl
      operation: Update
      time: "2021-02-25T13:05:29Z"
    name: rate-limit
    namespace: my-istio-app
    resourceVersion: "95358"
    selfLink: /apis/configuration.konghq.com/v1/namespaces/my-istio-app/kongplugins/rate-limit
    uid: 35205ef9-3fdf-4f0d-a5b6-f2029f021ca0
  plugin: rate-limiting
kind: List
metadata:
  resourceVersion: ""
  selfLink: ""

For sure, You are missing two things:
kubectl annotate service productpage ingress.kubernetes.io/service-upstream=true
and proper KongIngress configuration:
apiVersion: configuration.konghq.com/v1 kind: KongIngress metadata:     name: do-not-preserve-host route:   preserve_host: false upstream:   host_header: productpage.my-istio-app.svc

@andrzejjedrzejejwski  This is just to update the thread. I tried applying the missing annotation and the mentioned KongIngress manifest on my-istio-app namespace but It didn't resolve issue. I will try again deleting my cluster and do it from scratch and apply the KongIngress and missing annotations.

@andrzejjedrzejejwski This is just to update the thread. I tried applying the missing annotation and the mentioned KongIngress manifest on my-istio-app namespace but It didn't resolve issue. I will try again deleting my cluster and do it from scratch and apply the KongIngress and missing annotations.

do you solve this problem? I want to replace kong-ingress to apigateway too


ShaderBinaryBug Again!

Hi Everyone
I am really sorry for this but i thought that I Used an Intel GPU while i use an amd GPU:

Now, What is the solutoin for my problem?
My problem was that I am getting an opengl invalid enum and i think that the problem is due to GL_SHADER_BINARY_FORMAT_SPIR_V.
Here goes the image for the error that's occuring :

Now, if I am using an AMD GPU why that problem occurs @VagueLobster ?

Top line in the console output says

Renderer: Intel(R) HD Graphics 520
I'm gonna assume you're on a laptop, since laptops usually have a thing where they default to your CPU's integrated graphics and then use the GPU when you explicitly tell it to. My suggestion is to look up how to do this for AMD unless VagueLobster knows.

Another possibility if you use a desktop PC is that you connected your monitor to the motherboard instead of your graphics card. If this is the case, reconnect the monitor cable from the motherboard slot at the back of your PC and instead connect it to the GPU card itself. Then the problem should disappear.

Top line in the console output says

Renderer: Intel(R) HD Graphics 520

I'm gonna assume you're on a laptop, since laptops usually have a thing where they default to your CPU's integrated graphics and then use the GPU when you explicitly tell it to. My suggestion is to look up how to do this for AMD unless VagueLobster knows.
Another possibility if you use a desktop PC is that you connected your monitor to the motherboard instead of your graphics card. This causes your main GPU to be that of the motherboard, which is the CPU integrated graphics. (This is also why laptops default to the integrated GPU from the CPU, since the laptop monitors usually connect to the motherboard.) If this is the case, reconnect the monitor cable from the motherboard slot at the back of your PC and instead connect it to the GPU card itself. Then the problem should disappear.

@FlyingStudent I have to agree with Kirdow: Look it up on google on how to this for AMD, as i am using an Nvidia GPU myself!

Thanks you all I am really thankful for having such gorgeous community to help me ❤❤
I am going to try it out and tell you with the result before closing the comment.
Thanks again for your help @VagueLobstor, @Kirdow

Morning Everyone!
My problem ain't solved yet.
I learned how to change the current active GPU to AMD so i changed visual studio to use but as it turned out, I need to change the project I am running itself to use it because even though visual studio is using the AMD GPU my project runs on intel as the debugging window shows when i run my project.
Can anyone help me with that ?
Thank you all for support I really appreciate it .
Have a nice day

Update:
problem is nearly solved.
when i plug my power adapter it switch to the AMD GPU but when there's no power it doesn't automatically.
Working on it

Problem is solved !
Now, I am using my AMD GPU instead of my intel GPU.
Thank you all for your support. It was really helpful
Thanks Yan for this beautiful community that you have created ❤❤
Thanks @Kirdow , @VagueLobster  for your support . you were a huge help ❤

Please close the issue.


Error in instructions

In section Run the program, there are a couple of references to updates needed in GraphInstanceSet and GraphTopologyDelete:
Next, under the GraphInstanceSet and GraphTopologyDelete nodes, ensure that the value of topologyName matches the value of the name property in the previous graph topology:

"topologyName" : "CVRToAMSAsset""

In the latest version of the samples, however, which I've run just 3 days ago, these instructions don't apply to GraphTopologyDelete. This node has only:
{
            "opName": "GraphInstanceDelete",
            "opParams": {
                "name": "Sample-Graph-1"
            }
        },

And it has no topologyName as the text says.
I believe the other LVA  tutorial, for event-triggered recording, has the same error, only twice, also for the topologyUrl.

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

ID: 83493a18-75d3-21a6-a6b7-30f2c453ff4f
Version Independent ID: 9b48bec6-b5e2-befa-e11f-04daa78bd95e
Content: Continuous video recording to the cloud and playback from the cloud tutorial - Azure - Live Video Analytics on IoT Edge
Content Source: articles/media-services/live-video-analytics-edge/continuous-video-recording-tutorial.md
Service: media-services
Sub-service: lva-edge
GitHub Login: @Juliako
Microsoft Alias: juliako

Thanks for the feedback! We are currently investigating and will update you shortly.

Docs will be updated @lokijota. Thanks for the call out.

@lokijota I spoke the product group and I'll be rolling back that change as GraphTopologyDelete is indeed valid. The Name property should be updated the topologyName CVRToAMSAsset.


Custom Voice messages

I can't find detailed enough documentation about Custom Voice Messages.
When adding a custom greeting, there is no mention of what the 'Application' field does.
We'd like to know what the default wording is for each message in order to write our own custom messages.
The closest documentation is the table on this discussion: https://social.msdn.microsoft.com/Forums/en-US/5e8edc13-4888-4c51-b890-341c2bad0630/listen-to-default-mfa-voice-messages-to-create-new-custom?forum=windowsazureactiveauthentication
...but it is out of date as it does not say what the greeting name 'Activation' is and it includes 'Verification Denied Retry' which is no longer an option.

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

ID: 84b26750-880c-8f14-1628-436293de915e
Version Independent ID: 518b727f-79e8-b5c2-bcb3-ff4de3444c6f
Content: Configure Azure Multi-Factor Authentication - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-mfa-mfasettings.md
Service: active-directory
Sub-service: authentication
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

Hi @johncrowtherdcc Thank you for your feedback! We will review and provide an update as appropriate.

Hi,
Further to my first post, the old documentation also doesn't mention the greeting 'Authentication denied retry'.
John.

@johncrowtherdcc We are engaging the content team to update the documentation on this one.
@MicrosoftGuyJFlo Could you please have this verified with someone in engineering and see how best we can update the information on this page to support the customers ?

Pending feedback from the product group.

Added list of messages and default script to doc.

Thanks! This is exactly what I was looking for.
I think there is one tweak needed. The document has a message name of "Fraud reported	A fraud alert has been submitted." whereas the message name is really just "Fraud reported".


Is this codelab still accurate?  getting the following errors...

index.esm.js:356 Uncaught
{code: "app/no-app", message: "Firebase: No Firebase App '[DEFAULT]' has been cre…- call Firebase App.initializeApp() (app/no-app).", name: "[DEFAULT]", stack: "[DEFAULT]: Firebase: No Firebase App '[DEFAULT]' h…s.firebaseapp.com/scripts/FriendlyEats.js:196:16)"}
code
:
"app/no-app"
message
:
"Firebase: No Firebase App '[DEFAULT]' has been created - call Firebase App.initializeApp() (app/no-app)."
name
:
"[DEFAULT]"
stack
:
"[DEFAULT]: Firebase: No Firebase App '[DEFAULT]' has been created - call Firebase App.initializeApp() (app/no-app).↵    at cr (https://fs3friendlyeats.firebaseapp.com//firebase/5.1.0/firebase-app.js:1:32616)↵    at i (https://fs3friendlyeats.firebaseapp.com//firebase/5.1.0/firebase-app.js:1:34457)↵    at Object.f [as auth] (https://fs3friendlyeats.firebaseapp.com/__/firebase/5.1.0/firebase-app.js:1:33959)↵    at new FriendlyEats (https://fs3friendlyeats.firebaseapp.com/scripts/FriendlyEats.js:32:12)↵    at window.onload (https://fs3friendlyeats.firebaseapp.com/scripts/FriendlyEats.js:196:16)"
proto
:
Error

@mingsf do you have a hosted version of your codelab somewhere we could have a look? You can deploy a version of your codelab by doing firebase deploy and please provide us the URL.

Closing for inactivity


Tests refactoring

🚀 Feature
thinking about https://github.com/PyTorchLightning/metrics/pull/676#issuecomment-991161059 raised by @lucadiliello
we can make an assumption that there shall not be too many differences among the Python versions and OS, right?
in such a case we can have three lines of testing:

doctest on the package [we have now] which is testing examples in docs and imports
some general and most common tests [need to be specified] which would be reasonably fast and run and we run it on all configurations as it now
running full tests [as it is now] on Linux all python versions

Motivation
Boost the CI response but still, keep high test coverage.
Pitch

Alternatives
We can also think to run some limited testing on PRs and then set command which would before merging run some extensive testing. This extensive testing would also run on commit merge master
Additional context

Thoughts?
cc: @PyTorchLightning/core-metrics

#675
#733

In my experience, most times the difference lies in the Pytorch version that is being tested where something may work for version (x+1) but not for version x. So I would still do full Pytorch testing on each PR.

lest also explore auto-canceling via group: https://github.com/marketplace/actions/auto-cancel-redundant-workflow

concurrency is preferred because the auto-cancel repo is no longer maintained

we can benefit from principle that domains are independent, so we can for each PR determine if it is related only to domain level (adding new NLP metric) or it affects all metrics (for example changes in the base Metrics class)

Another point for discussion about static testing references #949


There are several different types of dependencies that should be transparently managed.

Containenv has the task of providing an all-in-one sandbox that seemlessly fulfills all the necessary functions for a given development process.
This means that the image produced for a given project needs an intelligently composed set of functions.
Discussions with @dangle have led to a tentative partition of image components into several categories:
(0)  project necessary (e.g. for a Python project Python itself will obviously always need to be installed)
(1)   team conventional (e.g. for a Python project some process may have dependencies that are commonly used by the team, but are not strictly required)
(2)   personal tools (e.g. for every project a developer may have a preferred tool set that should be installed into the containenv)
(3)   active user state
Obviously this partition is poorly defined, and requires refinement, moreover even when we converge on a reasonable partition of dependencies, how they will be handled will require implementation.

I believe (0) and (1) should be reified as separate Dockerfiles that live in the '.containenv' directory in the metasource (top level) directory of the 'contained' project.   I believe (0) and (1) should be composed at build time.
NOTE:  I hope to be pluggable with a rkt backend, but am constrained by ignorance to used Docker terminology.
I believe (2) should be reified as a Dockerfile in `~/.containenv'...   it should then also be composed...
I think I'm starting to believe that the containenv init command must consist of multiple build commands run in a strict order.
(3) should be -v mounted at run time.

Folks the products of this issue will let us know a lot about what other issues we need to create.

We've decided that "personal customizations" should be configured in:
PROJECT/.containenvrc which should be referenced in .gitignore.
This file is strictly OPTIONAL.

OK, the result is:
PROJECT DEPENDENCIES:
PROJECT/.containenv   -- mandatory, contains project dependencies, managed with revision control, shared
USER-PROJECT CONFIG:
PROJECT/.containenvrc -- optional, contains user dependencies, referenced in .gitignore, private
USER-GLOBAL CONFIG:
~/.containenvrc   -- optional, global-per-user config, private

Let's do this, once you're happy with the design, unassign yourself from the issue.
[11:57]
UNLESS your the last assignee...   in which case you should close the issue.

This is complete.

Oh...  this is closed when @szakeri closes it.

Thanks @szakeri !


cucumber-tag-expressions library naming

Summary
Naming of the tag-expressions library is inconsistent:

folder in monorepo is tag-expressions
npm package is cucumber-tag-expressions
ruby gem is cucumber-tag_expressions
Java package is tag-expressions

A linked problem is that we can not use file .internal-dependencies to link the package in the mono-repo as it will create a symlink in node_modules/tag-expressions which won't replace node_modules/cucumber-tag-expressions
Expected Behavior
The name of the folder should reflect the name of the package (which should be unique, not depending on implementations)
Possible Solution
Rename all "packages" as tag-expressions or cucumber-tag-expressions

Good writeup. We should prefix all packages with cucumber-. For NPM we could also use the @cucumber org - that would be cool. We just need to reclaim the org first - see https://cucumberbdd.slack.com/archives/C612KCP1P/p1574951184001100

Fixed by https://github.com/cucumber/cucumber/pull/825


ocamlPackages.git: 1.11.4 -> 1.11.5

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build ocamlPackages.git-unix

Success on x86_64-linux (full log)
Attempted: ocamlPackages.git-unix
Partial log (click to expand)
moving /nix/store/0qiskn0dcvl1xjdkf3q157n6jrsnipc4-ocaml4.05.0-git-unix-1.11.5/doc to /nix/store/0qiskn0dcvl1xjdkf3q157n6jrsnipc4-ocaml4.05.0-git-unix-1.11.5/share/doc
shrinking RPATHs of ELF executables and libraries in /nix/store/0qiskn0dcvl1xjdkf3q157n6jrsnipc4-ocaml4.05.0-git-unix-1.11.5
shrinking /nix/store/0qiskn0dcvl1xjdkf3q157n6jrsnipc4-ocaml4.05.0-git-unix-1.11.5/bin/ogit
shrinking /nix/store/0qiskn0dcvl1xjdkf3q157n6jrsnipc4-ocaml4.05.0-git-unix-1.11.5/lib/ocaml/4.05.0/site-lib/git-unix/git_unix.cmxs
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/0qiskn0dcvl1xjdkf3q157n6jrsnipc4-ocaml4.05.0-git-unix-1.11.5/lib  /nix/store/0qiskn0dcvl1xjdkf3q157n6jrsnipc4-ocaml4.05.0-git-unix-1.11.5/bin
patching script interpreter paths in /nix/store/0qiskn0dcvl1xjdkf3q157n6jrsnipc4-ocaml4.05.0-git-unix-1.11.5
checking for references to /build in /nix/store/0qiskn0dcvl1xjdkf3q157n6jrsnipc4-ocaml4.05.0-git-unix-1.11.5...
warning: SQLite database '/nix/var/nix/db/db.sqlite' is busy
/nix/store/0qiskn0dcvl1xjdkf3q157n6jrsnipc4-ocaml4.05.0-git-unix-1.11.5

Failure on aarch64-linux (full log)
Attempted: ocamlPackages.git-unix
Partial log (click to expand)
cannot build derivation '/nix/store/wyv8ih5yi9p1j509abk0hncr2j0jzpgh-ocaml4.05.0-git-1.11.5.drv': 16 dependencies couldn't be built
cannot build derivation '/nix/store/ars847rgr9qpvmswd4ci0dk2gsz0xqbm-ocaml4.05.0-x509-0.6.1.drv': 11 dependencies couldn't be built
cannot build derivation '/nix/store/48p2wnqdqx20hzqzz6ghzg963cyn0zkr-ocaml4.05.0-cohttp-lwt-1.0.2.drv': 8 dependencies couldn't be built
cannot build derivation '/nix/store/rfyczwnf69n3nbslvymifzwcphiyv5qb-ocaml4.05.0-conduit-lwt-1.0.0.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/4vvnjv6fl8ka1nzhsa7m59rgfsfcs90n-ocaml4.05.0-tls-0.9.0.drv': 14 dependencies couldn't be built
cannot build derivation '/nix/store/80frinlhf0kcsvz3j6992mvh15y7k17b-ocaml4.05.0-conduit-lwt-unix-1.0.0.drv': 7 dependencies couldn't be built
cannot build derivation '/nix/store/m4fy39fxd146qg6z9796n8ifi6x48j58-ocaml4.05.0-git-http-1.11.5.drv': 9 dependencies couldn't be built
cannot build derivation '/nix/store/9gpygs14ja1d2nigsyaix49gf7pl554i-ocaml4.05.0-cohttp-lwt-unix-1.0.2.drv': 10 dependencies couldn't be built
cannot build derivation '/nix/store/qcbmhpqa1x4hd1yicgdfng6grfrqiaf8-ocaml4.05.0-git-unix-1.11.5.drv': 9 dependencies couldn't be built
error: build of '/nix/store/qcbmhpqa1x4hd1yicgdfng6grfrqiaf8-ocaml4.05.0-git-unix-1.11.5.drv' failed


[SPARK-25856][SQL][MINOR] Remove AverageLike and CountLike classes

What changes were proposed in this pull request?
These two classes were added for regr_ expression support (SPARK-23907). These have been removed and hence we can remove these base classes and inline the logic in the concrete classes.
How was this patch tested?
Existing tests.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4548/
Test PASSed.

Test build #98106 has started for PR 22856 at commit 6becdd5.

Test build #98106 has finished for PR 22856 at commit 6becdd5.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Average(child: Expression) extends DeclarativeAggregate with ImplicitCastInputTypes
case class Count(children: Seq[Expression]) extends DeclarativeAggregate

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98106/
Test PASSed.

LGTM

Merged to master


Remove ga4gh-dream from s3 cache since it is used for others as well

Generalize the s3 cache path so other non-ga4gh-dream workflows can also have a cache

Coverage remained the same at 79.714% when pulling d28ac6853f0a74e2701ecabdccd2e13ff4927f6f on feature/generalizeS3Cache into 897a2803fcd9c24d31fdd35fa55c2623c53cc50b on develop.


Is it possible to use age and gpg backends simultaneously?

Summary
I have an existing gpg backed gopass setup that I've been using, and I'd like to have a new store that's age encrypted.  Is it possible to use both?  I created a new store with the steps below, and added a secret to it, but when it decrypted it used my gpg key instead of asking for my age passphrase.
If it's the answer is it's not possible to do, can you think of any workarounds?  Create a new gopass config and point at that instead of my existing config?
Steps To Reproduce

gopass init --path /home/kfken/.age-store --store agepass --crypto age
create -s agepass test
show test
prompted for gpg PIN

Expected behavior
Expected to be prompted for age passphrase and that's used to decrypt it
Environment

OS: Debian
OS version: 5.10.0-5-amd64 #1 SMP Debian 5.10.24-1 (2021-03-19) x86_64 GNU/Linux
gopass Version:

gopass 1.12.4-git+HEAD go1.16.2 linux amd64
<root>     - gpg 2.2.27 - git 2.31.1
agepass    -  age 0.0.1 - git 2.31.1

Installation method: built from source

Looking for options that don't' involve completely converting my gopass gpg encrypted secrets to all age secrets for now.  I definitely want to do that, but have been waiting for age yubkiey support.  I may just end up doing a full conversion anyway, but wanted to test out/experiment with a dual setup for the time being.

Sure, every mount can use a different Backend.
I think the problem is the -s flag to create. That is not supported.
You need to specify the correct path (gopass create -f age store/test) or select the correct store when asked.

Oh, actually the -s flag should be supported. Strange.

Ok, so gopass create with -s AND a secret name ignores the specified store.
If you omit either -s or the name (here: test) it will work.

Ok, so gopass create with -s AND a secret name ignores the specified store.
If you omit either -s or the name (here: test) it will work.

Ok, looks like it worked when I did create -s agepass.  If I omitted -s, I was prompted for a store, but if I chose agepass store, it still used my gpg keys to encrypt.
One other question: is it intentional that if you're in the "shell" environment ( gopass is run without args), that the age passphrase is cached?  I wasn't prompted for it again as long as I was in the shell session?

Using the gpg store when you select the agepass store sounds like a bug. However I did test it locally and couldn't observe anything like that.
The answer to the other question is a clear yes. The REPL (shell) mode was designed for use with the age backend because the gopass agent implementation was so poorly designed (by me) that I had to remove it to not risk security issues there. But without passphrase caching the age backend can be quite annoying to use. This is why we have the REPL shell.

Using the gpg store when you select the agepass store sounds like a bug. However I did test it locally and couldn't observe anything like that.

Here's the steps I took:
❯ gopass create agetest2
🌟 Welcome to the secret creation wizard (gopass create)!
🧪 Hint: Use 'gopass edit -c' for more control!
[ 0] Website Login
[ 1] PIN Code (numerical)
[ 2] Generic

Please select the type of secret you would like to create (q to abort) [0]: 2
2
🧪 Creating generic secret ...
  [1] Name                                   []: agetest2
  [2] Generate password?                     [Y/n/q]:
    [a] Human-pronounceable passphrase?      [y/N/q]:
    [b] How long?                            (q to abort) [24]:
    [c] Include symbols?                     [y/N/q]:
    [d] Strict rules?                        [y/N/q]:
[ 0] <root>
[ 1] agepass

Please select the store you would like to use (q to abort) [0]: 1
1
  [3] Enter zero or more key value pairs for this secret:
    [a] Name (enter to quit)                 []:
✅ Credentials saved to "agetest2"
✔ Copied agetest2 to clipboard. Will clear in 45 seconds.
❯ gopass show agetest2

Then I'm prompted for the PIN for yubikey opengpg slot, after which the new entry is displayed.
❯ gopass ls | head -n 10
gopass
├── agepass (/home/kfken/.age-store)
│   └── misc/
│       └── agetest
├── agetest2

The answer to the other question is a clear yes. The REPL (shell) mode was designed for use with the age backend because the gopass agent implementation was so poorly designed (by me) that I had to remove it to not risk security issues there. But without passphrase caching the age backend can be quite annoying to use. This is why we have the REPL shell.

Makes sense, I like the feature.

Thanks. This is indeed a bug in how a handle arguments to gopass create.


Integrate all submodules via postBuild to Python and R path

Sub-modules are not installed, they are only a code-clone of another repository into the sandbox project.
In order to make this usable in notebooks, there is a need to add the path to each of the sub-module in an automatic way to the project.
This may be realized by using the postBuild process.

From the mybinder docs I would say that it is better to use the start file that is run just before the user sessions starts. The postBuild is ran when the container is built, which, in our case, is now done from the separate env repository, where we actually do not yet have the submodules available. Hence, my hope would be to run the start script from our code repo directly when a new binder instance is launched.

For RAINLINK to run:

You need to switch to working directory “RAINLINK” (getwd("RAINLINK") in R shell).
Make a subdirectory “Figures”.
Install RAINLINK via command in R shell: install.packages("RAINLINK_1.3.tar.gz", repos=NULL,type="source")

R CMD INSTALL RAINLINK_1.3.tar.gz

Note that RAINLINK has been successfully tested using R, but R libraries from the Config.R file cannot (all) be loaded properly when using RStudio.

Things did not work out as planed at the beginning of this issue. Submodules are not initialized via the solution provided by #38.


Collaborate on a nunustudio project

Hi,
Is there a way for several people to collaborate on a nunustudio project? Or at least to merge 2 nunu projects together?

Hello
There is not a good proper way for people to colaborate in nunuStudio projects currently.
The closer thing to merging project is importing one of them into the other.
Cheers


[WIP] Test deploying caasp4 using containers

Hummm. httpException: 500: Server Error for url: https://engcloud.prv.suse.net:8774/v2.1/a5183948e4da445b9bb5cb0f05b74698/servers

Ofc this is just the start, to ensure terraform. All the rest would still need work.


Add “AvoidSpecificTermsInModuleNames” check

Now that we’re using credo_naming, we can take advantage of its other checks.
This pull requests adds the CredoNaming.Check.Warning.AvoidSpecificTermsInModuleNames check, which can prevent us from using OOP-style terms in our module names.
Here is the list of terms I’m suggesting we use for the moment:

Manager
Fetcher
Builder
Persister
Serializer
Helper
Helpers

I think this will make it easier for us to stick to our naming guidelines.

🦀 Requesting reviewers for this pull request:

@thermech (contributor with 0 commits in the last 90 days and 8 commits overall)
@olivierbuffon (expert reviewer for the elixir stack)


Increase font atlas dimensions guesstimate

Increase atlas size guesstimate, print warnings if atlas size is too small, and make sure all rectangles are properly initialized.
Fixes #2356.

@megagrump Thanks for the review! I'm merging this improvement but actually, afair, that value was decreased some time ago because using 1.4f generated very big texture atlas with empty space. Probably a best estimation mechanism should be designed.

Yeah, in my specific case it couldn't fit the last 5 glyphs in an estimated 512x512 atlas. After the change, a 1024x1024 atlas is used, with lots of empty space. 1024x512 would have been sufficient here.
A simple changle to improve packing efficiency would be sorting the glyphs by height beforehand.
I'll come back to this when I find the time and see if I can improve atlas generation.


Cannot read property 'appendChild' of undefined

Details
What browser were you using: Chrome - Version 86.0.4240.111 (Official Build) (64-bit)
What OS were you using: Windows 10 Pro - Version 1909OS Build 18363.1139
Project version you have: Master /build
Describe the bug

Importing the /build directory files directly into html causes a TypeError: "Cannot read property 'appendChild' of undefined".
Selecting a button element using the readme example causes a TypeError: "Cannot read property 'style' of null".

To Reproduce
Use the following HTML content, assuming the two build files are located relative to the HTML file in ../assets/js and assets/css.
For error 1. above, simply view the error in a browser console.
For error 2. above, click on the button and view the error in a browser console.
<!DOCTYPE HTML>
<html lang="en">
	<head>
		<script type="text/javascript" src="../assets/js/colr_pickr.min.js"></script>
		<link rel="stylesheet" type="text/css" href="../assets/css/colr_pickr.min.css" />
    </head>
    <body>
        <button id="test-button">Test</button>
    </body>
    <script type="text/javascript">
        const button = document.getElementById('test-button');
        let picker = new ColorPicker(button, '#ffcc99');
        button.addEventListener('colorChange', function (event) {
            const color = event.detail.color.hex;
        });
    </script>
</html>

Screenshots
Error 1

Error 2

Put the script tag you use the call the library below the HTML, thats proabably the problem.
I'll look in to not having to do this in the future devlopments.
Thanks, TEK

@fluxmaps

Ah beauty! Thanks for the speedy reply. The following script order now works:
<!DOCTYPE HTML>
	<head>
		<link rel="stylesheet" type="text/css" href="../assets/css/colr_pickr.min.css" />
    </head>
    <body>
        <button id="test-button">Test</button>
    </body>
</html>
<script type="text/javascript" src="../assets/js/colr_pickr.min.js"></script>
<script type="text/javascript">
    const button = document.getElementById('test-button');
    let picker = new ColorPicker(button, '#ffcc99');
    button.addEventListener('colorChange', function (event) {
        const color = event.detail.color.hexa;
    });
</script>

P.s. thanks for the awesome work, this product is fantastic.


[Feature Request] Option for set default output to custom directory

I'm not sure if Commander has a way to load options from CLI arguments. It'll be nice if we can support options in both CLI and options file.


miss makezero in slice init

I was running github actions to run linter makezero for top github golang repos.
see issues https://github.com/alingse/go-linter-runner/issues/1
and the github actions output https://github.com/alingse/go-linter-runner/actions/runs/9243209781/job/25427055647
====================================================================================================
append to slice `indexes` with non-zero initialized length at https://github.com/SpectoLabs/hoverfly/blob/master/core/templating/templating.go#L160:14
====================================================================================================

the indexes := make([]JournalIndex, len(journal.Indexes)) should be indexes := make([]JournalIndex, 0, len(journal.Indexes))

@alingse  this is being fixed. Closing this issue.


Update chirp to 20170126 and rename to chirp-daily

according to the download page the versioning has moved to a daily build model (only on days there are changes) and versioning is now date based
renamed to chirp-daily as per naming rules based on new application name

If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

[X] brew cask audit --download {{cask_file}} is error-free.
[X] brew cask style --fix {{cask_file}} reports no offenses.
[X] The commit message includes the cask’s name and version.

Do renames fall under tap_migrations/renames.json?

I wasn't aware that HBC had one @adidalal ?

Hmm, could have sworn we got that "for free" when we merge with Homebrew. Does that ring a bell, @vitorgalvao?
ie, a formula_renames.json should be handled by https://github.com/Homebrew/brew/blob/aaf70eb69656ab9f64a70c8daea4581c270766a6/Library/Homebrew/tap.rb#L451
IDK, now that you bring it up, actually. Always assumed we had it but never bothered to use it. Figured it would be good to start so as to stop "breaking" installs on renames.

Oh dear - I have renamed several Casks over my short time here, and never thought about this.

Me too @miccal!
OK so I messed with it and can't get it to work.
Created a formula_renames.json file with just this:
{
  "chirp": "chirp-daily"
}

Reinstall:
> brew cask reinstall chirp
Error: undefined method `latest?' for nil:NilClass
Follow the instructions here:
  https://github.com/caskroom/homebrew-cask/blob/master/doc/reporting_bugs/pre_bug_report.md

If this doesn’t fix the problem, please report this bug:
  https://github.com/caskroom/homebrew-cask#reporting-bugs

/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/download.rb:36:in `clear_cache'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/download.rb:14:in `perform'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:101:in `download'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/installer.rb:62:in `fetch'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/reinstall.rb:15:in `block in install_casks'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/reinstall.rb:6:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/reinstall.rb:6:in `install_casks'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:10:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:111:in `run_command'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:150:in `process'
/usr/local/Homebrew/Library/Homebrew/cask/cmd/brew-cask.rb:8:in `<top (required)>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/usr/local/Homebrew/Library/Homebrew/brew.rb:24:in `require?'
/usr/local/Homebrew/Library/Homebrew/brew.rb:100:in `<main>'

Removed it and attempted to install by name:
> brew cask install chirp
Error: No available Cask for chirp. Did you mean one of:
caskroom/cask/chirp-daily
caskroom/cask/chirp-daily
Error: nothing to install

Do you want me to add a comment about this to #29301 or start a new issue rather than continue here?

Mind creating a new issue? (just reference this thread) & copy/paste should be fine to get the discussion rolling.
And @miccal not a problem - I think it's just something that's slipped off the radar but would be nice as the project matures

Done @adidalal


Osheenkhare/people picker

Added app-people-picker with Code / Screenshots / Docs and Readme.
Also added the header in Readme

@osheenkhare - There is an existing sample for People picker linked from the People Picker in Adaptive Cards documentation.  If there are additional scenarios covered then should we merge it to same sample?

Hi @Wajeed-msft, this sample actually covers a different use case including Task Module / Adaptive Card and Message Extension with people picker

Fixed Adaptive Card fields


Sample Does Not Run

I get the following error when trying to run the Fraud Detection sample:
Caused by: io.smallrye.config.ConfigValidationException: Configuration validation failed:
SRCFG00014: The config property quarkus.langchain4j.openai.api-key is required but it could not be found in any config source
at io.quarkiverse.langchain4j.openai.runtime.OpenAiRecorder.chatModel(OpenAiRecorder.java:25)

You need to supply an OpenAI API key.
For example:
quarkus.langchain4j.openai.api-key=mykey

Added the API Key now I get:
dev.ai4j.openai4j.OpenAiHttpException: {
"error": {
"message": "The model gpt-4-1106-preview does not exist or you do not have access to it. Learn more: https://help.openai.com/en/articles/7102672-how-can-i-access-gpt-4.",
"type": "invalid_request_error",
"param": null,
"code": "model_not_found"
}
}

We use GPT 3.5 turbo by default, so you don't absolutely need to set a model.
That said, it seems like you configured one that your accound does not have access to.

This is the entry in the Fraud Detection sample application.properties. I did not change. I guess I'll just comment it out...
quarkus.langchain4j.openai.chat-model.model-name=gpt-4-1106-preview

Interesting... It works for my account.
That said, I have no idea what policies OpenAI has in place for limiting models

It looks like I have to pay separate from my ChatGPT+ account to get the current models.

Ah, correct! The API access is a different AFAIk

After commenting out the model I now get:
dev.ai4j.openai4j.OpenAiHttpException: {
"error": {
"message": "You exceeded your current quota, please check your plan and billing details. For more information on this error, read the docs: https://platform.openai.com/docs/guides/error-codes/api-errors.",
"type": "insufficient_quota",
"param": null,
"code": "insufficient_quota"
}
}
I tried putting in couple of different CCs but seems to be a problem with payment service....


In Network DAO proposal list, should not exhibit proposal description

Description
In Network DAO proposal list, should not exhibit proposal description
Steps To Reproduce

Log in...
Enter Network DAO proposal list Referendum tab

Current Behavior

Each proposal card exhibit description

Expected Behavior

Should not exhibit description, just exhibit title

Environment

Platform: PC
Node: v18.18.0
Browser: Chrome 126.0.6478.56

fixed confirm


wit_bindgen_wasmtime does not generate the newest wit bindings at build

Consider the following WIT files
use { info } from info

hey: func(i: info) -> info

record info {
	infoa: string,
	infob: string,
}

The following program compiles and generate the correct bindings for info and hey
let (import, _i) =
    import::Import::instantiate(&mut store, &module, &mut linker, |ctx| &mut ctx.data)?;
let info = import.hey(
    &mut store,
    import::InfoParam {
        infoa: "a",
        infob: "b",
    },
)?;

If I add one more field to info record, it should fail to compile.
record info {
	infoa: string,
	infob: string,
        infoc: string,
}

However, when I run cargo build, I still get my program builds successfully.
    Finished dev [unoptimized + debuginfo] target(s) in 1.28s

Is this an expected behavior? This is not a big issue since I can make a build.rs file to watch for wit file changes and re-build my rust program.

I found that if I combine both WIT files into one file, then adding a new field to info record will trigger an error from compiler, saying that the Rust program is missing a field.
hey: func(i: info) -> info

record info {
	infoa: string,
	infob: string,
	infoc: string,
}

error[E0063]: missing field `infoc` in initializer of `InfoParam<'_>`
  --> src/main.rs:23:9
   |
23 |         import::InfoParam {
   |         ^^^^^^^^^^^^^^^^^ missing `infoc`

Can you check to see what code is generated in these cases using cargo expand?

I believe this bug is roughly here-ish where the source files you pass into the macro are tracked for rebuilds but transitively-depended-on-files are not included in the list of files depended on. That's why it works when the files are merged into one but doesn't work when the files are split.
The fix here would be to change parsing to output the list of files that it consumed. That list of files would then be used instead of only the original input file as the list of files in the proc-macro that are depended on. I think this is also an issue for the rust-to-wasm generator as well.

I can submit a PR to fix this if no one is working on it @alexcrichton

Sure thing, feel free to tackle!

For now use has been removed and I'll remember to ensure this is fixed when re-added to the Rust macro support.


Add MIME::MediaType

This PR adds a MIME::MediaType type with methods for parsing mime media types as per RFC 2045 and RFC 2046 including support for optional parameters.
This type is put to use for reading and writing HTTP Content-Type headers and HTTP::Multipart boundaries replacing existing incomplete implementations.
This is not directly integrated with the MIME registry (added in #5765) but can be optionally used together.

@RX14 I removed sort from #to_s without an different implementation for #inspect. This can be improved later, if necessary.

Squash please!


sap_swpm: cross repository dependency to sap_launchpad

sap_swpm role is currently expecting python3-pip (ansible.builtin.pip) to be present on target server (not execution node).
This package is not installed using preconfigure roles and dependency is not clearly stated.
Pip is also not present in base images for SLES or RHEL.
Issue does not appear if sap_launchpad download is executed because it is installed here https://github.com/sap-linuxlab/community.sap_launchpad/blob/85d6cb9215ede07aaad98036939ad4f0c2310757/playbooks/sample-download-install-media.yml#L13
Proposal in separate PR: Add python3-pip into sap_swpm role.
Affected line
https://github.com/sap-linuxlab/community.sap_install/blob/5b920752aa818fcd9fe96ce740bffaa314baf99e/roles/sap_swpm/tasks/swpm.yml#L35

@marcelmamula Thanks for your suggestion. I agree that we cannot assume that customers would have executed the sap_launchpad role before running sap_swpm. So implementing the installation of python3-pip in sap_swpm is advisable.
Note: I will soon spend some time on trying to remove the dependency on the community.general.pids Ansible module. Once successful, we likely do not need psutil (and python3-pip) any more. We can then remove the package installation of python3-pip again.

Agree with @marcelmamula regarding this oversight.
Agree with @berndfinger also, the community.general.pids Ansible Module usage was intended as a short-term fix to watch the sapinst process (using shell ps to watch was not stable enough and would result in early exit of the Ansible Task).


Tolerate doclint. http://blog.joda.org/2014/02/turning-off-doclint-in…

…-jdk-8-javadoc.html

Linux with security manager Build 1391 is now running using a merge of 5931bac5993f271093aa551e694525058c1fe26b

Windows Build 1563 is now running using a merge of 5931bac5993f271093aa551e694525058c1fe26b

Linux Build 6485 is now running using a merge of 5931bac5993f271093aa551e694525058c1fe26b

Linux with security manager Build 1391 outcome was SUCCESS using a merge of 5931bac5993f271093aa551e694525058c1fe26b
Summary: Tests passed: 803, ignored: 320 Build time: 0:04:56

It will be part of next jboss-parent https://github.com/jboss/jboss-parent-pom/commit/1868ef628417e82fde1e442176d5f66eaa57ecb2

Windows Build 1563 outcome was SUCCESS using a merge of 5931bac5993f271093aa551e694525058c1fe26b
Summary: Tests passed: 2845, ignored: 375 Build time: 0:47:25

Linux Build 6485 outcome was FAILURE using a merge of 5931bac5993f271093aa551e694525058c1fe26b
Summary: Tests failed: 1 (1 new), passed: 2843, ignored: 375, muted: 1 Build time: 0:49:07
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.clustering.cluster.web.GranularWebFailoverTestCase(SYNC-tcp).testGracefulUndeployFailover: <no details avaliable>


Revert traceback console width to 100.

Type of changes

[X] Bug fix
[ ] New feature
[ ] Documentation / docstrings
[ ] Tests
[ ] Other

Checklist

[X] I've run the latest black with default args on new code.
[ ] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate. <- Seems unnecessary for a change this small.
[ ] I've added tests for new code.
[X] I accept that @willmcgugan may be pedantic in the code review.

Description
Please describe your changes here. If this fixes a bug, please link to the issue, if possible.
#493 I fix this. Just one change.
Looking in the blame, seems this was previously called code_width and was set to 88. This updates the Traceback Class to match the same parameters as the install() function.

Good spot. Thanks.


redefine institutionCode

currently institutionCode is defined as: The name (or acronym) in use by the institution having custody of the object(s) or information referred to in the record. I think these are two separate things, which should be separated: institution name and institution code.
Institution name could be derived from a resolvable institution identifier, but as long as that is not required and for readability of the data it would be nice to have that as extra field. For an institution name there could be aliases, alternative names, names in different languages, but for DwC it would be enough to have the official name (in English, if an English name is available). the code should not be a full name and is only useful if a reference to an (international) authoritative source for the code is included too (like Index Herbariorum or GRID). to make clear that the code is a label and not an identifier (it is not resolvable), it could be renamed to institutionCodeLabel. It may also be good to mention that it should be an international code, as there may be national variants in local languages.

This is the second time this concept has arisen. See also https://github.com/tdwg/dwc/issues/64.
Need to demonstrate demand and provide a change request following the template in the Guidelines for contributing.

Is there work in NCD that can/should inform this proposal?

@debpaul @mswoodburn ?

this relates to https://github.com/tdwg/cd/issues/149 InstitutionAcronym, see also discussion there. I am actually now thinking we might need a distinction between InstitutionAcronym and InstitutionCode, where InstitutionAcronym is the acronym of a current institution, to be used as a short label for an institution in e.g. GUIs where there is not always room to display a full name. No historical version needs to be kept, if an institution changes, both its name and acronym may change. Ideally these acronyms should be given by an authority like CETAF or IH to ensure these are regionally or globally unique. InstitutionCode on the other hand is used by an institution to provide a globally unique identifier for collections or specimens, as part of a 'darwincore triplet'. It should be part of a collection description rather than part of an institution description. Not globally unique and historical versions need to be kept, as they keep being used when a collection moves to a new institute or the institute name changes. In many cases the label for InstitutionCode and InstitutionAcronym may be the same, so this distinction may be confusing to people.


Why aren't Condition and ConditionSet being generated in our documentation?

Both missing from https://mdf.readthedocs.io/en/latest/api/Specification.html.
That page is generated from https://github.com/ModECI/MDF/blob/main/docs/README.md, which is a markdown export of https://github.com/ModECI/MDF/blob/main/src/modeci_mdf/mdf.py, which has classes Condition and ConditionSet...

The condition and the conditionset are not being generated in the documentation because it is not documented in the readme file, even though it is addressed in the mdf.py directory, It is the readme.md file that is being fed into the readthedocs documentation so it is only issues that are addressed in the readme.md that will be mirrored in the readthedocs documentation

The Specification.html page is generated from https://github.com/ModECI/MDF/blob/main/docs/sphinx/source/api/Specification.rst not the markdown file as i stated earlier

The above is a workaround, but needs cleaner / standard implementation

The above is a workaround, but needs cleaner / standard implementation
This has been solved here:
https://github.com/ModECI/MDF/actions/runs/4126125824

The remaining issue is that the docs for ConditionSet etc. should be automatically generated from the first line here:
https://github.com/ModECI/MDF/blob/c61fc54eb045d68238418cf5fbc18196512e136a/docs/generate.py#L44-L56
so doc_md_1 shouldn't be required. The way the markdown is generated from mod doesn't "connect" all the way down to ConditionSet as it should (and as it does for Node, Edge etc.)


[FEATURE] - infer-schema denied special caracters for XML

Infer-schema for XML file raise an error for invalid field name when "-" or ":" are found in attributes.
Exemple:

Exception in thread "main" java.lang.Exception: Invalid field name ->certificat-environnemental<-. Only letters, digits and '' are allowed. Other characters including spaces are forbidden.
Is it possible to add a feature to deal with that kind of file or do i have to transform my file before using starlake ?

Thanks @p1be for submitting this issue
As you probably know, data warehouses and parquet like file formats do not support '-' or ':' in their column names.
How would you like these chars to be translated into the target database.
Would it be fine if certificat-environnemental is translated to certificat_environnemental ?

Regarding the ':' char - aka namespace delimiter - we are now ignore the namespace when inferring the column name

@p1be Just published the enhancement you requested here in the snapshot branch.
Met me know if it solves your issue


Out-of-the-box syntax highlighting is broken for at least Java, PHP, & Ruby.

🐛 Bug Report
Out-of-the-box syntax highlighting is broken for at least Java, PHP, & Ruby.
There may be others...
Have you read the Contributing Guidelines on issues?
(Write your answer here.)
Yes.
To Reproduce
If possible, use https://new.docusaurus.io to create a CodeSandbox reproducible demo of the bug.
Sandbox didn't work. Received a docusaurus not found error when I tried to save changes, even after logging in.
(Write your steps here:)

Do a fresh install of Docusuarus using the classic theme, i.e.:

npx @docusaurus/init@latest init my-website classic

Add java, php, or ruby code to a Markdown page, such as:

Java:
import io.temporal.workflow.Workflow;
import io.temporal.workflow.WorkflowInterface;
import io.temporal.workflow.WorkflowMethod;
import org.slf4j.Logger;

public class GettingStarted {

    private static Logger logger = Workflow.getLogger(GettingStarted.class);

    @WorkflowInterface
    public interface HelloWorld {
        @WorkflowMethod
        void sayHello(String name);
    }

    public static class HelloWorldImpl implements HelloWorld {

        @Override
        public void sayHello(String name) {
            logger.info("Hello " + name + "!");
        }
    }
}

PHP:
#[ActivityInterface("file_activities.")]
interface FileProcessingActivities
{
    public function upload(string $bucketName, string $localName, string $targetName);

    #[ActivityMethod("transcode_file")]
    public function download(string $bucketName, string $remoteName);

    public function processFile(): string;

    public function deleteLocalFile(string $fileName);
}

Ruby:
class AsyncActivity < Temporal::Activity
  timeouts start_to_close: 120

  def execute
    logger.warn "run `bin/activity complete #{activity.async_token}` to complete activity"

    activity.async
  end
end

Run a local server to view the site.

Expected behavior
Code examples specified with a code type should have code specific syntax highlighting for popular languages like PHP, Ruby, and Java.
Actual Behavior
There is NO syntax highlighting for some langauges.
It works for some like Go and Javascript. But not all.
See screen shots:
Source:

Built site:

Your Environment

Public source code:
This is from a fresh install of YOUR source code.
But we use it in our docs site here:
https://github.com/temporalio/documentation

Public site url:
I won't upgrade my public site until this is fixed.
But here ya go: https://docs.temporal.io/

Docusaurus version used:
In depth testing was done using:
@docusaurus/core v2.0.0-alpha.72
@docusaurus/preset-classic v2.0.0-alpha.72

But some testing revealed issues starting with 2.0.0-alpha.68
I am using 2.0.0-alpha.66 which works find

Environment name and version (e.g. Chrome 78.0.3904.108, Node.js 10.17.0):
Node v15.2.1
Chrome Version 89.0.4389.90 (but tested across other browsers Safari & Firefox and same)
Operating system and version (desktop or mobile):
Mac OS Big Sur v11.2.3

Reproducible Demo
If possible, use https://new.docusaurus.io to create a CodeSandbox reproducible demo of the bug.
That did not work for me -
Just do a fresh install and see for yourself - takes 1 min.

According to the docusaurus doc,

By default, Docusaurus comes with this subset of commonly used languages.

Which doesn't include Java, PHP, or Ruby. You have to manually include them in the config.
I have encountered the same thing, and I wonder why they didn't choose these languages to support out-of-the-box...

Ah, yes just figuring this out.
The thing is - those languages used to be included by default.
I was confused because we upgraded our site, and discovered that some of the languages stopped working...
Appreciate it @Josh-Cena Thank you!


Clarify what is and is not verified by AnonCreds and what is the responsibility of the calling framework and application

Discussed on the AnonCreds Specification Working Group Call, May 8, 2023.
We need to be clear what is verified by AnonCreds and what is the responsibility of the caller to verify when using AnonCreds. For example, the following needs to be included in the AnonCreds specification.

In general, the caller must confirm that the presentation meets the business requirements of the verifier when the cryptographic verification done by AnonCreds is successful (e.g., verified=true may not be sufficient)
Encodings of revealed attributes must be checked by the caller.
The trustworthiness of credential issuers is the responsibility of the caller.

For example, if the restrictions on a referent is by schema, any issuer could issue a credential to the holder.
A trust registry may be useful for that purpose.

(To be confirmed) It is the responsibility of the verifier to make sure that all referents are included in the presentation.

AnonCreds will verify the cryptography of all included referents (presentations derived from source credentials), but not that all referents from the presentation request are included.
This can be feature in some cases, such as when the verifier requests multiple referents, but is willing to accept a presentation with only some of the referents. The AnonCreds presentation request format does not have a way for the verifier to convey that information to the holder.

The caller must decided if it is acceptable for Holders to leave some requested attributes unrevealed.
A caller may decide it is acceptable for a given business purpose for a presentation to be derived from a revoked credential.

This can be expressed by a verifier not including a revocation interval in the presentation request.

The business-purpose validity of a presentation MAY be carried out by the library/component invoking AnonCreds. For example, an Aries Framework may add some additional checks not covered by AnonCreds, such as verifying the encodings and ensuring all referents from the presentation request are included. However, there are some checks that are use case specific and can only be done by the calling application.

This has been clarified in the spec.  Adding to the checklist for the final review.


Create GitBook

Context
As user I want to have a documentation of the product so I can inform myself about all relevant topics.
Tasks
eg like: https://docs.astar.network/

write techincal documentation for modules so we can start onboarding with Acala
write how it works....
tokenomics etc.

Additional Information

the gitbook is already created.
shall we have subtasks for documentation?

https://github.com/gamedaoco/documentation

https://docs.gamedao.co/


NTRs: premature acrosome loss and regulation terms of it

Hi,
I would l like to ask for some new terms as discussed at the resent Signalling Workshop.

NTR: Regulation of premature acrosome loss

Synonym : Regulation of spontaneous acrosome loss
Suggested definition: Any process that modulates the frequency, rate or extent of premature acrosome loss.
Note:  If the release of the acrosome content occurs at the zona pellucida, consider using  GO:0060046 regulation of acrosome reaction.
It could be a child of regulation of fertilization.

In particular, the negative regulation of this term is the one that is more physiologically relevant.
NTR: Negative regulation of premature acrosome loss
Synonym : Negative regulation of spontaneous acrosome loss

Suggested definition: Any process that stops, prevents or reduces the frequency, rate or extent of premature acrosome loss.
Note: That occurs before the sperm reaches the zona pellucida.
That could be a child of "positive regulation of fertilization".
It might sounds a bit oxymoron but, physiologically, this is what is does. It prevents the premature acrosome loss, thus allows the reaction to happen at the right time and place, so positively regulates the fertilization process.
Question: Should that be specified in the definition to facilitate understanding and correct usage of the term?
Supporting publications for the existence of such regulators:

PMID:22228629; SPINK3

PMID:23430248; Spink13 (this publication has been used for creating  GO:1902225 negative regulation of acrosome reaction. But it's clearly referring to the premature acrosome loss. I would suggest obsoleting this term. I will create a different ticket for this, as there are some annotations that will need to be updated to the new term after it's created.)

I suppose the parent term should also be added to aid the building of the tree in the ontology.

NTR: premature acrosome loss
Synonym : Spontaneous acrosome loss
Suggested definition: The discharge, by sperm, of a single, anterior secretory granule before the sperm reaches to the zona pellucida of the oocyte. The process begins with the fusion of the outer acrosomal membrane with the sperm plasma membrane and ends with the exocytosis of the acrosomal contents.
Note:  If the release of the acrosome content occurs at the zona pellucida, consider using GO:0007340 acrosome reaction.
It could be a sibling of "acrosome reaction".
Thanks,
Penelope

Hi @pgarmari
I am still torn about creating the term 'premature acrosome loss' because I'm still not convinced it is a normal process. Can you remind me if there is a physiological relevance to the spontaneous release. If there isn't, maybe just a regulation term can be created. The term would not be a regulation of acrosome reaction, but would be a negative regulation of exocytosis.

Hi @ukemi ,
The 'premature acrosome loss' has been reported to facilitate the dispersal of the cumulus mass. It can be initiated by particular sperm factors or by progesterone released by the cumulus cells. Although there is still some discussion about this, most authors consider it normal.

There was a recent review about this..

"it is clear that the ZP may not be the only site of AR and that the cumulus mass may also be a physiological site of AR. We thus propose that some sperm initiate their AR when contacting with the cumulus cells, while some of them remain acrosome intact until they reach the ZP and bind to ZP3.
They say that "AR initiated at the site of the cumulus is for meaningful purposes that we are not entirely sure at the moment. Obviously, these acrosome reacted sperm may disperse the cumulus mass for other sperm population and it is the sacrifice of the acrosome-reacted sperm at the site of cumulus that paves the way for other sperm to follow through without undergoing AR until they reach ZP."
They attribute this to "..a sperm-released factor, NYD-SP8, that initiates AR upon sperm interaction with cumulus cells."
https://www.ncbi.nlm.nih.gov/pubmed/19153666
https://www.ncbi.nlm.nih.gov/pubmed/21380641

Other authors found that AR is induced by progesterone released by the cumulus cells.

https://www.ncbi.nlm.nih.gov/pubmed/26655718
In all case, when the AR  occurs, it allows the release of several types of sperm enzymes. "The release of these enzymes leads to a partial hydrolysis of the layers surrounding the oocyte, allowing the sperm to reach the oolema where the fusion between the two gametes occurs."


Update getting started guide

Changes include:

Whitespace fixes
Move the curly brackets for the tuple closer

I didn't see the curly brackets initially

Make it extra clear that we should use Desktop.Endpoint instead of use Phoenix.Endpoint
Various wording changes
Give some guidance about how to run your application at the end of setup and make it clear that phx.server should still be needed

Note: if you don't run phx.server (or set your endpoint configuration to server: true) you will get an error like:
[error] ** wx object server {:local, HappyHelperWindow} terminating
** Last message in was {:"$gen_cast", {:show, &HappyHelperWeb.Endpoint.url/0}}
** When Server state == %Desktop.Window{
  module: nil,
  taskbar: nil,
  frame: {:wx_ref, 35, :wxFrame, []},
  notifications: %{},
  webview: {:wx_ref, 53, :wxWebView, []},
  home_url: &HappyHelperWeb.Endpoint.url/0,
  last_url: nil,
  title: "Elixir.HappyHelperWindow",
  rebuild: 0,
  rebuild_timer: nil
}
** Reason for termination ==
** {:badarg,
 [
   {:erlang, :binary_to_existing_atom,
    ["Elixir.HappyHelperWeb.Endpoint.HTTP", :utf8],
    [error_info: %{module: :erl_erts_errors}]},
   {:elixir_aliases, :safe_concat, 1,
    [file: ~c"src/elixir_aliases.erl", line: 139]},
   {HappyHelperWeb.Endpoint, :url, 0,
    [file: ~c"lib/happy_helper_web/endpoint.ex", line: 3]},
   {Desktop.Window, :prepare_url, 1,
    [file: ~c"lib/desktop/window.ex", line: 668]},
   {Desktop.Window, :handle_cast, 2,
    [file: ~c"lib/desktop/window.ex", line: 606]},
   {:wx_object, :handle_msg, 5, [file: ~c"wx_object.erl", line: 493]},
   {:proc_lib, :init_p_do_apply, 3, [file: ~c"proc_lib.erl", line: 241]}
 ]}

Which isn't that easy to track down (probably took me ~30m of semi-distracted time)

Hmm, actually I see that the code snippet in the guide does have server: true, although that isn't reflected in the surrounding text. I've now mentioned server: true in the guide text.

Thanks for that !


[fastlane] Verify of rest-client dependency for sentry

Fixes https://github.com/fastlane/fastlane/issues/4848

👍