Changed type of ticket field ID from Integer to Long.

Fix for an ID value overflow problem.

@ngyewch thanks!

@johnou Thanks!


Evaluation cache abstraction

Added a caching abstraction to help reduce the impact of a bottleneck in evaluation work distribution. Further work is necessary to build upon the abstraction to hold and distribute common data between processes.

Provides tools for issue #434

Blocking off tomorrow to review this.


Cordova Support

Can you please add suuport for cordova like in Framework7
there is a Framework7 - Vue - Webpack Cordova Template
https://github.com/centrual/cordova-template-framework7-vue-webpack
It offers many hooks which help a lot in hybrid app building and fixing

@ahmedkassem - Realize cordova integration is on my list! :-)

Will be released with 1.1


Create Spin Dapr/k8s Tutorial

Reference the following examples to write Spin k8s documentation (structure and content tba)
https://github.com/radu-matei/spin-dapr-samples/tree/main
https://github.com/mikkelhegn/spin-k8s-scenarios/tree/main

Think about the learning paths and scenarios.
Figure out how we answer those questions and then serve our users.
Think about:

What problem does x solve; what is the critical path to achieve the problem.


Unable to build libtashkeel

New ubuntu machine 22.04
cloned repo, installed rust, tried to build libtashkeel by "cargo build --release", got this error. What could be the reason?
Compiling libtashkeel_base v1.5.0 (/root/libtashkeel/libtashkeel_base)
error[E0277]: the trait bound ort::Value: From<ArrayBase<OwnedRepr<i64>, Dim<[usize; 2]>>> is not satisfied
--> libtashkeel_base/src/inference_engine/ort.rs:26:22
|
26 |           let inputs = ort::inputs![
|  _____________^
27 | |             input_ids,
28 | |             diac_ids,
29 | |             input_length,
30 | |         ]?;
| |^ the trait From<ArrayBase<OwnedRepr<i64>, Dim<[usize; 2]>>> is not implemented for ort::Value, which is required by ort::Value: TryFrom<ArrayBase<OwnedRepr<i64>, Dim<[usize; 2]>>>
|
= help: the following other types implement trait From<T>:
ort::Value implements From<ort::Value<DynTensorValueType>>
ort::Value implements From<ort::Value<TensorValueType<T>>>
= note: required for ArrayBase<OwnedRepr<i64>, Dim<[usize; 2]>> to implement Into<ort::Value>
= note: required for ort::Value to implement TryFrom<ArrayBase<OwnedRepr<i64>, Dim<[usize; 2]>>>
= note: this error originates in the macro ort::inputs (in Nightly builds, run with -Z macro-backtrace for more info)
error[E0277]: the trait bound ort::Value: From<ArrayBase<OwnedRepr<i64>, Dim<[usize; 1]>>> is not satisfied
--> libtashkeel_base/src/inference_engine/ort.rs:26:22
|
26 |           let inputs = ort::inputs![
|  _____________^
27 | |             input_ids,
28 | |             diac_ids,
29 | |             input_length,
30 | |         ]?;
| |^ the trait From<ArrayBase<OwnedRepr<i64>, Dim<[usize; 1]>>> is not implemented for ort::Value, which is required by ort::Value: TryFrom<ArrayBase<OwnedRepr<i64>, Dim<[usize; 1]>>>
|
= help: the following other types implement trait From<T>:
ort::Value implements From<ort::Value<DynTensorValueType>>
ort::Value implements From<ort::Value<TensorValueType<T>>>
= note: required for ArrayBase<OwnedRepr<i64>, Dim<[usize; 1]>> to implement Into<ort::Value>
= note: required for ort::Value to implement TryFrom<ArrayBase<OwnedRepr<i64>, Dim<[usize; 1]>>>
= note: this error originates in the macro ort::inputs (in Nightly builds, run with -Z macro-backtrace for more info)
For more information about this error, try rustc --explain E0277.
error: could not compile libtashkeel_base (lib) due to 2 previous errors
warning: build failed, waiting for other jobs to finish...

@Andrey847 thanks for reporting this. Please try building with the latest commit.

Thanks! this commit helped.
After the updating the source, I' ve got this error:
= note: /usr/bin/ld: cannot find -lstdc++: No such file or directory
Which was fixed by these commands:

sudo apt-get update
sudo apt-get install build-essential
sudo apt-get install aptitude
sudo apt-get install libstdc++6

@Andrey847 thanks. I'm sure this would be a good reference for others.


Refactor BaseProblem.second_q_ops()

Summary
This PR is another self-container step part of #701.
It refactors the BaseProblem.second_q_ops() method to produce a tuple, with the first element being the main operator, and the second element a dictionary of auxiliary operators. This matches the input expected by Terra algorithms and avoids the common need of extracting the main operator from one larger dictionary of operators.
Details and comments
This PR also enables settings.dict_aux_operators by default and deprecates ListOrDict since Qiskit Nature will always be producing dictionaries of auxiliary operators in the new stack/workflow.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

69 of 81 (85.19%)  changed or added relevant lines in 23 files are covered.
13 unchanged lines in 5 files lost coverage.
Overall coverage increased (+0.06%) to 85.78%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_nature/list_or_dict.py
1
2
50.0%

qiskit_nature/second_q/mappers/qubit_converter.py
23
26
88.46%

qiskit_nature/second_q/algorithms/excited_states_solvers/excited_states_eigensolver.py
3
7
42.86%

qiskit_nature/second_q/algorithms/ground_state_solvers/ground_state_eigensolver.py
3
7
42.86%

Files with Coverage Reduction
New Missed Lines
%

qiskit_nature/second_q/algorithms/excited_states_solvers/excited_states_eigensolver.py
1
78.72%

qiskit_nature/second_q/algorithms/excited_states_solvers/qeom.py
1
80.97%

qiskit_nature/second_q/algorithms/ground_state_solvers/ground_state_eigensolver.py
1
81.61%

qiskit_nature/second_q/problems/base_problem.py
1
85.71%

qiskit_nature/list_or_dict.py
9
45.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
17059

Relevant Lines:
19887

💛  - Coveralls


Option to disable template caching.

When modifying Handlebars templates it's frustrating to reload the page after making changes. However, it's simple enough to make Foundry reload templates when an application renders, by simply removing the template from the template cache.
In Monster Blocks, I do the following:
Hooks.on("renderActorSheet", (...args) => {	// This is just for debugging, it prevents this sheet's template from being cached.
	if (!debugging()) return;

	let template = "modules/monsterblock/actor-sheet.html";
    delete _templateCache[template];
    console.debug(`Monster Block | removed "${template}" from _templateCache.`);
	console.log(args);
});

This specifically removes my template from the cache, but it's just as easy to purge the entire cache. It would be great if there were a feature of devMode that would purge the cache all the time, or otherwise disable the cache behavior.

and then loadTemplate in the Application call stack re-populates the cache just fine? That is nice.

Let's see, aside from deleting the cache every time something renders, one could patch this function:
/**
 * Get a template from the server by fetch request and caching the retrieved result
 * @param {string} path           The web-accessible HTML template URL
 * @returns {Promise<Function>}	  A Promise which resolves to the compiled Handlebars template
 */
async function getTemplate(path) {
	if ( !_templateCache.hasOwnProperty(path) ) {
    await new Promise((resolve, reject) => {
    	game.socket.emit('template', path, resp => {
    	  if ( resp.error ) return reject(new Error(resp.error));
	      const compiled = Handlebars.compile(resp.html);
	      Handlebars.registerPartial(path, compiled);
	      _templateCache[path] = compiled;
	      console.log(`Foundry VTT | Retrieved and compiled template ${path}`);
	      resolve(compiled);
	    });
    });
	}
	return _templateCache[path];
}

The best choice IMO is to patch getTemplate to first delete the cache entry, then call the original getTemplate function.
const originalGetTemplate = getTemplate;
getTemplate = async function(path) {
    if (_templateCache.hasOwnProperty(path)) delete _templateCache[path];
    return originalGetTemplate(path);
}

That seems like a solid way to do this, and I just put libwrapper's shim in last version so it'll be a simple and safe-ish patch.

Done with 1.6.0


[manageiq_foreman] List rest-client dependency

Ensure tests are run with the correct version

@gmcculloug Please review
cc @Fryguy @jrafanie

If we bump and lock it to 2.0.0 does that break anything else that uses rest-client?

@Fryguy we would have to do the same for our app Gemfile.  I don't see an official 2.0.0 release yet.

I'm more wondering about things like ovirt gem or other gems that use rest-client that we don't know about

There are 7 gems that depend on rest-client according to the Gemfile.lock...can we loosen this to something like ">=" and whatever minimum you need.
Or perhaps asking another way, why are you making this change?

Recently, Apipie-bindings v0.0.16 was released and caused failures on ManageIQ/manageiq VMDB test suite because of an incompatability with rest-client 2.x.  We lock the version of rest-client here.  Since this wasn't specified in manageiq_foreman, it installed the latest stable version (v1.8.0), ran the tests and everything passed. :disappointed: We would never actually run with 1.8 in production because of the other Gemfiles, so this test suite is giving potentially false positives right now.
In other gems... We don't need to lock them down to 2.x, but unfortunately our Travis builds run with 1.8.0, so we won't know if there is a problem like that until it blows up in production.

I'm fine with this... that's the version we lock down in the manageiq Gemfile.  Does "~> 2.0.0.rc1" allow "2.0.0.rc2" and "2.0.0"?

@jrafanie I don't know how to test that, do you have any ideas?  We can't do spec.add_runtime_dependency "rest-client", "~> 2.0.0"

@bdunne, I think this should work "~>2.0.0.rc1"
irb(main):008:0> Gem::Requirement.new("~>2.0.0.rc1").satisfied_by?(Gem::Version.new("2.0.0.rc2"))
=> true
irb(main):009:0> Gem::Requirement.new("~>2.0.0.rc1").satisfied_by?(Gem::Version.new("2.0.0"))
=> true
irb(main):010:0> Gem::Requirement.new("~>2.0.0.rc1").satisfied_by?(Gem::Version.new("2.0.1"))
=> true
irb(main):011:0> Gem::Requirement.new("~>2.0.0.rc1").satisfied_by?(Gem::Version.new("1.9.9"))
=> false


Docker build fail due to pip install

Describe the bug
Docker fails at installing pip packages: tables and tensorflow
To Reproduce
Steps to reproduce the behavior:

Download Oracle JRE available in /resources (/jre-8u251-linux-x64.tar.gz, the 221 version is not available)
Run docker build (as mentioned in your doc)
It will fail due to  multiprocess==0.70.7 (hangs), using multiprocess==0.70.9 to fix this
It will fail due to tables==3.5.2
If I upgrade to tables==3.6.1, then tensorflow==1.15.1 fails
if I remove tensorflow, then the whole install fail

I also needed to fix all the COPY and ADD command that were using absolute path which seens to point ti the root of the machine to put files in Docker, I changed to make sure it uses the current directory:
ADD /resources/jre-8u251-linux-x64.tar.gz /usr/local/oracle-jre8-installer-local
# becomes:
ADD ./resources/jre-8u251-linux-x64.tar.gz /usr/local/oracle-jre8-installer-local

I also download the dump (6G) and data (1.6G) before to make sure no issue happens with the download:
wget -O resources/ckg_080520.dump https://data.mendeley.com/datasets/mrcf7f4tc2/1/files/bf08667b-588f-4f40-b5fd-930f4e05368f/ckg_080520.dump?dl=1
wget -O resources/data.tar.gz https://data.mendeley.com/datasets/mrcf7f4tc2/1/files/c0d058a2-adfa-4b96-97d9-c9ec7fc5adb9/data.tar.gz?dl=1

Apart from that the Dockerfile is the same. So I am surprised that it completed for other users
The updated Dockerfile I am using can be found at https://github.com/vemonet/CKG
Expected behavior
Should manage to install pip packages that are published to Pypi
Screenshots
Error (stuck for more than 10min for 1.4MB):
Collecting more-itertools==7.0.0
  Downloading more_itertools-7.0.0-py3-none-any.whl (53 kB)
Collecting multi-key-dict==2.0.3
  Downloading multi_key_dict-2.0.3.tar.gz (8.4 kB)
Collecting multidict==4.5.2
  Downloading multidict-4.5.2.tar.gz (105 kB)
Collecting multiprocess==0.70.7
  Downloading multiprocess-0.70.7.tar.gz (1.4 MB)

Error message for  tables==3.5.2:
Collecting statsmodels==0.10.0
  Downloading statsmodels-0.10.0.tar.gz (14.0 MB)
Collecting tables==3.5.2
  Downloading tables-3.5.2.tar.gz (7.8 MB)
    ERROR: Command errored out with exit status 1:
     command: /usr/bin/python3 -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-p7zeh6o_/tables/setup.py'"'"'; __file__='"'"'/tmp/pip-install-p7zeh6o_/tables/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-ifcdl756
         cwd: /tmp/pip-install-p7zeh6o_/tables/
    Complete output (12 lines):
    /tmp/H5closeqa6mg_ii.c: In function ‘main’:
    /tmp/H5closeqa6mg_ii.c:2:5: warning: implicit declaration of function ‘H5close’ [-Wimplicit-function-declaration]
        2 |     H5close();
          |     ^~~~~~~
    /usr/bin/ld: cannot find -lhdf5
    collect2: error: ld returned 1 exit status
    * Using Python 3.8.2 (default, Apr 27 2020, 15:53:34)
    * USE_PKGCONFIG: True
    .. ERROR:: Could not find a local HDF5 installation.
       You may need to explicitly state where your local HDF5 headers and
       library can be found by setting the ``HDF5_DIR`` environment
       variable or by using the ``--hdf5`` command-line option.
    ----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.
The command '/bin/sh -c pip3 install --ignore-installed -r requirements.txt' returned a non-zero code: 1

Issue with tensorflow when I update tables. The issue stay with other version of tensorflow (which are available on pypi). It's complaining that it should be tensorflow 2
Collecting statsmodels==0.10.0
  Downloading statsmodels-0.10.0.tar.gz (14.0 MB)
Collecting tables==3.6.1
  Downloading tables-3.6.1-cp38-cp38-manylinux1_x86_64.whl (4.3 MB)
ERROR: Could not find a version that satisfies the requirement tensorflow==1.15.1 (from -r requirements.txt (line 169)) (from versions: 2.2.0rc1, 2.2.0rc2, 2.2.0rc3, 2.2.0rc4, 2.2.0)
ERROR: No matching distribution found for tensorflow==1.15.1 (from -r requirements.txt (line 169))
The command '/bin/sh -c pip3 install --ignore-installed -r requirements.txt' returned a non-zero code: 1

Desktop (please complete the following information):

OS: Ubuntu 18.04
Version: docker version:

Client: Docker Engine - Community
 Version:           19.03.11
 API version:       1.40
 Go version:        go1.13.10
 Git commit:        42e35e61f3
 Built:             Mon Jun  1 09:12:22 2020
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          19.03.11
  API version:      1.40 (minimum version 1.12)
  Go version:       go1.13.10
  Git commit:       42e35e61f3
  Built:            Mon Jun  1 09:10:54 2020
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.2.13
  GitCommit:        7ad184331fa3e55e52b890ea95e65ba581ae3429
 runc:
  Version:          1.0.0-rc10
  GitCommit:        dc9208a3303feef5b3839f4323d9beb36df0a9dd
 docker-init:
  Version:          0.18.0
  GitCommit:        fec3683

Additional context

I am really surprised that so many services are deployed and installed in a single Docker image
It would be much easier and stable to separate those services in separater containers and run it using docker-compose (and use this way to install CKG for anyone)

Getting the following error when removing tensorflow:
Running setup.py install for cffi: started
    Running setup.py install for cffi: finished with status 'error'
    ERROR: Command errored out with exit status 1:
     command: /usr/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-xl5eu5d2/cffi/setup.py'"'"'; __file__='"'"'/tmp/pip-install-xl5eu5d2/cffi/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record /tmp/pip-record-w645e0zt/install-record.txt --single-version-externally-managed --compile --install-headers /usr/local/include/python3.8/cffi
         cwd: /tmp/pip-install-xl5eu5d2/cffi/
    Complete output (56 lines):
    Package libffi was not found in the pkg-config search path.
    Perhaps you should add the directory containing `libffi.pc'
    to the PKG_CONFIG_PATH environment variable
    No package 'libffi' found
    Package libffi was not found in the pkg-config search path.
    Perhaps you should add the directory containing `libffi.pc'
    to the PKG_CONFIG_PATH environment variable
    No package 'libffi' found
    Package libffi was not found in the pkg-config search path.
    Perhaps you should add the directory containing `libffi.pc'
    to the PKG_CONFIG_PATH environment variable
    No package 'libffi' found
    Package libffi was not found in the pkg-config search path.
    Perhaps you should add the directory containing `libffi.pc'
    to the PKG_CONFIG_PATH environment variable
    No package 'libffi' found
    Package libffi was not found in the pkg-config search path.
    Perhaps you should add the directory containing `libffi.pc'
    to the PKG_CONFIG_PATH environment variable
    No package 'libffi' found
    running install
    running build
    running build_py
    creating build
    creating build/lib.linux-x86_64-3.8
    creating build/lib.linux-x86_64-3.8/cffi
    copying cffi/verifier.py -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/cparser.py -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/api.py -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/model.py -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/recompiler.py -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/lock.py -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/error.py -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/pkgconfig.py -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/backend_ctypes.py -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/commontypes.py -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/setuptools_ext.py -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/ffiplatform.py -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/vengine_cpy.py -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/cffi_opcode.py -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/__init__.py -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/vengine_gen.py -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/_cffi_include.h -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/parse_c_type.h -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/_embedding.h -> build/lib.linux-x86_64-3.8/cffi
    copying cffi/_cffi_errors.h -> build/lib.linux-x86_64-3.8/cffi
    running build_ext
    building '_cffi_backend' extension
    creating build/temp.linux-x86_64-3.8
    creating build/temp.linux-x86_64-3.8/c
    x86_64-linux-gnu-gcc -pthread -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O2 -Wall -g -fstack-protector-strong -Wformat -Werror=format-security -g -fwrapv -O2 -g -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 -fPIC -DUSE__THREAD -DHAVE_SYNC_SYNCHRONIZE -I/usr/include/ffi -I/usr/include/libffi -I/usr/include/python3.8 -c c/_cffi_backend.c -o build/temp.linux-x86_64-3.8/c/_cffi_backend.o
    c/_cffi_backend.c:15:10: fatal error: ffi.h: No such file or directory
       15 | #include <ffi.h>
          |          ^~~~~~~
    compilation terminated.
    error: command 'x86_64-linux-gnu-gcc' failed with exit status 1
    ----------------------------------------
ERROR: Command errored out with exit status 1: /usr/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-xl5eu5d2/cffi/setup.py'"'"'; __file__='"'"'/tmp/pip-install-xl5eu5d2/cffi/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record /tmp/pip-record-w645e0zt/install-record.txt --single-version-externally-managed --compile --install-headers /usr/local/include/python3.8/cffi Check the logs for full command output.
The command '/bin/sh -c pip3 install --ignore-installed -r requirements.txt' returned a non-zero code: 1

Excellent suggestions and brilliant description of the issues!
I have now updated the requirements.txt file to avoid problems in the installation. The tensorflow version used is no longer available and that caused the conflicts. The issue with multiprocess was never reported before but I have also upgraded the version just in case. Also, I have removed some of the packages that could cause errors with the pinned versions.
tensorflow is needed by at least one of the packages we use for imputation and cannot be removed.
There will be a error for the pygments package with an incompatible version with py2neo, however it does not affect and we are planning anyway to move to the supported driver (https://neo4j.com/docs/api/python-driver/current/) as soon as possible.
Tested creating an empty environment and running:
pip install -r requirements.txt
Regarding:

I am really surprised that so many services are deployed and installed in a single Docker image
It would be much easier and stable to separate those services in separated containers and run it using docker-compose (and use this way to install CKG for anyone)

This is a great suggestion! As may have noticed I am not an expert on docker and there are for sure ways to improve the container. I will look into building a better build using the compose functionality and I will be grateful if you send suggestions/solutions. THANK YOU!

@straussmaximilian tested the new reqs and experienced errors related to missing C libraries. I just noticed that the python version being installed is not the correct, it upgrades to the latest version 3.8. I keep the issue open until these errors are solved (on it).

Ok, thanks a lot!
I pulled your changes (from master) and building it on my side, I guess I will face similar issues at the moment
For the docker-compose it is a bit extra work at the beginning to cleanly separate, connect and initiate the different containers, but once it is done properly it is much easier to maintain! (and scale)

It was indeed the python version being installed. I modified how python 3.6 is installed and since I was at it also moved to openJDK. I am now running the final tests.

Closing and continuing here


Bugfix: add is_positive variable

On some systems, the vector values would be empty when resolving the second argument of the RegisterUnaryCallback call. This meant that the callback was always treated as positive. This caused issues if registering a transit vector with negative values. Creating an is_positive variable and resolving before the call fixes the issue.
P.S. Sorry for the triple PR, the CLA stuff is a bit tricky.

Thanks for the PR.
I will redo it myself to include the correct formatting.


Unrecognized font

So We have seen this issue time and time again and the docs are very explicit on how to get it to work, how ever my question for you is: is this an error with React native or you? because at this point I have followed all your instructions to a T, which you can see in this issue I raised with React Native, 8051.
Its pretty straight forward, if I run the application with the run button the fonts are added and life is grand. if I use their command to run the application: react-native run-ios then the fonts are missing and all hell breaks loose.
Can any one confirm or deny that this isn't me?

Hi All
i have the same problem

Maybe i have solved the problem
Try:
Install the reactive-native-vector-icons
watchman watch-del-all
rm -rf node_modules && npm install
npm start -- --reset-cache
now open another command window and
react-native run-ios
this work for me

@ogonbat  So I did exactly as you suggested, still the same error.

did you rnpm link?

My steps:
// I don't use watchman, but ... I do this any ways
watchman watch-del-all
rm -rf node_modules && npm install
rnpm link
npm start -- --reset-cache
react-native run-ios

Everything launches fine till I see:

same problem

This is not a bug with this project, however try to clean the build folder with rm -rf ./ios/build

for me the euro was in info.plist.
https://github.com/oblador/react-native-vector-icons#option-manually
make sure you have all the fun in the info.plit

So the way that I solved this was I had to follow React linking Libraries which is stated in this README for this project but it should be a requirement to do so. Once I manually copied over the actual library and linked it that way as well as doing rnpm link everything worked.

Happens to me to.
Fresh install and then "rnpm link"
reproduces.

Happens to me too :(
If indeed this is not an issue with the project, it would be great to have a documentation how to handle it.

Just run the project from Xcode instead? Have tried to make the docs as explicit as possible, if you have any suggestions a PR is welcome.

@oblador
I think the issue is that rnpm doesn't create a group in xcode called Fonts. I followed the manual guide and just doing that solved my issue

I'm having the same issue. I've doubled checked that the font is in Fonts, in Copy Bundle Resources and in Info.plist and it still don't work.
have tried to clean the project too.
weird.

I am having the same issue as above. I have double checked everything but no luck.

A mixture of following
watchman watch-del-all rm -rf node_modules && npm install rnpm link npm start -- --reset-cache react-native run-ios
and then following
https://facebook.github.io/react-native/docs/linking-libraries-ios.html
got it working for me. I suspect that maybe the rnpm does something that the manual instructions does not.

@grabbou: Any thoughts on this?

@cvongrim that works for me! Thanks!

Hey,
You could try this:

stop node server
start node server
react-native run-ios
Command + R (in simulator)

Worked for me.

I'm going to go ahead and close this, if you think the documentation is unclear about this I'll accept changes to the troubleshooting section.

same error still happening reread all the instructions and tried all the posted solutions

So for me to get it to work today, I only added the font file in resources that I had in plist and run in the xcode.


XS Breakpoint not working

Bug type
Component, Other
Component name
No response
What happened?
When combining breakpoints and spacing the xs-tag  has no effect. pa-xs-3 and pa-sm-3 should work the same. The same goes for justify-xs-center justify-sm-center and everything else.
Expected behavior
When decorating a component with pa-xs-3 and pa-sm-3 the render output should be the same at both screen sizes but are not.
Reproduction link
https://try.mudblazor.com/snippet/QYGcEGQGoShffzUe
Reproduction steps

Create a new VS 2022 blazor solution
Add the code like in https://try.mudblazor.com/snippet/wOcGuwwGoSkwNCSg to any page.
Run the solution

Relevant log output
No response
Version (bug)
6.0.6
Version (working)
No response
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[ ] I agree to follow this project's Code of Conduct

Because MudBlazor didn't have something-xs-value. An example if you use pa-3 pa-sm-6 it means pa-6 for sm and upper, so in this situation you have pa-3 for xs and pa-6 for others.

Weird. I'm sure I've seen examples in the documentation where xs is used in all sorts of documentation. But the documentation is being re-written at the moment I guess since the "spacing" section is gone.
Anyway. Your suggestion seem to be working so I've got a way forward. Thanks.

xs is default.

Which isn't mentioned anywhere in the documentation...

Which isn't mentioned anywhere in the documentation...

You are right not a clear explanation for now, it will be add in docs in next releases. And i also think it's predictable though that xs is the default.


Why is ErrNoRows thrown if no rows were updated?

Hello, referring to this piece of code:
// Update updates all columns of row specified by primary key in SQL database table with given record.
// If record implements BeforeUpdater, it calls BeforeUpdate() before doing so.
//
// Method returns ErrNoRows if no rows were updated.
// Method returns ErrNoPK if primary key is not set.
func (q *Querier) Update(record Record) error {
	err := q.beforeUpdate(record)
	if err != nil {
		return err
	}

	table := record.Table()
	values := record.Values()
	columns := table.Columns()

	// cut primary key
	pk := table.PKColumnIndex()
	values = append(values[:pk], values[pk+1:]...)
	columns = append(columns[:pk], columns[pk+1:]...)

	return q.update(record, columns, values)
}

Why Method returns ErrNoRows if no rows were updated. ?   Why is this an error? Users may sometimes want to update something regardless if no changes were made to a row.  Errors should be thrown only if they are unexpected, no?

In Go errors are not exceptions, they are totally expected. They are being returned, not thrown.
There are two reasons for ErrNoRows:

In my experience, not being to update a row by a primary is more often an error than not. I just checked the large proprietary code that uses reform. We have a lot of calls to Update there, but only a few checks for ErrNoRows. So that's just an optimization for a more common case (for us).
For consistency with other methods that work with primary keys: FindByPrimaryKeyFrom, Delete, etc.

Hey @AlekSi , thanks for the quick reply.
As a user, when I see the only value returned from an Update function is error, I expect that the database transaction failed.  In this case, it succeeded but still gives me an error.
Therefore, how can I differentiate between a "real" error thrown by SQL (i.e. a DB constraint wasn't met) and this "no rows were updated" error?  The only way I know is to compare the strings i.e. err.Error() == "sql: no rows updated..."
What about returning an int of the amount of rows that were actually updated (in this case 0?) and treat errors as only when a transaction fails?

You should compare value directly:
err := q.Update(&user)
if err == reform.ErrNoRows {
  err = nil  // that's fine if user not found
}
if err != nil {
  return err
}

Hey that makes sense to wait until v2.
In the meantime though, I think you might want to mention how Update will send an error even if the update worked if no rows were updated as this is a critical trip-up for folks (me, my dev team member both got super confused).
Lastly, FYI I tried this and it looks like one must do instead:
err.Error() == reform.ErrNoRows.Error()


Support for multiple database connections

consider connecting multiple databases for querying relevant data？

Thanks -- this is something that we've been thinking about for a while. What are the multiple databases that you're connecting to?
Is your use case that the user should be able to ask a question and then Vanna will decide which database connection to query?
Do you have human analysts who query multiple databases currently?

Can Vanna be integrated with Trino, considering that Trino has the capability for multiple database connections?

Yes, I would like multiple tables from different databases to be able to be joined together for querying, just like multiple tables within a single database. Since we have multiple systems corresponding to multiple databases, it is often necessary to connect to multiple databases for querying when retrieving data for analysis.

Thanks -- this is something that we've been thinking about for a while. What are the multiple databases that you're connecting to?
Is your use case that the user should be able to ask a question and then Vanna will decide which database connection to query?
Do you have human analysts who query multiple databases currently?

Yes! In my case, I have separate databases for metadata (schema tables) and business data tables. The schema tables contain information about the names of the databases where the business data tables reside. A human engineer will look up in the schema database firstly, then he will know which database and tables contain what he wants. This is my case.

Thanks -- this is something that we've been thinking about for a while. What are the multiple databases that you're connecting to?
Is your use case that the user should be able to ask a question and then Vanna will decide which database connection to query?
Do you have human analysts who query multiple databases currently?

Yes! In my case, I have separate databases for metadata (schema tables) and business data tables. The schema tables contain information about the names of the databases where the business data tables reside. A human engineer will look up in the schema database firstly, then he will know which database and tables contain what he wants. This is my case.

Same in my case. Is the feature available now or any plans for that ?

@zainhoda , I need to connect to multiple SQLite databases, where the user can ask a question, and Vanna should be able to determine which database to connect to and execute the query to retrieve the results.


Enable push-down of constant join conditions in cross and outer-joins

Summary of the changes / Why this improves CrateDB
This extends optimizer_move_constant_join_conditions_beneath_join to also provide filter-push-down for cross and outer joins.
Checklist

[x] Added an entry in the latest docs/appendices/release-notes/<x.y.0>.rst for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in the latest docs/appendices/release-notes/<x.y.0>.rst
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

@mfussenegger I made a second iteration to handle outer joins properly.  Outer-joins only apply constant join conditions on their non-preserved side:

LEFT JOIN will only filter on the RHS
RIGHT JOIN will only filter on the LHS

Therefore we can only extract the filter in these cases on top of the join

I think the real benefit of this optimization concept is that it converts non-equi-join to equi-joins which then leads to further optimizations. With having https://github.com/crate/crate/issues/15147 in mind, i think this is the right direction.

The test is still flaky, i will put this on hold.

@matriv Different approach with the same behaviour. This is 100 % save to do so.
cr> SELECT * FROM t1 LEFT JOIN t2 ON t1.a > 5 AND t1.a = t2.b;
+----+------+
|  a |    b |
+----+------+
|  3 | NULL |
|  6 |    6 |
|  8 |    8 |
|  9 |    9 |
|  2 | NULL |
|  5 | NULL |
|  1 | NULL |
|  7 |    7 |
| 10 |   10 |
|  4 | NULL |
+----+------+

cr> SELECT * FROM t1 RIGHT JOIN t2 ON t1.a > 5 AND t1.a = t2.b;
+----+----+
|  a |  b |
+----+----+
|  6 |  6 |
|  8 |  8 |
|  9 |  9 |
|  7 |  7 |
| 10 | 10 |
+----+----+

Will now also run the sql-logic tests on this.

thx @mkleen!
Left a minor comment. If it's not too much trouble maybe you can run and post a representative benchmark that shows the perf improvement we get.

This depends really on scenario and how much the filter will reduce the number of join operation to process by pushing it down to the collect. The impact of a filter push-down can be really significant e.g.:
Q: select count(*) from doc.t1 right join doc.t2 on t1.id = t2.id and t1.id < 1000
C: 1
| Version |         Mean ±    Stdev |        Min |     Median |         Q3 |        Max |
|   V1    |     6180.172 ±  107.879 |   6037.348 |   6157.541 |   6255.526 |   6353.298 |
|   V2    |       77.206 ±   55.086 |     56.349 |     59.384 |     62.273 |    233.831 |
├---------┴-------------------------┴------------┴------------┴------------┴------------┘
|               - 195.06%                           - 196.18%
There is a 100.00% probability that the observed difference is not random, and the best estimate of that difference is 195.06%
The test has statistical significance

Note: This is a benchmark where it works well, created on purpose for this use-case.

Yep, of course benchmarks depend on the exact query and data, but this one can show well how much this enchanced rule can help for such cases.


Aspect Ratio Differences

Hi,
when using the same custom preset for the docker and the GUI and the same input file I get different results.
The input file is 1920X1080 and if I use my custom preset (which I also use on other machines) I get an output of 1280X694 which is expected.
If I drop the file in the watch folder I get an output of 1280X1080 which obviously looks pretty weird!
Any idea why there should be a difference?
here is the log from the incorrect formatting via the watch folder
------- CONVERSION OUTPUT Fri Feb 23 11:27:35 UTC 2018 -------
[11:27:35] hb_init: starting libhb thread
[11:27:35] thread 7f304693aae8 started ("libhb")
HandBrake 1.0.7<PHONE_NUMBER>) - Linux x86_64 - https://handbrake.fr
4 CPUs detected
Opening /watch/Mr. Brooks 2007 BluRay 10Bit 1080p DD5.1 H265-d3g (1)-002.mkv...
[11:27:35] CPU: Intel(R) Xeon(R) CPU E5-1650 v4 @ 3.60GHz
[11:27:35]  - Intel microarchitecture Broadwell
[11:27:35]  - logical processor count: 4
[11:27:35] hb_scan: path=/watch/Mr. Brooks 2007 BluRay 10Bit 1080p DD5.1 H265-d3g (1)-002.mkv, title_index=1
udfread ERROR: ECMA 167 Volume Recognition failed
disc.c:317: failed opening UDF image /watch/Mr. Brooks 2007 BluRay 10Bit 1080p DD5.1 H265-d3g (1)-002.mkv
disc.c:418: error opening file BDMV/index.bdmv
disc.c:418: error opening file BDMV/BACKUP/index.bdmv
[11:27:35] bd: not a bd - trying as a stream/file instead
libdvdnav: Using dvdnav version 5.0.1
libdvdread:DVDOpenFileUDF:UDFFindFile /VIDEO_TS/VIDEO_TS.IFO failed
libdvdread:DVDOpenFileUDF:UDFFindFile /VIDEO_TS/VIDEO_TS.BUP failed
libdvdread: Can't open file VIDEO_TS.IFO.
libdvdnav: vm: failed to read VIDEO_TS.IFO
[11:27:35] dvd: not a dvd - trying as a stream/file instead
Input #0, matroska,webm, from '/watch/Mr. Brooks 2007 BluRay 10Bit 1080p DD5.1 H265-d3g (1)-002.mkv':
Metadata:
title           : Mr. Brooks 2007 BluRay 10Bit 1080p DD5.1 H265-d3g
encoder         : libebml v1.3.5 + libmatroska v1.4.8
creation_time   : 2018-02-23T11:25:58.000000Z
Duration: 00:01:00.07, start: 0.000000, bitrate: 4092 kb/s
Chapter #0:0: start 0.000000, end 60.068000
Metadata:
title           : (01)00:00:00:000
Stream #0:0: Video: hevc (Main 10), yuv420p10le(tv, bt709), 1920x1040 [SAR 1:1 DAR 24:13], 23.98 fps, 23.98 tbr, 1k tbn, 23.98 tbc (default)
Metadata:
BPS             : 3641623
BPS-eng         : 3641623
DURATION        : 00:01:00.060000000
DURATION-eng    : 00:01:00.060000000
NUMBER_OF_FRAMES: 1440
NUMBER_OF_FRAMES-eng: 1440
NUMBER_OF_BYTES : 27339485
NUMBER_OF_BYTES-eng: 27339485
_STATISTICS_WRITING_APP: mkvmerge v20.0.0 ('I Am The Sun') 64-bit
_STATISTICS_WRITING_APP-eng: mkvmerge v20.0.0 ('I Am The Sun') 64-bit
_STATISTICS_WRITING_DATE_UTC: 2018-02-23 11:25:58
_STATISTICS_WRITING_DATE_UTC-eng: 2018-02-23 11:25:58
_STATISTICS_TAGS: BPS DURATION NUMBER_OF_FRAMES NUMBER_OF_BYTES
_STATISTICS_TAGS-eng: BPS DURATION NUMBER_OF_FRAMES NUMBER_OF_BYTES
Stream #0:1(eng): Audio: ac3, 48000 Hz, 5.1(side), fltp, 448 kb/s (default)
Metadata:
BPS             : 448000
BPS-eng         : 448000
DURATION        : 00:01:00.064000000
DURATION-eng    : 00:01:00.064000000
NUMBER_OF_FRAMES: 1877
NUMBER_OF_FRAMES-eng: 1877
NUMBER_OF_BYTES : 3363584
NUMBER_OF_BYTES-eng: 3363584
_STATISTICS_WRITING_APP: mkvmerge v20.0.0 ('I Am The Sun') 64-bit
_STATISTICS_WRITING_APP-eng: mkvmerge v20.0.0 ('I Am The Sun') 64-bit
_STATISTICS_WRITING_DATE_UTC: 2018-02-23 11:25:58
_STATISTICS_WRITING_DATE_UTC-eng: 2018-02-23 11:25:58
_STATISTICS_TAGS: BPS DURATION NUMBER_OF_FRAMES NUMBER_OF_BYTES
_STATISTICS_TAGS-eng: BPS DURATION NUMBER_OF_FRAMES NUMBER_OF_BYTES
[11:27:35] scan: decoding previews for title 1
[11:27:35] scan: audio 0x1: ac3, rate=48000Hz, bitrate=448000 English (AC3) (5.1 ch)
Scanning title 1 of 1, preview 3, 30.00 %
Scanning title 1 of 1, preview 6, 60.00 %
Scanning title 1 of 1, preview 8, 80.00 %[11:27:36] scan: 10 previews, 1920x1040, 23.976 fps, autocrop = 0/0/0/0, aspect 16:9, PAR 1:1
Scanning title 1 of 1, preview 10, 100.00 %[11:27:36] libhb: scan thread found 1 valid title(s)

Using preset: myhd720P
title 1:

stream: /watch/Mr. Brooks 2007 BluRay 10Bit 1080p DD5.1 H265-d3g (1)-002.mkv
duration: 00:01:00
size: 1920x1040, pixel aspect: 1/1, display aspect: 1.85, 23.976 fps
autocrop: 0/0/0/0
support opencl: no
chapters:

1: cells 0->0, 0 blocks, duration 00:01:00

audio tracks:

1, English (AC3) (5.1 ch) (iso639-2: eng), 48000Hz, 448000bps

subtitle tracks:
[11:27:36] 1 job(s) to process
[11:27:36] json job:
{
"Audio": {
"AudioList": [
{
"Bitrate": 0,
"DRC": 0.0,
"Encoder": "copy:ac3",
"Mixdown": "none",
"PresetEncoder": "copy",
"Quality": -3.0,
"Samplerate": 0,
"Track": 0
}
],
"CopyMask": [
"copy:mp3",
"copy:aac",
"copy:ac3",
"copy:dts",
"copy:dtshd",
"copy:eac3",
"copy:truehd"
],
"FallbackEncoder": "ac3"
},
"Destination": {
"ChapterList": [
{
"Name": "(01)00:00:00:000"
}
],
"ChapterMarkers": false,
"File": "/output/Mr. Brooks 2007 BluRay 10Bit 1080p DD5.1 H265-d3g (1)-002.mkv",
"Mp4Options": {
"IpodAtom": false,
"Mp4Optimize": false
},
"Mux": "mkv"
},
"Filters": {
"FilterList": [
{
"ID": 6,
"Settings": {
"mode": 1
}
},
{
"ID": 11,
"Settings": {
"crop-bottom": 0,
"crop-left": 0,
"crop-right": 0,
"crop-top": 0,
"height": 1040,
"width": 1280
}
}
]
},
"Metadata": {
"Name": "Mr. Brooks 2007 BluRay 10Bit 1080p DD5.1 H265-d3g"
},
"PAR": {
"Den": 1,
"Num": 1
},
"SequenceID": 0,
"Source": {
"Angle": 0,
"Path": "/watch/Mr. Brooks 2007 BluRay 10Bit 1080p DD5.1 H265-d3g (1)-002.mkv",
"Range": {
"End": 1,
"Start": 1,
"Type": "chapter"
},
"Title": 1
},
"Subtitle": {
"Search": {
"Burn": true,
"Default": false,
"Enable": false,
"Forced": false
},
"SubtitleList": []
},
"Video": {
"ColorMatrixCode": 0,
"Encoder": "x265",
"Level": "",
"OpenCL": false,
"Options": "",
"Preset": "fast",
"Profile": "auto",
"QSV": {
"AsyncDepth": 4,
"Decode": true
},
"Quality": 21.0,
"Tune": "",
"Turbo": false,
"TwoPass": false
}
}
[11:27:36] starting job
[11:27:36] job configuration:
[11:27:36]  * source
[11:27:36]    + /watch/Mr. Brooks 2007 BluRay 10Bit 1080p DD5.1 H265-d3g (1)-002.mkv
[11:27:36]    + title 1, chapter(s) 1 to 1
[11:27:36]    + container: matroska,webm
[11:27:36]    + data rate: 4092 kbps
[11:27:36]  * destination
[11:27:36]    + /output/Mr. Brooks 2007 BluRay 10Bit 1080p DD5.1 H265-d3g (1)-002.mkv
[11:27:36]    + container: Matroska (libavformat)
[11:27:36]  * video track
[11:27:36]    + decoder: hevc
[11:27:36]    + filters
[11:27:36]      + Framerate Shaper (mode=1)
[11:27:36]        + frame rate: 23.976 fps -> constant 23.976 fps
[11:27:36]      + Crop and Scale (width=1280:height=1040:crop-top=0:crop-bottom=0:crop-left=0:crop-right=0)
[11:27:36]        + source: 1920 * 1040, crop (0/0/0/0): 1920 * 1040, scale: 1280 * 1040
[11:27:36]    + Output geometry
[11:27:36]      + storage dimensions: 1280 x 1040
[11:27:36]      + pixel aspect ratio: 1 : 1
[11:27:36]      + display dimensions: 1280 x 1040
[11:27:36]    + encoder: H.265 (libx265)
[11:27:36]      + preset:  fast
[11:27:36]      + profile: auto
[11:27:36]      + quality: 21.00 (RF)
[11:27:36]  * audio track 1
[11:27:36]    + decoder: English (AC3) (5.1 ch) (track 1, id 0x1)
[11:27:36]      + bitrate: 448 kbps, samplerate: 48000 Hz
[11:27:36]    + AC3 Passthru
[11:27:36] sync: expecting 1440 video frames
x265 [info]: HEVC encoder version 2.3
x265 [info]: build info [Linux][GCC 6.3.0][64 bit] 8bit
x265 [info]: using cpu capabilities: MMX2 SSE2Fast SSSE3 SSE4.2 AVX AVX2 FMA3 LZCNT BMI2
x265 [info]: Main profile, Level-4 (Main tier)
x265 [info]: Thread pool created using 4 threads
x265 [info]: Slices                              : 1
x265 [info]: frame threads / pool features       : 2 / wpp(17 rows)
x265 [info]: Coding QT: max CU size, min CU size : 64 / 8
x265 [info]: Residual QT: max TU size, max depth : 32 / 1 inter / 1 intra
x265 [info]: ME / range / subpel / merge         : hex / 57 / 2 / 2
x265 [info]: Keyframe min / max / scenecut / bias: 24 / 240 / 40 / 5.00
x265 [info]: Lookahead / bframes / badapt        : 15 / 4 / 0
x265 [info]: b-pyramid / weightp / weightb       : 1 / 1 / 0
x265 [info]: References / ref-limit  cu / depth  : 3 / on / on
x265 [info]: AQ: mode / str / qg-size / cu-tree  : 1 / 1.0 / 32 / 1
x265 [info]: Rate Control / qCompress            : CRF-21.0 / 0.60
x265 [info]: tools: rd=2 psy-rd=2.00 rskip signhide tmvp fast-intra
x265 [info]: tools: strong-intra-smoothing lslices=6 deblock sao
[11:27:36] sync: first pts video is 0
[11:27:36] sync: "(01)00:00:00:000" (1) at frame 1 time 0
[11:27:36] sync: first pts audio 0x1 is 360
[11:28:49] reader: done. 1 scr changes
[11:28:53] work: average encoding speed for job is 18.976942 fps
[11:28:53] vfr: 1440 frames output, 0 dropped and 0 duped for CFR/PFR
[11:28:53] vfr: lost time: 0 (0 frames)
[11:28:53] vfr: gained time: 0 (0 frames) (0 not accounted for)
[11:28:53] ac3-decoder done: 1877 frames, 0 decoder errors
[11:28:53] hevc-decoder done: 1440 frames, 0 decoder errors
[11:28:53] sync: got 1440 frames, 1440 expected
[11:28:53] sync: framerate min 23.981 fps, max 23.981 fps, avg 23.976 fps
x265 [info]: frame I:      9, Avg QP:17.34  kb/s: 7674.12
x265 [info]: frame P:    286, Avg QP:20.42  kb/s: 2821.82
x265 [info]: frame B:   1145, Avg QP:25.34  kb/s: 366.66
x265 [info]: Weighted P-Frames: Y:12.2% UV:1.7%
x265 [info]: consecutive B-frames: 2.4% 0.3% 0.3% 0.7% 96.3%

encoded 1440 frames in 77.47s (18.59 fps), 899.96 kb/s, Avg QP:24.32
[11:28:53] mux: track 0, 1440 frames, 6762183 bytes, 900.10 kbps, fifo 2048
[11:28:53] mux: track 1, 1877 frames, 3363584 bytes, 447.72 kbps, fifo 2048
[11:28:53] libhb: work result = 0
Encode done!
HandBrake has exited.

I have been doing some experimenting with this...
I have changed my custom present 3-4 times and removed the docker and reloaded but I still always get 1280X1084
I have also tried customizing an existing preset which although I get 1280X720 nothing I can do will make the sound passthrough (like I can make happen on my custom preset) it ALWAYS encodes even though I have removed encoding audio.. any help would be greatly appreciated! its all so close to being perfect!

For the aspect ratio, do you have the same behaviour for all your files or only a specific one?
Can you provide some screenshots showing your audio preset settings?

Hi,
I get the same issue on all files I have tried.
I am using my custom preset (in the log you can see it is called myhd720P
The Sound is doing exactly what it should which is passing through.
The docker works as expected but only if I use the GUI but it is via the watch folder is where the height goes wrong.
here are 2 files for example
Mr Brooks Original Aspect 1920X1040 GUI automatically correctly guesses the output 1280X694 but via the watch folder I get 1280X1080
Dr Zhivago Original Aspect 1920X800 GUI automatically correctly guesses the output 1280X534 but via the watch folder I get 1280X694

Hopefully this may be of help... it is a folder containing 2 encodes of the same file, one was done via the GUI and the other via the Watch Folder, I have included also the logs for each encode..   Same file, same preset.  Hope you can help me :-)  https://1drv.ms/f/s!AjnJhtmhYMJlhqYt1WhXLwAZCQT5Rg

Are you using the latest version of the container image?  If yes, can you try with the previous version (1.11.2) to see if you have the same behaviour?

I am using the latest as I just followed this instructions on this page, how do I specify a version?

In your docker run command, just use jlesage/handbrake:v1.11.2 instead of jlesage/handbrake for the repository.

Hi, I removed the old image and reloaded with 1.11.2 but now I get 1280X720??? I have uploaded the conversion log into the onedrive folder (https://1drv.ms/f/s!AjnJhtmhYMJlhqYt1WhXLwAZCQT5Rg) and also a screenshot of the handbrake GUI that proves the preset auto selects the aspect ratio from 1920X800 to 1280X534 as it should do but for whatever reason the watch folder is not able to select the correct height despite being the same preset?
Thanks for all your time on this...

Did you get distorted image or it encoded black bars?
Also, if you use one of the default preset (like Fast 720p30) without changing any setting, do you get the same result?

Hi, the video results are in the onedrive folder I shared, you will see it is distorted using my custom preset myhd720p but only when using the watch folder, the preset works when using the GUI
I just tried using the default fast 720p30 and although it outputs 1280X720 instead of 1280X594 like my preset does it looks great! (Does this mean it is encoding the black bars?)

I think the video I was talking about is not on OneDrive.  I was talking about the video you got with version 1.11.2: it has a different resolution of 1280x720, while the one with the latest version had a resolution of 1280X694.

My apologies, I have now added myhd720p custom preset result & the veryfast720p30 result to onedrive..

Is the video for veryfast720p30 the right one?  You said that it has a resolution of 1280x720, but the one on OneDrive has 1920x800.

Again! Apologies!
I accidentally uploaded the source.. the encode is there now :-)

Sorry I didn't had time to look at this... Trying to push some fixes for HandBrake.
However, since the veryfast720p30  preset is working better, did you compared with your preset to see the differences?
Also, I was not able to confirm if the auto-crop feature is something that should be working with the CLI...  It doesn't seems to have an argument for this feature and I don't know if it will pick up the option in the preset file.  Do you have more details in this regard?

Hi,
Sorry for my delay too I have been unwell..
The difference between veryfast720p30 and my own is that on my preset Anamorphic is disabled on my preset and enabled on veryfast720p30 I have however tried disabling it on my preset with no difference :-(
As for CLI I have never used it.. I am just using the preset that I have on several machines where the encoding works perfectly and trying to apply it to your docker.  I am sure it must be possible to do through CLI as the GUI I would imaging just uses the same parameters as CLI it is just prettier/easier?

That's what I would assume...  But that's obviously not the case.
Can you also upload a part of the original movie and your preset file (found under ghb/presets.json in your appdata folder)?

Hi, All uploaded, if you load the file segment into the GUI using my preset you will see that handbrake automatically chooses 1280X534 which is what I am trying to achieve.
Many Thanks Again!

Do you still have issue with this?  For me, it seems that enabling anamorphic (to loose) fixed the problem.

Hello,
I think the same problem got me here.
If I use the GUI with a custom preset it keeps the aspect ratio (720x406 in my case), if I use the watch folder it doesn't and makes the video 720x480.
The preset seems ok, "PictureKeepRatio" is set to true.
How can I make Handbrake keep aspect ratio when using watch folders?
Thank you!
                "PictureAutoCrop": true,
                "PictureBottomCrop": 0,
                "PictureCombDetectCustom": "",
                "PictureCombDetectPreset": "default",
                "PictureDARWidth": 0,
                "PictureDeblock": 0,
                "PictureDeblockCustom": "qp=0:mode=2",
                "PictureDeinterlaceCustom": "",
                "PictureDeinterlaceFilter": "decomb",
                "PictureDeinterlacePreset": "default",
                "PictureDenoiseCustom": "",
                "PictureDenoiseFilter": "off",
                "PictureDenoisePreset": "",
                "PictureDenoiseTune": "none",
                "PictureDetelecine": "off",
                "PictureDetelecineCustom": "",
                "PictureForceHeight": 0,
                "PictureForceWidth": 0,
                "PictureHeight": 480,
                "PictureItuPAR": false,
                "PictureKeepRatio": true,
                "PictureLeftCrop": 0,
                "PictureLooseCrop": false,
                "PictureModulus": 2,
                "PicturePAR": "off",
                "PicturePARHeight": 1,
                "PicturePARWidth": 1,
                "PictureRightCrop": 0,
                "PictureRotate": "disable=1",
                "PictureSharpenCustom": "",
                "PictureSharpenFilter": "off",
                "PictureSharpenPreset": "",
                "PictureSharpenTune": "",
                "PictureTopCrop": 0,
                "PictureWidth": 720,

Did you try to enable anamorphic and set it to loose?
"PicturePAR": "loose"

It worked! Thank you!


Type Inference Error with JSONB Functions and Parameters in pg2

Issue Description:
Summary:
I'm experiencing a type inference error when using the pg2 library with PostgreSQL JSONB functions that involve parameters. Specifically, when using functions like jsonb_array_elements and operators like ->>, PostgreSQL throws a polymorphic type error due to parameters being of type unknown.
This issue does not occur when using next.jdbc and honeysql, suggesting that it might be related to how pg2 handles parameter binding and type inference.
Steps to Reproduce:
Setup:
    PostgreSQL table test_table with a jsonb column data containing an array of JSON objects.
    Example data in data column:

    json

[
  {"score": "10", "quality": "high"},
  {"score": "8", "quality": "medium"}
]

Clojure Code Using pg2:
clojure
(require '[pg.jdbc :as jdbc]
         '[pg.honey :as sql])

(def ds ; your database datasource)

(def query
  {:select [[:->> :issue "score"] :score]
   :from   [[[:jsonb_array_elements :data] :issue]]
   :where  [:= [:->> :issue "score"] ?score]})

(jdbc/execute! ds (sql/format query ["10"]))

Expected Behavior:
The query should execute successfully, returning rows where the score field equals "10".

Actual Behavior:
The following error is thrown:

graphql

ERROR: could not determine polymorphic type because input has type unknown

Analysis:
The error suggests that PostgreSQL cannot infer the type of the parameter ?score because it is of type unknown.
This issue seems to be related to how pg2 handles parameter placeholders and type inference with JSONB functions and operators.
When using next.jdbc and honeysql, the same query executes without errors.

Workaround Attempts:
Explicit Casting: Adding explicit type casts to the parameter does not resolve the issue.

clojure

;; Attempt to cast the parameter to text
:where [:= [:->> :issue "score"] [:cast ?score :text]]
This results in the same error.

Using Inline Values: Replacing the parameter with an inline value works, but this is not ideal for queries that require parameterization.
clojure
:where [:= [:->> :issue "score"] "10"]

Additional Information:
Environment:
    pg2 version: `0.1.17`
    PostgreSQL version: `16.3`
    Clojure version: `1.11`

Request:
Is there a way to correctly handle parameter types with JSONB functions and operators in pg2?
If this is a bug, could it be addressed in a future update?
Any guidance or suggestions would be greatly appreciated.

Thank you for your assistance!

Hm, interesting. Let me reproduce it locally...

Just wondering if this text was generated with ChatGPT? Because it looks a bit strange. First, the query you posted above is incorrect. If you try to run it, you'll get an error.
This version looks closer to what you want:
(def query
  {:select [[:->> :issue "score"] :score]
   :from   [[[:jsonb_array_elements :data] :issue]]
   :where  [:= [:->> :issue "score"] ?score]})

I wrote a couple of new tests trying to reproduce it but I couldn't as they work fine. Here are two examples:
(deftest test-client-json-function-params
  (pg/with-connection [conn *CONFIG-TXT*]

    (pg/execute conn "create temp table foo123 (val jsonb)")
    (pg/execute conn "insert into foo123 values ($1)" {:params [{:foo {:bar {:kek 123}}}]})

    (let [res (pg/execute conn "select val #>> array['foo', 'bar', 'kek'] as val from foo123")]
      (is (= [{:val "123"}] res)))

    (let [res (pg/execute conn
                          "select val #>> array[$1, $2, $3] as val from foo123"
                          {:params ["foo" "bar" "kek"]})]
      (is (= [{:val "123"}] res)))

    (let [res (pg/execute conn
                          "select val #>> $1 as val from foo123"
                          {:params [["foo" "bar" "kek"]]})]
      (is (= [{:val "123"}] res)))))

;;
;; https://github.com/igrishaev/pg2/issues/17
;;
(deftest test-client-json-function-params-case-2

  (pg/with-connection [conn *CONFIG-TXT*]

    (pg/execute conn "create temp table test (data jsonb)")
    (pg/execute conn "insert into test values ($1)"
                {:params [[{:score "10" :quality "high"}
                           {:score "8" :quality "medium"}]]})

    (let [sql-map
          '{:select [[[:->> :issue "score"] :score]]
            :from   [[[:jsonb_array_elements ?data] :issue]]
            :where  [:= [:->> :issue "score"] ?score]}

          sql-vec
          (sql/format sql-map
                      {:params {:data [{:score "10" :quality "high"}
                                       {:score "8" :quality "medium"}]
                                :score "10"}})

          res
          (jdbc/execute! conn sql-vec)]

      (is (= [{:score "10"}] res)))))

Could you please provide a real, working query that causes the error you mentioned?

Another working case:
    (let [sql-map
          '{:select [:issue]
            :from   [[{:select [[[:jsonb_array_elements :data] :issue]]
                       :from [:test]}
                      :sub]]
            :where  [:= [:->> :issue "score"] "10"]}

          sql-vec
          (sql/format sql-map
                      {:params {:data [{:score "10" :quality "high"}
                                       {:score "8" :quality "medium"}]
                                :score "10"}})

          res
          (jdbc/execute! conn sql-vec)]

      (is (= [{:issue {:score "10", :quality "high"}}] res)))


Update upstream Prometheus

This PR merges upstream Prometheus (from https://github.com/prometheus/prometheus/commit/867d3bd78f514ffffcfd5da751cb2bd84c6aba72) into mimir-prometheus.
To be merged with merge commit.

Updating Mimir after merging this PR is currently blocked on https://github.com/prometheus/prometheus/pull/10857.


[Accessibility] Labels do not bind to related controls

Describe the bug
When clicking a label on the drop-in payment solution the related control does not receive focus. WCAG 2.1 - 1.3.1: Info and Relationships says relationships need to be programatically determined or available in text. We use labels and form controls bound together to achieve this but it does not work the first time the form is encountered.
The Name on card field works ok but the others do not, I suspect this is because they have an  as their child. Once the field is clicked then the behaviour works but it should work by clicking the label, this can benefit users with an impaired fine motor control.
To Reproduce
Steps to reproduce the behavior:

Go to payment step
Use a mouse to click label "Card number"
Use mouse to click label "Expiry date"
Use mouse to click label "CVC/CVV"

Expected behavior
Clicking the  with a mouse should activate the related field, just as "Name on card" does.
Screenshots

Desktop (please complete the following information):

OS: macOS Monterey 12.5
Browser: Safari 15.6

Additional context
https://www.w3.org/WAI/WCAG21/Techniques/html/H44

Hi @gazjoy - can you explain exactly what your Adyen-web implementation is i.e. what version, whether you are using Dropin or individual components? What exactly are we seeing in the screenshot you sent?
Because this functionality (click on Label, field gets focus) has been available since we launched the product

@sponglord
version is: "@adyen/adyen-web": "5.22.0"
implementation is: drop-in

Hi @gazjoy - this is odd, and Safari specific behaviour... I'll take a look


Allow test_saving_png_file to run during build_conda_pkg job

In #2323, we disabled test_saving_png_file during the conda build because we were getting out of memory from Popen commands issued by graphviz (see image below). This is pretty confusing and my guess is that the directory where the png files are stored in the docker container (pytest tmpdir fixture) is running out of space. The fact that this is all happening within docker may make debugging a bit more complicated.
One possible thing to try is to have each test delete any files it creates once the test finishes and see if it fixes the problem.

Outcome of this is replacement test coverage for saving .png for graphing.  Hopefully, creating this test helps us figure out what the original issue was and the mystery behind "test_env_placehold_placehold_..."


Stats.NumThreads gives wrong value

I'm using procfs for observing number of threads spawned by current process and simultaneously looking to procfs by simple cat /proc//status|grep Thread.
When count of threads if pretty low (under 100) results are the same. But when accoring to data extracted by cat from proc are pretty high (a few hundreds) then procfs library gives me 0. What is the reason of such inconsistency?
Source code can be found here

Running the example code, I wasn't able to reproduce the issue.  The only things I can think of that may have caused this are (1) maybe there was some difference on your system between /proc/<pid>/stat (which is read by procfs) and /proc/<pid>/status (which you were comparing) or (2) maybe there was a bug in the golang fmt.Fscan function which caused the integer to be read incorrectly.
If you are still able to reproduce this issue, I can take a deeper look, otherwise I'll close the issue.


Accessibility: NVDA emits errors in WT1.15 preview

Windows Terminal version
1.15.2283.0
Windows build number
25188.1000
Other Software
NVDA 2022.2.2
Steps to reproduce

Open bash in windows terminal
type nano /etc/hosts

Expected Behavior
No error in NVDA
Actual Behavior
NVDA emits the following error:
ERROR - NVDAObjects.behaviors.LiveText._monitor (18:21:06.798) - Dynamic_WinTerminalUIAEditableTextWithAutoSelectDetectionUIA._monitorThread (22344):
Error getting or calculating new text
Traceback (most recent call last):
  File "NVDAObjects\behaviors.pyc", line 386, in _monitor
  File "NVDAObjects\behaviors.pyc", line 350, in _getText
  File "diffHandler.pyc", line 62, in _getText
  File "baseObject.pyc", line 26, in __get__
  File "NVDAObjects\UIA\__init__.pyc", line 848, in _get_text
  File "NVDAObjects\UIA\__init__.pyc", line 555, in _getTextFromUIARange
  File "monkeyPatches\comtypesMonkeyPatches.pyc", line 32, in __call__
_ctypes.COMError: (-2147467259, 'Unspecified error', (None, None, None, 0, None))

I wasn't able to reproduce this with WT1.14, so therefore I'm filing this here since there might be a regression in UIA related code. @codeofdusk could you reproduce this?

I've also seen this, but not with wtNotifications.
This is probably related to broken assumptions when switching to/from the alt buffer.

This error indeed has nothing to do with the notifications implementation.
CC @carlos-zamora just want to make sure that this bug doesn't end up in a release, since that would be annoying for many sr users.

Yeah, that error code is E_FAIL so it's 100% that alt buffer thing.


The Ruby openssl extension was not compiled. Missing the OpenSSL lib?

rbenv install 2.0.0-p247 fails on Mac OS X 10.8.4 with the error:
The Ruby openssl extension was not compiled. Missing the OpenSSL lib?

I had no problem installing 1.9.3-p448.
Using ruby-build 20130628 via Homebrew.

@mislav I'd like to suggest a check based upon ldconfig -p | grep libssl as it is mentioned on this serverfault post.
If appreciated i'd like trying to create a pull-request for that issue.

@florianb That's a start, except it doesn't work on OS X.
Pull requests welcome of course.

sudo apt-get install libssl-dev works on ubuntu 15.04 :+1:

Some basic libs are required, recommend this:
sudo apt-get install git-core curl zlib1g-dev build-essential libssl-dev libreadline-dev libyaml-dev libsqlite3-dev sqlite3 libxml2-dev libxslt1-dev libcurl4-openssl-dev python-software-properties libffi-dev

I had problems trying to install Ruby 2.2.2 on Fedora 22, but the following worked for me:
dnf install openssl-devel
rbenv install 2.2.2
rbenv global 2.2.2

Ubuntu 14:
sudo apt-get install libssl-dev

patch worked for me when installing 2.2.2 and 2.2.3 on Arch using openssl 1.0.2.g-3

I'm sorry to say this but on macOS 10.12 with latest ruby-build this is still an issue.
It's really unfortunate but this:
CONFIGURE_OPTS="--with-openssl-dir=`brew --prefix openssl`"

... worked ...

macOS 10.13 High Sierra Beta 2
CONFIGURE_OPTS="--with-openssl-dir=`brew --prefix openssl`" rbenv install 2.3.4
Thank you @hugobast

@hugobast
Will try your solution as well, I encountered same error during installation as well.

Hi, for Debian Stretch install
apt-get install libssl1.0-dev
not libssl-dev

Hi, for Debian Stretch and ruby 2.2.3 install:
apt-get install libssl1.0-dev
not libssl-dev, then apply this:
https://github.com/rbenv/ruby-build/issues/834#issuecomment-160627207

For macports, this worked for me...
RUBY_CONFIGURE_OPTS="--with-openssl-dir=/opt/local/" rbenv install 2.4.1

when I installed rbenv through brew, I got this message
This formula is keg-only, which means it was not symlinked into /usr/local,
because Apple has deprecated use of OpenSSL in favor of its own TLS and crypto libraries.

If you need to have this software first in your PATH run:
  echo 'export PATH="/usr/local/opt/openssl/bin:$PATH"' >> ~/.bash_profile

For compilers to find this software you may need to set:
    LDFLAGS:  -L/usr/local/opt/openssl/lib
    CPPFLAGS: -I/usr/local/opt/openssl/include

have you tried adding these flags to rbenv install?

using RUBY_CONFIGURE_OPTS=--with-openssl-dir=/usr/local/Cellar/openssl/[latest version] rbenv install [version] worked for me finally..

using RUBY_CONFIGURE_OPTS=--with-openssl-dir=/usr/local/Cellar/openssl/[latest version] rbenv install [version] worked for me finally..

Works for me. macOS 10.14.6

using RUBY_CONFIGURE_OPTS=--with-openssl-dir=/usr/local/Cellar/openssl/[latest version] rbenv install [version] worked for me finally..

First, thank you so much. I failed at the  beginning because I copied the commands above guys' . You help me out of the trouble.
Here the exactly what we should do:
ls /usr/local/Cellar/openssl/

I was having this issue with OSX 10.15.3. The version of openssl that homebrew installed was  1.1 and I was trying to install ruby 2.3.8, which did not work.
I eventually got it to work using openssl version 1.0.2.
RUBY_CONFIGURE_OPTS=--with-openssl-dir=/usr/local/Cellar/openssl/1.0.2 rbenv install 2.3.8

I had this issue and official rbenv documentation as well as installation of a new version of ruby (2.7.1) solved it.
https://github.com/rbenv/ruby-build/wiki#updating-ruby-build


Set Environment's Ao Channel Effect to 1.0 by default

"Ao Channel Affect" was defaulting to 0, giving unexpected results. This meant that as soon as you set an AO texture on a material, there will no longer be SSAO on that material.
This led many users to think that Godot didn't support using SSAO and AO textures at the same time.
This closes #43957.

Should this pr also handle the importer changes?

Superseded by https://github.com/godotengine/godot/pull/45023.

-SSAO ao affect is gone because this confused users, min(ao,ssao) now used, should work in most cases.

This pull request could perhaps be remade for the 3.2 branch.

Superseded by https://github.com/godotengine/godot/pull/45023.

-SSAO ao affect is gone because this confused users, min(ao,ssao) now used, should work in most cases.

This pull request could perhaps be remade for the 3.2 branch.


Bump chai to 5.0.0

fix requires -> import

Superseeds: #983

See also https://github.com/chaijs/chai/issues/1578!
chai is now esm only and needs dynamic imports


CI: Update Jenkins build status URL

Now that the jenkins url has changed, we need to update
the build status URL.
Fixes: #391
Signed-off-by: Salvador Fuentes<EMAIL_ADDRESS>

adding DNM label as we need to use domain name instead of IP address.
Hopefully tomorrow we will have jenkins.katacontainers.io

The CI is still crashing due to CI problem
ERROR: URL 'http://<IP_ADDRESS>/job/kata-containers-runtime-ubuntu-16-04-master/' in the following files is invalid:
./README.md
./README.md
ERROR: URL 'http://<IP_ADDRESS>/job/kata-containers-runtime-ubuntu-16-04-master/badge/icon' in the following files is invalid:

The URL problem should be fixed by https://github.com/kata-containers/tests/pull/413.

I believe this is fixed, but I cant find "retrigger" button for jenkins even after I logined in.
travis-ci/pr is restarted by me and it's green now, so I think it should be fine to merge.
LGTM

Please close this in favour of #395


Some Non-Canonical Predicates in KG2.7.0

has_gene_product (assigned within ETL scripts) should be gene_product_of

match ()-[e]->() where e.predicate in ['biolink:treated_by', 'biolink:produced_by', 'biolink:preceded_by', 'biolink:superclass_of', 'biolink:has_gene_product', 'biolink:narrow_match', 'biolink:enabled_by', 'biolink:translation_of', 'biolink:has_biomarker', 'biolink:part_of', 'biolink:participates_in', 'biolink:derives_into', 'biolink:is_metabolite_of', 'biolink:expresses', 'biolink:transcribed_to', 'biolink:actively_involves', 'biolink:location_of', 'biolink:caused_by', 'biolink:condition_associated_with_gene'] return e.predicate, count(e)

e.predicate
count(e)

"biolink:has_gene_product"
41620

"biolink:part_of"
5523

Going forward, it might be good to have a validation script that compares kg2_util's edge labels to the Biolink model. Also, it would be good to check the biolink: predicates at the end of predicate-remap.yaml against the Biolink model.


Remove sui-test-validator binary from releases

Description
Remove sui-test-validator binary from releases
Test plan
👀

As per https://mysten-labs.slack.com/archives/C051YTB1GT1/p1714411744246009?thread_ts=1714330986.110369&cid=C051YTB1GT1 we are not doing this anymore


feat: filter peers using cache summary for getbitswap

Why

At the moment, GetBitswap will send requests to all connected peers, which can be inefficient. We therefore want to leverage the cache summaries (bloom filters) sent from the connected peers to filter out peers that definitely don't store the requested content. If there doesn't exist a bloom filter for a connected peer (I cannot think of a case where this happens atm), we still send out the request
What

Filter peers before sending getbitswap request using the bloom filters.

Demo

Notes

Checklist

[x] I have performed a self-review and commented my code, particularly in complex areas
[x] I have made corresponding changes to the tests
[x] I have made corresponding changes to the documentation
[x] I have run the app using my feature and ensured that no functionality is broken

I didn't create a unit test for this because all components used here are already unit tested (test_bitswap_get() and test_send_cache_summary()).


History not returning requested number of bars/periods, when trade data is partially unavailable.

Expected Behavior
My understanding is that when calling History with a constant number of periods/bars, Lean will intelligently handle getting you that exact number of bars by taking into account market hours, so that you can ultimately warm up your indicators, rolling windows, etc...
Actual Behavior
However, if for some reason data is unavailable for a period of time, you will not receive the expected number of bars, not even while using the fill forward setting.
Reproducing the Problem
See example below that can be ran in the CSharp Research environment. I'm requesting 2520 bars starting from 11/20/2020, and going backwards. However only 1035 bars are returned. When analyzing data for IMMP, you can see that there's no data available for 11/17/2020 which is what's causing this issue. I confirmed via the data explorer (https://www.quantconnect.com/data/tree/equity/usa/minute/i*/immp) and also via other charting packages. For whatever reason, data isn't available for the 17th.
That said, what I would expect to happen is that data would be filled forward from the last bar available on 11/16/2020 and Lean would return 2520 bars. This is how most charting platforms handle this, and how any indicator would be calculated.
Code Example (can run in research environment)
#load "../QuantConnect.csx"
var qb = new QuantBook();
qb.SetStartDate(2020, 11, 20);
var security = qb.AddEquity("IMMP", Resolution.Minute, Market.USA, fillDataForward: true, leverage: 1, extendedMarketHours: true);
var begin = qb.Time - TimeSpan.FromDays(27);
var end = qb.Time - TimeSpan.FromDays(22);
var history = qb.History(security.Symbol, 2520, Resolution.Minute);
Console.WriteLine($"History: {history.Count()}");
foreach(var bar in history) {
    Console.WriteLine($"[{bar.Time}] {bar}");
}

Potential Solution
Now in order to do this, I assume Lean would have to reference the actual data within the Time.GetStartTimeForTradeBars method, which could cause a performance hit, but if the FillForward setting is enabled, I think this is how things should work.
Let me know if you guys think this change/enhancements makes sense/is warranted, and if so we can take a crack at it when we get a chance, assuming it's not something you guys want to handle.
System Information
C# algo running on QuantConnect cloud environment running Lean Id: '10011'
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues
[x] I have provided detailed steps to reproduce the issue

Tricky one. Would prefer not to rely on fill forward and focus on fixing the data issue that made it happen. I think filling forward the gap would also be weird for many people and worse hide the issue


test: add tests for single file

Closes #16
Need to add a SilverBlaze.txt file and update previous snapshots because there is a parsing error with execa. Here's how a file name with spaces is parsed (log from argv() ).

Things I have tried but didn't work
 const { stderr, stdout, exitCode } = await run('-i assets/text/The\ Naval\ Treaty.txt');

 const { stderr, stdout, exitCode } = await run('-i "assets/text/The Naval Treaty.txt"');

 const { stderr, stdout, exitCode } = await run('-i \'assets/text/The Naval Treaty.txt\'');

 const { stderr, stdout, exitCode } = await run('-i assets/text/\'The Naval Treaty.txt\'');

@Andrewnt219, thanks for doing this.
I guess you running Windows. Filename with spaces is a known problem on Windows. On macOS/Linux, the first two commands would work.
Also, I already have a Silver Blaze.txt in assets/text/a.
To solve the problem with spaces in the filename, while you are here, can I ask you to change all sample file name with spaces from file name to file-name so next people don't have the same problem 😉?

@DukeManh So I guess WSL still isn't a proper Linux OS huh. Did try to run the same tests in there.

Also, I already have a Silver Blaze.txt in assets/text/a.

This one has space, so it'll cause the mentioned error.

To solve the problem with spaces in the filename, while you are here, can I ask you to change all sample file name with spaces from file name to file-name so next people don't have the same problem 😉?

Sure, but don't you also need files with spaces as the SSG should handle that case? I can run the SSG for files with space using ts-node, the error only happened during testing.

but don't you also need files with spaces as the SSG should handle that case

I think it does support filenames with spaces but when parsing arguments to execa we would have to escape \ to make it work because we are passing a JS string to it.
- const { stderr, stdout, exitCode } = await run('-i assets/text/The\ Naval\ Treaty.txt');
+ const { stderr, stdout, exitCode } = await run('-i assets/text/The\\ Naval\\ Treaty.txt');

The above worked for me.
@Andrewnt219

Makes sense. I've updated the tests and reverted unnecessary changes.


.net6 minimal API - OpenApiOperation Attribute does not work

Using Nswag.Annotations.OpenApiOperation does not seem to work with the new minimal API.
Example code:
var app = builder.Build();
app.MapPost("/Test", Test).WithTags("MyTag");

[OpenApiOperation("MySummary", "MyDescription")]
static string Test(string foo) {
    return foo;
}

This attribute is only supported on controller methods, I think you can use .WithName() for now...

.WithName() only sets the operationId.
I'm looking for the summary and description

Would it be somehow possible to use .WithMetadata() for this?

Confirm that something like .WithMetadata(new OpenApiOperationAttribute("summary", "description")) doesn't work either.


Update code formatting to comply with black & isort standards

Description
This PR updates the codebase to comply with black formatting standards and isort ordering of imports. These changes ensure that the code is consistently formatted and well-organized, enhancing readability and maintainability. No functional changes have been made to the code.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Screenshots (if appropriate)
Additional Notes
These formatting changes are part of my contribution to the GirlScript Summer of Code (GSSoC). Kindly assign this PR a level 2 and grant me the corresponding points. Thank you!

Fix the Conflicts.

@DiptiSanap fixed the conflicts!!

updated the branch

I have completed my tasks for this pull request and it is ready for review and merging. Could you please assign the 'gssoc' tag and the appropriate level to it? I propose that it be classified as 'level 2'. Thank you.


Migration guide from GNSP + GPP

Wiki? Blog post? Readme entry? To explain what should be changed with the new syntax.

I created an initial draft of the migration guide. Feel free to propose enhancements or just enhance it :-).
Among others, I wonder, if we should handle there migration from custom Nexus instances? Sonatype Nexus covers majority of (non-enterprise and non-Apache cases).

I just adjusted the wiki page to reflect the transitionCheckOptions change. Otherwise, it looks good to me!
Let's 🚢 it! 🙂


Replace goog.abstractMethod with @abstract

Replace all use of goog.abstractMethod with the @abstract JSDoc tag. Closure added support for this in 2016, and it is considered the preferred method of noting abstract classes/methods.

We should also put @abstract on all abstract classes to prevent use of new on them. This can either be done here, or split into a separate issue.


Not a structure, FieldChain (check_syntax)

Issue:
./src/nw/#jcs#bc_job_abap_debugging.prog.abap[418, 3]                      - Not a structure, FieldChain (check_syntax)

Source:

Definitions are in include:

Playground does not complain if the definition is inside the same file.

hmm, I cannot reproduce this
I'm guessing it has something to do with text-030, can you try deleting line 420? and see if the error is still there?

You could be right.. there are other errors as well
./src/nw/#jcs#bc_job_abap_debugging.prog.abap[418, 3]                      - Not a structure, FieldChain (check_syntax)
./src/nw/#jcs#bc_job_abap_debugging.prog.abap[446, 3]                      - Not a structure, FieldChain (check_syntax)
./src/nw/#jcs#bc_job_abap_debugging.prog.abap[453, 3]                      - Not a structure, FieldChain (check_syntax)
./src/nw/#jcs#bc_job_abap_debugging.prog.abap[460, 3]                      - Not a structure, FieldChain (check_syntax)
./src/nw/#jcs#bc_job_abap_debugging.prog.abap[467, 3]                      - Not a structure, FieldChain (check_syntax)
./src/nw/#jcs#bc_job_abap_debugging.prog.abap[475, 3]                      - Not a structure, FieldChain (check_syntax)
./src/nw/#jcs#bc_job_abap_debugging.prog.abap[483, 3]                      - Not a structure, FieldChain (check_syntax)
./src/nw/#jcs#bc_job_abap_debugging.prog.abap[491, 3]                      - Not a structure, FieldChain (check_syntax)
./src/nw/#jcs#bc_job_abap_debugging.prog.abap[504, 3]                      - Not a structure, FieldChain (check_syntax)
``

446: `  l_fcat-seltext_l = text-010.`
453: `l_fcat-seltext_l = text-011.`
...

They are all references to text-*.

is bc_job_abap_debugging.prog.abap an include or something special?

It is a report. Nothing special, just a tool to diagnose issues @ customer

hmm, any shadowing? eg. variables or types somewhere that redefine the text variable, like
DATA text TYPE c LENGTH 1.

yea, looks like its the issue, reproduced as:
DATA text TYPE c LENGTH 1.
WRITE TEXT-001.

ref https://github.com/abaplint/abaplint/pull/1251/commits/e82e598c5ae4a2e9245b8f4e39a4a5983148bcf1

try again, 2.39.23 published

Confirmed, there is a variable text :-)

Issue resolved. Thanks


Revert "Run these tests against any builds."

This reverts commit 66d575d106a7db4503fd74540967339c83a4abb7.
This is breaking local workflows. I wanted these tests to run locally, but too many build paths, etc.

Change Description

Change Type
Select ONE:

[ ] Documentation

[ ] Stability bug fix

[x] Other

[ ] Other - special case

Testing Changes
Select ANY that apply:

[ ] New Tests

[x] Existing Tests

[ ] Test Framework

[ ] CI System

[ ] Other

Consensus Changes

[ ] Consensus Changes

API Changes

[ ] API Changes

Documentation Additions

[ ] Documentation Additions

Closed in favor of pull request 10160.


more explicit docs for server.fsServe.root

Add a warning and explanations about https://github.com/vitejs/vite/issues/3347 in the docs.
See https://github.com/vitejs/vite/pull/3321#issuecomment-838271127

Thanks for the contributions! We decided to disable this restrictions by default for now and enable it the future version when we resolve the issues. See #3377.


Add drop_view to the rest catalog

@sungwy, thank you for your review. I will work on this tomorrow!

@sungwy Done!

LGTM! Thank you for adding this new API @ndrluis


add new PRs merged for 0.4.17 to release notes

maybe collect few of them instead of creating a single PR per PR?

@Czaki doing now :)

@Czaki : I'll likely merge this in the next 24 hours, then create one last PR (based on a branch in my fork) to update the 0.4.17 release notes with any other merged PRs, as well the summaries. Does that work for you?

My only worries are doubling commits by creating additional commits per merged PR. Only this.

My only worries are doubling commits by creating additional commits per merged PR. Only this.

Agreed. In future releases, I would be in favor of just creating one PR to track release notes and only merge it when the final RC has settled - at least when's there's only one release manager! There's no real value in merging it earlier as it just creates more commits on main for a file that cannot have merge conflicts.


Document GPX geometry output types

Outputting LineStrings as Routes and MultiLineStrings as Tracks seems a bit dubious to me, but at any rate it should be documented.
Docs only, no changes to logic.

Thanks @probins.


Migrate to manifest v3

Here is more information on it: chrome | firefox
One of the biggest changes I'm seeing is service workers instead of background.js now, and the opportunity to move towards promise based apis instead of the callback method chrome had us using.

Chrome
Manifest V3 is available beginning with Chrome 88, and the Chrome Web Store begins accepting MV3 extensions in January 2021.

Firefox
As of this writing, we are hoping to complete enough work on this project to support developer testing in Q4 2021 and start accepting v3 submissions in early 2022.

Firefox could be a blocker here until next year.

@erzhan-temir-mamyrov
would you like to take a look at it?


Use generic trust service method in the trust_ec2_resources method

A generic trust method was introduced in #92.

This was resolved by 464c4dc, looks like my commit reference was malformed.


ref callback not getting called at all

ref callback on Input component is not getting called at all.
Here is the test code:
import React, { Component } from 'react';
import {
  Button,
  Container,
  Content,
  Form,
  Item,
  Input,
  Label,
  Text
} from 'native-base';

export default class TestComponent extends Component {
  constructor(props) {
    super(props);
    this.refs = {};
  }

  componentDidMount() {
    console.log('=====>>>>>> 2. In componentDidMount() ....');
    this.refs.detailsField.focus();
  }

  render() {
    const { navigate } = this.props.navigation;
    return (
      <Container>
        <Content>
          <Form>
            <Item floatingLabel last>
              <Label>Details</Label>
              <Input
                ref={c => {
                  this.refs.detailsField = c;
                  console.log('=====>>>>>> 1 ....');
                  console.log(this.refs.detailsField);
                }}
                multiline={true}
                numberOfLines={10}
                style={{ height: 200, textAlignVertical: 'top' }}
                onChangeText={text => this.setState({ details: text })}
              />
            </Item>
          </Form>
        </Content>
      </Container>
    );
  }
}

ref callback is not getting called on rendering of Inpput component. Ideally we should see below console logs
=====>>>>>> 1 ....
=====>>>>>> 2. In componentDidMount() ....

But on console I am only seeing
=====>>>>>> 2. In componentDidMount() ....

Also seeing below warning / error message from React-Native (as I am calling this.refs.detailsField.focus() in componentDidMount()
Possible Unhandled Promise Rejection (id: 0):
TypeError: undefined is not an object (evaluating 'this.refs.detailsField.focus')
...
....

Looks like ref callback on Input field is not getting called at all.
Is my understanding wrong about ref or is it a bug ?
Please try this test case.

See #591


Mark version 0.1

Via python pip I'm able to install a "version 0.1.0", but it seems not reflected in this repos.

Okay. Just discovered, that pypi version is not from this repos but from there: https://github.com/zsprackett/python_sensu_plugin


add 'start' parameter to getusertransactions api page

sets db start row

Cool, this could be useful for some API clients :-) Thanks for the contribution!

Missed it, wrong target branch. Could you recreate this against development branch?


question in re licensing

[Enter feedback here]
On this page, you make reference to the following:
Note
Only SQL Server core-based licensing with Software Assurance or subscription licenses are eligible for Azure Hybrid Benefit. If you are using Server + CAL licensing for SQL Server and you have Software Assurance, you can use bring-your-own-license to an Azure SQL Server virtual machine image to leverage license mobility for these servers, but you cannot leverage the other features of Azure Hybrid Benefit.
I have a client that wishes to license SQL server in Azure on an 8c/56GB VM (a D13 SQL Managed Instance).  They have only 5 internal users- its not web facing.  The question is, can they use a SQL Server 2019 Standard license, with 5 User CALs and Software assurance to license this instance?  To license it under the Core model would be prohibitively expensive.
Incidentally, we're an indirect CSP and all of our CSP partners who I reached out to are thoroughly confused on this subject so some guidance might be needed.
thanks,
Tom

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

ID: e2600a44-9090-4144-b91d-326b027e617b
Version Independent ID: 329fdb8f-5623-7a04-9f6a-4f923820c4fc
Content: Change the license model for a SQL VM in Azure - SQL Server on Azure VM
Content Source: articles/azure-sql/virtual-machines/windows/licensing-model-azure-hybrid-benefit-ahb-change.md
Service: virtual-machines-sql
Sub-service: management
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

@tflorio
Thanks for your Question! We are currently investigating and will get back on this.

Hi @tflorio , thanks for reaching out! Yes, as the documentation states: If you are using Server + CAL licensing for SQL Server and you have Software Assurance, you can use bring-your-own-license to a SQL Server on Azure VM  image to leverage license mobility for these servers, but you cannot leverage the other features of Azure Hybrid Benefit.
My apologies, would you mind providing feedback for how we can clarify this further?
Thanks!
Masha from the SQL Docs team

Hope the provided information is helpful.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.

@VikasPullagura-MSFT  -
Sorry I didnt see your reply right away.  In re your question, I need clarity on what the features are that would be lost if you are using a Server + CAL licensing + Software Assurance to license SQL Server on an Azure VM.
I also would need some clarity- can you use this to license SQL Server Managed Instances as well?

Hey @tflorio wit AHB you get features such as HA/DR benefits which you wont be able to get via Server +CAL, Please check here for more information https://docs.microsoft.com/en-us/azure/azure-sql/virtual-machines/windows/business-continuity-high-availability-disaster-recovery-hadr-overview#free-dr-replica-in-azure


option-based conflicts

I want to introduce a feature: conflict that is triggered only by some option.
One may want to have some feature provided by one of the possible implementations and keeping both formulas installed, while disabling particular feature in one of the formulas.
Example: there is two uuid implementations ossp-uuid and util-linux, I want to make util-linux conflict with ossp-uuid only if it's build --with-uuid, e.g.
conflicts_with "ossp-uuid" if build.with? "uuid"

As in https://github.com/Homebrew/homebrew-core/pull/31378

I think the conflicts_with mechanism is already fairly unpleasant from a UX perspective and I'm not keen to expand its usage more than absolutely necessary, and options by being options aren't absolutely necessary. I would be very reluctant to see core go down this path.

@yurikoles Please ease off on the feature request issues. We're trying to remove options rather than add more so I don't see us adding more DSLs around it, sorry.

@MikeMcQuaid sorry for disturbing.

@yurikoles That's ok, don't worry about it 😄


predicted labels should read "Bird" and not "Tree" by default when use_bird_release()

I added a test and updated main.use_bird_release() to make sure that the bird label is automatically assigned instead of "Tree" default.

@henrykironde I cleaned up the change in the docs. This should be ready to go.

Looks good to me, Can you run yapf -i --recursive deepforest/ --style=.style.yapf  and commit amend to clean up this code
deepforest/main.py	(reformatted)
117
         self.config["score_thresh"] = 0.3

         # Set label dictionary to Bird
-        self.label_dict = {"Bird":0}
+        self.label_dict = {"Bird": 0}

I added another commit since it gets squashed when merged into main.


Change config format to recommendations example

Provides a config.php file in the root of the project, that can be used or merged with other defaults by the application

(Will probably never build because the master is still on Symfony rather than Silex)


One window has low fps

Hi
When I run the game in split screen, the first window has a lot of fps, the second window has 10-12
If I click on the second window with the mouse, then the fps changes to the second window

@Blackubic Always check the FAQ before asking questions:
https://www.splitscreen.me/docs/faq#25--why-are-my-in-game-frames-per-second-lowbetter-in-one-instance-than-in-the-others-when-using-nucleus-co-op


Allow overriding the way json parsing is done

Allow overriding the way json parsing is done. Example usage:
class JSONRPC_Server(jsonrpc_requests.Server):

    # override the loads method:
    def loads(self, response):
        return response.json(parse_float = decimal.Decimal)

This will make requests pass parse_float = decimal.Decimal to json.loads, which will make it parse all floats into decimal.Decimals rather than normal floats. This in turn makes it possible to avoid the very real rounding/precision errors that otherwise happens with returned bitcoin values when converting to satoshis (for example with decoderawtransaction)

This would be very useful to make it easily interoperable with pymongo's Extended Json, too.


MIDI unlearn an automation is not implemented

The MIDI unlearn function for an automation does not work. I just forgot tom implements the function as well as the test... dumb.

Fixed.


Update Jenkinsfile

seems a comment is not right?

I signed it!

Thanks for the contribution!
The canary in this case is in fact in the production namespace which allows it to be tested with the real production traffic.

Thanks for the contribution!
The canary in this case is in fact in the production namespace which allows it to be tested with the real production traffic.

I changed one reference from canary to production, but per your response, perhaps both comments should now say, "Change deployed image in production to the one we just built"


quiltmc trying to use internals of fabric loader

i was trying to install fabric api to play with some mods and i got this error
A mod crashed on startup! net.fabricmc.loader.impl.FormattedException: java.lang.RuntimeException: Could not execute entrypoint stage 'preLaunch' due to errors, provided by 'adventure-platform-fabric'! at net.fabricmc.loader.impl.launch.knot.Knot.init(Knot.java:164) at net.fabricmc.loader.impl.launch.knot.Knot.launch(Knot.java:68) at net.fabricmc.loader.impl.launch.knot.KnotClient.main(KnotClient.java:23) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) at java.base/java.lang.reflect.Method.invoke(Method.java:568) at crystallauncher.MinecraftConstructor.run(MinecraftConstructor.java:29) at crystallauncher.MineClient.start(MineClient.java:213) at crystallauncher.MineClient.main(MineClient.java:36) Caused by: java.lang.RuntimeException: Could not execute entrypoint stage 'preLaunch' due to errors, provided by 'adventure-platform-fabric'! at net.fabricmc.loader.impl.entrypoint.EntrypointUtils.lambda$invoke0$0(EntrypointUtils.java:51) at net.fabricmc.loader.impl.util.ExceptionUtil.gatherExceptions(ExceptionUtil.java:33) at net.fabricmc.loader.impl.entrypoint.EntrypointUtils.invoke0(EntrypointUtils.java:49) at net.fabricmc.loader.impl.entrypoint.EntrypointUtils.invoke(EntrypointUtils.java:35) at net.fabricmc.loader.impl.launch.knot.Knot.init(Knot.java:162) ... 9 more Suppressed: java.lang.ExceptionInInitializerError at org.quiltmc.qsl.command.impl.PreLaunchInitializer.onPreLaunch(PreLaunchInitializer.java:33) at net.fabricmc.loader.impl.entrypoint.EntrypointUtils.invoke0(EntrypointUtils.java:47) ... 11 more Caused by: java.lang.RuntimeException: Failed to load Classloader fields at org.quiltmc.qsl.base.impl.PreLaunchHacks.(PreLaunchHacks.java:49) ... 13 more Caused by: java.lang.NoSuchMethodException: net.fabricmc.loader.impl.launch.knot.KnotClassLoader.addURL(java.net.URL) at java.base/java.lang.Class.getMethod(Class.java:2227) at org.quiltmc.qsl.base.impl.PreLaunchHacks.(PreLaunchHacks.java:46) ... 13 more Caused by: java.lang.ExceptionInInitializerError at java.base/java.lang.Class.forName0(Native Method) at java.base/java.lang.Class.forName(Class.java:467) at net.fabricmc.loader.impl.util.DefaultLanguageAdapter.create(DefaultLanguageAdapter.java:50) at net.fabricmc.loader.impl.entrypoint.EntrypointStorage$NewEntry.getOrCreate(EntrypointStorage.java:117) at net.fabricmc.loader.impl.entrypoint.EntrypointContainerImpl.getEntrypoint(EntrypointContainerImpl.java:53) at net.fabricmc.loader.impl.entrypoint.EntrypointUtils.invoke0(EntrypointUtils.java:47) ... 11 more Caused by: java.lang.RuntimeException: Failed to load Classloader fields at net.kyori.adventure.platform.fabric.impl.AdventurePrelaunch.(AdventurePrelaunch.java:48) ... 17 more Caused by: java.lang.NoSuchMethodException: net.fabricmc.loader.impl.launch.knot.KnotClassLoader.addURL(java.net.URL) at java.base/java.lang.Class.getMethod(Class.java:2227) at net.kyori.adventure.platform.fabric.impl.AdventurePrelaunch.(AdventurePrelaunch.java:45) ... 17 more
so i went to the fabric api discussion thing and asked for help
i was told that quiltmc was trying to use fabric loader's internals that are removed on the latest version and that i needed to report it
Caused by: java.lang.NoSuchMethodException: net.fabricmc.loader.impl.launch.knot.KnotClassLoader.addURL(java.net.URL) at java.base/java.lang.Class.getMethod(Class.java:2227) at org.quiltmc.qsl.base.impl.PreLaunchHacks.(PreLaunchHacks.java:46)
help pls

i deleted the mods that used fabric api to try to make it work

Just for clarification: You're using quilt-loader, not fabric-loader?
If you are, try replacing fabric-api with quilt-standard-libraries (with bundled quilted-fabric-api): https://modrinth.com/mod/qsl

im using fabric loader, im here because of a comment on a discussion at fabric api's discussion thing

The problem is that quilt-standard-libraries no longer works on fabric. And one of your mods appears to have qsl bundled. You just have to figure out which one it is and remove it.
Alternatively, you could ofc try and run your pack on quilt loader, with the newest qsl from modrinth.

i removed some mods that use fabric + quilt and it worked, it was ok zoomer, thanks for the help!