Consolidate Scene3D with ign-gazebo's GzScene3D

We currently have 2 separate scene plugins in the Ignition ecosystem, one in ign-gui and one in ign-gazebo. The split into 2 scene plugins was something we did to move forward quickly, but the long-term goal is to combine those into a single plugin in a way that all Gazebo-independent features can be pushed into ign-gui so all downstream projects can take advantage of it and there's less duplication to maintain.
Some possible ways to achieve this:

Move all Gazebo-specific features to separate plugins that interact with the 3D scene
Make the ign-gui plugin extensible, similar to how we have ignition::gui::TransportPlotting and ignition::gazebo::Plotting.

we are using the Scene3D plugin in the delphyne project and have noticed some errors when loading mesh materials, but I believe this has already been fixed in ign-gazebo, so I'm cross-porting the fix from ign-gazebo's SceneManager in #191

Related PRs

MoveToHelper (ign-rendering) https://github.com/ignitionrobotics/ign-rendering/pull/311
New Scene3D interface https://github.com/ignitionrobotics/ign-gui/pull/217
New Scene3D plugin using the interface provided in ign-gui https://github.com/ignitionrobotics/ign-gazebo/pull/798

@chapulina, I have been working in adding small plugins to ign-gazebo and ign-gui. I updated the ign-gazebo PR with some of the plugins.

ign-gui

view_control  https://github.com/ignitionrobotics/ign-gui/pull/231/
Move to / follow  https://github.com/ignitionrobotics/ign-gui/pull/226

ign-gazebo

Select https://github.com/ignitionrobotics/ign-gazebo/pull/854
Transform controls https://github.com/ignitionrobotics/ign-gazebo/pull/854

I have created some branches to test this (Fortress). Because it requires some cherry picks:

ign-gazebo https://github.com/ignitionrobotics/ign-gazebo/tree/ahcorde/6/scene_manager_test
ign-gui https://github.com/ignitionrobotics/ign-gui/tree/ahcorde/6/new_scene_test
ign-common https://github.com/ignitionrobotics/ign-common/tree/ahcorde/3_to_4

Run the demo:
ign gazebo --gui-config src/ign-gazebo/src/gui/gui.config shapes.sdf -v 4

some branches to test this

Nice, I see the 3D scene is updated!
I assume all the plugins above should work with this combination of branches?

view_control #231

The ViewControl plugin is empty for me, no buttons

Move to / follow #226

Right-clicking the 3D scene doesn't give me a context menu to be able to choose these options.
Choosing them from the entity tree doesn't seem to work.

Select ignitionrobotics/ign-gazebo#854

I can select entities on the 3D scene and they get highlighted, but I don't see them highlighted on the entity tree. The same for the other direction.

Transform controls ignitionrobotics/ign-gazebo#854

They work if I:

Deselect all entities
Choose translate mode
Select entity

But don't work if I do:

Select entity
Choose translate mode

The ViewControl plugin is empty for me, no buttons

I called view_control but it's the plugin to move the camera with the mouse.

Right-clicking the 3D scene doesn't give me a context menu to be able to choose these options.

The Scene3D it's on ign-gui, the context menu is part of ignition-gazebo, I didn't find a way (yet) to modify the qml of the scene3d from ign-gazebo and include the.

Outstanding issue:

[ ] How to choose what camera plugins are being attached to, see https://github.com/ignitionrobotics/ign-gui/pull/231#discussion_r658353941)

Main PRs splitting the functionality:

https://github.com/ignitionrobotics/ign-gui/pull/221
https://github.com/ignitionrobotics/ign-gazebo/pull/813
https://github.com/ignitionrobotics/ign-gui/pull/226
https://github.com/ignitionrobotics/ign-gui/pull/231
https://github.com/ignitionrobotics/ign-gui/pull/247
https://github.com/ignitionrobotics/ign-gazebo/pull/854
https://github.com/ignitionrobotics/ign-gazebo/pull/988
https://github.com/ignitionrobotics/ign-gazebo/pull/900
https://github.com/ignitionrobotics/ign-gazebo/pull/983
https://github.com/ignitionrobotics/ign-gazebo/pull/1016
https://github.com/ignitionrobotics/ign-gazebo/pull/1014

TODO

[ ] Drag & drop from Fuel / meshes
[ ] Context menu
[ ] Port rendering fixes from GzScene3D to MinimalScene, such as https://github.com/ignitionrobotics/ign-gazebo/pull/774
[ ] Choose a render engine from the command line (this information needs to be sent to MinimalScene before it creates a scene.)

Drag & drop

ign-gui PR https://github.com/ignitionrobotics/ign-gui/pull/282
Still a branch https://github.com/ignitionrobotics/ign-gazebo/tree/ahcorde/plugin%2Fdrop_from_fuel

Context Menu

Added in this draft PR https://github.com/ignitionrobotics/ign-gazebo/pull/1014

Port rendering fixes from GzScene3D to MinimalScene

Added in this draft PR  https://github.com/ignitionrobotics/ign-gazebo/pull/1014

Choose a render engine from the command line (this information needs to be sent to MinimalScene before it creates a scene.)

https://github.com/ignitionrobotics/ign-gazebo/pull/1034
https://github.com/ignitionrobotics/ign-gui/pull/286

@chapulina can we consider this as done ?

Yes! :tada:


Do you have plans to support CoLLiE using 3D parallel training in LLaMA-Factory

https://github.com/OpenMOSS/CoLLiE
https://openlmlab-collie.readthedocs.io/zh-cn/latest/tutorials/collie-tutorial-6-parallel.html

we plan to support pipeline/tensor parallelism using more advanced libraries in the future

@hiyouga  Which advanced library will you integrate?


Review layouts and responsiveness in pages

There are inconsistencies in the page layouts, regarding how the content is positioned and scrolled. This is specially noticeable in the large size, and not in the smaller widget. So far, I've noticed that Fund account, Receive and Secret keys pages have vertically centered contents. When the viewport is shorter, the elements collide with the header and get cut off the viewport. Unlike the Send page, which has normal scrolling and positioning and any viewport size.
Scope

[x] Review all pages responsiveness (narrower and shorter viewports)
[ ] Propose a design spec for consistent page layouts going forward https://github.com/hirosystems/stacks-wallet-web/issues/2322
[ ] Check other issues about pages https://github.com/hirosystems/stacks-wallet-web/issues/1947

Examples

As @fbwoolf has been improving the responsiveness of the app, the first step would be to review this PR https://github.com/hirosystems/stacks-wallet-web/pull/2277 and check if the visual bugs persist.

#2277 Will fix all centering and scrolling issues and will get us to a better place in terms of layout and responsiveness. Still, I found more inconsistencies in the headings and navigation. The next step is re-define the spec for the layout and navigation.

Closing this off as I believe it to be outdated and we have improved our responsiveness a lot in https://github.com/leather-io/extension/pull/5176


Improve "time until rebase" to show hh:mm

[ ] when > 60 minutes remain, show "__ hours, __ minutes" or instead of just "__ hours"
Or any other time format you can come up with that makes sense and looks good!
[ ] Make sure the output text is localized properly

Discord user Ishmis (Sekula#0203) offered to take a stab at this

checked the dApp, this looks to be already implemented


SyntaxError: Unexpected identifier

Hi, since the update to fix opn, I get the following error. Mirror won't start. removing the module from the config allows the mirror to start correctly. Tried to rollback but failed.
The ^^^^ is under 'path'.
Thanks
pi@PicturePi:~/MagicMirror/modules/MMM-GooglePhotos $ node auth_and_test.js
/home/pi/MagicMirror/modules/MMM-GooglePhotos/node_modules/open/index.js:11
const wslToWindowsPath = async path => {
^^^^
SyntaxError: Unexpected identifier
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:549:28)
at Object.Module._extensions..js (module.js:586:10)
at Module.load (module.js:494:32)
at tryModuleLoad (module.js:453:12)
at Function.Module._load (module.js:445:3)
at Module.require (module.js:504:17)
at require (internal/module.js:20:19)
at Object. (/home/pi/MagicMirror/modules/MMM-GooglePhotos/auth.js:6:13)

It's very weird thing.
After git pull, Did you npm install in MMM-GooglePhotos directory?

Yes. I tried npm install. Then I renamed the MMM-GooglePhotos directory and started fresh.
Then I found this https://github.com/tapio/live-server/issues/305 which is the same issue and points to an out of date NodeJS. So I tried several different versions. But that didn't help the situation. On the mirror I currently have 8.9.1 and now get the error -
module.js:538
throw err;
^
Error: Cannot find module './credentials.json'
at Function.Module._resolveFilename (module.js:536:15)
at Function.Module._load (module.js:466:25)
at Module.require (module.js:579:17)
at require (internal/module.js:11:18)
at new Auth (/home/pi/MagicMirror/modules/MMM-GooglePhotos/auth.js:31:15)
at Object. (/home/pi/MagicMirror/modules/MMM-GooglePhotos/auth_and_test.js:16:14)
at Module._compile (module.js:635:30)
at Object.Module._extensions..js (module.js:646:10)
at Module.load (module.js:554:32)
at tryModuleLoad (module.js:497:12)
I haven't had time to try anything different. MagicMirror is up to date, pi is up to date.

Error: Cannot find module './credentials.json'
at Function.Module._resolveFilename

hmmm. this error is another thing. you've probably missed your credentials.json on newly installed module.

Have you still got a problem? I'll close this issue because there is no more feedback. Have a nice day.


Update original.md

add missed imported function

Thanks!

:tada: This PR is included in version 5.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Dropping Julia 0.4, upgrade to 0.5 and 0.6. Fix examples.

modified:   REQUIRE                           Compat version covering…… relevant changes in 0.6
modified:   examples/chat-client.html  Prepending messages -> Appending.
modified:   examples/chat.jl                0.6 update (take_buf, Array constructors)
modified:   examples/repl-client.html   Prepending messages -> Appending. Javascript feedback, no script error when connection close.
modified:   examples/server.jl              0.6 update (take_buf, Array constructors). 'Eval' error handling and feedback. More type stability.
modified:   src/WebSockets.jl               0.6 update (take_buf, Array constructors). Precompilation.
modified:   test/runtests.jl                   0.6 update (take_buf, Array constructors)

Coverage decreased (-0.8%) to 34.4% when pulling 01f33a9f9e19aa1793c1d8d0f6521cd763de91b0 on hustf:Ju_5_6 into 8a65b495f3f25761d0672764c70456bed8f176a6 on JuliaWeb:master.

Awesome! Thanks!! :+1:


Fails cross-compilation on musl

/home/vi/.cargo/registry/src/-3d9d141e372ea94e/renamore-0.3.0/src/linux.rs:26: undefined reference to renameat'
Shall it expose a cargo feature to force fallback and/or force usage of syscall or something like that without relying on C library?

From what I can tell, the musl standard library has renameat. It's right here. Still, the platform detection could be improved. The build.rs could check for the existence of the symbols. Doing a direct syscall is probably not a good idea because the syscall number would depend on the architecture (see the definitions in musl).

My current workaround is to build this file:
#define _GNU_SOURCE 1
#include <sys/syscall.h>
#include <unistd.h>

int renameat2(int oldfd, const char *old, int newfd, const char *new, unsigned int flags)
{
	return syscall(SYS_renameat2, oldfd, old, newfd, new, flags);
}

and use linker wrapper script in ~/.cargo/config's:
#!/bin/sh
cc "$@" /path/to/renameat2_amd64.o

Oh it's renameat2 not renameat! If the syscall exists and musl isn't providing a wrapper then that sounds like a problem with musl. In this case, checking for the existence of the renameat2 wrapper inside of build.rs wouldn't help you because you'd end up with the fallback even though the real thing exists.
So yeah (you edited your comment while I was writing mine!), this specific workaround for musl could be added so that the build.rs includes the above definition if the target_env="musl" and the symbol couldn't be found.

If the syscall exists and musl isn't providing a wrapper then that sounds like a problem with musl.

Musl seems to use renameat2 if it is available, but only expose renameat function.

I pushed something up. I haven't tested it. Let me know if it works for you.

Shall a special Cargo feature to turn off all renamore features (i.e. always fail rename_exclusive, always false rename_exclusive_is_atomic, always falls back in rename_exclusive_fallback) be exposed anyway, even with musl support being corrected?
This would allow users to continue building apps (in reduced mode) by specifying something like --features renamore/force-fallback.

Yeah I think so. It would be an easy workaround for future issues with platform detection.

I pushed something up. I haven't tested it. Let me know if it works for you.

It stops issuing exclusive renames on musl at all, even if I have my workaround active.
No mentions of renameat2 symbol in librenamore rlib.

I found some time to do some further investigation into this and I think I know what's going on.
The host target is aarch64-unknown-linux-gnu and I'm compiling for aarch64-unknown-linux-musl. Constructing the command to invoke the compiler on the test.c file is failing because the compiler is not installed. This causes an early-exit and the linker cfg variable is not set. I installed the compiler with sudo apt install musl-dev. After that, the command was able to run and the compiler returned a non-zero exit code as expected. What I didn't expect was that cfg!(target_env = "musl") was false even when passing --target aarch64-unknown-linux-musl. So the linux-musl.c file isn't compiled. When I comment out the if and let it run unconditionally, the tests pass.
Thinking about it a little more, it actually makes total sense. The build script is built for the host. It needs to run on the host. So of course the cfg variables will describe the host. To get the true target, you need to use the TARGET environment variable. With that, it now seems to be working. Although I seem to be having some problems on Windows but that's a separate issue (oh Windows!).
Let me know if this is working in your situation.

Trying with adbfa625bfb3242bef0739288b55d3adc1dff2f2:

x86_64-unknown-linux-gnu: works
x86_64-unknown-linux-musl + workaround: multiple definition of renameat2; /mnt/fluff/rust-targets/duplo/x86_64-unknown-linux-musl/debug/deps/librenamore-7a95ced6ab76a441.rlib(linux-musl.o):/mnt/bkel/vi/src/renamore/src/linux-musl.c:5: first defined here
x86_64-unknown-linux-musl without workaround: works, librenamore.rlib defines renameat2.
x86_64-unknown-linux-musl + workaround + CC environment variable points to the same compiler wrapper used for the workaround: works, librenamore.rlib does not define renameat2.

If non-Linux works properly then this version looks suitable for the v0.3.1 release.

If non-Linux works properly then this version looks suitable for the v0.3.1 release.

Well that's the thing. It's fine on Linux and macOS but I haven't been able to get it to work on Windows. Although I've realised that the way I've approached this is a little silly.

Linux added renameat2 in kernel version 3.15. The oldest supported LTS version of the Linux kernel is 4.14.
macOS added renamex_np in 10.12 (Sierra). The oldest supported macOS version is 11 (Big Sur).
Windows added MoveFileExW in Windows XP. The oldest supported LTS version of Windows is Windows 10.

I don't think anyone is creating new software for operating systems that aren't receiving security patches anymore. However, there is some special handling required on Linux due to the diversity of C standard library implementations. So I'll remove the macOS and Windows checks but leave the Linux code path as it is.
Making renameat2 a weak symbol makes sense for the sake of robustness so I'll do that. With that, I'll probably release this later today (likely within 12 hours).

v0.3.1 has been published with the fix. Thanks for your help on working through this issue.


cd to sub directory of directory with spaces does not work as slash gets removed

[ ] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[x] bash
[ ] zsh
[ ] fish

Problem / Steps to reproduce

I have following things in the bashrc file
source "$HOME/fzf-tab-completion-master/bash/fzf-bash-completion.sh"
bind -x '"\t": fzf_bash_completion'
export FZF_COMPLETION_AUTO_COMMON_PREFIX=true
[ -f ~/.fzf.bash ] && source ~/.fzf.bash

I think you should report the issue to https://github.com/lincheney/fzf-tab-completion


Unit Testing Failure

st_results/easy_perception_deployment/epd_test_init.gtest.xml
1: [==========] Running 3 tests from 1 test case.
1: [----------] Global test environment set-up.
1: [----------] 3 tests from EPD_TestSuite
1: [ RUN      ] EPD_TestSuite.Test_readSessionUseCaseConfigTextFile_EPDContainer
1: onnx_model_path = ./data/model/squeezenet1.1-7.onnx
1: unknown file: Failure
1: C++ exception with description "Load model from ./data/model/squeezenet1.1-7.onnx failed:Load model ./data/model/squeezenet1.1-7.onnx failed. File doesn't exist" thrown in the test body.
1: [  FAILED  ] EPD_TestSuite.Test_readSessionUseCaseConfigTextFile_EPDContainer (11 ms)
1: [ RUN      ] EPD_TestSuite.Test_setFrameDimension_EPDContainer
1: onnx_model_path = ./data/model/squeezenet1.1-7.onnx
1: unknown file: Failure
1: C++ exception with description "Load model from ./data/model/squeezenet1.1-7.onnx failed:Load model ./data/model/squeezenet1.1-7.onnx failed. File doesn't exist" thrown in the test body.
1: [  FAILED  ] EPD_TestSuite.Test_setFrameDimension_EPDContainer (0 ms)
1: [ RUN      ] EPD_TestSuite.Test_setInitBoolean_EPDContainer
1: onnx_model_path = ./data/model/squeezenet1.1-7.onnx
1: unknown file: Failure
1: C++ exception with description "Load model from ./data/model/squeezenet1.1-7.onnx failed:Load model ./data/model/squeezenet1.1-7.onnx failed. File doesn't exist" thrown in the test body.
1: [  FAILED  ] EPD_TestSuite.Test_setInitBoolean_EPDContainer (1 ms)
1: [----------] 3 tests from EPD_TestSuite (12 ms total)
1:
1: [----------] Global test environment tear-down
1: [==========] 3 tests from 1 test case ran. (12 ms total)
1: [  PASSED  ] 0 tests.
1: [  FAILED  ] 3 tests, listed below:
1: [  FAILED  ] EPD_TestSuite.Test_readSessionUseCaseConfigTextFile_EPDContainer
1: [  FAILED  ] EPD_TestSuite.Test_setFrameDimension_EPDContainer
1: [  FAILED  ] EPD_TestSuite.Test_setInitBoolean_EPDContainer

Solution
Disabling the generation of .json seems to resolve the issue. But why?

Solution :partying_face:
Just need to properly close the std::ofstream object via the following function call:
outputFileStream1.close();


Add support for parsing code_scanning_alerts in webhooks

Following up on the issue I created https://github.com/google/go-github/issues/2391.
Let me know if there is anything missing or needs fixing. I just tried to follow the patterns from the other types.
Cheers,
emil
Fixes: #2391.

Thanks @gmlewis! :D

Thank you, @raynigon !
Merging.


chore: use Depot for flagsmith-api image build

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[ ] I have added information to docs/ if required so people know about the feature!
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Changes
This makes the flagsmith-api image builds to be run by Depot.
How did you test this code?
Tested in the initial PR (https://github.com/Flagsmith/flagsmith/pull/3071).
flagsmith-api image should be pushed by Depot in CI.

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

Comparison is base (fc3bce0) 96.11% compared to head (e11367e) 96.11%.
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3179   +/-   ##
=======================================
  Coverage   96.11%   96.11%
=======================================
  Files        1059     1059
  Lines       32037    32074   +37
=======================================
+ Hits        30792    30829   +37
  Misses       1245     1245

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Broken loading of IDA libs in IDA 7.0 (Found in StructTyper)

Hello,
Setup:

Clean Install of IDA 7.0 with Flare-IDA install per README.md
WIndows 7 x64

When running the StructTyper plugin, I receive the following error:
Failed while executing plugin_t.run():
Traceback (most recent call last):
  File "C:/Program Files/IDA 7.0/plugins/struct_typer_plugin.py", line 47, in run
    idaapi.require('flare.struct_typer')
  File "C:\Program Files\IDA 7.0\python\ida_idaapi.py", line 153, in require
    m = importlib.import_module(modulename, package)
  File "C:\python27-x64\Lib\importlib\__init__.py", line 37, in import_module
    __import__(name)
  File "C:\Program Files\IDA 7.0\python\flare\struct_typer.py", line 48, in <module>
    g_dll = ctypes.windll[idaname + ".wll"]
  File "C:\python27-x64\Lib\ctypes\__init__.py", line 437, in __getitem__
    return getattr(self, name)
  File "C:\python27-x64\Lib\ctypes\__init__.py", line 432, in __getattr__
    dll = self._dlltype(name)
  File "C:\python27-x64\Lib\ctypes\__init__.py", line 362, in __init__
    self._handle = _dlopen(self._name, mode)
WindowsError: (126, 'The specified module could not be found')

The key error seems to be in:
g_dll = ctypes.windll[idaname + ".wll"]

Version 7.0 looks to have dropped the .will files (or I missed something on the installation).  I tried changing it to .DLL and now get the following:
    Failed while executing plugin_t.run():
    Traceback (most recent call last):
      File "C:/Program Files/IDA 7.0/plugins/struct_typer_plugin.py", line 47, in run
        idaapi.require('flare.struct_typer')
      File "C:\Program Files\IDA 7.0\python\ida_idaapi.py", line 153, in require
        m = importlib.import_module(modulename, package)
      File "C:\python27-x64\Lib\importlib\__init__.py", line 37, in import_module
        __import__(name)
      File "C:\Program Files\IDA 7.0\python\flare\struct_typer.py", line 70, in <module>
        print_type_to_one_line = g_dll.print_type_to_one_line
      File "C:\python27-x64\Lib\ctypes\__init__.py", line 375, in __getattr__
        func = self.__getitem__(name)
      File "C:\python27-x64\Lib\ctypes\__init__.py", line 380, in __getitem__
        func = self._FuncPtr((name_or_ordinal, self))
    AttributeError: function 'print_type_to_one_line' not found

Thanks for the report. IDA 7.0 SDK changed a lot. We'll be updating our plugins in the next few months as we have time Feel free to add additional bug reports as you encounter them so  we can track them.


Build Docker on release, not (just) push

Currently we build and push a docker image every time we push to main: https://github.com/neurobagel/bagel-cli/blob/1d4cb5217813cedfbe6d78fcb2821fac72059089/.github/workflows/cd.yml
That only really makes sense for testing builds, not for things we deploy to production (c.f. https://github.com/neurobagel/documentation/wiki/Release-process#what-happens-next-todo).
What we should do is:

[ ] make a second deploy workflow that listens for "release: released" (see here), have that build and tag with release number
[ ] only deploy those explicitly tagged images to production
[ ] release often (so we have nice things running in production), look into https://github.com/containrrr/watchtower or https://www.ansible.com/ (?)

This now works: https://github.com/neurobagel/bagel-cli/blob/main/.github/workflows/build_docker_on_release.yml


Ask server to skip cancel check

For these two tests, in some rare setting. The server may check fail at https://github.com/grpc/grpc/blob/b8759c2af4e234c724fc2898b0f444c51d0782cf/test/cpp/end2end/test_service_impl.cc#L200
The same argument is set for other cancel tests.

@yang-g can this server check if the network connection breaks? I see this check fails occasionally in cfstream_test. Would the same fix also be applicable to test/cpp/end2end/cstream_test?
Logs for failed test: https://source.cloud.google.com/results/invocations/a2cf9465-cd35-496f-a7e8-1cf6ca347c50/targets/grpc%2Fcore%2Fmaster%2Fmacos%2Fgrpc_cfstream/log

I believe IsCancelled may return true when the network connection breaks.
On Mon, Jun 3, 2019 at 2:00 PM rmstar<EMAIL_ADDRESS>wrote:

@yang-g https://github.com/yang-g can this server check if the network
connection breaks? I see this check fails occasionally in cfstream_test.
Would the same fix also be applicable to test/cpp/end2end/cstream_test?
Logs for failed test:
https://source.cloud.google.com/results/invocations/a2cf9465-cd35-496f-a7e8-1cf6ca347c50/targets/grpc%2Fcore%2Fmaster%2Fmacos%2Fgrpc_cfstream/log
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/grpc/grpc/pull/19219?email_source=notifications&email_token=ACNHICRCIPIWOTTI2DBUBJTPYWA77A5CNFSM4HSRAV72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW2VXWI#issuecomment-498424793,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACNHICUXDW6JQZVKW7A4EBLPYWA77ANCNFSM4HSRAV7Q
.

Test Failure #18416


MudSelect: Refactor control flow in MudSelect.razor.cs so subscriptions of the "OnKeyDown" event will be triggered correctly for the "Enter" key

Description
Replaced return with break to ensure method continues after opening the menu. Moved logic for selecting an option when the menu is already open into an else block for better readability. Made minor formatting changes for consistency.
Fixes #9594
How Has This Been Tested?

Created a simple component using MudSelect
Subscribed to the "OnKeyDown" event of the MudSelect with a handler that writes a console log whenever it is triggered
Pressed "Enter" key to verify that every "KeyDown" is logged to console

Type of Changes

[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)
[ ] Documentation (fix or improvement to the website or code docs)

Checklist

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

Does this impact the Autocomplete component as well?

I wouldn't see how, but I'm not super familiar with the solution yet.
The Autocomplete component does handle the OnKeyDown event aswell, however I couldn't find any logic related to the "Enter" key in particular.
Also, MudSelect already had the behavior I implemented when used in MultiSelect mode, that's why again I think it should have no effect on the Autocomplete component.

Thanks


[UPGRADE] kotlin 1.3.70

https://github.com/JetBrains/kotlin/releases/tag/v1.3.70

In progress now.

@jarmstrong
I accidentally converted this issue to a PR. The new binaries are now available. If you see any problem, please open a new issue. Again, apologies for messing up this issue.

Thanks very much will give it a spin!


Add loading icon when user taps 'Check again' on final page

From chat:

The spinner works on embedded pages, it just might need to have its CSS adjusted because the context around the  might be obscuring it.
You can manually show the spinner with daShowSpinner() and hide it with daHideSpinner().
When the spinner is showing, use the Inspect tool and find the div with the id of daSpinner.  It is at the end of the HTML, near the

Fixed in some commit for the multi-signer/multi-device stuff I think.


Only run tests affected by git changes

I believe there is an easy way for DoCIF to give us what was changed in the current PR and we can easily find the list of unit tests that are affected by that.
This might get complicated if we decide to make integration tests.
Still, I'm leaving this issue open to encourage investigation on doing this, but it's very low priority

smol note: bazel would let us do that.

@joshuamorton if you really want bazel I'm ok with it but like you'd have to make the PR yourself

At this point call it a slightly more serious "refactor to use lisp".

I'm ready to stop any and all lisp PRs.

Ok so actually we'd probably want to use something like this: https://circleci.com/docs/2.0/workflows/, but I'm not sure how it plays with DoCIF.

@joshuamorton this is actually a fairly complicated issue.
Originally, based on early descriptions of circleci 2.0, it seemed like I could keep the same workflow DoCIF provides (building a baseimage for caching so build performance is much improved), so I wanted to build a 2.0 backend for DoCIF. However circleci 2.0 ended up being a lot less powerful than I originally anticipated, which makes it it impossible to accomplish on 2.0.
very late in the 2.0 beta, circleci introduced workflows, which sovled half the issues I had with 2.0, but created many new ones. In my opinion, 2.0 is a limited, but polished product, while workflows is more of an alpha pushed out to release early. Despite this, I do think workflows is interesting, and I use it on a few of my projects.
For example, workflows do not support forked pull requests at all, and break email notifications (repeatedly send failure and pass emails), which breaks most of my personal use cases.
For your case, which currently does not use any of the baseimage caching features DoCIF provides, your best bet is probably to switch to circleci 2.0 without workflows. If you would like baseimage caching and easy local builds, you're pretty much forced to stick with 1.0 and DoCIF. Workflows might be interesting someday, but it's pretty broken right now.
I don't think that running tests only related to the current change is a goal that is needed at your current stage. Even for extremely large companies it's important to run all tests on all commits to make sure no complex interactions between dependencies are missed. Most of the time this is done to improve developer experience by improving CI time, but since most of your CI time is installing dependencies (not actually running tests), your CI time can be vastly improved by building a custom baseimage (to be manually updated) via 2.0, or using DoCIF's caching feature (which auto-updates images).

Even for extremely large companies it's important to run all tests on all commits to make sure no complex interactions between dependencies are missed.

As someone working at a Very Large Company, this isn't exactly true. It would significantly slow development, but the greater point is true. This is a very premature optimization.

As someone working at a Very Large Company, this isn't exactly true.

I guarantee that every competent large company is running all tests on all commits at some point in their build process. While this might be hidden from developers, or developers might turn this off for local testing, or it might be run only before merging into source control to speed things up, there are always cases that the build system simply cannot predict. (as a trivial example) if test A runs code that deletes dependencies for another unrelated project that happens to be colocated on the same host, it will break the other project despite being painfully easy to catch via it's tests.
If you are talking about google (since they seem to be the largest software company right now), their testing solution is not one to strive to achieve (imo at least), since most of their products are painfully buggy and regress often. For example, hangouts recently added a new forced popup on startup for me, which when closed, freezes all of hangouts, making it completely unusuable. The official homepage (google.com) currently has a bug with the javascript spell check notifier where it flickers on/off at random intervals and places. Both of these bugs (and many, many, others I've encountered) could be attributed to not running all tests on all commits, since they are complex interactions between different services (hangouts plugin/hangouts js; google search backend/google spellcheck/google search js), but are painfully obvious to an external observer or a trivial end to end test (in a non-dependent project). There are much better tested examples of software projects out there to learn from (which have public test infrastructure too)!
At any rate, copying someone else's <X> only places limits on what you <X>. Build a solution that works well for you, (regardless of how it works for others) and you'll be better off (with a much more stable and reliable buzzmobile) in the long run :smile:

I guarantee that every competent large company is running all tests on all commits at some point in their build process.

I can guarantee that this becomes computationally infeasible past a certain scale, but that's mostly beside the point here, like I said. If you want to have a longer conversation about testing methodology and infrastructure (at google), feel free to PM me.


/root/torch/install/bin/luajit: C++ exception

`~/neural-style# th neural_style.lua -style_image examples/inputs/brad_pitt.jpg -content_image examples/inputs/picasso_selfport1907.jpg -gpu 0 -output_image 123.png -backend clnn -optimizer adam
libthclnn_searchpath    /root/torch/install/lib/lua/5.1/libTHCLNN.so
/root/torch/install/bin/luajit: C++ exception`
i7 3770,16gb RAM, amd r290
with cpu work.
why not work with gpu?
how to fix?

I came across the same problem, have u fixed it already? However, I cannot work out with CPU either


Flask jsonify is not working. It removes keys from json response.

        ```org_schema = OrganisationSearchSerializer()
        result = org_schema.dump({'organisations': organisation_data})
       return jsonify(result)```

Sample Code
it gives output as
    {
        "organisations": []
    },
    {}
]```

Actual output should be

```[
    "data":{
        "organisations": []
    },
    "error": {}
]```

there is some bug in jsonify it removes the key

Please use Stack Overflow for questions about your own code. This tracker is for issues related to the project itself. Be sure to include a minimal, complete, and verifiable example.


Warn cellxgene explorer users that Safari will not be supported

[ ] Close Safari-specific bugs as won't fix.
[ ] cellxgene explorer must detect Safari and display a warning that Safari support is being deprecated by January 8 2021.
[ ] Supported browsers must be updated with a similar warning.

Issue moved to chanzuckerberg/cellxgene #1975 via ZenHub


cuda

Hey, does someone know if it should be possible to use cuda from jupyterlab app, either on windows or linux?
Also in the jupyterlab app, output seems to be very delayed. So it's quite hard to tell if it actually does anything. With pip, at least it fails faster when installing things, but as jupyterlab ships with conda as far as I know, some things would be easier to install from there anyway. So, am I doing anything wrong or is this expected behaviour?
As an example, this is when running !conda install pytorch torchvision cudatoolkit

(also somehow I saw no issue template)


[Run] UAC dialog box doesn't get focus after hitting enter on "Run As Administrator" button

Windows build number: 10.0.19041.329
PowerToys version: v0.19.1
PowerToy module for which you are reporting the bug (if applicable): Run

Steps to reproduce

Set UAC Settings to "Do not Dim"
Launch PowerToys Run
Type name of any program that you would like to start
Hit "tab" button in order to focus on "Run As Administrator" button or use Ctrl + Shift + Enter shortcut

Expected behavior
"User Account Control" dialog box appears with the action button "No" focussed. So you can use your keyboard to press either button on the UAC dialog box.
Actual behavior
"User Account Control" dialog box appears above the PowerToys search box but the UAC dialog itself doesn't get focussed. The PowerToys Run search box stays focussed. So you cannot use your keyboard to press any button on the UAC dialog box.
If you run any program from the Windows Start Menu the UAC dialog gets focussed right after you hit the "Run As Administrator" button.
Screenshots

Hm, this shouldn't be possible under the normal settings. Can you tell us which setting you use for UAC? Because It looks like you have the "do not block desktop" one.

yeah, sure. I use "Never notify" (do not dim my desktop)
The reason I raised this issue is that when I launch elevated Windows Powershell from "Win+X" menu the UAC dialog box is being focused so I can continue using the keyboard. I found this useful and that is why it would be great to have the same experience here.

hi @lrushl can you send a screen recording of the behaviour you want (or what happens with windows search?

I've recorded a new video of the issue that I am experiencing.
It consists of two parts.
The first part shows the expected result I'd like to get when I'm using PT Run for launching any program that requires the "User Account Control" dialog box to be shown.
By the expected behavior, I mean, that the next thing that gets focused is the "User Account Control" dialog box itself so that I can continue using my keyboard to navigate to the right dialog button and hit the enter button to continue.
The actual result is that the "User Account Control" dialog box shows but without focus, so I can't use my keyboard to make the decision and to continue work without workout (alt-tab until finding the "User Account Control" dialog box there) or using the mouse.

I can repro this, when I set my UAC settings "Do not dim desktop". setting this as a bug!

I can repro this, when I set my UAC settings "Do not dim desktop". setting this as a bug!

This looks like bug in Windows because when I run application as administrator from windows taskbar using Ctrl + Shift + Win + number from 0 to 9  or click icon of application using mouse cursor while holding Ctrl + Shift, UAC dialog is also not in focus.

This isn't fixed still and poses a serious vulnerability issue in my opinion, because users (such as me) can start to type their password and get exposed because the previous application is actually focused instead of the UAC dialog, if they happen to be in chat or something like that, they can send their password by mistake. Or even more frequently they could be in previously using a terminal and someone watching the screen will see the password.
I frequently do that when installing things that require elevated permissions using winget.

Can reproduce personally on W.11 today
I wonder if this is a bug in the first place. PT Run opens a new app and the UAC dialog is explicitly set to not block anything. Therefore, I think it opens as any other window and it just doesn't steal focus.

I think this is a Windows bug which specifically happens when you set UAC to not dim your desktop. Because if you don't want UAC to dim your desktop, it often spawns the UAC prompt behind opened windows instead of on top of windows.


How to Hide the CVC field ?

Hi guys !
Is it possible to hide the CVC input field when we use the  component ?
thanks for your help :)

It can be hidden on the CreditCardInput component, not currently supported by the LiteCreditCardInput.
<CreditCardInput           requiresCVC={false}         />
Documentation pretty clear on that.

@Mr-Cool thanks it works !


Reward function + Autoformating Walking Robot

cf #2 and #3
closes #2
Reformat everything using black & isort.
This is the current status of my work (still WIP, I need to convert things to the correct coordinate system).

@OndrejNepozitek I think I'm mostly ready on my side, the only thing I need to fix/update is the coordinate system (cf. my TODOs in the code).
For that I need to know whether we can ask the robot to be spawned at a specific location (will be (0, 0, 0)) with a specific orientation?
I will also add a rate limiter in the code soon (to limit the control frequency).

Thanks for your work.
How important is it to spawn the robot at exactly (0,0,0)? Is it possible to just subtract the initial position/orientation and pretend that it was spawned at (0,0,0)? The physics engine is slightly non-deterministic so I cannot make sure that the position is exactly (0,0,0). Also, the game doesn't like when I try to spawn the robot too close to the surface so there must be a slight drop from a height which also adds to the inaccuracies. So in theory I can get close to (0,0,0) but not precisely. Also, I can try to get close to a given orientation but it requires some manual tinkering so I can try to do that if you tell me what the orientation should be.

How important is it to spawn the robot at exactly (0,0,0)?

Not important, that would just simplify some code but not a problem.
I have some code ready to cope with that (It was used to track the real robot on the treadmill).

Also, I can try to get close to a given orientation but it requires some manual tinkering so I can try to do that if you tell me what the orientation should be.

no worry, we will just need some debug tomorrow and it should work.

Do you want to merge this pull request into the master branch or just keep it in the feature branch for the hackathon?

I would like to merge it once we have a working/tested version.
I created a branch "misc/veridream" in the rl zoo, so all we need to do try it is:
python train.py --algo tqc --env SpaceEngineers-WalkingRobot-IK-v0 --eval-freq -1

I also added a fake server (https://github.com/GoodAI/SpaceEngineersGym/blob/feat/reward-fn/tests/test_envs.py) so I can debug locally too ;)

I think I have a first (probably) working version.
I did some changes in the frame to make it easier to reason about:

the x axis will be aligned with the initial heading (forward vector) of the robot
the y axis will be perpendicular to the initial heading (- right vector)
the z axis will point upward (- gravity vector, up vector)

@OndrejNepozitek may I merge it?

Sure, it was no my TODO list for this morning. By the way, the last overnight training session wasn't able to further improve the average reward of the robot.

Sure, it was on my TODO list for this morning. By the way, the last overnight training session wasn't able to further improve the average reward of the robot.

ok, thanks for trying out =). I may have to do additional debug locally then before retrying.

ok, thanks for trying out =). I may have to do additional debug locally then before retrying.

I mostly suspect two things:

the speed may be still too high
the reward it is optimizing favor dangerous behavior (I will change that)

Otherwise, it could also be:

a bug in the coordinate system
not enough information as input to solve the task


Make BLANK_STR a static property on GridFilterModel.js

This would make it easier to re-use and maintain in apps.
Apps wouldn't have to make their own BLANK_STR constant and make sure it was in sync.
https://github.com/xh/hoist-react/blob/da53eb4e42d7433cc49bd2fec84c421d13cc95d7/cmp/grid/filter/GridFilterModel.js#L39

Let's call this BLANK_PLACEHOLDER.

FYI, I pushed this follow on commit straight to develop: https://github.com/xh/hoist-react/commit/0c6fb6bd17b898edd60a629e7d113c6881613488


Deployment issues using intel openvino(geti_sdk)

While deploying using geti_sdk i am able to install and import it successfuly but during getting server_details_from_environment i am getting error;
ValueError: Unable to load Geti Server details from environment file at path '.env', please make sure the file exists.
In my case I am not able to find .env file as its not generated by geti-sdk. I tried to create a .txt file with name .env but still not able to resolve the issue.
Please help me resolve it.

Hi, please have a look at the instructions for running the Jupyter notebooks. It contains a paragraph on how to set up your .env file here


Queue processing issue

I have a problem with this library. Jobs are added to the database but are not performed. Single event is created with name wp_queue_connections_databaseconnection and after few moments it disappears from Event list.
My code:
$items = [
        'test 1',
        'test 2',
        'test 3',
        'test 4',
    ];

    foreach ($items as $item) {
        wp_queue()->push(new \Some_Class\Backup_Job($item));
    }

    wp_queue()->cron();

Job class:
namespace Some_Class;

use WP_Queue\Job;

class Backup_Job extends Job
{
    public $name;

    /**
     * Backup_Job constructor.
     * @param $name
     */
    public function __construct($name)
    {
        $this->name = $name;
    }

    public function handle()
    {
        sleep(1);
    }

}

Jobs in DB:

Jobs are not fail.
This way you know if it's a problem with WP_Cron or the actual queue.
wp_queue()->worker()->process()

I'm having the exact same issue, even when I manually call wp_queue()->worker()->process()

I should add that it appears my issue was caused by problems in a script called by my Job class. Once I resolved this issue, manually calling the worker appears to have worked. That being said, I feel like this is when it should have been marked as a failed job.


Added strokeCap prop

I found myself working on some graphics which needed exactly the functionality arc-progress had to offer, except the capability to alter the strokeCap on a component level.
With my addition, you'll be able to define the strokeCap behavior on a prop level like so:
<CircularProgress
          size={200}
          width={12}
          prefill={0}
          fill={20}
          missingDegree={100}
          strokeCap="circular"
          style = {{marginTop:30}}
          tintColor={colors.accent}
          backgroundColor={colors.base} />

Good job, buddy


RS5 Emulator is missing

Now that RS5 is generally available, it'd be nice to have a corresponding emulator image.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0051f2fd-9bcd-7eb8-48f0-b8b170d42a49
Version Independent ID: 2c1a0a2b-6a3f-1a1d-caf1-4a06caf5411b
Content: Using the HoloLens emulator - Mixed Reality
Content Source: mixed-reality-docs/using-the-hololens-emulator.md
Product: mixed-reality
GitHub Login: @JonMLyons
Microsoft Alias: jlyons

These links were updated in PR #722 last week.
Thanks,
Jesse


Visual Studio Code

Currently there is no known vscode build for RV64GC.
https://github.com/Microsoft/vscode/wiki/How-to-Contribute
As described here, it lists the dependencies required to build vscode.
Node, npm and yarn appear to be the dependencies that are missing from debian.
There is work on node.js here https://github.com/v8-riscv/node but last time I tried to build all of this on my unmatched, I was unsuccessful.
There is an issue logged here https://github.com/v8-riscv/node/issues/5

VSCode has been successfully built and run on RISC-V (RV64GC) a few months ago.

Are the changes uptreamed already?

Are the changes uptreamed already?

IIRC not yet.

Gogo


Webcam not running on Mac due to multi threading

I am on Mac OS and am trying to make the webcam work. When I ran the command to initiate the webcam, the application started however it did not show the webcam playback. I took the code from under the play_video() function and integrated it into the main thread and it worked, however I was only getting a few a frame every few seconds.

Huh, that's interesting. Seems like this is partially a known issue with opencv on OSX:
https://github.com/opencv/opencv/issues/5150
Perhaps one of the fixes in that thread could help?
Also does saving video work? I think you should be able to save video by doing something like --video=0:video.mp4 from the webcam and pressing ctrl+c to pause and finalize the video.

Thanks. The set_start_spawn fix in that thread unfortunately did not work but now I have a better lead on the problem.
Also, video saving works somewhat. It seems really slow at parsing the frames but that could be because I edited the code to run on my cpu for now. Also, this error pops up:
Traceback (most recent call last):   File "eval.py", line 751, in play_video     % (repr(progress_bar), frames_displayed, num_frames, progress, fps), end='') OverflowError: cannot convert float infinity to integer Processing FPS: 0.76 | Video Playback FPS: 0.00 | Frames in Buffer: 11

Ah right, you'd get that error because the webcam has "infinite" frames (technically) so it can't print out the number of frames processed. You can just comment that print out if you want.


belindas-closet-nestjs_sprint-13_140_updating-dependencies

Resolves issue #140
For this pull request I researched each update independently to make sure they were not major changes. I updated the package.json file and added the nodejs 20 requirement for the engine. I also ran the server locally and created a test user and logged in successful after the update.

checkout the branch
run npm install
run npm run start:dev

Please checkout this notion link with pictures and some descriptions: https://www.notion.so/Belinda-s-Closets-Backend-127d514f21a080f1b7bdd8073f72a4b9?pvs=4

Good Job on this PR! After reviewing, I can see that the objective was to update various dependencies in package.json while ensuring compatibility with our current codebase. I appreciate the effort to independently research each update and confirm that none introduced breaking changes. Running the server locally, creating a test user, and logging in successfully post-update all demonstrate a thorough approach to testing. The addition of Node.js 20 as an engine requirement also aligns well with keeping the backend optimized for performance and security.
As a final check, I ran all the test suites, and they passed successfully, with 47 tests in total. The only minor issue encountered was a deprecation warning for the punycode module, which could be addressed by replacing it with a userland alternative in future sprints. For anyone interested in best practices for updating dependencies in Node.js projects, here’s a helpful resource: Managing Dependencies in Node.js.


Uncaught ReferenceError: _classCallCheck is not defined

If I click reset or revert after route change I get error Uncaught ReferenceError: _classCallCheck is not defined

npm run dev
click widgets from navi
click reset or revert
error happens

Seems that I get error while clicking reset/revert when the current route is not the same I used initially to load the application.
Tested with commit: f0f882756182b8b1042053168f754104750b7a78

This is due to redux-router putting react components in state: https://github.com/rackt/redux-router/issues/57

I'm getting this error when I put some components in the state. Is it possible to put components into the state and use them or is this a bad practice?


During control action host was not being passed in

https://bugzilla.redhat.com/show_bug.cgi?id=1434935
dialog_hosts should have been hosts

@gmcculloug @bzwei
Please review


Experiment with persistent data structures

We can actually get rid of dependency list if we just converted to persistent map for value equality semantics.
Might be negligible perf cost compared to iterating over dependency list and style map, but give us a more ergonomic API.
Size of bundle will increase dramatically if we use immutablejs though, maybe try https://github.com/bloomberg/record-tuple-polyfill or even building our own?

See #12


Report query: Overdue and lost

Report of all requests where the due date has passed and the state is not "Complete."
Report mockup: https://docs.google.com/spreadsheets/d/1KgfneTJZAQGow_87_Mra-xGqsAi9AmtWVnS0ph4VO4Y/edit#gid=0

At the June 9 meeting, we clarified that having requests with an "overdue" status is not required to begin working on this report. The query will select all requests where the due date has passed and the state is not "complete." At the moment, there are still no overdue requests in the test system, but I will continue to work on getting some in. I think there is bug interfering with this.

I would like to work on this issue

https://github.com/openlibraryenvironment/reshare-analytics/commit/c0efa8be727bca63b436cc3765f369a3c8d418a9


Code release

Hi,
You work is very interesting for me, and if I may, when you will release the corresponding code?
Thx
Chi

Hi, I am very interested in this work, and may I ask when you will release the corresponding code?


OCLint compilation commands file path

Trying to use OCLint with fastlane I get the following error:
{:DEFAULT_PLATFORM=>:ios, :ENVIRONMENT=>nil, :PLATFORM_NAME=>:ios, :LANE_NAME=>"ios test"}
[14:49:27]: Exit status of command 'cd /Volumes/Data/Projects/MyProject && oclint -report-type=pmd -o=oclint_report.pmd -rc=LONG_LINE=200 -max-priority-1=10 -max-priority-2=100 -max-priority-3=1000 ""' was 256 instead of 0.
LLVM ERROR: Could not auto-detect compilation database for file ""
No compilation database found in /Volumes/Data/Projects/MyProject or any parent directory
json-compilation-database: Error while opening JSON database: No such file or directory

My fastlane file has a lane that looks like this:
before_all do
    cocoapods
    xctest(
      scheme: "MyProjectScheme",
      destination: "name=iPhone 5s,OS=8.1",
      destination_timeout: 120, # increase device/simulator timeout, usually used on slow CI boxes
      reports: [{
        report: 'junit',
        output: './build-dir/test-report.xml'
        },
        {
          report: 'json-compilation-database',
          output: './build-dir/commands.json'
        }
      ]
    )
    gcovr(
      xml: true,
      output: "./code-coverage/cobertura-report.xml"
    )
end

lane :test do
    oclint({
      compile_commands: "./build-dir/commands.json",
      select_reqex: /MyProject/,
      report_type: 'pmd',
      max_priority_1: 10,
      max_priority_2: 100,
      max_priority_3: 1000,
      rc: 'LONG_LINE=200'
    })
end

I don't use this action myself, but I think this is because the non-existing directory. Try creating it manually and let me know if that works.

It's there. Actually, it doesn't open the path after the '.'

@cyupa How did you fix the issue?

I somehow ended up not having an json-compilation file on the build machine it was linked to cleaning of the actual workspace in Jenkins rather than a fastlane issue.


How to handle Done button tap or close authenticator tap of google or facebook authenticator?

Hello,
I need to handle Done or Close button tap in microsoft azure authentication webview.
Actually If I want to complete authentication process by done clicking or want to close the process of login through facebook or google by close tap then there isn't any event to handle this.
MobileServiceClient _client;
await _client.LoginAsync(UIApplication.SharedApplication.KeyWindow.RootViewController, provider, "uri scheme");
This actually returns MobileServiceClient as result. But If I handle webview by Done or close tapping then there is no event I am getting there. Anyone knows how to handle this ?

Hi @fazil1979,
Sorry for the delay in replying. It's possible to handle done/close tap events using try/catch block. Please take a look at this example: https://github.com/Azure/azure-mobile-apps-net-client/blob/899294bd9bfee1b67dcfc6d703845449733830c7/e2etest/iOS.E2ETest/UI/LoginViewController.cs#L97-L105
When done/close button is tapped then the exception will be thrown saying that login was canceled.
Please let me know if you have any questions.

@fazil1979, I'm closing this issue as we haven't heard from you for a while. Please feel free to reopen it if you have any questions.


Set a default request timeout

I'm not sure if this should be made configurable for now. 10s seems to be a reasonable value, and for decent usability 10s already feels like an infinite wait time.
For some of these, we might also want to print an error if the request timed out. Ultimately this is what motivated this change.

Does it really make sense to have a single timeout as default? I could imagine that this highly depends on the expected latency.
If we chose a single value, I think calling it default_timeout would help the reader in understanding that a timeout has not yet been set specifically for this request.

Does it really make sense to have a single timeout as default? I could imagine that this highly depends on the expected latency.

This mostly is about having a timeout at all. The actual value may end up being a minute even. The desired UX is "Hey, something's likely wrong on our end, requesting a status should not take a full minute, we're stopping here."
I've been toying around with the implications of the initial change I made on the side today and have done some stress testing under load, and it looks like having such a timeout is really only viable when connecting to the node.
What I'd really like to have is some sort of timeout until the result arrives, and after that no timeout at all, but I'm not sure if CAF allows for that easily.


no polishing windows found

Hi,
I'm having trouble getting nanopolish 0.10.2's vcf2fasta.  It seems that the nanopolish variants step works, producing a .vcf with a reasonable number of variants, but vcf2fasta produces an empty file and throws "error: no polishing windows found for<PHONE_NUMBER>_-PV2-_Final_Consensus."
It would be reasonable for there not to be any polishing windows because I'm working with a small viral genome, only ~7400 bp, and I didn't set a -w option.  But I've used nanopolish with this very same set of reads before, with very similar commands, and was able to get a phased .sam out of it.
There are two main differences between this attempt and my previous successes:
-I indexed the reads once and then ran nanopolish variants with and without the --fix-homopolymers option simultaneously
-Since nearly all of my reads cover the entire genome, I didn't feel like I'd need to use racon to make a draft genome.  For my successful attempts, I made a draft consensus from the reads assembled to a somewhat-similar virus from GenBank.  For this attempt, I used a consensus sequence for this same viral mixture, created from MiSeq reads.
I've also tried the --skip-checks option for vcf2fasta, with the same results.  I can't see a reason why these changes should make nanopolish expect a polishing window that I haven't defined.  Any thoughts?
For reference, here's the script I ran:
#!/bin/bash -l

#$ -pe smp 25
#$ -o BC04pipeline.out
#$ -b n
#$ -e BC04pipeline.err
#$ -N BC04pipeline
#$ -cwd
#$ -q all.q
echo Starting porechop at date
module load porechop/0.2.3
porechop -i ./fqPass -b porechop --discard_middle
echo Porechop complete at date.  Filtering reads by length.
module unload porechop/0.2.3
module load prinseq
prinseq-lite -fastq ./porechop/BC04.fastq -out_format 3 -out_good BC04_long -out_bad null -min_len 7000
echo Reads filtered at date.  Start mapping to reference.
module unload prinseq
module load minimap2/2.7
minimap2 -a -x map-ont --no-long-join -t 24<PHONE_NUMBER>_MiSeq.fasta BC04_long.fastq > BC04toMiSeq.sam & PIDMINIMAP=$!
echo Starting nanopolish index at date.
module load nanopolish/0.10.2
nanopolish index -d ./fast5Pass BC04_long.fastq & PIDNANOINDEX=$!
wait $PIDMINIMAP
module unload minimap2/2.7
echo Indexing started and first mapping complete at date.  Converting SAM to BAM.
module load samtools
module load tabix
module load bcftools
samtools view -S -b BC04toMiSeq.sam > BC04toMiSeq.bam
echo BAM produced at date.  Sorting.
samtools sort -o BC04toMiSeq.sort.bam BC04toMiSeq.bam
echo BAM sorted at date.  Indexing.
samtools index BC04toMiSeq.sort.bam
echo BAM indexed at date.  Producing draft consensus.
samtools mpileup -uf<PHONE_NUMBER>_MiSeq.fasta BC04toMiSeq.sort.bam | bcftools call -mv -Oz -o BC04vars.vcf.gz
tabix BC04vars.vcf.gz
cat<PHONE_NUMBER>_MiSeq.fasta | bcftools consensus BC04vars.vcf.gz > BC04draft.fasta
echo Draft complete at date.  Re-aligning reads to draft.
module unload samtools
module unload tabix
module unload bcftools
module load minimap2/2.7
minimap2 -a -x map-ont --no-long-join -t 24 BC04draft.fasta BC04_long.fastq > BC04toDraft.sam
echo Aligned to draft at date.  Converting to BAM.
module unload minimap2/2.7
module load samtools
samtools view -S -b BC04toDraft.sam > BC04toDraft.bam
echo BAM to draft produced at date.  Sorting.
samtools sort -o BC04toDraft.sort.bam BC04toDraft.bam
echo BAM to draft sorted at date.  Indexing.
samtools index BC04toDraft.sort.bam
echo BAM to draft indexed at date.  Waiting for nanopolish index.
module unload samtools
wait $PIDNANOINDEX
echo Nanopolish index finished by date.  Time to polish!
nanopolish variants --consensus --fix-homopolymers --ploidy 1 -t 13 -o BC04polished_FHP.vcf -r BC04_long.fastq -b BC04toDraft.sort.bam -g BC04draft.fasta & PIDFHP=$!
nanopolish variants --consensus --ploidy 1 -t 12 -o BC04polished_NoFHP.vcf -r BC04_long.fastq -b BC04toDraft.sort.bam -g BC04draft.fasta & PIDNOFHP=$!
wait $PIDNOFHP
echo NoFHP polishing complete at date.  Generating polished consensus.
nanopolish vcf2fasta -g BC04draft.fasta BC04polished_NoFHP.vcf > BC04polished_NoFHP.fasta
echo NoFHP polished consensus generated at date.  Phasing reads.
nanopolish phase-reads --reads BC04_long.fastq --bam BC04toDraft.sort.bam --genome BC04polished_NoFHP.fasta BC04polished_NoFHP.vcf > BC04phased_NoFHP.sam
echo NoFHP pipeline complete at date.  Waiting for FHP polishing.
wait $PIDFHP
echo FHP polishing complete at date.  Generating polished consensus.
nanopolish vcf2fasta -g BC04draft.fasta BC04polished_FHP.vcf > BC04polished_FHP.fasta
echo FHP polished consensus generated at date.  Phasing reads.
nanopolish phase-reads --reads BC04_long.fastq --bam BC04toDraft.sort.bam --genome BC04polished_FHP.fasta BC04polished_FHP.vcf > BC04phased_FHP.sam
echo FHP pipeline complete at date.
module unload nanopolish/0.10.2

Hi,
vcf2fasta requires nanopolish to run with the -w parameter. Since the genome is small in this case, you can just use the entire reference genome as the -w parameter (-w chromosome:1-7400).
Jared

I'm able to get it to work now, but perhaps not as intended?
I added -w :1-8000 to the nanopolish command, since I wasn't sure of the exact draft length and wanted to see what would happen if I overshot it.  Polishing yielded a reasonable-looking .vcf with "##nanopolish_window=:1-7429" in the header, but attempting to use vcf2fasta threw:
"error: first 1 bases are not covered by a polished window for contig .
error: one or more polishing windows are missing. Please check that all nanopolish variants --consensus jobs ran to completion"
I checked the actual length of the draft fasta (7430 bp) and manually changed the line in the .vcf to "##nanopolish_window=:0-7430."  Vcf2fasta worked with this file.
Do you think a different -w range would avoid this issue?  I'll try 0-8000 overnight tonight and see if that works.
Also, for reference, I looked back at a .vcf produced by polishing without -w, and it did contain a nanopolish_window header, ranging 0-[length-1].  This one worked with vcf2fasta.  Maybe differences in fasta formatting can mess with automatic window setting?

Ugh, sorry, I tried to designate a generic sequence name in between > and < signs, and the site must have thought it was a markup tag and ignored it.  So just pretend there's a generic sequence name wherever it makes sense ^^;
I'd go back and edit it, but the editing function doesn't work on IE, which is the only browser on this government box, until my software request goes through.

Setting the lower range of the window to 0 worked, so I'll close this.  Thanks for the support, and for an excellent set of tools!


Don't limit bank search and title explicitly to 15 characters

Before

After

Definitely comment up the rs2asm extensively since they are generally hard to follow and also we need to be able to identify what it is we changed since this would get missed.


Create a separate documentation for v0.9 and v0.10

There are some differences between v0.9 and v0.10 and the docs also need to be different for that purpose.

This is to update the docs content for tutorial, base clusters/ cluster specs etc so that each branch has docs to reflect what is developed/ available with the corresponding release.

@Rohitrajak1807 can you close this out, when the arlon init doc changes go in ?


Rounding option for klog start and klog stop

As discussed in https://github.com/jotaen/klog/discussions/156, when doing klog start or klog stop, there could be an option for it to round the time values to the nearest 5, 10, 15, 30 or 60 minute value.
Both should be possible:

On the fly: klog start --round=15m
Permanent via an env variable: KLOG_ROUND_START_STOP=15m

Resolved via https://github.com/jotaen/klog/pull/159.
For the permanent setting, see https://github.com/jotaen/klog/discussions/158.


[Serial]Storage Starting a VirtualMachineInstance [rfe_id:2288][crit:high][vendor:cnv-qe@redhat.com][level:component] With Cirros BlockMode PVC [test_id:1015] should be successfully started

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind bug

/kind enhancement

What happened:
tests/storage_test.go:796
Cirros login successfully
Unexpected error:
    <*kubecli.AsyncSubresourceError | 0xc0036a0c60>: {
        err: "Can't connect to websocket (500): websocket: bad handshake\n",
        StatusCode: 500,
    }
    Can't connect to websocket (500): websocket: bad handshake

occurred
tests/storage_test.go:807

Source: https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/4277/pull-kubevirt-e2e-k8s-1.17/1312368188956086273
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

KubeVirt version (use virtctl version):
Kubernetes version (use kubectl version):
VM or VMI specifications:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

/kind bug
/triage build-watcher

Happened again here:
https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/4152/pull-kubevirt-e2e-k8s-1.16/1313160032598102016

Happened again:
https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/4415/pull-kubevirt-e2e-k8s-1.18/1321250844049412096

/close
"Fixed" in #4522. It takes sometimes unusually long until the kubelet updates the certificate secrets.


Downgrade jsdom to 21 fix failed in node 14

Description
Downgrade jsdom to 21 to fix failed in node 14.
Here is the error detailed: https://github.com/opensearch-project/opensearch-build/issues/4115#issuecomment-1942990532
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test.

[ ] New functionality has user manual doc added.
[ ] Commits are signed per the DCO using --signoff.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Issue resolved https://github.com/opensearch-project/opensearch-build/issues/4115#issuecomment-1942990532

Backports:

#148
#149


il deleting manually issue

OS: win10
Python Version: latest_
i expected it to build an exe to to a specific folder.
(ı left the registry name empty because i dont know what it is if the registry name is doing this then what should i put it? please respond quickly if you can)
i cant delete it manually because it says the file is open in another program

add me on discord ill show you how to set it up

add me on discord ill show you how to set it up

can u also add me idk how

ye just add camsoooo

add me on discord ill show you how to set it up

i fixed the issue i just needed to wait a little.

my discord got banned.
Now, i need a little help in removing PySilon from my personal computer.
i don't know how to remove it.


Changed PasswordChangeSerializer to not include old_password when use…

…r has no usable password

Coverage remained the same at 96.931% when pulling b002ee3d4b4f516f6c451300bed837d7faf92e39 on sbenemerito:master into 9d1f65eedcd437003f5bef13fdc1f27d0fb0ee1a on Tivix:master.

Hi,
as a user of rest-auth, thanks for the contribution!
This repo is not maintained anymore, so the development moved to dj-rest-auth. (reference: https://github.com/Tivix/django-rest-auth/issues/568)
It may be best, if you move this PR there. (and upgrade to using dj_rest_auth)
new repo link: https://github.com/jazzband/dj-rest-auth (I'm not the upkeeper of that repo, it just makes sense for me to help you merge your PR)
Many Thanks,
Barney


Link to Arxiv HTML, not to PDF

The readme links to the Arxiv article PDF. Please consider updating the hyperlink to point to the respective HTML page instead. Some readers only want to read the abstract in the web page. For reference, the link I have is https://arxiv.org/abs/1809.00794

Updated. Thanks!


feat: added ping encoding and framing

Motivation
Implements RPLx ping message encoding (#167)
Description
Also refactors framing code to separate it from message building (Partially solves #126)
Closes #167

Marking it ready for review since it already solves #167.
There is still work to be done here but it can be addressed in the rest of the sub-tickets in #127
(ie. ping/pong decoding, frame decoding, listen loop, etc)


Showcases on phenomic.io not rendered properly in firefox

Ubuntu 16.04 / Firefox 54.0
The showcases do not display as they should (when compared to chrome) and take really long time to load, when going to next page (or next page not appearing at all)

Thanks for reporting. I am fixing this ATM! :)

Ah, thx! was already figuring this should be something temporary.. (found also next not working on chrome 60.0.3112.90)

yeah.. works like a charm again :tada:
thanx @MoOx !


Glossar function

Hi,
thank your for this cool app. A nice addition would be the ability to choose a glossar from a dropdown list for the selected source and target language. I'm guessing pretty much everyone who uses the Pro API has a need for glossars, be it short terms or product names.
The Deepl API should already provide all the necessary resources, but since the only way to add new entries to a glossar is to create a new one with all the old and new entries (and optionally to delete the old one) just configuring the glossar would not be enough.
Of course, the ability to manage glossaries (create, copy, etc) would be the icing on the cake. 💯

I also suggested the glossary function for DeepL. I would really be grateful if it could be implemented.
Cheers
Rob


Allow configuring applicaiton's description through the Application Configuration file

What would you like to be added:
Most of the application fields have been migrated to the Application Configuration file.
The Description field should be the same.
After that, we can remove the "Edit" button on the application details page and show the description box if only if it is containing a not empty string to avoid the ugly "No description" box.

Why is this needed:

/assign @nakabonne


Unable to generate VS 2019 solution with CMake

** Visual Studio 2019 Developer PowerShell v16.2.5
** Copyright (c) 2019 Microsoft Corporation

PS E:\CSO2LauncherS> cmake -G "Visual Studio 16 2019" ./
-- The CXX compiler identification is MSVC 19.22.27905.0
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.22.27905/bin/Hostx64/x64/cl.exe
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.22.27905/bin/Hostx64/x64/cl.exe -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- User did not specify build type, defaulting to Release build
CMake Error at CMakeLists.txt:20 (message):
The launcher must be built for 32 bit platforms.
-- Configuring incomplete, errors occurred!
See also "E:/CSO2LauncherS/CMakeFiles/CMakeOutput.log".
PS E:\CSO2LauncherS>
Originally posted by @felixabi in https://github.com/Ochii/cso2-launcher/issues/13#issuecomment-533845360

@felixabi
Try `cmake -G  "Visual Studio 16 2019 -A Win32 ./".
You should know that I haven't tested the launcher in Visual Studio 2019

okey

cmake -G "Visual Studio 16 2019 -A Win32 ./" - this code did not work
cmake -G "Visual Studio 16 2019" -A Win32 ./ - i tried that
-- The CXX compiler identification is MSVC 19.22.27905.0
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.22.27905/bin/Hostx64/x86/cl.exe
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.22.27905/bin/Hostx64/x86/cl.exe -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- User did not specify build type, defaulting to Release build
-- The C compiler identification is MSVC 19.22.27905.0
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.22.27905/bin/Hostx64/x86/cl.exe
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.22.27905/bin/Hostx64/x86/cl.exe -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
CMake Error at libs/PolyHook_2_0/CMakeLists.txt:215 (add_subdirectory):
The source directory
E:/CSO2LauncherS1/libs/PolyHook_2_0/capstone

does not contain a CMakeLists.txt file.
-- cotire 1.8.0 loaded.
CMake Error at launcher/CMakeLists.txt:241 (target_compile_options):
Cannot specify compile options for target "capstone-static" which is not
built by this project.
CMake Error at launcher/CMakeLists.txt:242 (target_link_options):
Cannot specify link options for target "capstone-static" which is not built
by this project.
-- CXX RC target launcher cotired without unity build.
-- Configuring incomplete, errors occurred!
See also "E:/CSO2LauncherS1/CMakeFiles/CMakeOutput.log".
CMakeOutput.log >
Microsoft for x86 (R) C/C++ Enhanced Compiler Version 19.22.27905
testCCompiler.c
Copyright (C) Microsoft Corporation. All Rights Reserved.
cl /c /Zi /W3 /WX- /diagnostics:column /Od /Ob0 /Oy- /D "WIN32" /D "_WINDOWS" /D "CMAKE_INTDIR="Debug"" /D _MBCS /Gm- /RTC1 /MDd /GS /fp:precise /Zc:wchar_t /Zc:forScope /Zc:inline /Fo"cmTC_9b3a8.dir\Debug\" /Fd"cmTC_9b3a8.dir\Debug\vc142.pdb" /Gd /TC /analyze- /errorReport:queue E:\CSO2LauncherS1\CMakeFiles\CMakeTmp\testCCompiler.c
cmTC_9b3a8.vcxproj -> E:\CSO2LauncherS1\CMakeFiles\CMakeTmp\Debug\cmTC_9b3a8.exe
Detecting C compiler ABI info compiled with the following output:
Change Dir: E:/CSO2LauncherS1/CMakeFiles/CMakeTmp
Run Build Command(s):C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/MSBuild/Current/Bin/MSBuild.exe cmTC_da80b.vcxproj /p:Configuration=Debug /p:Platform=Win32 /p:VisualStudioVersion=16.0 /v:m && .NET Framework için Microsoft (R) Build Engine Version 16.2.37902+b5aaefc9f
Copyright (C) Microsoft Corporation. All Rights Reserved.
Microsoft for x86 (R) C/C ++ Enhanced Compiler Version 19.22.27905
Copyright (C) Microsoft Corporation. All Rights Reserved.
CMakeCCompilerABI.c
cl /c /Zi /W3 /WX- /diagnostics:column /Od /Ob0 /Oy- /D "WIN32" /D "_WINDOWS" /D "CMAKE_INTDIR="Debug"" /D _MBCS /Gm- /RTC1 /MDd /GS /fp:precise /Zc:wchar_t /Zc:forScope /Zc:inline /Fo"cmTC_da80b.dir\Debug\" /Fd"cmTC_da80b.dir\Debug\vc142.pdb" /Gd /TC /analyze- /errorReport:queue "C:\Program Files\CMake\share\cmake-3.15\Modules\CMakeCCompilerABI.c"
cmTC_da80b.vcxproj -> E:\CSO2LauncherS1\CMakeFiles\CMakeTmp\Debug\cmTC_da80b.exe

I installed Visual Studio 2017 and I can't find Powershell.

and
Severity Code Description Project File Line Hide Status
Error CMake: E: \ CSO2LauncherS1 \ CMakeLists.txt: 20 (message):
The initiator must be created for 32-bit platforms. launcher E: \ CSO2LauncherS1 \ CMakeLists.txt 20

Try running git submodule update --recursive in the project directory, then try generating the project files again

PS E:\CSO2LauncherS2> git submodule update --recursive
PS E:\CSO2LauncherS2> cmake -G "Visual Studio 16 2019" -A Win32 ./
-- The CXX compiler identification is MSVC 19.22.27905.0
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.22.27905/bin/Hostx64/x86/cl.exe
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.22.27905/bin/Hostx64/x86/cl.exe -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- User did not specify build type, defaulting to Release build
-- The C compiler identification is MSVC 19.22.27905.0
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.22.27905/bin/Hostx64/x86/cl.exe
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.22.27905/bin/Hostx64/x86/cl.exe -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
CMake Error at libs/PolyHook_2_0/CMakeLists.txt:215 (add_subdirectory):
The source directory
E:/CSO2LauncherS2/libs/PolyHook_2_0/capstone

does not contain a CMakeLists.txt file.
-- cotire 1.8.0 loaded.
CMake Error at launcher/CMakeLists.txt:241 (target_compile_options):
Cannot specify compile options for target "capstone-static" which is not
built by this project.
CMake Error at launcher/CMakeLists.txt:242 (target_link_options):
Cannot specify link options for target "capstone-static" which is not built
by this project.
-- CXX RC target launcher cotired without unity build.
-- Configuring incomplete, errors occurred!
See also "E:/CSO2LauncherS2/CMakeFiles/CMakeOutput.log".

Can you send a list of files inside E:/CSO2LauncherS2/libs/PolyHook_2_0/capstone?

thanks sln succes fix capstone

Dear Ochii

????????????????
Local Windows Debugger Click Error


Refactored partition_type in parser.

This small PR changed partition_type in parser to hash or range from 0 or 1.

Looks good to me.


feat: Convert between Excel file and Table

Closes #138.
Closes #139 .
Summary of Changes
Created the methods to_excel_file() and from_excel_file() in Class Table.
Co-authored-by: patrikguempel<EMAIL_ADDRESS>

🦙 MegaLinter status: ❌ ERROR

Descriptor
Linter
Files
Fixed
Errors
Elapsed time

✅ PYTHON
black
3
3
0
1.31s

✅ PYTHON
mypy
3

0
2.05s

❌ PYTHON
ruff
3
3
4
0.05s

✅ REPOSITORY
git_diff
yes

no
0.03s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff
MegaLinter is graciously provided by

Now that #146 is merged, this PR has to be updated as well, so users can pass a path.

Please finish this next week before starting something new.

:tada: This PR is included in version 0.12.0 :tada:
The release is available on:

v0.12.0
GitHub release

Your semantic-release bot :package::rocket:


how to get process id of python interpreter that is executed in java program? How to debug Python code that is run from Java in Pycharm or other tools?

I want to debug Python code that is called from Java. Does Python interpreter is run by another process/sub process? If yes how can I get its pid? Not necessarily by code. Any utility is OK. (e.g htop, ...). But it seems that there is no new process for Python interpreter. How can I debug?
public static void main(String[] args)
{
    PyLib.startPython();
    PyModule module = PyModule.importModule("MyModel");
    PyObject obj = module.call("MyClass");
    MyInterface javaObj = obj.createProxy(MyInterface.class);

    javaObj.MyMethod();
}

Or main question, How can I debug it in Pycharm or other tools?
How can I set break points in MyMethod function in Pycharm  or other tools and pause code on them?

I found the solution. It is quite easy with pydev.
http://www.pydev.org/manual_adv_remote_debugger.html
https://www.jetbrains.com/help/pycharm/2016.3/remote-debugging.html
http://stackoverflow.com/questions/6989965/how-do-i-start-up-remote-debugging-with-pycharm

Thanks, I will update the docs with that info.


Facing model not initialized issue

Versions

sequelize: 5.10.0
sequelize-typescript: 1.0.0-beta.3
typescript: 3.6.2

I'm submitting a ...

[x] bug report
[ ] feature request
Actual behavior:
I am getting an error Model not initialized: <Model> cannot be instantiated. "<Model>" needs to be added to a Sequelize instance. when attempting to use insert data via new Model constructor within the sequelize.authenticate().then(() =>{}) promise sequence.  See IpnProcessor.ts snippet below for reference.
Expected behavior:

Steps to reproduce:
I have initialized the sequelize.ts with method addModels([<Model>]) before using the instance.  Inside the promise using Model's contstructor i am getting the exception error.
Related code:

// sequelize.ts
const options: SequelizeOptions = {
  operatorsAliases: Op,
  models: [IpnTransaction],
};

const sequelize = new Sequelize(DATABASE_URI, options);

sequelize.addModels([IpnTransaction]);

export default sequelize;

// app.ts
private initializeDatabaseConnection() {

  AppLogger.debug('Synchronizing models with the database.');
  const options: SyncOptions = { force: DATABASE_REBUILD };

  const watchOverSync = async () =>
    await sequelize
      .sync(options)
      .then(() => {
        AppLogger.debug('Finished synchronizing the models.');
      });

   watchOverSync();
}

// IpnProcessor.ts
sequelize
      .authenticate()
      .then(() => {
        if (!!IS_PROD) { AppLogger.debug('established connection successfully'); }
      })
      .then(() => {
        const newTransaction = new IpnTransaction(); // <-- Error happens here.

        return newTransaction.save();
      }).then((newTransaction) => {
        AppLogger.debug('successfully recorded transaction: ' + newTransaction.id);
      })
      .catch(err => {
        AppLogger.error(err);
      });

// IpnTransaction.ts
@Table
export class IpnTransaction extends Model<IpnTransaction> {

  constructor(values: any) {
    super(values);
  }

  @PrimaryKey
  @AutoIncrement
  @Column
  ipnTransactionId: number;

  @Column(DataType.STRING(100))
  protectionEligibility: string;

  @Column(DataType.STRING(100))
  payerId: string;

  @Column
  paymentDate: Date;
}

Hey @rroman81, thanks for reporting. Unfortunately I cannot reproduce it. Can you provide an example repo, please?

Closing this due to inactivity.

Closing this due to inactivity.


⚠ RPC server - Figment Solana has degraded performance

In 6787b3d, RPC server - Figment Solana ($FIGMENT_SOLANA_RPC_URL) experienced degraded performance:

HTTP code: 429
Response time: 121 ms

Resolved: RPC server - Figment Solana performance has improved in eeef09f.


Double-encoding happening on uploads

Filename is encoded twice: putFile and then in .prototype.request: this.url(filename)

Closed with 0.9.7


Italic theme variant

Many color schemes (like Night Owl or Monokai Pro) use italics to style specific keywords. Not only does this look great for many fonts (especially Operator and Dank Mono) it also makes it easier to distinguish those keywords from similarly colored companions.
PS: Italics are actually the only reason I still use Monokai Pro over Nord.

Thanks for your contribution :+1:
Nord makes use of italics for some elements all over different port projects (for consistency) and when it makes sense, e.g. for static variables / properties / class attributes to better differentiate from other „normal“ ones. Also static methods / functions as well as some builtin ones are used with with italic font.
I know some of these new themes and font you refer to and I've to say (personally) I absolutely dislike the style and this suddenly appeared trend. Almost every second word is italic and uses this illegible old antique curlicue fonts. Before this everyone tried to create better monospaced fonts (like the awesome Source Code Pro and Hack font families) and now it's like the exact opposite: Make the code look like a book from the ancient Romans.
Again, this is my personal opinion, but in some aspects this is reflected in Nord too :wink:
I know that VS Code provides a way to ship „sub-themes“ with one bundle so if this a features many users like to see we can implement a „italic version“ so users can select which they'd like to use (while the actual one stays the default). I'm not convinced yet since I don't know what would make sense to use italic fonts for other than the current ones (like all keywords and language specific elements, only user-defined ones, …).
Maybe some more ideas how to implement would be nice so a prototype can be created.

Bumping this in favor of using italics in comments for fancy fonts.

@WTFox, you can set comments to italic in your local VSCode settings with:
"editor.tokenColorCustomizations": {     "textMateRules": [       {         "scope": "comment",         "settings": {           "fontStyle": "italic"         }       }     ]   },

Thanks @ianrohde and @bashfulrobot, I guess this helps users that are currently looking for a “italic spiced” variant. Unfortunately, I'm still so convinced to bundle an italic variant for the extension. Personally I don't see any point in using italic styles for almost all language tokens making it harder to read and distinguish from actual “special“ elements like a local variable vs. a global/static variable.
I think a possible solution would be to add a section to the FAQ and troubleshooting guide (currently work in progress) in the docs that provides tips how to customize the theme for more italic styles, e.g. for users who switched from themes like Night Owl that makes heavy use of this style. It could link to some GitHub Gists of user-created styles as well as to the already existing guide about theme syntax customization that can then be referred to in order to guide users to the process of adding these styles.

Oh, one reason just came to my mind that could be a reason for an additional bundled italic variant:
:eye_speech_bubble:  Accessibility :wheelchair:
User that are affected by color blindness or limited vision skills especially for colors would benefit from it. Using a italic font style for syntax elements like language keywords and primitives (string, int, …) as well as language builtin types (struct, class, …) coming from the SDK/runtime would help to differentiate from user-created code like custom structs/classes or functions/methods.
I need to think about it again, maybe some input from actually affected users would be great so the variant could be developed together for the best possible optimization.

Imo, italics can be very beneficial for visually scanning blocks of code if they are not overused, even for people without any visual impairments. E.g., I personally like them a lot for return statements and comments.
I also do agree that they can look weird in direct comparison to the regular font. I've used a patched version of Operator Mono with programming ligatures in the past, but have switched to Dank Mono a while ago because I find the italics in that font blend in far nicer with the rest of the text.
On the topic of having a sub-theme, I'm not sure if that is really necessary and it could also lead to having a lot of sub-themes/variants quickly if you, e.g., decide to ship variants with different colors (e.g., a darker version) later.
Introducing yet another visual setting besides italics (I can't think of something specific right now, so let's call it x) would then cause something like this:

Regular
Regular (with italics)
Regular (with x)
Regular (with italics and x)
Dark
Dark (with italics)
Dark (with x)
Dark (with italics and x)

I think something like a useItalics setting might be more appropriate for this. And if that's not enough, people can always fine-tune it manually to their liking via editor.tokenColorCustomizations.textMateRules.

I use this settings:
"editor.tokenColorCustomizations": {         "textMateRules": [             {                 "scope": [                     "comment",                     "comment keyword",                     "comment markup.underline.link",                     "comment string",                     "comment punctuation.definition",                     "comment punctuation",                     "comment text",                     "entity.other.inherited-class",                     "entity.other.attribute-name",                     "entity.other.attribute-name.pseudo-class.css",                     "entity.other.pseudo-class.css",                     "entity.other.pseudo-element.css",                     "invalid",                     "markup.italic",                     "markup.quote",                     "source.sass variable.other",                     "source.sass variable.sass",                     "source.scss variable.other",                     "source.scss variable.scss",                     "source.scss variable.sass",                     "source.css variable.other",                     "source.css variable.scss",                     "source.less variable.other",                     "source.less variable.other.less",                     "source.less variable.declaration.less",                     "storage.type",                     "storage.modifier",                     "storage.type.modifier",                     "support.type",                     "entity.name.type.object.console",                     "support.type.property-name",                     "variable.parameter",                     "parameters variable.function",                     "variable.language",                     "variable.parameter.function.language.special.self.python",                     "variable.parameter.function.language.special.cls.python"                 ],                 "settings": {                     "fontStyle": "italic"                 }             },             {                 "scope": "markup.bold",                 "settings": {                     "fontStyle": "bold"                 }             },             {                 "scope": "markup.underline",                 "settings": {                     "fontStyle": "underline"                 }             },         ]     }, }

Thank you for your patience! 🙏🏼
It‘s been a while since I had free time to focus more on Nord, and my open source projects in general, and invest time in this issue due to work-life balance.
I recently published the first “Northern Post — The state and roadmap of Nord“ announcement which includes all details about the plans and future of the Nord project, including the goal of catching up with the backlog. This issue is part of the backlog and therefore I want to triage and process it to get one step closer to a “clean state“. Read the announcement about reaching the “clean“ contribution triage state in Nord‘s discussions for more details about the goal.
Due to the scope of this issue of not being “purely technical“, or being related to the project‘s code at all, I‘ll convert it into a discussion for Nord‘s central community hub on GitHub. This way the repository is easier to maintain by separating conversations about the actual code while also making help requests, ideas and other contributions from users more visible and supportable by the community.


ENH: new class method for constructing ASD and now supporting slicing Aligned/Alignments with FeatureMaps

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

49 of 49 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 92.04%

Totals

Change from base Build<PHONE_NUMBER>9:
0.03%

Covered Lines:
33730

Relevant Lines:
36647

💛  - Coveralls


add reports 24

Add report, Media Capture: How foreign states infiltrate African media
Fix: Flask dependencies and python runtime compatible with herokubuildpack 20 (dokku)

What needs to happen to this PR @thepsalmist ?

What needs to happen to this PR @thepsalmist ?

This should be merged, released & deployed on tools-v3.19.4


Thermite causes extreme tick lag in solo.

When playing with the mod,I get very bad tick lag,blocks take a long of time to drop and mobs move very difficultly.And it takes less than a minute to get kicked out with the message "disconnected" which brings me back to the server select screen.While leave and opening back you world does make the lag a bit better.It still eventually freezes the internal server and I get kicked.Here are the mods am using if thats useful since removing thermite removes the lag entirely.

Please consider this issue resolved as I just realised i have c2me installed which is incompatible

I don't have c2me installed, but I am getting this issue. Any ideas on what other mods could be causing this?

Sorry for the late reply but i found out that any mod that optimizes chunk or changes how they generate can cause this issue (i dont mean mods that change the terrain or biomes)


Allow inline many-to-many relation syntax for mongoDB

There is an untested feature on mongodb that the query engine relies on: embedded inline many-to-many relations:
model A {
    id  String  @id @map("_id") @db.ObjectId
    gql String?

    b_ids String[] @db.Array(ObjectId)
    bs    B[]      @relation(fields: [b_ids])
}

model B {
    id  String  @id @map("_id") @db.ObjectId
    gql String?

    a_ids String[] @db.Array(ObjectId)
    as    A[]      @relation(fields: [a_ids])
}

We improved relation validations recently, and that case that slipped through now triggers a working validation error. Unfortunately, this version of many to many relations on mongo doesn't follow the same pattern and it became impossible to write it the way the Query Engine expects it. We should stop validating that the types of the referencing fields match the referenced fields on MongoDB as a first fix, then validate that the type of the embedded IDs in the array match the type of the id field in the referenced records, maybe as a second pull request. We could assume it is always ObjectId, but I don't have enough mongo knowledge to say if that assumption will hold, so let's do go with the laxer solution first.

We just released a patch, you can now use the 3.4.1 version that includes the fix.

Let's reopen this, because there is more work to be done for a proper solution. The patch release fix simply skipped the validation, but we should adapt it.

Internal discussion lead to a decision here, and the work will most probably be represented in a new issue. When that is created, we can close this one.

Duplicate of #11553


[Enhancement] DNS Injection

This PR changes several parts related to actions k3d is taking to modify DNS in k3d managed clusters.
Here's an excerpt:

remove--no-hostip flag and the related disableHostIPInjection config option
inject host IP on every cluster startup (except when hostnetwork is chosen)

this includes an entry in /etc/hosts and in the CoreDNS configmap
fixes #401

inject host entries for every cluster network member container into the CoreDNS configmap

fixes #522

@iwilltry42  I noticed case, where entries are still missing in coredns config map. I'm using k3d 5.2.1 and WSL2 with Docker for Windows. In case that I restart system, k3d containers will get up, however coredns config map will only contain entry for k3d-k3s-default-server-0.
Same will happen, if I stop k3d-k3s-default-server-0 docker container and start it again. It's fixed once I run k3d cluster stop and k3d cluster start, because host.k3d.internal entry is injected again, but it doesn't seem to be persisted in docker container itself.
Would it be possible to persist coredns config map without having to explicitly restart k3d cluster with k3d CLI?

@ewawadr , to make the CoreDNS fix more resilient, we went the route of overwriting the CoreDNS config in the container filesystem, so K3s auto-deploys it. This makes the injection itself more reliable.
However, the case you're seeing now is something we cannot fix ourselves unfortunately, because K3s will re-generate the underlying file upon startup.
However, we have work in progress to change the whole process (I just don't know how far back it will be backported by the K3s team).

I'm creating a custom k3s image for some CUDA stuff and I see an error when attempting to use Ubuntu 20.04 as a base image.
INFO[0012] Injecting records for hostAliases (incl. host.k3d.internal) and for 2 network members into CoreDNS configmap...

ERRO[0014] Failed Cluster Start: error during post-start cluster preparation: error overwriting contents of /etc/hosts: Exec process in node 'k3d-gputest-server-0' failed with exit code '126'

This works fine if i use Ubuntu 18.04 as the base image.
Build this Docker file and attempt to fire up k3d with it:
https://github.com/jakkaj/k3d-gpu/blob/ce67b95baae7022eef5e63242359c596301a21ff/Azure/Dockerfile
The hosts file is locked if you exe in to the container, which from memory is correct because Docker controls the file and hence it will be locked?

The command that is failing is strange, and it's failing with 126 - unable to execute. If I can't figure it out I'll open an issue.
DEBU[0013] Executing command '[sh -c cat /tmp/-etc-hosts-qsSPeKgMSSKEwQIPZyiW > /etc/hosts]' in node 'k3d-gputest-server-0'

TRAC[0013] Exec process '[sh -c cat /tmp/-etc-hosts-qsSPeKgMSSKEwQIPZyiW > /etc/hosts]' still running in node 'k3d-gputest-server-0'.. sleeping for 1 second...

ERRO[0014] Failed Cluster Start: error during post-start cluster preparation: error overwriting contents of /etc/hosts: Exec process in node 'k3d-gputest-server-0' failed with exit code '126'


[Firebase Analytics] Show more data in debugView console

Is your feature request related to a problem? Please describe.
Feature request result from issue: #1245
Describe the solution you'd like
I would like to see the "parameter" screenName of the even "screen_view" in the DebugConsole. Just showing screen_view in a debug doesnt reveal where a user a clicked in terms of screen transitions. Dont know if its even worth extending this feature request to SteamView->Events.
Additional context
Possible DebugView with the extended information (assuming Dashboard was clicked via NamedRoute)

Hey, unfortunately this is out of the control of this plugin. You're best creating your own log event to handle this as you'd need to pass in custom information about what was clicked.
Closing but happy to help in here.