Deadlock when running alongside sh package

We have noticed a reproducible deadlock when running Sentry SDK (0.19.5) alongside sh (1.14.1) on Python 3.7.6 and 3.9.1.
The following works:
import sh
sh.cat(sh.false(_piped=True))

The following hangs:
import sentry_sdk
from sentry_sdk.integrations.logging import LoggingIntegration
import logging

SENTRY_URL = "..."

sentry_sdk.init(
    SENTRY_URL,
    integrations=[
        LoggingIntegration(
            event_level=logging.ERROR
        )
    ],
    debug=True,
)

import sh
sh.cat(sh.false(_piped=True)) # This hangs

After issuing SIGINT (Ctrl-C), I get the following output:
...
sh.cat(sh.false(_piped=True))
^C [sentry] DEBUG: Sending event, type:null level:error event_id:<redacted> project:<redacted> host:o238679.ingest.sentry.io
Exception in thread background thread for pid 6751:
Traceback (most recent call last):
  File "/usr/lib64/python3.7/threading.py", line 926, in _bootstrap_inner
    self.run()
  File "/usr/local/lib/python3.7/site-packages/sentry_sdk/integrations/threading.py", line 69, in run
    reraise(*_capture_exception())
  File "/usr/local/lib/python3.7/site-packages/sentry_sdk/_compat.py", line 54, in reraise
    raise value
  File "/usr/local/lib/python3.7/site-packages/sentry_sdk/integrations/threading.py", line 67, in run
    return old_run_func(self, *a, **kw)
  File "/usr/lib64/python3.7/threading.py", line 870, in run
    self._target(*self._args, **self._kwargs)
  File "/usr/local/lib/python3.7/site-packages/sh.py", line 1637, in wrap
    fn(*rgs, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/sh.py", line 2561, in background_thread
    handle_exit_code(exit_code)
  File "/usr/local/lib/python3.7/site-packages/sh.py", line 2265, in fn
    return self.command.handle_command_exit_code(exit_code)
  File "/usr/local/lib/python3.7/site-packages/sh.py", line 865, in handle_command_exit_code
    raise exc
sh.ErrorReturnCode_1:

  RAN: /usr/bin/false

  STDOUT:

  STDERR:

---------------------------------------------------------------------------
KeyboardInterrupt                         Traceback (most recent call last)
<ipython-input-2-62608a14b5f4> in <module>
      1 import sh
----> 2 sh.cat(sh.false(_piped=True)) # This hangs

/usr/local/lib/python3.7/site-packages/sh.py in __call__(self, *args, **kwargs)
   1518             stderr = open(str(stderr), "wb")
   1519
-> 1520         return RunningCommand(cmd, call_args, stdin, stdout, stderr)
   1521
   1522

/usr/local/lib/python3.7/site-packages/sh.py in __init__(self, cmd, call_args, stdin, stdout, stderr)
    782
    783             if should_wait:
--> 784                 self.wait()
    785
    786     def wait(self, timeout=None):

/usr/local/lib/python3.7/site-packages/sh.py in wait(self, timeout)
    846                 # main thread.  this allows them to bubble up
    847                 if self.process._stdin_process:
--> 848                     self.process._stdin_process.command.wait()
    849
    850             self.log.debug("process completed")

/usr/local/lib/python3.7/site-packages/sh.py in wait(self, timeout)
    829
    830             else:
--> 831                 exit_code = self.process.wait()
    832                 self._waited_until_completion = True
    833

/usr/local/lib/python3.7/site-packages/sh.py in wait(self)
   2486             timer.cancel()
   2487
-> 2488             self._background_thread.join()
   2489
   2490             if witnessed_end:

/usr/lib64/python3.7/threading.py in join(self, timeout)
   1042
   1043         if timeout is None:
-> 1044             self._wait_for_tstate_lock()
   1045         else:
   1046             # the behavior of a negative timeout isn't documented, but

/usr/lib64/python3.7/threading.py in _wait_for_tstate_lock(self, block, timeout)
   1058         if lock is None:  # already determined that the C code is done
   1059             assert self._is_stopped
-> 1060         elif lock.acquire(block, timeout):
   1061             lock.release()
   1062             self._stop()

KeyboardInterrupt:

Is there an obvious method for avoiding this deadlock?

You can run with default_integrations=False to disable the ThreadingIntegration. That will likely solve your issue.

You can run with default_integrations=False to disable the ThreadingIntegration. That will likely solve your issue.

Thank you for the quick response.
Setting default_integrations=False does avoid the deadlock, but also causes the event to not be logged by Sentry, which defeats the purpose of using Sentry.
Furthermore, I can confirm the ThreadingIntegration is causing the issue because manually importing all default integrations except ThreadingIntegration works:
import sentry_sdk
from sentry_sdk.integrations.logging import LoggingIntegration
from sentry_sdk.integrations.stdlib import StdlibIntegration
from sentry_sdk.integrations.excepthook import ExcepthookIntegration
from sentry_sdk.integrations.dedupe import DedupeIntegration
from sentry_sdk.integrations.atexit import AtexitIntegration
from sentry_sdk.integrations.modules import ModulesIntegration
from sentry_sdk.integrations.argv import ArgvIntegration
import logging

SENTRY_URL = "..."

sentry_sdk.init(
    SENTRY_URL,
    integrations=[
        LoggingIntegration(
            event_level=logging.ERROR
        ),
        StdlibIntegration(),
        ExcepthookIntegration(),
        DedupeIntegration(),
        AtexitIntegration(),
        ModulesIntegration(),
        ArgvIntegration(),
    ],
    debug=True,
    default_integrations=False,
)

import sh
sh.cat(sh.false(_piped=True)) # This doesn't hang, but event is not reported to Sentry

and adding ThreadingIntegration hangs:
import sentry_sdk
from sentry_sdk.integrations.logging import LoggingIntegration
from sentry_sdk.integrations.stdlib import StdlibIntegration
from sentry_sdk.integrations.excepthook import ExcepthookIntegration
from sentry_sdk.integrations.dedupe import DedupeIntegration
from sentry_sdk.integrations.atexit import AtexitIntegration
from sentry_sdk.integrations.modules import ModulesIntegration
from sentry_sdk.integrations.argv import ArgvIntegration
from sentry_sdk.integrations.threading import ThreadingIntegration
import logging

SENTRY_URL = "..."

sentry_sdk.init(
    SENTRY_URL,
    integrations=[
        LoggingIntegration(
            event_level=logging.ERROR
        ),
        StdlibIntegration(),
        ExcepthookIntegration(),
        DedupeIntegration(),
        AtexitIntegration(),
        ModulesIntegration(),
        ArgvIntegration(),
        ThreadingIntegration(),
    ],
    debug=True,
    default_integrations=False,
)

import sh
sh.cat(sh.false(_piped=True)) # This hangs

Is there something else that we can do to propagate these errors to Sentry without causing a deadlock?

Thank you for the quick response.
Setting default_integrations=False does avoid the deadlock, but also causes the event to not be logged by Sentry, which defeats the purpose of using Sentry.
Furthermore, I can confirm the ThreadingIntegration is causing the issue because manually importing all default integrations except ThreadingIntegration works:
import sentry_sdk
from sentry_sdk.integrations.logging import LoggingIntegration
from sentry_sdk.integrations.stdlib import StdlibIntegration
from sentry_sdk.integrations.excepthook import ExcepthookIntegration
from sentry_sdk.integrations.dedupe import DedupeIntegration
from sentry_sdk.integrations.atexit import AtexitIntegration
from sentry_sdk.integrations.modules import ModulesIntegration
from sentry_sdk.integrations.argv import ArgvIntegration
import logging

SENTRY_URL = "..."

sentry_sdk.init(
    SENTRY_URL,
    integrations=[
        LoggingIntegration(
            event_level=logging.ERROR
        ),
        StdlibIntegration(),
        ExcepthookIntegration(),
        DedupeIntegration(),
        AtexitIntegration(),
        ModulesIntegration(),
        ArgvIntegration(),
    ],
    debug=True,
    default_integrations=False,
)

import sh
sh.cat(sh.false(_piped=True)) # This doesn't hang, but event is not reported to Sentry

and adding ThreadingIntegration hangs:
import sentry_sdk
from sentry_sdk.integrations.logging import LoggingIntegration
from sentry_sdk.integrations.stdlib import StdlibIntegration
from sentry_sdk.integrations.excepthook import ExcepthookIntegration
from sentry_sdk.integrations.dedupe import DedupeIntegration
from sentry_sdk.integrations.atexit import AtexitIntegration
from sentry_sdk.integrations.modules import ModulesIntegration
from sentry_sdk.integrations.argv import ArgvIntegration
from sentry_sdk.integrations.threading import ThreadingIntegration
import logging

SENTRY_URL = "..."

sentry_sdk.init(
    SENTRY_URL,
    integrations=[
        LoggingIntegration(
            event_level=logging.ERROR
        ),
        StdlibIntegration(),
        ExcepthookIntegration(),
        DedupeIntegration(),
        AtexitIntegration(),
        ModulesIntegration(),
        ArgvIntegration(),
        ThreadingIntegration(),
    ],
    debug=True,
    default_integrations=False,
)

import sh
sh.cat(sh.false(_piped=True)) # This hangs

Is there something else that we can do to propagate these errors to Sentry without causing a deadlock?

Hey @tomazberisa, we have no other workarounds for sh. Is there a particular error you're missing?

Hey @tomazberisa, we have no other workarounds for sh. Is there a particular error you're missing?

Yes, neither of the errors outlined in my previous comment are reported:

In the case of ThreadingIntegration being disabled: an exception is raised (by sh) due to failing command (false), which is not reported to Sentry
In the case of ThreadingIntegration being enabled: we experience a deadlock which hangs indefinitely and blocks sh from raising an exception, thus blocking any downstream event destined for Sentry.

We also submitted a GitHub issue to the sh repo and their response was that Sentry's monkeypatching of the threading module is a potential root cause to this behavior. See here: https://github.com/amoffat/sh/issues/553

Yes, neither of the errors outlined in my previous comment are reported:

In the case of ThreadingIntegration being disabled: an exception is raised (by sh) due to failing command (false), which is not reported to Sentry
In the case of ThreadingIntegration being enabled: we experience a deadlock which hangs indefinitely and blocks sh from raising an exception, thus blocking any downstream event destined for Sentry.

We also submitted a GitHub issue to the sh repo and their response was that Sentry's monkeypatching of the threading module is a potential root cause to this behavior. See here: https://github.com/amoffat/sh/issues/553

Ok understood. I have looked into this deadlock for quite a while now and don't really understand what's happening. I believe you'll have to instrument sh manually for now, they should have some sort of hook to do that.

Ok understood. I have looked into this deadlock for quite a while now and don't really understand what's happening. I believe you'll have to instrument sh manually for now, they should have some sort of hook to do that.

I am experiencing this deadlock as well running Python 2.7 with Django, and our app has multiple threads performing background tasks. There are perhaps more than 1 open Python issues with respect to threading and deadlocks. One example here: https://bugs.python.org/issue25668

As there was a solution ("instrument sh by hand") I will close this issue.
I know this is not the optimal solution, but if someone can supply a PR that fixes this issue, I am more then happy to help you bring it into a shape so we can merge it!


Dashboards are not compatible with latest grafana/Influxdb

These dashboards are not working when trying to import to Grafana/InfluxDB/heapster Setup. Are they compatible with newer version of grafana?
Srinivas Kotaru

Can you please indicate which version of Heapster and which version of Graphana and InfluxDB you are using?

I'm using all latest versions
InfluxDB shell version: 1.2.2
Grafana : 4.2.0
Heapster : v1.3.0

I just updated to Grafana 4.2.0, but that image had different VOLUMEs compared to the heapster grafana image, that made it loose all the dashboards and settings. I solved this by taking the Grafana 4.2.0 image, and removing the VOLUME directive in the Dockerfile and then build/deploy that image.
@kotarusv Not sure if that's the issue you're running into, but this worked for me.


Search all SelectingItemsControl items with TextSearch on key input, not just realized ones

What does the pull request do?
This PR brings the behavior of SelectingItemControls with VirtualizingPanel presenters into parity with those controls without virtualizing presenters.
What is the current behavior?
Currently, SelectingItemsControls with VirtualizingPanel item presenters (such as ComboBox) do not search the full contents of their boxes with text search, only the currently realized controls. This means ~that if a ComboBox is closed, typing does nothing, and even when opened,~ (this was slightly wrong!) it will only jump to items immediately in the popup.
What is the updated/expected behavior with this PR?
Now, ComboBoxes and other SelectingItemsControls with virtualizing presenters will have proper TextSearch for their entire item sets.
How was the solution implemented (if it's not obvious)?
The anonymous method that existed in the SelectingItemsControl class previously has been moved into a static method in ItemsPresenter. It's now used there and in VirtualizingPanel to check search their items/children for text search. (Note: I don't know if there is a preferred place for a static method like this; happy to move as appropriate.)
Unit test case was added to an existing test and the test was adapted so that ComboBoxes in tests will use a VirtualizingStackPanel like the real controls do to ensure the tests fail without this change.
Checklist

[x] Added unit tests (if possible)?
[x] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/avalonia-docs with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues
Fixes #12490.

@jonko0493,
Please read the following Contributor License Agreement (CLA). If you agree with the CLA, please reply with the following:
@cla-avalonia agree

Contributor License Agreement
Contribution License Agreement
This Contribution License Agreement ( “Agreement” ) is agreed to by the party signing below ( “You” ),
and conveys certain license rights to the AvaloniaUI OÜ ( “AvaloniaUI OÜ” ) for Your contributions to
AvaloniaUI OÜ open source projects. This Agreement is effective as of the latest signature date below.
1. Definitions.
“Code” means the computer software code, whether in human-readable or machine-executable form,
that is delivered by You to AvaloniaUI OÜ under this Agreement.
“Project” means any of the projects owned or managed by AvaloniaUI OÜ and offered under a license
approved by the Open Source Initiative (www.opensource.org).
“Submit” is the act of uploading, submitting, transmitting, or distributing code or other content to any
Project, including but not limited to communication on electronic mailing lists, source code control
systems, and issue tracking systems that are managed by, or on behalf of, the Project for the purpose of
discussing and improving that Project, but excluding communication that is conspicuously marked or
otherwise designated in writing by You as “Not a Submission.”
“Submission” means the Code and any other copyrightable material Submitted by You, including any
associated comments and documentation.
2. Your Submission. You must agree to the terms of this Agreement before making a Submission to any
Project. This Agreement covers any and all Submissions that You, now or in the future (except as
described in Section 4 below), Submit to any Project.
3. Originality of Work.  You represent that each of Your Submissions is entirely Your
original work. Should You wish to Submit materials that are not Your original work,
You may Submit them separately to the Project if You (a) retain all copyright and
license information that was in the materials as you received them, (b) in the
description accompanying your Submission, include the phrase "Submission
containing materials of a third party:" followed by the names of the third party and any
licenses or other restrictions of which You are aware, and (c) follow any other
instructions in the Project's written guidelines concerning Submissions.
4. Your Employer. References to “employer” in this Agreement include Your employer or anyone else
for whom You are acting in making Your Submission, e.g. as a contractor, vendor, or agent. If Your
Submission is made in the course of Your work for an employer or Your employer has intellectual
property rights in Your Submission by contract or applicable law, You must secure permission from Your
employer to make the Submission before signing this Agreement. In that case, the term “You” in this
Agreement will refer to You and the employer collectively. If You change employers in the future and
desire to Submit additional Submissions for the new employer, then You agree to sign a new Agreement
and secure permission from the new employer before Submitting those Submissions.
5. Licenses.
a. Copyright License. You grant AvaloniaUI OÜ, and those who receive the Submission directly
or indirectly from AvaloniaUI OÜ, a perpetual, worldwide, non-exclusive, royalty-free, irrevocable
license in the Submission to reproduce, prepare derivative works of, publicly display, publicly perform,
and distribute the Submission and such derivative works, and to sublicense any or all of the foregoing
rights to third parties.
b. Patent License. You grant AvaloniaUI OÜ, and those who receive the Submission directly or
indirectly from AvaloniaUI OÜ, a perpetual, worldwide, non-exclusive, royalty-free, irrevocable license
under Your patent claims that are necessarily infringed by the Submission or the combination of the
Submission with the Project to which it was Submitted to make, have made, use, offer to sell, sell and
import or otherwise dispose of the Submission alone or with the Project.
c. Other Rights Reserved. Each party reserves all rights not expressly granted in this Agreement.
No additional licenses or rights whatsoever (including, without limitation, any implied licenses) are
granted by implication, exhaustion, estoppel or otherwise.
6. Representations and Warranties. You represent that You are legally entitled to grant the above
licenses. You represent that each of Your Submissions is entirely Your original work (except as You may
have disclosed under Section 3 ). You represent that You have secured permission from Your employer to
make the Submission in cases where Your Submission is made in the course of Your work for Your
employer or Your employer has intellectual property rights in Your Submission by contract or applicable
law. If You are signing this Agreement on behalf of Your employer, You represent and warrant that You
have the necessary authority to bind the listed employer to the obligations contained in this Agreement.
You are not expected to provide support for Your Submission, unless You choose to do so. UNLESS
REQUIRED BY APPLICABLE LAW OR AGREED TO IN WRITING, AND EXCEPT FOR THE WARRANTIES
EXPRESSLY STATED IN SECTIONS 3, 4, AND 6 , THE SUBMISSION PROVIDED UNDER THIS AGREEMENT IS
PROVIDED WITHOUT WARRANTY OF ANY KIND, INCLUDING, BUT NOT LIMITED TO, ANY WARRANTY OF
NONINFRINGEMENT, MERCHANTABILITY, OR FITNESS FOR A PARTICULAR PURPOSE.
7. Notice to AvaloniaUI OÜ. You agree to notify AvaloniaUI OÜ in writing of any facts or
circumstances of which You later become aware that would make Your representations in this
Agreement inaccurate in any respect.
8. Information about Submissions. You agree that contributions to Projects and information about
contributions may be maintained indefinitely and disclosed publicly, including Your name and other
information that You submit with Your Submission.
9. Governing Law/Jurisdiction. This Agreement is governed by the laws of the Republic of Estonia, and
the parties consent to exclusive jurisdiction and venue in the courts sitting in Talinn,
Estonia. The parties waive all defenses of lack of personal jurisdiction and forum non-conveniens.
10. Entire Agreement/Assignment. This Agreement is the entire agreement between the parties, and
supersedes any and all prior agreements, understandings or communications, written or oral, between
the parties relating to the subject matter hereof. This Agreement may be assigned by AvaloniaUI OÜ.
AvaloniaUI OÜ dedicates this Contribution License Agreement to the public domain according to the Creative Commons CC0 1.

@cla-avalonia agree

Everything was passing prior to me merging master... not sure what changed, could be transient failures?

@jonko0493 don't worry tests are a bit flaky atm

Hi folks! Is there something more I need to do to get this reviewed? It is needed for the application I'm developing and it's been a month since it was opened. Thanks!


Code highlighting

Nicely render code blocks in markdown
Suggest we use e.g. use https://github.com/timlrx/rehype-prism-plus
Acceptance

[x] Code sections are nicely rendered (i.e. syntax highlight) ✅ using rehype-prism-plus
[x] Have an example in demo page ✅ /demo/code-highlight
[x] Bonus: copy and paste button seen on hover

Tasks

[x] Do we use prism or highlight.js. ✅2022-06-29 Answer: prism i think (same speed, more plugins (?))
[x] Add the plugin ✅ 2022-07-11
[x] Add a section to demo to show it ✅ 2022-06-11
[x] Copy/paste button ✅ 2022-07-11

[x] Add component Pre.js to mdx

[x] Styles used - prism vs code dark from https://github.com/PrismJS/prism-themes

FIXED. Looks great. See https://flowershow.app/demo/wiki-links
PS: @khalilcodes one thing is i think rather than having in demo i think we put this in /docs/ since it is basically documenting and demoing a feature.


Add --match option to match a tag pattern (like git describe --match )

It would be very useful to have an option to only consider tags that match a pattern. Like git describe --match <pattern> does.
This would allow a usecase where you might have a file VERSION-TAG in your repo and you call git-semver as:
git-semver --no-prefix --match $(cat VERSION-TAG)
This would allow better branch management of your repo.
Here is a simplified scenario:
          D---E---F
         /
A---B---C---G---H---I

A is tagged v0.1.0-alpha and has VERSION-TAG as v0.1.0-alpha
B's version is 0.1.0-alpha.dev.1+meta
C is tagged with both v1.0.0-rc and v1.1.0-alpha. C's commit also changes VERSION-TAG to v1.0.0 (this will match the v1.0.0-rc tag).
D's commit changes VERSION-TAG to v1.1.0-alpha. D's version is 1.1.0-alpha.dev.1+meta. F's version is 1.1.0-alpha.dev.3+meta
G's versoin is 1.0.0-rc.dev.1+meta. H's version is 1.0.0-rc.dev.2+meta
I is tagged as v1.0.0 and I's version is 1.0.0

This sound like a legit idea. Would you feel comfortable to provide a PR?

This is now implemented in version v6.8.1. Let me know if this suits your needs.


Make test that contrasts are tab delimited

Sometimes the separators end up being spaces for strange reasons, and everything breaks. Or worse, it runs to completion but skips some of the contrasts and then the outputs are strange but the troubleshooting is extra hard.
Probably write a test into module 1.

And dealing with trailing whitespaces!

Closed with merge #244

Closed with merge https://github.com/R-ODAF/R-ODAF_Health_Canada/pull/244


Add notes to check version on problem

Signed-off-by: TomSweeneyRedHat<EMAIL_ADDRESS>Add a note scattered about the site to have people check their version before reporting an issue against Podman.  I included a "Latest Version" number on the README.md file and tried to get that auto update, but failed to find an easy way.  So went with a note reminder in version/version.go to update the README.md as well.  We do a similar thing in Buildah.

LGTM

You got GOFMT errors.
LGTM once these are fixed.

One rogue space zapped, fingers crossed for green test buttons.

@rh-atomic-bot r+


Use const to replace the hard code

like here : https://github.com/badboy-huaqiao/edgex-ui-go/blob/master/mqtt.go#L35
and here : https://github.com/badboy-huaqiao/edgex-ui-go/blob/master/proxyhandler.go#L33
and here : https://github.com/badboy-huaqiao/edgex-ui-go/blob/master/websocket.go#L35
many place : https://github.com/badboy-huaqiao/edgex-ui-go/blob/master/filter.go#L46

closing this...


Pluck out licenser metrics

We now run the LM on a new box and, well, we really should've been checking the whole time.
This causes the plugin to verify that the box is an LM before emitting metrics.

(assuming you got metrics in qa)
lgtm

:+1:


MaterialTextField怎么让text文本居中

Thank for using our package and pointing this issue. We have fixed this in new release 1.0.5.


Subscription stream transformer missing

Describe the issue
Subscription stream minnsin transformer to transform error into query result object
https://github.com/zino-app/graphql-flutter/blob/beta/packages/graphql/lib/src/core/query_manager.dart#L68
In case if there is an error in request the subscription stream does not handle errors correctly to match the query flow.
In case of error simulation in tests
when(
        mockHttpClient.send(any),
      ).thenThrow(Error());

you'll get an error
package:gql_http_link/src/link.dart 157:7                      HttpLink._executeRequest
package:gql_http_link/src/link.dart 97:32                      HttpLink.request
===== asynchronous gap ===========================
dart:async                                                     _AsyncStarStreamController.addError
package:gql_http_link/src/link.dart                            HttpLink.request
===== asynchronous gap ===========================
dart:async                                                     _asyncErrorWrapperHelper
package:gql_link/src/link.dart 125:36                          _LinkChain.request.<fn>.<fn>
package:gql_link/src/link.dart 126:8                           _LinkChain.request
package:graphql/src/core/query_manager.dart 68:17              QueryManager.subscribe
===== asynchronous gap ===========================
dart:async                                                     _StreamImpl.listen
package:graphql_flutter_bloc/src/subscription/bloc.dart 87:42  SubscriptionBloc.mapEventToState
===== asynchronous gap ===========================
dart:async                                                     _StreamImpl.listen
package:bloc/src/bloc.dart 240:7                               Bloc._bindEventsToStates
package:bloc/src/bloc.dart 22:5                                new Bloc
package:graphql_flutter_bloc/src/subscription/bloc.dart 18:9   new SubscriptionBloc
test/subscription_test.dart 21:9                               new TestSubscriptionBloc
test/subscription_test.dart 49:30                              main.<fn>.<fn>

ServerException(originalException: Instance of 'Error', parsedResponse: null)

Expected behavior
Transforms error into QueryResult object for consistency.
Temp workaround
subscription = client.subscribe(event.options).handleError(Object error) {
    QueryResult queryResult = QueryResult(source: QueryResultSource.network);

    queryResult.exception = coalesceErrors(
      exception: queryResult.exception,
      linkException: translateFailure(error),
    );

    // do with query result whatever you need
  });

closed by #768

:tada: This issue has been resolved in version 4.0.0-beta.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 4.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 4.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


DUPING GLITCH

Issue Description:
This Issue allows anyone to DUPLICATE ABSOLUTELY ANYTHING
using capacitor banks
What happens:
you start by having any power bank with any amount of power in it.
place a powered item in the bank to charge
use your hotkeys to take out the charging item
if done fast enough (its pretty slow) you will be able to take out a second of that same item
drop the first legit item
then drop anything you want out of your inventory
then move the duplicated item anywhere in your inv
you now have duped items
Affected Versions (Do not use "latest"):
EnderIO-1.10.2-<IP_ADDRESS>_beta
EnderCore-1.10.2-<IP_ADDRESS>-beta
Modpack: Direwolf20 (latest 1.0)

This was solved in a recent update of the modpack (version 1.1.0.


Make Marketo Forms Easy to Configure

Required Information

[x] AEM Version, including Service Packs, Cumulative Fix Packs, A
[x] ACS AEM Commons Version: All
[x] Reproducible on Latest? yes

Expected Behavior
As an administrator of AEM, I want to be able to easily configure the Marketo integration and understand any issues I have made with the configuration.
Actual Behavior
The Marketo integration does not allow for validation of the configuration before putting a form on the page and does not expose sufficient information either via the UI or logs to enable an administrator to validate the configuration.
Steps to Reproduce

Create a new Marketo Cloud configuration
Set configuration values according to tooltips and save
Configure embed component for the relevant site
Add an embed component onto the site
See that either you cannot select from forms or that the form does not display
Bang head against desk
Check logs and see that they are extremely unhelpful
Consider career in chicken farming
In desperation, the original developer who luckily now works on Adobe

Links
I'll also update the docs :-)

Screenshot of the Configuration Check modal:

And the success modal:


INW token logo update

On Subwallet, the logo for the INW token is the same as AZERO. Please help change it to the correct INW logo.
INW token logo (36x36): https://drive.google.com/file/d/1LtNH6eCqXiAabTyaJOG0vceqqFDqr3or/view?usp=sharing
Thank you!

Close the issue on this repository, move to "SubWallet-Chain" (https://github.com/Koniverse/SubWallet-Chain/issues/3)


Evolve[Doc]: Command as project description

A command example can be the description.

I think the project description is fine

I think the project description is fine


Adding Todos API resource

Adding GitLab Todo API resource to support ListTodos, MarkTodoRead and MarkTodosRead
https://docs.gitlab.com/ce/api/todos.html

thanks @svanharmelen for your time reviewing.  I've amended all the suggested changes except the interface{} on the Target field.
I'll take a look at that in bit and add as a separate commit to create the full struct for the Target field.

hey @svanharmelen, I made the suggested change to remove the interface{} on the Target field.   Since some of the fields are specific to the target type, i made some as pointers with omitempty so that they can be omitted?? Let me know if this works or if there is a better way to handle this case.
Thanks again.

Hi @eguevara! It's starting to look very nice 👍
As for the fields that are specific to a certain todo type, please do not make them pointers and do not add the omitempty tag. I would say keep the comment saying the fields are only used for specific types, but just thread the fields like all other fields.
Once that is done, we should be good to merge this one. Thx!

Thanks @svanharmelen.  I amended the comit to remove the pointers.

Thanks for your efforts @eguevara! LGTM 😀 🎉


Issues with tables

The tables are not being rendered, it generates it as plain text. When using other .md to pdf generators, it works, such as markdown to pdf. Is this a known issue ? Is there a work around ?
thanks !

This package uses markdown-it for md to html parsing. markdown-it converts by markdown spec. If you have an issue, please fill an issue in following repo:
https://github.com/markdown-it/markdown-it
My tables are rendering fine. I´m using markdown-themeable-pdf every day and with tables. Please check your syntax.


Escape double quotes for pluralized translations

Escape double quotes when transpiring translateJS and translate calls
When JSON is represented as a string literal between single quotes, double quotes need to be escaped twice and single quotes need to be escaped once. This PR adds the proper escapement to the double quotes. Previously double quotes weren't escaped at all would cause jambo to produce invalid JSON when a translation included a double quote.
This change also causes the transpiled calls to also escape all the double quotes surrounding JSON keys and values. While the additional escapes may seem unnecessary, they are still valid.  Previously these escapes were being created inside calls to ANSWERS.registerTemplate() because templates were double-quoted strings. With these new changes, the double quote escapes surrounding JSON keys and values inside ANSWERS.registerTemplate() calls are still the same. The only new changes are the extra escapes before double quotes within pluralized translations strings.
J=SLAP-636
TESt=manual, auto
Add new tests that cover the escaped double quotes. Also add more intermixed markdown tests.
Manually tests an internationalized jambo site and observe pluralized translations containing double quotes working that were previously broken.

What if we were to not use JSON serialization at all? When creating calls to ANSWERS.translateJS, we'd just pass an actual object. That'd be nice because we wouldn't have to serialize just to deserialize in translateJS. When creating calls to runtimeTranslation, we'd pass named arguments for each entry in the object. For example:
{ phrase: 'Blah blah', pluralForm1: 'blahs blahs' pluralForm2: 'blah blahs' }
would become:
{ runtimeTranslation phrase='Blah blah' pluralForm1='blahs blahs' pluralForm2='blah blahs' }.
Would this help the situation at all? As we discussed yesterday, I'm worried by the complexity of the escaping logic we have sprinkled around the codebase. I think that with the approach outline above, we'd just need to escape single quotes (since we wrap everything in them).

I think that makes a lot of sense. It's simpler and less error prone. Currently the SDK translator attempts a JSON parse on the phrase parameter. If it's successful, it assumes a pluralization is taking place. If it fails, it assumes it is translating a single string. It works, but it requires that we always send it valid JSON, which seems difficult to always get right as a string literal inside a HBS template. I'm going to try out Oliver's suggestion of the double stringify and see how robust that is. If that doesn't work, I think we'll want to make this change.

Spoke with @tmeyer2115 offline. He said that we can get this fix in now and we can revisit our JSON serialization and escaping with another item.


Update the DI config & the base project structure with initial components

Why
The current template project does not fully have DI configured & misses some initial components, such as ViewModel or UseCase. While we're trying to keep the base template clean and neat with only base components & setup, there is still the need to have a full DI configuration & some initial base components to start to reduce the team effort & avoid inconsistency usage.
Here is what we should add:

di.dart (at lib/di package).
network_module.dart to configure ApiService to be used in CredentialRepositoryImpl (at lib/di/module package).
request/response models should be located at lib/api/request and lib/api/response.
the model will be located at lib/model.
initial use_case and view_model, keep them clean asap.

Who Benefits?
Developers

@doannimble Note that we should ensure to have necessary unit tests 🙏


Change order to Payment Pending instead of on-hold

I think it makes more sense (and ties in better with other woocommerce payment plugins) to have the plugin stay in the payment pending state (wc-pending) until it has been paid (at which point people can choose through the plugin which state it should be in).
If it goes to on-hold, an order email is fired out to both the shop admin and the customer saying the order has been received but it shouldn't really do this until the payment is confirmed IMO.
Or perhaps if you disagree, we could be given the option to choose this as well?
Thanks

This is something that we go around and around about actually.  Probably the best solution is to provide the capability for the merchant to change the order disposition according to their business requirements.  What do you think?

Yes, I agree - that adds a lot of flexibility that way.
For me, I have changed it in my install for now - I ship physical products,
and the way my fulfillment/shipping software works it is better to have it
Pending until it is complete when it moves to Processing. For a shop that
sells downloadable/virtual products, it may be better to do it a completely
different way.
Like a lot of things - it probably just comes down to user preference (and
force of habit - I am used to it going to pending)
On Thu, Mar 12, 2015 at 3:00 AM, Rich Morgan<EMAIL_ADDRESS>wrote:

This is something that we go around and around about actually. Probably
the best solution is to provide the capability for the merchant to change
the order disposition according to their business requirements. What do you
think?
—
Reply to this email directly or view it on GitHub
https://github.com/bitpay/woocommerce-plugin/pull/50#issuecomment-78415661
.

If I recall correctly, the decision to have it go to On-Hold was to match the process of the gateways bundled with WooCommerce at the time when it was developed.  Looking at the latest version, it looks like On-Hold is set via the IPN if there is a discrepancy with the IPN and order values to allow for manual checking.
There may be users who already depend/expect the on-hold status for new orders with our plugin, so we won't pull this PR at its current state. However, allowing the merchant to decide via configuration is a solution we would accept so If you want to make this change into a configuration setting, code it and reopen the PR.


refactor: Common static analysis issues

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.staticanalysis.CommonStaticAnalysis?organizationId=T3BlblJld3JpdGU%3D

Doesn't help readability here; we can revisit this later once this has crystallized a bit more.


Example in transitioning to Shapely's functions

Hi,
We've been trying to switch to Shapely's implementation as per your suggestion but are having trouble. Our previous example looked something like this:
# Old approach
# Now get coords in the correct format
coords = points_to_coords(s_gpd.geometry)
region_polys, region_pts = voronoi_regions_from_coords(coords, bounding_shape)

# Reconstruct a dataframe with points and polys
s_gpd["site_voronoi"] = pd.Series()

# Loop through the region polys and assign them to the correct index of points
for key in region_polys:
    idx = region_pts[key][0]
    s_gpd.loc[s_gpd.index[idx], "site_voronoi"] = region_polys[key]

and we think the new approach would just be:
# New approach with Shapely
s_gpd["new_site_voronoi"] = voronoi_polygons(s_gpd.geometry, extend_to=bounding_shape)

But this seems to give a series of multipoint geometry with the bounding shape completely filled. Do you have any suggestions as to why that might be?

Sorry, but I neither maintain Shapely nor do I maintain or provide support for this repository anymore since I switched jobs. I simply don't have the time for this in my spare time. I suggest you head over to the Shapely documentation / GitHub issues for that question.

Okay thank you!


Settings: "Show other matching extensions" context

Issue Type: Bug
Testing #57304
Speaking of "other extensions" when I'm in settings (not extensions) misses context.

Maybe call it "Search in extension".
VS Code version: Code - Insiders 1.27.0-insider (43676d0cb7db94e2fa771f1d10bff10eaa01b453, 2018-08-28T05:12:24.162Z)
OS version: Darwin x64 17.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (8 x 2200)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 2, 3

Memory (System)
16.00GB (0.16GB free)

Process Argv
/Applications/Visual Studio Code - Insiders.app/Contents/MacOS/Electron --user-data-dir /Users/chrmarti/Development/repos/tmp/user2 --extensions-dir /Users/chrmarti/Development/repos/tmp/ext2

Screen Reader
no

VM
33%

Extensions: none

I would say "Show matching extensions" but I think it needs to be clear that the results exclude extensions that you already have installed.

The button's position is also not optimal. It is too far from the text field from which it uses the search text. Maybe move it to the ... menu below the filter field?

I'm going to call this a dupe of https://github.com/Microsoft/vscode/issues/57601


weight damage in .keras format

Packages

Keras-nightly

Describe

Saved model in .keras format
Reload it to check similarity but weight missmatch.

Gist

colab

Others
Potential similar issue. https://github.com/keras-team/keras/issues/19019

Hi @innat ,
I have replicated the reported behaviour with tensorflow and torch backends also. Attaching gist-tf and gist-torch.
Note: I tested with explicit serialization/deserialization for custom objects and still behaviour is same. Need to check the reason for this behaviour.

@SuryanarayanaY Thank you for the response. One of the suspected area in the code is mention below (possibly).
    def build(self, input_shape):
        self.safe_window_size = get_window_size(
            input_shape[1:-1], self.window_size
        )
        self.attention_layer = Attention(
            dim=self.dim,
            window_size=self.safe_window_size,
            num_heads=self.num_heads,
            qkv_bias=self.qkv_bias,
            qk_scale=self.qk_scale,
            attn_drop_rate=self.attn_drop_rate,
            proj_drop_rate=self.proj_drop_rate
        )

This is what I stated in this old ticket.
Let me know if I need to add any information.

Any update? cc. @nkovela1


Different systems have very different results？Using win10 always gets bad results.

When I run the project on colab or linux, everything is all right, but when I run it on my own notebook( win10, LTSC, 1660ti ) with the same source file , it always takes me bad results, as shown in the figure below. This problem makes me so puzzled, actually all the files and execution steps are the same, what strange environmental problems caused it, do you have any idea? Thanks very much.

I never used win10, so no idea. Maybe someone else could help you.

same here. at first i thouht torch 1.6.0 causes this issue, so i downgraded it to 1.0.0 but still same. colab results are better.


Can't npm install Version 0.12.0-beta.6

I get these errors when compiling with beta 6.
node_modules/nexus/dist/definitions/args.d.ts:58:36 - error TS2344: Type 'string' does not satisfy the constraint '"Int" | "Float" | "String" | "Date" | "Boolean" | "QueryFilterType" | "SortOrderEnum" | "StatusEnum" | "ID"'.

58   protected config: NexusArgConfig<string>;
                                      ~~~~~~

node_modules/nexus/dist/definitions/args.d.ts:59:54 - error TS2344: Type 'string' does not satisfy the constraint '"Int" | "Float" | "String" | "Date" | "Boolean" | "QueryFilterType" | "SortOrderEnum" | "StatusEnum" | "ID"'.

59   constructor(name: TypeName, config: NexusArgConfig<string>);
                                                        ~~~~~~

node_modules/nexus/dist/definitions/args.d.ts:60:34 - error TS2344: Type 'string' does not satisfy the constraint '"Int" | "Float" | "String" | "Date" | "Boolean" | "QueryFilterType" | "SortOrderEnum" | "StatusEnum" | "ID"'.

60   readonly value: NexusArgConfig<string>;
                                    ~~~~~~

node_modules/nexus/dist/definitions/inputObjectType.d.ts:30:32 - error TS2344: Type 'TypeName' does not satisfy the constraint '"Int" | "Float" | "String" | "Date" | "Boolean" | "QueryFilterType" | "SortOrderEnum" | "StatusEnum" | "ID"'.
  Type 'string' is not assignable to type '"Int" | "Float" | "String" | "Date" | "Boolean" | "QueryFilterType" | "SortOrderEnum" | "StatusEnum" | "ID"'.
    Type 'TypeName' is not assignable to type '"ID"'.
      Type 'string' is not assignable to type '"ID"'.

30   asArg(cfg?: NexusAsArgConfig<TypeName>): NexusArgDef<TypeName>;

I tried a build on nexus. There is no type error raised at that time. This is because none of the nexus runtime-generated types are present at that point. Once they are, then String and TypeName become invalid.

Did some more digging;
Only going to focus on some of the type errors right now;
First, the type error comes from the strings in here:
export declare class NexusArgDef<TypeName extends string> {
  readonly name: TypeName
  protected config: NexusArgConfig<string>
  constructor(name: TypeName, config: NexusArgConfig<string>)
  readonly value: NexusArgConfig<string>
}

coming from here:
export interface NexusArgConfig<T extends GetGen<"allInputTypes", string>>
  extends NexusAsArgConfig<T> {
  /**
   * The type of the argument, either the string name of the type,
   * or the concrete Nexus type definition
   */
  type: NexusArgConfigType<T>
}

coming from here:
export declare type GetGen<
  K extends GenTypesShapeKeys,
  Fallback = any
> = NexusGen extends infer GenTypes
  ? GenTypes extends GenTypesShape
    ? GenTypes[K]
    : Fallback
  : Fallback;

About this last block of code:

NexusGen is a global interface type; Nexus exports it. It is just an empty name. The macro-like Nexus typegen feature (build at runtime, apps, using Nexus) generates types that use declaration merging to extend NexusGen.
infer tells TypeScript to defer calculation of the type until build time (working explanation, probably leaky/bad)
GenTypes (infer GenTypes) is a type variable name, used to reference whatever the outcome of infer will be
GenTypesShape is the following names are record keys mapped to any:

export declare type GenTypesShapeKeys =
  | "context"
  | "inputTypes"
  | "rootTypes"
  | "argTypes"
  | "fieldTypes"
  | "allTypes"
  | "inheritedFields"
  | "objectNames"
  | "inputNames"
  | "enumNames"
  | "interfaceNames"
  | "scalarNames"
  | "unionNames"
  | "allInputTypes"
  | "allOutputTypes"
  | "allNamedTypes"
  | "abstractTypes"
  | "abstractResolveReturn";

finally if GenTypes is a subtype of the above said record (b/c values are any a simple key subset is sufficient to be a subtype) then we get to GenTypes[K] which if we look way back up we see k is allInputTypes

in order to understand what that lookup produces consider what the infer resolves to, once nexus typegen has run for an app, e.g.:

export interface NexusGenTypes {
  context: context.Context;
  inputTypes: NexusGenInputs;
  rootTypes: NexusGenRootTypes;
  argTypes: NexusGenArgTypes;
  fieldTypes: NexusGenFieldTypes;
  allTypes: NexusGenAllTypes;
  inheritedFields: NexusGenInheritedFields;
  objectNames: NexusGenObjectNames;
  inputNames: NexusGenInputNames;
  enumNames: NexusGenEnumNames;
  interfaceNames: NexusGenInterfaceNames;
  scalarNames: NexusGenScalarNames;
  unionNames: NexusGenUnionNames;
  allInputTypes: NexusGenTypes['inputNames'] | NexusGenTypes['enumNames'] | NexusGenTypes['scalarNames'];
  allOutputTypes: NexusGenTypes['objectNames'] | NexusGenTypes['enumNames'] | NexusGenTypes['unionNames'] | NexusGenTypes['interfaceNames'] | NexusGenTypes['scalarNames'];
  allNamedTypes: NexusGenTypes['allInputTypes'] | NexusGenTypes['allOutputTypes']
  abstractTypes: NexusGenTypes['interfaceNames'] | NexusGenTypes['unionNames'];
  abstractResolveReturn: NexusGenAbstractResolveReturnTypes;
}

the value picked is allInputTypes: NexusGenTypes['inputNames'] | NexusGenTypes['enumNames'] | NexusGenTypes['scalarNames']
string is not present there

AFICT fallback case will never be hit in the context of a Nexus typegen built app; that is, can either of the following be false in a case of a nexus typegen built  app?

? GenTypes extends GenTypesShape
? GenTypes[K]

I'm going with the present presumption that they cannot. I am also going under the presumption right now that the type returned by GenTypes[K] will never include a string.
If I am correct, then it seems the solution is to stop using string. But I have taken some guesses and may well be wrong, in which case the fix may be in a different part of the system.
Investigating further.

One place to learn more about infer is here https://www.typescriptlang.org/docs/handbook/release-notes/typescript-2-8.html

On my demo app, I modified the string and TypeName to be "String" types. At this point TypeScript is able to compile the app. As far as I can tell nothing broke. But doing this caused Nexus to fail type checking during build. So, I ended up with this approach that satisfies both sides:
diff --git a/src/definitions/args.ts b/src/definitions/args.ts
index 8205631..211c16f 100644
--- a/src/definitions/args.ts
+++ b/src/definitions/args.ts
@@ -65,7 +65,7 @@ export interface NexusArgConfig<T extends GetGen<"allInputTypes", string>>
 export class NexusArgDef<TypeName extends string> {
   constructor(
     readonly name: TypeName,
-    protected config: NexusArgConfig<string>
+    protected config: NexusArgConfig<GetGen<"allInputTypes", string>>
   ) {}
   get value() {
     return this.config;
diff --git a/src/definitions/inputObjectType.ts b/src/definitions/inputObjectType.ts
index a17dc3c..efcfa48 100644
--- a/src/definitions/inputObjectType.ts
+++ b/src/definitions/inputObjectType.ts
@@ -1,3 +1,4 @@
+import { GetGen } from "../typegenTypeHelpers";
 import { NexusTypes, NonNullConfig, withNexusSymbol } from "./_types";
 import { InputDefinitionBlock } from "./definitionBlocks";
 import { assertValidName } from "graphql";
@@ -35,7 +36,9 @@ export class NexusInputObjectTypeDef<TypeName extends string> {
   get value() {
     return this.config;
   }
-  asArg(cfg?: NexusAsArgConfig<TypeName>): NexusArgDef<TypeName> {
+  asArg(
+    cfg?: NexusAsArgConfig<GetGen<"allInputTypes", string>>
+  ): NexusArgDef<TypeName> {
     return arg({ ...cfg, type: this });
   }
 }

I simply continue to use the GetGen<"allInputTypes", string> type which produces string at nexus build time and some gql input type at nexus typegen time.
My understanding is still shaky, will do two things:

start a PR for further discussion with @tgriesser
analyze the code further to better understand how the type modelling/DSL is working around GQL input types. For example I found this comment and would like to understand it better:

export interface NexusAsArgConfig<T extends GetGen<"allInputTypes", string>>
  extends CommonArgConfig {
  /**
   * Configure the default for the object
   */
  default?: GetGen2<"allTypes", T>; // TODO: Make this type-safe somehow
}


Unable to use gax in Jest with jsdom test environment

See https://github.com/firebase/firebase-admin-node/issues/1135#issuecomment-859342414

We need to change isBrowser from checking the window to the complete feature detection. The main thing here is to check if we need to use window.fetch or node-fetch, I don't think we use isBrowser for anything else.


Version 1.10.0

Coverage remained the same at 92.959% when pulling f557f1040d7268d20c2a0652ab7928ffd1e0c22c on Juanlu001:v-1-10-0 into 25f43f2a1257d09cafd764e4b23ec832707d59d5 on satellogic:master.


Phone numbers in Android Chrome 56

When i enter a phone number on the demo page, the pointer jumps. Appears on Android 7 Chrome 56 # @

The new release 3.3.5-173 broke again in phone numbers in Android Chrome 56 and Samsung with Smart Keyboards.

@fakofsky , @diegoagudo ,
Retry with version 3.3.5-175

For mask currency ou numeric keep break

@diegoagudo?
It is mostly due to the numericInput option.  But even when setting it to false it doesn't let you enter the radixpoint. (caret positioning)
But phone entry should be ok,

Remove Phone alias #1981


Bump Dependencies #11799938832

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/11799938832

 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.


Add in documentation of what could go in rancher-compose.yml

Rancher Version:
Docker Version: 1.10.3
OS: CentOS
Steps to Reproduce:

Start a rancher environment with at least one host
Get to the Add Stack menu in the web UI
Try use the following docker-compose.yml

winning-calculation-service_webserver:
  image: hello-world

Results:
Activating (Bad response statusCode [422]. Status [422 status code 422]. Body: [code=InvalidCharacters, fieldName=name] from [http://localhost:8080/v1/projects/1a12/services])
Expected:
Rancher starts the hello world docker app
It appears that the underscore character is not a valid character for naming a service in Rancher.
It is a valid character in docker-compose however.
Also, it does not appear the Rancher supports version 2 of the docker compose file format.
Also also, is there a single place where the rancher-compose.yml file reference is stored?

Yes, we don't support the _ as the name needs to be compatible with our internal DNS.

Yes, there is an issue for v2 support. https://github.com/rancher/rancher/issues/3973

I'm not exactly sure what you mean by ", is there a single place where the rancher-compose.yml file reference is stored?" Are you looking for documentation on what could be included in it? Are you looking for where you could see it in the UI?

Hi Denise,
Thank you for the clarification on the underscore character, that makes sense.
I'm also happy to see there is a ticket to look at using version 2 syntax of docker compose.
What I meant by looking for a single place for the rancher-compose.yml reference is I was wanting to know a little more on what is possible to put into a rancher-compose.yml.
For example, on the docker-compose site the there is a very useful page which outlines all the possible values that can be put into a docker-compose.yml along with some explanation as to what they are used for.
I was wondering if there was a similar page that went through all the values that can be placed in a rancher-compose.yml, on one page, with a bit of explanation as to what each value means and what it does?

@doo-gl Currently, we don't have a section in the docs that lists out all the items that could go in rancher-compose.yml, but that's a great suggestion. I'll change this issue to be that one.

This is greatly missed :)

For reference, I did some digging and found this schema file which might be a little helpful.
https://github.com/rancher/rancher-compose-executor/blob/master/config/schema.go

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.

@deniseschannon Hi, this issue should be reopened as it is not related to v1.6. This is general issue that there is no rancher-compose.yml documentation.


topgrade: 3.7.1 -> 3.9.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/topgrade/versions.
meta.description for topgrade is: '"Upgrade all the things"'.
meta.homepage for topgrade is: '"https://github.com/r-darwish/topgrade"
Release on GitHub
Compare changes on GitHub

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.9.0 with grep in /nix/store/2knc3fk7v1f0cjbd2n4pms45dfik14gh-topgrade-3.9.0

directory tree listing: https://gist.github.com/ca516224faa87396dfc301f6e135ef72

du listing: https://gist.github.com/df120df9c60ec74f0fe16619f7447c34

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
topgrade

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/2knc3fk7v1f0cjbd2n4pms45dfik14gh-topgrade-3.9.0 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A topgrade https://github.com/r-ryantm/nixpkgs/archive/1cb1a8a37fcfbebad5ee2e12b0f4425e87fa1f82.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/2knc3fk7v1f0cjbd2n4pms45dfik14gh-topgrade-3.9.0
ls -la /nix/store/2knc3fk7v1f0cjbd2n4pms45dfik14gh-topgrade-3.9.0/bin

cc @filalex77 @hugoreeves for testing.

Thanks for submitting and merging this. 👍


TP2000-378 Add sid to conditions tab, add conditions link

TP2000-378 Add sid to conditions tab, add conditions link

Why

Tariff managers need additional info on measure conditions tab

What

Adds the sid to the measure conditions tab
Adds a link to the measure conditions tab from the tariff changes list

Codecov Report

Merging #610 (2ba1218) into master (c92a747) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #610   +/-   ##
=======================================
  Coverage   92.98%   92.98%
=======================================
  Files         324      324
  Lines       20373    20378    +5
  Branches     1629     1630    +1
=======================================
+ Hits        18943    18948    +5
  Misses       1131     1131
  Partials      299      299

Impacted Files
Coverage Δ

common/models/trackedmodel.py
94.57% <100.00%> (+0.04%)
:arrow_up:

common/tests/test_models.py
99.06% <100.00%> (+<0.01%)
:arrow_up:

measures/models.py
96.30% <100.00%> (+0.02%)
:arrow_up:

measures/tests/test_views.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Breakage due to systemd 242 parsing of \ in ExecStart= & other options

Issue description
It appears that since #61321, which upgraded systemd, systemd has a backwards-incompatible form of option parsing where trailing \s now escape empty lines up until the next line with text on it. This means that in a unit file like so:
[Unit]
After=kubelet-online.service
Before=kube-node-online.target
ConditionPathExists=/var/lib/kubernetes/secrets/ca.pem
ConditionPathExists=/var/lib/kubernetes/secrets/kube-proxy-client.pem
ConditionPathExists=/var/lib/kubernetes/secrets/kube-proxy-client-key.pem
Description=Kubernetes Proxy Service

[Service]
Environment="KUBECONFIG=/nix/store/xai0d9pw3wfy6fk66slf5x8pqild428z-kube-proxy-kubeconfig"
Environment="LOCALE_ARCHIVE=/nix/store/fnr9mys8l5224sqc7vgisgb66s5429f4-glibc-locales-2.27/lib/locale/locale-archive"
Environment="PATH=/nix/store/bw5a40lav31rqhvjc058izbkidn8xgcc-iptables-1.8.3/bin:/nix/store/kli9f92sdhhmvkgshzjlq2l3q52mh5q5-conntrack-tools-1.4.5/bin:/nix/store/k49nnjscfzgyrhbyfhf3pv2dmqiwi22w-kubectl-1.14.3/bin:/nix/store/i4r7xx7sj1bjgvj9p6dv59f1mb329ivw-coreutils-8.31/bin:/nix/store/dbbvnc30cjg8hdy77b2xhvsbwjkn3b8q-findutils-4.6.0/bin:/nix/store/22zkpn538ji8b3382syvs1kd3c34909v-gnugrep-3.3/bin:/nix/store/87wqi5b61fb90kgz95vbiwhdymfnw8q7-gnused-4.7/bin:/nix/store/izi6x20cncdvx742il1f38rycyk8b4wr-systemd-242/bin:/nix/store/bw5a40lav31rqhvjc058izbkidn8xgcc-iptables-1.8.3/sbin:/nix/store/kli9f92sdhhmvkgshzjlq2l3q52mh5q5-conntrack-tools-1.4.5/sbin:/nix/store/k49nnjscfzgyrhbyfhf3pv2dmqiwi22w-kubectl-1.14.3/sbin:/nix/store/i4r7xx7sj1bjgvj9p6dv59f1mb329ivw-coreutils-8.31/sbin:/nix/store/dbbvnc30cjg8hdy77b2xhvsbwjkn3b8q-findutils-4.6.0/sbin:/nix/store/22zkpn538ji8b3382syvs1kd3c34909v-gnugrep-3.3/sbin:/nix/store/87wqi5b61fb90kgz95vbiwhdymfnw8q7-gnused-4.7/sbin:/nix/store/izi6x20cncdvx742il1f38rycyk8b4wr-systemd-242/sbin"
Environment="TZDIR=/nix/store/9lyxln5y04lj5vg7npzvid0f8ampva1s-tzdata-2019a/share/zoneinfo"

ExecStart=/nix/store/gpx46s19nh708bqbia98iw4jcnlqhkbi-kubernetes-1.14.3/bin/kube-proxy \
          --bind-address=<IP_ADDRESS> \
          --cluster-cidr=<IP_ADDRESS>/16 \
           \
          --kubeconfig=/nix/store/xai0d9pw3wfy6fk66slf5x8pqild428z-kube-proxy-kubeconfig \
           \

ExecStartPre=/nix/store/vbpbg0ry6ingf4s8iyg9y8skil9pnbjv-unit-script-kube-proxy-pre-start
Restart=on-failure
RestartSec=5
Slice=kubernetes.slice
WorkingDirectory=/var/lib/kubernetes

The ExecStartPre=... line is included as part of the ExecStart= line. Since kube-proxy does not expect any arguments, this means that kube-proxy fails to start and enters a crash loop.
I see that #63197 has already been made to address this for Kubernetes (although it's incomplete, as I had this issue with kube-proxy in this case).
I also had the issue with earlyoom, since the optional lines mean that it can end up having a trailing \. It is likely that other services may also be affected (silently so, if they do not crash).
Steps to reproduce
Attempt to use either earlyoom or kube-proxy, in this case.
Technical details

system: "x86_64-linux"
host os: Linux 5.1.11, NixOS, 19.09.git.f8731a7 (Loris)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.2.2
channels(kevin): "nixpkgs-19.03pre167327.11cf7d6e1ff"
channels(root): ""
nixpkgs: /etc/nixos/nixpkgs

I can't switch to latest unstable. Is this related?
$ sudo nixos-rebuild switch
building Nix...
building the system configuration...
updating GRUB 2 menu...
activating the configuration...
setting up /etc...
reloading user units for dukzcry...
setting up tmpfiles
Detected unsafe path transition /var/log → /var/log/journal during canonicalization of /var/log/journal.
Detected unsafe path transition /var/log → /var/log/journal during canonicalization of /var/log/journal.
Detected unsafe path transition /var/log → /var/log/journal during canonicalization of /var/log/journal.
Detected unsafe path transition /var/log → /var/log/journal during canonicalization of /var/log/journal/02178b8a8b5b4420a759d702f1e7d4fa.
Detected unsafe path transition /var/log → /var/log/journal during canonicalization of /var/log/journal/02178b8a8b5b4420a759d702f1e7d4fa.
Detected unsafe path transition /var/log → /var/log/journal during canonicalization of /var/log/journal/02178b8a8b5b4420a759d702f1e7d4fa.
Detected unsafe path transition /var/log → /var/log/journal during canonicalization of /var/log/journal/02178b8a8b5b4420a759d702f1e7d4fa/system.journal.
Detected unsafe path transition /var/log → /var/log/journal during canonicalization of /var/log/journal/02178b8a8b5b4420a759d702f1e7d4fa/system.journal.
warning: error(s) occurred while switching to the new configuration

I can't update to latest unstable. Is this related?
$ sudo nixos-rebuild switch
building Nix...
building the system configuration...
updating GRUB 2 menu...
activating the configuration...
setting up /etc...
reloading user units for gnidorah...
setting up tmpfiles
Detected unsafe path transition /var/log → /var/log/journal during canonicalization of /var/log/journal.
Detected unsafe path transition /var/log → /var/log/journal during canonicalization of /var/log/journal.
Detected unsafe path transition /var/log → /var/log/journal during canonicalization of /var/log/journal.
Detected unsafe path transition /var/log → /var/log/journal during canonicalization of /var/log/journal/02178b8a8b5b4420a759d702f1e7d4fa.
Detected unsafe path transition /var/log → /var/log/journal during canonicalization of /var/log/journal/02178b8a8b5b4420a759d702f1e7d4fa.
Detected unsafe path transition /var/log → /var/log/journal during canonicalization of /var/log/journal/02178b8a8b5b4420a759d702f1e7d4fa.
Detected unsafe path transition /var/log → /var/log/journal during canonicalization of /var/log/journal/02178b8a8b5b4420a759d702f1e7d4fa/system.journal.
Detected unsafe path transition /var/log → /var/log/journal during canonicalization of /var/log/journal/02178b8a8b5b4420a759d702f1e7d4fa/system.journal.
warning: error(s) occurred while switching to the new configuration

systemd updates were included in the latest staging next PR (https://github.com/NixOS/nixpkgs/pull/64236) and has been merged.
This issue should be resolved on master.


How should 'score_thresh' be set to make sure ret is True?

In examples/python/test_ppf_estimator.py
# init ppf config
config = m3d.pose_estimation.PPFEstimatorConfig()
# init training param
config.training_param.rel_sample_dist = 0.04
config.score_thresh = 0.05
config.refine_param.method = m3d.pose_estimation.PPFEstimatorConfig.PointToPlane

What does rel_sample_dist mean?
When I set score_thresh to 0, I get "No match" as usual, I know 0 means Default, which is 0.05.
However, when I set score_thresh to 0.001, in rare cases, I still got "No match".
I assure config.score_thresh = 1e-10 may work when at least one result is needed.
Maybe there is a offical way to do it?

rel_sample_dist is a relative factor to control the sample distane for point clouds. the sample dist is equal to rel_sample_dist * model_diameter .
Did you see the printed infomation in console? score_thresh is use to supress the estimated pose, if there is no pose estimated, whatever you set this value, there is always no result.

Thanks for your answer!


In a cargo build, make note of when check steps finish

I occasionally use a develop <=> cargo build cycle to get my feedback (instead of cargo check). This is because in the case something does check out, I want it to indeed build.
However, while debugging errors, I have some minor anxiety staring at the terminal output wondering if the change I made worked or not, trying to tell from timing whether it has actually succeeded and has moved on to building or not.
I'd appreciate knowing when cargo build is done with the steps that cargo check normally does and has moved on to actually building things.
(Implementation-wise this could require rustc changes but filling a ticket here first given that this is the place I want the UX experience.)

This seems similar enough to #7265 that I'm going to close in favor of that so we have one place to discuss this and have an aligned way of changing the output, rather than trying to design two incompatible UXs separately.  If there is something I missed as to why this needs to be kept open on its own, let us know!


Change URL on "Learn how to save water" homepage button

Since the How you can help link came off the nav bar, we need to link that page somewhere else. Request is to change the Learn how to save water link so it goes to https://drought.ca.gov/how-you-can-help/.
This doesn't appear to be under editorial control in WP, but please let me know if it is.

Hi @thepegisin -- this is under editorial control, but I edited it (it wasn't obvious to me how to edit it, until I asked Jon) -- you have to select the text and use the link icon.
BEFORE

AFTER


Coinbase "advanced_trade_fill" type is not supported

First off, what a great project! I liked the original rp2, but the data wrangling was always the most difficult bit. This data loader is super easy to use and understand. Kudos!
One thing I ran into though is that with Coinbase introducing "advanced trading" earlier this year, some of my transactions for 2022 are labeled as  "type": "advanced_trade_fill", many of which appear to be throwing errors (redacted example below)
ERROR: Unsupported transaction type (skipping):
{
   "id":"[XXXXX]",
   "type":"advanced_trade_fill",
   "status":"completed",
   "amount":{
      "amount":"[XXX]",
      "currency":"BTC"
   },
   "native_amount":{
      "amount":"[XXX]",
      "currency":"USD"
   },
   "description":null,
   "created_at":"2022-03-10T15:34:41Z",
   "updated_at":"2022-03-10T15:34:42Z",
   "resource":"transaction",
   "resource_path":"/v2/accounts/[xxxxx]/transactions/[xxxxx]",
   "instant_exchange":false,
   "advanced_trade_fill":{
      "fill_price":"[XXX]",
      "product_id":"BTC-USD",
      "order_id":"[XXX]",
      "commission":"0"
   },
   "details":{
      "title":"Filled Bitcoin on buy",
      "subtitle":null,
      "header":null,
      "health":"positive"
   },
   "hide_native_amount":false
}

I have 86 errors in my log file, all of which print out these advanced_trade_fill type transactions.
Apologies if this has been brought up before - couldn't find a similar issue on search.
As a side note - perhaps should be its own thread - is there any way to constrain the time parameters of a data pull? Ex. I wanted to avoid this issue by just pulling 2021 transactions, but it looks like there was no way to do that.

Thanks for the kind words and for reporting a missing feature! Currently I don't have time to work on adding advanced trade to the Coinbase plugin, but if anybody wants to do it I'll be happy to review PRs (I tagged this issue as "Help Wanted" to increase visibility).
Regarding the time filter question: DaLI doesn't have one, but RP2 does. The rationale is that RP2 needs all the data to do computate taxes (but it can time-filter output after computation) and DaLI just reads everything because RP2 needs everything. What would be the use case for a DaLI time filter, since you can't (or rather shouldn't) feed RP2 incomplete data? Just wanted to understand if I'm missing something here.

I think this would be a good first issue me as I'd like to start contributing to the RP2 community. Could this issue please be assigned to me? Thank you!

Thanks for volunteering to help! Actually advanced_trade_fill was added here (I forgot to update the issue). However, while both sell and buy are implemented, only sell was tested and advanced trade conversions are not implemented at all. So I suggest the following:

start by testing the buy side, then
work on advanced trading conversions.

If this interests you I'll assign the bug to you. Thanks!

Thanks for volunteering to help! Actually advanced_trade_fill was added here (I forgot to update the issue). However, while both sell and buy are implemented, only sell was tested and advanced trade conversions are not implemented at all. So I suggest the following:

start by testing the buy side, then
work on advanced trading conversions.

If this interests you I'll assign the bug to you. Thanks!

That definitely interests me, please assign the bug to me. After this bug, my goal is to become a regular contributor. RP2/Dali are so unique in their space, and I'd love to put some work in so it continues to improve. 👍

Done! Welcome again and let us know if you have questions.

Quick update on this: I managed to test advanced trade buy and found a small bug, which I fixed here: https://github.com/eprbell/dali-rp2/commit/6828f04de0f62d6a29834004ca5d4f7164594116. So all that is missing is advanced trade conversions (regular conversions were already supported).


An OpenCV sneaky surprise that everyone should knows!

In OpenCV, when reading images, the format is set as BGR color space.
But the neural nets are expecting RGB color format. In most cases, when you are passing BGR to the neural nets, it would work fine.
But this cause the network to not perform at its best.
https://www.learnopencv.com/why-does-opencv-use-bgr-color-format/
So I suggest that you should be careful when using OpenCV's video.read() and cv2.imread() when passing it to the model!
Most of the scripts in this repository are suffering from this bug. So beware of the issue.

Update: it seems someone is aware of this issue: #167


fixed yml page

@lisawong19,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-autogen/overview/azure/DataLakeAnalytics.yml
:warning:Warning

Details

docs-ref-mapping/reference.yml
:white_check_mark:Succeeded

docs-ref-autogen/overview/azure/DataLakeAnalytics.yml

[Warning] Uid(azure.python.sdk.landingPage.services.Data.Lake.Analytics) has already been defined in docs-ref-services/key-vault.md.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Hi Yun,
Seems like the Data Lake Analytics page and Key Vault page got switched so I tried to fix the bug but it's still giving a warning. Can you check it out and see if there's something else that's wrong?
Thanks,
Lisa

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-services/data-lake-analytics.md
:warning:Warning

Details

docs-ref-mapping/reference.yml
:white_check_mark:Succeeded

docs-ref-services/authorization.md
:white_check_mark:Succeeded

docs-ref-services/billing.md
:white_check_mark:Succeeded

docs-ref-services/cdn.md
:white_check_mark:Succeeded

docs-ref-services/cognitive-services.md
:white_check_mark:Succeeded

docs-ref-services/commerce.md
:white_check_mark:Succeeded

docs-ref-services/consumption.md
:white_check_mark:Succeeded

docs-ref-services/container-registry.md
:white_check_mark:Succeeded

docs-ref-services/data-lake-store.md
:white_check_mark:Succeeded

docs-ref-services/devtest-labs.md
:white_check_mark:Succeeded

docs-ref-services/dns.md
:white_check_mark:Succeeded

docs-ref-services/event-hub.md
:white_check_mark:Succeeded

docs-ref-services/graph.md
:white_check_mark:Succeeded

docs-ref-services/iot.md
:white_check_mark:Succeeded

docs-ref-services/logic-apps.md
:white_check_mark:Succeeded

docs-ref-services/media-services.md
:white_check_mark:Succeeded

docs-ref-services/monitoring.md
:white_check_mark:Succeeded

docs-ref-services/network.md
:white_check_mark:Succeeded

docs-ref-services/notification-hubs.md
:white_check_mark:Succeeded

docs-ref-services/other.md
:white_check_mark:Succeeded

docs-ref-services/power-bi.md
:white_check_mark:Succeeded

docs-ref-services/recovery-services-backup.md
:white_check_mark:Succeeded

docs-ref-services/resources.md
:white_check_mark:Succeeded

docs-ref-services/scheduler.md
:white_check_mark:Succeeded

docs-ref-services/search.md
:white_check_mark:Succeeded

docs-ref-services/server-manager.md
:white_check_mark:Succeeded

docs-ref-services/traffic-manager.md
:white_check_mark:Succeeded

docs-ref-services/data-lake-analytics.md

[Warning] Uid(azure.python.sdk.landingPage.services.Data.Lake.Analytics) has already been defined in docs-ref-services/key-vault.md.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


STORM-2180: KafkaSpout ack() should handle re-balance

This is also fixed by https://github.com/apache/storm/pull/1696, please take a look.

Looking at the code #1696 is a much more complete solutions.  @knusbaum could you take a look at it and let us know what you think.

Fine by me.


Update zio-http to 3.0.0-RC4

About this PR
📦 Updates dev.zio:zio-http from 3.0.0-RC2 to 3.0.0-RC4
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio", artifactId = "zio-http" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio", artifactId = "zio-http" }
}]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Superseded by #723.


Renders during OcclusionQuery shows all geometry to fail depth test (OpenGL)

Description
Rendering captured during OcclusionQuery shows in the Texture view with DepthTest enabled all geometry always red (aka failing depth test) although it is clearly in front of other geometry. This happens always. Outside of OcclusionQuery the DepthTest shows green/red correctly but always read while an OcclusionQuery is running
Steps to reproduce

Use a program using OcclusionQuery.
Capture the scene.

Switch to Texture view.
Switch to Depth Test mode.

The geometry is rendered all in red but the parts in front of the depth map should be green. Since the occlusion query does succeed it is RenderDoc showing the wrong result.
Capture can not be shared. But any OcclusionQuery should trigger the bug.
In my test "depth test" is enabled with function LEQUAL and depth writes are disabled.  Depth write is disabled. OpenGL 4.6 context. Bug happens on both ATI and nVidia systems on 2 different Linux systems (GenToo, Kubuntu). Using RenderDoc 1.24 .

Made a screenshot. Both are the same captured geometry render but once with wireframe and once with depth test to show how the geometry (a light volume running along the hallway) is marked red if occlusion query is enabled.

I'm not able to reproduce this. I added an occlusion query around some geometry that should pass and fail the depth test in different places and the results were the same.
If you don't want to share your capture here, you can email it to me privately at<EMAIL_ADDRESS>If that's also not possible could you modify a sample program to reproduce the problem in the way you describe, if you say it works with any use of occlusion queries at all?

I can reproduce it with the smaller example application but the exported capture is nearly 200MB files large.

That's fine, I'm used to captures that are multiple GBs in size

I can 7zip it down to a bit more than 100MB but that's still to large. Doesn't fit through mail.

You can use any file sharing site of your choosing to share files that are more than a couple of MB and don't fit as an email attachment. Google drive, dropbox, onedrive, whatever else.

I've created another capture this time from a different angle. I have the impression I see where RenderDoc might have a bug in displaying the DepthTest information. I can get green color for pixels over cleared background but nowhere else. I have the suspicion RenderDoc uses the position after vertex shader for depth display not after fragment shader (which write gl_FragDepth). I've bookmarked one of the tests with all red: https://drive.google.com/file/d/1d8bHArDOV780MAd6itz2yVloyF2zn3Ke/view?usp=sharing

That would make sense, the overlay is implemented with shader replacement so any shader-modified depth values won't apply. It doesn't have anything to do with occlusion queries then?
I'll need to think if it's possible to solve this without shader patching (which is only really possible on Vulkan).

In that case, yes, it is not related to occlusion query but writing depth in the fragment shader. So it is a "limitation" not a "bug". Might help to note it somewhere in the docs or in the view itself that fragment depth writes are ignored.

This is now implemented.
Sorry it took longer than expected, it required a refactor on all APIs to be able support the feature.
Thank you for the original report and reproduction capture that included an important scenario of shader exported depth with no colour output just depth output.


[Issue] Azure Functions deployment reports success, but function not deployed because Oryx build failed

[x] Make sure you've installed the latest version using instructions in the wiki

Output from azd version
azd version 1.8.0 (commit 8246323c2472148288be4b3cbc3c424bd046b985)
Describe the bug
When deploying a JavaScript Azure Function with AZD, the installation of dependencies and build of the app is done remotely with Oryx.
When deploying the Functions, AZD reports a successful deployment as the zip upload succeeds, but the Function app is not deployed if the Oryx build fails and there's no easy way to detect and debug that when it happens.
To Reproduce

Git clone this repo: https://github.com/Azure-Samples/serverless-chat-langchainjs/
cd packages/api && npm rm typescript => We remove the typescript tool from the Functions API, causing the build to fail
azd deploy => Deployment success, but no Function app is actually deployed

Expected behavior
AZD should not report a successful deployment if Oryx build fails, or at least provide a way to check if an issue occurred and show the actual error logs from the Oryx build.
Environment
Information on your environment:
* Language name and version: TypeScript
* IDE and version : VS Code

@hemarina does your fix for #3260 also covers this one? cc @weikanglim

I'm using Linux Functions and I still ran into this issue again.
This is quite problematic as it reports deployment to be successful, and may fail on post deployment tasks (if there are any) making the users confused and the debugging not easy.
Any updates regarding this?


Allow to create mapping for type nested

There is currently no way to define nested type in mapping.. This pull request adds support for defining nested mapping type by passing options and adds tests around it.

Removed the integration tests (really not desired for a edge feature like this), cleaned, pushed, released.


[Do not merge] (PUP-3090) Update puppetmaster rack config for new paths

With PUP-4074, the master section was removed from the puppet.conf being
laid down by puppet during installation. Those settings are still
required by a master running under passenger, so this commit adds these
settings to the example config.ru in ext/rack. always_cache_features is
also included as a performance improvement for the master.

Depends on https://github.com/puppetlabs/puppet-specifications/pull/34

CLA signed by all contributors.

:+1: /cc @cprice404 @camlow325 @jeffmccune

LGTM, +1 if @jeffmccune is.

The puppet-specifications PR has been merged, so this should be GTG.

@jeffmccune feel free to merge if this looks right to you.


Example code doesn't run (import errors) with Elm 0.19

I'm running Elm 0.19. Having run elm install Skinney/keyboard-events, I then try and run the example code currently in the readme:
import Html exposing (Html, input)
import Keyboard exposing (Key(..))
import Keyboard.Events as Keyboard

autocomplete : Html Msg
autocomplete =
    input
        [ Keyboard.onKeyDown [(Escape, CloseAutoComplete) ]]
        []

This code produces the following error:
-- UNKNOWN IMPORT ------------------------------------------------- src\Main.elm

I cannot find a `Keyboard` module to import.
    import Keyboard
I cannot find that module! Is there a typo in the module name? The "source-directories" field of your elm.json tells me to only look in the src directory, but it is not there. Maybe it is in a package that is not installed yet?

@Skinney Oh okay. So your library just adds Keyboard.Events?

Yes


fix(deps): remove pinned half dependency

This commit removes the pinned version of the half crate, allowing consumers to resolve to a version of half that is compatible with other dependencies in the project.
I'm not sure if pinning to this exact version was intended — happy to close if this causes unintended behavior.
Thanks!

I think there once was an error with newer versions of half only building on rust >= 1.70 which was a problem, butt since the CI runs without problems it should be fine to remove the pin 👍

Sorry, I've been very busy and didn't have time to get around to this...
I think this should work, but the problem is if half gets resolved with 2.3, which raised the MSRV:
llm # cargo update -p half
    Updating crates.io index
    Updating half v2.2.1 -> v2.3.1
llm # cargo +1.65 run
    Updating crates.io index
  Downloaded llm-samplers v0.0.6
  Downloaded 1 crate (26.3 KB) in 0.71s
error: package `half v2.3.1` cannot be built because it requires rustc 1.70 or newer, while the currently active rustc version is 1.65.0
Either upgrade to rustc 1.70 or newer, or use
cargo update -p<EMAIL_ADDRESS>--precise ver
where `ver` is the latest version of `half` supporting rustc 1.65.0

I don't think it should be an issue because the CLI should resolve with the 2.2.1 lock, but I'm not sure about library use. We'll find out, I guess! 😅


nixos/bluetooth: don't install obex tools by default

Motivation for this change
Currently setting hardware.bluetooth.enable = true install the OpenOBEX tools along with bluez.
Since sharing files over bluetooth seems pretty uncommon and the packages are more than 200MB I propose we don't install them by default.

Only problem I see here could be like applications relying on obexftp being in the global environment and removing them would break them. (IIRC some plasma related utilities).

BlueZ still has obexd so this is more accurately removing the OpenOBEX related things.

If we accept this, it needs to be mentioned in release notes.

If we accept this, it needs to be mentioned in release notes.

Done.

worldofpeace: do you know more about the plasma situation? If needed this could be installed behind a guard like plasma5.enable.

@rnhmjoj I'm not sure.
Perhaps some maintainers/contributors of plasma5 will know
@bkchr @ttuegel

I rebased and moved the note to 20.03. Any update?

If needed this could be installed behind a guard like plasma5.enable.

As far as I know this is still required by Plasma. Could you please put it behind a guard?

Thank you @ttuegel. Is this ok?

Since sharing files over bluetooth seems pretty uncommon and the packages are more than 200MB I propose we don't install them by default.

@bjornfor are you aware of other desktop environments that need these tools?

No, I'm not aware of other DEs that need this. I was just worried there were, seeing that commit.

Since sharing files over bluetooth seems pretty uncommon

Personally, sharing files over bluetooth is something I would like to do, if it worked reliably. But I've never good enough experience with bluetooth on linux to rely on it :-/

I checked and it appears it's likely dolphin needed this for that functionality. So it probably won't be that a desktop environment needs, more that a program that uses this functionality will need the packages installed.


Sync eng/common directory with azure-sdk-tools for PR 1535

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/1535  See eng/common workflow

/check-enforcer override


Issues with Duplicates

Issue Type

[X] Bug

Description
When importing findings from CSV I am trying to mark findings as duplicates. According to the documentation Duplicate is a field that can be imported. However when I set the field to True in the CSV it is not pushing that value to the uploaded finding in DefectDojo. Is this working as intended or a bug?
Also, I have been testing and I cannot seem to get DefectDojo to Deduplicate findings. According to the System Settings when the Deduplicate findings box is checked it should automatically deduplicate findings that share a URL and have the same CWE or title. I have set these fields to be the same and DefectDojo has still not marked any of the findings as duplicates.
Install

[X] setup.bash

DefectDojo Version
1.5.2

Hi @JohnWickAppSec for dedupe, ensure that you have Celery running as well and/or you can dedupe in "realtime" by setting the user in the user settings to "blocking" mode. I don't believe the field duplicate in CSV does anything. I'll check the docs and update to reflect the behavior.

We are using Fortify as our scanning tool.

Changing the user setting to blocking mode fixed the issue. Thank you for your response.


Adding parameters such as version,copyright etc. to EXE files

Hello,
I use Nuitka to distribute my projects as standalone.
However, I was unable to add the values seen in the **Properties / details ** tab of the .exe file that was created.
Copyright, version, file description, Product name...
I use various 3rd party software for this.
I think Uses arduous.
It takes me a long time.
What do Nuitka users do for the situation I'm talking about?
I would like to listen.
I reviewed the NUITKA-Utilities repo and nuitka --help output.
I couldnt find it.
I may have missed it.
Finally, thanks to Kay Hayen and the nice people for whom I don't know her name for this wonderful project.

Please have a look at https://github.com/Nuitka/NUITKA-Utilities/blob/master/onefile-maker-windows.py.
You should be able to modify the parameters code from line numbers 250 onwards ...

Hi @JorjMcKie ,
The file here is not what I'm talking about.
I'm talking about the parameters of the .exe file.
Isn't NSIS to compress and setup the file?


[Sidebar] Stop working after moved to https

Semantic version: 2.2.6
My sidebar stop working after installed SSL. The sidebar do not hide / animating in when page is ready, it show up and stacked on the "pusher" div. Anyone have this issue and know how to fix it?
js:
$('.ui.sidebar').sidebar('attach events', '.menu.text .item').sidebar('setting',{debug:true});
html:

...

...

Hi @jdotchai,
Thank you for posting, but although it’s a valid usage question, we’ve limited GitHub Issues to bug reports and feature requests, keeping the board more manageable for maintainers; see the contributing guidelines for more information on what kind of posts should find themselves into the GitHub Issues board.
To get answers or feedback that might allow you to repost or reopen this issue, please use one of our other support resources:

StackOverflow
Gitter chatroom
Official forums


[chore] Fix internal tools and CI

What does this PR do?
Use the correct Go versions in internal tools and CI, also upgrade the tool versions

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 81.30%. Comparing base (47caf7c) to head (3a60b56).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #444   +/-   ##
=======================================
  Coverage   81.30%   81.30%
=======================================
  Files          48       48
  Lines        3681     3681
=======================================
  Hits         2993     2993
  Misses        606      606
  Partials       82       82

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


Implement mail draft

Basic workflow is this:

Client creates a Draft resource via the API

curl -X POST https://server/drafts/ -d '{'project_id': '<project_uuid>', 'from_user_id': '<from_user_id>', 'to_user_id': '<to_user_id'>'}

Service contacts DukeDS for the email addresses and the project name
Service sends an email using a template with the following values

from_email
to_email
project_name
project_url

Service records the action and updates the Draft status to Notified.

I'm having an issue implementing the case where the user has already sent the mail_draft and runs it again.
When sending a POST to drafts/ I get back a 400. (Bad Request) with the following json:
{"non_field_errors":["The fields project_id, from_user_id, to_user_id must make a unique set."]}'

I would like a way to know that the draft already exists.

One possibility is to return a 303. I haven't seen an example of this before but its in this RFC:
https://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html

Comments from PR and above have been moved to new issues


Additional syntax for Delete

Delete has a lot of repetitiveness in the syntax.
Delete.Column("AllowSubscription").FromTable("Users");
Delete.Column("SubscriptionDate").FromTable("Users");

proposed addition:
Delete.Columns("AllowSubscription", "SubscriptionDate").FromTable("Users");

or:
Delete.FromTable("Users").Column("AllowSubscription").Column("SubscriptionDate");

I like the first one for simplicity sake, but the second one is more in line with the current API. What do you think?

My vote goes for the second, although it seems that issue your are countering here could be resolved by writing an extension method

I actually would need to create a class for the FromTable to allow me to store the table name until I could move on to the columns. This class would probably also hold the original IDeleteExpressionRoot, so that I could use the original api to call Delete.Column("").FromTable("").

@khalidabuhakmeh I don't think an extension method would do the trick, but is it really a use case, how many times you need to delete multiple columns from a table

@tommarien we are oftern delete columns.

In theory, I have to delete multiple columns every time i add to an existing table (down migration).
Kind Regards,
Khalid Abuhakmeh
On Jan 27, 2012, at 5:04 AM, Tom Marien<EMAIL_ADDRESS>wrote:

@khalidabuhakmeh I don't think an extension method would do the trick, but is it really a use case, how many times you need to delete multiple columns from a table

Reply to this email directly or view it on GitHub:
https://github.com/schambers/fluentmigrator/issues/233#issuecomment-3684471

@khalidabuhakmeh adding collumn is perfectly handled by AutoReversingMigration

As long as there is a usecase for it, i don't mind the api improvement :)

@tommarien
But we are deleting columns in up stage too ;) But we have no inconveniences with deleting columns one by one.

@hazzik damn I have been using this library for a long time, must have missed AutoReversingMigration sneak in. The more you know I guess.
This syntax still might be helpful, but it's not completely necessary.


client.whisper() now rejects

client.whisper() now rejects, for now only when one of the followin notices arive within the _getPromiseDelay() timespan:

no_permission
whisper_invalid_login
whisper_invalid_self
whisper_limit_per_min
whisper_limit_per_sec
whisper_restricted
whisper_restricted_recipient

This closes #388
This is a breaking change

Hey @AlcaDesign, I came up with this solution but I don't have access to a known bot right now, so I can't test it properly.
I will add a test suite tomorrow, can you let me know if this solution makes sense?
I saw the __promiseWhisper event, which I used here, being emitted, but never listened on, does it cause any problems anywhere?

@AlcaDesign poke


Kubeflow Pipeline Token Auth does not read local environment variable KF_PIPELINES_SA_TOKEN_PATH

Describe the issue
Authentication fails when using token authentication for Kubeflow pipelines. Our deployment mounts the token in a path that is different to the default path of var/run/secrets/kubeflow/pipelines/token. Therefore we set the env variable using:
KF_PIPELINES_SA_TOKEN_PATH=/var/run/secrets/kubernetes.io/serviceaccount/token
The Run does not work.
To Reproduce
Steps to reproduce the behavior:

Go to Elyra and setup a kubeflow pipeline runtime that uses a token
Make sure the token is NOT in the default path of  /var/run/secrets/kubeflow/pipelines/token
In a Terminal, set the variable KF_PIPELINES_SA_TOKEN_PATH to the place where the token is mounted
Open pipeline example such as 'hello-generic-world.pipeline' and Run selecting the runtime you created in 1
See the error

Kubeflow authentication failed: Kubernetes service account token could not be read from /var/run/secrets/kubeflow/pipelines/token: [Errno 2] No such file or directory: '/var/run/secrets/kubeflow/pipelines/token'.
Deployment information
Describe what you've deployed and how:

Elyra version: 3.3.0
Operating system: Azure AKS, Kubeflow 1.4, Notebook server

Pipeline runtime environment
If the issue is related to pipeline execution, identify the environment where the pipeline is executed

Kubeflow Pipelines 1.4, Multi user with KUBERNETES_SERVICE_ACCOUNT_TOKEN auth
Argo workflows

Runtime configuration settings
If the issue is related to pipeline execution, document the runtime configuration settings from the Elyra UI, omitting confidential information.

This is odd. I just completed your listed steps in a local environment the environment variable value is picked up and the result is as expected:

install 3.3.0
open terminal window
export KF_PIPELINES_SA_TOKEN_PATH=/no/such/file
jupyter lab
create KFP runtime configuration, selecting KUBERNETES_SERVICE_ACCOUNT_TOKEN as auth type (specifying no credentials)
create KFP pipeline
pipeline run fails with:

You've indicated that you are using Azure AKS. If you've completed the steps there, can you please confirm that the environment variable was configured properly?

If you were running the test in a container in Kubernetes, can you list the environment variables in the pod where JupyterLab/Elyra is running, e.g. by issuing
kubectl exec <CONTAINER_ID> --namespace <YOUR_NAMESPACE> -- env

If the output does not include KF_PIPELINES_SA_TOKEN_PATH then this is likely caused by something in your setup.

I performed the steps from the Notebook Server within Kubeflow. You ran the steps in your machine. Let me check the env in the pod following your suggestion. Thanks

Hi @alexisc67, based on your responses it appears you're setting the environment variable from a terminal session launched from Lab/Elyra. This is not sufficient to set the variable into the server process. Instead, set/export the variable prior to starting the Lab/Elyra server. If your server resides in a container then the env should be set via the container launch facility, whatever that might be, so that it's present when Elyra goes to use it.

The issue was resolved adding a configuration poddefault to the namespace running the notebooks for:
KF_PIPELINES_SA_TOKEN_PATH=/var/run/secrets/kubeflow/pipelines/token
Thanks!

Hi @alexisc67,
How did you added configuration poddefault? I'm also running into same issues and on executing kubectl exec <CONTAINER_ID> --namespace <YOUR_NAMESPACE> -- env there is no KF_PIPELINES_SA_TOKEN_PATH variable


libcxx must be defined for poco to build

conan install .. -s compiler=clang -s compiler.version=3.8 --build
returns:
ERROR: Poco/1.7.2@lasote/stable: 'settings.compiler.libcxx' value not defined
Solved by:
conan install .. -s compiler=clang -s compiler.version=3.8 -s compiler.libcxx=libstdc++ --build

Hi!
This doesn't really seem an issue of the repo, more an issue of the Readme, right? I will fix it. Next time, you can directly modify it and do a PR!
Many thanks for reporting!

Will do. I'm new to Conan.io so wasn't sure if having to list the libcxx was a work around, or expected behavior.

Yes, expected behavior, it was a mistake in the readme :)

I think this issue could be closed