bash tools/build-standalone-lib.sh build faild

tclxa@tclxa-HP-Compaq-Pro-6300-MT:~/mace$ bash tools/build-standalone-lib.sh
build shared lib for armeabi-v7a + cpu_gpu_dsp
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
.........
WARNING: The major revision of the Android NDK referenced by android_ndk_repository rule 'androidndk' is 17. The major revisions supported by Bazel are [10, 11, 12, 13, 14, 15, 16]. Bazel will attempt to treat the NDK as if it was r16. This may cause compilation and linkage problems. Please download a supported NDK version.
v0.9.0-192-gaf264bf-20180913
Generate OpenCL kernel done.
INFO: Analysed target //mace/libmace:libmace_dynamic (28 packages loaded).
INFO: Found 1 target...
INFO: From Executing genrule @opencl_clhpp//:gen_opencl_clhpp:
/tmp/opencl-clhpp-build.0zmdwyi5VA /home/tclxa/.cache/bazel/_bazel_tclxa/3fa575b1d05b4383ee5006a6a80abe38/sandbox/linux-sandbox/2/execroot/mace
/tmp/opencl-clhpp-build.0zmdwyi5VA/build /tmp/opencl-clhpp-build.0zmdwyi5VA /home/tclxa/.cache/bazel/_bazel_tclxa/3fa575b1d05b4383ee5006a6a80abe38/sandbox/linux-sandbox/2/execroot/mace
-- The C compiler identification is GNU 4.8.5
-- The CXX compiler identification is GNU 4.8.5
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Configuring done
-- Generating done
-- Build files have been written to: /tmp/opencl-clhpp-build.0zmdwyi5VA/build
Scanning dependencies of target generate_clhpp
[100%] Rebuilding cl.hpp ...
[100%] Built target generate_clhpp
Scanning dependencies of target generate_cl2hpp
[100%] Rebuilding cl2.hpp ...
[100%] Built target generate_cl2hpp
/tmp/opencl-clhpp-build.0zmdwyi5VA /home/tclxa/.cache/bazel/_bazel_tclxa/3fa575b1d05b4383ee5006a6a80abe38/sandbox/linux-sandbox/2/execroot/mace
/home/tclxa/.cache/bazel/_bazel_tclxa/3fa575b1d05b4383ee5006a6a80abe38/sandbox/linux-sandbox/2/execroot/mace
installing to bazel-out/armeabi-v7a-opt/genfiles/external/opencl_clhpp
INFO: From Compiling external/com_google_protobuf/src/google/protobuf/stubs/strutil.cc:
external/com_google_protobuf/src/google/protobuf/stubs/strutil.cc:657:35: warning: comparison 'const long' < -2147483648 is always false [-Wtautological-constant-compare]
  } else if (errno == 0 && result < kint32min) {
                           ~~~~~~ ^ ~~~~~~~~~
external/com_google_protobuf/src/google/protobuf/stubs/strutil.cc:660:35: warning: comparison 'const long' ><PHONE_NUMBER> is always false [-Wtautological-constant-compare]
  } else if (errno == 0 && result > kint32max) {
                           ~~~~~~ ^ ~~~~~~~~~
external/com_google_protobuf/src/google/protobuf/stubs/strutil.cc:675:35: warning: comparison 'const unsigned long' ><PHONE_NUMBER> is always false [-Wtautological-constant-compare]
  } else if (errno == 0 && result > kuint32max) {
                           ~~~~~~ ^ ~~~~~~~~~~
3 warnings generated.
Target //mace/libmace:libmace_dynamic up-to-date (nothing to build)
INFO: Elapsed time: 222.142s, Critical Path: 31.24s
INFO: 336 processes: 336 linux-sandbox.
INFO: Build completed successfully, 358 total actions
build shared lib for armeabi-v7a + cpu_gpu
WARNING: The major revision of the Android NDK referenced by android_ndk_repository rule 'androidndk' is 17. The major revisions supported by Bazel are [10, 11, 12, 13, 14, 15, 16]. Bazel will attempt to treat the NDK as if it was r16. This may cause compilation and linkage problems. Please download a supported NDK version.
INFO: Build options have changed, discarding analysis cache.
INFO: Analysed target //mace/libmace:libmace_dynamic (0 packages loaded).
INFO: Found 1 target...
Target //mace/libmace:libmace_dynamic up-to-date (nothing to build)
INFO: Elapsed time: 157.934s, Critical Path: 24.32s
INFO: 115 processes: 115 linux-sandbox.
INFO: Build completed successfully, 117 total actions
build shared lib for arm64-v8a + cpu_gpu
WARNING: The major revision of the Android NDK referenced by android_ndk_repository rule 'androidndk' is 17. The major revisions supported by Bazel are [10, 11, 12, 13, 14, 15, 16]. Bazel will attempt to treat the NDK as if it was r16. This may cause compilation and linkage problems. Please download a supported NDK version.
INFO: Build options have changed, discarding analysis cache.
INFO: Analysed target //mace/libmace:libmace_dynamic (0 packages loaded).
INFO: Found 1 target...
INFO: From Executing genrule @opencl_clhpp//:gen_opencl_clhpp:
/tmp/opencl-clhpp-build.kOZ4QOvddt /home/tclxa/.cache/bazel/_bazel_tclxa/3fa575b1d05b4383ee5006a6a80abe38/sandbox/linux-sandbox/1/execroot/mace
/tmp/opencl-clhpp-build.kOZ4QOvddt/build /tmp/opencl-clhpp-build.kOZ4QOvddt /home/tclxa/.cache/bazel/_bazel_tclxa/3fa575b1d05b4383ee5006a6a80abe38/sandbox/linux-sandbox/1/execroot/mace
-- The C compiler identification is GNU 4.8.5
-- The CXX compiler identification is GNU 4.8.5
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Configuring done
-- Generating done
-- Build files have been written to: /tmp/opencl-clhpp-build.kOZ4QOvddt/build
Scanning dependencies of target generate_clhpp
[100%] Rebuilding cl.hpp ...
[100%] Built target generate_clhpp
Scanning dependencies of target generate_cl2hpp
[100%] Rebuilding cl2.hpp ...
[100%] Built target generate_cl2hpp
/tmp/opencl-clhpp-build.kOZ4QOvddt /home/tclxa/.cache/bazel/_bazel_tclxa/3fa575b1d05b4383ee5006a6a80abe38/sandbox/linux-sandbox/1/execroot/mace
/home/tclxa/.cache/bazel/_bazel_tclxa/3fa575b1d05b4383ee5006a6a80abe38/sandbox/linux-sandbox/1/execroot/mace
installing to bazel-out/arm64-v8a-opt/genfiles/external/opencl_clhpp
Target //mace/libmace:libmace_dynamic up-to-date (nothing to build)
INFO: Elapsed time: 195.159s, Critical Path: 41.61s
INFO: 156 processes: 156 linux-sandbox.
INFO: Build completed successfully, 171 total actions
build shared lib for linux-x86-64
INFO: Build options have changed, discarding analysis cache.
INFO: Analysed target //mace/libmace:libmace_dynamic (1 packages loaded).
INFO: Found 1 target...

ERROR: /home/tclxa/mace/mace/ops/BUILD:49:1: C++ compilation of rule '//mace/ops:ops' failed (Exit 1): gcc failed: error executing command
  (cd /home/tclxa/.cache/bazel/_bazel_tclxa/3fa575b1d05b4383ee5006a6a80abe38/execroot/mace && \
  exec env - \
    PATH=/home/tclxa/.local/share/umake/bin:/home/tclxa/bin:/home/tclxa/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/soft/jdk1.8.0_181/bin:/home/tclxa/bin:/soft/android-studio/bin:/snap/bin:/home/tclxa/bin \
    PWD=/proc/self/cwd \
  /usr/bin/gcc -U_FORTIFY_SOURCE -fstack-protector -Wall -B/usr/bin -B/usr/bin -Wunused-but-set-parameter -Wno-free-nonheap-object -fno-omit-frame-pointer -g0 -O2 '-D_FORTIFY_SOURCE=1' -DNDEBUG -ffunction-sections -fdata-sections '-std=c++0x' -MD -MF bazel-out/k8-opt/bin/mace/ops/_objs/ops/mace/ops/infer_conv2d_shape.pic.d '-frandom-seed=bazel-out/k8-opt/bin/mace/ops/_objs/ops/mace/ops/infer_conv2d_shape.pic.o' -fPIC -iquote . -iquote bazel-out/k8-opt/genfiles -iquote external/bazel_tools -iquote bazel-out/k8-opt/genfiles/external/bazel_tools -iquote external/com_google_protobuf -iquote bazel-out/k8-opt/genfiles/external/com_google_protobuf -iquote external/gemmlowp -iquote bazel-out/k8-opt/genfiles/external/gemmlowp -iquote external/tflite -iquote bazel-out/k8-opt/genfiles/external/tflite -isystem external/com_google_protobuf/src -isystem bazel-out/k8-opt/genfiles/external/com_google_protobuf/src -isystem bazel-out/k8-opt/bin/external/com_google_protobuf/src '-std=c++11' -fPIC -D_GLIBCXX_USE_C99_MATH_TR1 -DMACE_OBFUSCATE_LITERALS -DGEMMLOWP_USE_OPENMP -DMACE_USE_NNLIB_CAF -O3 -ffunction-sections -fdata-sections -Werror -Wextra -fopenmp -fno-canonical-system-headers -Wno-builtin-macro-redefined '-D__DATE__="redacted"' '-D__TIMESTAMP__="redacted"' '-D__TIME__="redacted"' -c mace/ops/infer_conv2d_shape.cc -o bazel-out/k8-opt/bin/mace/ops/_objs/ops/mace/ops/infer_conv2d_shape.pic.o)

Use --sandbox_debug to see verbose messages from the sandbox
In file included from mace/ops/infer_conv2d_shape.cc:15:0:
./mace/ops/infer_conv2d_shape.h: In member function 'mace::MaceStatus mace::ops::InferConv2dShapeOp<D, T>::Run(mace::StatsFuture*)':
./mace/ops/infer_conv2d_shape.h:46:46: error: cannot call member function 'T mace::OperatorBase::GetOptionalArg(const string&, const T&) const [with T = int; std::string = std::basic_string<char>]' without object
             "padding", static_cast<int>(SAME)));
                                              ^
./mace/ops/infer_conv2d_shape.h: In instantiation of 'mace::MaceStatus mace::ops::InferConv2dShapeOp<D, T>::Run(mace::StatsFuture*) [with mace::DeviceType D = (mace::DeviceType)0u; T = int]':
mace/ops/infer_conv2d_shape.cc:46:1:   required from here
./mace/ops/infer_conv2d_shape.h:73:7: error: enumeration value 'VALID' not handled in switch [-Werror=switch]
       switch (padding_type) {
       ^
./mace/ops/infer_conv2d_shape.h:73:7: error: enumeration value 'SAME' not handled in switch [-Werror=switch]
./mace/ops/infer_conv2d_shape.h:73:7: error: enumeration value 'FULL' not handled in switch [-Werror=switch]
./mace/ops/infer_conv2d_shape.h: In instantiation of 'mace::MaceStatus mace::ops::InferConv2dShapeOp<D, T>::Run(mace::StatsFuture*) [with mace::DeviceType D = (mace::DeviceType)0u; T = float]':
mace/ops/infer_conv2d_shape.cc:46:1:   required from here
./mace/ops/infer_conv2d_shape.h:73:7: error: enumeration value 'VALID' not handled in switch [-Werror=switch]
./mace/ops/infer_conv2d_shape.h:73:7: error: enumeration value 'SAME' not handled in switch [-Werror=switch]
./mace/ops/infer_conv2d_shape.h:73:7: error: enumeration value 'FULL' not handled in switch [-Werror=switch]
cc1plus: all warnings being treated as errors
Target //mace/libmace:libmace_dynamic failed to build
INFO: Elapsed time: 71.247s, Critical Path: 19.97s
INFO: 243 processes: 243 linux-sandbox.
FAILED: Build did NOT complete successfully

when i  use command "bash tools/build-standalone-lib.sh" , it`s turns error. i donot know why , Thank for you answer

Please reformat the issue according to: https://github.com/XiaoMi/mace/blob/master/ISSUE_TEMPLATE.md


support gzip and zstd HTTP transport compression to fetch remote resources

Golang net/http already transparently supports gzip compression.
This adds support for zstd compression

/ok-to-test

Are there known public registries that support zstd already?

@kzys AWS uses zstd for fargate. Registry has a PR open for assing support for zstd


Develop an SAP Fiori App Using SAP Business Application Studio

Group URL: https://developers.sap.com/group.appstudio-fiori.html
It would have been better, if a simple UI5 application with BAS was shown without database connectivity.
One similar to https://github.com/SAP/openSAP-ui5-course/tree/ui51/w1u5/
After a non DB application, then show a DB connection app.

Hi,
We will look into this.
Thank you for the suggestion.
Regards,
Pao

Hi,
Thanks for the suggestion. We will put it on our roadmap for future tutorials.
Regards,
Pao

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


Keyboard Stopped working on Samsung S20 android 11 in flutter InAppWebViewPage

environment:
sdk: ">=2.2.0 <3.0.0"
dependencies:
flutter:
sdk: flutter
cupertino_icons: ^1.0.2
http: ^0.13.1
provider: ^5.0.0
toast: ^0.1.5
shared_preferences: ^2.0.5
hidden_drawer_menu:
path: ./hidden_drawer_menu-1.1.2
connectivity:
get_it:
back_button_interceptor: ^5.0.0
flutter_inappwebview: ^5.3.2
add_2_calendar: ^2.0.2
image_picker: ^0.7.4
image_cropper: ^1.4.0
cached_network_image: ^3.0.0
package_info: ^0.4.0+16
webview_flutter: ^3.0.1
dev_dependencies:
flutter_test:
sdk: flutter

Hi @Nooshey, Thanks for filing the issue. If I am understanding it correctly,  the keyboard doesn't work with in app webview built with flutter_inappwebview plugin is that correct? if so then that issue must be reported in that plugin's repository. And If the issue is  with webview_flutter plugin then please share a minimum and complete reproducible code sample along with the output of flutter doctor -v.


Fraction Calculator

Calculator Functions
Fraction Calculator.

Contribution Event
GSSOC22
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


Measuring time elapsed with RUM

Is your feature request related to a problem? Please describe.
I want to track the performance of different places around our application. I know there is a functionality on the RUM SDK to measure. I know you guys already provides some ways to that but I think any of those can fit our needs.

GlobalRum.get().addTiming("hero_image") determine the time between the start of the current RUM view and the time when you call this function. That works greatly if you are measuring anything relative to the start of the current view, and I want to set startTrack manually and call endTrack manually, so we can measure time it takes for a function, or a segment of code to execute.
AndroidTracer works greatly also, but I'm worried about how it calculates the pricing because as far as I know, every different host is increasing the price of APM. On the client-side, how is calculated when a trace comes from a client? I mean, every mobile device is treated like a different host? Also, those traces are not integrated with RUM "easily", I can't see those traces from AndroidTracer on the Trace window on RUM sessions.

Describe the solution you'd like
Something completely integrated with RUM and visible from the session view. Allow to set startTrack and endTrack without only relying on relative start of the current RUM view.
Describe alternatives you've considered
Using a custom Timer on our project and calculate the time elapsed between a defined startTrack and endTrack, then send those results using the Logger class of Datadog.

Hi @Albertoperezs90 and thanks for submitting this feature request.
Regarding the pricing I know that it's slightly different for mobile, but you might want to reach your CSM contact for an official quote on that.
Your understanding of the addTiming method is correct, it measures timings since the beginning of the current view. Using a logger with a custom timing is indeed the workaround I'd recommend for this feature.
We'll keep this idea in mind and see how we can expand our product to include this in the future.

Thanks @xgouchet, just a question, can we create monitors or dashboard using the solution with Logger? I mean, it's possible to generate percentiles based on the information of Logger and from that percentiles, check if any value is bigger than X?

Yes, indeed, using the Logs to Metrics feature, you can create a metric based on a log attribute, and create monitors on them. And logs can also be used in graphs in your custom dashboards.

Thanks @xgouchet! closing this issue, tho It should be nice to include this feature on the future!


Computing the difference vectors for PCA

In the original repeng paper they mostly described the unsupervised version of PCA, where they randomly paired the hidden state vectors and performed PCA on these pairs. This is fundamentally different from the supervised implementation used here, and after reading the code I want to know if I am misunderstanding what your implementation is doing or if there is potential to improve your PCA methodology.
In the contrastive approach, we assume the hidden state vectors belong to one of two sets, A or B. When doing unsupervised PCA, items from these sets are randomly paired together without regards to to their label, e.g. the possible pair types are (Ai, Aj), (Bi, Bj), (Ai, Bj), (Bi, Aj). For simplicity I'll call these AA, BB, AB, and BA. If we assume vectors cluster closely with other vectors from their set, then the difference vectors for

AA and BB pairs should be ~0
AB pairs should be ~X
BA pairs should be ~-X

where X is a vector that points from set B to set A. Doing PCA on these difference vectors should then give you ~X as your first principal component.
However, it appears that in your supervised method you only have pairs (Ai, Bj), or AB. Taking the difference of these vectors should give you vectors that all center around a point X, with very low variance. In theory when these points are projected onto any arbitrary line they will all project to the same point, meaning PCA should not work very well. If this is what you are doing then one explanation for why it still works is that not all hidden states actually encode the representations you are trying to extract, making this the primary source of variance PCA is picking up on.
This question came up before in the realm of word vectors: https://stackoverflow.com/questions/48019843/pca-on-word2vec-embeddings
The simple fix for supervised PCA is to first compute a center point for each pair: Ck = (Ai + Bj)/2. Then, your difference vectors become Ai - Ck and Bj - Ck, such that you get two opposing difference vectors for each pair. Doing PCA on all of these difference vectors should then work a lot better since the variance due to the representations should be a lot larger.

Fixed by #34


Get this error

01_log_analysis_setup_1 is up-to-date
Starting 01_log_analysis_es01_1 ... done
Starting 01_log_analysis_es02_1 ... done
Starting 01_log_analysis_es03_1 ... done

ERROR: for kibana  Container "7e6470a0a53a" is unhealthy.
ERROR: Encountered errors while bringing up the project.

I installed docker.io, docker-compose. Docker service is running.

If you run docker-compose -d from within the vagrant machine (via vagrant ssh), do you still get the error?

If you run docker-compose -d from within the vagrant machine (via vagrant ssh), do you still get the error?

Hello. I tried as you suggest. Steps:

vagrant up
vagrant ssh
Clone this repository inside vagrant machine
Go to 01 and do docker-compose up -d.
Error didn't show. However the SSH session disconnected (it's a little odd).

Connection to <IP_ADDRESS> closed by remote host.
Connection to <IP_ADDRESS> closed.

Version of docker-compose in vagrant: Version: 1.25.0-1
I guess Parrot's upgraded kernel version to 5.16 so maybe docker or docker-compose is not compatible with current kernel.
I think alternative solution could be start docker-compose inside vagrant machine, and do port forwarding

Yupp! So the vagrant file was made so students all have the same base environment. In the 00_Setup/README.md I go through how to tweak some settings, but by default there's a port forward from 5601 on the host to 5601 on the guest allowing access to Kibana.
If you're running this on your host machine, which appears to be Parrot OS, you'll have to change the value of vm.max_mem_count or you'll have issues with the Elastic containers. This is done in the Vagrant machine for you out of the box. See the Vagrant files provision commands for exact syntax on doing so.

I didn't see the vm.max_mem_count but i found this sudo echo 'vm.max_map_count=262144' >> /etc/sysctl.conf;.
I also found this command sysctl -w vm.max_map_count=262144 on stack overflow https://stackoverflow.com/a/64543430 (my bad. I should searched this first).
This problem is fixed on my side. I'm closing this issue. It would be awesome if you can add a note about this issue in 01 because it could be common issue.


add compound tag test for nesting InlineText and Link

@rbarral We don't have a HTML parser in this repository, we just have a parser for the example XML format we are using. So these two are the same thing from the point of view of the generator functions:
Panic
<custom-link>Panic ~~~~https://www.facebook.com/panicmadrid/</custom-link>
The source format should not make a difference because the library only sees the contents of the tags not the HTML/XML tags.
Perhaps your issue is to do with your tags mapping e.g. maybe your parser is not mapping  to InlineText generator as it should?

@srilq
i made strong tag work so that's fine, and i managed to transform html links to custom-link, that's fine aswell.
maybe we should ask cni for a html parser example we can use in production, dont know tbh.

@rbarral I think that’s a good idea! I can’t make any promises of course.
To be clear, though: I think the idea of an “example” here is that it demonstrates usage rather than provides a general-use implementation. A production-ready HTML parser would be a separate product rather than an example.

thanks for your comments @srilq , problem is that we need that production-ready html parser right now :(, i'll stick to the xml example as long as i can but probably we will have to move to some php markdown library in order to perform the final migration :(.

@rbarral Please do not use the example parser. I wrote the parser as a proof-of-concept for using this generator library for converting hierarchical formats, but it is not tested at all and is probably completely broken for any real use case.
This library is designed to be used in combination with a parser. It does not provide any functionality for parsing content, only generating Markdown.

ok @srilq, in that case i'll replace the cni library with another one once i'm done with everything else before doing migrations for real.

@rbarral I think there's maybe a misunderstanding: the library is tested and is designed to be used in production, but the example parser (which is not part of the library) is not tested and should not be used in production. Hope this makes sense.

@srilq as long as there is no html parser for that library, it is of no use to us unfortunalely :(.


How to open/run terminal

I visit the Store for the app and have the 'You own this app' but don't see it anywhere on my system and I'm unable to actually run it. Win-R terminal only offers cmd. I saw one post that said Win-R wt but nothing was found there. Perhaps I have a Path problem? ver says: Microsoft Windows [Version 10.0.17763.1039]

Maybe you haven't actually installed it. I think that the "You own this app" message on the store is shown when you have already installed it previously but you uninstalled it or you're on another device. This is confusing.

Install is grayed out.

Terminal is not supported on 10.0.17763.0.

Yep - updated and Install is enabled. Thanks!


Readme image

Readme image doesn't show intended content, instead it portrays a tinypic error message.

Thanks, fixed


Revert to using esbuild for E2E test packaging

We previously switched from esbuild to browserify for bundling our E2E tests for running in the test runner (see PR #164), due to an upcoming incompatibility between esbuild and cucumber (see https://github.com/badeball/cypress-cucumber-preprocessor/issues/929).
We now find that since moving to TypeScript 5, browserify is struggling with tests that reference code that uses decorators. As the esbuild/cucumber incompatibility has been resolved, we move back to using esbuild to avoid this issue.

 Revert to using esbuild for E2E test packaging


Add holidays for Catalonia

For information, read and make sure you're okay with the CONTRIBUTING document.

[x] Tests with a significant number of years to be tested for your calendar.
[x] Docstrings for the Calendar class and specific methods.
[x] Calendar country / label added to the README.rst file,
[x] Changelog amended with a mention like: "Added <country> by @pseudo (#)"

the tests look great; I've checked the checkbox. You'd have to at least edit the README and the Changelog and check all the checkboxes when you're ready.

Ok done!
Changed README and CHANGELOG and removed comented code.

ah. almost done. I've digged the travis build and spotted an issue: https://travis-ci.org/novafloss/workalendar/jobs/148501567 if you look at this, you see there's no test_europe is executed, which is sad, because the coverage report doesn't seem to see it.
I think I know why, you have an ImportError. This import error has been introduced by the chmod you committed on various files: workalendar/europe/__init__.pyworkalendar/europe/__init__.py, workalendar/europe/spain.py, and also especially workalendar/tests/test_europe.py. I tried launch the tests from your branch using the previous standard chmod (644) and it worked again. Could you please change the access rights to the given files in order to have all the modules tested accordingly? thx.

for the record, it looks like it's a well-known bugfeature in nose: http://stackoverflow.com/questions/2378146/why-does-nose-finds-tests-in-files-with-only-644-permission

@ferranp do you need some help?

Ok, file permissoions restored.
Sorry for this, I work in a mixded Windows/linux environment and sometimes the permissions get changed without noticing.

@ferranp no problem, it happens (and BTW, I'd have never thought it had caused such an issue...)


Cron-server PoW signal monitor Bot

A Discord bot to monitor the PoW signal sent from the Lilypad cron server. The bot will monitor the Arbiscan address 0xd10d15cc705f7d2558352b1212a9b3685155d93d looking for method 0xb681f2fd. If this method is not transacted on this address within 1 hour send a warning message to the Lilypad Discord (ask the team which channel to use).
Info:
Testing

Test the bot using a test Discord server or ask the Lilypad team for access to a channel in the Lilypad Discord.

Implementation

The Lilypad team will provide the Discord webhook and Arbiscan API URLs as Github secrets in this repo. Let the team know which ENV variables you intend to use so they can be added.

Address on Arbitrum Sepolia

https://sepolia.arbiscan.io/address/0xd10d15cc705f7d2558352b1212a9b3685155d93d

@galaxyxtwo, this one ca be assigned to me

Implemented by: https://github.com/Lilypad-Tech/community-bots/pull/8


Use separate thread pools for input and output in Netty

Use separate thread pools for input and output in Netty as recommended in the Netty documentation.

Part of #7968


src: release Node.js version 8.10.0-cr

This commit updates the node to 8.10.0 candidate release version.
I'm was not sure if this PR should be opened against the 8.x branch or not. Feel free to close and open against the correct (perhaps a new branch) or let me know and I'll do that.

@danbev not sure what would be best for this. Maybe a new branch 8.next? I don't want a candidate release to be what you get if you pull 8.x from docker hub

@lance A branch named like that sounds good to me. I'll need to update this with the environment variable for node-gyp (that is why I marked it as in progress). Might not get around to that until tomorrow. I an create a branch and reopen this PR against it.


Flesh out URLError information upon URLSessionTask cancellation

The URLError received upon URLSessionTask cancellation by a URLSessionTaskDelegate or completion handler has a blank userInfo dictionary. This is an inconsistency when compared with macOS's Foundation and it makes actual error handling harder than it could be.
This PR fills it out to match the fields filled out by macOS Foundation.
I previously reported this as https://bugs.swift.org/browse/SR-13017, and I believe this fixes that (if a fix is even desirable, that is.)

There are 6 other URLError constructors in the FoundationNetworking project, and their userInfo situation is quite inconsistent. Would it be desirable to change that?

The testcase is ok, I ran it under DarwinCompatibilityTests and it works correctly against Darwin.
However the test failed on both Linux and macOS. Note that currently the TestURLSession tests are disabled and need to be re-enabled in the Tests/Foundation/main.swift file. URLSession still has some other issues that makes the tests generally flaky.
Note in this case it looks like request?.url  was nil so the failingURL and failureURLString entries werent added.

I thought it ran in the TestFoundation target of the Xcode project, but maybe it didn't. (I should have tried to crash the test to make sure!) Building on Linux failed yesterday, and didn't have enough time to fix that so I crossed my fingers and submitted it. I'll try again.

@spevans I haven't had time to solve my Linux build, but now I've gotten the test to properly run from Xcode.

I think we should fix the URLError.failingURL and .failureURLString properties that your original tests exposed. The following seems to fix it in my tests:
diff --git a/Sources/Foundation/NSError.swift b/Sources/Foundation/NSError.swift
index 0b55deea..ff71279e 100644
--- a/Sources/Foundation/NSError.swift
+++ b/Sources/Foundation/NSError.swift
@@ -919,12 +919,12 @@ extension URLError {

     /// The URL which caused a load to fail.
     public var failingURL: URL? {
-        return _nsUserInfo[NSURLErrorFailingURLErrorKey._bridgeToObjectiveC()] as? URL
+        return _nsUserInfo[NSURLErrorFailingURLErrorKey] as? URL
     }

     /// The string for the URL which caused a load to fail.
     public var failureURLString: String? {
-        return _nsUserInfo[NSURLErrorFailingURLStringErrorKey._bridgeToObjectiveC()] as? String
+        return _nsUserInfo[NSURLErrorFailingURLStringErrorKey] as? String
     }
 }

I wasn't sure what _bridgeToObjectiveC() was supposed to do, so I didn't go that route. Also: should it be in a different PR?
This being said, there are other things with URLError: none of the properties added in iOS 13 are on it yet. Obviously they're for either darwin-specific or not-yet-supported things, but shouldn't they be there, returning nil?

Yes I think a separate PR to fix up the properties in URLError would be a good start. I think the
private var _nsUserInfo: [AnyHashable : Any] {
        return _nsError.userInfo
    }

can be changed to a [String: Any] and then the properties fixed up.
It just requires adding some tests constructing URLError and then testing the properties and validating the tests in the DarwinCompatibilityTests project which will run the tests against the native Foundation. It would be good to add in the missing properties as well.

@swift-ci test

I just tested it locally on Linux with the TestURLSession tests enabled and it passed ok.

@swift-ci test

Thanks!


Close PRs opened in sandbox, and notify requestor of this action

A small number of PRs, such as #10, have been opened in the Sandbox repo. Rather than simply leaving them open indefinitely we should

Add a bot to auto-comment on new PRs in the sandbox, informing the creator that the PR will be closed in future without being merged
Add a bot to auto-close stale PRs in the sandbox

This was taken care of in #13. Closing this issue now.


It still showing test ads even in live

Still showing Test ads even my app in live, i removed isTesting line
please help me thank you

leave the line and replace true with false
isTesting = false;

Same issue

Duplicate to https://github.com/ratson/cordova-plugin-admob-free/issues/69


Implement Essential matrix extraction from Matches Features (We need the Camera calibration matrix before!)

Implemented in https://github.com/makra89/Visual-Odometry-Cpp/pull/10


Question on vector index segment load store

Hi, i am having a question regarding the constraints on illegal instruction of vector index seg load store.
Let's say the setting for vtype is e16 m4, and the instruction is vluxseg2ei32.v. For my understanding, under this configuration, the instruction is legal. Because for index load store, I think the "nfemul <=8 " constraint only applys for the data vector, which in this case emul=lmul=4  and nf is 2.  But if taking index vector into account, the emul would be 8 and the nfemul will come to 16. I am not sure if I am right on this that this instruction is legal.
Hope my question is clear.

You are correct; NF only factors into the legality of the data vector.

Another way to think about it is that there is only one index per segment.


Npm istall issue with version 0.10.x

Hello,
Whilst specifying version "db-migrate": "0.10.0-beta.4" in the package.json file then calling npm install it seems skips over installing node-db-migrate. After the install is complete all the packages have been installed apart from db-migrate. If I specify the version to be 0.9.x this issue doesn't occur. This is slightly problematic if you want to use the module's API options and run it via a .js file. Any idea what might be going on?
Package.json content - https://gist.github.com/Siron/44ec86475c7f15085a53

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/28053316-npm-istall-issue-with-version-0-10-x?utm_campaign=plugin&utm_content=tracker%2F73887&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F73887&utm_medium=issues&utm_source=github).

I need more information from you:

node version used
npm version used

However, I have tried with many npm versions and node versions and all worked. How do you come to the conclusion it was skipped?

Information:

Node 0.10.29
Npm - 2.11.3

Here are the steps I used

rm -rf node_module/
npm install
ls -l node_modules

Everything is listed apart from db-migrate. If I then call npm update && npm install, then it gets installed. But I always have to call npm update first.
If I change the version to 0.9.x for db-migrate then everything is fine with just npm install. This happens across develop, integration and all other environments where jenkins handles the builds.

First of all, you might consider updating node (0.10.40), as there are
probably many security issues with your version, apart from you're using
a quite old one.
However, I've tested with the exact same details and your package json,
no problems here.
ls -l node_modules/
drwxr-xr-x 4 tobi users 4,0K  7. Nov 14:08 body-parser/
drwxr-xr-x 6 tobi users 4,0K  7. Nov 14:08 bower/
drwxr-xr-x 4 tobi users 4,0K  7. Nov 14:08 coffee-script/
drwxr-xr-x 4 tobi users 4,0K  7. Nov 14:08 compass/
drwxr-xr-x 3 tobi users 4,0K  7. Nov 14:08 compression/
drwxr-xr-x 5 tobi users 4,0K  7. Nov 14:08 db-migrate/
drwxr-xr-x 6 tobi users 4,0K  7. Nov 14:08 ejs/
drwxr-xr-x 4 tobi users 4,0K  7. Nov 14:08 express/
drwxr-xr-x 5 tobi users 4,0K  7. Nov 14:08 glob/
drwxr-xr-x 3 tobi users 4,0K  7. Nov 14:08 method-override/
drwxr-xr-x 3 tobi users 4,0K  7. Nov 14:08 morgan/
drwxr-xr-x 4 tobi users 4,0K  7. Nov 14:08 request/
drwxr-xr-x 3 tobi users 4,0K  7. Nov 14:08 serve-favicon/
drwxr-xr-x 7 tobi users 4,0K  7. Nov 14:08 snapsearch-client-nodejs/

Am 07.11.2015 um 14:04 schrieb Siron:

Information:

Node 0.10.29
Npm - 2.11.3

Here are the steps I used

rm -rf node_module/
npm install
ls -l node_modules

Everything is listed apart from db-migrate. If I then call npm update
&& npm install, then it gets installed. But I always have to call npm
update first.
If I change the version to 0.9.x for db-migrate then everything is
fine with just npm install. This happens across develop, integration
and all other environments where jenkins handles the builds.
—
Reply to this email directly or view it on GitHub
https://github.com/db-migrate/node-db-migrate/issues/326#issuecomment-154702236.

It was a local build issue. Sorry about that. All fixed now

@Siron Great to hear! Do not hesitate to ask again if you encounter any issues.


Refine the contributing doc to more readable and easy to get started

as title
Fixed #14
Signed-off-by: acezen<EMAIL_ADDRESS>

Maybe we can add a paragraph to introduce how to contribute to documentation


Chart editor beat snaping doesn't work

Describe your bug here.

seems pretty straight forward, the beat snap doesn't do anything even with 16th or 192nd
Crash Log or logcat logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua and Source
Have you open preload state?
Yes
What is your build target?
Desktop (Linux, Windows or Mac)
Did you edit anything in this build? If so, mention or summarize your changes.
I changed language.hx and titlestate.hx

psych already removed this function,use zoom in/out


helm.Options.Logger option is ignored when building dependencies

I am trying to silence the helm modules log output during templating like so:
opts := helm.Options{
    SetValues: map[string]string{/* values go here */},
    Logger:    logger.Discard,
}
helm.RenderTemplate(t, &opts, "./path/to/chart", "test", []string{"templates/templatefile.yaml"})

However, while this eliminates the log output from the template command, the dependency build command still outputs via terratests logger:
TestDeploymentAnnotations 2023-04-24T17:16:37+02:00 logger.go:66: Running command helm with args [dependency build ../../]
TestDeploymentAnnotations 2023-04-24T17:16:38+02:00 logger.go:66: Hang tight while we grab the latest from your chart repositories...
TestDeploymentAnnotations 2023-04-24T17:16:38+02:00 logger.go:66: ...Successfully got an update from the "ingress-nginx" chart repository
TestDeploymentAnnotations 2023-04-24T17:16:38+02:00 logger.go:66: ...Successfully got an update from the "prometheus-community" chart repository
TestDeploymentAnnotations 2023-04-24T17:16:39+02:00 logger.go:66: ...Successfully got an update from the "bitnami" chart repository
TestDeploymentAnnotations 2023-04-24T17:16:39+02:00 logger.go:66: Update Complete. ⎈Happy Helming!⎈
TestDeploymentAnnotations 2023-04-24T17:16:39+02:00 logger.go:66: Saving 1 charts
TestDeploymentAnnotations 2023-04-24T17:16:39+02:00 logger.go:66: Downloading common from repo https://charts.bitnami.com/bitnami
TestDeploymentAnnotations 2023-04-24T17:16:39+02:00 logger.go:66: Deleting outdated charts
TestClientTlsCertSecretAnnotations 2023-04-24T17:16:39+02:00 logger.go:66: Running command helm with args [dependency build ../../]
TestClientTlsCertSecretAnnotations 2023-04-24T17:16:39+02:00 logger.go:66: Hang tight while we grab the latest from your chart repositories...
TestClientTlsCertSecretAnnotations 2023-04-24T17:16:40+02:00 logger.go:66: ...Successfully got an update from the "ingress-nginx" chart repository
TestClientTlsCertSecretAnnotations 2023-04-24T17:16:40+02:00 logger.go:66: ...Successfully got an update from the "prometheus-community" chart repository
TestClientTlsCertSecretAnnotations 2023-04-24T17:16:40+02:00 logger.go:66: ...Successfully got an update from the "bitnami" chart repository
TestClientTlsCertSecretAnnotations 2023-04-24T17:16:40+02:00 logger.go:66: Update Complete. ⎈Happy Helming!⎈
...

I believe that the reason for this is the line below - terratest seems to pass an empty options struct reference when it should probably transfer the Logger option from the options struct passed to the method?
https://github.com/gruntwork-io/terratest/blob/e0c790c2147f13219c8df62ff2f5035232c392ae/modules/helm/template.go#L38
For some context: I am currently building a large set of templating tests for a helm chart and terratests log output can make it difficult to determine which tests failed at a glance, as one needs to scroll through quite a lot of console output.
To make test failures more visible, I'm trying to silence terratest logging when running go test without -v.

Fix released in https://github.com/gruntwork-io/terratest/releases/tag/v0.41.23


[3.x] SpriteFramesEditor Add animation searchbox

(~will do master version after this is approved~) master version: #49488
Closes https://github.com/godotengine/godot-proposals/issues/2681. Made searchbox toggleable so it won't take any vertical space if not needed.

before
after

Do I need to add added strings somewhere (for translations)?

Do I need to add added strings somewhere (for translations)?

No, as this is handled automatically when Akien synchronizes translations.

Changes done, now it by default looks like this:

Oops, not the whole "Filter animations" placeholder was visible. Should be fixed now:

"Filter Animations" should be capitalized as per #63175.

"Filter Animations" should be capitalized as per #63175.

Seems like #61488 wasn't backported though so I don't think I should change that. In 3.x it's still "Filter nodes" etc.


android dialog when permission is blocked

Bug report

Summary
when call
let result = await check(PERMISSIONS.ANDROID.READ_EXTERNAL_STORAGE);
and result is RESULTS.BLOCKED , android dialog is opened.
Environment info
react-native info output:
info Fetching system and libraries information...
System:
OS: Linux 5.4 Ubuntu 18.04.5 LTS (Bionic Beaver)
CPU: (4) x64 Intel(R) Core(TM) i5 CPU       M 480  @ 2.67GHz
Memory: 272.98 MB / 7.63 GB
Shell: 4.4.20 - /bin/bash
Binaries:
Node: 12.18.3 - /usr/local/bin/node
Yarn: Not Found
npm: 6.14.8 - /usr/local/bin/npm
Watchman: 4.9.0 - /home/linuxbrew/.linuxbrew/bin/watchman
SDKs:
Android SDK:
API Levels: 23, 28, 29
Build Tools: 28.0.3, 29.0.2
System Images: android-23 | Intel x86 Atom_64, android-23 | Google APIs Intel x86 Atom_64
Android NDK: Not Found
IDEs:
Android Studio: Not Found
Languages:
Java: 1.8.0_272 - /usr/bin/javac
Python: 2.7.17 - /usr/bin/python
npmPackages:
@react-native-community/cli: ^4.12.0 => 4.12.0
react: 16.13.1 => 16.13.1
react-native: ^0.63.2 => 0.63.2
npmGlobalPackages:
react-native: Not Found
Library version: 2.2.2
Describe what you expected to happen:
I want to show my dialog that have a go to settings button!

@samaneh-kamalian First, hello. Could you add more context?

It seems that everything is normal here, if it returns RESULTS.BLOCKED, uses an Alert (https://reactnative.dev/docs/alert) + openSettings() to display a dialog to open app settings.

Thanks for your quick answer.
tIt works well but to coordinate with the rest of the project, I want to use the dialogue I created myself but android default dialogue is opened and my dialogue is not shown.
Is there any way?

Please provide more code than just a single line.

    async requestStoragePermission() {

        let result = await check(PERMISSIONS.ANDROID.READ_EXTERNAL_STORAGE);
        if (result === RESULTS.GRANTED)
            return true;
        else if (result === RESULTS.BLOCKED) {
            this.setState({
                dialogVisible: true,
                dialogTitle: ApplicationTexts.UiTexts.changePermissionTitle,
                dialogContent: ApplicationTexts.UiTexts.changePermissionContent,
                actionOfFirstBtnOfDialog: this.inVisibleDialog,
                textOfFirstBtnOfDialog: ApplicationTexts.UiTexts.changePermissionAction_noAction,
                appearSecondBtnForDialog: true,
                actionOfSecondBtnOfDialog: this.goToSettingsPage,
                textOfSecondBtnOfDialog: ApplicationTexts.UiTexts.changePermissionAction_goToSettingsPage
            });
        }
        else if (result === RESULTS.DENIED) {
            let granted = await request(PERMISSIONS.ANDROID.READ_EXTERNAL_STORAGE);
            if (granted === RESULTS.GRANTED)
                return true;
            else
                return false;

        }

    }

I guess something like this will work:

async function requestStoragePermission() {
  const permission = PERMISSIONS.ANDROID.READ_EXTERNAL_STORAGE;
  const result = await check(permission);

  if (result === RESULTS.GRANTED) {
    return true;
  }
  if (result === RESULTS.UNAVAILABLE) {
    return false;
  }
  if (result === RESULTS.DENIED) {
    return (await request(permission)) === RESULTS.GRANTED;
  }
  if (result === RESULTS.BLOCKED) {
    return new Promise((resolve) => {
      Alert.alert(
        'Open settings',
        'In order to grant permission, which is super needed, please open settings and…',
        [
          {
            text: 'Cancel',
            onPress: () => console.log('Cancel Pressed'),
            style: 'cancel',
          },
          {text: 'Open settings', onPress: () => resolve(openSettings())},
        ],
        {cancelable: false},
      );
    });
  }
}

Perfect. thanks a lot.


⚠ Bofrost.de has degraded performance

In 57079c0, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1008 ms

Resolved: Bofrost.de performance has improved in e5b7c2e.


Convert a matrix to its sparse matrix form

add in array folder

Hey @akshitagupta15june, I would like to contribute to this issue, can you please assign it to me?

I would like to contribute too


support multiple key servers for token and sign

The old API still works but marked deprecated already. When you call JwtHelper.verifyJwt(), the third parameter is a boolean to indicate if the JWT is an OAuth 2.0 access token or not.

 support multiple key servers for token and sign


Upload has been blocked due to rate limiting. Please try again later.

It seems like the Sandbox is throwing this error when uploading, even if the change size is well below the maximum. I'm dealing with the same problem right now in my own project https://codeberg.org/NeatNit/osm-bus-stops-to-sandbox
I suspect this is thrown because of the file size of the uploaded OsmChange. So if you split the changeset over, say, 20 different uploads it might work. I'll try this soon in my own project, but it will take a while because it turns out I have to fix an unrelated problem first.

This might be different between users. Found this: https://github.com/openstreetmap/openstreetmap-website/blob/master/app/controllers/api_controller.rb#L201
  def check_rate_limit(new_changes = 1)
    max_changes = ActiveRecord::Base.connection.select_value(
      "SELECT api_rate_limit($1)", "api_rate_limit", [current_user.id]
    )

    raise OSM::APIRateLimitExceeded if new_changes > max_changes
  end

I'm still not sure I'm looking at the right thing, but if so, it seems to have a different limit per user.


Upmerge december

Zephyr 2.1.0

child of https://github.com/NordicPlayground/fw-nrfconnect-nrf/pull/1647

@mbolivar : Done. Thank you for the detailed explanation.

@mbolivar : Done. Thank you for the detailed explanation.

Thanks! As discussed elsewhere, I've got a rebase-and-then-cleanup branch started here:
https://github.com/mbolivar/fw-nrfconnect-zephyr/tree/mergeup-249-rebase-rewrite
I've squashed the CI commits, various multi-image patches, and various socket.h and net_ip.h commits together and reordered the results to ease reading. Our diff from upstream is smaller now and should be easier to deal with going forward.
We can upstream three of these patches here, here, and here.
There are also three temphacks we can revert entirely or revert/rewrite in a cleaner way using zephyr_linker_sources().

@SebastianBoe @hakonfam It is impossible to make DFU. When DFU server request to erase image collection bank, the primary bank is erased instead of secondary. This is something inherited from master, but I think better to fix it within upmerge patch series.
(Reproduced using smp_svr zephyr sample)

Verified local DFU (dfu candidate flashed to device) for mcuboot and app upgrade. Also verified dfu for http_application_update sample.

@nvlsianpu : Please re-asses your review.

Some checks failed. Please fix and resubmit.
Identity/Emails issues
149950abadca2684ec209e8948ce8034bc74ac18: author email (Sebastian Bøe<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
dfaaa1caa2a1f97693a1d6b6ba4715ff23ef5a08: author email (Sebastian Bøe<EMAIL_ADDRESS>needs to match one of the signed-off-by entries.
Tip: The bot edits this comment instead of posting a new one, so you can check the comment's history to see earlier messages.


Slim down the Docker images

This changes to a multi-stage build, and To run the application, we only need package.json, yarn.lock, build.js, node_modules and src/
Changes done

ensure what's in the lockfile is installed, keeps dependencies correct
build the necessary files
remove devDependencies
copy files from build stage to runtime stage
starts the server directly, brings down startup time significantly

Reduces the docker image with 193MB

@electerious another thing I might open a PR on, depends if you're interested, is a hardened Docker image. Kinda like I've done here https://github.com/omBratteng/social.hashweb.org/blob/develop/Dockerfile#21
Uses a distroless base image, to reduce possible attack surface

Pull Request Test Coverage Report for Build 741

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 85.526%

Totals

Change from base Build 740:
0.0%

Covered Lines:
756

Relevant Lines:
840

💛  - Coveralls

Seeing as Ackee itself, awaits for being able to connect to the MongoDB, is the wait script necessary?

Thanks for taking a closer look at the dockerfile!

ensure what's in the lockfile is installed, keeps dependencies correct
remove devDependencies

👍

build the necessary files
copy files from build stage to runtime stage
starts the server directly, brings down startup time significantly

I guess this won't work. The build uses environment variables which can be changed by the user. E.g. ACKEE_DEMO and NODE_ENV are effecting the build process.

Seeing as Ackee itself, awaits for being able to connect to the MongoDB, is the wait script necessary?

The connection function is async, but it doesn't wait and throws an error when MongoDB isn't available.

The connection function is async, but it doesn't wait and throws an error when MongoDB isn't available.

Well, not Ackee, but mongoose, has a 30s timeout on trying to connect.
https://docs.mongodb.com/manual/reference/connection-string/#urioption.serverSelectionTimeoutMS
Starting server without a mongodb
$ yarn server
yarn run v1.22.10
$ node src/index.js
[Ackee] › …  awaiting  Connecting to mongodb://localhost:27017/ackee
[Ackee] › ✖  fatal     MongooseServerSelectionError: connect ECONNREFUSED <IP_ADDRESS>:27017
...

Starting docker without mongodb
Creating ackee ... done
Attaching to ackee
ackee    | Docker-compose-wait starting with configuration:
ackee    | ------------------------------------------------
ackee    |  - Hosts to be waiting for: [mongo:27017]
ackee    |  - Timeout before failure: 30 seconds
ackee    |  - Sleeping time before checking for hosts availability: 0 seconds
ackee    |  - Sleeping time once all hosts are available: 0 seconds
ackee    | ------------------------------------------------
ackee    | Checking availability of mongo:27017
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Host mongo:27017 not yet available
ackee    | Timeout! After 30 seconds some hosts are still not reachable

The only benefit of wait in docker, is if starting the mongodb instance takes longer than 30 seconds

I guess this won't work. The build uses environment variables which can be changed by the user. E.g. ACKEE_DEMO and NODE_ENV are effecting the build process.
That is true.

Could change CMD to be CMD /wait && yarn start. That way it will, by default, run the build process.
But add documentation, that you can run tell Docker to run yarn server if you don't need a custom build.
Or wise versa, leave it as-is. And add documentation that you can tell docker to run CMD /wait && yarn start.
That way, it is already pre-built if you want a fast start. But you can still build it.

Well, not Ackee, but mongoose, has a 30s timeout on trying to connect.
https://docs.mongodb.com/manual/reference/connection-string/#urioption.serverSelectionTimeoutMS

That's cool. I wasn't aware that there's a timeout. We should change the connectTimeoutMS option of Mongoose to 60000 to ensure that MongoDb is available. You can remove the wait script.

Could change CMD to be CMD /wait && yarn start. That way it will, by default, run the build process.
But add documentation, that you can run tell Docker to run yarn server if you don't need a custom build.
Or wise versa, leave it as-is. And add documentation that you can tell docker to run CMD /wait && yarn start.
That way, it is already pre-built if you want a fast start. But you can still build it.

I know that it's not perfect to wait until it's compiled, but I want to keep it simple. Users should be able to start the image without accidentally getting old files.
An automatic alternative: We could check if ACKEE_DEMO is true, NODE_ENV is development or ACKEE_TRACKER is set and compile the files when any of those cases is true. They're the only env variables affecting the build.

Well, not Ackee, but mongoose, has a 30s timeout on trying to connect.
docs.mongodb.com/manual/reference/connection-string/#urioption.serverSelectionTimeoutMS

That's cool. I wasn't aware that there's a timeout. We should change the connectTimeoutMS option of Mongoose to 60000 to ensure that MongoDb is available. You can remove the wait script.

I have bumped connectTimeoutMS to be 60000, and removed wait script

Could change CMD to be CMD /wait && yarn start. That way it will, by default, run the build process.
But add documentation, that you can run tell Docker to run yarn server if you don't need a custom build.
Or wise versa, leave it as-is. And add documentation that you can tell docker to run CMD /wait && yarn start.
That way, it is already pre-built if you want a fast start. But you can still build it.

I know that it's not perfect to wait until it's compiled, but I want to keep it simple. Users should be able to start the image without accidentally getting old files.

Reverted it back to run the start command, but still keep the earlier build stage. So if a user wants, they can tell Docker to run it with yarn server.

An automatic alternative: We could check if ACKEE_DEMO is true, NODE_ENV is development or ACKEE_TRACKER is set and compile the files when any of those cases is true. They're the only env variables affecting the build.

I actually was thinking about that, by using an entrypoint file, that checks for the env variables, and then determines if it should run build or start it directly. But that could be in another PR

Perfect 🙌 I've learned a few new things about Docker thanks to this PR.
I've also realised that we can install dependencies with yarn install --production --frozen-lockfile directly, because the build tools are part of the normal dependencies (I know that's not perfect, but it's because npm start builds the files first). Currently implementing those changes.

The Docker image is now around 58% smaller 😎


Bonfires not saving after "Repeat a String repeat a string"

I noticed my Bonfires have stopped saving my work after Basic Algorithms #9
Challenge http://freecodecamp.com/challenges/bonfire-repeat-a-string-repeat-a-string has an issue. Please describe how to reproduce it, and include links to screenshots if possible.

Closing as duplicate of #2894
Please see https://github.com/FreeCodeCamp/FreeCodeCamp/wiki/Searching-for-Your-Issue-on-Github
Happy Coding!


[BUG]The scheduled refresh feature of the query is not working.

Currently, the sharing page of Redash cannot refresh to load the latest data. It is hoped that this function can be added. Polling can be carried out when the page is refreshed or loaded.
Or the shared page may have data refresh settings.

Or a feature could be provided to disable cache, so that the data is reloaded every time the user opens the page.

I am currently refreshing data in batches using the method below, but it is very slow.
payload = {
  "parameters": {
    "DataSource": "us-one",
    "site": "us"
  },
  "max_age": 1
}
header = {
  "Authorization": "Key bF4R7JEgNU97J9YYl6dRLuBmERlhWbQuymGdNRXf",
  "content-type": "application/json"
}

def refresh(payload,header):
    import requests
    try:
        sql = """select max(id) from queries"""
        count = execute_query("redash",sql)
        # print(count)
        vlaue = count['rows'][0]['max']
        print(vlaue)
        for id in range(vlaue):
            print(id)
            url = "https://redash.amiemall.com/api/queries/"+str(id)+"/results"
            print("start")
            response = requests.request("POST", url, json=payload, headers=header)
            print("end")
            print(response.text)

    except:
        print('err2')
    rlt = {"columns": [{"name": "uname","type": "string","friendly_name": "uname"}],"rows": [{"uname": "test"}]}
    return rlt

result = refresh(payload,header)

In fact, I am currently using the following Python code to periodically refresh data through Redash API.
refer to：
import requests

def refresh(url,payload,header):
  try:
    # url = "http://<IP_ADDRESS>:5500/api/queries/1/results"
    print("start")
    response = requests.request("POST", url, json=payload, headers=header)
    print("end")
    print(response.text)
    return response.text
  except:
    print('err')
    return 'err'

if __name__ == '__main__':
  # url = "https://redash.amiemall.com/api/queries/47/results"
  for site,country in [("mall","ghana"),("mall","ghanab2b"),("mall","viet nam b2b")]:
    payload = {
      "parameters": {
        "country": country,
        "site": site
      },
      "max_age": 1
    }
    header = {
      "Authorization": "Key bF4R7JEgNU97J9YYl6dRLuBmERlhWbQuymGdNRX0",
      "content-type": "application/json"
    }
    for id in range(100):
      print(id)
      url = "http://<IP_ADDRESS>:5500/api/queries/"+str(id)+"/results"
      refresh(url,payload,header)
      print("Call completed.")

The same to #31

I don't know #61 maybe help you!

I don't know #61 maybe help you!

Thank you, I will wait for the current docker image issue to be resolved, and then I will give it a try.

I don't know #61 maybe help you!

Due to certain network reasons (GFW), the Docker Hub is now banned. This makes it inconvenient for me to test Docker images. Can you take a look at this issue?

Oh wow.    If Docker Hub is permanently banned in China, that's going to be a problem for us (and others).
We can probably set up a mirror of our images somewhere.  I have a few dedicated servers online with unmetered bandwidth (eg no download cost) for another project, so we could probably use one of those.  It won't be a today thing though.

@gaecoli Due to certain network reasons (GFW), the Docker Hub is now banned. This makes it inconvenient for me to test Docker images. Can you take a look at this issue?

You can use proxy.

@gaecoli
This hasn't completely solved my problem.

@gaecoli This hasn't completely solved my problem.

Maybe in query refresh schedule has problem.


pass templates/adapters info to the server

It seems PR #1911 accidentally removes the connection between
the server and those info. E2E tests start failing because of
this.
What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

none

/assign @geeknoid

@jmuk: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
e9558d3d06d98c5bd7d2de334c5355bd2280db57
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Hi @jmuk , the tests seems to be passing on the PR under question and the other PRs submitted after that. Is it the e2e post submit that is failing ?

LGTM

/lgtm
/approve

/retest


Multi Camera issue on Android 8.1.0

Required Info

Camera Model
{ D435 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
Android  8.1.0

Kernel Version (Linux Only)
Android 8.1.0

Platform
Red Hydrogen One/Snapdragon 835  / etc..

SDK Version
{220 }

Language
Java+JNI

Segment
{Smartphone/AR }

When I create a device like this and try to get the serial number from it.
The  function always return the same one.
MainActivity.java
.......
for( int i=0; i< mRsContext.queryDevices().getDeviceCount();i++) {
            mPipeline[i] = new Pipeline();
            mConfig  = new Config();

            device=devices.createDevice(i);
            String serialnumber = device.getInfo(CameraInfo.SERIAL_NUMBER);
           mConfig.enable_device(serialnumber);

//               mConfig.enable_device(devicenmae[i]);
mConfig.enableStream(StreamType.DEPTH, 640, 480);
mConfig.enableStream(StreamType.COLOR, 640, 480);
            mPipeline[i].start(mConfig);

..................
config.c
JNIEXPORT void JNICALL
Java_com_intel_realsense_librealsense_Config_nEnableDevice(JNIEnv *env, jclass type_, jlong handle,
jstring serialnumber) {
rs2_error *e = NULL;
const char *nativeString = (*env)->GetStringUTFChars(env, serialnumber, 0);
rs2_config_enable_device((rs2_config *) handle, nativeString, &e);
(*env)->ReleaseStringUTFChars(env, serialnumber, nativeString);
handle_error(env, e);
}
I tried to use the serial number which is printed on bottom of the camera. The first camera works fine. But, the second camera doesn't start.

@David-168 The latest SDK v2.31 has multi camera example for Android. https://github.com/IntelRealSense/librealsense/tree/master/wrappers/android/examples/multicam
Please try to see if same issue on this. Thanks!

@David-168 Any update from your side? Thanks!

@David-168 Sorry that we didn't get response from you for weeks. Will close this issue at this point. Please feel free to create new one if you still have questions or issues. Thanks!


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: sarahyeahh sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Add bootlint 0.16.8 and 1.0.0

@jmervine can you upload this (including the /latest endpoint and ping me?

@XhmikosR Done

0.16.8 -> 0.16.8
1.0.0 -> 1.0.0
1.0.0 -> latest

Perfect, thanks!
CDN purged for latest.


Testing for 0.5.0 release

Hi folks
We've released https://github.com/Day8/re-frame/releases/tag/v0.5.0-alpha1. Try it out in your apps and let us know what you think. There are two changes that need particular attention:

Dynamic subscriptions - https://github.com/Day8/re-frame/pull/108. Try them out and let us know if the API feels right for this. I'm pretty sure that we don't have any memory leaks in our implementation of dynamic subscriptions, but let us know if something seems funky. More docs for this will be forthcoming
Switching from (core.async/timeout 0) to (yield) in the router dispatch loop - https://github.com/Day8/re-frame/pull/121. We need to give control back to the browser while we're processing for it to do Browser Things. We previously used (timeout 0) to do this. The time taken for the  (timeout 0) channel to close is browser specific, but in general takes at least 4 ms. This meant we couldn't process more than 250 events/second. The new yield implementation, helpfully shared by https://github.com/pkobrien is much faster. Again, we think this is implemented correctly, but let us know if you see anything funky.

Thanks!

summary: re: (yield) change: I see a nice improvement in my app's performance, but perhaps we can get an even better balance between the needs of the browser and the needs of the router.
I'm writing an app that uses svg to allow the user to draw items onto a drawing surface. Currently I have one entry in the db when a new item is being created by a mouse drag and a separate entry for the list of items that already exist. In the "mouse up" handler for a creation drag, I dispatch two re-frame events: :end-drag, and :create-item. Before the yield change this gave a significant visual glitch as the drawing surface was redrawn on each yield: first without the new item (after :end-drag was handled) and then again with the new item in the existing items list (after :create-item was handled).
In this case, the effect of the yielding to the browser before handling each event was a visual flicker ("there" "not there" "there") during item creation. The duration of "not there" is greatly reduced in my setup after the change to use (yield) instead of waiting for (timeout 0). Sweet!
It does appear though that even with the new faster yield, software down the stack from re-frame (reagent/react/chrome) is rendering my svg every time the router loop yields. That seems like too much deference to UI handling over router event loop handling.
I see in the comments that the purpose of the yield to make sure the browser gets some time in case the handler hogs the cpu for a long time. I suspect many handlers are extremely fast (manipulations of the db atom) and it seems to me we should try to handle as many as we can in some reasonably small time slice rather than yielding before every event.
I tried this modification to the router yielding algorithm: when there is a new event to handle, check if it's been at least 5 ms since we most recently returned from a yield. If yes, yield now and start a new 5 ms timer when the yield returns. If no, handle the event without yielding first.
This removed the visual glitch in my case and I think makes a better tradeoff between the needs of the browser and the needs of the router. Fast handlers will execute much faster on average and even if a slow handler slips in and starts at just under 5 ms after a yield returned, the effect on the responsiveness of the browser will be worse by at most 5 ms.
Here's a sketch of some code that's working for me locally:

(defn yield-chan
  [flush? yield-time now]
  (cond flush?
        (do (flush) (timeout 20))
        (>= now yield-time)
        (yield)))

(defn router-loop
  []
  (go-loop [yield-time 0]
    (let [event-v (<! event-chan)
          yield-time (if-let [ch (yield-chan (:flush-dom (meta event-v))
                                             yield-time
                                             (system-time))]
                       (do (<! ch)
                           (+ (system-time) 5))
                       yield-time)]
      (try
        (handle event-v)
        (catch js/Object e
          (do
            (purge-chan)
            (router-loop)
            (throw e))))
      (recur yield-time))))

@scgilardi I've been thinking about this and have an approach that we want to try involving running handler functions immediately. Do you have a public example of your SVG app we can use as a test case? Don't worry if you don't have it available, it's not critical.

I updated and my app works as usual. It's a drop-in upgrade so far. Good stuff!

@danielcompton Thanks! I extracted a public example out here: https://github.com/scgilardi/re-frame-svg-demo . It lets you draw rectangles and demonstrates the flicker I'm talking about.

Updated my app and didn't have any issues. Good work! I will try to play with dynamic subscriptions in the next few days and see how it goes.

No issues as well. Just less noise in the JS console! Thanks :)

@scgilardi there's a new version of the router loop in develop.
I've dumped using core.async completely. The new strategy is to process ALL events in the queue at once without yielding.  But to not process any NEW events, created in this cycle. They will be done next time.  So there is no yielding at all when processing current events.
Obviously this is experimental but I have high expectations.
It is not deployed to clojars yet (next couple of days).  In the meantime, to use it, you'll have to:

checkout develop
lein install to put it your local maven repo
- change your deps to be for [re-frame "0.5.0-alpha2"]

[re-frame "0.5.0-alpha2"]  is now in clojars.   Could you test it please?
There's a new event processing (router) loop. I replaced use of core.async with a hand-rolled solution. As a result:

events will be handled more quickly after their dispatch (<1ms vs 5ms??)
groups of events queued up will be handled in a batch, one after the other, without yielding to the browser (previously re-frame yielded to the browser before handling every single event)

I doubt this will affect normal apps. But it could affect games which depend on existing timings. Maybe.  It could affect apps which dispatch large volumes of events (telemetry?) very quickly. Maybe.
 We can now think in terms of a packet of events being handled without yield. I updated my demo at https://github.com/scgilardi/re-frame-svg-demo and it's beautifully flicker-free after being made compatible with the new router policy.
Nicely done!
I have a few thoughts based on reviewing the code:

I think this portion of the state machine is buggy: [:do-paused :add-event] -> [:paused #(-add-event this arg1)]. Except when the current state is :quiescent, adding an event should not change the state. In particular with the current code, adding an event while processing the event that previously caused a pause will result in a subsequent illegal state transition [:paused :done-paused]. The buggy portion should be changed to [:do-paused :add-event] -> [:do-paused #(-add-event this arg1)] .

I suggest a couple of name changes:

":do-paused" -> ":resuming" for easier understanding. I think it's a general improvement, but also by being more distinct from :paused, it may have helped avoid the bug above.
":done-paused" -> ":finish-resuming" (for consistency with :finish-running)

It seems unfortunate that the entire queue is flushed on any exception. I can see the reasoning and it would certainly be a good policy for a re-frame app never to throw out of a handler to avoid the issue entirely.

I wonder if at some point a notion like "a packet of events" could be made more concrete and only the members of the packet could be discarded on an exception. This scenario comes to mind: I handle event A by generating events B, C, and D and the new router will execute all of those without pause in the next router time slice. But into the mix comes event W which is unrelated to processing A--maybe it's a reminder whose time has arrived. With the current router, if W ends up at the end of the same router time slice as B, C, and D and C throws, D will not be processed (probably good), but also W will not be processed (unfortunate).
it might be useful to be able to mark B, C, and D as being derived from processing events in the previous router time slice and only flush marked events on an exception in one of them.

unmarked events (perhaps events that were queued from outside a handler) would still be processed.

It's probably more complexity than it's worth to try to add this kind of nuance to how exceptions are handled by the router, but I thought I'd mention it in case someone can think of an elegant way to throw away only the bathwater in a case like this.

@scgilardi  Thanks!   I greatly appreciate that thoughtful review.
You are 100% right about the bug.  Thanks!   Also, I've adopted your name changes.  Good suggestions.  I'll release an alpha3.
Regarding the handling of events post an exception, it seems to me there is no strategy which will work generally, so sticking with something utterly simple seems right.  For example, what happens if W  (from a websocket) gets interleaved with your C D.  Mind you, I have a selfish angle here in not looking too hard for solutions.  In our apps, when an Exception is thrown,  we do an "undo" to put app-db  back into a known-good-state and try to let the user continue on.  So clearing events works in well with that.

@mike-thompson-day8 you're welcome! I'm glad it was helpful.
Another quick one: The condp = in the -fsm-trigger body can be replaced with case with the benefit of constant time, hash-based dispatch. (I'm not sure it's faster in a way that matters, but it is an ideal use-case for `case

see alpha3  about to appear in clojars

v0.5.0 now released.


type error when using LiveImage

BlueSky [45]: testplan = Count([bpmAD], 5, 1)
BlueSky [46]: cb = LiveImage('bpmAD_image_array_data')
BlueSky [47]: gs.RE(testplan, cb)
/home/xf05id1/.conda/envs/collection/lib/python3.4/site-packages/bluesky/run_engine.py:1279: UserWarning: A TypeError("unhashable type: 'numpy.ndarray'",) was raised during the processing of a event Document. The error will be ignored to avoid interrupting data collection. To investigate, set RunEngine.ignore_callback_exceptions = False and run again.
  "and run again." % (exc, name.name))

This error is fixed.
The correct field name to use is 'bpmAD_image' which has the correct shape.
please close this issue.


ServiceAccountCredentials need way to modify UserAgent after serialization

We accept a user_agent in [init] (https://github.com/google/oauth2client/blob/master/oauth2client/service_account.py#L40) but that doesn't cover deserialization via the Credentials base class from_json method.
Presently to allow for caching, one must need to inherit from the private _ServiceAccountCredentials and hard-code user agent and scopes into to_json / from_json methods.

Agreed. We should add factory overloads to from_stream, but also provide a way to clone the credentials with a changed user agent, so that it can be used with the Default Credentials also. Tracking this bug as part of completing Usable Auth.

Thank you for creating this issue, however, this project is deprecatedand we will only be addressing critical security issues. You can read moreabout this deprecation here.
If you need support or help using this library, we recommend that you ask yourquestion on StackOverflow.
If you still think this issue is relevant and should be addressed, pleasecomment and let us know!


Embedded images are not displayed

Hey!
I tend to embed images into my Markdown documents. This has been shown to work well in the past as I could easily move my documents between different markdown applications, without having to care about asset files and file/http references.
However this does not work at Pico CMS.
Example
Document:
2020-05-05-brettspiele-kartenspiele-offline.md
PicoCMS

ruby-kramdown / jekyll

Content

Pandoc

Pico output html
img src / ![img]() was stripped

Version info
I have the PicoCMS that is bundled with NextCloud. CHANGELOG.md in the picocms folder mentions 2.1.4 on top, so guess this is the release version used.

Hey @gsantner,
I just downloaded your example file and gave it a test.  It worked fine for me:

I'm not really sure what the issue is here, but it definitely should be working.  I tested it using the standalone release of Pico (it's what I already had deployed for testing).  I'm not sure what differences there are, if any between the standard and Nextcloud versions of Pico though.  It could also be a difference somewhere in the rest of your stack (like the PHP version for instance), but unfortunately that's all a bit outside of my expertise.
@PhrozenByte, do you have any idea what could cause this issue?

This is caused by Pico CMS for Nextcloud: data URLs are removed by HTMLPurifier for security reasons. However, I'm not aware of any security implications for images (except SVGs maybe), so we probably can lower HTMLPurifier's defaults here. Will look into this and create a new release of Pico CMS for Nextcloud then.

Should I recreate the issue in a nextcloud pico repo? But guess you have the ability to move the issue anyway as well

Fixed in https://github.com/nextcloud/cms_pico/commit/eb2609af65cdc2c294aebae8b8162ef234e76112, you can give it a try. However, please note that you can't use WebP images. HTMLPurifier unfortunately can't validate images of this type, you'll have to use PNG, JPEG or GIF instead. You might wanna ask ezyang/htmlpurifier whether they want to implement this.

well this is bad, because 99% of my markdown documents have webp embedded (for size). Not a single png/jpg

thanks for quick fix.
though bad to hear as it not fixes it for my use. because 99% of my markdown documents have webp embedded (for size). Not a single png/jpg

You might wanna switch to Pico (w/o Nextcloud), no limitations there :smiley:

+1 from me on that approach too.
I used to accomplish this a while back by simply simlinking my Pico folders from Nextcloud. If I'm remembering right, the original files had to live in Nextcloud because it would either ignore or overwrite symlinks. I wasn't using files_external though, Nextcloud was completely unaware I was doing anything special.
But this was hands-down the best setup I ever had for deploying pages!
I could save a file locally and have the changes almost instantly reflected on my server.
I'm assuming that Pico for Nextcloud offers a similar experience (I've never used it), but doing it manually worked great for me for years.
My current git-based approach also leaves a lot to be desired, so I hear you there.
(Unfortunately for me though, I'm not using Nextcloud at the moment.)
But yeah, I'd definitely recommend this solution if it sounds like it fits your use case.

Me neither (using nextcloud yet), so far on all ends issues and bugs getting into the way.
I'm evaluating it currently, and privacy issues here, password leaks there, no automatic sync here for android there...the list goes on and on ^^

Ah, my bad, I just assumed that if you started with the Nextcloud version of Pico, you were probably already using it. 😅
There's plenty of file syncing solutions out there though.  If the automatic syncing appeals to you, you could try something lighter like Syncthing.  Whatever solution you decide on, just sync your content and assets folders with it and you should be good (or maybe just content in your case).  Since you can't generally run code from these folders, it ends up being a decent balance of convenience and security on the Pico side of things. 😉
Anyway, good luck with your search and let us know if there are any other questions we could answer for you. 😁

I use synthing + jekyll currently :D

As said, may be better to recreate the bug report on nextcloud repository? Looks not like there anything  Pico developers can do about the issue :D

As said, may be better to recreate the bug report on nextcloud repository? Looks not like there anything Pico developers can do about the issue :D

Doesn't make much of a difference whether I answer you here or there :laughing:
We can't do anything about this. Pico CMS for Nextcloud is designed to allow arbitrary users to create websites, thus we can't allow them to use any active content (like JavaScript), otherwise we would create a whole lot of security issues. For this reason we use HTMLPurifier in Pico CMS for Nextcloud, that removes any potentially insecure contents. I'm pretty sure that embedding a WebP image is no security issue at all, but HTMLPurifier simply doesn't know this rather new format - and thus removes it. As I said, you can head over to the developers of HTMLPurifier and ask them whether they add WebP support - if they do, Pico CMS for Nextcloud will support it, too.
Anyway, from your comments I assume that you don't actually want arbitrary users of your Nextcloud to create their own websites, you just want to use this for yourself, right? In this case you won't need Pico CMS for Nextcloud, using the app has no significant advantage for you. The only thing you have to do is to sync Pico's content and assets dir with your Nextcloud. You can either use Nextcloud's files_external app, or, as explained by Maya, symlink the folders. This solves all your issues, embedding WebP images will work just fine then. Just give it a try :+1:

@PhrozenByte
Thanks for help, I've reported the issue on htmlpurifier repo. And if I see correctly it's just one line of additional change required to add webp to a array.
Is there a release out at NextCloud-pico already with your change included? If so, I would test it with just that one line of change 😄


Custom Title Page: extending the default page throws Name Error

I need a custom title page to align the formatting to a company standard.
I followed the guide at https://docs.asciidoctor.org/pdf-converter/latest/extend/.
I am not so familiar with Ruby and Prawn. However, I wanted to modify the default title page.
My idea was to use the ink_title_page method from https://github.com/asciidoctor/asciidoctor-pdf/blob/main/lib/asciidoctor/pdf/converter.rb#L3708. I copied it over to the ExtendedPDFConverter and called asciidoctor with it:
 asciidoctor-pdf -v -a compress  --trace -r ./pdf-converter/extended-pdf-converter.rb -r asciidoctor-lists -o main.pdf  "main.adoc"

in the output I get the following error:
pdf-converter/extended-pdf-converter.rb:10:in `ink_title_page': uninitialized constant ExtendedPDFConverter::AttributeList (NameError)
        from C:/Ruby30-x64/lib/ruby/gems/3.0.0/gems/asciidoctor-pdf-2.3.2/lib/asciidoctor/pdf/converter.rb:179:in `block in convert_document'
        from C:/Ruby30-x64/lib/ruby/gems/3.0.0/gems/asciidoctor-pdf-2.3.2/lib/asciidoctor/pdf/ext/prawn/extensions.rb:1025:in `perform_on_single_page'
        from C:/Ruby30-x64/lib/ruby/gems/3.0.0/gems/asciidoctor-pdf-2.3.2/lib/asciidoctor/pdf/converter.rb:179:in `convert_document'
        from C:/Ruby30-x64/lib/ruby/gems/3.0.0/gems/asciidoctor-pdf-2.3.2/lib/asciidoctor/pdf/converter.rb:152:in `convert'
        from C:/Ruby30-x64/lib/ruby/gems/3.0.0/gems/asciidoctor-2.0.17/lib/asciidoctor/document.rb:955:in `convert'
        from C:/Ruby30-x64/lib/ruby/gems/3.0.0/gems/asciidoctor-2.0.17/lib/asciidoctor/convert.rb:118:in `convert'
        from C:/Ruby30-x64/lib/ruby/gems/3.0.0/gems/asciidoctor-2.0.17/lib/asciidoctor/convert.rb:190:in `block in convert_file'
        from C:/Ruby30-x64/lib/ruby/gems/3.0.0/gems/asciidoctor-2.0.17/lib/asciidoctor/convert.rb:190:in `open'
        from C:/Ruby30-x64/lib/ruby/gems/3.0.0/gems/asciidoctor-2.0.17/lib/asciidoctor/convert.rb:190:in `convert_file'
        from C:/Ruby30-x64/lib/ruby/gems/3.0.0/gems/asciidoctor-2.0.17/lib/asciidoctor/cli/invoker.rb:129:in `block in invoke!'
        from C:/Ruby30-x64/lib/ruby/gems/3.0.0/gems/asciidoctor-2.0.17/lib/asciidoctor/cli/invoker.rb:112:in `each'
        from C:/Ruby30-x64/lib/ruby/gems/3.0.0/gems/asciidoctor-2.0.17/lib/asciidoctor/cli/invoker.rb:112:in `invoke!'
        from C:/Ruby30-x64/lib/ruby/gems/3.0.0/gems/asciidoctor-pdf-2.3.2/bin/asciidoctor-pdf:46:in `<top (required)>'
        from C:/Ruby30-x64/bin/asciidoctor-pdf:25:in `load'
        from C:/Ruby30-x64/bin/asciidoctor-pdf:25:in `<main>'

on line 10 of extended-pdf-converter.rb:
          logo_image_attrs = (AttributeList.new $2).parse %w(alt width height)

Was it the correct way to start with the default title page? How can I fix it.
Many Thanks

You usually can't just copy code out of one converter and into another without some modification. In this case, since your class is not inside the Asciidoctor module, it cannot find AttributeList. You would need to replace it with ::Asciidoctor::AttributeList.
In the future, please direct usage questions to the project chat at https://chat.asciidoctor.org. Thanks!


Support steelseries prime wireless mouse

I just got the steelseries prime wireless mouse and would like to use this library to query their battery level. How can I help you find the magic numbers and add support for them?

how i can find produt id steelseries prime wireless by my slelf

Mice won't be supported. Sorry.


feat: MvPolynomial.X_ne_zero

Proves that a variable is not the zero polynomial, analogous to Polynomial.X_ne_zero.

Is there any reason this shouldn't be simp? The single-variable version is not, but it seems useful if it's allowed.

Is there any reason this shouldn't be simp? The single-variable version is not, but it seems useful if it's allowed.

How can simp use a ≠ lemma?

Is there any reason this shouldn't be simp? The single-variable version is not, but it seems useful if it's allowed.

How can simp use a ≠ lemma?

Well, it can of course close the goal X  ≠ 0...

maintainer merge

Can you add the simp attribute also to one variable case? In another PR it's fine.


Error output loading e.g. 3ds meshes

Due to the merge of #1317 (subsequently changed in #1352), I am now seeing the following error for my 3ds meshes when using rviz 1.13.3 in ROS Melodic on Ubuntu 18.04:
[ERROR] [1554754289.703535903, 1.974491951]: /rviz: XML parse error [package://my_package/meshes/fancy_mesh.3ds]: XML_ERROR_PARSING_TEXT
Reason: loadMeshFromResource calls meshFromAssimpScene for 3ds files, and then subsequently getMeshUnitRescale is called even if the file is not a Collada XML file.

I have the same error for OBJ mesh files.

Addressed in #1365.


document group

for #1343
TODO:

[ ] this is already very long (too long)
[ ] include comments on using the sort{ fy: "x", limit: 10} input option.
[ ] reducer scope? (as in transforms/bin)
[ ] fix types for the {reduce()} function (& create test plots with an explicit scope)

I've just added an incomplete commit that shows an underlying type bug with ChannelReducers. Not sure how to solve this.

The incomplete test is in test/plots/penguin-island-mode.ts; it fails because our types aren't certain that the reduce function is called with two arguments

That’s only true in this branch because you changed the definition of ReducerImplementation. In the main branch TypeScript seems to infer that it’s a ReducerImplementation:

But, I do think there is ambiguity and likely a bug here introduced by #1310 (to fix #1309). Take this code:
Plot.groupX({y: "count", fill: {reduce: f}}, {x: "species", fill: "island"})

In this case, it’s hard to tell whether the fill output channel is being specified as

a reducer with a scale override, in which case f is a ReducerFunction (or a ReducerImplementation object)
a reducer implementation, in which case f is a ReducerImplementation["reduce"]

Currently we assume (1)
https://github.com/observablehq/plot/blob/2ac08e266d1dd9743870d2c1ebbaf56ceb992524/src/transforms/group.js#L181
and if you want a reducer implementation, you would need another level of nesting to get (2):
Plot.groupX({y: "count", fill: {reduce: {reduce: f}}}, {x: "species", fill: "island"})

but this should be considered a bug since we didn’t intend to change the interpretation. Hmm, tricky! I suppose we could look at the length of the reduce function to disambiguate but that could be brittle.

I wonder if we should make the map/reducer implementations much less ambiguous by using a symbol, e.g., {[Plot.reduce]: …} instead of {reduce: …} to declare an implementation. That would be possible like so:
export declare const reduce: unique symbol;

export interface ReducerImplementation {
  [reduce](index: number[], values: any[]): any;
}

It wouldn’t be backwards compatible, though… 🤔

Yes, please don’t make the label change in this documentation PR. It’s more subtle than it looks and is now delaying adding the documentation.

Got it—I'm reverting all this. I was following my comment above, but it took me too far :)
Also this seems to work, but I wouldn't know how to express it as an interface:
export type ReducerImplementation =
  | {
      reduce(index: number[], values: any[]): any;
    }
  | {
      reduce(index: number[], values: any[], summary?: any): any;
      scope: "data" | "facet";
    };

export type BinReducerImplementation =
  | {
      reduce(index: number[], values: any[], extent: {x1: any; y1: any; x2: any; y2: any}): any;
    }
  | {
      reduce(index: number[], values: any[], summary?: any, extent?: {x1: any; y1: any; x2: any; y2: any}): any;
      scope: "data" | "facet";
    };


Backport workaround for Intel HPC platform version

Intel HPC platform version number is changed, as shown in this commit https://github.com/aws/aws-parallelcluster-cookbook/commit/afce2a687fada887e57784c497b8eecf12d564f6
Workaround is required in order to use Intel HPC in all versions <=2.7.0

Signed-off-by: Rex<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Linter (and thus Travis status check) failures are expected. No need to backport the commit that pins isort in my opinion.


SkParagraph regresses time to first frame on iOS

Starting with the Engine roll here https://github.com/flutter/flutter/commit/55e8a2cd5bc319f9aa9f28f7e3e68ec7672d4a04 containing only the SkParagraph flag flip, there is a big regression in time to first frame, and first frame build times on iOS.
https://flutter-flutter-perf.skia.org/e/?begin=1643075719&end=1643149114&keys=Xf98240cb6f1160c361037486d85f96fa&num_commits=50&request_type=1&xbaroffset=27306
On chat @jason-simmons wrote:

SkShaper's font loader may be a major factor in the high cost of the initial text layout (https://github.com/google/skia/blob/main/modules/skshaper/src/SkShaper_harfbuzz.cpp#L266)
SkShaper's create_hb_face is calling SkTypeface::openStream, and on iOS it appears that the openStream implementation is making copies of all of the font's tables.
On Android it looks like the font data is being loaded on demand during calls to other HarfBuzz APIs used by create_hb_face.

Maybe related: https://github.com/flutter/flutter/issues/94419

The SkShaper_harfbuzz create_hb_face function has another code path that calls hb_face_create_for_tables with a callback that loads font tables on demand.
This is similar to how Libtxt operates and avoids copying the complete font asset when loading fonts.
https://github.com/flutter/flutter/issues/69566 has some previous discussion about this.  In particular, loading the entire font can noticeably increase memory usage for large fonts such as emoji.
@bungeman @Rusino Would it make sense for SkShaper to provide a flag that Flutter could set on iOS that would opt into use of hb_face_create_for_tables?

No, a flag would be wrong. I don't have time to work on this right now, but we understand what needs to be done.

@jason-simmons @bungeman Is there a Skia issue we should cross-reference here, or is this github issue sufficient to track this?

@heathmill: Is there anything we can do to elevate the priority of this issue? This regression may be severe enough to reconsider flipping the switch for the next stable release.

I submitted the fix yesterday (Feb, 7) to Skia main branch. It should be in Flutter now.

The Skia change has rolled through to the framework, but the benchmark has not recovered. Is there follow-up work needed in the Engine to use the new API? Should we re-assign this to @jason-simmons?

No, it should work without any changes.

To clarify, it looks like the memory benchmarks have recovered:
https://flutter-flutter-perf.skia.org/e/?begin=1644284719&end=1644314715&keys=X5844b97c0a37edd15278ccad36a0a878&num_commits=50&request_type=1&xbaroffset=27492
But the startup regression has not recovered:
https://flutter-flutter-perf.skia.org/e/?begin=1642718442&end=1644344757&keys=Xf98240cb6f1160c361037486d85f96fa&requestType=0&xbaroffset=27306

So, by not using all that extra memory we didn't save any time?

I've been unable to reproduce the performance issue.
I've tried various benchmarks (complex_layout_ios__start_up, flutter_gallery_ios__start_up, flutter_gallery_ios__transition_perf) but the metrics have been consistent between SkParagraph and Libtxt on the iPhone 7 that I have.
The devicelab is using an iPhone 6s - possibly this issue is device related?
The dashboard is not showing any issues for the Android benchmarks, so presumably this involves something specific to the iOS platform.  I can confirm that the SkTypeface_Mac::onOpenStream calls are no longer happening on the latest head.

@godofredoc Do we have any newer iPhones in the devicelab?

No yet, we are planning to replace the 6s with iphone 11s but we haven't started creating the replacement plan.

So, what are we going to do with it? I don't have iPhone 6 available for testing, and I don't think that we should tune up our software for old devices...

Just to clarify, one of the selling points of Flutter is support for older devices, which is important for customers targeting NBU. That's why we have older devices in the lab. Data I could find is a little out of date, but suggests that iPhone 6s is still ~5% of users, which sounds small, but in absolute terms that's millions of people.
That's why I'm interested in at least determining the cause of the regression. @chinmaygarde do you or @bdero have an iPhone 6s handy, or know if we have one in the device library?

I don't have a 6s on hand, but I went ahead and ordered one. I can repro/debug it on ~Feb 16th if we haven't figured it out by then.

@bdero cool. It'll be good to have one on hand. @jmagman has one as well, and mentioned that she'd take a stab at reproing.

 However it does look like there is still a ~10% regression in timeToFirstFrameMicros in the runs from @jmagman.
@jason-simmons Would the next step be to collect before/after timeline traces?

Closing this in favor of tracking progress on performance in https://github.com/flutter/flutter/issues/94419.

Let me know if you need anything else from that iPhone 6S.


Help with refactor ESP32 support

Hello @donRaphaco, I'm removing the libatomic from the extensions repositories and adding it to rcutils package, check this: https://github.com/micro-ROS/micro_ros_setup/pull/171/checks?check_run_id=953530520
But in the case of the ESP32 I'm not able to remove the folder components/libatomics and these two lines without having building errors.
Is possible for you to remove this libatomic dependency for the ESP32 board support? I'm working in this PR: https://github.com/micro-ROS/freertos_apps/pull/18 You can commit there if you wish.
Thanks!

It should be fixed with 5c87578


[SPARK-19336][ML][Pyspark]: LinearSVC Python API

What changes were proposed in this pull request?
Add Python API for the newly added LinearSVC algorithm.
How was this patch tested?
Add new doc string test.

cc @hhbyyh Thanks!

Test build #71945 has started for PR 16694 at commit abafaeb.

Test build #71945 has finished for PR 16694 at commit abafaeb.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71945/
Test FAILed.

Test build #71946 has started for PR 16694 at commit 98bd7e7.

Test build #71946 has finished for PR 16694 at commit 98bd7e7.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71946/
Test FAILed.

Test build #71948 has started for PR 16694 at commit 2980e67.

Test build #71948 has finished for PR 16694 at commit 2980e67.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71948/
Test PASSed.

Test build #72080 has started for PR 16694 at commit e2e9943.

Test build #72080 has finished for PR 16694 at commit e2e9943.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72080/
Test PASSed.

Merged build finished. Test PASSed.

LGTM, thank you!
Merging with master


Deprecated package gitbeaker/node

When installing there is a warning given by npm:
npm WARN deprecated<EMAIL_ADDRESS>Please use its successor @gitbeaker/rest

Thanks for reminding, I'll try to migrate these days, or PR welcome even better.


feat(blog): add Camunda Modeler v3.3.0 release blog post

Looking forward to your feedback. ❤
Related to camunda/camunda-modeler#1480

Overall, it looks good to me. I left one comment and IMO with that slight change it can be merged.

Should we mention that copy and paste has been completely reworked (not only interaction, but internals, too).
From our point of view it is now completely fixed.

We are proud to announce the release of Camunda Modeler v3.3.0. It comes with a couple of significant improvements to the BPMN modeling experience as well as many bug fixes.

Are we that vague all the time? If I read a blog post I usually like to get teasered (i.e. new copy and paste interaction / navigation / everything).

Should we mention that copy and paste has been completely reworked (not only interaction, but internals, too).
From our point of view it is now completely fixed.

Well, do we want people to know about how broken it was before?

Not looking back, but looking into the future, yes. Those who know about the current limitations may see that as a major advantage.

Regarding the past => "fix existing limitations" sounds much better than everything was lost :see_no_evil:

Your choice :wink:

"Ricarding the limitations of copy & paste, it's all good now."

Thank you for your feedback. I've made a few adjustments. Let's merge. 🔥

Have we decided whether the title will be Camunda Modeler v3.3.0 or v3.3? @nikku

Yes, we decided for 3.3

Updated the title and the release date (today, 4 PM).