CI: Test minimum dependencies

This sets up Travis to test minimum dependencies. I don't have permissions to turn on Travis for SDCflows.

Hello @effigies, thank you for submitting the Pull Request!

In the file tools/update_requirements.py:

Line 2:1: F401 'sys' imported but unused
Line 20:1: E302 expected 2 blank lines, found 1
Line 28:1: E305 expected 2 blank lines after class or function definition, found 1

To test for issues locally, pip install flake8 and then run flake8 sdcflows.

@oesteban This is failing with nipype 1.3.1 and traits 6.0.0. The issue was fixed in 1.4.2 (https://github.com/nipy/nipype/pull/3159).
Not sure what the best way to handle sudden API breakages in packages a couple levels upstream would be. It might be that testing minimum dependencies in a package with this big a dependency tree is just not feasible.

Not sure what the best way to handle sudden API breakages in packages a couple levels upstream would be. It might be that testing minimum dependencies in a package with this big a dependency tree is just not feasible.

Yeah, I don't have an answer to that either. What do you feel would be the best course of action on this matter, at this point?

This PR, amongst other things, has triggered the combo d01d88229ea535beeebfa8f76fb29b802f08783b, 01c8188735ffa76f870997b47a27718625f4b143 (which means, has made me revise all the dependencies) - so it is a good thing to merge asap.
Iterating over this, we can try to expand this base to follow up dependencies of niworkflows, nipype, pybids and templateflow. Including something like that, we should be in acceptable shape.


Autogenerate component imports at build time.

Splitting this off of #3168. This is a step towards #3168 that adds value to several different scenarios, whether or not we decide to build #3168 or not.
In what area(s)?

/area runtime
Describe the feature
I propose that we implement the following:
Our build system should take a configuration file as an input with a list of components to include in the build. Based on this input, we should generate a component loader that is used by cmd/daprd/main.go. The output would be a daprd binary that only contains the desired modules, and nothing else. This gives us two short term benefits:

Ease of adding new components to dapr/dapr code.
Anyone that wants can show up with their own input file and have their own flavor.

In the long term, if we want to maintain flavors as in the long term as this proposal suggests, doing so becomes as easy as maintaining a yaml file.
I took a first pass at this, and I propose the following format for the component list yaml file:

root_package: "github.com/dapr/components-contrib/"
component_typenames:
  secretstores:
    - runtime_callback: WithSecretStores
      components:
        - path: "aws/secretmanager"
        - path: "azure/keyvault"
...
  bindings:
    - runtime_callback: WithInputBindings
      components:
        - path: "aws/kinesis"
        - path: "zeebe/jobworker"
...
    - runtime_callback: WithOutputBindings
      components:
        - path: "alicloud/oss"
        - path: "apns"
...

With that data, and some minor changes to some component initialization conventions, the code generation piece of this would be relatively straightforward. We could be building daprd instances with arbitrary components in no time.
Release Note

RELEASE NOTE: Imported components are now configurable.

Personally, I'd like to see json vs. yaml since it's easier to generate, build an API around and you don't have to remember to quote certain things.

There's some prior art doing this with Caddy which may be useful.

Personally, I'd like to see json vs. yaml since it's easier to generate, build an API around and you don't have to remember to quote certain things.

True, but yaml is pretty much the default everywhere else with dapr. And since all json is yaml (with a few edge case exceptions), json will still be accepted and valid.

There's some prior art doing this with Caddy which may be useful.

Oh, thanks! I'll be sure and check this out.

created issue to track Dapr docs for this feature - https://github.com/dapr/docs/issues/2085

Moved to 1.7


Screen flicker when running application

Hi, first of all thanks for this project, it has been very useful.
I have built an application on top of it, which renders ER diagrams using graphviz by sourcing a database schema file. Check the code here.
There are two output branches on that app, one that spits out a text representation (in the DBML form) and another one that uses graphviz-java to output Dot/Svg/Png files.
What I (and other users) noticed is that, when running the graphviz-java path, my entire screen flickers for a brief moment (in other words, it blinks in black and then back to normal).
FWIW, I'm on a MacBook with MacOS Catalina 10.15.4 (19E287).
Since the idea for my project is to be run quite frequently (many times a day), the flicker starts to get a tad annoying.
I glanced over the other issues in this repo but couldn't find references. Can you please advise on how to prevent it?

Hi there
I noted this too (also on Mac) and I suspect it has to do with the Graal Javascript engine.
But I haven't investigated on this yet.

The cause was using AWT classes without drawing on the screen. Using headless mode fixes this in version 0.16.2.

Btw. as you're using kotlin, you might be interested in using graphviz-kotlin?

What is (or where is) graphviz-kotlin?

Excellent, thank you very much for the quick turnaround. I verified the fix for this 👍

I have plans on trying out graphviz-kotlin soon, but for the first releases I preferred the java API due to the extensive documentation in the PRs/issues of this repo. Now that FloorPlan has a somewhat covered usecase, we could try replacing the java bindings with kt ones.

graphviz-kotlin is a kotlin DSL for graphviz. It's here: https://github.com/nidi3/graphviz-java/tree/master/graphviz-kotlin 😄


Update postgres test connection form

Currently postgres allows only host name, user and password in the config form. Update it to take port and database as well.

Can we add two more related tasks in this ticket?

Add the integrations for JDBC compatible DBs (via connection string).
Credentials are visible for Postgres currently after saving.

Noted, will release a fix for all all 3 issues and JDBC connection string connector as part of new feature.

We have released a fix for these.


Conversation notification settings

Closes https://github.com/yunity/karrot-backend/issues/490
Needs https://github.com/yunity/karrot-backend/pull/500
Adds a notification setting to the group wall.
It supports to trigger the conversation mute via URL parameter for supporting unsubscribe links.
I made it work when the user logs in during the process, as I guess it's a typical use case for people to stop their e-mail subscription when they haven't visited for a long time and their session expired.


Test looks wrong.

See:
https://github.com/EricSmekens/jsep/blob/79defb44cd2212f93a004dd6e7524c10d4d1fcf4/test/tests.js#L11

Oops, thanks, fixed. Thankfully, it didn't appear to affect any actual tests.


Add .ssh/config

machines.sh aliases shell commands that assume a .ssh/config is in place.. but I couldn't find an appropriate .ssh/config

Can't add it, because it has secrets. And you can't include one SSH config in another, unfortunately.
I avoided the issue in 6b8ec0f75554f2d25d531a43aa8e962ae1c613a5 by making the aliases more explicit about the machines to which they connect.

Years later, you now can Include SSH config files. So I did it with 31f689c75b416ddd2500d6e34b2bc9735f10f507.


[do not merge] Preview with new results

Preview link

@rcannood this PR contains both the new results and the new metadata -- why do I not see them in the preview?

Ah, current setup requires running https://github.com/openproblems-bio/website/blob/update_wowchemy_and_benchmarks/content/benchmarks/_generate_task_pages.py after changing the data file. Didn't automate that part because of #102.

I'll run the command tomorrow morning :)

Gotcha. I can include that in the gh action for future runs. Thanks!
On Sun, 5 Feb 2023, 17:08 Robrecht Cannoodt, @.***>
wrote:

I'll run the command tomorrow morning :)
—
Reply to this email directly, view it on GitHub
https://checkpoint.url-protection.com/v1/url?o=https%3A//github.com/openproblems-bio/website/pull/121%23issuecomment-1418277988&g=ODMzYTRlMmY4MjM0Y2RkOQ==&h=ZTJiYTZiZTA0OWU4Nzg5NjEzODNkOGM5MGNmNTE3ZGNlMWRmNTQ5NmZhZGMyNDAzN2ZiMTJkNjNlYmEyY2QwYQ==&p=YzJlOmltbXVuYWk6YzpnOjdiM2ExOGIwZmE3Y2QzMmJlYjRlMTY0OTQ5ZDQxZGYxOnYxOmg6VA==,
or unsubscribe
https://checkpoint.url-protection.com/v1/url?o=https%3A//github.com/notifications/unsubscribe-auth/AUHCMASUPLWP3U34F44XCLDWWAQGPANCNFSM6AAAAAAUR7XR4E&g=OGU3YjFlNTE0ODA4OTAwMw==&h=NjFkYTk4YjJmNjcyMDQ1YjExMmQ1MmJlYmU4ZmU3YjkzMDZjMmFiODI5MTI5Y2FjYTAxNTQxYjY5NjRiYmMyMg==&p=YzJlOmltbXVuYWk6YzpnOjdiM2ExOGIwZmE3Y2QzMmJlYjRlMTY0OTQ5ZDQxZGYxOnYxOmg6VA==
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
PLEASE NOTE: The information contained in this message is privileged and
confidential, and is intended only for the use of the individual to whom it
is addressed and others who have been specifically authorized to receive
it. If you are not the intended recipient, you are hereby notified that any
dissemination, distribution, or copying of this communication is strictly
prohibited. If you have received this communication in error, or if any
problems occur with the transmission, please contact the sender.

Gotcha. I can include that in the gh action for future runs. Thanks!

In essence, since I'm using quarto to render hugo pages, the full command to build the website would be quarto render && hugo --gc --minify .... However, I wouldn't put it in an Action for now; at least until #102 is resolved.

@rcannood please rerender, thanks!


LineWidth value not respected when using anti-aliasing above 1

Minimal working example:
figure(1), clf;
z = linspace(0,4*pi,250);
x = 2*cos(z) + rand(1,250);
y = 2*sin(z) + rand(1,250);
scatter3(x,y,z,'Marker', 'x', 'LineWidth', 4);
export_fig -a4

The saved figure will display points at default line width rather than the specified value.

Cannot reproduce - it looks good on my system. Perhaps you're not using the latest version of export_fig.


smack.py: fix case issue which causes IOTOS-1407

Please explain in the commit message what the problem is and why fixing it is important.
Referencing an internal bug tracker is not enough. Instead, describe it so that everyone can understand and then add "Fixes: IOTOS-1407".

Commit message updated.


turn off filenames cutoff

What's the Problem?
in icon layout the filenames are cut off and only first few characters are shown
Solution/Idea
show full filename like in default explorer

Alternatives
currently it is like
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
Version: <IP_ADDRESS>
Windows Version
OS Version: 10.0.19043.1526
Comments
No response

Merging with #1033


Divide class that holds settings for customizing default behavior

I will suggest adding a new class(Settings) that holds settings for customizing default behavior of MyBatis (separating properties to a new class).
The motivation of this changes is avoiding circular reference on configuration properties feature of spring boot (mybatis-spring-boot). See spring-projects/spring-boot#10702.
In the current implementation of mybatis-spring-boot, it uses the Configuration as nested configuration properties as follows:
@ConfigurationProperties(prefix = MybatisProperties.MYBATIS_PREFIX)
public class MybatisProperties {
  // ...
  @NestedConfigurationProperty
  private Configuration configuration;
  // ...
}

And the Configuration has many circular reference property in nested properties. This class structure is incompatible with configuration properties feature provided by spring boot.
If this changes will be merged, I'll replace the type of nested configuration property to the Settings from the Configuration as follows: (not change property name for keeping compatibility with the old version as possible)
@ConfigurationProperties(prefix = MybatisProperties.MYBATIS_PREFIX)
public class MybatisProperties {
  // ...
  @NestedConfigurationProperty
  private Settings configuration;
  // ...
}

and I will use Configuration.from(Settings)(added factory method) for converting the Settings to Configuration as follows:
// ...
public class MybatisAutoConfiguration {
  // ...
  @Bean
  @ConditionalOnMissingBean
  public SqlSessionFactory sqlSessionFactory(DataSource dataSource) throws Exception {
    // ...
    Settings settings = this.properties.getConfiguration();
    Configuration configuration = null;
    if (settings != null ) {
      configuration = Configuration.from(settings);
    }
    if (configuration == null && !StringUtils.hasText(this.properties.getConfigLocation())) {
      configuration = new Configuration();
    }
    // ...
  }
  // ...
}

@harawata @emacarron @eddumelendez
What do you think about this changes?
Please review.
Thanks!

I've withdrawn this PR because fixed on Spring Boot 2.0.0.M6.


Rancher service links not working

Rancher Version:
1.1.0-dev5
Docker Version:
1.10.3
OS and where are the hosts located? (cloud, bare metal, etc):
Ubuntu 15.04 running on AWS cloud
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single node Rancher, internal DB
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Create two machines, one docker container on each machine (A and B). Link A to B using Service links.
Results:
When hitting B from A the link resolve to B's IP instead of B hosts's IP and that prevents us from reaching the container.
Expected:
The link should be accessible

This is not an issue, we were missing ports 4500, 2376 and 500


Re-design dedup to not reuse upsert mechanism

Here are some of the main differences between dedup and upsert:

Dedup is done when ingesting data from the stream (apply to consuming segment only), and no need to track valid docs. The duplicate records are simply dropped
Dedup window (TTL of the metadata) is a must have to reduce the metadata size
There is no need to track the record location in the dedup metadata. We do want to track timestamp for the dedup window

One potential solution for the dedup window is to keep 2 rotating maps, each storing metadata for one dedup window, and once the old map is completely out of the dedup window, clear it and use it as the new map.

I am currently investigating this ticket and working on an initial design document -- will be updating the ticket pretty soon.


Fixed #2307

This fixes #2307 by adding a "devices" setting in docker platform config. I also updated the documentation accordingly.
I tested my changes on a custom role and I successfully made a fuse mount with the following settings in my docker platform:
  - name: debian
    image: debian:latest
    #privileged: True
    devices:
      - /dev/fuse:/dev/fuse:rwm
    capabilities:
      - SYS_ADMIN
    security_opts:
      - apparmor=unconfined

PR Type

Feature Pull Request

@sprat Please fix the DCO as I cannot bypass it.

No problem, I'll fix that when I come back to home.

The merge request should be ok. One job failed due to problem with apt install which has nothing to do with my merge request.


Right click for dropdowns no longer works on beta

Expected Behavior
Right clicking on some collections, projects and http requests would open up the dropdown for that component.
Actual Behavior
You can no longer right click to open a dropdown for the component.
Reproduction Steps

Go to a collection
Right click a http request in sidebar
Dropdown does not open

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
This is likely bugged due to #5615 reworking of the dropdowns.
Insomnia Version
2023.1.0-beta.0
What operating system are you using?
Windows
Operating System Version
Windows 11
Installation method
Download from main website
Last Known Working Insomnia version
No response

Hi @xXBuilderBXx thank you for reporting this! Seems this particular functionality was removed in recent dropdown refactors we did. We'll update you here if there are fixes/changes on this. Feel free to create other issues of things you find 🙇

Hi @xXBuilderBXx thank you for reporting this! Seems this particular functionality was removed in recent dropdown refactors we did. We'll update you here if there are fixes/changes on this. Feel free to create other issues of things you find 🙇

Why would you do this? This is adding one more extra click to open the drop down menu. This is bad, and you should feel bad about it. Please, return this functionality back.
Also, relevant xkcd: https://xkcd.com/1172/

Still missing this very useful feature :(

@filfreire Any update on this? Will we get the functionality back?

Same issue here;
Version: Insomnia 2023.1.0
Build date: 3/9/2023
OS: Linux x64 5.19.0-35-generic
Electron: 22.0.0
Node: 16.17.1
V8: <IP_ADDRESS>-electron.0
Architecture: x64

$ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=22.10
DISTRIB_CODENAME=kinetic
DISTRIB_DESCRIPTION="Ubuntu 22.10"

This is a problem bugging me as well.
I looked at it for a short time and it seems like the commit e38ece1ab7a45671778fef9326fe4c18c4554154 where the dropdown was completely refactored (re-)introduced
the problem. The value requestActionsDropdown.current in sidebar-request-row.tsx is always null. I am not a react expert and do not fully understand how the DropdownHandle refs are supposed to work, but maybe someone is able to dive deeper into this?

Hi @filfreire, do we have any updates on this issue?

Hi @filfreire, do we have any updates on this issue?

It is over a month, and no response.

I hope we can bring this feature back. :(


request: Twitter Color Emoji

You have EmojiOne sheet,
any chance of a  Twitter Color Emoji one too?
I found this rep Twitter Color Emoji SVGinOT Font that may be used, there are raw images under assets/twemoji-svg.
It should be newer and better than EmojiOne, using Twitter's Unicode-10 emoji with diversity and country flags.

The twitter emojis are already included here in their PNG format.


Fix markdown formatting (add a space after #s)

A markdown header must have a space between the #s and the text.

thanks @yuchi


Different / weird key mappings in terminal in Chrome

OS/Web Information

Web Browser: Chrome 93.0.4577.63 64 bits / Firefox 91.0.2 64 bits
Local OS: Ubuntu 20.04
Remote OS: Ubuntu 20.04
Remote Architecture: amd4
code-server --version: 3.11.1 c680aae973d83583e4a73dc0c422f44021f0140e

Steps to Reproduce

With a french keyboard
Open a terminal under Chrome
Type some non-letter keys (example : @&é"'...)

Expected
The letters @&é"' shoud appear in terminal
Actual

with firefox : the letters @&é"' appear in terminal (good)
with chrome : the letters '123456 appear in terminal (bad)

Screenshot

Notes
This issue is only present in terminal, for editors all keys are OK for all browsers.
Changing vscode keyboard layout from us to fr have absolutely no effect.
Trying to remap keys in keyboardLayout.json have no effect.
Code-server runs from these 2 Dockerfiles :
https://dev.solidev.net/school/develspace/develspace-services/-/blob/develop/base/Dockerfile
https://dev.solidev.net/school/develspace/develspace-services/-/blob/develop/vscode/Dockerfile
This issue can be reproduced in VS Code: No

Hmm 🤔 I wonder if this is an upstream issue. Do you mind seeing if this can be reproduced in Xterm.js?
In the meantime, I'll see if I can reproduce locally. Just gotta find a French keyboard...

I'm using a French keyboard input on my macOS + Brave and I'm not sure if I'm reproducing or not 🤔 Very hard to tell since I'm not sure which keys I should be hitting. Let me know exactly what keys I should hit on a US QWERTY keyboard and I can try to reproduce.
https://user-images.githubusercontent.com/3806031/131890979-9e921850-f398-43e2-bd7a-3b2d5377d9e3.mov
Other thought, could this be a browser issue?

I have the exact same problem (and I'm french too).
In fact, the numbers act like an US QWERTY keyboard, so when hitting the first number key, we get a 1 and should get a &. Combined with MAJ we get a ! and should get a 1.
If this can help, I'm on Ubuntu and my keyboard is french but my language is English :

And Opera browser does not have the problem, even being a chrome-like browser.

@jsjoeio nice catch, the behaviour is exactly the same for xterm.js, in https://xtermjs.org/ for example (and also in jupyterlab).
@bcouetil exact, this seems to be a Chrome-only problem, i just tried https://w3c.github.io/uievents/tools/key-event-viewer.html on the following browsers

Chromium Version 93.0.4577.63 (Build officiel) snap (64 bits)
Edge Version 95.0.997.1 (Version officielle) dev (64 bits)
Firefox 91.0.2 (64 bits)

while in Chrome, keypress and keydown events are mismatched :

So this is clearly an upstream issue, feel free to close this.

Closing in favor of https://bugs.chromium.org/p/chromium/issues/detail?id=1245196

Wow, that's cool to see you figured out it's upstream in Chromium! Thanks for double-checking and linking upstream!


Add a TTD live recorder API sample.

This new sample serves to illustrate how to use TTD's live recorder API in order to control precisely what is recorded in the trace file.

Note that, as of the start of this PR, some pieces needed to run it are still in flight.

well .. it's ready when we are ready to publish. github added this "ship it" when I clicked approve. :)

In reply to:<PHONE_NUMBER>


Error on New task creation

My actions before raising this issue

[X] Read/searched the docs
[X] Searched past issues

I am trying to annotate a video on cvat of size 157mb. It was working for the past 1 month and I was annotating videos of size <500mb. But suddenly a week ago I got this error.

I tried after 2 days and today also and I was always getting the same error.
I even tried running cvat locally using docker but its giving me the same error.
Please help.
Expected Behaviour

The task is successfully created.
Current Behaviour

Error while creating the task.
Possible Solution

Steps to Reproduce (for bugs)

Context

I am currently doing a project for school which requires annotating videos.
There aren't any free tools other than cvat which could help me achieve this task. So I started working with cvat for creating bounding boxes for YOLOv5 algorithm.
Your Environment

Operating System and version (e.g. Linux, Windows, MacOS): Windows
I am using the cvat website for creating the task

Next steps
You may join our Gitter channel for community support.

Hi, I'm afraid it will be difficult to find these logs on cvat.org, so could you please collect and attach the server logs from your local instance after a failed task creation attempt? Command is: docker logs cvat> cvat.log
Thanks

@azhavoro I tried uploading a 157 mb video today locally and the task was created successfully. But then I tried to create a another task with a video of size 222 mb, the task creation failed this time. https://drive.google.com/file/d/1FvsY4HAnki3zVhMc0ttDTzTv3JUAYY-v/view?usp=sharing Please have a look at the logs in this link. Thank you so much! 🙏

@lokeshn011101
You probably have copied labels from created task with Ctrl+C and pasted them with Ctrl+V to the new task, right?
So, it is a wrong way because Raw data contains database IDs and looks like you try to create labels with primary keys which already exist.
Solution: To use button "Copy" in the top right corner of labels form (instead of Ctrl+C).

Damn. That was a silly thing that I missed. Thanks for taking your time and pointing it out! The issue is now resolved!

Let's consider it like appeal to actions. You are not the first person, who was confused with this issue.
I will reopen it and mark as a enhancement, to find a better solution.

Let's treat the issue as a UX bug. No need to have a separate button if the usual way to copy is CTRL+C.


Should allow setting color of TextStoryItem

Version: 0.0.5
There is no action to set color of TextStoryItem at its options dialog. This results in text showing in default-white color which in non-edit mode isn't visible when placed inside (say as child) a speech bubble (which is usually of white fill color)
Add ColorPicker dialog and respective button at popup of TextStoryItem

can currently only set by coping to clipboard, pasting to a text editor and then copy-pasting back
e.g.
object TextStoryItem: TTextStoryItem
Size.Width = 600
Size.Height = 300
Text = 'γεια σου'
TextColor = claGoldenrod
end
more color constants at
https://docwiki.embarcadero.com/RADStudio/Sydney/en/Colors_in_FireMonkey
(should also be able to add custom color values in ARGB but have to see in what format Delphi serializes them)


TestErrTypeRedefined

Codecov Report

Merging #98 (23b2935) into main (f05da50) will increase coverage by 0.00%.
The diff coverage is 70.00%.

@@           Coverage Diff           @@
##             main      #98   +/-   ##
=======================================
  Coverage   93.75%   93.76%
=======================================
  Files          14       14
  Lines        5557     5564    +7
=======================================
+ Hits         5210     5217    +7
  Misses        268      268
  Partials       79       79

Impacted Files
Coverage Δ

type_var_and_const.go
98.32% <70.00%> (+0.03%)
:arrow_up:

Continue to review full report at Codecov.

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


Issue with Angular 11

Hi,
Adding nats.ws to a Angular 11 project is throwing me the following error
Error: ./node_modules/nats.ws/nats.js 131:12
Module parse failed: Unexpected token (131:12)
File was processed with these loaders:

<EMAIL_ADDRESS><EMAIL_ADDRESS>You may need an additional loader to handle the result of these loaders.
| }));
| class Messages {

messages;

|     constructor(){
|         this.messages = new Map();
Can you help me?
Regards

Try https://github.com/nats-io/nats.ws/blob/master/developer_notes.md

Verified that angular 13 works fine.


Navigation is not functional on HoloLens 2 in the Hand Interaction Gesture Events example scene

Describe the bug
The Navigation feature in the Hand Interaction Gesture Events example scene does not follow the touch point in the blue cube when it is activated on HoloLens 2. It does not work with near or far interaction. The Manipulation feature only works with far interaction. See the attached video of the issue for more clarity. This issue does not happen with the Unity in-editor player.
To reproduce
Steps to reproduce the behavior:

Deploy the Hand Interaction Gesture Events Example scene to HoloLens 2.
Use far interaction with the blue cube to activate the Manipulation and Navigation detection spheres.
Observe issue.

Expected behavior
Near and Far interaction are detected with the Manipulation and Navigation detection spheres on the HoloLens 2
Screenshots

Your setup (please complete the following information)

Unity Version 2919.4.10f1
MRTK Version UPM 2.5.0-preview.20200910.3
Visual Studio 2019

Release
ARM
ARM64
Remote Machine

Target platform (please complete the following information)

HoloLens 2

@elbuhofantasma do you know if this also happens with 2.4?

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


Support for multi-line tables in pyproject.toml

I do have a few per-file ignores in my project[1]. Do you think it is possible to add support for multi-line tables?
This works:
[tool.ruff]
line-length = 89
per-file-ignores = {"tests/federation/printer/*" = ["E501"], "tests/test_printer/test_basic.py" = ["E501"], "tests/pyright/test_federation.py" = ["E501"], "tests/test_printer/test_schema_directives.py" = ["E501"]}

This does not:
[tool.ruff]
line-length = 89
per-file-ignores = {
  "tests/federation/printer/*" = ["E501"],
  "tests/test_printer/test_basic.py" = ["E501"],
  "tests/pyright/test_federation.py" = ["E501"],
  "tests/test_printer/test_schema_directives.py" = ["E501"]
}

Error:
error expected a table key, found a newline at line 196 column 21

I get the same error in VS Code:

My understanding is that multi-line tables are supported by TOML: https://github.com/toml-lang/toml/pull/904
but it's a relatively new feature :)
Anyway, I'm fine with having the ignores on one line, it's not a big deal, I opened the issue just in case it is easy to implement and for others that might stumble on the same problem :)
[1] https://github.com/strawberry-graphql/strawberry/pull/2367/files#diff-50c86b7ed8ac2cf95bd48334961bf0530cdc77b5a56f852c5c61b89d735fd711R196

I'm not an expert in TOML and how tables work there, but you can use the following syntax for this as of now:
[tool.ruff.per-file-ignores]
"tests/federation/printer/*" = ["E501"]
"tests/test_printer/test_basic.py" = ["E501"]
"tests/pyright/test_federation.py" = ["E501"]
"tests/test_printer/test_schema_directives.py" = ["E501"]

It doesn't look to me like the PR your linked has been published under a new TOML version, so for now the spec doesn't allow these kinds of tables.

I'm not an expert in TOML and how tables work, but I ran into this problem in a project too. You can use the following syntax for this at the moment:
[tool.ruff.per-file-ignores]
"tests/federation/printer/*" = ["E501"]
"tests/test_printer/test_basic.py" = ["E501"]
"tests/pyright/test_federation.py" = ["E501"]
"tests/test_printer/test_schema_directives.py" = ["E501"]

ah! great suggestion!

It doesn't look to me like the PR your linked has been published under a new TOML version, so for now the spec doesn't allow these kinds of tables.

didn't notice that! thanks for checking 😊
I'll close the issue for now then!

It ain't pretty, but I like one section per tool. The following works, which is good enough for me.
[tool.ruff]
per-file-ignores = {"btrack/models.py" = [
    "UP006",
    "UP007",
], "btrack/config.py" = [
    "UP006",
    "UP007",
]}


Custom Entity Models YoungSoulluoS Fork

CurseForge link
none
CurseForge Mod Distribution
No response
Modrinth link
none
Source/other link
https://github.com/YoungSoulluoS/cemFork
Mod file size
1.11 MB
License
(any other license)
What it does
This is simply a fork of Custom Entity Models (but for Fabric) and I thought I'd put it here since the original has not been updated for a while
Why should it be in the modpack
It is more up-to-date than the original, including CEM support for new mobs and fixing other ones, as well as having a fully compatible (from my testing) fork of the now very popular Fresh Animations by FreshLX Optifine resourcepack (Fabric fork by YoungSoulluoS), which is obviously not as ideal as the original FA working (most mobs turn invisible with it), but is definitely ahead of the original CEM for Fabric
Why shouldn't it be in the modpack

It has nothing but the original CEM for Fabric Curseforge and Modrinth links at time of writing, so may be difficult to include in a modpack
Its a fork and comes with those issues

Additional details
Putting this here for now as it would provide better Optifine resources support, however naturally it comes with all the issues of a fork

Interesting. I guess I could keep this issue open for reference, but I am actually expecting ETF to implement a better support for this because he is also working on the feature.

Interesting. I guess I could keep this issue open for reference, but I am actually expecting ETF to implement a better support for this because he is also working on the feature.

So maybe close this issue?


Fix WatchedResource data race

fix #51685

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

@wulianglongrd: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
65ffbf8b4182fc703e2403e892ef97a122c55afe
link
true
/test lint

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

@wulianglongrd: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
65ffbf8b4182fc703e2403e892ef97a122c55afe
link
true
/test lint

gencheck_istio
65ffbf8b4182fc703e2403e892ef97a122c55afe
link
true
/test gencheck

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

/test all

I'm not sure if this fix makes sense. In theory there are data races in the core process (but I haven't really encountered them in production), for example:
read：
https://github.com/istio/istio/blob/27a44759d96d7e532a62321c8a2e2f33f6920e1f/pilot/pkg/xds/ads.go#L635
write：
https://github.com/istio/istio/blob/27a44759d96d7e532a62321c8a2e2f33f6920e1f/pkg/xds/server.go#L405

I think they cannot run concurrently, donot they run in a goroutine.

I think they cannot run concurrently, donot they run in a goroutine.

You are correct, my example above is incorrect.

Its only the debug interfaces that do the concurrent access


isb_indicator_top_content_layout can not set by java

@wudaotiantian you can do this like: seekBar.getIndicator().setTopContentView(yourTopView); ,more info you can read readMe.md


[#10615] Add new Main classes to SpringBootDetector

#10615

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

Thanks for the contribution.
One last thing before merge, let's squash the commits into a single.
Could you please squash the commits and force-push?

I'm done.
Thank you for contribution guiding


Define Spectroscopy Use Case Notebook

Afaisst:  Currently working with Thomas Lai on JWST spectrum use case.
Once that is done, how to generalize?

It would be nice to think about how this could eventually be turned into a Fornax demo notebook.  Fornax demo notebooks need to:

[ ]  touch all three archives, IRSA, MAST, HEASARC
[ ]  Involve big data
[ ]  runs to completion on the SMCE

It is ok if the notebook reproduces work already published, ie., a good notebook would be one that tries to reproduce some seminal work, or expand upon some previous work.  But, of course, original ideas are also good.
No one will judge the science case, so the ties to all three archives can be tenuous.  Real science doesn't have a three archive requirement.
Lastly, I think it is important to be something you are actually interested in so that the work is more fun and might lead to a publication, which is a win win for all.

Vandana suggested it would look good if there was Euclid & DESI and/or SPHEREX & SDSS data in the spectroscopy use case.

A list of possible spectra:
IRSA:

Keck spectra on COSMOS. All reduced and ready to use, about 10,000 spectra
Spitzer IRS spectra. We have the experts in house.
IRTF has a large library of stellar spectra
SPHEREx: Spectra/cubes hosted at IRSA, but that takes a couple of years.
Euclid: Spectra hosted at IRSA in the near future?

MAST:

HST spectra (slit and slitless): Need our own reduction for good quality. Slitless grism spectra need to be extracted by us.
JWST spectra (slit and cubes): likely reduction. For JWST MIRI, we can build on Thomas' work.

SDSS:

A lot of optical spectra. Reduced and ready to use.

HEASARC

They don't really have "spectra". But could as ancillary data, for example matching to Chandra

Long term, among some COSMOS folks we have an NSF and ADAP program to re-reduce all spectra that exist on COSMOS. We so far focussed on Keck and Subaru. This still takes a while and in the end we could store them at IRSA. This would give us the possibility to use optimally reduced and calibrated spectra on COSMOS.
What the notebook should do:
We could follow the same theme as for the lightcurve notebook: Create a tool to query spectra in the different archives for a give position on the sky. The difference is that spectra are sparse, so there is not much overlap. The biggest overlap is likely in the famous fields. For example, COSMOS is covered by Keck, Spitzer, IRTF, SPHEREx, Euclid, HST, JWST, and SDSS. Alternatively, we could think of SDSS+Spitzer as pair (see below).
Another difficulty is that some of the spectra are not (or barely/badly) reduced. The reduction of the retrieved raw spectra would need to be included in such a notebook.
The notebook would conclude with providing UV to IR spectra for a given source.
Possible science themes
Think of science that can benefit from combining spectra at different wavelengths.

finding and studying extreme galaxies in the low-redshift. For example metal-poor galaxies that could serve as analogs of high-redshift galaxies (green peas, etc). The search for these could involve Machine Learning as well. We could combine UV, optical, and IR spectra of the brightest such galaxies to study the connection between different properties (stars, dust, metals, gas). A science goal would be to study the dust/PAH emission of metal-poor galaxies (what do their dust-grains look like?) That could inform observations of high-z galaxies with JWST. There do exist many studies (for example the evolution of the ionized continuum escape fraction as a function of redshift using local high-z analogs: [Faisst (2016)])(https://ui.adsabs.harvard.edu/abs/2016ApJ...829...99F/abstract)
Connecting to the lightcurve notebook: We can gather the spectra of the variable AGN and characterize their black hole masses from broad emission lines. Study the feeding of massive and less massive black holes, or connect variability to metallicity of gas content of the host galaxies. There must be studies about that in the literature, which would allow us to compare our results.

Discussion from Nov 1, 2023
We discussed the above topcis. A summary is below

We don't want to re-reduce the spectra on MAST. That is too time consuming. However, we can use them as "quick-look". Check what sources have JWST spectroscopy.
It is fine to design the notebook around available large spectroscopy surveys (such as SDSS, DESI, Spitzer, etc). We first have to understand how to ingest these data before we can combine them with Euclid and SPHEREx and other NASA missions.
Better to focus on different science case other than AGN to make use cases attractive for different science fields.
Another component is to put spectra in Parquet file format. A good starting point is to convert the Spitzer/IRS spectra to Parquet and store them on the cloud.

Created a new directory on GitHub and added a notebook template that we can use: https://github.com/fornax-navo/fornax-demo-notebooks/tree/spectroscopy_faisst_Dec723

Updates:

From Brigitta: "Re the IRSA spectro datasets from Andreas’ table, Angela has confirmed that the ones available from Gator are available through TAP, too, so we can use astroquery or pyvo"
Completed prototype of HST spectrum query

Updates:

Herschel spectroscopy is now added to the notebook (by Jessica Krick). However, it takes very long, mainly because need to download ~800MB of data for a single spectrum. For now, we called this good enough. Needs to be resolved (with ESA?) for larger numbers of sources.
v1 of the spectroscopy notebook is complete (July 24, 2024)


Mac OS X 10.8.5 and reliance on "en.lproj"

I've found that on Mac OS X 10.8.5, Electron requires the "en.lproj" folder that comes with the default build of Electron. If you delete that folder, it fails to load with a Segmentation Fault: 11 error.
This doesn't occur on anything newer than 10.8.5. I have a hunch, although untested, that it also causes Electron to not load on 10.7.5.
To recreate:

download the latest release from https://github.com/atom/electron/releases
find the Electron.app file, and right-click and choose "Show Package Contents"
navigate to Contents/Resources
either delete or rename en.lproj

If you try starting, you should see that it errors. As a side point, you can delete any other .lproj folder.
Thanks,
Scott.

The *.lproj files are required by Electron and Chromium for certain things to work, and we don't guarantee Electron would work if you remove them.

Okay, that's fair enough. Seems like it's an omission from the documenetation at http://electron.atom.io/docs/v0.32.0/tutorial/application-distribution/ perhaps?


Implementing dp_timelapse_process into cytosnake

This pull requests introduced a new workflow into cytosnake known as dp_timelapse_process,where it processing morphological features extracted by Deep Profiler from time lapse images.
When adding this feature, also some changes had to occur in order to maintain the current usage experience of cytosnake
Module Updates!
New modules added!
setup_init.py → Handles different variations on how to setup files for workflows.
config_utils.py → handles interactions like (loading, writing) configs with cytosnake
feature_utils.py → Normally functions that focuses on dealing features is normally done in pycytominer, however,  functions that are not implemented pycytominer will be placed in this module.
Module changes
errors.py → new errors added WorkflowNotFoundError and BaseFileNotFound
Updated configs!
New configs added!
dp_aggregator_config.yaml → responsible for aggregating Deep Profiler features
dp_data_configs.yaml → responsible for creating the DeepProfiler data type in order to use Deep Profiler specific functions and methods.
changes in configs
configuration.yaml → includes the newly added configs
Scripts
New scripts added!

dp_aggregate.py →aggregates deep profiler features
dp_normalize.py →normalize deep profiler features
build_dp_consnsus.py →builds a consensus from deep profiler features

Environment Files
New env file added!

dp_process.yaml → used for processing deep profiler features

Additional Notes
Issue: Repetitive code in the newly added scripts
New scripts contain repetitive functions calls to load config files.
Since cytosnake is not pip install-able (yet). The modules cannot be imported into script workflows since we are declaring different environments in snakemake.
Therefore, the solution to this repetitive code is to make cytosnake pip install-able and add it to our workflow environment files.

@gwaybio I have attended all your comments!

also, @axiomcura, can you please update the title of this PR to correspond to the updated workflow name. (it's no longer timelapse specific, i believe)


Support checkbox/radio button

The handleDomFormChange method doesn't currently account for the checkbox or radio input types:
https://github.com/Astrocoders/reform/blob/4553bd8b21e3d82ba772e6def066dc98578a36f5/re/ReForm_Helpers.re#L1-L4
I tried fixing this by adding a switch:
let handleDomFormChange = (handleChange, event) => {
  let domEl = ReactDOMRe.domElementToObj(ReactEventRe.Form.target(event));

  switch (domEl##type_) {
  | "checkbox" => handleChange(domEl##checked);
  | "radio" => handleChange(domEl##checked);
  | _ => handleChange(domEl##value);
  };
};

But this doesn't work as Reform currently only supports strings, so it's not as easy as I first thought. Any thoughts on how to fix this?

I think we'll include a helper to handle better boolean values while implementing this https://github.com/Astrocoders/reform/issues/23

Awesome 🙂 Let me know if I can help in any way

Closing in favor of #23


Be explicit about python3

Some Linux distributions will use python3 not python for Python 3 scripts. We should use that instead of assuming that python is now Python 3.

Fair enough. I was just fixing a problem I encountered working outside a virtual environment. In the virtual environment, python3 is also a valid way to invoke the interpreter.


particle minecraft: dust size argument accepts large numbers greater than 1

Also, the size of the particles increases up to a value of about 4
Anything greater than that is treated as its maximum.

:tada: This issue has been resolved in version 1.11.2 :tada:
The release is available on:

npm package (@latest dist-tag)
Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


Timing condition in Container.logs() - may raise DockerException rather than NoSuchContainer if container exits

This is because the utils.stream_stdout_and_stderr() function doesn't handle the different error outputs like utils.run() does. Could we factor this logic out and use it from the stream_stdout_and_stderr() function too?

That seems like a good improvement!


Use VS Code Notebooks as source of truth and INotebook as a wrapper

For #12189

INotebookModel is now a wrapper around VS Code Notebooks.
This way, we don't need to keep INotebookModel in sync with VS Code Notebooks.

Codecov Report

Merging #13154 into master will decrease coverage by 59.60%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master   #13154       +/-   ##
===========================================
- Coverage   59.60%    0.00%   -59.61%
===========================================
  Files         672       14      -658
  Lines       36803       38    -36765
  Branches     5170        3     -5167
===========================================
- Hits        21938        0    -21938
+ Misses      13764       38    -13726
+ Partials     1101        0     -1101

Impacted Files
Coverage Δ

src/client/debugger/constants.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

src/client/common/process/constants.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

src/client/common/platform/constants.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

src/client/datascience/notebook/constants.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

src/client/interpreter/autoSelection/constants.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

...t/datascience/jupyter/jupyterDataRateLimitError.ts
0.00% <0.00%> (-50.00%)
:arrow_down:

src/client/datascience/baseJupyterSession.ts

...c/datascience-ui/react-common/settingsReactSide.ts

...ient/datascience/jupyter/kernels/kernelSelector.ts

src/client/datascience/kernel-launcher/types.ts

... and 634 more

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 e33af82...689b7fc. Read the comment docs.


feat(tutorial): complete step 1

Closes #
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

@alisonjoseph Done, can you please have another look?

@alisonjoseph Any updates on this one?

@kellychurchill Do you please mind having a look at my last PR too? I just need this one in to apply for the badge

@vladbalanescu Netlify didn't trigger a re-build of this PR's deploy preview, but your change to Sass import looks good!


Cargo does not provide a .wasm file in the correct location when building for wasm32-unknown-emscripten

When building a project with the wasm32-unknown-emscripten target:
cargo build --target=wasm32-unknown-emscripten
a .js file is output in the target/wasm32-unknown-emscripten/debug folder, but not a corresponding .wasm file.
Instead, the .wasm file appears in the target/wasm32-unknown-emscripten/debug/deps folder, named with the metadata suffix.
The expected behavior would be to output both the .js and .wasm in the debug folder. The .js loader file is also trying to load .wasm file from its own folder with the metadata suffix.
If you run rustc --target=wasm32-unknown-emscripten --print=file-names main.rs, you'll see that rustc only prints main.js as an output file, even though emscripten will produce both main.js and main.wasm.
cargo 0.23.0-nightly (8118b02ac 2017-09-14)
rustc 1.22.0-nightly (26015da01 2017-09-23)
x86_64-pc-windows-msvc
emcc 1.37.21

I'm having the same problem. I'd be interested in helping to fix it - but would need to be pointed to the right area of cargo src.

I think this is the same case as https://github.com/rust-lang/cargo/issues/4500#issuecomment-330237356, neither Cargo nor rustc unfortunately knows about this output file.

How would you go about telling rustc/cargo about this file? Would it be better to do it in rustc, or cargo? In cargo, I've found cargo::opt::cargo_rustc::Context::calc_target_filenames, which seems to be one place it could be added.

@derekdreery indeed! So up to this point we've avoided encoding anything target-specific in Cargo, it relies solely on rustc for this sort of information. That being said I think it's time to break that rule! If Cargo were to handle this then I believe yeah, that location is where the fix would go.

Yeah I was worried that I wasn't allowed to do anything target-related, but yes this file doesn't really have anything to do with rustc.

I think ideally rustc would inform Cargo about this as it tends to have more target-specific information, but for the time being I think we can go ahead and implement it in Cargo (in a conservative fashion) and we can move it around later if need be

OK cool.
The situation will change anyway if emscripten comes out of the build chain, and llvm is compiled directly to wasm.

@Herschel @derekdreery I opened a PR #4570 to address the issue. If you have time, can you please check if it works for you? I do not have environment with emscripten installed.
$ git clone -b target-specific-artifacts https://github.com/tatsuya6502/cargo.git
$ cd cargo
$ cargo build --release

cargo binary will be found in target/release directory.
Thanks!

Cool! Could you submit it as a PRINTING?

Cool! Could you submit it as a PR?

Yes, I submitted it as a PR. But before it gets merged, I think one of us needs to verify that it actually solves the problem. The PR contains fixes for wasm and Windows DLL (#4500), and I verified the latter. I cannot verify the earlier because I do not have emscripten now.

Thanks for your work, @tatsuya6502!
The .wasm file now appears in the correct folder. However, the .js file is still trying to load the .wasm file with the incorrect name including the metadata suffix. (for example, web_test-4d1ad28c85a2ee45.wasm instead of web-test.wasm).
Shee this gist showing the incorrect line in the .js:
https://gist.github.com/Herschel/df22af2b04d69c0ecb6c447a54765d54#file-web-test-js-L1625
Ideally cargo would generate the correct filename in the .js file. You can either:
a) Replace (module_name)-(suffix).wasm with (module-name).wasm in the .js file.
b) Append var Module = {wasmBinaryFile: "web-test.wasm"}; to the top of the js file.
c) Maybe there is some way to get emscripten to generate the js with the correct filename?

@Herschel Thank you for testing it!
Hmm, I think another option could be:
d) Do not ask rustc to add the metadata (-C metadata) for wasm bin target, like this for cdylib.
@alexcrichton Do you have any preference/suggestions?

@tatsuya6502 sorry I'm not sure what the question is about, I'll try to read the PR in more detail tomorrow

I took option d) and added another commit https://github.com/rust-lang/cargo/pull/4570/commits/f9a541b5f93e66bc5e02c67ecd585d87fb1185cb to PR #4570. I installed emsdk (incoming) to Fedora and tested wasm32-unknown-emscripten target.
I will test x86_64-pc-windows-msvc target after work. (It is almost 10am now in my timezone.)

I tested x86_64-pc-windows-msvc, works great now! Thank you!

@alexcrichton Sorry for asking an unclear question. My original question was "is it OK to remove metadata suffix from wasm target files (js, wasm) in deps directory?" The wasm file name was embedded in js file, so we should avoid to copy wasm file something like "debug/deps/wasm_test-0f5ba9409d843569.wasm" to "debug/wasm_test.wasm".
I added a summary of changes on the PR. Hope this helps.


Binary Sensors missing

Two binary_sensors that were working, are now showing as unavailable.
The sensors are:

binary_sensor.ev_car_is_online
binary_sensor.ev_car_is_active

Version of the custom_component

0.0.12.
Installation method (hacs / manual)
HACS
Installation method of hass (venv, docker, hassio,...)
Configuration

Add your configuration here.

Describe the bug
They were working until about 18:00 last night.   When they stopped working I upgraded to 0.0.12.
Looks like the latest code base removes them?
Debug log

Add your logs here.

These sensors made the integration crash because they are no longer available in the api.
So i've removed them.

That’s disappointing,
Do you know if this is a temporary or permanent change to the API?

Superseded, can now be closed.


[DOCS]: Typing and Documentation Issues with onShippingAddressChange

Is there an existing issue for this?

[X] I have searched the existing issues

Documentation Link
No response
Describe the problem
Hello,
The onShippingAddressChange prop of the PayPalButtons component does not seem to be correctly typed, is non-compliant with the paypal-js SDK, and is not documented.
For example:
      <PayPalButtons
        onShippingAddressChange={(address, actions) => {
          // @ts-ignore
          return actions.reject(address.ERRORS.ADDRESS_ERROR);
        }}
      />

The paypal-js SDK appears to expose two methods, actions.reject and actions.resolve, but here only reject is available.
By default, the displayed message is something like "unable to update your address." I eventually managed to display a different message by using an untyped errors property available on the address object and passing it to the reject function, which according to its type, does not accept any parameters.
Additionally, the messages provided in the ERRORS property are not translated according to the current language of the client, which is the case with paypal-js.

Describe the improvement
No response
Anything else?
No response

bump


how add lazyLoad, beforeInteractive and etc.. when use in Nextjs

Hello,
We need gtm scripts to be added after page load. In nextjs, we use script tags along with strategy to handle this case. which includes things like lazy load, afterInteractive,... But we need it to be dynamic for some reason. In this case, we do not have access to the mentioned strategies. How can we manage these things using this package?

Nextjs Doc Address: https://nextjs.org/docs/basic-features/script

check this out https://github.com/vercel/next.js/tree/canary/examples/with-google-tag-manager


Other url-pattern on Swagger-ui to separate documentation (With Jersey)

Hi everyone!
I wanted to know if there is a way to have an other url pattern to access Swagger doc on Swagger ui. Im working with Jersey with Eclipse IDE. For exemple, im using LocalHost:8080/api/restapi/swagger.json right now to access my doc for my rest services but i want to make doc for an other package (so not restapi) and I want to be able to acess it with an other url like this : LocalHost:8080/api/newPath/swagger.json.
Can you tell me how to do it step by step? I searched a lot but could not find what I'm looking for.
Thanks in advance!
Edit : all my swagger configurations are in Web.xml of my API so I don't have a class like "Application" or "AppConfig".

  <servlet-mapping>         <servlet-name>SpringMVC</servlet-name>         <url-pattern>*.do</url-pattern>     </servlet-mapping>
how to fix it?


v2/card

GET /card
GET /card/:uuid
POST /card
DELETE /card/:uuid

GET /card
GET /card/:uuid
POST /card - couldn't POST, but you could show me how
DELETE /card/:uuid

Not super validated and some funny HTTP codes, but 3/4 are working.


Update README.md

added contributors information to the readme file

Won't merge this as I cannot tell if you're a bot. Sorry.


Add default config.json to jar

Codecov Report

Merging #175 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master     #175      +/-   ##
============================================
- Coverage     28.71%   28.67%   -0.05%
  Complexity      322      322
============================================
  Files            43       43
  Lines          2828     2832       +4
  Branches        335      335
============================================
  Hits            812      812
- Misses         1933     1937       +4
  Partials         83       83

Impacted Files
Coverage Δ
Complexity Δ

.../redhat/red/build/finder/DistributionAnalyzer.java
69.41% <0%> (ø)
42 <0> (ø)
:arrow_down:

...rc/main/java/com/redhat/red/build/finder/Main.java
9.78% <0%> (-0.11%)
5 <0> (ø)

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 011ab50...79a02bc. Read the comment docs.


Logger not wrapped properly

Would be great if we could use something like Pino as the logger:
const logger = require("pino")({ level: "info" });
const { ServiceBroker } = require("moleculer");
const broker = new ServiceBroker({ logger });
broker.logger.info("hi");

TypeError: Cannot read property 'write' of undefined
at LOG (node_modules\pino\lib\tools.js:102:9)
at EventEmitter. (node_modules\moleculer\src\logger.js:63:7)

After all, it's really fast and supports custom log levels and child loggers.

Thanks. It's weird. I will able to check it on the next week. I'm on holiday right now.
Or you can send a fix PR if you found the problem.

I'm having troubles with bunyan too :(

As I'm relying on some additional functionality offered by pino I opted in for a workaround.
Instead of providing a logger to moleculer's constructor I override the broker's logger and use mixins to override it per-service:
const logger = bindings => pino.child(bindings); // spawns child logger with additional base bindings

const broker = new ServiceBroker({ ... });
broker.logger = logger({ svc: "broker" });
broker.logLevel = broker.logger.level;

const loggerMixin = {
  created() {
    this.logger = logger({ svc: this.name });
  },
};

const serviceN = {
  mixins: [loggerMixin],
  ...
};

@WoLfulus: I assume you can use the same approach.

My approach was a little bit different, and maybe uglier.
This is what I did:
    const logger = bunyan.createLogger({
        name: process.env.SERVICE_NAME || os.hostname(),
        level: process.env.LOG_LEVEL || "trace"
    });

    const log = (name, text) => {
        logger[name](text.toString("utf8"));
    };

    const broker = new ServiceBroker({
        logger: {
            trace: (text) => log("trace", text),
            debug: (text) => log("debug", text),
            info: (text) => log("info", text),
            warn: (text) => log("warn", text),
            error: (text) => log("error", text),
            fatal: (text) => log("fatal", text),
        },
    });

I'm considering this issue. I plan I'm going to extend the logger broker options. It can accept Function. In this case, the broker send every log entries to this function, so you can call your custom logger module.
Example:
const broker = new ServiceBroker({
    logger: function(entry) {
        myLogger[entry.level].apply(entry.args)
    }
});

The entry is an object and have the following properties:
{
    module: "broker", 	// Others: "service", "tx", "transit", ...etc
    service: "users", 	// service name of the `module` is "service"
    level: "info", 	// log level of `this.logger.info` method
    args: [		// Arguments of `this.logger.` method
        "Test message",
        {
            a: 5
        }
    ]
}

@deividasjackus @WoLfulus what do you think? It can solve the logger problems? Or any suggestion?

LGTM.
Has all the info I need and I'll be able to get rid of those prefixes and put it into a proper field on the logs (easy to filter in kibana for example).
Also, I don't know if it's bunyan related, but I had to encode the message to utf8, or else all the messages were being printed as H e l l o   w o r l d. on the console.

@icebob I like the proposal, but would like to propose that instead of passing a function that gets called w/ every logger call to the broker constructor, you pass a function that takes contextual logger w/ data provided by moleculer and returns a new logger instance. Example:
const broker = new ServiceBroker({
    logger: function(bindings) {
        return pino.child(bindings); // in the case of pino
        return bunyan.child(bindings); // in the case of bunyan
        return new WinstonContext(winston, '', bindings); // in the case of winston + winston-context
    },
});

In other words, moleculer calls the above user-supplied function only once per module/service during creation w/ { nodeID, module, service } as the arguments, uses the returned logger.
This also means that broker.logLevel gets deprecated in favor of broker.logger.level. FWIW, that being handled by any of the mentioned loggers instead of moleculer is a good thing 😉

@deividasjackus Thanks, I'm going to consider your proposal.
Btw, I fixed the original bug. So from now you can use your original code without workaround in v0.8.5
// With Pino
const logger = require("pino")({ level: "info" });
const broker = new ServiceBroker({ logger });

// Or with Bunyan
const bunyan = require("bunyan");
const logger = bunyan.createLogger({ name: "my-broker" });
const broker = new ServiceBroker({ logger });

broker.logger.info("hi");

I'm working on it in the next-logger branch.
Current state:
Pino
const pino = require("pino")({ level: "debug" });
const broker = createBroker({
    logger: bindings => pino.child(bindings)
});

Output:

Bunyan
const bunyan = require("bunyan");
const logger = bunyan.createLogger({ name: "moleculer", level: "debug" });
const broker = createBroker({
    logger: bindings => logger.child(bindings)
});

Output:

Default console logger
const broker = createBroker({
    logger: console,
    logLevel: "debug"
});

New console output:

The bindings contains

module - type of module: broker, service, cacher, transit, transporter
service - name of service if the module == "service"
nodeID - nodeID from broker options
namespace - namespace from broker options (new v0.9 feature)

What do you think @deividasjackus @WoLfulus ?

@icebob aside from the addition of namespace (great btw!) I can't see if there's anything different from what I proposed - so obviously I'm in love 😆. Default logger has been improved and now uses chalk as well, correct? Hope I didn't miss anything :)
So far I've continued to manually override the broker/service loggers, but have made slight changes to the bindings. Purely a matter of taste, but if there's ever a time to share mine I guess it's before v1.0:

component instead of module (still don't like either one)
ver to include service version (be aware that pino reserves v for storing the used log format version)
ns instead of namespace (purely out of will to save 7 bytes per long line)

On a slight off-note:
You might know this already, but when you're using pino (can't speak for bunyan or winston) you can use either its .pretty formatter, or the bundled CLI version of the same formatter (for piping raw logs):

I've personally found their CLI formatter very easy to customize.
Display the logs in any format you want.
Add log processing logic, i.e. somewhat of a hook (e.g., sending logs to centralized storage).

Hope someone finds anything of the mentioned above useful :)

Thank you for your suggestion. I'm changing the bindings props to a shorter name. But in case the component is not too long? :) Maybe it can be comp ?
The service version is a good idea. I missed to add it but it's important.

@icebob: I agree, module probably isn't the right name for it and component is too long.
Brainstorm off the top of my non-yet-caffeinated head: unit, scope, part, comp, mod?

I prefer the comp or mod

Of those two options I'm actually leaning towards mod 😆 Also picking it would mean that time doesn't have to be spent on updating documentation 👍

:laughing: OK, mod is better :)

I finished the implementation. I'm starting to update changelog.
Btw, there is an other new feature what I implemented. If you are using the built-in logger: console you can set a logFormatter and you can change the format of log messages.
E.g.:
const broker = new ServiceBroker({
    logger: console,
    logFormatter(level, args, bindings) {
        return level.toUpperCase() + " " + bindings.nodeID + ": " + args.join(" ");
    }
});
broker.logger.warn("Warn message");
broker.logger.error("Error message");

Output:
WARN norbi-pc2: Warn message
ERROR norbi-pc2: Error message

I think from now every logging logic in Moleculer is customizable :)

@icebob excellent!
Last thing: think it's relevant to list the bindings moleculer uses itself in the documentation so users know what's considered reserved (overrides are still possible: broker.logger.error({ mod: "peanut" }).

Good point! I added it to changelog first.


Support global namespace reference in JavaScript

VSCode Version: 1.8.0

I asked this question on SO but it is more like a feature request for Salsa so I copy it here:
Say I have two files:
// a.js
Abc.foo = function(x) { return x; };

// b.js
Abc.foo(1);

The feature request is to enable "Goto Definition [F12]" to jump from b.js Abc.foo() to a.js

Thanks. I think this is already being tracked by: https://github.com/Microsoft/TypeScript/issues/13271
If you instead write:
// a.js
var Abc = {
    foo: function(x) { return x; }
}

//b.js
Abc.foo(1);

I am able to use go to definition just fine. The root problem here seems to be that Salsa does not understand properties added to the namespace after it is created.
Please let me know if that does not sound correct.

I see. Also, I notice that:
// a.js
Abc = { ... }  // without var

does not work either.
Together with Microsoft/TypeScript#13271, that means The JavaScript namespace style doesn't work that well:
// Abc/def/xyz.js
Ext.ns('Abc.def');
Abc.def.xyz = { ... }

// Abc.def/pqr.js
Ext.ns('Abc.def');
Abc.def.pqr = { ... }


Recorder not working in latest versions of browser

I tried with Firefox and chrome. Record seems working when we see the counter. But the output doesn't have any sound. I have checked my implementation as well as the demo that you provided.
Chrome version : Version 66.0.3359.139 (Official Build) (64-bit)
Firefox Version :  60.0 (64-bit)
Please do help.

Remember to put the Microphone level to a default of > 0. Currently in the demo code it is set to 0 for some reason. Took me a while to find out as well.
Like:
microphoneLevel.gain.value = 0.8;
Something else I had to "fix" was to comment out this line:
mixer.connect(audioContext.destination);
Because it was making a feedback loop from microphone to speaker (creating a creepy sound effect).
Hope this helps you.

me too getting this issue but with
Could not get audio media device: TypeError: Failed to execute 'getUserMedia' on 'MediaDevices': Malformed constraint: Cannot use both optional/mandatory and specific or advanced constraints.

This looks like a fantastic amount of work.  What needs to be done to get this project up and running?

@derek-adair if you're talking about me then it's just some funding and I'll get to fix this and get it up and running in no time ;)

@meteorplus - nope.  I was asking with the intention of doing this myself.  Do you have a list of action items or bugs that need working on anywhere?  Should we fork this?

Upon further inspection its not clear if this project is working or not.  I assumed the worst when looking at the non-functioning demo.

@derek-adair I can get you a working demo ASAP and test it on all browser and even on mobile too

Was getting no sound as well from Chrome. I commented out and replaced this line thusly:
//~ var constraints = { audio: true, video:false }

    var constraints = { audio: {
        volume: 1.0,
    }, video:false }

Works perfectly now.

Nice after two years at least

@derek-adair if you're talking about me then it's just some funding and I'll get to fix this and get it up and running in no time ;)

?

@derek-adair if you're talking about me then it's just some funding and I'll get to fix this and get it up and running in no time ;)

?

??

@derek-adair if you're talking about me then it's just some funding and I'll get to fix this and get it up and running in no time ;)

?

??

???


Expose tempvars2 on the runtime & minor code changes.

Please read the comments I put on the git commit for why stuff was changed.
As per what cst1229 did, for letting others have access to the extension stuff then lets us access the previously accessible runtime variables from another extension.
And the rest is just some minor code changes.

I'll look into this more clearly later, but it's looking fine as of now.


MM-15161 show bot badge and icon for ephemeral posts from bots

Summary
This adds the profile image, username and bot badge to ephemeral posts for bots.
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/10640
Related Pull Requests

Has mobile changes https://github.com/mattermost/mattermost-mobile/pull/2740

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

@scottleedavis Do you have a version of the remind plugin that can be used to test this?

Spinmint test server created at: https://i-0e4939484f312b71d.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-0e4939484f312b71d

@scottleedavis Do you have a version of the remind plugin that can be used to test this?

@hanzei  sort of.  If you use the ' ephemeral-bot-posts' branch on the repo, and thing typed into /remind .... will return an ephemeral and a normal test posts.

Great, thanks Scott!

Thank you! It appears to be working for me

Is there any additional testing I should take into account?

Spinmint test server destroyed

@saturninoabril I rebased off of master, and then I noticed a lot appears now changed in this PR beyond the two small changes.
I am use to keeping a PR/MR up to date with master and that is what I thought I had done, and this has not happened before to me.
Please advise the best course of action.  I apologize.

@scottleedavis looks like you need to merge from master, and re-run the tests. Do you mind doing it so we can merge?

@svelle I think this can be merged anyway. It has no merge conflicts.

Done @levb

Thank you very much for this PR @scottleedavis!


[Bug] Import CSV Data Into Model works on file with 30 rows but not with 95.000 rows

Version of Archi, Operating System

Archi v4.4.0, on Windows 10 x64.
Expected Behaviour

Import CSV Data Into Model working on a .csv file with 95.000 rows or more.
Actual Behaviour

I'm trying to import into Archi an elements.csv file with approximately 95.000 rows.
However, I received the following error message importing:

Could not find concept: (Some ID that in this elements.csv file does not exist).

I already ran a test and reduced the same elements.csv file to only 30 rows and in this way Archi imported elements.csv file correctly.
That is, with this test I was able to verify if there was any error with respect to the data, and it was identified that there is no inconsistency in the data.
I took a quick look at the CSVImporter.java file and in the ArchimateModelUtils.getObjectByID() function, maybe left something pass, and as I mentioned earlier, the ID returned along with the CSVImporter_10 message does not exist in elements.csv file.
I could not recognize exactly why the code returned this error, but again, I took a quick look.
Steps to Reproduce the Behaviour

Create the elements file in MS Excel (with approximately 95,000 rows) containing an ArchimateModel type and the rest of the objects as Artifacts type.
Save the elements file in .csv format using the UTF-8 encoding.
In Archi, create a new Empty Model.
Without renaming the Empty Model or any other modification, import the elements.csv file through the CSV Data Into Model.

PS: By doing the same process, however, reducing the 95,000 rows to 30 rows, this same process works as expected.
I'm new using Archi, I'm not sure if this can be considered a bug.
But anyway, I would like to know if it is in fact a bug or something else.
Anyway, I apologize if I have opened this issue in the wrong way.
Best Regards.

This can happen if there is a wrong object id in the relations.csv file. The source and target IDs should match entries in the elements.csv file. Did you check this?

Using the CSV Data Into Model option I can not import the three files (elements.csv, properties.csv and relations.csv) together.
I need to import them one at a time. In this case, I do not know if there is a correct import order, but at this first moment I'm trying to import the elements.csv file first.
So I could not check what you commented because the reported problem occurs when I try to import only the elements.csv file, (and I can not import all three files at once).

As long as the three file names match ("elements.csv", "relations.csv", "properties.cvs") you only need to select one file for import. So if you import "elements.csv" file the others will be imported also.
Search for the incorrect ID in the "relations.csv" file, and you will find it in the Source or Target column.

Sorry for the delay in answering it.
I have not realized that if I import elements.csv file the others will be imported also.
So I was looking for an error in the elements.csv file instead of looking for something in the relations.csv file.
This worked for me. Thank you!
Just one more thing, I fixed the problem reported above, but now when importing I'm getting the CSVImporter_2 error:
Incorrect record size.
What exactly does that mean? I understand that there is a condition to check if the size of the elements record is correct.
But I can not understand why I'm getting this error.

Incorrect record size means that there are not enough, or too many, entries in a row. You have to check there are the exact right amount in each row.

OK. Thank you!
Once the main problem has been clarified and solved, I will be closing this issue.
Any other problems, I'll get in touch.
Best Regards.


Will there be config.w32?

What did you do? If possible, provide a simple script for reproducing the error.

I want to install openswoole extension using php-sdk.
but when I want to configure, it show "Must be run from the root of the extension source"
then I search it and found that it need config.w32, otherwise it won't work
so my question is. Will there be config.w32?

What did you expect to see?

create openswoole.dll

What did you see instead?

"Must be run from the root of the extension source" in php-sdk

What version of OpenSwoole are you using (show your php --ri openswoole)?

4.12.0

What is your machine environment used (show your uname -a & php -v & gcc -v) ?

Window
PHP 8.1.12 (ts)
gcc version 6.3.0

Unfortunately, openswoole doesn’t work on windows OS. You can find more about how to install at https://openswoole.com/docs/get-started/installation

awh ok , thanks


Error starting revad from sciencemesh-testing branch

I think we need to add some config entries to make these errors go away:
2023-06-13 08:01:05.341 ERR ../reva/pkg/notification/notificationhelper/notificationhelper.go:77 > connecting to nats failed, notifications will be disabled error="connection to nats server at '' failed: nats: no servers available for connection" pid=301883 pkg=rhttp service=ocs
2023-06-13 08:01:05.342 INF ../reva/pkg/rhttp/rhttp.go:215 > http service enabled: ocs@/ocs pid=301883 pkg=rhttp
2023-06-13 08:01:05.343 INF ../reva/pkg/rgrpc/rgrpc.go:147 > grpc server listening at tcp:<IP_ADDRESS>:19000 pid=301883 pkg=rgrpc
2023-06-13 08:01:05.385 ERR ../reva/pkg/notification/notificationhelper/notificationhelper.go:77 > connecting to nats failed, notifications will be disabled error="connection to nats server at '' failed: nats: no servers available for connection" pid=301883 pkg=rhttp service=ocdav
2023-06-13 08:01:05.386 INF ../reva/pkg/rhttp/rhttp.go:215 > http service enabled: ocdav@/ pid=301883 pkg=rhttp
2023-06-13 08:01:05.386 ERR ../reva/cmd/revad/runtime/runtime.go:211 > error starting the http server error="http service sciencemesh could not be started,: sciencemesh: provider_domain is missing from configuration" pid=301883

For the notifications part, it's "benign" and @javfg already fixed the message AFAIU.
The last message is important: you need to specify provider_domain in the [http.services.sciencemesh] section (it was already mandatory, now reva won't start) and you can actually drop the invite_link template

@michielbdejong yes! I already have a patch ready so the connection to NATS is only performed if there is at least something configured for the notifications. That way the error will be an info log instead.
The failure to start should not be related to those errors, tho.

And mesh_directory_url should be "https://sciencemesh.cesnet.cz/iop/meshdir"?


Skip tests generated by plugins (like pep8) for discovery command.

To do not fail on discovery command and do not track their results.
For #7287

[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[ ] Title summarizes what is changing
[ ] Has a news entry file (remember to thank yourself!)
[ ] Appropriate comments and documentation strings in the code
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated
[ ] Test plan is updated as appropriate
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed)
[ ] The wiki is updated with any design decisions/details.

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

Codecov Report

Merging #8242 into master will decrease coverage by 0.73%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #8242      +/-   ##
==========================================
- Coverage   59.01%   58.28%   -0.74%
==========================================
  Files         504      504
  Lines       23090    22723     -367
  Branches     3733     3728       -5
==========================================
- Hits        13627    13244     -383
- Misses       8583     8591       +8
- Partials      880      888       +8

Impacted Files
Coverage Δ

...nt/datascience/jupyter/invalidNotebookFileError.ts
0% <0%> (-75%)
:arrow_down:

.../client/datascience/jupyter/jupyterConnectError.ts
0% <0%> (-66.67%)
:arrow_down:

...science/jupyter/jupyterKernelPromiseFailedError.ts
0% <0%> (-50%)
:arrow_down:

...ent/datascience/jupyter/jupyterWaitForIdleError.ts
0% <0%> (-50%)
:arrow_down:

...cience/jupyter/jupyterDebuggerNotInstalledError.ts
50% <0%> (-30%)
:arrow_down:

src/client/common/utils/platform.ts
64.7% <0%> (-23.53%)
:arrow_down:

src/client/language/characters.ts
12% <0%> (-21.34%)
:arrow_down:

src/client/testing/codeLenses/main.ts
60% <0%> (-17.78%)
:arrow_down:

src/client/datascience/cellFactory.ts
72.34% <0%> (-17.32%)
:arrow_down:

src/client/common/platform/pathUtils.ts
66.66% <0%> (-5.56%)
:arrow_down:

... and 35 more

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 acd7a3d...8388ec1. Read the comment docs.

The changes look good, might need a rebase to master to get around the test failures.

FWIW, I am fairly sure that the old code (parsing the pytest CLI output) used to ignore plugin-generated nodes.  If it matters then we can investigate in #8425.


preconnect is not possible in sass where i am using import for google fonst

Welcome! Please use this template for reporting documentation issues on https://developers.google.com/web/.
For questions about Chrome DevTools, checkout the
google-chrome-devtools tag on StackOverflow.
For questions about Chrome, check the google-chrome
tag. If you think you've found a Chrome bug, check out our
feedback page for more details on how to file it.
Page Affected: https://developers.google.com/web/TODO
What needs to be done?
Please provide description of what's needs to be done here. Please be sure to include the link to the original page
and any supporting links/data.

[ ] Appropriate type-something label has been added.
[ ] Appropriate section-something label has been added.

Hi, Can you let us know the page of WebFundamentals you are referring to, if you are here to report a bug that is not related to WebFundamentals (developers.google.com/web) then please close this issue and see the feedback page  on how to file a good bug.
Thanks!


fully set up example of communication with Rock7 and quick plotting

(wait a bit for more bugs to be crushed before updating on this repo).


Series doesn`t work correctly with sparse at MODIN_STORAGE_FORMAT=omnisci

When I run at MODIN_STORAGE_FORMAT=omnisci:
import numpy as np
import pandas
import modin.pandas as pd
pd.Series(pandas.arrays.SparseArray(np.array([3, 1, 2, 3, 4, np.nan])))

I have error:
TypeError: Sparse pandas data (column F___reduced__) not supported.

@modin-project/modin-omnisci  could someone please check if this still happens?
@devin-petersohn do we still want to support sparse data?..


Problem with Game.GetUserInput

I have created a configuration menu for my Autodrive mod. Now at all value types I call Game.GetUserInput so the user can enter a new value. After pressing the return/enter key I expect the input line to close and the menu should reappear but instead the input line just pops up again. I guess I'm just doing something wrong. So here is a bit of code. ChangeCarMoversPerimeter is called when the user hits return on the appropriate menu item.
      private void ChangeCarMoversPerimeter()
      {
         UI.ShowSubtitle("Enter new perimeter value in meters.");
         int number;
         var success = int.TryParse(
            Game.GetUserInput(
               _currentConfig.CarMoversPerimeter
                  .ToString(CultureInfo.InvariantCulture), 30), out number);
         if(success)
            _currentConfig.CarMoversPerimeter = number;
         else
            UI.ShowSubtitle("That's not a valid number!");
         ReOpenMainMenuAt(_currentlySelectedIndex);
         DisplayAndSafeNewSettingOf("CarMoversPerimeter", _currentConfig.CarMoversPerimeter);
      }
//...
      private void ReOpenMainMenuAt(int index)
      {
         _currentlySelectedIndex = index;
         View.AddMenu(_mainMenu);
         _keyMenuIsOpen = false;
         _mainMenuIsOpen = true;
         _mainMenu.Items[0].Deselect();
         _mainMenu.SelectedIndex = index;
         _mainMenu.Items[index].Select();
         OnMainMenuSelectedIndexChanged(new SelectedIndexChangedArgs(index));
      }

Got it. It turned out it was the key bindings for the menu handling.
...
//ActivateKey = Keys.Return;
//BackKey = Keys.Escape;
//DownKey = Keys.Down;
//LeftKey = Keys.Left;
//RightKey = Keys.Right;
//UpKey = Keys.Up;
...
All is good after commenting them out. Looks like I need to unsubscribe from the key event or the return key on GetUserInput is also used to activate the menu item again, which causes this kind of loop.

I think we shouldn't process menu keys if GetUserInput is activated.

I was too happy too soon. Sometimes it works but most of the times not.
@IncoCode You mean I should unsubscribe from all menu related events before I call GetUserInput?

@RainerHilmer i mean add check here: https://github.com/crosire/scripthookvdotnet/blob/dev/source/Script.cpp#L161
if (UserInputActivated) return; // i forgot the proper name of this native :D

or something.

Feature-Duplicate of #252.
Also, do you increment _currentlySelectedIndex anywhere?

Increment? No, _currentlySelectedIndex gets set in the SelectedIndexChanged handler.
private void OnMainMenuSelectedIndexChanged(SelectedIndexChangedArgs e)
{
_currentlySelectedIndex = e.SelectedIndex;
var propertyValue = GetPropertyValueForSelectedMainMenuIndex(e.SelectedIndex);
if(string.IsNullOrEmpty(propertyValue.ToString()))
{
UI.ShowSubtitle(""); //removes the previous display.
return;
}
UI.ShowSubtitle(_mainMenuItems[e.SelectedIndex].Caption
+ IS_CURRENTLY + GetPropertyValueForSelectedMainMenuIndex(e.SelectedIndex));
}
@IncoCode Oh now I get you. Your reply was not directed towards me but to the team. So there is nothing I can do about it?

Any idea for a workaround? The users are waiting for the release of V5.0.0 of VAutodrive and I promised them an in-game configuration menu.

Just disable all the menu keys and restore them after the input is closed?

I found the culprit. It's not only the menu keys but also the line I commented out here:
...
private void ReOpenMainMenuAt(int index)
{
_currentlySelectedIndex = index;
View.AddMenu(_mainMenu);
_keyMenuIsOpen = false;
_mainMenuIsOpen = true;
_mainMenu.Items[0].Deselect();
//_mainMenu.SelectedIndex = index;
_mainMenu.Items[index].Select();
OnMainMenuSelectedIndexChanged(new SelectedIndexChangedArgs(index));
}
...
So is there any other way to highlight a certain menu element without the SelectedIndexChanged event being fired? It's this event that calls ChangeCarMoversPerimeter again and so I end up in a loop. Hmm I could detach that event and attach it again after the selction. I'll be back with the result...

This issue can be closed now


Release updates

Oh actually could you also update the test dockerfiles to use 5.10 now that our releases will be built with 5.10?

Yeah everything else looks okay, just updating test.dockerfile. test-ubi9.dockerfile, and test-amazonlinux2.dockerfile to use 5.10 and having the test run is all that's left.

@swift-server-bot test this please

@swift-server-bot test this please

@swift-server-bot test this please

@swift-server-bot test this please


Geocoding error in long tables (and unexpected size growth)

[from jira: https://cartodb.atlassian.net/browse/CDB-3599]
A user (and me) is not able to georeference his table with 127K rows.
A copy of the table is here:
https://team.cartodb.com/u/iriberri/tables/carto_20_281_29

Georeference by Countries using country column
Georeferenciation error. Table was 13 MBs initially. It grows 10x times without being georeferenced at all. After a vacuum full it returns to its original size.
This is not a problem of the values of the column itself. I have made a distinct(country) in the table and I get all the values georeferenced. The problem comes when I try to geocode the full table.
I have tried splitting the table into pieces. Up to 1K rows more or less it geocodes correctly. Another sets of 10K and 50K rows fail, and it also does another set of 100K rows.
There are no errors in the console.
cartodb_georef_status column is created after the error and set to null.
We have had two cases in SupportBee about this today.

fixed with batching


[CHANGELOG] remove docs updates from changelog

PR description
Besu docs used to be part of besu repo.
This is why documentation updates are listed in the besu changelog for older versions of besu
This PR removes those entries from the changelog - no longer relevant and also containing links that are likely to be or become out of date.
Fixed Issue(s)

Thanks for sending a pull request! Have you done the following?

[ ] Checked out our contribution guidelines?
[ ] Considered documentation and added the doc-change-required label to this PR if updates are required.
[ ] Considered the changelog and included an update if required.
[ ] For database changes (e.g. KeyValueSegmentIdentifier) considered compatibility and performed forwards and backwards compatibility tests

Locally, you can run these tests to catch failures early:

[ ] unit tests: ./gradlew build
[ ] acceptance tests: ./gradlew acceptanceTest
[ ] integration tests: ./gradlew integrationTest
[ ] reference tests: ./gradlew ethereum:referenceTests:referenceTests

Think this is a better option than keeping links up to date as per #7558
We could go further and delete anything that's already been released - discussion in discord https://discord.gg/hyperledger


Azure deployment - plugin:elasticsearch - Request Timeout after 1500ms

Using the new Azure Market place template mentioned in this post: https://www.elastic.co/blog/microsoft-azure-marketplace-elasticsearch-kibana-and-more-now-available I deployed an Elastic cluster but get the below error:
plugin:elasticsearch - Request Timeout after 1500ms
plugin:marvel - Waiting for Elasticsearch

Hey @MedAnd, @markwalkom forwarded on your email related to this too. Would you be able to indicate:

what cluster configuration you installed e.g. number of data nodes, ES version
if you're using an internal/external load balancer
if you set up a jump box
how you successful created indexes on the cluster.

Have you always seen both of those statuses, or were they initially green? How long have you had the cluster spun up for?

Hi @russcam
Thank you for the follow-up.

After deploying, in Azure I can see the following Resources (the version was 2.01):

elasticsearchMaster
esdata-set
esdatavm0
esdatavm1
esdatavm2
kibana-vm
masterVm0
masterVm1
masterVm2

I believe the load balancer is external:

Protocol/port
TCP/9200, TCP/9300
Backend pool
LBBE (3 virtual machines)
Probe
esProbe (TCP:9200)
NAT rules
0 inbound

I believe there is no jumpbox. I'm also connecting to Kibana directly on kibana-nsg TCP/5601.

I connected directly to ES on loadBalancer IP port 9200 and issued commands via Chrome Postman plugin to create indexes.

I've always seen the same failures. The cluster has been running for a couple of days. I hope the above helps, I could also privately send you the logon details?

@MedAnd The easiest way to avoid this is to change the load balancer to internal during generation.
However, you can change the IP address within the /opt/kibana/config/kibana.yml to the public IP for the load balancer. Via the template, it cannot guess the public IP so it can't fill it in for you.

@pickypg As we are working with two Azure ARM deployed clusters (one of them being Elastic) I've opted for the external load balancer. I had to change the yml file as mentioned above. All working now, thank you for the assistance.
PS. Wondering if deployment to an existing Azure vNet will be supported in future versions of the Elastic ARM template?

@MedAnd I would recommend opening a new issue for deploying a cluster to an existing vNet so it doesn't get lost in the comments here :smile:

This has been fixed, see https://github.com/elastic/azure-marketplace/issues/18#issuecomment-183303402 for the details.

@pickypg which user should be able to login and change the /opt/kibana/config/kibana.yml file?
I've only been able to ssh into the kibana server with es_admin, and it does not have the rights to make changes to the file ..

You will need to ssh into the box using the username/pass|key provided here:

Hoping to get a new release pushed to the production marketplace within the next weeks.

Thanks, I got another reply from Martijn on email, I was only missing "sudo" to get the correct permission level to save! 👍


Allow TLS client session resumption

In order to reduce the EVE device network traffic, the TLS hello exchange
is a big part of the inbound total traffic, and the server side certificate chain
does not need to be sent every time with the client session resumption.

Initialize the TLS config client session cache for resumption
align the code so that tlsConfig can be reused for subsequent sessions
add outbound authen additional length to the total sent content
add 'nim', 'diag' and 'zedcloud' for controller metrics
add 'DidResume' count for the cloud metrics
changed zedrouter probing to 'http' instead of default 'https'

Also, with these changes does the loguploader still fail to resume? If so are there any timers we can adjust on the server side?

as long as the next packet gap > 300 seconds, it will fail. Yes I think the nginx has some configs, but the real thing is later for Cloudflare, which i can not test at the moment. but i'm sure there is configs for this.


feat: spread Table.Row props

closes #717

@moonayyur you should be able to merge the PR yourself now (I granted you the permissions for it)
Please "squash and merge" and improve the commit message as it is what is going to end up in the changelog.

@stropitek I'm still not authorized to merge the PR


Hashing in C++

Is your feature request related to a problem? Please describe.
Using Hashing in C++ various problems can be solved like  Is sudoku valid, triplet sum in array, Key pair, Intersection of two arrays etc.
Describe the solution you'd like
Problems like  Is sudoku valid, triplet sum in array, Key pair, Intersection of two array will be solved using hashing.
Do you want to work on it
C++

Multiple problems cannot be raised in a single issue, please raise them in individually and kindly before raising an issue check if it's previously present or not.


Number Guesser

Hi! I would like to add a number guesser game using html css and js

Cool, go ahead

please assign me this issue. I would like to contribute in it.


add nomos image to build/publish (#345)

this image is not intended for use with Config Sync backend components. This image simply contains the nomos binary, and is intended to support use cases such as containerized CI/CD.

/assign @nan-yu
/assign @tiffanny29631

/lgtm


Enter key stops working

I reported the issue here: https://github.com/Microsoft/vscode/issues/53972
The enter key stops working and won't go to the next line. It's unclear how to replicate, as it happens seemingly at random. It only fixes itself after restarting vscode, or disabling/re-enabling the extension.
Since removing endwise, I haven't had this problem anymore.

Hmm. To make this extension work, we completely overload the enter key and handle line breaks and stuff. As you said its random, my guess would be that the extension host is crashing and the associated command does no longer execute.
But: It doesn't have to be this extension that is causing the host crash, I've seen this behaviour with a couple of other commands too in the past.
If it happens again, would you mind checking the Developer Tools (Menu Help->Toggle Developer Tools) and share the most recent lines from there? That might give at least a clue where to start looking…

I just now had the same issue. It only happens to certain files and disabling endwise fixed the issue. I restarted code several times as well.
I managed to reproduce the issue with this file:
#
module A
  module B
    module C
      module D
        module E
          class F

            description <<-EOD.strip_heredoc
            Blah blah
            EOD

          end
        end
      end
    end
  end
end

When I hit enter the Code Helper process goes to 100% cpu.

UPDATE: Well I've now done another couple of hours of extension wack a mole and I apologize because it seems endwise is not the issue. Not sure how I got to that in the first round.
The offending extension for me is highlight-matching-tag.

@wulffeld Already thought so, the code example didn't look to fancy to cause a problem in itself.
The way VSCode's extension architecture is designed makes it a little tough to find the proper cause of a freeze. There is a clear separation with the extension host running in a separate process to protect the editor itself from misbehaving plugins, which is brilliant.
But on the other side of the fence, all extensions run in the same process and have pretty much the same problem as every Node application: If a part of the system blocks the event loop, everyone gets stuck.
When I started this extension, I already had a bad gut feeling realising that I need to take over the Enter key completely as a keybinding. Not only does it need to handle the "end" 2 lines below the current cursor position, it literally writes the newline symbol, figures out the auto-indentation that would normally occur and then moves the cursor to the end of the next line.
I would have really preferred to just listen to a keypress event, let VSCode do its thing on the first line and only handle the next one appending the "end". But there is currently no API for doing so (as I remember, there was a reasoning behind not having keypress events).
Long story short, as soon as something is available to either use keypress events or release the binding when the extension host freezes, I try to work around this problem. But currently we are stuck with this and can only hope that other extension authors write code that doesn't crash. (But hey, at least I can now blame other programmers from time to time, that's nice too :D)


Create template to import games into scummvm

The template should be applicable to most scummvm games where they can be generated at the end of the installation to prevent the users from needing to manually add the games into scummvm

scummvm was added as a role.  Games that run on scummvm will need to have the role scummvm in their config so that the engine can be installed and configured prior to installing the game