07/05/2018 00:56 : Create Pull Request for branch master -> prTesting1_2018070500

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


return error in na_ofi_cq_read

let na_ofi_cq_read return NA_PROTOCOL_ERROR instead of NA_SUCCESS if the
error code is FI_EIO.
Signed-off-by: Yulu Jia<EMAIL_ADDRESS>

thanks! I agree it should return that. I can't remember now though if there was any specific reason for not returning NA_PROTOCOL_ERROR, have you checked with @liuxuezhao ?

BTW, for your error handling (in cart or IOF), simply relying on the return code from HG_Forward() for error handling probably not enough, in some case the fi_tsend()'s failure cannot be catched, better to check if its completion callback is called.

Fixed in b4e7cb221ec3cdde1f233a3e9febe16263552136


Remove Amount::MAX and replace it with the value in MAX_MONEY

#3688 is quite a convincing argument for limiting the value of Amount to MAX_MONEY.  Also we have #620, which I've read twice in the past and don't want to read again because its so long but I think if we do the following we can close both issues

Make Amount::MAX equal to Amount::MAX_MONEY
Deprecate/remove max_money ?
Enforce the invariant in all addition and multiplication functions

If someone has the patience to read through #620 again and check I haven't forgotten anything that would be great.

Enforce the invariant in all addition and multiplication functions

Can this be done independently from adding something like UncheckedAmount at the same time? In other words, shouldn't this task (making MAX MAX_MONEY) be part of a broader task to split Amount into two (one enforcing invariants, one not)? Or does this task assume using i64 for arithmetic operations that may exceed 21M?

@jirijakes
This was discussed here: https://github.com/rust-bitcoin/rust-bitcoin/issues/620 where we discussed that very question.

This was discussed here: #620 where we discussed that very question.

Without a conclusion, right? So I am wondering whether the point I quoted (enforce 21M invariant in Amount) assumes anything about that or not. In other words, after this task is closed as finished, what shall I do if I need to add bitcoin amounts together when I expect them to exceed 21M?

Convert them to plain old u64 and add them? Or said another way, what does adding UncheckedAmount provide if the type is not used anywhere in the rust-bitcoin API?

Convert them to plain old u64 and add them?

Thanks, that answers the question I had.

Or said another way, what does adding UncheckedAmount provide if the type is not used anywhere in the rust-bitcoin API?

I think there are some benefits but this issue is not the place to discuss them.

Cool, agreed with all three suggestions in the OP.
Also agreed that UncheckedAmount does not have a clear benefit over u64 and that it's definitely not needed for the MAX_MONEY thing, since users can always just use u64 (or u128 or whatever).


DIA-NN 1..8.1 vs 1.9.2 performance benchmark

PXD026600 was selected as benchmark datasets (narrow window and wide window). We compared peptides and proteins identified at q-value 0.01. The peptide number has increased a little. The number of protein quantified essentially the same. It is likely that large-scale data will show the difference better, according to DIA-NN document

@daichengxin thanks a lot for this great benchmark and been so quick. Im wondering the following:

for the new version of DIANN are we using all the parameters in the right way?
Are we losing some features/ids because of the parallelisation?

Im tagging @vdemichev in case he can help to find out if these are the expected results.

Seems fine I guess

We further tested the large-scale PXD030304 datasets using quantms with DIA-NN 1.8.1 and DIA-NN 1.9.2, and comparing with original paper results.
Identified #PSM

1.8.1
1.9.2
original results

252015828
285336801
204148465

Identified PSM at Global.Q.value < 0.01:

quantified peptides:

quantified proteins:

The quantms with DIA-NN 1.8.1 results seems to be ok and intersects well with the original results as well at peptide level. The weird thing is that 1.9.2 overlap is poor at PSM and peptide level although it identifies to more psm and peptides. Is it a change in the scan of the input spectra or something else?

Any warnings printed by 1.9.2?

@daichengxin we shouldn't compute PSMs only features, please remove the PSMs concept because PSMs are difficult to figure out in the original results.
I would be interested to understand the differences in protein IDs between original results and DIANN 1.9.2.


[HOLD] Feature: Update participant creation [LEI-247]

Refs: https://openscience.atlassian.net/browse/LEI-247
Purpose

Simplify ids by using numbers instead of alphanumeric ids
Only create unique ids that do not already exist to prevent an error when attempting to create accounts

Summary of changes

Make generated ids use only numbers
Check to make sure no ids are duplicated in the same batch, and that the generated ids do not match an existing participant id

Will resume after https://github.com/CenterForOpenScience/experimenter/pull/120 goes in


Add missing update of Conditions description in YAMLs

Signed-off-by: Carlo Lobrano<EMAIL_ADDRESS>

We already have some, or did you mean a different thing?

e2e
package tests

I mean a test for missing manifest updates.
Something along  make generate manifests bundle verify-no-changes (didn't check Makefile / test config on what's the best way to add this)

I mean a test for missing manifest updates. Something along make generate manifests bundle verify-no-changes (didn't check Makefile / test config on what's the best way to add this)

ah got it! Sure I'll add it in a follow up

/hold cancel

/retest


func only matches kretprobe_trampoline on kretprobes

# bpftrace -e 'kretprobe:vfs_* { @[func] = count(); }'
Attaching 53 probes...
cannot attach kprobe, Invalid argument
Warning: could not attach probe kretprobe:vfs_dedupe_get_page.isra.21, skipping.
^C

@[kretprobe_trampoline]: 438

Would be nice if func could match the function on kretprobe, intsead of only ever returning kretprobe_trampoline. If that's not possible, it should be documented in the reference guide.

#101 is a separate issue -- currently func is implemented by fetching the pc and unlike with kretprobe stacks I think it would be quite ugly to fake pc as well in-kernel (maybe it's not even practical to do this). So maybe bpftrace should instead get the stack but only the first entry? I'm not sure how much of an impact on the BPF code size this would be though...

I think bpf_get_stack (as opposed to _get_stackid) would let us just fetch the top-most entry of the stack...


Cannot select items while keyboard open

I'm using multiselect inside scrollview and when keyboard is opened to search multiselect items and when i select item, first it closes the keyboard then only i can select. how can i select items while keyboard is open? im also added keyboardShouldPersistTaps='handled' in scrollview. still the same. any fix for this?

flatListProps={{keyboardShouldPersistTaps: 'handled'}} adding this property fixed the issue.


ENH: HiGHS submodule and update

Reference issue

What does this implement/fix?

remove HiGHS source tree from the SciPy repository
create and use a fork of the official HiGHS repository for easier upstream/downstream change management
modifications to Cython wrappers to catch up to the current HiGHS API

includes changes to pxd files to add/update data members and class interfaces

Additional information

Inclusion of submodules in SciPy was not yet standard practice when HiGHS was first integrated.  It was discussed but ultimately a vendor approach won out.  With the inclusion of Boost, UNURAN, and PROPACK submodules, it makes sense to move HiGHS to a submodule as well, especially given the development pace of the project and the unreasonable review burden that would be put on SciPy maintainers to keep an up-to-date version.
Blockers:

Auto write of High.log to working directory, see https://github.com/ERGO-Code/HiGHS/issues/586
Moving https://github.com/mckib2/highs into SciPy's org

@mdhaber Building and passing tests on Linux, will try Windows/Mac hopefully tomorrow evening.  Please take a look and suggest how we can make this as easy as possible for SciPy reviewers

Please take a look and suggest how we can make this as easy as possible for SciPy reviewers

Deletions make sense, but maybe add brief comments in the PR about what the new code is for?

@mdhaber I've struggled for a few hours now trying to get Windows to run for me.  It's building fine with the latest changes, but I get the following error when trying to run tests:
ImportError while importing test module 'C:\Users\nmckibben\Documents\scipy\scipy\optimize\tests\test__linprog_clean_inputs.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
C:\Python310\lib\importlib\__init__.py:126: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
scipy\optimize\__init__.py:399: in <module>
    from ._optimize import *
scipy\optimize\_optimize.py:33: in <module>
    from scipy.sparse.linalg import LinearOperator
scipy\sparse\linalg\__init__.py:120: in <module>
    from ._isolve import *
scipy\sparse\linalg\_isolve\__init__.py:6: in <module>
    from .lgmres import lgmres
scipy\sparse\linalg\_isolve\lgmres.py:7: in <module>
    from scipy.linalg import get_blas_funcs
scipy\linalg\__init__.py:198: in <module>
    from ._misc import *
scipy\linalg\_misc.py:4: in <module>
    from .lapack import get_lapack_funcs
scipy\linalg\lapack.py:818: in <module>
    from scipy.linalg import _flapack
E   ImportError: DLL load failed while importing _flapack: The operating system cannot run %1.

https://github.com/scipy/scipy/issues/13717 seems related -- I'm not building with clang-cl or Pythran and I'm getting a different error than reported there, but maybe the compiler is the issue.  I'm not sure at this point.  I always forget how fun it is to set Windows up for the first time...

: ( Sorry to hear that. Do you want me to test whether I get the same error?

: ( Sorry to hear that. Do you want me to test whether I get the same error?

Sure, that would be helpful.  Any hints on how to get clang-cl to be used instead of cl?  I have both MSVC BuildTools, mingw, and llvm installed (all through chocolately)

You probably saw my comments starting here. I really didn't know what I was doing. My strategy to use one was to force-uninstall the other.

I have SCIPY_USE_PYTHRAN=0 right now because ASV was having trouble with it being 1. I get an error about CMakeLists.txt.

(scipydev) C:\Users\matth\scipy>git clean -dxf
Skipping repository benchmarks/env/43aced745fbfea41add6be22ac8298b4/project
Removing site.cfg

(scipydev) C:\Users\matth\scipy>python setup.py build_ext --inplace
Cythonizing sources
Running scipy\linalg\_generate_pyx.py
Running scipy\special\_generate_pyx.py
Running scipy\stats\_generate_pyx.py
Processing scipy\cluster\_hierarchy.pyx
Processing scipy\cluster\_optimal_leaf_ordering.pyx
Processing scipy\cluster\_vq.pyx
Processing scipy\fftpack\convolve.pyx
Processing scipy\interpolate\interpnd.pyx
Processing scipy\interpolate\_bspl.pyx
Processing scipy\interpolate\_ppoly.pyx
Processing scipy\io\matlab\_mio5_utils.pyx
Processing scipy\io\matlab\_mio_utils.pyx
Processing scipy\io\matlab\_streams.pyx
Processing scipy\linalg\cython_blas.pyx
Processing scipy\linalg\cython_lapack.pyx
Processing scipy\linalg\_cythonized_array_utils.pyx
Processing scipy\linalg\_decomp_update.pyx.in
Processing scipy\linalg\_matfuncs_sqrtm_triu.pyx
Processing scipy\linalg\_solve_toeplitz.pyx
Processing scipy\ndimage\src\_cytest.pyx
Processing scipy\ndimage\src\_ni_label.pyx
Processing scipy\optimize\_bglu_dense.pyx
Processing scipy\optimize\_group_columns.pyx
Processing scipy\optimize\cython_optimize\_zeros.pyx.in
Processing scipy\optimize\tnc\moduleTNC.pyx
Processing scipy\optimize\_highs\cython\src\_highs_constants.pyx
Processing scipy\optimize\_highs\cython\src\_highs_wrapper.pyx
Processing scipy\optimize\_lsq\givens_elimination.pyx
Processing scipy\optimize\_trlib\_trlib.pyx
Processing scipy\signal\_max_len_seq_inner.pyx
Processing scipy\signal\_peak_finding_utils.pyx
Processing scipy\signal\_sosfilt.pyx
Processing scipy\signal\_spectral.pyx
Processing scipy\signal\_upfirdn_apply.pyx
Processing scipy\sparse\_csparsetools.pyx.in
Processing scipy\sparse\csgraph\_flow.pyx
Processing scipy\sparse\csgraph\_matching.pyx
Processing scipy\sparse\csgraph\_min_spanning_tree.pyx
Processing scipy\sparse\csgraph\_reordering.pyx
Processing scipy\sparse\csgraph\_shortest_path.pyx
Processing scipy\sparse\csgraph\_tools.pyx
Processing scipy\sparse\csgraph\_traversal.pyx
Processing scipy\spatial\_ckdtree.pyx
Processing scipy\spatial\_hausdorff.pyx
Processing scipy\spatial\_qhull.pyx
Processing scipy\spatial\_voronoi.pyx
Processing scipy\spatial\transform\_rotation.pyx
Processing scipy\special\cython_special.pyx
Processing scipy\special\_comb.pyx
Processing scipy\special\_ellip_harm_2.pyx
Processing scipy\special\_test_round.pyx
Processing scipy\special\_ufuncs.pyx
Processing scipy\special\_ufuncs_cxx.pyx
Processing scipy\stats\_biasedurn.pyx
Processing scipy\stats\_qmc_cy.pyx
Processing scipy\stats\_sobol.pyx
Processing scipy\stats\_stats.pyx
Processing scipy\stats\_boost\src\beta_ufunc.pyx
Processing scipy\stats\_boost\src\binom_ufunc.pyx
Processing scipy\stats\_boost\src\hypergeom_ufunc.pyx
Processing scipy\stats\_boost\src\nbinom_ufunc.pyx
Processing scipy\stats\_unuran\unuran_wrapper.pyx
Processing scipy\_lib\messagestream.pyx
Processing scipy\_lib\_ccallback_c.pyx
Processing scipy\_lib\_test_deprecation_call.pyx
Processing scipy\_lib\_test_deprecation_def.pyx
Running from SciPy source directory.
lapack_opt_info:
lapack_mkl_info:
No module named 'numpy.distutils._msvccompiler' in numpy.distutils; trying from distutils
customize MSVCCompiler
  libraries mkl_rt not found in ['C:/Users/matth/Anaconda3/envs/scipydev\\Library\\lib']
  NOT AVAILABLE

openblas_lapack_info:
  libraries openblas not found in ['C:\\opt\\64\\lib']
get_default_fcompiler: matching types: '['gnu', 'intelv', 'absoft', 'compaqv', 'intelev', 'gnu95', 'g95', 'intelvem', 'intelem', 'flang']'
customize GnuFCompiler
Could not locate executable g77
Could not locate executable f77
customize IntelVisualFCompiler
Could not locate executable ifort
Could not locate executable ifl
customize AbsoftFCompiler
Could not locate executable f90
customize CompaqVisualFCompiler
Could not locate executable DF
customize IntelItaniumVisualFCompiler
Could not locate executable efl
customize Gnu95FCompiler
Found executable C:\msys64\mingw64\bin\gfortran.exe
Using built-in specs.
COLLECT_GCC=C:\msys64\mingw64\bin\gfortran.exe
COLLECT_LTO_WRAPPER=C:/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.2.0/lto-wrapper.exe
Target: x86_64-w64-mingw32
Configured with: ../gcc-10.2.0/configure --prefix=/mingw64 --with-local-prefix=/mingw64/local --build=x86_64-w64-mingw32 --host=x86_64-w64-mingw32 --target=x86_64-w64-mingw32 --with-native-system-header-dir=/mingw64/x86_64-w64-mingw32/include --libexecdir=/mingw64/lib --enable-bootstrap --with-arch=x86-64 --with-tune=generic --enable-languages=c,lto,c++,fortran,ada,objc,obj-c++ --enable-shared --enable-static --enable-libatomic --enable-threads=posix --enable-graphite --enable-fully-dynamic-string --enable-libstdcxx-filesystem-ts=yes --enable-libstdcxx-time=yes --disable-libstdcxx-pch --disable-libstdcxx-debug --disable-isl-version-check --enable-lto --enable-libgomp --disable-multilib --enable-checking=release --disable-rpath --disable-win32-registry --disable-nls --disable-werror --disable-symvers --disable-plugin --with-libiconv --with-system-zlib --with-gmp=/mingw64 --with-mpfr=/mingw64 --with-mpc=/mingw64 --with-isl=/mingw64 --with-pkgversion='Rev5, Built by MSYS2 project' --with-bugurl=https://github.com/msys2/MINGW-packages/issues --with-gnu-as --with-gnu-ld --with-boot-ldflags='-pipe -Wl,--dynamicbase,--high-entropy-va,--nxcompat,--default-image-base-high -Wl,--disable-dynamicbase -static-libstdc++ -static-libgcc' 'LDFLAGS_FOR_TARGET=-pipe -Wl,--dynamicbase,--high-entropy-va,--nxcompat,--default-image-base-high' --enable-linker-plugin-flags='LDFLAGS=-static-libstdc++\ -static-libgcc\ -pipe\ -Wl,--dynamicbase,--high-entropy-va,--nxcompat,--default-image-base-high\ -Wl,--stack,12582912'
Thread model: posix
Supported LTO compression algorithms: zlib zstd
gcc version 10.2.0 (Rev5, Built by MSYS2 project)
  libraries openblas not found in ['C:\\opt\\64\\lib']
Library openblas was not found. Ignoring
  libraries openblas not found in ['C:\\opt\\64\\lib']
Runtime library openblas was not found. Ignoring
  FOUND:
    library_dirs = ['C:\\Users\\matth\\scipy\\build\\openblas_lapack_info']
    libraries = ['openblas_lapack_info']
    language = f77
    define_macros = [('HAVE_CBLAS', None)]

  FOUND:
    library_dirs = ['C:\\Users\\matth\\scipy\\build\\openblas_lapack_info']
    libraries = ['openblas_lapack_info']
    language = f77
    define_macros = [('HAVE_CBLAS', None)]

blas_opt_info:
blas_mkl_info:
  libraries mkl_rt not found in ['C:/Users/matth/Anaconda3/envs/scipydev\\Library\\lib']
  NOT AVAILABLE

blis_info:
  libraries blis not found in ['C:/Users/matth/Anaconda3/envs/scipydev\\Library\\lib']
  NOT AVAILABLE

openblas_info:
  libraries openblas not found in ['C:\\opt\\64\\lib']
get_default_fcompiler: matching types: '['gnu', 'intelv', 'absoft', 'compaqv', 'intelev', 'gnu95', 'g95', 'intelvem', 'intelem', 'flang']'
customize GnuFCompiler
customize IntelVisualFCompiler
customize AbsoftFCompiler
customize CompaqVisualFCompiler
customize IntelItaniumVisualFCompiler
customize Gnu95FCompiler
Using built-in specs.
COLLECT_GCC=C:\msys64\mingw64\bin\gfortran.exe
COLLECT_LTO_WRAPPER=C:/msys64/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/10.2.0/lto-wrapper.exe
Target: x86_64-w64-mingw32
Configured with: ../gcc-10.2.0/configure --prefix=/mingw64 --with-local-prefix=/mingw64/local --build=x86_64-w64-mingw32 --host=x86_64-w64-mingw32 --target=x86_64-w64-mingw32 --with-native-system-header-dir=/mingw64/x86_64-w64-mingw32/include --libexecdir=/mingw64/lib --enable-bootstrap --with-arch=x86-64 --with-tune=generic --enable-languages=c,lto,c++,fortran,ada,objc,obj-c++ --enable-shared --enable-static --enable-libatomic --enable-threads=posix --enable-graphite --enable-fully-dynamic-string --enable-libstdcxx-filesystem-ts=yes --enable-libstdcxx-time=yes --disable-libstdcxx-pch --disable-libstdcxx-debug --disable-isl-version-check --enable-lto --enable-libgomp --disable-multilib --enable-checking=release --disable-rpath --disable-win32-registry --disable-nls --disable-werror --disable-symvers --disable-plugin --with-libiconv --with-system-zlib --with-gmp=/mingw64 --with-mpfr=/mingw64 --with-mpc=/mingw64 --with-isl=/mingw64 --with-pkgversion='Rev5, Built by MSYS2 project' --with-bugurl=https://github.com/msys2/MINGW-packages/issues --with-gnu-as --with-gnu-ld --with-boot-ldflags='-pipe -Wl,--dynamicbase,--high-entropy-va,--nxcompat,--default-image-base-high -Wl,--disable-dynamicbase -static-libstdc++ -static-libgcc' 'LDFLAGS_FOR_TARGET=-pipe -Wl,--dynamicbase,--high-entropy-va,--nxcompat,--default-image-base-high' --enable-linker-plugin-flags='LDFLAGS=-static-libstdc++\ -static-libgcc\ -pipe\ -Wl,--dynamicbase,--high-entropy-va,--nxcompat,--default-image-base-high\ -Wl,--stack,12582912'
Thread model: posix
Supported LTO compression algorithms: zlib zstd
gcc version 10.2.0 (Rev5, Built by MSYS2 project)
  libraries openblas not found in ['C:\\opt\\64\\lib']
Library openblas was not found. Ignoring
  libraries openblas not found in ['C:\\opt\\64\\lib']
Runtime library openblas was not found. Ignoring
  FOUND:
    library_dirs = ['C:\\Users\\matth\\scipy\\build\\openblas_info']
    libraries = ['openblas_info']
    language = f77
    define_macros = [('HAVE_CBLAS', None)]

  FOUND:
    library_dirs = ['C:\\Users\\matth\\scipy\\build\\openblas_info']
    libraries = ['openblas_info']
    language = f77
    define_macros = [('HAVE_CBLAS', None)]

Traceback (most recent call last):
  File "C:\Users\matth\scipy\setup.py", line 622, in <module>
    setup_package()
  File "C:\Users\matth\scipy\setup.py", line 618, in setup_package
    setup(**metadata)
  File "C:\Users\matth\Anaconda3\envs\scipydev\lib\site-packages\numpy\distutils\core.py", line 135, in setup
    config = configuration()
  File "C:\Users\matth\scipy\setup.py", line 528, in configuration
    config.add_subpackage('scipy')
  File "C:\Users\matth\Anaconda3\envs\scipydev\lib\site-packages\numpy\distutils\misc_util.py", line 1016, in add_subpackage
    config_list = self.get_subpackage(subpackage_name, subpackage_path,
  File "C:\Users\matth\Anaconda3\envs\scipydev\lib\site-packages\numpy\distutils\misc_util.py", line 982, in get_subpackage
    config = self._get_configuration_from_setup_py(
  File "C:\Users\matth\Anaconda3\envs\scipydev\lib\site-packages\numpy\distutils\misc_util.py", line 924, in _get_configuration_from_setup_py
    config = setup_module.configuration(*args)
  File "scipy\setup.py", line 18, in configuration
    config.add_subpackage('optimize')
  File "C:\Users\matth\Anaconda3\envs\scipydev\lib\site-packages\numpy\distutils\misc_util.py", line 1016, in add_subpackage
    config_list = self.get_subpackage(subpackage_name, subpackage_path,
  File "C:\Users\matth\Anaconda3\envs\scipydev\lib\site-packages\numpy\distutils\misc_util.py", line 982, in get_subpackage
    config = self._get_configuration_from_setup_py(
  File "C:\Users\matth\Anaconda3\envs\scipydev\lib\site-packages\numpy\distutils\misc_util.py", line 924, in _get_configuration_from_setup_py
    config = setup_module.configuration(*args)
  File "scipy\optimize\setup.py", line 130, in configuration
    config.add_subpackage('_highs')
  File "C:\Users\matth\Anaconda3\envs\scipydev\lib\site-packages\numpy\distutils\misc_util.py", line 1016, in add_subpackage
    config_list = self.get_subpackage(subpackage_name, subpackage_path,
  File "C:\Users\matth\Anaconda3\envs\scipydev\lib\site-packages\numpy\distutils\misc_util.py", line 982, in get_subpackage
    config = self._get_configuration_from_setup_py(
  File "C:\Users\matth\Anaconda3\envs\scipydev\lib\site-packages\numpy\distutils\misc_util.py", line 924, in _get_configuration_from_setup_py
    config = setup_module.configuration(*args)
  File "scipy\optimize\_highs\setup.py", line 64, in configuration
    _major_dot_minor = _get_version(
  File "scipy\optimize\_highs\setup.py", line 51, in _get_version
    with open(CMakeLists, 'r', encoding='utf-8') as f:
FileNotFoundError: [Errno 2] No such file or directory: 'C:\\Users\\matth\\scipy\\scipy\\_lib\\highs\\CMakeLists.txt'

@mdhaber Did you pull the new highs submodule?  Take a look in scipy/_lib/highs to make sure it's not empty

Hah of course not : P
No errors for me!

Great news, though I'm still not getting anywhere on my Windows build.  I'm going to try uninstalling everything and starting fresh with Anaconda

Just verified everything runs on MacOS and for now I trust you that it works on Windows.  Are we ready to open a PR against the main scipy repo?

Almost. Just review the diff for unintended changes (e.g. there are changes to stats at the bottom that don't look related) first. Then when you post, please add those comments about changes (other than deletions) and why they are necessary. I'd avoid complicating this PR with unnecessary changes.

Almost. Just review the diff for unintended changes (e.g. there are changes to stats at the bottom that don't look related) first. Then when you post, please add those comments about changes (other than deletions) and why they are necessary. I'd avoid complicating this PR with unnecessary changes.

I figured out what was happening -- apparently I had extra commits in my fork's master branch which was causing "dirty" diffs.  Rebasing everything back onto the upstream master seems to have resolved it


NVHPC not respecting startup checks

From #102

@wilfonba
Compiling with GNU shows the real problem:
"Unsupported combination of precision and parallel IO. Please use precision == 2 when enabling parallel_io. Exiting ..."
It appears it's simply not been implemented yet. NVHPC doesn't show these error messages for checking the input files.

@sbryngelson
This signals a more significant problem. The GNU compilers are respecting our m_start_up.f90 checks (where this error comes from), but NVHPC compilers are not.

Interesting, I just tested this on Phoenix with NVHPC and it did catch the error:

Looking at the code, there aren't really any pre-processor directives associated with GNU/NVHPC compilers when calling m_start_up. The only conditional I see is checking if proc_rank == 0 before calling s_check_input_file().
@wilfonba I did notice that the error message is sandwiched in between MPI error messages. Perhaps it did print but went unnoticed?

I think what's actually happening on Summit is that MPI_abort() is called before the print statement is flushed to stdout. I know in C/C++ a simple fix for this would be to call "fflush(stdout)" after every print statement in the input checking code, and an equivalent solution would be very easy to add to MFC if it exists. I think "call flush()" before every "call MPI_Abort()" in these input-checking files may be enough to fix this issue. Summit seems to buffer output in different ways than other computers, ie. output is displayed in the terminal every few hundred-time steps in some cases. I'll take a look at "call flush()" and its effect this evening. @anshgupta1234 @sbryngelson

Interesting -- I did not know about this. Would an mpi_barrier() of some kind fix it? Meaning that one of the MPI ranks is getting to mpi_abort() before the print statement, so nothing is printing. If you put a barrier before the print statement, it would have to wait, then print, then abort. I assume this is somewhere in the actual s_mpi_abort() subroutine in m_mpi_common.

Ok so what if changed this routine:
https://github.com/MFlowCode/MFC/blob/d929c4b1f40633330314bc87ee22495144acfe3f/src/common/m_mpi_common.fpp#L313-L326
to instead have an optional character argument that prints before aborting?
something like
subroutine s_mpi_abort(prnt)
	character(len=*), intent(in), optional :: prnt

    if (present(prnt)) then
        print*, print
    end if

#ifndef MFC_MPI
        stop 1
#else
		! wait for MPI ranks!!
		call MPI_Barrier(MPI_COMM_WORLD, ierr)
        ! Terminating the MPI environment
        call MPI_ABORT(MPI_COMM_WORLD, 1, ierr)
#endif
    end subroutine s_mpi_abort

However, thinking more about this, I don't know why the PRINT statement already in MFC pre/post process before s_mpi_abort() doesn't print for every MPI rank already (meaning my hypothesis could be wrong).
https://github.com/MFlowCode/MFC/blob/d929c4b1f40633330314bc87ee22495144acfe3f/src/pre_process/m_start_up.fpp#L217-L221C13
For example, for a 128 core job it seems like it should print that statement 128 times as present, and I'm not sure why it doesn't. Regardless,

If there's one thing I know for sure, it's the above change would shorten the code length!

@sbryngelson I don't think you need to wait for all MPI ranks but it might be useful if MFC does some clean up operations beforehand and you want to make sure that they complete. In that case, I would add if (proc_rank .eq. 0) before printing. As an extra caution, I would still add an fflush just to make sure.

@henryleberre agreed.

@henryleberre Wouldn't adding if (proc_rank == 0) cause problems if this function is called from a different rank?

@wilfonba Indeed, just like the MPI_Barrier it would cause problems if this wasn't called on all ranks.. One solution is to have a timeout. It depends how we want to use s_mpi_abort.

@sbryngelson the pull request I made currently works, as far as I can tell. Most of the calls to s_mpi_abort are in m_start_up files which are only ran on one processor, so dealing with MPI_ranks isn't necessary.


performance with index

Medici is slow when the number of records reach 30k

I think the Readme is misleading.
I'm assuming there is no performance issue over 30k record when running index.
Is that correct?
related to #12

Thanks! Readme got updated. :)
Is it better now?


add 1.24 ad-test

What this PR does / why we need it:
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Codecov Report

Merging #1173 (dd9e55c) into main (0e836a1) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@           Coverage Diff           @@
##             main    #1173   +/-   ##
=======================================
  Coverage   38.98%   38.98%
=======================================
  Files          25       25
  Lines        1980     1980
=======================================
  Hits          772      772
  Misses       1106     1106
  Partials      102      102

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add docker.options support - fixed #32

Fixes: #32
When using this extension with a linux docker image, the file permissions
are not always correct. So there is a need to specify the docker options
to set the user-id and group-id.
This change adds the following configuration options:

pandoc.docker.enabled: Replaces the existing pandoc.useDocker. Any
existing configurations using the old value will be migrated over and
the old value is deprecated. Warning will be shown when migrating.
pandoc.docker.options: Set this to provide options to the docker run
command. As an example you can use --user $(id -u):$(id -g) to set
the correct user-id and group-id.
pandoc.docker.image: Can be used to override the default docker image
to a specified value.

@raven42 I don't quite have the setup to test this properly but I fixed up some TypeScript issues and built and ran and got the migration messages as I never used the extension with a container. Changes OK with you?

@raven42 I don't quite have the setup to test this properly but I fixed up some TypeScript issues and built and ran and got the migration messages as I never used the extension with a container. Changes OK with you?

I'm not sure that would work as intended. It was using the same globalValue lookup for each if check. I pushed up another change to differentiate between the global config, workspace config, and folder config. Does that look correct?

@raven42 I don't really understand the statement. Sorry, this was to fix typescript type complaints, I don't think it changed the functionality, did it?

@raven42 Although it looks like whatever you added in the last commits kept the type fixes, so if you're happy with the combination of changes then I am happy!

is it master or main?

@sharifm-informatica it should be main, I'm not sure when master crept back

I suspected an issue. I cannot find that new code merged anywhere in this repo at the moment. Furthermore, I did not find it in release 4.7 either. Many thanks for the great work it changed many lives or at least workflows.

I cannot find the code in this pull request anywhere. How can I help.

@raven42 @sharifm-informatica Oh dear I don't know what kind of mess I created there, re-merged and pushed a new release with the code.

@raven42 Honestly I have no idea what happened, but we're all good now :) Thanks for the contribution!


Removing traling spaces from end of lines.

This is useful because when one is copying lines from term.js all those   characters are also copied and what you will see when you paste is a bunch of useless spaces which are usually undesired. This patch fixes this issue by removing traling spaces when we are constructing the lines. There is a caveat for empty lines. We need to leave at least one space there or otherwise the output will lose lines.

@chjj Could you please take a look at this? Thanks a bunch.

@chjj That would be great if you could look at this. Thanks.


send email when aggregation update is finished

notify the requester / project owner when the aggregation export has finished uploading

LGTM


Vsphere cpi.yml missing resource_pool

In order to get vms created in the appropriate resource pool we had to make the following change to the vsphere/cpi.yml
      clusters:
      - ((vcenter_cluster)): {}

changed to
      clusters:
      - ((vcenter_cluster)):
          resource_pool: ((vcenter_resource_pool))

We were thinking about making this a pull request since it's a simple change but then the documentation would need to be updated as well at https://bosh.io/docs/init-vsphere.html so maybe it's more appropriate for the bosh team to handle it.

We found an ops file to handle this correctly. We think a bit more documentation around it would be helpful.


Display share indicator when listing files

When a resource (file/folder) has been shared we should show the appropriate share indicator.

@C0rby could you update this issue / close it if it was finished off in the current sprint (sync bot suggests so)

So there are multiple things to this.

Propfinds include shareinformation for direct shares (users,groups)

The indicators for direct shares are displayed for the items in the root and in subfolders

Propfinds to not include share information for public links

Share indicators for public links are not shown in the root but in subfolders
This is because the webui does an extra request in subfolders to receive a list with shares

Including public link share information into the propfind response is a bit difficult since it would require to do a request against the share manager for every file in the propfind response. This would slow down the process significantly which is why we haven't implemented this yet.
If this has a high priority/is a strict requirement than we need to take another look and possibly refactor the sharing architecture.

This looks all solved to me. See also https://github.com/owncloud/ocis/issues/1038#issuecomment-990711232.
Please reopen and describe if there is still a TODO.


fix: Update the QA categorization mappings file

Signed-off-by: Akash Nayak<EMAIL_ADDRESS>

Fixes - https://github.com/konveyor/move2kube/issues/1154


 How do I fix this? I've tried restarting both the background worker and web app with no effect.

Hey @axsuul, that's odd. It'd be nice to see what's happening with that stuck job. Do you see anything in your logs?
How are you running Huginn? You could definitely unstick this by executing delete from delayed_jobs; on your SQL server.

I have the same issue after upgrading to e3f79bf84bb6d2593aef563133edcaa32ae47d38.  After >10 hours event propagation gets stuck.  Restarting huginn machine entirely makes it run for a while.  I see events being processed but then next day it's stuck.
I also added another delayed_job in Procfile (2 in total from 1), but that did not fix the issue.
Nothing seems to be eating up CPU.
I reviewed the logs and this is the only thing that seems relevant - other
I, [2017-04-28T06:48:04.526608 #145]  INFO -- : 2017-04-28T06:48:04-0700: [Worker(host:huginn pid:145)] Error while reserving job: could not obtain a connection from the pool within 5.000 seconds (waited 5.000 seconds); all pooled connections were in use

@budric Try to increase the DATABASE_POOL variable in your .env file. I am using 30 without having issues but that depends a bit on the amount of Agents that are used.

@dsander I modified .env file and ran rake production:export for changes to take effect.  The huginn-jobs-1 could not be stopped - presumably this is the one that's stuck propagating events.  Here's --trace output if that helps pin down the problem in any way.
Aside from restarting the machine it doesn't want to go down.
$>sudo bundle exec rake production:export --trace
** Invoke production:export (first_time)
** Invoke production:check (first_time)
** Execute production:check
** Execute production:export
** Execute production:stop
Stopping huginn ...
ok: down: huginn-web-1: 150s, normally up
rake aborted!
'sv stop huginn-jobs-1' exited with a non-zero return value: timeout: run: huginn-jobs-1: (pid 145) 107999s, want down, got TERM
/home/huginn/huginn/lib/tasks/production.rake:85:in `run'
/home/huginn/huginn/lib/tasks/production.rake:77:in `block (2 levels) in run_sv'
/home/huginn/huginn/lib/tasks/production.rake:93:in `with_retries'
/home/huginn/huginn/lib/tasks/production.rake:76:in `block in run_sv'
/home/huginn/huginn/lib/tasks/production.rake:75:in `each'
/home/huginn/huginn/lib/tasks/production.rake:75:in `run_sv'
/home/huginn/huginn/lib/tasks/production.rake:33:in `block (2 levels) in <top (required)>'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/task.rb:250:in `block in execute'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/task.rb:250:in `each'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/task.rb:250:in `execute'
/home/huginn/huginn/lib/tasks/production.rake:57:in `block (2 levels) in <top (required)>'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/task.rb:250:in `block in execute'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/task.rb:250:in `each'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/task.rb:250:in `execute'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/task.rb:194:in `block in invoke_with_call_chain'
/usr/lib/ruby/2.3.0/monitor.rb:214:in `mon_synchronize'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/task.rb:187:in `invoke_with_call_chain'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/task.rb:180:in `invoke'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/application.rb:152:in `invoke_task'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/application.rb:108:in `block (2 levels) in top_level'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/application.rb:108:in `each'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/application.rb:108:in `block in top_level'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/application.rb:117:in `run_with_threads'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/application.rb:102:in `top_level'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/application.rb:80:in `block in run'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/application.rb:178:in `standard_exception_handling'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/lib/rake/application.rb:77:in `run'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/gems/rake-12.0.0/exe/rake:27:in `<top (required)>'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/bin/rake:23:in `load'
/home/huginn/huginn/vendor/bundle/ruby/2.3.0/bin/rake:23:in `<main>'
Tasks: TOP => production:export

When the process doesn't stop gracefully try: sudo bundle exec rake production:force_stop

@dsander, thanks.  Increasing the DATABASE_POOL to 30 seems to have fixed the problem.  It's been up over the weekend and events are still being propagated.

Nice, let's hope it stays that way :)

I have the same issue as this. I'm running Huginn on Docker, with the following docker-compose.yml file:
version: '3'

services:
  web:
    image: huginn/huginn
    restart: always
    ports:
      - "<IP_ADDRESS>:3000:3000"
    volumes:
      - ./mysql-data:/var/lib/mysql
    env_file: .env

My .env file has the following lines:
PORT=3000
RAILS_ENV=production
FORCE_SSL=true
IMPORT_DEFAULT_SCENARIO_FOR_ALL_USERS=false
SMTP_DELIVERY_METHOD=sendmail
RAILS_ENV=production
DATABASE_POOL=60

DOMAIN=<hidden>
APP_SECRET_TOKEN=<hidden>
INVITATION_CODE=<hidden>
EMAIL_FROM_ADDRESS=<hidden>
TWITTER_OAUTH_KEY=<hidden>
TWITTER_OAUTH_SECRET=<hidden>
GITHUB_OAUTH_KEY=<hidden>
GITHUB_OAUTH_SECRET=<hidden>
DROPBOX_OAUTH_KEY=<hidden>
DROPBOX_OAUTH_SECRET=<hidden>

I'm unable to run bundle exec rake production:export --trace as I get the following:
** Invoke production:export (first_time)
** Invoke production:check (first_time)
** Execute production:check
[ FAIL ] Please install runit:

sudo apt-get install runit

In my jobs page, I get this:

This is the total of my agents, right now:

Any thoughts? Happy to do some basic diagnostics!

I'm having the same issues with the Docker and MariaDB setup.
Increasing the DATABASE_POOL didn't help.
The Docker logs show nothing and inside the Docker container, the existing directory /app/logs is completely empty.
bundle exec rake production:export also exits with [ FAIL ] Please install runit:
Any more ideas?

Same problem here using default docker container (only one container with everything)
Perhaps connecting to an external DB could solve the problem.
Another thing that would be nice would be to add better health checks to the container like: "If there are more than 50 jobs on the queue, the container is unhealthy". Then, the docker swarm should restart it.


0.9.14 changelog & breaking changes

Arch linux package was updated to 0.9.14 which doesn't yet have a changelog under https://github.com/Alexays/Waybar/releases.
The update broke, among others, the ability to format custom plugins like this:

The <b> would previously make the A bold. The plugin is a bash script outputting json.

For me the whole bar looks different:
Old:

New:

See 1753 or 1726

indeed, thanks for pointing out the formatting issue is #1726.
on top of that there are a good amount of additional issues including the unclear release state with an existing git tag landing in downstream packages without a proper release.

The clock is now rendering grey text for me, even though I have this in my config:
#clock {
  color: #ffffff;
}

@Alexays do you think we could expect 0.9.15 reasonably soon, now that for example #1687 is reverted, and only styling seems to be the problem?
Although I understand the intention behind deleting a tag, it is generally not the best practice to do, but more importantly, there is over 250 commits of more than 5 months of contributions since 0.9.13, it would be awesome to get all those improvements into the hands of the users 🙂

Although I understand the intention behind deleting a tag, it is generally not the best practice to do

I agree, things got very confusing when I couldn't find the upstream 0.9.14 release corresponding to the one on OBS. Any announcement would have been appreciated (open an issue, add a notice to the README, ...).

0.9.15 released, sorry for the mess

Wow that was quick. Thanks a lot, very much appreciated

Thanks @Alexays. openSUSE OBS package got updated

Arch too, thanks for the swift response! 👍

Thanks for addressing this confusion quickly.
Suggestion: If tags should not be picked up by downstream packagers (and only GH releases) I'd suggest adding a small section for packagers on the README would be great to avoid confusion. Generally, when a tag pops up packagers will assume it's ready to ship (unless it says alpha or alike).

0.9.15 doesnt fix the issue in second post. (arch linux)
0.9.15

0.9.13

Same on openSUSE Tumbleweed:
0.9.15

0.9.13

Similar issues.
With 0.9.13:

With 0.9.15:

Main regressions I have:

text for modules without on-click is different
fixed width format with the network module is not working anymore

There's also an issue with "format": "", these no longer display the arrows correctly. The foreground colour isn't set and the arrow doesn't take up the entire block

0.9.15 does nothing good for me on openSUSE/Tumbleweed, see https://github.com/openSUSE/openSUSEway/issues/79.

There's also an issue with "format": "", these no longer display the arrows correctly. The foreground colour isn't set and the arrow doesn't take up the entire block.

Can confirm. My bar uses it, and now looks completely differently.
Old:

New:

I'm assuming there are backwards-incompatible changes in Waybar, but I haven't had much luck finding how to migrate my old configs properly.


Environment variable for the data path possibly inconsitent

Describe the bug
Just discussed with @micbar the env variable defining the location of the ocis data folder:

We describe in our admin docs the env variable OCIS_BASE_DATA_PATH in Define the Infinite Scale Data Path. This is the only variable known to me so far defining the data path depending on the deployment. I got the infos from @wkloucek and I remember that he wrote code to define the locations depending on the deployment.
But is seems, that there is an additional env variable STORAGE_USERS_OCIS_ROOT at Storage-Users Service Configuration doing the same thing.

To be clarified:

Do these env's do the same thing
If yes, the OCIS_BASE_DATA_PATH variable must be added to the storage_users service
Where do these env's differ - if they do
Hmmm, usually global overwrites service, but does this apply in this case to the data path? How can it differ for what use case?
Needs checking if both env's are properly used in the code for the same purpose
The current description of STORAGE_USERS_OCIS_ROOT is not sufficient and needs improvement

base path by default:

$HOME/.ocis/ - binary setup
/var/lib/ocis - container setup

after starting ocis init and ocis server without extra vars we have following structure:
PWD/.ocis/

config
idp
idm
storage
....

envs:
OCIS_BASE_DATA_PATH
OCIS_CONFIG_DIR
STORAGE_USERS_OCIS_ROOT

to change config dir PWD/.ocis/config we use env OCIS_CONFIG_DIR=newConfigDir
to change only PWD/.ocis/storage/users dir we use env STORAGE_USERS_OCIS_ROOT=newPath
to change PWD/.ocis dir we use env OCIS_BASE_DATA_PATH=newPath

there is a difference between  OCIS_BASE_DATA_PATH and STORAGE_USERS_OCIS_ROOT
we cannot add OCIS_BASE_DATA_PATH to storage_users because it changes path not only for storage

Update, post a discussion with @kobergj we can bring in some light, hopefully I got it correct.
We have two data types that are covered by the variables above. User data and system relevant data. and we have config data

If only using/defining OCIS_BASE_DATA_PATH, user data, system relevant data and config are on the same path. This is because STORAGE_USERS_OCIS_ROOT and OCIS_CONFIG_DIR defaults to OCIS_BASE_DATA_PATH.
If env vars are defined, you can separate the data stream:
OCIS_BASE_DATA_PATH will contain system relevant data
STORAGE_USERS_OCIS_ROOT will contain the user data
OCIS_CONFIG_DIR contains the config files

Example of system relevant data:
idm
idp
nats
proxy
search
storage/metadata
store
thumbnails
This leads to following conclusions

Currently we have no description about this - docs relevant
This can positively increase security - docs relevant
OCIS_CONFIG_DIR is not well described without the impact - docs relevant
The description and default values for OCIS_CONFIG_DIR/OCIS_BASE_DATA_PATH (have not seen them) and STORAGE_USERS_OCIS_ROOT (available) needs an update before GA - ocis repo
Proposal:
-- default_value set to empty for both
-- description add for both: If not set, defaults to OCIS_BASE_DATA_PATH
-- For STORAGE_USERS_OCIS_ROOT, Path for the persistence directory. --> Path for the persistence directory of user files only.
-- OCIS_BASE_DATA_PATH (I do not know the current text, but it should look like) --> Path for the persistence directory for config, system and user files.

Note, normally a service env overwrites the global env for that service. Woudnt it be good in that case to have the OCIS_BASE_DATA_PATH added in the storage_users env STORAGE_USERS_OCIS_ROOT?
which in the same way would be true for OCIS_CONFIG_DIR adding OCIS_BASE_DATA_PATH?

If only using/defining OCIS_BASE_DATA_PATH, user data, system relevant data and config are on the same path. This is because STORAGE_USERS_OCIS_ROOT and OCIS_CONFIG_DIR defaults to OCIS_BASE_DATA_PATH.

here is bug:

clean .ocis dir rm -rf /.ocis
run init: OCIS_BASE_DATA_PATH=/Users/scharfviktor/.ocisNew ocis/bin/ocis init

Actual: config created in default dir .ocis

Looks like ocis init does not evaluate OCIS_BASE_DATA_BATH.
It gets the path from https://github.com/owncloud/ocis/blob/f91a0b45a50fa9110d559dcbeeee8e5bab8c93f4/ocis-pkg/config/defaults/paths.go#L48 which does not evaluate the env var and only defaults to the home dir or whatever is hardcoded in BaseConfigPathValue

Sorry, I think there is still some confusion here.
OCIS_CONFIG_DIR does not default to $OCIS_BASE_DATA_PATH/config it is an independent config value.
STORAGE_USERS_OCIS_ROOT however does to default to $OCIS_BASE_DATA_PATH/config if not set.

@kobergj you are right, my fault.
Accodring to the documentation OCIS_CONFIG_FILE
The default locations for config files are:

For container images (inside the container)
/etc/ocis/

For binary releases
$HOME/.ocis/config/

The default data path again seems to be dependent on if you use binary or container install, see: container mount option
--mount type=bind,source=$PWD/ocis/ocis-config,target=/etc/ocis \
--mount type=bind,source=$PWD/ocis/ocis-data,target=/var/lib/ocis \

and looking into the default of STORAGE_USERS_OCIS_ROOT it is ~/.ocis/storage/users which I guess is the same for STORAGE_USERS_OCIS_ROOT.
IMHO we need to look if the above is true and how to make the default for the data path in the same mind as teh config - usage dependent.
We programatically need to fix this and then do a proper documentation.

I think we have the solution now, with many thanks to @rhafer.

The admin docs need to be updated describing the usecase and the defaults.
Yes, there is more than one default, the dependency is based on the build (binary/container)
OCIS_CONFIG_DIR
OCIS_BASE_DATA_PATH
STORAGE_USERS_OCIS_ROOT

The description of STORAGE_USERS_OCIS_ROOT needs to be updated.
-- Make the usecase clear
-- Note that the default is build dependent, referencing to read the docs for details

Note, that we should take the chance to also update the both OCIS_ variables from above in the same way we do for item 2. Else this knowledge gets lost.


[Bug] 404 on iOS 16.6 after updating to v1.10.1

Trying to access xCloud produced error 404 after updating to better xcloud v1.10.1. The error cleared after about 10 refreshes.

better xcloud settings:
block_social_features: false block_tracking: true disable_bandwidth_checking: false hide_dots_icon: true prefer_ipv6_server: false reduce_animations: false screenshot_button_position: "bottom-left" server_region: "default" skip_splash_video: false stats_conditional_formatting: false stats_opacity: 80 stats_position: "top-left" stats_quick_glance: false stats_show_when_playing: false stats_text_size: "0.9rem" stats_transparent: false stream_hide_idle_cursor: false stream_preferred_locale: "default" stream_simplify_menu: true stream_target_resolution: "auto" stream_touch_controller: "default" user_agent_custom: "" user_agent_profile: "default" use_desktop_codec: false version_current: "1.10.1" version_last_check: [1691365032](tel:1691365032) version_latest: "1.10.1" video_brightness: 100 video_contrast: 100 video_fill_full_screen: false video_saturation: 100

Please use only this issue for this 404 error: https://github.com/redphx/better-xcloud/issues/78


Animations from GLB

The GLB is here and it's looking fine: https://github.com/webaverse/app/blob/master/animations/animations.glb
This will need a few more strafe reverses in the GLB before I can successfully load it:
`left_strafe_walking_reverse`,
`right_strafe_walking_reverse`,
`left_strafe_reverse`,
`right_strafe_reverse`,

Closing in factor of #1133.


Evaluates nil as true in conditions without equivality test

This condition will treat nil/none values as `true:
if new
    << id
end

The condition works with an equality test:
if new == true
    << id
end

https://github.com/opset/openset/pull/33


Add tests for getHtmlTitleBody

added test cases for getHtmlTitleBody() when input is:

text file with title
text file without title
markdown file

Looks great, thanks for your changes :)


Update CONTRIBUTING.md

fixed some minor typos

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


feat: add rust binding for two adic fri pcs

Description
This PR adds rust binding for TwoAdicFriPcs.

For 9c112b7, 4 to 8 seems correct.


Configure Ruleset version for Plugin  via .editorconfig file

Hi
On our projects my team uses ktlint-intellij-plugin (dev environment) and jlleitschuh/ktlint-gradle plugin (rarely on dev environment, always on CI/CD server).
In this Gradle plugin we can set ktlint version as described here. That works well, because we can store it in repository and it will be applied automatically in all dev environments and CI/CD.
As for ktlint-intellij-plugin I have to manually set ktlint version via IDE settings in each project. This problem becomes more painful when we have many developers using same project. Also, different projects may use different ktlint version.
Could you consider to add an option to fetch Ruleset version (ktlint version) from .editorconfig file?
I suppose it should be some new custom setting specifically prefixed for your plugin (like Idea has own settings with prefix ij_). So it can be stored in a repo and automatically applied in all dev environments where ktlint-intellij-plugin is installed. If setting is not found, then default behaviour can be used. I suppose now your plugin by default uses some hardcoded ktlint version. In UI it appears as Ruleset version: default (recommended).
Something like this

P.S. It is clear that fetching ktlint version from jlleitschuh/ktlint-gradle plugin has no sense.

Could you consider to add an option to fetch Ruleset version (ktlint version) from .editorconfig file?

Please commit file .idea/ktlint-plugin.xml to your version control. In this way, all preferences of the plugin will be shared amongst all developers on the project.

Need to try. Thank for the idea.

@paul-dingemans thank you for quick respond.
It works. Also I've discovered Idea feature Required Plugins. So Idea IDE can notify user, if plugin is not installed.
Here what I end up (notes for those who what to implement the same):
.gitignore
.idea/*
!.idea/externalDependencies.xml
!.idea/ktlint-plugin.xml

.idea/externalDependencies.xml - configured via UI.
<?xml version="1.0" encoding="UTF-8"?>
<project version="4">
  <component name="ExternalDependencies">
    <plugin id="com.nbadal.ktlint" />
  </component>
</project>

!.idea/ktlint-plugin.xml - configured via UI.
<?xml version="1.0" encoding="UTF-8"?>
<project version="4">
  <component name="KtLint plugin">
    <ktlintMode>MANUAL</ktlintMode>
    <ktlintRulesetVersion>V1_0_1</ktlintRulesetVersion>
  </component>
</project>

Keep in mind, files in .idea foldet not updated immediately (click OK or Apply in Settings), but only after IDE quit.

Tnx for sharing back the required plugin settings which indeed might be helpful in cases like you describe. As you now have a working solution, I will close the issue.


attribution error

I have an issue where this plugin uses gem source CODEOWNERS instead of project root CODEOWNERS.
I have gem "redis-store" and all my files are being attributed to tubbo via https://github.com/redis-store/redis-store/blob/master/CODEOWNERS
commenting out that source file in local cache, this plugin then uses gem "omniauth-auth0"'s CODEOWNERS, etc. perhaps especially when those gems use .github/CODEOWNERS instead of root CODEOWNERS
Complications may include

remote (docker) Ruby Framework, so gem source gets cached in RubyMine directory
project CODEOWNERS is more than ~500 lines (I could get first ~500 lines to parse by deleting gem CODEOWNERS files from caches)
project CODEOWNERS is in root, but project also has a .github directory

plugin wasn't finding CODEOWNERS in root today, but had previously.
moving to .github/CODEOWNERS had some success

CODEOWNERS file had some syntactic errors (trailing comments)

did not fully recover after deleting syntactic errors

Hi, how can I reproduce the issue? I tried cloning the repo you mentioned, but after bundle install I have no CODEOWNERS files in the project file tree other than the one in the project root with a single * @tubbo line (I assume ruby doesn't install dependencies into the project subdir, like npm does)

I was not able to reproduce today with the original complex project.
With an example project, it did not produce with just redis-store, but it did reproduce for me when I added omniauth-auth0
The example project is at https://github.com/stevenjonescgm/intellij-codeowners-gem-attribution.
test/system/ex_300_spec.rb was identified as owned by
*	@auth0/dx-sdks-engineer

After reproducing, I tried switching ruby interpreter/gems to localhost and that worked correctly/did not reproduce error.
Due to some caching in the plugin, I needed to close and reopen the project window before the change of interpreter was evaluated.
I changed the interpreter back to the remote Docker and restarted then issue reproduced again.
Reproducing may involve the plugin parsing some information, then subsequent runs after modifying dependencies. So if it does not reproduce for you on 62213906c862f3d9a620f159d90449cc00fe0ed3, then maybe reset to an earlier commit to configure the IDE then pull and install gems (docker compose up will run bundle install on the remote)


added functionality for scraping based on recency

adds timedelta parameter, i.e. allowing scraping of only the most recent tweets instead of pages alone. Note that it still cuts off based on pages

Such a good idea! I think it can would be better if works standalone (instead of based on pages).
If you can update documentations too we I can merge.

The problem is that without the pages functionality, you lose flexibility
of cutting off early in case a particular account has done a massive amount
of tweeting over the time period (in which case your end application will
slow down).
On Wed, Feb 12, 2020 at 12:39 PM Buğra İşgüzar<EMAIL_ADDRESS>wrote:

Such a good idea! I think it can would be better if works standalone
(instead of based on pages).
If you can update documentations too we I can merge.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bisguzar/twitter-scraper/pull/106?email_source=notifications&email_token=AEW57PZWTB2NIZJIHUCLMGLRCQX6DA5CNFSM4KSOOOJ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELRWEQQ#issuecomment-585327170,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEW57PYL3FRHN5PBLPPCMULRCQX6DANCNFSM4KSOOOJQ
.

--
-Jacob Alperin-Sheriff

I see, but documentation still needed.


Django >= 4.1 <5 wants a default form renderer

If not set, automated testing output gets cluttered with deprecation warnings.
Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] 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 change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Most deprecation warnings come from django-formtools. I tried to fix this in django-formtools and hope a 2.4 release will be made soon, then I'd like to rerun the tests with an updated django-formtools to see if this PR is still needed or not.

django-formtools 2.4 was released today. Together with the changes of #555, this patch should no longer be necessary. Thanks anyway!


tokenName alignment and tokenValue width in inspect

Is your feature request related to a problem? Please describe.
The inspect panel does not have an organised way of displaying the tokenNames & Values - user is having to haphazardly scan the section to identify the tokens

Proposal:

The tokenValues container should have the same width and not move into the next line like with 4,8 of the label token
This should then result in the tokenNames being aligned left - on the same vertical line

I think we should

force a max width of the one we have today
Use the same width for the color swatches (just the container, not the actual swatch) so that also colors being and the same line
Same for none value tokens and any other tokens that use an icon instead of a text
Truncate each value preview so the text doesnt wrap, is on 1 line (no ..., just cap the text)
Have a tooltip for each of those that shows the full token (the same tooltip we show when you hover a token in the Tokens tab)

I think we should

force a max width of the one we have today
Use the same width for the color swatches (just the container, not the actual swatch) so that also colors being and the same line
Same for none value tokens and any other tokens that use an icon instead of a text
Truncate each value preview so the text doesnt wrap, is on 1 line (no ..., just cap the text)
Have a tooltip for each of those that shows the full token (the same tooltip we show when you hover a token in the Tokens tab)

This request has been moved to our new feedback tool. Go check out https://tokensstudio.featurebase.app/ to create a free account. You can upvote and comment on existing posts, make a new request, and subscribe to be notified when requests move into our roadmap.

Here's the URL to the featurebase post that is linked to this ticket: https://tokensstudio.featurebase.app/p/enhance-inspect-page-in-the-plugin


dts: rt1170: Add dts for fast gpio

rt1170 has two group fast gpio and shared the same
interrupt source. Now add the dts definition for the
fast gpio
Signed-off-by: Crist Xu<EMAIL_ADDRESS>

@CristXu, I think you need to change your commit message to start with:
dts: rt1170: Add dts for fast gpio

LGTM. Does this issue also exist on the RT1160?

Yes, The 1160 has the same issue, but seems that rt1160 does not set these pins yet. If we also need the fast-gpio of rt1160, we also need do this, I think.


Any API call is failing while server is up and running

Hi,
I'm trying to connect between docker images and having this code
new Flagsmith({
        environmentKey: configService.get('FLAGSMITH_KEY') || '',
        apiUrl: 'http://<IP_ADDRESS>:9000/',
      });

The first call to the API using:
flagsmith?.getEnvironmentFlags(); fails with

Error: Invalid request made to Flagsmith API. Response status code: 500

The issue is I can curl outside Docker, that means server is up and running fine:
curl 'localhost:9000/api/v1/flags/' -H 'x-environment-key: ##########xN4y7cm'

[{"id":1,"feature":{"id":1,"name":"test_feature","created_date":"2023-01-23T14:32:09.986573Z","description":null,"initial_value":"holla","default_enabled":false,"type":"STANDARD"},"feature_state_value":"holla","environment":1,"identity":null,"feature_segment":null,"enabled":false}]

Thanks a lot :)

my apiUrl should have been http://<IP_ADDRESS>:9000/api/v1/ my bad .


GradleRunner.withPluginClasspath isolates plugin-under-test classloader from other test project plugins

GradleRunner.withPluginClasspath() isolates plugin-under-test and its dependencies  in the test project  from test project plugins/buildscript classloader leading to unexpected user errors.
Expected Behavior
Plugin and its dependencies passed via GradleRunner.withPluginClasspath() are merged into test project plugins/buildscript Classloader so behaviour is the same as consume plugin build published via local repository.
Current Behavior
Gradle isolates withPluginClasspath dependencies in a separate Classloader. This leads to wrong classes loaded by plugin in the test, while real project loads correct classes.
Context

Upon investigating this Kotlin Gradle plugin issue I've discovered user plugin in the test loads old ApiVersion class from plugin transitive dependency. In the real project this dependency will be replaced to newer version by KGP runtime dependency declaration.
Steps to Reproduce
Check KGP issue - it has project with reproduce steps.

I think I am facing the same problem. The Gradle plugin I want to test has a compileOnly dependency for Android Gradle Plugin. When I attempt to use GradleRunner.withPluginClasspath(), I get a ClassNotFound error. However, respective dependency (AGP) is added to the classpath in the test project and therefore should be available but due to the isolation described it doesn't work as expected. I verified that the test case works fine If I set up a real project.

I have just published a combo plugin + support library to make use of TestKit easier. A blog post will be forthcoming, but for now the README might be enough to get some people started.  The "whys" are right at the top. I'd love feedback from anyone who gives it a try. Note that it currently wants you to put all your gradle tests in src/functionalTest, but I might make that configurable at some point
https://github.com/autonomousapps/dependency-analysis-gradle-plugin/tree/main/testkit

I'm also running into this problem in a way that result in a more subtle error.
My scenario is similar to the one from https://github.com/gradle/gradle/issues/7974. I put together a minimal reproduction in
repro.zip. The tests demonstrate the broken behavior.
In my case I have the following plugin dependencies:
com.palantir.baseline-error-prone -> net.ltgt.errorprone
com.example.project.my-plugin -> net.ltgt.errorprone

The com.palantir.baseline-error-prone plugin depends on and applies the net.ltgt.errorprone plugin.
// com.palantir.baseline-error-prone plugin

project.getPluginManager().apply(ErrorPronePlugin.class);

My plugin also depends on the the net.ltgt.errorprone plugin.
// build.gradle

dependencies {
    implementation 'net.ltgt.gradle:gradle-errorprone-plugin:3.1.0'
}

As a result, the generated plugin-under-test-metadata.properties file has an implementation-class property which contains gradle-errorprone-plugin-3.1.0.jar.
In my test project, I apply the com.palantir.baseline-error-prone plugin. If I don't apply my plugin, then I can use withPlugin to detect the application of the net.ltgt.errorprone plugin.
plugins {
    id 'java'
    id 'com.palantir.baseline-error-prone'
}
project.getPluginManager().withPlugin('net.ltgt.errorprone') {
    // This is printed
    println "Found errorprone plugin"
}

But if I apply my plugin, then withPlugin does not detect the application of the net.ltgt.errorprone plugin.
plugins {
    id 'java'
    id 'com.palantir.baseline-error-prone'
    id 'com.example.project.my-plugin'
}
project.getPluginManager().withPlugin('net.ltgt.errorprone') {
    // This is not printed
    println "Found errorprone plugin"
}

When applying the ErrorPronePlugin in the com.palantir.baseline-error-prone plugin, the plugin is loaded with the standard buildscript classloader and the resulting RegistryAwarePluginImplementation.pluginId is null.
When we later try to lookup the plugin by ID, because the pluginId is null, we fallback to the classloader lookup. Except now the classloader is the MultiParentClassLoader which contains the TestKit injected classpath. So the ErrorPronePlugin class we load is different than the ErrorPronePlugin class loaded when we applied it in the com.palantir.baseline-error-prone plugin.
https://github.com/gradle/gradle/blob/28aca86a7180baa17117e0e5ba01d8ea9feca598/subprojects/core/src/main/java/org/gradle/api/internal/plugins/DefaultPluginRegistry.java#L258-L266


Hotfix/DS-330 Remove deprecated card

Jira
https://pegadigitalit.atlassian.net/browse/DS-330
Summary
Remove deprecated Card component.
Details
Remove deprecated Card component, its docs, and a few mentions of this component in tests and build files. I renamed the docs folder for "Card (Replacement)" to "Card" but stopped there. I did not rename all the docs pages from "card-replacement-x" to "card-x", and I did not rename the <bolt-card-replacement> element itself. I figure that will happen later.
How to test
Build is passing and there are no regressions to the "replacement" card. Spot check other components for any obvious regressions.
Release notes

Bolt Card: @bolt-components-card/card.twig has been removed in v3.x. Use @bolt-components-card-replacement/card-replacement.twig instead.
Bolt Band: items prop has been removed in v3.x. If you are still using the items prop to add pinned content to a band, see docs on how to use the pinned_content prop instead.

@remydenton updated to include Band updates: https://github.com/boltdesignsystem/bolt/pull/2102/commits/b671a74ad1d85a0489944f6704ef9b219463eb27


Generated Nix expression references out-of-scope pkgs

Minimal repro here:
https://gist.github.com/isomorpheme/514b826ca0f9bf2ee28aa5ca27863d8b
Running nix build on that flake will produce an error like
error:
       … while calling the 'derivationStrict' builtin

         at /builtin/derivation.nix:9:12: (source not available)

       … while evaluating derivation 'my-example-haskell-lib-<IP_ADDRESS>'
         whose name attribute is located at /nix/store/7adgvk5zdfq4pwrhsm3n9lzypb12gw0g-source/pkgs/stdenv/generic/make-derivation.nix:348:7

       … while evaluating attribute 'propagatedBuildInputs' of derivation 'my-example-haskell-lib-<IP_ADDRESS>'

         at /nix/store/7adgvk5zdfq4pwrhsm3n9lzypb12gw0g-source/pkgs/stdenv/generic/make-derivation.nix:402:7:

          401|       depsHostHostPropagated      = elemAt (elemAt propagatedDependencies 1) 0;
          402|       propagatedBuildInputs       = elemAt (elemAt propagatedDependencies 1) 1;
             |       ^
          403|       depsTargetTargetPropagated  = elemAt (elemAt propagatedDependencies 2) 0;

       error: undefined variable 'pkgs'

       at /nix/store/4hnqbxmqsjm7gqngszp1xvgx87i3bh5d-cabal2nix-tz-<IP_ADDRESS>/default.nix:23:34:

           22|   ];
           23|   preConfigure = "export TZDIR=${pkgs.tzdata}/share/zoneinfo";
             |                                  ^
           24|   homepage = "https://github.com/ysangkok/haskell-tz";

The top-level trigger is the dependency on tz; commenting it in my-example-haskell-lib makes the build succeed.
The custom preConfigure seems to come from here, but if so I have no idea how the ${pkgs.tzdata} bit wasn't interpolated already. In hackage-packages.nix it also shows up, but there it's fine because pkgs is bound at the top of the file. The generated default.nix that triggers the error doesn't have this structure.
I've tried using cabal2NixArgsOverrides as a workaround, but it doesn't seem to work (I might've been using it wrong, though).
This may be an upstream issue with cabal2nix, but I couldn't tell for sure, so I'm starting here.

Ah, it's actually added by cabal2nix itself: https://github.com/NixOS/cabal2nix/blob/2099a1f4594f621bb1a2879b793b860aefe4c027/cabal2nix/src/Distribution/Nixpkgs/Haskell/FromCabal/PostProcess.hs#L177
So I think this is indeed an upstream issue after all.

@isomorpheme Thanks for looking into this.
I sent a PR to cabal2nix that should somewhat fix this: https://github.com/NixOS/cabal2nix/pull/611.
This problem will require a separate fix in stacklock2nix as well.

This has made its way to Nixpkgs: https://github.com/NixOS/nixpkgs/pull/278674/commits/0c2ff42913035c83d56b53aeafd24b39b31d4152
But we still need this same fix in stacklock2nix, I think.


Feature/icsaas 394 add state to facet filters

add state
use session storage to persist facet filters
style no result for facet filter options
sort alphabetically facet filter options
refactor sessionStorage with util functions

UPDATE:
@nhirrle I fixed the behavior, should be better now as expected :D (sorry about that, I got confused)
https://user-images.githubusercontent.com/81755864/152823036-d844988d-7ae4-4392-adb4-997b259fd677.mov


How does the Persformer consider the height of lane lines?

Hi, the results of your work are impressive. But I have a question. As you mention "the height of lane lines has to be considered when we project perspective lanes into BEV space" in the paper. how does the Persformer  take the height of lane lines into consideration ?
Whether the Persformer does just take advantage of the power of the self-attention and cross-attention mechanism to improve the performance of the normal situations such the relative even roads?

It is a great question. Compared to 1-1 mapping, the cross-attention mechanism takes surrounding areas in the image, which naturally corresponds to different heights, into consideration. Some more sophisticated ideas could be like how BEVFormer does.


The force_text function has been removed in Django 3.1 and later versions, but it is used in django-cms-4.1.0rc3

in django-cms-4.1.0rc3, the following import is used:
from django.utils.encoding import force_text
I am using Django 4.1.9, which now use force_str instead of force_text
on https://pypi.org/project/django-cms/4.1.0rc3/#description it says django-cms-4.1.0rc3 support Django 4.1.9, but in actual django-cms-4.1.0rc3 has dependencies on force_text

mistake


Add more traces for state-lock management

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10050/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10071/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10075/
Test FAILed.

alluxio-bot, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10078/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10080/
Test PASSed.

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.2 please

alluxio-bot, cherry-pick this to branch-2.2 please


Fix responsiveness of RGA GUI disassembly view

The GUI becomes unresponsive for several seconds whenever the disassembly view
gains or loses focus due to the code unnecessarily applying a stylesheet to
the entire frame just to change border color, which seems to be a leftover
functionality, considering that the color was only affected by the API used
which is now chosen at application launch so the border color does not provide
much value. According to the Qt documentation applications should not call
setStylesheet() outside of the constructor because it's very costly and the
current code calls it on focus change.
This change fixes that by getting rid of all the code using setStylesheet()
to change the disassembly view's border.
Also, some missing header includes were added which most likely were
uncovered by compiling against Qt 5.11.2 contrarily to Qt 5.9.2 recommended
in the RGA documentation.

Note: the reason the entire rgIsaDisassemblyView.cpp is marked as changed is because it contained inconsistent line endings previously.


Error Message "Could not reach the server" on mounting to FastAPI

Describe the bug
When trying to run Chainlit mounted on FastAPI , the Chainlit app seems to be failing on the path "/chainlit" with the error message:

Could Not Reach the Server

Accompanied by this in the terminal:

Translated markdown file for en-US not found. Defaulting to chainlit.md.

The Chainlit app runs perfectly fine when run separately.
To Reproduce
Error observed for Chainlit v1.1.404, normal behavior for v1.1.400
Mounting chainlit on FastAPI app
mount_chainlit(app=app, target="chatbot/chatbot.py", path="/chainlit")
Run FastAPI app:
 uvicorn main:app --host <IP_ADDRESS> --reload
Expected behavior
App should be served as expected unless I'm doing something wrong
Desktop (please complete the following information):

OS: Windows10
Browser: Chrome
Version 128.0.6613.120

@sajsax10 Since you are specifying a path, this is likely related to #1313, #1262, and #1317
As far as I'm aware the last release that didn't have this critical bug was 1.1.402, but unfortunately the Chainlit company doesn't seem to be taking it very seriously :(

@stephenrs We are. Please refrain from attacking us, it's unpleasant and unproductive. We appreciate your support but please be aware that we are doing are very best with the bandwidth we have.
@sajsax10 @stephenrs Could you please try and validate whether #1337 resolves your issue?

@dokterbob I'm not attacking you, I'm pointing out my observations, and I'm glad to see you're finally showing some indications that you're paying attention to critical issues like this one.
I would recommend that you get your emotions under control, and stop whining and being so defensive. None of this is personal. Focus on the work and smile about it :)

@dokterbob Just a quick follow-up. You seem to spend a considerable amount of time thinking about and talking about how you feel. So, I think it would be helpful for you to devote some time to thinking about how people in this community feel as a result of your actions, inactions, or lack of communication.
People who encountered this particular problem were completely blocked on their entire project if they didn't know they could downgrade to 1.1.402. This is likely an actual crisis for some people. So, I'm thinking about them and how they feel and doing my best to make sure they are informed about where your priorities appear to be so they don't get stuck in a crisis situation - because that feels terrible.
Also, if you guys make it to the point of having paying customers, what people are going to be saying here is quite likely going to be much more emotionally triggering than anything you'll hear from me - and they won't be focused on helping anyone except themselves. Clearly, my interest is in helping you and the community. So, it might best serve your interests to start thinking about us as customers - OUR happiness is more important than yours, such is the nature of the real world of business.

Closing this as the case as the reproduction seems to fully match the example https://docs.chainlit.io/integrations/fastapi which was tested to be resolved by #1337. Very l33t, much wow.
We're doing a release today.

Cool. Note there's 355 issues that YOU can see.

Not sure how this is relevant...and maybe if you communicated your priorities or didn't ignore questions about them it wouldn't matter at all which issues are public and which are not.

Sending long write ups don't help there, it detracts from limited bandwidth we have.

More whining. Wow.
But hopefully you're learning that when you tell lies about the status of a product (which is illegal in the EU), your "limited bandwidth" is of no concern to the public you lied to and they don't owe you anything. The debt is entirely yours to deliver what you have promised or face worse consequences than people who "annoy" you.
Glad to see some energy and some focus on helping the people of this community get their work done, however.


Develop creating person logic for PersonService

Validation process:

check required fields
check if the person already exists

Results:
Information about a person saves in the database
Creating a request with wrong information about the person returns an error result with a description.

@itbeard Should the system check if a person with FristName and LastName already exists in the database?

@itbeard Should the system check if a person with FristName and LastName already exists in the database?


Blurred image

Good afternoon! The issue #50 seems to be closed, but the image is still not sharp. For example, the same image is used. Can you help me?

Closing due to no response.


fix: don't display deps when there are none

removing something that used to be a debug-statement.

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

v5.4.7
GitHub release

Your semantic-release bot :package::rocket:


#185 Add searching to dropdown

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Withdraw Private Method Missing?

Is there a reason the "Withdraw" method is not in the private query method struct?

Because Kraken doesn't support it. See API manual = https://www.kraken.com/help/api

Private user funding
NOTE:This is a tentative funding API and may be updated in the future. Please refer to the main API page for more information on using the API.

But I see no reason why they are not added...

Merged. Thx for the feedback!


"iterNChildren" shoud accept "nil" for "iter" arguments.

This is the same problem already encountered for widgets, but for the TreeModel methods. "nil" is processed as as special case by the GTK library. There are several methods which use this trick.
For now, I will define an overloaded function.

In fact, the situation is very different, and more complicated. The type "Iter" defines an object not a reference or a pointer to an object, so "nil" is not an acceptable value here.
I don’t know how manage this special case, but the interface of "gtk_tree_model_iter_n_children" is really ugly.

Yes, I did that by intent, I did it also for oldGTK3 low level wrapper. For simple structs like GdkColor or TextIter it makes sense to avoid a Nim proxy object. Those objects are generally advertised by gobject-introspection with marker user-alloc. The advantage when we use no proxy objects is, that we can allocate them simple on the stack like "var iter: TextIter", so we do not need "var iter = newTextIter()". And it saves a few bytes in executable. Passing to C library works fine because {.byRef.} pragma is applied, so Nim Compiler passes pointers, what is even the default. Handling nil is a bit complicated indeed, maybe we have to add overloaded procs manually indeed. Will continue tomorrow...

I tried to fix that with this minimal patch:
$ diff gen.nim ~/gintro/tests/
431,435d430
<     # ------------------
<     if isFunctionInfo(info) and (gFunctionInfoGetFlags(info).int and GIFunctionInfoFlags.WRAPS_VFUNC.ord) == 0:
<       if userAlloc and mayBeNil:
<         str.add(" | ptr " & str & " = nil")
<     # ------------------

Now we have something like
proc gtk_text_iter_backward_search*(self: TextIter; str: cstring; flags: TextSearchFlags;
    matchStart: var TextIter; matchEnd: var TextIter; limit: TextIter | ptr TextIter = nil): gboolean {.
    importc: "gtk_text_iter_backward_search", libprag.}

So it should be possible to pass a nil pointer or nothing instead of a textiter when gobject-introspection allows NULL. I hope this work, have no code for testing currently.

It doesn’t work in my program. I have written a small test. The first call to "iterNChildren" compiles, of course, but there is still an error in the second one.
import gintro/[gtk, gobject, glib]

proc p(model: TreeModel, iter: TreeIter) =
  discard model.iterNChildren(iter)

proc q(model: TreeModel) =
  discard model.iterNChildren(ptr TreeIter(nil))

Interesting, thanks. I think I have to ask Araq.
iter: TreeIter | ptr TreeIter = nil
seems to generate some headache for the compiler.

#The problem seems to be the = nil
# But that is what we want -- we want to pass an TreeIter or nil

type
  TreeIter* {.pure, byRef.} = object
    stamp*: int32

  TreeModel* = ref object of RootObj

proc iterNChildren*(self: TreeModel; iter: TreeIter | ptr TreeIter = nil): int =
  discard

var model: TreeModel
var iter: TreeIter

discard iterNChildren(model, iter)

# yyy.nim(14, 22) template/generic instantiation from here
# yyy.nim(8, 70) Error: type mismatch: got (nil) but expected 'TreeIter = object'

Yes, removing "= nil" makes it compile, but to pass "nil", we have to write "(ptr TreeIter)(nil)". Not very elegant.
Using an overloaded procedure may work, I suppose.
Another way to manage this would be to define some "null" TreeItem object for the high level interface which would be translated as "nil" when calling the low level interface. I have no idea what could be such a "null" TreeItem, though.
I understand the reason to use "object" rather than pointer here. It is also he way TreeIter is defined in the GTK3 interface (as a struct), but the strong typing in Nim makes it complicated to manage here.
I have to leave for two days. I will work on my programs again probably on Thursday. Bye.

Yes an overloaded proc seems to be necessary for this case.
I will try to fix all the open bugs in the next days, bye. Well, except the signal emission, that will take longer.

Closing with this patch applied:
$ diff tests/gen.nim ~/gitest/tests/gen.nim
434c434,437
<         str.add(" | ptr " & str & " = nil")
---
>         #str.add(" | ptr " & str & " = nil")
>         var h = str
>         if h.startsWith("var "): h = h[4 .. ^1]
>         str.add(" = cast[ptr " & h & "](nil)[]")

Now this compiles:
import gintro/[gtk, gobject, glib]

proc p(model: TreeModel, iter: TreeIter) =
  discard model.iterNChildren(iter)

proc q(model: TreeModel) =
  discard model.iterNChildren()

A bit strange is it for var parameters, we have only very few like
pango.nim:proc transformRectangle*(self: Matrix; rect: var Rectangle = cast[ptr Rectangle](nil)[]) =

It works fine. But, if we want to pass explicitely "nil", we have to use "castptr TreeIter[]" which is awful. We don’t need to do that, of course, but somebody who looks at the GTK3 documentation may think it is the way to go. It is something which will have to be documented.

Great that it works. Yes we have to document how it is used. That nil default does not work is indeed a known bug:
https://github.com/nim-lang/Nim/issues/1201


Where could I find the file 'data/dow30.csv'?

thanks!

you can download on yahoo finance.
samsung data
click download csv

I do NOT mean the stocks data downloaed from yahoo finance.
I mean the configuration file in the source file 'config.py', at line 3:
names_file = 'data/dow30.csv'
thanks all the same!

I think readme leads to download data from yahoo.
Downloads adjusted daily returns of a configurable date range and set of stocks from Yahoo Finance, concatenates them all into a long sequence, and trains an LSTM to predict future returns based on the sequence of past returns.

But I don't know why author didn't contain that file.

stocks_list = ['NKE', 'AXP', 'BA', 'C', 'CAT', 'DD', 'DIS', 'GE', 'CSCO', 'HD', 'CVX', 'V', 'IBM',
'INTC', 'JNJ', 'JPM', 'KO', 'MCD', 'MMM', 'GS', 'MRK', 'MSFT', 'PFE', 'PG', 'AAPL', 'UTX', 'VZ',
'WMT', 'XOM', 'TRV']
this is the dow30 stocklist，you can have this


Large Balance Decrease - Multichain USDT Bridge Augist Drop rate

Hi!
I have been watching the performance for this bot for the month of August and it looks like it has a high drop rate on ETH. I would appreciate if you can check it out, and consider sharding as a potential solution to mitigate this.

solved in https://github.com/NethermindEth/forta-starter-kits/pull/85/files


Spanish translation for alerts

You fixed the 'Oh Snap!' bug in the template as well, thanks @jdarias !

my pleasure, @ziahamza .


Extend swagger to push messages

Extend the Swagger to push messages (CAP...) for testing to the AAR Backend

see #1


Add drag and drop for music files

Using love.filedropped and love.directorydropped.

Added in https://github.com/nabakin/drop/commit/8bbf26f05bc21db9c2f9de0ba48b80af4ad30b68


Write support

I had opened a PR against ReadStat.jl a while ago to implement writing: https://github.com/queryverse/ReadStat.jl/pull/87
However, it does not seem it's going to be merged as that package is not supposed to pull in Tables.jl. I could port the implementation to ReadStatTables.jl, which we are using anyway to read these files, so it makes sense from our perspective.
Would you be interested in such a PR? If yes, do you have any suggestions before I get started transferring the code, any particular interface you already had in mind?

Thanks for making the offer!
The write support is definitely something that will be added to this package at some point, tentatively in the v0.3.0 release. I haven't done so immediately as I plan to finish some remaining enhancement on the reading part before getting started on the writing part. But, to get things move faster on the writing part, it seems to make sense to have an experimental write support in the v0.2.x phase. I would probably get back to you in several days on this.

That sounds good, thank you. I'd be willing to work on this to get a first version working as soon as possible.

@jkrumbiegel If you are still interested in taking the lead on the writing part, I will be happy to incorporating these changes into the package.
Regarding the design, I imagine that there could be two options: 1) directly iterating over any Tables.jl-compatible table specified by the user in a fashion similar to the PR to ReadStat.jl; 2) converting the table to ReadStatTable before writing the data. The first option avoids copying data but the row iteration could be somewhat slow depending on the type of that table. The second option may involve some data copying but row iteration with ReadStatTable should be fast and it also allows collecting all metadata including value labels in one place.
Regarding the file organization, please put functions that merely wrap the ccall in src/wrappers.jl in an order that matches how they show up in readstat.h. Please add a new file named src/writer.jl for code that handles the writer. There could be an additional file named src/writestat for the interface exposed to end users, including docstrings. The function for end users is tentatively named writestat.

I will take a look next week, sounds good!

I am closing this issue as the first attempt on write support is arriving with the v0.2.3 release. Future development will target toward the v0.3 series due to planned breaking changes.


[native_assets_builder] Builder out/ folder

Closes:

https://github.com/dart-lang/native/issues/98

coverage: 98.642%. remained the same when pulling 817bd022ff7c9eae3bffba0370216b3d3950c887 on builder-out-folder into 63083306d69058424308b9e441295f8863e39a45 on main.

Thanks @HosseinYousefi !


Add Appendix Effort

Codecov Report

Merging #255 (7508e76) into master (5f2b43b) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #255   +/-   ##
=========================================
  Coverage     70.31%   70.31%
  Complexity      801      801
=========================================
  Files           111      111
  Lines          2732     2732
  Branches        366      366
=========================================
  Hits           1921     1921
  Misses          701      701
  Partials        110      110

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Loosen Semver to Match Kedro Deps

https://github.com/kedro-org/kedro-viz/blob/93caf4ab3db516d40249ea7f1e86648310176e1b/package/requirements.txt#L1
semver~=3.0
uvicorn[standard]~=0.22.0
https://github.com/getindata/kedro-vertexai/pull/125

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender


invoke on Mac: ALT/OPT+CMD+0

It would be helpful to others of the keystrokes to invoke on Mac:
ALT/OPT+CMD+0
were included in the instructions.

I've just updated the README and put in a pull request :-)


Fix Emscripten link rule wrong arguments position

Fix the LINK_EXECUTABLE_RULE makefile rule in Emscripten builds to have
the correct list of arguments: the rule was incorrectly written as if
there was an additional first argument that specifies the name of the
executable to be linked. In reality, as documented in the "interface"
file and implemented in the NaCl build scripts, there's no such
argument, and the TARGET variable should be used for that purpose.
This fixes the mistake introduced in the original PR #208. The umbrella
issue that tracks the Emscripten build implementation is #177.

Thanks for the quick review!


After changing the album, the automatically selected image is not selected.

In single image selection, if you click Next on the first image automatically selected for the first time after changing the album, the image of the previous album will be processed.

This is a very serious bug indeed, I'm trying to fix it for the next (swift5) version :)
Will report back when it's fixed !

@jasudev This should be fixed by https://github.com/Yummypets/YPImagePicker/pull/340, this will be part of the upcoming release :)

This is now solved in https://github.com/Yummypets/YPImagePicker/releases/tag/4.0.0


Cant search multiples pages while working with Google()

from search_engines import Google, Bing, Yahoo, Ask, Dogpile, Duckduckgo, Aol, Mojeek, Startpage
google = {}
action_words = ['affimative action']

def search_word(words):
      for word in words:
            engine = Google()
            results = engine.search(word, pages = 10)
            links = results.links()
            num = 1
            for link in links:
                google[num] = link
                num = num+1

search_word(action_words)

The output shows only the first 10 links. It would be really helpful if you can let me know what I am doing wrong. I would like to extract all the links in the first 10 pages.
I checked with bing and yahoo which are working fine and I am able to get all the links in the first 10 pages.
Duckduckgo and Startpage are not working

Thanks for bringing this to my attention. It seems we have to change the selectors for those three search engines.

Fixed the next page selector for Google, now it should return more than one pages. I think Duckduckgo and Startpage made much more drastic changes to their HTML structure and it will take some time to update the code. It's probably doable (unless Js is absolutely necessary for HTML to  render) and I'll take care of it when I have a couple of hours to spare.

Thank you! I am looking forward to the update.


No error, no response

I'm initializing correctly with: var fcm = new FCM(mykeyhere);
That returns a pretty object that looks correct.
Later in the story, I do a:
        var fcmMessage = {
          to: [ channel ],
          data: {
            "content-available": 1,
            attachment_id: idgoeshere,
          },
          notification: {
            title: 'New attachment',
            body: "Message goes here"
          }
        };

followed up by a:
        fcm.send(fcmMessage, function(error, response){
          if (error) {
            return console.error("error while sending", channel, "push notification", error);
          } else {
            console.log("sent to fcm", channel, "with response", response);
          }
        });

The log shows that response is "sent to fcm channelhere with response null".
If the fcm object initializes correctly, and there is no error in the fcm.send callback, why is response null?
What is the disconnect?
Thanks.

As an update, I copy-pasted the example on the fcm-push main page, put in my key and change the message = {} slightly.  Same 'null' success response:
var FCM = require('fcm-push');

var serverKey = '';
var fcm = new FCM('*********');

var message = {
    to: '***************', // required fill with device token or topics
    data: {
        "content-available": 1,
        attachment_id: 'tango',
    },
    notification: {
        title: 'Title of your push notification',
        body: 'Body of your push notification'
    }
};

//callback style
fcm.send(message, function (err, response) {
    if (err) {
        console.log("Something has gone wrong!");
    } else {
        console.log("Successfully sent with response: ", response);
    }
});

Same problem here, when the message is sent the response is null

After much research, I found the issue.  I was sending to a Firebase topic (not a device token).  So my 'to' field in my message needed to be prefixed with "/topics".    Instead of "channel_name" I changed it to "/topics/channel_name" and it worked.
Postmortem: This should be detected and the error messaging should be improved, or the docs themselves should be improved.

@morfie78 do you think the detection should be as feature of this npm or as firebase doc/feature?

I think it would be a nice thing for the docs on this npm to add this note, in order to avoid issues like this in the future.


M90GL functionality

Description

unused assault rifle with non-functioning under-barrel grenade launcher

Thanks for fixing it, always looked like a cool admin weapon for downstream

pro this is an issue

Thanks for fixing it, always looked like a cool admin weapon for downstream

Only just now realised this ain't a pr lmfao

professional


EC2 service controller

New ACK Service Controller
Support for EC2
List of API resources
List the API resources in order of importance to you:

[resource 1]
[resource 2]

Hi @mdykes-gw, would you mind filling out a list of the EC2 API resources that you would prioritize? It would be helpful for us to plan this (very very large) controller. (Last time I generated the EC2 service controller it was over 300K lines of code...)

I think this is a good starting point. Would love to hear what others would like!

Instance (or w/ EC2Fleet request type: Instant)
LaunchTemplate
Volume
Snapshot(s)
Image
Fleet
SpotFleet
CapacityReservation
PlacementGroup
ManagedPrefixList

@mdykes-gw what do you think of @bwagner5's resource list above? Would that be a good priority list in your opinion?

@jaypipes - would be great to see this aws ec2 service controller land into this project - especially for things like resource "adoption" os coming . any thoughts on effort to land such an initial support?

Hi @gadinaor-r7! We're aiming to tackle core EC2 VPC networking primitives in the second half of this year (things like security groups, subnets, etc). See #489 for information about that. For the resources listed in @bwagner5's comment above, it may be possible to get to some of those later this year.

/remove-lifecycle stale

/remove-lifecycle stale

/lifecycle frozen