unable to open some images due to unsupported color space

Some jpg images can't be open by the iiif server, for instance accessing
https://iiif.bdrc.io/bdr:I1KG15427::I1KG154280003.jpg/full/,600/0/default.jpg
gives
javax.imageio.IIOException: Incompatible color conversion
	at com.sun.imageio.plugins.jpeg.JPEGImageReader.checkColorConversion(JPEGImageReader.java:997)
	at com.sun.imageio.plugins.jpeg.JPEGImageReader.readInternal(JPEGImageReader.java:1102)
	at com.sun.imageio.plugins.jpeg.JPEGImageReader.read(JPEGImageReader.java:1058)
	at io.bdrc.iiif.image.service.ReadImageProcess.readImage(ReadImageProcess.java:238)

Note that other Java jpg readers also fail, for instance:
java.io.IOException: de.digitalcollections.turbojpeg.TurboJpegException: Unsupported color conversion request

The original image is the following:

The problem seems to be that the color space is CMYK which is not handled by Java... There's a tutorial that we could use to sort of hack a solution... but it looks quite complex and I'm not sure we want to use that (it does some heavy conversion and will take a lot of processing time). Perhaps it would be best to just make sure that all the web images are not CMYK...
@jimk-bdrc I think one easy way to handle that would be to add
"cs": "cmyk"
in the dimensions.json when the color space is not rgb? That way I can just skip these images on BUDA...

@eroux editing the dimensions.json will be a good stop-gap solution for now.  the better fix for existing works will be to audit the archive for CMYK files and convert them to RGB.  for incoming works we should add a test to Audit Tool to check for CMYK files and raise an error when they are found.

sounds good yes!

Yes to adding it to audit tool and volume-manifest-builder. Who's volunteering to audit the web image library?  Note that we have the whole image library here in web format ( with the internal buckets) on xttvY:~j.../prod/archive-ops-118/archive.tbrc.org/Works and also in //TBRCRS5/NetBackup

well, I'm happy to help but I'm not sure I'm part of the call to volunteers :) But this works for one image:
$ identify -format '%[colorspace]' /tmp/I1KG154280003.jpg
CMYK

this could be integrated in a script that goes through all the images... maybe there are some scripts already doing this kind of things that could be used as templates?

@eroux yes, that's the correct imagemagick incantation to find these CMYK files.  something like this can be used to convert to RGB:
mogrify -profile  /Library/Application\ Support/Adobe/Color/Profiles/Recommended/AdobeRGB1998.icc -colorspace sRGB /tmp/I1KG154280003.jpg
not a hard script to write but it'll take quite alot of time to run "identify" across millions of web images i think

sure, and to resync them once this is done too... what is your estimation of how much time does this kind of things take in general? I guess IO is the bottleneck?

sure, and to resync them once this is done too

this goes without saying.  :)

what is your estimation of how much time does this kind of things take in general? I guess IO is the bottleneck?

most definitely bound by io.  we'd have to do some testing to know for sure my guess is that it takes a few seconds to get the colorspace information using imagemagick on a conventional hard drive.  if you were to multiply those few seconds by millions of images this would very quickly become a tedious process.    we could likely be sped up by executing the process on a ssd or otherwise having the files "in memory" somehow.
now that we're talking about this I wonder if running this through a Java or Python api would be more efficient.

A few seconds?? On my machine with an SSD drive:
time identify -format '%[colorspace]' /tmp/I1KG154280003.jpg
CMYK
real	0m0,067s

if we multiply that by 20M we get 6 months... so that seems way too much... so a Python script would be better yes.

@eroux any possibility of BUDA fixing these in realtime on the iiif server?  catching the error, converting the image to a proper color space on-the-fly, and then presenting the newly updated image.

I'll gice it a try next week but unfortunately the Java API us pretty
bad...
Le jeu. 6 août 2020 à 18:35, Travis<EMAIL_ADDRESS>a écrit :

@eroux https://github.com/eroux any possibility of BUDA fixing these in
realtime on the iiif server? catching the error, converting the image to a
proper color space on-the-fly, and then presenting the newly updated image.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/buda-base/buda-iiif-server/issues/57#issuecomment-670039028,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAO3REB4QMHSNCKXJ34NR3R7LLTHANCNFSM4PWLH7AQ
.

sounds good.  I think that would be ideal.  making this tweak to the iiif server and a fix to audit tool would be the most effective solution.

There may be more to this issue than a Java API:
Stack Overflow PIL CMYK to RGB There's balancing and adjusting.

@eroux writes

A few seconds?? On my machine with an SSD drive:
time identify -format '%[colorspace]' /tmp/I1KG154280003.jpg
CMYK
real	0m0,067s

if we multiply that by 20M we get 6 months... so that seems way too much... so a Python script would be better yes.

Of course, most of that time was spent launching the executable, loading the 10MB of image processing library & so forth. So even doing this from a file list would be a huge savings.
Prboably the best architecture is to write one scan script to read, emit all the CMYK files, and then pass their filenames into a python CMYK converter.
This could all run on our processing servers, bodhi and sattva.
I have to rewrite v-m-b to read from disk, so if I write a short python routine to asynchronously scan our disk library for CMYK images.

Converting IIIF images on the fly sounds slow and risky - Java APIs don't always work as expected (@eroux said recently they've sworn off Java for image processing, gone over to PIL).  There's a risk that any Java could be a programming black hole, and would most def slow the experience down.
Also, when you convert at that late stage, we lose (or someone would have to write) a signal that the image group needs converting. There gets to be digression between the source bits (which AO manages) and the display bits.
It's cleaner to have AO fix things once by detecting before processing (audit tool) and converting using Python.
That way, the same tool can be used for the backlog and for ongoing.

Thanks @jimk-bdrc.  Excellent assessment here and outline for a path forward.  I hadn't realized Java was so clunky.  If the Python API performs well then that certainly seems the way to go.  I also agree with your reuse strategy - that we create a common library of code to pull from for both detection and conversion.

I've just made a small test with a few readers, and it seems the TwelveMonkeys reader works well, it converts the image into sRGB so that it can then be handled by Java. So we should implement a failover to that reader on the server. It's not ideal as it requires some processing, changes the quality, etc. but at least it will work. I suspect these CMYK images are not too common and would appear only once or twice per volume... so I can live with that. @MarcAgate can you implement the failover to the TwelveMonkeys reader when this exception is raised?

Done as of commit d66844d
See
https://iiif.bdrc.io/bdr:I1KG15427::I1KG154280003.jpg/full/,600/0/default.jpg for cmyk image
and
https://iiif.bdrc.io/bdr:V4CZ5369_I1KG9127::I1KG91270034.jpg/full/1250,/0/default.jpg for normal image

Can we close this for now ?


Accessors are not working

Describe the bug
I have a model named Audit and it has a auditable_type field which is multi field with types keyword and text .
I tried to use accessors to format the retrevived auditable_type and realized that accessors are not working . even accessor function did not called .
Audit.php
<?php

...

class Audit extends Model
{
    ...

    public function getAuditableTypeAttribute(string $value)
    {
        return collect(array_filter(explode('\\', $value), function ($item) {
            return $item != null && $item != '';
        }))->last();
    }

}

AuditRepository.php
$column = "auditable_type.keyword";
Audit::distinct($column)->get($column)->pluck([$column]); // we expect that accessor format the field .

To Reproduce
Create a model using laravel-elasticsearch package and try to implement an accessor for a text field .
Expected behavior
We Expect that model accessor format the desired field correctly .

Hi, is hard to see what you're trying to do, but I can spot a few issues with what you have shared.
Your getAuditableTypeAttribute() accessor is faulty. Accessor methods do not have arguments, like (string $value). They work as attributes of a current record (or an instantiated model class):
  public function getFullNameAttribute()
  {
      return $this->first_name.' '.$this->last_name;   // returns $user->full_name
  }

Secondly, you can't perform database actions on accessors since these columns do not exist at a database level. They are handled by Laravel further up the stack. Getters create attributes from existing records, and setters manipulate fields before saving.
If you try to find, sort, etc on an accessor-defined field with MySQL you will run into the same limitations.

About first tip  :
based on the laravel@8 documentation :
<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Model;

class User extends Model
{
    /**
     * Get the user's first name.
     *
     * @param  string  $value
     * @return string
     */
    public function getFirstNameAttribute($value)
    {
        return ucfirst($value);
    }
}

As you can see, the original value of the column is passed to the accessor, allowing you to manipulate and return the value

And about second tip :
I have been mentioned that auditable_type is a text field on my elasticsearch instance and therefore it exists at databse level .

Anyway I guess the problem is about useing .keyword because of that auditable_type is multi field .

I see, I was mistaken on both.
I did run a simple test on my end and it should work, this is what I did:
public function getCodeAttribute(int $value)
{
    return $value + 1000000;
}
// code is a field with a possible value of 1 - 5

UserLog::distinct('code')->get('code'); returned:

UserLog::distinct('code')->get('code')->pluck(['code']); produced:

The .keyword would throw it off for sure, what happens if you omit it? ie:
$column = "auditable_type";
Audit::distinct($column)->get($column)->pluck([$column]);

Can you try re-index by having the field as a keyword first, then (assuming you want full-text search for this field) as text
$index->keyword('auditable_type');

..then:
$index->text('auditable_type');

Then the operation should use the keyword type as the default and you can omit .keyword
See: https://github.com/pdphilip/laravel-elasticsearch?tab=readme-ov-file#migrations
Let me know

I have been re-index with your explained changes and sorting and filtering have been fixed . but in my search i have to use .text extenstion for this field .

I'm using laravel@8 with<EMAIL_ADDRESS>.
Thanks for your attention .

@alirezaImani-f4L3e the latest version v2.8.7 fixes this. You'll be able to execute distinct queries with the keyword flag, and the mutators will parse it correctly:
$column = "auditable_type.keyword";
Audit::distinct()->get($column);


License

Hi,
Thank you for releasing this awesome project!
May I ask what the license is?
Thank you!

This is a demo for git http server, you can freely modify and use it if you want.

Thanks! Does this mean it is CC0 licensed?

Thanks! Does this mean it is CC0 licensed?

Yes

Thank you! Opened #2 for clarity


Get-SafeLinksPolicy | Format-Table Name,IsEnabled

When I use the new version of the module, IsEnabled is no longer working.
PS C:\Temp> Get-SafeLinksPolicy | Format-Table Name,IsEnabled,IsDefault
Name                                         IsEnabled IsDefault

Customer_Safe_Links
Safe Links
Built-In Protection Policy
Strict Preset Security Policy1637335373227
Standard Preset Security Policy1637335358918

@MWPatterson2000 Thank you for your feedback. I was able to reproduce the issue. Isenabled and isdefault are no longer available. Could you please point me to the document with this information I'll be happy to modify it.
Thanks
Sri

@msbemba Thank you very much for the contribution and digging deep into it.  @MWPatterson2000 We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if you see an improvement is required in the specific area of this documentation. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

Here is the document, I was away with limited to no internet access for the past 8 days
https://docs.microsoft.com/en-us/powershell/module/exchange/get-safelinkspolicy?view=exchange-ps

@MWPatterson2000 Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/office-docs-powershell/pull/9501
Once the author approves, the changes will be live.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @MWPatterson2000 Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


jellyfish version?

INSTALL
pangenie]$ cd build/
build]$ cmake ..
-- The C compiler identification is GNU 9.3.0
-- The CXX compiler identification is GNU 9.3.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /xxx/software/miniconda3/bin/x86_64-conda-linux-gnu-cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /xxx/software/miniconda3/bin/x86_64-conda-linux-gnu-c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Checking for module 'jellyfish-2.0'
--   Found jellyfish-2.0, version 2.2.10
-- /Parastor300s_G30S/zhangjj/software/miniconda3/envs/pangenie/include/jellyfish-2.2.10
-- /parastor300/work01/zhangjj/software/PanGenie/pangenie/build/src
-- Configuring done
-- Generating done
-- Build files have been written to: /parastor300/work01/zhangjj/software/PanGenie/pangenie/build
build]$ make
Scanning dependencies of target PanGenieLib
[  1%] Building CXX object src/CMakeFiles/PanGenieLib.dir/emissionprobabilitycomputer.cpp.o
[  2%] Building CXX object src/CMakeFiles/PanGenieLib.dir/copynumber.cpp.o
[  4%] Building CXX object src/CMakeFiles/PanGenieLib.dir/commandlineparser.cpp.o
[  5%] Building CXX object src/CMakeFiles/PanGenieLib.dir/columnindexer.cpp.o
[  7%] Building CXX object src/CMakeFiles/PanGenieLib.dir/dnasequence.cpp.o
[  8%] Building CXX object src/CMakeFiles/PanGenieLib.dir/fastareader.cpp.o
[ 10%] Building CXX object src/CMakeFiles/PanGenieLib.dir/genotypingresult.cpp.o
[ 11%] Building CXX object src/CMakeFiles/PanGenieLib.dir/histogram.cpp.o
[ 13%] Building CXX object src/CMakeFiles/PanGenieLib.dir/hmm.cpp.o
。。。
[ 38%] Linking CXX executable PanGenie-graph
/xxx/software/miniconda3/bin/../lib/gcc/x86_64-conda-linux-gnu/9.3.0/../../../../x86_64-conda-linux-gnu/bin/ld: cannot find -ljellyfish-2.0
collect2: error: ld returned 1 exit status
make[2]: *** [src/CMakeFiles/PanGenie-graph.dir/build.make:104: src/PanGenie-graph] Error 1
make[1]: *** [CMakeFiles/Makefile2:166: src/CMakeFiles/PanGenie-graph.dir/all] Error 2
make: *** [Makefile:114: all] Error 2
build]$ jellyfish --version
jellyfish 2.2.10
Does the version of Jellyfish have to be 2.0?

No, version 2.2.10 should be fine. Are you installing PanGenie inside of a conda environment, or without conda?


Confusing behavior of deconcatenate method with both new_column_names and autoname argument?

Brief Description

I first realized this because mypy is complaining of the type of new_column_name :
janitor/functions.py:1103: error: Argument 1 to "len" has incompatible type "Union[List[str], Tuple[str], None]"; expected "Sized"
janitor/functions.py:1116: error: Argument 1 to "enumerate" has incompatible type "Union[List[str], Tuple[str], None]"; expected "Iterable[str]"

This should have been resolved by the following signature modification new_column_names: Optional[Sequence[str]] however mypy still complains
janitor/functions.py:1104: error: Argument 1 to "len" has incompatible type "Optional[Sequence[str]]"; expected "Sized"
janitor/functions.py:1117: error: Argument 1 to "enumerate" has incompatible type "Optional[Sequence[str]]"; expected "Iterable[str]"

So it is basically the Optional that is giving trouble to mypy, because we don't ostensibly check of
if new_column_names is None:

I think this is because we authorize the presence of both non None arguments new_column_names and autoname even though in this case  new_column_names is actually ignored and redefined using autoname. (looking at the last test test_deconcatenate_column_autoname in tests/functions/test_deconcatenate_column.py).
I believe that the behavior of the method in this case is not necessarily obvious in this case for the user, so

should we raise an exception (like only one of the two arguments at a time)?
should we keep this behavior but change the code to trick mypy? (I think that using an additional typed argument to avoid redefining new_column_names might work)
should we just silence mypy for this particular case?

Type checkers are pretty rad, forcing us to clarify the behaviour of functions! I like where this issue is going: semantics and language matter more than we might think!
So in terms of how to go forward, let me see.
I think the simplest and also most principled way forward is what you've suggested: raise an exception.
@zbarry and @hectormz, I think both of you have had the most experience with type checkers on the team. If you have a moment, would you be able to chime in here? No pressure, btw, if you're busy, feel free to ignore.

@ericmjl @VPerrollaz Checking it out now and will see if I have anything worthwhile contribute!

Just to let you know I "implemented" solution 2 (keeping the behavior and adding an internal typed argument) in the branch mypy_configuration of my fork.
But that is only for "demonstration" purposes and not because I think it is the best idea.

@hectormz @VPerrollaz Any updates on this?


Display LCC as an extra "module"?

Related: #974

The Languages, Countries, and Codes (LCC) standard is an OMG standard that is not part of FIBO.  It could be included in the FIBO viewer if you like - it's also available from the OMG and I believe from data.world.

This means, however, if we choose to depend on other ontologies (like SKOS, Dublin Core, etc.), the emerging Standard Business Reporting Metamodel (SBRM), we will have a similar decision to make about them.

SKOS and DC are small and people are not very likely to click on SKOS and DC props.
If SBRM is large, I'd also suggest it should be included in the viewer.
Perhaps you need a new section "External ontologies" or a new colored dot to distinguish those.

This is purely an issue with what is displayed in FIBO Viewer, right?
There will be a general problem in that such external ontologies have URIs not controlled by EDMC so URIs cannot be redirected to the EDMC/FIBO viewer site and URI. Unless FIBO viewer hacks any such external URIs which occur in FIBO to replace them with something under the edmcouncil.org domain. Which I'd say would be misleading and somewhat unethical.
The external ontology owners are likely to put in place their own viewing mechanism: GLEIF has already, and OMG is likely to.

You already display external terms (thanks!). The issue is whether you can somehow incorporate them in the left-hand navigation menu.
If you do, indicate them with a purple dot or something similar.
I understand the technical difficulties related to resolution. If you cannot incorporate them in the navigation without taking control of OMG domain resolution (that is infeasible), cancel this issue.


[PLUGIN REQUEST] csvview.nvim

Field
Description

Plugin
csvview.nvim

Homepage
https://github.com/hat0uma/csvview.nvim

Nixpkgs
false

Extra Information

csvview.nvim is a lightweight CSV file viewer plugin for Neovim. With this plugin, you can easily view and edit CSV files within Neovim.

Features

Displays the CSV/TSV file in a tabular format using virtual text.
Dynamically updates the CSV view as you edit, ensuring a seamless editing experience.
Asynchronous parsing enables comfortable handling of large CSV files.
Supports two display modes:

highlight: Highlights the delimiter.
border: Displays the delimiter with │.

Customizable delimiter character.

NixVim specific
An auto-enable option would be nice. To automatically call CsvViewEnable for csv buffers.
Something like:
{
  #...

  plugins.csvview.autoEnable = {
    enabled = true;
    delimiter = ",";
  };
}

Opened https://github.com/NixOS/nixpkgs/pull/360611


Improve analyzer warnings by disambiguating elements

When importing two libraries which export classes with the same name (but hiding the class from one library import), the analyzer gives a warning like this one:
$ dartanalyzer file.dart
[warning] The type 'List<Field>' used in the 'for' loop must implement Iterable
with a type argument that can be assigned to 'Field'. (file.dart, line x, col y)

This warning could be improved by qualifying the two Field classes with the library from which they came from.
The clang c++ compiler would for a similar example say emit this warning:
file.cc:10: error: no viable conversion from 'Field' to 'foobar::Field'

$ dartanalyzer --version
dartanalyzer version 1.22.1

I agree. Analyzer is smart enough to disambiguate in some cases, but missed this case.

Very similar to #32178 but probably different specific problem.

Likely still an issue.


Add hathi rsync manage command based on import utility logic

Codecov Report

Merging #472 (a1301b8) into develop (a32937e) will increase coverage by 1.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           develop      #472      +/-   ##
============================================
+ Coverage    98.98%   100.00%   +1.01%
============================================
  Files          113         5     -108
  Lines         5906        78    -5828
  Branches         8         8
============================================
- Hits          5846        78    -5768
+ Misses          60         0      -60


[WFCORE-5167] Use individual ApacheDS dependencies to avoid beinging in additional shaded content.

https://issues.redhat.com/browse/WFCORE-5167

@jamezp The reasoning here is the version was already defined here to avoid a conflict with any parent version.  A Jira issue has been raised to clean up ApacheDS dependencies further but this change is not the time to refactoring the use of ApacheDS across the whole testsuite.

@darranl Okay. What we've done before is add a separate property. @jmesnil do you have a preference?

This pom already had the versions inlined and not in a separate property, changes like refactoring into properties can come under the next Jira.

@darranl thanks


[EMB-3] TypeScript syntax

Purpose
Follow-up from #68
Summary of Changes

Use class syntax

actions and component attributes still need to live in the .extend({ }) object
using private property initializers until otherwise needed

async / await instead of .then
Refactored a few things to use modern syntax / readability
Conform to linting rules (partial exception for tests, which need to be written)

Side Effects / Testing Notes
This will require a full regression, but most likely that will need to be done before the next release. All of the functionality should remain the same.
Ticket
https://openscience.atlassian.net/browse/EMB-3
Reviewer Checklist

[ ] meets requirements
[ ] easy to understand
[ ] DRY
[ ] testable and includes test(s)
[ ] changes described in CHANGELOG.md

Coverage remained the same at 0.0% when pulling ae760021402bf698838cbeec4e9df7249360c3ea on feature/typescript into 589568b57d00e8e3491842ca589fbd8d6706b3d8 on develop.

I've fixed the /quickfiles route. The problem was that when you override beforeModel in the class definition, there's no way to call this._super. You may see a quick flash of of the template when it is rendering, but the cas authenticated route mixin is supposed to replace the window.location, forcing a redirect. However, this doesn't block execution, so the template may render. I put in a template with message saying that it will redirect to handle this. This is an edge-case, because there are no direct links to /quickfiles, other than on the OSF navbar, which only displays the link if the user is authenticated. It's merely a vanity route. CenterForOpenScience/osf.io#8182 changes the OSF navbar to go directly to the /<user guid>/quickfiles/ route.

Nice, thank you much. That extra reload was unfortunate, and this is much better.


ENYO-1220: Support /lib/enyo location for Enyo path and minifier

Enyo-DCO-1.1-Signed-off-by: Jason Robitaille<EMAIL_ADDRESS>

Closing this as it is against master and we will not be accepting it for future releases.


[BUG] system-agent does not respect maxFailures if running the same plan post restart

Rancher Server Setup

Rancher version: n/a. Tested latest version of the system-agent, v0.3.7
Installation option (Docker install/Helm Chart):

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc):

Proxy/Cert Details: n/a

Information about the Cluster

Kubernetes version: n/a
Cluster Type (Local/Downstream): n/a

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider): n/a

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

If custom, define the set of permissions: n/a

Describe the bug
The system-agent will fail to run a plan multiple times if that plan had previously succeeded, has the same contents, and is configured with a maxFailures setting >1.
For example, Rancher will deliver a one-time-plan to the system-agent each time that a node connects to a cluster. This plan invokes the system-agent-installer-rke2 in order to install or update rke2 on the node as needed. This will happen on initial registration, as well as after a reboot of the node. In the event that this plan has a maxFailures value greater than zero, that value will only be respected the first time that the system agent applies the plan, but not on subsequent applications post reboot.
This is due to the appliedChecksumKey not being cleared after the system-agent restarts. The system-agent will always apply the plan at least once after it restarts. However, if the content of the plan has not changed, then the system-agent will compute the same checksum that is stored in the appliedChecksumKey field. This results in the system agent skipping further applications of the plan as it thinks the plan has already been successfully applied, even if the most recent application has failed.
This was discovered while investigating how to better handle https://github.com/rancher/rancher/issues/42458 in the planner. In some situations for Windows nodes, the system-agent-installer-rke2 plan may fail to properly execute post reboot. Ideally, the planner would attempt to retry this plan multiple times before marking it as 'failed', as the initial failure may be transient.
To Reproduce
Reproducing this issue while executing the system-agent-installer-rke2 plan is difficult to do, as linux nodes reliably execute this plan and Rancher is hard coded to set the maxFailures count to 1. This is more easily reproduced in the context of https://github.com/rancher/rancher/issues/42458. The planner would need to be updated to set a maxFailure and failureThreshold value of >1 for worker plans, and a Windows worker would need to be provisioned on vSphere.
Result
In the event that the one-time-plan application fails post reboot, the maxFailures property will not be respected
Expected Result
The plan is reapplied up to either the maxFailures or failureThreshold values configured for that plan, even if that plan has previously been successfully applied.
Screenshots

Additional context

Validated this against head https://github.com/rancher/rancher/commit/891b100f6c4873c7d8a7dca329a5854c1ba3763a. The test results have been recorded in Qase test run /RM 2592. The below test scenarios were tested:
ENVIRONMENT DETAILS

Rancher install: Helm
Rancher version:

Fresh install: head https://github.com/rancher/rancher/commit/688f1aff0eda9da28d88fc0167b8ae95378b9575
Upgrade: v2.9.2 -> head https://github.com/rancher/rancher/commit/891b100f6c4873c7d8a7dca329a5854c1ba3763a

TEST RESULT

#
Scenario
Result

1
[Fresh install] Perform graceful restart guest OS on Windows nodes in downstream node driver cluster
:white_check_mark:

2
[Fresh install] Replace Windows nodes and perform graceful restart guest OS
:white_check_mark:

3
[Upgraded Rancher] Perform graceful restart guest OS on Windows nodes in downstream node driver cluster
:white_check_mark:

4
[Upgraded Rancher] Replace Windows nodes and perform graceful restart guest OS
:white_check_mark:

5
[Upgraded] Rancher] Add Windows node to pre-existing cluster post-upgrade
:white_check_mark:


Bug: KeyError when The Event has no Summary Field

➜  Downloads i3-agenda
Traceback (most recent call last):
File "/usr/bin/i3-agenda", line 8, in
sys.exit(main())
File "/usr/lib/python3.8/site-packages/i3_agenda/init.py", line 97, in main
events = getEvents(service, allowed_calendars_ids, args.maxres,
File "/usr/lib/python3.8/site-packages/i3_agenda/init.py", line 147, in getEvents
all.append(Event(event['summary'], is_allday(start_time), unix_time, end_time))
KeyError: 'summary'

Looks like some commit broke it cuz it used to work for me on Manjaro i3. After adding a summary field, it works again.

Isn't summary just the event name? how do you even create an event without a title?

The rectangular area:

Are you sure? Can you reproduce it this way? Because I never enter a description and it doesn't crash for me

Somehow I can't reproduce it now, but I am sure that's the problematic field since the error went away as soon as I put in some text there. (Previously, reinstalling did not help.)

seems like you misunderstood the error.
the summary field is the title and it only exists in a struct I created (not in the API).
so the field you showed in the picture has nothing to do with that
however, it seems like there is a bug as seen in the logs you sent

I read the logs you sent, seems like the error occurs in this line
https://github.com/rosenpin/i3-agenda/blob/master/i3_agenda/__init__.py#L147
The summary field is the title as far as I know, this is how it works when I use it so there shouldn't be a different in your instance (unless there was a change to the API and you have a newer version than I)
I'll look into the error when I get the time

First. Thank you for this wonderful package. I just wanted to report that trying to install this with a fresh install through AUR i get a similar error. Credentials were created and look find. When I run with the -u flag I get.
Traceback (most recent call last):
  File "/bin/i3-agenda", line 33, in <module>
    sys.exit(load_entry_point('i3-agenda==1.4', 'console_scripts', 'i3-agenda')())
  File "/usr/lib/python3.9/site-packages/i3_agenda/__init__.py", line 100, in main
    events = getEvents(service, allowed_calendars_ids, args.maxres,
  File "/usr/lib/python3.9/site-packages/i3_agenda/__init__.py", line 173, in getEvents
    all.append(Event(event['summary'],
KeyError: 'summary'

I can see the .i3agenda folder is created, with the token.pickle. The .cache.txt file ends up being an empty array.
If you have any ideas please let me know. This looks like a fantastic script and I appreciate the time you took to build it.

:bug: Just tried this out and I have the same error as @snide. This looks like a fantastic module. <3

I looked into this and with patch in PR #48, the code no longer runs into this error. However I'm not sure if that's a complete fix. At the moment, after running the script just with the -c flag set, I get this output:

Do you literally get "%d/%m"? or the actual day/month?
I think maybe shared events really be the reason for this because I don't really have any shared events and I never ran into such issue

@snide do you have shared events in your calendar?
Could you try @indradhanush patch?

Do you literally get "%d/%m"? or the actual day/month?

Yes, indeed. I was confused and thought that maybe I am missing some additional configuration? I will try this out without a shared calendar and see if I am seeing a similar behaviour.

I'll try to run the patch later today.
On Fri, Feb 19, 2021, 3:56 AM Indradhanush Gupta<EMAIL_ADDRESS>wrote:

Do you literally get "%d/%m"? or the actual day/month?
Yes, indeed. I was confused and thought that maybe I am missing some
additional configuration? I will try this out without a shared calendar and
see if I am seeing a similar behaviour.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rosenpin/i3-agenda/issues/41#issuecomment-782029790,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACPHJ5VPQ7IRSI42HB7FY3S7ZGXJANCNFSM4QQEN6QQ
.

I checked today where the next / current event description was publicly available from the shared calendar and the description along with the timestamp was printed as the output. However, if the event description is not publicly available, I still see the %d/%m in the output.
And on that note, I was also thinking that users may only want to use i3-agenda as what is the next meeting in their calendar that they need to be at instead of seeing a notification for the next meeting from every shared calendar. Would it be possible to maybe optionally exclude all shared calendars? That is if the user wanted, they may set a flag like --exclude-shared=foo,bar,baz and that would only exclude these three calendars from the events list?
What do you think?

Solved at #88


I have error on nuxt 3

please help me with this problem
When building a project in webstorm, the console says:
[10:05:07 AM]  ERROR  Pre-transform error: Failed to parse source for import analysis because the content contains invalid JS syntax. If you are using JSX, make sure to name the file with the .jsx or .tsx extension.
and on the page
500
[vite-node] [plugin:vite:import-analysis] [VITE_ERROR] virtual:nuxt:C:/someadresstoproject/.nuxt/routes.mjs:10:3
8  |      redirect: icons_45pageK3BpbB8TSHMeta?.redirect,
9  |      component: () => import("C:/someadresstoproject/node_modules/@nuxtjs/svg-sprite/dist/runtime/components/icons-page.vue").then(m => m.default || m)
10 |    }
|    ^
11 |  ]
at virtual:nuxt:C:/someadresstoproject/.nuxt/routes.mjs:10:3

Same here, started with some routine maintenance yesterday.
[09:14:01]  ERROR  Pre-transform error: Failed to parse source for import analysis because the content contains invalid JS syntax. If you are using JSX, make sure to name the file with the .jsx or .tsx extension.

Seemingly seems to be an issue with this package and Nuxt 3.10.0

Seemingly seems to be an issue with this package and Nuxt 3.10.0, @fenn1ks - try using Nuxt 3.9.0
thanks, I'll try it now, but I'd like to use the actual version of Nuxt

@Zellement This also works on 3.9.3

@fenn1ks
fork

works on 3.10.2 NUXT


Fix post-commit python failure.

addresses #30513 regarding docker error (https://github.com/apache/beam/issues/30513#issuecomment-2468551929)

My fix doesn't clear the PostCommit Python problem. The previous docker problem is gone, but now there are other problems related to it. I am going to revert https://github.com/apache/beam/commit/81f35ab62298a2ec9fadeded82461b363b6401db then.


Minor:  Should say WDF instead of KMDF

Instead of KMDF it should say "WDF (KMDF and UMDF V2) " so it is clear that it applies to both.
"framework calls a KMDF function or filter driver's event callback function"

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

ID: 8e10578f-45fa-4666-15b9-67942520cd6a
Version Independent ID: 779e15f8-0987-d4d2-b31c-c906f4830729
Content: Power-Up Sequence for a Function or Filter Driver - Windows drivers
Content Source: windows-driver-docs-pr/wdf/power-up-sequence-for-a-function-or-filter-driver.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @tedhudek
Microsoft Alias: tedhudek

Just pushed a fix, should be live soon.


Create TCP socket

As a user
I need develop better TCP connections
So that  I can integrate the PBX system without loosing any data.
Details and Assumptions

https://docs.python.org/3/howto/sockets.html

Acceptance Criteria
Given
When
Then

close #1.


snapshot/git/gitdb: implement tags backend

Tags backend uploads and downloads snapshots as git tags.
Enable Auto-Uploading in GitDB. When a Snapshot is created, it can be auto-uploaded to a remote (that is an uploader).
Change some (internal-to-GitDB) code to make more sense and be more composable. (e.g., now we return a snapshot instead of a snap.ID)
Expand db_test to have more DBs in the system to test distribution.
rename {local,stream}Snap -> {local,stream}Snapshot (because I forgot to when I renamed snap -> snapshot; oops!)

Current coverage is 59.72% (diff: 51.16%)

Merging #206 into master will decrease coverage by 0.13%

@@             master       #206   diff @@
==========================================
  Files            80         81     +1
  Lines          3736       3786    +50
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           2236       2261    +25
- Misses         1308       1322    +14
- Partials        192        203    +11

Powered by Codecov. Last update cd63f42...08b100d


Introduce Options for the httpclient

Currently, slow up and downloads fail because the timeout is currently hardcoded to 10 sec.
This PR allows configuring the HTTPClient as needed. It also introduces timeout and insecure config options for the oc services. I tried not change the behavior or introduce config options when a config struct was not available, eg for the CLI up and download commands.
For mentix I updated the example config.
I also made the oidc client use the rhttp httpclient, so it should also send traces.

@butonic Can you rebase?


feat(providers): add emojis and user

Please describe the changes this PR makes and why it should be merged:
Add client.emojis and client.user as providers
This commit adds these 2 new providers to make it easier to access client data as it is already done with other data
Status and versioning classification:

This PR changes the library's interface (methods or parameters added)

LGTM, need to update docs

Docs updated! https://github.com/necordjs/necord.org/pull/251


when my code is error, and i fix it  but the browser not change .i need to rerun npm start

when my code is error, and i fix it  but the browser not change .i need to rerun npm start
anyone know this?

What version of react-scripts are you using?

Please refer to the relevant section of the docs.


Primitive interface updates for SamplerV2

Part of https://github.com/Qiskit/qiskit-ibm-runtime/issues/1131
Support SamplerV2 input/output:

RFC
Input/output would also be PUB based
Output is per shot measurement, with convenience functions to returns counts/probabilities

Based on the following Qiskit PRs https://github.com/Qiskit/qiskit/pull/11566 (replaced https://github.com/Qiskit/qiskit/pull/11264)

SamplerV2 Container https://github.com/Qiskit/qiskit/pull/11542
BaseSamplerV2 https://github.com/Qiskit/qiskit/pull/11529

Work should be done off the experimental branch where some of the work is already done (https://github.com/Qiskit/qiskit-ibm-runtime/pull/1224)

cc: @gadial

The BaseSamplerV2 has been closed, and the BaseEstimatorV2 is in the merge queue. My guess is that the runtime repo should not be blocked two state vector implementations (https://github.com/Qiskit/qiskit/pull/11566 and https://github.com/Qiskit/qiskit/pull/11227) but only on  BaseEstimatorV2.
That being said, the two state vector implementations will hopefully be merged tomorrow.

Completed in 0.21.0 release


topic/filter in relayer

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

8 of 16 (50.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 63.815%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

chainsight-cdk/src/config/components.rs
1
3
33.33%

chainsight-cdk-macros/src/canisters/relayer.rs
7
13
53.85%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
4573

Relevant Lines:
7166

💛  - Coveralls


flows: hoster setup + proof of storage

@todo

[ ] improve hosting setup for encoder
[ ] improve hosting setup for hoster
[ ] improve hosting setup for attestor
[ ] improve hosting setup for chain
[ ] improve proof of storage for hoster
[ ] improve proof of storage for attester
[ ] improve proof of storage for chain
[ ] add accounting logic to hosting setup
[ ] add payment logic to proof of storage

improve hosting setup for encoder

improve hosting setup for hoster

improve hosting setup for attestor

improve hosting setup for chain

improve proof of storage for hoster

improve proof of storage for attestor

improve proof of storage for chain

add accounting logic to hosting setup

add payment logic to proof of storage


Restrictions

[Enter feedback here]
Please advise if there are any country-level restrictions on using MS Teams outside the UK?
Will my students face issues if they travel to certain countries? If so, will it please be OK to advise which countries current block meetings via MS Teams.

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

ID: ca7f5d7b-fdf3-f9a7-46e7-689258f33cf9
Version Independent ID: 86f27884-7c71-ee17-aa4e-5b3e12125814
Content: Meetings in Microsoft Teams - Microsoft Teams
Content Source: Teams/Tutorial-Meetings-in-Teams.yml
Service: msteams
GitHub Login: @LolaJacobsen
Microsoft Alias: lolaj

@Pixie7 Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


What im doing wrong with installing go-sdl2

Hi i have a problem with go-sdl2 first i create go module next install all package on debian next write some sdl2 code in golang and go mod tidy then i go build it and there's this loong warning
# github.com/veandco/go-sdl2/sdl
In file included from _cgo_export.c:4:
hints.go:152: warning: "SDL_HINT_JOYSTICK_ROG_CHAKRAM" redefined
In file included from /usr/local/include/SDL2/SDL.h:46,
                 from ../../go/pkg/mod/github.com/veandco/go-sdl2@v0.4.24/sdl/sdl_wrapper.h:5,
                 from events.go:4,
                 from _cgo_export.c:4:
/usr/local/include/SDL2/SDL_hints.h:820: note: this is the location of the previous definition
  820 | #define SDL_HINT_JOYSTICK_ROG_CHAKRAM "SDL_JOYSTICK_ROG_CHAKRAM"
      |
In file included from _cgo_export.c:4:
hints.go:153: warning: "SDL_HINT_X11_WINDOW_TYPE" redefined
In file included from /usr/local/include/SDL2/SDL.h:46,
                 from ../../go/pkg/mod/github.com/veandco/go-sdl2@v0.4.24/sdl/sdl_wrapper.h:5,
                 from events.go:4,
                 from _cgo_export.c:4:
/usr/local/include/SDL2/SDL_hints.h:1882: note: this is the location of the previous definition
 1882 | #define SDL_HINT_X11_WINDOW_TYPE "SDL_X11_WINDOW_TYPE"
      |
# github.com/veandco/go-sdl2/sdl
../../go/pkg/mod/github.com/veandco/go-sdl2@v0.4.24/sdl/hints.go:152: warning: "SDL_HINT_JOYSTICK_ROG_CHAKRAM" redefined
  152 | #define SDL_HINT_JOYSTICK_ROG_CHAKRAM ""
      |
In file included from /usr/local/include/SDL2/SDL.h:46,
                 from ../../go/pkg/mod/github.com/veandco/go-sdl2@v0.4.24/sdl/sdl_wrapper.h:5,
                 from ../../go/pkg/mod/github.com/veandco/go-sdl2@v0.4.24/sdl/hints.go:4:
/usr/local/include/SDL2/SDL_hints.h:820: note: this is the location of the previous definition
  820 | #define SDL_HINT_JOYSTICK_ROG_CHAKRAM "SDL_JOYSTICK_ROG_CHAKRAM"
      |
../../go/pkg/mod/github.com/veandco/go-sdl2@v0.4.24/sdl/hints.go:153: warning: "SDL_HINT_X11_WINDOW_TYPE" redefined
  153 | #define SDL_HINT_X11_WINDOW_TYPE ""
      |
In file included from /usr/local/include/SDL2/SDL.h:46,
                 from ../../go/pkg/mod/github.com/veandco/go-sdl2@v0.4.24/sdl/sdl_wrapper.h:5,
                 from ../../go/pkg/mod/github.com/veandco/go-sdl2@v0.4.24/sdl/hints.go:4:
/usr/local/include/SDL2/SDL_hints.h:1882: note: this is the location of the previous definition
 1882 | #define SDL_HINT_X11_WINDOW_TYPE "SDL_X11_WINDOW_TYPE"
      |
# github.com/veandco/go-sdl2/sdl
In file included from events.c:1:
hints.go:152: warning: "SDL_HINT_JOYSTICK_ROG_CHAKRAM" redefined
  152 | #define SDL_HINT_JOYSTICK_ROG_CHAKRAM ""
      |
In file included from /usr/local/include/SDL2/SDL.h:46,
                 from ../../go/pkg/mod/github.com/veandco/go-sdl2@v0.4.24/sdl/sdl_wrapper.h:5,
                 from events.go:4,
                 from events.c:1:
/usr/local/include/SDL2/SDL_hints.h:820: note: this is the location of the previous definition
  820 | #define SDL_HINT_JOYSTICK_ROG_CHAKRAM "SDL_JOYSTICK_ROG_CHAKRAM"
      |
In file included from events.c:1:
hints.go:153: warning: "SDL_HINT_X11_WINDOW_TYPE" redefined
  153 | #define SDL_HINT_X11_WINDOW_TYPE ""
      |
In file included from /usr/local/include/SDL2/SDL.h:46,
                 from ../../go/pkg/mod/github.com/veandco/go-sdl2@v0.4.24/sdl/sdl_wrapper.h:5,
                 from events.go:4,
                 from events.c:1:
/usr/local/include/SDL2/SDL_hints.h:1882: note: this is the location of the previous definition
 1882 | #define SDL_HINT_X11_WINDOW_TYPE "SDL_X11_WINDOW_TYPE"
      |
# github.com/veandco/go-sdl2/sdl
In file included from hints.c:1:
hints.go:152: warning: "SDL_HINT_JOYSTICK_ROG_CHAKRAM" redefined
  152 | #define SDL_HINT_JOYSTICK_ROG_CHAKRAM ""
      |
In file included from /usr/local/include/SDL2/SDL.h:46,
                 from ../../go/pkg/mod/github.com/veandco/go-sdl2@v0.4.24/sdl/sdl_wrapper.h:5,
                 from events.go:4,
                 from hints.c:1:
/usr/local/include/SDL2/SDL_hints.h:820: note: this is the location of the previous definition
  820 | #define SDL_HINT_JOYSTICK_ROG_CHAKRAM "SDL_JOYSTICK_ROG_CHAKRAM"
      |
In file included from hints.c:1:
hints.go:153: warning: "SDL_HINT_X11_WINDOW_TYPE" redefined
  153 | #define SDL_HINT_X11_WINDOW_TYPE ""
      |
In file included from /usr/local/include/SDL2/SDL.h:46,
                 from ../../go/pkg/mod/github.com/veandco/go-sdl2@v0.4.24/sdl/sdl_wrapper.h:5,
                 from events.go:4,
                 from hints.c:1:
/usr/local/include/SDL2/SDL_hints.h:1882: note: this is the location of the previous definition
 1882 | #define SDL_HINT_X11_WINDOW_TYPE "SDL_X11_WINDOW_TYPE"
      |
# github.com/veandco/go-sdl2/sdl
In file included from log.c:1:
hints.go:152: warning: "SDL_HINT_JOYSTICK_ROG_CHAKRAM" redefined
  152 | #define SDL_HINT_JOYSTICK_ROG_CHAKRAM ""
      |
In file included from /usr/local/include/SDL2/SDL.h:46,
                 from ../../go/pkg/mod/github.com/veandco/go-sdl2@v0.4.24/sdl/sdl_wrapper.h:5,
                 from events.go:4,
                 from log.c:1:
/usr/local/include/SDL2/SDL_hints.h:820: note: this is the location of the previous definition
  820 | #define SDL_HINT_JOYSTICK_ROG_CHAKRAM "SDL_JOYSTICK_ROG_CHAKRAM"
      |
In file included from log.c:1:
hints.go:153: warning: "SDL_HINT_X11_WINDOW_TYPE" redefined
  153 | #define SDL_HINT_X11_WINDOW_TYPE ""
      |
In file included from /usr/local/include/SDL2/SDL.h:46,
                 from ../../go/pkg/mod/github.com/veandco/go-sdl2@v0.4.24/sdl/sdl_wrapper.h:5,
                 from events.go:4,
                 from log.c:1:
/usr/local/include/SDL2/SDL_hints.h:1882: note: this is the location of the previous definition
 1882 | #define SDL_HINT_X11_WINDOW_TYPE "SDL_X11_WINDOW_TYPE"
      |
# github.com/veandco/go-sdl2/sdl
In file included from system.c:1:
hints.go:152: warning: "SDL_HINT_JOYSTICK_ROG_CHAKRAM" redefined
  152 | #define SDL_HINT_JOYSTICK_ROG_CHAKRAM ""
      |
In file included from /usr/local/include/SDL2/SDL.h:46,
                 from ../../go/pkg/mod/github.com/veandco/go-sdl2@v0.4.24/sdl/sdl_wrapper.h:5,
                 from events.go:4,
                 from system.c:1:
/usr/local/include/SDL2/SDL_hints.h:820: note: this is the location of the previous definition
  820 | #define SDL_HINT_JOYSTICK_ROG_CHAKRAM "SDL_JOYSTICK_ROG_CHAKRAM"
      |
In file included from system.c:1:
hints.go:153: warning: "SDL_HINT_X11_WINDOW_TYPE" redefined
  153 | #define SDL_HINT_X11_WINDOW_TYPE ""
      |
In file included from /usr/local/include/SDL2/SDL.h:46,
                 from ../../go/pkg/mod/github.com/veandco/go-sdl2@v0.4.24/sdl/sdl_wrapper.h:5,
                 from events.go:4,
                 from system.c:1:
/usr/local/include/SDL2/SDL_hints.h:1882: note: this is the location of the previous definition
 1882 | #define SDL_HINT_X11_WINDOW_TYPE "SDL_X11_WINDOW_TYPE"
      |

but ./app is creating when i am starting it there's this error message:
solindek_@debian:~/Documents/Tal$ ./app
panic: No available video device

goroutine 1 [running, locked to thread]:
main.main()
        /home/solindek_/Documents/Tal/main.go:7 +0x254

im using code from the example please help me!

Hi @SolindekDev, could you let me know your version of SDL2?


Get Container Registry Github Action working on Azure

The github action should now be working in the latest commit in the azure branch on the qhub-cloud repo


Make server-admins creation migration idempotent.

If server-admins has an entry in erchef, skip its creation and continue to permission adds.
Fixes https://github.com/chef/chef-server/issues/822
Build

LGTM 👍

Tested it out. Should be g2g.

LGTM


[RC3] Cannot resolve @intlify/message-compiler on pnpm

Environment
Nuxt project info:                                                                              9:22:24 AM

Operating System: Linux
Node Version:     v20.5.1
Nuxt Version:     3.6.5
Nitro Version:    2.5.2
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      app, css, plugins, components, modules, googleFonts, i18n, ignore, imports, router, ssr, sourcemap, telemetry, nitro, runtimeConfig, vite
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
No response
Describe the bug
When I upgrade from RC2 to RC3, I get the error:
 ERROR  Cannot resolve @intlify/message-compiler on pnpm! please install it on 'node_modules'

Additional context
audit=false
engine-strict=false
loglevel=error
fund=false
node-linker=hoisted
progress=false
shamefully-hoist=true
strict-peer-dependencies=true
update-notifier=false

Logs
Nuxi 3.6.5
 ERROR  Cannot resolve @intlify/message-compiler on pnpm! please install it on 'node_modules'
  at tryResolve (node_modules/@nuxtjs/i18n/dist/module.mjs:217:9)
  at async setupAlias (node_modules/@nuxtjs/i18n/dist/module.mjs:453:53)
  at async setup (node_modules/@nuxtjs/i18n/dist/module.mjs:1439:5)
  at async normalizedModule (node_modules/@nuxt/kit/dist/index.mjs:2140:17)
  at async installModule (node_modules/@nuxt/kit/dist/index.mjs:2409:95)
  at async initNuxt (node_modules/nuxt/dist/index.mjs:3238:7)
  at async loadNuxt (node_modules/nuxt/dist/index.mjs:3314:5)
  at async loadNuxt (node_modules/@nuxt/kit/dist/index.mjs:2537:19)
  at async Object.invoke (node_modules/nuxi/dist/chunks/prepare.mjs:31:18)
  at async _main (node_modules/nuxi/dist/cli.mjs:49:20)
 ELIFECYCLE  Command failed with exit code 1.

Thank you for your reporting!
Unfortunately we could not reproduce your issue
we need a minimum reproduction to ensure that we reproduce your issue.
Issue guideline saying:

Please use a template below to create a minimal reproduction, and provide to us your issue cases
👉 https://stackblitz.com/github/nuxt/starter/tree/v3-stackblitz

Could you give us it please? 🙏

Hi, we have the same issue. I have here a minimal reproduction https://stackblitz.com/edit/github-7endol?file=package.json

Adding other library as well with the same error message:

@joan-domingo

Hi, we have the same issue. I have here a minimal reproduction https://stackblitz.com/edit/github-7endol?file=package.json
It started with the beta-13.
Let me know if you need more help

I think your issue is related to #2098

@joan-domingo

Hi, we have the same issue. I have here a minimal reproduction https://stackblitz.com/edit/github-7endol?file=package.json
It started with the beta-13.
Let me know if you need more help

I think your issue is related to #2098

Might be but I can't seem to find a way to make it work by overriding the other intlify libraries version. If anyone has some hints let me know. Thanks!

Thank you for your reporting!
Unfortunately we could not reproduce your issue we need a minimum reproduction to ensure that we reproduce your issue.
Issue guideline saying:

Please use a template below to create a minimal reproduction, and provide to us your issue cases
👉 https://stackblitz.com/github/nuxt/starter/tree/v3-stackblitz

Could you give us it please? 🙏

Hi @kazupon, I'm trying to create a reproduction like you asked.
So far, I've found that when I remove "@intlify/eslint-plugin-vue-i18n": "2.0.0", from my package.json, the error is gone.

When I use "@intlify/eslint-plugin-vue-i18n": "3.0.0-next.3", the error is gone.

Hi, we have the same issue. I have here a minimal reproduction https://stackblitz.com/edit/github-7endol?file=package.json

Looks like installing the latest versions resolves this issue.


Suggestion: Don't really need systemd container

Hello,
You don't really need systemd for the Docker deployment. Instead you should use restart=always it is a better way to keep the server alive even if the container crash.
Add this to your docker command:
docker run -d \
    --name valheim-server \
    -p 2456-2458:2456-2458/udp \
    -v $HOME/valheim-server/config:/config \
    -v $HOME/valheim-server/data:/opt/valheim \
    -e SERVER_NAME="My Server" \
    -e WORLD_NAME=Neotopia \
    -e SERVER_PASS=secret \
    --restart always
    lloesche/valheim-server

You could also add docker-compose this way:
version: '3'

services:
  valheim-server:
    image: lloesche/valheim-server
    ports:
       - 2465-2458:2465-2458/udp
    volumes:
      - $HOME/valheim-server/config:/config
      - $HOME/valheim-server/data:/opt/valheim
    environment:
      - SERVER_NAME=My Server
      - WORLD_NAME=Neotopia
      - SERVER_PASS=secret
    restart: always

And then deploy with docker-compose up -d

We've had that topic a couple of times. The existing docs are just some examples of how to start the container. You are of course free to run the container whichever way you like 🙂
Some might even prefer another containerizer like podman or extract the image into a systemd-nspawn controlled environment. In the end it's just a bunch of Linux processes namespaced into a cgroup.
I for one like to run the server like all my other system services and control it using systemctl hence the service unit.
If you feel the existing README should be improved let's make a PR and add your notes. Maybe within the Deployment section create a new subsection Deploying with Docker only or something like that?


[Proposal] Automatically regenerate the docs on push to main

Proposal
Automatically regenerate the docs on push to main.
Motivation
The committed .rst files are sometimes out of date if not explicitly regenerated in a PR, as running sphinx-apidoc locally showed differences on a recent PR.
There are inconsistencies in the documentation that is being served through GH Docs and the actual state of the repository. Recently the "torchtyping helper module" was in the documentation, but the module had been removed from the repository. It is proposed to add automatic rst generation in the CI pipeline to keep the documentation up-to-date.
Pitch
The proposed solution is to add automatic rst generation in the CI pipeline.
This can be accomplished by adding a step in the GH action that runs sphinx-apidoc whenever changes are pushed to the repository (I think! CC @dkamm @Aprillion )
This will ensure that the documentation stays consistent with the state of the repository.
Checklist
[x] I have checked that there is no similar issue in the repo (required)

Fixed in #398


Menu next to profile window displays incorrectly when opened when site is scrolled down

Menu next to profile window displays incorrectly when opened when site is scrolled down, this occurs in about, profile and group sections consistently

Fixed.


How to change test method names

Hi,
Is there a way to change the test method names of test cases not just test case name .I know that the test case name can be changed by implementing ITEST iterfac and implementing its method but thats not what I am looking.I looking to change the name of methods so that I can show my test reports with my custom name?
Please provide inputs ASAP.
Best Regards

Hi,
What is the reporter you are using?

so that I can show my test reports with my custom name

The reporter is supposed to use the test case name (and not the name of the method), and the name of the method is not supposed to be changed.

I am using default reporter provided by testng.
my  tag has many classes in it, it displays in the following format packagename.classname.methodname on jira, my final aim is to show the test method name as my custom name not the one I am provided by testing

As workarround, you can use your own reporter which will generate the output you expect: http://testng.org/doc/documentation-main.html#logging-reporters

@Udit2 Could you share a sample (a test class + output) of what you are expecting?

Closing this issue since there's no response from the user.


[DRAFT] Ami minimum age

Fixes #5382
Description
This is a draft PR that proposed a possible implementation for the introduction of a "ami minimum age" parameter.
I opened it as a draft to get early feedback.
Not sure if this feature is already worked on by the maintainers or if there are other plans about it.
How was this change tested?
It's not exactly tested (a part for a very small subset of use cases, locally), all the unit tests are missing.
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Basically the proposed change is the following:
we query SSM for the recommended Amis, and then we check with the ec2 describe-image if the creation date complies with the newly introduced amiminimumage parameter. If not, we pick the next ami that matches the required minimum age.
some assumptions here:

if a recommended ami is more recent than the required miMinumAge, we need to retrieve anyway a "valid" image (is this even true? This test suite seems to suggest that in other scenarios we accept we don't have a valid ami)
we can't query the SSM parameter history of the recommended amis (I have a very powerful role and I couldn't get the parameter history, so I assumed the parameter was protected)
the "approved" eks node images are always published with the naming convention of amazon-eks-arm64-node-<eks-version>- or similar. It sounds fragile, I hope I'm missing something: the ec2 api doesn't allow to specify the image location in the filter, so I'm wondering if the name of the ami is robust enough.

For brevity, the code only implements the proposed change for AL2 family, to gather some feedback about the strategy overall, so the other families are missing (on purpose)

Hey @cebernardi can you go into a bit of your use case here? For features like these, it would be nice to talk through what you're trying to achieve and if this is the best way to go about it. Usually the best way to do this is to open a RFC (review for comments) or design doc that details the use-cases and the reason you picked this solution.

I was picking up from this discussion, the amiMinimumAge was proposed by @jonathan-innis , and it seemed perfect for the use case discussed on the issue.
I'm happy to open a RFC, but given that's Jonathan's idea, would you still suggest me to open one?

I think it'd be best to open a RFC, as it seems like he was just proposing an idea, but there wasn't much discussion on if we want to do this.

Sounds good!

@njtran I reverted the code and added a design document.
Please let me know if I interpreted your request correctly (I couldn't find any reference to RFC format on the documentation)

should I also change it from draft to a proper PR?

Hey @cebernardi, planning on taking another look at this next week, but in the meantime, there's some discussion amongst the maintainers to figure out how to handle the issues that are motivating you to cut this. I'll post here when I have more information on this.

Hey @cebernardi, I know I said I would give you an update on this last week, but still figuring some things out. We're currently figuring out if there are any additional guardrails that we can implement in clusters that would allow you to control the rate that new AMIs are rolled out in your cluster. Can you give some more details about how you detected the issue with the AMI you had? How would you have best liked to see the issue detected?

Hi @njtran
we detected the issue because we got a spike of applications failing to schedule on the affected nodes.
The spike was happening both in test and production, which was confusing because we couldn't immediately link it to any change we deployed and we were actively monitoring.
We're not testing specifically new AMIs, but we have synthetic use-cases running twice a day in our test environments.
Ideally the spike would have been happening only in test environments, so the higher environments would have been spared.
Does it answer your question?

@cebernardi Ah so you use automatic upgrades (no AMISelectorTerms) in both dev and production?

Yes

@cebernardi Ah so you use automatic upgrades (no AMISelectorTerms) in both dev and production?

Yes

It may be worth your time to read through https://karpenter.sh/docs/tasks/amitasks/#choosing-ami-tasks. We added some more details on what we think is the best way to model your AMI rollouts. Not saying this fixes everything, but this should help a bit while we're still figuring out things on our end.

Closing this due to a conversation that @jonathan-innis and @cebernardi had at KubeCon that we don't think minimum age is the right approach here.

we tested the approach suggested here, and we found it adds an (avoidable, in my opinion) overhead on the operational side.
Please see this comment.
Can we reopen this?


[SUBS] Add ability to capture additional customer data through the checkout widget

As a merchant,
I'd like to capture additional customer data,
when the customer subscribes using the checkout widget.
The customer data is mainly meant as the e-mail address. The widget should treat the data as ephemeral, i.e. it will pass it through using callbacks but will not actually store it anywhere.
To consider:

should filling the fields be optional or required

if it's required then it should be validated before the user can do the transactions

anything else besides e-mail? (like a phone number)
should the callback be blocking, i.e. not asynchronous like a listener

To get an idea how on Stripe's checkout builder you can enable capturing of "phone number" and other additional data: https://checkout.stripe.dev/configure We want to capture for e-mail.om

Investigated how to use custom data.


unable to enumerate USB device

I bought a fresh STM32F103 (blue pill), as I initially bought a CKS32F103.
The problem now is, that the device will not be recognized as a USB device.
Aug 17 11:39:07 tobias-linux kernel: usb 1-4: new full-speed USB device number 115 using xhci_hcd
Aug 17 11:39:07 tobias-linux kernel: usb 1-4: Device not responding to setup address.
Aug 17 11:39:07 tobias-linux kernel: usb 1-4: Device not responding to setup address.
Aug 17 11:39:08 tobias-linux kernel: usb 1-4: device not accepting address 115, error -71
Aug 17 11:39:08 tobias-linux kernel: usb usb1-port4: unable to enumerate USB device

If I plug in my clone stm32, I will get the following
Aug 17 11:39:54 tobias-linux kernel: usb 1-4: new full-speed USB device number 116 using xhci_hcd
Aug 17 11:39:54 tobias-linux kernel: usb 1-4: device descriptor read/64, error -71
Aug 17 11:39:54 tobias-linux kernel: usb 1-4: New USB device found, idVendor=26ba, idProduct=8003, bcdDevice= 2.00
Aug 17 11:39:54 tobias-linux kernel: usb 1-4: New USB device strings: Mfr=1, Product=2, SerialNumber=3
Aug 17 11:39:54 tobias-linux kernel: usb 1-4: Product: Boot Switch
Aug 17 11:39:54 tobias-linux kernel: usb 1-4: Manufacturer: Stecman
Aug 17 11:39:54 tobias-linux kernel: usb 1-4: SerialNumber: D000000000E7
Aug 17 11:39:54 tobias-linux kernel: usb-storage 1-4:1.0: USB Mass Storage device detected
Aug 17 11:39:54 tobias-linux kernel: scsi host7: usb-storage 1-4:1.0

But flashing was successful of the new real STM32.
$ make flash PROGRAMMER=stlink
  LD	boot-swtich.stm32f103c8.elf
  OBJCOPY	boot-swtich.stm32f103c8.bin

Firmware size (boot-swtich.stm32f103c8.elf):

   text	   data	    bss	    dec	    hex	filename
   8128	    112	   1020	   9260	   242c	boot-swtich.stm32f103c8.elf

Flash: 12% (8240 / 65536 bytes)
RAM: 5% (1132 / 20480 bytes)

BINARY=boot-swtich.stm32f103c8.elf openocd -f interface/stlink.cfg -c "transport select hla_swd" -f target/stm32f1x.cfg -f openocd.cfg -c 'program_and_run ()'
Open On-Chip Debugger 0.11.0-rc2
Licensed under GNU GPL v2
For bug reports, read
	http://openocd.org/doc/doxygen/bugs.html
hla_swd
Info : The selected transport took over low-level target control. The results might differ compared to plain JTAG/SWD
attach
Info : clock speed 1000 kHz
Info : STLINK V2J17S4 (API v2) VID:PID 0483:3748
Info : Target voltage: 3.205253
Info : stm32f1x.cpu: hardware has 6 breakpoints, 4 watchpoints
Info : starting gdb server for stm32f1x.cpu on 3333
Info : Listening on port 3333 for gdb connections
target halted due to debug-request, current mode: Thread
xPSR: 0x01000000 pc: 0x080016a8 msp: 0x20005000
Info : device id = 0x10006412
Info : flash size = 32kbytes
shutdown command invoked

Do you have any idea? Unfortunately I'm no hardware programmer.
I'm thankful for any help :)

What's the frequency of the crystal on your new board? The code in clock.c is set up for an 8MHz crystal, but 12MHz is also common. There'll be a number etched or printed on the crystal something like this:

For a 12MHz crystal, you should only need to change the argument on line 86 in clock.c to RCC_CFGR_PLLMUL_PLL_CLK_MUL4. The USB peripheral needs a clock of 48MHz (4 x 12 = 48), but it'll be getting 72MHz if this is the case (6 x 12 = 72).

It's also a 8MHz crystal 😿

Further, I've checked all soldering points for shorts. Nothing suspicious

Hmm, a few things to check/try if you haven't already:

Disconnecting and connecting the controller (it could be stuck in halt)
Try a different USB cable as a sanity check
Verify a simple blink program runs (just replace your main.c contents with this)

#include "clock.h"

#include <libopencm3/stm32/gpio.h>
#include <libopencm3/stm32/rcc.h>

int main(void)
{
    init_clocks();

    rcc_periph_clock_enable(RCC_GPIOC);
    gpio_set_mode(GPIOC, GPIO_MODE_OUTPUT_2_MHZ, GPIO_CNF_OUTPUT_PUSHPULL, 0xFFFF);

    while (1) {
        gpio_toggle(GPIOC, 0xFFFF);
        for (uint32_t i = 0; i != 0xFFFFF; i++) {}
    }
}

Confirm the part number on the microcontroller chip. My example says STM32F103C8T6, but worth checking what you've got there as I can't quite read it in your photo.
Connect USB without the programming wires connected
Try a different USB cable

The "Device not responding to setup address....device not accepting address 115, error -71" message comes up even if the controller is held in reset or running firmware that doesn't use the USB peripheral, so it could really be anything, but should be possible to work through by a process of elimination.

Ahh, thanks for that hint.
I have a stm32f103c6t6 and defining the DEVICE env did the trick.
Thanks so much! :)

You could add stm32f103c6 as tested device to the README.md :) 🥳

I've updated this to build against the lowest-spec part in the f103 series, which will work out of the box for any variant :+1:
If you're curious, the issue here will have been the stack pointer getting set to an address past the end of your chip's smaller RAM, probably causing a hard-fault (which is handled by an infinite loop by default).

Ah, that makes total sense. Thank you for the explanation 🙂
Does it make sense to mention the semi-functional cks32f103? This chip will work, but it will not be detected to 100%.
My new blue pill is working perfectly, like you've described.

I've added a note to the troubleshooting section about the issues you had with the CKS chip 👍
That part number doesn't work when given to the DEVICE build variable, but you're right that it should be mentioned. Thanks!


feat(tuner): add default projection head for ssl

ProjectionHead, first proposed in SimCLR paper, is adding a 2-3 layer MLP on top of the embedding model, this trick is able to bootstrap the representation learning. This has became a common practice in contrastive representation learning.
This PR will:

create a simple MLP head in tailor, replace the old bottleneck_net.
add tests.
restructure the tailor tests, now fixtures of tailor moved out from the tailor to conftest.

[x] pytorch
[x] paddle
[x] keras

chop the projection head once training is finished

@bwanglzu I'm sorry to argue this trick.I think we need some experiments to see whether it is reasonable to chop the projection layer for the retrieval task.
In related papers, they declared that the projection layer resulted in degraded performance on downstream tasks (i.e., classification task). However, they did not evaluate the performance of the retrieval task.

@numb3r3 just had a discussion with @tadejsv , we'll keep the projection head as bottleneck_layer, so we don't remove it anymore.  It will be used as a tailor module.
Is it sounds reasonable to you?

Looks good! Just two comments. :)


How to configure linting in code editors?

I tried using atom and Visual Studio Code - rubocop is working, but without rspec rules.
My setup in VSC:
"ruby.lint":{
  "rubocop": {
     "lint": true,
     "rails": true,
     "require": ["rubocop-rspec"] # this also did not help
  }
}

To be honest, I'm not sure. I remember trying to get this to work with vscode when I was trying it out about 6 months ago, but I never got it quite right.
I'd suggest you ask over at vscode-ruby if you can't make headway.
We do have some various quirks about require order, etc. that can be problematic since there isn't (yet) an official rubocop extension api, so that could be related but I don't have any concrete suggestions for you at the moment.
One thing you might want to check is if "lint": true disables non-lint cops as well as enabling the lint cops. If so, it might exclude the rubocop-rspec linting, but that's just speculation--I'm not sure how it's implemented (though I think rubocop -l has this effect).

Posted an issue: https://github.com/rubyide/vscode-ruby/issues/352

solved with e.g. vscode-ruby  0.22.2


Native method in exception stack causes botsing to fail

Characteristics

Issue Type: [bug]
Reproducibility: [always]
Severity: [major]
Tool/Service/Component: [Botsing, maven plugin]
Execution Environment: [Linux Ubuntu 18.04]
Reporter: [Pierre-Yves Gibello<EMAIL_ADDRESS>
Description
When the exception stack contains a native method call, botsing fails.
Steps to reproduce
Exception stack sometimes contains lines like this one:
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
Botsing fails with following error:
[ERROR] Failed to execute goal eu.stamp-project:botsing-maven:1.0.3:botsing (default-cli) on project authzforce-ce-core: Error executing Botsing: For input string: "(Native Method" -> [Help 1]
[...]
Caused by: java.lang.NumberFormatException: For input string: "(Native Method"
Other files and URLs
Attached exception stack (from authzforce OW2 project - https://gitlab.ow2.org/authzforce ), stored in /tmp/ex1 file, causes failure for mvn eu.stamp-project:botsing-maven:botsing -Dcrash_log=/tmp/ex1 -Dtarget_frame=2 -e
(launched in authzforce/core dir).
ex1.log

Thank you for opening this issue. We are working on this issue to resolve it for the next release. For now, you can remove the frames which do not contain the line number.

Hi @gibello, frames with Native Method and Unknown Source are not supported by botsing-reproduction right now.
The reason is that we need the line numbers where the exception is thrown (or is propagated) in the frames to avoid being stuck with exceptions thrown at the wrong places during the search process.
From a user perspective, you should remove those frames from the stack trace and rerun botsing with the cleaned up stack trace. @pasquy73 is writing a botsing-preprocessing utility to automate that process. See #9 for more details.

Ok. Got a code snippet that extracts exceptions from any log file (eg. a tomcat log file), and returns them as a list of String. If useful for you ?
Code follows...
    /**
     * Extract exceptions from a log stream (eg. an app server log file).
     * @param in The log stream (file or so) from which exceptions must be extracted.
     * @return A list of exceptions extracted in the input stream, null if none.
     * @throws IOException
     */
    public List<String> extractExceptions(BufferedReader in) throws IOException {
    	LinkedList<String> ret = null;
    	String line;
    	String previous = "";
    	Pattern pattern = Pattern.compile("(^\\tat |^Caused by: |^\\t... \\d+ more)");
    	Pattern nested = Pattern.compile("; nested exception is: *$");
    	StringBuffer currentException = null;
    	while((line = in.readLine()) != null) {
    		Matcher matcher = pattern.matcher(line);
    		if(matcher.find()) {
    			if(currentException == null) currentException = new StringBuffer(previous);
    			currentException.append("\n" + line);
    		} else if(currentException != null) {
    			Matcher nestedMatcher = nested.matcher(line);
    			if(nestedMatcher.find()) {
    				currentException.append("\n" + line);
    			} else {
    				if(ret == null) ret = new LinkedList<String>();
    				ret.add(currentException.toString().trim());
    				currentException = null;
    			}
    		}
    		previous = line;
    	}
    	return ret;
    }

This should be solved by the stack trace preprocessor in release botsing-1.0.4.

Fine, thanks. I also fixed my code (the snippet above) because it misses the last exception if EOF occurs within exception parsing. The regexp has also been modified, because logs don't always put tabs but sometimes blanks instead at the beginning of some exception stack lines (eg. tomcat logs do that).
Don't know if you use it or not, but new code follows:
    public List<String> extractExceptions(BufferedReader in) throws IOException {
    	LinkedList<String> ret = null;
    	String line;
    	String previous = "";
    	// Pattern more permissive than it could : should be ^\\t and never ^(\\t|\\s+)
    	// but required when exceptions are carelessly copied/pasted (with tabs replaced by spaces)
    	Pattern pattern = Pattern.compile("^(\\t|\\s+)at |^Caused by: |^(\\t|\\s+)... \\d+ more");
    	Pattern nested = Pattern.compile("; nested exception is: *$");
    	StringBuffer currentException = null;
    	while((line = in.readLine()) != null) {
    		Matcher matcher = pattern.matcher(line);
    		if(matcher.find()) {
    			if(currentException == null) currentException = new StringBuffer(previous);
    			currentException.append("\n" + line);
    		} else if(currentException != null) {
    			Matcher nestedMatcher = nested.matcher(line);
    			if(nestedMatcher.find()) {
    				currentException.append("\n" + line);
    			} else {
    				if(ret == null) ret = new LinkedList<String>();
    				ret.add(currentException.toString().trim());
    				currentException = null;
    			}
    		}
    		previous = line;
    	}
    	// Don't forget last exception if EOF at the midst of an exception stack
    	if(currentException != null) {
			if(ret == null) ret = new LinkedList<String>();
			ret.add(currentException.toString().trim());
    	}
    	return ret;
    }


auth: increase verbosity of oidc parsing errors

This is now ready, however the CI fails with unrelated errors. @labkode do you have an idea why?


GEODE-4517: Remove CacheFactory.getAnyInstance call from CliUtil

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

This PR is based off the HEAD of #1487
For initial review, ignore the first two commits.

Precheckin green.

Accidentally pushed the branch rebased on the newest develop, but no changes were made from the previous green precheckin.


fix(client): modernize stripe form

This pr migrates our client stripe implementation from react-stripe-elements to @stripe/react-stripe-js library and uses hooks to manage state.

Tested this locally with the test cards in the Stripe docs and everything LGTM.


Replace blobstore TestLargeAmountOfData with config lookup

TestLargeAmountOfData takes upwards of three minutes, so this patch replaces it with a test that reads the ssl renegotiation configuration from postgres by running psql in a container. This takes a second or less to perform.

LGTM

LGTM


persistent groupings

Related Issue
#206
Description
incomplete implementation with demo.
Bug: Table with different persistence-id still loads the saved groupings.
Impacted Areas in Application
GroupBar
MaterialTable
Additional Notes
I wanted to share my initial approach so you can comment on it :)
The deep copy of the columns array feels hacky, but that is probably just a JS thing...

Hi! Can we get the conflicts resolved on this?

I finally had some time to review this, somewhat "in-depth", but I really, really like this idea. It's an incredible idea!
The first thing that popped into my head, which may be dumb, is using cookies - since that will be "under the hood" - do we need to ask for permission to save cookies? Idk. Seems like a minor concern, just wanted to mention it.
I think this idea could be expanded upon (we have seen multiple issues where people using remote data want to "cache" how someone has selected rows, etc... on a previous page.
Do you have any opinion on making this "caching via cookies" more of a stand-out feature? What I mean by that is, imagine if we had some abstract way that would allow for the consumer to easily cache things that people seem to want cached. Or are we good with just overriding for now?
Great idea and implementation! Thanks for this.

Glad you like it!
I actually did choose local storage over cookies, as it is more suited for caching user data (far easier to use). The legal implications of using local storage or cookies are the same. GDPR applies to both and requires user consent. I rarely saw it in the wild, but here is a german example of a privacy policy mentioning local storage: https://www.zdf.de/dsgvo/datenschutz-156.html
I think making sure that whoever reads the docs about cache related features gets the hint that local storage is used and GDPR or local laws may apply, is enough. Consent should be handled by other librarys, maybe we can link to some popular ones in the docs.
Yeah I think row selection could be saved in a simlar way. Give the table an ID, save selected row IDs and check all matching rows once data is there. But I think building something like a central store to handle storing and loading data would be to much.
Using local storage is a one liner. We could collect savedGroupings and savedSelections in an object within the local storage to keep it a little cleaner.
{
  material-table-core-groupings: [{ id, data }, { id, data }],
  material-table-core-selections: [{ id, data }, { id, data }],
  other-local-storage-enty: 'foo',
}

@FalkF thank you! My fault, I should have realized you were using local storage.  Thanks for your feedback, I really appreciate the help.
@Domino987 what do you think? Are you good with merging this? I think adding this component improves things.

@Domino987 Bump :)

@Domino987 thoughts on this?

Technicality looks good, but there might be some side effects we are not aware yet. SO yes, lets merge it and add it to the website as well.

I dont want to stress anyone, but is anything else needed from my side or can I help by adding docs to the site? :)


Python Crash

Import the NetCDF-CF file linked below
Create a new Python variable named speed, and check the three 3D variables vx, vy, and vz
Leave the script field blank (or enter anything, it doesn't matter) and click "Test"
Vapor will crash due to not being able to find a time coordinate variable

Data: https://drive.google.com/open?id=1bxftiSGwyRFYsQazu4mACDHlhYvLGS-u
Note we can render without a time coordinate variable, but we crash in python.

The test data set for this has been added to /glade/p/cisl/vast/vapor/Bugs/1806


Thumb2: RRX

Version and Platform (required):

Binary Ninja Version: 3.4.4251-dev, 3f761e0a
OS: macos
OS Version: 13.3
CPU Architecture: arm64

IDA Pro
ROM:00004B76                 MOV.W           R1, R1,RRX

BN
🚫000004b76  4fea3101   rrx     r1, r1

Why the difference?

V35 should search for "Encourage Salesman Prompt Delay" to find the database.

This instruction is a preferred synonym for MOV instructions with shifted register operands. IDA isn't wrong - its output is what you'll find in the ARM manual. But, neither are we.


Improve Radial Chutes In Symmetry Handling

Current system is too basic (checks if even amount exist) and punishes player if they use an odd number even if they are in symmetry.

Recoded system. Now checks programmatically for symmetry and calculates accordingly


fix: do not allow using ref functions after provider has been disposed

fixes #3522
Using ref functions of the disposed provider can lead to unexpected behaviors e.g providers may never be disposed and hang around indefinitely if subscribed to using Ref of a disposed provider.
@riverpod
Future<void> future1(Future1Ref ref) async {}

@riverpod
Future<void> future2(Future2Ref ref) async {
  await Future.delayed(const Duration(seconds: 3));
  await ref.watch(future1Provider.future);
}

In the above example future1Provider will still be watched forever if future2Provider is disposed before timer expires.
Will handle checklist items if this even correct solution.

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).

[ ] I have updated the CHANGELOG.md of the relevant packages.
Changelog files must be edited under the form:
## Unreleased fix/major/minor

- Description of your change. (thanks to @yourGithubId)

[ ] If this contains new features or behavior changes,
I have updated the documentation to match those changes.

oh great, then it should be closed.


Upgrade to tower-lsp 0.14.1 and tokio 1.6

Changed

Upgrade to tower-lsp 0.14.1 (see release notes for 0.14.0 and 0.14.1).
Upgrade to tokio 1.6 (single-threaded flavor, like before).

Fixed

Use async-aware tokio::sync::Mutex instead of std::sync::Mutex to avoid stalling the tokio executor.

Removed

Remove unused lsp-types dependency from Cargo.toml.

New release of tower-lsp is finally out, as of a few days ago. Please let me know if you run into any issues! :heart:

thanks


Storybook 6.5 peerDependencies react 18.1.0

With the new 6.5.x Storybook release and your package as dependency will break the 'npm install'.
Due the addons with ' || ^18.0.0' in it comes react '18.1.0' but your plugin dont accept that.
https://github.com/chromaui/storybook-addon-pseudo-states/blob/d58f2f794da324d0ebaa87a0bedb10f21119dd47/package.json#L58
    "react": "^16.8.0 || ^17.0.0",
    "react-dom": "^16.8.0 || ^17.0.0"

https://github.com/storybookjs/storybook/blob/4b849f9bcd1d76bd6a016d50f7f1cb8f7809ed31/lib/addons/package.json#L55
    "react": "^16.8.0 || ^17.0.0 || ^18.0.0",
    "react-dom": "^16.8.0 || ^17.0.0 || ^18.0.0"

or maybe i'm wrong :D
cheers

@chromaui Any movement on this?


PunkBuster False Negatives

SteamDB link to the game
https://steamdb.info/app/203290/
https://steamdb.info/app/24960/
https://steamdb.info/app/209870/
https://steamdb.info/app/2620/
https://steamdb.info/app/2630/
https://steamdb.info/app/17300/
https://steamdb.info/app/9050/
https://steamdb.info/app/21090/
https://steamdb.info/app/21120/
https://steamdb.info/app/13520/
https://steamdb.info/app/220240/
https://steamdb.info/app/9460/
https://steamdb.info/app/226700/
https://steamdb.info/app/32770/
https://steamdb.info/app/389430/
https://steamdb.info/app/47790/
https://steamdb.info/app/3970/
https://steamdb.info/app/2210/
https://steamdb.info/app/2200/
https://steamdb.info/app/35450/
https://steamdb.info/app/9010/
https://steamdb.info/app/234510/
https://steamdb.info/app/19830/
https://steamdb.info/app/15000/
https://steamdb.info/app/13540/
https://steamdb.info/app/880850/
https://steamdb.info/app/1873030/
What should it be detected as?
PunkBuster

You can cross off most of those now. Unfortunately, i couldn't figure out the rest.
https://github.com/SteamDatabase/FileDetectionRuleSets/pull/483

You can cross off most of those now. Unfortunately, i couldn't figure out the rest. #483

also, when you create a pr to fix the issue please add "closes #issue" to the description, so it does not have to be closed manually after the merge

I would have done that if i could completely resolve the issue.


Dotenv wrap environments with =

Laravel Version: 5.8.4
PHP Version: 7.3.2

Description:
Dotenv failed parsing environments contained =
Steps To Reproduce:
php artisan passport:keys --force
cat storage/oauth-public.key
cat storage/oauth-private.key

and insert it's in .env as a multiline strings like laravel/password environments (PASSPORT_PRIVATE_KEY, PASSPORT_PUBLIC_KEY)
Links
Related: #27850

Sorry, problem not in =, but multiline environments not works as expected

Problem solved: we drop docker-compose env_file configuration option, because not supported multilines variables.
https://github.com/docker/compose/issues/3527


layout mobile friendly

A couple of changes, use grid layout so it's easy to make it mobile friendly.

@radames Excellent thank you!


component list F450 quadrotor

Dear MRS team,
Would it be possible to provide the component lists that I would need to order to build and fly the exact same F450 drone as you have?

450 frame kit
motors: are these the ones?
props: are these the ones?
ESCs: are these ESCs the ones you use? Are these the ones that give the issues when the motors are braking?
battery: you use 4S, which capacity and C rating? What is nominal flight time?
radio transmitter? Do I need a radio transmitter for each member of the swarm?
receiver
pixhawk 4, is plastic casing good enough?
which SD card for pixhawk is recommended (R/W speed, storage)?
companion computer: which exact intel NUC version and specs (CPU, GPU, RAM, SSD...) would be the least that has enough compute power for running your software? Which one do you recommend? Does it need a fan? Does it require an Ethernet port or any other ports to certainly add? Do you recommend one with or without a fan? Do you just buy the full NUC and disassemble it to put only the board on the drones? How is the NUC powered? Is an Intel processor required to run your software or should AMD also work without issues? Is your software effectively using the multiple CPU cores or would you advice to invest in a stronger lower core CPU?
which exact connection cable between intel NUC and PX4 do you use? It seems to be an FTDI serial to usb cable, but which one?
wifi modules for NUC?
wifi ground based router for swarm communication?
normal gps? which rtk gps do you use?
other sensors?
telemetry module?
ground station hardware required to fly a small swarm?
anything else?

Is there anything that you custom 3D printed? Would you mind sharing the .STL files for 3D printing?
Thanks!
Best,
Bryan

Hello, sorry for the late reply.

F450 frame kit, we buy the copies from china since the original is not made anymore
Motors are correct
We use the original DJI props, we found out that some of the copies have worse performance, but they still work
We use these ESCs: https://hobbyking.com/en_us/turnigy-multistar-32bit-51a-race-spec-esc-2-6s-opto.html?queryID=b8f5dd1784099a8f3e6aa86811b2b4a1&objectID=74316&indexName=hbk_live_magento_en_us_products  There is no problems with them.
Battery is Tattu 4S 6750 mAh 25C
Receiver is Optima SL, transimtter is Hitec Flash 8, each drone has to have its own receiver, but with larger swarms we have just one transmitter connected to all the receivers.
Pixhawk 4 plastic case is fine, all our drones have them.
Any SD card will do for pixhawk, it is just for config files and low rate logging.
Currently we recommend NUC10i7FNK with 16 GB RAM (8 GB works too, but compilation can sometimes run out of memory, but no issues during runtime) and at least 128 GB SSD, more if you want to record a lot of data from your flights. We buy the NUCs without RAM and SSD, then disassemble them, install the RAM and SSD and mount them on the drones. The NUC is powered straight from the battery, it has 12-19V input voltage range, so a 4S battery can be plugged right in the NUC. AMD can also run our system. The system is using multiple cores, so more cores is good.
We are using the regular FTDI clone boards like this - https://www.aliexpress.com/item/4001323460121.html?spm=a2g0o.productlist.0.0.7b83e8dcbCXILT&algo_pvid=1c2070da-7f8f-47ad-a8a0-83f3b4ca9cb6&algo_expid=1c2070da-7f8f-47ad-a8a0-83f3b4ca9cb6-5&btsid=2100bdd016034491508824193e00b8&ws_ab_test=searchweb0_0,searchweb201602_,searchweb201603_
One end is connected to the TELEM2 port, the other goes to the NUC USB. The NUC has two USB ports on Picoblade headers, we use those to connect the FTDI board (this is a much tougher connection, but it requires to make a custom cable, desolder the mini USB from the FTDI board and replace it with directly soldered cables).
NUC already has a WIFI module integrated, we use that one.
For bigger swarm experiments, we attach a WIFI router to one of the drones, so that they can carry it with them, otherwise we leave it somewhere on the ground. We use this WIFI router - TP-LINK TL-WR902AC as it can be powered through USB (directly from the NUC or from a powerbank or laptop).
We use this GPS - https://shop.holybro.com/pixhawk-4-gps-module_p1094.html
We have the Emlid Reach 2 RTK on order right now. We used the first generation and it works well, so I can recommend it.
Other sensors include Garmin Lidar Lite to measure altitude, Bluefox  global shutter cameras for UVDAR/OpticFlow/VIO/Other stuff, Realsenses, RPLidar A3, Ouster 3D lidars, Terabee and FLIR thermal camers and more, I cannot recall all of them :D.
We do not use any telemetry modules, we communicate with the drone over WIFI, or with the RC transmitter.
We do not have any dedicated ground stations, only thing you need is just a laptop and a WIFI router.
We print most of the sensor mounts, the landing legs, battery holders and other things. I made a new repo for the 3D models - https://github.com/ctu-mrs/3D_models and put some of the F450 models there. If you need other models we can add them as well.

Thanks a lot @DanHert for the detailed responses!
Some follow up questions:

should your ESCs be used in order to set  the required ESC configs? Would these cheaper ESCs be giving the same results or does it require some change of the config parameters?
do you think this cheaper transmitter is ok. Or does your transimtter Hitec Flash 8 offer some additional features which are required?
For what concerns the pixhawk's SD card I read here dropouts can result in wrong logged data. PX4 proposes the SanDisk Extreme U3 32GB, since it is least sensitive to dropouts. Did you experience dropouts and do you have an effective strategy to get rid of them?
I had already ordered these boards, do you think they should work too?
So with 1 x $1899 REACH RS2 the pixhawk 4 GPS on each drone in a swarm becomes centimeter precise? Or do you need >1 base station? What's the reason you upgraded from the REACH RS+?

Thanks!
Bryan

The other ESCs should be fine, there is no need to use our exact config
We only use Hitec Flash or Hitec Aurora transmitters which work well. I cannot guarantee you if the cheap one will work too, but probably yes.
We only check the Pixhawk logs when we have a crash/failsafe land which cause we cannot deduce from the outputs of the mrs_uav_system. And if the card is not full, the logs are always there, never been a problem. We use the microSD card which comes with the Pixhawk 4.
yes, these boards shoud be ok.
You need one RS2 as a basestation, and then each drone needs its own reach M2 + LoRa radio for corrections. The Reach RS+ is a singleband receiver, while RS2 is a multiband one, which means faster fix time, longer possible baseline, better performance in worse conditions (blocked sky) etc...

Hi @DanHert ,
Which cabling and connectors do you use to power up and transmit signals from the Emlid reach M2 and the Lora radio to other components of the drone F450 drone?
Which Emlid specific cabling/connectors do you use (if any) and which cabling/connectors did you buy elsewhere for making your RTK solution work?
Thanks for the help!
Best,

Hi @DanHert ,
Which cabling and connectors do you use to power up and transmit signals from the Emlid reach M2 and the Lora radio to other components of the drone F450 drone?
Which Emlid specific cabling/connectors do you use (if any) and which cabling/connectors did you buy elsewhere for making your RTK solution work?
Thanks for the help!
Best,

Hello, I tried to power the Emlid directly from pixhawk, but pixhawk does not provide enough power. So our solution is to connect the emlid with USB to the NUC. So no additional cables are needed, just the one to connect lora and then a USB cable to the NUC.

Hello, I tried to power the Emlid directly from pixhawk, but pixhawk does not provide enough power. So our solution is to connect the emlid with USB to the NUC. So no additional cables are needed, just the one to connect lora and then a USB cable to the NUC.

Btw this also means that the RTK data will not get into pixhawk, only the NUC. So if you want to fly with RTK in feedback, you need to fly autonomously using the onboard computer, pixhawks position hold will still only use the regulart GPS. You can pipe the rtk data into pixhawk too, you just need an aditional cable.

Btw this also means that the RTK data will not get into pixhawk, only the NUC. So if you want to fly with RTK in feedback, you need to fly autonomously using the onboard computer, pixhawks position hold will still only use the regulart GPS. You can pipe the rtk data into pixhawk too, you just need an aditional cable.

@DanHert , I want to fly using RTK gps as position feedback using mrs uav system. So if I understood you correctly,  I just need 1 usb from emlid reach M2 to NUC and I don't need to pipe RTK data from NUC to pixhawk?

@DanHert , I want to fly using RTK gps as position feedback using mrs uav system. So if I understood you correctly,  I just need 1 usb from emlid reach M2 to NUC and I don't need to pipe RTK data from NUC to pixhawk?

Yes, that is correct.

Yes, that is correct.


Initiate the benchmark script for Linux

Add utils to get Linux device information.
Move common arguments to its own module.

Ping : )


Support module wrapping for debugging on Linux

In addition to the added tests, this fixes Driver/modulewrap.swift. I did also find a bug while working on this where we aren't always planning an autolinking job when we should. I'm pretty sure that's an unrelated issue with how we're setting the object file format of triples though.

@swift-ci test Linux

@swift-ci please test Linux

@swift-ci please test Linux

@swift-ci please test Linux

@swift-ci please test Linux

@swift-ci please test Linux

@swift-ci please test Linux

@swift-ci please test Linux

@swift-ci please test Linux

@swift-ci test

@swift-ci please test macOS

This is breaking Swift CI (https://ci.swift.org/job/oss-swift-incremental-RA-linux-ubuntu-18_04/8618/consoleFull#-6367346333122a513-f36a-4c87-8ed7-cbc36a1ec144):
<EXPR>:0: error: BuildPlanTests.testExplicitSwiftPackageBuild : threw error "unableToFind(tool: "swift-autolink-extract")"
02:34:23 Test Case 'BuildPlanTests.testExplicitSwiftPackageBuild' failed (0.662 seconds)
02:34:23 Test Suite 'BuildPlanTests' failed at 2020-08-11 09:33:59.671
02:34:23 	 Executed 1 test, with 1 failure (1 unexpected) in 0.662 (0.662) seconds
02:34:23 Test Suite 'Selected tests' failed at 2020-08-11 09:33:59.672
02:34:23 	 Executed 1 test, with 1 failure (1 unexpected) in 0.662 (0.662) seconds