Support for mutually conflicting options

I want to offer multiple ways to define parameters for operation. For example parameters are following: paper size, paper margins, and various other parameters. And, there should be at least two ways to define them:

--paper-definition-json FILE, or
--paper-definition-xml FILE, or
--paper-size VALUE..., --paper-margins VALUE..., --paper-param-a ..., --paper-param-b, and so on...

These options are reused in same mix-in in various multiple sub-commands. Is there any (considered) support for such cases? Or, should I create own mixin content validation mechanism, and call it inside each sub-command using the mixin?
While I was writing this, I thought, that mixin PaperMixin has got Paper getPaperDefinition() method,.. validation can happen there, no addition code needed inside each sub-command.
But, as I have already wrote this, .. it might be good idea to implement automatic validation of conflicting options, as it could be reusable in other applications.

Yes, picocli currently does not do validation (other then when converting types).
When I need validation I usually create a separate class for this so I can reuse it from multiple commands.
Does that answer your question?

And, is there any standard (reusable) way to output errors of input arguments from custom validation logic? Also, what approach would you suggest for handling this, and indication about this in --help?

Hmm, the official standard for validation is jsr 303, but that may be a bit heavy weight for a stand-alone application.
There’s several ways to do this. You can throw an exception or you can have a validate method (it’s a good idea to allow returning multiple error messages from this method). Or you can invent your own annotations and inspect those at runtime.
Generally it’s a good idea to keep it simple though.

So, I guess, there's not gonna be any support for InvalidCommandlineArgumentException with properties argument and reason, in picocli? Which would have been caught and outputted pretty output with exit(1)?

Yes, at the moment I'm not planning to add support for validation to picocli.

What I’ll probably do is write an article using picocli with JSR-380 Beans Validation.

@remkop, looking forward to see it :-)


Post fix log fix

Post Fix spamming logs bug
The postfix_message_info plugin is currently logging an unnecessary amount of debugging information. As of now the plugin will log anytime some connects or disconnects from the SMTP server and when daemon services are started because the plugin failed to parse the log. This information is not necessary and this PR is looking to suppress the amount of logs produced currently by the plugin.
Example issue:
Failed to parse: (kp server logs)
KingPhisher.Plugins.Server.postfix_message_info.Plugin WARNING  [plugin: postfix_message_info] failed to parse postfix log line: 6546

Line 6546: ( postfix logs )
king-phisher postfix/smtpd[54970]: disconnect from localhost[<IP_ADDRESS>] ehlo=2 starttls=1 mail=1 rcpt=1 data=1 quit=1 commands=7

Test Steps:
[ ] Successfully enabled the postfix_message_info plugin.
[ ] Send a message sucesfully
[ ] Verify the plugin did not spam the kp logs with failed to parse warning.

Looks good, I'll land this in a moment.


No module named 'mltu.tensorflow'

I am using Pythong 3.8.17. mltu==0.1.4 and Tensorflow 2.10.0. when I try to run train.py it gives this.
raceback (most recent call last):
  File "train.py", line 7, in <module>
    from mltu.tensorflow.dataProvider import DataProvider
ModuleNotFoundError: No module named 'mltu.tensorflow'

I recommend using newest version of mltu


Measure code coverage for Gradle plugin

Expected Behavior
Test coverage for the Gradle plugin is measured by JaCoCo. Results are uploaded to Codecov after a successful CI build.
Current Behavior
Neither happens.
Context
Tracking code coverage for all modules means coverage and hopefully quality will improve over time.
Unfortunately JaCoCo doesn't run out of the box on tests that use Gradle TestKit, so this will require a bit of trickery to get working properly.

Blocked until https://github.com/arturbosch/detekt/issues/1754 is resolved.

Codecov is fine again. Therefore, I'm closing this.

Codecov is fine again. Therefore, I'm closing this.

Not quite, the unit tests (those not relying on Gradle TestKit) result in coverage, but not the integration tests which don't generate coverage info due to https://github.com/gradle/gradle/issues/1465
So yes, there's some coverage info, but it's not complete. You can see for example coverage info for the Detekt task which shows no coverage of the @TaskAction: https://codecov.io/gh/detekt/detekt/src/e3fd6c88cc0477084de3e69de7c901390683ac08/detekt-gradle-plugin/src/main/kotlin/io/gitlab/arturbosch/detekt/Detekt.kt#L203

Not quite, the unit tests (those not relying on Gradle TestKit) result in coverage, but not the integration tests which don't generate coverage info due to https://github.com/gradle/gradle/issues/1465
So yes, there's some coverage info, but it's not complete. You can see for example coverage info for the Detekt task which shows no coverage of the @TaskAction: https://codecov.io/gh/detekt/detekt/src/e3fd6c88cc0477084de3e69de7c901390683ac08/detekt-gradle-plugin/src/main/kotlin/io/gitlab/arturbosch/detekt/Detekt.kt#L203

It would still be good to do this

Is this even doable from our side? Shouldn't we just wait for https://github.com/gradle/gradle/issues/1465 or just accept that we are not going to get this reports?

There's a plugin we could try. Let's keep this open.


How do I write data to an existing native HDF5 file?

I want to write data to the local HDF5 file, your library does not seem to provide this method, or I did not find it. So far I've seen write methods that overwrite my native files. Does your library support such writing methods? Thank you!

Hi, thanks for trying out PureHDF. Modifying existing files is out of scope oft this library unfortunately, because that would complicate things very much due to the complexity of the HDF5 specification.


feat: switch next-api-og-image with @vercel/og

Description

Linked Issues
Preview / Screenshot

Photo
Video

Checklist

[ ] I have followed commit guideline to create PR title (https://conventionalcommits.org/en/v1.0.0/)

📝 Changed routes:

/api/generate

Commit a6176fdec2c05b6f5e421a767d3934afd16a53b3 (https://og-dm6yl90px-sozonome.vercel.app).


Enable some form of Slack notifications parsing

Hi,
right now mentioning slack users or teams is pretty tedious as it has to be done with slack IDs. Think
<!subteam^ID|team>  or <!@U12345|nickname> instead of normal @team or @nickname.
This is because slack does not parse messages received from webhooks (default parse=none).
I think parse=full is unnecessary, but adding link_names=1 here https://github.com/prometheus/alertmanager/blob/2433eebf481029d6f4b68c1874618d4cbe1d8d3c/notify/impl.go#L465 should be helpful.
What do you think? Are there any reasons why this is not there or will you accept a simple PR? :)

I think that makes sense. I only wonder whether it should be configurable (or always on), but probably configurable.

is this gonna be merged any time soon ?

Following up now, sorry for letting this slip

We have same issue, can you fix it?

https://github.com/prometheus/alertmanager/pull/912 added this.

Looks like nowadays link_names: true + mentioning like @username stopped working. I tried even <@username> - no luck, it just prints this text as is. The only solution is to use user id, like <@USERID>. You can find user id using this manual https://help.workast.com/hc/en-us/articles/360027461274-How-to-find-a-Slack-user-ID.
I think this change is related to the blog post https://api.slack.com/changelog/2017-09-the-one-about-usernames.


Update the versioning logic for the Godot Android Editor

This is necessary to separate subsequent uploads to the Google Play store as each upload needs to increment the version code.
Also updates the set of supported screen size to exclude small devices (e.g: watches). Affects less than 0.5% of supported devices.

3.x version

This is necessary to separate subsequent uploads to the Google Play store as each upload needs to increment the version code.

I don't think this change will be sufficient then. We usually have multiple alpha / beta / RC builds, but as I understand it this change only gives you the possibility to upload a single build for each. So if you upload 3.6 beta 1, then you can't do updates until the 3.x branch is ready to move to RC.

Can we pass this as an argument to scons or gradle at build time

Yeah, I'll create an issue to track the task.
We can't automate the upload process until the app is first reviewed and released to the store, so at the moment the process will be manual for which the current approach should work.
Once we're able to fully automated the upload process, then we should update it to pass the build number at build time.

Yeah, I'll create an issue to track the task.

Added https://github.com/godotengine/godot-proposals/issues/5410 to track the issue.

Thanks!


Option to disallow unused checker suppressions

As code evolves, some suppressions may be added that become unnecessary later one. However, these are easy to miss during refactoring and remains as benign. Unfortunately, this leaves open the possibility of a new error being introduced and uncaught. It can be unfeasible to audit a large code base for unnecessary suppressions after the fact, and generally shouldn't be an error as each tool has its own subset.
It would be nice if ErrorProne allowed its checks to be verified when suppressed. If the suppression is enabled but no error was reported, then the failure it reported. This would be hooked in ErrorProneScanner and disabled by default.
A scenario is null data flow analysis (https://github.com/uber/NullAway/issues/100) where false positives are suppressed. Then as the code or checker evolves these become potential hazards.

I don't really have anything to add to this other than I think this is, at least in theory, a great idea! I've no idea how practical it would be, but considering just the initial idea here, I'd be very, very happy to see something like this implemented.


Feature request: Introduce DataSharedFacetValue and MethodSharedFacetValue subtypes

Moving forward, we don't want to mix and match methods and data in shared facets. We want to think of data facets as pure data, much like Redux state, and method facet as flat lists of available API calls from the remote.
Additionally, MethodSharedFacetValues could automatically do a .bind on the methods it includes, to avoid issues with selecting individual methods from the facet and losing the this reference.

Closing this as we are exploring better alternative APIs for shared facets.


[APMSP-1317] Fetch agent info endpoint

What does this PR do?
Add AgentInfoFetcher to fetch the agent info endpoint and keep it updated
Motivation
The info endpoint stores some essential parameters for client side stats
Additional Notes
Anything else we should know when reviewing?
How to test the change?
Describe here in detail how the change can be validated.

Codecov Report
Attention: Patch coverage is 91.93548% with 15 lines in your changes missing coverage. Please review.

Project coverage is 73.24%. Comparing base (9705ec2) to head (aad19e4).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #619      +/-   ##
==========================================
+ Coverage   73.13%   73.24%   +0.11%
==========================================
  Files         252      254       +2
  Lines       35959    36127     +168
==========================================
+ Hits        26300    26463     +163
- Misses       9659     9664       +5

Components
Coverage Δ

crashtracker
20.53% <ø> (ø)

datadog-alloc
98.73% <ø> (ø)

data-pipeline
90.12% <ø> (ø)

data-pipeline-ffi
0.00% <ø> (ø)

ddcommon
83.08% <ø> (ø)

ddcommon-ffi
69.52% <ø> (ø)

ddtelemetry
59.10% <ø> (ø)

ipc
83.63% <ø> (ø)

profiling
84.26% <ø> (-0.01%)
:arrow_down:

profiling-ffi
77.42% <ø> (ø)

serverless
0.00% <ø> (ø)

sidecar
40.12% <ø> (ø)

sidecar-ffi
0.00% <ø> (ø)

spawn-worker
50.36% <ø> (ø)

tinybytes
91.66% <ø> (ø)

trace-mini-agent
70.88% <ø> (ø)

trace-normalization
98.25% <ø> (ø)

trace-obfuscation
95.73% <ø> (ø)

trace-protobuf
77.67% <ø> (ø)

trace-utils
93.28% <ø> (+0.41%)
:arrow_up:

FYI, this branch needs a Jira ticket associated with it.

Requires https://github.com/DataDog/libddprof-build/pull/52 to be merged


Refresh contribute page

What does this change

Incorporate feedback on how we link to issue labels
Link to list of mixins and mixin ideas
Offer mentoring
Link to quickstart

Preview at https://deploy-preview-1121--porter.netlify.app/contribute/
What issue does it fix
N/A
Notes for the reviewer
🍫
Checklist

[ ] Unit Tests
[x] Documentation
[ ] Schema (porter.yaml)

@iennae I've updated the PR with your suggested changes, please take another look.


Update data-retention-policy.rst

Adding docs for custom data retention policies.

Summary

Ticket Link

Thank you so much for this great PR, @wiersgallak!
The introduction contains this line:
"In Mattermost Enterprise E20, you can set a custom policy to manage how long messages and file uploads are kept in Mattermost channels and direct messages."
We should update this as follows:
"In Mattermost Enterprise E20, you can set a global retention policy as well as custom retention policies to manage how long messages and file uploads are kept in Mattermost channels and Direct Messages.

@cwarnermm I made some editorial updates again along with addressing your feedback.  I felt like we were being a bit redundant and overusing the word "specified".  I also removed the sections that indicated a restart was required.


How to run http in configuration.yaml file with lets encrpyt without ECONNRESET errors?

I setup a duckdns server with letsencrpyt and when I edit the configuration.yaml file to set my api password, ssl keys and cert. homebridge throws an econnreset error but when I comment the http: command out in configuration.yaml it works again.
Is there a way to connect homebridge to work with http line in configuration.yaml?

Edit: Figured it out. I just needed to edit my homebridge config.json to go through my domain rather than the local ip.


Support hosting multiple subgraphs

CLI configuration is shared among subgraphs.

See comments on #227 for specs.


Add Proguard rules to library using consumerProguardFiles

Will fix #351

Friendly ping @patedit

Thanks @vanniktech - Can we rename proguard-rules.txt to proguard.txt as per Android' suggestion? http://tools.android.com/tech-docs/new-build-system/aar-format

Done

Thanks a lot!


Fix a typo preventing to use the full int64_t range when using gt<> and lt<> validators

Hi everyone,
This PR fixes #199. Since ROS uses int64_t to store integer parameters, int64_t numeric limits should be used instead of int.
i simply corrected the template files and the associated unit tests.
Any thoughts ?

My pleasure, happy to help.


DefaultCredentialsError is raised when getting client using JSON credentials on Docker

I'm trying to connect to Datastore from inside Docker container running on my local machine when I got this DefaultCredentialsError.
Python 2.7.13 (default, Apr 18 2017, 22:03:41)
[GCC 6.3.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> from google.cloud import datastore
D1004 04:45:28.018781655     107 env_linux.c:66]             Warning: insecure environment read function 'getenv' used
>>> client = datastore.Client()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python2.7/site-packages/google/cloud/datastore/client.py", line 204, in __init__
    project=project, credentials=credentials, _http=_http)
  File "/usr/lib/python2.7/site-packages/google/cloud/client.py", line 211, in __init__
    _ClientProjectMixin.__init__(self, project=project)
  File "/usr/lib/python2.7/site-packages/google/cloud/client.py", line 165, in __init__
    project = self._determine_default(project)
  File "/usr/lib/python2.7/site-packages/google/cloud/datastore/client.py", line 221, in _determine_default
    return _determine_default_project(project)
  File "/usr/lib/python2.7/site-packages/google/cloud/datastore/client.py", line 77, in _determine_default_project
    project = _base_default_project(project=project)
  File "/usr/lib/python2.7/site-packages/google/cloud/_helpers.py", line 179, in _determine_default_project
    _, project = google.auth.default()
  File "/usr/lib/python2.7/site-packages/google/auth/_default.py", line 281, in default
    credentials, project_id = checker()
  File "/usr/lib/python2.7/site-packages/google/auth/_default.py", line 142, in _get_explicit_environ_credentials
    os.environ[environment_vars.CREDENTIALS])
  File "/usr/lib/python2.7/site-packages/google/auth/_default.py", line 65, in _load_credentials_from_file
    'File {} was not found.'.format(filename))
google.auth.exceptions.DefaultCredentialsError: File "/root/.google/##################.json" was not found.

My Docker container is from jfloff/alpine-python:2.7 image. I have set Docker volume to let this container access credentials file from the host and I have already make sure that Python can see this file. What I don't understand is that if I try to call function _load_credentials_from_file directly, it can read credentials file without problem. Only got DefaultCredentialsError when calling through client = datastore.Client()
>>> from google.auth._default import _load_credentials_from_file
>>> _load_credentials_from_file('/root/.google/##################.json')
(<google.oauth2.service_account.Credentials object at 0x7f0b47ef3f90>, u'############')

Another thing that I noticed is that when I import datastore using from google.cloud import datastore I got the following warning (You can also see the warning at the stacktrace above)
Warning: insecure environment read function 'getenv' used

So I googled around this warning and see that it might have something to do with grpc library.

https://github.com/grpc/grpc/issues/8104
https://github.com/nodejs/docker-node/issues/436

Now I have no idea how to fix issue, please help me
Thank you

@panuta Thanks for filing. Have you seen our auth guide? Also, is the filename in the error:
google.auth.exceptions.DefaultCredentialsError: File "/root/.google/##################.json" was not found.

the same as the filename you are manually loading?
>>> _load_credentials_from_file('/root/.google/##################.json')

@dhermes Yes, I did read and follow the guide. I download json key and set its path to  GOOGLE_APPLICATION_CREDENTIALS environment variable according to the guide.
Also filename is is the same. I copy&paste it :)
To give you more info, I have uploaded my application to Google Compute Engine and run it from there using the same code that I run on my local machine. It runs perfectly fine. Data was saved to Datastore correctly. So this issue only happened inside Docker on my local machine.
Here is my Dockerfile and requirements that I'm using
FROM python:3.6-alpine3.6
ENV PYTHONUNBUFFERED 1

# System dependencies
RUN apk add --no-cache \
        tzdata \
        build-base \
        bash \
        libffi-dev \
        openssl-dev \
        libxml2-dev \
        libxslt-dev

# Set Timezone
RUN cp /usr/share/zoneinfo/Asia/Bangkok /etc/localtime
RUN echo "Asia/Bangkok" > /etc/timezone

# Python environments
COPY ./requirements/scrapyd.txt /tmp/requirements.txt
RUN pip install -r /tmp/requirements.txt

# scrapyd
RUN mkdir -p /scrapyd/data
COPY ./compose/scrapyd/scrapyd.conf /scrapyd/scrapyd.conf

VOLUME /scrapyd/data
EXPOSE 6800

WORKDIR /scrapyd
CMD ["scrapyd"]

scrapy
scrapyd

slack_log_handler

google-cloud-datastore

NOTE that since I asked this question, I have changed Docker image from jfloff/alpine-python:2.7 to python:3.6-alpine3.6 but the issue is still persisted.

@jonparrott Can we move this over to https://github.com/GoogleCloudPlatform/google-auth-library-python/?
@panuta Thanks for the details. It's very surprising that _load_credentials_from_file fails to find the file when called by google.auth.default() but not when called directly.
How are you running Docker on your local machine? The file in /root/.google won't appear there unless you do something like --volume /some/local/path:/root/.google when you call docker run.

@dhermes I solved it. It's my own fault. I have set env variable with double quote around the string like
GOOGLE_APPLICATION_CREDENTIALS="/root/.google/############.json"

Took me a while to see this obvious error. Such an embarrassing mistake. Thank you for trying to help me figuring it out.

Good deal! 👍

This saved my day ! Thanks panuta

Thanks @panuta, actually stumbled upon making the exact same mistake!

I made exactly the same mistake, thanks to share the case.

you gys how are you getting it right i have also the same error

@lewisMachilika when you set the environment variable, make sure to use single quote instead of double-quotes surrounding your path, i.e.
export GOOGLE_APPLICATION_CREDENTIALS=‘/path/to/your_credentials.json’
The reason so many people are having this issue is probably because they use double-quotes in the documentation:
https://cloud.google.com/docs/authentication/getting-started#auth-cloud-implicit-python
Someone should fix this in order to avoid further confusion amongst developers.

Hello guys.
I have still this issue, and after trying all steps written above, I am still getting same error - file not found.
Strange is - script is looking for file in GOOGLE_APPLICATION_CREDENTIALS - it mean, it can read it from ENV, but complains about not finding it.
Any other sugestion how to solve this?
Thanks.
ERROR
google.auth.exceptions.DefaultCredentialsError: File /Users/janfalta//Users/janfalta/key.json  was not found
this is my statement in BASH_PROFILE
export GOOGLE_APPLICATION_CREDENTIALS='/Users/janfalta//Users/janfalta/key.json '
PRINTENV
printenv TERM_PROGRAM=Apple_Terminal SHELL=/bin/bash TERM=xterm-256color TMPDIR=/var/folders/k1/vxmdz_591lqf_k1p3czxb4_m0000gn/T/ GOOGLE_APPLICATION_CREDENTIALS=/Users/janfalta//Users/janfalta/key.json  Apple_PubSub_Socket_Render=/private/tmp/com.apple.launchd.7VuVFJmiK4/Render TERM_PROGRAM_VERSION=421.2 OLDPWD=/Users/janfalta TERM_SESSION_ID=7C78DC52-1147-46F5-A419-55BD12BEE6C4 USER=janfalta SSH_AUTH_SOCK=/private/tmp/com.apple.launchd.2Bof01U0Gw/Listeners PATH=/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin LANG=cs_CZ.UTF-8 XPC_FLAGS=0x0 PS1=$ \u@\e[1;36m [\w]:\e[m \n XPC_SERVICE_NAME=0 SHLVL=1 HOME=/Users/janfalta LOGNAME=janfalta _=/usr/bin/printenv

@faltajan in your case it may be that this path:

GOOGLE_APPLICATION_CREDENTIALS=/Users/janfalta//Users/janfalta/key.json

is incorrect (/Users/janfalta/ is duplicated).


Add Docs for 1.23.3

This updates the docs for 1.23.3.
I've removed the 1.23.2 version from the website and replaced it with 1.23.3 as they're effectively equal.

Messages

:book:

Thank you very much for making our website better :heart:!

  Generated by :no_entry_sign: dangerJS against 148c69336d2d2a4fad19341c5c395e6efc6dee61


verify the v1 ingress resources in the getting stated guide

Is there an existing issue for this?

[X] I have searched the existing issues

Problem Statement
It seems that example Ingress resources like this one are broken in the Getting Started guide, because they use the v1beta1 schema and the v1 apiversion.
Proposed Solution
Ensure that all items from the guides that have been updated to ingress v1 work.
Additional information
No response
Acceptance Criteria
No response

https://github.com/Kong/docs.konghq.com/pull/3157 seeks to correct these.

Having not received further review on https://github.com/Kong/docs.konghq.com/pull/3157, merging that closing this based on the spot checks I performed for that since I made the same changes to Ingresses throughout. That makes the corrected Ingresses available in the v2 doc branch for the audit tickets, and the audit tickets for individual docs should find any that were missed.


Web - Workspace - Inviting a member offline does not cross the name out

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Access staging.new.expensify.com
Sign into a valid account
Click on profile > Workspace > Manage members
Invite a member
Turn off internet connection
Remove the member while offline

Expected Result:
User expects the name to be crossed out while offline
Actual Result:
The name just simply disappears from the members list
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/216486595-6b97ca63-fa6c-4658-be41-852b64b67389.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team / @Tushu17
Slack conversation:
View all open jobs on GitHub

Not repro in Production

Working for me on both staging:

And production:

Going to close this one out for the moment

@bfitzexpensify This is issue happening in Web ONLY.

@kbecciv sorry, the first screenshot looks like the desktop app, but it is in web.
Here's a clearer screenshot:

Are you able to reproduce?

Also could not repro on latest staging web. Closing for now


[Bug] Treemap itemstyle color set is not working

Version
5.3.0
Link to Minimal Reproduction
https://codepen.io/gurucharan1/pen/rNpJewq?editors=0010
Steps to Reproduce
option = {
  series: {
    type: "treemap",
    data: [{
      name: "Food",
      children: [{
        value: 3,
        name: "Fruit",

      }, {
        value: 9,
        name: "Meat",
      }, {
        value: 19,
        name: "Veggies",
      }]
    }],
    itemStyle: {
      color: "rgba(215, 49, 49, 1)"
    }
  }
}

Current Behavior
We have set color for treemap using itemstyle color property. It is not being respected.
Expected Behavior
Treemap should be plotted with color set in itemstyle color property
Environment
- OS:Windows
- Browser: Chrome
- Framework:Angular

Any additional comments?
No response

Seems to be a unexpected behavior. You can use series.color as a workaround
option = {
  series: {
    type: 'treemap',
    data: [
      {
        name: 'Food',
        children: [
          {
            value: 3,
            name: 'Fruit'
          },
          {
            value: 9,
            name: 'Meat'
          },
          {
            value: 19,
            name: 'Veggies'
          }
        ]
      }
    ],

    color: ['rgba(215, 49, 49, 1)']
  }
};

So this means I can't color the nodes using a "callback function" the same way you can color the bar chart items for example? https://echarts.apache.org/en/option.html#series-bar.itemStyle.color
Will this become an option in the future?


AttributeError: module 'site' has no attribute 'getsitepackages'

Issue description
python version 3.7.3
pipenv version 2018.11.26
When I try to use site.py#getsitepackages() in pipenv environment, the below error occurrs.
AttributeError: module 'site' has no attribute 'getsitepackages'
When I use the same module in pip environment, the error does not occur.
The cause is probably that pipenv doesn't refer site.py installed  in python environment.
I checked the installed directory.
pipenv --venv
$HOME/.local/share/virtualenvs/$hash
ls -l $HOME/.local/share/virtualenvs/$hash/lib/python3.7 | grep site.py
-rw-rw-r--   1 $user $user 28389 Jun  6 07:13 site.py

The aforementioned site.py does not have getsitepackages() method.
The other modules refer to the modules installed in python environment.
E.g.
os.py
ls -l $HOME/.local/share/virtualenvs/$hash/lib/python3.7 | grep os.py
lrwxrwxrwx   1 $user $user    51 Jun  6 07:13 os.py -> /usr/local/pyenv/versions/3.7.3/lib/python3.7/os.py

Expected result
Even in pipenv environment, site.py#getsitepackages() method is available.
Steps to replicate
Install python3.7.3
Install pipenv by pip
Run the below
pipenv run python -c "import site; site.getsitepackages()"

It is an issue with virtualenv, which copies the bundled version of site.py to the venv when it is created. (https://github.com/pypa/virtualenv/issues/737)
Close it for now

This now breaks pywin32 installation, starting from pywin32==226. :(

This started happening today.
I installed a package in pipenv and started throwing this error.
I then deleted the environment and created a new, now it won't work.
File "C:\Users\--\.virtualenvs\project-nNknfD7x\lib\site.py", line 791, in <module>
  main()
File "C:\Users\--\.virtualenvs\project-nNknfD7x\lib\site.py", line 768, in main
  paths_in_sys = addsitepackages(paths_in_sys)
File "C:\Users\--\.virtualenvs\project-nNknfD7x\lib\site.py", line 280, in addsitepackages
  addsitedir(sitedir, known_paths)
File "C:\Users\--\.virtualenvs\project-nNknfD7x\lib\site.py", line 211, in addsitedir
  addpackage(sitedir, name, known_paths)
File "C:\Users\--\.virtualenvs\project-nNknfD7x\lib\site.py", line 179, in addpackage
  exec(line)
File "<string>", line 1, in <module>
File "C:\Users\--\.virtualenvs\project-nNknfD7x\lib\site-packages\win32\lib\pywin32_bootstrap.py", line 14, in <module>
  for maybe in site.getsitepackages():
AttributeError: module 'site' has no attribute 'getsitepackages'

This started happening today.
I installed a package in pipenv and started throwing this error.
I then deleted the environment and created a new, now it won't work.
I can't even open python.exe

Delete the virtual environment, change the pywin32 environment in your Pipfile to "==225", and then run pipenv update.
If you don't have a pywin32 requirement, it's probably required by another required package. Just add pywin32 with the "==225" version into your Pipfile.

I've gotten this issue on Python 3.7.0, Windows 10, in a virtualenv.
As a workaround that seems to be holding, outside of the virtualenv, I did:
python

Python 3.7.0 (v3.7.0:1bf9cc5093, Jun 27 2018, 04:59:51) [MSC v.1914 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import site
>>> site.getsitepackages()
['C:\\Users\\maludwig\\AppData\\Local\\Programs\\Python\\Python37', 'C:\\Users\\maludwig\\AppData\\Local\\Programs\\Python\\Python37\\lib\\site-packages']

Then I simply edited the site.py in my virtualenv, and added the missing function:

def getsitepackages():
    return ['C:\\Users\\maludwig\\AppData\\Local\\Programs\\Python\\Python37', 'C:\\Users\\maludwig\\AppData\\Local\\Programs\\Python\\Python37\\lib\\site-packages']

It's a hacky workaround, but hopefully this helps someone else.

work for me
thanks :beer:
def getsitepackages():
    return ['C:\\Users\\***\\AppData\\Local\\Programs\\Python\\Python37', 'C:\\Users\\***\\.virtualenvs\\Wolfram-OchuDyJ5\\Lib\\site-packages']

Happened to me on my windows CI too :

Fatal Python error: initsite: Failed to import the site module
Traceback (most recent call last):
  File "c:\\users\\circleci\\.virtualenvs\\project-ohnhamvy\\lib\\site.py", line 769, in
    main()
  File "c:\\users\\circleci\\.virtualenvs\\project-ohnhamvy\\lib\\site.py", line 746, in main
    paths_in_sys = addsitepackages(paths_in_sys)
  File "c:\\users\\circleci\\.virtualenvs\\project-ohnhamvy\\lib\\site.py", line 279, in addsitepackages
    addsitedir(sitedir, known_paths)
  File "c:\\users\\circleci\\.virtualenvs\\project-ohnhamvy\\lib\\site.py", line 202, in addsitedir
    addpackage(sitedir, name, known_paths)
  File "c:\\users\\circleci\\.virtualenvs\\project-ohnhamvy\\lib\\site.py", line 170, in addpackage
    exec(line)
  File "", line 1, in
  File "c:\\users\\circleci\\.virtualenvs\\project-ohnhamvy\\lib\\site-packages\\win32\\lib\\pywin32_bootstrap.py", line 14, in
    for maybe in site.getsitepackages():', "AttributeError: module 'site' has no attribute 'getsitepackages'"

We don't have pywin32 as a direct dependency and I don't think pywin32==225 is viable long term solution.
Can we reopen this issue ?

Thanks @maludwig.  Your work around worked well.  Hope there's a long term solution soon cause I have a lot of different virtual environments.  😭

I received this same error message with pyInstaller package. The reason is because I was from inside a virtualenv.
I fixed it by downgrading pyinstaller==5.0.1

It is an issue with virtualenv, which copies the bundled version of site.py to the venv when it is created. (https://github.com/pypa/virtualenv/issues/737)

For those (like me) living under a rock: This was fixed in virtualenv 20.0+, and upgrading to pipenv 2023.2.18+ will pull the fixed virtualenv version. You then have to re-create the pipenv environments.


XSD association fails

I've just installed XmlNotepad on my new laptop and I'm having trouble to associate it to open xsd extension.
There hasn't been any problem with xml but the system ignores XmlNotepad when selecting "open with..." in XSD. I'm not even able to find XmlNotepad.exe file to manually select it.
I've tried ClickOnce installer, then uninstalled, then tried with winget. I think am walking around with no success.

I made a video for you showing how to do it:
https://youtu.be/n-6sSUSlN34

That was great. Thank you very much.
I've had managed to associate it but now it has also set the proper icon.


Add HTTP status codes

HTTP::Response was missing some HTTP status codes from recent RFC (eg: 308, 423, etc).

Thanks!


Bump identity.org.mgt version

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6641054109

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6641054109
Status: success


css | fix donation modal for mobile devices

this is how it looks today:

Hi, I’d like to work on this issue. Could you please assign it to me?

Hi @rivka14 ! Thanks!
Have you seen the #945 pull request that @yontank submitted? Just making sure we won't have collisions.

Solved in #945 . Thanks @yontank !


Fix the Table of Contents in README.md

First, synchronize headers with contents' name.
Equalize uppercase or lowercase letters with headers. Add a question mark.
Second, fix the link of "What is Burrow ?", which didn't work.
Third, change the loction of  "Future work" and "License".
Because the text has a different order with the table of contents.

most if not all of this was fixed on develop in #700


Add guidelines for artifacts

This is an alternate take on #1023. The goal is to document the "scratch layer" from #1025, and include details from opencontainers/artifacts that will allow us to eventually archive the repo.

Nits were moved to #1042. I'll reopen a new PR with the decision to add an artifactType field to the image manifest we made on yesterday's call.

Help
@.***
On Fri, Mar 24, 2023, 12:36 PM Brandon Mitchell @.***>
wrote:

Nits were moved to #1042
https://github.com/opencontainers/image-spec/pull/1042. I'll reopen a
new PR with the decision to add an artifactType field to the image
manifest we made on yesterday's call.
—
Reply to this email directly, view it on GitHub
https://github.com/opencontainers/image-spec/pull/1029#issuecomment-1483175799,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5O3HGWMIVQKD75W4GAS3SDW5XLS3ANCNFSM6AAAAAAVHRRLBQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


cannot query on strings in v5

System.NotSupportedException: "Method CompareString not available to convert to BsonExpression (CompareString(x.Name, value(testLiteDB.MainWindow+_Closure$__0-0).$VB$Local_customer.Name, False))."

how to reproduce:
just perform any kind of query (Query/Find/FindOne) on a string property, using Lambda-Epressions.
works in v4.

Hi @BangL, can you write this query example? In v5 lambda expressions must be converted into BsonExpression.... in v4, if not possible, LiteDB uses Linq to Objects

my example was something like this:
Dim oldObj As MyClass = Items.FindOne(Function(x) x.Identifier = newObj.Identifier)
while newObj was of type "MyClass" as well.
i've already migrated to Items.FindOne($"$._id = '{newObj.Identifier}'") overload, which solves the problem for me.
but the overload for linq does not work this way.

might be a conflict with how VB handles lambda. not sure

Stacktrace:
bei LiteDB.LinqExpressionVisitor.VisitMethodCall(MethodCallExpression node)
bei System.Linq.Expressions.MethodCallExpression.Accept(ExpressionVisitor visitor)
bei LiteDB.LinqExpressionVisitor.VisitAsPredicate(Expression expr, Boolean ensurePredicate)
bei LiteDB.LinqExpressionVisitor.VisitBinary(BinaryExpression node)
bei System.Linq.Expressions.BinaryExpression.Accept(ExpressionVisitor visitor)
bei System.Linq.Expressions.ExpressionVisitor.Visit(Expression node)
bei System.Linq.Expressions.ExpressionVisitor.VisitLambda[T](Expression1 node)    bei LiteDB.LinqExpressionVisitor.VisitLambda[T](Expression1 node)
bei System.Linq.Expressions.Expression1.Accept(ExpressionVisitor visitor)    bei System.Linq.Expressions.ExpressionVisitor.Visit(Expression node)    bei LiteDB.LinqExpressionVisitor.Resolve(Boolean predicate)    bei LiteDB.BsonMapper.GetExpression[T,K](Expression1 predicate)
bei LiteDB.LiteCollection1.Find(Expression1 predicate, Int32 skip, Int32 limit)
bei testLiteDB.MainWindow..ctor() in C:\Users\BangL\source\repos\testLiteDB\testLiteDB\MainWindow.xaml.vb: Zeile20

@BangL In C#, the equality operator is a "real" operator and the BsonMapper successfully converts it to a to an expression. However, in VB, the string equality operator is converted to a call to CompareString, and the BsonMapper doesn't know what to do with it.
In previous versions of LiteDB, lambdas that were not sucessfully converted to expressions were stored and executed after the results were retrieved and mapped to object. However, this feature was removed in v5 due to performance issues - if you ran your query on a collection with 1 million entries, all of them would be retrieved before the lambda expression was executed, which is very inefficient.
This feature may be recreated in v5.1 in a more efficient way, however please note that there might be a significant performance hit.

very good to know. thanks for the explanation!
for me personally this problem is solved then, i will go BsonExpressions only from now on.
despite the performance issues and the vb.net incompatiblity, i would suggest putting the [DeprecatedAttribute] onto the lambda overloads though.


Search migration

Should be merged alongside search migration branch on backend (uses new routes)

CS Lower level undergrad also has this issue.


improve: pretty error message on ts-node import error

closes #111
TODO

[x] tests

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

npm package (@next-major dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


added a random state to glove class

While creating paragraph vectors I noticed that each initialization would be wildly different. This PR sets a random_state to the Glove class similar in spirit to those used in sklearn. Note that currently pre-trained models will have no random state set, and need to be set explicitly by the user.

Looks great, thanks!


Explicit timeout handling on Promise API

Currently, when a timeout is specified on a promise, the Promise returned by timeout fails with a TimeoutException when the upstream promise did not resolve beforehand. Handling of a timeout relies on the same operations as "regular" failures.
If in the failure handling you want to distinguish between a timeout or application failure, you'll need to do an instanceof check on the failure, which hurts code readability a bit. Adding explicit handlers specifically for timeout related failures could improve the readability of application code.
For example, by extending the Promise API with methods like:

Promise<T> onTimeout(Runnable callback)
Promise<T> recoverTimeout(Supplier<? extends T> recovery)
Promise<T> recoverTimeoutWith(Supplier<Promise<? extends T>> recovery)
Promise<T> fallbackTimeoutTo(Promise<? extends T> fallback)

Though I think application code could benefit from this, I do also see some downsides. Primarily on how these proposed methods would coexist with the use of the existing failure handling operations:

For example, when there is both an onFailure and an onTimeout specified, when the timeout triggers, both handlers would be called. I guess preventing the onFailure callback in this case would be a backwards incompatible change, and therefor not an option.
Also the order of operations would matter. Specifying the recoverWith before recoverTimeoutWith, could render the latter useless when the former would also (inadvertently) handle the TimeoutException case. This is not really new, as the current API also permits multiple recoverWith being chained, which also requires care to ensure each function handles the case(s) it is supposed to.

Any thoughts on this? Would such a change be an acceptable extension? Are there perhaps alternative approaches to timeout handling to be considered?

I am convinced this is not a good idea. It would require to spec to define two callback queues (for non-timeout callbacks and for timeout callbacks). And then also to call the right call back depending on the exception type of the failure.
The correct answer here is to do the instanceof check you already mention. Timeout is just one of many exception types which can be present in a promise failure. Recovery code may need to take special action for any number of exception types including TimeoutException.
See https://github.com/bndtools/bnd/blob/271c260085fcc2dd54e912247bdd2c44e6bde902/biz.aQute.bndlib/src/aQute/bnd/http/HttpClient.java#L165-L201 where a recovery callback has special handling for multiple exception types including TimeoutException case. Having separate callback queue would complicate recovering from a failure since you need to register for when the failure is a TimeoutException and for when the failure is not a TimeoutException and then possibly duplicate code in both callbacks.
So I do not see sufficient value in additional API for one specific exception type. The additional complexity does not add sufficient value. And I am sure such a change would not be backwards compatible with existing code.

it's pretty nice as it is if you use Java pattern matching switch:
p.recoverWith(it -> switch (it.getFailure()) {
	case TimeoutException te -> timeoutFailure(te);
	case Throwable t -> generalFailure(t);
});

I think we should not change API where the Java language is improving on our behalf.

I am convinced this is not a good idea. It would require to spec to define two callback queues (for non-timeout callbacks and for timeout callbacks). And then also to call the right call back depending on the exception type of the failure.

I was not necessarily suggesting a separate callback queue for timeout related callbacks, but more like additional timeout-specific convenience operations based on the current capabilities/structure. Separate callback queues would address some of the downsides I mentioned, but I fully understand that would be a risky change while also trying to retain the current behaviour: that is most likely not worth it.

The correct answer here is to do the instanceof check you already mention. Timeout is just one of many exception types which can be present in a promise failure. Recovery code may need to take special action for any number of exception types including TimeoutException.
See https://github.com/bndtools/bnd/blob/271c260085fcc2dd54e912247bdd2c44e6bde902/biz.aQute.bndlib/src/aQute/bnd/http/HttpClient.java#L165-L201 where a recovery callback has special handling for multiple exception types including TimeoutException case. Having separate callback queue would complicate recovering from a failure since you need to register for when the failure is a TimeoutException and for when the failure is not a TimeoutException and then possibly duplicate code in both callbacks.

That is a nice intricate recovery example, thanks for pointing me to it!

So I do not see sufficient value in additional API for one specific exception type. The additional complexity does not add sufficient value. And I am sure such a change would not be backwards compatible with existing code.

I see the risk of opening the door for more specific handling, though in this case I would argue that the exception in question is part of the same API, and is raised by the implementation itself; having specific "handler APIs" to deal with that, in addition to the existing failure handling, to me would not be a strange addition.
Assuming timeout related convenience functions could be added, without messing with the underlying structures (like the callback queues) and existing behaviour (where the example you pointed to depends on), could this be an addition that would be considered?

Thanks for this proposal, I think it would be a useful addition! I left some comments on  #461 with some suggestions on the API.


Provide a mechanism for programmatically changing a field's placeholder text.

I think @nwhittaker is asking for more than just updating the custom .val() function.
In the example he provided, after:
$('#myinput').attr('placeholder', 'something else');
I would expect
$('#myinput').val();
to return an empty string, but also as @nwhittaker says:
$('#myinput')[0].value
should return "something else", and the user should actually see the new placholder "something else".
That will require more change than just the custom .val().

Well, unless every time the plugin wants to access the placeholder attribute, should always get it without caching it as it is done now. Doing it so, will reduce performance but will always have the latest placeholder value. Unless an extension is made for the attr() function also along with the custom val() but I prefer the first solution have to be tested if it could work for all those cases.


[New] Show device location history

Description
This PR implements Show device location history in Maps category.
URL to README: https://github.com/Esri/arcgis-maps-sdk-swift-samples/tree/Ting/ShowLocationHistory/Shared/Samples/Show device location history
Linked Issue(s)

common-samples/issues/3820

How To Test
Run the sample and see if the track always works, and the objects get deallocated correctly when the sample quits.
Screenshots

To Discuss

Do we need to weakly capture self in the standalone tasks?
Make sure the for await newLocation in locationDisplay.dataSource.locations section is correct. This sample's key part is  to demonstrate async stream is the equivalent of locationChangedHandler pattern.

This is low priority, no hurry to review.

Thanks!


Update vSphere CPI CSI default node selector label

The default node selector label in the vSphere charts needs to be updated to node-role.kubernetes.io/control-plane(Add a dash) once this issue has been closed. This is so that the chart can be deployed in a RKE or RKE2 cluster with default values.
Related issue: https://github.com/rancher/rancher/issues/32604

Closing this; the CPI and CSI charts were updated to use node affinity to handle the discrepancy in the node selector labels and the cluster type enum logic was removed.
https://github.com/rancher/charts/pull/1448
https://github.com/rancher/charts/pull/1451


[FEATURE] Enhancement - Config Rule Enhancements (NIST800-53 Conformance pack, SCPs)

Following the addition of the capabilities to deploy config rules at scale:

migrate the NIST800-53 conformance pack into the Accelerator configuration file (~100 rules)
add SCP's to protect deployed config rules (PBMMAccel- prefixed or tagged) - already protected
add script to convert AWS Conformance pack into ASEA config file format for deployment
NOTES:
Conformance packs are essentially just a deployment engine, for which the ASEA also is
Today, organization conformance packs are limited in rule count, do not allow targeting different packs per OU, and have limits on the number of supported parameters - for these reasons we are deploying config rules directly.

Shipping in v1.2.4

Shipping in v1.2.4


Cleaned up k8s intro.

@djosborne I think we're stepping on each others toes.  I've got a PR up here that also gives this doc a makeover: #247.
We should chat and figure out what to do!

I do like how you've laid this out.  I think we can probably combine what you've done here with what I've got in my PR.

LGTM


Add interpreter and executor tests for join operator

Enhancement

Extract join related tests to separate files.
Add more join tests.

/assign


Native data types

API server returns decimal values as str. Should be float. Same problem with the streaming pricing.
>>> response = ctx.pricing.get(account_id, instruments='EUR_USD')
>>> pprint(json.loads(response.raw_body))
{'prices': [{'asks': [{'liquidity': 10000000, 'price': '1.11095'}],
             'bids': [{'liquidity': 10000000, 'price': '1.11082'}],
             'closeoutAsk': '1.11099',
             'closeoutBid': '1.11077',
             'instrument': 'EUR_USD',
             'quoteHomeConversionFactors': {'negativeUnits': '0.80307094',
                                            'positiveUnits': '0.80291619'},
             'status': 'tradeable',
             'time': '1478199233.623775422',
             'unitsAvailable': {'default': {'long': '0', 'short': '0'},
                                'openOnly': {'long': '0', 'short': '0'},
                                'reduceFirst': {'long': '0', 'short': '0'},
                                'reduceOnly': {'long': '0', 'short': '0'}}}]}

We have chosen to represent all fixed point decimal numbers in the REST API as strings to ensure that precision delivered is exactly as intended. Internally we use a fixed point representation and do fixed point math. If a client wishes to convert the provided values into a floating point representation and perform math on them for their own models they are welcome to, but the over-the-wire representation for these values is a string.

I'm confused by the fact that minimumTradeSize, maximumPositionSize and maximumOrderUnits are decimal numbers. Do you allow trading fractional units on v20 engine?
{'displayName': 'EUR/USD',
 'displayPrecision': 5,
 'marginRate': '0.01',
 'maximumOrderUnits': '100000000',
 'maximumPositionSize': '0',
 'maximumTrailingStopDistance': '1.00000',
 'minimumTradeSize': '1',
 'minimumTrailingStopDistance': '0.00050',
 'name': 'EUR_USD',
 'pipLocation': -4,
 'tradeUnitsPrecision': 0,
 'type': 'CURRENCY'}

Good observation. The v20 engine is capable of dealing with fractional units, however this is not a feature that OANDA is offering at this time.

I would like to get back to this issue again.
This is really inconvenient that you force using str instead of native data types. I'm not suggesting you should change your over-the-wire representation. Just wrap it properly within the wrapper. Higher level of abstraction is what wrapper is for. Isn't it?
Just had the following experience:
>>> ctx.order.create(account_id, order=ctx.order.LimitOrderRequest(instrument='EUR_USD', price=1.07903, units=1000000)).body
{'errorMessage': "Invalid value specified for 'order.units'"}
>>>
>>>
>>> ctx.order.create(account_id, order=ctx.order.LimitOrderRequest(instrument='EUR_USD', price=str(1.07903), units=str(1000000))).reason
'Created'

I would strongly suggest implementing native data types within the wrapper and then casting it to str whenever it's required on the server side.

That sounds reasonable.
I'll also let you configure the context to allow for automatic conversion of the numeric types from string to their native type. This will likely be enabled by default.

In my opinion representing decimal numbers as strings creates unnecessary processing overhead on the client side. That's especially the case when processing streaming rates.

you mean in the JSON wire format?

Yes. If someone wants to control the precision they can use displayPrecision for rounding or string formatting.
It's not GA yet, right? In that case you'd better change it earlier than later. Bids & asks are for math operations, not for printing as strings.

Decimal numbers will always be rendered as strings over the wire as long as we are using JSON as the wire format. For example the difference between
{ "number" : 1.2345 }
and
{ "number" : "1.2345" }
over the wire is simply two "'s.
Regardless of the above representation, the client (either at the library or app level) will have to convert the string "1.2345" into a float. It's far too late to change the representation for this version of the API. I will be adding the option to convert to native types in the library though.

With 200-300 ticks/s that means 200-300 extra function calls and god knows how many extra CPU cycles per second. For what? For no reason. Just because you decided representation is more important than the data itself. It's NOT.

I'm claiming that it's not an extra function call, it's a relocated function call. Converting from string -> float must always be done, either inside of the json.loads() call or after the json parsing (by the v20 library or the client code).

You might be right :)

When are you planning to add option to convert to native types in the library?

i'm just doing some testing, should push it out tonight or tomorrow morning

Native data types seems to be broken in 3.0.12. It is returning str instead of float.
Sample code:
            for msg_type, msg in response.parts():
                if msg.type == 'PRICE':
                    print('Bid {}\tAsk {}'.format(
                        type(msg.bids[0].price),
                        type(msg.asks[0].price),
                    ))

Returns:
Bid <class 'str'>	Ask <class 'str'>
Bid <class 'str'>	Ask <class 'str'>

Used to return float in 3.0.11


Transpiling with webpack creates undefined effects

...
    "redux-saga": "^1.0.2",
    "redux-saga-router": "^2.2.0",
    "redux": "^3.7.2",
    "redux-persist": "^5.8.0",
    "redux-query-sync": "^0.1.7",
    "webpack": "^4.32.0",
    "webpack-dev-server": "^3.4.1",
    "webpack-manifest-plugin": "^2.0.4",
    "autoprefixer": "7.1.2",
    "@babel/core": "^7.4.4",
    "@babel/preset-env": "^7.4.4",
    "babel-eslint": "10.0.1",
    "babel-jest": "^24.8.0",
    "babel-loader": "^8.0.6",
    "babel-plugin-styled-components": "^1.10.0",
    "babel-preset-react-app": "^9.0.0",
...

effects.js:10 Uncaught TypeError: Cannot read property 'call' of undefined
    at Object../node_modules/redux-saga-router/lib/effects.js (effects.js:10)
    at __webpack_require__ (bootstrap:723)
    at fn (bootstrap:100)
    at Object../node_modules/redux-saga-router/lib/router.js (router.js:24)
    at __webpack_require__ (bootstrap:723)
    at fn (bootstrap:100)
    at Object../node_modules/redux-saga-router/lib/index.js (index.js:16)
    at __webpack_require__ (bootstrap:723)
    at fn (bootstrap:100)
    at Module../src/core/sagas.js (reducers.js:1504)

@fooloomanzoo  Any solution to this problem? I am facing the same issue.

If you share a repository with the issue reproduced I can take a look at it.


D programming language implementation

Codecov Report

Merging #36 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #36   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines          93     93
  Branches        5      5
=====================================
  Hits           93     93

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 4e0d919...491e8e8. Read the comment docs.

there is another one: https://github.com/Extrawurst/ulid-d

@Extrawurst feel free to submit a PR to include your library in the list!

@enckse Thanks! 👍


Invalid serialized string

This test outputs an invalid json string:
unittest {
	struct Point
	{
		float[2] coordinates;
	}

	Point().serializeToJson.toPrettyString.writeln;
}

{
	"coordinates": [
		undefined,
		undefined
	]
}

I think there should be null instead of undefined

For which version does this occur? This is supposed to be fixed since #1548 was merged (contained in 0.7.30).

vibe-d:data 0.7.30-alpha.3+commit.30.g82b9d0e if it's fixed, I think we can close this


Debug scripts with root priviliges

Is there a way to run script with root privileges with configuration option or somekind of workaround. I tried to put "sudo" in launch.json but with no luck.

No there is no option for that.
As a workaround, i see 2 options:

https://askubuntu.com/questions/803343/how-to-run-visual-studio-code-as-root/1027243

change every command in script that requires elevation to start with sudo, (hint: https://stackoverflow.com/questions/82256/how-do-i-use-sudo-to-redirect-output-to-a-location-i-dont-have-permission-to-wr) - sudo will ask for password only once

That helped enough.

Please add "sudo": true in the configuration file and then run the script with sudo priviliges


Previews of image attachments are not displayed, regardless of settings

No matter what settings are set, I cannot get graphical attachments to appear in-line.
I have several security cameras that email photos from when movement is captured, and instead of showing me the actual images, k9 shows me these grey boxes with icons.
How can I force k9 to actually show me the images? I'm using the most recent version available in the /releases section here on github.
Screenshots

Environment

K-9 Mail version: 5.718
Android version: 10
Device: Oneplus 7 pro
Account type: IMAP

That's probably because the camera doesn't use the proper MIME type (i.e. image/jpeg) for the attachments.
We'll add some code to make image previews work if the MIME type is application/octet-stream (which basically says "I don't know what kind of content this is") and the file extension is a known value for a supported image format. Hopefully that will make things work for you.
Still, you should report this as a bug to the vendor of your security camera.

These images are from an axis camera, and I'll report this to them as well.


feat: Auto-configure emulator when connecting with JDBC

Connecting to the Cloud Spanner emulator with JDBC can be a little cumbersome, as

The JDBC driver can only connect to existing instances and databases. A freshly started emulator does not contain any of those.
The user needs to either specify the host and port number of the emulator, and set usePlainText to true, OR set the SPANNER_EMULATOR_HOST environment variable.

The first point makes it very difficult to execute tests against the emulator when using only JDBC, as the instance and database needs to be created by an external program/script before a JDBC connection can be created. It also makes it very difficult to connect to the emulator from generic tools like DBeaver.
Adding a connection option to the JDBC driver to automatically configure and connect to the emulator would make testing a lot easier. When the option is set, the JDBC driver would:

Automatically connect to the default host:port combination of the emulator. This can be overridden by the user if they have chosen a custom host/port.
Automatically turn off authentication and use plain text communication.
Automatically create both the instance and the database that are mentioned in the connection string if any of these do not already exist. Any existing instance/database should be used if it already exists.

This feature has been enabled and released in version 2.0.0 thanks to https://github.com/googleapis/java-spanner/pull/931


Pubsub hp

@pongad please take a look, this adds support for falling back to pull and ack when no streaming pull support and fixes the flakiness on the tests

Changes Unknown when pulling 68d79ad1a92d2cb026ebc15d19d63e75b6862b3b on davidtorres:pubsub-hp into ** on GoogleCloudPlatform:pubsub-hp**.

@davidtorres Could you change the commit email? The one you're using, while undoubtedly yours, seems to be running into cla troubles.

Also cc @garrettjonesgoogle

@pongad comments addressed please take a look

@pongad Not sure about the flakyness, can you point me to where you see this?

Changes Unknown when pulling 0d82897a5bef7183c13e94065e50e11cc08e6071 on davidtorres:pubsub-hp into ** on GoogleCloudPlatform:pubsub-hp**.

@davidtorres I'll approve the change from my end, please also address Garrett's comments

@davidtorres The flake I thought I saw seems to have disappeared :smile:

Changes Unknown when pulling 6ef9df77d52a80860476c8fd73d6702732790400 on davidtorres:pubsub-hp into ** on GoogleCloudPlatform:pubsub-hp**.

LGTM, I don't want to block on more refactoring right now.

Changes Unknown when pulling f8c8dc6de6187426eac21198f50ad35768271569 on davidtorres:pubsub-hp into ** on GoogleCloudPlatform:pubsub-hp**.


add more specific error type in ErrorHeaderKind (#14809)

What is changed and how it works?

Issue Number: Close #14808

What's Changed:
Add more specific error type of raftstore error to metrics

Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

None

/retests

/test pull-unit-test

/test pull-unit-test

/test pull-unit-test


Pulse is ignoring timezone setting

Pulse Version
1.2.5
Laravel Version
11.25.0
PHP Version
8.3.10
Livewire Version
3.5.8
Database Driver & Version
No response
Description
After updating Laravel to version 11, Pulse stopped honoring the timezone setting. This may be due to updating Carbon to version 3. Data is presented according to UTC time zone, even though the time zone is set to "Europe/Warsaw" (both in Laravel and PHP configuration).
Steps To Reproduce
Set your Laravel timezone to something other than "UTC" and observe the timeline in your Pulse charts.

Hi, @maciek-szn, thanks for reporting this. I've just some testing and wanted to confirm what I'm seeing is what you are reporting.
I can see that everything still works as expected. The graphs still show the right data in the right places, e.g., data from "now" is always at the right-hand side.
The problem being that when you hover over the dates is you see UTC time in the browser rather than your own timezone.
You would like to see Warsaw's time in this popup, i.e., 2024-10-02 00:36:00 rather than the same time but in UTC format.

I feel that we should likely display the time in your browsers timezone, rather than the applications timezone - potentially with UTC alongside.

You would like to see Warsaw's time in this popup, i.e., 2024-10-02 00:36:00 rather than the same time but in UTC format.

Indeed. In Laravel 10 (i.e. Carbon 2) everything works fine. I didn't look into the code, but maybe that's the reason.

Closing this one as we have a PR open for this: https://github.com/laravel/pulse/pull/417


refactored query client + added env YDB_GO_SDK_QUERY_SERVICE_USE_SESS…

…ION_POOL for use internal/pool in query client + added SLO test for query client with pool

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report
Attention: Patch coverage is 75.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 42.91%. Comparing base (39d0d61) to head (6567cb9).

Files
Patch %
Lines

internal/query/session.go
75.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1306      +/-   ##
==========================================
+ Coverage   41.36%   42.91%   +1.55%
==========================================
  Files         321      258      -63
  Lines       33792    24359    -9433
==========================================
- Hits        13977    10454    -3523
+ Misses      19346    13462    -5884
+ Partials      469      443      -26

Flag
Coverage Δ

go-1.20.x
42.91% <75.00%> (-0.65%)
:arrow_down:

go-1.21.x
?

go-1.22.x
?

macOS
42.91% <75.00%> (+2.01%)
:arrow_up:

ubuntu
?

unit
42.91% <75.00%> (+1.55%)
:arrow_up:

windows
?

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


pr

see commits for details

Please don't merge, configuring travis

okay travis works, merge at will


Chart and legend colors are displayed differently

Description
Chart and legend colors are displayed differently
Related Issue : #3936

How Has This Been Tested?

If you put one dimension value on the cross shelf and set the color setting to the dimension value, the legend color is displayed strangely.
Need additional checks?
Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

deploy to 2


Parametric smartcontracts

It should be possible to forward the event that triggered the execution of a trigger into the body of the trigger, at least in the WASM format.

Blocked by #2314


WFLY-9802: Minimize WildFly-specific stuff in welcome-content

https://issues.jboss.org/browse/WFLY-9802

Can one of the admins verify this patch?

@bstansberry could you check this PR ? https://issues.jboss.org/browse/WFLY-9802 was filed by you.

this is ok to test

Thanks, @mwpark92! I have one administrative question before we can merge what I think is your first commit to WildFly / WildFly Core:
We require that all contributions be made under the terms of the MIT License, http://www.opensource.org/licenses/mit-license.php. Do you agree to the these terms?


Same snippet of JS (wasm_inline_js.rs)

I have an error with the latest released version of cargo-web:
   Compiling showcase v0.1.0 (file:///home/travis/build/DenisKolodin/yew/examples/showcase)
    Finished release [optimized] target(s) in 6m 48s
    Garbage collecting "showcase.wasm"...
    Processing "showcase.wasm"...
thread 'main' panicked at 'internal error: same snippet of JS (by value)
is used with two different shims; please report this!', src/wasm_inline_js.rs:105:17
note: Run with `RUST_BACKTRACE=1` for a backtrace.

https://travis-ci.org/DenisKolodin/yew/jobs/408902303#L885
It works locally where I had updated cargo-web from master. I think the new release can fix that.

@DenisKolodin Sorry for the delay and thanks for the reminder!


Add additional step required for parameters to cascade from Main component

[EDIT by guardrex to fix the code blocks]
The documentation is lacking one crucial step (and another is incorrect). The steps related to editing the Main.razor file should include the following:
<Found Context="routeData">
  <CascadingValue Value="@KeypadViewModel">
    <RouteView RouteData="@routeData" DefaultLayout="@typeof(MainLayout)" />
    <FocusOnNavigate RouteData="@routeData" Selector="h1"/>
  </CascadingValue>
</Found>

The next steps that state you need to add a private field to MainLayout.razor should be updated to:
[CascadingParameter]
public string? KeypadViewModel { get; set; }

Based on my testing, this is the only way to get the value passed using the RootComponent.Parameters property to actually cascade from root. After making these changes, the parameters work as expected.

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

ID: 5e560e9f-cf5e-d6ab-6ba3-3bf7a45986a5
Version Independent ID: 5e560e9f-cf5e-d6ab-6ba3-3bf7a45986a5
Content: Pass root component parameters in ASP.NET Core Blazor Hybrid
Content Source: aspnetcore/blazor/hybrid/root-component-parameters.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Thanks @UnusualJustin ... I'll take a look at this first thing tomorrow morning.

@guardrex, I'm not sure how relevant it is, but for my testing I'm using MAUI BlazorWebView / .NET 7. If I can be of any other help (or if you discover that I'm incorrect), please let me know.

... and the 🧀 moved in the sample app. The files that the document says are in the sample app aren't present, so I'll need to track those down. Even if I do find them for new cross-links, I think it's best if the topic keeps a copy of them in the text, so I'm going to add them to the topic.

It either moved or the link was never correct. That sample app is over here ...
https://github.com/davidbritch/dotnet-maui-samples/tree/main/XAML/XamlSamples/XamlSamples
... but I would like to see about the topic hosting the sample code to prevent this from happening again.

I actually found my local sample app for this experience, and it runs here ...

I'll check your remarks against the code that I have. Stand-by .........................

But I thought that the KeypadViewModel was assigned in the dict passed to the params in MainPage.xaml.cs, and then the Keypad component's KeypadViewModel comes from that dictionary.
public MainPage()
{
    InitializeComponent();
    rootComponent.Parameters = new Dictionary<string, object> { { "KeypadViewModel", new KeypadViewModel() } };
}

We need to ping the engineer who wrote the article and built the sample ... @davidbritch ... David, can you take a look at this issue and confirm that the view model instance is passed correctly using the article/sample's guidance? If not, do we need to make the changes that @UnusualJustin is suggesting?

I see what you mean. Let's let @davidbritch take a look and confirm, given that this content/sample came from him or from the sample code in the sample app over there, then we can proceed to change the example with his blessing. He should be along in a bit to take a look at this.

I've just looked at this and I agree with @UnusualJustin . From the doc it looks like:

A KeypadViewModel is instantiated in MainPage.xaml.cs that's passed to the KeypadViewModel property in Main.razor.
Another KeypadViewModel is instantiated in MainLayout.razor, which then cascades down the component hierarchies.

new KeypadViewModel is instantiated in MainLayout.razor, which is then cascaded. So while that shows the technique, IMO a better example would be to cascade KeypadViewModel from MainPage.xaml.cs into MainLayout.razor.

I think I figured out what I'm doing here .... MAYBE 🙈😆. I agree with your suggested testing approach, @UnusualJustin, of setting an Id property value in the dictionary to confirm the behavior with ...
rootComponent.Parameters =
    new Dictionary<string, object>
    {
        {
            "KeypadViewModel",
            new KeypadViewModel()
            {
                Id = "Set in MainPage.xaml.cs"
            }
        }
    };

... and that did work using your suggested code updates ...

I'll see about writing up the changes now, and I'll ping you both on the PR for review.


Added option to turn off fancy discord messages

With this setting disabled the bot will not touch any Discord message, just relay it to Factorio. The messages from Factorio still have the icon.

Will be added when #5 gets merged, which also makes deleting messages optional.


gremlin 3.5 (dependabot proposal) causes breakage

In recent dependabot PRs, and update to gremlin 3.5.0 has been proposed.
However in testing for #5291 a variety of problems were noted

open lineage graph connector needs to use org.apache.commons:commons-configuration2 instead of commons-configuration:commons-configuration (minor import change)
Some FVTs using the graph repository fail. This includes a) not initializing (classpath?), issues with searches (bug, or bug fix?)

Opening issue to relook at this and capture exact data, Sticking with gremlin 3.4.x until resolved (and until other dependency updates are merged so as to get a clean comparison)

See #5275

Example failure:
DataManagerOMASCreateDatabaseIT > testCreateDB(String) > org.odpi.openmetadata.accessservices.datamanager.fvt.execution.DataManagerOMASCreateDatabaseIT.testCreateDB(String)[2] STANDARD_OUT
    ==========================================
    CreateDatabaseTest FVT failed
    Number of Failed tests: 1
    Captured Exceptions: 1
     ==> org.odpi.openmetadata.fvt.utilities.exceptions.FVTUnexpectedCondition with message createDatabaseTable
    FVTUnexpectedCondition{testCaseName='CreateDatabaseTest', errorDescription='createDatabaseTable', caughtException=PropertyServerException{reportedHTTPCode=500, reportingClassName='org.odpi.openmetadata.commonservices.ffdc.RESTExceptionHandler', reportingActionDescription='getDatabaseTablesByName', reportedErrorMessage='OMAG-REPOSITORY-HANDLER-500-001 An unexpected error org.odpi.openmetadata.repositoryservices.ffdc.exception.RepositoryErrorException was returned to getEntitiesByName by the metadata server during getDatabaseTablesByName request for open metadata access service Data Manager OMAS on server servergraph; message was OMRS-REPOSITORY-400-055 An unexpected java.lang.ClassCastException exception was received from a repository connector during the findEntitiesByProperty operation which had message: class org.apache.tinkerpop.gremlin.structure.util.empty.EmptyGraph cannot be cast to class org.janusgraph.graphdb.database.StandardJanusGraph (org.apache.tinkerpop.gremlin.structure.util.empty.EmptyGraph and org.janusgraph.graphdb.database.StandardJanusGraph are in unnamed module of loader org.springframework.boot.loader.LaunchedURLClassLoader @1b604f19)', reportedErrorMessageId='OMAG-REPOSITORY-HANDLER-500-001', reportedErrorMessageParameters=[OMRS-REPOSITORY-400-055 An unexpected java.lang.ClassCastException exception was received from a repository connector during the findEntitiesByProperty operation which had message: class org.apache.tinkerpop.gremlin.structure.util.empty.EmptyGraph cannot be cast to class org.janusgraph.graphdb.database.StandardJanusGraph (org.apache.tinkerpop.gremlin.structure.util.empty.EmptyGraph and org.janusgraph.graphdb.database.StandardJanusGraph are in unnamed module of loader org.springframework.boot.loader.LaunchedURLClassLoader @1b604f19), getDatabaseTablesByName, Data Manager OMAS, servergraph, org.odpi.openmetadata.repositoryservices.ffdc.exception.RepositoryErrorException, getEntitiesByName], reportedSystemAction='The system is unable to process the request because of an internal error.', reportedUserAction='Verify the sanity of the server.  This is probably a logic error.  If you can not work out what happened, ask the Egeria community for help.', reportedCaughtException=null, reportedCaughtExceptionClassName='null', relatedProperties=null}}

As per https://github.com/JanusGraph/janusgraph/pull/2619, this will be addressed in JanusGraph 0.6. As such will close this issue and ignore the dependabot report for now


Is there official sample used typescript for draw chart?

Documentation Is:

[X] Missing or needed?
[X] Confusing
[ ] Not sure?

Please Explain in Detail...
I used to use version 2.9.4. That typescript definition is excellent.
Now I update project and use version 3.9.1 and :

I confused!
Some options is not define or not work (I mentioned some of them in the issues)

Your Proposal for Changes
Now!
Is there official sample used typescript for draw chart?
Example
No response

No there is none, if you follow the documentation everything should work fine, also typescript should point out any errors you are misconfiguring so I don't really understand what you might be having issues with especially since you dont provide any issues you are having.
So if options are not defined/working please add them so we can take a look at those

Closing since you did not respond with what typings miss or what is confusing.


Add new logo for powerful seal

A new logo for our Powerfulseal friends!
Signed-off-by: Chris Aniszczyk<EMAIL_ADDRESS>Resolves https://github.com/bloomberg/powerfulseal/issues/60

We'll need to know the copyright holder's name for the logo artwork (LF or CNCF or whatever is correct), and a suitable license will need to be chosen for it as well since the Apache 2 license on the repo isn't really appropriate for artwork. CC-BY-SA or CC-BY would be fine, depending on what the copyright holder prefers for downstream users of the logo.

We are OK with licensing under CC-BY as the logo creator.

Thanks. What should the copyright holder's name be?

Linux Foundation

OK, then we can add a line to README.md like this:
PowerfulSeal logo Copyright 2018 The Linux Foundation, and distributed under the Creative Commons Attribution (CC-BY-4.0) license.

Perfect ! Thanks so much, @caniszczyk and @kpfleming 👍


Slipstream can be used to make portals between instances

@TracyTheBoy Funnily enough this spell actually handles instances correctly. The bug is really "Naxxramas is not registered as an instance in InstanceSetup.cs".

In 6d59239a66fe0f67a23ff8917b28095f84370a29 Naxxramas got moved a bit so this specific case should be fixed.
Also I found this piece of code in InstanceSetup.cs
    var naxxramas = new Instance("Naxxramas", Regions.NaxxramasInside);       naxxramas.AddDependency(preplacedUnitSystem.GetUnit(Constants.UNIT_U01X_HEART_OF_NAXXRAMAS));       naxxramas.AddDependency(preplacedUnitSystem.GetUnit(Constants.UNIT_E013_NAXXRAMAS_SCOURGE));       InstanceSystem.Register(naxxramas);
So it looks like there is a bug with the spell after all, what do you think @YakaryBovine ?

Yup, looks like it. Let's leave this issue open and I'll check it out.

@TracyTheBoy It turned out to indeed be an issue with Naxxramas configuration. The "NaxxramasInside" region doesn't actually cover the entire instance.


maxVersion and lookup for tags in merged branches should be default behavior

Current lookup of commits relies on a walk of all commits reachable from the HEAD without any control on how merge & parent commits are.
By default jgitver stops the lookup on the first encountered version tag.
With #49 a new parameter controls, if jgitver stops or if continues the lookup to find other candidates version tags ; choosing in the end the greatest found.
Instead of parsing the commits in a random order as it is today using the standard RevWalk, a better control of the graph parsing should be done and instead of stopping at the first tag encountered, jgitver should retrieve the first tag in each branch it is able the reach from the HEAD and then keep only the greatest.

@djarosz what is your opinion?

I think this behaviour would be more natural and easer to understand so I think it is good idea.
With current behaviour only the only way to backport some changes from other branches (i.e production branch fixes) without polluting version numbers (as it does in my current projects) is to do cherry-picking. Using simple 'git merge' in those situations would be more natural in my opinion.
But still changing default behaviour might (?) introduce some incompatibilities for current users.

But still changing default behavior might introduce some incompatibilities for current users.

It might but I prefer to correct it right now instead of spreading a "bug feature".
I quickly implemented it this morning and all tests passed correctly ; I'll put the changes today later.
It has also the benefit that bringing it as a default does not require any of the plugins to implement additional behavior.
Thanks for your very quick feedback.

Great. And of course after merging Your code my PR in plugin projects can be rejected.


No throw literal

Will approach this issue at a later time

Will approach this issue at a later time


Cleanup stuck pods

Closes: https://github.com/astronomer/issues/issues/1557
In summary, we would like to delete any pod that is older than X minutes and not at full container readiness

some difference in flake8 locally and in CI so give me a sec to sort that out

ah ok. can reproduce locally after:
 pip install -e '.[test]

I'll wait on @schnie 's review before merging because this one is a significant behavior change.

I will need to follow up by integrating this into AC. I'm going to leave this PR open so I remember to do that.


Minor update needed in the French translation

I have checked the following:

[X] I've searched existing issues and found nothing related to my issue.

Describe the feature you want to add
Minor changes occurs in the English version of the Readme file that need to be translated into the French version (for isntance, the Github Sponsor link).
This issue is part the PR requirements.
Mockups or Images of the feature
N/A

@sanjai0py @lohxt1 the PR for this issue was closed so this issue likely can be closed.


how to handle multi byte characters ?

my code like this
                .push(
                    Button::new(&mut self.reset_button, Text::new("ＲＥＳＥＴ"))
                        .on_press(Message::Reset),
                )

but the result showed this

iced supports multi byte characters?

See #33.


On macOS keyboard shortcut is shown as Ctrl+Enter instead of Cmd+Enter

Describe the Issue
Even though on macOS it would seem that Ctrl+Enter does not work & Cmd+Enter does; website still tells you to use Ctrl+Enter.
I think this might end up being confusing for some - and should be fairly straightforward of a change.
Affected Page
All coding pages
Steps to Reproduce
You must use macOS for this issue.

Go to any coding challenge
Type your changes
You will see button underneath says "Check your code (Ctrl + Enter)"
Press Ctrl + Enter
Nothing will happen.
Press Cmd + Enter
Expected behaviour will occur.

Expected behavior
It should tell the user correct shortcut (Cmd + Enter) instead of (Ctrl + Enter) depending on the platform.
Screenshots

System

Device: M1 MacBook Pro
OS: macOS 12.0.1
Browser: Safari
Version: 15.1

Additional context
No response

I've created the PR however;

I've updated non English translations as well in here to use dynamic values; however not really sure how to align it with the crowdin. Since updated translations should coincide with this change; any opinions on this?

https://github.com/freeCodeCamp/freeCodeCamp/pull/46428#issuecomment-1151568332

Since updated translations should coincide with this change; any opinions on this?

Do not worry, I think you are putting too much effort into it. Simply drop the changes to those files and as the bot mentioned, they will get updated eventually via our translation workflow.
As for the bug itself:
I am 100% sure at some point we introduced a regression, because Ctrl + Enter (in addition to the implicit Cmd + Enter) was working since the inception of the code editor.
@freeCodeCamp/dev-team do you happen to know if this was just an accident with the new editor?

I can't check this myself, since I don't have a mac, but it should work. This is the code:
https://github.com/freeCodeCamp/freeCodeCamp/blob/1083f1d5c5a58a4b399f1af4842abd96443aff0b/client/src/templates/Challenges/classic/editor.tsx#L427-L431
and
https://github.com/freeCodeCamp/freeCodeCamp/blob/1083f1d5c5a58a4b399f1af4842abd96443aff0b/client/src/templates/Challenges/components/Hotkeys.tsx#L45
@moT01 could you try and reproduce the issue?

@ojeytonwilliams
I've shot a short video of the behaviour I am seeing; if that helps.
https://www.loom.com/share/9d590c0470c449b69e8fa83cbce5de45

That does help, yeah. It seems to be ignoring the command button entirely, which is odd.
I can't investigate myself, but this will help anyone that has access to a Mac.

Thanks for the video @tugkankibar 👍

I am 100% sure at some point we introduced a regression ... do you happen to know if this was just an accident with the new editor?

I don't recall if ctrl+enter ever worked on mac - I may or may not have ever used it. I just hit buttons without looking 😬
I'm not sure if you are trying to show something specific with those code snippets @ojeytonwilliams - that's the code on main. Which results in the current behavior.
Here's the current behavior I see (all on mac) - there's some nuance...
New RWD:
Execute: only command+enter works on mac
Submit: Enter alone works, so any key+enter works
Legacy RWD, JS, etc:
Execute: Only cmd+enter works
Submit: Enter alone does not work - both ctrl+enter or command+enter do work
Multiple choice Python video questions:
Execute: Enter alone does not work - ctrl+enter and cmd+enter work
Submit: Enter alone does not work - both ctrl+enter or command+enter do work
Projects with URL inputs - e.g. front end libs cert project, or backend api challenges, etc:
Execute: Enter alone works cause it's a form I suppose, which also means any key+enter works
Submit: Enter alone does not work - both ctrl+enter or command+enter do work
The last three groups there have the same submit modal, so the behavior is the same on all of them. What I see is that the submit modal, and the video question both allow [ctrl or cmd] +enter - we should see what those are doing.

K, I think I found a fix.
ctrl+enter works when the hotkeys package/component is used. So crtl+enter actually does work on the legacy challenges when the instructions are focused. When the editor is focused, those hotkeys don't get used, the actions/hotkeys we add to the editor take over.
I made ctrl+enter work when the editor is focused by changing that line 431 @ojeytonwilliams mentioned to this:
keybindings: [monaco.KeyMod.CtrlCmd | monaco.KeyCode.Enter, monaco.KeyMod.WinCtrl | monaco.KeyCode.Enter],
That should work. There's a number of actions in that file that use the same hotkeys - lines 457 and 469 for example. There may be others. I think we should be safe to change all those.

Nice work, Tom.  I thought CtrlCmd was "Ctrl or Command", but it must be Ctrl on Windows and Command on Mac :man_facepalming:
I guess this has never worked properly.

@moT01 happy to give it a look and update the PR. Thank you 🙏

Nice work, Tom. I thought CtrlCmd was "Ctrl or Command", but it must be Ctrl on Windows and Command on Mac 🤦♂

Here's an issue I had created sometime back when working on the editor. Apparently the logic goes like this - the first word is Windows and the second one is Mac button.


Add prometheus in helm chart

Signed-off-by: yeya24<EMAIL_ADDRESS>What problem does this PR solve?
What is changed and how does it work?
Check List
Tests

Unit test
E2E test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has Go code change
Has CI related scripts change
Has Terraform scripts change

Side effects

Breaking backward compatibility

Related changes

Need to update the documentation

Does this PR introduce a user-facing change?:

Add prometheus in helm chart

Codecov Report

Merging #230 into master will increase coverage by 2.86%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #230      +/-   ##
==========================================
+ Coverage   40.78%   43.64%   +2.86%
==========================================
  Files          20       18       -2
  Lines         738      685      -53
==========================================
- Hits          301      299       -2
+ Misses        404      353      -51
  Partials       33       33

Impacted Files
Coverage Δ

controllers/timechaos_controller.go

api/v1alpha1/timechaos_types.go

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 c281c88...6a654e6. Read the comment docs.

Updated. PTAL


Advancing Wiki

[ ] coding standards
[ ] explanation of project

overview of architecture of project (models, etc.) was removed from the acceptance criteria for the current sprint, because the architecture still needs a lot of improvement and a stable base in order to correctly clearly define it using models.


Detox test setup

PR: https://github.com/Blockchain-Powered-eSIM/LPA/pull/42

Basic Detox setup for end to end testing
Add tests
Write build script to update config of mmkv-storage library

Build command: detox build --configuration android.emu.debug
Test command: detox test --configuration android.emu.debug

https://github.com/Blockchain-Powered-eSIM/LPA/issues/46#issuecomment-2050698893
This is the current status for the issue, the error hit when running detox test --configuration android.emu.debug needs to be resolved. Next action item would be unit test integration

The error mention there are no modules find as mentioned in jest.config.fs in e2e:
/** @type {import('@jest/types').Config.InitialOptions} */
module.exports = {
  rootDir: '..',
  testMatch: ['<rootDir>/e2e/**/*.test.js'],
  testTimeout: 120000,
  maxWorkers: 1,
  globalSetup: 'detox/runners/jest/globalSetup',
  globalTeardown: 'detox/runners/jest/globalTeardown',
  reporters: ['detox/runners/jest/reporter'],
  testEnvironment: 'detox/runners/jest/testEnvironment',
  verbose: true,
};

const baseConfig = require("../base.jest.config.js");

module.exports = {
  ...baseConfig,
  setupFilesAfterEnv: ["./init.ts"],
  setupFiles: [
    "<rootDir>/../../../node_modules/react-native/jest/setup.js",
    "<rootDir>/../../../node_modules/react-native-gesture-handler/jestSetup.js",
    "<rootDir>/../../../test/setup.ts",
  ],
  testPathIgnorePatterns: ["/node_modules/"],
};

there is no setup.js in node_modules/react-native/jest
there is no react-native-gesture-handler in node_modules
there is no test file in root directory

error resolved, Detox tests are now running and communicating, the verification is it indicated by the WebSocket (ws) messages between the Detox server and client.
what's happening and what need to check next:
WebSocket Messages:
ws-server get data: The server receives a request to check if the app is ready (isReady).
ws-server send data: The server is sending a response or forward another isReady check.
ws-client send message: The client sends a message back, possibly indicating its current status or asking for the next instruction.
using detox test --configuration android.emu.debug --loglevel trace helps identify at which point the interaction fails or stalls, giving more insight into whether the issue lies with the app's readiness or the test setup.
updated codebase

The detox setup is tested now , the next is to write specific test cases and test them manually using emulator.
Keeping this test setup and testing using emulator for now.

test cases:

[x] App Launch Test
[x] Modal Visibility Test
[x] Unique ID Fetch Test
[x] Modal Dismissal Test

npm run :
><EMAIL_ADDRESS>test
> detox test --configuration android.emu.debug

09:34:03.799 detox[78599] B jest --config e2e/jest.config.js
09:34:06.951 detox[78600] i firstTest.spec.js is assigned to emulator-5554 (Pixel_3a_API_34_extension_level_7_arm64-v8a)
09:34:06.953 detox[78600] i App Launch Test: should launch the app successfully
09:34:09.312 detox[78600] i App Launch Test: should launch the app successfully [OK]

09:34:09.313 detox[78600] i Modal Visibility Test: should initially hide the modal
09:34:09.444 detox[78600] i Modal Visibility Test: should initially hide the modal [OK]

09:34:09.445 detox[78600] i Unique ID Fetch Test: should fetch and display unique ID in modal
09:34:10.922 detox[78600] i Unique ID Fetch Test: should fetch and display unique ID in modal [OK]

09:34:10.924 detox[78600] i Modal Dismissal Test: should dismiss the modal when "Back" button is pressed
09:34:12.092 detox[78600] i Modal Dismissal Test: should dismiss the modal when "Back" button is pressed [OK]

 PASS  e2e/firstTest.spec.js (7.712 s)
  App Launch Test
    ✓ should launch the app successfully (2358 ms)
  Modal Visibility Test
    ✓ should initially hide the modal (131 ms)
  Unique ID Fetch Test
    ✓ should fetch and display unique ID in modal (1476 ms)
  Modal Dismissal Test
    ✓ should dismiss the modal when "Back" button is pressed (1167 ms)

Test Suites: 1 passed, 1 total
Tests:       4 passed, 4 total
Snapshots:   0 total
Time:        7.783 s, estimated 8 s
Ran all test suites.

added e2e/README.md to guiding through tests

closing this issue since unit tests is merged, here's the PR .


Verify the late-failing implementation of lookup

in issue https://github.com/haskell/containers/issues/794 and PR
https://github.com/haskell/containers/pull/800 an alternative
implementation of lookup is proposed which checks if the key is as
expected only when reaching the leaf. This means less branching in the
success case or when the lookup key shares long prefixes with existing
values, but may do more work when the lookup key is far away from
existing keys.
Not saying that it’s particularly doubtful that the changed code is
correct, but since we have the machinery, why not verify it? So here we
go, and yes, it goes through.
I don’t think we need to merge the PR like this, and can probably wait
for the code to reach a containers release, and then include it there.
Also, this PR currently contains a bunch of local hacks that I used to
get going again (now that I use NixOS), which need to be removed, merged
separately, or made obsolete by better changes to the setup.

https://github.com/haskell/containers/pull/800 got merged, so maybe we should merge this too

Ah, maybe not: As I said in the  PR description, we should just update the version of containers that we verify


Remove Euler angles from logfiles

@tumbili @thomasgubler we discussed this a while back, did you proceed with this? As it will induce confusion during the transitional phase, it would be good to digest this rather sooner than later.
@DrTon does FlightPlot support quaternion conversion already?

I started this indeed. You said that you would like a vtol status topic publishing a quaternion of the transformation. Do you mean the quaternion of the transformed attitude? Because the transformation cannot really be expressed as a rotation it is really more a change of body frame axes. We could instead introduce three new parameters in the vtol module which indicate the change of axes from one mode to the other, e.g.
[1,2,3] -> no transformation, same attitude
[-3,2,1] -> x axis becomes negative of old z axis, y stays the same, z becomes old x axis

@DrTon does FlightPlot support quaternion conversion already?

Just implemented it in FlightPlot (in master but not yet in binary), only need to adjust default fields names and test.

@tumbili, why not to use "fixed wing style" attitude and just add rotation in attitude and position controllers for VTOL mode? In this case attitude and attitude setpoints will be consistent and you will always clearly determine orientation of vehicle. To avoid singularities we can use quaternions, and to display orientation of VTOL in "copter style" I can add additional rotation to FlightPlot.

Using two attitude representations for the same vehicle is very confusing IMO...

@DrTon This generally sounds like a good idea keeping in mind the following:
The reason why we transformed attitude was because it allowed us to use the conventional body axes of both multicopter and fixed wing. Therefore, we only had to make minor changes in the fw_att_contoller to make things work. If you want to use one set of body axes then you need to transform all quantities for either fixed wing or multicopter module. If this is not related with unnecessary complexity then I agree to go with your proposal.

There is no complexity associated with it. The VTOL controller should emit the VTOL status topic, which allows the fixed wing controller to pick the transformation off it and apply it once. Same for logging: VTOL status holds the transformation and the plotter applies it locally.
The current state was ok to get things up and running, but it ties layers and apps together that have nothing to do with each other, hinting that we need to improve its architecture.
Since we apply axis changes to sensors based on a rotation matrix, I'm not sure why we couldn't do the same for the VTOL, if its done in the right coordinate frame.

@LorenzMeier Basically what your are saying is the same that we have at the moment:
Fixed wing attitude controller checks if is_vtol= true and if yes it applies the transformation (hardcoded though). Your proposal is just a bit cleaner but I wouldn't say it breaks any ties between layers.
And with rotation you probably mean something like this https://github.com/PX4/Firmware/blob/master/src/lib/conversion/rotation.cpp which is exactly what is done in the fw_att_control module at the moment just only for one case.
@DrTon @LorenzMeier I think in the end we will be looking (at least I hope so) for a controller that can handle a VTOL as ONE vehicle without always having to differentiate between fixed wing and VTOL.


it seems not working on Ubuntu 24.04 LTS

it seems not working on Ubuntu 24.04 LTS

Could you add more details?


MIB2 STD ZR Developer mode activation error

I am trying to enable developer mode in 5F module using ODIS engineering getting error "some adaptations could not be written.. function not available" I am new to this world...any guidance would be appreciated. I don't have VCDS or other interfaces.

This is the repository for the MIB2 High, not the standard.


Use maven wrapper to lock mvn version

Azure pipeline fails due to Tycho bug in Maven 3.6.1.
https://bugs.eclipse.org/bugs/show_bug.cgi?id=546463
Use mvnw to lock the version.

@Flanker32 This project fails to build with JDK 10+ . Here I simply set the ubuntu dist to trusty which uses JDK8