Windows Build Error

Where have you installed OpenCV?
https://github.com/homuler/MediaPipeUnityPlugin#desktopunityeditor-1

Unity Can Run

Where have you installed OpenCV?
https://github.com/homuler/MediaPipeUnityPlugin#desktopunityeditor-1
Now,UnityEditor Can Run, But Build For Exe,Not Find DLL mediapipe_c

this is mu Bundle And  sure Windows Platform

Please tell me your system information in the way like this format and also share the Editor.log.
See also https://github.com/homuler/MediaPipeUnityPlugin/issues/260#issuecomment-910148422 and https://github.com/homuler/MediaPipeUnityPlugin/issues/260#issuecomment-910151456 , since it may be the same issue.

DEBUG (build.py): Running bazel build -c opt //mediapipe_api:mediapipe_proto_srcs
Starting local Bazel server and connecting to it...
DEBUG: /private/var/tmp/_bazel_zl/6de371e6cb5a92efe862db2fb0286ee5/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'com_google_absl' because it already exists.
DEBUG: /private/var/tmp/_bazel_zl/6de371e6cb5a92efe862db2fb0286ee5/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'com_google_protobuf' because it already exists.
DEBUG: /private/var/tmp/_bazel_zl/6de371e6cb5a92efe862db2fb0286ee5/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'com_google_googletest' because it already exists.
DEBUG: /private/var/tmp/_bazel_zl/6de371e6cb5a92efe862db2fb0286ee5/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'com_github_gflags_gflags' because it already exists.
DEBUG: /private/var/tmp/_bazel_zl/6de371e6cb5a92efe862db2fb0286ee5/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'rules_python' because it already exists.
DEBUG: /private/var/tmp/_bazel_zl/6de371e6cb5a92efe862db2fb0286ee5/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'build_bazel_rules_apple' because it already exists.
DEBUG: /private/var/tmp/_bazel_zl/6de371e6cb5a92efe862db2fb0286ee5/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'build_bazel_rules_swift' because it already exists.
DEBUG: /private/var/tmp/_bazel_zl/6de371e6cb5a92efe862db2fb0286ee5/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'build_bazel_apple_support' because it already exists.
DEBUG: /private/var/tmp/_bazel_zl/6de371e6cb5a92efe862db2fb0286ee5/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'bazel_skylib' because it already exists.
DEBUG: /private/var/tmp/_bazel_zl/6de371e6cb5a92efe862db2fb0286ee5/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'pybind11' because it already exists.
DEBUG: /private/var/tmp/_bazel_zl/6de371e6cb5a92efe862db2fb0286ee5/external/tf_runtime/third_party/cuda/dependencies.bzl:51:10: The following command will download NVIDIA proprietary software. By using the software you agree to comply with the terms of the license agreement that accompanies the software. If you do not agree to the terms of the license agreement, do not use the software.
ERROR: Analysis of target '//mediapipe_api:mediapipe_proto_srcs' failed; build aborted: The repository's path is "/usr/local/opt/android-sdk" (absolute: "/usr/local/opt/android-sdk") but a symlink could not be created for it, because: /usr/local/opt/android-sdk (No such file or directory)
INFO: Elapsed time: 2.392s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (22 packages loaded, 22 targets configured)
currently loading: @rules_foreign_cc//foreign_cc/private/shell_toolchain/toolchains ... (4 packages)
Traceback (most recent call last):
File "/Users/zl/data/MediaPipeUnityPlugin/build.py", line 392, in
Argument().command().run()
File "/Users/zl/data/MediaPipeUnityPlugin/build.py", line 109, in run
self._run_command(self._build_proto_srcs_commands())
File "/Users/zl/data/MediaPipeUnityPlugin/build.py", line 51, in _run_command
return subprocess.run(command_list, check=True)
File<EMAIL_ADDRESS>line 528, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['bazel', 'build', '-c', 'opt', '//mediapipe_api:mediapipe_proto_srcs']' returned non-zero exit status 1.
Build Android aar Error

Well, please write your System Information! (cf. https://github.com/homuler/MediaPipeUnityPlugin/blob/d58ab72c8149fce183ce804aef2e218f74ff70dc/.github/ISSUE_TEMPLATE/build-installation-issue.md)
It's created as a Windows issue, but you're running it on macOS now, right?
And apparently you've changed the command but you've not written the new command, so I cannot get what you're trying to do.
What I can tell you now is that maybe you are trying to build for Android, but are failing because the path to the SDK (i.e. ANDROID_HOME) is wrong.

The repository's path is "/usr/local/opt/android-sdk" (absolute: "/usr/local/opt/android-sdk") but a symlink could not be created for it, because: /usr/local/opt/android-sdk (No such file or directory)

So maybe you need to install Android SDK and NDK, and set the path to ANDROID_HOME and ANDROID_NDK_HOME.

好吧，请写下您的系统信息！（参见https://github.com/homuler/MediaPipeUnityPlugin/blob/d58ab72c8149fce183ce804aef2e218f74ff70dc/.github/ISSUE_TEMPLATE/build-installation-issue.md）
它是作为 Windows 问题创建的，但您现在正在 macOS 上运行它，对吗？
显然你已经改变了命令，但你没有编写新命令，所以我无法理解你想要做什么。
我现在可以告诉您的是，您可能正在尝试为 Android 构建，但由于 SDK（即ANDROID_HOME）的路径错误而失败。

存储库的路径是“/usr/local/opt/android-sdk”（绝对：“/usr/local/opt/android-sdk”）但无法为其创建符号链接，因为：/usr/local/opt /android-sdk（没有那个文件或目录）

所以也许你需要安装Android SDK和NDK，并将路径设置为ANDROID_HOME和ANDROID_NDK_HOME。

I have the SDK and NDK installed, but where should I get my repository address from
Modify the

I get this error building on windows whats the problem
INFO: Analyzed target //mediapipe_api:mediapipe_assets (0 packages loaded, 0 targets configured).
INFO: Found 1 target...
ERROR: C:/users/bbc/downloads/mediapipeunityplugin-master/mediapipeunityplugin-master/mediapipe_api/graphs/instant_motion_tracking/BUILD:5:8: Executing genrule //mediapipe_api/graphs/instant_motion_tracking:asset3d failed: (Exit 127): bash.exe failed: error executing command
cd /d C:/_bzl/lbr7m5x6/execroot/mediapipe_api
SET PATH=C:\msys64\usr\bin;C:\msys64\bin;C:\Windows;C:\Windows\System32;C:\Windows\System32\WindowsPowerShell\v1.0;C:\Users\bbc\bin;D:\Programs\Git\mingw64\bin;D:\Programs\Git\usr\local\bin;D:\Programs\Git\usr\bin;D:\Programs\Git\usr\bin;D:\Programs\Git\mingw64\bin;D:\Programs\Git\usr\bin;C:\Users\bbc\bin;C:\Program Files (x86)\Common Files\Intel\Shared Libraries\redist\intel64\compiler;C:\Program Files\Common Files\Oracle\Java\javapath;C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0;C:\Windows\System32\OpenSSH;C:\Program Files (x86)\Intel\Intel(R) Management Engine Components\DAL;C:\Program Files\Intel\Intel(R) Management Engine Components\DAL;C:\Program Files (x86)\Intel\Intel(R) Management Engine Components\IPT;C:\Program Files\Intel\Intel(R) Management Engine Components\IPT;C:\Program Files\Intel\WiFi\bin;C:\Program Files\Common Files\Intel\WirelessCommon;D:\Programs\Git\cmd;C:\src\flutter\bin;C:\Program Files\nodejs;C:\ProgramData\chocolatey\bin;C:\Program Files\CMake\bin;D:\OpenCv\CurrentBin\bin;D:\OpenCv\CurrentBin\bin;D:\AppData\AndroidSdk;D:\AppData\AndroidSdk\platform-tools;C:\Program Files\Java\jdk-15.0.2\lib;C:\Program Files\Java\jdk-15.0.2\bin;C:\msys64\usr\bin;C:\Bazel\bazel.exe;C:\Bazel;C:\Bazel\nuget.exe;C:\Python39\Scripts;C:\Python39;C:\Users\bbc\AppData\Local\Programs\Python\Python39\Scripts;C:\Users\bbc\AppData\Local\Programs\Python\Python39;C:\Users\bbc\AppData\Local\Microsoft\WindowsApps;D:\Programs\Microsoft VS Code\bin;C:\Users\bbc\AppData\Local\GitHubDesktop\bin;C:\src\flutter\bin;C:\Users\bbc\AppData\Roaming\npm;D:\OpenCv\CurrentBin\bin;D:\OpenCv\CurrentBin\bin;D:\AppData\AndroidSdk;D:\AppData\AndroidSdk\platform-tools;C:\Program Files\Java\jdk-15.0.2\bin;C:\Program Files\Java\jdk-15.0.2\lib;C:\Program Files\Java\jdk-15.0.2\bin;C:\msys64\usr\bin;C:\Bazel\bazel.exe;C:\Bazel;C:\Bazel\nuget.exe;D:\Programs\Git\usr\bin\vendor_perl;D:\Programs\Git\usr\bin\core_perl
SET PYTHON_BIN_PATH=C://Python39//python.exe
SET RUNFILES_MANIFEST_ONLY=1
C:\msys64\usr\bin\bash.exe -c source external/bazel_tools/tools/genrule/genrule-setup.sh; unzip -p external/com_google_mediapipe/mediapipe/examples/android/src/java/com/google/mediapipe/apps/instantmotiontracking/assets/robot/robot.obj.uuu.zip > bazel-out/x64_windows-opt/bin/mediapipe_api/graphs/instant_motion_tracking/robot/robot.obj.uuu
Execution platform: @local_execution_config_platform//:platform
0 [main] unzip (18684) D:\Programs\Git\usr\bin\unzip.exe: *** fatal error - cygheap base mismatch detected - 0x180352408/0x18034C408.
This problem is probably due to using incompatible versions of the cygwin DLL.
Search for cygwin1.dll using the Windows Start->Find/Search facility
and delete all but the most recent version.  The most recent version should
reside in x:\cygwin\bin, where 'x' is the drive on which you have
installed the cygwin distribution.  Rebooting is also suggested if you
are unable to find another cygwin DLL.
Target //mediapipe_api:mediapipe_assets failed to build
INFO: Elapsed time: 0.714s, Critical Path: 0.33s
INFO: 6 processes: 2 internal, 4 local.

This problem is probably due to using incompatible versions of the cygwin DLL.

Well, it's totally different problem so I'd rather like you to create a new issue.
You're running bash, that is installed by msys64, but calling unzip that is installed by Git for Windows.

The exact cause cannot be determined by this alone, but

If you're running command from Git for Windows, then use Command Prompt instead.
If you've not installed unzip to msys64, install unzip (see Dockerfile)

Hey when i'm trying to build i have this error:
D:\Projets\Unity\MediaPipeUnityPlugin-0.6.2>C:/Python38/python build.py clean Traceback (most recent call last):   File "build.py", line 392, in <module>     Argument().command().run()   File "build.py", line 360, in __init__     build_command_parser.add_argument('--resources', action=argparse.BooleanOptionalAction, default=True) AttributeError: module 'argparse' has no attribute 'BooleanOptionalAction'

Please use Python >= 3.9.0
https://github.com/homuler/MediaPipeUnityPlugin#prerequisites

Please use Python >= 3.9.0
https://github.com/homuler/MediaPipeUnityPlugin#prerequisites

Indeed, my bad. Thanks you !

win10
the same error
$ python build.py build --desktop cpu --include_opencv_libs -v
Loading:
Loading: 0 packages loaded
DEBUG: C:/_bzl/2kbbcyl2/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'com_google_absl' because it already exists.
DEBUG: C:/_bzl/2kbbcyl2/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'com_google_protobuf' because it already exists.
DEBUG: C:/_bzl/2kbbcyl2/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'com_google_googletest' because it already exists.
DEBUG: C:/_bzl/2kbbcyl2/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'com_github_gflags_gflags' because it already exists.
DEBUG: C:/_bzl/2kbbcyl2/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'rules_python' because it already exists.
DEBUG: C:/_bzl/2kbbcyl2/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'build_bazel_rules_apple' because it already exists.
DEBUG: C:/_bzl/2kbbcyl2/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'build_bazel_rules_swift' because it already exists.
DEBUG: C:/_bzl/2kbbcyl2/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'build_bazel_apple_support' because it already exists.
DEBUG: C:/_bzl/2kbbcyl2/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'bazel_skylib' because it already exists.
DEBUG: C:/_bzl/2kbbcyl2/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'pybind11' because it already exists.
DEBUG: C:/_bzl/2kbbcyl2/external/tf_runtime/third_party/cuda/dependencies.bzl:51:10: The following command will download NVIDIA proprietary software. By using the software you agree to comply with the terms of the license agreement that accompanies the software. If you do not agree to the terms of the license agreement, do not use the software.
Analyzing: target //mediapipe_api:mediapipe_proto_srcs (0 packages loaded, 0 targets configured)
ERROR: D:/msys64/home/pq/mpp/WORKSPACE:302:23: fetching android_ndk_repository rule //external:androidndk: Either the path attribute of android_ndk_repository or the ANDROID_NDK_HOME environment variable must be set.
DEBUG: Rule 'rules_python' indicated that a canonical reproducible form can be obtained by modifying arguments shallow_since = "1564776078 -0400"
DEBUG: Repository rules_python instantiated at:
D:/msys64/home/pq/mpp/WORKSPACE:35:23: in
C:/_bzl/2kbbcyl2/external/rules_pkg/deps.bzl:33:10: in rules_pkg_dependencies
C:/_bzl/2kbbcyl2/external/bazel_tools/tools/build_defs/repo/utils.bzl:233:18: in maybe
Repository rule git_repository defined at:
C:/_bzl/2kbbcyl2/external/bazel_tools/tools/build_defs/repo/git.bzl:199:33: in
ERROR: Analysis of target '//mediapipe_api:mediapipe_proto_srcs' failed; build aborted: Either the path attribute of android_ndk_repository or the ANDROID_NDK_HOME environment variable must be set.
INFO: Elapsed time: 0.155s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded, 0 targets configured)
FAILED: Build did NOT complete successfully (0 packages loaded, 0 targets configured)
INFO (build.py): Building protobuf sources...
DEBUG (build.py): Running bazel --output_user_root C:/_bzl build -c opt --action_env PYTHON_BIN_PATH=C://Users//pq//AppData//Local//Programs//Python//Python39//python.exe //mediapipe_api:mediapipe_proto_srcs
Traceback (most recent call last):
File "D:\msys64\home\pq\mpp\build.py", line 392, in
Argument().command().run()
File "D:\msys64\home\pq\mpp\build.py", line 109, in run
self._run_command(self._build_proto_srcs_commands())
File "D:\msys64\home\pq\mpp\build.py", line 51, in _run_command
return subprocess.run(command_list, check=True)
File "C:\Users\pq\AppData\Local\Programs\Python\Python39\lib\subprocess.py", line 528, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['bazel', '--output_user_root', 'C:/_bzl', 'build', '-c', 'opt', '--action_env', 'PYTHON_BIN_PATH=C://Users//pq//AppData//Local//Programs//Python//Python39//python.exe', '//mediapipe_api:mediapipe_proto_srcs']' returned non-zero exit status 1.
why window compiling need  Android NDK
but the doc don't have the step

@wildfire2015

Either the path attribute of android_ndk_repository or the ANDROID_NDK_HOME environment variable must be set.

I guess you've not set ANDROID_NDK_HOME properly.


[Feature Request]: Task Entry Behavior Modification

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Description:
Currently, when users enter a new task, the tick button is already marked as completed (ticked). This feature request aims to change this behavior so that the tick button is left blank for new tasks. Users should manually tick the button upon completing the task.
Expected Behavior:
When a new task is created, the tick button should be in an unchecked state.
The tick button should only be checked once the user manually marks the task as completed.
Record

[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

I can work on this issue

Sure @rsnj5 . Your point is valid. You can start working on this issue. Keep upstream from the main branch before starting. Thankyou for your contribution.


Hosting Flet web apps in FastAPI

Adding docstrings to
classes/methods/functions will be awesome... don't you think so?


Add boot from USB Stick option

I found that working with the SD Card on Raspberry Pi can sometimes damage the SD Card. (Not related to Minke, just in general, removing re-flashing etc and then snap!)
Is it possible to add the Boot from USB storage option? Easier when experimenting with software to flash to a USB stick and plug in.
https://www.raspberrypi.org/documentation/hardware/raspberrypi/bootmodes/msd.md

Reading about booting on the Raspberry Pi 3B+, it seems to suggest that it would boot of a USB drive without changes. So .. maybe .. flashing the MinkeBox code into a flash drive and plugging it in will "just work"? Have you tried?


Conditional indexing

This is undocumented, but it appears the searchIndexShouldBeUpdated method can be used to determine whether or not a record should be updated in the index. At the moment, I've set this to only happen if the subject, body or slug are dirty. I can't think of any other scenarios where we might want to update it.
As part of this PR, I also added a migration that updates the last_activity_at timestamp for all records in the database. This should have run as part of https://github.com/laravelio/laravel.io/pull/761, but I had removed last_activity_at from the fillable model fields during the review process and didn't update the migration accordingly.

Can we make a command out of the migration? A migration without any schema changes doesn't really makes sense I think.


UI issues.

Is there supposed to be depth here?

Top and left edge is not looking as per the UX.

Check UX for font, size and colour

The shade difference does not look right as per the ‘Note’ area. Also should be top aligned. Too much space between the top of the area and the text. Same with text on empty F&F

After send success, the "View Account" button is not responsive. It takes few seconds to go to Account screen.

Verified this issue on dev app v1.9.5(337)


QoL changes to the queue manager and whats reported

Description
This PR fixes a few issues relating to more verbose reports from the queue managers as to the pass/fail rate of jobs, a verbosity toggle, and you can now just scale up in dask rather than adapt.
Fixes
Status

[ ] Changelog updated
[x] Ready to go

We also need print the first error message in qcfractal-manager --test.

We also need print the first error message in qcfractal-manager --test.

So I chose to print the last error message instead because it was a one liner with nothing to track. I can do first if thats what you would prefer.
https://github.com/MolSSI/QCFractal/pull/234/files#diff-12fe35ea3c88dd77f827cc649ee2df2dR476

@dgasmith
Trying to adjust the server looks like it broke a few other things.

            Can provide more accurate results if the optimization was ill-behaved,
            but pulls additional data from the server and may take longer.
        return_figure : Optional[bool], optional
            If True, return the raw plotly figure. If False, returns a hosted iPlot. If None, return a iPlot display in Jupyter notebook and a raw plotly figure in all other circumstances.

        Returns
        -------
        plotly.Figure
            Description

        Raises
        ------
        TypeError
            Description
        """

        show_spec = True
        if isinstance(specs, str):
            specs = [specs]
            show_spec = False

        if isinstance(entries, str):
            entries = [entries]

        # Query all of the specs and make sure they are valid
        for spec in specs:
            self.query(spec)

        traces = []

        xmin = 1e12
        xmax = -1e12
        # Loop over specifications
        for spec in specs:
            # Loop over indices (groups colors by entry)
            for index in entries:

                record = self.get_entry(index)

                td = self.df.loc[index, spec]
                min_energy = 1e12

                # Pull the dict apart
                x = []
                y = []
>               for k, v in td.final_energies().items():
E               AttributeError: 'float' object has no attribute 'final_energies'

collections/torsiondrive_dataset.py:414: AttributeError

Make UFF and B3LYP-D3 in caps. I have this fixed in my feature branch.

That fixes the first problem locally, but not the second where the td seems to get set to a float?

I am not sure. I will review and merge and patch in another PR.


Key-repeat does not work for letters

On OSX 10.11.3 key repeat works for everything except normal letters.
Expected behavior:
When holding down a letter key (with or without shift) the editor repeats the letter according the rate and delay set for key-repeat in the OS
Actual behavior:
The letter is only output once and the key has to be hit again to repeat the letter. The mouse cursor flashes in sync with the expected key-repeat but nothing happens.

Interesting, works in both Chrome and Safari on 10.10.5 for me.

This also didn’t work for me initially, but…
It‘s probably an OS settings issue. Try doing the same key-holding behavior in the browser’s URL bar. My guess is that it doesn’t work there either.
I had to make a change to my system settings via Terminal. Just paste this line, hit enter, then restart any app that is misbehaving:
defaults write NSGlobalDomain ApplePressAndHoldEnabled -bool false
After this change, the example on the Draft.js homepage works for key-repeat.

 I am using chrome and the omnibar didn't work with key-repeat either. Running your defaults write worked. Thanks @gscottolson :-)
Closing as a non-issue.


Extend djangoSelect2 in js won't work

I am trying to adding own settings to djangoSelect2 function in js.
I create a new js file and include in my own inherited widget's media js, like this
# ajax-select2.js
;(function ($) {
  $(function () {
    $('.django-select2').djangoSelect2({
        placeholder: "Type to search",
    });
  });
}(this.jQuery));

# forms.py
class AjaxSelect2Widget(ModelSelect2Widget):
    class Media:
        js = ["js/ajax-select2.js"]

However, my widget won't work.(It doesn't show the placeholder.)
Need some help, thank you.

Hi @ktshen,
this is a very good question.
Did you know, that form Media on both forms and widgets inherits from super classes. Therefore the original script will be still there unless you do:
class Media:
    extend = False
    js = ['path/to/my/script']

That being said, there are better ways to this. I recommend only adding your own JS script, if you want add functions as options, eg. if you want to change the way select2 renders or parses search results. In all other cases i would just use data- attributes.
class MyForm(froms.Form):
    myfield = forms.ChoiceField(
        widget=Select2Widget(attrs={'data-placeholder', 'Type to search'})
    )

If you really need to change your JS script, make sure to use a different class name:
MyWidget(Select2Widget):
    def build_attrs(*args, **kwargs):
        attrs = super().build_attrs(*args, **kwargs)
        attrs['class'] = 'my_class'
        return attrs

I hope that helps you. Let me know if you have any questions :)

Thank you so much @codingjoe,
I turned out to build my own select2 widget.
However, I still have a question that confuses me a lot. Since I like djangoSelect2 original js settings, so I would like to "extend" it. I thought I could simply pass my own option to $.extend({},options)
(in this case, option is {"placeholder": "Type to search.."} ), why isn't it effective?

HI @ktshen i think it was, but the widget was already initialised, therefore you script didn't have any effect. But good, if you found a way :)
Let me know if there are places where I can improve the documentation.


CVAT projects enhancement suggestions

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

Expected Behaviour

We have several proposals to add some features in CVAT projects

[ ] Necessary to add an option to choose the current status "in progress" or "annotation" regardless of jobs count inside a task. For example, if we deal with a project which has one-job tasks, it's impossible to visually understand and track tasks which have been started.

[ ] Add the "assigned to" button on the main project page.

In this case there is no need to open each task and assign it on someone.

[ ] Develop and option to arbitrarily dump annotations inside a project. Now we have a good option to dump annotations for a whole project (export project dataset), but it isn't enough in cases when there is a need to dump annotations only for a part of completed tasks inside a project, for example. I see it can look like the following solution:
Pressing the "export project dataset" button, a user sees two options: 1. Export full project 2. Choose tasks for dumping.
In the second option, he chooses a format and tasks he's going to dump. Seems, a checkbox near each task will be the most convenient decision.

[x] Add filters to sort tasks inside a project depending on their status (pending, in progress, completed) and assignees.

Context

The mentioned enhancements will allow to improve the current CVAT projects UI and enlarge its functionality.

Another enhancement to the CVAT project that will be useful for my purpose it to have statuses like "Completed" "In Progress" and "Pending" that we currently do for CVAT Tasks.
This will make it easier to filter which CVAT projects are "Complete" or "In Progress"  or not started yet (Pending). A Project can be though of as complete if all tasks/jobs in it are "Complete" and "In Progress" if at least 1 task/job is no longer in "Pending" state.


Some question on build.gradle

Hi Gergely Kőrössy,
Sorry for that! Since I don't find any better channel to contact you to ask about non-project thingy, I decide to open this issues in order able to ping you.
Recently, I wish to fork some Android library projects from GitHub, perform some modification, publish to maven (or jcenter) for my own usage purpose.
So that I can use them by having the following in my build.gradle
compile 'com.yccheok:my-forked-library-v7:24'

After reading https://inthecheesefactory.com/blog/how-to-upload-library-to-jcenter-maven-central-as-dependency/en
I expect I shall saw some similar info in your build.gradle
apply plugin: 'com.android.library'
ext {
	bintrayRepo = 'maven'
	bintrayName = 'fb-like'

	publishedGroupId = 'com.inthecheesefactory.thecheeselibrary'
	libraryName = 'FBLike'

However, I found none. They just seem like a normal build.gradle for non-library project.
May I know what is the secret behind, of publishing your Android-Support-Preference-V7-Fix library?
Thank you very much!
Cheok

@yccheok It's in the preference-v7 module: https://github.com/Gericop/Android-Support-Preference-V7-Fix/blob/master/preference-v7/build.gradle

@Gericop Wow. Thanks for the speedy respond. My bad! I should check more carefully. Thank you very much.


Webpack-dev-server "e.transport is null" browser error

I'm accessing the webpack-dev-server through an nginx proxy from a local HTTPS connection and I'm receiving the error "e.transport is null" on line 2 column 15229 with "...tbeatTimeoutTimer=setTimeout(function(){e.transport.onClose()},this.heartbeatTim..."
Here is my upstream conf:
upstream webpack-7002beeb-55e9-434f-8311-f730bc5f4317 {
    server localhost:8090;
}

Then inside of the vhost:
    location ~ ^/(css|js|fonts)/(.+)$ {
        proxy_pass http://webpack-7002beeb-55e9-434f-8311-f730bc5f4317;
        proxy_redirect off;
        proxy_buffering off;
        rewrite ^/js/webpack-dev-server.js$ /webpack-dev-server.js break;
    }

Am I missing something?  My PHP Ratchet forwarding is working correctly.

If this is still relevant, can you re-open this at the webpack-dev-server repo?


SMTP mail driver won't send emails

Im trying to configure cachethq to send emails to subscrivers, but emails never reach destiny.
this is how my docker-compose.yml configuration
mysql:
  restart: always
  ports:
  - "3306:3306"
  environment:
    MYSQL_ROOT_PASSWORD: password
    MYSQL_USER: cachethq
    MYSQL_PASSWORD: password
    MYSQL_DATABASE: cachethq
  image: mysql:latest
  stdin_open: true

cachethq:
  restart: always
  ports:
  -  "8000:8000"
  environment:
    DB_DRIVER: mysql
    DB_HOST: mysql-server
    DB_DATABASE: cachethq
    DB_USERNAME: root
    DB_PASSWORD: password
    MAIL_DRIVER: smtp
    MAIL_HOST: mail.example.com
    MAIL_PORT: '25'
    MAIL_USERNAME<EMAIL_ADDRESS>    MAIL_PASSWORD: somepassword
    MAIL_ADDRESS<EMAIL_ADDRESS>    MAIL_NAME<EMAIL_ADDRESS>  image: cachethq/docker:latest
  links:
  - mysql:mysql-server
  stdin_open: true

Im using my email server.

Looks like we don't actually setup the requirements for the SMTP driver (postfix). Will need to add this into the base image.

Until we get around to adding postifx to the image, I recommend using something like Mailgun or Amazon SES to provide external SMTP.

May revisit this at a later time, but for now please just use Mailgun for sending emails.

I believe this is still an issue. I've tried versions 2.3.6 - 2.3.9 and it is present in all of them. No emails are being sent, and there is no subscribe button.

Is there a solution for this issue anywhere? thanks for the help in advance.

Is MAIL_DRIVER=smtp still a problem?

Still a problem
smtp is not working on my side (with personal smtp or with outlook)
Connection could not be established with host smtp-mail.outlook.com :stream_socket_client(): unable to connect to tcp://smtp-mail.outlook.com:587 (Operation timed out)


Implement internal image build logic

soma image build will be done with the following procedure:

[x] Copy repository directory to tmpdir
[x] Render proper template file to tmpdir
[x] Run docker image build on tmpdir
[x] Remove tmpdir

#22


tx-generator: Merge reusable code with Cardano.API.IPC

There is some code duplication in Cardano.API.IPC /Cardano.API.InMode and Cardano.Benchmarking.GeneratorTx.SubmissionClient.
In particular the conversion functions between API types and Consensus types.

This overlaps with https://github.com/input-output-hk/cardano-node/issues/3770
This needs an extension of cardano-api/coordination with cardano-api maintainers

Is there any more code exposed (or exposable) in the API that is currently duplicated in the generator?


Error running GreatExpectationsOperator

Describe the bug
When running a great_expectation suite on Google Cloud Composer using BigQuery as a datasource, I'm getting the follwing error:
[2021-07-23 17:40:24,866] {xcom.py:237} ERROR - Could not serialize the XCom value into JSON. If you are using pickles instead of JSON for XCom, then you need to enable pickle support for XCom in your airflow config.
[2021-07-23 17:40:24,866] {taskinstance.py:1457} ERROR - Object of type ValidationOperatorResult is not JSON serializable
Traceback (most recent call last):
  File "/opt/python3.8/lib/python3.8/site-packages/airflow/models/taskinstance.py", line 1113, in _run_raw_task
    self._prepare_and_execute_task_with_callbacks(context, task)
  File "/opt/python3.8/lib/python3.8/site-packages/airflow/models/taskinstance.py", line 1287, in _prepare_and_execute_task_with_callbacks
    result = self._execute_task(context, task_copy)
  File "/opt/python3.8/lib/python3.8/site-packages/airflow/models/taskinstance.py", line 1320, in _execute_task
    self.xcom_push(key=XCOM_RETURN_KEY, value=result)
  File "/opt/python3.8/lib/python3.8/site-packages/airflow/utils/session.py", line 65, in wrapper
    return func(*args, session=session, **kwargs)
  File "/opt/python3.8/lib/python3.8/site-packages/airflow/models/taskinstance.py", line 1896, in xcom_push
    XCom.set(
  File "/opt/python3.8/lib/python3.8/site-packages/airflow/utils/session.py", line 62, in wrapper
    return func(*args, **kwargs)
  File "/opt/python3.8/lib/python3.8/site-packages/airflow/models/xcom.py", line 88, in set
    value = XCom.serialize_value(value)
  File "/opt/python3.8/lib/python3.8/site-packages/airflow/models/xcom.py", line 235, in serialize_value
    return json.dumps(value).encode('UTF-8')
  File "/opt/python3.8/lib/python3.8/json/__init__.py", line 231, in dumps
    return _default_encoder.encode(obj)
  File "/opt/python3.8/lib/python3.8/json/encoder.py", line 199, in encode
    chunks = self.iterencode(o, _one_shot=True)
  File "/opt/python3.8/lib/python3.8/json/encoder.py", line 257, in iterencode
    return _iterencode(o, 0)
  File "/opt/python3.8/lib/python3.8/json/encoder.py", line 179, in default
    raise TypeError(f'Object of type {o.__class__.__name__} '
TypeError: Object of type ValidationOperatorResult is not JSON serializable

To Reproduce
Steps to reproduce the behavior:

Dag File
GE config file
GE suite

Expected behavior
It should run and push the result to XComs
Environment (please complete the following information):

Cloud Composer Verison: 1.17.0
Airflow Version: 2.0.2
Great Expectations Version: 0.13.22

Additional context
If I set do_xcom_push to False it works but I can't fetch the results after execution.

i've faced the same issue when using GreatExpectationsOperator and pandas s3 datasource (but i'm using GCS, not S3)

For Someone in the future needed help
Add return_json_dict=true work for me
ge_data_context_root_dir_with_checkpoint_name_pass = GreatExpectationsOperator(
        task_id="ge_data_context_root_dir_with_checkpoint_name_pass",
        data_context_root_dir=ge_root_dir,
        checkpoint_name="test-checkpoint",
        return_json_dict = True,
    )

@kmandagie  : I am trying to use Great Expectations Operator with Postgres datasource and facing the same issue. I tried AIRFLOW__CORE__ENABLE_XCOM_PICKLING=true ( this seems to be deprecated in Airflow 2.0 )
return_json_dict = True,
None of these worked.

@keerthiis : that setup works for me in airflow version 2.2.5 and version 2.3,
with this package version
great-expectations==0.15.5
airflow-provider-great-expectations==0.1.4
I don't know about other version


display an array in collapsible view

Hi,
I have an array like this
[
{
"category_id":"169",
"name":"Computer & Tablets",
"children":[
{
"category_id":"698",
"children":[
{
"category_id":"1",
"name": "IPhones"
},
{
"category_id":"2",
"name": "Smart Phones"
},
{
"category_id":"3",
"name": "Basic Phones"
}
],
"name":"Computer & Laptop Accessories",
},
{
"category_id":"786",
"name":"Computer Peripherals",
"children":[
{
"category_id":"1",
"name": "Charger"
},
{
"category_id":"2",
"name": "Charger Connector"
},
{
"category_id":"3",
"name": "Datacable"
}
],
},
{
"category_id":"680",
"name":"Computers",
"children":[
{
"category_id":"1",
"name": "Laptop"
},
{
"category_id":"2",
"name": "Computer Pheriperals"
}
],
}
],
"thumb":"Computers.png"
},
{
"category_id":"1027",
"name":"Mobiles & Accessories",
"children":[
{
"category_id":"153",
"name":"Mobiles",
"image":" mobile1.png"
},
{
"category_id":"178",
"name":"Mobile Accessories",
"image":" audio.png"
}
],
"thumb":" Mobiles.png"
},
{
"category_id":"740",
"name":"Personal & Health Care",
"children":[
{
"category_id":"631",
"name":"Health Care",
"image":" shopbycategory/healthcare.png"
},
{
"category_id":"727",
"name":"Personal Care",
"image":" shopbycategory/personalcare.png"
}
],
"thumb":" Health Care.png"
}
]
how can i display this in react native collapsible view ? i am new to react native can u pls help me with that?

I want to open single item at a time and i want to show icons at right hand end side.. can u please help me with that

I want to open single item at a time and i want to show icons at right hand end side.. can u please help me with that


Incorporate Forklift logo into upstream builds (masthead? welcome page?)

Just a nice to have. Logos are here: https://github.com/konveyor/community/tree/main/brand/logo

Closing this since we added the logo to the About page.


finish threebot server factory

need to wait for #709  (openresty)
todo

[ ] test chatbot
[ ] authentication through openresty (need add on there probably) for chatbot
[ ] packaging (kristof)
[ ] integration of gundb client (need auth with master passwd)
[ ] integration graphql client (need auth with master passwd)
tbd

i have error when get openresty to start it ( and test is fail j.servers.openresty.test() )
jumpscaleX ( development_jumpscale)
DigitalMeX ( development_jumpscale)

i passed the error above but have problem

it can't serve index.etlua if i add index.html it works good  in /sandbox/var/web/html/chat
@abom  any suggestion for that i think the problem in config somehow

That's a configuration problem for sure, you need use openresty directives like content_by_lua_block if you running a lapis app, not serving etlua files as html files.
See https://leafo.net/lapis/reference/getting_started.html#creating-an-application/nginx-configuration

chatbot doc : https://github.com/threefoldtech/digitalmeX/blob/development_jumpscale/DigitalMe/servers/threebot/README.md
branch development_jumpscale


Make generated copyright year be "now"

Always use the current year in generating output files, rather than
trying to base is on the modtime of the script or input, as that can
vary depending on the ability of the local OS to keep those accurate.
Fixes #10744

When I originally set up those scripts, I used the timestamp on the input file(s). I know realize that was too fragile, for systems that muck with or otherwise don't preserve timestamps very well.
This should backport easily.

I have marked this as urgent, as it can affect anyone using make update.

Merged.
339638b586e77c189e8294f931b99b03d1f08dbd Make generated copyright year be "now"


Add Annotation Toolbar Component

Overview
This PR adds an annotation toolbar component for drawing points, rectangles, and polygons, and for adding their labels through popups.  The label name field in the popup has autocomplete feature based on labels already created.
Editing and/or deleting labels, shapes, and geometries will be implemented in a future PR - which probably not in this component but on the sidebar
The import button now renders a hard-coded feature collection, which is passed to the component to be combined with other drawn shapes.  Hope this could result in a quicker process when adding the real import feature.
The export button now logs all annotations to the console in geojson format, which is for future developments.
Checklist
~- [ ] Styleguide updated, if necessary~
~- [ ] Swagger specification updated, if necessary~
~- [ ] Symlinks from new migrations present or corrected for any new migrations~
Demo
Add a new shape and label (with autocomplete).

Notes

The stylesheet is located at app-frontend/src/app/components/map/annotateToolbar/annotateToolbar.scss. @designmatty edit, move, remove them as you see fit.
This PR has two commits. The first has its own PR in here.  The second contains the updates for this PR.
Validations on label fields will be a task for the next round.

Testing Instructions

Go to /projects/edit/:project_id/annotate

Click on import to load the hard-coded feature collection and see if it renders.  OR don't import it and go to the next step.

Click on any shape type button on the annotation tool to draw shapes and add labels.

Create multiple labels with same names to test if the autocomplete works as intended.

Click on 'Cancel' on the popup to see if it will delete the shape from the map.

Click on "Import" button in the middle of drawing polygons, if you have not done so before to see if it adds the hard-coded shape in and does not interfere any drawn annotations.

Click on "Export" and check if all drawn annotations are logged there in geojson with correct labels.

Closes #2277

Another comment, the popup's z-index is lower than the toolbar so that if you click close to the toolbar, the popup is under it. The z-index of the popup should be higher.


Add a router for pages in the app

Summary
We need to have multiple pages in our app (data entry, data visualization are the two primary ones). To support this, let's add a router for handling pages
Work Scope

[ ] Create a router
[ ] Create an example data visualization page
[ ] Migrate existing data viz to that example page

Let's not close issues until they are merged into main with a relevant pull request @TheLumpiest

This is actually done now


controller_impl::on_irreversible called multiple times with same block

Occasionally on_irreversible is called multiple times with the same block which means the fork_db is signaling it multiple times. It seems to happen at the start of every chain and occasionally when switching forks.
Under normal non-forking operation it doesn't seem to happen.
For now the handler simply suppresses it, but we should probably avoid signaling it multiple times to begin with.

The code in question has been refactored significantly, and this issue is no longer relevant.  As of 9463efdfe2672e3058ac5701f01fbdd5525afada, controller_impl::on_irreversible no longer exists.


add scan jobs in CI

Coverage increased (+0.05%) to 54.853% when pulling 1a1b9a59a04f025b5d0187647bba10d9a2d0f43d on wy65701436:add-scan-job-log into 6e6bb80b872ce1cdd1438bbcc7767372c7b1fa68 on vmware:master.


Correct localization_3d.launch

Also, be consistent with 2D and with documentation.
ISSUE=https://github.com/googlecartographer/cartographer/issues/1056

@wally-the-cartographer merge


feat(LENS-1390): add support for incremental catalog updates in Java SDK

Added the capability to use the stream mechanism in the Push API to enable incremental updates as per https://docs.coveo.com/en/l62e0540

@y-lakdar Can you merge this for me please as I'm not authorised to. Thanks

@y-lakhdar can you merge this in for me please as I'm not authorised to. Thanks.


Map over all chapters to grab and save images

AC:
Will map over all chapters and save the collection of images on each page as a stand alone PDF with title of Chapter as file name
Dev Notes:
Can start at Chapter 0 and continue to go to 'next' chapter to pull data until 'next' is no longer populated

Completed with the addition of commit ea3bac8


[W5.7] [W16-1] ERIC YANG JIA WEI

For your review

Also, I think the commit messages can be more explanatory.

PR should be present tense. Do not use past tense such as 'added'.

Good job 👍 you can close the PR now

Approved by TA


Reveal removes all listeners on init

When the reveal JS component is initialized on a specific element, all listeners in the .reveal namespave are removed, even those that were added by the user.
The culprit is the .off('.reveal') on this line: https://github.com/zurb/foundation/blob/master/js/foundation/foundation.reveal.js#L50
That causes code like this to fail:
var elem = $('<div data-reveal />');
$('body).append(elem);

// Attach a listener
elem.on('opened.fndtn.reveal', function () { console.log('Reveal was opened'); });

 // This causes an init, which clears all listeners on elem
elem.foundation('reveal', 'open', { url: 'http://example.org' });

If you have a solution, a PR would really help people out. We are working on a release for next week.

I don't have a solution unfortunately. I worked around the problem by manually initializing the reveal before I add my own listeners.
I suspect that init is only called once for each element, even if you call $(elem).foundation('reveal') multiple times. If that is the case, the .off('.reveal') can probably simply be removed.
But I'm not sure that's the case. https://github.com/zurb/foundation/commit/711a594e3325dca941d9e865490c6870e8980c4a seems to indicate it can be called multiple times (but that was Foundation 4). If that is the case, you should keep a list of all registered event handlers, and on subsequent inits, only remove the listeners that were added during the previous init, instead of removing all listeners.

@sandermarechal Typically, those event listeners are placed after Foundation has been initialized in the document. Alternatively, your callback function can be passed as an argument in the settings for reveal. Is there a specific reason for this particular use case?

Yes, DOM-manupulating libraries such as React.js, Ractive and many more. They all cause new elements to be added to the page. In that case, events may be added before the element is initialized. Foundation shouldn't throw away those listeners.

I agree with @sandermarechal , as I'm having the same problem of discarded event listeners after Foundation's init.


vpu acceleration not working on kirkstone for a i.mx8 mini

Hi guys,
On kirkstone (cog --version 0.14.1 (WPE WebKit 2.38.2)) , when reproducing a video(.mov) with cog, the video is too slow (audio is correct at normal speed). Using gstreamer, the video is reproduced correctly.
If using an older version of cog (from a gatesgarth build), video play works fine (cog --version 0.9.90 (WPE WebKit 2.32.1))
My relevant sha1 layers are:
meta-yocto-bsp       = "HEAD:d3e378397395542179a8497fbff4b610f6f7e802"
meta-oe
meta-python
meta-networking
meta-webserver       = "HEAD:1f31570d0795da90083d1dbf28127c90908e30ee"
eta-freescale       = "HEAD:3f5fc5810cfb7be61b487ade0a7ecfaeafa3d2aa"
meta-webkit          = "HEAD:eb31f54387bfd309ec3e936b5f19febaac8045bb"
I am using a kernel 5.15 version, but I also see the same problem using a 5.4 kernel version.
Maybe is needed to start the video with some specific argument? I am just starting the video as "cog <path_to_file.mov>".
I also realize that with this kirkstone version, the mouse pointer is lost.
In the previous version that I used, (gatesgarth, cog 0.9.90, WPE WebKit 2.32.1), it works.
Is there any way to enable full debug to deep into the issue?

It was a memory problem on my device allocating the cma memory


New package: SharedMATLABEngine v0.1.1

Registering package: SharedMATLABEngine
Repository: https://github.com/jonniedie/SharedMATLABEngine.jl
Created by: @jonniedie
Version: v0.1.1
Commit: a0d5832966ff9ee72ac2e08799491c696432481e
Reviewed by: @jonniedie
Reference: https://github.com/jonniedie/SharedMATLABEngine.jl/commit/a0d5832966ff9ee72ac2e08799491c696432481e#commitcomment-42442248

@jonniedie ^

@jonniedie Can you please re-trigger Registrator? This will fix the merge conflict.
[noblock]


Business case not calculating correctly for LES#id=332

Delayed job is having a problem trying to calculate the business case with a merit curve.
Tried to price a value collection containing 0 values with a curve containing 35040
/var/www/etmoses/releases/20151019103323/app/models/market/curve_tariff.rb:14:in `price_of'
/var/www/etmoses/releases/20151019103323/app/models/market/payment_rule.rb:28:in `call'
/var/www/etmoses/releases/20151019103323/app/models/market/relation.rb:27:in `price'
/var/www/etmoses/shared/bundle/ruby/2.1.0/gems/activesupport-4.1.8/lib/active_support/core_ext/enumerable.rb:22:in `map'
/var/www/etmoses/shared/bundle/ruby/2.1.0/gems/activesupport-4.1.8/lib/active_support/core_ext/enumerable.rb:22:in `sum'
/var/www/etmoses/releases/20151019103323/app/services/finance/business_case_calculator.rb:36:in `row'
/var/www/etmoses/releases/20151019103323/app/services/finance/business_case_calculator.rb:26:in `block in cells'
/var/www/etmoses/releases/20151019103323/app/services/finance/business_case_calculator.rb:21:in `map'
/var/www/etmoses/releases/20151019103323/app/services/finance/business_case_calculator.rb:21:in `cells'
/var/www/etmoses/releases/20151019103323/app/services/finance/business_case_calculator.rb:10:in `block in rows'
/var/www/etmoses/releases/20151019103323/app/services/finance/business_case_calculator.rb:9:in `map'
/var/www/etmoses/releases/20151019103323/app/services/finance/business_case_calculator.rb:9:in `rows'
/var/www/etmoses/releases/20151019103323/app/services/finance/business_case_creator.rb:16:in `financials'
/var/www/etmoses/releases/20151019103323/app/services/finance/business_case_creator.rb:10:in `calculate'
/var/www/etmoses/releases/20151019103323/app/jobs/business_case_calculator_job.rb:12:in `perform'

@antw is this by any chance fixed with 95a9c80 ? :pray:


[newrelic-infra] Update to 1.7.0 and add OWNERS

Signed-off-by: JF Joly<EMAIL_ADDRESS>What this PR does / why we need it:
This PR

updates the New Relic infrastructure daemon set to version 1.7.0.
adds a OWNERS file to this chart

Note to reviewers
We, New Relic, are willing to keep the helm chart up to date with the latest version of our daemon set; to do so more efficiently, I'm adding a OWNERS file with New Relic people + @rk295 who has been active in maintaining this chart.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x ] DCO signed
[ x] Chart Version bumped
[ x] Variables are documented in the README.md

@rk295 @areina @douglascamata Could you please review this PR?

/lgtm

Gah, I thought I had out an OWNERS file in this chart 🙁. Approved from my point of view.

I'm adding a OWNER file now

I added a OWNERS file and update both the PR description and the head comment.
I hope that clarifies the intent which is to keep this chart updated.

/assign @lachie83 would you be able to look at this PR please? I have added a OWNERS file so that we don't need to bother you in the future :-)

/approve

/assign @scottrigby

@scottrigby Would you please take a look at this PR? Thanks for your help!

Please rebase your changes to latest master.

@verwilst done, rebased and ready for merge!

/lgtm

I invited everyone as collaborator. That's necessary for OWNERS files to work.

Thanks @unguiculus I added the Owners in Chart.yaml.

/ok-to-test

/lgtm


Revert "feat(container): update image ghcr.io/onedr0p/home-assistant ( 2024.9.3 → 2024.10.1 )"

Reverts adampetrovic/home-ops#689

Reverting due to: https://github.com/home-assistant/core/issues/127709


[WIP] disable ITW for x86_32

waiting CI job result  to approve and testing artefacts
Expecting ITW no present anymore on MSI x86-32 but always present on MIS x64

https://ci.adoptopenjdk.net/job/build-scripts/job/release/job/test_installer_windows/11/

LGTM, I have looked at msi produced for jre hostpot 32 and 64 from job https://ci.adoptopenjdk.net/job/build-scripts/job/release/job/test_installer_windows/11/

I have downloaded the artifacts from https://ci.adoptopenjdk.net/job/build-scripts/job/release/job/test_installer_windows/11/ and confirmed:

IcedTea Web appears as install option for x64 jdk and jre, but not for x86-32 jdk and jre
There is no javaws.exe in the installed x86-32 jdk and jre
Application https://docs.oracle.com/javase/tutorialJWS/samples/uiswing/AccessibleScrollDemoProject/AccessibleScrollDemo.jnlp launches OK when choosing the javaws.exe as the launch application for either an installed x64 jdk or x64 jre


Clean up data after the tests complete.

Closes #311.
Checklist

[ ] Unit tests
[ ] End-to-End tests
[ ] Code coverage checked
[x] Coding standards
[ ] Error Handling
[ ] Security/performance considered?
[ ] Deployment changes considered?
[ ] README updated

Can you also remove any data on external? The issue with the data is both incomplete data on IntSchema and ExtSchema causing either errors when Async worker runs or errors when Internal web tries to view incomplete claim data copied across.
Also please delete for all e2e tests excluding the first-time-* ones.

Closing this PR for now. We have decided to clean up all end-to-end test data.


Command-G does not show/hide servers panel

Issue Type: Bug
The Cmd-G shortcut only show the servers panel, but does not hide the servers panel.
Recommendation

Make these shortcuts show and hide panels rather than only show them.

Azure Data Studio version: Azure Data Studio 1.0.0 (cab8f3e37c9ac4367e0dacef5185da08bd8021c2, 2018-09-20T21:41:50.803Z)
OS version: Darwin x64 17.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7920HQ CPU @ 3.10GHz (8 x 3100)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
16.00GB (7.94GB free)

Process Argv
/Applications/Azure Data Studio.app/Contents/MacOS/Electron

Screen Reader
no

VM
0%

Extensions (4)

Extension
Author (truncated)
Version

agent
Mic
0.33.0

import
Mic
0.2.0

profiler
Mic
0.2.0

sql-vnext
Mic
0.6.9

Hi @gengelbms - thanks for logging this...many moons ago :)
We're going through the backlog and @lewis-sanchez and I reviewed this request, and we also looked at the list of shortcuts within the Command Palette (type keyboard shortcuts to see everything).
Within keyboard shortcuts, we have CTRL + SHIFT + D documented as View: Show Connections, and CTRL + B documented as Toggle Primary Side Bar Visibility.
If you are anywhere in ADS and use CTRL + SHIFT + D, it changes focus to the connections side bar.  Where CTRL + B opens or closes whatever sidebar you are in (Connections. Notebooks, etc.).  This behavior is by design, and my suggestion would be to use CTRL + B to open/close the side bar, no matter where you are.
Hope that helps!


Changed image to reflect UI.

I changed the image named windows-19-1.png to reflect changes in UI.

I'm changing the image, so I'm closing the pull request.


Corrupted zip file when exporting product

Support reference number
came across when reproducing the scenario from the below ticket
https://jira.commercetools.com/browse/SUPPORT-2920
Description
When there is a attribute in the same name as a keyword of our platform, for ex. "productType", then performing product export as a csv, outputs an corrupted zip file. On opening this zip file, it creates another file with extension .cpgz.
When performing the same product export to get an excel we get an error "There are duplicate header entries"
Expected Behavior
Same error as we get when exporting as excel.
Current Behavior
Corrupted zip file, which on opening creates file with extension .cpgz

As this is caused by a faulty data structure, there's not much that can be done from here
fyi @VasumathiJayakumar


Investigate self-contained metadata and source resolvers

As discussed briefly here #189

How about at startup we create a csproj like we do, and keep the state in the resolver - then each time a resolve is invoked in the resolver by the compiler, we will check if a given entry is part of the state (== part of csproj). If yes, nothing, if no, we update the existing csproj with the new entry and run restore again.
For performance reasons we can combine both the current approach (to not generate n calls to dotnet restore at startup) and the state-based approach which should kick in when compiler resolves the directives.
I think in either case a change in Roslyn may be required because a resolver at the moment can only resolve a reference into a single PortableExecutionReference.

That might work. We could probably also simply project the state of the resolver so that we can continue to do the PortableExecutionReference hack until it is resolved in Roslyn. I need to think about this for a while :) For #load it should be doable indeed. I'll dive into this once 0.16.0 is out the door 😄


Need a way to force validation of all text via IonReader

Recent changes to the stack use newer IonTextReader implementation with speed-optimized behavior when skipping over values that can fail to detect some syntactic or semantic errors.  This is a change in behavior from earlier releases.
I think its important to at least provide the option to do full validation, otherwise the same input may throw exceptions on some use cases and not on others. I'd prefer such correctness enforcement to be the default, but can see the value of optional optimization.
Imported from ION-161,IONJAVA-159

Specifically, using next() is insufficient to deeply parse a values (e.g. integer) so malformed data requires using the appropriate APIs to force the parsing today.


purge: fix purge of lvm devices

using shell module seems to be the only way to make this task working
on rhel based distribution AND debian based distributions.
on ubuntu, using command ansible module fails like following
(not due to sudo usage or not):
ok: [osd1] => changed=false
  cmd: command -v ceph-volume
  failed_when_result: false
  msg: '[Errno 2] No such file or directory: ''command'': ''command'''
  rc: 2

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1653307
Signed-off-by: Guillaume Abrioux<EMAIL_ADDRESS>

perhaps it's better to consider using which instead of command -v to avoid using shell ansible module.
@dsavineau any thought?

jenkins test pipeline


webSocketNotConnected

WalletConnect 1.6.10
Starscream 3.1.2

DApp WebSocket Always webSocketNotConnected

add PROJECT_ID to the configuration file

Already added

https://github.com/WalletConnect/WalletConnectSwiftV2/issues/739
can connect, but not support iOS 14

got the same error
SDK version: 1.6.11
Starscream <IP_ADDRESS>

@lilyDong are you connecting from China?

@lilyDong are you connecting from China?

Yes. And I use relay.walletconnect.org , not  relay.walletconnect.com. When I don't use any vpn, it's ok. But there will be an error "webSocketNotConnected" when I use vpn. For our app, some logic needs vpn.

SDK version 1.6.14. I can't connect websocket when I am in China without any VPN, but can connect when I use VPN.


Add -DSWIFT_RUNTIME_ENABLE_COW_EXISTENTIALS=1 when compiling the front end

This should fix building a --debug-build
A debug build fails tests because the SWIFT_RUNTIME_ENABLE_COW_EXISTENTIALS macro is not enabled when building frontend files. For some reason that only affects DEBUG builds.
./utils/build-script --release-debuginfo --debug-swift

@swift-ci Please smoke test and merge

Thanks!


Raised PHP7 cphalcon branch to the official final LTS 3.0.x

Phalcon 3.0.0 final (LTS) released
https://blog.phalconphp.com/1
2.1 is dead, all hail 3.0
As mentioned above, 2.1 will not be fully backwards compatible. As a result, we are changing the version number to 3.0.

Thank you


.NET Core 2.1.1 doc updates

The SDK dlc links at https://download.microsoft.com/download/D/0/4/D04C5489-278D-4C11-9BD3-6128472A7626 aren't working. Just mentioning in case it isn't a known issue.

Nice, updating homebrew casks now


Full 1.16 support

This PR ports all the features to the 20w20a features to 1.16-pre5.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: marcus8448:x: hYdosYou have signed the CLA already but the status is still pending? Let us recheck it.

just going to add some bug fixes here and there


Wrong wallet appears to pay the fee for claiming IOT

Describe the bug
I have two wallet accounts. Each owns a couple of Hotspots. I claimed all the IOT/MOBILE associated with each account and that seemed to work fine. Except that one wallet paid the SOL fee for all the transactions.
To Reproduce

Select a wallet account.
Go to Collectibles tab.
Claim all rewards.
Go back to $ tab and select the other wallet account.
Go to collectibles tab.
Claim all rewards.

Expected behavior
I expected that all IOT and MOBILE would be claimed. It was. I expected that each wallet account would pay the fee for claiming rewards for that wallet. (But, one wallet account paid all fees.)
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):

Device: iPhone13 mini
OS: 16.4.1
App Version: Not sure, either 2.0.2, 2.0.4, 2.0.5

Additional context
If needed, you can contact me regarding transaction details.

Try updating to v2.0.7. This should be fixed

Was that a known issue that has been fixed?  I have upgraded to that version, but haven't done a transaction yet.


is there a trick to using the classNamePrefix parameter in the maven plugin?

When I set <classNamePrefix>test</classNamePrefix>, it has no effect on my generated classes.  Am I doing it wrong?

nevermind. was using 0.4.14; problem was resolved by pulling most current version.


Support for acpid protocol

Is your feature request related to a problem? Please describe.
None of the existing protocols nor the ones planned are able to detect speakers getting plugged into an audio jack.
Is your feature request related to a way you would like Akri extended? Please describe.
No support for using devices plugged into an audio jack. Basically a new protocol for acpid should be added.
Describe the solution you'd like
Ability to use speakers/headphones/microphones plugged into an audio jack using the acpid protocol.
Describe alternatives you've considered
As of now one could only use usb speakers with udev.
Additional context
Still confused why I can't simply just mount /dev/snd
Here is the output of a test I ran which involved unplugging and plugging some headphones in using acpi_listen
kelly@rocky:~$ acpi_listen
jack/headphone HEADPHONE unplug
jack/microphone MICROPHONE unplug
jack/headphone HEADPHONE plug
jack/microphone MICROPHONE plug

Hi @kferrone, even if plugged in via the headphone jack, I would expect the speakers to still be a part of the Linux device file system and therefore discoverable via udev. Like you said, your speakers may be under /dev/snd in which case you could use the udev rule SUBSYSTEM=="sound". But I am assuming there are other sound devices there besides your speakers, so you may need a more specific rule.  Determining the udev rule has certainly been tricky

We have discovered the /dev/snd directory is special and the entire folder is needed unlike other devices in /dev.
Well at least all devices related to at least one sound card and the seq and timer as well.
Also the only true 'device' for sound is the sound card and anything you plug into the Audio jack is a device connected to a device, a meta device I guess.
Basically UDEV would only ever see you un/plugging the sound card, not speakers through the audio jack. Therefore acpid would be needed to discover anything connecting to the audio jack.
Another issue with sound and UDEV is using HDMI or USB for audio. Basically UDEV would trigger from USB and HDMI but how would you know it is an audio device? You would still need to mount dev/snd to play any sound out of it, probably would need to mount the device for USB or HDMI as well.

Thanks for confirming that udev doesn't fit the scenario of audio jack or determining specific sound devices! ACPID sounds like a good protocol, especially since it captures other events too besides connecting to the audio jack such as the ones you listed in Slack ( sleep/power button, power cables getting plugged in or out). My question with this protocol is determining what would need to be mounted into brokers. For example, a jack/headphone HEADPHONE plug ACPID event would need /dev/snd mounted by some other event may need a different device directory mounted.

Further discussion on ACPID as a protocol can be referenced from this thread in Slack.

Further discussion on ACPID as a protocol can be referenced from this thread in Slack.

@kferrone , is this a scenario that should be prioritized?

Maybe. I just discovered it is what is needed to discover headphones/aux inputs. Would be a nice addition to Akri. I have no idea how to implement such a thing though. I only needed it for a home project so I could easily plug speakers into any node and have snapcast client run as the broker.

I could see us using this rust ALSA library to search for devices and then if the desired one is found, then the entire /dev/snd directory is mounted into the broker. It dont think it would be too complex of a discovery handler and could work off our template. If you are interested in implementing it, I'm certainly happy to help. It also doesnt have to be in Rust, rather it could be in any language so long as it implements our gRPC interface.


Create hexagon logo

I crated a Logo, but am unsure whether to upload it to the images folder and then change the website code, or to remove the existing logo file and upload the new logo with the logo.png name....?

Just delete the logo file - will be fine I suppose.


Add support for virtual wan s2s module

add https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/vpn_site to create virtual WAN based S2S IPSec connections, as well to add custome IKEv2 policies.

hi @geraldfehringer this is currently under dev and should be here in June release

Coming with 5.4.0


Authorization Implementation

Implement authorization functionality base on organization schemas

Pushed first implementations:

https://github.com/Solidalert/api/pull/16
https://github.com/Solidalert/api/pull/18

#18

#16


BUG: NavBar in cart.html

Cant see the navbar in the cart.html.

please assign this to me.
@sunny0625 @khushi-joshi-05

This issue is already solved.
Try to contribute to different issues.


can't read Characteristic

Prerequisites

[X] I checked the documentation and FAQ without finding a solution
[X] I checked to make sure that this issue has not already been filed
[X] I'm sure that question is related to the library itself and not Bluetooth Low Energy or Classic in general. If that so, please post your question on StackOverflow.
[X] I'm running the latest version

Question
got this error when I try to read
[BleError: Characteristic f0e60004-4b8c-41d8-a61c-eeafa4eb6077 read failed for device 94:54:93:17:63:73 and service f0e60001-4b8c-41d8-a61c-eeafa4eb6077]

React native: 0.73.3
react-native-ble-plx: 3.1.2

Looks like you're using the same ID for the service and the characteristic

Hi @blood-romantic
Could you provide more context? I would need information on how your ble service looks like and how you try to write to it. The more code and context will you provide the easier it will be to find the issue.

Hey @blood-romantic
If your problem is solved can you please close the issue?
It sparks joy for maintainers ✨

Hi @blood-romantic
Do you still need help with the issue?

may i know how the issue is solved

even i getting the dame issue
Error reading characteristic: [BleError: Characteristic 233bf001-5a34-1b6d-975c-000d5690abe4 read failed for device 00:09:1F:87:DF:1B and service 233bf000-5a34-1b6d-975c-000d5690abe4]
ERROR  Error reading characteristic: [BleError: Device 00:09:1F:87:DF:1B is not connected]
ERROR  Connection failed: [TypeError: Cannot read property 'charCodeAt' of undefined]


Bug located in ... q3d.export_matrix_data()

Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

Description of the bug
In 24R2 Q3D, the call to native API ExportMatrixData is missing <pass> argument, so the arguments after that are set incorrectly.

Steps To Reproduce

q3d = Q3d(...)  #Open a q3d project with DCRL and ACRL solutions
q3d.export_matrix_data("results.csv", "DC RL, AC RL", matrix_type="Maxwell,Spice,Couple", export_ac_dc_res=False, precision=5, field_width=8, use_sci_notation=False)
You will see in the csv ac_dc_res is there and precision is 8. Arguments are shifted.

Which Operating System are you using?
Linux
Which Python version are you using?
3.8
Installed packages
pyaedt and dependent packages

Hi @gene820715 ,
The PyAEDT method export_matrix_data is wrapping the AEDT API method ExportMatrixData:
https://github.com/ansys/pyaedt/blob/main/src/ansys/aedt/core/q3d.py#L424
I think the values passed to the AEDT API methods are correct, maybe you missed passing any of them. PyAEDT methods try to reduce the number of needed arguments if possible, and you probably pass the same order as the AEDT API method, but the PyAEDT method has different orders.
I encourage you to use the best practice of passing the name of the argument, then the order does not matter:
export_matrix_data(file_name=...., problem_type=...., variations=..., setup=..., ...)
Attached is the documentation:
https://aedt.docs.pyansys.com/version/stable/API/_autosummary/ansys.aedt.core.q3d.Q3d.export_matrix_data.html

@gene820715 You are right.
Let me investigate what this parameter is doing, and I will create a new pull request.


Show dedicated placeholder message in empty channel feeds when there are only muted topics

When a channel feed has messages only in muted topics, the default "There are no messages here." placeholder message can be confusing. In this situation, we should use a dedicated message (formatted like the first line of the default message):

This feed is empty, because you have muted all the topics in this channel.

If there are no messages in the channel at all (muted or unmuted), we should just show the default message.

CZO thread

Hi @alya,
I would like to work on this issue, but I understand my previous PR is still pending review. Once that PR is merged, I would like to claim this issue and continue contributing. Please let me know if that’s possible.

@zulipbot claim

@zulipbot claim

@userAdityaa
As mentioned earlier, I have been actively working on this issue. It would be helpful if you could unassign yourself from it. Thanks for your understanding!

#31609 does not appear to have useful work towards implementing this. The message_list_data data structure should, possibly with API extensions, make it easy to check this -- it'll be a case where _items is empty (well, we already know that if we're displaying an empty feed message) but _all_items is not.

@zulipbot claim

@zulipbot claim

@radagon12 Are you still working on this issue? I'd like to work on this issue.

/abandon

@zulipbot abandon

@radagon12 Are you still working on this issue? I'd like to work on this issue.

You can take the issue

@zulipbot claim


How to Track Y position of floading panel view

i want to add button on top of floating view that floats with the view

Hi @forTJ555,
I do something similar in my apps, basically what I do is attach the view (in your case the button) to the top of the floating panel view.
Here's how:
    let panelController: FloatingPanelController = ....
    let buttonView: UIView = ...

    panelController.view.addSubview(buttonView)
    buttonView.translatesAutoresizingMaskIntoConstraints = false

    let constraints: [NSLayoutConstraint] = [
      buttonView.bottomAnchor.constraint(equalTo: panelController.surfaceView.topAnchor,
                                         constant: 0),
      buttonView.rightAnchor.constraint(equalTo: panelController.surfaceView.rightAnchor,
                                         constant: 0),
      buttonView.leftAnchor.constraint(equalTo: panelController.surfaceView.leftAnchor,
                                         constant: 0)
    ]

    NSLayoutConstraint.activate(constraints)

Feel free to change the constant values to add padding, or change/replace the left/right anchors with something more suitable to your needs :)
Cheers!
Federico

floatingViewController.surfaceView.frame.origin.y

Hi guys,
I'm trying to achieve something similar and it's working fine.
However there is one small issue: when dragging the floating panel, the button follows it properly. But when releasing the panel, the button will not move in an animated manner (like the panel does), and will immediately go to its next position.
@zntfdr Did you achieve to animate the buttonView position change as well ?
Thanks!

Hi @paulnicolet,
I manage to get the following by using the code mentioned above:

I don't do any changes beside adding the constraints as mentioned 😃

@zntfdr thanks for your help man :)


SFT lora ends with higher loss

I've run the training without changing any hyperparameter except for batch size and gradient accumulation steps to match the global batch size on two machines.
The first run is exactly as in repo, gets eval loss 1.0667:
https://wandb.ai/evgeniizh/huggingface/runs/pskgg48d
The second one adds warmup (https://github.com/huggingface/alignment-handbook/pull/31 https://github.com/huggingface/alignment-handbook/pull/71) and uses TRL from master (which fixes https://github.com/huggingface/alignment-handbook/issues/61) and gets eval loss of 1.0927
https://wandb.ai/evgeniizh/huggingface/runs/9ez7kl7s
The official SFT model gets much lower loss of 0.99
https://huggingface.co/alignment-handbook/zephyr-7b-sft-lora

Possibly related to https://github.com/huggingface/alignment-handbook/issues/45


fix: updgrade zod to 3.22.4

Upgrade zod to address https://security.snyk.io/vuln/SNYK-JS-ZOD-5925617

was fixed in #198


Problem: diff engine defies expectation. Proposal: redraw strategy as a virtual node attribute

I've mentioned before that it's a bit confusing when what the author conceives of as a completely distinct element is interpreted by the diff engine as a modification upon another. Here's a trivial example:
return m( '.' + loading ? 'loading' : 'loaded', 'thing' );

If I were to describe the construction above, I'd say its a "thing with 1 of 2 classes, depending on a condition". And this is what the diff engine will interpret it as when loading changes.
return loading ? m( '.loading', 'thing' ) : m( '.loaded', 'thing' );

But I'd describe the snippet above as "1 of 2 things, depending on a condition".
The difference is contrived in those examples. The real issue is when you have large nested structures, and the diff engine reaches a node which, like the cached node, is simply a div with perhaps a class or 2 and a config element. Consider the following scenario:
function view(){
	return condition ? widget1 : widget2;
}

function widget1(){
	return m( '.sprawling', m( '.nested', m( '.structure' ), { config : pluginX } ) );
}

function widget2(){
	return m( '.something', m( '.completely', m( '.unrelated' ), { config : pluginY } ) );
}

condition starts as true, then an event switches it to false, and the view redraws. Now pluginY executes for the first time with isInitialized as true, and pluginX's context.onunload never fired. This is completely unintuitive.
The standard approach to this problem is to assign unique IDs or keys on any element that could be confused for another, but that leaves you with the grim task of managing identities yourself — which becomes unmanageable at scale.
So my proposal is for a special redraw attribute which would introduce extra logic in the diff engine. The principle is to supersede the subtree directive by making the functional equivalent of the m.redraw.strategy flag available on a granular level from within the view — and then extending the flag with a deep value, which extends the diff logic to accommodate the problem under discussion.

none would behave as subtree: "retain" currently does, and effectively aborts the build process for the node in question.
all is the opposite and builds the subtree from scratch.
diff isn't checked for: this is the current default behaviour, which compares tagName, attribute keys, and the values for key and id.
deep goes some way further than current behaviour and attempts to compare attribute values as well.

What constitutes a reasonable 'deep' comparison (which I'm well aware is a perennial problem in JS programming). These are the credible options AFAIC:

Compare primitive values only.
Compare all values by identity.
Compare all values by identity, except functions: compare these by Function.prototype.toString.

1 solves the example, and all my current expectations. I could fork this functionality relatively easily. 2 is more comprehensive and allows for differentiating elements that are statically identical but which have different config and event handlers. 3 is a Terrible Idea for the sake of example that uses non-standard behaviour in an attempt to cater for expectation shortfall in 2, when the function is not the exact same instance, but looks the same. This will still lead to expectation shortfall in all sorts of common scenarios.
What do we think?

Hi Barney
Thanks for suggesting this, it's definitely something worth thinking about.
As it stands, I think the proposal wouldn't work because of the complexity in Mithril's current diffing heuristics (e.g. how should these flags be treated when m.redraw.strategy is "all" or "none"?).
Also, I don't see any obvious use cases for an all flag. If someone wanted that, they could conceivably even use element.innerHTML, but I'm having a very hard time thinking of a valid reason why someone would purposely use that as a rendering strategy.
And since the semantics for none and diff flags already exist, the only new thing would be the deep flag. If I understand correctly, the use case for this is to be able to recreate an element instead of diffing it based on some logic that is not covered by the current diff conditional (i.e. a shouldComponentUpdate of sorts, except that it affects the component lifecycle instead of whether diffing should happen for its children).
The only thing that is pertains to lifecycles is config, so I think this proposal essentially boils down to the need to have some way of resetting the isInitialized flag based on some arbitrary condition. Since config functions are already hooks to arbitrary computations, this is somewhat doable via something like this:
var config = function(el, isInitialized, ctx) {
  if (shouldRecreate()) {
    ctx.onunload()
    isInitialized = false
  }
  //rest of config goes here
}

I fully admit this code is ugly as hell (and it doesn't even work in some cases), but I'm presenting it as food for thought in terms of what is the minimum amount of impact required to implement the idea needed in this proposal.
A proper solution would need to actually recreate the DOM element (in order to play well w/ libraries that don't provide destroy methods). This is something my snippet obviously doesn't handle correctly.
So theoretically, the simplest solution would be an attribute that takes a function, which is then called in the diff engine's if statement for whether an element should be recreated. The question would then be whether the function overrides the conditional altogether or not (presumably yes).

@lhorie sorry for the delay in getting back to you. I've been thinking about this on and off and become slightly less convinced about it. But that's still plenty of conviction remaining ;)

As it stands, I think the proposal wouldn't work because of the complexity in Mithril's current diffing heuristics (e.g. how should these flags be treated when m.redraw.strategy is "all" or "none"?).

I'd say these top level flags are overrides. Isn't that how { subtree } currently behaves?

Also, I don't see any obvious use cases for an all flag.

Neither do I, it would be an awful idea in practice. This is in part due to the dubious aim of maintain semantics upstream, but also of trying to rationalise how the logic would work compared to those.

The only thing that is pertains to lifecycles is config

Good point. As with above, I can't think of any reason why somebody would want to continuously destroy and rebuild an element every redraw.

I fully admit this code is ugly as hell

…But it offers a way to demonstrate proof of concept using current Mithril, rather than wading into core hackery without a clear outcome in mind. I'll try and get that working and take a step back to see if some kind of standardized function invoked by a new key is worthwhile. Good suggestion.

… positional identity … which is essentially the core of the problem…

Mithril.Elements is ingenious, but the whole problem AFAIC is that positional and key-based identity are kludges that happen to work in the simplest situations – the notion of multiple instances of the same module at different locations in the DOM, without awareness of each-other, is the ultimate contender for when these methods break down. I maintain that the 'comprehensive diff' (ie, has the element changed at all or are we just passing over?) is a useful condition in and of itself (at least, it would solve all my problems).

The other question is what would the function need to look like in application space

I'll get back to you :)

This would be immensely useful,
I remember having some troubles with including css files in m().
So having something like
m( 'head', [
    m( 'link[rel=stylesheet][href=/styles/main.css]', { redraw: 'diff' } )
] );

to make sure the css file isn't reloaded on route change would in my opinion be much better than having to specify m.redraw.strategy( 'diff' ); in all top level controllers.
furthermore having m.redraw.strategy( 'diff' ) in all controllers adds complexity when it comes to config functions.
This is by no means preventing me from using mithril effectively, all my config functions are simply written using this pattern:
var config = function( forceRedraw ) {
    return function( element, isInitialized ) {
        if ( !isInitialized || forceRedraw ) {
            // Do stuff
        }
    };
};

So I believe having a redraw strategy local to the element would by no means be a roadblock, but would rather force users to use a pattern such as the one above on only those elements which require a local redraw strategy.

Update: there's some preliminary code to address this in the ctx-reuse-flag branch.
Usage:
m("div", {config: config})

function config(el, isInit, ctx) {
  ctx.reuse = true //don't recreate on route change if not needed
}

or when using w/ strategy("diff") in controller constructors:
m("div", {config: config})

function config(el, isInit, ctx) {
  ctx.reuse = false //recreate on route change
}

Misc: Currently flag is write-only (i.e. it does not report default state). Flag name sucks, suggestions welcome.
Status:

tested: yes
documented: no

The flag is now called retain

Brilliant stuff @lhorie! This feels just right to me — extending the ctx argument's build-loop  interactions is intuitive and simple. I think this should work with components whose controllers are initialised in the view loop too… Can you push the branch?

:+1:


The new catch method is not good enough (?)

I often used the following pattern in the older version:
func executeJob() -> Promise<Void> {
  request()
  .done {
     // Do something
  }.catch { e in
     // Schedule a local push notification, etc
  }
}

Basically I "tapped" the error and process it, but I still wanted it to be passed back to whatever that called executeJob (thing like logging, or scheduling push notifications really belongs to executeJob, not its caller).
This pattern is no longer possible with PMK 6.0. An alternative is to use the tap function, but it seems to be designed for debugging purpose -- e.g. it executes async and I have to do stuff like guard case .rejected(let error) = result else { return }.
I understand the reasons why catch has been changed, but how I can produce the same effects in PMK 6.0?

Use recover.

I feel I must ask, what situations are you doing something but only for an error condition? It suggests misuage.

Thanks @mxcl. It does solve my problem.
Not sure if I understand what you asked... My use case is: I have jobs (which are basically network requests). These are initiated by users and may complete or fail. Push notifications are used in both cases to notify users. These jobs are managed by a manager so that they can be retried later if they fail. It is this manager that processes the errors further, e.g. saving error messages so that they can be displayed later, or when the app restarts.
I guess my issue comes down to responsibility separation between components: a component may want to do something (most likely logging, or push notification in my case) when an error happens, but still want to pass it along the chain.


fix sm4-xts aarch64 assembly implementation bug

Tested on kunpeng920, to turn on 'VPSM4_EX_CAPABLE'.

Checklist

[ ] documentation is added or updated
[x] tests are added or updated

fix #23316

The CI failure is relevant.

LGTM

Ping for second approval.

This pull request is ready to merge

Merged to the master and 3.2 branches. Thank you for your contribution.


Declare if canonical_sphinx extension is safe for parallel reading

When using the  canonical_sphinx extension I get a build warning:
WARNING: the canonical_sphinx extension does not declare if it is safe for parallel reading, assuming it isn't - please ask the extension author to check and make it explicit
WARNING: doing serial read

Here is an example for such a declaration: https://github.com/networkmanager-clients/python-sdbus/commit/ae11d3239a85e3a48de1ae2d4f9fe2be00329385

These fixes have already been implemented some time ago with the commit 79697de, but the version of canonical_sphinx that is on Pypi has not been updated in some time. I'm working to get the new version pushed to pypi - but in the interim I recommend using:
canonical-sphinx[full] @ git+https://github.com/canonical/canonical-sphinx@main

Please note that the most recent change to this extension moves a significant amount of functionality to the [full] optional installation as there is an internal need for a minimal installation.

This is canonical-sphinx-extensions not canonical_sphinx (confusing I know). I believe all the extensions here are parallel safe already.  The canonical_sphinx extension has had this fix implemented but the version on Pypi has not been updated. I'm working to remedy that, but in the interim I would recommend using:
canonical-sphinx[full] @ git+https://github.com/canonical/canonical-sphinx@main

Please note that a recent update has pushed a series of functions to the optional [full] installation as there was a need for a minimal install for some applications.


Array Implementation Code in C++

Hi, @ankit-sr
Please check PR #96. Please let me know if any update is required.


pick team on entry

before starting the game, pick an experiment to defend.
random suggestions

ALICE many hitpoints, more weasels
ATLAS slower weasels, expensive transformers
CMS many hit points, faster weasels
LHCb cheaper transformers with fewer hp
LHCf smaller hit box of detector (weasels walk by) weasel gives only 60 JTN

closing all my cern related issues as won't-pursue-anymore / don't care anymore - regardless of status


bundle update --conservative mimemagic

Updates mimemagic to 0.3.5
Delivers Pivotal #17747704,
fixing Travis CI build Error documented therein.

Coverage remained the same at 92.979% when pulling b8edbab63c160c9052d88d246e9bec5a8afce0db on 177477041-mimemagic-update into ad63e47bbeebd256beadb8638b328ac5d5c625cd on master.


Path gets reset when user manually changes it

Description
The user cannot manually change the URL of a page they're visiting with Flet. Consider this simple Flet example and watch the following screencast.
https://user-images.githubusercontent.com/1115018/216831157-d27bf814-a2e2-4fe7-a538-fffe0541c930.mp4
Code example to reproduce the issue:
import flet as ft

def main(page: ft.Page):
    page.add(
        ft.Text(f"This is the path: {page.route}")
    )

if __name__ == "__main__":
    flet_path = ''
    flet_port = 8001
    ft.app(name=flet_path, target=main, view=None, port=flet_port, route_url_strategy="path")

Describe the results you received:
When the user tries to change a URL, in an existing window, the URL reverts.
Describe the results you expected:
The user should be able to update the URL, as is normal with web sites.
Additional information you deem important (e.g. issue happens only occasionally):
If you open a new tab, and enter a URL there, it works. However, modifying an existing tab (open to a Flet app) does not allow for URL changes done by the user.
Flet version (pip show flet):
Name: flet
Version: 0.4.0
Summary: Flet for Python - easily build interactive multi-platform apps in Python
Home-page:
Author: Appveyor Systems Inc.
Author-email<EMAIL_ADDRESS>License: MIT
Location: /Users/kkinder/Library/Caches/pypoetry/virtualenvs/fletend-9mHHD83p-py3.11/lib/python3.11/site-packages
Requires: flet-core, httpx, oauthlib, packaging, watchdog, websocket-client, websockets
Required-by:

Operating system:
Version 13.2 (22D49)
Additional environment details:
Poetry (version 1.3.1)

Just ran into the same issue. It seems the RouteChangeEvent just doesn't fire.

Investigated a bit: the event handler converter method, called convert_route_change_event() in page.py seems to get called when one manually changes the route. However, that change doesn't result in a RouteChangeEvent' being returned because for some reason, the data of the event it converts always contains an unchanged route and thus None` is returned from a guard clause.

This only occurs when url strategy is set to "path", "hash" works as expected.  This does not occur on Safari because safari actually does an entire page load on manual url change.  Brave, Chrome and Firefox do not.

Still occurs in 0.5.2.  Its not an option to use hash paths for me, even temporarily because I am replacing an existing app and need to use the same paths.

yes, on a new tab you can paste a link in and it works as expected.  However, I have never seen an app that does not allow me to navigate by typing in URLs if I chose to.
My use case is that a user can get a 'public' link to an object which they can paste into the URL bar, but if they are also a  member of the site, they get more actions available to them on the public page. So they go to the login window, login, then paste the public URL back in, but it fails and they can never see the public link again without opening a new tab.  This is very surprising to the user because they expect that typing an URL will do the right thing.

Will take a look.


Should split install and start

Current helper script pulls down all required files, modifies them then starts a Singleton or HA cluster.
This should be broken into 2 scripts:

the first simply pulls down the environment; and
the second does the modification and install.

Some environments such as openshift will require intermediary steps between these 2 phases.

The quick start has been re-worked such that there is only a configure.sh script, which does the second part: modification and install. Pulling down the environment is part of the quick start procedures.

This can be closed now

agree


test

Signed-off-by: surabhi-dudhediya<EMAIL_ADDRESS>Changelog

Add
Fix
Update

Reviewed by
@developer_github_id
Linked issue
#issue_number

wrong pr


Update things now that Bitcoin Core master is usable

Updates setup_environment.sh to build Bitcoin Core master branch instead of my hww branch.
Updates the Bitcoin Core usage documentation.

utACK


deps: update pino-roll to v1.2.0

@mcollina I think this PR failed because of the flackiness of the CI/test. Anyway pino-roll already in the main branch (and CI is passing). This is a PR for the v1.x branch.
https://github.com/platformatic/platformatic/pull/2907


Avoid calling handleClientConnection if initializeClientConnection fails

Description

This is a follow-on to #219, which split out the socket accept from the protocol initialization for SSL connections, in order to perform the latter on a separate thread.
In the case where the socket initialization fails, such as with an SSL handshake error, following #219 we would log the error but then attempt to handle the client connection anyway, resulting in unnecessary work and further errors.
This PR moves the do/catch logic for initializeClientConnection so that we only try to call handleClientConnection if the former was successful.
I also updated the log messages to document the source of a connection when it fails initialization.
Motivation and Context

Avoids unnecessary attempts to handle a connection for which the SSL initialization has failed, and reporting of further errors as a consequence.
How Has This Been Tested?

I have run the Kitura-net tests and they are passing.  Note that there is an intermittent test failure exposed by the test intrduced in #219 - which intentionally generates a bad connection to an SSL listener - which will be resolved by https://github.com/IBM-Swift/BlueSSLService/pull/42.
Checklist:

[ ] I have submitted a CLA form
[ ] If applicable, I have updated the documentation accordingly.
[ ] If applicable, I have added tests to cover my changes.

Codecov Report

Merging #221 into master will decrease coverage by 0.27%.
The diff coverage is 70%.

@@            Coverage Diff             @@
##           master     #221      +/-   ##
==========================================
- Coverage   74.02%   73.75%   -0.28%
==========================================
  Files          31       31
  Lines        4146     4145       -1
==========================================
- Hits         3069     3057      -12
- Misses       1077     1088      +11

Flag
Coverage Δ

#CHTTPParser
56.11% <ø> (ø)
:arrow_up:

#KituraNet
73.75% <70%> (-0.28%)
:arrow_down:

Impacted Files
Coverage Δ

Sources/KituraNet/HTTP/HTTPServer.swift
79.09% <70%> (-1.81%)
:arrow_down:

Sources/KituraNet/IncomingSocketManager.swift
85.45% <0%> (-5.46%)
:arrow_down:

...ces/KituraNet/Server/ServerLifecycleListener.swift
79.16% <0%> (-4.17%)
:arrow_down:

Sources/KituraNet/IncomingSocketHandler.swift
76.99% <0%> (-1.88%)
:arrow_down:

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 b3b3c5d...c173d65. Read the comment docs.

I'm accepting the apparent reduction in coverage - this is really because of a fix I added to BlueSSLService which no longer (erroneously) triggers an error handling path.  Raised #222 for a proper test for this path.


Slate Editor

Slate Roadmap

[x] Lower risk: by companies picking up Slate as editor (RedTurtle and kitconcept)
[x] Backend (make plone.restapi support slate in addition to draftjs)
[x] Transforms indexing in p.restapi
[x] Transforms for resolveuid
[x] Frontend
[x] integrate slate into Volto core
[ ] Migration
[ ] Migration steps - Transforms for migration inplace?

Features / UX

[ ] link sidebar (external/internal links, anchor links)
[ ] UI decide buttons, bar looking like the current one
[ ] Plone 6 Upgrade Steps
[ ] Sidebar - what do we want to show?
[ ] Extensibility story
[ ] Documentation
[ ] Configuration
[ ] Extensibility
[ ] features/plugins

Questions:

[ ] Current Title block (one liner fixed rich text)?

Slate Roadmap

Lower risk: by companies picking up Slate as editor (RedTurtle and kitconcept)
Backend (make plone.restapi support slate in addition to draftjs)
Transforms indexing in p.restapi
Transforms for resolveuid
Frontend
integrate slate into Volto core
Migration
Migration steps - Transforms for migration inplace?

Features / UX
link sidebar (external/internal links, anchor links)
UI decide buttons, bar looking like the current one
Plone 6 Upgrade Steps
Sidebar - what do we want to show?
Extensibility story
Documentation
Configuration
Extensibility
features/plugins
Questions:
Current Title block (one liner fixed rich text)?

@tiberiuichim I moved your comment to the main description of the ticket. We can use a checklist there to keep track of our progress.

What's the current roadmap on 'Replace draftJS with Slate editor'?
Unfortunatly Volto microsprints on Slate are not public anymore. I understand that you focus on associates. But this is on Plone 6. I do consume and contribute.
As soon as Volto Slate editor is not anymore a third party product but indeed Plone 6:

Is the inline editor style menu still a feature? https://github.com/eea/volto-slate/issues/109
fixed editor toolbar is skipped: yes, mywords. I convinced the customer.
…

@ksuess

the inline style menu paragraph style should be fixed. I don't plan on removing it
fixed editor toolbar... I'm sorry, I'm the one who messed with it. I need to revamp it. I've tried to improve the performance of the editor and the responsiveness of the toolbar and I found that the code is confusing and I've tried to simplify it. I can bring it back, but the plan for the future is to make the toolbar "fixed" all the time, using the Quanta toolbar. Probably not sooner then autumn, though, as Volto would have to ship with Quanta toolbar support. If you want, and I'm sorry about this, you can make a PR in volto-slate that brings back the fixed toolbar support and we'll merge it.

Slate is merged in Volto core since 16.0.0-alpha.15.


Update References usage

References only actually operates on DefinitionNames so make that
clear.

Nice 👍 I think this makes it a lot clearer, and it’s such a small change.

Pull Request Test Coverage Report for Build 109724682

-17 of 17 (0.0%)  changed or added relevant lines in 8 files are covered.
196 unchanged lines in 10 files lost coverage.
Overall coverage increased (+3.5%) to 33.106%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

hack/generator/pkg/astmodel/array_type.go
0
2
0.0%

hack/generator/pkg/astmodel/definition_name.go
0
2
0.0%

hack/generator/pkg/astmodel/enum_type.go
0
2
0.0%

hack/generator/pkg/astmodel/map_type.go
0
2
0.0%

hack/generator/pkg/astmodel/optional_type.go
0
2
0.0%

hack/generator/pkg/astmodel/package_definition.go
0
2
0.0%

hack/generator/pkg/astmodel/struct_type.go
0
2
0.0%

hack/generator/pkg/astmodel/primitive_type.go
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

hack/generator/pkg/astmodel/array_type.go
1
36.67%

hack/generator/pkg/astmodel/definition_name.go
1
0%

hack/generator/pkg/astmodel/enum_type.go
1
0%

hack/generator/pkg/astmodel/map_type.go
1
26.83%

hack/generator/pkg/astmodel/optional_type.go
1
0%

hack/generator/pkg/astmodel/primitive_type.go
1
53.33%

pkg/zips/env.go
3
0%

pkg/zips/types.go
25
54.76%

pkg/zips/client.go
75
3.96%

pkg/zips/template.go
87
26.32%

Totals

Change from base Build 109606322:
3.5%

Covered Lines:
436

Relevant Lines:
1317

💛  - Coveralls


Pull Request For Hacktober Hello-world

What the program do ?
kmcauliffe6.java prints my GitHub handle
In what programming language it is written?
Java

This pull request contains conflicts. Please fix it and reopen


Update ansible job util

New rackhd.py is now merged to ansible repo : https://raw.githubusercontent.com/ansible/ansible/devel/contrib/inventory/rackhd.py
Support for args --list and --host are requirements in ansible inventory scripts

@benbp @zyoung51 @jlongever

+1

+1

test this please

+1

:+1:


SendToAddress RPC call

wallet attached to client (for RPC calls that requires open wallet)
added composition of raw Contract transaction
basing on raw Contract TX, added ‘sendtoaddress’ RPC call
added interface to get wallet balance and implementation for NeoScan server

LGTM


#serialized_json for empty collection

Hi,
Behavior for serializer passed empty collection is to return nil
I think it would make more sense for the serializer to return []. Ideally would work with empty Array and empty Relation.
##### Current behavior
UserSerializer.new([]).serialized_json
=> nil

##### Desired behavior(?)
UserSerializer.new(User.where.not(id: nil)).serialized_json
=> "[]"

https://github.com/Netflix/fast_jsonapi/issues/462


missing packaging \ entry point

at time of writing, a user can't just pip install sPRUNED, the whole Installation\Running structure is missing

spruned.py is a good entry point.
missing argparser & package distribution

PR #49