Cut on log of critical value rather than raw value

Hi @HajimeKawahara and @ykawashima,
I'm working with the dev branch and had problems querying the new HDF5 tables via the crit keyword. I still can't figure out exactly what the cause of the problem is, but the where command in pd.read_hdf is not respecting the critical Sij0. I'm wondering if this has to do with the floats we use to represent small values of Sij0. For example, I've been using TiO with crit~1e-16 and finding that the MdbExomol object that gets returned is empty, even though there should be some lines in my wavelength range.
In my experimentations, I found that I could fix this if I alter the where query based on the log of the Sij0 values, which is what I've implemented in this PR. This makes for a larger HDF5 database, but the tradeoff is that you can then successfully read-in only portions of it. This PR will only "work" if you delete your existing HDF archives.
If you have any insight on how to improve this PR to make the old behavior work correctly, your help is appreciated and I'll happily close this PR 😄

After some more tests, unfortunately this seems not to be working correctly either.

Hello @bmorris3 !
Can you give us an example that occurs the error when using the current implementation?

Sure, it looks like this:
In [1]: import pandas as pd

In [2]: trans=pd.read_hdf('/.database/TiO/48Ti-16O/Toto/48Ti-16O__Toto.trans.hdf', where=['Sij0 > 1e-16'])

In [3]: trans
Out[3]:
Empty DataFrame
Columns: [i_upper, i_lower, A, nu_lines, Sij0, logSij0]
Index: []

OH, this could be the problem! Is it possible that the crit/Sij0 values have changed compared to the previous version with the feather implementation? I was using the crit that gave me a certain number of lines before, but now it gives me no lines every time. It looks like the maximum Sij0 is different by orders of magnitude compared to the previous implementation.

Hi @bmorris3
Thank you so much for pointing this out! This is just from my quick check, but I have realized that when using Toto TiO line list for the first time, the code somwhow yields the following warning when calculating S0, which did not happen when I checked for Li2015 CO line list.

/home/kawashima/anaconda3/lib/python3.8/site-packages/exojax-0.5.5-py3.8.egg/exojax/spec/exomol.py:24: RuntimeWarning: invalid value encountered in true_divide
S0=-Agnp.exp(-hcperk*elower/Tref)np.expm1(-hcperknu_lines/Tref)
.broad is used.

As this may be related, I will look into it. Sorry.

No problem, thanks for the quick response!

It looks like the TiO file has some zero values for the nu_lines column:
In [2]: trans = pd.read_hdf('.database/TiO/48Ti-16O/Toto/48Ti-16O__Toto.trans.hdf')
trans

In [3]: trans
Out[3]:
          i_upper  i_lower             A      nu_lines          Sij0    logSij0
0             601      271  3.482300e-28      0.000000           NaN        NaN
1             602      272  3.592300e-28      0.000000           NaN        NaN
2             603      273  3.712100e-28      0.000000           NaN        NaN
3             604      274  3.831200e-28      0.000000           NaN        NaN
4             605      275  3.961100e-28      0.000000           NaN        NaN
...           ...      ...           ...           ...           ...        ...
58983947   130411   127539  3.845600e-10  30000.011066  1.083500e-44 -43.965171
58983948   165661   162738  2.142300e-05  29999.813031  4.884841e-50 -49.311150
58983949   164872   163548  2.142100e-05  29999.813031  4.884384e-50 -49.311190
58983950   142828   141383  9.992600e-05  29999.997066  1.212398e-65 -64.916355
58983951   142101   142182  9.993300e-05  29999.997093  1.212483e-65 -64.916324

This might be causing the problems in the exomol.Sij0 function which divides by nu_lines**2 here.

Yes, I have also realized that. I found that e_lower and e_upper are calculated to be the same values for some lines for the Toto TiO line list, resulting in zero values for nu_lines. But I have not started to check why this happens. Sorry.

It looks like this is a problem with the original transition file:
In [1]: import pandas as pd

In [2]: dat = pd.read_csv('.database/TiO/48Ti-16O/Toto/48Ti-16O__Toto.trans.bz2',compression="bz2",sep="\s+",nam
   ...: es=("i_upper","i_lower","A","nu_lines"))

In [3]: dat
Out[3]:
          i_upper  i_lower             A      nu_lines
0             601      271  3.482300e-28      0.000000
1             602      272  3.592300e-28      0.000000
2             603      273  3.712100e-28      0.000000
3             604      274  3.831200e-28      0.000000
4             605      275  3.961100e-28      0.000000
...           ...      ...           ...           ...
58983947   130411   127539  3.845600e-10  29999.992723
58983948   165661   162738  2.142300e-05  29999.995951
58983949   164872   163548  2.142100e-05  29999.995961
58983950   142828   141383  9.992600e-05  29999.997066
58983951   142101   142182  9.993300e-05  29999.997093

[58983952 rows x 4 columns]

Perhaps a simple where condition in exomolapi.read_trans could skip over these lines with zero nu_lines values?

It seems that switching trans_lines=True in pickup_gE avoids this problem! Should trans_lines=True be the default?

Glad to hear that it works though I am a bit wondering whether the lines with nu_lines=0 can be just ignored.
As nu_lines information is the optional information as written in Table 12 of  Exomol's paper (please see also the explanation at the beginning of pickup_gE @HajimeKawahara wrote), trans_lines=False is set to be the default as far as I understand.

Note that the reason why trans_lines=False is default is that some line list in ExoMol does not have information (see #16).

This is just for your reference, but among all the 58,983,953 transitions in the Toto TiO line list, 3,402 transitions have zero values for the nu_lines in the original transition file. On the other hand, when calculated using the upper and lower energy difference like done here, 5,370 transitions have zero values for nu_lines since the lower and upper state IDs somehow indicate exactly the same energy levels.
I have confirmed that there are 3,249 transitions that satisfy the above both two cases, meaning that 3,402-3,249=153 transitions that have zero values for the nu_lines in the original transition file, but the nu_lines calculated from the upper and lower energy difference are non-zero. I am not sure whether those 153 lines are reliable, but you can include them if you want. This also means that 5,370-3,249=2,121 lines have non-zero values for the nu_lines in the original transition file, but the nu_lines calculated from the upper and lower energy difference are zero.
I think it is better to include a warning or something when nu_lines result in zero values to avoid this kind of this issue. I will work on it. Thank you very much for pointing this out @bmorris3!

Closed because this issue was fixed by #104

Hello ! Related to your discussion above, I see you're moving to HDF5.
We're doing a similar move in RADIS https://github.com/radis/radis/pull/306 but reading HDF5 from Vaex instead of Pandas (warning, it's not compatible. VaeX's HDF5 is column-based ; Pandas's PyTables is row-based). Saves memory and also time (may be relevant for @ykawashima). Proof of concept showing how querying/filtering is faster in Vaex can be found here: test_vaex.ipynb
We can discuss it further on the Radis Slack.

Hello @erwanp! Thank you very much for your useful information. Indeed, I have been worried that reading HDF5 from Pandas and saving Pandas to HDF5 are a bit slow. I will try using Vaex. Thank you very much!!

Keep us updated. @gagan-aryan is also working on similar improvement !


Issue search

It would be nice to do a text-based search on issues. (Feature request)
Does not have to be on description / comments - title would be a good start.
It can be like the "filters" at GitHub (more customization), but can be a lot simpler than that - at least for the start.

That's the way we are heading to, thanks!

I +1 this request.
I would also like the search box to show applied filters. Much like the filter box in GitHub (meaning to prefill the box with something like "tag:tagname milestone:milestonename" if those filters are applied)

I believe that this project could be helpful: https://github.com/etsy/hound

A thousand times yes!

Please follow on https://github.com/gogits/gogs/issues/13


Escape windows args with double quotes instead of single quotes

Right now, we wrap certain windows args (like filepaths) in single quotes, which don't get stripped out by the windows shell.

Awesome.

Tests are super flaky...did what I could to fix them but we've got serious race conditions. I can't repro any of these in my local environments.


can't run nk-example

Hi,
nk-example crash on my workstation, which infos can I provide to help debugging this issue ?
`[forth@mjolnir nk-example]$ go build -v
github.com/golang-ui/glfw
github.com/golang-ui/nuklear/nk
github.com/golang-ui/nuklear/nk
In file included from ../../nk/impl_glfw.go:16:0:
../../nk/nuklear_glfw_gl3.h: Dans la fonction ‘nk_glfw3_render’:
../../nk/nuklear_glfw_gl3.h:242:18: attention : assignment makes pointer from integer without a cast [-Wint-conversion]
vertices = glMapBuffer(GL_ARRAY_BUFFER, GL_WRITE_ONLY);
^
../../nk/nuklear_glfw_gl3.h:243:18: attention : assignment makes pointer from integer without a cast [-Wint-conversion]
elements = glMapBuffer(GL_ELEMENT_ARRAY_BUFFER, GL_WRITE_ONLY);
^
github.com/golang-ui/nuklear/cmd/nk-example`
`[forth@mjolnir nk-example]$ ./nk-example
2016/09/27 11:47:04 glfw: created window 400x500
fatal error: unexpected signal during runtime execution
[signal SIGSEGV: segmentation violation code=0x1 addr=0x5d8cc002 pc=0x4f7098]
runtime stack:
runtime.throw(0x57e87b, 0x2a)
/usr/lib/go/src/runtime/panic.go:566 +0x95
runtime.sigpanic()
/usr/lib/go/src/runtime/sigpanic_unix.go:12 +0x2cc
goroutine 1 [syscall, locked to thread]:
runtime.cgocall(0x51a210, 0xc42004dae8, 0xc400000000)
/usr/lib/go/src/runtime/cgocall.go:131 +0x110 fp=0xc42004da98 sp=0xc42004da58
github.com/golang-ui/nuklear/nk._Cfunc_nk_glfw3_render(0x8000000000001, 0x3f80000000020000)
??:0 +0x45 fp=0xc42004dae8 sp=0xc42004da98
github.com/golang-ui/nuklear/nk.NkGLFW3Render(0x8000000000001, 0x3f80000000020000)
/home/forth/go/src/github.com/golang-ui/nuklear/nk/nk.go:3482 +0x39 fp=0xc42004db08 sp=0xc42004dae8
main.gfxMain(0x22a8090, 0x834bb0, 0xc42000c3c0)
/home/forth/go/src/github.com/golang-ui/nuklear/cmd/nk-example/main.go:145 +0x2ba fp=0xc42004dd18 sp=0xc42004db08
main.main()
/home/forth/go/src/github.com/golang-ui/nuklear/cmd/nk-example/main.go:87 +0x652 fp=0xc42004df48 sp=0xc42004dd18
runtime.main()
/usr/lib/go/src/runtime/proc.go:183 +0x1f4 fp=0xc42004dfa0 sp=0xc42004df48
runtime.goexit()
/usr/lib/go/src/runtime/asm_amd64.s:2086 +0x1 fp=0xc42004dfa8 sp=0xc42004dfa0
goroutine 17 [syscall, locked to thread]:
runtime.goexit()
/usr/lib/go/src/runtime/asm_amd64.s:2086 +0x1
goroutine 5 [syscall]:
os/signal.signal_recv(0x0)
/usr/lib/go/src/runtime/sigqueue.go:116 +0x157
os/signal.loop()
/usr/lib/go/src/os/signal/signal_unix.go:22 +0x22
created by os/signal.init.1
/usr/lib/go/src/os/signal/signal_unix.go:28 +0x41
goroutine 6 [select, locked to thread]:
runtime.gopark(0x589010, 0x0, 0x57579a, 0x6, 0x18, 0x2)
/usr/lib/go/src/runtime/proc.go:259 +0x13a
runtime.selectgoImpl(0xc42002a730, 0x0, 0x18)
/usr/lib/go/src/runtime/select.go:423 +0x11d9
runtime.selectgo(0xc42002a730)
/usr/lib/go/src/runtime/select.go:238 +0x1c
runtime.ensureSigM.func1()
/usr/lib/go/src/runtime/signal1_unix.go:304 +0x2f3
runtime.goexit()
/usr/lib/go/src/runtime/asm_amd64.s:2086 +0x1
goroutine 7 [select]:
github.com/xlab/closer.(*closer).wait(0xc42007a000)
/home/forth/go/src/github.com/xlab/closer/closer.go:102 +0x2a2
created by github.com/xlab/closer.newCloser
/home/forth/go/src/github.com/xlab/closer/closer.go:94 +0x262`

@F0rth
Thanks for the report, I honestly haven't tested it on Linux yet. Here nk_glfw3_render crashes for some reason. You can debug with GDB (go build -work, then gdb) if you want.
I want to research that case, could you provide you linux workstation config? uname, distro and stuff.

@F0rth I switched to http://github.com/go-gl/glfw, this may help.

so I put a breakpoint at 28 and nk-example crashed after the 74th line:
`
74			select {
(gdb) n
Thread 1 "nk-example" received signal SIGSEGV, Segmentation fault.
nk_draw_list_fill_poly_convex (list=list@entry=0x8375c8 <glfw+10984>,
points=0xa3f470, points_count=4, color=..., aliasing=)
at /home/forth/go/src/github.com/golang-ui/nuklear/nk/nuklear.h:6964
6964	            ids[1] = (nk_draw_index)(vtx_inner_idx + ((i-1) << 1));
(gdb)
my config:
distro: Archlinux
uname: Linux mjolnir 4.7.5-1-ARCH #1 SMP PREEMPT Sat Sep 24 13:04:22 CEST 2016 x86_64 GNU/Linux
go:
go version go1.7.1 linux/amd64
gcc:
Utilisation des specs internes.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-pc-linux-gnu/6.2.1/lto-wrapper
Cible : x86_64-pc-linux-gnu
Configuré avec: /build/gcc-multilib/src/gcc/configure --prefix=/usr --libdir=/usr/lib --libexecdir=/usr/lib --mandir=/usr/share/man --infodir=/usr/share/info --with-bugurl=https://bugs.archlinux.org/ --enable-languages=c,c++,ada,fortran,go,lto,objc,obj-c++ --enable-shared --enable-threads=posix --enable-libmpx --with-system-zlib --with-isl --enable-__cxa_atexit --disable-libunwind-exceptions --enable-clocale=gnu --disable-libstdcxx-pch --disable-libssp --enable-gnu-unique-object --enable-linker-build-id --enable-lto --enable-plugin --enable-install-libiberty --with-linker-hash-style=gnu --enable-gnu-indirect-function --enable-multilib --disable-werror --enable-checking=release
Modèle de thread: posix
gcc version 6.2.1 20160830 (GCC)
`

It compiles here (latest Ubuntu Xenial) but it crashes immediately after creating the application window.

@vruz thanks for the report, I'll try to debug this case on the next weekend, tomorrow gonna move to dotGo, so no time for playing with Linux..

No worries. I'd be willing to look into this problem and offload some of
the work from you debugging it if you give me some rough pointers or
starting directions where the problem could possibly occur. But no rush,
I'll check back next week!
On 8 October 2016 at 17:38, Maxim Kupriianov<EMAIL_ADDRESS>wrote:

@vruz https://github.com/vruz thanks for the report, I'll try to debug
this case on the next weekend, tomorrow gonna move to dotGo, so no time for
playing with Linux..
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang-ui/nuklear/issues/1#issuecomment-252447099,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABboiYddnsdsMMbVSlL8kKJd7FN9qLIks5qx_9JgaJpZM4KHduA
.

Related #2, probably the nuklear_glfw_gl3.h backend for Nuklear should be replaced. Will research that this Saturday.

@F0rth @vruz guys sorry for delay. I've managed to run this thing on Windows, the problem with windows was that some old versions don't support OpenGL 3. So I migrated to OpenGL 2.1 backend and changed imports.
Please update the lib and check if this helps, because this should definitely do the trick.

No worries, thank you for following up.
Re: Windows. I'm on Linux right now, but I'll test it and see what happens.
On 29 October 2016 at 09:45, Maxim Kupriianov<EMAIL_ADDRESS>wrote:

@F0rth https://github.com/F0rth @vruz https://github.com/vruz guys
sorry for delay. I've managed to run this thing on Windows, the problem
with windows was that some old versions don't support OpenGL 3. So I
migrated to OpenGL 2.1 backend and changed imports.
Please update the lib and check if this helps, because this should
definitely do the trick.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang-ui/nuklear/issues/1#issuecomment-257089763,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABbonbsYWjJLspD9HOYQuT_hmPWrhGmks5q40AAgaJpZM4KHduA
.

it's working ! (on archlinux).

Gj everyone, thanks for the reports.


Running predictions based on user input.

Hi can you give a sample implementation on how to run your model on specific user input from the pre-trained models? Can you guide with the demo pipeline to predict aspect sentiment for a user input in real-time.

I wanted to do some inference (Aspect Extraction and Aspect sentiment classification) using the hugging face transformer model. Can you share some guidelines on how to do that ?  On using AutoModel .. the model downloaded looks to be a plain BERT Model. Do we need to an additional layer over it ?
Any help will be appreciated!

we didn’t release a fine-tuned model on a specific task.
but you can easily tune on using semieval data.
best,
hu
On Tue, Feb 23, 2021 at 5:56 AM Prasad Nageshkar<EMAIL_ADDRESS>wrote:

I wanted to do some inference (Aspect Extraction and Aspect sentiment
classification) using the hugging face transformer model. Can you share
some guidelines on how to do that ? On using AutoModel .. the model
downloaded looks to be a plain BERT Model. Do we need to an additional
layer over it ?
Any help will be appreciated!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/howardhsu/BERT-for-RRC-ABSA/issues/17#issuecomment-784219960,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACRK37ZXQQJ4I6BDNFZKQQ3TAOXZXANCNFSM4VQLJFVA
.

--
Homepage: https://howardhsu.github.io/
Linkedin: https://www.linkedin.com/in/hu-xu-9852403b/
Google Scholar: https://scholar.google.com/citations?user=SaH2yWMAAAAJ
Twitter: https://twitter.com/Hu_Hsu
Email<EMAIL_ADDRESS>

we didn’t release a fine-tuned model on a specific task. but you can easily tune on using semieval data. best, hu
On Tue, Feb 23, 2021 at 5:56 AM Prasad Nageshkar @.***> wrote: I wanted to do some inference (Aspect Extraction and Aspect sentiment classification) using the hugging face transformer model. Can you share some guidelines on how to do that ? On using AutoModel .. the model downloaded looks to be a plain BERT Model. Do we need to an additional layer over it ? Any help will be appreciated! — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#17 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACRK37ZXQQJ4I6BDNFZKQQ3TAOXZXANCNFSM4VQLJFVA .
-- Homepage: https://howardhsu.github.io/ Linkedin: https://www.linkedin.com/in/hu-xu-9852403b/ Google Scholar: https://scholar.google.com/citations?user=SaH2yWMAAAAJ Twitter: https://twitter.com/Hu_Hsu Email<EMAIL_ADDRESS>
Thank you.


-fsanatize=undefined with -order -group gives an warning

Describe the bug
@eder-matheus @maliberty
Seems like -fsanatize=undefined found some uninitialized data... https://clang.llvm.org/docs/UndefinedBehaviorSanitizer.html
tools/OpenROAD/src/ppl/src/munkres/src/Hungarian.cpp:227:13: runtime error: signed integer overflow:<PHONE_NUMBER> +<PHONE_NUMBER> cannot be represented in type 'int'

Two uninitialized integers are added together:
https://github.com/The-OpenROAD-Project/OpenROAD/blob/d5b908a31e00957ed6c695144c76859863d29a4c/src/ppl/src/munkres/src/Hungarian.cpp#L227
See below, this message:
/home/oyvind/OpenROAD-flow-scripts/tools/OpenROAD/src/ppl/src/munkres/src/Hungarian.cpp:227:13: runtime error: signed integer overflow:<PHONE_NUMBER> +<PHONE_NUMBER> cannot be represented in type 'int'

Without -fsanatize=undefined
./build_openroad.sh --local --clean-force

(/usr/bin/time -f 'Elapsed time: %E[h:]min:sec. CPU time: user %U sys %S (%P). Peak memory: %MKB.' /home/oyvind/OpenROAD-flow-scripts/tools/install/OpenROAD/bin/openroad -exit -no_init  /home/oyvind/OpenROAD-flow-scripts/flow//scripts/io_placement.tcl -metrics ./logs/asap7/ce-array_ALUCE/base/3_2_place_iop.json) 2>&1 | tee ./logs/asap7/FooBar/base/3_2_place_iop.log
OpenROAD v2.0-7907-gd5b908a31
[deleted]
[INFO PPL-0044] Pin group: [ceEnableOut]
[INFO PPL-0080] Mirroring pins ceEnableOut and ce_enable.
Found 0 macro blocks.
Using 1u default distance from corners.
Using 2 tracks default min distance between IO pins.
[WARNING PPL-0042] Unsuccessfully assigned I/O groups.
[WARNING PPL-0092] Pin group of size 256 does not fit any section. Adding to fallback mode.
[WARNING PPL-0042] Unsuccessfully assigned I/O groups.
[WARNING PPL-0092] Pin group of size 384 does not fit any section. Adding to fallback mode.
[WARNING PPL-0078] Not enough available positions (53) in section (120.0, 58.572)-(120.0, 77.676) at edge RIGHT to place the pin group of size 59.
[ERROR PPL-0089] Could not create matrix for groups. Not available slots inside section.
Error: io_placement.tcl, 13 PPL-0089
Command exited with non-zero status 1
Elapsed time: 0:01.17[h:]min:sec. CPU time: user 1.12 sys 0.04 (99%). Peak memory: 211064KB.
make: *** [/home/oyvind/OpenROAD-flow-scripts/flow/Makefile:504: results/asap7/FooBar/base/3_2_place_iop.odb] Error 1

With -fsanatize=undefined
./build_openroad.sh --local --clean-force --openroad-args -DCMAKE_CXX_FLAGS=-fsanitize=undefined

Elapsed time: 0:10.56[h:]min:sec. CPU time: user 10.40 sys 0.15 (99%). Peak memory: 719284KB.
(/usr/bin/time -f 'Elapsed time: %E[h:]min:sec. CPU time: user %U sys %S (%P). Peak memory: %MKB.' /home/oyvind/OpenROAD-flow-scripts/tools/install/OpenROAD/bin/openroad -exit -no_init  /home/oyvind/OpenROAD-flow-scripts/flow//scripts/io_placement.tcl -metrics ./logs/asap7/ce-array_ALUCE/base/3_2_place_iop.json) 2>&1 | tee ./logs/asap7/FooBar/base/3_2_place_iop.log
OpenROAD v2.0-7907-gd5b908a31
[deleted]
Found 0 macro blocks.
Using 1u default distance from corners.
Using 2 tracks default min distance between IO pins.
/home/oyvind/OpenROAD-flow-scripts/tools/OpenROAD/src/ppl/src/munkres/src/Hungarian.cpp:227:13: runtime error: signed integer overflow:<PHONE_NUMBER> +<PHONE_NUMBER> cannot be represented in type 'int'
[WARNING PPL-0042] Unsuccessfully assigned I/O groups.
[WARNING PPL-0092] Pin group of size 256 does not fit any section. Adding to fallback mode.
[WARNING PPL-0042] Unsuccessfully assigned I/O groups.
[WARNING PPL-0092] Pin group of size 384 does not fit any section. Adding to fallback mode.
[WARNING PPL-0078] Not enough available positions (53) in section (120.0, 58.572)-(120.0, 77.676) at edge RIGHT to place the pin group of size 59.
[ERROR PPL-0089] Could not create matrix for groups. Not available slots inside section.
Error: io_placement.tcl, 13 PPL-0089
Command exited with non-zero status 1
Elapsed time: 0:02.01[h:]min:sec. CPU time: user 1.88 sys 0.12 (99%). Peak memory: 702680KB.
make: *** [/home/oyvind/OpenROAD-flow-scripts/flow/Makefile:504: results/asap7/FooBar/base/3_2_place_iop.odb] Error 1

Expected Behavior
No difference with and without -fsanatizer=undefined
Environment
Git commit: d5b908a31e00957ed6c695144c76859863d29a4c
kernel: Linux 5.19.0-40-generic
os: Ubuntu 22.04.2 LTS (Jammy Jellyfish)
cmake version 3.22.1

To Reproduce
Checking for a public test-case, but the above clue will hopefully help.
Relevant log output
No response
Screenshots
No response
Additional Context
No response

Saw this one as well, might be worth a look:
/home/oyvind/OpenROAD-flow-scripts/tools/OpenROAD/src/gpl/src/nesterovBase.cpp:344:36: runtime error: signed integer overflow: -2147483648 -<PHONE_NUMBER> cannot be represented in type 'int'
/home/oyvind/OpenROAD-flow-scripts/tools/OpenROAD/src/gpl/src/nesterovBase.cpp:344:50: runtime error: signed integer overflow: -2147483648 -<PHONE_NUMBER> cannot be represented in type 'int'
[NesterovSolve] Iter: 1 overflow: 0.995667 HPWL: 7577714
[NesterovSolve] Iter: 10 overflow: 0.995671 HPWL: 2144047

@eder-matheus
https://stackoverflow.com/questions/2204176/how-to-initialise-memory-with-new-operator-in-c

@antonblanchard Your tip with -fsanatize=undefined paned out, it would seem. :-)


Migrate to Net Core 3.1

Net Core 2.2 which we use as a target framework for the project is not supported anymore.
Next LTS release for Net Core is 3.1 (Official support policy).
We should consider retargeting the project to NetCore 3.1.
It could not only bring us a framework that receives regular updates but also brings a feature that could minimize standalone package size and increase performance.

The project was in Net Core 3.1, the only reason we downgraded it so we could run it in the office computers. (Visual Studio 2017 is friend only with NetCore 2.2)
If we see we don't need to run it there any more, we should upgrade again.
Or we can ask for VS2019 hahah

OK, it's worth discussing at the office and put a solution here 😊


[Candidate 3v3 Nocky Mountains ZH v2] Units can move through GLA ornament, but not build over it

Observed behaviour
Units can move through GLA ornament, but not build over it.
Expected behavior
Units can not move through GLA ornament or can build over it.
Screenshots

yes you need to destroy it first to build there it got a hp bar

As designed?

yes


6157 Segmentation fault

Dear fmriprep community,
I get the following error when running fmriprep_docker-1.0.6.post2 on my unbuntu 16.
/usr/lib/ants/antsBrainExtraction.sh: line 117:  6157 Segmentation fault      (core dumped) $cmd
Return code: 1
Can anyone help me with that issue?
Thank you,
Boris

Most likely memory issue, have a look here https://github.com/poldracklab/fmriprep/issues/756#issuecomment-335968692

Hi @tragus1, could you confirm that this is a memory issue?

Yes problem solved it is a memory issue

Thanks for the feedback!


adding build-project time in test/jamfile

Description
Adding checks for time coordinate for detection in CI
References
#159
Tasklist

[ ] Add test case(s)
[ ] Ensure all CI builds pass
[ ] Review and approve

@Zyro9922 the tests are still failing. Did I make another mistake??


Changed from named single elements in varargs

IntelliJ IDEA 2017.3 gives this warning message:
Kotlin: Assigning single elements to varargs in named form is deprecated

The warning is removed if the names of the single arguments are removed.
If this pull request is accepted, I will make another one which removes the same warning in other files.

Thanks! I've fixed the warning in a different way: no more varargs, and more correct type for the references argument.


create JavaScript version

ditch Java2D - the drawing quality is abysmal compared to (modern) js frameworks like

Konva https://github.com/konvajs/konva or
Three.JS

http://raphaeljs.com/reference.html
https://jonobr1.github.io/two.js/
http://jdan.github.io/isomer/
https://www.javascripting.com/view/raphael
or plain HTML5 canvas drawing...


Not working debug mode in Edge on windows 10

I am using Edge Browser in Windows 10, the path to the dart sdk is served incorrectly.
I get this error message:

Invalid argument (requestedUri): handlerPath "null" and url "null" must combine to equal requestedUri path "/C:/work/flutter/lib/src/widgets/framework.dart".: Instance of '_Uri' dart:isolate  _RawReceivePortImpl._handleMessage

Hi @sharklead
Which version of Flutter are you using?
Looks like a duplicate of https://github.com/flutter/flutter/issues/60235
Can you please upgrade to the latest  master and try again
flutter channel master
flutter upgrade --force
flutter doctor -v
Thank you

Thanks you !
I solved my problem with this issue.


How to connect with the firebase?????

Follow this link or documentation to connect the app with firebase https://firebase.google.com/docs/android/setup


feat: allow one or more CSRs per request

Determined by CONTENT_TYPE, Steward will accept one CSR and issue one Certificate, or will accept a Vec<Certficiate> and return a struct containing separate Vec<Certificate>s, one for the issuing chain, and one for the certificates issued.
Signed-off-by: Richard Zak<EMAIL_ADDRESS>

Closes #57


chore: tag container_test as no-remote-exec in js_image e2e test

Reason?

Reason?

It's failing at HEAD. No docker installed on the RBE worker. Although I'm not sure when it started to fail.


getting error

when I run sudo pihole5-list-tool I get this error:
Traceback (most recent call last):   File "/usr/local/bin/pihole5-list-tool", line 10, in <module>     sys.exit(main())   File "/usr/local/lib/python3.7/dist-packages/ph5lt.py", line 82, in main     print(color('    \u250c\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2510', fg='#b61042'))   File "/usr/lib/python3.7/encodings/iso8859_15.py", line 19, in encode     return codecs.charmap_encode(input,self.errors,encoding_table)[0] UnicodeEncodeError: 'charmap' codec can't encode characters in position 21-64: character maps to <undefined>

I know it's a different error, but have you tried setting/changing your locale/language options as talk about in #4 ?

Thx for the hint, this solved the problem!
Update worked quite well, nice tool!


Revert "force stable dependencies to prevent unwanted dependency evol…

…ution and build errors"
This reverts commit dc36fc0580f898ad0dfbdcaee4782f19f59bd31b.
The previous commit made the data binding for whatever reason impossible. The tables are not shown anymore on the entry page.

Data binding again fixed without reverting to dymanic dependencies.
See c4a7aba99d8a89e47ce015764aab7c7388dce60b.


key hijacking?

Microsoft PowerToys version
0.43.0
Running as admin

[X] Yes

Area(s) with issue?
Keyboard Manager
Steps to reproduce
Sometimes when starting up, keyboard manager "captures" or "hijacks" left ctrl key which I have assigned as part of a key remapping
PowerToysReport_2021-08-22-09-49-49.zip

✔ Expected Behavior
My remaping is just this

and it should only control those keys in the remapping scenario
❌ Actual Behavior
Sometimes after windows login, left ctrl stays in a "depressed state" as if somehow my remapping made the key unavailable for the rest of the system, this is worked around by repeatedly pressing said key physically until windows "reconquers" the key and everything else works fine from then on.

Other Software
No other app involved except windows itself

I'm closing this issue because I've found others with very similar symptoms


flinkx编译Kafka源码部分抛错

Caused by: org.apache.maven.plugin.compiler.CompilationFailureException: Compilation failure
/opt/software/flinkx-1.12_release/flinkx-connectors/flinkx-connector-kafka/src/main/java/com/dtstack/flinkx/connector/kafka/source/KafkaDynamicSource.java:[359,61] unreported exception X; must be caught or declared to be thrown

This issue has been too long to review it and reopen new one if necessary.


Midi usage prompt only shows when resuming Main activity after plugging in midi device

Currently a midi device (eg. midi keyboard controller) needs to be plugged in prior to starting the app in order for the "allow to use midi" dialog prompt to be shown. If a midi controller is plugged in while the app is running, need to leave and resume the Main activity (eg go to app Settings menu and return) for the prompt to be shown.

I'm not sure if a plug-in event is available using the current legacy "usb direct" midi implementation, so this may only be possible to fix when adding support for the new Android Midi API per #3


fix inconsistent behavior with the 'istio_agent_cert_expiry_seconds' metric

Please provide a description of this PR:
Fixes: https://github.com/istio/istio/issues/50356
	// Cert expire time by default is createTime + sc.configOptions.SecretTTL.
	// Istiod respects SecretTTL that passed to it and use it decide TTL of cert it issued.
	// Some customer CA may override TTL param that's passed to it.

For ROOTCA, the expire time is meanlingless, we should just record this for WorkloadKeyCertResourceName cert
Please check any characteristics that apply to this pull request.

[] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

LGTM, can you add a ut

I'm not sure if it possible, but will try.

In response to a cherrypick label: #52005 failed to apply on top of branch "release-1.21":
Applying: fix inconsistent behavior with the 'istio_agent_cert_expiry_seconds' metric
Applying: release-notes
Applying: add ut
Using index info to reconstruct a base tree...
M	security/pkg/nodeagent/cache/secretcache_test.go
Falling back to patching base and 3-way merge...
Auto-merging security/pkg/nodeagent/cache/secretcache_test.go
CONFLICT (content): Merge conflict in security/pkg/nodeagent/cache/secretcache_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0003 add ut
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #52022

In response to a cherrypick label: #52005 failed to apply on top of branch "release-1.21":
Applying: fix inconsistent behavior with the 'istio_agent_cert_expiry_seconds' metric
Applying: release-notes
Applying: add ut
Using index info to reconstruct a base tree...
M	security/pkg/nodeagent/cache/secretcache_test.go
Falling back to patching base and 3-way merge...
Auto-merging security/pkg/nodeagent/cache/secretcache_test.go
CONFLICT (content): Merge conflict in security/pkg/nodeagent/cache/secretcache_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0003 add ut
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #52023

In response to a cherrypick label: #52005 failed to apply on top of branch "release-1.22":
Applying: fix inconsistent behavior with the 'istio_agent_cert_expiry_seconds' metric
Applying: release-notes
Applying: add ut
Using index info to reconstruct a base tree...
M	security/pkg/nodeagent/cache/secretcache_test.go
Falling back to patching base and 3-way merge...
Auto-merging security/pkg/nodeagent/cache/secretcache_test.go
CONFLICT (content): Merge conflict in security/pkg/nodeagent/cache/secretcache_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0003 add ut
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #52024

In response to a cherrypick label: #52005 failed to apply on top of branch "release-1.22":
Applying: fix inconsistent behavior with the 'istio_agent_cert_expiry_seconds' metric
Applying: release-notes
Applying: add ut
Using index info to reconstruct a base tree...
M	security/pkg/nodeagent/cache/secretcache_test.go
Falling back to patching base and 3-way merge...
Auto-merging security/pkg/nodeagent/cache/secretcache_test.go
CONFLICT (content): Merge conflict in security/pkg/nodeagent/cache/secretcache_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0003 add ut
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".


Code fix for using statements

I made an (initial) implementation for a code fix that will reorder all using statements in the correct order and place them on the correct location.
Can you please review this and discuss if this is the way to go?

Updated to reflect the current state

I'm treating this as blocked prior to implementing the rest of the using diagnostics (#288, #289, #290).

:question: Can you update the unit tests for the diagnostics from DiagnosticVerifier to CodeFixVerifier?
:question: In the case of directive trivia surrounding using statements outside of a namespace how will these be handled?
Moving the using statement into the namespace in the following example will cause a compiler error. This is handled correctly by the diagnostic. Pointing it out as a heads up.
using System.Reflection;

[assembly: AssemblyVersion("<IP_ADDRESS>")]

namespace TestNamespace { }

@vweijsters I'd love to include this, but first I want to make sure it accounts for a couple of things to make sure we don't break users' code.

When SA1200 is suppressed, the code fix should not attempt to move the using declarations inside a namespace.

When the file contains items declared outside a namespace (such as @Noryoko's example), or when the file contains more than one namespace, the code fix should not attempt to move using declarations inside a namespace.

The code fix should account for #1161 when reordering directives. In addition, the code fix should not attempt to move using directives inside a namespace if the group contained unbalanced preprocessor directives.

I've been busy the last couple of days. This one is first on my list for the coming week

Thanks for the update. :smile: I marked it as in progress with a tentative milestone of Beta 10.

Updated with all remarks above. Also rebased/squashed.

Can you add a test that includes a file header?

I'll send a follow-up pull request to address the final feedback.


VertexPath.CalculateClosestPointOnPathData optimization

This PR contains an optimized version of VertexPath.CalculateClosestPointOnPathData, which positively affects the performance of:

VertexPath.GetClosestPointOnPath
VertexPath.GetClosestTimeOnPath
VertexPath.GetClosestDistanceAlongPath

The idea of the optimization is to perform all the math and specifically "ClosestPointOnLineSegment" in local-space, because then there is no need to transform every point in the path to world-space.
This works by transforming the incoming "worldPoint" from world-space to "VertexPath local space" only once. The result CalculateClosestPointOnPathData result is then transformed from local-  to world-space again.
On my machine, profiling before and after the optimization shows:

Old implementation: 5.58ms - 14.62ms
New implementation: 0.25ms - 0.40ms

I've used the scene "GetClosestPointOnPath Test" to test and profile the new implementation against the old one.

Great optimization, thanks!


Added missing \n after app_port definition in backend/dalle_playground_backend.ipynb

Added missing \n after app_port definition in backend/dalle_playground_backend.ipynb

Dupe of https://github.com/saharmor/dalle-playground/pull/64 with slightly different patch, feel free to close.

Thanks for surfacing this! Got fixed as part of https://github.com/saharmor/dalle-playground/commit/83c27cef066f4130fe98dac8d1c41aadbd372cb5


pack it bruh

any events with a nil in it will not return anything past the nil if you don't

I cannot believe that in the entire time this thing has existed that hasn't been tested.


Powershell Core Install-Module AzureRM.NetCore silently dies

OS:  Windows Subsystem for Linux, Debian 9
Powershell Core:  6.0.4
PowershellGet:  1.6.0
Issue:  When running Install-Module AzureRM.NetCore, the status of the module installation begins normally.  Immediately after Azure.Storage.Netcore is downloaded, the installation silently dies.

Please file this issue with the engineering team:
https://github.com/azure/azure-powershell/issues


HTTP 400 "Missing Required Header for CSRF protection."

 -v -H 'Content-Type: application/json' -X POST -d '{<EMAIL_ADDRESS>"className":"com.microsoft.spark.test.SimpleFile" }' "https://mysparkcluster.azurehdinsight.net/livy/batches"

...on HDInsight 3.6, produces this error:

HTTP ERROR: 400

Problem accessing /batches. Reason:

    Missing Required Header for CSRF protection.

Powered by Jetty://

The solution is to add an extra header: -H "X-Requested-By: admin"

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

ID: 4472f8f1-7879-22cc-9c8d-5ccca7f0358a
Version Independent ID: 19443dd3-1562-c748-2cc4-7e19b3aa01f4
Content: Use Livy Spark to submit jobs to Spark cluster on Azure HDInsight
Content Source: articles/hdinsight/spark/apache-spark-livy-rest-interface.md
Service: hdinsight
GitHub Login: @nitinme
Microsoft Alias: nitinme

@ishepherd Thanks for the feedback. We are actively investigating and will get back to you soon.

@ishepherd thanks for the catch and the suggested fix.

@ishepherd Thanks for figuring out this quirk.
I am going to add the extra header switch to the POST examples in this article.
MicrosoftDocs/azure-docs-pr#46995
Thanks, Jason
Supporting information:
Without the extra header, the POST requests may fail due to the tightened security in Ambari.
The security can be relaxed if needed to avoid code change. You can disable the CSRF for Livy in the Ambari config, which may open security a bit, so that may be undesirable.
https://community.hortonworks.com/articles/35145/how-to-resolve-csrf-protection-error-while-adding.html

vi /etc/ambari-server/conf/ambari.properties
api.csrfPrevention.enabled=false
#ambari-server restart

Related to:
https://github.com/jupyter-incubator/sparkmagic/issues/285
https://stackoverflow.com/questions/39456705/ambari-api-post-complaining-csrf-protection
https://community.hortonworks.com/articles/35145/how-to-resolve-csrf-protection-error-while-adding.html
Ambari Doc improvement was made in 09/25/2015 according to this doc bug, but I can't find the official ambari reference for that setting. https://issues.apache.org/jira/browse/AMBARI-4337

#please-close


SW-4514 Remove EnumTable secondary constructors

In preparation for adding a flag to generated enums to control whether or not
their display names should be localizable, update the enum configuration to
use the primary constructor everywhere. The secondary constructors existed solely
to avoid having to use listOf() for single-use enums, but the minor cleanliness
gain will mostly go away once the call sites need a localization-related
parameter.

Current dependencies on/for this PR:

main

PR #1529

PR #1530   👈

PR #1531

This stack of pull requests is managed by Graphite.


C API: some functions are unimplemented but not documented

The following functions in C API are actually not implemented in rust, but not marked as unimplemented in the documentation. This may cause linker errors when downstream bindings try to consume the headers and emit foreign function declarations.
wasm_frame_copy
wasm_func_as_extern_const
wasm_global_as_extern_const
wasm_table_as_extern_const
wasm_memory_as_extern_const

Thanks for the report! I'll look to update this documentation in https://github.com/bytecodealliance/wasmtime/pull/2897 as I touch a whole bunch of other docs too.

I think this was fixed in #2897


Added Interactive Effect option to start progress of effect when key is released

This pull request proposes the following changes:
Implements #623
Hope everything is okay with the code style.

Looks good! Thanks for the contribution!


test fix for language-questionanswering

API change check
API changes are not detected in this pull request.


Drop vector indexes before dropping table/database

Fixes a bug where we were dropping vector indexes when the connection pool is disposed, which lead to missing vector indexes.

/gcbrun


Remove @types/react-hot-loader from TypeScript template

Description
This PR simply removes the no-longer-needed dependency on @types/react-hot-loader.
I wasn't sure if an update to the CHANGELOG was warranted. If it is, let me know and I'll be happy to add that.
UPDATE: Added summary to CHANGELOG.md
Changes/Tasks

[x] Remove @types/react-hot-loader from TypeScript template

Motivation and Context

react-hot-loader bundles its own types since v4.0.0

When running $ npx react-static create with the TypeScript template (before this change), npm would give the following warning:

npm WARN deprecated<EMAIL_ADDRESS>This is a stub types definition. react-hot-loader provides its own type definitions, so you do not need this installed.

Types of changes

[ ] Refactoring/add tests (refactoring or adding test which isn't a fix or add a feature)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] I have updated the documentation accordingly
[x] I have updated the CHANGELOG with a summary of my changes
[ ] My changes have tests around them

Any change warrants a CHANGELOG entry. I'll merge when you add!

@SleeplessByte Cool, that makes sense. I've updated the CHANGELOG. Thanks!

Thank you for your contribution @noahbrenner. It was just released.


Get to India

Hey everyone. A friend of mine is getting married in India soon and I'm trying to raise funds to get there as I abandoned money earlier this year to experiment with alternative sustainable lifestyles.
However, being moneyless means I can't attend the wedding. In turn, I've put together this crowd funding campaign: https://life.indiegogo.com/fundraisers/get-ben-helen-to-india-for-wedding-workshops/x/1800303
If the funds are raised, I'll do a week of workshops/talks/trainings in India on Node.js Programming, Open-Source Living, and whatever else you'd like to learn from myself and my wife.
Any help will be deeply appreciated. I'll also do a personalised thank you to everyone who donates.
So please tweet all about it, and share it, and donate to it. The sooner the money gets raised, the sooner I can prepare for the trip. Thank you.

FAiled.


Added very barebones VR support

This PR should give us rudimentary VR support. It is the bare minimum to avoid getting sick when using a VR headset, but some camera types should be avoided, such as Follow or Watch.
I also included a small typo fix. As far as I know, that's the only place that function shows up in the whole repository and it only exists in that small scope.

I also want to add that I've changed how the internal state tracking of the camera scripts work. If anyone sees a problem with this, let me know as soon as possible.


Git file transport does not work inside the container image

We can't point to a git repository using the file protocol when the application is run as a container.
failed to clone from file:///work/git-repo to /work/work-dir: exec: \"git\": executable file not found in $PATH"

Obviously, the Distroless base image does not have git installed and neither do we want to ship git with the application. It's odd that go-git uses the binary in the first place. Maybe we are using it wrong.
https://github.com/go-git/go-git/blob/b5b59f5fc9f9c0cd3ad1329c814b6a0d8126e675/plumbing/transport/file/client.go

It was easy to write a failing test by resetting PATH with https://pkg.go.dev/testing#T.Setenv.
I tried to fix the test and discovered the use case is not supported by go-git.
From https://github.com/go-git/go-git/blob/master/COMPATIBILITY.md

Feature
Status
Notes

file://
partial
Warning: this is not pure Golang. This shells out to the git binary.

I suggest closing the issue.

Thanks for looking into it. We can just add a note to the documentation to say that the file transport is not supported by the container.


Add section about installing plugins feature

This is available when using sb-mig tool, with help, command. We don't need to document it itself in repository.


Version Number for got-statsd-client

Hi,
I noticed that your client does not have any version number or official release number for it. Would it be possible for you guys to tag the code with a version number or create a zip file with version number?
Thanks,
--Tejas

Sure. I can tag a version.

tag added. https://github.com/cactus/go-statsd-client/releases/tag/v1.0.0


working on the timestamps of videos-list/27-cameron.md

I am working on the timestamps for video 27 Cameron Davidson, Microbial cell counting
#11

@bsenst This issue did not seem to close automatically.
For next time, it may be necessary to write text exactly as I done previously:
Closes #17 rather than what was done with this one, which was closes issue #17
I am closing this issue #17 via pull request #18


Add support for climber hardware

This PR:

Adds controls for climbing
Adds climb statemachine and subsystem

@catarinaburghi this has not actually been tested on the bot yet

Sarah has hooked up the hardware. I think we can test this

We need to rename the positions

Replaced in favour of #146


Package.load fails with InvalidLineError

OS: MacOS Mojave 10.14.4 (18E226)
Python Version: 3.7.3
t4 Version: 0.0.10
Used normal pkg.build(), build manifest was created and seemingly looks good, tried to load manifest and received error InvalidLineError. Traceback:
---------------------------------------------------------------------------
JSONDecodeError                           Traceback (most recent call last)
/usr/local/anaconda3/envs/aicsdatapackaging/lib/python3.7/site-packages/jsonlines/jsonlines.py in read(self, type, allow_none, skip_empty)
    158         try:
--> 159             value = self._loads(line)
    160         except ValueError as orig_exc:

/usr/local/anaconda3/envs/aicsdatapackaging/lib/python3.7/json/__init__.py in loads(s, encoding, cls, object_hook, parse_float, parse_int, parse_constant, object_pairs_hook, **kw)
    347             parse_constant is None and object_pairs_hook is None and not kw):
--> 348         return _default_decoder.decode(s)
    349     if cls is None:

/usr/local/anaconda3/envs/aicsdatapackaging/lib/python3.7/json/decoder.py in decode(self, s, _w)
    336         """
--> 337         obj, end = self.raw_decode(s, idx=_w(s, 0).end())
    338         end = _w(s, end).end()

/usr/local/anaconda3/envs/aicsdatapackaging/lib/python3.7/json/decoder.py in raw_decode(self, s, idx)
    354         except StopIteration as err:
--> 355             raise JSONDecodeError("Expecting value", s, err.value) from None
    356         return obj, end

JSONDecodeError: Expecting value: line 1 column 1 (char 0)

The above exception was the direct cause of the following exception:

InvalidLineError                          Traceback (most recent call last)
<ipython-input-4-0c27a554d04e> in <module>
      1 import t4
----> 2 pkg = t4.Package.load("/allen/aics/modeling/t4/manifests/aics/pipeline_integrated_cell/2019-04-16T18:33:28.json")
      3 pkg

/usr/local/anaconda3/envs/aicsdatapackaging/lib/python3.7/site-packages/t4/packages.py in load(cls, readable_file)
    512         """
    513         reader = jsonlines.Reader(readable_file)
--> 514         meta = reader.read()
    515         meta.pop('top_hash', None)  # Obsolete as of PR #130
    516         pkg = cls()

/usr/local/anaconda3/envs/aicsdatapackaging/lib/python3.7/site-packages/jsonlines/jsonlines.py in read(self, type, allow_none, skip_empty)
    162                 "line contains invalid json: {}".format(orig_exc),
    163                 line, lineno)
--> 164             six.raise_from(exc, orig_exc)
    165
    166         if value is None:

/usr/local/anaconda3/envs/aicsdatapackaging/lib/python3.7/site-packages/six.py in raise_from(value, from_value)

InvalidLineError: line contains invalid json: Expecting value: line 1 column 1 (char 0) (line 1)

Maybe relevant: The path provided to load is a symlink to the actual manifest.
/allen/aics/modeling/t4/manifests/aics/pipeline_integrated_cell/2019-04-16T18:33:28.json -> /allen/aics/modeling/t4/manifests/aics/pipeline_integrated_cell/.quilt/packages/3ac72ee165e825341cba7803e474faaa0fb51c0a33f5cfd705dd0c153413483e

More context:
pkg.build() was ran with options:
hash = pkg.build(name="{}/{}".format(self.package_owner, self.package_name), registry=self.build_path)
self.package_owner = aics
self.package_name = pipeline_integrated_cell
self.build_path = Path(f"/allen/aics/modeling/t4/manifests/{self.PACKAGE_OWNER}/{self.PACKAGE_NAME}")

Tested Package.load() on non-symlink path, same error.

One quick comment would be to try using Package.browse("{self.PACKAGE_OWNER}/{self.PACKAGE_NAME}") instead. Package.load() is a low-level API.

I was able to reproduce this error locally.
When you open a package by name via t4.Package.browse, it eventually executes the following method:
    @classmethod
    def _from_path(cls, uri):
        """ Takes a URI and returns a package loaded from that URI """
        src_url = urlparse(uri)
        if src_url.scheme == 'file':
            with open(parse_file_url(src_url)) as open_file:
                pkg = cls.load(open_file)
        elif src_url.scheme == 's3':
            body, _ = get_bytes(uri)
            pkg = cls.load(io.BytesIO(body))
        else:
            raise NotImplementedError
        return pkg

Which calls on load to open the manifest. load constructs a jsonlines.Reader from the file handle and uses that to do a line-by-line read. Either the docs are wrong, or the code changed without me knowing. I will update the docs accordingly.

Nice catch! I would update the docs to use the context manager instead of just open.
import t4
with open("/path/to/manifest.json", "r") as manifest_obj:
    pkg = t4.Package.load(manifest_obj)

pkg

Feel free to close issue!


v0.22.0 breaks webrtcICEUDPMuxAddress

Which version are you using?
v0.22.0
Which operating system are you using?

[ ] Linux amd64 standard
[x] Linux amd64 Docker
[ ] Linux arm64 standard
[x] Linux arm64 Docker
[ ] Linux arm7 standard
[ ] Linux arm7 Docker
[ ] Linux arm6 standard
[ ] Linux arm6 Docker
[ ] Windows amd64 standard
[ ] Windows amd64 Docker (WSL backend)
[ ] macOS amd64 standard
[ ] macOS amd64 Docker
[ ] Other (please describe)

Describe the issue
webrtcICEUDPMuxAddress was working previously, but seems to be broken after upgrading to v0.22.0.
Switching to webrtcICETCPPMuxAddress with -p 8189:8189 still works as expected.
Describe how to replicate the issue

docker run -p 8889:8889 -p 8189:8189/udp \
-e MTX_PATHS_TEST_SOURCE=rtsp://test-server:8554/stream \
-e MTX_WEBRTCICEHOSTNAT1TO1IPS=<IP_ADDRESS> \
-e MTX_WEBRTCICEUDPMUXADDRESS=:8189 \
aler9/rtsp-simple-server

Did you attach the server logs?

 DEB [WebRTC] [conn <IP_ADDRESS>:44760] peer connection state: closed
 INF [WebRTC] [conn <IP_ADDRESS>:44760] closed (deadline exceeded)
 DEB [WebRTC] [conn <IP_ADDRESS>] [s->c] HTTP/1.1 404 Not Found
Access-Control-Allow-Credentials: true
Access-Control-Allow-Origin: *
Server: mediamtx

Did you attach a network dump?

no
EDIT:
Works as expected when rolling back down to v0.21.6:
docker run -p 8889:8889 -p 8189:8189/udp \
-e RTSP_PATHS_TEST_SOURCE=rtsp://test-server:8554/stream \
-e RTSP_WEBRTCICEHOSTNAT1TO1IPS=<IP_ADDRESS> \
-e RTSP_WEBRTCICEUDPMUXADDRESS=:8189 \
aler9/rtsp-simple-server:v0.21.6

Hello, i've investigated the issue and it turned out that it's a regression in pion/ice.
I've sent a PR with a fix here: https://github.com/pion/ice/pull/550

added in MediaMTX v0.22.1


Packages required for selinux_policy_module

On RHEL/CentOS, package 'selinux-policy-devel' is required to add new policy modules.

Hi.
Can I get a little more detail, like specific OS, error?
Thanks
Nitzan Raz
http://backslasher.net
On Sat, Jun 11, 2016 at 12:16 AM, Jason Martin<EMAIL_ADDRESS>wrote:

On RHEL/CentOS, package 'selinux-policy-devel' is required to add new
policy modules.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/BackSlasher/chef-selinuxpolicy/issues/46, or mute
the thread
https://github.com/notifications/unsubscribe/AG3He2e4q9camMz9IQAmLzSoWFvByU7wks5qKdQzgaJpZM4IzVmF
.

https://forums.fogproject.org/topic/6154/selinux-policy demonstrates the issue:
make: /usr/share/selinux/devel/Makefile: No such file or directory
make: *** No rule to make target `/usr/share/selinux/devel/Makefile'.  Stop.

I think I have a test for that in the kitchen (runnable via kitchen test).
I'm not in position for testing - could you please check it?
If not I'll do it some other time

The test passes as your 7.2 box comes with that packages preinstalled, but it is not part of the vanilla installation.

https://github.com/BackSlasher/chef-selinuxpolicy/blob/f52ee57/recipes/install.rb#L22
Did you include selinux_policy::install in your run?

I am apparently blind.

Thanks for using and reporting, GL etc
If we need to stress it more in the README I'll accept PRs! :)
Nitzan Raz
http://backslasher.net
On Sat, Jun 11, 2016 at 5:09 PM, Jason Martin<EMAIL_ADDRESS>wrote:

Closed #46 https://github.com/BackSlasher/chef-selinuxpolicy/issues/46.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/BackSlasher/chef-selinuxpolicy/issues/46#event-689416828,
or mute the thread
https://github.com/notifications/unsubscribe/AG3HewOWjJ9QH1TdVzrkPM9aHv5YTlkMks5qKsGlgaJpZM4IzVmF
.


Train from scratch on my GPUs.

Thanks for sharing your awesome work.
Your works have been a good baseline to me :)
I have some questions about training from scratch to reproduce your results.
Unfortunately, I possess 4 RTX 4090 GPUs, so I adjusted GRAD_ACCUM_STEPS to set ALL_BATCH_SIZE=4096.
However, I couldn't get the same results as you. My results are more degraded than yours (e.g., kobe size model).
So, following your previous work PoolFormer, I set ALL_BATCH_SIZE=1024, --lr 1e-3, --warmup-epochs 5, and keep DROP_PATH=0.025.
Q1) If I tune ALL_BATCH_SIZE, do I also need to change the value of DROP_PATH?
Q2) I wonder if there is a rule to set DROP_PATH depending on model sizes.
Q3) I ask your advice if there are something tips to reproduce your results with a small number of GPUs .

Hi @DoranLyong ,
Thank you so much for your recognition of our work.
A1) In general, no need.
A2) I don't have a clear and specific rule for DROP_PATH. My simple rule is the DROP_PATH of the larger model should be larger or equal to that of the smaller model.
A3) Maybe try EMA, although I am not sure whether it works.
Besides, could you please share the results of the MambaOut-Kobe you trained? I may try to reproduce it on you similar setting to help find the solution.
Thank you so much once again.

Thanks for your concern :)
$\text{Batch Size per GPU} = \frac{\text{Total Batch Size}}{\text{Number of GPUs} \times \text{Gradient Accumulation Steps}}=\frac{4096}{8 \times 4}=128$
This is your setting. So, I try to meet the same batch size per GPU and keep the learning rate like below.
1st try
DATA_PATH=/workspace/dataset/ImageNet2012
CODE_PATH=/workspace/projects/MambaOut # modify code path here

ALL_BATCH_SIZE=4096
NUM_GPU=4
GRAD_ACCUM_STEPS=8 # Adjust according to your GPU numbers and memory size.
let BATCH_SIZE=ALL_BATCH_SIZE/NUM_GPU/GRAD_ACCUM_STEPS

MODEL=mambaout_kobe
DROP_PATH=0.025

cd $CODE_PATH && sh distributed_train.sh $NUM_GPU $DATA_PATH \
--model $MODEL --opt adamw --lr 4e-3 --warmup-epochs 20 \
-b $BATCH_SIZE --grad-accum-steps $GRAD_ACCUM_STEPS \
--drop-path $DROP_PATH

Next, I tried to change the learning rate following below rule.
$\text{New Learning Rate} = \text{Old Learning Rate} \times \frac{\text{New Batch Size per GPU}}{\text{Old Batch Size  per GPU}} =  4 \times 10^{-3} \times \frac{64}{128} = 2 \times 10^{-3}$
So, I increase the GRAD_ACCUM_STEPS in double and decrease the learning rate as 2e-3
2nd try
DATA_PATH=/workspace/dataset/ImageNet2012
CODE_PATH=/workspace/projects/MambaOut # modify code path here

ALL_BATCH_SIZE=4096
NUM_GPU=4
GRAD_ACCUM_STEPS=16 # Adjust according to your GPU numbers and memory size.
let BATCH_SIZE=ALL_BATCH_SIZE/NUM_GPU/GRAD_ACCUM_STEPS

MODEL=mambaout_kobe
DROP_PATH=0.025

cd $CODE_PATH && sh distributed_train.sh $NUM_GPU $DATA_PATH \
--model $MODEL --opt adamw --lr 2e-3 --warmup-epochs 20 \
-b $BATCH_SIZE --grad-accum-steps $GRAD_ACCUM_STEPS \
--drop-path $DROP_PATH

I stopped to train at Epoch 249 and the top accuracy rate is 75.5.
In my experience with your MetaFormer baseline training from scratch, the accuracy should be larger than it at this epoch to reach 80.0 accuracy within Epoch 300.

summary.csv
I'd like to share my training log. I have no idea why it saturated early :s

Hi @DoranLyong , I reproduced MambaOut-Kobe on 8 GPUs of A5000 and it can achieve 80.1. Logs see
mambaout_kobe_bs4096_gpu8_ga2_lr4e-3_dp0025.csv.
My environment is
Hardware: 8 GPUs of A5000
PyTorch: 1.11.0
CUDNN: 8
timm: 0.6.11
My training command is
DATA_PATH=/local_home/dataset/imagenet
CODE_PATH=/home/yuweihao/code/MambaOut # modify code path here

ALL_BATCH_SIZE=4096
NUM_GPU=8
GRAD_ACCUM_STEPS=2 # Adjust according to your GPU numbers and memory size.
let BATCH_SIZE=ALL_BATCH_SIZE/NUM_GPU/GRAD_ACCUM_STEPS

MODEL=mambaout_kobe
DROP_PATH=0.025

cd $CODE_PATH && sh distributed_train.sh $NUM_GPU $DATA_PATH \
--model $MODEL --opt adamw --lr 4e-3 --warmup-epochs 20 \
-b $BATCH_SIZE --grad-accum-steps $GRAD_ACCUM_STEPS \
--drop-path $DROP_PATH --native-amp

Therefore, you may try:

Set up environment with package versions the same as me.
Set up batch size as 4096 (I did not try other batch sizes on GPUs, so I am not sure whether other batch sizes work). You can also use --native-amp and reduce GRAD_ACCUM_STEPS to accelerate training.

okay I solved my issues :)

It was the version issues of pytorch and timm.
after matching my env to your env, I could get the same results like you.
For pytorch and timm, it seems that there are some minor different between the latest version and the previous version.


[feature] Add support for Openstack API

Description

OpenStack is a free, open standard cloud computing platform. It is mostly deployed as infrastructure-as-a-service (IaaS) in both public and private clouds where virtual servers and other resources are made available to users.

Source : Wikipedia
Ability to disply server names and ip adresses in specific or all tenants.
Describe the solution you'd like
Add Openstack as a new cloud provider to cloudlist. gophercloud seems to be an interesting library to implement Openstack API interactions.
I may be interested developing this feature myself if main contributors are open to the idea of adding this provider.

I may be interested developing this feature myself if main contributors are open to the idea of adding this provider.

@S0obi please feel free to open a PR for this.


[UI] Enhance news card to use image and color pallete to add variations.

Add text color and background image for visual variations.

Depends on #34

Moved to icebox, because adding color is not needed at the moment.


Streamlined UI

The UI could be cleaner. Credentials via chrome style options page, simplify the main UI, respond to what information the API returns.
Jumping off point: https://docs.google.com/presentation/d/1mJv7rMnQUXzvJzDmAIo3JDZJFi9e5eg8eoYwFo2SzEE/edit#slide=id.g100df090a1_0_36

Covered in the new version coming soon


Implement webhooks management

The latest version of appcat-postgresql uses validation webhooks which require certificates to work. Currently the chart is being installed with predefined certificates (step 4). This step has to be managed by the component.
Implementation ideas:

Component crafts a secret which references certificates from Vault. ArgoCD will populate that sensitive information from Vault to valid secret data during deployment. This new secret webhook.existingSecret needs to be available in the Helm Chart.
Add a pre-installation job for Argo to create the secret with self-signed certificates. We will still need webhook.existingSecret for that matter.

Acceptance Criteria

The secret should be created in the same namespace as the operator

Notes
we need:

argocd pre-hook
bash code to create certificates
a job definition to trigger the certificate creation
use hardcoded name for the secret

There are much easier ways for Openshift: https://docs.openshift.com/container-platform/4.10/security/certificates/service-serving-certificate.html (also supports patching Webhook configs)
And since our first target platform is Openshift anyway, that's good enough for now.
My vote is for going that route with service certificates, no need to script something in an ArgoCD job when not yet needed.

I agree, but let's keep this open for when we want to support plain k8s.

To support plain K8s in the future, we may consider https://github.com/projectsyn/k8s-service-ca-controller which is hopefully released by then.


getFeatureRemainings() returns error if queried against a feature with value of true/false

For example $subscription->getFeatureRemainings('analytics') will error with "Unsupported operand types: string - int". I think in this case we could silently return null (helpful when we are getting user's features details "in bulk")?

Fixed in #94 , will be in next patch version for v4 and merged also to v5 development.


diy mode sonoff mini firmware 3.5

Hi,
I've upgraded to the latest firmware using ewelink and activated diy mode after that.
I've successfully been able to go the web interface and entering my home wifi SSID and password.
after led was fast double blinking.
I don't see any DHCP requests on my router (Mikrotik).
Just to sure I did not mistype anything I'm trying the following which does not work.
when I try to put into pairing mode and into compatible pairing mode. I'm able to reconnect to the ap, but the device is not showing me the webpage again.
any suggestions from your side?

Check to see if you have mDNS on your system. That's what eWelink uses to find you mini. I don't have DNS, and therefore I have about 10 Mini's that are bricked.


Seperate CommonServiceLocator from SolrNet

This change is backward compatible for our SolrNet package endusers.
CI build is passing

Create CommonServiceLocator.SolrNet project and a test project
Moved all relevant code over from SolrNet and SolrNet tests to these new projects.
Rename SolrNet nuget package to SolrNet.Core
Create SolrNet package that includes CommonServiceLocator.SolrNet in order to prevent breaking changes.

#302

Thanks @gjunge


HDFS-16086. Add volume information to datanode log for tracing

JIRA: HDFS-16086
To keep track of the block in volume, we can add the volume information to the datanode log.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 47s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
32m  2s

trunk passed

+1 :green_heart:
compile
1m 24s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
1m 15s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
checkstyle
1m  2s

trunk passed

+1 :green_heart:
mvnsite
1m 22s

trunk passed

+1 :green_heart:
javadoc
0m 55s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 25s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
3m 16s

trunk passed

+1 :green_heart:
shadedclient
18m 30s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 16s

the patch passed

+1 :green_heart:
compile
1m 21s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

-1 :x:
javac
1m 21s
/results-compile-javac-hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04.txt
hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 generated 36 new + 467 unchanged - 36 fixed = 503 total (was 503)

+1 :green_heart:
compile
1m 12s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
javac
1m 12s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 55s
/results-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs-project/hadoop-hdfs: The patch generated 2 new + 406 unchanged - 0 fixed = 408 total (was 406)

+1 :green_heart:
mvnsite
1m 16s

the patch passed

+1 :green_heart:
javadoc
0m 48s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 19s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
3m 23s

the patch passed

+1 :green_heart:
shadedclient
18m 52s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
343m 48s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 48s

The patch does not generate ASF License warnings.

434m 20s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.namenode.TestDecommissioningStatusWithBackoffMonitor

hadoop.hdfs.server.namenode.ha.TestEditLogTailer

hadoop.hdfs.server.namenode.ha.TestBootstrapStandby

hadoop.hdfs.server.namenode.TestDecommissioningStatus

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3136/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3136

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux e9b7aacb928e 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / d459a94ce489b6700d470488466c28b6d49d5873

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3136/1/testReport/

Max. process+thread count
2220 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3136/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 48s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
31m 41s

trunk passed

+1 :green_heart:
compile
1m 24s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
1m 14s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
checkstyle
1m  1s

trunk passed

+1 :green_heart:
mvnsite
1m 23s

trunk passed

+1 :green_heart:
javadoc
0m 55s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 24s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
3m 13s

trunk passed

+1 :green_heart:
shadedclient
18m 53s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 17s

the patch passed

+1 :green_heart:
compile
1m 19s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

-1 :x:
javac
1m 19s
/results-compile-javac-hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04.txt
hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 generated 36 new + 467 unchanged - 36 fixed = 503 total (was 503)

+1 :green_heart:
compile
1m 11s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
javac
1m 11s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 55s

hadoop-hdfs-project/hadoop-hdfs: The patch generated 0 new + 398 unchanged - 8 fixed = 398 total (was 406)

+1 :green_heart:
mvnsite
1m 15s

the patch passed

+1 :green_heart:
javadoc
0m 49s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 18s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
3m 19s

the patch passed

+1 :green_heart:
shadedclient
18m 47s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
341m 37s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 36s

The patch does not generate ASF License warnings.

431m 48s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.namenode.TestDecommissioningStatusWithBackoffMonitor

hadoop.hdfs.server.namenode.ha.TestEditLogTailer

hadoop.hdfs.server.namenode.ha.TestBootstrapStandby

hadoop.hdfs.TestDFSShell

hadoop.hdfs.server.datanode.fsdataset.impl.TestFsVolumeList

hadoop.hdfs.server.namenode.TestDecommissioningStatus

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3136/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3136

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux c8292eb298de 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / cbac4f095fa1e7de6cbdf836095909d828ab0699

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3136/2/testReport/

Max. process+thread count
2001 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3136/2/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

These UTs works fine locally and are unrelated to the change.
Hi @tasanuma @aajisaka  @jojochuang @Hexiaoqiao , could you please review the code when you have time. Thanks.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 48s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
33m 31s

trunk passed

+1 :green_heart:
compile
1m 25s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
1m 16s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
checkstyle
1m  2s

trunk passed

+1 :green_heart:
mvnsite
1m 25s

trunk passed

+1 :green_heart:
javadoc
0m 55s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 26s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
3m 15s

trunk passed

+1 :green_heart:
shadedclient
18m 58s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 16s

the patch passed

+1 :green_heart:
compile
1m 18s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

-1 :x:
javac
1m 18s
/results-compile-javac-hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04.txt
hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 generated 36 new + 467 unchanged - 36 fixed = 503 total (was 503)

+1 :green_heart:
compile
1m  9s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
javac
1m  9s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 54s
/results-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs-project/hadoop-hdfs: The patch generated 1 new + 406 unchanged - 0 fixed = 407 total (was 406)

+1 :green_heart:
mvnsite
1m 16s

the patch passed

+1 :green_heart:
javadoc
0m 49s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 19s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
3m 21s

the patch passed

+1 :green_heart:
shadedclient
18m 51s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
347m 56s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 38s

The patch does not generate ASF License warnings.

440m 13s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.namenode.TestDecommissioningStatusWithBackoffMonitor

hadoop.hdfs.server.namenode.ha.TestEditLogTailer

hadoop.hdfs.server.namenode.ha.TestBootstrapStandby

hadoop.hdfs.TestDFSShell

hadoop.hdfs.server.datanode.fsdataset.impl.TestFsVolumeList

hadoop.hdfs.server.datanode.fsdataset.impl.TestFsDatasetImpl

hadoop.hdfs.server.namenode.TestDecommissioningStatus

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3136/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3136

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 9d9da12a285a 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 0fc5f0fdc02bd5e8ae7df769d8eb8385de43c96e

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3136/3/testReport/

Max. process+thread count
2070 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3136/3/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

These failed UTs work fine locally.

Hi @jojochuang , could you please take a look again? Thanks.

Hi @ayushtkn , could you please take a quick look at this. Thanks.

The checkstyle warnings are old, unrelated. Mergeing the PR.

The checkstyle warnings are old, unrelated. Mergeing the PR.

Thanks @jojochuang for the merge.


Error in console when loading as player

Thanks for working on this module. Well crafted and styled.
When I load FoundryVTT with a player user I get the following error in the console.
Error message
Uncaught (in promise) SyntaxError: Unexpected end of JSON input
    at JSON.parse (<anonymous>)
    at loadWindowPositions (utils.js:15:25)
    at Module.restoreWindows (utils.js:26:2)
    at Function.onReady (Monarch.js:219:9)

  | loadWindowPositions | @ | utils.js:15
-- | -- | -- | --
  | restoreWindows | @ | utils.js:26
  | onReady | @ | Monarch.js:219
  | await in onReady (async) |   |
  | _call | @ | foundry.js:294
  | callAll | @ | foundry.js:253
  | setupGame | @ | foundry.js:5566
  | await in setupGame (async) |   |
  | _initializeGameView | @ | foundry.js:6498
  | _initializeView | @ | foundry.js:6474
  | initialize | @ | foundry.js:5430
  | await in initialize (async) |   |
  | 🎁call_wrapped | @ | libWrapper-wrapper.js:511
  | 🎁libWrapperInit | @ | libWrapper-api.js:793
  | await in 🎁libWrapperInit (async) |   |
  | 🎁Game.prototype.initialize#0 | @ | libWrapper-wrapper.js:187
  | window.addEventListener.once | @ | foundry.js:62436

Not sure if this connected to the players. Or if this could also happen as GM.
with the GM account however, I do not get this error.
further resources
https://stackoverflow.com/questions/30621802/why-does-json-parse-fail-with-the-empty-string
Potential solution
Old
https://github.com/zeel01/monarch/blob/04badbf35ac75f738cbb65142e3025c8134ade09/scripts/utils.js#L14
New
const data = window.localStorage.getItem("monarch-windows") || '{}';

Explanation of the change
When using the "Old" from above in
https://github.com/zeel01/monarch/blob/04badbf35ac75f738cbb65142e3025c8134ade09/scripts/utils.js#L15
The variable data is an empty string. Which JSON.parse can't parse.
Using either "{}" or maybe '""' (double quotes in single quotes) should yield the desired behavior.
This should also make the || {} in line 15 obsolete I think. As data will always be a parsable string.
This could be wrong if window.localStorage.getItem("monarch-windows") will return something else then valid json or null.

What's really confusing is why this error can happen in the first place. Have you tried reading out the storage object to see what's inside of it and why it doesn't appear to be valid JSON?
Your suggestion though is a good one, but does this error only happen if the value is empty? Or is it happening because some invalid data was stored that fails to parse but won't be falsy to the || operator?


set COOP, COEP, and CORP headers for desktop

This PR is for https://github.com/microsoft/vscode/issues/116715 and sets COOP, COEP and CORP headers so that the renderer and worker extension host are cross origin isolated. Tho, for webviews this doesn't work yet

Please review. The renderer and the web worker extension host will be cross origin isolated with this PR. Web views won't. I still needed to add headers to them to load in the stricter context (CORP, COEP). I have tested with our markdown preview, our images preview, luna paint, and a webview panel that loads resources from the internet (not from within the extension) but please review/test carefully.

Thanks @jrieken! I just tested a few webview extensions. Here's what I've found so far:

✅ Md preview seem to work (including references to external images)
✅ Git lens welcome views seem to work
❌ Live preview extension doesn't seem to work. I see [main 2022-04-06T21:38:38.049Z] CodeWindow: failed to load (reason: ERR_BLOCKED_BY_RESPONSE, code: -27)

I'm taking a look into the live preview issue now

The request that fails:http://<IP_ADDRESS>:3000/index.html. This is for the iframe that the live preview extension creates (which is inside the two iframes that we create for each webview). This request does not set a cross-origin-resource-policy, which causes it to be blocked by the browser
I think this document may be relevant: https://github.com/camillelamy/explainers/blob/main/anonymous_iframes.md


ARROW-16571: [Java] Update .gitignore to exclude JNI-related binaries

Adds three lines to gitignore to exclude three folders containing binaries and other build output produced by the JNI build process. The folders:

java-dist/
java-native-c/
java-native-cpp/

are created in the root arrow directory when cmake is run

Could you show your command lines to build JNI related binaries?

Could you show your command lines to build JNI related binaries?

I edited the PR to include a link to the instructions, and the actual steps I followed for macOS.


RenderStack object was given an infinite size during layout.

This occurs when trying to add a Position to a Text object in a stack. This is illustrated by stackNotWorking, I get the following exception when using it. For working example use stackWorking. Position is supposed to work in a stack, and indeed it does work for the image, but if you add a Position to the Text it causes the error.
Here is working code to reproduce the error.
import 'package:flutter/material.dart';
import 'test_list.dart';

// test_page.dart
class TestPage extends StatefulWidget {
  TestPage();

  @override
  TestPageState createState() {
    return TestPageState();
  }
}

class TestPageState extends State<TestPage> {
  void initState() {
    super.initState();
  }

  @override
  Widget build(BuildContext context) {
    var key = GlobalKey<ScaffoldState>();
    return Scaffold(
      key: key,
      body: Row(
          children: [
            Flexible(flex: 180, child: TestList()),
            Flexible(flex: 20, child: Text("test")),
          ])
    );
  }
}

// test_list.dart
// import 'package:flutter/material.dart';

class TestList extends StatelessWidget {
  final Map<int, String> aMap = { 1:'test1', 2:'test1', 3:'test1' };
  TestList();

  @override
  Widget build(BuildContext context) {
    return _buildList(context);
  }

  ListView _buildList(context) {
    return ListView(
        shrinkWrap: true,
        children: [
          for ( MapEntry entry in aMap.entries )
            stackNotWorking( entry.key ),
        ] );
  }

// This works
  Widget stackWorking( int index ) {
    return Stack(
        children: <Widget>[
          Positioned(
              left: 90,
              child: Icon(Icons.home, color: Colors.black, size: 20,) ),
          Text( 'a line of text' )
        ]
    );
  }

// This DOESN'T works
  Widget stackNotWorking( int index ) {
    return Stack(
        children: <Widget>[
        Positioned(
          left: 90,
          child: Icon(Icons.home, color: Colors.black, size: 20,) ),
        Positioned(
          left: 110,
          child: Text( 'a line of text' ) )
    ]
    );
  }
}

flutter doctor -v
[√] Flutter (Channel stable, v1.12.13+hotfix.5, on Microsoft Windows [Version 10.0.18362.592], locale en-US)

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.1)
[√] Android Studio (version 3.2)
[√] IntelliJ IDEA Community Edition (version 2019.3)
[!] IntelliJ IDEA Ultimate Edition (version 2016.3)
    X Flutter plugin not installed; this adds Flutter specific functionality.
    X Dart plugin not installed; this adds Dart specific functionality.
    X This install is older than the minimum recommended version of 2017.1.0.
[√] Connected device (1 available)

! Doctor found issues in 1 category.

logs
════════ Exception caught by rendering library
The following assertion was thrown during performLayout():
RenderStack object was given an infinite size during layout.

This probably means that it is a render object that tries to be as big as possible, but it was put inside another render object that allows its children to pick their own size.
The nearest ancestor providing an unbounded height constraint is: RenderIndexedSemantics#31bbb relayoutBoundary=up14 NEEDS-LAYOUT NEEDS-PAINT NEEDS-COMPOSITING-BITS-UPDATE
...  parentData: index=0; layoutOffset=0.0 (can use size)
...  constraints: BoxConstraints(w=370.3, 0.0<=h<=Infinity)
...  semantic boundary
...  size: MISSING
...  index: 0
The constraints that applied to the RenderStack were: BoxConstraints(w=370.3, 0.0<=h<=Infinity)
The exact size it was given was: Size(370.3, Infinity)

See https://flutter.dev/docs/development/ui/layout/box-constraints for more information.

The relevant error-causing widget was:
  Stack file:///E:/dev/purified/lib/test_list.dart:35:12
When the exception was thrown, this was the stack:
#0      RenderBox.debugAssertDoesMeetConstraints.<anonymous closure> (package:flutter/src/rendering/box.dart:1960:9)
#1      RenderBox.debugAssertDoesMeetConstraints (package:flutter/src/rendering/box.dart:2028:6)
#2      RenderBox.size=.<anonymous closure> (package:flutter/src/rendering/box.dart:1745:7)
#3      RenderBox.size= (package:flutter/src/rendering/box.dart:1747:6)
#4      RenderStack.performLayout (package:flutter/src/rendering/stack.dart:520:7)
...

The relevant error-causing widget was:
  ListView file:///E:/dev/purified/lib/test_list.dart:13:12
══════════════════════════════════════════════════════════════

There is a work around for this, but it remains a fundamental flaw IMHO.
The work around is to add a non positioned Text field at the end of the stack.
return Stack(         children: <Widget>[         Positioned(           left: 90,           child: Icon(Icons.home, color: Colors.black, size: 20,) ),         Positioned(           left: 110,           child: Text( 'a line of text' ) ),         Text( '' )      ]

The issue is because the stack has an unbounded height, yet has no non-positioned child to base its size on.
What you probably wanted to do instead is make the Text a non-positioned child and give it padding so that it sits 110pt from the left.

I got the same error,my flutter version is 1.17.3-stable

I use SingleChildScrollView,its child is Stack,Stack has a child with positioned,I got the same error,my flutter version is 1.17.3-stable.
if i add Boxconstraints to Stack or remove positioned,the error is gone.
Because the content's height is unsure,so i use SingleChildScrollView.If i use column,and did not add Boxconstraints to column,it works well.

Pretty much the same situation for me. In my case I use a CustomScrollView with a SliverToBoxAdapter that contains a Stack.

@robertflesch
Using https://github.com/flutter/flutter/issues/49631#issuecomment-617884531 implementation, the RenderStack issue isn't observed anymore:
// This works
  Widget stackWorking( int index ) {
    return Stack(
        children: <Widget>[
          Positioned(
              left: 90,
              child: Icon(Icons.home, color: Colors.black, size: 20,) ),
          Text( 'a line of text' )
        ]
    );
  }

// This DOESN'T works
  Widget stackNotWorking( int index ) {
    return Stack(
        children: <Widget>[
          Positioned(
              left: 90,
              child: Icon(Icons.home, color: Colors.black, size: 20,) ),
          Padding(
            padding: EdgeInsets.only(left: 110),
            child: Text( 'a line of text' ),
          )
        ]
    );
  }
}

Does this solution resolve your issue ?

@robertflesch
Using https://github.com/flutter/flutter/issues/49631#issuecomment-617884531 implementation, the RenderStack issue isn't observed anymore:
// This works
  Widget stackWorking( int index ) {
    return Stack(
        children: <Widget>[
          Positioned(
              left: 90,
              child: Icon(Icons.home, color: Colors.black, size: 20,) ),
          Text( 'a line of text' )
        ]
    );
  }

// This DOESN'T works
  Widget stackNotWorking( int index ) {
    return Stack(
        children: <Widget>[
          Positioned(
              left: 90,
              child: Icon(Icons.home, color: Colors.black, size: 20,) ),
          Padding(
            padding: EdgeInsets.only(left: 110),
            child: Text( 'a line of text' ),
          )
        ]
    );
  }
}

Does this solution resolve your issue ?

Indeed, thank you for addressing it.

Indeed, thank you for addressing it.


Use schema defined in uri for consul health check

Good catch!

Will you update the nuget packages?


Start cloud 6.x development

Remove Cloud 3.x related cloudsources, add susecloud6/Milestone
targets.

:+1:


CreationPage iteration21 make confirmation button green and Azure app update timed

Description
After discussion in our Live Online Prototype Demo today,
the feedback from our users indicated that another
iteration of the confirmation message on CreationPage,
was in order.
The ugly confirmation message,
has now been replaced with a prettier
green button.
Future iterations might improve this confirmation step even further.
Also, this is a test of the live update of our online Azure app. The
update will be timed after merge, so that stakeholders might be
later informed on how fast our pipeline is running, and when they
can expect to see the change.
Related Issue(s)

#90

Verification

[X] Your code builds clean without any errors or warnings
[X] Manual testing done (required)
[ ] Relevant automated test added (if you find this hard, leave it and we'll help out)
[ ] All tests run green

Experiment failed: after 21 minutes, the Azure web app is not updated with the new feature
(tested on Safari and Chrome on Mac, cache cleaned etc). Also on PlasticPC Edge, no update.


feat(bazel): map operation_metadata.proto to common_proto

Properly generate dependencies for google/cloud/common/operation_metadata.proto for the proto_library target and the Java & Go targets.

I've tested this in googleapis with the google/cloud/filestore API, the impetus for this change.


⚠ TorrentLeech has degraded performance

In 24d63c1, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 4688 ms

Resolved: TorrentLeech performance has improved in af2a8b6 after 7 minutes.


Fortinet-GetEntityDetails not authenticating against Fortigate

Describe the bug
Receive error "Unexpected character encountered while parsing value: <. Path '', line 0, position 0."  when running function app Fortinet-GetEntityDetails
To Reproduce
Steps to reproduce the behavior:

Assign Fortinet-FortiGate-ResponseOnIP playbook to analytics rule
Trigger rule so that logic app fires
Logic app will get to "check address object is already in firewall" and will hang.
Check function app monitor to confirm that API is calling Fortinet-GetEntityDetails and that error is generated

Expected behavior
API should run through so that logic app can complete.
Screenshots

Issue is due to the access_code not being appended to the request to the fortigate and reply has the following when unauthorised.

Hi,
Wanted to check in to see if there were any thoughts on this one?

@lior-tamir : This looks to be related to Playbooks, can you please take a look.. Thanks !!!

Hi,
Wanted to check in to see if there was anything we could look at on this one?

We are working on this one and will update once we have solution.

Hi @scottbrien01  this bug is addressed now. Shall we close this issue ?

We will close this issue, please reopen incase you still face any issues... Thanks!!!


Prototype improvements

Following features should be implemented:

Logs of user interactions such as answers to questions, started & ended materials etc.
Logs should be displayed

Since Laravel & PhP is not a requirement and the timeline is on the essence, we'll abandon this issue and focus on the tech-stack required for this app.


Doing a getAllSubscriptions call payload doesn't match data returned

I'm doing a call to get all the subscriptions along the lines of,
var subscriptionsApi = new ArtikCloud.SubscriptionsApi();
subscriptionsApi.getAllSubscriptions({}, callback);

But the payload returned does not match what the library code expects.
Payload returned
    aid: 'xx',
    messageType: 'message',
    uid: 'xx',
    sdtid: 'xx',
    description: 'Send messages from Artik to Kinesis',
    subscriptionType: 'xx',
    awsKey: 'xx',
    awsRegion: 'ap-southeast-1',
    awsKinesisStreamName: 'xx',
    status: 'ACTIVE',
    createdOn:<PHONE_NUMBER>218,
    modifiedOn:<PHONE_NUMBER>218 } ]

https://github.com/artikcloud/artikcloud-js/blob/master/src/model/SubscriptionArray.js#L46
This line is expecting a 'subscriptions' property but the payload does not have that property, it's just an array of objects.

Thanks for reporting this. We will look into it.

@fud thanks for sending.   In the json response you provided, I'm interested in seeing the 'parent' keys that was cutoff.   Could you share this?   Specifically, the 2nd parameter data parameter from the callback:
var callback = function(error, data, response) {
   console.log(JSON.stringify(data));
}

Can you confirm something like the following.   Array wrapped by a "data" key.
{
   "data":[
      {
         "id":"1673d7394737480d847023acfc50c9d",
         ...
      },
      {
         "id":"7dfb1e6290e848d9b798079651df11d",
         ...
      }
   ],
   "total":2,
   "offset":0,
   "count":2
}

FYI, I debugged by doing an output of console.log(response.res.text); Because at that stage the data has been unpacked through the subscriptionsApi.getAllSubscriptions call.
{
    "count": 1,
    "data": [
        {
            "aid": "691e4cd494164c48bcec989827accc58",
            "awsKey": "xx",
            "awsKinesisStreamName": "xx",
            "awsRegion": "ap-southeast-2",
            "createdOn":<PHONE_NUMBER>218,
            "description": "Send messages from Artik to Kinesis",
            "id": "39214c5abd5b432e9c416ea3493d21e8",
            "messageType": "message",
            "modifiedOn":<PHONE_NUMBER>218,
            "sdtid": "dtea63cceab0a44c83b29c714bb79ee51e",
            "status": "ACTIVE",
            "subscriptionType": "awsKinesis",
            "uid": "b7a08456a8474cb9b226d0e7f1e6a22b"
        }
    ],
    "offset": 0,
    "total": 1
}

@fud  - thank you for confirming and the quick response.   Team has discussed issue —  at this time fix is planned to change SubscriptionArray.js (and other similar issue) to conform based on the HTTP response.   Fix is planned for next release.
//SubscriptionArray.js
 if (data.hasOwnProperty('data')) {
        obj['data'] = ApiClient.convertToType(data['data'], [Subscription]);
 }


Fix doctests not running with --target=HOST.

There was a regression in #8167 where cargo test --target=$HOST stopped running doctests. This caused doctests to silently stop running in rust-lang/rust (https://github.com/rust-lang/rust/issues/73286).  This PR restores the original behavior where --target=$HOST behaves as-if it is a normal host test.
There was a discussion about this at https://github.com/rust-lang/cargo/pull/8167#pullrequestreview-402196323, but I think I let it slip through the cracks.

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+
Oops sorry about that!

:pushpin: Commit 1bf67a04023e91d72b9ed573651f0fc6c484b150 has been approved by alexcrichton

:hourglass: Testing commit 1bf67a04023e91d72b9ed573651f0fc6c484b150 with merge 4e14cc101a6cce226caf03d376d81cbb9218c33a...

:sunny: Test successful - checks-azure
Approved by: alexcrichton
Pushing 4e14cc101a6cce226caf03d376d81cbb9218c33a to master...