text:
namespace: Allow container to join pid namespace of agent

This is a debug-only feature and is disabled by default reflected
by the value of "enable_agent_pidns" in the configuration.toml
which is set to false by default.
Allow a container to join the pid namespace of the agent.
This is allowed using an environment variable "AGENT_PIDNS".
Only if this variable is set and configuration explicitly enabled to
allow this, does the runtime set the "agentPidNs" flag in the
CreateContainer grpc request sent to the agent.
Fixes #2633
Signed-off-by: Archana Shinde archana.m.shinde@intel.com

@devimc @jodh-intel  @bergwolf Please take a look.

why only this feature can be enabled through the container's environment? what's the difference with the rest of features (use_vsocks, accelerators, etc)?

@devimc We want to support this feature on a per container basis, rather than a sandbox/pod basis. If k8s supported container specific annotations, I would have used those. That would have been ideal, but unfortunately container specific annotations are not supported in k8s today. Hence the decision to go with container's environment.

@devimc We want to support this feature on a per container basis, rather than a sandbox/pod basis. If k8s supported container specific annotations, I would have used those. That would have been ideal, but unfortunately container specific annotations are not supported in k8s today. Hence the decision to go with container's environment.

thanks @amshinde , that make sense, how about enabling this feature if and only if AGENT_PIDNS == true && enable_agent_pidns == true ? this way we are sure that the admin (root) explicitly allow containers to join the agent's pid namespace. I don't like the idea of having annotations for debugging features.

@devimc I am following the exact same logic that you mentioned. Have a look:
https://github.com/kata-containers/runtime/pull/2723/commits/8d8fd6dcbf06ed7933b26bd20e1ed483d19da92a#diff-7ba6ce1174401126aaba6a1cff25434dR1475

@jodh-intel @devimc Can you take another look?

/test

@jodh-intel @bergwolf Any other comments?

/test

/test

/test

/test-ubuntu

/test-ubuntu

authors: amshinde devimc


text:
build: check golang version meets min req.

Check that the system golang version is new enough to build with
according to the data from the versions.yaml file.
Update the verions in the versions.yaml accordingly, and add a note
describing what the 'newest-version' item represents.
Note, we only do a minimum requirement check, and are not checking
against the 'newest-version' info from the yaml.
Fixes: #148
Inspired-by: Wei Zhang zhangwei555@huawei.com
Idea-by: James O. D. Hunt james.o.hunt@intel.com
Signed-off-by: Graham Whaley graham.whaley@intel.com

Replaces #762
@jodh-intel @chavafg and if you are around @WeiZhang555 ptal.
Let's see what the CI thinks of this....
/test

Oh, this has hit the same struct align issue as #744
virtcontainers/pkg/mock/cc_proxy_mock.go:24:18:warning: struct of size 160 could be 152 (maligned)
virtcontainers/monitor.go:15:14:warning: struct of size 80 could be 72 (maligned)
Build step 'Execute shell' marked build as failure
Performing Post build task...

@jodh-intel - did we have a plan?

@jodh-intel thanks for the info. Hmm, I suspect both our PRs are going to sit for a short while then until we/somebody can fix up the cri-containerd tests :-( but, at least we found that before merging!

Adding dnm till cri tests are fixed.

@grahamwhaley I submitted a fix to cri-containerd that has been merged in master: https://github.com/containerd/cri/pull/941
I am not sure which version of cri we are using, we can consider applying that patch till it becomes part of a release.

The cri-containerd version information is in versions.yaml so we should be able to bump the version on this PR to have the tests pull in that version I think:

https://github.com/kata-containers/runtime/blob/master/versions.yaml#L139..L145

OK, let me see if I can bump that version. But, the patch from @amshinde (btw, thank you! ;-) ) is not in a release yet, and I suspect may not be for some time (hard to tell) - so, I'll have to see if our yaml maybe supports pinning to a commit version.

It should do, yes.

/test

Looks like we need to add yq to travis as well.
$ make
make: yq: Command not found
golang.mk:12: *** "ERROR: cannot determine minimum golang version".  Stop.

What's the recommended way there @jodh-intel @chavafg - do I try to add it into the .travis.yml or somehow tie it into .ci/static-checks.sh? I suspect figuring out the 'correct' way to get travis to add packages is preferable?

Hmm, and strange osbuilder fail on 16.04initrd:
Digest: sha256:c26d7183c5d6341ef2eb8a8c7e6e2053a7f9d03e7a156bca2dd17c46629cff44
Status: Downloaded newer image for golang:1.9.2-alpine3.7
 ---> f1c23f11852e
Step 2/2 : RUN apk update && apk add git make bash gcc musl-dev linux-headers apk-tools-static libseccomp libseccomp-dev &&     echo -e '#!/bin/sh\nuname -m' > /usr/bin/arch && chmod +x /usr/bin/arch
 ---> Running in 8985e9408842
fetch http://dl-cdn.alpinelinux.org/alpine/v3.7/main/x86_64/APKINDEX.tar.gz
Build timed out (after 5 minutes). Marking the build as aborted.
Build was aborted

and seems to have hit the no-nesting issue on 16.04 image.
mumble...

and.... seems I blew the line length check for the meta description in the versions.yaml. spinning again...
versions.yaml
  188:81    error    line too long (92 > 80 characters)  (line-length)

/test

I had a look at the Travis item. They show using apt in the before_install section, and yq does not have a standard ubuntu package, but on their page they show either a snap install or adding a ppa like (here to our before_script section):
before_script:
  -sudo add-apt-repository ppa:rmescandon/yq
  -sudo apt update
  -sudo apt install yq -y
...

Thoughts @jodh-intel @chavafg ? I'm ready to add that patch to this PR and spin it...

I had a look at the Travis item. They show using apt in the before_install section, and yq does not have a standard ubuntu package, but on their page they show either a snap install or adding a ppa like (here to our before_script section):
before_script:
  -sudo add-apt-repository ppa:rmescandon/yq
  -sudo apt update
  -sudo apt install yq -y
...

Thoughts @jodh-intel @chavafg ? I'm ready to add that patch to this PR and spin it...

fwiw there's a install_yq function in tests .ci/lib.sh you may use too, but I checked and there's no way to call it without some wrapper script.

/test
let's see if the yq travis apt fix works...

@grahamwhaley ping :)

Updated. rebased to pick up the relevant golang version.yaml changes. Bumped the known max version from 1.11 to 1.11.1. Moved the travis apt yq init into the 'install' section, so let's see if that fixes travis.
/cc @marcov @jodh-intel

@grahamwhaley Travis still failing, you need to add the -y flag to both apt and add-apt-repository to avoid them waiting for a user input.

Thanks @marcov . A quick surf says that -y for update is redundant (but accepted as a global option), as update does not prompt for anything ever - but, I went 'full hog' and added -y -qq to all three lines. Let's see how it flies...

Well, I guess that didn't fly - try again...
add-apt-repository: error: no such option: -q
The command "sudo add-apt-repository -y -qq ppa:rmescandon/yq" failed and exited with 2 during

/me starts to wave arms.....
OK, is this telling me that there is no yq package for trusty - probably coz we'd really like to be on xenial, but Travis is not iirc....?  @marcov @jodh-intel
gpg: keyring `/tmp/tmp8gq8qnjl/secring.gpg' created
gpg: keyring `/tmp/tmp8gq8qnjl/pubring.gpg' created
gpg: requesting key CC86BB64 from hkp server keyserver.ubuntu.com
gpg: /tmp/tmp8gq8qnjl/trustdb.gpg: trustdb created
gpg: key CC86BB64: public key "Launchpad PPA for Roberto Mier Escandón " imported
gpg: Total number processed: 1
gpg:               imported: 1  (RSA: 1)
OK
7.59s$ sudo apt update -y -qq
W: http://ppa.launchpad.net/couchdb/stable/ubuntu/dists/trusty/Release.gpg: Signature by key 15866BAFD9BCC4F3C1E0DFC7D69548E1C17EAB57 uses weak digest algorithm (SHA1)
E: The repository 'http://ppa.launchpad.net/rmescandon/yq/ubuntu trusty Release' does not have a Release file.
The command "sudo apt update -y -qq" failed and exited with 100 during .

Yep, I vote we try to bump Travis, presuming a newer version is available.
When we tried this waaaay back on ClearContainers iirc we ran into some issues, but that was like >18months ago probably, and hopefully the world has moved on a bit since then.
What do you think @jodh-intel @chavafg - go directly to a bionic if we can, otherwise try a xenial?
This feels like a new PR to me  which I'm happy to try as a pre-cursor to getting this PR landed on day.

Unluckily we are still stuck in 2014: https://github.com/travis-ci/travis-ci/issues/9460

@grahamwhaley, I'd suggest adding a before_install step that just install yq using a bash script. You can take inspiration from tests/.ci/lib.sh install_yq function.

/me hacks up a script, and fires off travis again - I will not be surprised if this gives me something that still needs to be fixed up.

Golly, travis seems to have passed!
/test

Centos 7.4 CI failed in a strange manner - I'll nudge a rebuild....
Stderr:
Running command '/usr/bin/docker [docker ps -a -f name=6hzLfwdjtT2qHNBymvssMAasZZBmP5 --format {{.Status}}]'
[docker ps -a -f name=6hzLfwdjtT2qHNBymvssMAasZZBmP5 --format {{.Status}}]
Timeout: 60 seconds
Exit Code: 0
Stdout:
Stderr:
•

Summarizing 1 Failure:

[Fail] CPUs and CPU set updating cpus and cpuset of a running container [It] should have the right number of vCPUs
/tmp/jenkins/workspace/kata-containers-runtime-centos-7-4-PR/go/src/github.com/kata-containers/tests/integration/docker/cpu_test.go:432

Ran 208 of 220 Specs in 1316.180 seconds
FAIL! -- 207 Passed | 1 Failed | 0 Pending | 12 Skipped --- FAIL: TestIntegration (1354.25s)
FAIL

@jodh-intel @WeiZhang555 PTAL and merge if you're fine with this :)

LGTM
Thanks!

Since it was probably copied from https://github.com/kata-containers/runtime/pull/762, note that the issue number in the commit is incorrect. Anyone know what it should be?

Golang 1.10 is mentioned as an issue in https://github.com/kata-containers/runtime/issues/148
So, maybe there should have been another Issue to fix just the go 1.10 item - I dunno - better ask the owners of #148 if it is a done deal or not.

source: gharchive/pull-request

authors: WeiZhang555 amshinde grahamwhaley jodh-intel marcov raravena80 sboeuf

repo: kata-containers/runtime

license_type: permissive

license_source: github-api


text:
Comment Snippet for Code

Let's use the following snippet as a standard if we want to add comments to certain sections in our codes.
This is based on jsdocs.
http://usejsdoc.org/index.html

just comment

/**
 * Comment Heading
 * something about this comment in here
 * additional information here
 */

with params

/**
 * Comment Heading
 * something about this comment in here
 * additional information here
 * @param {number} - The # value.
 * @param {string} - The string value.
 */

no params just return

/**
 * Comment Heading
 * something about this comment in here
 * additional information here
 * @return {number} The # value.
 */

with params and return

/**
 * Comment Heading
 * something about this comment in here
 * additional information here
 * @param {number} - The # value.
 * @param {string} - The string value.
 * @return {number} The # value.
 */

Sounds good. Documenting this somewhere would be good.

documentation now adheres to this, closing.

source: gharchive/issue

authors: entonbiba thijstriemstra


text:
Avoid warning on PHP 7

There is no need for the reference parameter here (its arguments are objects like “new QRNumber”), and expressions can’t be passed as references.

thanks a lot!

authors: kazuhikoarase kohler

repo: kazuhikoarase/qrcode-generator

license: MIT


text:
required_unless() doesn't seem to work

Rust Version
1.13.0
Affected Version of clap
v2.18.0
Expected Behavior Summary
accept --list as an only arg
Actual Behavior Summary
error: The following required arguments were not provided:
    --iface <INTERFACE>
    --file <TESTFILE>
    --server <SERVER_IP>

Steps to Reproduce the issue
call with --list
Sample Code or Link to Sample Code
        .arg(Arg::from_usage("-l, --list 'List available interfaces (and stop there)'"))
        .arg(Arg::from_usage("-i, --iface=[INTERFACE] 'Ethernet interface for fetching NTP packets'")
            .required_unless("list"))
        .arg(Arg::from_usage("-f, --file=[TESTFILE] 'Fetch NTP packets from pcap file'")
            .conflicts_with("iface")
            .required_unless("list"))
        .arg(Arg::from_usage("-s, --server=[SERVER_IP] 'NTP server IP address'")
            .required_unless("list"))
        .arg(Arg::from_usage("-p, --port=[SERVER_PORT] 'NTP server port'")
            .default_value("123"))
        .get_matches();

Debug output
*DEBUG:clap: fn=from_usage;
*DEBUG:clap: fn=new; usage="-l, --list \'List available interfaces (and stop there)\'"
*DEBUG:clap: fn=parse;
*DEBUG:clap: iter; pos=0;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=short_or_long;
*DEBUG:clap: fn=short;
*DEBUG:clap: setting short: l
*DEBUG:clap: setting name: l
*DEBUG:clap: iter; pos=1;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=short_or_long;
*DEBUG:clap: fn=long;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: setting name: list
*DEBUG:clap: setting long: list
*DEBUG:clap: iter; pos=10;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=help;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: setting help: List available interfaces (and stop there)
*DEBUG:clap: iter; pos=55;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: vals: None
*DEBUG:clap: fn=from_usage;
*DEBUG:clap: fn=new; usage="-i, --iface=[INTERFACE] \'Ethernet interface for fetching NTP packets\'"
*DEBUG:clap: fn=parse;
*DEBUG:clap: iter; pos=0;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=short_or_long;
*DEBUG:clap: fn=short;
*DEBUG:clap: setting short: i
*DEBUG:clap: setting name: i
*DEBUG:clap: iter; pos=1;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=short_or_long;
*DEBUG:clap: fn=long;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: setting name: iface
*DEBUG:clap: setting long: iface
*DEBUG:clap: iter; pos=11;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=name;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: setting val name: INTERFACE
*DEBUG:clap: iter; pos=22;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=help;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: setting help: Ethernet interface for fetching NTP packets
*DEBUG:clap: iter; pos=69;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: vals: Some({0: "INTERFACE"})
*DEBUG:clap: fn=from_usage;
*DEBUG:clap: fn=new; usage="-f, --file=[TESTFILE] \'Fetch NTP packets from pcap file\'"
*DEBUG:clap: fn=parse;
*DEBUG:clap: iter; pos=0;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=short_or_long;
*DEBUG:clap: fn=short;
*DEBUG:clap: setting short: f
*DEBUG:clap: setting name: f
*DEBUG:clap: iter; pos=1;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=short_or_long;
*DEBUG:clap: fn=long;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: setting name: file
*DEBUG:clap: setting long: file
*DEBUG:clap: iter; pos=10;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=name;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: setting val name: TESTFILE
*DEBUG:clap: iter; pos=20;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=help;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: setting help: Fetch NTP packets from pcap file
*DEBUG:clap: iter; pos=56;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: vals: Some({0: "TESTFILE"})
*DEBUG:clap: fn=from_usage;
*DEBUG:clap: fn=new; usage="-s, --server=[SERVER_IP] \'NTP server IP address\'"
*DEBUG:clap: fn=parse;
*DEBUG:clap: iter; pos=0;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=short_or_long;
*DEBUG:clap: fn=short;
*DEBUG:clap: setting short: s
*DEBUG:clap: setting name: s
*DEBUG:clap: iter; pos=1;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=short_or_long;
*DEBUG:clap: fn=long;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: setting name: server
*DEBUG:clap: setting long: server
*DEBUG:clap: iter; pos=12;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=name;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: setting val name: SERVER_IP
*DEBUG:clap: iter; pos=23;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=help;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: setting help: NTP server IP address
*DEBUG:clap: iter; pos=48;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: vals: Some({0: "SERVER_IP"})
*DEBUG:clap: fn=from_usage;
*DEBUG:clap: fn=new; usage="-p, --port=[SERVER_PORT] \'NTP server port\'"
*DEBUG:clap: fn=parse;
*DEBUG:clap: iter; pos=0;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=short_or_long;
*DEBUG:clap: fn=short;
*DEBUG:clap: setting short: p
*DEBUG:clap: setting name: p
*DEBUG:clap: iter; pos=1;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=short_or_long;
*DEBUG:clap: fn=long;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: setting name: port
*DEBUG:clap: setting long: port
*DEBUG:clap: iter; pos=10;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=name;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: setting val name: SERVER_PORT
*DEBUG:clap: iter; pos=23;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: fn=help;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: setting help: NTP server port
*DEBUG:clap: iter; pos=42;
*DEBUG:clap: fn=stop_at;
*DEBUG:clap: vals: Some({0: "SERVER_PORT"})
*DEBUG:clap: fn=get_matches_with;
*DEBUG:clap: fn=create_help_and_version;
*DEBUG:clap: Building --help
*DEBUG:clap: Building --version
*DEBUG:clap: Begin parsing '"--list"' ([45, 45, 108, 105, 115, 116])
*DEBUG:clap: Starts new arg...Maybe
*DEBUG:clap: fn=possible_subcommand
*DEBUG:clap: fn=parse_long_arg;
*DEBUG:clap: Does it contain '='...No
*DEBUG:clap: Found valid flag '--list'
*DEBUG:clap: Checking if --list is help or version...Neither
*DEBUG:clap: fn=parse_flag;
*DEBUG:clap: macro=validate_multiples!;
*DEBUG:clap: fn=groups_for_arg; name=list
*DEBUG:clap: No groups defined
*DEBUG:clap: macro=arg_post_processing!;
*DEBUG:clap: Is '--list' in overrides...No
*DEBUG:clap: Does '--list' have overrides...No
*DEBUG:clap: Does '--list' have conflicts...No
*DEBUG:clap: Does '--list' have requirements...No
*DEBUG:clap: macro=_handle_group_reqs!;
*DEBUG:clap: fn=add_val_to_arg;
*DEBUG:clap: adding val: "123"
*DEBUG:clap: fn=groups_for_arg; name=port
*DEBUG:clap: No groups defined
*DEBUG:clap: fn=validate_value; val="123"
*DEBUG:clap: macro=arg_post_processing!;
*DEBUG:clap: fn=fmt
*DEBUG:clap: Is '--port <SERVER_PORT>' in overrides...No
*DEBUG:clap: fn=fmt
*DEBUG:clap: Does '--port <SERVER_PORT>' have overrides...No
*DEBUG:clap: fn=fmt
*DEBUG:clap: Does '--port <SERVER_PORT>' have conflicts...No
*DEBUG:clap: fn=fmt
*DEBUG:clap: Does '--port <SERVER_PORT>' have requirements...No
*DEBUG:clap: macro=_handle_group_reqs!;
*DEBUG:clap: fn=validate_blacklist;blacklist=[]
*DEBUG:clap: fn=validate_num_args;
*DEBUG:clap: fn=_validate_num_vals;
*DEBUG:clap: fn=create_usage;
*DEBUG:clap: fn=create_usage_no_title;
*DEBUG:clap: args_in_groups=[]
*DEBUG:clap: fn=fmt
*DEBUG:clap: fn=fmt
*DEBUG:clap: fn=fmt
*DEBUG:clap: fn=needs_flags_tag;
*DEBUG:clap: iter;f=list;
*DEBUG:clap: fn=groups_for_arg; name=list
*DEBUG:clap: No groups defined
*DEBUG:clap: Flag not required...(returning true)
*DEBUG:clap: args_in_groups=[]
*DEBUG:clap: fn=fmt
*DEBUG:clap: fn=fmt
*DEBUG:clap: fn=fmt
*DEBUG:clap: fn=create_usage;
*DEBUG:clap: fn=create_usage_no_title;
*DEBUG:clap: fn=smart_usage;
*DEBUG:clap: args_in_groups=[]
*DEBUG:clap: fn=fmt
*DEBUG:clap: fn=fmt
*DEBUG:clap: fn=fmt
*DEBUG:clap: fn=fmt
*DEBUG:clap: fn=color;
*DEBUG:clap: Color setting...Auto
*DEBUG:clap: fn=error;
*DEBUG:clap: fn=is_a_tty;
*DEBUG:clap: Use stderr...true
*DEBUG:clap: fn=good;
*DEBUG:clap: fn=is_a_tty;
*DEBUG:clap: Use stderr...true

Thanks for posting this! I should have fixed quickly :wink:

I've got this fixed in a local branch, should have it uploaded and the PR through shortly. Then I'll upload the new version to crates.io and post back here.

Great thanks !

v2.19.0 is out on crates.io

authors: bestouff kbknapp

repo: kbknapp/clap-rs

license: mit

license_source: bigquery


text:
Radio's checked attribute set, but not fully applied

Hello there,
first of all, thanks for your work ! i started a new project with your lib instead of using vuejs :)
I encounter a strange bug, or maybe i do something wrong (?) : https://codepen.io/anon/pen/wPXKrR
When you navigate the first run is perfect, but if you go to the page 1 from the last page multiple time, checked becomes wrong, in the debugger i see a 'checked' attribute but the radio is definitely not checked.
It sounds like #25
any idea ?

It looks like the code has a number of problems.

The data overrides the props—you don't need to declare the props in data.
You pass data to the component as a string rather than its' appropriate type (boolean)
You have multiple child elements in the component template.

I don't think it's a problem with Moon, but I'll look into it.

Thanks for your reply, i tried a lots a things, and it apear that is not the proper way to hanlde my use case :/
Even if i solve your checklist, this doesent work as expected.
I think there is an issue about dynamics component generation but my moonjs knowleges dont able me to go further.
Regards

Hello,
An other issue with radio inputs : probably related to the one above
https://codepen.io/anon/pen/NXxEQq
It works at first run, but stop working correctly after the second cycle.
An other question : the "typeof" seems to be wrong ?

This was fixed in v1, using m-literal with a dom modifier. The typeof is not wrong, since you passed the value as a string (using template braces {{}} passes as a string within an attribute). Passing through m-literal will pass it as a number.
The fix can be seen here (using the current development version of v1).

Thanks, You are, powerful.
Regards

No problem.

authors: Marc-oO kbrsh

repo: kbrsh/moon


text:
Allow resources in data source kustomization_overlay from another terraform resource

would be good to allow resources in data source kustomization_overlay from another terraform resource block. maybe using resources_raw ?
use case :

we still didn't have the kustomization directory, and we would like to generate it on the fly,

for example :

we want to install flux using terraform flux provider. we know that data.flux_install.main.content have yaml that required to be installed, but we want to patch it before install, then we use kustomization_overlay to generate the patch, with resources inputed from data.flux_install.main.content

My opinion on this topic hasn't changed, I don't think it makes sense for this provider to support more than Kustomize does itself. https://github.com/kbst/terraform-provider-kustomization/issues/114

authors: kholisrag pst

repo: kbst/terraform-provider-kustomization


text:
added responds to keyboard var

Introduce a variable respondsToKeyboard that determines whether Floaty responds to keyboard notifications

Thank you :)

authors: kciter kochrt

repo: kciter/Floaty


text:
syncer: improve mutation + admission

syncer: move secret renaming to mutation
Signed-off-by: Steve Kuznetsov skuznets@redhat.com

syncer: add an interface for mutation
Signed-off-by: Steve Kuznetsov skuznets@redhat.com

syncer: move configmap mutation into mutators
Signed-off-by: Steve Kuznetsov skuznets@redhat.com

syncer: move serviceaccount mutation to mutators
Signed-off-by: Steve Kuznetsov skuznets@redhat.com

syncer: add a concept for admission
Signed-off-by: Steve Kuznetsov skuznets@redhat.com

/assign @jmprusi @ncdc

This PR IMO over-abstracts both the mutator interface and the admission as a separate concept. It adds complexity that won't help us.

/close

::no-prow::

authors: ncdc stevekuznetsov sttts

repo: kcp-dev/kcp


text:
Event filtering seems incorrect

Hey Keith:
Been using the library and I'm basically trying to poll my nodes that control loads to monitor their energy consumption and write the contents to a lambda database.
I initially created an XML file that had the node names and the variables I wanted to log.  I read these into program then I thought I would create a polling loop to log them.  What I remembered is that you have the way cool event system that can create callbacks.  So I've been playing around with events.
If I setup the event handler to log callbacks for "" or "devices." or even "devices.." these all work.  I can watch the event loop and see the event status.  So I thought I would get clever and just setup a callback for "devices..Watts." and I can't get that to trigger.  I've tried ".Watts.*" and just about every other combo I can think of.
Is there something I am missing in configuring the callbacks?  I have been plugging away for a while without much luck.

Sounds good.  Thanks for the continued effort and the feedback on using the
threading wait call.  My main thread is currently not doing anything but as
I start to expand the project I suspect that I will want to do useful work
there.  When I get home this evening I'll see if my logger is still running
and will attempt to use the other device callback masks to see if they
trigger.
On Wed, Jan 31, 2018 at 11:57 AM, Kevin Schlosser notifications@github.com
wrote:

and the reason to not use the sleep is because of how it blocks. I never
did a whole lot of poking about into it but the use of
threading.Event().wait() is the better way. I am not sure how the inner
mechanics work. Its something I have read over and over again. I guess if
it works then keep it.. But it is a thread so you can use it for something
other then sleeping..
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kdschlosser/MiCasaVerde_Vera/issues/2#issuecomment-362016689,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AhSL44-M666F_J1LDr5gfNNzFBgpgOHYks5tQKmlgaJpZM4RvlpC
.

OK sorry this took so long. it was a very large undertaking to get this all right, much bigger then I thought it would be.
But... You will be happy to know that I found the issue with the events. So those are now good to go. And everything has threading locks. so different threads cannot interact with the same object at the same time.
The unfortunate thing is because of how the data is stored if a device is being updated it is going to cause another thread to wait until the update for that device is done.. this process is very quick but that was the only way to go about it.

I am thinking I may implement an update routine that can be called by the user. this way if you simply want to create a single thread that will do an update and then iterate through specific devices you can.
This will allow you to create a single thread that would handle the passing of the watts to your database and any changes there may be would get handled with no other code needed. This would be the ost simplistic approach.
If you plan on using this library for other things besides the watts thing then you would use the polling loop that is built in and have to create your own thread to keep the flow of data rolling into your DB.

Hi Kevin:
Great news.  I will download the code this evening.  I think that the thread blocking won't be a major issue.  If the database write collides with the device update and it blocks for a bit it should be OK.   I am currently updating the database at a 1 minute interval and polling the vera on a 30 second interval so there should be plenty of time for the database thread to complete before the next vera polling event.
I have my basic application running but started refactoring into cleaner classes.  I do think that there was an issue with attributes still crashing with the last code drop but I will try the latest with the thread enhancements.  I had the vera loop stop running after a few days but since I put in an exception handler  and I wasn't logging my output to a file I can't confirm.  I had it crash again last night but haven't reviewed my log files to confirm my hypothesis.
I changed my write to database thread to write the vera wattage plus measurements from my whole house energy monitor and my pool pump controller so I have a more complete view of my overall consumption.  In the write to database thread I poll the whole house monitor and pool controller to get the measurements to write.  If I add a poll for the vera devices as you proposed it would put all of my data measurements and the  write to database in that one thread.
I will be adding other systems and also logging temperature, dim level, etc. from the Vera moving forward.  Since I am planning to log more that just wattage I had to rewrite my data structures to be generic and handle other parameters.
Below is an example of how I am logging the data and visualizing it in grafana.  There are 2 example images:

showing only the vera devices

showing the vera devices, my pool pump, and other (my whole house energy monitor minus the total vera consumption minus the pool pump consumption)

I'll try out the updated library and give you some feedback in the next few days.
Thanks again for the great support and quick turnaround.

I'm about to download and install your latest changes.
I did check my log file and I got an exception but not the attribute exception that I expected.  Looks like the connection to the vera got reset which threw an exception.  If I wanted to handle that exception would I need to create a new vera instance and reload the device list?
I'll dig through the code and try to figure it out as well but thought you might have an idea.
Thanks.

Forgot to include the traceback.  Looks like i got a couple of exceptions.  Including one to my home energy monitor.  The first few look like they are vera related though and maybe addressed with the most recent changes.
The connection aborted was tied to my neurio home energy monitor not the vera.
Exception in thread Thread-847: Traceback (most recent call last):   File "/usr/lib/python2.7/threading.py", line 801, in __bootstrap_inner     self.run()   File "/usr/lib/python2.7/threading.py", line 754, in run     self.__target(*self.__args, **self.__kwargs)   File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/__init__.py", line 577, in _data_handler     self.__update(self.devices, 'devices', data)   File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/__init__.py", line 465, in __update     obj.update_node(data.pop(key, None), True)   File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/devices.py", line 119, in update_node     found_device.update_node(device, full)   File "/home/sabolcik/.MiCasaVerde_Vera/core/devices/binary_light_1.py", line 464, in update_node     state['service']   File "/home/sabolcik/.MiCasaVerde_Vera/core/devices/binary_light_1.py", line 452, in check_value     variable.replace('.', '')   File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/event.py", line 51, in notify     event_handler.event_object = event_object   File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/event.py", line 79, in event_object     self.__callback(self)   File "<stdin>", line 125, in device_callback   File "/usr/local/lib/python2.7/dist-packages/neurio/__init__.py", line 412, in get_local_current_sample     r = requests.get(url, headers=headers)   File "/usr/local/lib/python2.7/dist-packages/requests/api.py", line 72, in get     return request('get', url, params=params, **kwargs)   File "/usr/local/lib/python2.7/dist-packages/requests/api.py", line 58, in request     return session.request(method=method, url=url, **kwargs)   File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 508, in request     resp = self.send(prep, **send_kwargs)   File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 618, in send     r = adapter.send(request, **kwargs)   File "/usr/local/lib/python2.7/dist-packages/requests/adapters.py", line 490, in send     raise ConnectionError(err, request=request) ConnectionError: ('Connection aborted.', error(104, 'Connection reset by peer'))

Just for the sake of completeness, I forgot that I needed to change a few things in the library to get it to run last time.  When I downloaded the latest code I got the same errors.  I'll document them here and the default arguments that I used to make them go away:
Traceback (most recent call last):
  File "<stdin>", line 265, in <module>
  File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/__init__.py", line 332, in connect
    return __Vera(ip_address)
  File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/__init__.py", line 293, in __init__
    _VeraBase.__init__(self, ip_address)
  File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/__init__.py", line 422, in __init__
    self.__update(self.scenes, 'scenes', data)
  File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/__init__.py", line 465, in __update
    obj.update_node(data.pop(key, None), True)
  File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/scenes.py", line 113, in update_node
    found_scene = Scene(self, **scene)
  File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/scenes.py", line 172, in __init__
    self.timers = Timers(self, timers)
  File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/scenes.py", line 909, in __init__
    self.timers = list(Timer(self, scene, **timer) for timer in timers)
  File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/scenes.py", line 909, in <genexpr>
    self.timers = list(Timer(self, scene, **timer) for timer in timers)
TypeError: __init__() got an unexpected keyword argument 'interval'
```
Previously I had simply changed the constructor to add the parameter interval with a value of 0
`class Timer(object):

    def __init__(
        self,
        parent,
        scene,
        id,
        name='',
        type='',
        enabled=1,
        days_of_week='',
        time='',
        next_run=0,
        last_run=0,
        **interval = 0**
    ):`

Similar issue with the Trigger object

`Traceback (most recent call last):
  File "<stdin>", line 265, in <module>
  File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/__init__.py", line 332, in connect
    return __Vera(ip_address)
  File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/__init__.py", line 293, in __init__
    _VeraBase.__init__(self, ip_address)
  File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/__init__.py", line 422, in __init__
    self.__update(self.scenes, 'scenes', data)
  File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/__init__.py", line 465, in __update
    obj.update_node(data.pop(key, None), True)
  File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/scenes.py", line 113, in update_node
    found_scene = Scene(self, **scene)
  File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/scenes.py", line 171, in __init__
    self.triggers = Triggers(self, triggers)
  File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/scenes.py", line 677, in __init__
    Trigger(self, scene, **trigger) for trigger in triggers
  File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/scenes.py", line 677, in <genexpr>
    Trigger(self, scene, **trigger) for trigger in triggers
TypeError: __init__() got an unexpected keyword argument 'users'
`
Updated constructor:
`class Trigger(object):
    def __init__(
        self,
        parent,
        scene,
        name,
        device=None,
        template=None,
        enabled=1,
        lua='',
        encoded_lua=0,
        arguments=[],
        last_run=0,
        last_eval=0,
        users = 0
    ):`
With those changes I got the code running again.  There may be a more elegant way to fix those missing parameters but wanted you to be aware.

OK I updated the library to add the fixes for those items.
I also added threaded events. what this means is that the callback will get called inside of it's own thread for each event that is generated. I added this feature for 2 reasons. first is because I added the locks. the second is so that if a user has long running code inside of the event callback this will not stall the update process from continuing.
I also did quite a bit of code cleanup.
And I incremented the library version so when you run this it will automatically generate new files.
I do not know if you know this or not. But you can check out the files if you like. if you are running on a Windows system the files are located in the %appdata%/MiCasaVerde_Vera folder. and on Linux/Crapintosh in the ~/.MiCasaVerde_Vera folder.
If you change any of the generated files or if you have data corruption on your HDD and the files get damaged, the library will know and it will rebuild them.

and as far as that exception goes. that is not a problem with the library. that is an issue with a connection to something else. and you are running the code that makes that connection from inside of the update thread the vera library is running. My suggestion is to do a base exception catching as soon as the callback gets called. This way it won't stall the library from running
Exception in thread Thread-847:
Traceback (most recent call last):
File "/usr/lib/python2.7/threading.py", line 801, in __bootstrap_inner self.run()
File "/usr/lib/python2.7/threading.py", line 754, in run self.__target(*self.__args, **self.__kwargs)
File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/init.py", line 577, in _data_handler self.__update(self.devices, 'devices', data)
File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/init.py", line 465, in __update obj.update_node(data.pop(key, None), True)
File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/devices.py", line 119, in update_node found_device.update_node(device, full)
File "/home/sabolcik/.MiCasaVerde_Vera/core/devices/binary_light_1.py", line 464, in update_node state['service']
File "/home/sabolcik/.MiCasaVerde_Vera/core/devices/binary_light_1.py", line 452, in check_value variable.replace('.', '')
File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/event.py", line 51, in notify event_handler.event_object = event_object
File "/home/sabolcik/.local/lib/python2.7/site-packages/micasaverde_vera/event.py", line 79, in event_object self.__callback(self)
File "", line 125, in device_callback File "/usr/local/lib/python2.7/dist-packages/neurio/init.py", line 412, in get_local_current_sample r = requests.get(url, headers=headers)
File "/usr/local/lib/python2.7/dist-packages/requests/api.py", line 72, in get return request('get', url, params=params, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/requests/api.py", line 58, in request return session.request(method=method, url=url, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 508, in request resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 618, in send r = adapter.send(request, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/requests/adapters.py", line 490, in send raise ConnectionError(err, request=request)
ConnectionError: ('Connection aborted.', error(104, 'Connection reset by peer'))
I organized the traceback so you can read it easier. But each step through the traceback is a stack trace. if you follow the stacktrace down the list you will see that the error happens inside of the callback from a requests connection error.
def callback(event):
    try:
        # do your code here
        pass
    except Exception:
        import traceback
        traceback.print_exc()

this will print out the traceback but not cause your program to stall if you want to output it to a file
def callback(event):
    try:
        # do your code here
        pass
    except Exception:
        import traceback
        with open(r'Some\Error\File\Location\error.log', 'a') as f:
            f.write('\n' + traceback.format_exc() + '\n')

what i mean by check out the files. is that when you run connect to the vera for the first time is this library will read information from the vera from a variety of ways. I have developed a way to use this information to write code dynamically. The benefit from this is the library is able to work with almost any plugin/device for the vera.. current and future, because the library is technically speaking tailored to your vera and what you have installed on it.
those generated python files are located in the directory i provided above. so if you want to know what variables are available. or what functions can be called on a device that is where you can do some poking about.

I wanted to know how you are making out with this and is everything working ok for ya?

So far so good.
I have added some other data sources to my logger (Nest thermostat, whole house energy monitor) and those devices (which are not under the control of the Vera) have crashed a few times preventing me from a long continuous run with the Vera.  The Vera library itself seems solid.  I haven't had any crashes with it.
At one point I thought there was a problem with the library because it was reporting energy usage for a device I was certain was turned off.  But when I checked that device it turns out it was actually on.
Thanks for the quick turnaround on the library and the very useful tool.  I'll close out this issue.

no worries m8. you should be able to begin the process of removing scenes form the Vera to lighten the load on it. it is much easier to code them up using this library and some python scripting. And you can use the example I gave that is a kind of a console for it. I may build in some kind of a GUI into it.
Once i figure out the device creation ends of things it will really open the doors up. because any device API that is written for python can end up being a device on the vera.

authors: kdschlosser sabolcik


text:
[feature] 회원정보 찾기 페이지 마크업, 유효성 검사

작업 상세 내용

[ ] 이메일 인증 내용
[ ] 인증번호 입력
[ ] 비밀번호 재설정 섹션
[ ] 완료버튼

참고 사항

이메일 인증번호 유효시간 타이머는 후 순위

지정 부탁드려요

authors: hyerani yujiseok


text:
Make image tasks pausable

Add -pause method to DFImageTask
DFImageManager should be able to put tasks back into suspended state
Add optional pause to the operation, returned by DFImageFetching protocol. Waiting for #29
Make NSProgress pausable
Support pause in image fetchers

This feature is available in Nuke, DFImageManager is deprecated.

repo: kean/DFImageManager


text:
sometimes empty image

hello
i am in the process of porting to DFImageManager from SDWebImage :) - one of the main reason, the GIF support.
right now i have a few collectionviews (inside uitabbarcontroller) each with a bunch of cells (mtl.image is  DFImageView).
doing something like
[mtl.image prepareForReuse];
[mtl.image setAllowsAnimations:NO];
[mtl.image setImageWithResource:[NSURL URLWithString:obj.imageUrl]];

loading the same datasource, into two tabs (collection views) - sometimes does not load images in one collectionview, and in the other it does (mainly on GIFS), i have no clue where to start debugging, any hints?
regards
helmut

Thanks, I'll take a look into it. My first guess is that it is somehow related to the a way FLAnimatedImageView renders images. There is a lot of stuff going on in FLAnimatedImage class and you are trying to display the same FLAnimatedImage instance in two separate image views.

Hi @hjanuschka, did you figure out what was the issue?

i actually removed dfimagemanager, and stayed with SDWebimage (Atleast for now)
had following problems beside the problem with flanimated image
*) image loading didn't seem to be canceled, wich resulted in a really slow loading behavior on fast scrolling
the FLAnimated image problem, seems to be, that we cannot change the flanimated image i came up with the following solution:
using: https://github.com/jagbolanos/FLAnimatedImage_AFNetworking (for async loading of flanimatedimages)
inside my cellforitematindexpath
[[mtl.image subviews]    makeObjectsPerformSelector:@selector(removeFromSuperview)];
if([obj.imageUrl_Orig hyp_containsString:@"gif"]) {
        [mtl.image setImage:nil];

        FLAnimatedImageView * faiv = [[FLAnimatedImageView alloc] initWithFrame:CGRectMake(0,0, mtl.image.frame.size.width, mtl.image.frame.size.height)];
        [mtl.image setAutoresizesSubviews:YES];
        faiv.contentMode=mtl.image.contentMode;

        faiv.autoresizingMask=UIViewAutoresizingFlexibleWidth|UIViewAutoresizingFlexibleHeight;
        faiv.clipsToBounds=YES;
        __weak __typeof(faiv)weakfaiv = faiv;
        NSURLRequest * ur=[[NSURLRequest alloc] initWithURL:[NSURL URLWithString:obj.imageUrl]];

        [faiv setAnimatedImageWithURLRequest:ur
                            placeholderImage:nil
                                     success:^(NSURLRequest *request, NSHTTPURLResponse *response, FLAnimatedImage *animatedImage){
                                         weakfaiv.animatedImage = animatedImage;

                                         //do more things
                                     } failure:^(NSURLRequest *request, NSHTTPURLResponse *response, NSError *error){
                                         //handle failure
                                     }];

        [mtl.image addSubview: faiv];

    } else {

        [mtl.image sd_setImageWithURL:[NSURL URLWithString:  obj.imageUrl] placeholderImage:nil options:SDWebImageCacheMemoryOnly completed:^(UIImage *image, NSError *error, SDImageCacheType cacheType, NSURL *imageURL) {
            if (image && cacheType == SDImageCacheTypeNone)
            {

            }

        }];
    }

Hi @hjanuschka

*) image loading didn't seem to be canceled, wich resulted in a really slow loading behavior on fast scrolling

Do you have a way to reproduce this, preferably on version 0.7.0? Please show how you configure DFImageManager (or which subspecs you installed if you don't change sharedManager) and the way you start and cancel image requests.

using: https://github.com/jagbolanos/FLAnimatedImage_AFNetworking (for async loading of flanimatedimages)

Thanks, I'll take a look at this.

I've done some extensive testing of GIFs and it actually works really well https://youtu.be/YctDm3L4uXw. Tested with both NSURLSession and AFNetworking subspecs from develop branch.
I've also made some minor changes in _DFURLFetcherTaskQueue that would allow for instant NSURLSessionTask cancellation. Previous releases were resuming and cancelling tasks with a micro-delay to prevent NSURLSession trashing during extremely fast scrolling. This change only affects DFImageManager/NSURLSession subspec.
New release (0.7.1) would be available shortly, your feedback would be much appreciated.

I actually did find and fix an issue with cancellation of fetch operations (see #71). It's a rare issue, but it might happen quiet frequently depending on the way you start and cancel image requests.

Regarding cancellation. It's also a matter of how you implement UITableView/UICollectionView delegate:
- (void)collectionView:(UICollectionView *)collectionView willDisplayCell:(UICollectionViewCell *)cell forItemAtIndexPath:(NSIndexPath *)indexPath {
    DFImageView *imageView = (id)[cell viewWithTag:15];
    if (!imageView) {
        imageView = [[DFImageView alloc] initWithFrame:cell.bounds];
        imageView.autoresizingMask = UIViewAutoresizingFlexibleWidth | UIViewAutoresizingFlexibleHeight;
        imageView.tag = 15;
        [cell addSubview:imageView];
    }

    NSURL *URL = _photos[indexPath.row];
    [imageView setImageWithResource:URL targetSize:[self _imageTargetSize] contentMode:DFImageContentModeAspectFill options:nil];
}

- (void)collectionView:(UICollectionView *)collectionView didEndDisplayingCell:(UICollectionViewCell *)cell forItemAtIndexPath:(NSIndexPath *)indexPath {
    DFImageView *imageView = (id)[cell viewWithTag:15];
    [imageView prepareForReuse];
}

Bad thing is here

DFImageView *imageView = (id)[cell viewWithTag:15];

You should check type of "imageView"

if (!imageView || ![imageView isKindOfClass:[DFImageView class]])

instead of

if (!imageView)

because imageView sometimes becomes UICollectionViewCell-like class

authors: bmxius hjanuschka kean


text:
Update documentation 1

updated Home, Overview, Tutorial
added Onboarding
deleted writing to gooddata

Fixed nav, rebased and fixed permalink.

authors: hhanova ujovlado

repo: keboola/connection-docs


text:
adds PanelWithDetails story

Fixes #123

este tam prehodim logiku, lebo takto to matie:
labelOpen: 'Hide details'

authors: janmichek ujovlado

repo: keboola/indigo-ui


text:
Also add support for SSM secrets in AWS TriggerAuthentication

Proposal
Besides accessing secrets from secretsmanager, similar to secrets-store-csi-driver-provider-aws, we should be able to fetch secure secrets from SSM. This will allow more flexibility for implementation
Use-Case
This will allow users already using SSM to not have to copy their secrets
Is this a feature you are interested in implementing yourself?
No
Anything else?
No response

Hello
I'm not sure if I get your point totally. Does SSM mean AWS Systems Manager? Assuming that yes, I think that we can extend the current implementation of the AWS Secret Manager and support calling to the new service (with another parameter to declare it).
WDYT @kedacore/keda-contributors ?

Yeah, I like this

authors: JorTurFer sneljo1 zroubalik

repo: kedacore/keda


text:
Add DynamoDB Scaler

Signed-off-by: Samuele Resca samuele.resca@gmail.com

Provide a description of what has been changed
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[X] Tests have been added
[X] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[X] A PR is opened to update the documentation on https://github.com/kedacore/keda-docs/pull/680
[ ] Changelog has been updated

Fixes #2482

Thanks for your PR! Would you mind opening and linking a PR for the docs please?

Looking good, just a few minor nits on formatting.
Also a big ask is e2e testing. Do you think you can implement e2e tests for this scaler, similar to this https://github.com/kedacore/keda/tree/main/tests? We should have KEDA related AWS account soon CC @tomkerkhove

I'm happy to implement the e2e tests for this. Should I do it in this PR?

Looking good, just a few minor nits on formatting.
Also a big ask is e2e testing. Do you think you can implement e2e tests for this scaler, similar to this https://github.com/kedacore/keda/tree/main/tests? We should have KEDA related AWS account soon CC @tomkerkhove

I'm happy to implement the e2e tests for this. Should I do it in this PR?

Yes please :)

Looking good, just a few minor nits on formatting.
Also a big ask is e2e testing. Do you think you can implement e2e tests for this scaler, similar to this https://github.com/kedacore/keda/tree/main/tests? We should have KEDA related AWS account soon CC @tomkerkhove

I'm happy to implement the e2e tests for this. Should I do it in this PR?

Yes please :)

Regarding the e2e testing. I'm considering spanning up an instance using the following image for testing purposes: https://hub.docker.com/r/amazon/dynamodb-local. Until we don't have an AWS account. It should mimic the dynamodb on AWS, except for the fact that multiple regions are not supported

@zroubalik  I updated the PR with the tests.  I executed the E2E test locally with my personal subscription and looks good to me. Let me know when you have the account available.

@zroubalik I updated the PR with the tests. I executed the E2E test locally with my personal subscription and looks good to me. Let me know if I can do something else.

@JorTurFer @tomkerkhove how should we approach this? Do we know if there's any progress on the AWS account from CNCF?

@zroubalik I updated the PR with the tests. I executed the E2E test locally with my personal subscription and looks good to me. Let me know if I can do something else.

@JorTurFer @tomkerkhove how should we approach this? Do we know if there's any progress on the AWS account from CNCF?

If the e2e test is running, I'd keep it commented and I'd enable it when the account is ready. We already have other scalers without any e2e test, so I wouldn't block this PR.
Other option is using DynamoDB Local but I'm not sure if that development version could be enough for our e2e test case. Even if the local version is enough, I'd prefer using the real version, but this is an option

@zroubalik I updated the PR with the tests. I executed the E2E test locally with my personal subscription and looks good to me. Let me know if I can do something else.

@JorTurFer @tomkerkhove how should we approach this? Do we know if there's any progress on the AWS account from CNCF?

If the e2e test is running, I'd keep it commented and I'd enable it when the account is ready. We already have other scalers without any e2e test, so I wouldn't block this PR. Other option is using DynamoDB Local but I'm not sure if that development version could be enough for our e2e test case. Even if the local version is enough, I'd prefer using the real version, but this is an option

Yeah, let's disable e2e tests for now and enable once we have the account.

No AWS account yet indeed. I will check

authors: JorTurFer samueleresca tomkerkhove zroubalik


text:
i am in love

please help. i am in love with this library, yet i am unable to marry it

This is a complicated and depressing area of the legal code.  Software is largely treated as sub-human chattel.  Perhaps try petitioning equal rights for general AI, and then work down to utilities?

repo: keenerd/jshon


text:
Investigate rejected transactions

Running a benchmark on a totally new Madara instance results in some rejected transactions on both ERC20 and ERC721 benchmarks. We should investigate why does transactions are being rejected


text:
feat: add ABI to contract class ✨

Feature Request
Describe the Feature Request

The ABI should be read from the compiled contract when declaring a new contract class.
Describe Preferred Solution

Should be done in a similar fashion as program and entry_points_by_type.
Once implemented // TODO: Uncomment once we have ABI support can be uncommented.
Some thinking should be put into the "why" of this issue, what should the ABI be used for.
For example :

Sequencer contract verification
Explorer

Hey @EvolveArt glad to help on this one. just not sure on where do we find the "compiled contract".
Can you give some advices? Thank you !

Hey @EvolveArt glad to help on this one. just not sure on where do we find the "compiled contract". Can you give some advices? Thank you !

Thank you for your help! Compiled contracts can be found under the ressources folder for now as .json files.
You can see that we load them in tests when using get_contract_class(include_bytes!({path}))

authors: Cheelax EvolveArt

repo: keep-starknet-strange/madara


text:
[Frontend] Improve dropdown component

Improve the current dropdown component to automatically close when a user clicks anywhere else on the screen.
Note: Ensure the implementation does not interfere with other UI elements.

Please can I work on this

Hi @0xChqrles I'd love to work on this

Hello @0xChqrles My name is Ugo, a full stack dev currently learning Cairo. i would love to contribute to this effort. Please can i be assigned this task? will get it done in no time

i would love to work on this, will get it done ASAP!

@0xChqrles Since, dropdown was added by you in my last PR, I can quickly implement this.
Can you please assign it to me ?

Can I work on this?

@0xChqrles Can we please close this one or something is still pending ?

authors: 0xChqrles JoshdfG Joy-Adah Signor1 Ugo-X kateberryd mananbordia


text:
Disk shader does not work in HDRP (2019.1.0f2)

Hi Keijiro,
I tried the disk shader on HDRP in Unity 2019.1.0f2, I get the magenta shader of death (with no errors). Any idea why?
I have a volumetric capture of an actor as a sequence of vertex and colors lists (each frame has a list of vertices and a list of colors). Do you have a suggestion on how to render them apart from a geometry shader?

Basically there is no compatibility in shaders between the legacy render pipeline and HDRP, so you can't simply use the PCX builtin shaders.
It might sound like a little bit strange idea but the Visual Effect Graph is the handiest way to visualize a point cloud on HDRP.
https://twitter.com/_kzr/status/1101395833541414912
Unfortunately, I have no time for writing a tutorial about the usage. Please check VFX Graph tutorials/articles. You can find some of them by googling.

I'm closing this issue now. Please feel free to reopen for further questions.

authors: keijiro sergiobd

repo: keijiro/Pcx

license: Unlicense


text:
Add support for "types" and "typings" from TypeScript

see https://www.typescriptlang.org/docs/handbook/declaration-files/publishing.html
/cc @keithamus

👍 will be released as 1.5.0 shortly

thanks :)

repo: keithamus/sort-package-json


text:
Explore text() function to check if it can be improved for more websites - to check

For eg https://www.worldometers.info/coronavirus/
The current logic is return document.body.innerText || document.body.textContent
But document.documentElement.innerText will grab everything. To review how it can retrieved better.

dump `text()` to result.txt

dom return document.body.innerText
dump `dom_result` to body.txt

dom return document.documentElement.innerText
dump `dom_result` to documentElement.txt

All above yields the same results --> result.txt

documentElement includes stuffs like header which shouldn't be included for text extraction.
Will need more examples why document.documentElement.innerText works better than document.body.innerText

repo: kelaberetiv/TagUI


text:
Improve synchronous execution of JS code and TagUI steps, 2. Support complex nested for loops with if, else if, else

In some scenarios, the order of execution is not fully synchronous. The short summary is JavaScript is asynchronous by nature while CasperJS tries to make it synchronous -> but there is still gap in TagUI for some cases to convert human language into fully synchronous CasperJS JavaScript code.
Following shows an example and this issue has another example. In that previous issue, a potential fix was abandoned as it compromises backward compatibility for existing scripts. A big change in many parts of tagui_parse.php is needed to implement this fix.
There could be slight backward compatibility issues but this change is necessary to make automation scripts more deterministic and drive automation actions and behavior more consistently. Below the example is a list of items to evaluate and check before committing to master.
Replication example:
a=1
echo "checkpoint 1. a = " + a

if (a>0)
{
    a = 20
    echo "checkpoint 2. a = 20"
}

echo "checkpoint 3. a = " + a

Output is not synchronous:
checkpoint 1. a = 1
checkpoint 3. a = 1
checkpoint 2. a = 20

List of items to evaluate on local bleeding edge version before committing to master -

[x] works for sample flows 1 to 9 (except 3 & 4)
[x] sample flow 3 use of var
[x] sample flow 4 while loop handling
[x] works for normal TagUI steps
[x] works for popup step
[x] works for single frame
[x] works for double frame
[x] works for if and else
[x] works for for loops
[x] works for while loop
[x] nested for, if, else, else if
[x] works for py and r integrations
[x] works for Sikuli visual automation
[x] works for JavaScript code
[x] works for JavaScript functions
[x] works for default, chrome, headless
[x] works for test option testing mode
[x] works for positive test cases

Breaks in backward compatibility in exchange for deterministic automation execution

var now strictly not supported for variables definition to be in scope. previously for some steps  (eg echo, api), var variables will still be in scope
frame and popup after one another requires { and } steps. in general conditions (for, if else, else if, while) and popup / frame need { and } steps if used consecutively to prevent ambiguity
random use of { and } blocks without purpose no longer works. for example, when { and } steps are not used with if, else, else if, for, while popup, frame

Backward compatibility point 3 is now addressed by automatically supporting random code blocks by users that are not related to conditions or loops or frame or popup steps. I can't really think of a reason why a user would want to do that, perhaps for organizing sections of steps or keeping things readable. Nevertheless, updated codebase to support such random use of { and } to maximize backward compatbility.
For backward compatibility point 1, it is easy to update TagUI translation engine to always convert var variable_name = xxx to variable_name = xxx but this prevents advanced users who wants to limit scope of a variable within a section of JavaScript code from implementing good programming practices. Thus keep it flexible so that users can write variable_name = xxx according to TagUI documentation, or use var variable_name = xxx if they wish to limit scope for just that few consecutive JS statements.
For backward compatibility point 2, it is possible to write a pre-parsing algorithm in the translation engine to address that. However, that encourages bad scripting writing and ambiguity in code. Thus keeping it such that consecutive uses of conditions or loops or frame / popup steps require explicitly using { and } to define the code blocks in scope for such implementations. That promotes deterministic script-writing and easier maintenance as the script will show exactly what it is trying to do instead of leaving it to TagUI parsing engine to fill in the gaps.

List of items to evaluate and check before committing to master -

[ ] works for sample flows 1 to 9
[ ] works for normal TagUI steps
[ ] works for popup step
[ ] works for frame step
[ ] works for if and else
[ ] works for for loops
[ ] works for while loop
[ ] nested loops & conditions
[ ] works for py and r integrations
[ ] works for Sikuli visual automation
[ ] works for JavaScript code
[ ] works for JavaScript functions
[ ] works for positive test cases

Above is an initial working commit for this, keeping issue open for any post commit issues.
More details of the issue, checklist and break in backward compatibility here - https://github.com/kelaberetiv/TagUI/issues/150#issue-319436500

Made above commit with following comments (to reduce backward compatibility)
After #150, random use of { and } blocks without purpose no longer works.

For example, when { and } steps are not used with if, else, else if, for, while popup, frame.

This commit addresses that, so that random use of { and } still works.

Backward compatibility point 3 is now addressed by automatically supporting random code blocks by users that are not related to conditions or loops or frame or popup steps. I can't really think of a reason why a user would want to do that, perhaps for organizing sections of steps or keeping things readable. Nevertheless, updated codebase to support such random use of { and } to maximize backward compatbility.
For backward compatibility point 1, it is easy to update TagUI translation engine to always convert var variable_name = xxx to variable_name = xxx but this prevents advanced users who wants to limit scope of a variable within a section of JavaScript code from implementing good programming practices. Thus keep it flexible so that users can write variable_name = xxx according to TagUI documentation, or use var variable_name = xxx if they wish to limit scope for just that few consecutive JS statements.
For backward compatibility point 2, it is possible to write a pre-parsing algorithm in the translation engine to address that. However, that encourages bad scripting writing and ambiguity in code. Thus keeping it such that consecutive uses of conditions or loops or frame / popup steps require explicitly using { and } to define the code blocks in scope for such implementations. That promotes deterministic script-writing and easier maintenance as the script will show exactly what it is trying to do instead of leaving it to TagUI parsing engine to fill in the gaps.
This latest change is now usable from cutting edge version.

User reported issue with below, which shows ERROR - number of step { does not tally with with }
js begin
if (a==1) a=20;
js finish

This has to be fixed to be backward compatible with existing scripts -> to look into a fix.

Made commit above with following comments -
Code that are within integrations code blocks should skip special processing. This commit fixes that. For example, following should work and not having { and } auto-padded to after the if statement and after the js finish.
a = 1
js begin
if (a == 1) a = 2;
js finish

Issue found for this overhaul with using non-english languages. Non-English languages such as Chinese, French, Russian, were never included in the positive test cases as that wasn't a critical part of TagUI features and use cases. This has to be resolved, tracking it here first on the issue.

Above commit fixes translation issue for this overhaul, otherwise scripts using non-English languages won't work (eg Chinese, Serbian, Hungarian, German.. etc).

Query from user on the var backward compatibility
Noticed that your new update does not support the below – we have been using the var functions a fair bit to move files around in the automation process, is there a new function that is replacing this?

My reply on the impact of that particular use scenario
It might work from what you are describing. Can you try, and see if your existing scripts using var breaks? var variable_name = 'xxx' defines a local variable which only exists in the context of that consecutive series of JavaScript statements. The lines of JS code immediately following the var line should still be able to access the variable just as before the overhaul.
In this overhaul, it will only not be accessible in some subsequent steps such as api step or echo step. Previously api or echo step would be able to access variables defined with var immediately preceding. As the break in backward compatibility is for a minor use scenario which was originally not intended, I opted to go ahead with the overhaul without automatically removing the var in user's JavaScript statements. If automatically remove var from the JavaScript lines, it will prevent advance users from explicitly declaring a local variable for use in that few lines of JS code.

Closing issue for now since no further problems reported,
this latest change is now usable from cutting edge version.

Hi @kensoh
and thank you for this new release,
I needed it for the same above problem and found it in your comment here....
Great!
How can I subscribe to be aware of each new releases? (is it some feed o email subscribtion is available?)
Thanks for your work ! I 'm using it intensivly and please let me know how can pay you (at least for a beer!)

Hey @adegard it's our pleasure! I used to work on TagUI full-time on my own for a year, but now AI Singapore hires me to work on it and continue developing it. There isn't a mailing list now, would like to have incremental improvements every couple of weeks and releases every couple of months.
Thanks for your encouragement! I have an old paypal account. If you buy a drink and I'll probably use that to buy instant coffee to share with my friends in office. But there is really no need to pay, this is purely open-source contribution for people to have access to fun and useful automation tools!

Thanks @adegard for your generosity!
Below is the outcome of your 'beer treat' ---> more coffee for conversion to code

Your welcome Ken!
I'm happy so your project can go futher with some interns help.
In two weeks with your help we made 3 bots for our little HR recluting agency:

one to auto post job ads on portals
one to auto answer to new job candidates posts on 1 portale
one to automatically download pdf employee tax sheet from national istitute  (reducing hand working by 2-3 days a month!)

We will continue to use it for the simplicity of language ("natural" vs pantomjs...) but also  because it run headless (vs Uipath or other)
Have a nice day!
Arnaud

authors: adegard kensoh


text:
could not patch umodbus: numpy not found

I see this message when attempting to use this add-on in HA
could not patch umodbus: numpy not found
Should I be concerned?   I'm able to connect to the following UK based inverter, SUNSYNK-3.6K-SG01LP1 and get some readings but I'm getting "Timeout" conditions - I'm going to re-do my cabling and see if that fixes it.
Thanks

The addon does not use numpy, so simply ignore the warning.
Timeouts is mainly cabling& interference.
Check the projects github readme for some pointers…. even I get them a couple of times per day

Thank you so much for the very swift response and clarification. My first mistake was to assume the Cat5e patch cable I adapted was wired to T568A and in fact this is not the case, it's T568B I think.
Once I got the right cabling it started working. I am getting a Timeout condition every 1 or 2 mins but I'm upgrading the cable to a solid copper core and will use STP/SFTP type cabling instead, just in case of interference.
My inverter manual says Pin1 and Pin 7 are RS485B and Pin 2 and Pin8 are RS485A, Ground being Pin 3 and 6.
Is it important to cable up both wires in each case.... Could that help with the timeouts do you think?
Do I need to do anything to confirm my testing of my specific model - SUNSYNK-3.6K-SG01LP1

r.e Modbus Server ID, I have a single inverter and yet my installer has set the inverter as "Slave" - the parallel box is not checked. Could this be causing any issue?  The ModBus SN is 0 and that is what I have configured.

Server ID must not be 0. Try 1. I doubt that the master checkbox makes a difference here…
I would only wire up a single RS485 pair (and make sure its on a twisted pair)

I've changed the cable to a top quality SFTP (shielded) solid core cable and still the timeouts in the log appear approx every 1-2mins however the data does appear to be flowing nicely into MQTT and HA.
What do you think the issue is?   The actual inverter taking longer than expected by the code or my USB-RS485 Wave adapter? (the one you recommended in your notes)

Do you still have the Solarman Dongle reading data? It might also cause contention to read information...

Sort of. They don’t come with Solarman now. I have the new Sunsynk Wi-Fi dongle and app- I think it’s some kind of OEM’ing of software. I have asked Sunsynk support if this could be intervening but no response yet. I will chase. I can try removing the dongle I guess and looking at the logs.

I would try removing it for some test period… it queries the same Inverter MCU, very likely it’s over a shared bus as well

@rcotgrov Were you able to improve on the amount of timeouts?

I actually have not tried removing the Wi-Fi dongle as yet. Other things in life have taken priority. Sent from my iPhoneOn 22 Oct 2022, at 08:26, e1ace @.***> wrote:
@rcotgrov Were you able to improve on the amount of timeouts?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

Ok to close this? From the logs, the timeout every 10 seconds seems to be two different pollers competing for the RS485 bus (i.e. This Addon, every second and the Wifi dongle, every 10 seconds)

Yes, please close. I suspect you are right.
On Wed, Nov 9, 2022 at 10:24 AM Johann Kellerman @.***>
wrote:

Ok to close this? From the logs, the timeout every 10 seconds seems to be
two different pollers competing for the RS485 bus (i.e. This Addon, every
second and the Wifi dongle, every 10 seconds)
—
Reply to this email directly, view it on GitHub
https://github.com/kellerza/sunsynk/issues/64#issuecomment-1308533948,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB2NPCEJWJP7LVHB4PJH7XLWHN3UJANCNFSM6AAAAAAQVUB7IM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thx👍

authors: e1ace kellerza rcotgrov

repo: kellerza/sunsynk


text:
Handle AutoValue Optionals

Fixes #52

Will land after #54 as CI job won't pass until that lands

repo: kelloggm/object-construction-checker


text:
redis backend hashes support

Hey guys, is it possible to add support for redis hashes?
They suggest it as best practice for memory optimization (https://redis.io/topics/memory-optimization), so would be nice to have it.
Thanks!

Fixed in #604

authors: agrakhov okushchenko

repo: kelseyhightower/confd


text:
Assertion error

Hi,
I'm getting an assertion error on line 0, which is the whole code, so it's not very specific and hard to check/fix.
Here's what I'm getting:
Assertion failed on line 0: "no code found at the location."
Any help's appreciated!

let me fix that for you:

What a legend! Wouldn't have thought about that myself.

authors: Siand ctubio

repo: kelseyhightower/nocode


text:
Inclusao de 92 cidades com seus respectivos codigos ibge

Após filtrar bastante, consegui evitar duplicação de cidades e adicionar os códigos do IBGE (Deu um trabalhinho bom). As cidades repetidas existentes são cidades com o mesmo nome mas de estados diferentes.

Muito bom @jonathancbarros, valeu.

authors: jonathancbarros kelvins

repo: kelvins/Municipios-Brasileiros


text:
error anonymous define

How can I change maphilight to not anonymous define?
Error: Mismatched anonymous define() module: function(a)

I'd need more information than that. I.e. where are you seeing this error? (In browser? Some other tool?)

authors: importwholesales kemayo

repo: kemayo/maphilight


text:
Default to env['SENTRY_DSN'] if none is passed

We should allow the client to be configurable via an environment DSN variable (its common on the Heroku platform, and supported by PHP and Python).
Being that I have no idea how Java works, I'm only somewhat confident this is possible, but ideally it would just look like:
RavenClient client = new RavenClient()

Your Pull request satisfies this requirement. Closing issue.

authors: dcramer kencochrane

repo: kencochrane/raven-java


text:
Information regarding https://kenkoooo.com/atcoder/resources/merged-problems.json

Hi Team,
I am planning to use this api: https://kenkoooo.com/atcoder/resources/merged-problems.json for one of my personal hobby projects. I need some information and clarification regarding this api endpoint:

How often is this information updated?
I am planning to hit this endpoint once in every 5 minutes. Is this okay with you guys?

Thanks.

Thank you for the clarification!

How often is this information updated?

It's updated after a contest is over. That means it's updated at most twice a week.

I am planning to hit this endpoint once in every 5 minutes. Is this okay with you guys?

Yes, you can, if you handle 304 responses:
https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/304
Once you get a JSON file, please add an HTTP header If-Modified-Since. If the file is updated, the server will return the new file with 200. If there's no update, the server will return an empty response body with 304.
Thanks!

Hi @kenkoooo I think this file is updated more than 2 times a week, I ran my script for about 12 hours and it has updated more than 3 times already. Is this expected?

@ankur-kayal I didn't notice. How were they different?

I am not logging that but checked that the If-last-modified header indeed change. And it matches with my logs.

I guess the file content wasn't updated. The header was automatically updated by AWS. I didn't expect that behavior, but it shouldn't be a problem. You don't need to change your logic. Thank you for putting If-Modified-Since header.

authors: ankur-kayal kenkoooo

repo: kenkoooo/AtCoderProblems


text:
Weird behaviour with 'pipenv shell' and interactive python on Windows, every command causes a SyntaxError

I have no idea what happens here.
Totally trivial project with one dependency, otherwise empty. Now i run 'pipenv' to install, which works just fine. Then i run 'pipenv shell' to run a python shell in the venv. Any interactive command is greeted with a SyntaxError exception, even trivial stuff like import sys or print 1.
If i activate the venv directly via the Scripts\activate.ps1 script in the venv, everything works as expected.
This is a naked Python 2.7.13 x64 install on a Windows Server 2012R2 with the default powershell, no packages installed at all. No special shell wrappers or anything nasty installed as far as i can see. The only thing modified is the prompt to show the venv that is active, but the bug happens without that mod too.
PS  C:\pipenv_test> cat Pipfile
[[source]]
url = "https://pypi.python.org/simple"
verify_ssl = true

[packages]
requests = "*"

PS  C:\pipenv_test> pipenv --version
pipenv, version 5.1.2
PS  C:\pipenv_test> pew version
0.1.26
PS  C:\pipenv_test> C:\Python27amd64\python.exe --version
Python 2.7.13
PS  C:\pipenv_test> pipenv --python=C:\Python27amd64\python.exe install
Creating a virtualenv for this project...
Using C:\Python27amd64\python.exe to create virtualenv...
Running virtualenv with interpreter C:\Python27amd64\python.exe
New python executable in c:\venvs\pipenv_test-O9HVN3Wq\Scripts\python.exe
Installing setuptools, pip, wheel...done.

Virtualenv location: c:\venvs\pipenv_test-O9HVN3Wq
No package provided, installing all dependencies.
Pipfile found at C:\pipenv_test\Pipfile. Considering this to be the project home.
Pipfile.lock not found, creating...
Locking [dev-packages] dependencies...
Locking [packages] dependencies...
Updated Pipfile.lock!
Installing dependencies from Pipfile.lock...
[================================] 5/5 - 00:00:09
To activate this project's virtualenv, run the following:
 $ pipenv shell
PS  C:\pipenv_test> pipenv shell
Launching subshell in virtual environment. Type 'exit' to return.
Windows PowerShell
Copyright (C) 2014 Microsoft Corporation. Alle Rechte vorbehalten.

PS {VENV: pipenv_test-O9HVN3Wq} C:\pipenv_test> python --version
Python 2.7.13
PS {VENV: pipenv_test-O9HVN3Wq} C:\pipenv_test> python
Python 2.7.13 (v2.7.13:a06454b1afa1, Dec 17 2016, 20:53:40) [MSC v.1500 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> print 1
  File "<stdin>", line 1
    print 1
           ^
SyntaxError: invalid syntax

BUT in the same shell (right after the strange behaviour above) this works just fine:
PS  C:\pipenv_test> pipenv --venv
c:\venvs\pipenv_test-O9HVN3Wq
PS  C:\pipenv_test> . C:\venvs\pipenv_test-O9HVN3Wq\Scripts\activate.ps1
(pipenv_test-O9HVN3Wq) PS {VENV: pipenv_test-O9HVN3Wq} C:\pipenv_test> python
Python 2.7.13 (v2.7.13:a06454b1afa1, Dec 17 2016, 20:53:40) [MSC v.1500 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> print 1
1
>>> ^Z

So, this looks like it may be an issue with newlines. The shell is launched on Windows with Popen and the universal_newlines flag set. I'm wondering if your machine is having issues with the newline being passed from the enter keypress in your shell. I'm not immediately sure why this is an issue on your instance but not our previous windows test runners.

I tried disabling universal_newlines in pipenvs cli.py in the shell command, but that had no effect.
Will try a few things more on monday, the envvar PYTHONUNBUFFERD in the subprocess looks suspicious, maybe pyexpect messes with the console settings?

A few more data points. pew workon works as expected, so the issue is really in the pipenv part.
PS  C:\pipenv_test> pipenv shell
Launching subshell in virtual environment. Type 'exit' to return.
Windows PowerShell
Copyright (C) 2014 Microsoft Corporation. Alle Rechte vorbehalten.

PS {VENV: pipenv_test-O9HVN3Wq} C:\pipenv_test> python
Python 2.7.13 (v2.7.13:a06454b1afa1, Dec 17 2016, 20:53:40) [MSC v.1500 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import sys
  File "<stdin>", line 1
    import sys
              ^
SyntaxError: invalid syntax
>>> ^Z

PS {VENV: pipenv_test-O9HVN3Wq} C:\pipenv_test> exit
PS  C:\pipenv_test> pew workon pipenv_test-O9HVN3Wq
Launching subshell in virtual environment. Type 'exit' to return.
Windows PowerShell
Copyright (C) 2014 Microsoft Corporation. Alle Rechte vorbehalten.

PS {VENV: pipenv_test-O9HVN3Wq} C:\pipenv_test> python
Python 2.7.13 (v2.7.13:a06454b1afa1, Dec 17 2016, 20:53:40) [MSC v.1500 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import sys
>>>
>>> ^Z

Replacing pypenv shell with this also works just fine:
#!python
import subprocess
import sys

cmd = ['pew.exe', 'workon', 'pipenv_test-O9HVN3Wq']
p = subprocess.Popen(cmd, shell=True, universal_newlines=True)
p.communicate()
sys.exit(p.returncode)

More weirdness:
Just importing pipenv in above script makes it fail:
#!python
import subprocess
import sys

# Once this happens, it fails
import pipenv

cmd = ['pew.exe', 'workon', 'pipenv_test-O9HVN3Wq']
p = subprocess.Popen(cmd, shell=True, universal_newlines=True)
p.communicate()
sys.exit(p.returncode)

So something inside pipenv.cli must break console support.

This is pretty bizarre behaviour. I'm not sure why importing pipenv would cause the shell to do this if the code works otherwise. I haven't been able to reproduce this on a windows machine and don't have access to a copy of Windows Server 2012R2. My only suggestions for debugging at this point is to try removing some dependencies from pipenv.cli and see if one of them is a noticeable culprit. My initial hunch is possibly click_completion?

Found it, as i already guessed in the beginning. os.environ['PYTHONUNBUFFERD']='1' inside the delegator package breaks it. It should probably just set that for subprocesses spawned by delegator and not globally.
This breaks:
#!python
import subprocess
import sys
import os

os.environ['PYTHONUNBUFFERD']='1'

cmd = ['pew.exe', 'workon', 'pipenv_test-O9HVN3Wq']
p = subprocess.Popen(cmd, shell=True, universal_newlines=True)
p.communicate()
sys.exit(p.returncode)

Guess this should be fixed in delegator. Unless you want to add a workaround in pipenv. Feel free to close it here.

Fixed with delegator 0.13.

I cannot reproduce this with either 11.10.0 or master. Everything works as it should. I’m on Windows 10 Pro, Python 2.7.14. We also didn’t change anything related to shell AFAICT.

Ok, this is weird. At work I additionally used a plain fresh Windows 10 VM to test - same error. After your comment I booted my Windows 10 Pro laptop at home and tried that, same error. Maybe I am doing something stupid - I recorded a video as well if that may help: https://www.youtube.com/watch?v=TPtjsJQaJG8&feature=youtu.be

As above I’d be willing to guess this is a result of PYTHONUNBUFFERED or a system encoding issue. When you say ‘fresh’ did you set any environment variables, etc?

Sorry for being unspecific - by fresh I mean it's right after the installation of windows 10 (eng) completes. I have a Virtualbox Image with a snapshot right after Windows 10 finished installing, so I installed python 2.7.14, git for windows (which might set env vars), opened the regular cmd prompt as well as powershell and ran pipenv install followed by pipenv shell. If it matters, the other tested systems are Thinkpad and Dell laptops running english Windows 10 pro as well, with the same results.

PYTHONUNBUFFERED is indeed 1 for the process.
How can I find out my delegator version? If I install delegator manually in the virtualenv, delegator-0.0.3 is pulled in.
Maybe one of you could push me in the right direction to help further - I saw that PYTHONUNBUFFERED  is only set when the process not "blocking" - I might not understand enough about what is meant by blocking here, but if the python interpreter is run, wouldn't that be a blocking process?
Thanks for all the help so far!

PYTHONUNBUFFERED shouldn’t be set by default, so you need to unset it (and find out whatever set it if unsetting it works). For Powershell:
Remove-Item Env:\PYTHONUNBUFFERED

authors: nateprewitt schlenk techalchemy uranusjr woelper

repo: kennethreitz/pipenv


text:
CheckPoint option

Is there a option for me to save checkpoint while training and then continue training from the checkpoint. Cause right now I cannot start from where I stopped but to start over again

The model is saved at the end of each epoch if the total loss is less than the previous epoch total loss. This is done using a ModelCheckpoint callback. So to resume where you left off, you just have to use the load option the next time you want to initialize your training

authors: QuandoSJ magloiretouh


text:
CONTRIBUTING.md - incorrect command

Problem Description:
CONTRIBUTING.md states

This project uses nps and you can run npm start to see what scripts are available.

npm start tries to serve the website (and expects it to be built - which makes it fail on a freshly cloned repos)
It tells the user that it couldn't find .next/build-stats.json, which isn't a great first experience ;)

Suggested Solution:
Change the npm start to npm start help which does show a list of scripts.

Additionaly, the Working with locales section could use a
monospace block thingy

on the host file contents.
I am willing to do a PR for both of those problems, should it be a single pull request or two separate?

Thanks! You can make a single pull request for both, that's fine 😀

authors: irth kentcdodds

repo: kentcdodds/glamorous-website


text:
Translations in french [WIP]

What:
Translations in french

Why:
Closes #110

How:
Translate:

[x] Core pages
[x] Basis pages
[x] Advanced pages
[ ] Api pages
[ ] Integrations pages
[ ] Examples pages

Codecov Report

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

@@           Coverage Diff           @@
##           master     #112   +/-   ##
=======================================
  Coverage   96.19%   96.19%
=======================================
  Files          57       57
  Lines         210      210
  Branches       20       20
=======================================
  Hits          202      202
  Misses          7        7
  Partials        1        1

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 e1320d3...6da444a. Read the comment docs.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #112   +/-   ##
=======================================
  Coverage   96.19%   96.19%
=======================================
  Files          57       57
  Lines         210      210
  Branches       20       20
=======================================
  Hits          202      202
  Misses          7        7
  Partials        1        1

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 e1320d3...6da444a. Read the comment docs.

Much better 👍

On Jun 14, 2017, at 08:20, Forresst notifications@github.com wrote:
@forresst commented on this pull request.
In pages/basics/content/fr/install.js:

Il dépend aussi de ~react~ et ~glamor~ donc vous aurez aussi besoin d'eux dans votre projet (si vous ne les possédez pas déjà) :

npm install --save react glamor

REMARQUE : Si vous utilisez React v15.5 ou une version supérieure, vous aurez également besoin de

~prop-types~ : ~npm install --save prop-types~

Vous pouvez ensuite utiliser l'un des formats du module :

~main~: ~dist/glamorous.cjs.js~ - s'exporte en tant que module CommonJS

~global~: ~dist/glamorous.umd.js~ et ~dist/glamorous.umd.min.js~ - s'exporte en tant que module umd qui est consommable dans plusieurs environnements, le plus significatif de manière globale.

~jsnext:main~ et le module : ~dist/glamorous.es.js~ - s'exporte en utilisant la spécification des modules ES, vous aurez besoin de configurer webpack pour faire usage de ce fichier, faites-le en utilisant la propriété resolve.mainFields.

Le cas d'utilisation le plus courant consomme ce module via CommonJS :
@Macxim maybe:
Ce module est utilisé le plus souvent via CommonJS :
What do you think?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Check this out: https://fr.rc.glamorous.rocks/
Wooo!

What's the Something below Installation though?

@Macxim It is the subtitle
I guess it was a test that could be put to space ? @kentcdodds

Yes, we should remove that! Thanks for pointing it out 😅

authors: Macxim codecov-io forresst kentcdodds


text:
Not scrolling to last 'section' of nav

I have 2 slick carousels synced.  One acting as a sort of 'thumbnail nav' for the other.  This thumbnail nav shows more slides (5) then it scrolls (1).  If you try to go directly to one of the slides in last 'section' of the thumbnail nav, the slide in the main area will update correctly, but the thumbnail nav will not scroll to SHOW which slide was selected.
Fiddle is here: https://jsfiddle.net/8h905ta4/
Click on the 'click to go to 7th slide..' button at the top to see the problem.  Using version 1.5.7.

Thanks for reporting, this issue also happens with the latest version

Is there a quick patch available for this?

authors: ahmadalfy m-jeune steacha

repo: kenwheeler/slick


text:
Home Page Elements not showing in Mobile like they do in desktop

Hello!

My Home page looks great and works as it should on desktop, but doesn't seem to be appearing correctly on mobile. Any way to fix this? Thanks for your help!
Respectfully,
Walter
(PS. Laborator Support crew suggested I come here to ask about the problem Hi Walter,

It is weird how that happens! I have been checking about this about an hour and didn't find what is the problem. I have the same set of products and it was not happening! I suggest you to contact plugin author and ask if he could discover where the issue is coming from?
https://github.com/kenwheeler/slick/issues Click New Issue and then describe your problem.
Kind regards,
Arlind

====================================================================
[  http://www.cabalcrafted.com/  ]
use this jsfiddle to reproduce your bug:
http://www.cabalcrafted.com/
we will likely close your issue without it.
====================================================================
Steps to reproduce the problem

...  Go to http://cabalcrafted.com on a desktop
...  Go to http://cabalcrafted.com on a mobile phone
...  Observe how the mobile phone doesn't yield a viewable / usable home page as compared to the desktop version.

====================================================================
What is the expected behaviour?
...  The expected behaviour is for the product carousel, and the portfolio items beneath it to size itself and display appropriately for the mobile phone
====================================================================
What is observed behaviour?
...  The observed behavior is that the mobile device yields a home page that does not display product carousel, or portfolio items. Neither does it display the automated type (as shown Desktop screens.) Makes home page, unrecognizable, and prevents any interaction for user.
====================================================================
More Details

Which browsers/versions does it happen on? iphone 6
Which jQuery/Slick version are you using? - Not sure
Did this work before? - This is the first time I'm using the product carousel feature \

closing support req

authors: leggomuhgreggo oruta


text:
There is no release notes for v1.9.0 (changelog)

Hello everyone! Does anyone knows where I can find release notes for v1.9.0 (changelog)?
P.S. I've checked the releases page but there's no any info about v1.9.0 https://github.com/kenwheeler/slick/releases
Thanks!

@leggomuhgreggo @kenwheeler Can we get the changelog for v1.9.0 slick? Thanks!

Yes, there isn't a tag for 1.9.0 either. 🙂

authors: jahaganiev pedrosanta


text:
slickPause() not working

var popupSlide = $('.popup-con').slick({
autoplay: true,
slidesToShow: 1,
slidesToScroll: 1,
prevArrow: '.popup-prev',
nextArrow: '.popup-next'
});
$('.btnPause').on('click', function () {
popupSlide .slickPause();
});

@josephpspiteri @kenwheeler Not sure about anyone else, but pauseOnHover was exactly the issue I was having — setting that to false solved the issue for me.

If this is the last carousel we'll ever need, maybe add some tests to ensure everything works as intended?

I'm sorry, I missed the part where you paid for this and now I'm required to do what you want and can have shit talked to me?

@kenwheeler :+1

You got any ideas on how I can visually regression test this? I'm all ears.

@kenwheeler - calm down, I didn't mean to kick you or make demands.
The issue here just doesn't seem like such an edge case to me. Maybe setting up some form of basic tests could benefit all of us (ie. less issues).
For visual tests, something like https://github.com/Huddle/PhantomCSS could work, but it might be overkill.

I've wanted to do it for a while and I've tried with very little benefit

Here's a fiddle of the original issue pause slideshow
I can't seem to reproduce the error. May have been fixed in the meantime - or the problem arise under different circumstances.
@kenwheeler Given the amount of options the plugin provides, there are bound to be problems in some kind of scenario. I would just assume that including tests would make it easier for users to pinpoint the specific problem (and hopefully give them the opportunity to fix it themselves).

@kenwheeler - calm down, I didn't mean to kick you or make demands.
😞😒

@supermensa it's an open source project, why don't you go ahead and create them? make some positive contributions 😌

@simeydotme  I would argue that suggesting you add tests - or any other form of discussion within an open source project - is a positive contribution.
I was merely pointing out that while claiming that X is the best something - or the last X you'll ever need and struggling with 127 open issues, where most of them doesn't follow your contributing markdown rules; my suggestion is that maybe you're going about it in the wrong way.
I'd like to help out, but the original author(s) would probably be the best to determine how tests are managed (framework, scope etc.). I believe that if you have it, people will use it.

@supermensa There area 127 issues because:
a. I'm a new father
b. It was just the holidays
c. I have a day job
So for the last month or two I haven't had the kind of time I wish I had to devote to open source. Also of those 127 issues, I'd say a solid quarter are feature requests, a quarter are people who don't know javascript asking how to set it up, another quarter are people who do know javascript but are trying to make it do something unreasonable in some ridiculous edge case, and the other quarter are legitimate.
If I didn't have to respond to the first 3/4 and manage all those issues, I'd be able to stay more on top of the legit 1/4.
Slick is the best carousel. Can you really argue that it isn't? I don't think that because one edge case slips through the cracks it qualifies it as a shitty carousel. And what can I do about people not following contributing.markdown? We are sitting here , bullshitting, on an issue that didn't follow contributing.markdown. I asked for a reduced test case in jsFiddle that took 6 months to get delivered.
I need to see context. This could be the _.paused state acting up, or there could be not enough slides (line 344) or any other external variable.
In regards to tests, Slick has like 30 options, thats like 4000 different possible combinations. In 5-6 different browsers, on 3 operating systems. And half of the shit that can go wrong can't even be reliably tested. Look at my other libraries, the tests are there. This one is special. Its just not that cut and dry, or the tests would be there already.
And thank you for your contribution, because it is positive. I appreciate it.

@kenwheeler Thanks for that response.
Regarding issues. I'd imagine it would be hell to maintain a popular project like this.
I'd suggest being more ruthless and close issues that doesn't follow guidelines - and be strict in choosing which features to focus on and which to ignore (especially regarding weird edge cases). No one is probably ever going to build a slider that is perfect for everyone.
(Congrats on becoming a father. Skip the tests & go play!)

Anybody reading this, I found the solution to my issue here:
http://stackoverflow.com/questions/29707645/slick-carousel-functions-not-executing
Instead of calling .slickPause(), or .slickPlay(), call .slick('slickPause') or .slick('slickPlay').
Thanks for the awesome slider @kenwheeler !!

Your welcome @GrateHairyBallz

i solved the problem finally!
It's better to pause the slick animation with slick('slickpause') on mouse over, and then you can run it again on mouse out.
$('.slideshow').mousedown(function() { 		$(this).slick('slickPause');     }); $('.slideshow').mouseout(function() { 		$(this).slick('slickPause');     });

authors: AdibTE GrateHairyBallz kenwheeler poopsplat simeydotme supermensa xianghongai


text:
redsocks2编译错误

make[4]: Leaving directory '/home/andy/lede/build_dir/target-x86_64_musl/redsocks2-0.67'
touch /home/andy/lede/build_dir/target-x86_64_musl/redsocks2-0.67/.built
rm -rf /home/andy/lede/build_dir/target-x86_64_musl/redsocks2-0.67/.pkgdir/redsocks2.installed /home/andy/lede/build_dir/target-x86_64_musl/redsocks2-0.67/.pkgdir/redsocks2
mkdir -p /home/andy/lede/build_dir/target-x86_64_musl/redsocks2-0.67/.pkgdir/redsocks2
install -d -m0755 /home/andy/lede/build_dir/target-x86_64_musl/redsocks2-0.67/.pkgdir/redsocks2/usr/sbin
install -m0755 /home/andy/lede/build_dir/target-x86_64_musl/redsocks2-0.67/redsocks2 /home/andy/lede/build_dir/target-x86_64_musl/redsocks2-0.67/.pkgdir/redsocks2/usr/sbin
install -d -m0755 /home/andy/lede/build_dir/target-x86_64_musl/redsocks2-0.67/.pkgdir/redsocks2/etc/redsocks2
install -m0644 ./files/redsocks2.template /home/andy/lede/build_dir/target-x86_64_musl/redsocks2-0.67/.pkgdir/redsocks2/etc/redsocks2/config.template
install: cannot stat './files/redsocks2.template': No such file or directory
make[3]: *** [Makefile:55: /home/andy/lede/build_dir/target-x86_64_musl/redsocks2-0.67/.pkgdir/redsocks2.installed] Error 1
make[3]: Leaving directory '/home/andy/lede/feeds/small/redsocks2'
time: package/feeds/small/redsocks2/compile#1.89#0.87#11.28
ERROR: package/feeds/small/redsocks2 failed to build.
make[2]: *** [package/Makefile:116: package/feeds/small/redsocks2/compile] Error 1
make[2]: Leaving directory '/home/andy/lede'
make[1]: *** [package/Makefile:110: /home/andy/lede/staging_dir/target-x86_64_musl/stamp/.package_compile] Error 2
make[1]: Leaving directory '/home/andy/lede'
make: *** [/home/andy/lede/include/toplevel.mk:231：world] 错误 2

同错

+1

+1

先回退到helloword仓库的版本，远程获取的https://github.com/semigodking/redsocks.git 的PKG_SOURCE_VERSION版本有误，尽量去修复

先回退到helloword仓库的版本，远程获取的https://github.com/semigodking/redsocks.git 的PKG_SOURCE_VERSION版本有误，尽量去修复

感谢大佬

最近更新bug太多我直接回退到1月9日
cd small/luci-app-passwall
git reset --hard ecd92aa64174f5e7a200678b4122891764a6b245