Swedish string updates

Here are a couple of updates that I got from a Swedish reviewer by email.
@iah Do these look OK to you?

Line 87 is wrong. Is should be "rensas" instead of "rensa".
Line 115 is removed?
The best,
iah

Thanks for the quick feedback @iah. I've changed rensa to rensas in line 87.
Regarding link 115, it's still there - just a trailing space was removed automatically by my text editor.

@fgei This is an update to the Swedish PR you merged the other day, so should be ready to go.

Thanks!

And thank you for the quick merge again!


Prevent user to deselect tag once it is selected.

How can I stop deselect of tag once it is selected.

Got solution.


fix: Remove expandable "products using" package in "Package details" page

https://issues.redhat.com/browse/TC-1051
I've seen that the "backstraces" field was never implemented and always return an empty array. As a consequence we always have an empty expandable section in the "package details => products using package" table.
This is the line where the backstraces are set https://github.com/trustification/trustification/blob/main/spog/api/src/service/guac.rs#L321
backtraces: vec![]

So it makes sense to me just to disable the expandable section for all rows as we don't expect to see data there.

Do we really want to remove the dependency information altogether?

No, this change should only remove the expandable area of the table. The expanded area never shows data as we currently don't generate data for it in the backend.
Before:

After:

Yes, I understand that. And I agree, if that field will always be an empty array, then it makes sense to remove this.
However, my question is more of: does it make sense to just abandon the idea of having those backtraces? Wouldn't it make more sense, if feasible, to provide them? Which might be some that @dejanb could answer.
If we decide we give up on those, then yes: let's remove it.

As far as I can see this query, FindDependentProduct does not return backtraces (@mrizzi correct me if I'm wrong). I think, as most of the current SBOMs are flat and we don't want to invest too much time in the current implementation, we should remove it.

As far as I can see this query, FindDependentProduct does not return backtraces (@mrizzi correct me if I'm wrong). I think, as most of the current SBOMs are flat and we don't want to invest too much time in the current implementation, we should remove it.

@dejanb you're right: if guac-rs doesn't enhance data with further requests towards guac, the invoked findDependentProduct GraphQL endpoint returns an array of SBOM [1]
[1] https://github.com/trustification/guac/blob/v0.3.29/pkg/assembler/graphql/schema/search.graphql#L67

@ctron @dejanb @mrizzi It seems we all agree that the expanded area the the table/UI should be removed as we don't expect data generated by the backend right?
Could you approve this PR please?


New App: Checkiday

Description
The world's most trusted holiday website is now on Tidbyt! Included are thousands of holidays, which are timezone-award and you can enable NSFW holidays (off by default) if you wish.
Copilot

🤖 Generated by Copilot at 836bc76
Summary
🎉🌎🕒

This pull request adds a new applet called Checkiday, which displays a list of today's holidays from the Checkiday API on the Tidbyt device. The pull request includes the main code file checkiday.star and the manifest file manifest.yaml for the applet. The applet has some configuration options for the user, such as NSFW mode and location.

Checkiday applet
Shows today's holidays
On the Tidbyt screen

Walkthrough

Create and configure a new applet to display today's holidays from the Checkiday API (link, link)

I have read the CLA Document and I hereby sign the CLA


Add Okta server selection

Add support for selecting Okta region, closes #48.

You are still excluding the Okta preview accounts with this change... if you are serious about making any change to support this properly you have to consider the preview accounts, and accounts that can operate under a custom URL, which would be a URL not using any of the three known Okta domains.

@peter-murray Thanks for the feedback. I haven't really interacted too much with Okta, I didn't know about these other options. I'm happy to review this change and take these into consideration.

Works well as expected.
As peter said, this is not the ultimate solution but ... the beginning to solve the okta domains problem.
From my side I prefer to merge this PR instead of "wait the golden solution"

Yeah, totally agree the prompt is pretty crap. I had literally just picked Go up when I wrote this and did the first thing that came to mind. I wasn't really wanting to mess with this too much. I'll try to write up something with a bit of a cleaner solution. I'm using this modification currently, it works but is less than optimal.

Pretty sure this is superseded by #92.


[SPARK-44325][SQL] Use PartitionEvaluator API in SortMergeJoinExec

What changes were proposed in this pull request?

SQL operator SortMergeJoinExec updated to use the PartitionEvaluator API to do execution.
Why are the changes needed?

To avoid the use of lambda during distributed execution.
Ref: SPARK-43061 for more details.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Updated 1 test case, once all the SQL operators are migrated, the flag spark.sql.execution.useTaskEvaluator will be enabled by default to avoid running the tests with and without this TaskEvaluator

CC @cloud-fan @viirya @dongjoon-hyun @yaooqinn @beliefer @HyukjinKwon

@viirya I think the benefit is that, we make it more clear what gets serialized and sent to the executor side.

LGTM+1

thanks @vinodkc and @cloud-fan @viirya @beliefer. merged to master


Regenerate Icons?

This is a super helpful project. The current icon set is a bit out of date.
Is it possible for you to regenerate the icons with the latest version of the SVGs? It would be great if your could share the script used to generate the files so others can regenerate them themselves.

I managed to regenerate the black images myself. I'm on a Mac and this one liner worked for me:
cd /path/to/clone/of/github.com/primer/octicons
for svg in *.svg; do rsvg-convert -h 300 $svg > /path/to/output/target/$(echo $svg | sed -e 's/\.svg/.png/'); done

Inspired by this superuser answer.

Hey @skwashd ! I was offline from my github notifications during the summer in between roles. I appreciate you calling out the out-of-dateness.
If you want, I'll happily merge a PR with the work you've done. Otherwise, I can get to this in a day or two.

@webdog No worries. I got the black ones I needed and moved on. I didn't bother trying to generate the white ones. All I have is the one liner I shared above.

@skwashd Sounds good. I'll update the icons here shortly. Thanks again 😺


Font licensing restrictions

It looks like the Alverata font has to be directly licensed for a fee (or you access via Adobe Creative Cloud). We should be clearer that if third parties wish to use the Alverata font they will need a direct license to cover their use (especially if in a commercial context). The same goes for Open Sans but at least that's free as it's available in Google Fonts.
We could also make some suggestions for fonts similar to Alverata that are free - I'll have a look.

I'll update the README and put in a pull request...

this is fixed in #26


Add Helm chart release workflow

Publish './charts' to gh-pages branch on git tags with 'chart' prefix.
Signed-off-by: stefanprodan<EMAIL_ADDRESS>Ref: #1424
Release procedure

create OSM release
wait for the container image to be push to Docker Hub
set the container image tag in charts/osm/values.yaml and charts/osm/Chart.yaml
bump the chart version in charts/osm/Chart.yaml
create a PR with the above changes and merge it to main
create a git tag from main in the format chart/vx.x.x where vx.x.x is the chart version
push the git tag to origin
GH Actions runs helm-gh-pages that packages the charts, updates index.yaml and pushes the changes to gh-pages branch
GH Pages publishes the new version, making the chart available at https://openservicemesh.github.io/osm

Note that the above procedure works for any number of charts in the ./charts dir without any changes to the GH Actions workflow.
Applicable areas:

[x] Tests / CI System

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?
Yes, I'm the author of the GH Action.

Codecov Report

Merging #1509 into main will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1509      +/-   ##
==========================================
+ Coverage   46.42%   46.55%   +0.13%
==========================================
  Files         109      109
  Lines        4605     4605
==========================================
+ Hits         2138     2144       +6
+ Misses       2466     2460       -6
  Partials        1        1

Impacted Files
Coverage Δ

...ertificate/providers/tresor/certificate_manager.go
77.22% <0.00%> (+5.94%)
:arrow_up:

Continue to review full report at Codecov.

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

Publish './charts' to gh-pages branch on git tags with 'chart' prefix.
Signed-off-by: stefanprodan<EMAIL_ADDRESS>Ref: #1424
Release procedure

create OSM release
wait for the container image to be pushed to Docker Hub
set the container image tag in charts/osm/values.yaml and charts/osm/Chart.yaml
bump the chart version in charts/osm/Chart.yaml
create a PR with the above changes and merge it to main
create a git tag from main in the format chart/vx.x.x where vx.x.x is the chart version
push the git tag to origin
GH Actions runs helm-gh-pages that packages the charts, updates index.yaml and pushes the changes to gh-pages branch
GH Pages publishes the new version, making the chart available at https://openservicemesh.github.io/osm

Note that the above procedure works for any number of charts in the ./charts dir without any changes to the GH Actions workflow.

Hey @stefanprodan, thanks for this useful change. Would you mind updating docs/release_guide.md with these details?

@shashankram I've added the chart release procedure to the release guide.

Wondering if we should release the chart as part of the release workflow kicked off by the tag update as well.

Yes that should work if you also bump the chart version in Chart.yaml. The idea behind having a dedicated tag for charts is for being able to release a chart fix independent of the OSM releases.


Added Neopixel Example

Summary
Example app using the neopixel driver that displays a rainbow pattern.
Impact
Should not impact other apps.
Testing
Tested app with single and multiple pixel chains with various update speeds.

@curuvar could you squash all patch into one by:
git rebase --interactive HEAD~5

why not squash the rename patch into the first?

@curuvar it's better to squash the change for comment in the main patch .

@xiaoxiang781216, yes, I have been trying to remember to squash before I push changes.  It turns out the change in question was make when I clicked the "Commit Suggestion" button on gustavonihei's comment.  Is it best to avoid this practice, or is there some way to have it squash from the GitHub site itself?

@xiaoxiang781216, yes, I have been trying to remember to squash before I push changes. It turns out the change in question was make when I clicked the "Commit Suggestion" button on gustavonihei's comment. Is it best to avoid this practice, or is there some way to have it squash from the GitHub site itself?

I normally update the change locally instead from the web interface. Another way is enable Squash and Merge:


[small] Fix unused import

When running test this line shows up as an unused import (this file does not compile for non-test so it seems CI did not catch it)

Yeah, okay it looks like it is not as simple :(


Adds functionality to trigger (via snafu) dropping of Ceph OSD caches

A sample file is included in resources to allow the user to manually
deply a privileged pod in the rook-ceph namespace with the capabilities
of listening on an HTTP port for a request to drop caches and then
translating that to a set of direct ceph commands. The fio CR file is
updated to accpet the options to trigger the cache drop, and some minimal
changes have been made to the fio role to pass the option to snafu.
Documentation has been updated to explain the requirements.

Results for minikube_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_ycsb
Pass
1
00:00:40

test_fiod
Pass
0
00:02:39

test_fs_drift
Pass
1
00:00:55

test_smallfile
Fail
3
00:11:11

test_pgbench
Pass
0
00:00:53

test_iperf3
Pass
0
00:01:43

test_sysbench
Pass
0
00:00:35

test_uperf
Pass
0
00:03:55

test_byowl
Pass
0
00:00:32

test_hammerdb
Pass
0
00:01:05

Results for kni_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_ycsb
Pass
0
00:00:56

test_fiod
Pass
0
00:04:59

test_fs_drift
Pass
0
00:01:15

test_smallfile
Fail
3
00:12:58

test_pgbench
Pass
0
00:03:27

test_iperf3
Pass
0
00:01:50

test_sysbench
Pass
0
00:00:43

test_uperf
Pass
0
00:04:15

test_byowl
Pass
0
00:00:37

test_hammerdb
Pass
0
00:01:54

Overall happy with this, one question is the rook_ceph_drop_cache_pod.yaml  should this YAML also be in bohica? I am fine with this change either way, just curious what others think.

IMO we are already causing ourselves ridiculous amounts of pain with all of the inter-dependent code segregation, and my preference is to stop proliferating that behavior. That said, I don't care enough to argue against if it is deemed that the YAML isn't a ripsaw resource and should otherwise be placed in $other_project.

Overall happy with this, one question is the rook_ceph_drop_cache_pod.yaml  should this YAML also be in bohica? I am fine with this change either way, just curious what others think.

IMO we are already causing ourselves ridiculous amounts of pain with all of the inter-dependent code segregation, and my preference is to stop proliferating that behavior. That said, I don't care enough to argue against if it is deemed that the YAML isn't a ripsaw resource and should otherwise be placed in $other_project.

Is the issue that we have Ripsaw / SNAFU and now Bohica? Is your argument we should just have Ripsaw / SNAFU ?

This discussion has gone deeper in-channel, but to close the loop here, I don't mind the move of the yaml to snafu, and I would prefer it go there rather than bohica.

Looping back on this PR since @aakarshg brought it up in conversation. The work was already done here to exclude the cache dropping privileged pod spin-up from ripsaw, replacing that automation with documentation. The open question is about whether the example yaml provided to the user for deploying that pod manually be included here in the ripsaw role or pointed to elsewhere in snafu or bohica. It seems silly that this PR has been sitting for weeks untouched on this point, so let's just make a decision and move forward.
IMO, the yaml should stay here.


feat: implement  x-tagGroup feature

Description
Docusaurus should support nested  sidebar which already implemented in ReDocly as x-tagGroups.
Motivation and Context
Most of the complex Apis need nested resources in one openapi spec. This feature aims to solve this problem within one resource instead of having multiple openapi resources.
How Has This Been Tested?
jest tests added for valid openapi files.
Screenshots (if appropriate)
tags:
  - name: tag1
    description: Everything about your restaurant
    x-displayName: Tag 1
  - name: tag2
    description: Tag 2 description
    x-displayName: Tag 2
  - name: tag3
    description: Tag 3 description
    x-displayName: Tag 3

x-tagGroups:
  - name: Tag 1 & 2
    tags:
      - tag1
      - tag2
  - name: Trinity
    tags:
      - tag1
      - tag2
      - tag3
  - name: Last Two
    tags:
      - tag2
      - tag3

Types of changes

New feature (non-breaking change which adds functionality)

Checklist

[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes if appropriate.
[x] All new and existing tests passed.

Hi @bencagri, looks like there are a few linter errors and some new merge conflicts. Let me know if I can assist with resolving these issues, thanks!

hi @sserrata, conflicts were resolved and also linter errors fixed.

@bencagri, any plans/thoughts on supporting generated category index pages, similar to how non-x-tag-groups are handled?

@bencagri, any plans/thoughts on supporting generated category index pages, similar to how non-x-tag-groups are handled?

hi Steven, actually this should be document writer's choice -from my point of view-. The generator should act as proof of concept, not to generate the docs that are not given.
if you think this is a must, then to provide such a feature, we might generate docs under something uncategorized even if category has not been configured.
But again, this wouldn't be a part of its job. What is your thought?

Makes sense to me. I was speaking more about the tag groups that are generated, i.e. whether or not you think they should link to index pages when they are clicked.

hmm. i missed this point. correct, i dont think we should provide index pages, as they are already a category, no?

Sorry, it took me a minute to remember the name of the option: categoryLinkSource
docs: https://docusaurus-openapi.tryingpan.dev/#sidebaroptions
reference: https://github.com/PaloAltoNetworks/docusaurus-openapi-docs/blob/54399a2b85b25b9bb754558d0e3ca61b0e9bb27b/packages/docusaurus-plugin-openapi-docs/src/sidebars/index.ts#L149

hi @sserrata, i think we should point that categoryLinkSource is disregardred  if the groupPathsBy is set as tagGroup

@sserrata it shows introduction in every dropdown/tagGroup and contains same info please have a look


How to build libtensorflowlite.so with TensorFlow ops supported for armdf?

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: No
TensorFlow installed from (source or binary): source
TensorFlow version: tf-nightly
Python version: 3.6.9
Installed using virtualenv? pip? conda?: virtualenv
Bazel version (if compiling from source): 3.7.2
GCC/Compiler version (if compiling from source): 7.5
CUDA/cuDNN version: 10.2
GPU model and memory: GTX 1070ti

Describe the problem
I need Tensorflow OPS to run tflite model inference in C++.
But how to build libtensorflowlite.so (for armdf) with TensorFlow ops ?
In document, it mentioned:
Add the TensorFlow ops delegate library dependency to the build dependencies: tensorflow/lite/delegates/flex:delegate
So how to do this?
Modify BUILD file in directory tensorflow_src/tensorflow/lite/ ?
Provide the exact sequence of commands / steps that you executed before running into the problem
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

@thaink could you take a look?

Actually, what I need is a C++ library that support interpreting TF ops as libtensorflowlite_flex_jni.so did on Android.

@zhangguanqun if you use the jni how about building tensorflow-lite.aar and tensorflow-lite-select-tf-ops.aar?

@thaink I did build tensorflow-lite.aar, and it works fine on my Android app.
Now I want to run my TFLite model on Raspberry Pi using pure C++ interface.

You can add "tensorflow/lite/delegates/flex:delegate" to the deps field of the rule "tensorflowlite" in tensorflow/lite/BUILD

@thaink Thank u, I will try it.

@thaink
Build report following errors:
no such package '@local_cuda//': The repository '@local_cuda' could not be resolved and referenced by '@cub_archive//:cub'
Build command:
bazel build --config=elinux_armhf -c opt //tensorflow/lite:libtensorflowlite.so

After change deps from  ["@local_cuda//:cuda_headers"] to ["@local_config_cuda//cuda:cuda_headers"] in file bazel-tensorflow-src/external/cub_archive/BUILD.bazel, I got the following errors:
ERROR: /home/zhangguanqun/tensorflow-master/tensorflow/core/kernels/BUILD:4919:18: C++ compilation of rule '//tensorflow/core/kernels:scatter_op_gpu' failed (Exit 1): aarch64-linux-gnu-gcc failed: error executing command /home/zhangguanqun/.cache/bazel/_bazel_zhangguanqun/9a90fa5c1777f045d849f4cfb9aa189a/external/aarch64_linux_toolchain/bin/aarch64-linux-gnu-gcc -fstack-protector -g0 -O2 -DNDEBUG -ffunction-sections ... (remaining 141 argument(s) skipped)
aarch64-linux-gnu-gcc: error: unrecognized command line option '-Xcuda-fatbinary=--compress-all'
aarch64-linux-gnu-gcc: error: unrecognized command line option '--no-cuda-include-ptx=all'; did you mean '--no-chkp-instrument-calls'?
aarch64-linux-gnu-gcc: error: unrecognized command line option '--cuda-include-ptx=sm_61'
aarch64-linux-gnu-gcc: error: unrecognized command line option '--cuda-gpu-arch=sm_61'
aarch64-linux-gnu-gcc: error: unrecognized command line option '-nvcc_options=relaxed-constexpr'
aarch64-linux-gnu-gcc: error: unrecognized command line option '-nvcc_options=ftz=true'
Target //tensorflow/lite:libtensorflowlite.so failed to build

Tensorflow source version: 2.5.0-dev20210210
CUDA version: 10.2

@zhangguanqun What platform are you building on? I am not sure about this error, too.

@thaink
Ubuntu 18.04 Desktop

Did you run ./configure?

@thaink
Yes, I have already successfully build tensorflow-lite.aar with the same TF source and the AAR file is works fine on my Android App.

Or, can I build libtensorflowlite.so without CUDA?
It seems like all errors are related to CUDA.

Right. cub_archive is only pulled in if if_cuda is true.
So it should not be included here.
Could you double check  and share the content of .tf_configure.bazelrc file?

does adding --config=monolithic help?

@thaink
Do you mean run ./configure and set with CUDA=False?
I run configure file with CUDA setting true, here is my .tf_configure.bazelrc file:
build --action_env PYTHON_BIN_PATH="/home/zhangguanqun/anaconda3/bin/python3"
build --action_env PYTHON_LIB_PATH="/home/zhangguanqun/anaconda3/lib/python3.8/site-packages"
build --python_path="/home/zhangguanqun/anaconda3/bin/python3"
build --action_env CUDA_TOOLKIT_PATH="/usr/local/cuda-10.2"
build --action_env TF_CUDA_COMPUTE_CAPABILITIES="6.1"
build --action_env LD_LIBRARY_PATH="/usr/local/cuda-10.2/lib64:/usr/lib/x86_64-linux-gnu/"
build --action_env GCC_HOST_COMPILER_PATH="/usr/bin/x86_64-linux-gnu-gcc-7"
build --config=cuda
build:opt --copt=-Wno-sign-compare
build:opt --host_copt=-Wno-sign-compare
build --action_env ANDROID_NDK_HOME="/home/zhangguanqun/Android/android-ndk-r21b"
build --action_env ANDROID_NDK_API_LEVEL="21"
build --action_env ANDROID_BUILD_TOOLS_VERSION="30.0.3"
build --action_env ANDROID_SDK_API_LEVEL="30"
build --action_env ANDROID_SDK_HOME="/home/zhangguanqun/Android/Sdk"
test --flaky_test_attempts=3
test --test_size_filters=small,medium
test --test_env=LD_LIBRARY_PATH
test:v1 --test_tag_filters=-benchmark-test,-no_oss,-no_gpu,-oss_serial
test:v1 --build_tag_filters=-benchmark-test,-no_oss,-no_gpu
test:v2 --test_tag_filters=-benchmark-test,-no_oss,-no_gpu,-oss_serial,-v1only
test:v2 --build_tag_filters=-benchmark-test,-no_oss,-no_gpu,-v1only

Should I configure CUDA = false?

Adding --config=monolithic does not help.

UPDATE:
If I set with CUDA to False in runing configure file, .tf_configure.bazelrc file change to
build --action_env PYTHON_BIN_PATH="/usr/bin/python3"
build --action_env PYTHON_LIB_PATH="/usr/lib/python3.6/dist-packages"
build --python_path="/usr/bin/python3"
build:opt --copt=-Wno-sign-compare
build:opt --host_copt=-Wno-sign-compare
build --action_env ANDROID_NDK_HOME="/home/zhangguanqun/Android/android-ndk-r21b"
build --action_env ANDROID_NDK_API_LEVEL="21"
build --action_env ANDROID_BUILD_TOOLS_VERSION="30.0.3"
build --action_env ANDROID_SDK_API_LEVEL="30"
build --action_env ANDROID_SDK_HOME="/home/zhangguanqun/Android/Sdk"
test --flaky_test_attempts=3
test --test_size_filters=small,medium
test:v1 --test_tag_filters=-benchmark-test,-no_oss,-gpu,-oss_serial
test:v1 --build_tag_filters=-benchmark-test,-no_oss,-gpu
test:v2 --test_tag_filters=-benchmark-test,-no_oss,-gpu,-oss_serial,-v1only
test:v2 --build_tag_filters=-benchmark-test,-no_oss,-gpu,-v1only

Than I try to rebuild libtensorflowlite.so with the same command, but got some errors with OPEN-SSL :(
ERROR: /home/zhangguanqun/.cache/bazel/_bazel_zhangguanqun/9a90fa5c1777f045d849f4cfb9aa189a/external/com_github_grpc_grpc/BUILD:1659:16: C++ compilation of rule '@com_github_grpc_grpc//:grpc_secure' failed (Exit 1): arm-linux-gnueabihf-gcc failed: error executing command /home/zhangguanqun/.cache/bazel/_bazel_zhangguanqun/9a90fa5c1777f045d849f4cfb9aa189a/external/armhf_linux_toolchain/bin/arm-linux-gnueabihf-gcc -fstack-protector -g0 -O2 -DNDEBUG -ffunction-sections ... (remaining 68 argument(s) skipped)
In file included from /usr/include/openssl/evp.h:16,
                 from /usr/include/openssl/x509.h:18,
                 from external/com_github_grpc_grpc/src/core/tsi/ssl_transport_security.h:28,
                 from external/com_github_grpc_grpc/src/core/lib/security/security_connector/security_connector.h:33,
                 from external/com_github_grpc_grpc/src/core/lib/security/credentials/credentials.h:35,
                 from external/com_github_grpc_grpc/src/core/lib/http/httpcli_security_connector.cc:35:
/usr/include/openssl/bio.h:690:1: error: expected constructor, destructor, or type conversion before 'DEPRECATEDIN_1_1_0'
 DEPRECATEDIN_1_1_0(int BIO_get_port(const char *str, unsigned short *port_ptr))
 ^~~~~~~~~~~~~~~~~~
In file included from /usr/include/openssl/asn1.h:23,
                 from /usr/include/openssl/objects.h:15,
                 from /usr/include/openssl/evp.h:28,
                 from /usr/include/openssl/x509.h:18,
                 from external/com_github_grpc_grpc/src/core/tsi/ssl_transport_security.h:28,
                 from external/com_github_grpc_grpc/src/core/lib/security/security_connector/security_connector.h:33,
                 from external/com_github_grpc_grpc/src/core/lib/security/credentials/credentials.h:35,
                 from external/com_github_grpc_grpc/src/core/lib/http/httpcli_security_connector.cc:35:
/usr/include/openssl/bn.h:183:43: error: 'BN_ULONG' does not name a type; did you mean 'SHA_LONG'?
 int BN_abs_is_word(const BIGNUM *a, const BN_ULONG w);
                                           ^~~~~~~~
                                           SHA_LONG
/usr/include/openssl/bn.h:186:39: error: 'BN_ULONG' does not name a type; did you mean 'SHA_LONG'?
 int BN_is_word(const BIGNUM *a, const BN_ULONG w);
                                       ^~~~~~~~
                                       SHA_LONG
/usr/include/openssl/bn.h:214:22: error: 'BN_ULONG' was not declared in this scope
 int BN_num_bits_word(BN_ULONG l);
                      ^~~~~~~~
/usr/include/openssl/bn.h:214:22: note: suggested alternative: 'SHA_LONG'
 int BN_num_bits_word(BN_ULONG l);
                      ^~~~~~~~
                      SHA_LONG
/usr/include/openssl/bn.h:266:1: error: 'BN_ULONG' does not name a type; did you mean 'SHA_LONG'?
 BN_ULONG BN_mod_word(const BIGNUM *a, BN_ULONG w);
 ^~~~~~~~
 SHA_LONG
/usr/include/openssl/bn.h:267:1: error: 'BN_ULONG' does not name a type; did you mean 'SHA_LONG'?
 BN_ULONG BN_div_word(BIGNUM *a, BN_ULONG w);
 ^~~~~~~~
 SHA_LONG
/usr/include/openssl/bn.h:268:28: error: 'BN_ULONG' has not been declared
 int BN_mul_word(BIGNUM *a, BN_ULONG w);
                            ^~~~~~~~
/usr/include/openssl/bn.h:269:28: error: 'BN_ULONG' has not been declared
 int BN_add_word(BIGNUM *a, BN_ULONG w);
                            ^~~~~~~~
/usr/include/openssl/bn.h:270:28: error: 'BN_ULONG' has not been declared
 int BN_sub_word(BIGNUM *a, BN_ULONG w);
                            ^~~~~~~~
/usr/include/openssl/bn.h:271:28: error: 'BN_ULONG' has not been declared
 int BN_set_word(BIGNUM *a, BN_ULONG w);
                            ^~~~~~~~
/usr/include/openssl/bn.h:272:1: error: 'BN_ULONG' does not name a type; did you mean 'SHA_LONG'?
 BN_ULONG BN_get_word(const BIGNUM *a);
 ^~~~~~~~
 SHA_LONG
/usr/include/openssl/bn.h:288:37: error: 'BN_ULONG' has not been declared
 int BN_mod_exp_mont_word(BIGNUM *r, BN_ULONG a, const BIGNUM *p,
                                     ^~~~~~~~
/usr/include/openssl/bn.h:323:24: error: variable or field 'BN_consttime_swap' declared void
 void BN_consttime_swap(BN_ULONG swap, BIGNUM *a, BIGNUM *b, int nwords);
                        ^~~~~~~~
/usr/include/openssl/bn.h:323:24: error: 'BN_ULONG' was not declared in this scope
/usr/include/openssl/bn.h:323:24: note: suggested alternative: 'SHA_LONG'
 void BN_consttime_swap(BN_ULONG swap, BIGNUM *a, BIGNUM *b, int nwords);
                        ^~~~~~~~
                        SHA_LONG
/usr/include/openssl/bn.h:323:46: error: expected primary-expression before '*' token
 void BN_consttime_swap(BN_ULONG swap, BIGNUM *a, BIGNUM *b, int nwords);
                                              ^
/usr/include/openssl/bn.h:323:47: error: 'a' was not declared in this scope
 void BN_consttime_swap(BN_ULONG swap, BIGNUM *a, BIGNUM *b, int nwords);
                                               ^
/usr/include/openssl/bn.h:323:57: error: expected primary-expression before '*' token
 void BN_consttime_swap(BN_ULONG swap, BIGNUM *a, BIGNUM *b, int nwords);
                                                         ^
/usr/include/openssl/bn.h:323:58: error: 'b' was not declared in this scope
 void BN_consttime_swap(BN_ULONG swap, BIGNUM *a, BIGNUM *b, int nwords);
                                                          ^
/usr/include/openssl/bn.h:323:61: error: expected primary-expression before 'int'
 void BN_consttime_swap(BN_ULONG swap, BIGNUM *a, BIGNUM *b, int nwords);
                                                             ^~~
/usr/include/openssl/bn.h:332:1: error: expected constructor, destructor, or type conversion before 'DEPRECATEDIN_0_9_8'
 DEPRECATEDIN_0_9_8(int
 ^~~~~~~~~~~~~~~~~~
/usr/include/openssl/bn.h:403:1: error: expected constructor, destructor, or type conversion before 'DEPRECATEDIN_0_9_8'
 DEPRECATEDIN_0_9_8(int BN_get_params(int which)) /* 0, mul, 1 high, 2 low, 3
 ^~~~~~~~~~~~~~~~~~
In file included from /usr/include/openssl/objects.h:15,
                 from /usr/include/openssl/evp.h:28,
                 from /usr/include/openssl/x509.h:18,
                 from external/com_github_grpc_grpc/src/core/tsi/ssl_transport_security.h:28,
                 from external/com_github_grpc_grpc/src/core/lib/security/security_connector/security_connector.h:33,
                 from external/com_github_grpc_grpc/src/core/lib/security/credentials/credentials.h:35,
                 from external/com_github_grpc_grpc/src/core/lib/http/httpcli_security_connector.cc:35:
/usr/include/openssl/asn1.h:555:7: error: expected constructor, destructor, or type conversion before 'unsigned'
 const unsigned char *ASN1_STRING_get0_data(const ASN1_STRING *x);
       ^~~~~~~~
In file included from /usr/include/openssl/x509.h:22,
                 from external/com_github_grpc_grpc/src/core/tsi/ssl_transport_security.h:28,
                 from external/com_github_grpc_grpc/src/core/lib/security/security_connector/security_connector.h:33,
                 from external/com_github_grpc_grpc/src/core/lib/security/credentials/credentials.h:35,
                 from external/com_github_grpc_grpc/src/core/lib/http/httpcli_security_connector.cc:35:
/usr/include/openssl/ec.h:274:1: error: expected constructor, destructor, or type conversion before 'DEPRECATEDIN_1_2_0'
 DEPRECATEDIN_1_2_0(int EC_GROUP_get_curve_GFp(const EC_GROUP *group, BIGNUM *p,
 ^~~~~~~~~~~~~~~~~~
/usr/include/openssl/ec.h:543:1: error: expected constructor, destructor, or type conversion before 'DEPRECATEDIN_1_2_0'
 DEPRECATEDIN_1_2_0(int EC_POINT_get_affine_coordinates_GFp(const EC_GROUP *group,
 ^~~~~~~~~~~~~~~~~~
/usr/include/openssl/ec.h:631:1: error: expected constructor, destructor, or type conversion before 'size_t'
 size_t EC_POINT_point2oct(const EC_GROUP *group, const EC_POINT *p,
 ^~~~~~
In file included from /usr/include/openssl/x509.h:25,
                 from external/com_github_grpc_grpc/src/core/tsi/ssl_transport_security.h:28,
                 from external/com_github_grpc_grpc/src/core/lib/security/security_connector/security_connector.h:33,
                 from external/com_github_grpc_grpc/src/core/lib/security/credentials/credentials.h:35,
                 from external/com_github_grpc_grpc/src/core/lib/http/httpcli_security_connector.cc:35:
/usr/include/openssl/rsa.h:239:1: error: expected constructor, destructor, or type conversion before 'int'
 int RSA_generate_key_ex(RSA *rsa, int bits, BIGNUM *e, BN_GENCB *cb);
 ^~~
In file included from /usr/include/openssl/dsa.h:25,
                 from /usr/include/openssl/x509.h:26,
                 from external/com_github_grpc_grpc/src/core/tsi/ssl_transport_security.h:28,
                 from external/com_github_grpc_grpc/src/core/lib/security/security_connector/security_connector.h:33,
                 from external/com_github_grpc_grpc/src/core/lib/security/credentials/credentials.h:35,
                 from external/com_github_grpc_grpc/src/core/lib/http/httpcli_security_connector.cc:35:
/usr/include/openssl/dh.h:142:1: error: expected constructor, destructor, or type conversion before 'int'
 int DH_generate_parameters_ex(DH *dh, int prime_len, int generator,
 ^~~
In file included from /usr/include/openssl/x509.h:26,
                 from external/com_github_grpc_grpc/src/core/tsi/ssl_transport_security.h:28,
                 from external/com_github_grpc_grpc/src/core/lib/security/security_connector/security_connector.h:33,
                 from external/com_github_grpc_grpc/src/core/lib/security/credentials/credentials.h:35,
                 from external/com_github_grpc_grpc/src/core/lib/http/httpcli_security_connector.cc:35:
/usr/include/openssl/dsa.h:103:1: error: expected constructor, destructor, or type conversion before 'int'
 int DSA_sign(int type, const unsigned char *dgst, int dlen,
 ^~~
/usr/include/openssl/dsa.h:127:1: error: expected constructor, destructor, or type conversion before 'int'
 int DSA_generate_parameters_ex(DSA *dsa, int bits,
 ^~~
In file included from external/com_github_grpc_grpc/src/core/tsi/ssl_transport_security.h:28,
                 from external/com_github_grpc_grpc/src/core/lib/security/security_connector/security_connector.h:33,
                 from external/com_github_grpc_grpc/src/core/lib/security/credentials/credentials.h:35,
                 from external/com_github_grpc_grpc/src/core/lib/http/httpcli_security_connector.cc:35:
/usr/include/openssl/x509.h:728:1: error: expected constructor, destructor, or type conversion before 'DEPRECATEDIN_1_1_0'
 DEPRECATEDIN_1_1_0(ASN1_TIME *X509_CRL_get_nextUpdate(X509_CRL *crl))
 ^~~~~~~~~~~~~~~~~~
Target //tensorflow/lite:libtensorflowlite.so failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 724.445s, Critical Path: 115.39s
INFO: 3503 processes: 148 internal, 3355 local.
FAILED: Build did NOT complete successfully

My OpenSSL version is: 1.1.1  11 Sep 2018

right. cuda should be config to no.
I am was able to reproduce a similar build error, let me check.

@thaink  🙏
Maybe I should downgrade openssl.

I hope this is helpful.

According to https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/tools/pip_package/build_pip_package_with_bazel.sh#L39
you need to hide the system openssl. And you also need to modify /usr/include/curl/curlbuild.h

"--config=monolithic --config=noaws --config=nogcp --config=nohdfs --config=nonccl" is required for Flex build.

To avoid #1, it's better to use container build tool "tensorflow/tools/ci_build/ci_build.sh" and create your own build script.

@terryheo Thanks, advice 1st did help, errors about openssl did not exist, but, at the end of building, linking error reported:
ERROR: /home/zhangguanqun/tensorflow-master/tensorflow/lite/BUILD:887:24: Linking of rule '//tensorflow/lite:libtensorflowlite.so' failed (Exit 1): arm-linux-gnueabihf-gcc failed: error executing command /home/zhangguanqun/.cache/bazel/_bazel_zhangguanqun/9a90fa5c1777f045d849f4cfb9aa189a/external/armhf_linux_toolchain/bin/arm-linux-gnueabihf-gcc ... (remaining 1 argument(s) skipped)
stderr (/home/zhangguanqun/.cache/bazel/_bazel_zhangguanqun/9a90fa5c1777f045d849f4cfb9aa189a/execroot/org_tensorflow/bazel-out/_tmp/actions/stderr-7638) exceeds maximum size of --experimental_ui_max_stdouterr_bytes=1048576 bytes; skipping
Target //tensorflow/lite:libtensorflowlite.so failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 2182.429s, Critical Path: 474.31s
INFO: 4542 processes: 493 internal, 4049 local.
FAILED: Build did NOT complete successfully

How about increase the value of --experimental_ui_max_stdouterr_bytes?

@thaink
Yes, I have increated max_stdouterr_bytes to<PHONE_NUMBER>, and rebuilding is doing now, this may take several tens of minutes.

Finally made it !
Thanks to @abattery @terryheo @thaink, Appreciate.
Cmd that successfully build libtensorflowlite.so:
bazel build --config=monolithic --config=noaws --config=nogcp --config=nohdfs --config=nonccl --config=elinux_armhf --experimental_ui_max_stdouterr_bytes=1073741819 -c opt //tensorflow/lite:libtensorflowlite.so

@zhangguanqun how did you solve the problem below?
"  The repository '@local_cuda' could not be resolved and referenced by '@cub_archive//:cub' "


Improve Scoreboard API

This PR improves the recently added Scoreboard API. A list of the changes is below:

Add getScoreboard() method to Objective and Team
Add getObjective() method to Score
Add getMainScoreboard() method to the registry
Correct method signature of ScoreboardBuilder#build()

There's no 'main scoreboard' - scoreboards are serialized with a particular world.

@Aaron1011 Ah, I see now. I'll remove the method from the PR.

Teams and objectives can be added to multiple scoreboards.

@Aaron1011 Hm, in that case should the methods return Sets instead?

@caseif: I think that sounds good.

Can you fix the return type of ScoreboardBuilder#reset()? It should probably return ScoreboardBuilder and not Scoreboard. (From #550)

@Minecrell Fixed.

@caseif Did you? You didn't push.

@Minecrell Sorry, my mistake.


Update CDVLocation.m

Just added background mode to this function:

(void)startLocation:(BOOL)enableHighAccuracy
{
[self.commandDelegate runInBackground:^{

This stops iOS from giving warnings that the Plugin should be executing in background mode, and also stops random crashes of the plugin and makes it more stable on iOS.
Platforms affected
iOS
What does this PR do?
Increase Stability by running Function in Background Mode.
What testing has been done on this change?
iOS tests shows that without this PR, warning is showing and also plugin crashes at some points.
With the PR plugin gains stability, and no warning found.

Cordova CI Build has completed successfully.
Commit     - Link
Dashboard - Link

Builder Name
Console Output
Test Report
Device Logs

Windows 10  Store
Link
Link
Link

iOS 9.3
Link
Link
Link

iOS 10.0
Link
Link
Link

Android 4.4
Link
Link
Link

Android 5.1
Link
Link
Link

Ping @shazron for a review.
@avishekcode have you signed an Apache ICLA?

@filmaj Nope. Is that a requirement? I didn't think there would be any licensing issue, since I am contributing to an already open project, where the terms are already in place. I would think licensing might be required if I created a entirely new project.

@avishekcode in general it is required, I believe there is some wiggle room for trivial contributions, but in this case, I think the changes are significant enough that we'll need an ICLA signed from you.
Check out: https://www.apache.org/licenses/#clas
If you wouldn't mind filling out + signing the ICLA here: https://www.apache.org/licenses/icla.pdf and e-mailing it to<EMAIL_ADDRESS>then I think that's all we need to safely pull in your code.
Thanks so much!

@filmaj Greatly appreciate for letting me know.
I just submitted the Individual Contributor License Agreement to<EMAIL_ADDRESS>via email.
Not sure if I need to do anything else.
Thanks!

Just received the ICLA. Thanks!

Hey @avishekcode, I'm trying to reproduce these warnings you mentioned:

This stops iOS from giving warnings that the Plugin should be executing in background mode, and also stops random crashes of the plugin and makes it more stable on iOS.

How do they look like? I cannot seem to reproduce with the latest master release of this plugin (without your code changes). I am simply issuing a getCurrentLocation call when my app starts up.

@avishekcode what device are you testing it on? @filmaj I believe if he is using an older phone, it might affect timings

@shazron I've been trying to reproduce these warnings on a pretty old iOS device: iPhone 5S, running iOS 10.2. Haven't been able to repro anything that looks weird in the device console.

Hmm not sure then :/
Regarding my line comments on this PR, I realize that we have another instance not following the rules in the existing code 😱  https://github.com/apache/cordova-plugin-geolocation/blob/9f9496f0577ff66fb2449ac21b28ee1f0252aca2/src/ios/CDVLocation.m#L197

@shazron @filmaj
Hi!
I literally, just downloaded the current plugin, and without my modification, I get these warnings constantly via Xcode, while testing on iPhone 5s:
2017-04-26 17:45:32.151697-0400 TabCab Inc[3927:2025999] THREAD WARNING: ['Geolocation'] took '10.250000' ms. Plugin should use a background thread.
2017-04-26 17:45:32.256468-0400 TabCab Inc[3927:2025999] THREAD WARNING: ['Geolocation'] took '33.067871' ms. Plugin should use a background thread.
2017-04-26 17:45:52.700650-0400 TabCab Inc[3927:2025999] THREAD WARNING: ['Geolocation'] took '25.637939' ms. Plugin should use a background thread.
2017-04-26 17:46:18.191620-0400 TabCab Inc[3927:2025999] THREAD WARNING: ['Geolocation'] took '10.547852' ms. Plugin should use a background thread.
Device: iPhone 5s, but I believe it happens even in newer phones, if I am using the plugin for applications like Uber (Motion capture in the background).
Now, I apologize if I broke some rules while trying to implement the solution. I will try to dig into the rules, and any quick suggestion how to fix the problem otherwise?
My solution puts the function in background thread, stops all warnings, and it increases the performance of the plugin and the overall app.
My use of the plugin in my applications, is as follows:
function getAccurateGPS(GPSok,GPSerr,GPSing,options,Abort)
{
var lastPos,locECnt=0,watchID,navTim;options=options || {};
if(Abort){clearTimeout(navTim);navTim=null;navigator.geolocation.clearWatch(watchID);}
else
{	var checkLocation=function(position)
{	lastPos=position;locECnt=locECnt+1;	//Ignore first event unless it's the only one because some devices send cached location even with maxaimumAge=0
if((position.coords.accuracy<=options.desiredAccuracy) && (locECnt>1)){clearTimeout(navTim);navTim=null;navigator.geolocation.clearWatch(watchID);foundPos(position);}else{GPSing(position);}
};
var stopTrying=function(){navigator.geolocation.clearWatch(watchID);foundPos(lastPos);};
var onError=function(error){clearTimeout(navTim);navTim=null;navigator.geolocation.clearWatch(watchID);GPSerr(error);};
var foundPos=function(position){GPSok(position);};
if(!options.maxWait)			options.maxWait=10000;
if(!options.desiredAccuracy)	options.desiredAccuracy=5;
if(!options.timeout)			options.timeout=options.maxWait;
options.maximumAge=0;options.enableHighAccuracy=true;
watchID=navigator.geolocation.watchPosition(checkLocation,onError,options);
navTim=setTimeout(stopTrying,options.maxWait);//Timeout
}
}
Basically, my applications require constant motion tracking, and I try to capture the most accurate position available.
Please let me know if there is anything else you need from me, or if you have any suggestions.
Thanks!

Okay, with that sample code of using the watch, I am able to reproduce one warning of this on an iPhone 5S running iOS 10.2:
2017-04-26 17:14:37.015628 HelloCordova[273:48525] THREAD WARNING: ['Geolocation'] took '44.689941' ms. Plugin should use a background thread.

I need to run the app from xcode to see that warning, though - that warning does not show in the Device Console.
Now when using @avishekcode's patch, the warning is gone.
So I'm fine with merging this patch in as long as any comments from @shazron are addressed.


Bug fixes for reducer, styles and state

Hey, Makeda, I need to see a picture of how the front end looks after this pull request on here before I merge it, I also need to see a trello ticket waiting for review, with a picture associated with it and a trello pr link to this pr.

[](url

)

 Code-Cleanup


docs

create manual google cloud + openvpn + db1000n

I had to amend commits in the repo for privacy and legal reasons (mostly precaution but still), could you please recreate your fork or rebase it on the latest history?


Dashboard query acts oddly and throws errors

Found this during the back port to 5.6.9, may exist in 6.x.
Load up the regression dashboard, and attempt to change the query, will get an error.

fixed in 5.6.9


When is the task assembly loaded?

Is the task loaded from assembly when UsingTask is evaluated or when the task is used for the first time?

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

ID: 96b3f5bf-9736-884e-9d08-e9422b4891a5
Version Independent ID: fa34a483-01a9-8062-ac26-a0c8de0821b5
Content: UsingTask Element (MSBuild) - Visual Studio
Content Source: docs/msbuild/usingtask-element-msbuild.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @ghogen
Microsoft Alias: ghogen

It's loaded the first time it's required.


Remove the scroll bar after “Who uses EvalAI”

Observed Behaviour
Scroll bar is visible under Who uses EvalAI on normal screen. The scroll bar is used for medium and small screen. Find attached screenshot.
Expected Behaviour
The scroll bar should not be visible on normal screen.
@RishabhJain2018

@RishabhJain2018 @gautamjajoo  In PR #108 I have removed the scroll bar and also build is successfull  but surge is not able to login. Please look into this issue.

Hi @Abhi58 , Thanks for the PR. I'll surely fix the surge issue by tonight.

@RishabhJain2018 Thanks.


Support distribution as a standalone JS file

Similar to Ton Web:
<script src="tonweb.js"></script>

const tonweb = new window.TonWeb();

It will be very useful for people building TON apps with toolchains that don't rely on NPM

I am not sure how we can sign this js files and confirm that it is authentic while in general we can trust NPM since breaking NPM probably very hard task for any hacker (I suppose they are using immutable and signed data at rest).

Two ideas come to mind:

Put the dist JS file in this repo either pushed with the other sources or as a Github Release artifact. Developers would trust the file is by you the same way they trust the source and they will host it by themselves on their websites when they use it.

Create a package.json script npm run build:web that will allow developers to build the standalone dist JS file by themselves from source.

This is the way Tonweb does it and it seems secure enough, don't you think?

much needed feature.


Show an "add fields?" button/link on Fields page when none present

If no fields are defined, we should show a link or button asking if the user wants to add fields; this should toggle edit mode.

This issue is not relevant anymore using Apigility 1.1.0.


Added unit tests for Klf6 frameshift, fix bug in frameshift translation

Found a scary bug in the frameshift_coding_effect.py. The logic for translating frameshifts was wrong for some cases, added unit tests and fix.
Fixes https://github.com/hammerlab/varcode/issues/151, https://github.com/hammerlab/topiary/issues/52

This change is

To confirm: no intended change in the logic other than removing nucleotide_after from case 2 and nucleotides_after from case 3?

@tavinathanson Confirmed! Everything else is just refactoring to shrink the code and make it possible to test the insertion logic separately.

Coverage decreased (-0.3%) to 86.687% when pulling 0443ea5d4968b483c1aabacb6a36d7f8157be9e9 on Klf6-mm10-frameshift-test into<PHONE_NUMBER>7331accb66ac7cca67ffd408c622a7 on master.

Coverage increased (+0.1%) to 87.089% when pulling 73d59decb3e201ed147bb45a52c6684e49fe355d on Klf6-mm10-frameshift-test into<PHONE_NUMBER>7331accb66ac7cca67ffd408c622a7 on master.

Coverage increased (+0.4%) to 87.348% when pulling d0b851ceb28deb1a6c62e4985edc5c2b3078e541 on Klf6-mm10-frameshift-test into<PHONE_NUMBER>7331accb66ac7cca67ffd408c622a7 on master.

LGTM. Some of this logic took me a while to wrap my head around, though it's not immediately clear to me how to simplify.

Coverage increased (+0.9%) to 87.881% when pulling f29784284429c1cb823b1b23759c07ab058413f8 on Klf6-mm10-frameshift-test into<PHONE_NUMBER>7331accb66ac7cca67ffd408c622a7 on master.


NuGet package

dear sir,
have you considered creating NuGet package for your library? it would be really helpful for .NET part of the world.
thx :+1: ,
d

Hey @rostacik
Thanks for your interest and sorry for the late response. Right now, this library is on npm and bower and I'm not planning any other package manager. But of course you can fork, maintain your own copy and publish to your community.
Cheers.


Added transactionSync endpoints

ok something really weird happened here.. tons of files were apparently changed, but with no noticeable differences. Maybe something weird with the text encoding??


Cannot Install

Hi, this will no longer load as an integration as neither standard or expert config can be selected - on submit the "Unknown Error Occurred" message is given.

Hello,
Since the last version V5.0.0 , I have added the use of matplotlib. Perhaps it is related to this.
Have you restarted HA. It has taken sometimes for me .
Please can you look in home-assistant.log if there is a keyword "Traceback" related to worldtidesinfocustom.
Thanks in advance
Philippe

Hi Phillippe,
Nothing in the logs relating to worldtides, yes I did a couple of restarts but the error remains. I ve not used this integration before and was about to take out an admiralty subscription but a HAC integration would be much better.
Thanks in advance for helping me :)
Regards,
Nigel Reburn

On 29 May 2021, at 14:00, jugla @.***> wrote:
Hello,
Since the last version V5.0.0 , I have added the use of matplotlib. Perhaps it is related to this.
Have you restarted HA. It has taken sometimes for me .
Please can you look in home-assistant.log if there is a keyword "Traceback" related to worldtidesinfocustom.
Thanks in advance
Philippe
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/jugla/worldtidesinfocustom/issues/10#issuecomment-850829751, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUD67DBNMNDLAR4QPNJFTATTQDQOTANCNFSM45YIPZGQ.

Please, which type of HA version do you have ?
I use the core version on raspberry PI4. Just to be sure, can you look inside the following directory :
/srv/homeassistant/lib/python3.9/site-packages if there are files/directory of matplotlib ?

As a workaround, have you tried within configuration.yaml
with
##configuration.yaml example
sensor:
  - platform: worldtidesinfocustom
    name: royan_tides
    api_key: YOUR_API_KEY
    latitude: 45.61949378902948
    longitude: -1.0318721687376207

Perhaps, it will failed but with a clear trace in home-assistant.log

Here a try with bergen (norway):

Hi, I have HA loaded on an UnRaid Unix server so I don’t see that Python directory.
Regards,
Nigel Reburn

On 29 May 2021, at 14:28, jugla @.***> wrote:
Please, which type of HA version do you have ?
I use the core version on raspberry PI4. Just to be sure, can you look inside the following directory :
/srv/homeassistant/lib/python3.9/site-packages if there are files/directory of matplotlib ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/jugla/worldtidesinfocustom/issues/10#issuecomment-850833698, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUD67DD37BXO4EVJTFZAW7TTQDTWFANCNFSM45YIPZGQ.

I could try this if I csn find that yaml file (and my API key).
Regards,
Nigel Reburn

On 29 May 2021, at 14:36, jugla @.***> wrote:
As a workaround, have you tried within configuration.yaml
with
##configuration.yaml example
sensor:

platform: worldtidesinfocustom
name: royan_tides
api_key: YOUR_API_KEY
latitude: 45.61949378902948
longitude: -1.0318721687376207
Perhaps, it will failed but with a clear trace in home-assistant.log

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/jugla/worldtidesinfocustom/issues/10#issuecomment-850834877, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUD67DBNUTHQACFHZ7OZD73TQDUXPANCNFSM45YIPZGQ.

That looks absolutely perfect for what I want to do - I do hope I can get it to work1
Regards,
Nigel Reburn

On 29 May 2021, at 14:39, jugla @.***> wrote:
Here a try with bergen (norway):
https://user-images.githubusercontent.com/59493499/120072381-f2226d00-c093-11eb-9b96-147645786630.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/jugla/worldtidesinfocustom/issues/10#issuecomment-850835268, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUD67DHA5ZUXMAYLRPLPSI3TQDVARANCNFSM45YIPZGQ.

You succeed in !

API key is from wordtidesinfo server : you can get a temporary key
the configuration.yaml is below HomeAssistant home directory. In ~ /.homeassistant folder.

Ok thanks Phillippe, I will take a look later and let you know how it goes.
Regards,
Nigel Reburn

On 29 May 2021, at 14:59, jugla @.***> wrote:
You succeed in !
API key is from wordtidesinfo server : you can get a temporary key
the configuration.yaml is below HomeAssistant home directory. In ~ /.homeassistant folder.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/jugla/worldtidesinfocustom/issues/10#issuecomment-850838194, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUD67DBKC77YRQ7XBVFGFJTTQDXMDANCNFSM45YIPZGQ.

Hi Phillipe,
Thanks so much for your help - I am going to try to do this with a different site now as I didn’t realise it required a subscription - thanks/sorry.
Regards,
Nigel Reburn

On 29 May 2021, at 15:04, Nigel Reburn @.***> wrote:
Ok thanks Phillippe, I will take a look later and let you know how it goes.
Regards,
Nigel Reburn

On 29 May 2021, at 14:59, jugla @.*** @.***>> wrote:
You succeed in !
API key is from wordtidesinfo server : you can get a temporary key
the configuration.yaml is below HomeAssistant home directory. In ~ /.homeassistant folder.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/jugla/worldtidesinfocustom/issues/10#issuecomment-850838194, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUD67DBKC77YRQ7XBVFGFJTTQDXMDANCNFSM45YIPZGQ.

Hello
Any news Nigel ?
I will close this issue : if you have other questions , feel free

Hi Phillippe,
No I have no further questions thanks, please do close the issue and thanks very much for your help. I may come back to you to use this at a later date if the other system does not work.
Regards,
Nigel Reburn

On 3 Jun 2021, at 11:11, jugla @.***> wrote:
Hello
Any news Nigel ?
I will close this issue : if you have other questions , feel free
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/jugla/worldtidesinfocustom/issues/10#issuecomment-853755449, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUD67DCOVKN4LLUZFK6AZI3TQ5INJANCNFSM45YIPZGQ.

Hi. Just for information. I had same error (or similar): "Unknown Error Occurred" message and some errors related to the integration in logs (a few restarts tried). I try to configure as sensor (config.yaml), and after that the integration worked properly. After the integration works, I delete the sensor because HA said the "Platform worldtidesinfocustom does not generate unique IDs". After delete the sensors (config.yaml) all working ok. Thanks for the integration.

Hello,
Thanks !
Please can you tell, what was your configuration i.e. worldtidesinfocustom never installed ?
Have you kept the trace (log) ?
Thanks in advance
Philippe

Hi @jugla . Yes,  worldtidesinfocustom never installed before. Sorry, I'm not kept the error logs. Anyway, I'll try to delete integration to check if the error come back with no config.yaml 'trick'

Hi again. I have tried to reproduce the issue, but it seems that after configure it as sensor (config.yaml) the first time, no errors are coming. That's include delete and reinstall integration

Thanks !
I think it's because, matplotlib takes some time to be installed . This installation is triggered by HA when it analyses the worldtidesinfo's manifest.json file.
So when you remove the integration, you have left matplotlib.
It is my current guess.

Therefore, all we have to do is wait a few minutes after the first restart. I have experienced similar situations with other HACS integrations. Thanks @jugla

Thanks @somar05 !


Parse attributes from custom element

I want to pass attribs  in my custom element like this:
<my-custom-element attribute-one="[1,2,3,4]" ></my-custom-element>
and receive that into my preact component as an Array, and not a String. For now, im parsing all my props before rendering:
for(let key in this.props){
	try {
		this.props[key] = JSON.parse(this.props[key])
	} catch (e) {}
}

Do you guys think that is a nice improvement for the project? If you agree, i can make a PR with that, just let me know!

One option here would be to add a hint in the property name - something like <c-e prop$="[1,2,3">

I implemented some fairly simple logic for this in my org's forked vue wc wrapper around here, where users would pass a JSON to a prop with a -json suffix, then map that -json back to the "real" prop. It helped us keep the seamlessness between consuming components as Web Components or just as plain Vue components.
For clarity:
<x-element something-json="[1,2,3]">...</x-element>
Would get mapped to something inside the component, Array.isArray(something) would evaluate to true.


html_safe values lose safety

Given config/locales/test.en.rb:
{
    en: {
        test: '<i>test</i>'.html_safe,
    }
}

I expect that:
I18n.t('test').html_safe? #=> true

In v0.8.5 this actually returns false.
This used to work but I'm not sure exactly at which version this stopped working.

Hey @glebm :wave:
Thanks for reporting this issue. I attempted to reproduce this issue today and was unable to. Here are my exact steps:

Create a new rails app rails new i18n-377
Create config/locales/test.en.rb in this Rails app with the content specified
In the Gemfile, specify gem 'i18n', '0.8.5' explicitly.
Create a new file called repro.rb at the rails app root, with content of p I18n.t('test').html_safe?
Run this file with rails runner repro.rb
Observe the output is true, not false.

Could you please try to reproduce this in a new app and if you're able to do so, put that up on GitHub?
Thanks!

Hi @radar!
It must be one of the gems I have causing this.
I'll need to add them to a demo app one by one to figure out which one is causing this.
This might take a while :sweat_smile: Closing the issue for the time being

A surprising development! The issue is fixed in v0.8.6. :smile:

Hooray! Thanks for letting me know :)

On 11 Jul 2017, at 08:14, Gleb Mazovetskiy<EMAIL_ADDRESS>wrote:
A surprising development! The issue is fixed in v0.8.6. 😄
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Questions

Adds support for retrieving Questions and their available Responses, in both singular and collection form.
@shaisachs there's a tiny bit of refactoring left to do (the common responses, parsing auth, etc). There was a little bit of crazy refactoring in this pr that I wanted an early look at:

Refactored all resource tests into a common file that creates the tests
Refactored the ngpvanapi-client into its own folder with sub-clients

@shaisachs it's all revved up and ready to go

There is something about passing around this trio of functions (unauthorized/badRequest/success) that I kind of dislike aesthetically.  Part of me would like to create some kind of class called, say, NGPVANAPIResponseStrategy, and to send an instance of that strategy through the call chain.  This strategy in turn provides the unauthorized/badRequest/success trio internally.  And of course we can then create a sort of stock strategy used by almost all of our endpoints.  Maybe we don't need to do that just yet but it's a thought.  (And yes I know I was the one who started all this nonsense. :)

all fixed except for the issue I made and the validate discussion for tomorrow

Sweet!  I'm sure we can get that figured out quickly.

Ship it!


[feature request] port benchmarks to google benchmark

you have a simple benchmark framework in tensorflow/tensorflow/core/platform/test_benchmark.h but i was curious if you'd be interested in integrating with github.com/google/benchmark instead.
i'm happy to help with the integration if you are.

We're working on a benchmarking framework and some infrastructure. I'll let @ebrevdo comment.

Gchat me and we can discuss.

@dominichamon, @ebrevdo: What was the result of the offline discussion?

No concrete plans to merge code, but backward compatible commits are welcome.

Since no action has been taking in a while and it appears that there isn't compelling motivation to use the google framework, closing for now. @annarev, reopen if you feel this is likely to be a priority.


Verify challenge resource matches request domain

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Security.KeyVault.Administration


[sunder] make plugins available for kafka connect

this will fix issue 40 Kafka connects Plugins are not working. #40

Thanks for the fix ! Can you also add another directory that the user can optionally mount, this way we support user connectors as well ? Call it "/connectors"

Thanks for the fix ! Can you also add another directory that the user can optionally mount, this way we support user connectors as well ? Call it "/connectors"
Sure will do that

done.


Question about zxingwriter for datamatrix code

Hey,
Can anyone tell me how to force the writer to generate squared data matrix code?
Thanks in advance.

This is discussed in #581


Add pre-commit hook to git repo and point the documentation to it

Do we recommend the EDM Council hook or the one on the shared drive? If the latter, we need to add it to the repo. In either case we need to make sure the documentation is correct. The relevant documentation is gistStyleGuide.md.

From Michael: There is a pre-commit file in this folder: ...\Offerings\gist\git4gist\ which, I think, was a reworked one from edmc changed by Ted. Have a look.
The task is to:

Choose which version of the pre-commit hook to use.
Add it to the repository.
Update the notes in gistStyleGuide and Contributing.md.

See #331 - both issues require the pre-commit hook. Consider writing a single script per function (serialization, adding language tags), with a single pre-commit hook that calls each one. Or is this too much baggage?

Michael has raised questions about using the pre-commit hook and thus requiring serialization before commit, so this needs further review.

Use of the pre-commit hook cannot be enforced, so the issue is just to provide it and documentation.

Enforcement is done by reviewers, who should require serialization.


jit lax_numpy.roll

This was making tracing slow for code with lots of rolls.

I'm still pretty confident that allowing the user to specify both static argument names and numbers would work. (We had an open issue where we are discussing this somewhere, but I can't find it right now.)

I'm still pretty confident that allowing the user to specify both static argument names and numbers would work

Yeah, I think that could be done.

See https://github.com/google/jax/issues/1936 for discussion on keyword arguments


Improve an error message

Description:
Consider the following code
int a = b;
var b = 6;
public function main() {

}

Gives the error incompatible types: expected 'int', found 'other' which is less informative. Need to improve the  error msg to something like  variable declared with 'var' does not support variable forward referencing
Suggested Labels (optional):

Type/Improvement
Team/CompilerFE

Why should this result in a compilation error anyway?

Why should this result in a compilation error anyway?

Yes, we can support this from the implementation


Add property to identify mockbukkit environment

Description

This PR adds a system property to identify mockbukkit environment.
This can be used for checking if we are in a unittest.
This would help solve having multiple constructors in the main class.
Checklist
The following items should be checked before the pull request can be merged.

[x] Code follows existing style.
[ ] Unit tests added (if applicable).

Particularly, this lets us validate at runtime rather than just within the test folder. In updating from MockBukkit 2 (1.18) to 3 (1.20.4) we noticed the constructor which used to be invoked (Slimefun(JavaPluginLoader loader, PluginDescriptionFile description, File dataFolder, File file)) no longer is and instead it's just the default no param one (Slimefun()). This means we can no longer rely on the constructor being used (this change was made in https://github.com/MockBukkit/MockBukkit/pull/747)
Therefore, for us to know we're within a unit test within our standard code, we need an identifier. Java properties are a nice way to do this.
The alternatives are hacky and not something we'd like to do, we can add a new constructor and parameter to every single MockBukkit.load or we can check if the class loader has mockbukkit in the package name  or other similar alternatives
We think this would be a low cost but good general improvement.

I'm not the best qualified person for deciding whether this is a good idea. But I do believe that we should not have this behavior as it encourages running slightly modified code when testing. If this would have been a good idea, why haven't junit done anything similar?
Instead of doing that, you can also do something like this:
public static final boolean IS_TEST = isRunningTestI();

private static boolean isRunningTest() {
        try {
            Class.forName("org.junit.Test");
            return true;
        } catch (ClassNotFoundException e) {
            return false;
        }
}

I hope this solves your problems

Oh, it 100% is not good and not recommended practice however, within a plugin env it's quite difficult to cover all cases. We had to make a number of PRs to just get 1.20.4 working for example and that will continue. There's also just other plugin interaction (slimefun works with like 15-20 other plugins...) which I wish we could easily test but we cannot.
The aim for us is definitely to move off using a secondary code path and not covering specific code but it just isn't simple today.
This PR is not a requirement for us and we already have a new way (a simple class detection like you alluded to). It was more of a "we're having to do this so others may be too". Y'all are obviously the best judges for if this should go in. Feel free to close if y'all do not wish for it to land :)

All good, this is just my opinion. I will leave this open for a while if anyone dissagrees with me in the dev team.

I agree with the stance brought forward by @Thorinwasher

I also agree with Thorinwasher. We shouldn't have an official way in the library to do this. If it needs to be done, you can use a utility method like he posted.


HNC: Add recommended RBAC rules in README

For rules like 'don't change/delete hc instances', instead of implementing them in webhooks, we can just ask users to use RBAC rules to enforce them.

I don't really find a good place to put this specific RBAC rules in the README. I think it may be okay not to add it since user are not likely to add a permission on hierarchyconfigurations any way? WDYT @adrianludwin ?

If user's don't give permission to hierarchyconfigurations, then no-one can use them :)
Maybe add the documentation somewhere in here?
https://docs.google.com/document/d/1R4rwTweYBWYDTC9UC-qThaMkPk8gtflr_ycHOOqzBQE/edit#heading=h.fbb92511bj1z

Just left a footnote here: https://docs.google.com/document/d/1R4rwTweYBWYDTC9UC-qThaMkPk8gtflr_ycHOOqzBQE/edit?disco=AAAAGZZ-WLI. Do you think this is good enough?

Thanks for the explanation and the update on the doc!


Update typescript-starter

Typescript-starter now has an interactive CLI generator:

50th pull request! I am accepting it without review! Just kidding, I did a review.
Thank you @bitjson ! Really nice project

Thanks!


:set doesn't work for 'cot' and 'bkc' after :setlocal

Problem:  :set doesn't work for 'cot' and 'bkc' after :setlocal.
Solution: Clear the local flags when using :set.

thanks


Modernize docker compose usage & language

I think the old versions of docker-compose have percolated out of dev environments by now, since v2 was released in 2020. See https://docs.docker.com/compose/releases/migrate

docker-compose –> docker compose
docker-compose.yml –> compose.yml
Remove deprecated "version" tag from compose.yml (it is causing warnings in the dev env)

@LocoDelAssembly You mentioned a concern with the change in chat I think?

Missed this one. File rename and removing version seems compatible with the docker version that comes with Ubuntu 22.04 LTS (still supported). However I'd remove the unzip mount point as it is not necessary.
The version number was 2 because I think were using something that was removed in later versions, something related to health checks or detecting when the container is ready, but not currently needed since startup script in app container waits for db to come up before letting TW run.


Update yarn.lock

Update versions in yarn.lock

Thanks @jadnast. Lockfile was just updated on the main branch, so closing this PR.


Fix static analayzer warning for uninitialized variable

Sorry to be so noisy about this somewhat trivial issue.  I am still getting used to how github works for collaboration.  This patch is pretty much the same thing as pull-request/issue #449 except this time I put the change on a branch so I can work on some other things without polluting the change set.  I also think I made the variable name a little better.
The issue is that if you run the analyzer you get an initialized variable warning.  It is most likely a false positive (assuming the server conforms correctly to the OAuth2 spec) but I think it is better to get rid of the warning clutter.
RestKit/Code/Network/RKOAuthClient.m:119:30:{119:81-119:90}: warning: Argument in message expression is an uninitialized value
                          NSError *error = [NSError errorWithDomain:RKRestKitErrorDomain code:errorCode userInfo:userInfo];
                                           ^                                                  ~~~~~~~~~
             1 warning generated.

If this change turns out to be unacceptable, that is fine.  I'm in it for the learning experience of it.

Ray -
Thanks for the patch. I tweaked the error name a little bit and filled in some blank comments around your changes before merging to master.


[BUG] chunks flickering

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

play 3arthh4ck

Expected behavior
Chunks are flickering, this is not an crash. If I move to an different chunk, it works only for that chunk, and others flicker
Crashlog
Not an crash
Additional context
Heres a video, steamable don't wok for me :C
https://seupload.com/shared/7j3cfqcwpwid1kdmzpo91g_ir11tb0p1xmv6yo8wp_ydxsm63ndqjhi7xywneqcyqhgrhoz732fnsnqux9chyqr1pp2i0rt7yvp-9hkrocyzz-z8knq02xqsu24pos3s

Same here

use 1.6.2

use 1.6.2

😎


Documentation: Default Fonts

Hello,
I wonder which Fonts pdfkit supports "out-of-the-box".
As far as I see we have "Times-Roman" and "Helvetica".
Where can I find more Informations about this?

The available fonts are listed in this folder: lib/font/data

Courier, Courier-Bold, Courier-Oblique, Courier-BoldOblique
Helvetica, Helvetica-Bold, Helvetica-Oblique, Helvetica-BoldOblique
Times-Roman, Times-Bold, Times-Italic, Times-BoldItalic
Symbol
ZapfDingbats

The characters supported by the default fonts are there: https://github.com/devongovett/pdfkit/issues/483#issuecomment-199059845

Awesome! Works like a charm!
Thanks!


Unable to open from Cloud Drive any Dropbox account

Hi,
I'm unable to enter my Dropbox credentials (for either type of Dropboxes) : page is pixellised, and only 2 buttons work : the upper left box, going to register page ; and the 'forgot password' button, going to a complete broken page.
Brief: can't click on 'connect with google' nor 'connect' so i can't do anything.
KeePass 2.35
KeePassLibC 1.32
KeeAnywhere 1.3.1
Thanks for your work ! :)

Could you please attach a screenshot of this misbehavior?

Cloing due to inactivity


Cannot Generate Open API swagger.json file if inheriting from ITableEntity From Azure.Data.Tables

Describe the issue
We recently have run into an issue where after upgrading our Azure Storage SDK to support managed identity we can no longer generate the swagger.json file. The error we receive is:
Method 'get_ETag' in type '' from assembly ', Version=X, Culture=neutral, PublicKeyToken=null' does not have an implementation.
at System.Reflection.CustomAttribute._CreateCaObject(RuntimeModule pModule, RuntimeType type, IRuntimeMethodInfo pCtor, Byte** ppBlob, Byte* pEndBlob, Int32* pcNamedArgs)
at System.Reflection.CustomAttribute.CreateCaObject(RuntimeModule module, RuntimeType type, IRuntimeMethodInfo ctor, IntPtr& blob, IntPtr blobEnd, Int32& namedArgs)
at System.Reflection.CustomAttribute.AddCustomAttributes(ListBuilder1& attributes, RuntimeModule decoratedModule, Int32 decoratedMetadataToken, RuntimeType attributeFilterType, Boolean mustBeInheritable, ListBuilder1 derivedAttributes)
at System.Reflection.CustomAttribute.GetCustomAttributes(RuntimeModule decoratedModule, Int32 decoratedMetadataToken, Int32 pcaCount, RuntimeType attributeFilterType)
at System.Reflection.CustomAttribute.GetCustomAttributes(RuntimeMethodInfo method, RuntimeType caType, Boolean inherit)
at System.Reflection.RuntimeMethodInfo.GetCustomAttributes(Type attributeType, Boolean inherit)
at System.Attribute.GetCustomAttributes(MemberInfo element, Type attributeType, Boolean inherit)
at Microsoft.Azure.WebJobs.Extensions.OpenApi.Core.DocumentHelper.GetOpenApiResponses(MethodInfo element, NamingStrategy namingStrategy, VisitorCollection collection, OpenApiVersionType version)
at Microsoft.Azure.WebJobs.Extensions.OpenApi.Document.Build(Assembly assembly, OpenApiVersionType version)
at Microsoft.Azure.WebJobs.Extensions.OpenApi.OpenApiTriggerFunctions.RenderSwaggerDocument(OpenApiHttpTriggerContext openApiContext, HttpRequest req, String extension, ExecutionContext ctx, ILogger log)
To Reproduce

Create an Http Trigger
Ensure you are running Azure.Data.Tables version 12.8.1
Have the Request Body Type include a class that inherits from ITableEntity
Attempt to Generate swagger.json file by visiting URL upon building

Expected behavior
We can generate the swagger.json file when using native Microsoft Libraries.
Additional context
I did some initial investigation and this seems related to feature request #327 where we either need to added the allowance of custom visitors or include E_Tag in the core code. For now I've created Tags that exclude the affected endpoints so I can compile the Open API spec. Any additional help would be greatly appreciated.

I am experiencing the same issue

I was able to solve this issue after updating the SDK to the latest.
<PackageReference Include="Microsoft.NET.Sdk.Functions" Version="4.4.0" />
Make sure that you are using the OpenApi package based on the project type ( in process or isolated )


Bug: Hourly Emissions Streaming endpoints missing JSON schema on swagger

Test Case
Steps to Reproduce

Navigate to the Stream API on Swagger https://api.epa.gov/easey/dev/streaming-services/swagger/#/
Open the Hourly emissions state, facility, or national endpoints
The system does not contain a JSON schema for the Hourly emissions state, facility, or national endpoints and swagger displays an error message

Open Defect in TestQuality

Test Move Hourly Aggregation stream endpoints to the Stream API in Run Move Hourly Aggregation stream endpoints to the Stream API (linked to Cycle Aggregation).

Defect resolved. Schema available on swagger


sample to get channel site url

sample to get channel site url

Thanks so much @pkbullock , I will be mindful of trailing spaces in the future.


HTTPException does not have the expected behavior

If the take_token() condition is not satisfied (False), the script should go to HTTPException, but the app is breaking, and it should just return an error in the API.

I believe it should be something like this:
return JSONResponse(             status_code=status.HTTP_429_TOO_MANY_REQUESTS,             content={"detail": "Too many requests"},         )


Improves the service health check

This improves the service health check by checking the runningCount against the desiredCount, which helps in the case where an application is running at a lower scale.
This would report that the service is unhealthy and prevent the next node from being terminated before the task is running on another ECS instance.

This update changes the workflow logic so that the upgrade terminates if the ECS cluster is unhealthy and the ECS cluster size is the desired size of the auto-scaling group.
This means that a new ECS node has come online that AWS says is healthy, but our services are not running at desired capacity, which means there is an issue.  This scenario can be replicated by changing IAM permissions or Docker registry authentication -- that would cause the ECS node to be unable to reach specific AWS resources or pull Docker containers from the registry.
If that happens, you will see similar output:
Starting upgrade of container instances:
pagebuilder-tools (i-01d446c25393272cd): initial
pagebuilder-tools (i-01d446c25393272cd): replacing
pagebuilder-tools (i-01d446c25393272cd): replacing
pagebuilder-tools (i-01d446c25393272cd): replacing
================================================================================
Cluster Health
================================================================================
[{u'activeServicesCount': 1,
  u'clusterArn': u'arn:aws:ecs:us-east-1:397853141546:cluster/pagebuilder-tools',
  u'clusterName': u'pagebuilder-tools',
  u'pendingTasksCount': 0,
  u'registeredContainerInstancesCount': 3,
  u'runningTasksCount': 2,
  u'status': u'ACTIVE'}]
================================================================================
Service ops-tools-sandbox is not running at the desired scale
ECS cluster upgrade failed.


Incorrect Job Name in metrics prometheus_target_interval_length_seconds

What did you do?

Deployed Prometheus with default configuration
Deployed Node Exporter
Updated prometheus.yml configuration file and added node_exporter scrape job with 10s interval and set prometheus scrape job to 12s interval

Open prometheus web console http://localhost:9090/graph and browse metrics prometheus_target_interval_length_seconds
What did you expect to see?
It should have returned following time series
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="10s",job="node_exporter",quantile="0.01"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="10s",job="node_exporter",quantile="0.05"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="10s",job="node_exporter",quantile="0.5"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="10s",job="node_exporter",quantile="0.9"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="10s",job="node_exporter",quantile="0.99"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="12s",job="prometheus",quantile="0.01"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="12s",job="prometheus",quantile="0.05"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="12s",job="prometheus",quantile="0.5"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="12s",job="prometheus",quantile="0.9"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="12s",job="prometheus",quantile="0.99"}
What did you see instead? Under which circumstances?
It returns following time series. Every job label value is stamped as prometheus.
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="10s",job="prometheus",quantile="0.01"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="10s",job="prometheus",quantile="0.05"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="10s",job="prometheus",quantile="0.5"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="10s",job="prometheus",quantile="0.9"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="10s",job="prometheus",quantile="0.99"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="12s",job="prometheus",quantile="0.01"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="12s",job="prometheus",quantile="0.05"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="12s",job="prometheus",quantile="0.5"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="12s",job="prometheus",quantile="0.9"}
prometheus_target_interval_length_seconds{instance="localhost:9090",interval="12s",job="prometheus",quantile="0.99"}
Environment
Local Ubuntu VM:
cat /etc/os-release
NAME="Ubuntu"
VERSION="18.04.5 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 18.04.5 LTS"
VERSION_ID="18.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic

System information:

Linux 5.4.0-52-generic x86_64

Prometheus version:
prometheus, version 2.22.0 (branch: HEAD, revision: 0a7fdd3b76960808c3a91d92267c3d815c1bc354)
build user:       root@6321101b2c50
build date:       20201015-12:29:59
go version:       go1.15.3
platform:         linux/amd64

Alertmanager version:
NA

Prometheus configuration file:

# my global config
global:
  scrape_interval:     15s # Set the scrape interval to every 15 seconds. Default is every 1 minute.
  evaluation_interval: 15s # Evaluate rules every 15 seconds. The default is every 1 minute.
  # scrape_timeout is set to the global default (10s).

# Alertmanager configuration
alerting:
  alertmanagers:
  - static_configs:
    - targets:
      # - alertmanager:9093

# Load rules once and periodically evaluate them according to the global 'evaluation_interval'.
rule_files:
  # - "first_rules.yml"
  # - "second_rules.yml"

# A scrape configuration containing exactly one endpoint to scrape:
# Here it's Prometheus itself.
scrape_configs:
  # The job name is added as a label `job=<job_name>` to any timeseries scraped from this config.
  - job_name: 'prometheus'
    scrape_interval: 12s
    # metrics_path defaults to '/metrics'
    # scheme defaults to 'http'.

    static_configs:
    - targets: ['localhost:9090']

  - job_name: 'node_exporter'
    scrape_interval: 10s
    static_configs:
      - targets: ['localhost:9100']

Alertmanager configuration file:

NA

Logs:

NA

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.

Thanks Brian... But, just in case, if this is a bug how discussing in mailing list can help getting the fix? Do contributors take part in the conversations in the mailing list?

I do keep an eye out for bugs on the list, however this is documented behaviour with no indications of a bug so the mailing list is a better place for it.


Remove unexpectedly introduced .DS_Store files

The .DS_Store files introduced by the commit 1519195a3179dc8d6e1cd412a4513e919abca43d in this repo, but the name of this file is in .gitignore file. so I pretty sure those files are committed unexpectedly

@dreamoving take a look at this, .DS_Store files contain sensitive data of the committer's devices. it should be removed ASAP

Thanks for your reminder！ We have removed the .DS_Store files from this project


Cant get barcode detection to wok

Using the example provided, ive added the following lines to viewDidLoad()
        self.cameraEngine.metadataDetection = .barcode

        self.cameraEngine.blockCompletionCodeDetection = { codeObject in
            print(codeObject.stringValue)
            let valueCode = codeObject.stringValue
            let frameCode = (codeObject as AVMetadataObject).bounds
            print(frameCode)
//            self.displayLayerDetection(frame: frameCode)
        }

and am unable to detect ean13 codes. is there something else i need to do?

after a few hours of tinkering, i've figured out the problem...
inside CameraEngineMetadataOutput.swift the method

func captureOutput(_ captureOutput: AVCaptureOutput!, didOutputMetadataObjects metadataObjects: [AnyObject]!, from connection: AVCaptureConnection!)

was replaced with

metadataOutput(_ output: AVCaptureMetadataOutput, didOutput metadataObjects: [AVMetadataObject], from connection: AVCaptureConnection)


WIP: rename 'FunctionContext' to 'ValidationContext'

fixes #29 :slightly_smiling_face:
EDIT: hrmph :confused: tests seemed to pass locally.. I'll give this another look and see why this didn't work out. For now I will flag this as a WIP

Did you git submodule update --init locally?

Ahh, thanks for pointing this out! I'll take care of that and ping you once this is ready for review. Thanks :smiley_cat:

Hmm, that problem seems to happen when I check out master as well. I'll try and fix that in a separate PR, and rebase this once that's landed :slightly_smiling_face:

👍 Thanks!


Label alignment tweaks

The CMS label when "in-frame" should be aligned in the top left corner with the same distance from both axes.

Sure, I can do that.
It was not the default setting in the "old" TDR style. I kept that code as much as possible.
At the moment, it's set as a fixed fraction of each of the pad lengths.
I can make the fraction equals, this would be safe to have and not depending on the squared/rectangular shapes.
Alternatively, I can give a fix spacing, but has to be large enough to avoid overlaps in any possibility.

addressed in #13

PR merged

Sorry to reopen this, but can yo also update the example pdfs with the change?

Yeah, sounds good.

I can work on migrating this repo to using LFS now

Now the repo is using LFS

See #14.
I fixed it also for the ratio pad.
I also removed the annoying cut of the y-axis when having the ratio pad drawn.

https://github.com/cms-cat/cms-root-style/pull/14 merged


Add support for Hypre with hip

Enables us to run with Hypre on AMD GPUs.

[x] CMAKE build

Ex1p works for me :-)
 ./ex1p -d hip
Options used:
   --mesh ../data/star.mesh
   --order 1
   --no-static-condensation
   --no-partial-assembly
   --device hip
   --visualization
Device configuration: hip,cpu
Memory configuration: host-std,hip
Number of finite element unknowns: 82561

 Num MPI tasks = 1

 Num OpenMP threads = 1

BoomerAMG SETUP PARAMETERS:

 Max levels = 25
 Num levels = 7

 Strength Threshold = 0.250000
 Interpolation Truncation Factor = 0.000000
 Maximum Row Sum Threshold for Dependency Weakening = 0.900000

 Coarsening Type = PMIS
 measures are determined locally

 No global partition option chosen.

 Interpolation = extended+i interpolation

Operator Matrix Information:

             nonzero            entries/row          row sums
lev    rows  entries sparse   min  max     avg      min         max
======================================================================
  0   82561   739201  0.000     4   11     9.0  -1.221e-15   1.915e+00
  1   15187   300885  0.001     6   25    19.8  -2.998e-15   2.420e+00
  2    2615    66307  0.010     8   34    25.4  -4.226e-15   2.472e+00
  3     402    10166  0.063     7   35    25.3  -1.032e-14   2.471e+00
  4      63     1277  0.322     8   34    20.3   1.487e-14   2.427e+00
  5      10       98  0.980     9   10     9.8   1.382e+00   2.535e+00
  6       1        1  1.000     1    1     1.0   2.684e+00   2.684e+00

Interpolation Matrix Information:
                    entries/row        min        max            row sums
lev  rows x cols  min  max  avgW     weight      weight       min         max
================================================================================
  0 82561 x 15187   0    4   3.8   1.138e-02   7.036e-01   0.000e+00   1.000e+00
  1 15187 x 2615    1    4   3.9   4.212e-03   8.855e-01   9.386e-02   1.000e+00
  2  2615 x 402     0    4   3.8   3.379e-03   8.302e-01   0.000e+00   1.000e+00
  3   402 x 63      0    4   3.6   3.885e-03   7.500e-01   0.000e+00   1.000e+00
  4    63 x 10      0    4   2.9   2.394e-03   5.433e-01   0.000e+00   1.000e+00
  5    10 x 1       0    1   0.6   9.140e-02   2.553e-01   0.000e+00   1.000e+00

     Complexity:    grid = 1.221388
                operator = 1.512356
                memory = 1.963636

BoomerAMG SOLVER PARAMETERS:

  Maximum number of cycles:         1
  Stopping Tolerance:               0.000000e+00
  Cycle type (1 = V, 2 = W, etc.):  1

  Relaxation Parameters:
   Visiting Grid:                     down   up  coarse
            Number of sweeps:            1    1     1
   Type 0=Jac, 3=hGS, 6=hSGS, 9=GE:     18   18     9
   Point types, partial sweeps (1=C, -1=F):
                  Pre-CG relaxation (down):   0
                   Post-CG relaxation (up):   0
                             Coarsest grid:   0

   Iteration :   0  (B r, r) = 0.51194
   . . .
   Iteration :  20  (B r, r) = 2.10548e-25
Average reduction factor = 0.245671

All that is missing now is just the CMAKE build.

Now works with cmake!

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

Merged in next for testing...

@camierjs, @YohannDudouit, @tzanio, @artv3, I merged this PR for testing, however, if you have any further feedback we can still address it before merging into master.

@artv3 Could you please share your HYPRE configuration? I'm trying to run ex1p. I can successfully run ./ex1p -d hip -pa but I can't get HypreBoomerAMG (i.e., full assembly) to work. I have tried both with & without --enable-unified-memory in my HYPRE configuration.

Hi @wcdawn,
What version of hypre are you using? For CUDA or HIP we require hypre >= 2.22.1 and I've tested v2.22.1, and v2.23.0. The new v2.24.0 probably works too but I have not tested it myself. For building hypre I use something like this:
CUCC="${HIP_CXX}" \
./configure \
  --with-hip \
  --with-gpu-arch="${HIP_ARCH}" \
  CXX="${MPICXX}" \
  CC="${MPICC}" \
  --prefix="${PREFIX_HYPRE}" \
  --disable-fortran

where HIP_CXX is a wrapper for hipcc that also adds flags for compiling and linking with MPI (this may or may not be important depending on the issue you are seeing).
Do you get a build error in hypre, mfem, or you get a runtime error when running ex1p?

@v-dobrev I was using the master branch of Hypre from GitHub but I've switched to v2.23.0 for now.
I'm getting a runtime error when I run ./ex1p -d hip.
Memory access fault by GPU node-1 (Agent handle: 0x2e16900) on address 0x7fe2d101a000. Reason: Page not present or supervisor privilege.
[jezebel:940172] *** Process received signal ***
[jezebel:940172] Signal: Aborted (6)
[jezebel:940172] Signal code:  (-6)
[jezebel:940172] [ 0] /lib/x86_64-linux-gnu/libpthread.so.0(+0x153c0)[0x7fe4104973c0]
[jezebel:940172] [ 1] /lib/x86_64-linux-gnu/libc.so.6(gsignal+0xcb)[0x7fe3f44c418b]
[jezebel:940172] [ 2] /lib/x86_64-linux-gnu/libc.so.6(abort+0x12b)[0x7fe3f44a3859]
[jezebel:940172] [ 3] /opt/rocm-5.0.0/hip/lib/../../lib/libhsa-runtime64.so.1(+0x72847)[0x7fe3f401b847]
[jezebel:940172] [ 4] /opt/rocm-5.0.0/hip/lib/../../lib/libhsa-runtime64.so.1(+0x7545b)[0x7fe3f401e45b]
[jezebel:940172] [ 5] /opt/rocm-5.0.0/hip/lib/../../lib/libhsa-runtime64.so.1(+0x18757)[0x7fe3f3fc1757]
[jezebel:940172] [ 6] /lib/x86_64-linux-gnu/libpthread.so.0(+0x9609)[0x7fe41048b609]
[jezebel:940172] [ 7] /lib/x86_64-linux-gnu/libc.so.6(clone+0x43)[0x7fe3f45a0293]
[jezebel:940172] *** End of error message ***

I get no such error when I use partial assembly (i.e., -pa).
 Thanks!


Optional trailing space

Hi,
On my routes if I omit the trailing space, the view does not return the instance.
I managed to create a fix with a decorator like this :
def check_trailing_slash(view):
    def wrap(request, *args, **kwargs):
        path = kwargs['path']
        if path.endswith('/'):
            return view(request, *args, **kwargs)
        else:
            uri = request.build_absolute_uri()
            return redirect(
                uri.replace(path, f"{path}/"),
                permanent=True,
            )

    return wrap

It works but i found the fix a bit ugly.
Is there something I missed with your library to manage this case ?

Hi, I added trailing_slash param to view(...), try smth like this:
mptt_urls.view(model='gallery.models.Category', view='gallery.views.category', slug_field='slug', trailing_slash=False)
New version of the package (2.0.4) is uploaded to PyPi.
Lmk if it works for you.


lodash: signatures of the method _.template changed

https://lodash.com/docs#template
https://lodash.com/docs#_ (description of the chainable wrapper)

thanks mate!


Automatic download of Past Broadcasts [Feature Request]

Could it be possible to automatically download past broadcasts?
Obviously there would be the issues of polling the API too constantly and getting TL banned.

Exactly that. Too many requests get you banned. And a single search is not a single request but 5-10 requests to the API depending on the content and the amount of different games (2 per second are allowed). I like the idea and it is definitely not the first time I receive this reqeust but I do not see a possibility with the current API to implement that.


Remove unused variables?

The following variables are declared, allocated, then deallocated.
culmSpeciesProduced(:), culmReactionNumber(:), culmRates(:), culmSpeciesLoss(:), culmReactionNumberLoss(:), culmRatesLoss(:)
@rs028 Can I safely remove these, since they do nothing? If there's no particular plan to use them then I would vote to remove them to declutter the code. They can always be re-added later.

@spco I think so. If they don't do anything and there is no indication of what they were intedend for I think it is okay to get rid of them.


[BUG] На сайте кнопка Следующая страница всегда ведет на первую

Describe the bug
A clear and concise description of what the bug is.
Please complete the following information:

Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

I mean this is a problem in vitepress, I'll update to the latest version and check if it's fixed there

I thought the problem was with vitepress itself. It turned out that the link settings there are a little specific. I fixed this in the latest update.


BabylonJS hello world!

Please assign the issue to me or ignore!
By making a PR you are agreeing to that contributing a hello world project allows the repo/project owner to use the hello world app in examples for their own use.

[ ] only the most simple basic hello world app
[ ]  must say "Hello World" on the main component
[ ]  only include the minimal components (1 ideally)
[ ]  no routing, not extras whatsoever
[ ] no styling
[ ]  nothing else !
[ ] must work by running npm install and npm start (or equivalent)
[ ] must add to the readme
[ ]  must be in the folder named -app

Hey @Yonet do you still have bandwidth to add babylon example?

Yes, thank you for reminding @manekinekko

FYI @manekinekko PR