build(tvm_utility): remove download logic from CMake and update documentation

Description

See https://github.com/autowarefoundation/autoware.universe/issues/3137
According to discussion above remove downloading logic from building stage. For now it will be possible to download model files either by ansible or manually as explained in this PR
Also documentation about the package is updated to remove mentions about downloading. Kindly ask for maintainers to read throw the doc to check that it still consistent especially Security consideration part.
Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

I don't understand why it bothers anyone that there is an option to download default-compiled models. As you see the default is to not download the models, and there is already an option for the user to provide their own compiled/tuned models; so to me it looks like it already adheres to the proposed plan in https://github.com/autowarefoundation/autoware.universe/issues/3137.

ambroise-arm The main issue with current approach that it will produce empty Debian package by default as there is no ansible step in their build. And it is not possible to download or provide models in other way. They should be part of the source tree.
May be you can propose the other way to deal with it? May be it is possible to move model compilation  step further as it done with onnx2trt conversion? So the model files doesn't need for package compilation.
For now it was offered in Software WG to add header files for default model to the source tree. And the model weights will be provided by the user or downloaded by ansible script. If user want to use his models he will need to rebuild the package or just provide the deploy_param.params file if he just retrain the model.
Correct me, please, if get wrong how TVM works in general.

@ambroise-arm

I don't understand why it bothers anyone that there is an option to download default-compiled models. As you see the default is to not download the models, and there is already an option for the user to provide their own compiled/tuned models; so to me it looks like it already adheres to the proposed plan in https://github.com/autowarefoundation/autoware.universe/issues/3137.

Requiring packages to have access to internet during build step can make them vulnerable to supply chain attacks, in addition to not being able to make builds reproducible (e.g. for audits). For example, the Debian and ROS buildfarms restrict access to the internet for packages so they must be built in isolation. The end goal is so that all that logic for downloading artifacts done in CMake is removed and instead use ansible (or instruct the user to download the artifacts manually).

The main issue with current approach that it will produce empty Debian package by default as there is no ansible step in their build.

I don't understand how removing the option to download models with cmake solves this. The problem is that we lack an ansible step, not that there is a cmake download step as an option. At least from what I see. Is the ansible step mutually exclusive with the cmake download step? I don't know what this ansible step would look like (if you have example code somewhere I am interested), but I suppose it can be used together with the other ways to get the artifact, in the same way that the cmake logic currently prefers user-provided models, and if there are none try to download one.

May be it is possible to move model compilation step further as it done with onnx2trt conversion? So the model files doesn't need for package compilation.

Yes, but not with the current implementation. I had started to rework the way tvm_utility worked to allow to remove the need for the artifact to be present at compile time, but I didn't see much interest in TVM packages so I parked the work. Maybe I could continue on that, not immediately though.

Requiring packages to have access to internet during build step can make them vulnerable to supply chain attacks, in addition to not being able to make builds reproducible (e.g. for audits).

It is not a requirement but a convenience option. Currently user-provided artifacts are preferred over downloading one.

The end goal is so that all that logic for downloading artifacts done in CMake is removed and instead use ansible (or instruct the user to download the artifacts manually).

Ok, if there has been a definitive decision to remove the cmake downloads from all packages for whatever reason I can live with that. But can we agree it isn't justified by any of the points above?

I've updated tvm_utils. As example for other tvm based packages. Introduced new constructor for InferenceEngineTVM with additional parameter autoware_data_path. So package expect to find artifacts for inference (as deploy_graph.json, deploy_lib.so, deploy_param.params) in autoware_data/tvm_utility/ with the same structure as it was in tvm_utility/data folder.
Back compatible with other tvm packages as previous constructor is kept in place

@ambroise-arm Can you please elaborate a bit how tvm_utility should be reworked:

I had started to rework the way tvm_utility worked to allow to remove the need for the artifact to be present at compile time, but I didn't see much interest in TVM packages so I parked the work.

So we can consider such option as one possible solution.

@lexavtanke  At a high level InferenceEngineTVM needs to have its constructor interface changed so that it requires getting the network name and paths to the files to be dynamically loaded at runtime (the .so, .json and .params) instead of reading it from the artifact header file. And the remaining parameters from that header file need to be extracted from the network at runtime (with TVM's runtime_mod.GetFunction(...)). So that there is no dependency on the artifacts at build time.
All the tvm packages in autoware need to be changed to use the new interface. The path to those .so, .json and .params need to become parameters of the nodes making use of tvm_utilty. That probably needs to be integrated with the new way of getting the artifacts you were talking about, in order to make it easy for the user to select a specific backend, and potentially switching between them without too much effort (and also user-provided artifacts).
And then there can be optimizations that can be done to https://github.com/autowarefoundation/modelzoo/ in order to stop generating that header file that won't be used anymore.

@ambroise-arm What is the actual purpose of the test in TVM_utility? to be sure that model will be loaded during the inference and inference will happen, correct?
If so, we can provide the small model (as sum a + b), add it in source tree and use it during the the test. YOLOv2 Tiny will work as example for the user, so he will be able to build and run it if he want.
I guess it is more simple and proper solution for the current situation. As it provide simple test TVM_utility as a tool for other packages and provide example to the user.
What do you think?
@esteve What are you thoughts about it as well?

What is the actual purpose of the test in TVM_utility? to be sure that model will be loaded during the inference and inference will happen, correct?

Yes, it doesn't matter what kind of inference, it is just to exercise the TVM runtime and the tvm_utility interface.

we can provide the small model (as sum a + b), add it in source tree and use it during the the test

Committing the compiled artifacts from TVM to autoware may be a problem as they contain binary files. You may want to double check with maintainers if it is ok.

@esteve What are you thoughts about it as well?

Adding a test model to the tree is fine by me, that's the same as adding a JPEG image as a test fixture, which we already do. My only concern was that some test fixtures were distributed separately from the code, which is problematic, but as long as everything is accessible when cloning this repository, then it's ok.
If the test model is too big, please use git-lfs to add it.

Okay, thank you @ambroise-arm and @esteve I will implement it.

If so, we can provide the small model (as sum a + b), add it in source tree and use it during the the test. YOLOv2 Tiny will work as example for the user, so he will be able to build and run it if he want.

We shouldn't add binary model files to the git in Autoware Universe. We won't activate Git LFS for the Universe repositories too, if such files need to be a part of the repo, related package(s) should be moved to an external repository and added to the autoware.repos file.
So far, tvm_utility/README.md this repository performs 2 independent tasks:

Testing a yolov2 tiny model
Providing tvm_utility-extras.cmake file for other packages to use.

These should be separated into 2 packages.
The package responsible for downloading and testing the packages should be an external repository (Like tvm_testing), can have Git LFS or any other downloading method to test it. (Not distributed as a runtime package)
The actual part that is used by other packages can stay in the utility. It is the utility part.
Right now none of the TVM packages are used in anywhere and they are not tested in CI at all.
Maybe separating them like this will allow us to actually test the parts we require.
By separating the heavy test-required parts (requiring to download models) from cheap test parts.

@xmfcx

We shouldn't add binary model files to the git in Autoware Universe. We won't activate Git LFS for the Universe repositories too, if such files need to be a part of the repo, related package(s) should be moved to an external repository and added to the autoware.repos file.

I'd say it depends on how big the model files are. If they are tiny (properly tiny), I don't see the problem with adding them to this repository.
However, I'm not opposed to having a separate repository called tvm_testing which may contain the models and has Git-LFS enabled.

@esteve I've checked the file sizes:
mfc@mfc-leo:~/Downloads/aaa/yolo_v2_tiny$ tree -s -h
[4.0K]  .
├── [4.0K]  llvm
│   ├── [ 24K]  deploy_graph.json
│   ├── [161K]  deploy_lib.so
│   ├── [ 43M]  deploy_param.params
│   └── [1.6K]  inference_engine_tvm_config.hpp
└── [4.0K]  vulkan
    ├── [ 23K]  deploy_graph.json
    ├── [270K]  deploy_lib.so
    ├── [ 43M]  deploy_param.params
    └── [1.6K]  inference_engine_tvm_config.hpp

It's 42~MB for each architecture.
Downloaded and extracted networks-x86_64.tar.tar from here probably_requires_admin_access

I think we can remove the https://github.com/autowarefoundation/autoware.universe/blob/ad69c2851b7b84e12c9f0c3b177fb6a9032bf284/common/tvm_utility/tvm_utility-extras.cmake file since we will download the models with ansible.
And only other function of this package was to test a tvm pipeline. And we can move this out of universe to do that.
tvm_utility shouldn't be a part of debian packages.
And for the dependencies, like:

https://github.com/leo-drive/autoware.universe/blob/6ef6b548392232a366fa7a4263bb95e8802755a7/perception/lidar_centerpoint_tvm/package.xml#L22
these can be turned into <test_depend>

@lexavtanke I can help with creating a new repo for "tvm_testing under awf github.

@xmfcx if we want to test tvm_utility ability to load model, we need only  small model with size in the printscreen.

We don't need to have full yolo model in the repo.
And as I understand the main idea of the tvm_utility package is to provide pipeline for other packages like lidar_centerpoint_tvm and  lidar_apollo_segmentation_tvm.

It's very small then, it's OK to have it in the git tree, considering that we have some large images in the documentation parts of this repository.

It'd good to store it as git-lfs, Git repositories are not the best at storing binary data.

Still, tvm_utility doesn't need to have a debian package since it has no other function than being a reference package and having tests in it.

That's irrelevant, even if we don't need a Debian package for tvm_utiliy, it's good practice to keep test fixtures in the same place as tests to avoid admin overhead and to make sure tests are easily reproducible. Other projects like for example PCL do that as well, don't see a compelling reason why Autoware should be different.

It'd good to store it as git-lfs, Git repositories are not the best at storing binary data.

This is stating the obvious but I wish it was a simple decision to just switch this repository to utilize git-lfs.
About introducing git-lfs, my concerns are stated in this issue and can be discussed there.
An additional thing I've learned about Github LFS billing is,

In forks, bandwidth and storage usage count against the root of the repository network.
Forking and pulling a repository counts against the parent repository's bandwidth limit.
Source: https://docs.github.com/en/billing/managing-billing-for-git-large-file-storage/about-billing-for-git-large-file-storage#about-billing-for-git-large-file-storage

I've realized this when trying to pull LFS files of https://github.com/lgsvl/simulator and it said the owner didn't pay for the LFS billing.
I had to fork the repo, turn it into an archive, download it as a zip to get access to the LFS files since they didn't pay for the bills.
And forked repo wasn't even able to pull LFS files because forked repo billing is still tied to the original repo.
Only way to keep working with it was to recreate the repository and push it there.
Because of these reasons, I don't see it as a good option for current Universe repository.

That's irrelevant, even if we don't need a Debian package for tvm_utiliy, it's good practice to keep test fixtures in the same place as tests to avoid admin overhead and to make sure tests are easily reproducible. Other projects like for example PCL do that as well, don't see a compelling reason why Autoware should be different.

The reason I wanted to move the package out of the Universe repository was following:

The package had 2 functions:

Testing the tvm pipeline with a reference implementation.
Providing a cmake download script.

Download function is being moved to ansible, now it is not needed anymore.
If testing the pipeline requires a 40MB download, we can't have it in Autoware Universe, it needs to move out.

With recent information, I've learned that it's just about 25KB, I am also not too inclined to move it out too.

But with all these information, relevant or not, this package doesn't need a debian package because it is just a test package.
Still, I think it is better to rename this package to tvm_testing since that's the only job it does.

For reference, additional previous discussion on using git-lfs (here specifically for TVM models): https://github.com/orgs/autowarefoundation/discussions/2508

this package doesn't need a debian package

I agree with this part.

Still, I think it is better to rename this package to tvm_testing since that's the only job it does.

It is not there only for testing. This file: https://github.com/autowarefoundation/autoware.universe/blob/main/common/tvm_utility/include/tvm_utility/pipeline.hpp provides a common interface for all Autoware packages making use of TVM to access the models and organize the inference pipeline. Renaming to tvm_testing sounds misleading to me.

@ambroise-arm

this package doesn't need a debian package

I agree with this part.

I disagree. If other packages depend on tvm_utility and those packages generate Debian packages, it most definitely needs a Debian package.

After removing the artifact download functionality from tvm_utility, its only export will be the header file, which is of no use to other packages at runtime. And in my head the Autoware debian packages would only contain compiled binaries and other files needed at runtime. But also I haven't really followed the debian package discussions, so I may be completely off. Disregard my previous comment if it doesn't match reality.

@ambroise-arm Debian packages are built one by one in isolation to ensure that dependencies are correct, they are not built all at once from the source tree, therefore if a package depends on anything at build time from another package, that dependency needs to be packaged and exported to downstream packages.
In this case, the Debian package for tvm_utility would include the necessary header files that downstream packages depend on so that the code for such dependent packages can be compiled.
If you have any questions about how the Debian generation process works, I'll be happy to answer them.

@esteve and @ambroise-arm I've updated the PR. Now there is a small test case. It run a small model with one node abs. It computes element-wise absolute of input tensor.

I kept yolo_v2_tiny in the package as example pipeline. So the user can build and test it if he want. It's build is disableв by default.
For now artifact ansible role should be updated to download models for lidar_centerpoint_tvm and lidar_apollo_segmentation_tvm. Also documents for manual downloading should be updated.
And final change should be done in TVM packages. tvm_utility::pipeline::InferenceEngineTVM constructor should receive path to the autoware_data folder with models files.
Сmake function get_neural_network builds dependency for tvm packages and should be renamed. I think it will be better to do in separate PR.

@lexavtanke thanks for addressing my feedback, I think the small model you've added looks good.
As for the yolo_v2_tiny model, if you still want it to be included and not add the model files, it should be as an example program and not as a test. Test are meant to test the code and if the goal of the the yolo_v2_tiny example is to show how to use tvm_utility, then it should be moved to an examples folder and rewritten as a program instead of a test, a user could pass a path as an argument to the example program, so the model could be stored anywhere. Otherwise it's confusing whether the test is a test or an example.
We can discuss in the future moving the model fixtures to a separate repository if we want to store them with git-lfs, but for now it's not possible to enable git-lfs for autoware.universe

@esteve Thanks for your feedback. I've updated yolo_v2_tiny example. It is separated form the test case completely, and user can build it separately and launch with the launch file if he wants.
I also separated x86 model and arm model as @ambroise-arm offered in direct messages. So for now tests will works for both systems.
There is an only issue for now. As there is binary model files spell-check throws errors on it. Do you now work around for it? Is it possible to ignore the whole file? If so how can it be done? Should CI/CD scripts be modified?
I think that PR is ready for a review again.

@esteve Do you know who has right to update this file?
It's used by spell-check and I think .so file extension should be added to it.

I have found the repo for spell-check-dict https://github.com/tier4/autoware-spell-check-dict/tree/main and will PR.

@angry-crab could you approve this PR when you have a moment? You're one of the codeowners of this package, I approved these changes, but they won't merge unless a codeowner approves it as well. Thanks!

Am I not a codeowner of tvm_utility?

@ambroise-arm probably, but only @angry-crab had the codeowner logo, I assumed he was the only one. Anyway, @ambroise-arm thanks for approving this PR


adds 1-second leeway to JWT validation to prevent intermittent except…

…ions

Should it be fixed in newest version?

@Kak3n v2.0.0-RC5 and on have had this change.

thanks ok.
2016-05-25 18:49 GMT+02:00 Brent Shaffer<EMAIL_ADDRESS>
@Kak3n https://github.com/Kak3n v2.0.0-RC5
https://github.com/google/google-api-php-client/releases and on have
had this change.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/google/google-api-php-client/pull/828#issuecomment-221632975

On my server, the system time is 2 min prior to the current time so I got this problem. The system admin may have to run the ntp service for date updates.

Mine shows 5 hours later than what what the Google Token returns.

@jslewis90 I imagine this is an issue with your server's date/time configuration.

Now is impossible to set leeway. Please rollback this fix.

@martinfree You're right, this is not an optimal solution because it pollutes global scope. I'll submit a fix that ensures the adjustment is localized. It would be great if the Firebase\JWT class made leeway configurable per instance as well as statically.

@bshaffer thank you for considering my suggestion. Cheers.

In google-api-php-client/src/Google/AccessToken/Verify.php line 213 you are not adding 1 to JWT leeway like you mentioned in comments but rather setting it to one. Kindly fix it.
`
if (property_exists($jwtClass, 'leeway')) {
// adds 1 second to JWT leeway
// @see https://github.com/google/google-api-php-client/issues/827
$jwtClass::$leeway = 1;
}
`

This solution didn't worked with me. So, I tried adding 60 seconds to JWT leeway:
if (property_exists($jwtClass, 'leeway')) {
// adds 1 second to JWT leeway
// @see #827
$jwtClass::$leeway += 60;
}

Hello,
I have that same problem me too, The problem is solving when i changing $leeway variable value as "+=60", but i do not want change in core file of package. But i don't find any way overwrite this variable on another place.
There is any way changing that variable another place?

I'm sorry for the sarcasm that's about to come, but @hamzawain7 is right. The "geniuses" that "fixed" this are not adding to the leeway, but instead they're overwriting whatever value you set to it. For now having to manually edit the vendor files will have to do, although a fix would be appreciated. Thanks.

@FG-Developer You can pass a custom JWT object to Google_Client.
$jwt = new \Firebase\JWT\JWT;
$jwt::$leeway = 60; // adjust this value

// we explicitly pass jwt object whose leeway is set to 60
$this->client = new \Google_Client(['jwt' => $jwt]);

Refer this post.


Intellisense comments tooltip does not support Chinese character display

From @lishali12345 on June 2, 2016 7:8

VSCode Version:Version 1.1.1 (1.1.1)
OS Version: OSX EI Capitan Version 10.11.5

Steps to Reproduce:

The Chinese character in the second screen shot display like "??????". Please let me know If I can do any config to make this right, thanks a lot.
Copied from original issue: Microsoft/vscode#7120

I see. Thanks

From @joaomoreno on June 2, 2016 7:57
@jrieken This is hover. And clearly the div in the hover is able to render chinese characters. Shouldn't this be forwarded to the csharp guys?

I'm betting this will be addressed by https://github.com/OmniSharp/omnisharp-roslyn/pull/598.

I'm here just want to make something clear. When  I upgrade the Visual Studio Code to v1.3.1 and C# extension to v1.3.0, then the Chinese characters show in intellisense comments very well, just like the pic:

But when I just upgrade the Visual Studio Code to v1.4.0 release in July, the problem comes back again. So I roll back the Code to v1.3.1 to make me feel better with Chinese characters comments.
Thanks.

This issue really should be on the https://github.com/OmniSharp/omnisharp-vscode repo. However, this is now fixed for the next release of the C# Extension for VS Code. The latest beta with this change is available here.


Unable to install Apache Tomcat

Hello,QT,
I am trying to install the Apache Tomcat server using the documentation of Digital Ocean Documentation {https://www.digitalocean.com/community/tutorials/how-to-install-apache-tomcat-8-on-ubuntu-16-04} and i am losing at the point of updatig my service file and exiting my service file.
Please help me!!

@vishwendh105 Can you tell what is the issue which you are getting.

please post related error screenshot.

Hello, Sorry QT for the delay
Here is the screen shot of the error message and trying different commands to save and exit the file in terminal commands like :wq,^x,^b,

^X,^B etc. Please let me know the correct command.

check your installed version of tomcat and then try like this sudo systemctl start tomcatx where x is your tomcat version number.

I Had tried that command but it shows tomcat is NOT FOUND
On Thu, 23 Jul 2020 at 16:49, srmanikanta<EMAIL_ADDRESS>wrote:

check your installed version of tomcat and then try like this sudo
systemctl start tomcatx where x is your tomcat version number.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/QT-DevOps/DevOpsIssues/issues/920#issuecomment-662950850,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMG5ZEPQDTPOKCOYOIXMXGLR5AMERANCNFSM4OXG45XA
.

you are executed some commands for tomcat installation so please post tomcat installation commands here i will tell you which version of tomcat your are installed.


fix Issues #19 transfer amount precision problem

fix Issues #19 transfer amount precision problem

Thanks for your PR.
close #19 .


Update to Schemas

Summary
Update to schemas for consistency with the spec document.
Details and comments

Added versions to the schemas
Split registers into memory and registers
Changed conditional structure with gates (added bfunc command)
Added backend configuration information about conditional logic

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

Thanks @dcmckayibm !


Hide validation details by default using Bootstrap theme

Closes #3

Hide validation details by default, toggled with "Show details"/"Hide details" link
Validation error message now uses  Bootstrap alert theme
Fix problem with validation of idspace if not on first line of file (it still checks that it is at the start of a line)

This is working just how I want it to. Thanks!
For the implementation, please try refactoring the code to avoid building HTML with JavaScript. I'm hoping it will be possible to specify all the required elements in the HTML template, then just update them in JavaScript as needed: show/hide, change alert-* class, change text.
In general, feel free to use jQuery in this code, since we're already loading it for Bootstrap.


Ensures memory request and limit is same for rabbitmq

The rabbitmq operator has recommendation that the memory request and limit be the same. This patch ensures that we request 10G memory and also have a 10G limit.

I think it makes sense to have requests but we can eliminate that all of our limits. This was there from our initial impl, and something that we don't really need.

@sulochan I think it makes sense to have requests but we can eliminate that all of our limits. This was there from our initial impl, and something that we don't really need.

@cloudnull This is a recommendation from rabbitmq.
https://www.rabbitmq.com/kubernetes/operator/using-operator#:~:text=The RabbitMQ high-water mark,limits as the same value.&text=Note%3A It's possible for RabbitMQ,cause an immediate OOM kill.
For memory resource, setting request and limit both is recommended and to the same value. From what I gather from looking at the code, the limit is used to calculate overhead and therefore affects high watermark and overall evm health. Judging by the overhead calculation this is more important for large nodes since the cut off seems to be either 20% or 2Gi.
There is actually a specific check on the controller for just this.
See: https://github.com/rabbitmq/cluster-operator/blob/main/internal/status/no_warnings.go#L41
Was there a reason why we didn't / don't want it set ?

In general there's a common recommendation to set memory and cpu limits, however, I've come to understand is that limits are performance impacting and give the application no room to grow or otherwise leverage idle resources available on the node; I think our current limit approach has created an artificial scale problem.
re: https://home.robusta.dev/blog/stop-using-cpu-limits
For every opinion piece out there there's a counter piece
re: https://dnastacio.medium.com/why-you-should-keep-using-cpu-limits-on-kubernetes-60c4e50dfc61
I kinda agree with both arguments and can see how scaling groups could be better leveraged to allow our pods to grow / shrink; however, given the capabilities of our expected infrastructure, I think we can eliminate all of our limits as they stand today. Should we want to change that in the future we could do so as a feature, making better use of both horizontal and verticle scaling groups in a more targeted approach.

Yeah, I agree to a large extent on NO CPU limits.
However, I do not believe this applies for Memory Limits. Best practice recommendation for Memory is to always have request and limit and request == limit, which is the recommendation from rabbitmq with the cluster warning check.
I suggest, that I will update this PR to 16Gi request/limit from the current 10Gi that i modified and try that out. We can always increase if we need.
How do you feel about that ?


Date localization

How can I localize the date formatting?

Hey @zeusfly-spb, can you explain with examples as to what you mean by localizing the format?

Hello, xaksis! Thanks for the response, I have already solved the problem in php. What can you say about dynamically changing the width of columns in a table, such as colResizeable?

@zeusfly-spb if you mean setting it programmatically, you can do it via "width" property of column. If you mean via user dragging to change width, that is currently not supported.
either way, i'm going to close this ticket since you seem to have solved it.


error on running kbst local apply on macos

Hey,
getting an error when running
kbst local apply
│ Error: failed to generate kubeadm config content: failed to get kubernetes version from node: failed to get file: command "docker exec --privileged cm-dsml-dev-loc-eks-f6e2913-control-plane cat /kind/version" failed with error: exit status 1
│
│   with module.eks_zero.module.cluster.kind_cluster.current,
│   on .terraform/modules/eks_zero/kind/_modules/kind/main.tf line 26, in resource "kind_cluster" "current":
│   26: resource "kind_cluster" "current" {
│
╵
2022/01/21 11:25:50 error watching repository: watching 'manifests/bases' failed: lstat manifests/bases: no such file or directory

running on macOS Monterey v12.1 with the m1 chip, if that helps.

Is there a workaround available for this issue?

Closing because local cmd has been removed.


Fix iotbsp and some minor formating changes

ms-iot/bsp replaced with ms-iot/rpi-iotcore

OPS Build status updates of commit fa032ce:
:clock10: Preparing: average preparing time is 42 sec(s)

OPS Build status updates of commit fa032ce:
:clock10: Incremental building: average incremental building time is 18 sec(s)

OPS Build status updates of commit fa032ce:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/learn-about-hardware/SoCsAndCustomBoards.md
:white_check_mark:Succeeded

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.

OPS Build status updates of commit c810951:
:clock10: Preparing: average preparing time is 42 sec(s)

OPS Build status updates of commit c810951:
:clock10: Incremental building: average incremental building time is 18 sec(s)

OPS Build status updates of commit c810951:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/learn-about-hardware/SoCsAndCustomBoards.md
:white_check_mark:Succeeded

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.

OPS Build status updates of commit 70b4ece:
:clock10: Preparing: average preparing time is 42 sec(s)

OPS Build status updates of commit 70b4ece:
:clock10: Incremental building: average incremental building time is 18 sec(s)

OPS Build status updates of commit 70b4ece:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/learn-about-hardware/SoCsAndCustomBoards.md
:white_check_mark:Succeeded

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.

OPS Build status updates of commit ea8d091:
:clock10: Preparing: average preparing time is 42 sec(s)

OPS Build status updates of commit ea8d091:
:clock10: Incremental building: average incremental building time is 18 sec(s)

OPS Build status updates of commit ea8d091:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/learn-about-hardware/SoCsAndCustomBoards.md
:white_check_mark:Succeeded

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.


Web Component and Verify website display different information in case the last file was edited without content credentials

Hi!
I’m not sure if this is an issue, but I found the following a bit confusing from the user's perspective: it looks like the Web Component and the Verify website react in different ways in case the file was exported without content credentials and the previous content ingredient contains content credentials.
Steps to reproduce:

Export an image with content credentials
Open the same image again with content credentials

Make some changes
Export it again without content credentials

Use the Web Component and Verify website to display the content credentials of the last exported image

Expected behavior:
The Web Component and Verify websites should display the same CAI metadata (if any).
Actual behavior:
The Verify website displays “This file was edited without content credentials” but the Web Component displays the CAI metadata from the image with content credentials.
Additional information:

I used Photoshop 24.4.1 to edit the image(s).
I have attached two screenshots and the image used for testing here.

Test image: https://github.com/contentauth/c2pa-js/files/11295944/image.zip

Thanks for brining this up @francesco-smartframe - this is definitely not intended and we prioritize fixing this with the proper user experience. Also appreciate the test image as that is very helpful.


Runelite doesn't work with GPU at all

Until the last update I would set my Nvidia GPU as the primary renderer for the game's graphics and it worked flawlessly. Now it is always shown as inactive and I can't play with the GPU anymore, which makes other activities a pain. I'm submitting this as an issue becuase I've tried configuring the Nvidia in many ways and it hasn't been fruitful.
GFX model: Nvidia GeForce 840M (in case it can be helpful)
Feel free to delete this if it doesn't fit as a client issue, I'll understand

Are you sure you are running latest launcher? E.g 1.5.1

The latest I can download from runelite.net is 1.3.6. Where do I download 1.5.1 from?

1.5.1 is the launcher version, not the client version. If you have downloaded runelite recently your launcher version is most likely up to date.

Any update? Launcher update should solve this.

Sorry, had lots of things to do irl. I confirmed by reinstalling and even downloading from the launcher repo that the launcher version is indeed 1.5.1. The GPU is not showing any activity. I've tried assigning the .exe again in the Nvidia Control Panel, but it didn't work :(

I'm experiencing the same issue. Win 10, i5-7300, GTX 1060. I'd prefer RuneLite to run on the GTX 1060, so I can watch livestreams better, since that consumes a lot of CPU as well.

I guess this is not even an issue after all, OSRS runs on CPU so I dont understand where is the problem.


dummy pr

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
What is the link to the Apache JIRA
(Please create an issue in ASF JIRA before opening a pull request,
and you need to set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g. HDDS-XXXX. Fix a typo in YYY.)
Please replace this section with the link to the Apache JIRA)
How was this patch tested?
(Please explain how this patch was tested. Ex: unit tests, manual tests)
(If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)

One other background process needs to be tested: RocksDBCheckpointDiffer::pruneOlderSnapshotsWithCompactionHistory  I don't see any tests for it here.  Am I missing them?

I'd also like to see one more test, where we actually do a small snap diff and confirm the results are as expected.

One other background process needs to be tested: RocksDBCheckpointDiffer::pruneOlderSnapshotsWithCompactionHistory I don't see any tests for it here. Am I missing them?

You are not missing them. I missed it. Looking into it.


todo-md-rs: init at 1.0.0

Description of changes

A project of mine to manage todos while storing them in markdown and having customizable paths to integrate with pkm tools.
I hope it's not too irrelevant as I'd like to install it the proper way (tm)
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I think it might be better suited for https://github.com/nix-community/NUR

I think it might be better suited for nix-community/NUR

What do you think? Then you can still install it the "proper way" through NUR.

What do you think? Then you can still install it the "proper way" through NUR.

Hey, I'm sorry for neglecting this PR. Ever since the semester started I couldn't find the time to get back to this.
Nevertheless, this seems like the optimal route to take with my little project and thank you kindly for making me aware of its existence :].


fix Link binlog files according to consistent rowsets

In previous implementations, binlog files would be linked according to making snapshot requests. However, sometimes not all requests can be executed directly. For example, when a certain version in missing_version does not exist, it will fallback to the full snapshot. Therefore, it is correct to link binlog files according to consistent rowsets.

run buildall


feat(cdk/bidi): support auto direction value

Adds support for the auto value of the dir attribute. The value gets resolved to ltr or rtl based on browser's locale.
Fixes #10186.
Note: I didn't include a test, because mocking navigator.language is difficult.

The feedback has been addressed.

nice work <3 @crisbeto crisbeto


Dev-compiler assumes & , | and ^ only exists on int.

I am adding non-short-circuit boolean operators to the bool class.
In doing so, I find DDC generating incorrect code, where the a & b is compiled into something similar in JavaScript, followed by >>0. This looks like an assumption that if the operations was allowed on a native object, then it must be an integer operation, without actually checking that it was an integer operation.
See: https://dart-review.googlesource.com/c/sdk/+/25240 and the error in:
https://logs.chromium.org/logs/dart/buildbucket/cr-buildbucket.appspot.com/8937943941959450128/+/steps/ddc_tests/0/stdout
If compiling the code:
main(List<String> args) {
 bool b = args.isEmpty;
 print(b | b);
}

the resulting JS code contains:
   core.print((dart.notNull(b) | dart.notNull(b)) >>> 0);

where it should instead call the interceptor for bool.operator|.

thanks! Not sure if you got my message when you were working on your CL, but I already made a CL to fix this (https://dart-review.googlesource.com/c/sdk/+/71228). Anyways I'm happy to verify it works and land it :)

Verified that my fix works with your patch. I will see if I can submit it on its own to unblock you. Otherwise you could mark DDC failing and I'll fix it in my CL.

Your change looks good, so I'll hold for it (less status file juggling is always better). Thanks!


Add more code formatting

Since import sorting seems to be outside of Black's scope, we'll be using something else for more opinionated code formatting and import sorting

Seems like the best one is isort, so gonna use that with default settings

Also adding ssort as well

Closed by #40


Re-factor code for variants long table

Inspired by @saramonzon awesome pandas implementation, the Python script has been re-written and now produces the output below:
SAMPLE,CHROM,POS,REF,ALT,FILTER,DP,REF_DP,ALT_DP,AF,GENE,EFFECT,HGVS_C,HGVS_P,HGVS_P_1LETTER,LINEAGE,CALLER
SAMPLE1_PE,MN908947.3,241,C,T,PASS,489,4,483,0.99,orf1ab,upstream_gene_variant,c.-25C>T,.,.,B.1,ivar
SAMPLE1_PE,MN908947.3,1875,C,T,PASS,92,62,29,0.32,orf1ab,missense_variant,c.1610C>T,p.Ala537Val,p.A537V,B.1,ivar
SAMPLE1_PE,MN908947.3,3037,C,T,PASS,213,0,213,1.0,orf1ab,synonymous_variant,c.2772C>T,p.Phe924Phe,p.F924F,B.1,ivar
SAMPLE1_PE,MN908947.3,11719,G,A,PASS,195,9,186,0.95,orf1ab,synonymous_variant,c.11454G>A,p.Gln3818Gln,p.Q3818Q,B.1,ivar
SAMPLE1_PE,MN908947.3,14408,C,T,PASS,572,2,570,1.0,orf1ab,synonymous_variant,c.14143C>T,p.Leu4715Leu,p.L4715L,B.1,ivar
SAMPLE1_PE,MN908947.3,20268,A,G,PASS,338,0,338,1.0,orf1ab,stop_lost,c.20003A>G,p.Ter6668Trpext*?,p.*6668Wext*?,B.1,ivar

TODO

[ ]  Add to ONT pipeline
[ ] Test if you use --skip_pangolin / --skip_snpeff
[ ] Do we want all of those columns in the long output? e.g. FILTER
[ ] Double-check format of table for --variant_caller ivar and --variant_caller bcftools is the same

Thanks to @ErikaKvalem for the python implementation, and @svarona  for the initial R implementation :)

Nice! Added @ErikaKvalem to the Credits section https://github.com/nf-core/viralrecon/pull/268/commits/1bdebea7a6890cd1b6f6288170787c432febdd59 and invited her to become a member of the nf-core org.


Data augmentation strategy includes translation of odd/even slices

Some 2D scans (e.g. PSIR, STIR) include images that are acquired by combining odd and even slices separately. This is susceptible to patient motion as consecutive slices can have a larger duration to be acquired. This also leads to challenging issues with registration to 3D scans (e.g T2w).
Suggestion by @jcohenadad: A good data augmentation strategy would be to shift/translate odd and even slices in a scan for training training a multi-contrast model.

Some 2D scans (e.g. PSIR, STIR) include images that are acquired by combining odd and even slices separately.

Not exactly. This is called a 'sequential' 2D acquisition wherein each slice is acquired separately, and the timing is 2, 4, 6, ... 1, 3, 5, etc. So "odd" and "even" are not acquired as two separate blocks, but rather do manifest as two separate entities in that their timing is shifted. Eg: it is possible that motion would occur at the last 10% of the acquisition, in which case odd/even would be perfectly aligned from slices 1->n, but then screwed up from n+1->m.

The section "Multislice imaging" in this chapter gives a good explanation/more details of how the acquisition is done.


Incorrect copyright Division in footer

Where to find the issue
Footer of each page on https://www.coronawarn.app/
Describe the issue
The copyright Content Division element in the footer of each page is incorrectly terminated with
:

© 2020-2022 - The authors of the Corona-Warn-App open-source project

Steps to reproduce
Run https://validator.w3.org on https://www.coronawarn.app/en/ and note errors

"End tag footer seen, but there were open elements"
"Unclosed element div"

Suggestion
Change the closing tag of copyright to

PR https://github.com/corona-warn-app/cwa-website/pull/3224 submitted to resolve this issue

Following the merge of PR #3224 running https://validator.w3.org/ on https://www.coronawarn.app/en/
no longer outputs the error "End tag footer seen, but there were open elements"

The remaining reasons for the message: "Unclosed element div" are covered in issue https://github.com/corona-warn-app/cwa-website/issues/3236

Closing, as this issue is now resolved.


Error only in production: Cannot read properties of undefined (reading 'name')

I've used your library in the past and it worked great, but now that I'm using it in a different app I'm running into the weirdest issue that appears only when running in production, even if it's still running locally.
It's in a<EMAIL_ADDRESS>app, and I have an API route that calls this method:
export const syncBib = async (appConfig: any, bibId: number = 1) => {
    const bibpath = appConfig.bibPath
    const f = new FileData(path.join(appConfig.fsRoot, bibpath))
    if(!f.exists()){
        console.warn(`Could not find a bibliography .bib file at path ${bibpath}. Your bibliography will not sync until this is resolved. Make sure to add the appropriate "bibPath" field to your appConfig.ulld.json file.`)
        return {
            errorKey: "bibPathFileDoesNotExist"
        } satisfies ErrorResponse<"bibManager">
    }
    const content = f.getContent()
    const parsed = parseBibFile(content)
    const entries = BibEntry.fromFsList(parsed.entries_raw)
    const { citations, userDefined } = getFormattedCslCitation(content, appConfig)
    let connectOrCreateEntries: Prisma.BibEntryCreateOrConnectWithoutBibInput[] = []
    const ids = citations.getIds() as string[]
    const lowerCaseIds = ids.map((x) => x.toLowerCase())
    for (const entry of entries) {
        const _args = entry.connectOrCreateArgs()
        _args.create.htmlCitation = citations.format('bibliography', {
            format: 'html',
            template: userDefined ? "user-defined" : 'apa',
            entry: entry.getIdInBibFile(ids, lowerCaseIds)
        })
        connectOrCreateEntries.push(_args)
    }
    return await prisma.bib.upsert({
        where: {
            id: bibId
        },
        create: {
            filename: bibpath,
            entries: {
                connectOrCreate: connectOrCreateEntries
            },
            lastSync: new Date()
        },
        update: {
            filename: bibpath,
            entries: {
                connectOrCreate: connectOrCreateEntries
            },
            lastSync: new Date()
        },
        include: {
            entries: {
                include: {
                    tags: true,
                    citationGroups: true,
                    MdxNotes: {
                        select: {
                            title: true,
                            id: true,
                            href: true,
                        }
                    }
                }
            }
        }
    })
}

The error appears to occur here:
     _args.create.htmlCitation = citations.format('bibliography', {
            format: 'html',
            template: userDefined ? "user-defined" : 'apa',
            entry: entry.getIdInBibFile(ids, lowerCaseIds)
        })

I am getting back the proper id in that getIdInBibFile method, and the user-defined model is applied in the getFormattedCslCitation function that looks like this:
import "@citation-js/plugin-csl";
import "@citation-js/plugin-bibtex";
import Cite from "citation-js";
import path from "path";
import { AppConfigSchemaOutput } from "@ulld/configschema/types";
import { FileData } from "@ulld/utilities/fileClass";

export const getFormattedCslCitation = (
    content: string,
    appConfig: AppConfigSchemaOutput,
) => {
    let useUserDefined = false;
    if (appConfig.cslPath) {
        let f = new FileData(path.join(appConfig.fsRoot, appConfig.cslPath), false);
        if (f.exists()) {
            const cslFileContent = f.getContent();
            if (cslFileContent) {
                let config = Cite.plugins.config.get("csl");
                if(config){
                    useUserDefined = true;
                    config?.templates.add("user-defined", cslFileContent);
                } else {
                    console.error("Could not gather csl config. Cannot parse citations with user provided csl format.")
                }
            }
        }
    }
    let citations = new Cite(content);
    return { citations, userDefined: useUserDefined };
};

This all works flawlessly in development, but I've spent 3 days now working on this error. This is the complete output here:
Don't judge me for my home directory.
TypeError: Cannot read properties of undefined (reading 'name')
    at t.XmlJSON.addMissingNameNodes (/Users/bigsexy/Desktop/current/ulldSandbox/ulldApp/.next/server/chunks/34354.js:17:43244)
    at new t.Engine (/Users/bigsexy/Desktop/current/ulldSandbox/ulldApp/.next/server/chunks/34354.js:17:67469)
    at u (/Users/bigsexy/Desktop/current/ulldSandbox/ulldApp/.next/server/chunks/34354.js:1:373946)
    at c (/Users/bigsexy/Desktop/current/ulldSandbox/ulldApp/.next/server/chunks/34354.js:1:374213)
    at bibliography (/Users/bigsexy/Desktop/current/ulldSandbox/ulldApp/.next/server/chunks/34354.js:1:374923)
    at tl (/Users/bigsexy/Desktop/current/ulldSandbox/ulldApp/.next/server/chunks/34354.js:1:28027)
    at g.tm [as format] (/Users/bigsexy/Desktop/current/ulldSandbox/ulldApp/.next/server/chunks/34354.js:1:28107)
    at d (/Users/bigsexy/Desktop/current/ulldSandbox/ulldApp/.next/server/chunks/34354.js:17:2746)
    at async v (/Users/bigsexy/Desktop/current/ulldSandbox/ulldApp/.next/server/app/api/events/onSync/route.js:1:12672)
    at async /Users/bigsexy/Desktop/current/ulldSandbox/ulldApp/node_modules/next/dist/compiled/next-server/app-route.runtime.prod.js:6:36258

Since the addMissingNameNodes appears to be a part of the csl plugin, and the only references I could find to it came from the APA csl content that citation-js provides, I tried this with the provided APA style, the provided harvard1 style, and a csl file of my own that generates the AIP citation style. Both the AIP and the APA style work fine in development, but in production they fail with the above error. I didn't test the Harvard style in development, or the Vancouver style in either development or production.
I've also tried applying the 'force-dynamic' status to the API route which is more Next specific, but still no luck.
If you want to look at the project
There are two projects on my profile, one is a large monorepo called ulld, and the other a small React/Next application that utilizes most of the monorepo, ulldApp. The app is generated by a build script to populate most of the useful bits, but you should be able to test the API route without running the build script, although it will require a file called appConfig.ulld.json at the root of the project that looks something like this to not blow up.
Thanks, and this is an awesome library regardless.
If there's any other information that might help, please let me know. I've been working on this project for 6+ months, and I'm a week away from wrapping it up but this bug is the single thing getting in the way of the last little bit of tidying up that I need to do and it's driving me insane.
The versions look like this:

Package
Version

@citation-js/plugin-csl
0.7.14

@citation-js/plugin-bibtex
0.7.14

citation-js
0.7.15 (tried 0.7.14 as well)

Thank you for the report. The error occurs in the citeproc-js dependency, the csl plugin is mostly a wrapper around that dependency. There was a similar issue with Next.js producing errors in citeproc-js code last year as well (#194), I will have to investigate.

I think I might be able to figure it out a bit quicker if you could send ulldApp/.next/server/chunks/34354.js (that way I don't have to set up the build process myself), but I will take a look regardless.

I was able to set up a minimal example myself, so I don't need the file anymore.

Could you try Next.js v15.0.0-canary.121? Or are there too many breaking changes to try?

Looks like swc is optimizing a function like the following (it's parseXml):
function foo (arg) {
    const a = { b: [] }
    const c = [a.b]
    c[0].push(arg)
    return a.b[0]
  }

into the useless
function foo (n) {
  return [[]][0].push(n),
  [][0]
}

But I think it has been fixed in a recent update, as with the canary v15 it produces
function(e) {
  let n = [];
  return [n][0].push(e),
  n[0]
}

Oh lol the bug report in swc actually mentioned Citation.js as well https://github.com/swc-project/swc/pull/8956

I can try it with Next 15. There are quite a few moving pieces in the generated app that don't appear in the template so it might introduce some issues, but right now my biggest concern is citation-js. If you took a look at my bio it kind of describes how this app came to be, and in the original app that isn't generated from a config file citation-js works flawlessly. It's<EMAIL_ADDRESS><EMAIL_ADDRESS>and<EMAIL_ADDRESS>in that app, and<EMAIL_ADDRESS>I tried using those citation-js related versions in the current app and still had the same issue, but I'll give this a shot with<EMAIL_ADDRESS>It wouldn't even be the end of the world to move back to 14.0.4 if it gets this to work.

Update
Thanks so much for looking at this so quickly. Once you narrowed it down to swc, I was able to disable swc with swcMinify: false in the next.config.mjs file. I didn't attempt to build with<EMAIL_ADDRESS>because it requires<EMAIL_ADDRESS>which would introduce quite a few issues and I wasn't sure if that would even be successful, but in<EMAIL_ADDRESS>the swcMinify option will be removed entirely and be forced to be true.

in<EMAIL_ADDRESS>the swcMinify option will be removed entirely and be forced to be true according to some comments in their code.

Oh wow, I didn't realise that. That seems like a bad idea given how many issues there seem to be.

I created another issue here if you want to keep track of it, just to let them know about the issue with swc since they seem to be kind of aware that forcing people to use swc might introduce a lot of issues.


InputGroup needs onSubmitEditing

I'm having problems implementing onSubmitEditingto InputGroup:
<InputGroup style={styles.input}>
       <Icon style={{color: '#fff'}} name='md-mail' />
       <Input placeholder='EMAIL' />
</InputGroup>

In react native this already works:
<TextInput
  style = {styles.titleInput}
  returnKeyType = {"next"}
  autoFocus = {true}
  placeholder = "Title"
  onSubmitEditing={(event) => {
    this.refs.SecondInput.focus();
  }}
/>
<TextInput
  ref='SecondInput'
  style = {styles.descriptionInput}
  multiline = {true}
  maxLength = {200}
  placeholder = "Description" />

Regards,
Peer from Polygoal

Did you try using onSubmitEditing with Input inside TextInput?

Try this!

I had a similar problem, and the solution above didn't work.
I managed to get it to work by replacing
this.refs.SecondInput._textInput.focus()
with
this.refs.SecondInput._root.focus()
Here's the whole thing:
<Container>
        <Image source={require( "../../assets/hipster-cards.png" )} style={styles.backgroundImage}>
        <Content ref={c => this._content= c}>
          <KeyboardAvoidingView
            behavior='position'
            style={styles.form}
            >
            <Form>
              <InputGroup
                style={styles.email}
                regular>
              <Input
                returnKeyType='next'
                placeholder='Email'
                keyboardType='email-address'
                value={this.state.email}
                onChangeText={
                  ( email ) => this.setState({ email })
                }
                onSubmitEditing={
                  (event) => {
                    console.log("onSubmitEditing");
                    console.log(this.refs);
                    console.log(this.refs.SecondInput);
                    this.refs.SecondInput._root.focus();
                    }
                  }
                />
            </InputGroup>
            <InputGroup style={styles.password} regular>
              <Input
                secureTextEntry
                ref='SecondInput'
                returnKeyType='done'
                placeholder='Password'
                value={this.state.password}
                onChangeText={
                  ( password ) => this.setState({ password })
                }/>
            </InputGroup>
            <View style={styles.buttonRow}>
              <Button style={styles.buttons} light>
                <Text>Sign Up</Text>
              </Button>
              <Button style={styles.buttons} primary>
                <Text>Sign In</Text>
              </Button>
            </View></Form>
          </KeyboardAvoidingView>

      </Content>
      </Image>
      </Container>

Apologies if it's spaghetti -- I'm still a beginner.

@trevor-coleman This is definitely an issue that should be reopened (or a new issue should be submitted). When passing refs to NativeBase Components I expect the ref callback to return the raw element for basic components like Input. Either this should be fixed, or what I think is a better solution, this should be documented.

This should help: https://reactjs.org/docs/forwarding-refs.html

Its been long time that InputGroup is deprecated


Create a document DB plug in/connector (part 2)

Created by: @bayegan Dec 07 2021 at 20:47 CET
Last updated: Jul 19 2022 at 10:35 CET
ClickUp ID: IK-1934
As part of IK's authorization service, create a document DB plug in/connector to be used by the ingest engine.  This is a POC and use Vipps/Curriki as relevant use cases.
Acceptance criteria:
Document results
Suggest next steps
Present findings in a huddle meeting for this sprint
Attachment(s):
https://github.com/indykite/clickup-attachments/blob/master/e3eb789b-9465-452b-b8c6-76110f1a3980.pdf
https://github.com/indykite/clickup-attachments/blob/master/7d223aa5-f6de-4ca9-adc3-326460178dac.p7s
https://github.com/indykite/clickup-attachments/blob/master/6bb308f4-d92a-45b0-81d0-a698a1143ef3.pdf
https://github.com/indykite/clickup-attachments/blob/master/2c5adf5c-63c7-45f7-821a-18a03291a30e.json

Commented by: @bayegan Nov 29 2021 at 02:16 CET
Added documentation from Vipps.


chore(sdk): make sdk transition files public

Issue being fixed or feature implemented

Some files of the SDK were not public so they couldn't be accessed from the TUI where they are needed.
What was done?

Just made the files public.
How Has This Been Tested?

Breaking Changes

Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added or updated relevant unit/integration/functional/e2e tests
[x] I have added "!" to the title and described breaking changes in the corresponding section if my code contains any
[x] I have made corresponding changes to the documentation if needed

For repository code-owners and collaborators only

[ ] I have assigned this pull request to a milestone

Does this affect Dash Platform stability testing (iaw your guide), through use of Platform TUI ? Should we wait on the merger of this pull request ?

Does this affect Dash Platform stability testing (iaw your guide), through use of Platform TUI ? Should we wait on the merger of this pull request ?
Edit : or is the relevant part possibly affecting Platform stability testing already merged through pull request 1832 ?

@qwizzie this will just allow for the NFT-related state transitions to be used in the TUI, both in the strategy tests once I add them, and also just in the Contracts screen.


Download dll for BulletSharp 2.83 differs from the one in Demos projects

I did an upgrade to your latest BulletSharp version 2.83, but could not compile because Matrix and Vector3 weren't available in the dll. Using those from SharpDX didn't work either. Then I checked your Demo projects and saw that the dll used there differs a few bytes, although having the same version number. Taking the dll from Demos I was able to compile.

Forgot to mention, it's about the SharpDX BulletSharp dll.

Only the dll in the "Release Generic" folder will have the Matrix and Vector3 classes. Others such as the SharpDX version will use Matrix/Vector3 classes from SharpDX.
The demos also use the Generic version, because it's not dependant on any external library and so you can switch between graphics frameworks (by pressing F8).

Thank you for the quick answer! Totally makes sense. But then I think my problem is that the BulletSharp dll is built with the unsigned SharpDX dll. Is this possible? As we're using the signed one, it won't compile.

Yes, it's linked to the unsigned version. I'll provide a version linked to signed SharpDX in the future. This must be related to https://code.google.com/p/bulletsharp/issues/detail?id=73
For now, you could compile BulletSharp manually if you need this.

2.83.5 includes a version for signed SharpDX:
https://drive.google.com/folderview?id=0B2TPMwtggT0jfmRzRm1RQkJOYjZXVWV6Y2FwZGxKV1cxWE1ZTVlWd2paZ2RWLTE5MTEtMEk#list


HTTP POST: [FromBody] Attribute - TypeLoadException

Seems like there is still something wrong with the controller generation on POST Endpoints - I receive a typeload exception during export :   KY.Generator.targets(37, 5): Error: System.TypeLoadException occurred. Could not load type 'Microsoft.AspNetCore.Mvc.ModelBinding.IConfigureEmptyBodyBehavior' from assembly 'Microsoft.AspNetCore.Mvc.Abstractions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'.
... this seems somehow  to be related to the [FromBody] Attribute => If I remove the Attribute the generator runs, unfortunately the endpoint doesn't work in this case ;)
Full log attached:
Could not load type 'Microsoft.AspNetCore.Mvc.ModelBinding.IConfigureEmptyBodyBehavior' from assembly 'Microsoft.AspNetCore.Mvc.Abstractions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'.
   at System.ModuleHandle.ResolveTypeHandleInternal(RuntimeModule module, Int32 typeToken, RuntimeTypeHandle[] typeInstantiationContext, RuntimeTypeHandle[] methodInstantiationContext)
   at System.Reflection.RuntimeModule.ResolveType(Int32 metadataToken, Type[] genericTypeArguments, Type[] genericMethodArguments)
   at System.Reflection.CustomAttribute.FilterCustomAttributeRecord(MetadataToken caCtorToken, MetadataImport& scope, RuntimeModule decoratedModule, MetadataToken decoratedToken, RuntimeType attributeFilterType, Boolean mustBeInheritable, ListBuilder`1& derivedAttributes, RuntimeType& attributeType, IRuntimeMethodInfo& ctorWithParameters, Boolean& isVarArg)
   at System.Reflection.CustomAttribute.AddCustomAttributes(ListBuilder`1& attributes, RuntimeModule decoratedModule, Int32 decoratedMetadataToken, RuntimeType attributeFilterType, Boolean mustBeInheritable, ListBuilder`1 derivedAttributes)
   at System.Reflection.CustomAttribute.GetCustomAttributes(RuntimeModule decoratedModule, Int32 decoratedMetadataToken, Int32 pcaCount, RuntimeType attributeFilterType)
   at System.Reflection.CustomAttribute.GetCustomAttributes(RuntimeParameterInfo parameter, RuntimeType caType)
   at System.Reflection.RuntimeParameterInfo.GetCustomAttributes(Type attributeType, Boolean inherit)
   at System.Attribute.InternalParamGetCustomAttributes(ParameterInfo param, Type type, Boolean inherit)
   at System.Attribute.GetCustomAttributes(ParameterInfo element, Boolean inherit)
   at System.Reflection.CustomAttributeExtensions.GetCustomAttributes(ParameterInfo element)
   at KY.Generator.AspDotNet.Readers.AspDotNetControllerReader.IsFromBodyParameter(ParameterInfo parameter)
   at KY.Generator.AspDotNet.Readers.AspDotNetControllerReader.Read(AspDotNetReadConfiguration configuration, List`1 transferObjects)
   at KY.Generator.AspDotNet.Commands.AspDotNetReadControllerCommand.Run(IOutput output)
   at KY.Generator.Command.CommandRunner.Run(IEnumerable`1 commands, IOutput output)
   at KY.Generator.Reflection.Commands.AttributesCommand.Run(IOutput output)
   at KY.Generator.Command.CommandRunner.Run(IGeneratorCommand command, IOutput output)
   at KY.Generator.Generator.Run()

Sorry for that - this seems to be related to a messed up nuget package caching!
Finally all builds perfectly => I cleaned up the nuget and build folder => After all the build runs without any issues!

Sorry for that - this seems to be related to a messed up nuget package caching!
Finally all builds perfectly => I cleaned up the nuget and build folder => After all the build runs without any issues!


SQLite connections not properly closed

SqliteConnection is not being closed correctly.
Repro: repeatedly call this method using a context configured to use Sqlite. It should always return false, but will return true on occasion.
public bool Test_IsConnectionOpenAfterDispose()
{
	DbConnection connection;
	using(var context = new MyContext())
	{
		context.Hats.Add(new Hat { Color ="red"});
		context.SaveChanges();
		connection = context.Database.AsRelational().Connection.DbConnection;
	}

	return connection.State == ConnectionState.Open;
}

Possibly due to Reactive-Extensions/Rx.NET#106?

I believe Reactive-Extensions/Rx.NET#106 only affects async paths though.

I believe this was fixed with https://github.com/aspnet/Microsoft.Data.Sqlite/pull/98. I have not been able to reproduce.


Alt Key accelerators aren't rendered

Menu, Button, etc.
I looked into it and figured this much out:

There is a AccessKeyHandler which listens at the visual root level.
Whenever a control is part of the parent it registers (somehow) with it to be notified when an AccessKey should be displayed.
The AccessKeyHandler has an owner set to it, and when it has an owner it registers for key events and toggles the ShowAccessKeys property, which triggers all of the children to render with the _ on the character.
Each element looks for the AccessKeyHandler (somehow) and subscribes to changes on the ShowAccessKeys property.
ConsoleWindow was not mapping Alt key correctly, it was coming through as a space.

I updated ConsoleWindow.cs to look like this:
 private void HandleKeyInput(WindowsConsole.InputRecord inputRecord)
 {
     WindowsConsole.KeyEventRecord keyEvent = inputRecord.KeyEvent;
     RawInputModifiers modifiers =
         ModifiersFlagTranslator.Translate(keyEvent.dwControlKeyState);
     char character = keyEvent.UnicodeChar;
     Avalonia.Input.Key key;
     if (character == '\0')
     {
        // modifier only key, there is no character, avalonia maps this as a Key.LeftAlt key etc.
         switch (modifiers)
         {
             case RawInputModifiers.Control:
                 key = Avalonia.Input.Key.LeftCtrl;
                 break;
             case RawInputModifiers.Alt:
                 key = Avalonia.Input.Key.LeftAlt;
                 break;
             case RawInputModifiers.Shift:
                 key = Avalonia.Input.Key.LeftShift;
                 break;
             default:
                 key = Avalonia.Input.Key.None;
                 break;
         }
     }
     else
         key = DefaultNetConsole.ConvertToKey((ConsoleKey)keyEvent.wVirtualKeyCode);

     RaiseKeyPress(key, character, modifiers, keyEvent.bKeyDown, (ulong)Stopwatch.GetTimestamp());
 }

Much to my dismay, this did not fix it. I think that somehow the accesskeyhandler is not being registered correctly because our root window is a ConsoleWindow, but I haven't figured out where/how that happens.  It looks like it might have something to do with TopLevel, which we don't even implement?  Each control calls RegisterServices and inside there it seems to create an accesshandler...?
Anyhow. I though I would file this bug with my learnings to hopefully help either me or the next guy who comes along and tries to fix this.

I don't understand what should work. What is AccessKey? I see HotKey on the Button for instance.

When you press the ALT key, all of the accelerator letters will get an underscore rendered under them.

So ALT+F => opens File menu
ALT+F + O => Opens File Menu, selects Open menuItem
etc.

Stack trace when access key was pressed on Avalonia under Linux:
MainWindow.Button_OnClick() at /home/jinek/Repo/Trash/AvaloniaApplication1/AvaloniaApplication1/MainWindow.axaml.cs:line 86
Interactive.<AddHandler>g__InvokeAdapter|4_0<Avalonia.Interactivity.RoutedEventArgs>() at /_/src/Avalonia.Base/Interactivity/Interactive.cs:line 67
Interactive.<>c__4<RoutedEventArgs>.<AddHandler>b__4_1()
EventRoute.RaiseEventImpl() [3]
EventRoute.RaiseEvent() [3]
Interactive.RaiseEvent() [3]
Button.OnClick()
Button.OnAccessKey()
Button.<>c.<.cctor>b__15_0()
RoutedEvent<RoutedEventArgs>.<>c__DisplayClass1_0<Button>.<AddClassHandler>g__Adapter|0()
RoutedEvent.<>c__DisplayClass23_0.<AddClassHandler>b__0()
AnonymousObserver<(object, RoutedEventArgs)>.OnNext()
LightweightObservableBase<(object, RoutedEventArgs)>.PublishNext()
LightweightSubject<(object, RoutedEventArgs)>.OnNext()
RoutedEvent.InvokeRaised()
EventRoute.RaiseEventImpl() [2]
EventRoute.RaiseEvent() [2]
Interactive.RaiseEvent() [2]
AccessKeyHandler.OnKeyDown()
[Lightweight Method Call]
MethodInvoker.Invoke()
RuntimeMethodInfo.Invoke()
Delegate.DynamicInvokeImpl()
EventRoute.RaiseEventImpl() [1]
EventRoute.RaiseEvent() [1]
Interactive.RaiseEvent() [1]
KeyboardDevice.ProcessRawEvent()
InputManager.ProcessInput()
TopLevel.HandleInput() at /home/jinek/.config/JetBrains/Rider2024.3/resharper-host/SourcesCache/29f3b7d33f648d907396766a137fb623e567f92649bd8e1d4b1f8ac8e20e247/TopLevel.cs:line 744
X11Window.DispatchInput()
RawEventGrouper.Dispatch()
ManualRawEventGrouperDispatchQueue.DispatchNext()
X11PlatformThreading.RunLoop()
DispatcherFrame.Run()
Dispatcher.PushFrame()
Dispatcher.MainLoop()
ClassicDesktopStyleApplicationLifetime.Start()
ClassicDesktopStyleApplicationLifetimeExtensions.StartWithClassicDesktopLifetime()
Program.Main()

Yeah I think the buttons works it's the visualization that doesn't

I've done handling for Windows (CTRL, SHIFT and ALT added).
The question:
Access key drawing is done by AccessText.RenderCore:
private protected override void RenderCore(DrawingContext context)
        {
            base.RenderCore(context);
            int underscore = Text?.IndexOf('_') ?? -1;

            if (underscore != -1 && ShowAccessKey)
            {
                var rect = TextLayout!.HitTestTextPosition(underscore);

                var x1 = Math.Round(rect.Left, MidpointRounding.AwayFromZero);
                var x2 = Math.Round(rect.Right, MidpointRounding.AwayFromZero);
                var y = Math.Round(rect.Bottom, MidpointRounding.AwayFromZero) - 1.5;

                context.DrawLine(
                    new Pen(Foreground, 1),
                    new Point(x1, y),
                    new Point(x2, y));
            }
        }

@tomlm We probably can not draw this properly, right? Do we need some own special AccessText control?

Oh awesome! Just make the pen stroke thickness  UnderlineThickness  and it will turn the text to TextDecoration underline!
internal class DrawingContextImpl : IDrawingContextImpl
    {
        public const int UnderlineThickness = 10;

oh I see, I think a custom AccessText with UnderlineThickness.

I filed a bug with Avalonia, https://github.com/AvaloniaUI/Avalonia/issues/17763
I may try to do a PR back to Avalonia...unless you feel inspired to do it.

But how does it help us. Its entire string being rendered, will be entire string underline, not just one symbol

no, it's using a pen over the top of characters to draw the underline. Avalonia doesn't use the font system to underline, it draws them with pens.  It's drawing an underline under just the accelerator key character.
TextBlock normally uses the FontMetric.UnderlineThickness to do that, and so I use a thickness of 10 to signify that we should apply TextDecoration of underline to the pixel.

Blarg.

I want to check what we've got on linux and default .net console also

@tomlm UX question: in some cases we can not receive ALT key alone.
Should we always draw the underline? Or should we define hotkey to switch on underlines?

My suggesion:
Draw understrike always. But expose a method to switch it off and bind some keys to show them, like:
public static AppBuilder BuildAvaloniaApp()
         => AppBuilder.Configure<App>()
                .UseConsolonia()
                .UseAutoDetectedConsole()
                .NoAccessKeysAlwaysOn(new KeyGesture(Key.A, KeyModifiers.Alt)) // !
                .LogToException();

I think leaving them on is fine. That's what  Windows 3.x  used to do.


osinfo-db: 20230308 -> 20230719

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/osinfo-db/versions.
meta.description for osinfo-db is: Osinfo database of information about operating systems for virtualization provisioning tools
meta.homepage for osinfo-db is: https://gitlab.com/libosinfo/osinfo-db/
meta.changelog for osinfo-db is: https://gitlab.com/libosinfo/osinfo-db/-/commits/v20230518
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
found 20230518 with grep in /nix/store/7cr9wansya8lf9iz3gc3gwr21fwb1ihb-osinfo-db-20230518
found 20230518 in filename of file in /nix/store/7cr9wansya8lf9iz3gc3gwr21fwb1ihb-osinfo-db-20230518

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

22 total rebuild path(s)

22 package rebuild(s)

First fifty rebuilds by attrpath
cinnamon.cinnamon-gsettings-overrides
cinnamon.nemo-fileroller
cinnamon.nemo-with-extensions
dropbox-cli
eiciel
gnome-photos
gnome.file-roller
gnome.gnome-boxes
gnome.gnome-control-center
gnome.gnome-terminal
gnome.nautilus
gnome.nautilus-python
gnomeExtensions.gtk4-desktop-icons-ng-ding
libosinfo
nautilus-open-any-terminal
osinfo-db
pantheon.file-roller
pantheon.file-roller-contract
phosh
phosh-mobile-settings
tracker-miners
virt-manager

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/7cr9wansya8lf9iz3gc3gwr21fwb1ihb-osinfo-db-20230518 \
  --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 osinfo-db https://github.com/r-ryantm/nixpkgs/archive/f4f7cab23523aa8e73f399cfe28adb234e8f25ec.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/7cr9wansya8lf9iz3gc3gwr21fwb1ihb-osinfo-db-20230518
ls -la /nix/store/7cr9wansya8lf9iz3gc3gwr21fwb1ihb-osinfo-db-20230518/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 failed to build:

virt-manager

20 packages built:

cinnamon.cinnamon-gsettings-overrides
cinnamon.nemo-fileroller
cinnamon.nemo-with-extensions
dropbox-cli
eiciel
gnome-photos
gnome.file-roller
gnome.gnome-boxes
gnome.gnome-control-center
gnome.gnome-terminal
gnome.nautilus
gnome.nautilus-python
gnomeExtensions.gtk4-desktop-icons-ng-ding
libosinfo
nautilus-open-any-terminal
osinfo-db
pantheon.file-roller-contract
phosh
phosh-mobile-settings
tracker-miners

Maintainer pings
cc @bjornfor for testing.

@ofborg build virt-manager

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/osinfo-db/versions.
meta.description for osinfo-db is: Osinfo database of information about operating systems for virtualization provisioning tools
meta.homepage for osinfo-db is: https://gitlab.com/libosinfo/osinfo-db/
meta.changelog for osinfo-db is: https://gitlab.com/libosinfo/osinfo-db/-/commits/v20230719
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
found 20230719 with grep in /nix/store/aihxixbzl1kp3hrb08ab7m2r1fvq9an7-osinfo-db-20230719
found 20230719 in filename of file in /nix/store/aihxixbzl1kp3hrb08ab7m2r1fvq9an7-osinfo-db-20230719

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

23 total rebuild path(s)

23 package rebuild(s)

First fifty rebuilds by attrpath
cinnamon.cinnamon-gsettings-overrides
cinnamon.nemo-fileroller
cinnamon.nemo-with-extensions
dropbox-cli
eiciel
gnome-photos
gnome.file-roller
gnome.gnome-boxes
gnome.gnome-control-center
gnome.gnome-terminal
gnome.nautilus
gnome.nautilus-python
gnomeExtensions.gtk4-desktop-icons-ng-ding
guestfs-tools
libosinfo
nautilus-open-any-terminal
osinfo-db
pantheon.file-roller
pantheon.file-roller-contract
phosh
phosh-mobile-settings
tracker-miners
virt-manager

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/aihxixbzl1kp3hrb08ab7m2r1fvq9an7-osinfo-db-20230719 \
  --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 osinfo-db https://github.com/r-ryantm/nixpkgs/archive/e91373c5062767bfb1286a64a940ece12c0808dd.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/aihxixbzl1kp3hrb08ab7m2r1fvq9an7-osinfo-db-20230719
ls -la /nix/store/aihxixbzl1kp3hrb08ab7m2r1fvq9an7-osinfo-db-20230719/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 failed to build:

virt-manager

21 packages built:

cinnamon.cinnamon-gsettings-overrides
cinnamon.nemo-fileroller
cinnamon.nemo-with-extensions
dropbox-cli
eiciel
gnome-photos
gnome.file-roller
gnome.gnome-boxes
gnome.gnome-control-center
gnome.gnome-terminal
gnome.nautilus
gnome.nautilus-python
gnomeExtensions.gtk4-desktop-icons-ng-ding
guestfs-tools
libosinfo
nautilus-open-any-terminal
osinfo-db
pantheon.file-roller-contract
phosh
phosh-mobile-settings
tracker-miners

Maintainer pings
cc @bjornfor for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/osinfo-db/versions.
meta.description for osinfo-db is: Osinfo database of information about operating systems for virtualization provisioning tools
meta.homepage for osinfo-db is: https://gitlab.com/libosinfo/osinfo-db/
meta.changelog for osinfo-db is: https://gitlab.com/libosinfo/osinfo-db/-/commits/v20231027
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
found 20231027 with grep in /nix/store/lj7my22dpfxzzcgplzw8hqr9mknhg1xp-osinfo-db-20231027
found 20231027 in filename of file in /nix/store/lj7my22dpfxzzcgplzw8hqr9mknhg1xp-osinfo-db-20231027

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

23 total rebuild path(s)

23 package rebuild(s)

First fifty rebuilds by attrpath
cinnamon.cinnamon-gsettings-overrides
cinnamon.nemo-fileroller
cinnamon.nemo-with-extensions
dropbox-cli
eiciel
gnome-photos
gnome.file-roller
gnome.gnome-boxes
gnome.gnome-control-center
gnome.gnome-terminal
gnome.nautilus
gnome.nautilus-python
gnomeExtensions.gtk4-desktop-icons-ng-ding
guestfs-tools
libosinfo
nautilus-open-any-terminal
osinfo-db
pantheon.file-roller
pantheon.file-roller-contract
phosh
phosh-mobile-settings
tracker-miners
virt-manager

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/lj7my22dpfxzzcgplzw8hqr9mknhg1xp-osinfo-db-20231027 \
  --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 osinfo-db https://github.com/r-ryantm/nixpkgs/archive/085adadc0bd3f80eb3a5c94d0ea822cf8740d0fd.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/085adadc0bd3f80eb3a5c94d0ea822cf8740d0fd#osinfo-db

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/lj7my22dpfxzzcgplzw8hqr9mknhg1xp-osinfo-db-20231027
ls -la /nix/store/lj7my22dpfxzzcgplzw8hqr9mknhg1xp-osinfo-db-20231027/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 failed to build:

virt-manager

21 packages built:

cinnamon.cinnamon-gsettings-overrides
cinnamon.nemo-fileroller
cinnamon.nemo-with-extensions
dropbox-cli
eiciel
gnome-photos
gnome.file-roller
gnome.gnome-boxes
gnome.gnome-control-center
gnome.gnome-terminal
gnome.nautilus
gnome.nautilus-python
gnomeExtensions.gtk4-desktop-icons-ng-ding
guestfs-tools
libosinfo
nautilus-open-any-terminal
osinfo-db
pantheon.file-roller-contract
phosh
phosh-mobile-settings
tracker-miners

Maintainer pings
cc @bjornfor for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/osinfo-db/versions.
meta.description for osinfo-db is: Osinfo database of information about operating systems for virtualization provisioning tools
meta.homepage for osinfo-db is: https://gitlab.com/libosinfo/osinfo-db/
meta.changelog for osinfo-db is: https://gitlab.com/libosinfo/osinfo-db/-/commits/v20231215
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
found 20231215 with grep in /nix/store/c9g3rmfb7n31wwj6i5fhzfpkrnkjdck2-osinfo-db-20231215
found 20231215 in filename of file in /nix/store/c9g3rmfb7n31wwj6i5fhzfpkrnkjdck2-osinfo-db-20231215

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

23 total rebuild path(s)

23 package rebuild(s)

First fifty rebuilds by attrpath
cinnamon.cinnamon-gsettings-overrides
cinnamon.nemo-fileroller
cinnamon.nemo-with-extensions
dropbox-cli
eiciel
gnome-photos
gnome.file-roller
gnome.gnome-boxes
gnome.gnome-control-center
gnome.gnome-terminal
gnome.nautilus
gnome.nautilus-python
gnomeExtensions.gtk4-desktop-icons-ng-ding
guestfs-tools
libosinfo
nautilus-open-any-terminal
osinfo-db
pantheon.file-roller
pantheon.file-roller-contract
phosh
phosh-mobile-settings
tracker-miners
virt-manager

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/c9g3rmfb7n31wwj6i5fhzfpkrnkjdck2-osinfo-db-20231215 \
  --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 osinfo-db https://github.com/r-ryantm/nixpkgs/archive/660b2fbf4017a4113400035a9d9dd7452bd399b4.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/660b2fbf4017a4113400035a9d9dd7452bd399b4#osinfo-db

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/c9g3rmfb7n31wwj6i5fhzfpkrnkjdck2-osinfo-db-20231215
ls -la /nix/store/c9g3rmfb7n31wwj6i5fhzfpkrnkjdck2-osinfo-db-20231215/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.
nixpkgs-review took longer than 45m and timed out

Maintainer pings
cc @bjornfor for testing.

Superseded by #279266.


open /root/.docker/cert.pem: no such file or directory

regression from #25
via @jverdeyen

Thanks :)

Fixed. Note that the docker hub image takes ~5min to update.

That was fast! Thanks!


Potential thread bug when multiple dataset are requested from GSIP

While testing a prototype harvester, we generated HTTP 500 errors.  In this special case, we asked for three datasets from the Node (https://geoconnex.ca/gsip/info/LOD_Node/CAN_Hydro_LOD_Node) "simultaneously" (in quick succession).  The system eventually recovers (?)

this might be it..
https://github.com/eclipse-ee4j/jersey/issues/3850
the outputstream cuts before the streaming is over ?

it stalls right here
RDFDataMgr.write(sw,ModelUtil.getAlternateModel(mdl),format);

Seems that RDFDataMgr.write balks when the file is too large. LIMIT 10000 in the SPARQL "fixes" the problem

This might be related:
http://jersey.576304.n2.nabble.com/returning-an-output-stream-td1321529.html
when the client cancels the connection, the StreamingOutput keeps sending data

Could not narrow the problem, really RDFDataMgr.write(sw,ModelUtil.getAlternateModel(mdl),format); stalling when the number of triples > ~80000 .  I just added a LIMIT to the statement and tweaked jersey.config.server.contentLength.buffer (did not make any change) and push this to avoid application crashing.  problem not resolved, just contained

a quick fix , without deploying a new version is to edit templates/node_all.ftl and add a LIMIT clause
eg:
# the whole thing
CONSTRUCT {?s ?p ?o.}
WHERE {?s ?p ?o.}
LIMIT 20000

commit faa52b6d8565dd5d04d0fb31090a25d293f69a59


OpenGL.error.Error: Attempt to retrieve context when no valid context

Running on a Ubuntu server without a physical monitor.
I have tried add data_path, but it's not working.
Does that need a monitor to work?
Also I am wondering are there any other python tools(not necessarily with a viewer) that can edit gaussian splat results, like combining 2 gaussian point clouds into one with specified relative poses?
Traceback (most recent call last):
File "wkx/splatviz/run_main.py", line 233, in
main()
File "/home/wkx/anaconda3/envs/gs-view/lib/python3.10/site-packages/click/core.py", line 1157, in call
return self.main(*args, **kwargs)
File "/home/wkx/anaconda3/envs/gs-view/lib/python3.10/site-packages/click/core.py", line 1078, in main
rv = self.invoke(ctx)
File "/home/wkx/anaconda3/envs/gs-view/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/home/wkx/anaconda3/envs/gs-view/lib/python3.10/site-packages/click/core.py", line 783, in invoke
return __callback(*args, **kwargs)
File "wkx/splatviz/run_main.py", line 226, in main
viz = Visualizer(data_path=data_path, use_gan_decoder=use_decoder)
File "splatviz/run_main.py", line 39, in init
super().init(title="splatviz", window_width=1920, window_height=1080, font=self.regular_font_path, code_font=self.code_font_path)
File "wkx/splatviz/gui_utils/imgui_window.py", line 46, in init
self._imgui_renderer = _GlfwRenderer(self._glfw_window)
File "wkx/splatviz/gui_utils/imgui_window.py", line 113, in init
super().init(*args, **kwargs)
File "/home/wkx/anaconda3/envs/gs-view/lib/python3.10/site-packages/imgui_bundle/python_backends/glfw_backend.py", line 20, in init
super(GlfwRenderer, self).init()
File "/home/wkx/anaconda3/envs/gs-view/lib/python3.10/site-packages/imgui_bundle/python_backends/opengl_backend.py", line 61, in init
super(ProgrammablePipelineRenderer, self).init()
File "/home/wkx/anaconda3/envs/gs-view/lib/python3.10/site-packages/imgui_bundle/python_backends/base_backend.py", line 17, in init
self._create_device_objects()
File "/home/wkx/anaconda3/envs/gs-view/lib/python3.10/site-packages/imgui_bundle/python_backends/opengl_backend.py", line 146, in _create_device_objects
gl.glVertexAttribPointer(
File "/home/wkx/anaconda3/envs/gs-view/lib/python3.10/site-packages/OpenGL/latebind.py", line 63, in call
return self.wrapperFunction( self.baseFunction, *args, **named )
File "/home/wkx/anaconda3/envs/gs-view/lib/python3.10/site-packages/OpenGL/GL/VERSION/GL_2_0.py", line 469, in glVertexAttribPointer
contextdata.setValue( key, array )
File "/home/wkx/anaconda3/envs/gs-view/lib/python3.10/site-packages/OpenGL/contextdata.py", line 58, in setValue
context = getContext( context )
File "/home/wkx/anaconda3/envs/gs-view/lib/python3.10/site-packages/OpenGL/contextdata.py", line 40, in getContext
raise error.Error(
OpenGL.error.Error: Attempt to retrieve context when no valid context

Running on a Ubuntu server without a physical monitor. I have tried add data_path, but it's not working. Does that need a monitor to work?

Yes the viewer requires a Monitor. There are plans in the future to have the rendering on a server but currently it has to render and display the results on the same machine.

Also I am wondering are there any other python tools(not necessarily with a viewer) that can edit gaussian splat results, like combining 2 gaussian point clouds into one with specified relative poses?

I am currently not aware of any other python tools that allow editing of 3DGS scenes. Supersplat is a web viewer that allows some editing, however, its not in python and the rendering is quiet slow for large scenes.

OK Thank you for your answer. I have switched to a local machine and ran it without trouble.


Added quick start guide for new users

I added the quickstart guide from the readme to the documentation. I think especially for new users reaching the documentation through blogposts (...) it's far simpler then to check out the readme on this GitHub page.

Ping?

@aleontiev Could you please take a look at this?

@aleontiev Could you please take a look at this?

changes have been included


add instructions and checks for the Discord auth setup

right now, we just point folks to the Python SDK's docs, but it's an involved process

this gets a lot easier with #72


Request

Hey Folks, I really much like what you do! Especially since you try to reduce complexity for a normally hard problem.
Yet I'd super appreciate if you could reduce this a little further. Do you think that is possible?
All that setup and stuff looks rather difficult to me. Would be great!
Thank you.

Hi @ChrisChross  Appreciate that, and it's entirely fair!

For pure viz, you may enjoy our csv/xls no-code workflow: https://www.graphistry.com/blog/graphistry-2-35-9-no-code-graph-analysis-for-your-csvs-ha-for-multi-gpus-run-on-a100-cuda-11-0

... we're pushing on getting connector configs + dashboarding to that level too :)

Meanwhile, any particular aspects you'd especially like replaced, and how?

Will close for now as now directly actionable. We'll keep pushing!


fix: configure Vite config optimizeDeps.entries option

fixes https://github.com/sveltejs/kit/issues/11793
We need to specify optimizeDeps.entries for the Vite config to scan for dependencies that should be optimised because we have set rollupOptions.build.input and that overrides Vite's default scanning behaviour. This prevents the excessive Vite dependency optimisation and reloading on page navigation during development when it discovers dependencies used by the page.
Note that this PR merges the optimizeDeps.entries specified by the user's Vite configuration instead of overriding it. We considered just adding these to the Vite user config so it's more visible but there's no way to retrieve the dynamic path of the Kit routes folder and hook file names which are configurable in svelte.config.js.
Additional context from the maintainer's Discord chat.

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

Edits

[x] Please ensure that 'Allow edits from maintainers' is checked. PRs without this option may be closed.

great to see the added test working :+1:


Fix Graph Editor unintentional camera orbit in render view

This fixes the logic that detects whether the cursor is in the render view. Previously, there were three known issues that are fixed with this change:

The camera would orbit when grabbing the scrollbar located next to the render view.
The camera would orbit when clicking a menu option in the menu dropdown overlapping the render view.
The previous logic to detect the cursor in the render view did not consider that the render view moves up when scrolling. As a result, the camera would still orbit when clicking parts of the Node Property Editor that were previously occupied by the render view.

This change fixes issue #1984 and another similar issue I noticed during this work (see comment in issue).

This looks really promising, @LeaRe, and I'm CC'ing @lfl-eholthouser for her expertise with this code.

This looks great, @LeaRe! Thanks


Added required symbol to Array and Object fieldset captions.

Refactored Arrays to use TitleField, added required prop to TitleField. Updated the README to reflect the changes.
Referencing: https://github.com/mozilla-services/react-jsonschema-form/issues/31

FYI, the test fail is not related to my changes: https://travis-ci.org/mozilla-services/react-jsonschema-form/jobs/125439098

Yes, failing test is related to #162.

The patch looks pretty good, thanks. Could you please add some tests?

I added a few tests and updated the ObjectField test.

Thank you. Looks good with a few nits to be addressed.

Landed in c73e039515cf783607831466b954f0be626d30bb.