Separate thrift handler for frontend service

What changed?

Created ThriftHandler wrapper on top of Handler interface. At this point it only forward requests as is to underlying handler.
Covered ThriftHandler with units tests. Test that requests are forwarded to underlying handler and response is returned as is.
Updated handler registration. Actual handler registration within YARPC dispatcher was moved to thrift specific handler.

Why?
This is purely a mechanical refactoring that introduces additional ThriftHandler wrapper. For now it only forwards requests to the original handler. Later on this will become a point for conversion between internal types and thrift types. Underlying handler will be independent of rpc protocol types.

How did you test it?

Covered ThriftHandler with unit tests
Started server locally

Potential risks
None, purely mechanical change.

Coverage increased (+0.08%) to 59.539% when pulling 670decf3c525e48c5cde7a49874fbac1b17b4d0f on vytautas-karpavicius:thrift-frontend-handler into 81c18343ce2b585ead06ef5c3dd349eada6d70fc on uber:master.


Also handle image/jpeg mimetype

On my MacBook, I was getting this mimetype when I tried to drag-and-drop
an image from the Finder.
Fixes: #70

As we are moving toward the client/user paradigm gradually, do you think you could replicate your PR on the /client repo as well?

Okay. I'll try to figure out what you are doing there and create a PR for that.


Inconsistent version indexing

Package Name: azure-keyvault-secrets
Package Version: 4.4.0
Operating System: Ubuntu 20.04
Python Version: 3.9.13

Describe the bug
Using the same command to get and iterate through versions of two different secrets starts with different version "offsets". One where the first version is the current version as shown in the Azure portal view, and the other starts from the most recent "Older version". The only difference I can see between the two secrets is that the first only has one older version while the second has more than ~45 older versions.
Example
If the current versions of Secret_1 and Secret_2 are a000000 and b000000, respectively the current implementation returns versions as:
Offset      Secret_1       Secret_2
0           a000000        b000001
1           a000001        b000002
2           a000002        b000003

To Reproduce
for secret in [secret_1, secret_2]:
    print(f"Current:\t{secret.name}\n\t{secret.properties.version}")
    versions = secret_client.list_properties_of_secret_versions(secret.name)
    for i, v in enumerate(versions):
        print(f"{i}:\t{v.name}\n\t{v.version}")

Expected behavior
Expect indexing to be consistent.
Example
Offset      Secret_1       Secret_2
0           a000000        b000000
1           a000001        b000001
2           a000002        b000002

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'KeyVault:0.21158333,Service Bus:0.13389705,Storage:0.09951048'

Hi @keriksson-rosenqvist, thanks for the feedback. We'll investigate asap.

Hi @keriksson-rosenqvist, thank you again for opening an issue! This behavior seems to be because of how the list of secret version properties is returned by the Key Vault service. Rather than ordering the version properties by the time at which the secret version was created, the service appears to order the list by the name of the secret version string (0-9, then a-z).
For example, if I run the code snippet you provided when I have secret_one and secret_two which each have five versions, I get (with adjusted formatting post-execution):
Current:        secret-one        86114ff30d1341eb910309d0ef143326

0:              secret-one        03ad90deaf5b4d91b809ac6abd35dc9e
1:              secret-one        573180db6a2240b3b28badf3e2000e54
2:              secret-one        86114ff30d1341eb910309d0ef143326
3:              secret-one        8a371b4b30f74151ac5a1bcf378a457f
4:              secret-one        91792547ac9243e4a83695f0691ae70f
5:              secret-one        e9a8b3216f9e4afda28823caaf66030f

Current:        secret-two        d56f3e18824b4bbf92850259d4303a89

0:              secret-two        34aa75bb2a184701823e621c263cf73d
1:              secret-two        5b0a36b68c7442d18bb6440aad934961
2:              secret-two        d56f3e18824b4bbf92850259d4303a89
3:              secret-two        e04a6ad1e6434659a818fffbabadc6fc
4:              secret-two        eca7728ea7a141a88341b919bd871222
5:              secret-two        f03f7e331f36469783e7449450953b5e

This output order is produced deterministically, consistent the alphanumeric ordering of the version strings. Is this consistent with what you've seen? Please let me know if I misunderstood your question and if there's something else that you're asking about.
cc: @jlichwa, is this ordering in the Key Vault response intentional and correct?

@mccoyp Yes you are correct. Apologies for misinterpreting the functionality of list_properties_of_secret_versions(). However is there a key word argument or other input to that I can use to get the versions ordered by date?

@keriksson-rosenqvist The results of list_properties_of_secret_versions unfortunately can't be sorted through function input because of limitations in what we receive from the service response. However, it shouldn't take too much work to do some sorting on the application end -- for example:
secret_one_version_properties = [props for props in secret_client.list_properties_of_secret_versions("secret-one")]
sorted_properties = sorted(secret_one_version_properties, key=lambda props: props.updated_on, reverse=True)

This gives us the properties for each secret version, ordered from newest to oldest. The limitation here is that the updated_on property is only accurate to the second, so versions that were updated in the same second will be ordered alphanumerically by version string:
Current:        86114ff30d1341eb910309d0ef143326

03ad90deaf5b4d91b809ac6abd35dc9e        2022-06-30 00:44:27+00:00
86114ff30d1341eb910309d0ef143326        2022-06-30 00:44:27+00:00
573180db6a2240b3b28badf3e2000e54        2022-06-30 00:44:26+00:00
8a371b4b30f74151ac5a1bcf378a457f        2022-06-30 00:44:26+00:00
91792547ac9243e4a83695f0691ae70f        2022-06-30 00:44:26+00:00
e9a8b3216f9e4afda28823caaf66030f        2022-06-30 00:44:26+00:00

This should suffice, though, for the more common case when there aren't many secret versions being generated within the same second.


Report error when having a circular template constraint

fix #2788

: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:@typespec/compiler

You can try these changes at https://cadlplayground.z22.web.core.windows.net/prs/2932/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/prs/2932/


Bumps brokerapi from v10 to v11

#187356491
Authored-by: Ryan Wittrup<EMAIL_ADDRESS>All unit tests ran and passed as expected - the SDK uses the domain models package from broker API, which did not include any breaking changes in the V11 release

I don't think we should be go mod vendoring in this repo.
Consumers may vendor this library but this repo has traditionally not vendored its dependencies.

Can you squash your commits together so we don't have one with vendoring and one without?
Otherwise, LGTM.


Add fediverse creator tag

The feature
We should add the fediverse creator tag that got added in the Mastodon 4.3 release. For us, I believe it would look like:
<meta name="fediverse:creator" content="<EMAIL_ADDRESS>/>
and I think the right place to put it would be in templates/layouts/layout.html.
Why?
This would increase discoverability of the Mastodon account when links are being shared on the fediverse.
Examples and inspiration
See examples in the first link above.

(Note: Author attribution for the domain govdirectory.org has already been enabled in our Mastodon account.)

I'm curious, are there other examples of it being used to reference brand/website accounts?

I'm curious, are there other examples of it being used to reference brand/website accounts? This usage seems more like a self reference to me similar to the intent of rel="me".

I haven't seen any in the wild yet, which can be explained by it being a new feature.
But the difference from rel=me is that if someone would share any link from govdirectory, let's say https://www.govdirectory.org/germany/Q491637/ then the link preview window on Mastodon would have a bottom bar with "More from Govdirectory" on it. So, rather than verifying an account through one link, it's giving discoverability of it through many links.

The feature has been around for a few months and I see it quite often but never in this way. It has always been used to point to the creator of a specific work not globally for a brand/website, the documentation also use it in the former way.

I think that before 4.3 it was only available as an experiment for a few media houses, who used it for the accounts of their journalists on the media house's self-hosted Fediverse instances.

No it has been available for everyone for at least a few months, it has just not been visible for Mastodon instances not following the nightly builds.

Either way, I don't see why that matters? The documentation does not say that it has to point to a personal account or that it cannot point to only one account.

We do have a system for this particular use and the intent with this new one seems different so I could imagine this practically being abuse. That's why I'm asking for examples where it's being used in a similar fashion to what's proposed here, I wouldn't want to be the outlier...

I find it unlikely someone would think it is abuse, we put a relevant value of the expected type in that tag. And it looks like the man working on the documentation of the feature also thinks it is ok.

That's good enough for me!


Adding VocabularySubauthority, a QA adapter based on Vocabulary/VocabularyTerm

Following on #1184

Coverage remained the same at 100.0% when pulling 7e91325d8b5a44c1c410665d73c2566e6a62f373 on sub-vocabularies-qa into 9ee26e9140c140863a63171f931e2298bde07fcb on master.


intermittent router TestAddTCPRoute failure

[router] 2015/01/26 02:46:07.627255 Started PUT /routes
[router] 2015/01/26 02:46:07.636804 Completed 200 OK in 9.516203ms
2015/01/26 02:46:09 http: TLS handshake error from <IP_ADDRESS>:35166: router: route not found or TLS not configured
2015/01/26 02:46:10 Error while processing update from etcd: address already in use, &etcd.Node{Key:"/router/tcp/1b0c7224572324e8772d17a2cc5134a5", Value:"{\"id\":\"1b0c7224572324e8772d17a2cc5134a5\",\"type\":\"tcp\",\"config\":{\"port\":45000,\"service\":\"test\"}}", Dir:false, Expiration:(*time.Time)(nil), TTL:0, Nodes:etcd.Nodes(nil), ModifiedIndex:0x3, CreatedIndex:0x3}

----------------------------------------------------------------------
FAIL: tcp_test.go:97: S.TestAddTCPRoute

tcp_test.go:107:
    r := addTCPRoute(c, l, portInt)
setup_test.go:114:
    c.Fatalf("timeout exceeded waiting for %s %s", event, id)
... Error: timeout exceeded waiting for set

discoverd process exited: signal: killed
etcd process exited: signal: killed
2015/01/26 02:46:17 http: proxy error: read tcp <IP_ADDRESS>:53355: connection reset by peer
2015/01/26 02:46:19 http: proxy error: read tcp <IP_ADDRESS>:34675: connection reset by peer
2015/01/26 02:46:35 no backends found
2015/01/26 02:46:36 no backends available
2015/01/26 02:46:36 no backends available
OOPS: 28 passed, 1 FAILED

https://s3.amazonaws.com/flynn-ci-logs/20150126024502-4900a97d-build-5fcf7750155101573da49727c12caab71a06fe77-2015-01-26-02-46-45.txt
/cc @bgentry @benburkert

Might be related to #893.

Another example of this: https://s3.amazonaws.com/flynn-ci-logs/20150217192510-2c130ba2-build-23cbdfa586dcf001d9d0c0353686f6148eb236d7-2015-02-17-19-36-07.txt
There is also a panic in TestInitialTCPSync, not sure if it is related.

Another instance: https://s3.amazonaws.com/flynn-ci-logs/20150221183924-86218d4b-build-1441f2e7cc255e2ff4a3e35248e9a4bcc69968b0-2015-02-21-18-50-10.txt
Also has the TestInitialTCPSync panic; seems it might be caused by state overflowing between tests.


make shell install script

for mac/linux

Can you tell me in details what needs to be done ?

A shell script that downloads the binary of this projects and put its somewhere on the path.
So you can install this without having to manually setup all this stuff, oh-my-zsh example:


Is posible if can read from buffer?

Hi, whether do we have any way can allow read hdf from buffer memory ?
Example :
H5File.OpenRead(byte[] buffer)

You can wrap the buffer into a memory stream and then read from it:
var data = new byte[] { xxx };
using var stream = new MemoryStream(data);
using var h5File = H5File.Open(stream);

thank you, look nice,.


Remove commented code

I think we can merge this. I think we can add the removed code back when we have a mature Python Backend. We can probably also remove/replace the Python tab on the GUI with C or X64 tab.


Can't move the cursor to some parts of the code

Issue Description
Every 2 or 3 challanges (sometimes even in a row but I can't seem to understand what's the cause) I can't access far parts of the lines that I need to edit. The cursor just jumps to the next line or doesn't move at all.
I recorded a video of the behaviour: https://drive.google.com/open?id=0B_zGmwAmO3kCQ0VhWXZ0TlFoaEk
Interesetingly enough, reloading the page doesn't help. But the reset button on the left does. Furthermore, the bug also dissapears if I shrink the browser window (is stil gone after maximizing it afterwards).
Browser Information
Tried both the newest Chrome and Firefox.

Operating System: Windows 7 64-bit

@m3h0w Thanks for reporting an issue. This is a duplicate of a long-known issue with the editor. See: https://github.com/freeCodeCamp/freeCodeCamp/issues/7847. It will be fixed when the beta site replaces the current site.

For everyone having the mouse cursor problem, try Chrome Canary, works fine


Atom fails to maximize using F11 on Linux

On Linux hitting F11 will toggle windows between current size and maximized. Atom does not behave this way on Linux.
The desktop I am using is Kubuntu 16.10.
Atom 1.15.0

Thanks for reaching out!
We require the template to be filled out completely on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
Thanks for understanding and meeting us half way :grinning:


fix: full text search with limit may return an incorrect results

fix #3264
this happens when limit is set and there are some docs the their estimated score is greater than threshold

Codecov Report
Attention: Patch coverage is 98.55072% with 1 line in your changes missing coverage. Please review.

Project coverage is 79.07%. Comparing base (022135b) to head (b7cce72).
Report is 1 commits behind head on main.

Files with missing lines
Patch %
Lines

rust/lance-index/src/scalar/inverted/wand.rs
0.00%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3284      +/-   ##
==========================================
+ Coverage   79.00%   79.07%   +0.06%
==========================================
  Files         246      246
  Lines       86900    87539     +639
  Branches    86900    87539     +639
==========================================
+ Hits        68655    69221     +566
- Misses      15377    15454      +77
+ Partials     2868     2864       -4

Flag
Coverage Δ

unittests
79.07% <98.55%> (+0.06%)
:arrow_up:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add Tinybird components & colors

Remove Icons, Tooltip
Remove unused classes
Rename theme colors
fix config typing

@dmytro-tinybird going to say no to the introduction of the internal Tinybird theme package unless it's made public, so that Mockingbird UI can continue to be run standalone; are there any changes in this PR that should be saved and brought into the FakerV8 PR?


Add a CI workflow for changelog

Just a reminder to complete the CHANGES.md file.
Shamelessly stolen from gospel repository.

comment.md

Maybe, as there are multiple packages in the repo, we should indicate which package is concerned in the CHANGES.md entries. Or is it overkill?


@bentley/imodeljs-native 2.19.7

Need to run rush change.
Sorry, forgot to push it

Superseded by 2.19.8.


Setting the default user to "nobody" in the Dockerfile.

Changing CMD to ENTRYPOINT so that commandline arguments can be passed to unifi_exporter without specifying the binary. I think the docker-compose.yaml file won't work without this.

Seems fine to me, but I don't use the Dockerfile at all. Merging since we can revert later if needed.

For sure this Dockerfile works properly while specifying the additional arguments on the commandline of "docker run" (and runs properly as nobody). It's the docker-compose.yaml file I'm not sure about, I don't use them.


Order onr comparisons by cardinality for short-circuiting

This is a small optimization that carries forward the idea set in onrEqual. The comparisons are swapped so that the higher cardinality check happens first to allow for faster short-circuiting.

I have read the CLA Document and I hereby sign the CLA

recheck


Move/add strings and translations

Translations and pushing checkout v4 (as pipeline breaks on 3.6.0)
(and apologies for commit message being stuck)

Might need improvements (which also can be done later) - but this should mainly clean up the backlog of the PRs


Inconsistent weather data for Haarlem, the Netherlands

I've been using this service for my current city (Haarlem), but while I get the right values for the in-day breakdown, the summary looks totally out of place.
I cross-referenced the data with other weather services and something doesn't add up. Check out the attached screenshot for example:

Maybe I don't get something, but the summary (or is it instant/real-time?) view is completely misaligned with the day forecast. It doesn't check with other weather services, and it's also plain wrong when I step outside the house literally.
I did check the weather in Harlem, New York and seemed close enough for me to suspect there might be some sort of a name collision?

Seeing the same issue with Bengaluru, India.
It showed up as Mist yesterday, and Haze today. Both around noon-time. Definitely incorrect.

Ref: https://wttr.in/Bangalore

Seems to be the same case. Changed title to be more generic.

Potentially duplicate of https://github.com/chubin/wttr.in/issues/819

same thing for Kyiv today.
Google, Apple, OpenWeather all just fine with 17-19 celsius
while gettin in console from request https://wttr.in/kyiv?format=%t;%h
11*;66%

Incorrect data for Los Angeles and other cities. I recommend everyone to stop using this application until the owner @chubin  addresses the issue.

People, don’t panic, that is just the data that we get from the upstream provider, and has with the app itself nothing to do. Use format=j1 option to see the raw data. Please let me know if the raw data is correct, and it is displayed incorrectly. We will try to fix the issue then. Otherwise, the only solution here is to drop this upstream provider and switch to another one. I already paid for the data for the many months upfront, so it won’t happen without serious arguments (and again: if the problem is not with the data, but with the app itself, please let me know)

Thanks. I ceased to panic.
It is a problem with the upstream provider then, because the data is also off with format=j1

Don't know if it is helpful, but I see the same in Oslo:
Weather report: 59.92565,10.76668

     \  /       Partly cloudy
   _ /"".-.     +13(12) °C
     \_(   ).   ← 19 km/h
     /(___(__)  10 km
                0.0 mm
                                                       ┌─────────────┐
┌──────────────────────────────┬───────────────────────┤  Thu 15 Jun ├───────────────────────┬──────────────────────────────┐
│            Morning           │             Noon      └──────┬──────┘     Evening           │             Night            │
├──────────────────────────────┼──────────────────────────────┼──────────────────────────────┼──────────────────────────────┤
│    \  /       Partly cloudy  │    \  /       Partly cloudy  │  _`/"".-.     Thundery outbr…│    \  /       Partly cloudy  │
│  _ /"".-.     26 °C          │  _ /"".-.     +29(27) °C     │   ,\_(   ).   25 °C          │  _ /"".-.     16 °C          │
│    \_(   ).   ↓ 5-6 km/h     │    \_(   ).   ↙ 8-9 km/h     │    /(___(__)  ↗ 3-4 km/h     │    \_(   ).   → 1-3 km/h     │
│    /(___(__)  10 km          │    /(___(__)  10 km          │     ⚡‘‘⚡‘‘  9 km           │    /(___(__)  10 km          │
│               0.0 mm | 0%    │               0.0 mm | 0%    │     ‘ ‘ ‘ ‘   0.0 mm | 0%    │               0.0 mm | 0%    │
└──────────────────────────────┴──────────────────────────────┴──────────────────────────────┴──────────────────────────────┘

Current time is noon (12:16), so it is quite off as you can see (13 vs 29 degrees)


Clarification regarding PCI-Express requirements

I have multi-GPU setup, one NVidia GTX 960 and another AMD RX 560 Baffin 16CU/1024 cores.
Due to size restrictions I can plug GTX 960 only to PCIEX16 slot and RX 560 to PCIEX4 slot
MB is GA-B150M-D3H-DDR3  All of the PCI Express slots conform to PCI Express 3.0 standard and according to the documentation PCI Express 3.0 should be enough
In such a configuration ROCM driver complains on atomics on PCIEX4 slot - not allowing to run neither OpenCL nor HIP - dmesg reports PCI rejects atomics
Notes:

ROCM driver works when the RX 560 installed in PCIEX16 slot but for this purpose I need to remove GTX 960 card.
When AMDGPU-PRO drivers where installed AMD RX 560 worked on PCIX4 slot - but it does not support hip (and so hipCaffe/tensorflow-hip)

Questions:

why atomics are not supported it should be PCI Express 3 connection?
Is it possible to enable RX 960 on second PCIEX4 slot without atomics

Setup:

Using ROCM 1.9.2 from official repositories
Running Ubuntu 16.04, Kernel 4.15

It's possible that the second PCIe slot on your motherboard is actually connected to the B150 chipset. Looking at the CPU support list for this motherboard, even the highest-end processor it supports only includes 16 PCIe gen 3 lanes directly connected to the CPU. As such, my suspicion is that PCI1, PCI2, and PCIEX4 are connected through the chipset.
The datasheet for the B150 chipset makes no mention of support for the optional atomics extension to PCIe 3.0. Note that PCIe 3.0 compatibility does not mean that a slot supports PCIe 3.0 atomics, as those are a feature that PCIe 3.0-compliant devices (including bridges) need not support.
It is not possible at this time to enable gfx8 devices without the use of PCIe atomics. See this discussion for many more details about this.

The problem that putting all mid range GPU like 560/570/580 out of scope for large range of applications including use of eGpu via thunderbolt. Additionaly it is somthing not obvious when you get a GPU and discover that hip/rocm would not work in perfectly normal PCIE slot, like NVIDIA gpus do

I'll note that I've given a much longer discussion of why we do not currently support gfx8 without PCIe atomics in the linked thread. I appreciate your request and the reasons behind it, but I don't plan to rehash the same discussion here.


Typing check using signature, allowing to use object methods as RPC endpoints

@trollfot thank you for the PR! :raised_hands:
Unfortunately, the tests are failing and I think "inspect" is not a good solution for thorough type checking. I have tried it before but forgot why didn't use it :disappointed:


Reverted logo path, updated gitattributes

Updated gitattributes to describe images as binary, list out specific extensions for special treatment
Signed-off-by: keithbloomfield<EMAIL_ADDRESS>

This PR restores the path to the project logo in the README, and prevents git from corrupting image files

Yeah, we need to specify that "text" files have LF line endings.
https://stackoverflow.com/questions/9976986/force-lf-eol-in-git-repo-and-working-copy
Last I checked Git-Bash on WIndows auto converts local files to CRLF, which is fine, until they run ./bin/start -b, so we need to make sure that any sourcecode has forced LF.

Restored text eol=lf


Add ACE medical "doctor" status switching from lockers

title

Shouldn't we place the medical skills selection on a different place than gear selection ?
Alternatively, should we manage the skill based on the steam ID ?

As already discussed, another place to change medic status is a waste of time for players, as they will go to briefing and then know what is the mission and roles. They will go to the locker to get equipment, and change medic status if needed. Then they will rendez-vous to the meeting point.
As for steam id, it is not yet decided to go this way.

Done https://github.com/team-gsri/GSRI-Opex/commit/0f8dc3bbe8284d290d652049d9b9ffbddbf8e60b


Alignment of item in card in index.html.

Cards items were not aligned properly in index.html.
Before :-

After :-

@TejasVarshney Please resolve the conflicts in the mentioned files. Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you!


Update cloud-services-guestos-msrc-releases.md

Updated with September release details

@ibthampy : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver Would you be able to help approve this merge too?

@ibthampy This content should be updated in the private repo. I'm sending you email now. We'll close this PR.
cc @DavidCBerry13 @tfosmark


ui-jq

Can you please put back the ui-jq module?
It is still useful in my opinion...

ui-jq module is still available from this repository as it used to be. If there are new patches/version of ui-jq, then we might be thinking about splitting it. But if there is not a strong demand, it will be left as is.

I can't find "ui-jq" module in the new release 3.0...
Which is the correct way to get it using bower ?

My last comment wasn't very clear. ui-jq is not maintained anymore, but you can still get the latest version available via:
bower install angular-ui-utils#jq-0.2.3

This is clear now ;-)
Thanks


devcontainer is using Python 2.7 for pre-commit

Report
When I try to execute pre-commit run --all-files using devcontainers I get an error related with outdated python version (and also it's unmaintained)
There is also another problem related with the End Of Line in bash script because my local computer is windows and the container is UNIX.
An unexpected error has occurred: CalledProcessError: command: ('/usr/bin/python', '-mvirtualenv', '/root/.cache/pre-commit/repoJzxre_/py_env-python2.7', '-p', '/usr/bin/python')
return code: 1
expected return code: 0
stdout: (none)
stderr:
    Traceback (most recent call last):
      File "/usr/lib/python2.7/runpy.py", line 163, in _run_module_as_main
        mod_name, _Error)
      File "/usr/lib/python2.7/runpy.py", line 111, in _get_module_details
        __import__(mod_name)  # Do not catch exceptions initializing package
      File "/usr/local/lib/python2.7/dist-packages/virtualenv/__init__.py", line 3, in <module>
        from .run import cli_run, session_via_cli
      File "/usr/local/lib/python2.7/dist-packages/virtualenv/run/__init__.py", line 13, in <module>
        from .plugin.activators import ActivationSelector
      File "/usr/local/lib/python2.7/dist-packages/virtualenv/run/plugin/activators.py", line 6, in <module>
        from .base import ComponentBuilder
      File "/usr/local/lib/python2.7/dist-packages/virtualenv/run/plugin/base.py", line 5, in <module>
        from backports.entry_points_selectable import entry_points
    ImportError: No module named entry_points_selectable

Check the log at /root/.cache/pre-commit/pre-commit.log

Expected Behavior
I expect to be able to run  pre-commit run --all-files  and get the linting status
The UNIX EOL should be persisted, ignoring the OS in *.sh (Bash in Windows should still recognize it)
Actual Behavior
I am not able to run  pre-commit run --all-files  and get the linting status (and I need to run it using the GitHub Action) xD
All EOL change according the OS
Steps to Reproduce the Problem

Create a fresh branch from master
Run Visual Studio Code with support for dev containers
Once the container starts, execute pre-commit run --all-files

Anything else?
I'm fixing it for myself, I can create the PR in a few days if it's useful :)

More or less nothing on the planet still supports Python 2. You'll need to use Python 3. 2.7 was end-of-lifed January 1 2020.

Sure, but the devcontainer is using python 2, that's the problem :) https://github.com/kedacore/keda/blob/main/.devcontainer/Dockerfile#L73-L74
Also, there is another problem with the EndOfLine in bash scripts

That's a more specific issue :)

Sorry, my fault
I thought that the term dev container is enough explicit, but probably I was assuming more than I should assume

Ouch, I have just seen that you created the PR first. Up to you to close mine if you want but please, take in consideration the .gitattributes file :)


Dwarfeh: new patterns not detected by destructor code

Needed for Dwarf exception handling support. If I'd had this test case in the test suite before, it would have saved me a day :-)

Auto-merge toggled on


Migration to newer ESLint version

Modify the eslintrc.tt template for ESLint 2.x.x. See issue
PS: In the template package.tt, the version of eslint-plugin-react is now also up-to-date (version 4.3.x).

@sungwoncho

Thanks. Will let you know when released.


Multiple form types for a single widget

The "widgets" that allow searching can be expanded to more than just check boxes and range searches. Each data field could actually have multiple ways of search on it (for example, both range and check boxes for Cassini rev_no). In addition:
From Mark:
When we get around to dealing with asteroid and comet targets, which we are sort of committed to dealing with as part of our PDART program, we will have to handle target names in multiple ways too--we can't have a checkbox for every asteroid, but we also don't want to get rid of the checkboxes for planets and moons, so the UI will have to provide both options.
This was actually part of the thinking behind OPUS2--that there could be a variety of UI widgets inside any of those constraint boxes.
[Old content, see #354, #581]
For example, I would love to see options to select ranges for things like incidence, emission, phase, latitude and longitude using sliders. Wouldn't it be cool if you could see a map of Enceladus and then draw a box around the region you want to search for?]

Phases of implementation:

Allow sub-categories in checkbox fields
Allow checkbox fields to be hidden
API supports string searching on checkbox fields
API supports string completion on checkbox fields
UI supports adding a string-search version of the widget for checkbox fields in addition to the normal widget

Phase 1, 2, and 3 are finished (although there is no way to search on alias strings).
The details of #4 are yet to be determined.


Add absolute_import to init

In one of my projects in python 2.7, I ran into a problem of distinguishing between a module in the package vs a package of the same name in the python path. I wanted to import an external package, but since I had a module in the package directory of the same name (that was a bad naming convention),
it was importing that module.
from  module import MyClass # is module a module in the package or a package in the python path ?

The solution was to add the line
from __future__ import absolute_import
# Now the imports are clear
from module import MyClass # imports MyClass from package called module
from .module import MyPackageClass # import MyPackageClass form package module called module

This also allows you to import a module within package in the following way which I think is one of the recommended ways in python 3
from . import module

Looks like maybe this was sorted out on Python 2.7? https://www.python.org/dev/peps/pep-0328/

@arokem
Checkout a repository https://github.com/rbiswas4/quick_absimport_test which I made quickly for the purpose. You can install this using the setup file through the usual commands. The point is that this package contains a module [mynumpy.py] (https://github.com/rbiswas4/quick_absimport_test/blob/master/test_imports/mynumpy.py)  which has a class called numpy. In test.py I want to write a function that prints out numpy.arange(0., 1., 0.1) in ways that it actually calls numpy.arange(0., 1., 0.1) or the module tests_imports.numpy.arange in clearly distinguishable ways. (Obviously, it would be stupid for someone to name a class by the well-known and highly used name of numpy, but this distinction is not so clear if the class in question is under development by multiple user in a multiple packages doing somewhat similar things.)
With the files in the repository which include the from __future__ import absolute_import on python 2.7
(Anaconda : Python 2.7.11 |Anaconda 2.1.0 (x86_64)| (default, Dec  6 2015, 18:57:58)) ,
I can do this correctly. Without the future statement, only the package numpy class works, the call to numpy ends up with a strange error:
Using future
# to call the numpy we know
>>> import test_imports
>>> test_imports.check(0.)
x
[ 0.   0.1  0.2  0.3  0.4  0.5  0.6  0.7  0.8  0.9]
3.0

While if I replace the import numpy
statement with (an uncommented version of) the previous line
to call the package class numpy:
# Call the numpy class defined in *this package itself*

>>> import test_imports
>>> test_imports.check(0.)
x
HELP
(0.0, 1.0, 0.1)
3.0
>>>

This is all good.
If I get rid of the  from __future__ import absolute_import
statement at the top of test.py,
# Try calling the numpy we know
>>> import test_imports
>>> test_imports.check(0.)
x
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/rbiswas/.local/lib/python2.7/site-packages/test_imports/test.py", line 10, in check
    print(np.arange(0., 1., 0.1))
AttributeError: 'module' object has no attribute 'arange'
>>>

though the attempt to call the numpy defined within this package works correctly.
While there are probably other ways of obtaining the correct behavior, adding this absolute_import seems like a useful thing to do to minimize potential confusion.

I'm convinced. Where do we put this? In every .py file, or just in the __init__.py file?

You should put it in all files so that 2 and 3 are consistent.

#41

Probably resolved through #41, but feel free to reopen if I misunderstand.


Implementation Questions

Hi there,
I'm developing a generative MIDI app called Midinous and there have often been requests to write out MIDI files of the sequences that play. Due to the unpredictable nature of the sequences in Midinous, writing a complete MIDI file all at once (like the example provided) isn't really practical. Ideally I'd like to write to the stream writer as the sequence plays and once the sequence stops, build the MIDI file for use elsewhere.
Every note that plays in Midinous has a note, velocity, channel, and duration value, but the duration is specified as a decimal so not cleanly tied to a grid. What's more, events can be "off grid" as well, meaning notes may be aligned to fractional numbers like 1.333_ quarter notes.
Do you have any tips to implement Hearn.Midi into my project, or is this not the intended implementation of the library?
Thanks!

I see in the example that the rendering of the song is done as one big concatenation. Can these methods be called independently to increase the length of the stream? If so, I think I could use your suggestion and build to the file as the sequence plays and write it out only at the end.

Yes, these are equivalent
midiStreamWriter
        .WriteStartTrack()
            .WriteNote(0, MidiNoteNumbers.C4, 127, NoteDurations.WholeNote);

and
midiStreamWriter.WriteStartTrack();
midiStreamWriter.WriteNote(0, MidiNoteNumbers.C4, 127, NoteDurations.WholeNote);

You will need to keep a reference to the midiStreamWriter and call Dispose() when done.  The example is scoped within a using statement which disposes automatically when complete.

Fantastic! Thank you so much. I'm grabbing the NuGet and I will let you know how this works out.

Your library has worked well, taking the guesswork out of writing MIDI files with an easy-to-use library! I did modify MidiStreamWriter to remove all of the duration and note length logic, including the enums, as I simply didn't need those due to my current implementation, and I added a method for writing note off messages. Closing as satisfied!


Feature/update to polkadot v0 9 22

Updated Substrate to polkadot-0.9.22

Is it intentional to have feature/update-to-polkadot-v0-9-20 as a target instead of the dev?
Yep as we did not merge v0.9.20 to master-cere.


[CARBONDATA-2985]Fix issues in Table level compaction and TableProper…

Issue :-

If 2nd Level compaction is 1 like 2,1 or 6,1 then only 1st time compaction is done subsequent compaction are ignored . ( like if 2,1 is given then only 0.1 is segment is created and other segments are ignore forever )
Table level compaction does not support ,0 as 2nd level compaction value but system level compaction supports same.

Solution :-

if 2nd level compaction value is 1 then user does not want 2nd level compaction at all which  mean 2nd level compaction can be set to 0.
remove check to support 2nd level compaction as 0 in table level.

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?
NA
[ ] Any backward compatibility impacted?
NA
[ ] Document update required?
NA
[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
Added

How it is tested? Please attach test report.

Is it a performance related change? Please attach the performance test report.
Any additional information to help reviewers in testing this change.
FT is added

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
NA

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/670/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8933/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/865/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/709/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8972/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/904/

LGTM


Expose scheduled event ID on PendingNexusOperationInfo

Why?
So the UI can correlate the info from describe with the history event.

What is the use case? Does this use case not also apply to pending activity info?

Pending activity info can be correlated to the history event using activity_id. There's nothing to link the nexus operation today.

Activity ID is (usually) generated by us, I wonder if the analogue here is request ID instead event ID?

Activity ID is (usually) generated by us, I wonder if the equivalent here is the recently-added request ID instead of event ID?

Both can be used, the event ID is easier to work with though.

My only concern is that we haven't traditionally used event identifiers outside of history. Maybe this is showing a need for an identifier unrelated to history for correlation reasons. When talking about a Nexus operation on a workflow, will we expect users to use event identifier as opposed to the other things (activities, child workflows, updates, etc) that have non-history-specific identifiers?

My only concern is that we haven't traditionally used event identifiers outside of history. Maybe this is showing a need for an identifier unrelated to history for correlation reasons. When talking about a Nexus operation on a workflow, will we expect users to use event identifier as opposed to the other things (activities, child workflows, updates, etc) that have non-history-specific identifiers?
What will be put in OTel tracing spans for this operation end-to-end?

The thing with nexus operations is that callers don't get to set the ID, it's only available in async operations and set by the handler. Request ID is allocated by the server so doesn't have much meaning to the user.
The otel spans will be propagated via nexus_header on the command/event, we don't put IDs on otel spans AFAICT: https://github.com/temporalio/sdk-typescript/blob/a6849afe39cb51cafdc933e217c227206d4edacd/packages/interceptors-opentelemetry/src/workflow/index.ts#L80.

I see, yeah we don't put outbound activity/child identifiers. I do think we should consider a way for users to identify Nexus operation invocations besides their scheduled event ID in history. But if others are ok with this being the identification identifier, no prob.

I see, yeah we don't put outbound activity/child identifiers. I do think we should consider a way for users to identify Nexus operation invocations that isn't the scheduled event ID in history. But if others are ok with this being the identifier, no prob.

I don't think it hurts to add this field right now though. If we do come up with such an identifier, we could add that later too.
In the meantime users can get more information in the history event, such as request ID and inputs.


Create library of datasets to be re-used instead of generating a new dataset in each benchmark pipeline

Pr #78 goes in that direction, but we'll need to separate the generation pipeline from the training pipeline, TBD


component update

component update in order to render only that what has been v

closing this PR in favor of : https://github.com/donseba/go-partial/


Release Google.Cloud.AIPlatform.V1Beta1 version 1.0.0-beta11

Changes in this release:
New features

Add workbench_runtime and kernel_name to NotebookExecutionJob (commit 83da91f)
Add new Status field to DeployedModel in Endpoint (commit 5846fb6)
Add new RequiredReplicaCount field to DedicatedResources in MachineResources (commit 5846fb6)
Support streaming and multi class methods in Reasoning Engine v1beta1 API (commit 6bbb600)
Introduce HybridSearch and Ranking configuration for RAG (commit 15ac66e)
Introduce VertexAiSearch integration for RAG (commit 15ac66e)
Trajectory eval metrics added to evaluation service proto (commit 1c70f59)
Add CustomEnvironmentSpec to NotebookExecutionJob (commit 92e884c)

Documentation improvements

A comment for field encryption_spec in message .google.cloud.aiplatform.v1beta1.NotebookExecutionJob is changed (commit 83da91f)
A comment for field class_method in message .google.cloud.aiplatform.v1beta1.StreamQueryReasoningEngineRequest is changed (from steam_query to stream_query) (commit d2f716f)
Added support for multiple class_methods in QueryReasoningEngine (commit 6bbb600)
Added support for StreamQueryReasoningEngine (commit 6bbb600)
A comment for field api_key_config in message .google.cloud.aiplatform.v1beta1.JiraSource is changed (commit 15ac66e)
A comment for field partial_failure_gcs_sink in message .google.cloud.aiplatform.v1beta1.ImportRagFilesConfig is changed (commit 15ac66e)
A comment for field partial_failure_bigquery_sink in message .google.cloud.aiplatform.v1beta1.ImportRagFilesConfig is changed (commit 15ac66e)
A comment for field rag_file_parsing_config in message .google.cloud.aiplatform.v1beta1.ImportRagFilesConfig is changed (commit 15ac66e)
A comment for field source_uri in message .google.cloud.aiplatform.v1beta1.RagContexts is changed (commit 15ac66e)
Fixed typo for field use_strict_string_match in message .google.cloud.aiplatform.v1beta1.ToolParameterKVMatchSpec (commit 1c70f59)

Created release for Google.Cloud.AIPlatform.V1Beta1-1.0.0-beta11


Host guest env reorg

This is a major reorg of the host crate -- so major it even renames the root crate to stellar-contract-env! We'll need to rename this repo if we go along with this.
Along the way, the following (with motives):

Switches from one crate with cfg-controlled bits to 4 crates in a dependency diamond. This is to make both configs testable without having to toggle the cfg; this is silly but a consequence of how feature flags work in rust, there's no way to have two turned on at once at the root so we need two separate crates (and another with the common parts). This is actually the recommended approach to mutually exclusive features. Bleh.
Do some renaming for fewer compound words. We need a name for the interface between host and guest as well as the implementations of the interface as seen from host and guest perspectives. I tried a lot of words and the simplest I cold find was Host, Guest and Env for the interface they both implement.
Make a handle-containing-and-environment type (actually two: EnvVal and EnvObj) so that operator overloading, Ord and Eq can work at least on EnvVal and EnvObj types. The latter are needed so that we can reach through an Env to resolve or compare objects deeply when needed; the former are needed because we store them in HostMaps and such and need a uniform type, and also because even in the guest we need some type to return from functions that return maybe-an-object like the i64 injection function.
Implement Ord and Eq on the EnvVal and EnvObj types, in terms of the presence of an Env, by reaching through the Env. This should let users in the guest deep-compare objects the same way the host does.
Implement the Guest type as a unit struct, defining Object = EnvObj<Guest> when cfg(guest).
Implement the Host type as the former HostContext and Object = EnvObj<Weak<Host>> when cfg(host)

This very roughly seems to all hold together, but it's late and I didn't get it entirely .. ergonomic. In particular I expect @leighmcculloch will want to fiddle a lot with the conversion functions and maybe actually add a wrapper around Rc<Host> and implement Env for that too (or instead of WeakHost) for when users manually construct or pass-around a strongly-referenced Host. So there's stuff to do. But overall I think this gets us a fair ways along in the direction we were discussing today.

We'll need to rename this repo if we go along with this.

Requesting a rename in https://github.com/stellar/terraform/pull/787.


Disable mouse acceleration

I'm on my laptop and mouse acceleration causes the camera to flick. To get a more accurate position and make the experience more pleasant on laptops, I tweaked the example.

Thanks @FlokiTV for your contribution! Seems reasonable to do, so LGTM 🚀

it's a small fix, I'm getting familiar with the repo to contribute more!


JS: extractUrlsWithIndices does not extract URL when exclamation point is ending char in the path

The extractUrlsWithIndices function does not properly recognize the full URL if the ending character in the path is an exclamation point (!). With a URL as such:
https://www.domain.com/locale/#!?key1=value1&key2=value2
The twitter-text library will only recognize 'https://www.domain.com/locale/#' as being the URL instead of 'https://www.domain.com/locale/#!?key1=value1&key2=value2'.

I found a workaround with a combination of encoding exclamation marks and adding an extra / so there is never a trailing exclamation mark


DCOS-39832 elected leader tab

Testing
Launch a multi-master cluster. Look at the data it shows, go on the control panel for your cluster (AWS) and kill the machine that is, running.
Go to: http://localhost:4200/#/nodes/masters
Trade-offs
GET_MASTER quickfix
The state store only updates the MesosState once it
parsed the entire state, which means that we wait seconds
on a large cluster, before we are able to show a piece
of information, we get almost instantly (GET_MASTER).
GET_MASTER parser elected and started
I added code to the parser to add elected and started time to the master_info object. Not quite sure I like adding those there but do not have any better idea. I considered another structure like master_events with both. I also do not like to add them flatten to the mesos state we keep
Toggle elected and started time
We dont have this behaviour as a reusable component, so I will create a ticket (ticket here) to extract this component to the UI-kit, we can then use it later.
Dependencies
Depends on #3118 for tabs

This build is broken, but it is building here #3127

continued in #3127


how to save the chart in local?

how to save the chart in local?

it's simple:
// your graph
file, _ := os.Create("demo.png")
_ = graph.Render(chat.PNG, file)

Render's second parameter is an io.Writer


Error installing dependencies when forking the repo

The following occurs when trying to run npm install as stated in the Getting Started section of the README
npm ERR! fetch failed https://github.com/FreeCodeCamp/loopback.git#fix/no-password
npm WARN retry will retry, error on last attempt: Error: fetch failed with status code 406
npm ERR! fetch failed https://github.com/FreeCodeCamp/loopback.git#fix/no-password
npm WARN retry will retry, error on last attempt: Error: fetch failed with status code 406
npm ERR! fetch failed https://github.com/FreeCodeCamp/loopback.git#fix/no-password
npm ERR! Linux 3.16.0-4-amd64
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install"
npm ERR! node v0.12.0
npm ERR! npm  v2.5.1

npm ERR! fetch failed with status code 406
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <http://github.com/npm/npm/issues>

This seems to be caused by pointing to the repository directly when it should be defining the package tarball url. Changing it to FreeCodeCamp/loopback#fix/no-password seems to proceed with normal installation.
Referencing: https://github.com/npm/npm/issues/7708#issuecomment-84640042

Yes I have had this issue also. The fix i found is to npm install again then do bower install and the npm install again.This (for me at least) fixes the problem.

@alexlincoln I you're happy that your fix works you can go ahead and close this issue and leave the pull request! Nice work by the way :+1:


Added Truck tour problem (Hackerrank)

Have you read the Contributing Guidelines on Pull Requests?
Yes, I have.
Description
Added this problem to Queue folder  since this problem uses Circular Queue.
Link to the problem: https://www.hackerrank.com/challenges/truck-tour/problem
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.

Related Issues or Pull Requests
Fixes issue no. #806
Fill up the DCP form
If you are a part of DCP 21, fill up this form after the PR is merged.
The links provided in the form will be checked thoroughly, if someone tries fooling around, strict actions will be taken against the individual.

I'm sorry, I didn't realized there are empty files.

Thank you @noviicee

@smv1999 now can you please remove the changes requested label from this PR.


How to parse streamresult in   List convertStreamResultFirebase(

Im trying to pasrse the streamresult in   List convertStreamResultFirebase(
How can I do that?

so the streamResult in the parameter you will got it from my widget:
it is dynamic, because it will store the type you passed to the widget through the getBookingStream
In the implementation of my widget it looks like this:
` StreamBuilder(
stream: widget.getBookingStream(
start: startOfDay, end: endOfDay),
builder: (context, snapshot) {
if (snapshot.hasError) {
return widget.errorWidget ??
Center(
child: Text(snapshot.error.toString()),
);
}
                  if (!snapshot.hasData) {
                    return widget.loadingWidget ??
                        const Center(child: CircularProgressIndicator());
                  }

                  ///this snapshot should be converted to List<DateTimeRange>
                  final data = snapshot.requireData; **//THIS IS THE MOST IMPORTANT LINE**
                  controller.generateBookedSlots(
                      widget.convertStreamResultToDateTimeRanges(
                          streamResult: data));`

So the final data=snapshot.requireData will be the fetched data which you store in your firebase project, and this will be passed to the convertStream function, which you will receive on your end.
Probably, this streamResult will be a list of object, but for firebase specific, it is a queryresult, in my case:

But it depends on how you store your data in firebase.
so your data is with firebase a QueryResult, but if you did not use withConverter method like I did in the example you also need to deserialize the data, because by default it is a map:
 List<DateTimeRange> converted = []       for (var i = 0; i < data.size; i++) {         final item = data.docs[i].data();         converted.add(DateTimeRange(start: (item.bookingStart!), end: (item.bookingEnd!)));       }   return converted;

Accoding to the firebase example, what is the ApiRepository?


Setting to set the verifier arguments

Now it is possible to set the arguments passed to the compiler or the language server, but to my knowledge I cannot set the arguments passed to Dafny when it verifies the file (every time I save). Is there currently a way to do this that I'm missing?

Right now I need to set allocated to a different value but I think that a more general setting would be more useful.

This is now fixed with the last release of VSCode plugin and Dafny 3.10. You can now specify the same options as the dafny CLI in the settings


GO:0004972 NMDA glutamate receptor activity - new parent

Could
GO:0004972 NMDA glutamate receptor activity
be
is_a GO:0005234 extracellular-glutamate-gated ion channel activity?
In fact, they may even be equivalent...
Currently they are in parallel branches of GO:0005230 extracellular ligand-gated ion channel activity.
It should also be
is_a  GO:0022843 voltage-gated cation channel activity (mainly Ca2+, but also Mg2+)
Birgit

GO:0004972 NMDA glutamate receptor activity
be
is_a GO:0005234 extracellular-glutamate-gated ion channel activity?
Yep.  But they are not equivalent, as there at least to other types apart from NMDA with quite different activities.
These things are pretty complicated:
Also lets Na+ and K+ out.  Mg2+ is an inhibitor (part of the mechanism of voltage gating).  Activation happens when Gly and hyperpolarisation co-incide.
I've actually been working on this as an example of a compound process.  Here're my notes:
https://docs.google.com/document/d/1tBhuA_-xgbeqehZD9NdZWbHy8ruM9YFnJQ2Qpcw8lmQ/edit#

That's coincidence! NMDARs are my next family! I guess you are looking at these as part of the synapse project.
You are right, I forgot about the AMPA and kainate Rs. So I guess it's actually their common parent,
GO:0004970 ionotropic glutamate receptor activity
that should be
is_a GO:0005234 extracellular-glutamate-gated ion channel activity
And why has
GO:0005234 extracellular-glutamate-gated ion channel activity
not parent
is_a  GO:0008066 glutamate receptor activity?
It is parent to
GO:0004970 ionotropic glutamate receptor activity,
the parent of
GO:0004972 NMDA glutamate receptor activity (and AMPA & kainate glutamate receptor activity)
I think it links to the ticket (number ?) discussion whether a ligand-gated channel is a receptor as the community/literature regards them as receptors. I think you where going to add some axioms, then the reasoner should pick that up, right?
Fascinating biology (or rather - biophysics)! I never did anything neurological and I'm learning as I'm going along. Membrane potentials are still a somewhat black box.

Fixed this up some time ago as part of general work on ionotropic ion channels.

(Note - likely to be refined further & modeled slightly differently as part of MF refactoring in 2017)


Dev

separate loops for separate file types are brought back and added return value insertion it them

sorry was gonna create a pr for my fork


ServiceAccountCredential with valid access token failing with 401 when passed as HttpClientInitializer

I have created a service account client with domain wide delegation.  It has the role of Project Owner, and I have added the scopes:

https://www.googleapis.com/auth/admin.directory.orgunit
https://www.googleapis.com/auth/admin.directory.user

I have a very simple test program that is based off the example presented here except that example uses user credentials, not a service account.
I have modified the example (see code below) to try to make it work with ServiceAccountCredential.  So far, I have not been able to make it work.  When I attempt to use the Orgunits service, it throws an exception from ClientServiceRequest.cs#116 with the text {"Google.Apis.Requests.RequestError\r\nLogin Required [401]\r\nErrors [\r\n\tMessage[Login Required] Location[Authorization - header] Reason[required] Domain[global]\r\n]\r\n"}
This is puzzling to me, b/c the service account credential I load from the file looks fine to me, and in order to ensure it is working note that I am forcing it to authenticate and get an access token.  Indeed, it succeeds just fine in getting an access token and prints it on the console every time.
It isn't until I pass the credential as the HttpClientInitializer and try to use the service that it ends up failing.  My understanding was that this HttpClientInitializer was to be used by the service to authenticate, but when I breakpoint and step the code, I walk it right up to the point where it issues the HTTP request, and there are no headers added for authentication, and indeed it never makes an attempt to authenticate or use the access token that has already been populated.
I'm at a loss here.  I must be missing something fairly simple?
using Google.Apis.Admin.Directory.directory_v1;
using Google.Apis.Admin.Directory.directory_v1.Data;
using Google.Apis.Auth.OAuth2;
using Google.Apis.Services;
using System;
using System.Collections.Generic;
using System.Threading;

namespace DirectoryQuickstart
{
    class Program
    {
        static string[] Scopes = { DirectoryService.Scope.AdminDirectoryUser, DirectoryService.Scope.AdminDirectoryOrgunit };
        static string ApplicationName = "slea-crm";
        static string Secret = "client_secret.json";
        static string CustomerId = "redacted";

        static void Main(string[] args)
        {
            ServiceAccountCredential credentials = GoogleCredential.FromFile(Secret).CreateScoped(Scopes).UnderlyingCredential as ServiceAccountCredential;
            // Force auth to get access token
            if (credentials.RequestAccessTokenAsync(CancellationToken.None).Result)
            {
                Console.WriteLine("access token: " + credentials.Token.AccessToken);
            }

            // Create Directory API service.
            var service = new DirectoryService(new BaseClientService.Initializer()
            {
                HttpClientInitializer = credentials,
                ApplicationName = ApplicationName,
            });

            // Fetch Org Units
            OrgunitsResource.ListRequest request = service.Orgunits.List(customerId: CustomerId);
            IList<OrgUnit> orgUnits = request.Execute().OrganizationUnits;

            if (orgUnits != null && orgUnits.Count > 0)
            {
                foreach (var orgUnit in orgUnits)
                {
                    Console.WriteLine("{0} ({1})", orgUnit.Name, orgUnit.OrgUnitPath);
                }
            }
            else
            {
                Console.WriteLine("No orgunits found.");
            }
            Console.Read();
        }
    }
}

My end goal is to primarily use the OrgUnit and User service by a server-side app, that will allow the server to generate a federation of OrgUnit and Users underneath them.  This is why I set it up as a domain wide delegate and went the route of using a service account credential.

I'm actually pretty certain that all I need to do is something like the following to make this demo orgunit service request succeed:

request.Headers.Add("Authorization", "Bearer " + credentials.Token.AccessToken);

OR

request.URL += "?access_token=" + credentials.Token.AccessToken

But I don't have access to the internals of the request that is built, so I cannot do it, and I see no other way via the API to trigger this to happen.

How did you verify the Authorization header was not being added to the request? You definitely shouldn't need to do this yourself with a ModifyRequest action.
I don't know much about domain delegation, but looking at the delegation guide here you may need to set the service account User: GoogleCredential.FromFile(Secret).CreateScoped(Scopes).CreateWithUser(user)
Which version of the Google.Apis.Admin.Directory.directory_v1 package are you using?

How did you verify the Authorization header was not being added to the request?

I just step-debug the code, and walk it right up to the point where it issues the System.Net.HttpRequest and looked at the Headers which were blank.
After more investigation, I'm still trying to resolve whether the issue is in the Client / my misuse of the API? / or on the backend services.
The version of the package I have is Google.Apis.Admin.Directory.directory_v1 v<IP_ADDRESS>0
One thing I've been trying is to use the ServiceAccountCredential to generate an access token, dump it to the console, then copy/paste it into a curl command.  That still isn't working.
So, I've followed the instructions here and walked through the steps again and again and again.
If I use the Try This API side bar, it works when I use the my_customer customer id and my root org's customer id C03rfhq7f  But if I run it in curl using an access token I get from the .NET code above...
$curl -vvv -H "Authorization: Bearer ya29.REDACTED" "https://www.googleapis.com/admin/directory/v1/customer/C03rfhq7f/orgunits?type=all"
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to www.googleapis.com (<IP_ADDRESS>) port 443 (#0)
* TLS 1.2 connection using TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256
* Server certificate: *.googleapis.com
* Server certificate: Google Internet Authority G3
* Server certificate: GlobalSign
> GET /admin/directory/v1/customer/C03rfhq7f/orgunits?type=all HTTP/1.1
> Host: www.googleapis.com
> User-Agent: curl/7.54.0
> Accept: */*
> Authorization: Bearer ya29.c.REDACTED
>
< HTTP/1.1 401 Unauthorized
< Vary: X-Origin
< WWW-Authenticate: Bearer realm="https://accounts.google.com/", error=invalid_token
< Content-Type: application/json; charset=UTF-8
< Date: Fri, 04 May 2018 15:32:02 GMT
< Expires: Fri, 04 May 2018 15:32:02 GMT
< Cache-Control: private, max-age=0
< X-Content-Type-Options: nosniff
< X-Frame-Options: SAMEORIGIN
< X-XSS-Protection: 1; mode=block
< Server: GSE
< Alt-Svc: hq=":443"; ma=2592000; quic=51303433; quic=51303432; quic=51303431; quic=51303339; quic=51303335,quic=":443"; ma=2592000; v="43,42,41,39,35"
< Accept-Ranges: none
< Vary: Origin,Accept-Encoding
< Transfer-Encoding: chunked
<
{
 "error": {
  "errors": [
   {
    "domain": "global",
    "reason": "required",
    "message": "Login Required",
    "locationType": "header",
    "location": "Authorization"
   }
  ],
  "code": 401,
  "message": "Login Required"
 }
}
* Connection #0 to host www.googleapis.com left intact

I'm a bit at a loss here.

@chrisdunelm that link you posted is interesting https://developers.google.com/admin-sdk/directory/v1/guides/delegation#instantiate_an_admin_sdk_directory_service_object but do you know if there's a .NET example for it?  Trying to map the code from Java/Go/Python -- I'm fluent in all of those, but obviously the classes/types are totally different so I'm studying the API to try to port it.

Huh, I'm a little dense today.  That was it.  This was the missing piece of the puzzle.  Thank you @chrisdunelm
GoogleCredential.FromFile(Secret).CreateScoped(Scopes).CreateWithUser(user)

Glad you've sorted it out :)


chore: Process payment error

lnd is returning a 200 status code even if the payment was not successfully. This will check if the payment_error field is set, and if so report the payment as error.

bors r+


Fix double Error wrapping

Since $error already is an instance of the Error class this results in wrapping an Error by another Error which is useless and complicates error handling. In my opinion this should be fixed.

Coverage increased (+0.001%) to 86.311% when pulling 7efd5f935c985298a7e833731500efbb109f718b on enumag:patch-8 into 2dcc81e4d7f01470e66c72a2a1db8050c8b0b70a on webonyx:master.

Actually, since Error::$message is public maybe we can just prepend $messagePrelude and not create new Error instance at all. Thoughts?

Actually, since Error::$message is public maybe we can just prepend $messagePrelude and not create new Error instance at all. Thoughts?

Not anymore, actually. Make sure your fork is up to date with the latest changes.

Ah, right, I'm locally on 0.13 still. In that case we can keep this PR as is.

Just realized I can't update to 0.14 yet due to overblog dependency. Is the v0.13 branch still maintained? Should I rebase this PR to target it instead?

You can open another PR that targets https://github.com/webonyx/graphql-php/tree/0.13.x


Plans to implement skiprows in readtable?

I saw mention of adding support for skiprows in #292 but that was back in 2013. I'm using version 0.8.5 and am getting this message when I try to use it:
ERROR: ArgumentError: Argument 'skiprows' is not yet supported.
Speaking for myself I'd find this feature to be very useful. Does anybody have plans for finishing it?

Actually readtable will be deprecated soon in favor of CSV.jl, so no further development of it will take place here.


Upon live reload of the app, make any windows re-appear at the last positions they'd been dragged to

Quite annoying that it keeps snapping back to the dead centre of the screen.

Closing, as this is specific to NodeGUI rather than Svelte NodeGUI.


fix absl import issue #25282

fix issue when using:
from tensorflow.python.platform.googletest import mock

there is code in tensorflow/python/platform/googletest.py
from absl.testing.absltest import *

but absl.testing.absltest have mock object after absl>=0.4.0
this case is occur after commit 87cc788e1cc
@revan
87cc788e1cc (Revan Sopher           2019-01-23 13:14:20 -0800
https://github.com/tensorflow/tensorflow/issues/25282

Thanks for the fix! I think we should be able to bump this all the way to 0.7.0, after this commit: https://github.com/tensorflow/tensorflow/commit/661103ed87cdf0f5a792100b543006b71d62697a
Can you try 0.7.0?

@revan OK,I have rebased this commit to absl-py==0.7.0

@revan Hello,is this PR in progress

Hi @zjjott, I've retriggered the tests and am waiting for a green signal.

@rthadur submit failed due to unrelated linter error, can we force?

@rthadur submit failed due to unrelated linter error, can we force?

@revan we will wait until it gets merged internally.


Postgres

Branch to stay dormant as POC for now. Will need new work to Postgres things when we're ready to move to CF anyhow.


Jade transition to Pug

Jade is renamed to Pug.  Although dependencies on Jade work, the code is not maintained from what I have gathered as all the new changes will be under the pug package.  There are some syntax differences, so moving from Jade to Pug may require verifying that all the views still work.
Refrence: https://github.com/pugjs/pug#rename-from-jade

please search before creating issues

https://github.com/mochajs/mocha/issues/2200#issuecomment-235493858


add the ability to upload files to GCS using a service account

Sorry, wrong pull request target :)


Can't open .scpt files

Now the extension can only open applescripts which format is .applescrit, but can't open and edit .scpt file.

Why do you say “Now” when this package never had support to open .scpt and never will unless Microsoft introduces an load equivalent to onWillSaveTextDocument. VSCode does not open binary files, let alone run them through a decode function prior to opening.
You can request (or upvote) such an API method at Microsoft/vscode.


Fix SampleChannelBass reporting wrong state

This PR fixes SampleChannelBass reporting wrong state.

Calling Stop no longer makes IsAlive false.
Calling Play after disposal no longer makes enqueuedPlaybackStart true, so that Playing won't be true (this can be tested with TestDoesNotRestart because SampleChannel is disposed after the end of playback with #6397).

Do not merge this yet. What should we do for looping samples? Stop no longer makes IsAlive false, but looping sample will be a zombie because it doesn't end. Should we make SampleChannels not resumable?

So what happens when the channel is played, manually stopped, and the ISample is just left to live (as we never dispose samples in game)? It sounds like in this case samples would never be disposed, no?

Yeah, my idea is to consider Stop() the same as playback end. That way everything is safe except when you set ManualFree = true, which is the only case you need to make a conscious effort to handle it.


Unexpected behavior of focused window with picom

I have use picom to blur all currently unfocused windows. In sowm the focused window gets blurred as well, which makes it very difficult to distinguish between focused and unfocused windows, especially when you can't have borders that change colors.

Yes. This is likely due to no EWMH/ICCCM support. Borders can also be applied via a patch to sowm. I don't personally use a compositor so this isn't something I've noticed or cared about fixing.


Parsing of mach-o symbol tables, generation of debug maps, and resolution through object files

Notes to self: I ran into what I think is a libdwarf issue so temporarily blocked on https://github.com/davea42/libdwarf-code/issues/217. I also noticed odd behavior with universal binaries so I'll need to dig into that more later.


[DOC] Add changelog page to docs

https://sphinx-github-changelog.readthedocs.io/en/stable/

@mathdugre not sure the workflow here is working, maybe the GitHub Action is only for PRs and not issues?


Login as user doesn't work

After login as user (not admin) crash:
in laravel.log:
[2019-04-13 09:49:40] local.ERROR: Argument 1 passed to App\Album::App\{closure}() must be an instance of Illuminate\Database\Eloquent\Builder, instance of Illum
inate\Database\Query\Builder given, called in /var/www/html/lp1204/vendor/laravel/framework/src/Illuminate/Database/Query/Builder.php on line 309 {"exception":"[
object] (Symfony\\Component\\Debug\\Exception\\FatalThrowableError(code: 0): Argument 1 passed to App\\Album::App\\{closure}() must be an instance of Illuminate\
\Database\\Eloquent\\Builder, instance of Illuminate\\Database\\Query\\Builder given, called in /var/www/html/lp1204/vendor/laravel/framework/src/Illuminate/Data
base/Query/Builder.php on line 309 at /var/www/html/lp1204/app/Album.php:316)

used last commit (3589248)
same behavior at mysql and postgresql

I will double check that.

Should be fixed now (as I can also connect via another User).
Thank you for bringing this case up. I will need to add a new test for that. :)


Stretch columns to fit width

Is it possible to stretch the columns (evenly) to fill the width of the parent container?
Also, how do you remove the empty column at the end of the grid?

Empty column? You mean the leftover space if your columns are not wide enough to fill?
There is no built in fill mode. I considered adding it but it adds a lot of extra work. Let me see if I can come up with a simple solution.

@jassmith We ran into this issue as well, is it complicated to add support for percentage width? 🙏

The problem is how this interacts with resizing. I think they are basically mutually exclusive.

Im struggling to imagine the dynamic model here, as you resize elements all the percentage elements sizes change which changes the start position of the column which creates an unintended resize delta on your next mouse motion. The only realistic solution here seems to be that resizing columns should "lock" percentage columns at their current size for the duration of the resize.

Fixed by #406 but there is still work to be done

You can see the oddball behavior with the resizing I was talking about.

@jassmith If I don't allow resizing, can I define columns width in percentage? Is it supported?


fix: namespace import for Experimental

What changes are proposed in this pull request?
fix namespace import for Experimental, from jdk.jfr.Experimental to org.apache.spark.annotation.Experimental

/azp run

Codecov Report

Merging #1780 (d1a8a8b) into master (8dc4a58) will decrease coverage by 24.22%.
The diff coverage is n/a.

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

Thank @mhamilton723 to force merging it


Update button on Instances

Feature Request
Feature Information:

There should be a button that updates the instance.
(Switch off instance -> Update -> Switch on instance)
For example, I can't update the maininstance. To be able to update it I have to turn off the instance. But then I can't get into the interface, because I, who manage AMP, don't have direct access to the Linux server.
https://www.bilder-upload.eu/bild-62285f-1554845951.png.html
I confirm:

[x] that I have searched for an existing feature request matching the description.

This already exists in ADS in the right-click menu of an instance.

You can only press it if the instance is off, but there should be a way to update it even if the instance is on.

e.g. I can't update ADS01

ADS can't update itself (yet) - that's just done via ampinstmgr update ADS, the update option should be available while it's running but I'll check into that.


Page layout for k8s.io/docs/concepts/workloads/controllers/deployment/ is broken

Problem:
Page layout for Deployment page is broken.
Proposed Solution:
Fix or rollback to previous version
Page to Update:
k8s.io/docs/concepts/workloads/controllers/deployment/

Hi @supirman
Can you describe the symptoms you saw?

I'm glad it's not just me - I'm seeing quite a few formatting issues today across numerous pages in chromium/firefox as well. Here are some screenshots:
 - e.g the "blockquote" stuff everywhere; text boxes are not looking right either. Source
I happened to look at this page today and yesterday, it used to be that the different distributions (ubuntu vs Fedora/RHEL) were tabs you could select, similar to here, that's now murked. The code snippets are also full of extra whitespaces for some reason.
As an aside - really loving these docs, thanks so much for maintaining them, they're excellent!

that is how Deployment page looks like right now for me in chrome in French language:

the reason is broken html: there is a bunch of elements that are on the same level with

, although they should be inside of it
check it:
good html (English version):

broken html (French version):

only English and Chinese versions of the page have correct html in them.
English version also had broken html yesterday.

@ArtemAlagizov if you log the bug with the French page into its own issue, that lets the French localization team pick it up and work on it. (If you want to submit a PR directly yourself, that is also OK).

@sftim got it, thanks! i will submit a separate bug issue

fixed in #20946
/close

@k8s-ci-robot thank you robot

/close
fixed in #20920


allow switch out of buffertools

this package relies on buffertools, which uses node-gyp
on windows, that kinda sucks, as there is no dist/executable shipped in the buffertools npm, it does a rebuild from source
this basically means that flowdock has a dependency on python2.7 and visual studio
thats compounded when you look at hubot-flowdock
So, for the functionality this seems to be giving, thats quite a heavy barrier
is there any way you could either:

remove buffer tools for something that works out of the box across platforms
make it pluggable, so I can at least include a local copy

Replacing the JSON streaming with eventsource could work. Windows users seem to be a quite small minority, so this isn't a very high priority for us. That said, we welcome any pull requests to tackle this.

Err.. Ok.  We might look at trying to plug that for you..
On Fri, Feb 13, 2015 at 10:35 pm, Ville Lautanala <notifications<EMAIL_ADDRESS>wrote:
Replacing the JSON streaming with eventsourcehttps://github.com/aslakhellesoy/eventsource-node could work. Windows users seem to be a quite small minority, so this isn't a very high priority for us. That said, we welcome any pull requests to tackle this.

Reply to this email directly or view it on GitHubhttps://github.com/flowdock/node-flowdock/issues/20#issuecomment-74339396.

Sorry – I didn't mean to write off Windows support. We should get this fixed eventually, but I don't expect anyone to immediately jump to work on this. In the meantime, pull requests either by replacing our current uses of buffertools with pure javascript or some other methods would be appreciated especially since we don't have Windows development machines and our knowledge about Node on Windows is pretty much based on issues like this.

Ok – will push something up if one of us gets a chance. For now we are getting round it by including a built version of buffertools and changing our gitignore so that gets pulled with the rest of our hubot repo


Vulnerabilities because of body-parser library - 1.20.2

Hi Team,
While using "@cap-js-community/odata-v2-adapter", I am getting a vulnerability for "body-parser" library version 1.20.2.
Could you please update if/when a new update can be expected to fix this issue?
Thanks
Aman

Thanks for the information. We will release a new version beginning of next week.
We have an open dependency to body-parser, so once you update your package-lock you will already get the latest patch release 1.20.3 solving the vulnerability.


PM4 Support

Here soon, PM3 support is to be dropped, so this plugin will have to be updated for PM4, which I will only support from here on out, even if PM3 is not dropped yet.

An update to this. After further consideration, 3.x support will be dropped in favor of 4.x support.


Unhandled rejection Error on fetch for app/main.js

i get this when running npm install :
Unhandled rejection Error on fetch for app/main.js at file:///C:/wamp/Projects/1                 4.auth with express/app/main.js
Error: ENOENT: no such file or directory, open 'C:\wamp\Projects\14.auth                  with express\app\main.js'
at Error (native)
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\ch                 okidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for fsevents@                 1.0.15: wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"                 })

I tried npm install 3 more times an it just worked.

persistence pays off :)


Redid favicons using instructions in Docsy docs

Removed favicon partial
https://www.docsy.dev/docs/adding-content/iconsimages/#add-your-favicons
The easiest way to do this is to create a set of favicons via http://cthedot.de/icongen (which lets you create a huge range of icon sizes and options from a single image) and/or https://favicon.io/, and put them in your site project’s static/favicons directory. This will override the default favicons from the theme.
Generated new favicons using  http://cthedot.de/icongen

When I copy/paste the deploy preview link, Slack still displays the bowl of oatmeal instead of the Armory shield.  shrug


Node Creation Bug

When attempting to seed my db, I came across this bug:
ivy_league = Community.create(name: "Ivy League")

%W( Yale Harvard Cornell ).each do |name|
	School.create(name: name, child_of: [ivy_league])
end

School.create(name: "The College of New Jersey")

This will incorrectly mark "The College of New Jersey" node as a child of ivy_league, while creating that node first:
School.create(name: "The College of New Jersey")

ivy_league = Community.create(name: "Ivy League")

%W( Yale Harvard Cornell ).each do |name|
	School.create(name: name, child_of: [ivy_league])
end

produces the correct result.

Oh, wow, this is an interesting bug.  We'll definitely take a look, thanks!

If I'm reading it correctly this is the culprit. It sets an @_association_attributes hash at the class level, {association_name: target_obj}, then it's keeping the value for all subsequent passes. I think I see a fix for it, I'll poke at it now.

Just pushed a fix.  Are you running off master?

Yep, running off master. Seems to be behaving differently now - though not as intended:
tcnj = School.create(name: 'The College of New Jersey')

ivy_league = Community.create(name: "Ivy League", slug: "ivy-league")

%W( Yale Harvard Cornell ).each do |name|
	School.create(name: name, slug: name.parameterize, child_of: [ivy_league])
end

The above now adds tcnj as a child of ivy_league three times: https://cloudup.com/cOHDuY7aqhN
While moving the creation of tcnj to the bottom creates 3 self-referential child_of relationships on ivy_league: https://cloudup.com/cEiHzjXY-d0
Odd :/

Damn it. I wonder if that is my fault. Can you lock your Gemfile at gem 'neo4j', github: 'neo4jrb/neo4j', ref: '765ec93f37225049ddd88c959a95f237337e097e' and tell me if it works?

Works :D

Hmm... That's no good. I'll have to add your example to Brian's tests and see if I can pin down what's causing it.

Oh no! I'm very new to neo4j, but ping me if I can help

Thank you! Using master, reporting issues, and following up with us is a huge help.

:smile: :+1:

Do you have a sec that you could pop into http://gitter.im/neo4jrb/neo4j? I can't reproduce this but might be overlooking something.

I'll be heading out for the night a few minutes but I'll be signed into Gitter all day tomorrow if you have a minute. In the meantime, try a bundle update neo4j with your gemfile not locked to that one commit. Nothing is jumping out at me that'd cause what you're reporting and I can't recreate it, so I wonder if it's just a weird glitch, but let's poke at it more to be sure.

I poked around a bit, ran brew update && brew upgrade (which installed a new neo4j homebrew package), created a rails app from scratch, and it seems to be working now. Will follow up tommorrow.
Though I did come across a few other quirks: https://github.com/neo4jrb/neo4j/issues/763 and https://github.com/neo4jrb/neo4j/issues/760

yep, looks like i was running neo4j 2.2.0 - all systems go on 2.2.1

I don't think it was a 2.2.0 issue, seems like other wires crossed somewhere. Glad it's all working, though! Was nervous there for a sec. ;-)


Improper Parsing of Chocolatey Parameter

What You Are Seeing?
Failure to parse Chocolatey parameter
What is Expected?
Correctly parsing Chocolatey parameter
How Did You Get This To Happen? (Steps to Reproduce)
boxstarter script.ps1
where script.ps1 only contains
choco install -y cmake --installargs "ADD_CMAKE_TO_PATH=System"
However,
choco install -y cmake --installargs="ADD_CMAKE_TO_PATH=System" works fine, which makes me think it's simply a parsing issue.
Output Log

Full Log Output

Boxstarter starting Calling Chocolatey to install --installargs. This may take several minutes to complete...
Boxstarter starting Calling Chocolatey to install --installargs. This may take several minutes to complete...
Installing the following packages:
cmake;ADD_CMAKE_TO_PATH=System
By installing you accept licenses for the packages.
cmake v3.15.2 already installed.
Use --force to reinstall, specify a version to install, or try upgrade.
ADD_CMAKE_TO_PATH=System not installed. The package was not found with the source(s) listed.
If you specified a particular version and are receiving this message, it is possible that the package name exists but the version does not.
Version: ""
Source(s): "https://chocolatey.org/api/v2/"

this is somehow related to #265 - Boxstarter definitely needs some work on it's command parsing, as it simply tries to install an array of packages after a choco install command.
note, this should only applies to "Boxstarter packages", not when using BoxstarterShell.

note that #404 does NOT resolve this bug.

@mwallner You happy to take this one on?

In theory Call-Chocolatey already handles arbitrary arguments passed to chocolatey.
function Call-Chocolatey {
    param(
        [string]$command,
        [string[]]$packageNames=@('')
    )
    $chocoArgs = @($command, $packageNames)
    $chocoArgs += Format-ExeArgs $command @args
    Write-BoxstarterMessage "Passing the following args to Chocolatey: $chocoArgs" -Verbose

.. relates to #145

Oh, the code in https://github.com/chocolatey/boxstarter/blob/master/Boxstarter.Chocolatey/Chocolatey.ps1#L301 only checks for = separated parameters, imo this is the correct place for fixing this.

I've just written a couple of tests and verified: there definitely is a bug in choco command parsing that leads to argument reordering before those args are passed to choco.exe.
i.e.:
choco Install -y pkg --source blah --installargs "ADD_CMAKE_TO_PATH=System"

I've added a couple of Debug-print statements to function chocolatey:
Write-BoxstarterMessage "chocolatey wrapper: PSBoundParameters: '$($PSBoundParameters.Keys | % { "`r`n  $($_) -> $($PSBoundParameters[$_])" })`r`n'" -Verbose
Write-BoxstarterMessage "chocolatey wrapper: args: $($args | % { $_ })" -Verbose

which gives me
VERBOSE: Boxstarter: chocolatey wrapper: PSBoundParameters: '
  command -> Install
  packageNames -> --source
'
VERBOSE: Boxstarter: chocolatey wrapper: args: -y pkg blah --installargs ADD_CMAKE_TO_PATH=System

that finally leads to
Install --source -y pkg blah --installargs ADD_CMAKE_TO_PATH=System

being passed to choco.exe.
(which is bad, as '--source blah' needs to be kept in correct order, also the quotes of the '--installargs'  parameter are lost).
After all, function cinst, function choco, function cup as well as function chocolatey need to be fixed to keep the original parameter order.

Build is failing in AppVeyor. Once that is resolved we can get to merging this.
Awesome work @mwallner. Thanks for picking this up and completing it!