HIP: Implementation of CUE for Values Processing and Validation (Superseding Original JSON Schema Support)

---
hip: 9999
title: "Implementation of CUE for Values Processing and Validation (Superseding Original JSON Schema Support)"
authors: [ "Jiří Altman<EMAIL_ADDRESS>Jan von Löwenstein ]
created: "2024-08-07"
type: "feature"
status: "draft"
---

Abstract
This HIP proposes the implementation of CUE (Configuration Unification Engine) as the primary tool for processing and validating Helm values files, replacing the current JSON Schema support. CUE, being a superset of YAML, JSON, and JSON Schema, extends the existing capabilities while maintaining full backward compatibility. This ensures that all existing Helm charts and configurations remain fully supported, with the added benefit of enhanced validation, better defaulting, and easier schema management.
Motivation
The current JSON Schema implementation for validating Helm values files has several significant limitations:

Lack of Updates: The existing JSON Schema implementation in Helm is based on Draft-07 and has not been actively developed or maintained. It supports only a subset of the JSON Schema specification and contains non-standard issues, leading to inconsistent behavior and limited validation capabilities.

Complexity: Writing and maintaining JSON schemas for Helm values files often requires deep expertise, making it challenging for users who are not deeply familiar with JSON Schema to create and manage their configurations.

Feature Limitations: Helm Values lack advanced features like defaulting, referencing of values, and flexible schema definitions, which are increasingly necessary for handling complex Helm charts.

CUE addresses these issues by providing:

Support for Newer JSON Schema Specs: CUE brings out-of-the-box support for newer JSON Schema specifications, ensuring compatibility with modern JSON Schema features and addressing the limitations of the current implementation.

Simplified Schema Management: CUE simplifies the process of writing and managing schemas, making it more accessible for users who are not experts in JSON Schema. This lowers the barrier to entry and allows more users to effectively utilize Helm's validation capabilities.

Enhanced Features: CUE natively supports better defaulting, flexible referencing of values, and other advanced configuration management features, which are critical for complex Helm deployments.

Rationale
The decision to integrate CUE as the primary validation tool in Helm is based on several factors:

Backward Compatibility: CUE's nature as a superset of YAML, JSON, and JSON Schema ensures that all existing Helm configurations and JSON Schemas will continue to work without any changes. This seamless transition is a key advantage for users.

Support for Modern Standards: CUE's support for the latest JSON Schema specifications means Helm users will benefit from modern features and bug fixes that were not available in the old implementation.

Ease of Use: CUE's design makes it easier for users to write and understand schemas, without requiring deep expertise in JSON Schema. This democratizes the use of Helm and makes it more accessible to a broader audience.

Native Advanced Features: CUE's built-in features like better defaulting, value referencing, and flexible schema definitions provide a more powerful and flexible validation system, allowing users to manage more complex configurations with ease.

Consistency and Flexibility: CUE's ability to handle multiple data formats and its integration with YAML and JSON make it a perfect fit for Helm's needs. It allows for more sophisticated configurations while ensuring consistency across different environments.

Community Feedback: The Helm community has expressed the need for a more capable validation system. CUE addresses these needs without sacrificing the stability or compatibility of existing charts.

Specification
The implementation will involve several key changes:

CUE Integration: Helm will integrate CUE as a new validation engine for values files. This will involve embedding CUE's runtime into Helm and providing command-line options to enable CUE-based validation.

Continued Support for JSON Schema: While CUE will be the primary tool for new validations, JSON Schema will continue to be supported, allowing users to choose the most appropriate tool for their needs.

Documentation: Comprehensive documentation and migration guides will be provided to assist users in understanding how to use CUE for validation, as well as how to continue using their existing JSON Schemas if preferred.

Backward Compatibility Assurance: Since CUE is fully backward compatible with YAML, JSON, and JSON Schema, no changes will be required for existing charts. The system will maintain full support for all current validation schemas and formats.

Community Feedback Mechanism: A feedback mechanism will be set up to collect user experiences and issues related to the adoption of CUE, ensuring that the transition is as smooth as possible.

Backwards compatibility
The introduction of CUE will not disrupt any existing Helm configurations, as CUE is a superset of YAML, JSON, and JSON Schema. This means that all existing values.yaml files, JSON Schema validations, and related configurations will continue to work as expected. Users can adopt CUE at their own pace, with the assurance that their current setups will remain fully functional. Additionally, CUE will extend support for newer JSON Schema specifications, ensuring continued relevance and support for evolving standards.
Security implications
There are no known security implications associated with this change. However, the new validation capabilities offered by CUE will be reviewed to ensure that they do not introduce any new vulnerabilities, particularly in how they process and validate input.
How to teach this
To educate users about this change:

Documentation: Update the Helm documentation to include a section on using CUE, with examples and best practices.
Migration Guides: Although full backward compatibility is maintained, migration guides will be provided for users who wish to take full advantage of CUE's capabilities.
Workshops and Webinars: Host online workshops and webinars to demonstrate the benefits of CUE and how to use it effectively in Helm charts.
Community Engagement: Engage with the community through forums, GitHub discussions, and social media to answer questions and gather feedback.

Reference implementation
A proof-of-concept implementation has been developed and is available in PR https://github.com/helm/helm/pull/13236. This implementation demonstrates the integration of CUE with Helm and includes basic examples of values file validation using CUE.
Rejected ideas

Enhancing JSON Schema: Extending JSON Schema to meet the needs of Helm was considered but ultimately rejected due to the complexity and limitations of JSON Schema in handling non-JSON formats like YAML.
Custom Validation System: Developing a custom validation system for Helm was also considered, but CUE's existing ecosystem and support made it the preferable choice.

Open issues

Performance Impact: The performance impact of integrating CUE into Helm is still under evaluation. Benchmarks and performance testing will be conducted to ensure that CUE does not negatively affect Helm's performance.
Community Adoption: Encouraging the community to adopt CUE and transition away from JSON Schema may require additional efforts, including incentives and extended support during the transition period.

References

CUE Language Documentation
Helm Documentation
JSON Schema Documentation

Closes #13260

The code changes included in this PR/HIP represent an initial test to evaluate the feasibility and practicality of integrating CUE into Helm. The original test and proof-of-concept (POC) code, developed with contributions from @loewenstein-sap (kudos 🎖), extended the relevant parts of Helm to allow the loading of a values.cue file using CUE as an equivalent alternative to values.yaml. This test was successful, as evidenced in the commit 566d1287.
Subsequently, I prepared another POC (but a little bit different) that aims to completely and accurately replace the outdated JSON schema validator with CUE, ensuring full backward compatibility. The existing values.schema.json is processed directly by CUE, which handles the necessary transformation and validation—tasks previously managed by the github.com/xeipuuv/gojsonschema package:
https://github.com/JuryA/helm/blob/45b6bf43ac653394201c5e26b74a43d9471d55b0/pkg/chartutil/jsonschema.go#L68-L109
Our approach was further validated using several Helm Charts containing values.schema.json, confirming that it works. This justifies the CUE integration with Helm and demonstrates that it has significant benefits.

[!IMPORTANT]
I would like to emphasize that I am not an experienced Go developer, and I apologize in advance for any errors or shortcomings in the code I have published. I would greatly appreciate it if, during the open discussion of this HIP, developers who identify any deficiencies could help address them.
Honestly, I don't feel fully confident handling the entire CUE integration into Helm on my own, but I am willing to attempt it with the guidance of someone more experienced."

Hello @mattfarina,
Could you kindly advise me on the procedure for submitting this HIP proposal?
I appreciate your assistance. 🙂

Apologies, I've been meaning to get to this PR for a while. Thanks for the contrib. I like cue, and agree that there is likely good utility for Helm to support for values management. But the HIP text should be submitted as a PR over here: https://github.com/helm/community/tree/main/hips
For a significant change to Helm. Like supporting cue. A HIP is required. A HIP both lets the details (scope, specification, limitations, etc) be discussed outside of the implementation. And serves as a record of the outcome of those discussions in a referenable document (github PRs do not function well here). Then the PR/implementation can follow from the HIP.

Apologies, I've been meaning to get to this PR for a while. Thanks for the contrib. I like cue, and agree that there is likely good utility for Helm to support for values management. But the HIP text should be submitted as a PR over here: helm/community@main/hips

I have created PR #364 in the Helm Community repository with my HIP proposal. Please review it at your convenience.

Hello @gjenkins8,
I hope you're doing well! 😊
I wanted to check if any further action is required on my part. Additionally, could you advise on the best way to initiate a formal discussion? Should I consider mentioning anyone specific in the new PR? 🤔 I’m unsure whether the proposal for the new HIP has gained enough visibility. 👀
Thank you so much for your guidance! 🙏


DDS game textures wrong colour

When previewing game textures from midnight club 3 dub edition remix with PCSX2-EX the textures are displayed wrongly a character texture is more blue than also red textures are also blue oddly not sure why this happens hopefully could be fixed willing to share the .DDS Files if want to investigate

Same issue as issue #26

This is more of a PCSX2-EX gsdx issue than it is cupscale. It would be better to ask the gsdx plugin author to implement RGBA dumping/converting/loading since Cupscale is behaving as intended.


force: true not working for @import?

Hello there.
I am using node-sass-middleware in a very simple configuration, I just want it to write the file, and Express serves it as static. So far, so good.
It works fine when the output file is not present, but does not regenerate if one of my @imports changes, not even with force:true
Maybe I am using this wring, it's entirely possible.
app.use(sass({
  src: path.join(__dirname, '/views/client/styles'),
  dest: path.join(__dirname, '/views/client/styles'),
  force: true,
  debug: true,
  outputStyle: 'compressed',
  prefix: '/styles'
}));

Any clues as to why this might be, and how I can avoid that?
Cheers,
Martin

Nevermind, I am being a fool today.


Build: Create - show expected build resistance based on coil setup

This will be a bit more advanced as it requires all coil data (queried), which includes:

wireId
coil size
wraps

Also update reisistance based on:

single/dual coil
serial/parallel coils (half/double of resistance)

Resistance at all could be fetched from server or computed on background - this will be much simpler.
Ok, compute resistance of build on background, if not provided, user should update it.

Done.


Page header CSS

In our current implementation of the page, there's an issue with the CSS where the background color doesn't fill the page header section. I'm not able to reproduce this issue in the extension demo.
Is it possible to adjust the CSS to use padding instead of margin on the pfext-page-layout__title element, similar to how spacing is handled for page sections in PF? Or is it possible to use PF components for these page sections instead?

Occurs here

Difference between live QS preview and above usage is that there are two extra wrappers

 and
 AFAIK
 wrapper is breaking the flex layout. See the issue fixed in screenshot above for fix: inherit the display and flex-direction properties from parent.

Fixed in product


Title: Pairs Game using JS

Is your feature request related to a problem? Please describe.
The logic of the game will be:

There are a given number of cards on the table facing down.
The player needs to find all the pairs.

Describe the solution you'd like.
I will use different images to create match cards game and implement it using HTML , CSS and JS.
I can also integrate the time limiting feature as well as restrict the number of guesses to make it more fun and competitive.
Describe alternatives you've considered.
None
Add any other context or screenshots about the feature request here.
No response
@swapnilsparsh  kindly assign the issue to me.

@kyra-09 Thank you for providing your insight but it is not required as of now


How can I get the element that was repositioned?

I would like to access the element that was reordered it in the reordered callback. Currently it seems to only give me all the elements. Is this possible currently?

Right now I don't think this is possible. One option is to keep track of the elements outside of the reorder call then do the diff after the reordering.

Couldn't the element be passed along as a parameter to the callback?

They could as a second parameter I suppose (right now the first parameter is the full set of elements). I don't have any bandwidth to add this feature though (pull requests are welcome).

Closing pending pull request.


Typo in line 225

"programmers," not "progammers"

Not sure if I'm doing this right ... thanks for providing access to the book!

Thanks!


ERR wrong number of arguments for 'get' command

Hello,
I am getting error while retriving striing from redis.
var client = redis.createClient();
client.set(channel, channel_subscribers_count);
client.get(channel, function (err, message) {
console.log('Get value of channel : ' + message);
});
Error ReplyError: ERR wrong number of arguments for 'get' command
I am using redis npm version 2.6.3 and redis server version 3.2.5

Can't reproduce and as a very basic feature, this is very unlikely that the client made a mistake here.
Are you certain the error is coming from this code? Would you be so kind and show the whole part? And are you certain channel is a String? As an addition: you can activate the debug mode and get good stack traces. That way you can check if the error is indeed coming from that line of code.

sc-simple-broker.zip
In this zip file index.js contain code
In subClient.on('subscribe', function (channel, count) {}) function client.get() is working perfectly but when i try to get value in  SCSimpleBroker.prototype.unsubscribeSocket() then i am getting Error ReplyError: ERR wrong number of arguments for 'get' command

As you can see in the arguments of that function it should receive the socket, the channel name that you want to unsubscribe from and optional the callback:
SCSimpleBroker.prototype.unsubscribeSocket = function (socket, channel, callback) { /* ... */ }

If you do not pass the channel name and no callback to the function node_redis is going to think you did not pass any arguments at all and does not warn. Currently it is not really possible to improve this and warn in your case as some people pass an explicit undefined as callback to node_redis and we can't distinguish this case from yours.

Ok Thanks :+1:


java.lang.NullPointerException: Parameter specified as non-null is null: method com.mathroda.domain.model.NewsDetail., parameter description

java.lang.NullPointerException: Parameter specified as non-null is null: method com.mathroda.domain.model.NewsDetail., parameter description

Because description has a chance of getting null, its type needs to be set to nullable.
It is recommended to set everything to be nullable, because it is common for the backend to return null.

@tenSunFree Thanks for reporting the issue. it will be fixed soon 🚀


Define format for resource file

As a user I want to be able to further tail my usage by defining some standard behaviour via a resource file.
Goal: Define a format for the resource file which is understandable by the user, but still easy to parse and secure for the application
DoD:

[ ] Format defined
[ ] PoC file attached to this issue

https://docs.fileformat.com/system/ini/


Make changes to the handling of projector_countdowns used as something

Every meeting has a projector_countdown used as poll-countdown and a projector_countdown used as list_of_speakers-countdown by default. Those projector_countdowns cannot be deleted. Therefore, hide the function to delete them. Also, we should implement a toggle activate/de-activate these projector_countdowns. If they are deactivated, they should be hidden.

Disabling the delete function of these countdowns was already implemented by #771


[Category] update category's status FE

update FE status category

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.


Docker image with gpu support?

My apologies if I just missed this somewhere, but is there a version of the docker that has cuda support? Or is there a way to use the nvidia-docker that I haven't quite grasped?
Thanks!

We don't currently have a pre-built Magenta image with gpu support. However, it should be fairly easy to build one. Just modify the Dockerfile to depend on tensorflow/tensorflow:N.N.N-devel-gpu instead of the version without -gpu, and that should work.


chore(release): v5.6.0

Description

Related Issue

Type of Change

[ ] ✨ feat -- New feature (non-breaking change which adds functionality)
[ ] 🛠 fix -- Bug fix (non-breaking change which fixes an issue)
[ ] ❌ ! -- Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 refactor -- Code refactor
[ ] ✅ ci -- Build configuration change
[ ] 📝 docs -- Documentation
[ ] 🧪 test -- Test
[x] 🗑 chore -- Chore

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 61.73%. Comparing base (9960053) to head (d84e0b0).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #910   +/-   ##
=======================================
  Coverage   61.73%   61.73%
=======================================
  Files          33       33
  Lines        1061     1061
=======================================
  Hits          655      655
  Misses        406      406

Flag
Coverage Δ

win32
59.33% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Fails as it tries to delete the same package twice

Hi,
we tried to apply this action to the siemens/kas repository, but there it fails as it tries to delete the same package twice (in the same run):
[...]
 deleting package id: 183706397 digest: sha256:44871a3c4628a192347a09e39e7e6f5bbdb7e68f95fa8ab3914be3d2301aca7d architecture: amd64
[...]
 deleting package id: 183706397 digest: sha256:44871a3c4628a192347a09e39e7e6f5bbdb7e68f95fa8ab3914be3d2301aca7d architecture: amd64
Error: Package version not found. - https://docs.github.com/rest/packages/packages#delete-package-version-for-an-organization

Full CI output: https://github.com/siemens/kas/actions/runs/10011297384/job/27675632510

I believe it's related to #36. Where multiple mult architecture manifests point to the same underlying image (which didn't not change between builds). I'm working on a fix for that right now.

I've refactored the core logic df5575c, during which I discovered that this issue was that the child images where not being guarded against this scenario where different multi arch images reference the same platform image digest. I believe I've fixed this issue now (currently on main branch). I'll hopefully release later this week.
If there is any way to test this that would be great.
https://github.com/dataaxiom/ghcr-cleanup-action/blob/df5575cf39073931f0ab604166a54216398b1eee/src/main.ts#L258

I've added a test case c75dc10 from #36 to the project CI workflow which tests this issue indirectly.

Hi, I gave it a try in on ghcr.io/fmoessbauer/kas, a registry copy of the official ghcr.io/siemens/kas registry. There, it worked correctly. Job log: https://github.com/fmoessbauer/kas/actions/runs/10347778072/job/28639207637
Note, that the registry copy was created with skopeo sync --all --src docker --dest docker ghcr.io/siemens/kas/kas ghcr.io/fmoessbauer/kas/ and by that only contains artifacts which are referenced. AFAIK, the OCI registry spec does not provide an endpoint to list all artifacts.

That's great to hear. I'm planning to release v1.0.8 later this week to incorporate the latest fixes.

Looking at you test run yesterday, I think it didn't recreate the same setup, likely as the images copied didn't contain references to the same platform images. Which your original log showed.
I've implemented an additional test 7f993f3 which outputs the following action log, which is very close to your original:
https://github.com/dataaxiom/ghcr-cleanup-action/actions/runs/10361019261/job/28681206924
deleting all untagged images
 deleting package id: 257029904 digest: sha256:4b45a528e7526cc38ea35abdcce905c8cbeab1c6d25f78469bd308273cb53edf
 skipping deletion of sha256:f027150874f5e7fd517721736267f83a8eb7b0d40d6d10033f347fbbe2175f18 as it's in use by another image
 skipping deletion of sha256:9e9cf370930547e1eb4683d53348196bbfeecc6243f0c43787c7eb81488f8fd9 as it's in use by another image
 deleting package id: 257029900 digest: sha256:3338b3399136bf4d92641049f93b9b4cb2a7341f40e1fd442184065038aa7a9d application/vnd.in-toto+json
 deleting package id: 257029902 digest: sha256:2b4fdc5b5200ca15f84f4dff65910d18417060314456fdcff10f209b0faf4a14 application/vnd.in-toto+json
 deleting package id: 257029892 digest: sha256:372a4e37c8f870657e7f18e2d18c32ddf66c8056ecdc3ae6de1b70644aaaf1ef
 deleting package id: 257029888 digest: sha256:f027150874f5e7fd517721736267f83a8eb7b0d40d6d10033f347fbbe2175f18 architecture: amd64
 deleting package id: 257029889 digest: sha256:9e9cf370930547e1eb4683d53348196bbfeecc6243f0c43787c7eb81488f8fd9 architecture: arm64
 deleting package id: 257029887 digest: sha256:3b01dd7f8f628ece2819f7319a468efcd5b42e9e0d3ff49245c9f173c6790780 application/vnd.in-toto+json
 deleting package id: 257029890 digest: sha256:192b7f18a2d2045922e699b9a18d3b940ab08bec1e68c6813a124d77d16aad73 application/vnd.in-toto+json
validating multi-architecture/referrers images:
 no errors found
cleanup statistics:
 multi architecture images deleted = 2
 total images deleted = 8

The important change is that it doesn't delete platform images where they are in use elsewhere. Which can happen by buildx cached images apparently.

I've implemented an additional test https://github.com/dataaxiom/ghcr-cleanup-action/commit/7f993f33af9dfb768ec7a9ba00b38e5967b75345 which outputs the following action log, which is very close to your original (except fixed):

Thanks, much appreciated. Recreating these conditions on a registry is challenging, as you cannot easily create a 1 to 1 copy of an existing one. And you also don't want to test new features against production repos (we learned that the hard way when applying the actions/delete-package-versions to kas, which STILL eats multiarch images). That's why we came here in the first place :smile:

That's why I came here also, haha.  I couldn't find an action that didn't eat packages and seamlessly worked in forked repos.
I've released v1.0.8/v1 ff99a6e, so closing this issue.
Thanks for your feedback/help.


Fixing Download button

Just fixing URL for Main Download button, from {{ site.baseurl }}/download/ to {{ site.baseurl }}/downloads/"

Thank you!


review used security roles

Currently, we sometimes use azure security roles which give more permissions than really required. For example, Storage Contributor has rights to read and delete other data, which is not required.
One possible option is to create and use Azure custom roles

Here is one example, not sure though on blobServices actions
{
    "properties": {
        "roleName": "Storage Blob Data Sender",
        "description": "Can only send new data to storage blob, cannot read or delete anything.",
        "assignableScopes": [
            "/subscriptions/6558e738-8188-4771-a5fb-b62f974f971c/resourceGroups/AzmiEnvironment"
        ],
        "permissions": [
            {
                "actions": [
                    "Microsoft.Storage/storageAccounts/blobServices/containers/delete",
                    "Microsoft.Storage/storageAccounts/blobServices/containers/read",
                    "Microsoft.Storage/storageAccounts/blobServices/containers/write",
                    "Microsoft.Storage/storageAccounts/blobServices/generateUserDelegationKey/action"
                ],
                "notActions": [],
                "dataActions": [
                    "Microsoft.Storage/storageAccounts/blobServices/containers/blobs/write"
                ],
                "notDataActions": []
            }
        ]
    }
}


[stdlib] Mark UInt128.Words as frozen

It seems we forgot to label this new struct as @frozen which inhibits the optimizer's ability to specialize use cases of UInt128. Let's mark it as such. This is an ABI break, but hopefully this type isn't being used broadly yet as it's still pretty new and in beta.

@swift-ci please test


demo_imgs.py not working

Thank you for sharing your incredible work. I was trying to run the levocr pretrained model on couple of demo images.
demo_imgs script wasn't running, resolved it by making few changes in the demo_imgs.py
Following are the changes I have made  -
At Line 141 model.module.load_state_dict(torch.load(opt.saved_model, map_location=device),strict=False)
At Line 150  AlignCollate_demo = AlignCollateTest(imgH=opt.imgH, imgW=opt.imgW)
At Line 171   vision_final_pred, _ = converter.encode_levt(vision_preds_str, src_dict, device=device, batch_max_length=pred_vision.size(1))
So,when I ran the script, this was the output

Can you help me to interpret the output or the right way to run the demo script?

It looks like you didn't load the correct model, which one are you using?

@wdp-007 Thank you for the reply. I was using LevOCR-model from the given pretrained models

I re-downloaded the model and it works fine in my environment
python3 demo_imgs.py  --imgH 32 --imgW 128  --max_iter 2 --batch_size 16 --model_dir levocr_model.pth --rgb --th 0.5 --demo_imgs demo_imgs
demo_imgs/5_BALLYS.png
Visual Prediction: bally
Iteration1:Del-Plh-Ins bally-bally<unk>-ballys
Iteration2:Del-Plh-Ins ballys-ballys-ballys
Iteration3:Del-Plh-Ins ballys-ballys-ballys
=======================================
demo_imgs/88_Celebrating.png
Visual Prediction: gelebrating
Iteration1:Del-Plh-Ins elebrating-<unk>elebrating-celebrating
Iteration2:Del-Plh-Ins celebrating-celebrating-celebrating
Iteration3:Del-Plh-Ins celebrating-celebrating-celebrating
=======================================
demo_imgs/100_MANCHESTER.png
Visual Prediction: manchlsste
Iteration1:Del-Plh-Ins manchste-manch<unk>ste<unk>-manchester
Iteration2:Del-Plh-Ins manchester-manchester-manchester
Iteration3:Del-Plh-Ins manchester-manchester-manchester
=======================================
demo_imgs/229_REGISTER.png
Visual Prediction: regester
Iteration1:Del-Plh-Ins regster-reg<unk>ster-register
Iteration2:Del-Plh-Ins register-register-register
Iteration3:Del-Plh-Ins register-register-register
=======================================
demo_imgs/276_MOTORS.png
Visual Prediction: moroes
Iteration1:Del-Plh-Ins mos-mo<unk><unk><unk>s-morors
Iteration2:Del-Plh-Ins moors-mo<unk>ors-motors
Iteration3:Del-Plh-Ins motors-motors-motors

Did you download this model：https://github.com/AlibabaResearch/AdvancedLiterateMachinery/releases/download/v1.0.3-LevOCR-model/levocr_model.pth？

@wdp-007 Yes,I am using the same model. Can you share the updated demo_imgs.py  file,I am getting errors with the current script.
Ex : At Line 171, converter.encode_levt_tgt doesn't exist in the utils file.

Sorry for that, I have updated the demo_imgs.py, you can try again.

Thanks for the updated script,it's working now @wdp-007


AlreadyExistsError: file already exists

Traceback (most recent call last):
File "local_rl_loop.py", line 122, in
rl_loop()
File "local_rl_loop.py", line 108, in rl_loop
main.train(working_dir, gather_dir, next_model_save_file, generation_num=1)
File "/xxx/minigo/main.py", line 110, in train
dual_net.export_model(working_dir, model_save_path)
File "xxx/minigo/dual_net.py", line 450, in export_model
tf.gfile.Copy(filename, destination_path)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/lib/io/file_io.py", line 385, in copy
compat.as_bytes(oldpath), compat.as_bytes(newpath), overwrite, status)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/errors_impl.py", line 473, in exit
c_api.TF_GetCode(self.status.status))
tensorflow.python.framework.errors_impl.AlreadyExistsError: file already exists

This is super strange; are you using the temporary directory still?  what OS is this?

it's look fine last code


Prevent invalid input

Use cases:

applies only to sync validation
opt-in feature to prevent user from typing values that would be invalid
should of course work with custom validators also
iron-input has a corresponding property called prevent-invalid-input

I’d be willing to trade this feature for #16. I have an initial proto for the feature that I’ve been thinking that would cover that use case: https://github.com/vaadin/vaadin-input/tree/proto/validation-feedback

paper-input has allowed-pattern also, which does the same thing. Not exactly sure which one is better, to have a separate property for this, or to use the other validation properties.


conflict with JumpPad++ Failed to connect to server 1.7.10 + Russian language on the client

AdvancedRocketry-1.7.10-1.0.19 (b)
JumpPad++ 2.3.8
Failed to connect to server
/ERROR] [FML/]: Failed to connect to server: there are 2 missing blocks and items /DEBUG] [FML/]: Missing list: [JumpPad++:Launch Pad, JumpPad++:Launch Pad]
I can not go on your server
The bug is shown when 2 modes together, on detachment all ok.
log:
server
https://gist.github.com/3e4a407a8d56c3a2b2e4d8bc52accdb7
client
https://gist.github.com/2efa8923b52e0b69d776bf1dbb240ab6
/ERROR] [FML/]: Failed to connect to server: there are 2 missing blocks and items /DEBUG] [FML/]: Missing list: [JumpPad++:Launch Pad, JumpPad++:Launch Pad]

Are any blocks set to false in the "JumpPad++.cfg" on either the client or server?

@zmaster587 Yes checked. equal.
the weird thing is that JumpPad++, there is no Launch Pad
looking in NEI
now it will show the configs. maybe they matter....

advancedRocketry
https://gist.github.com/71f57b1794f25c1b3fea0d37ad06cc91
JumpPad++.cfg
https://gist.github.com/12f72cd15079bf7f41a8ee87b4e9e45d

What if you remove neiintegration, UnicodeFontFixer, and liteloader?

@zmaster587 left the current 3 mod. not what to change, also swears...
win 10 java 8.131

I just wanted to make sure, because in the logs you gave me before you did have liteloader installed and I was unsure if you tested without it.
(https://gist.github.com/SvEgiiVEteR/3d33e744ec7515797a3fff0cf1fb10d9 line 215)
(https://gist.github.com/SvEgiiVEteR/2efa8923b52e0b69d776bf1dbb240ab6 line 943)
You do appear to be running the same forge version I'm testing with so that seems ok.
Does the issue persist if you reset the world on the server?

@zmaster587 I tried and with LL and without
Now, as I wrote above. Current with 3 modes.
The problem has not disappeared .....

Does the issue persist if you reset the world on the server? If you go into a single player world do the jump pad blocks show up in the creative menu?

I came across different bugs...even the stupid and illogical... but this conflict won it.....
and Yes forgot to write. map every time I delete at server startup.
and Kaie blocks exactly to look for in a single world?
Launch Pad is definitely not..

The launch pad is some internal name, it won't show up in world, I'm just curious if any of the launch blocks are showing up in the creative menu?

checked all the blocks are put into the world
hm...
I noticed one strange
dump
block
JumpPad++:Launch Pad,412,true,null,T145.pads.blocks.BlockPad
JumpPad++:Light Sensor,413,true,null,T145.pads.blocks.BlockLightDetector
item
JumpPad++:Launch Pad,412,true,JumpPad++,T145.pads.blocks.BlockPadItem
JumpPad++:Light Sensor,413,true,JumpPad++,net.minecraft.item.ItemBlock <<<<<<<<<

The light sensor doesn't have variations based on metadata, so it can afford to use the default itemblock, so that seems normal.  Can you check to see if a single player world have all 4 of those entries as well?

@zmaster587
checked they are the same I wrote above about the SP
now checked in MP
all the same
JumpPad++:tile.pad.name,165,true,null,T145.pads.blocks.BlockPad
JumpPad++:Light Sensor,166,true,null,T145.pads.blocks.BlockLightDetector
and
JumpPad++:tile.pad.name,165,true,JumpPad++,T145.pads.blocks.BlockPadItem
JumpPad++:Light Sensor,166,true,JumpPad++,net.minecraft.item.ItemBlock
of course I had to remove your mod. what would go in MP

So the first entry was from single player with AR and Jumppad in at the same time?

@zmaster587
Yes
and the second current with the Jumppad without AR. because MP

while I was testing the SP now. I went with MP.
and now noticed that the MP fell with such a failure..
https://gist.github.com/afe14fdb9ef0fd7f543807c9a114720c
like wine around in the JumpPad

So then the client definatly has the blocks loaded when trying to connect to the server.  Usually if the client has blocks the server does not have, it still lets you join.
Would it be possible to zip an instance of the server and client (mods and config folders should suffice) where this issue occurs?  I tried pulling AR, libvulpes, jumppads++ from curse and had no problems connecting to a server with my setup.

Can you send it anyway so I can be 100% sure all of the files are identical.  It would also allow me to verify no corruption is involved

Also that crash appears to be caused by a cow getting on top of a booster pad, but the program does not check to see if it's a player or not and just assumes it's a player anyway

I suppose if you like I can upload a copy of the instance that I verifed works on my end for you to try if you'd prefer that.

Yes, I can send all. wait)

@zmaster587
I can't understand. just went to the server.
start close again and AGAIN will NOT LET me.
hmm..I think it's in the RUSSIAN language.
1 Russian language is the gateway to the server.
changed to ENG again will not let me.
closed the game, launched the and go - with ENG joined the server.
checked not once.
blame the Russian translation of the game..... if not difficult to check. BUT the game is required to restart.

Let me check something... I wonder if it's some locale problem

@zmaster587 I checked сеqчас 10 times with ENG all is well.
I think your mod is probably trying to move the Launch Pad in this fashion.
why?
because when I created a dump file this
JumpPad++:Launch Pad,412,0,false,Acceleration Pad
JumpPad++:Launch Pad,412,1,false,Annihilation Pad
JumpPad++:Launch Pad,412,2,false,Aqua Pad
JumpPad++:Launch Pad,412,3,false,Ballistic Pad
JumpPad++:Launch Pad,412,4,false,Booster Pad
JumpPad++:Launch Pad,412,5,false,End Pad
JumpPad++:Launch Pad,412,6,false,Impact Landing Pad
JumpPad++:Launch Pad,412,7,false,Incineration Pad
JumpPad++:Launch Pad,412,8,false,Jump Pad
JumpPad++:Launch Pad,412,9,false,Jump Pad+
JumpPad++:Launch Pad,412,10,false,Jump Pad++
JumpPad++:Launch Pad,412,11,false,Portal Pad
JumpPad++:Launch Pad,412,12,false,Regeneration Pad
JumpPad++:Launch Pad,412,13,false,Slow Pad
JumpPad++:Launch Pad,412,14,false,Speed Pad
JumpPad++:Light Sensor,413,0,false,Light Sensor
all Launch Pad was translated as "стартовая площадка" BUT no JumpPad perevodu there should be a Launch Pad
and this game will not let me on the server, I think that these things not because of the translation..

Launch pad is the internal name of the block, if it changes to anything else, the server may not recognise it.  Each mod is also responsible for its own blocks.  AR does not change Jump Pad's block names or vice versa.
I can also confirm I can replicate the issue when changing the language though, so now I can see about nailing an exact cause down.

it is strange that without your mod, no problems with the Russian language in this mod is not shown.
I just know that you also have block Launch Pad. I think it's no coincidence.

I think you may be on to something with that, I have also noticed that with AR removed jumppad++ no longer registers jumppad++:Launch Pad

If it was not difficult to change the name of the block and translate your Launch Pad, then it would be possible to check for the sake of interest what will happen ...

That's what I'm in the process of doing.  Though I still am trying to figure out how a translated name got used at all.  I'm registering it with a hardcoded constant string.

@zmaster587 Well, maybe the reason is not in your mod.
Maybe the second mod tries to translate itself. Looking for translation Launch Pad finds from you and translates.

I tried changing the internal name of the Launch Pad in AR from "pad" to "pad2", now i get this when i try to connect:
[22:12:07] [Netty Client IO #2/INFO] [FML]: Found a missing id from the world JumpPad++:tile.pad.name
[22:12:07] [Netty Client IO #2/INFO] [FML]: Found a missing id from the world JumpPad++:tile.pad.name

I think I need to replace the "Launch Pad" myself with something else, for example LauncherPad

I tried replacing it with "LeWeirdBug" in the lang and I get this:
[22:21:33] [Netty Client IO #2/INFO] [FML]: Found a missing id from the world JumpPad++:LeWeirdBug
[22:21:33] [Netty Client IO #2/INFO] [FML]: Found a missing id from the world JumpPad++:LeWeirdBug

I went and changed the internal name of the block to match the registry name and updated the lang files.
http://<IP_ADDRESS>:8080/job/Advanced Rocketry/job/AdvancedRocketry/job/master/20/changes
Appears to fix the issue.

@zmaster587 I confirm the problem has disappeared)
Thank you very much


Chrome Extension as packaged app apparently no longer works after 2.18 release

User reports in the the Chrome Store Support tab that the packaged app doesn't work on the ChromeBook anymore.
"Chromebook Compatibility
I'm no longer able to read ePubs on a Chromebook with your Chrome App. Very disappointing!
Lynda VanWinkle"
This has not yet been verified on the Readium side.

@ryanackley reports that he tested in on a Chromebook and it worked fine. I will follow up with the user.

Got no response from the user.  Closing as cannot reproduce.


Provide an example list of simulation ids to import

To demonstrate the capabilities of the application to reviewers/users we should create an exported file of simulations ids.
These simulations can have various combinations of files and simulators to represent successful and failed simulations demonstrating the feature of visibility into the failures and the role of specifying the correct simulator for given formats/algorithms.
Examples:

Bionetgen reading smbl
Vcell running unsported algorithm
Same long running Omex archive on multiple simulators

This is the idea for the test suite. For now, I'll copy omex archives I use to test the individual simulators over to the test suite. I'll organize this later this week.
Note, BioNetGen only supports the BNGL language. This will need to be used with different images from the other simulators.

To clarify, for this I meant a simulations.json file exported from the front end. This could be used to demonstrate the features of various cases. A simulation run in which bionetgen was used to run an omex file with SBML would show an example of a failed simulation, with the error logs providing the explanation (unsupported language).
It would be simpler for a reviewer to import a simulations.json file into the front end than to run various different omex files with various options themselves

What's a simulations.json file?

The file that can be exported from https://run.biosimulations.org/simulations

Ah. I see. This is a good idea. I can add a UI element to "Import sample simulations" without the user even having to download + upload a JSON file.
I can create a JSON file within the repo for the same simulations. This can be imported as a JSON module.

That would certainly work better. Is a json file even needed? Is a list of ids sufficient to load in the information?

The file could look like this:
[
  {"id:" "<sim-id-1>"},
  {"id:" "<sim-id-2>"},
  ...
]

The other attributes can be loaded from the API. This might require a slight edit to the simulation service which manages the local storage.

I can take care of making the UI work with this.

PR #1761 adds a UI element with a button to try loading existing simulations.
The list of example simulations are stored here apps/dispatch/src/app/components/simulations/browse/example-simulations.json. The only attribute that's needed is the id attribute. I kept the others so we can easily see what each example corresponds to.
@moraru, to generate the list of example simulations

Clear your list of saved simulations by deleting the local IndexedDB for runBioSimulatons. Can be done from Chome developers tools (Application tab).
Use the submit simulations page to load simulations
Use the export button on the view simulations page to output the list of examples to a file.

Implemented by apps/dispatch/src/app/components/simulations/browse/example-simulations.json


opencv: only use the C++ API

This is a more complete version of https://github.com/IntelRealSense/librealsense/pull/3091 which completely gets rid of the C API usage.
Note that the C++ API used in this PR was already available in OpenCV 2.4.

Ok, the rs-grabcut example was a false alarm (bad calibration and USB2), everything else worked really well, thank you @devernay and @s-trinh!


torchvision.io.read_video mirrors videos of BDD100k and sometimes rotates them 180 degrees

🐛 Bug
The function torchvision.io.read_video(path, start_pt, end_pt, pts_unit='sec') flips videos horizontally and for around half of the videos, also rotates them 180 degrees. The videos in question are from the dataset BDD100k. I believe they have been collected with an iPhone, and are in .mov format.
To Reproduce
Steps to reproduce the behavior:
import torch
import torchvision as tv
import matplotlib.pyplot as plt

vid1, _, _ = tv.io.read_video('data/bdd100k/bdd100k/videos/train/011f8481-b14725fa.mov', 9.5, 10.5, pts_unit='sec')
vid2, _, _ = tv.io.read_video('data/bdd100k/bdd100k/videos/train/019c8305-3a3be0d6.mov', 9.5, 10.5, pts_unit='sec')

# vid1.size() -> torch.Size([31, 1280, 720, 3])

plt.imshow(vid1[0].permute(1, 0, 2))
plt.show() # See first figure
plt.imshow(vid2[0].permute(1, 0, 2))
plt.show() # See second figure

Figures:

Videos when opened with QuickTime:

ffprobe provides:
> ffprobe data/bdd100k/bdd100k/videos/train/011f8481-b14725fa.mov
ffprobe version 4.3.1 Copyright (c) 2007-2020 the FFmpeg developers
  built with Apple clang version 12.0.0 (clang-1<IP_ADDRESS>)
  configuration: --prefix=/usr/local/Cellar/ffmpeg/4.3.1_4 --enable-shared --enable-pthreads --enable-version3 --enable-avresample --cc=clang --host-cflags= --host-ldflags= --enable-ffplay --enable-gnutls --enable-gpl --enable-libaom --enable-libbluray --enable-libdav1d --enable-libmp3lame --enable-libopus --enable-librav1e --enable-librubberband --enable-libsnappy --enable-libsrt --enable-libtesseract --enable-libtheora --enable-libvidstab --enable-libvorbis --enable-libvpx --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxml2 --enable-libxvid --enable-lzma --enable-libfontconfig --enable-libfreetype --enable-frei0r --enable-libass --enable-libopencore-amrnb --enable-libopencore-amrwb --enable-libopenjpeg --enable-librtmp --enable-libspeex --enable-libsoxr --enable-videotoolbox --disable-libjack --disable-indev=jack
  libavutil      56. 51.100 / 56. 51.100
  libavcodec     58. 91.100 / 58. 91.100
  libavformat    58. 45.100 / 58. 45.100
  libavdevice    58. 10.100 / 58. 10.100
  libavfilter     7. 85.100 /  7. 85.100
  libavresample   4.  0.  0 /  4.  0.  0
  libswscale      5.  7.100 /  5.  7.100
  libswresample   3.  7.100 /  3.  7.100
  libpostproc    55.  7.100 / 55.  7.100
Input #0, mov,mp4,m4a,3gp,3g2,mj2, from 'data/bdd100k/bdd100k/videos/train/011f8481-b14725fa.mov':
  Metadata:
    major_brand     : qt
    minor_version   : 512
    compatible_brands: qt
    encoder         : Lavf57.71.100
  Duration: 00:00:40.09, start: 0.000000, bitrate: 3949 kb/s
    Stream #0:0(eng): Video: h264 (High) (avc1 / 0x31637661), yuv420p(tv, smpte170m/bt709/bt709), 720x1280, 3946 kb/s, 30.06 fps, 60 tbr, 19200 tbn, 38400 tbc (default)
    Metadata:
      rotate          : 270
      handler_name    : VideoHandler
      encoder         : H.264
    Side data:
      displaymatrix: rotation of 90.00 degrees

> ffprobe data/bdd100k/bdd100k/videos/train/019c8305-3a3be0d6.mov
ffprobe version 4.3.1 Copyright (c) 2007-2020 the FFmpeg developers
  built with Apple clang version 12.0.0 (clang-1<IP_ADDRESS>)
  configuration: --prefix=/usr/local/Cellar/ffmpeg/4.3.1_4 --enable-shared --enable-pthreads --enable-version3 --enable-avresample --cc=clang --host-cflags= --host-ldflags= --enable-ffplay --enable-gnutls --enable-gpl --enable-libaom --enable-libbluray --enable-libdav1d --enable-libmp3lame --enable-libopus --enable-librav1e --enable-librubberband --enable-libsnappy --enable-libsrt --enable-libtesseract --enable-libtheora --enable-libvidstab --enable-libvorbis --enable-libvpx --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxml2 --enable-libxvid --enable-lzma --enable-libfontconfig --enable-libfreetype --enable-frei0r --enable-libass --enable-libopencore-amrnb --enable-libopencore-amrwb --enable-libopenjpeg --enable-librtmp --enable-libspeex --enable-libsoxr --enable-videotoolbox --disable-libjack --disable-indev=jack
  libavutil      56. 51.100 / 56. 51.100
  libavcodec     58. 91.100 / 58. 91.100
  libavformat    58. 45.100 / 58. 45.100
  libavdevice    58. 10.100 / 58. 10.100
  libavfilter     7. 85.100 /  7. 85.100
  libavresample   4.  0.  0 /  4.  0.  0
  libswscale      5.  7.100 /  5.  7.100
  libswresample   3.  7.100 /  3.  7.100
  libpostproc    55.  7.100 / 55.  7.100
Input #0, mov,mp4,m4a,3gp,3g2,mj2, from 'data/bdd100k/bdd100k/videos/train/019c8305-3a3be0d6.mov':
  Metadata:
    major_brand     : qt
    minor_version   : 512
    compatible_brands: qt
    encoder         : Lavf57.71.100
  Duration: 00:00:40.16, start: 0.000000, bitrate: 4085 kb/s
    Stream #0:0(eng): Video: h264 (High) (avc1 / 0x31637661), yuv420p(tv, smpte170m/bt709/bt709), 720x1280, 4082 kb/s, 30.03 fps, 29.97 tbr, 19200 tbn, 38400 tbc (default)
    Metadata:
      rotate          : 90
      handler_name    : VideoHandler
      encoder         : H.264
    Side data:
      displaymatrix: rotation of -90.00 degrees

Expected behavior
I expect tv.io.read_video to provide images similar to what would be displayed in QuickTime.
Environment

PyTorch / torchvision Version (e.g., 1.0 / 0.4.0): 1.7.1 / 0.8.2
OS (e.g., Linux): macOS Big Sur
How you installed PyTorch / torchvision (conda, pip, source): Conda
Build command you used (if compiling from source): N/A
Python version: 3.8.5
CUDA/cuDNN version: N/A
GPU models and configuration: N/A
Any other relevant information:

Best regards,
Joakim
cc @bjuncek

Hi @joakimjohnander , thank you very much for reporting this - this indeed does look like a weird bug.
Would it be possible for you to upload the two videos somehow for me to check out more thoroughly?
I suspect it's an issue with FFMPEG filter parameters somehow getting mixed up in the decoder, but haven't observed on our videos. Understanding this would be very helpful.
Best,
Bruno

Hey, thanks for swift reply. Does this link work for you?
https://liuonline-my.sharepoint.com/:f:/g/personal/joajo88_ad_liu_se/EheEo5wWNLlGsN9GSCnfRUgBo6u-q0SjezmZwR7kKd_YUg?e=XeyfG9

Yup, perfect. I'll take a look at this tonight :)
Cheers,
Bruno

Hi @joakimjohnander
sorry for the long response, in addition to the holidays, this one is in particular a PAIN to debug, and I still don't have a clear idea on what is happening. In short it seems like these videos are somehow encoded in a different height/width order and FFMPEG (what's behind VideoReader and libraries with similar functionalities) seems to be misinterpreting it.
pyAV and "pure" ffmpeg have the same issue, whilst the openCV interprets the file correctly despite being based on FFMPEG as well.
My hypothesis is that it's a codec/format issue of some sort, but I haven't yet identified the source of confusion.
Sorry this is taking so long, I'll have a discussion with fmassa offline to see if someone can lend me a hand with this.
Cheers,
Bruno

Hi @joakimjohnander
sorry for the long response, in addition to the holidays, this one is in particular a PAIN to debug, and I still don't have a clear idea on what is happening. In short it seems like these videos are somehow encoded in a different height/width order and FFMPEG (what's behind VideoReader and libraries with similar functionalities) seems to be misinterpreting it.
pyAV and "pure" ffmpeg have the same issue, whilst the openCV interprets the file correctly despite being based on FFMPEG as well.
My hypothesis is that it's a codec/format issue of some sort, but I haven't yet identified the source of confusion.
Sorry this is taking so long, I'll have a discussion with fmassa offline to see if someone can lend me a hand with this.
Cheers,
Bruno

Hi - so I've done a bit more research on the subject and it seems that the autorotate flag is built into the FFMPEG binary as a filter graph (source]. This is confirmed by the fact that when I read in the file with openCV that is linked to the the same installation of FFMPEG used in pyAV and torchvision, the file is still rotated as well.
@fmassa we could keep this as something to have in mind if and when we are able to get ffmpeg packaged, but until then I'm afraid there is no easy fix for this other than compiling your own ffmpeg with the autorotate option on.
Note that an old ffmpeg issue ticket sorts this as fixed; but this is definitely not the case by default, with FFMPEG installed from conda-forge.

Hi - so I've done a bit more research on the subject and it seems that the autorotate flag is built into the FFMPEG binary as a filter graph (source]. This is confirmed by the fact that when I read in the file with openCV that is linked to the the same installation of FFMPEG used in pyAV and torchvision, the file is still rotated as well.
@fmassa we could keep this as something to have in mind if and when we are able to get ffmpeg packaged, but until then I'm afraid there is no easy fix for this other than compiling your own ffmpeg with the autorotate option on.
Note that an old ffmpeg issue ticket sorts this as fixed; but this is definitely not the case by default, with FFMPEG installed from conda-forge.


NIFI-9275 Upgrade Jetty from 9.4.43 to 9.4.44

Description of PR
NIFI-9275 Upgrades Jetty from 9.4.43 to 9.4.44 across all modules. Updates include replacing individual dependencies with the Jetty Bill-of-Materials dependency to maintain version consistency and avoid unnecessary declarations. The jetty-runner dependency is not covered in jetty-bom and remains as a separate dependency declaration.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

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

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

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

[X] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[X] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[X] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

Closing this PR after additional review of the Bill-of-Materials change impacting the explicit configuration of provided scope for several Jetty dependencies. Planning to resubmit a new PR without the BoM changes.


Add Catalog Management API Spec

Adds the customer-facing API spec for Discovery Catalog Management V3.
This will be available in the next release, so we figured this is the best place to expose this spec to users. Let me know if this isn't the appropriate place for it.

@joerideg the main branch is protected, can you merge this, thanks!

@akshgpt7 apologies, the branch was not meant to be locked. Feel free to merge it!


Allow plugin name alias

I hit an issue similar to https://github.com/buildkite/agent/issues/520 and solved it with the comment about using ssh://. While this totally works it gets pretty verbose as I use my plugin several times throughout the pipeline yaml.
Does anyone have thoughts on supporting a plugin alias with supporting config or env vars? Something like:
steps:
  - command: "echo whee"
     plugins:
       alias://awesome_plugin:
         ...

With a plugin alias list in the agent config file.
I feel like this is sort of a nit-pick issue but I'm happy to drop in a PR for it. I mostly wanted to take a quick pulse check on the style/appetite for a change like this before I wrote any code.

This could also be interesting to handle plugin renames.
For example, we have created a Buildkite plugin internally to share a lot of commands that we use in many of our projects and pipelines. As the plugin grew in functionality, the original name we picked for the plugin does not reflect its purpose anymore, so we'd like to rename it… but many of our pipelines already reference it with the current name, so that may prove tricky to handle the transition (including if you take into consideration that our many projects also have multiple active branches)
With the ability to declare plugin aliases in the agents, we could consider creating an alias for the current name, then deploying it and update our pipelines to reference the plugin via its alias… and then when we're ready, rename the plugin repo and update the alias in the agents in one swoop, which would apply the change to all pipelines and client repos and branches at once.


How the way best custom template dspace 7?

Hi,  today I have problem.
I don't have not idea, how to custom template dspace7?
becuease I read some source recommend "Don't custom in the core template".
I want to clone original template to custom theme but
the some HTML file much use original TS file,
If I copy TS original file to custom, I find configurate is difficult and so many.
Whold you have idea how to custom template dspace 7?
Thank you very much.

Please see the user interface customization documentation at https://wiki.lyrasis.org/display/DSDOC7x/User+Interface+Customization
If you have more specific questions, feel free to ask them via one of our official support channels: https://wiki.lyrasis.org/display/DSPACE/Support  GitHub issues is only for suspected bugs and new features.


bump axe-core-api gem

It's been several months since we bumped this gem.
The latest axe-core-api update pulls in more recent axe-core which will ensure better coverage.

Got this error...

NoMethodError: undefined method `execute_script' for #Capybara::Cuprite::Browser:0x00005577a0b77ee8

saving for a future time. I dont have capacity for this.


Turbo frame contains busy and complete when navigating history

I've noticed that when using 7.3.0, my  which I navigate inside using data-turbo-action="advance" incorrectly shows the busy attribute when going back in history.
e.g, initial load is a regular frame, navigate (css displays a loading style due to turbo-frame[busy], frame loads (all good so far). Navigate again, looks fine. Now go back and the frame is stuck on busy.
Is anyone else experiencing this issue?

Sorry, normal navigation is actually fine. The issue occurs when I have a form with an action=GET
<form method="get" action="/my-path" data-turbo-frame="my-frame">

@dhh Any reason why the issue has been closed? I've got the same problem that the history back stuff breaks when using a form with method GET.


Move-to-Spam Button in email view

Please add a button left to the trash button in the email view, which moves the current email into the spam folder and views the next email.

See #5300

Thank you so much!


Appname camelCase with numbers

I'm building a plugin and brand called "bed66" but wpack.io fails to start because plugin is not considered in camelcase.
Any advice or guidance?

✖ could not start server.
please review the following errors
❯  appName must be in camelCase in project config.
❯  Currently bed66, try bed66

Hi,
This is a restriction in naming things. Simply try bedSixSix or bedSixtySix

Yes, that is what I did to get around the restriction.  Thank you.


Provide delivery/live stage redundancy

Creating a scalable delivery stage automatically by simply providing the number of Replication Live Servers and minimum and maximum number of Content Application Engines.
This should bring up the desired number of RLS, and create a suitable deployment of live CAEs and an appropriately parameterized Horizontal Pod Autoscaler.

A fixed number of Live CAEs can be configured by setting with.delivery.minCae. The number of RLS, and an HPA for the Live CAEs are still to be implemented.

e5782eb7dbfa68b4d7847c8cbde41b3ce5357a68 implements creation of RLSes. WIth this, a fully redundant setup is possible (for example, delivery.rls=2 and delivery.minCae=2).

Additional scaling to be discussed in #90.


Subclass pyviz_comms extension and add delete hook

This PR subclasses the extension from pyviz_comms ensuring that independently of which extension you load all plots are cleaned up correctly when a cell is deleted, re-executed or cleared.
Should not be merged until pyviz_comms 0.7 containing the extension has been released.

Depends on https://github.com/pyviz/pyviz_comms/pull/21

The push build has passed. Merging.


tests (ticdc): remove ticdc github action ci

What problem does this PR solve?

Issue Number: close #9460
What is changed and how it works?
The test cases in Github Action Ci of TiCDC are already covered in tests/integration_tests, so these unstable CI pipelines can just be removed.
Check List
Tests

No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
None

Please state the reasons for removing such tests in the pr body.

Perhaps the relevant test files should be deleted? Ref: https://github.com/pingcap/tiflow/tree/master/tests/mq_protocol_tests

/test all

/check-issue-triage-complete

/test cdc-integration-mysql-test

/retest


DecimalField trailing zeros fix

Working on a fix for this issue.

I am closing this PR. I was blaming django-auditlog for the problem that you can see here reproducible by the tests. But it appears that the problem is in django itself. It seems that django is changing the Decimal value it stores when you save again.

I am closing this PR. I was blaming django-auditlog for the problem that you can see here reproducible by the tests. But it appears that the problem is in django itself. It seems that django is changing the Decimal value it stores when you save again.

I don't know if it's a django problem, when saving an integer value, for example, in the database it is saved with decimal places.
Tavelz, the auditlog should treat decimal values and not differentiate between equal values, 0 should be equal to 0.00, I believe.


"muti-user": is it wanted?

https://github.com/serverpod/serverpod/blob/bee2cfe19bac000f3f09517d4a66b439f83e8cc3/templates/serverpod_templates/projectname_server_upgrade/deploy/aws/scripts/install_dependencies#L36
Seems a pretty important typo

This typo also exists in the documentation and should be fixed there simultaneously.


PoS RemoteNodeIndexer

Current dependencies on/for this PR:

feature/proof-of-stake

PR #830

PR #768

PR #857

PR #858   👈

PR #792

PR #793

This stack of pull requests is managed by Graphite.


Add GitHub Action action.yml

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#40  👈
#39
#38
#37
#36
#35
#34
#33
#32
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @JackPlowman and the rest of your teammates on  Graphite

Merge activity

Sep 26, 3:18 AM EDT: @JackPlowman merged this pull request with Graphite.


Fix windows addon labels

Fixes bug described in https://github.com/prometheus-operator/kube-prometheus/issues/1099

Thank you! :100:


[Specification] Windows project template

Abstract
Develop a Visual Studio 2017 solution file that will enable a user to download a zip file of content (including a solution file), open the solution, hit run, and have the code Just Work (tm).
Full description
Python.net contains an example project that shows how to embed a Python interpreter into a C# project. However, this project doesn't work "out of the box" - it requires a bunch of configuration before it will work. It is part of a larger project, so you need to modify the solution file to set the starting project; then, when you do run it, you get errors because Python27.dll can't be found - which is problematic if you're trying to use Python 3.5.
I want to be able to give someone a zip file that contains the solution file and source code, have them open the file, run it, and get a useful executable.
The deliverable is the zip file, containing a Visual Studio 2017 solution file, the C# source code from the Python.net console demo, and any other support files necessary. I imagine this will include:
A copy of Python. The zip file should contain the official embedded python zip file in a known location relative to the solution file. However, you cannot depend on a system installation of Python (or anything else other than Visual Studio).
An installation of Python.net. Ideally, this will be in a directory other than the site-packages provided by the embedded Python install
When compiled, the executable should have a custom icon; any icon will do for sample purposes, but it should also be available in the zip file.
The ideal layout for the final zip file would be:

Windows

MyProject.sln
MyProject.cs

MyProject

Resources

myproject.ico

Python

(contents of embedded Python zip file)

app_packages

(installed version of python.net)

app

MyProject

__init__.py
app.py

The solution file should output "MyProject.exe" in the MyProject directory.
Bonus points
For bonus points:

The default source from the console demo of Python.net will require a command line argument; modify the sample code to invoke app.py with the contents of app and app_packages in the PYTHONPATH for the purposes of execution.

The output app should be a gui app, not a console app. When MyProject.exe is executed, a black console window should not be displayed.

Add a target to the solution file that produces an MSI installer with contents of the "MyProject" directory, including the executable, embedded python, app_packages and app.

I replied on Twitter:
Denis Akhiyarov (@denfromufa) Tweeted:
@PyBeeWare The pythonnet.sln is actually configured by setup.py during build step. I recommend this troubleshooting wiki: https://t.co/K7oUdIsAQp https://twitter.com/denfromufa/status/852717207247368193?s=17

You can grab nuget packages for pythonnet and Python to embed them in a C# project. This hasn't been documented or tested yet:
https://github.com/Microsoft/PTVS/issues/2304

I can't help on the MSi installer (perhaps pynsist or constructor?). But here is how to hide console in c#:
http://stackoverflow.com/a/3571628/2230844

Maybe wix for the MSI installer?

@NathanW2 You can certainly use wix for building full-blown MSI installers, like the original CPython installer:
https://github.com/python/cpython/tree/master/Tools/msi
But this requires some knowledge of wix. That's why I suggested pynsist or constructor, both based on nsis.
Anyway the installer is secondary, the primary issue here is how to include cpython in pythonnet solution and build it into stand-alone application.
I also did not understand why @freakboy3742 picked VS 2017 - it still has a lot of churn with Python. I would recommend waiting for next update and use VS 2015 in the meantime.
But main issue I have is WHY do you need to embed CPython in C#?

I didn't pick NSIS because I want a proper MS Installer, not just an installer-like tool. One of the guiding principles of the BeeWare suite is to build tools that are indistinguishable from native tools. Part of that is an installation process that is as native as possible as well. From everything I've read, NSIS installers don't meet that requirement, because they don't fit into the transactional installation process Windows provides.

I picked VS2017 because that's what you get when you go to the Microsoft website. If there's a better option, I'm all ears.

What I'm trying to do is provide an artefact on disk that can be used to start a Toga application. cx_freeze and pyinstaller aren't options because they depend on the existence of system Python installation. I want a completely isolated standalone installer that works when I just give you just the application installer. The approach I want to use is much the same as I'm using on iOS - a bare bones compiled executable that is a bootstrap to start the Python interpreter and start the app. This lets you attach an icon to the executable, and if you ship the embedded Python library, you've got an isolated, standalone environment.

On top of all of this, Windows isn't my platform of choice, I haven't touched Visual Studio in almost 15 years, and haven't much C# at all - none of it on Windows. I've tried to pull apart the "console" example, but it's intertwined with a bunch of other stuff in the PythonNet project, isn't easily isolated, and obviously requires some sort of pre-configuration (which AFAICT isn't documented anywhere). I tried the "just drop the PythonNet library into a new project" approach, and then spent the next couple of hours debugging an error that turned out to be related to the fact that the default project targets MSIL, but PythonNet is targeted at x64 (which I can only assume is Windows-speak for x86-64). I'm up against a time deadline, so I was hoping for someone to crack this nut as an interesting but simple problem if you are a domain expert, rather than having to spend a couple of weeks learning how to drive Visual Studio again.
Filling in some extra context - eventually, after a couple of days getting no progress, I gave up on the idea of using an executable and switched to trying to use a batch/command script starting the python executable that ships with the embedded Python installer. However, that failed because the embedded interpreter doesn't honour PYTHONPATH (by design). Hence my question about whether there was any way to pass in PYTHONPATH.
I have found a solution - using Powershell to generate a shortcut link - but AFAICT, it's not relocatable, and the icon has the shortcut overlay on it. It's not ideal, but it "works", for some definition of work. But again - Windows isn't my platform of choice, and I'm up against a deadline, so this will have to do for now.
If someone wants to provide an alternative - like what I've described in this ticket, or anything better - I'll happily integrate it.

If I get some time I can help you with wix at least. I have just done it
for qgis to make MSI packages.
On Wed, 19 Apr 2017, 4:47 PM Russell Keith-Magee<EMAIL_ADDRESS>wrote:

I didn't pick NSIS because I want a proper MS Installer, not just an
installer-like tool. One of the guiding principles of the BeeWare suite is
to build tools that are indistinguishable from native tools. Part of that
is an installation process that is as native as possible as well. From
everything I've read, NSIS installers don't meet that requirement, because
they don't fit into the transactional installation process Windows provides.
2.
I picked VS2017 because that's what you get when you go to the
Microsoft website. If there's a better option, I'm all ears.
3.
What I'm trying to do is provide an artefact on disk that can be used
to start a Toga application. cx_freeze and pyinstaller aren't options
because they depend on the existence of system Python installation. I want
a completely isolated standalone installer that works when I just give you
just the application installer. The approach I want to use is much
the same as I'm using on iOS - a bare bones compiled executable that is a
bootstrap to start the Python interpreter and start the app. This lets you
attach an icon to the executable, and if you ship the embedded Python
library, you've got an isolated, standalone environment.
On top of all of this, Windows isn't my platform of choice, I haven't
touched Visual Studio in almost 15 years, and haven't much C# at all - none
of it on Windows. I've tried to pull apart the "console" example, but it's
intertwined with a bunch of other stuff in the PythonNet project, isn't
easily isolated, and obviously requires some sort of pre-configuration
(which AFAICT isn't documented anywhere). I tried the "just drop the
PythonNet library into a new project" approach, and then spent the next
couple of hours debugging an error that turned out to be related to the
fact that the default project targets MSIL, but PythonNet is targeted at
x64 (which I can only assume is Windows-speak for x86-64). I'm up against a
time deadline, so I was hoping for someone to crack this nut as an
interesting but simple problem if you are a domain expert, rather than
having to spend a couple of weeks learning how to drive Visual Studio again.
Filling in some extra context - eventually, after a couple of days getting
no progress, I gave up on the idea of using an executable and switched to
trying to use a batch/command script starting the python executable that
ships with the embedded Python installer. However, that failed because the
embedded interpreter doesn't honour PYTHONPATH (by design). Hence my
question about whether there was any way to pass in PYTHONPATH.
I have found a solution - using Powershell to generate a shortcut link

but AFAICT, it's not relocatable, and the icon has the shortcut overlay
on it. It's not ideal, but it "works", for some definition of work. But
again - Windows isn't my platform of choice, and I'm up against a deadline,
so this will have to do for now.

If someone wants to provide an alternative - like what I've described in
this ticket, or anything better - I'll happily integrate it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pybee/briefcase/issues/30#issuecomment-295125176, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAXS3INz6C_7KATdt9aBfxQj1Z-LbgG6ks5rxa4AgaJpZM4M9Q-A
.

cx_freeze and pyinstaller aren't options because they depend on the
existence of system Python installation

What do you mean by this.  They are only required on the dev machine. Once
you use cx_freeze you should have a standalone app.  I have used py2exe in
the past but it's the same idea.
On Wed, Apr 19, 2017 at 4:48 PM, Nathan Woodrow<EMAIL_ADDRESS>wrote:

If I get some time I can help you with wix at least. I have just done it
for qgis to make MSI packages.
On Wed, 19 Apr 2017, 4:47 PM Russell Keith-Magee<EMAIL_ADDRESS>wrote:

I didn't pick NSIS because I want a proper MS Installer, not just an
installer-like tool. One of the guiding principles of the BeeWare suite is
to build tools that are indistinguishable from native tools. Part of that
is an installation process that is as native as possible as well. From
everything I've read, NSIS installers don't meet that requirement, because
they don't fit into the transactional installation process Windows provides.
2.
I picked VS2017 because that's what you get when you go to the
Microsoft website. If there's a better option, I'm all ears.
3.
What I'm trying to do is provide an artefact on disk that can be used
to start a Toga application. cx_freeze and pyinstaller aren't options
because they depend on the existence of system Python installation. I want
a completely isolated standalone installer that works when I just give you
just the application installer. The approach I want to use is much
the same as I'm using on iOS - a bare bones compiled executable that is a
bootstrap to start the Python interpreter and start the app. This lets you
attach an icon to the executable, and if you ship the embedded Python
library, you've got an isolated, standalone environment.
On top of all of this, Windows isn't my platform of choice, I haven't
touched Visual Studio in almost 15 years, and haven't much C# at all - none
of it on Windows. I've tried to pull apart the "console" example, but it's
intertwined with a bunch of other stuff in the PythonNet project, isn't
easily isolated, and obviously requires some sort of pre-configuration
(which AFAICT isn't documented anywhere). I tried the "just drop the
PythonNet library into a new project" approach, and then spent the next
couple of hours debugging an error that turned out to be related to the
fact that the default project targets MSIL, but PythonNet is targeted at
x64 (which I can only assume is Windows-speak for x86-64). I'm up against a
time deadline, so I was hoping for someone to crack this nut as an
interesting but simple problem if you are a domain expert, rather than
having to spend a couple of weeks learning how to drive Visual Studio again.
Filling in some extra context - eventually, after a couple of days
getting no progress, I gave up on the idea of using an executable and
switched to trying to use a batch/command script starting the python
executable that ships with the embedded Python installer. However, that
failed because the embedded interpreter doesn't honour PYTHONPATH (by
design). Hence my question about whether there was any way to pass in
PYTHONPATH.
I have found a solution - using Powershell to generate a shortcut link

but AFAICT, it's not relocatable, and the icon has the shortcut overlay
on it. It's not ideal, but it "works", for some definition of work. But
again - Windows isn't my platform of choice, and I'm up against a deadline,
so this will have to do for now.

If someone wants to provide an alternative - like what I've described in
this ticket, or anything better - I'll happily integrate it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pybee/briefcase/issues/30#issuecomment-295125176,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXS3INz6C_7KATdt9aBfxQj1Z-LbgG6ks5rxa4AgaJpZM4M9Q-A
.

@NathanW2 I must have been doing something wrong then, because I'm fairly sure from my experimentation that cx_freeze ended up putting files in my system (well, venv) Python install.

Shouldn't do from what I know. It should just give you a standalone exe
with no need for anything else.
On Wed, 19 Apr 2017, 6:03 PM Russell Keith-Magee<EMAIL_ADDRESS>wrote:

@NathanW2 https://github.com/NathanW2 I must have been doing something
wrong then, because I'm fairly sure from my experimentation that cx_freeze
ended up putting files in my system (well, venv) Python install.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pybee/briefcase/issues/30#issuecomment-295151563, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAXS3PcHwp3jCz4a2Si_ttJqTNZ1kgeeks5rxb_dgaJpZM4M9Q-A
.

This was resolved and released in v0.2.0


Upgrade Containerd in main from 1.7.7 to 1.7.8

Subject says it all.

This is an auto-generated PR. Will approve and merge this one when the PR for the generator script https://github.com/flatcar/scripts/pull/1330 is approved.


Add definitions

Basically definitions objects holds data types that are consumed and
produced by operations.
This allows definitions to be defined using metadata like this:
Metadata = #{
   get ....
   post ....
   definitions => #{
       vendor => #{
          type => <<"object">>,
          required => [ <<"id">>, <<"name">>],
          properties => #{
            <<"id">> => #{
                type => <<"integer">>,
                format => <<"int64">>
               },
            <<"name">> => #{
                type => <<"string">>
               }
           }
         }
    }}

So get, post can use to references schema like this:
parameters => [
    #{name => <<"vendor">>,
      in => <<"body">>,
      description => <<"Vendor to add">>,
      required => true,
      schema => #{
          <<"$ref">> => <<"#/definitions/newVendor">>
      }
    }
],
responses => #{
              <<"200">> => #{description => <<"Succesful operation">>,
                             schema => #{<<"$ref">> => <<"#/definitions/vendor">>}}

Hi,
This PR is the general idea. Am I in the right direction?

Hi @joaohf. I wouldn't tie those definitions to a single handler.
In my mind they should be somewhat global.
Therefore, they should be in the application environment. Something like
Definitions = application:get_env(cowboy_swagger, definitions, [])

Which is what we're doing today with global_spec. As a matter of fact, can't they be included there already?

Hi,
Ok, Makes sense.
So will use application environment.
Thanks.


SP5 Heatmap: How to calculate Heatmap from the minimum data value

Question: How to calculate Heatmap not from 0, but from the minimum data value?
The problem is that the values that come to me may not be distributed from 0, but, for example, in the interval between 80-100% (where 100% is the maximum data value). I also use a Colorbar based on this Heatmap, which would also be a good idea to start with a minimum value so as not to confuse the end user. How to do it?
Next, I will provide screenshots of graphs (two different sets of values) from Excel with my data distribution (for ease of perception, I built a graph reflecting what percentage the number is of the maximum) and with those obtained after the Heatmap + ColorBar pair.
Incorrect:

Please ignore the green bar under the "red graph", this is a piece of my form on which I place the Heatmap and ColorBar
Correct:

A suggestion:
For this type of heat map, the data can be represented in logarithmic coordinates or dB. We find the maximum value of the data and set a number slightly larger than that value as the maximum value of the heat map.
If the data is represented by logarithm, we can set a reasonable Decade for the heat map to ensure that the minimum value of the chart can be distinguished.
Similarly, if the data is represented in dB, we can set the range of dB.
The following figure is an example represented by logarithm.with a maximum value of 2.509 and a minimum value of 35.759e-6. The maximum value of the heat map is set to 2.6, with a range of 5 Decades, so the minimum value that can be represented is 26e-6.

The key code is:
https://github.com/ScottPlot/ScottPlot/blob/44ee711df9f646ee82b124e1f98b8a4c5f597fc0/src/ScottPlot5/ScottPlot5/Plottables/Heatmap.cs#L332
use sample code
https://scottplot.net/cookbook/5.0/Heatmap/HeatmapColormap/
Original sample's heatmap range [0, 228]

If a new range is forcibly specified [100, 500]
public Range GetRange() => new(100, 500);

If a new range is forcibly specified [100, 150]
public Range GetRange() => new(100, 150);

If I understand the code correctly, I think Heatmap already spans from min to max, not zero to max.  That being said, I will submit a PR that adds an arbitrary range you can force for rescaling the colormap.


[Feature]: Make a unstable latest docker image

🚀 The feature, motivation and pitch
I'm trying to test Lora with quantization using BnB and I'm using Docker to run vLLM. However, I can't find a docker image, on docker hub, with a tag that matches the 'latest' version. The latest tag of the docker images correspond til the stable version - not latest.
So have you considered makes a tag for the 'latest' (unstable) version for Docker?
Alternatives
No response
Additional context
No response

+1


netreceive and netsend errors on Windows

I'm testing osc.receive and osc.send objects (within PlugData), but they don't work since the underlying netreceive and netsend objects throw the following error when trying to connect to any (valid) port:
netreceive: bad host or port? Either the application has not called WSAStartup, or WSAStartup failed.  (10093)
10093 is WSANOTINITIALISED, which according to Mcirosoft documention means:

Successful WSAStartup not yet performed.
Either the application has not called WSAStartup or WSAStartup failed. The application may be accessing a socket that the current active task does not own (that is, trying to share a socket between tasks), or WSACleanup has been called too many times.

I think this should be moved to the plugdata repo, since I see that netreceive and netsend are provided by pure-data itself.


Fixes the OpenDyslexic extension readme

About
Came across this readme today and wanted to help with a few fixes
Impacted Areas in Application
The application should not be impacted.
Only changes occurred within the README.

Fixed the Chrome extension link
Fixed the Firefox add on link
Fixed the anchor tag href URL to go to the new .org website
Fixed the path for the logo

Hey @olitreadwell
Looks great, thank you for the pull request.
Will get an OpenDyslexic on the Edge store this weekend.


Gateway tests reorg and cleanup

Simplify gateway test structure by removing gateway/tests crate. Instead, this creates a gateway/ln-gateway/tests folder to host gatewayd functional tests
closes #1783

Codecov Report
Base: 63.26% // Head: 63.28% // Increases project coverage by +0.02% :tada:

Coverage data is based on head (913e845) compared to base (976a520).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1703      +/-   ##
==========================================
+ Coverage   63.26%   63.28%   +0.02%
==========================================
  Files         124      124
  Lines       24703    24703
==========================================
+ Hits        15629    15634       +5
+ Misses       9074     9069       -5

Impacted Files
Coverage Δ

fedimint-server/src/multiplexed.rs
88.75% <0.00%> (-2.50%)
:arrow_down:

fedimint-server/src/net/connect.rs
94.40% <0.00%> (-0.25%)
:arrow_down:

modules/fedimint-wallet/src/lib.rs
79.32% <0.00%> (-0.24%)
:arrow_down:

fedimint-server/src/net/peers.rs
89.42% <0.00%> (ø)

fedimint-server/src/lib.rs
76.48% <0.00%> (+0.25%)
:arrow_up:

fedimint-server/src/consensus/mod.rs
80.95% <0.00%> (+0.79%)
:arrow_up:

fedimint-core/src/core.rs
95.51% <0.00%> (+1.24%)
:arrow_up:

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

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

I'll go ahead and merge this. We'll fix the typos in the next PR :)


WIP: Limit the greenboot package version to 0.15.4 until bootloader issue is resolved

See THEEDGE-4006

/test microshift-metal-tests

/close


az cli failing on webapp deployment

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp deployment user set
Errors:
No module named 'decorator'
Traceback (most recent call last):
python3/dist-packages/fabric/connection.py, ln 5, in <module>
    from invoke.vendor.six import StringIO
ModuleNotFoundError: No module named 'invoke.vendor.six'

...

python3/dist-packages/fabric/connection.py, ln 10, in <module>
    from decorator import decorator
ModuleNotFoundError: No module named 'decorator'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az webapp deployment user set --user-name {} --password {}

Expected Behavior
Environment Summary
Linux-5.4.0-33-generic-x86_64-with-glibc2.29
Python 3.8.2
Shell: bash

azure-cli 2.0.81

Extensions:
azure-devops 0.17.0

Additional Context

Fresh installation of Ubuntu 20.04 LTS. I ran az login, when azure cli was not installed.
pi@copper:~/git/azure-mol-samples-2nd-ed/03/prod$ az

Command 'az' not found, but can be installed with:

sudo apt install azure-cli

After logging in successfully with 'az login', the az webapp deployment command was run, which is the described in the initial issue.
I was able to resolve my issue by uninstalling the previous az installation, and installing using the documented azure cli instructions.
pi@copper:~/git/azure-mol-samples-2nd-ed/03/prod$ sudo apt-get remove azure-cli

curl -sL https://aka.ms/InstallAzureCLIDeb | sudo bash

webapp

duplicate with #13660

hi @asnogordo is your az cli  installed via pypi? If yes, pls see https://github.com/Azure/azure-cli/issues/13343#issuecomment-635811624 for reference.

@asnogordo yes, following the doc is the right way. The package comes with Ubuntu repo is old and has some issues.

#14011


Can't read log output on dark theme Chrome or FireFox because the font color isn't high contrast

Operating System: Darwin 17.4.0
Node Version: 9.5.0
NPM Version 5.6.0
webpack Version 4.1.1
webpack-hot-client Version: 1.3.0

[x] This is a bug
[ ] This is a feature request
[ ] This is a modification request

Code

See https://github.com/joshunger/bugrepro
Expected Behavior
A user can read the logs.
Actual Behavior
Can't read the logs because of the text color.

For Bugs; How can we reproduce the behavior?
Clone https://github.com/joshunger/bugrepro
cd bugrepro
yarn
yarn start

Open browser.
Change devtools to Dark Theme in Chrome or FireFox.
Load URL.
For Features; What is the motivation and/or use-case for the feature?

@nchevobbe

@shellscape assign to me and I'll send you a PR.  I just need the info from Chrome folks.

@garbee can you help on the Chrome side?

@shellscape can I just remove?  Why was it there originally?

I'm not sure what Chrome/FF in particular have to do here. This is about the color contrast as styled by the application, not something done by default by the browsers.
If there is a specific console statement that is difficult to read by default, then please file bugs on the appropriate tool's issue tracker. That would mean the problem is reproducible by simply running a certain console.log statement for example manually in the terminal.

With the way that css works in the browser consoles, you need to reset the color back to something. So we're going to have to figure out how to detect when contrast is needed, or what the system css variables are on each. Detecting high contrast situations would probably be preferable, though.

With JS in Chrome you can check chrome.devtools.panels.themeName for either default or dark options to know which mode the user has selected. This however, only works with the built in themes, not any custom themes users may have enabled using the experimental flag for custom ones. I think with Firefox you just check for the dark mode class on the html or body node.

OK so that's two relatively reasonable detection methods. We can then consider custom themes to be edge-cases, and users can request that similar "dark mode" classes be added to the root node(s) by the theme authors.

Custom themes can't modify the node classes. Custom themes are an edge-case where we literally can't directly support it. Unless there is a set css var that is looked for first which is never set internally. Then the internal variable is the fallback. i.e. color: var(--global-check, --internal-color) This would look for the --global-check variable initially, and if set use that. Which a custom theme author would need to know about and write the variable for. If it is not set, then the --internal-color variable is used which is our set var in this system.

Thanks for the help @Garbee happy 🌮 Thursday @shellscape if you want to assign to me I'll add it to my PR list.

Will do.

I'll take this next week. 👍

@joshunger any action on this? if it's on the back burner, we'd rather close and get it off the books, and re-open when it can be circled back to.

Feel free to close it.


Add data-requirements endpoint

Summary
This PR adds a /Measure/$data-requirements endpoint, to use the new fqm-execution calculateDataRequirements method.
The endpoint takes in a measure object, which is a Bundle resource containing the Measure resource and all the Library resources.
Since patient isn't required for this call, I created a new input type DataRequirementsBody in src/types/server_types that only accepts a measure bundle.
Testing
POST to /Measure/$data-requirements and /Measure/%24data-requirements (the url-encoded version) with a measure bundle as a parameter, and make sure you get back a Library object. Compare the Library object to one that came directly from fqm-execution.

NOTE: Draft until after projecttacoma/fqm-execution#70 gets merged

ready to roll?

ready to roll?

Maybe wait for mine with the npm audit resolutions?

Ah true then we can include both in a new fqm-exec release. NIce

We can tag an fqm-exec release after Hoss's pr gets merged, then I can update this bad boi

Alright should be good to update to<EMAIL_ADDRESS>

Updated. Also bumped the minor version.

Actual functionality looks good to me, the README is a bit weird now though:

So I really hate to be that guy, but....


WIP: Add commit sha to filepath api

This is an attempt to give a solution to #12840
I'm not really satisfied with the tests yet, as the repo that is used in the test has only one commit, it's not really possible to check that the returned commit is correct.

This looks OK to me. Just would like you to confirm that this matches the github api.

This looks OK to me. Just would like you to confirm that this matches the github api.
It appears that this does not match the Github API.
If we want the same functionality as the github api, we should probably go this way to solve the same problem:
https://github.com/go-gitea/gitea/issues/12840#issuecomment-700221496

Would you prefer that approach?

in general, I think we would better return 'lastcommit' rather than commit, as this is not obvious very obvious in some cases.

This is in progress for a while now, I'm closing it to avoid stale pull requests. Please reopen when it's ready for review.


Ambigous call on QueryContainerDescriptor.Wildcard(field, value)

NEST/Elasticsearch.Net version: 5.4
Elasticsearch version: 5.5
Description of the problem including expected versus actual behavior: After upgrading to NEST 5.4 (from 5.2.2) I get an exception on QueryContainerDescriptor.Wildcard(field, value).
The call is ambiguous between the following methods or properties: 'QueryContainerDescriptor<T>.Wildcard(Expression<Func<T, object>>, string, double?, RewriteMultiTerm?, string)' and 'QueryContainerDescriptor<T>.Wildcard(Expression<Func<T, object>>, string, double?, MultiTermQueryRewrite, string)'
I use this feature to do Wildcard searches, because the alternative Term does not work like a 'like' search.
var query = value.Contains("*") ? (Func<QueryContainerDescriptor<T>, QueryContainer>)
                            (bs => bs.Wildcard(propConfig.KeySelector, value)) : // wildcard search
                            (bs => bs.Term(propConfig.KeySelector, value)); // default term search

Steps to reproduce:

call .Wildcard on a QueryContainerDescriptor with 2 parameters: field, value

Misc:
I also noticed that .Wildcard is new deprecated. What is the recommended method to replace this? I can't find this in any release notes.

Hey @TSSchoof, that particular method call for WildCard() was deprecated in NEST 5.3.0 because of #2688. With the deprecated warning, do you see the message "Use overload that accepts MultiTermQueryRewrite"?
Looks like we didn't have a test to catch the method call ambiguity 😢  It's easily resolved by specifying rewrite though
(bs => bs.Wildcard(propConfig.KeySelector, value, rewrite:(MultiTermQueryRewrite)null))

The deprecated methods will be removed in the next major release.

Thanks! I don't see that message though. It's probably a setting in Visual Studio somewhere.

That's no good! Do you see it in the build output?

Well when I'm not using the rewrite the build fails and I just get the exception.

The override is not deprecated so that does not show any warnings.

No worries, thanks for looking. I'm sure Visual Studio used to show the Obsolete message within the intellisense bubble 😕

No problem! I would have assumed so too. Not sure if it's something with my Visual Studio (2017) or just in general.


refactor: decoupling seeding from strapi bootstrap

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)

This allows us to simplify the startup a fair bit.

Not that it is really relevant to this pull-request, but I would like to be able to run the start commands from the root directory. Like I want to be able to start the backend and the frontend with the same command npm run develop. It is a hassle every time I have to change the directories to start them separately.

That's on my list. Hopefully by the end of today I should have a PR to simplify the whole process a fair bit.

Oh wow I dint know I could do this with Strapi. Thanks Oliver.


Update terragrunt.hcl

Ran Apply for 0 projects:

Locks and plans deleted for the projects and workspaces modified in this pull request:

dir: . workspace: default


Added function to test all the events from selected period

Function put into unit-tests.gs. Function updated with suggestion from @GioBonvi from comment under commit 00486664c7f49f43d81e712ba675dcdd45aa1f2c

I'd wait until we have merged #58 before making any change or review to this PR since it depends on functions and settings discussed there.

I am trying to decide wether this is the optimal way to go, considering that:

Google APIs are not rate-limited, but have daily limits (honestly they are quite high limits except for the number of email sent);
there is a hard limit on the execution time of a cusstom function (30 seconds);
we still would have to go through the logs manually looking for errors/imperfections, which is not a really reliable method;

I would say that right now as we don't have a better idea it's kinda ok.
It has helped me to test major-rewriting branch anyway.

I've finally found some time to dedicate to this project, sorry for the extremely long pauses.
I think you are right, since this code is just for us to run some tests and it's not targeted towards users we should inculde this since we don't have anything better.
The only thing I'd ask you is to parametrize the function, turning testDate and numberOfDaysToTest from hard-coded variables to function parameters.
The function can then be called with the appropriate parameters from unitTests() or any other function that may require it.
Thank you for your work and patience

Done, I guess.

I also find a little bug in the function (there was no test if mailContent isn't null), now it should work OK.

Good, did you test it?

Yup, looks like it works

I've just edited a comment for function a little bit.

Ok, there is just some tweaking left to do to comply with semistard JS style.
If you want I can fix them by rebasing the commit on this branch or I can relay the 5 or 6 small changes to you so that you can fix them by yourself. Tell me which way you prefer.

Tell me what's wrong, I will know for future and I'll try not to do those mistakes again.

Figuring out this global variables thing took me some time, but I guess I've corrected what was against JS semistandard.