[Zeppelin-2676] recent notes list

What is this PR for?
Adding recent notes feature: every user now can see his recent notes on homepage. Also you can add recent list on your custom home page.
What type of PR is it?
Feature
Todos
What is the Jira issue?
https://issues.apache.org/jira/projects/ZEPPELIN/issues/ZEPPELIN-2676
How should this be tested?
Go to homepage. Open some note. Return to homepage. This note has appeared in recent list.
If you open note, which is already in recent list, this note move to the beginning of the recent list.
Screenshots (if appropriate)

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? Yes (for add recent list to custom homepage) (done)

I have a suggestion.

Place Recent before Notebook (more upper side)
Limit the maximum number of recent notebooks.

That's because users are accessing recent notebook more and
if there are lots of notebooks, the recent menu will not be seen without scrolling

@1ambda
Number of recent is limited: MAX_RECENT_SIZE = 10 (https://github.com/apache/zeppelin/pull/2449/files#diff-289945053c8adc894aa4838564528ba5)
Place Recent before Notebook: good idea, done

Ready to review

Ready to review

Ready to review

I think we have a comment thread on this PR
https://github.com/apache/zeppelin/pull/2449#discussion_r127705438

Ready to review

@1ambda @zjffdu
Thank you for your comments. Classes have been renamed to match its functionality

Ready to review (red for other tests )

Ready to review

LGTM!

Ready to review

@Leemoonsoo, @felixcheung, @zjffdu I think it's good to go.
Any concerns about backend code like persistency, ...?

Ready to review

Ready to review

Ready to review

Ready to review

Ready to review

Ready to review

Need rework.


POST command fails with updates from PR 54

PR https://github.com/IBM-Swift/Kitura-net/pull/54 added a "Connection: close" header, which cases certain POST commands using HTTP.request() to fail.
Here is the verbose output from CURL when used with HTTP.request().
*   Trying <IP_ADDRESS>...
* Connected to a.example.com (<IP_ADDRESS>) port 443 (#0)
* TLS 1.2 connection using TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256
* Server certificate: *.example.com
> POST /exampleapi HTTP/1.1
Host: a.example.com
Accept: */*
Content-Type: application/json
Connection: close
Content-Length: 39624
Expect: 100-continue

* Done waiting for 100-continue
* We are completely uploaded and fine
< HTTP/1.1 200 OK
< Content-Type: application/json; charset=UTF-8
< Vary: X-Origin
< Vary: Referer
< Date: Thu, 11 Aug 2016 21:14:20 GMT
< Server: ESF
...
< Accept-Ranges: none
< Vary: Origin,Accept-Encoding
< Connection: close
<
* SSLRead() return error -9806
* Closing connection 0

Commenting out line 290 of ClientRequest.swift allows HTTP POST requests to work.
//headers["Connection"] = "close"
To make the KituraNet HTTP class more general purpose, adding the "Connection: close" header should be optional.

Interesting, I'm having the exact same problem and I posted an issue at the exact same time. See #69

@dsperling  It does look like ["Connection"] = "close" is causing the problem, but I'm also worried about the consequences of omitting it. When it's not included, the request succeeds but this is logged at the very end:
* Connection #0 to host a.example.com left intact

Honestly I'm not 100% sure what this means, but it sounds like maybe the connection is being left open? I can't say for sure, but I imagine there was a reason somebody added that header in the first place.

It was added in a PR for the "keep alive implementation", but not sure what it does yet.
Curl, which is the underlying library, does leave the connection open.  However, when you call curl_easy_cleanup() (which is called in deinit()) finally closes the connection.
https://curl.haxx.se/libcurl/c/curl_easy_cleanup.html

Maybe @shmuelk can tell us more about the close header.
PS, you accidentally typed 45 in the title instead of 54.

Without Connection: Close, the connection is suppose to get held open by the server. When that happened, against Kitura, we failed to get output. As HTTP.request is always a single request, I saw no need not send Connection: close.

@shmuelk Hmm, I wonder why it's causing issues with SSL on certain sites. In my case the issue occurs when accessing the Google Maps API (see #69). Is it possible that it's not following a redirect properly, or something like that?

@collinhundley interesting that you see the issue with Google Maps.  I see the exact POST problem with a different Google service.
@shmuelk Do you have a test case where things were failing without "Connection: Close"?  Was it a GET?  Maybe we could figure out a solution without having to close the connection after each transaction.  At a minimum we should add a property to control this by verb or a Bool.
It would be great to allow a single KituraNet HTTP instance to transact multiple times with the same host on the same socket.  A big savings in socket setup time and TLS negotiation.

I should point out that my problem is actually with a GET to the Google API, so the issue isn't isolated to POST requests.

@collinhundley Interesting that GET fails as well.  I have confirmed the bug on Linux so we can eliminate Apple's App Transport Security (ATS) as a culprit.

Unfortunately all that I remember right now, is that things were failing when Kitura tried to keep the socket open and the client didn't get any output. I'm pretty sure it was one of the Kitura-net or Kitura Unit tests that was failing, i.e. timing out waiting for output.
@dsperling Says in his PR that he has removed the addition of the connection: Close header and the Kitura-net and Kitura Unit tests are still passing. I don't know what has changed, but I'm happy to merge the PR (after a minor change mentioned on the PR).

@shmuelk Fixed the typo. I double checked the unit tests on Kitura-net and Kitura on Linux and Mac.  All looks good.
If the need for the "Connection: close" has disappeared, the other option to fix this is to simply remove that one line that adds the header.  That would simplify the end() function which would no longer need the optional close Boolean.  Users that want "Connection: close" behavior can always add the header themselves.
If you want to revert to the simpler end() functions, let me know and I can modify the PR.


pkg/operator/staticpod/controller/installerstate: Include reason in InstallerPodContainerWaitingDegraded

To make messages like:
level=error msg=Cluster operator kube-scheduler Degraded is True with InstallerPodContainerWaiting_ContainerCreating::NodeController_MasterNodesReady::StaticPods_Error: NodeControllerDegraded: The master nodes not ready...
level=error msg=InstallerPodContainerWaitingDegraded: Pod "installer-7-ci-op-kqngdf8m-b58e2-8x4c5-master-2" on node "ci-op-kqngdf8m-b58e2-8x4c5-master-2" container "installer" is waiting for 10m9.799978979s because ""

slightly more helpful when processing data like:
$ curl -s https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/logs/release-openshift-ocp-installer-e2e-azure-4.8/1374791880151142400/artifacts/e2e-azure/pods.json | jq -r '.items[] | select(.metadata.name == "installer-7-ci-op-kqngdf8m-b58e2-8x4c5-master-2").status.containerStatuses[] | select(.name == "installer").state.waiting'
{
  "reason": "ContainerCreating"
}

The ContainerCreating condition was already available via InstallerPodContainerWaiting_ContainerCreating, but when all we have to go on is the reason, repeating it in the message saves folks the effort of finding and parsing it out of the aggregate reason.

/lgtm

/approve
Follow-up is fine.


Fix deadlock in status reporting and add retry on failure

Fixes #869 by grabbing the neutron db-access lock and retrying on timer.

ML2 driver has moved to the networking-calico project, this will need to be rebased onto that.

retest this please

retest this please

retest this please

Another chef timeout:
retest this please

retest this please

retest this please
(This will be the first PR build since moving the mechanism driver, so if there are problems I'd like to know.)

Also @fasaxc: you've got conflicts.

This pull request needs to move to gerrit (against networking-calico).

The mechanism driver part of this has been moved to gerrit: https://review.openstack.org/#/c/249897/

Abandoning this approach and moving code to networking-calico...


How to gradually decrease the opacity ?

i already done this simple script
Hotspot.hotspots.forEach(
        (hotspot) => {
            let fadingInterval = setInterval(() => {
                hotspot.setupState({
                    state: "idle",
                    attributes: {
                        backgroundOpacity: hotspot.states.idle.attributes.backgroundOpacity - .01,
                    }
                })
                console.log(hotspot.states.idle.attributes.backgroundOpacity)

                if (hotspot.states.idle.attributes.backgroundOpacity < 0) {
                    clearInterval(fadingInterval)
                    scene.remove(hotspot)
                }

            }, 10)
        }
    )

hotspot here is like the button on example.
i have tried to directly change the backgroundOpacity but the opacity wouldn't change.
is there any other method to work around this ?

Hi,
Here is a fiddle showing how to tween the background opacity : https://jsfiddle.net/felixmariotto/j6a2xe3L/3/
As explained in the doc, setupState and setState, are used to record a component states and switch between them with less code. It's not really appropriate to tween a value.


Enable LeaderElect for both scheduler and controller

fix https://github.com/kubernetes-incubator/kube-arbitrator/issues/299

thanks for your contribution, merged it :)


azureml-core Installation error

azureml-core:
1.46.0:
Ubuntu:
3.8.10:

Describe the bug
When attempting to install azureml-core on the Linux System, the installation process fails with the message on the command prompt:
Traceback (most recent call last):
File "/usr/bin/pip", line 11, in <module>
  load_entry_point('pip==20.0.2', 'console_scripts', 'pip')()
File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 490, in load_entry_point
  return get_distribution(dist).load_entry_point(group, name)
File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 2854, in load_entry_point
  return ep.load()
File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 2445, in load
  return self.resolve()
File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 2451, in resolve
  module = __import__(self.module_name, fromlist=['__name__'], level=0)
File "/usr/lib/python3/dist-packages/pip/_internal/cli/main.py", line 10, in <module>
  from pip._internal.cli.autocompletion import autocomplete
File "/usr/lib/python3/dist-packages/pip/_internal/cli/autocompletion.py", line 9, in <module>
  from pip._internal.cli.main_parser import create_main_parser
File "/usr/lib/python3/dist-packages/pip/_internal/cli/main_parser.py", line 7, in <module>
  from pip._internal.cli import cmdoptions
File "/usr/lib/python3/dist-packages/pip/_internal/cli/cmdoptions.py", line 24, in <module>
  from pip._internal.exceptions import CommandError
File "/usr/lib/python3/dist-packages/pip/_internal/exceptions.py", line 10, in <module>
  from pip._vendor.six import iteritems
File "/usr/lib/python3/dist-packages/pip/_vendor/__init__.py", line 65, in <module>
  vendored("cachecontrol")
File "/usr/lib/python3/dist-packages/pip/_vendor/__init__.py", line 36, in vendored
  __import__(modulename, globals(), locals(), level=0)
File "<frozen importlib._bootstrap>", line 991, in _find_and_load
File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
File "<frozen importlib._bootstrap>", line 655, in _load_unlocked
File "<frozen importlib._bootstrap>", line 618, in _load_backward_compatible
File "<frozen zipimport>", line 259, in load_module
File "/usr/share/python-wheels/CacheControl-0.12.6-py2.py3-none-any.whl/cachecontrol/__init__.py", line 9, in <module>
File "<frozen importlib._bootstrap>", line 991, in _find_and_load
File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
File "<frozen importlib._bootstrap>", line 655, in _load_unlocked
File "<frozen importlib._bootstrap>", line 618, in _load_backward_compatible
File "<frozen zipimport>", line 259, in load_module
File "/usr/share/python-wheels/CacheControl-0.12.6-py2.py3-none-any.whl/cachecontrol/wrapper.py", line 1, in <module>
File "<frozen importlib._bootstrap>", line 991, in _find_and_load
File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
File "<frozen importlib._bootstrap>", line 655, in _load_unlocked
File "<frozen importlib._bootstrap>", line 618, in _load_backward_compatible
File "<frozen zipimport>", line 259, in load_module
File "/usr/share/python-wheels/CacheControl-0.12.6-py2.py3-none-any.whl/cachecontrol/adapter.py", line 5, in <module>
File "<frozen importlib._bootstrap>", line 991, in _find_and_load
File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
File "<frozen importlib._bootstrap>", line 655, in _load_unlocked
File "<frozen importlib._bootstrap>", line 618, in _load_backward_compatible
File "<frozen zipimport>", line 259, in load_module
File "/usr/share/python-wheels/requests-2.22.0-py2.py3-none-any.whl/requests/__init__.py", line 95, in <module>
File "<frozen importlib._bootstrap>", line 991, in _find_and_load
File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
File "<frozen importlib._bootstrap>", line 655, in _load_unlocked
File "<frozen importlib._bootstrap>", line 618, in _load_backward_compatible
File "<frozen zipimport>", line 259, in load_module
File "/usr/share/python-wheels/urllib3-1.25.8-py2.py3-none-any.whl/urllib3/contrib/pyopenssl.py", line 46, in <module>
File "/home/eric/.local/lib/python3.8/site-packages/OpenSSL/__init__.py", line 8, in <module>
  from OpenSSL import SSL, crypto
File "/home/eric/.local/lib/python3.8/site-packages/OpenSSL/SSL.py", line 19, in <module>
  from OpenSSL.crypto import (
File "/home/eric/.local/lib/python3.8/site-packages/OpenSSL/crypto.py", line 3224, in <module>
  utils.deprecated(
TypeError: deprecated() got an unexpected keyword argument 'name'
Sys error. excepthook:
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/apport_python_hook.py", line 72, in apport_excepthook
  from apport.fileutils import likely_packaged, get_recent_crashes
File "/usr/lib/python3/dist-packages/apport/__init__.py", line 5, in <module>
  from apport.report import Report
File "/usr/lib/python3/dist-packages/apport/report.py", line 32, in <module>
  import apport.fileutils
File "/usr/lib/python3/dist-packages/apport/fileutils.py", line 12, in <module>
  import os, glob, subprocess, os. path, time, pwd, sys, requests_unixsocket
File "/usr/lib/python3/dist-packages/requests_unixsocket/__init__.py", line 1, in <module>
  import requests
File "<frozen importlib._bootstrap>", line 991, in _find_and_load
File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
File "<frozen importlib._bootstrap>", line 655, in _load_unlocked
File "<frozen importlib._bootstrap>", line 618, in _load_backward_compatible
File "<frozen zipimport>", line 259, in load_module
File "/usr/share/python-wheels/requests-2.22.0-py2.py3-none-any.whl/requests/__init__.py", line 95, in <module>
File "<frozen importlib._bootstrap>", line 991, in _find_and_load
File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
File "<frozen importlib._bootstrap>", line 655, in _load_unlocked
File "<frozen importlib._bootstrap>", line 618, in _load_backward_compatible
File "<frozen zipimport>", line 259, in load_module
File "/usr/share/python-wheels/urllib3-1.25.8-py2.py3-none-any.whl/urllib3/contrib/pyopenssl.py", line 46, in <module>
File "/home/eric/.local/lib/python3.8/site-packages/OpenSSL/__init__.py", line 8, in <module>
  from OpenSSL import SSL, crypto
File "/home/eric/.local/lib/python3.8/site-packages/OpenSSL/SSL.py", line 19, in <module>
  from OpenSSL.crypto import (
File "/home/eric/.local/lib/python3.8/site-packages/OpenSSL/crypto.py", line 3224, in <module>
  utils.deprecated(
TypeError: deprecated() got an unexpected keyword argument 'name'

The original exception was:
Traceback (most recent call last):
File "/usr/bin/pip", line 11, in <module>
  load_entry_point('pip==20.0.2', 'console_scripts', 'pip')()
File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 490, in load_entry_point
  return get_distribution(dist).load_entry_point(group, name)
File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 2854, in load_entry_point
  return ep.load()
File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 2445, in load
  return self.resolve()
File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 2451, in resolve
  module = __import__(self.module_name, fromlist=['__name__'], level=0)
File "/usr/lib/python3/dist-packages/pip/_internal/cli/main.py", line 10, in <module>
  from pip._internal.cli.autocompletion import autocomplete
File "/usr/lib/python3/dist-packages/pip/_internal/cli/autocompletion.py", line 9, in <module>
  from pip._internal.cli.main_parser import create_main_parser
File "/usr/lib/python3/dist-packages/pip/_internal/cli/main_parser.py", line 7, in <module>
  from pip._internal.cli import cmdoptions
File "/usr/lib/python3/dist-packages/pip/_internal/cli/cmdoptions.py", line 24, in <module>
  from pip._internal.exceptions import CommandError
File "/usr/lib/python3/dist-packages/pip/_internal/exceptions.py", line 10, in <module>
  from pip._vendor.six import iteritems
File "/usr/lib/python3/dist-packages/pip/_vendor/__init__.py", line 65, in <module>
  vendored("cachecontrol")
File "/usr/lib/python3/dist-packages/pip/_vendor/__init__.py", line 36, in vendored
  __import__(modulename, globals(), locals(), level=0)
File "<frozen importlib._bootstrap>", line 991, in _find_and_load
File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
File "<frozen importlib._bootstrap>", line 655, in _load_unlocked
File "<frozen importlib._bootstrap>", line 618, in _load_backward_compatible
File "<frozen zipimport>", line 259, in load_module
File "/usr/share/python-wheels/CacheControl-0.12.6-py2.py3-none-any.whl/cachecontrol/__init__.py", line 9, in <module>
File "<frozen importlib._bootstrap>", line 991, in _find_and_load
File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
File "<frozen importlib._bootstrap>", line 655, in _load_unlocked
File "<frozen importlib._bootstrap>", line 618, in _load_backward_compatible
File "<frozen zipimport>", line 259, in load_module
File "/usr/share/python-wheels/CacheControl-0.12.6-py2.py3-none-any.whl/cachecontrol/wrapper.py", line 1, in <module>
File "<frozen importlib._bootstrap>", line 991, in _find_and_load
File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
File "<frozen importlib._bootstrap>", line 655, in _load_unlocked
File "<frozen importlib._bootstrap>", line 618, in _load_backward_compatible
File "<frozen zipimport>", line 259, in load_module
File "/usr/share/python-wheels/CacheControl-0.12.6-py2.py3-none-any.whl/cachecontrol/adapter.py", line 5, in <module>
File "<frozen importlib._bootstrap>", line 991, in _find_and_load
File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
File "<frozen importlib._bootstrap>", line 655, in _load_unlocked
File "<frozen importlib._bootstrap>", line 618, in _load_backward_compatible
File "<frozen zipimport>", line 259, in load_module
File "/usr/share/python-wheels/requests-2.22.0-py2.py3-none-any.whl/requests/__init__.py", line 95, in <module>
File "<frozen importlib._bootstrap>", line 991, in _find_and_load
File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
File "<frozen importlib._bootstrap>", line 655, in _load_unlocked
File "<frozen importlib._bootstrap>", line 618, in _load_backward_compatible
File "<frozen zipimport>", line 259, in load_module
File "/usr/share/python-wheels/urllib3-1.25.8-py2.py3-none-any.whl/urllib3/contrib/pyopenssl.py", line 46, in <module>
File "/home/eric/.local/lib/python3.8/site-packages/OpenSSL/__init__.py", line 8, in <module>
  from OpenSSL import SSL, crypto
File "/home/eric/.local/lib/python3.8/site-packages/OpenSSL/SSL.py", line 19, in <module>
  from OpenSSL.crypto import (
File "/home/eric/.local/lib/python3.8/site-packages/OpenSSL/crypto.py", line 3224, in <module>
  utils.deprecated(
TypeError: deprecated() got an unexpected keyword argument 'name'

To Reproduce
Steps to reproduce the behaviour:

Enter "pip install azureml-core" onto the command prompt to install azureml-core

Expected behaviour
The expectation would be a successful installation of the azureml-core library.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Storage:0.56320536,App Services:0.06313863,Azure.Core:0.04440899'

@hlin863 thank you for reaching out. @bandsina could you please take a look and advise?
cc/ @azureml-github

@tjprescott, thank you for taking the time to reply and redirecting the problem.
The error is occurring on my repository link:
https://github.com/hlin863/Final_Year_Project/issues/1
Would you like to check "pipeline.py" for any possible issues?

This seems like an issue with pyOpenSSL package. Could you please try uninstalling the package: pyOpenSSL from the environment?


Update README.md example

Due to issue https://github.com/influxdb/influxdb/issues/3467, the field key named user makes InfluxDB return an error. This change prevents users from running into the bug, which will likely be fixed in 0.9.3.

Good catch, fixed that already in the Tests, but forgot to adjust the Readme, thanks


Detailed command to run RO with PreRes-18

I used RO public ckpt cifar10_l2_eps128.pth and train preResnet18SSL model with env CUDA_VISIBLE_DEVICES=0,1,2,3 python3 cifar10_defense.py --md_path=checkpoint/cifar10_l2_eps128.pth    --batch-size=1024 --contrastive_bs=1024 --epochs=100 --res18, but I failed to get similar results shown in the paper (28.45 vs 30.99 in my case, 52.40 vs 54.59 in paper).
Is there any hyperparameter I need to change here? Thx!

I believe you use the wrong normalization. For input image's normalization
function, do you subtract mean and divided by var? Undo yours should give
you the right results.
On Sun, Apr 24, 2022 at 6:44 PM Bowen Chen @.***> wrote:

I used RO public ckpt cifar10_l2_eps128.pth
https://drive.google.com/drive/folders/110JHo_yH9zwIf1b12jKoG6dRonrow9eA
and train preResnet18SSL model with env CUDA_VISIBLE_DEVICES=0,1,2,3
python3 cifar10_defense.py --md_path=checkpoint/cifar10_l2_eps128.pth
--batch-size=1024 --contrastive_bs=1024 --epochs=100 --res18, but I
failed to get similar results shown in the paper (28.45 vs 30.99 in my
case, 52.40 vs 54.59 in paper).
Is there any hyperparameter I need to change here? Thx!
—
Reply to this email directly, view it on GitHub
https://github.com/cvlab-columbia/SelfSupDefense/issues/3, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEUBBS2H4V5Q6GVA73OSZGLVGXFFDANCNFSM5UG4LTAA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I used the normalization
def normalize(X):
    return transforms.functional.normalize(X, (0.4914, 0.4822, 0.4465), (0.2023, 0.1994, 0.2010))

from which I can get correct acc without attack from RO's ckpt I downloaded. Or did you mean I should retrain RO by without normalization? Thx!

One of them should work. 28.45 does not reproduce RO's results.So you
should use the other normalization.
On Sun, Apr 24, 2022 at 9:27 PM Bowen Chen @.***> wrote:

I used the normalization
def normalize(X):
return transforms.functional.normalize(X, (0.4914, 0.4822, 0.4465), (0.2023, 0.1994, 0.2010))
from which I can get correct acc without attack from RO's ckpt I
downloaded. Or did you mean I should retrain RO by without normalization?
Thx!
—
Reply to this email directly, view it on GitHub
https://github.com/cvlab-columbia/SelfSupDefense/issues/3#issuecomment-1107968755,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEUBBS4FMT6CCDVFM2EVOMDVGXYHVANCNFSM5UG4LTAA
.
You are receiving this because you commented.Message ID:
@.***>

Thx for the reply. But I am still confused, which normalization should I use? I just checked their cifar training code, it seems that they are using (X - mu)/std.

Yes, I also checked my code and it uses (X - mu)/std. It seems your low acc
is due to not reproducing the robustoverfit model in the first place. Try
run their code and see if you can reproduce it, then apply our method on
top.
On Sun, Apr 24, 2022 at 9:33 PM Bowen Chen @.***> wrote:

Thx for the reply. But I am still confused, which normalization should I
use? I just checked their cifar training code, it seems that they are using (X

mu)/std.

—
Reply to this email directly, view it on GitHub
https://github.com/cvlab-columbia/SelfSupDefense/issues/3#issuecomment-1107971452,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEUBBS27PYZVXGGZZLLYTZTVGXY5FANCNFSM5UG4LTAA
.
You are receiving this because you commented.Message ID:
@.***>


Vulnerability roundup 114: ncurses-6.2-abi5-compat: 1 advisory [7.1]

search, files

[ ] CVE-2022-29458 CVSSv3=7.1 (nixos-21.11)

CVE details
CVE-2022-29458
ncurses 6.3 before patch 20220416 has an out-of-bounds read and segmentation violation in convert_strings in tinfo/read_entry.c in the terminfo library.

Scanned versions: nixos-21.11: 3c5ae9be1f1.

Resolved through be09e32117d51bae9f05951fbd69c3c530aa03ad.


docs: replace the reference to co on the async function

Codecov Report

:exclamation: No coverage uploaded for pull request head (patch-3@832fa4a).

Continue to review full report at Codecov.

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


What is the cause of this ERROR?

Installing requirements
Installing tensorrt requirements
Traceback (most recent call last):
File "G:\Radiata\launch.py", line 224, in
prepare_environment()
File "G:\Radiata\launch.py", line 206, in prepare_environment
install_tensorrt()
File "G:\Radiata\launch.py", line 133, in install_tensorrt
assert (
AssertionError: Could not find TensorRT. Please check if it is installed correctly.

To use TensorRT mode, TensorRT must be installed on your computer.
https://docs.nvidia.com/deeplearning/tensorrt/install-guide/index.html

Thank you very much! I was able to solve the problem!

@SERO3452 could you give steps as to how you fixed it.
I'm getting the error as well through a conda env.
I also made the system environment variables.


add ability to send message to settings tab

delete !tab.url.indexOf('about:') === 0

on about and other system page not working content script


DXCDT-547: Add websocket connection and send data on connect

🔧 Changes
This PR adds the websocket connection and sends the branding data on connect. The code is only partially tested. More e2e testing will follow at a later point in time.
📚 References

🔬 Testing

📝 Checklist

[x] All new/changed/fixed functionality is covered by tests (or N/A)
[x] I have added documentation for all new/changed functionality (or N/A)

Codecov Report
All modified lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (feature/auth0-universal-login-customize@b20738a). Click here to learn what that means.

Additional details and impacted files
@@                            Coverage Diff                             @@
##             feature/auth0-universal-login-customize     #863   +/-   ##
==========================================================================
  Coverage                                           ?   73.26%
==========================================================================
  Files                                              ?       93
  Lines                                              ?    12138
  Branches                                           ?        0
==========================================================================
  Hits                                               ?     8893
  Misses                                             ?     2726
  Partials                                           ?      519

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


Using a function for test name formatting

Clear and concise description of the problem
Hello,
I'd like to supply a function for a test name. Something like this:
test.for([
  [1,'a']
])(
  ([one, a]) => `${one * 2} ${a}`, // my custom function that produces a test name
  ([one, a]) => {
    // some test
  }
})

A specific use case might be test suites that accept strings with multiple new lines. Relying on %s results in hard-to-read test output. Providing a function that automatically escapes new lines would clean up said test output, and I don't believe the current formatting tools provide such a mechanism.
Suggested solution
I'd like the ability to pass a function in as test name, that accepts the same arguments as the test function and is expected to return a string.
Alternatively, if there was a way to extend the behavior of %s (or other templating replacements) that could achieve the same thing.
Alternative
It appears this same topic was brough up here, though with minimal discussion: https://github.com/vitest-dev/vitest/issues/2403
That issue references this PR: https://github.com/vitest-dev/vitest/pull/2405 which includes this comment:

I see it still expects a function in this PR. The test name should always be a string, lets not change that.

(This PR is also linked): https://github.com/vitest-dev/vitest/pull/2405
It appears that formatName is set up to accept a function, but the function is never called; just the function's name is returned if a function is provided:
function formatName(name: string | Function) {
  return typeof name === 'string'
    ? name
    : name instanceof Function
      ? name.name || '<anonymous>'
      : String(name)

Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

I think I had a similar need before. Not sure about a proper solution, but workaround I think of is:
// manually write title as first element
test.for([
  ['1 and a', 1, 'a']
  ['2 and b', 2, 'b']
])('%s', ([_, x, y]) => {
  // ...
})

// or generate first element (probably type not inferred)
test.for(
  [
    [1, 'a']
    [2, 'b']
  ].map(([x, y]) => [`${x} and ${y}`, x, y])
)('%s', ([_, x, y]) => {
  // ...
})

Thanks for the suggestions @hi-ogawa. I'm doing something similar to your second example, mapping over the test cases to replace \n to \\n so that it prints nicely in the test output, and then again in the tests replacing back from \\n to \n. But it adds a lot of cruft and I'm reusing this pattern all through my tests (my particular use case deals with a lot of new line inputs).

Duplicate of https://github.com/vitest-dev/vitest/issues/2093
I still think the current syntax is enough and introducing different behaviour for test(Function) and test.for(Function) is not justifiable.
If you need a different value for the title, you can add it to the for array like @hi-ogawa explained.

I agree on closing this for now.


feat: add checkstyle as new lint format

What/Why/How?
Add another format for linting: Checkstyle. Can be used to pipe linting results into Jenkins, for example. Implementation is adapted from the one found in ESLint.
Reference
Testing
Screenshots (optional)
Check yourself

[x] Code is linted
[ ] Tested with redoc/reference-docs/workflows
[x] All new/updated code is covered with tests

Security

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

Adding @skadinna to review docs.

Thanks for updates @twz123!
I'll wait for somebody to confirm docs update and merge it after.

Thank you @twz123 !


run: Add an option to emit the list of a script dependencies

It can be useful for external programs to know the list of files a script
depends on. For instance, a program can watch that list of files and run jk
when one of them changes as it's possible running the script will produce a
different output.
We current emit only file dependencies:

import-file: a file imported through an import statement
read-file: a file read by std lthe output has changed as well.ib

$ jk run -d read-simple.js
[
  {
    "kind": "import-file",
    "path": "/home/damien/go/src/github.com/jkcfg/jk/read-simple.js",
    "specifier": "read-simple.js"
  },
  {
    "kind": "import-file",
    "path": "/home/damien/go/src/github.com/jkcfg/jk/foo.js",
    "specifier": "./foo"
  },
  {
    "kind": "read-file",
    "path": "/home/damien/go/src/github.com/jkcfg/jk/svc-myapp.yaml"
  }
]

Fixes: #172

It's difficult to do a test case with the current infrastructure because the file path returned are absolute (so there's no ambiguity) and so will be different on each machine. Will think about something.

All green and with a test :) :tada:


Fix: Allow Single Letter Names

Codecov Report

Merging #248 (1090727) into master (cd3497e) will increase coverage by 0.06%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #248      +/-   ##
============================================
+ Coverage     38.44%   38.50%   +0.06%
- Complexity      556      557       +1
============================================
  Files           178      178
  Lines          3283     3283
  Branches        389      389
============================================
+ Hits           1262     1264       +2
+ Misses         1883     1881       -2
  Partials        138      138

Impacted Files
Coverage Δ

...ain/java/seedu/notor/logic/parser/NotorParser.java
36.63% <ø> (+1.98%)
:arrow_up:

Continue to review full report at Codecov.

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


Corrected meta description to use SEO-supplied meta description, if supplied.

We have not been using this field, even though Michael has been supplying it.

This is partial fix for Issue #305 which corrects the top level meta description tag.


feature request: pin in retrospect

it would be useful to be able to pin chunks and their children, that are already existent in the network.
eg. swarm-cli pin eba7925a4d82c33affaf95fbf3885096487ceb27d8c9f1601f73db4f8ec7624a

With the 0.6.0 release we have added 3 commands:
pinning pin
pinning unpin
pinning list


Fix e2e test after registry changes

What type of PR is this:
/kind tests

I wonder if we should define these projectType and componentName values as constants. We have been going back and forth a lot lately, as a temporary solution, it might help to define constants.

/override windows-integration-test/Windows-test
PVC problem
/NoCluster-Integration-tests/NoCluster-Integration-tests
Non applicable

/override windows-integration-test/Windows-test
PVC problem
/override NoCluster-Integration-tests/NoCluster-Integration-tests
Non applicable

/override ci/prow/v4.11-integration-e2e


Update env.example

With the Persons API going into production, we should document the two
variables.

You can remove MEMBERS_API_HOST as well


[DEV-10893] - create request interactive mode

see this
dist/apono_darwin_arm64/apono requests create -i sdaf
Error: if any flags in the group [integration resource-type resources permissions] are set they must all be set; missing [permissions resource-type resources]


Cannot share on WP8.1 and Windows 10 desktop

When calling the share function I get this error on the javascript console:
Unable to get property 'share' of undefined or null reference
the code I used is:
window.plugins.socialsharing.share('Message only', null, null, null)
What am I doing wrong?

If I log window.plugins.socialsharing.share to the console I can see the function is defined correctly though


CMake: Added support for building in a Linux-hosted cross-compile env.

Proposed changes
CMake: Added support for building in a Linux-hosted cross-compile env.
Linux uses "aarch64" for "arm64".
Cf. https://github.com/JuliaPackaging/Yggdrasil/pull/9761
Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING document
[x] I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have updated the necessary documentation (if needed)

Don't need this variable anymore, closing.


Align panel name and properties with panel purpose

Based on the documented purpose of this panel, the name of the panel and any related panel properties (i.e. github repository, gitter channel, etc.) and references to them should be updated as well.

Panel has voted on changing the name to "Authorization Panel", per the minutes from 6/17.

I've noticed that solid organization has two teams https://github.com/orgs/solid/teams?query=auth

Authorization Panel
App Authorization Panel

Probably we want merge one into the other one.


Network errors not handled properly

When a network error occurs, the currently-active socket may be inadvertently dropped, as pointed out in https://github.com/quartiq/minimq/pull/24#issuecomment-792432265

At the time of writing my comment on #24, there is such a concern because socket_is_connected() didn't consider the case where the socket is None(). This would happen when read()/write() returns an error due to some network error (e.g. disconnection), where the socket reference after being take()-n would be left None(), meaning the next polling on the client would panic when trying to unwrap the socket by calling socket_is_connected().
Since #35 has been merged, there shouldn't be any further issue of having the socket dropped when read()/write() raises error. When polling, the client would call the updated socket_is_connected(), which guarantees to replace the empty socket reference with an open socket wrapped as Some(). Therefore, I believe the active socket can now no longer be dropped from the memory inadvertently.

But, I'm still interested to know why #24 removes the blocking mechanism in read() but keeps it in write(). Would there be any inadvertent wait on writes, and should I create a new issue based on this question?

I agree that we shouldn't be blocking on writes either, but I need to think about the implications of not being able to write to the network port.
If we don't block, we have to internally cache a received message for the future, which may be possible. I'll investigate the next steps.

Closing this issue in favor of https://github.com/quartiq/minimq/issues/38, as the original bug described in this issue was fixed in #35


heatmap layer (and other layers)

I am using the Gmaps4Rails gem and by following the wiki/readme, I have successfully created a map with points (pins).
The javascript looks like this:
handler = Gmaps.build('Google');
handler.buildMap({
provider: {
},
internal: {
id: 'map'
}
},
function(){
json_array = #{raw(@hash.to_json)};
markers = handler.addMarkers(json_array);
_.each(json_array, function(json, index){
json.marker = markers[index];
});
createSidebar(json_array);

handler.bounds.extendWith(markers);
handler.fitMapToBounds();

});
I would like to create a heatmap layer that can be toggled on and off, which would show instead of the pindrops. Despite looking through the Wiki as well as the Google Maps api (https://developers.google.com/maps/documentation/javascript/examples/layer-heatmap) I am quite lost as to how to initialize the layer and show it. Does it go into the the same handler variable or do I have to create something new entirely?
I've also posted on stackoverflow: http://stackoverflow.com/questions/28261039/creating-a-heatmap-layer-in-gmaps4rails, though no one's really looking. I also saw that someone had mentioned the trafficlayer in the issues, but there didn't seem to be a resolution. (or that it just wasn't implemented yet, but I couldn't figure out any implementation for other layers.)

of course its doable, but its out of the gem's scope.
It would demand some work and sorry but I wont do free consulting here.

Hey,
Can you please tell me how to use the heat map layer on the gmaps4rails.


No module named functools_lru_cache

When I run faucet_mininet_test.py , I got the following problem:

sudo ./faucet_mininet_test.py
required binary/library ryu-manager --version version is 4.6
required binary/library ovs-vsctl --version version is 2.5
required binary/library tcpdump -h version is 4.7
required binary/library nc present (nc from the netcat-openbsd)
required binary/library vconfig present (the VLAN you are talking about)
required binary/library fuser -V version is 22.2
required binary/library mn --version version is 2.2
required binary/library exabgp --version version is 3.4
required binary/library pip show influxdb version is 3.0
required binary/library pylint --version version is 1.6
required binary/library curl --version version is 7.5
Problem importing module variables.py: No module named functools_lru_cache
Problem importing module variables.pyc: No module named functools_lru_cache
No config file found, using default configuration
Problem importing module variables.py: No module named functools_lru_cache
Problem importing module variables.pyc: No module named functools_lru_cache
No config file found, using default configuration

I installed functools32, but the error persists.
If I run test_config.py it works without problems

I installed faucet in other machine and it works fine


ci: lint yarn lockfiles to analyze and detect security issues

Motivation
Lockfile changes are hard to review and we should make sure we don't introduce a malicious dependency either accidentally or from an external contribution. Don't trust, verify.
Description
Add CI job to lint yarn lockfiles to analyze and detect security issues
It only takes 4 seconds to execute the lint task

Example of detected issue
> npx lockfile-lint --path yarn.lock --allowed-hosts npm yarn --validate-https
detected invalid host(s) for package<EMAIL_ADDRESS>    expected: registry.npmjs.org,registry.yarnpkg.com
    actual: registry.evil.org

 ✖ Error: security issues detected!

Nice, although it doesn't seem to check a lot of things?

This will detect a malicious packages being injected into your lockfile which is the main concern, it does not verify integrity as you would have to pull the package and cross check the hash but this is already done by yarn itself when installing. There are a few more checks we could enable but those don't seem that useful.

Might make sense to also consider npm audit.

I think there is a yarn equivalent for it too. The problem with audit is that you have to differentiate between dev vs. prod dependencies and you have to run it continuously (cron job or similar) and not on a code change. Definitely something we could improve but a lot might already be covered by Github security alerts and Dependabot PRs bumping vulnerable packages.

Might make sense to automate this too. (yaml lint ?)

If it is cheap to run it is probably good idea to add that to the CI.

:tada: This PR is included in v1.16.0 :tada:


Implement ! as a way of ending dialog

The purpose of this is to have a clean way of hiding the HUD during
dialog. The VM gets in_dialog in its context and hides the HUD.
When the ! (alias end_dialog) command is called, the context
is reset. The HUD is also restored unless the event flag NO_HUD
is set.

PR comment: the VM context is there in case someone gets inventive
and wants to create checking if we've been in a dialog state for suspiciously
long or in another state where it should be impossible or whatever.
Most likely when Escoria becomes a true asset with debug facilities, that's
when it'll be most useful, in simply viewing the VM state!

Sounds good :)
Thanks !


Add support for user config file

Hello,
First, thanks for this useful project I begin to use, with really great features !
I've checked quickly, but I haven't see the management of a user config file (as .cookiecutterrc) to manage user default answers on top of template default answers.
The order of override could be (from top to bottom):

Template default values
If existing user values (from ~/.copierrc ?)
Last used values if already deployed and .copier-answers.yml exists

What do you think about that ?

Absolutely needed!


Targetare button not available.

ENVIRONMENT: production
COMPANY : Loreal
URL: https://scrapers.peviitor.ro/
Browser: Mozilla Firefox
Device: Laptop
OS: Windows 10
STEPS TO REPRODUCE:
Open the URL in the browser
Run scraper for : "Loreal" company.
Go to the top and click " Targetare"
EXPECTED RESULTS :
A new page is opened with the company name " Loreal" on "Targetare"
ACTUAL RESULTS :
There is no "Targetare" button.

Fixed!

A new page is opened with the company name " Loreal" on "Targetare"


fix(curriculum): limit hardcoding roman numerals

Checklist:

[X] I have read and followed the contribution guidelines.
[X] I have read and followed the how to open a pull request guide.
[X] My pull request targets the main branch of freeCodeCamp.
[X] I have tested these changes either locally on my machine, or GitPod.

Ref  #54063

I added two tests. One for numbers less than negative one and one for numbers greater than four thousand. This is the only amount of testing I can add without implementing the solution to the problem inside one of the tests.

Another option for some randomness, instead of full-fledged solution, could be composing randomized number to convert from some pieces.
For example:
Randomize tens 10-90
Randomize ones 1-9
Then compose expected result from the conversions:
{ 1: 'I', 2: 'II', 3: 'III', 4: 'IV', 5: 'V', 6: 'VI', 7: 'VII', 8: 'VIII', 9: 'IX',
  10: 'X', 20: 'XX', 30: 'XXX', 40: 'XL', 50: 'L', 60: 'LX', 70: 'LXX', 80: 'LXXX', 90: 'XC' }

These ranges are fully arbitrary.


unstable-vite-express template: hmr reports 'Could not Fast Refresh' on trivial edit to _index.tsx

Reproduction

pnpm create remix@latest --template remix-run/remix/templates/unstable-vite-express
Change h1 text from Welcome to Remix to Welcome to Remix!

System Info
System:
    OS: macOS 14.2.1
    CPU: (12) arm64 Apple M2 Max
    Memory: 4.28 GB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.8.1 - ~/Library/pnpm/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 9.8.1 - /opt/homebrew/bin/npm
    pnpm: 8.14.1 - ~/Library/pnpm/pnpm
  Browsers:
    Chrome: 120.0.6099.234
    Edge: 120.0.2210.144
    Safari: 17.2.1
  npmPackages:
    @remix-run/dev: ^2.5.1 => 2.5.1
    @remix-run/express: ^2.5.1 => 2.5.1
    @remix-run/node: ^2.5.1 => 2.5.1
    @remix-run/react: ^2.5.1 => 2.5.1
    vite: ^5.0.0 => 5.0.12

Used Package Manager
pnpm
Expected Behavior
Page updates without hmr reporting "Could not Fast Refresh"
Actual Behavior
Page updates and hrm reports "Could not Fast Refresh"
2:02:29 PM [vite] hmr update /app/routes/_index.tsx, /app/routes/_index.tsx?client-route
2:02:29 PM [vite] hmr invalidate /app/routes/_index.tsx?client-route Could not Fast Refresh. Learn more at https://github.com/vitejs/vite-plugin-react/tree/main/packages/plugin-react#consistent-components-exports
Note that if the meta is commented out in _index.tsx, this does not happen.
export const meta: MetaFunction = () => {
  return [
    { title: "New Remix App" },
    { name: "description", content: "Welcome to Remix!" },
  ];
};

I get the expected behavior when I revert to v2.5.0.


getting error when ng serve/ng build

ERROR in ./node_modules/yahoo-finance2/api/lib/fetchDevel.js
Module not found: Error: Can't resolve 'crypto' in '/Users/*****/AngularProject/node_modules/yahoo-finance2/api/lib'
ERROR in ./node_modules/yahoo-finance2/api/lib/fetchDevel.js
Module not found: Error: Can't resolve 'fs' in '/Users/*******/AngularProject/node_modules/yahoo-finance2/api/lib'
ERROR in ./node_modules/yahoo-finance2/api/lib/fetchDevel.js
Module not found: Error: Can't resolve 'path' in '/Users/******/AngularProject/node_modules/yahoo-finance2/api/lib'

Node modules like fs, path, and crypto aren't being found, as angular runs in the browser. fetchDevel relies on the local filesystem, so it won't work in the browser. I think that the best solution is to just not use { devel: true }.

I am installing this package in a angular project and once we have data, I want to show in the UI.. Other yahoo finance APIs are working like that but not this one..so please fix this.

where can I remove this { devel: true }?

Remove it from the "module options" parameter of any function (like search, autoc, historical, etc.). When you call something like await yahooFinance.search('AAPL', {}, { devel: true }), you are calling the search function and invoking the fetchDevel function instead of the regular fetch function. You are adding the { devel: true } there, and that doesn't work in the browser.

Still getting same error when you call like above.. I will wait until you actually fix this.

Hey @santokt, thanks for your report.  If this is happening without { devel: true } anywhere, it sounds like your bundler (ng) is trying to bundle fetchDevel.js even though it's conditionally imported, and without that condition being met.  I had a quick look at the angular project and it seems that this is indeed an unsupported configuration.  I'll put some thought into how else we can do this on our side and report back here.
I'll also note that this project is primarily intended to be run in node.  You're going to have some other challenges running in the browser (CORS, etc).  I'll add a note about that in the README (and track #54).

Yes I agree with you on Other issues like CORS. That we can handle by creating proxy.
Have a look at this npm package stock info..here I can get data directly in Angular.(with cors issue)
https://www.npmjs.com/package/stock-info

Thanks, @santokt, that's super helpful.  It looks like they're using axios there instead of the native browser fetch, I'll do some experimentation and see if we can take the same approach.  Thanks for bringing that to our attention, I'll report back here once I've had a chance to look into this.

(deleted previous message)
@santokt, did you mean you're using a proxy with stock-info?  I think I misunderstood you the first time.  Ok so you have a proxy and you just need node-yahoo-finance2 to run in the browser without the fetchDevel error...  will update you on this :)

@santokt, think this is fixed now.  If you're happy to run a development copy of the library, would be great if you could confirm (since I don't have any experience with angular's builder), but either way, pretty sure this will solve the issue and it will be in our next release later this week (you'll get an automatic notice here when that happens).

Sure.I can test the dev copy.how can I get and install that.Pls let me know and I will update you the test results.

Thanks, @santokt, that will be great.  I added some instructions at https://github.com/gadicc/node-yahoo-finance2/blob/devel/CONTRIBUTING.md#local-dev.  If anything is not clear or doesn't work, let me know.  If it's more complicated than you were expecting, feel free to just wait for the upcoming release.

Now i am not getting in installation, but while call getting below error.I guess you guys published the latest version..i just installed the latest version and tried to call in Angular.
validateAndCoerceTypes.js:143 Uncaught ReferenceError: process is not defined
at Object../node_modules/yahoo-finance2/api/lib/validateAndCoerceTypes.js (validateAndCoerceTypes.js:143)
at webpack_require (bootstrap:79)
at Object../node_modules/yahoo-finance2/api/lib/moduleExec.js (moduleExec.js:22)
at webpack_require (bootstrap:79)
at Object../node_modules/yahoo-finance2/api/index-common.js (index-common.js:8)
at webpack_require (bootstrap:79)
at Object../node_modules/yahoo-finance2/api/index-browser.js (index-browser.js:6)
at webpack_require (bootstrap:79)
at Module../src/app/yf/yf.component.ts (app.module.ts:19)
at webpack_require (bootstrap:79)

Thanks for reporting back, @santokt.  Looks like @PythonCreator27 had you covered in #50 before I even got out of bed this morning :grin: So this is fixed now... it will be in the next release (probably in a few days again).
If you don't want to wait, I think you could probably just add the following line in your app somewhere before calling yahooFinance:
window.process = window.process || {};

just once, and in the main scope of a file, not inside a function or anything.  To be clear, that's in your own app, not in the yahoo-finance source.  Should side-step this issue for you until the next release is out.

@santokt Has this been fixed in the latest version, or not?

Yes, I have tested and it worked ..Great job guys..Happy to help you on testing :)

Awesome news!  Thanks, @santokt, for reporting back.  Shweet, let us know if you come across any other issues.
Thanks again to @PythonCreator27 for your patch on this and for following up :)

Looks like this issue is back.
I'm using v2.4.3 with Angular and I'm getting these errors during build:
./node_modules/yahoo-finance2/dist/esm/src/env-node.js:21:2-17 - Error: export 'URLSearchParams' (imported as 'URLSearchParams') was not found in 'url' (possible exports: Url, format, parse, resolve, resolveObject)

./node_modules/yahoo-finance2/dist/esm/src/lib/fetchDevel.js:4:0-20 - Error: Module not found: Error: Can't resolve 'fs' in '\node_modules\yahoo-finance2\dist\esm\src\lib'

./node_modules/yahoo-finance2/dist/esm/src/lib/fetchDevel.js:5:0-28 - Error: Module not found: Error: Can't resolve 'crypto' in '\node_modules\yahoo-finance2\dist\esm\src\lib'

Hi All,
I also get this issue in v2.4.3 (with Angular 15.1.6).
Thanks

Hey @guaycuru adn @Airmime, thanks for the updated reports.
I'll definitely need help with this as I don't know anything about Angular.
Did I understand correctly that you're trying to run this in the browser?  In that case, I'm not sure why the Angular bundler is loading the node vs browser code.  (As a side note, although you can run yf2 in the browser in theory, in practice, there are a lot of other gotchas for this, notably, you'll need a node or equivalent server regardless to bypass the CORS restrictions).


Hubot Is Typing

Are there any plans to support the "... is typing" message type for Hubot? This would be really useful for some of the longer-lived ChatOps type commands I've been working on to indicate that something is happening behind the scenes.
I've had a quick experiment with trying to make it work by sending a custom message (as with sending an attachment), but not had any joy, so thought I'd open the question up to see if this is something anyone else has attempted.

There was some discussion about it over in https://github.com/github/hubot/issues/1023 but nothing concrete happened.
It looks like this is the relevant RTM event: https://api.slack.com/events/user_typing .  robot.adapter.client is an instance of https://github.com/slackhq/node-slack-client if you can figure out how to send that event back over it.

I ran into this same problem using python-slackclient.  This page (scroll down) has some details on how to do this, but you just need to send a custom JSON object with type set to 'typing', channel set to the channel ID where the bot is "typing," and id set to 1 over the RTM connection.

This would be cool indeed. :)

I think I found a workaround to add this feature
please check my PR
https://github.com/slackhq/hubot-slack/pull/281

@technicalpickles has the right idea. This might be fixed in the latest node-slack-sdk, marking as such.

Closing this as fixed in 4.0. Please reopen if I am mistaken!


Category tree

Is your feature request related to a problem? Please describe.
There is only a single level of categories.
Describe the solution you'd like
Category could have a parent category and so a category could have many category children.
This would result to a tree of categories. It is very basic to implement in model but more difficult to implement well in rendering and views.
We will need some performant enough to render a category tree without make too many big queryset.
Did Category URL will have to consume multiple name to get a high URI ? This would need to validate every category name for correct relationship.
There may be some missing points for this subject yet.

I think since tags features is on the way into a 0.5.x release, multiple categories in an article become counterproductive, especially with category tree where the multiplicity will lead to very complex queryset.
Also there is a question about the  article listing from a top category should be recursive to its children or not.

The article listing should not be recursive into category children, such as "Cheese" only list its direct article and not the ones from "Camembert". However, a convenient method may exists to recursively retrieve all article from a category and its children, but it would be only for further specific usages. The category tree feature may be released without this.

Work is on the way for this on branch category_tree.
django-treebeard has been chosen to manage parenting instead of simple foreign key to avoid performance issue with querysets about parenting because simple foreign keys would lead to as many queryset as they are items in the tree, instead some tree management like with treebeard just need one and have better method to navigate in tree.
However treebeard is not something easy to implement and we also have many things to manage aside because the parenting tree is almost creating a new tree structure aside the translation one. Basically the two tree cannot collaborate in content display, we will only be able to display parenting tree when we are listing only a single language at once.
Admin todo
We will have to adapt  queryset related to category but first treebeard parenting management have to be done in admin so we are able to continue on front usage.

[ ] Add django-treebeard to requirements and install into sandbox;
[ ] Add treebeard use on Category;
[ ] Fix tests that may fail because of new Category fields;
[ ] Add 'parent' virtual field on admin form to manage parenting (we cannot use the shipped admin list from treebeard since of multiple languages, it have to be done only in the detail form for now);
[ ] Clean 'parent' value to avoid breaking constraints (parent is always on the same language, when changing language it must be checked again possible current parent language);
[ ] Excludes possible current category children from 'parent' field listing (parent selecting its child as parent lead to broken tree and orphan circular reference);
[ ] Use the  'parent' field value to compute the tree node attributes
[ ] Parent field initial value have to be computed from the path in edition form;
[ ] CategoryParentChoiceField improvement for a better display if there is any;
[ ] CategoryTreeModelChoiceField improvement for a better display if there is any;

Front todo
Have to be thought about what to do.
API todo
API should still work well without anything more but the parent and children should be displayed in Category detail.

Done in v0.9.0


slurm-gcp-v6-controller / pre-existing-network-storage - '$controller' not added to mounts

Describe the bug
Module scripts located in community/modules/scheduler/schedmd-slurm-gcp-v6-controller/modules/slurm_files/scripts/ (develop branch)
I am creating a v6 cluster using pre-existing-network-storage with the server_ip set to $controller in the blueprint. However the start up script fails to mount the storage and times out
[root@clusterb6b-controller ~]# tail -f /slurm/scripts/setup.log
run: ['create-munge-key', '-f']
run: ['systemctl', 'restart', 'munge']
Set up network storage
Temporary failure in name resolution, retrying in 1
Temporary failure in name resolution, retrying in 2
Temporary failure in name resolution, retrying in 4
Temporary failure in name resolution, retrying in 8
Temporary failure in name resolution, retrying in 16
Temporary failure in name resolution, retrying in 32
Temporary failure in name resolution, retrying in 64
Temporary failure in name resolution, retrying in 128
Temporary failure in name resolution, retrying in 256
[Errno -2] Name or service not known
Traceback (most recent call last):
  File "/slurm/scripts/setup.py", line 494, in <module>
    main()
  File "/slurm/scripts/setup.py", line 468, in main
    {
  File "/slurm/scripts/setup.py", line 335, in setup_controller
    setup_network_storage(log)
  File "/slurm/scripts/setup_network_storage.py", line 100, in setup_network_storage
    ext_mounts, int_mounts = separate_external_internal_mounts(all_mounts)
  File "/slurm/scripts/setup_network_storage.py", line 91, in separate_external_internal_mounts
    return separate(internal_mount, mounts)
  File "/slurm/scripts/util.py", line 698, in separate
    return reduce(lambda acc, el: acc[pred(el)].append(el) or acc, coll, ([], []))
  File "/slurm/scripts/util.py", line 698, in <lambda>
    return reduce(lambda acc, el: acc[pred(el)].append(el) or acc, coll, ([], []))
  File "/slurm/scripts/setup_network_storage.py", line 88, in internal_mount
    mount_addr = util.host_lookup(server_ip)
  File "/slurm/scripts/util.py", line 687, in wrapper
    raise captured_exc
  File "/slurm/scripts/util.py", line 680, in wrapper
    return f(*args, **kwargs)
  File "/slurm/scripts/util.py", line 1160, in host_lookup
    return socket.gethostbyname(host_name)
socket.gaierror: [Errno -2] Name or service not known
Aborting setup...
run: ['wall', '-n', '*** Slurm setup failed! Please view log: /slurm/scripts/setup.log ***']

*** Slurm setup failed! Please view log: /slurm/scripts/setup.log ***

I believe the server_ip in the storage mounts should contain the host name instead of $controller, similar to how the second mount shows cluster9f3-controller successfully here?
Resolved network storage mounts: [{'fs_type': 'nfs', 'local_mount': '/home', 'mount_options': 'defaults,nofail,nosuid', 'remote_mount': '/home', 'server_ip': '$controller'}, {'server_ip': 'cluster9f3-controller', 'remote_mount': '/opt/apps', 'local_mount': '/opt/apps', 'fs_type': 'nfs', 'mount_options': 'defaults,hard,intr'}, {'fs_type': 'nfs', 'local_mount': '/opt/cluster', 'mount_options': 'defaults,nofail,nosuid', 'remote_mount': '/opt/cluster', 'server_ip': '$controller'}]
Separating external and internal mounts
Checking if mount is internal: {'fs_type': 'nfs', 'local_mount': '/home', 'mount_options': 'defaults,nofail,nosuid', 'remote_mount': '/home', 'server_ip': '$controller'}
Temporary failure in name resolution, retrying in 1
Temporary failure in name resolution, retrying in 2
Temporary failure in name resolution, retrying in 4
...

Steps to reproduce

Create a VPC and subnet
Create cluster using the blueprint
Check the instances logs

Expected behavior
Storage should be successfully mounted and timeout should not happen
Actual behavior
Timeouts as shown in the above logs
Version (gcluster --version)
gcluster version - not built from official release
Built from 'develop' branch.
Commit info: v1.37.1-167-g1d7dc338-dirty
Terraform version: 1.9.3
(tested with Terraform 1.4 too)
Blueprint
If applicable, attach or paste the blueprint YAML used to produce the bug.
blueprint_name: cluster-b6be43f5

vars:
  project_id: ofetest
  deployment_name: cluster-b6be43f5
  region: us-central1
  zone: us-central1-c
  enable_cleanup_compute: True
  enable_bigquery_load: False
  instance_image_custom: True
  labels:
    created_by: testofe-server

deployment_groups:
- group: primary
  modules:
  - source: modules/network/pre-existing-vpc
    kind: terraform
    settings:
      network_name: proper-hound-network
      subnetwork_name: proper-hound-subnet-2
    id: hpc_network

  - source: modules/file-system/pre-existing-network-storage
    kind: terraform
    id: mount_num_1
    settings:
      server_ip: '$controller'
      remote_mount: /opt/cluster
      local_mount: /opt/cluster
      mount_options: defaults,nofail,nosuid
      fs_type: nfs

  - source: modules/file-system/pre-existing-network-storage
    kind: terraform
    id: mount_num_2
    settings:
      server_ip: '$controller'
      remote_mount: /home
      local_mount: /home
      mount_options: defaults,nofail,nosuid
      fs_type: nfs

  - source: community/modules/project/service-account
    kind: terraform
    id: hpc_service_account
    settings:
      project_id: ofetest
      name: sa
      project_roles:
      - compute.instanceAdmin.v1
      - iam.serviceAccountUser
      - monitoring.metricWriter
      - logging.logWriter
      - storage.objectAdmin
      - pubsub.admin
      - compute.securityAdmin
      - iam.serviceAccountAdmin
      - resourcemanager.projectIamAdmin
      - compute.networkAdmin

  - source: community/modules/compute/schedmd-slurm-gcp-v6-partition
    kind: terraform
    id: partition_1
    use:
    - partition_1-nodeset
    settings:
      partition_name: batch
      exclusive: True
      resume_timeout: 500

  - source: community/modules/compute/schedmd-slurm-gcp-v6-nodeset
    id: partition_1-nodeset
    use:
    - mount_num_1
    - mount_num_2
    settings:
      bandwidth_tier: platform_default
      subnetwork_self_link: "projects/ofetest/regions/us-central1/subnetworks/proper-hound-subnet-2"
      enable_smt: False
      enable_placement: False
      machine_type: c2-standard-4
      node_count_dynamic_max: 1
      node_count_static: 0
      disk_size_gb: 50
      disk_type: pd-standard

  - source: community/modules/scheduler/schedmd-slurm-gcp-v6-controller
    kind: terraform
    id: slurm_controller
    settings:
      cloud_parameters:
        resume_rate: 0
        resume_timeout: 500
        suspend_rate: 0
        suspend_timeout: 300
        no_comma_params: false
      machine_type: n2-standard-2
      disk_type: pd-standard
      disk_size_gb: 120
      service_account_email: $(hpc_service_account.service_account_email)
      service_account_scopes:
        - https://www.googleapis.com/auth/cloud-platform
        - https://www.googleapis.com/auth/monitoring.write
        - https://www.googleapis.com/auth/logging.write
        - https://www.googleapis.com/auth/devstorage.read_write
        - https://www.googleapis.com/auth/pubsub
      controller_startup_script: |
        #!/bin/bash
        echo "******************************************** CALLING CONTROLLER STARTUP"
      compute_startup_script: |
        #!/bin/bash
        echo "******************************************** CALLING COMPUTE STARTUP"
      login_startup_script: |
        #!/bin/bash
        echo "******************************************** CALLING LOGIN STARTUP"
    use:
    - slurm_login
    - hpc_network
    - partition_1
    - mount_num_1
    - mount_num_2

  - source: community/modules/scheduler/schedmd-slurm-gcp-v6-login
    kind: terraform
    id: slurm_login
    settings:
      num_instances: 1
      subnetwork_self_link: "projects/ofetest/regions/us-central1/subnetworks/proper-hound-subnet-2"
      machine_type: n2-standard-2
      disk_type: pd-standard
      disk_size_gb: 120
      service_account_email: $(hpc_service_account.service_account_email)
      service_account_scopes:
        - https://www.googleapis.com/auth/cloud-platform
        - https://www.googleapis.com/auth/monitoring.write
        - https://www.googleapis.com/auth/logging.write
        - https://www.googleapis.com/auth/devstorage.read_write

Output and logs
N/A - blueprint is successfully deployed
Execution environment

OS: Rocky 8.5
Shell: bash
go version: go1.22.5
Terraform: both 1.4 and 1.9

Other info
I have added a quick fix to the resolve_network_storage function that is located in setup_network_storage.py (the "for mount in mounts.values()" loop) as I noticed similar logic relating to $controller in util.py
def resolve_network_storage(nodeset=None):
    """Combine appropriate network_storage fields to a single list"""

    if lkp.instance_role == "compute":
        try:
            nodeset = lkp.node_nodeset()
        except Exception:
            # External nodename, skip lookup
            nodeset = None

    # seed mounts with the default controller mounts
    if cfg.disable_default_mounts:
        default_mounts = []
    else:
        default_mounts = [
            NSDict(
                {
                    "server_ip": lkp.control_addr or lkp.control_host,
                    "remote_mount": str(path),
                    "local_mount": str(path),
                    "fs_type": "nfs",
                    "mount_options": "defaults,hard,intr",
                }
            )
            for path in (
                dirs.home,
                dirs.apps,
            )
        ]

    # create dict of mounts, local_mount: mount_info
    mounts = mounts_by_local(default_mounts)

    # On non-controller instances, entries in network_storage could overwrite
    # default exports from the controller. Be careful, of course
    mounts.update(mounts_by_local(cfg.network_storage))
    if lkp.instance_role in ("login", "controller"):
        mounts.update(mounts_by_local(cfg.login_network_storage))

    if nodeset is not None:
        mounts.update(mounts_by_local(nodeset.network_storage))

    # Replace $controller with the actual hostname in all mounts
    for mount in mounts.values():
        if mount['server_ip'] == '$controller':
            mount['server_ip'] = cfg.slurm_control_host

    return list(mounts.values())

This successfully gets the startup scripts to run and gets the login and controller nodes online
Setting up network storage
Resolving network storage
Resolved network storage mounts: [{'fs_type': 'nfs', 'local_mount': '/home', 'mount_options': 'defaults,nofail,nosuid', 'remote_mount': '/home', 'server_ip': 'clusterad2-controller'}, {'server_ip': 'clusterad2-controller', 'remote_mount': '/opt/apps', 'local_mount': '/opt/apps', 'fs_type': 'nfs', 'mount_options': 'defaults,hard,intr'}, {'fs_type': 'nfs', 'local_mount': '/opt/cluster', 'mount_options': 'defaults,nofail,nosuid', 'remote_mount': '/opt/cluster', 'server_ip': 'clusterad2-controller'}]
External mounts: [], Internal mounts: [{'fs_type': 'nfs', 'local_mount': '/home', 'mount_options': 'defaults,nofail,nosuid', 'remote_mount': '/home', 'server_ip': 'clusterad2-controller'}, {'server_ip': 'clusterad2-controller', 'remote_mount': '/opt/apps', 'local_mount': '/opt/apps', 'fs_type': 'nfs', 'mount_options': 'defaults,hard,intr'}, {'fs_type': 'nfs', 'local_mount': '/opt/cluster', 'mount_options': 'defaults,nofail,nosuid', 'remote_mount': '/opt/cluster', 'server_ip': 'clusterad2-controller'}]
Instance is controller, using external mounts
Creating backup of fstab
Restoring fstab from backup
Mounting fstab entries
Handling munge mount
About to run custom scripts
Determined custom script directories: [PosixPath('/slurm/custom_scripts/controller.d')]
Collected custom scripts: [PosixPath('/slurm/custom_scripts/controller.d/ghpc_startup.sh')]
Custom scripts to run: /slurm/custom_scripts/(controller.d/ghpc_startup.sh)
Processing script: /slurm/custom_scripts/controller.d/ghpc_startup.sh
Running script ghpc_startup.sh with timeout=300
run: /slurm/custom_scripts/controller.d/ghpc_startup.sh
ghpc_startup.sh returncode=0
stdout=******************************************** CALLING CONTROLLER STARTUP
This is the startup script for the controller on cluster 3

Unfortunately Slurm still isn't configured correctly as shown below, so it is possibly not being replaced elsewhere in the module too. Happy to provide more info if required.
s[root@clusterad2-controller ~]# srun -p batch hostname
srun: Required node not available (down, drained or reserved)
...
[root@clusterad2-controller ~]# cat /var/log/slurm/slurmdbd.log
...
[2024-08-05T14:38:07.093] select/cons_tres: select_p_reconfigure: select/cons_tres: reconfigure
[2024-08-05T14:38:07.093] select/cons_tres: part_data_create_array: select/cons_tres: preparing for 2 partitions
[2024-08-05T14:39:06.000] SchedulerParameters=bf_continue,salloc_wait_nodes,ignore_prefer_validation
[2024-08-05T14:42:08.506] sched: _slurm_rpc_allocate_resources JobId=1 NodeList=clusterad2-partition1node-0 usec=2040
[2024-08-05T14:42:09.769] _update_job: setting admin_comment to GCP Error: Permission denied on locations/{} (or it may not exist). for JobId=1
[2024-08-05T14:42:09.769] _slurm_rpc_update_job: complete JobId=1 uid=981 usec=112
[2024-08-05T14:42:09.780] update_node: node clusterad2-partition1node-0 reason set to: GCP Error: Permission denied on locations/{} (or it may not exist).
[2024-08-05T14:42:09.780] Killing JobId=1 on failed node clusterad2-partition1node-0
[2024-08-05T14:42:09.780] update_node: node clusterad2-partition1node-0 state set to DOWN

Hi @scott-nag , thank you for reporting!
To be fixed by https://github.com/GoogleCloudPlatform/slurm-gcp/pull/194

this is working now perfectly, thank you for the quick fix!

https://github.com/GoogleCloudPlatform/slurm-gcp/pull/194 is included in the latest release.


Fix use-after-free in timer manager

Fixes #11815

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences


720p

Not showing 720p links anymore!!

Example?

https://maple-ytdl.herokuapp.com/?id=T2pdmZhDXfo&format=1
The example video has 720p but no link in json.

This video doesn't have 720p.

My bad there are two demo url's in your example :(


Automatically rescans when download cancelled

Describe the bug
The software rescans assets every time a download is cancelled, even if this was the first download to happen.
To Reproduce
Steps to reproduce the behavior:

Click on a route with downloadable assets
Click on 'Download packages'
Click on 'Cancel'
Rescan happens

Expected behavior
If this was the first download and it has been cancelled by user : never rescan.
Else, let the user have the choice to rescan or not (maybe through a setting "Automatically rescan : Yes or No").
It would also be nice, since the rescan takes some time, if it didn't prevent the user to start downloading packages meanwile.
OS: Windows 10
Version <IP_ADDRESS>

Most probably resolvced somewhere in process.
If not, please reopen the issue.


fix(configx): goroutine leak

#661 attempted to improve context usage in configx by using context.Background() instead of storing context in the struct.
With the changes from #666, watcherx however does not automatically close channels, so configx started leaking go routines.
We can actually easily avoid storing context, but also passing it down to the watcher such that context cancelation automatically trickles down, no need for a .Close() function.

I find the use of contexts for cancelation of background workers a little bit wonky. Constructors should generally not take contexts as arguments, in my opinion. That's not a thing in the Go standard library either. Contexts are for request-scoped data.
 But we definitely need some docs around these functions, clearly explaining what they use the context for and how and when any background goroutines are reclaimed.

I think a better API for the config provider thingy would be to not spawn background goroutines directly in the constructor. Instead, have a dedicated opt-in function [for example func (*Provider) BackgroundReload() (stop func())], which makes it obvious where the goroutines are coming from and makes it harder to leak them.


Expected spend allocator not constraining budget or adjusting optimal spend

Hello all
I am attempting to run the allocator for an expected spend level on a model I have saved as an RDS object. The output is strange, where the Initial vs. Optimized Budget Allocation is the same across all channels, but Total Spend Increase is negative. Furthermore, the Optimized Mean Response is universally lower. I do not run into this issue when doing max historical response.
I attached a screenshot of the results. The code is as follows: (Note that I get the same issue when I don't constrain date_min or date_max)

rds_input <- "~/Documents/Robyn MMMs/MyRobyn_v5_1-651-3.RDS"
select_model <- "1_651_3"
start_date <- "2022-01-01"
end_date <- "2022-12-01"
budget <- 400000
n_days <- 31
rds <- readRDS(rds_input)
InputCollect <- rds$listInit$InputCollect
OutputCollect <- rds$listInit$OutputCollect
paid_vars <- InputCollect$paid_media_spend
AllocatorCollect <- robyn_allocator(
InputCollect = InputCollect
, OutputCollect = OutputCollect
, select_model = select_model
, scenario = "max_response_expected_spend"
, channel_constr_low = rep(0.7, length(paid_vars))
, channel_constr_up = rep(1.5, length(paid_vars))
, export = TRUE
, expected_spend = budget
, expected_spend_days = n_days
, date_min = start_date
, date_max = end_date
)

Can you retry with a larger spend, like 1mio? It should look more intuitive. the -30% total spend means that the 400k is 30% less than your historical average in the same length (31days), thus all responses also decrease because of less spend. Check the allocator output table to understand details.

Just an additional comment: since version 3.7.1 you can save Robyn models with JSON files which will contain all the necessary information to replicate them. We won't keep supporting RDS files even though it might still work.

Ok, thank you! Increasing the spend seems to have fixed it.


docs: update artifact names in image to image notebook

Update artifact names in image to image notebook
The artifact names of the image-to-image tutorial only worked without finetuner/ prefix since they are not in the finetuner account. Howver, there the same artifacts are in the finetuner account with slightly different names. Therefore the names needed to be changed from *-data to *-da

[ ] This PR references an open issue
[x] I have added a line about this change to CHANGELOG

We instead pushed the correct artifacts with the names in the documentation


Nextcloud integration

Integrating this app into nextcloud and syncing via webdav would be great.

Duplicate of #59 , #14
Leaving it open anyways for now

We are looking into sharing and sync functionality across all our apps. This feature will sadly still take some time..

Thanks for your quick reply. I had read those issues before posting.
With integrating this app into nextcloud I meant having a web ui in nextcloud that can be downloaded from the nextcloud app store and consecutively integrating the webdav sync. Therefore I don't think it is a duplicate.

Ah, okay. Thank you for the explanation.
I've deleted the marking of duplicate Issues and we will discuss this within the team and see if this feature will make it into future releases.

Sync with e.g. Nextcloud might be good.
I think it would be fine also to have other ways, e.g. just an easy import/export (XML, Json, etc.).

In the mean time, an IMHO helpful description of how to use NextCloud's task sync for shopping lists: https://ownyourbits.com/2017/12/30/sync-nextcloud-tasks-calendars-and-contacts-on-your-android-device/#comment-5761

I would prefer a generic WebDAV sharing process. Not everyone wants to own and manage nextcloud or use nextcloud hosted by them. In my case, I'd just have an apache server with webdav mod.


[newrelic-pixie-integration] Expose timeout in values.yml

Pixie integration has default timeouts for metric collection that might be too low for some environments. These timeouts can be overridden through environment variables (https://github.com/newrelic/newrelic-pixie-integration/blob/main/internal/config/config.go), but those cannot be easily set from the values.yml file.

Closing as this has been descoped from the roadmap.


MySQL auto_increment value is not current in table properties

DBeaver 22.1.1 does not display the correct (current) auto increment value in the Auto Increment field. However, the value is correct in the DDL section. Here's an example:

The AI field contains 1707219 while the correct value is 1707223 as displayed in DDL. In effect we have two different values. Refreshing does not help.
This appears to be caused by MySQL caching the value when getting it from information_schema or by SHOW TABLE STATUS. From my research current value can be retrieved with these methods:

Use SHOW CREATE TABLE and extract the value from the output.
Use SET information_schema_stats_expiry=0 before querying information_schema or SHOW TABLE STATUS (see docs).
Run ANALYZE TABLE before querying information_schema or SHOW TABLE STATUS.

I think DBeaver should use some method for fetching up to date information. Probably method 1 or 2 would be suitable here. At the very least fresh information should be fetched when I refresh the Properties view with the button or F5.

Hello @lemon-juice,
DDL is read separately from the metadata you see on top, so that's why they aren't in sync.
By refreshing the editor, you refresh the lower panel that has DDL in it. To refresh the contained object, you have to refresh that object in the navigator tree. Perhaps we can improve this behavior.

I have the same behavior. The refreshing won't work for me.
I will get the error too, if I newly start DBeaver and open the table in the editor.


[webapi] Update suite.json to support bdd for aio package

Update suite.json to support bdd for aio package in tct-geoallow-w3c-tests,
tct-geodeny-w3c-tests, tct-notification-w3c-tests

Please provide the UI result

@Honry  I have updated it following your comments

Thanks, so it works well on all-in-one package?

@Honry Test packages for aio build successfully,  bdd tests got pass.

LGTM


[BUG][Functional] Security Hub - Unable to automatically disable security framework

Required Basic Info

Accelerator Version:  v1.2.3
Install Type: ALL
Install Branch: Standalone
Upgrade from version: N/A

Describe the bug

the Accelerator properly enables and configures security hub
the Accelerator properly deploys the selected security frameworks
no ability to disable a security framework automatically, once enabled
Specifically customer enables AWS, PCI and CIS frameworks
customer then wishes to disable PCI frame work
PCI framework is removed from ASEA control, but remains activated in all accounts

Failure Info

No error messages

Expected behavior
Changing this:
    "security-hub-frameworks": {
      "standards": [
        {
          "name": "AWS Foundational Security Best Practices v1.0.0",
          "controls-to-disable": ["IAM.1"]
        },
        {
          "name": "PCI DSS v3.2.1",
          "controls-to-disable": ["PCI.IAM.3", "PCI.KMS.1", "PCI.S3.3", "PCI.EC2.3", "PCI.Lambda.2"]
        },
        {
          "name": "CIS AWS Foundations Benchmark v1.2.0",
          "controls-to-disable": ["CIS.1.20", "CIS.1.22", "CIS.2.8"]
        }
      ]
    },

to this
    "security-hub-frameworks": {
      "standards": [
        {
          "name": "AWS Foundational Security Best Practices v1.0.0",
          "controls-to-disable": ["IAM.1"]
        },
        {
          "name": "CIS AWS Foundations Benchmark v1.2.0",
          "controls-to-disable": ["CIS.1.20", "CIS.1.22", "CIS.2.8"]
        }
      ]
    },

should disable the PCI DSS framework (or any/all frameworks removed)
would it be easier if we added a "disabled": true instead? i.e.

        {
          "name": "PCI DSS v3.2.1",
          "controls-to-disable": [],
          "disabled": true
        },

Additionally, while not originally a requirement

Could we do the same for individual controls (assuming not an extensive amount of extra work?

i.e. "controls-to-disable": ["CIS.1.20", "CIS.1.22", "CIS.2.8"] changing to "controls-to-disable": ["CIS.1.20", "CIS.1.22"] would re-enable controls "CIS.2.8"

QUESTION

Would it be easier to simply update the codebase to use this feature and simplify the codebase all at the same time:
https://github.com/aws-samples/aws-secure-environment-accelerator/issues/516

Shipping in v1.2.4

Shipping in v1.2.4


Prevent submissions that are "Penn State Only" with an open access license.

Feature Request
Depositors sometimes select "Penn State Only" access and apply a license that would allow the Libraries to share the work publicly (i.e., any license other than "rights reserved"). In these case, the access setting and the license conflict with one another and we should do a better job to notify users about this conflict so they can change either the license or the access setting.
Describe the solution you'd like
If a user selects "Penn State Only" access and an OA license (anything other than "In Copyright (Rights Reserved)") they get a warning:

The access and license settings conflict with each other. Please set access to "Public" (preferred), or set the license to "In Copyright (Rights Reserved)."

This validation should not prevent updates by administrators to existing works where the access and license settings are in conflict.
We use URLs to identify licenses in config/authorities/licenses.yml. We might use a setting like the following to configure the validation above:
psu_access_required_license: [https://rightsstatements.org/page/InC/1.0/]

This validation should not prevent updates by administrators to existing works where the access and license settings are in conflict.

@srerickson remind me, do admins use the same form as plebeians to do this work? Or is there a special admin interface?

admins use the same form, which probably complicates implementation of this.

It sorta does complicate it, but also sorta doesn't because of the way Rails does validations. I have a plan.

@rschenk might be worth keeping in the big picture plan in mind as you do this (#1153) -- or not.


Remove obsolete tests

As discussed in #3007, it might be worth to remove the tests in tests/tests/old. Or is there some reason these are still there?

No, they can definitely go.
I'm still holding out hope that some kind soul will help migrate the legacy
simpletests, but those are in a different directory and since they're not
phpunit they probably aren't screwing stuff up.
On Sat, Oct 24, 2015 at 12:51 AM, Antti Hukkanen<EMAIL_ADDRESS>wrote:

As discussed in #3007 https://github.com/concrete5/concrete5/issues/3007,
it might be worth to remove the tests in tests/tests/old. Or is there
some reason these are still there?
—
Reply to this email directly or view it on GitHub
https://github.com/concrete5/concrete5/issues/3052.


allow dynamic strings to be used in metrics_format option

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then  say 'jenkins, test it'.

Apologies @aaronhanson , I somehow missed this PR. This functionality was just put in by me in an accidental case of duplicate work :(. We'll try to be better about it next time!

Cool, good to know.


Fix PowerShell from statement to be from current location

Please use mcr.microsoft.com for the best support.  https://azure.microsoft.com/en-us/blog/microsoft-syndicates-container-catalog/

Thanks for this! I'm checking internally about the CLA bit and will update here as soon as I know more.

@TravisEz13 the instructions I'm getting is to close this PR with a note and do the change myself, explaining.
Would that be OK with you or do you want to reconsider signing the CLA?

CLA was signed

FYI: The example is using the latest tag which does change distribution periodically.  We moved from Ubuntu 16.04 to 20.04 as .NET dropped support for 16.04 in their latest release.  This is probably okay for your sample, but I just wanted to point it out.

Thanks!
I'll run CI before merging and we'll see if having the newest version works or not. If it doesn't I'll pin to whatever works until I have the time to update.

CI failures are unrelated to this PR and already being tracked here:

Cloud Sql: #1311
Storage: #1272

I'll force merge. Thanks again for this @TravisEz13


bl_setArmorTextures crashes upon use

The title says it all. Tried your example and it also crashes.

Crash log please?
Also, MFW I get this bug report for the 3rd time

BL crashlog or CatLog?

Never mind, found the issue. Yes, it needs ModPE scripts enabled to work.

oh, ill try now

@jose-vm I can make this work with ModPE scripts turned off if you need it. If you don't need it, just close the issue.

@zhuowei Why you don't make things that could work alongside addons work by default?
PD: Did I express myself well?

It works, but you need to have ModPE script enabled :b

Should I close it?

@InusualZ I do this for a few important things (extended item IDs, initializing Substrate), but I never test with scripts disabled, so a lot of the time I don't even know that stuff stops working with scripts disabled

@zhuowei Yep, keep it
Also, make it useable without ModPE script support please


Cannot run Julia-defined function from example in README

I am running Julia 0.6.4 and Python 3.7.0. I tried running the following example from the README to use a Julia-defined function
jul_f = de.eval("""
function f(du,u,p,t)
  x, y, z = u
  sigma, rho, beta = p
  du[1] = sigma * (y - x)
  du[2] = x * (rho - z) - y
  du[3] = x * y - beta * z
end""")
u0 = [1.0,0.0,0.0]
tspan = (0., 100.)
p = [10.0,28.0,2.66]
prob = de.ODEProblem(jul_f, u0, tspan, p)
sol = de.solve(prob)

But I get the error
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-2-c07076744217> in <module>
     10 tspan = (0., 100.)
     11 p = [10.0,28.0,2.66]
---> 12 prob = de.ODEProblem(jul_f, u0, tspan, p)
     13 sol = de.solve(prob)

ValueError: Julia exception: ArgumentError("reducing over an empty collection is not allowed")

Strangely, the example below that uses a Python-defined function works correctly for me
def f(du,u,p,t):
    x, y, z = u
    sigma, rho, beta = p
    du[0] = sigma * (y - x)
    du[1] = x * (rho - z) - y
    du[2] = x * y - beta * z

numba_f = numba.jit(f)
u0 = [1.0,0.0,0.0]
tspan = (0., 100.)
p = [10.0,28.0,2.66]
prob = de.ODEProblem(numba_f, u0, tspan, p)
sol = de.solve(prob)

Thanks for the help!

The current release might be only v1.0 compatible?

I've tried this on Julia 1.0.2, but now I cannot import diffeqpy
>>> import diffeqpy
>>> from diffeqpy import de
Traceback (most recent call last):
  File "/home/mikeyu/.pyenv/versions/3.7.0/lib/python3.7/site-packages/julia/core.py", line 808, in __getattr__
    return getattr(self.__julia, name)
AttributeError: 'Julia' object has no attribute 'add_module_functions'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/mikeyu/.pyenv/versions/3.7.0/lib/python3.7/site-packages/diffeqpy/de.py", line 2, in <module>
    _jul = setup()
  File "/home/mikeyu/.pyenv/versions/3.7.0/lib/python3.7/site-packages/diffeqpy/__init__.py", line 20, in setup
    jul.add_module_functions('DifferentialEquations')
  File "/home/mikeyu/.pyenv/versions/3.7.0/lib/python3.7/site-packages/julia/core.py", line 810, in __getattr__
    return getattr(Main, name)
  File "/home/mikeyu/.pyenv/versions/3.7.0/lib/python3.7/site-packages/julia/core.py", line 107, in __getattr__
    return self.__try_getattr(name)
  File "/home/mikeyu/.pyenv/versions/3.7.0/lib/python3.7/site-packages/julia/core.py", line 132, in __try_getattr
    raise AttributeError(name)
AttributeError: add_module_functions

I confirmed that PyCall works for the example on PyCall's README
julia> using PyCall

julia> @pyimport math

julia> math.sin(math.pi / 4) - sin(pi/4)
0.0

I also confirmed that pyjulia works for the example on pyjulia's README
>>> from julia import Base
>>> Base.sind(90)
1.0
>>> from julia import Main
>>> Main.xs = [1, 2, 3]
>>> Main.eval("sin.(xs)")
[0.8414709848078965, 0.9092974268256817, 0.1411200080598672]

@tkf ?

As for the 1.0.2 error, it is expected as Julia 1.x (or rather PyJulia 0.2) only works with diffeqpy on git master.  I'm not sure what's going on with Julia 0.6.4 case.  I'll look into it later.
Meanwhile, you can try the following with Julia 1.x and PyJulia 0.2:
from julia import Main

Main.eval("""
using DifferentialEquations
using PyCall

PyCall.PyObject(::typeof(DiffEqBase.solve)) =
    PyCall.pyfunctionret(solve,Any,Vararg{PyCall.PyAny})

const inspect = pyimport("inspect")

function DiffEqBase.numargs(f::PyCall.PyObject)
    haskey(f,:py_func) ? _f = f[:py_func] : _f = f
    if PyCall.pyversion < v"3.0.0"
        return length(first(inspect[:getargspec](_f)))
    else
        return length(first(inspect[:getfullargspec](_f)))
    end
end
""")

from julia import DifferentialEquations as de

then try the README example:
# Use Main.eval, not de.eval:
jul_f = Main.eval("""
function f(du,u,p,t)
  x, y, z = u
  sigma, rho, beta = p
  du[1] = sigma * (y - x)
  du[2] = x * (rho - z) - y
  du[3] = x * y - beta * z
end""")

u0 = [1.0,0.0,0.0]
tspan = (0., 100.)
p = [10.0,28.0,2.66]
prob = de.ODEProblem(jul_f, u0, tspan, p)
sol = de.solve(prob)

This solution works. Thanks!

#36 should fix it.
Re:

<ipython-input-2-c07076744217> in <module>
     10 tspan = (0., 100.)
     11 p = [10.0,28.0,2.66]
---> 12 prob = de.ODEProblem(jul_f, u0, tspan, p)
     13 sol = de.solve(prob)

ValueError: Julia exception: ArgumentError("reducing over an empty collection is not allowed")

This was actually doing something similar to
jul_f = """
function f(du,u,p,t)
  x, y, z = u
  sigma, rho, beta = p
  du[1] = sigma * (y - x)
  du[2] = x * (rho - z) - y
  du[3] = x * y - beta * z
end"""  # yes, String!
ODEProblem(jul_f, u0, tspan, p)

in Julia.  This is because now (due to PyJulia change) diffeqpy.de.eval really points to DifferentialEquations.eval which is an identity function for a String.  Using Main.eval instead is the way to go.

Yes, now that tests are passing let's do a release and close this.

@tkf I added you as an author in the METADATA

Thanks! :)

Alright, a v1.0-compatible version is on PyPI now.


Fix compatibility with Python 3.11 by updating the vendored version of attrs

Issue description
When running latest version of pipenv (2022.10.25) on Python 3.11 it shows a runtime warning:
$ pipenv --version
/__venv__/lib64/python3.11/site-packages/pipenv/vendor/attr/_make.py:876: RuntimeWarning: Running interpreter doesn't sufficiently support code object introspection.  Some features like bare super() or accessing __class__ will not work with slotted classes.
  set_closure_cell(cell, cls)
pipenv, version 2022.10.25

This has been fixed in attrs version 22.1.0 (see https://github.com/python-attrs/attrs/pull/969). However, pipenv currently vendors version 21.2.0 (see https://github.com/pypa/pipenv/blob/main/pipenv/vendor/vendor.txt#L1).
Please update the vendored version of attrs to 22.1.0 or later to fix compatibility with Python 3.11
Expected result
$ pipenv --version
pipenv, version 2022.10.25

Actual result
See issue description.
Steps to replicate
Launch any pipenv command on Python 3.11.
$ pipenv --version

Note that this is blocking the update of pipenv in Fedora: https://src.fedoraproject.org/rpms/pipenv/pull-request/56

Fix by #5450.


refactoring

refactoring layout architeture more efficiently
refoctoring api scopes

Layout protocol in SwiftLayout currently is being to some kind of concrete anonymous type. and all apis in Layout should be implemented by using for custom implementations not UIView. may be it is time to admit that situation in limits.

ok done. I should allow to some of rests to me.


Are there any TypedChainOfThoughtHint?

Thank you for your wonderful work.
I would like to ask, you created TypedChainOfThought based on typed predictor. But I don’t seem to have found a way to add hints to this COT. Do you have anything similar to TypedChainOfThoughtHint?

There isn’t, all predictors will become implicitly typed within a few months…

Thanks for your reply, but I'm still confused about what the 'implicitly typed' means. Can you be more specific? Thank you.


Combat Reach Doesn't Scale with Size (Better Combat Rebirth & Other Combat Reach Mods)

Here I am again, hi.
This is a bug with the Better Combat Rebirth mod, where your reach (against entities only) doesn't get shorter at smaller scales. Also, the features of BCR don't actually trigger when reaching mobs farther than you normally would at normal size. Basically, the reach BCR uses doesn't scale as it should. A similar problem also occurs with the mod Potion Core, where your reach against other mobs is stuck at normal size, and you can't actually hit anything far away when big (haven't tested if this still persists, but it has happened before)
To reproduce, just use BCR (or similar combat reach changing mod, like Potion Core) and CM together, and change your size. Then just hit some mobs farther than you're supposed to, or outside normal size reach.
Combat reach should scale along with your size.
A lot of mods, but the two important ones are BCR and CM.
Forge Version: 2847
Mod Version: Any, but I used the latest beta.
Didn't actually report this one until now, I found this long ago. Hopefully it doesn't add too much more work before the release.

It is their bug, because they just hardcoded the reach distance (btw wtf is this "longerAttack" config anyway, lol), instead of using the (vanilla!) reach attribute (e.g. player.getEntityAttribute(EntityPlayer.REACH_DISTANCE).getAttributeValue() instead of just 5.0).
This mod does (in the beta atm) scale the attribute, so there is no bug here on my side.
You may ask them to fix that, I am not proactively fixing all the mods :shrug:
This is (almost, because they could've avoided that) the same as asking me to fix modded entities not having their particles scaled, or like any custom logic that depends on distances somehow - the mod tries to ensure that at least vanilla entities and their effects are scaled, weird mod interactions where something will not be scaled are unavoidable.
I try to fix mod compat crashes and fixed that rfp issue on inertia and because it was very easy to do (and not their fault they do not integrate with my mod lol).