GlobalPhaseGate causing circuit.depth() error

Environment

Qiskit version: 1.0.2
Python version: 3.11.9
Operating system: Windows

What is happening?
Greetings,
Hope all are well. I have noticed that when you apply the GlobalPhaseGate to a circuit, and then try to retrieve the depth of the circuit, it causes an error due to the GlobalPhaseGate instruction not using any qubit indices. I think this should be fixed inside the .depth() method as perhaps a special case.
How can we reproduce the issue?
from qiskit import QuantumCircuit, transpile
from qiskit.circuit.library import GlobalPhaseGate

qc_qiskit = QuantumCircuit(2, 2)

qc_qiskit.cx(0, 1)

transpiled_qc = transpile(qc_qiskit,
                          basis_gates=['u3', 'cx'],
                          optimization_level=3,
                          seed_transpiler=0)

transpiled_qc.append(GlobalPhaseGate(2.5928519981876477), [], [])

transpiled_qc.global_phase

transpiled_qc.depth()

This will output the following error:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[15], [line 19](vscode-notebook-cell:?execution_count=15&line=19)
     [15](vscode-notebook-cell:?execution_count=15&line=15) transpiled_qc.append(GlobalPhaseGate(2.5928519981876477), [], [])
     [17](vscode-notebook-cell:?execution_count=15&line=17) transpiled_qc.global_phase
---> [19](vscode-notebook-cell:?execution_count=15&line=19) transpiled_qc.depth()

File c:\Users\A.C.EA\OneDrive\Documents\GitHub\QICKIT\.venv\Lib\site-packages\qiskit\circuit\quantumcircuit.py:1909, in QuantumCircuit.depth(self, filter_function)
   [1906](file:///C:/Users/A.C.EA/OneDrive/Documents/GitHub/QICKIT/.venv/Lib/site-packages/qiskit/circuit/quantumcircuit.py:1906)             reg_ints.append(idx)
   [1907](file:///C:/Users/A.C.EA/OneDrive/Documents/GitHub/QICKIT/.venv/Lib/site-packages/qiskit/circuit/quantumcircuit.py:1907)             levels.append(op_stack[idx] + 1)
-> [1909](file:///C:/Users/A.C.EA/OneDrive/Documents/GitHub/QICKIT/.venv/Lib/site-packages/qiskit/circuit/quantumcircuit.py:1909) max_level = max(levels)
   [1910](file:///C:/Users/A.C.EA/OneDrive/Documents/GitHub/QICKIT/.venv/Lib/site-packages/qiskit/circuit/quantumcircuit.py:1910) for ind in reg_ints:
   [1911](file:///C:/Users/A.C.EA/OneDrive/Documents/GitHub/QICKIT/.venv/Lib/site-packages/qiskit/circuit/quantumcircuit.py:1911)     op_stack[ind] = max_level

ValueError: max() arg is an empty sequence

What should happen?
Inside the .depth() method, the code should simply pass when the instruction is a GlobalPhaseGate. Since it's not, because GlobalPhaseGate has no qubit index as parameter, it causes the levels variable to be [], thus raising the error.
Any suggestions?
I tried this and it worked:
def depth(
        self,
        filter_function: Callable[..., int] = lambda x: not getattr(
            x.operation, "_directive", False
        ),
    ) -> int:
        """Return circuit depth (i.e., length of critical path).

        Args:
            filter_function (callable): A function to filter instructions.
                Should take as input a tuple of (Instruction, list(Qubit), list(Clbit)).
                Instructions for which the function returns False are ignored in the
                computation of the circuit depth.
                By default filters out "directives", such as barrier or snapshot.

        Returns:
            int: Depth of circuit.

        Notes:
            The circuit depth and the DAG depth need not be the
            same.
        """
        # Assign each bit in the circuit a unique integer
        # to index into op_stack.
        bit_indices: dict[Qubit | Clbit, int] = {
            bit: idx for idx, bit in enumerate(self.qubits + self.clbits)
        }

        # If no bits, return 0
        if not bit_indices:
            return 0

        # A list that holds the height of each qubit
        # and classical bit.
        op_stack = [0] * len(bit_indices)

        # Here we are playing a modified version of
        # Tetris where we stack gates, but multi-qubit
        # gates, or measurements have a block for each
        # qubit or cbit that are connected by a virtual
        # line so that they all stacked at the same depth.
        # Conditional gates act on all cbits in the register
        # they are conditioned on.
        # The max stack height is the circuit depth.
        for instruction in self._data:
            levels = []
            reg_ints = []
            for ind, reg in enumerate(instruction.qubits + instruction.clbits):
                # Add to the stacks of the qubits and
                # cbits used in the gate.
                reg_ints.append(bit_indices[reg])
                if filter_function(instruction):
                    levels.append(op_stack[reg_ints[ind]] + 1)
                else:
                    levels.append(op_stack[reg_ints[ind]])
            # Assuming here that there is no conditional
            # snapshots or barriers ever.
            if getattr(instruction.operation, "condition", None):
                # Controls operate over all bits of a classical register
                # or over a single bit
                if isinstance(instruction.operation.condition[0], Clbit):
                    condition_bits = [instruction.operation.condition[0]]
                else:
                    condition_bits = instruction.operation.condition[0]
                for cbit in condition_bits:
                    idx = bit_indices[cbit]
                    if idx not in reg_ints:
                        reg_ints.append(idx)
                        levels.append(op_stack[idx] + 1)

            if getattr(instruction.operation, 'name') == 'global_phase':
                levels = [0]

            max_level = max(levels)
            for ind in reg_ints:
                op_stack[ind] = max_level

        return max(op_stack)

Thanks for the report - this looks like a clear mistake in depth.  I can post a PR in a sec that does a little bit of tidying up of the function and makes it handle a few other things it's currently missing.

Greetings dear Jake,
Hope you are well. May I ask when the commit will be merged so that I can pip install it in my environment?

Greetings dear @jakelishman,
Hope you are well. May I ask when the commit will be merged so that I can pip install it in my environment?


Data Relationships

It would be useful to have a data model/record capable of linking to another model as a single reference or a collection of references (i.e. an array). Ideally, these would be pointers to another store and the specific record within the store.

Completed in 1.0.57.


Blank pages.

Missing content?
There is no content on: https://developers.flow.com/mobile/get-started
The other pages in this section are basically empty as well.

Thanks for the heads up Mack :)

https://github.com/onflow/mobile-docs/pull/8

closing


Project Browser is filled with TinyIDE stacks

Is there a way to hide the TinyIDE stacks in the Project Browser? Your files swamp the ones I'm trying to work on.
Thanks
Kelly Janz
kdjanz - at - porta dot ca

Hi Kelly, sorry about the late reply.
Yes the way LiveCode treats behavior stacks for plugins is not ideal., I had a few years back asked for an enhancement which would allow the user the option to hide stacks and scripts from the project browser if the stacks were in the plugins folder, I must revisit this.
I'm at this very moment building a no behavior version of TinyIDE. When I've completed this I will send you a copy to try out, this should be later today or tomorrow.
All the best
Andy

Thanks. I’ll look for that.
Kelly
Sent from the iPad

On Apr 20, 2020, at 5:50 AM, Andy Piddock<EMAIL_ADDRESS>wrote:

Hi Kelly, sorry about the late reply.
Yes the way LiveCode treats behavior stacks for plugins is not ideal., I had a few years back asked for an enhancement which would allow the user the option to hide stacks and scripts from the project browser if the stacks were in the plugins folder, I must revisit this.
I'm at this very moment building a no behavior version of TinyIDE. When I've completed this I will send you a copy to try out, this should be later today or tomorrow.
All the best
Andy
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi Kelly, in the folder 1 stack version, you will find a single stack version of TinyIDE.
I haven't had time to fully test all of the functions yet, so please let me know if you find any problems.
All the best
Andy


LC0004 false positives (workspaces)

Scenario:

App A has a table A without "LookupPageID" and "DrilldownPageID"
App A has a listpart page A for table A

LC0004 is not raised when opening app A. (OK)

App B has a dependency on App A
App B has a list page B for table A
A workspace exists with App A and B

LC0004 is raised when opening the workspace (NOK)
warning LC0004: Property "LookupPageID" and "DrilldownPageID" must be filled in table "A" because it is used in page "B" (list)
The warning should only be raised for tables / pages part of the current app. Pages from apps higher in the dependency tree should be excluded from the rule.

I've modified this rule and this should be resolved in the (pre)release version of v0.30.10.

The version v0.30.10 of the LinterCop is now released. If would you still encounter this issue feel free to reopen this one.


DartFixKind.ORGANIZE_IMPORTS disagrees with LintNames.directives_ordering

tl;dr
LintNames.directives_ordering (which I agree with) prefers code like this
import 'travis.dart';
import 'travis/lib.dart';

DartFixKind.ORGANIZE_IMPORTS orders code like this:
import 'travis/lib.dart';
import 'travis.dart';

I hacked up the test which I would like to pass. 😄
diff --git a/pkg/analysis_server/test/src/services/correction/fix/organize_imports_test.dart b/pkg/analysis_server/test/src/services/correction/fix/organize_imports_test.dart
index 868dbabae2f..65c90a10baf 100644
--- a/pkg/analysis_server/test/src/services/correction/fix/organize_imports_test.dart
+++ b/pkg/analysis_server/test/src/services/correction/fix/organize_imports_test.dart
@@ -4,6 +4,7 @@

 import 'package:analysis_server/src/services/correction/fix.dart';
 import 'package:analysis_server/src/services/linter/lint_names.dart';
+import 'package:analyzer/src/error/codes.dart';
 import 'package:analyzer_plugin/utilities/fixes/fixes.dart';
 import 'package:test_reflective_loader/test_reflective_loader.dart';

@@ -40,4 +41,28 @@ import 'dart:io';
 void main(Stream<String> args) { }
 ''');
   }
+
+  Future<void> test_organizeImports2() async {
+    await resolveTestUnit('''
+//ignore_for_file: uri_does_not_exist
+import 'travis/lib.dart';
+
+import 'travis.dart';
+
+void main(Stream<String> args) { }
+''');
+    await assertHasFix(
+      '''
+//ignore_for_file: uri_does_not_exist
+import 'travis.dart';
+import 'travis/lib.dart';
+
+void main(Stream<String> args) { }
+''',
+      errorFilter: (input) {
+        print(input);
+        return input.errorCode != CompileTimeErrorCode.URI_DOES_NOT_EXIST;
+      },
+    );
+  }
 }

CC @jwren  @scheglov

CC @pq re LINT

Just some history: The quick fix uses the ImportOrganizer, which also implements the "Sort Members in Dart File" feature in IDEs. The lint was written long after ImportOrganizer was and disagrees with it. IIRC, the Flutter team has yet another way of sorting imports that doesn't match either of the pre-existing orders.
While I strongly agree that it would be nice for everything to be consistent, there's an open question as to which pieces should be changed and to what.


React is needed during testing

Describe the bug
I'm trying to reproduce an issue in a different library (for reference issue: https://github.com/blitz-js/superjson/issues/196, PR: https://github.com/blitz-js/superjson/pull/197). For that I'm using QueryClient and dehydrate, but I'm getting the following error:
Cannot find module 'react' from<EMAIL_ADDRESS>
    Require stack:
<EMAIL_ADDRESS><EMAIL_ADDRESS>      src/index.test.ts

This is the whole test:
  const queryClient = new QueryClient();
  const input = { props: { dehydratedState: dehydrate(queryClient) } };

  const stringified = SuperJSON.stringify(input);
  const parsed: any = SuperJSON.parse(stringified);

  expect(parsed).toEqual(input);

Your minimal, reproducible example
https://codesandbox.io/s/tanstack-query-react-dependency-bnkx3l?file=/src/index.test.js
Steps to reproduce

Go to the reproduction codesandbox
Go to the tests tab
The test doesn't pass

Expected behavior
I think the test should pass, since it's not using any hook, just the QueryClient and dehydrate.
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform

OS: MacOS
Browser: Brave
Version: 1.39.11

react-query version
4.2.1
TypeScript version
No response
Additional context
No response

QueryClient and dehydrate are both part of the query-core. So if you only depend on that, the test works:
https://codesandbox.io/s/tanstack-query-react-dependency-forked-x2e2v9?file=/src/index.test.js
- import { QueryClient, dehydrate } from "@tanstack/react-query";
+ import { QueryClient, dehydrate } from "@tanstack/query-core";

basically: if you depend on react-query, you need react. If you don't need react, don't depend on react-query :)

Nice! Good to know :)
This is a weird place to say this, but I'm not sure there's a good place. Thanks for being so helpful all over the place: issues (this one), discussions (https://github.com/TanStack/query/discussions/4024), PR (https://github.com/TanStack/query/pull/4010), and even a StackOverflow Question I asked about NextJS SSR (https://stackoverflow.com/a/73342248/5452969). And those are just the places where you've helped me directly. I've definitely been helped by answers you left to other people. Thanks! 🙏


Throw dll missing exception when failed to load oci.dll at runtime

When OCI_IMPORT_RUNTIME defined and oci.dll is not found following message thrown:
OCILIB has not been initialized
but it should throw this message:
Cannot load OCI shared library (oci.dll)

Thanks


Sandbox MSBuild

As mentioned by Jason here, we need to properly sandbox our use of MSBuild when loading solutions. I'd rather not lose everything on the machine (or worse...).
Unfortunately, I'm not sure what the appropriate steps to take here are. I'm reading up on running partially trusted code in a sandbox, but I'm not sure if this is good enough: http://msdn.microsoft.com/en-us/library/bb763046(v=vs.110).aspx

Sounds like that might not work either: https://twitter.com/ThisIsJoshVarty/status/551216170175447041

What would we run in the sandbox? MSBuild, because the concern is that the build script may contain malicious code?
Does it mean that the assembly with restricted permissions would be  Microsoft.CodeAnalysis.Workspaces.Desktop.dll?
The article says that one of the permissions that can be granted is being able to use a virtual filesystem which is restricted to that assembly only. Either I'm wrong or we will need to devise a mechanism to return SB_Files from that filesystem to the actual filesystem.

Whoops, missed Jason's tweet. Low privilege user account sounds like it could do the trick and be relatively straightforward. Let's hear his response about elevation.

Working on this in https://github.com/AmadeusW/SourceBrowser/tree/SecureGenerator
The code is done and should work. This will be verified after we successfully update the server to support newest Roslyn DLLs. I will create a pull request with details then.


Can RTMP stream be password protected

Hi iizukanao,
I wanted to know if the RTMP stream be password protected, and also is it possible to add stream.key instead of /live/picam in RTMP

As for "password protected", do you want a Basic HTTP authentication?
Also, regarding "stream.key", does it mean you would like to use a URL of your choice instead of the fixed one?

Hi, iizukanao, I use ngrok tcp forwarding for the RTMP stream to watch it
on VLC, i have seen some camera RTSP streams which are username:password
protected while watching on VLC, .
If some thing similar can be implemented on the RTMP stream from Picam ,
because RTMP stream from Picam is consistent.
As we use Encrypted Key on youtube, if some thing similar can be used
while watching on VLC, from security point of view.
On Mon, Aug 9, 2021 at 8:50 PM Nao Iizuka @.***> wrote:

As for "password protected", do you want a Basic HTTP authentication?
Also, regarding "stream.key", does it mean you would like to use a URL of
your choice instead of the fixed one?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/iizukanao/picam-streamer/issues/47#issuecomment-895314017,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJFDX6UQJN2MPH5A7EP7T4TT37W25ANCNFSM5BZNCCMQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Unfortunately, that is not supported in picam-streamer at this time. As an alternative, you can secure your pi by setting up a firewall using ufw so that only some source IP addresses are allowed to access the stream.

Thankyou for your prompt Reply🙏🙏🙏🙏


Remove version: FXHOME.HitFilmExpress version 18.0.12407.12817

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://1-965-10440-2.b.cdn13.com/hitfilm-express/HitFilmExpress_2021.3_(18.0.12407.12817).msi
Response Error: The remote server returned an error: (421) Misdirected Request.
(Automated message - build 912)

Hi @Exorcism0666,
This manifest has the highest version number for this package. Is it available from another location? (This might be in error if the version is switching from semantic to string, or string to semantic.)
(Automated response - build 887.)

URL: https://1-965-10440-2.b.cdn13.com/hitfilm-express/HitFilmExpress_2021.3_(18.0.12407.12817).msi
Response status code does not indicate success: 421 (Misdirected Request).
(Automated message - build 887)

For moderators or/and anyone else

[!Caution]
As a result of several personal problems I've had over the last few months + my final exams coming up very soon, I'm currently unable to look at my GitHub notifications every day and maintain my executions correctly.
If you detect a certain problem on a certain pul request:

Either you can close the pull request directly without mentioning me.
Or you can send me a message on X (Twitter): @Exorcism0666

Thank you for your understanding, and I'm sorry if some pull requests can't be approved due to the situation I'm in.

This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping Exorcism0666.

[!warning]
With the new version of Komac, numerous Pull Requests may contain bugs or unwanted elements. If such issues arise, please tag russellbanks so that these bugs can be addressed in future updates.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

URL: https://1-965-10440-2.b.cdn13.com/hitfilm-express/HitFilmExpress_2021.3_(18.0.12407.12817).msi
Response status code does not indicate success: 421 (Misdirected Request).
(Automated message - build 887)

URL: https://1-965-10440-2.b.cdn13.com/hitfilm-express/HitFilmExpress_2021.3_(18.0.12407.12817).msi
Response status code does not indicate success: 421 (Misdirected Request).
(Automated message - build 887)

URL: https://1-965-10440-2.b.cdn13.com/hitfilm-express/HitFilmExpress_2021.3_(18.0.12407.12817).msi
Response status code does not indicate success: 421 (Misdirected Request).
(Automated message - build 887)


Write a user feedback summary about EDS value

I interviewed Christian Leon Christensen about how he uses and value EDS. This will be used as a qualitative valuation of EDS, but could also be used as a case article to be published to our community.

I added the summary of the interview here under User meetings/EDS value


As a user, I want a better homescreen with a different layout. Move 'Grocery list', 'Pantry', 'Meal plans' to the bottom, with icon for each thing.

We decided to changed 'Meal plans' to 'Recipes'


[stable/kube-state-metrics] Add imagePullSecrets in serviceAccount

Signed-off-by: Tanguy Buchier<EMAIL_ADDRESS>What this PR does / why we need it:
Add the ability to specify the imagePullSecrets field in the serviceAccount
Allow you to specify the serviceAccount name even if rbac is enabled
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/assign @lachie83


Fix sphinx issues for azure-monitor-opentelemetry and exporter

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


Global State Handling

TimeLine for handling state :

[ ] use zustand for statehandling

We have to create a separate file in a folder named stores.
I have created a store for subscription.


More fine-grained DI for management node types. Don't allocate processing resources on Router

Remove DruidProcessingModule, QueryableModule and QueryRunnerFactoryModule from DI for coordinator, overlord, middle-manager.
Add RouterDruidProcessing not to allocate processing resources on Router.

The same goal as in #4410.

@gianm could you please review this pr?

Looks cool! Changing around the module injection order can change the lifecycle ordering through. As such this probably needs good testing internally to make sure the different instance types can all run with this change.

The integration tests, at least, check it for the major types.

Moved into 0.10.1 as discussed in the dev sync


rhods_test_jupyterlab: rework artifact extraction

/cc @kpouget

/test jh-at-scale

/test ods-jh-at-scale

On Fri, Jun 3, 2022 at 9:47 PM openshift-ci[bot] @.***>
wrote:

@kpouget https://github.com/kpouget: The specified target(s) for /test
were not found.
The following commands are available to trigger required jobs:

/test 4.10-e2e-mlperf-ssd-pr-test
/test 4.10-gpu-operator-e2e
/test 4.10-images
/test 4.7-gpu-operator-e2e
/test 4.7-images
/test 4.7-test-commit
/test 4.8-gpu-operator-e2e
/test 4.8-images
/test 4.8-test-commit
/test 4.9-gpu-operator-e2e
/test 4.9-images
/test 4.9-test-commit
/test gpu-operator-e2e
/test images
/test nfd-images
/test nfd-operator-e2e-master
/test nto-e2e-master
/test nto-images
/test ods-images
/test ods-jh-at-scale
/test ods-jh-at-small-scale
/test test-pr

Use /test all to run the following jobs that were automatically triggered:

pull-ci-openshift-psap-ci-artifacts-master-images

In response to this
https://github.com/openshift-psap/ci-artifacts/pull/370#issuecomment-1146306509
:
/test jh-at-scale
Instructions for interacting with me using PR comments are available here
https://git.k8s.io/community/contributors/guide/pull-requests.md. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository.
—
Reply to this email directly, view it on GitHub
https://github.com/openshift-psap/ci-artifacts/pull/370#issuecomment-1146306547,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZVQIUN5X77XLTYFF2HMHLVNJOL5ANCNFSM5XZ7QGRA
.
You are receiving this because you were mentioned.Message ID:
@.***>

/test ods-jh-at-scale

/test ods-jh-at-scale

/test ods-jh-at-scale
On Sat, Jun 4, 2022 at 11:21 AM openshift-ci[bot] @.***>
wrote:

@kpouget https://github.com/kpouget: The following test failed, say
/retest to rerun all failed tests or /retest-required to rerun all
mandatory failed tests:
Test name Commit Details Required Rerun command
ci/prow/ods-jh-at-scale a2083e4
https://github.com/openshift-psap/ci-artifacts/commit/a2083e417ec54bdd54c54048ea5f193fd877dd2a
link
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift-psap_ci-artifacts/370/pull-ci-openshift-psap-ci-artifacts-master-ods-jh-at-scale/1532987156996296704
true /test ods-jh-at-scale
Full PR test history
https://prow.ci.openshift.org/pr-history?org=openshift-psap&repo=ci-artifacts&pr=370.
Your PR dashboard
https://prow.ci.openshift.org/pr?query=is:pr+state:open+author:kpouget.
Instructions for interacting with me using PR comments are available here
https://git.k8s.io/community/contributors/guide/pull-requests.md. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository. I understand the commands that are listed here
https://go.k8s.io/bot-commands.
—
Reply to this email directly, view it on GitHub
https://github.com/openshift-psap/ci-artifacts/pull/370#issuecomment-1146575659,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZVQIX6RXFBIJFY3DHCPPLVNMN2XANCNFSM5XZ7QGRA
.
You are receiving this because you were mentioned.Message ID:
@.***>

/test ods-jh-at-scale

/test ods-jh-at-scale
On Sat, Jun 4, 2022 at 4:07 PM openshift-ci[bot] @.***>
wrote:

@kpouget https://github.com/kpouget: The following test failed, say
/retest to rerun all failed tests or /retest-required to rerun all
mandatory failed tests:
Test name Commit Details Required Rerun command
ci/prow/ods-jh-at-scale 73c92ef
https://github.com/openshift-psap/ci-artifacts/commit/73c92efddb73f941e61413f8bb2598d3dea6ed21
link
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift-psap_ci-artifacts/370/pull-ci-openshift-psap-ci-artifacts-master-ods-jh-at-scale/1533060960732844032
true /test ods-jh-at-scale
Full PR test history
https://prow.ci.openshift.org/pr-history?org=openshift-psap&repo=ci-artifacts&pr=370.
Your PR dashboard
https://prow.ci.openshift.org/pr?query=is:pr+state:open+author:kpouget.
Instructions for interacting with me using PR comments are available here
https://git.k8s.io/community/contributors/guide/pull-requests.md. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository. I understand the commands that are listed here
https://go.k8s.io/bot-commands.
—
Reply to this email directly, view it on GitHub
https://github.com/openshift-psap/ci-artifacts/pull/370#issuecomment-1146619215,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZVQIWGZBQYQ7TMFY6QDLTVNNPI3ANCNFSM5XZ7QGRA
.
You are receiving this because you were mentioned.Message ID:
@.***>

/test ods-jh-at-scale

/test ods-jh-at-scale

/test ods-jh-at-scale
On Sun, Jun 5, 2022 at 11:05 AM openshift-ci[bot] @.***>
wrote:

@kpouget https://github.com/kpouget: The following test failed, say
/retest to rerun all failed tests or /retest-required to rerun all
mandatory failed tests:
Test name Commit Details Required Rerun command
ci/prow/ods-jh-at-scale 1e6afff
https://github.com/openshift-psap/ci-artifacts/commit/1e6afff78ef7f52162bb99d2d82c90e560798937
link
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift-psap_ci-artifacts/370/pull-ci-openshift-psap-ci-artifacts-master-ods-jh-at-scale/1533348987355009024
true /test ods-jh-at-scale
Full PR test history
https://prow.ci.openshift.org/pr-history?org=openshift-psap&repo=ci-artifacts&pr=370.
Your PR dashboard
https://prow.ci.openshift.org/pr?query=is:pr+state:open+author:kpouget.
Instructions for interacting with me using PR comments are available here
https://git.k8s.io/community/contributors/guide/pull-requests.md. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository. I understand the commands that are listed here
https://go.k8s.io/bot-commands.
—
Reply to this email directly, view it on GitHub
https://github.com/openshift-psap/ci-artifacts/pull/370#issuecomment-1146769045,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZVQISWNTFRILHHJBIEWULVNRUURANCNFSM5XZ7QGRA
.
You are receiving this because you were mentioned.Message ID:
@.***>

/test ods-jh-at-scale

/test ods-jh-at-scale
On Sun, Jun 5, 2022 at 3:54 PM openshift-ci[bot] @.***>
wrote:

@kpouget https://github.com/kpouget: The following test failed, say
/retest to rerun all failed tests or /retest-required to rerun all
mandatory failed tests:
Test name Commit Details Required Rerun command
ci/prow/ods-jh-at-scale 10b648b
https://github.com/openshift-psap/ci-artifacts/commit/10b648b7eccbba9203c4c0ab4ff312ef4c2ef0a4
link
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift-psap_ci-artifacts/370/pull-ci-openshift-psap-ci-artifacts-master-ods-jh-at-scale/1533416606137323520
true /test ods-jh-at-scale
Full PR test history
https://prow.ci.openshift.org/pr-history?org=openshift-psap&repo=ci-artifacts&pr=370.
Your PR dashboard
https://prow.ci.openshift.org/pr?query=is:pr+state:open+author:kpouget.
Instructions for interacting with me using PR comments are available here
https://git.k8s.io/community/contributors/guide/pull-requests.md. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository. I understand the commands that are listed here
https://go.k8s.io/bot-commands.
—
Reply to this email directly, view it on GitHub
https://github.com/openshift-psap/ci-artifacts/pull/370#issuecomment-1146809283,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZVQITOKD3VFGZ3TQL7OO3VNSWSZANCNFSM5XZ7QGRA
.
You are receiving this because you were mentioned.Message ID:
@.***>

/test ods-jh-at-scale

base version working again: https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift-psap_ci-artifacts/370/pull-ci-openshift-psap-ci-artifacts-master-ods-jh-at-scale/1533522124273094656

/test ods-jh-at-scale

test looks good, merging


Setup

Vet clinic database: create animals table

Create an animals table. 💯
Insert given data into the table. 💯
Write queries for animals table. 💯

Feel free to review 🧑🏻💻 the PR and let me know if there are any changes to do! 🤓

Hi @Rustamxon7 👋,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


add for syntax to be null

Fixes https://github.com/SamyPesse/draft-js-prism/issues/4 .

Thanks 🍻


Any way to increase the truncated query length in Art workbook?

Hi, I'm working with a customized Art workbook, and there is "Queries and Protocol Creation" dashboard which gives "details" info in tooltips for each query.
The "details" column contains info of the executed SQL query, but it is truncated in a short length with a '~<<query-trunc>>~' string.
Is there any way in Logshark options to increase the truncated length, in order to see the whole executed SQL query?
Thanks for help!

I believe that this is in the zip log output , as I also have the same issue.


1.11-SNAPSHOT:  com.evernote.android.job.JobManagerCreateException

Seeing these reported for the first time on the Play store with my app using 1.11-SNAPSHOT.
Android 7.0
com.evernote.android.job.JobManagerCreateException
com.evernote.android.job.JobManager.
java.lang.RuntimeException:
at android.app.ActivityThread.handleBindApplication(ActivityThread.java:5884)
at android.app.ActivityThread.-wrap3(ActivityThread.java:0)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1718)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:154)
at android.app.ActivityThread.main(ActivityThread.java:6688)
at java.lang.reflect.Method.invoke(Method.java:0)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1468)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1358)
Caused by: com.evernote.android.job.JobManagerCreateException:
at com.evernote.android.job.JobManager.(JobManager.java:184)
at com.evernote.android.job.JobManager.create(JobManager.java:107)
at com.dvtonder.chronus.WidgetApplication.onCreate(WidgetApplication.java:166)
at android.app.Instrumentation.callApplicationOnCreate(Instrumentation.java:1032)
at android.app.ActivityThread.handleBindApplication(ActivityThread.java:5881)
And on Android 6.0 and 4.2
com.evernote.android.job.JobManagerCreateException
com.evernote.android.job.JobManager.
java.lang.RuntimeException:
at android.app.ActivityThread.handleBindApplication(ActivityThread.java:6431)
at android.app.ActivityThread.access$1800(ActivityThread.java:229)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1887)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:148)
at android.app.ActivityThread.main(ActivityThread.java:7331)
at java.lang.reflect.Method.invoke(Method.java:0)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1230)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1120)
Caused by: com.evernote.android.job.JobManagerCreateException:
at com.evernote.android.job.JobManager.(JobManager.java:184)
at com.evernote.android.job.JobManager.create(JobManager.java:107)
at com.dvtonder.chronus.WidgetApplication.onCreate(WidgetApplication.java:166)
at android.app.Instrumentation.callApplicationOnCreate(Instrumentation.java:1037)
at android.app.ActivityThread.handleBindApplication(ActivityThread.java:6428)

Yes, this one is new, see here: https://github.com/evernote/android-job/blob/master/library/src/main/java/com/evernote/android/job/JobManager.java#L184
This exception is thrown when all APIs are disabled, e.g. when the services in the manifest are disabled. This usually happens only with rooted or modded devices. In this case the library can't work and you would schedule jobs and wonder why none of those jobs are running.
At the moment I don't have a better solution to notify you about this issue as early as possible. Do you have any recommendation or better way to deal with this?

Hmm, no, I cannot think of a better way of handling it.  I suppose all I can do it catch that exception in my app and display an error message.  Without the Jobs running my app is useless, may a well throw a big red warning triangle :)

Yes, this was my intention how to handle it.

For me it's an Android 7 device, crashlytics reports as not rooted.
What is the reason this crashes?

Xposed? I have no idea, honestly. Any idea what I should do instead?

I got this report in Android 6.0. I suspend android device may not finish boot?

No, it means that no API is supported for some reason. That's the way we setup the JobManager to prevent the crash.
  private void setupJobManager() {
    JobCat.addLogPrinter(new EvernoteJobCreator.JobLibraryLogger());
    JobApi.setForceAllowApi14(true);

    JobManager jobManager = JobManager.create(this);
    jobManager.getConfig().setGcmApiEnabled(false); // is only important for Android 4.X
    jobManager.addJobCreator(new EvernoteJobCreator());
  }

Note that setForceAllowApi14 is deprecated and will be moved to a different place in version 1.2.0.

@vRallev Now that 1.2.0 is out, this answer no longer works, as you anticipated. It seems the location for setting GcmApiEnabled, and setForceAllowApi14 have moved. What is the solution for 1.2.0?

JobConfig. There you can enable/disable APIs.

If I understand correctly, this answer provides work around for the situations where JobManagerCreateException is thrown. Is that correct? If so, even in 1.2.0, do we still need to (1) force allow api 14 (2) disable GcmApi for Android 4.X as shown above?

That's correct. It depends on you. We disable the GCM API, because we only want to use the platform version.

@vRallev For us all reports so far are from LG devices running android 7.0. We only support api > 16. So we shouldn't see this error?
On API 24 & 25 android.app.job.JobService is supported. So why does the JobApi.getDefault(context) return a JobApi.V_14 object.

So, JobApi.setForceAllowApi14(true)  in v 1.2.+ it should become JobConfig.setForceAllowApi14(true)
But  what to set instead of  jobManager.getConfig().setGcmApiEnabled(false) to have the same effect? I found _ JobConfig.forceApi(new GCM)_ but what object needed to send to it.

@vRallev
So, JobApi.setForceAllowApi14(true) in v 1.2.+ it should become JobConfig.setForceAllowApi14(true)
But what to set instead of jobManager.getConfig().setGcmApiEnabled(false) to have the same effect? I found _ JobConfig.forceApi(new GCM)_ but what object needed to send to it.

@vRallev For us all reports so far are from non-rooted LG devices running android 7.0. We only support api > 16. So we shouldn't see this error?

Same here, minSdk 16 and on Crashlytics we see a lot of
com.evernote.android.job.JobManagerCreateException: All APIs are disabled, cannot schedule any job

especially on non-rooted devices running android 7.0 (Moto G4).
@vRallev could you please clarify how force allow JobConfig.forceApi(JobApi.V_14) and JobConfig.setApiEnabled(JobApi.GCM, false) could help in this scenario?

would this be the correct workaround?
  try {
    JobManager.create(this).addJobCreator(new EvernoteJobCreator());
  } catch (JobManagerCreateException e) {
    JobConfig.forceApi(JobApi.V_14);
    JobConfig.setApiEnabled(JobApi.GCM, false);
    JobManager.create(this).addJobCreator(new EvernoteJobCreator());
  }
}

This is how we setup the library. We don't use a try-catch block.
    JobConfig.addLogger(new EvernoteJobCreator.JobLibraryLogger());
    JobConfig.setForceAllowApi14(true);
    JobConfig.setApiEnabled(JobApi.GCM, false); // is only important for Android 4.X

    JobManager.create(this).addJobCreator(new EvernoteJobCreator());

We've got the same crash. I'm reading docs and there is written that setApiEnabled is intended only for test purposes. Is it correct?

implementation 'com.evernote:android-job:1.2.6'
Fatal Exception: java.lang.RuntimeException: Unable to create application com.droid4you.application.wallet.Application: com.evernote.android.job.JobManagerCreateException: All APIs are disabled, cannot schedule any job
Galaxy S5 Mini: 6.0.1 (SM-G800H)
Never seen before on older android-job versions. User is using our app for 5 months, so it didn't crashed on 1.2.5 or previous, but start with 1.2.6 (for now just one user with one crash)

@vRallev
        JobConfig.addLogger(new MyJobCreator.JobLibraryLogger());
        JobConfig.setForceAllowApi14(true);
        JobConfig.setApiEnabled(JobApi.GCM, false); // is only important for Android 4.X
        JobManager.create(this).addJobCreator(new MyJobCreator());

I have used the above as mentioned by you. But I am unable to find MyJobCreator.JobLibraryLogger() in the first line. Any pointers here?

@swapnilgt
This should be your class which implements JobCreator

Be aware of tool:node="replace" in your AndroidManifest file.
That leads to the same problem.


Dropbox daily monthly

Hi! Bumped into this issue recently. The problem is Travis is using a global phpunit and it is not working the same as before. Have a look at https://github.com/kompisbyran/Kompisbyran/pull/190 and see if you can solve it this way in this repo too.

Hey @mctdev-marius !
Thanks for this PR, this is quite interesting! You guys always have nice ideas that I had never thought about! 😃
But there is one disadvantage about this, I'm afraid that may complicate the bundle, and not cross-storage compatible for a non-vital feature.
I'm sure we can find a simpler way to do it, and managing it directly right from the crontab manager.
What about just specifying the remote_path directly from the command line? Let me show you an exemple:
#every day
1 1 * * * SYMFONY__REMOTE__PATH=/normal/path php app/console --env=prod dizda:backup:start >/dev/null 2>&1
#first day of the month
1 1 1 * * SYMFONY__REMOTE__PATH=/different/path php app/console --env=prod dizda:backup:start >/dev/null 2>&1

More details here http://symfony.com/doc/master/configuration/external_parameters.html
Then you can add more crons to delete oldest files in both directories if more than X items: http://stackoverflow.com/questions/25785/delete-all-but-the-most-recent-x-files-in-bash
What do you guys think?

Hello @dizda ,
and thanks for your work on this project!
I did not know about the external parameters so thanks for that, it can be very helpful. As for the crons to delete old files it will not work in my case (DropBoxSDK storage).
You see, my integration is not meant for all the storage is just (for now) DropboxSDK oriented.
In fact all the code add (except for the Travis PHP 7 part) is on the configuration (so i can make the user set up the right parameters) and DropboxSDK client file.
What it dose is uploads the backup archive to different DropBox directory based on the day of month, then it checks the dropbox file list to see if it needs to delete files based on the settings for DropboxSDK.
This can be very useful if your dropbox account has a space limit and u don't want to have an infinite number of backups eating up your subscription plan.
I normally have a 5 files for the daily and 2 for monthly.
Please let me know if i'm missing the long term problems that can complicate a  already grate bundle (yours).

Yeah sorry, I didn't get than you were talking about removing remote files on the cloud storage rather than the ones on your local...
So I might be agree on the utility to remove files if the total number is higher than a specified amount.
But I'm really not sure that other people would need to upload the backup to specific directories depending on the current day.
To me, you should handle that on your side, but let's see what other people think?

Yes i agree, the directory and the limit number based on the folder they end up in can be set via crons so you can make of whit out extra code to decide where to upload the created backup.
I add it because i did not know about the parameters sent via cl so i need to keep my monthly/daily backups organized and limit the amount of backups stored meaning i don't end up whit 2 backups if it's the last day of month (say you wanna share only the monthly backups whit your client).
Let's see if i'm the only one needing this running this way or if it can bee useful to others.


Add ability to automatically disable 2FA when in developer mode

Closes #3 and is backwards-compatible.
New possible combinations

disable 2FA throughout (existing)

Build in the ability to automatically disable 2FA if dev mode is enabled.

disable 2FA only when Magento is set to developer mode (new)

We also need an option (env var) to keep 2FA enabled in dev mode if debugging is needed.

enable 2FA at all times (existing)

@drpayyne I find the naming implemented in your PR confusing. One toggle saying "2FA is enabled, default to yes", and another toggle saying "disable only in developer, default to yes" -- one uses a positive statement, and another uses a negative statement. See https://www.obsidiansecurity.com/blog/why-to-stop-writing-negative-code/ for more info.
I strive to only use positive statements as it makes a lot more sense and makes things easier to follow.
This module is only intended to be used in development, and the only way it would be a breaking change is if they ran it on production while in developer mode. Since developer mode causes tons of inherent security issues, disabling 2FA by default, if a site has developer mode enabled on prod, is really not of my concern. It's not any worse than writing errors to the browser.
That said, 100% of the developers installing this module would like 2FA disabled for dev, so this update shouldn't be breaking as "enable in developer mode" would default to Yes. In dev, developer mode is always on. This update would also allow them to toggle 2FA back on in developer mode if they wish. I think the multiple toggles makes sense here. It would also allow us to get rid of the need to set config variables after install, so someone can just install it and 2FA would automatically be disabled in developer mode.

Thank you so much for the detailed reply @markshust. I understand and agree with you.
I will follow up with the changes very soon. Thank you!

@drpayyne I made some updates, let me know what you think 0cd8e1f (#13)
I simplified the initial idea by adding a "Disable 2FA in developer mode" toggle. By default this is set to Yes, so 2FA is auto-disabled when in developer mode. If set to No, it falls back to the other 2 config options. So, this should be backwards-compatible, unless if they are in developer mode, in which it auto-disables 2FA. This is a breaking update, so I'll need to tag as 2.0.

@drpayyne made a few other updates before merging in. I really like this workflow/toggle, it's sort of a combination of both of our ideas. Testing and works really well! Tagging 2.0.0.

Awesome! Looks amazing.


replace ci.opensearch.org artifact source with nexus

Description
failing tests due to preparation for 2.6 release and OpenSearch / job-scheduler dependencies
Issues Resolved
https://github.com/opensearch-project/observability/issues/1420
Check List

[ ] New functionality includes testing.

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

[ ] New functionality has been documented.

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

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

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

not relevant anymore


DEPRECATION WARNING with APT

@benyanke,
Thanks for posting!
I'll get around to this shortly, for my own record here's a solid remediation:

https://stackoverflow.com/a/52743335

@benyanke,
I've rebuilt my images to update Ansible to 2.7.1 on Python 2.7.15+... I'm not able to replicate it under these conditions.
Are you able to provide some details about python version, ansible version and operating system?

Sorry - just came back to this, and it's still occuring I'm on 18.04, with the following ansible version info:
ansible 2.7.10
  config file = /home/benyanke/.ansible.cfg
  configured module search path = [u'/home/benyanke/nextcloud/ansible/library']
  ansible python module location = /usr/lib/python2.7/dist-packages/ansible
  executable location = /usr/bin/ansible
  python version = 2.7.15rc1 (default, Nov 12 2018, 14:31:15) [GCC 7.3.0]

I've also updated the role:
┌─[☢ ]─[🔑]─[~/nextcloud/ansible/playbooks/linux-desktops]─[7.04ms]
└──╼ ansible-galaxy install fubarhouse.golang --force
- changing role fubarhouse.golang from 2.8.0 to unspecified
- downloading role 'golang', owned by fubarhouse
- downloading role from https://github.com/fubarhouse/ansible-role-golang/archive/2.8.0.tar.gz
- extracting fubarhouse.golang to /home/benyanke/.ansible/roles/fubarhouse.golang
- fubarhouse.golang (2.8.0) was installed successfully

I'm quite sure this one will be resolved by #101.
Any chance you could look over it to see if this is true?
TASK [role_under_test : Go-Lang | Install dependencies] ************************
task path: /etc/ansible/roles/role_under_test/tasks/tasks-Debian.yml:3
<localhost> ESTABLISH LOCAL CONNECTION FOR USER: root
<localhost> EXEC /bin/sh -c 'echo ~root && sleep 0'
<localhost> EXEC /bin/sh -c '( umask 77 && mkdir -p "` echo /root/.ansible/tmp/ansible-tmp-1555012194.59-53742339537298 `" && echo ansible-tmp-1555012194.59-53742339537298="` echo /root/.ansible/tmp/ansible-tmp-1555012194.59-53742339537298 `" ) && sleep 0'
Using module file /usr/local/lib/python2.7/dist-packages/ansible/modules/packaging/os/apt.py
<localhost> PUT /root/.ansible/tmp/ansible-local-6689NLQtbX/tmpJJgd7l TO /root/.ansible/tmp/ansible-tmp-1555012194.59-53742339537298/AnsiballZ_apt.py
<localhost> EXEC /bin/sh -c 'chmod u+x /root/.ansible/tmp/ansible-tmp-1555012194.59-53742339537298/ /root/.ansible/tmp/ansible-tmp-1555012194.59-53742339537298/AnsiballZ_apt.py && sleep 0'
<localhost> EXEC /bin/sh -c '/usr/bin/python /root/.ansible/tmp/ansible-tmp-1555012194.59-53742339537298/AnsiballZ_apt.py && sleep 0'
<localhost> EXEC /bin/sh -c 'rm -f -r /root/.ansible/tmp/ansible-tmp-1555012194.59-53742339537298/ > /dev/null 2>&1 && sleep 0'
ok: [localhost] => {
    "cache_update_time":<PHONE_NUMBER>,
    "cache_updated": false,
    "changed": false,
    "invocation": {
        "module_args": {
            "allow_unauthenticated": false,
            "autoclean": false,
            "autoremove": false,
            "cache_valid_time": 0,
            "deb": null,
            "default_release": null,
            "dpkg_options": "force-confdef,force-confold",
            "force": false,
            "force_apt_get": false,
            "install_recommends": null,
            "only_upgrade": false,
            "package": [
                "curl",
                "gcc",
                "git",
                "findutils",
                "make",
                "rsync",
                "tar"
            ],
            "pkg": [
                "curl",
                "gcc",
                "git",
                "findutils",
                "make",
                "rsync",
                "tar"
            ],
            "purge": false,
            "state": "present",
            "update_cache": null,
            "upgrade": null
        }
    }
}

Source: https://travis-ci.org/fubarhouse/ansible-role-golang/jobs/518922054

Yes - I assume this is handled by that existing issue. Will test as I'm able!


FE - Add Data Categories to AccessManualWebhook

Warning
Dependent on #3270

Description
The manual webhook configuration page will get a new data category dropdown added to the form. It will allow selecting one data category and be required.

moving to done


Auto mounting override labels and manual conversion

Feature request to label the overriden mounting values additionally with the mounted direction rather then just saying it's overriden. Also the ability to take the values assigned by the auto mount process and convert them to the manual positions.

sure, but I wouldn't show this values in a normal context. If it's added, it should be in developer mode


Add more info on landing page

The landing page is a bit hard to grasp without context. We should add an info text to briefly explain what Bristlemouth is and what this project is about.

Fixed in #19


chore: upgrade sharp version to avoid vulnerability

fix https://github.com/elastic/synthetics/security/dependabot/14

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-06-03T19:33:13.620+0000

Duration: 16 min 42 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
165

Skipped
2

Total
167

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.


insulin unit and basal dosage

I wonder about insulin unit used in this simulator.
Other thing is basal dosage in basal-bolus controller. The calculation is:
basal = u2ss * BW / 6000
What is the standard used in this controller? Also I can not find out what is u2ss. It is better if there is explanation about variables.
Thank you.

The basal I use (basal = u2ss * BW / 6000) is under unit "U/min", where 1 U/min = 6000 pmol/L. "u2ss" is the steady state insulin rate per kilogram, and its unit is pmol/(L * kg).
Documentation is really an issue. I need to find time to address that. Thanks!

Hey @jxx123!
Further, in controller/basal_bolus_ctrller.py, line 56
bolus = bolus / env_sample_time
scales the bolus from "U/env_sample" to "U/min", right? Then, in simulation/env.py, there is
'insulin += tmp_insulin / self.sample_time'
again, so the stored output is divided by env_sample_time. The division from this line should be omitted, right?

Hey @jxx123!
Further, in controller/basal_bolus_ctrller.py, line 56:
bolus = bolus / env_sample_time
scales the bolus from "U/env_sample" to "U/min", right? Then, in simulation/env.py, there is
insulin += tmp_insulin / self.sample_time
again, so the stored output is divided by env_sample_time. The division from this line should be omitted, right?

Hi @EemeIi , the reason to divide bolus by env_sample_time is that the original bolus is the total amount (U), and the division is to convert it to rate (U/min).
For the line insulin += tmp_insulin / self.sample_time, the purpose is just to compute the average insulin rate in the sample time (see this comment).  The reason for this is that the patient simulation sample time is 1 minute (see the mini_step function), but I would like to support arbitrary sample time in the simulator, hence I record the average values for all the variables including insulin.

Hello,  can you write down and upload the detailed documentation of the algorithm in your spare time?

Hi I have added the documentation for the controller. I closed this issue for now, please feel free to reopen it if there is anything unclear. Thanks.

1 U/min = 6000 pmol/L
@jxx123 How can this be? The right hand side has no units of time. Should u2ss be instead pmol/L/min since it's a rate?

Hi, I'm wondering how u2ss is calculated? Or it's an independent parameter that needs to be input instead of calculated?


[UBS courier/Personal data] More than 4 addresses are saved

Environment: All environments
Reproducible: always
Build found: 18.02.2022
E.g. "commit 147fa44"
Preconditions
Preconditions:

Login to GreenCity as a User https://ita-social-projects.github.io/GreenCityClient/#/
UBS Order form is opened
All required fields on the tab “Деталі замовлення” are filled

Steps to reproduce

Go to 'Personal Data' page
Scroll down to 'Garbage collection address'

Actual result
More than 4 addresses are saved
Attached bug screenshots

Expected result
The limit on the page - 4 addresses
User story and test case links
#2533
Test case
Labels to be added
"Bug", Priority ("pri: "), Severity ("severity:"), Type ("UI, "Functional"), "API" (for back-end bugs).

No longer up to date


Aggregation/Alignment on GCP Stackdriver metric

Proposal
It's crucial to have ability applying aggregation/alignment on GCP Stackdriver metric in order to both fetch less data from GCP API and get exactly what you need.
For instance, aggregator and aligner would be useful:
triggers:
- type: gcp-stackdriver
  metadata:
    projectId: my-project-id
    filter: 'metric.type="loadbalancing.googleapis.com/https/backend_latencies" AND resource.type="https_lb_rule" AND resource.label.forwarding_rule_name="k8s2-..."'
    aggregator: 'mean | sum | 95th_percentile | etc.'
    aligner: 'delta | sum | 95th_percentile | etc.'
    targetValue: '10' # latency in ms
    credentialsFromEnv: GOOGLE_APPLICATION_CREDENTIALS_JSON

Use-Case
No response
Anything else?
No response

Is it doable @RamCohen ?


feat: proxy page style

modify proxy page style
refactor proxy node card and collapse as component

this part of code needs review


Addresses #1698 edit synonym for leukocyte

Addresses #1698 edit synonym for leukocyte
Originally "immune cell" was requested to be a new term.
Due to the points made here, instead of creating a new term, the synonym "immune cell" for leukocyte has been changed from exact synonym to related synonym.

could you check on the use case for immune cell - not sure if this would meet the requesters requirements (do as a follow up)
That being said fully agree with this change so will approve it

could you check on the use case for immune cell - not sure if this would meet the requesters requirements (do as a follow up)
That being said fully agree with this change so will approve it

Great, thank you, @shawntanzk. The requester has commented that this is an acceptable solution.


Need some explanation on a data preparation code.

Hi, I found this part of the code confusing. Could you give some explanations? I put some comments that I didn't understand. Thanks.
    for ann in data['annotations']:
        segs = ann['segmentation']
        new_segs = []
        for seg in segs:
            assert len(seg) > 0 and len(seg) % 2 == 0
            if len(seg) < 4:
                new_segs.append(seg + [0, 0, seg[0], seg[1]]) # why do you add these?
            if len(seg) == 4:
                new_segs.append(seg + [seg[0], seg[1]])
            else:
                new_segs.append(seg)
            new_segs.append(seg) # Why do we need this line?
        ann['segmentation'] = new_segs

I am also interested in learning more about this. Can someone help us?

Hi, @neouyghur @iranroman
Sorry for the late reply.
For if len(seg) < 4: new_segs.append(seg + [0, 0, seg[0], seg[1]]), this is because a polygon should have at least 3 points, aka 6 coordinates. Here is another reference.
For new_segs.append(seg), this is bug... Thanks for pointing it out. Luckily, it doesn't make effect since the duplicated polygon points won't change the mask.

@PeizeSun  Could you briefly explain the purpose of pascal_part_one_json.py? thanks.

@PeizeSun Could you briefly explain the purpose of pascal_part_one_json.py? thanks.

When we build the dense semantic correspondence between novel object and base object, we expect there is only one object in the image, otherwise the correspondence maybe messed up. So we select those images that have only one object by pascal_part_one_json.py

For if len(seg) < 4: new_segs.append(seg + [0, 0, seg[0], seg[1]]), this is because a polygon should have at least 3 points, aka 6 coordinates. Here is another reference.

Thanks for your reply. However, I am a bit confused by the (0,0).  If len(seg)<4, then it is only a point. If you are going to add (0,0), the mask will be a line. This might introduce some incorrect GT.

@PeizeSun

For if len(seg) < 4: new_segs.append(seg + [0, 0, seg[0], seg[1]]), this is because a polygon should have at least 3 points, aka 6 coordinates. Here is another reference.

Thanks for your reply. However, I am a bit confused by the (0,0). If len(seg)<4, then it is only a point. If you are going to add (0,0), the mask will be a line. This might introduce some incorrect GT.

Yes, it will introduce incorrect GT. This error is caused by annotation. Under the incorrect annotation, we don't know what is the correct GT, unless we re-annotate. Therefore, this code is only used for not causing errors to following code.

@PeizeSun  In this case, Will this if len(seg) < 4: new_segs.append(seg + [ seg[0], seg[1], seg[0], seg[1]]) be better?

Probably. Maybe one point is better than one line.

@PeizeSun  Thanks for your help. I close this issue.


Using syntactic sugar fails.

Might be a good idea to document prominently that one must use e.g. (vec ...) instead of [...]. This is a stumbling block for new users.
You can use the hash-map syntactic sugar {:a 12} successfully due to #7 fix.

Hi,
Vectors are not yet supported. They are on the roadmap. Only the data types under Runtime -> Object System -> Objects are implemented.
Best,


Crash on Initialization

Describe the bug
Bugsnag Performance crashes on initialization.

Edit 1: We were able to fix the crash by setting networkRequestCallback in the config to  { _ in return .init() } which I'm not sure if it's the right approach.
Edit 2: We are using the SDK initialization and not the plist initialization.
Environment

BugSnag Performance library version: 1.6.0
BugSnag Error Monitoring library version (if any): 6.29.0
CocoaPods version:
iOS/tvOS/macOS version(s): iOS
Simulator or physical device: Simulator
Xcode version: 15.4
Swift version (if applicable): 5.9

Hi @nchegireddy
Thank you for raising this issue. We now have some work on our backlog aimed at fixing this issue. Our current aim is to have a patch release out for this early next week. We'll be sure to keep you informed via this thread once the patch is released.
For background, The issue exists for users who call the reportNetworkRequestSpan API to report their own network spans but who don't set a callback. This crept in during a recent enhancement and unfortunately was a combination that wasn't in our end-to-end tests. You can set a no-op callback to avoid the issue until we get the patch release out.

Hi @nchegireddy
Thank you for raising this with us. We are currently working on a patch for this issue. Our current aim is to have a release out early next week. We'll be sure to keep you informed via this thread once the patch is released.
For background, The issue exists for users who call the reportNetworkRequestSpan API to report their own network spans but who don't set a callback. This crept in during a recent enhancement and unfortunately was a combination that wasn't in our end-to-end tests. You can set a no-op callback to avoid the issue until we get the patch release out.

@clr182 Great to know that a fix is already in the works! We landed on the same workaround of setting the no-op callback, as you mentioned. We'll keep an eye out for the patch and get that integrated as soon as it's ready.

I get no crashes on 1.6.1, so closing this issue.


GOMC analysis update: rerun and tested mean energy, pressure and density values of NPT methane.

attached two files that verify that the added analysis code does not change the outcome of the methane run compared to when the methane run, when the analysis code was run after the simulation. Mean energy, pressure and density values agree to 6+ digits between each other and values provided by reproducibility planning excel.
rclonedData.zip
startToFinishData.zip


WIP: Linux support

This PR adds Linux support.
Here's what currently works:

Launching the game
Launching the GC server
Registry wiping
Builtin proxy
Downloading things
Service control
Installing CA certs (on Debian and Arch-based distros)
3dmigoto

Here's what I'm unsure about:

Akebi/Acrepi
Reshade Injector
Anything not on the previous list

Rebased on main, updated anime-launcher-sdk, squashed "Fix build on Windows" 8ff6397 into the commit that introduced that issue and complied with clippy.

What sorts of uncertainties do you have regarding reshade and Akebi/Acrepi/etc.? Are you unable to test them, or do you know they don't work?

I'm not sure if they work.
I don't have a copy of Akebi with which I could test the game with.
As for reshade, I haven't got around to it yet.

Okay, that's fine. I would feel fine merging without knowing if they work or not, I'll give a full code review when I have some free time :+1:

Reshade does not work. Video: https://youtu.be/hYzUbkH8R14
Should I still add the code necessary for running the injector from the launcher?

I think it should be kept, no way to know if a new Wine version would just fix it or whatever. I would imagine it to be a similar situation with Akebi-likes, so we can just assume they would work the same.

Moved terminal detection to term-detect, updated anime-launcher-sdk and complied with clippy (I missed one commit last time).

Oops. I was cleaning up branches and did not expect renaming the linux branch to close the PR.
Since https://github.com/MrLGamer/GIMI-for-Linux was updated, I'll reopen the pull request with the part2 branch, which does not contain support for 3dmigoto.


Why does 3.1.1 requires context again?

In version 3.0.2+1, context is not needed when initializing ScreenUtil.init().
Why in 3.1.1 it's required again?

Because of the mechanism of flutter, there will be unpredictable problems when getting screen data without context.
I am sorry about it.


Search request

Description:

Short Description of the needed Epic in the below format

As an IAO analyst or ministry client

I want to be able to search for a request

so that I can find an individual request

What are the most critical features being built as part of this Epic

Expected Outcome – Justification why this Feature must be built, what is expected value and justification for priority.

Sizing

T-shirt size:

[ ] S
[ ] M
[ ] L
[ ] XL
(S=1-2 Sprints, M=2-3 Sprints, L=4-5 Sprints, XL=not sure, probably more than 5 Sprints).

Risks and Assumptions*

[ ] What are assumptions?
[ ] What are the risks?
[ ] Have we done an RFC on a technical approach?
[ ] Are the dependencies (technical, business, regulatory/policy) known?
[ ] Has work on this Epic been prioritized with key stakeholders?

closing this epic as I believe it was inadvertently opened on the wrong repo


Firestore Java: Can't cancel a query from ApiStreamObserver

I want to implement an in-memory filter for a query (to reduce the number of indexes on Firestore, but also to implement features like fuzzy search) in places where I know that the number of query results is small, like hundreds but not thousands. I want to return results with a paged query: starting with a cursor, filter in memory until I have enough results to return a page or there are no more results.
I cannot do this with the current streaming implementation in Firestore SDK. What I am trying is:

Create a query with a big limit, say 1000
Start streaming
When I have enough values for a page, collect the page contents and next cursor, and return inside an ApiFuture:

// the CursorList class to return after filtering in memory
public class CursorList<T> {

  List<DocumentSnapshot> data;
  String cursor;

}

// prime up the test
var collection = FirestoreSingleton.getInstance().collection("Test");
collection.document("0").create(Map.of("order", 0)).get();
collection.document("1").create(Map.of("order", 1)).get();
collection.document("2").create(Map.of("order", 2)).get();
var query = collection.orderBy("order", Query.Direction.ASCENDING).limit(1000);

// run the in-memory filter
final int pageSize = 10;
final List<DocumentSnapshot> filteredResult = Lists.newArrayListWithExpectedSize(pageSize);
final SettableApiFuture<CursorList> future = SettableApiFuture.create();

query.stream(
  new ApiStreamObserver<DocumentSnapshot>() {
    DocumentReference cursor = null;

    private boolean passes(DocumentSnapshot value) {
      // Dummy in-memory filter implementation
      return value.get("order") % 2 == 0;
    }

    @Override
    public void onNext(DocumentSnapshot value) {
      if (passes(value)) {
        if (filteredResult.size() < pageSize) {
          filteredResult.add(instance);
        } else {
          cursor = value.getReference();
          triggerResult();

          //
          // TODO: I'd like to also cancel the streaming,
          // but it's not possible
          //

        }
      }
    }

    @Override
    public void onError(Throwable t) {
     future.setException(t);
    }

    private void triggerResult() {
      future.set(new CursorList(filteredResult, cursor));
    }

    @Override
    public void onCompleted() {
      triggerResult();
    }
  }
);
return future;

With the current implementation, an ApiStreamObserver cannot cancel the execution of a query, which will keep going until 1000 results (or whatever limit I set) have been processed, even if the future has already been resolved.
Did I miss something?  From what I could see, there is a StreamObserver that I could use to cancel a stream in GRPC, but Firestore is not supporting that. I can ditch streaming and use batches of queries, but it would be less efficient and a messier implementation.
Any help is welcome.

Sorry for replying late.
Pagination is typically done with limit/limitToLast/startAt/endAt queries. See: https://googleapis.dev/java/google-cloud-firestore/latest/index.html
Also see: https://firebase.google.com/docs/firestore/query-data/query-cursors#java_1 for some examples.
Hope this helps!

@wu-hui This does not help. The main issue is that I do not know the size of the pages in advance, because I need to do some filtering using my own code that cannot be done on Firestore indexes (for example, fuzzy search or regexp). This leaves me with two choices:

Iterate through multiple pages in memory, until I have enough filtered items to return a results page. Since each page returned by Firestore reads all results in memory in a List, I need to keep the page size reasonable.
Use ApiStreamObserver and process results one by one until I have enough to return a results page. This would be preferable because the implementation is simpler (see example above), but I have no way to stop the stream when I have completed a page.

Both cases return both the page of results (filtered in memory, using my own code) and the cursor to continue searching. With the current status of the SDK, option 2 does not seem possible.

Hey @icoloma
Sorry for not closing the issue without fully understanding your requirement.
Unfortunately this is not supported currently. I can see we support it in the future, but I cannot say when will this happen.
I created an internal issue to track this, and I will try to report back here when we do have a timeline.


fix: wrong link in the local-setup

This PR fixes #914

LGTM but please fix the markdown errors :)

I hope it's working fine now :) @mowies


Support block subranges

Figure out the right way to support reading a subrange of block content. For literal blocks, this is easy - just need to open a stream, skip some portion of the content, and wrap a bounded stream around that. Both file and s3-backed blocks support subrange reading.
Maybe the stat metadata can include a marker for whether the reader function supports being invoked with range arguments. If so, call it like (reader start end).

Merged in 4a0df2953037cdb7811663d6b2b4816d3d525e33. Rather than explicit metadata, the open function just tries to call reader with two args, and if it throws an ArityException it falls back to the naive skip + bounded input stream approach.


Bumped ccx-rules-ocp version

Codecov Report
Patch and project coverage have no change.

Comparison is base (59604fc) 73.46% compared to head (3cd2345) 73.46%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #488   +/-   ##
=======================================
  Coverage   73.46%   73.46%
=======================================
  Files          11       11
  Lines         603      603
=======================================
  Hits          443      443
  Misses        135      135
  Partials       25       25

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


GetAccountAsync does not work with Guest account in single tenant configuration

Hi,
I'm using this sample because it's the closest to our application, i.e. using .NET Framework.
Our requirement is to support external user (guest account) authentication on our AAD tenant.
The authorization is performed on AD groups assigned to these guest accounts in our AAD tenant.
Therefore, we need to configure the application in single tenant mode (using our tenant id), otherwise the access token will be issued by the guest account's tenant and the call to Microsoft Graph API to retrieve authenticated user's AD groups will be performed on the tenant of the user instead of ours.
The problem is that in single tenant mode, the ClaimsPrincipal.Current.GetAccountId() does not return the same value as the TokenCacheNotificationArgs.SuggestedCacheKey :
Indeed, GetAccountId returns the "objectId.tenantId" from our AAD tenant, whereas the SuggestedCacheKey manages for some reason (couldn't find out how) to retrieve the correct HomeAccountId of the user's tenant.
As a result, GetAccountAsync returns null and AcquireTokenSilent doesn't work.
Thank you for your help.

@jennyf19. We also need to improve this sample by adding a call to the user info endpoint.
It seems we'd better have Microsoft.Identity.Web support ASP.NET directly


[Android] NullPointerException @ Track.toMediaMetadata

Receiving this crash log in production log reports.
java.lang.NullPointerException:
  at com.guichaguri.trackplayer.service.models.Track.toMediaMetadata (Track.java:113)
  at com.guichaguri.trackplayer.service.metadata.MetadataManager.updateMetadata (MetadataManager.java:194)
  at com.guichaguri.trackplayer.service.MusicManager.onTrackUpdate (MusicManager.java:191)
  at com.guichaguri.trackplayer.service.player.ExoPlayback.onPositionDiscontinuity (ExoPlayback.java:242)
  at com.guichaguri.trackplayer.service.player.ExoPlayback.onTimelineChanged (ExoPlayback.java:222)
  at com.google.android.exoplayer2.ExoPlayerImpl$PlaybackInfoUpdate.notifyListeners (ExoPlayerImpl.java:771)
  at com.google.android.exoplayer2.ExoPlayerImpl.updatePlaybackInfo (ExoPlayerImpl.java:708)
  at com.google.android.exoplayer2.ExoPlayerImpl.handlePlaybackInfo (ExoPlayerImpl.java:639)
  at com.google.android.exoplayer2.ExoPlayerImpl.handleEvent (ExoPlayerImpl.java:584)
  at com.google.android.exoplayer2.ExoPlayerImpl$1.handleMessage (ExoPlayerImpl.java:125)
  at android.os.Handler.dispatchMessage (Handler.java:105)
  at android.os.Looper.loop (Looper.java:173)
  at android.app.ActivityThread.main (ActivityThread.java:6698)
  at java.lang.reflect.Method.invoke (Native Method)
  at com.android.internal.os.Zygote$MethodAndArgsCaller.run (Zygote.java:240)
  at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:782)

By the way, thank you for your recent updates w/ fixing crashes - my crash reports have gone down significantly! But, still receiving this one from time to time

This issue seems to be related to the track URL not being set. I'll be closing this for inactivity, feel free to reopen it if you're still facing the issue.


add callout for json writer space needed

We had another customer curious about this behavior so thought it might be useful to have it in the docs to hopefully make it easier to find.
cc @daisukeiot

/azp run c - client - ci

@ahsonkhan let me know if you would like this modified further but going to merge for now.


Perform check using implicit list length

Resolves #34
Hello!
Pull request overview

Perform compute capabilities check using implicit list length instead of ccs is not None, as ccs is always a list, so this if-statement is currently always True.

Details
The function within which the exception from #34 occurs:
https://github.com/TimDettmers/bitsandbytes/blob/9b5f2eda8fbd3f042c4af7ed1b870525d4668f2a/bitsandbytes/cuda_setup/main.py#L99-L109
This function takes ccs from get_compute_capabilities:
https://github.com/TimDettmers/bitsandbytes/blob/9b5f2eda8fbd3f042c4af7ed1b870525d4668f2a/bitsandbytes/cuda_setup/main.py#L65-L95
As you can see, ccs is initialized as a list, and is only optionally appended to. In short, it is never None, so ccs is not None is always True, even if ccs is an empty list.
I've resolved this by using the implicit list length check using if ccs:. The new traceback from the Colab provided by @justheuristic (Thank you for raising the issue, and providing the Colab) is:

Traceback
===================================BUG REPORT===================================
Welcome to bitsandbytes. For bug reports, please submit your error trace to: https://github.com/TimDettmers/bitsandbytes/issues
For effortless bug reporting copy-paste your error into this form: https://docs.google.com/forms/d/e/1FAIpQLScPB8emS3Thkp66nvqwmjTEgxp8Y9ufuWTzFyr9kJ5AoI47dQ/viewform?usp=sf_link
================================================================================
CUDA_SETUP: WARNING! libcudart.so not found in any environmental path. Searching /usr/local/cuda/lib64...
CUDA SETUP: CUDA runtime path found: /usr/local/cuda/lib64/libcudart.so
CUDA exception! Error code: no CUDA-capable device is detected
CUDA exception! Error code: initialization error
CUDA SETUP: Highest compute capability among GPUs detected: None
CUDA exception! Error code: no CUDA-capable device is detected
CUDA SETUP: CUDA version lower than 11 are currenlty not supported for LLM.int8(). You will be only to use 8-bit optimizers and quantization routines!!
CUDA SETUP: Detected CUDA version 00
CUDA SETUP: TODO: compile library for specific version: libbitsandbytes_cuda00_nocublaslt.so
CUDA SETUP: Defaulting to libbitsandbytes.so...
CUDA SETUP: CUDA detection failed. Either CUDA driver not installed, CUDA not installed, or you have multiple conflicting CUDA libraries!
CUDA SETUP: If you compiled from source, try again with `make CUDA_VERSION=DETECTED_CUDA_VERSION` for example, `make CUDA_VERSION=113`.

/usr/local/lib/python3.7/dist-packages/bitsandbytes/cuda_setup/paths.py:21: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('/usr/local/nvidia/lib64'), PosixPath('/usr/local/nvidia/lib')}
  "WARNING: The following directories listed in your path were found to "
/usr/local/lib/python3.7/dist-packages/bitsandbytes/cuda_setup/paths.py:99: UserWarning: /usr/local/nvidia/lib:/usr/local/nvidia/lib64 did not contain libcudart.so as expected! Searching further paths...
  f'{candidate_env_vars["LD_LIBRARY_PATH"]} did not contain '
/usr/local/lib/python3.7/dist-packages/bitsandbytes/cuda_setup/paths.py:21: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('"/usr/local/bin/dap_multiplexer","enableLsp"'), PosixPath('{"kernelManagerProxyPort"'), PosixPath('["--ip=<IP_ADDRESS>"],"debugAdapterMultiplexerPath"'), PosixPath('"<IP_ADDRESS>","jupyterArgs"'), PosixPath('true}'), PosixPath('6000,"kernelManagerProxyHost"')}
  "WARNING: The following directories listed in your path were found to "
/usr/local/lib/python3.7/dist-packages/bitsandbytes/cuda_setup/paths.py:21: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('/env/python')}
  "WARNING: The following directories listed in your path were found to "
/usr/local/lib/python3.7/dist-packages/bitsandbytes/cuda_setup/paths.py:21: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('module'), PosixPath('//ipykernel.pylab.backend_inline')}
  "WARNING: The following directories listed in your path were found to "

---------------------------------------------------------------------------

Exception                                 Traceback (most recent call last)

[<ipython-input-1-49909ea8fb42>](https://localhost:8080/#) in <module>
----> 1 import bitsandbytes

5 frames

[/usr/local/lib/python3.7/dist-packages/bitsandbytes/cextension.py](https://localhost:8080/#) in initialize(self)
     25                 print('CUDA SETUP: CUDA detection failed. Either CUDA driver not installed, CUDA not installed, or you have multiple conflicting CUDA libraries!')
     26                 print('CUDA SETUP: If you compiled from source, try again with `make CUDA_VERSION=DETECTED_CUDA_VERSION` for example, `make CUDA_VERSION=113`.')
---> 27                 raise Exception('CUDA SETUP: Setup Failed!')
     28             self.lib = ct.cdll.LoadLibrary(binary_path)
     29         else:

Exception: CUDA SETUP: Setup Failed!

I would argue that the new traceback is much clearer.

Tom Aarsen

Thank you!


Abletoshowpost

aa

I mistakenly requested to original repository.
It is my fault. I am so sorry.


Pixi is encapsulated in a 'Self Invoking Anonymous Function'...

Hi, I was just trying to write a patch for my own project that would allow me to scale my spine data but I have found I cant overwrite stuff because it is all hidden within a function. See #1344
With the PIXI namespace this should not be necessary as the global scope would not be polluted.
Perhaps this is just related to the "spine" class. Because it is hidden within the function there is no way to reference it and therefore if I was to try and change how it is parsed, I have no reference to it.
Perhaps to use Spine, users should be required to load the library themselves rather than PIXI have a nested version.
Thanks.

You mean you can't override the spine implementation (not PIXI's but Esoteric's)? I'm confused as to what "stuff" you are referring to.

I want to change the function "PIXI.JsonLoader.prototype.onJSONLoaded" so that I can change what happens when spine data is loaded.
I want to have an external script called "pixi.patches.js" that I load after pixi and therefore overwriting this method.
The prolem is in my "patches" file, I have no reference to "spine" because it is being deglared inside the PIXI annon function.
When i run:
        // spine animation
        var spineJsonParser = new spine.SkeletonJson();
        spineJsonParser.scale = 0.5;
        var skeletonData = spineJsonParser.readSkeletonData(this.json);
        PIXI.AnimCache[this.url] = skeletonData;
        this.onLoaded();

I get an error because spine is not in the global scope.
Either I could load the esotericsoftware version externally (is it the same?) so it is in the global scope or it needs to be accessible through the PIXI namespace.
Does that make sense?

Yes, so you are trying to access the internal spine runtime. We don't expose that in v2, but we can talk about doing it for v3.
The runtime is not the same as what you load from Esoteric directly, so that wouldn't work.

Yeah, I it might be a good idea because there are features that can be used (such as scaling) but it is not accessible. Would be good to have more control over this in V3 or perhaps make necessary modifications so that it can use the Esoteric version directly rather than having a custom version.

Ha, I just ran into exactly the same problem! For now I'm working around it in my project by removing the self invoking function that wraps pixi since I don't need to use pixi that way. Glad to see somebody has already logged this as an issue.

Pixi's spine runtime is now exposed in v3.


formatted index.html for my portfolio

For the record: This is (temporarily) the base branch of #3


Logs are not sent via UDP

Summary
Most of the messages are sent but some sporadic ones ones are not and the error is reflected in logs (see below)
Are you aware of any issue that might cause this? Any config we might tune for this matter?
We will keep monitoring this issue and in case of news I'll post them here.
Thanks in advance for your time.
Steps To Reproduce

Enable UDP log
Read file /usr/local/kong/logs/error.log

Additional Details & Logs

Kong version 0.11.0
Operating System Ubuntu 16.04

Error message (IPs are redacted):
2017/12/01 08:42:40 [alert] 1401#0: *5057272 send() failed (90: Message too long), context: ngx.timer, client: xx.xx.xx.xx, server: <IP_ADDRESS>:80
2017/12/01 08:42:40 [error] 1401#0: *5057272 [lua] handler.lua:28:  [udp-log] could not send data to xxxx.xxxx.xxxx.9900: message too long, context: ngx.timer, client: xx.xx.xx.xx, server: <IP_ADDRESS>:80

Datagram is bigger than an MTU on the path and it cannot be fragmented.
the Linux/C error codes

#define EMSGSIZE 90 /* Message too long */

The MTU is the maximum size for 1 package that can be sent. For Ethernet this MTU is 1500.

I am closing this issue down sinde @WALL-E seems correct (thanks!). @pachico , if you feel your issue isn't solved, please reopen.

We ended up having to use TCP indeed.
However, I don't understand the purpose of a UDP logging system that will send messages bigger than what a UDP package can handle.
Cheers

@pachico  Please confirm the following two values when logs are not sent via UDP

response size ngx.var.bytes_sent
request size ngx.var.request_length


Add DeleteOpts for container deletion

ping @dmcgowan
Closes #1025
Signed-off-by: Michael Crosby<EMAIL_ADDRESS>

Codecov Report

Merging #1079 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1079   +/-   ##
=======================================
  Coverage   54.51%   54.51%
=======================================
  Files          16       16
  Lines        1229     1229
=======================================
  Hits          670      670
  Misses        386      386
  Partials      173      173

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 a33e8d0...3f45e43. Read the comment docs.

LGTM


[home-assistant] bump home-assistant to use common chart 2.1.0

Description of the change
Bump common chart library dependency to 2.1.0
Benefits
This should help solve issue with code-server & settings persistence
Checklist

[x] Chart version bumped in Chart.yaml according to semver.
[x] Title of the PR starts with chart name (e.g. [home-assistant])
[x] Chart is using our common library as a dependency.
[ ] (optional) Variables are documented in the README.md

@billimek renovate got this :)
https://github.com/k8s-at-home/charts/pull/417

bah, I saw renovate run and not open anything.  Thanks!

@onedr0p just beat me to it 😄


OG Images Part 2

A true show art generator.
To be used as

Show art
OG images

Pass in dimension into the url to get a specific container size


RH: blocks from Autocad sometimes not converted when sent with other geometry

Sometimes blocks are not converted (with no warning or error message) in the Rhino connector when sent from AutoCAD.
Tested with manually imported blocks from Rhino, which may be relevant for troubleshooting.

Cannot reproduce this! all blocks from different test files coming in just fine 🤷♀


UART communication between CC1312 and ESP32

Communicating MCUs through the UART port
This feature is necessary to communicate both MCUs
We have a bunch of options available, for example:

SLIP - Serial Line Internet Protocol -

https://tools.ietf.org/html/rfc1055

CSLIP - Compressed SLIP -

https://tools.ietf.org/html/rfc1144

KISS - keep it simple, stupid -

https://en.wikipedia.org/wiki/KISS_(TNC)

Modbus RTU

https://www.rtautomation.com/technologies/modbus-rtu/
https://ipc2u.com/articles/knowledge-base/modbus-rtu-made-simple-with-detailed-descriptions-and-examples

PR #83 add  a communication layer
also V.A.I.N.A adds with the PR #93  a basic functionality for to use IPV6 addresses

SLIP and VAINA are merged, so it's fine to close this, the radio and the esp32 talk to each other nicely right now :)


fix: CLI link for stacks was broken

Found an issue with the CLI when on the stack selection. The link includes ) in the link when clicking the link from the terminal. This resulted in a 404 page.

Hey @jwoods-getlabs, since this will be changing remix code, can you target the dev branch instead?

Hey @jwoods-getlabs, since this will be changing remix code, can you target the dev branch instead?

Yeah, I will resubmit for the dev branch.


refactor: Drop Server User Table

🍰 Pullrequest
drop server users table. Add isAdmin to users table.
Issues

None

Todo

[X] None

Has the deployment in backend branch merged


Finish support for inputs and missing data

The major pieces are in place and Gibbs sampling should work, but we have a few more TODO's to fix. Here's a list (which is surely incomplete):

[x] Write a demo script for the input driven SLDS and make sure the results pass the sniff test.
[ ] Write some tests for the SLDS in general
[ ] Update the mean field _set_gaussian_expected_stats for both the general setting and the missing data setting.
[ ] write Cython functions to replace the slow einsums for computing info form potentials, especially for the meanfield case.

Quick update: SLDS has been updated to mostly support inputs. We still need to do some work on the set_gaussian_expected_stats functions, but these will be nearly identical to their counterparts in pylds.

I believe the _set_gaussian_expected_stats functions now support both inputs and missing data and will work with both VBEM and mean field VI.   I'm going to close this issue as the remaining todo list is about tests and performance, neither of which have to do with inputs and missing data.


Add route table

https://github.com/cybercongress/go-cyber/blob/main/x/grid/spec/04_messages.md
convert to tables all types of messages
keep txhash, height and timestamp

add indexes on everything expect coin

MsgEditRouteAlias
didn't find such raws


fix: show error message for invalid receipt

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/28343
PROPOSAL: https://github.com/Expensify/App/issues/28343#issuecomment-1738414257
Tests

Login to any account
Send a Money Request to another account, using the Scan Receipt option
Click on the Receipt image to open the preview popup
Click on the “Three dots” icon >> select the “Replace” option
Replace the current image with an unsupported file. Ex: CORRUPT.pdf
Verify that the error message is shown and users can download the invalid file
Click clear error icon, verify that the errors are removed, and the previous file is shown

[x] Verify that no errors appear in the JS console

Offline tests

Same as above
QA Steps

Login to any account
Send a Money Request to another account, using the Scan Receipt option
Click on the Receipt image to open the preview popup
Click on the “Three dots” icon >> select the “Replace” option
Replace the current image with an unsupported file. Ex: CORRUPT.pdf
Verify that the error message is shown and users can download the invalid file
Click clear error icon, verify that the errors are removed, and the previous file is shown

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else

[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)

[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/113963320/93a399b7-6ab1-4332-9602-c37931869674

Android: mWeb Chrome

https://github.com/Expensify/App/assets/113963320/526a9d20-b300-45c5-b940-f247ac93369b

iOS: Native

https://github.com/Expensify/App/assets/113963320/706c9693-8a5d-4125-9026-c8f0b561c915

iOS: mWeb Safari

https://github.com/Expensify/App/assets/113963320/3220c2ad-36e4-4751-ac50-85cae5e53093

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/113963320/bc832ef2-51ee-488a-bdb9-053d2fc2f098

MacOS: Desktop

https://github.com/Expensify/App/assets/113963320/055b327f-440b-43d7-8e15-06e00459dfcb

https://github.com/Expensify/App/actions/runs/7703963164

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Cherry-picked to staging by https://github.com/francoisl in version: 1.4.33-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to staging by https://github.com/jasperhuangg in version: 1.4.34-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
cancelled 🔪

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/jasperhuangg in version: 1.4.34-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@tienifr I think this PR caused this regression, could you please look into it?

🚀 Deployed to production by https://github.com/francoisl in version: 1.4.34-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Standby mode metric

Hi!
I just added the host standbyMode metric. Not a big change at all.
https://code.vmware.com/apis/196/vsphere/doc/vim.HostSystem.StandbyMode.html

@rmontenegroo Hello and Thank you for the PR! This all looks good but is there any way we could get a test for it this metric?

Sure! Trying to follow your CI script test. Using python 3.6.9 here but I got
(python3.6) rodmonte@deadpool:~/develop/vmware_exporter$ pytest --cov=. -v tests/unit
Traceback (most recent call last):
  File "/home/rodmonte/develop/python3.6/bin/pytest", line 5, in <module>
    from pytest import main
  File "/home/rodmonte/develop/python3.6/lib/python3.6/site-packages/pytest.py", line 13, in <module>
    from _pytest.fixtures import fixture, yield_fixture
  File "/home/rodmonte/develop/python3.6/lib/python3.6/site-packages/_pytest/fixtures.py", line 839, in <module>
    class FixtureFunctionMarker(object):
  File "/home/rodmonte/develop/python3.6/lib/python3.6/site-packages/_pytest/fixtures.py", line 841, in FixtureFunctionMarker
    params = attr.ib(convert=attr.converters.optional(tuple))
TypeError: attrib() got an unexpected keyword argument 'convert'

right after pip installing requirements.
Any tip?

Does this help? https://github.com/pytest-dev/pytest/issues/5903

@Jc2k Thanks for the tip!
I changed requirements-test.txt so as to get tests working here:
pytest_docker_tools==0.2.0
pytest==5.4.1
pytest-cov==2.8.1
pytest-twisted==1.12
codecov==2.0.15
flake8==3.6.0