Mockup vs API review

Check the API coverage for the mockups

Homepage
https://marvelapp.com/1e21bb3#13388532
Most important findings for homepage:

the OpenAQ's data section requires calls to 4 different endpoints
the Compare Locations will involve quite some processing (pagination, filtering on date, etc)

OpenAQ's data
The four stats in this section are available through 4 separate endpoints:

locations - https://api.openaq.org/v1/locations

measurements - https://api.openaq.org/v1/measurements

countries - https://api.openaq.org/v1/countries

sources - https://api.openaq.org/v1/sources

The response always follows the same structure:
Response:
{
  "meta": {
    "found": 2555
  }
}

Compare locations
The data for the chart comes from the measurements endpoint.
The response is paginated, with a default limit of 100, and a max limit of a 1000 results. A 100 results should be good for 24 hours of measurements with a 15 minute interval. A weekly chart with that reporting interval will require a little more than 1000 measurements.
Things to note:

this may require pagination, depending on the reporting interval
the endpoint contains all the meta information for the chart, except for proper country name (it returns the ISO code)
it is possible to request data for a specific time-frame, but the date parameter filters the result on utc time, not on local time

Example:
https://api.openaq.org/v1/measurements?parameter=pm25&location=Bukhiin urguu&limit=100&date_from=2016-07-04T00:00:00&date_to=2016-07-11T00:00:00
Recommended for you
All the information can be fetched using the locations endpoint, specifying a set of coordinates and max radius in meters.
Example:
https://api.openaq.org/v1/locations?coordinates=52.3723,4.8792&radius=2000

Locations page
https://marvelapp.com/1e21bb3#13386008
Download data button
Comes from the measurements endpoint.
Locations sections
The country filter can be constructed using the countries endpoint
The data for the results can be fetched from the locations endpoint
Notes:

it is not possible to specify multiple parameters and countries, which we need for the filters
there is no endpoint that provides all the parameters in the system
the # cities would have to be calculated client-side, loading all the data. I'd suggest we drop this.

Map
https://marvelapp.com/1e21bb3#13414770
The map data can be fetched using the latest endpoint, passing the has_geo parameter.
Example: https://api.openaq.org/v1/latest?has_geo
Two endpoints are necessary for the other locations functionality in the popup:
1. locations endpoint passing coordinates + radius to determine locations in proximity
2. latest endpoint for the values of those locations

@olafveerman to your point about # of cities on Locations page above, does https://api.openaq.org/v1/cities get you what you need?


error: RPC failed; curl 18 transfer

Tried on a fresh install of Ubuntu 18.04.6 on VirtualBox 6.1 on Windows 10
Build end, puts git version  2.38.0-rc1 in /usr/bin/git
But when I try to clone a repo, it says:
remote: Enumerating objects: 26528, done.
remote: Counting objects: 100% (150/150), done.
remote: Compressing objects: 100% (136/136), done.
error: RPC failed; curl 18 transfer closed with outstanding read data remaining
error: 6687 bytes of body are still expected
fetch-pack: unexpected disconnect while reading sideband packet
fatal: early EOF
fatal: fetch-pack: invalid index-pack output

I'm not able to replicate this locally. The problem you're having looks to be rooted in your internet connection. I'd double check your local configuration as well.
Googling your error seems to point to this as the culprit: https://stackoverflow.com/a/73079738/1478636

Appreciate your quick answer. It certainly makes sense what you say as it´s hard to believe git was that much broken. But I have tried many different possibilities, mostly with fresh ubuntus on VMs.
I have a good internet and even if it was its fault I would have noticed the problem being intermitent. I also have no antivirus, not even the W defender and the firewall is off. It's a developer machine, there's nothing of permanent value (I mean, the physical win machine).
From the recipes in your links, some of them were old stuff to me, others new, I tried them to no avail...
I know there are workarounds, like shallow clones, etc, but it's a makefile that uses git, so these don't apply.
I ran out of ideas (on the git side). I'm thinking now of blaming either the machine or its network settings or dive into that makefile and make all git clone shallow...
Thanks again for your help


Exception when using diectory prefix

When creating a file using a prefix, the file uploads, but then the following exception is thrown:
Code: 404
Value: The specified parent path does not exist.
details (if any): <?xml version="1.0" encoding="utf-8"?><Error>ParentNotFound<Message>The specified parent path does not exist.

When a file is created or updated, the metadata is fetched from Azure as a separate operation. It looks like that operation is failing, since the file does get created and in the correct place. So I'm guessing the metadata fetch is not being given the correct path.

This is working now.


User-guide or tutorial for using linked libraries with peer dependencies in CRA v2

Is there a guide or tutorial on how to use libraries that are linked (yarn link or link:<> with CRA v2.  I have multiple CRA apps and multiple shared libraries that work fine with CRA v1 but break in V2 with Module not found: Can't resolve '<<package>>'.  I've been googling, searching issues, and experimenting to try and find the solution.  There's lots of comments on this but no definitive answer/guide.  Do I need to use yarn workspaces, lerna, install my peer-dependencies as regular dependencies in my libraries, or what?  Trial-and-erroring on a non-trivial project size is very time-consuming and it would be great if there was a definitive answer/guide on how to make this work.  Supporting shared libraries is very important and something that is worthy of being addressed explicitly.  I'm currently stuck using<EMAIL_ADDRESS>because of this.  I would love to upgrade as my project is large, build times are long, and I'm hopeful CRA v2 will help.

Since CRA 2 doesn't have out of box support for a monorepo, the error you got is probably more about the way you set things up and less about CRA.  I am not sure what would cause a setup to stop working compared to CRA 1.1.5 unless I know more about your setup.
I generally wouldn't recommend yarn link in a monorepo setup.  The reason is that if you do a yarn install, you will blow up whatever you've linked before, and every developer has to do this linking step on their own.
lerna is generally more useful if you are publishing your packages or running a lot of scripts across all workspaces.  It doesn't sound like you are doing that lerna is strictly optional.
yarn workspace is useful if you are always using packages from head.  If you want to fix versions of your internal dependencies on the other hand, you should publish or use the result from yarn pack.
If you've setup your workspaces properly, you should not find errors such as Module not found: Can't resolve '<<package>>'.  Double check the packages are correctly linked in the root node_modules.  Please refer to https://yarnpkg.com/lang/en/docs/workspaces/ on setting it up.

@bugzpodder Thanks for thorough response and info.  I was really leaning towards using yarn workspaces.  Based on the document and what I'm doing it seemed to make sense... and you're reinforcing that.  I will try using workspaces and see if that resolves my issues.  Stay tuned...

Closing issue.  Converting to yarn workspaces indeed fixes the above Module not found: Can't resolve '<<package>>' error and provides for a much improved DX.  I feel a little slow for only now converting... oh well.  @bugzpodder thanks for the assistance.  It would be good if there were a comprehensive guide/how-to for setuping a non-trival project with multiple packages and CRA apps but I don't believe that belongs here.


'contentUrl' & 'entityId' redirection link

In this section for both 'contentUrl' & 'entityId', update the hyperlink to below:
https://learn.microsoft.com/en-us/microsoftteams/platform/resources/schema/manifest-schema#statictabs

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

ID: e44c95e7-3258-df7e-224d-d60ea1cac46c
Version Independent ID: e44c95e7-3258-df7e-224d-d60ea1cac46c
Content: Build tabs for meeting - Teams
Content Source: msteams-platform/apps-in-teams-meetings/build-tabs-for-meeting.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: surbhigupta

Hi @surbhigupta12 - Could you please help here to update the doc as mentioned?


Code refactoring in subject area. And fixed bug for find all categories without request params.

Code refactoring in subject area. I brought the code to a more readable form. Also, along the way, I fixed some shortcomings in the form of calling deprecated methods and the like.
Removed unnecessary double "new HashSet". But I’m still worried that these are mutable collections and that in 95% of cases they’re simply not usedRemoved unnecessary double "new HashSet". But I’m still worried that these are mutable collections and that in 95% of cases they’re simply not used

In case of commit  https://github.com/odpi/egeria/pull/3050/commits/d0860ebe3897c4860c3413736290cab5eb00b044
I noticed that the toString method returns a StringBuilder which is not correct based on the logic. I decided to correct this defect, but I realized that a lot of architectural logic was already tied to this method. You can fix it in absolutely all classes and simultaneously remove a lot of duplicated code, but I was worried about the question "Will there be a backward compatibility problem?". So I rolled back all my changes, but missed it.

Before fixing the bug, when requesting all categories without parameters to
_http://localhost:8080/servers/test/open-metadata/access-services/subject-area/users/admin/categories_

I was get next exception:
{
    "class": "InvalidParameterExceptionResponse",
    "relatedHTTPCode": 200,
    "responseCategory": "InvalidParameterException",
    "exceptionClassName": "org.odpi.openmetadata.accessservices.subjectarea.server.handlers.ErrorHandler",
    "exceptionErrorMessage": "OMAS-SUBJECT-AREA-400-035 Invalid parameter processing request findCategory.",
    "exceptionSystemAction": "The system is unable to process the request.",
    "exceptionUserAction": "Please correct the parameter and retry."
}

After I fixed the bug and i've filling catalog via org.odpi.openmetadata.accessservices.subjectarea.samples.SubjectAreaDefinitionSample.
And I was get next the picture when I was called all categories again, I get only one entity instead of all and with each request different.
The fact is that the findCategory method in the class org.odpi.openmetadata.accessservices.subjectarea.handlers.SubjectAreaCategoryHandler
has the following limitation:
response = getCategory(userId, entityDetail.getGUID());
                    if (response.getResponseCategory() == ResponseCategory.Category) {
                        CategoryResponse categoryResponse = (CategoryResponse) response;
                        Category category = categoryResponse.getCategory();
                        categories.add(category);
                    } else {

That is, if an entity arrives with a different ResponseCategory other than Category, the method does not add (in this case, SubjectAreaDefinition).
The problem is clearly of an architectural nature, as when adding any new ResponseCategory, the method will grow "if else", which will make the code less supported. The architecture of work in this case is worth rethinking.
This problem is absent if you simply use the Category entity, so I changed the logic of SubjectAreaDefinitionSample in my next commit to have a working example.
Thx. I hope for a speedy review, because current error is present in production.

I also noticed that in the class org.odpi.openmetadata.accessservices.subjectarea.server.services.SubjectAreaRESTServicesInstance there are two big unused methods "getRelationshipsFromGuid" written a year ago

@immmus Thanks for your pr. I am finding it difficult to review this change as there is a lot of reformatting. It would really help if you could split this up into small minimal changes each with an issue and pr. If there are refactors then an issue and pr for just that refactor. A targeted fix on the bug in master you talk about could be quickly reviewed if that fix were by itself.

@immmus Thanks for your pr. I am finding it difficult to review this change as there is a lot of reformatting. It would really help if you could split this up into small minimal changes each with an issue and pr. If there are refactors then an issue and pr for just that refactor. A targeted fix on the bug in master you talk about could be quickly reviewed if that fix were by itself.

Hi, if you say about request for search all categories then it here https://github.com/odpi/egeria/pull/3050/commits/0d0d7752568adc4540a4e6f64097f9ed4b7a4bf4

I am not sure this fix is correct, the change means that sequencingOrder and sequencingProperty will not be applied when there is no searchCriteria. Also all API changes need to be tested in the SubjectAreaFVT. SubjectAreaFVT tests all the APIs that are known to work.
I am thinking

I suspect if we specify .* when we are requested with null as the searchCriteria, we should then be able to use the findEntitiesByPropertyValue and the sequencingOrder and Property.
I have not looked into what searchCriteria of null means to OMRS, whether this means it looks for null values or unspecified values. The Conformance tests may/should check the behaviour; I have not investigated.
I suspect an equivalent change would be need to Project Glossary and Term processing

Yes, to search for all Terms, Glossaries and Projects, the same logic of work is used. And in my correction, I relied on their logic of work, considering it correct. The searchCriteria of null means that we are looking for everything without any search criteria, as I understand it.
If null searchCriteria, sequencingOrder and sequencingProperty should be used. Then the logic of work for all of them is written and works incorrectly.
By the way, I ran the tests, they all passed. After all, they absolutely do not check the logic of work for sequencingOrder and sequencingProperty as it turned out.

Please can you split this into separate issues and have pr for each:
-one for the reformats (not refactor)

one for the search issue
one for the warnings around hashmaps.
any others?
It would be so much easier to review and handle.

Please can you split this into separate issues and have pr for each:
-one for the reformats (not refactor)

one for the search issue
one for the warnings around hashmaps.
any others?
It would be so much easier to review and handle.

Well, I’ll try to scatter all these tasks on different pr, it will take time and this pr will need to be closed

@immmus  I am actually in the middle of refactoring subject area around the error messages to change the SubjectAreaError processing to use the new MessageDefinitions as part of #3070. I am going to complete this refactor and push that change (I will try to remember to format the file after my changes :-) - as I have left in a few formatting errors - which you have rightly picked up on); I will then endeavour to work with you so we can make the appropriate changes for the issues you have raised.

@immmus I am actually in the middle of refactoring subject area around the error messages to change the SubjectAreaError processing to use the new MessageDefinitions as part of #3070. I am going to complete this refactor and push that change (I will try to remember to format the file after my changes :-) - as I have left in a few formatting errors - which you have rightly picked up on); I will then endeavour to work with you so we can make the appropriate changes for the issues you have raised.

I made all my changes in separate pr as you asked. Waiting for your approve.
https://github.com/odpi/egeria/pull/3071
https://github.com/odpi/egeria/pull/3069
https://github.com/odpi/egeria/pull/3067

@davidradl  I can to close this? You can are checking my new pr's


Bump dependencies

Bump dependencies.
Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

@maarcingebala any possibilities of bumping graphene as well? because all graphql libraries are using graphql core v3 except graphene v2 (which saleor is using), this will allow the possibility of not using graphene django in plugins and use any alternatives, because currently, the only thing that supports graphql core v2 is graphene django (same version as graphene)

@LeOndaz We would love to do that but I think we are still blocked. Some work has been done recently to unblock it, but i need to double-check with the team where we are with this. Here is a PR with more details: https://github.com/saleor/saleor/pull/8503


fix inverted logic in docs

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @therealprof (or someone else) soon.
Please see the contribution instructions for more information.


dhcpd.conf generated with wrong pool { } section

I introduced a new vlan and created a new pool for that interface. I got this in the log file:
DHCPDISCOVER from xx:xx: via vlan01: network <IP_ADDRESS>/24: no free leases

In dhcpd.conf I found this
subnet <IP_ADDRESS> netmask <IP_ADDRESS> {
  pool {
    option domain-name-servers <IP_ADDRESS>;
    range <IP_ADDRESS> <IP_ADDRESS>;
  }

  option routers <IP_ADDRESS>;
  option domain-name-servers <IP_ADDRESS>;
  default-lease-time 3600;
  max-lease-time 4000;

}

If I remove the nameserver option, it looks like that
subnet <IP_ADDRESS> netmask <IP_ADDRESS> {
  pool {
    range <IP_ADDRESS> <IP_ADDRESS>;
  }

  option routers <IP_ADDRESS>;
  option domain-name-servers <IP_ADDRESS>;
  default-lease-time 3600;
  max-lease-time 4000;

}

I'm running OPNsense 22.7.3_2-amd64 .
Cheers, Lutz

closed and recreated #6006


[Documentation]: Automated Metadata Extraction for Better Retrieval + Synthesis Nodes Reusage Question

Documentation Issue Description
In your  metadata extraction documentation
You are creating the nodes utilizing the node_parser with
orig_nodes = node_parser.get_nodes_from_documents(docs)
# take just the first 8 nodes for testing
nodes = orig_nodes[20:28]

Then the ingestion_pipeline is created with
pipeline = IngestionPipeline(transformations=[node_parser, *extractors_1])
Finally, it's being run with
nodes_1 = pipeline.run(nodes=nodes, in_place=False, show_progress=True)

does not this run the node_parser on the inputs again?
Documentation Link
https://docs.llamaindex.ai/en/stable/examples/metadata_extraction/MetadataExtraction_LLMSurvey.html#automated-metadata-extraction-for-better-retrieval-synthesis

@dosu-bot run_transformations method uses the instance variable self.transformations which has the node_parser in it. NodeParser class' __call__ method calls the get_nodes_from_documents method so it might be running. I did not find any type checks in the code as well.

I still need feedback on this, it seems that it's called twice?

ping

@ravi03071991 yea just a bug, didn't mean to parse the nodes twice


Change PIN feature: work in progress

Description
This PR begins to implement a change pin feature for the user.
This is still a work in progress, but i wanted to get some eyes on the code for now.
Included is a recording of the mechanics, the UI still needs to be worked on!
https://user-images.githubusercontent.com/12700801/111392579-2204cb80-8674-11eb-9322-1b488d88f937.mp4
Other changes
Changes some types.
Tested
Testing on iOS for now.
Related issues

Fixes #[issue number here]

Backwards compatibility

Looks good so far. Just need some UI tweaks and also it probably shouldn't go to the restore existing wallet screen. It's doing that now because of the first time sign up flow right?

Looks good so far. Just need some UI tweaks and also it probably shouldn't go to the restore existing wallet screen. It's doing that now because of the first time sign up flow right?

im not entirely sure... this is the part im debugging right now. could it be something with verification refactoring? and yes some UI tweaks! haha

closed for new PR


disable / enable flag in .rclone.conf for remotes

Have a flag in .rlcone.conf to enable/disable a remote ?
It would be great if we can just disable few remotes instead of deleting them from the conf file.
This should stop them from showing up in the listremotes and won't allow any operations on it.
Would like to know what others think!

Not a bad idea... Why do you need it?

To avoid confusion for the less informed people on the team. While temporarily not using a cloud provider, admin can disable it in the .conf file instead of deleting. Not a high priority task, just a could be better feature.

@ncw, I just started on this enhancement and want your opinion on how to build this.
I seem that the essayist way of adding this feature is to add a disabled variable inside Option inside RegInfo in every fs.
My point of view is that the option to disable a remote only should be visible when a remote is edited, and not when it initially is created.
By adding this enhancement using the Option structure, the user will be prompted with a question if the remote should be disabled when a new remote is configured. We could maybe circumvent this from happening, or just leave it to be default enabled.
What do you think?


README.md: Add information from blog, reference PkgCacheInspector

Fix: #18

Very nice, thanks so much @mkitti!


Custom Album Path

Is your feature request related to a problem? Please describe.
My pictures folder is already extremely cluttered by various apps, with pictures I don't need in my gallery nor in my Nextcloud sync. I would like to use the Upload Files Feature of the Nextcloud App to sync all my sorted Albums to Nextcloud, but without all the other junk in my Pictures folder.
Describe the solution you'd like
It would be great to be able to adjust the default album path, so I can choose "pictures/albums/" instead of "pictures/" and sync only the albums subfolder to Nextcloud.
Describe alternatives you've considered

Having an app that integrates with Nextcloud like Les Pas and allows you to sync new albums directly to nextcloud would an alternative. However I don't like the private app data aspect of the app and the lacking features of gallery management. Your app would suit my needs much better, even without Nextcloud integration.
very few apps use sub folders for there albums, which would already be enough for me (like the default OnePlus Gallery "DCIM/myAlbums/"). However, choosing this app versus your vastly superior one would be heartbreaking 😭.
syncing all the other junk in my "pictures/" folder to nextcloud as well.

Additional context
When I granted the permission for your app to write to the pictures folder, it looked like everything needed to set a custom path was already in place, except for the part where any other folder than "pictures/" would be accepted.

Duplicate:

#295

In the meantime for your case, you can easily create the new album in a subfolder of "Pictures" by giving the album name "albums/Holidays", instead of "Holidays".

However, choosing this app versus your vastly superior one would be heartbreaking 😭.

Flattery, yes! :D

Thanks a lot, that is a great middle way. I didn't know this was possible. And sorry for the dublicate, I really tried to find anything related to this topic.

No worries ^^


Update install docs and add Docker article

Fixes AB#1681601
@dcaro I made the changes we discussed. Please review and sign off.

Docs Build status updates of commit 620cade:
:clock10: Full build: the average full build time is 23 min(s) 9 sec(s), this is based on the last 320 build(s) for this repository.

Docs Build status updates of commit 620cade:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-conceptual/azps-3.6.1/azureps-in-docker.md
:warning:Warning
View (azps-3.6.1)
Details

docs-conceptual/azps-3.6.1/install-az-ps-msi.md
:warning:Warning
View (azps-3.6.1)
Details

docs-conceptual/azps-3.6.1/install-az-ps.md
:warning:Warning
View (azps-3.6.1)
Details

docs-conceptual/azps-3.6.1/release-notes-azureps.md
:warning:Warning
View (azps-3.6.1)
Details

docfx.json
:white_check_mark:Succeeded

docs-conceptual/azps-3.6.1/toc.yml
:white_check_mark:Succeeded
View (azps-3.6.1)

docs-conceptual/azps-3.6.1/azureps-in-docker.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

docs-conceptual/azps-3.6.1/install-az-ps-msi.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

docs-conceptual/azps-3.6.1/install-az-ps.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

docs-conceptual/azps-3.6.1/release-notes-azureps.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 30b0196:
:clock10: Full build: the average full build time is 23 min(s) 12 sec(s), this is based on the last 321 build(s) for this repository.

Docs Build status updates of commit 30b0196:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

docs-conceptual/azps-3.6.1/azureps-in-docker.md
:white_check_mark:Succeeded
View (azps-3.6.1)

docs-conceptual/azps-3.6.1/install-az-ps-msi.md
:white_check_mark:Succeeded
View (azps-3.6.1)

docs-conceptual/azps-3.6.1/install-az-ps.md
:white_check_mark:Succeeded
View (azps-3.6.1)

docs-conceptual/azps-3.6.1/release-notes-azureps.md
:white_check_mark:Succeeded
View (azps-3.6.1)

docs-conceptual/azps-3.6.1/toc.yml
:white_check_mark:Succeeded
View (azps-3.6.1)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


AOT - Module did not self-register

I am attempting to create a very simple "Hello World" C# project for AOT, following the documentation here https://github.com/microsoft/node-api-dotnet/blob/main/Docs/node-module.md
The AOT module was created with the latest preview version of .NET 8.0.0-rc.2, using Microsoft Visual Studio Community 2022 (64-bit) - Preview Version 17.8.0 Preview 6.0.
I included the following packages and updated the project settings according to the docs:
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>net8.0</TargetFramework>
    <PublishAot>true</PublishAot>
    <PublishNodeModule>true</PublishNodeModule>
    <ImplicitUsings>enable</ImplicitUsings>
    <Nullable>enable</Nullable>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.JavaScript.NodeApi" Version="0.4.18" PrivateAssets="all" />
    <PackageReference Include="Microsoft.JavaScript.NodeApi.Generator" Version="0.4.18" PrivateAssets="all" OutputItemType="Analyzer" ReferenceOutputAssembly="false"/>
  </ItemGroup>

</Project>

Below is the only code in the project marked with the JSExport Attribute
using Microsoft.JavaScript.NodeApi;

namespace AotTest
{
    [JSExport]
    public class MyClass
    {
        public void Hello()
        {
            Console.WriteLine("Hello World from C# AOT!");
        }
    }
}

I was able to get it to compile and publish a .node file and .d.ts file which I referenced in a nodeJS TypeScript project.
However, when attempting to load the module, I receive the following error

node:internal/modules/cjs/loader:1327
return process.dlopen(module, path.toNamespacedPath(filename));
Error: Module did not self-register: 'full-path-to-aot-module\AotTest.node'.
at Module._extensions..node (node:internal/modules/cjs/loader:1327:18)
at Module.load (node:internal/modules/cjs/loader:1091:32)
at Module._load (node:internal/modules/cjs/loader:938:12)
at Module.require (node:internal/modules/cjs/loader:1115:19)
at require (node:internal/modules/helpers:130:18)
at Object.
...
code: 'ERR_DLOPEN_FAILED'
}
Node.js v20.9.0

Are you able to run the aot-module example project? It is very similar to what you have.
Check that the source-generator produced the module registration code. After the build you can find the generated .g.cs source file under obj/Debug/net8.0/win0x64/generated/, and it should have lines similar to this near the top, which should enable Node.js to register the module:
public static class Module
{
    [UnmanagedCallersOnly(EntryPoint = "napi_register_module_v1")]
    public static napi_value _Initialize(napi_env env, napi_value exports)
        => Initialize(env, exports);

In your .csproj file, try updating the <PackageReference> items to match how they are in the example. I'm not sure if your extra attributes there are causing a problem. (The documentation might be out of date.)

Thanks for the feedback.  The generated .g.cs file was not being generated.  I tried the example project and was getting the same result.  In both cases, I was attempting to publish the project using the IDE, which produced a .node file but it would not register.  I decided to delete the publish profile created by the IDE and just run dotnet publish from the CLI, which correctly builds the module and allowed me to import the example project successfully in NodeJS.
When I attempted to do the same thing in my test project, I had to specify a runtime identifier when publishing using dotnet publish -r win-x64.  I assume this is because the example project inherited this from the parent folder configuration.   Afterwards I was able to get it to work in my node app.  The only thing I had to change, was adding //@ts-ignore above the import because Typescript did not recognize the file as a module.

I was attempting to publish the project using the IDE

I haven't tried that; I don't know exactly what it does. The .g.cs file is generated by a C# compiler source-generator that should run during any build, independent of the publish step. So I don't understand why it wouldn't get generated when building/publishing from the IDE.

The only thing I had to change, was adding //@ts-ignore above the import because Typescript did not recognize the file as a module.

It should be possible to get TypeScript type-definitions from importing the module, since a .d.ts is auto-generated also. That may depend on your project settings (commonjs vs ES modules) and the syntax used to import, so I'd need more info to troubleshoot.

I haven't tried that; I don't know exactly what it does. The .g.cs file is generated by a C# compiler source-generator that should run during any build, independent of the publish step. So I don't understand why it wouldn't get generated when building/publishing from the IDE.

I don't know the difference either.  I did a little research on the differences between dotnet build  and Visual Studio/MSBuild.  For the most part, they should produce the same result, but I did come across this SO post claiming that Analyzers targeting .NET Core/.NET 5+ will only run from dotnet build, which might explain why NodeApi.Generator is not producing the generated code.  In fact, I have created a more complex project that compiles from Visual Studio, but throws errors from the CLI, so there is definitely something different about how the project is compiled.
Knowing there is at least a workaround by building and publishing from the CLI, I am going to close this issue.
FYI, my main interest in this project is possibly having a better way to call Soap APIs from our Node application.  While there are several npm packages out there, I have not found an elegant solution for Soap in NodeJS/TypeScript.  I work with some very large third party Soap APIs with thousands of models and .NET is very good at generating client code using svcutil.exe.  I was hoping to be able to generate the client code that consumes these APIs and create C# projects that can be called from Node.

Analyzers targeting .NET Core/.NET 5+ will only run from dotnet build

Yes that would explain it: NodeApi.Generator is an analyzer (source-generator) that targets .NET 6 or later. I wasn't aware of that limitation of analyzers in VS. I assume it's because VS MSBuild runs on .NET Framework 4.x. We could investigate making the analyzer target .NET Standard 2.0 or .NET Framework to support that.

I was hoping to be able to generate the client code that consumes these APIs and create C# projects that can be called from Node.

Interesting!


core: remove some refs to server key

The idea here is to remove refs to InstanceInfo and Server, so that they can be encapsulated more easily.


Basic CSS: Use Hex Code to Mix Colors

https://www.freecodecamp.org/learn/responsive-web-design/basic-css/use-hex-code-to-mix-colors

If a hex triplet (i.e. #F00) is used in the bodies of the .red-text and .green-text selectors, the tester complains:
The hex code for the color red should be used instead of the word red.
The hex code for the color green should be used instead of the word green.

Don't penalize learners for being smart!

I agree. The tests are here. The regex for the second and fourth tests should be changed to allow for either version of the hex code.

If there is an agreement about allowing the triplet hex for red, green and blue, I will like to take it

Looks like someone already jumped on it @YemiAdebayo. Want to take a look at that PR and see if it looks good?


make Vradar optional for Menu and Vlims optional in Display

Issue #17
Turn some V variable into optional arguments, those are initialized internaly.
In execute_two.py I accessed Vradar using MainMenu.Vradar instead creating function  Main.Menu._getVradar(). I don't think we need a get function here, do you agree?

Just ran the code and maybe I failed to notice this before but self.fieldnames is not defined anywhere so there is a error in _check_default_field in plot.py.  If you are going to make more changes soon would you stick this in the init?  Otherwise, I'll do at a later date.  The code still functions as is so I consider this minor for the time being.

I will add that to my next PR, some time later today

Thanks!

I'm not getting this error, self.fieldnames is being defined in self.NewRadar() that should be executed priory to any call of self._check_default_field().
There is another error "  'Display' object has no attribute 'ax'  " that was caused by this PR, however I already have a solution using issue #19.
Anyway, the code is working, I will keep an eye open to see if this turn back up and how to make  Display Start-up less vulnerable to such things.

Hmm weird.  I still get the above stated error, but I can't reproduce the 'ax' error.
Will have to keep an eye on this.


Feature request: Auto-dispose family provider that uses a type as key with its return type being parametrised by the key-type

So far, we can have auto-dispose family providers with a fixed return type.
But, let's look at the following example:
class Car {}

class BMW extends Car {}

class Audi extends Car {}

class CarRepository {
  final all = StateProvider<List<Car>>((ref) => [BMW(), Audi(), BMW()]);

  // get all cars of a specific type
  AutoDisposeProvider<List<T>> byType<T>() => ???
}

final carRepository = CarRepository();

void build(BuildContext context) {
  final bmws = useProvider(carRepository.byType<BMW>());
}

I was thinking how to implement byType in a generic way and came up with the following (imperfect) solution:
///
/// Equality is implemented to check the passed `keyType`.
///
class _AutoDisposeProviderTypeFamilyKey {
  _AutoDisposeProviderTypeFamilyKey(this.keyType, this.create);

  final Type keyType;
  final dynamic Function(ProviderReference ref) create;

  @override
  int get hashCode => keyType.hashCode;

  @override
  bool operator ==(Object other) => other is _AutoDisposeProviderTypeFamilyKey && other.keyType == keyType;
}

Type _typeOf<T>() => T;

///
/// A family of `AutoDisposeProvider`s that are keyed by their type.
///
class AutoDisposeProviderTypeFamily {
  AutoDisposeProviderTypeFamily();

  /// manages the actual objects, and delegates to `create` in `_AutoDisposeProviderTypeFamilyKey`
  final _map = Provider.autoDispose.family<dynamic, _AutoDisposeProviderTypeFamilyKey>((ref, key) => key.create(ref));

  ///
  /// Lookup the value by the given `Key` type. `create` is used to construct the `value` if the key did not exist so far.
  ///
  AutoDisposeProvider<Value> call<Key, Value>(Value Function(ProviderReference ref) create) {
    // we are getting the actual object from `_map` but need to wrap it into another AutoDisposeProvider to apply the cast
    return Provider.autoDispose<Value>(
      (ref) => ref.watch<dynamic>(_map(_AutoDisposeProviderTypeFamilyKey(_typeOf<Key>(), create))) as Value,
    );
  }
}

Now, we can finish CarRepository:
class CarRepository {
  final all = StateProvider<List<Car>>((ref) => [BMW(), Audi(), BMW()]);

  final _byType = AutoDisposeProviderTypeFamily();

  // get all cars of a specific type
  AutoDisposeProvider<List<T>> byType<T>() => _byType<T, List<T>>((ref) => ref.watch(all).state.whereType<T>().toList());
}

@rrousselGit, probably you can give some advice how to do it better...

That's a hard one.
I don't think there's a clean way to handle generic providers, because of one problem: generic constraints.
If we want to support generics, it is a given that we'd want to support T extends Something. But that's hardly doable in a reusable way
Could you expand on why you need a generic provider?
Why not do:
abstract class Cars {
  static final all = StateProvider<List<Car>>((ref) => [BMW(), Audi(), BMW()]);

  static final bmws = Provider.autoDispose<List<BMW>>((ref) => ref.watch(all).whereType<BMW>());
  static final audis = Provider.autoDispose<List<Audi>>((ref) => ref.watch(all).whereType<Audi>());
}

void build(BuildContext context) {
  final bmws = useProvider(Cars.bmws);
}

After all, the list of Car subclasses is known. And the declaration of the provider is a one-liner.

Actually, the reason is that I have a potentially big union and don't want to provide variables for all of them individually.

But you don't really have the choice. You could otherwise easily have a memory leak or unperformant code.
Why do you need a byType to begin with? Maybe the entire idea of filtering the list by type is incorrect.
For instead why do:
final bmws = watch(Cars.byType<BMW>());

when you can do:
final bmws = watch(Cars.all).whereType<BMW>();

That achieves strictly the same thing.

Actually, I thought leveraging the family gives me caching for free...

Oh that's a fair point. But there's hardly a better solution.
Maybe use hooks and do:
final cars = watch(Cars.all);
final bmws = useMemoized(() => cars.whereType<BMW>(), [cars]);

Although I still believe the original Cars.bmw is the way to go. It's not perfect, but that's still better than hacking a watch(byType<T>()) which is likely to have numerous issues.

This is what I would do:
 class Cars {
  static final all = StateProvider<List<Car>>((ref) => [BMW(), Audi(), BMW()]);
  static final singleton = Cars();
  final cars = Provider.autoDispose.family<List<Car>, Type>((ref, type) => ref.watch(all).state.where((c) => type.isSubtype(c)).toList());

}

extension Subtype on Object {
  bool isSubtype<T>(T type){
    return this is T;
  }
}

extension CarTypeReader on Reader {
  List<T> readCarType<T>(){
    return this(Cars.singleton.cars(T)).cast<T>();
  }
}
List<T> useCarType<T>() {
  return useProvider(Cars.singleton.cars(T)).cast<T>();
}

void build(BuildContext context, ScopedReader read) {
  final bmws = read.readCarType<BMW>();
  final audi = useCarType<Audi>();
}

The fact that you need a "cast" in the build method mostly defeats the purpose imo
The point of extracting it in a provider is to have the provider behave as a cache.
But that "cast" means we're creating a new list whenever the widget rebuild

Could you cast the list using as List<T> do you think?
This is a case where the static metaprogramming language proposal would be helpful.

Could you cast the list using as List<T> do you think?

It won't because the list instance is a List<Car>, not List<BMW>

My solution in the initial post was exactly to overcome the limitations of Dart by delegating the construction to the call site where I have access to the type.
To implement without a fancy delegate, we would require second order generics like:
typedef Create = Value<Key> Function<Key, Value<Key>>(Key key);
Then you could have something like Create<T, List<T>> and resolve the delegate and move the code directly into the family-constructor.

So you could create your own 'family' by caching it yourself:
Then you would have the type when the provider is being created.
Not as easy as using a family, but more what you want it seems.

abstract class Cars {
  static final all = StateProvider<List<Car>>((ref) => [BMW(), Audi(), BMW()]);
  static final Map<Type, dynamic> _cache = {};
  static AutoDisposeProvider<List<T>> cars<T>() {
    if (!_cache.containsKey(T)) {
      _cache[T] = Provider.autoDispose<List<T>>((ref) {
        final allCars = ref.watch(all).state;
        ref.onDispose(() {
          _cache.remove(T);
        });
        return allCars.whereType<T>().toList();
      });
    }
    return _cache[T] as AutoDisposeProvider<List<T>>;
  }
}

void build(BuildContext context) {
  final bmws = useProvider(Cars.cars<BMW>());
}

Actually, I wanted an abstraction of what I did in the initial post, potentially to be included in the riverpod package.

It's a lot of work for something fairly niche and with a reasonable workaround.
Support in riverpod means support for all the combinations, so autoDispose vs non-autoDispose and all the providers.
Unless that's something a large number of people desires, I won't work on this.

Closing...


Add class CroppedBitmap

What does the pull request do?
Added class CroppedBitmap which allows to render a part of Source in Image Control
How was the solution implemented (if it's not obvious)?
I added some changes to ImagePage of ControlCatalog to demonstrate CroppedBitmap usage

I've glanced over WPF documentation to understand this class and I think this needs more work to be useful.
In examples this class can be used directly from XAML like:
<CroppedBitmap x:Key="croppedImage"
      Source="{StaticResource masterImage}" SourceRect="30 20 105 50"/>

This is currently not possible in this implementation and it would limit usability.
Docs: https://docs.microsoft.com/en-us/dotnet/framework/wpf/controls/how-to-crop-an-image
WPF implementation: https://github.com/dotnet/wpf/blob/master/src/Microsoft.DotNet.Wpf/src/PresentationCore/System/Windows/Media/Imaging/CroppedBitmap.cs

I tried to realize this functionality and  I have one problem. If I want to use CroppedBitmap directly in XAML I need to inherit it from AvaloniaObject and add setters for Source and SourceRect properties. In this case this class will cease to be immutable. I haven't find any way to notificate the parent Image Control to render when changes occurs in it Source. Can anybody help me with it?

@vmelnikov You need to implement IAffectsRender on your IImage implementation.

@vmelnikov You need to implement IAffectsRender on your IImage implementation.

Thanks it works

Can save croppedBitmap as a jpg?

@nir34 please open a Q&A or ask on telegram https://t.me/Avalonia if you have a question. Closed issues are not a good place to ask. I think jpg is not supported atm, but png is which is even better in most cases. Try MyBitmap.Save()


[SPARK-43624][FOLLOWUP][PS][CONNECT] Moving  into Spark functions

What changes were proposed in this pull request?
This is follow-up for https://github.com/apache/spark/pull/41660 to improve the reusability of functions.
Why are the changes needed?
To improve the reusability of ewm.
Does this PR introduce any user-facing change?
SF.ewm is added, but it's mostly purposed to internal usage.
How was this patch tested?
It's just rearrangement, so the existing CI should pass.

cc @zhengruifeng FYI

The test fails FYI:
File "/__w/spark/spark/python/pyspark/pandas/window.py", line 2529, in pyspark.pandas.window.ExponentialMoving.mean
Failed example:
    df.ewm(com=0.1).mean()
Exception raised:
    Traceback (most recent call last):
      File "/usr/lib/python3.9/doctest.py", line 1336, in __run
        exec(compile(example.source, filename, "single",
      File "<doctest pyspark.pandas.window.ExponentialMoving.mean[1]>", line 1, in <module>
        df.ewm(com=0.1).mean()
      File "/__w/spark/spark/python/pyspark/pandas/window.py", line 2549, in mean
        return super().mean()
      File "/__w/spark/spark/python/pyspark/pandas/window.py", line 2457, in mean
        return self._apply_as_series_or_frame(mean)
      File "/__w/spark/spark/python/pyspark/pandas/window.py", line 210, in _apply_as_series_or_frame
        return self._psdf_or_psser._apply_series_op(
      File "/__w/spark/spark/python/pyspark/pandas/frame.py", line 881, in _apply_series_op
        applied.append(op(self._psser_for(label)))
      File "/__w/spark/spark/python/pyspark/pandas/window.py", line 211, in <lambda>
        lambda psser: psser._with_new_scol(func(psser.spark.column)),  # TODO: dtype?
      File "/__w/spark/spark/python/pyspark/pandas/window.py", line 2450, in mean
        col_ewm = SF.ewm(scol, unified_alpha, self._ignore_na)
      File "/__w/spark/spark/python/pyspark/pandas/spark/functions.py", line 159, in ewm
        return Column(sc._jvm.PythonSQLUtils.pandasCovar(col._jc, alpha, ignore_na))
      File "/__w/spark/spark/python/lib/py4j-<IP_ADDRESS>-src.zip/py4j/java_gateway.py", line 1322, in __call__
        return_value = get_return_value(
      File "/__w/spark/spark/python/pyspark/errors/exceptions/captured.py", line 179, in deco
        return f(*a, **kw)
      File "/__w/spark/spark/python/lib/py4j-<IP_ADDRESS>-src.zip/py4j/protocol.py", line 330, in get_return_value
        raise Py4JError(
    py4j.protocol.Py4JError: An error occurred while calling z:org.apache.spark.sql.api.python.PythonSQLUtils.pandasCovar. Trace:
    py4j.Py4JException: Method pandasCovar([class org.apache.spark.sql.Column, class java.lang.Double, class java.lang.Boolean]) does not exist
    	at py4j.reflection.ReflectionEngine.getMethod(ReflectionEngine.java:321)
    	at py4j.reflection.ReflectionEngine.getMethod(ReflectionEngine.java:342)
    	at py4j.Gateway.invoke(Gateway.java:276)
    	at py4j.commands.AbstractCommand.invokeMethod(AbstractCommand.java:132)
    	at py4j.commands.CallCommand.execute(CallCommand.java:79)
    	at py4j.ClientServerConnection.waitForCommands(ClientServerConnection.java:182)
    	at py4j.ClientServerConnection.run(ClientServerConnection.java:106)
    	at java.lang.Thread.run(Thread.java:750)

merged to master


Create request body pipe reader by decorating the connection pipe reader

This should be more efficient than pumping request body data into a new pipe.

Let's punt this away.

@halter73 Assigning both of us for now. Based on costing, we can decide who actually does this later.


Support netcoreapp3.1

Hi, I've been using this lib in a project and a recent updated dropped support for netcoreapp3.1 and moved to net5.0. So firstly that's a breaking change without a semver major version bump. However, if netcoreapp3.1 were to be added in conjunction with net5.0 (which shouldn't be a problem since the project has no dependencies) it would not be a breaking change.
If you're not interested in supporting runtimes/frameworks that you don't use... that's cool I'll just ship a fork. 👍🏻

my mistake @damianh, I'll look into it tomorrow! sometimes it's easy to overlook semver in a big project

my mistake @damianh, I'll look into it tomorrow! sometimes it's easy to overlook semver in a big project

Cool. Some nice libs/code here :)

Cool. Some nice libs/code here :)

Should be fixed this evening :) after we bump everything to 5.0.2

@damianh can you try 2.2.11?

Works great, cheers @CumpsD !


chore: add sepolia testnet

Adds Sepolia testnet.  Tested with CLI, works fine.

Looks good! Thanks :heart:


Google browser Clone

I am Anurag Mukherjee, a participant of GSSoC'21.
Aim
The aim of the script is to make a google browser clone with the help of PyQt5 module in python.
Details
The browser will contain all the basic functionalities that a browser should have. It has a home button which takes the browser directly to the home page, a back button, which takes to the previous URL, a forward button, and a reload button which refreshes the whole web page. Apart from these it also has a editable URL area, where we can directly write the URL of our respected website.
Modules and packages will be used:-

PyQt5
sys

A sample image of the browser is shown below:-

We already have an implementation for this: https://github.com/avinashkranjan/Amazing-Python-Scripts/tree/master/Broswer


LocalAI model selection

Being integrated int the https://github.com/enricoros/big-agi/compare/main...feaure-model-sources branch. Steadily. Not ready yet.

This request has been merged. The code and support has trickled in over time. This PR does not apply anymore, but the support is now on mainline.
Example: config:

Example: output:


[stable/metabase] Add tolerations and affinity

Signed-off-by: Amit Oren<EMAIL_ADDRESS>What this PR does / why we need it:
Add tolerations and affinity in addition to the node selector already present
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @paulczar

davidkarlsen

/assign @davidkarlsen

/ok-to-test

@davidkarlsen ping


Wireless Controller won’t work on Steam Deck

The controller will connect just fine to the steam deck itself, but once in minecraft it doesn’t work. I use the prism launcher, idk if that has anything to do with it.

The controller is likely just missing a mapping, look at the new section in the README to find out how to fix it.


Flow control

It would be very nice to see some form of flow control for the air. Even the simple ability to make the climate control unit separate the networks (so it does not act like anormal pipe and merges them) and to select the input/output network for air would be amazing.
This would allow users to build a climate control network with one intake fans array and have several separate temperatures in controlled output networks.

I see your point. I thought of that a while back that we should be able to Share Intake Fans across the climate control units. I guess it will need some big changes that might affect people's existing saves. I will need to see more on this on how I can include this!
Thanks for the suggestion. I will let you know if it's feasible to include this or not.


Check functionality #28, download and unzip

https://github.com/USGS-R/mda.streams/pull/28

A little confused as to why the ... isn't passed to item_file_download in both cases:
#set the intermediate (staging) destination for downloaded gzip file
  if (isGzipped(file_list$url)){
    stage_file <- tempfile(fileext = paste0(get_ts_extension(), '.gz'))
    stage_file = item_file_download(item$id, file_list$fname, stage_file, ...)
    out_destination = gunzip(stage_file, destname = destination,
                             temporary = FALSE, skip = FALSE, overwrite = FALSE, remove = TRUE, BFR.SIZE = 1e+07)
  } else {
    out_destination = item_file_download(item$id, file_list$fname, destination)

  }

Probably no good reason. Fix it.

Seems like it should be passed the second time, but not the first. Primary case:
#'@param ... additional arguments passed to \code{\link[sbtools]{item_file_download}},
#'for example \code{overwrite_file}


Update dev deploy instructions

Should we also bring that as the standard way of doing DEV in template-service-flask?
Looks good to me, but maybe you see some caveats I don't


Iss 311

closes #311

I confirm that the errors described in the issue are corrected.
All changes in the files appear directly related to the issue description.
I confirm that all tests run successfully on my machine.
Please note that Github has identified two warnings in
frontend/src/ui/Results/Results.stories.tsx
(1) 'Provider' is defined but never used
(2) 'createStore' is defined but never used
I see that both are imported.  The code that uses them has been commented out,
therefore I suggest removing the two import statements, and all code that uses them.
(I saw no issues when I did that and tested it on my local machine)

Thanks for the review @LewisStaples ; merging this code in now.


Avoid esp32-s3 compile error

Related: #227
No guarantee that everyone‘s ESP32-S3 boards will work, but it worked for my ESP32-S3 board.It will not affect the ESP32 boards, because the new added content has a judgment on the chip type by like blow code:
#if CONFIG_IDF_TARGET_ESP32
// old...
#elif CONFIG_IDF_TARGET_ESP32S3
// new added...
#endif

But the this added may have some impact:Link. I do some simple test it no impact with my device(both esp32 and esp32-s3 device) and simple demos.

Hi @imliubo!
Thank you for this contribution!
Which display driver do you use with esp32-s3?

It will not affect the ESP32 boards, because the new added content has a judgment on the chip type by like blow code

ESP32 compiles with your changes, but does not run correctly.
For example, esp_clk_cpu_freq function is missing from the espidf module, but is used by ili9XXX.py.
That's because you filter clk.h (and other important headers such as gpio.h)
What was your motivation for removing these headers from the espidf module?

I do some simple test it no impact with my device(both esp32 and esp32-s3 device) and simple demos.

If you complete the tasks in https://github.com/lvgl/lv_binding_micropython/issues/227 you will be eligable to get paid for your contribution!

Question for @kisvegabor: https://github.com/lvgl/lv_binding_micropython/issues/227 offers 100 USD for adding support for ESP32-S2, ESP32-C3, ESP32-S3.
What does LVGL offer for fully implementing only one of these flavours (esp32-s3) including testing, videos etc.?
I would recommend paying the total sum of 100$ for fully implementing even one of these ESP flavours (but it's your decision of course).

Which display driver do you use with esp32-s3?
I use a C code driver(not ili94341.py).

ESP32 compiles with your changes, but does not run correctly.
For example, esp_clk_cpu_freq function is missing from the espidf module, but is used by ili9XXX.py.
That's because you filter clk.h (and other important headers such as gpio.h)
What was your motivation for removing these headers from the espidf module?

emmm... I use my own display driver,so I am not test the ili9341.py driver.I remove those headers beacuse it has some problem with ESP32-S3 compile, looks need figrue out why the error happen instead of just remove these files :), late I will try again.

If you complete the tasks in https://github.com/lvgl/lv_binding_micropython/issues/227 you will be eligable to get paid for your contribution!

Actually this also was my work, so no need any paid. But thanks for this awesome project. I would be happy to contribute some very little help for this.
Anyway, I will try fix these mistakes late.
Have a nice day :)

Question for @kisvegabor: https://github.com/lvgl/lv_binding_micropython/issues/227 offers 100 USD for adding support for ESP32-S2, ESP32-C3, ESP32-S3.
What does LVGL offer for fully implementing only one of these flavours (esp32-s3) including testing, videos etc.?
I would recommend paying the total sum of 100$ for fully implementing even one of these ESP flavours (but it's your decision of course).

Actually this also was my work, so no need any paid. But thanks for this awesome project. I would be happy to contribute some very little help for this.

I know it's not a life changer amount of money, but that's what we can offer now and I'd really like to give it to people who work on making LVGL better. We are a little bit tight on budget  but 70 USD for the S3 part still work.

A liitle process for esp32s3.
Simple test with example3 and use ili9XXX.py driver, but esp32s3 has some problem.
video
Left is esp32, right is esp32s3.

It seems for some reason you DMA can't send out all the bytes in the flush_cb. Maybe the DMA has limit for number of bytes in one transfer?

It seems for some reason you DMA can't send out all the bytes in the flush_cb. Maybe the DMA has limit for the number of bytes in one transfer?

@kisvegabor  Yeah, looks had some limit, but I am not deep in check this,it can be work by reducing display buf_size (set factor=8).
Test result with this example:

esp32 with ili9xxx.py driver OK.
esp32s3 with ili9xxx.py driver OK(need reduce display buf_size for now).

video
It need more test with different lcds, and still has some issuse with esp-idf module, because some API is not support esp32s3 for now,see related issues:
IDFGH-8771 and IDFGH-8773, To fix this, I added 3 functions (temporary solution).
IDFGH-8772, to fix this, I add lldesc.h to FILTER, because i think this funciton is not use?If I am wrong please told me :)
All test is build with esp-idf v4.4.2-1b16ef6cfc2479a08136782f9dc57effefa86f66
@amirgon has any suggest?
Have a nice day.

emmm... I use my own display driver,so I am not test the ili9341.py driver

Could you tell us more about your display driver?
Also, note that there are two different ili9341 drivers: one that uses DMA driver/esp32/ili9XXX.py and a slower pure Python one driver/generic/ili9xxx.py that is not ESP32-specific.
Which one are you trying? Or both?

Actually this also was my work, so no need any paid. But thanks for this awesome project. I would be happy to contribute some very little help for this.

Thanks!!

and still has some issuse with esp-idf module, because some API is not support esp32s3 for now,see related issues:

It's ok to filter out some espidf headers that are not in use.
Many headers are included unintentionally (because headers we want include other headers we don't want), so as long as  the APIs we want are not excluded, it's ok to filter out (and in fact helps because it reduces program size)
These are the headers that we want and that are included explicitly:
https://github.com/lvgl/lv_binding_micropython/blob/c544c51c97b38907709a0cd7ea7ca8ab96410cbb/driver/esp32/espidf.h#L117-L130
So I think you can filter out other headers such as rtc_io.h, lldesc.h, esp_eth.h, etc. for all ESP flavors.
If a header is required but specific functions are missing (For example, we need gpio.h, but gpio_input_get_high and gpio_output_set_high are missing) then I think it's ok to provide an empty implementation that invokes an error as long as these functions are not in use today in our code (at least until the issues your opened on esp-idf are resolved).

Could you tell us more about your display driver?

Yeah, it kind like TFT_eSPI, when not use LVGL, it still has many draw functions can be use.

Also, note that there are two different ili9341 drivers: one that uses DMA driver/esp32/ili9XXX.py and a slower pure Python one driver/generic/ili9xxx.py that is not ESP32-specific.
Which one are you trying? Or both?

This one driver/esp32/ili9XXX.py, not driver/generic/ili9xxx.py.

If a header is required but specific functions are missing (For example, we need gpio.h, but gpio_input_get_high and gpio_output_set_high are missing) then I think it's ok to provide an empty implementation that invokes an error as long as these functions are not in use today in our code (at least until the issues your opened on esp-idf are resolved).

Yes,need espressif resolved this.

@imliubo Is this ready from your side or do you plan to make more changes?
I think mkrules.cmake can be simplified by filtering some headers from all ESP flavours instead of specifically from esp32s3.

I only have ESP32 and ESP32-S3 for now

I think that even if your PR covers only ESP32 and ESP32-S3, it's good enough for now.
We can always add C3 and S2 later.
So basically what's missing is simplifying mkrules.cmake a little, and testing.

LGTM! @imliubo This patch is all I need for my S3 board.


Documentation

Following Documentation Milestone.

@Rockster34 @caio1478 @CineXMike Will do the Documentation

@Rockster34 Will you do this?

Yes i'll do that

@Rockster34 Any progress?

Documentation is Paused. Since we're rewriting Core.


agm-marker not showing

I'm trying to implement angular map on my app, but nothing seems to work.
In my html file I added the following code:
<div class="TestMap">
	<agm-map [latitude]="lat" [longitude]="lng">
  <agm-marker [latitude]="lat" [longitude]="lng"></agm-marker>
</agm-map>
</div>

On my typescript component file I added:
import { Component, OnInit } from '@angular/core';
import { AgmCoreModule } from '@agm/core';
import { AgmMap } from '@agm/core';

export class LuminaireMapMonitorComponent implements OnInit {

	title: string = 'My first AGM project';
	lat: number = 51.678418;
	lng: number = 7.809007;

  constructor() { }
  ngOnInit() {
  }
}

And on my css I added:
.TestMap {
  height: 300px;
}

And nothing works.

you have to place the css class at the map element

It still doesn't work
I changed my html to
<agm-map class="agmMap"  [latitude]="lat" [longitude]="lng">
  <agm-marker [latitude]="lat" [longitude]="lng"></agm-marker>
</agm-map>

and the css to
.agmMap {
  height: 300px;
}

What am I doing wrong?

Same issue

Try changing css from

> .agmMap {
>   height: 300px;
> }

to:
agm-map {
  height: 300px;
}

You need to add css width to the agm-map element. I used the attribute directive fxFlex on mine.


Proposal for a logo

The idea is to take the parquet logo and put it into a globe.

That looks nice!
Two comments:

Can you add the actual file to the repo? (now it's only included through a link to githubusercontent in the readme)
We probably need to check with the Apache Parquet project that such a derivative logo is not a problem

Very good points Joris. Have added the files. Not sure if I can link to the file on the README?
Anybody has a connection the Apache Parquet project that we can ask about this? Would be good also to get their opinion on the initiaitve!
George Percivall is an OGC member I know is part of the Apache somehow, but anybody else has a contact to ask if this is ok for them?

Apache itself has a trademark policy (https://www.apache.org/foundation/marks/), but I don't know if such a derivative logo is discussed in there. Parquet also has a dev mailing list that could be used to reach out. And cc @shangxinli who is the current PMC chair of Apache Parquet.

Not sure if I can link to the file on the README?

Yes, that should be possible, like ![](relative/path/file.png)

If design support is needed for this, OGC has a graphics team on retainer that is able to support, especially since there is a solid scope of work laid out already. They created our APIs and Rebrand materials and are hungry to get more involved in the dev space.

Oh good to know! I am waiting for an answer from the Parquet team and
depending on that we decide.
Thanks
On Fri, Mar 25, 2022 at 4:13 PM Jfath1508 @.***> wrote:

If design support is needed for this, OGC has a graphics team on retainer
that is able to support, especially since there is a solid scope of work
laid out already. They created our APIs and Rebrand materials and are
hungry to get more involved in the dev space.
—
Reply to this email directly, view it on GitHub
https://github.com/opengeospatial/geoparquet/pull/48#issuecomment-1079126732,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA7GOY3RUHG2FNV55DAZFDVBXJZFANCNFSM5RRTE76A
.
You are receiving this because you authored the thread.Message ID:
@.***>

No need. On it already.
On Fri, Mar 25, 2022 at 5:18 PM Chris Holmes @.***>
wrote:

Anybody has a connection the Apache Parquet project that we can ask about
this? Would be good also to get their opinion on the initiaitve!
There's some good connections at Planet, I'll follow up with them.
—
Reply to this email directly, view it on GitHub
https://github.com/opengeospatial/geoparquet/pull/48#issuecomment-1079185822,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA7GO7IMNPEUTBP4KTC7QDVBXROZANCNFSM5RRTE76A
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Javier de la Torre
(US) +1<PHONE_NUMBER> | (ES) +34 911 165 823 | @.***
https://spatial-data-science-conference.com/2022/london/index.html#signup

Like to help, but generally, the Parquet community doesn't allow the Parquet logo to be used that way. Sorry about that!

So as per discussion separately with the Apache Foundation, they will prefer if we do not use their logo as a base for the initiative. So lets cancel this PR and @Jfath1508 would be great if we could get some support for a logo from OGC yes.

@Jfath1508 any update on this? Thank you so much!

Closing this, recent consensus was we shouldn't aim for an independent logo as we hope that Parquet eventually just has our geo stuff as a standard and people could just use the parquet logo.


Remove get-earn-protocol-* lambdas

Remove get-earn-protocol-* lambdas

Summary by CodeRabbit

Chores

Removed the addEarnProtocolConfig function and its references from the project, indicating the Earn Protocol configuration is no longer part of the stack setup.
Deleted multiple configuration files including package.json, Dockerfile, and various TypeScript and ESLint configurations, which may impact the build and deployment processes.
Removed GraphQL queries related to fleet and portfolio details, which may affect data retrieval capabilities for users interacting with the Earn Protocol.

@coderabbitai pause

Added Added GetVaultsRaw, GetVaultRaw methods to the sdk

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

Please upload report for BASE (dev@c362513). Learn more about missing BASE report.
Report is 1 commits behind head on dev.

Files with missing lines
Patch %
Lines

...entation/ArmadaManager/ArmadaManagerUsersClient.ts
0.00%
2 Missing :warning:

Additional details and impacted files
@@          Coverage Diff           @@
##             dev     #545   +/-   ##
======================================
  Coverage       ?   70.76%
======================================
  Files          ?       25
  Lines          ?      260
  Branches       ?       19
======================================
  Hits           ?      184
  Misses         ?       75
  Partials       ?        1

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

Added usage examples and tests:
it('should getVaults', async () => {
  const raw = await sdk.armada.users.getVaultsRaw({
    chainInfo,
  })
  expect(raw.vaults).toHaveLength(1)
  expect(raw.vaults[0]?.id).toEqual(`${fleetAddress.value}`)
})

it('should getVault', async () => {
  const raw = await sdk.armada.users.getVaultRaw({
    poolId: ArmadaVaultId.createFrom({
      chainInfo,
      fleetAddress,
    }),
  })
expect(raw.vault?.id).toEqual(`${fleetAddress.value}`)


Allow decoding resource-only APKs

Fixes #1517 (with test)
Allow decoding APKs with only resources, i.e., with no Java classes.
Basic test APK created with https://github.com/therealchjones/blank-android and copyright-free

@therealchjones great! Thank you!


As a user, I want the option of zooming in on the timetree on desktop and with buttons, so that the content and functionality is accessible in a variety of modes.

Add buttons and function to zoom in and out of the tree on desktop and mobile to address WCAG requirements  (can't rely on gesture/scroll wheel)
Add zoom capability on desktop for accessibility (low vision)

In the following Figma links you can find the +/- button styles and states on mobile and desktop along with notes on interaction:

Default view on mobile and desktop,
selected leaf on mobile,
and interaction with zoom buttons on desktop

@gissoo question about the desktop zoom, which of these do you prefer?
When the tree is zoomed, allow leaves to display behind the info panel:

Cut the tree off at the edge of the container (overflow is hidden), no leaves behind the panel:

Rebecca and I viewed option 1 in max zoom and it looks great! But we revised the height of the zoom and full screen buttons on desktop to match the height of the panel so they don't look out of place

completed in #218


Creating a citable bibliography for VSCode Jupyter or make this repo citable

Describe the feature you'd like.
This is not really a feature. I have benefited a lot from VSCode Jupyter in my research and wanted to add the citation in the acknowledgement. However, I could not find a bibliography entry for VSCode Jupyter extension to put in acknowledgement and bibliography.
Possible solutions
Would you please consider make this repo citable? https://guides.github.com/activities/citable-code/
or even better a comprehensive bibtex entry like:
@Article{harris2020array,
 title         = {Array programming with {NumPy}},
 author        = {Charles R. Harris and K. Jarrod Millman and St{\'{e}}fan J.
                 van der Walt and Ralf Gommers and Pauli Virtanen and David
                 Cournapeau and Eric Wieser and Julian Taylor and Sebastian
                 Berg and Nathaniel J. Smith and Robert Kern and Matti Picus
                 and Stephan Hoyer and Marten H. van Kerkwijk and Matthew
                 Brett and Allan Haldane and Jaime Fern{\'{a}}ndez del
                 R{\'{i}}o and Mark Wiebe and Pearu Peterson and Pierre
                 G{\'{e}}rard-Marchant and Kevin Sheppard and Tyler Reddy and
                 Warren Weckesser and Hameer Abbasi and Christoph Gohlke and
                 Travis E. Oliphant},
 year          = {2020},
 month         = sep,
 journal       = {Nature},
 volume        = {585},
 number        = {7825},
 pages         = {357--362},
 doi           = {10.1038/s41586-020-2649-2},
 publisher     = {Springer Science and Business Media {LLC}},
 url           = {https://doi.org/10.1038/s41586-020-2649-2}
}

AlternativeS
Or a simple bib entry like

T. A. Driscoll, N. Hale, and L. N. Trefethen, editors, Chebfun Guide, Pafnuty Publications, Oxford, 2014.

Since this is a public repo, you can reference the repo by URL.


Prebuilt Removal: Investigate Microsoft.CodeAnalysis,2.8.2 in arcade

Arcade defines a static version package reference for Microsoft.CodeAnalysis,2.8.2 here:
https://github.com/dotnet/arcade/blob/14abaee3dba41fbe608431fb6a4f4b2435dcac33/src/Microsoft.DotNet.Build.Tasks.Packaging/src/Microsoft.DotNet.Build.Tasks.Packaging.csproj#L60-L61
This is causing multiple indirect prebuilts.  There is a comment which states:
Need to reference version 2.8.2 of CodeAnalysis so that we don't have downgrade issues with System.Reflection.Metadata version
This should be investigated to see if source-build can use a source-built version for this package.

/cc @markwilkie @chcosta

@joperezr , can you take a look?

That reference should be now removed from the Packaging task. We removed the task that depended on it and it seems now that we forgot to also remove the extra package dependency.

@joperezr - Thanks, that's great to hear!


Missing fluent/fluentd-kubernetes-daemonset:v1.16.3-debian-s3-amd64-1.0 in Docker Hub

The image fluent/fluentd-kubernetes-daemonset:v1.16.3-debian-s3-amd64-1.0 is missing in Docker Hub https://hub.docker.com/r/fluent/fluentd-kubernetes-daemonset, please help build and upload it.

related with #1455

If I remember right, I published it manually in the previous release (v1.16.2).
It's not documented yet in MAINTINING.md because it's not ideal procedure.
We should replace Docker Hub's build automation with GitHub Actions.

Same here, I could not find this image, but I can pull 1.16.2


soroban-rpc: limit the execution duration of the jrpc requests

What
Limit the execution duration of the jrpc requests.
Why
Limiting the execution duration of the jrpc requests allow us to ensure reasonable performance characteristics and provide means for the hosting providers to plan for a desired SLA goals.
Known limitations
The implementation would still require the underlying handlers to respect the passed-in context, and abort the processing as soon as possible. This is not covered in this PR, and will be handled in a subsequent PR.
Additional notes

The unit tests for the backlog request limiter were extensively enhanced
The limiters configuration parameters were restricted to the TOML configuration file to avoid cluttering the environment variable and the command line interface.

@sreuland note that from configuration perspective, it's really important for us to have good and independent control of each of the endpoints. When monitoring our partner's usage, we would be able to tune individual endpoint without any binary change.

I'm generally less concerned about the increased overhead here in maintenance : I don't expect to have very frequent change in the API, and following a "pattern" isn't overwhelmingly risky. I am concerned that we would run into a situation where we won't have the right "configuration" tools in place - in particular around server and security configurations.


Ran into exception while running validator: is_module_hosted_on_github

Ran into exception while running validator: is_module_hosted_on_github
Stacktrace: ValueError: Attempt to enable Github-specific features when module path does not belong to github
go-template is still under development, if you feel that the error is incorrect, you are encouraged to raise an issue at:
https://github.com/notsatan/go-template
ERROR: Stopping generation because pre_gen_project hook script didn't exit successfully
Hook script failed (exit status: 246)

Hi @hmthanh, given that you've closed the issue on Github I'll assume you're no longer running into this error and that go-template is working as expected!
If this is not the case (and you happened to close this issue by mistake), feel free to re-open this issue — note that I'll need the input parameters you used that lead to this issue.


WFLY-14989 Inside naming remove unnecessary unboxing to clean code

Issue: https://issues.redhat.com/browse/WFLY-14989

@ropalka as we have discussed can you please approve this PR

Thanks! @boris-unckel done the changes.

/retest


Support function calls that don't elide errors

Given three functions f, g, and h:
func f() (int, error) { return 1, nil }
func g(int) { }
func h(int, error) { }

We previously supported g(f()) but not h(f()) because
we greedily elided errors, causing the call of h to fail
because only one parameter was passed in.
Rewrite (*typecheck.Checker).exprPartialCall to support
this case by improving how we detect how many parameters
there are.
To do this we take a hint from go/types and introduce
the unpack function which abstracts the Go spec's
complexity around how many values an expression like f()
actually yields, since it depends on context.
As a nice side effect we end up supporting slightly more
of the Go spec, too.

I ended up rewriting the function to be more aligned with go/types, which makes it easier to follow the Go spec I think. Let me know if you think the changes are too large, though, and I can probably find a smaller delta that accomplishes the same thing.

Hi @eandre, I really appreciate your work on this. I know this is not easy. But I looked up go/types as you mentioned, and it looks like the code is copied from there. Right now neugram doesn't include any code not under the copyright of the authors (other than what's in the vendor directory) and to incorporate some would require sorting out attribution and the LICENSE file. I don't want to do that just yet.
I'm going to close this pull request to avoid any confusion. However you have lots of good clearly original code in here, and I do want to see this fixed. If you want to break out what's yours into a new pull request and either rewrite the code, or leave it as a TODO, please do.

Ah, duh :) I'll rewrite those parts and open a new pull request. My apologies, I just didn't think clearly!

No worries, and thank you for pull requests. Really great work.


stator_statorerror table needs cleanup job

query I ranSELECT
    table_name,
    pg_size_pretty(table_size) AS table_size,
    pg_size_pretty(indexes_size) AS indexes_size,
    pg_size_pretty(total_size) AS total_size
FROM (
    SELECT
        table_name,
        pg_table_size(table_name) AS table_size,
        pg_indexes_size(table_name) AS indexes_size,
        pg_total_relation_size(table_name) AS total_size
    FROM (
        SELECT ('"' || table_schema || '"."' || table_name || '"') AS table_name
        FROM information_schema.tables
    ) AS all_tables
    ORDER BY total_size DESC
) AS pretty_sizes;

source

   1   │                           table_name                          | table_size | indexes_size | total_size
   2   │ --------------------------------------------------------------+------------+--------------+------------
   3   │  "public"."stator_statorerror"                                | 824 MB     | 33 MB        | 857 MB
   4   │  "public"."users_identity"                                    | 12 MB      | 1816 kB      | 13 MB
   5   │  "public"."activities_post"                                   | 7336 kB    | 3568 kB      | 11 MB
   6   │  "public"."users_inboxmessage"                                | 10 MB      | 384 kB       | 10 MB
   7   │  "public"."activities_post_mentions"                          | 584 kB     | 1016 kB      | 1600 kB
   8   │  "pg_catalog"."pg_proc"                                       | 856 kB     | 336 kB       | 1192 kB
   9   │  "public"."activities_postattachment"                         | 800 kB     | 104 kB       | 904 kB
  10   │  "pg_catalog"."pg_attribute"                                  | 616 kB     | 256 kB       | 872 kB
  11   │  "public"."activities_fanout"                                 | 464 kB     | 312 kB       | 776 kB

I’ve been running Takahē on a DB “server” with 1GB of storage, and this table seems to have caused the disk to fill up, locking up the PostgreSQL process.
I’m not much of a DBA and don’t really want to change that… is this a bug? Or perhaps it reveals a missing feature, some kind of occasional cleanup transaction that would delete old records from this table?
(I’m generally running from tip of main, although sometimes with a few days lag. I haven’t run migrations in awhile and tbh it’s a little unclear to me how I’m supposed to know when I need to run them…)

Oh yes, we need something to trim this table periodically. You can just delete everything from it for now.

Mine ran up to 3.8 GB after a week 😂

Oh yes, we need something to trim this table periodically. You can just delete everything from it for now.

So something like TRUNCATE "public"."stator_statorerror"; ?

TRUNCATE quickly removes all rows from a set of tables. It has the same effect as an unqualified DELETE on each table, but since it does not actually scan the tables it is faster. Furthermore, it reclaims disk space immediately, rather than requiring a subsequent VACUUM operation. This is most useful on large tables.

https://www.postgresql.org/docs/current/sql-truncate.html

I use TRUNCATE TABLE public.stator_statorerror RESTART IDENTITY RESTRICT; for this.

I can understand you use RESTART IDENTITY, but why you use also RESTRICT ?
I read that the effect is:

Refuse to truncate if any of the tables have foreign-key references from tables that are not listed in the command. This is the default.

But in the code I can't see any foreign-key to the StatorError model:
https://github.com/jointakahe/takahe/blob/main/stator/models.py#L212

I just removed the table in favour of the runner logs and/or using something like Sentry; it was mostly for my debugging use during initial development!


⚠ Fortnite has degraded performance

In f346fed, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1590 ms

Resolved: Fortnite performance has improved in cac08d4.


Add paypal as a donate option

Pending bank transfer to complete final process.

Must get done ASAP.

Add to header.

@davidmolina is this still needed?

Rick, negative. Close issue.
Seperately issue, add additional donation options, ex. $5, $10, $20... $2000

On Jun 30, 2016, at 6:59 PM, Rick<EMAIL_ADDRESS>wrote:
@davidmolina is this still needed?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Breaks where in related query

Hi,
Recently I updated Adonis, Lucid, and soft-deletes to their latest versions, and now where query in preload is broken.
Following are the version updates:
@adonisjs/core               ^5.5.3  →   ^5.8.3
@adonisjs/lucid             ^17.1.0  →  ^18.0.0
adonis-lucid-soft-deletes    ^1.3.3  →   ^1.4.1
I get the "Cannot read properties of undefined (reading 'method')" error.
So I inspected to find the error and seems like the issue is caused by the toQuery() method on line 32 of the following file:
https://github.com/lookinlab/adonis-lucid-soft-deletes/blob/develop/src/SoftDeletes/index.ts
Replacing const isGroupLimitQuery = query.toQuery().includes('adonis_group_limit_counter') with
let sqlQuery = query.toSQL().toNative()
const isGroupLimitQuery = sqlQuery.sql.includes('adonis_group_limit_counter'); seem to fix the issue.
I can create a PR if that helps.
Thanks

Hi @atharahmed
Can you send callstack of error "Cannot read properties of undefined (reading 'method')"?

Please see attached image:

I'm trying to test the suggested fix a fork of the package.

I dont think what replace toQuerytotoSQLwill fix this issue. Both methods callapplyWhere()` where this error occurs
Need check why firstItem is undefined.

My suggested fix does not work but it seems like when you transform the lucid object to SQL query it adds an empty array in chainable of lucid which throws the error.
You can recreate the issue by replacing .preload('authors', (authors) => authors.groupLimit(2)) with .preload('authors', (authors) => authors.where('name', 'John')) in relations.spec.ts file line 226

Fixed in v1.4.2


code: update to give warning if not collecting logs

@ideepika shellcheck is now complaining. If you are using VSCode, I would recommend you install the recommended extensions to have shellcheck run locally as well.


Simple StackPanel widget as in WPF(C#) ?

please, i need simple sizing and positioning logic, like StackPanel widget in WPF.
current sizepos() logic is not so easy for begginers to figure out the look of the app. specially when not using layoutdesigner.
eventually i end up wasting more time on sizing and testing widgets in the window rather than actually developing the app.
you just put the widgets in the panel, simply set the width and height of the widget and it does the rest.

I made something similar to wxWidgets grid sizer out of Splitter Ctrl. You can specify the number of columns and the rows will be added depending on the number of added Ctrls. GridLayout can be nested as you wish for complex grid layouts.
https://github.com/M0untacir/GridLayout

@M0untacir Nice, thank you thank you thank you. that's what i needed.
you made it such it can be easily used as Vertical and Horizontal Panel. actually i didn't expect i'll see a solution this soon.
and a complete showcase. let me test it. i'll continue this thread in your repo
they should include this component built in U++ 👍🏼

What is your desired outcome? Buttons in a column? (That is what docs for StackPanel show as an example, but I bet it is more complicated than that). Mirek

@mirek-fidler actually the StackPanel logic is simple, but there is more than that.
it's a powerful container to organize components in a easy to understand way.

@M0untacir Nice, thank you thank you thank you. that's what i needed. initially i thought i can base it on ArrayCtrl, so it was SplitterCtrl. hmm it's orientation is currently vertical, but you made it such it can be easily used as Vertical and Horizontal Panel with a little modification. actually i didn't expect i'll see a solution this soon. you provided a complete showcase. let me test it. i'll continue this in it's own repo they should include this component built in U++ 👍🏼

The SplitterCtrl was the easiest route.
Thank you, glad i could help


Spelling and Grammar fixes

Just felt the need to add them

this is just for the README, fyi

oh thx epic


feat(pv): directly mount to container

Quick follow up to https://github.com/cdk8s-team/cdk8s-plus/pull/483 based on https://github.com/cdk8s-team/cdk8s/pull/889#discussion_r836943364.

💚 All backports created successfully

Status
Branch
Result

✅
k8s-20/main

Questions ?
Please refer to the Backport tool documentation


Support SHA512 for buildpack dependencies

Describe the Enhancement
Make it possible to define checksum = "sha512:..." in the buildpack.toml metadata.dependencies (instead of just {sha256,md5})
Possible Solution

Motivation
Some dependency providers (in our case Apache Spark collected from archive FTP) publish sha512 only, in such case it's not a trival task to use it in buidlpacks.

Transferred. I haven't looked at what it will take to implement this for libpak. My guess is that we'll do this as part of v2 (hopefully libcnb v2 is to be released soon, so then we can get working on libpack v2). That said, if someone wants to look into this and it can be done in a non-breaking way, we could add it to v1 as well.


Remove Unused Dependency: Colorama

Summary
Hello @Fizzadar,
I hope you're doing well! I've just opened this pull request that proposes the removal of the unused colorama dependency from the setup.py configuration file. It's part of an ongoing research endeavor focusing on the identification and elimination of code bloat within software projects. Your insights on this would be really valuable.
Rationale
The colorama library was integrated into the project via ea1410a, and was later utilised on
pyinfra_cli/__main__.py. However, its usage was discontinued in a subsequent commit, ca3ca9c, where it was removed in favor of click . Despite this change in the codebase, colorama  continued to be listed in the project's dependency files. Removing this unused dependency can reduce the overall footprint of the application, mitigate potential security risks, and simplify the dependency management process.
Changes

Removed the colorama  dependency from the setup.py file.

Impact

Reduced package size: Eliminating this unused dependency will decrease the overall size of the installed packages.

Simplified dependency tree: A leaner list of dependencies aids in more straightforward project maintenance and faster installations.

Hello @Fizzadar,
I hope this message finds you well. We are currently engaged in a research project focused on optimizing projects by identifying and eliminating unnecessary dependencies. Your insights and expertise as a maintainer would greatly contribute to our research efforts.
Could you please take a moment to review these changes? Your input would be highly valuable to us.
Thank you for your consideration!


ocenaudio: 3.11.1 -> 3.11.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/ocenaudio/versions.
meta.description for ocenaudio is: "Cross-platform, easy to use, fast and functional audio editor"
meta.homepage for ocenaudio is: "https://www.ocenaudio.com"
meta.changelog for ocenaudio is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/nix/store/rwbwq84smb1d3war2g561ijsqf9hfirb-ocenaudio-3.11.2/bin/.ocenvst-wrapped passed the binary check.
Warning: no invocation of /nix/store/rwbwq84smb1d3war2g561ijsqf9hfirb-ocenaudio-3.11.2/bin/.ocenaudio-wrapped had a zero exit code or showed the expected version
/nix/store/rwbwq84smb1d3war2g561ijsqf9hfirb-ocenaudio-3.11.2/bin/ocenvst passed the binary check.
/nix/store/rwbwq84smb1d3war2g561ijsqf9hfirb-ocenaudio-3.11.2/bin/ocenaudio_license.txt passed the binary check.
Warning: no invocation of /nix/store/rwbwq84smb1d3war2g561ijsqf9hfirb-ocenaudio-3.11.2/bin/ocenaudio had a zero exit code or showed the expected version
2 of 5 passed binary check by having a zero exit code.
1 of 5 passed binary check by having the new version present in output.
found 3.11.2 with grep in /nix/store/rwbwq84smb1d3war2g561ijsqf9hfirb-ocenaudio-3.11.2
found 3.11.2 in filename of file in /nix/store/rwbwq84smb1d3war2g561ijsqf9hfirb-ocenaudio-3.11.2

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

3 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
ocenaudio

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/rwbwq84smb1d3war2g561ijsqf9hfirb-ocenaudio-3.11.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/rwbwq84smb1d3war2g561ijsqf9hfirb-ocenaudio-3.11.2
ls -la /nix/store/rwbwq84smb1d3war2g561ijsqf9hfirb-ocenaudio-3.11.2/bin

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

1 package built:

ocenaudio

Maintainer pings
cc @onny for testing.

Result of nixpkgs-review pr 152105 run on x86_64-linux 1

1 package built:

ocenaudio


Please make storage configurable

Please make "storage" configurable:
sata0: "raid:#{config.qemu_disk_size},format=qcow2",

not everyone has a storage called raid, so it would be helpful, if this could be set by config.
Otherwise you'll get an error on boot like
create failed - storage 'raid' does not exists

Fixed in version 0.0.8


Server crashes with error: Closed already requested. Readable stream is unavailable.

I have await for request in server.get('*', async (context) => {. at server.tsx.
(I need to obtain Supabase session before SSR rendering start to do SSR routing correctly based on authenticated Supabase user permissions).
If I press [F5] key several times (< 7) in browser then server breaks with error Cannot enqueue chunk when underlying stream is not readable. Readable stream is unavailable. How to fix?!
See

https://github.com/d9k/citations-supabase-demo/blob/5d50bbf1a842a0a149bf658621be46782a286e90/server.tsx#L116
https://github.com/d9k/citations-supabase-demo/blob/5d50bbf1a842a0a149bf658621be46782a286e90/src/app/providers-constructors/supabase-ssr.tsx#L74

Can easily be reproduced with
await new Promise((resolve) => setTimeout(resolve, 300));

and press [F5] 5+ times in browser rapidly

Tried to upgade Deno (1.37.1 -> 1.38.4).
Error stack changed:
error: Uncaught TypeError: Readable stream is unavailable.
          controller.enqueue(encodeText(bufferedString));
                     ^
    at transformStreamDefaultControllerEnqueue (ext:deno_web/06_streams.js:4006:11)
    at TransformStreamDefaultController.enqueue (ext:deno_web/06_streams.js:6247:5)
    at https://deno.land/x/ultra@v2.3.8/lib/stream.ts:54:22
    at Object.action (ext:deno_web/02_timers.js:150:11)
    at handleTimerMacrotask (ext:deno_web/02_timers.js:64:10)
    at eventLoopTick (ext:core/01_core.js:184:21)

I prepared minimal example with error reproduction:
https://github.com/d9k/d9k-problems-examples/tree/main/ultra/286-readable-stream-is-unavailable
Please help

This is reproduced by simply adding asynchronous processing to the handler.
import { createServer } from "ultra/server.ts";

const server = await createServer({
  importMapPath: import.meta.resolve("./importMap.json"),
  browserEntrypoint: import.meta.resolve("./client.tsx"),
});

function wait(ms: number): Promise<void> {
  return new Promise<void>((resolve) => {
    setTimeout(() => resolve(), ms);
  });
}

function App() {
  return <html></html>;
}

server.get("*", async () => {
  await wait(200);

  const result = await server.render(<App />);

  return new Response(result, {
    headers: {
      "content-type": "text/html; charset=utf-8",
    },
  });
});

Deno.serve(server.fetch);

So, this would also occur in a throw promise, using useQuery.
There may be a fatal error in server.render.


Devel

Merging additional speed-ups to split heuristic

@joshua-d-campbell looks good to me. Let's talk about the cCG.splitY optimization; I'm not sure I get it from the commit message 😅


changes don't work

hello, can you tell me how to properly load a modified digital file through uassetgui into the game? I have nothing, I save the uasset file, then through cmd write it, but the game does not see the changes, do you have a detailed description of how to write new files in the game, Thank you.

Write the name of the game and, if possible, the version of the UE

Try consulting other modders and resources for the specific game you are trying to mod


Fix actions parameter naming mistake in README

hello there @kutovoys! I updated it, thanks. (together with adding an option to replace the whole directory and not only a single file)


Minified nsis-compat-updater Builds

Now we distribute only a bundle generated by webpack, which should take no more than 200KB and has no dependency, cheers!


Consider a way to support/add equalJSX method

I am currently using tape and I am considering switching to AVA.
There is just one piece missing atm: https://github.com/atabel/tape-jsx-equals
As you can see tape-jsx-equals use extend-tape to extends tape.
Are you open to add support of equalJSX in the core? If not will it be possible to provide a way to easily extends AVA?

Note: this is assertion is doable using https://github.com/algolia/react-element-to-jsx-string

Also please note that another method is really handy when testing react component: includeJSX. This one ignore whitespace differences which can be very handy.

@MoOx The ability to add custom assertions is on the roadmap.

You can already use any custom assertion. The only minus is that assertion counting doesn't work. We plan to add support for adding custom assertions, see #49.

FYI, here is how I do for now
import "babel-core/register"
import test from "ava"

import React, { Component } from "react"
import { createRenderer } from "react-addons-test-utils"
// import ReactDOMServer from "react-dom/server"
import expect from "expect"
import expectJSX from "expect-jsx"
expect.extend(expectJSX)

import MyComponent from ".."

test("MyComponent default render", (t) => {
  const renderer = createRenderer()

  renderer.render(
    <MyComponent />
  )
  expect(
    renderer.getRenderOutput()
  )
  .toEqualJSX(
    <div />
  )
})

Let's continue discussion in #446. Any future improvements will benefit from our solution to #25.


add wiki or README to describe the rename and package relocation

After released(both for spring-data-cosmosdb and spring-data-documentdb)

add another wiki or README about package rename
announce the old package name spring-data-documentdb will no longer be released after this relocation release.

Release document:
https://github.com/Microsoft/spring-data-cosmosdb/releases/tag/v2.0.3-docdb


Type specifiers should be supported in declare module

Right now, this is a syntax error:
declare module "a" {
  import {type t} from "b";
}

The error is:

Imports within a declare module body must always be import type or import typeof!

However, it should be allowed because a type specifier is a type, which is what the error is essentially saying that it isn't.

This is now supported