Spaces in file names results in NotFoundError

I have some files that have spaces in their names. Like Polykey Logo.svg.
When requested from the browser, it looks like https://polykey.io/docs/images/Polykey%20Logo.svg.
When using getAssetFromKV, this ends up not working and throws a NotFoundError.
When I changed the file name to polykey-logo.svg, it all worked fine.
I saw this is an old issue cloudflare/kv-asset-handler#75 that said it was fixed, but this is definitely not the case.
I'm using:
    "@cloudflare/kv-asset-handler": "^0.2.0",
    "wrangler": "^2.0.23"

And my compatibility date:
    "wrangler": "^2.0.23"

We haven't heard from you in while so I'm going to close this issue for now. If you're still running into problems feel free to comment with more details and we can investigate further.


 Now we want to change our
implementation to use responseDisposition with File.getSignedUrl.
But in order to use this, we have to clear metadata.contentDisposition which is impossible with the current API.  All we can do, is use File.setMetadada({contentDisposition: ""}) but this will not remove metadata.contentDisposition but set it to an empty string.
Because the content-disposition set in an object's metadata always takes precedence over File.getSignedUrl({responseDisposition: ...}) the resulting Content-Disposition-header is always an empty string.
Reading on the JSON-API I see that there is actually a update operation that allows to completely (re-)set an object's metadata. But this operation is not exposed in the NodeJS-API.
Any hint to a workaround for the meantime would be greatly appreciated.

FYI, the workaround for my current problem (Content-Disposition cannot be removed) is as follows:
        const [metadata] = await file.getMetadata()
        const [acl] = await file.acl.get()
        delete metadata.contentDisposition
        await (file as any).request({
            method: "PUT",
            uri: "",
            qs: {
                userProject: blob_bucket.userProject,
            },
            json: {
                acl,
                ...metadata,
            },
        })

You should be setting contentDisposition to null to delete it.  From the docs:
      /* // Assuming current metadata = { hello: 'world', unsetMe: 'will do' }
       * file.setMetadata({
       *   metadata: {
       *     abc: '123', // will be set.
       *     unsetMe: null, // will be unset (deleted).
       *     hello: 'goodbye' // will be updated from 'world' to 'goodbye'.
       *   }
       * }, function(err, apiResponse) {
       *   // metadata should now be { abc: '123', hello: 'goodbye' }
       */ });

Stupid me. This works totally fine. I close this issue now.


Unit Tests

While testing, I figured I'd upgrade deck.gl to the next version since I didn't want to have to deal with testing utils we'd never use.  I also changed the names of the layers since we're testing now.
I have also taken testing off travis and made it into a pre-push hook.  I don't think it's worth my sinking more time into getting WebGL2 enabled in a headless browser only on travis.
Fixes #6 and #101

@ilan-gold :

Fixes #6 and #101

It's not smart enough to autoclose 100 unless "Fix" is right in front of it. (I'll close it now.)


MFILE format consistency

In GitLab by @emiralle on Jul 17, 2019, 23:22
I'm currently coupling COSSAN and PROCESS to perform UQ analyses with this software. COSSAN can be applied to 3rd party solvers (e.g, PROCESS).
The way it works is by treating this 3rd party solver as a black-box. COSSAN writes an input file (IN.DAT) and reads an output file (MFILE.DAT).
The MATLAB class in charge of reading the solver output is called Extractor and there are a few ways to define how are we going to read the data. The simplest way is by associating the output of a variable to a specific number of row and column. There are other methods, but all of them suffer of the problem I'm going to explain next.
PROCESS (as far as I'm concerned) has three different MFILE formats. For example, when ifail=1 the MFILE looks like this:
 # PROCESS #
 # Power Reactor Optimisation Code #
 # PROCESS #
 # Power Reactor Optimisation Code #
 PROCESS_version_number__________________________________________________ (procver)_____________________     "1.0.13   "
 Date_of_run_____________________________________________________________ (date)________________________     "15/07/2019"
 Time_of_run_____________________________________________________________ (time)________________________     "23:13"
 User____________________________________________________________________ (username)____________________     "enmidol"
 PROCESS_run_title_______________________________________________________ (runtitle)____________________     "Run Title (change this line using input variable 'runtitle')
 PROCESS_tag_number______________________________________________________ (tagno)_______________________     "1.0.13-499-g78dfc55"
 PROCESS_last_commit_message_____________________________________________ (commsg)______________________     "Reformat lines for inclusion in .bashrc as code.,"
 # Numerics #
 VMCON_error_flag________________________________________________________ (ifail)_______________________              1

But when ifail=6 we have this:
 # PROCESS #
 # Power Reactor Optimisation Code #
 # PROCESS #
 # Power Reactor Optimisation Code #
 PROCESS_version_number__________________________________________________ (procver)_____________________     "1.0.13   "
 Date_of_run_____________________________________________________________ (date)________________________     "15/07/2019"
 Time_of_run_____________________________________________________________ (time)________________________     "23:13"
 User____________________________________________________________________ (username)____________________     "enmidol"
 PROCESS_run_title_______________________________________________________ (runtitle)____________________     "Run Title (change this line using input variable 'runtitle')
 PROCESS_tag_number______________________________________________________ (tagno)_______________________     "1.0.13-499-g78dfc55"
 PROCESS_last_commit_message_____________________________________________ (commsg)______________________     "Reformat lines for inclusion in .bashrc as code.,"
 # Numerics #
 # PROCESS COULD NOT FIND A FEASIBLE SOLUTION #
 # PROCESS COULD NOT FIND A FEASIBLE SOLUTION #
 VMCON_error_flag_(ifail)________________________________________________ ______________________________              6

So there are two extra rows, and we won't be actually reading the rows where we think our variables are.
The other format appears when error_status = 3:
 # PROCESS #
 # Power Reactor Optimisation Code #
 # PROCESS #
 # Power Reactor Optimisation Code #
 PROCESS_version_number__________________________________________________ (procver)_____________________     "1.0.13   "
 Date_of_run_____________________________________________________________ (date)________________________     "15/07/2019"
 Time_of_run_____________________________________________________________ (time)________________________     "23:13"
 User____________________________________________________________________ (username)____________________     "enmidol"
 PROCESS_run_title_______________________________________________________ (runtitle)____________________     "Run Title (change this line using input variable 'runtitle')
 PROCESS_tag_number______________________________________________________ (tagno)_______________________     "1.0.13-499-g78dfc55"
 PROCESS_last_commit_message_____________________________________________ (commsg)______________________     "Reformat lines for inclusion in .bashrc as code.,"
 # Errors and Warnings #
 # Errors and Warnings #
 PROCESS_error_status_flag_______________________________________________ (error_status)________________              3
 Final_error_identifier__________________________________________________ (error_id)____________________             74

This is only a problem when instead of looking for determined numbers of rows and columns we look for keywords -this is one of the options available with the Extractor class-. COSSAN would enter in a loop trying to find keywords in the MFILE that don't actually exist.
If you use the Extractor with rows and columns you would be able to perform UQ analyses, but when ifail=6 and error_status=3 the variables you look for in the output will get wrong values (when the variables are shifted 2 rows forward) and NaNs. This is not really a problem if you are only looking for successful runs (ifail=1), as COSSAN will find the values for the defined variables (you can then filter the arrays saving the ones that correspond to ifail=1, as you know those would be the real values). However, you won't be able to explore regions of unfeasible runs (ifail=6).
So possible solutions:
The easiest way to fix this would be to remove the two extra rows included in the MFILE when ifail=6. These only state that PROCESS couldn't find a possible solution, so no major data would be lost with this.
This fix is suitable if we are interested in defining output variables in COSSAN using rows and columns, which can be a bit tedious. If we want to extract these variables by just telling COSSAN the name of the variable in the MFILE, we would have to solve the issue with the error_status=3, as COSSAN would enter in a loop trying to find keywords that are not inside.
I think the explanation may not be fully clear as you are not familiar with the COSSAN interface, and I feel this would be much easier to explain with a proper live example. Anyway, if you have any question please do let me know.
Thanks,
Enrique
MFILE_ifail1.DAT
MFILE_ifail6.DAT
MFILE_errorstatus3.DAT

We provide MFILE readers in the io subpackage of PROCESS already, and this issue is outdated


[DS-4031] Updated link to DRIVER guidelines.

https://jira.duraspace.org/browse/DS-4031

Merging, this is obvious, just a comment change.


Better docz 😃

Switch to Docz
Noticeable features

Fully-customizable
Powered by Gatsby
MDX support
Supports JavaScript and TypeScript
Plugins and themes

Can you try and run this locally and see how this is going to look? Just as an example. It's hard to decide if this is ready to merge if we can't see the final version.
Note: While your doing this maybe you can look into seeing if it is possible to host this through github pages. That wasmy biggest limitation last time and I would ideally like docs to be on github pages.

Can you try and run this locally and see how this is going to look? Just as an example. It's hard to decide if this is ready to merge if we can't see the final version.

I will add some screenshots of the generated website. Also, you can fully-customize the theme if you do not like the current style.

Note: While your doing this maybe you can look into seeing if it is possible to host this through github pages. That wasmy biggest limitation last time and I would ideally like docs to be on github pages.

And yes, it does support GitHub Pages.


[graphql] add timestamp for TransactionBlock

Description
This PR adds the timestamp field for the TransactionBlock type. It also adds a corresponding example for querying a transaction block by its digest.
Test Plan
How did you test the new or updated feature?

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

Thanks @stefan-mysten ! Curious what was the motivation for this change (because I notice this wasn't one of the fields in the existing GraphQL schema)? Here's the high level review:
* Please also add this change to the draft schema.

* What does this timestamp represent? Is it the time the transaction was signed, executed, finalised, indexed? Please update the docs to reflect which one it is.

* I'm hoping it's the time that the transaction was finalised (🤞), because that's the most useful of these timestamps, in which case, that belongs on `TransactionBlockEffects`, not `TransactionBlock`.

Awesome, thank you for asking the right questions. When firing up the Explorer and checking a transaction block, we get the timestamp information: https://suiexplorer.com/txblock/EGoc6mhhXg4fm6vmg4od3P6jyDaAMhNJ73uiD96M97KF
To be honest, I do not know precisely which timestamp this is in the explorer, but I assume it is the same as the checkpoint's timestamp.
For the code I wrote, that timestamp is the indexed transaction's timestamp. Currently, if we want to get the transaction block's timestamp, one has to fetch its related checkpoint, and get its timestamp. That's an extra query to the DB, so I was hoping that we can avoid that. TransactionBlockEffects does not have a timestamp field, so maybe we should add it there instead.

Just double checked: the timestamp comes from CheckpointHandler (IndexedTransaction (indexer)  -> StoredTransaction (indexer) -> TransactionBlock (graphql)

@amnn - I will move this field to TransactionBlockEffects


Change water_heater devices as per architectural discussion

Description:
This adds on/off functionality along with the setting of the temprature for a water_heater device.
Code is based on the climate device.
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#7767
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io (PR pending)

Current temperature would be nice to have while you are at it.

In that case, a good reference is the architecture discussion for boiler that was sidestepped when water heater was added a good reference point. https://github.com/home-assistant/architecture/issues/59

In that case, a good reference is the architecture discussion for boiler that was sidestepped when water heater was added a good reference point. home-assistant/architecture#59

I agree - it seems the current water_heater implementation was created/deployed with very little appreciation of that discussion.  IIRC, it was implemented during the most recent hacktoberfest...
Currently, the water_heater has no current_temperature!!
Nor is it able to tell you if it in on, or off!

Changing the entity model requires approval in an issue in our architecture repo.

FWIW, I have started a discussion.

I have reviewed this PR.

current_temp must go back in.  If it is not required fro a particular entity, the support flags can be set appropriately, and it will be a non-breaking change.

state should be STATE_OFF, STATE_ON, or STATE_HEATING, as per https://github.com/home-assistant/architecture/issues/59

@zxdavb I agree I also would like to see the ON/OFF atleast.
One problem though is that the ON/OFF wasn't working yet with this PR and I haven't taken a look yet... Hence the WIP label.

I just added the current temprature back in as well to have the changes all in one PR :-)!

Change title to: "Change water_heater devices as per architectural discussion"

This PR seems to have gone stale. Closing it.


feat: treat kcl.mod file as a compile unit

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

feat: treat kcl.mod file as a compile unit

2. What is the scope of this PR (e.g. component or file name):

feat: treat kcl.mod file as a compile unit
kclvm/config/src/modfile.rs
kclvm/config/src/vfs.rs
kclvm/driver/src/lib.rs

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

kclvm/config/src/testdata/kcl.mod

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
64 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.009%) to 71.082%

Files with Coverage Reduction
New Missed Lines
%

kclvm/config/src/cache.rs
4
98.06%

kclvm/config/src/modfile.rs
10
93.67%

kclvm/driver/src/lib.rs
50
82.32%

Totals

Change from base Build<PHONE_NUMBER>:
-0.009%

Covered Lines:
53860

Relevant Lines:
75772

💛  - Coveralls


qualquertítulo

asd asdf

@HCley Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


implement tutorials catalog

PR#
Parent
Child

4/4
#661
none

Added

Added a tutorial catalog

Created litdev-tutorial-card

A data pipeline that renders and creates tutorial manifests

Particular items that need review

[ ] Data pipeline architecture

[ ] packages/lit-dev-content/site/tutorials/tutorials.11tydata.js

[ ] Component implementation

[ ] packages/lit-dev-content/src/components/litdev-tutorial-card.ts

[ ] UI of the catalog

[ ] packages/lit-dev-content/site/tutorials/index.html

Access at the preview link below + /tutorials/
How should we go about the categories?
Do the categories look good?
Arthur should take a look at the wording of the sections and possibly even the sections that should exist

Build may be broken?

It's up now. Just took a while to deploy.

Woah! Just tested this with many elements (copy pasted) and checked responsiveness. It looks SO good!!!

Just copy pasted the existing blocks and updated sizes. It's responsive. Can tab through. Screenreader works great!

@AndrewJakubowicz added a ?mods=tutorialCatalog mod


hotfix(changelog) missing v0.9.1 fixes/additions

Full changelog

Added missing fixes/additions to the CHANGELOG for 0.9.1

Should have been manually merged to both next+master


fix: pin to major version 1

🚀 Deployed on https://66c50cb1f9bb2a5ff5c13293--cozy-fox-0defec.netlify.app


Add --force option to knife acl add to override best practice

Fixes #37

I'm 👍. @jeremiahsnapp thoughts?

Sounds great. @drrk can you also add that to acl_bulk_add.rb please?

Closing this out since we never heard back


Rendering components from views

It may be useful to render components from views. I came across this while trying to use slippers components with HTMX.

Hey @mixxorz, I was testing the same thing this morning as the stack I'm working on is django with htmx.
So what I found out is that it does work, even if you wrap the htmx "call" in a fragment you could use it as a parameter to another slipper component:
# inventory/views.py

class IOHistoryList(generic.ListView):
    model = IOHistory
    slug_field = 'code'
    template_name = 'ui/inventory/IOHistory.html' # <-- this is a slipper component

    def get_queryset(self):
        queryset = super().get_queryset()
        return queryset.filter(
            product__code=self.kwargs.get('code')
        )

# ui/templates/IOHistory.html

IO History

    {% for history in object_list %}

        {{ history.date|date:'Y/m/d' }}

          {% if history.quantity_in %}
            ↓{{ history.quantity_in }}
          {% else %}
            ---
          {% endif %}

          {% if history.quantity_out %}
            ↑{{ history.quantity_out }}
          {% else %}
            ---
          {% endif %}

    {% endfor %}

# templates/inventory/product_detail.html
....
{% fragment as ioHistory %}

{% endfragment %}

{% Card header=cardHeader content=ioHistory %}

The page is loaded just fine

And also the "htmx" response from my slipper component

And I think that works just fine because the slipper component it's nothing different than a Django template, the only scenario where this may not work is if you're not rendering the template in your views, but would appreciate if you could include the error you faced bc I'm deciding to use slipper on a mid-size project here and would be great to know any possible issues in advance o/


data storage

Thank you very much for your experiments and new ideas. So far, I have successfully run your code and successfully trained to get the output image, but I want to use the newly generated image. How do I save the output image separately? Instead of just saving the long picture in figures

Hi, thanks for your interest! You'll likely have to modify the _make_results_im function in either transform_models.py (https://github.com/xamyzhao/brainstorm/blob/master/src/transform_models.py#L620) or segmenter_model.py (https://github.com/xamyzhao/brainstorm/blob/master/src/segmenter_model.py#L777).

Hello, I have a question. What do the vol and seg data in atlas, train, and validation in the data folder correspond to? If vol is data and seg is label, why both atlas and train files? This makes me a little puzzled.

When I was training my stroke data set, the data input size was (8, 256, 256), and I also encountered the same problem as another person: when I trained transform color and --aug_sas, the code pointed to you before The trained model will report an error: ValueError: Error when checking input: expected input_1 to have shape (160, 192, 224, 1) but got array with shape (8, 256, 256, 1). How can I solve this problem?

@cue1997 if you search for the term "160, 192, 224" in this repository, you can see the places where this input size is defined: https://github.com/xamyzhao/brainstorm/search?q=160%2C+192%2C+224. You'll likely want to update all of these to reflect the sizes in your dataset.

I'm sorry I didn't explain my situation clearly. I have modified all (160, 192, 224) places, and can train flow-fwd, flow-bkd and mri-100unlabeled --aug_rand. But when training color-unet and --aug_sas, the error mentioned above will be reported

Are you sure you are loading your own trained model? Your first error sounds like your model is still expecting the old input shape, which implies that you are loading the pretrained model.

Hello, thank you very much for your previous guidance, I am very sorry for not responding to you in time. My data is ready for training, but the training effect of the stroke data set I used is very poor. Have you tried stroke data training before? I put a ct image in vol and a mask in seg. Is there a problem with my file placement method?

Can you elaborate on what you mean by the "training effect is very poor"? Does this mean that you are able to train, but you observe that the registration (spatial transformation) performance is bad?
I would expect the registration performance to be worse on a 2D dataset than on a 3D dataset, since this method does not explicitly handle occlusions.

Yes, the results of the training pictures are not good. Could you give me your email, I can package my code and data and send it to you


timestamp for AV material

As an end user,
I would like all AV materials to have a time stamp while playing,
so I can mark and skip to relevant sections.

Related to issue #76

Already covered in user stories to view A/V objects. Closing.


Does BatchUpdate Work With Top/Take?

The Batch update code looks like it does not work when the query has a Take commandin it.  Take generates a first line that looks something like this:
"SELECT TOP(@__p_0) [d].[ColumnName]"
which cause GetBatchSql to not be able to correctly determine the tableAlias, plus, the entire Top statement will get cut out of the final Update Sql.
                string escapeSymbol = (databaseType == DbServer.SqlServer) ? "]" : "."; // SqlServer : PostrgeSql
                tableAlias = sqlQuery.Substring(8, sqlQuery.IndexOf(escapeSymbol) - 8);

I think there needs to be code to look for Top and produce the correct Update statement.  If Skip and Take are used together, it will result in entirely different Sql, so that should also be addressed.

I have the same problem, but in BatchDelete. The 'TOP(100)' statment is not complete, this is the result: "DELETE [OP(100) [table_alias]
FROM..."

@borisdj I made this if you want to check it, this could inspire you for the enhancement:
` private static Task<(string TopStatement, string TableAlias, string Sql)> GetBatchSql(IQueryable query) where T : DomainEntity
{
var sqlQuery = query.ToSql();
var selectStmLength = "SELECT".Length;
var escapeSymbolEnd = "]";
var escapeSymbolStart = "[";
string tableAliasEnd = sqlQuery.Substring(selectStmLength, sqlQuery.IndexOf(escapeSymbolEnd) + 1 - selectStmLength).Trim(); // "TOP(10) [table_alias]" / "[table_alias]"
int tableAliasStartIndex = tableAliasEnd.IndexOf(escapeSymbolStart);
string tableAlias = tableAliasEnd.Substring(tableAliasStartIndex); // "[table_alias]"
string topStatement = tableAliasEnd.Substring(0, tableAliasStartIndex); // "TOP(10) " / if TOP not present in query this will be a Substring(0,0) == ""
        int indexFROM = sqlQuery.IndexOf(Environment.NewLine);
        string sql = sqlQuery.Substring(indexFROM, sqlQuery.Length - indexFROM);
        sql = sql.Contains("{") ? sql.Replace("{", "{{") : sql; // Curly brackets have to be escaped
        sql = sql.Contains("}") ? sql.Replace("}", "}}") : sql;

        return Task.FromResult((topStatement, tableAlias, sql));
    }`

@sky-daniel-paula
I have accepted your commit and new NuGet 3.0.3 is published.
Thx for contribution.


Give a less confusing example for requirements parsing

In the API reference for requirements parsing:
https://setuptools.readthedocs.io/en/latest/pkg_resources.html#requirements-parsing
There is offered an example PickyThing<1.6,>1.9,!=1.9.6,<2.0a0,==2.4c1 that strongly implies commas mean logical OR.  But PEP440 says commas mean logical AND.
I've edited it into a more realistic example to avoid this potential confusion.
Related is https://github.com/pypa/pip/issues/2744, which was promptly closed some time ago but never actually fixed (maybe just a simple oversight - ping @dstufft?)

Thanks for the explanation and fix!


Refine staticFilePath to perform sanity check

I have:

[x] Based work on latest develop branch
[x] Followed the contribution guide
[x] Looked for lint in my changes with hlint . (lint found code you did not write can be left alone)
[x] Run the test suite: $(nix-build -A selftest --no-out-link)
[ ] Updated the changelog
[ ] (Optional) Run CI tests locally: nix-build release.nix -A build.x86_64-linux --no-out-link (or x86_64-darwin on macOS)

Damn it :/ I am forgetful


Build failing for MVC Deployment

OSX, running "dotnet build"
DOTNET VERSION: 2.1.301
Blazor app project file:
<Project Sdk="Microsoft.NET.Sdk.Web">
  <PropertyGroup>
    <TargetFramework>netstandard2.0</TargetFramework>
    <RunCommand>dotnet</RunCommand>
    <RunArguments>blazor serve</RunArguments>
    <LangVersion>7.3</LangVersion>
  </PropertyGroup>
  <ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore.Blazor.Browser" Version="0.5.0-preview1-10334" />
    <PackageReference Include="Microsoft.AspNetCore.Blazor.Build" Version="0.5.0-preview1-10334" />
    <DotNetCliToolReference Include="Microsoft.AspNetCore.Blazor.Cli" Version="0.5.0-preview1-10334" />
  </ItemGroup>
</Project>

MVC app project file:
<Project Sdk="Microsoft.NET.Sdk.Web">
  <PropertyGroup>
    <TargetFramework>netcoreapp2.1</TargetFramework>
  </PropertyGroup>
  <ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore.App" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.Sqlite" Version="2.1.1" />
    <PackageReference Include="Microsoft.VisualStudio.Web.CodeGeneration.Design" Version="2.1.1" />
    <PackageReference Include="Microsoft.AspNetCore.Blazor.Server" Version="0.5.0-preview1-10334" />
  </ItemGroup>
  <ItemGroup>
    <ProjectReference Include="..\blazorApp\blazorApp.csproj" />
  </ItemGroup>
</Project>

Error upon MVC app publish:
JPMB:mvcApp joshuapearce$ dotnet build
Microsoft (R) Build Engine version <IP_ADDRESS>72 for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.
Restoring packages for /Users/joshuapearce/Dev/captainProton/mvcApp/mvcApp.csproj...
Restore completed in 19.64 ms for /Users/joshuapearce/Dev/captainProton/blazorApp/blazorApp.csproj.
Restore completed in 44.73 ms for /Users/joshuapearce/Dev/captainProton/blazorApp/blazorApp.csproj.
Restore completed in 638.92 ms for /Users/joshuapearce/Dev/captainProton/mvcApp/mvcApp.csproj.
blazorApp -> /Users/joshuapearce/Dev/captainProton/blazorApp/bin/Debug/netstandard2.0/blazorApp.dll
Blazor Build result -> 17 files in /Users/joshuapearce/Dev/captainProton/blazorApp/bin/Debug/netstandard2.0/dist
/usr/local/share/dotnet/sdk/NuGetFallbackFolder/microsoft.aspnetcore.razor.design/2.1.1/build/netstandard2.0/Microsoft.AspNetCore.Razor.Design.CodeGeneration.targets(69,5): error MSB4062: The "Microsoft.AspNetCore.Razor.Tasks.RazorTagHelper" task could not be loaded from the assembly /usr/local/share/dotnet/sdk/NuGetFallbackFolder/microsoft.aspnetcore.razor.design/2.1.1/build/netstandard2.0/../../tasks/netstandard2.0/Microsoft.AspNetCore.Razor.Tasks.dll. Assembly with same name is already loaded Confirm that the  declaration is correct, that the assembly and all its dependencies are available, and that the task contains a public class that implements Microsoft.Build.Framework.ITask. [/Users/joshuapearce/Dev/captainProton/mvcApp/mvcApp.csproj]
Build FAILED.
/usr/local/share/dotnet/sdk/NuGetFallbackFolder/microsoft.aspnetcore.razor.design/2.1.1/build/netstandard2.0/Microsoft.AspNetCore.Razor.Design.CodeGeneration.targets(69,5): error MSB4062: The "Microsoft.AspNetCore.Razor.Tasks.RazorTagHelper" task could not be loaded from the assembly /usr/local/share/dotnet/sdk/NuGetFallbackFolder/microsoft.aspnetcore.razor.design/2.1.1/build/netstandard2.0/../../tasks/netstandard2.0/Microsoft.AspNetCore.Razor.Tasks.dll. Assembly with same name is already loaded Confirm that the  declaration is correct, that the assembly and all its dependencies are available, and that the task contains a public class that implements Microsoft.Build.Framework.ITask. [/Users/joshuapearce/Dev/captainProton/mvcApp/mvcApp.csproj]
0 Warning(s)
1 Error(s)
Time Elapsed 00:00:02.71

I don't see any indication that this is Blazor-specific, as opposed to being an issue with Razor or .NET generally. It looks likely that your .NET SDK installation is incorrect in some way. I'd recommend clearing it away and reinstalling the .NET SDK.


The mini-charts don't have a mouse-hover info box or a click event to explain what they do.

On /students/8/profile?column=ela
There is a red mini-chart there without an x or y descriptor, they do this sort of thing on etrade for plotting charts, but they have a more well defined square outline so you can get a feel for where the line is relative to the borders.  The border of these charts should be descernable so you can tell the difference between a high horizontal vs low horizontal.
Clicking on them should bring up the full description of what they are and help on how to interpret the chart.

@sentientmachine This is good feedback, thanks!
I think we'll need to validate this in real user testing, but if you or anyone else wants to try making some improvements here that would be great, and I'm happy to help try them out with folks in Somerville.
Some other thoughts:

the top of the profile page should be an at-a-glance summary, so it'd be good to keep in mind simplicity while trying to clarify
more details are always available by clicking into that section, so that may allow more a streamlined summary up top

Also, you had a great idea in-person to add a tutorial or something like that.  I think that's another great way to approach this, with either a written tutorial page or something more interactive built into the site.  Any prototypes of these would be most welcome too! :)


[RESTEASY-2942] upgraded dependencies to match quarkus/bom/application

We need to think about what to do here. The MP upgrades are higher than the versions currently in WildFly which may be a problem. The tests did pass so maybe it's not a huge deal.

We need to think about what to do here.

For now, I'm thinking of RESTEasy 4.6.1.Final, which is for Quarkus. Ideally, when we get to 4.7.0.Final, we can find a compromise that works for both WildFly and Quarkus. It would be great if they both can use 4.7.0.Final.
Anyway, I'll create a pull request for 4.6.1.Final.

I cloned RESTEASY-2931 to RESTEASY-2942 for main branch.

@ronsigal If we do it in 4.6 we have to do it in 4.7. We can't have a newer version released with older dependencies. If this is required for 4.6 we need to come up with a solution because 4.7 needs to go into WildFly otherwise we're back to 3.x and we'll have to port Jakarta EE 9 changes into 3.x.

Replaced by #2856


Bugfix/multiroot cleanup

Added methods:

root_value_for_template for getting root value based on template
all_root_paths returning all root paths for all platforms for current project

Wrappers in Anatomy class - for easier calls

find_root_template_from_path
all_root_paths
path_remapper

Added new feature:

TemplateResult has rootless attribute where is filled path except root key.


Make it work with clap 3.0.0-beta.2

Addressing breaking changes introduced in clap 3.0.0-beta.2.

Ah cool, thanks!


A Running Error Using Tensorflow 1.8.0

My tensorflow version is 1.8.0
I clone the code to my computer and download the mnist_cluttered_60x60_6distortions.npz dataset into ./data/.
When I run the main.py, I get the following ERROR:
Loading the data...
Building ConvNet...
Traceback (most recent call last):
  File "/home/xinlab/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/tensor_util.py", line 517, in make_tensor_proto
    str_values = [compat.as_bytes(x) for x in proto_values]
  File "/home/xinlab/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/tensor_util.py", line 517, in <listcomp>
    str_values = [compat.as_bytes(x) for x in proto_values]
  File "/home/xinlab/anaconda3/lib/python3.6/site-packages/tensorflow/python/util/compat.py", line 67, in as_bytes
    (bytes_or_text,))
TypeError: Expected binary or unicode string, got -1

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "main.py", line 422, in <module>
    main()
  File "main.py", line 275, in main
    pool3_flat, pool3_size = Flatten(pool3)
  File "/home/LiChenyang/spatial-transformer-network/utils/layer_utils.py", line 85, in Flatten
    layer_flat = tf.reshape(layer, [-1, num_features])
  File "/home/xinlab/anaconda3/lib/python3.6/site-packages/tensorflow/python/ops/gen_array_ops.py", line 6113, in reshape
    "Reshape", tensor=tensor, shape=shape, name=name)
  File "/home/xinlab/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/op_def_library.py", line 513, in _apply_op_helper
    raise err
  File "/home/xinlab/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/op_def_library.py", line 510, in _apply_op_helper
    preferred_dtype=default_dtype)
  File "/home/xinlab/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 1104, in internal_convert_to_tensor
    ret = conversion_func(value, dtype=dtype, name=name, as_ref=as_ref)
  File "/home/xinlab/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/constant_op.py", line 235, in _constant_tensor_conversion_function
    return constant(v, dtype=dtype, name=name)
  File "/home/xinlab/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/constant_op.py", line 214, in constant
    value, dtype=dtype, shape=shape, verify_shape=verify_shape))
  File "/home/xinlab/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/tensor_util.py", line 521, in make_tensor_proto
    "supported type." % (type(values), values))
TypeError: Failed to convert object of type <class 'list'> to Tensor. Contents: [-1, None]. Consider casting elements to a supported type.
LiChenyang@ubuntu:~/spatial-transformer-network$

Any idea to fix this problem?

I guess the error is due to the last dimension of the target shape is None(i.e. [-1, None]) when execute the code: layer_flat = tf.reshape(layer, [-1, num_features]).
I try to print the h_trans and pool3 in main(), and I get:
Tensor("AddN:0", shape=(?, ?, ?, 1), dtype=float32)
Tensor("Relu_5:0", shape=(?, ?, ?, 128), dtype=float32)
The second and third dimension of h_trans is '?', which makes num_features in Flatten() None.
To verify this idea, I add a new function in ./utils/layer_utils.py:
def Flatten_test(layer):
	"""
	Handy function for flattening the result of a conv2D or
	maxpool2D to be used for a fully-connected (affine) layer.
	"""
	# layer_shape = layer.get_shape()
	# # num_features = tf.reduce_prod(tf.shape(layer)[1:])
	# num_features = layer_shape[1:].num_elements()

	num_features = 8 * 8 * 128
	layer_flat = tf.reshape(layer, [-1, num_features])

	return layer_flat, num_features

Manually calculate the num_features and change:
pool3_flat, pool3_size = Flatten(pool3)
into
pool3_flat, pool3_size = Flatten_test(pool3)
Finally, I can successfully run the main.py.
However, I still puzzle about why the num_features  is None in Flatten()


TDE in standard edition

I am aware that this is prerelease information, but I have to ask: Is it on purpose that TDE (transparent data encryption) is listes as a feature that is available on Standard Edition? If so, this would be a rather significant change from previous versions :-)

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

ID: 48cb4a50-bcd6-38bc-187b-27cc36576219
Version Independent ID: 38c1d92e-25d6-6237-9a04-6ca13bbb1a09
Content: What's new in SQL Server 2019 - SQL Server
Content Source: docs/sql-server/what-s-new-in-sql-server-ver15.md
Product: sql
Technology: release-landing
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

@Jagoop Thank you for the question.
There hasn't been any official mention of this feature supported as of now, assigning this to the author for review.

This has been confirmed @Jagoop in the details release yesterday: https://docs.microsoft.com/en-us/sql/sql-server/editions-and-components-of-sql-server-version-15?view=sql-server-ver15#RDBMSS

Has there been any official confirmation from Microsoft about this?  TDE in Standard is a BIG DEAL and should be a headlining feature change if true.

Thanks for the comment, @Jagoop - TDE is available in SQL Server 2019 Standard Edition.
In addition to the updates in the docs, we announced it in a blog post


fix: added babel-polyfill to fix IE11 support

Resolves #207
Impact: minor
Type: bugfix
Work
Added babl-poyfill to styleguide for IE11 support.
Testing

Start the component lib and view it in IE11
Component pages should now be loading

@mikemurray  I think I've got this working now. I moved the import of "babel-polyfill" to the styleguide.config and it looks to be making it to the production build now.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

@nnnnat @mikemurray does this close https://github.com/reactioncommerce/reaction-next-starterkit/issues/153 ?


Exceptions: added base init call for proper Exception inheritance

This is needed for proper exception handing, for example by celery (to pack/unpack exceptions args attribute is used, and in this case it appears to be empty), but possible by other systems too.

At least google has this as top result when searching OAuth2AuthExchangeError: You must provide a client_id. Thanks @vgavro


simple_actions: 0.2.2-1 in 'foxy/distribution.yaml' [bloom]

Increasing version of package(s) in repository simple_actions to 0.2.2-1:

upstream repository: https://github.com/DLu/simple_actions.git
release repository: https://github.com/DLu/simple_actions-release.git
distro file: foxy/distribution.yaml
bloom version: 0.11.2
previous version for package: 0.2.1-1

Not sure why checks aren't running

I'm going to close this and reopen to see if I can get the checks to run.


Revise command line arguments

Go through the currently supported command line arguments and make sure they are consistent and don't conflict with each other. Support for -h (instead of?) -?

Whats wrong with -?    I use that all the time.    But we probably should go through and make sure the available options are consistent with conventions and internally consistent.

Mostly that -h seems to be more typical as a short form (for unix) help, and since most users will likely be using the unix form of arguments they should be handled well. Supporting both would probably be good, since it looks like some unix commands also support -?.

For Federate Info
static const ArgDescriptors InfoArgs{
  {"broker,b"s, "address of the broker to connect"s},
  {"name,n"s, "name of the federate"s},
  {"corename"s, "the name of the core to create or find"s},
  {"core,c"s, "type of the core to connect to"s},
  {"type,t"s, "type of the core to connect to"s},
  {"coretype"s, "type of the core to connect to"s},
  {"offset"s, "the offset of the time steps"s},
  {"period"s, "the period of the federate"s},
  {"timedelta"s, "the time delta of the federate"s},
  {"rttolerance"s, "the time tolerance of the real time mode"s},
  {"rt_lag"s, "the amount of the time the federate is allowed to lag realtime before corrective action is taken"s},
  {"rt_lead"s,
   "the amount of the time the federate is allowed to lead realtime before corrective action is taken"s},
  {"coreinit,i"s, "the core initialization string"s},
  {"coreinitstring"s, "the core initialization string"s},
  {"maxiterations"s, ArgDescriptor::arg_type_t::int_type,
   "the maximum number of iterations a federate is allowed to take"s},
  {"log_level"s, ArgDescriptor::arg_type_t::int_type, "the logging level of a federate"s},
  {"separator"s, "separator character for local federates"s},
  {"inputdelay"s, "the input delay on incoming communication of the federate"s},
  {"outputdelay"s, "the output delay for outgoing communication of the federate"s},
  {"brokerport"s, ArgDescriptor::arg_type_t::int_type, "port number for the broker priority port"s},
  {"localport"s, "port number for the local receive port"s},
  {"autobroker"s, ArgDescriptor::arg_type_t::flag_type,
   "tell the core to automatically generate a broker if needed"s},
  {"port"s, ArgDescriptor::arg_type_t::int_type, "port number for the broker's port"s},
  {"flags,f"s, ArgDescriptor::arg_type_t::vector_string, "named flag for the federate"s}};

For Broker base  (passed in through coreinitstring)
{"name,n", "name of the broker/core"},
  {"federates,f", ArgDescriptor::arg_type_t::int_type, "the minimum number of federates that will be connecting"},
  {"minfed,m", ArgDescriptor::arg_type_t::int_type, "the minimum number of federates that will be connecting"},
  {"maxiter", ArgDescriptor::arg_type_t::int_type, "maximum number of iterations"},
  {"logfile", "the file to log message to"},
  {"loglevel", ArgDescriptor::arg_type_t::int_type,
   "the level which to log the higher this is set to the more gets logs (-1) for no logging"},
  {"log_level", ArgDescriptor::arg_type_t::int_type,
   "the level which to log the higher this is set to the more gets logs (-1) for no logging"},
  {"fileloglevel", ArgDescriptor::arg_type_t::int_type, "the level at which messages get sent to the file"},
  {"consoleloglevel", ArgDescriptor::arg_type_t::int_type, "the level at which message get sent to the console"},
  {"minbrokers", ArgDescriptor::arg_type_t::int_type,
   "the minimum number of cores/brokers that need to be connected (ignored in cores)"},
  {"identifier", "name of the core/broker"},
  {"tick", "number of milliseconds per tick counter if there is no broker communication for 2 ticks then "
           "secondary actions are taken  (can also be entered as a time like '10s' or '45ms')"},
  {"dumplog", ArgDescriptor::arg_type_t::flag_type,
   "capture a record of all messages and dump a complete log to file or console on termination"},
  {"networktimeout",
   "milliseconds to wait to establish a network (can also be entered as a time like '500ms' or '2s') "},
  {"timeout",
   "milliseconds to wait for a broker connection (can also be entered as a time like '10s' or '45ms') "}};

For network interface
{"interface"s, "the local interface to use for the receive ports"s},
  {"local_interface"s, "the local interface to use for the receive ports"s},
  {"broker,b"s, "identifier for the broker"s},
  {"broker_address"s, "location of the broker i.e network address"s},
  {"brokername"s, "the name of the broker"s},
  {"brokerinit"s, "the initialization string for the broker"s},
  {"max_size"s, ArgDescriptor::arg_type_t::int_type, "maximum message buffer size (16*1024)"s},
  {"max_count"s, ArgDescriptor::arg_type_t::int_type, "max queue size for the message (256)"s},
  {"local"s, ArgDescriptor::arg_type_t::flag_type, "use local interface(default)"s},
  {"ipv4"s, ArgDescriptor::arg_type_t::flag_type, "use external ipv4 addresses"s},
  {"ipv6"s, ArgDescriptor::arg_type_t::flag_type, "use external ipv6 addresses"s},
  {"server"s, ArgDescriptor::arg_type_t::flag_type, "specify that the network connection should be a server"s},
  {"os_port"s, ArgDescriptor::arg_type_t::flag_type,
   "specify that the ports should be allocated by the host operating system"s},
  {"autobroker"s, ArgDescriptor::arg_type_t::flag_type,
   "allow a broker to be automatically created if one is not available"s},
  {"client"s, ArgDescriptor::arg_type_t::flag_type, "specify that the network connection should be a client"s},
  {"reuse_address"s, ArgDescriptor::arg_type_t::flag_type, "allow the server to reuse a bound address"s},
  {"external"s, ArgDescriptor::arg_type_t::flag_type, "use all external interfaces"s},
  {"brokerport"s, ArgDescriptor::arg_type_t::int_type, "port number for the broker priority port"s},
  {"localport"s, "port number for the local receive port"s},
  {"port"s, ArgDescriptor::arg_type_t::int_type, "port number for the broker's port"s},
  {"portstart"s, ArgDescriptor::arg_type_t::int_type, "starting port for automatic port definitions"s}};

available for brokers
 {"root"s, ArgDescriptor::arg_type_t::flag_type, "specify whether the broker is a root"s},

For tcpSS core and broker
{ "connections"s, ArgDescriptor::arg_type_t::vector_string,"target link connections"s },
{"no_outgoing_connections"s, ArgDescriptor::arg_type_t::flag_type,"disable outgoing connections"s} };

For broker apps
  {"name,n", "name of the broker"},
  {"type,t", R"lit(type of the broker ("(zmq)", "ipc", "test", "mpi", "test", "tcp", "udp"))lit"}};

For all other apps
{"local", ArgDescriptor::arg_type_t::flag_type,
                                          "specify otherwise unspecified endpoints and publications as local( "
                                          "i.e.the keys will be prepended with the player name"},
                                         {"stop", "the time to stop the player"},
                                         {"quiet", ArgDescriptor::arg_type_t::flag_type,
                                          "turn off most display output"}

For echo apps
{"delay", "the delay with which the echo app will echo message"}

For player
  {"datatype", "type of the publication data type to use"},
  {"marker", "print a statement indicating time advancement every <arg> period during the simulation"},
  {"timeunits", "the default units on the timestamps used in file based input"}

for recorder
{"tags", ArgDescriptor::arg_type_t::vector_string,
   "tags to record, this argument may be specified any number of times"},
  {"endpoints", ArgDescriptor::arg_type_t::vector_string,
   "endpoints to capture, this argument may be specified multiple time"},
  {"sourceclone", ArgDescriptor::arg_type_t::vector_string,
   "existing endpoints to capture generated packets from, this argument may be specified multiple time"},
  {"destclone", ArgDescriptor::arg_type_t::vector_string,
   "existing endpoints to capture all packets with the specified endpoint as a destination, this argument may be "
   "specified multiple time"},
  {"clone", ArgDescriptor::arg_type_t::vector_string, "existing endpoints to clone all packets to and from"},
  {"capture", ArgDescriptor::arg_type_t::vector_string,
   "capture all the publications of a particular federate capture=\"fed1;fed2\"  supports multiple arguments or a "
   "comma separated list"},
  {"output,o", "the output file for recording the data"},
  {"allow_iteration", ArgDescriptor::arg_type_t::flag_type, "allow iteration on values"},
  {"verbose", ArgDescriptor::arg_type_t::flag_type, "print all value results to the screen"},
  {"marker", "print a statement indicating time advancement every <arg> period during the simulation"},
  {"mapfile", "write progress to a map file for concurrent progress monitoring"}

for source
{"default_period", "the default period publications"}

for tracer
{"tags", ArgDescriptor::arg_type_t::vector_string,
   "tags to record, this argument may be specified any number of times"},
  {"endpoints", ArgDescriptor::arg_type_t::vector_string,
   "endpoints to capture, this argument may be specified multiple time"},
  {"sourceclone", ArgDescriptor::arg_type_t::vector_string,
   "existing endpoints to capture generated packets from, this argument may be specified multiple time"},
  {"destclone", ArgDescriptor::arg_type_t::vector_string,
   "existing endpoints to capture all packets with the specified endpoint as a destination, this argument may be "
   "specified multiple time"},
  {"clone", ArgDescriptor::arg_type_t::vector_string, "existing endpoints to clone all packets to and from"},
  {"allow_iteration", ArgDescriptor::arg_type_t::flag_type, "allow iteration on values"},
  {"capture", ArgDescriptor::arg_type_t::vector_string,
   "capture all the publications and endpoints of a particular federate capture=\"fed1;fed2\"  supports multiple "
   "arguments or a comma separated list"}

the helics_broker executable will take a first arg as "term" to start and interactive terminal
the helics_app takes first argument as the name of the app

also for all parsing there is
("help,?", "produce help message")
        ("version,v","display a version string")
		("config-file", po::value<std::string>(), "specify a configuration file to use");

I think there are four things with command line arguments that seem off:

support -h for help (more Linux programs seem to support -h than -?; -? could stay as an option with the same meaning)
brokers should recognize --coretype for consistency with apps; apps should recognize -t for setting the core type (along with possibly removing -c and --core; there would already be 4 option names to chose from, and being a bit more specific with the long form option names for the type would be more aligned with the --corename options)
only the broker supports log_level with an underscore (according to the help message printed)
the broker -b option seems to have a different meaning (identifier) from other apps (address) (other apps don't have a broker_address option)

One question that came up, which might be something for the docs to explain: how is -b or --broker for setting a broker identifier different from the name?

BrokerBase applies to all cores and brokers, and all the apps take all the options for a federate


Pin GitHub actions to git hashes

Pin our GitHub Action configs to explicit git hashes.
This reduces the potential for compromised dependencies to undermine our own repo security.

Codecov Report
Base: 89.33% // Head: 89.33% // No change to project coverage :thumbsup:

Coverage data is based on head (0d45d1c) compared to base (4a0f207).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #45   +/-   ##
=======================================
  Coverage   89.33%   89.33%
=======================================
  Files           7        7
  Lines         497      497
=======================================
  Hits          444      444
  Misses         48       48
  Partials        5        5

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Can you remind me how to rerun the fuzzing check?

Can you remind me how to rerun the fuzzing check?

There's a "re-run" button on the action details page, but the issue seems to be upstream in the fuzzer action/library itself :/


Not support responsive size?

Describe the bug
I use useVirtualize with useWindowSize for the responsive item size.
  const { width } = useWindowSize();

  const columnVirtualizer = useVirtualizer({
    horizontal: true,
    count: 100,
    getScrollElement: () => parentRef.current,
    estimateSize: () => width / 10,
    overscan: 5,
  })

Even though the width is changed, columnVirtualizer doesn`t rerendered.
There is the code about the Virtualizer instance in useVirtualizerBase (/pacakges/react-virtual/src/index.tsx, line:39)
  const [instance] = React.useState(
    () => new Virtualizer<TScrollElement, TItemElement>(resolvedOptions),
  )

Even if the resolvedOptions have changed,  initializing the instance only once.
Is this intentional?
Your minimal, reproducible example
https://codesandbox.io/p/sandbox/rough-tdd-4wfpkr
Steps to reproduce
Refer to codesand box
Expected behavior
When I resize the window, items should rerender with new size
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
macOS
tanstack-virtual version
latest
TypeScript version
5.2.2
Additional context
No response
Terms & Code of Conduct

[x] I agree to follow this project's Code of Conduct
[x] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

@jijiseong that is correct, estimateSize is not part of caching strategy, to re-create virtual items you can use,
getItemKey: React.useCallback((index) => index, [width]),

https://codesandbox.io/p/sandbox/sweet-sun-f35n4r?file=%2Fsrc%2FApp.tsx%3A17%2C5-17%2C62

I'm trying to figure out how is it possible to retain the scroll position when the window or container is resized?
https://codesandbox.io/p/sandbox/quizzical-snowflake-9t5rrk?workspaceId=4639fe9e-259b-4807-83e7-6dc48e992cc6
https://github.com/user-attachments/assets/ac672971-5ab4-4e7b-a236-5f1edea449db


Cleanup

Updates to make use of built in ansible vars as well as a JSON config file.

Tested here. No errors reported.


Can be plugin v1.x compatible with cordoba-plugin-firebase?

Hi,
I  have push messages in my Ionic 2 app using GCM for Android with this plugin and I want to use Firebase analytics using cordova-plugin-firebase. When both plugins are installed Android platform cannot be build.
For the time being, I am not planning to move my push messaging service to Firebase and I want to stick to version 1.x.
Can both plugins be used at the same time?
Regards,
Xavi
Expected Behaviour
I want to install in Ionic 2 this plugin and cordoba-plugin-firebase,  and compile the Android platform.
Actual Behaviour
When both plugins are installed in a Ionic 2 project, Android platform build ends with an error:
Error: /Volumes/Data/xavi/work/code/ionic/incubapps/just-today-ionic/platforms/android/gradlew: Command failed with exit code 1 Error output:
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':mergeDebugResources'.
> /Volumes/Data/xavi/work/code/ionic/incubapps/just-today-ionic/platforms/android/res/values/strings.xml: Error: Found item String/google_app_id more than one time

platforms/android/res/values/strings.xml:
<?xml version='1.0' encoding='utf-8'?>
<resources>
    <string name="app_name">Just Today</string>
    <string name="launcher_name">@string/app_name</string>
    <string name="activity_name">@string/launcher_name</string>
    <string name="google_app_id" translatable="false">XXXXXXXX</string>
    <string name="google_app_id">1:XXXXXXXX:android:XXXXXXXX</string>
    <string name="google_api_key">YYYYYYYY</string>
</resources>

The string google_app_id is defined twice and causes an error. I'm not an expert and the problem can much deeper and complex, but maybe if phonegap-plugin-push v1.x uses a name like gcm_app_id the problem is solved.
Reproduce Scenario (including but not limited to)
Steps to Reproduce

Install latest Ionic CLI 2.2.1.
Install Cordova 6.5.0.
Create a brand new project with: ionic start --v2 myApp tabs.
Install cordoba-plugin-firebase plugin.
Install phonegap-plugin-push plugin.
Build and run with ionic run android.

Platform and Version (eg. Android 5.0 or iOS 9.2.1)
Android 5.x and up.
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Not relevant.
Cordova CLI version and cordova platform version
Ionic info:
Your system information:

Cordova CLI: 6.5.0
Ionic Framework Version: 2.2.0
Ionic CLI Version: 2.1.17
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 1.1.4
ios-deploy version: 1.9.1
ios-sim version: 5.0.13
OS: OS X El Capitan
Node Version: v6.9.5
Xcode version: Xcode 8.2.1 Build version 8C1002

cordova platform version android:
Installed platforms:
  android 6.1.2
  browser 4.1.0
  ios 4.3.1

Plugin version
phonegap-plugin-push 1.10.0 "PushPlugin"
cordova-plugin-firebase 0.1.19 "Google Firebase Plugin"

@xmellado sorry, I won't be back porting the changes to the 1.x stream that get rid of the google_app_id. You'll have to move to the 2.x stream. Even if we did get rid of the google_app_id there is a good chance there would be other conflicts with regards to the GCM and FCM libraries. Furthermore both the push plugin and firebase plugin would both be trying to handle the notifications which would cause issues in your app.
What analytics functions do you want to use?

Hi,
@macdonst I would like to use only the function logEvent() from the cordoba-firebase-plugin, I don't need anything else. I need to log a set of custom events that we have defined to track user's navigation in our app.
I have tried to import directly Firebase as web project (JavaScript) in Ionic 2, but analytics are not available for web projects.
Regards,
Xavi

I tried to use the version 1.x.x but is not working with Firebase plugin. Ok. But the version 2.x.x-rc4 give me this error:
Error: cmd: Command failed with exit code 1 Error output:
FAILURE: Build failed with an exception.

Where:
Script 'C:\app\platforms\android\phonegap-plugin-push\android-push.gradle' line: 12

What went wrong:
A problem occurred evaluating root project 'android'.

Failed to apply plugin [id 'com.google.gms.google-services']
For input string: "+"

Please, I need help!

@performatric if you browse the issues you'll see that that is not fixed yet, hopefully on the next release.

I am having the same issue with rc5, when it'll be solved ?


ENDOC-640 Entando Entando Identity Management System Documentation

@Lyd1aCla1r3 Looks good, just need to add in v7.1 as well.


1.25.2 ansible-operator is not picking up 8.7 ubi.

Bug Report

What did you do?
podman image inspect  quay.io/operator-framework/ansible-operator:v1.25.2  | jq .[0].Labels
{
  "architecture": "x86_64",
  "build-date": "2022-10-26T11:06:57",
  "com.redhat.component": "ubi8-container",
  "com.redhat.license_terms": "https://www.redhat.com/en/about/red-hat-end-user-license-agreements#UBI",
  "description": "The Universal Base Image is designed and engineered to be the base layer for all of your containerized applications, middleware and utilities. This base image is freely redistributable, but Red Hat only supports Red Hat technologies through subscriptions for Red Hat products. This image is maintained by Red Hat and updated regularly.",
  "distribution-scope": "public",
  "git_commit": "77dd4e2ff165c0317254d55ff01ec1f8d0f1c34a",
  "io.buildah.version": "1.26.2",
  "io.k8s.description": "The Universal Base Image is designed and engineered to be the base layer for all of your containerized applications, middleware and utilities. This base image is freely redistributable, but Red Hat only supports Red Hat technologies through subscriptions for Red Hat products. This image is maintained by Red Hat and updated regularly.",
  "io.k8s.display-name": "Red Hat Universal Base Image 8",
  "io.openshift.expose-services": "",
  "io.openshift.tags": "base rhel8",
  "maintainer": "Red Hat, Inc.",
  "name": "ubi8",
  "release": "990",
  "summary": "Provides the latest release of Red Hat Universal Base Image 8.",
  "url": "https://access.redhat.com/containers/#/registry.access.redhat.com/ubi8/images/8.6-990",
  "vcs-ref": "f1ee6e37554363ec55e0035aba1a693d3627fdeb",
  "vcs-type": "git",
  "vendor": "Red Hat, Inc.",
  "version": "8.6"
}

What did you expect to see?
As stated on the 1.25.2 release note, the base image should have been picking up v8.7 UBI.

What did you see instead? Under which circumstances?
Latest commit to the Dockerfile of ansible-operator is https://github.com/operator-framework/operator-sdk/commit/2bda9e1e767a9007f97422739fa3bbe62cc05a6f which is about a month old.
I think commit of updating base.ansible-operator is missing on ansible-operator dockerfile.

Environment
Operator type:

/language ansible

Possible Solution
Modify  Dockerfile with updated build tag of base.ansible-operator image

Both #6179 and #6180 have been merged and if my understanding is correct the image should be updated automatically within the next day or so. I'll check back on Monday next week and verify whether or not the update has gone through.

Just to follow up, I took a look to see if it has been updated but our automated image refresh job failed this morning. I kicked it off again to see if it was just a flake, but if not I will try to investigate why it is failing so we can get this image rebuilt ASAP

Update: It seems like we would need to go through the process of another patch release to get this taken care of. I have this on my todo list but after some discussion offline with other maintainers we aren't marking this as something that is urgent and rather something that I will get around to when I have some extra time.

@everettraven Thanks for the update, do you have the ETA for next patch release?

@daezaa Due to this week being the week of Thanksgiving in the United States I would give a rough estimate of mid-late next week

@daezaa I just pushed out a new patch release (v1.25.3) that updates ansible images to properly pull in the UBI 8.7 image.
I have verified that quay.io/operator-framework/ansible-operator:(v1.25.3 | v1.25 | latest) all currently pull in the UBI 8.7 image with podman image inspect ....
Here is the labels output from podman image inspect quay.io/operator-framework/ansible-operator:latest:
 {
            "architecture": "x86_64",
            "build-date": "2022-11-01T04:29:09",
            "com.redhat.component": "ubi8-container",
            "com.redhat.license_terms": "https://www.redhat.com/en/about/red-hat-end-user-license-agreements#UBI",
            "description": "The Universal Base Image is designed and engineered to be the base layer for all of your containerized applications, middleware and utilities. This base image is freely redistributable, but Red Hat only supports Red Hat technologies through subscriptions for Red Hat products. This image is maintained by Red Hat and updated regularly.",
            "distribution-scope": "public",
            "git_commit": "aa0243440429e60568c27df5c7c22976f2b23a16",
            "io.buildah.version": "1.26.2",
            "io.k8s.description": "The Universal Base Image is designed and engineered to be the base layer for all of your containerized applications, middleware and utilities. This base image is freely redistributable, but Red Hat only supports Red Hat technologies through subscriptions for Red Hat products. This image is maintained by Red Hat and updated regularly.",
            "io.k8s.display-name": "Red Hat Universal Base Image 8",
            "io.openshift.expose-services": "",
            "io.openshift.tags": "base rhel8",
            "maintainer": "Red Hat, Inc.",
            "name": "ubi8",
            "release": "929",
            "summary": "Provides the latest release of Red Hat Universal Base Image 8.",
            "url": "https://access.redhat.com/containers/#/registry.access.redhat.com/ubi8/images/8.7-929",
            "vcs-ref": "a995512a05037e3b60bbb1bf9fa6e394063131c3",
            "vcs-type": "git",
            "vendor": "Red Hat, Inc.",
           "version": "8.7"
}

Since this should now be resolved as of Operator SDK v1.25.3 I am going to close this issue. Please feel free to reopen if there are any other issues.


About Denoising Vision Transformers

I noticed this paper "Denoising Vision Transformers", and its point is that most ViT models have artifacts, which can be simply removed using this method. Have you considered integrating this plugin?

Will try it, still not sure its usage though.


Potential feature: auto rename duplicate file names

Congrats to the nice and useful tool!
Just one thing I struggled with: Since some pictures might have the same time stamp (taken within the same second), they get overwritten when sorting by the last picture having the file name corresponding to this second.
BR

Hi thanks for giving it a try!
This was by design as I had lots of datasets that I knew were full of duplicates due to years of poor media management.
Due to the parallel processing of the media files it makes it a little bit more complicated to handle duplicates as everything is sorted out of order but shouldn't be too hard to add options on ways to handle duplicates.
What is better:

Rename duplicates <new_name>_1.extension?
Copy them to a duplicates folder instead of overwriting so you can see what was duplicated (similair to "failed_to_sort" or "other" files)?

Cheers

Hi Josh!
I used the script for exactly the same reason: to clean up the mess created during years. And: it does an awesome and super-fast job.
The issue with the duplicates is mainly caused by serial pictures taken when using the burst function of the camera.
Quickly tried to hack in the lines to auto-rename the duplicates myself (see below) but, as you mentioned: out of 20 files, some 2-4 get overwritten due to parallel task handling, which I’m not yet experienced enough in coding to deal with.
My personal preference would be option 1: “Rename duplicates <new_name>_1.extension”
Regards from the Swiss Alps!
Chris

temp_new_name = "{}{}{}_{}{}{}{}{}".format(
            str(dtime.year).zfill(4),

            str(dtime.month).zfill(2),

            str(dtime.day).zfill(2),

            str(dtime.hour).zfill(2),

            str(dtime.minute).zfill(2),

            str(dtime.second).zfill(2),

            '',

            os.path.splitext(source_image_path)[1],

        )

        new_path = os.path.join(

            destination_dir, str(dtime.year).zfill(4), str(dtime.month).zfill(2)

        )

        os.makedirs(new_path, exist_ok=True)

        i = 0

        while os.path.exists(os.path.join(new_path,temp_new_name)):

            i += 1

            name, ext = os.path.splitext(temp_new_name)

            temp_new_name = name[:15] + '_' +str(i) + ext

        new_name = os.path.basename(temp_new_name)

From: Josh Thorpe<EMAIL_ADDRESS>Sent: Saturday, 20 February 2021 02:11
To: ThorpeJosh/image-sorting-tool<EMAIL_ADDRESS>Cc: Meimec<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [ThorpeJosh/image-sorting-tool] Potential feature: auto rename duplicate file names (#17)
Hi thanks for giving it a try!
This was by design as I had lots of datasets that I knew were full of duplicates due to years of poor media management.
Due to the parallel processing of the media files it makes it a little bit more complicated to handle duplicates as everything is sorted out of order but shouldn't be too hard to add options on ways to handle duplicates.
What is better:

Rename duplicates <new_name>_1.extension?
Copy them to a duplicates folder instead of overwriting so you can see what was duplicated (similair to "failed_to_sort" or "other" files)?

Cheers
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/ThorpeJosh/image-sorting-tool/issues/17#issuecomment-782494477 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AS4OYNSB7UP2KWBVRFSN6N3S74DY7ANCNFSM4XZCYJTQ .  https://github.com/notifications/beacon/AS4OYNRBZFMPF2D3H4ZXN5TS74DY7A5CNFSM4XZCYJT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOF2R6WDI.gif

Hi Chris,
I've been without internet for several days now (greetings from Australia via our National Broadband Network). What you've done would have been my first attempt. But if what you said is true then between the time a thread checks if the temp name exists but before writing the file, another thread checks and thinks the file doesn't exist yet.
The better solution is to share a set of filenames between all the threads and check if the filename exists in the set and add the filename to the set before writing the file to disk.
My phone doesn't take burst shots so I'll have to find some on the web to test with.
I'll see how I go and post back.
Feel free to make a pull request too if you have any luck.
This blog has a decent explanation of how I perform multiprocessing with a pool of workers to sort the images. 1st example is identical.
https://www.codesdope.com/blog/article/multiprocessing-using-pool-in-python/

Hi Josh,
Fun to hear that the network in AU is still an issue – remember that same experience/pain when staying in Karatha / Dampier.
Thanks for the thorough explanation and the interesting proposal for an approach to get it implemented. I’ll see if I find time to work on it the next days.
Cheers!

Hi Chris,
Karratha is a long way from anywhere. I just had 5 days with no internet (broadband or mobile, complete blackout) only 50km from Melbourne. Fun times. I was spoilt with fast reliable internet in Stuttgart when I worked there and also in Singapore, so I get more frustrated with AU internet than most Aussies do.
I have added this feature to the GUI and passed the checkbox boolean through to the sorting function that runs in each thread. It is ready for your changes. It is on a "duplicates" branch: github.com/ThorpeJosh/image-sorting-tool/pull/19
I am still unsure the best way to go about each worker/thread communicating so they don't simultaneously choose and write the same filename.

@Meimec Hope you're well.
Almost 3 years later but this should now be implemented in the latest release . PyPI Link
I'll close this issue but if you still run into issues with this new feature, please reopen.


Seeeduino XIAO (SAMD21G18 Cortex-M0+)

Is this library compatible my Seeeduino XIAO?
SAMD21G18 Cortex-M0+
This is what I get trying to run UsageExample.ino:
WARNING: library FastAccelStepper claims to run on avr, esp32 architecture(s) and may be incompatible with your current board which runs on samd architecture(s).
In file included from c:\Users\Nathaniel\Documents\Arduino\libraries\FastAccelStepper\src/FastAccelStepper.h:10:0,
                 from c:\Users\Nathaniel\Documents\Arduino\FilmFix\FilmFix.ino:52:
c:\users\nathaniel\documents\arduino\libraries\fastaccelstepper\tests\pc_based\stubs.h:5:0: warning: "pgm_read_byte_near" redefined
 #define pgm_read_byte_near(x) (*(x))

In file included from C:\Users\Nathaniel\AppData\Local\Arduino15\packages\Seeeduino\hardware\samd\1.8.1\cores\arduino/Arduino.h:36:0,
                 from C:\Users\NATHAN~1\AppData\Local\Temp\arduino-sketch-4B354A760F649884E833B790D5F4E104\sketch\FilmFix.ino.cpp:1:
C:\Users\Nathaniel\AppData\Local\Arduino15\packages\Seeeduino\hardware\samd\1.8.1\cores\arduino/avr/pgmspace.h:110:0: note: this is the location of the previous definition
 #define pgm_read_byte_near(addr) pgm_read_byte(addr)

In file included from c:\Users\Nathaniel\Documents\Arduino\libraries\FastAccelStepper\src/FastAccelStepper.h:10:0,
                 from c:\Users\Nathaniel\Documents\Arduino\FilmFix\FilmFix.ino:52:
c:\users\nathaniel\documents\arduino\libraries\fastaccelstepper\tests\pc_based\stubs.h:19:0: warning: "abs" redefined
 #define abs(x) ((x) > 0 ? (x) : -(x))

In file included from C:\Users\NATHAN~1\AppData\Local\Temp\arduino-sketch-4B354A760F649884E833B790D5F4E104\sketch\FilmFix.ino.cpp:1:0:
C:\Users\Nathaniel\AppData\Local\Arduino15\packages\Seeeduino\hardware\samd\1.8.1\cores\arduino/Arduino.h:135:0: note: this is the location of the previous definition
 #define abs(x) ((x)>0?(x):-(x))

C:\Users\NATHAN~1\AppData\Local\Temp\arduino-sketch-4B354A760F649884E833B790D5F4E104\libraries\FastAccelStepper\FastAccelStepper.cpp.o: In function `FastAccelStepperEngine::stepperConnectToPin(unsigned char)':
c:\Users\Nathaniel\Documents\Arduino\libraries\FastAccelStepper\src/FastAccelStepper.cpp:82: undefined reference to `StepperQueue::isValidStepPin(unsigned char)'
c:\Users\Nathaniel\Documents\Arduino\libraries\FastAccelStepper\src/FastAccelStepper.cpp:116: undefined reference to `StepperQueue::queueNumForStepPin(unsigned char)'
C:\Users\NATHAN~1\AppData\Local\Temp\arduino-sketch-4B354A760F649884E833B790D5F4E104\libraries\FastAccelStepper\FastAccelStepper.cpp.o: In function `FastAccelStepper::setDirectionPin(unsigned char, bool)':
c:\Users\Nathaniel\Documents\Arduino\libraries\FastAccelStepper\src/FastAccelStepper.cpp:476: undefined reference to `fas_queue'
C:\Users\NATHAN~1\AppData\Local\Temp\arduino-sketch-4B354A760F649884E833B790D5F4E104\libraries\FastAccelStepper\FastAccelStepper.cpp.o: In function `FastAccelStepper::move(long)':
c:\Users\Nathaniel\Documents\Arduino\libraries\FastAccelStepper\src/FastAccelStepper.cpp:545: undefined reference to `fas_queue'
C:\Users\NATHAN~1\AppData\Local\Temp\arduino-sketch-4B354A760F649884E833B790D5F4E104\libraries\FastAccelStepper\RampGenerator.cpp.o: In function `RampGenerator::_startMove(long, long)':
c:\Users\Nathaniel\Documents\Arduino\libraries\FastAccelStepper\src/RampGenerator.cpp:139: undefined reference to `noInterrupts()'
c:\Users\Nathaniel\Documents\Arduino\libraries\FastAccelStepper\src/RampGenerator.cpp:147: undefined reference to `interrupts()'
collect2.exe: error: ld returned 1 exit status
Compilation error: Error: 2 UNKNOWN: exit status 1

I replaced defined(ARDUINO_ARCH_AVR) with defined(ARDUINO_ARCH_AVR) || defined(ARDUINO_ARCH_SAMD)  to add my Seeeduino. Now I am getting:
error: 'MAX_STEPPER' was not declared in this scope
   FastAccelStepper* _stepper[MAX_STEPPER];

From the datasheet the Seeduino Xiao could drive three steppers:
Three 24-bit Timer/Counters for Control (TCC), with extended functions:
• Up to four compare channels with optional complementary output
• Generation of synchronized pulse width modulation (PWM) pattern across port pins
But it is quite an effort to implement. If there are a couple of comments wanting this chip to be supported, then I will look into it. So I keep it open, for others to register their wish. A patch (MIT licensed) would be welcomed, too

Thank you -- very much appreciated.
I am still new to the github concept and I am not sure what this would entail: "A patch (MIT licensed) would be welcomed, too"; I plan on using it privately, and found (url) to do the job for now. It would be nice if I could run it faster so I can use 1/8 or 1/16 steps.

You are in so much luck....I'm not sure what the seeduino xiao is, so I'm not sure if it is an atmel sam3x8e like the DUE, but I'm pretty close to forking/committing/pull-requesting Due support.  You're addition of ARDUINO_ARCH_SAM makes me wonder.
I didn't look too closely at the full line.  My code is using the PWM generator peripheral, not synchronous, to generate up to 8 channels...but not all pins are easily accessible, and some share functions with other pins you're likely to want.  I arbitrarily capped it at 6.  gin66 is correct, this was a non-trivial effort.  It probably would have been easier for him, and I suspect he will need to "fix" some of my code (or tell me what to fix).
I chose NOT to use the timers because I need them to read servo PWM signals.  It looks like those timers have a rise time capture, as well as a fall time capture, so no ISR delay on pulses that are synchronous and end within a few microseconds of one another.  I'm working pretty hard to get this committed tonight....as evident by the fact I just registered a github account (I havent had need/want to contribute/collaborate until very recently).  Anyhow, off to fork...

You can give my fork a shot and see if it works...I didn't look at the datasheet for the SAMD21G18, so I honestly have no idea.
https://github.com/clazarowitz/FastAccelStepper

So, I did just have a look at the data sheet for the SAMD21G18.  Sorry, my fork will definatly not work.  The M0 and M3 from atmel are just too different.  Looks like my #defines will need to be updated.  ARDUINO_ARCH_SAM is far too broad.


updated plex-media-player (<IP_ADDRESS>-9c69e092)

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


Update Hyperterm to Hyper

See https://github.com/zeit/hyper/releases/tag/0.8.1

Thank you for the contribution. It needed some fixes, so they were made in 9e5b3db2d122990b81e9686903577379e769f963. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


LoopVectorization Unroll

Support LoopVectorization using VecUnrolls in codegen
At this point, it's mostly for the sake of making future optimizations easier, but there are already a few benchmarks that improved a little.
Opening this PR now to track coverage and possible test failures here.

Codecov Report

Merging #47 (7260669) into master (112e944) will decrease coverage by 2.29%.
The diff coverage is 14.14%.

@@            Coverage Diff             @@
##           master      #47      +/-   ##
==========================================
- Coverage   62.11%   59.81%   -2.30%
==========================================
  Files          32       32
  Lines        3571     3725     +154
==========================================
+ Hits         2218     2228      +10
- Misses       1353     1497     +144

Impacted Files
Coverage Δ

src/VectorizationBase.jl
68.53% <ø> (ø)

src/fmap.jl
60.83% <0.00%> (-3.34%)
:arrow_down:

src/llvm_intrin/binary_ops.jl
73.68% <0.00%> (-2.00%)
:arrow_down:

src/llvm_intrin/conversion.jl
71.20% <0.00%> (-2.97%)
:arrow_down:

src/llvm_intrin/unary_ops.jl
72.22% <0.00%> (-9.03%)
:arrow_down:

src/promotion.jl
38.15% <0.00%> (-2.12%)
:arrow_down:

src/strided_pointers/stridedpointers.jl
35.97% <0.00%> (-2.85%)
:arrow_down:

src/llvm_intrin/memory_addr.jl
61.05% <9.37%> (-6.72%)
:arrow_down:

src/llvm_intrin/vbroadcast.jl
65.71% <11.11%> (-5.12%)
:arrow_down:

src/vector_width.jl
50.00% <20.00%> (-1.36%)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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


HTML beautifying not working

Description
The results of beautification are not what I expect.
Input Before Beautification
This is what the code looked like before:
<!DOCTYPE html>

    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <meta http-equiv="X-UA-Compatible" content="ie=edge">
    <title>Document</title>

X

Expected Output
The beautified code should have looked like this:
<!DOCTYPE html>

    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <meta http-equiv="X-UA-Compatible" content="ie=edge">
    <title>Document</title>

X

Actual Output
The beautified code actually looked like this:
<!DOCTYPE html>

    <meta charset="UTF-8">
    <meta name="viewport"
        content="width=device-width, initial-scale=1.0">
        <meta http-equiv="X-UA-Compatible"
            content="ie=edge">
            <title>Document</title>

X

Steps to Reproduce

Add code to Atom editor
Run command Atom Beautify: Beautify Editor
This beautified code does not look right!

Note: It doesn't matter if wrap_attributes is set to force or auto
Debug
Here is a link to the debug.md Gist: https://gist.github.com/nivv/737386623d093b42df615da6ad401037
Checklist
I have:

[x] Tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[X] Reloaded (or restarted) Atom to ensure it is not a caching issue
[X] Searched through existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue
[X] Filled out the Input, Expected, and Actual sections above or have edited/removed them in a way that fully describes the issue.
[X] Generated debugging information by executing Atom Beautify: Help Debug Editor command in Atom and added link for debug.md Gist to this issue

Seems related to https://github.com/beautify-web/js-beautify/issues/1103. Maybe we can close this one

Agreed, closing.

This has been resolved by the 3rd party. Any chance of publishing with correct reference to js-beautify?

Can we get this updated as @larsandreasness says? Seems silly to not have the latest fixes from js- beautify

Reinstalling atom-beautify triggers a fresh install of all of the dependencies, and is the recommended solution 👍 .
Feel free to submit a Pull Request to update the version numbers in the package.json, however it would be a tedious process to try and stay up to date and publish a new release for every single dependency in the package.json 😉 .


Getting :  com.google.api.gax.core.GaxProperties.getLibraryVersion(Ljava/lang/Class;Ljava/lang/String;)Ljava/lang/String Error while connecting to Cloud Spanner Via java code

Thanks for stopping by to let us know something could be better!
PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.
Please run down the following list and make sure you've tried the usual "quick fixes":

Search the issues already opened: https://github.com/googleapis/google-cloud-java/issues
Check for answers on StackOverflow: http://stackoverflow.com/questions/tagged/google-cloud-platform

If you are still having issues, please be sure to include as much information as possible:
Environment details

Specify the API at the beginning of the title (for example, "BigQuery: ...")
General, Core, and Other are also allowed as types
OS type and version:
Java version:
google-cloud-java version(s):

Steps to reproduce

?
?

Code example
// example

Stack trace
Any relevant stacktrace here.

External references such as API reference guides used

?

Any additional information below
Making sure to follow these steps will guarantee the quickest resolution possible.
Thanks!

Exception in thread "main" java.lang.NoSuchMethodError: com.google.api.gax.core.GaxProperties.getLibraryVersion(Ljava/lang/Class;Ljava/lang/String;)Ljava/lang/String;
at com.google.api.gax.grpc.GaxGrpcProperties.(GaxGrpcProperties.java:41)
at com.google.cloud.spanner.spi.v1.GapicSpannerRpc.(GapicSpannerRpc.java:161)
at com.google.cloud.spanner.SpannerOptions$DefaultSpannerRpcFactory.create(SpannerOptions.java:71)
at com.google.cloud.spanner.SpannerOptions$DefaultSpannerRpcFactory.create(SpannerOptions.java:66)
at com.google.cloud.ServiceOptions.getRpc(ServiceOptions.java:472)
at com.google.cloud.spanner.SpannerOptions.getSpannerRpcV1(SpannerOptions.java:270)
at com.google.cloud.spanner.SpannerImpl.(SpannerImpl.java:177)
at com.google.cloud.spanner.SpannerOptions$DefaultSpannerFactory.create(SpannerOptions.java:61)
at com.google.cloud.spanner.SpannerOptions$DefaultSpannerFactory.create(SpannerOptions.java:56)
at com.google.cloud.ServiceOptions.getService(ServiceOptions.java:459)
at com.kohls.googlespanner.connections.CloudSpannerConnecton.getConnectionDetails(CloudSpannerConnecton.java:25)
at com.kohls.googlespanner.connections.CloudSpannerConnecton.main(CloudSpannerConnecton.java:59)

@KandasamySurya, this seems like it could be a conflict with other dependencies.  Are you using other Google Cloud libraries?  Can you please share the versions of the cloud libraries you are using?

ok sure sir

Spanner1.zip
These are the jars I have used

Spanner2.zip
Also these

@KandasamySurya, are you using maven?  If so, can you please do a mvn dependency:tree?

Spanner2.zip has different versions for the same artifacts.  That's going to be a problem.
Another problem is that you have dependencies for google cloud libraries that may be incompatible; you're using google-cloud-(storage|spanner|logging) that are not compatible.  Please see here for list of compatible versions.

ok . I will check with the compatible versions. Thanks a lot for your valuable input


[ZEPPELIN-1015] Cron job fails to run a paragraph when multiple type of interpreter is being used

What is this PR for?
Cron job can fail when notebook uses multiple types of paragraphs.
Problem reported here http://apache-zeppelin-users-incubating-mailing-list.75479.x6.nabble.com/Cron-job-fails-to-run-a-paragraph-that-runs-correctly-manually-tt2265.html
What type of PR is it?
Bug Fix
Todos

[x] - Fix
[x] - Unittest

What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1015
How should this be tested?
Create two paragraphs in the notebook
First takes longer than second (last) paragraph.
First paragraph and second paragraph should use different interpreter.
If cron schedule the notebook with 'auto-restart interpreter on cron execution' checked.
Then interpreters will be restarted when second paragraph finished, but first paragraph is still running.
That may cause abort of first paragraph run.
Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

LGTM

LGTM

Rebased to resolve conflicts to master.
Thanks @felixcheung and @cloverhearts for review.
Merge if there're no more discussions.

LGTM too, thanks @Leemoonsoo I've also tried with same interpreter and also works

@jmrr Thanks for verifying!


fix a bug that spring-mvc set an error endpoint name

fix a bug that spring-mvc set an error endpoint name if the controller class annotation implements an interface.

recently, I find some error logs in the OAP server like this

after decoding the endpoint id, I found the endpoint name is empty. after adding more logs in the trace segment receiver, I found the error from the spring-MVC plugin.
if the controller class annotation implements an interface, it can not get the RequestMapping. in the spring-cloud  tech stack, it is common sense, like RPC framework is feign
example
@RequestMapping(value = "/for")
public interface A {
    @RequestMapping
    String bar();
}

@RestController
public class B implements A{

    @Override
    public String bar() {
        return "oops";
    }
}

result in oap server :

Fix

[ ] Add a unit test to verify that the fix works.

[x] Explain briefly why the bug exists and how to fix it.

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.

[x] Update the CHANGES log.

You'd better add this case into the test scenarios about Spring framework.  @darcydai

You'd better add this case into the test scenarios about Spring framework. @darcydai

Here are those test scenarios about Spring framework:

spring-4.3.x-scenario
spring-3.1.x-scenario
spring-4.1.x-scenario
spring-4.3.x-scenario

You'd better add this case into the test scenarios about Spring framework. @darcydai

Here are those test scenarios about Spring framework:

spring-4.3.x-scenario
spring-3.1.x-scenario
spring-4.1.x-scenario
spring-4.3.x-scenario

I will do this later

You'd better add this case into the test scenarios about Spring framework. @darcydai

Here are those test scenarios about Spring framework:

spring-4.3.x-scenario
spring-3.1.x-scenario
spring-4.1.x-scenario
spring-4.3.x-scenario
I found  a scenarios can only define one entry service,can i add more scenario in spring mvc for the special case ?
@ascrutae

I found a scenarios can only define one entry service,can i add more scenario in spring mvc for the special case ?

You don't need to. Do a cycle call inside the entrance service. You could have as many /PATH as you like, unless timeout due to too slow

I found a scenarios can only define one entry service,can i add more scenario in spring mvc for the special case ?

You don't need to. Do a cycle call inside the entrance service. You could have as many /PATH as you like, unless timeout due to too slow

thanks, i got it

I found a new bug when I coding the test case for spring MVC, I has fix it in last commit.

DO NOT MERGE UNTIL I FINISH SCENARIOS TEST CASE

I found a new bug when i coding the scenarios for the spring MVC plugin. i have fixed in last commit
org.apache.skywalking.apm.plugin.spring.mvc.commons.interceptor.RequestMappingMethodInterceptor#getRequestURL

the annotation RequestMapping with the value[value()] array length equals zore represent an empty string in spring MVC concept.

RequestMapping in spring mvc 3.x and 4.1  has no the path method [path()], if RequestMapping with  with the value[value()] array length equals zore in spring mvc 3 will case no such method error;

because of AliasFor annotation, no need to call path(), I have tested

do not  merge before i finish scenario test for spring mvc.

do not merge before I finish the scenario test for spring MVC.

I expect you could update the plugin test scenarios.

@darcydai Please fix your test case.

Seems still have style issues?

Seems still have style issues?

seems failed scenario test is not spring MVC, how to trigger  the task run again

Seems still have style issues?

seems failed scenario test is not spring MVC, how to trigger the task run again

I will try to rerun, I think we have located the issue.


[BOOK] Chapter 15 should have adwaita-icon-theme included in macOS install command

When using the book to install libadwaita on macOS, I tried running an example (blank) application with just a titlebar and an about dialog. However, I could not add an application icon to the about dialog window because it kept appearing as the missing icon texture. After doing some more digging into this issue, it turned out that the machine must have the Homebrew package adwaita-icon-theme also installed.
As a result, the command should be changed to:
brew install libadwaita adwaita-icon-theme
Machine: iMac Retina 4K, 21.5" 2017 running macOS Ventura 13.6.7
Homebrew version is 4.3.10
GTK version is 4.14.4
Libadwaita version is 1.5.2

Thanks for reporting @MonsieurRacecar!
Can you please open a pull request to fix this? :)


Add Explicit POST Method to Curl Calls

Using -T flag defaults to PUT method
Fixes compatibility with "slack compatible" APIs like rocketchat

Thanks!


attr.define seems to break init_subclass (works with attr.s)

I like to use __init_subclass__ to implement simple plug-in systems, e.g.:
class OptionType:
    name: str
    __types__: dict[str, "OptionType"] = {}

    def __init_subclass__(cls, **kwargs):
        super().__init_subclass__(**kwargs)
        print(cls.name, cls)
        cls.__types__[cls.name] = cls

    @classmethod
    def instance_for(cls, name):
        return cls.__types__[name]()

class Int(OptionType):
    name: str = "int"

class IntRange(Int):
    name: str = "int-range"

print(OptionType.instance_for("int"))

When this is run, __init_subclass__ is run exactly once per class:
int <class '__main__.Int'>
int-range <class '__main__.IntRange'>
<__main__.Int object at 0x101462c70>

This also works with attr.s:
import attr

class OptionType:
    name: str
    __types__: dict[str, "OptionType"] = {}

    def __init_subclass__(cls, **kwargs):
        super().__init_subclass__(**kwargs)
        type_name = cls.name._default
        print(type_name, cls)
        cls.__types__[type_name] = cls

    @classmethod
    def instance_for(cls, name):
        return cls.__types__[name]()

@attr.s
class Int(OptionType):
    name: str = attr.ib("int")

@attr.s
class IntRange(Int):
    name: str = attr.ib("int-range")

print(OptionType.instance_for("int"))

int <class '__main__.Int'>
int-range <class '__main__.IntRange'>
Int(name='int')

However, when I use attr.define, everything falls appart:
import attr

_OPTION_TYPES: dict[str, "OptionType"] = {}

@attr.define
class OptionType:
    name: str

    def __init_subclass__(cls, **kwargs):
        super().__init_subclass__(**kwargs)
        type_name = attr.fields(cls).name.default
        print(type_name, cls)
        _OPTION_TYPES[type_name] = cls

    @classmethod
    def instance_for(cls, name):
        return _OPTION_TYPES[name]()

@attr.define
class Int(OptionType):
    name: str = "int"

@attr.define
class IntRange(Int):
    name: str = "int-range"

print(OptionType.instance_for("int"))

NOTHING <class '__main__.Int'>  # WHAT?
int <class '__main__.Int'>
int <class '__main__.IntRange'>  # ONOES! :-O
int-range <class '__main__.IntRange'>
IntRange(name='int')  # 😿

The main reason is, that __init_subclass__ is now called for every class in the inheritance hierarchy. That means that:

the base class must also use attrs.define (or attr.fields(cls) (or cls.name.default) will break)
__types__ must move out of the class or I’ll get a TypeError: 'member_descriptor' object does not support item assignment (because since it is typed, an attrs attribute is created for it).
IntRange overrides the class for int, so OptionType.instance_for("int") now returns an IntRange() instead of an Int().

I have not yet time to dig deeper into it but I always had the impression that attr.define is just an alias for attr.s with nicer defaults but that seems not to be the case. 🤔

It's probably __slots__ interfering with it. Try the attr.s example with slots=True.

of course, i'm open for another hack to make some feature work with slots ;)

@Tinche: It's probably __slots__ interfering with it. Try the attr.s example with slots=True.

The third example works if i set define(slots=False).  If I also set auto_attribs=False, I can resemble the behavior of the second example.  However, this only works with attr but not with the attrs import:
import attr

@attr.define(slots=False, auto_attribs=False)
class OptionType:
    name: str = attr.field()
    __types__: dict[str, "OptionType"] = {}

    def __init_subclass__(cls, **kwargs):
        super().__init_subclass__(**kwargs)
        type_name = attr.fields(cls).name.default
        print(type_name, cls)
        cls.__types__[type_name] = cls

    @classmethod
    def instance_for(cls, name):
        return cls.__types__[name]()

@attr.define(slots=False)
class Int(OptionType):
    name: str = "int"

@attr.define(slots=False)
class IntRange(Int):
    name: str = "int-range"

print(OptionType.instance_for("int"))

@hynek: of course, i'm open for another hack to make some feature work with slots ;)

That’s exactly the answer I was hoping for.  Please notify me when you implemented this! 🤪

Where exactly would you like that to be?
It's currently prominently in the attrs.define docs:

The hint in braces is, compared to the amount and potential severness of the issues, not visible enough. I would use an admonition (attention or caution maybe) with a strong advice for users to acutally read the slotted classes gotchas.

Looks good, thanks :)


Create missing directories required by the services/operators

:warning: TBD  – not ready for work
It must be clarified if the agent is responsible for creating missing directories required by the services/operators.
Furthermore it must be specified who defines the directories and all directories must be listed.

clarification process is started here with ADR "worker node directories".
https://github.com/stackabletech/documentation/blob/adr-worker-node-directories/modules/adr/pages/drafts/ADRnnn-worker-node-directories.adoc


Support for class methods

Hi! First of all I loved the idea so far, thanks for making it open source! I also thought of something like this but I never actually implemented it. Sorbet is a little bit heavy lift and RBS is too far from runtime to me, but this is a simple implementation. I even think it may be eventually be able to export to RBS for IDE support (an entirely different conversation, but wanted to mention it)
While trying it out i found out it does not have support  for class methods. Here's an example on how to reproduce
require 'delivered'

class User
  extend Delivered::Signature

  sig String, age: Integer, returns: String
  def initialize(name, age:)
      "User #{name} created with age #{age}"
  end

  sig String, returns: String
  def self.example(user)
      123
  end
end

User.example(1)
# should throw error because of the integer as parameter, and also because of the integer as returned value

I may be able to submit a PR for this in the short term, but wanted to submit the issue first, in case you or someone else wants to do it before

More than happy for you to submit a PR, thx. I would expect that I would need this in not too distant future