ADD: histogram kernel

Small PR for adding histogram kernel functionality
This comes with some small enhancement on the kernel abstract class

Enable a constant list which will be integrate in kernel source as #define CONSTANT_NAME CONSTANT_VALUE.
Possibility to add parameters with tags which are not listed in kernel tag list, this is useful for complex kernel writing.

This should close:

#70

WARNING: switch kernel submodule to clesperanto_kernel branch before merge

Codecov Report

Merging #71 (2f50bce) into master (26aa19c) will decrease coverage by 0.08%.
The diff coverage is 87.05%.

@@            Coverage Diff             @@
##           master      #71      +/-   ##
==========================================
- Coverage   89.22%   89.13%   -0.09%
==========================================
  Files         119      131      +12
  Lines        6069     6691     +622
==========================================
+ Hits         5415     5964     +549
- Misses        654      727      +73

Impacted Files
Coverage Δ

clic/includes/core/clesperanto.hpp
77.77% <ø> (ø)

clic/src/tier1/cleHistogramKernel.cpp
79.68% <77.96%> (ø)

clic/includes/core/cleKernel.hpp
80.00% <80.00%> (ø)

tests/histogram_test.cpp
82.75% <81.13%> (ø)

tests/binary_not_test.cpp
83.33% <81.81%> (ø)

tests/binary_and_test.cpp
84.84% <83.60%> (ø)

tests/binary_or_test.cpp
84.84% <83.60%> (ø)

tests/binary_subtract_test.cpp
84.84% <83.60%> (ø)

tests/binary_xor_test.cpp
84.84% <83.60%> (ø)

clic/src/core/cleKernel.cpp
81.33% <93.33%> (+0.87%)
:arrow_up:

... and 8 more

Continue to review full report at Codecov.

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


Support for the official Pinterest api

With the attached changes, this Omniauth strategy would support the official Pinterest auth workflow.
It would be great if you could merge this PR and also push a newer version of the gem. I already increased the gem version, but maybe you'll name it more final, like 2.0.0.

not sure how to upvote pull requests, so just want to second, would be great if you could update this to support official pinterest

Hi @chikeabuah, until @jot finds the time to release a new version you can go with:
gem 'omniauth-pinterest', github: 'endil/omniauth-pinterest', branch: 'supporting-official-api'
in your Gemfile. I'll keep the repository up until the new gem is released.
@jot: If you give me access to the repository and RubyGems, I could help with the maintenance.

@jot: thanks! :+1:

@nlsrchtr I've also made you an owner on RubyGems.

Great, thanks! Just released version 2.0.0 and updated the repository.

@jot @nlsrchtr thanks guys!


Document HashToField security level

The OPCODE HashToField requires that the hash function heuristically behaves as a random oracle.
Kobi Gurkan pointed out that the security level should also be documented.
We can either:

Accept  a min security parameter . This would allow more flexibility if the underlying proving system can provide less than 128 bits and you do not need it.

Document ACIR indicating that we always require 128 bit security.

This will be left open until there is a specific place to document opcode


UI Bug in map activity

Steps to reproduce

Select any city. Select Hangout or any other map item
Following UI comes up:

Expected behaviour

There is space between card item & right edge of the screen. Remove that.
Add margin between more button & right edge of the card
Make Locate and More button smaller
Change the color of blue marker items to orange (the same as the colour on the more button). The blue markers does not look that distinguished on the map

Taking this up.


feat(sdk/js): update artifacts

This PR aims to support two endpoints that were missing from the Agent Protocol Specification in the JS SDK:

POST /agent/tasks/{task_id}/artifacts -> To upload artifacts for a given task
GET /agent/tasks/{task_id}/artifacts/{artifact_id} -> To download artifacts from a given task
GET /agent/tasks/{task_id}/artifacts -> Get the list of artifacts from a given task

It's worth mentioning that I've noticed that you started the migration of these SDK into their own repositories, but since it's a WIP I decided to open the PR here - If you feel that I should open the PR in the other repository just let me know (I also saw that it has some broken links).
I also have some questions regarding how the steps are handled - I noticed that in the Python SDK they're first created and the Status is defined as created but in JS we're not doing that; I updated it to be completed once the handler has finished but tbh I don't think this is handled okay, so I am looking for feedback here
Also, we're able to run a test suite in our agent with these changes. This is how I tested things :smile:

oh amazing. @jzanecook and I will review. Thank you!

PR migrated to https://github.com/AI-Engineers-Foundation/agent-protocol-sdk-js/pull/1


GUI for Howdy (config especially)

I am interested in creating a Qt-based GUI for configuration of Howdy for Hacktoberfest. Would this be approved?

I am interested in creating a Qt-based GUI for configuration of Howdy for Hacktoberfest. Would this be approved?

I think that I will create an organization for hosting this project myself.

Hey! Sorry for not getting back to you sooner, i don't monitor closed issues very well. A GUI would be very much appreciated, especially if it's in something like GTK so it could be distributed as howdy-gkt for instance

OK, we'll see what I come up with. I don't know GTK, so it will need to be Qt.

From: boltgolt<EMAIL_ADDRESS>Sent: Saturday, October 3, 2020 5:52:10 AM
To: boltgolt/howdy<EMAIL_ADDRESS>Cc: Loren Burkholder<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Subject: Re: [boltgolt/howdy] GUI for Howdy (config especially) (#449)
Hey! Sorry for not getting back to you sooner, i don't monitor closed issues very well. A GUI would be very much appreciated, especially if it's in something like GTK so it could be distributed as howdy-gkt for instance
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/boltgolt/howdy/issues/449#issuecomment-703078246, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANINLHLWYENRHDGKKACN35TSI3X4VANCNFSM4RYZNYFQ.

Any news on this very interesting subject ?

Well. It turns out that I gave up on the idea since my laptop's camera isn't IR and therefore is very bad for facial recognition. Maybe someday if and when I get an IR camera....

@mcgivrer there's a working GUI on the beta branch, but it's waiting for other changes in the 3.0.0 version

Really would be great, so the UX is better when adding faces. It'd be great if it could have a shortcut in Settings.


SDK Version Reporting Via Constant

Testing against my local dev environment, I can see the correct user agent reflecting in the API's logs:
api_1           | USER AGENT:
api_1           | JavaServiceSDK/4.9.0

The Java Release template has been updated to reflect that this needs to be bumped manually for now.

Let's just make sure we squash this on merge


Scale in image augmentations?

Notice that scale=(0.08, 1.0), is 0.08 too small, causing the cropped image to lose image details and even the tag information in the image

Hi there,
This is the default number set by Ross.
You can try different hyperparameter during your own training.


Fixing small typo in contribute.mdx

Description (required)
I updated contribute.mdx.
Related issues & labels (optional)

Suggested label: i18n

Lunaria Status Overview
🌑 This pull request will not trigger status changes.

Learn more
Lunaria automatically ignores changes on specific PRs by adding a ignored keyword in its title. Found: typo.
You can change this by either removing the keyword above from the PR's title, or modifying the ignoreKeywords property in your Lunaria configuration file.

Tracked Files

Locale
File
Note

en
contribute.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

Hi @wpplumber ! This is not a typo!  😄
Astro Docs Docs is the name of our website that holds the contributing guides for Astro Docs. It's the "Astro Docs" documentation!

Thank you for your recent Arabic PRs, and I hope we'll have them reviewed and merged soon!  🙌

Welcome @sarah11918, 😄

It's the "Astro Docs" documentation!

I missed that, thank you.


[PR for #76]: Created Templates to add new feature, report bugs and pull requests.

Related Issue
Closes: #76
Description of Changes
Created ".github" folder and added new templates under "issue_template" to report bugs and add new features.
Created file under ".github" folder and added new template for PR.
This template which you can see right now if pull request template.
Checklist:

[x] My code adheres to the established style guidelines of this project.
[x] I have conducted a self-review of my code.
[x] I have included comments in areas that may be difficult to understand.
[x] I have made corresponding updates to the project documentation.
[x] My changes have not introduced any new warnings.

Screenshots

Original
Bug Report

Original
New Feature

Original
Pull Request

Please provide any necessary screenshots to illustrate the changes made.

@sourabhsikarwar please review.
Thanks a lot for considering.


compose: remove color block from stream picker.

Fixes: This PR Solves #28796

Screenshots and screen captures:
Colorblock removed from New Message Compose Area

Colorblock removed from Moved Message Compose Area

Self-review checklist

[X] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[X] Each commit is a coherent idea.
[X] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[X] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Thanks! Please clean up your commit history and post again to request a review. See here for guidelines.

@alya I have revised the commit messages according to this guide. Would you mind reviewing the changes?
Mentioning because I am unable to request a review

Also, squash all your commits into 1, they all do the same thing, and individually they don't pass the checks, so in this case just 1 commit makes more sense.

There have been many files that were changed, renamed, or deleted. Once the reviews are complete, I will combine these commits into a single one.

I may have made a mistake by removing stream_bar.js, as well as the mock_stream_header_colorblock thing. I will be looking into this. help appriciated

@Mahhheshh Please be more thorough with your work before requesting for another round of review. Make sure to review your own work before asking others for reviews

I would love to receive your feedback on this Pr, @alya.

I take it you've addressed @N-Shar-ma 's feedback? If so, it's helpful to post a comment explicitly stating that.

The DM icon is cut off. Please post a comment with a screenshot when you've addressed this.

Looks good to me other than that!

should it be like this

or this

the difference is the left down cornor

The icon looks normal to me in the PR's current state... :thinking:
I'm not sure what @alya meant exactly since it is hard to make out from that screenshot. Might be a secondary monitor rendering issue?

If that's the case, and @N-Shar-ma 's feedback has been addressed, you can use Zulipbot to add the "chat.zulip.org review" label.

The second screenshot looks like it fixes the issue I was seeing.
👍🏽

@zulipbot "chat.zulip.org review"

@zulipbot add "chat.zulip.org review"

conflict related to stream_bar.ts.

Including this in today's chat.zulip.org deployment for feedback.

Feedback seems to be positive on this design change, so merged, thanks @Mahhheshh and @N-Shar-ma!


How to use arguments?

Hello there!
Would it be possible to use arguments with a statechart component?
If yes, then how?

yes! it is possible -- I think you want to use @context https://github.com/NullVoxPopuli/ember-statechart-component#api
and then be sure to handle the ARGS_UPDATE event on the machine for when the args change

I am sorry, I should have been more specific.
I was already aware of the arguments @config and @context.
What I meant what to use regular arguments to be accessed by this.args.
I guess it is not possible, so a combination of some data passed to @context and some functions to @config is needed.
@NullVoxPopuli

Yeah, inside the statechart, there aren't really any ember-specific additions. All dana is managed on the context, so that following the xstate docs works 1 to 1


How to compile physfsrwops.c?

I am trying to use PhysicsFS with SDL2, but I did not see an option to compile physfsrwops.c with cmake. Do I have to do it myself, or am I just missing something?

It's just example code; copy the files into your project.


Update Reconnect Prompt

https://microbit-global.monday.com/boards/1356069004/pulses/1368916444

Not sure if this is best look, but asking Lucy can be a separate step

Can we give the dialog a title and the same vertical spacing as other dialogs. Might need a bit more width for a heading. Maybe "micro:bit connection lost"?

Can we give the dialog a title and the same vertical spacing as other dialogs. Might need a bit more width for a heading. Maybe "micro:bit connection lost"?


⚠ Bofrost.be has degraded performance

In 8f47ab3, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1617 ms

Resolved: Bofrost.be performance has improved in a77d10c after 36 minutes.


[SMALLFIX] Removed explicit argument type in AbstractLocalAlluxioCluster

Removed an explicit argument type in the AbstractLocalAlluxioCluster class.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10065/
Test PASSed.

LGTM


模型量化失败，出现“ValueError: cannot convert float NaN to integer”错误

问题描述：
量化onnx模型时，选择asymmetric_affine-u8的量化方式，quantized_algorithm="kl_divergence"，量化失败并提示ValueError: cannot convert float NaN to integer，原模型输入格式为单通道图像，量化数据集为npy格式
环境：
ubuntu18.04， rknn版本1.7.1， python3.6.7
代码：
rknn = RKNN()

rknn.config(
        mean_values=[[0]],
        std_values=[[1]],
        reorder_channel="0 1 2",
        quantized_dtype="asymmetric_affine-u8",
        epochs=40,
        batch_size=16,
        quantized_algorithm="kl_divergence",
        # quantized_algorithm="normal",
        optimization_level=3,
    )

rknn.load_onnx(model="./Model/onnx/test.onnx")

rknn.build(do_quantization=True, dataset="./dataset/dataset_npy.txt", rknn_batch_size=-1)

rknn.export_rknn("./Model/rknn/tset.rknn")

rknn.release()

按此份代码运行时，在rknn.build的时候失败，提示错误信息:
E Catch exception when building RKNN model!
E Traceback (most recent call last):
E   File "rknn/base/RKNNlib/app/medusa/quantization.py", line 64, in rknn.base.RKNNlib.app.medusa.quantization.Quantization._run_quantization
E   File "rknn/base/RKNNlib/quantization/quantize_manager.py", line 379, in rknn.base.RKNNlib.quantization.quantize_manager.QuantizeManager.calculate_params
E   File "rknn/base/RKNNlib/quantization/quantize_manager.py", line 174, in rknn.base.RKNNlib.quantization.quantize_manager.QuantizeManager._cal_quantize_param_all
E   File "rknn/base/RKNNlib/quantization/asymmetric_quantizer.py", line 34, in rknn.base.RKNNlib.quantization.asymmetric_quantizer.AsymmetricQuantizer.cal_quantize_param_all
E   File "rknn/base/RKNNlib/quantization/utils.py", line 145, in rknn.base.RKNNlib.quantization.utils.get_max_min_range
E ValueError: cannot convert float NaN to integer
E Please feedback the detailed log file <log_feedback_to_the_rknn_toolkit_dev_team.log> to the RKNN Toolkit development team.
E You can also check github issues: https://github.com/rockchip-linux/rknn-toolkit/issues

如果把：
quantized_algorithm="kl_divergence"

改成：
quantized_algorithm="normal"

则可以量化，但是有警告信息：
W tensor @Add_Add16_113:out0 seems to be always 0, user might try to remove the correlative layer manually
W tensor @Conv_Conv16_115:out0 seems to be always 0, user might try to remove the correlative layer manually
W tensor @Add_Add15_123:out0 seems to be always 0, user might try to remove the correlative layer manually
W tensor @Conv_Conv15_125:out0 seems to be always 0, user might try to remove the correlative layer manually
W tensor @Add_Add14_129:out0 seems to be always 0, user might try to remove the correlative layer manually
W tensor @Conv_Conv14_131:out0 seems to be always 0, user might try to remove the correlative layer manually
W tensor @Conv_Conv16_115:weight seems to be always 0, user might try to remove the correlative layer manually
W tensor @Conv_Conv15_125:weight seems to be always 0, user might try to remove the correlative layer manually
W tensor @Conv_Conv14_131:weight seems to be always 0, user might try to remove the correlative layer manually

但是量化出来的模型推理结果都是0，与预期不符合。
出现的原因可能是什么？有没有可能是原模型的一些层的值本来就比较小，导致量化后太小了直接为0？

请问解决了么 求教


feat(node): add hash operation endpoint

Context
Adds an endpoint to hash operations in Jstz so that user's can have consistent hashing without wasm dependency. A slightly nicer UX would be an indutry standard binary encoding that is consistent across rust, typescript and other languages but existing solutions are not effortless drop ins and will require additional effort at a later time.

Description

Adds the hashing endpoint
Updates openapi

Manually testing the PR
cargo test -p jstz_node api_doc_regression

POST /operations/hash doesn't sound right to me, though I don't think there's a better verb for this operation.
What is the actual use case of this endpoint? Can we replace it by making POST /operations return the hash?

We need the operation hash (as part of signature generation) to inject the operation :(


Documentation on transferring data from other systems

We should add some documentation on how to import data from other database systems.
e.g. sqlite -> duckdb, postgres -> duckdb, etc

Great idea!
Are you thinking about some examples with the Postgres FDW , or using Python/Pandas as an intermediary? Or maybe using the CLI for each?

I was thinking about a simple dump -> load pipeline for each just using the CLI, as a starting point. Perhaps we could add one for Python as well, using the various Python connectors for the systems.

More important once we have stable storage

I think docs for the new postgres/sqlite/mysql scanners would be useful, a matrix sounds nice.


WebSocket client

There is a websocket server in spark, is there any websocket client currently implemented or planned?

Why would you want to add a client to a web server project?

Not necessarily a part of this project, but I was asking because I don't particularly want to use netty if there was something much more compact available.

@perwendel was talking about creating a (non-websocket) client library actually, but I'm guessing he doesn't have the time currently :smile:

I'll close this, since it's not really related to the Spark core.


Alerting Integration

Proposal
Include the potential to installation and manipulate alerts based on Prometheus metrics. When thresholds are breached, the dashboard can cause notifications.

It seems like you opened this issue against the wrong repository. Feel free to reopen otherwise.


[BUG]  iOS Doesn't Load Image

Describe the bug
In iOS, PhotoManager loads the thumbnails of images, but won't load the originBytes or File of the image. In some cases it will load the images. Screenshots seem to always work, for example.
The code below is how I get the assets:
var result = await PhotoManager.requestPermissionExtend();
    if (result.isAuth) {
      eventState.textMode = false;
      final albums = await PhotoManager.getAssetPathList(onlyAll: true);
      final album = albums.first;
      final recentAssets = await album.getAssetListPaged(page, 40);
      List<Map<String, dynamic>> tempLoadedAssets =
          await Future.wait(recentAssets
              .map((asset) async => {
                    "asset": asset,
                    "thumb": await asset.thumbData,
                    "valid": await asset.exists && asset.type == AssetType.image
                  })
              .toList());

And this code is how I load the image:
FutureBuilder(
                          future: myState.selectedImage?.originBytes,
                          builder: (context, snapshot) {
                            if (snapshot.data == null) return Container();

                            final bytes = snapshot.data as Uint8List;
                            return FadeIn(
                                key: ValueKey(snapshot.data),
                                duration: Duration(milliseconds: 200),
                                child: PhotoView(
                                    loadingBuilder: (context, _) => Center(
                                        child: CircularProgressIndicator()),
                                    controller: myState.photoController,
                                    minScale: PhotoViewComputedScale.covered,
                                    backgroundDecoration:
                                        BoxDecoration(color: Colors.grey[200]),
                                    imageProvider: MemoryImage(bytes)));
                          })

On iPhones, the image often never loads. I've tried loading using originBytes and the load method. It will load some types of images, like screenshots, as I mentioned above.
Flutter version
Flutter (Channel beta, 2.6.0-5.2.pre, on Microsoft Windows [Version 10.0.19043.1348], locale en-US)
• Flutter version 2.6.0-5.2.pre at C:\Users\jared\Software\flutter_windows_2.5.1-stable\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 400608f101 (8 weeks ago), 2021-09-15 15:50:26 -0700
• Engine revision 1d521d89d8
• Dart version 2.15.0 (build 2.15.0-82.2.beta)
Smartphone (please complete the following information):

Device: iPhone11
OS:  iOS 15.0.2
Version: 1.3.9+1

Not sure how, but the issue is fixed. I did change a few things: for example, I changed requestPermission to requestPermissionExtend as shown above.

Not sure how, but the issue is fixed. I did change a few things: for example, I changed requestPermission to requestPermissionExtend as shown above.

Issue is not fixed. Error still appears, and images only load in debug mode, but still broken in release mode. Some images will still load in iOS, by the way. Screenshots work, for example.

I'm not getting any useful logs or errors from your comments, at present.

[core] "Error returned from daemon: Error Domain=com.apple.accounts Code=7 "(null)""
[PAAccessLogger] Failed to log access with error: access=<PATCCAccess 0x282c46700> accessor:<<PAApplication 0x2801c3660 identifierType:auditToken identifier:{pid:926, version:2688}>> identifier:96B577E3-1A84-4551-AC4B-B50A18E35270 kind:intervalEnd timestampAdjustment:0 tccService:kTCCServicePhotos, error=Error Domain=PAErrorDomain Code=10 "Possibly incomplete access interval automatically ended by daemon"

Have you seen anything like this error before? Here is my info.plist. Anything else that might be helpful?
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
	<dict>
		<key>CFBundleDevelopmentRegion</key>
		<string>$(DEVELOPMENT_LANGUAGE)</string>
		<key>CFBundleExecutable</key>
		<string>$(EXECUTABLE_NAME)</string>
		<key>CFBundleIdentifier</key>
		<string>$(PRODUCT_BUNDLE_IDENTIFIER)</string>
		<key>CFBundleInfoDictionaryVersion</key>
		<string>6.0</string>
		<key>CFBundleName</key>
		<string>whatado</string>
		<key>CFBundlePackageType</key>
		<string>APPL</string>
		<key>CFBundleShortVersionString</key>
		<string>$(MARKETING_VERSION)</string>
		<key>CFBundleSignature</key>
		<string>????</string>
		<key>CFBundleVersion</key>
		<string>$(CURRENT_PROJECT_VERSION)</string>
		<key>LSRequiresIPhoneOS</key>
		<true/>
		<key>NSAppTransportSecurity</key>
		<dict>
			<key>NSAllowsArbitraryLoads</key>
			<true/>
		</dict>
		<key>NSPhotoLibraryUsageDescription</key>
		<string>Whatado needs your permission to access photos in order to upload events with photos.</string>
		<key>UIBackgroundModes</key>
		<array>
			<string>fetch</string>
			<string>remote-notification</string>
		</array>
		<key>UILaunchStoryboardName</key>
		<string>LaunchScreen</string>
		<key>UIMainStoryboardFile</key>
		<string>Main</string>
		<key>UIStatusBarHidden</key>
		<false/>
		<key>UISupportedInterfaceOrientations</key>
		<array>
			<string>UIInterfaceOrientationPortrait</string>
		</array>
		<key>UISupportedInterfaceOrientations~ipad</key>
		<array>
			<string>UIInterfaceOrientationPortrait</string>
			<string>UIInterfaceOrientationPortraitUpsideDown</string>
			<string>UIInterfaceOrientationLandscapeLeft</string>
			<string>UIInterfaceOrientationLandscapeRight</string>
		</array>
		<key>UIViewControllerBasedStatusBarAppearance</key>
		<false/>
	</dict>
</plist>

Could it perhaps have something to do with images on iCloud?

Still, nothing relates. Please follow up on how the example is handled with iCloud or other images.


Search and replace input text fields are unusable with multi-line text

VSCode Version:

Commit: 91899dcef7b8110878ea59626991a18c8a6a1b3e
Date: 2020-07-23T15:51:39.791Z
Electron: 7.3.2
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Linux x64 5.7.9-100.fc31.x86_64

OS Version:

Fedora 30 (I think)
Linux 5.7.9-100.fc31.x86_64 #1 SMP Fri Jul 17 17:18:38 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Steps to Reproduce:

Press CTRL+SHIFT+H to initiate a global replace
Paste multiple lines of text into the Search box (e.g. 10+ lines)
Try to use the cursor keys to move around the input box - weird things happen and the input box is unusable
Use the mouse wheel to scroll and click in different spots to place the cursor - weird things happen and the input box is unusable
Repeat this for the second input box (replace text input box)

Does this issue occur when all extensions are disabled?: Sorry, I haven't tried this.

I can repro - not sure what happened here


fix values_for

This commit fixes the generated values_for methods. A test for that already existed and failed with
 test: Campaign should return all defined values of a given bitmask attribute. ERROR
    undefined local variable or method `webprintemailphone' for #<Class:0x7f89e23cae60>

With this commit it passes again.

+1

cheers!


archive/tar: Does not handle extended pax values with nulls

What version of Go are you using (go version)?

go version go1.12.5 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What did you do?
Read a tar file generated by git archive:
$ git clone https://github.com/SSW-SCIENTIFIC/NNDD.git
$ cd NNDD
$ git archive --format tar c21b98da2ca7f007230e696b2eda5da6589fe137 > nndd.tar

Alternatively you can fetch this archive directly from github: https://github.com/SSW-SCIENTIFIC/NNDD/tarball/c21b98da2ca7f007230e696b2eda5da6589fe137
If you try to read this with golang's archive/tar you will get a tar.ErrHeader on one of the entries. Here is a reproduction program:
package main

import (
	"archive/tar"
	"compress/gzip"
	"fmt"
	"io"
	"log"
	"net/http"
	"os"
)

func main() {
	// Read tar from file if specified, otherwise fetch directly from github
	var r io.Reader
	if len(os.Args) > 1 {
		f, err := os.Open(os.Args[1])
		if err != nil {
			log.Fatal(err)
		}
		defer f.Close()
		r = f
	} else {
		resp, err := http.Get("https://github.com/SSW-SCIENTIFIC/NNDD/tarball/c21b98da2ca7f007230e696b2eda5da6589fe137")
		if err != nil {
			log.Fatal(err)
		}
		defer resp.Body.Close()
		gr, err := gzip.NewReader(resp.Body)
		if err != nil {
			log.Fatal(err)
		}
		defer gr.Close()
		r = gr
	}

	tr := tar.NewReader(r)
	for {
		_, err := tr.Next()
		if err == io.EOF {
			break
		}
		if err != nil {
			log.Fatal(err)
		}
	}
	fmt.Println("Successfully read tar")
}

If you try read the same file with bsdtar or gnutar they both read the whole tarball. bsdtar (depending on version) will log a warning to stderr but still extract the rest of the tarball.
The archive contains a symbolic link which instead of a path for value, contains a large value which includes null. The validatePAXRecord explicitly disallows null from appearing in the value for PAX values for links. This leads to tar.Reader.Next() returning tar.ErrHeader.
https://github.com/golang/go/blob/go1.12.5/src/archive/tar/strconv.go#L321-L322
I reported this to the git mailing list<EMAIL_ADDRESS>Read that issue for more details on the "bad" entry in the archive.
The opinion there seems to be that generating values that aren't C-strings for links is valid. Not all tarballs are destined for filesystems. And the latest bsdtar and gnutar do not complain about the value. gnutar will write the symbolic link value upto the first null.
What did you expect to see?
I think any of the follow behaviours would be better:

Be able to read the rest of the tar if encountering a recoverable error.
Allow nulls in all PAX values.
Truncate up to first null in PAX value. (consistent with gnutar)

What did you see instead?
tar.ErrHeader on a tarball both bsdtar and gnutar can read.

@dsnet

FYI I am happy to contribute a fix for this, just need guidance on what is the appropriate fix.

The specification for PAX regarding linkpath says:

The pathname of a link being created to another file, of any type, previously archived. This record shall override the linkname field in the following ustar header block(s). The following ustar header block shall determine the type of link created. If typeflag of the following header block is 1, it shall be a hard link. If typeflag is 2, it shall be a symbolic link and the linkpath value shall be the contents of the symbolic link. The pax utility shall translate the name of the link (contents of the symbolic link) from the encoding in the header to the character set appropriate for the local file system. When used in write or copy mode, pax shall include a linkpath extended header record for each link whose pathname cannot be represented entirely with the members of the portable character set other than NUL.

This suggest that NUL is not a valid character to appear the extended record for linkpath.

Be able to read the rest of the tar if encountering a recoverable error.

This seems reasonable, but it requires a scrub through of the entire implementation to identify fatal vs. non-fatal errors. It also needs an API design to surface non-fatal errors.
One problem is that the implementation has always returned tar.ErrHeader for all types of parse errors (both fatal and non-fatal ones). The compatibility agreement technically requires us to continue to return the same sentinel error in the same circumstances. Who uses this obscure feature, I have no idea.

When used in write or copy mode, pax shall include a linkpath extended header record for each link whose pathname cannot be represented entirely with the members of the portable character set other than NUL.

This suggest that NUL is not a valid character to appear the extended record for linkpath.

I read over that bit of the spec a few times. I believe it is saying use an extended header record if set(linkpath) - (set(portable_characters) - set(\0)) != set(). Using the set notation to try and clear up the ambiguity. With that interpretation it is fine to include nul.
However, the spec also states UTF-8. But the archive/tar implementation doesn't validate that due to many archives not following spec. I think the same case can be made here. Especially given gnutar and bsdtar do not choke on this. For example the CR which introduced the validatePAXRecord cited what gnutar and bsdtar does in practice for its implementation.

Be able to read the rest of the tar if encountering a recoverable error.

This seems reasonable, but it requires a scrub through of the entire implementation to identify fatal vs. non-fatal errors. It also needs an API design to surface non-fatal errors.
One problem is that the implementation has always returned tar.ErrHeader for all types of parse errors (both fatal and non-fatal ones). The compatibility agreement technically requires us to continue to return the same sentinel error in the same circumstances. Who uses this obscure feature, I have no idea.

I agree, I think that rules out this possibility without further investigation. Given what gnutar and bsdtar do, I would be tempted to just remove the contains nul validation. If that was done, the effect on existing use cases:

Writers: Could now generate tars which previously would of failed. When the tars are read how it is handled would depend on implementation. I think this is fine, since it is the same as not writing valid utf-8. Additionally I doubt anyone relied on the behaiour of failing with nul in linkpath.
Readers: Instead of failing, we would pass on linkpaths with null in them. This could lead to failures in later parts of a program. This seems acceptable to me, since it would of failed anyways. Additionally the error is more likely to be more descriptive than the error value tar.ErrHeader.


fix: modify url regex to filter

Description Of Changes
Resolves UIKIT-3707
AS-IS
A string \w $ sign(e.g. $123.12) or<EMAIL_ADDRESS>created unnecessary underline with link

TO-BE

@HoonBaek Yeah sounds like a idea too!

@HoonBaek , actually, share an example :)


[5.1] saveMany receives collection OR array to hasOneOrMany

saveMany() receiving a collection OR array. In Illuminate\Database\Eloquent\Relations\hasOneOrMany. See issue #7144. Backwards Compatible no BCs.

The saveMany() method of the Illuminate\Database\Eloquent\Relations\hasOneOrMany relationship can only receive an array of models. The API across eloquent usually you can pass in collections or arrays. So this behaviour seems strange and not consistent with the rest of the API.

Understandable. In that case it would be a pointless BC.


Support for new Heap JS API

Heap is discontinuing support of their current javascript API/bindings on June 1, 2016:
https://heapanalytics.com/docs/updating-custom-api

wasn't really keeping this up to date, but new JS merged in #9


⌛ Add score stability (array with min depth)

Test  | time/score-stability-1-mindepth-7
Elo   | 0.53 +- 3.94 (95%)
SPRT  | 40.0+0.40s Threads=1 Hash=128MB
LLR   | -0.16 (-2.25, 2.89) [0.00, 3.00]
Games | 10492: +2630 -2614 =5248
Penta | [146, 1249, 2446, 1253, 152]
https://openbench.lynx-chess.com/test/1009/

Feature added as part of #1223


Choice of router

I'm curious to know why you chose page instead of react-router.  Are you willing to share your thinking on routers?
thanks!

Oh the reason is trivial: back in the day when I wrote this repo I hadn't experience with RR so I chose page. The pro is that page is a v1.x while RR is (even now) in flux. The major concern is that this app is not "isomorphic". When I will find the time (and RR will hit v1.0) I'll try to replace page with RR and make the code sharable between client and server.
Besides I'm not a big fan of the current version of RR:

it's not truly isomorphic without this patch: git://github.com/bobpace/react-router.git#transitionContext
defining the router via jsx is weird
mixing routes definitions with layout (as suggested in its docs) can be misleading, they should be separate concerns
I'd prefer to define middlewares without using react components

RR 1.0.0 on master


[CP][beta channel] Cherry pick a regression for how the dartanalyzer handles exclude file paths

commit(s) to merge: https://github.com/dart-lang/sdk/commit/37a91c04b19df62e37c5d4d727926f518ab30c71
merge instructions: clean merge
What is the issue:
There was a regression for how the dartanalyzer handles analysis exclude paths in some situations.
What is the fix:
We updated the exclude path handling to better track which directory the exclude path should be relative to.
Why cherrypick:
This is likely a lower impact issue for users - a low frequency issue - however, cherry-picking this will simplify the beta to stable roll process (as per @athomas).
Risk:
Low - this is a small patch and a clean merge.
Link to original  issue(s):
https://github.com/flutter/flutter/issues/81335
/cc @kevmoo @mit-mit @whesse @athomas @vsmenon @franklinyow @scheglov

CC: @kf6gpe @zanderso

From an infra perspective we want to pick this so we can upgrade the checked-in SDK to the 2.13 beta (and later stable). Otherwise we'll have to settle for a dev release with this fix.

@athomas - can you verify that the above  fix - 37a91c0 - addresses the issue? I think we would cherry-pick this, contingent on us knowing that this is a real fix. I expect that's just re-try the sdk upgrade + a successful engine roll?

@christopherfujino as FYI, in case he hasn't seen this.

@athomas - can you verify that the above fix - 37a91c0 - addresses the issue? From a discussion, this cherry-pick would be approved contingent on us knowing that this is a real fix. I expect that's just re-try the sdk upgrade + a successful engine roll?

There's a bit a of chicken & egg problem here because "retrying the checked-in SDK upgrade" would mean CP'ing this to beta and then upgrading the checked-in SDK.
@aam Was this issue discovered on 3xH testing? In that case one way to resolve it would be to attempt an upgrade to a dev release that has the fix and then running the 3xH tryjob to verify that the issue has been fixed. It doesn't prove that it was exactly this fix but at least that there was some fix on master that resolved the issue.

@aam Was this issue discovered on 3xH testing?

No, this was discovered when engine->dart roller failed (https://github.com/flutter/flutter/issues/81335).
To verify the fix one could roll third_party_dart/tools/sdks in locally checked-out flutter/engine, build host_debug_unopt and run third_party/dart/tools/sdks/dart-sdk/bin/dartanalyzer --options flutter/analysis_options.yaml --packages=flutter/flutter_frontend_server/.dart_tool/package_config.json flutter/flutter_frontend_server

Cherry-picking 37a91c0 on top of 2.14.0-49.0.dev, building host_debug_unopt and running dartanalyzer from it against flutter_frontend_server shows that it indeed fixes the problem.
$  out/host_debug_unopt/dart-sdk/bin/dartanalyzer --log --verbose --options flutter/analysis_options.yaml --packages=flutter/flutter_frontend_server/.dart_tool/package_config.json flutter/flutter_frontend_server
Analyzing flutter/flutter_frontend_server...
No issues found!

@aam, thanks much for confirming!
@franklinyow - adding the cp-approved label (but please confirm).

Confirm approval.

Merged to beta in 4511eb2a779a612d9d6b2012123575013e0aef12 (2.13.0-211.14.beta).


0.0.18-alpha.4 unable to record

As I can't add the components as usual, I added them via script in the start method (see my issue before), then I tried to record with videokit and I got the following exception:
InvalidOperationException: Operation is not valid due to the current state of the object. VideoKit.Internal.VideoKit.Throw (VideoKit.Internal.VideoKit+Status status) (at ./Library/PackageCache/ai.natml.videokit@0.0.18-alpha.4/Runtime/Internal/VideoKit.cs:692) VideoKit.MediaRecorder.get_sampleRate () (at ./Library/PackageCache/ai.natml.videokit@0.0.18-alpha.4/Runtime/MediaRecorder.cs:126) VideoKit.MediaRecorder.Append (VideoKit.AudioBuffer audioBuffer) (at ./Library/PackageCache/ai.natml.videokit@0.0.18-alpha.4/Runtime/MediaRecorder.cs:157) VideoKit.Sources.SceneAudioSource+<>c__DisplayClass11_0.<.ctor>b__0 (System.Single[] data, System.Int32 channels) (at ./Library/PackageCache/ai.natml.videokit@0.0.18-alpha.4/Runtime/Sources/SceneAudioSource.cs:121) VideoKit.Sources.SceneAudioSource+AudioSourceAttachment.OnAudioFilterRead (System.Single[] data, System.Int32 channels) (at ./Library/PackageCache/ai.natml.videokit@0.0.18-alpha.4/Runtime/Sources/SceneAudioSource.cs:131)
My Start method to add the components:
public void Start(){
        gameObject.AddComponent<VideoKitAudioManager>();
        VideoKitRecorder recorderComponent = gameObject.AddComponent<VideoKitRecorder>();
        recorderComponent.recordingAction = VideoKitRecorder.RecordingAction.Custom;
        recorderComponent.videoMode = VideoKitRecorder.VideoMode.Texture;
        recorderComponent.audioMode = VideoKitRecorder.AudioMode.AudioListener;
        recorderComponent.audioListener = Camera.main.GetComponent<AudioListener>();

    }

My start recording mehtod:
public async void StartRecording()
    {
        gameObject.GetComponent<VideoKitRecorder>().OnRecordingCompleted.RemoveAllListeners();
        gameObject.GetComponent<VideoKitRecorder>().OnRecordingCompleted.AddListener(StopRecording);
#if UNITY_ANDROID
        if (!Permission.HasUserAuthorizedPermission(Permission.ExternalStorageWrite))
        {
            Permission.RequestUserPermission(Permission.ExternalStorageWrite);
        }
#endif

        if (GetComponent<VideoKitRecorder>().status == VideoKitRecorder.Status.Recording)
        {
            await GetComponent<VideoKitRecorder>().StopRecording();
            GameObject.Find("RecBtn").GetComponent<Blinking>().stopBlink();
        }
        else if (GetComponent<VideoKitRecorder>().status != VideoKitRecorder.Status.Recording && !recorderIsBusy)
        {
            recorderIsBusy = true;
            GetComponent<VideoKitRecorder>().texture = GameObject.Find("Plane").GetComponent<MeshRenderer>().material.mainTexture;
            GetComponent<VideoKitRecorder>().resolution = VideoKitRecorder.Resolution.Custom;
            bool record1080p = Settings.settingsObj["sections"]["Others"]["Record quality"]["current"]["value"] == 1080;
            int recordingWidth = record1080p ? 1920 : 1280;
            int recordingHeight = record1080p ? 1080 : 720;
            if (Settings.settingsObj["sections"]["Others"]["Record format"]["current"]["value"] == "keepDeviceRatio")
            {
                if (Screen.width > Screen.height)
                {

                    recordingWidth = (int)((float)Screen.width / (float)Screen.height * (float)recordingHeight);
                }
                else
                {

                    recordingWidth = (int)((float)Screen.height / (float)Screen.width * (float)recordingHeight);
                }
                //Make sure width is even as this could cause problems on android
                if (recordingWidth % 2 == 1)
                {
                    recordingWidth += 1;
                }
            }
            Debug.Log("width" + recordingWidth + "height" + recordingHeight);

            if (Screen.width > Screen.height)
            {
                GetComponent<VideoKitRecorder>().customResolution = new Vector2Int(recordingWidth, recordingHeight);
            }
            else
            {
                GetComponent<VideoKitRecorder>().customResolution = new Vector2Int(recordingHeight, recordingWidth);
            }
            GetComponent<VideoKitRecorder>().keyframeInterval = 1;
            GetComponent<VideoKitRecorder>().videoBitRate = recordingWidth * recordingHeight * 30;

            if (PlayerPrefs.GetInt("mic") == 0 || AudioPeer.instance.audioDevice == null)
            {
                GetComponent<VideoKitRecorder>().audioMode = VideoKitRecorder.AudioMode.AudioListener;
            }
            else
            {
                GetComponent<VideoKitRecorder>().audioMode = VideoKitRecorder.AudioMode.AudioDevice;
            }
            GameObject.Find("RecBtn").GetComponent<Blinking>().startBlink();
            GameObject.Find("Notification").GetComponent<UINotificationManager>().Notify("", "Started recording!", "Recording started successfully.");
            await GetComponent<VideoKitRecorder>().StartRecording();
        }
    }

I used the following settings:

Do you have any idea what could cause this problem?

@marvpaul are you running this on the Windows editor by any chance?

@marvpaul are you running this on the Windows editor by any chance?

No it's Mac os, unity 2022.3.10f1

I also created a simple test project where I was able to reproduce the issue. I just added VideoKit to an empty project and started recording when you click on the white square in the scene. Then the error occurs.
https://github.com/marvpaul/videokit

Repro'ed, fix coming!

Fixed in 0.0.18-alpha.6. Also @marvpaul your access key is public in the repro you shared. I strongly recommend generating a new access key, and deleting that one to prevent leakage.
To omit the VideoKit settings from git tracking, add the following to your .gitignore:
# VideoKit
ProjectSettings/VideoKit.asset

Sounds good.


Fix tests

This will fix tests and add missing class-transformer

All tests pass now, i think this MR is ok

Ping @kotivuori did you got the previous message? This is looking good and ready to merge


Cannot close game window when having a panel on it

Godot version:

Can reproduce in beta5 and beta4, not happening in beta3
OS/device including version:

Linux Mint Debian Edition
Issue description:

When having a panel in a scene, closing the window using the (x) button, causes Godot to disconnect from it but not close it. Then the window remains unresponsive.
It does not happen if you don't move the panel from it's default position
Steps to reproduce:
1- Create a scene with a root node (Tested with both Node and Node2D)
2- Add a Panel (Control)
3- Move the panel (for some reason, I noticed it's not happening if the panel is at it's parent position)
3- Play the scene
4- Close the window using the (x) button
Here Godot disconnects from the scene, the game does not close and stays unresponsive.
Minimal reproduction project:

Panel.zip

Can't reproduce in OSX. It might be a System problem, but as it's happening in beta4 and 5 but not in beta3, I guessed something changed in between.

Can't reproduce, Arch Linux, KDE.
Can you try to get a backtrace using gdb (and just pressing CTRL-C after it freezes)?

Can't reproduce either with 3.1 beta 5 on Mageia 7 x86_64, KDE. Nor with a build from the master branch.

It's probably a system problem. I noticed that the only thing is that it takes about 4 minutes to close instead of not closing at all.


Unbound block list , How to know from query log that if domain refused or not?

I have configured an Unbound server with domain block list, blocking works perfectly fine. But from the query log, i can only see that blocked domain is queried, that's it. I tired with
server:
local-zone: "00009483.com" always_refuse

and increased log level as well, but i cannot able to find any indication that this domain refused / blocked . Is there any configuration do i need to update ? or is it possible to add custom query message for blocked domains?

Dear James,
Thank you for using Unbound.
We use GitHub Issues for bugs and feature requests only.
For questions like configuration or support, we use the Unbound community mailing list, see https://lists.nlnetlabs.nl/mailman/listinfo/unbound-users.
Can you resend your question to the mailing list?
Thank you,
-- Benno


docs: Remove sidecar-http-proxy config from upgrade instructions

Fixes: https://github.com/cilium/cilium/commit/4ccf124629942f7ab1c461485fe7b8f3ed3caf94
Fixes: https://github.com/cilium/cilium/pull/4531
Signed-off-by: Romain Lenglet<EMAIL_ADDRESS>

This change is

test-docs-please


Development: Handle exam users file persistency directly

Checklist
General

[ ] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[X] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[X] I chose a title conforming to the naming conventions for pull requests.

Server

[X] Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
[X] I strictly followed the server coding and design guidelines.
[X] I documented the Java code using JavaDoc style.

Motivation and Context

I want tor remove the last instance of using temporary files after upload instead of saving the file directly. Additionally, I want to remove the deprecated EntityFileHelper.
Description

I replaced persisting and deleting the the files exam user's signature and image with direct calls. Then I removed the deprecate helper class
Exam Mode Testing

Ensure that the iOS app for the exam conduction still works as expected
Testserver States

[!NOTE]
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked

Review Progress

Performance Review

[x] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

[x] Code Review 1
[x] Code Review 2

Exam Mode Test

[ ] Test 1
[ ] Test 2

Test Coverage

Class/File
Line Coverage
Confirmation (assert/expect)

src/main/java/de/tum/in/www1/artemis/service/exam/ExamRegistrationService.java
92.8%

src/main/java/de/tum/in/www1/artemis/service/exam/ExamUserService.java
80.8%

src/main/java/de/tum/in/www1/artemis/service/user/UserService.java
92.9%

src/main/java/de/tum/in/www1/artemis/web/rest/ExamResource.java
96.0%

src/main/java/de/tum/in/www1/artemis/web/rest/ExamUserResource.java
94.7%

Summary by CodeRabbit

Refactor

Streamlined file management by removing outdated methods and fields, and integrating FileService and FilePathService for efficient file operations.

Bug Fixes

Ensured non-null return values in file saving operations to prevent potential errors.

Thank you for the fix @nityanandaz. We should cover this case with regression tests. As we want to merge this PR ASAP, I'll add regression tests for this in a follow-up to not block the workflow.


XMS-PDFT Gradients No Error

I need to check to see if calling XMS-PDFT gradients yields an error if .nuc_grad_method() is called on the object. If not, then this is a fairly minor (but also not) bug issue.

Not an issue. It will raise an exception if someone tries to run XMS-PDFT gradients.


Updated r-oxbow's dependencies

Hallo! Out of curiosity, I was looking at this package (r-oxbow) use of extendr. I ran cargo update and then had to fix these things in lib.rs. But the extendr-fn gave this error:
error[E0277]: the trait bound `extendr_api::Robj: From<Vec<(u64, u16)>>` is not satisfied
  --> src/lib.rs:76:1
   |
76 | #[extendr]
   | ^^^^^^^^^^ the trait `From<Vec<(u64, u16)>>` is not implemented for `extendr_api::Robj`
   |
   = help: the following other types implement trait `From<T>`:
             <extendr_api::Robj as From<&'a [T; N]>>
             <extendr_api::Robj as From<&'a [T]>>
             <extendr_api::Robj as From<&'a mut [T; N]>>
             <extendr_api::Robj as From<&Altrep>>
             <extendr_api::Robj as From<&Primitive>>
             <extendr_api::Robj as From<&Vec<T>>>
             <extendr_api::Robj as From<&extendr_api::Complexes>>
             <extendr_api::Robj as From<&extendr_api::Doubles>>
           and 53 others
   = note: this error originates in the attribute macro `extendr` (in Nightly builds, run with -Z macro-backtrace for more info)

That's because it wants to turn Vec<(u64, u16)> to an Robj. I cannot imagine this was ever possible to do. If this is strictly necessary to do, I'd like to hear how it was possible before, and stuff like that if at all possible.


Disable assembly generation in numba?

When generating llvm IR, does numba generates assembly code? if so, we need to be able to disable it since rbc only needs LLVM IR. This might speedup things a little bit.

x-ref: issue #87

I believe this can be achieve by setting the flag no_compile on irtools.py


Form submission

Hello, I'm trying to use your plugin (great work by the way :) ) but I have a problem integrating it into my form.
When I add a new tag or delete one of the default tags, the form gets submitted which redirects me to a new page. Also the server does not receive the tags from the form request that get's submitted.
I've googled and looked at your documentation but didn't find any solution.

Hi, i have the same issue. The reaseon i guess you are using plugin inside  element. If you remove probably it will gonna solve.

Yes, it's inside a form element. I've also figured out I could probably move it outside of a form and handle it in separate ajax request but I would prefer to keep it inside as It's a part of the form. Regarding the form request I had and idea that I could create another component as a wrapper for smart-tagz and bind the value to some hidden input / hidden multi select (so it would be an array of tags, not a string) but that feels like a hacky solution.

@Niekto350 thanks for reporting the issue.  can you point me to the repo where this issue can be reproduced.

I don't have one, only a bigger local project where I'm trying to use your plugin as a part of a bigger system. But I think I could try to create one for you.
BTW: What are the chances that I check the issue the same minute as you post your comment? :)

Okay, I've created a dummy repo for you available at https://github.com/Niekto350/smart-tagz-demo
Just clone it, npm install and run npx mix to compile it. After that open index.html from to public directory. If you click on a (x) button to delete the tag, it get's submitted. If you add a new tag, it also submits the form.
The form should only be submitted by hitting the submit button (or enter at some other form field of course).

@Niekto350 Here is a hack that I have bee using
<smart-tagz           @keypress.enter.prevent           @click.prevent           inputPlaceholder="Create Tags ..."           :allowPaste="{delimiter: ','}"           :allowDuplicates="false"           :maxTags="20"           :on-changed="setTags"       />

Thank you, I've tried something similar (@keyup.enter.prevent.stop) but it wasn't working, I'm still new to Vue so maybe I've did something wrong.
For now I've rewritten the tagging system plugin into another custom vue component built on top of different pure js library as I needed to get it working and could not wait longer for the fix. Once this get's fixed, I may come back to it and rewrite it once again to the original smart-tagz solution.

@Niekto350 Here is a hack that I have been using
<smart-tagz
  @keypress.enter.prevent
  @click.prevent
/>

@click.prevent
Works like a charm!
            <smart-tagz v-if="loaded" autosuggest inputPlaceholder="Select Tags..." :sources="formData.tags"                         :maxTags="5"                         :on-changed="updateTags"                         @click.prevent             />
    updateTags(tags) {       this.tags = tags     },

I ended up with a custom component and binding to a v-model
            <smart-tags-component v-if="loaded"                                   :maxTags="5"                                   :sources="formData.tags"                                   v-model="tags"             />
`

`


Painting into a mask

Hello - I need to be able to paint only in a masked area.
Can you tell how that could be done?
Many thanks.

I'm also interested in what's the best way to implement this feature. For example, limit the drawing area within the closest bounds of an SVG image..

In the end I got a great solution to this. Very accurate.

In the end I got a great solution to this. Very accurate.

Awesome! Is it possible to share the algorithm or the code so I could try it?

Let me know if you need some explanation:
override func touchesEnded(_ touches: Set, with event: UIEvent?) {
if !swiped {
// draw a single point
drawLine(from: lastPoint, to: lastPoint)
}
// Merge tempImageView into mainImageView
UIGraphicsBeginImageContext(mainImageView.frame.size)
mainImageView.image?.draw(in: view.bounds, blendMode: .normal, alpha: 1.0)
tempImageView?.image?.draw(in: view.bounds, blendMode: .normal, alpha: opacity)
mainImageView.image = UIGraphicsGetImageFromCurrentImageContext()
UIGraphicsEndImageContext()

print("currentBodyMask = \(currentBodyMask)")
let maskImageview   = UIImageView()
maskImageview.image = UIImage(named: currentBodyMask)

maskImageview.contentMode = .scaleAspectFit
maskImageview.frame = mainImageView.bounds
print(mainImageView.bounds)

mainImageView.mask = maskImageview
tempImageView.image = nil

}
Mask image example:

Many thanks!

Thanks, guys! @askrav-anoda  @askrav, did you manage to mask the drawing to the closest path of an SVG/Shape?


Add cost alarms for CBS resources

Summary
Alarms should be setup to alert us when CBS resource costs exceed a threshold.

After a discussion with AWS, we could setup the following billing alarms:

Each hour, compare the cost of account resources from current to previous hour.
Warning if 10% increase.
Critical if 50% increase.

This will catch early, unexpected ramp up costs.


Major/go v2 sdk (#2)

CHANGES
GENERAL

Standarized letme configuration files to .ini to match aws credentials.
Upgraded dependencies to Go 1.22
Upgrade go SDK to v2.
Version name is now New Horizons.

OBTAIN
Refactor:

Extracted and simplified logic from main function.
Created two functions:

AssumeRoleChained
AssumeRole

Stored new credentials through ini library for ease of use and validation.
Created LoadAwsCredentials and LoadAwsConfig to store credentials on .aws/credentials and .aws/config respectively.
Change DynamoDB query from Scan to Get (requires name as primary key).
Validation of credentials now doesn't require to use assume rol.
Credentials v1 is supported for credentials without MFA.

CONFIG && CONFIG-FILE

Update functions to use ini library.
Create ini files instead of toml.

LIST

Extracted  logic from main function.
Fixed table length on output. Now is calculated based on the item.Name with max length.
Removed local flag as it's unused.

UTILS

Removed non-used functions.
Removed non-used structs.

REMOVE

Simplified validations.

Instead of checking an array now checks values on a map.
For verifying that a context is managed by letme, now uses the ini module to check if the comment matches the one used by letme.

[ ] Remove references to hectorruiz-it and letme-alpha directory.
[ ] Update README.md

Hi @hectorruiz-it , thanks for your contribution. I will be looking at this as soon as I can.
Best,


Trust: Rephrase setting trust message

Testing #122209

Should be instead:

This setting can only be applied in a trusted workspace.

Same for JSON file hover:


Dockerize etcd

@Smana Can you try again? It installs calico_plugin before running kubelet.

With these commits, the role works fine.
However there's a required python module which has to be installed manually.
We need to work on the role "common" which will install all the requirements.

I've cleaned commits (force-push).


Add current time line on Timetable

Idea Description
It must be fun 👍
Reference images and links

I would like to try this issue 🙋

Thanks! Assigned 👍


Fix typo in documentation

Fix typo in documentation
Signed-off-by: Vladimir Nachev<EMAIL_ADDRESS>

This change is

Coverage decreased (-0.005%) to 45.936% when pulling 3e9b2286b95da32f5af34420783cd97aebae0c48 on vpnachev:fix-typo-in-documentation into 9d3374d12af94d1888cfde1de2318774a8b0ebb9 on cilium:master.

test-docs-please


fix(preset-attributify): filter invalid extraction

Fixes #2037

Simple and Violent😅, but it can solve the problem temporarily. I think the best way is to update the reg.

Simple and Violent😅, but it can solve the problem temporarily. I think the best way is to update the reg.

I tried, but couldn't come up with one. In my opinion @antfu's suggestion is better. I updated the PR with his suggestion. Now we can still target attributes that end with :, or people who need it.

This broke my use case ;(
Minimal repro:


feat: support specify project when create provider

Signed-off-by: lixd<EMAIL_ADDRESS>
What type of PR is this?

/kind feature
What this PR does / why we need it:
support specify project when create provider
Which issue(s) this PR fixes:

Fixes #
Special notes for reviewers:

Does this PR introduced a user-facing change?

None

Additional documentation, usage docs, etc.:

/assign @x893675

/lgtm
/approve


make loading screen more beautiful

before

after

Maybe I don’t fully understand...but what if you have an app with both light and dark theme? It looks like the background will always be Colors.white, disregarding the theme?
Maybe omit the container, since you can‘t get data from Theme.of(context)?

I think it is simple example for understand state management, and now it
only has light theme
Vào 6:15 Th 5, 25 thg 6 2020 nohli<EMAIL_ADDRESS>đã viết:

Maybe I don’t fully understand...but what if you have an app with both
light and dark theme? It looks like the background will always be
Colors.white, disregarding the theme?
Maybe omit the container, since you can‘t get data from Theme.of(context)?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rrousselGit/river_pod/pull/12#issuecomment-649121839,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRATJF2UJ32FKACMMFVN7LRYKCKLANCNFSM4OG777HA
.

This example is in complete rewrite, so that will likely disappear soon.
But thanks!


Remove unnecessary caching

The use of cache decorator was unnecessary and was breaking local dev.
Because the return value was a uri and that cannot be JSON serialized.
Unnecessary as everything used in this method is already cached, hence the caching on top was unnecessary.

Will merge once CI goes green even if not apporved, as this is breaking local dev.


⚠ Redacted has degraded performance

In 9a23a88, Redacted ($RED) experienced degraded performance:

HTTP code: 200
Response time: 11425 ms

Resolved: Redacted performance has improved in bc2fe2a.


Ensure settings sync is resilient to server cache inconsistencies

Right now the cache is the source of truth on the server but there is an extreme edge case where it could become stale. Hopefully it can be fixed on the server-side but I want to make sure if this ever does happen that the client is resilient to this and can self-repair rather than going into a request loop that never fixes itself.
Consider the following scenario:

Machine A: User updates settings
Machine A: Request to write settings is made to the server where the cache becomes stale (database and cache do not match)
Machine B: Does a sync but it doesn't take into account the change Machine A made since the cache is stale
Machine B: User updates a settings, triggering a new request to write latest settings
Machine B: Server returns 412 because it doesn't know about the actual latest settings record

Now the important part: Machine B must request to get the new latest settings without providing an if-none-match header. This will ensure that the cache is skipped and the latest version is fetched directly from the database. After this the new write should succeed and the cache will be repaired.
TL;DR: If a write fails because of a 412, the following request for the latest must not include if-none-match.

Ideally Feb as it should be simple

May I know why the writes from B go in a loop? Are you saying that server cache will not get updated at all with the write from A?

MB asks server for latest, server says everything is fine with 304 (cache is used if it exists)
MB asks with the lastSyncRef as If-None-Match header which is not the latest on the server because MA has written. So, server should not return 304 right?

No, it would be 304 because the ask for latest went to the cache and the write went to the db and they differed. This scenario is to work around if the server gets into state where the cache has stale data, for example if the cache is offline for a few minutes. These things would self heal on the server after 6 minutes anyway as the cache item would expire, also this sort of thing will likely rarely if ever happen but I'm just trying to cover all bases. The self-healing would not occur if regular polling continues or a request loop happens.

I am not comfortable to introduce this debt in client where the server is not returning what it is expected.

also this sort of thing will likely rarely if ever happen but I'm just trying to cover all bases

Introducing such a debt is hard to change in future. So not sure if its a good idea to do this on speculation. It would be good to know why it happened (when it happened) and figure out the root cause and fix it ?

Could we add one of the request assertion tests for this too?

Sure

@Tyriar Not sure how to verify this during testing.

I would suggest @Tyriar to verify

Verified via review, it's pretty difficult to verify this properly. If all syncs go via AbstractSynchroniser.sync then it looks good.


Bumb py-spy to v0.3.7g2

Description
Use new py-spy version which adds Python-specific suffixes
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the relevant documentation.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Bump* lol

@Jongy Should I merge?

Yes


add max_slider_value to config

Notes
For sliders that cannot reach the full analog range, use max_slider_value in the config.yaml to adjust the range from 0 to max_slider_value.
I wrote this because I made the mistake of using Logarithmic Potentiometers, which only reach a value of 900.
Converting a logarithmic value to linear, wasn't ideal for range / sensitivity.
So adjusting the max value in range, works very well to remedy this issue.

Hey there @barnumc, thank you for your interest in contributing to deej.
As per the project's contribution guidelines, I'm currently not accepting PR submissions for deej's main fork. That being said, I encourage you to continue working on these changes to your heart's content in your own fork of the project.


Expected 88 from C header, got 80 from PyObject

command i ran:
from mmdet3d.apis import init_detector, inference_detector
errors:
Traceback (most recent call last):
File "", line 1, in
File "/home/lupeng/pycharmProjects/mmdetection3d/mmdet3d/apis/init.py", line 1, in
from .inference import inference_detector, init_detector, show_result_meshlab
File "/home/lupeng/pycharmProjects/mmdetection3d/mmdet3d/apis/inference.py", line 8, in
from mmdet3d.core import Box3DMode, show_result
File "/home/lupeng/pycharmProjects/mmdetection3d/mmdet3d/core/init.py", line 1, in
from .anchor import *  # noqa: F401, F403
File "/home/lupeng/pycharmProjects/mmdetection3d/mmdet3d/core/anchor/init.py", line 1, in
from mmdet.core.anchor import build_anchor_generator
File "/home/lupeng/pycharmProjects/mmdetection/mmdet/core/init.py", line 6, in
from .mask import *  # noqa: F401, F403
File "/home/lupeng/pycharmProjects/mmdetection/mmdet/core/mask/init.py", line 2, in
from .structures import BaseInstanceMasks, BitmapMasks, PolygonMasks
File "/home/lupeng/pycharmProjects/mmdetection/mmdet/core/mask/structures.py", line 6, in
import pycocotools.mask as maskUtils
File "/home/lupeng/anaconda3/envs/mmdetection3d/lib/python3.7/site-packages/pycocotools/mask.py", line 3, in
import pycocotools._mask as _mask
File "pycocotools/_mask.pyx", line 1, in init pycocotools._mask
ValueError: numpy.ndarray size changed, may indicate binary incompatibility. Expected 88 from C header, got 80 from PyObject
my environment:
cuda=10.1
pytorch=1.7
numpy=1.19.5

我的环境：
cuda == 10.1
但是，我修改了python的版本。
原來python的版本是python==3.7，但是現在我將其修改成python==3.6.1 (conda create -n open-mmdet3d python==3.6.1)
下載pytorch==1.6 (conda install pytorch==1.6.0 torchvision==0.7.0 cudatoolkit=10.1 -c pytorch)
安装mmcv==1.2.6
但是使用pycocotools的时候出现了找不到pycocotools模块的相关问题，解决办法是
(1) pip uninstall pycocotools
(2) pip install mmpycocotools
(3) pip uninstall mmpycocotools
(4) pip install mmpycocotools
参考：https://github.com/open-mmlab/mmdetection/issues/3761

I tried
pip uninstall pycocotools
pip uninstall mmpycocotools
pip install mmpycocotools

but it doesn't work...
My environment

cuda=10.2
Python==3.8
torch==1.7.0
mmcv-full==1.3.0
mmdet==2.11.0
mmdet3d==0.12.0
mmpycocotools==12.0.3

can anyone help me??

改一下环境试一试？试一下我提供的环境

I found that I got error with Python==3.8.8 although got no error with Python==3.6.9