Upgrade Mongo Management Studio.app to v1.7.0

Updates Mongo Management Studio to the most recent version, 1.7.0

Thank you for the contribution. It was merged directly as 2c470c89f5a3235ca07eac6d44978906dc283a1a to keep commit history cleaner. Your contribution is still credited to you.

:+1:


spanner-cli 0.10.0 does not work with go 1.19

Tried installing spanner-cli@latest (0.10.0) and I got this error tcp_keepalive_unix.go:27:2: cannot find package.
Seems it's not compatible with go 1.19 yet?

I was able to successfully install spanner-cli 0.10.0 with a fresh-installed go1.19.13.
Here is my download log:
$ go version
go version go1.19.13 linux/amd64

$ go install github.com/cloudspannerecosystem/spanner-cli@latest
go: downloading github.com/cloudspannerecosystem/spanner-cli v0.10.0
go: downloading cloud.google.com/go/spanner v1.55.0
go: downloading cloud.google.com/go v0.111.0
go: downloading github.com/apstndb/gsqlsep v0.0.0-20230324124551-0e8335710080
go: downloading github.com/chzyer/readline v0.0.0-20180603132655-2972be24d48e
go: downloading github.com/jessevdk/go-flags v1.4.0
go: downloading github.com/olekukonko/tablewriter v0.0.4
go: downloading github.com/xlab/treeprint v1.0.1-0.20200715141336-10e0bc383e01
go: downloading google.golang.org/api v0.155.0
go: downloading google.golang.org/grpc v1.60.1
go: downloading google.golang.org/protobuf v1.32.0
go: downloading github.com/golang/protobuf v1.5.3
go: downloading github.com/googleapis/gax-go/v2 v2.12.0
go: downloading github.com/json-iterator/go v1.1.12
go: downloading go.opencensus.io v0.24.0
go: downloading google.golang.org/genproto/googleapis/rpc v0.0.0-20240102182953-50ed04b92917
go: downloading google.golang.org/genproto v0.0.0-20231212172506-995d672761c0
go: downloading cloud.google.com/go/iam v1.1.5
go: downloading cloud.google.com/go/longrunning v0.5.4
go: downloading google.golang.org/genproto/googleapis/api v0.0.0-20240102182953-50ed04b92917
go: downloading github.com/mattn/go-runewidth v0.0.8
go: downloading golang.org/x/exp v0.0.0-20230310171629-522b1b587ee0
go: downloading golang.org/x/oauth2 v0.15.0
go: downloading go.opentelemetry.io/otel v1.21.0
go: downloading go.opentelemetry.io/otel/trace v1.21.0
go: downloading github.com/modern-go/concurrent v0.0.0-20180228061459-e0a39a4cb421
go: downloading github.com/modern-go/reflect2 v1.0.2
go: downloading github.com/golang/groupcache v0.0.0-20210331224755-41bb18bfe9da
go: downloading cloud.google.com/go/compute/metadata v0.2.3
go: downloading go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc v0.46.1
go: downloading cloud.google.com/go/compute v1.23.3
go: downloading golang.org/x/time v0.5.0
go: downloading golang.org/x/net v0.19.0
go: downloading github.com/envoyproxy/go-control-plane v0.11.1
go: downloading go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.46.1
go: downloading github.com/google/s2a-go v0.1.7
go: downloading golang.org/x/sys v0.15.0
go: downloading github.com/go-logr/logr v1.3.0
go: downloading go.opentelemetry.io/otel/metric v1.21.0
go: downloading github.com/felixge/httpsnoop v1.0.4
go: downloading github.com/googleapis/enterprise-certificate-proxy v0.3.2
go: downloading github.com/cncf/xds/go v0.0.0-20230607035331-e9ce68804cb4
go: downloading github.com/envoyproxy/protoc-gen-validate v1.0.2
go: downloading github.com/go-logr/stdr v1.2.2
go: downloading github.com/cespare/xxhash/v2 v2.2.0
go: downloading github.com/cncf/udpa/go v0.0.0-20220112060539-c52dc94e7fbe
go: downloading golang.org/x/text v0.14.0
go: downloading golang.org/x/sync v0.5.0
go: downloading github.com/census-instrumentation/opencensus-proto v0.4.1
go: downloading golang.org/x/crypto v0.17.0

$ spanner-cli -h
Usage:
  spanner-cli [OPTIONS]

spanner:
  -p, --project=       (required) GCP Project ID. [$SPANNER_PROJECT_ID]
  -i, --instance=      (required) Cloud Spanner Instance ID [$SPANNER_INSTANCE_ID]
  -d, --database=      (required) Cloud Spanner Database ID. [$SPANNER_DATABASE_ID]
  -e, --execute=       Execute SQL statement and quit.
  -f, --file=          Execute SQL statement from file and quit.
  -t, --table          Display output in table format for batch mode.
  -v, --verbose        Display verbose output.
      --credential=    Use the specific credential file
      --prompt=        Set the prompt to the specified format
      --history=       Set the history file to the specified path
      --priority=      Set default request priority (HIGH|MEDIUM|LOW)
      --role=          Use the specific database role
      --endpoint=      Set the Spanner API endpoint (host:port)
      --directed-read= Directed read option (replica_location:replica_type). The replicat_type is optional and either READ_ONLY or READ_WRITE

Help Options:
  -h, --help           Show this help message

Invalid options

Could you share the full details of go version and go install github.com/cloudspannerecosystem/spanner-cli@latest output?

Hi,
I tried to install spanner-cli on Linux. There were no error messages and it did not install. What am I doing wrong ?
 Definitely needed. Yes, I set the PATH correctly. That is how it is able to find go. Since whereis is returning nothing - I think there is an install failure. First time I am using something where go needed to be installed first, so I could be doing something wrong.
Any help appreciated.
Thanks.
Sundar Mudupalli

@sundar-mudupalli-work Could you run the following commands?
$ go env
$ ls ${GOPATH}/
$ ls ${GOPATH}/bin

Furuyama-san,
Yes, that is right - that was the issue. I did not set my path to include $HOME/go/bin. For first time users like me, could you add this to the readme. This was not mentioned in the go install link on your site - it is possible it is common knowledge for Go users.
I checked out the tool and it will work great for the things I do - working with many databases at once - usually helping customers migrate from one to another.
Thank you.
Sundar Mudupalli

I'm glad that you can solve the issue. Sure, I'll update README accordingly.
The issue was solved, so I'll be closing this issue. Thanks!

This was super helpful for me to find. 😞 Spent to much time trying to figure out why i didn't have it.


travis: only require sudo on docker/i686 builds

Signed-off-by: Paul Osborne<EMAIL_ADDRESS>

Based on http://stackoverflow.com/questions/34668468/use-sudo-true-with-single-element-of-build-matrix-in-travis-ci

With the added coverage, this certainly doesn't make things faster (especially at peak hours), but it is faster than it would be for the same amount of coverage using sudo for everything.
r? @kamalmarhubi

Ping @kamalmarhubi

Ooh! I didn't know you could specify sudo per-matrix entry. Nice!
@homu r+ a6967d6

:zap: Test exempted - status


Comparison Operators

This pull request aims to add support for -gt, -lt, -ge, -le and (i/c) variants for most primitive types.  This tracks well against the built-in Powershell functionality via the added ReferenceTests.  It is implemented via the various IComparer::Default or case relevant StringComparer functions.  There are two added reference tests that are failing on the Pash side still.  I wasn't sure how best to handle them at this time.
"a" -clt "A" == False in Pash, is True in Powershell
"a" -cge "A" == True in Pash, is False in Powershell
"a" -cgt "A" == True in Pash, is False in Powershell
These are all related and it looks like Powershell is forcing lowercase to be less than uppercase.  Most days of the week "A" == 65 and "a" == 97 and "a" > "A" when comparing these as ASCII ordinals.  I'm happy to continue to preserve the output of StringComparer instead of forcing the same Powershell behaviour.  The floor is open for discussion on this.  At a minimum if we keep this behaviour I should mark these specific tests as failing/ignoring (how do we do this)?
resolves: #373

We should match the real PowerShell behavior.

I agree with Jay.
Although I'm not a fan of this kind of strange behavior, the goal of being PS-compatibillty has higher priority than doing it "right".
Is anybody aware of how the Powershell internal string comparison works?

Inverting the string case prior to the comparison fixes this.  I've updated the PR.

Looks good to me :+1:

Good stuff.
One concern: There's a bunch of duplication between comments and code. Would you be willing to eliminate that duplication?


Redirect to https

SSL cert is in place and DNS is updated to point at the right heroku endpoint. Just need to force https
@jszwedko - let me know if you need anything else

http://adoptadrain.sfwater.org/ redirects to https://adoptadrain.sfwater.org/ now


Please add inverter type 702 für DS3-S

Adding ECU-B gave me the following log entry:
ERROR (MainThread) [custom_components.apsystems_ecur.config_flow] APSystemsInvalidData exception: Unsupported inverter type 702 please create GitHub issue.

My APsystem solar was just installed and I'm getting the same HA log entry as @alpapst above.
APSystemsInvalidData exception: Unsupported inverter type 702 please create GitHub issue.
Traceback (most recent call last):
File "/config/custom_components/apsystems_ecur/config_flow.py", line 45, in async_step_user
test_query = await self.hass.async_add_executor_job(ap_ecu.query_ecu)
File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/config/custom_components/apsystems_ecur/APSystemsSocket.py", line 158, in query_ecu
data = self.process_inverter_data()
File "/config/custom_components/apsystems_ecur/APSystemsSocket.py", line 348, in process_inverter_data
raise APSystemsInvalidData(error)
custom_components.apsystems_ecur.APSystemsSocket.APSystemsInvalidData: Unsupported inverter type 702 please create GitHub issue.
Thanks for your work.

Sorry for my late response, I was on vacation. There are two ways for solving this.

Edit APSystemsSocket.py on line 48 add the 702 inverter type (restart home assistant afterwards)
recommended method: download my repository at https://github.com/HAEdwin/homeassistant-apsystems_ecur/archive/refs/heads/main.zip and replace APSystemsSocket.py (restart home assistant afterwards)

@alpapst @imonlinux  Did my response solve you issues? If so @alpapst please close this issue for I'm not able to moderate this. Thank you!

Thanks for the instructions. I just made the change and it is now working as expected.
Thanks again for your work.

@alpapst please close this issue for I'm not able to moderate this. Thank you!

@HAEdwin Sorry for the late response, I was on vacation and had no access to my GitHub account. Thank you for your support!


AIP-6005 Use PVC as tmp path in S3()

depends on KF Profiles addition of METAFLOW_ARTIFACT_LOCALROOT env variable in https://gitlab.zgtools.net/analytics/artificial-intelligence/ai-platform/aip-infrastructure/kubeflow-k8s-profiles/-/merge_requests/530

Is the file src/kfp expected in this PR?

Is the file src/kfp expected in this PR?
not at all! nice catch! This must have been added by VSCode PR creation.


Zip encoding

Fixes #597. We pass zipfile the Cloudstorage file stream so we don't hold all the contents in memory.
Tried it on Hog and successfully downloaded the zip (98M). Extracted and got some duplicate files (a lot, actually). They look like this:
replace ****@berkeley-edu-2015-09-10_05:50:37-246979/group_members_****.json? [y]es, [n]o, [A]ll, [N]one, [r]ename:

This looks good to merge once the the merge conflicts are fixed! Thanks @knrafto

        File "/base/data/home/apps/s~ok-server/staging.387415767741270679/app/utils.py", line 612, in subms_to_gcs
          add_subm_to_zip(subm, Submission, zipfile, result)
        File "/base/data/home/apps/s~ok-server/staging.387415767741270679/app/utils.py", line 557, in add_subm_to_zip
          name, file_contents = subm.data_for_zip(result)
        File "/base/data/home/apps/s~ok-server/staging.387415767741270679/app/api.py", line 1016, in data_for_zip
          messages = obj.get_messages()
      AttributeError: 'FinalSubmission' object has no attribute 'get_messages'

Ugh. Should we use FinalSubmission.revision or .submission?

Let's get the actual submission. I can't see us batch downloading revisions anytime soon.

What's the difference?

Revisions are the composition revisions (resubmits well after the deadline)

Thanks for catching it. Can you test?

It failed Travis.
I think you have to use ModelProxy in utils.
Traceback (most recent call last):
File "/home/travis/build/Cal-CS-61A-Staff/ok/server/tests/integration/test_utils.py", line 66, in test_add_subm_to_zip
zipfile = utils.add_subm_to_zip(subm, result.class, zipfile, result)
File "/home/travis/build/Cal-CS-61A-Staff/ok/server/app/utils.py", line 567, in add_subm_to_zip
if isinstance(submission, FinalSubmission):
NameError: global name 'FinalSubmission' is not defined
--
Sumukh Sridhara
On September 26, 2015 at 6:03:52 PM, Kyle Raftogianis<EMAIL_ADDRESS>wrote:
Thanks for catching it. Can you test?
—
Reply to this email directly or view it on GitHub.

Got an encoding error again. I don't get it:
  File "/base/data/home/apps/s~ok-server/staging.387437672181890858/app/utils.py", line 626, in subms_to_gcs
    add_subm_to_zip(subm, zipfile, result)
  File "/base/data/home/apps/s~ok-server/staging.387437672181890858/app/utils.py", line 572, in add_subm_to_zip
    return add_to_zip(zipfile, file_contents, name)
  File "/base/data/home/apps/s~ok-server/staging.387437672181890858/app/utils.py", line 132, in add_to_zip
    str(contents).encode('utf-8'))
UnicodeEncodeError: 'ascii' codec can't encode characters in position 2103-2105: ordinal not in range(128)

That did the trick. Ready to merge now!

Deployed and used successfully!


Option to use YAML for config files

Sorry, but I have to disagree with this.
I wouldn't want to pull a YAML dependency into my project unless I'm using YAML files. I do actually use YAML files in my fleek project, but I still think it should be optional.
I'll go one step further (perhaps too far) and suggest that maybe fleek shouldn't load from files at all, because:

I have came across issues to do with the current working directory, relative paths, etc when working with some of the fleek modules in the past. Sure, issues like that can be fixed, but...
It is easy to require('./swagger.json') or use a YAML library and load the spec file into an object, and then pass that object into fleek.

Here's what I've done in my project to support both JSON and YAML documents:
// Loads one spec file and returns it as an object.
function loadSpecFile(specPath) {
  switch (path.extname(specPath)) {
    case '.json': {
      return require(specPath);
    }
    case '.yml':
    case '.yaml': {
      return yaml.safeLoad(fs.readFileSync(specPath, 'utf8'));
    }
    default: {
      throw new Error('unsupported spec type: ' + specPath);
    }
  }
}

The result gets passed to fleek-router as the swagger document.


NullReferenceException ... Microsoft.MixedReality.Toolkit.Teleport.TeleportCursor.SetVisibility

Describe the bug
Our unity log on the hololens was thowing the error over and over - despite the Teleportation system being toggled off
NullReferenceException: Object reference not set to an instance of an object.
at UnityEngine.Component.get_gameObject () [0x00000] in <00000000000000000000000000000000>:0
at Microsoft.MixedReality.Toolkit.Teleport.TeleportCursor.SetVisibility (System.Boolean visible) [0x00000] in <00000000000000000000000000000000>:0
at Microsoft.MixedReality.Toolkit.Utilities.Solvers.HandConstraint+d__62.MoveNext () [0x00000] in <00000000000000000000000000000000>:0
at UnityEngine.SetupCoroutine.InvokeMoveNext (System.Collections.IEnumerator enumerator, System.IntPtr returnValueAddress) [0x00000] in <00000000000000000000000000000000>:0
at UnityEngine.MonoBehaviour.StartCoroutine (System.Collections.IEnumerator routine) [0x00000] in <00000000000000000000000000000000>:0
at Microsoft.MixedReality.Toolkit.Utilities.Solvers.HandConstraint.SolverUpdate () [0x00000] in <00000000000000000000000000000000>:0
at Microsoft.MixedReality.Toolkit.Utilities.Solvers.Solver.SolverUpdateEntry () [0x00000] in <00000000000000000000000000000000>:0
at Microsoft.MixedReality.Toolkit.Utilities.Solvers.SolverHandler.LateUpdate () [0x00000] in <00000000000000000000000000000000>:0
UnityEngine.MonoBehaviour:StartCoroutine(IEnumerator)
Microsoft.MixedReality.Toolkit.Utilities.Solvers.HandConstraint:SolverUpdate()
Microsoft.MixedReality.Toolkit.Utilities.Solvers.Solver:SolverUpdateEntry()
Microsoft.MixedReality.Toolkit.Utilities.Solvers.SolverHandler:LateUpdate()
To reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Not to get null reference exceptions
Screenshots
If applicable, add screenshots to help explain your problem.
Your setup (please complete the following information)

Unity Version 2019.4.17f
MRTK Version 2.5.1

Target platform (please complete the following information)

HoloLens 2

Additional context
Disabling ParabolicPointer from the Input System's Pointer Options cleared it up.

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all-new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visit https://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


fixing (the) future

@biocore/qiita-dev can we get a couple of reviews here? This fixes the future and new biom issues in qiita.

Shouldn't six be in the setup.py now?

We have been using six for other things AKA I didn't introduce this dependency. My guess is that is being installed by other at installation.

Well, accordingly to the code we where using future.utils.six. If six is now a direct dependency of Qiita, it should be in the setup.py. I think it is not safe to assume that it gets installed by one of our dependencies.

it’s an python dependency, but would still be good to explicitly install it here.

On Oct 28, 2015, at 10:31, Antonio Gonzalez<EMAIL_ADDRESS>wrote:
We have been using six for other things AKA I didn't introduce this dependency. My guess is that is being installed by other at installation.
—
Reply to this email directly or view it on GitHub https://github.com/biocore/qiita/pull/1516#issuecomment-151920024.

I can add if that's what we agree on. However, I feel that we should keep relaying on ipython to install it as this way we are sure to not have any version conflicts, what do you think?

I would recommend against it, in case that Ipython drops such dependency, we will be missing it. However, if others agree with this solution ok with me.

Isn't six part of future?
On Oct 28, 2015 11:43 AM, "Jose Navas"<EMAIL_ADDRESS>wrote:

I would recommend against it, in case that Ipython drops such dependency,
we will be missing it. However, if others agree with this solution ok with
me.
—
Reply to this email directly or view it on GitHub
https://github.com/biocore/qiita/pull/1516#issuecomment-151925269.

Not since 0.14 ...

Pretty sure the string io imports can be made from "io" which is a built in
module. See how we did it in biom
On Oct 28, 2015 12:06 PM, "Antonio Gonzalez"<EMAIL_ADDRESS>wrote:

Not since 0.14 ...
—
Reply to this email directly or view it on GitHub
https://github.com/biocore/qiita/pull/1516#issuecomment-151935834.

Tried io but all tests are using strings and io.StringIO expects unicode. Then realized that in other sections of qiita we were using from six so decided to go that route.

—
Reply to this email directly or view it on GitHub
https://github.com/biocore/qiita/pull/1516#issuecomment-151944635.

Or even importing unicode_literals from future. Good overview of the
advantages / drawbacks of the two approaches here
http://python-future.org/unicode_literals.html
On Oct 28, 2015 12:35 PM, "Jose Navas"<EMAIL_ADDRESS>wrote:

I think you just need to add a u in front of the strings so they're
unicode.
—
Reply to this email directly or view it on GitHub
https://github.com/biocore/qiita/pull/1516#issuecomment-151944635.

I can move back and add a u or use unicode_literals but not sure what's the benefit over six?


two dynamic lists

Does this library support two dynamic lists? For example if I have two different binders that each have a dynamic list, would I need to do anything differently to get it to work?
Currently I'm getting indexing issues when using ListBindAdapter.
What's the best way to do this? thanks in advance

After adding/removing/replacing items in binder, you need to notify dataset changed methods like the following line.
https://github.com/yqritc/RecyclerView-MultipleViewTypesAdapter/blob/master/sample/src/main/java/com/yqritc/recyclerviewmultipleviewtypesadapter/sample/binder/Sample2Binder.java#L54

I realized that my issue was I was overriding the getItemViewType method and that was causing issues with my two different lists since they are dynamic. I ended up removing my own version of the getItemViewType and seems to be working perfectly.
Thanks for the help!!


Repeating group, row deletion, dropdown: ResizeObserver loop completed with undelivered notifications.

Description of the bug
When deleting a row while another row is open for editing in the "pet" repeating group in our test-app, we get the error: "ResizeObserver loop completed with undelivered notifications."
I have not looked deeply into this, but i know we have had issues before with this error message appearing for the comboboxes when they change size. This repeating group contains comboboxes, so that might be worth looking into.
If relevant, fix for ResizeObserver error in DS:
https://github.com/digdir/designsystemet/pull/1594
Steps To Reproduce

Open the app "frontend-test" and go to the repeating group page "kjæledyr".

Additional Information
No response

Re-tested this and it has been resolved by using the DS combobox.


Fixes envoy rock build

The checksum of quiche-envoy-integration changed, causing the envoy build to fail.
xref: https://github.com/envoyproxy/envoy/issues/36563

/cc @bschimke95


[ISSUE] Multinode training issue on two different server having different number of usable GPU!

System Info
Accerlerate

Multinode training

All is the latest version

Information

[ ] The official example scripts
[X] My own modified scripts

Tasks

[ ] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[X] My own task or dataset (give details below)

Reproduction
I have two servers: named by SV1, SV2.
However, I can only use gpu id: 0,1,2,3,4,5,6 in SV1, and I can only use gpu id:2,4 in SV2.
In this case, there pops up error assertion error.
Therefore, I have modified num_processes 14, 4 on each server, but it seems works but this model has stucked with loading DDP on _verify_param~~ function.
Note that, I was not issuing in this github page before trial and error for changing numerous parameter, but I did by myself really bunch of parameter change.

SV1 accel_config.json

compute_environment: LOCAL_MACHINE
debug: false
distributed_type: MULTI_GPU
downcast_bf16: 'no'
gpu_ids: 0,1,2,3,4,5,6
machine_rank: 0
main_process_ip: xxx.xxx.xxx.123
main_process_port: 456
main_training_function: main
mixed_precision: bf16
num_machines: 2
num_processes: 7
rdzv_backend: static
same_network: true
tpu_env: []
tpu_use_cluster: false
tpu_use_sudo: false
use_cpu: false

SV2 accel_config.json

compute_environment: LOCAL_MACHINE
debug: false
distributed_type: MULTI_GPU
downcast_bf16: 'no'
gpu_ids: 2,4
machine_rank: 1
main_process_ip: xxx.xxx.xxx.123
main_process_port: 456
main_training_function: main
mixed_precision: bf16
num_machines: 2
num_processes: 2
rdzv_backend: static
same_network: true
tpu_env: []
tpu_use_cluster: false
tpu_use_sudo: false
use_cpu: false

Expected behavior
I would like to know how to run with two different number of usable GPUs.

I’m not sure this is actually possible. I’ll look into it but I’m unsure.

I believe this bug can be solved by slight modification of DDP or similar sync module. :)

This is because
I observed it works in the following case (unimportant parameters are omiited):

SV1 accel coinfig

gpu_ids: 0,1,2,3
machine_rank: 0
main_process_ip: xxx.xxx.xxx.123
main_process_port: 456
num_machines: 2
num_processes: 8 (due to 4*2 (num of gpus * num of machines) )

SV2 accel config

gpu_ids: 0,1,2
machine_rank: 1
main_process_ip: xxx.xxx.xxx.123
main_process_port: 456
num_machines: 2
num_processes: 6 (due to 3*2 (num of gpus * num of machines) )

How can I know num of gpus * num of machines work?
Once I runned the code where the following case:

SV1 accel config

gpu_ids: 0,1,2,3,4,5
machine_rank: 0
main_process_ip: xxx.xxx.xxx.123
main_process_port: 456
num_machines: 2
num_processes: 8 (due to 6+2 (num of gpus in SV1 + num of gpus in SV2) )

SV2 accel config

gpu_ids: 3,4
machine_rank: 0
main_process_ip: xxx.xxx.xxx.123
main_process_port: 456
num_machines: 2
num_processes: 8 (due to 6+2 (num of gpus in SV1 + num of gpus in SV2) )

Then, I surprisingly observed the following nvidia-smi.

SV1 nvidia-smi

0 memory alloc
1 memory alloc
2 memory alloc
3 memory not alloc
4 memory not alloc
5 memory not alloc

SV2 nvidia-smi

3 memory alloc
4 memory alloc

Then, I guessed why this happends and I inferred if the number of gpu (= number_of_process / num_machines) is allocated?.
Then my guessing is almost correct in other case.
But, it is not the official way, therefore I would like to know how we can run it in official way.


Update difference-of-squares tests

Adds two missing tests as well as a stub for the student.

I think I ran afoul of how the test runner pastes the example into the stub where the tests are defined. I'll troubleshoot later this week.


Reapply "[native] Real shared libraries in APK /lib directories (#9154)"

Revert b795a653460417af9da75f94e4a487a2665faa7f (Reapply 5ebcb1dd1503648391e3c0548200495f634d90c6)
Cherry-pick 230ebb0a54fde42dc160c7aadf5744ef3199c675
Cherry-pick 3fbc4510b0ef4c36589cbccac0e118e9c0a20eaa

Use rebase-and-merge to merge this, so we get 3 commits.
MAUI Test Integration should hopefully pass to know MAUI will build after merging.

Latest changes, I just fixed up the .apkdesc files that had conflicts.


What is the max dimension for binary vectors?

Currently the milvus limitation is 32768 for binary vector, but I believe this is for float vector?
https://github.com/milvus-io/milvus/issues/30285
What should be the right limitation?

@xiaofan-luan
knowhere has no dimension limitation for both float and binary vector

@xiaofan-luan knowhere has no dimension limitation for both float and binary vector

Then please check and fix on Milvus side?

close it for now.
it is 32768 * 8


Domains that my organization should allow to use CC

Hi,
My organization plans to block access to unnecessary domains.
We want to block .blob.core.windows.net, will this affect CC usage?
Regards.

Hi @MK202204,
Thanks for raising the query. We will discuss with our team and get back to you with proper response soon.

Hi @gsv022 & Team,
When I checked the wiki, it was stated that CC uses Azure Table Storage as a data store, so access to ".table.core.windows.net" is necessary, but access to ".blob.core.windows.net" is unnecessary.
Please let me know if this understanding is wrong.
Thankyou.

@MK202204 ,
We do not recommend making any changes to the deployed Azure resources as it may impact the overall functionality. Our finding says that there is a dependency on blob storage as well. Hence disabling blob.core.windows.net will effect the functionality.
Please let us know if have any queries.

@gsv022
Thankyou for your reply.
OK.Rather than allowing *.blob.core.windows.net, I want to allow only domains used in my organization.
I think that specifying the storage account as shown below will not effect the functionality.
https://.blob.core.windows.net
How about this?
please let me know your opinion.
Thanks.

@MK202204 , Ideally this should work. But you need to check it on your own. We can't give assurance on this.

@MK202204 ,Please let us know if you have any queries related to CC. Otherwise we can make it to closure.

@gsv022
Thankyou for your reply.
I understand your answer.
Please close this thread.


docs(changelog): implement conventional commits preset

build(deps): add conventionalcommits preset
docs(changelog): adds conventional commit settings

Description

Improves the generation of the changelog by using the conventionalcommits preset.
This will allow us to generate a changelog with information about the changes made in a specific release.
How has this been tested?

Ran the semantic-release command locally as a dry run and checked the changelog generated.
Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed
[ ] I have assigned a reviewer for this PR - no other reviewers available
[x] I have assigned myself as the PR assignee
[x] I have added the relevant labels to the PR
[x] I have linked all the relevant issues and project to the PR - no issues available
[x] I have made sure to follow the Conventional Commits Specification for my PR and commit titles

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat(types): provide TypeScript types

Refs #12

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Bcrypt - Template function

Is your feature request related to a problem? Please describe.
I would like to start managing ArgoCD as IAC from my primary cloud platform - AWS.   In order to create and manage access passwords for ArgoCD for users in secrets manager, the application uses bcrypt hashes of passwords in the secret it references.
I am unable to transform the bcrypt version of the password and so now have to stand up a "bcrypt the value in secrets manager" policy which is not easily captured in configuration compared to something like { bcrypt .password }.
Describe the solution you'd like
I would like to have a brcypt function available that basically proxies the golang bycrypt library GenerateFromPassword function.
Example usage:
// Using default cost
{ bcrypt .password }
// Using specified cost
{ bcrypt .password 4 }

Describe alternatives you've considered
The alternative I will be going with is to create a CDK construct for "Argo Passwords" that bcrypts the secret in the external source.  There will be manual documentation around this for myself and others to remember that updating this secret requires a transform step.
Additional context
N/A

Just stumbled across the exact same issue and would love this being implemented 😄

Just want to add here, that I went looking into the source and it looks like sprig currently does bcrypt with default cost value set using the golang bcypt library.
{{ .secretValue | bcrypt }}

You can find the implementation here: https://github.com/Masterminds/sprig/blob/581758eb7d96ae4d113649668fa96acc74d46e7f/crypto.go#L54


Bug fix: improve templater option reliability

This pull request improves the reliability and speed of the templater option.
The following things are addressed:

unsaved text is no longer lost when pressing the button (fixes #95)
template reliably remains in the button definition, not overwritten (fixes #82)
await-ed Templater prompt commands now work (fixes #70 and possibly part of #59?)
proper order of operations is guaranteed through use of await instead of setTimeout

as a side bonus, the operations happen more quickly.

I appreciate that you're working on a new version of Buttons but I think this fix might benefit others in the meantime if you would be willing to release it.

This would probably solve the issue of briefly flashing and not working with templater?

https://github.com/shabegom/buttons/issues/191
https://github.com/shabegom/buttons/issues/173
https://github.com/shabegom/buttons/issues/184
https://github.com/shabegom/buttons/issues/119

Potentially, a smaller PR without conflicts could also be merged to resolve the issues above:
https://github.com/shabegom/buttons/pull/169


[WIP] PR: Add SpyderPlugin class to public API

This adds a new class to our API (SpyderPlugin) to be able to create plugins without an associated dockwidget. Or in other words, non-graphical plugins. Pinging @jitseniesen who has been asking for this for a long time.
This is needed to make LSPManager a proper plugin, so it's part of #7724.
Note: This is still a WIP, but I'm not making it a draft PR because Travis doesn't run on those.
TODO:

[x] Move most of BasePluginWidget __init__ to BasePluginWidgetMixin __init__
[ ] Don't register plugin if check_compatibility fails
[x] Actually add SpyderPlugin
[ ] Add default code for register_plugin.

Hello @ccordoba12! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/api/plugins.py:

Line 47:5: E265 block comment should start with '# '

@CAM-Gerlach, the docstrings in the api module are meant to be read by developers creating their plugins, so they don't need to follow PEP 257.

However, I'll agree with some of your suggested changes.

the docstrings in the api module are meant to be read by developers creating their plugins, so they don't need to follow PEP 257.

That's exactly when following PEP 257 is most important, in the template of a public API. I don't see how it, of all things, merits a contrived exception.

Because by making the docstrings shorter to follow Pep 257 we lose clarity.

@goanpeca, this is ready for review. I updated the PR description to explain what I've done here.

@andfoy, I'd like to know your opinion on this PR. What do you think about the API I came up with here? Please read the description to understand my rationale and read it as an external developer to see if it's clear enough to build your own plugins.

This is ready. We can later revisit how the API was defined, but right now we need this so that @andfoy can continue working.


update readme w/ comparison to flake-parts

Hi, I recently stumbled on this tool while searching for alternatives to flake-parts. I found this post on discourse that compares the difference between the tools.
I took the text and added it to readme since I think it could be helpful to refer to these differences when learning about the project.
Feel free to edit/suggest changes, thanks!

I don't think this should be added to the readme. If theres strengths of flakelight that are missing, that can be added to the features list.
I feel that adding a comparison to another library to the readme sets the stage as if the library exists to compete with the the other, rather than to stand on its own. Its also something that is likely to go out of date as other libraries evolve, and is unlikely to be updated as that would require keeping up with flake-parts development. It could be also interpreted as antagonism towards flake-parts. As such, it doesn't feel correct to me to have this in the readme.
It is valuable information for someone evaluating the library though, which is why I made the discourse post. Perhaps we should just link the discorse page from the readme.

@accelbread Makes sense, agree with your thoughts. I updated the PR, thanks.

Thanks!


Some improvements

Nice work. How about setting MTBF to seconds rather than days. Since currently it is an integer, you can't go below daily restarts.
You should also consider whitelisting namespaces.
Why are you setting mbtf on a label rather than in the config map?
When killing pods, it would be nice to know their replication scale to find out their kill frequency.

How about setting MTBF to seconds rather than days. Since currently it is an integer, you can't go below daily restarts.

Having a termination frequency in seconds feels out of scope of the intention of this project. The idea of Kube Monkey (really, Chaos Monkey) is to simulate failure scenarios in a production environment. Having failures occur every few seconds will be counter-productive.

You should also consider whitelisting namespaces.
Currently Kube Monkey works only on an opt-in basis so having a Whitelist has limited benefits. I do like the idea of allowing Kube monkey to function on an 'opt-out' basis as well, in which case a whitelist would definitely make a lot more sense.

Why are you setting mbtf on a label rather than in the config map?
MTBF is configured per app (Kube deployment). Setting it via a label means that Deployment owners can set the parameters of Kube Monkey for their app without having to re-configure Kube Monkey itself. So, if I run and operate Kube Monkey in your environment for all services running in that environment, you as a service owner can independently tune the parameters for how Kube Monkey handles your app without having to go through me.

When killing pods, it would be nice to know their replication scale to find out their kill frequency.
I'm not sure I understand you on this. Could you elaborate.

Closing since I haven't heard back. Feel free to re-open if you have a follow up.


Attempting to fix binary memory leak in ETS flusher

Coverage: 78.771% (+0.04%) from 78.731% when pulling 44605d6541885d292d9665c1dd36649ef77de618 on binary_memory_leak_in_flusher into 6a002fb029cf00eeadfcfef7ce984cd7a0f83493 on master.


introduce Repository layer

The Repository is there for connecting Redis and Azure Blob.
The base has at lest three interfaces;
Task<IEnumerable<T>> FindAllAsync()
Task UpdateAsync<T>(T entity)
Task AddAsync<T>(T entity)

Also the base repository has one protected interface:
Task<IEnumerable<T>> FindAllWithNoCacheAsync,
all three public interface will internally call this interface, to make sure the data is always up to date.


Both: Example renewal notification letter / email / text

link to #452

I believe v1 of this has been done and is being tested

@EUzkuraityte @sanjaypoyzer @ingasiler I've put all the confirmation & reminder emails / texts in here: https://drive.google.com/drive/folders/0B_rqDnauwjr8RFd5eURIS0tJa1k Outstanding: Letter sent by post.


Add jsx extension to transform and testMatch in jest-preset

Summary
A lot of time people will write react components and tests in files with extension .jsx. So, tests ending with this extension don't run and the component files written with this extension don't get transpiled.
Changelog

[General] [Fixed] - Add jsx extension to transform and testMatch in jest-preset
Test Plan

Please change your own Jest configuration in your project template to overwrite this stuff in the template. I think we don't want to promote using jsx specifically in RN.

Ya sure 👍. But a bit curious, any specific reasons for not promoting the use of jsx extension with RN.

@cpojer - Could you please clarify your comment when you get a chance? Is there a reason why you don't want to promote jsx in RN?  Since you have tsx allowed in the babel preset, are you gently nudging people to use tsx instead?

@navilan Absolutely! Originally, jsx was invented because people weren't used to compiling their plain js files and it wasn't common to add syntax extensions to JS. It was a way to get the community to accept custom flavors of JavaScript. The reason tsx exists is because originally TypeScript's syntax conflicted with JSX. That's why they invented tsx which doesn't have this incompatibility.
Nowadays, people compile their js files. I do not think we need to promote usage of jsx over js.  As you can see, tsx is still necessary.
Note that not including this change into React Native only means this won't be a default but you can still change your project to include jsx files across your project.

Thanks for the clear reply @cpojer. Much appreciated.


Nix build: 'eta-build-shell' not found

Following the installation instructions for installing eta with Nix fails.
Description

I tried running
$ git clone --recursive --depth 1 https://github.com/typelead/eta
$ cd eta
$ nix-shell -A eta-build-shell

Expected Behavior

It should start the interactive nix shell.
Actual Behavior

The command fails with the message:

error: attribute 'eta-build-shell' in selection path 'eta-build-shell' not found

Steps to Reproduce
Follow the eta installation instructions with Nix.
Context
I was trying to install eta.
Your Environment

Did you install an older version of Eta/Etlas before?
No

Current Eta & Etlas version:

Operating System and version:
Debian 4.9.110-3+deb9u6 (2018-10-08)

Link to your project:

@puffnfresh Do the instructions need to be updated?

Yes, the updated instructions are here:
https://github.com/typelead/eta/blob/master/docs/0-user-guides/0-eta-user-guide/1-installation/5-nix.md
I think the website needs to be regenerated.

Should be updated now, closing.


[BEAM-12898] Minor fix to Kubernetes.groovy postBuildScript.

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

run seed job

R: @ibzib

LGTM
Can you run a oneshot using a manual trigger to verify this is correct before submitting?

Run BigQueryIO Batch Performance Test Java Avro

Run Java InfluxDbIO Performance Test

Run Java InfluxDbIO Performance Test

Looks like the test was running but not showing up in Github: https://ci-beam.apache.org/job/beam_PerformanceTests_InfluxDbIO_IT/915/console

Run Java InfluxDbIO Performance Test

It's fine if the test fails. It looks like it's running the post build script correctly regardless.

this is passing but not reflected on github https://ci-beam.apache.org/job/beam_PerformanceTests_InfluxDbIO_IT/918/


Cuisine module for cockroachdb

To work with gowncloud

moved to https://github.com/Jumpscale/prefab9/issues/5


Segfaults on master

➜  try_gluon git:(master) ✗ lldb cargo run
(lldb) target create "cargo"
Current executable set to 'cargo' (x86_64).
(lldb) settings set -- target.run-args  "run"
(lldb) r
Process 67725 launched: '/Users/brendan/.cargo/bin/cargo' (x86_64)
Process 67725 stopped
* thread #2, stop reason = exec
    frame #0: 0x0000000100b0419c dyld`_dyld_start
dyld`_dyld_start:
->  0x100b0419c <+0>: popq   %rdi
    0x100b0419d <+1>: pushq  $0x0
    0x100b0419f <+3>: movq   %rsp, %rbp
    0x100b041a2 <+6>: andq   $-0x10, %rsp
Target 0: (cargo) stopped.
(lldb) bt
* thread #2, stop reason = exec
  * frame #0: 0x0000000100b0419c dyld`_dyld_start
(lldb)

That is a strange backtrace. Are you sure it is not cargo that crashes? try_gluon itself shouldn't do anything with dynamic libraries.

Since it runs fine for me I am 95% certain this segfault was fixed by https://github.com/gluon-lang/gluon/pull/398 .
Closing.


feat(tracing): add tracing config schema

From now on we can use "$ref": "https://ory.sh/schemas/tracing.schema.json" in the config schemas instead of having to copy it around.
See related https://github.com/ory/keto/issues/662

That is unfortunately not a good idea. I have done this mistake in Ory Oathkeeper before. Here are the reasons:

You are referencing a non-versioned URL. This will eventually update, be refactored, changed. All older builds will no longer work as soon as this changes. If you want to reference a URL, it must be versioned (like a git hash).
You are referencing a HTTP URL. This means that Ory Keto will not work in envs without network access. We got hit by this in Ory Oathkeeper as well

If you want to centralize the config, it needs to be ensured that it uses embed somehow, and in particular not referencing any URL, in order to get reproducible builds.
Otherwise, this LGTM afaict


Create types corresponding to evaluation results

Currently there are only types corresponding to Logo tokens. We also need a result type that will represent the outcome of evaluating an expression or result.

Turns out it was not needed. I decided to overload the LogoToken type to represent the results of an evaluation. Strictly speaking, a valid evaluation can only use a subset of constructors defined by LogoToken. Once I learn to use GADTs better, I might be able to have a more robust type definitions. Till then this should suffice.
Check out changes made in 56b374109


[cloud_functions] App crashes on iOS when UInt8List is used as parameter

Describe the bug
iOS app crashes when Uint8List is used as parameter.
On Android  FormatException: Invalid envelope is thrown.
To Reproduce
Call a Firebase Cloud Function with an Uint8List as parameter.
Expected behavior
As an Uint8List basically is (implements) List<int>, I would expect Uint8List to work as parameter
Additional context
Currently possible workaround is to use List.from(myUint8List).

Beware when I implemented the workaround mentioned above worked on the flutter side, but on Node cloud function side I had to convert the Array (List) back to a buffer:
Buffer.from(list);

I don't think it's possible to pass a Uint8List parameter to a native platform without wrapping it. See here

Your statement is false. Uint8List is specifically listed here: https://flutter.dev/docs/development/platform-integration/platform-channels#codec
Further, the link you provided is not helpful at all. In fact, the Stackoverflow specifically shows that passing Uint8List ist possible. https://stackoverflow.com/a/58869349

Apologies, will reopen. I guess that may have been true in that moment in time. Just to clarify, I didn't say it wasn't possible, just that it had to be wrapped i.e. placed within a List.

Alright, sorry for being a little rude.
I think in httpsFunctionCall, parameters containing FlutterStandardTypedData  types require special treatment. https://github.com/FirebaseExtended/flutterfire/blob/master/packages/cloud_functions/cloud_functions/ios/Classes/FLTFirebaseFunctionsPlugin.m#L74

Hey @lschmierer,  I tried  to pass through an obj-c representation of a buffer as a parameter of the cloud function call here:

// sent an object "parameter" from dart code like so: { "list": Uint8List(100) }
  NSData * buffer = [[parameters objectForKey:@"list"] data];

  NSDictionary * params = @{
    @"list": buffer
  };

  [function callWithObject:params
                completion:^(FIRHTTPSCallableResult *callableResult, NSError *error) {
                  if (error) {
                    result.error(nil, nil, nil, error);
                  } else {
                    result.success(callableResult.data);
                  }
                }];

But it still loses connection to device. Looking at the iOS for cloud functions documentation, I don't think a buffer is an accepted parameter. The same appears true for the android cloud function call. Not sure if you have any thoughts on a viable solution? If not, I will update the documentation with accepted parameter types to warn users.

So maybe check if the parameter is FlutterStandardTypedData and if so copy from the buffer to an array?
NSData *data = ...

UInt8 buf[data.length];
[data getBytes:buf length:data.length];

The copy needs to happen anyway. Using  List.from(myUint8List) it happens on the Dart side.
Int32List, Int64List and Float64List Dart types will probably require the same handling as they are all represented as FlutterStandardTypedData (wrapped NSData).
On Android bye[], int[], long[] and double[] need to be converted to List<> types.
Like
  Byte[] array = ....
  List<Byte> list = Arrays.asList(array);

which is zero copy, I think.


feat: add qgis example with earthquake data

Also, you should rename to examples/geopandas. QGIS is a GUI application 😄.

Also, you should rename to examples/geopandas. QGIS is a GUI application 😄.

Ah yeah if you run pixi run start it should actually start the GUI. If not please let me know.


ERROR: relation "zulip.zerver_message" does not exist

Using this manual
https://zulip.readthedocs.io/en/stable/production/install.html
after running
./zulip-server-*/scripts/setup/install --certbot --[email=emoxam<EMAIL_ADDRESS>--hostname=domain.example.com

i get error
+ POSTGRES_USER=postgres
++ crudini --get /etc/zulip/zulip.conf postgresql database_name
++ echo zulip
+ DATABASE_NAME=zulip
++ crudini --get /etc/zulip/zulip.conf postgresql database_user
++ echo zulip
+ DATABASE_USER=zulip
++ su postgres -c 'cd / && psql -v ON_ERROR_STOP=1 -Atc "SELECT 1 FROM pg_database WHERE datname='\''zulip'\'';"'
+ '[' 1 = 1 ']'
++ su postgres -c 'cd / && psql -v ON_ERROR_STOP=1 -Atc '\''SELECT COUNT(*) FROM zulip.zerver_message;'\'' zulip'
ERROR:  relation "zulip.zerver_message" does not exist
LINE 1: SELECT COUNT(*) FROM zulip.zerver_message;
                             ^
+ records=

Zulip installation failed (exit code 1)!

The install process is designed to be idempotent, so you can retry after resolving whatever issue caused the failure (there should be a traceback above). A log of this installation is available in /var/log/zulip/install.log`

It might help to stop by "production help" on chat.zulip.org and explain how you got here; it looks like you unexpectedly have a database called zulip on your postgres instance, but it doesn't contain one of the typical tables -- did you create that manually?
In any case, the failure is in logic trying to check if your database doesn't have existing Zulip server content that we'd prefer not to delete without prompting.

It seems pointing to PSQL 15 fixed the problem..
./zulip-server-*/scripts/setup/install --certbot<EMAIL_ADDRESS>--hostname=zulip.example.com --postgresql-version=15
Anyway, i didn't create any base because if installer see any PSQL installed it gives a command to delete it and doesn't continue without this.

@emoxam: That "solved" the problem by making a clean PostgreSQL cluster which didn't have the empty zulip database in it.
Pushed #29242 to make this still work, however you got into the situation.


Scenario Report - Bar Chart doesn't render correctly

For each scenario, it renders a bar chart for overview of each step. However, the chart doesn't render a bar for each step.
Some links from the sample report:
Link 1
Link 2
Link 1
Thanks

It actually does. But the scale is one second, so you don't see the quick steps.

Is there a way, we can set the configuration. So that the scale will get adjusted automatically based on the execution duration? That wauy, it will populate all the bars.

Sure that would be a possibility. There is no way to set this from the outside at the moment but if you want to create a PR, feel free to do so. Otherwise, we can keep this in mind for a future release.

Fixed in 2.9.0


VRTK_Pointer requires the VRTK_ControllerEvents component warning shown when ControllerEvents is already added

Environment
VRTK Git 2065a7f9e3c202b480dac80c1416d2b2d39a584d
Unity 5.5.1f1
Vive
SteamVR 1.2.1
Steps to reproduce
Start with Scene: 034_Controls_InteractingWithUnityUI
Open scene Additive: 036_Controller_CustomCompoundPointer
from scene 36 copy component: VRTK->RightController->VRTK_Pointer
paste it in scene 34 to:  VRTK->RightController
Change Button_Activation and Selection_Button on the new script to: Button_Two_Press
from scene 36 copy component: VRTK->RightController->VRTK_BezierPointerRenderer
paste it in scene 34 to:  VRTK->RightController
Link the pointer renderer from the new pointer to the VRTK_BezierPointerRenderer
Remove scene 36
At this point the first controller works as expected
Where things go wrong
Delete VRTK->LeftController
Duplicate VRTK->RightController
Rename it to LeftController
Drag the reference for left controller to reference on the VRTK object
Now none of the cursors work, and this error shows up repeatedly, even though Controller Events are on both parent objects.

Expected behavior
After Duplicating the right controller settings for the left controller, and setting the references both controllers should behave the same way.
Current behavior
After Duplicating controller settings, all UI Pointers are broken, and complain about references not existing that do.

This seems like a bit of a weird set up.
Why are you opening a scene to copy the pointer into a scene that already contains a pointer?

One pointer is for the compound bezeier pointer for teleporting, the other is the pointer for interacting with the ui.
I copied the other pointer from another scene so I wouldn't need to describe the settings I had on the other pointer, so you'd end up with the exact same setup I had. You could just as easily attach the same pointer script, instead of copying it from the other scene.

Can you provide simple steps to reproduce where you're not copying one script from another scene, nor duplicating game objects.
I'm not entirely sure what the issue is. It reads like it's a configuration issue which is likely caused by trying to copy/paste things.
Can you recreate the issue with just setting up scripts from new?


Update how-to-pronunciation-assessment.md

suggest to land on tech blog

@Melissa-Ma-speech : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

#sign-off

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Allow experimenter to configure API details in the front-end (without modifying config file)

This should carry over across experiments - perhaps an "API key" field (or API key per supported language model) in Settings?
Also add validation upon adding an "LLM Mediator" during the chat component in case the experimenter tries to add a mediator without having already added their key - e.g. "You don't have a Gemini 3.5 API key listed! Please do so in Settings."

Fixed by refactor


Find a more standard terminal than foot

The default installation comes with an initial set of applications configured as launchers in the dock.
Ideally, these applications are referring to tools that are widely available -- terminal, web browser, ... .
As of 2024-10-28, the pre-configured launcher applications are Chrome (with the Wayland flags), Firefox (with the Wayland environment variable) and foot as a terminal.
Neither is eg. installed by default on Debian.

Potential source of inspiration: https://anarc.at/blog/2022-09-19-wayland-terminal-emulators/


Feature request : need ability to retrieve full bundle at once

Hi,
in our project, we make a lot of calls to the 'translate' function with several times the same keys.
It would be preferable for us to translate a full bundle once, and then manually associate the translated labels to our objects.
For ex, let's say i have a list of users and for each, i want to display '<user_preferred_greeting> ' with  <user_preferred_greeting> being translated.
On the html i will have a ng-repeat containing :
{{user.name}}
But if i have hundreds of users with only 3 different greetings, i'll have hundreds of requests to the gp server.
In this case, it would be more efficient to translate the whole bundle containing the 3 greetings translated, store them locally and use something like :
{{ctrl.translatedGreetings[user.preferred_greeting]}} {{user.name}}
Although I noticed that for each translation, the whole bundle is requested from the server. So i feel like it wouldn't be too much of a change to add this feature (from what i've seen at least ;) ). I'd be happy to give it a try if you think that can be an interesting feature to add.
thanks

Hey there -- Great to see you using the gp-angular client for G11n Pipeline. @srl295 and @yumaoka and @steveatkin what do you think? Let's start the conversation.

sounds like some good ideas… let me look at what this would take.  Also cc @ckoberlein


Deprecate --charts flag in favor or --manifests

--charts is confusing because there's a manifests/charts subdir.

@ostromart: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
a17a71cc768dad15964c4f9a3db1aad140861ca1
link
/test gencheck_istio

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

@ostromart: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
a17a71cc768dad15964c4f9a3db1aad140861ca1
link
/test gencheck_istio

unit-tests_istio
a17a71cc768dad15964c4f9a3db1aad140861ca1
link
/test unit-tests_istio

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

@ostromart: PR needs rebase.

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

@ostromart: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
a17a71cc768dad15964c4f9a3db1aad140861ca1
link
/test gencheck_istio

unit-tests_istio
a17a71cc768dad15964c4f9a3db1aad140861ca1
link
/test unit-tests_istio

integ-ipv6-k8s-tests_istio
a17a71cc768dad15964c4f9a3db1aad140861ca1
link
/test integ-ipv6-k8s-tests_istio

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

@ostromart: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

analyze-tests_istio
f2e04cf0f7fc220c2ccd12ba6178225553c2707e
link
/test analyze-tests_istio

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


How do I upload files and hidden input values directly to s3 with detailed control?

Type of issue

[ ] Bug report
[ ] Feature request
[x] Support request

Uploader type

[ ] Traditional
[x] S3
[ ] Azure

Support Request
Fine Uploader version
5.14.3
Question
I have read everything I can find in the documentation, including all of the setup documentation for s3 and the "form support" page.
Here is my scenario:
I have a form which allows users to attach up to five images. Those images are uploaded to Heroku via a standard form submission, which often times out. Because of that, I need to use direct uploads to s3. The images are managed on the Rails server using the carrierwave gem.
Here is the plan I have come up with:
I would like to upload each file directly to s3, and in a callback, modify my HTML. I need to use the s3 url as the value of a new hidden input which will be submitted in the form to my server and used by carrierwave as a remote image url. Some of my forms also have images which are base64 encoded as values of hidden inputs.
If someone changes the file chosen with a file input, I need to be able to upload that file again, processing the callback with the url, etc.

I have been reading the documentation for hours (as well as other days) and this is as far as I have gotten:
var uploader = new qq.s3.FineUploaderBasic({
  autoUpload: false,
  request: {
    endpoint: 'https://' + s3BucketName + '.s3.amazonaws.com',
    accessKey: accessKey
  },
  signature: {
    endpoint: '/s3/signature'
  },
  uploadSuccess: {
    endpoint: '/s3/success'
  },
  iframeSupport: {
    localBlankPagePath: '/success.html'
  },
  retry: {
    enableAuto: true // defaults to false
  },
  validation: {
    allowedExtensions: ["png", "jpg"]
  }
});

It is worth mentioning that I need to be able to control the entire process, and I don't want any special UI generated by fine uploader. The users click through a wizard and when they click "next" and arrive at the last step, I am going to fire off the s3 uploads. If they go back and change a file, I need to be able to upload only the changed file. I show them a progress bar already which I will update with the status of the s3 uploads.
Thank you in advance!

It is worth mentioning that I need to be able to control the entire process, and I don't want any special UI generated by fine uploader.

Where is your current UI?


FIrebaseAnimatedLists with reverted and inverseSorted asserts()

Steps to Reproduce
Using "FirebaseAnimatedList" to update from a remote DB.
It will trigger item insertion/removal as the remote DB changes.
When the FirbaseAnimatedList is configured with:
FirebaseAnimatedList (reverse: true, sort: (DataSnapshot a, Datasnapshot b) => b.key.compareTo(a.key))
It will show the results in sorted in inverted order, and in reverse order.
Effectively pushing the list "down" rather than "up" (default).
However, inserting new elements will now occur on index -1 (before the first index).
And will trigger an exception in debug builds on:
https://github.com/flutter/flutter/blob/5c44057d853caab9ed372268e69e55f2a159a573/packages/flutter/lib/src/widgets/animated_list.dart#L533
However, this is not a real issue, and the
final int itemIndex = _indexToItemIndex(index);
corrects the index location later on.
Expected results:
It should not except on debug builds, relax the assert() constrain.
Actual results:
Asserts on https://github.com/flutter/flutter/blob/5c44057d853caab9ed372268e69e55f2a159a573/packages/flutter/lib/src/widgets/animated_list.dart#L533
with index = -1, and the list ends up in a corrupted state (the last element is removed).
DRAFT SUBMISSION, I will update it later with more details

Logs

Hello @DarkZeros. Can you please provide you flutter doctor -v and flutter run -v logs, as well as the contents of your pubspec.yaml file? Would be great if you can provide a minimal, reproducible example as well. Thank you


Control the maximum number of words indexed

Currently the indexer does is not limited to index a certain amount of words. This could produce enormous updates and make the DocIndex index growth fast.
We must also make sure that a word index does not wrap when converting from an usize into an u16, we could use the TryInto trait.
The indexer must be separated from the serializer to make things more modular and configurable.

This is no more relevant, we close it, feel free to comment here or open a new issue.
The future version of MeiliSearch will be limited to 1000 words per fields.


Remove parsing folder

Will help clear up diffs for commits, and can be
regenerated by developers on clone

We actually need this for pylint/mypy to have any clue what's happening when it checks the frog_parser file


Improve the verification of v value in signature

Verify the range of v in sign more strictly based on official documentation or code.

Bitcoin: Certainly, there are no issues. Simply adhere to BIP137 in its entirety.
Ethereum: Based on his code, it could be 0, 1, 27, or 28. However, according to the description in EIP155, it could also be v = CHAIN_ID * 2 + 35 or v = CHAIN_ID * 2 + 36.
I see EIP-155 mentioned here

Test cases are being improved.

@janx Could you review this change, especially the Ethereum part.

LGTM except the annotation


Transition in FAQ page and responsive

Related Issue
Fixes:  #2424
Description
Added animation and transition effect on FAQ page and improved UI.
Made the section responsive for all screen also
Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
https://github.com/anuragverma108/SwapReads/assets/168854094/39d716cf-a5ae-49c6-8e5f-b40fbcccf043
https://github.com/anuragverma108/SwapReads/assets/168854094/04b31d65-9a41-499c-a0e4-9622570773ee
Checklist:

[X]  I have made this change from my own.
[X]  My code follows the style guidelines of this project.
[X]  I have performed a self-review of my own code.
[X]  I have commented my code, particularly in hard-to-understand areas.
[X] My changes generate no new warnings.
[X]  I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers and screenshots after making the changes.

@huamanraj Sir, now I have resolved conflicts and opened new PR.
Please review this PR now.

@huamanraj Sir, now I have resolved conflicts and opened new PR.
Please review this PR now.

good work 👏


fix: MD048/code-fence-style

Code fence style [Expected: backtick; Actual: tilde]

@nschonni : Thanks for your contribution! The author(s) have been notified to review your proposed change.

This fix looks good in all 6 articles. Thanks @nschonni
Publishing in this repository is suspended until Tuesday, September 22. Merging will resume at 7 AM Pacific Tuesday.
This PR is ready to merge on 9/22.

No problem, I'd suggest a small change to the .markdownlint config for MD048, but didn't want to add it to end up having it block the PR. The true should be changed to:
    "MD048": {
        "style": "backtick"
    }


Update to version 5.0.0

Checklist before submitting

[ ] Did you read the contributor guide?
[ ] Did you update any affected documentation?
[ ] Did you write any tests to validate this change?

Description
Update open-ce-builder to version 5.0.0.
Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

Codecov Report

Merging #71 (c9f4a3b) into main (407257e) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #71   +/-   ##
=======================================
  Coverage   96.62%   96.62%
=======================================
  Files          23       23
  Lines        2010     2010
=======================================
  Hits         1942     1942
  Misses         68       68

Flag
Coverage Δ

unittests
96.62% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

open_ce/__init__.py
100.00% <100.00%> (ø)

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 407257e...c9f4a3b. Read the comment docs.

Codecov Report

Merging #71 (c9f4a3b) into main (407257e) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #71   +/-   ##
=======================================
  Coverage   96.62%   96.62%
=======================================
  Files          23       23
  Lines        2010     2010
=======================================
  Hits         1942     1942
  Misses         68       68

Flag
Coverage Δ

unittests
96.62% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

open_ce/__init__.py
100.00% <100.00%> (ø)

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 407257e...c9f4a3b. Read the comment docs.


Hue scene palette

Attached to this issue you will find an approximation of the color palettes provided by the 58 scenes that were part of the official hue app for android version 4.17.0. The palette data is not in the format used by this project but it should be pretty straightforward.
These were approximated using the colorthief library probably similar to as documented by this person:
https://www.jeroendruwe.be/diy-hue-scenes-in-home-assistant-using-the-deconz-integration/
If you want to give that a go yourself, you can find the images in the apk in the version mentioned in
\assets\flutter_assets\packages\preset_scenes\assets\scenes.
As said, these aren't the exact colors provided by the app as the app likely uses a different algorithm to create the 5 color palette, but they might be either a starting point or just good enough.
Either way, I just wanted to throw some pointers regarding that topic into the world. That's why this issue is a bit low-ish effort.
Feel free to use or not use this information however you like :)
hue.json.txt

Awesome, thank you!

(I may or may not be down a rabbit hole trying to decompile the Hue app right now.)

It took a while, but we now not only have approximations but the actual colors set by the app:
https://gist.github.com/Hypfer/a0a8b5b9429831a7306ec4300077eaaa

@Hypfer Ah, very nice. I'll update that in! As for reverse-engineering the apps, well, it's not that easy since they're written in Flutter, and the Dart decompilation scene isn't really there...


Issue with predicates?

Hello @pmoura !
I started writing some examples but it seems that something is broken. Can you help?
I'll assign this to you if that's okay.
Here's what I did:
?- A = new(1,2).
A = new(1, 2).

?- mid(A, Mid).
ERROR: Unknown procedure: mid/2 (DWIM could not correct goal)
?- midpoint(A, Mid).
ERROR: Unknown procedure: midpoint/2 (DWIM could not correct goal)
?- A = [1,2].
A = [1, 2].

?- midpoint(A, Mid).
ERROR: Unknown procedure: midpoint/2 (DWIM could not correct goal)

?- magnitude(A, M).
ERROR: Unknown procedure: magnitude/2 (DWIM could not correct goal)

I see that the predicates work only with a representation of the Interval as a list [a,b]. So they don't work with the new predicate, for now.
Question: will the user have access to the shorthands of the predicates, eg mid?
Thanks, and apologies for not having started earlier.

There's a big misunderstanding here. Assuming you have loaded the library, the predicates are available by sending messages to the interval_arithmetic object:
?- logtalk_load(loader).
...

?- interval_arithmetic::new(1,2,Interval).
Interval = ...

I.e. Logtalk is not based on an import/export semantics that you find with Prolog modules. In the code you wrote, loading the library doesn't make the predicates available for calling without reference to the library objects as in your sample queries.
You can use implicit message sending from within an object using the uses/2 directive.

Oh yes !!!
Thanks.flr the correction!

Michail L. Liarmakopoulos, MSc
On Tue, Jan 18, 2022, 11:22 Paulo Moura @.***> wrote:

There's a big misunderstanding here. Assuming you have loaded the library,
the predicates are available by sending messages to the
interval_arithmetic object:
?- logtalk_load(loader).
...
?- interval_arithmetic::new(1,2,Interval).
Interval = ...
I.e. Logtalk is not based on an import/export semantics that you find
with Prolog modules. In the code you wrote, loading the library doesn't
make the predicates available for calling without reference to the library
objects as in your sample queries.
You can use implicit message sending from within an object using the
uses/2 https://logtalk.org/manuals/refman/directives/uses_2.html
directive.
—
Reply to this email directly, view it on GitHub
https://github.com/mlliarm/ia/issues/14#issuecomment-1015269995, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABMQENUCBCLNZCQRXOV34ULUWU5PLANCNFSM5MGN5TYQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


Option to disable auto update

🤷🏽♂ Current Behaviour
The software updates itself without any warn and I can't disable it.
🎯 Expected Behaviour
There should be an option to disable the auto-update. I am having a problem with the new version and trying to use the older one but it updates itself automatically.
🏡 Environmental Context
Operating System
Windows 10

@axtTOKQf
Thanks for your report. The feature has been added to the latest release:
https://github.com/sidneys/desktop-dimmer/releases
Please download & review.
Cheers,
S

Thank you!

@axtTOKQf
You're welcome ☺


Issue replacing Text's fontFamily while state changes..

[*] Review the documentation: https://facebook.github.io/react-native
[*] Search for existing issues: https://github.com/facebook/react-native/issues
[*] Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment
React Native Environment Info:
    System:
      OS: macOS High Sierra 10.13.6
      CPU: (8) x64 Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz
      Memory: 126.22 MB / 16.00 GB
      Shell: 3.2.57 - /bin/bash
    Binaries:
      Node: 8.11.3 - ~/.nvm/versions/node/v8.11.3/bin/node
      Yarn: 1.10.1 - /usr/local/bin/yarn
      npm: 5.6.0 - ~/.nvm/versions/node/v8.11.3/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
      Android SDK:
        API Levels: 23, 25, 26, 27, 28
        Build Tools: 27.0.3, 28.0.0, 28.0.2, 28.0.3
    IDEs:
      Android Studio: 3.2 AI-181.55<IP_ADDRESS>56338
      Xcode: 10.1/10B61 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.6.3 => 16.6.3
      react-native: 0.57.8 => 0.57.8
    npmGlobalPackages:
      react-native-cli: 2.0.1

Description
Hi..
I am reporting an issue concerning TextInput in Android where fontFamily not changing on state changes..
Scenario :

on Initial component load i am using normal fontFamily (using state) to display place holder. (Working Fine)
When enter value in textField then fontFamily state should change & display Bold text. (Working Fine)
Again if I remove values from the textfield & if the text field value is empty then the state should update to normal fontFamily (Not Working - i can see state is updated but view is not changing)
Again enter value in the textfield It should show bold fontFamily but it showing the unknown fontFamily

When inspecting the changes State and Style are changing perfectly.. Not sure the problem in the view
Initial Load :

if value is not empty:

After Clearing the input Field:

Note : However in IOS all these things working fine.
Font Family used:  "Montserrat-ExtraBold",  "Montserrat-Medium";

Reproducible Demo
https://snack.expo.io/@premdev077/aW5wdX
https://snack.expo.io/HkCg9wexE (Font is not loaded)

I can't reproduce your issue, I got this:

Hello there 👋 this issue doesn't have a repro (which means, a react-native init-ed project with the minimal changes that leads to creating the same issue you are reporting). Unfortunately, I have no way of helping you in a meaningful way – there is no easy way for me to recreate the situation and check that the issue reported is still there when changing the code.
Because of this, we are going to close this issue - but if a repro is shared, we are happy to reopen it 🤗

please use underscores in font file names, and it should work.


Remove Python 3.8 from cookiecutter build matrix

Fixes #59
Changes made in this Pull Request:

Removed Python 3.8 from the cookiecutter build matrix

LGTM. Worth adding Python 3.11? With the MDAKit workshop underway, I think this is rather high priority, otherwise users trying it out for the first time  will have CI faling (see https://github.com/RMeli/spyrmsdkit/actions/runs/5023108612/jobs/9007339706).


Change processing of EFS data to xarray

Feature Request
With the introduction of ReadEfs in PR #202, more EFS data is retained for sector coupling studies. The issue is, if many sectors are using the EFS data and try to process it at the same time I believe memory consumption skyrockets as I am sometimes (and seemingly randomly) getting killed errors when running. This has not been an issue for electricity only studies or if running the script by itself.
Suggested Solution
I believe xarray will be much more memory efficient at processing EFS data.
Additional Info
No response

One way we could significantly reduce data size significantly would be to aggregate to substations earlier on in the workflow. Right now bus:sub is a many:1 ratio where all buses have the same GPS location based on the the substation. This would involve making many modifications throughout the workflow, so no quick task. We would probably want to change build_base_network to aggregate bus to sub.


wasmtime-wasi-crypto missing symmetric_state_clone

wasi-crypto has seen some changes recently, and wasmtime-wasi-crypto is missing a function. Some of the guest types have also changes. Can we have an update to pull in the latest wasi-crypto features?
impl super::wasi_ephemeral_crypto_symmetric::WasiEphemeralCryptoSymmetric for WasiCryptoCtx {
  |   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ missing `symmetric_state_clone` in implementation

CC: @npmccallum, @sunfishcode

Is this related?
unknown import: `wasi_ephemeral_crypto_symmetric::symmetric_state_open` has not been defined

From this code: https://github.com/ArneVogel/wasi-crypto-bench/blob/main/bencher/src/main.rs


Added GE override feature

Added an override option to not use the GE for Ironmen.
Not sure about the change to gradle.properties this was auto-generated

Addresses #4


version number (2.0.1 -> 2.1.0)

This issue is the same as this one

see the resolution of pac4j/jax-rs-pac4j#20


[Feat] Add Digital Input Adapter for Enhanced Menu Navigation

These commits introduces the following changes:

Digital Input Adapter:

Added a new DigitalInputAdapter class to handle various digital inputs for menu navigation.
Supports multiple types of digital inputs including buttons, switches, and other digital sensors.
Provides debouncing functionality for stable readings.

Example Usage:

Provided example sketch (DigitalInputAdapter.ino) demonstrating the usage of various digital inputs with the LcdMenu.
Added another example sketch (DigitalInputWithRotaryAdapter.ino) showing how DigitalInputAdapter can be used in conjunction with other input adapters like SimpleRotaryAdapter. This example demonstrates that double press actions, which are often not read properly, can be substituted with a dedicated button for more reliable input.

These changes enhance the flexibility and usability of the menu system by allowing various digital inputs to interact seamlessly with the menu.

You can utilize something like that https://github.com/madleech/Button

You can utilize something like that https://github.com/madleech/Button

It's one less dependency for the user to install which is good, so I'm keeping it, had this problem #203 where dependencies are not installed automatically, fixed it by removing the line in properties file, they user will have to install it manually if it's not already installed.

As improvement it could be nice to specify type of event for adapter:
DigitalInputAdapter upBtn(&menu, 2, PRESSED, UP);
With following possibilities:

HIGH_LEVEL
LOW_LEVEL
PRESSED
RELEASES
HOLD
DOUBLE_CLICK

Something like that. Maybe in different PR

As improvement it could be nice to specify type of event for adapter: DigitalInputAdapter upBtn(&menu, 2, PRESSED, UP); With following possibilities:

HIGH_LEVEL
LOW_LEVEL
PRESSED
RELEASES
HOLD
DOUBLE_CLICK

Something like that. Maybe in different PR

Actually, I don't really want it to be that complicated. It's meant to be an easy way to just bind it to the pin and execute call an action when it becomes HIGH.
I can still evaluate adding some of those later also if users request it.
If you see the example of DigitalInputWithRotaryAdapter, it works well when paired with another input method. If the the requirements of the dev restrict the dev from having only push buttons and not more than 4, then maybe double tap can be added so that instead of 4 (UP, DOWN, ENTER, BACKSPACE) you can use 2 (2 actions per BTN). In most cases you'll need only three buttons (UP, DOWN & ENTER).


Decide if service.name should be required

It isn't required to set OTEL_SERVICE_NAME, however the SDKs will set service.name to unknown_service when it isn't configured. Making it required allows users to correctly set it upfront, rather than finding out later that the service name isn't set.
That being said, there are a few different mechanisms out there that set the service name automatically (i.e. resource detectors)
Originally posted by @tsloughter in https://github.com/open-telemetry/opentelemetry-configuration/issues/6#issuecomment-1524153529

Initially I was in favour of making it required because the config file is expected to wholly be used by itself and not be layered with other configuration sources (eg env vars or SDK options). Making service.name required would help prevent the cycle of users not naming their services, seeing unknown_service and then going back to name their services.
However, I setting service.name in the config file means a single config file cannot be used by multiple similar services without edits. I think this is a very valuable feature and will increase adoption of the config file.

However, I setting service.name in the config file means a single config file cannot be used by multiple similar services without edits. I think this is a very valuable feature and will increase adoption of the config file.

Can still achieve this by doing something like:
resource:
  attributes:
    service.name: ${env:OTEL_SERVICE_NAME}

@MikeGoldsmith right, re-using this config would require service.name to be set via an environment variable:
resource:
  attributes:
    service.name: ${env:OTEL_SERVICE_NAME}

... what @jack-berg said 😄

@codeboten the SDK's resource detector runner does it after running all detectors.

However, I setting service.name in the config file means a single config file cannot be used by multiple similar services without edits. I think this is a very valuable feature and will increase adoption of the config file.

Can still achieve this by doing something like:
resource:
  attributes:
    service.name: ${env:OTEL_SERVICE_NAME}

Ah, that is nice. Thanks for pointing out how to evaluate env vars 👍🏻

I'll have to check if any others do similar to Erlang. I can't imagine we are the only ones who are able to get the service name without the user essentially setting it twice.

I asked around and one other is .net which also has the ability to discover the service name https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation/blob/6f4e6a67994d8b042f74351c828b4e2b115079c7/src/OpenTelemetry.AutoInstrumentation/Configurations/ServiceNameConfigurator.cs#L24


Refactoring

Provide refactoring for javaparser classes

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here


Do not break when page it RTL

Greetings. We ran into an issue where the resize sensor was not behaving correctly when the dir of the html tag was set to rtl ( https://github.com/learningequality/kolibri/issues/1851). So a simple fix was to just set the dir of the resize sensor to ltr to prevent it from inheriting the dir. Thanks for this great library!

Good catch, thanks!