Unable to load .spydata files containing numpy object arrays

Issue Report Checklist

[X] Searched the issues page for similar reports:
https://github.com/spyder-ide/spyder/issues/10013 - same issue, but no followup

[X] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice

[X] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)

[X ] Could not reproduce inside jupyter qtconsole (if console-related) (not applicable?)

[X] Tried basic troubleshooting (if a bug/error)

[X] Restarted Spyder
[X] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[X ] Tried the other applicable steps from the Troubleshooting Guide

[X ] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
When attempting to load .spydata files that contain numpy object arrays, i get the following error:

I believe the issue is in iofuncs.load_dictionary()
https://github.com/spyder-ide/spyder/blob/9b3e311c3289adeb104d7b60c013e8b34d43b091/external-deps/spyder-kernels/spyder_kernels/utils/iofuncs.py#L415
This stack exchange post provides workaround by modifying the default for np.load() to allow_pickle=True
I'm not sure if this is legitimately a bug or, perhaps a security feature, as per this warning in the numpy documentation here

What steps reproduce the problem?

Create a numpy array with dtype='object', for example

Save environment as foo.spydata using 'Save Data' button above Variable Explorer Pane
Restart spyder and attempt to load foo.spydata using 'Import Data' button above Variable Explorer Pane

What is the expected output? What do you see instead?
Expected out put is that the 'objArray' variable is imported. Instead, error above occurs "Object arrays cannot be loaded when allow_pickle=False"
Versions

Spyder version: 4.0.1
Python version: 3.7.3 (64-bit)
Qt version: 5.9.6
PyQt version: 5.9.2
Operating System name/version: Windows 10

Dependencies

atomicwrites >=1.2.0         :  1.3.0 (OK)
chardet >=2.0.0              :  3.0.4 (OK)
cloudpickle >=0.5.0          :  1.3.0 (OK)
diff_match_patch >=20181111  :  20181111 (OK)
intervaltree                 :  None (OK)
IPython >=4.0                :  7.12.0 (OK)
jedi =0.14.1                 :  0.14.1 (OK)
nbconvert >=4.0              :  5.6.1 (OK)
numpydoc >=0.6.0             :  0.9.2 (OK)
pexpect >=4.4.0              :  4.8.0 (OK)
pickleshare >=0.4            :  0.7.5 (OK)
psutil >=0.3                 :  5.6.7 (OK)
pygments >=2.0               :  2.5.2 (OK)
pylint >=0.25                :  2.4.4 (OK)
pyls >=0.31.2;<0.32.0        :  0.31.7 (OK)
zmq >=17                     :  18.1.1 (OK)
qdarkstyle >=2.7             :  2.8 (OK)
qtawesome >=0.5.7            :  0.6.1 (OK)
qtconsole >=4.6.0            :  4.6.0 (OK)
qtpy >=1.5.0                 :  1.9.0 (OK)
rtree >=0.8.3                :  0.9.3 (OK)
sphinx >=0.6.6               :  2.4.0 (OK)
spyder_kernels >=1.8.1;<2.0.0:  1.8.1 (OK)
watchdog                     :  None (OK)
cython >=0.21                :  0.29.15 (OK)
matplotlib >=2.0.0           :  3.1.3 (OK)
numpy >=1.7                  :  1.18.1 (OK)
pandas >=0.13.1              :  1.0.1 (OK)
scipy >=0.17.0               :  1.4.1 (OK)
sympy >=0.7.3                :  1.5.1 (OK)

@dalthviz, please take a look at this one asap to see if we can have it for 4.1.0.

@ccordoba12 as @nkleinbaer said the issue is related with the allow_pickle having as default value False because of security reason. Should I set allow_pickle to True in spyder-kernels ?

@nkleinbaer, this only happens for object arrays? Or any kind of array?

I've only experienced it w/ object arrays

the numpy docs make it seem like it would only be an issue w/ object arrays
https://docs.scipy.org/doc/numpy/reference/generated/numpy.load.html

@dalthviz, I'm by no means an expert, but I think the security issue with allow_pickle=True is moot in this case, since the load and save functions for .spydata files are using pickle anyways?

@ccordoba12 as @nkleinbaer said the issue is related with the allow_pickle having as default value False because of security reason. Should I set allow_pickle to True in spyder-kernels ?

That's not true because someone could put malicious code in a spydata file and send it to another user. So we'll have to inform users about the potential dangers of opening those files every time and they will have to agree on accepting them or not.

Right, but isn't that already a risk when loading ANY spydata file, not only those containing numpy arrays (given that all non-numpy data loaded from a pickle file anyways)? My point was that allowing pickling for numpy in order to fix this bug would not be any less secure than it already is.
If you want to warn before loading anything from a pickle file that sounds like a good idea, but broader issue than fixing this specific bug.

@ccordoba12 I recently noticed this issue is still open and tested it out to confirm it is still happening in a recent release (5.1.5).
I still think that usingallow_pickle=True is a fine solution. Above, you mentioned the concern that it would allow someone to put malicious code in a spydata file and send it to another user. But this risk already exists, because all objects besides np.ndarrays are serialized and deserialized with pickle. So allowing it for numpy objects as well doesn't create any additional risk

This issue isn't resolved for Spyder version 5.3.2 and 1.23.3.

Hi,
I had this problem and the way I got around it was to convert the numpy array with dtype = 'object' to a list (using np.tolist() ) and then save the environment using spyder's interface.
After using this method spyder loads the .spydata with no errors.  ( I haven't changed the allow_pickle at all)
In my case, I have used  dtype='object' to either store something other than a float ( a decimal number ) or a ragged nested sequence. so saving it as a list might be a better option anyways.

Hi @moreza14 !
Thanks, I ended up doing the same thing, converting it to a list. :)


Phases tests

moved id/refs => tag/follow
added tests for each phase/ phase_manager
fixed minor bugs
renamed transform to extraction

LGTM


DATA_ROOT env variable

Thank you for sharing your great work, I had a suggestion to change the DATA_ROOT environmental variable to something else like CVL_DATA_ROOT. As DATA_ROOT is quite a general name, it can already be used for some other tasks in a system.
Let me know your thoughts ...
Tnx

How did you solve the problem?

You can simply search for the all matches to DATA_ROOT using IDEs like vscode and replace the matches with CVL_DATA_ROOT.

@parsa-ra Can u give me  a working collab code to use for my case
I am havinf many troubles related to input and all(With the hugging face code)


The navigation bar improvement

The navigation bars could show the user where he/she is currently to make navigation easier

This enhancement has been added!


Approval request

What is your username and discriminator?
Anónimo#2973
Why did you join Discord Linux?
To resolve simple & complex Linux problems that I am facing by interacting with other dev's
Where did you hear about Discord Linux?
google search (discord.me)
What is the passphrase that was assigned to you by the bot?
https://blargbot.xyz/output/526258162458746168

From sans#8719 :
That is the link to the page your passphrase will appear on. Please read over it and reply with the passphrase once you are done.

From sans#8719 :
Closing as this form is for those without phones.


Specifying FP_DEF_TEMPLATE crashes the flow.

Description
Specifying FP_DEF_TEMPLATE crashes the flow. Issue traced to: apply_def_template.py. The bug is that click converts arguments cases to lowercase, but the CLI def expects templateDEF and userDEF. Tried manually fixing this. But instead got another bug of the same nature in defutils. Didnt debug further, decided to report.
You might be interested, in this too. If I fix this bug and specify non existing path to DEF it crashes with FileNotFound but the log does not make sense. Maybe a sanity check in OpenLane's scripts can be added?
Environment
Kernel: Linux v5.11.0-46-generic

Distribution: ubuntu 21.04

Python: v3.9.5 (OK)

Container Engine: docker v20.10.10 (OK)

OpenLane Git Version: 2022.01.13_01.51.43

pip:click: INSTALLED

pip:pyyaml: INSTALLED

pip:venv: INSTALLED

---

PDK Version Verification Status: OK

---

Git Log (Last 3 Commits)

c01e53a 2022-01-12T18:40:17-03:00 Fix for make pdk (#861) - Vitor Bandeira -  (HEAD -> master, tag: 2022.01.13_01.51.43, origin/master, origin/HEAD)

e33dd27 2022-01-12T20:23:47+02:00 [BOT] Update openroad_app (#818) - Openlane Bot -  ()

45cb357 2022-01-12T19:50:06+02:00 Global Routing Fixes & Enhancements (#856) - Donn -  ()

Reproduction Material
Any design with specified FP_DEF_TEMPLATE crashes.
Expected behavior
It does not crash.
Logs
[INFO]: Changing layout from /openlane/designs/user_analog_project_wrapper/runs/RUN_2022.01.17_10.12.50/tmp/floorplan/3-initial_fp.def to /openlane/designs/user_analog_project_wrapper/runs/RUN_2022.01.17_10.12.50/tmp/floorplan/4-io.def

[INFO]: Applying DEF template...

[ERROR]: during executing: "openroad -python /openlane/scripts/apply_def_template.py --def-template /openlane/designs/user_analog_project_wrapper/macros/def/user_analog_project_wrapper.def /openlane/designs/user_analog_project_wrapper/runs/RUN_2022.01.17_10.12.50/tmp/floorplan/4-io.def"

[ERROR]: Exit code: 1

[ERROR]: Last 10 lines:

    cli()

  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 1128, in __call__

    return self.main(*args, **kwargs)

  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 1053, in main

    rv = self.invoke(ctx)

  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 1395, in invoke

    return ctx.invoke(self.callback, **ctx.params)

  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 754, in invoke

    return __callback(*args, **kwargs)

TypeError: cli() got an unexpected keyword argument 'userdef'

[INFO]: Calculating Runtime From the Start...

[INFO]: Generating Final Summary Report...

[INFO]: Design Name: user_analog_project_wrapper

Run Directory: /openlane/designs/user_analog_project_wrapper/runs/RUN_2022.01.17_10.12.50

Source not found.

----------------------------------------

LVS Summary:

Source: /openlane/designs/user_analog_project_wrapper/runs/RUN_2022.01.17_10.12.50/logs/finishing/user_analog_project_wrapper.lvs.lef.log

Source not found.

----------------------------------------

Antenna Summary:

No antenna report found.

[INFO]: check full report here: /openlane/designs/user_analog_project_wrapper/runs/RUN_2022.01.17_10.12.50/reports/final_summary_report.csv

[INFO]: Saving runtime environment...

[ERROR]: Flow failed.

    while executing

"flow_fail"

    (procedure "try_catch" line 17)

    invoked from within

"try_catch $::env(OPENROAD_BIN) -python $::env(SCRIPTS_DIR)/apply_def_template.py --def-template $::env(FP_DEF_TEMPLATE) $::env(CURRENT_DEF)"

    (procedure "apply_def_template" line 4)

    invoked from within

"apply_def_template"

    (procedure "run_floorplan" line 39)

    invoked from within

"[lindex $step_exe 0] [lindex $step_exe 1] "

    (procedure "run_non_interactive_mode" line 54)

    invoked from within

"run_non_interactive_mode {*}$argv"

    invoked from within

"if { [info exists flags_map(-interactive)] || [info exists flags_map(-it)] } {

	puts_info "Running interactively"

	if { [info exists arg_values(-file)..."

    (file "./flow.tcl" line 387)

After fixing the issue, apply_def_template.py contains an error, defutil.py is called without a command to execute. Probably missing merge_components
subprocess.check_output([

        "openroad",

        "-python",

        f"{scriptsDir}/defutil.py",

        "merge_components",

        "--output", userDEF,

        "--input-lef", "/dev/null",

        userDEF, templateDEF

    ], stderr=subprocess.PIPE)

After fixing all of this:
INFO PSM-0064] Number of voltage sources = 54.

[INFO PSM-0040] All PDN stripes on net VPWR are connected.

 I am pretty much stuck.

UPD: Last issue was caused by out of memory. So only two above mentioned bugs exist right now. Hopefully will make a .def based design that can be added to list of designs to properly test this kind of bugs.

Looks like #864 by @donn fixes this issue, can the pull request be merged?

Not yet, no. It's not ready. I'll push a more granular fix for this on Monday.

New update: Looks like the first issue, with userDEF and templateDEF is not fixed by the pull request. IDK why it's not reprudicble in the CI, I assume, because of Click version mismatch?


Update Excel.Worksheet.Protect.md

Docs Build status updates of commit 4e000fc:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch lindalu-msft-patch-2 has been deleted or has been force pushed: fatal: Couldn't find remote ref refs/heads/lindalu-msft-patch-2

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Easily manage etcd cluster with env vars

Exporting  etcdctl environment variables, allow cluster managers to run etcdctl commands without providing arguments.
Currently when you logging in the etcd cluster you can't run etcdctl commands except you provide all the args.
$ etcdctl --cert-file=/etc/etcd2/ssl/etcd-client.pem --ca-file=/etc/etcd2/ssl/ca.pem --key-file=/etc/etcd2/ssl/etcd-client-key.pem  --endpoints=https://ip-10-0-0-5.eu-west-1.compute.internal:2379 cluster-health
member f5654f631de6e415 is healthy: got healthy result from https://ip-10-0-0-5.eu-west-1.compute.internal:2379
cluster is healthy

After this PR will be easy for run etcdctl commands
$ etcdctl cluster-health
member f5654f631de6e415 is healthy: got healthy result from https://ip-10-0-0-5.eu-west-1.compute.internal:2379
cluster is healthy

Also this is important on #49 and signaling cloudformation when the etcd cluster is healthy.

Current coverage is 54.72% (diff: 100%)

Merging #58 into master will not change coverage

@@             master        #58   diff @@
==========================================
  Files             4          4
  Lines          1038       1038
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            568        568
  Misses          388        388
  Partials         82         82

Powered by Codecov. Last update 7237203...a47d7d6

@gianrubio I'm impressed with your work 👍

Thanks for the contribution! Merging.


Build failed with libbcc 0.19.0-21d866e2

When trying to run one of the examples, I get a build failed with the following error
# github.com/iovisor/gobpf/bcc
bcc/module.go:98:40: too many arguments in call to _Cfunc_bpf_module_create_c_from_string
        have (*_Ctype_char, number, **_Ctype_char, _Ctype_int, _Ctype__Bool, nil)
        want (*_Ctype_char, _Ctype_uint, **_Ctype_char, _Ctype_int, _Ctype__Bool)
bcc/module.go:230:28: too many arguments in call to _C2func_bcc_func_load
        have (unsafe.Pointer, _Ctype_int, *_Ctype_char, *_Ctype_struct_bpf_insn, _Ctype_int, *_Ctype_char, _Ctype_uint, _Ctype_int, *_Ctype_char, _Ctype_uint, nil)
        want (unsafe.Pointer, _Ctype_int, *_Ctype_char, *_Ctype_struct_bpf_insn, _Ctype_int, *_Ctype_char, _Ctype_uint, _Ctype_int, *_Ctype_char, _Ctype_uint)
bcc/module.go:261:33: too many arguments in call to _C2func_bpf_attach_uprobe
        have (_Ctype_int, uint32, *_Ctype_char, *_Ctype_char, _Ctype_ulong, _Ctype_int, number)
        want (_Ctype_int, uint32, *_Ctype_char, *_Ctype_char, _Ctype_ulong, _Ctype_int)

The definitions for the functions seem to have dropped the last attribute. I am running the libbcc version 0.19.0-21d866e2 looked up from bcc_version.h.
root@linux-dev:/usr/include/bcc# grep -r bcc_func_load . -A 3
./bcc_common.h:int bcc_func_load(void *program, int prog_type, const char *name,
./bcc_common.h-                  const struct bpf_insn *insns, int prog_len,
./bcc_common.h-                  const char *license, unsigned kern_version,
./bcc_common.h-                  int log_level, char *log_buf, unsigned log_buf_size);
--
./bpf_module.h:  int bcc_func_load(int prog_type, const char *name,
./bpf_module.h-                    const struct bpf_insn *insns, int prog_len,
./bpf_module.h-                    const char *license, unsigned kern_version,
./bpf_module.h-                    int log_level, char *log_buf, unsigned log_buf_size);

After modification of the function definitions, I am able to run the examples, but as
sudo -E /usr/local/go/bin/go run examples/bcc/execsnoop/*.go

and not as
$ sudo -E /usr/local/go/bin/go run examples/bcc/execsnoop/execsnoop.go
# command-line-arguments
examples/bcc/execsnoop/execsnoop.go:241:10: undefined: newOutput

Environment info
$ lsb_release --all
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 20.04.2 LTS
Release:        20.04
Codename:       focal

$ go version
go version go1.15.6 linux/amd64

$ uname -a
Linux linux-dev 5.4.0-70-generic #78-Ubuntu SMP Fri Mar 19 13:29:52 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

$ apt show libbcc
Package: libbcc
Version: 0.10.0-1
Priority: optional
Section: misc
Source: bcc
Maintainer: Brenden Blanco<EMAIL_ADDRESS>Installed-Size: 58.3 MB
Provides: libbpfcc, libbpfcc-dev
Depends: libc6, libstdc++6, libelf1
Conflicts: libbpfcc, libbpfcc-dev
Homepage: https://github.com/iovisor/bcc
Download-Size: 15.8 MB
APT-Manual-Installed: no
APT-Sources: https://repo.iovisor.org/apt/bionic bionic/main amd64 Packages
Description: Shared Library for BPF Compiler Collection (BCC)
 Shared Library for BPF Compiler Collection to control BPF programs
 from userspace.

The go binding should be made working with the newest version of bcc, there shall be special tag added for any changes that map to certain commit in the bcc repo. Hopefully can cut releases alongside BCC release (if BCC has releases).


Ensure registry cache does not interfere with returned errors

If there is a cache miss, and an upstream error we were hiding the upstream error.

Bug introduced in: https://github.com/weaveworks/flux/pull/376


Convert SLES 15 docker image to artifactory location

This changes the location of the SLES 15 SP3 build environment image to one that is built and stored on Artifactory. This source image is also closer to native SLES 15 as it uses the standard published image available from SUSE.

@heemstra since the version remains the same, the RPM on Arti that main builds will clobber/replace what's there already. The LiveCD will pick it up next build, so will CSM/rpm YAMLs and the CSM tarball, however we do need to still update assets.sh to pull the newly built LiveCD (since it has the timestamp and ghash in it).


feat(api): implement labware.set_offset in 2.18

The python protocol api's labware.set_offset() command lets you set a labware position check offset programmatically. This is useful in protocols run outside the app (i.e. through Jupyter, through opentrons_execute).
When we moved to the protocol engine, we didn't support running those protocols outside the app, and therefore we didn't implement this method on those new API versions.
Now, we allow that again, so implement that method again.
This relies on dispatching AddLabwareOffset actions at arbitrary times, because it will set an offset for the location in which the labware is currently present. This allows the user to set offsets without having to wrangle with the engine's internal definitions of where something is, at the cost of the user having to spread their offset calls around their protocol if they have protocols that load things in one place and later move them to another.
Closes RSQ-29
To Leave Draft

[ ] tests
[ ] general agreement that this is the right path

How are we going to communicate to users where they need to call set_offset?
Seems to me that documenting that might be harder than documenting how to use the engines internal definitions of where something is

How are we going to communicate to users where/when they need to call set_offset? Seems to me that documenting that might be harder than documenting how to use the engines internal definitions of where something is.
I really disagree with this. I think it's very documentable to say "use set_offset by calling it right after you move a labware"
Don't each one of the internal definitions of the locations correspond to, "an offset we calculate in LPC"?
yes, but it's annoying (see below) to actually wrangle the various function calls, unions, isinstance checks, and type imports to do it correctly.
Can you maybe provide an example of what users would have to define if we went the other route?

if you went the other way, they'd have to do this: https://github.com/Opentrons/opentrons/pull/14940/files#diff-6f733f7119a96d280a77b7a2c54d99087fa07c7759299c4ce029c48ec0c1c668R103-R154 or we'd have to provide some function to look it up

@sfoster1, your response to my earlier comment helped me understand.
I thought users would get confused about keeping track of where the labware was at. But they have to do that regardless.
My mistake, you're correct, the downside of having to do all those imports and describing to the user how to use said imports, is much more difficult than just telling them to call set offset.
This looks good to me so far

Cool, TY!
I think we'll need to document on the public set_offset() method that identical labware in the same location will share the same offset. This is the opposite of what the PAPIv≤2.13 behavior was, and what is currently documented.

Right, definitely - it's now adding a labware offset. Can do.

@ecormany Would love your opinion on the documentation strategy here

Cool, TY!
I think we'll need to document on the public set_offset() method that identical labware in the same location will share the same offset. This is the opposite of what the PAPIv≤2.13 behavior was, and what is currently documented.

https://github.com/Opentrons/opentrons/pull/14967


Relocate and Rename SumN

The name -- and having to put the count in >= -- are non-obvious.  It also prevents you from adding both a filter and weight at the same time.  I suggest making a TopN input box below the >= value (maybe only active if Sum is selected).

Yeah, I've put a 'Pick Top' box in, but haven't committed to cleaning out Sum/SumN (they sort of work the same now)


High performance view statistics loader

WIP I wrote a high performance (non-ActiveRecord) class for loading out view statistics as a 2-d array of...
[
  [Date, counts for that day, cumulative total of all counts]
]

@rschenk Looks good to me. Bypassing the ActiveRecord stuff really improves performance here, right? I'm assuming the main use case for this is powering the analytics graphs? I was wondering if there'd ever be the need to query a date range in the array of results and if another data structure might help with that. I can't think of one...

@awead Yep that's exactly its intention is to power the graphs. Instantiating an ActiveRecord for each row would be expensive and pointless, since the graphs want to be fed a 2d-array anyways, so why not cut out the middle man.
This class can totally have date range filtering, and batched/page loading added to it later if need be


proxy port not working with docker build build-arg

Description of problem:
--build-arg doesn't work with proxy on different port than 80. Our Proxy ' http://proxy.my.url:8080'
docker version:
Docker version 1.9.1, build a34a1d5
docker info:
Containers: 22
Images: 31
Server Version: 1.9.1
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 75
 Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.19.0-43-generic
Operating System: Ubuntu 15.04
CPUs: 4
Total Memory: 15.38 GiB
Name: xxxxxx
ID: 7TGI:DQ64:JJEB:ODJT:FTNV:M44I:2YKJ:VGOQ:DAH6:IN4F:RXK3:XL57
WARNING: No swap limit support

uname -a:
Linux BERN30035062A 3.19.0-43-generic #49-Ubuntu SMP Sun Dec 27 19:43:07 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

Environment details (AWS, VirtualBox, physical, etc.):
physical ubuntu 15.04

How reproducible:
dockerfile
FROM debian:jessie

RUN apt-get -y update

Variant (1)
docker build --no-cache --build-arg http_proxy="proxy.my.url:8080" .
docker build --no-cache --build-arg http_proxy=proxy.my.url:8080 .

Actual Results:
...
W: Failed to fetch http://httpredir.debian.org/debian/dists/jessie/Release.gpg  Cannot initiate the connection to 8080:80 (<IP_ADDRESS>). - connect (22: Invalid argument)

Expected Results:
...
Get:1 http://security.debian.org jessie/updates InRelease [63.1 kB]
Get:2 http://security.debian.org jessie/updates/main amd64 Packages [241 kB]
Ign http://httpredir.debian.org jessie InRelease
Get:3 http://httpredir.debian.org jessie-updates InRelease [136 kB]
...

Variant (2)
Steps to Reproduce:
docker build --no-cache --build-arg http_proxy=http://proxy.my.url:8080 .
docker build --no-cache --build-arg http_proxy=http://proxy.my.url\:8080 .
docker build --no-cache --build-arg 'http_proxy=http://proxy.my.url:8080' .
docker build --no-cache --build-arg http_proxy='http://proxy.my.url:8080' .
docker build --no-cache --build-arg http_proxy='http://proxy.my.url\:8080' .

Actual Results:
...
Failed to fetch http://security.debian.org/dists/jessie/updates/Release.gpg  Could not resolve 'proxy.my.url'

Expected Results:
...
Get:1 http://security.debian.org jessie/updates InRelease [63.1 kB]
Get:2 http://security.debian.org jessie/updates/main amd64 Packages [241 kB]
Ign http://httpredir.debian.org jessie InRelease
Get:3 http://httpredir.debian.org jessie-updates InRelease [136 kB]
...

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

part of the issue is that your daemon can't find "proxy.my.url" - see the last error message - so you'll need to fix that issue outside of docker on your host.
the other error (variant 1), might be a golang issue since it appears that the env var is being picked up but being parsed incorrectly.  The docs say the "http://" part of the url is optional but from your output (and my quick test) it sure looks like it might be required.

i have removed all docker images. To show you that the proxy is available from my host.
docker build --build-arg http_proxy=http://proxy.my.url:8080 --build-arg https_proxy=http://proxy..my.url:8080 .
Sending build context to Docker daemon 2.048 kB
Step 1 : FROM debian:jessie
jessie: Pulling from library/debian
523ef1d23f22: Pull complete
140f9bdfeb97: Pull complete
Digest: sha256:f968f10b4b523737e253a97eac59b0d1420b5c19b69928d35801a6373ffe330e
Status: Downloaded newer image for debian:jessie
 ---> 140f9bdfeb97
Step 2 : RUN apt-get -y update
 ---> Running in 6df2a92b3992
Err http://httpredir.debian.org jessie InRelease

Err http://httpredir.debian.org jessie-updates InRelease

Err http://httpredir.debian.org jessie Release.gpg
  Could not resolve 'proxy.my.url'
Err http://httpredir.debian.org jessie-updates Release.gpg
  Could not resolve 'proxy.my.url'
Err http://security.debian.org jessie/updates InRelease

Err http://security.debian.org jessie/updates Release.gpg
  Could not resolve 'proxy.my.url'
Reading package lists...
W: Failed to fetch http://httpredir.debian.org/debian/dists/jessie/InRelease

W: Failed to fetch http://httpredir.debian.org/debian/dists/jessie-updates/InRelease

W: Failed to fetch http://security.debian.org/dists/jessie/updates/InRelease

W: Failed to fetch http://httpredir.debian.org/debian/dists/jessie/Release.gpg  Could not resolve 'proxy.my.url'

W: Failed to fetch http://httpredir.debian.org/debian/dists/jessie-updates/Release.gpg  Could not resolve 'proxy.my.url'

W: Failed to fetch http://security.debian.org/dists/jessie/updates/Release.gpg  Could not resolve 'proxy.wdf.sap.corp'

W: Some index files failed to download. They have been ignored, or old ones used instead.
 ---> 9154c5ddec19
Removing intermediate container 6df2a92b3992
Successfully built 9154c5ddec19

This is just saying it can't resolve proxy.my.url, meaning DNS lookup is failing for your proxy address.

Also, you missed a replace on your proxy URL in the last line of the failed command.

@cpuguy83 what can i do? the build command did not have a dns option.

Can pass in the IP instead of the DNS name, or you can provide a DNS server at the daemon level docker daemon --dns <ip> --dns <ip>

it works :+1:
docker build --no-cache --build-arg http_proxy=http://x.x.x.x:8080 --build-arg https_proxy=http://x.x.x.x:8080 .

or alternative set dns servers for systemd /etc/systemd/system/docker.service.d/dns.conf first google dns then our own
[Service]
ExecStart=
ExecStart=/usr/bin/docker daemon -H fd:// --dns <IP_ADDRESS> --dns x.x.x.x --dns x.x.x.x

@gimler is this resolved for you?


pkg/client/stream: fix deadlock for client methods

Client.queryOffset/queryPublisherSequence/StreamStats were fixed to prevent deadlock in case of request timeout (for example rabbit server is down). Now these methods returns error immidiately if it occured and doesn't try to read from data chanel, because in case of request timeout it hangs (no other gorouitine would write to this channel).
Also Client is extended with additional field socketCallTimeout, for testing purposes (decrease running time of new tests).
Closes #273

@HustonMmmavr Thank you for the PR
At the moment we are busy with other tasks. We will have a look as soon as possible.
Please have a bit of patience :)

@HustonMmmavr I am testing the PR and it seems to be ok. May I ask to resolve the conflicts so I will merge!
Thank you

Thank you

Thanks)


Added support for tracing_port

proxy client needs tracing_port to be specified, if exists

@howardyoo please make sure you bump the package version before merging this

Okay.. which package version are you referring to? Not sure what to update...

Okay.. which package version are you referring to? Not sure what to update...

This one here https://github.com/wavefrontHQ/wavefront-pyformance/blob/master/setup.py#L24

Ok... updated the version to 1.0.1

Ok... updated the version to 1.0.1

Version 1.0.1 is current one [*0], so it should be 1.0.2 for you.
The reason why we need an unique version every time is that we deploy every merged commit to PyPI [*1] automatically, and it doesn't allow overwriting existing uploads.
[*0] https://github.com/wavefrontHQ/wavefront-pyformance/blob/master/setup.py#L24
[*1] https://test.pypi.org/project/wavefront-pyformance/#history

Ok, changed the version to 1.0.2

We'll have to wait for @sushantdewan123 or @vikramraman to merge this


Espresso: improved preset handler + more presets

Requirements
Checklist

[x] I have read the Contributing Guide. Don't lie! 😉
[x] My PR is on a custom branch and is not named main.
[ ] I have added relevant unit tests.

Notes

Your PR will likely not be merged without proper and thorough tests, even if it's for a "simple" fix.
If you are an external contributor, you will see a comment from @buildbot-princeton. This is solely for the maintainers.
When your code is ready for review, ping one of the active maintainers.

Summary of Changes
More presets for the espresso calculator, I am not done yet but I open this PR to discuss a little bit more about presets. For now I created new presets for specific cases (metals, semiconductors) where I copied the pseudopotentials part from the previous two presets.
Should it be possible for the user to specify more than one preset i.e:

(..., preset = 'my-param-preset, pseudopotentials = 'my-pp-preset', ...)
(..., presets = ['my-param-preset, 'my-pp-preset'], ...)
(..., preset = 'my-big-preset-containing-both--pp--and-param', ...)

Can one of the admins verify this patch?

Thanks for the info, indeed the 'parent' mechanism is all I was looking for, I used it in pretty much every new presets. As you will see I changed the function the parse_pp_and_cutoff in espresso.utils to reflect the new changes in presets.
Something you might not like is the new ability to modify the Atoms object directly by a preset, If not I can probably find another way to do what I want.
I added a test to make sure everything is working correctly and that the priority between calc_default, user defined kwargs and presets kwargs is respected.

@tomdemeyere: Great! Glad that feature could help streamline things.
Would you be able to walk me through the logic for having the pbc: [True, True, True] in the preset? Maybe once I understand the context a bit more, I can help make a suggestion that is a bit less roundabout. It would indeed be nice to mess with the Atoms object if possible, but more-so I am looking to understand the use case you have in mind.

Specifying pbc: [True, True, False] would make things work better with ase.io.write_espresso_in for example. If you specify the kspacing (which is a single float), the k-grid generated with the specified pbc's would have no k-point along the third dimension.
This is completely avoidable, for example by manually calling the function kspacing_to_grid from ase as a workaround. I just thought that changing Atoms inside presets might be an interesting idea that you might want to hear about, but after second thoughts changing it might raise a few other problems

Ah thanks for clarifying! Yeah, I think I'd prefer this to be taken care of outside the presets. After all, the user may set no preset and also this behavior is not immediately obvious to a new user. In general, I like to abide by the principle of least surprise. Let's keep setups to just input parameters to the calculator.

@Andrew-S-Rosen Presets still contains atoms_info but it is not used to modify the Atoms object, instead, it creates an atoms copy and calls spacing_to_grid manually.
Creating an entire copy might be overkill for now, but will be useful in the future if other checks are performed
This should be pretty good to go as it is, I suspect there are some details you might want to check

@Andrew-S-Rosen Presets still contains atoms_info but it is not used to modify the Atoms object, instead, it creates an atoms copy and calls spacing_to_grid manually.
Creating an entire copy might be overkill for now, but will be useful in the future if other checks are performed
This should be pretty good to go as it is, I suspect there are some details you might want to check
EDIT: re-reading your last comment I realise you probably want no "atoms_info" in presets at all, I will find a way to delete them

Thanks for the update!
One option that you might prefer is to create a separate module called slabs.py where the default preset is set to one of your slab presets, and in that recipe you could also set atoms.pbc as needed. See the analogous module for VASP. This is entirely optional though. I just wanted to let you know of an alternate way to think about this.
Regarding the atoms info, a reason that I'm not a huge fan of the atoms_info key in the YAML file is that it is effectively a hidden feature. I try to ensure that the YAML parameters have 1:1 correspondence with calculator parameters, so it's obvious what can and can't be set. Here, the YAML is supporting some "hidden convenience magic". Also, one might think it would be atoms.info that is being set when in reality, that's not the case.
I'm not sure off-hand if there are other parameters behaving similarly, e.g. kspacing. If there are any parameters set in the YAML that are not direct arguments for the calculator, please take a moment to add them as defined, optional keyword arguments to the Espresso class that way they are documented and more generic.
Otherwise, this looks good and I'm happy to merge once that minor detail is sorted out. Thank you!

@tomdemeyere: For a simple fix, we can consider adding and documenting a kwarg to the Espresso calculator that is slab: bool = False. If set to True, then the PBCs can be set appropriately. Then in the YAML, you could have slab: true.

@tomdemeyere: For a simple fix, we can consider adding and documenting a kwarg to the Espresso calculator that is slab: bool = False. If set to True, then the PBCs can be set appropriately. Then in the YAML, you could have slab: true.

With these solutions, a user who uses the ESM presets without setting this additional parameter or without using the specific workflow/job would be very likely to create a non-functional calculation (ESM in QE refuses any k-points that is not in the x y 0 plane) which is not ideal for a preset.
So far I am trying to find a solution for a preset that is:

Self-contained (no need for additional parameters for a calculation to be functional)
Does not use any kind of hidden features (although we could argue that input_atoms is a parameter of the Espresso calc)

Additionally, I think it is very important that users can feel they have a global grasp of what is going on when using quacc (and by that, I mean which parameters are doing what and when), and adding such a specific "slab" might be confusing in this regard. When I was trying a different workflow engine, I quickly got overwhelmed by how many parameters there were, and understanding the overlap between them was complicated and time-consuming.
For now, I am thinking about the possibility of turning kspacing into a 3-element list, that would allow the preset to specify the spacing in each direction. It's not ideal because it does not respect the original definition of the calculator though...

It sounds like this might be resolved by having the dedicated slab workflows. But I will trust your best judgment in the end.
I agree that minimizing the number of non-standard parameters is ideal for the sake of the user.

No satisfying solution here in my opinion, slab presets will not have any k-point information. It will be up to the user to specify them.
It should be good to go if you are happy with it

@tomdemeyere: Thanks for this nice PR. I am happy to merge as-is. We can always revisit things later if desired.


Add Copy operation to the radial menu

Now that we can Copy and Paste with ⌘-C and ⌘-V (see #2498), it would be nice to add a Copy operation to the radial menu for discoverability.
@samanpwbb, can you work your magic to make a copy icon?
Not sure whether or not a "Paste" button makes sense here too.

I take this as a different 'copy'. Using keys to copy can utilise the clipboard. Copying using the annular menu ought to work differently - i.e. as per the move function that leaves the original in place.

Not sure whether or not a "Paste" button makes sense here too.

I think it'd be frustrating to have copy without paste. This does mean we'd have to enable right-clicking on blank areas of the map and move the edit menu controller out of modeSelect.

Now that we've done #7577, the edit menu is also accessible on touch devices. Thus, implementing menu buttons for copy and paste would make these operations possible on mobile devices where keyboard shortcuts aren't generally available.


Add Bundler to staging a doc request page

Fixing the following PR: https://github.com/kubernetes/kubernetes.github.io/pull/1796

This change is

This works best if we add github-pages gem to the gemfile.

e.g. https://github.com/kubernetes/test-infra/pull/1508/files

@jaredbhatti is this good to go or are there more changes that need to be made?

@mikedanese What's the specific fix for the instructions?

It's probably not a big deal, this looks fine


Revamp ktest R package following Python package refactoring

Minor refactoring of ktest R package :

fix some minor issues (c.f. #10 for examples)
use new version of ktest Python package under the hood (following its refactoring)

Ready to merge.
See r-pkg/README.md and [r-pkg/vignettes](r-pkg/vignettes] for more details about using the new version of ktest R package.


[New game]: Tic-Toc-Toe Game

🎮 Game Request
Game Setup:
Tic-Tac-Toe is played on a 3x3 grid, making a total of 9 cells.
Two players take turns, one using "X" and the other using "O."
Starting the Game:
Decide who goes first, often done by flipping a coin or another random method.
The first player places their mark ("X" or "O") on any empty cell on the grid.
Taking Turns:
Players take turns placing their marks on empty cells.
Once a cell is marked, it cannot be used again in the game.
Winning the Game:
A player wins the game if they get three of their marks in a row, horizontally, vertically, or diagonally.
The game ends as soon as a player wins, and they are declared the winner.
Restart the Game:
After A player wining A restart button is visible, wher user click restart button it start a new game.
Point down the features
We need two player to play this game.
Select program in which you are contributing
hacktoberfest
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Please assing me this issue , so I can work on this.

Hey @pabitra0011 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @pabitra0011!
We are already having a similar game request in #2333 👀
Make sure you come up with a cool unique idea 😀
Waiting for your new game idea 💗.

Hey @pabitra0011 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


debug: Send errors on the control channel

We were always sending the response that was received on executing agent
callback function for a command, on the control channel.
In case of an error, this response is empty and hence the error is not
propogated to the proxy at all, resulting in an empty message sent to
the proxy in case of error.
Instead check for error case and send the error string on the control
channel. With this, we can finally see the errors occurring in the agent
in the proxy logs.
Signed-off-by: Archana Shinde<EMAIL_ADDRESS>

@sboeuf Can you take a look

LGTM

Good catch @amshinde! :smile:


[Android] Overflow:hidden not work properly

When the parent view (overflow: 'hidden') has borderRadius set, the child view is not clipped to the shape of the parent.

demo reproducing the issue: https://rnplay.org/apps/69TRrw

Thanks for the report. Unfortunately overflow: hidden is not respected on android (see https://facebook.github.io/react-native/docs/known-issues.html#the-overflow-style-property-defaults-to-hidden-and-cannot-be-changed-on-android). I think @kmagiera has previously posted about the clipping issue for rounded corners and can chime in here.

Nothing more to add here. Unfortunately this is a known problem and is not being worked on as mentioned in the docs. Closing this as it's already on the list. If you guys have any proposal on how this could be implemented without causing significant perf issue in rendering pipeline feel free to reopen and post your ideas here

We're using imgix, so I think we'll be able to add ?fm=png&mask=ellipse and be on our way. Dodged a bullet there but will encounter moderate problems elsewhere.
I'll submit a PR to note that in addition to hidden always being true, it doesn't obey borderRadius, which is not currently noted in the docs. (I assumed that if hidden were true, it would be clipped by the borderRadius just fine.)

@kmagiera May I ask where to find the comment or discussion about clipping issue with rounded corners that you posted before?

@Kudo: I was referring to the link posted by Andy just above my comment: https://facebook.github.io/react-native/docs/known-issues.html#the-overflow-style-property-defaults-to-hidden-and-cannot-be-changed-on-android

@kmagiera I see.
The feature is really useful for some cornered UI construction.
Would like to know more detail of your mentioned significant perf drops and see if we can make this feature happens.

More info:
this demo (https://rnplay.org/apps/WlY9aA) shows if the outer control is an image, the borderRadius property will totally not working.

We're working on an alternative UI implementation on Android that will fix this -- it's still in testing internally so don't expect it for a few more months, but do know we are working on getting a fix.

Any updates on this issue ?

I have a fairly cringe-worthy fix using styles that works for me in the meantime. This is a UI for creating group avatars out of user avatars. Similar to Facebook Messengers group avatars.
https://rnplay.org/apps/qXLzlw

@bsinc that's a great solution! thx.
just clarifying what you did:
  <View style={styles.circle}>
            <Image style={styles.image} />
            <View style={styles.fixCircleClipping}/>

          </View>

  circle: {
    width: circleSize,
    height: circleSize,
    borderRadius: circleSize / 2,
    overflow: 'hidden',
  },
  fixCircleClipping: {
    position: 'absolute',
    top: -circleFixBorder,
    bottom: -circleFixBorder,
    right: -circleFixBorder,
    left: -circleFixBorder,
    borderRadius: circleSize / 2 + circleFixBorder / 2,
    borderWidth: circleFixBorder,
    borderColor: bgColor
  },

@bsinc That's brilliant, thank you!

@astreet Is there any updates?
Thanks

Is this still a known issue? I can't find the known issues page anymore.

@astreet this will be fixed in 0.32? When 0.32 comes out?

@Duelsik Was this hinted to being fixed in 0.32 or are you just asking? The release notes for 0.32 do not mention it anywhere.

Hey all, sorry you are being hit by this -- a couple engineers at FB are actively working on open sourcing our internal fix for this, but it's not a simple change so unfortunately it'll take a while longer. Tentatively, they are hoping to have it open sourced by the end of the year.

@astreet Thanks for the info! I eagerly await the fix!
I wonder if there is some temporary hack I can do with css to imitate this? The fix mentioned by @yonatanmn doesn't cover all cases.

@bsinc Thank you!

I would also be interested in a temporary hack to fix this problem.

Try bsinc's hack. It worked for me.

It doesn't work for me because I basically have a grid of circles.

Added a product pain: https://productpains.com/post/react-native/add-overflow-support-to-android
Everyone upvote so FB knows how much we want it :)

@astreet any timeline on open sourcing the fix?
Many thanks!

No timeline right now but work is still in progress. I reopened this issue because there were some dupes with less info and it's less spam to just have this issue open.

I solved this by setting zIndex: 100 on the component, now overflow: hidden works for me on android!

@saberking @julienvincent Yeah not sure how that works.
Isn't this going to be fixed in Android: Nodes in RN 0.41? @lacker
https://github.com/facebook/react-native/tree/master/ReactAndroid/src/main/java/com/facebook/react/flat

@madjam002 My guess is that nodes won't immediately fix it, but it's probably the start

@julienvincent 0.40.0. I still have not managed to get borderRadius to work at all, but being able to use overflow hidden for other stuff is still really useful. And yes, I was amazed when that solved the problem. If you like i can make an example

@saberking This issue is specific to border-radius + overflow: hidden. The overflow:hidden property without any border-radius was never broken.

@julienvincent OK maybe it is not broken but nowhere in the docs does it mention that you have to set zIndex in order to use overflow: hidden. Should I raise this as a separate issue?

That also sounds rather unlikely. It's more likely you are not using overflow: hidden as intended.

      <Animated.View style={{
            top: this.state.currentOuterTopOffset,
            overflow: 'hidden',
            ...this.props.style
          }}
          onStartShouldSetResponder={() => true}
          onMoveShouldSetResponder={() => true}
          onResponderGrant={this.responderGrant.bind(this)}
          onResponderMove={this.responderMove.bind(this)}
          onResponderRelease={this.responderRelease.bind(this)}>
        <Animated.View style={{ top: this.state.currentInnerTopOffset }}>
          {this.props.children}
        </Animated.View>
      </Animated.View>

zIndex must be also be set for overflow: 'hidden' to work here. It definitely seems like a bug to me

I would suggest making a vanilla reproduce repo to demonstrate the problem you are experiencing and then opening an issue.

On RN 0.41.2 you can try to add
import com.facebook.react.uimanager.UIImplementationProvider;
import com.facebook.react.flat.FlatUIImplementationProvider;
and
@Override
protected UIImplementationProvider getUIImplementationProvider() {
return new FlatUIImplementationProvider();
}
in ReactNativeHost class.

Since the view is not clipping the child image, I apply a border-radius to my image as well:
<View style={styles.imageContainer}>
  <TouchableOpacity onPress={props.onChangeProfilePhoto}>
    <View style={styles.changeProfileButton}>
      {profile.photo
        ? <Image
          style={styles.image}
          source={{ uri: profile.photo }} />
        : <Icon
          name='person'
          style={{ fontSize: 60 }} />
      }
    </View>
  </TouchableOpacity>
</View>

const styles = StyleSheet.create({
  ...,
  imageContainer: {
      alignItems: 'center',
      justifyContent: 'center',
      backgroundColor: ColorDefaults.background.darkerGrey,
      flex: 0.25,
  },
  image: {
      width: 98,
      height: 98,
      borderRadius: 50,
  },
  changeProfileButton: {
      width: 100,
      height: 100,
      justifyContent: 'center',
      alignItems: 'center',
      borderRadius: 50,
      borderWidth: 2,
  },
  ...,
};

@ryanhomer it is not a solution. Only a workaround in 5% of the case.

Here is my reproduction of the problem: https://snack.expo.io/ByMfo50hx

If just want to clip overflow sub views, you can set removeclippedsubviews to true with overflow: 'hidden' and it would work.

@desmond1121 Can you modify my snack above and show you can do it with removeclippedsubviews?

@henrikra Sorry for my misunderstood,  removeClippedSubviews would remove out bound sub views but not clip in bound views.. so definitely it's not work with your problem.

@bsinc  it's so smart . Just wanted to add a little advice , never use borderRadius as props (i repeat props , not style) in Image , like
     <Image source={some source} borderRadius={50} /> .

   Just Never !!

Using in scrollview will crash the app on repeated scrolling , and you will ruin the day finding where the bug is  , even debugger won't tell you whats wrong . Its may be a out of memory problem . If its not within ScrollView or ListView (not sure), its fine . I almost ruined 4 hours finding the bug . Use the @bsinc  trick instead .

+1 for this to have on android

+1

+1

Please upvote on product pains (https://react-native.canny.io/feature-requests/p/add-overflow-support-to-android) or upvote this issue rather than spamming "+1" comments.
@mica16 @diegorodriguesvieira

I'd just like to throw in another data point for setting zIndex as a workaround (from @saberking's post). I'm working with sprite sheets and ran into the issue where the entire sprite sheet was being shown on Android (worked fine for iOS).
render() {
  return (
    <View style={styles.container}>
      <View style={styles.imageWrapper}>
        <Image
          style={styles.image}
          source={{uri: [...]}}
        />
      </View>
      <Text style={styles.text}>Without zIndex: 1</Text>
      <View style={styles.imageWrapper2}>
        <Image
          style={styles.image}
          source={{uri: [...]}}
        />
      </View>
      <Text style={styles.text}>With zIndex: 1</Text>
    </View>
  );
}

var styles = StyleSheet.create({
  container: {
    backgroundColor: '#f9f8f5',
    flex: 1,
    padding: 20,
    alignItems: 'center'
  },
  imageWrapper: {
    overflow: 'hidden',
    width: 86,
    height: 86
  },
  imageWrapper2: {
    overflow: 'hidden',
    width: 86,
    height: 86,
    zIndex: 1
  },
  image: {
    width: 196,
    height: 86
  },
  text: {
    paddingTop: 10,
    paddingBottom: 30
  }
});

This will render:

Verified on a Nexus 5 with Android 7.0. Note that you don't need to specify as high a value as 100. Specifying a zIndex of 1 was sufficient to solve the issue for me.

This is a must have. Much of UI nowadays is based on rounded corners with subclipping.

Temporary workaround is positioning multiple "absolute" views with a solid border color on top of the image (using z-index) to fill in the gaps. Works if your background is a solid color.

<View style={{height:215, alignItems:'center', flex:1}}>
    <View style={{position:'absolute', height:215, width:deviceWidth*0.95, borderWidth:10, borderRadius: 5, borderColor:'white'}}></View>
    <View style={{position:'absolute', height:215, width:deviceWidth*0.95, borderWidth:10, borderRadius: 20, borderColor:'white'}}></View>
    <Image
        resizeMode="cover"
        style={{height:215, width:deviceWidth*0.9, zIndex:-1}}
        source={this.props.image}
    />
</View>

Yesterday I suddenly discoveree a simple solution . Just added backgroundColor in Image style and added borderRadius and it worked liked magic. Tried it on android . May be works on IOS too

@yonatanmn if borderRadius is 5?

@PARAGJYOTI this is not a solution. look at https://github.com/facebook/react-native/issues/3198#issuecomment-286122690 you are refering to the same workaround.

There isn't much more to be said in regard to this issue. Could we please lock conversations on this thread until further development.

@wollld - what about it? should work as far as I see

+1

+1

Please....
Stop +1 ... Upvote it here: https://react-native.canny.io/feature-requests instead.
Stop spaming workarounds, they dont solve the issue, feel free to create a PR to fix it instead.
Could we please lock conversations until the issue is solved or there exists a PR to solve it?
@astreet @hramos

Hi there! This issue is being closed because it has been inactive for a while. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. Either way, we're automatically closing issues after a period of inactivity. Please do not take it personally!
If you think this issue should definitely remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release candidate? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.

If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contribution guide, and feel free to hop into #react-native if you need help planning your contribution.

This is fixed with D5917111.


bins command messes up if only a single value is present like 0

xsv map 'count(links, "twitter.com")' tcount lemonde-internet-slim.csv | xsv bins -s tcount | xsv hist


Run app, config, and service resources in the root run context

This way, the restart notifications take action at the end of the Chef run
instead of at the end of the dnsmasq_local resource's child run context.

Coverage remained the same at 100.0% when pulling 4bd2893575f181e5d3528b418416798939f2457b on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling 4bd2893575f181e5d3528b418416798939f2457b on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling 4bd2893575f181e5d3528b418416798939f2457b on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling 4bd2893575f181e5d3528b418416798939f2457b on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling 4bd2893575f181e5d3528b418416798939f2457b on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling 4bd2893575f181e5d3528b418416798939f2457b on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling 4bd2893575f181e5d3528b418416798939f2457b on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling 4bd2893575f181e5d3528b418416798939f2457b on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling 4bd2893575f181e5d3528b418416798939f2457b on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage decreased (-0.9%) to 99.127% when pulling 724b50f0e33c44fc707aec0713cef8718744c2a1 on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling 336f70c47eadab68a3b0c64e36227c921b10c208 on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling 336f70c47eadab68a3b0c64e36227c921b10c208 on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling 336f70c47eadab68a3b0c64e36227c921b10c208 on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling 336f70c47eadab68a3b0c64e36227c921b10c208 on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling 336f70c47eadab68a3b0c64e36227c921b10c208 on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling 336f70c47eadab68a3b0c64e36227c921b10c208 on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling 336f70c47eadab68a3b0c64e36227c921b10c208 on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling 336f70c47eadab68a3b0c64e36227c921b10c208 on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling 336f70c47eadab68a3b0c64e36227c921b10c208 on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling bad68e8e913ae3d0da436a2f261bc8e15d3169ec on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling bad68e8e913ae3d0da436a2f261bc8e15d3169ec on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling bad68e8e913ae3d0da436a2f261bc8e15d3169ec on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling a04bb400954199e6dd0c1eeeb2589839af8a5a78 on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling a04bb400954199e6dd0c1eeeb2589839af8a5a78 on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.

Coverage remained the same at 100.0% when pulling a04bb400954199e6dd0c1eeeb2589839af8a5a78 on jdh-run-contexting into c4909a35b0344a70d7dbfdf9220746c303903502 on master.


Update http4s-client, http4s-dsl, ... to 0.21.23

Updates

org.http4s:http4s-client
org.http4s:http4s-dsl
org.http4s:http4s-server

from 0.21.19 to 0.21.23.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch

Superseded by #37.


Running the tests gives a segmentation fault.

@rizar I followed the instructions in README to run the tests. However, I am getting seg faults for all the tests I tried. My guess is that code is not getting the correct data. Are there additional steps I need to take to properly set the run time paths?

By "tests", do you mean the content of "tests" folder, or do you mean the actual experiments? The former was included in the repository by mistake and should be deleted.

I am sorry for the confusion. I mean the actual experiments billion words and ted used in the paper.

OK, I see. Can you please provide a bit more information about the exact
commands that you ran?
On Mon, 19 Dec 2016 at 21:01 Shivam Vats<EMAIL_ADDRESS>wrote:

I am sorry for the confusion. I mean the actual experiments billion words
and ted used in the paper.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rizar/actor-critic-public/issues/1#issuecomment-268133363,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAn8Yi2riOFcxLm6T9Q6e6njSzz-MKeFks5rJzcSgaJpZM4LQih3
.

source env.sh.
cd exp/billion_words
 $LVSR/bin/run.py train autoencoder3 configs/autoencoder3.yaml
cd ../ted
./create_dataset.sh (I first set the path to MOSES)
$LVSR/bin/run.py train ted12 config/ted12.yaml

Error:
Segmentation fault```

Hold on, did the step 3 work for you? You should've downloaded the data first.

https://github.com/rizar/actor-critic-public/blob/master/exp/billion_words/README.md is now updated

I did download the data (I put it in lvsr/datasets though).
I still get the same error after putting the data in FUEL_DATA_PATH. I am surprised to see a seg fault with Python. Are you using C wrappers in your code? From fuel's docs, it seems that it needs data as  a .hdf5 file. Does it need to be converted to fuel's format?
Also, in the configs for ted the data filename is set as TED/de-en/ted.h5 (I would expect it to be ted.h5).

No, their is no C code in our implementation. But libhdf5 could trigger
one. But let's focus on one experiment. Do you have a segfault when you are
trying to run on TED or on billion_words?
On Wed, 21 Dec 2016 at 13:27 Shivam Vats<EMAIL_ADDRESS>wrote:

I did download the data (I put it in lvsr/datasets though).
I still get the same error after putting the data in FUEL_DATA_PATH. I am
surprised to see a seg fault with Python. Are you using C wrappers in your
code? From fuel's docs, it seems that it needs data as a .hdf5 file. Does
it need to be converted to fuel's format?
Also, in the configs for ted the data filename is set as TED/de-en/ted.h5
(I would expect it to be ted.h5).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rizar/actor-critic-public/issues/1#issuecomment-268601668,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAn8Yof13Ky_9ziHFe0I6XT0m9OYGTMWks5rKW-ggaJpZM4LQih3
.

Both.

OK, let's focus on billion words then. In order to help you, I would need
to know at which line of lvsr/main.py the segmentation fault happens.
On Wed, 21 Dec 2016 at 13:34 Shivam Vats<EMAIL_ADDRESS>wrote:

Both.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rizar/actor-critic-public/issues/1#issuecomment-268603602,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAn8YoyIGF1WO2OazlRzwl3ZfnqvJHJUks5rKXFDgaJpZM4LQih3
.

I figured out the issue. Importing fst is leading to a seg fault. I tried reinstalling pyfst and openfst but it doesn't work. The code should work without fst, right?

Yes, this an obsolete import. Feel free to remove it. I will remove it
later as well. Thanks for the heads-up!
On Wed, 21 Dec 2016 at 14:47 Shivam Vats<EMAIL_ADDRESS>wrote:

I figured out the issue. Importing fst is leading to a seg fault. I tried
reinstalling pyfst and openfst but it doesn't work. The code should work
without fst, right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rizar/actor-critic-public/issues/1#issuecomment-268621297,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAn8YtAcvcARoC5oFt02ky7S03NNC9aTks5rKYJogaJpZM4LQih3
.

Thanks a lot for your prompt help! :)
Would it be possible for you to give me a rough estimate of the training time? I am using a GTX 960, with 4GB memory.

I am afraid this amount of memory might not be sufficient. On a K80 the
pretraining (first actor, then critic) takes about 36 hours, and then you
can run the main stage for as long as you want...
On Wed, 21 Dec 2016 at 14:56 Shivam Vats<EMAIL_ADDRESS>wrote:

Thanks a lot for your prompt help! :)
Would it be possible for you to give me a rough estimate of the training
time? I am using a GTX 960, with 4GB memory.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rizar/actor-critic-public/issues/1#issuecomment-268623261,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAn8YqIz_JcTVDlrZPr9RpHxkfghjYhxks5rKYRQgaJpZM4LQih3
.

Hi @rizar @shivamvats I have been following the same procedure listed in this issue but I do however get a valueError when running the billion_word training experiment ($LVSR/bin/run.py train autoencoder3 configs/autoencoder3.yaml) which says "if target_source != 'text': raise valueError." (from ..lvsr/datasets/text.py) I would like to know the python version this experiment was run on and any other tips I can use to debug my error?


[Change]: Questionnaire summary: Containerisation

Summary
The percentage splits of responses to the SATRE survey questions on support for containers a TRE.
Source
SATRE specification survey responses
Detail
Survey responses:
11.1.a. Support for running containers (e.g. Docker, Podman, Apptainer)

No opinion
Not important
Nice to have
Important
Essential

23.81%
2.86%
19.05%
30.48%
23.81%

Where
TRE optional features
Proposal
Container support in TRE optional features as those who thought it essential are in the minority.
Who can help
Anyone

I wonder if the type of containerization being used is useful to frame this discussion?  i.e. genuinely not requiring root access  (Podman) as this is very likely to not be granted to users.

Root access is only a major problem on shared VMs, if each user has their own VM it doesn't matter as much. user-mod containers reduce the risk further, but I wouldn't say it's necessary.

What would these containers be allowed to be used for?  To enable scripts and loading of data into an approved (legal, ethical, funded) research project environment seems an essential capability.  Lots of research projects will need to bring data in that may be regularly updated or stored as large files somewhere.  being able to schedule automatic linkage using approved login user credentials to a project TRE seems essential.
Providing a VM for the user to do whatever they want seems totally not appropriate.  Why would that be needed in a trusted research environment.  Is this another meaning for "compute" power?  If so then cloud native solutions provide this.
So do we need to collect what people are wanting these Containers for?

I think it would be nice to add support for running containers as an optional under the software tools capability.

So do we need to collect what people are wanting these Containers for?

@manics do you have an answer for @crickpetebarnsley point here? Would the idea be that the TRE has access to a subset of approved (e.g. Docker) containers from an (internal?) container registry, similar to how we might have an approve list of (e.g. Python) packages for a TRE?

Containers are worth highlighting because they're a self-contained way of making software available, and they're particularly useful for reproducible pipelines. However, ultimately they're just another way of packaging software, so the considerations that apply to making other applications or software repositories available are also applicable to containers. This might be individual container images that are made available in an internal container registry, or it might be making a subset of public containers available.
I don't think we need to explicity ask what people need them for, that can be handled as part of deciding what software or other tools to make available in a TRE.


Fixed small typo

Description of the change:
There was a small typo which needed fixing.
.....SMTP setup <for for> easy transactional email Integration... is changed to:
.......SMTP setup <for> easy transactional email Integration.......
Reason for the change:
It was a typo which had to be fixed.
Link to original source: https://sendgrid.com/docs/for-developers/partners/magento/

Hello @geekysrm,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


Unlimited XP Bug with turned on Electromagnet

Found a similar issue here: https://github.com/SleepyTrousers/EnderIO/issues/1658
Hovering at one certain point gives the player unlimited XP. There are no XP-related blocks nearby.
Take a look here: http://youtu.be/g2XolyKFb3E
FTB Infinity 1.7 (v1.8.2) Mod Pack
Minecraft Version 1.7.10
Forge <IP_ADDRESS>8
EnderIO-1.7.10-<IP_ADDRESS>1

I'm at a loss on this issue, just as I was before. The magnet shouldn't be causing anything like this.

Hey,
the bug is gone now. Two changes were made:
The server got an update to the newest FTB Infinity Modpack (v1.9) and therefore a sever restart.
Second, there was a quarry hole with water source blocks on top (and of cource floating water) next to it (not the one i am hovering in in the video,) which has been filled.
Greets
duff

I have the latest Ender io and am still experiencing this issue. Also massibe server lag with the magnet and using mekanism Atomic disassembler.
EnderIO-1.7.10-<IP_ADDRESS>7_beta
EnderCore-1.7.10-<IP_ADDRESS>_beta
I'll be creating a new issue.


Python file with specific name not syntax highlighting

We have a file that is not receiving diff syntax highlighting in a pull request.
This is a new file and the file's path is:
staticdhcpd/unit_tests/staticDHCPd/extensions/official/test_httpdb.py
(It's under a private repo)
It seems that any changes to the files name (a few tried are test_httpdb2.py, test_httpd.py, httpdb_test.py) make it get syntax highlighting on the diff.
Syntax highlighting is correctly applied when just viewing the file (not as a diff).
Any ideas?

Any chance you could reproduce the bug on a public repository? I don't see any obvious reason why it wouldn't be highlighted so it's going to be difficult to debug if we can't reproduce :/

Closing as stale.


go.mod: bump rest of otel libs

Bump these two OTEL libs to be like the rest of the OTEL libs. We had a strange issue where service.name was not set even though it was in environment variables. ~Bumping these libs fixes the problem.~

Actually, just bumping the libs doesn't fix the problem. I have added this commit https://github.com/thanos-io/thanos/pull/6447/commits/de1b25df96ba51b8b571610b564c3c671ab9cb7e which only sets service.name if something has been provided through the YAML config. Let me also add a test case for this.

Added a test case that currently doesn't work on main.


Limit Parquet Page Row Count By Default to reduce writer memory requirements with highly compressable columns

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

There is a discussion on the mailing list about default settings for page sizing, and one of the suggestions is that the page row count limit should be enabled by default, where currently it is not. Row groups are limited to 1M rows by default, there is some suggestion pages should be limited to 20,000.
Creating this issue to track
Describe the solution you'd like

Describe alternatives you've considered

Additional context

Maybe we can use some data from @XiangpengHao 's analysis in https://github.com/apache/arrow-rs/issues/5770 as part of this decision

Specifically I think @tustvold 's proposal is to change the default value of this
https://docs.rs/parquet/latest/parquet/file/properties/struct.WriterProperties.html#method.data_page_row_count_limit
Currently it seems there is no row count limit (limit is usize::max):
https://github.com/apache/arrow-rs/blob/5a24119f6634d66b83556ecbcfbd54ae3f1620c7/parquet/src/file/properties.rs#L356
Note that @XiangpengHao  hit this when trying to write large numbers of rows: https://github.com/apache/arrow-rs/issues/5828

I think setting a default page size to 20k is a good idea -- the default "no limit" can result in substantial memory usage on the writer, that people seem to think is a limit of the parquet format itself.
Another potential default value per page would be 64K (the size used in BtrBlocks)

We just found another example in InfluxDB where setting the data page limit to 64k dramatically lowered the amount of memory written to the parquet writer

label_issue.py automatically added labels {'parquet'} from #5957


Update gatsby-node.ts

What if we pass the whole object so its much easier to query other properties rather than just the fileAbsolutePath?

Your code creates relatedFileAbsolutePaths field with a self remark node, so it does not work as you intended and I reject this PR.
However, you are right, and that's exactly what I felt should be fixed!
I fixed this issue today and you can try<EMAIL_ADDRESS>

Sorry, I found that the above implementation creates a critical performance issue of the gatsby graphql server, so I published v1.0.1 to reverte the plugin to v0.0.8...


TreeView - Programmatically Set Item Visual Bug

Bug type
Component
Component name
MudTreeView
What happened?
When you programmatically select an item, the selected style doesn't change.
Expected behavior
The selected color should apply new selected item.
Reproduction link
https://try.mudblazor.com/snippet/QEmwvlwAzIPhRoAX
Reproduction steps

...
Relevant log output
No response
Version (bug)
Latest
Version (working)
No response
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

FYI, programmatic selection is removed in v7 in #8661. You have to select via @bind-SelectedValue(s)


Feat(Storage): add samples for soft-delete and restore buckets

Added Storage Samples:

Get a Soft Deleted Bucket
List Soft Deleted Buckets
Restore a Soft Deleted Bucket

@thiyaguk09  please change the title of PR Feat(Storage) ==> samples (Storage)

@thiyaguk09 please change the title of PR Feat(Storage) ==> samples (Storage)

Thank you for the approval. This repository is exclusively for PHP samples, so we typically follow the 'feat', 'fix' conventions. I am adhering to the existing PRs that have been raised.
https://github.com/GoogleCloudPlatform/php-docs-samples/pulls?q=is%3Apr+is%3Aclosed


BasicOAuth example code reuses existing token from bot service

Bot Info

SDK Platform: .NET
SDK Version: <IP_ADDRESS>
Active Channels:
Deployment Environment: Local Emulator

Issue Description
When using the basic oauth sample with the emulator, previous sign ins are used rather than re prompting for credentials. Once I have logged in the bot framework is continually retrieving the same token, even when I create a new chat session. This also seems to be the case for the deployed bot at well.
          var oauthClient = activity.GetOAuthClient();
                var token = await oauthClient.OAuthApi.GetUserTokenAsync(activity.From.Id, ConnectionName).ConfigureAwait(false);

this code snippet seems to the the culprit, the activity.From.Id is always set to "default-user" when the conversation starts up. So new conversations are finding the token of the previous user. I have found outside the emulator I can control this by setting a new userid on the initialization of the chat client, but this seems a little too risky to depend on the client for providing the userid to authenticate with. Am I missing something here with how the userid should be used? It seems like once the user logs in anyone could run the javascript to initialize a new chat window with just knowing another user's id.
I could initialize with a userid + generated value when the page is loaded so someone from any machine can't just try a userid, but then if the user reloads the chat page, they would have to sign in again - which I don't want either. I'm not sure how best to proceed.
Reproduction Steps

Configure bot for auth
Use the BasicOAuth sample
Once authenticated create new emulator sessions

Expected Behavior
The ability to create new user sessions
Actual Results
Existing token is used and the user is "signed in" regardless of who the token belongs to.

So I did find this: https://stackoverflow.com/questions/44906266/ms-bot-framework-emulator-chang-user-id
Where you can change the default settings of the emulator, which is cumbersome, but doable, I'm curious though how this should be working in the bot service in terms of keeping track of the tokens without just trusting what the user passes in.

If you need to restart a session with the emulator, you can send ' /deleteProfile' and it should clear out UserData.
UserId should be kept private (hashed, or encrypted).  It is specific to the user.

This is more of an emulator issue than a bot builder issue.  Please post this to the emulator repository if you require further assistance: https://github.com/Microsoft/BotFramework-Emulator/issues/new


autocomplete / autosuggest

I've started to look into autocompletion and autosuggest for p5 functions. The Khan Editor handles this really well:
http://khan.github.io/live-editor/demos/simple/
They're also using the ACE editor.
If you type ScratchpadAutosuggestData into the console you can see how the data is formatted, from this document
It's sorta similar to the data.json that p5 generates with grunt yuidoc for the reference pages.
A difference is that their name is "line(x1, y1, x2, y2)" rather than just "line," and they include a link to an example (in our case this would be the reference page at p5js.org). But we can probably generate both of those when parsing our data.json:
        {
            "file": "src/core/2d_primitives.js",
            "line": 198,
            "description": "Draws a line (a direct path between two points) to the screen. The version\nof line() with four parameters draws the line in 2D. To color a line, use\nthe stroke() function. A line cannot be filled, therefore the fill()\nfunction will not affect the color of a line. 2D lines are drawn with a\nwidth of one pixel by default, but this can be changed with the\nstrokeWeight() function.",
            "itemtype": "method",
            "name": "line",
            "params": [
                {
                    "name": "x1",
                    "description": "the x-coordinate of the first point",
                    "type": "Number"
                },
                {
                    "name": "y1",
                    "description": "the y-coordinate of the first point",
                    "type": "Number"
                },
                {
                    "name": "x2",
                    "description": "the x-coordinate of the second point",
                    "type": "Number"
                },
                {
                    "name": "y2",
                    "description": "the y-coordinate of the second point",
                    "type": "Number"
                }
            ],

sounds great!

The Tern library is another option
http://ternjs.net/doc/manual.html#protocol
It could be added to a user's text editor of choice, and built into ACE with something like this:
http://sevin7676.github.io/Ace.Tern/demo.html
Seems like we'd generate data in this format: http://ternjs.net/doc/manual.html#typedef
I'll keep playing with this

Autocomplete is a terrific tool and somewhat of a must for any good editor.  However, it's also particularly tricky to balance its features with the clean simplicity of a beginner experience.  I'm excited for the possibilities you mention above and happy to chat about how autocomplete works in Processing (we went around in many circles about integrating it) if that can be of help.

Does anyone have a Tern definition json file for p5 yet? would it be easier to generate it with tern's condense tool or through yuidoc?


The displayed waveform and audio cannot be corresponding when dealing with long audio file

Hello!
I meet a problem using label-studio. When I import long audio file (about 25-min long, 25MB), the displayed waveform and audio cannot correspond to each other after I adjust the zoom axis below the timeline. But after I compress the audio file by adopting lower bit rate, the issue gets solved.
So I wonder why this problem emerges, and by the way, would there be any other problem if I compress the audio file?
Thank you very much for your help!

@wanderin-earth Could you send us an example of source audio and converted audio?

@wanderin-earth Could you send us an example of source audio and converted audio?
I think there won't be other problem if you use compressed audio.

I've sent you the files by email. Thanks!

@wanderin-earth Could you check the master branch of LS now, I hope we've fixed this issue (in this PR https://github.com/heartexlabs/label-studio/pull/806)


Section "CMake Project + fetch content " in README.md

Query/Question
The section "CMake Project + fetch content" suggests to use CMAKE_FETCH_CONTENT to integrate the sdk in my project.
We dont have vcpkg. When doing as instructed:
FetchContent_Declare(
    azuresdk
    GIT_REPOSITORY      https://github.com/Azure/azure-sdk-for-cpp.git
#    GIT_TAG             2850c5d32c8a86491b49e801433b8f186fa81745
    GIT_TAG             c3dc7321577fa99ddbf473fdf2a9454c2e388702
)

FetchContent_GetProperties(azuresdk)
if(NOT azuresdk_POPULATED)
    FetchContent_Populate(azuresdk)
    # Adding all Azure SDK libraries
    # add_subdirectory(${azuresdk_SOURCE_DIR} ${azuresdk_BINARY_DIR} EXCLUDE_FROM_ALL)
    # Adding one Azure SDK Library only (Storage blobs)
    add_subdirectory(${azuresdk_SOURCE_DIR}/sdk/storage/azure-storage-blobs ${azuresdk_BINARY_DIR} EXCLUDE_FROM_ALL)

    #add_subdirectory(${azuresdk_SOURCE_DIR}/sdk/core/azure-core ${azuresdk_BINARY_DIR} EXCLUDE_FROM_ALL)
endif()

I get the error:
-- Configuring incomplete, errors occurred!
Vcpkg integrate step.
CMAKE_TOOLCHAIN_FILE is not defined. Define it for the user.
AZURE_SDK_DISABLE_AUTO_VCPKG is not defined. Fetch a local copy of vcpkg.
Vcpkg commit string used: 94ce0dab56f4d8ba6bd631ba59ed682b02d45c46
No transport adapter was selected, using libcurl as the default option for POSIX.
CMake Error at /......../_deps/azuresdk-src/sdk/storage/azure-storage-blobs/CMakeLists.txt:37 (find_package):
  By not providing "Findazure-storage-common-cpp.cmake" in CMAKE_MODULE_PATH
  this project has asked CMake to find a package configuration file provided
  by "azure-storage-common-cpp", but CMake did not find one.

  Could not find a package configuration file provided by
  "azure-storage-common-cpp" (requested version 12.3.0) with any of the
  following names:

    azure-storage-common-cppConfig.cmake
    azure-storage-common-cpp-config.cmake

  Add the installation prefix of "azure-storage-common-cpp" to
  CMAKE_PREFIX_PATH or set "azure-storage-common-cpp_DIR" to a directory
  containing one of the above files.  If "azure-storage-common-cpp" provides
  a separate development package or SDK, be sure it has been installed.

-- Configuring incomplete, errors occurred!

When just adding azure-core it works. My try to include the azure-storage is even smaller as the sample in REAMDE.md that includes the whole sdk.
Many thanks in advance for clarification and support.
Why is this not a Bug or a feature Request?
I assume I did misunderstood or overlooked something. Maybe a clarification in the README.md would be helpful for others as well.
Setup (please complete the following information if applicable):

OS: 5.15.0-67-generic #74~20.04.1-Ubuntu SMP Wed Feb 22 14:52:34 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
IDE : QtCreator
tried with several versions (e.g. #2850c5d32c8a86491b49e801433b8f186fa81745 or #c3dc7321577fa99ddbf473fdf2a9454c2e388702)

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ x ] Query Added
[ x ] Setup information Added

@RickWinter  we need to check if the getting-started steps in README has stopped working for some reason.


Limit creeper explosion reconstruction to owned areas

I like the reconstruction capability for creeper explosions, but I want them to be normally destructive when away from the places where players own their homes.
I don't know if the answer is an object that you have in your home that marks a zone around it as 'reconstructable', or using the existing region system, but it would be a really nice feature.
I haven't spent any time looking into the implementation, to see how hard this is yet.

I agree this would be a nice feature.
Regarding implementation: If you want a setting per region-group, it's going to be quite some work. If you are fine with a global toggle then it's relatively simple and you can mostly copy existing code. Similar to the portals <-> regions integration, the admin module would need a callback like this one in portals, but slightly adjusted so it can check whether a coordinate is in a region.
https://github.com/oddlama/vane/blob/e71365b547f5b0dc2be48aae093a1785e37d99d9/vane-portals/src/main/java/org/oddlama/vane/portals/Portals.java#L352-L359
In the initialization of vane-regions, it will provide that callback to vane-admin if necessary (we want regions to softdepend on admin not the other way around). That can also be mostly copied from the RegionPortalIntegration.java as a new RegionAdminIntegration.java.
Then a single config needs to be added to vane-admin like only_in_regions which determines where the creeper rebuild applies. And in the explosion events just remove all blocks that are not in a region if that toggle is set.


cpu: Fix Ruby/x86 pio port connections

Fixes #1033
In the BaseCPU object _uncached_interrupt_response_ports is a class variable, not an instance variable. #1004 changed the explicit self._uncached_interrupt_response_ports to use extend. This caused the list of ports to be extended for all cores, which caused problems when using a system with more than 1 core.
This reverts the extend part of the change, but keeps the rest.
Change-Id: I6dc7d6da6763048d82960229d34933a3a2ac36e0

Just FYI @nmosier. This took a bit to track down.

Thanks for fixing this! Sorry I was MIA (submission deadline…) and for breaking it in the first place.


Approvals screen: Request type "Review" for SIAs - please add Name and Focus fields when you go to the View screen

When using the Approvals screen and the Achievement Pathway is any "SIA" and the Request Type is "Review": when you click the "View" hyperlink, the resulting screen shows only the Review questions and answers of the SIA but does not show the "Name" and "Focus" fields  for the SIA.
This makes it almost impossible to know what the youth was doing for the SIA for when doing the Review.
As a starting point could we please have the "Name" and "Focus" fields added for the SIA?
If it is possible when doing this change, it would be great to have all the fields for the SIA on the screen, but this is not as important as having the Name and Focus fields.

I just saw that the screen we need here is already in the system in the youth Basecamp > Special Interest Areas > Awarded section > click the View hyperlink on an SIA that has been awarded and you get the whole SIA screen including name, Goals, How, and Who headings.
That screen could be improved by adding in a "Review" title above the review questions and answers that are there.

Hi,
I believe you are raising an issue regarding Terrain, not Topo. I don't do anything with Terrain, follow the support/help info in Terrain.
Simon.

Darn Simon,
You are exactly right.  I thought that i'd tapped straight into the
developers for Terrain via GitHub!  Bummer, and thanks for letting me
know.  Please delete my requests.
Cheers,
Andrew
On Mon, 9 Sept 2024 at 13:35, Simon Batson @.***> wrote:

Hi,
I believe you are raising an issue regarding Terrain, not Topo. I don't do
anything with Terrain, follow the support/help info in Terrain.
Simon.
—
Reply to this email directly, view it on GitHub
https://github.com/NomisNostab/Topo-Blazor/issues/179#issuecomment-2337044814,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BLCXB2L3RE4PM7EYIKDBIWDZVUJQBAVCNFSM6AAAAABNZOWDDGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZXGA2DIOBRGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Sorry Simon, please delete these 3 request as you are right, they are for Terrain. :P

Closed as these are Terrain issues


Validator should accept params on content-type

application/ld+json;charset="..." is still correct, but not equal to the string being tested for.
The test should use startswith() rather than ==

Was fixed by ef77436da7412195cc0901a87dccf32f20fb3510


[App Development] Website updates

The website for the WG currently shows the old meeting cadence of 1st and 3rd Wednesday of every month. That should be updated to reflect the new schedule.
There is also a link for WG issues that filters by the wg-app-development label, but clicking through shows no open issues.  The label that's being used (e.g. here) seems to be https://github.com/cncf/tag-app-delivery/labels/wg-appdev

I can take this one if someone wants to assign it to me.

was fixed with https://github.com/cncf/tag-app-delivery/pull/772


MODIFYING THE SIMULATOR

HEY THERE PLEASE IT IS VERY IMPORTANT I WOULD LIKE TO KNOW IF IT IS POSSIBLE TO ADD "OCCUPANCY GRID MAP" around the ROBOT ? HOW , which file do i need to modify ? THANK YOU

Hey Dadwi,
have you tried grabbing the sensor information and build a map on your own?


Ability to exclude some directives contents

Is your feature request related to a problem? Please describe
This is mostly a feature request because it's related to openresty which adds additional directives to the nginx configuration, and I'm not sure that's a core concern for this organisation.
When using *_by_lua_bock directives the parser seems to get confused and adds configuration outside the block into the lua block.
Example config
    header_filter_by_lua_block {
      cors_allow_for([[some\.domain\.name$]])
    }
    include includes/common;

This generates JSON which looks like
...
                {
                  "directive": "header_filter_by_lua_block",
                  "line": 32,
                  "args": [],
                  "block": [
                    {
                      "directive": "cors_allow_for([[some\\.domain\\.name$]])",
                      "line": 33,
                      "args": []
                    },
                    {
                      "directive": "error_page",
                      "line": 2,
                      "args": [
                        "404",
                        "=",
                        "@error"
                      ]
                    },
                    {
...

where the error_page directive and subsequent directives come from the included file.
Describe the solution you'd like
To get a valid representation of the configuration.  The approach suggested here is to identify specific directives to exclude them from further parsing.  This means the offending directive will still appear in the output, but its block will be empty.  If these directives could be identified by command line options then the solution would work for as-yet unforeseen directives.
Describe alternatives you've considered
Fixing the parser to correctly process the *_by_lua_block directives (probably by turning them into strings).  I don't think this is a great option because any nginx addon could add a directive which could fall afoul of the same issue.
Additional context
Add any other context or screenshots about the feature request here.

It looks like the python version handles this already - https://github.com/nginxinc/crossplane/blob/master/crossplane/ext/lua.py

Apologies, this feature already exists - https://github.com/nginxinc/nginx-go-crossplane/blob/main/parse.go#L52

On reflection, this is a bug in the parser which is not solved by ignoring the directive.  When the directive is ignored, the parse will still eat the config and then exclude those other directives/blocks from the payload!

So I haven't actually addressed the bug here, it still stands.  A config file which illustrates the issue I have is
http {
   server {
      header_filter_by_lua_block {
         cors_allow_for([[invest\.vic\.gov\.au$]])
      }
      error_page 404 /handler;
   }
}

Fixing the bug would only partially solve my problem, which is that I want to keep the lua code, but not have it parsed into the Directive structure.
I've implemented another option for directives that should be handled similarly to the ignored directives.  The difference is that we don't discard their content, instead we collect it and put it in the comment field for the directive in question.
This appears to resolve the immediate issue I'm having, but it's not MR worthy - https://github.com/nginxinc/nginx-go-crossplane/compare/main...asharpe:nginx-go-crossplane:skip-block-contents?expand=1

This moves the logic into the lexer and provides a much better result for what I want.  My current thinking is that *_by_lua are already handled as strings, *_by_lua_file are just directives with args, it's just *_by_lua_block that's causing an issue.
This change allows the lexer to identify *_by_lua_block directives and capture the contents verbatim, thus turning them into an arg for the relevant directive.  The output of this suits me perfectly, it would be great to get some feedback from the maintainers on the approach here, and what might be required to get this to land!
https://github.com/nginxinc/nginx-go-crossplane/compare/main...asharpe:nginx-go-crossplane:by-lua-block?expand=1

The problem apparently is because the parser cannot detect end of lines, as we don't add the ";" after each directive.
Example that works:
		content_by_lua_block {
			tcp_udp_configuration.call();
		}

if you remove the ";" (which is right on by_lua blocks btw) it breaks

I am not sure if this is the same issue, but I have figured out that Lua parsers are not used by default, so I created some example here: https://gist.github.com/rikatz/2928938c6d354366e7b30e8cffc69bf1


refactor dns code to live in a custom Collector in collector.py #64

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

Comparison is base (37061af) 98.19% compared to head (3b79cbb) 96.79%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #66      +/-   ##
==========================================
- Coverage   98.19%   96.79%   -1.41%
==========================================
  Files           9       11       +2
  Lines        1110     1124      +14
==========================================
- Hits         1090     1088       -2
- Misses         20       36      +16

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


Alias do not work on controller

Hi,
I'm used to use annotations system to decrypt my ID's. If i wanna make a redirection (redirectToRoute), with a encrypt function service, i have alias bug (nzo_url_encryptor)
I'm on Symfony 4.
Thx a lot for your help.
Djb

I have to create an alias in my services.yaml to use UrlEncryptor in my controller
nzo_url_encryptor:
class:
Nzo\UrlEncryptorBundle\UrlEncryptor\UrlEncryptor
public: true
Everything it's ok !
Nice work ! thx !


Port AB player

More or less 1-1 port of Abyssbox's song player. The current issues that need to be fixed are:

The song player doesn't immediately resize itself upon load (this is an issue in AB)
The prompt code is kinda hacked in, so it doesn't account for the Frosted Glass preference (this is an issue in AB)
The X on the song layout prompt doesn't show up (this is probably a mistake I made during porting)

Fixes #153

@choptop84

Hola

So like, should I accept this review idk why I was pinged

Prob to review the code and see what's wrong with it (issues addressed)

does this mean you're able to see those buttons on the upper left corner of the editor?

does this mean you're able to see those buttons on the upper left corner of the editor?

No, the mobile buttons are not part of this pull request