SyntaxError: Unexpected end of JSON input

When create a issue from Mattermost  using /jira command
e.g.
/jira issue create this is a test
Can't search or select Assignee/Label/Assigner/Owner/....
It shows  "SyntaxError: Unexpected end of JSON input"

no cule

@rogerhsu1121 Could you please share some more information:

What is the version of the plugin you're using
Are you using Jira cloud or Jira server and what version of Jira server if that's the case?
Could you please share any console logs from the network tab of the relevant AJAX requests that are unsuccessful with along their responses (to investigate the error)

@rogerhsu1121 Could you please share some more information:

What is the version of the plugin you're using
Are you using Jira cloud or Jira server and what version of Jira server if that's the case?
Could you please share any console logs from the network tab of the relevant AJAX requests that are unsuccessful with along their responses (to investigate the error)
Thanks for reply

JIRA Plugin Version is Jira (jira - 3.2.4)
I use Jira Server (Version v7.2.7)
I got some log from Mattermost  ->System console->Server logs

{"timestamp":"2023-04-19 09:51:22.442 +08:00","level":"error","msg":"websocket.NextReader: closing websocket","caller":"app/web_conn.go:830","user_id":"bc4jme54sp8mjmgbhayacnozmc","error":"read tcp <IP_ADDRESS>:8065-><IP_ADDRESS>:50854: i/o timeout"}
{"timestamp":"2023-04-19 10:01:09.442 +08:00","level":"info","msg":"SimpleWorker: Job is complete","caller":"jobs/base_workers.go:88","worker":"ExpiryNotify","job_id":"yawhbxm1tjy8dq5rxd1b9oec5o"}
{"timestamp":"2023-04-19 10:11:09.520 +08:00","level":"info","msg":"SimpleWorker: Job is complete","caller":"jobs/base_workers.go:88","worker":"ExpiryNotify","job_id":"674oq6h6hbgfzj65sqew6nxmue"}
{"timestamp":"2023-04-19 10:13:42.986 +08:00","level":"error","msg":"Received an unknown request as a Jira plugin:\nGET /plugins/jira/plugins/servlet/streams?use-accept-lang=true&streams=user+IS+DennisChen&local=true&maxResults=10 HTTP/1.1\r\nHost: <IP_ADDRESS>:8065\r\nAccept-Encoding: gzip,deflate\r\nAccept-Language: en-us, en;q=0.9, en-us;q=0.8, en;q=0.7, *;q=0.5\r\nConnection: Keep-Alive\r\nUser-Agent: Apache-HttpClient/4.4.1 (Java/1.8.0_102)\r\n\r\n","caller":"app/plugin_api.go:974","plugin_id":"jira"}
{"timestamp":"2023-04-19 10:21:09.599 +08:00","level":"info","msg":"SimpleWorker: Job is complete","caller":"jobs/base_workers.go:88","worker":"ExpiryNotify","job_id":"uifswjw4uffrbqegzwujwzj8ac"}
{"timestamp":"2023-04-19 10:23:26.174 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"failed to return any results","Path":"/api/v2/get-search-users","Method":"GET","query":"instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com&project=TIT2&q="}
{"timestamp":"2023-04-19 10:23:26.197 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"403 Forbidden: : Request failed. Please analyze the request body for more details. Status code: 403","Path":"/api/v2/get-search-autocomplete-fields","Method":"GET","query":"fieldName=Labels&fieldValue=&instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com"}
{"timestamp":"2023-04-19 10:23:26.221 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"403 Forbidden: : Request failed. Please analyze the request body for more details. Status code: 403","Path":"/api/v2/get-search-autocomplete-fields","Method":"GET","query":"fieldName=Requirement+Version&fieldValue=&instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com"}
{"timestamp":"2023-04-19 10:23:26.259 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"failed to return any results","Path":"/api/v2/get-search-users","Method":"GET","query":"instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com&project=TIT2&q="}
{"timestamp":"2023-04-19 10:23:26.348 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"failed to return any results","Path":"/api/v2/get-search-users","Method":"GET","query":"instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com&project=TIT2&q="}
{"timestamp":"2023-04-19 10:23:29.417 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"failed to return any results","Path":"/api/v2/get-search-users","Method":"GET","query":"instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com&project=TIT2&q=ssss"}
{"timestamp":"2023-04-19 10:23:31.543 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"failed to return any results","Path":"/api/v2/get-search-users","Method":"GET","query":"instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com&project=TIT2&q=ss"}
{"timestamp":"2023-04-19 10:23:33.483 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"failed to return any results","Path":"/api/v2/get-search-users","Method":"GET","query":"instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com&project=TIT2&q=ggg"}
{"timestamp":"2023-04-19 10:23:35.701 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"failed to return any results","Path":"/api/v2/get-search-users","Method":"GET","query":"instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com&project=TIT2&q=s"}
{"timestamp":"2023-04-19 10:23:37.124 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"failed to return any results","Path":"/api/v2/get-search-users","Method":"GET","query":"instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com&project=TIT2&q=ssssss"}

@rogerhsu1121 Could you please share some more information:

What is the version of the plugin you're using
Are you using Jira cloud or Jira server and what version of Jira server if that's the case?
Could you please share any console logs from the network tab of the relevant AJAX requests that are unsuccessful with along their responses (to investigate the error)
Thanks for reply

JIRA Plugin Version is Jira (jira - 3.2.4)
I use Jira Server (Version v7.2.7)
I got some log from Mattermost  ->System console->Server logs

{"timestamp":"2023-04-19 09:51:22.442 +08:00","level":"error","msg":"websocket.NextReader: closing websocket","caller":"app/web_conn.go:830","user_id":"bc4jme54sp8mjmgbhayacnozmc","error":"read tcp <IP_ADDRESS>:8065-><IP_ADDRESS>:50854: i/o timeout"}
{"timestamp":"2023-04-19 10:01:09.442 +08:00","level":"info","msg":"SimpleWorker: Job is complete","caller":"jobs/base_workers.go:88","worker":"ExpiryNotify","job_id":"yawhbxm1tjy8dq5rxd1b9oec5o"}
{"timestamp":"2023-04-19 10:11:09.520 +08:00","level":"info","msg":"SimpleWorker: Job is complete","caller":"jobs/base_workers.go:88","worker":"ExpiryNotify","job_id":"674oq6h6hbgfzj65sqew6nxmue"}
{"timestamp":"2023-04-19 10:13:42.986 +08:00","level":"error","msg":"Received an unknown request as a Jira plugin:\nGET /plugins/jira/plugins/servlet/streams?use-accept-lang=true&streams=user+IS+DennisChen&local=true&maxResults=10 HTTP/1.1\r\nHost: <IP_ADDRESS>:8065\r\nAccept-Encoding: gzip,deflate\r\nAccept-Language: en-us, en;q=0.9, en-us;q=0.8, en;q=0.7, *;q=0.5\r\nConnection: Keep-Alive\r\nUser-Agent: Apache-HttpClient/4.4.1 (Java/1.8.0_102)\r\n\r\n","caller":"app/plugin_api.go:974","plugin_id":"jira"}
{"timestamp":"2023-04-19 10:21:09.599 +08:00","level":"info","msg":"SimpleWorker: Job is complete","caller":"jobs/base_workers.go:88","worker":"ExpiryNotify","job_id":"uifswjw4uffrbqegzwujwzj8ac"}
{"timestamp":"2023-04-19 10:23:26.174 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"failed to return any results","Path":"/api/v2/get-search-users","Method":"GET","query":"instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com&project=TIT2&q="}
{"timestamp":"2023-04-19 10:23:26.197 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"403 Forbidden: : Request failed. Please analyze the request body for more details. Status code: 403","Path":"/api/v2/get-search-autocomplete-fields","Method":"GET","query":"fieldName=Labels&fieldValue=&instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com"}
{"timestamp":"2023-04-19 10:23:26.221 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"403 Forbidden: : Request failed. Please analyze the request body for more details. Status code: 403","Path":"/api/v2/get-search-autocomplete-fields","Method":"GET","query":"fieldName=Requirement+Version&fieldValue=&instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com"}
{"timestamp":"2023-04-19 10:23:26.259 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"failed to return any results","Path":"/api/v2/get-search-users","Method":"GET","query":"instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com&project=TIT2&q="}
{"timestamp":"2023-04-19 10:23:26.348 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"failed to return any results","Path":"/api/v2/get-search-users","Method":"GET","query":"instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com&project=TIT2&q="}
{"timestamp":"2023-04-19 10:23:29.417 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"failed to return any results","Path":"/api/v2/get-search-users","Method":"GET","query":"instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com&project=TIT2&q=ssss"}
{"timestamp":"2023-04-19 10:23:31.543 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"failed to return any results","Path":"/api/v2/get-search-users","Method":"GET","query":"instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com&project=TIT2&q=ss"}
{"timestamp":"2023-04-19 10:23:33.483 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"failed to return any results","Path":"/api/v2/get-search-users","Method":"GET","query":"instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com&project=TIT2&q=ggg"}
{"timestamp":"2023-04-19 10:23:35.701 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"failed to return any results","Path":"/api/v2/get-search-users","Method":"GET","query":"instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com&project=TIT2&q=s"}
{"timestamp":"2023-04-19 10:23:37.124 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"failed to return any results","Path":"/api/v2/get-search-users","Method":"GET","query":"instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com&project=TIT2&q=ssssss"}
seems can't solve this problem

Thanks @rogerhsu1121 for the extra information. We'll investigate the issue and we'll get back as soon as possible.

@m1lt0n I tried to reproduce the error but I was unable to do so. In the server logs shared by @rogerhsu1121, he is getting 403 (forbidden) error responses from the Jira APIs. So, I think it is some permission issue on the Jira side. I tried to change the permission schema, user's groups and project permissions related to the assignee field but the error was not getting reproduced.
@rogerhsu1121 Is there any specific permission you are using on your Jira side that you think might be causing the issue?

@raghavaggarwal2308 One issue I see here is that the frontend is expecting to parse JSON, but is instead parsing a string that does not contain valid JSON. In this case, is the server responding with an error, but not in JSON form? If we can have the server return an error in a structure that the frontend can understand, then we can properly show the error. Even if the error to be shown is "Error fetching data" or something like that
Then there is the issue of "why did this error occur" that we are discussing here. The error isn't specific enough to tell us why it's happening. I don't have any ideas on what about our usage of the API would cause this. It looks like we're doing our best to log the error we get from Jira's API. I don't think there's much we can do about the error itself.
Relevant logs for reference:
{"timestamp":"2023-04-19 10:23:26.197 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"403 Forbidden: : Request failed. Please analyze the request body for more details. Status code: 403","Path":"/api/v2/get-search-autocomplete-fields","Method":"GET","query":"fieldName=Labels&fieldValue=&instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com"}
{"timestamp":"2023-04-19 10:23:26.259 +08:00","level":"error","msg":"ERROR: ","caller":"app/plugin_api.go:974","plugin_id":"jira","Status":"500","Error":"failed to return any results","Path":"/api/v2/get-search-users","Method":"GET","query":"instance_id=https%3A%2F%2Ftpe-jira2.fihtdc.com&project=TIT2&q="}
I've seen "Please analyze the request body for more details" many times before, though I'm not sure it's always associated with 403. I'm thinking the 403 may even be used incorrectly by Jira in this case, though once again we don't have much insight on the error itself so this is just a guess on my part.


allow None for IntervalMetadata.upper_bound

This is to address #1450 , where the upper bound of the interval features (or seq length) is unknown at the time the object is created.
It adds a get_right_bound() function in the cython code to get the max value from the interval tree. Setting upper_bound to None is now more efficient than setting it to a actual value because it avoids the repetitive bound check for adding interval to the intervalmetadata. It only checks once when upper_bound is set.
@ebolyen or @jairideout , this is ready for review.

Thanks @RNAer! See my comment on #1450 about a potential change to this API. Also, please add a changelog note.

@jairideout and @ebolyen , added the alternative IntervalMetadata constructor for upper bound update purpose (from line 535-563). One thing I am not sure is the API, whether it should be a static method (as currently defined) or a normal method (def clone(self, upper_bound) -> new IntervalMetadata). It seems very subtle to me. any suggestion?

@RNAer, a few test failures btw

Thanks for looking at it, @wasade ! I am aware of the test failure. wanna get feedback on the alternative constructor before fixing them.

What about adding an optional from=None parameter to the existing constructor, where from accepts an IntervalMetadata object?
def __init__(self, upper_bound, from=None):
    ...

That sounds good too - does the copy works in __init__? Anyway, I still need a function to do the heavy-lifting construction because the _rebuild_tree decorator needs to be applied to it (line 536, 553). Prone to define it as normal class method instead of a static

I was thinking the constructor would behave similar to __copy__ by 1) setting self._upper_bound and 2) looping over the Interval objects and calling self.add(). add() already takes care of the error-checking and can be reused in the constructor.

ah, sorry, you are right. I was overthinking and forgetting add() does checking.

I think this is ready for final review. thanks for all the comments!

Thanks, @jairideout ! I replied to a few of your comments - would you mind responding? I will address all your other comments.

Yep, I just followed up

@jairideout , does changelog really need update? Because he API changes is not new and compatible backwards. If so, then which section should I put it?

@jairideout , all comments are addressed except for changelog

@jairideout , does changelog really need update? Because he API changes is not new and compatible backwards. If so, then which section should I put it?

Yes. As detailed in the code review guidelines and in the pull request template, all public-facing changes are documented in the changelog. There are several sections in the changelog to categorize changes; for example, "features", "bug fixes", "performance enhancements", "backwards-incompatible changes", etc. Any change visible to a user should be documented in the changelog, not just backwards-incompatible changes. This is why we have several sections to categorize the types of changes present in a release.
For this PR, there are two public-facing changes. Both are new features, so they would go under the "features" section of the changelog:

skbio.interval.IntervalMetadata constructor has a new keyword argument, copy_from, for creating an IntervalMetadata object from an existing IntervalMetadata object with specified upper_bound.

skbio.interval.IntervalMetadata constructor allows None as a valid value for upper_bound. An upper_bound of None means that the IntervalMetadata object has no upper bound.

concat is not parallel to merge function in the sense that concat
returns a new object while merge modify the current object and that
concat requires upper_bound defined to add up coordinate spaces while
merge works fine with unbounded upper bound. I added check in merge in
the previous comment. It should not raise an error neccessarily when
upper_bound=None. I explained in line 673. could you check if it makes
sense to you?
On Fri, Dec 2, 2016 at 2:10 PM, Jai Ram Rideout<EMAIL_ADDRESS>wrote:

@jairideout commented on this pull request.
In skbio/metadata/_interval.py
https://github.com/biocore/scikit-bio/pull/1460:

     self._is_stale_tree = True

 @classonlymethod

@experimental(as_of="0.5.1")
def concat(cls, interval_metadata):

def _concat(cls, interval_metadata):

ok. will add it, although I think it's fine to raise error if
upper_bound=None.
In your previous implementation, did merge raise an error if
upper_bound=None? You added an explicit check and TypeError to concat for
this case, so I was wondering if this same check needed to be added to
merge.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/biocore/scikit-bio/pull/1460, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA9CeDE2WE7JPa1iDnzOwi7aWjYDkdysks5rEJdKgaJpZM4K2DlJ
.

Note _rebuild_tree is a decorator and only apply to the function that
operate on IntervalMetadata object. It is not modifies the private state
of an object directly, only when it is used to decorate a function that
access interval tree. The motivation leaving it outside is, I think,
because it can be used on other functions outside the class.
On Fri, Dec 2, 2016 at 2:04 PM, Jai Ram Rideout<EMAIL_ADDRESS>wrote:

@jairideout commented on this pull request.
In skbio/metadata/_interval.py
https://github.com/biocore/scikit-bio/pull/1460:

@@ -512,6 +520,30 @@ def init(self, upper_bound):
# Indicates if the IntervalTree needs to be rebuilt.
self._is_stale_tree = False

   if copy_from is not None:

       for interval in copy_from._intervals:

           # Only need to shallow-copy `bounds` and `fuzzy`

           # because their elements are immutable.

           bounds_cp = interval.bounds[:]

           fuzzy_cp = interval.fuzzy[:]

           metadata_cp = copy.copy(interval.metadata)

           self.add(bounds_cp, fuzzy=fuzzy_cp, metadata=metadata_cp)

def _rebuild_tree(method):

I don't see how moving this outside the class is an improvement. Now you
have a private function that modifies private state on an object, which is
not good practice.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/biocore/scikit-bio/pull/1460, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA9CeHRpX_NZDnCel_AIKJ0AntnOKQI8ks5rEJXugaJpZM4K2DlJ
.

I am clear now. Thanks for explanation! will add.
On Fri, Dec 2, 2016 at 1:35 PM, Jai Ram Rideout<EMAIL_ADDRESS>wrote:

@jairideout https://github.com/jairideout , does changelog really need
update? Because he API changes is not new and compatible backwards. If so,
then which section should I put it?
Yes. As detailed in the code review guidelines
https://github.com/biocore/scikit-bio/blob/master/REVIEWING.md#changelog
and in the pull request template
https://github.com/biocore/scikit-bio/blob/master/PULL_REQUEST_TEMPLATE.md,
all public-facing changes are documented in the changelog. There are
several sections in the changelog to categorize changes; for example,
"features", "bug fixes", "performance enhancements",
"backwards-incompatible changes", etc. Any change visible to a user
should be documented in the changelog, not just backwards-incompatible
changes. This is why we have several sections to categorize the types of
changes present in a release.
For this PR, there are two public-facing changes. Both are new features,
so they would go under the "features" section of the changelog:

skbio.interval.IntervalMetadata constructor has a new keyword
argument, copy_from, for creating an IntervalMetadata object from an
existing IntervalMetadata object with specified upper_bound.
skbio.interval.IntervalMetadata constructor allows None as a valid
value for upper_bound. An upper_bound of None means that the
IntervalMetadata object has no upper bound.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/biocore/scikit-bio/pull/1460#issuecomment-264568750,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA9CePrwU6-hZSk952qsd_1U4pqqLjxdks5rEI8fgaJpZM4K2DlJ
.

concat is not parallel to merge function in the sense that concat
returns a new object while merge modify the current object and that
concat requires upper_bound defined to add up coordinate spaces while
merge works fine with unbounded upper bound. I added check in merge in
the previous comment. It should not raise an error neccessarily when
upper_bound=None. I explained in line 673. could you check if it makes
sense to you?

Makes sense, thanks.

Note _rebuild_tree is a decorator and only apply to the function that
operate on IntervalMetadata object. It is not modifies the private state
of an object directly, only when it is used to decorate a function that
access interval tree. The motivation leaving it outside is, I think,
because it can be used on other functions outside the class.

I disagree -- the decorator is tightly coupled with IntervalMetadata and accesses private state on those objects (_is_stale_tree, _interval_tree, and _intervals), so it should continue to exist as a method on IntervalMetadata to avoid breaking encapsulation.
The decorator is only being applied to IntervalMetadata methods right now, but if it were to be applied to other functions in the future, it is possible to use the decorator as it is right now (i.e. defined as a method instead of a function):
@IntervalMetadata._rebuild_tree
def foo():
    ...

One issue with putting _rebuild_tree inside class is: you can prevent people to use it as a normal method instance._rebuild_tree(xxx) (although this is a private func). It causes error because it can't not be decorated with staticmethod.

One issue with putting _rebuild_tree inside class is: you can not prevent people to use it as a normal method like instance._rebuild_tree(xxx) (although this is a private func). It causes error because it can't be decorated with staticmethod to avoid this.

This is a problem with any private method, because nothing is truly private in Python. Even if you refactor it into a private function, users can still call that private function.
If a tree-rebuilding API is desired, it would be better to promote this API to a public method instead of a decorator. Then self.rebuild_tree() can be called appropriately within IntervalMetadata methods, and users outside the class can use it if desired. It sounds like it could be a useful public API in its own right, as a user may want to explicitly rebuild the tree before querying (or other operations).
I don't think the current refactoring of the private method into a private function has any benefits, and it has drawbacks that I noted in earlier comments. If you think refactoring the API into a public instance method is a good idea, it would be better to include that API change in a separate PR b/c it is orthogonal to the changes in this PR.

You misunderstood me. I was not talking about Python private. I was saying _rebuild_tree can't be decorated with staticmethod as I mentioned previously and its 1st parameter is not self, thus calling instance._rebuild_tree (which is perfectly legitimate syntax-wise) will cause error.
Multiple persons liked the idea of decorator before, because it isolates the tree build logic out and syntax-wise it is clean to just decorate the methods that access interval tree. I don't think we should reverse this design. And _interval_tree is private, we shouldn't expose this tree rebuild API to users; Otherwise, users may rebuild tree duplicately  or get confused whether they should rebuild tree before query or other operations accessing interval tree.
With that being said, if you still insist putting this back into the class, I can reverse this function to the previous state.

@RNAer I see your concern about the _rebuild_tree method. It definitely isn't a "method" in its present context, so pulling it out as a function does initially seem logical to keep the decorator syntax. However I would argue that it is actually pretty simple to disambiguate the first argument so that we have a method which can act both as a decorator in a static calling context (inside class definition) and as proper method (still private) for rebuilding a tree in a bound context. This would preserve the encapsulation of the class which is something we work very hard to maintain in scikit-bio (not to mention having very intentional API boundaries).
That being said, it seems like this code is frankly a little complicated for what is does, and a simple instance method like @jairideout suggested would suffice, and even make good sense as a part of the public API.
In either case, it seems like this method isn't hurting any public API and it probably isn't worth discussing this much further in the context of this PR. Perhaps making a new PR specifically targeted around refactoring this decorator makes the most sense for moving forward.

agree with leaving it for future. I will reverse the change wrt it.
On Tue, Dec 6, 2016 at 12:11 PM, Evan Bolyen<EMAIL_ADDRESS>wrote:

@RNAer https://github.com/RNAer I see your concern about the
_rebuild_tree method. It definitely isn't a "method" in its present
context, so pulling it out as a function does initially seem logical to
keep the decorator syntax. However I would argue that it is actually pretty
simple to disambiguate the first argument so that we have a method which
can act both as a decorator in a static calling context (inside class
definition) and as proper method (still private) for rebuilding a tree in a
bound context. This would preserve the encapsulation of the class which is
something we work very hard to maintain in scikit-bio (not to mention
having very intentional API boundaries).
That being said, it seems like this code is frankly a little complicated
for what is does, and a simple instance method like @jairideout
https://github.com/jairideout suggested would suffice, and even make
good sense as a part of the public API.
In either case, it seems like this method isn't hurting any public API and
it probably isn't worth discussing this much further in the context of this
PR. Perhaps making a new PR specifically targeted around refactoring this
decorator makes the most sense for moving forward.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/biocore/scikit-bio/pull/1460#issuecomment-265258862,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA9CeJFcozKTHKSsf4bAsVOHtOUItF3Hks5rFcFtgaJpZM4K2DlJ
.

ready for final review? Thanks!

Thanks!
On Wed, Dec 7, 2016 at 11:35 AM, Jai Ram Rideout<EMAIL_ADDRESS>wrote:

Merged #1460 https://github.com/biocore/scikit-bio/pull/1460.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/biocore/scikit-bio/pull/1460#event-885785087, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA9CeOzNrOLvIr4cAPsB2xqivzjrAuVAks5rFwp6gaJpZM4K2DlJ
.


fix: workflow ts hotfix

Changes

#933  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @MasterPtato and the rest of your teammates on  Graphite


elementValue fix for editable content

<input type="text" is editable, so .val() should be used instead of text()

Which problem does this change solve? How to reproduce?

Hi @bogatykh,
If you're trying to fix the same issue as #2211, it was already fixed and released as part of version 1.19.0. Otherwise, please provide more information about your issue.
Thanks!


Remove SignalProducer.buffer usage

With regard to https://github.com/ReactiveCocoa/ReactiveCocoa/issues/2831.

Looks good. 👍
I really hate this bit of code.

I really hate this bit of code.

I agree. This looks a bit ugly. :disappointed:

It was ugly before. That wasn't a condemnation of your changes her. 😄

Oh, that was my misunderstanding. ☺


Sections with lots of tags are not searchable by all tags

Describe the bug
sections with a lot of tags aren't searchable in smallish terminals.
It looks like because the tags are trimmed to a certain length before being passed to fzf then fzf can't seem to search them.
For instance, if you search js then you wouldn't find the js snippets in a small enough terminal.
To Reproduce
Steps to reproduce the behavior:

Open a terminal
Shrink the terminal
Ctrl+g to use navi
Search for js
See that no js snippet is found

Expected behavior
Be able to search snippets through all tags.
Screenshots

You can see that js tag is part of this cheat
https://github.com/denisidoro/navi/blob/f8e8da018bcd2598a47260581faad2cbafad7f3a/cheats/javascript.cheat#L1
Versions:

OS: ChromeOS (with debian linux)
Shell Version: zsh 5.3.1

That's a known problem but I'm afraid there's no way to fix it :disappointed:
fzf won't allow one to search for something that isn't displayed on the screen.
And if I don't trim columns, they won't be aligned and potentially you'll only be able to see the first column.

Related: https://github.com/junegunn/fzf/issues/1440#issuecomment-443480229

😞
That's a shame because it kinda makes the tags a bit redundant for me. I'd rather put these tags as the description of the commands which is given a lot more space in the UI.
I know I tend to search using keywords rather than sentences, as well as most other developers I know.
But, In theory it's possible, if we were to call fzf without letting take over the terminal (--filter) and then draw the UI ourselves, simply using fzf for a fuzzy search we could decouple the filtered string from the displayed string.
Also, #227 could help here as we'd send the full line to skim as it's just a library, and again, build the UI ourselves.

Supporting skim does seem to be a nice idea.
Regarding your use-case, this would be helpful: https://github.com/denisidoro/navi/issues/202

Looking forward to being able to resize columns :)

Hi! Sorry for the late reply.
Being 100% honest here: this is a feature it would be nice to have but unfortunately I haven't had much free time lately, so I have no plans to implement this in the near future.
If anyone could submit a PR for this, I'll gladly accept it, though! 👍
(One additional comment: skim is already supported --as a binary, not as a library. Maybe this issue already has a workaround?)


Checked out items view

Finished checked out items view

Tables are not implemented yet in "Checked Out Items" view.


attention_is_all_you_need:  install.py Error: C3861: '_PyGen_Send': identifier not found

When following the instructions from https://learn.microsoft.com/en-us/windows/ai/directml/gpu-pytorch-windows
I end up with calling install.py of the attention_is_all_you_need demo.
The build errors with:
spacy/strings.cpp(10212): error C3861: '_PyGen_Send': identifier not found
It seems this issue has already been discovered by cython: https://github.com/cython/cython/issues/3876
and is related to Python 3.8.10 (possibly also 3.8.9). The installation instructions do not mention a specificy python version, so depending on the miniconda version and date of setup, a fairly new python will be installed.

Quick Fix: Hardcode a stable version in the documentation of gpu-pytorch-windows
Final Fix: Make the build work with 3.10+ Versions of python

retried with<EMAIL_ADDRESS>--> One error because "spacy" couldn't be installed, because of "creating link error to de" or so. I didn't memorise the exact message, but there was a hint I might not install as admin.
So I reopened the shell with admin rights, and then the python install.py ran through.
-> I am not sure now, if the 3.10 Python version is really the issue (there I didn't realise the admin suggestion and installed spacy manually).
--> The documentation doesn't say "run/install as admin". In general, python installs requiring aren't ideal. Can this be fixed  to be non-admin dependent?


[SUPPORT] Print the error but do not stop the tests

What
The job availability-tests/470 has failed. We think it actually
shouldn't matter to us that much, if there is a one of connection issue
with the concourse API. We don't mean to get rid of the error
completely, therefore we're still printing it, allowing the script to
continue it's loop and hoping that the issue has resolved it self in the
mean time.
How to review

Code review
Run the deployment from this branch and expect the
availability-tests to pass

Thanks 👍


sort imports using LocalPrefix

Implement https://github.com/ent/ent/issues/2170.
The second commit is generated by go generate ./....

I like very much. Let's see what @a8m thinks.

Thanks for your contribution @Thearas, and for your review @masseelch ❤
Please, see my comment on https://github.com/ent/ent/issues/2170#issuecomment-980009054, and let's continue the discussion there.


Make line protocol round or parse precision as expected

Line protocol for precision can do two things:

If specified and a time is provided, it needs to use a multiplier to get it back to nanoseconds
If specified, but no time is provided, we need to truncate down to the time to observe the precision.

:+1:

Thanks!


⚠ Bofrost.de has degraded performance

In 7671d28, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1531 ms

Resolved: Bofrost.de performance has improved in 2d587b4 after 57 minutes.


fix: dataset urls

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Motivation and Context / Related issue
ML datasets urls fix
How Has This Been Tested (if it applies)

Checklist

[ ] The documentation is up-to-date with the changes I made.
[ ] I have read the CONTRIBUTING document and completed the CLA (see CLA).
[ ] All tests passed, and additional code has been covered with new tests.

Fixed in other PR; thanks @fteytaud


[Bug]: No metrics exposed at /metrics with 0.5.2 (0.5.1 is fine), possible regression?

Your current environment
Collecting environment information...
PyTorch version: 2.3.1+cu121
Is debug build: False
CUDA used to build PyTorch: 12.1
ROCM used to build PyTorch: N/A

OS: Ubuntu 22.04.4 LTS (x86_64)
GCC version: (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
Clang version: Could not collect
CMake version: version 3.30.0
Libc version: glibc-2.35

Python version: 3.10.12 (main, Nov 20 2023, 15:14:05) [GCC 11.4.0] (64-bit runtime)
Python platform: Linux-5.15.0-107-generic-x86_64-with-glibc2.35
Is CUDA available: True
CUDA runtime version: Could not collect
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration:
GPU 0: NVIDIA H100 80GB HBM3
  MIG 3g.40gb     Device  0:

Nvidia driver version: 550.54.15
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

CPU:
Architecture:                       x86_64
CPU op-mode(s):                     32-bit, 64-bit
Address sizes:                      46 bits physical, 57 bits virtual
Byte Order:                         Little Endian
CPU(s):                             192
On-line CPU(s) list:                0-191
Vendor ID:                          GenuineIntel
Model name:                         Intel(R) Xeon(R) Platinum 8468
CPU family:                         6
Model:                              143
Thread(s) per core:                 2
Core(s) per socket:                 48
Socket(s):                          2
Stepping:                           8
CPU max MHz:                        3800.0000
CPU min MHz:                        800.0000
BogoMIPS:                           4200.00
Flags:                              fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc art arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc cpuid aperfmperf tsc_known_freq pni pclmulqdq dtes64 monitor ds_cpl vmx smx est tm2 ssse3 sdbg fma cx16 xtpr pdcm pcid dca sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand lahf_lm abm 3dnowprefetch cpuid_fault epb cat
_l3 cat_l2 cdp_l3 invpcid_single cdp_l2 ssbd mba ibrs ibpb stibp ibrs_enhanced tpr_shadow vnmi flexpriority ept vpid ept_ad fsgsbase tsc_adjust bmi1 avx2 smep bmi2 erms invpcid cqm rdt_a avx512f avx512dq rdseed adx smap avx512ifma clflushopt clwb intel_pt avx512cd sha_ni avx512bw avx512vl xsaveopt xsavec xgetbv1 xsaves cqm_llc cqm_occup_llc cqm_mbm_total cqm_mbm_local split_lock_detect avx_vnni avx512_bf16 wbnoinvd dtherm ida arat pln pts hwp hwp_act_window hwp_epp hwp_pkg_req avx512vbmi umip pku ospke
 waitpkg avx512_vbmi2 gfni vaes vpclmulqdq avx512_vnni avx512_bitalg tme avx512_vpopcntdq la57 rdpid bus_lock_detect cldemote movdiri movdir64b enqcmd fsrm md_clear serialize tsxldtrk pconfig arch_lbr amx_bf16 avx512_fp16 amx_tile amx_int8 flush_l1d arch_capabilities
Virtualization:                     VT-x
L1d cache:                          4.5 MiB (96 instances)
L1i cache:                          3 MiB (96 instances)
L2 cache:                           192 MiB (96 instances)
L3 cache:                           210 MiB (2 instances)
NUMA node(s):                       2
NUMA node0 CPU(s):                  0,2,4,6,8,10,12,14,16,18,20,22,24,26,28,30,32,34,36,38,40,42,44,46,48,50,52,54,56,58,60,62,64,66,68,70,72,74,76,78,80,82,84,86,88,90,92,94,96,98,100,102,104,106,108,110,112,114,116,118,120,122,124,126,128,130,132,134,136,138,140,142,144,146,148,150,152,154,156,158,160,162,164,166,168,170,172,174,176,178,180,182,184,186,188,190
NUMA node1 CPU(s):                  1,3,5,7,9,11,13,15,17,19,21,23,25,27,29,31,33,35,37,39,41,43,45,47,49,51,53,55,57,59,61,63,65,67,69,71,73,75,77,79,81,83,85,87,89,91,93,95,97,99,101,103,105,107,109,111,113,115,117,119,121,123,125,127,129,131,133,135,137,139,141,143,145,147,149,151,153,155,157,159,161,163,165,167,169,171,173,175,177,179,181,183,185,187,189,191
Vulnerability Gather data sampling: Not affected
Vulnerability Itlb multihit:        Not affected
Vulnerability L1tf:                 Not affected
Vulnerability Mds:                  Not affected
Vulnerability Meltdown:             Not affected
Vulnerability Mmio stale data:      Not affected
Vulnerability Retbleed:             Not affected
Vulnerability Spec rstack overflow: Not affected
Vulnerability Spec store bypass:    Mitigation; Speculative Store Bypass disabled via prctl and seccomp
Vulnerability Spectre v1:           Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:           Mitigation; Enhanced IBRS; IBPB conditional; RSB filling; PBRSB-eIBRS SW sequence; BHI BHI_DIS_S
Vulnerability Srbds:                Not affected
Vulnerability Tsx async abort:      Not affected

Versions of relevant libraries:
[pip3] flashinfer==0.0.9+cu121torch2.3
[pip3] numpy==1.26.4
[pip3] nvidia-nccl-cu12==2.20.5
[pip3] torch==2.3.1
[pip3] torchvision==0.18.1
[pip3] transformers==4.42.4
[pip3] triton==2.3.1
[conda] Could not collect
ROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: 0.5.2
vLLM Build Flags:
CUDA Archs: Not Set; ROCm: Disabled; Neuron: Disabled
GPU Topology:
GPU0    CPU Affinity    NUMA Affinity   GPU NUMA ID
GPU0     X      1,3,5,7,9,11    1               N/A

Legend:

  X    = Self
  SYS  = Connection traversing PCIe as well as the SMP interconnect between NUMA nodes (e.g., QPI/UPI)
  NODE = Connection traversing PCIe as well as the interconnect between PCIe Host Bridges within a NUMA node
  PHB  = Connection traversing PCIe as well as a PCIe Host Bridge (typically the CPU)
  PXB  = Connection traversing multiple PCIe bridges (without traversing the PCIe Host Bridge)
  PIX  = Connection traversing at most a single PCIe bridge
  NV#  = Connection traversing a bonded set of # NVLinks

🐛 Describe the bug
When switching the Container image from 0.5.1 to 0.5.2 the Prometheus metrics usually exposed at /metrics are gone.
0.5.1 - INFO:     xxx.xxx.xxx.xxx:56742 - "GET /metrics HTTP/1.1" 200 OK
0.5.2 - INFO:     xxx.xxx.xxx.xxx:45698 - "GET /metrics HTTP/1.1" 404 Not Found
I looked at the release notes, but did not see any change about a required config parameter or similar to have the metrics enabled.

cc @simon-mo

@simon-mo @youkaichao Maybe it's this PR: https://github.com/vllm-project/vllm/pull/5090/
Looking around the metrics endpoint, I believe there is an annotation `@router.get("/metrics") missing somewhere around
https://github.com/EthanqX/vllm/blob/8cf22579e36ea95686611ac7e697c02dd29a8aee/vllm/entrypoints/openai/api_server.py#L71-L77

cc @EthanqX
and cc @DarkLight1337  if you can fix it in #6431 by the way, or add another pr to fix it.

Please refer to #6434 for more details on release timeline.

Argh .. sorry, I was even subscribed to that one :disappointed:

How do you get certain metrics like say for example avg generation throughput or something similar that is available in the /metrics endpoint to the caller?
As it is currently, these metrics being part of the vLLM engine to be specific, sit on the server and the caller isn't aware of them. The server however has a endpoint to monitor, but I can't seem to see how I can access those individually rather than see the hosted ones.
Are there any examples to maybe curl this endpoint and grab them individually per request and get it back to the caller or a way to send them with the request. I believe sending it with the request would be easy but would also mean adding things to the OpenAI chat completions function itself from what I understand.

@frittentheke how do you collect/use these metrics?

@frittentheke how do you collect/use these metrics?

Scaping via Prometheus. These are aggregated metrics of the vLLM instance, NOT per request.


Unable to view Published app in experience cloud

Describe the bug
We have created an app using “DX Experience Shell” , It is approved and published, status in console states its Published, But we are unable to view under Experience cloud -> App builder Catlog
To Reproduce
Steps to reproduce the behavior:
The build and deployment log as been attached
Expected behavior
Able to see published app under experience cloud -> App catalog
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Try using the proper commands please.
You are running aio app:project:select instead of aio console:project:select and aio app:workspace:select instead of aio project:workspace:select ... all of these are causing exceptions internally, and even though you are 'auto-suggested' the correct command, it is hard to know if this is causing the issue.
The recommended way to bind a code-repo to a dev-console project is to go to the project/workspace page, and select Download All
This will give you a json file with all the config for the project/workspace, then you simply run
aio app use <downloaded-file.json>

Hi @purplecabbage , i have the same issue, pushed an application but it doesn't appear in App Builder Catalog.
Please do share what are the steps int and deploy a project once we download the JSON file from the Project

Hi @purplecabbage I am unable to Download JSON since the app published in production workspace is Read Only, though I do have admin access.

@igirish-kumar i had the same issue and couldn't download the json but i executed this command in the terminal
aio app use -w Production
It changed both .aio and .env files with the production configurations which I consider as same as what we need
After that I ran
aio app deploy
but it also didn't deploy the application in app-builder catalog, it ran successfully and deployed the in the long URL with the devmode=true and localhost=...

But I noticed this row in the terminal skipping publish phase...
What does that mean?
@purplecabbage please advise what we should do in order to publish the app since all configurations look correct and production workspace is selected. How can we list the app in the catalog?
Thanks

btw, i also received an email that the app is successfully approved and published
our application is now available in Experience Cloud. If you have any questions, please refer to the [App Builder documentation](https://www.adobe.io/apis/experienceplatform/project-firefly/docs.html).
All good but the app still doesn't appear

@igirish-kumar Even though the CLI has been configured to use the Production workspace of your application, the project you are working on may still be configured to use the Stage workspace locally. I would suggest running aio app use in your project's directory and inspecting the .env file in the project directory to ensure production values are present before trying again.
@NickRumenov Was the application you created initialized as Standalone Application or as DX Experience Cloud SPA v1? It must be initialized as the latter in order to show up in the App Builder Catalog. You can verify this by looking at the app in Exchange. If the app has the App Builder ExcShell subheading (see below), then it should show up in App Builder Catalog after approval. If it does not, then you may need to go through some migration steps, which I will document and link here once they're complete.

@purplecabbage , The issue still exists even after using production workspace and we could see some errors on executing  aio app deploy as below :
✔ Built 4 action(s) for 'dx/excshell/1'
✔ Building web assets for 'dx/excshell/1'
✔ Deployed 8 action(s) for 'dx/excshell/1'
✔ Deploying web assets for 'dx/excshell/1'
Your deployed actions:
web actions:
-> https://25605-mvplabservicecenter.adobeio-static.net/api/v1/web/dx-excshell-1/generic
-> https://25605-mvplabservicecenter.adobeio-static.net/api/v1/web/dx-excshell-1/getProfile
-> https://25605-mvplabservicecenter.adobeio-static.net/api/v1/web/dx-excshell-1/getExperienceEvents
-> https://25605-mvplabservicecenter.adobeio-static.net/api/v1/web/dx-excshell-1/postEvent
non-web actions:
-> dx-excshell-1/__secured_generic
-> dx-excshell-1/__secured_getProfile
-> dx-excshell-1/__secured_getExperienceEvents
-> dx-excshell-1/__secured_postEvent
To view your deployed application:
-> https://25605-mvplabservicecenter.adobeio-static.net/index.html
To view your deployed application in the Experience Cloud shell:
-> https://experience.adobe.com/?devMode=true#/custom-apps/?localDevUrl=https://25605-mvplabservicecenter.adobeio-static.net/index.html
⠋ Updating Extension Points 'endpoints' in Workspace=Production... ›   Error: [CoreConsoleAPISDK:ERROR_UPDATE_WORKSPACE_ENDPOINTS] 400 - Bad Request
›   ([{"template":"ERR_MSG_RETRY_OR_CONTACT","message":"{"errors":[{"error_code":"1400","message":"Problem occured - App in
›   DRAFT|REJECTED|RETRACTED status can only be updated"}]}"}])
⠙ Updating Extension Points 'endpoints' in Workspace=Production...

@igirish-kumar It seems like you are trying to update extension point configuration for an application that is already published. Try revoking the application, giving the deploy another try, and then re-publishing the application.

@MichaelGoberling , That worked, Thanks for all your support.

@igirish-kumar Glad to help.
@NickRumenov Following up, have you been able to view your application in Experience Cloud?

Closing this as resolved.  If you still have issues @NickRumenov please open a new issue.

Ok i will test it, thanks for the support


chore: support RxSwift 6

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


Elastic profile enhancement

Enhancements for having these 3 things at Elastic Profile level

Agent auto-register timeout
Name space
Security Token

If these are not specified , the default values from Plugin are used.
However if they are specified the Elastic profile ones take precedence.

Hi @saifally, these changes make sense. We'll review them and merge soon. Thanks. cc: @GaneshSPatil

Related: https://github.com/gocd/gocd/issues/3591

Hi @saifally and @GaneshSPatil, thanks for your thoughts on supporting multiple k8s clusters via elastic profile settings. Unfortunately we didn't design elastic profiles for this purpose and including cluster configurations at the elastic profile level pollutes this design. This would introduce unintended side-effects, one example of which is breaking the current status report page.
I would recommend we hold off on this change at the moment until we design the support for multiple clusters more intentionally. I don't have a timeline for you at the moment, but we'll revisit this feature and send out an update on what we plan to do soon.
\cc: @arvindsv @ketan

@sheroy  thanks for the quick feedback.
Again, I will defer the multi cluster design to you guys.
After all you have to be consistent across Marathon,Swarm,Kubernetes and ECS.
Happy to contribute again once there is enough clarity.
For this one , are you guys still keen to accept the initial change of
Namespace and Token at Elastic profile level.
Its quite easy for me to remove the cluster details from Elastic profile front end.
Behind the scene they can still be held in a variable for each agent instance which to me is the right way to go about it because that is point in time detail.
For example, with the current implementation , without my PR, if the cluster details are changed while a few agents/pods are in progress.
They can not be terminated since they are running in old cluster.
And last and most important :)
The status page is not broken with my changes  I have done enough testing on it.
It needs to be enhanced to show multi cluster details.
For this PR, the only additional detail it is showing is namespace.

Hi @saifally, sorry for the delayed response on this.

For this one , are you guys still keen to accept the initial change of
Namespace and Token at Elastic profile level

I would rather we hold off on this change since we will be changing the elastic profile settings soon. We might as well just make this change once after thinking it through.

Any update on this?

Hello, @saifally @wstewartii @sheroy ---
As part of GoCD release v19.3.0, we've added support to create multiple clusters for elastic agent plugin. We can use GoCD v19.3.0 and kubernetes-elastic-agents v3.0.0-156 to use new multi-cluster feature.
More information around Multi-cluster support can be found on:

Release Notes
Elastic Agent Documentation
Configure K8s Elastic Agent Documentation

I am closing this ticket, marking as resolved.
Feel free to reach out us/reopen the ticket in case you have queries or questions.
Thanks,
Ganesh


Scala Kinesis Enrich: add health check endpoint

The old description is below - but I think this is an odd architecture, and it would be better to have a passive endpoint inside SKE that any external healthcheck system could use...
Old description
It is possible to monitor a long-running application using the Consul TTL health check:

https://groups.google.com/d/msg/consul-tool/22X5_Op_V7A/GioJW27rjlYJ
https://www.consul.io/docs/agent/checks.html

Take a look here:

https://consul.io/docs/agent/http/agent.html#agent_check_pass

You want the /v1/agent/check/{pass,warn,fail}/ endpoints.
Let's assume that our provisioning has registered the check ID.
Therefore the SKE HOCON should contain something like:
  consul {
    check-id: "xxx"
    get-frequency: x.seconds
  }

That'll be great!

Could use HTTP or gRPC or Serf...

Hi guys, any news on this feature?

hey @andrebrov , we're going to way of metrics you can read more about it in our RFC

@BenFradet is any news regarding monitoring componenets of Stream Pipeline, especially Stream Enricher.

We have included a set of metrics in beam-enrich (section "Implementation" in the RFC) and we're planning on doing the same for stream-enrich but there is no clear timeline at this time.

Migrated to https://github.com/snowplow/enrich/issues/None

Why was this closed? Was the feature added?

Hi @zoubinirani. I was migrating all ticket to the new repository for enrich assets and something went wrong with this one (as doesn't show the proper link). Re-created it in https://github.com/snowplow/enrich/issues/243


Better FFTLog

This PR:

Is part of #704
Fixes a bug in our implementation of FFTLog (which wasn't affecting our code before)
Makes the FFTLog interface a bit easier to use for 2D and 3D-like Hankel transforms. It also adds more documentation.
Makes this directly accessible through python.

Pull Request Test Coverage Report for Build 4131

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 96.053%

Totals

Change from base Build 4127:
0.0%

Covered Lines:
3602

Relevant Lines:
3750

💛  - Coveralls

OK, done. Back to you @beckermr !

It's just flake8's authoritarian attitude. I'll fix it in a bit.


Can't install fatal error: /lib/python3.8/site-packages/torch/include/ATen/ops/argmax.h: No such file or directory

Everytime I try to compile from source, I get this kind of error:
fatal error: <path-to-conda-env>/lib/python3.8/site-packages/torch/include/ATen/ops/argmax.h: No such file or directory
Everytime it is some different file in the ATen folder of pytorch which it says is missing but it is not.
If I reinstall pytorch (compile from source nightly cuda 12.4 version), this disappears but following appears
/home/skatar6/TransformerEngine/transformer_engine/common/util/../common.h:20:10: fatal error: cuda_fp8.h: No such file or directory

Can you check your CUDA version (e.g. with nvcc --version)? TE requires CUDA 11.8 or newer, which includes cuda_fp8.h.
If your CUDA version is supported, I suspect the problem is from CMake. If you have multiple CUDA installations on your system, it might be detecting and using an old version. Try following one of these instructions to force it to use the right CUDA version. It may be helpful to pass the --verbose flag to pip install so that you can see the CMake build logs.

Thanks for the useful links. I think my machine had several CUDA, cuDNN, pytorch, and disk issues. I installed everything from scratch and after much experimentation, the following command could install the package finally:
mkdir -p $TMPDIR && mkdir -p $CMAKE_TEMP_DIR && mkdir -p $BUILD_DIR
TMP_DIR=/home/$USER/tmp MAX_JOBS=4 CUDA_HOME=$CUDA_HOME CUDNN_PATH=$CUDNN_PATH CC=$CC CXX=$CXX \
    pip -v install --no-deps --cache-dir /home/$USER/tmp/pip-cache git+https://github.com/NVIDIA/TransformerEngine.git@stable

Infact I would recommend MAX_JOBS=1 to others.


[Bug]: In peer review, deletion proposal is sent even if no reason is chosen

Summary
In peer review, the deletion proposal is sent even if no reason is chosen.
Steps to reproduce

Start peer review
When asked whether the picture is in scope, tap No
Without selecting any reason, submit

Expected behaviour
Should not be submittable until at least a reason is chosen.
Actual behaviour
Submitted without any reason listed. This simply shouldn't happen, as it results in a very weird (and useless) proposal.
Device name
No response
Android version
All
Commons app version
3.1.1 master prodDebug
Device logs
No response
Screen-shots
No response
Would you like to work on the issue?
Prefer not

@nicolas-raoul, i would like to work on this


Correctly handle double quotes when setting meta descriptions

If the first paragraph of a page includes double quotes, the plugin should remove or escape them so that they don't break the description attribute of the <meta> HTML element.

Fixed in https://github.com/prcr/mkdocs-meta-descriptions-plugin/commit/c30b6149b48922533220c41d350898a1a1df9015.


Improved 'Using typeof' waypoint

Now properly refers to typeof as an operator vs a method.
Added a link to MDN's article about typeof.
Added some <code> tags and extra auxiliary text.
Changed the examples and test cases to not use parenhteses.

Fix #3047

Didn't realize that #3016 was, in fact, a pull request.. :neutral_face: :palm_tree:
Closing this in favour of that one. Might rebase and reopen to add the MDN link and aux text when @prashcr's PR is merged.


Internationalization of texts

Hi,
Is there a way to change the language of the PaginatedDataTable2 texts?
For example the text of Rows per page.
Thanks.
Regards,
Jose

Hi,
Loclization is supported and works exactly as in stock widgets. You can try this by chaging the sample in the following way:

Add localization de[endency to pubspec.yaml

  flutter_localizations:
    sdk: flutter

Add 2 properties to MaterialApp in main.dart

      localizationsDelegates: [
        GlobalMaterialLocalizations.delegate,
        GlobalWidgetsLocalizations.delegate,
        GlobalCupertinoLocalizations.delegate,
      ],
      supportedLocales: [
        const Locale('en', ''),
        const Locale('be', ''),
        const Locale('ru', ''),
      ],
      locale: Locale('ru', ''),

See that paginated table controls are translated:


chore: remove -vvv flag

Removing -vvv flag as Github worflow UI is not able to display it due to its large size. Example, https://github.com/sablier-labs/flow/actions/runs/10989737602/job/30508576133

tbh i would keep it, maybe it is not useful when there is a long sequence, but for shorter ones, it improves the debugging
it doesn't hurt if we keep it, wdyt?

I would like to keep it as well but see here, the UI can't even load the log and the subsequent failed/passed tests. It just keep trying to load it.

UI can't even load the log and the subsequent failed/passed tests

yeah, that's for long  sequences
i said to keep it for: shorter ones

One more recent run, not able to display the log at all: https://github.com/sablier-labs/flow/actions/runs/10993870570/job/30524285735

fair enough, let's remove it


Fix issue where initializeAll does not properly use initializer

TypeInfo_StaticArray.initializer only returns an initializer for one element. In the case where initializeAll is given a range of static arrays of types with elaborate assign, random data would be used. This is also a memory corruption issue, since memcpy was reading past the end of the initializer. I tested the unit test before making the fix, and it was failing.

Current coverage is 88.79% (diff: 100%)

Merging #4736 into master will increase coverage by <.01%

@@             master      #4736   diff @@
==========================================
  Files           121        121
  Lines         74154      74168    +14
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          65840      65854    +14
  Misses         8314       8314
  Partials          0          0

Powered by Codecov. Last update e15e7c0...bf49940

LGTM

@DmitryOlshansky yeah, copied from the bug report. I've removed it now.

Auto-merge toggled on


Cascadia Code static vs variable (VF) rendering

Windows Terminal version
1.16.10262.0
Windows build number
10.0.22000.1574
Other Software
Cascadia Code release 2111.01
Steps to reproduce

Remove Cascadia Cove that comes bundled with Windows Terminal [1]
Note that the bundled version is Cascadia Cove 2111.01 VF TrueType outlines
Manually install Cascadia Cove 2111.01 static Regular TrueType outlines
Start Windows Terminal
Take screenshot of text with capitals and small-letter i at some bigger size (effect is not size dependent though)
Close Windows Terminal, remove Cascadia Cove font
Install Cascadia Cove 2111.01 VF TrueType ourlines
Start Windows Terminal and repeat previous steps
Compare screenshots

[1] For people like me that is hard, here is a manual:

MAINTAINER EDIT: DO NOT FOLLOW THESE STEPS. Changing the permissions on WindowsApps is a one-way street to pain.

Gain access to C:/Programs/WindowsApps
Set youself as owner for C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_1.15.3466.0_x64__8wekyb3d8bbwe
Set youself as owner for all 4 font files in that folder
Add full access right for everyone on the 4 fonts
Move the 4 fonts to another location (e.g. desktop) - deleting is impossible
Reboot
Enjoy to be able to install whichever Cascadia Code you like and use Windows Terminal with that

Note that a Windows Terminal update reinstalls the bundled font, of course.

I'm not entirely sure if this should be reported here or at Cascadia Code. But the issue can not be seen on WordPad and also not in terminals on other platforms, so I guess it is something specific to Windows Terminal.
I tried experimental.rendering.software true and false (and also experimental.useAtlasEngine) but the outcome is unchanged.
Related issues, that as far as I can see all are due to the effect described in this issue.

https://github.com/microsoft/cascadia-code/issues/371
https://github.com/ryanoasis/nerd-fonts/issues/998
https://github.com/ryanoasis/nerd-fonts/issues/519
https://github.com/adam7/delugia-code/pull/79

The problem with the issues above was that they all compare the VF to the static ones and the VF is hard to get rid of.
Expected Behavior
Both fonts should look the same. With maybe a very slight weight difference because one is interpolated on the fly.
Actual Behavior

The heights (capital and x) is visibly different
The dot on the small letter i is noticable down (or stays in place while the dotless-i glyph resizes)

The effect is even visible in the font preview window.
For easier visual comparison I use a renamed copy of Cascadia Cove Regular ttf with the name Delugia.
The first image shows that Delugia and Cascadia Cove static render the same (at least no really noticable differences):

The second image now compares Delugia and Cascadia Cove VF ttf. Because of the comparison above this is equivalent to comparing Cascadia Cove static to Cascadia Cove VF:

Ok, then please advice me how I can get rid of the bundled font?
The steps described above are done in order to be able to bugfix (or rather triage) a problem, not for any 'normal' user.

Couldn't you just install the 12 variants of "CascadiaCodePL" and then compare "Cascadia Code PL" with the builtin "Cascadia Code"? That way you wouldn't have to delete any fonts to reproduce the issue.
According to Visual TrueType we're rendering the text correctly.
@DHowett there seems to be something strange going on with the static variants of Cascadia Code. If I load them into VTT they look identical to the VF variant, but once I load the GASP table, I get warnings in VTT and the glyphs are suddenly 1px taller. With the lowercase "w" character at 12pt and 144dpi for instance, both initially load as 12px tall, but once the GASP table has loaded for the static variants its suddenly 13px tall.

Couldn't you just install the 12 variants of "CascadiaCodePL" and then compare

In principle I could, but whatever the result I would never know if that is a difference between the vanilla and the PL version.
But thanks, that is still a good idea :+1:

According to Visual TrueType we're rendering the text correctly.

I do not know that tool. But WordPad renders both fonts also identical. And Glyphs' TextPreview tool (used to check fonts after creation) also renders both identical :thinking: So far Windows is the only platform where I witnessed the problem.
One thing lingers still in the back of by head, I believe I had a discussion with aaronbell about the static fonts and that they are different in some way; unfortunately I can not find that thread anymore.

After re-reading the issue I linked myself for the 3rd time:

https://github.com/microsoft/cascadia-code/issues/371

I guess this is the exact effect we see here. So it might be the autohinting that 'breaks' the static fonts; and has nothing to do with Windows Terminal per se.
I also have a user comment who sees the problem with Visual Studio on Windows.
So I guess this Issue is wrong here (as speculated in the beginning).
Thank you so much for looking into it!
I close this now, sorry for raising a red-herring.

That's alright, thanks for following up!


Step 1 of debug: Remove bcftools sort

Step 1 of changes to debug: https://centrepopgen.slack.com/archives/C030X7WGFCL/p1704924872033969?thread_ts=1704685344.162619&cid=C030X7WGFCL

Ok I have merged this branch with mergeSTR-shard. I will not merge this branch into main. Closing this PR now.


Uninstall

How to remove this from repo?

Hello. I am not sure what you mean, but I guess you are saying you have installed the library via pip install? If yes, you can use pip uninstall to remove any installed library.


Revise FrEIA install instructions

Hi thanks for open-sourcing your code! I was trying to get things running but needed to install FrEIA at a specific commit to get things to work as they are. I updated the README accordingly.

PR https://github.com/VLL-HD/GIN/pull/3 will make this redundant


remove unused default value

@tyneises Can you confirm that you meant this to go to develop, not patch? Apologies, we're dragging a bit on getting the patch branch set up.

@tyneises Can you confirm that you meant this to go to develop, not patch? Apologies, we're dragging a bit on getting the patch branch set up.

Thanks, develop (not patch) is fine


Fixes NODE-2335 (non-compliant DNS Seedlist Discovery parsing.)

Description
See: https://jira.mongodb.org/browse/NODE-2335
This PR fixes Fixes non-compliant DNS Seedlist Discovery parsing.
What changed?
Currently the TXT record values override the querystring values, spec states the opposite must be true, so the uri parser was fixed.
The spec test was also not being performed properly so that was fixed as well to add coverage.

Thanks for the contribution @mpilar ! This will go out in our v3.4.0 release tomorrow


Fix assignfixedaddress #106

Address is required for RPC assignfixedaddress.
Address can be repeatedly assigned.
Key will be removed from keypool if the given address's key is in the keypool.

pass


Not able to override inputs defaults with query input

rank-profile
rank-profile hybrid inherits closeness {
        inputs {
            query(wTags) : 1.0
            query(wPopularity) :  1.0
            query(wTitle) : 1.0
            query(wVector) : 1.0
        }
        first-phase {
            expression {
                query(wTags) * rawScore(tags) +
                query(wPopularity) * attribute(popularity) +
                query(wTitle) * bm25(title) +
                query(wVector) * closeness(field, embedding)
            }
        }
        match-features {
            rawScore(tags)
            attribute(popularity)
            bm25(title)
            closeness(field, embedding)
            distance(field, embedding)
            query(wVector)
        }
    }

Query with input override:
vespa query \
    'yql=select title, artist from track where {targetHits:100}nearestNeighbor(embedding,q) or userQuery()' \
    'query=total eclipse of the heart' \
    'type=weakAnd' \
    'hits=2' \
    'ranking=hybrid' \
    'input.query(q)=embed(e5, "Total Eclipse Of The Heart")' \
    'input.query(qVector)=10.0'

result response
{
    "timing": {
        "querytime": 0.005,
        "summaryfetchtime": 0.0,
        "searchtime": 0.006
    },
    "root": {
        "id": "toplevel",
        "relevance": 1.0,
        "fields": {
            "totalCount": 1181
        },
        "coverage": {
            "coverage": 100,
            "documents": 95666,
            "full": true,
            "nodes": 1,
            "results": 1,
            "resultsFull": 1
        },
        "children": [
            {
                "id": "index:tracks/0/f13697952a0d5eaeb2c43ffc",
                "relevance":<PHONE_NUMBER>3147683,
                "source": "tracks",
                "fields": {
                    "matchfeatures": {
                        "attribute(popularity)": 100.0,
                        "bm25(title)": 22.591334631476823,
                        "closeness(field,embedding)": 1.0,
                        "distance(field,embedding)": 0.0,
                        "query(wVector)": 1.0,
                        "rawScore(tags)": 0.0
                    },
                    "title": "Total Eclipse Of The Heart",
                    "artist": "Bonnie Tyler"
                }
            },
            {
                "id": "index:tracks/0/57c74bd2d466b7cafe30c14d",
                "relevance":<PHONE_NUMBER>8655665,
                "source": "tracks",
                "fields": {
                    "matchfeatures": {
                        "attribute(popularity)": 100.0,
                        "bm25(title)": 12.03241051547921,
                        "closeness(field,embedding)": 0.6700445710774405,
                        "distance(field,embedding)": 0.49243803049099677,
                        "query(wVector)": 1.0,
                        "rawScore(tags)": 0.0
                    },
                    "title": "Eclipse",
                    "artist": "Kyoto Jazz Massive"
                }
            }
        ]
    }
}

As can be seen from the matchfeatures, the `query(wVector)' is unchanged (1.0) and not the expected (10)
Output with trace
 rankproperties={\"qVector\":[10.0],\"q\":[tensor<float>(x[384]):[-0.5........]

Looks like it's handled correctly in the container, but not in proton?

Missspelled input query value.. closing as invalid.


Python Packages Install Fails in Docker action

Can't install Python packages like Numpy in Docker because of the absence of certain libraries.
Packages only install if the below command is included in docker file, is it possible to integrate this in the Docker Skeleton.
RUN mkdir -p /tmp/build
&& cd /tmp/build/
&& wget http://www.netlib.org/blas/blas-3.6.0.tgz
&& wget http://www.netlib.org/lapack/lapack-3.6.1.tgz
&& tar xzf blas-3.6.0.tgz
&& tar xzf lapack-3.6.1.tgz
&& cd /tmp/build/BLAS-3.6.0/ && gfortran -O3 -std=legacy -m64 -fno-second-underscore -fPIC -c *.f
&& ar r libfblas.a *.o && ranlib libfblas.a && mv libfblas.a /tmp/build/.
&& cd /tmp/build/lapack-3.6.1/
&& sed -e "s/frecursive/fPIC/g" -e "s/ ..// /g" -e "s/^CBLASLIB/#CBLASLIB/g" make.inc.example > make.inc
&& make lapacklib
&& make clean
&& mv liblapack.a /tmp/build/.
&& cd /
&& export BLAS=/tmp/build/libfblas.a
&& export LAPACK=/tmp/build/liblapack.a
&& cd /action; pip install -r requirements.txt
&& rm -r /tmp/build

Partially related to #1980 and allowing for more runtime kinds.

I will close this as duplicate of https://github.com/apache/incubator-openwhisk/issues/2300.


Rango swallows key presses when dialog window is open

For example, when you right click something and open the 'save as' dialog window, commands to write one or more keys are still consumed by rango. This makes it a little bit more difficult to write a file name in the dialog window as you can't use the single letter commands there.

This is on firefox by the way

This is a difficult issue to solve. The way we press letters with Rango is we check if there's no document focused or there's no hints with those letters and then we tell talon back to press those keys. The problem is we can't send commands to Rango while the dialog is open so the request never goes through.
The only way to solve this is to reduce the timeout in rango-talon if the command is directClickElement and if we get a timeout error then type those letters. I'll see what I can do. Other than that if you're using knausj_talon you can always use press <letters> to type those letters.


error when erasing an element from a immer::flex_vectorstd::variant/optional/any

If I use a std::variant or std::optional or std::any with an immer::flex_vector like in this code:
#include <immer/flex_vector.hpp>
#include <variant>
// #include <optional>
// #include <any>

int main()
{
    using Vector = immer::flex_vector<std::variant<int, double>>;
    // using Vector = immer::flex_vector<std::optional<int>>;
    // using Vector = immer::flex_vector<std::any>;

    Vector v{1, 2, 3, 4};
    Vector v2 = v.erase(2);

    return 0;
}

I get the following error when trying to erase an element:
error: call of overloaded 'uninitialized_move(std::variant<int, double>*, std::variant<int, double>*, std::variant<int, double>*)' is ambiguous
Full output:
[build] Starting build
[proc] Executing command: d:\msys64\mingw64\bin\cmake.EXE --build d:/projects/build/test/immer_test/debug --config Debug --target all -- -j 6
[build] Scanning dependencies of target immer_test
[build] [ 50%] Building CXX object CMakeFiles/immer_test.dir/test02.cpp.obj
[build] In file included from D:/projects/install/debug/include/immer/flex_vector.hpp:23:0,
[build]                  from D:\projects\src\test\immer_test\test02.cpp:1:
[build] D:/projects/install/debug/include/immer/detail/rbts/rrbtree.hpp: In instantiation of 'void immer::detail::rbts::concat_mut_lr_l(immer::detail::rbts::rrbtree<std::variant<int, double>, immer::memory_policy<immer::free_list_heap_policy<immer::cpp_heap>, immer::refcount_policy>, 5, 4>&, immer::detail::rbts::rrbtree<std::variant<int, double>, immer::memory_policy<immer::free_list_heap_policy<immer::cpp_heap>, immer::refcount_policy>, 5, 4>::edit_t, immer::detail::rbts::rrbtree<std::variant<int, double>, immer::memory_policy<immer::free_list_heap_policy<immer::cpp_heap>, immer::refcount_policy>, 5, 4>&, immer::detail::rbts::rrbtree<std::variant<int, double>, immer::memory_policy<immer::free_list_heap_policy<immer::cpp_heap>, immer::refcount_policy>, 5, 4>::edit_t)':
[build] D:/projects/install/debug/include/immer/flex_vector.hpp:505:22:   required from 'static immer::flex_vector<T, MemoryPolicy, B, BL>&& immer::flex_vector<T, MemoryPolicy, B, BL>::concat_move(std::true_type, immer::flex_vector<T, MemoryPolicy, B, BL>&&, immer::flex_vector<T, MemoryPolicy, B, BL>&&) [with T = std::variant<int, double>; MemoryPolicy = immer::memory_policy<immer::free_list_heap_policy<immer::cpp_heap>, immer::refcount_policy>; unsigned int B = 5; unsigned int BL = 4; std::true_type = std::integral_constant<bool, true>]'
[build] D:/projects/install/debug/include/immer/flex_vector.hpp:371:25:   required from 'decltype(auto) immer::operator+(immer::flex_vector<std::variant<int, double> >&&, immer::flex_vector<std::variant<int, double> >&&)'
[build] D:/projects/install/debug/include/immer/flex_vector.hpp:424:24:   required from 'immer::flex_vector<T, MemoryPolicy, B, BL> immer::flex_vector<T, MemoryPolicy, B, BL>::erase(immer::flex_vector<T, MemoryPolicy, B, BL>::size_type) const & [with T = std::variant<int, double>; MemoryPolicy = immer::memory_policy<immer::free_list_heap_policy<immer::cpp_heap>, immer::refcount_policy>; unsigned int B = 5; unsigned int BL = 4; immer::flex_vector<T, MemoryPolicy, B, BL>::size_type = long long unsigned int]'
[build] D:\projects\src\test\immer_test\test02.cpp:14:26:   required from here
[build] D:/projects/install/debug/include/immer/detail/rbts/rrbtree.hpp:962:39: error: call of overloaded 'uninitialized_move(std::variant<int, double>*, std::variant<int, double>*, std::variant<int, double>*)' is ambiguous
[build]                      uninitialized_move(r.tail->leaf(),
[build]                      ~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~
[build]                                         r.tail->leaf() + r.size,
[build]                                         ~~~~~~~~~~~~~~~~~~~~~~~~
[build]                                         l.tail->leaf() + tail_size);
[build]                                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~
[build] In file included from D:/projects/install/debug/include/immer/detail/rbts/node.hpp:25:0,
[build]                  from D:/projects/install/debug/include/immer/detail/rbts/rrbtree.hpp:24,
[build]                  from D:/projects/install/debug/include/immer/flex_vector.hpp:23,
[build]                  from D:\projects\src\test\immer_test\test02.cpp:1:
[build] D:/projects/install/debug/include/immer/detail/util.hpp:43:6: note: candidate: auto immer::detail::uninitialized_move(Iter1, Iter1, Iter2) [with Iter1 = std::variant<int, double>*; Iter2 = std::variant<int, double>*]
[build]  auto uninitialized_move(Iter1 in1, Iter1 in2, Iter2 out)
[build]       ^~~~~~~~~~~~~~~~~~
[build] In file included from d:/msys64/mingw64/include/c++/7.3.0/memory:65:0,
[build]                  from D:/projects/install/debug/include/immer/detail/util.hpp:28,
[build]                  from D:/projects/install/debug/include/immer/detail/rbts/node.hpp:25,
[build]                  from D:/projects/install/debug/include/immer/detail/rbts/rrbtree.hpp:24,
[build]                  from D:/projects/install/debug/include/immer/flex_vector.hpp:23,
[build]                  from D:\projects\src\test\immer_test\test02.cpp:1:
[build] d:/msys64/mingw64/include/c++/7.3.0/bits/stl_uninitialized.h:862:5: note: candidate: _ForwardIterator std::uninitialized_move(_InputIterator, _InputIterator, _ForwardIterator) [with _InputIterator = std::variant<int, double>*; _ForwardIterator = std::variant<int, double>*]
[build]      uninitialized_move(_InputIterator __first, _InputIterator __last,
[build]      ^~~~~~~~~~~~~~~~~~
[build] In file included from D:/projects/install/debug/include/immer/flex_vector.hpp:23:0,
[build]                  from D:\projects\src\test\immer_test\test02.cpp:1:
[build] D:/projects/install/debug/include/immer/detail/rbts/rrbtree.hpp:975:39: error: call of overloaded 'uninitialized_move(std::variant<int, double>*, std::variant<int, double>*, std::variant<int, double>*)' is ambiguous
[build]                      uninitialized_move(r.tail->leaf(),
[build]                      ~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~
[build]                                         r.tail->leaf() + remaining,
[build]                                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~
[build]                                         l.tail->leaf() + tail_size);
[build]                                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~
[build] In file included from D:/projects/install/debug/include/immer/detail/rbts/node.hpp:25:0,
[build]                  from D:/projects/install/debug/include/immer/detail/rbts/rrbtree.hpp:24,
[build]                  from D:/projects/install/debug/include/immer/flex_vector.hpp:23,
[build]                  from D:\projects\src\test\immer_test\test02.cpp:1:
[build] D:/projects/install/debug/include/immer/detail/util.hpp:43:6: note: candidate: auto immer::detail::uninitialized_move(Iter1, Iter1, Iter2) [with Iter1 = std::variant<int, double>*; Iter2 = std::variant<int, double>*]
[build]  auto uninitialized_move(Iter1 in1, Iter1 in2, Iter2 out)
[build]       ^~~~~~~~~~~~~~~~~~
[build] In file included from d:/msys64/mingw64/include/c++/7.3.0/memory:65:0,
[build]                  from D:/projects/install/debug/include/immer/detail/util.hpp:28,
[build]                  from D:/projects/install/debug/include/immer/detail/rbts/node.hpp:25,
[build]                  from D:/projects/install/debug/include/immer/detail/rbts/rrbtree.hpp:24,
[build]                  from D:/projects/install/debug/include/immer/flex_vector.hpp:23,
[build]                  from D:\projects\src\test\immer_test\test02.cpp:1:
[build] d:/msys64/mingw64/include/c++/7.3.0/bits/stl_uninitialized.h:862:5: note: candidate: _ForwardIterator std::uninitialized_move(_InputIterator, _InputIterator, _ForwardIterator) [with _InputIterator = std::variant<int, double>*; _ForwardIterator = std::variant<int, double>*]
[build]      uninitialized_move(_InputIterator __first, _InputIterator __last,
[build]      ^~~~~~~~~~~~~~~~~~
[build] mingw32-make.exe[2]: *** [CMakeFiles\immer_test.dir\build.make:63: CMakeFiles/immer_test.dir/test02.cpp.obj] Error 1
[build] mingw32-make.exe[1]: *** [CMakeFiles\Makefile2:67: CMakeFiles/immer_test.dir/all] Error 2
[build] mingw32-make.exe: *** [Makefile:83: all] Error 2
[build] Build finished with exit code 2

Thanks for the report. Can you tell me which compiler are you using (incl version and platform)? Thanks!

Sorry, I forgot to write that info in the previous message.
Compiler: GCC 7.3.0, under MSYS2/MinGW64
Platform: Windows 10, 64 bits

Ohhh, I see the problem now. I will patch it over the weekend. Thanks!

A fix is on the way. Thanks again for reporting this!
To ensure responsive support in the future, if you are using Immer in a commercial project, please consider sponsoring the project (via Patreon or just drop me an email for other agreements smile )

Thanks.
Well, I'm not a programmer, I just program for hobby. I'm actually an artist and what got me interested in programming is basically the stuff related to image processing and graphics in general. I'm unemployed currently but I'll try to contribute something. I've also heard that tipeee is a better donation platform for people in europe (I'm from Spain).
Like I said, I'm an artist (painter/illustrator), so if you need some help in that area sometime I'm happy to help (some of my work: http://deiflou.tumblr.com/).

Aha! Thanks for the tips :)
I'm from Spain too (from Huelva) but I've been living 6 years in Berlin already. I find it amazing that as an artist you are into advanced programming topics like immutable data-structures in C++. I used to paint too when I was younger but over the years that has become a very secondary hobbie. Your porfolio looks amazing indeed! Once again: thanks for all the feedback and thorough bug reports! Hasta pronto :)


Remove most references from operators

Light changes to implement feedback from (https://github.com/HandmadeMath/HandmadeMath/issues/163). Passes tests, haven't checked codegen to see if this is an improvement yet.
I left the references in the &operator{*,/,+,-}= type operators because AFAIK they are needed (happy to change them too if wrong).

Sure, fine by me!


Provide a way to update proxy certs without re-deploying

Connected to https://github.com/Azure/aks-hybrid/issues/271
Custom certs deployment to Linux nodes is connected to proxy configs what caused issues. We now use a DaemonSet to deploy and update them.
https://github.com/Elektronenvolt/aks-hybrid/blob/main/Linux-certificate-ds.yaml


Revert "Skip a few broken packets. Disable stream processing once thr…

…eshold has been exceeded."
This reverts commit 83bb3cc1ebed271d6b8e0fa165bc51afccb3f4f9.
This change made the decoder unable to play after hitting EOS for the first time

Generally it would be better to repair a broken feature than just remove it. I think you just need to set isEnabled=true after a seek operation.
You can also take a look at my fork of the lib. It is a much improved version with lots of bug fixes and new features.

@lukasf, thanks for your comment. Based on your feedback, does this look better?
Of course I'd like to fix a broken feature instead of removing it, but sometimes that's the easiest fix with limited information. :)

Yes this looks fine!


Application-insights client for .Net Core Vuejs web application

Can not  get Application-insights client from .Net Core Vuejs web application. How to do it ?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d56ecb14-9e66-5313-a381-d2824ec3d1e2
Version Independent ID: ff056ac8-ae76-be10-6895-757333a28e05
Content: Azure Application Insights for JavaScript web apps - Azure Monitor
Content Source: articles/azure-monitor/app/javascript.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @lgayhardt
Microsoft Alias: lagayhar

Thank you for the detailed question. We are currently investigating and will update you when we have accurate information to provide.


Compilation problem

The compilation goes smoothly until this point:
AlleleParser.cpp: In member function ‘void AlleleParser::openBams()’:
AlleleParser.cpp:43: error: ‘class BamTools::BamMultiReader’ has no member named ‘SetIndexCacheMode’
AlleleParser.cpp:43: error: ‘NoIndexCaching’ is not a member of ‘BamTools::BamIndex’
make[1]: *** [AlleleParser.o] Error 1
make: *** [all] Error 2
Could you suggest how I can fix this problem?  Thanks.

Are you able to produce this error when you build freebayes using the
following commands?
git clone --recursive git://github.com/ekg/freebayes.git
cd freebayes
make
On Sat, Dec 10, 2011 at 12:12 PM, shelbirussell <
<EMAIL_ADDRESS>
wrote:

The compilation goes smoothly until this point:
AlleleParser.cpp: In member function void AlleleParser::openBams():
AlleleParser.cpp:43: error: class BamTools::BamMultiReader has no member
named SetIndexCacheMode
AlleleParser.cpp:43: error: NoIndexCaching is not a member of
BamTools::BamIndex
make[1]: *** [AlleleParser.o] Error 1
make: *** [all] Error 2
Could you suggest how I can fix this problem?  Thanks.

Reply to this email directly or view it on GitHub:
https://github.com/ekg/freebayes/issues/23

Thanks.  This worked like a charm, I just needed to download git.


Add WireLib.ParseEscapes

New WireLib function ParseEscapes. Takes a string input generated by the user (such as from a TextEntry panel) and parses escape characters out. The output is the escaped string. This is a complete implementation of Luajit's escape sequences.
Purpose:
Some user inputs have their own implementation of escaping characters. This is meant to be a unified and thorough alternative and supports the entire Lua escape sequence library.
Notes:
This differs from Lua's escaping only in that invalid escape sequences are skipped instead of erroring.
I've only implemented this in constant value more as a demonstration than anything. Any user input should be changed to use this if desired. E2's tokenizer, for example, is a noteworthy target.

Noticed E2 string literals could advantage from this. No idea where in the toolchain that would be, though.

Feels a bit uncomfortable adding any more passes. Also I exposed HexToNum because I thought it could also be useful but let me know if that's overstepping.

HexToNum can just be tonumber(x,16)

Of course. I knew I was missing something.

The implementation of this keeps bugging me, obvious global functions or not. I'm going to draft it until it feels right.

That's not even the multi-pass gsub. I'm going to bed.

I'd like to see an implementation more like

local singleCharEscapes =
{
	n = "\n", r = "\r", t = "\t", ["\\"] = "\\",
	["'"] = "'", ["\""] = "\"", a = "\a",
	b = "\b", f = "\f", v = "\v"
}

local multiCharEscapes =
{
	x = function(arg)

	end
}

str = string.gsub(str, "(\\(.?)([^\\]?[^\\]?))", function(str, i, arg)
	return singleCharEscapes[i] or (multiCharEscapes[i] and multiCharEscapes[i](arg)) or str
end)

Not as simple against unicode but I'll see what I can do.

I can only feasibly get that idea to make sense by still having unicode parsed separately, and in that case it only parses one multi condition which is \x.

I think I'll settle for this which derives from your idea but inlines each case, which looks almost like what I started with... huh.

You can't do two passes otherwise stuff like \\u will get parsed by the unicode despite it should parse to \u

Ah, you're right. Sorry, I'm not being thorough enough.

I tried your suggestion again and realized it was simpler than I had thought. The only thing I'd like to know is if you wouldn't mind having it inlined instead.

Looks pretty nice. I don't mind if its inlined.

One last thing that we might want to be done is to benchmark it and limit the string length depending on how much it can do. Like maybe feed it a string.rep("\    ", 100)

Hex should be specifically two characters long.

All right, sure

On my 2015 CPU in a server filled with sents, benchmark is between 0.001 to 0.004 seconds with 27075 semi-randomized characters and sequences with roughly 6290 matches. Even string.reping it to ~31440 matches is almost always less than a tick (0.009 seconds average). So, really inconsequential.

And I was just about to include it in the tokenizer. Oh well, another time.


ci: automate upgrade AWS CDK

Reason for this change
The AWS CDK version for this repository has not been updated. However, since it is inefficient for contributors to update this every time, we need automate process.
Description of changes
I implemented GitHub Actions that create a pull request to upgrade the AWS CDK version at 0 UTC every day.
Description of how you validated changes
I tested on https://github.com/WinterYukky/projen-auto-upgrade-cdk/pull/2.
Checklist

[x] My code adheres to the CONTRIBUTING GUIDE

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

I don't think this is a good idea. It unnecessarily forces users to update to the latest version of aws-cdk-lib when they just want to upgrade @open-constructs/aws-cdk.
But then I also think that changing a peerDeps constraint should require a major version change. 😅
As a possible solution I can only suggest what I have suggested before: A well-define regular cycle for the release of new major versions which will include lifting the minimal version requirement to whatever is latest at that time.

Ah, I see. That's right.
First, We should documentation about major version release schedule.


changed the download version of protobuf to 3.6.0

Wrote a fix to bug #22536

@angersson  Changes in pr as discussed in #23744


Upgrade build to use Maven 3.9.1

Impact

[ ] Bug fix (non-breaking change which fixes expected existing functionality)
[X] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
Liquibase build was not working with Maven  3.9.0 . Skipping it and going straight to 3.9.1

So... maven 3.9.2 is out. I'm pulling this back to development to test with this new version.


Fix : Bug in the count of characters and minutes read [fixes #1]

Fix : Bug in the count of characters and minutes read [fixes #1]

I dint get what is recorded text. can you share a screen recording about that?

I dint get what is recorded text. can you share a screen recording about that?

In this you have the option to record the text by clicking start button and adding of the notes.

but im not able to see any recording. these buttons do nothing when i check

but im not able to see any recording. these buttons do nothing when i check

After clicking start you have to speak something so that it can be recorded.

please check it again. i have updated the code.
my mic was not detecting sound, that's why some confusion happened. sorry for that !

please check it again. i have updated the code. my mic was not detecting sound, that's why some confusion happened. sorry for that !

ok no problem.


Formatter is trying to add a star to a zero-argument functor type

module type B = () => {

}

would be formatted into
module type B = (*) => {

}

, causing compile error.

probably duplicate of #467

fixed in master


replace dashmap

What's the benefit of replacing DashMap with RwLock<HashMap>>? Since it's is easy to use, I can't see reason to replace it.


Help on remote throws Http exception

?help

see "help server starting..."

?plot
An unhandled exception of type 'System.Net.HttpListenerException' occurred in mscorlib.dll
Additional information: An operation was attempted on a nonexistent network connection
   at System.Net.HttpResponseStream.BeginWrite(Byte[] buffer, Int32 offset, Int32 size, AsyncCallback callback, Object state)
   at System.IO.Stream.<>c.<BeginEndWriteAsync>b__53_0(Stream stream, ReadWriteParameters args, AsyncCallback callback, Object state)
   at System.Threading.Tasks.TaskFactory`1.FromAsyncTrim[TInstance,TArgs](TInstance thisRef, TArgs args, Func`5 beginMethod, Func`3 endMethod)
   at System.IO.Stream.BeginEndWriteAsync(Byte[] buffer, Int32 offset, Int32 count)
   at System.IO.Stream.WriteAsync(Byte[] buffer, Int32 offset, Int32 count, CancellationToken cancellationToken)
   at System.IO.Stream.<CopyToAsyncInternal>d__27.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
   at Microsoft.Common.Core.StreamExtensions.<CopyAndFlushAsync>d__0.MoveNext() in C:\RTVS\src\Common\Core\Impl\Extensions\StreamExtensions.cs:line 10
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
   at Microsoft.R.Host.Client.BrokerServices.RemoteUriWebService.<GetResponseAsync>d__4.MoveNext() in C:\RTVS\src\Host\Client\Impl\BrokerServices\RemoteUriWebService.cs:line 62
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
   at Microsoft.R.Host.Client.BrokerServices.WebServer.<DoWorkAsync>d__22.MoveNext() in C:\RTVS\src\Host\Client\Impl\BrokerServices\WebServer.cs:line 77
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.Common.Core.TaskExtensions.<>c.<DoNotWaitThreadContinuation>b__6_0(Object s) in C:\RTVS\src\Common\Core\Impl\Extensions\TaskExtensions.cs:line 91
   at System.Threading.QueueUserWorkItemCallback.WaitCallback_Context(Object state)
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
   at System.Threading.QueueUserWorkItemCallback.System.Threading.IThreadPoolWorkItem.ExecuteWorkItem()
   at System.Threading.ThreadPoolWorkQueue.Dispatch()
   at System.Threading._ThreadPoolWaitCallback.PerformWaitCallback()

Couldn't repro second time.

I am not able to repro this.

I think question if we should be handling it.

https://github.com/Microsoft/RTVS/commit/f4755eedde9c2354cf6fdf3ca5e92ccafb40472d


Local and Remote Extensions are Disappearing After Restart of the VSCode 1.74.0

All extensions are disappearing in the local pc and remote server (using Remote - SSH) after the restart of VSCode. This issue started after the new update. Installed extensions are no longer showing in the extension panel.

Same here. We have a portable mode setup: A set of extensions is installed into the VSCode installation folder, everything is packed together and then shipped to various workstations. Until VSCode 1.73.1 this worked great, now in VSCode 1.74.0 no single extension is visible upon startup.
The portable setup is created with
%VSCODEDIR%/bin/code.cmd --extensions-dir %VSCODEDIR%/data/extensions --install-extension %FILENAME%.vsix --force

The extension host log reports nothing unusual.
When I install addition extensions manually, these extensions are available.
Windows machines.

Educated guess: The portable mode extension folder is not scanned anymore ?

Same. But I don't have remote extensions, only local. Installed extensions can be displayed by typing @installed in the search bar.

Can you all please confirm following?

Are the extensions running and extensions are not shown in UI?
Does reloading the window helps?

Are the extensions running and extensions are not shown in UI?

Extensions do work BUT not shown.

Does reloading the window helps?

Reloading window does NOT help.

Are the extensions running and extensions are not shown in UI?

Extensions do work BUT not shown.

Does reloading the window helps?

Reloading window does NOT help.

However, refresh button above the search bar does help.

I can confirm the VSCode extensions are empty in the remote extension folder in the server (Remote development using SSH). I can install extensions and disappear after restarting of VSCode. However, locally install extensions are loading but not showing on the extensions side panel.

If I understand correctly, extensions are running but not shown. They are not shown when you open VS Code, but after refreshing using the button they appear.
@ndgayan How are you installing the extensions in the remote?
Can you please share following logs

F1 > Open View... > Shared
F1 > Open View... > Remote
F1 > Toggle Shared Process > Console

In the portable mode case described above:

Extensions are not neither shown nor present at all.
The content of the 'data/extensions' folder seems to be ignored.
If I install further extensions locally (non-portable), these ones are working fine and are shown.

Could be a duplicate of https://github.com/microsoft/vscode/issues/168703
May I know if the extensions views/sections are collapsed? Can you please do following and let me know if it worked?

Expand the extensions views/sections and reload the window?

@FrankBlabu Can you please file separate issue for the portable mode with steps to reproduce.

/duplicate