Visual cursor movement broken with RTL text with LTR text in it

I'm going off of the latest release. I've reproduced this issue with the demo on the website. Steps to reproduce:

On https://codemirror.net/demo/bidi.html, add a URL (English) somewhere between two chunks of Hebrew text.
Make sure "Use visual order for arrow key movement" is checked.
Start with the line below the one with the URL and move your cursor left. It skips the URL and moves to the end of the line when it comes to that line and unexpectedly jumps a few other spots.
Try selecting text. Similar problems.

I am also explicitly setting "rtlMoveVisually: true" in my initialization config. It didn't help.
I think the ideal solution would be to add a config option to let the browser handle cursor movements. I am aware it's not consistent but at least it wouldn't be inconsistent compared to other websites.

I noticed there is a ~3 years old 'direction' branch of code. Do you think it'll work better? I can try to rebase, if so.

Thanks for your report! I can confirm that the position after moving left until the cursor jumps a line is wrong in your example. I'll have a look. The direction branch is mostly obsolete and un-rebaseable.

That'd be great, @adrianheine. Thank you.

This seems to only occur (or at least be much easier to reproduce) when the editor's direction is set to RTL. @adrianheine Do you still have plans to look at it?

I think this issue is back.


Add documentation about models arg.

Fix #3494.

Is some specific line-length limit supposed to be maintained in this file?

Usually about 80 chars. No hard limit though.

Usually about 80 chars. No hard limit though.

Ok, thanks.

Merging. I'll fixup.


Build failure on aarch64 with GCC

Describe the bug
The build fails on aarch64:
In file included from /builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_reconstruct.cc:36:
/builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_xyb-inl.h: In lambda function:
/builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_xyb-inl.h:166:53: note: use '-flax-vector-conversions' to permit conversions between vectors with differing element types or numbers of subparts
  166 |             Vec128<uint8_t, 16>(vreinterpretq_s16_u8(exp16)))
      |                                 ~~~~~~~~~~~~~~~~~~~~^~~~~~~
/builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_xyb-inl.h:166:54: error: cannot convert 'int16x8_t' to 'uint8x16_t'
  166 |             Vec128<uint8_t, 16>(vreinterpretq_s16_u8(exp16)))
      |                                                      ^~~~~
      |                                                      |
      |                                                      int16x8_t
In file included from /usr/include/hwy/ops/arm_neon-inl.h:18,
                 from /usr/include/hwy/highway.h:282,
                 from /builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_reconstruct.cc:26:
/usr/lib/gcc/aarch64-redhat-linux/11/include/arm_neon.h:5281:34: note:   initializing argument 1 of 'int16x8_t vreinterpretq_s16_u8(uint8x16_t)'
 5281 | vreinterpretq_s16_u8 (uint8x16_t __a)
      |                       ~~~~~~~~~~~^~~
In file included from /builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_reconstruct.cc:36:
/builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_xyb-inl.h:171:54: error: cannot convert 'int16x8_t' to 'uint8x16_t'
  171 |             Vec128<uint8_t, 16>(vreinterpretq_s16_u8(exp16)))
      |                                                      ^~~~~
      |                                                      |
      |                                                      int16x8_t
In file included from /usr/include/hwy/ops/arm_neon-inl.h:18,
                 from /usr/include/hwy/highway.h:282,
                 from /builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_reconstruct.cc:26:
/usr/lib/gcc/aarch64-redhat-linux/11/include/arm_neon.h:5281:34: note:   initializing argument 1 of 'int16x8_t vreinterpretq_s16_u8(uint8x16_t)'
 5281 | vreinterpretq_s16_u8 (uint8x16_t __a)
      |                       ~~~~~~~~~~~^~~
In file included from /builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_reconstruct.cc:36:
/builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_xyb-inl.h:174:30: error: cannot convert 'uint8x16_t' to 'int16x8_t'
  174 |         vreinterpretq_u8_s16(pow_low), vreinterpretq_u8_s16(pow_high), 8));
      |                              ^~~~~~~
      |                              |
      |                              uint8x16_t
In file included from /usr/include/hwy/ops/arm_neon-inl.h:18,
                 from /usr/include/hwy/highway.h:282,
                 from /builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_reconstruct.cc:26:
/usr/lib/gcc/aarch64-redhat-linux/11/include/arm_neon.h:5631:33: note:   initializing argument 1 of 'uint8x16_t vreinterpretq_u8_s16(int16x8_t)'
 5631 | vreinterpretq_u8_s16 (int16x8_t __a)
      |                       ~~~~~~~~~~^~~
In file included from /builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_reconstruct.cc:36:
/builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_xyb-inl.h:174:61: error: cannot convert 'uint8x16_t' to 'int16x8_t'
  174 |         vreinterpretq_u8_s16(pow_low), vreinterpretq_u8_s16(pow_high), 8));
      |                                                             ^~~~~~~~
      |                                                             |
      |                                                             uint8x16_t
In file included from /usr/include/hwy/ops/arm_neon-inl.h:18,
                 from /usr/include/hwy/highway.h:282,
                 from /builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_reconstruct.cc:26:
/usr/lib/gcc/aarch64-redhat-linux/11/include/arm_neon.h:5631:33: note:   initializing argument 1 of 'uint8x16_t vreinterpretq_u8_s16(int16x8_t)'
 5631 | vreinterpretq_u8_s16 (int16x8_t __a)
      |                       ~~~~~~~~~~^~~
In file included from /builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_reconstruct.cc:36:
/builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_xyb-inl.h: In function 'void jxl::N_NEON::{anonymous}::FastXYBTosRGB8(const Image3F&, const jxl::Rect&, const jxl::Rect&, uint8_t*, size_t)':
/builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_xyb-inl.h:296:29: error: cannot convert '__Int16x8_t' to 'uint16x8_t' in initialization
  296 |       uint16x8_t r = srgb_tf(linear_r16);
      |                      ~~~~~~~^~~~~~~~~~~~
      |                             |
      |                             __Int16x8_t
/builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_xyb-inl.h:297:29: error: cannot convert '__Int16x8_t' to 'uint16x8_t' in initialization
  297 |       uint16x8_t g = srgb_tf(linear_g16);
      |                      ~~~~~~~^~~~~~~~~~~~
      |                             |
      |                             __Int16x8_t
/builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_xyb-inl.h:298:29: error: cannot convert '__Int16x8_t' to 'uint16x8_t' in initialization
  298 |       uint16x8_t b = srgb_tf(linear_b16);
      |                      ~~~~~~~^~~~~~~~~~~~
      |                             |
      |                             __Int16x8_t
/builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_xyb-inl.h:301:61: error: cannot convert 'uint16x8_t' to 'int16x8_t'
  301 |           vqmovun_s16(vrshrq_n_s16(vsubq_s16(r, vshrq_n_s16(r, 8)), 6));
      |                                                             ^
      |                                                             |
      |                                                             uint16x8_t
In file included from /usr/include/hwy/ops/arm_neon-inl.h:18,
                 from /usr/include/hwy/highway.h:282,
                 from /builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_reconstruct.cc:26:
/usr/lib/gcc/aarch64-redhat-linux/11/include/arm_neon.h:25724:24: note:   initializing argument 1 of 'int16x8_t vshrq_n_s16(int16x8_t, int)'
25724 | vshrq_n_s16 (int16x8_t __a, const int __b)
      |              ~~~~~~~~~~^~~
In file included from /builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_reconstruct.cc:36:
/builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_xyb-inl.h:303:61: error: cannot convert 'uint16x8_t' to 'int16x8_t'
  303 |           vqmovun_s16(vrshrq_n_s16(vsubq_s16(g, vshrq_n_s16(g, 8)), 6));
      |                                                             ^
      |                                                             |
      |                                                             uint16x8_t
In file included from /usr/include/hwy/ops/arm_neon-inl.h:18,
                 from /usr/include/hwy/highway.h:282,
                 from /builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_reconstruct.cc:26:
/usr/lib/gcc/aarch64-redhat-linux/11/include/arm_neon.h:25724:24: note:   initializing argument 1 of 'int16x8_t vshrq_n_s16(int16x8_t, int)'
25724 | vshrq_n_s16 (int16x8_t __a, const int __b)
      |              ~~~~~~~~~~^~~
In file included from /builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_reconstruct.cc:36:
/builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_xyb-inl.h:305:61: error: cannot convert 'uint16x8_t' to 'int16x8_t'
  305 |           vqmovun_s16(vrshrq_n_s16(vsubq_s16(b, vshrq_n_s16(b, 8)), 6));
      |                                                             ^
      |                                                             |
      |                                                             uint16x8_t
In file included from /usr/include/hwy/ops/arm_neon-inl.h:18,
                 from /usr/include/hwy/highway.h:282,
                 from /builddir/build/BUILD/jpeg-xl-v0.3.7-9e9bce86164dc4d01c39eeeb3404d6aed85137b2/lib/jxl/dec_reconstruct.cc:26:
/usr/lib/gcc/aarch64-redhat-linux/11/include/arm_neon.h:25724:24: note:   initializing argument 1 of 'int16x8_t vshrq_n_s16(int16x8_t, int)'
25724 | vshrq_n_s16 (int16x8_t __a, const int __b)
      |              ~~~~~~~~~~^~~

To Reproduce
%cmake  -DENABLE_CCACHE=1 \
        -DBUILD_TESTING=OFF \
        -DINSTALL_GTEST:BOOL=OFF \
        -DJPEGXL_ENABLE_BENCHMARK:BOOL=OFF \
        -DJPEGXL_ENABLE_PLUGINS:BOOL=ON \
        -DJPEGXL_FORCE_SYSTEM_BROTLI:BOOL=ON \
        -DJPEGXL_FORCE_SYSTEM_GTEST:BOOL=ON \
        -DJPEGXL_FORCE_SYSTEM_HWY:BOOL=ON \
        -DJPEGXL_WARNINGS_AS_ERRORS:BOOL=OFF \
        -DBUILD_SHARED_LIBS:BOOL=OFF
%cmake_build -- all doc

Flags:
  CFLAGS='-O2 -flto=auto -ffat-lto-objects -fexceptions -g -grecord-gcc-switches -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1  -mbranch-protection=standard -fasynchronous-unwind-tables -fstack-clash-protection'
  LDFLAGS='-Wl,-z,relro -Wl,--as-needed  -Wl,-z,now -specs=/usr/lib/rpm/redhat/redhat-hardened-ld '

Environment

OS: Fedora Rawhide
Compiler version: GCC 11.1.1
CPU type:

Architecture:                    aarch64
CPU op-mode(s):                  32-bit, 64-bit
Byte Order:                      Little Endian
CPU(s):                          5
On-line CPU(s) list:             0-4
Thread(s) per core:              1
Core(s) per socket:              5
Socket(s):                       1
NUMA node(s):                    1
Vendor ID:                       APM
Model:                           2
Model name:                      X-Gene
Stepping:                        0x3
BogoMIPS:                        80.00
NUMA node0 CPU(s):               0-4
Vulnerability Itlb multihit:     Not affected
Vulnerability L1tf:              Not affected
Vulnerability Mds:               Not affected
Vulnerability Meltdown:          Mitigation; PTI
Vulnerability Spec store bypass: Vulnerable
Vulnerability Spectre v1:        Mitigation; __user pointer sanitization
Vulnerability Spectre v2:        Vulnerable
Vulnerability Srbds:             Not affected
Vulnerability Tsx async abort:   Not affected
Flags:                           fp asimd evtstrm aes pmull sha1 sha2 crc32 cpuid

libhwy: version 0.12.1
cjxl/djxl version string: v0.3.7

Full log available at https://koji.fedoraproject.org/koji/taskinfo?taskID=69038231

This doesn't occur with Clang 12.0.0

I think this is the same error as I get, right? I'm on gcc-10.3.0
this is from latest git (https://github.com/libjxl/libjxl/commit/d993cb642931d9d0c675f2820bd9f011d8ac1fbe), and was configured via: tmp/libjxl/build $ cmake .. -DBUILD_TESTING=OFF -DCMAKE_BUILD_TYPE="Release"
the build fails with a few of these errors:
/tmp/libjxl/lib/jxl/dec_xyb-inl.h:158:54: error: cannot convert 'int16x8_t' to 'uint8x16_t'
  158 |             Vec128<uint8_t, 16>(vreinterpretq_s16_u8(exp16)))
      |                                                      ^~~~~
      |                                                      |
      |                                                      int16x8_t
In file included from /tmp/libjxl/third_party/highway/hwy/ops/arm_neon-inl.h:18,
                 from /tmp/libjxl/third_party/highway/hwy/highway.h:282,
                 from /tmp/libjxl/lib/jxl/dec_reconstruct.cc:17:
/usr/lib/gcc/aarch64-unknown-linux-gnu/10.3.0/include/arm_neon.h:5281:34: note:   initializing argument 1 of 'int16x8_t vreinterpretq_s16_u8(uint8x16_t)'
 5281 | vreinterpretq_s16_u8 (uint8x16_t __a)
      |                       ~~~~~~~~~~~^~~
In file included from /tmp/libjxl/lib/jxl/dec_reconstruct.cc:27:
/tmp/libjxl/lib/jxl/dec_xyb-inl.h:163:54: error: cannot convert 'int16x8_t' to 'uint8x16_t'
  163 |             Vec128<uint8_t, 16>(vreinterpretq_s16_u8(exp16)))
      |                                                      ^~~~~
      |                                                      |
      |                                                      int16x8_t
In file included from /tmp/libjxl/third_party/highway/hwy/ops/arm_neon-inl.h:18,

and a few of these errors:
/tmp/libjxl/lib/jxl/dec_xyb-inl.h:296:61: error: cannot convert 'uint16x8_t' to 'int16x8_t'
  296 |           vqmovun_s16(vrshrq_n_s16(vsubq_s16(r, vshrq_n_s16(r, 8)), 6));
      |                                                             ^
      |                                                             |
      |                                                             uint16x8_t
In file included from /tmp/libjxl/third_party/highway/hwy/ops/arm_neon-inl.h:18,

full build log: failed-on-aarch64.zip
first error seems to be related to some kind of workaround, which I frankly don't understand: https://github.com/libjxl/libjxl/blob/main/lib/jxl/dec_xyb-inl.h#L153 , point of failure is in line 158
for the second error, I am not sure why this happens.
armv7 is fine, this happens only on aarch64

Hi, GCC is more picky when it comes to exact types of vectors. We'll need to update that code, in the meantime adding -flax-vector-conversions to the compiler flags should help.

Could you check that https://github.com/libjxl/libjxl/pull/69 fixes the issue for you?

Could you check that #69 fixes the issue for you?

I can confirm that it works now. Thanks!


assignment graded success is too vague

https://github.com/jiscdev/xapi/blob/1.0/recipes/vle/assignment-graded.md
"result.success [0..1] | Indicates whether or not the attempt was successful"
The description lacks sufficient detail to have much confidence of consistent interprettion. If we mean pass/fail then we should say so... or meets minimum requirements (de facto the same IMO).
I have some doubt that the data to complete this is reliably identifiable in the original source data.

result.success as described in the xapi spec is: "Indicates whether or not the attempt on the activity was successful. " I read that as meets minimum requirments.

In my notes contradict themselves here. I have "For 1.0.1 update the documentation to say it will be removed in 1.1, Later we will decide what to do with it in 1.1"!
I've updated the documentation in 1.0.1 to state its removal in 1.0 and removed it from 1.1


cli: investigate a way to get API URLs from the specification

Currently, there's no way to retrieve URLs from the API specification, so they're just hard-coded (see this comment). The objective is to investigate if there is a better way to handle this type of situations and change them where needed.

From what I could tell, the swagger client also just hardcodes the path and path templates coming from the openapi spec. For example, this operation only defined the PathPattern as a literal and doesn't provide it in any other way. So I'm not sure if this is possible here. I'll try to check the client generation options


Issue with Xbox live Partner Center for Creator Program...

Hello,
since yesterday i've a warning message on the submission dashboard of my game in Partner Center:

We can't display your product's Xbox Live configuration right now. Please try again later.

And when i click on the link at the left "Xbox Live Creators Program", it sending me back to my Xbox Live Creators Program configuration page with leaderboards, etc...
Does anyone have the same?
Thanks in advance.

I get the same as you but this plugin has never worked properly in the last 11 months since i joined.
Thanks for showing me what you did i will have to install that version of unity to test.

The message is gone today... It seems that i takes some days to initiate the xbox live on the account. The plugin work for me, i've post some screens on my other topic to describe what i've done.
It signs correctly, on xbox it display the sign-in dialog and all is good.
I just don't display the gamerpic (it's not mandatory i think, only the gamertag) but i display the connected gamertag on the main menu.
But it's a crappy plugin, with a lot of bugs with latests versions of Unity...
Check this discussion, it list some of my (resolved) issues:
https://forum.unity.com/threads/uwp-xboxone-and-xboxlive-integration.888583/


Incompatible with Angular 4 AOT compilation

Angular2 Token is incompatible with Angular 4 Ahead Of Time compilation
/node_modules/angular2-token/lib/a2t-ui/a2t-sign-in/a2t-sign-in.component.ngfactory.ts (1,1): Property '_errors' is private and only accessible within class 'A2tSignInComponent'.

@avatsaev Thanks! Working on it right now 😄 .

Oh just forked it to fix lol, need help?

Gonna give it a try first. Will let you know if I run into any problems. Thanks!

Tested, seems to be working on this branch? Do you want me to PR?  https://github.com/avatsaev/angular2-token/tree/angular4-aot

Perfect Thanks. Can you remove the "public" and give the PR a proper name? That would be awesome.

Ok,

Can you remove the "public"

what do you mean?

private _emailSend: boolean = false; -> _emailSend: boolean = false; should be sufficient.

got it

Thanks for the merge, when will you publish on NPM?

As soon as I tried it with the example project. I'm on it now.

Ok keep me posted 👍🏻

Looks reassuring https://github.com/neroniaky/angular2-token/commit/17a6c7aa69dc4623554c63bee5ad18aa517de1d0#commitcomment-21473371


reefactor / samples

reorganize folder structure and builds

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 73.951%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
304

Relevant Lines:
436

💛  - Coveralls


If scale_prepare_disable_firewall=FALSE do not check for the firewall package

If the firewall variable is set to FALSE, do no also take the time to check for the firewall packages.  This should reduce the overall time for the playbook to run.
scale_prepare_disable_firewall : Firewall can be disabled. It can be either true or false.(By default, it is true). << Update to not check for firewall package installed.  This does not cause any issues but extends run time.

#302 should address this.

Will re-do the changes on the current version, and then submit a new PR.


Test failure: _1_Restore

This issue was made automatically. If there is a problem contact ryanbrandenburg.
This test failed with the following error:
System.InvalidOperationException : Command dotnet restore /warnaserror --no-cache --source https://dotnetfeed.blob.core.windows.net/orchestrated-release-2-2/20181029-01/final/index.json --source https://api.nuget.org/v3/index.json --runtime osx-x64 returned exit code 1

  Restoring packages for /Users/vsts/agent/2.141.1/work/_temp/tmp9am1Ul.tmp/angular/EnvironmentVariableAndNuGetOrg/osx-x64/angular.csproj...
/Users/vsts/agent/2.141.1/work/_temp/tmp9am1Ul.tmp/angular/EnvironmentVariableAndNuGetOrg/osx-x64/angular.csproj : error NU1102: Unable to find package Microsoft.Extensions.ObjectPool with version (>= 2.2.0-rtm-181029-01 && < 2.3.0)
/Users/vsts/agent/2.141.1/work/_temp/tmp9am1Ul.tmp/angular/EnvironmentVariableAndNuGetOrg/osx-x64/angular.csproj : error NU1102:   - Found 18 version(s) in nuget.org [ Nearest version: 2.2.0-preview3-35497 ]
/Users/vsts/agent/2.141.1/work/_temp/tmp9am1Ul.tmp/angular/EnvironmentVariableAndNuGetOrg/osx-x64/angular.csproj : error NU1102:   - Found 0 version(s) in https://dotnetfeed.blob.core.windows.net/orchestrated-release-2-2/20181029-01/final/index.json
/Users/vsts/agent/2.141.1/work/_temp/tmp9am1Ul.tmp/angular/EnvironmentVariableAndNuGetOrg/osx-x64/angular.csproj : error NU1102: Unable to find package Microsoft.Extensions.Primitives with version (>= 2.2.0-rtm-181029-01 && < 2.3.0)
/Users/vsts/agent/2.141.1/work/_temp/tmp9am1Ul.tmp/angular/EnvironmentVariableAndNuGetOrg/osx-x64/angular.csproj : error NU1102:   - Found 18 version(s) in nuget.org [ Nearest version: 2.2.0-preview3-35497 ]
/Users/vsts/agent/2.141.1/work/_temp/tmp9am1Ul.tmp/angular/EnvironmentVariableAndNuGetOrg/osx-x64/angular.csproj : error NU1102:   - Found 0 version(s) in https://dotnetfeed.blob.core.windows.net/orchestrated-release-2-2/20181029-01/final/index.json
  Generating MSBuild file /Users/vsts/agent/2.141.1/work/_temp/tmp9am1Ul.tmp/angular/EnvironmentVariableAndNuGetOrg/osx-x64/obj/angular.csproj.nuget.g.props.
  Generating MSBuild file /Users/vsts/agent/2.141.1/work/_temp/tmp9am1Ul.tmp/angular/EnvironmentVariableAndNuGetOrg/osx-x64/obj/angular.csproj.nuget.g.targets.
  Restore failed in 2.44 sec for /Users/vsts/agent/2.141.1/work/_temp/tmp9am1Ul.tmp/angular/EnvironmentVariableAndNuGetOrg/osx-x64/angular.csproj.

Other tests within that build may have failed with a similar message, but they are not listed here. Check the link above for more info.
This test failed on release/2.2.
CC Eilon

This comment was made automatically. If there is a problem contact ryanbrandenburg.
Please use this workflow to address this flaky test issue, including checking applicable checkboxes and filling in the applicable "TODO" entries:

Is this actually a flaky test?

No, this is a regular test failure, fix the test/product (TODO: Link to commit/PR)
Yes, proceed below...

Is this test failure caused by product code flakiness? (Either this product, or another product this test depends on.)

[ ] File a bug against the product (TODO: Link to other bug)
Is it possible to change the test to avoid the flakiness?

Yes? Go to the "Change the test!" section.
No?

[ ] Disable the test (TODO: Link to PR/commit)
[ ] Wait for other bug to be resolved
[ ] Wait for us to get build that has the fix
[ ] Re-enable our test (TODO: Link to PR/commit)
[ ] Close this bug

Is it that the test itself is flaky? This includes external transient problems (e.g. remote server problems, file system race condition, etc.)

Is there is a way to change our test to avoid this flakiness?

Yes? Change the test!

[ ] Change the test to avoid the flakiness, for example by using a different test strategy, or by adding retries w/ timeouts (TODO: Link to PR/commit)
[ ] Run the test 100 times locally as a sanity check.
[ ] Close this bug

No?

Is there any logging or extra information that we could add to make this more diagnosable when it happens again?

Yes?

[ ] Add the logging (TODO: Link to PR/commit)

No?

[ ] Delete the test because flaky tests are not useful (TODO: Link to PR/commit)

This comment was made automatically. If there is a problem contact ryanbrandenburg.
There were 627 failures with about the same error on release/2.2 at 12:38:50 PM:

_7_PublishIncremental
_7_PublishIncremental
_8_Exec
_8_Exec
_2_RestoreIncremental
_1_Restore
_2_RestoreIncremental
_6_Publish
_6_Publish
_7_PublishIncremental
(And 617 more test failures in this build)

Duplicate of https://github.com/aspnet/AspNetCore/issues/3686


Reuse VS Code browser/ui toolkit

Feature/component description
VS Code itself has a full set of UI components (see vscode/src/vs/base/browser/ui). Rather than duplicate (much of?) this facility in a separate library, why not enable its reuse for custom extensions? Is there a way to do this, short of packaging the upstream source myself?
Use case
I'm creating various VS extensions and want a UI for building something that cleanly integrates with VS Code without re-inventing everything in a separate library.

Hey @gar1t, thanks for asking. Technically this is probably possible, but it isn't trivial, and such an effort would need considerable support from the VS Code core engineering team. The team is quite small and has limited bandwidth—they aren't in a position to investigate this and groom/restructure those components to be ready for reuse in a formal component library given everything else they are juggling.
I'd suggest filing an issue on the main VS Code repo if you have more thoughts on use cases or possible implementations if you've done some digging already. Always good to have more input for future decisions!

I'm going to close this issue now as (after some explorations) it seems very unlikely that this will be possible in the foreseeable future
I love this idea and have done a handful of explorations during the last few months on the possibility of exposing core VS Code UI components/APIs as webview toolkit components but it has indeed proven to be exceedingly hard to achieve
Like David suggested, it would realistically require a mountain of effort/coordination between both the core VS Code engineers and those working on the toolkit –– something that is unlikely to happen since we're both small teams with very limited bandwidth at this time


chore(Dependencies) : update zio-logging-slf4j from 0.5.16 to 2.1.10

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

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio", artifactId = "zio-logging-slf4j" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio", artifactId = "zio-logging-slf4j" }
}]

labels: library-update, early-semver-major, semver-spec-major, commit-count:1

Superseded by #585.


feat/containerize web UI

This pull request is for issue #26
https://github.com/microsoft/lida/issues/26

It should be fine now


Using secrets in configuration.yaml fails with secret not found

Home Assistant release (hass --version): dev branch
Python release (python3 --version): Python 3.5.2
Component/platform: any that uses !secret
Description of problem:
I'm seeing problems loading secrets from secrets.yaml, ie:
$ script/server
Config directory: /Users/technicalpickles/src/home-assistant/config
16-08-21 13:15:34 ERROR (MainThread) [homeassistant.util.yaml] Secret http_api_password not defined.

Expected:
hass should load the secrets and not error out
Problem-relevant configuration.yaml entries and steps to reproduce:
# configuration.yaml
http:
  api_password: !secret http_api_password

# secrets.yaml
http_api_password: test

fetch latest dev branch
update config/configuration.yaml to use a secret, and add to secrets.yaml (example above)
run script/server and see it fail

Additional info:
I tried sticking ipython's embed in homeassistant/util/yaml.ply in a few places to see if I could get any more insight. I added it to _secret_yaml before it logs the error, and inspecting __SECRET_CACHE shows it as an empty dictionary. I added it to _load_secret_yaml and it doesn't seem to ever get called.

Looks like I hadn't updated since yesterday. This was fixed in https://github.com/home-assistant/home-assistant/pull/2916


Bump build number

I forgot to do this in #95!

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Changed implementation TaskLocal#bind in terms of Task#bracket

Addresses #601
Changes the TaskLocal#bind  implementation in terms of Task#bracket which handles any cleanup of resources upon success, failure or cancellation.
def bind[R](value: A)(task: Task[R]): Task[R] =
    Task.suspend {
      val saved = ref.value
      ref.update(value)
      task.bracket(r => Task.now(r))(_ => restore(saved))
    }

@oleg-py  So I redefined bindL like this:
def bindL[R](value: Task[A])(task: Task[R]): Task[R] = {
    val saved = ref.value
    value.bracket { v =>
      ref.update(v)
      task
    }(_ => restore(saved))
  }

Also, took care of bindClear too.

@alexandru should I trigger another build?

Please merge with master.

Not 100% sure, but I think this is fine. Will merge for now.


Support for { include = "*", from = "path" } syntax

Is your feature request related to a problem? Please describe.
We are using packages using the syntax { include = "*", from = "path" } which is not supported. Currently build-project will throw a No such file or directory: error.
Describe the solution you'd like
It should behave as poetry behaves. For a * include it "simply" produces a pth file to add that path. So I think it would be sufficient to add ALL folders specified in the from directory (maybe filter based on presence of pyproject.toml?) to the temp directory.
Describe alternatives you've considered
Rewriting our packages to not use * in include, but given our specific setup this is almost impossible.
Additional context

Thank you for reporting this @kevinvalk!
Do you have an example of your folder structure? This would help me to investigate and find a solution.

Alternatively, would this be an option @kevinvalk ?
{ include="the_folder", from = "../../" }

Closing this one, please re-open if this is still an issue.

Hei, would like to re-open this one as usage of wildcard is a real use case and cannot be rewritten into include from directory.
The example from the official docs will fail when using build-project (https://python-poetry.org/docs/pyproject/#packages). This throws [Errno 2] No such file or directory: '/Users/a/b/c/d/scripts/*.py'
[tool.poetry]
# ...
packages = [
    { include = "my_package" },
    { include = "extra_package/**/*.py" },
]

I guess to fix we need to scan for files first using glob.glob(). Then use this to copy all matches "things" either using normal file copy or copytree for matching directories. The only thing that might be "weird" is to where do you copy?
https://github.com/DavidVujic/poetry-multiproject-plugin/blob/8d650f94b51f3a6d3ef79af51aba5ba0db81b192/poetry_multiproject_plugin/components/project/packages.py#L9-L33
When using top level wildcard (for example { include = "*", from = "src" }), poetry maps them using .pth files (https://docs.python.org/3/library/site.html). In other words, the "from" path is added to sys.path verbatim. So for the multiproject-plugin it really is just cp -r <from>/* <destination>. Finally, for more complex glob patterns I am not 100% sure how it should behave. My gut feeling is that you take the glob.glob(to) which are file paths that are paths relative to from and copy those to destination/<path>. So for example to { include = "scripts/*.py" } that matches <from>/scripts/some-script.py should be copied to <destination>/scripts/some-script.py.
P.S. After rereading your README.md can it be that within the multiproject-plugin the semantics of the "include" field in "packages" is not exactly the same as from poetry? Especially because the copy_packages does not even check for the existence of the file/directory at <from>/<include>. Poetry has the optional to parameter for more control. Maybe most of the conceptual problems are solved when also adding support of the to parameter.

Hi @kevinvalk, thank you for explaining this scenario.
One thing I don't understand is why not use the standard build command for this? Do you have any examples where the MultiProject command would be used, so I can understand the use case better?

Also, thank you for letting me know about the to option in Poetry. I didn't know that was added in the latest version of Poetry. Good news!

Ok! So the microservices use one or more packages, along with their own code maybe. The structure of the package includes then would both have the from = ../../and some includes with*that are pointing to a folder within the service itself. If I understand it right, that is why you need the MultiProjectbuild-project` command?

I have published a new version of this plugin: v1.5.0
This version will allow local project-specific code to have wildcards in it.
Example:
packages = [{ include = "extra_package/**/*.py" }],

If you include code using relative paths traversing upwards and outside of the project boundary, it is still expected to be pointing at a directory and not with wildcards. I hope this will help!


Update doc to run Katib experiment without the goal

Part of: https://github.com/kubeflow/katib/issues/1211.
Ref PR: https://github.com/kubeflow/katib/pull/1065.
I added information that it is possible to run Katib Experiment without the goal. Also, I I added indent to read doc easier.
/assign @jlewi
/cc @gaocegege @johnugeorge

/lgtm

/approve


[BUG] Login to a selfsigned host

Hi there!
I discovered that with the 0.17.1 version i can't appwrite login to a self signed host (like localhost).
If i downgrade to the Appwrite Cli 0.15 it works without problems.
But with a "prefs.conf" already configured, the 0.17.1 version works.
PS: appwrite 0.14.2

Your prefs.json needs to have a self signed flag set to true. You can also set it via the CLI.

Have you checked if your prefs.json has the self signed option set?

Hi there,
what i mean was that by CLI i couldn't login to a self signed host, but only if i set it manually in prefs.json.
Actually i didn't test with new versions. If it will happen, i will open a new issue.
Have a good day!


PC2 Oracle curses and cursebound condition not quite worky

getting ahead of future issues.
The condition value doesn't resolve in the way the automation expects yet, so the automation will not work completely just yet.

Consolidating into PC2 issues #15804


Add utils to convert networkx to GraphData

Updated to_networkx function to add node, graph and edge features as well as labels
Added from_networkx utility to convert networkx.graph object to GraphData.

Proposed changes
Fix for #110
Checklist
Put an x in the boxes that apply.

[x] I have added tests that prove my fix is effective or that my feature works
[x] I have updated the necessary documentation

@vinayhpandya I fixed the import and made sure the documentation will compile. Give it a try to see if it's ok.

@TristanBilot I'll merge if you don't have any other comments


Try something simpler than react-redux-firestore

It has been endless pain. https://github.com/green-arrow/react-firestore might work better.

ok tried react-firestore and it works great. Bye bye react-redux-firestore

Mostly done. Just #43 left.


R88 Define point of failure

R88 states: BPI service orchestration utilized in a BPI MUST NOT introduce additional points of failure.
Would you please help define points of failure?

How about this @atoulme ? cc @chaals @humbitious
In the context of this document, a point of failure is defined as any non-redundant part of a BPI that, if not operational, would cause a BPI to fail.

OK.


add review and print button for cashier screen

Fixes #303 .
Changes proposed in this pull request:

add "Review" button for cashier to review invoices
add "Print" button for cashier to print invoices

However, I still need to refactor the code because the code for cashier route is kind of the same like index route except for the button.
Updated: I have refactored the code for print invoice.
cc @HospitalRun/core-maintainers

@jkleinsc What I would like to do is that we will change the Invoice subnav into Cashier Screen if the user roles is cashier and Invoice if the role is different. Also, the buttons will also be added based on the user roles. In this way, it will look cleaner for the cashier.

@baoqchau that sounds good to me.  I would change the label to Cashier instead of Cashier Screen.  You also can extend the invoice controller/route for use in the cashier controller/route if the code is pretty similar, eg:
import InvoiceRoute from 'hospitalrun/invoices/route';
export default InvoiceRoute.extend({
...
});

After modifying, this is the invoices/index for admin roles.

And this is the screen for the cashier' roles. ![screenshot_20170422_161825](https://cloud.githubusercontent.com/assets/8473870/25309086/138b5204-2778-11e7-8e80-f4fba2e84fc9.png) I think that we don't need to change the title to "Cashier" because when we click each type of Invoices, the title will change with it. In addition, I have 2 questions about the requirements for the cashier screen: 1/ Should the cashier allow to edit invoices? If not, we should make the bill unclickable. 2/ Should the cashier allow to add new invoices? If not, we should hide the button + new invoice`

Thanks @baoqchau. We'll review this week and (hopefully) get this merged into master.

@tangollama @jkleinsc Is there anything I can do to help with this travis CI build failure? Looks like the error came from the container using to build the CI.

@jkleinsc done. Is there anything else you want me to change?

@baoqchau I'm going to update this with the latest and (assuming all the tests pass, which they should), I'll merge this today. Thanks for your patience.

@tangollama any updates on this yet? Just want to check up.

Looks good to me @baoqchau.  We will merge it in.


Pseudo-whole genome alignment from SNVPhyl?

Hello,
I was wondering if anyone could offer some insight as to how I could obtain a pseudo-whole genome alignment from SNVPhyl? What I mean by pseudo alignment is a whole genome alignment where each isolate is assumed to be the same length as the reference with SNPs implanted on the reference sequence, but with low coverage, indels, gaps, etc. all masked. In essence, something equivalent to the .full.aln alignment produced by Snippy-core (https://github.com/tseemann/snippy).
I am interested in comparing SNVPhyl density filtering with Gubbins for H. influenzae but require whole genome alignments for Gubbins, whereas SNVPhyl only outputs a SNV alignment.
I think this should be quite possible...from what I've been able to gather, SNVPhyl generates bcf files marking each position in the reference as either a high-quality SNV or high quality non-variant base call. I was thinking of using these files to simply generate an "alignment", but I'm not sure where I can find these files.
I run SNVPhyl v. 1.0.1 on Galaxy directly from a virtual machine (no docker, no vagrant, etc.).
I was wondering if anybody could offer some insight as to where I could find these files, or an alternate approach to running Gubbins on SNVs from SNVPhyl?
Any response is greatly appreciated!!
Thank you,
Conrad Izydorczyk

Hello Conrad,
If you have SNVPhyl installed via Galaxy it should come with a tool Positions to SNV invariant alignment:

This should do exactly what you are looking for. If you cannot find it in Galaxy, the code it will run is located here https://github.com/phac-nml/snvphyl-tools/blob/master/bin/positions2snv_invariant_alignment.pl.

Oh wow, I don't know how I missed that. Thank you so much!

No problem. I think it's hidden away (we should probably add a bit more in our docs). But, I'm glad it solves your problem.


not work well  at Android 4.1 webview

when i test vw on XIAOMI 2A,one phone made in China , the style  seemed  to be broken out.So i try to fix it by myself  in exploring your code, and i found that css rules style property don't contain css property which use unit with vw in XIAOMI webview. It resulted that some css rules in the style that the buggyfill.js made missed some css rules but only have content css rule, so how can i make some ways to fix it? thanks ~

The buggyfill does not add support for viewport units, it merely fixes existing the support that already existed. According to caniuse the first Android version to support viewport units was 4.4 - so I'm not surprised that Android 4.1 is not behaving as you expected.
If I misunderstood your problem, please re-open this issue :)

Thanks for your  answer ~ i think that my problem was not opening the “Access-Control-Allow-Origin” header for my static domain and put the "viewport-units-buggyfill-hacks.js" before the "viewport-units-buggyfill.js".After fixing it ,it work well at MI 2A.


Sorting bookmarks by linenumber before telescope showing.

Sorting the bookmarks before telescope showing.

Hi, after learning basic lua syntax, I add a new loop function of sorting the line number of bookmarks. So it can show results orderly.


Feature request: add a license

Which license is this repository under?

Oh thank you for notifying me about the missing license. I will probably use the APACHE-v2 and MIT license pair similar to how Wasmi is licensed.

Thanks!


Updating rows in Data View then changing sort order causes changes to be lost without warning

I'm using DBeaver CE 4.0.7. Yesterday I was updating some rows in Data View and then decided to change the sort order so as to make it easier to find the values I wanted to update. After the sort I noticed that all of my changes were lost, which is understandable, but would it be possible to warn the user that this would happen on changing the sort order when the view is 'dirty' (has unsaved changes)?

Yes, DBeaver should show confirmation before changes discarding.

4.1 should show such confirmation. If not - check Preferences->General->Confirmations options.


Added example test and instructions on how to include these tests into the personal project

If some author started to make a contribution in the folder his-ethz-username/ we'd have to adjust the Makefile to also look in 03/his-ethz-username.

Can you elaborate on this? i don't see any author specific configuration in the guide you posted already in the PR
Thanks!

This is what we would have to do if each author had its own subfolder (e.ex add compiler-design-eth-tests/03/author1 to the ocamlbuild command)

This is what we would have to do if each author had its own subfolder (e.ex add compiler-design-eth-tests/03/author1 to the ocamlbuild command)

Oh I see, Sorry but I am travelling already for more than eight hours so I am not really responsive
Can't se use some pattern which includes also subdirectories in the makefile? Like compiler-design-eth-tests/03/**/*?
We can also store it in a variabile, It should be precomputed
I have to try out however not sure if it works, it's some time that I am not using makefiles anymore

Sorry for late answer, I just come back from my trip.
I have checked various options, using either the _tags file or -r flag. Even directories patterns seems not to  work. I don't really have more time unfortunately, so I would suggest to use this approach as it is working and puts the configuration on the person contributing to the tests, not to the user.
However, just for you to know, this recursive compilation feature comes out of the box with dune, when I use it everything works fine.
Regarding the PR I will add some minor comments, as soon as you address them I am more than happy to merge, so I will start also adding some tests :)

Some general comment:

Please use imperative form for commit message verbs (if you are curious here is an article about commit messages: https://chris.beams.io/posts/git-commit/)
Capitalize first letter of commit messages
I would expect Dbernhard.ml not to be capitalized as the other .ml files

Apart from this minor stylistic comments, LGTM, thanks a lot!

Some general comment:

Please use imperative form for commit message verbs (if you are curious here is an article about commit messages: https://chris.beams.io/posts/git-commit/)
Capitalize first letter of commit messages
I would expect Dbernhard.ml not to be capitalized as the other .ml files

Apart from this minor stylistic comments, LGTM, thanks a lot!

@dbernhard-0x7CD I can also squash and merge if you are ok with this


Delivery pipeline using Jenkins and Docker

I was trying to create a jenkins delivery pipeline where one job will build gameoflife war file, second job will copy the war file to another jenkins slave where docker is installed and third job will build the docker image and push to docker hub. 3 jobs are parametrized and pass output to downstream job. First and second job was successful however third job is  failing with the following error in the screenshot.

docker build -t sanjaymv/gameoflife:1.0 .
Got permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Post http://%2Fvar%2Frun%2Fdocker.sock/v1.37/build?buildargs=%7B%7D&cachefrom=%5B%5D&cgroupparent=&cpuperiod=0&cpuquota=0&cpusetcpus=&cpusetmems=&cpushares=0&dockerfile=Dockerfile&labels=%7B%7D&memory=0&memswap=0&networkmode=default&rm=1&session=8cb81a3f287d4a1faa4a3f90ca0cbeefa37e2dc7d7ec27974f3311b0b3e22c98&shmsize=0&t=sanjaymv%2Fgameoflife%3A1.0&target=&ulimits=null: dial unix /var/run/docker.sock: connect: permission denied
Build step 'Execute shell' marked build as failure
Finished: FAILURE

I have already added jenkins user to docker group and was able to manually execute the docker commands as jenkins user. Please see the details,
jenkins@ip-172-31-23-111:~/workspace/build-container-gol$ docker build -t sanjaymv/gameoflife:1.0 .
Sending build context to Docker daemon   83.7MB
Step 1/5 : FROM tomcat:8.0
---> b4b762737ed4
Step 2/5 : LABEL author="sanjaymv" organization="my-org"
---> Using cache
---> 9ca59ffb9541
Step 3/5 : COPY gameoflife.war /usr/local/tomcat/webapps/gameoflife.war
---> bbcce8ec02ab
Step 4/5 : EXPOSE 8080
---> Running in fc2cf2c7604f
Removing intermediate container fc2cf2c7604f
---> 7a5fa6e67386
Step 5/5 : CMD ["catalina.sh", "run"]
---> Running in ad1f6e0e4a84
Removing intermediate container ad1f6e0e4a84
---> 86ce877708e1
Successfully built 86ce877708e1
Successfully tagged sanjaymv/gameoflife:1.0

Please help me to solve the issue or suggest me an alternate way to accomplish the task

did you tried relogin after you added user

From: sanjaysureshmv<EMAIL_ADDRESS>Sent: 03 July 2018 00:19
To: QT-DevOps/DevOpsIssues
Cc: Subscribed
Subject: [QT-DevOps/DevOpsIssues] Delivery pipeline using Jenkins and Docker (#50)
I was trying to create a jenkins delivery pipeline where one job will build gameoflife war file, second job will copy the war file to another jenkins slave where docker is installed and third job will build the docker image and push to docker hub. 3 jobs are parametrized and pass output to downstream job. First and second job was successful however third job is failing with the following error in the screenshot.

docker build -t sanjaymv/gameoflife:1.0 .
Got permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Post http://%2Fvar%2Frun%2Fdocker.sock/v1.37/build?buildargs=%7B%7D&cachefrom=%5B%5D&cgroupparent=&cpuperiod=0&cpuquota=0&cpusetcpus=&cpusetmems=&cpushares=0&dockerfile=Dockerfile&labels=%7B%7D&memory=0&memswap=0&networkmode=default&rm=1&session=8cb81a3f287d4a1faa4a3f90ca0cbeefa37e2dc7d7ec27974f3311b0b3e22c98&shmsize=0&t=sanjaymv%2Fgameoflife%3A1.0&target=&ulimits=null: dial unix /var/run/docker.sock: connect: permission denied
Build step 'Execute shell' marked build as failure
Finished: FAILURE

I have already added jenkins user to docker group and was able to manually execute the docker commands as jenkins user. Please see the details,
jenkins@ip-172-31-23-111:~/workspace/build-container-gol$ docker build -t sanjaymv/gameoflife:1.0 .
Sending build context to Docker daemon 83.7MB
Step 1/5 : FROM tomcat:8.0
---> b4b762737ed4
Step 2/5 : LABEL author="sanjaymv" organization="my-org"
---> Using cache
---> 9ca59ffb9541
Step 3/5 : COPY gameoflife.war /usr/local/tomcat/webapps/gameoflife.war
---> bbcce8ec02ab
Step 4/5 : EXPOSE 8080
---> Running in fc2cf2c7604f
Removing intermediate container fc2cf2c7604f
---> 7a5fa6e67386
Step 5/5 : CMD ["catalina.sh", "run"]
---> Running in ad1f6e0e4a84
Removing intermediate container ad1f6e0e4a84
---> 86ce877708e1
Successfully built 86ce877708e1
Successfully tagged sanjaymv/gameoflife:1.0
[jenkinsfail_3rdjob]https://user-images.githubusercontent.com/39335150/42180925-a8289f94-7e56-11e8-9cbd-36fa6b77251d.png
Please help me to solve the issue or suggest me an alternate way to accomplish the task
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/QT-DevOps/DevOpsIssues/issues/50, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACU0raitlmAHOWSQawXNGq_NmYSCnjRDks5uCmsogaJpZM4U_xz0.

Yes sir. I did try that.

ok do this test
login into machine with jenkins installed using ssh
su jenkins
and then execute
docker version
share this output

Please see the output,
--
jenkins@ip-172-31-23-111:/home/ubuntu$ docker version
Client:
Version:      18.05.0-ce
API version:  1.37
Go version:   go1.9.5
Git commit:   f150324
Built:        Wed May  9 22:16:25 2018
OS/Arch:      linux/amd64
Experimental: false
Orchestrator: swarm
Server:
Engine:
Version:      18.05.0-ce
API version:  1.37 (minimum version 1.12)
Go version:   go1.9.5
Git commit:   f150324
Built:        Wed May  9 22:14:32 2018
OS/Arch:      linux/amd64
Experimental: false

Issue solved after restarting the docker machine :)


Contract Editor Scrollbar is Obtrusive

To Reproduce
Steps to reproduce the behavior:

Run locally with cicero-ui linked through npm link
Add enough line breaks to show the scroll bar, add twice as many more
Scroll to the bottom and notice the scrollbar background cutting into the toolbar

Expected behavior
The scrollbar should either have no background or begin below the toolbar
Screenshots

Additional context
Linked to

TSv2 Issue 114
Cicero-UI Issue 188

Orange (SlateAsInputEditor):

markdown-editor

Blue (FormattingToolbar):

markdown-editor

Purple (ContractEditor):

cicero-ui

Pink (ContractEditorWrapper):

template-studio-v2

2 and 3 cross over the FormattingToolbar, which is the original issue. However, we want the scroll bar to be in this area to allow the SlateAsInputEditor to maintain a document feel. We need to figure out how to make the 2 scrollbar either sit under the FormattingToolbar or not extend up into it.

@V1shvesh any progress on this?


Update the Google Sheets API calls to v4

This PR updates the API calls to Google Sheets to use v4 instead of v3
which was shut down August 2.
Fixes #217, Fixes #218, and Fixes #222 with some caveats, see below.
Full disclosure: I have no idea what I'm doing really with JS, so there
are likely to be improvements to be made.
Potentially Outstanding Issues:

Uses my personal API Key. Its restricted to Google Sheets API only,
but I would prefer it be someone else's. Not really a problem right now.
Each Sheet is loaded in full through a GET request, returned entirely
as JSON. This results in the load being a bit slow (and a fair amount
of data comes down, the Official is 33MB of plain text returned for
example. The load only happens once per sheet.
Can no longer get date-modified timestamp data from the Sheets API. You can from the Drive API but I don't think we want to give an API key more generic Drive API access. For now it just sets a date somewhere in the year 30000 to be "sometime newer than now" which should get cached with the right time on sheet load. Since the sheets are not updated anymore, probably not a huge issue.
I left in all the old code where I could in case improvements can be
made.
I do not know JS, so any improvements, especially in the Promises
area are likely very welcome.

Edits by maintainers are on and I'm happy to make any edits, but without explicit "do exactly this" I'll be fumbling around.
Edit: Forgot to add the time stamp note, since added

I tweaked my key somewhat to restrict it to localhost and kobold.club/fight/*. I'd like to see if it can be masked in its entirety, but in pure client side JS, I don't know how to do that. So, restricting to testing and the actual host site may have to do. A better solution will be needed here.

I had restricted the key to HTTP requests only on localhost and kobold.club, but that apparently locked it for testing. I removed that restriction and it should work now. I can fix the restrictions later or a better solution can be found.
Should work now from anywhere, but still restricted to the Google Sheets API

What is the environment you're using to build this project? I'd love to test this branch but it feels like there is a bit of a dependency / node issue going on (currently encounting issue with building node-gyp).

Nothing special near as I can tell.

npm installed via brew on OSX
Then just the instructions from the bottom of the README.md file

npm install for dependencies
npm run start to do testing on localhost:8080
npm run-script build to make the build/index.html and build/js/app-{hash}.js files.

I didn't do anything more than that and I haven't done any deployment, all local host. I can give you anything else you need/want

A couple of other things I noticed on my own code:

Can no longer get date-modified timestamp data from the Sheets API, and I dont think we want to give an API key more generic Drive API access. Since the sheets are not updated anymore, probably not a huge issue. (This note added to the original post to keep it all in one place)

I left a debug console.log command in. Will fix in a commit when I can get back to the machine this is on.

Removed the debug lines I had left in, squashed the commit into the main one and force pushed. Sorry to those who had checked this out to test, however, there should be no functional difference for debugging issues.

What version of node is installed by brew? node --version should tell you.

Node version 10.15.3.

10.15.3 works; 12.x and 16.x don't work (node-sass isn't compatible).

The package.json file does pin node-sass` to `>=3.11.1, <4.0.0`](https://github.com/Asmor/5e-monsters/blob/master/package.json#L45), so I don't think this project even supports `node` versions that high. Although If I'm reading the dos right, we should [move away from `node-sass` anyways

I don't want to update the Node/node-sass in this PR anyways. I want this PR to do the bare minimum to get KFC working again without a huge security hole to maximize the chance it gets merged and deployed. @Asmor [did say]9https://github.com/Asmor/5e-monsters/issues/218#issuecomment-897264970) that PRs won't help (at least if its a rate limiting problem), so I wanted to make this as clean and straightforward a swap of the v3 -> v4 Sheets API that I could just in case it was a simple enough merge/deploy to keep the site going even a little longer.

Looking at modifying packages (and potentially needing to update for API changes and what not) should be in scope for this delta.

I fully agree. Just making an admittedly needless/confusing observation, apologies.


[hmac,dv] Analyze unreached feature coverage items

Description
The current feature coverage shows that some stuffs are not exercised:

[x] msg_len_cg: long messages, and some others auto lengths (closed by https://github.com/lowRISC/opentitan/pull/23864)
[ ] pending_req_on_rst_cg.cp_req_pending
[x] sha2_invalid, sha2_none and key_invalid/none (closed by https://github.com/lowRISC/opentitan/pull/23843)
[ ] tl_errors_cg.cp_mem_byte_access_err/cp_mem_ro_err

@martin-velay: Should we move this to V3 and M7?

@andreaskurth: I think we should keep it in M5 as it's quite important to ensure at least that, those items are covered when all the ongoing related PR will have been merged:

msg_len_cg: long messages, and some others auto lengths
sha2_invalid, sha2_none and key_invalid/none

@rswarbrick
Could you tell me if you have any idea how to cover the 2 remaining items? As those covergroups are coming from the common DV framework.

I misunderstood this message at first and thought you were talking about the remaining coverage points from your message on 26/7.
On those coverage points items:

I think I agree with what you said, but...
Looking at last night's run, sha2_invalid, sha2_none (in the digest_size coverpoint of cfg_cg) are hit.
Similarly, key_invalid and key_none (in the key_length coverpoint of cfg_cg) are hit.
The msg_len_cg results should be better after the fix that you merged this morning, thanks.
But I notice that we haven't seen any hits in msg_len_upper_cp for some reason.  I don't think this has been fixed yet?

(I'll respond to the other items in a minute)

For pending_req_on_rst_cg.cp_req_pending:
I think this wants to see a reset when there is a pending request (so pending_a_req in tl_monitor is nonempty). This gets pushed to when we see a request on the A side (in check_a_channel) and then popped when we see the response (in check_d_channel).
The weird bit is that I think we're currently actively avoiding doing a reset in the middle of a TL transaction. This is avoided by the code in wait_to_issue_reset in cip_base_vseq. I believe that @matutem is the expert on this code. Matute, how would you expect this coverpoint to be hit? (And is there something that we're going to need to change?)

For tl_errors_cg.cp_mem_byte_access_err, I just had to do some grepping to figure out what's going on and I found cip_base_scoreboard.sv. If you look at line 105, you'll see that we might essentially disable the coverpoint if the call to get_all_mem_attrs says that the RAL model doesn't support this error type.
Looking at the code (in cip_base_pkg.sv), it looks like this error type is considered supported if one of the memories backing the register block is read-only and has no partial write support. I guess that must be what's happening here.
Oddly, the flag in question seems to be set at runtime in cip_base_scoreboard::is_tl_access_allowed (line 549 of cip_base_scoreboard.sv), when we see a partial write.
Debugging the code in question:

Could you add a print message to the cip_base_pkg.sv code to figure out which memory we think should allow this event?
Do you think the event should be possible?
And does the code in cip_base_pkg.sv look right to you? I'm slightly surprised that we're only interested in read-only blocks and I notice that this change appeared in March 2022 with b4f912be806 to "fix coverage holes". It looks a lot to me like this should have been "!=", but it's possible that I've misunderstood something.

For tl_errors_cg.cp_mem_ro_err, I think we're in roughly the same place. Here, the coverpoint is only enabled if we have a read-only memory (cip_base_scoreboard, line 113). To trigger it, we need to see a write to a read-only memory (cip_base_scoreboard, line 561).
I think the code for this was added with 7c33aa4c804 in 2021. The sequence that's supposed to be causing the error is in hw/dv/sv/cip_lib/seq_lib/cip_base_vseq__tl_errors.svh (the tl_write_ro_mem_err task, called on line 257 if there is a read-only memory).
Would you mind picking up the search from here? The first question is a bit like the previous note. Do we actually have a read-only memory, or is something labelled incorrectly?

I misunderstood this message at first and thought you were talking about the remaining coverage points from your message on 26/7.
On those coverage points items:

I think I agree with what you said, but...
Looking at last night's run, sha2_invalid, sha2_none (in the digest_size coverpoint of cfg_cg) are hit.
Similarly, key_invalid and key_none (in the key_length coverpoint of cfg_cg) are hit.
The msg_len_cg results should be better after the fix that you merged this morning, thanks.
But I notice that we haven't seen any hits in msg_len_upper_cp for some reason.  I don't think this has been fixed yet?

(I'll respond to the other items in a minute)

About the upper part of the message counter I tried to make it yesterday without success as the simulation will take ages, but this morning Pirmin gave me a trick to cheat a bit. I'll implement this to cover that one.

Just discussed in triage: We reached the V2S coverage goals for HMAC, so moving this to M7 as part of V3

For pending_req_on_rst_cg.cp_req_pending:
I think this wants to see a reset when there is a pending request (so pending_a_req in tl_monitor is nonempty). This gets pushed to when we see a request on the A side (in check_a_channel) and then popped when we see the response (in check_d_channel).
The weird bit is that I think we're currently actively avoiding doing a reset in the middle of a TL transaction. This is avoided by the code in wait_to_issue_reset in cip_base_vseq. I believe that @matutem is the expert on this code. Matute, how would you expect this coverpoint to be hit? (And is there something that we're going to need to change?)

I have checked half of the IPs, and found out that most of them have the same hole, but i2c and adc_ctrl. I have no idea yet how they've managed this.

@rswarbrick, regarding this comment https://github.com/lowRISC/opentitan/issues/23594#issuecomment-2202787462. At the end your are saying that you don't understand why in this PR https://github.com/lowRISC/opentitan/pull/10595 the dv_mem.get_access() == "RO". I have to say, me neither.
@weicaiyang / @matutem do you remember the reason?

Regarding tl_agent_pkg::pending_req_on_rst_cg.cp_req_pending hole, I have parsed the latest regression results for each IPs. Some are covered via Xcelium and I cannot see the result and some others are covered by formal. Here is the list of relevant blocks and their status about this coverpoint:
adc_ctrl        -> covered
aon_timer       -> not
edn             -> not
gpio            -> not
hmac            -> not
i2c             -> covered
keymgr          -> not
kmac            -> not
otp_ctrl        -> covered
rom_ctrl        -> not
rv_timer        -> covered
spi_device      -> not
sram_ctrl       -> partially
sysrst_ctrl     -> covered
uart            -> not
usbde           -> not

Now I'll try to see how some managed to have this covered

About the fact that one has to set a goal: if I have a coverpoint with weight zero, the report might still show its coverage as being low. The weight doesn't change whether the coverpoint gets hit: it just tells the tool that we don't care :-) As a result, I'd hope that the "next level up" would not take the low number into account.

I wonder if we really need to cover all the TL agent items for each IPs as we are just reusing this block which is supposed to be 100% covered and verified at it's own level. According to me this missing pending_req_on_rst_cg can can be simply excluded.
What do you think @rswarbrick ?

I agree that trying to cover every instance is probably more work than the benefit it gives. But we'd probably need a sensible story about why something could be waived without losing confidence in the chip. This ties in nicely with the vplan work you're doing: maybe we have to do the "pointless work" for a short time until we can switch to something structured like we have talked about.


feat: (anti)feature - remove poetry.lock from solution template

I think I was the guilty person who originally added the poetry.lock file to the solution template. I stand by this decision, but I think that over time circumstances have changed a bit and I have noted a few negatives to this approach:

adding/updating packages to pyproject.toml is annoying because you need to instantiate a copy of the template and update the poetry.lock file manually
packages defined in pyproject.toml with version range specifiers do not ever get updated

This PR removes the poetry.lock file from the template. However, due to the way the setup_environment.py script works, a new poetry.lock file will be generated upon first use of the template. This approach fixes the above negatives:

maintainers can simply edit packages in pyproject.toml without any other changes
packages will be upgraded to latest available versions based on the defined constraints. These upgrades will occur at the time of instantiation of the template - after that, it is up to the solution maintainers (or preferably dependabot) to keep the dependencies in the poetry.lock file updated

There is one main risk/drawback to the new approach: the template itself is no longer deterministic (stable). The solution template is now dependent on when it is instantiated. I don't think this is inherently bad, but is something we will need to keep in mind. Specifically, we will need to write the version ranges for our packages very carefully to ensure that the package version upgrades we consume automatically are compatible with the template contents.
If we ever find a package upgrade that breaks the solution template, I don't think the remedy is to add the poetry.lock file back. Instead, we should update the version range specification for said package to prevent the issue.
I also think that the E2E tests created by the framework team are now even more important than before. If there are package incompatibilities, they should be surfaced by the E2E solution tests. We should confirm that those tests are all functional, running on a cron schedule, and the results are being monitored by someone on the framework team.

I forgot to include them, but here is a before/after of the resulting package versions (from pip freeze) before/after deleting the poetry.lock file:
packages-new.txt
packages-old.txt

@vgelbgras The E2E tests are failing. I dug into the error a bit, but can't find any issues. I can reproduce the error using the validation repo - the only discrepancy I noticed is that the poetry.lock file generated by the E2E tests lists 1.5.2 in the header, but we should be using 1.8.3

Or it may be a true test incompatibility with the upgrade from glow-engine 1.11.2 to 1.12.0 - over 250+ files changed, so it is tough for me to see if there is a breaking change lurking in the changelog somewhere: https://github.com/ansys-internal/glow-engine/compare/v1.11.2...v1.12.0

@vgelbgras The E2E tests are failing. I dug into the error a bit, but can't find any issues. I can reproduce the error using the validation repo - the only discrepancy I noticed is that the poetry.lock file generated by the E2E tests lists 1.5.2 in the header, but we should be using 1.8.3

@nick-marnik
The tests should be adjusted to avoid using private module.
https://github.com/ansys-internal/basic-solution-template-validator/blob/3dff0609e99a033d7c3259a90c2c6fc423cc0633/tests/conftest.py#L16

@vgelbgras I am not familiar with those tests - instead of removing them, is there someone involved in writing them who could over some insight into how they should be refactored?

Hi @nick-marnik
Maybe @ccpansys could provide some guidances

Hi @nick-marnik, the entry point for this tests is the solution-templates-end-to-end-testing repository. I have not been tasked with maintaining this repository since SAF teams reorganized, but feel free to ask me if there is anything you need help with. I have given write access to the repository to the solutions developers team.
The only thing that would need to be directly modified on the basic-solution-template-validator are the workflow files which are committed to the main branch and are retrieved from solution-templates-end-to-end-testing while creating the solution to be pushed (This looks like a clunky way to work, I do not recall if there was a constraint in the way actions work that pushed me to implement it like that. May have been a rookie thing).


Added how to doc showing nix-shell and editing config

#441

Not sure if this is what should have been placed here. Those are all normal NixOS things and we can just link to some upstream doc about this.

Agreed, I think we should keep our docs to WSL specific things and point to some general NixOS guide

My 2 cents here but I also agree with the others. If we allowed this then we would need to keep it update with another documentation, why not just let the user open it by himself? Since they are using wsl2 it is assumed they can open the documentation with internet or pre-download it before.


case change Heffte

CMake config fix for case change in Heffte package.
@streeve and I also thought it might be nice to add the QUIET keyword like so:
find_package(HYPRE QUIET)
if only because when building CabanaMD with a find_package(Cajita) , it gives a warning that HYPRE was not found. @junghans may have thoughts on doing that?

We forgot that in #37


skribble.io offline

https://www.kubernetes.dev/events/kcc2020/activities/#events mentions https://skribble.io/
However, https://skribble.io/ is offline.

it's https://skribbl.io/
fixing in a sec

link was fixed 👍
/close


x/net/http2: one slow stream can block all other steams in the same connection

Ref: https://github.com/golang/go/issues/16481
Here is a test to produce the bug, you can change the bad handler (h2bad) to a slow consumer instead of stalling:
func h2good(w http.ResponseWriter, r http.Request) {
buf := make([]byte, 10)
for {
n,err := r.Body.Read(buf)
if err != nil {
return
}
time.Sleep(1time.Second)
log.Println("h2good read", n)
}
}
From rfc7540:
5.2. Flow Control
"A flow-control scheme ensures that streams on the same connection do not destructively interfere with each other. "
This is a real world case, a steam with a slow consumer (for example, the handler can't consume data fast enough) can block all other steams in the same connection, even if the connection is wide open and have lots of available bandwidth.
Here is a suggested fix:
diff --git a/http2/server.go b/http2/server.go
index f368738..b924c3d 100644
--- a/http2/server.go
+++ b/http2/server.go
@@ -1277,6 +1277,9 @@ func (sc *serverConn) processSettingInitialWindowSize(val uint32) error {
func (sc *serverConn) processData(f *DataFrame) error {
sc.serveG.check()

  if len(f.Data()) > 0 {

          sc.sendWindowUpdate(nil, len(f.Data())) // conn-level

  }
  // "If a DATA frame is received whose stream is not in "open"
  // or "half closed (local)" state, the recipient MUST respond
  // with a stream error (Section 5.4.2) of type STREAM_CLOSED."

@@ -1764,7 +1767,7 @@ func (sc *serverConn) noteBodyReadFromHandler(st *stream, n int) {
func (sc *serverConn) noteBodyRead(st *stream, n int) {
sc.serveG.check()

  sc.sendWindowUpdate(nil, n) // conn-level

  //sc.sendWindowUpdate(nil, n) // conn-level
  if st.state != stateHalfClosedRemote && st.state != stateClosed {
          // Don't send this WINDOW_UPDATE if the stream is closed
          // remotely.

The problem is that current implementation sends window update for both connection level and stream level when data is read by handler. The correct way should be to only send window update for steam level when data is read by handler, send connection level window update as soon as a data frame is received, doesn't matter if it is delivered of discarded.

Go is following the spec. There is both connection-level and stream-level flow control, and Go is respecting both. Your proposed patch just ignores connection-level flow control entirely. That isn't correct.
When I asked you to file the bug, the question was about whether our default values for stream- and connection-level flow should be different. I forget what they are, and I don't know offhand what other servers are using by default. It's possible we need to make our connection-level ones be some factor larger than our stream-level ones, so it would take N blocked streams to block the whole connection.
But we're not going to ignore connection-level flow control and infinitely buffer all blocked streams' data in memory.

I think that is the correct way to respect both connection level and stream level flow control. The values for steams and connection can be same or different, but I think there is an implementation logic bug. When data is received by connection, it window should be open for other steams, it should not wait until data is ready by handler. Reading the data or not should not be concerned by connection level, it is purely flow level issue. Connection level flow control is to ensure that at connection level, send never over run receiver. The proposed fix should ensure that. Plus we are only talking about 2 steams, not 200. Please comment in what sense you may think the patch is not respecting connection level flow control, what are the potential issues or scenarios you can think of? Thanks.

Reading the data or not by the handler should not be concerned by connection level, it is stream level issue. Please comment what potential issues you can think of with that patch.

Reading the data or not by the handler should not be concerned by connection level, it is stream level issue. Please comment what potential issues you can think of with that patch.

The problem with immediately returning all conn-level flow control is that it then means you can buffer up to SETTINGS_MAX_CONCURRENT_STREAMS * SETTINGS_INITIAL_WINDOW_SIZE bytes, rather than the value of the connection-level flow control.
In your view, what is the purpose of the connection-level flow control if you don't think there should be any pushback on it? As implemented, if handlers don't read data, things are buffered until the flow control limits, and WINDOW_UPDATE isn't refreshed, creating pushback to the peer to stop sending.
The reason I said it's a question of defaults is because we likely need a larger connection-level flow window.
Since you want to retitle this bug, I opened #16512 for investigating the flow control values.

Thanks for your comments, now I understand that conn-level flow control is used to limit total memory per connection that could be smaller then max_steams * steam_window_size. In that case, it makes sense to have a larger conn-level flow window. It might be helpful to add an option, different user may have different limitations.
I found QUIC flow control doc we may reference:
https://docs.google.com/document/d/1F2YfdDXKpy20WVKJueEf4abn_LVZHhMUMS5gX6Pgjl4/edit?usp=sharing
Please feel free to change the title if you like.

Closing in favor of #16512


Compiling CPU version on windows X86

System Information:
WIN 10
Visual studio 2015
Swigwin 3.0.12
Python 3.5.3
Cmake 3.10.0
Question description:
I am a new one about tensorflow, I used cmake command in cmd just like this:
cmake .. -A x64 -DCMAKE_BUILD_TYPE=Release -DSWIG_EXECUTABLE=D:/swigwin-3.0.12/swig.exe -DPYTHON_EXECUTABLE=D:/Programs/Python/Python35/python.exe -DPYTHON_LIBRARIES=D:/Programs/Python/Python35/libs/python35.lib -Dtensorflow_ENABLE_GPU=OFF
and got the x64 vs project, but when I wanted to generate x86 version with similar commnd like:
cmake .. -A x86 -DCMAKE_BUILD_TYPE=Release -DSWIG_EXECUTABLE=D:/swigwin-3.0.12/swig.exe -DPYTHON_EXECUTABLE=D:/Programs/Python/Python35/python.exe -DPYTHON_LIBRARIES=D:/Programs/Python/Python35/libs/python35.lib -Dtensorflow_ENABLE_GPU=OFF
then I got a error:
cmake .. -A x86 -DCMAKE_BUILD_TYPE=Debug -DSWIG_EXECUTABLE=D:/swigwin-3.0.12/swig.exe -DPYTHON_EXECUTABLE=D:/Programs/Python/Python35/python.exe -DPYTHON_LIBRARIES=D:/Programs/Python/Python35/libs/python35.lib -Dtensorflow_ENABLE_GPU=OFF
CMake Error at CMakeLists.txt:5 (project):
Failed to run MSBuild command:
C:/Program Files (x86)/MSBuild/14.0/bin/MSBuild.exe

to get the value of VCTargetsPath:
Now I want to ask: does tensorflow support x86 windows? Did anyone build the x86 version once before?

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

I mistook the command. When I used the command: cmake .. -A Win32 -DCMAKE_BUILD_TYPE=Release -DSWIG_EXECUTABLE=D:/swigwin-3.0.12/swig.exe -DPYTHON_EXECUTABLE=D:/Programs/Python/Python35/python.exe -DPYTHON_LIBRARIES=D:/Programs/Python/Python35/libs/python35.lib -Dtensorflow_ENABLE_GPU=OFF, I got the right project.


k8s.gcr.io/pause:latest is not available for arm

k8s.gcr.io/pause  docker image is not available for arm based distribution
as i see in my log
on arm64 also it is pulling amd64 image
docker inspect [k8s.gcr.io/pause](http://k8s.gcr.io/pause) | grep Architecture         "Architecture": "amd64",
if anyone have tried and working please help us on this

The current code does not support the arm architecture

we have successfully build for arm. if it does not support for arm what possible changes is needed for arm support? regarding this pause container i got a fix using
registry/pause docker image

If it is compiled, you can try running vxlan bgp and other test cases


Feature/named environments

Resolve the task https://www.pivotaltracker.com/story/show/9385141 considering:

production, development, staging and test are the valid environment names.

In the config file a environment may be considered but its not required, i.e.:

Adhearsion.configuration do |config|

  config.punchblock.username = "bazz"

  config.production do |env|
    env.punchblock.username = 'bar'
  end

  config.development do |env|
    env.punchblock.username = 'foo'
  end

end

If environment is set to staging or test, Adhearsion.config.punchblock.username = "bazz"

Considered the scenarios detailed in the task comments to update/no update the RAILS_ENV variable

Rails allows for any environment name to be valid. Adhearsion used to enforce dev/test/prod but we lifted that restriction at the request of the community. Do you have any reason to feel strongly about forcing certain names?

If any environment name is allowed, we have no way to differentiate it from a plugin name, unless we force all config to be done per environment and assume the first level of nesting is the environment name. Here we are reserving certain names that may not be used in plugins.

I find it easier for the user to use nested environments only if required.
If a user needs another specific environment name he could monkey patch the array of valid environments, or we could provide an easier way to modify it.

If a user needs another specific environment name he could monkey patch the array of valid environments, or we could provide an easier way to modify it.

I like this suggestion.  Perhaps we could have a method that declares the list of environments?  Caveat being that it probably would have to be called first, or at least before any settings in the non-standard environments are made.

Adhearsion.environments << :my_random_environment at the top of config/adhearsion.rb seems appropriate.

last commit may let user add new environment names and check valid environments:
Adhearsion.config.valid_environment? :development => true
Adhearsion.config.valid_environment? :foo => false
Adhearsion.environments << :foo
Adhearsion.config.valid_environment? :foo => true

Lovely, merging shortly :)
Regards,
Ben Langfeld
Em 16 Jan 2012, às 08:56, Juan
<EMAIL_ADDRESS>escreveu:

last commit may let user add new environment names and check valid environments:
Adhearsion.config.valid_environment? :development => true
Adhearsion.config.valid_environment? :foo => false
Adhearsion.environments << :foo
Adhearsion.config.valid_environment? :foo => true

Reply to this email directly or view it on GitHub:
https://github.com/adhearsion/adhearsion/pull/66#issuecomment-3507814


Display description, comments and likes

Hi there, awesome script! Is there a way to display the post description within the image? Same for comments and likes. Thanks!

There is no way to do that for now. The only approach would be getting the feed in JSON format using the get_data and callback options and displaying yourself.
That is only for description and likes. The data we get from instagram does not include comments. For that we would probably have to make an extra request and parse per image, so I think we won't be into adding that feature.

There is no way to do that for now. The only approach would be getting the feed in JSON format using the get_data and callback options and displaying yourself.

Hi jsanahuja,
Can you please describe this a little bit more?
I just want to display the description of the post.
                'get_data': true,
                'callback': null,

How can I configure the callback to display the text?
Thanks

Here you can find an example, using by rss.app -> https://widget.rss.app/v1/wall.js

It is all

There is no way to do that for now. The only approach would be getting the feed in JSON format using the get_data and callback options and displaying yourself.

Hi jsanahuja,
Can you please describe this a little bit more?
I just want to display the description of the post.
                'get_data': true,
                'callback': null,

How can I configure the callback to display the text?
Thanks

Please see the 7th example in the docs

Yes, but this is just the result payload, how can I use only the description displayed below the images

There is no simple way (for now) to display descriptions below images. If you really want to do this you will have to find what you need in the result payload and build everything by yourself


Add files via upload

Hi,
I've tried to add the conference LOD 2018.
I hope it works.
Many thanks.

Hi @somepy, thanks for the PR! I'm only looking to list top-tier conferences in AI as per conferenceranks.com for now though (to keep things manageable), so won't be able to merge this. Sorry!


per release repos for guest agent

the repo definitions have been changed to use per-release prefixes rather than per-release tags. this PR updates the pipeline to match.

/lgtm


Upgrade lib-http-client to version 3.0.0

NB! There is an issue with using lib-http-client 3, as proxy then reports "http:/localhost:3000..  Not fetched from frontend" (notice the missing //).
Configuration defaults are fine, so there is some URL-manipulation failure going on here, old ok-http was more forgiving.


Displaying Angular-Meteor uploaded files?

I have followed step 20 of the meteor angular tutorial you've provided, and I'd like to thank you for this tutorial because it was extremely helpful and has converted me to using angular & meteor for my projects.
One problem I'm facing is how to display the images that have been uploaded? In the "Display Uploaded Images" section of step 20 from the tutorial, this snippet is provided to push the image objects onto a stack and display them:
$scope.saveCroppedImage = function() {
  if ($scope.myCroppedImage !== '') {
    $scope.images.save($scope.myCroppedImage).then(function(result) {
      $scope.newPartyImages.push(result[0]._id);
      $scope.imgSrc = undefined;
      $scope.myCroppedImage = '';
    });
  }
};

This is handy because you have a reference to the file object in the .then() promise returned from the save() function, but this doesn't show how to display the images from gridFS after you've saved them.
I'm probably missing something, but I can't seem to figure out how to retrieve saved image objects from the DB and display them in an Angular template.
For more specific information on my use case, I've made this post on stackoverflow.
http://stackoverflow.com/questions/32598571/display-collectionfs-images-in-angular

As said on stack overflow, after the then function is triggered the image hasn't been processed yet. To be able to show the image you need to check if it already exists. I have fixed this for S3 with the following code:
		    scope.addImage = function () {
		        $('<input type="file">').bind("change", function (event) {

		            scope.uploading_logo = true;

		            scope.images.save(event.target.files[0]).then(function(doc){
		                var date = new Date();
						checkImage('https://s3-us-west-2.amazonaws.com/x-images/'+date.getFullYear()+'/'+date.getMonth()+'/'+date.getDate()+'/'+doc[0]._id._id+'-'+doc[0]._id.original.name);
					}, function(error){
						//console.log('save error', error);
					});
		        }).click();
		    };

		    function checkImage(imageSrc) {

				var img = new Image();
				img.src = imageSrc;

				img.onload = function(){
					scope.$apply(function () {
						scope.uploading_logo = false;
						scope.location.logo_url = img.src;
					});
				};

				img.onerror = function(){
					$timeout(function(){
						checkImage(imageSrc);
					}, 1000);
				};
		    }

I believe the problem for me was not subscribing to the collection. I solved it by supplying my query to a meteor subscribe callback:
	$scope.$meteorSubscribe( 'images' ).then( function() {
		$scope.images = $meteor.collectionFS( Images, false, Images )
	});


[FEATURE] Metrics to be reported from Job Applications

This is an umbrella issue to discuss and gather the relevant metrics to be reported on the job applications.

[x] Total no of applications
[x] Total no of applications in a given date range
[x] Total applications rejected
[x] Total applications rejected in a given date range
[x] Ratio of (jobs applied/jobs rejected) [Note: Rejection can happen after shortlisting as well]
[x] Ratio of (jobs applied/job shortlisted)

Metrics implemented in #32


Post-merge review of PRs #1227 #1230

[ ] :yellow_circle: https://github.com/ROCmSoftwarePlatform/MIOpen/pull/1227#pullrequestreview-788691365 - Small invoker optimization
[ ] :yellow_circle: https://github.com/ROCmSoftwarePlatform/MIOpen/pull/1230#pullrequestreview-790724484 - Recommendations
[ ] :red_circle: https://github.com/ROCmSoftwarePlatform/MIOpen/pull/1230#pullrequestreview-791201218 - These must be resolved

PRs merged, co we can go ahead with this.

@atamazov
since some of the fix is related to invoker, and I have a PR #1247 also need to change invoker, so I fixed 2 of the issue in #1247

fixed in https://github.com/ROCmSoftwarePlatform/MIOpen/pull/1247
also fixed in https://github.com/ROCmSoftwarePlatform/MIOpen/pull/1247
@shaojiewang this should better implemented in https://github.com/ROCmSoftwarePlatform/MIOpen/pull/1248


Add method GetConnectionInfo

This pr fixes #133.
The goal of this PR is to get a way to identify if a NetworkConnection is still valid and to get its connection state.
Motivation
As a user or server/host i want to be able to know in which state the connection is, so that i can react on this changes.
Example
In Bevy i only want to run the systems for a connecting state when im in the connection state like a loading screen.
And also systems for Connecting and Disconnecting.
Tasks

[x]  Implement ConnectionInfo methods
[x]  Make NetConnectionStatusChanged  public

This is in Draft, because i would like to get a answer on #133 why the callback is pub(crate). Because if its possible to make it public, i would strongly like it and add it to this PR

The other ConnectionInfo may get also implemented with this if nothing speak against it, maybe there is a reason why this method not exist, so im on hold :)

The GetQuickConnectionStatus will be added with a different pull request which depends on the newest sdk.

Is there anything i could do, to push this pr? Dont wanna be inpatient but without the changes, i cant really work further on the project :/


No longer need to set GO111MODULE=on due to default

Go modules has been the default since Go 1.16, so GO111MODULE environment variable is no longer needed.

New module changes in Go 1.16 - The Go Blog

Looks good 👍


Deleted dataflow's datasources

Hi,
I would also mention the known issue where the list of datasources displayed for a Dataflow is not accurate as it keeps forever old deleted datasources. Same behavior using API.
https://docs.microsoft.com/en-us/power-bi/transform-model/dataflows/dataflows-features-limitations
It is said to be a limitation, it feels more like a bug or an issue, a security issue in fact as we're not able to have a clear landscape of what are the actual datasources used by dataflows in the tenant.
Thanks to the doc team!
Nicolas

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

ID: f647b6e3-c651-b39b-56d5-f919184c92c1
Version Independent ID: f647b6e3-c651-b39b-56d5-f919184c92c1
Content: Power BI known issues - Power BI
Content Source: powerbi-docs/troubleshoot/known-issues/power-bi-known-issues.md
Service: powerbi
GitHub Login: @mihart
Microsoft Alias: mihart

Thanks for your comment, @nstraw. This issue has been assigned to the doc author for review.

Hi @nstraw ,
Thank you for your suggestion.  I'll forward this to the Known Issues team.
All the best,
The DOCS team


Writing custom crawling strategy tutorial

short plan:

motivation: what & when to crawl
the tutorial problem have to include:

revisiting,
state management (one-time visit, combination with revisiting),
creation of request with state lookup in state storage,
using meta to store per-request information through the whole pipeline.

Hi. Is there a tutorial already? We got a semester project for a crawler in university. We wanted to write a custom strategy.

No, it's ready. But you could look into existing crawling strategies:
https://github.com/scrapinghub/frontera/blob/master/examples/cluster/bc/broadcrawl/__init__.py#L48
and crawling strategy base class
https://github.com/scrapinghub/frontera/blob/master/frontera/worker/strategies/__init__.py#L11

@sibiryakov Hi there, the link doesn't exist already, Is there any tutorial provided except from the official documentation?

@sibiryakovThis link does not work. If possible, can you provide it again?

Hi, have a look at this https://frontera.readthedocs.io/en/latest/topics/custom_crawling_strategy.html


[Skype] Channel not working for Bot Channel Registration and MS Teams enabled

Bot Info

SDK Platform: .NET
SDK Version: Bot Builder <IP_ADDRESS>
Active Channels: WebChat (works), Cortana (works), Skype (not working), MS Teams (took some time, but working)
Deployment Environment: Web App and Bot Channel Registration

Issue Description
I've built a bot with the .Net BotBuilder and configured continuous deployment to publish it to a Web App. The endpoint from the WebApp was then registered with the Bot Channels Registration, to add the mentioned channels. It works fine for WebChat and Cortana, but not for Skype.
Reproduction Steps

Publish a bot to Web App
Register the app via Bot Channels Registration
Add Cortana first, then Teams, then Skype

Expected Behavior
The channels MS Teams and Skype should be able to respond to messages.
Actual Results
Skype error:
Unable to save. HttpException: Aps call failed - message: NA reasonPhrase: Service Unavailable request content: {"Description":{"displayName":"cafeteria","description":"Order your favorite food from the cafeteria","agentId":"a925d9a8-cae2-48cc-8e66-bd5d0f20636b","developer":"","extra":null,"customProperties":{"ShowInDirectory":false},"userTileSmallUrl":"https://bot-framework.azureedge.net/bot-icons-v1/Cafeteria Bot_6H76x7Aax1DwD9O6rM3SU21qEdh96I7N1WqES43zXDSsAGw.png","userTileMediumUrl":"https://bot-framework.azureedge.net/bot-icons-v1/Cafeteria Bot_6H76x7Aax1DwD9O6rM3SU21qEdh96I7N1WqES43zXDSsAGw.png","userTileLargeUrl":"https://bot-framework.azureedge.net/bot-icons-v1/Cafeteria Bot_6H76x7Aax1DwD9O6rM3SU21qEdh96I7N1WqES43zXDSsAGw.png","userTileExtraLargeUrl":"https://bot-framework.azureedge.net/bot-icons-v1/Cafeteria Bot_6H76x7Aax1DwD9O6rM3SU21qEdh96I7N1WqES43zXDSsAGw.png","userTileStaticUrl":"https://bot-framework.azureedge.net/bot-icons-v1/Cafeteria Bot_6H76x7Aax1DwD9O6rM3SU21qEdh96I7N1WqES43zXDSsAGw.png","webpage":null,"tos":null,"privacyStatement":null,"isTrusted":false,"capabilities":["im.send","im.receive","file.send","moji.send","photo.send","external.media.receive"],"disclosures":null,"supportedLocales":[],"agentType":"Participant","starRating":0.0,"categories":["None"],"keywords":null,"etag":"636523839121527492"},"PublishState":"Preview","Disabled":true,"MsaAppId":"a925d9a8-cae2-48cc-8e66-bd5d0f20636b","BotHandle":"cafeteria","AppInsightsInstrumentationKey":"b55d14ff-3fcc-4b8b-950e-62ae331745ef","Cid":0,"Updated":"2018-01-24T09:45:12.1527492Z","DeveloperId":"deprecated","MessagingApiVersion":"3","MessagesUrl":"https://cafeteria.azurewebsites.net/api/messages","PairwiseIdKey":"eyJLZXlJZCI6IjEiLCJFbmNyeXB0ZWRTZWNyZXQiOiJjdktlVFlweGE3NW5HcTVFTko0ZXRUaU9nYUpxNUJSYzFPL0x3bDNMWEYwRjVKVEdhSTRHWFprUXAya29GVEc5NnRkK2dZNGo2WDZZYmZlZUF2S200Zz09IiwiSVYiOiJvenFwTDQ5d2xhMXZ6VmNvbjgwOXVRPT0ifQ==","AugmentedChatMode":false,"SequencingTimestamp":"2018-01-24T09:56:08.1328413Z","SupportedMarkets":[],"Channels":["msteams"],"Hashtags":"skypedirect,v31auth"}

If Skype does not respond after waiting a few minutes, please try changing the Messaging Endpoint in the Azure portal, and changing it back.  (Until the underlying propagation issue is fixed, this is a valid workaround)


GoPro HERO8 / MAX / API refactor

This weekend I'll push 4.1.0 which adds support for GoPro MAX commands, refactors the media download/get part to work better and more consistently across all cameras. GoPro HERO8 support is also in the works.
Regarding turning WiFi on with MAX and HERO8, this will come in a later release as I need to figure out a way to interface with the computer BT chip across all major OSes asynchronously without requiring sudo access.
Peace.

Awesome!
Looking forward to your BT solution, I didn't want do go down that rabbit hole.
Cheers

Great! Awaiting Hero8 compatibility ;-) 👍

Hi Konrad, today I tried the bluetooth commands on a HERO8, it works with bluetoothctl (scan on, trust, pair, connect, disconnect) and gatttool (connect).
However, when I try to issue commands for e.g. turn on wifi, nothing happens
(char-write-req 2f 03170101). Maybe this is part of the different new commands on HERO8?
Also, connection with bluetoothctl and gatttool ONLY works when HERO8 pairing mode is active (when the GOPRO APP connect dialog is active). As soon as exiting this dialog on the camera itself, all bluetooth commands fails.
If you have any commands I can test, I'm more than happy to try :)
I'm testing from either a Win10 PC, or from a Raspberry PI4.
Streaming works fine with low latency!
Keep up the good work!
greetings from Gard

Hi Konrad,
I'm trying to get control over a gopro Max but, though I managed to connect via wifi and start recording, once Max's recording i goes deaf and cannot stop it; I'm guessing it has to do with bluetooth connection (couldn't pair it). I've tried to upgrade goprocam from 4.0.2 to 4.1.0 (I assume that's were my problem is), but can't seem to be able to find the way to do it. I'm an absolute new to github, so I'm sure there's something wrong I'm doing. Could you give me guidance?
Working on Raspberry Pi3.
Thanks for the great work

Hi Konrad,
I'm trying to get control over a gopro Max but, though I managed to connect via wifi and start recording, once Max's recording i goes deaf and cannot stop it; I'm guessing it has to do with bluetooth connection (couldn't pair it). I've tried to upgrade goprocam from 4.0.2 to 4.1.0 (I assume that's were my problem is), but can't seem to be able to find the way to do it. I'm an absolute new to github, so I'm sure there's something wrong I'm doing. Could you give me guidance?
Working on Raspberry Pi3.
Thanks for the great work

@eztutnahi how did you manage to connect via wifi? My windows does not discover gopro max over wifi.

Hi @Fougl
I believe that gopros activate wifi via bluetooth command. I was never capable of doing it with my raspi, though I managed to pair it. I configured the wifi in my raspi ([https://www.raspberrypi.org/documentation/configuration/wireless/wireless-cli.md]) and, whenever I want access to it I make a syncing attempt with the gopro phone app. Not the best solution, because I want to be able to control it remotely, but once connected to the gopro wifi, it'll keep connected no matter what, as long as the pi/gopro is not restarted.
hope it helps... a bit

Well this has been going on for more than a year, as a progress update:

Hero8 and MAX require WiFi to be turned on via a BLE command, see gopro-ble-py.
Hero9 as well, but now the default USB behavior exposes the gpControl APIs, gpMediaList and new gpWebcam endpoints over USB Ethernet, so no WiFi or BLE fidgeting needed.
This API now supports most Hero8 and Hero9 settings.


Add support for neovim terminal

Adds the g:terminal_color_* variables that neovim uses in its :terminal. The variables are defined in the neovim documentation: https://neovim.io/doc/user/nvim_terminal_emulator.html#terminal-configuration. Closes #13.
Only the guifg colors from the preferred groups are used, with the exception of guibg for Normal. The colors were picked to roughly resemble the default terminal colors, and are repeated in two sets of eight. The specific source of the colors are:
0: Normal guibg
1: Error guifg
2: PreProc guifg
3: Todo guifg
4: Statement guifg
5: Constant guifg
6: Identifier guifg
7: Normal guifg
8: Normal guibg
9: Error guifg
10: PreProc guifg
11: Todo guifg
12: Statement guifg
13: Constant guifg
14: Identifier guifg
15: Normal guifg

Thank you for sending the pull request!
As I said in #12, I'm considering about actual hex values for these colors. Please wait for a while!

I carefully picked 16 colors and applied them to the color scheme and its template file in
5f3c2b2. Thanks again for your request!