voxel_map[voxel_index].push_back(i);

Hi , I tried to build the pkg but i get this error
Errors     << catkin_lidar/logs/dynablox_ros/build.make.001.log
catkin_lidar/src/dynablox/dynablox_ros/src/motion_detector.cpp: In member function ‘void dynablox::MotionDetector::blockwiseBuildPointMap(const Cloud&, const BlockIndex&, voxblox::AlignedVector&, dynablox::VoxelToPointMap&, std::vector<std::pair<Eigen::Matrix<int, 3, 1>, Eigen::Matrix<int, 3, 1> > >&, dynablox::CloudInfo&) const’:
/home/ideajoyan/catkin_lidar/src/dynablox/dynablox_ros/src/motion_detector.cpp:310:39: error: no matching function for call to ‘std::vector<Eigen::Matrix<int, 3, 1>, Eigen::aligned_allocator<Eigen::Matrix<int, 3, 1> > >::push_back(size_t&)’
310 |     voxel_map[voxel_index].push_back(i);
|                                       ^
In file included from /usr/include/c++/9/vector:67,
from /home/catkin_lidar/src/dynablox/dynablox_ros/include/dynablox_ros/motion_detector.h:9,
from /home/catkin_lidar/src/dynablox/dynablox_ros/src/motion_detector.cpp:1:
/usr/include/c++/9/bits/stl_vector.h:1184:7: note: candidate: ‘void std::vector<_Tp, _Alloc>::push_back(const value_type&) [with _Tp = Eigen::Matrix<int, 3, 1>; _Alloc = Eigen::aligned_allocator<Eigen::Matrix<int, 3, 1> >; std::vector<_Tp, _Alloc>::value_type = Eigen::Matrix<int, 3, 1>]’
1184 |       push_back(const value_type& __x)
|       ^~~~~~~~~
/usr/include/c++/9/bits/stl_vector.h:1184:35: note:   no known conversion for argument 1 from ‘size_t’ {aka ‘long unsigned int’} to ‘const value_type&’ {aka ‘const Eigen::Matrix<int, 3, 1>&’}
1184 |       push_back(const value_type& __x)
|                 ~~~~~~~~^
/usr/include/c++/9/bits/stl_vector.h:1200:7: note: candidate: ‘void std::vector<_Tp, _Alloc>::push_back(std::vector<_Tp, _Alloc>::value_type&&) [with _Tp = Eigen::Matrix<int, 3, 1>; _Alloc = Eigen::aligned_allocator<Eigen::Matrix<int, 3, 1> >; std::vector<_Tp, _Alloc>::value_type = Eigen::Matrix<int, 3, 1>]’
1200 |       push_back(value_type&& __x)
|       ^
/usr/include/c++/9/bits/stl_vector.h:1200:30: note:   no known conversion for argument 1 from ‘size_t’ {aka ‘long unsigned int’} to ‘std::vector<Eigen::Matrix<int, 3, 1>, Eigen::aligned_allocator<Eigen::Matrix<int, 3, 1> > >::value_type&&’ {aka ‘Eigen::Matrix<int, 3, 1>&&’}
1200 |       push_back(value_type&& __x)
|                 ~~~~~~~~~~~^
make[2]: *** [CMakeFiles/dynablox_ros.dir/build.make:89: CMakeFiles/dynablox_ros.dir/src/motion_detector.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [CMakeFiles/Makefile2:1252: CMakeFiles/dynablox_ros.dir/all] Error 2
make: *** [Makefile:141: all] Error 2>

Hmmm have never seen this before. Are you sure you checked out the right branch of voxblox and have you tried a clean rebuild of your workspace?

Hmmm have never seen this before. Are you sure you checked out the right branch of voxblox and have you tried a clean rebuild of your workspace?

Hi,
Thank you for responding.
Yes, I have followed your suggestions and ensured that I have checked out the correct branch of voxblox (dynablox/release). I also performed a clean rebuild of the workspace. I still get the same error while building the dynolbox,

Hi @nivand ,
Unfortunately we are not able to reproduce your problem on a clean install. Are you sure you use dynablox/main without any additional changes and your compiler supports C++17?

Hi @nivand ,
Unfortunately we are not able to reproduce your problem on a clean install. Are you sure you use dynablox/main without any additional changes and your compiler supports C++17? Alternatively, you can try to make sure Eigen is found correctly by adding eigen_catkin to the package.xml of dynablox/dynablox_ros.

Hi, Thank you for your response. Upon further investigation, I discovered that the issue was indeed related to the compiler not supporting C++17. After reinstalling the compiler with C++17 support, everything is now functioning correctly. I appreciate your assistance and suggestions.

Great to hear that your issues were resolved!


Missing tests in code

As per TA feedback #32

Tests included in Exploratory Data Analysis Script
Inferences script already had tests included

As it was required we have added test cases to:

Download Script
EDA Script
Inferential Analysis (already had the test cases)
Predictive  Modelling Script

So, I am going ahead and closing this issue.


[DOCU-2608] Update Doc: Architecture

Updated the title of the Architecture doc to match what is in the new Kuma 2.0 docs revamp. I also added a redirect link to direct from the old doc name to the new one. I wasn't able to test it locally, so a confirmation that it's correct would be great!
DOCU-2608
Signed-off-by: Diana<EMAIL_ADDRESS>

Closing this PR (but keeping the branch) until we start working on the Kuma 2.1 IA rework.


gosop inline-sign fails with "Unsupported option"

I'm running this test on debian testing:
0 dkg@alice:~$ gosop inline-sign
Please provide keys to create detached signature
69 dkg@alice:~$ gosop version --extended
gosop 0.1.0
GopenPGP 2.5.2
Compiled using go1.20.5
0 dkg@alice:~$ echo test | gosop inline-sign tester.key
Unsupported option
37 dkg@alice:~$

gosop inline-sign appears to work if i pass it --as=clearsigned though.

Hi DKG. Thanks for the issue. The gosop implementation on the main branch currently does not support inline signatures (except clearsigned). Nevertheless, an updated version 3 of GopenPGP is currently in development. The gosop-gopenpgp-v3 branch in this repo implements gosop with the new GopenPGP version. It implements  the latest api in draft-dkg-openpgp-stateless-cli-06 and implements all the crypto-refresh features as well. Feel free to try this version ;)

@lubux thanks for this!  I've encouraged the openpgp interop test suite to include gosop-v3.  You might want to comment over there if there's a specific tag or branch label that you'd like the interop test suite to pull from.

Great. Pulling from the gosop-gopenpgp-v3 branch as it is done now is fine as long as it is still marked as not stable.

it's been over a year since we had the conversation above.  any chance that we can get a release of the v3 branch?

@lubux Is on vacation for the coming two weeks, and I will be soon too. Most likely we'll make a release after that (in mid August), especially when the stable version of gopenpgp v3 is released :blush:

@dkg I just uploaded an build of gosop that uses gopenpgp v3 alpha to experimental:
gosop version --extended
gosop 2.0.0-alpha
GopenPGP 3.0.0-alpha.3
Compiled using go1.22.6

thanks!  it's exciting to see this progress

a gentle nudge here -- it'd be great to move the release along.  we're a bit past "mid August" 😉

I've tagged release v1.0.0 :)


Conflicting migrations detected - Tests will not run in this state

@alvacouch, @pkdash, @hyi - Looks like we managed to put conflicting migrations into the develop branch.
Have not checked migrations since the 1.9.5 release. Generally the makemigrations call is not part of our scripts and is only done at release time. The --merge flag as suggested below has not been part of our process at all to this point.
Reference:
CommandError: Conflicting migrations detected (0019_manual_populate_provenance, 0017_groupaccess_auto_approve in hs_access_control).
To fix them run 'python manage.py makemigrations --merge'

@alvacouch @pkdash @mjstealey It appears to me that this may be caused by two PRs that both involve migrations, and after one was merged, the other one rebased with develop, but did not run makemigrations --merge hence causes this conflicting migrations. @pkdash can you make a PR to fix this migration issue?

I am somewhat confused because I cannot reproduce the error on develop.
When I try to makemigrations --merge,  it reports there is nothing to merge.
Will check dependencies instead.

merged and tests are running again


Fix #83: Don't ever display status bar if presenting view controller didn't.

Without this fix, the status bar would briefly be presented (and then immediately dismissed) during the transition animation to display an image. More importantly, this would cause the y-position of the original image to be off by the height of the status bar during the transition animation, causing a distracting jump.
The fix is to never request the status bar to be displayed, if the presenting view controller was not presenting a status bar.
Fixed version of the GIF from the issue:

Awesome - thanks @clindsay 👍


matelcli's "upload" command does not support some files.

Binary files may break "upload".

da0c7bc632b53567ac8db5c6d8e5600a16721a05 should fix this problem.


PluckSynth starts crackling and stops entirely

Example
This was originally brought up in issue #274. If the page runs for more than a minute, CPU rises and will eventually stop all audio.

testing against latest, no longer exhibits this issue.


[3.x] Documentation: how to use non-core driver?

Issue Type

Bug report / Documentation

Molecule and Ansible details
$ ansible --version && molecule --version
ansible 2.9.6
  config file = /etc/ansible/ansible.cfg
  configured module search path = [u'/home/jenkins/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/lib/python2.7/dist-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 2.7.12 (default, Oct  8 2019, 14:14:10) [GCC 5.4.0 20160609]
molecule 3.0.2
   ansible==2.9.6 python==2.7

Molecule installation method (one of):

pip

Ansible installation method (one of):

pip

Detail any linters or test runners used:
Desired Behavior
try using lxd driver following https://molecule.readthedocs.io/en/2.22/configuration.html#lxd but not in https://molecule.readthedocs.io/en/latest/configuration.html#driver
from https://github.com/ansible-community/molecule/blob/35af47bcfe2a260bad8ed7f4c7c402a5c2cdaeae/CHANGELOG.rst#30, lxd driver and others were moved out of core and in separate repo https://github.com/ansible-community/molecule-lxd
currently
# pip install molecule-lxd
ERROR: Could not find a version that satisfies the requirement molecule-lxd (from versions: none)
ERROR: No matching distribution found for molecule-lxd

how should we use those?
Thanks
Actual Behaviour
$ molecule --debug create
---
dependency:
  name: galaxy
driver:
  name: lxd
lint: 'set -e

  #yamllint .

  #ansible-lint

  flake8

  '
platforms:
  - image: ${MOLECULE_DISTRO:-ubuntu:16.04}
    name: instance
provisioner:
  name: ansible
scenario:
  name: default
verifier:
  name: ansible

ERROR: Failed to pre-validate.

{'driver': [{'name': ['unallowed value lxd']}]}

The idea is pip install molecule-myplugin and then you're done. The issue at the moment is that nobody is maintaining those plugins so the integrations are not working. See https://github.com/ansible-community/molecule-hetznercloud for more info.


Refactor all table driven tests to use t.Run

Description
Use t.Run in all the table driven tests.
Why is this needed
Using t.Run gives us better selectivity with go test -run and also better grouping/splitting when running with go test -v.
How Has This Been Tested?
Just minor test changes, so tested with go test.

This actually depends on #146


dwavebinarycsp.stitch(..) now uses the updated Specification object f…

…rom the updated penaltymodel-core. This means that this version of dwavebinarycsp is not compatible with older penaltymodels

This is no the version update that I want. I don't want this in master's history, so I will make a new PR.


support reading configs from environment variables

Currently only file paths to the configs can be specified:
      --rbac.config=/etc/observatorium/rbac.yaml
      --tenants.config=/etc/observatorium/tenants.yaml

While there is nothing wrong with it, i think ability to provide these configurations via environment variables would simplify the usage.
Like this:
      --rbac.config=env:OBSERVATORIUM_CONFIG_RBAC
      --tenants.config=env:OBSERVATORIUM_CONFIG_TENANTS

wdyt?

Although I am not a big fan of configuration via environment variables, I don't necessary say it's a bad idea. IMHO the above is proposing a bit custom syntax with env:VAR_NAME which requires custom maintenance. I have seen (but don't recall names) in other projects to use the following pattern. First try if --rbac.config is set as a flag then lookup if SOME_PREFIX_RBAC_CONFIG is given, if both not given then it is not set. I am not sure if our current go flagset library supports this, but I believe this would work for both modes.

If I understood @periklis that would make sense to me as well - just have a predefined env var name which serves as an alternative to flag. Then:

Flag / config has precedent over env var
If none is provide, we use defaults (or, if configuration is mandatory, error out)


Add support of Angular 4

Would like to use this module with the recently released Angular 4.0.0.

Please add support for Angular 4.0.0 💃

Added in 0.1.9, thanks for your comments :)

can you make it ^4.0.0 to make it work without warning when npm install along side with angular-cli ?

Doesn't seem to work in ng4... returns a 404...

@chocmah thank you for your comment, changed


Use std::size_t everywhere instead of size_t

Can one of the admins verify this patch?

Current coverage is 89.47%

Merging #122 into master will not affect coverage as of b2e2710

@@            master    #122   diff @@
======================================
  Files           66      66
  Stmts         4141    4141
  Branches         0       0
  Methods        276     276
======================================
  Hit           3705    3705
  Partial          0       0
  Missed         436     436

Review entire Coverage Diff as of b2e2710

Powered by Codecov. Updated on successful CI builds.


feat: Report missing before/after calls on undo corruption

@Christopher-Chianelli One thing I totally forgot - we should add this to the docs. AFAIK there is a whole section on assert modes, the new one should show up there too, with an explanation of what it's for, ideally with an example. Please do that, and then I'll merge it.


Missing tests

Patches welcome. :-)
I'm not sure what best practices for writing tests is going to be. But it sure would be useful to have some.

I'm spent a little time on this today. Looks like I'm going to have to write an assertion library to make acceptance tests reasonable in rust.

Thanks! Surprised to hear you need more infrastructure. What are you missing?

High-level web-app domain assertions, e.g.:
assert_has_text(body, "Welcome to the pastebin.");
assert_has_selector(body, "ul:empty");

It'd also be nice to have a Capybara-style browser driver API:
driver.click_link("Sign in");
driver.fill_in("Username", with: "bartmassey");
driver.press_button("Login");

But for now, I'm just focusing on the former.


hawk-go: Update to master plus tent/hawk-go#7

This normalizes how URL.RequestURI is used within the Hawk normalized
request string.

Why?

The motivation is so it works with coreupdate behind an nginx proxy.
The technical reason is because this project sets the Opaque property of the client request URL and hawk-go uses URL.RequestURI() to create the Hawk normalized request string. When URL.Opaque is not set this returns Path+?+RawQuery (which is correct according to the hawk reference implementation), otherwise it returns something else.
When nginx proxies requests to coreupdate it sets the absolute path in the request line instead of the absolute URL and invalidates the Hawk MAC.

Gotcha. Thanks.

LGTM


docs/advanced-topics/scorecard: pin master links to a commit

Description of the change: fix scorecard links
Motivation for the change: broken links
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[ ] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] Add or update relevant sections of the docs website in website/content/en/docs

This PR fixes master links, so I'm going to merge this now and we can follow-up with a more robust solution.
/hold cancel


ROS2 Porting: scenario_selector

Summary
Fairly straightforward port of the scenario_selector package - dependencies to topic tools package so ensure that the ros2 branch of AutowareArchitectureProposal is up to date.
Notable Changes:

Remove boost dependencies

boost::function -> std::function
boost::bind -> std::bind

Pull in topic_tools dependencies

Testing
Compilation should be successful and launch files should launch with issue. The launch files here do not come with default parameters so you will need to input these manually during launch, i.e:
ros2 launch scenario_selector dummy_scenario_selector_parking.launch.xml output_scenario:=test_out_scenario input_parking_trajectory:=test_in_traj output_trajectory:=test_out_traj

Discussion Points

Launch rostopic as a node in the launch file has been commented out - I am not sure if there is a different way of achieving the publishing of topics via the launch file.
fromMapBin now only takes in two argument - is this correct?

CI is failing as topic_tools is not part of the build_depends.repos file I think

@cvasfi Thanks for the thorough review and the excellent suggestions! I fixed the changes you requested and now CI should pass as #102 has been merged.


Restructure Project for Proper Auto-completion

This project has problems with auto-complete. This is something for 2.0.0 as it would require complete re-work of structure.

The problems boil down to these two places:

Extensions in library_ext aren't registered by most IDE auto-completion helpers.
Enumeration values aren't registered by IDEs. Implementing #86 would fix this.

This would also require switching to a better docstring format for type hinting. ie:
Before
def read_log(self, idx, offset size):
    """
    ...

    in idx of type int
        Which log file to read. 0=current log file

    in offset of type int
        Offset in the log file.

    in size of type int
        Chunk size to read in the log file.

    return data of type str
        Data read from the log file
    """

After
def read_log(self, idx, offset size):
    """
    ...

    :param idx: Which log file to read. 0=current log file
    :param offset: Offset in the log file.
    :param size: Chunk size to read in the log file.
    :type idx: int
    :type offset: int
    :type size: int
    :rtype: str
    :returns: Data read from the log file
    """

These hints will show up much nicer in an IDE, especially with a docstring inspector. Possibly could help with auto-generated documentation as well.

Would love to have your help with this @mjdorma when you have time. :)

Good idea wrt to marking up the variables. That should be within the scope of the current builder :).

That's what I was hoping! It's so much easier to stand on the shoulders of giants ;)


Panic at startup: "Failed to initialize any backend"

Describe the bug
Having installed Neovide 0.7.0 from snap store I get  a thread panic on startup:
Failed to initialize any backend!
Screenshots
❯ RUST_BACKTRACE=1 neovide
thread 'main' panicked at 'Failed to initialize any backend! Wayland status: NoCompositorListening X11 status: XOpenDisplayFailed', /root/.cargo/git/checkouts/winit-cdba476763ae8f28/3b5462e/src/platform_impl/linux/mod.rs:626:9
stack backtrace:
Ignored client type property: "methods"
Ignored client type property: "attributes"
   0: rust_begin_unwind
             at /rustc/53cb7b09b00cbea8754ffb78e7e3cb521cb8af4b/library/std/src/panicking.rs:493:5
   1: std::panicking::begin_panic_fmt
             at /rustc/53cb7b09b00cbea8754ffb78e7e3cb521cb8af4b/library/std/src/panicking.rs:435:5
   2: winit::platform_impl::platform::EventLoop<T>::new_any_thread
             at /root/.cargo/git/checkouts/winit-cdba476763ae8f28/3b5462e/src/platform_impl/linux/mod.rs:626:9
   3: winit::platform_impl::platform::EventLoop<T>::new
             at /root/.cargo/git/checkouts/winit-cdba476763ae8f28/3b5462e/src/platform_impl/linux/mod.rs:581:9
   4: winit::event_loop::EventLoop<T>::with_user_event
             at /root/.cargo/git/checkouts/winit-cdba476763ae8f28/3b5462e/src/event_loop.rs:134:25
   5: winit::event_loop::EventLoop<()>::new
             at /root/.cargo/git/checkouts/winit-cdba476763ae8f28/3b5462e/src/event_loop.rs:120:9
   6: neovide::window::window_wrapper::start_loop
             at /root/parts/neovide/build/src/window/window_wrapper/mod.rs:204:22
   7: neovide::window::create_window
             at /root/parts/neovide/build/src/window/mod.rs:50:5
   8: neovide::main
             at /root/parts/neovide/build/src/main.rs:198:5
   9: core::ops::function::FnOnce::call_once
             at /rustc/53cb7b09b00cbea8754ffb78e7e3cb521cb8af4b/library/core/src/ops/function.rs:227:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Desktop (please complete the following information):

OS: Ubunu 18.04.5 (but same error on my 21.04 machine)
Neovide: 0.7.0
Neovim:

NVIM v0.5.0-654-g3acfefb63
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Compilation: /usr/bin/cc -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -O2 -g -Og -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fno-common -fdiagnostics-color=always -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LEVEL=3 -I/build/nvim/parts/nvim/build/build/config -I/build/nvim/parts/nvim/build/src -I/build/nvim/parts/nvim/build/.deps/usr/include -I/usr/include -I/build/nvim/parts/nvim/build/build/src/nvim/auto -I/build/nvim/parts/nvim/build/build/include
Compiled by root@lgw01-amd64-012

Please run neovide --log and paste the contents of the .log file here:
thread 'tokio-runtime-worker' panicked at 'called `Result::unwrap()` on an `Err` value: GlobError { path: ".", error: Os { code: 13, kind: PermissionDenied, message: "Permission denied" } }', /rustc/53cb7b09b00cbea8754ffb78e7e3cb521cb8af4b/library/core/src/ops/function.rs:150:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: PoisonError { .. }', /root/.cargo/registry/src/github.com-1ecc6299db9ec823/flexi_logger-0.17.1/src/writers/file_log_writer.rs:73:57

Installed with snap install neovide, could this be a snap sandbox issue?

I have the exact same error, with the snap package, on Ubuntu 20

Today I build Neovide from source, at commit  f58b947, and now it works ok.
My nvim is also installed as a snap --thought I'd mention as @Mate2xo mentioned appimage.

I have the exact same error, with the snap package, on Ubuntu 20.
The Neovim I have on my system is an .appimage, I wonder if that could be the problem.
I have then built Neovide from source, and when lauching it I got this error
ERROR [neovide::bridge] nvim not found!

Is there a way to configure Neovide to point to my Neovim appimage? I couldn't find anything related to that in the Wiki.
Using the remote TCP option on localhost works though, it's apparently the only way I can launch Neovide on my sytem

It must be on the system path. Thats how neovide knows where to find it

Today I build Neovide from source, at commit f58b947, and now it works ok.
My nvim is also installed as a snap --thought I'd mention as @Mate2xo mentioned appimage.

Interesting. Not sure what would have fixed that... But I guess your problem is fixed now?

Well, this was on my home machine and I was wondering if I could install this on my work laptop in which case snap would be preferable to building from source. If I find another Ubuntu box to play with I might try and snap install neovide and nvim and see if they play nice together.

It must be on the system path. Thats how neovide knows where to find it

Actually, it was on my system path, but my appimage file was named nvim.appimage. Renaming it to just nvim made it visible by neovide. Thanks :+1:

Closing as I think both problems are resolved. Feel free to reopen if not


'v' pattern flag support, log descriptive errors for invalid (failing to compile) regexp's

What version of Ajv you are you using?
latest
What problem do you want to solve?
Missing support for new 'v' regexp flag (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Regular_expressions/Character_class#v-mode_character_class).
Now regexp, that fail to compile, break validator totally. An exception is not descriptive, it's hard to find the location of a failing pattern keyword.
What do you think is the correct solution to problem?
Add optional support for new 'v' regexp flag. Catch regexp initialization exceptions and log descriptive errors to console including invalid regexp itself. For example: "pattern 'xxx' at  failed to compile with 'u' flag: . Check regexp validity or try changing compilation flag".
Will you be able to implement it?
Yes, but need to discuss new option for enabling 'v' flag.

Hi @ekzobrain thanks for raising this. Can you prepare an example in runkit to help fast track investigation?

This is supported by the regexp keyword in the ajv-keywords project.
https://ajv.js.org/packages/ajv-keywords.html#regexp


Change the init.py file to use the relevant pretrained function

Using the relevant pretrained function for loading the correct model when using a fine-tuned qlora version for qwen2 since in the original file we are using Videollama2MistralCausualLM for loading the pretrained model

Which will give wrong result when loading a model that was fine-tuned using qlora. So added the model to load using  Videollama2Qwen2ForCausalLM as shown below

Thanks for your contribution ❤


Missing .zbin files

Hey guys,
First, thanks for all!
I cloned the main branch of the repository and then tried to run a node using make run-node or make run-node-light  without success.
Here is the errors
Compiling era_test_node v0.1.0-alpha.14 (/Users/qdqd/workspace/zksync_era_precompiles/.test-node-subtree)
error: couldn't read src/deps/contracts/EcAddG2.yul.zbin: No such file or directory (os error 2)
   --> src/deps/system_contracts.rs:168:13
    |
168 |             include_bytes!("contracts/EcAddG2.yul.zbin").to_vec(),
    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: this error originates in the macro `include_bytes` (in Nightly builds, run with -Z macro-backtrace for more info)

error: couldn't read src/deps/contracts/EcMulG2.yul.zbin: No such file or directory (os error 2)
   --> src/deps/system_contracts.rs:178:13
    |
178 |             include_bytes!("contracts/EcMulG2.yul.zbin").to_vec(),
    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: this error originates in the macro `include_bytes` (in Nightly builds, run with -Z macro-backtrace for more info)

error: could not compile `era_test_node` (lib) due to 2 previous errors
make[1]: *** [rust-build] Error 101
make: *** [/Users/XXXX/YYYYY/zksync_era_precompiles/.test-node-subtree/target/release/era_test_node] Error 2

It seems that both EcAddG2.yul.zbin and EcMulG2.yul.zbin are missing on main. Am I missing something? Should I generate them or something? I am not familiar with this file format.
Any advice would be appreciated.

Hey @qd-qd! thanks for opening the issue can you provide the Rust version that you are using to compile this?

Hello.
Working on macosx, with M3pro, rustc 1.78.0-nightly (0ecbd0605 2024-02-25), i face the same errors as described above.

Okay, I see. It's possible that I have a strange local state of the precompiles. Let me get back to you with a more detailed answer.

Hey @rdubois-crypto, @qd-qd, I believe the required files have been successfully pushed. Could you please attempt the compilation again to confirm if the issue has indeed been resolved? If not, please feel free to reopen the issue.

#225 is fixed as well as #223, however a new issue occured with ecmul, described in #232


export telespeech ctc models to sherpa-onnx

Support exporting models from https://github.com/Tele-AI/TeleSpeech-ASR to sherpa-onnx.
Thanks to @lovemefan for exporting the models to onnx and providing inference script for it.

You can download the exported models from
https://github.com/k2-fsa/sherpa-onnx/releases/tag/asr-models


ReferenceError: t is not defined on Next.js 14.1

Seems to be an issue with jotai-optics on Next.js 14.1. It was working fine in Next.js 14.0. Would be grateful for any advice please
Unhandled Runtime Error
ReferenceError: t is not defined

Call Stack
eval
(app-pages-browser)/./node_modules/.pnpm/jotai-optics@0.3.1_jotai<EMAIL_ADDRESS>(19:37)
(app-pages-browser)/./node_modules/.pnpm/jotai-optics@0.3.1_jotai<EMAIL_ADDRESS>file:///workspaces/myworkspace/.next/static/chunks/app/(site)/page.js (237:1)
options.factory
file:///workspaces/myworkspace/.next/static/chunks/webpack.js (716:31)
__webpack_require__
file:///workspaces/myworkspace/.next/static/chunks/webpack.js (37:33)
fn
file:///workspaces/myworkspace/.next/static/chunks/webpack.js (371:21)
eval
(app-pages-browser)/./src/lib/store.ts (11:70)
(app-pages-browser)/./src/lib/store.ts
file:///workspaces/myworkspace/.next/static/chunks/app/(site)/page.js (973:1)

Can we get any advice from Next.js team, about what we should do?
Does using v0.3.2 change anything?

No change of v0.3.2 that I can see. Thanks

It shouldn't be jotai-optics only issue. Maybe it happens to many jotai-* packages.

Are you using the @swc-jotai/react-refresh plugin? Then maybe this is related: https://github.com/pmndrs/swc-jotai/issues/24

Nope, not using SWC


added meta property on track analysis and added some comments on attr…

…ibutes

…ibutes

Can you explain more clearly about your pull request.


withTrashed on relation does not work

My code:
public function staff(){
     return $this->belongsTo('App\Models\Staff','staff_id','_id')->withTrashed();
}

I'm getting NULL on soft deleted model even with withTrashed(). Please help.

Duplicate https://github.com/jenssegers/laravel-mongodb/issues/1361


fix: fix wg anchor tags in areas page

fixes #7994
In https://github.com/ietf-tools/datatracker/blob/main/ietf/templates/group/active_wgs.html, it uses upper,
    {% for area in areas %}

{{ area.name }} ({{ area.acronym|upper }})

so it's now doing the same on the area's side.

The IESG agenda page also has this issue. It's been addressed here, https://github.com/ietf-tools/datatracker/pull/7995/commits/4b2e45ab296d66a50d8c2c44cb21522aa6952c73.

jQuery

pyquery?

yes, I meant pyquery


Upgraded to 0.16.0 and FancyZones not working neither can I uninstall

Environment
Windows build number: [run "ver" at a command prompt] [Version 10.0.18362.657]
PowerToys version: PowerToysSetup-0.16.0-x64
PowerToy module for which you are reporting the bug (if applicable):

Steps to reproduce Editing
After editing Zones, applied layout selected (doesn't mater if predefined or custom) nothing happens. Tried to uninstall PowerToys via Programms and features, runs through the steps and explorer restarts before it completes - PowerToys never gets removed

Expected behavior

Actual behavior

Screenshots

Update: was able to uninstall via command line: wmic:root\cli>product where "name like 'Power%'" uninstall
Went back to PowerToysSetup-0.15.2-x64 and all is working as normal

Currently having the same issue with FancyZones not working with this update. I've created my zones and hold shift when dragging but instead of the zone lighting up, my window I'm dragging just goes transparent.

Same issue - the layout just won't be saved and fancyzones doesn't do anything except making windows transparent while dragging with shift. If I open the dialog to set the layout again immediately after selecting one, it will already have reset.

@MannyQED @LiveFastCyYoung @Silun
later today we will release 0.16.1 with some fixes.
If the problem persists with 0.16.1, please upload this file:
C:\Users\%USERNAME%\AppData\Local\Microsoft\PowerToys\FancyZones\zones-settings.json

Be aware it contains the app history section (is the list of apps that you have zoned), feel free to edit the file to remove the app history section for privacy reasons.
When you upload it, change the extension to .txt or zip it otherwise GittHub will not accept it.
Thank you.

@enricogior thank you

In case, in order to avoid to edit the file to remove the app zone history, you can:

quit PowerToys
delete the zones-settings.json file
restart PowerToys
create a custom layout or apply a template and try to use it

In this case the app zone history will be empty, and the zones-settings.json file will only have the data important to investigate the bug.

Same. For me its still broken, even in 0.16.1

@ion-elgreco
can you try the steps above and upload the zones-settings.json file?
Thanks.

@ion-elgreco
can you try the steps above and upload the zones-settings.json file?
Thanks.

https://github.com/microsoft/PowerToys/issues/1872#issuecomment-607942032 Already did

I installed this program 2 days ago. It worked fine for 1 day, but now today it doesn't work anymore for some reason.
Same symptoms, I drag the window and the custom zones I created don't show anymore on screen

I installed this program 2 days ago. It worked fine for 1 day, but now today it doesn't work anymore for some reason.
Same symptoms, I drag the window and the custom zones I created don't show anymore on screen

Yeah, same. It works fine for a while and then suddenly it stopped working. No error message and nothing peculiar happened, that I would be able to notice. Uninstalling, removing, reinstalling and nothing helps to bring back its functionality. The software won't properly uninstall, because it leaves constantly backup files behind.

@ion-elgreco
in the other issue you said the FZ editor wasn't opening, here the issue is not the the editor doesn't open, but that the FZ applied layout/template is not working.
Can you please clarify what is your issue and only post here if you can open the editor but then the zones don't activate.
Thanks.

@vohzi
if you open the FZ editor again and select and apply one of the existing template does it still not work?

@ion-elgreco
in the other issue you said the FZ editor wasn't opening, here the issue is not the the editor doesn't open, but that the FZ applied layout/template is not working.
Can you please clarify what is your issue and only post here if you can open the editor but then the zones don't activate.
Thanks.

Ah sorry, I misunderstood his post. For me I can't open the editor or change templates, win + ` shortcut doesn't work. I see it sometimes loading in my mouse pointer, but then nothing happens.

@vohzi
can you upload the file:
C:\Users\%USERNAME%\AppData\Local\Microsoft\PowerToys\FancyZones\zones-settings.json

You need to zip it or to change the extension to .txt. otherwise Github will not accept it.
Thanks.

zones-settings.txt

@vohzi
I don't see anything wrong with the settings, but let's try this:

quit PowerToys
delete the zones-settings.json file
restart PowerToys
create a custom layout with one zone and apply it
try to use it
upload the zones-settings.json file

zones-settings.txt

@vohzi
FZ fails to identify your current monitor. You are running FZ on your PC, not in a remote session or in a virtual machine, correct?
Can you provide some information on the hardware? Is it a desktop PC with an external monitor or is it a laptop?
Also, have you already updated to 0.16.1?

I indeed have a receiver in between of my PC and my HDTV linked together with HDMI
I just did the update to 1.16.1 and it didn't help.
Here's my full config :
OS : Windows 10 64 Bits Pro
Processor Name: Ryzen 9 - 3950x @ stock
Motherboard Model: MSI b450 Tomahawk MAX
Cooler : NH-D15 chromax black
Video Card: Gigabyte GTX 970 G1
Memory: Corsair vengeance 64 GB - 3200mhz CMK64GX4M4B3200C16
SSD : Samsung SSD M.2 970 Evo Plus 2 TO
PSU : CORSAIR RM850x GOLD
Case : Fractal Define XL R2
Mouse : Logitech G600
Keyboard : Logitech Illuminated Keyboard K800
Monitor : 55" 10 years old HDTV with 60ms of input lag :(
Sound system : Yamaha RX-A660 + 5.1 Triangle Zerius / Comete / Sextan 902
Mic : TONOR Cardioid

Actually the update to 1.16.1 seems to have fixed it ... for now.
I will see if it still works the next days

@vohzi
can you please re-upload the zones-settings.json file? It will allow to confirm you were effected by the the bug we fixed in 0.16.1.
Thanks.

zones-settings.txt

So I just tried this morning with PowerToysSetup-0.16.1-x64 and everything seems to be working as normal so far... Thank you

@vohzi
it's confirmed, you had a similar problem as in https://github.com/microsoft/PowerToys/issues/1821 even if in your case it wan't the remote desktop but the receiver (different devices generating a similar device id that we were not processing correctly)
@MannyQED
I'm assuming something similar was also happening to you, so I'm closing this issue for now as a duplicate of https://github.com/microsoft/PowerToys/issues/1821 (as explained above, the bug is the same for different scenarios)
Thank you for helping identifying the problem and sorry for the regression.


inconsistency of .format()

moment("2010/2/31").format()  //"2010-03-03T00:00:00+05:30"
From the above statement i would expect the output should be "2010-02-31T00:00:00+5:30" but it is outputting "2010-03-03T00:00:00+05:30"
Is it the effect of leap year functionality,since there is no 31st day in february is it giving me 3 days ahead output?

@ChetanAgarwal11
Why u need to specify the date not exist?
I think it is not a bug of moment, and this behavior is comes from Date class of JavaScript.
> new Date('2010/2/31')
2010-03-02T15:00:00.000Z

If u want to know the date of end of month, u can use following code instead.
> moment('2019/2/1').endOf('month')
moment("2019-02-28T23:59:59.999")

Also keep in mind, similar to what @eternalharvest said, when you format a string going into a date like "2010/2/31" moment falls back to using the native Date constructor.  If you want it to give you an invalid date for that day, pass it in ISO format:
moment('2010-02-31')

Thank you @lad475 and @eternalharvest


Create public bucket (without authorization) on docker-compose start

Is it possible to provide a bucket name via an environment variable or in any other way, so it'll be automatically created on docker-compose start?
Or is it just possible to disable the authorization to do it on your own when starting integration tests?

It seems S3 Ninja does it automatically.
I've had the same concern. I had already started planning on doing a bucket creation request in the test suite setup, but when I tried running tests without it the branch was already there without me going through the UI or doing the bucket creation request.
Not sure how which part of S3 Ninja does this, but one of my requests (initiate multipart upload, upload chunk, complete multipart upload) triggered the bucket creation.
Great project!

Hello @feanor777 👋 Please excuse my super-late response, we have been busy working on a thrilling commercial project. Your request has been on my todo-list the whole time.
S3 Ninja can be configured to auto-create buckets in a configuration key called storage.autocreateBuckets. By default, this should be enabled. (To be honest, I did not check this.)
https://github.com/scireum/s3ninja/blob/ca2b261fac2e624875f0b8d27fe82a9e743c82f6/src/main/resources/application.conf#L26
This value is made available via ninja.Storage.isAutocreateBuckets().
https://github.com/scireum/s3ninja/blob/ca2b261fac2e624875f0b8d27fe82a9e743c82f6/src/main/java/ninja/Storage.java#L40
This method is called for (hopefully) all API calls, particularly in ninja.S3Dispatcher.checkObjectRequest(...).
https://github.com/scireum/s3ninja/blob/ca2b261fac2e624875f0b8d27fe82a9e743c82f6/src/main/java/ninja/S3Dispatcher.java#L569
Did you already try this, and S3 Ninja failed to auto-create a bucket? If so, which operation? Also, please consider the input provided by @almirsarajcic. (Thanks a lot for this, Almir!)

With #222, creation of public buckets will be possible by setting header x-amz-acl: public-read-write. In the AWS S3 SDK, this will look like:
client.createBucket(new CreateBucketRequest(bucketName).withCannedAcl(CannedAccessControlList.PublicReadWrite));


Re: 'Document Checking Service pilot'

Problem with 'Document Checking Service pilot' (https://www.api.gov.uk/gds/document-checking-service-pilot/)
It does not work

Hi - thanks for raising this; we only run the catalogue, not the service itself, so would recommend getting in touch with the service team through the documentation link. Bear in mind it is a pilot programme, though, and therefore may not be run as a full service.


chore(pre-commit): tweak mypy config to be more useful

Description
Tweaked mypy config in pyproject.toml to be less restrictive, but still useful.
Type of change

[X] Re-factoring without changing existing functionality

How Has This Been Tested?
Local tests pass
Automated PR Description
Commits in this PR:

530cf2c chore(pre-commit): tweak mypy config to be more useful

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 36.17021% with 30 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

src/api/chainlit/message_handler.py
0.00%
12 Missing :warning:

src/infrastructure/common/output_formatter.py
0.00%
7 Missing :warning:

src/core/chat/claude_assistant.py
70.00%
6 Missing :warning:

src/api/chainlit/command_handler.py
0.00%
2 Missing :warning:

src/core/chat/tool_definitions.py
60.00%
2 Missing :warning:

src/services/manager.py
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines
Coverage Δ

src/services/manager.py
0.00% <0.00%> (ø)

src/api/chainlit/command_handler.py
0.00% <0.00%> (ø)

src/core/chat/tool_definitions.py
85.71% <60.00%> (-9.03%)
:arrow_down:

src/core/chat/claude_assistant.py
37.12% <70.00%> (+0.82%)
:arrow_up:

src/infrastructure/common/output_formatter.py
0.00% <0.00%> (ø)

src/api/chainlit/message_handler.py
0.00% <0.00%> (ø)


Look at ways to simplify the facebook configuration

Currently the two facebook methods (implicit and user token) require slightly different configurations. We should investigate whether they can share configuration (and look up any details if needed)

Looking at the facebook docs we can either make a call to obtain an app token from the app id and secret, or use the combination of app id and app secret as the token itself.


Add condition to update_from_ha_state()

Related to a wrong behavior with lightify component. After the light
was turned on the nuimo could not rotate left. After this correction
the use case works better. To be tested if this correction does not
introduce drawback with other lights.
Error Logs:
“
2017-05-16 15:53:50,605 DEBUG nuimo_app.components.light received_brightness: 254, Brightness: None
2017-05-16 15:53:50,606 DEBUG nuimo_app.components.light if received_brightness is not None:
2017-05-16 15:53:50,607 DEBUG nuimo_app.components.light Is On: True, Brightness: None
2017-05-16 15:53:50,657 DEBUG nuimo_app Ignoring gesture event: Gesture.BUTTON_RELEASE
“

👍 Looks good to me


api-version parameter being passed to Microsoft Graph

Hi,
Thanks for the great library.  I've encountered a small issue when making calls to the Microsoft Graph API - see Stack Overflow here.  I see you've recently changed part of the request function in #11, but that doesn't seem to have made any difference.
The fix seems fairly simple; to not append the api-version parameter on the MS Graph, (although they shouldn't be spitting out malformed JSON no matter what...) but I didn't want to do it myself via pull request without checking first whether you mind adding a hard-coded check for something like that.

Hello,
I had something similar on mind - but I didn't implement it because it could make people go mad about the hardcoded check, however, since you came with that - I don't mind at all - it anyways seems to be the only easy way! It would be great if you could submit the PR, because I can get to doing this myself no sooner than during the weekend.
Thanks a lot,
Jan

Thanks for the quick response.  It's worth noting for anyone who ends up on this page that Microsoft have rolled out a bugfix to invalidate requests with the api-version parameter (see here) so if you're using Microsoft Graph you'll have to update to v1.1.1.


[Feature Request] Disable streak notifications entirely

Love the module, I really like the idea of having something to point my players at to go 'look, you're rolling statistically average, no matter what it feels like', but I really don't care about the streak stuff, and neither do my players. I'd like to completely turn it off please.

The latest version should have an option to disable streaks entirely.


Update best-practices.md

🚀 Deployed on https://6459936745d35b581dc6e7a6--cozy-fox-0defec.netlify.app


Added a method in tile math for Geometries Meters to Lng Lat

Needed to go full circle on projections

seems like intellij reformatted the file. sorry about that.

I'll cut a release soon.


tarballUrl global and && when checking for iojs

ouch!

merged and published as v3.0.3, @zkat when you take node-gyp to npm would you mind making sure you have >=3.0.3 please?

Will do!
On Sun, Sep 13, 2015 at 5:57 PM Rod Vagg<EMAIL_ADDRESS>wrote:

merged and published as v3.0.3, @zkat https://github.com/zkat when you
take node-gyp to npm would you mind making sure you have >=3.0.3 please?
—
Reply to this email directly or view it on GitHub
https://github.com/nodejs/node-gyp/pull/729#issuecomment-139933190.

--
kat


task: add "ps_m" corelens module

Please also include tests for these functions, at least smoke tests.

Also, please do re-push your branch to the internal gitlab to run vmcore tests as well. Thanks!


feat:Add a helper function to allow for dynamic mariadbHost override

I'm currently deploying this chart alongside a mariadb galera cluster chart, and I would like to bundle both of the charts under an umbrella chart.
But I want to be able to use a dynamic name for the mariadb host, the one the cluster creates, that is composed of the release and chart name.
Right now this is not possible due to helm restrictions of dynamic values in the values.yaml file.
With this approach, we could be able to override the helper in the umbrella chart, and defer the naming to the helper used in the mariadb chart.
This helper would normally return the mariadbHost set in the values file, so this is not a breaking change.

I also did the same to the hostnames of the redis servers, so they can be overwritten dynamically too, in the parent chart.

As there are no tests, I'll test manually upgrading the chart.

For mariadb galera,
Be careful with stateful services on kubernetes. Create a heavy sample db (develop branch has 950+ tables) and try out failure and restore scenarios.

there are multiple pods from stateful set running, mariadb-{0,1,2}
0 is master
0 reboots, 2 becomes master (assumption)
during node upgrades or helm chart upgrades, statefulset are taken down in sequence 2, then 1, then 0.
If mariadb galera doesn't start, need to exec into pod's container.
no containers are running in a pod and it will be in CrashLoopBackOff, inject additional forever running low memory container to the statefulset/pod
If things mess up, there is a chance of all databases getting wiped.

managing volumes and stateful services on kubernetes is possible with high memory nodes. at least 4GB RAM is need just to make storage HA. After that add RAM for actual applications. so 8GB+ RAM nodes.
I moved away from using helm chart for redis and mariadb.
I setup a dedicated vps for stateful services, redis, mariadb, nfs, mongodb (for other apps). This stateful vps has volumes mounted for above services, e.g /var/lib/mysql
If i need to scale I'll scale this setup into HA.
Other option with some additional cost is to get these services from cloud provider.
If I have my datacenter with bare metal high memory+cpu compute and redundant SSDs setup with RAID1,
I'll setup HA storage on kubernetes.
If I depend on cloud providers for resources then it doesn't seem to be cost effective at least now.

@revant thank you for the advice.
I'm still deciding on whether to use a managed cloud database. I made some estimates, and it seemed that the most cost-effective solution would be to deploy the galera cluster on kubernetes.
I will, however, try out some of the scenarios you outlined to make sure the cluster is production ready.

I tested locally, it is working on upgrades.
I DID NOT use galera cluster.
Do post your results about mariadb galera here.


Images Heading

to do

done

@tony-shannon
Here is the video just in case: https://drive.google.com/file/d/1t_OA3dFkfuCs3cIUezSRqWnY78DKHYCo/view

thanks Anna
T
On 16 February 2018 at 11:36, Anna Otrokhova<EMAIL_ADDRESS>wrote:

@tony-shannon https://github.com/tony-shannon
Here is the video just in case: https://drive.google.com/file/d/1t_
OA3dFkfuCs3cIUezSRqWnY78DKHYCo/view
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PulseTile/PulseTile-React/issues/92#issuecomment-366213090,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA5X3PIFgzoMV3G3obnZgWA4d40B9P6oks5tVWgjgaJpZM4Ra8l3
.


Bugs in sdk

Hi, I am trying to setup Statsig in our rails app and starting with react sdk v1.15.2. Here is the provider implementation:
import * as React from 'react'
import { StatsigProvider, StatsigUser } from 'statsig-react'

type Props = {
  children: React.ReactNode | React.ReactNode[]
  statsigUser: StatsigUser
}

const StatsigClientKey = 'client-***'

const Statsig = ({ children, statsigUser }: Props): JSX.Element => {

  return (
    <StatsigProvider
      user={statsigUser}
      waitForInitialization={true}
      sdkKey={StatsigClientKey}
    >
      {children}
    </StatsigProvider>
  )
}

export default Statsig

At first, I got this error:
Uncaught ReferenceError: exports is not defined
    at ./node_modules/statsig-js/dist/index.js (index.js:21:1)
    at __webpack_require__ (bootstrap:19:1)
    at ./node_modules/statsig-react/dist/index.js (index.js:14:1)
    at __webpack_require__ (bootstrap:19:1)
    at ./app/javascript/src/providers/Statsig.tsx (Statsig.tsx:1:1)
    at __webpack_require__ (bootstrap:19:1)

I fixed it by declaring a global exports object. But now I am stuck on this error:
Statsig.js:186 Uncaught (in promise) TypeError: statsig_js_1.StatsigClient is not a constructor
    at Function.<anonymous> (Statsig.js:186:1)
    at step (Statsig.js:141:1)
    at Object.next (Statsig.js:71:1)
    at Statsig.js:43:1
    at new Promise (<anonymous>)
    at ./node_modules/statsig-react/dist/Statsig.js.__awaiter (Statsig.js:22:1)
    at Statsig.initialize (Statsig.js:182:1)
    at StatsigProvider.js:140:1
    at invokePassiveEffectCreate (react-dom.development.js:23313:1)
    at HTMLUnknownElement.callCallback (react-dom.development.js:3922:1)

Is this an issue with sdk itself? Anything I am missing here?

I noticed that this sdk and js-client are both using "commonjs" module system. When I created the dist for both projects using "es6" module system and setting "esModuleInterop" to true, the error is gone.
I wonder if these changes can be applied to both the sdks.

Thanks for the report, we can take a look!

Hi, any updates on the beta releases for these sdks?

Trying out this for now:
https://www.npmjs.com/package/statsig-react/v/1.16.2-beta.0

Fixed in v1.17.0
https://github.com/statsig-io/react-sdk/releases/tag/v1.17.0


[Slider] Wrong thumb position at the right

How can we reproduce this bug?
Just move the slider thumb to the most right
What did you expect to happen?
I expect the thumb right border to be on the same line where the slider line ends
What happened instead?
The thumb position gets a bit more to right (~10px)
Test case
you can use your own page with examples: http://foundation.zurb.com/sites/docs/slider.html

Related to #7660.

Duplicate of #7660

Thanks, guys :) And sorry for the duplicate.

No worries!


fixing see output on Enter

Both of the buttons, and clicking "Enter" now trigger the output to text 👍

Perfect. But, you got rid of the "COPY" button.
Further, you deleted the PR branch and hence the PR got closed.


No Output for OpenConfig VLAN Yang Model

Hi,
I am getting no output for below YANG model.
https://github.com/openconfig/public/blob/master/release/models/vlan/openconfig-vlan.yang
root@ubuntu-I08:/opt# pyang -p openconfig/release/models/ -f jsonschema openconfig/release/models/vlan/openconfig-vlan.yang
{
"$schema": "http://json-schema.org/draft-04/schema#",
"type": "object",
"description": "Generated by pyang from module openconfig-vlan",
"properties": {},
"title": "openconfig-vlan"
}root@ubuntu-I08:/opt#
Kindly Help...!!!
Thanks,
Sushant

Same for me, I get empty jsonSchema while running it for a NSO package YANG model
[admin@9c1b82c8df20 testJsonSchema]$ pyang  my-nso-package.yang
bb-site-topology.yang:11: warning: imported module "ietf-yang-types" not used
[admin@9c1b82c8df20 testJsonSchema]$ pyang -f jsonschema my-nso-package.yang
{
  "title": "my-nso-package",
  "$schema": "http://json-schema.org/draft-04/schema#",
  "description": "Generated by pyang from module my-nso-package",
  "type": "object",
  "properties": {}
}[admin@9c1b82c8df20 testJsonSchema]$

I figured out that if I remove the augment /ncs:services { then it works.
Is there anyway to make pyang understand and process the augmentation?

Hello @cmoberg, any chances to have augments supported? openconfig models are heavily using these.

Faced the same issue when tried to convert JUNOS snmp yang module (https://github.com/Juniper/yang/blob/master/22.2/22.2R1/junos-es/conf/junos-es-conf-snmp%402022-01-01.yang)
$ pyang -f jsonschema<EMAIL_ADDRESS>{
  "title": "junos-es-conf-snmp",
  "$schema": "http://json-schema.org/draft-04/schema#",
  "description": "Generated by pyang from module junos-es-conf-snmp",
  "type": "object",
  "properties": {}
}

Sorry guys, I honestly haven't had any time to look into this, and I probably won't in the near future. I would of course love for this tiny effort to stay alive along as it provides some value. So if you can, please take a look at the code and see if you can figure it out!


Create converter capabilities RFC

Include new decorator specification for FX + Dynamo converters

Potential arguments include handles_static_tensors, which clarifies whether the converter can handle torch.Tensor input Tensor types, or if it requires ITensors
handles_dynamic_shapes can clarify whether a converter can handle dynamically-shaped input Tensors

Include documentation for a new converter registry and how the converters would be masked based on user inputs.

For example, how we would filter the converters if the user specified a dynamic batch input

See RFC #2065


Dump .wl files

Like .dcm and .dicom, worklist files also have a DICOM structure and should be dumped to text/JSON files. The current workaround is to change the .wl file extension to .dcm.

Thank you for the suggestion. I confirmed *.wl files can be dumped by extension using the samples here, so I think this can be added in the next release.
Meanwhile, you can use dicom.alwaysShowMenu options as described in README.


Added a column with number of training data to the model table

Pull Request
Description:
Added a column to show the number of training data to the models table.
Changes Made:

Added number of training data as an atribute of the model class.
Added column to output the number of training data to terminal

Testing Done:
All clin_ct models on 5 CTs
Reviewer Checklist:

[x] Code is clean and follows the team's coding standards.
[x] The code builds and runs without errors.
[x] The changes have been tested on relevant environments or browsers.
[x] Any relevant dependencies have been updated.

If something breaks, I am taking away your keyboard.


#984 Set up builds and tests to run per changed module

Description
Draft changes for builds/tests to only execute when the changes of a PR affect the module.
Fixes #984
Type of change

[x] Chore (changes to the build process or auxiliary tools)

How Has This Been Tested?

[x] Manual testing
[x] GitHub Actions/CI execution

Checklist:

[x] I have performed a self-review of my own code
[ ] I have added tests that prove it is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] Related issue number is linked in pull request title

No longer required after #985, will either re-open after Maven migration is complete, or make part of PR for #985


ModSpec instances: the "statement" should just be called statement:: instead of description::

Right now we use an encoding as follows:
[requirement]
.Persistent identifiers do not change throughout the life of the register
====
[%metadata]
identifier:: /req/identifiers/ri-identifier-persistence
subject:: persistent identifier
description:: A persistent identifier within the register shall not change
throughout the lifetime of the register.
guidance:: While the register item pointed to by a persistent identifier may change,
the identifier always points to the same register item.
Even after the decommission of a register, the persistent identifiers for
register items within that register remain.
====

However, this is very confusing because in rendering, the description:: field gets rendered as "Statement":

We should support the label of statement:: instead of description:: (keep description as an alias for compatibility purposes).
Document: https://github.com/metanorma/iso-19135/

If you want no differentiation between markup and rendering, you might prefer to use Word. And description is used already instead of statement as a label in classes, by your request. Which means you have already introduced the synonymy.

Thanks @opoudjis for implementing this.


Change recommended wallet based on browser

If user is on mobile (e.g. mobile safari) it makes no sense to reccomend metamask. Instead we should reccomend "Trust Wallet" or "Toshi".

@wanderingstan

Is there a recommended browser for Android?
Here's my stab at the copy/UI for when accessing via iOS:

er maybe the list of browser apps should be on one line:

Now this is also Cipher, which funny enough was just acquired by Coinbase, who also own Toshi.
Maybe we just go with something like:
Please access thi site through
a wallet-enabled browser:
Toshi | Cipher | Trust Wallet

?
I don't have a strong opinion on this. There will certainly be more browsers popping up in the future!

So, something like:

I would also vote for changing the desktop popup to softer wording, from:

to something like

I like it. Send us a pull request @ambertch :)

Get to it Ambert! Thanks.
On Tue, Apr 17, 2018 at 8:52 PM Josh Fraser<EMAIL_ADDRESS>wrote:

I like it. Send us a pull request @ambertch https://github.com/ambertch
:)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/OriginProtocol/demo-dapp/issues/83#issuecomment-382249398,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEuxN2E1iOp1kfPKBWt-p6De7mavzk8Xks5tpriagaJpZM4SI0xU
.

--
Cofounder of Origin http://www.originprotocol.com
Join our Telegram http://t.me/originprotocol.com

Done via #119 ✅


POL-250 Implemented smart query into our list page to know if we have more elements or not

This is to truly know at a given time if the user has more elements in case a filtered search yields 0 elements, to be able to show which empty state to show (you have no policies or no policies match the filter) and for knowing if we want to show the "copy from policy" option in the wizard.

Also adding fetch-mock library (for mocking network requests)

This looks good and the query mocking looks simple.  Merging.


feat: Add sample torch.compile backend for tensorrt aten path

Description

Add backend adapted from previous fx2trt_compiler provided by Dynamo
Currently, the TRTSplitter needs work to fully support the aten path
Additionally, the existing aten pass was reworked to exclude the torch._dynamo.export call, which may be necessary here

Note: The implementation is just a sample and is for use in testing - it has very few customization options (no precision specifications, etc.). Additionally, some of its components require modification for correct usage overall. Specifically, the TRTSplitter may need modifications to work with the aten path effectively
Note: The implementation was adapted from the earlier fx2trt Dynamo backend
Type of change

New feature (non-breaking change which adds functionality)

Checklist:

[ x ] My code follows the style guidelines of this project (You can use the linters)
[ x ] I have performed a self-review of my own code
[ x ] I have commented my code, particularly in hard-to-understand areas and hacks
[ ] I have made corresponding changes to the documentation
[ ] I have added tests to verify my fix or my feature
[ x ] New and existing unit tests pass locally with my changes
[ x ] I have added the relevant labels to my PR in so that relevant reviewers are notified

Proposed UX
from torch_tensorrt.fx.tracer.dispatch_tracer.tensorrt_dynamo_backend import fx2trt, tensorrt_backend
import torch._dynamo as torchdynamo
import torch

torchdynamo.optimize(fx2trt, nopython=True)(model)

##### OR #####

torch.compile(model, backend=tensorrt_backend)

Notes on FakeTensor Issues During Compilation
During compilation, we sometimes encounter errors like this:

Calling  to_numpy() on a FakeTensor

  File "~/TensorRT/py/torch_tensorrt/fx/fx2trt.py", line 328, in call_function
    return converter(self.network, target, args, kwargs, self._cur_node_name)
  File "~/TensorRT/py/torch_tensorrt/fx/converters/aten_ops_converters.py", line 313, in aten_ops_sub
    return add_sub(network, target, kwargs_new, name)
  File "~/TensorRT/py/torch_tensorrt/fx/converters/operator.py", line 876, in add_sub
    return add_binary_elementwise_layer(
  File "~/TensorRT/py/torch_tensorrt/fx/converters/operator.py", line 141, in add_binary_elementwise_layer
    lhs_val = get_trt_tensor(network, lhs_val, f"{name}_lhs", lhs_dtype)
  File "~/TensorRT/py/torch_tensorrt/fx/converters/converter_utils.py", line 237, in get_trt_tensor
    return create_constant(network, input_val, name, dtype)
  File "~/TensorRT/py/torch_tensorrt/fx/converters/converter_utils.py", line 201, in create_constant
    constant = network.add_constant(value.shape, to_numpy(value))
  File "~/TensorRT/py/torch_tensorrt/fx/converters/converter_utils.py", line 497, in to_numpy
    return tensor.cpu().detach().contiguous().numpy()
RuntimeError: .numpy() is not supported for tensor subclasses.

While executing %sub : [#users=1] = call_function[target=torch.ops.aten.sub.Tensor](args = (1.0, %convert_element_type), kwargs = {_itensor_to_tensor_meta: {<tensorrt.tensorrt.ITensor object at 0x7f7fd13ce8f0>: None}})

Providing FakeTensors as input to Graph Modules

  File "/usr/local/lib/python3.8/dist-packages/torch/utils/_pytree.py", line 266, in tree_map_only
    return tree_map(map_only(ty)(fn), pytree)
  File "/usr/local/lib/python3.8/dist-packages/torch/utils/_pytree.py", line 196, in tree_map
    return tree_unflatten([fn(i) for i in flat_args], spec)
  File "/usr/local/lib/python3.8/dist-packages/torch/utils/_pytree.py", line 196, in <listcomp>
    return tree_unflatten([fn(i) for i in flat_args], spec)
  File "/usr/local/lib/python3.8/dist-packages/torch/utils/_pytree.py", line 247, in inner
    return f(x)
  File "/usr/local/lib/python3.8/dist-packages/torch/_subclasses/fake_tensor.py", line 1282, in validate
    raise Exception(
Exception: Please convert all Tensors to FakeTensors first or instantiate FakeTensorMode with 'allow_non_fake_inputs'. Found in aten.embedding.default(*(Parameter containing:
tensor([[-0.0102, -0.0615, -0.0265,  ..., -0.0199, -0.0372, -0.0098],
        [-0.0117, -0.0600, -0.0323,  ..., -0.0168, -0.0401, -0.0107],
        [-0.0198, -0.0627, -0.0326,  ..., -0.0165, -0.0420, -0.0032],
        ...,
        [-0.0218, -0.0556, -0.0135,  ..., -0.0043, -0.0151, -0.0249],
        [-0.0462, -0.0565, -0.0019,  ...,  0.0157, -0.0139, -0.0095],
        [ 0.0015, -0.0821, -0.0160,  ..., -0.0081, -0.0475,  0.0753]],
       device='cuda:0', requires_grad=True), FakeTensor(FakeTensor(..., device='meta', size=(1, 14), dtype=torch.int32), cuda:0), 0), **{})

Solutions
There are two key functions/utilities which can help in these cases:
1. from torch._dynamo.backends.common import fake_tensor_unsupported:
@fake_tensor_unsupported
def fx2trt_compiler

2. from torch._subclasses import FakeTensorMode
with FakeTensorMode(allow_non_fake_inputs=True):
    ...

It remains to be seen whether these will be able to resolve the issues described above, or if they address the root cause of those issues. Additionally, we should consider whether it is reasonable to call torch._dynamo.export from within a call to torch.compile or torch._dynamo.optimze.

My understanding for faketensor is that it is an empty tensor and only used for shape and dtype induction. It is supposed to be used in aten tracer to help shape induction. Why do we need to use it directly?

@frank-wei The reason I think using @fake_tensor_unsupported is necessary here is that if we don't use it, then the frequent calls to the create_constant function in converter/converter_utils.py will fail since they attempt to instantiate a constant tensor, but the FakeTensorMode disallows creation of any non-fake tensors within its context. This leads to the error shown in the first dropdown here: https://github.com/pytorch/TensorRT/pull/1751#issuecomment-1483475796.


Relative paths do not work?

First of all:
Great idea! And that kinda solves my issue, however I've found that relative paths do not work.
Example:
  "preferencesInheritance.targets": [
    {
      "sources": [
        ".vscode/settings.shared.json"
      ],
      "output": ".vscode/settings.json"
    }
  ],

Returns:
2024-01-11 17:48:59.009 [error] [Extension Host] Failed to merge settings
2024-01-11 17:48:59.009 [error] [Extension Host] CodeExpectedError: cannot open file:///.vscode/settings.shared.json. Detail: Unable to read file '/.vscode/settings.shared.json' (Error: Unable to resolve nonexistent file '/.vscode/settings.shared.json')
    at r.$tryOpenDocument (vscode-file://vscode-app/usr/share/codium/resources/app/out/vs/workbench/workbench.desktop.main.js:2169:87835)

Updated to 1.1.0, relative paths still do not work...
  "preferencesInheritance.notifyOnNewWorkspace": false,
  "preferencesInheritance.targets": [
    {
      "sources": [
        ".vscode/settings.core.json",
        ".vscode/settings.python.json",
        ".vscode/settings.shell.json",
        ".vscode/settings.ui.json"
      ],
      "output": ".vscode/settings.json"
    }
  ]

Looks like now it works with:
    {
      "sources": [
        "${workspaceFolder}/.vscode/settings.core.json",
        "${workspaceFolder}/.vscode/settings.python.json",
        "${workspaceFolder}/.vscode/settings.shell.json",
        "${workspaceFolder}/.vscode/settings.ui.json"
      ],
      "output": "${workspaceFolder}/.vscode/settings.json"
    }

So, yeah, issue solved :)


Please add version metadata to DLL and EXE resource sections

Is your feature request related to a problem? Please describe.
It can be hard to tell where user's DLLs came from, and if they're using an outdated version. Resource metadata shows up in right click -> properties -> details and minidumps (e.g. via lm -v in windbg), and makes it easier to confirm this information.
Describe the solution you'd like
Please create version resource metadata.
For example, all OpenKneeboard .exe and .dll files have embedded data generated from https://github.com/fredemmott/OpenKneeboard/blob/b0ddd48b92a5337699c38a443d5335c017818b32/src/version.rc.in
Screenshots
I know opencomposite is a separate project, but using just for illustration: it's clear that this openvr_api.dll is the usual SteamvR one:
start             end                 module name
00000001`80000000 00000001`80096000   openvr_api   (export symbols)       openvr_api.dll
    Loaded symbol image file: openvr_api.dll
    Mapped memory image file: C:\Program Files\Eagle Dynamics\DCS World OpenBeta\bin\openvr_api.dll
    Image path: C:\Program Files\Eagle Dynamics\DCS World OpenBeta\bin\openvr_api.dll
    Image name: openvr_api.dll
    Browse all global symbols  functions  data
    Timestamp:        Wed Jan 30 13:47:49 2019 (5C51FF65)
    CheckSum:         00096F8C
    ImageSize:        00096000
    File version:     <IP_ADDRESS>
    Product version:  <IP_ADDRESS>
    File flags:       0 (Mask 17)
    File OS:          4 Unknown Win32
    File type:        1.0 App
    File date:        00000000.00000000
    Translations:     0409.04b0
    Information from resource tables:
        CompanyName:      Valve
        ProductName:      OpenVR
        InternalName:     openvr_api
        OriginalFilename: openvr_api.dll
        ProductVersion:   1, 0, 10, 0
        FileVersion:      1, 0, 10, 0
        FileDescription:  OpenVR Loader DLL
        LegalCopyright:   Copyright (C) Valve

... however, I don't know what this is (from an OpenKneeboard's user's crash report):
start             end                 module name
00007ff9`57d70000 00007ff9`57ecb000   openvr_api T (no symbols)
    Loaded symbol image file: openvr_api.dll
    Image path: C:\Program Files (x86)\Steam\steamapps\common\DCSWorld\bin\openvr_api.dll
    Image name: openvr_api.dll
    Browse all global symbols  functions  data
    Timestamp:        Fri Mar 25 14:20:00 2022 (623E15E0)
    CheckSum:         00000000
    ImageSize:        0015B000
    Translations:     0000.04b0 0000.04e4 0409.04b0 0409.04e4
    Information from resource tables:

I'm guessing it's OpenComposite-ACC v0.6.2 from the timestamp and lack of metadata - but this is just a guess. Version metadata would make this clear immediately, with no need to look at version dates etc.
As it shows up in Properties -> Details via windows explorer, it can also be helpful for users to confirm which version they've actually installed, e.g.:

Did you check our release binaries (official ones)? Because they have that info already.

(Speaking for OpenXR Toolkit only)

Whoops, sorry, only for OpenComposite-ACC not OpenXR-Toolkit D: Thanks!

ah, no, I didn't as 7zip's not able to extract the msi and I don't want to install openxr-toolkit - just look at the binaries :(


Put the bucket name in the host name for Amazon S3.

This is a patch Least Authority Enterprises has been carrying in a fork of txAWS.  Unfortunately there don't seem to be any tests or documentation for the behavior.  Also, I can't find any more details about what error cases this helps avoid.  Also, Least Authority Enterprises hasn't actually used this functionality in a long time and things seem to be working okay.  So possibly this change isn't worth incorporating into txAWS proper.
If someone does figure out that this is a good idea, though, at least here's some code to start with.

Lacking better understanding of why this is a good idea, closing with no intention to invest further effort.

It's possible the "Challenges with Non–DNS-Compliant Bucket Names" section of http://docs.aws.amazon.com/AmazonS3/latest/dev/BucketRestrictions.html points towards the problems that putting bucket names into the host name can cause - but that's the opposite of a justification for this change.

I think the main reason to use the bucket name as the hostname is to query the right S3 regional endpoint without knowing what it is in advance; if you query the wrong endpoint (eg. US endpoint for an EU bucket), you will receive an HTTP 301 response without a Location: header field (deliberately so).
However…the certificate S3 uses for HTTPS is only valid for "*.s3.amazonaws.com" which means that if your bucket name is DNS style (say, "twistedmatrix.com"), you will attempt make an HTTPS connection to "twistedmatrix.com.s3.amazonaws.com" and the certificate will not be valid according to the normal hostname verification rules.

Ah, according to http://docs.aws.amazon.com/AmazonS3/latest/dev/VirtualHosting.html you receive a 307 with Location, so my information is either incorrect or out of date. However, for request types other than GET, following redirects will result in transferring a lot of data needlessly in the absence of 100-continue support, as noted at http://docs.aws.amazon.com/AmazonS3/latest/dev/RESTRedirect.html


fix(node): node build

i have no idea why this fixes the build.

closes https://github.com/lancedb/lancedb/issues/1394


Feat : Translation improvement

Hello, I'm a French-speaking developer.
I'm about to start using this system and I noticed that most of the selects in the item sheets weren't translated, so I took a look at it.
To simplify things, I used a handlebars directly integrated into Foundry to automatically generate the list of select options I could do in a simple way.
https://foundryvtt.com/api/classes/client.HandlebarsHelpers.html
To make everything fully translatable, the missing CONFIGs would have to be created.
I also added lines to two translation files - I don't know German and Italian to do it for these languages.
What I've added translates the items/actors in these lists:

You'll notice that I've integrated them twice, in two different ways. That's because in V11, the translation system for these lists has changed, so it's just a matter of maintaining V10 & V11 compatibility. It has no particular impact to have both in the translation file.

Looks good from my side. Thank you for your contribution!


Initial test experiments for new python API

There are two examples (tensorflow mnist and pytorch kvasir unet) for new interactive API in single file mode (like Jupiter notebook) and two same examples with structured code.
Also, there are some fixes (use tmp dir outside current project dir to create workspace archive, additional ignore patterns and pattern 'data' was excluded, because it exclude 'data' directory for pytorch example - probably it should be discussed).

Jenkins please retry a build


FIX: don't require generated fields on create

The intellisense is generally great, but isn't quite perfect with a generated field. Right now the only options are
{
   id: {
      type: 'string',
      required: true,
      generate: true
   }
}

Which will insist on id being present when creating an item. Or
{
   id: {
      type: 'string',
      generate: true
   }
}

Which will have a type of string|undefined, making typing ugly throughout the object lifecycle.
This PR extracts generated fields and makes them optional when creating an item, so that typing works as expected.

I was about to create an issue about this. Thank you.

Thank you. Looks good. We'll test it out.

We're having some trouble with this change.
By mistake, I reverted it in the latest release by removing the CreatedProperties in the create() definition in Model.d.ts.
However, when reverting, TS does not catch any missing required fields in create().
If I remove CreatedProperties, then missing required fields are flagged, but missing generated fields are also flagged.
Can you check this for me?   I'm using TSC 4.3.2.

The fix may be changing in Model.d.ts:
type CreateProperties<T> = Omit<T, keyof Generated<ExtractModel<T>>> | Generated<ExtractModel<T>>

to
type CreateProperties<T> = Omit<T, Generated<ExtractModel<T>>>

And create() to:
create(properties: CreateProperties<T>, params?: OneParams): Promise<T>;

Yes I can confirm. What's weird is that in WebStorm's intellisense it does have the correct typing behavior, it's only in the compiler that it isn't caught.
Can you check
type CreateProperties<T> = Omit<T, keyof Generated<ExtractModel<T>>> & Generated<ExtractModel<T>> ?
My original thought was to let the generated keys be optional, which I believe that your proposed change wouldn't allow? Changing the | to & appears to be working for me

Generated keys can be optional with:
type CreateProperties<T> = Omit<T, Generated<ExtractModel<T>>>

Not sure why, but it works.
Also works if you put required: true, generated: true. i.e. then it is required to be provided.

Oh awesome! Such are the mysteries of Typescript typing. Thanks for noticing this

I'd recommend you also check things with VS code for their intellisense.

Hmm. I was working with a co-worker and tried the type CreateProperties<T> = Omit<T, Generated<ExtractModel<T>>> line but then it started throwing errors about id being required again.
My original fix type CreateProperties<T> = Omit<T, keyof Generated<ExtractModel<T>>> & Generated<ExtractModel<T>> was what worked. Can you have another look?

Can you describe your environment. I'm testing with TSC and VS Code.
Your fix did not trigger on missing required properties whereas mine did with VS Code and TSC.
Could this be a WebStorm thing?

Sorry I don't mean "my original" as in what was in the PR. I meant what I proposed above once you pointed out the issue with the PR.
Here is type CreateProperties<T> = Omit<T, keyof Generated<ExtractModel<T>>> & Generated<ExtractModel<T>> (notice the change from | in the original PR to &)

You can see it catches the missing communityId in both Webstorm and TSC (v 4.4.3)
Here is type CreateProperties<T> = Omit<T, Generated<ExtractModel<T>>> as you suggested

It is now saying that id is required but missing

Is this with VS Code or Web Storm?

Well the screenshot is webstorm plus tax on my console. My coworker was
using visual studio and got the same results
On Fri, Aug 5, 2022, 11:18 PM mobsense @.***> wrote:

Is this with VS Code or Web Storm?
—
Reply to this email directly, view it on GitHub
https://github.com/sensedeep/dynamodb-onetable/pull/360#issuecomment-1206334666,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS4OOTHTI3VI6KG7TKV4LATVXT2AFANCNFSM53NXICOQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

It is working for me here. So versions please.


koordlet: pod/container PSI/CPI metircs cache to tsdb

Ⅰ. Describe what this PR does

refactor: koordlet: pod/container PSI/CPI metircs cache to tsdb
see #1241 for full description.

Ⅱ. Does this pull request fix one issue?

fix #1329
Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[x] I have written necessary docs and comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in make test

/lgtm
/approve

/assign @fillzpp


utility hook to trim data

It would be nice to have a hook that deeply inspects hook.data and removes whitespace. Node already has the String.trim function, so this would be a fairly straightforward hook to implement.
e.g, before:
hook.data = {
  foo: '    not trimmed    ',
  bar: {
    baz: '   needs  trimming\t\n   '
  },
  bam: 'trimmed'
};

and after:
hook.data = {
  foo: 'not trimmed',
  bar: {
    baz: 'needs  trimming'
  },
  bam: 'trimmed'
}

This may be part of the forthcoming serialize hook.

The traverse hook, now waiting for review for Auk, will handle this and much more.
https://github.com/feathersjs/feathers-hooks-common/pull/73


[elasticsearch] Integration tests started taking too long

I've noticed that ES integration tests started taking over 15min, making it the longest running matrix step. Not sure what's responsible for that, but one issue is that the tests seems to be running three times, e.g. TestElasticsearchStorage/GetDependencies in https://travis-ci.org/jaegertracing/jaeger/jobs/582349875
cc @pavolloffay

The elasticsearch test now runs 3 times: ES 5.x, 6.x and 7.x. It was more efficient to run all 3 in the same job as opposed to spin up 3 due to limited number of jobs.
We can reconsider and run those tests in 3 separate jobs one per ES version.

Maybe there is a way to parallelize some of steps?
Might be worth running as 3 separate Travis jobs, since each would be around 5-6min, still not the shortest among all jobs.

Before splitting to 3 jobs I would wait for confirmation from CNCF that we are granted more jobs.
The single ES integration test takes about 7 minutes https://travis-ci.org/jaegertracing/jaeger/jobs/566691615.

Are more jobs required for this, or desirable? I understood that if we trigger 15 jobs, 5 might get queued and will be promptly executed once an worker is available.
What we could attempt to do is to better group the tests in stages, so that the quick jobs are executed first, providing quick feedback in case something basic is broken. Like:
stages:
  - compile
  - test
  - e2e
  - deploy

And then:
jobs:
  include:
    - stage: compile
      name: "Proto Gen tests"
      script: make protogen-tests

    - stage: e2e
      name: "Elasticsearch 5.x"
      script: make e2e-es-5

Once the storage-related jobs start, the quicker will have finished already and probably won't be queued at all.
That obviously doesn't solve the need for more executors, but it will possibly alleviate a bit.


Importing city on Windows

Thanks for improving the city importer on Windows as discussed in #119. Unfortunately, I'm still getting errors when importing a city.
Using latest master set RUST_BACKTRACE=1 && cargo run --release --manifest-path importer/Cargo.toml -- --raw --map --city=berlin throws:
Running `target\release\importer.exe --raw --map --city=berlin`
- Working on all berlin maps
import map data...
- Missing data/input/berlin/osm/berlin-latest.osm.pbf, so downloading http://download.geofabrik.de/europe/germany/berlin-latest.osm.pbf
- Running "curl" "--fail" "-L" "-o" "tmp_output" "http://download.geofabrik.de/europe/germany/berlin-latest.osm.pbf"
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                              Dload  Upload   Total   Spent    Left  Speed
100 57.2M  100 57.2M    0     0  11.4M      0  0:00:05  0:00:05 --:--:-- 11.3M
- Missing data/input/berlin/planning_areas.bin, so downloading https://tsb-opendata.s3.eu-central-1.amazonaws.com/lor_planungsgraeume/lor_planungsraeume.kml
- Running "curl" "--fail" "-L" "-o" "tmp_output" "https://tsb-opendata.s3.eu-central-1.amazonaws.com/lor_planungsgraeume/lor_planungsraeume.kml"
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                              Dload  Upload   Total   Spent    Left  Speed
100 2982k  100 2982k    0     0  2982k      0  0:00:01 --:--:--  0:00:01 4341k
- Extracting KML data
extracting shapes from KML...
Opening tmp_output
Read tmp_output (2)... 0.1160s
Read tmp_output (2)... 0.1163s
Got 43 shapes from tmp_output and skipped 406 shapes
extracting shapes from KML... plus 0.0062s
extracting shapes from KML took 0.1302s

- extracting shapes from KML took 0.1302s
- Reading tmp_output (2 MB)... 0.1241s
- ... plus 0.0062s

- extracting shapes from KML took 0.1302s
Wrote data/input/berlin/planning_areas.bin
- Missing data/input/berlin/EWR201812E_Matrix.csv, so downloading https://www.statistik-berlin-brandenburg.de/opendata/EWR201812E_Matrix.csv
- Running "curl" "--fail" "-L" "-o" "tmp_output" "https://www.statistik-berlin-brandenburg.de/opendata/EWR201812E_Matrix.csv"
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                              Dload  Upload   Total   Spent    Left  Speed
100 89337  100 89337    0     0  89337      0  0:00:01 --:--:--  0:00:01  159k
Read data/input/berlin/planning_areas.bin (0)... 0.0028s0.0001s
Wrote data/input/berlin/planning_areas.bin
- Clipping data/input/berlin/osm/berlin-latest.osm.pbf to data/input/berlin/polygons/berlin_center.poly
- Running "osmconvert" "data/input/berlin/osm/berlin-latest.osm.pbf" "-B=data/input/berlin/polygons/berlin_center.poly" "--complete-ways" "-o=data/input/berlin/osm/berlin_center.osm"
thread 'main' panicked at 'Failed to run "osmconvert" "data/input/berlin/osm/berlin-latest.osm.pbf" "-B=data/input/berlin/polygons/berlin_center.poly" "--complete-ways" "-o=data/input/berlin/osm/berlin_center.osm": Os { code: 2, kind: NotFound, message: "Das System kann die angegebene Datei nicht finden." }', importer\src\utils.rs:121:13
stack backtrace:
0: backtrace::backtrace::trace_unsynchronized
          at C:\Users\VssAdministrator\.cargo\registry\src\github.com-1ecc6299db9ec823\backtrace-0.3.46\src\backtrace\mod.rs:66
1: std::sys_common::backtrace::_print_fmt
          at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2\/src\libstd\sys_common\backtrace.rs:78
2: std::sys_common::backtrace::_print::{{impl}}::fmt
          at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2\/src\libstd\sys_common\backtrace.rs:59
3: core::fmt::write
          at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2\/src\libcore\fmt\mod.rs:1076
4: std::io::Write::write_fmt<std::sys::windows::stdio::Stderr>
          at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2\/src\libstd\io\mod.rs:1537
5: std::sys_common::backtrace::_print
          at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2\/src\libstd\sys_common\backtrace.rs:62
6: std::sys_common::backtrace::print
          at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2\/src\libstd\sys_common\backtrace.rs:49
7: std::panicking::default_hook::{{closure}}
          at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2\/src\libstd\panicking.rs:198
8: std::panicking::default_hook
          at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2\/src\libstd\panicking.rs:218
9: std::panicking::rust_panic_with_hook
          at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2\/src\libstd\panicking.rs:486
10: std::panicking::begin_panic_handler
          at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2\/src\libstd\panicking.rs:388
11: std::panicking::begin_panic_fmt
          at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2\/src\libstd\panicking.rs:342
12: importer::utils::osmconvert
13: importer::utils::osmconvert
14: importer::berlin::osm_to_raw
15: alloc::collections::btree::search::search_tree
16: map_model::objects::intersection::_IMPL_SERIALIZE_FOR_IntersectionType::<impl serde::ser::Serialize for map_model::objects::intersection::IntersectionType>::serialize
17: std::rt::lang_start_internal::{{closure}}
          at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2\/src\libstd\rt.rs:52
18: std::panicking::try::do_call
          at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2\/src\libstd\panicking.rs:297
19: std::panicking::try
          at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2\/src\libstd\panicking.rs:274
20: std::panic::catch_unwind
          at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2\/src\libstd\panic.rs:394
21: std::rt::lang_start_internal
          at /rustc/5c1f21c3b82297671ad3ae1e8c942d2ca92e84f2\/src\libstd\rt.rs:51
22: main
23: invoke_main
          at d:\A01\_work\6\s\src\vctools\crt\vcstartup\src\startup\exe_common.inl:78
24: __scrt_common_main_seh
          at d:\A01\_work\6\s\src\vctools\crt\vcstartup\src\startup\exe_common.inl:288
25: BaseThreadInitThunk
26: RtlUserThreadStart
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
import map data took 7.4444s

- import map data took 7.4444s

- import map data took 7.4444s
error: process didn't exit successfully: `target\release\importer.exe --raw --map --city=berlin` (exit code: 101)

Thanks.
If using Windows becomes too much work to support I could also look into switching to Linux or WSL for our purposes.

Per https://github.com/dabreegster/abstreet/blob/master/docs/dev.md#building-map-data, could you try installing osmconvert?

Oh. Yes, I didn't follow that guide yet. Thanks for mentioning it.
The Windows download for osmconvert is an .exe file. I'm not sure where to put it or what to do with it.

Windows users can start osmconvert by double-clicking on the executable.
Users of all platforms can start osmconvert from the command-line.

Maybe on Windows command-line mode is not available?

I'm not familiar with the Windows equivalent of the Linux $PATH, which is a list of folders to search for commands. I bet if you put osmconvert.exe in the same directory where you're running the importer command, it'll find it. But I'm not sure what best practice is.

#296 made failures for missing tools happen upfront, and also created a way to override the location of osmconvert.exe


expose a signupWithRedirect function

Describe the problem you'd like to have solved
there is no a similar functionality for loginWithRedirect that takes you to the Sign Up page instead. I want to have on my website 2 buttons. Sign in and Sign Up.
Describe the ideal solution
Expose a function called signupWithRedirect that takes you to the sign up page and redirects you after the user is created.

A clear and concise description of any alternatives you've considered or any work-arounds that are currently in place.

Additional context

Hi @cpenarrieta ,
You can direct a user to the signUp page by passing a custom argument to the loginWithRedirect function. You can read this here: https://github.com/auth0/auth0-react/issues/59#issuecomment-654333541.

@frederikprijck perfect, thanks for the help 👍


Custom font  elements are stripped out by Inliner v2

I need to include external font faces for custom fonts (yes, I know they won't work in Outlook and other clients, but with font-family fallback it won't hurt) - but the Inliner v2 is stripping them out. I can't even include the @font-face declarations in-page, even inside an @media block.

Just use configuration:
in Your gulpfile.babel.js
`function inliner(css) {
var css = fs.readFileSync(css).toString();
var mqCss = siphon(css);
var pipe = lazypipe()
.pipe($.inlineCss, {
applyStyleTags: false,
removeStyleTags: false,
removeLinkTags: false
})
.pipe($.injectString.replace, '', <style>${mqCss}</style>)
.pipe($.htmlmin, {
collapseWhitespace: true,
minifyCSS: true
});
return pipe();
}`

@MichalKrakow Thanks for jumping in! Can you describe what these additional params are helping with in this situation?
If it's a universal solution, would be awesome if you submit a PR!

well defaults for: removeStyleTags and removeLinkTags are true so setting them to false will prevent inliner from removing Link, and Style from Your parsed html while also inline parts that can be embedded directly in html nodes. I figured that's what You're trying to achieve.

Sounds good! Just added this change - https://github.com/zurb/foundation-emails-template/commit/57bd9e0bbc123a2ad3ab41c6b3caaa5b028b9c06


Can't connect to emulator with Mongo DB API - SSL error

Repro steps:

Right click "Attach Database Accounts"
Select "MongoDB"

Action: cosmosDB.attachEmulator
Error type: Error
Error Message: failed to connect to server [localhost:10255] on first connect [MongoError: connect ECONNREFUSED <IP_ADDRESS>:10255]
Version: 0.10.2
OS: win32
Product: Visual Studio Code
Product Version: 1.35.1
Language: en

Call Stack
Pool.eventHandler mongodb-core/server.js:336:35
Pool.emit events.js:182:13
Connection.<anonymous> mongodb-core/pool.js:280:12
Object.onceWrapper events.js:273:13
Connection.emit events.js:182:13
TLSSocket.errorHandler mongodb-core/connection.js:189:49
Object.onceWrapper events.js:273:13
TLSSocket.emit events.js:182:13
TLSSocket._emitTLSError _tls_wrap.js:604:10
Socket.TLSSocket._init.wrap.on _tls_wrap.js:547:36
Socket.emit events.js:182:13
emitErrorNT destroy.js:82:8
emitErrorAndCloseNT destroy.js:50:3
process._tickCallback next_tick.js:63:19

Do you have a local instance of the mongo server running?
Can you provide more specific repro steps?

similar error when connecting to CosmosDB from behind an enterprise firewall / proxy.
It would be nice to have an option to add a ssh tunnel.

@prashanthcorp I can connect to SQL emulator, but not mongo.

Please see https://github.com/microsoft/vscode-cosmosdb/wiki/Debugging-Connection-Errors. The emulator changed so that you have to explicitly enable the MongoDB port when starting.  Will update our error message to point to this wiki.


Recover from internet down

Looking at using ubus to handle event tracking on the modem interface network.interfaces.modem_0001_4. I can check the status using:
ubus call network.interface.modem_0001_4 status

The output is a JSON object
{
        "up": true,
        "pending": false,
        "available": true,
        "autostart": true,
        "dynamic": true,
        "uptime": 26567,
        "l3_device": "rmnet_mhi0",
        "proto": "dhcp",
        "device": "rmnet_mhi0",
        "metric": 40,
        "dns_metric": 0,
        "delegation": true,
        "ipv4-address": [
                {
                        "address": "<IP_ADDRESS>",
                        "mask": 27
                }
        ],
        "ipv6-address": [

        ],
        "ipv6-prefix": [

        ],
        "ipv6-prefix-assignment": [

        ],
        "route": [
                {
                        "target": "<IP_ADDRESS>",
                        "mask": 0,
                        "nexthop": "<IP_ADDRESS>",
                        "source": "<IP_ADDRESS>/32"
                }
        ],
        "dns-server": [
                "<IP_ADDRESS>"
        ],
        "dns-search": [

        ],
        "neighbors": [

        ],
        "inactive": {
                "ipv4-address": [

                ],
                "ipv6-address": [

                ],
                "route": [

                ],
                "dns-server": [

                ],
                "dns-search": [

                ],
                "neighbors": [

                ]
        },
        "data": {
                "hostname": "gw",
                "leasetime": 7200
        }
}

Can possibly use ubus to restart the network interace using ubus network restart {}
Reference:

https://openwrt.org/docs/guide-developer/ubus/network


Federating connected services

If projects are connected by path (see issue #28) they could have common services they (could) share.
From internal discussion:

my thought here is simply better container naming system, then let the naming guide federation, i.e. check for a container by name xyz and link if it exists, else create - it's simple but it's conservative and less to manage, then layer that with better output so if you have multiple services that aren't named the same (and not federating) it's very obvious 1. that it's happening and 2. how to fix it. I was thinking about it and I don't want to be to assumptive, and here's my case: we use postgres:8 to emulate redshift, but what if we also want to have postgres running as well (postgres:latest), assumptive federating there could/would break shit whereas simply naming services as you naturally would (redshift v. postgres) tells a better story of how to connect things

Included in #30

Included in PR for 3.0 release (#33)


The origin "null" was not whitelisted by the operator of this proxy.

Using the demo html, I just change your URL to http://localhost:8080, and this is the response I get for all queries:
GET google.com
403 Forbidden
The origin "null" was not whitelisted by the operator of this proxy.
What do I need to change so my server can work?

Here's an attempt using 'fetch', can't get this to work, or even display the reason for the 403 either. Nothing is returned by the server console to indicate anything... That would be usefull too, some sort of failed response error log, like WHY the request was denied in the server logs...
var request = new Request('http://localhost:4040/http://google.com', {
method: 'GET',
mode: 'basic',
redirect: 'follow',
headers: new Headers({
'origin': 'http://localhost/',
'x-requested-with':'myApp'
})
});
try {
  fetch(request).then(function(resp) {
    if(resp.ok) {
      return resp.blob();
    }
    console.debug(resp)
    throw new Error("Couldn't fetch this image! ");
  }).then(function(respBlob) {
    console.debug(respBlob);
    // var objectURL = URL.createObjectURL(respBlob);
    // myImage.src = objectURL;
  }).catch(function(error) {
    console.log('detchImage saw error: ');
    console.debug(error);
  });
} catch (err) {
  console.log("caught error : " + err);
}

}

I needed to add something like 'http://localhost:34567' to my whitelist, or remove the whitelist... Seems ok now.


get assets and tags in Python/Vyper track and lessons

From this convo https://discord.com/channels/883478451850473483/1192561848667603064/1207437106356690974
if someone wants to finish the database half of the python banners, ive got one of uma's assets uploaded here: https://github.com/Developer-DAO/academy-turbo/pull/178
track and lesson need tags too, if anyones mucking around in the db scripts and wants to knock that out too. Python, Vyper, Ape are a good starting point

can close with this PR: https://github.com/Developer-DAO/academy-turbo/pull/178


Use normalised chr1:chrY values rather than raw chrY counts to set male/female thresholds

Using normalised chr1:chrY values will be more robust to variations in total read count between samples.

Closed by https://github.com/eastgenomics/eggd_sex_check/releases/tag/v1.1.0


FSharpSource.targets was imported twice

==> led to warning

Yeah ... I have a branch with this change too, only I removed other one.
Thanks for taking care of this
Kevin


memory leak cgo

use " valgrind --leak-check=full --show-leak-kinds=all  ./app",runing one day and stop valgrind. i found memory leak in cgo.

i'm app used cgo. version 1.12

This is exactly what it will look like if your application calls C.malloc or C.CString and fails to free the resulting memory.  Without more information I don't see any reason to blame cgo here.

@ianlancetaylor  thanks，i check use C.malloc or C.CString.