pylint 2.11 reports false positive R0401 (cyclic-import)

Actual behavior
Since version 2.11.0, pylint reports cyclic-import in the zhmccli project:
make pylint
pylint --disable=fixme --rcfile=.pylintrc --output-format=text setup.py zhmccli/__init__.py zhmccli/_cmd_adapter.py zhmccli/_cmd_capacitygroup.py zhmccli/_cmd_cpc.py zhmccli/_cmd_hba.py zhmccli/_cmd_info.py zhmccli/_cmd_lpar.py zhmccli/_cmd_metrics.py zhmccli/_cmd_nic.py zhmccli/_cmd_partition.py zhmccli/_cmd_port.py zhmccli/_cmd_session.py zhmccli/_cmd_storagegroup.py zhmccli/_cmd_storagevolume.py zhmccli/_cmd_vfunction.py zhmccli/_cmd_vstorageresource.py zhmccli/_cmd_vswitch.py zhmccli/_helper.py zhmccli/_version.py zhmccli/zhmccli.py   tests/__init__.py tests/function/__init__.py tests/function/test_global_opts.py tests/function/test_info.py tests/function/utils.py
************* Module tests.function.utils
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_vstorageresource -> zhmccli._cmd_port -> zhmccli._cmd_adapter -> zhmccli._cmd_cpc) (cyclic-import)
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_vstorageresource -> zhmccli._cmd_port) (cyclic-import)
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_vstorageresource -> zhmccli._cmd_storagegroup -> zhmccli._cmd_cpc) (cyclic-import)
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_vstorageresource -> zhmccli._cmd_port -> zhmccli._cmd_adapter) (cyclic-import)
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_nic -> zhmccli._cmd_partition -> zhmccli._cmd_cpc) (cyclic-import)
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_capacitygroup -> zhmccli._cmd_partition -> zhmccli._cmd_cpc) (cyclic-import)
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_hba -> zhmccli._cmd_partition -> zhmccli._cmd_cpc) (cyclic-import)
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_info) (cyclic-import)
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_lpar -> zhmccli._cmd_cpc) (cyclic-import)
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_nic -> zhmccli._cmd_partition -> zhmccli._cmd_metrics) (cyclic-import)
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_partition -> zhmccli._cmd_cpc) (cyclic-import)
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_vstorageresource -> zhmccli._cmd_storagegroup) (cyclic-import)
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_adapter -> zhmccli._cmd_cpc) (cyclic-import)
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_session) (cyclic-import)
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_storagevolume -> zhmccli._cmd_port -> zhmccli._cmd_adapter -> zhmccli._cmd_cpc) (cyclic-import)
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_vfunction -> zhmccli._cmd_partition -> zhmccli._cmd_cpc) (cyclic-import)
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_nic -> zhmccli._cmd_partition -> zhmccli._cmd_storagegroup -> zhmccli._cmd_port -> zhmccli._cmd_adapter -> zhmccli._cmd_cpc) (cyclic-import)
tests/function/utils.py:1:0: R0401: Cyclic import (zhmccli -> zhmccli._cmd_vswitch -> zhmccli._cmd_cpc) (cyclic-import)

Pylint 2.10.0 did not report that issue.
Expected behavior
Pylint 2.11 should not report that issue.
Execution environment

zhmccli version: master
zhmcclient version:
Operating system (type+version):
HMC version:
CPC version:

PRs #235 in master and #236 in 1.0.2 have a workaround for the issue by disabling pylint cyclic-import.
I have created pylint issue


Forking i18n-ally

To complete the forking process the following tasks are pending:

[ ] Publish on https://open-vsx.org/
[ ] Change README.md logo
[ ] Change VSCode sidebar logo
[ ] List supported frameworks again

What is the reason for this fork?

We rely on i18n-ally in some projects, but had a little bug inside the Transloco framework support. In October 21 I've sent a pull request about this and since then I never got any feedback. I've asked if I can support the project as a contributor here, but the owners never really got in touch with me.
The project itself has quite some technical dept already (webpack 4, outdated vscode integration, test coverage, dependency lock in) and there is not really an alternative, so I thought it's a good idea to fork the project.


fix(amplify-category-function): check for new function when adding permissions

Description of changes

A function attempting to add permissions is handled as new function when is not found in amplify meta.
Issue #, if available

fix #7970
Description of how you validated changes

Unit Testing yarn lerna --scope amplify-category-function run test
Manual Testing

Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added
[x] Relevant documentation is changed or added (and PR referenced)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #8017 (57fe47d) into master (b0600c8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #8017   +/-   ##
=======================================
  Coverage   54.04%   54.04%
=======================================
  Files         592      592
  Lines       33380    33380
  Branches     6917     6917
=======================================
  Hits        18040    18040
  Misses      14568    14568
  Partials      772      772

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b0600c8...57fe47d. Read the comment docs.


[New Port Request] bliss

Library name
bliss
Library description
bliss is an open-source tool for computing canonical labelings and automorphism groups of graphs. Its algorithms are described in the following papers, please cite them if you use bliss in your research or software tools.
Source repository URL
https://users.aalto.fi/~tjunttil/bliss/download.html
Project homepage (if different from the source repository)
https://users.aalto.fi/~tjunttil/bliss/
Anything else that is useful to know when adding (such as optional features the library may have that should be included)
No response

Keep it open.

Any pointers for a person to start porting a new library?


pynautobot reqeust error

Environment

Python version: 3.6.8
Nautobot version: 1.0.3

Steps to Reproduce

When I am trying to match a tag when running a nornir script
nb_url= 'https://test-nautobot.lab.com'
    nb_token= "hiddenhidden"
    """Nornir testing."""
    nr = InitNornir(
        inventory={
            "plugin": "NautobotInventory",
            "options": {
                "nautobot_url": nb_url,
                "nautobot_token": nb_token,
                "ssl_verify": False,
            },
            "transform_function": "populate_creds",
        },
    )

Expected Behavior
to match the filtered tag

Observed Behavior
File "/home/test/pynet_nautobot/lib64/python3.6/site-packages/pynautobot/core/query.py", line 226, in _make_call
    raise RequestError(req)
pynautobot.core.query.RequestError: The request failed with code 504 Gateway Time-out but more specific details were not returned in json. Check the Nautobot Logs or investigate this exception's error attribute.

I believe the original report was at https://github.com/nautobot/pynautobot/issues/23, and includes some additional details not presented here.

Thanks for the report. Can you provide more details as to where exactly the error is being seen? Is it coming from your InitNornir call itself, or from subsequently when you're using the returned object to run a specific query? Is Nornir actually a factor at all or is the error coming from a direct call to pynautobot?

thanks for looking into it. Yes it is generated from the init call itself
python PynetBoxCallableBackup.py  -f tag:test_api
> /home/test/pynet_nautobot/nornir_scripts/PynautobotCallableFilter.py(13)CallableFilterPynautobot()
     12
---> 13     nb_url= 'https://nautobot.lab.com'
     14     nb_token= "hidden"

ipdb> n
> /home/test/pynet_nautobot/nornir_scripts/PynautobotCallableFilter.py(14)CallableFilterPynautobot()
     13     nb_url= 'https://nautobot.lab.com'
---> 14     nb_token= "hidden"
     15     """Nornir testing."""

ipdb> n
> /home/test/pynet_nautobot/nornir_scripts/PynautobotCallableFilter.py(16)CallableFilterPynautobot()
     15     """Nornir testing."""
---> 16     nr = InitNornir(
     17         inventory={

ipdb> n
> /home/test/pynet_nautobot/nornir_scripts/PynautobotCallableFilter.py(18)CallableFilterPynautobot()
     17         inventory={
---> 18             "plugin": "NautobotInventory",
     19             "options": {

ipdb> n
> /home/test/pynet_nautobot/nornir_scripts/PynautobotCallableFilter.py(20)CallableFilterPynautobot()
     19             "options": {
---> 20                 "nautobot_url": nb_url,
     21                 "nautobot_token": nb_token,

ipdb> n
> /home/test/pynet_nautobot/nornir_scripts/PynautobotCallableFilter.py(21)CallableFilterPynautobot()
     20                 "nautobot_url": nb_url,
---> 21                 "nautobot_token": nb_token,
     22                 "ssl_verify": False,

ipdb> n
> /home/test/pynet_nautobot/nornir_scripts/PynautobotCallableFilter.py(22)CallableFilterPynautobot()
     21                 "nautobot_token": nb_token,
---> 22                 "ssl_verify": False,
     23             },

ipdb> n
> /home/test/pynet_nautobot/nornir_scripts/PynautobotCallableFilter.py(26)CallableFilterPynautobot()
     25         },
---> 26         runner={"plugin": "threaded", "options": {"num_workers": 20}},
     27         logging = {"enabled" : False}

ipdb> n
> /home/test/pynet_nautobot/nornir_scripts/PynautobotCallableFilter.py(27)CallableFilterPynautobot()
     26         runner={"plugin": "threaded", "options": {"num_workers": 20}},
---> 27         logging = {"enabled" : False}
     28     )

ipdb> n
/home/test/pynet_nautobot/lib64/python3.6/site-packages/urllib3/connectionpool.py:1020: InsecureRequestWarning: Unverified HTTPS request is being made to host '-nautobot.lab.com'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/1.26.x/advanced-usage.html#ssl-warnings
  InsecureRequestWarning,
/home/test/pynet_nautobot/lib64/python3.6/site-packages/urllib3/connectionpool.py:1020: InsecureRequestWarning: Unverified HTTPS request is being made to host '-nautobot.lab.com'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/1.26.x/advanced-usage.html#ssl-warnings
  InsecureRequestWarning,
pynautobot.core.query.RequestError: The request failed with code 504 Gateway Time-out but more specific details were not returned in json. Check the Nautobot Logs or investigate this exception's error attribute.

So during the init call for this it is using the REST API through pynautobot to gather device information. The specific all is a pynautobot get all devices - https://github.com/nautobot/nornir-nautobot/blob/develop/nornir_nautobot/plugins/inventory/nautobot.py#L111-L125
It is likely having a timeout within there from the gateway of gathering the devices.
How many devices are within Nautobot at this point in time?

it is working intermittently. We have around 2300 devices and 2000 cables

I checked both CPU and Memory. All looks good for the last two weeks. Even in this condition, I am having the same problem.
Tasks: 188 total,   1 running, 187 sleeping,   0 stopped,   0 zombie
%Cpu(s):  1.0 us,  2.7 sy,  0.0 ni, 96.3 id,  0.0 wa,  0.0 hi,  0.0 si,  0.0 st
KiB Mem : 16247540 total,   951580 free,  2618128 used, 12677832 buff/cache
KiB Swap:  4190204 total,  4184060 free,     6144 used. 12696888 avail Mem

could it be pagination problem? how I can change the pagination from 1000 to more ?


add websocket config to settings

Motivation and Context

Adds websockets allowed_request_origins setting to the settings files, so that they can be overwritten in e.g. a settings.local.yml
Your Changes

Description:
Added setting to settings.yml, as well as production and development
Type of change (select all that apply):

[x] Bug fix (non-breaking change which fixes an issue)

Testing

Questions and Comments (if applicable)

Checklist

[x] I have performed a self-review of my own code.
[x] I have verified that the pre-commit.ci checks have passed.
[x] I have verified that the CI tests have passed.
[x] I have reviewed the test coverage changes reported on Coveralls.
[x] I have made changes to the documentation and linked to that pull request below.

Pull request to make documentation changes (if applicable)

https://github.com/MarkUsProject/Wiki/pull/196
Pull request to make documentation changes (if applicable)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 91.814%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
38230

Relevant Lines:
41074

💛  - Coveralls

@david-yz-liu thanks for the review! Updated the naming. I did add the setting to application.rb, which I think is sufficient? At the very least, the setting has the correct value when read from the rails console.

You're totally right and I forgot that file existed - thanks for the catch @david-yz-liu ! I made both action_cable and web_socket_allowed_request_origins optional because I don't think they are strictly necessary to actually having markus up, but let me know if you have other thoughts.


License added

Change Copywright to "Copyright (c) 2020-21 Application Library Engineering Group" in LICENSE

Yes done!


ENH: linalg: Cholesky factorization update/downdate

Closes https://github.com/scipy/scipy/issues/8188
This PR picks up where a previous now seemingly abandoned PR left off (https://github.com/scipy/scipy/pull/8286) to implement update and downdate for the Cholesky factorization.
On top of the previous PR rank-k updates are also implemented using an algorithm described in [a paper] [0] (https://www.cs.utexas.edu/users/flame/pubs/flawn41.pdf) by van de Geijn and Van Zee. The rank-1 downdates use the algorithm by Pan [1] (unfortunately I couldn't find a non-paywalled paper), and the updates use the same modification as in #8286.
The code is in cython for speed. A simple benchmark comparing rank-1, rank-2 and rank-20 updates to redoing the factorization from scratch shows that the rank-1 update has good speedup, but the rank-k updates are not as good.

If deemed necessary it should be possible to

fix the different performance depending on the contiguity of input matrices
use block rotations to speed up the rank-k updates (not sure exactly how, references are sparse on this).

Let me know what your thoughts on this are,
Giacomo
[0]: Van de Geijn, R. A., Van Zee, F. G. "High-Performance Up-and-Downdating via Householder-like Transformations". ACM Trans. Math. Softw. 38, 1-17 (2011).
[1]: Pan, C-T. "A modification to the LINPACK downdating algorithm." BIT Numerical Mathematics 30.4 (1990): 707-722.

Would be great to see this added to scipy! Fast rank one updates would be super useful.

Hello @Giodiro 👋🏻  - I know it's been a long time, apologies for the lack of response! Would you be willing to continue working on this? It seems like there is some interest for this change. Please let us know if you need assistance. Cheers!

Hi @melissawm
Sure! Other than a cleanup and following @matusvalo 's suggestion, I think the next step is getting a proper review?
Other than that, deciding whether we want to keep rank-1 update code only or include rank-p updates as well (they're implemented separately) would be useful.


BUG: Clarify install instructions

We should make it clear that people should install Python2 for now, due to Mayavi.
Also, we need to make sure the instructions still work. Apparently Anaconda has been having some problems due to PyQt5 / matplotlib / mayavi junk. I suspect we'll need to add a line, at least for a little while, along the lines of conda install numpy scipy sklearn mayavi matplotlib to force dependency resolution. And this stuff should be backported.
Thanks @bloyl for suffering through the existing directions.

I have installed mne-python and been through this recently so I'd be happy to help.
I tried to reproduce my installation steps this morning, but I have an issue when importing numpy (ImportError: libopenblasp-r0-ea933816.2.18.so: cannot open shared object file: No such file or directory)
Steps to reproduce:
conda create --name mne python=2.7 ipython && source activate mne
conda install numpy scipy mayavi scikit-learn matplotlib
(mne) mathurin@mathurin-HP-EliteBook-840-G3:~$ which ipython
/home/mathurin/anaconda3/envs/mne/bin/ipython
(mne) mathurin@mathurin-HP-EliteBook-840-G3:~$ ipython
Python 2.7.12 |Continuum Analytics, Inc.| (default, Jul  2 2016, 17:42:40)
Type "copyright", "credits" or "license" for more information.

IPython 5.1.0 -- An enhanced Interactive Python.
?         -> Introduction and overview of IPython's features.
%quickref -> Quick reference.
help      -> Python's own help system.
object?   -> Details about 'object', use 'object??' for extra details.

In [1]: import numpy
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-1-5a0bd626bb1d> in <module>()
----> 1 import numpy

/home/mathurin/anaconda3/envs/mne/lib/python2.7/site-packages/numpy/__init__.py in <module>()
    178         return loader(*packages, **options)
    179
--> 180     from . import add_newdocs
    181     __all__ = ['add_newdocs',
    182                'ModuleDeprecationWarning',

/home/mathurin/anaconda3/envs/mne/lib/python2.7/site-packages/numpy/add_newdocs.py in <module>()
     11 from __future__ import division, absolute_import, print_function
     12
---> 13 from numpy.lib import add_newdoc
     14
     15 ###############################################################################

/home/mathurin/anaconda3/envs/mne/lib/python2.7/site-packages/numpy/lib/__init__.py in <module>()
      6 from numpy.version import version as __version__
      7
----> 8 from .type_check import *
      9 from .index_tricks import *
     10 from .function_base import *

/home/mathurin/anaconda3/envs/mne/lib/python2.7/site-packages/numpy/lib/type_check.py in <module>()
      9            'common_type']
     10
---> 11 import numpy.core.numeric as _nx
     12 from numpy.core.numeric import asarray, asanyarray, array, isnan, \
     13                 obj2sctype, zeros

/home/mathurin/anaconda3/envs/mne/lib/python2.7/site-packages/numpy/core/__init__.py in <module>()
     12 if 'GOTOBLAS_MAIN_FREE' not in os.environ:
     13     os.environ['GOTOBLAS_MAIN_FREE'] = '1'
---> 14 from . import multiarray
     15 os.environ.clear()
     16 os.environ.update(envbak)

ImportError: libopenblasp-r0-ea933816.2.18.so: cannot open shared object file: No such file or directory

Contrary to what's mentionned here, conda install libgfortran==1
does not change the error.
And the versions as shown by pip freeze are:
(mne) mathurin@mathurin-HP-EliteBook-840-G3:~$ pip freeze
apptools==4.2.1
backports.shutil-get-terminal-size==1.0.0
configobj==5.0.6
decorator==4.0.10
enum34==1.1.6
envisage==4.4.0
ipython==5.1.0
ipython-genutils==0.1.0
matplotlib==1.4.3
mayavi==4.4.0
mne==0.13
nibabel==2.1.0
nose==1.3.7
numpy==1.9.3
path.py==0.0.0
pathlib2==2.1.0
pexpect==4.0.1
pickleshare==0.7.4
prompt-toolkit==1.0.8
ptyprocess==0.5.1
pyface==4.4.0
Pygments==2.1.3
pyparsing==2.0.3
pysurfer==0.6
python-dateutil==2.5.3
pytz==2016.7
scikit-learn==0.16.1
scipy==0.17.1
simplegeneric==0.8.1
six==1.10.0
traitlets==4.3.1
traits==4.4.0
traitsui==4.4.0
VTK==5.10.1
wcwidth==0.1.7

I cannot install numpy=1.10 because of mayavi requirements, nor numpy=1.8 because of fontconfig requirements.

The numpy import can be fixed using numpy=1.9.0 and libgfortran=1
Now import surfer fails (surfer is installed with pip, with pip correctly linked to the environment), with error message:
/home/mathurin/anaconda3/envs/mne/lib/python2.7/site-packages/traits/trait_base.py in <module>()
     32 from os.path import dirname, exists, join
     33
---> 34 from . import _py2to3
     35
     36 # backwards compatibility: trait_base used to provide a patched enumerate

ImportError: cannot import name _py2to3

Removing and reinstalling traits as suggested here does not solve the problem.

It turns out installing numpy=1.9.0 along with libgfortran==1 allows me to import numpy. I had issues importing surfer, caused by import traits.api (ImportError: cannot import name _py2to3), which was apparently due to conflicting install of traits=4.5.0 and 4.4.0 in ~/anaconda3/pkgs. As adviced here I removed the traits related folders and reinstalled traits=4.4.0 with conda, which solved the issue. But now import surfer fails, with error message:
/home/mathurin/anaconda3/envs/mne/lib/python2.7/site-packages/tvtk/array_handler.py in <module>()
     26
     27 # Enthought library imports.
---> 28 from tvtk.array_ext import set_id_type_array
     29 from tvtk.common import is_old_pipeline
     30

ImportError: numpy.core.multiarray failed to import

I have in the past had such problems ... but using pip to upgrade matplotlib and numpy after installing everything via conda seems to have worked for me

I wouldn't recommend using pip to install numpy in an Anaconda install. Anaconda by default uses MKL, and installing via pip will not. I'm a bit surprised this didn't break a lot of stuff.

I can't manage to find out why pysurfer isn't built with the correct version of numpy; 1.9.0 is the only one installed in my environment. I have also tried building pysurfer from source, but I still obtain RuntimeError: module compiled against API version a but this version of numpy is 9

I have not found a clean way to install a working version of mayavi using only conda. However I'm posting this as it might help  others in the future: my solution was to install everything with conda, then update mayavi to 4.5 and numpy to 1.11 with pip:
conda create --name test_mne python=2.7  matplotlib scipy scikit-learn ipython numpy mayavi
source activate test_mne
pip install -U mayavi numpy

It is also worth adding that if I launch ipython --matplotlib=qt, the following import crashes
from mayavi import mlab
with error ValueError: API 'QDate' has already been set to version 1
The workaround is to launch ipython without options, and as first command input
from pyface.qt import QtGui, QtCore
%matplotlib qt

@Eric89GXL

We should make it clear that people should install Python2 for now, due to Mayavi.

Are you sure? I installed mayavi on python 3 with conda:
conda install -c menpo mayavi

and I have no problems with mayavi (except occasional window freezing in jupyter which I also get for matplotlib in some cases).

I'm not sure if we should recommend using an unofficial repository.
@agramfort WDYT?

Yeah, it's not official but it was recommended once in mayavi github issue. I can only say that it works for me.

I also use an unofficial mayavi build on conda with py35
i tried many things but apparently using vtk7 from satra and mayavi from
pip works on my machine (TM)

This combination seems to work for me (OS X, Python 2.7):
$ conda install numpy scipy matplotlib mayavi h5py pyqt=4.10 pygments=1
$ pip install mne imageio pysurfer
With the caveat mentioned by @mathurinm  that I have to import mlab before %matplotlib qt

We should probably also break up the Python install page (https://mne-tools.github.io/stable/install_mne_python.html) to have the advanced steps separate. That way the install instructions almost fit on a single page, which is more user-friendly.

+1

Just since this hasn't been mentioned in this thread, I notice that Anaconda released mayavi 4.5, that should make the installation simpler again.

do you have a link about his mayavi package?

A link to what @agramfort ?

saying that mayavi is now on anaconda in py3k

It's not in Python 3, I just noticed it with conda install mayavi (and it does not appear if this is run from a py3 env).

ok so no update then

the update I’m referring to is 4.4->4.5

On Jan 11, 2017, at 2:27 PM, Alexandre Gramfort<EMAIL_ADDRESS>wrote:
ok so no update then
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/mne-tools/mne-python/issues/3723#issuecomment-271867911, or mute the thread https://github.com/notifications/unsubscribe-auth/AAI5a9caE4MF2q2CBlHunrUYt3fckvD5ks5rRNi4gaJpZM4Kmyer.

Just for completeness. We recounted the same problem when trying to reproduce this example.


Roles feature

Will review all code and double-check everythings in working order. Code coverage is done and builds are passing.

Reviewing the code I see I need to make the menu items dependant on role too. 🐰


Feature/wallet preparetx

WIP

need to write testcases

This closes #6

Sorry this PR is so large ><; ListUnspent() is longer than I thought it would be.
This PR:

refactored Wallet creation. DB and Wallet creation separated into 2 functions.
implemented ListUnspent(). Its complexity comes from converting Credit type into btcjson.ListUnspentResult type :,(

This PR:

added privatePassphrase as a parameter in the wallet struct
refactored CreateWallet() and Open() to return wallet struct instead of interface
implemented wallet.ListUnspent()

This PR:

CreateWallet returns a Wallet interface instead of wallet struct (changed from an earlier PR)
added getter functions for wallet's manager and txStore


Fix Issue 10126 - Make TaskPool terminate on its own or improve docs

From the docs of the function taskPool it's clear, that the current behaviour is intentional. Therefore I've choosen to improve the docs. :-)

Auto-merge toggled on


models/os: Always first normalize the device type slug

Change-type: patch
Signed-off-by: Thodoris Greasidis<EMAIL_ADDRESS>
Contributor checklist

[ ] Includes tests
[ ] Includes typings
[ ] Includes updated documentation
[ ] Includes updated build output

@balena-ci I self-certify!


Exec tachikoma update<PHONE_NUMBER>3448

:hamster::hamster::hamster:   Strategy none
Powered by Interval Pull Request as a Service - Tachikoma.io Announcement: 2015-06-28 Updated permissions - Tachikoma.io

Coverage remained the same at 68.0% when pulling 9cf9c29d970ae0241080d51b46bcea2863551eac on pazjacket:tachikoma/update-20160626163448 into 0c477cea399180e0dc0455b31d685efdbee0f3af on sanemat:master.


The output is whole dark image

I cannot get correct result from the python code, but when I use the command like
' autocolorize grayscale.png -o colorized.png' I get the correct results
My code is following:
import cv2
import os
import autocolorize
path2 = 'input/'
path3 = 'output/'
files= os.listdir(path2)
classifier = autocolorize.load_default_classifier()
for ff in files:
gray_image = cv2.imread(path2+ff)
rgb_image = autocolorize.colorize(gray_image, classifier=classifier)
cv2.imwrite(path3+rgb_image,rgb_image)

I cannot get correct result from the python code, but when I use the command like
' autocolorize grayscale.png -o colorized.png' I get the correct results
My code is following:
import cv2
import os
import autocolorize
path2 = 'input/'
path3 = 'output/'
files= os.listdir(path2)
classifier = autocolorize.load_default_classifier()
for ff in files:
gray_image = cv2.imread(path2+ff)
rgb_image = autocolorize.colorize(gray_image, classifier=classifier)
cv2.imwrite(path3+rgb_image,rgb_image)

How do you make out with it ?

You need to use the data reading and writing method in the code:
import cv2
import os
import autocolorize
import numpy as np
import torch
import time
from tqdm import tqdm

def load(path, dtype=np.float64):
   """
   Loads an image from file.
   Parameters
   ----------
   path : str
       Path to image file.
   dtype : np.dtype
       Defaults to ``np.float64``, which means the image will be returned as a
       float with values between 0 and 1. If ``np.uint8`` is specified, the
       values will be between 0 and 255 and no conversion cost will be
       incurred.
   """
   import skimage.io
   im = skimage.io.imread(path)
   if dtype == np.uint8:
       return im
   elif dtype in {np.float16, np.float32, np.float64}:
       return im.astype(dtype) / 255
   else:
       raise ValueError('Unsupported dtype')

def save(path, im):
   """
   Saves an image to file.
   If the image is type float, it will assume to have values in [0, 1].
   Parameters
   ----------
   path : str
       Path to which the image will be saved.
   im : ndarray (image)
       Image.
   """
   from PIL import Image
   if im.dtype == np.uint8:
       pil_im = Image.fromarray(im)
   else:
       pil_im = Image.fromarray((im * 255).astype(np.uint8))
   pil_im.save(path)

path2 = '/home/rpf/tgp/Colorization/colorization_valset/celebahq_val_256/'
path3 = '/home/rpf/tgp/Colorization/colorization_valset/celebahq_val_256_results/'
files = os.listdir(path2)
import caffe

caffe.set_mode_gpu()
classifier = autocolorize.load_default_classifier()
time_costs = []
skip = 5
for ff in tqdm(files):
   skip -= 1
   gray_image = load(path2 + ff)
   # torch.cuda.synchronize()
   start = time.time()
   rgb_image = autocolorize.colorize(gray_image, classifier=classifier)
   # torch.cuda.synchronize()
   end = time.time()
   cost = (end - start) * 1000
   if skip < 0:
       time_costs.append(cost)
   save(path3 + ff, rgb_image)
print(np.average(time_costs))


Resize on rotated node is not accurate.

Describe the Bug
The NodeResizer does not take into account the rotation of the node.
Your Example Website or App
https://reactflow.dev/docs/examples/nodes/resize-rotate/
Steps to Reproduce the Bug or Issue
Rotate a node to 90degrees and then, try to resize it. You'll see that the resizing is logical but not accurate for a human.
Expected behavior
As a user, I expected to resize a rotated node easily, but I am seeing that the node is not rotated like a human could expect.
Screenshots or Videos
No response
Platform

OS: Windows
Browser: Firefox
Version: 11.3.3

Additional context
No response

the same issue

The NodeResizer doesn't really support rotated nodes, because the rotation is not a built-in feature. I will leave this open until we make a decision if we want to integrate rotation.

We will not add rotation as a built-in feature for React Flow. If you need resizing and rotation, you would need to implement resizing in the custom node on your own since the built-in NodeResizer doesn't work with rotated nodes.
We will update the docs to make this more clear.


Remove Grammar doc comments

These grammars were most of the time incomplete, incorrect and in the case of recovery wrong by definition. Instead of having misleading doc comments, let’s delete them entirely.

As incorrect as some of them are, I did find them quite useful. These comments helped me trace down what I needed (to use/change) for #1414. I'm not really opposed to this PR, but I do kind of worry that once they're deleted, the correct ones won't be added back.
Also as far as I understand, it's sometimes necessary for the parser's logic to deviate from the grammar, in order to produce good diagnostics. I wonder if there is a good way to document both the correct grammar and the looser grammar the parser uses, and the relation between their production rules?

Thinking more about this, the path forward here should be to generate a grammar for the Swift language from the syntax nodes because they will automatically be kept up-to-date if the layout of the syntax tree changes.

@WowbaggersLiquidLunch I’m generating the grammar in https://github.com/apple/swift-syntax/pull/1771. Do you think that would be a sufficient replacement of the current comments?

@swift-ci Please test

@swift-ci Please test


Bridge response error, please check the docs

I hosted this using hostinger and got this error. What are the possible reasons and how do I fix it?

Hello @jamiechoi can you look the backend file response?... Did you clone the repo with --recursive param? (in order to include the dependencies of php-classic framework)

Cloned the github repo with --recursive, removed all the dependencies, using Php bridge for connectivity, still getting the same "Bridge response error". Please suggest some way out.

Hi @sagarora77 Are you using another angular lib/framework that performs the requests to the backend bridge?

Hi @joni2back ,
No, I am not using any framework, just trying to implement it in localhost with valid ftp credentials and verified it with FileZilla. Having classic php handler in place. Need your help to figure out the exact problem.

Hi @sagarora77 Better install apache server xampp or wamp with Filezilla which will execute the code in handler.php

@sagarora77 @boloindia Of course you need to use also a php web server to run the ftp-connection handler :smile:

Thanks man for the support!
I solved the issue, the port number was the issue with my server.
Warmest regard
Sagar Arora
On Fri, Feb 5, 2016 at 6:11 PM, Jonas Sciangula Street <
<EMAIL_ADDRESS>wrote:

@sagarora77 https://github.com/sagarora77 @boloindia
https://github.com/boloindia Of course you need to use also a php web
server to run the ftp-connection handler [image: :smile:]
—
Reply to this email directly or view it on GitHub
https://github.com/joni2back/angular-filemanager/issues/82#issuecomment-180336743
.

--
Warm regards,
Sagar Arora

@joni2back I am getting the same error as well. As suggested by you, what exact values i need to put into config.js.

hi
I also have a problem. I have the following error
Error 405 - Bridge response error, please check the API docs or this ajax response
In your opinion, what's the problem?!

@itdotnet

Edit your index.html file by adding things,

@sachendrapal
what things?!
I thank you for your help

Hi @itdotnet
Are you using a backend server to have a connector for the filemanager?
Please, open your browser developer tools, and let do a screenshot in the ajax response...

hi @joni2back
i just used script files As mentioned in the documentation in visual studio and I run the project.
I have attached the output you

In addition, error in the FileZila to be shown

Hi, seems dont have the backend running well..
Are you really using an apache/nginx server..?

my project run in visual studio. how to i use apache server?!
I start Apache on xampp. but i dont know how to sync the two together.
apache service port changed my desired Because it was interfering.

@itdotnet If you have installed PHP, you can open a console in angular-filemanager folder then run
php -S<IP_ADDRESS>:8000

I'm having a 500 internal server error. I edited /bridges/php/includes/Ftp.php.
Angular-Filemanager runs on my debian server with apache2 server at https://ws.peastuti.com/angular-filemanager/ with this config
protected $_hostname  = 'peastuti.asuscomm.com/shares/folder/';
 protected $_username  = '------';
 protected $_password  = '------';
 protected $_port      = 21;
 protected $_timeout   = 90;
 protected $_passive   = true;
 protected $_debug     = false;
 protected $_conn   = false;
 protected $_ssl   = false;

Hi @peastuti , could you give the php error log for this error 500?
you could add this in the ftp.php code
error_reporting(E_ALL); ini_set('display_errors', E_ALL);

Nothing happens, here a screenshot

@peastuti You should check the PHP error_log of this server in order to know that error...

[Mon Aug 01 13:39:48.648549 2016] [:error] [pid 25283] [client <IP_ADDRESS>:10121] PHP Fatal error:  Class 'AngularFilemanager\\LocalBridge\\stream' not found in /var/www/ws.peastuti.com/public_html/angular-filemanager/bridges/php-local/LocalBridge/Rest.php on line 167 [Mon Aug 01 13:39:54.421605 2016] [:error] [pid 25202] [client <IP_ADDRESS>:10122] PHP Notice:  Undefined index: action in /var/www/ws.peastuti.com/public_html/angular-filemanager/bridges/php-local/LocalBridge/FileManagerApi.php on line 171 [Mon Aug 01 13:43:02.291250 2016] [:error] [pid 24798] [client <IP_ADDRESS>:64439] PHP Fatal error:  Uncaught exception 'InvalidArgumentException' with message 'Invalid connection' in /var/www/ws.peastuti.com/public_html/angular-filemanager/bridges/php/includes/Ftp.php:114\nStack trace:\n#0 /var/www/ws.peastuti.com/public_html/angular-filemanager/bridges/php/includes/Ftp.php(544): PHPClassic\\Ftp->_isConnected()\n#1 /var/www/ws.peastuti.com/public_html/angular-filemanager/bridges/php/includes/Ftp.php(561): PHPClassic\\Ftp->close()\n#2 [internal function]: PHPClassic\\Ftp->__destruct()\n#3 {main}\n  thrown in /var/www/ws.peastuti.com/public_html/angular-filemanager/bridges/php/includes/Ftp.php on line 114, referer: https://ws.peastuti.com/angular-filemanager/

I've received this error now:
Error 200 - Bridge response error, please check the API docs or this ajax response

Hi,
I'm getting the same error on 000webhost, but I noticed when I took out the width and height specifications from my css statement, the error went away?

I had the same problem when using WAMP PHP version and I resolve it by setting up an original version of PHP from the official website. Don't forget to update the Path environment variable which include the PHP folder you want to use.

Getting the same Error 405, in developer mode in the browser on line 99 in angular.min.js getting this error "Failed to load resource: the server responded with a status of 405 (Method Not Allowed)" how do i fix it?


[Doc][SLM] Update documentation to rollout SLM

This PR is the S1 part of https://github.com/mlc-ai/mlc-llm/issues/1420

The compile-time portion of the SLM documentation is ready (S1 in https://github.com/mlc-ai/mlc-llm/issues/1420). @rickzx will follow up with S2.
Will update the runtime portion of the documentation in a separate PR -- can only be done after we update all the precompiled model lib and weights.
cc @tqchen @junrushao @davidpissarra


Create handler for Report page

Work to be done:

Create a new url that links to {{url}}/report
P.S A route for /report/ currently exists and it's provided by the viewsets, you'll need to delete that to avoid a clash

when a user visits that url let it display a page where the user can see some basic report.
For now it can show an empty page that just shows the total number of people that have had breakfast or lunch - if possible include a date picker that can be used to crunch report from one date to another.

You can get some design ideas from this bootstrap theme.

@PriscillaSam Kindly hold off on this.


Update package-lock, engines.node >=16<EMAIL_ADDRESS>

This updates package-lock to lockfileVersion:2
needle to 3.2.0

https://github.com/tomas/needle/compare/v2.6.0...v3.2.0

I also moved engines.node to >=16 as all versions < 16 are EOL

I undid changing engines.node to 16, if that is needed it should be in its own pr.


Remove WSLPath

Checklist

[X] I've looked through the issues and pull requests for similar reports

Describe your issue
Currently, we use wsl -e wslpath -a ... to find the WSL2 path of a file for our mount point on Windows. This poses a few issues:

It requires external commands, which add significant overhead.
It does not work for UNC paths.

A simple test is as follows:
wsl -e wslpath -a "\\server\share"
wslpath: \\server\share

What target(s) are you cross-compiling for?
No response
Which operating system is the host (e.g computer cross is on) running?

[ ] macOS
[x] Windows
[ ] Linux / BSD
[ ] other OS (specify in description)

What architecture is the host?

[x] x86_64 / AMD64
[ ] arm32
[ ] arm64 (including Mac M1)

What container engine is cross using?

[ ] docker
[ ] podman
[ ] other container engine (specify in description)

cross version
latest main build
Example
No response
Additional information / notes
No response

for context, wslpath was added in https://github.com/cross-rs/cross/pull/665


Cannot access protected property 'EqualityContract' here

Given:
internal static class FooFactory
{
    public static Foo Create(Bar bar) => new();
}
record Foo;
record Bar;

Generated test:
[TestMethod]
public void CreatePerformsMapping()
{
    // Arrange
    var bar = new Bar();

    // Act
    var result = FooFactory.Create(bar);

    // Assert
    Assert.AreSame(bar.EqualityContract, result.EqualityContract);
}

Gives me this error:
Cannot access protected property 'EqualityContract' here.

Resolved in 0.127 🎈

Oh - forgot to say - thanks for the issue 😁


Use Environment.TickCount instead of DateTime.UtcNow when waiting for some task

Summary

Improve performance when scripts wait for the signals or when some task (e.g. loading some model) waits for something or the wait timeout expires at the cost of the resolution of the timer. It will be limited to in the range of 10 milliseconds to 16 milliseconds, but GTA V itself causes major issues when running at more than 187 FPS, so the new resolution shouldn't be a major issue too much.
No longer get influcenced by clock adjustment by the user

Why I made this PR instead of directly mading commits? Because I wonder there are better designs instead of exposing GetTickCount64 via WinApiWrapper. If there was Environment.TickCount64 in .NET Framework, I'd directly made commits for the change.

Environment.Tick does exist in .NET Framework, which IMHO would be cleaner to use. Just has the nasty detail of returning a signed 32-bit value, which can wrap around after just 25 days of uptime, so may have to add an extra check to ensure an overflow won't trigger an infinite loop. E.g.:
try
{
	checked
	{
		int resumeTime = Environment.TickCount + ms;

		do
		{
			waitEvent.Release();
			continueEvent.Wait();
		}
		while (Environment.TickCount < resumeTime);
	}
}
catch (OverflowException)
{
	// Tick count overflowed, can't wait until it settled again =(
}

I think we have a workaround without calling GetTickCount64. Calculate how much tick is elapsed since we don't expect the elapsed time to exceed more than about 25 days.

Ah right, that works. But IMHO this doesn't warrant a new class and file added, can just do this inline in all the places.


brew-bundle: fix Tty usage.

This has changed in Homebrew/brew.

Don't merge this until we've tagged a new Homebrew/brew version.


OTA example application

Hi! Really thanks for this application.  Great work!
We were trying to do OTA firmware updates and I spent so much time while reading your code,  there was so much stuff going in your application so I created an example application that just implements OTA progress. I tried to keep the code clean and simple and wrote some inline comments to help the reader. I am leaving a link to the example applications here so if there are any other people trying to implement OTA, I think it will be easier to understand what is going on.
Thanks!
https://github.com/sensemore/slilabs-ota

Thanks a lot for this.


api: Add go-http-metrics to collect API metrics

Motivation
By integrating go-http-metrics, we can gain detailed insights into the performance and behavior of our APIs.

Why do we need a dedicated dependency for this? We already use github.com/grpc-ecosystem/go-grpc-middleware which has a metrics provider: https://github.com/grpc-ecosystem/go-grpc-middleware/tree/main/providers/prometheus
Wouldn't this work too?

This middleware does not show metrics for individual endpoints, nor does it indicate whether the response is a success or failure.

bors merge

bors merge

bors merge

bors merge


feat: redact secrets in debug logging

This PR improves the debug logging logic in the core,
so that secrets are not shown in the debug logs anymore.
FYI: this PR targets the debug-logging feature branch.

Update: After experimenting with your feature branch in my local sandbox, I opened a PR with a handful of changes (most of which we discussed on slack) that fix the issues that I ran into.
PR: https://github.com/IBM/python-sdk-core/pull/210


Don't reference the old post.sh script in docs, use the bin/solr post command.

Ref guide fix, not planning JIRA.

Description
Please provide a short description of the changes you're making with this pull request.
Solution
Please provide a short description of the approach taken to implement your solution.
Tests
Please describe the tests you've developed or run to confirm this patch implements the feature or solves the problem.
Checklist
Please review the following and check all that apply:

[ ] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[ ] I have created a Jira issue and added the issue ID to my pull request title.
[ ] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended, not available for branches on forks living under an organisation)
[ ] I have developed this patch against the main branch.
[ ] I have run ./gradlew check.
[ ] I have added tests for my changes.
[ ] I have added documentation for the Reference Guide

This change looks awkward, but I think it's right... @HoustonPutman ??


Required fields not working in version 0.5.0 on Umbraco version 7.5.9 assembly: 1.0.6247.17495

Hello,
We are having an issue with your plugin in version 0.5.0 on Umbraco version 7.5.9 assembly: 1.0.6247.17495.
The problem we are having is with required fields: a user is are allowed to submit empty fields for parameters that are required in the DocType.
We are unable for various reasons to upgrade to a higher version of Umbraco, and the highest version of your plugin we can install on this instance is 0.5.0.
Any assistance would be helpful and appreciated.

Closing this as v0 is unsupported


v6.ifconfig.co

Chain issues 	Incorrect order, Extra certs, Contains anchor
Common names 	tar.io   MISMATCH
Alternative names 	atbapi.tar.io git.tar.io tar.io
Trusted 	No   NOT TRUSTED

SSLlabs SSL Report: v6.ifconfig.co (2001:16d8:ee03:0:0:0:cafe:d00d)
$ curl -vvk https://v6.ifconfig.co
* STATE: INIT => CONNECT handle 0x600057790; line 1402 (connection #-5000)
* Rebuilt URL to: https://v6.ifconfig.co/
* Added connection 0. The cache now contains 1 members
*   Trying 2001:16d8:ee03::cafe:d00d...
* STATE: CONNECT => WAITCONNECT handle 0x600057790; line 1455 (connection #0)
* Connected to v6.ifconfig.co (2001:16d8:ee03::cafe:d00d) port 443 (#0)
* STATE: WAITCONNECT => SENDPROTOCONNECT handle 0x600057790; line 1562 (connection #0)
* ALPN, offering h2
* ALPN, offering http/1.1

   "status": 404,
   "message": "route not found"
 * STATE: PERFORM => DONE handle 0x600057790; line 1955 (connection #0)
 * multi_done
 * Connection #0 to host v6.ifconfig.co left intact

Yes? It looks like somebody was careless with HSTS?

https://v6.ifconfig.co/

Unable to communicate securely with peer: requested domain name does not match the server’s certificate.
HTTP Strict Transport Security: true
HTTP Public Key Pinning: false

v6.ifconfig.co uses an invalid security certificate.
The certificate is only valid for the following names: atbapi.tar.io, git.tar.io, tar.io
Error code: SSL_ERROR_BAD_CERT_DOMAIN

@k0nsl
$ curl -sik https://v6.ifconfig.co
HTTP/1.1 404 Not Found
Server: nginx
Date: Thu, 14 Jul 2016 20:11:32 GMT
Content-Type: application/json
Content-Length: 51
Connection: keep-alive
Vary: Accept-Encoding
Access-Control-Allow-Methods: GET
Access-Control-Allow-Origin: *

{
  "status": 404,
  "message": "route not found"
}

HTTPS is not supported for v6.ifconfig.co. Please see #15 and #18.

But  HSTS Preloading	Chrome  Edge  Firefox  IE    Not in: Tor
of SSLlabs SSL Report in 2001:16d8:ee03::cafe:d00d aka ifconfig.co
create this torouble
@k0nsl assertion partly true
enough to force a single request http to https routes inside the browser

@gema-arta Yes, HSTS was mistakenly turned on in the past, and it is on for ifconfig.co, but not for v6.ifconfig.co.

I found one problem. The vhost for ifconfig.co was incorrectly adding includeSubdomains in the HSTS header, causing HSTS to be enabled for v6.ifconfig.co as well. This has now been fixed.
Before:
`Strict-Transport-Security: max-age=31536000; includeSubdomains; preload``
After:
Strict-Transport-Security: max-age=31536000; preload


requesting continuous integration badge for the confidence use of Viz.js

Can you please use any of the free cloud-based continuous integration with Viz.js?
Including the badge will let us know the test cases that work with your build version.
Both Emscripten and Graphviz are evolving and I have seen when tools get updated they tend to break some test cases after transpiling using Emscripten.

Addressed in 1b2d0ba32e163941fdd70302dea9f60e016f21b6 and 379e63d5e255762d5d1b3dfd787e057ac594f5f8


Correct sysv init script to check if sockets are free. This solve pro…

…blem during restart when old process stop but sockets are still in use and then new one can't start.
I found that consul agent after stop sometimes is leaving open connection which can last for me up to 45 seconds (this is the highest value which I found during tests). This fix is solving the problem by adding the loop which check if sockets are free and after that is finishing stop function in init script. So when we try to start again we are sure that process will not crash.

This is a pretty complicated way to solve this.
Can you solve this with less complexity by simply waiting for the pid to die?

The problem is that pid is dead but socket is still opened without process
assign to it. So init script thinks that everything is ok.
16 lip 2015 16:42 "Kyle Anderson"<EMAIL_ADDRESS>napisał(a):

This is a pretty complicated way to solve this.
Can you solve this with less complexity by simply waiting for the pid to
die?
—
Reply to this email directly or view it on GitHub
https://github.com/solarkennedy/puppet-consul/pull/155#issuecomment-121977571
.

@bartlomiejsawicki Hey, trying to understand this problem a little better, can you provide some more information?

In sysctl.conf can you put net.ipv4.tcp_tw_reuse = 1 (don't forget to run sysctl -p) and then try restarting via the init script a few times?
What is your TCP TIME-WAIT timeout?
You mentioned this happens sometimes, out of 10 restarts how many take more than a few seconds?
Does it happen during certain config changes? main config (config.json) vs service/check/watch changes? or just randomly?
Can you try replicating this issue with the agent directly, something like this (ensure you pass the same options as the init script):

# /usr/local/bin/consul agent -pid-file /var/run/consul/consul.pid -config-dir <condif_dir> <extra options>
^C
# /usr/local/bin/consul agent -pid-file /var/run/consul/consul.pid -config-dir <condif_dir> <extra options>
^C
# /usr/local/bin/consul agent -pid-file /var/run/consul/consul.pid -config-dir <condif_dir> <extra options>
^C
...

Some jargon on all this:
http://vincent.bernat.im/en/blog/2014-tcp-time-wait-state-linux.html
http://www.lognormal.com/blog/2012/09/27/linux-tcpip-tuning/
https://rtcamp.com/tutorials/linux/sysctl-conf/

I think we can close it, sorry for answering so late but new version of your branch was solving the problem so this change is not needed.


Avatar

I'd be happy to work on the Avatar component but would need some visual guidance. Is there anything to reference?

Would need canvas to reduce bit depth tho, or maybe css custom filter

@tyler-reitz @goodmind I think something this simple would be great:

Avatar component is frequently used in AppBars as a user navigation or in chat apps so we could do something similar to material-ui avatar. Also it would be nice to implement the letter avatar too (that's great as a placeholder when user didn't set any profile pic yet) 😊
Maybe we could use canvas like @goodmind mentioned to apply some filters or slightly pixelate the circle edges but that's not essential.

@arturbien but this isn't real app :P

@goodmind yes it's a design. Just like any UI design you get from designers to convert it into something real. 😊

We are not attempting to exactly RECREATE Windows95 components. We are going to make them usable in the modern mobile / web apps. Thus, any help from UI / UX designers would be EXTREMELY appreciated. The challenge is to come up with new component designs / layouts that are broadly used in modern UIs, that weren't present back in 95.

@arturbien thanks for the notes. That pixelated circle looks :ok_hand: but I'm not sure there's any sane CSS to pull it off with. Maybe the recessed border from @goodmind's ex. would work instead?

@tyler-reitz I looked this up the other day and it should definitely be possible with a canvas.

@tyler-reitz how would this border look on the circle?

@goodmind probably not as good as the example provided by @arturbien. @maracuja-juice says the pixelated border effect is achievable with canvas anyway, so no need to fix what ain't broke.

#30 For now I added simple Avatar component with 'noBorder' and 'square' prop. What do you think?

#30


MacOS workflow is failing

For some reason the workflow started failing when running cargo tarpaulin on MacOS.

Looks like deleting the cached executable and letting it recompile worked :shrug:


New Component: TextField File Input

It would be nice to have a text input field with a button for file selection / input. Core features:

[x] file is supplied as StringProperty
[x] filepath is shown in text field
[x] Button to select a file (normal javafx file open view)
[x] valid property to see if the file exists

@malafeev i think this works, how can i upload it?

I released it.
Should be available in maven central in a couple of hours.


add java image to crossdock tests

Coverage increased (+0.1%) to 85.564% when pulling 387a35e199e462caa1869949ec093861a896361d on java-crossdock-tests into 645f9ef60ef115d04c9260cf12b558754c6f3c2c on master.

LGTM!


Question on vertices placement in poly meshes generated by Recast

Assuming I'm using cs=0.3 and ch=0.3 for building poly meshes - would it be valid to assume (for each poly mesh separately) that there will be at most 1 generated vertex per voxel in voxels grid with 0.3x0.3x0.3 voxels?
I'm asking because I've been assuming that and in practice, it worked... until recently when I've found that for complex geometry it's not always the case, see e.g. this situation:

The white grid reflects the cs of Recast. Teal lines are the parts of mesh generated by Recast. In all cases except one, there is at most 1 vertex per cell.
@memononen or anyone?

The white grid does not seem to match the values you pass to recast. All the vertices should have same relation to the white grid (i.e. at center or at corners, depends how you aligned the grid).

@memononen okay, so you say my assumption is right and the only thing I did wrong is the offset of the white grid?

Recast puts the vertices at the corners of the voxels. I'm not sure how to translate to your assumption. Anyways, your grid does not look like it matches he cellsize that is passed to Recast. It looks like like the white grid size is 1.0, if the Recast cell size is 0.3.

@memononen thanks, I think it answers my question :) just one more thing - is it guaranteed that there will be at most 1 vertex per corner?

If the corner is at tile border, then there will be a vertex on each tile. Also if you have multiple levels in your navmesh, then looking from top down, there can be multiple vertices in the same place. Otherwise there should be just one per voxel corner.


Make it possible to configure a cron task so that it can have one concurrent execution

Some tasks might need to have a guarantee that just one instance is executing against a given repository, or against all repositories. We need to implement this sort of limitation.
For more details on cron tasks, check here.

seems that org.carlspring.strongbox.cron.jobs.OnePerRepositoryJavaCronJob#beforeScheduleCallback not prevent from parallel task execution but just don't allow to create same(identical) Cron Jobs for repository.
Within this issue we should guarantee that same Job will not produce parallel executed tasks.

@carlspring I will fix it
The anticipated solution is that task will be canceled if same already triggered and not finished.

Okay, @sbespalov !
Thanks for looking into this!

also we probably need a follow up task to have configurable exclusive repository/storage access for CronJobs.

805


[ UX  / DASHBOARD ] Display message in empty list of references

In dashboard, when the user hasn't added any reference yet, the area is just empty.
And I think it would be nice to have a little tutorial video to show how to do it.
We should display a message to encourage the user to send their first reference:
"Vous n'avez pas encore contribué en nous envoyant une référence !
Voir le tutoriel"

I have only displayed a little message for when there's no contribution. I created a ticket for it. So to be tested: only the display of message if the contributions are empty


debugging mode active

the room numbers have been turned on again

Fixed. This time I revised the logic to avoid having the deployment script get confused in the future.


Normalize storage_cost and storage_rebate in gas summary

Right now we include both storage_cost and storage_rebate in the gas summary. In many cases they can be identical when we mutate objects. This can be confusing.
We could normalize it by making one of them 0. We could do so either at per-object level, or the transaction level.

https://github.com/MystenLabs/sui/pull/8101
conversation will follow there

It seems this feature can get problematic.
Particularly with programmable transactions, but even with regular ones with different gas charging, a user can run out of gas if normalized values are used. Simple example:
programmable transaction
command 1- execution cost: 100 storage cost: 30 rebate: 20
command 2- execution cost: 100 storage cost: 20 rebate: 20
command 3- execution cost: 100 storage cost: 10 rebate: 40
if normalized, the required cost would be "execution cost: 300 storage cost: 0 rebate: 20"
non normalized (current) cost would be "execution cost: 300 storage cost: 60 rebate: 80"
Gas to pay - in client code (wallet and more) - is computed as execution_cost + storage_cost so in the normalized case users would have to provide 300 while they would have to provide 360 in the current model.
In the normalized case the gas would not be enough to run all the "commands" and the transaction will fail, which is obviously problematic

it seems storage cost is all computed at the end so the match above is incorrect and we could do this.
Keeping it open

@dariorussi @lxfind - is this still relevant? If not, let's close this.


Use the proper stop method to use the scanner methods multiple times

Previously, it was impossible to call the scanner.start method multiple times because the scanner was never stopped properly. All the Tessels that were originally found would still be considered found so no new Tessels would be returned. I also changed the default type of the scanners to null instead of undefined because that seemed more idiomatic.

looks good!


Simplify calculation for Sweden + Finland

Slightly simpler/faster/more elegant calculation of All Saints Day and Midsummer Day in Finland and Sweden.

Thanks for the PR! Looking good; better than my initial code, haha.


🐛 FIX: Skip copy id from entity to model on create action

fix "ER_DUP_ENTRY: Duplicate entry '38517658' for key 'PRIMARY'" error


Set locale variables automatically for the integrated terminal

VSCode Version: 1.2.0
OS Version: OSX 10.11.5

Steps to Reproduce:

I use ZSH for shell.
Open Integrated Terminal and run locale.
Locale is not set properly. Here is my result:

LANG=
LC_COLLATE="C"
LC_CTYPE="C"
LC_MESSAGES="C"
LC_MONETARY="C"
LC_NUMERIC="C"
LC_TIME="C"
LC_ALL=

Expected:
An optional similar to the default Terminal app in OSX. It makes sense to have it ON by default.
Option from Terminal settings:

Workaround:
Add the following to your profile (~/.zhrc for ZSH):
export LC_ALL=en_US.UTF-8
export LANG=en_US.UTF-8

This might be OS X-specific, seems fine in Linux (vscode/bash left, gnome-terminal/bash right)

If anyone has insight on how locale works on Mac it would be appreciated, PRs welcome.

VSCode Version: 1.4.0 Insiders (commit 217e0b5
OS Version: OS X 10.11.5（15F34）
Shell: /bin/bash,
LANG variable is not explicitly set. It is automatically set by the system.
following is a result of running the locale command on the OS X Terminal (not vscode terminal).
Kazuyuki-no-MacBook-Pro:~ satokaz$ locale
LANG="ja_JP.UTF-8"
LC_COLLATE="ja_JP.UTF-8"
LC_CTYPE="ja_JP.UTF-8"
LC_MESSAGES="ja_JP.UTF-8"
LC_MONETARY="ja_JP.UTF-8"
LC_NUMERIC="ja_JP.UTF-8"
LC_TIME="ja_JP.UTF-8"
LC_ALL=

Please tell me the behavior of expected "terminal.integrated.setLocaleVariables" setting.
In the Japanese locale, it will be set as follows.
"terminal.integrated.setLocaleVariables":true (default) :
bash-3.2$ locale
LANG="ja.UTF-8"
LC_COLLATE="C"
LC_CTYPE="C"
LC_MESSAGES="C"
LC_MONETARY="C"
LC_NUMERIC="C"
LC_TIME="C"
LC_ALL=

"Ja.UTF-8" is not correct.
"Ja_JP.UTF-8" is correct.
"terminal.integrated.setLocaleVariables": true

"terminal.integrated.setLocaleVariables": false
This is the value you expected Looks.

It looks like when the "terminal.integrated.setLocaleVariables":false is set to the correct value has been set.

Ping @chrmarti, sometimes vscode's locale is only the language, not dialect eg. "ja". This is likely why it's set to ja, not ja_JP. As for why it's working on this version of OS X, I have no idea?

We use electron's app.getLocale() to read the locale, which limits it's values to those electron has translations for, that includes 'ja', but not 'ja_JP': http://electron.atom.io/docs/api/app-locales/
We could use some other means to get at the full locale identifier (like https://www.npmjs.com/package/os-locale).

@dbaeumer, would it make sense to make platform.locale be the full OS's locale rather than just what electron supports? Having that will make sense for other (than shells) external processes that support more locales than electron too.

Thank you for the details.
It is considered to be difficult to derive the OS loclae from Electron's app.getLocale().
short locale is also not supported.
In addition, can not care of the LC_* environment variables.
It is desirable to be automatic in the locale setting, but this implementation is thought that only lead to confusion.
Currently, That prompt the user to set the export LANG="" environment variable to the configuration file of the shell (.zprofile, .zshrc, .bash_profile, .bashrc ...) to use, I think it is good to describe to the FAQ.
Microsoft/vscode project is great.
Thanks,
Kazuyuki

This should be fixed in 1.10

@chrmarti I decided against this since the npm modules I looked at when implementing this all spawn processes and this is on the critical start up path and we can't load any code without knowing which locale to use. IMO if someone really needs the real OS locale then they can use a corresponding npm module.


'copy:bowerjs' errors - stratos won't build as app

Deployment type

[X] Cloud Foundry Application (cf push)
[ ] Kubernetes, using a helm chart
[ ] Docker, using docker compose
[ ] Docker, single container deploying all components
[ ] Other (please provide below)

Expected behaviour
console is deployed
Actual behaviour
/tmp/tmp.pmmHSUMuqy/src/golang.org/x/sys/windows/ztypes_windows.go exists, skipping...
/tmp/tmp.pmmHSUMuqy/src/golang.org/x/sys/windows/ztypes_windows_386.go exists, skipping...
/tmp/tmp.pmmHSUMuqy/src/golang.org/x/sys/windows/ztypes_windows_amd64.go exists, skipping...
[03:52:05] Finished 'dedup-vendor' after 2.64 s

<EMAIL_ADDRESS>build /tmp/app
gulp clean:dist && gulp
[03:52:07] Using gulpfile /tmp/app/gulpfile.js
[03:52:07] Starting 'clean'...
[03:52:07] Finished 'clean' after 20 ms
[03:52:07] Starting 'clean:dist'...
[03:52:07] Finished 'clean:dist' after 17 μs
[03:52:08] Using gulpfile /tmp/app/gulpfile.js
[03:52:08] Starting 'build'...
[03:52:08] Starting 'clean'...
[03:52:08] Finished 'clean' after 21 ms
[03:52:08] Starting 'prepare-frontend'...
[03:52:09] Finished 'prepare-frontend' after 1.14 s
[03:52:09] Starting 'copy:configjs'...
[03:52:09] Starting 'copy:bowerjs'...
[03:52:18] 'copy:configjs' errored after 9.08 s
[03:52:18] Error: Unknown system error -122: Unknown system error -122, mkdir '/tmp/app/dist'
at Error (native)
[03:52:18] 'build' errored after 10 s
[03:52:18] Error in plugin 'run-sequence(copy:configjs)'
Message:
Unknown system error -122: Unknown system error -122, mkdir '/tmp/app/dist'
Details:
errno: -122
code: Unknown system error -122
syscall: mkdir
path: /tmp/app/dist
Stack:
Error: Unknown system error -122: Unknown system error -122, mkdir '/tmp/app/dist'
at Error (native)
[03:52:23] 'copy:bowerjs' errored after 14 s
[03:52:23] Error: Unknown system error -122: Unknown system error -122, mkdir '/tmp/app/dist'
at Error (native)
npm ERR! Linux 4.4.0-1027-gke
npm ERR! argv "/tmp/node6.11.3/node-v6.11.3-linux-x64/bin/node" "/tmp/node6.11.3/node-v6.11.3-linux-x64/bin/npm" "run" "build"
npm ERR! node v6.11.3
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build: gulp clean:dist && gulp
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script 'gulp clean:dist && gulp'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the stratos-ui package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     gulp clean:dist && gulp
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs stratos-ui
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls stratos-ui
npm ERR! There is likely additional logging output above.
npm ERR! Linux 4.4.0-1027-gke
npm ERR! argv "/tmp/node6.11.3/node-v6.11.3-linux-x64/bin/node" "/tmp/node6.11.3/node-v6.11.3-linux-x64/bin/npm" "run" "build"
npm ERR! node v6.11.3
npm ERR! npm  v3.10.10
npm ERR! path npm-debug.log.341517281
npm ERR! code Unknown system error -122
npm ERR! errno -122
npm ERR! syscall open
npm ERR! Unknown system error -122: Unknown system error -122, open 'npm-debug.log.341517281'
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     https://github.com/npm/npm/issues
npm ERR! Please include the following file with any support request:
npm ERR!     /tmp/app/npm-debug.log
Failed to compile droplet
Exit status 223
Staging failed: Exited with status 223
Destroying container
Successfully destroyed container

FAILED
Error restarting application: BuildpackCompileFailed
Steps to reproduce the behavior
git clone https://github.com/SUSE/stratos-ui.git
cf push
Log output covering before error and any error statements
Insert log here

@abonillasuse
I'm unable to reproduce this issue. I've tested against the latest SCF from https://github.com/suse/scf and using the latest released helm chart version 2.2.0-beta.
The issue appears to be a lower level issue in your SCF deployment. The error indicates that the build tried to create a folder in on the file system and that failed.
I suspect your environment is unable to compile buildpacks. To test this, can you push a simple app that uses a buildpack with a URL.
For instance push a go app that uses the buildpack https://github.com/cloudfoundry/go-buildpack.
Underlying exception was:
Message:
Unknown system error -122: Unknown system error -122, mkdir '/tmp/app/dist'
Details:
errno: -122
code: Unknown system error -122
syscall: mkdir
path: /tmp/app/dist
Stack:
Error: Unknown system error -122: Unknown system error -122, mkdir '/tmp/app/dist'
at Error (native)

If you are able to reproduce this issue on different environments, please provide us access to one such environment for further investigation and/or talk to the SCF team for them to investigate this.


Restructure project structure

Project structure is more idiomatic and representative of a go project
now, all commands can now be found at cmd/
This is the first step towards simplifying/removing the Makefile

Looks good, merging!


Double underscores break dynamic wildcards parsing

Snakemake version
5.13.0+14.g1804ad75
Describe the bug
Placeholders separated by double underscores (e.g., data.{sample}__{id}.csv) are not parsed correctly in case of dynamic workflows.
Logs
Building DAG of jobs...
Provided cores: 1 (use --cores to define parallelism)
Rules claiming more threads will be scaled down.
Job counts:
        count   jobs
        1       all
        1       parse
        2

[Mon Apr  6 15:45:16 2020]
rule parse:
    input: data_list.csv
    output: data.{*}__{*}.csv (dynamic)
    jobid: 1
    wildcards: sample=__snakemake_dynamic____, id=snakemake_dynamic__

Subsequent jobs will be added dynamically depending on the output of this job
Waiting at most 5 seconds for missing files.
MissingOutputException in line 5 of C:\Dev\test_snakemake\Snakefile:
Job completed successfully, but some output files are missing. Missing files after 5 seconds:
data.{sample}__{id}.csv
This might be due to filesystem latency. If that is the case, consider to increase the wait time with --latency-wait.
Shutting down, this might take some time.
Exiting because a job execution failed. Look above for error message
Complete log: C:\Dev\test_snakemake\.snakemake\log\2020-04-06T154516.037995.snakemake.log

Minimal example

Create a data_list.csv containing:

A
B
C

Create a Snakefile with:

rule all:
    input:
        dynamic('data.{sample}__{id}.csv')

rule parse:
    input:
        'data_list.csv'
    output:
        dynamic('data.{sample}__{id}.csv')
    run:
        with open(str(input)) as f:
            samples = [line.rstrip('\n') for line in f]

        for id, sample in enumerate(samples):
            f = open(f"data.{sample}__{id}.csv", "w")
            f.close()

Additional context
Tested on a conda environment on a Window 10 machine. Snakemake was pip-installed from source.
environment.yml specification:
name: snakemake
channels:
  - defaults
dependencies:
  - python
  - pip
  - pip:
    - git+https://github.com/snakemake/snakemake.git#egg=snakemake

Hi Jack,
I think this is unlikely to be fixed as dynamic rules are deprecated in favour of the more robust checkpoint feature, and will be removed soon.
Looking at the example (and I appreciate it's a very simplified one), if you actually have the data_list.csv file available before the workflow starts then you don't need dynamic outputs at all. You can enumerate the list of samples into a global variable before defining any rules. So something like:
data_list = [l.strip() for l in open('data_list.csv')]

rule all:
   input: [ "data.{}__{}.csv".format(l, i) for i, l in enumerate(data_list) ]

rule report_on_sample:
    output: 'data.{sample}__{id}.csv'
    shell:
        'touch {output}'

Now if the {id} here is actually something that needs to be calculated within the job, I'd say that in the Snakemake way of working it doesn't belong as being part of the filename (I've tried to do this myself and it just never turned out to be a smart idea). But if you really need that to happen then a checkpoint rule will allow it.
Hope that is helpful.
TIM


Workflow creation

We recently converged on a new (stateful) protocol for live workflows: https://github.com/scipp/beamlime/blob/51853589b5510c70ac52527fddb45a5c1212e88b/src/beamlime/workflow_protocols.py#L22-L31
This works, but while working on scipp/esssans#179 it became clear that something is missing: We will have to configure the underlying (non-streaming) workflow with parameters, typically given by a user, before fully creating and running the live workflow. In the linked PR this is circumvented by hard-coding for a specific run, but this is only possible because we a testing with and event stream generated from a fixed file.
I think we will want to make the base workflow (a sciline.Pipeline) somehow injectable into the live workflow, e.g., as in make_sample_run_workflow from scipp/esssans#179. As the users will need to provide parameters we should design this in a way that allows for using our existing workflow widgets module for this. The building blocks we need are:

The live workflow.
The base workflow.
Parameters for the live workflow.
Parameters for the base workflow.

There is no 1:1 mapping between these. For example, we may have:

Live workflow chosen from (transmission-run, sample-run).
Base workflow chosen from (loki, skadi)

I don't know yet if this will be freely composable or if we need to define a per-workflow list (for example, a live workflow could expose a property that defines all supported base workflows).
It is probably easiest to sit down together and just play with a few options, to see what works, then update the live workflow protocol accordingly.

I think scipp/sciline#92 could be valuable for this: We could use widgets to setup a pipeline, save the pipeline with params, the run Beamlime which can restore the pipeline. This would decouple the running of Beamlime from the widgets (which currently run only in a Jupyter Notebook).


Bump org-mgt service version

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/5484346946

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/5484346946
Status: success


Allow Users with exactly 3 or exactly 16 characters to be found.

Currently, if a user has a username 3 characters long or exactly 16 they won't pass the length check and will return not found. This adds a greater than and less than comparison operator to prevent this.

We we discussed on discord, it should be just a normal else, because accounts with less than 3 character displayNames exist (created before the restriction was added)
{
  "id": "b3d1cf3822814b75af8b069628065cfd",
  "displayName": "A"
}


Successful  delete organization gives 204 response code.

Describe the issue:
Successful  delete organization gives 204 response code.
How to reproduce:

Setup the organization management feature https://docs.google.com/document/d/1MWf_rzJGuYObMtlLIyVOBHa8qpKIZ89ZJoPSWuHsaxQ/edit?usp=sharing
Create oraganization under root organization
Delete organization

Request:
curl --location --request DELETE 'https://localhost:9443/o/10084a8d-113f-4211-a0d5-efe36b082211/api/server/v1/organizations/ad16484e-05e8-472c-8a0a-af490287237e' \
--header 'Authorization: Basic YWRtaW46YWRtaW4=' \
--header 'Cookie: JSESSIONID=1D0B039F53D753052520FB201B7839D1; requestedURI=../../815abb2b-9507-434d-a928-90606dd2bba2/api/server/v1/identity-providers'

Response

Expected behavior:
Delete the oraganization and response code should be 202
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: https://wso2.org/jenkins/view/products/job/products/job/product-is/org.wso2.is$wso2is/4139/
OS: mac
Database: H2
Userstore: JDBC

Optional Fields
Related issues:

Suggested labels:

Returning 204 HTTP status code for the successful organization deletion request is the expected response.
HTTP status code 202 - Accepted is used if the server accepts the request but not deleted the resource until some workflow completion.

@aaujayasena Shall we close this issue since there is nothing to fix? Refer https://github.com/wso2/product-is/issues/14174#issuecomment-1179770656


Raise the profile of the SDK feeds to encourage upgrading

This is WIP. Only did Go so far so we can just review that and then I'd do the same for the rest of the SDKs. Note that I'd be normalizing the "Links" section of the respective SDKs as part of this change.

LGTM. @ChristieQ has been contacted for input.

@ChristieQ when you get a chance, can you check this out?

Looks good to me! Thanks :)

@everett-toews do you want me to merge this first or wait until you're done with the rest of the SDKs?

Wait please

@rdodev Okay. The rest of the SDKs are done.

Cool beans! Let me give it a last check and merge.


Vertex API Support

What do you think about supporting Claude via Vertex?
https://docs.anthropic.com/en/api/claude-on-vertex-ai
I'm happy to take a swing at it.

Welcome to submit a PR


math::multiply should be able to handle empty matrix or vectors

Description
The math lib function multiply requires that rows() and cols() are positive.  This should be changed to non-negative in order to support matrices or vectors of size zero.
Example
see https://github.com/stan-dev/cmdstanpy/issues/203
Expected Output
All rules for output sizes work.  That is, matrix multiplication is (M x N) times (N x K), yielding an (M x K) matrix, even if some are zero.  Same for row-vector/matrix mulitply and matrix/vector multiply.
Current Version:
v3.1.0

@syclik  - this issue needs someone who can handle the C++ and knows the math.   I'm timing myself out on this one.   maybe @dpsimpson, @charlesm93  or @bbbales2 could review the list of functions and give us a ruling as on which of m1.rows, m1.cols, m2.rows, m2.cols - can be zero or must be positive?

Copying the testcase from https://github.com/stan-dev/cmdstanpy/issues/203:
data {
  int num_points;
  matrix[num_points, 1] features;
  int labels[num_points];
}
parameters {
  vector[1] x;
}
model {
  x[1] ~ normal(0, 1);
}
generated quantities {
  real test_log_likelihood;
  test_log_likelihood = bernoulli_logit_lpmf(labels | features * x);
}

I've run it in cmdstanr with list(num_points=0, features=matrix(0, 0, 1), labels=integer()) as data. If I change the check_multiplicable in multiply.hpp to a check_size_match("multiply", "m1.cols", m1.cols(), "m2.rows", m2.rows()) the testcase runs:
> fit$summary()
# A tibble: 3 x 10
  variable       mean  median    sd   mad    q5      q95  rhat ess_bulk ess_tail
  <chr>         <dbl>   <dbl> <dbl> <dbl> <dbl>    <dbl> <dbl>    <dbl>    <dbl>
1 x[1]         0.0323  0.0668 0.995 0.987 -1.59  1.63     1.00    1470.    1847.
2 test_log_l…  0       0      0     0      0     0       NA         NA       NA
3 lp__        -0.495  -0.226  0.704 0.308 -1.86 -0.00240  1.00    1736.    2144.

Similar change should be done to the rev/fwd versions. Is this enough or do you need a more general solution?

@mcol  - I think this needs a more general solution.   if this works, that would be great.
yesterday afternoon I edited the issue, but it would seem that my edits got lost  - here's a summary:
check_multiplicable  is used in a lot of places - approx 40 math functions.
check_multiplicable enforces too strict a definition of multiplicability - it requires that both matrices are non-empty.  however, Stan users are using models that sometimes have empty matrices because the language and the I/O allow it.    this bug was reported for multiply, but we need an audit of the math library, because it potentially exists for other operations as well.

Replaced by more general issue #1689.


icu: missing make dep

icu needs clang.
@sdsddsd1

icu builds fine with gcc. Your problem is probably ccache messing things up. Have a look here https://freenode.logbot.info/kisslinux/20200715#c4400989 .  When disabeling ccache fixes your issue we should find a solution to add this to the package.

Oh thanks, that's would be great if there is a solution with ccache.

I took the solution from our firefox package to bypass ccache which works for me.
export CC="${CC:-/usr/bin/cc}"
export CXX="${CXX:-/usr/bin/c++}"

This solution should not break setups where clang/llvm is the default. @dilyn-corner @konimex is this right?

This works.
Of course the easy fix is to not use ccache in the first place.

Thanks

@eudaldgr : can this be closed?

Okey

Okey


Build and Upload Controls fails even on clean project

Steps to Reproduce

Create new preact project.
Select Build and Upload Controls

Error
Project builds locally fine via Restart Webpack, but errors out when trying to Build and Upload Controls showing the following error:

We ran into a problem compiling your controls. You can save the compilation output, which should contain more details about what went wrong.

Compilation output shows no errors:
webpack-output-2018-09-04T20-29-38.457Z.txt
Started having problems with my project, and, after testing with a clean new project, I found the problem was universal.
Editor Details
Version: 1.0.1
Mixer User ID: 53919132

Encrypted Editor State: {"id":"304d88821499bcad","password":"da2Roj8BJWls/lLHez8JqwQ3WeuPbykPfQ66bTAKq94="}

Messing around with network setting corrected the issue. The issue appears to have been with the upload, not with the compilation.

Hi, thanks for reporting the issue! Azure is having some issues at the moment in the region where our control data is stored. You may experience some issues authoring bundles until that clears up.


DateFormatter applied in xyzRow.ts are respected in xyzGrid.ts but not in xyzForm.ts

Hi all,
I know there are plenty of issues/questions regarding Date- and DateTimeFormatter and I have read through many of them but I simply don't get the Display of a date within a form correct:
I have applied a DateFormatter to a field within xyzRow.ts like this:
[DateTimeKind(DateTimeKind.Utc), DateFormatter(DisplayFormat = "yyyy-MM-dd")]
 [DisplayName("Time Stamp Upload"), Column("TimeStamp_Upload"), ReadOnly(true)]

Problem:
This results that in the xyzGrid.ts, the Date is correctly displayed but not correctly displayed within the xyzForm.ts.

I have read in other issues that this depends on the culture Setting in web.config but there is of Course no culture which Displays the date in the desired Format.
Question:
Is there a way to easily apply the dateformatter also to a form field so that the form also Displays the date in the form: yyyy-MM-dd or if not, how can this be accomplished?
Thanks for your time and
with Kind regards,
John

changing culture in web.config might be helpful
    `<globalization culture="ts-ZA" uiCulture="ts-ZA" />`

As explained by Volkan, you can : 1. should change culture in web.config to something that uses dd/MM/yyyy, or set your preferred format in LayoutHead, setting your prefs in ScriptCulture object constructor.'

As explained by Volkan, you can 1. change settings in web.config or setting ScriptCulture in _LayoutHead.cshtml
(@Html.Raw(Serenity.JSON.Stringify(new ScriptCulture() {DateFormat = "yyyy-MM-dd" }))
this gives results expected. Other Culture settings can be changed

@kilroyFR ,
thanks for your feedback - this solves the problem described in my issue. @dfaruque  - thanks also for your feedback.
With kind regards,
John


Fix issue that tomcat plugin cannot capture exception

Coverage increased (+0.01%) to 17.187% when pulling f76b2a8862bf441e117d93de77d777b61b25d94b on ascrutae:zhangxin/fix/tomcat-error-capture into aac05bce7879d2510c9e980f88645565dbb8937d on wu-sheng:master.

Coverage increased (+0.01%) to 17.187% when pulling 6f6556fa33f2aa89ffa7daf21134da912b9a4595 on ascrutae:zhangxin/fix/tomcat-error-capture into aac05bce7879d2510c9e980f88645565dbb8937d on wu-sheng:master.

Coverage increased (+0.01%) to 17.187% when pulling fa42cd47b5989e5dc3b1ebfc0accea6b30098c1f on ascrutae:zhangxin/fix/tomcat-error-capture into aac05bce7879d2510c9e980f88645565dbb8937d on wu-sheng:master.

Ref: #262


Fix iopub callback invocation

Currently, every iopub handler is invoked for every message type in the shim. Here's a fix that also checks the message type.
Things I'm not sure about:

Is iopub.output supposed to be invoked for display_data only? Or is it supposed to handle other messages as well?
I didn't touch onReply since there's only one callback in there at the moment.

I expect output should be called for both display_data and execute_result. Otherwise :+1:.

@parente thanks for this!  I think this is good.  FWIW the shim was modeled after this, https://github.com/jupyter/notebook/blob/master/notebook/static/services/kernels/kernel.js#L891

Added handling of execute_result too.

FWIW the shim was modeled after this

Does it make more sense then to have just one handler function per channel name instead of breaking it down by channel name and select message types?  Or are there clients already relying on the channel+type callbacks?
In our use, we wind up setting every iopub callback to the same function anyway, one that switches on message type.

Does it make more sense then to have just one handler function per channel name instead of breaking it down by channel name and select message types? Or are there clients already relying on the channel+type callbacks?

The manager inside the jupyter/notebook widget manager Javascript relies on the callbacks object behaving how it is described in kernel.js.  It probably makes more sense for this logic to live downstream, just because the shim is supposed to make jupyter-js-services behave exactly like that API.
I think you're the first consumer of this API, so I'm happy to iterate on things upstream (here).  I've merged this, but it'll make sense to revert it if we agree the logic should live downstream.

Currently, every iopub handler is invoked for every message type in the shim. Here's a fix that also checks the message type.

Actually, is it possible that this is a problem with jupyter-js-services ?

@parente gah sorry for the noise, it's totally something I just missed when I was implementing it: https://github.com/jupyter/notebook/blob/master/notebook/static/services/kernels/kernel.js#L806
This is definitely something that should live here, upstream! thanks!


Add disableHistory option (Fixes #44)

This PR implements the disableHistory option for the RAGChat class as requested in issue #44. Here's what this PR includes:

Added disableHistory option to ChatOptions
Updated RAGChat class to handle the new option
Implemented logic to skip history recording when disableHistory is set to true
Added tests to verify the new functionality

This feature allows users to disable history recording for specific chat interactions, providing more flexibility in how chat history is managed.
Testing:

Added unit tests for the new functionality
Manually tested the feature to ensure it works as expected

Note: Some tests may fail due to missing environment variables. I've used the QStash token as suggested, but please let me know if any adjustments are needed.

@enesakar is this what you imaged? Let me know.

cok tesekkurler @eraykeskinmac

devamini bekleriz @eraykeskinmac

@enesakar Çok teşekkürler abi! Devamı için çalışıyorum, projeyi inceledikçe daha fazlasını yapmak istiyorum.

@eraykeskinmac This is good, but I think we should also consider whether we want to fetch history from the store. @enesakar what do you think?

when historyDisabled, the SDK should not call redis (to fetch history)

@eraykeskinmac  Addition to what we have right now, can you disable history fetch when disableHistory is active?

@ogzhanolguncu  Sure i will add that feature to disable history fetch when disableHistory is active

@eraykeskinmac There were lots of conflicts I moved all your commits to https://github.com/upstash/rag-chat/pull/57 here. Your contributions are still there so nothing to worry about.


Replace FQDN with short names when configuring availability group

Fixes #85

Does this impose additional name resolution requirements e.g. need to configure DNS or /etc/hosts with both short names and fqdns?
Does this impose additional x509 cert generation requirements e.g. need to have both short names and fqdn in either the cert subject or Subject Alt Name?

Does this impose additional name resolution requirements e.g. need to configure DNS or /etc/hosts with both short names and fqdns? Does this impose additional x509 cert generation requirements e.g. need to have both short names and fqdn in either the cert subject or Subject Alt Name?

There are requirements for name resolution indeed, thank you for bringing this up. MS docs list their requirements here: https://docs.microsoft.com/en-us/sql/linux/sql-server-linux-availability-group-configure-ha?view=sql-server-ver15#prerequisites
I need to link to this document from the readme.
Configuring both short names and fqdns must be fine, the most important thing is to have short names.
I do not know about additional x5099 cert requirements, I do not think so. The requirement for short names comes from Microsoft, I guess they would mentiont if so.

Yeah, it doesn't say anything about the certs - it uses sql to generate some sort of certificates, so perhaps it hides that.  If we're using that method to generate the certs, then we don't have to mention it.

I discussed this with Amit from Microsoft, and he says that using FQDN for endpoint URLs is recommended, so I will stick to it. Will then check how it goes with Brian. I will keep changes to readme.

Hi @spetrosi, when I ran in to this issue, in my /etc/hosts file on the hosts, I had both FQDN and short names:
<IP_ADDRESS> r8-server1.example.com r8-server1 <IP_ADDRESS> r8-server2.example.com r8-server2 <IP_ADDRESS> r8-server3.example.com r8-server3 <IP_ADDRESS> r8-server4.example.com r8-server4
But if I used the FQDN in the Ansible inventory file, I ran in to the previously discussed issues.

What happens if the shortnames come first e.g.
<IP_ADDRESS> r8-server1 r8-server1.example.com

?

@richm I would need to re-setup my environment to try that out, and I'm not sure how soon I would be able to get that setup again.  The "hosts"(5) man page indicates that the shorter name should be an alias (i.e. not the first entry), and the example in the man page has FQDN listed first, then short name, as I had configured.

@richm I would need to re-setup my environment to try that out, and I'm not sure how soon I would be able to get that setup again. The "hosts"(5) man page has an example with the FQDN listed first, then short name, as I had configured.

If I recall, the order can affect how entries are returned and used by the calling application.

@richm @spetrosi using the latest mssql role from Galaxy, I tried this again.  When using short names in the Ansible inventory file, everything appears to works properly.
When using FQDNs in the Ansible inventory file, with this /etc/hosts file on each node:
<IP_ADDRESS> r8-server1 r8-server1.example.com
<IP_ADDRESS> r8-server2 r8-server2.example.com
<IP_ADDRESS> r8-server3 r8-server3.example.com
<IP_ADDRESS> r8-server4 r8-server4.example.com
The cluster does not come up properly:
[root@r8-server2 ~]# pcs status
Cluster name: ag1
Cluster Summary:

Stack: corosync
Current DC: r8-server2.example.com (version 2.1.2-4.el8_6.2-ada5c3b36e2) - partition with quorum
Last updated: Mon Aug 22 08:18:56 2022
Last change:  Mon Aug 22 08:14:53 2022 by root via cibadmin on r8-server2.example.com
3 nodes configured
4 resource instances configured

Node List:

Online: [ r8-server2.example.com r8-server3.example.com r8-server4.example.com ]

Full List of Resources:

virtualip	(ocf::heartbeat:IPaddr2):	 Started r8-server4.example.com
Clone Set: ag_cluster-clone [ag_cluster] (promotable):

ag_cluster	(ocf::mssql:ag):	 FAILED Master r8-server4.example.com (Monitoring)
ag_cluster	(ocf::mssql:ag):	 FAILED r8-server2.example.com (Monitoring)
ag_cluster	(ocf::mssql:ag):	 FAILED r8-server3.example.com (Monitoring)

Failed Resource Actions:

ag_cluster_monitor_11000 on r8-server4.example.com 'not running' (7): call=105, status='complete', last-rc-change='Mon Aug 22 08:18:55 2022', queued=0ms, exec=5919ms
ag_cluster_monitor_12000 on r8-server2.example.com 'not running' (7): call=86, status='complete', last-rc-change='Mon Aug 22 08:18:55 2022', queued=0ms, exec=5613ms
ag_cluster_monitor_12000 on r8-server3.example.com 'not running' (7): call=79, status='complete', last-rc-change='Mon Aug 22 08:18:55 2022', queued=0ms, exec=5508ms

Daemon Status:
corosync: active/enabled
pacemaker: active/enabled
pcsd: active/enabled
sbd: active/enabled

Hi @briansmith0, I wonder if the issue is in the ha_cluster role trying to guess the names that you want to use.
When running this role against beaker systems, I needed to provide short node names in the inventory. Also, I set up a private network on beaker to have a virtualip reserved, so provided my custom IP addresses too. The resulting inventory looked like this:
all:
  hosts:
    isvqe-01.example.com:
      ansible_user: root
      mssql_ha_replica_type: primary
      ha_cluster:
        node_name: isvqe-01
        pcs_address: isvqe-01
        corosync_addresses:
          - <IP_ADDRESS>
    veritas6.example.com:
      ansible_user: root
      mssql_ha_replica_type: synchronous
      ha_cluster:
        node_name: veritas6
        pcs_address: veritas6
        corosync_addresses:
          - <IP_ADDRESS>
    veritas7.example.com:
      ansible_user: root
      mssql_ha_replica_type: witness
      ha_cluster:
        node_name: veritas7
        pcs_address: veritas7
        corosync_addresses:
          - <IP_ADDRESS>

Galaxy still have the initial HA version, I am planning to publish new version to galaxy tomorrow, For now, I recommend you trying with the repo. You will be able to do this to setup DB and then setup HA:
    - name: Set facts to create a test DB on primary as a pre task
      set_fact:
        mssql_pre_input_sql_file: create_example_db.j2
        __mssql_test_db_name: "{{ mssql_ha_db_name }}"
      when: mssql_ha_replica_type == 'primary'

    - name: Run on all hosts to configure HA cluster
      include_role:
        name: linux-system-roles.mssql

Hi @spetrosi thanks for the info, with this inventory file it seemed to work:
all:
hosts:
r8-server2.example.com:
mssql_ha_replica_type: primary
ha_cluster:
node_name: r8-server2
pcs_address: r8-server2
r8-server3.example.com:
mssql_ha_replica_type: synchronous
ha_cluster:
node_name: r8-server3
pcs_address: r8-server3
r8-server4.example.com:
mssql_ha_replica_type: synchronous
ha_cluster:
node_name: r8-server4
pcs_address: r8-server4
Is this possibly a bug in the ha_cluster role, or is this the intended behavior?  Based on the HA cluster documentation it looks like you should be able to use a FQDN for the node name:  https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/8/html-single/configuring_and_managing_high_availability_clusters/index

ha_cluster by default uses inventory hostnames for node names and pcs addresses. And I suppose in our case we need short addresses.
Did you run your test that succeed with the new role version or with the same version that you run the test that failed?

@spetrosi this testing was done with the galaxy version.  If there is a requirement for the SQL role to use short names, can we document this in the README.md?  Thanks again for your help with this issue!

@briansmith0 of course, it's wip in #92, I'll add it as a hard requirement.


[UI] Setup instructions fail

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):
On Mac with Homebrew, follow the Usage steps.
brew install node watchman yarn
git clone 'https://github.com/rancher/ui'
cd 'ui'
./scripts/update-dependencies
Result:
Build failed with error code: 1
Other details that may be helpful:
10 warnings and 8 errors generated.
make: *** [Release/obj.target/binding/src/binding.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/Users/tiefling/Documents/Dev/ui/node_modules/node-gyp/lib/build.js:262:23)
gyp ERR! stack     at ChildProcess.emit (events.js:200:13)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:272:12)
gyp ERR! System Darwin 18.6.0
gyp ERR! command "/usr/local/Cellar/node/12.4.0/bin/node" "/Users/tiefling/Documents/Dev/ui/node_modules/node-gyp/bin/node-gyp.js" "rebuild" "--verbose" "--libsass_ext=" "--libsass_cflags=" "--libsass_ldflags=" "--libsass_library="
gyp ERR! cwd /Users/tiefling/Documents/Dev/ui/node_modules/node-sass
gyp ERR! node -v v12.4.0
gyp ERR! node-gyp -v v3.8.0
gyp ERR! not ok
Build failed with error code: 1
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
Installation option (single install/HA):

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Machine type (cloud/VM/metal) and specifications (CPU/memory):
Kubernetes version (use kubectl version):

(paste the output here)

Docker version (use docker version):

(paste the output here)

after
npm install -g<EMAIL_ADDRESS>&& rm -rf ~/.node-gyp
worked for me.

Please let us know in a comment if you can still reproduce the issue, and we'll reopen it


New package: OrbitReadout v0.1.0

Registering package: OrbitReadout
Repository: https://github.com/yurier/OrbitReadout.jl
Created by: @yurier
Version: v0.1.0
Commit: 8ccd63a15d47e586d7db5a5ee773b630d657e7c0
Git reference: HEAD
Description: An algorithm to detect differences in orbits associated to classes.

@yurier please see the message above.  Once you fix the automerge issues you can register again the new revision without changing the version number and this pull request will be automatically updated.
[noblock]


Is there a way to combine the coverage.txt file if I run my tests in multiple parallel containers?

Currently, the coverage-path parameter accepts only a single file path. What's the way to get the coverage.txt file in case we're running tests in parallel?

The proper way to use coverage is to generate a proper report using the coverage-summary-path: ./coverage/coverage-summary.json.
In your case, you can also use multiple-files like:
multiple-files: |
      My-Title-1, ./coverage_1/coverage-summary.json
      My-Title-2, ./coverage_2/coverage-summary.json

You can read more in the main readme of the action.
P.S. The usage of | tee ./coverage.txt is mainly for very limited and simple use-cases.


Pre-generate default config during container build

There is no point in generating static configuration in runtime.
Also get rid of hardcoding ignition.conf in favour of using config-dir.
Rename the default configuration so that it always goes first (and thus
can be overridden).

/approve

/approve
huh?

/approve
huh?

Prow is on PTO :). It started to show some flakiness and being re-build right now. Sorry for that.

/approve

/lgtm


problem with fMRI QC

each of my tasks are crashing at BigPlot and measures nodes.
The crash files look like they're picking up the column labels from the output of ComputeDVARS

ahh this is a repeat of #322, I'll close this