Configurable keybindings

I would love to have a more sublime-text like terminal editor.
This project seems to drift more towards vim.
It might be possible to get both, if keybindings where completely user-configurable with a config-file like sublime does it.
Would you be interested in a pull request for something like this?
Goals:

Be able to simply copy&paste a sublime-keybindings file (of course most actions are not yet supported).
Generate one for the current state as well and make default
Safe those and user-defined overwrites (future step) under XDG-paths
Make all possible commands available via keybindings or Command prompt

Rough overview
Simple keybinding
	{ "keys": ["ctrl+q"], "command": "exit" },

More complex keybinding
	{ "keys": ["left"], "command": "move", "args": {"by": "characters", "forward": false} },
	{ "keys": ["ctrl+left"], "command": "move", "args": {"by": "words", "forward": false} },
	{ "keys": ["shift+left"], "command": "move", "args": {"by": "characters", "forward": false, "extend": true} },
	{ "keys": ["ctrl+shift+left"], "command": "move", "args": {"by": "words", "forward": false, "extend": true} },

This seems to be very flexible and I think would allow for a vim-like binding as well as sublime-like or maybe even emacs.
Code changes
For now, I would stick with the simple bindings first (although args is not hard to do with rust Enums):

Have a function that parses the value of "keys" to map to Event::Key(...)
Have a function that parses the value of "command" to enum Command
Enhance Command to encompass all possible commands
Create a global HashMap<Event::Key, Command> out of steps 1 and 2.
Rewrite the match-statements in Tui, Editor and Command to all map to the same functions

This might not be the direction you want to go, which is fine. If so, I would simply create a hard fork and diverge, if thats ok with you.

Looking at the suggested config I see that the only input is key/shortcut, while for modal editors it should also be a mode and some previous info like 2dd. Maybe it would be ok to have a context as an argument for this config to be flexible enough. Anyway, having a command pallete with an ability to control editor without any shortcuts is a must, I think this is the direction, where it should go.

Modal editing can be done (sublime has a Vintage mode for vi-keybindings, based on this keymap and some (don't know enough here) smaller plugins).
But it is definitely not a first-class citizen for this kind of config.
Lets do it like this: I currently work on a fork (https://github.com/msirringhaus/xi-term). I pack all keymap-related changes in a separate branch. You can then take a look at it and decide for yourself.

Anyway, the best approach to have this in xi-core itself

Just fyi: My fork now has support for undo/redo, copy+paste, moving around + selections, and multiple cursors.
It is not modal, but I think you could take 90% as is, if you want.

My fork now has support for undo/redo, copy+paste, moving around + selections, and multiple cursors

@msirringhaus woah that's awesome!

Lets do it like this: I currently work on a fork (msirringhaus/xi-term). I pack all keymap-related changes in a separate branch. You can then take a look at it and decide for yourself.

I'll take a look at your branch as soon as possible (which should be Monday).

Well, that branch has now "long" been merged into master. I think you can just look at HEAD and take what you like.

In core/cmd.rs mostly the naming of things is taken from sublimes keymap file. And, obviously, serde-derives make sense only for those files.
In those files all the parsing from user-input (be it prompt or keyboard shortcuts) to rust enum "Command" happens. Everything below that doesn't care how the commands are derived.
But again, that editor has no modes (which is exactly what I want), so if you want to go in the direction of vim, you need to restructure of course.

@msirringhaus would you mind opening a PR so we can discuss the changes easily? I have your branch checked out locally already but I can't comment about the code.

open


Exit code 0 when actually failing

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.02.11. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[X ] I've verified and I assure that I'm running youtube-dl 2018.02.11

Before submitting an issue make sure you have:

[X ] At least skimmed through the README, most notably the FAQ and BUGS sections
[X ] Searched the bugtracker for similar issues including closed ones
[X ] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[X ] Question
[ ] Other

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
I am trying to develop an aplication with youtube-dl. When downloading a video to a very nested subfolder in order to reach the maixmum path length youtube-dl outputs the error
ERROR: unable to open for writing: [Errno 2] No such file or directory: 'South Park_December 17, 1997_1_0110_South Park 110 - Mr. Hankey, der Weihnachtskot - 1. Ak-e6039bb4-ed00-11e0-aca6-0026b9414f30.mp4.part-Frag0.part'
(Used southpark for example). The process then exits with error code 0 (success) indicating that everything went well. I have also tested this beahivour with mistyped url's (404). Checking for the exitcode is therefore not working. Also if i start youtube dl within a c# process i cant get the error output. If i redirect all output it is still visible within my console window so it seems like the error message is  returned some other way.
How can i catch all errors programmaticaly when using youtube-dl and why is the exit code always 0?

>py -3.6 -m youtube_dl BaW_jenozKc  -o '%(title)400s.%(ext)s'
[youtube] BaW_jenozKc: Downloading webpage
[youtube] BaW_jenozKc: Downloading video info webpage
[youtube] BaW_jenozKc: Extracting video information
ERROR: unable to open for writing: [Errno 2] No such file or directory: "'                                                                                youtube-dl test video ''_ä↭𝕐.mp4'.f137.part"

>echo %ERRORLEVEL%
1

Most likely you're getting error code of wrapping process not youtube-dl.

I have got it working by starting youtube-dl directly as a Process and setting the WorkingDirectory first. -o does not work for multiple files.

-o does not work for multiple files.

Not true.


When a run the cmake config using the execute_process the config show scape codes.

Hello.
The code next code is to fix small details in config. I run the cmake config using a
execute_process( COMMAND cmake ${CMAKE_CURRENT_SOURCE_DIR}/git_repo  -DSOCI_CXX11:BOOL=ON -DSOCI_TESTS:BOOL=OFF -DWITH_MYSQL:BOOL=ON WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/build )

But the config messages outputs from soci had scape codes like the next
Configuring SOCI:
[cmake] -- SOCI_VERSION                             = 4.0.3
[cmake] -- SOCI_ABI_VERSION                         = 4.0
[cmake] -- SOCI_SHARED                              = ON
[cmake] -- SOCI_STATIC                              = ON
[cmake] -- SOCI_TESTS                               = OFF
[cmake] -- SOCI_ASAN                                = OFF
[cmake] -- SOCI_CXX11                               = ON
[cmake] -- LIB_SUFFIX                               = 64
[cmake] -- Looking for SOCI dependencies:
[cmake] -- Threads:
[cmake] -- CMAKE_THREAD_LIBS_INIT                   = -lpthread
[cmake] -- Boost:
[cmake] -- BOOST_RELEASE_VERSION                    =
[cmake] -- BOOST_INCLUDE_DIR                        =
[cmake] -- BOOST_LIBRARIES                          =
[cmake] -- MySQL:
Full external/libraries/soci/CMakeLists.txt
cmake_minimum_required(VERSION 3.11-3.18)

# MAIN_ROOT_SOURCE_DIR come from the top most CMakeLists.txt file
message(STATUS "MAIN_ROOT_SOURCE_DIR=${MAIN_ROOT_SOURCE_DIR}")
#message(STATUS "CMAKE_SYSTEM_INCLUDE_PATH=${CMAKE_SYSTEM_INCLUDE_PATH}")

# Emulate the ExternalProject_Add but in config time.
# Because the Fetch_Declare, Fetch_Populate, Fetch_* no work. Never create the static libraries.
if ( NOT EXISTS "${CMAKE_CURRENT_BINARY_DIR}/build" OR
     NOT EXISTS "${CMAKE_CURRENT_BINARY_DIR}/install" OR
     FORCE_BUILD_INSTALL_SOCI_LIBRARY )

  if ( NOT EXISTS "${PROJECT_SOURCE_DIR}/git_repo/src/CMakeLists.txt" )

    execute_process( COMMAND git clone https://github.com/SOCI/soci ${PROJECT_SOURCE_DIR}/git_repo )

  endif()

  file( MAKE_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/install )
  file( MAKE_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/build )

  execute_process( COMMAND cmake ${CMAKE_CURRENT_SOURCE_DIR}/git_repo -DSOCI_CXX11:BOOL=ON
                                                                      -DSOCI_TESTS:BOOL=OFF
                                                                      -DWITH_MYSQL:BOOL=ON
                                                                      #-DSOCI_NO_COLORS_IN_CONFIG:BOOL=OFF
                                                                      # The CMAKE_COLOR_MAKEFILE in off.
                                                                      # Not work for the soci config messages
                                                                      -DCMAKE_COLOR_MAKEFILE:BOOL=OFF
                   WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/build )

  execute_process( COMMAND cmake --build ${CMAKE_CURRENT_BINARY_DIR}/build )

  execute_process( COMMAND cmake --install ${CMAKE_CURRENT_BINARY_DIR}/build
                                 --prefix "${CMAKE_CURRENT_BINARY_DIR}/install" )

endif()

# Manual create the target soci_core and soci_mysql
add_library( soci_core STATIC IMPORTED GLOBAL )
set_target_properties( soci_core PROPERTIES IMPORTED_LOCATION
                       ${CMAKE_CURRENT_BINARY_DIR}/install/lib/libsoci_core.a )

target_include_directories( soci_core INTERFACE
                            ${CMAKE_CURRENT_BINARY_DIR}/install/include )

add_library( soci_mysql STATIC IMPORTED GLOBAL )
set_target_properties( soci_mysql PROPERTIES IMPORTED_LOCATION
                       ${CMAKE_CURRENT_BINARY_DIR}/install/lib/libsoci_mysql.a )

# Ubuntu package mysqlclient-dev only had support to pkgconfig not for cmake.
# We cannot use findPackage(MySQL)
include(FindPkgConfig)

#pkg_check_modules(LIBMYSQLCLIENT REQUIRED mysqlclient)

pkg_get_variable(MYSQL_INCLUDE_PATH mysqlclient includedir)

message(STATUS "MYSQL_INCLUDE_PATH=${MYSQL_INCLUDE_PATH}")

# The next are required by soci. Because in the mysql backend the include
# are "#include <mysql.h>" and not "#include <mysql/mysql.h>"
target_include_directories( soci_mysql INTERFACE ${MYSQL_INCLUDE_PATH} )

#target_include_directories( soci_mysql INTERFACE /usr/include/mysql )

target_link_libraries( soci_mysql INTERFACE soci_core mysql_client mysql_services )

Reference screenshot. Sorry be patient with me ;)

I think ideal would be to disable colours automatically if the output is not the terminal, as is common under Unix. There is also a convention to use NO_COLOR environment variable to do it, see https://no-color.org/
Do you think either of those could be implemented in CMake rather than adding an extra option for this?

Do you think either of those could be implemented in CMake rather than adding an extra option for this?

Ok perfect let me try. I never hear before about the NO_COLOR environment variable. But interesting try the possibility.

Yes it work!!!. Passing the environment var before the execute_command.
  set(ENV{NO_COLOR} 1)
  execute_process( COMMAND cmake ${CMAKE_CURRENT_SOURCE_DIR}/git_repo -DSOCI_CXX11:BOOL=ON
                                                                      -DSOCI_TESTS:BOOL=OFF
                                                                      -DWITH_MYSQL:BOOL=ON
                                                                      # The CMAKE_COLOR_MAKEFILE in off.
                                                                      # Not work for the soci config messages
                                                                      -DCMAKE_COLOR_MAKEFILE:BOOL=OFF

We not need the new option.
Thanks!!!.


kpt.dev show should widgets for filing/fixing doc issues

Similar to GCP docs, there should be two icons on every page:

File an issue for that doc page
Create a PR for quick fixes to that doc page

Great suggestion.  This will help get fixes quickly.  Extra credit - the template for the issue should be a "doc bug"


Dynamic Auto Scaling Executors - Kubernetes

Hi Team,
Please let me know when the dynamic scaling out /scale in feature with executors would be available in spark with kubernetes ?
It was suppose to be in 3.0.0

https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/blob/master/docs/user-guide.md#dynamic-allocation

@liyinan926 could you please provide more details on the wording "limited form of dynamic allocation" ? Why is it limited and what would a "full" support be? Thank you.


ci: test all features combinations in a matrix

We currently only test with the default features, but we have a handful of different features that can be toggled on and off at the user's discretion. We should use the test matrix strategy to test with those features on and off to make sure we didn't break any build setup.

@JoseSK999 those 50 combinations would be all building floresta and floresta-cli, or you plan into compiling some subcrates by themselves (e.g. build floresta-chain directly)? Because then we could cache only builds of floresta and florestad.
If all those combinations appear when building floresta and florestad, I think caching is important. Otherwise, CI would take forever.

Compiling every individual crate by itself yes. The exact combinations per crate (after #275) is:
chain: 2 combinations
cli: 1
common: 8
compact-filters: 1
electrum: 1
watch-only: 4
wire: 2
floresta: 16
florestad: 16
Currently not all combinations of the sub-crate features appear when building floresta and florestad.
Btw we would need to build in --release mode which lowers a lot the compiled artifact size, although it is slower.

Can we try to cache the last 32 and see what happens?

I guess, we could try it first on forks rather than in upstream/master

Sure! Do you have CI enabled on your fork? I think I can enable on mine if it's needed


Requesting more data in the Detail-Graph

If you have a nice detail-graph in front of you and notice some suspicious behaviour near one or both of the sides (or just want to get some larger overview), you have to manually adjust the start or end date.
Even worse, if you want to symmetrically expand the display, e.g. fetch a week more on both sides, you need to adjust both of the dates after each other.
What are nice ways to deal with this? Some random ideas:

Have a "+ 1 week" button that symmetrically extends the range on both sides
Zooming out using the mouse wheel when the graph is fully zoomed out expands the range symmetrically on both sides. I have no idea how well - or even if - this can be implemented, but it has some pros and cons:

It seems reasonable intuitive. You are fully zoomed out and zoom out further, so you are moving the boundaries
If you just want to zoom out and quickly scroll the mouse wheel, you might accidentally expand the window and trigger a refetch

Panning the graph (Shift+Dragging) beyond the edge causes the start date to be expanded by a week in this direction after you let go of the mouse (Not sure if you can implement that in echarts, but it might be possible)
Use a https://vuetifyjs.com/en/components/date-pickers/#range for the start / end date which might make these adjustments more tolerable
Make a custom widget similar to the sliders below the graph where you can interactively select the range you want
Probably many better and different ideas I missed

Current choices and status:

Dragging to the left / right is possible in Dygraphs (the "Fast" graph) but not really in echarts so it might only be implemented in Dygraphs
Using the mouse wheel to symmetrically extend the fetched datapoints has a working prototype for echarts and should be able to be implemented for Dygraphs as well. The size of the new window will likely be a fraction of the visible range, capped at a few months and clamped to the current date to the right.


I'll check if pycodestyle supports ignoring all errors.

I read pytest-pep8's documents.
.. warning::

    Running pep8 tests on your project is likely to cause a lot of
    issues.  This plugin allows to configure on a per-project and
    per-file basis which errors or warnings to care about, see
    pep8ignore_.  As a preliminary advise, if you have
    projects where you don't want to care at all about pep8 checks,
    you can put configure it like this::

        # content of setup.cfg (or pytest.ini)
        [pytest]
        pep8ignore = * ALL

By the way, I wonder if you want to ignore all erros, you don't have to run pytest-codestyle.
Why do yout want this support?

I wanted to use it with specific files. In example: docs/source/conf.py ALL rather than * ALL.

I'm sorry to tell you that I cannot implement your request function because I have to change the current config style.
Currently, you can run pytest ... src tests, which means exludeing ./docs directory.
And patches are welcomed :)

No problem, thanks for your reply! :blush:


investigate different meters ABC - is abcjs not picking up note length for animate playback

music correct but animation not

Audio playback and score following is correct for all the following. This appears to discount the L: hypothesis.
X:72
M:4/4
K:Cmaj
eccccccc|eccccccc|eccccccc|eccccccc|

X: 73
M:4/4
K:Cmaj
L:1/4
eccc|eccc|eccc|eccc|

X: 74
M:4/4
K:Cmaj
L:1/8
eccccccc|eccccccc|eccccccc|eccccccc|

X: 75
M:4/4
K:Cmaj
L:1/2
ec|ec|ec|ec|

@boblsturm Could you find a reproducible case?

(also, just fixed that inconsequential but jarring X: inconsistent space)

Also, re-reads original title and goes to change those meters to 6/8...

Oh, look. We're running the latest abcjs, 5.1.2. There is one subsequent commit: Correct the speed of animation in 3/8 meter.
https://github.com/paulrosen/abcjs/commit/185bcf78e2021967cd0f404d16f0d940a8fadcf4

Adding "Q:1/4=120" to the ABC corrects the animation for a 6/8 tune:
https://themachinefolksession.org/tune/6
On 04/07/2018 12:53, Toby Harris wrote:

Oh, look. We're running the latest abcjs, 5.1.2. There is one
subsequent commit: Correct the speed of animation in 3/8 meter.
paulrosen/abcjs@185bcf7
https://github.com/paulrosen/abcjs/commit/185bcf78e2021967cd0f404d16f0d940a8fadcf4
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tobyspark/folk-rnn-webapp/issues/47#issuecomment-402443697,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJy6ZJLXII3UOqQyP8FIVAbu6O9Wtuk0ks5uDJ7FgaJpZM4UHD1q.

--
Cheers.
-Bob L. Sturm

Bob L. Sturm, Lecturer of Digital Media
School of Electronic Engineering and Computer Science
Centre for Digital Music
Eng. 111
Queen Mary University of London
Mile End Road London E1 4NS
United Kingdom
http://www.eecs.qmul.ac.uk/~sturm/
https://highnoongmt.wordpress.com
https://soundcloud.com/sturmen-1
https://www.youtube.com/channel/UC7wzmG64y2IbTUeWji_qKhA

I've filed this on abcjs https://github.com/paulrosen/abcjs/issues/155
As that latest commit shows, Paul is working on this code, so I'll see if the report prompts a fix while he's got the workings still in his head.

In the meantime, adding "Q:1/4=120" to the abc before sending to abc2midi would work.


I keep on getting a strange error when trying to run music-gen plug in.

Hope all is well,
I keep running into this error:
 RuntimeWarning: Couldn't find ffmpeg or avconv - defaulting to ffmpeg, but may not work   warn("Couldn't find ffmpeg or avconv - defaulting to ffmpeg, but may not work", RuntimeWarning) Traceback (most recent call last):   File "ML_Models/music_gen.py", line 22, in <module>     nd = Nendo(config=NendoConfig(plugins=["nendo_plugin_musicgen"]))   File "/Users/adelabdalla/.pyenv/versions/3.8.12/lib/python3.8/site-packages/nendo/main.py", line 73, in __init__     self._load_plugins(plugin_names=plugin_names)   File "/Users/adelabdalla/.pyenv/versions/3.8.12/lib/python3.8/site-packages/nendo/main.py", line 141, in _load_plugins     plugin_instance = self._load_plugin(module_name=module_name)   File "/Users/adelabdalla/.pyenv/versions/3.8.12/lib/python3.8/site-packages/nendo/main.py", line 129, in _load_plugin     raise schema.NendoPluginLoadingError( nendo.schema.exception.NendoPluginLoadingError: Failed to import plugin 'nendo_plugin_musicgen'. Error: dlopen(/Users/adelabdalla/.pyenv/versions/3.8.12/lib/python3.8/site-packages/av/_core.cpython-38-darwin.so, 2): Library not loaded: @loader_path/libX11.6.dylib   Referenced from: /Users/adelabdalla/.pyenv/versions/3.8.12/lib/python3.8/site-packages/av/.dylibs/libavformat.60.3.100.dylib   Reason: no suitable image found.  Did find:         /Users/adelabdalla/.pyenv/versions/3.8.12/lib/python3.8/site-packages/av/.dylibs/libX11.6.dylib: cannot load 'libX11.6.dylib' (load command 0x80000034 is unknown)         /Users/adelabdalla/.pyenv/versions/3.8.12/lib/python3.8/site-packages/av/.dylibs/libX11.6.dylib: cannot load 'libX11.6.dylib' (load command 0x80000034 is unknown)
  When trying to execute:

nd = Nendo(config=NendoConfig(plugins=["nendo_plugin_musicgen"]))
I'm not sure what is wrong with my local development. I've installed musicgen throguh pip install nendo etc however I keep getting this error. Any insight would be appreciated.

Hi, thanks for trying out nendo!
The error message sounds like you may be missing ffmpeg on your local install. Please also make sure that you did install all the requirements.
From the error I'm assuming you're on MacOS, so can you try and run: brew install ffmpeg libsndfile portaudio

Hi Aaron thanks for replying. I installed both but for some reason still no avail. Is there any other advice you have?

If you are using an ARM based Mac and/or conda to manage your python enviroments, you could try to reinstall ffmpeg via:
brew uninstall ffmpeg

# in your conda env
conda install ffmpeg

Another issue could be that your python enviroment is not able to locate ffmpeg.
Can you run which ffmpeg and echo $PATH | grep ffmpeg and post what these commands return?

@aa221 flagging this issue for inactivity since there has been no response in 2 weeks. Have you been able to resolve this by now? If so, can you share your solution with the community? If it still persists, feel like sharing some more details about your environment with us, so we can help you debug this?


Testing and polishing app

Improved UI by turning helper buttons into popup helper messages on hover to decrease clutter.
Also dealt with this issue:
https://github.com/wincowgerDEV/OpenSpecy/issues/9#issue-823745168
@zsteinmetz I think this is what you were thinking for that feature request?

I was able to get the buttons to now completely turn off the process if they are inactivated, and I put all of the tools in the little cog button which will allow us to scale up as we add more tools without making things overly cluttered.

Nice work! You just need one extra click now to adjust the sliders. Bearable?

@zsteinmetz I think this is what you were thinking for that feature request?

If we stay with those cog buttons, should be maybe rather place them on the right and add some spacing between the sliders and the text? This would make it more logic from a user's perspective, I guess.

I can figure out another way to add more features without bogging down the
ui. I think it should be alright for now.
On Fri, Mar 19, 2021, 4:45 AM Zacharias Steinmetz @.***>
wrote:

Nice work! You just need one extra click now to adjust the sliders.
Bearable?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/wincowgerDEV/OpenSpecy/pull/28#issuecomment-802775359,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGMUJU5F62YVTGATCXFKTRTTEM2NTANCNFSM4ZNL5KYA
.

I agree with the space between the text and slider. It was giving me some
trouble. As for where the cog should be, if you try out the cogs you'll see
that they end up pooping out in a location right below the slider which to
me made intuitive sense but I'm open to other ideas. Also it's good to have
the cog information not cover the plot so you don't want it too far to the
right.
On Fri, Mar 19, 2021, 5:15 AM Zacharias Steinmetz @.***>
wrote:

@zsteinmetz https://github.com/zsteinmetz I think this is what you were
thinking for that feature request?
[image: image]
https://user-images.githubusercontent.com/26821843/111695777-af6d2a80-87f0-11eb-9eb4-a5136a1d5580.png
If we stay with those cog buttons, should be maybe rather place them on
the right and add some spacing between the sliders and the text? This would
make it more logic from a user's perspective, I guess.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/wincowgerDEV/OpenSpecy/pull/28#issuecomment-802791929,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGMUJU7FP5BJCPCU6G3CK7TTEM57TANCNFSM4ZNL5KYA
.

Hmm, doesn't seem ideal to me. You'd like to decide for switching a tool on or off first and do the fine tuning afterwards with the sliders. Given that the default language is English reading from left to right, this is not consistent. I'll contemplate a little more on that.

There might be a way we can have the tool button open to the left or
something too which would satisfy all the things.
On Fri, Mar 19, 2021, 6:37 AM Zacharias Steinmetz @.***>
wrote:

Hmm, doesn't seem ideal to me. You'd like to decide for switching a tool
on or off first and do the fine tuning afterwards with the sliders. Given
that the default language is English reading from left to right, this is
not consistent. I'll contemplate a little more on that.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/wincowgerDEV/OpenSpecy/pull/28#issuecomment-802838368,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGMUJU3T3S2OH4QR3LISXRLTENHQZANCNFSM4ZNL5KYA
.


Disable Auto-Updater for 2.1.0

We haven't addressed how we're going to do (or even if we'll support) auto-updating. This issue will disable the auto-updater until we can address this properly

Can this be closed, or do we want to come up with some ad-hoc way to check GitHub releases and let the user know that a new version is available for download?

Yup! PR #36 addressed this and can be closed.


handle grouped cgroup mounts (e.g. cpu,cpuacct)

While trying to get garden running on CoreOS I noticed that the linux_backend does not account for grouped cgroup mounts which seem to become more prevalent these days:
cat /proc/1/cgroup
9:perf_event:/
8:cpuset:/
7:devices:/
6:memory:/
5:freezer:/
4:cpu,cpuacct:/
3:blkio:/
2:net_cls,net_prio:/
1:name=systemd:/

The attached changes make garden-linux start in environments with multi cgroup mounts. For systems without multi group mounts this doesn't change anything.

Hi @databus23 could you make your membership of the SAP organisation public so cfdreddbot can tell that you're a member of an organisation that has signed the CLA please?

@julz I signed an individual CLA last week. The contribution was performed off work. I'm currently not part of the SAP organization and don't even know who inside of SAP would be responsible to add me to the SAP organization.

@databus23 - great, thanks! Going to close and re-open the issue to see if that'll get dreddbot to re-judge :)

yay it worked! :) I've unblocked the story in our tracker to pull this in - thanks again

Thank you, @databus23.

Hey @databus23,
Just a quick note to let you know that we've recently reverted this PR as it caused an unexpected failure in our test suite.
It seems that there is a subtle difference in behaviour here when deploying garden-linux on a fresh VM vs when simply updating the garden-linux job. It also seems that it doesn't happen on bosh-lite, only 'real' deployments.
We see the deployment complete successfully but the garden-integration-tests start to report the following error:
Expected error:
    <*errors.errorString | 0xc20802bca0>: {
        s: "container: start: exit status 2",
    }
    container: start: exit status 2
not to have occurred

After some investigation, we realised that the issue only happens on a fresh VM because cgroups are only mounted if they are not already mounted.
Note that we use ubuntu for our tests and CI pipelines.
If you'd like to address the above test failure and resubmit the PR we'd be more than happy to accept. Let us know if you have any questions or thoughts.
Thanks very much,
Alberto and Ed.


Fix multipart requests, where params are objects instead of strings

Hi @afilina
Thanks for the pull request, I've fixed a few issues remaining with the UI aspect of Mimic to show the requests with file uploads and also let you match according to the file being sent.
Will love if you could try it out and give feedback before it goes into an official release.
You can try it out by installing mimic@next e.g npm install mimic@next.
Thanks!

@morsdyce I tried it and it fixes the upload requests, but I can't find how to re-enable the status bar. I manually edit the LocalStorage to bypass for now.

Thanks, happy it worked out!
Could you clarify what you mean about re-enabling the status bar? I'm not sure what you mean.


Implemented SK450 DeadCat frame support

Implemented SK450 DeadCat frame support
Implemented the way to specify motor output scale which is required for SK450 DeadCat as it has asymetrical arms (front arms are longer than back ones)

Looks good to me.

You will want to rebase this onto master like this:
git fetch upstream
git rebase upstream/master --interactive

https://help.github.com/articles/about-git-rebase/
I expect one conflicting line, but since you did both changes, that should be trivial. I assume this is flight tested? If / once it is, I'm happy to merge it.
Thanks!

Rebased and flight tested. Works fine

What is the difference between the 'w' / wide mixer and the dc mixer?

They have different angles (5 deg difference for front rotors, 6 deg back). In addition DC's one has extra correction for the arms length (back arms are 96% of the front)

Ok, got it!

Changes Unknown when pulling ba89c9eae26ba990317ff0b66d42e114b8c7dffa on anton-matosov:SK450DeadCat.v2 into ** on PX4:master**.


IPv6 with OKD4 UPI on OpenStack fails

Describe the bug
With a fresh install of OKD4 on OpenStack/UPI, the IPv6 configuration fails since NetworkManager defaults to stable-privacy. OpenStack expects the VM to use eui64 and does not allow any other IPv6 addresses (i.e. those are blocked by firewall rules on the hypervisor). Note that this is a feature of port-security, but it is best practice have port-security enabled.
Version
Cluster version is 4.7.0-0.okd-2021-08-07-063045
How reproducible
100% reproducible
References
Related bug-reports
https://github.com/coreos/fedora-coreos-tracker/issues/907
https://bugzilla.redhat.com/show_bug.cgi?id=1743161

A workaround for installations with networkType:OpenshiftSDN exists by using the following butane configuration:
variant: openshift
version: 4.8.0
metadata:
  name: 98-nmconnection
  labels:
    machineconfiguration.openshift.io/role: master
storage:
  files:
    - path: /etc/NetworkManager/system-connections/default.nmconnection
      mode: 0600
      contents:
        inline: |
          [connection]
          id=Wired Connection
          type=ethernet
          autoconnect-retries=1
          multi-connect=3
          permissions=
          [ethernet]
          mac-address-blacklist=
          [ipv4]
          dhcp-timeout=90
          dns-search=
          method=auto
          [ipv6]
          addr-gen-mode=eui64
          dhcp-timeout=90
          dns-search=
          method=auto
          [proxy]

For networkType:OVNKubernetes this needs to be fixed by adjusting https://github.com/vrutkovs/machine-config-operator/blob/ea229fbb1eb3b68d0014a0b06de27addc1dc473e/templates/common/_base/files/configure-ovs-network.yaml#L120

So in configure-ovs-network.yaml we need

      dhcp6_client_id=$(nmcli --get-values ipv6.dhcp-duid conn show ${old_conn})
      if [ -n "$dhcp6_client_id" ]; then
        extra_brex_args+="ipv6.dhcp-duid ${dhcp6_client_id} "
        extra_brex_args+="ipv6.ipv6.addr-gen-mode 0 "
      fi

and in IPv6 mode we'll have br-ex set to use eui64?
Do we know if that would affect IPv6 on other platforms?

Hi thanks for opening this Bug. We are attempting same in vmware
environment : vmware 7.6
okd               : okd4.7
network       : ovn-kubernetes
We read "IPv6 is supported only on bare metal clusters"  https://docs.okd.io/latest/networking/ovn_kubernetes_network_provider/about-ovn-kubernetes.html.
Does that mean if anybody tries in vmware or openstack hypervisors , they wont work ?

So in configure-ovs-network.yaml we need

      dhcp6_client_id=$(nmcli --get-values ipv6.dhcp-duid conn show ${old_conn})
      if [ -n "$dhcp6_client_id" ]; then
        extra_brex_args+="ipv6.dhcp-duid ${dhcp6_client_id} "
        extra_brex_args+="ipv6.ipv6.addr-gen-mode 0 "
      fi

and in IPv6 mode we'll have br-ex set to use eui64?

dhcp6_client_id is not set (no DHCPv6 is in use, just SLAAC), so we would need something like:
      # create bridge; use NM's ethernet device default route metric (100)
      if ! nmcli connection show br-ex &> /dev/null; then
        nmcli c add type ovs-bridge \
            con-name br-ex \
            conn.interface br-ex \
            802-3-ethernet.mtu ${iface_mtu} \
            802-3-ethernet.cloned-mac-address ${iface_mac} \
            ipv4.route-metric 100 \
            ipv6.route-metric 100 \
            ipv6.addr-gen-mode eui64 \
            ${extra_brex_args}
      fi

Do we know if that would affect IPv6 on other platforms?

No, but previously Fedora CoreOS was defaulting to addr-gen-mode=eui64, i.e. my previous installation (from last year) was doing the correct thing from the start, albeit with networkType:OpenshiftSDN.

Does that mean if anybody tries in vmware or openstack hypervisors , IPV6 dual stack will not work irrespective of OKD4.7/OKD4.8?

Note that I opened this bug since it prevents us from accessing the OpenStack VMs over IPv6,and it is therefore not (directly) related to dual-stack Kubernetes. This feature allows us to connect directly to the OpenStack VMs (since we do route the IPv6 traffic) without requiring the configuration of floating IPs for all VMs.
When enabling dual-stack Kubernetes I would suspect that it requires disabling of port-security (or at least adjusting the allowed-address-pairs in OpenStack, since not doing this would prevent any communication from unknown addresses.

Thanks @pjoomen , Shall i open a separate bug for Dual Stack support in okd4.7, 4.8 on vmware ?

Do we know if that would affect IPv6 on other platforms?

To re-iterate:
The fact that Fedora CoreOS defaults to ipv6.addr-gen-mode=stable-privacy is a sane choice for workstations, but does not make any sense for server-type installations. ipv6.addr-gen-mode=eui64 gives us a predictable IPv6 address since it is constructed from the combination of the route-announcement and the mac-address. This address is (for OpenStack) the one that is configured as the address to expect traffic from. Traffic from other IPv6 addresses, originating from the VM, is disallowed (if no further configuration is taken).
The proper way to deal with this issue, is to have the default of the base operating system changed. This would also allow us to have access to the VMs before OKD corrects the error.
The two ways to deal with this issue as mentioned in this report, are viable workarounds (until a more permanent fix is in place).

Thanks @pjoomen , Shall i open a separate bug for Dual Stack support in okd4.7, 4.8 on vmware ?

Yes, that makes sense to me.

Note the Fedora CoreOS stance on this issue: https://github.com/coreos/fedora-coreos-tracker/issues/907#issuecomment-901329628 We want to fix it in the base but it might be some time.
For now Thomas and I suggested in the BZ to have whatever is calling nmcli to also specify the ipv6.addr-gen-mode as well.

No, but previously Fedora CoreOS was defaulting to addr-gen-mode=eui64, i.e. my previous installation (from last year) was doing the correct thing from the start, albeit with networkType:OpenshiftSDN.

Yes. The change in behavior was unintended. See https://github.com/coreos/fedora-coreos-tracker/issues/513#issuecomment-887032823

It does not seem to be that FCOS 35 has fixed this.  Am I correct in making that assumption?

The following butane file can be used as a workaround when provisioning the cluster: (works with OKD 4.7/4.8/4.9)
`variant: openshift
version: 4.9.0
metadata:
name: 97-eui64
labels:
machineconfiguration.openshift.io/role: master
systemd:
units:
- name: configure-eui64.service
enabled: true
contents: |
[Unit]
Before=multi-user.target
Wants=network-online.target
After=network-online.target
    [Service]
    Type=oneshot
    ExecStart=/usr/local/bin/configure-eui64.sh
    ExecStartPost=/usr/bin/touch /tmp/has_executed
    RemainAfterExit=no

    [Install]
    WantedBy=multi-user.target

storage:
files:
- path: /usr/local/bin/configure-eui64.sh
mode: 0700
contents:
inline: |
#!/bin/bash
nmint=(ovs-if-br-ex br-ex)
      for i in "${nmint[@]}";do
        if [ "$(nmcli con show "${i}"|awk '/ipv6.addr-gen-mode/ {print $2}')" == "stable-privacy" ]; then
          # /etc/NetworkManager/systemConnectionsMerged = OVERLAY
          echo "eui64 is not enabled for connection ${i}! Enabling now!"
          nmcli con mod "${i}" ipv6.addr-gen-mode eui64 && nmcli con up "${i}"

          # /etc/NetworkManager/system-connections = UNDERLAY
          # This needs to be changed to make the change persistent after reboot
          sed -i 's/stable-privacy/eui64/' /etc/NetworkManager/system-connections/"${i}".nmconnection
        fi
      done

`


Move node link strategy from view to report

It would be better to:

Store the link strategy with the report instead of the view so link strategy can also be used when doing a rerun to find counterpart checkpoints to check whether they need to be stubbed or not
Remove code in frontend to find counterpart node and call a backend api that will return the counterpart checkpoint for a report that is compared with another report for which the user selects a specific checkpoint. It will simplify the frontend and all code related to finding counterpart checkpoints based on a link strategy will be on one place

See also:
https://github.com/wearefrank/ladybug-frontend/issues/314 (will be fixed when the above is done)
https://github.com/wearefrank/ladybug-frontend/issues/593 (will be fixed when the above is done)
https://github.com/wearefrank/ladybug/commit/47456163a5b5cb34dbdac54488116d9e39356a3c (node link strategy was added to view)

Backend code has been changed to store link method in report. Dropdown has been added to old GUI. Link method will be used when running a report and in the compare tab.
https://github.com/wearefrank/ladybug/commit/e1a63b6484a290f3c363433bdc87f3520e41f0c5
REST api still has to be changed to allow the new GUI to make use of the backend code to find counterpart checkpoints:
https://github.com/wearefrank/ladybug/commit/e1a63b6484a290f3c363433bdc87f3520e41f0c5#diff-3d280ea4d464a0a5a7d164b85aacd0d0c6a03a99293b6ae3fc0b00a01935c148R766


Is this Error Message Correct?

Quote: a valid fragment contains only unreserved characters, reserved characters or '%'.
This seems pretty dubious to me.
If a fragment can contain unreserved characters and reserved characters... isn't that all characters?
If this is some quirk of the URL spec, it's really not clear as a casual reader.
https://github.com/icerpc/icerpc-csharp/blob/57995b702436169798fc3fe5fd513e6f09591397/src/IceRpc/ServiceAddress.cs#L522-L532

This is not a bug. The URI spec defines all these terms. See for example https://en.wikipedia.org/wiki/Percent-encoding.

The broader point of my comment was:
"an error message which requires niche domain knowledge or reading a spec isn't a great error message".
But this is a niche error, so it's fine. Listing the characters individually would be a pain.

After reading the spec, I still believe the error message is in fact technically incorrect, and even worse, I think our logic is incorrect too. TL;DR these characters shouldn't be valid in a fragment "#" / "[" / "]", but our comment suggests they are (these are in the reserved character set), and our implementation doesn't seem to reject them either.

Taking the definitions from the spec, a valid fragment is:
fragment    = *( pchar / "/" / "?" )
pchar       = unreserved / pct-encoded / sub-delims / ":" / "@"
unreserved  = ALPHA / DIGIT / "-" / "." / "_" / "~"
pct-encoded = "%" HEXDIG HEXDIG

reserved    = gen-delims / sub-delims
gen-delims  = ":" / "/" / "?" / "#" / "[" / "]" / "@"
sub-delims  = "!" / "$" / "&" / "'" / "(" / ")" / "*" / "+" / "," / ";" / "="

Expanding this gives us that a fragment can contain:
fragment    = *( unreserved / pct-encoded / sub-delims / ":" / "@" / "/" / "?" )

NOTE: HEXDIG is contained within the unserved character set, so pct-encoded only gives us a % here
fragment    = *( unreserved / sub-delims / "%" / ":" / "@" / "/" / "?" )

However, our error message claims that a fragment can contain:
unreserved characters, reserved characters or '%'
We agree with the spec on unreserved and %, so lets take those our of list for fragment. This leaves us with:
fragment    = *( sub-delims / ":" / "@" / "/" / "?" )

Per our comment, this should be equivalent to the reserved character set: reserved = gen-delims / sub-delims
Again, we agree on sub-delims, so let's take that out. We are left with:
*( ":" / "@" / "/" / "?" )    vs    gen-delims

THESE ARE NOT THE SAME THING.
gen-delims  = ":" / "/" / "?" / "#" / "[" / "]" / "@"

Per my reading of the spec these characters are not valid in a fragment: "#" / "[" / "]".
However, both our comment, error message, and implementation seem fine letting them through.
https://github.com/icerpc/icerpc-csharp/blob/57995b702436169798fc3fe5fd513e6f09591397/src/IceRpc/ServiceAddress.cs#L198

Just as a sanity check, C#'s URI class does state that it must adhere to RFC 3986 to be considered well-formed.

Per my reading of the spec these characters are not valid in a fragment: "#" / "[" / "]".

I believe these characters are valid in fragments. In particular, I don't see any reason why they would be invalid. The rules are about avoiding ambiguities. The fragment is at the very end of your URI - it's simply everything after the '#' character.

I guess disallowing # helps reduce ambiguity, even if it isn't strictly necessary?
And maybe allows for more optimized performance when getting the fragment, since you can search from the back of the string for # if you know there can only be one of them.
That's just speculation on my part though.
A more concrete statement from the spec (emphasis mine):

A host identified by an Internet Protocol literal address, version 6
[RFC3513] or later, is distinguished by enclosing the IP literal
within square brackets ("[" and "]").  This is the only place where
square bracket characters are allowed in the URI syntax.

See https://dotnetfiddle.net/L7dKKQ
The whole point here is to do what the platform does. It's not about "enforcing the spec" when the platform does not.
When we decode a Slice2-encoded service address, we count on the .NET Uri class to perform all URI-related validations. We don't want to validate the already Uri-validated string in case the Microsoft Uri class is not conformant enough. So we just adopt the same lax rules as Microsoft. These may be more lax than the spec but they don't introduce any ambiguity.
Note: when we decode a Slice1-encoded service address with a facet, we use Uri.EscapeDataString to create the fragment:
https://github.com/icerpc/icerpc-csharp/blob/bbeb9356f0fec09205c426e99e64e0f0c40fd54e/src/IceRpc/Internal/FragmentSliceDecoderExtensions.cs#L14
EscapeDataString percent-escapes # and other reserved characters:
https://learn.microsoft.com/en-us/dotnet/api/system.uri.escapedatastring?view=net-8.0
So the fragment we get in this case is fully URI-spec compliant.

On the decoding side, I totally agree. As long as we can decode it, it doesn't really matter if the URI is perfectly-to-spec, and we shouldn't waste effort doing so. If it decodes, it's fine.
But on the encoding side, I think it would be safer to follow the spec. Otherwise a "valid" C# URI might be an "invalid" Rust URI.
Having a URI's validity depend on which language (and even which crate in Rust) you're using defeats the point of these being uniform and standardized IMO. What do you think?

Also, if this was some significant effort, I would of dropped this a while ago.
But, we're already performing our own validation of URIs, that's why we have this _notValidInFragment field in the first place. Right? All we'd have to do is add \\[\\]# to the list of characters, and maybe spend an hour reviewing the other rules as well.
IMO, following the spec here would be a pretty small effort.

What do you think?

You're missing the point.
The most common way to create a ServiceAddress in C# is from a Uri. When you do that, the IceRPC code doesn't perform extra validation to make triple-sure Uri did not let through some invalid character.
Then - for consistency - when you construct a ServiceAddress by hand, and set your own fragment, we perform the same lax but unambiguous validation as System.Uri.

I see your point.
Keeping consistency with the language is a nice-to-have, but more importantly implementing my 'fix' would be pretty pointless, since it's so rare to construct a ServiceAddress 'by hand'.
I'm still a little worried that this might cause problems down the road, since different languages will probably have inconsistent levels of strictness. But I'm going to close this, since we don't have many languages yet, and this is pretty niche. I mean, you need to use some very strange looking URIs to even get into this concern in the first place.
P.S. I'm dropping this, but am still planning to fix the small typos we found, and (IMO) clarify some minor wordings.


Improve data visualization in registry section

The current implementation of distribution by type diagram in Registry section uses circles. It is a bad pattern in data visualization. Let me explain.
When we visualize data with circles we need to calculate the area with weird for human mind formula A = πr², where A represents area (value of data), r - radius, π=3.1415. Because of r² and π it's hard for users to understand the dimensions and visually compare different circles.
Let's imagine a circle. Now try to imagine another one smaller twice.
Common answer is to reduce width(diameter) of circle twice, but it's wrong.

I suggest to use rectangles instead. This is why:

Current:

Suggestion:
I suggest the following design. Note that dimensions are inaccurate.

Even better
Users don't need to hover if number fit the rectangle.

It will also better align with tree map design of distribution by SDG chart
I can see that visualization is rendered in this repo so I opened issue here
@lacabra what do you think?

Thanks @Rolikasi, I absolutely love it, thanks for your very detailed explanation, I very much appreciate your time to explain it so clearly. I am in full support of changing this visualization!
Are you still thinking of leaving the hovering in place? because I agree with you that is not needed if it fits within the rectangles above, but it is still useful to know where rectangles overlap (say how many nominees are labeled both software and data?)

@lacabra, I'm happy to help!
I agree that hovering is useful and we need to leave it in place.

@Rolikasi I would love for you to cross this off your list in the coming month. I've given it some thought and realize that there is one more constraint that you should be aware off. The current display of software, data, content and standard is incomplete because we have a fifth type: ai model. This potentially creates a major issue, which is the impossibility of displaying an intersection between 5 sets given 4-side square figures, though I claim that there will never be one single DPG that can be all of 5 types at once. The most common overlap right now is between software, data and content which is a 3-way overlap, when we introduce ai model I think that will overlap with data and software, possibly with content.


Feature/improve error handling

Changes introduced:

Implement Error types for various parse failures.
Modify the entry parsing regex to only allow leading whitespace
Add tests to support additions/changes.
Documentation and badge improvements

Codecov Report

Merging #5 into master will decrease coverage by 0.03%.
The diff coverage is 81.88%.

@@            Coverage Diff            @@
##           master      #5      +/-   ##
=========================================
- Coverage   84.43%   84.4%   -0.04%
=========================================
  Files           4       4
  Lines         482     545      +63
=========================================
+ Hits          407     460      +53
- Misses         75      85      +10

Impacted Files
Coverage Δ

src/pgn.rs
81.06% <43.47%> (-1.92%)
:arrow_down:

src/dbc.rs
87.38% <89.56%> (+2.55%)
:arrow_up:

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 afb99c1...dd2f22e. Read the comment docs.


Play backwards at timeout

Hi (again) biohzrdmx :)
Another request/question and I FULLY understand if you don't want to spend more time on this, but just a small proposal that I think could be a cool feature.
On timeout at the end of the video it would play the video backwards, like a "boomerang" effect? When it hits "start" again, it would begin to play again "forward" ? :)
Thanks again for a great piece of code :)
All the best
Peter

Sorry - Just delete it - Found a simple and obvious solution :)


[FEATURE] Upload to multiple remote hosts

Host urls are listed with the '--hosts' command line flag.
CODIUS_HOST environment variable is deprecated
[Finishes #89122638]

+r

Review has been requested, but I see no reviewers assigned. Here's who wins:
+r assign stevenzeiler
Come on down and claim your prize!

Review has been requested, but I see no reviewers assigned. Here's who wins:
+r assign stevenzeiler
Come on down and claim your prize!

LGTM :+1:

The build is good and I'm seeing 1 +1s from the following folks:

tdfischer

As such, I'll attempt to merge and close this pull request.


Install Issue: createsuperuser Fails

Hello,
I am trying to install and run netbox on Ubuntu 16.04. I followed the instructions in "getting-started.md" up to the point where I need to create the first user. Here is the output of running ./manage.py createsuperuser:
/opt/netbox/netbox# ./manage.py createsuperuser
Couldn't import dot_parser, loading of dot files will not be possible.
Traceback (most recent call last):
File "./manage.py", line 10, in
execute_from_command_line(sys.argv)
File "/usr/local/lib/python2.7/dist-packages/django/core/management/init.py", line 353, in execute_from_command_line
utility.execute()
File "/usr/local/lib/python2.7/dist-packages/django/core/management/init.py", line 345, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/usr/local/lib/python2.7/dist-packages/django/core/management/base.py", line 348, in run_from_argv
self.execute(*args, **cmd_options)
File "/usr/local/lib/python2.7/dist-packages/django/contrib/auth/management/commands/createsuperuser.py", line 52, in execute
return super(Command, self).execute(*args, **options)
File "/usr/local/lib/python2.7/dist-packages/django/core/management/base.py", line 399, in execute
output = self.handle(*args, **options)
File "/usr/local/lib/python2.7/dist-packages/django/contrib/auth/management/commands/createsuperuser.py", line 86, in handle
default_username = get_default_username()
File "/usr/local/lib/python2.7/dist-packages/django/contrib/auth/management/init.py", line 189, in get_default_username
auth_app.User._default_manager.get(username=default_username)
File "/usr/local/lib/python2.7/dist-packages/django/db/models/manager.py", line 122, in manager_method
return getattr(self.get_queryset(), name)(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/django/db/models/query.py", line 381, in get
num = len(clone)
File "/usr/local/lib/python2.7/dist-packages/django/db/models/query.py", line 240, in len
self._fetch_all()
File "/usr/local/lib/python2.7/dist-packages/django/db/models/query.py", line 1074, in _fetch_all
self._result_cache = list(self.iterator())
File "/usr/local/lib/python2.7/dist-packages/django/db/models/query.py", line 52, in iter
results = compiler.execute_sql()
File "/usr/local/lib/python2.7/dist-packages/django/db/models/sql/compiler.py", line 848, in execute_sql
cursor.execute(sql, params)
File "/usr/local/lib/python2.7/dist-packages/django/db/backends/utils.py", line 64, in execute
return self.cursor.execute(sql, params)
File "/usr/local/lib/python2.7/dist-packages/django/db/utils.py", line 95, in exit
six.reraise(dj_exc_type, dj_exc_value, traceback)
File "/usr/local/lib/python2.7/dist-packages/django/db/backends/utils.py", line 64, in execute
return self.cursor.execute(sql, params)
django.db.utils.ProgrammingError: relation "auth_user" does not exist
LINE 1: ...user"."is_active", "auth_user"."date_joined" FROM "auth_user...
Have I incorrectly configured something? Is there an issue using 16.04?
Any assistance would be greatly appreciated.

Looks like your database hasn't been created yet. Did you run
./manage.py migrate

before trying to create the superuser?

Thanks for the quick response. Yes, I ran that just a few minutes ago. I ran it from my user with sudo; however, I noticed that the instructions in "getting-started.md" all use the root prompt. I will try running ./manage.py migrate as root.

Okay, that worked. However, I am still receiving the following error:

Couldn't import dot_parser, loading of dot files will not be possible.

@vcsalex those dor_parser warnings are OK to ignore for now unless something has changed @jeremystretch ?

Okay, thank you for your assistance.

ok

FYI I've removed the note in the "getting started" doc about graphviz being optional.


Remove 'my_new' permission for workdays

In ability.rb a new permission my_new was introduced to ensure that the 'add working hours' link in the sidebar of app/views/projects/show.html.erb was not rendered if the user was not authorized not access the project#show page.
This should be replaced with the normal new ability check.
@derari do you get this?

Thanks @derari !


ERR: Unsupported compression type: 3 causing seg fault

I tried to search the bash manual for the string "-z" which resulted in  the following error:
ERR: Unsupported compression type: 3
[1]    8200 segmentation fault (core dumped)  ag "-z" bash_manual.txt

I suppose it is to do with -z being an option for ag. It did work with:
ag -- "-z" bash_manual.txt

This is on Ubuntu 20 using zsh.
Maybe ag should not treat quoted strings as potential options.

I don't think this is an issue with ag specifically. Many Unix utilities have the same "problem". Try for instance creating a file named -z with the command touch -z.
There is no difference between passing "-z" or -z as the argument. The shell will remove the quotes before passing the argument to ag.
Your solution is correct, by using -- to indicate there will be no further option arguments. This works for touch too: touch -- -z.


Multiple pathParams don't work with non-standard url path

Following code will throw exception:
... when().get("/search;{criteria}={value}", "name", "John") ...
java.lang.IllegalArgumentException: Path parameters were not correctly defined. Undefined path parameters are: value.
Meanwhile, next code works ok (difference in slash and equals sign):
... when().get("/search;{criteria}/{value}", "name", "John") ...
Update. And also this code works:
... when().get("/search;{c}={value}", "name", "John") ...

Looks like a bug indeed. Thanks for reporting.

This was a very specific bug. It only manifested itself when one had defined multiple templates between subresources in the request path or (as in your case) when one had multiple templates after the last / IF at the same time the template name was longer than the template value. I've now deployed a new snapshot that should address this issue. Please try version 2.9.1-SNAPSHOT after having added the following Maven repository:
<repositories>
        <repository>
            <id>sonatype</id>
            <url>https://oss.sonatype.org/content/repositories/snapshots/</url>
            <snapshots />
        </repository>
</repositories>


bump vec_map to 0.7.0

There are no changes in API, only bumped dependency on serde.

Coverage remained the same at 91.37% when pulling e8d002929d1f6a98ff36f57e1439293bbe48e76e on ignatenkobrain:patch-1 into 10c1d1bd3e5f1d7c358f378405362c20ec60203f on kbknapp:master.

@ignatenkobrain thanks, but @frewsxcv just beat you to it 😜


Component result

Coverage remained the same at 97.005% when pulling 5175a878b475cd779b5c7ed7d08fa117fcc1a1b1 on gaggala:componentResult into e821ce0a5cfb2aa21b77afc06d985030a8b16c9f on plusnew:master.


Issue with responsive plots using image

Generating a responsive plot with image appears to not work since panel was updated to version 1.0.0. I'm aiming to open a plot in my web browser, which will responsively fill the screen. However, the update to panel (and also the change in the dependent version of bokeh) seems to have broken the ability of image to do this.
I'm working on Fedora 33, and the issue was initially raised to me by someone using my nctoolkit package on macOS M1.
The code below should reproduce the problem on all 1.0.x versions of panel. Instead of filling the web browser the plot gets squashed.
import xarray as xr
from threading import Thread
import hvplot.xarray
import hvplot as hv
import panel as pn
from bokeh.plotting import show

import hvplot.xarray

hv.extension("bokeh")
hv.Store.renderers

import nctoolkit as nc

ds = xr.tutorial.open_dataset('air_temperature').load()

def ctrc():
    """
    A basic control to send a message to the terminal
    """
    print("Press Ctrl+C to stop plotting server")

intplot = ds.hvplot.image("lon", "lat", "air",
        dynamic = True,
        rasterize = False,
        responsive = True
        )

t = Thread(target=ctrc)
t.start()
bokeh_server = pn.panel(intplot, sizing_mode="stretch_both").show(
    threaded=False
)

If I run this using the latest version of panel I get a squashed plot that cannot be read.

However, it is fine in panel version 0.14.4.

I've already got the required updates in a branch, will push those up tomorrow and then get a 0.8.4 release out asap.

Excellent, @philippjfr . Keep up the good work.


Use git lfs

@Marius1311
I think it would be best to recreate the repo/purge the history (the second option is preferred) and use git-lfs. I've tested this if it's possible to download the data using scanpy.read(..., backup_url=...) and it works.
Pros:

currently, I download 230MB, which is a bit too much for such a small repo
it will speedup the download (to run the tutorials, you don't need to download the datasets in the datasets directory)
using GIT_LFS_SKIP_SMUDGE=1 git clone ...
gallery can be finished (I don't know what the rate limits on figshare are, but with gh, we've never had an issue before)

Cons:

1GiB limit: currently, we'd be using ~180MiB (30 for pancres, 150 + pancreas_preprocessed)
needs git-lfs to download without a warning (see last point in Todos)

Todos:

[ ] for gallery, we want to download the file only once, so the path should be absolute
[x] I need to triple authenticate before I push - this will break automatic the travis pipeline in the main repo,
but this does not happens with a personal access token
[x] git-lfs has to be installed to "properly" download the repo (i.e. the clone succeeds, the checkout fails - this automatically remove the files and adds the changes to the staging area, ergo we need git-lfs on the main repo travis, which is not an issue)

Mike, I don't think we need to change anything here since this repository is not meant to be downloaded. This is why we separated cellrank from cellrank_notebooks. This is literally only meant to store our data and tutorials.
Since there is always a risk of something going wrong when doing these sorts of changes, I would not like us to do this right now and to delay this to later, if really necessary.

I think this is necessary. I need to close this issue.

Note to self: https://christoph.ruegg.name/blog/git-howto-mirror-a-github-repository-without-pull-refs.html

Done via https://github.com/theislab/cellrank_notebooks/commit/62aaa2c2d09eb00e839ac9f4693ec429b7631159.
The commits have been rewritten using, it's very easy to use. I will do the same for the main repo (no git-lfs there, for now).


Question: Declaring permissions on Google Play

Hello,
Apologies for sort of abusing the issues section as this doesn't directly relate to this package but there's no discussions tab on this repo and I'm running out of places to ask.
How have people been publishing their apps to Google Play? We've been trying along with contacting support for the last month to no avail, they've switched from the form to using the play console. Only problem is that to submit a request to get permission to use the required permissions you need an app release with those permissions, and we can't create an app release since we don't have permission to use those permissions even in the internal testing track...
Does anyone have any insights on what we're missing?
Thanks and apologies again for hijacking the issues section.

Hey!
No worries. I have now enabled the discussions section of the repo. I did not realise that it's not enabled by default. Can you move it there?

Yep, moved to discussions https://github.com/matinzd/react-native-health-connect/discussions/156 thanks


Page summary reports contain duplicates

I only discovered this by chance and I'm not sure how many other instances there are of but at least in the 2022-06-01 desktop query the test 220609_Dx5G_1613T (www.jnj.com) appears twice and this shouldn't be the case.

Agreed, something doesn't look right. There are 3,252 duplicate WPT IDs in the 2022_06_01_desktop table.
It's straightforward to remove the duplicates but we should also try to understand how it happened. FWIW there are 0 duplicates in the July table, so this may be unique to the streaming pipeline.

mobile - 3946 duplicates
bquxjob_14def594_1830fe5f0ad.csv
SELECT wptid, COUNT(1) as count_wptids, ARRAY_AGG(DISTINCT url) as urls, ARRAY_LENGTH(ARRAY_AGG(DISTINCT url)) as count_urls
FROM `httparchive.summary_pages.2022_06_01_mobile`
GROUP BY 1
HAVING COUNT(1) > 1
ORDER BY 2 DESC

desktop - 3252 duplicates
bquxjob_16304447_1830fe8e23a.csv
SELECT wptid, COUNT(1) as count_wptids, ARRAY_AGG(DISTINCT url) as urls, ARRAY_LENGTH(ARRAY_AGG(DISTINCT url)) as count_urls
FROM `httparchive.summary_pages.2022_06_01_desktop`
GROUP BY 1
HAVING COUNT(1) > 1
ORDER BY 2 DESC

These older jobs are harder to research, but similar to #133, this is likely due to issues encountered when implementing the streaming pipeline around that time. Since we will no longer use the streaming approach to loading BigQuery, this should not be expected in the future.
I have deduplicated these tables which were loaded around the time streaming was still in use:

httparchive.summary_pages.2022_06_01_mobile
httparchive.summary_pages.2022_06_01_desktop
httparchive.summary_pages.2022_05_01_desktop (mobile had no duplicates)
httparchive.summary_requests.2022_06_01_mobile
httparchive.summary_requests.2022_06_01_desktop
httparchive.summary_requests.2022_05_01_desktop
httparchive.summary_requests.2022_05_01_mobile

Great thanks @giancarloaf. Looks like we can resolve this.


Add owned bucket count metric for leader election

For #1523.

Introduce a metric called controller_owned_bucket_count (welcome better name)
Will report  that metric if both pod name and reconciler work queue name are available.
Add work queue name to LeaderAwareFuncs for webhook controllers.

Verified in Knative serving.

/assign  @mattmoor @vagababov

Do we actually serialize promote/demote now?

Part of them are serialized by lock, where I added the metric reporting: https://github.com/knative/pkg/blob/1127f59d7b73b75db42fed5ba9acce04ce4e1d77/reconciler/leader.go#L92

Is this still ongoing?

@yanweiguo any news?


3.3.4 starrocks 查询kudu表时间问题

In one word，only by following the SQL below can the data retrieval, statistics, and display be correct.only by following the SQL below can the data retrieval, statistics, and display be correct.（只有按照如下SQL，才能取数、统计和显示 都正确）
select id,
hours_sub(createtime, 8)  -- notes: display needs -8 hours（显示正确需要 -8小时）
from kudu_hive_catalog.db_noah.kudu_login a
where a.createtime >= '2024-10-11'  -- notes：>= not needs to process （大于等于却不需要处理）
and a.createtime < hours_add('2024-10-12', 8) -- notes：< needs +8 hours （小于的话 需要＋8小时）
The reason is that StarRocks is in the Shanghai time zone, while Kudu is in the UTC 0 time zone.
However, in order to achieve the correct result, the processing is not consistent.
The requirement is: from the user's perspective, the time handling should be unified (for example, adding or subtracting 8 hours to all), or the best approach would be for StarRocks to handle everything, so that when querying Kudu, users don't need to do anything with the time fields (whether in the SELECT clause or WHERE clause).（
原因是 starrocks是 上海时区，kudu是UTC 0时区，但是为了达到一个正确的结果，处理上不统一； 诉求是：对用户来说，处理时间字段能统一（比如全部都加或者减8小时）或者最好的方式是StarRocks处理好一切，查询kudu时，时间字段（不论在select子句还是where子句）用户什么都不用处理）

@SQLMaestroPro Thanks for the feedback, I have reproduced the problem and will fix it in these two days.


Moving the ui and workers steps earlier in the pipeline

Description
Let's deploy the base infrastructure before we do the final mirror and disconnect. This means will be
pulling from the HUB all the images and content needed for these steps.
Signed-off-by: Flavio Percoco<EMAIL_ADDRESS>Fixes # (issue)
Type of change
Please select the appropriate options:

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] This change is a documentation update

Testing
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Tested in production
[x] Also tested in CI

Test Configuration:

Versions: 4.10.9 / 2.4 / 4.9
Hardware: real hardware

Checklist

[x] I have performed a self-review of my own code
[ ] If a change is adding a feature, it should require a change to the README.md and the review should catch this.
[ ] If the change is a fix, it should have an issue. The review should make sure the comments state the issue (not just the number) and it should use the keywords that will close the issue on merge.
[ ] A change should not be merged unless it passes CI or there is a comment/update saying what testing was passed.
[ ] PRs should not be merged unless positively reviewed.

:tada: This PR is included in version 1.1.13 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Rework on Camera

Done in commit https://github.com/Sygmei/ObEngine/commit/1a897b5826a4d397a8a3dd6a411bf51e8c017906


fix: [Android TV] Text should not be accessible by default

Summary
Recent changes in the Text component make it accessible and pressable by default. On Android TV, this has the side effect of making it focusable. Only controls (Touchable, Pressable, TextInput, etc.) should be focusable.
Fixed by changing Text.js so that a Text component must explicitly have accessible={true} in order to be pressable or focusable on TV platforms.
Fixes #389 .
Test Plan
Tested with the TVInputDemo test app, and RNTester.

This is an interesting topic. I don't think accessible elements should be focusable but there's a line that explicitly does that: https://github.com/react-native-tvos/react-native-tvos/blob/tvos-v0.69.6/ReactAndroid/src/main/java/com/facebook/react/views/view/ReactViewManager.java#L58
Gotta check if it's really a necessity on Android but seemed strange to me 😄 I'm interested, and will check this one.
Your fix would do the trick as well, though. Thanks!

Thanks @yusufyildirim ! One thing I do need to check is that this change doesn't break VoiceOver (which is supported on Apple TV at least). If that turns out not to be an issue, then we are good. If it is, then we need a different fix.

@yusufyildirim I'm making one more adjustment, to make this change happen only on Android TV. It is not needed on Apple TV and could affect VoiceOver there.


Add LICENSE.md file to specify CC0 license

Discussed in issue #182.

With one review approving this PR (thanks @erget) and the conventions repo version of this PR merged and all looking good, I'm going to merge this.


Considering add combo service support?

Is there possible to add combo service for unpkg.com?
For example, concat files if we use such syntax:
https://unpkg.com/normalize.css@5.0.0/normalize.css,highlight.js<EMAIL_ADDRESS>

I'd prefer not to because the URLs are super ugly and it just complicates things for little gain. But if enough people want it sure, I'll consider it.


Update doc link and fix deprecated syntax

Documentation link for Map was returning 404.
ℹ Another way of doing this would be to delete the sample function, declare a module attribute @sample %{foo: 'bar', baz: 'quz'}, and then use @sample instead of sample() in the code below. Feel free to reject this PR depending on your personal preferences.

I am closing this PR, because I believe the attribute syntax is easier to read than the function syntax. I will add a commit with the attribute syntax instead.


[Form Validation] - Error message only appears after second blur

If form validation is configured to run on 'blur' and no errors already exist, the validation error message at the top of the form is not made visible until the second blur has been triggered.
Looking at the code it would seem that if the input field does not have the error class, then it simply validates that field, but if it does then the entire form is validated. In the form validation method, the error class is added to the form itself if validation fails, but the single field validation method does not do this, so the error class is not added to the form even when the fields validation fails, thus the error message is not displayed. The second blurring of the input then triggers the validate form method due to it already having failed, thus the form gets the appropriate class and the message shows.
Here is a fiddle demonstrating. If you simply click in the input, then out of it, the field will go red but no message will appear. Repeat this, and the message will appear
http://jsfiddle.net/qvwkzr0u/3/

when will this be fixed? It seems like a huge bug to me, probably not going to use semantic-ui's form validation because of it :(

@ziarno7 you can temporarily work around it, albeit fairly ugly. I would love to see a fix added to SUI, but for now you can add your own listeners to the onInvalid form settings. If you do this (and you can basically clone the existing one) then you can easily add it.
$.fn.form.settings.onInvalid = function(errors) {
    ...
    if (!$(this).closest('.ui.form').hasClass('error')) {
      $(this).closest('.ui.form').addClass('error');
    }
    ...
}

I've got this (amongst other things) in the codebase which is working for me. As I say, it's a little ugly and should be added to SUI, but this should keep things working until then

I couldn’t pinpoint when it was changed, but it seems that on: blur doesn’t trigger error messages at all, even after the second time; can you please confirm?

I tried examples from the doc at http://semantic-ui.com/behaviors/form.html#/examples
Only submit causes error messages.

I elaborated on @pareeohnos idea and came up with an ok work-around. I post it here in case it can be of use to someone:
$.fn.form.settings.onValid = function() {
    var form = $(this).closest('.ui.form');
    var list = form.find('.ui.error.message > .list');

    $(this).closest('.field').find('span[id^="validator-"]').each(function (index, messageElement) {
        list.children().each(function (i, c) {
           if ($(c).text() == $(messageElement).text()) {
               $(c).remove();
           }
        });
    });

    if (!list.children().length) {
        form.removeClass('error');
    }
};

$.fn.form.settings.onInvalid = function(errors) {
    var form = $(this).closest('.ui.form');
    var list = form.find('.ui.error.message > .list');

    errors.forEach(function (error) {
        var hasError = false;
        list.children().each(function (i, c) {
            if ($(c).text() == error) {
                hasError = true;
                return false;
            }
        });

        if (!hasError) {
            list.append('<li>' + error + '</li>');
        }
    });

    form.addClass('error');
};

It assumes that you created a span inside the div.field containing the error message that you want to display when necessary. As far as I know, it does not disrupt the display of messages when 'on' is set to submit or change.

I meet the same problem too, only submit will cause error messages

This has been fixed in the community fork Fomantic-UI by https://github.com/fomantic/Fomantic-UI/pull/1809
See your adjusted jsfiddle here http://jsfiddle.net/lubber/74ohc0p2/


tests: adjustments for Windows

Alter the path separators for Windows paths.  This makes 66% of the
remaining tests pass on Windows.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please test

@swift-ci please test and merge

@swift-ci please test and merge


SPARK-12868: Allow Add jar to add jars from hdfs/s3n urls.

What changes were proposed in this pull request?
Add jar command fails when given s3n or hdfs urls. This changes fixes that.
How was this patch tested?
unit test added, it was just copied from a previously opened PR https://github.com/apache/spark/pull/10797 so credit should be given to the person who originally posted the patch.

Can one of the admins verify this patch?

Can one of the committers take a look at this PR?

Can one of the committers take a look at this PR?

Request for review one more time.

Ping.

Request for review.

@srowen would this make sense to you?

Hi @Parth-Brahmbhatt, is it still active? If so is there any argument against https://github.com/apache/spark/pull/14720#discussion_r87908473?

It has been fixed by https://github.com/apache/spark/pull/17342.

I will propose to close this after a week if it is still inactive or no objection to ^.

closing.


List-generations empty.

Issue description
With efi & grub2.
Server (In grub menu 3 entries available):
root@server> nix-env --list-generations
root@server>

Laptop (Only 2 entries found, but in efi menu more than 100 entries exists.
nix-store --gc, nix-collect-garbage -d doesn't clean them...):
root@nixos_laptop> nix-env --list-generations
   1   2017-11-05 19:22:24
   2   2017-11-06 09:07:46   (current)
root@nixos_laptop> ls /boot/loader/entries/
nixos-generation-216.conf  nixos-generation-218.conf  nixos-generation-220.conf  nixos-generation-222.conf
nixos-generation-217.conf  nixos-generation-219.conf  nixos-generation-221.conf  nixos-generation-223.conf

Steps to reproduce
Server configuration.nix - https://gist.github.com/MrSorcus/a52ecc23ee126bc847599241f560641b
Laptop configuration.nix - https://gist.github.com/MrSorcus/3aa2f023d580d9a4a54416e917b0e821
nix-install
Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...)
Laptop: 18.03pre119245.5cfd049a03 (Impala)
Server: 18.03.git.eece875 (Impala)
Nix version: (run nix-env --version)
Laptop & Server: nix-env (Nix) 1.11.15
Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion)
Laptop: 18.03pre119245.5cfd049a03
Server: 18.03.git.eece875
Sandboxing enabled: (run grep build-use-sandbox /etc/nix/nix.conf)
Laptop: build-use-sandbox = false
Server: build-use-sandbox = true

Did you mean to list system generations?
nix-env --list-profiles --profile /nix/var/nix/profiles/system

Maybe. If --profile required, why in the manual nothing written about that?
Or i can't find it is...
Without --profile list/rollback/remove generations doesn't work...
But in manual all commands are written without this argument...

People do not tend to manipulate their system profiles with nix-env directly often, I think. Which part of the manual are you referring to?

https://nixos.org/nix/manual/#sec-profiles
https://nixos.org/nix/manual/#operation-list-generations
https://nixos.org/nix/manual/#operation-delete-generations
https://nixos.org/nix/manual/#operation-switch-generation
https://nixos.org/nix/manual/#operation-rollback

The Nix manual doesn't deal with anything related to the system profile, since that's a NixOS-specific thing. Instead, check out the NixOS manual and see if that helps.

Ok. Sorry, but how to delete old generations from uefi|grub?
nix-collect-garbage -d? Or nix-store --gc?
With grub2 after using both commands, entries not deleted from menu.

nix-collect-garbage -d or --delete-older-than ... as root is right. :)

Ok, good. Why old entries still exist in grub/uefi menu? Bug or not?

 Otherwise I think the current system will be prevented from being gc'ed.

Yes.

Hmm, not sure what's going on then. Perhaps someone else can weigh in.

@MrSorcus Perhaps you want to alter the issue text to refer to the problem at hand. The current one is a bit misleading. :)

@srhb Now good?

Are you sure you're running it as root? As in, sudo nix-collect-garbage -d.

root@nixos_laptop> whoami
root

Update.
Looks like that now it's working. But i'm not sure... I will try it later with clean installation.

On a clean installation, everything is just as empty.
nix-env --list-generations does not show records.
Either the hands are curves, or Nix.

It would be helpful if there is a parameter like --list-system-generations.

Perhaps we should have nixos-env that sets the --profile by default?

Is there any progress about it?


Delete  GoogleUtilities/README.md

Removes dated README in GoogleUtils directory.

@paulb777 @maksymmalyhin Is this something I should remove or should we keep it as a reminder of sorts?

@paulb777 @maksymmalyhin Is this something I should remove or should we keep it as a reminder of sorts?

I think removing the App Delegate Swizzler recommendation makes sense now. Later we will add another recommendation.


JSON export should represent dates as ISO8601

The json exports seem to serialize dates in Javascript format:
"addAccountDateInDate": "datetime.datetime(2009, 12, 19, 19, 46, 47)",

Rather than an ISO-based representation or similar. It would be better if these were represented as an ISO8601-compatible string.

It looks like the culprit is make_accounts_presentable which appears to try to find field values that are datetime objects and replaces them with a "printable" representation, but which I believe ends up being a non-standard representation.
Alternately, mintapi could use https://github.com/rconradharris/jsondate, which is a drop-in replacement for the python json module that does the right thing in terms of serializing out to ISO8601.


[bugfix] prevent synthetic onClick event bubble up

This temporary fix solves issue #345

Thx! This issue still remains with later iOS as well. Fixed bug  #131 for me.

However outside of Cordova this creates some bugs for me


Per-scope asciidoctor attribute values

Is it possible to provide different attribute values per jekyll scope?
ie:
asciidoctor:
  attributes:
     - imagesdir='/images'
defaults:
  -
    values:
      layout: "default"
  -
    scope:
      path: "asciidoctor1/subdirectory"
      type: "pages"
    values:
      layout: "custom"
      asciidoctor:
        attributes:
            imagesdir: "/different/path"

Which would override the value of imagesdir specified in asciidoctor->attributes for the given path?
Alternatively, all scope values could be blindly made available to asciidoctor:
defaults:
  -
    values:
      layout: "default"
  -
    scope:
      path: "asciidoctor1/subdirectory"
      type: "pages"
    values:
      layout: "custom"
      imagesdir: "/different/path"

(just two examples of possible syntax).
At this time I'm finding that the scope concept is entirely disconnected from asciidoctor configuration - there is no way to reference values between them in either direction (being able to reference them would be another way to reduce duplicated configuration)
If there is a way to do any of the above already, please point me in the right direction.

The layout part of this issue was reported again in #187 and a solution has been found.
Built-in AsciiDoc attributes such as imagesdir cannot yet be defined per scope. They can only be defined globally or in the document itself.

I'd guess this is related, I managed to enable asciidoctor-bibtex on my Jekyll site, but apparently the bibtex-file-attribute has to be set globally and cannot be set on a per document basis. So it is only possible to have one bibliography file for the whole site, which does not scale well.

Have you tried the soft set (@) modifier? https://docs.asciidoctor.org/asciidoc/latest/attributes/assignment-precedence/#altering-the-assignment-precedence


[mod_arithmetic] Organize modular addition

new top-level module mod_arithmetic that will contain modular arithmetic.
Move things from factoring/ and addition/ to this module
Simplify naming: [C][Mod]Op[K]

More to come after this

@tanujkhattar or @fdmalone can you ptal


Migrate to SciMLBase v2

This pr updates Integrals.jl to the new interface in SciMLBase v2, and needs https://github.com/SciML/SciMLBase.jl/pull/535 before merging.
It is a big rewrite since all of the calls to the algorithms needed to be rewritten to use the new interface correctly. And the deprecated interface still works! (There may be an edge case when the user specifies nout=1, but otherwise I checked all the old tests pass.)
This pr also improves how the IntegralsCuba.jl and IntegralsCubature.jl packages call the C-libraries so that the integrand can use arrays of any shape.
The AD was also updated to the new interface, including a bugfix for nested forward-mode AD, and there are opportunities to extend this pr by adding user-provided jacobians to IntegralFunction and BatchIntegralFunction.
This pr may fix many of the open issues. I think it addresses the following:
#169
#186
#164
#67
#86
#76
#40
And possibly others.
This should get the main branch up and running again.

One last test failure and it looks legitimate.

The solution is that we need to just get rid of the subpackages and make them extension packages. Since this is a breaking change we might as well fold it into the other breaking changes here. This will make maintenance a lot easier since now we don't need to independently release.

Okay that took a bit but 🎉 !


Convert from a Juypter notebook to a stand-alone program and containerize

The jupyter notebook needs to be converted to a stand alone program where others can contribute.
The program needs to be containerize along with data so that anyone can run the application without installing any component by cloning the docker.

Check this to know more about containerization.

The jupyter notebook has been changed to py file.

Containerization of the project is done by  #12


Howto import csv with custom formats

I have csv files that have colon seperated values wrapped by quotes. It is really easy importing these formats using the custom csv package that comes with python as it has a well defined set of attributes for handle different csv formats. The tablib support for csv on the other side seems very limited. Is there a way to use the use csv.DictReader as an alternative import approach for importing data? What feature of the tablib.Dataset class are used within the import workflow of import-xport that csv.DictReader lacks?
X":"COMMENT":"CORES":"EDITIONKEY":"FARM":"FQDN-LOGIN":"HOSTNAME":"HT-CORES":"HW-FREQUENCY":"HWARCH":"KERNELARCH":"LASTUPDATE":"LICENSEKEY":"MEMORY":"MODEL":"NAME":"PARENT":"SERIAL":"SHORTNAME":"SOCKETS":"SOURCE":"UUID":"VCENTER":"VCENTERPATH":"VIRTUALHUMAN":"VLT-CODE":"POWERSTATE":"CONSOLE"
"C":"":"4":"":"":"machine03p141b.home.sec.local":"machine03p141":"4":"":"":"x86_64":"2016-01-04":"":"":"":"":"564df441-1e6a-b0ca-88fb-f9ece7fa3907":"":"machine03p141":"":"SSH":"613081d7b9f56b42f414be51edbf1ca61fc229a6":"":"":"OS Instance":"6":"":"appl95p01b.home.sec.local"
"C":"":"4":"":"04":"":"":"4":"":"":"":"2016-01-04":"":"12288":"":"machine04p600":"7fe6c80ccc87e8081bf06940d2f404cfaaa4afab":"":"machine04p600":"1":"API":"421b66a1-bb86-c5e1-97e1-d029ab3f1f00":"":"Datacenters/home_virt_dc/host/ESXi/dc-APPZ-2_14_2680/machine04s62.home.sec.local/machine04p600":"Virtual Machine":"5":"poweredOn":"appl95p01b.home.sec.local"
"C":"ED-Medienarchiv App 1.1 Q":"4":"":"":"machine01p74.home.local":"machine01p74":"4":"":"":"x86_64":"2016-01-04":"":"":"":"":"564d933f-592a-ae54-8fa5-6fd9f4cffbaa":"":"machine01p74":"":"SSH":"c4f53878e6ac55041a2db176c13e2bc5a4108e3d":"":"":"OS Instance":"6":"":"appl90p01.home.sec.local"

The **kwargs of tablib's import_set() are directly passed on to Python's csv.reader(). Unfortunately, django-import-export's Format.create_dataset() does not pass any additional arguments to import_set() which leaves you with the option of subclassing your own Format and adding the dialect there.

Thanks for your finding. Can you explain why not add the **kwargs to the base_formats? I'd create a pull request for this.

If you want a custom delimiter, you can do the following:
Create a new Format class and overwrite the create_database method:
`class SemicolonCSV(base_formats.CSV):
def get_title(self):
    return "semicolon_csv"

def create_dataset(self, in_stream, **kwargs):
    return tablib.import_set(in_stream, format='csv', delimiter=';')`

But you must also create a class based on ImportMixin to overwrite get_import_formats method, adding the new format:
`class CustomImportMixin(ImportMixin):
def get_import_formats(self):
    """
    Returns available import formats.
    """
    self.formats += [SemicolonCSV]
    return [f for f in self.formats if f().can_import()]`


accessKeyId / secretAccessKey ?

sorry if i've missed something but where do you put your accessKeyId / secretAccessKey ?
also would this work for koa ?

They are coming from ENV vars and automatically picked up by aws-sdk, we do not ever specify keys/secrets anywhere in the app.

Never used koa, but I assume it should be able to use standard SessionStore implementation, which this thing is. If it does not - then prob not...

ok thanks.


Android support

Currently unreal sdk does not support android OS.
Can you please provide the support on android.

Audio-only support for iOS would be great too.
Looking forward to have Quest Pro (android VR) and iPad (iOS) XR experience with voice chat from external users.

By any chance, is there any news about a tentative date for Android feature? Thanks in advance!

@malcriado3d Sorry about the delay, I was on holiday. I believe there has been significant progress made in the plugin's underlying component so this feature may come fairly soon, but I haven't had the time to check @codlab 's work yet. Maybe he can say more :)

is there any news on support for Android devices? thanks in advance!

Are there any timeline updates for the Android support? The big project is coming in October, and we would love to have Dolby comms integration.

We currently have 3 remaining facing issues that are not specifically about unreal but the overall layer underneath it. Those are about early spatialisation, audio device selection beside a last one a bit more important in term of work which is video related (sender side). I'm confident we can have those first 2 by the end of august - for the video, it'll be a bit more work (but at least without any crashes by that time as well) 👀

Hhhmmm it's a bit of uncharted territory but I'll try to push something forward either way 🤔 (beside sending video as this one is definitely an unknown on my side, can't commit to a clear timeline rn !)

Thank you for your quick answer!
Is it possible to have some early tests running (even if it is crashing)? Our client (Porsche) is heavily invested in the project and we would love to prepare all the necessary implementation (like voice chat ui) ready on our side.

phih369, I am the product manager at Dolby for the plugin. I would like to talk with you about your project, and to see how we can assist you further. If you are interested, please contact me at gary.makin with a domain name of dolby.com.

Version 1.2.0 contains an experimental plugin build for Android. It supports arm64-v8a devices and UE5.3 only. Please see this tutorial for how to use it. Since the Android version is based on a different C++ SDK which has not been released, it may require some additional "ifdefing" to make both a desktop and an Android version work at the same time. The plugin can work on an older UE5 but it would require additional configuration. This project will not longer be maintained so this experimental build is the best we can do right now.


Add twig functions

I'd like to have a version function that gets the 3 variables injected into the DI
application.version - the application's version (the format depends on the configured versioning strategy)
application.version_hash - an MD5 digest of the version
application.release_date - an [RFC 3339](https://tools.ietf.org/html/rfc3339) formatted date on which the version was last incremented

<footer>{{ app.name }} {{ version() }} {{ version_hash() }} {{ version.release_date() }} </footer>

They could also be injected as twig globals.
AbstractBundle makes this kind of stuff easier, but it's Symfony 6.4+ only.
(Okay, I'll stop with the issues!  I'm just kinda excited about using this...)

Actually, I'd like a 'application' to contain more than the version info, I'd like to it contain name, code, github repo, etc.  That's above and beyond this bundle, but I noticed that the prefix is bizkit, and I wonder if you had ideas for including more than just versioning.  Since you're already tightly tied with the versioning system, I'd love to have the name (the "name" property in composer.json) and branch also injected into the DI.  And a link to the github repo.
Above and beyond this bundle, but just brainstorming about things I'm looking at when adding the version to the applications.