Lock updating is very slow

Updating a lockfile can be very slow. A standard pipenv lock easily takes well over a minute for me:
$ time pipenv lock
Locking [dev-packages] dependencies…
Locking [packages] dependencies…
Updated Pipfile.lock (abef76)!

real	1m56.988s
user	0m21.805s
sys	0m2.417s

This means every time I need to install, uninstall or upgrade a package I need to take a 2-minute break to wait for pipenv to finish updating its lockfile. I'm not sure why that is; yarn and npm seem to perform a  similar task but only take seconds, even for projects that have many many more dependencies.

Same here im trying it out right now and installing django is already at 10 minutes and going

seems that if you try and install a couple at a time its slow but if you do one at a time works a bit faster

If you can provide a Pipfile that reproduces the slow behavior it would be helpful -- thanks

Python dependencies require us to fully download and execute the setup files of each package to resolve and compute

Is this still the case when the package has its own Pipfile.lock, or will pipenv use that when possible to determine dependencies?

will pipenv use that when possible to determine dependencies?

No. Pipenv is an application depedency resolution tool. When a dependency is used as a Python package by another application, it is no longer an application. Its dependencies are determined by setup.py (or setup.cfg or whatever you use to upload to PyPI). Loading dependencies from a lock file is a sure route to dependency hell, we will likely never ever do that.

It's still super slow

@iddan Thanks for the reminder, Captain Obvious!

Sorry, as an OSS maintainer I know how sometimes issues can get dismissed because of age. Just wanted to state that even the issue received discussion it’s still relevant

@techalchemy note of --skip-lock above is wonderful. This should be a more accessible or publicized option. Can we set it as a default somewhere?

@techalchemy what if it takes 20 minutes with my brand new mac pro?

This took 38 minutes on my machine to create the lock file. Running Windows 10 and using Python 3.7

Only numpy and pillow were already installed and it took <1 seconds to install numba and 25 minutes to lock it. Does pipenv forcibly compile every lib on lock or how does this work?

FYI persistent skip lock is just waiting on someone to flip the switch for auto_envvar_prefix which is a click setting.  I've been 100% focused on core functionality so I haven't had a chance to look at this yet but I suspect it isn't that difficult

TLDR; Typical pipenv install invocation: Time: 144.82 real        33.68 user         5.77 sys. With --skip-lock: Time:  4.54 real         5.33 user         0.87 sys.
Pandas-datareader install fails on first attempt, possibly cause of lock hanging. Is this an issue anyone else is seeing with other packages?
Using version 2018.11.26
$ pipenv --version
pipenv, version 2018.11.26

Contents of requirements.txt
sklearn-pandas
sklearn
pandas
numpy
matplotlib
statsmodels

Typical pipenv install invocation. Timed execution using time (BSD).
Results: 144.82 real        33.68 user         5.77 sys
$ time pipenv install -r requirements.txt
Requirements file provided! Importing into Pipfile…
Pipfile.lock (0fdb67) out of date, updating to (a65489)…
Locking [dev-packages] dependencies…
Locking [packages] dependencies…
✔ Success!
Updated Pipfile.lock (0fdb67)!
Installing dependencies from Pipfile.lock (0fdb67)…
  🎅   ▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉ 15/15 — 00:00:04
To activate this project's virtualenv, run pipenv shell.
Alternatively, run a command inside the virtualenv with pipenv run.
      144.82 real        33.68 user         5.77 sys

Invoking w\ --skip-lock
Results:  4.54 real         5.33 user         0.87 sys
$ time pipenv install -r requirements.txt --skip-lock
Requirements file provided! Importing into Pipfile…
Installing dependencies from Pipfile…
  🎅   ▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉ 6/6 — 00:00:01
To activate this project's virtualenv, run pipenv shell.

This is pretty consistently happening, essentially pipenv going away around town while "locking" something: why is this issue closed?
Understand the --skip-lock is the way to go, but it's not clear at all why "installing" takes a few seconds, and "locking" takes forever: it would be great  if at least the lock command would emit some progress/update logs: as things stand, it's not even clear if it just got stuck in some sort of while True forever...
I'd at least like to know if it's me doing something wrong, or just a "feature" of pipenv.

In our project pipenv lock is so so so slow. It definitely impacted our normal use. Adding a new package becomes a real pain for us now. Is there anyway we can debug this behavior?

I'm trying to install PyTorch and it took 20 minutes to lock and then it pulls the following error
Installing initially failed dependencies…
[pipenv.exceptions.InstallError]:   File "/usr/local/lib/python3.6/dist-packages/pipenv/core.py", line 1992, in do_install
[pipenv.exceptions.InstallError]:       skip_lock=skip_lock,
[pipenv.exceptions.InstallError]:   File "/usr/local/lib/python3.6/dist-packages/pipenv/core.py", line 1253, in do_init
[pipenv.exceptions.InstallError]:       pypi_mirror=pypi_mirror,
[pipenv.exceptions.InstallError]:   File "/usr/local/lib/python3.6/dist-packages/pipenv/core.py", line 859, in do_install_dependencies
[pipenv.exceptions.InstallError]:       retry_list, procs, failed_deps_queue, requirements_dir, **install_kwargs
[pipenv.exceptions.InstallError]:   File "/usr/local/lib/python3.6/dist-packages/pipenv/core.py", line 763, in batch_install
[pipenv.exceptions.InstallError]:       _cleanup_procs(procs, not blocking, failed_deps_queue, retry=retry)
[pipenv.exceptions.InstallError]:   File "/usr/local/lib/python3.6/dist-packages/pipenv/core.py", line 681, in _cleanup_procs
[pipenv.exceptions.InstallError]:       raise exceptions.InstallError(c.dep.name, extra=err_lines)
[pipenv.exceptions.InstallError]: ['Collecting pytorch==1.0.2 (from -r /tmp/pipenv-pb00kf8t-requirements/pipenv-vae35p2d-requirement.txt (line 1))', '  Using cached https://files.pythonhosted.org/packages/ee/67/f403d4ae6e9cd74b546ee88cccdb29b8415a9c1b3d80aebeb20c9ea91d96/pytorch-1.0.2.tar.gz', 'Building wheels for collected packages: pytorch', '  Building wheel for pytorch (setup.py): started', "  Building wheel for pytorch (setup.py): finished with status 'error'", '  Running setup.py clean for pytorch', 'Failed to build pytorch', 'Installing collected packages: pytorch', '  Running setup.py install for pytorch: started', "    Running setup.py install for pytorch: finished with status 'error'"]
[pipenv.exceptions.InstallError]: ['ERROR: Complete output from command /home/alex/.local/share/virtualenvs/pytorch-umelu-tG/bin/python3 -u -c \'import setuptools, tokenize;__file__=\'"\'"\'/tmp/pip-install-hix3yz6v/pytorch/setup.py\'"\'"\';f=getattr(tokenize, \'"\'"\'open\'"\'"\', open)(__file__);code=f.read().replace(\'"\'"\'\\r\\n\'"\'"\', \'"\'"\'\\n\'"\'"\');f.close();exec(compile(code, __file__, \'"\'"\'exec\'"\'"\'))\' bdist_wheel -d /tmp/pip-wheel-f_h8w1pz --python-tag cp36:', '  ERROR: Traceback (most recent call last):', '    File "<string>", line 1, in <module>', '    File "/tmp/pip-install-hix3yz6v/pytorch/setup.py", line 15, in <module>', '      raise Exception(message)', '  Exception: You tried to install "pytorch". The package named for PyTorch is "torch"', '  ----------------------------------------', '  ERROR: Failed building wheel for pytorch', '    ERROR: Complete output from command /home/alex/.local/share/virtualenvs/pytorch-umelu-tG/bin/python3 -u -c \'import setuptools, tokenize;__file__=\'"\'"\'/tmp/pip-install-hix3yz6v/pytorch/setup.py\'"\'"\';f=getattr(tokenize, \'"\'"\'open\'"\'"\', open)(__file__);code=f.read().replace(\'"\'"\'\\r\\n\'"\'"\', \'"\'"\'\\n\'"\'"\');f.close();exec(compile(code, __file__, \'"\'"\'exec\'"\'"\'))\' install --record /tmp/pip-record-xr7o93_5/install-record.txt --single-version-externally-managed --compile --install-headers /home/alex/.local/share/virtualenvs/pytorch-umelu-tG/include/site/python3.6/pytorch:', '    ERROR: Traceback (most recent call last):', '      File "<string>", line 1, in <module>', '      File "/tmp/pip-install-hix3yz6v/pytorch/setup.py", line 11, in <module>', '        raise Exception(message)', '    Exception: You tried to install "pytorch". The package named for PyTorch is "torch"', '    ----------------------------------------', 'ERROR: Command "/home/alex/.local/share/virtualenvs/pytorch-umelu-tG/bin/python3 -u -c \'import setuptools, tokenize;__file__=\'"\'"\'/tmp/pip-install-hix3yz6v/pytorch/setup.py\'"\'"\';f=getattr(tokenize, \'"\'"\'open\'"\'"\', open)(__file__);code=f.read().replace(\'"\'"\'\\r\\n\'"\'"\', \'"\'"\'\\n\'"\'"\');f.close();exec(compile(code, __file__, \'"\'"\'exec\'"\'"\'))\' install --record /tmp/pip-record-xr7o93_5/install-record.txt --single-version-externally-managed --compile --install-headers /home/alex/.local/share/virtualenvs/pytorch-umelu-tG/include/site/python3.6/pytorch" failed with error code 1 in /tmp/pip-install-hix3yz6v/pytorch/']
ERROR: ERROR: Package installation failed...

installing with pip in the environment work fine! This is really a show stopper. Going back to requirements.txt...

This is the workaround I use for now:
export PIPENV_SKIP_LOCK=true

Then pipenv install foo won't be locking and you can lock manually when you have time by running pipenv lock.

raise Exception(message)', '    Exception: You tried to install "pytorch". The package named for PyTorch is "torch"', '

Pretty sure the last line says all you need:

You tried to install "pytorch". The package named for PyTorch is "torch"

Locking dependencies is important so I don't think "skip lock" is a lasting solution. At the same time, I simply don't buy that "locking dependencies" (whatever that could possibly entail under the hood) is maximally optimized as it is now and functionally requires many minutes or hours to complete. Indeed, my pipenv lock ran for several minutes on a Pipfile that has a laughable 5 dependencies before failing—stack attached at bottom—and during that time used only 10–15% of available CPU and a little sip of memory.
Can we at least put forth some group effort to profile and determine the bottlenecks? I have a feeling there are just some silly low hanging fruit in there just waiting to take this process into reasonable runtime.
pipenv version 2018.11.26
For Pipfile:
[[source]]
url = "https://pypi.org/simple"
verify_ssl = true
name = "pypi"

[packages]

[dev-packages]
keras = "*"
tensorflow = "~=1.13"
setuptools = "*"
wheel = "*"
twine = "*"

[requires]
python_version = ">=3.6"

Pipfile.lock (63b275) out of date, updating to (5e165c)…
Locking [dev-packages] dependencies…
Traceback (most recent call last):
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/vendor/pexpect/expect.py", line 109, in expect_loop
    return self.timeout()
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/vendor/pexpect/expect.py", line 82, in timeout
    raise TIMEOUT(msg)
pexpect.exceptions.TIMEOUT: <pexpect.popen_spawn.PopenSpawn object at 0x105a17210>
searcher: searcher_re:
    0: re.compile('\n')

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/bin/pipenv", line 11, in <module>
    load_entry_point('pipenv==2018.11.26', 'console_scripts', 'pipenv')()
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/vendor/click/core.py", line 764, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/vendor/click/core.py", line 717, in main
    rv = self.invoke(ctx)
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/vendor/click/core.py", line 1137, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/vendor/click/core.py", line 956, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/vendor/click/core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/vendor/click/decorators.py", line 64, in new_func
    return ctx.invoke(f, obj, *args, **kwargs)
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/vendor/click/core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/vendor/click/decorators.py", line 17, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/cli/command.py", line 254, in install
    editable_packages=state.installstate.editables,
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/core.py", line 1992, in do_install
    skip_lock=skip_lock,
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/core.py", line 1221, in do_init
    pypi_mirror=pypi_mirror,
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/core.py", line 1068, in do_lock
    lockfile=lockfile
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/utils.py", line 649, in venv_resolve_deps
    c = resolve(cmd, sp)
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/utils.py", line 517, in resolve
    result = c.expect(u"\n", timeout=environments.PIPENV_INSTALL_TIMEOUT)
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/vendor/delegator.py", line 215, in expect
    self.subprocess.expect(pattern=pattern, timeout=timeout)
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/vendor/pexpect/spawnbase.py", line 341, in expect
    timeout, searchwindowsize, async_)
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/vendor/pexpect/spawnbase.py", line 369, in expect_list
    return exp.expect_loop(timeout)
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/vendor/pexpect/expect.py", line 119, in expect_loop
    return self.timeout(e)
  File "/usr/local/Cellar/pipenv/2018.11.26_2/libexec/lib/python3.7/site-packages/pipenv/vendor/pexpect/expect.py", line 82, in timeout
    raise TIMEOUT(msg)
pexpect.exceptions.TIMEOUT: <pexpect.popen_spawn.PopenSpawn object at 0x105a17210>
searcher: searcher_re:
    0: re.compile('\n')

As an addendum to previous comment, running pip lock separately took a reasonable amount of time, ~15 seconds, after first running pip install --skip-lock. So perhaps the post-install invocation of lock is out of date or otherwise faulty. :)

FYI I found tensorflow is the culprit of the slow/timing out lock, if that helps profile pipenv! (Still consider this a pipenv problem...)

Tensorflow is one of many packages that can cause pipenv to basically become a useless tool. I like the suggestion of group profiling, though. I think that's an excellent idea to start tacking this problem. Just to re-iterate, PEP 566 enabled enumerating dependence (via pypi) without loading the entire source, which may be helpful: https://github.com/pypa/pipenv/issues/1914#issuecomment-457965038

@brandonrobertz from what I see, downloading all packages of the dependencies is where most time is spent. This also has been confirmed before.
How to verify this:

Try creating a new Pipfile and installing scipy (for example) in it with locking enabled
Wait until locking completes. (Takes about 5 minutes on my machine)
remove Pipfile.lock
run pipenv lock - locking now will take very little time (6 seconds on my machine), because all packages are already downloaded and kept in Pipenv cache which is normally in ~/.cache/pipenv

Here's the Dockerfile I used to test this:
FROM python:3.6
ENV WORKDIR=/work/
WORKDIR /work/
RUN python3 -m pip install --upgrade pip
RUN python3 -m pip install pipenv
RUN PIPENV_SKIP_LOCK=true pipenv install scipy
RUN date
RUN pipenv lock
RUN date
RUN rm Pipfile.lock
RUN pipenv lock
RUN date

@shtratos Yeah, that makes sense and others have suggested that in this issue thread. Downloading and parsing dependencies is expensive. Some of those steps seem like they can be eliminated by pulling from the pypi dependency API.
For some libraries this will probably not work, due to poor quality and bad practices (not using setup.py or requirements.txt). But since some of the major offenders seem to be very popular libraries (tensorflow, numpy), implementing this with a fallback to the super slow process might be a good path forward.

Can you point me in the direction of where to find that code? I could take a stab at parallelizing it in a fork.

I think https://github.com/pandas-dev/pandas/ may be a problem? It's a common point with the time for me too.
Although pytest may also be an issue :\

I doi not think so, it works fine on my machine, the problem seems to be more general than that

In my case the problem seems to be pylint. It always hangs on locking when simply running pipenv install pylint see https://github.com/pypa/pipenv/issues/2284#issuecomment-569457752

We are aware and have many issues tracking this topic. See #1785 #1886 #1891 and PR #1896
npm and yarn have the advantage of not having to fully download and execute each prospective package to determine their dependency graph because the dependencies are specified in plaintext. Python dependencies require us to fully download and execute the setup files of each package to resolve and compute. That's just the reality, it's a bit slow. If you can't wait 2 minutes or you feel it's not worth the tradeoff, you can always pass --skip-lock.
Closing to track in the other issues.

3 out of the 4 other issues are now closed and the remaining one hasn't seen any activity since 2018. This issue still persists, so maybe re-opening it would be a good idea?

Python dependencies require us to fully download and execute the setup files of each package to resolve and compute

I don't think that is still true for wheels, which should be the majority of packages now?

I know I at least have to build the wheel for dlib every time, which is horrendous.
The process of resolving dependencies should be cached somewhere on a per-package-version basis, even if it requires another remote lookup on the client each time to get the tree (it shouldn't, you could just cache it locally too after the fact). For example, package repos could easily store resolved dep trees. The additional network hit for the cached resolved dep tree would always be faster than downloading an entire package and computing.

FWIW I've removed pipenv from all my projects (this being the one main reason, there are others).
virtualenv + pip (with requirements.txt) now work pretty well, even for Prod deployments; and in any event, one deploys a fully-formed container these days; after getting really into pipenv, I no longer see the point of it.

Please reopen this issue.
Otherwise pipenv will never be the reference packaging tool

It's still super slow.
I recommend going with Poetry.

Could we signal that this is still a problem by re-opening this issue?

@lyle-nel I don't think re-opening this specific issue would help since the content of it is dated, for example,

Python dependencies require us to fully download and execute the setup files of each package to resolve and compute. That's just the reality, it's a bit slow.

This is no longer true about pipenv as it gets the package hashes from the pypi API.  There is a merged PR connected here that relates back to the issue at hand being resolved.  If the community still feels that locking is very slow (too slow) and that there might be some optimization that can be done that wouldn't compromise the integrity of locking (I am unsure) then at this stage, the path forward is to open a new issue with as much detail as possible, including method for reproducing the issue using the latest version of pipenv, and any suggestions on how to improve it would make the ticket even better.

For what it's worth, it still feels too slow. Is there a different issue where this is tracked?

@truekonrads and others -- I am locking conversation on this issue because the originally issue from 2018 of why it was slow has been resolved.   For all of the people that come here to comment in the modern day that pipenv feels really slow, just know that it is super hard to support you because you are generally providing zero examples.  You have none of the default from the issue report template information, no reproduction case or supporting evidence, no version info, whether you are using a private pypi (because that will be a lot slower since there is no API to collect package hashes like there is for pypi so everything has to be downloaded and built to determine the hash).   If you think that there is a need or realistic chance to optimize pipenv speed then please have at opening a brand new issue report with complete context.  However note that my analysis has not revealed anything with current pipenv version just wasting time just and slow for no reason, so if you do open a new issue report please try to support your claim with as much details and data as possible.


Permission denied to access property "apply"

Hey,
I am getting the following unhandled error reported on Bugsnag-js. What could be causing it?
We are using:
"@bugsnag/js": "^5.0.2",
"angular": "~1.7.2",
Error
/
Permission denied to access property "apply"
Jan 21st, 2019, 06:37:30 UTC
STACKTRACE
Error Permission denied to access property "apply"
../../../../../node_modules/@bugsnag/browser/plugin-inline-script-content/inline-script-content.js:37:26 init/e.onreadystatechange

I'm getting this error, on Firefox 69.0.0 and 70.0.0:
node_modules/@bugsnag/browser/dist/bugsnag.js:2275   init/t.onreadystatechange
if (typeof prev === 'function') prev.apply(this, arguments);

@timscott - can you confirm you're using at least version 6.4.1 which includes a fix for #608?

@tomlongridge - From Gemfile.lock:
bugsnag (6.11.1)

@tomlongridge - Ignore my last comment. That's the server side ruby gem. I'm on 6.4.0. I will upgrade.


Colorbar label when plotting wave energy spectra (m2).

When plotting
ds_energy = ds.efth.spec.to_energy()
p = ds_energy.plot.pcolormesh(ax=wrax_cham, as_period=True, as_log10=False, cmap=cmap, cbar_ax=cbar_ax)
The plot shows the label Energy density $m^2s/deg$ insted of the one given by to_energy(): Energy spectra $m^2$.
I fixed it manually by doing
p.colorbar.set_label('Energy $m^2$')
But doing some debugging found that the culprit is on line 369 of plot.py (_plot2d(plotfunc)) when calling  _freq_or_period_clean_attributes(darray, as_period=as_period) as it erases the previous attributes. I suggest including an if to read if there's a different standard_name already in place.
Regards,

I am trying to reproduce the problem but I get the label given by to_energy() when I do:
import matplotlib.pyplot as plt
from wavespectra.input.swan import read_swan

ds = read_swan("./tests/sample_files/swanfile.spec").isel(lat=0, lon=0, time=0)
ds_energy = ds.efth.spec.to_energy()
p = ds_energy.spec.plot(as_period=True, as_log10=False)
plt.show()

mmm...don't really know. I ran your example (downloaded the swanfile.spec files):
p = ds.spec.plot(as_period=True, as_log10=False)
plt.show()
and get:

the I ran:
p = ds_energy.spec.plot(as_period=True, as_log10=False)
plt.show()
and got:

I did a new pip install wavespectra[extra] a got version 3.8.1 and tried again but got the same. It is not a big deal as I can manually change the label and the values do agree. It must be something on my version. Thanks anyway.

And thank you and all the development team for this very nice library!

Oh sorry, now I understand what happened. pip install wavespectra points to the fork created by rafa-guedes a couple of years ago, that's why we install from the source. We also try to minimize the divergence between our version and that fork but it's hard because we have different development priorities.


formula_installer: check if dependencies have already been fetched.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Given a pkg_a that depends on pkg_b, it is redundant to list pkg_b as a dependency when running the following commands:

brew install pkg_b pkg_a
brew upgrade (when both pkg_a and pkg_b are outdated)

Can you provide some example before/after output illustrating why you think this change is better? Thanks!

Makes sense, thanks again @FnControlOption!


Is there any way to disable lua filter for particular service  or particular route

If you are reporting any crash or any potential security issue, do not
open an issue in this repo. Please report the issue via emailing
<EMAIL_ADDRESS>where the issue will be triaged appropriately.
Title: One line description
Description:

Describe the issue.

[optional Relevant Links:]

Any extra documentation required to understand the issue.

I think this answers your question :-)
https://www.envoyproxy.io/docs/envoy/latest/configuration/http/http_filters/lua_filter#per-route-configuration


Dev mmz

Updated schema
Added pypi-publish workflow
Fixed metadata

Cross-referencing issue: https://github.com/mite-standard/mite_schema/issues/1


Autorest Regen Preview to 3.0.0-alpha.20230420.4 by Dapeng Zhang from refs/pull/3279/merge

Triggered from https://github.com/Azure/autorest.csharp/pull/3279

API change check
API changes are not detected in this pull request.


Parse Server documentation on Stackoverflow

Stackoverflow has recently launched Stackoverflow Documentation. I thought it would be great to create a community documentation for Parse Server.
Please feel free to commit your proposals here:
http://stackoverflow.com/documentation/parse-server/commit

While this is interresting, there is already a documentation repository here: https://github.com/parseplatform/docs and also here https://parseplatform.github.io/

Let's see if anone is interested in this.
I personally consider Stackoverflow is a better place to share my knowledge.

@andrew8712 you are 100% right, this is not the place to get help on implementation questions, nor scaling issues, but for reporting bugs on the code itself! I'd love to see a striving stackoverflow community!

Feel free to use Stack Overflow Docs to share any specific examples related to Parse Server. For broad, general docs you can always send a PR to the docs repo.


is client.write() thread safe?

Hi, influxdb-python is so awesome.
I would like to ask that, given only 1 client object, is client.write() thread safe?
def write(data):
    client.write(data)

if __name__ == "__main__":
    client = InfluxDBClient('localhost', 8086, 'admin', 'admin', 'db')
    thread.start_new_thread( write, (data1,))
    thread.start_new_thread( write, (data2,) )

or I shall create 2 clients:
def write(data, client):
    client.write(data)

if __name__ == "__main__":
    client1 = InfluxDBClient('localhost', 8086, 'admin', 'admin', 'db')
    client2 = InfluxDBClient('localhost', 8086, 'admin', 'admin', 'db')
    thread.start_new_thread( write, (data1, client1))
    thread.start_new_thread( write, (data2, client2) )

Thanks,

The client uses requests Session underneath. https://github.com/requests/requests/issues/1871

Great question with a vague answer.
@tcloaa, does it mean .write is thread safe?


301 Redirect error?

I am calling the following URL from PHP using Curl:
http: //github-awards.com/api/v0/users/
And recently I started getting the following data:
< title > 301 Moved Permanently < /title>
 301 Moved Permanently

  nginx / 1.6.2

Did anything changed recently with the API ? If so, how can I avoid this issue?
Calling the same URL directly from the browser seems to work normally.

The website moved to http://git-awards.com/ , you should update your script.

Thanks for the quick answer! Awesome work here! Keep it up!


charts version upgrade of jenkins-x/exposecontroller-service

change jenkins-x/exposecontroller-service to version 1.0.7

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: garethjevans
If they are not already assigned, you can assign the PR to them by writing /assign @garethjevans in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @enschede. Thanks for your PR.
I'm waiting for a jenkins-x member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/test static
/test prow
/test tekton

/test static

/test prow

/test ng

@enschede: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

ng
4b0aa587e8274debd21d6c54eaacb6975aa70b5b
link
/test ng

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test ng

@enschede: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

ng
4b0aa587e8274debd21d6c54eaacb6975aa70b5b
link
/test ng

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


do not canonicalize hostnames if canonicalize_hostname is false

This is useful when running teuthology in a lab that does not enforce
the same naming conventions as the Ceph community lab.
http://tracker.ceph.com/issues/11844 Fixes: #11844
Signed-off-by: Loic Dachary<EMAIL_ADDRESS>

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1492/
Tests passed for this pull request.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1493/
Tests passed for this pull request.

Why is this necessary? I can see it breaking lots of things.

Nothing works outside of sepia / typica if it's not possible to disable it.

That's the thing: that's just not true. We have an entirely separate internal lab that inspired my writing this. Does lab_domain not help you?

I see what you mean, my bad.


sink 4 more arrows as the third player

What is your question?
more or less it's something like Bonedoogle, add another arrow box for a third player (opponent) in the middle of the song

ironically enough, there IS a Bonedoggle port that uses this, just download the mod and look for the script
https://gamebanana.com/mods/372092

thank


Update README (added credits and useful links), incr version to 2.1.7

Also, I made changes to the PyPI readme renderer for the div align issue. See https://github.com/pypa/readme_renderer/commits?author=akamhy


SPManagedAccount not getting the right value from Test-DSCResource

Hi,
I'm using the SPManagedAccount DSC resource and I have no issue to add an account but when the configuration is running again, the test seems to not return the right value because it is going through the SET again and I'm getting an error because the account is already registered.
I'm currently using the SharePointDSC module with version <IP_ADDRESS>
In SharePoint, if I'm trying the command get-SPManagedAccount with the same value as I'm using in DSC, it is returning me the object.
That is why I think the Test-DSCResource is returning $false while it should be $true.
Thanks
Dave

This was not an issue. The NetBIOS name of our domain is different and SharePoint is adding the managed account with the NetBIOS domain name. When the test was occuring, it was looking with the domain name and not the NetBIOS domain name. I managed to fix it by modifying the username in the credentials to the value that should work.


Nested elements (Bootstrap Panels)

Hello.
I have the following scenario: I'm using bootstrap panels in order to present some React components. In order to get them to work, the css styles and others have to be applied to a child div. What I would like accomplish is pretty much this:
<ReactGridLayout>

<ReactGridLayout>

Is there any way to replicate this scenario?

I just solved this problem. In order to get it to work, all I had to do was actually setting the height property from the inner div, like this:
<ReactGridLayout>

<ReactGridLayout>


Fix MNIST test_parse_images

Fixes #131.
Download and cache MNIST data prior to running tests.
This hopefully ensures stable test runs by preventing “corrupt deflate stream” errors caused by incomplete downloads.
I've tried to rerun the tests 5 times without any issues so far.

@chaseWillden Can you take a look at this also. Just to be two persons reviewing this.


Does it support hours, miniutes and seconds?

Hi,
This components looks very neat for vue, but does it support hours, miniutes and seconds?
Thanks!

Thanks, not support yet.


Fix nodejs examples npm install

npm i in iota-sdk/bindings/nodejs/examples fails with
npm ERR! code EUNSUPPORTEDPROTOCOL
npm ERR! Unsupported URL Type "link:": link:../

The merged PR actually didn't fix it (I think it only worked because something was still cached)
When rebuilding it either fails with yarn or npm, never got it working with both

We decided to use yarn everywhere https://github.com/iotaledger/iota-sdk/pull/1953


Refit integration and tests

📋 What's changed

Integrated Refit.
Added Unit tests.
Refactored code and project structure.

💡 Other proposed changes

Add Github Actions to run tests and automatically push the latest version to Nuget.

I can implement the above suggested changes if you wish.
⚠ Note that you should not merge this PR if you want me to do these proposed changes

These changes result in a slight speed increase:
New = {00:00:04.8158022}
Old = {00:00:05.1037871}

However i mostly made those changes to make the project testable and maintainable (And because i was kinda bored).
If you have any questions @Riku32 feel free to ask. 😄


Issues after v3

After the release of version 3 issues will be blocked, all the discussion will be on the forum

http://forum.metroui.org.ua


Springs don't entirely take deltatime in account accordingly

After having tested with multiple different re-occurring springs, the higher the fps the faster the springs perform. It seems as if deltatime isn't properly being taken in account.
This can be easily tested in an empty place and by switching your fps between 30 and uncapped (or 60) with an fps unlocker.

Which version are you using?

I'm currently using the latest version (to my knowledge) 1.0.1


first working implementation with user text selection preventing swipe

fixes #21
@rubenstolk can you try this?

This change is

Sorry @MeTaNoV, missed this. Will test it now!

Hi @MeTaNoV, doesn't seem to work. When I have a simple <paper-input> within a page, selecting the text in that input field still triggers the swipe...


Increase the verbosity of conda make test

Some tests were failing -- https://ci.int.mosaicml.com/job/scratch/job/Conda Publish/15/. Difficult to debug without verbosity.

Closing as #1011 incorporates this change.


Compilation Issues In VS Solution

Apologies if I'm missing something here, I've never developed mods before although I am very experienced in .Net and VR development so thought I'd try to get involved in this and maybe future mods.
I've cloned the repo but I'm seeing the solution (TwoForksVr.sln) doesn't compile - I'm seeing a lot of errors, so I was hoping to query them as they might be expected.
Incorrectly defined namespaces, eg "namespace TwoForksVr.Settings;"
Shouldn't the namespaces surround the class? I'm getting compilation issues right now because they don't.
Update: This is a C#10 thing! I'm clearly behind the times.
Unresolved Dependencies
It seems my VS is also unable to resolve the SteamVR dependency. Looking at the prebuild steps, it looks like they're copied from the UnityHelper - Is there any info anywhere how the UnityHelper should be built? It looks like it should be opened in Unity but that's as much as I can work out.
The BepInEx.Core also seems to be unresolved as well, and I'm unable to locate the right package on Nuget. Is there anything in particular we need to be doing to resolve this dependency?
If there's a developer guide anywhere or if there's a better place to ask these questions please let me know.

Turned out to be an issue with me using an older version of VS. Updated to 2022 and now everything compiles fine.

Sorry that I haven't added build instructions yet! I'll make sure to add them asap. Glad you managed to get it working.
What version of VS were you trying to use? I would expect this project to work with VS 2019 too, don't wee why it wouldn't. And I guess I need to make sure it specifies the C# version explicitly to avoid problems like these.
The problem with missing DLLs like steamvr os because you need to build the project once for the DLLs to get generated. So I guess you ended up doing that accidentally? Sorry about the confusion.
By the way, may I ask what kind of contribution you want to make? You can talk to me on Discord about it if you want to.

Sorry that I haven't added build instructions yet! I'll make sure to add them asap. Glad you managed to get it working.
What version of VS were you trying to use? I would expect this project to work with VS 2019 too, don't wee why it wouldn't. And I guess I need to make sure it specifies the C# version explicitly to avoid problems like these.
The problem with missing DLLs like steamvr os because you need to build the project once for the DLLs to get generated. So I guess you ended up doing that accidentally? Sorry about the confusion.
By the way, may I ask what kind of contribution you want to make? You can talk to me on Discord about it if you want to.

No worries, I guess this is in LTS now anyway so more changes are unlikely.
I was using VS2019 which doesn't support C#10, but it also obviously had an impact on other things too like the resolving of dependencies. Once I upgraded then everything fell into place.
In regards to what I'm planning, more familiarising myself initially then I hope to re-engineer this into other games starting with Among Trees.
Will reach out on Discord shortly as I suspect I'll have a number of questions.

You can join the Flat 2 VR Discord and find me there with the same name: https://discord.gg/gEEqTVFzvD
Plus you'll have other people helping you with this.
I didn't realize VS2019 didn't support C#10. I might try to change that just to make it easier to contribute to. Anyway, I added some proper build instructions for anyone else who might need them, thanks!


lose state in drag and drop

Hi.
in each row I have a swipeRow that in the hidden layer  have a count down circle ( a number is calculated every 30 seconds).
when I drag one Item or delete it, computational parameters are mixed and return error.
but after seconds it works ok.
I used react-native-draggable-flatlist and  had this problem. I think your library does not have same problem. do you have any solution?
thank you.

Hi,
Use my library so

Hi,
Use my library so

I used it but it has same problem!

Hi,
Use my library so

I used it but it had same problem!

Hi,
Use my library so

I used it but it had same problem!

any solution?


SWS action "Set selected tracks MIDI input to...." no work

"Set selected tracks MIDI input to...." no work!
Cycle Action, it is necessary, on the selected track assign the midi channel.

"Set selected tracks MIDI input to...." actions do work as expected here... Can you please attach a project demonstrating the issue (just the .RPP file) and tell me on which track the problem is supposed to occurs?
TIA

I figured out how it works. Thank you!
But I have not found a function that would change input the selected track. For example with mono:Left to midi Channel All.


Add cargo {test,doc,...} --all-features

When running tests on travis, you usually want to enable all the features when running cargo test. However this requires manually passing all non-default-features one-by-one. It is easy to forget one.
An option should be added to enable all the features.
The follow-up should be to submit a PR or open an issue to travis so that its engine runs cargo test --verbose --all-features instead of just cargo test --verbose by default.

I’d also like cargo test without argument to default to testing with a multiple sets of features specified in Cargo.toml. Currently I have a Makefile to test both with and without a feature, but I never remember to actually use it locally. (Travis-CI uses it, though.)
test:
	cargo test --features query_encoding
	cargo test

An option should be added to enable all the features.

Hmm. What about combinations of features? for example
[features]
a = []
b = []

Would there be 3 runs:

--features ""
--features "a"
--features "b"

Or also a fourth?

--features "a b"

I'd personally expect this to just pass --features "a b" in that case, re-running all the tests may be a bit much.

Since Cargo doesn’t know how features interact with each other, I think this should be left to the library author. The default would be a single run with no features enabled (unless given on the command line), but Cargo.toml could specify multiple runs, and for each of them the set of features to enable.
[lib]
test_features = [[], ["query_encoding"]]

[lib]
test_features = [[], ["a"], ["b"], ["a", "b"]]

I'm currently working on implementing support for a basic --all-features flag.
The scope of my approach to it (just enabling all flags for the project / crate) is not with @SimonSapin's proposal, which might also involve #2980.
I personally would prefer doing what seems most intuitive, just enabling all features.
Having specific feature sets could be nice as well, but I think that this flag would be most useful for building documentation or installing certain binaries.
Testing is obviously another possible use case, but I don't think that a test (as part of a feature) that requires another feature being disabled is not that feasible.

I implemented it in #3038, but without special feature sets for testing.


[SPARK-46416][CORE] Add @tailrec to  HadoopFSUtils#shouldFilterOutPath

What changes were proposed in this pull request?
This pr adds the @scala.annotation.tailrec annotation to the shouldFilterOutPath function in HadoopFSUtils.
Why are the changes needed?
To improve performance.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Pass GitHub Actions
Was this patch authored or co-authored using generative AI tooling?
No

Merged into master. Thanks @yaooqinn


Dabit industries added

All good. Merging.

Thanks for your confirmation!  -Yoon

On Feb 23, 2015, at 2:39 PM, Marcus Liebhardt<EMAIL_ADDRESS>wrote:
Merged #1 https://github.com/turtlebot/map/pull/1.
—
Reply to this email directly or view it on GitHub https://github.com/turtlebot/map/pull/1#event-239928913.


SQLMetadata managers do not have retry strategy on polling

SQL metadata mangers do not have a smart retry strategy when they are polling their databases. A retry strategy should retry (with backoff) on acute errors and emit an alert if the retries fail.
This conversation came up in https://github.com/druid-io/druid/pull/1366
Currently the managers simply skip that polling period and try again the next time around.

Would wrapping the calls in something like SQLMetadataStorageActionHandler.retryingHandle work? (It would have to be made into a static I suppose)
Fwiw, I just noticed that Postgres doesn't have a reasonable implementation of isTransientException, and filed #1382 for that.


Make PolyBind to fail if property value is not found (fixes #4369)

When there is a polyBind:
PolyBind.createChoice(binder, "key", Key.get(Foo.class), Key.get(DefaultFooImpl.class));

And the value in config key=nonFound couldn't be found in classpath, dependency injection must fail with error instead of using DefaultFooImpl. Because it's 100% a error either in classpath or a typo in configuration, and silently consuming this problem may lead to hard-to-find bugs and not met expectations.
Fixes  #4369

👍 after CI

This PR always fails integration tests, probably it is not a transient failure. However I cannot understand how connections failures in Netty related to this PR.

The problem looks that a coordinator failed to start on a docker machine (https://github.com/druid-io/druid/issues/4359#issuecomment-306661974). Maybe there are some places where a coordinator use the default value?

Good job, integration tests 😄

Related: #4380

@b-slim could you please review the newer version

LGTM

@b-slim could this PR be merged?

Just want to note here that this PR appears to have changed behavior of PolyBind.createChoiceWithDefault(..) which is used in some custom extensions. On upgrade to 0.10.1, that led to error like...
"
Unknown provider[xx] of Key[type=com.MyClass, annotation=[none]], known options[yy]
  at io.druid.guice.PolyBind.createChoiceWithDefault(PolyBind.java:105) (via modules: com.google.inject.util.Modules$OverrideModule -> com.MyNamespaceExtractionModule)
  at io.druid.guice.PolyBind.createChoiceWithDefault(PolyBind.java:105) (via modules: com.google.inject.util.Modules$OverrideModule -> com.MyNamespaceExtractionModule)
  while locating com.MyClass
  ...
  ...
"

and following type of code...
        PolyBind.createChoiceWithDefault(
                 binder,
                 TYPE_PREFIX,
                 Key.get(MyExtractionCacheManager.class),
                 Key.get(OnHeapMyExtractionCacheManager.class),
                 "onHeap"
         ).in(LazySingleton.class);

was required to be replaced with

         PolyBind
                 .createChoiceWithDefault(binder, TYPE_PREFIX, Key.get(MyExtractionCacheManager.class), "onHeap")
                 .in(LazySingleton.class);
          PolyBind
                 .optionBinder(binder, Key.get(MyExtractionCacheManager.class))
                 .addBinding("onHeap")
                 .to(OnHeapMyExtractionCacheManager.class)
                 .in(LazySingleton.class);

@leventov does that look expected ?

@himanshug it looks like there is a problem with createChoiceWithDefault()


Userbot and selfbot issues

Currently, several minor issues were reported when running a selfbot or a userbot:

[x] (CNext) RequireOwner attribute does not correctly recognize an owner

This is minor, and thus should be released as 2.2.x

Fixed in 2.3.


Add unit tests for redis.clients.jedis.util.KeyMergeUtil

Hi,
I've analysed your code base and noticed that redis.clients.jedis.util.KeyMergeUtil in the jedis module is not fully tested.
I've written some tests that cover this class with the help of Diffblue Cover.
Hopefully, these tests should help you detect any regressions caused by future code changes. If you would find it useful to have additional tests written for this repository, I would be more than happy to look at other particular classes that you consider important.

LGTM!

@sazzad16 why 3.0.2?


Add some kind of omnitool/ideas for upgrades

The dark pick already has the spoon and travel upgrades which make life a whole lot easier during the game. However, when out exploring and collecting resources, I generally just end up taking my Atomic Dissassembler from Mekanism due to its multi-purpose nature to save inventory space, and I really miss having the same features in my EnderIO tools (which is my favorite mod and what I try to use for everything).
So what I'm thinking is maybe add another tier of tool that combines all other tools into one, so that you can harvest/attack anything in your path with the same tool (plus it doubling as a Travel Staff). This could be done either by upgrading the pick with the specific stuff you want (something like the spoon but for harvesting wood, for example) or (which I personally think is the cooler option) by adding a new tool (maybe crafted by combining all dark tools?).
This, of course, should not be cheap (which is a major gripe I have with the paxels and atomic dissassembler from mekanism). Some ideas I have are:

Crafting the omnitool (which would be expensive on it's own) and then upgrading it with each tool you want to have the functionality of (you could even require the component tool to be fully empowered, and having the upgrades not transfer over, to make it more expensive). So, for instance, if I want my omnitool to act as both a pick and a sword, I would have to add a (maybe fully upgraded) pick and sword to it in an anvil.

The aesthetics of it could change based on what upgrades you have, similar to Tinker's Construct's tools. So if you add a sword, it would look like you actually added some sort of functional blade to it. (I don't know how feasible such a thing is to do)

It would be really really awesome if it could also work as a wrench, which is something I think no omnitool-like item so far does.

Maybe adding an upgrade for it so that you can optionally have it not have durability and instead rely entirely on power (the drawback being that if you run out of power it's back to being like using your fists). Reinforced obsidian would be a good item to use in such an upgrade, since obsdian already gives the Unbreaking enchant, reinforced obsidian could make the EnderIO tools unbreakable.

Maybe have a way to control the speed? I have a Dark Pick with Efficiency V and Spoon, which is great when mining but absolutely terrible when trying to break stuff (specially dirt) with any precision. This is of course bad planning on my part, but a real omnitool should be able to be useful under any circumstance one finds oneself in, be it breaking that one block or pretending to be a human quarry. I understand, though, that this might be too close to an atomic dissassembler to be worth it.

On the subject of human quarries, maybe a way to break multiple blocks at once? IIRC the flux-infused pickaxe can mine a 3x1x1 hole, and there's a pick from Thaumcraft that can mine an offset 5x5x1 hole (great for building rooms and large tunnels). Maybe a way to add some presets like that? (They could each be an independent upgrade, and if the omnitool is a new item and not just a dark pick full of upgrades, it could be so that adding those upgrades to a Dark Pick is permanent and in the omnitool they become new modes of operation and not permanent changes, to keep to the idea that it is the ultimate-tier tool that's useful for everything)

Two words: MAGNET UPGRADE. No more wasted hotbar space. Bonus points if there's a way to make it only work for the stuff the tool mines and not everything around you (although I think this would have to work more like instant teleportation to your inventory than a proper magnet... which, now that I think of it, is very Ender-y and cool).

Those are just some ideas I had, they are not polished and certainly I don't expect them to be accepted in their entirety and as-is. Even if the omnitool is out of the question (which I would completely understand), maybe some of those upgrade ideas could be worth something to the existing tools.

Paxel, combine all 3 tools into a paxel and add all the upgrades into the paxel from all 3 tools as normal.


Enable history-substring-search as a default module

Enable history-substring-search as a default module
Resolves #1868
Proposed Changes

Enable history-substring-search as a default module

The motivation
I love prezto at least this 4 or 5 years. Thanks for this so useful project!
This time, I have casual updated to my own local prezto.
Then all dotfiles around zsh has been removed with my careless operation... 😇
All of them are completely my mistaken.
Then I have tried to get back my loving environment again.
Honestly, It taken no short time.
And I noticed, I mostly loving the feature is the history-substring-search.
And I noticed again, making it is a default is suggested in https://github.com/sorin-ionescu/prezto/issues/1868.
I believe it makes happy for new users of prezto. So I have created this PR.
I'm afraid...
I think 2 consideration points exist.
https://github.com/sorin-ionescu/prezto/blob/704fc46c3f83ca1055becce65fb513a533f48982/README.md#usage

Prezto has many features disabled by default.

I guess it is an important philosophy of this project. It makes light and fast this prezto behaviors.
Actually, the default loading modules looks not changed this 9 years!
https://github.com/sorin-ionescu/prezto/blob/704fc46c3f83ca1055becce65fb513a533f48982/runcoms/zpreztorc#L30-L42
https://github.com/sorin-ionescu/prezto/commit/cc7e43b2422d044c98da2b5c5a1ebf6e8b2ac40d
It sounds this suggestion might not fit for prezto philosophy, but I couldn't keep to stop sending this PR from my motivation. :bow:
But I think this change does not make much slower behaviors, I have checked with below.
 But this ordering works fine in my machine.
If this has any problem, please pointing out. 🙏

I use it a lot and am +1 as well!
While we are usually conservative in updating default in runcoms/* (it is expected to be very conservative by default and among other things, it requires git rebase requires manual resolution when a change happens), in this case, I agree the benefit is relevant without any obvious downside.
I'll go ahead and apply this. We can always revisit this in case it causes too mush disruption.

Merged, thank you @kachick!

@jeffwidman @indrajitr Thanks for your approval! 🙏


Documentation "quick-start"

Hello
I have some questions regarding the documentation in https://github.com/COMBINE-lab/alevin-fry#a-quick-start-run-through-on-sample-data
I read and worked through the quick-start and pull the latest singularity image, singularity pull docker://combinelab/usefulaf:latest
There some discrepancies between that version and the documentation:

"Building the splici reference and index": there is no parameter -l 91, but there is a parameter -r.
"Quantifying the sample":  error: Invalid value 'u' for '--forced-cells <FORCED_CELLS>': invalid digit found in string

So, I am not sure whether the documentation fits to the latest (docker/singularity) version?
Thank you and best regards,
R.
PS: and when I run with it, for example, with option --knee I get an unknown error
singularity exec --cleanenv --bind $AF_SAMPLE_DIR:/workdir --pwd /usefulaf/bash usefulaf_latest.sif simpleaf quant -1 /workdir/data/pbmc_1k_v3_fastqs/pbmc_1k_v3_S1_L001_R1_001.fastq.gz,/workdir/data/pbmc_1k_v3_fastqs/pbmc_1k_v3_S1_L002_R1_001.fastq.gz -2 /workdir/data/pbmc_1k_v3_fastqs/pbmc_1k_v3_S1_L001_R2_001.fastq.gz,/workdir/data/pbmc_1k_v3_fastqs/pbmc_1k_v3_S1_L002_R2_001.fastq.gz -i /workdir/human_CR_3.0_splici/index -o /workdir/quants/pbmc1k_v3 --knee -c v3 -r cr-like -m /workdir/human_CR_3.0_splici/ref/transcriptome_splici_fl86_t2g_3col.tsv -t 8
2023-02-02T12:28:23.288555Z  INFO simpleaf: deserializing from File { fd: 3, path: "/afhome/simpleaf_info.json", read: true, write: false }
2023-02-02T12:28:23.288601Z  INFO simpleaf: prog info = ReqProgs { salmon: Some(ProgInfo { exe_path: "/opt/conda/bin/salmon", version: "1.9.0" }), alevin_fry: Some(ProgInfo { exe_path: "/opt/conda/bin/alevin-fry", version: "0.8.1" }), pyroe: Some(ProgInfo { exe_path: "/opt/conda/bin/pyroe", version: "0.7.1" }) }
Error: custom geometry string doesn't contain ';' character

Thanks! Can you please share the full command you are using fir quantification?  We'll fix bugs in the documentation that may have arisen from the continued development of the software.

Hi!
This the command I used, same as in previous post but with line breaks
singularity exec --cleanenv     \ --bind $AF_SAMPLE_DIR:/workdir     \ --pwd /usefulaf/bash usefulaf_latest.sif     \ simpleaf quant     \ -1 /workdir/data/pbmc_1k_v3_fastqs/pbmc_1k_v3_S1_L001_R1_001.fastq.gz,/workdir/data/pbmc_1k_v3_fastqs/pbmc_1k_v3_S1_L002_R1_001.fastq.gz     \ -2 /workdir/data/pbmc_1k_v3_fastqs/pbmc_1k_v3_S1_L001_R2_001.fastq.gz,/workdir/data/pbmc_1k_v3_fastqs/pbmc_1k_v3_S1_L002_R2_001.fastq.gz     \ -i /workdir/human_CR_3.0_splici/index     \ -o /workdir/quants/pbmc1k_v3     \ --knee -c v3 -r cr-like     \ -m /workdir/human_CR_3.0_splici/ref/transcriptome_splici_fl86_t2g_3col.tsv     \ -t 8
in the doc there is this line -f u -c v3 -r cr-like , but I replaced -f u by --knee because -f u raises an error.
I hope this helps.
Thank you!

Ohh! I see. The problem is that we have massively upgraded simpleaf but not this particular documentation page. Great catch!  The simpleaf docs can be found here.  We will update this QuickStart to be correct.
In addition to the -l to -r difference you found during indexing, you should change your quant command in the following way.

change -c v3 to -c 10xv3
replace —-knee with -u

This should then work equivalently to what is currently in the tutorial.  Please let us know if this works, and great catch. Thanks for reporting it!
—-Rob

Thanks, Rob!
The above parameters for the docker/singularity images worked. No more errors regarding input parameters.
The workflow finished with the message Error: quant failed with exit status ExitStatus(unix_wait_status(512))
singularity exec --cleanenv     --bind $AF_SAMPLE_DIR:/workdir     --pwd /usefulaf/bash usefulaf_latest.sif     simpleaf quant     -1 /workdir/data/pbmc_1k_v3_fastqs/pbmc_1k_v3_S1_L001_R1_001.fastq.gz,/workdir/data/pbmc_1k_v3_fastqs/pbmc_1k_v3_S1_L002_R1_001.fastq.gz     -2 /workdir/data/pbmc_1k_v3_fastqs/pbmc_1k_v3_S1_L001_R2_001.fastq.gz,/workdir/data/pbmc_1k_v3_fastqs/pbmc_1k_v3_S1_L002_R2_001.fastq.gz     -i /workdir/human_CR_3.0_splici/index     -o /workdir/quants/pbmc1k_v3     -u -c 10xv3 -r cr-like     -m /workdir/human_CR_3.0_splici/ref/transcriptome_splici_fl86_t2g_3col.tsv     -t 82023-02-02T13:04:55.123087Z  INFO simpleaf: deserializing from File { fd: 3, path: "/afhome/simpleaf_info.json", read: true, write: false }
2023-02-02T13:04:55.123152Z  INFO simpleaf: prog info = ReqProgs { salmon: Some(ProgInfo { exe_path: "/opt/conda/bin/salmon", version: "1.9.0" }), alevin_fry: Some(ProgInfo { exe_path: "/opt/conda/bin/alevin-fry", version: "0.8.1" }), pyroe: Some(ProgInfo { exe_path: "/opt/conda/bin/pyroe", version: "0.7.1" }) }
2023-02-02T13:04:55.123209Z  INFO simpleaf: cmd : "/opt/conda/bin/salmon" "alevin" "--index" "/workdir/human_CR_3.0_splici/index" "-l" "A" "-1" "/workdir/data/pbmc_1k_v3_fastqs/pbmc_1k_v3_S1_L001_R1_001.fastq.gz" "/workdir/data/pbmc_1k_v3_fastqs/pbmc_1k_v3_S1_L002_R1_001.fastq.gz" "-2" "/workdir/data/pbmc_1k_v3_fastqs/pbmc_1k_v3_S1_L001_R2_001.fastq.gz" "/workdir/data/pbmc_1k_v3_fastqs/pbmc_1k_v3_S1_L002_R2_001.fastq.gz" "--threads" "8" "-o" "/workdir/quants/pbmc1k_v3/af_map" "--sketch" "--chromiumV3"
2023-02-02T13:07:22.899766Z  INFO simpleaf: cmd : "/opt/conda/bin/alevin-fry" "generate-permit-list" "-i" "/workdir/quants/pbmc1k_v3/af_map" "-d" "fw" "--unfiltered-pl" "/afhome/plist/10x_v3_permit.txt" "--min-reads" "10" "-o" "/workdir/quants/pbmc1k_v3/af_quant"
2023-02-02T13:07:32.961164Z  INFO simpleaf: cmd : "/opt/conda/bin/alevin-fry" "collate" "-i" "/workdir/quants/pbmc1k_v3/af_quant" "-r" "/workdir/quants/pbmc1k_v3/af_map" "-t" "8"
2023-02-02T13:07:38.327151Z  INFO simpleaf: cmd : "/opt/conda/bin/alevin-fry" "quant" "-i" "/workdir/quants/pbmc1k_v3/af_quant" "-o" "/workdir/quants/pbmc1k_v3/af_quant" "-t" "8" "-m" "/workdir/human_CR_3.0_splici/ref/transcriptome_splici_fl86_t2g_3col.tsv" "-r" "cr-like"
Error: quant failed with exit status ExitStatus(unix_wait_status(512))

Maybe I should now switch to the documentation here, https://alevin-fry.readthedocs.io/en/latest/index.html, and tutorials referenced there.
Also, the docker/singularity images are not mentioned anymore in the "readthedocs" documentation. Would you then recommend to use only the versions from bioconda and not from docker/singularity ?
Many thanks!

@DongzeHE any thoughts about this error above? It gets to the very last step. Is there any difference in where e.g. pyroe puts the t2g vs roe?

Hi @rob-p
Yes, now everything worked perfectly and without warnings nor errors.
Then, I will continue using the Singularity/Docker image and will try out the tutorials provided through the readthedocs manual.
Many thanks for your rapid support!
Best regards,
Roland

Hi @schmucr1,
Sorry for the inconvenience. As simpleaf copies and pastes the t2g files when building the index, if you don't want to check the name of the t2g_3cols.tsv file every time, you can also pass the t2g_3col.tsv file in the index folder, which is in the same directory as ref, to -m.  This file is always named as t2g_3col.tsv.
-m /workdir/human_CR_3.0_splici/index/t2g_3col.tsv

Best,
Dongze

Hi @DongzeHE and @rob-p
Thank you for the explanations. I will consider this naming conventions for the upcoming runs.
On another note and not related to this issue, do you think that alevin-fry is able to process data generated with the DNBelab C Series Single-Cell Library Prep Set, ie not 10x data ? I am mostly interested in a macaque data set published here: https://www.nature.com/articles/s41586-022-04587-3#Sec8
Many thanks and kind regards,
R.

Hi @schmucr1,
Short answer:
Yes. You need to pass the location of UMI and Cellular barcode in the technical read (read 1) as a custom geometry to simpleaf. See here for details.
Long answer:
Upon checking, I think alevin-fry is able to handle this dataset. However, it will be a little bit tricky because the geometry specification (the positions of UMI and cellular barcode in read1) of that protocol is unclear to me.
In the paper you shared, it mentioned that they used the prep set as another study. And in that study, the authors referred to the original preprint of that prep set.
The interesting thing is the three studies listed above did not apply the same sequencing strategy.

In the original preprint introducing that prep set, the author said

For C4 scRNA-seq data, the cell barcodes (base 1 to base 10 and base 17 to base 26) and UMIs (base 32 to 41) are in read 1 and the cDNA reads are in read 2.

In the paper you shared, the authors used a different sequencing strategy and said they used PISA for processing the sequencing reads before aligning. However, I cannot find the commands for processing FASTQ files in the corresponding GitHub repository.

41-bp read length for read 1 and 100-bp read length for read 2.

In the other study, the sequencing strategy was

The read structure was paired-end with Read 1, covering 30 bases inclusive of 10-bp cell barcode 1, 10-bp cell barcode 2 and 10-bp unique molecular identifier (UMI), and Read 2 containing 100 bases of transcript sequence, and 10-bp sample index.

Therefore, I would suggest you ask the authors about the geometry specification of the prep set they used and set the appropriate custom geometry in simpleaf accordingly.
Best,
Dongze

@DongzeHE,
I agree, the right course of action, first, is to really understand the geometry.  From there, figuring out how to provide it should be straightforward.
One point of information is that while the mapping tools (both salmon and piscem) could support such geometry, I think that simpleaf may currently have some silly limitations about having a single contiguous barcode.  That is if you look in the docs, I'm not sure if you could easily do something like B1[1-10;17-26]. Of course, there is no difficulty supporting this since both salmon and piscem support multi-part barcodes in the backend, we'd just have to write that support.  Ultimately, what happens is that the "simpleaf" format is parsed, and then converted to the appropriate geometry type for either salmon or piscem.  Once we know the library type we are dealing with, we can figure out the easiest way to get the data into the pipeline (a stop-gap solution would just be to preprocess read 1 e.g. with awk or some such) so that the barcode is contiguous.
--Rob


Implement cookie for theme choice

where did you copy-paste get- and setCookie from? :D (I don't like how they named things, it makes the boiler code way less readable. It's now part of your codebase, so I'd change the namings ;))
getCookie and setCookie functions are great candidates for lib-like functionality. (functionality that doesn't belong to any component really and that's potentially reusable across the applicaiton)
wanna try to add a src/lib folder to your application and extract these functions there?

Yeaaaahh I forgot about it... You've mentioned it already. ;) And yes it felt a bit wrong to put it there. I will change it as requested.

where did you copy-paste get- and setCookie from? :D (I don't like how they named things, it makes the boiler code way less readable. It's now part of your codebase, so I'd change the namings ;))

From w3schools. ;) I'll change the naming.

coool!


bugfix: i18n display. Fixes #301

Codecov Report

Merging #302 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #302   +/-   ##
==========================================
  Coverage      19.37%   19.37%
  Complexity       289      289
==========================================
  Files             91       91
  Lines           3639     3639
  Branches         566      566
==========================================
  Hits             705      705
  Misses          2859     2859
  Partials          75       75

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 851f0c7...1483c6f. Read the comment docs.


feat(tutorials): update feature branch tutorials

Description
This PR is just for updating the tutorials feature branch, cause it is really old. We still need this feature branch to work on the initial tutorial section.
No need for a review, cause the review is done, when the tutorial branch is merged to main. There are so many changes, cause the structure of tractusx.github.io has changed

No need for a review, cause the review is done, when the tutorial branch is merged to main. There are so many changes, cause the structure of tractusx.github.io has changed

As it is a external contribution we need a review here.

@carslen : What do you mean with external contribution?

Hi @stephanbcbauer as we already synced i would also decline this PR, because of the size an that we can not be sure if something will be missed if we merge it into main.
as already pointed out in our conversation, If such big PR come into our way we need to open a IP-Issue against the EF. see https://www.eclipse.org/projects/handbook/#ip-faq point 2.

So if there urgent things you will need to change on main i would please you to open a new PR from a different branch eg. feat/new-feature or in our case doc/update-documentation

Hey all, since the PR does not target main, but a feature branch, I can see why it is so big.
It just takes the whole history from main. So content, that has already gone through regular PRs.
@stephanbcbauer can you confirm that?

Hey all, since the PR does not target main, but a feature branch, I can see why it is so big. It just takes the whole history from main. So content, that has already gone through regular PRs.
@stephanbcbauer can you confirm that?

yes can confirm this :) the initial feature branch was created month ago :) and unfortunatly was never rebased :( and yes the changes were already reviewed :)
But to provide better feelings... i created a new feature branch and published the intial changes (created by felix gerbig) -> this is our base to work on the tutorial.

@carslen : What do you mean with external contribution?

The PR is initiated from a fork of this repository.

@carslen i am confused :) cause all teams work with forks and provide changes to origin with a PR from fork -> origin. Whats wrong here? Did i missunderstand something?


Roadmap?

Let me start with mentioning that I love to see this repository being worked on again. It's one of the coolest web apps I've seen in a while. I'd be interested if you have any concrete plans for future features of this project?
New web APIs from the recent years have made me want to fork or reimplement ShareDrop several times, but I have never found the time to do so.
I'm particularly curious if you're planning to integrate features related to PWAs or the Web Share Target API?

Thanks!

New web APIs from the recent years have made me want to fork or reimplement ShareDrop several times, but I have never found the time to do so.

That's exactly my problem :)
I'd like to rewrite it in (p)React, add service workers, replace PeerJS library with something modern (e.g. simple-peer), maybe even replace the current custom transfer protocol with webtorrent, look into improving transfer speed, fix bugs, improve error handling, figure out how to make it work on iOS etc. Web Share Target API is something I haven't looked into at all yet, but it seems like a nice fit for ShareDrop.
Unfortunately, I have no idea when/if I find time to work on that.

The old, ubiquitous problem I guess. 😁

+1 for Service Worker and Progressive Web App. It would be awesome to have sharedrop installable as PWA on almost every device and offline too


Failed to minify the code from this file ./node_modules/getstream/src/lib/client.js:159

I'm having trouble compiling with npm run-script build when using the getstream library.
Steps to reproduce:

Download sample app from https://codesandbox.io/s/jvm683w6pw
npm install
npm run-script build

And the output:
 $ npm run-script build
><EMAIL_ADDRESS>build /Users/jimmy/Downloads/jvm683w6pw
> react-scripts build

Creating an optimized production build...
Failed to compile.

Failed to minify the code from this file:

 	./node_modules/getstream/src/lib/client.js:159

Read more here: http://bit.ly/2tRViJ9

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `react-scripts build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/jimmy/.npm/_logs/2018-03-14T11_38_21_345Z-debug.log

As you can see the app is just a vanilla sample React app from https://codesandbox.io and the only addition was adding getstream as a dependency and import stream from "getstream";.
When trying with UglifyJS on my other project I seem to get a more verbose error message:
Unexpected character '`' [./~/getstream/src/lib/client.js:159,0][static/js/main.f24a4d28.js:94724,41]

Thanks for the report. Will investigate.

On https://codesandbox.io/s/jvm683w6pw when I click Refresh on the IDE Browser I see this error message:
Error
/node_modules/getstream/src/lib/client.js hasn't been transpiled yet.

Hope if helps :)

Thanks @dwightgunning

Fixed on master. Will be included in an upcoming package / release to be published to npm.
Thanks again for raising the issue.


sem: remove limitation of irq context when do sem_trywait

Summary
sem: remove limitation of irq context when do sem_trywait
Impact
allow mm_takesemaphore in IRQ, just try get
Testing

NOT alloc memory, just get the mm lock to check corruption

This will actually make the free part succeed, and we will be freeing memory if we successfully get the lock on the first try. malloc in the other hand only DEBUGVERIFY the check, when debug options are disabled we are then allocating memory and possibly without holding the lock?

Is this a good idea? Allocating memory in an interrupt context can effect latency and add jitter.
This will actually make the free part succeed,

The intent of change is allowing https://github.com/apache/incubator-nuttx/pull/4759 can do the check even in the interrupt context. So, malloc/free shouldn't call from the interrupt context.

and we will be freeing memory if we successfully get the lock on the first try. malloc in the other hand only DEBUGVERIFY the check, when debug options are disabled we are then allocating memory and possibly without holding the lock?

How about we add DEBUGASSERT in malloc/free to catch the wrong usage?

Is this a good idea? Allocating memory in an interrupt context can effect latency and add jitter.
This will actually make the free part succeed,

The intent of change is allowing #4759 can do the check even in the interrupt context.

and we will be freeing memory if we successfully get the lock on the first try. malloc in the other hand only DEBUGVERIFY the check, when debug options are disabled we are then allocating memory and possibly without holding the lock?

How about we add DEBUGASSERT in malloc/free to catch the wrong usage?

Why not make the  mm/mm_heap/mm_sem.c change dependent on CONFIG_DEBUG_MM?
Effectively the code is left as is when CONFIG_DEBUG_MM is not defined and as in this PR when it is defined?

@davids5 Good idea! done

@davids5 Please review the newest version


Fix typo

This patch just fixes typo introduced in commit 4d737b6427ecf78ec3301d0de77196bd7ad9cf8c.
Signed-off-by: Peter Lemenkov<EMAIL_ADDRESS>

Just a note - Travis-CI job failure isn't related to my patch. It fails while installing packages for testing.

Hi @lemenkov , I know, I was just trying to fix travis stuff ;)


fix(transformer/optional-chaining): avoid creating a useless reference when noDocumentAll is true

Merge activity

Dec 13, 8:28 AM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
Dec 13, 8:28 AM EST: A user added this pull request to the Graphite merge queue.


Subtitles background is no longer visible

Describe the bug
Since the 0.1.4 update I can't see the subtitles background regardless of color/alpha channel settings.
Additional info (please complete the following information):

Device: galaxy s23
Android version: 14
App Version: Release v0.1.4

seems to have been caused by this
https://github.com/mpv-player/mpv/issues/14777
an easy fix

should be fixed, can you check the nightly to make sure?


Can we add an snapshot option so users can choose whether to include receipts in snapshot by themselves?

Can we add an snapshot option so users can choose whether to include receipts in snapshot by themselves?
Originally posted by @janx in https://github.com/cryptape/cita/issues/531#issuecomment-487859019

@ouwenkg @keroro520

same as #323


include gimms version number 0

The gimms package was recently updated to include version 1 of the gimms dataset and set version 1 as the default. Version 1 changes the file structure so most of get_ndvi_data will have to be redone if we switch to it. This will stick to version 0 for now.

Thanks @sdtaylor. Does version one have later dates than version 0?

Yes it does. 0 goes to 2013 and 1 goes to 2015, so it's something we'll want to update to.


Setup Project & react app

ran the create-react-app script to make the basic react template
setup issue templates
setup github workflow for CI stuff:

build project
run tests
make sure code is formatted according to prettier's standards

readme file

Note: the React code is in ./react because ROS and maybe flask are probably going to have to be added in the future to be able to actually interact with the robot, and they should not be in the same directory lmao

I am going to merge this without a review because this is pretty essential to do any code lol


Add an episode detail page to dashboard

We need a page in the dashboard that provides details about an episode (~ the data available on ClickHouse).
It should link to the corresponding inference pages in #590.

I'll share a mockup before implementation.


How to get trafficlights and buildings shown?

Hi, very great work!
When I am running the example (with carla 0.9.13), I can get the map and ego vehicle shown, but without any trafficlights and buildings. Like this:

So, how do you get  trafficlights and buildings shown on your demo2.jpg ?

Hi Carlaviz:0.9.13 does not support building display. You may want to upgrade to the latest version with latest Carla simulator.


Handle event logistics, including scheduling, invitations, and follow-up communications.

Am interested in the volunteer logistic aspect


add license title

It's not strictly required, but it's useful metadata, and part of the recommended license template text (see http://choosealicense.com/licenses/isc/ and https://opensource.org/licenses/isc-license)

[Current coverage][cc-pull] is 74.36%

Merging [#897][cc-pull] into [master][cc-base-branch] will increase coverage by 0.02%

@@             master       #897   diff @@
==========================================
  Files           128        128
  Lines          6742       6742
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           5012       5014     +2
+ Misses         1730       1728     -2
  Partials          0          0

Powered by Codecov. Last updated by [9d0b4e8...9c6839a][cc-compare]
[cc-base-branch]: https://codecov.io/gh/tonioo/modoboa/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/tonioo/modoboa/compare/9d0b4e8533ba02a20f26137ac2b4e2a25c13fe74...9c6839a3a9427bb2b96a24d26e6c3448379f9302
[cc-pull]: https://codecov.io/gh/tonioo/modoboa/pull/897?src=pr

Indeed. Thank you.


Email plugin does not choose proper channel configuration

Copy of https://github.com/mirumee/saleor/pull/7876 for Saleor 3.1
Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-4227 Email plugin does not choose proper channel configuration


NameError: name 'cdist' is not defined

Hi,
thank you for the repo and paper. It is a really great work!
Unfortunately, I tried to run your repo but I received a NameError: name 'cdist' is not defined in ECCV22-FOSTER/models/base.py", line 132, in _eval_nme.
I simply fixed the error adding the following import at the beginning the file models/base.py
from scipy.spatial.distance import cdist

I opened this issue to improve the repo. Probably you simply missed it 😉
Best,
Niccolò

Thank you so much for your attention and help!👍 I think I deleted this line by mistake when deleting my code comments.


Build failed in ARM64(Ubuntu 18.04 )

Could it be buit on ARM64 servers? I tried to execute ./buildall.sh on ARM64, but it failed when build binutils package, and the config.log as http://paste.openstack.org/show/786963/ Could you please have a look?

Then I enter binutils/binutils-2.26.1, config and make manually, it success.


⚡ Feature | EXPLORE - Add login modal on follow button

On the Explore page, when logged out, we need to display the "follow" button to convert the users.
When the logged-out user clicks on the "follow" button, display the modal below:

https://www.figma.com/file/elxKahbV8rj85IWOEyZeVu/Product-design?node-id=4187%3A29767

@QuentinSaubadu can we do a generic modal that can be used for any part where the user is required? Meaning remove the "to follow writers" etc.. have some generic text

@QuentinSaubadu can we do a generic modal that can be used for any part where the user is required? Meaning remove the "to follow writers" etc.. have some generic text

100%! That's a good idea.
We could replace the text by:

Login to continue
To proceed, you must first be logged in.
Button> Cancel Button>Go to login page

If you're ok with that, I'll make it on Figma and edit this issue @pradel

@QuentinSaubadu I think the content of the modal should show the benefits of joining Sigle, eg: list the features you will get by creating a accounts / logging-in. Wdyt?

@QuentinSaubadu @pradel as we are adding the login prompt on the user cards which are used for the explore and profile page, I'm wondering if it also makes sense to add the same logic for other areas in the app where we currently hide the follow button (we also hide it in the profile card on public story)—wdyt?

@QuentinSaubadu @pradel as we are adding the login prompt on the user cards which are used for the explore and profile page, I'm wondering if it also makes sense to add the same logic for other areas in the app where we currently hide the follow button (we also hide it in the profile card on public story)—wdyt?

@gregogun that's a good idea, let's do that.
@pradel I thought we already discussed it and find out that wasn't a priority, and that we could improve the modal later on, but I can try to make something quick. @gregogun I might change a little bit if I manage to put the list of features.

Final version on FIGMA: https://www.figma.com/file/uPVMeBq0bERyViiRMXzLN8/Sigle-design-system?node-id=3138%3A7449
@gregogun @pradel


Add support for prompt caching (image + video)

To solve #136
We can load KV cache (attention sink) and compute values for new image and prompt tokens.


Adding Sub Column Support

Hi there.
I had reviewed the list of feature requests and I don't think I saw this particular one - if it's a duplicate feature request - my appologies.
I'd like to know if it's possible to add sub-column support.
Take a look at the attached screen shot.

see #955 #530

Thanks.  See those issues now.


Revert Chrome Refresh 2023 or keep it avoidable

Google started to introduce a UI refresh, starting from M115. The change is called "Chrome Refresh 2023". There are many bug reports, UX issues and negative feedback about this change and Google seems to be stubborn enough not to pay attention and force it upon all of its users, like they did with DirectWrite, however this change has already received even more negative feedback. With M125 release, Google is expiring (and later removing) the flags that can disalbe the changes of Chrome Refresh 2023.
Chrome Refresh 2023 is already in latest Thorium-Win7 (now: 122.0.6261.168) as it's based on M122 but the feature is turned off by default via flags. From a desktop user point of view it's an anti-feature that ruins the classic desktop experience (with mouse and keyboard), wastes vertical real estate, replaces icons to colorless unrecognizable ones. These changes are even visually incompatible with a compact desktop running Windows 7, Vista or XP.
My suggestions are the following, whichever is easier to implement.

Completely remove Chrome Refresh 2023 and keep the old looks.

Keep flags related to Chrome Refresh 2023 so Thorium users can turn them on/off.
Introduce a configuration option under Settings > Appearance to turn on/off the whole (or parts of this) UI change.

Chrome Refresh 2023 flags

#chrome-refresh-2023 Chrome Refresh 2023
#chrome-refresh-2023-ntb Chrome Refresh 2023 New Tab Button
#chrome-refresh-2023-top-chrome-font Chrome Refresh 2023 Top Chrome Font Style
#chrome-webui-refresh-2023 Chrome WebUI Refresh 2023
#customize-chrome-side-panel Customize Chrome Side Panel
This may not be necessary. It's about a bug that if you don't turn this flag to Disabled, Chrome Refresh 2023 flags are all ignored. See: https://issues.chromium.org/issues/330756862

Issues
Issues related to Chrome Refresh 2023, from the Chromium Bug Tracker.

https://issues.chromium.org/issues/335553327
https://issues.chromium.org/issues/332574933
https://issues.chromium.org/issues/332081796
https://issues.chromium.org/issues/330842283
https://issues.chromium.org/issues/330743875
https://issues.chromium.org/issues/330753883
https://issues.chromium.org/issues/330589210
https://issues.chromium.org/issues/330443177
https://issues.chromium.org/issues/330373923
https://issues.chromium.org/issues/329953462
https://issues.chromium.org/issues/327022941
https://issues.chromium.org/issues/41490387
https://issues.chromium.org/issues/40947720

Negative user feedback
Negative feedback from mostly advanced users who suffered a UX degradation because of Chrome Refresh 2023.

https://support.google.com/chrome/thread/269780757
https://support.google.com/chrome/thread/269812926
https://support.google.com/chrome/thread/269774466
https://support.google.com/chrome/thread/269868881
https://support.google.com/chrome/thread/269982965
https://support.google.com/chrome/thread/269956094
https://support.google.com/chrome/thread/269946774
https://support.google.com/chrome/thread/269800018
https://www.reddit.com/r/chrome/comments/1c5jgmu/icons_in_chrome_tabs_are_misplaced/
https://www.reddit.com/r/chrome/comments/1c695ef/how_to_get_tab_favicons_back_to_normal/
https://www.reddit.com/r/chrome/comments/1c5zvnx/disabled_chrome_refresh_2023_and_customize_chrome/
https://www.reddit.com/r/chrome/comments/1c5xeja/broken_icon_placements_after_update_how_to_fix/
https://www.reddit.com/r/chrome/comments/1c6ette/why_are_my_tab_icons_higher_up_than_normal/

@highbaser I've already done this in regular Thorium M123 because I hate it too.
When this repo gets rebased on Chromium/Supermium 123, it will have those fixes too.
However, Chromium 125 completely removes all the older code related to non-CR23 UI. In that case it will be alot harder to maintain, and depending on the amount of files involved, I might use CR23 UI. I would like to make a patch to at least put the tab search button back on the right side again though.

As far as I know, Supermium devs have reworked the entire tab drawing logic in order to have a look&feel that fits into the XP ecosystem. You advertise thorium-legacy (Thorium-Win7) as something that fits into the Win7 ecosystem aero tab bar, etc. Chrome Refresh 2023 ruins this all so I think it's as much vital to keep it as important it was to create all your compatibility patches for Win7. Also, Supermium is planning to keep the UI before refresh in the long term.

It seems these imbeciles can't help but change for the sake of change and not let people choose their damn design. If it were so good people wouldn't be trying to avoid it like the plague it is. This kind of thing made me abandon Firefox years ago.


fix(chore): remove stub type definition for handlebars

<EMAIL_ADDRESS>This is a stub types definition. handlebars provides its own type definitions, so you do not need this installed.

cc @juanpicado

Thanks, good catch, I will test it when I have a keyboard in front of me.

The lock file changes don't seem to show up on my local machine :thinking:

The lock file changes don't seem to show up within my local machine :thinking:

The lock file changes don't seem to show up within my local machine 🤔

That's odd, I run yarn install in my local env and I have changes. How are you actually doing it?

My local environment was all messed up. Guess it's fine now :thinking:

Thanks again @jamesgeorge007

@juanpicado As you may have noticed we've our project: Mevn-CLI which is still in the infancy state. We're not getting enough reach which is really important for a project of such sort. You could really help us out by spreading the world :smile:

Also, we're accepting financial contributions: open-collective
Twitter-handle


dind-flexvolume still has issues with parallel use

Seems that on bionic, we can no longer just mount and unmount directly in the flexvolume driver whenever required, as there's trouble with mounting the same device into two locations (sentinal and dotmesh-server).
https://gitlab.dotmesh.com/dotmesh/dotmesh-sync/-/jobs/99595
As the flexvolume driver is asked to mount directly into a container's filesystem (or, at least, into a container-specific path), and we previously relied on Linux letting us mount the same loopback image in more than one location, we probably need to:

Have a reference counting system to track multiple mounts of the same PV
Mount it just once, in some location alongside the image file
bind-mount that out to where it's asked for via the FV API
Unmount the bind mounts when asked to via the FV API
On the last unmount, unmount the loopback device

And, for extra points, we can also make the dind-flexvolume driver write to .../cleanup-actions.XX file in the test root with explicit unmounts and loopback-device-destroys  to ensure that we always get cleanup of all the mounts it makes.

I might have been wrong about this - let's investigate the cause of the error in the linked build:
Aug 05 07:42:18 === RUN   TestKubernetesOperator/PVCMakeNew
Aug 05 07:42:19 panic: runtime error: index out of range [recovered]
Aug 05 07:42:19 	panic: runtime error: index out of range

I might have seen errors in different runs. Kubernetes PVC dind-flexvolume was separately complaining about running out of loopback devices. Investigate more...

Aug 06 09:39:31 3539: 2018/08/06 09:38:49.476652 [system] result: exit status 32, mount: /dev/loop2 is already mounted or /dotmesh-test-pools/513130633933877-ebb659b0/k8s-cluster-513130633933877-0-node-2/pods/38b78f83-995c-11e8-a1fa-02f42d51d1a0/volumes/dotmesh.io~dind/pvc-38b6dafe-995c-11e8-a1fa-02f42d51d1a0 busy
Aug 06 09:39:31        /dev/loop2 is already mounted on /dotmesh-test-pools/513130633933877-ebb659b0/k8s-cluster-513130633933877-0-node-2/pods/38b8b500-995c-11e8-a1fa-02f42d51d1a0/volumes/dotmesh.io~dind/pvc-38b6dafe-995c-11e8-a1fa-02f42d51d1a0

https://gitlab.dotmesh.com/dotmesh/dotmesh-sync/-/jobs/99819

Also https://gitlab.dotmesh.com/dotmesh/dotmesh-sync/-/jobs/99818

Done!


Fix initial Android support

Add force touches.
Fix UI focus system and touch screen system.
Fix Moved event.
Fix examples README.
Update rodio to 0.13 with Android support.
Get rid of supported_android_features as all features are now supported.
Add Android build CI with cargo-apk.
Example in bevy works well:

I think its worth trying to squeeze this in to the release. testing on all platforms now (to be safe)

UI touch "kind of" works for me now. If i "press" a button it stays pressed until i touch something else.

I think its probably good enough for now though.

Yes. It's still a lot to enhance and rework. But at least it works. I even did a paint app


problem installing new radiusdesk script with ansible

Hi.. im trying to installing the new script that you have published...
i want to install in localhost, i put in ssh "ssh localhost" i write yes and i put the password..
but when i try to install he show me this error
http://prntscr.com/a89882

What user are you using to run the installer, is it the root user or a regular user? The installer assumes you are running as root, there is a patch that will be pushed within the weekend but until then, make sure you are logged in as the root user on your control machine/host

im using root

Right now, I am trying to recreate the problem but it seem to be working fine. Just an aside, do you have Python 2.7+ installed?

Python 2.7.6

@j0r1993 I have push a fix for this issue after I finally reproduced the bug. You can clone a fresh copy of the installer or just edit your servers file and ensure the following change:
[default]
localhost

Becomes:
[default]
localhost ansible_connection=local ansible_ssh_user=root

now work.. but there is 1 error i show you
http://prntscr.com/a8c6rb

Can you try updating your Ansible version from 1.5.x to the recent one? Follow this guide and after installing the updated Ansible, run this the installer again and get back to us.

updated but same error..

there isn't no radiusdesk folder installed
http://prntscr.com/a8chb2

Looks like the installation did not complete that's why. Are you still getting the error after the update of Ansible?

yes
TASK [radiusdesk : Downloading RadiusDesk CakePHP Source from Subversion (Ubuntu)] ***
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: AttributeError: 'int' object has no attribute 'startswith'
fatal: [localhost]: FAILED! => {"changed": false, "failed": true, "parsed": false}

I have created a new bugfix branch and kindly test and revert back.
To checkout the bugfix branch, just do the following:
git checkout -b bugfix_18 http://github.com/muffycompo/radiusdesk-installer

work!!.. all installation passed without error!! you are fantastic :D thanks for all you doing for us

I am glad I could help fix it for you. I will merge the changes to the master branch to ensure others don't have the same problem and thank you too for trying it out because without your feedback, this fix will have not been implemented.

Dear muffy i am facing below this issue when installing on centos 7 on cloud
TASK [radiusdesk : Downloading RadiusDesk CakePHP Source from Subversion (RHEL/C                                                                                entOS 6+)] ***
fatal: [localhost]: FAILED! => {"changed": true, "cmd": "svn checkout -r 1951 ht                                                                                tp://svn.code.sf.net/p/radiusdesk/code/trunk/rd_cake /tmp/rd_sources/rd_cake", "                                                                                delta": "0:00:00.674130", "end": "2017-02-19 07:58:07.820619", "failed": true, "                                                                                rc": 1, "start": "2017-02-19 07:58:07.146489", "stderr": "svn: E175002: Unable t                                                                                o connect to a repository at URL 'http://svn.code.sf.net/p/radiusdesk/code/trunk                                                                                /rd_cake'\nsvn: E175002: OPTIONS of 'http://svn.code.sf.net/p/radiusdesk/code/tr                                                                                unk/rd_cake': could not connect to server (http://svn.code.sf.net)", "stdout": "                                                                                ", "stdout_lines": [], "warnings": ["Consider using subversion module rather tha                                                                                n running svn"]}
RUNNING HANDLER [radiusdesk : Start Nginx] *************************************
RUNNING HANDLER [radiusdesk : Restart Nginx] ***********************************
RUNNING HANDLER [radiusdesk : Start PHP-FPM] ***********************************
RUNNING HANDLER [radiusdesk : Restart PHP-FPM] *********************************
to retry, use: --limit @/root/radiusdesk-installer/rd-installer-ansible.                                                                                retry
PLAY RECAP *********************************************************************
localhost                  : ok=14   changed=9    unreachable=0    failed=1
[root@iberrycloud radiusdesk-installer]# cd..

@iberywifi, the issue is with sourceforge, I tried to reproduce the error and I had the same thing, trying it manually also yielded the same results, it times out when doing an "svn checkout" of radiusdesk trunk. Hopefully it gets fixed and the installer should work as normal.

Is there any other way to install other than from sourceforge.
On Mon, Feb 20, 2017 at 3:30 PM, Mfawa Alfred Onen<EMAIL_ADDRESS>
wrote:

@iberywifi, the issue is with sourceforge, I tried to reproduce the error
and I had the same thing, trying it manually also yielded the same results,
it times out when doing an "svn checkout" of radiusdesk trunk. Hopefully it
gets fixed and the installer should work as normal.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/muffycompo/radiusdesk-installer/issues/18#issuecomment-281035880,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYroyindFw01WO07ubeRMVasIk-bB6E5ks5reWQ5gaJpZM4Hj_qE
.


Add support for BIP 70

Fields of interest:

(general) buttonData: a JSON string, passed along to the server upon a completed payment
clientIdentifier: a hash of the client ID / app, passed back to the server upon a completed payment


Better Documentation for Types

Add more useful docs to module roots in the types crate. This should include examples of how to define types with default mappings, create custom mapping etc.
Base the docs around what users are likely going to want to do, and provide alternatives and details on the method docs themselves.

Once #59 is implemented, there should be enough to cover some standard scenarios in the docs.

Waiting on #65

This is improving in general along with the module design

Pick this issue up again once the core types are implemented so the docs can be worked on as a whole.

The state of the types documentation is much better now.
Examples of building types with compiler plugins will be added as they're done as per normal.


Add executable permissions to Cocoapods and docs publishing

Travis Deploy phase failed on the Cocoapods publishing step with error code 127. This is probably due to the fact that there were no executable permissions on release-to-cocoapods.sh. Same for publish-documentation.sh.

:tada: This PR is included in version 0.37.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Don't allow :s delimiters not allowed by vim

Instead of the '/' which surrounds the pattern and replacement string,
you can use any other single-byte character, but not an alphanumeric
character, '', '"'' or '|'.

(Vim docs)
I'm open to suggestions on how to improve the code!

Looks fine. Thanks!