Completion is not working with zsh on macOS

Maybe this is related to #226 ?
$ pipsi install pipenv
Running virtualenv with interpreter /Users/markus/.local/venvs/pipsi/bin/python3.6
Using real prefix '/usr/local/var/pyenv/versions/3.6.1'
New python executable in /Users/markus/.local/venvs/pipenv/bin/python3.6
Also creating executable in /Users/markus/.local/venvs/pipenv/bin/python
Installing setuptools, pip, wheel...done.
Collecting pipenv
Collecting pew>=0.1.26 (from pipenv)
  Using cached pew-0.1.26-py2.py3-none-any.whl
Requirement already satisfied: pip in ./.local/venvs/pipenv/lib/python3.6/site-packages (from pipenv)
Collecting virtualenv (from pipenv)
  Using cached virtualenv-15.1.0-py2.py3-none-any.whl
Collecting pythonz-bd>=1.10.2 (from pew>=0.1.26->pipenv)
  Using cached pythonz_bd-1.11.4-py3-none-any.whl
Collecting virtualenv-clone>=0.2.5 (from pew>=0.1.26->pipenv)
Requirement already satisfied: setuptools>=17.1 in ./.local/venvs/pipenv/lib/python3.6/site-packages (from pew>=0.1.26->pipenv)
Collecting resumable-urlretrieve; python_version == "3.6" (from pythonz-bd>=1.10.2->pew>=0.1.26->pipenv)
  Using cached resumable_urlretrieve-0.1.5-py2.py3-none-any.whl
Collecting requests (from resumable-urlretrieve; python_version == "3.6"->pythonz-bd>=1.10.2->pew>=0.1.26->pipenv)
  Downloading requests-2.18.2-py2.py3-none-any.whl (88kB)
    100% |████████████████████████████████| 92kB 1.7MB/s
Collecting certifi>=2017.4.17 (from requests->resumable-urlretrieve; python_version == "3.6"->pythonz-bd>=1.10.2->pew>=0.1.26->pipenv)
  Using cached certifi-2017.4.17-py2.py3-none-any.whl
Collecting urllib3<1.23,>=1.21.1 (from requests->resumable-urlretrieve; python_version == "3.6"->pythonz-bd>=1.10.2->pew>=0.1.26->pipenv)
  Downloading urllib3-1.22-py2.py3-none-any.whl (132kB)
    100% |████████████████████████████████| 133kB 3.4MB/s
Collecting idna<2.6,>=2.5 (from requests->resumable-urlretrieve; python_version == "3.6"->pythonz-bd>=1.10.2->pew>=0.1.26->pipenv)
  Using cached idna-2.5-py2.py3-none-any.whl
Collecting chardet<3.1.0,>=3.0.2 (from requests->resumable-urlretrieve; python_version == "3.6"->pythonz-bd>=1.10.2->pew>=0.1.26->pipenv)
  Using cached chardet-3.0.4-py2.py3-none-any.whl
Installing collected packages: certifi, urllib3, idna, chardet, requests, resumable-urlretrieve, pythonz-bd, virtualenv, virtualenv-clone, pew, pipenv
Successfully installed certifi-2017.4.17 chardet-3.0.4 idna-2.5 pew-0.1.26 pipenv-5.1.2 pythonz-bd-1.11.4 requests-2.18.2 resumable-urlretrieve-0.1.5 urllib3-1.22 virtualenv-15.1.0 virtualenv-clone-0.2.6
  Linked script /Users/markus/.local/bin/pipenv
Done.

$ pipenv --version
pipenv, version 5.1.2

$ _PIPENV_COMPLETE=source pipenv
Traceback (most recent call last):
  File "/Users/markus/.local/venvs/pipenv/lib/python3.6/site-packages/pipenv/vendor/click_completion.py", line 409, in get_auto_shell
    import psutil
  File "/Users/markus/.local/venvs/pipenv/lib/python3.6/site-packages/pipenv/vendor/psutil/__init__.py", line 136, in <module>
    from . import _psosx as _psplatform
  File "/Users/markus/.local/venvs/pipenv/lib/python3.6/site-packages/pipenv/vendor/psutil/_psosx.py", line 14, in <module>
    from . import _psutil_osx as cext
ImportError: cannot import name '_psutil_osx'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/markus/.local/bin/pipenv", line 11, in <module>
    sys.exit(cli())
  File "/Users/markus/.local/venvs/pipenv/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/Users/markus/.local/venvs/pipenv/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 692, in main
    _bashcomplete(self, prog_name, complete_var)
  File "/Users/markus/.local/venvs/pipenv/lib/python3.6/site-packages/pipenv/vendor/click_completion.py", line 297, in _shellcomplete
    echo(get_code(prog_name=prog_name, env_name=complete_var))
  File "/Users/markus/.local/venvs/pipenv/lib/python3.6/site-packages/pipenv/vendor/click_completion.py", line 387, in get_code
    shell = get_auto_shell()
  File "/Users/markus/.local/venvs/pipenv/lib/python3.6/site-packages/pipenv/vendor/click_completion.py", line 415, in get_auto_shell
    raise click.UsageError("Please explicitly give the shell type or install the psutil package to activate the"
click.exceptions.UsageError: Please explicitly give the shell type or install the psutil package to activate the automatic shell detection.

$ _PIPENV_COMPLETE=source-zsh pipenv

#compdef pipenv
_pipenv() {
  eval $(env COMMANDLINE="${words[1,$CURRENT]}" _PIPENV_COMPLETE=complete-zsh  pipenv)
}
if [[ "$(basename ${(%):-%x})" != "_pipenv" ]]; then
  autoload -U compinit && compinit
  compdef _pipenv pipenv
fi

But it's not completing anything ...
$ eval $(_PIPENV_COMPLETE=source-zsh pipenv)
$ pipenv i<Tab>
-- file --
Applications/             Downloads/                Movies/                   VirtualBox\ VMs/

I have the same issue on macOS, but it works in a created virtual environment:
$ pipenv check
check      -- Checks PEP 508 markers provided in Pipfile.
install    -- Installs provided packages and adds them to...
lock       -- Generates Pipfile.lock.
run        -- Spawns a command installed into the...
shell      -- Spawns a shell within the virtualenv.
uninstall  -- Un-installs a provided package and removes it...
update     -- Updates pip to latest version, uninstalls all...

BTW, It's really slow. I guess it requires Python running to provide a complementation?

it does.

do you have requests[security] installed in the same virtualenv? that'd slow things down a lot.

That's why I recommend installing pipenv w/ pipsi.

Not working here at all. :(

@kennethreitz I really have requests installed, but I cannot image how could requests slow down shell autocomplimentation.
I have another doubt -- when I did source pipenv in zsh, I got the full help doc of ImageMagic! And ends with:
import: delegate library support not built-in '' (X11) @ error/import.c/ImportImageCommand/1294.
from: can't read /var/mail/pipenv
/usr/local/bin/pipenv: parse error near `)'

What may this be related to?

@kxxoling but do you have requests[security] installed? (e.g. pyopenssl?) it vastly slows down import time of requests.

The library being used for completion is here:
https://github.com/click-contrib/click-completion

@kennethreitz I didn't install requests[security] literally, but pyOpenSSL, cryptography and idna are all installed. Maybe they are required by other dependencies.

If you're using Python 2.6 I believe pipenv will install requests[security] automatically. Anyway, this is why you're seeing such slow responsiveness.

@kennethreitz I'm using Python 2.7.13 actually. But I still don't understand why would requests[security] slows down pipenv autocomplete. Could this be optimised?
I tried eval "$(_PIPENV_COMPLETE=source-zsh pipenv)" in a Python 3.6 environment, the responsiveness is much faster and acceptable now.

This is working for me, at least on my Linux machine.

@tilgovi what shell and command are you using?

In both cases it's zsh. On the Mac it's ZSH with homebrew. Dotfiles are the same (sync'd through git). Works fine on Linux but not MacOS, for some reason.

Ooh, wait. I was forgetting to quote it on my Mac.
eval "$(_PIPENV_COMPLETE=source-zsh pipenv)"
... yields "basename: illegal option -- z"

@tilgovi I thought you said it was working?

@tilgovi try removing the quotes ...

BTW.: It works now for me (outside and inside a virtual env) but I have no clue why...

@kennethreitz Sorry to revive a closed issue, but I have the exact same problems using pipenv with zsh (both installed through Homebrew) on macOS: Doing eval "$(pipenv --completion)" gives me:
basename: illegal option -- z
usage: basename string [suffix]
       basename [-a] [-s suffix] string [...]

Interestingly, the completion for pipenv works after I do that manually in the shell. If I put the eval in my .zshrc I don't get the error, but I also don't get the completion... :(

Ok, I found a workaround which "works for me":tm:, but expects oh-my-zsh to be used:
pipenv --completion >> .oh-my-zsh/custom/pipenv_completion.zsh

All files there get auto-loaded on zsh startup. Bonus: It's faster since pipenv does not need to be executed on shell startup. I'm sure there are standard/non-oh-my-zsh places that can be used instead, but my vanilla-zsh-fu is not strong enough for this.


Replicate content in Telegram

This has been implemented with IFTTT, outside the scope of the code in this repo. Closing.


Use lazy task configuration in gradle/gradle buildSrc's integration-testing, kotlin-dsl, packaging, and performance subprojects

Expected Behavior

Current Behavior

Context

Steps to Reproduce (for bugs)

Your Environment

Build scan URL:

PR: https://github.com/gradle/gradle/pull/5618


AtlasTextures don't work with Patch9Frames (and related GUI skinning)

Tested on Godot 2.0.3 and 66d513bb2ab86ab75d727cce031cfb226501cab4, Windows 10 64 bits
My GUI is skinned from a single atlas image, so I want to use AtlasTextures in my theme.
However it doesn't respects the texture region :(
This is the case on every GUI element that can be skinned using the 9-patch approach, and it forced me to use separate textures everytime.
Is this intented, or is it a bug?
Example project:
Patch9FramesAtlas.zip

What's the status on this in 2.1? IIRC there was work done on AtlasTextures and Patch9Frames.

I'm not sure, though AtlasTextures are still a bit weirdly handled by the engine. (e.g. see https://github.com/KOBUGE-Incubator/smart-terrain-plugin/blob/master/lib/smart_texture_helpers.gd#L23-L29)

2.1.5 - still not working
3.1.1 - does not work if created through the scene itself. But it works if you make it local (which means that it is necessary that AtlasTexture be local by default). It also works if you first create AtlasTexture as a resource in the file system, and then apply it to the scene.


Fix ClientPayloadError while reading

Closes https://github.com/fsspec/s3fs/issues/786
The _call_and_read pattern was already implemented for _cat_file in https://github.com/fsspec/s3fs/pull/601 so I'm just applying this pattern again. I don't know the library well enough to judge if this can be refactored somehow else but I would prefer to separate fix from the refactor

There were no tests added in https://github.com/fsspec/s3fs/pull/601 so I don't know how to proceed. If there are any suggestions on how to implement tests, I'm happy to follow up

I would prefer to separate fix from the refactor

That is fine

If there are any suggestions on how to implement tests, I'm happy to follow up

Unfortunately, there isn't a way I know of to get the moto server or any other mock to cause the case you were seeing. Se, as long as this doesn't break anything, but it fixes your workflow, we should be good.

@martindurant can you enable the workflow for CI, please?

I wonder if there's anything we can do about a retry wrapping around _call_s3, which itself has retries. Once the read on the body fails, I don't know whether it's possible/useful to try to repeat just that part.

As it is right now, if the read on the body fails, the entire request is repeated. I think this is a requirement since I doubt the body is still valid after such a failure.
The fact that there is a duplicated retry in case of an exception in _call_s3 is unfortunate but I don't think this is a big deal. By default, this is retrying for 5s with a backoff of 1.7**i * 0.1, i.e. a full retry cycle take 1.89s + request time. Doing this again before failing is fine. I would also like to point out that this is the exact way _cat_file is doing it.
I see three options

Keep it as is and merge since this pattern is already used in _cat_file.
add a toggle to _call_s3 that allows one to disable retries. Simple and I would be fine doing this in this PR. However, I don't think we'll be doing the code base a favor.
Refactor the entire retry mechanism, likely remove it from _call_s3 and find a different abstraction/entry point that works better.

is definitely out of scope since I don't know the code base well enough to do this. This would likely require a holistic view on the code base and we'd need to review all _call_s3 invocation to decide where and why a retry is appropriate. I suspect there are more patterns that should be retried as a whole but I don't know. I encourage this kind of refactoring but I am the wrong person for it.

@martindurant is it possible to get a patch release with this once merged?

OK, I will merge this as it is, and consider refactoring in the future.
Thanks for contributing!
A release should be out in the next few days.


Oshioke

clashing files

have you updated ur repo so I can clone it
On Thu, Sep 9, 2021 at 2:09 PM Ehis Edemakhiota @.***>
wrote:

clashing files
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ehizman/eden/pull/8#issuecomment-916077040, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ATN3FKNVNPNRYT7EHJHSTFLUBCWZ7ANCNFSM5DW4DAYQ
.


Shade com.google.thirdparty

Guava 16 introduced packages starting with com.google.thirdparty which need to be shaded as well. I am augmenting all current shadings of Guava regardless of version to prevent future issues.
@ssalinas @jhaber

Thanks @stevegutz LGTM


KUBE-5984 - ensure that scale-ups always occur when there are starved pods

This fixes https://github.com/atlassian/escalator/issues/224

This solves #224

If you want this to close the issue, you'll need to use the keywords from here - https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword, e.g. closes/fixes/resolves etc.

Would be good to see in the description of this PR what the change does to fix the issue - I had to look through the code to see how it addresses the original issue

LGTM - thanks for the contribution!


Update tutorials

This PR resolve issues #1057 , #1058  and #995

@raphaeltimbo , I noticed very different results and plots than the previous ones, I'm pushing the before/after below:
Tutorial part 2 (Tutorial - Running Analyzes) in Rotor model:

Campbell Diagram:

Bode Plot:

This branch had several conflict issues, we will create a new solution.


Please disable feature runtime-rng by default

We introduce aHash  in our database system. By default, this crate will generate different hash results in distributed nodes, which is disastrous.
So I propose disabling the feature runtime-rng by default.

why was this issue closed when there are no PR:s that solve the issue?

Can you do default-features = false to get rid of it?
There is already a way to use ahash with fixed seeds in order to guarantee stable hashes.
Therefore I suppose it is close because RTFM.

Can you do default-features = false to get rid of it?
There is already a way to use ahash with fixed seeds in order to guarantee stable hashes.
Therefore I suppose it is closed because RTFM.

wouldn't it be more appropriate then to label it as "not planned" rather than "closed"? Because "closed" suggests that an actual solution has been implemented in response to the issue (rather than a pre-existing solution making the issue unnecessary)


[ESP32]Add CI for ESP32C3

Problem
CI need to be add since C3 support has be added for all-clusters-app
Change overview
Add CI for ESP32C3-DevKitM
Testing
Successful CI

@andy31415 Can you please updated the docker hub? we have update ESP-IDF version to v4.3 in #6345 .

@saurabhst @msandstedt @Damian-Nordic @jepenven-silabs @mspang @andy31415 ?


[fix][broker] Release the entry in getEarliestMessagePublishTimeOfPos.

Motivation
Release the entry after using it in getEarliestMessagePublishTimeOfPos.
Documentation

[x] doc-not-needed
(Please explain why)

Hi @Technoboy-
Looks like this problem not exist at branch-2.9 could you help to review it?

Hi @Technoboy- Looks like this problem not exist at branch-2.9 could you help to review it?

yes, right. Removed related label.


Farewell to Acorn.

One of the most uncomfortable things right now for me is the existence of acorn in the renderer.
Literally, the only place where acorn is used to is to generate something what we call a statementTypeMap https://github.com/abelljs/abell-renderer/blob/main/src/compiler.js#L38
Statement Type Map is an array of the types of statements we have in the code. This is eventually used to determine if the last statement is an assignment or expression.
E.g.
const a = 3;
a + b;
b = 9;

Will output-
['VariableDeclaration', 'ExpressionStatement', 'AssignmentExpression']

Why?
JavaScript returns the value when you assign. So if you run a = 3 in console, it will return 3, And so does only writing 3, And so does writing 2 + 1.
And in vm module, we only get what the code outputs.
But hey, how do we know if it is 3 coming from assignment or 3 coming from expression? We use the statement type maps!
So when it is an assignment, we don't return a value and thus printing a blank string.
Why acorn is a problem?
Using a whole frikin JavaScript parser just to tell if the code has assignment in the end is an overkill!
Solution
Probably write a custom tokenizer so instead of doing the whole parsing, we can just tokenize and figure out the statement type map from tokens.
Performance Improvements-
Whole parsing takes around 27ms for 3 lines of js code. Whereas tokenizing can take 10ms. This difference will be much larger for larger scripts.

I just did the performance check and apparently the 27ms are just for the initial parsing but after that every other parsing happens in around 2-3ms so even moving away from acorn won't have much of the performance boost. I think this is something that we can tackle later in the future when saving few ms are important to performance.


Xanadu

source: https://github.com/opentecture/mindmapping/issues/31
Xanadu / Ted Nelson

https://en.wikipedia.org/wiki/Ted_Nelson
https://en.wikipedia.org/wiki/Project_Xanadu

Original 17 rules

Every Xanadu server is uniquely and securely identified.
Every Xanadu server can be operated independently or in a network.
Every user is uniquely and securely identified.
Every user can search, retrieve, create and store documents.
Every document can consist of any number of parts each of which may be of any data type.
Every document can contain links of any type including virtual copies ("transclusions") to any other document in the system accessible to its owner.
Links are visible and can be followed from all endpoints.
Permission to link to a document is explicitly granted by the act of publication.
Every document can contain a royalty mechanism at any desired degree of granularity to ensure payment on any portion accessed, including virtual copies ("transclusions") of all or part of the document.
Every document is uniquely and securely identified.
Every document can have secure access controls.
Every document can be rapidly searched, stored and retrieved without user knowledge of where it is physically stored.
Every document is automatically moved to physical storage appropriate to its frequency of access from any given location.
Every document is automatically stored redundantly to maintain availability even in case of a disaster.
Every Xanadu service provider can charge their users at any rate they choose for the storage, retrieval and publishing of documents.
Every transaction is secure and auditable only by the parties to that transaction.
The Xanadu client–server communication protocol is an openly published standard. Third-party software development and integration is encouraged.

http://ted.hyperland.com/

All the above notes are in:
https://opentecture.github.io/mindmapping/#graphql-3d/README.md
There is nothing here that needs acting on for now


updates

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@abbfbcf). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #5   +/-   ##
=========================================
  Coverage          ?   73.85%
=========================================
  Files             ?        4
  Lines             ?      306
  Branches          ?        0
=========================================
  Hits              ?      226
  Misses            ?       80
  Partials          ?        0

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 abbfbcf...44770ae. Read the comment docs.


Change the type of window.orientation to number

fix issue #3884

Should it be number or string | number?

I didn't find anywhere except the IE spec that defines window.orientation as string, it seems that in most cases (iOS, Android) it is used as number. But I agree that as it is an unstable property, it might be better to use string | number

I can confirm setting the value to string | number allows my orientation checks to compile without errors. Thank you. My problem now has nothing to do with you guys but with the Visual Studio Code team, who have their own lib.d.ts, etc. as resources in the app package instead of using TypeScript's versions. Atom seems fine. I'm just casting the orientation numbers to <any> for now until TypeScript 1.6 and VSCode 0.6 are released.
Finally, I'd like to say thank you for all the hard work you guys are putting in to making TypeScript so awesome. I absolutely love TypeScript :+1:

:+1:
@chocolatechipui for the record, you can change your user settings in VS Code and set "typescript.tsdk" to a custom location. If you install our nightly (npm install -g typescript@next), you can point to that version of TypeScript's bin folder.

:+1:

@DanielRosenwasser Thanks. That tip for VSCode made my day :100:


bug

if i run the code python opens a new tab named main and i get half of screen i cannot see full screen so i cant run jarvis

Hello @DTXGAMERYT
Actually, it's not a bug, the application is not responsive, and the resolution of the screen is made according to my screen, for the small screen it may enlarge, So if you are interested you can make it responsive and I'm happy to accept the PR.
Thank you.

can u help me where can i change resolution?

You can change that from JarvisUI.py Lines:17.

@DTXGAMERYT  Closing this issue, if you still have any other issue, please feel free to open a new issue.


Add migration to add projectId column in the jobTestReports model and upsert projectId to all existing jobTestReports

https://github.com/Shippable/pm/issues/9648

migration script is successfully run on rc


SSR rendering

This might be related to my setup (running with next.js) but I am having issues rendering Tablet on the server. Has anyone had any success with SSR here?

this was an issue with my css loader, resolved

Hey man. Sorry to dig this up, but I'm having troubles configuring my loader with Next too. Would you mind sharing your relevant settings please ? Thanks 🙏


Fine-tune lora errors with invoked with: array?

Hi,
I've setup everything well with the requirements for lora fine-tune.
mlx==0.0.7
and others also installed well.
I've used the well converted model with quantized but, after i put
python lora.py --model ./mlx_model --train --iters 600

and got an errors below
Traceback (most recent call last):
  File "/Users/soonde17/anaconda3/envs/mlx/mlx_0.0.7/mlx-examples/lora/lora.py", line 324, in <module>
    train(model, train_set, valid_set, opt, loss, tokenizer, args)
  File "/Users/soonde17/anaconda3/envs/mlx/mlx_0.0.7/mlx-examples/lora/lora.py", line 222, in train
    for it, batch in zip(
  File "/Users/soonde17/anaconda3/envs/mlx/mlx_0.0.7/mlx-examples/lora/lora.py", line 191, in iterate_batches
    batch_arr[j, : lengths[j]] = batch[j]
TypeError: __array__(): incompatible function arguments. The following argument types are supported:
    1. (self: mlx.core.array) -> numpy.ndarray

Invoked with: array([1, 2401, 28747, ..., 266, 28742, 2], dtype=int64), dtype('int32')

after i've got above errors, i've tried models that mlx team uploaded on hf but, got a same error messages like above.
model is the mlx-community/Mistral-7B-Instruct-v0.2 btw.
Please let me know how should i fix that.
Thank you.

Looks like a bug from the old example,  would you please pull the latest main branch. It should fix the issue


Bug: Missing dist folder in 2.0.32 npm build

https://www.npmjs.com/package/vue3-google-login/v/2.0.32?activeTab=code

I'm having same problem 👍🏻

Same! Builds are breaking

How does that even happen?!

@escral @mertcelen @pojdrovic @Firu115 Sorry for the trouble guys, I was trying to automate the build process, silly me forgot to add npm run build to the YML file 😟
Fixed this in the latest version 2.0.33

We forgive you!


Can we use actual html element or even "ref" for the scrollTo target?

From the examples, it seems we always need to pass in a string of the target HTML ID?
If this is not true, can you please update the docs so that it's not a hunting show when trying to figure this out?
Thanks

https://github.com/rigor789/vue-scrollto/blob/197b365e2914b0e47470d1d10ffc6bfc127a4151/src/utils.js#L13-L18
is what resolves the target - I guess passing an html element, or a ref would also work, but I have not tested this directly (across different browsers).
I don't currently have the capacity to work on this, but if you can test it across browsers, feel free to do so, and let me know how it goes.

Thanks! Unfortunately, I don't have space currently for a complete test, but I'll leave feedback when I do.
For the moment I'm just generating dynamic ids for my elements and that works like a charm.

I'm using this.$scrollTo(this.$refs['myReference']) just fine.  this.$refs.myReference works too, but is giving me an argument warning in WebStorm.


Logging fabio-output to log file

Hi guys,
is there a possibility to write the console output from fabio to a log file?
The only way I found so far was logging the access  to stdout, but not a file.

No. This should be handled by your supervisor which does the log file redirection.
—
Frank Schröder

On 28. Jun 2018, at 09:46, Matthias Rosenauer<EMAIL_ADDRESS>wrote:
Hi guys,
is there a possibility to write the console output from fabio to a log file?
The only way I found so far was logging the access to stdout, but not a file.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@Roseidon how are you running fabio today?

Closing due to lack of response.  Future users - your system should be more than capable of taking the stdout of fabio and writing it to a system log.  Check the systemd/docker/upstart documentation for more information.


Fixes the daily functional tests

Updated and consolidated test framework packages.

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll

Pull Request Test Coverage Report for Build 63229

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 71.155%

Totals

Change from base Build 63131:
0.0%

Covered Lines:
4344

Relevant Lines:
6105

💛  - Coveralls

Pull Request Test Coverage Report for Build 63232

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 71.155%

Totals

Change from base Build 63131:
0.0%

Covered Lines:
4344

Relevant Lines:
6105

💛  - Coveralls


Migrate completely custom sections from the admin to declarative approach.

Summary
Migrate completely custom sections from the admin to declarative approach.
Added custom component too, to easy add custom widgets to the admin when needed.
Added migration of logging section.
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed
[x] Added or updated unit tests (required for all new features)

@jespino feel free to merge this when ready


[resotocore][fix] Only use config roots to validate

Code of Conduct
By submitting this pull request, I agree to follow the code of conduct.

Codecov Report
Patch coverage: 100.00% and no project coverage change

Comparison is base (8f65b5e) 89.61% compared to head (ddab7d3) 89.61%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1492   +/-   ##
=======================================
  Coverage   89.61%   89.61%
=======================================
  Files          74       74
  Lines       11818    11819    +1
=======================================
+ Hits        10591    10592    +1
  Misses       1227     1227

Flag
Coverage Δ

resotocore
89.61% <100.00%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...tocore/resotocore/config/config_handler_service.py
88.69% <100.00%> (+0.06%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fast tap or "tap, hold, drag" on "< Back" (back link) not always responding

First of, thanks for this amazing project!!
I do however experience a small issue. In the simulator/browser everything works fine when clicking the "< Back" (back link). On a device however "< Back" does not always respond:

when tapping it fast;
when you tab, hold and drag.

It is clearly visible that in both scenario's the "active-state" class gets applied, you can see the opacity changing (to 0.3) but the page is not going back as expected.
In iOS however, as long as you start you "touchstart" on a "< Back" link, no matter if you make a drag movement afterwards, or tap it really fast, you always go back.
Framework7 v1.0.2
iPhone 6, iOS 8.1.3 (real device)
Deployed the "/dist" folder from the master to the device (PhoneGap).

you may try to increase fastClicksDistanceThreshold app parameter

Thanks, I'll look into it and let you know if it fixed things for me.

That worked for me. I am thinking the default for it should be more like 20 and not 0.
I have downloaded many F7 apps, and they all seem to have that problem on retina devices (maybe because movement is more common on denser screens)

Likely related to #444

Time has passed and #444 was merged


OpenStack: empty/unprovided attributes are always "added" and thus seen as required change

Certain(?) lists/maps are always added if empty:
Terraform detected the following changes made outside of Terraform since the last "terraform apply":

  # module.kubermatic.kubermatic_cluster.renku has been changed
  ~ resource "kubermatic_cluster" "test" {
        id                 = "abcd1234"
      + labels             = {}
        name               = "test"
      + sshkeys            = []
        # (6 unchanged attributes hidden)

        # (1 unchanged block hidden)
    }
  # module.kubermatic.kubermatic_node_deployment.renku has been changed
  ~ resource "kubermatic_node_deployment" "test" {
        id                 = "test-nodedeployment"
        name               = "test-nodedeployment"
        # (5 unchanged attributes hidden)

      ~ spec {
            # (2 unchanged attributes hidden)

          ~ template {
                # (1 unchanged attribute hidden)

              ~ cloud {
                  + bringyourown = {}

                    # (1 unchanged block hidden)
                }

                # (2 unchanged blocks hidden)
            }
        }
    }

It doesn't matter whether I define these lists and maps or not, they're always being added as long as they're empty.  I figure the provider should ignore empty lists/maps (and surely not add some that are not even defined in the manifest) but what really matters is how the KKP API treats empty lists/maps.

Uhm...now that I opened the ticket, I can't reproduce it anymore. Maybe it only happens once after initial cluster/nod_deployment creation? Not sure, but closing this for now.


Improve resource deallocation in FUSE

What changes are proposed in this pull request?
This improves resource deallocation in two ways:

Close FileSystemContext
Close an optional thread pool on normal exit (before it is only destroy on error)

Why are the changes needed?
Resource deallocation

alluxio-bot, merge this please


Added greeting trigger, PM trigger, link title lookup trigger

Additions:
-Greeting trigger AKA Doormat functionality: When a person enters a groupchat, he is greeted with "Hello Name".
-Link title lookup trigger: Posted links are responded to with the page title.
-PM trigger: Basically ChatReplyTrigger, but responds via private message to reduce spam.
Take my code with a grain of salt, I'm an amateur. Which is probably why I needed the PM Trigger and some additions in chatBot and baseTrigger. Maybe you can think of a better way to implement that.
That's also the reason I messed up the loop, apparently. (According to codacy) :I But you get the idea.
Thank you for your hard work guys, this bot is amazepants.
p.p.S.: Pulling again didn't help, apparently I had to push to my dev branch instead of master. Don't ask why :v Seems to be compatible now.

@Divinux please see 63ab0a0f646da08fee3b30334fb70d62f1da86a6. I've removed chatPmTrigger and replaced it with an option in chatReplyTrigger. Does this do what you need?

@Efreak Oh, this is so much neater. Nice work.

Cool. When I get around to it, I'll probably add something like this to baseTrigger to make this global across all triggers.


SQLSTATE[42S22]: Column not found: 1054 Unknown column '0' in 'field list'

Please help
{"draw":4,"recordsTotal":0,"recordsFiltered":0,"data":[],"error":"Exception Message:\n\nSQLSTATE[42S22]: Column not found: 1054 Unknown column '0' in 'field list' (SQL: select count(*) as aggregate from (select panichd_tickets.id, panichd_tickets.created_at, panichd_tickets.subjectassubject, panichd_tickets.hiddenashidden, panichd_tickets.contentascontent, panichd_tickets.interventionasintervention, panichd_tickets.status_idasstatus_id, panichd_statuses.nameasstatus, panichd_statuses.colorascolor_status, panichd_priorities.colorascolor_priority, panichd_categories.colorascolor_category, panichd_tickets.start_dateasstart_date,  0-CONVERT(date_format(panichd_tickets.start_date, '%Y%m%d%h%i%s'), SIGNED INTEGER) as inverse_start_date, CASE panichd_tickets.limit_date WHEN NULL THEN 0 ELSE 1 END as has_limit, panichd_tickets.limit_dateaslimit_date,  0-CONVERT(date_format(panichd_tickets.limit_date, '%Y%m%d%h%i%s'), SIGNED INTEGER) as inverse_limit_date, panichd_tickets.limit_dateascalendar, panichd_tickets.updated_atasupdated_at, panichd_tickets.completed_atascompleted_at, panichd_tickets.agent_id, group_concat(agent.name) AS agent_name, panichd_priorities.nameaspriority, panichd_priorities.magnitudeaspriority_magnitude, panichd_members.nameasowner_name, panichd_tickets.user_id, panichd_tickets.creator_id, panichd_tickets.category_id, panichd_categories.nameascategory, group_concat(panichd_tags.id) AS tags_id, group_concat(panichd_tags.name) AS tags, group_concat(panichd_tags.bg_color) AS tags_bg_color, group_concat(panichd_tags.text_color) AS tags_text_color, 0asdeleted_owner, (select count(*) from panichd_attachmentswherepanichd_tickets.id=panichd_attachments.ticket_id) as all_attachments_count, (select count(*) from panichd_commentswherepanichd_tickets.id=panichd_comments.ticket_idandtypein (reply, note, completetx) andtypein (reply, complete, completetx, reopen)) ascomments_count, (select count(*) from panichd_commentswherepanichd_tickets.id=panichd_comments.ticket_idandtypein (reply, note, completetx) andtypein (reply, complete, completetx, reopen) andpanichd_comments.updated_at> 2018-04-07 00:00:00) asrecent_comments_countfrompanichd_ticketsinner joinusersonusers.id=panichd_tickets.user_idinner joinpanichd_membersonpanichd_members.id=panichd_tickets.user_idinner joinpanichd_statusesonpanichd_statuses.id=panichd_tickets.status_idinner joinpanichd_membersasagentonagent.id=panichd_tickets.agent_idinner joinpanichd_prioritiesonpanichd_priorities.id=panichd_tickets.priority_idinner joinpanichd_categoriesonpanichd_categories.id=panichd_tickets.category_idleft joinpanichd_taggablesonpanichd_tickets.id=panichd_taggables.taggable_idandpanichd_taggables.taggable_type= PanicHD\\PanicHD\\Models\\Ticket left joinpanichd_tagsonpanichd_taggables.tag_id=panichd_tags.idwherecompleted_atis null anduser_id= 1 andhidden= 0 anduser_id= 1 andhidden= 0 group bypanichd_tickets.id) count_row_table)"}

Hi, Same problem here.
This thing doesn't work :(
Did you figure out what was going on?

Hi,
Same here  any news about that ?

i don't remember how and if i resolved this problem... :( i'm sorry
Il giorno mer 13 giu 2018 alle ore 17:25 Fab-Miami<EMAIL_ADDRESS>ha scritto:

This project is dead, nobody is answering :(
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/panichelpdesk/panichd/issues/2#issuecomment-396978874,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADiFJdlcOeuCF-YndcfxG8GFzgHB73dHks5t8S7-gaJpZM4TLVhl
.

Hi guys,
@Fab-Miami This project isn't dead. It's just I haven't time to dedicate to it since a few weeks ago. That bug is known to me and It is on my plans to publish a patch for it. I'm sorry I can't tell you right know how to solve it exactly, but I guess I remember it's about an array key in TicketsController, but I'm not sure.
As soon as I can get back into it I'll give more feedback.


Compare by value when memoizing

I ran across a use case where this check makes me lose all benefits of memoization.
I'm pickling a data structure where a couple of fairly long strings are repeated many times, so memoizing it should make the resulting stream a lot shorter.
However, the instances are not considered equal, because they are only equal by value, not by reference. Now this doesn't necessarily matter with Strings, and maybe not for many different classes either.
So an option where the == check is removed, or replaced by a .equals call could be considered for addition in my opinion.

Also, in these cases, the identityHashCode(obj) call would have to be replaced by obj.hashCode.
What do you think?

In your java code, I would expect that these strings were already referencing the same object (in the object graph that you're trying to serialize) ?   Any idea why that is not the case?
Regarding the identityHashcode, this was a deliberate change to improve the pickler's performance. See  #18  I'm hesitant to change it back because it may affect Spark too much

They come from C++ via SWIG, this is why they are different instances. But this should not be important. Created in any way, equal strings (being immutable in Java... - and maybe not just strings, but any other immutable objects as well) should be memoized in a way that the value only occurs once in the stream in my opinion. This could reduce the output size, and maybe improve depickling performance/footprint.

I respectfully disagree here: checking for duplicate objects based on equality is very expensive and will not occur often in practice. So the general case suffers from a possible optimization for certain use cases.  Note that pickle itself, in Python, will also not memoize identical strings if they're not already the same object.
Is there any way you can intern these strings in your java code so that you solve the identity problem that way?

I see your point, and of course, some overhead is unavoidable when checking for equivalence. It is not clear to me however, if it would be slower than computing the hash one additional time (I presume it would have to be computed anyways when inserting into the memo lookup HashMap). Many times when two objects are not equal, the check returns very early, while hash computation has to "go through" the whole content anyways.
Of course, I could easily intern the Strings, but this could be optionally done by Pyrolite as well, so it is readily available the next time someone needs it. Also, since not all objects provide a handy .intern() method, it might not be completely trivial in some cases. Simply adding an option to check for value equivalence when memoizing could work in all cases I can think of right now. And I still think this (minor?) overhead in Java could be worth it considering the stream size and depickling speed. Would have to measure it though.
I don't want to force it at all, I'm just putting my thoughts out. :)

Can you come up with a pull request for this perhaps? That allows us to investigate the performance properties as well.

I sent the pull request, just so you can see what I was talking about. I don't consider all the changes in it to be of high enough quality for merging yet. Let me know what you think!

It's been over a month now. Did anyone have a chance to take a look at my PR?

no, I haven't yet looked at it but thank you for the heads-up. I'll try to evaluate it shortly

@torokati44 well I've looked at the code and I like it! Thanks for taking the time to make it configurable and adding a test example.  I'm kinda surprised by the results to be honest.  The regular hashCode() seems just a tiny bit slower in the worst case scenario than the identityHashcode, and using your changes the pickling speed actually improves a lot in a happy scenario because the pickle stream is a lot smaller.
I'm actually considering getting rid of the identityHashcode now that I learned this... but still, I only did a tiny test

Thanks! Well, without identityHash(), storing and reconstructing the exact same object hierarchy won't be possible of course. I don't need this personally (since the result will still be "equivalent", as far as .equals() is concerned, and for plain data it's perfectly fine), but I thought it was done this way because having this capability was a requirement. In the end, your project, your decision. :)

No, there's no such requirement. The reason to use identityHash was because of the pull request mentioned above where it seemed to give a reasonable performance gain in Spark

@mengxr do you see any issues of reverting back to regular hashCode() in light of the discussion above?

@torokati44 I went ahead and merged your changes. The compare by value is now also the default again for the pickler. When people want to revert to object identity comparison they can do so via the new constructor parameter for the Pickler class

Thank you!
Is it really the default? The inline field initializer indeed says =true, but when the unary constructor delegates to the binary one, it passes the false as valueCompare. IIRC that will override the inline default. This inconsistency can be misleading.
Also, I just now noticed that I misspelled the word 'Also' in the comment of the valueCompare field...

Hmm, I changed it to true?:
https://github.com/irmen/Pyrolite/blob/b05daacfa464fa5d51a0c8b672721b8da26744fc/java/src/main/java/net/razorvine/pickle/Pickler.java#L109

Oh, yes, you did. I was looking at my original commit, sorry.
It's too late around here, and GitHub UI wasn't entirely obvious about the fact that you altered it.
And I guess I can live with that comment typo, it's not the end of the world.


Append text to Queer in AI social event description

Could some please update the queer in AI social event description (on https://virtual.acl2020.org/socials.html) to

The ACL QueerInAI social is an event for queer folks who work in NLP or AI to share their experiences, network with others, and discuss queer issues in research. To facilitate this, the social will involve a mix of mini-testimonials and breakout discussions. All event information will remain confidential to respect the privacy and safety of participants. People of all identities (including allies) are invited to attend!

diff: I've added the last sentence "People of all identities (including allies) are invited to attend!"

@prabhat1081 can you fix this one?

https://github.com/acl-org/acl-2020-virtual-conference-sitedata/pull/152


Pie pan duping

When you uncraft a cooked pumpkin pie into the edible pie and pie pan, you end up with two pie pans rather than 1.

fixed


Vex dialogues not closing properly on Chrome OS X latest

Chrome 44.0.2403.157
If you click on the page as the modal is fading out you can prevent the modal from fully closing. If you continue to click on the page the 'vex-open' class won't be removed from the body and the modal will capture all the click events preventing the user from interacting with the page.
Seems to be chrome only. I tested in latest versions of FF and Safari any they didn't show this behaviour. If you stop clicking around on the page the modal will close and the 'vex-open' class is removed from the body after a couple seconds.
Steps to reproduce:

open vex dialogue in chrome.
close dialogue
click on the page as the modal is closing and continue to click the page.

I'm thinking this is probably a bug with Chrome but thought I would report it anyway incase others are having this issue.

Seems the issue is a bug with Chrome latest stable. I downloaded Canary and it seems the bug has been fixed there. Clicking on the page while the animation is running seems to prevent or reset the 'animationend' event from firing. If that event doesn't fire then the rest of the close code doesn't run.
Since this looks to be fixed in the Version 47.0.2497.0 canary (64-bit) and in Version 46.0.2490.6 dev (64-bit) I wouldn't worry too much about this. IF someone really does need this fixed I'd suggest a monkeypatch to use setTimeout() to manually call the needed close functions and trigger vexClose and vexAfterClose events.


Investigate the deletion of dist/near-api-js.js file

We need to investigate the deletion of near-api-js.js. Should we keep both near-api-js.js and near-api-js.min.js?
According to this discussion, we need to delete near-api-js.js before 1.0 release: https://github.com/near/near-api-js/pull/631

I am working on this issue


About 10% of test runs fail currently due to an event listener never being fulfilled

I'm not certain what the cause of this bug is,
https://github.com/textileio/js-threads-client/commit/19211d608b99cce0214d1add56f3e689a12f571c/checks?check_suite_id=398973899#step:11:57
some possibilities,

[x] check if something wrong with the update created
[x] check if test setup with the async listener isn't right
[ ] check if js-threads-client isn't properly returning events
[ ] check if go-threads isn't firing the event on time or at all

they seem to be failing still, but sources narrowed down a bit

This appears to actually be due to https://github.com/textileio/go-threads/issues/245 which is itself due to https://github.com/improbable-eng/grpc-web/issues/543

Is this still an issue @andrewxhill ?

yep

Definitely try integrating a new threads release. Some bug fixes in there.

just leaving a note, that this is still an issue. @carsonfarmer mentioned a maybe fix in a recent update, but just saw it again.


Need to clearly mentioned about the reCaptcha prompt when user is locked

My suggestion

reCaptcha is reuqired to prompt after Lock user accounts on maximum failed attempts and not required to prompt after Prompt reCaptcha after max failed attempts as after the max failed attempts the account is locked and users cannot login.
But https://is.docs.wso2.com/en/latest/learn/setting-up-recaptcha/ documentation not clearly states that  there is no prompt of reCaptcha after Prompt reCaptcha after max failed attempts
Need to add these information

Related issue
https://github.com/wso2/product-is/issues/13195

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Log in options

I removed the line that was separating the two different log in options and I modified the css so that both options look similar.

Commit: 6a22a2e


feat(page): add omitBackground option for page.pdf method

Adds omitBackground option for Page.pdf to allow for transparent backgrounds. This option is available in the screenshot api, this change unifies this for both .screenshot and .pdf.
Related issues: #6906  #2545

@googlebot I signed it!

Thanks. Looks like the tests are only setup to compare pngs. Any suggestions for alternative ways to get some basic test coverage?

Well if that's not currently set up to handle PDFs, I think we can skip it for now.
But maybe you can create a tracking bug for this on this repository so that we can take a look at it in the future.
Please also link to this pull request from that new issue. Thanks!


[TCGC] bump tcgc 0.46.1

bump tcgc 0.46.1

:x: There is undocummented changes. Run chronus add to add a changeset or click here.
The following packages have changes but are not documented.

:x:@azure-tools/typespec-client-generator-core

The following packages have already been documented:

:white_check_mark: @azure-tools/typespec-azure-resource-manager

Show changes
@azure-tools/typespec-azure-resource-manager - feature ✏

x-ms-skip-url-encoding should be replaced with allowReserved

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs


increased spi buffer sizer

I am using a chip board and I followed these instructions https://bbs.nextthing.co/t/spi-support-on-chip-pro/15150/38 to increase the size of spi buffer from 4096 to 16384. I ran my code and got the following error
Traceback (most recent call last):
File "lights.py", line 98, in
neo.refresh()
File "lights.py", line 82, in refresh
spi.xfer2(stick, 3000000, 0, 8)
TypeError: Non-Int/Long value in arguments: b6cb5e18.
I believe it is because of this line in xfer2
char    wrmsg_text[4096];
I tried changing it but it still gave me the error

According to https://www.raspberrypi.org/forums/viewtopic.php?t=124472&p=837140#p1138833, there are several occurances of the magic number 4096 (hardcoded).

Would be much better if spidev either

allowed changing the buffer size with a method call. The buffers for xfer are allocated in the xfer itself so just in theory the size can be changed between calls to xfer no problem
or just hid the fact that buffer of limited size is used - it can collect a chunk of data from the list passed to fill the buffer, flush it, collect the next chunk etc. Because this is what people end up doing anyway

        for i in range(0, len(data), chunk_size):
            chunk = data[i:i + chunk_size]
            self.spi.xfer(chunk)

it works, just not as efficient as it would be if done by xfer itself.
Still, allowing setting buffer size may not be the worst idea too because by changing /sys/module/spidev/parameters/bufsiz (on Raspberry at least) I can increase that buffer but there is no way to communicate this optimisation to spidev. (Well, if spidev hid the fact it is using limited buffer AND also got default size from this file that would be just brilliant, right? :) )

Pull request are welcome. 😉

@gnbl, your solution works without problems.  Thanks!

Okay, @doceme , here comes the patch :) https://github.com/doceme/py-spidev/pull/75
I decided not to touch any of the existing methods and added xfer3. It accepts lists of arbitrary size and splits large lists into smaller chunks and transfers them.
It also gets maximum SPI buffer size from /sys/module/spidev/parameters/bufsiz but caps it to 65535 because I believe DMA is not possible with larger buffers. If it fails to read /sys/module/spidev/parameters/bufsiz, it defaults to the standard 4K.
I have very little knowledge on Python C APIs so I do not really understand tricks in the original code like this one https://github.com/doceme/py-spidev/blob/master/spidev_module.c#L450-L453 . However from what I read, it looks like PySequence_Fast increments the counter even if it returns you the same object so you always have to do Py_DECREF(seq) which original code does not do in many cases (like error handling or just the normal return). Again, not sure about all of this.

I had follow this tutorial and the problem had fixed: https://youtu.be/1VJDhAClKMc.
Thanks.


Local cache via Service Worker, Toasts and member reactivation.

This merge will add

a service worker (with cache first for all the dependencies excepts for the IHM Rest-API)
Toasts (Red and Green) for errors and feedback (#2)
A button to reactivate members that are deactivated (#5)

Missing Toasts for :

[ ]  New/update bill successfully
[ ]  Remove/deactivate member successfully


Compatibility with Jelly Bean?

Is there a technical reason for android:minSdkVersion="18" in the manifest? I wanted to download the app from F-Droid for a Jelly Bean device.

I thought I was making a call to an API that required SDK 18 or higher. But at the moment I don't see it. I've just made a build with the minimum API set to 16 and put it at https://dl.dropboxusercontent.com/u/19663978/android_apps_gsmlp-debug.apk
If you could check that it works okay I would appreciate it. If it does, I'll bump the version number and push it to GitHub.

Thank you! I downloaded your build and generated a database on the device with both data from OpenCellId and MLS. Then I checked that it works as a location provider. There were no problens for me.


CenterMask implementation

As a better and more interesting model than YOLOACT, It would be really nice to have CenterMask model implemented.

Thanks for your suggestion! We have planned to implement several instance segmentation models in the near future. Although CenterMask is not in our plan, we welcome the community to create a pull request for it.


Remove Osaro

b1b1e90107d804e5317c8527129ceaededdcb162a6737112b88712742a08bb72

Removing Contributor #1418

@IvanDanilov66 please update ur branch based on the latest development branch. There's will be unnecessary commits if not done so.

Squashing & merging


Forecast Report - Wrong Currency symbol

New Forecast report showing Values with US Dollar symbol ($).  I use UK Pound symbol (£).  Either remove $ or respect the currency setting

ahhh I saw that too and (hopefully) fixed it in #2703

Hey @d-whitaker  the fix for this should be in our current release! If you notice the issue still exists and you're definitely running the newest version of the toolkit, please reopen! Thanks!
Please note, all browser can take up to an hour to provide updates to all users. You can force an update if it's available for you by going to chrome://extensions and clicking the "Update" button. Firefox users can go to about:addons, click the setting cog, and click "Check for updates...".


Pagination

Where We make some change in new and edit the previous pagination and sorting applied goes and it loads the data randomly.
Please fix this as soon as possible.

Closing because the issue is old. If you still need help, please reopen.


72 character limit?

Hi,
Why is this code generating identical hashes?  It seems like it ignores anything beyond 72 characters.
<html>
<head>
<script<EMAIL_ADDRESS><script>
var string1 = "Lorem ipsum dolor sit amet, vix eu consequat contentiones. Cu salutandi "
var string2 = "Lorem ipsum dolor sit amet, vix eu consequat contentiones. Cu salutandi accusamus incorrupte eos.  Est vivendo conceptam signiferumque ex, nam at wisi rebum."
hash1 = TwinBcrypt.hashSync(string1, "$2y$10$kJuyLKJVZ5K0OOaaqftTb.");
console.log(hash1);
hash2 = TwinBcrypt.hashSync(string2, "$2y$10$kJuyLKJVZ5K0OOaaqftTb.");
console.log(hash2);
</script>
</head>
</html>

I was using Chrome, but just tried in Firefox, and its console showed more detail:
TypeError: asm.js link error: ArrayBuffer byteLength 0x2000 is not a valid heap length. The next valid length is 0x10000 twin-bcrypt.min.js:6:1244
TypeError: asm.js type error: Disabled by 'asmjs' runtime option twin-bcrypt.min.js

No idea if its related though.
Thanks!

OK - so I just found out that this is a limit with bcrypt in general, so I'll close this.  If you're interested, I'm using twin-bcrypt here for a couple of use cases: http://www.cryptoquiz.cafe/ - it's pretty self explanatory - but I'll need to swap it out for probably SHA256 for the second use case.  Thanks!

Hi John !
Actually the limit is 72 bytes not characters. Thanks for pointing it, I updated the README.md.


Getting the HLSL Semantic Name using the Reflection API

Hello! Is there is any way by which I can get the semantic name that I used in my HLS shader? That is, I have a shader like
struct vsOut
{
    float4 v_Position  : SV_POSITION;
    float3 v_Normal    : M_NORMAL;
    //....
};

vsOut main(vsIn input)
{
    //....
}

Is there is any way by which I can get the name M_NORMAL from the spirv of this vertex shader by using the SPIRV Cross Reflection API? Thanks in advance 🙂

Try looking for an HlslSemantic decoration.

Yes, HlslSemantic is the way to go here. Compiler::get_decoration_string can deal with this.


[WIP] [don't merge] Link python extensions against shared libawkward

An alternative approach to close #209. This doesn't work yet. Just wanted to see if this passes CI.
If there is some mismatch between typeinfo symbols due to copies that
may appear due to static linking, this might fix it. Interestingly,
this instead turns 13 test failures into 171 test failures of the same
kind:
RuntimeError: missing boxer for Content subtype
This also removes "CXX_VISIBILITY_PRESET hidden" from libawkard
interface as it currently hides typeinfo symbols.

Sorry that the tests didn't start automatically. Apparently, my Azure Pipelines aren't set up to run on PRs from forks. I'm going to look that up.

/test

/AzurePipelines help

/AzurePipelines buildtest-awkward

/AzurePipelines run buildtest-awkward

/AzurePipelines run buildtest-awkward


⚠ SceneTime has degraded performance

In 5bcd86a, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 2709 ms

Resolved: SceneTime performance has improved in 29df7a2 after 6 minutes.


Optimize equivariant transformer

I've started looking into how to improve the speed of the equivariant transformer.  I'm opening this issue as a place to report my progress.
I'm mainly interested in the speed of running simulations with it: predicting energy and forces for a single molecule.  I'm less concerned about training speed.  As a starting point, I tried loading molecules of various sizes and computing forces.  For the small molecules we're mainly interested in, the time is completely dominated by overhead and barely affected by the number of atoms.  It ranged from about 11.5 ms for molecules with around 20 atoms, up to 13.1 ms for one with 94 atoms.
Viewing it with the PyTorch profiler shows the GPU running lots of tiny kernels with long gaps in between.  The GPU is sitting idle most of the time.  In contrast, if I profile it during training with a batch size of 64, the GPU usage is very high.  There might still be ways to make it faster, but at least operating on batches keeps the GPU from being idle.
There are two possible approaches to making it faster: improve the PyTorch code, or just replace it with custom CUDA kernels.  The latter is likely to be more effective, so that's what I'm inclined to do.  The custom kernels would be added to NNPOps.

Any thoughts on this question?  Is there actually a reason to change the functional form, or is it really just a matter of not putting RBFs at short distances?
I found that the lower cutoff also changes the model in a significant, unrelated way.  When accumulating interactions, you multiply the attention weight by the cutoff function.
https://github.com/torchmd/torchmd-net/blob/57b4d51f222f3f66405f6b4c71511099144ebd19/torchmdnet/models/torchmd_et.py#L308
Without a lower cutoff, the cutoff function goes to 1 at short distances, which allows self interactions.  You get contributions to the sum for each atom interacting with itself.  With a lower cutoff, the cutoff function goes to 0 which eliminates self interactions.  Did you really intend it to have that effect?  If not, which behavior did you intend (with or without self interactions)?

We actually start distributing the RBFs only beyond the lower cutoff in GaussianSmearing and ExpNormalSmearing. This does not require changing the functional form and in fact we don't even use the cutoff function with a lower cutoff in these two classes.
The change in functional form is important where we want to discard interactions between atoms below the lower cutoff as in the message passing step of the graph network:
https://github.com/torchmd/torchmd-net/blob/57b4d51f222f3f66405f6b4c71511099144ebd19/torchmdnet/models/torchmd_gn.py#L247-L248
Here it is not enough to rely on the RBF distribution as the filter generating network (self.net) might produce non-zero entries in the filter even if all inputs are 0.
Regarding the ET, however, it is not intended that self interactions disappear when lower cutoff is larger than 0. The model should definitely have self interactions. Maybe we could explicitly allow interactions if the distance r_ij == 0.

We do not want to throw away self interactions. I believe the locations where this is currently happening with lower cutoffs larger 0 are your 1st, 5th and 6th points. However, it is still important that we restrict the attention mechanism to interactions between the lower and upper cutoff inside EquivariantMultiHeadAttention if we want to keep the lower cutoff as an option.
I personally haven't used the lower cutoff so far but as far as I know it has helped in other projects, however, using the graph network and not the ET. Maybe it makes sense to remove the lower cutoff option for the ET (or altogether)? @giadefa

The correct solution, it seems to me, is just to make sure your training data includes close contacts so the model can accurately learn that they produce huge energies.

I completely agree but depending on the type of application it might be very difficult to obtain data, which includes close contacts (e.g. existing MD data, which is too expensive to rebuild/augment with close contacts).

When two atoms come very close together, that has a huge effect on the energy. Forcing it to instead have no effect at all won't produce anything the least bit realistic.

That is if the neural network potential is the only component in the energy calculation. Poorly sampled regions, like close contacts in MD, can be handled by a more robust but less accurate prior model.

So, when two beads come together the NN energy is just anywhere and so the
forces. Forces can be strongly attractive at 1A and strongly repulsive at
1.2A. Ideally one would like to regularize the forces, but this involves
the gradient of the gradient of the energy in the loss.
A lower cutoff allows for zero forces in that region and it is the only
solution we found for coarse-graining simulations.
For quantum data the lower cutoff is less important, so you might just
ignore it, if it is simpler. We only use it  for coarse-graining.
g
On Wed, Dec 1, 2021 at 5:02 PM Peter Eastman @.***>
wrote:

It sounds like you're saying the lower cutoff should only be applied for
certain purposes, not others? Here are all the ways it gets used in the ET
model:

In the Distance class, to mask out any pairs (including self
interactions) whose distance is less than the cutoff. This leads to them
being ignored when computing embeddings and when accumulating interactions.
In ExpNormalSmearing to modify the value of alpha
In ExpNormalSmearing to modify where the RBFs initially get placed
(but they can move during training)
To modify the functional form of computing the RBF values
In NeighborEmbedding to alter the functional form while summing over
neighbors
In EquivariantMultiHeadAttention to alter the attention weights

Do you really want it to have all of these effects, or should it be
restricted to a subset of them?
I still don't understand the reason for having the lower cutoff in the
first place. @giadefa https://github.com/giadefa said
At close distances there is no data to train on. Having a lower cutoff
produces nicer models.
That doesn't make sense to me. If you try to apply a model to cases far
outside the training data, you probably won't get good results. That
includes having atoms much closer together than ever happened in the
training data. But having the model pretend those close contacts don't
exist is not going to help in any way. When two atoms come very close
together, that has a huge effect on the energy. Forcing it to instead have
no effect at all won't produce anything the least bit realistic.
The correct solution, it seems to me, is just to make sure your training
data includes close contacts so the model can accurately learn that they
produce huge energies.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/torchmd/torchmd-net/issues/45#issuecomment-983841859,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB3KUOWV7CZJWUNFNM7MP43UOZIKBANCNFSM5HTXARSQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

For me it is fine to remove the lower cutoff for ET
On Wed, Dec 1, 2021 at 6:06 PM Philipp Thölke @.***>
wrote:

We do not want to throw away self interactions. I believe the locations
where this is currently happening with lower cutoffs larger 0 are your 1st,
5th and 6th points. However, it is still important that we restrict the
attention mechanism to interactions between the lower and upper cutoff
inside EquivariantMultiHeadAttention if we want to keep the lower cutoff
as an option.
I personally haven't used the lower cutoff so far but as far as I know it
has helped in other projects, however, using the graph network and not the
ET. Maybe it makes sense to remove the lower cutoff option for the ET (or
altogether)? @giadefa https://github.com/giadefa
The correct solution, it seems to me, is just to make sure your training
data includes close contacts so the model can accurately learn that they
produce huge energies.
I completely agree but depending on the type of application it might be
very difficult to obtain data, which includes close contacts (e.g. existing
MD data, which is too expensive to rebuild/augment with close contacts).
When two atoms come very close together, that has a huge effect on the
energy. Forcing it to instead have no effect at all won't produce anything
the least bit realistic.
That is if the neural network potential is the only component in the
energy calculation. Poorly sampled regions, like close contacts in MD, can
be handled by a more robust but less accurate prior model.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/torchmd/torchmd-net/issues/45#issuecomment-983921681,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB3KUOTCJKVXCQZ2RTKLYF3UOZP3TANCNFSM5HTXARSQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Poorly sampled regions, like close contacts in MD, can be handled by a more robust but less accurate prior model.

Perhaps a way of handling that would be to add a regularization term encouraging the NN energy contribution to go to zero at short distances?  Or just create some artificial training samples that represent very close contacts and specify the energy as equal to the energy of the prior model?

We tried with the training samples at low distance but did not work,
because the NN is many-body. You can fix the two-body but then you have
problems with 3 body interactions going wild.
On Wed, Dec 1, 2021 at 6:18 PM Peter Eastman @.***>
wrote:

Poorly sampled regions, like close contacts in MD, can be handled by a
more robust but less accurate prior model.
Perhaps a way of handling that would be to add a regularization term
encouraging the NN energy contribution to go to zero at short distances? Or
just create some artificial training samples that represent very close
contacts and specify the energy as equal to the energy of the prior model?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/torchmd/torchmd-net/issues/45#issuecomment-983932318,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB3KUOXW67DOGA3AWAVGVLTUOZRIVANCNFSM5HTXARSQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


adding reflections as a dependency

Description
adding reflections as a dependency.
POST _plugins/_ppl
{
  "query": "source=iris_data | fields petal_length_in_cm,petal_width_in_cm | kmeans centroids=3"
}

This command was breaking and the debug log is:
Caused by: java.lang.NoClassDefFoundError: org/reflections/Reflections
opensearch-node1         | 	at org.opensearch.ml.common.MLCommonsClassLoader.loadMLAlgoParameterClassMapping(MLCommonsClassLoader.java:66) ~[?:?]
opensearch-node1         | 	at org.opensearch.ml.common.MLCommonsClassLoader.loadClassMapping(MLCommonsClassLoader.java:51) ~[?:?]
opensearch-node1         | 	at org.opensearch.ml.common.MLCommonsClassLoader.lambda$static$0(MLCommonsClassLoader.java:39) ~[?:?]
opensearch-node1         | 	at java.security.AccessController.doPrivileged(AccessController.java:569) ~[?:?]
opensearch-node1         | 	at org.opensearch.ml.common.MLCommonsClassLoader.<clinit>(MLCommonsClassLoader.java:38) ~[?:?]
opensearch-node1         | 	at org.opensearch.ml.common.output.MLOutput.fromStream(MLOutput.java:35) ~[?:?]
opensearch-node1         | 	at org.opensearch.ml.common.transport.MLTaskResponse.<init>(MLTaskResponse.java:38) ~[?:?]
opensearch-node1         | 	at org.opensearch.ml.common.transport.MLTaskResponse.fromActionResponse(MLTaskResponse.java:55) ~[?:?]
opensearch-node1         | 	at org.opensearch.ml.client.MachineLearningNodeClient.lambda$getMlPredictionTaskResponseActionListener$8(MachineLearningNodeClient.java:191)

Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Is this issue only for 2.x? If not, normally we send PR to main branch first.

Is this issue only for 2.x? If not, normally we send PR to main branch first.

Yeah we found this issue in 2.7. Should I raise a PR for main branch?

Codecov Report

Merging #1558 (6ab50ed) into 2.x (8c32040) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##                2.x    #1558   +/-   ##
=========================================
  Coverage     97.18%   97.18%
  Complexity     4098     4098
=========================================
  Files           371      371
  Lines         10445    10445
  Branches        703      703
=========================================
  Hits          10151    10151
  Misses          287      287
  Partials          7        7

Flag
Coverage Δ

sql-engine
97.18% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Raised a PR for main branch: https://github.com/opensearch-project/sql/pull/1559

Could you please add integration test?

Could you please add integration test?

Thanks @Yury-Fridlyand , good point. I think it takes effort to do that. Considering this issue blocks OS2.7 release and we have verified manually this PR will fix the issue, how about we merge this PR first? We can continuously improving the integ testing.

Closing as duplicate https://github.com/opensearch-project/sql/pull/1559


A subsequent request after a 100-expect failed requests fails incorrectly.

If a PUT request fails with a 4XX status where expect headers are set, then the next request if the same socket is reused seems to fail with the same 401 status even though the server actually responds with say a 200. We have a test case in test_aws_request.py test_state_reset_on_connection_close which seems to explicitly close the socket, though if the server actually responds without a Connection: close, then due to keepalives, when we read the next response we seem to incorrectly read the previous response

This behaviour isn't seen in most S3 regions as many of them return an explicit Connection:Close on 4XX statuses, though if that isn't done (as the specs do not mention close on error) and the connection is kept-alive, then you'd run into the next response being read incorrectly by httplib

I've some sort of code (tested against ceph-radosgw) which demonstrates the behaviour,
https://gist.github.com/theanalyst/161d21767a78d2a9b4487c722aa6ea1a
with asyncio aiohttp (which probably doesn't use python's httplib?) a reused TCP connection that sends a 4XX status and later success status seemed to work as expected,
a boto client with a 400 (or any 4XX status) will actually raise an exception on the first successful request (wireshark logs show a 200) though requests after that will work as expected
aws-sdk-go also succeeds similar to aiohttp, and the network logs clearly in all cases show a 2XX status which boto seems to incorrectly parse.
While aws services may force a Connection: Close header on 4XX status, nothing on the rfc or the api specs actually require this behaviour, so nothing prevents the socket from being kept open (provided the connection:close wasn't explicitly sent by the server) for services, so it is just a client error that hasn't been spotted yet

@theanalyst - Thank you for your post. I would mark this as enhancement.

@theanalyst - Thank you for providing detailed information on this issue. I will mark this as enhancement.

the issue still exists on a recent check


[Bug]: github.py should not be named github, circular import issue

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Describe the bug and reproduction steps
Try running: python openhands/server/listen.py and you will get this error
ERROR:root:  File "/home/ubuntu/openhands/openhands/server/listen.py", line 23, in <module>
    from openhands.server.github import (
  File "/home/ubuntu/openhands/openhands/server/github.py", line 3, in <module>
    from github import Github
  File "/home/ubuntu/openhands/openhands/server/github.py", line 3, in <module>
    from github import Github

ERROR:root:<class 'ImportError'>: cannot import name 'Github' from partially initialized module 'github' (most likely due to a circular import) (/home/ubuntu/openhands/openhands/server/github.py)

I fixed this by renaming github.py to github_utils.py
OpenHands Installation
Docker command in README
OpenHands Version
No response
Operating System
None
Logs, Errors, Screenshots, and Additional Context
No response

Hi there! Are you able to open a PR with your change and get some 👀 on it please? If you ran into it, it's likely others will too..

Okay will do
On Thu, Nov 21, 2024 at 6:21 AM mamoodi @.***> wrote:

Hi there! Are you able to open a PR with your change and get some 👀 on it
please? If you ran into it, it's likely others will too..
—
Reply to this email directly, view it on GitHub
https://github.com/All-Hands-AI/OpenHands/issues/5168#issuecomment-2491343941,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BL3P7PWNLFYRGDOSIJKNKRT2BXT5FAVCNFSM6AAAAABSGHRWXKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJRGM2DGOJUGE
.
You are receiving this because you authored the thread.Message ID:
@.***>


fix(sql): set empty string as default compression alg

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
bug fix

What is the current behavior? (You can also link to an open issue here)
resolutions created before the compression feature cannot be loaded because the steps_compression_alg column is filled with NULL as value.

What is the new behavior (if this is a feature change)?
NULL value are replaced by empty strings, the column is not nullable anymore.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
no

Other information:

CDS Report test#1491.0 ✘

tests and lint

golangci-lint ✔
unit-test and integration-tests ✘

CDS Report test#1492.0 ✘

tests and lint

golangci-lint ✔
unit-test and integration-tests ✘


How to display on top of the keyboard?

How can I show the dropdown on top of the keyboard?

@frkncngz Did you figure this out? I've found this problem too.


Specify Directory AllowOverride in docs

@jamiesnape PTAL.

LGTM.

Apparently Ansible had the right idea, but it never made it to the docs:
provisioning/ansible/roles/apache2/templates/etc-apache2-sites-available-vagrant.conf.j2#L5

I've updated to be consistent with Ansible.  PTAL.

LGTM.


Publish of a crate was missed

Crate name: stacker 0.1.6
Build failure link: https://docs.rs/crate/stacker/0.1.6/builds
Additional details:
stacker 0.1.6 was released 5 days ago. docs.rs appears to not have been able to pick-up on this and never scheduled a build?
Another crate published at approx the same time did get built fine: https://docs.rs/crate/psm/0.1.5/builds

Weird, did a rebuild and it was fine. There were a few bugs after the rustwide rollout, so this crate might have encountered one of them. The docs are available here now: https://docs.rs/stacker/0.1.6/stacker/


Fix error when paginating queries

When paginating we were setting up the circle-token with multiple
values because request did params.Add("circle-token", ...) rather
than using params.Set.
That results in CircleCI returning a 401.
This fixes the issue.

:+1:

Thanks @ernesto-jimenez!
I'm curious which endpoint(s) you are paginating to see if it makes sense to add support to the library. I'm currently only aware of /recent-builds.
Regardless, this change makes sense to me. Thanks for including tests!


Increase Keylimit of Request Form

Hello together,
I have a large view, with a lot of bindings. When I serialize the data and send them to the server with an AJAX request, not all keys are represent in the Request.Form. There is a limit of 1000 keys. Is there a way to increase the limitation?
Regards

Nancy version: 1.4.4

Nancy host

[x] Nancy.Hosting.Self 1.4.1
[x] Nancy.Viewengines.Razor 1.4.3

Environment (Operating system, version and so on): Windows 10

.NET Framework version: 4.6.1

I'm not aware of a limit of 1000 keys. What makes you think there's a limit?

You can set the StaticConfiguration.RequestQueryFormMultipartLimit property, which defaults to 1000:
https://github.com/NancyFx/Nancy/blob/dbdbe9428caea06c17d34a507c9216fe35abadc4/src/Nancy/StaticConfiguration.cs#L17


expose encoded values

User could use cadenceClient service API to get the closing event of a given workflow, however, that would only give you the blob data of the closing event. We need a way to convert that blob to strong typed values for user to consume.
Move the interface definition to internal to avoid circular dependency.

Coverage decreased (-0.1%) to 67.169% when pulling 936ab0e3a5f5a574bcb3ca435a06a164e2a7924b on yiminc:encoded_value into 570324de7bb1008983846c97ee4d80d24e277854 on uber-go:master.

I don't understand the need for this change. The encoded package contains only interfaces. So I'm not sure how it can have circular dependency.


Wrote the relnote

@denis-shuvalov @udh-mendix I wrote a release note for the changes to alerts (see PR https://github.com/mendix/docs/pull/5170). Let me know any comments.

@denis-shuvalov I've updated the release note per your comments, can you take a look and verify that all is correct?


1.2.0 issue with jruby-<IP_ADDRESS> on Travis

FWIW our build failed when 1.2.0 was released, see activeadmin/pull_requests #6644  Reverting to 1.1 fixed the build for us.
P.S. You may want to tag your 1.2 release in GitHub.

@varyonic thanks for opening this. I'm a longtime user of Active Admin, big fan.
I'm going to add JRuby to the CI pipeline and we'll see what we see ...
P.S. I forgot to git push --tags, it's been corrected.

The test suite (such as it is) passes on JRuby. Moreover, when I install it under JRuby <IP_ADDRESS>, on an otherwise pristine-ish system, an appropriate binary downloads:
$ ls -lt ~/.chromedriver-helper/2.35/linux64/*
-rwxr-xr-x 1 flavorjones flavorjones 7903592 2018-02-16 22:41 /home/flavorjones/.chromedriver-helper/2.35/linux64/chromedriver*
-rw-r--r-- 1 flavorjones flavorjones 3721349 2018-02-16 22:41 /home/flavorjones/.chromedriver-helper/2.35/linux64/chromedriver_linux64.zip

and I can run chromedriver:
Starting ChromeDriver 2.35.528139 (47ead77cb35ad2a9a83248b292151462a66cd881) on port 9515

Can you help me out with any further information?

Same results with JRuby <IP_ADDRESS> (which was the version running in the failed travis run):
chromedriver-helper $ rvm install "jruby-<IP_ADDRESS>"
Searching for binary rubies, this might take some time.
Found remote file https://s3.amazonaws.com/jruby.org/downloads/<IP_ADDRESS>/jruby-bin-<IP_ADDRESS>.tar.gz
Checking requirements for mint.
Requirements installation successful.
jruby-<IP_ADDRESS> - #configure
jruby-<IP_ADDRESS> - #download
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 20.0M  100 20.0M    0     0  6417k      0  0:00:03  0:00:03 --:--:-- 6418k
jruby-<IP_ADDRESS> - #validate archive
jruby-<IP_ADDRESS> - #extract
jruby-<IP_ADDRESS> - #validate binary
jruby-<IP_ADDRESS> - #setup
jruby-<IP_ADDRESS> - #gemset created /home/flavorjones/.rvm/gems/jruby-<IP_ADDRESS>@global
jruby-<IP_ADDRESS> - #importing gemset /home/flavorjones/.rvm/gemsets/jruby/global.gems..
jruby-<IP_ADDRESS> - #generating global wrappers.......
jruby-<IP_ADDRESS> - #gemset created /home/flavorjones/.rvm/gems/jruby-<IP_ADDRESS>
jruby-<IP_ADDRESS> - #importing gemsetfile /home/flavorjones/.rvm/gemsets/default.gems evaluated to empty gem list
jruby-<IP_ADDRESS> - #generating default wrappers.......

curiosity jruby-<IP_ADDRESS> (master)
chromedriver-helper $ ls -lt ~/.chromedriver-helper/2.35/linux64/*
ls: cannot access '/home/flavorjones/.chromedriver-helper/2.35/linux64/*': No such file or directory

curiosity jruby-<IP_ADDRESS> (master)
chromedriver-helper $ ruby -v
jruby <IP_ADDRESS> (2.3.3) 2017-12-07 929fde8 OpenJDK 64-Bit Server VM 25.151-b12 on 1.8.0_151-8u151-b12-0ubuntu<IP_ADDRESS>-b12 [linux-x86_64]

curiosity jruby-<IP_ADDRESS> (master)
chromedriver-helper $ gem install chromedriver-helper
Fetching: io-like-0.3.0.gem (100%)
Successfully installed io-like-0.3.0
Fetching: archive-zip-0.11.0.gem (100%)
Successfully installed archive-zip-0.11.0
Fetching: nokogiri-1.8.2-java.gem (100%)
Successfully installed nokogiri-1.8.2-java
Successfully installed chromedriver-helper-1.2.0
4 gems installed

curiosity jruby-<IP_ADDRESS> (master)
chromedriver-helper $ ls -lt ~/.chromedriver-helper/2.35/linux64/*
-rwxr-xr-x 1 flavorjones flavorjones 7903592 2018-02-16 22:47 /home/flavorjones/.chromedriver-helper/2.35/linux64/chromedriver*
-rw-r--r-- 1 flavorjones flavorjones 3721349 2018-02-16 22:47 /home/flavorjones/.chromedriver-helper/2.35/linux64/chromedriver_linux64.zip

curiosity jruby-<IP_ADDRESS> (master)
chromedriver-helper $ which chromedriver
/home/flavorjones/.rvm/gems/jruby-<IP_ADDRESS>/bin/chromedriver

curiosity jruby-<IP_ADDRESS> (master)
chromedriver-helper $ chromedriver --verbose
Starting ChromeDriver 2.35.528139 (47ead77cb35ad2a9a83248b292151462a66cd881) on port 9515
Only local connections are allowed.

Closing, as the OP did not provide any additional information to assist in reproducing the issue.


Can't install because nextjs version is too low

do
npm i next-use-contextual-routing

get this error:
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ^10.0.0 || ^11.0.0" from<EMAIL_ADDRESS>

hi @toomuchdesign! came here to report the same thing.. any chance this can be updated to support the latest next js version?


Packages published with alpha tag while not necessarily being alpha

├─ ilp-protocol-stream@npm:2.7.2-alpha.0
Are all the packages actually alpha?

Opened in wrong repo somehow :/


Exception when using an empty "secret key" string

If you get an error like this, you're creating AWS4Auth() with an empty "secret key" string.
>>> requests_aws4auth.AWS4Auth('username','', 'eu-west-1', 's3')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python2.7/site-packages/requests_aws4auth/aws4auth.py", line 246, in __init__
    self.regenerate_signing_key(secret_key=secret_key)
  File "/usr/lib/python2.7/site-packages/requests_aws4auth/aws4auth.py", line 291, in regenerate_signing_key
    secret_key = secret_key or self.signing_key.secret_key
AttributeError: 'NoneType' object has no attribute 'secret_key'

It's not something real users would want to do, but found this in testing code that previously ran with V2 Signature awsauth.S3Auth. This patch works around the error:
diff -u /usr/lib/python2.7/site-packages/requests_aws4auth/aws4auth.py~ /usr/lib/python2.7/site-packages/requests_aws4auth/aws4auth.py
--- /usr/lib/python2.7/site-packages/requests_aws4auth/aws4auth.py~	2017-02-15 19:17:02.000000000 +0000
+++ /usr/lib/python2.7/site-packages/requests_aws4auth/aws4auth.py	2017-02-18 00:18:15.425646504 +0000
@@ -288,7 +288,7 @@
                                    self.signing_key.secret_key is None):
             raise NoSecretKeyError

-        secret_key = secret_key or self.signing_key.secret_key
+        secret_key = secret_key or (self.signing_key and self.signing_key.secret_key) or ''
         region = region or self.region
         service = service or self.service
         date = date or self.date

@sam-washington I am experiencing this behavior using fallback env variables in my unit tests and it's very annoying. Could we change this behavior?


Version mismatch in CMakeLists

https://github.com/akielaries/openMTPK/blob/ecf824d5063e61e7a7fa99116c98506235a46a6a/CMakeLists.txt#L38
I believe the latest project version is 0.9.7 according to PyPI and 0.9.4 according to GitHub release tags. CMakeLists.txt stills points to 0.5. Perhaps we can keep all these version numbers in sync somehow.

This is a good point. I wonder if we could just omit the version number for CMake. If not it could be nice to perhaps make it dynamic by including it into some sort of pipeline.

I'm looking to publish a RPM for the package in the Fedora Project, making it available for use as dnf install openmtpk. I was able to build the package successfully but at the cost of some changes (like making the libopenMTPK shared instead of static, adding -fPIC to compilation flags for tests and making use of gtest from a system level and not fetch directly from CMake).
The last one was majorly because Fedora builds happen inside a chroot environment and networking is natively disabled. I'll create a different project for the said Fedora builds and share it with you for you to review.
As for the changes, let them be as patch for now for the builds I'm taking. Once they look good and are reviewed by you, you can pull them into the upstream. I'd also like to discuss with you more about the package publishing.

Thank you very much for doing this! This is pretty cool. In terms of making openMTPK a static vs shared library, it doesn't matter too much to me, there are benefits to both and it would take some maturing of the project to really boil down our use cases. These changes are very much welcome and I'd love to review them! Thank you for all your help so far. Feel free to open a separate issue or continue this thread to discuss about the package publishing.
In response to this earlier statement...

I believe the latest project version is 0.9.7 according to PyPI and 0.9.4 according to GitHub release tags

So far, the versions for the Python API and the actual source code are to be treated differently at the moment. The source code gets updated more frequently than the Python API and mostly to avoid constantly publishing new versions to PyPI, in this early stage it can't be helped as we're trying to get things right regarding the CI/CD pipeline currently in place

solved in #99 by fetching the dirty git tag using git describe --tags from the CMakeLists.txt file


A better way to create cita config

Description
TBD
Steps to Reproduce
Omit
Versions
develop
Additional Information
-------------Chinese-----------------
是否有更好的生成链配置的方法

A proposal first

That may be a tricky modification.
Once we put configuration in  a file and change the way we config nodes. the integrate tests(CI) will hava a big impact.
My opinion: Making --contract_arguments more detailed.
optional arguments:
  -h, --help            show this help message and exit
  --authorities AUTHORITY[,AUTHORITY[,AUTHORITY[,AUTHORITY[, ...]]]]
                        Authorities (addresses) list.
  --chain_name CHAIN_NAME
                        Name of the new chain.
  --nodes IP:PORT[,IP:PORT[,IP:PORT[,IP:PORT[, ...]]]]
                        Node network addresses for new nodes.
  --super_admin SUPER_ADMIN
                        Address of super admin.
  --contract_arguments Contract.Argument=Value [Contract.Argument=Value ...]
                        Update constructor arguments for system contract. Can
                        be specify more than once.
  --timestamp TIMESTAMP
                        Specify a timestamp to use.
  --resource_dir RESOURCE_DIR
                        Chain resource directory.
  --grpc_port GRPC_PORT
                        grpc port for this chain
  --jsonrpc_port JSONRPC_PORT
                        jsonrpc port for this chain
  --ws_port WS_PORT     websocket port for this chain
  --enable_tls          The data is encrypted and transmitted on the network

That may be a tricky modification.
Once we put configuration in a file and change the way we config nodes. the integrate tests(CI) will hava a big impact.
My opinion: Making --contract_arguments more detailed.
optional arguments:
  -h, --help            show this help message and exit
  --authorities AUTHORITY[,AUTHORITY[,AUTHORITY[,AUTHORITY[, ...]]]]
                        Authorities (addresses) list.
  --chain_name CHAIN_NAME
                        Name of the new chain.
  --nodes IP:PORT[,IP:PORT[,IP:PORT[,IP:PORT[, ...]]]]
                        Node network addresses for new nodes.
  --super_admin SUPER_ADMIN
                        Address of super admin.
  --contract_arguments Contract.Argument=Value [Contract.Argument=Value ...]
                        Update constructor arguments for system contract. Can
                        be specify more than once.
  --timestamp TIMESTAMP
                        Specify a timestamp to use.
  --resource_dir RESOURCE_DIR
                        Chain resource directory.
  --grpc_port GRPC_PORT
                        grpc port for this chain
  --jsonrpc_port JSONRPC_PORT
                        jsonrpc port for this chain
  --ws_port WS_PORT     websocket port for this chain
  --enable_tls          The data is encrypted and transmitted on the network

first, not decide to use file yet.
second, the CI is a part job of this instead of the reason to deny it.


option to add custom gif image or widget

I was wondering if there is anyway to add custom image or gif image or some widget as progress indicator

The progress indicator can be any type of widget. So, for example, a widget containing a custom image can be used.


ctPointLabels is not defined

Hi, I've found chartist to be an excellent tool. I customized my charts to a great extent by modifying the css file.
However, when I try to get any of the proposed plugins that you posted, I cannot get the to work. I'm srry if this is a newbie question, but that's what I'm. What I've tried so far are the following options:
OPTION A:

download the chartist-plugin-ctpointlabels.js and saved it in the "chartist/dist" folder, right alongside chartist.js and chartist.css which I know are working.
Added the plugin description in the configuration options of the chartist.Line call

        $scope.config = {
              high: 15,
              low: 0,
              showArea: true,
              showLine: true,
              showPoint: true,
              fullWidth:true,
              axisY: {
                 showLabel: false,
                 showGrid: false,
                 offset: 5
              },
              axisX: {
                showLabel: false,
                showGrid: false
              },
              plugins: [
                ctPointLabels({
                  textAnchor: 'middle'
                }),
                ],

        };

Result: All I get is an error stating that ctPointLabels is not defined: " ReferenceError: ctPointLabels is not defined"

OPTION B:

download the chartist-plugin-ctpointlabels.js, copied its code and placed it whitin chartis.js (on the same level as @module Chartist.Base - line 1153 or Chartist.Class - line 1044) o even at the very begging of the file (line 1).
and 3) same as OPTION A

It's worth noting that the chart is working fine with the original config options (with the the " plugins" section commented). The original working options are:
$scope.config = {
high: maxVal,
low: 0,
showArea: true,
showLine: true,
showPoint: showPoints,
fullWidth:true,
axisY: {
showLabel: false,
showGrid: false,
offset: 5
},
axisX: {
showLabel: false,
showGrid: false
}
// ,
// plugins: [
//   ctPointLabels({
//     textAnchor: 'middle'
//   }),
//  ],
        };

Thanks for any help or guide on how to properly setup the workspace for plugins (I checked the chartistjs plugin section, but I couldn't find a solution)

Hey there. Thanks for the detailed issue! Actually youre doing it right with option A but the example is wrong in the readme! It should be Chartist.plugins.ctPointLabels and not ctPointLabels.

Ill update the readme

You didnt mention that youve included the script of the plugin in your html. So in case youve missed that youd also need to do this.

Hi, thanks for the prompt reply. I had included the script in the html, so it may be the naming of the plugin. Later today I'll try and let you know if it works.

Great! It worked like a charm! thanks! I will post some other questions on the main library repository later. Thanks for the awesome work!

Ok, just to let you know. Your code is incredibly flexible. I just wanted one particular set of data to have labels on the points. And it was as easy as placing a conditional statement on the 'draw' event emitter...
Now I'm playing around with the idea of placing legends close to each of the curves.

Done, I use the data.x and data.y to place the legend for the GOAL line

Cool! :-) glad it worked out for you nicely.

The README never got changed FYI. Thankfully stumbled across this issue. Although no more errors, just labels aren't being displayed now 🤔

Same here, README is still wrong. Confused me a bit.


Query Engine - What should be used :  input_type="search_document". or serach_query

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
 When iam inserting in Vector db , Iam using the same Embed_model as    input_type="search_document". below Code where iam also retrieving the query engine where iam using the same embedmodel , Iam still gettting response .Should i use the  input_type="search_query"  which one is better and why ?

  embed_model = CohereEmbedding(
    cohere_api_key=cohere_api_key,
    model_name="embed-english-v3.0",
    input_type="search_document"

    )
    service_context = ServiceContext.from_defaults(
    llm=llm, embed_model=embed_model)

    # load storage context
    load_storage_context = StorageContext.from_defaults(persist_dir=index_persist_dir)

    # load index
    loaded_index = load_index_from_storage(
    storage_context=load_storage_context, service_context=service_context
    )

    query_engine = loaded_index.as_query_engine(
    node_postprocessors = [cohere_rerank]
    )

search query should be used for queries, and search document for ingesting data
tbh the embedding class could maybe be doing this automatically