[ENH] Minkowski distance

Reference Issues/PRs
Fixes #484
What does this implement/fix? Explain your changes.
This PR introduces a new distance measure minkowski_distance.
Any other comments?
PR checklist
For all contributions

[ ] I've added myself to the list of contributors. Alternatively, you can use the @all-contributors bot to do this for you.
[x] The PR title starts with either [ENH], [MNT], [DOC], [BUG], [REF] or [GOV] indicating whether the PR topic is related to enhancement, maintenance, documentation, bugs, refactoring or governance.

For new estimators and functions

[ ] I've added the estimator to the online API documentation.

For developers with write access

[x] Optionally, I've updated aeon's CODEOWNERS to receive notifications about future changes to these files.

Hello, sorry for the late reply. The people who are more familiar with distances are a bit busy at the moment.
There appears to be an issue with your example, the output does not patch the expected https://github.com/aeon-toolkit/aeon/actions/runs/6869352045/job/18682151763?pr=904#step:6:399

hey @MatthewMiddlehurst, thanks for response! . May I know what is the ground-truth framework we're testing it against ?

Hello, sorry for the late reply. The people who are more familiar with distances are a bit busy at the moment.
There appears to be an issue with your example, the output does not patch the expected https://github.com/aeon-toolkit/aeon/actions/runs/6869352045/job/18682151763?pr=904#step:6:399

Thanks for pointing it out! I will immediately correct it.

him, thanks for this, we will look early next week @chrisholder

Hi @akshatvishu thanks for the contribution! Look great just a few more things to add so people can call your distance properly:

Add your distance to the valid distances array located at the bottom of aeon/distances/_distance.py. Will look something like:

    {
        "name": "minkowski",
        "distance": minkowski_distance,
        "pairwise_distance": minkowski_pairwise_distance,
    },

Add your distance to the utility functions in aeon/distances/_distance.py. Add to the following functions:

line 87 distance function
line 227 pairwise_distance function

Add your distance to the various docstring tables in aeon/distances/_distance.py. You will need to add it to the following functions:

Line 705 get_distance_function
Line 757 get_pairwise_distance_function

Hi @chrisholder and @MatthewMiddlehurst,
Thank you both for your valuable feedback during the initial review. I've carefully considered and incorporated all the suggested changes into the latest commits. Hopefully everything will work fine this time!

Looks like the tests are passing now. Would you mind taking a look through @chrisholder just to make sure everything is still good before merging?

I will resolve the conflict and merge this for the release. Thanks for the contribution @akshatvishu (and dealing with testing woes)!


is_of_type does not work as expected with Annotated[Generic[T], ...]

Note this is in python 3.9 where typing.Annotated exists. I'm using pytypes from
pip install git+https://github.com/Stewori/pytypes.git
import pytypes
import typing
CT = typing.TypeVar('CT')
class Variable(typing.Generic[CT]):
    pass
pytypes.is_of_type(Variable[int](), Variable[int]) # returns True as expected
pytypes.is_of_type(Variable[int](), typing.Annotated[Variable[int], 'a']) # returns False unexpected

pytypes.is_of_type(3, typing.Annotated[int, 'a']) # returns True as expected

After stepping through the is_of_type function I end up in _issubclass_Generic.
I suspect there is some issue in _issubclass_Generic here:
          elif origin is not None and \
                  _issubclass(_origin(subclass), origin, bound_Generic, bound_typevars,
                          bound_typevars_readonly, follow_fwd_refs, _recursion_check):

takes an origin of the subclass so Variable[int] turns into Variable and origin of typing.Annotated is Variable[Int] so clearly _issubclass(Variable, Variable[int]) will not go through later parts of the function will basically all return False, so I think this is the place that needs to work.
Anyways let me know if I can be of any more help in resolving this issue.

Please note that Python 3.9 is not supported at the moment. Also 3.7 and 3.8 are not fully supported (but mostly). If you can make this particular issue work, feel free to do it (PR welcome). However, I suspect a broader 3.9 support would be a larger effort, which I currently cannot afford. (Changes in typing between 3.8 and 3.9 were manifold)


Attempt to use page language when formating release date

Fixes
Release date always being in English despite the language setting.
What is the current behavior?
Release date shown under the video is always in English
What is the new behavior?
Release date under the video tries to get the format based on language.
(Defaults to English when language doesn't match any locale in moment.)
Other information
For English language using LL changes the format from: December 7th, 2022 to December 7, 2022
But MMMM Do, YYYY doesn't work well with other languages
Screenshot
Release date for German language new vs current

PR Checklist

Toggle...
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Please check all that apply to this PR using "x":

[ ] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[ ] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

Nice! I think 7th is fine.

Noticed that there is some other places with similar thing.
Don't know if correct solution to the issue, but at least all is done here, in case it is


Fail Scapegoat on warnings

I'm using Scapegoat to check my project using gradle (and it's awesome, thank you so much). I would like warnings to cause the compilation step to fail; is there a way to cause any warnings to result in a compilation error?
Thanks!

you can always use -Xfatal-warnings
and you also have an option to override levels, see https://github.com/sksamuel/sbt-scapegoat#inspection-warning-level-overrides

@siddhuwarrier @mccartney
should we close this issue?


https://github.com/advisories/GHSA-257q-pv89-v3xv lists Nuget twice.

The last line item is Maven, not Nuget. Pls correct.

@cheburakshu Thanks for the heads up! org.webjars.npm:jquery is corrected to the Maven ecosystem now.

In the future, you are welcome to create a pull request in https://github.com/github/advisory-database to suggest changes to advisories and generate a community contribution. If the pull request is merged, the person who makes the pull request gets credit on the advisory.

Sure thing, thanks for fixing this!!


fix: refactor CLI for integration into AsyncAPI CLI

Description
This PR refactors the CLI setup so it can be integrated into the AsyncAPI CLI along side making it easier to maintain.

Hi @jonaslagoni, We have a merge conflict here.


Generic Pipeline crash error

Hi, I'm getting a generic pipeline crash error on the current release.
It seems though that this was solved on the master branch 3 months ago. Any plans to release a new patch version for this?
I tested running on master, but I don't feel comfortable enough depending on an edge version. Right now I have to switch back and forth whenever I have a problem in the pipelines I want to debug.
PS.: Thanks for the awesome work in this lib guys. It's really really cool!

going to let @oltarasenko decide on the new release :slightly_smiling_face:

I know +1 isn't welcome on GitHub generally, but since there are so few people here... +1 :)

+1 here as well :/


Cannot build docs/Dockerfile without Makefile

Because of the VOLUME , and GIT_COMMIT files you cannot build this dockerfile without the make file.  This should not be the case because the automated build system cannot create these containers for us.  We should either check in stubs of these files or modify the script not to fail

this was closed by new docs stuff


[WIP] Add Donate button

It's WIP as I still need to get the KMD and BTC address from Pondsea.

Looks great.
Could maybe do with a little more text, I'm not sure exactly what to put there but maybe something like:

HyperDEX is a free Open Source project. You can help support to support HyperDEX development by donating to the following addresses.

Maybe throw a ❤ emoji in there somehwere?

How about this?

Btw, open source should not be capitalized when used in a sentence: https://english.stackexchange.com/questions/157178/capitals-or-not-for-open-source

That looks great.

Btw, open source should not be capitalized when used in a sentence

Cool, did not know that, always thought it was like a title of a type of software.

Maybe move the ❤ to the beginning of the text to match the first start modal title:

✨ Thanks for Helping Us Test HyperDEX!

✨ Thanks for Helping Us Test HyperDEX!

How about we change this one instead to have the emoji at the end? It's more common for emojis to be after the text. They're not icons. And with the heart it looks unaligned when it's in front:

Sure, makes sense.

@lukechilds Ok. I'll do that later after this PR. The emoji in the intro modal is in the localized text, so changing the position of it will invalidate all the localizations for the string... I will look for a workaround.

Is this good to go now?

Beautiful


Allowing non-ASCII characters in email addresses (aka RFC 6532 support)

I received a Swift_RfcComplianceException when trying to send an email that contained a 'u' with an umlaut in the local part of the email address (eg. "müller@example.com").
Currently, SwiftMailer officially supports RFC 2822, which only provides for ASCII characters in the local part of the email address. Are there any plans to add support for RFC 6532? This would allow non-ASCII characters to be used in both the local-parts and domain of an email address.

I was just hunting around the code in preparation for making the necessary changes myself when I discovered that there is a major upgrade in the works (v6 on the master branch) that resolves this issue. The new version ditches a lot of the pre-existing Mime code and utilises another library for email address validation, which works with UTF8 characters. The only catch is that v6 will not work with PHP versions < 7.0.

Closing as this is indeed something part of the upcoming version 6.

Is there a possibility for you to make this change on the version 5? Couldn't use version 6 due to php requirements :/

@PierreRambaud Sorry for taking so long to respond. I'm not sure it would be a good idea to merge this change into the 5.x branch because it relaxes a very important validation rule that a lot of old applications might be relying on.
However, I can offer two alternative methods for including the change in your project:

Fork 5.x, make the change again on a new branch, add your repo to the "repositories" section of your composer.json file and then change the reference to swiftmailer to be         "swiftmailer/swiftmailer": "dev-your-branch-name". As long as you get all the references right, composer will magically pull in your branch instead of the 5.x branch from the swiftmailer repo.

Manually patch the two files using the following commands (and related patch files):

patch -N -r ignore.rej vendor/swiftmailer/swiftmailer/lib/classes/Swift/Mime/Grammar.php < swift1.patch
patch -N -r ignore.rej vendor/swiftmailer/swiftmailer/lib/classes/Swift/Mime/Headers/MailboxHeader.php < swift2.patch
swift1.patch:
-84c84
-<         self::$_grammar['atext'] = '[a-zA-Z0-9!#\$%&\'\*\+\-\/=\?\^_`\{\}\|~]';
----
->         self::$_grammar['atext'] = '[a-zA-Z0-9!#\$%&\'\*\+\-\/=\?\^_`\{\}\|~\\p{L}]';

swift2.patch:
-343c343
-<         if (!preg_match('/^'.$this->getGrammar()->getDefinition('addr-spec').'$/D',
----
->         if (!preg_match('/^'.$this->getGrammar()->getDefinition('addr-spec').'$/Du',

Before I switched to SwiftMailer 6 I used the second option and added those two commands to the scripts section of my composer.json file. I set things up so that they would be run automatically every time I ran composer install.
I hope that helps.

Hello.
I have a similar problem with Swiftmailer 6.1.2:
Method Swift_Mime_SimpleHeaderSet::__toString() must not throw an exception,
caught Swift_AddressEncoderException: Non-ASCII characters not supported in local-part

I have the ł character in the e-mail address which is perfectly acceptable when sending emails via any client like Gmail or Outlook.

Furthermore, addresses which contain UTF-8 characters in the local part pass the Egulias\EmailValidator\Validation\RFCValidation::isValid() check used by Swiftmailer.

@KKSzymanowski You need to use Swift_AddressEncoder_Utf8AddressEncoder and Swift_Transport_Esmtp_SmtpUtf8Handler. See “Internationalized Email Addresses” on https://swiftmailer.symfony.com/docs/messages.html#setting-bcc-recipients.

Thanks @c960657 for the link. But if I use:
$utf8Encoder = new Swift_AddressEncoder_Utf8AddressEncoder();
$transport->setAddressEncoder($utf8Encoder);

after this code the $transport object is always null. Any thoughts about this?

@KKSzymanowski I have the same issue with Swiftmailer (v6.2.1).
Swift_SmtpTransport luckily gives the ability to set extension handlers and address encoder, however
$smtpUtf8 = new Swift_Transport_Esmtp_SmtpUtf8Handler();
$transport->setExtensionHandlers([$smtpUtf8]);
$utf8Encoder = new Swift_AddressEncoder_Utf8AddressEncoder();
$transport->setAddressEncoder($utf8Encoder);

solves issue only partially.
There is a problem with Swift_Mime_SimpleHeaderFactory class. This class takes address encoder as the last argument in a constructor, and by default, it is Swift_AddressEncoder_IdnAddressEncoder.
During Swift_SmtpTransport construction, Swiftmailer resolves dependencies (of its parent's class) out of the container, and the problem is with container configuration.
To fix this issue (in the most simple way) and allow non-ascii emails, you need to override some of the registered dependencies:
\Swift_DependencyContainer::getInstance()
    ->register('transport.smtp')
    ->asNewInstanceOf('Swift_Transport_EsmtpTransport')
    ->withDependencies([
        'transport.buffer',
        'transport.smtphandlers',
        'transport.eventdispatcher',
        'transport.localdomain',
        'address.utf8addressencoder', // this is changed
    ])

    ->register('transport.smtphandlers')
    ->asArray()
    ->withDependencies([
        'transport.authhandler',
        'transport.smtputf8handler', // this is added
    ])

    ->register('mime.headerfactory')
    ->asNewInstanceOf('Swift_Mime_SimpleHeaderFactory')
    ->withDependencies([
        'mime.qpheaderencoder',
        'mime.rfc2231encoder',
        'email.validator',
        'properties.charset',
        'address.utf8addressencoder', // this is changed
    ]);

@fabpot do you think it would be feasible to make that address encoder thing parametrized (via config, environment variable, whatever) and use that during dependencies registration? It would be developer's choice to use idn or utf8 address encoder...


WSL2 not using all the cpu available.

Version
Microsoft Windows [Version 10.0.22000.675]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
5.472
Distro Version
Ubuntu 22
Other Software
Windows 11
Repro Steps
load your cpu on a WSL2 Ubuntu 22 distro
Expected Behavior
CPU on the task manager at 100%
Actual Behavior
Only using 50% of my cpu
Diagnostic Logs

Could you check the Details tab or use Process Explorer (https://docs.microsoft.com/en-us/sysinternals/downloads/process-explorer) instead?
It appears some parts of Task Manager will not show the actual CPU usage for WSL processes. It is at least that is true for WSL1.


Xray report resources don't support import

Describe the bug
Xray report resources, namely, xray_vulnerabilities_report, xray_violations_report, xray_licenses_report, don't support import opposed to the documentation that claims otherwise.

xray_vulnerabilities_report
xray_violations_report
xray_licenses_report

Requirements for and issue
Planning failed. Terraform encountered an error while generating this plan.

│ Error: resource xray_vulnerabilities_report doesn't support import
│
│

Planning failed. Terraform encountered an error while generating this plan.

│ Error: resource xray_violations_report doesn't support import
│
│

Planning failed. Terraform encountered an error while generating this plan.

│ Error: resource xray_licenses_report doesn't support import
│
│

Jfrog artifactory version: 7.77.1
XRay version: 3.86.10
Terraform version: 1.5.5
Expected behavior
Resources should support import.

@neatcoder Thanks for the report. Looks like a bug from first glance. I'll add this to our plan to investigate.

@neatcoder After familiarize myself with this part of the code (haven't touched it in many months), these report resources do not support import. So the documentations are incorrect and I'll be removing the "Import" section.

Any plans for extending the import support to these resources in future?

@neatcoder Before we can add import support to the provider, the Xray API must support getting all the report generation detail.


Intallation documentation

Dear colleagues,
I hope this message finds you well. I'm a PhD student at Waterloo interested in trying your method to simulate driven dynamics of a flux qubit under noise with long-range correlation. I wonder if INSTALL.rst is up to date? As it stands I can only see "pip install ." which does not work. I have tried with sbc and spinbosonchain as well but neither was found. Many thanks in advance for your reply.
Sincerely,
Chi Zhang
University of Waterloo

Hi I'd like to close my own thread. Reading the installation more carefully would have told me that I would need to change my directory to the cloned directory first. I'm leaving this up in case others make similar mistakes as me.


Incompatibility with Android Studio 4.1.1

The last version compatible with Android Studio (4.1.1) is v25.1 and it's impossible to install the version v32.0.
Can you make it compatible with the latest version of Android Studio?

More info:
Android Studio 4.1.1
Build #AI-201.87<IP_ADDRESS>53283, built on November 5, 2020
I checked the "plugin.xml" file and it's because the minimal version is 202 (<idea-version since-build="202" />)
The last version of Android Studio is 201.87<IP_ADDRESS>53283.
Is there a constraint for the minimum version to be 202?


use pretty pure grammar in entity diff view

Summary

use pretty pure grammar in entity diff view
How did you test this change?

[ ] Test(s) added
[x] Manual testing (please provide screenshots/recordings)
[ ] No testing (please provide an explanation)

before:

after:

fix your css on your local computer :)

@YannanGao-gs  can you do a code search whereever we convert to pure code, see if anyplace else needs the pretty flag. should almost be universally set


Multiple data centres broken

When using the -n 4:4 argument to the populate command, the data centre value does not get saved in node.cfg.
Then when using the updateconf the cassandra-topology.properties is reverted back to default.

This is happening again.


Issue with using ObjectnessBING

Having trouble running BING:
 cv::saliency::ObjectnessBING bing;     bing.setTrainingPath("/home/raman/opencv/opencv_contrib/modules/saliency/samples/ObjectnessTrainedModel");     bing.setBBResDir("result");     std::vector<cv::Vec4i> bounding_boxes;     bing.computeSaliency(img_c, bounding_boxes);     std::vector<float> scores = bing.getobjectnessValues(); // float only
I am getting following errors:
error: undefined reference to cv::saliency::ObjectnessBING::ObjectnessBING()' error: undefined reference to cv::saliency::ObjectnessBING::setTrainingPath(std::string)'
etc
If i declare  cv::saliency::Objectness bing1;  I get following error message:
error: cannot declare variable 'bing1' to be of abstract type 'cv::saliency::Objectness'
cv::saliency::Objectness bing1;
^
i think the problem is that i cant add "#include "opencv2/saliency/saliencySpecializedClasses.hpp" to my code. The compiler gives error again :
/usr/local/include/opencv2/saliency/saliencySpecializedClasses.hpp:70: error: expected class-name before '{' token
{
^
etc.
Is there a sample code of bing running successfully.. all the examples i have seen are of 2015 may or before and the opencv3.1 code/design seems to have changed after that/

managed to resolve it by reinstaling 3.1. closing


Add syntax highlighting to web application

The shaclex web application contains textareas to input both RDF data and ShEx/SHACL schemas. It would be nice to add syntax highlighting during the input process.

Done


Msis

Msis:

pyarrow interprets DATE column as datetime.
removed the "static" flux variant of extract profile

Start L2:

Use StepFunctionsStartExecution task to start L2
Map State and Pass to tweek to the right input

Looks good, but I'm wondering what the effect of adding the compile step in odin-cal is. Does this mean all data up until now needs to be re-run? We were planning to do so anyway, but I'd like to know.

Probably no effect, compared with precompiled in repo.


Bug: can't export the "selected file," but I have no files selected

Debug log ID
547B2KIC-refs-euc
What happened?
I'm seeing the same filename of 'pdf'.  In addition, when I export a collection with "export files" clicked (screenshot 1),   I get an error popup saying it can't export the "selected file," but I have no files selected (screenshot 2).  I believe that was working yesterday or the day before.
I don't get an error if I unclick "export files"

547B2KIC-refs-euc does not contain any errors. Did the problem occur before you sent a debug log? I also see no indication in the log of an export being performed.

an error popup saying it can't export the "selected file," but I have no files selected (screenshot 2)

That's unfortunate phrasing from Zotero but it's just the general message you get when anything went wrong during export, even if nothing relating files was exported.

I can't currently replicate the problem, so I will need a debug log with the error in it. Can you first upgrade to 6.7.97?

I just upgraded, and then realized that I already was on 6.7.97 -- now it's 6.7.98, and the "an error occurred" problem is gone.  So thanks for that fix, although the "pdf" filename thing is still there.
Here's the logid.  GC5RSAR3-refs-euc

I see an autoexport being added in that log, but there is no error in the log.

If you can reproduce this, I need a new log.

Closing for inactivity


Is there a way to change selected tab header?

Basically, I want to change the backgroundColor and the label color whenever you select a tab, but I don't know If there is a property or a method to do so. For example, I have this 2 tabs:

And I want them to look like this:

As you can see, depending on which one is selected, its styling changes.

Hi @FacuAcosta! You just have to use the renderHeader prop to do so. Then you can styled your header as you wish (example here).


Support Multiple Targets

Currently only one single target can be used. To implement the functionality to use several different targets the following steps are necessary

[ ] Create a TargetActor which represents the target and can contain content
[ ] Move the target related functions in CFVuforiaActorComponent to this new actor
[ ] Depending on the trackable that was detected, show the content of the corresponding TargetActor and move the camera relatively to the TargetActor's position

Sure, it is easy to add a detecting target name. You have to do add
in CFVuforiaActor.cpp  after 146 linie:
		TargetName = trackable.getName();
in CFVuforiaActor.h in public section
	 	UPROPERTY(EditAnywhere, BlueprintReadOnly, Category = "Vuforia") 	FString TargetName;
And in effect you can put the node 'get TargetName' and compare it to your prefer name.
In next step you can make own database on vuforia sdk website, download and put it in plugins/CFVuforia/Assets and change the name in CFVuforiaActorComponent.cpp. In effect you got own, individual markers, which are detected by boolean variables.

Thank you for your help @valake, cool idea that defenitely works. Just send us a pull request :-)
The solution that @geovie ment, was more a visual one. To have a plane with the targert rendered on (the TargertActor), which has its own content as a child on it. I think having a representation of the target visible in the scene is easyer to understand for users that are not so familiar with AR and in order to suppuort multiple target tracking (or any other form or object tracking) in a further step, different TargetActors will be necessary :-)


use opn instead of open

opn is currently maintained, unlike open.
Also, it works correctly on my machine. Unlike open, which opens three Firefox tabs: www.http.com, file:////localhost and 6060/README.md :D

Thanks!


[BUG] Noon message not working

Describe the bug
The noon message is no longer sent since the new version.
Expected behavior
A message should be sent at least in the Sonia guild for the general channel.

Since there is no coverage is it difficult to test but I think that this is somehow related to a potential undefined value not checked when accessing a channel.
It could just break the whole thing and stop the process.
I added a nullish check to make sure that the channel exists without throwing an error and I tested it locally with a minute schedule and it works.


Explanation of # of Steps and # of Ensembles

Can you please specify in general terms what is number of steps and number of ensembles? I do not have a good understanding of diffusion and flow matching models. Also, what would you suggest as a good combination. I see the defaults set to 2 & 4. But as I increased the number of steps, I am seeing more details in the depth map.

SD-based depth estimators utilize multiple shots/images and compute their average. The ensemble size corresponds to the number of shots, equivalent to the "batch size" in Stable Diffusion UIs. The steps parameter represents the number of denoising iterations, with higher values generally yielding better results until reaching a convergence point. In my experience, I've found that 4 steps and 4 ensembles is optimal.

Thank you for the explanation!


How to define domain without 'domainPath'?

I'm trying to integrate this library with my application. One issue I run into is that currently I 'compile' all my code (client and server) into a single JavaScript file (long story). Anyway, this currently prevents me from using this library since the domain definitions are loaded from files located by the 'domainPath' option.
Could you imagine another way to load the definitions? Something not based on loading files? I'm not familiar with the code base yet, so I couldn't find another way myself ...

Never mind, I reconfigured my server build :)


Please update the mik3y/usb-serial-for-android lib to 3.7.0

Hey bastengao)
Could you please update the mik3y/usb-serial-for-android lib to 3.7.0, because at the moment my devices is not supported in 3.4.3 but supported at the 3.7.0.
Thank you!

Thank you very much, I'll test it in the next few days and report back.

@bastengao Thanks now it works well!


print error for git-bash using oh-my-zsh  with autosuggestions

Windows Terminal version
1.11.3471.1
Windows build number
10.0.19042.0
Other Software
git-bash
zsh(with msys2pack unpacked to git root)
oh-my-zsh(for zsh)
autosuggestions(for zsh)
Steps to reproduce
add git-bash to terminal
launch a git-bash tab
launch zsh
input some long word
Expected Behavior

Actual Behavior

Wow that is wild. That kinda looks like the newline/autoreturn bug that we see in mingw with some frequency, but it's... different.

it works fine with ... WSL

By "WSL" I'm assuming you mean in the vintage console, instead of in the Terminal? That would be extra curious. I would expect this to be broken in the same way in both...

You can see the image under "Expected Behavior" which use the origin git-bash console instead of in the Terminal , it works fine
I install the oh-my-zsh with autosuggestions under wsl, and launch it with the Terminal, it works fine too
but for the image shown blow "Actual Behavior", I launch git-bash with the Terminal, and tap some word, it gets wrong outprint
Anyway, this problem is solved by switch zsh-autosuggestions verison from v0.7.0 to v0.6.4, and I guess it has nothing related to the Terminal


Add disk req for STAR (resolves #447)

Needed for caching

@hannes-ucsc — Ready for review.

Just for the record: I expressed my consent verbally.


jellyfin-mpv-shim: 2.7.0.post2 -> 2.8.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for jellyfin-mpv-shim is: Allows casting of videos to MPV via the jellyfin mobile and web app
meta.homepage for jellyfin-mpv-shim is: https://github.com/jellyfin/jellyfin-mpv-shim
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.8.0 with grep in /nix/store/9ws8ligmh7g0q8zcd18hyh1apx9g9xg8-jellyfin-mpv-shim-2.8.0
found 2.8.0 in filename of file in /nix/store/9ws8ligmh7g0q8zcd18hyh1apx9g9xg8-jellyfin-mpv-shim-2.8.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

jellyfin-mpv-shim

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/9ws8ligmh7g0q8zcd18hyh1apx9g9xg8-jellyfin-mpv-shim-2.8.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A jellyfin-mpv-shim https://github.com/r-ryantm/nixpkgs/archive/da11bdb8ba2bf991dc53c585711836cdf9621c90.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/da11bdb8ba2bf991dc53c585711836cdf9621c90#jellyfin-mpv-shim

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/9ws8ligmh7g0q8zcd18hyh1apx9g9xg8-jellyfin-mpv-shim-2.8.0
ls -la /nix/store/9ws8ligmh7g0q8zcd18hyh1apx9g9xg8-jellyfin-mpv-shim-2.8.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages built:

jellyfin-mpv-shim
jellyfin-mpv-shim.dist

Maintainer pings
cc @jojosch for testing.

seems like jellyfin-mpv-shim is broken on unstable, might need a python3Packages.mpv upgrade: https://github.com/jaseg/python-mpv/commit/d96eaf7e64f1063ae16f88abfaa14c2997b3291d
With the jellyfin-mpv-shim from this PR and from master, I get this error:
2024-06-10 20:52:20,772 [   ERROR] mpv: main: Command loadfile: argument index can't be parsed: option requires parameter.

I'll try with to do the python3Packages.mpv upgrade


I can't run game if use MU3Input.dll included in ongeki-io

If i don't use MU3Input.dll, the game will run normally, but the moment edit segatools.ini to specify the path to MU3Input.dll, the game will not run.
I attached some pictures

I will not use MU3Input.dll, the game will run without any problems by passing through the 'システムプロセス' entry normally.

When load MU3Input.dll, the game no longer progresses beyond 'システムプロセス'
Is there anything I missed?
Overwriting the files in configs.zip but I did not solve the problem.
If load ongeki-io version 2.0.0 of dll, it runs normally, but I couldn't use the controller because it's an older version.
I tried everything all over again and changed the ongeki-io version and kept trying, but it didn't work out..
Game versions I tried were SDDT 1.45 and SDDT 1.39

I think this issue may be caused by differences in the AMDaemon version. When my AMDaemon starts running, it outputs its version number. The absence of a version number in your screenshot may indicate that it’s not running, or that you’re using a different version than I am.

Is there any action I can take?
Even though the game is running fine, but I can't see the version of AMDaemon in my logs.

Could you provide a copy of your package folder? Please exclude its subfolders to reduce the file size.

package.zip
yes, I attached the file.

I'm sorry, but I can't help you with this. I used your package to start AMDaemon and then launched the game, and I didn't encounter this issue.

Ok thank you, I'll try running it on another PC
So I think it's probably a system environment issue, not a file issue.
But I don't understand why the problem doesn't occur with ongeki-io 2.0.0

I finally solved this problem after 3 days..
This symptom occurred the same when running on another PC and it was related to a network conflict.
To fix this, It must need to completely remove all wireless connections except the wired connection in IOConfig. (I didn't remove the wireless connection even when I was wired.)
Can you please add a solution to the user manual for people who have encountered similar situations as me?

Hello
Thank you for your feedback.
Your issue has been added to the User-Reported Issues section of the wiki.
User-Reported Issues


Add minimum alignment support for loongarch64

The loongarch64-unknown-linux-gnu was added as a tier 3 target, add minimum alignment support for loongarch64 now.
Thanks

:pushpin: Commit 824abba7bf973b6e024aa29778a5e62fbdaedf88 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 824abba7bf973b6e024aa29778a5e62fbdaedf88 with merge 53e6e0e8b0ef243aebba1d0deeefc259e55946af...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 53e6e0e8b0ef243aebba1d0deeefc259e55946af to master...


Sulu UI backend sorting is a bit confusing

The Sulu UI backend sorting is a bit confusing, in my opinion the most important content from a user perspective is:

Web-Spaces (in Web-Spaces: 1. Homepage 2. Content)
Assets
CRM
Settings

The reason for the sorting is, that if you have multiple webspaces the other modules will be pushed down. With many webspaces even out of sight. That's why we put them on the bottom. The sorting within the webspace module will be changed so the "Homepage" will be fist, followed by "Content". Which is the more obvious solution.

In that case this issue will be fixed by https://github.com/sulu-cmf/sulu/pull/789

That PR fixes only the homepage/content order, not the order of the sections!

As Bernd has described it's the only change we will apply, so it will be fixed afterwards.


createAnimatedComponent does not support stateless functional components; use a class component instead.

Description
I just started a react native project to play around using react-native init and did the linking step.
Changed the default buildToolsVersion of my project from 23.0.1 to 25.0.0, which is the minimum sdk version required.
Only added this line of code to the default react-native project generated:
import Animated from 'react-native-reanimated;
And I get this strange error:
`createAnimatedComponent` does not support stateless functional components; use a class component instead.

I guess this could be related to this https://github.com/kmagiera/react-native-reanimated/commit/cc4ab7fbe704d219a01db00dd440c5600caf70b9 ?
Steps to reproduce:

react-native init PlaygroundApp
yarn add react-native-reanimated
react-native link react-native-reanimated

Versions:
"react": "16.4.1",
"react-native": "0.56.0",
"react-native-reanimated": "^1.0.0-alpha.4"

Hey @rodolfovilaca! Thanks for reaching out. This issue has already been reported and fixed around the same time you posted this issue. Please check if alpha.5 works ok for you https://github.com/kmagiera/react-native-reanimated/releases/tag/1.0.0-alpha.5


Start markdown

this is an h1
this is an h2
this is an h3
this is h6

This is a test of markdown

THIS IS A TEST OF MARKDOWN

This is a test on Markdown


Angular fails to build inside docker when target directory for dist does not exist

Bug Report
Affected Package
Angular compiler(?)
Is this a regression?
Yes, it worked in 12.0.x. Doesn't work in 12.1.0.
Description
When running ng build --configuration production inside a docker image, and the target dist directory does not exist, build fails with the following error (before compiling ES2015 to ESM2015 messages appear):
angular/angular#15 31.41 An unhandled exception occurred: ENOENT: no such file or directory, stat '/usr/app/dist/project'
angular/angular#15 31.41 See "/tmp/ng-WYeBTH/angular-errors.log" for further details.
Can't get to the log, since the image is thrown away.

Error does not occure when running ng build locally on my Windows machine.
Error occurs on docker hub and on my local machine's Docker desktop in multiple projects.
Error does not occure while building a library, but on every frontend (app).

Minimal Reproduction
Dockerfile
FROM node AS build
WORKDIR /usr/app
COPY ./package*.json ./
RUN npm install
COPY ./ ./
RUN npm run build --configuration production
FROM nginx
WORKDIR /usr/app
COPY --from=build /usr/app/dist/project/ /usr/share/nginx/html/
COPY ./nginx/angular.conf /etc/nginx/conf.d/default.conf
Reproducing with (nearly minimal) repo:
https://github.com/rongem/spi/
Exception or Error

An unhandled exception occurred: ENOENT: no such file or directory, stat '/usr/app/dist/project'

Your Environment
Angular Version:

Angular CLI: 12.1.0
Node: 14.16.0
Package Manager: npm 7.19.0
OS: win32 x64

Angular: 12.1.0
... animations, cli, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1201.0
@angular-devkit/build-angular   12.1.0
@angular-devkit/core            12.1.0
@angular-devkit/schematics      12.1.0
@schematics/angular             12.1.0
rxjs                            6.6.7
typescript                      4.2.4

Anything else relevant?
Docker version 20.10.7, build f0df350, and also Docker hub autobuild feature
Only Docker related, and libs are not affected
Workaround inside Dockerfile is pasting mkdir between copy and ng build
COPY ./ ./
RUN mkdir dist/project
RUN npm run build:all

Duplicate of #21202.


Add bacteria test-data for nf-core/modules

This PR adds some basic test-data for some bacteria specific nf-core/modules that don't quite work with SARS-CoV-2 test data. I think its a good start, since there are cases (e.g. vcf) where some of the existing test-data works great.  Overtime, bacteria-specific test data can be added as the need arises.
In these test data I've selected Candidatus Portiera aleyrodidarum which a bacterial endosymbiont for White Flies. It has a genome size of 358,242bp, so it plays quite nicely with basic testing.
I'm currently using this for testing in Bactopia (https://github.com/bactopia/bactopia-tests), and here's example of GitHub Action (https://github.com/bactopia/bactopia/actions/runs/1256507737)
About the data

bacteria/genome/*
These data are all based of GCF_000292685

bacteria/illumina/fastq/*
These data are based off FASTQs from SRX1390609

bacteria/nanopore/fastq/test.fastq.gz
While I could have used any Nanopore data (e.g. subsampled E. coli), I chose to use Nanopore data to match C. aleyrodidarum. The only Nanopore data for C. aleyrodidarum was in the form of metagenomic data from ERX3774356. I extracted reads C. aleyrodidarum reads and thats the file found here.
Full details on how I created this are here: https://github.com/bactopia/bactopia-tests/tree/main/data/fastq/nanopore

Since this is adding a new group of test data to the modules test-data, your feedback and guidance is very much appreciated.

Would GFFs like in this PR be worthwhile to add? https://github.com/nf-core/modules/pull/777
Also should I add species specific test data, like these: https://github.com/nf-core/modules/pull/773 https://github.com/nf-core/modules/pull/772

Need to update the README too @rpetit3 . Will merge in for now. Thanks alot.


[BEAM-2085] Fixups for Python resource hints.

Allow resource_hints in Python pipeline option for consistency with experiments and dataflow_service_options.
Raise ValueError instead of Keyerror for proper errors message when hint parsing fails.

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

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

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

GitHub Actions Tests Status (on master branch)

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

R: @robertwb

precommit failure unrealted:
11:17:03 apache_beam.io.gcp.tests.pubsub_matcher: ERROR: Timeout after 400 sec. Received 0 messages from projects/apache-beam-testing/subscriptions/wc_subscription_output3afcfa29-f037-486c-ad2f-ab0c6d841cd3.
11:17:03 --------------------- >> end captured logging << ---------------------
11:17:03 Worker logs: https://console.cloud.google.com/dataflow/jobs/us-central1/2021-04-21_11_02_57-16440804195861241526?project=apache-beam-testing
11:17:03
11:17:03 ----------------------------------------------------------------------
11:17:03 XML: nosetests-preCommitIT-df-py37.xml
11:17:03 ----------------------------------------------------------------------
11:17:03 XML<EMAIL_ADDRESS>11:17:03 ----------------------------------------------------------------------
11:17:03 Ran 1 test in 872.802s
11:17:03
11:17:03 FAILED (failures=1)
11:17:03
11:17:03 > Task :sdks:python:test-suites:dataflow:py37:preCommitIT_streaming_V2 FAILED
11:17:03


Set up, configure and read temp probes

DS18B20

https://github.com/jerabaul29/OpenMetBuoy-v2021a/issues/93
https://cdn.sparkfun.com/datasheets/Sensors/Temp/DS18B20.pdf?_gl=1*152fmo2*_ga*MTI5MTg4NDI3MC4xNzA4NDE2Nzcy*_ga_T369JS7J9N*MTcwOTczMTE5MS4yMC4xLjE3MDk3MzMwMzAuNjAuMC4w

Maybe use: https://www.analog.com/en/products/ds2482-100.html

https://github.com/bartweber/one-wire-ds2482


Testing  linkml-runtime v1.3.0rc7

@hrshdhgd do we still need this?

I don't think so. It was just a test Chris wanted me to run.


allow rgw multisite endpoint re-configuration

If I already have a existing rgw multisite setup, but I want to reconfigure the endpoints due to new IP/dns layout current playbooks can't be used to update the existing zones and zonegroups

This is still a valid bug


"ID / account number" question type: define UI/UX/functionality

We have some a mockup for this question type, but we need to get more detail on the final UI/UX. In Miro, please help us define:

What the error text for each possible type of error looks like, and where on/near the widget it's shown
How the widget looks when it's in a default state (nothing entered), a focused state, a valid state (correct input entered), and an error state.
Suggested pixel dimensions for all input field(s) in questions of this type.
How the custom functionality, if any, should work.
Anything else needed to define how this question type works.

Ideally, include 2-3 mockups showing what the question looks like in several different states (e.g. you've inputted half the answer, you've inputted all the answer but there's an error...)
See also #453, the central issue for this question type.
Feel free to only take on part of this task, by the way.

See #453 for mockups!

Sorry, the Miro link from above broke. See tiny.cc/cvf-flow.


Login with Social Media

Fixes #463
Description
This PR includes a new button for the Home page (on the EVM side) which provides the users a way to authenticate using social media providers without even having a wallet installed.
Notes

I removed the timeout when sending tokens because it was annoying. Maybe we should discuss if we want this timeout or not.
I'm currently using my own app id for testing because I needed to do some tests directly. We need to decide which Identity providers are we going to be using and enable them in the app registration.

Test scenarios

go to this link
Press the "Login with Social Media" button

A menu with 4 social media authentication providers should appear

Choose any provider and press the button

a window will pop up to ask you for a pin (If is your first time, it will ask you to repeat the pin)
after confirmation

the window will load and close automatically
you will be logged in with a zero balance account

Copy your address, and send to that address some tokens from another account you control
After you have some tokens in your balance, go to send page and try to send them (make sure you have some TLOS)
When pressing Send button, a window will pop up
Enter PIN and confirm

the transaction should go with no problems.

Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have cleaned up the code in the areas my change touches
[x] My changes generate no new warnings
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings
[x] I have removed any unnecessary console messages
[x] I have included all english text to the translation file and/or created a new issue with the required translations for the currently supported languages
[x] I have tested for mobile functionality and responsiveness
[x] I have added appropriate test coverage

[ ] update appId (testnet)
[ ] update appId (mainnet)

Re-opening this PR that I think was closed by mistake


Isnt there a better way to wrap REST APIs in a client?

I think there might even be a julia package that does this. Right now I am hard-coding the block structure from existing blocks. Potential support could be:

creating blocks, deployments
creating flows, tasks

Again, Prefect had a blurb that they were working on a Julia SDK.

Yes, there are established code-generation techniques to automatically create the REST API client package.
AWS.jl design document describes one approach, covers all AWS services.
Swagger.jl (OpenAPI) packge is a code generator plugin to create your own client library. Its Java based.
So... yes there is a right way, but they are a heavy lift. Maybe the Swagger way is actually pretty easy -- not ready to put effort into learning it just now.
So far the hand rolled small-scale is the best approach. I only need a small set of API calls to run the Julia part of my deployed jobs. Don't need to deploy or define flows in Julia -- just need the following:

access to block information
logging from Julia process

Notes from slack, other examples of API wrapping:
Trello.jl
Airtable.jl
They say avoid AWS.jl as an example, lot of design driven by the huge size of the API.

New one that is very clean example, MatterMost.jl. OpenAPI generated interface.
So what else is needed for writing Julia code that orchestrates with Prefect?

@flow decorators functionality easy in julia, even macro-style
prefect cli, it works the same? I suppose so. And running a flow from file is julia --file=... type of thing. The worker is local process, the observability comes from the @flow macroized calls from the julia script.


Refactors linearizer acc to a Dict

moving in the direction of supporting multiple reduceops (sequential and parallel), this lets the linearizer track multiple accumulators

I picked reduce_acc


multiply blend mode fix for spatial materials

The blending code that was being used for BLEND_MODE_MUL was the same as the code for BLEND_MODE_MIX so I've fixed this.
Fixes #11822

Cherry picked into 3.0.3.


Can I use ncm source for ncm2?

Can it be compatible?

No, it needs to be ported.

Understand, thank you for your reply


Grab the users custom avatar or default avatar to display properly

Details

I believe this came from https://github.com/Expensify/App/pull/20276/ because it was using getDefaultAvatar which will only ever grab the default avatar for a user and never one they might have uploaded. I believe this new method will first grab a custom one if present then default to the default avatar if it needs to but honestly I'm really confused by UserUtils.getAvatar since you have to pass it the url you think you want 🤷
Fixed Issues

$ https://github.com/Expensify/App/issues/20683
Tests

[X] Verify that no errors appear in the JS console

Ensure you have a custom avatar uploaded for your user
open up a chat with another user
click on the chat name to show the member list (or navigate it however you choose)
Hover over the avatar which should show a tooltip of the avatar again (?)
hover over the user's name/login which should show a tooltip of the avatar.
Ensure the avatar is the same in both those tooltips

Offline tests

Ensure you have a custom avatar uploaded for your user
open up a chat with another user
click on the chat name to show the member list (or navigate it however you choose)
Hover over the avatar which should show a tooltip of the avatar again (?)
hover over the user's name/login which should show a tooltip of the avatar.
Ensure the avatar is the same in both those tooltips

QA Steps

[x] Verify that no errors appear in the JS console

Ensure you have a custom avatar uploaded for your user
open up a chat with another user
click on the chat name to show the member list (or navigate it however you choose)
Hover over the avatar which should show a tooltip of the avatar again (?)
hover over the user's name/login which should show a tooltip of the avatar.
Ensure the avatar is the same in both those tooltips

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

Is this a regression PR?

@fedirjh Can you test your proposed solution if it works well?

So then is this because we're halfway between transitioning the PERSONAL_DETAILS and PERSONAL_DETAILS_LIST onyx keys?

which was identified here https://github.com/Expensify/App/issues/20672 and solved with https://github.com/Expensify/App/pull/20713/?

@bondydaa ok so should we just live with this problem for a bit and let it be handled in here https://github.com/Expensify/App/pull/20473/files#diff-cfc1b0041979edd1cf120bf6f6b714a0c81f0562ee1aafbda836ce61aae559d5? we plan to merge that today

okay closing this then.


[Bug]: size unit suffix 'b' should be optional

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Environment

Version or commit-id (e.g. v0.1.0 or 8b23a93): 3dfc8253f54c90c78de2985c01f26468c516657d
Hardware parameters:
OS type:
Others:

Actual Behavior
Configure memory-capacity to 10Gi, and start the mo-service
error: invalid suffix 'gi'

This issue is introduced by upgrading https://github.com/docker/go-units from v0.4.0 to v0.5.0, the commit https://github.com/docker/go-units/commit/737572633c434ce2d80ba3fbe2d8ea15d7d821ff change the unit parsing algorithm from regex to basic string parsing, which introduce a bug that the b or B suffix is now required.
xref: As documented in the package itself,  the b suffix is optional, which is also documented in https://en.wikipedia.org/wiki/Binary_prefix
Expected Behavior
Both 10Gi and 10GiB should be valid unit.
Steps to Reproduce
No response
Additional information
No response

reported to https://github.com/matrixorigin/matrixone/issues/9507


Deployment success icon seems off on Ubuntu

Is this intentional?
I think that the icons should appear before the tasks description after a task finishes, and not in the place of the letter "x" in "Executing".

Yes, it's a bug. On Mac OS x everything looks normal. Do you know why this happens on Ubuntu?

I have no idea, sorry. :/

I can confirm it works perfectly on Mac:

Really strange behavior.

Hi, I just debug about this problem , it is about ascii cursor control char. based wiki:https://en.wikipedia.org/wiki/ANSI_escape_code  I found 2 way to fix this on ubuntu(no mac for test).
'✔' will have 2 chars position,  so I have add "\033[3D"  (means move cursor left 3 char). or  just add "\r" to go back to  line start position.
Deployer\Console\Output\OutputWatcher line 42
$this->output->write("\033[k\033[1A\033[3D➤\n", false, $type);
or
$this->output->write("\033[k\033[1A\r➤\n", false, $type);

Deployer\Executor\Informer line 52
$this->output->write("\033[k\033[1A\033[3D<info>✔</info>\n");
or
$this->output->write("\033[k\033[1A\r<info>✔</info>\n");

any one can help to test on mac ?

Wow, cool! Can you create PR so i can test it?

Fixed.

excellent :)


Update Article “roudgara-forest”

Automatically generated by Netlify CMS

👷 Deploy Preview for 4shame processing.

Name
Link

🔨 Latest commit
72ef86a36a3c8a7a9e118e65864839b44f40614a

🔍 Latest deploy log
https://app.netlify.com/sites/4shame/deploys/633b0da346904300087d437c


How to make workflow tasks execution process as synchronous instead of asynchronous

I have a workflow consists of two tasks.
We are having status for task execution level and workflow execution level.
I want workflow status to be updated as completed, If both tasks executions is completed.
But somehow, workflow status is shown as completed, even though second task is in scheduled state.
Is it because of asynchronous behavior of workflow execution?
If yes can anyone suggest how to make my workflow execution as synchronous?

Can you post your task and workflow definitions?

Closing this as we haven't been able to reproduce this issue. Please reopen if you still encounter this and provide workflow and task definitions for the same.


Checks accessibility in the #slice and #duplicate methods of ByteBuf

Motivation:
The ByteBuf#slice and ByteBuf#duplicate methods should check an accessibility to prevent creation slice or duplicate of released buffer. At now this works not in the all scenarios.
Modifications:
Add missed checks.
Result:
More correct and consistent behavior of ByteBuf methods.

Has anyone done it in an app that creates a nifty jar in  drag & drop?

@fenik17 thanks!


Align PayPal API with iOS

Summary of changes

WIP

Checklist

[ ] Added a changelog entry

Authors

List GitHub usernames for everyone who contributed to this pull request.

the listener implementation looks like what we did on web fallback, so i think it looks good.


base.scss or _base.scss?

The file is _base.scss, but we import base.scss. Why?
https://github.com/funkhaus/vuepress/blob/master/src/App.vue

That's a SASS convention - you need the leading underscore to show that the file is a partial meant to be imported elsewhere. See this page under "Partials" and "Import."

Ok, so the _ is a sass thing, so why do we import base.scss then? Does the compiler know to add the underscore to it? I think it’s a typo.

@drewbaker it's not a typo, this is how sass requires you to do it. Partials must be named with a leading underscore, and when imported they must be referred to without the underscore.
Can't say I agree with it, but that's the way it is. As Sander mentioned for more info you can read the import section of this page.

OK that’s what I wanted to know. You have to import without the underscore. Super strange they did that, the import statement is a path that doesn’t exist!


Liveness and Readiness Probes And Spring Boot

If Spring Boot Actuator is not on the class path and the livenessProbe and readinessProbe are not configured why are they added to the deployment yaml?  It can end up in Kubernetes thinking the app is down.

My experience on the subject is that for web applications, its better to have a probe on the http port than nothing.
For example, if you try to run automated tests without a probe at all, its most likely that it will fail, as the pod will appear as ready prematurely.
@ryanjbaxter: wdyt?

@iocanel what does kubernetes do when a path is not specified in the probe?  Does it just hit the root of the app by default?  The root does not necessarily have to return anything.  I would think the functionality should be not added to the Deployment yaml unless specifically configured by the user or if we can detect that 1.) actuator is on the classpath then automatically configure it to hit /actuator/health.

@ryanjbaxter: I think that it hits the root. That's not really a problem.
The problem that I see, is that its possible that the request perfromed has side-effects and could potentially be a source of problems.  So, I think that it should be changed as you suggest.

@iocanel any idea when you will merge those changes into to master?

As soon as I address the CI issues (a day or two).
On Mon, Jun 24, 2019, 23:07 Ryan Baxter<EMAIL_ADDRESS>wrote:

@iocanel https://github.com/iocanel any idea when you will merge those
changes into to master?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ap4k/ap4k/issues/223?email_source=notifications&email_token=AADCEWEZ3MY5N2MTAOEULIDP4ESRVA5CNFSM4HTBDZHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYOCBWQ#issuecomment-505159898,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADCEWD76JG6XPHSKEFUWK3P4ESRVANCNFSM4HTBDZHA
.

Meged, will release soon.


Coulomb subduction playground

Hi Kevin,
here's a framework for testing out coulomb filters for joint ruptures. I've put it in a more or less random package for now since it's all experimental.
Do you have access to our repo? If not we'll have to work out how we can both work on the same branch.

There's a main method in RuptureMerger that opens a rupture set with crustal and subduction ruptures (get it from https://share.gns.cri.nz/123S72ZHG3ZK/rupset-disjointed.zip.html ) and merges ruptures into one particular subduction rupture. There's commented out code to use 50 ruptures instead of just one. But It's rather slow already and creates a lot of ruptures.
There's a RuptureJumpDistFilter that will ensure that ruptures are within 5km of each other.
A MultiRuptureCompatibilityFilter is expected to return a PlausibilityResult as well as a RuptureJump. Not sure if the jump is important. I guess we could just as well take the first section of each rupture and use those for the jump. But in case we need something more specific, the filter can return it. Any filter that you write won't need to return any jump as the RuptureJumpDistFilter already returns one.

Hope this all makes sense and does what you need for now. Not wedded to any of this. Happy to make any changes you need!

Looks like a great start! I won't have time to try it out until the second half of next week, but it looks good. One thought is that you might rename the package from MultiRupture to multiRupture as per the usual convention.

@kevinmilner will you need knowledge about the structure of the subduction ruptures, e.g. a 2DRuptureTreeNavigator?

@kevinmilner will you need knowledge about the structure of the subduction ruptures, e.g. a 2DRuptureTreeNavigator?

Not for the simple filters I'm proposing we start with. We can bite that bullet when/if the time comes (and we hopefully won't have to).

@kevinmilner here's a small rupture set that can be used during development. It comes with a handy report that helps you choose specific rupture combinations. http://nzshm-sundry-reports.s3-website-ap-southeast-2.amazonaws.com/reports/catalogue-6-June-24/

Hi @voj,
I put together a simple MultiRuptureCompatibilityFilter implementation that checks that the that at least X % of interactions from one rupture to the other are positive (i.e., slip on one encourages slip on the other). What's the best way for me to contribute it? Should I make my own branch on opensha that forks from your subduction playground then submit a PR to your branch, or do you want to give me access to your branch directly?
A few thoughts that I had while implementing this:

Not sure that MultiRuptureCompatibilityFilter.apply(...) needs to take the previous result as an argument.
Not sure why MultiRuptureCompatibilityResult has a Jump attached, how do you see that being used? Could probably just use PlausibilityResult.
Might eventually want to make MultiRuptureCompatibilityFilter extend ShortNamed if we end up using multiple of them at a time and want more advanced diagnostics.

Also, do you have the rupset-disjointed.zip file referenced in the main method of RuptureMerger somewhere? It looks like the rupture set you linked to above might only has a few, and none of them pass my initial Coulomb tests. That's because the 2 test subduction ruptures you included are both either partially or fully down-dip of the crustal faults, which is not Coulomb compatible (but the up-dip patches, 2, 3, and 4, are). Here are some SCEC-VDO screenshots showing this; green are the 'source' patches in the Coulomb calculation, and I'm plotting compatibility where red is compatible and blue is incompatible:

I did a test calculation where I built my own subduction ruptures using each unique combination of the uppermost 3 ruptures; I got 256 combined ruptures, here's the resulting rupture set:
mergedRupset.zip

@kevinmilner oh, exciting news! So happy to see you've made progress 🎉

Here's a new link to the rupture set. I just learned that it's automatically deleted from the sharing server after 7 days https://share.gns.cri.nz/04XJHYXQR9GN/rupset-disjointed.zip.html
@chrisbc will give you access to our repo so we should be able to both work on my branch 👍
previous result in MultiRuptureCompatibilityFilter.apply(...): OK, I'll remove it
MultiRuptureCompatibilityResult's Jump: I wasn't sure if it's actually important to know where the jump was taken. I'm more than happy to take this out since a single section-to-section jump might not make sense in this context anyway.
ShortNamed: I'll do that too.

That's surprising to me that those ruptures are incompatible 🤔 I was going off some RSQSim earthquakes like these (not by me) from the same area

Do you think the incompatibility is due to the specific properties of these ruptures or is there a general incompatibility with the subduction rupture down-dip of the crustal rupture?
I'm far from being a  SCEC-VDO power-user. Is this a built-in feature or debugging code you've added specifically for this?

@kevinmilner I've pushed changes incorporating your suggestions

Here's a Coulomb calculation using a bunch of the subduction sections as sources, showing that some of the crustal faults are encouraged and others aren't:

It seems to vary by fault type, maybe liking strike-skip crustal ruptures more. Here's a plot of rake, where red is right-lateral strike-slip, blue is reverse, and yellow is normal:

The Coulomb calculates are a poorly documented feature of SCEC-VDO. I just pushed some updates so you should pull first, but when you load a rupture set into VDO you can choose Color By -> Stiffness. You select your first source subsection by ctrl + clicking on it, and then you can add additional ones by shift + ctrl + clicking on them. It can be a bit sluggish, especially for large rupture sets.
Thanks for the updates and giving me access, I'll  merge my initial ideas in shortly!

OK I just committed my draft Coulomb filters in org.opensha.sha.earthquake.faultSysSolution.ruptures.multiRupture.impl, and updated the main method of RuptureMerger to use them.
MultiRuptureNetCoulombPositiveFilter: This ensures that the ruptures are net positive in a Coulomb sense, summing all interactions from the source rupture to the receiver rupture and testing that the sum is >0. It can either be specified to require that either crustal -> subduction or subduction -> crustal is positive (Directionality.EITHER), or to require that both are positive (Directionality.BOTH). This type of filter weights nearby interactions greatest.
MultiRuptureFractCoulombPositiveFilter: This ensures that at least threshold fraction of interactions between the source rupture and the receiver rupture are positive (regardless of distance or strength of that interaction).
It first does a pre-cache of all coulomb calculations in parallel to speed things up because the rupture building loop is serial as currently implemented when only using a single nucleation rupture. It stores that cache in /tmp and will reuse it. It may take a little while to calculate the first time!
I do find that it still prefers to include corupture where the crustal fault is near the top of the subduction rupture, but that's not always true, e.g.:

Here are some test rupture sets using the single subduction rupture and the full set of crustal ruptures:
Fraction interactions >= 0.75:
mergedRupset_5km_cff0.75IntsPos_from97653.zip
Net Coulomb positive:
mergedRupset_5km_cffNetPositiveBOTH_from97653.zip
Both combined:
mergedRupset_5km_cff0.75IntsPos_cffNetPositiveBOTH_from97653.zip
I'm trying to build a full rupture set (using all possible subduction and crustal ruptures), we'll see how it goes.

@kevinmilner that's awesome! I've been looking at it today. So many things to play with! I've been able to run a rupture build, it really does take a while to build that cache. Let me know how long it takes you to build a cache for everything!

In SCEC VDO, is there a way to set the focal point so that it's easier to use for NZ? The help document is not reachable, and I've found were the default focal point is hard coded. But I'm hoping for an easier way than hacking that in :-)

Are the parameters that go into the stiffness calculation unlikely to change? I.e. will we really only have to calculate that cache once?

SCEC VDO, is there a way to set the focal point so that it's easier to use for NZ? The help document is not reachable, and I've found were the default focal point is hard coded. But I'm hoping for an easier way than hacking that in :-)

@voj I think I can help with this .... I have some notes somewhere on using scecVDO with an NZ configuration

@kevinmilner that's awesome! I've been looking at it today. So many things to play with! I've been able to run a rupture build, it really does take a while to build that cache. Let me know how long it takes you to build a cache for everything!

I downsampled to 2km patches and built a full rupture set in about 4.5 hours...it has 26 million ruptures!!! So we'll clearly have to build better filters.

* In SCEC VDO, is there a way to set the focal point so that it's easier to use for NZ? The help document is not reachable, and I've found were the default focal point is hard coded. But I'm hoping for an easier way than hacking that in :-)

The simplest way is to go the "Graticule" tab, click the drop down where you see "California" and select "NewZealand" (which I just added in my commits early today). I usually then go to "Political Boundaries" and enable "Oceania" to get NZ (and a bunch more), or you can double click on Oceania and enable NZ specifically.

* Are the parameters that go into the stiffness calculation unlikely to change? I.e. will we really only have to calculate that cache once?

We never changed them in CA (except for some early sensitivity tests). It's possible that other parameters would be better for this style of rupture, but I would say we can treat them as unlikely to change for now. Even if we do change them, we're unlikely to change it frequently.

* I can't seem to make `Color By -> Stiffness` work. When I select it it's all greyed out, which I assume is what it's supposed to look like. But ctrl+click doesn't on a fault section doesn't seem to have nay effect.

Had you loaded a rupture set first? That's the first step, then click the "Visible" checkmark next to it in the table below the browse button, then you can start coloring and such. All faults will initially be gray when you select stiffness, but you should be able to ctrl+click then.

@kevinmilner @chrisbc Thanks, the "Graticule" tab was what I needed to change the focal point 👍
And I finally have the stiffness visualisation working. It seems ctrl+ click doesn't do anything, but shift+ctrl+click does work. Maybe it's a Windows thing?
4.5 hours is amazing! And the result of that is probably the best joint rupture set we have yet 🤩 Is that 4.5 hours including building the stiffness cache? And going to a 1k resolution would take about 4 times as long?
26 million ruptures is a much, much better result than I had thought! The theoretical worst case of rupture combinations at 5km max distance is in the billions. In the meantime, I've looked into thinning. I might be able to give you a rupture set with fewer subduction and crustal ruptures tomorrow. That might bring the result into more manageable territory. With any luck I won't have messed up the fault sections list and the stiffness cache will be reusable.
Oh, this is very exciting!

It seems ctrl+ click doesn't do anything, but shift+ctrl+click does work. Maybe it's a Windows thing?

Oops, I misremembered the key combos! It's shift + click for the first selection, ctrl + shift + click for additional. But with none selected, adding ctrl doesn't do any harm. So that explains it!
The 4.5 hours was for both caching and building the rupture set. When doing the whole set I don't precache as the rupture set is built in parallel anyway and will thus do the caching work in parallel. I would expect that a 1km run would take longer but not 4x. I think 2km spacing is totally defensible for this sort of run though, especially during the prototyping phase.
I just pushed a commit with some tweaks for full calculation. I also posted my rupture set and 2km cache file here in case you want to take a look or rerun reusing my cache. Note that I'm running this on my absolute beast of a machine (64 thread ryzen threadripper 7975) so it might take significantly longer on a laptop even if using my cache file.

Hi @kevinmilner
I've had a closer look at the rupture set I gave you. It includes all sections but for some reasons not all of our ruptures. I've created a new rupture set that includes all ruptures (download it here). It'll be better to use this new rupture set. The fault sections are identical apart from the parent ids of the subduction sections. The stiffness cache you created with the 4.5 hour run still works for this rupture set, but it doesn't seem to be quite complete for these rupture combinations.
I have also created lists of rupture ids that can be used to select subsets of this new file. There's new code in RuptureMerger that makes use of these lists (download them here)

filteredRuptures.txt heavily thinned subduction ruptures (ruptures need to be 10% larger than the previous rupture size, ruptures of the same size should not overlap) and crustal ruptures (max cumulative azimuth change 10deg, max cumulative rake change 10deg)
filteredRuptures-allCrustal.txt heavily thinned subduction ruptures, all crustal ruptures
filteredRuptures-straightCrustal.txt heavily thinned subduction ruptures, crustal ruptures have max cum azimuth change of 10deg
filteredRuptures-uniformRakeCrustal.txt heavily thinned subduction ruptures, crustal ruptures have max cum rake change of 10deg

filteredRuptures.txt cuts the source ruptures right down. With this file, I can run RuptureMerger in just over 2 minutes on my 6-core machine.

@kevinmilner I've generated a basic rupture report to get some data on how well the approach is working. You can find one that is based on the strongest thinning here.
The code is on this branch. In order to make it work, I needed to slightly change how we create splays in order to not trip up the ClusterRuptures module. Old rupture sets need to be re-generated as they would crash ClusterRuptures.
I've also stuffed the stiffness cache as a module into the rupture set to make sure we use the correct data for the report. We'll have to see how practical this is in the long run. I see this as a tool to use while we experiment.
Here is one interesting example of a rupture. It happens to be the rupture with the longest number of sections.
A couple of properties strike me as odd with this one. Not sure if this is due to my ignorance or if there's something there:

This rupture was accepted because stiffness was above the threshold when using crustal as the source. Does this mean that we're saying that this enormous subduction fault was triggered by the relatively small crustal rupture?
There's interaction between the crustal rupture and subduction sections in the far north, hundreds of kilometers away. Is this realistic?

I'm thinking we could have a distance-cutoff for section-to-section stiffness. Then we'd also make it impossible for this crustal rupture to trigger the whole subduction fault.

@kevinmilner to try out my suggestion above ☝ to limit coulomb at a distance, I've created a new report that explores stiffness with different distance cut-offs http://nzshm-sundry-reports.s3-website-ap-southeast-2.amazonaws.com/reports/stiffness-20-June-24/. Here's the new version of the interesting example with a bunch of plots at different distance cut-offs at the bottom (subduction as the source in the top rows, crustal in the bottom rows). Note how this time, none of them reaches the threshold.

Great work, @voj! I'm just coming back up to speed.
It looks like you're plotting the fraction of interactions that are positive in your stiffness reports. It all comes down to what summary statistic you take from the set of all patch-to-patch calculation, and this is where you're setting it up to use the fraction positive.
The fraction positive doesn't take distance into account at all, it just registers if something is positive or negative, so that is why you see such strong far-field compatibility in your example plots. I think we could make the case that we want both crustal -> subduction and subduction -> crustal to be strongly positive, so maybe you should add a Dicretionality.BOTH to the call to the MultiRuptureFractCoulombPositiveFilter constructor (as I already had in there for MultiRuptureNetCoulombPositiveFilter.
Speaking of MultiRuptureNetCoulombPositiveFilter, that differs in that it instead sums all Coulomb interactions, thus weighting stronger near-field ones greater than weaker far-field interactions. That filter is supposed to be already doing basically what you were trying to do with the distance cutoff.
I like using a combination of these two filters: are the near-field interactions net positive (MultiRuptureNetCoulombPositiveFilter), and is the whole thing also compatible including far-field interactions (MultiRuptureFractCoulombPositiveFilter)?

@kevinmilner yes, I didn't really explain what I was plotting 😅 I set up the AggregatedStiffnessCalculator the same way MultiRuptureFractCoulombPositiveFilter does to have a closer look at the 0.75 threshold.
I set the directionality to BOTH for MultiRuptureFractCoulombPositiveFilter as you suggested and ran a new report. Now the largest rupture by section count is this one http://nzshm-sundry-reports.s3-website-ap-southeast-2.amazonaws.com/reports/stiffness-21-June-24/resources/jointStiffness9842/index.html
The extra plots where stiffness is limited by distance seem to show that we have to go quite far until the rupture is above the 0.75 threshold.
Reading your comments and talking to Chris here shows to me that I still don't quite understand the stiffness values well enough to properly interpret this. I hope we can clear all that up when we meet next week!

@voj oh cool, I better understand what you were getting at with the distances yet. Maybe some sort of intermediate option to balance the near and far field would be to specify the threshold as a function of distance?

@kevinmilner yes, I was thinking of something like that. I'll whip up something and see how it looks.


Compatibility View Settings in IE

Comment from a user: "We at [...] were struggling because we couldn't collapse the VLM.  We finally discovered that it was because in the Compatibility View Settings option in the IE Tools menu, "Display intranet sites in Compatibility View" option was ticked.  When I unticked that, then collapse/expand worked.  After some research, I discovered that I didn't have to worry about that option if I inserted a new Meta tag - .  I added this line to the new stylesheet and didn't have the problem any more."

Is this related: IE8 won't load JavaScript file in “Compatibility View.” ?
I checked define.xml + stylesheet with Inspector in Firefox 84.0.1 (64-bit), and on initial load, there are 4 warnings:
1.Unknown property ‘-moz-border-radius’.  Declaration dropped.
2. Unknown property ‘-moz-box-shadow’.  Declaration dropped.
3. Error in parsing value for ‘padding-left’.  Declaration dropped.
    Elements matching selector: ul.SubClass
4. Expected declaration but found ‘#000000’.  Skipped to next declaration.
    Elements matching selector: .arm-summary-resultdisplay

However, the expand/collapse vlm buttons work as expected.
Would it be better to have only one button to toggle "expand/collapse vlm"?

Is this related: IE8 won't load JavaScript file in “Compatibility View.” ?
I checked define.xml + stylesheet with Inspector in Firefox 84.0.1 (64-bit), and on initial load, there are 4 warnings:
1.Unknown property ‘-moz-border-radius’.  Declaration dropped.
2. Unknown property ‘-moz-box-shadow’.  Declaration dropped.
3. Error in parsing value for ‘padding-left’.  Declaration dropped.
    Elements matching selector: ul.SubClass
4. Expected declaration but found ‘#000000’.  Skipped to next declaration.
    Elements matching selector: .arm-summary-resultdisplay

However, the expand/collapse vlm buttons work as expected.
Would it be better to have only one button to toggle "expand/collapse vlm"?

One button would not solve the issue?

One button would not solve the issue?


Address not Parsed Properly

I tried parsing this address using the web interface.  The address was found here:
http://www.tdcj.texas.gov/unit_directory/unit_information.html
P.O. Box 48, Substation #1; Galveston, TX 77555
The semicolon appears to be what they use for a newline delimitter in the table

what would the correct parsing be?

Probably Substation should be the SubaddressType, but I'm not really sure.  Right now it throws the Substation in PlaceName.  It also puts Galveston in the PlaceName and this causes an exception.
For my purposes, I don't know what the substation is for when delivering mail so I just removed it from the string before giving it to the parser.

Added this into a bunch of new training data for PO Boxes. Thanks @ckeeney!


Prepare submission material

Submission checklist

[x] 2 Icons (for Ledger device and Ledger Manager)
[x] Companion app (binaries or package, for Windows/MacOS/Linux)
[x] Contact information
[x] Link to tutorial hosted on third party website
[x] Video of your application running on the Ledger device

Details in: https://docs.google.com/document/d/1Rgn_myiYeftg9zUAfb5veJuPBoHH0eq3HS4YGyGCc5k/edit?usp=sharing

Ledger app was submitted on 20 April 2021


Introspect environment variable from Dockerfile

Is your feature request related to a problem? Please describe.
Setting up a new template for a container requires copy/pasting all environment variables already defined using Dockerfile ARG/ENV directive.
Describe the solution you'd like
All variables defined in the Dockerfile could show up in the Configuration section with their current value.
Or alternatively; in the array list having an option to source from current ENV value defined in Dockerfile.
Describe alternatives you've considered
Manually copy all ARG/ENV directives from Dockerfile to the templates.json definition.
Additional context
Dockerfile:
ARG DEBUG=*,-nodemon,-nodemon:*,-snapdragon:*,-engine,-engine:*,-socket.io:*,-express:*,-socket.io-parser,-compression,-body-parser:*,-send,-mquery
ENV DEBUG=$DEBUG

templates.json:
    "env": [
      {
        "name": "DEBUG",
        "label": "DEBUG",
        "default": "*,-nodemon,-nodemon:*,-snapdragon:*,-engine,-engine:*,-socket.io:*,-express:*,-socket.io-parser,-compression,-body-parser:*,-send,-mquery"
      },

Have you looked at creating templates by calling the Portainer API? I do this to create templates using a simple Powershell script.

Thanks for the tip Nigel. I might take a look at that later! although if I do that, maybe it is also as easy as using the API from within the app so I should look to see if I can change it.

I will close this, as custom templates were changed in the upcoming 2.0 release via #3861 and there is now no longer a way to create custom container templates therefore no container template environment variables


New LDS-02 problem with measures count

ISSUE TEMPLATE ver. 0.4.0

Which TurtleBot3 platform do you use?

[ X] Burger
[ ] Waffle
[ ] Waffle Pi

Which ROS is working with TurtleBot3?

[ ] ROS 1 Kinetic Kame
[ ] ROS 1 Melodic Morenia
[ X] ROS 1 Noetic Ninjemys
[ ] ROS 2 Dashing Diademata
[ ] ROS 2 Eloquent Elusor
[ ] ROS 2 Foxy Fitzroy
[ ] etc (Please specify your ROS Version here)

Which SBC(Single Board Computer) is working on TurtleBot3?

[ ] Intel Joule 570x
[ ] Raspberry Pi 3B+
[ X] Raspberry Pi 4
[ ] etc (Please specify your SBC here)

Which OS you installed on SBC?

[ X] Raspbian distributed by ROBOTIS
[ ] Ubuntu MATE (16.04/18.04/20.04)
[ ] Ubuntu preinstalled server (18.04/20.04)
[ ] etc (Please specify your OS here)

Which OS you installed on Remote PC?

[ ] Ubuntu 16.04 LTS (Xenial Xerus)
[ ] Ubuntu 18.04 LTS (Bionic Beaver)
[ X] Ubuntu 20.04 LTS (Focal Fossa)
[ ] Windows 10
[ ] MAC OS X (Specify version)
[ ] etc (Please specify your OS here)

Specify the software and firmware version(Can be found from Bringup messages)

Software version: [x.x.x]
Firmware version: [x.x.x]

Specify the commands or instructions to reproduce the issue.

HERE

Copy and Paste the error messages on terminal.

Traceback (most recent call last):
  File "/opt/ros/noetic/lib/python3/dist-packages/rospy/topics.py", line 750, in _invoke_callback
    cb(msg)
  File "./real_exp.py", line 594, in laser_scan_callback
    new_blocks, LL, CC, x, y = self.obstacles_parser.get_obstacles(np.array(dt.ranges), fillna='else', state=self.new_state)
  File "./real_exp.py", line 280, in get_obstacles
    degrees = degrees[~nans]
IndexError: boolean index did not match indexed array along dimension 0; dimension is 360 but corresponding boolean dimension is 244

Please describe the issue in detail.

Hi there!
Thank you so much for the support you give to the users. It's time for me to reach out to you.
I have worked with TB3 before with the LDS01 version of the laser rangefinder and everything was fine.
We bought new robots and had some problems with the new version of the laser rangefinder. The thing is that every time (measure) a different number of measurements (arrays lenght) come in, according to the documentation the resolution = 1 deg.
So, we have three TB3s with the old lidar and five pieces TB3 with the new ones.
On the old one it works, on the new one it doesn't.
The drivers have been updated, the bashrc file has been updated according to the tutorial.
Nevertheless, the problem remains.
Is it normal now that the number of measurements (leght of the array) is different each time or not?
In total we have 3 TB3 with the old Lidar and 7 TB3 with the new Lidar. The tests were done on all of them.
But, with old lidar it works fine, with new one not.
Hope for your help.
Kind regards,
Dmitrii
bringup_msg.txt

@ROBOTIS-Will
Hi there,
Do you have any ideas?

Hi @dimadobriy
I'm sorry about the delayed response.
The LDS-02 sensor has a fixed sampling frequency(2300Hz), the faster RPM will decrease the resolution (higher than 1°) which will reduce the size of the array data in each rotation.
Unfortunately, the motor of the LDS-02 does not have the PWM controller which makes it difficult to control the RPM.
Thank you.

Hi @ROBOTIS-Will
Thanks for your reply.
I didn't modify RPM. So it seems it should by default.
However, each rotation I receive different size of the array.
See attach. Prev execution could be another, say, 240-241.

Hi @dimadobriy
Since the USB2LDS does not have a feedback controller to control the RPM of the motor of the LDS-02, the sensor will rotate with whatever the duty cycle it is provided via PWM#1 of the USB2LDS.
While the motor is rotating, the sensor will collect data with 2300Hz therefore, depending the RPM of the motor, the size of the data array could vary.
The USB2LDS is configured to generate 70% duty cycle to the motor at PWM#1 pin.
In order to reduce the duty cycle to lower the RPM (which will improve the resolution and increase the size of the array), you need to adjust resistors(R16, R17).
For example, replacing the 3.9Kohm R16 to 1.5Kohm will reduce the duty cycle from 70% to 60%.
The equation of the output duty cycle is : 1 - { R17 / (R16 + 2*R17) }
However, please be aware that this modification will void your product warranty so please perform this at your own risk.
Thank you.

Hi @ROBOTIS-Will
Thank you for your reply. I really appreciate it.
Do I understand correctly that in this case the resolution of 1 degree is not respected?
I made some adjustments in my code. By correcting the resolution I solved the problem.
Thank you for your patience.

@dimadobriy
Yes, unless you have a feedback controller to control the PWM signal, a exact 1° resolution may be hard to achieve.
In case the RPM gets lower than 383, you will get more than 360 data per each rotation.
Thank you!

@dimadobriy
How do you solve this problem??
I have same problem.


Change to leave all logs

Now we leave log only last 1MB about first process.
So we very hard to find error, when occur to little error.
We should be able to leave all log to optional.
@junoyoon How think about this?

OK. Add this configuration on 3.4.1


Create a canonical Arcs on-disk Repo structure

As part of the ingest design it's become clear that we need to have a well-documented and consistent structure for on-disk artifacts when running Arcs in a server environment.
The pain points come from:

Current server is in-memory and doesn't everything goes away on restart. (Makes testing hard)
On disk support is available in cloud deployment, but it is not being tested.
We need to really think about versioning and how the system as a whole can work well with new versions  of Runtime.

To that end I propose that we support an on-disk format for the Arcs Repo, and a set of commands to manage it.  To make things easy for developers and administrators we should consider mimicing existing repo systems like git.
The on disk repo might look something like this:
   .arcs/
      config.json
          {"version": 1,
            ...
          }
      db/

Looks like an early duplicate of https://github.com/PolymerLabs/arcs/issues/2810


Dio requests tracking

Hi I would like to learn how to track dio requests

Because we track at the dart:io level datadog_tracking_http_client will track dio requests just fine.
If you're having any issues with it, please reopen with more details about what problems you're having.


it works, but when i go fullscreen webpage becomes invisible

thanks for this project, please can you help? my site has not been uploaded to domain. still coding in xampp localhost.. have you ever seen this problem.. what you advise me?? when i click fullscreen it works.. my site becomes invisible too.. when exit full screen page becomes visible again.. thanks

Hello.
I have problem using this script.
When i go to full screen some styles and Scroll not working.
Please help.


Downloading Buildpack failing

I'm trying to push this app and revision. This is what I'm getting:
Counting objects: 866, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (721/721), done.
Writing objects: 100% (866/866), 163.04 KiB | 0 bytes/s, done.
Total 866 (delta 520), reused 244 (delta 98)
-----> Building flowcells...
-----> Multipack app detected
       =====> Downloading Buildpack: https://github.com/heroku/heroku-buildpack-apt.git
remote: ERROR: Build failed: exec: job exited with status 1
To https://git.cubipaas.bihealth.org/flowcells.git
 ! [remote rejected] HEAD -> master (pre-receive hook declined)

I'm running a fresh install of flynn v20160918.1.
Debug information is available here: https://gist.github.com/a5e7921bb4a3873ff3bd89aa5611dab0
What can I do?

It looks like you have pulled in the apt buildpack, but don't have an Aptfile, which could cause that error. Try again after committing an Aptfile.


Roll Engine from b09cf532697b to d25b3a60e521 (3 revisions)

https://github.com/flutter/engine/compare/b09cf532697b...d25b3a60e521
2022-02-16<EMAIL_ADDRESS>Add a CSV output option to displaylist_benchmark_parser.py and add a README detailing how to run and process benchmarks on iOS/Android (flutter/engine#31481)
2022-02-16<EMAIL_ADDRESS>Revert mark idle (flutter/engine#31487)
2022-02-16<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from fbvo-hhGc... to EyfkLwBRC... (flutter/engine#31488)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from fbvo-hhGcTFQ to EyfkLwBRCpp8
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.