Add module for jmh benchmarks

Can you explain in a little more detail what this is and why it's useful?

It is very convenient tool to that allows measure and analyze performance.
For instance, simple java -jar benchmarks.jar -t 8 allowed me to capture the performance of Beans.newInstance(Beans.class) under 8 concurrent threads (see https://github.com/cglib/cglib/pull/53)

Yup, I understand that jmh is for benchmarking.
Do you plan to add more benchmarks?
The framework is nice, but I'm hesitant to add it if all we're going to have is a benchmark for a single method.

I hope the suite will grow over time.
Even a single benchmark helps validating lots of PRs: #50, #51, #53, #54

I am not sure over the value of this benchmark. An absolute value does not give any indication as it is machine and runtime-context dependant. One would need to add some form of baseline to have a comparison value. Also, what does the flight recorder profile help here? Again, without a baseline, the results are dubious.

@raphw, the benchmark is like a unit test for performance.
For instance, if one adds a feature, the benchmark can be used to validate "before" and "after" performance.
If benchmark is missing, there is no way to figure out if the performance was broken or not.

Also, what does the flight recorder profile help here?

Flight recorder is for investigation purposes.
For instance, when you research "before and after", flight recorder is very helpful to identify "excessive memory allocation" kind of problems.
Does that answer your questions?

I very much understand the intention. But the test can only be run in two iterations, one before and one after a change is applied. These benchmarks need then to be run on the same machine under the exact same setup. Ideally, one would have some sort of baseline benchmark to capture this variation as JMH benchmarks should rather be interpreted by their ratio then their absolute value. I see that this pull request has a benefit but I would rather see such a baseline added to it.

Do you mean I should add a file with java -jar benchmarks.jar output to the PR?
I've never seen that in the wild.

No, not at all. Such a machine-dependant result would make no sense as a comparison value. A baseline benchmark is measured to capture the benchmark overhead to set it into proportion as the raw number of the benchmark does not mean a lot. For example, it is difficult to measure the performance a micro-operation such as adding an element to a set. To put the result into proportion I would need to add another benchmark that captures the noise of the machine to create a baseline number. If the result of the set operation yields a result that is only insignifficantly different, it means that the benchmark is probably not capturing the intended event:
@Benchmark
Object baseline() {
  return new Object();
}

@Benchmark
Object actualBenchmark(){
  Set<Object> set = new HashSet<>();
  set.add(new Object());
  return set;
}

I did however not yet find a good baseline for your proposed benchmark. Can you think of something?

Finally I got you.
Here's the baseline for BeansBenchmark
    @Benchmark
    public Object newInstance() {
        return new Beans();
    }

Anything else?

That was too obvious. Good enough! If you add it, I merge the benchmark.

I've rebased the PR.
Here's what I get out of java -jar benchmarks.jar -prof gc
Benchmark                                        Mode  Cnt     Score     Error   Units
BeansBenchmark.baseline                          avgt    5     3,280 ±   0,107   ns/op
BeansBenchmark.baseline:·gc.alloc.rate.norm      avgt    5    16,000 ±   0,001    B/op
BeansBenchmark.newInstance                       avgt    5   138,938 ±   8,258   ns/op
BeansBenchmark.newInstance:·gc.alloc.rate.norm   avgt    5   264,000 ±   0,001    B/op


unknown command "init" for "rportcli"

Hi,
Rportcli doesn't seem to recognize any commands besides "exec". I am trying to run the initial configuration after download and I got this error:
$rportcli  init
Error: unknown command "init" for "rportcli"
Run 'rportcli --help' for usage.
FATA[0000] unknown command "init" for "rportcli"

The same happens trying to override CONFIG_PATH:
$ CONFIG_PATH=/tmp/config.json rportcli init
Error: unknown command "init" for "rportcli"
Run 'rportcli --help' for usage.
FATA[0000] unknown command "init" for "rportcli"

Version:
$ rportcli version
Version:
BuildDate  2022-06-06 10:26:33.240667384 -0300 -03 m=+0.000358307
GoVersion  go1.14
GitBranch  refs/tags/v0.0.1pre1
GitCommit  f6c6d3f4c98805617153416a40b1c7a20bb8e5be
GoArch     amd64
GoOS       linux

Help:
$ rportcli help
Rport cli

Usage:
  rportcli [flags]
  rportcli [command]

Available Commands:
  exec
  help        Help about any command
  version     Print the version number of RportCLI

Flags:
  -h, --help      help for rportcli
  -v, --verbose   verbose output

Use "rportcli [command] --help" for more information about a command.

Exec --help:
$ rportcli exec --help
Usage:
  rportcli exec {{PATH_TO_SCRIPT}} [flags]

Flags:
  -h, --help   help for exec

Global Flags:
  -v, --verbose   verbose output

Is it a bug or am I missing something?
Thank you very much.

OK, I just realized I was installing a previous version of rportcli.
However, I still got errors when I try to run rportcli for the first time:
$ rportcli init -s http://localhost:9123 -l admin  -p ieWah2hai
WARN[2022-06-09T14:41:45.252090] unexpected end of JSON input
WARN[2022-06-09T14:41:45.253901] cannot unmarshal error response : unexpected end of JSON input
FATA[2022-06-09T14:41:45.253991] config verification failed against the rport: the specified item doesn't exist

Any ideas?

Ok, I just realized I had a previous version of rportcli. However, I still got errors when trying to init:
$ rportcli version
Version: 0.3.1
BuildDate  2022-06-09 14:45:24.357358293 -0300 -03 m=+0.001642948
GoVersion  go1.16.15
GitBranch
GitCommit
GoArch     amd64
GoOS       linux

$ rportcli init -s http://localhost:9123 -l XXXXXXXXXXXX -p XXXXXXXXXXXXXX
WARN[2022-06-09T14:46:02.744334] unexpected end of JSON input
WARN[2022-06-09T14:46:02.746780] cannot unmarshal error response : unexpected end of JSON input
FATA[2022-06-09T14:46:02.746833] config verification failed against the rport: the specified item doesn't exist

Ok, I just realized I had a previous version of rportcli. However, I still got errors when trying to init:
$ rportcli version
Version: 0.3.1
BuildDate  2022-06-09 14:45:24.357358293 -0300 -03 m=+0.001642948
GoVersion  go1.16.15
GitBranch
GitCommit
GoArch     amd64
GoOS       linux

$ rportcli init -s http://localhost:XXXX -l XXXXXXXXXXXX -p XXXXXXXXXXXXXX
WARN[2022-06-09T14:46:02.744334] unexpected end of JSON input
WARN[2022-06-09T14:46:02.746780] cannot unmarshal error response : unexpected end of JSON input
FATA[2022-06-09T14:46:02.746833] config verification failed against the rport: the specified item doesn't exist

@gbatistuzzo Can you try just rportcli init and go through the init process interactively? Does this work for you?


fix: unused to json call

Summary

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

!bench


Problems

Ive fixed many issues with IO kit (i think it was on my end) but anyways when i try to click the go button it crashes my device and idk if its supposed to install cydia or what (i assume it was considering it had cydia.app and all of the essintaial cydia components in the bootstrap.tar file so)

Problem #5 already states this same issue.


Install iTorch on Mac crashes

I'm installing iTorch on mac and when I run itorch I get a crash.  I'm sure I'm doing something pretty dumb but I can't figure out what.
Thanks in advance!
Here's my log:
pmerrill-osx:extra pmerrill$ brew install zeromq
Warning: zeromq-4.1.3 already installed
pmerrill-osx:extra pmerrill$ brew install openssl
Warning: openssl-1.0.2d_1 already installed
pmerrill-osx:extra pmerrill$ luarocks install luacrypto OPENSSL_DIR=/usr/local/opt/openssl/
Installing https://raw.githubusercontent.com/rocks-moonscript-org/moonrocks-mirror/master/luacrypto-0.3.2-1.src.rock...
Using https://raw.githubusercontent.com/rocks-moonscript-org/moonrocks-mirror/master/luacrypto-0.3.2-1.src.rock... switching to 'build' mode
Archive:  0.3.2.zip
0639ea2f65dbe5f11c2d874527f3019ce5293ac5
creating: luacrypto-0.3.2/
inflating: luacrypto-0.3.2/.gitignore
extracting: luacrypto-0.3.2/AUTHORS
inflating: luacrypto-0.3.2/CMakeLists.txt
inflating: luacrypto-0.3.2/COPYING
inflating: luacrypto-0.3.2/ChangeLog
inflating: luacrypto-0.3.2/INSTALL
inflating: luacrypto-0.3.2/Makefile.am
inflating: luacrypto-0.3.2/Makefile.in
extracting: luacrypto-0.3.2/NEWS
inflating: luacrypto-0.3.2/README
inflating: luacrypto-0.3.2/aclocal.m4
inflating: luacrypto-0.3.2/config.guess
inflating: luacrypto-0.3.2/config.h.in
inflating: luacrypto-0.3.2/config.sub
inflating: luacrypto-0.3.2/configure
inflating: luacrypto-0.3.2/configure.ac
inflating: luacrypto-0.3.2/depcomp
inflating: luacrypto-0.3.2/dist.cmake
creating: luacrypto-0.3.2/doc/
inflating: luacrypto-0.3.2/doc/Makefile.am
inflating: luacrypto-0.3.2/doc/Makefile.in
creating: luacrypto-0.3.2/doc/us/
inflating: luacrypto-0.3.2/doc/us/examples.html
inflating: luacrypto-0.3.2/doc/us/index.html
inflating: luacrypto-0.3.2/doc/us/license.html
inflating: luacrypto-0.3.2/doc/us/luacrypto-128.png
inflating: luacrypto-0.3.2/doc/us/manual.html
inflating: luacrypto-0.3.2/install-sh
inflating: luacrypto-0.3.2/ltmain.sh
inflating: luacrypto-0.3.2/luacrypto.pc.in
inflating: luacrypto-0.3.2/luacrypto.sln
inflating: luacrypto-0.3.2/luacrypto.vcproj
creating: luacrypto-0.3.2/m4/
inflating: luacrypto-0.3.2/m4/libtool.m4
inflating: luacrypto-0.3.2/m4/ltoptions.m4
inflating: luacrypto-0.3.2/m4/ltsugar.m4
inflating: luacrypto-0.3.2/m4/ltversion.m4
inflating: luacrypto-0.3.2/m4/lt~obsolete.m4
inflating: luacrypto-0.3.2/missing
creating: luacrypto-0.3.2/rockspecs/
inflating: luacrypto-0.3.2/rockspecs/luacrypto-git-1.rockspec
creating: luacrypto-0.3.2/src/
inflating: luacrypto-0.3.2/src/Makefile.am
inflating: luacrypto-0.3.2/src/Makefile.in
inflating: luacrypto-0.3.2/src/lcrypto.c
inflating: luacrypto-0.3.2/src/lcrypto.h
creating: luacrypto-0.3.2/tests/
inflating: luacrypto-0.3.2/tests/Makefile.am
inflating: luacrypto-0.3.2/tests/Makefile.in
creating: luacrypto-0.3.2/tests/ca/
inflating: luacrypto-0.3.2/tests/ca/README
inflating: luacrypto-0.3.2/tests/ca/ca.crt
inflating: luacrypto-0.3.2/tests/ca/ca.key
inflating: luacrypto-0.3.2/tests/ca/server.crt
inflating: luacrypto-0.3.2/tests/ca/server.csr
inflating: luacrypto-0.3.2/tests/ca/server.key
inflating: luacrypto-0.3.2/tests/ca/server.key.insecure
inflating: luacrypto-0.3.2/tests/encrypt.lua
extracting: luacrypto-0.3.2/tests/message
inflating: luacrypto-0.3.2/tests/open_seal.lua
inflating: luacrypto-0.3.2/tests/pkeytest.lua
inflating: luacrypto-0.3.2/tests/rand.lua
inflating: luacrypto-0.3.2/tests/run-tests
inflating: luacrypto-0.3.2/tests/test.lua
extracting: luacrypto-0.3.2/tests/tmp.rnd
inflating: luacrypto-0.3.2/tests/x509_ca.lua
export MACOSX_DEPLOYMENT_TARGET=10.5; gcc -O2 -fPIC -I/Users/pmerrill/torch/install/include -c src/lcrypto.c -o src/lcrypto.o -I/usr/local/opt/openssl/include
export MACOSX_DEPLOYMENT_TARGET=10.5; gcc -bundle -undefined dynamic_lookup -all_load -o crypto.so -L/Users/pmerrill/torch/install/lib src/lcrypto.o -L/usr/local/opt/openssl/lib -Wl,-rpath,/usr/local/opt/openssl/lib: -lcrypto
Updating manifest for /Users/pmerrill/torch/install/lib/luarocks/rocks
luacrypto 0.3.2-1 is now built and installed in /Users/pmerrill/torch/install/ (license: MIT)
pmerrill-osx:extra pmerrill$ git clone https://github.com/facebook/iTorch.git
fatal: destination path 'iTorch' already exists and is not an empty directory.
pmerrill-osx:extra pmerrill$ cd iTorch
pmerrill-osx:iTorch pmerrill$ luarocks make
ipy=$(which ipython)
if [ -x "$ipy" ]
then
ipybase=$(ipython locate)
rm -rf $ipybase/profile_torch
ipython profile create torch
echo 'c.KernelManager.kernel_cmd = ["/Users/pmerrill/torch/install/bin/itorch_launcher","{connection_file}"]' >>$ipybase/profile_torch/ipython_config.py
echo "c.Session.key = b''" >>$ipybase/profile_torch/ipython_config.py
echo "c.Session.keyfile = b''" >>$ipybase/profile_torch/ipython_config.py
mkdir -p $ipybase/profile_torch/static/base/images
mkdir -p $ipybase/kernels/itorch
cat kernelspec/kernel.json | sed "s@LUA_BINDIR@/Users/pmerrill/torch/install/bin@" > $ipybase/kernels/itorch/kernel.json
cp kernelspec/*.png $ipybase/kernels/itorch/
cp ipynblogo.png $ipybase/profile_torch/static/base/images
mkdir -p $ipybase/profile_torch/static/custom/
cp custom.js $ipybase/profile_torch/static/custom/
cp custom.css $ipybase/profile_torch/static/custom/
cp itorch /Users/pmerrill/torch/install/bin/
cp itorch_launcher /Users/pmerrill/torch/install/bin/
cp -r ~/.ipython/profile_torch ~/.ipython/profile_itorch
cmake -E make_directory build && cd build && cmake .. -DCMAKE_BUILD_TYPE=Release -DCMAKE_PREFIX_PATH="/Users/pmerrill/torch/install/bin/.." -DCMAKE_INSTALL_PREFIX="/Users/pmerrill/torch/install/lib/luarocks/rocks/itorch/scm-1" && make
else
echo "Error: could not find ipython in PATH. Do you have it installed?"
fi
[ProfileCreate] Generating default config file: u'/Users/pmerrill/.ipython/profile_torch/ipython_config.py'
-- Found Torch7 in /Users/pmerrill/torch/install
-- Configuring done
-- Generating done
-- Build files have been written to: /Users/pmerrill/torch/extra/iTorch/build
cd build && make install
Install the project...
-- Install configuration: "Release"
-- Installing: /Users/pmerrill/torch/install/lib/luarocks/rocks/itorch/scm-1/lua/itorch/_env.lua
-- Installing: /Users/pmerrill/torch/install/lib/luarocks/rocks/itorch/scm-1/lua/itorch/bokeh.lua
-- Installing: /Users/pmerrill/torch/install/lib/luarocks/rocks/itorch/scm-1/lua/itorch/completer.lua
-- Installing: /Users/pmerrill/torch/install/lib/luarocks/rocks/itorch/scm-1/lua/itorch/gfx.lua
-- Installing: /Users/pmerrill/torch/install/lib/luarocks/rocks/itorch/scm-1/lua/itorch/init.lua
-- Installing: /Users/pmerrill/torch/install/lib/luarocks/rocks/itorch/scm-1/lua/itorch/IOHandler.lua
-- Installing: /Users/pmerrill/torch/install/lib/luarocks/rocks/itorch/scm-1/lua/itorch/main.lua
-- Installing: /Users/pmerrill/torch/install/lib/luarocks/rocks/itorch/scm-1/lua/itorch/Plot.lua
-- Installing: /Users/pmerrill/torch/install/lib/luarocks/rocks/itorch/scm-1/lua/itorch/test.lua
-- Installing: /Users/pmerrill/torch/install/lib/luarocks/rocks/itorch/scm-1/lua/itorch/util.lua
-- Installing: /Users/pmerrill/torch/install/lib/luarocks/rocks/itorch/scm-1/lua/itorch/bokeh-0.7.0.min.js
-- Installing: /Users/pmerrill/torch/install/lib/luarocks/rocks/itorch/scm-1/lua/itorch/bokeh-0.7.0.min.css
Updating manifest for /Users/pmerrill/torch/install/lib/luarocks/rocks
itorch scm-1 is now built and installed in /Users/pmerrill/torch/install/ (license: BSD)
pmerrill-osx:iTorch pmerrill$
pmerrill-osx:iTorch pmerrill$ itorch
/Users/pmerrill/torch/install/bin/itorch: line 19: jupyter: command not found
pmerrill-osx:iTorch pmerrill$ itorch notebook
Traceback (most recent call last):
File "/usr/local/bin/ipython", line 11, in
sys.exit(start_ipython())
File "/Library/Python/2.7/site-packages/IPython/init.py", line 118, in start_ipython
return launch_new_instance(argv=argv, **kwargs)
File "/Library/Python/2.7/site-packages/traitlets/config/application.py", line 591, in launch_instance
app.initialize(argv)
File "", line 2, in initialize
File "/Library/Python/2.7/site-packages/traitlets/config/application.py", line 75, in catch_config_error
return method(app, *args, **kwargs)
File "/Library/Python/2.7/site-packages/IPython/terminal/ipapp.py", line 305, in initialize
super(TerminalIPythonApp, self).initialize(argv)
File "", line 2, in initialize
File "/Library/Python/2.7/site-packages/traitlets/config/application.py", line 75, in catch_config_error
return method(app, *args, **kwargs)
File "/Library/Python/2.7/site-packages/IPython/core/application.py", line 386, in initialize
self.parse_command_line(argv)
File "/Library/Python/2.7/site-packages/IPython/terminal/ipapp.py", line 300, in parse_command_line
return super(TerminalIPythonApp, self).parse_command_line(argv)
File "", line 2, in parse_command_line
File "/Library/Python/2.7/site-packages/traitlets/config/application.py", line 75, in catch_config_error
return method(app, *args, **kwargs)
File "/Library/Python/2.7/site-packages/traitlets/config/application.py", line 487, in parse_command_line
return self.initialize_subcommand(subc, subargv)
File "", line 2, in initialize_subcommand
File "/Library/Python/2.7/site-packages/traitlets/config/application.py", line 75, in catch_config_error
return method(app, *args, **kwargs)
File "/Library/Python/2.7/site-packages/traitlets/config/application.py", line 418, in initialize_subcommand
subapp = import_item(subapp)
File "/Library/Python/2.7/site-packages/ipython_genutils/importstring.py", line 31, in import_item
module = import(package, fromlist=[obj])
ImportError: No module named notebook.notebookapp
pmerrill-osx:iTorch pmerrill$

have you installed ipython / jupyter? it seems to say:  jupyter: command not found

Cool thanks that did the trick!!!  I needed a sudo infront of the pip to get jupyter to install.
I'm testing and finding that the image/audio/video are not working.  Here's an example of the output console log where the image is not drawing like in the example.  Any ideas what might be going on?
Thanks again in advance!
"In [2]: itorch.image({image.lena(), image.lena(), image.lena()})
Out[2]:
In [3]: "

They don't work in itorch console, you'll have to open an itorch notebook
(that opens in the browser)
On Tuesday, October 6, 2015, watts4speed<EMAIL_ADDRESS>wrote:

Cool thanks that did the trick!!! I needed a sudo infront of the pip to
get jupyter to install.
I'm testing and finding that the image/audio/video are not working. Here's
an example of the output console log where the image is not drawing like in
the example. Any ideas what might be going on?
Thanks again in advance!
"In [2]: itorch.image({image.lena(), image.lena(), image.lena()})
Out[2]:
In [3]: "
—
Reply to this email directly or view it on GitHub
https://github.com/facebook/iTorch/issues/71#issuecomment-145739268.


fix the requirements.txt file

Codecov Report

Merging #13 into master will decrease coverage by 5.49%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master      #13     +/-   ##
=========================================
- Coverage   98.51%   93.02%   -5.5%
=========================================
  Files          12       13      +1
  Lines         202      215     +13
  Branches       14       20      +6
=========================================
+ Hits          199      200      +1
- Misses          2        8      +6
- Partials        1        7      +6

Impacted Files
Coverage Δ

color_gamma_analyzer/tests/test_xml_util.py
86.66% <0%> (-13.34%)
:arrow_down:

color_gamma_analyzer/tests/test_diapason.py
87.5% <0%> (-12.5%)
:arrow_down:

color_gamma_analyzer/tests/test_converter.py
87.5% <0%> (-12.5%)
:arrow_down:

color_gamma_analyzer/tests/test_brightness.py
90.9% <0%> (-9.1%)
:arrow_down:

color_gamma_analyzer/tests/test_img_loader.py
91.3% <0%> (-8.7%)
:arrow_down:

...gamma_analyzer/tests/test_brightness_processing.py
91.3% <0%> (-8.7%)
:arrow_down:

color_gamma_analyzer/__init__.py
100% <0%> (ø)

Continue to review full report at Codecov.

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


On Dr search, when user hits ENTER key, the modal flashes

When a user searches for a doctor, nurse, or CO, the modal that displays the result flashes for a moment and then the search field is cleared.

This was being caused by loading the custom.js file twice.


Cannot decrease the reference clock frequency for u200

Dear Sir/Madam,
I am using the Vitis 2019.2 tool to implement some code on the U200 board. I found the timing requirement cannot be satisfied using the default 300MHZ frequency so I tried to decrease it but failed. I have attached the .txt version of my Makefile. I have made the following changes:
TARGET := hw
HOST_ARCH := x86
SYSROOT :=
+FREQ := 200000000
.PHONY: build
build: $(BINARY_CONTAINERS)
Building kernel
$(TEMP_DIR)/$(KERNEL).xo: src/$(KERNEL).cpp
mkdir -p $(TEMP_DIR)
$(VPP) $(CLFLAGS) --temp_dir $(TEMP_DIR) -c -k $(KERNEL) --kernel_frequency 200 -o'$@' '$<'
$(BUILD_DIR)/$(KERNEL).xclbin: $(BINARY_CONTAINER_kernel_OBJS)
mkdir -p $(BUILD_DIR)
$(VPP) $(CLFLAGS) --temp_dir $(BUILD_DIR) -l --kernel_frequency 200 -o'$@' $(+)
Building Host
$(EXECUTABLE): check-xrt $(HOST_SRCS) $(HOST_HDRS)
$(CXX) $(CXXFLAGS) $(HOST_SRCS) $(HOST_HDRS) -o '$@' $(LDFLAGS)
I also tried to the following command to generate the botstream
faketime 'last year' make all TARGET=hw DEVICE=xilinx_u200_xdma_201830_2 FREQ=200000000 -B
Makefile.txt
The above attempts can only change the HLS synthesis results, but the timing results in the xilinx_u200_xdma_201830_2_bb_locked_timing_summary_routed.rpt file still remains unchanged.
Can anyone help me to reduce the PL frequency?

Can you please post your query on the Xilinx Forum?
https://support.xilinx.com/s/topic/0TO2E000000YKYAWA4/vitis-acceleration-acceleration?language=en_US


Make moving topics between streams faster in large organizations

In large organizations (like chat.zulip.org), moving a topic to a different stream can be very slow (seconds to tens of seconds). We should fix this.

This might not be the solution we choose, but using topic IDs would make this much faster.

Topic IDs would not make this faster, at least in isolation, because the bulk of the cost/complexity is the work around deleting UserMessage rows where required for security. In any case, #26899 gets about a 2x speedup on top of the work for this completed in 8.0. It's probably still too slow, but to some extent that's because we've signed up to do quadratic work in such cases with the product design.


Write "barrier factor" instead of "form factor" in TR-029

Form factor has multiple meanings, barrier factor is less ambiguous.

Basically:

Internal Structure Form Factors
These form factors describe the distribution of charge and current within hadrons. They provide information about the internal structure and are essential for understanding elastic and inelastic scattering processes.
They depend on the momentum transfer $Q^2$ (space-like) or s (time-like).

Blatt–Weisskopf Form Factors (Barrier Factors)
also known as barrier factors, are used in partial wave analyses of resonance decays. They account for the angular momentum barrier effects in the decay of resonances and the production of particles.
These form factors depend on the orbital angular momentum 𝐿 of the interacting particles and the radial distance (or momentum) involved in the interaction.
They are designed to regularize the centrifugal barrier effects and are typically expressed using spherical Hankel functions or other related functions.


"Stopping" group simulation

When a group is no longer executing (i.e. when its go signal is low) we want to ensure that nothing is happening. Currently this is accomplished by the fact that the group assignments won't be evaluated when we're not simulating the group. However, this means that the values mapped to each of the ports are stale/incorrect as they derive from assignments that are no longer running. The observed behavior in verilog is that all these assignments take on the value of zero, but it may be advantageous later to specifically model unavailable/invalid values (see #542). Regardless, we need a step/method to run after simulating a group to reflect the impact of the go signal going low. The reason to make this a separate step, rather than rolling it in with group simulation is that if and while statements can read values for branching which should be valid.

A version of this exists now with the finish_group_interpretation. Issue can be resolved pending later PR and additional verification.


Travis: Also build with openjdk-ea

Coverage decreased (-0.007%) to 95.246% when pulling a3a1904e5964a97a20d5132569beb2901d2cd919 on PascalSchumacher:travis_openjdk_ea into 8e507bd0b188b497ed371b8a6772b0e6ad9bd1cd on apache:master.


Registered asset amounts are not being formatted with precision in spend.tsx

When the marina.sendTransaction is called from the api, the popup from sign.tsx doesn't format the amount with the precision found within the registry
The asset in the picture below has a precision of 0 but is shown with a precision of 8:

@tiero still present in master, should be fixed by #450


generate in-page toc

this is a feature request
Please add the option to generate TOC (using confluence's toc macro) within a page given a heading structure. It also helps to create deeplinks. If it is available. please improve the documentation so that this info can be found easier.

confluence_metadata

The Table of Contents Macro macro does not have a directive support yet, but we can plan to add a confluence_toc directive type in attempt to support this.
At this time, a workaround would be to add something like this to use Confluence's TOC macro:
(default)
.. raw:: confluence_storage

    <ac:structured-macro ac:name="toc" ac:schema-version="1" />

(parameter examples; see Confluence's TOC tree macro for more macro options)
.. raw:: confluence_storage

    <ac:structured-macro ac:name="toc" ac:schema-version="1">
        <ac:parameter ac:name="maxLevel">6</ac:parameter>
        <ac:parameter ac:name="minLevel">2</ac:parameter>
        <ac:parameter ac:name="outline">true</ac:parameter>
        <ac:parameter ac:name="type">flat</ac:parameter>
    </ac:structured-macro>

To my own understanding, Confluence's TOC tree macro is for local page header/TOC only. I believe a matching reStructuredText equivalent would be the use of the contents directive, such as follows:
.. contents::
    :local:

This will provide support for things like backlinks; but will not have capabilities such as Confluence flat-type TOCs.

Initial support has been introduced (#660) to support Confluence's Table of Contents macros through Sphinx-supported directives. This change has been added into the main branch and should be made available next stable release. If users wish to use this capability before a release is made, the development version of this extension can be installed.

v1.9 is now available on PyPI -- marking as closed.


Documentation points to nightly

In e.g. https://docs.rs/depot/0.2.0/depot/queue/struct.Queue.html the Iterator link points to https://doc.rust-lang.org/nightly/core/iter/iterator/trait.Iterator.html (which is broken).
It should probably point to the stable documentation.

Thanks for catching this @bestouff, as well as for filing the other issues. One thing I want to point out is this excerpt in the README:

Development is currently stalled as other projects are currently higher priority.
Consider this project a preview of the API -- the on-disk format will change to accomodate better performance and efficient item removal. The intention is for there to be first-class implementations for Java and Rust.

Just mentioning it in case you didn't see that - i.e. the quality isn't where it needs to be yet.
I do intend to get back to it and finish both Rust/Java impls, hopefully over this summer :)

I saw it yes. I'm just opening issues to track progress :)


TEAM.md updated - Rajesh

added github link and sort bio.

Thanks for this PR.
U need 2 code review approval.
Approach others in twitter group or discord channel (https://discord.com/invite/jBj2zMR)
Post ur pr link in those grp.


'undefined is not a function' when promise function not defined

Using 1.4.0-beta.5, the following stack trace seems to be the result of an error() function not being defined during promise rejection. The rejection still bubbles up, however the "undefined is not a function" is logged in the console.
http://plnkr.co/edit/JtaNeNEcxFLPH5ByuHI8
TypeError: undefined is not a function
at angular.js:9564
at processQueue (angular.js:14018)
at angular.js:14034
at Scope.$get.Scope.$eval (angular.js:15236)
at Scope.$get.Scope.$digest (angular.js:15051)
at Scope.$get.Scope.$apply (angular.js:15341)
at done (angular.js:9847)
at completeRequest (angular.js:10037)
at XMLHttpRequest.requestLoaded (angular.js:9978)

That is because the httpError variable is undefined until the assignment takes place (which is after calling .error.  That is the same as doing $http(...).error(undefined).
See function hoisting vs var hoisting.

@jbedard is right. Closing as not a valid issue

The problem is that the error is not nice at all and since it's thrown usually asynchronously with respect to where the callback was registered, it's really hard to debug.
Could we just silently ignore, if the callback is not a function? Would this break Promises A+ spec?

OK, so this is hard to debug but it is a JS development error not a failure in Angular. Wherever there is code that takes a callback, you have to have defined the callback before you pass it to the code.
Silently ignoring if the callback is not a function will make this even more difficult to debug! There won't even be an error but the error callback will silently not be called if the HTTP request errors.
I think we could throw an error if you call .error() with a thing that is not a function. For example in this case the code is effectively calling .error(undefined), which is not what the error() method expects.

I created a PR with this in mind. @olore and @IgorMinar is this an acceptable fix?
Now if you fail to pass in a valid function to either error or success you get an instant error.

I think this will give more clarity to the situation.

So, if we want to keep those callbacks close to promises callbacks we should ignore non-function arguments: https://promisesaplus.com/#point-23

The only reason the promise spec allows us to ignore params is because they are all optional but I am pretty sure that passing no parameters is always an error. In the case of error and success there is only one parameter, so it never makes sense to fail to pass the parameter.

Well, the spec says "if it is not a function", so I'm not sure it is only about optional params

Guys thanks for seeing this one through.
@IgorMinar appreciate you opening this one back up and describing our twitter convo more clearly


Don't send notifications when the user comments on their own post

Obviously the user knows they're commenting on their own post, so we shouldn't notify them.

This is duplicated of #738 , do you mind to follow that issue and close this one?
Thanks!


review + comment routes

Updated Task

[ ] Display all transactions in orders page and display review/comments component for anonymous rating feature

some are done


Will there be performance issues running WSL2 on an Intel big.LITTLE CPU ?

The current kernel version of WSL2 is: Linux <IP_ADDRESS>-microsoft-standard-WSL2, but Intel's Thread Director Coming to Linux 5.18 to Fix Alder Lake Performance Issues. This means that WSL2 may have thread scheduling issues when running on Intel big.LITTLE CPUs.
Should/can I upgrade the kernel version? Or is there a way to run WSL2 only on P cores or E cores?
Thanks.

It doesn't matter which virtual CPU the Linux kernel inside WSL schedules on.
It's the hypervisor's scheduler that actually decides which physical CPU to run on.
(You can see this by running taskset 1 yes > /dev/null and looking at top 1 and taskmgr.exe.)
And those features are hidden by virtualization.

@0xbadfca11
But I'm a bit concerned about the hypervisor's scheduler scheduling my CPU-bound process in WSL2 onto the E core.

similar issue is being tracked here #9121


N-Hop Lifting (Graph to Combinatorial)

Add N-Hop lifting to combinatorial complex where the n-hop neighbourhood is a 2+k rank hyper edge. This lifting technique is mentioned in  Hajij et al, 2023 where the proof that this lift is a Combinatorial Complex is also described
Tags:
Existing lift from literature | connectivity-based | deterministic

Hello @martin-carrasco! Thank you for your submission. As we near the end of the challenge, I am collecting participant info for the purpose of selecting and announcing winners. Please email me (or have one member of your team email me) at<EMAIL_ADDRESS>so I can share access to the voting form. In your email, please include:

your first and last name (as well as any other team members)
the title of the method you implemented
the input domain of the method you implemented
the output domain of the method you implemented
your pull request number (#28)

Before July 12, make sure that your submission respects all Submission Requirements laid out on the challenge page. Any submission that fails to meet this criteria will be automatically disqualified.


Empty lines in the log file between json lines

Filebeat will get the error
{"log.level":"error","@timestamp":"2022-12-02T16:54:15.327Z","log.logger":"reader_json","log.origin":{"file.name":"readjson/json.go","file.line":74},"message":"Error decoding JSON: EOF","service.name":"filebeat","ecs.version":"1.6.0"}
This is caused I suspect because of the blank spaces in the log files

Bug fixed!

This was probably fixed by https://github.com/RichardoC/kube-audit-rest/commit/c54beebee13d726c52a19d33316a04531eacc19b


BluetoothDevice name is not what I expect on iOS devices

Often times the scan results yield names that I do not expect, and cannot change when I'm using an iOS device.

From iOS CBPeripheral docs:

A peripheral may have two different name types: one that the device advertises and another that the device publishes in its database as its Bluetooth low energy Generic Access Profile (GAP) device name. If a peripheral has both types of names, this property returns its GAP device name.

The GAP device name is prioritized over anything else.

If anyone wants a helpful tip, I assume the Advertising name isn't going to change, and I create a map that lasts all the runtime.
Then, at the same time I scan results, I annotate this name in a map:
something like:
extension BluetoothDeviceNameUtils on List {
void saveLocalNamesFromScanResults(List scanResults) {
scanResults.forEach((it) {
results[it.device.id.id] = it.advertisementData.localName;
});
}
}
now, what I do, if I'm interested in this name, I create an extension on the device that will tell me that name or the default if there's any error grabbing it.
String get advertisingName => results[this.id.id] ?? this.name;
(Keep your map at runtime!)

This works for me. I have done changes in flutter_blue (src/bluetooth_device.dart) and add new method BluetoothDevice.fromProtos(protos.ScanResult p)

then one more change in (src/flutter_blue.dart) see above screenshot.

If anyone wants a helpful tip, I assume the Advertising name isn't going to change, and I create a map that lasts all the runtime. Then, at the same time I scan results, I annotate this name in a map:
something like:
  extension BluetoothDeviceNameUtils on List<ScanResult> {
    void saveLocalNamesFromScanResults(List<ScanResult> scanResults) {
      scanResults.forEach((it) {
        results[it.device.id.id] = it.advertisementData.localName;
      });
    }
  }

now, what I do, if I'm interested in this name, I create an extension on the device that will tell me that name or the default if there's any error grabbing it.
  extension BluetoothDeviceExtensions on BluetoothDevice {
    String get advertisingName => results[this.id.id] ?? this.name;
  }

(Keep your map at runtime!)

Thanks, for giving me the idea that's why I achieved it.


Adding an info msg for docker pull

Hey,
as the docker hub is very slow for pulling images sometimes. I would suggest adding a message to "docker pull " indicating that the first download was started. It costs me about 2 hours wondering why there is nothing happening, until i wait long enough to see something. I only waited so long, because i was googleing what i'm doing wrong. The first message printed is something like "download completed". What do you guys think about it?

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

Not sure I understand your request, docker pull shows a progress bar (e.g. https://m.youtube.com/watch?v=ED6lN8OqXoI) what change are you requesting to be made?

If it would look like in the video it would be awesome! That is more, i expected. Sadly, for me it looks different.
core@core01 ~ $ docker info
Containers: 9
Images: 47
Server Version: 1.9.1
Storage Driver: overlay
 Backing Filesystem: extfs
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 4.3.0-coreos
Operating System: CoreOS 877.1.0
CPUs: 4
Total Memory: 3.808 GiB
Name: core01
ID: KL6V:NIKH:77LK:OSTO:M6NG:TZWE:3MDZ:YGSY:SB3N:4BFV:GHWF:ZSWS
core@core01 ~ $ docker images
REPOSITORY          TAG                 IMAGE ID            CREATED             VIRTUAL SIZE
gitlab/gitlab-ce    latest              43aef83b9f72        4 days ago          1.33 GB
jenkins             latest              988e2e1b7418        3 weeks ago         707.8 MB
core@core01 ~ $ docker version
Client:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.3
 Git commit:   4419fdb-dirty
 Built:        Thu Nov 26 02:50:10 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.3
 Git commit:   4419fdb-dirty
 Built:        Thu Nov 26 02:50:10 UTC 2015
 OS/Arch:      linux/amd64
core@core01 ~ $ uname -a
Linux core01 4.3.0-coreos #2 SMP Thu Nov 26 02:49:19 UTC 2015 x86_64 Intel(R) Core(TM) i5-2430M CPU @ 2.40GHz GenuineIntel GNU/Linux
core@core01 ~ $ docker pull ubuntu
Using default tag: latest

This last line lasts for about 10 minutes.   Important: This registry version will not be supported in future versions of docker.

I switched to CoreOS Beta because i through it would solve it. Maybe it's somek kind of bug? Hmm...

I access the server via ssh, but that is not a problem, isn't it?

CoreOS distributes their own builds of docker, and apply some modifications, so I think you should try contacting them first; based on the git commit version 4419fdb-dirty, this is the commit they built from (and applied their patches to it); https://github.com/coreos/docker/commit/4419fdb, perhaps that's what makes a difference here? Can you open this issue in the CoreOS issue tracker?

Okay, thanks. Maybe thats it

We ship a minimal set of patches to the Docker we ship in CoreOS. https://github.com/coreos/docker/commit/4419fdb was reverted because it broke socket activation and our fix was accepted upstream. FWIW, Docker 1.9.1 on CoreOS is behaving the same way as Docker 1.7.1 on my Gentoo system. I don't see the animated progress bar, but I do see the percentages.

@crawford is that on a "fresh" pull (i.e., an image that doesn't exist yet locally)?

@thaJeztah yes. I cleaned up before I ran the test. docker rmi $(docker images -aq)

Strange.
@aaronlehmann any idea here why progress bars are not visible?

It looks like my terminal was too thin.
$ stty -a
speed 38400 baud; rows 58; columns 95; line = 0;
...

Apparently, 95 columns is not enough... After widening it, I see the progress bars on both my machine and in CoreOS.

ah, interesting :)

Ah, yes. I've been confused by this too in the past. Ideally the progress bar would be formatted in a way that works with 80 column terminals. Not showing it unless the terminal is significantly larger is weird, to say the least.


Update oryx build base image to buster

Update the base image in KuduLite:
mcr.microsoft.com/oryx/build:20210708.1
Tested the build with github.com/Hazhzeng/KuduLite, branch: user/hazeng/funcdeplog
This branch contains the latest changes in AAPT-Antares-kudulite repository
https://github.com/Hazhzeng/kudulite/tree/user/hazeng%2Ffuncdeplog
This should fix the issue of GLIB_C building errors due to inconsistency between current remote build environment (debian 9.12) and runtime environment (debian 10.8)

PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which has an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] If applicable, the PR references the bug/issue that it fixes in the description.
[x] New Unit tests were added for the changes made and CI is passing.

KuduLite moved to internal repo


services.AddMvc (Core 1.1) throwing exception after VS 15.8.0 update

I have a project that is dotnet core 1.1 (and needs to stay there for the time being).  In my startup code I have the following:
        public void ConfigureServices(IServiceCollection services)
...
                services.AddMvc(config =>
                {
                    var policy = new AuthorizationPolicyBuilder().RequireAuthenticatedUser().Build();
                    config.Filters.Add(new AuthorizeFilter(policy));
                    config.Filters.Add(new CorsAuthorizationFilterFactory(Configuration["CorsPolicy"]));
                    config.Filters.Add(new RefreshJWTResultFilter(Options.Create(GetTokenOptions())));
                    config.Filters.Add(typeof(FaultTestingFilter));

                    config.Filters.Add(typeof(IncidentExceptionFilter));
                });

After updating Visual Studio to 15.8.0, this code throws the following exception:
"System.Collections.Generic.KeyNotFoundException"
The given key was not present in the dictionary.

With stacktrace:
   at System.Collections.Generic.Dictionary`2.get_Item(TKey key)
   at Microsoft.AspNetCore.Mvc.Internal.DefaultAssemblyPartDiscoveryProvider.CandidateResolver.ComputeClassification(String dependency)
   at Microsoft.AspNetCore.Mvc.Internal.DefaultAssemblyPartDiscoveryProvider.CandidateResolver.ComputeClassification(String dependency)
   at Microsoft.AspNetCore.Mvc.Internal.DefaultAssemblyPartDiscoveryProvider.CandidateResolver.ComputeClassification(String dependency)
   at Microsoft.AspNetCore.Mvc.Internal.DefaultAssemblyPartDiscoveryProvider.CandidateResolver.<GetCandidates>d__4.MoveNext()
   at System.Linq.Enumerable.<SelectManyIterator>d__17`2.MoveNext()
   at System.Linq.Enumerable.WhereSelectEnumerableIterator`2.MoveNext()
   at Microsoft.Extensions.DependencyInjection.MvcCoreServiceCollectionExtensions.GetApplicationPartManager(IServiceCollection services)
   at Microsoft.Extensions.DependencyInjection.MvcCoreServiceCollectionExtensions.AddMvcCore(IServiceCollection services)
   at Microsoft.Extensions.DependencyInjection.MvcServiceCollectionExtensions.AddMvc(IServiceCollection services)
   at Profile.Startup.ConfigureServices(IServiceCollection services)

I've simplified this to simply:
services.AddMvc();
and get the same error.
Source control indicates nothing in the startup file or package/reference list for this program has changed.
The project targets 4.6.1
   <TargetFramework>net461</TargetFramework>
Here are the relevant packages:
    <PackageReference Include="Microsoft.AspNetCore.Authentication.JwtBearer" Version="1.1.2" />
    <PackageReference Include="Microsoft.AspNetCore.Authorization" Version="1.1.2" />
    <PackageReference Include="Microsoft.AspNetCore.Diagnostics" Version="1.1.2" />
    <PackageReference Include="Microsoft.AspNetCore.Identity" Version="1.1.2" />
    <PackageReference Include="Microsoft.AspNetCore.Mvc" Version="1.1.3" />
    <PackageReference Include="Microsoft.AspNetCore.Routing" Version="1.1.2" />
    <PackageReference Include="Microsoft.AspNetCore.Server.IISIntegration" Version="1.1.2" />
    <PackageReference Include="Microsoft.AspNetCore.Server.Kestrel" Version="1.1.2" />
    <PackageReference Include="Microsoft.DotNet.ProjectModel" Version="1.0.0-rc3-003121" />
    <PackageReference Include="Microsoft.Extensions.Configuration.EnvironmentVariables" Version="1.1.2" />
    <PackageReference Include="Microsoft.Extensions.Configuration.FileExtensions" Version="1.1.2" />
    <PackageReference Include="Microsoft.Extensions.Configuration.Json" Version="1.1.2" />
    <PackageReference Include="Microsoft.Extensions.Logging" Version="1.1.2" />
    <PackageReference Include="Microsoft.Extensions.Logging.Console" Version="1.1.2" />
    <PackageReference Include="Microsoft.Extensions.Logging.Debug" Version="1.1.2" />
    <PackageReference Include="Microsoft.Extensions.Logging.Filter" Version="1.1.2" />
    <PackageReference Include="Microsoft.Extensions.Options.ConfigurationExtensions" Version="1.1.2" />
    <PackageReference Include="Microsoft.IdentityModel.Tokens" Version="5.1.4" />

I'm very broken now, and really don't know how to proceed.  Any suggestions are very much appreciated.

cc @dsplaisted
Seems like a dup of https://github.com/aspnet/Home/issues/3132, although that issue should have been fixed with the 2.1.400 SDK. Could you
a) Run dotnet --info from the command line and paste the contents
b) Build the app and paste the contents of bin\Debug\net461\${YourAppName}.deps.json  (a gist works great for this)
In the meanwhile, https://github.com/aspnet/Home/issues/3132#issuecomment-388082725 has some possible workarounds to unblocks yourself.

I've attached the requested information.  I'm attempting to absorb the workarounds.  Thank you so much for responding.
dotnet.info.txt
Sale.deps.json.txt

@EltonInAtlanta I've reopened - https://github.com/aspnet/Mvc/issues/7983 - to see if we could get this patched. You could also consider short-circuiting MVC's application part discovery code that produces this error. Here's the code you would have to add to your Startup:
var manager = new ApplicationPartManager();
var assemblies = new[] { "Sale", "Horizon.Common.Tokens", "Mpp.Core.Infrastructure", "Mpp.Core.Sale" };

foreach (var name in assemblies)
{
    var assembly = Assembly.Load(new AssemblyName(name));
    manager.ApplicationParts.Add(new AssemblyPart(assembly));
}

services.AddSingleton(manager);
services.AddMvc();

This should avoid being pinned to an earlier version of the SDK.

@pranavkm I changed the file name on my global.json and dropped this code in to my startup.cs.  It works great.

Using https://github.com/aspnet/Mvc/issues/7983 to track further work.

Hi,
I'm new on github so sorry if it's the wrong way to do it.
I have exactly the same issue since .net Core SDK 2.1.400 and I was wondering how you define which assembly were causing it?
Thanks

@Gimgur, you'd want to add any assemblies that contain MVC controllers. If you need help with this, feel free to open a new issue in https://github.com/aspnet/Mvc.


Merged master into proxy-auth and moved enum into SDK...

...in order to get a build of this branch

See https://github.com/xapi-project/xen-api-sdk/pull/85/commits/2c9c5d937facd738905b92f0910c630078bb24e3 for changes regarding enum. The rest are merges from master branch.

Looks ok. Merging.


cmake parsing KERNEL* files incorrectly

Currently cmake function ParseMakefileVars (which is used for parsing KERNEL* files) skips lines with if, ifeq, ifneq and just converts all assignments in config file to cmake variable assignments. It leads to execution of both if and else branches. For example in
ifeq($(DGEMM_UNROLL_M), 8)
DGEMMINCOPY = a
else
DGEMMINCOPY = b
endif

cmake just sets DGEMMINCOPY to a and then immediately resets in to b. This means wrong kernel will be called. And this is common pattern in KERNEL* files to use such if-else. So I have a lot of examples, when variable is reset to wrong value and it brokes performance or correctness. Often it is reference version instead of optimized. But sometimes it leads to segfaults.

Known defect unfortunately. The KERNEL files are basically Makefiles in disguise, so the gmake handling of conditionals would need to be reimplemented in ParseMakefileVars (which currently only knows to extract key-value pairs).

First attempt at a fix in #2584 - but requires an else branch to be present and does not yet support nested conditionals

Actually this appears to work quite well even without explicit support for nested conditionals.


Add a DatabricksRunner

There have been some discussions lately like https://dask.discourse.group/t/dask-on-databricks-clusters/2086/8 about being able to side load a Dask cluster onto a Databricks multi-node cluster. A Runner could be a good model for this.
A Databricks cluster can be given an init script that gets executed on every node of the cluster. We could create a runner that can be launched as an init script as a background process.
It looks like Databricks sets environment variables like DB_IS_DRIVER and DB_DRIVER_IP so the scheduler could start on the driver node and all of the workers can find it because they know the driver IP address.
It probably doesn't make sense to support running client code as that would likely be driven from a notebook.

Thanks for that @benrutter. I just tried your example and it worked for me!

Yeah you can see the result of 11 printed in my screenshot.
Here's the configuration I used.

Not sure how I missed in the screenshot! Could be the 40 seconds, I'll see if I can get it to work by extending it.

Yup, I think it must have either:

The 40 seconds not being enough
The driver node not having a wait time, so being callable ahead of workers

I don't really know enough about when databricks considers a cluster "ready", but given I was starting up a cluster to immediately run a notebook, its possible I just needed to wait an extra 40 seconds being scheduling the job.
Either way, tried again with a 100 second wait both before the worker nodes, and after the dask set up on the driver node, and all worked great.
Nice! What would be involved in setting up a runner to use this method?


Initial wrapping of the text-field

Fixes #3

This change is

Review status: 0 of 8 files reviewed at latest revision, all discussions resolved.

demo/time-picker-basic-demos.html, line 10 at r1 (raw file):

Plain time picker

Demo titles should use Title Case

src/vaadin-time-picker.html, line 24 at r1 (raw file):

    </style>
    <vaadin-time-picker-text-field
      value="[[value]]"

Because of this binding, I can programmatically set timePicker.value = "foo" and it gets reflected in the text field. Also changing the text to "bar" will revert the value back to (invalid) "foo". Add tests for this (should not allow setting invalid value programmatically).

src/vaadin-time-picker.html, line 26 at r1 (raw file):

      value="[[value]]"
    >
      <slot name="prefix" slot="prefix"></slot>

Why is this text-field feature specifically exposed?

src/vaadin-time-picker.html, line 79 at r1 (raw file):

        }

        _setValue(e) {

If this is supposed to be an event listener function for "change" event, rather name it to "_onChange". It would be hard to guess what _setValue(event) api means.

src/vaadin-time-picker.html, line 87 at r1 (raw file):

          const parts = /^(\d{2}):(\d{2})$/.exec(str);
          if (!parts) {
            return this.value;

Doesn't make sense that a function called parseTime(str) returns this.value

src/vaadin-time-picker-text-field.html, line 3 at r1 (raw file):


Only set $path if not already set

Some users of the GOV UK Elements sass have their own existing file structure for image assets. When the $path was added it created urls starting with /public/images but it would be more flexible to allow other users to set their own value. By making the assignment optional, if a developer has defined $path in their top level file it will use that instead. Alternatively move the $path assignment out of the helper and up to the main.scss file

As _helpers.scss shows which partials are imported from the govuk front end toolkit, I'd like to keep the $path variable here, rather than moving it up into main.scss.
Ideally, I'd set a _vars.scss file above this and keep all Sass variables the same place, however, all other variables used are being set by the front end toolkit (for this project).
I've added the missing import to make it clear _url-helpers.scss is a dependency.
I'll merge this so it doesn't interfere if $path is already declared, apologies for the slow response on this one - I was away last week.


Github Actions module 'Ajv' not found

Current behavior
I am developing a TypeScript project on macos and it successfully compiles (tsc) locally but when the following github actions script is run:
build:
    needs: lint
    runs-on: ubuntu-latest
    steps:
    - uses: actions/checkout@v3
    - name: Use Node.js 20.x
      uses: actions/setup-node@v3
      with:
        node-version: 20.x
        cache: 'npm'
    - run: npm ci
    - run: npm run compile

it fails with the output:
Run npm run compile

><EMAIL_ADDRESS>compile
> tsc

Error: src/common/validation.ts(1,3[4](https://github.com/{redacted}/job/17324159764#step:6:5)): error TS230[7](https://github.com/{redacted}/job/17324159764#step:6:8): Cannot find module 'Ajv' or its corresponding type declarations.
Error: Process completed with exit code 2.

(* I 'redacted' my project info)
my package.json:
{
  "name": "hello_world",
  "version": "1.0.0",
  "description": "hello world sample for NodeJS",
  "main": "app.js",
  "repository": "https://github.com/awslabs/aws-sam-cli/tree/develop/samcli/local/init/templates/cookiecutter-aws-sam-hello-nodejs",
  "author": "SAM CLI",
  "license": "MIT",
  "scripts": {
    "unit": "jest",
    "lint": "eslint 'src/**/*.ts' --quiet",
    "lint-warn": "eslint 'src/**/*.ts'",
    "lint-fix": "eslint 'src/**/*.ts' --quiet --fix",
    "compile": "tsc",
    "test": "npm run compile && npm run unit"
  },
  "dependencies": {
    "@aws-sdk/client-secrets-manager": "^3.254.0",
    "@hapi/joi": "^17.1.1",
    "ajv": "^8.12.0",
    "esbuild": "^0.14.14",
    "jwks-rsa": "^3.0.1",
    "jwt-decode": "^3.1.2",
    "lodash": "^4.17.21",
    "mongoose": "^6.11.2"
  },
  "devDependencies": {
    "@types/aws-lambda": "^8.10.92",
    "@types/hapi__joi": "^17.1.9",
    "@types/jest": "^29.2.0",
    "@types/lodash": "^4.14.195",
    "@types/node": "^18.11.4",
    "@typescript-eslint/eslint-plugin": "^5.10.2",
    "@typescript-eslint/parser": "^5.10.2",
    "esbuild-jest": "^0.5.0",
    "eslint": "^8.8.0",
    "eslint-config-prettier": "^8.3.0",
    "eslint-plugin-prettier": "^4.0.0",
    "jest": "^29.2.1",
    "prettier": "^2.5.1",
    "ts-node": "^10.9.1",
    "typescript": "^4.8.4"
  }
}

I have tried switching the runner os to 'macos-latest' to match my local environment but it gives the same result. The only thing I can think of is something I have installed locally that the runner needs but 'ajv' is in the list of dependencies so it should be installed by the 'npm ci' step of the githb action script. Any help is greatly appreciated!
Expected behavior
github actions runner succeeds just like running the script locally.
Steps to reproduce the issue

run the above github actions script with a TypeScript project that has ajv as a dependency

Environment

os: macos

Wrong repo, my apologies


Structured format display attribute multiple props

Addresses issue #241

Could you also submit a CLA here
Please review our contribution information here
Thank you
Kevin

@KevinRansom I did submit a CLA yesterday, was there something I missed?

No .. it just takes a while to wind through system, thank you for submitting this.
Kevin

@latkin I am a bit unsure about the direction you want me to take regarding the tests. I'm working on adding tests (and addressing shortcomings found via those tests), and for now I have them in a separate test folder (sformat), was that what you were suggesting doing?

I am suggesting that you extend the set of test cases found here

Ok, I've checked in those additions.

I've synced this and played around a bit.  Works quite well for most scenarios.  Some edge cases might need attention, though
Depth limit With multiple properties, the cost of recursive format definitions explodes. I would look for ways to prevent code like below from freezing up fsi. At the very least, there should be test coverage for basic depth limit cases.
[<StructuredFormatDisplay("{X} {X}")>]
type Foo() =
   member __.X = Foo()

Foo() |> printfn "%A"

Compat Old behavior simply looked at string between first { and last }, then checked if a property with that name exists.  New behavior does not maintain compat with this when there is { or } in property names.
e.g. both of below used to print "abc" but now print something else.
[<StructuredFormatDisplay("{{}}")>]
type Foo() = member __.``{}`` = "abc"

[<StructuredFormatDisplay("{One} {Two}")>]
type Foo() =
    member __.``One} {Two`` = "abc"
    member __.One = 123
    member __.Two = 456

For maximum back-compat, the right approach might be to use the old algorithm to start with, and only go to the new algorithm if the former fails to find a matching property.

If that's the only compat problem I think we can live with that :)

Regarding the depth-limit, is there some sort of standard/guideline for establishing a max recursion depth? I could easily add a depth parameter to the recursive function and bail out if some arbitrary limit is reached, but want to ask if there is a standard for this already established.
I'll let you two hash out whether I should make any modifications regarding backwards compatibility or not :)

There is already a depth limit parameter passed to the function. See depthLim param here, checked/modified in various places. There is some effort in place to intelligently limit the depth if we suspect things will get expensive (see here, not sure how well this works) I suggest that we do something along the same lines here.  The default depth limit is 100.  Without throttling, we will naively attempt 2^100 property evaluations for my example above. :scream:
For the compat item -- for the purposes of compat I guess it's sufficiently corner case that it's ok.  However this also means we are removing support for displaying properties with { or } in general. Are we ok with that?

I added some throttling very similar to the example you pointed me to (thanks for that, and the reminder that there already was a depth parameter), and a simple test case using your exact example.

I've given the compat issue more thought, and I could rollback/simplify the logic I added to try to avoid exceptions being thrown by mis-matched/nested brackets. In other words, I could simplify things to just blindly grab the text between a { and } while accounting for nested brackets. So, if two { are encountered before a }, then consider the second } as the property name boundary. That would still leave unanswered how to handle single brackets in property names or "flipped" brackets (e.g. member __.``One} {Two`` = "abc"). My only thought there would be to brute-force the different possible property names until a valid one is discovered (start with the smallest possible parsed property name, and keep grabbing more text before the next bracket).
My original thinking was that, in "normal" circumstances, these brackets would not be used for property names and I should consider any nested or mis-matched brackets as a user error. If that isn't a fair assumption, I am fine updating my thinking.
With an eye towards backwards-compatibility, perhaps the hybrid approach proposed by @latkin would be best. Try the original logic and just grab everything between the first { and the last } and try to find the corresponding property. If that throws an exception, then try this new recursive logic. Then the only limitation is that users couldn't use properties with brackets in their names going forward unless they were the only property used.

The old behavior seems wrong to me. Seems there should be an escape char for embedded brackets.
[<StructuredFormatDisplay("{\{\}}")>]
type Foo() = member __.``{}`` = "abc"

[<StructuredFormatDisplay("{One\} \{Two}")>]
type Foo() =
    member __.``One} {Two`` = "abc"
    member __.One = 123
    member __.Two = 456

Have any decisions been reached on this? Is this good as is? Should I make the change to first try the old method for backwards compatibility?

No, it's fine, you don't need to try the old method

There should be a way to escape brackets, both to support properties with brackets in their names and to allow for bracket literals in the display string.

Should that bracket-escaping behavior be part of this PR or tracked separately?

I think in the PR we want at least the ability to escape brackets in property names, so that this isn't a feature takeback (will still be a compat break, but with a workaround).  Ability to escape bracket literals outside of properties would be a totally new capability, so I'm fine with that being tracked separately if it doesn't come along for free with the former.
As a point of reference, DebuggerDisplayAttribute, which does essentially the same thing but for the debugger engine, uses \ escaping for brackets (unfortunately this isn't doc'ed, though it's known).

Alright, I should be able to get to the bracket escaping this weekend.

I spent way too much time trying to figure out how that DebuggerDisplayAttribute does it's handling of escaped brackets without any luck, so I just rolled my own implementation using regular expressions. I've added multiple test cases and everything seems good. I did notice that I was only able to find one other file that uses regexes (TypeProvidersImpl.fs), so I readily acknowledge that regexes are unusual here and might be frowned upon. I'm certainly open to other strategies, but this seemed the most straight-forward way to handle this.

The build failed due to a timeout???

Yes, there is a 40 min limit

I'm not sure what, if anything, my specific changes had to do with the timeout, but it appears it timed out during this MSBuild task:
C:\projects\visualfsharp-radou>"C:\Program Files (x86)\MSBuild\12.0\Bin\MSBuild.exe" vsintegration\fsharp-vsintegration-build.proj /p:Configuration=Release
This is the last line I can find from the output:
Setting up Visual Studio for debugging extensions. This one-time operation may take a minute or more.
C:\Program Files (x86)\Microsoft Visual Studio 12.0\Common7\IDE\devenv.exe /RootSuffix FSharp /ResetSettings General.vssettings /Embedding /Command File.Exit
I see the same issue occurred in the last 4.0 build so I am assuming this isn't specific to my PR.

This is working pretty well, thanks for the recent update.
What is the behavior you are aiming for?  Unbalanced/invalid use of non-escaped brackets causes bailout to default ToString?  This seems to be the case sometimes, but other times invalid brackets are tolerated and used as if they were escaped.

The behavior I am aiming for is naively try to use anything between an opening bracket and closing bracket as a property name. If that fails, then the old behavior is invoked, with an error message returned. If unbalanced brackets are found outside of an expected property name grouping, then bailing out to ToString() is done. I recognize the inconsistency and would probably lean towards using error messages in all cases, but I was hesitant to make that decision unilaterally.

The broad approach is that ToString is used if the format string is ill-formed, wheras an error message is used if the format string is fine but the specified properties can't be read. That's how StructuredFormatDisplay works today, and how DebuggerDisplay works, and we should just keep it that way.
The question is what do we consider to be an ill-formed format string.  Your approach is similar to what DebuggerDisplay does, so I think that's fine.
Though with that in mind I wonder why these don't both work, then?
[<StructuredFormatDisplay("{A} } {B}")>]  // prints "A } B", ok
[<StructuredFormatDisplay("{A} {B} }")>]  // defaults to ToString. Why not "A B }" ?

I have a good handle on why that is, and it is predominantly remnants of the old logic that slipped through the cracks. I'll hopefully have these scenarios handled in the next few days when I have time.

I have now got my test cases standardized to always revert to ToString() if a stray, unescaped bracket is detected.

Cool, great work.  Updated set of tests looks reasonable, I'll take another look soon.

Looks good to me.  Nice job, thanks for hanging in there and iterating on the design. This is a much more useful feature now :)
I do see a problem on portable profiles, but it's not in your new code.  Looks like existing getProperty can nullref in portable for some property names.  This can be tracked in a different bug. It works fine on desktop profiles.
test cenwoiwe12: expected
	'<StructuredFormatDisplay exception: Method 'Test+CheckDisplayAttributes12+Foo.Val{Hello' not found.>' but produced
	'<StructuredFormatDisplay exception: Object reference not set to an instance of an object.>'


style(containers): change order of container action buttons to match …

…ordering of quick action buttons in the container list page.
The ordering in the "quick actions" row in the container list is:

Log
Inspect
Stats
Console

Thanks @fabiang :+1:


docs(modal): add isInverse example

Also update isInverse example in storybook
fix #802

Codecov Report

Merging #809 (6ca22f2) into dev (c765e88) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #809   +/-   ##
=======================================
  Coverage   86.43%   86.43%
=======================================
  Files         175      175
  Lines        4453     4453
  Branches     1521     1521
=======================================
  Hits         3849     3849
  Misses        297      297
  Partials      307      307

Continue to review full report at Codecov.

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


Custom objects

I'd use noa engine for a personal project where I have to load more than 200-300 different textures, depending on player positions (there would be rooms, and in each room there are pictures like a galery).
My question is, does the engine only loads the textures if they are visible, or they'll load anyways? and plus question: does custom meshes only rendered/added to the scene to the position when they'd appear on the scene?

Hey, glad to hear it and I hope the project is useful.
Very generally, the goal of this engine is to abstract away some complexities of dealing with voxels (chunking, meshing, raycasting, etc), and not do too much else. So for anything that's not a voxel, stuff like loading or unloading assets is mostly down to the 3D engine.
With that said, by custom meshes you mean voxels that use a user-defined mesh instead of a square, right? If so, then here's how things work currently: to define an object mesh, you create it and noa saves a copy, but it's not added to the scene. At this point I'd imagine any related textures are in browser memory but not in GPU memory. Then once the scene begins, noa creates and renders all voxels within a certain range of the player, and for any custom meshes it will create mesh instances from the original. Then as the player moves around, noa creates new voxels and instances and destroys the old ones that got too far away.
Does that help? Basically noa doesn't cull anything based on visibility, just on range. Of course, if needed you could manually delete voxels when you know the player has left that room, and add them back if they return, etc.

Yeah, that helped thank :), what option should I tweak to set "voxel/mesh/object" loading, unloading distance?

These settings here :smile:


Finish prob rept adoption into issue cred present proof

Addresses https://github.com/hyperledger/aries-cloudagent-python/issues/1194


Cleanup unused code in ProcessInfo

Cleaned up unused code in ProcessInfo.
resolves: rdar://122976909

@swift-ci please test


chore(deps): update dependency @types/lingui__core to v2.7.1

This PR contains the following updates:

Package
Type
Update
Change

@types/lingui__core
devDependencies
minor
2.2.1 -> 2.7.1

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

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

v6.0.0
GitHub release

Your semantic-release bot :package::rocket:


MODINREACH-347: Record Contribution Errors Should not Cause Contribution Jobs to Halt

MODINREACH-347 (https://issues.folio.org/browse/MODINREACH-347)
Purpose

When errors occur during record contribution, the error should be logged, the record skipped, and the overall contribution job should continue running until all instances have been processed for contribution. In the current implementation, the job is completing after a few thousand records have been processed after receiving an unrecognized error code from the central.

Approach

Redesign of existing Kafka Consumer
Implemented concurrency.
Implemented Kafka retry along with Spring retry.
Implemented policy for when what retry should trigger.
New counting logic added which is used to stop Kafka consumer from processing in case of total completion of records or retry exhaustions.
Restructure of existing business logic in order to fit the current implementation.
Test cases added and modified accordingly.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.5 out of 6 committers have signed the CLA.:white_check_mark: steveellis:white_check_mark: Arghya22:white_check_mark: arin-epm:white_check_mark: gurleenkaurbp:white_check_mark: Vignesh-kalyanasundaram:x: Vignesh-KalyanasundaramVignesh-Kalyanasundaram seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@Arghya22 rather than comment every line, I would prefer if you could go through it again line by line and please make the following changes:

Remove necessary characters from log messages (for example: ::, ---, ->, -->>, =  and probably others). This just clutters things up and doesn't add value.
If a log message has a parameter, introduce that consistently with a colon and a space log.info(Some message: {}, param). You do this some places but not others.
Always use  the {} syntax rather than concatenating the string with + when you have a parameter or parameters in your log messages.
Check your use of spaces throughout. For example, method parameters should always have a comma and a space between them. Sometimes you have no spaces (or some with spaces and some without). Sometimes you introduce conditions without a space like if(something.
Remove unnecessary empty lines.
Some of the info messages that you have seem more like debug messages. Either change them to debug if you think they might be useful in the future, or remove them entirely.
Look at each of your log messages and imagine yourself trying to debug a production system. Make sure each message succinctly and describes with proper grammar what is being logged and if there is something like a param or params that can uniquely identify it, add that. For example you have two InnReachConnectionException occur---. These are each separate things.

Also @Arghya22 please go through the code smells report on sonar and fix as many as you can. These are almost always worth addressing. As for the ones about code not being addressed in unit tests, let's discuss that. It may be that we can ignore those for now since you're at 80% coverage, but the rest are easy and necessary fixes.


Add Redditor.trophies()

Close #956

Fixed. Anyway, just for curiousity, why those kinds are configurable?

Kinds are configured because the type id tX doesn't always match up on custom installs as they do with Reddit's default. For instance, at AppFolio we have a custom install of Reddit for internal use, and there I have overwritten the kind mapping in my praw.ini file to:
message_kind: t6
submission_kind: t5
subreddit_kind: t4

Fixed. Coveralls says coverage decreased but I'm not sure how to fix it (it seems uncovered statements are covered in Python 2 build).

Sometimes coveralls doesn't update the status after one of the builds goes through.

Thanks for the PR! 🎆


poly: add a test for lower_mul with a divisor that has a non-1 leading coefficient

The initial lowering for poly.mul involves computing a remainder of the product modulo the ring's polynomial modulus. This in turn computes an inverse of the leading coefficient of the divisor. E.g., for a poly mod of 3x^1024 + 1, this would be the inverse of 3 modulo the coefficient modulus of the ring.
There are two more cases to test: that the division uses the correct inverse when an inverse exists (e.g., 3 mod 1024), and that the lowering fails with a sensible error message when the leading coefficient has no inverse (e.g., 2 mod 1024).

This can be added via the testing framework introduced by https://github.com/google/heir/pull/210


Update to 1.20

Here are a couple small changes that I made to update the mod to 1.20.

Thanks for the update! Merging now

Hello!  Discovered the mod some months ago, love that you made a fork and updated to newer versions of Minecraft.
Some feedback:

In your next update, can you add the option to adjust the padding/margin variables from the config file? So that one can more precisely align it with other HUDs.
If the position is selected as "bottom_left" it interferes when typing to the in-game chat. Perhaps it could auto hide when the input box is displayed. (or give the option 1, to allow the possibility to fine tune the HUD position by ourselves)

PS: Apologies if this is the wrong location to provide feedback. Was unable to find other ways.
Best wishes!


[#5411] support option for tracing embedded library

@emeroad
What do you think about adding a variable in DefaultFrameAttachment class
such as
private ServiceType disabledLibrary

I think it's better to use DefaultTrace.addScope (String)
It is recommended not to modify core api as possible.

Codecov Report

Merging #5413 into master will decrease coverage by 0.02%.
The diff coverage is 3.44%.

@@            Coverage Diff             @@
##           master    #5413      +/-   ##
==========================================
- Coverage    40.9%   40.88%   -0.03%
==========================================
  Files        2761     2761
  Lines       83526    83553      +27
  Branches    11131    11134       +3
==========================================
- Hits        34165    34157       -8
- Misses      46364    46399      +35
  Partials     2997     2997

Impacted Files
Coverage Δ

...avercorp/pinpoint/profiler/context/AsyncTrace.java
31.48% <0%> (-3.94%)
:arrow_down:

...orp/pinpoint/profiler/context/AsyncChildTrace.java
0% <0%> (ø)
:arrow_up:

...ercorp/pinpoint/profiler/context/DefaultTrace.java
62.75% <0%> (-6.19%)
:arrow_down:

...ercorp/pinpoint/profiler/context/DisableTrace.java
33.33% <16.66%> (-3.04%)
:arrow_down:

...p/pinpoint/rpc/client/WriteFailFutureListener.java
56.25% <0%> (-18.75%)
:arrow_down:

...vercorp/pinpoint/rpc/packet/ServerClosePacket.java
52% <0%> (-16%)
:arrow_down:

...rp/pinpoint/rpc/server/PinpointServerAcceptor.java
84.93% <0%> (-4.8%)
:arrow_down:

...navercorp/pinpoint/rpc/common/SocketStateCode.java
82.92% <0%> (-2.44%)
:arrow_down:

...rc/main/java/com/navercorp/pinpoint/test/Item.java
70% <0%> (ø)
:arrow_up:

...cluster/zookeeper/ZookeeperClusterDataManager.java
64.7% <0%> (+2.2%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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


Fangyi Cao HW

question 4a is incomplete other than that should be fine
thanks!!

Thank you!

4.a is close enough, you just need print(R_s.to('m')) . I'll give it to you.


No support for transactWrite API

As I see it the new transactWrite APi is not supported yet. Would it be possible to add support for this similar to the batchWrite

@warrify  If you need a working typescript library for this today, checkout [dynamo-easy] (https://github.com/shiftcode/dynamo-easy)
disclaimer: I am one of the maintainers of dynamo-easy

@michaelwittwer Thanks I checked it out and liked it very much 👍

Is this on the roadmap?


Is is possible to generate borders with Markdown tables

The panel group redirected me to your issue tracker to see if markdown can provide remedy for this issue.
I am using panel that uses markdown and I am trying to generate a table with borders. Here is the code I am using in panel:
import panel as pn
from bokeh.resources import INLINE

SimpleTable = pn.panel("""
|     Time    | Number of Trial with Results | Unique Units |
|:-----------:|:----------------------------:|:------------:|
| 20 Apr 2018 |            30,763            |    21,094    |
|  7 Feb 2019 |            34,751            |    23,733    |
| 12 Apr 2019 |            35,926            |    24,548    |
|             |                              |              |

""")

SimpleTable.save('SimpleTable', resources=INLINE)

The code generates a nice table, yet without borders.

Now the question is whether you can force table borders to be drawn in markdown or are visual look aspects need to be handled elsewhere by panel, bokeh, or some CSS.
It may be a question open for debate - will be happy to get feedback.

For what it's worth, I think what Panel is doing with the input above is effectively:
import markdown

str = """
|     Time    | Number of Trial with Results | Unique Units |
|:-----------:|:----------------------------:|:------------:|
| 20 Apr 2018 |            30,763            |    21,094    |
|  7 Feb 2019 |            34,751            |    23,733    |
| 12 Apr 2019 |            35,926            |    24,548    |
|             |                              |              |
"""

extensions = ['markdown.extensions.extra', 'markdown.extensions.smarty']
html = markdown.markdown(str, extensions=extensions, output_format='html5')

The resulting html looks like this in a web browser if put in a bare page:

And like this in a Jupyter notebook cell:
from IPython.display import HTML
HTML(html)

So it's possible to style it, presumably using the usual table CSS properties.

In that case, unless Markdown developers want to add more features related to CSS style, it seems we can close this in this issue tracker.

Python-Markdown simply outputs standard HTML for tables.  As explained in the Markdown syntax rules "HTML is a publishing format; Markdown is a writing format." Whether borders are displayed and/or the style of those borders are publishing matters and outside the scope of Markdown. However, basic CSS rules for styling the HTML will work fine. In fact, as @jbednar points out, different tools use different default styles for the same input. If you want to use custom styles, you need to explore what provisions your tool uses to define custom styles.

Thanks Waylan,
There is a temporary solution found that can be found here: https://github.com/pyviz/panel/issues/668
It may not be very elegant, yet it seems to work for a simple example. Thanks for the feedback.


Not allowed to use NOT to filter on Status

Filtering on status and unable to use NOT Status==Deleted. A dialog is displayed with an explanation that we didn't find elucidating.

The use of expression trees for stream selection is not yet powerful enough to allow the user to specify any old shape of expression.
There are various limitations caused by the underlying storage of that meta data and the way we currently have to query it.
The error seems clear enough, the problem I found is that the processing filter UI doesn't give you the same dialog, but that error is instead dumped to the console and the pipeline fails to process. This is completely invisible to the UI.


should system.ListenAndServeMetrix take ctx?

		// TODO: should system.ListenAndServeMetrix take ctx?

https://github.com/filecoin-project/bacalhau/blob/117a53f7eac3b9650d5264adb8dc5f1958f6f32b/cmd/bacalhau/serve.go#L201

Probably a no, given that it uses a CleanupManager for lifecycle management 👀

SG, feel free to remove the TODO and close this if you're sure :)

Currently system.ListenAndServeMetrics takes a ctx. Closing the issue
https://github.com/filecoin-project/bacalhau/blob/main/pkg/system/metrics.go#L16


License?

Would you mind adding a license to this project? I'd love to play with this but can't check it out without knowing the terms of use.  Thanks!

Sorry about that! I just added (via mobile) the MIT license to match the CommandLineParaer itself.


Cannot install: cannot find package "github.com/hashicorp/consul/consul/structs"

Cannot install:
 $ go get -u github.com/outbrain/consul-raftdb-reader
package github.com/hashicorp/consul/consul/structs: cannot find package "github.com/hashicorp/consul/consul/structs" in any of:
	/usr/lib64/golang/src/github.com/hashicorp/consul/consul/structs (from $GOROOT)
	/home/stas/go/src/github.com/hashicorp/consul/consul/structs (from $GOPATH)

Looks like github.com/hashicorp/consul/consul/structs is gone somewhere

I had the same issue but I got around it by adding the contents of the structs directory manually..
Download the zip file here
https://github.com/hashicorp/consul
Copy the contents of "consul-master.zip\consul-master\agent\structs" to your go path as specified in the error above
/home/stas/go/src/github.com/hashicorp/consul/consul/structs (from $GOPATH)
then cd into your consul-raftdb-reader directory
cd /root/go/src/github.com/outbrain/consul-raftdb-reader
then run
go install
That should do it

Pull request for this change was rebased and merged to the base branch, Thanks!


Exposing livenessProbe and readinessProbe Configuration in Task Manager Pod Spec

Right now the following is the default provided:
  livenessProbe:
    failureThreshold: 5
    initialDelaySeconds: 5
    periodSeconds: 60
    successThreshold: 1
    tcpSocket:
      port: 6123
    timeoutSeconds: 10

Occasionally we encounter an unusual situation where the task manager did not reach RPC port start up, and was terminated by k8s. But when the port starts up while the pod is in terminating state, the job gets submitted, and killed eventually, creating a restart loop (which after some time resolves when magically the task manager starts before 25 seconds).
I can work on this and create pull request here if you think this is useful.

Hi @hjwalt yeah I think that exposing these can be useful indeed; If you can create a PR please do.

Hi @hjwalt and @regadas ,
I was as well facing this and raised it in issue 94. I will remove it there, because now it is duplicated. I have two more points.

In the GCP Operator it was actually working to set livenessProbe and readynessProbe. I'm not sure, why it has been removed. I thought it might be helpful to mention this.
Would you mind to implement it for the jobmanager as well?

@numbnut sure, I'll add the same for job manager

Hi @hjwalt , thank you!

Closed via #111


WIP: upgrade sops to v3.8.1

Description
Fixes #2890 .

TODOs
Read the Gruntwork contribution guidelines.

[ ] Update the docs.
[x] Run the relevant tests successfully, including pre-commit checks.
[x] Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
[x] Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Upgrade sops to new location with dependencies.

Migration Guide

Merged in another PR


standardize dim checks

Fix for #125

Further work blocked, awaiting bug #153 to be addressed

@tennlee @nicholasloveday @rob-taggart Replacing all calls with the new version of gather_dimensions.
I think I caught all the old stuff and the references to the two gather_dimensions versions in docstrings.
I'll also raise another issue to make it so keyword arguments must be used instead of positional as this can lead to bugs with default args.

@tennlee Bit of a messy conflict resolution as develop had moved on quite a bit in three weeks, but hopefully resolved it all correctly.
I've modified, as suggested:

gather_dimensions accept dimensions rather than passing full data
use the previous tests for gather_dimensions

@tennlee Added further tests for coverage

Closing this MR. Another MR has been raised to address this work.


How about retain prop? (easy & cheap Relay Classic caching behaviour)

Would it make sense to implement a prop to disable GC (a'la Relay Classic behaviour) to this package?
I find it wasteful, even annoying to fetch all the data again & again in most of my own apps. Very few (browser) apps are very long-lived & I don't belive most of our apps ever get memory problems, especially if you don't target emerging markets (very low-end devices).
It would also be pleasantly flexible for different strategies, for example:

List view - dynamic, updates often, don't retain: <QueryRenderer other={"props"}/>
Single item view - retain it: <QueryRenderer retain other={"props"}/>

It would be very cheap because it would only be checked againts on componentWillUnmount or what?

But how would one even implement this? I think something like this, haven't seen any side-effects so far..
_release() {
  // ... _rootSubscription & _pendingFetch code -> nothing to do with retaining data

  // reference to retained data ("protected" from garbage collection) - jackpot, add condition
  if (!this.props.retain && this._selectionReference) {
    this._selectionReference.dispose();
    this._selectionReference = null;
  }
}

I haven't 100% figured out yet how onError & onNext might affect retained data in different situations.

@unirey sounds good to me. Do you mind opening a PR?

I added the retain prop in the latest version 2.0.1. Please let me know if this works for your use case!


UnhandledPromiseRejectionWarning: Error: Too Many Requests

All good until you start having lots of calls. Then it stops responding

I am also getting too many requests. After multiple requests to FACEAPP api.  Have you got any solution to this?
Is any any barrier to number of requests per hour or something like that?

As far as I can tell you can process 50 images every 10 minutes.
This is implemented on their servers and there is nothing that this library can do to work around it.

Some sites in my niche have found a way
Example:
https://en.nametests.com/test/what-would-you-look-like-as-a-bald-person/30984/
This is a site with millions of visits and they have lots of tests using FaceApp API and they do not seem to have this issue.
How did they do it? Do they have some kind of official partnership with FaceApp?

faceapp.js is an unofficial reverse-engineering of the way the mobile clients work, so they're subject to any of their limitations.
I would assume that the site linked probably have a more official implementation from the company themselves. It also seems that the rate limits are a recent addition as I've only started getting told about it being a problem in the last couple weeks or so.
I don't have any plan to attempt to get around them as that would be breaking their terms of service. You may do so at your own risk.


Pairing multiple devices?

I noticed your Keycard applet has a configurable number of pairing slots, which is hardcoded to 5 here. This would imply you want to allow multiple pairings for a given card.

Aside: We at GridPlus actually discovered issues with this (namely, a user who tries to pair with a new device when the card is at its limit would be unable to use the card since only a paired device can unpair). We have implemented an alternative that simply overwrites the first pairing slot whenever a pairing is made - this is better for our use case.

However, it seems like a user can only pair if a pairingSecret is loaded (used here) and the only way to load it is via the INIT command (here you can see the command getting blocked if pairingSecret has already been set).
Is this intentional? It seems odd to have the following both be true:

5 pairing slots
Only the client that initializes the card can ever pair with it

Is there another way to allow pairing with multiple devices?

Aside: We are working on a "re-init" function (PR here), which allows a new device to load its own pairingSecret after the card has been initialized. Of course this means that the old pairing will be invalid, but this is not a problem for us (see earlier Aside).

That's not exactly how it works. The pairingSecret is a key generated from an externally known secret. In our case is the PBKDF2 output of a password, but this is not enforced by the card.
When the card is uniniitalized it has no credentials, so no PIN/PUK/Pairing password. In this state the card accepts a pairing-less one-shot SecureChannel to bring the credentials on-card. The credentials can then be changed by the user at any time (all of them). Changing the pairingSecret does not break existing pairings, because the actual pairing key is different for each pairing and stored individually.
You can pair up to 5 devices at the same time, with the user inserting the pairing password on each. The entire pairing mechanism is needed to counter active MITM attacks and that's why an off-band secret is needed. Overwriting a slot would lock-out a previously paired device, that's why we do not allow it. Although that could be an option if implemented in a way which requires user interaction/authentication.
There are scenarios where pairing is not desired, such as at a POS. We already provided a subset of commands which can be executed without pairing.

Interesting. Can you point me to how a client can update the pairingSecret?
I do see how additional pairings can be made, but they seem to require knowledge of the pairingSecret, which I would assume shouldn't be passed from client to client.

Please check https://status.im/keycard_api/sdk_pinless.html about this

Got it, thanks for clarifying!


support windows iso on linux

Etcher tool is very cool for make bootable usb iso but i see just support linux iso not support windows
Pleas make etcher is better support windows iso on linux
Thanks

Duplicate of https://github.com/resin-io/etcher/issues/210.
Etcher currently supports any image that is bootable out of the box. Windows images need special treatment to be made bootable, that we might implement at some point.


Permission dialog never displayed when Target Android Version is set to "Automatic"

Bug
On Android 6.0+ The plugin doesn't display a permission dialog when the Target Android Version is set to "Automatic - user target framework version (API 25)"
The ReadMe does say

You MUST set your Target version to API 23+ and Compile against API 23+

Does setting the Target Android Version to  "Automatic - user target framework version (API 25)" meet this criteria, or does the instruction mean that the Target Android Version must be set explicitly?
Version Number of Plugin: 1.2.1
Device Tested On: Nexus 5x running Android v7.1.1
Simulator Tested On: N/A
IDE: Xamarin Studio

Expected Behavior
Plugin should display the permission dialog
Actual Behavior
No permission dialog is displayed
Steps to reproduce the Behavior
1.)Uninstall existing app
2.)Open Android Manifest
3.)Set Target Android Version to "Automatic - user target framework version (API 25)"
4.)Install and run app

Closing this issue because I just noticed this one opened previously:
https://github.com/jamesmontemagno/PermissionsPlugin/issues/36
Basically: The Target has to be explicitly because otherwise its not included in the AndroidManifest.xml.


加载预训练模型，动转静态图报错。

export_model.py
import paddle
from config import get_config
from cait import build_cait as build_model

# config files in ./configs/
config = get_config('./configs/cait_xxs24_224.yaml')
# build model
model = build_model(config)
# load pretrained weights
model_state_dict = paddle.load('./cait_xxs24_224.pdparams')
model.set_state_dict(model_state_dict)
# dynamic to static
input_spec = paddle.static.InputSpec([None, 3, 224, 224], 'float32', 'image')
model = paddle.jit.to_static(model, input_spec=[input_spec])
# save inferenc model
paddle.jit.save(model, "infer/cait_xxs24_224")

TypeError: In transformed code:
File "/home/aistudio/CaiT/cait.py", line 493, in forward
x = self.forward_features(x)
File "/home/aistudio/CaiT/cait.py", line 481, in forward_features
# Self-Attention blocks
File "/home/aistudio/CaiT/cait.py", line 373, in forward
x = self.attn(x)
File "/home/aistudio/CaiT/cait.py", line 228, in forward
    B, H, C = x.shape # H: num_patches
    qkv = self.qkv(x).chunk(3, axis=-1)
    q, k, v = map(self.transpose_multihead, qkv) #[B, num_heads, num_patches, single_head_dim]
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE

    q = q * self.scale

File "/home/aistudio/CaiT/cait.py", line 220, in transpose_multihead
new_shape = x.shape[:-1] + [self.num_heads, self.dim_head]

TypeError: can only concatenate tuple (not "list") to tuple

Thanks for the issue.
We are fixing this problem now and will update once finished.


phantomjs 1.8.1 crash

<EMAIL_ADDRESS>commented:

Version 1.8.1
What steps will reproduce the problem?
accessing a js heavy page (backbone app)
Which operating system are you using?
Linux 3.2.0-36-generic #57-Ubuntu
Converted stacktrace included!

Disclaimer:
This issue was migrated on 2013-03-15 from the project's former issue tracker on Google Code, Issue #1039.
:star2:   1 people had starred this issue at the time of migration.

Outdated.


[Request] Could we add request-info for this repo?

We have seen some issues / PRs that have no information on the description.
To the admins: Would it make sense to have a probot app installed for this repo?

Done :)


Get ScholarSphere working with the Arkivo API and improve the documentation

We should probably talk this through before merging. I'm not sure the tests pass because this upgrades the stack to the latest in Sufia master (which contains an Arkivo fix). So this Sufia PR needs merging before #185 will work: https://github.com/projecthydra/sufia/pull/1313

rm -rf ./node_modules/zotero/node_modules/ws/node_modules/bufferutil

Talked about this on the 9/14 standup. We're going to let this PR :cool: out until after Hydra Connect, in hopes that Zotero's "My Publications" goes live in production before then (giving us a thing that we can test locally and predictably).

@mjgiarlo Looks good.  How will we know when My Publications goes online?

@DanCoughlin or I should get contacted, which we can pass on.


feat: documentation moved to mintlify with api extractor and typedoc markdown generator for typescript code files

Adding guide as mention in Issue #30

:tada: This PR is included in version @wapijs/wapi.js-v0.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


#36 - Save locations for easy access

A user may save/unsave their favorite places upon searching for a location and clicking on the save button.
They may then view an updated list of of their saved places by going to My Places from the nav menu.

Overall great changes! I like this feature, it's really handy. I only have a few comments
Functionality-wise:
I noticed that when you press on the directions button on a saved place, it brings you to the search fragment (as intended) but the destination is an address & the navigation button cannot be clicked (it is not highlighted - Figure 1). You have to press again on the address listed below to enable the navigation toggle. Would it be possible to look into this and fix it?
I compared with when pressing directions on a place from the info card (Figure 2), it brings you to search fragment and the name of the place is displayed (instead of the address) and the navigation button is highlighted (can be clicked) -> Figure 3.
Figure 1

Figure 2

Figure 3

TLDR;
Figure 1 should be like Figure 3 -> the navigation button is highlighted upon pressing directions to a place & the destination textbox should display the name of the place instead of the address (debatable)
Code-wise:
Some formatting nitpicks. Mostly empty lines.
As discussed in the meeting, a fix will be discussed to "fetch the current location" - as I wrote in my last inline comment, it has to do with handling a Task.

Functionality-wise:
I noticed that when you press on the directions button on a saved place, it brings you to the search fragment (as intended) but the destination is an address & the navigation button cannot be clicked (it is not highlighted - Figure 1).

Fixed with ff83447 during a pair programming session. Thank you for the fix @makchamp
Only minor formatting comments are left  😄

When pressing the navigation button from a place card in the My Place fragment, the destination textbox shows the name of the place AND its respectful address. Thanks @makchamp for the fix ->
7604a6f
All the comments have been addressed by @meshgin . I fixed the conflicts with the master branch and committed a last refactoring change. Will merge now 🛩


Update _Windows-Install-Prerequisites.rst

With a note about the Python path.
Solves #3914
I would suggest to merge this both for the Humble tag and latest, if possible.

Should I squash these commits? Or will that happen during merging?

Should I squash these commits? Or will that happen during merging?

I'll do it during the merge.

@Mergifyio backport iron rolling