TranspilerError with Unconventional Physical Qubit Indices in Layout

Environment

Qiskit version: 1.2.4
Python version: 3.12.6
Operating system: macOS Sequoia 15.1

What is happening?
Providing unconventional physical qubit indices (e.g., -1,<PHONE_NUMBER>) in the initial_layout for stress testing results in a TranspilerError without a clear explanation. The error states that the layout is “not full,” which is misleading as all virtual qubits are mapped. This behaviour limits testing scenarios where users intentionally supply extreme indices.
Full Traceback:
Traceback (most recent call last):
  File "~/lib/python3.12/site-packages/qiskit/transpiler/passmanager.py", line 464, in wrapper
    return meth(*meth_args, **meth_kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/transpiler/passmanager.py", line 226, in run
    return super().run(
           ^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/passmanager/passmanager.py", line 232, in run
    _run_workflow(program=program, pass_manager=self, callback=callback, **kwargs)
  File "~/lib/python3.12/site-packages/qiskit/passmanager/passmanager.py", line 292, in _run_workflow
    passmanager_ir, final_state = flow_controller.execute(
                                  ^^^^^^^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/passmanager/base_tasks.py", line 218, in execute
    passmanager_ir, state = next_task.execute(
                            ^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/passmanager/base_tasks.py", line 218, in execute
    passmanager_ir, state = next_task.execute(
                            ^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/passmanager/base_tasks.py", line 218, in execute
    passmanager_ir, state = next_task.execute(
                            ^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/transpiler/basepasses.py", line 195, in execute
    new_dag, state = super().execute(
                     ^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/passmanager/base_tasks.py", line 98, in execute
    ret = self.run(passmanager_ir)
          ^^^^^^^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/transpiler/passes/layout/apply_layout.py", line 56, in run
    raise TranspilerError("The 'layout' must be full (with ancilla).")
qiskit.transpiler.exceptions.TranspilerError: "The 'layout' must be full (with ancilla)."

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "~/quantum_circuit_builder.py", line 2002, in <module>
    main()
  File "~/quantum_circuit_builder.py", line 1953, in main
    try_angles_qiskit()
  File "~/quantum_circuit_builder.py", line 1876, in try_angles_qiskit
    transpiled_qc = transpile(qc_test, simulator, initial_layout=layout)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/compiler/transpiler.py", line 391, in transpile
    out_circuits = pm.run(circuits, callback=callback, num_processes=num_processes)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/transpiler/passmanager.py", line 441, in run
    return super().run(circuits, output_name, callback, num_processes=num_processes)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/transpiler/passmanager.py", line 466, in wrapper
    raise TranspilerError(ex.message) from ex
qiskit.transpiler.exceptions.TranspilerError: "The 'layout' must be full (with ancilla)."
~ % python3 quantum_circuit_builder.py
Installing qiskit-aer...
Running in test mode..
Traceback (most recent call last):
  File "~/lib/python3.12/site-packages/qiskit/transpiler/passmanager.py", line 464, in wrapper
    return meth(*meth_args, **meth_kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/transpiler/passmanager.py", line 226, in run
    return super().run(
           ^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/passmanager/passmanager.py", line 232, in run
    _run_workflow(program=program, pass_manager=self, callback=callback, **kwargs)
  File "~/lib/python3.12/site-packages/qiskit/passmanager/passmanager.py", line 292, in _run_workflow
    passmanager_ir, final_state = flow_controller.execute(
                                  ^^^^^^^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/passmanager/base_tasks.py", line 218, in execute
    passmanager_ir, state = next_task.execute(
                            ^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/passmanager/base_tasks.py", line 218, in execute
    passmanager_ir, state = next_task.execute(
                            ^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/passmanager/base_tasks.py", line 218, in execute
    passmanager_ir, state = next_task.execute(
                            ^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/transpiler/basepasses.py", line 195, in execute
    new_dag, state = super().execute(
                     ^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/passmanager/base_tasks.py", line 98, in execute
    ret = self.run(passmanager_ir)
          ^^^^^^^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/transpiler/passes/layout/apply_layout.py", line 56, in run
    raise TranspilerError("The 'layout' must be full (with ancilla).")
qiskit.transpiler.exceptions.TranspilerError: "The 'layout' must be full (with ancilla)."

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "~/quantum_circuit_builder.py", line 2002, in <module>
    main()
  File "~/quantum_circuit_builder.py", line 1953, in main
    try_angles_qiskit()
  File "~/quantum_circuit_builder.py", line 1876, in try_angles_qiskit
    transpiled_qc = transpile(qc_test, simulator, initial_layout=layout)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/compiler/transpiler.py", line 391, in transpile
    out_circuits = pm.run(circuits, callback=callback, num_processes=num_processes)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/transpiler/passmanager.py", line 441, in run
    return super().run(circuits, output_name, callback, num_processes=num_processes)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "~/lib/python3.12/site-packages/qiskit/transpiler/passmanager.py", line 466, in wrapper
    raise TranspilerError(ex.message) from ex
qiskit.transpiler.exceptions.TranspilerError: "The 'layout' must be full (with ancilla)."

How can we reproduce the issue?
from qiskit import QuantumCircuit, transpile, Aer
from qiskit.transpiler import Layout

qc_test = QuantumCircuit(2, 2)  # 2 qubits, 2 classical bits
qc_test.h(0)
qc_test.cx(0, 1)
qc_test.measure_all()

simulator = Aer.get_backend('qasm_simulator')
layout = Layout({qc_test.qubits[0]: -1, qc_test.qubits[1]:<PHONE_NUMBER>})  # Extreme indices
transpiled_qc = transpile(qc_test, simulator, initial_layout=layout)

What should happen?

Clear error messages should identify the problem with the layout. For example: "Physical qubit index -1 is invalid; indices must be non-negative and within the backend's qubit range."
An option to bypass strict validation (for testing purposes) would allow intentional stress testing of the simulator and transpiler.

Any suggestions?

Documentation: Update the initial_layout documentation to clarify that the layout must map all logical qubits and any ancilla for compatibility.
Relax Constraints for Virtual Backends: Consider allowing partial layouts for virtual backends where ancilla qubits are not strictly necessary. For example, automatically handling unmapped qubits or issuing a warning instead of raising an error.

Why would we add an option to pass bad types into the transpiler? This kind of error is exactly protection against things going wrong more mysteriously further down - good library design doesn't have type checking within every single function, it normalises user input as it's encountered, then can assume the invariants that were enforced once, rather than wasting the user's time doing it over and over.
These "unconventional" physical indices are simply incorrect physical indices per Qiskit's data model, which specifies that physical qubit indices in a layout are the integers 0 to n-1. The message can potentially be improved slightly, but the error is correct.
The standard way to pass initial_layout is as a list of integers, which is the virtual mapping - no ancillas are needed in this case. We can potentially expand that to work with Layout too, but I can't remember if there's a reason that's harder.
Fwiw, it's not necessarily good practice for a Python project to optimise for somebody "stress testing" the input types to every function. The allowed types are documented, and typically you should use static code analysis like Mypy to check typing correctness, not expect that every function spends runtime enforcing it - this penalises the happy paths for mistakes that are mostly only made deliberately (as you are), or are obvious as soon as the first exception is raised.

I really appreciate the insight into Qiskit’s design philosophy. It makes sense that the focus is on performance and optimising for the “happy path,” which I’m sure is what the majority of users need.
That said, since we’re a group of academic researchers stress testing the framework (definitely not the “typical user” 😅), we’re deliberately pushing edge cases to test how robustly unexpected inputs are handled.
I also appreciate you mentioning that initial_layout could potentially be expanded to work better with Layout and that the error message could be improved slightly. Those sound like great steps to make the experience even smoother.
By the way, could you point us to the documentation you mentioned about the allowed types for initial_layout? It’d be super helpful for guiding our future tests.
Thanks again for all your time and effort - it’s clear a lot of thought goes into making Qiskit work so well, and these discussions help us better understand the design choices.

Exploring hyperparameters is a common practice in software engineering methods, not only in the context of reliability and security but also for methods like search-based optimization and genetic algorithms. Consequently, knowing sensible values for these parameters is crucial for automating optimization processes.
In this case, it would be helpful if documentation clarified the following:

"-1" is invalid: we cannot use negative indices because they don’t correspond to any physical qubit.
Large indices (e.g., 100000000): These are fine for the simulator because, unlike the hardware, it doesn't impose strict limits on the number of qubits. However, it will be good to know the maximal number tested, or what the spec is expecting as a maximal value the quantum platform can handle.

Additionally, support for a wider range of "good values," even those less commonly used, will be super useful for such methods.

When you're deliberately pushing incorrect types, it's worth considering whether these are things that are actually likely to occur by mistake, or whether they're only going to happen if somebody does it deliberately very wrong, and what it would cost to insert checks in those situations.
Python as a language is almost always used via an interpreter, and the typing is highly dynamic.  This means it's always going to be possible to pass something bad to a function, whereas in a static and strict type system, a compiler might reject it from even building.  We don't have that luxury in Python, so any time we want to do a type check, we have to trade off the fact that it needs to happen at run-time, not compile-time, so it penalises all users, even ones who are following the documentation.  We typically make this trade-off by ensuring that high-level user-facing functions normalise and type check inputs (directly with isinstance or indirectly by regular Python duck-typing assumptions), and then let the lower-level functions all assume that the data is in valid formats, and undergo undefined behaviour if it isn't.  For top performance, we may also choose to expose lower-level functions directly to users, with caveats that the user needs to be more vigilant, because there are far fewer guardrails.

Yes, you are entirely free to explore changing the values you pass in and auto-optimising those, just like lots of successful projects have done using Qiskit.
The maximum number of qubits reported by the simulator is set by the simulator.  The example above instantiated a CPU-bound, RAM-based statevector simulator, and will have determined the maximum number of qubits based on those constraints.  If you'd instantiated a different simulator, the constraints would have been different, because it would have been a completely different device.  You can see what number of qubits the backed supports by looking at the BackendV2.num_qubits property (simulator.num_qubits in the example above).  On my machine, with my amount of RAM, that simulator reports having 31 qubits.  It might be a little different on your machine, if you've got more/less RAM than I do.  The simulator you're using is not a part of Qiskit SDK (this repo), and the Qiskit compiler doesn't treat simulators specially - to us, a simulator is just like any other backend we're targetting, and it's up to the simulator to provide a hardware-like description of itself and what it has available.
If you used, for example, qiskit_ibm_runtime.fake_provider.FakeTorino as your backend instead, it would report having 127 qubits, like the real hardware device has.  If you choose to make your own custom Target, you can put any number of qubits in there, if you define the device topology and available instructions, etc.  I know that people have successfully used Qiskit to compile to (imaginary) 10,000+ qubit devices - you just need to have a backend descriptor (a Target) to tell the compiler about it.
Beyond that, your inputs to transpile need to be logically consistent with each other.  If your backend only has 31 qubits, a layout that attempts to map the 10,000,000th is not a valid input - the allowed sets of values to the tranpsile function are interdependent, just like how you can't ask a QuantumCircuit to put an X gate on virtual qubit 10,000,000 if you said your circuit was only going to use 31 qubits.
The transpiler is entirely correct to raise an error in the example given at the top.  The only real troubles that I see are that the error message could be clearer and have happened sooner in the pipeline, and that we don't have a huge amount of centralised documentation about the common data representations of the transpiler.  Both of those can be improved.

Thanks for the detailed explanation! I think adding a check much earlier sounds like a good solution.
More specific documentation is super useful especially since the analysis of these documents commonly is done by Gen-AI tools, which might miss the context you added here. In the end, we are trying to build models that can sensibly explore these systems.

@vili-1 @karineek I'm wondering, can you please elaborate more on the use-case you are after here? Is the end goal a specific quantum problem you are trying to solve with Qiskit (and IBM Quantum) or is it more of a generic testing/fuzzing/automation story?

@eliarbel Our short-term goal is to test and validate the quantum simulators. In the long term (and under the hood), we are developing a robust, publishable model for researchers to use in the broader scientific context. More concise documentation can help to automate at least part of the process of building the model.
You can see the public project page: https://github.com/karineek/QuantumSimulatorValidatorFuzz.
We can give further details once the current paper is published.


Disable XFA tests because of timeouts with Firefox (issue 17311)

/botio test

Looking at the logs above it's not just the reference tests that are "broken" now, but it seems that we also get a few new unit-test failures as well :-(

I filed a bug for Firefox:
https://bugzilla.mozilla.org/show_bug.cgi?id=1865917

/botio test

@Snuffleupagus, in waiting for bug 1865774 to be fixed, I removed nightly 122.0a1 from the bots and replace them by 121.0a1 from the 13th of November (https://archive.mozilla.org/pub/firefox/nightly/2023/11/2023-11-13-09-15-05-mozilla-central/). It's the best I can do for the moment.


Rename Sonar projects

All sonar projects should have the same naming pattern like:
"JavaScript client API of the Dolphin Platform (dolphin-platform-javascript)"
"Description (technical name)"

This is done...hence closing.


Provide a minified version

Hey,
I was looking for a diacritics version on bower, and found only yours (and an angular version but it's for a non-angular website).
Could you provide a minified version? thanks

Hi,
I'm not sure it is
https://marijnhaverbeke.nl/uglifyjs?code_url=https://raw.githubusercontent.com/Nijikokun/Diacritics.js/master/diacritics.js

I'm not sure it is relevant to commit a minified file, but you can easily create one yourself.
For example: https://marijnhaverbeke.nl/uglifyjs?code_url=https://raw.githubusercontent.com/Nijikokun/Diacritics.js/master/diacritics.js

I know but usually libraries come with a minified version otherwise people have to minify it every time there's an update
anyway, just a thought

:+1: Will do this in the coming days.


Set latest kubevirtci provider to include k8s-1.14.6

Update hack/config-default.sh to include new kubevirtci sha,
run 'make generate' to include changes for k8s-1.14.6

What this PR does / why we need it:
Add support for k8s-1.14.4 as kubevirt CI environment.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

Add support for k8s-1.14.4 as kubevirt CI environment

let's see if it works
/test pull-kubevirt-e2e-k8s-1.14.6

Ran 376 of 453 Specs in 13236.005 seconds
SUCCESS! -- 376 Passed | 0 Failed | 1 Pending | 76 Skipped

:+1:
failures on other lanes are unrelated

@dhiller just merged PR in kubevirtci, please update sha to the merge commit (and run make generate again)

/test pull-kubevirt-e2e-k8s-1.14.6

Travis is green, will merge, the image didn't change :)
Nice job @dhiller  :+1:
last step:

please remove the k8s 1.11 job from prow, it's disabled for a while now, nobody will need it anymore
please make the 1.13 job optional (no clue how pending PRs react when we directly delete it)
and make the new 1.14 job mandatory :slightly_smiling_face:


Fix #1373 Support the init scope in some VolumeChoosers

When multiple volumes are used, the accumulo init step can randomly create the system tables in any of the volumes. This change introduces the init property in the volume chooser & let the user configure in which volume the system tables like metadata & root tables are to be created.
For ex: general.custom.volume.preferred.init=hdfs://<namenode>/accumulo creates the system tables in HDFS during accumulo init.
Additionally it was observed, after the init step, starting accumulo creates metadata and root tables on the default volume with rfiles. Trying to fix this with config -t accumulo.metadata -s table.custom.volume.preferred=hdfs://<namenode>/accumulo and compact -w did not help. May be something to discuss separately.

Thanks for the PR! The changes look really good. I only have very minor comments/suggestions. The test changes are also really nice.

Thanks for reviewing the PR @ctubbsii @keith-turner. Based on the comments, I worked with @billierinaldi & committed the new changes to the PR. Let us know for any further comments/questions?

Looks really good! Awesome, @karthick-rn !

Thanks @ctubbsii

I'm just kicking off the ITs with this change, in my Jenkins instance, to be sure there aren't any surprises in our testing suite, but then I can merge after that.
 I thought you had contributed to Accumulo before (perhaps accumulo-testing?), but if not:
If you wish to be added as a contributor to https://accumulo.apache.org/people/ , please open a pull request to add yourself at https://github.com/apache/accumulo-website/edit/master/pages/people.md
If you haven't already, please consider subscribing to our developer mailing list (https://accumulo.apache.org/contact-us/) and introducing yourself. :smiley_cat:

I'm just kicking off the ITs with this change, in my Jenkins instance, to be sure there aren't any surprises in our testing suite, but then I can merge after that.
 I thought you had contributed to Accumulo before (perhaps accumulo-testing?), but if not:
If you wish to be added as a contributor to https://accumulo.apache.org/people/ , please open a pull request to add yourself at https://github.com/apache/accumulo-website/edit/master/pages/people.md
If you haven't already, please consider subscribing to our developer mailing list (https://accumulo.apache.org/contact-us/) and introducing yourself. 😺

Thanks for the update.
GitHub is right there!, this is my first contribution to Accumulo. In the past I had contributed to fluo-muchos & accumulo-website.

I ran the ITs, and it looks like VolumeChooserIT has some tests failing with these changes. It's probably some no-longer-valid assumptions some of the old tests were making during the Initialize phase.
You should be able to reproduce the failures with:
mvn clean verify -Dtest=blah -Dit.test=VolumeChooserIT -Dcheckstyle.skip -Dspotbugs.skip
@karthick-rn Are you able to take a look at those tests and include a fix with this PR?

I ran the ITs, and it looks like VolumeChooserIT has some tests failing with these changes. It's probably some no-longer-valid assumptions some of the old tests were making during the Initialize phase.
You should be able to reproduce the failures with:
mvn clean verify -Dtest=blah -Dit.test=VolumeChooserIT -Dcheckstyle.skip -Dspotbugs.skip
@karthick-rn Are you able to take a look at those tests and include a fix with this PR?

Sure! @ctubbsii

I ran the ITs, and it looks like VolumeChooserIT has some tests failing with these changes. It's probably some no-longer-valid assumptions some of the old tests were making during the Initialize phase.
You should be able to reproduce the failures with:
mvn clean verify -Dtest=blah -Dit.test=VolumeChooserIT -Dcheckstyle.skip -Dspotbugs.skip
@karthick-rn Are you able to take a look at those tests and include a fix with this PR?

Sure! @ctubbsii

Hi @ctubbsii, Sorry for the delay in getting back on this. I worked with @billierinaldi to get through the failing tests and pushed the new changes to this PR. Thanks @billierinaldi for your help!
We also noticed the test phase of maven sometimes times out with the below error
`[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.accumulo.test.VolumeChooserIT
[ERROR] Tests run: 5, Failures: 0, Errors: 5, Skipped: 0, Time elapsed: 300.273 s <<< FAILURE! - in org.apache.accumulo.test.VolumeChooserIT
[ERROR] waLogsSentToConfiguredVolumes(org.apache.accumulo.test.VolumeChooserIT)  Time elapsed: 60.014 s  <<< ERROR!
org.junit.runners.model.TestTimedOutException: test timed out after 60 seconds
at app//org.apache.accumulo.test.VolumeChooserIT.waLogsSentToConfiguredVolumes(VolumeChooserIT.java:349)
[ERROR] twoTablesPreferredVolumeChooser(org.apache.accumulo.test.VolumeChooserIT)  Time elapsed: 60.006 s  <<< ERROR!
org.junit.runners.model.TestTimedOutException: test timed out after 60 seconds
at app//org.apache.accumulo.test.VolumeChooserIT.configureNamespace(VolumeChooserIT.java:221)
at app//org.apache.accumulo.test.VolumeChooserIT.twoTablesPreferredVolumeChooser(VolumeChooserIT.java:279)
[ERROR] twoTablesDiffChoosers(org.apache.accumulo.test.VolumeChooserIT)  Time elapsed: 60.005 s  <<< ERROR!
org.junit.runners.model.TestTimedOutException: test timed out after 60 seconds
at app//org.apache.accumulo.test.VolumeChooserIT.createAndVerify(VolumeChooserIT.java:306)
at app//org.apache.accumulo.test.VolumeChooserIT.twoTablesDiffChoosers(VolumeChooserIT.java:323)
[ERROR] includeSpecialVolumeForTable(org.apache.accumulo.test.VolumeChooserIT)  Time elapsed: 60.003 s  <<< ERROR!
org.junit.runners.model.TestTimedOutException: test timed out after 60 seconds
at app//org.apache.accumulo.test.VolumeChooserIT.configureNamespace(VolumeChooserIT.java:221)
at app//org.apache.accumulo.test.VolumeChooserIT.includeSpecialVolumeForTable(VolumeChooserIT.java:337)
[ERROR] twoTablesRandomVolumeChooser(org.apache.accumulo.test.VolumeChooserIT)  Time elapsed: 60.007 s  <<< ERROR!
org.junit.runners.model.TestTimedOutException: test timed out after 60 seconds
at app//org.apache.accumulo.test.VolumeChooserIT.createAndVerify(VolumeChooserIT.java:306)
at app//org.apache.accumulo.test.VolumeChooserIT.twoTablesRandomVolumeChooser(VolumeChooserIT.java:299)
[INFO]
[INFO] Results:
[INFO]
[ERROR] Errors:
[ERROR]   VolumeChooserIT.includeSpecialVolumeForTable:337->configureNamespace:221 » TestTimedOut
[ERROR]   VolumeChooserIT.twoTablesDiffChoosers:323->createAndVerify:306 » TestTimedOut ...
[ERROR]   VolumeChooserIT.twoTablesPreferredVolumeChooser:279->configureNamespace:221 » TestTimedOut
[ERROR]   VolumeChooserIT.twoTablesRandomVolumeChooser:299->createAndVerify:306 » TestTimedOut
[ERROR]   VolumeChooserIT.waLogsSentToConfiguredVolumes:349 » TestTimedOut test timed ou...
[INFO]
[ERROR] Tests run: 5, Failures: 0, Errors: 5, Skipped: 0`

@ctubbsii After fixing the first error in VolumeChooserIT (which was that ClientContext was always reading the instance ID from ZooKeeper), we saw another error from PerTableVolumeChooser that was due to no volume chooser being configured for the init or default scope. Previously, the user could just configure a volume chooser for the logger scope, since that was the only scope where we were constructing a volume chooser. Since introducing volume choosers for the init scope would require the user to change their config, Karthick's latest patch falls back to a random chooser when the neither the desired nor the default scope are specified. This preserves the old behavior for the init scope, but changes the behavior for the logger scope (which previously threw an error instead of falling back to random).

Karthick's latest patch falls back to a random chooser when the neither the desired nor the default scope are specified.

@billierinaldi the comments on #253 discuss reasons for avoiding falling back to the random chooser.  I am not sure if those reasons apply to these changes though, just wanted to make you aware of those.

Thanks, @keith-turner. I did see that issue, which was why I wanted to explicitly discuss the desired behavior. My impression from reading over the comments was that it was decided that misconfiguration should definitely throw an error, but I was less clear on whether it is important that absent configuration should throw an error. I suppose there are three possibilities:

require a new configuration property when using the preferred and per table volume choosers (general.custom.volume.preferred.default or general.custom.volume.preferred.init must be specified)
fall back to the random chooser instead of throwing an exception when scope-specific and default properties are not specified (absent configuration no longer throws an error for the logger scope, implemented in the current patch)
handle absent configuration differently for the logger and init scopes, so that init falls back to random while logger throws an exception (preserving existing behavior for both scopes, while allowing new configurability for the init scope).

My order of preference is 2, 3, 1, but I am open to any of these and to other suggestions.

Based on the intent of the previous issue, only option 1 seems acceptable to me, as it is the only one that properly protects against partial / improper configuration. Any fall-back to a random chooser can place data improperly on a system, that is very expensive to correct later. In general, I think it's a good idea to require explicit configuration from a user, rather than the system to try to infer user's intentions from incomplete or incorrect configuration.
The behavior I strongly prefer: the default config should work out of the box, without any special configuration (it does, as the default is the RandomVolumeChooser), but if the other choosers are used, they must be fully configured, with explicit configuration for any relevant scope they will be executed for. Here, I think this means that during system initialization, the corresponding init scope configuration property, or the default scope property, must be set.

@ctubbsii Are there semver implications to changing how the PerTableVolumeChooser and PreferredVolumeChooser need to be configured?

@ctubbsii Are there semver implications to changing how the PerTableVolumeChooser and PreferredVolumeChooser need to be configured?

I don't think we consider our configuration to be "public API" in the same way that our Java API is "public API" for the purposes of semver. The "custom" props that we support for various implementation-specific classes of server-side plugin mechanisms even less so (they are more like API for that specific plugin, rather than "Accumulo" API), but it's all still user-facing, so we should be careful. What did you have in mind?

The new commit implements option 1, the VolumeChooserIT passed for @billierinaldi however the same resulted in time out for me. It would be good if someone else could verify the IT passes as well. Thanks

The new commit implements option 1, the VolumeChooserIT passed for @billierinaldi however the same resulted in time out for me. It would be good if someone else could verify the IT passes as well. Thanks

I'll try to run them and report back here.
By the way, the timeouts we set on our ITs are often arbitrary... as a last-ditch effort to not let a stuck test run forever. For most of these ITs, the actual run time depends greatly on the hardware you're using, and are much more likely to time out on a VM or laptop than they are on a dedicated bare metal server used for CI. Most of the ITs can flex their timeout by passing -Dtimeout.factor=2 (or 3, or whatever) to the mvn command used to build them, but some of them have hard-coded timeouts that can't be changed. If you're using reasonably good hardware and it still times out with the default timeout factor of 1, then it might indicate a need to adjust the default timeout. But, I would recommend trying to increase the timeout factor first.

The new commit implements option 1, the VolumeChooserIT passed for @billierinaldi however the same resulted in time out for me. It would be good if someone else could verify the IT passes as well. Thanks

I'll try to run them and report back here.
By the way, the timeouts we set on our ITs are often arbitrary... as a last-ditch effort to not let a stuck test run forever. For most of these ITs, the actual run time depends greatly on the hardware you're using, and are much more likely to time out on a VM or laptop than they are on a dedicated bare metal server used for CI. Most of the ITs can flex their timeout by passing -Dtimeout.factor=2 (or 3, or whatever) to the mvn command used to build them, but some of them have hard-coded timeouts that can't be changed. If you're using reasonably good hardware and it still times out with the default timeout factor of 1, then it might indicate a need to adjust the default timeout. But, I would recommend trying to increase the timeout factor first.

I have already tried increasing the timeout to -Dtimeout.factor=3 that @billierinaldi provided upon noticing the timeout earlier, however in my case it doesn't seem to fix. I'm running the mvn on my mac, could be there is a subtle difference on the hardware that might be causing this.

@karthick-rn  I checked out your branch locally and ran VolumeChooserIT.  I did not hang for me.

@karthick-rn I checked out your branch locally and ran VolumeChooserIT a few times. I did not hang for me.

Thanks @keith-turner for checking this. As I'm the only one facing the timeout, it should be something down to my machine. Let me know if you have any further comments on the PR?

My IT run aged out of my CI server over the December holiday season... I'll re-run to verify. I expect it to pass, but will confirm tomorrow and merge if there are no issues.

The timeout issue that I was facing when running maven disappeared after increasing the maven heap (export MAVEN_OPTS="-Xmx4G"). Thanks @billierinaldi for the suggestion.

@ctubbsii are you finished reviewing this?  I can merge it if you are done.

No. My CI server got stuck on some old builds, and the one I had queued to test this change never ran. I rebooted it. It should be done today, and I can get this merged before the weekend is over.

All build tests passed, except for two flaky tests that are unrelated to this. Going to merge now.


IndentationError error in 3.9.5

Describe the bug
When running ofscraper there is an indentation error
$ ofscraper -l DEBUG
Traceback (most recent call last):
  File "/home/user/.local/bin/ofscraper", line 5, in <module>
    from ofscraper.__main__ import main
  File "/home/user/.local/lib/python3.11/site-packages/ofscraper/__main__.py", line 4, in <module>
    import ofscraper.runner.load as load
  File "/home/user/.local/lib/python3.11/site-packages/ofscraper/runner/load.py", line 6, in <module>
    import ofscraper.runner.run as run
  File "/home/user/.local/lib/python3.11/site-packages/ofscraper/runner/run.py", line 5, in <module>
    import ofscraper.commands.picker as picker
  File "/home/user/.local/lib/python3.11/site-packages/ofscraper/commands/picker.py", line 1, in <module>
    import ofscraper.commands.check as check
  File "/home/user/.local/lib/python3.11/site-packages/ofscraper/commands/check.py", line 12, in <module>
    import ofscraper.api.archive as archived
  File "/home/user/.local/lib/python3.11/site-packages/ofscraper/api/archive.py", line 5
    _______  _______         _______  _______  _______  _______  _______  _______  _______
IndentationError: unexpected indent

To Reproduce
Install 3.9.5, run ofscraper -l DEBUG
Expected behavior
ofscraper startup
System Info

OS: Ubuntu 22.04
Browser Chrome
Version 3.9.5
python 3.11

Same here.
OS: Win10, x64
Python: 3.12

+1

same here in docker container

+1

+3

Sneed.

same here. python 3.11.2, debian 12 lxc container.

Seems to be fixed with 3.9.6!

can confirm, the upgrade to 3.9.6 fixed it for me as well.


fatal error: runtime: bsdthread_register error

Environment:
OS X El Capitan
CommandLineTools version 7
Go version 1.7
Protobuf c++ tar 2.4.1
go get -u -v github.com/golang/protobuf/proto
go get -u -v github.com/golang/protobuf/protoc-gen-go
Everything works fine in OS X Yosemite
protoc --go_out=. test.proto
fatal error: runtime: bsdthread_register error
runtime stack:
runtime.throw(0x11ab0ca, 0x21)
/usr/local/go/src/runtime/panic.go:596 +0x95 fp=0x7fff5fbffa80 sp=0x7fff5fbffa60
runtime.goenvs()
/usr/local/go/src/runtime/os_darwin.go:108 +0xa0 fp=0x7fff5fbffab0 sp=0x7fff5fbffa80
runtime.schedinit()
/usr/local/go/src/runtime/proc.go:486 +0xa1 fp=0x7fff5fbffaf0 sp=0x7fff5fbffab0
runtime.rt0_go(0x7fff5fbffb20, 0x1, 0x7fff5fbffb20, 0x0, 0x1, 0x7fff5fbffc58, 0x0, 0x7fff5fbffc66, 0x7fff5fbffc82, 0x7fff5fbffc92, ...)
/usr/local/go/src/runtime/asm_amd64.s:158 +0x183 fp=0x7fff5fbffaf8 sp=0x7fff5fbffaf0
--go_out: protoc-gen-go: Plugin failed with status code 2.

I don't have a Mac machine with that environment to repro this issue. Does this still occur when you use Go1.8.1?
protoc-gen-go doesn't use any unsafe, so I highly suspect that this is not an issue specific to protoc-gen-go, but possibly the Go runtime. I remember OSX having a history of introducing breaking changes that the Go toolchain is not happy with.

Closing as stale. If this is still happening, feel free to re-file the issue or re-open. Thanks.


Added exception when replanner remove_{action/goal} fails

Now the integrated meta-engine replanner does not silently fail when a remove_action or remove_goal does not change the problem.

Thinking about this feature, we could insert a flag that determines if the user wants an exception when the method fails (default behavior) or he prefers a silent fail.

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.01% :tada:

Comparison is base (6d3cbec) 85.03% compared to head (d76209e) 85.05%.

:exclamation: Current head d76209e differs from pull request most recent head d4c1db0. Consider uploading reports for the commit d4c1db0 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #474      +/-   ##
==========================================
+ Coverage   85.03%   85.05%   +0.01%
==========================================
  Files         198      198
  Lines       26019    26035      +16
==========================================
+ Hits        22125    22143      +18
+ Misses       3894     3892       -2

Files Changed
Coverage Δ

unified_planning/engines/replanner.py
98.52% <100.00%> (+3.61%)
:arrow_up:

unified_planning/test/test_replanner.py
100.00% <100.00%> (ø)

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


Question on 1-st training

Hi!
https://github.com/SxJyJay/MSMDFusion/blame/7b5b2741e693ba8007c95e3e8951e4e67fbc47ed/configs/transfusion_nusc_voxel_L.py#L244-L244C10

In this line, you mention that the value of samples_per_gpu is 2, but in the previous part this value is set to 4. I am a little confused by this.
When I set the workers_per_gpu to 0 as the code does, I find that the utilization rate of my gpus (8 RTX  4090) is low and it must spend 14 days to train the model, is that nomal?
Due to the second problem, I set workers_per_gpu to 16 to make better use of my gpus. Besides, the value of samples_per_gpu is 4. After training,I get the following result:
pts_bbox_NuScenes/NDS: 0.6944, pts_bbox_NuScenes/mAP: 0.6464
which is a little bit lower than yours. Is this normal？If not, is that because of the change of workers_per_gpu?

Hope for yous reply!
Best

I remember that setting samples_per_gpu to 2 can probably improve the performance a little bit. And workers_per_gpu determines the number of workers during fetching data, thus it does not affect model performances.

I remember that setting samples_per_gpu to 2 can probably improve the performance a little bit. And workers_per_gpu determines the number of workers during fetching data, thus it does not affect model performances.

Got it! Thanks a lot!


Playback stuck

When trying to play flv from remote server, first this message appears in console:
flv.min.js:3 Fetch failed loading: GET <video url>
Then, at certain point playback freezes and player stops responding to any controls.
This doesn't occur with smaller files.

check http request / response

@xqq Thanks for your reply. Here's request/response screenshot. It shows no errors, and when I send same request with my rest client it is fine too. CORS are enabled on server.


[TC-LTIME-1.1, 1.2] Time Format Localization Cluster--Feature-map value mismatch.

Problem:
As per test plan [TC-LTIME-1.1, 1.2] FeatureMap attribute showing value mismatch error when executed, DUT should
responds with feature map value 1 but currently DUT responding feature map value has 0.
Expected behavior:- DUT should responds the feature map value as 1
Actual behavior:- DUT is responding the feature map value as 0
Steps to reproduce:- ./chip-tool timeformatlocalization read feature-map 1 0
System configuration
Chip-tool - RPI-4, 8GB RAM
DUT - RPI - RPI-4, 8GB RAM
Platform : Onnetwork
Commit-ID:  22de5a04f7c3f3d10bc06ad20bf50823c42cfbed
Dut application: all-clusters-app
Test-Plan Reference:-
https://github.com/CHIP-Specifications/chip-test-plans/blob/master/src/cluster/timeformatlocalization.adoc#tc-ltime-1-1-global-attributes-dut-as-client
Spec Reference:-
https://github.com/CHIP-Specifications/connectedhomeip-spec/blob/master/src/service_device_management/LocalizationTimeFormat.adoc#TimeFormatLocalizationCluster
Log:
LTIME.txt

Expected behavior:- DUT should responds the feature map value as 1

@Survensa Why?  Test plan says, for the expected behavior:

bit 0 is set to 1 If CALFMT is true.

Is CALFMT true in this case?

@bzbarsky-apple CALFMT is set to true because the attributes "ActiveCalendarType" and "SupportedCalendarTypes" depending on the feature CALFMT - CalendarFormat is implemented so I believe that it should be listed in featuremap.


Prepare 0.2.0

PR adds:

Attrs expanded by impl_scope now require an explicit grouping token, just like #[proc_macro_attribute]
impl_default now supports a where clause

Changelog for 0.2.0:

Added impl_scope!
Added #[impl_default]

Test failed: cargo clippy --all -- -D warnings -A unknown_lints — due to unused fn. This is a base lint, not a Clippy lint.
Anyway, the unused function is known and to be resolved later (it will likely become part of the public API), so I'll ignore the failure for now.


[Bug] [A11y] Contrast on Episode Cards in most themes

The description text for an episode is very difficult to read in all themes except light mode. In certain modes (dark and Sentry) the title can also be difficult to read.
There should either be a backing behind the text or the card should be formatted such that the text will always render below the episode number.

Thanks a ton. there was a background color the text, but it seems to not be applied in light mode.

Hey @wesbos  @therealphatmike  any thoughts on utilising the ch unit here for the line length of the description in the card? Baymard did research around the optimal line length. Andy bell employs the ch unit in a similar way on his set.studio blog cards, only with a max 50ch width.
Before: Spanning full width of the card.

After: This example uses max-width: 65ch.

@garyb1 It's not a bad suggestion, it just may go against the intended design ethos. The interplay of the text and the large episode number in the background may be an intentional aesthetic that they don't want to lose--I personally feel the site loses a bit of it's "grunge" factor by making the separations too neat.
It's worth noting that this has implication on both the episode title and the description text.
Here's an example of an episode where the title flows onto the episode number as well.

With a background color on the text, we get something akin to

With the max-width we get

In my opinion, the max-width doesn't feel like it's a component for the same site. Obviously, Scott and Wes have the last word on that, but I figured I'd throw in my two cents.
As far as providing a bg color on the description text, again I've come to the same conclusion

bg color applied:

max width applied;

Again, I feel it just loses...something.


Trains are causing huge lagg somehow and my friend always gets kicked because of a network error

Describe the Bug
Hey wolveix, first of all thank you so much for your great work here. Everything is working great except Trains. My friend says that he can only drive about 100km/h before the Server starts to lagg. That's pretty confusing because my Server has 4 CPU Cores with 4.2GHz each and 16GB of RAM, so it shouldn't normally lagg. I have allocated 6GB to the Docker container, but I think it's only using about 3-4. I don't know that much about satisfactory, because I'm only hosting the server for him, but it would be great if you could help us out :)
He also gets this Error when he's trying to join: https://cdn.felo.gg/WhatsApp Bild 2024-05-18 um 11.26.46_64b8c4d9.jpg
Your Runtime Command or Docker Compose File
Default
System Specs (please complete the following information):
If you're on Linux, just paste the following block as a single command, and paste the output here.
===== START ISSUE REPORT =====
OS:  Linux rs-zap466398-2 6.1.0-20-amd64 #1 SMP PREEMPT_DYNAMIC Debian 6.1.85-1 (2024-04-11) x86_64 GNU/Linux
CPU: Intel(R) Xeon(R) CPU E5-2667 v2 @ 3.30GHz
BIOS pc-i440fx-8.1  CPU @ 2.0GHz
RAM: 2GB/15GB
HDD: 56GB/473GB (13% used)
===== END ISSUE REPORT =====

Update for the network error:
"I join while the server is running and I get in. Apparently, starting while I'm joining is a problem" - my Friend

Update for the Train issue:
He always gets kicked when trying to drive with the Train

Here is the Error that gets displayed when he is being kicked:
Error: UEngine::BroadcastNetworkFailure: FailureType = ConnectionTimeout, ErrorString = UNetConnection::Tick: Connection TIMED OUT. Closing connection..  Closing connection..  Closing connection..  Thanks for your kind words. These are game issues, and not issues with this Docker image. Nevertheless, the information below may help.
Try setting SERVERSTREAMING=false (this will drastically increase the server's RAM usage). Also try increasing TIMEOUT=60 or to a higher value.

Hey, thanks for helping me. I'll try out these settings later today and will let you know if that helps :)

Okay that did sadly not work at all. Setting it to SERVERSTREAMING=false completely overheated my server and using TIMEOUT=60 didnt change anything :/
Is there anything else I can try?

@Felitendo after looking up your CPU, I don't believe it's really sufficiently powerful to host a Satisfactory server. Game servers, particularly Satisfactory, are very single-core heavy. This would explain the poor performance you're experiencing, particularly given that you were unable to host the server with SERVERSTREAMING=false.


Swiper 10.2.0 run time type issues for SwiperContainer and SwiperOptions

Check that this is really a bug

[X] I confirm

Reproduction link
https://stackblitz.com/edit/swiper-angular-example-wjegdv?file=src%2Fapp%2Fapp.component.ts,src%2Fapp%2Fapp.module.ts
Bug description
Getting runtime type error when upgrading from swiper 9 to swiper 10.2.0

SwiperContainer interface shows type error for pagination, scrollbar, autoplay
Property 'pagination' does not exist on type 'SwiperContainer'.
The Doc says to import SwiperOptions from 'swiper/types' but it doesn't work
https://swiperjs.com/swiper-api#typescript-definitions
Module '"../../../node_modules/swiper/types"' has no exported member 'SwiperOptions'.

Instead, I have to import it from
import { SwiperOptions } from 'swiper/types/swiper-options';
to make it work.
Working example with swiper 9.4.1
https://stackblitz.com/edit/swiper-angular-example-fkzr93?file=src%2Fapp%2Fapp.component.ts,src%2Fapp%2Fapp.module.ts
Expected Behavior

SwiperContainer should not show run-time type error
Import SwiperOptions from 'swiper/types' should work as per Doc

Actual Behavior

SwiperContainer interface shows type error for pagination, scrollbar, autoplay
Property 'pagination' does not exist on type 'SwiperContainer'.
The Doc says to import SwiperOptions from 'swiper/types' but it doesn't work
https://swiperjs.com/swiper-api#typescript-definitions
Module '"../../../node_modules/swiper/types"' has no exported member 'SwiperOptions'.

Instead, I have to import it from
import { SwiperOptions } from 'swiper/types/swiper-options';
Swiper version
10.2.0
Platform/Target and Browser Versions
Chrome is up to date Version 115.0.5790.110 (Official Build) (64-bit)
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

Update to TypeScript v5


fix small typo left over from PR19

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

👍


BitMarket

http://bitmarket.ro/

I know that only adding ERC20 assets is supported here.
Can you tell me where to find documentation for adding BEP20 assets?
Thank you very much! ! !


FIleSystem won't show external files in res://

I am still novice and following step by step tutorials. This bug has rendered me unable to even start a tutorial to be honest. Whatever I import into the file I can't see in Godot unless that file is made inside Godot. No external resources show up.

Please follow the issue template.
Give Godot version, OS and any other details.

Currently this is a feature. A file which is not recognized by the engine is ignored. Those files cannot be used by the engine directly and have to be imported in some way. A lot of files are also imported like .png so they mostly represent named resources which happen to live inside the virtual res:// filesystem used by Godot to map resources in some way.
Do you suggest that there should be an option/switch to show all files?

This bug has rendered me unable to even start a tutorial to be honest.

What part of the tutorial makes it impossible/difficult to follow? You can request to clarify tutorials at godot-docs specifically.

Whatever I import into the file I can't see in Godot unless that file is made inside Godot. No external resources show up.

There's currently a usability issue of having to refocus the Godot editor in order to force the rescan of externally modified files in order for them to be reimported again, just as a hint.
It's also possible to create an arbitrary text file with the script editor. Saving such a file won't show it up in the FileSystem dock either unfortunately.
See also Import plugins.

It was just discussed on discord. The problem was that the start and end folders where godot projects itself, containing a project.godot file. Therefore they are not shown in the FileSystem dock.
I reproduced this in godot 3.2.1

I'd like to see the option to have an unfiltered view of all files in my res:// directory. I'll close the issue now as it was my mistake, not a bug, but a feature.

It was just discussed on discord. The problem was that the start and end folders where godot projects itself, containing a project.godot file. Therefore they are not shown in the FileSystem dock.

Actually I've heard rumors about having nested project support, namely when it comes to plugin development as they are usually created as Godot projects at first, so we'll see how it goes.


The cache is not adequately described.

What happens if you request a service/char with cache mode, but the requested service/char is not present in the cache? Does it fall back to retrieval from the device?

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

ID: 2ef1a204-e7f5-2424-d0d5-a3331de022dc
Version Independent ID: 73aa1eba-bd66-d4fd-ac78-a3ac1d9f00fb
Content: BluetoothCacheMode Enum (Windows.Devices.Bluetooth) - Windows UWP applications
Content Source: winrt-api-build/xml/Windows.Devices.Bluetooth/BluetoothCacheMode.xml
Product: uwp
Technology: winrt-reference
GitHub Login: @stevewhims
Microsoft Alias: stwhi

Hi, @tony-gutierrez, I've reached out for confirmation. I'll update this thread when I have more info.
Thanks!
Steve

Hi, @tony-gutierrez, I've reached out for confirmation. I'll update this thread when I have more info.
Thanks!
Steve

I'm still seeking clarification about this internally. I'll respond when I have all the info.
Thanks again!
Steve

UPDATE: The feature team is talking about this internally. I just pinged the thread to get it moving again. Hope to have an answer soon.
Thanks for your patience.
-Steve

So, I'll go ahead and close this issue for now since we're not yet ready with a response nor with documentation updates. I've asked the feature team to let me know when we're ready for doc updates here, and then I'll be able to come back and make them. You can still respond to the thread while it's closed. Thanks for your patience.
-Steve

Uhhhh... Seems to me that those are all good reasons for the issue to
remain open?
On Tue, Apr 6, 2021, 9:39 PM Steven White @.***> wrote:

Closed #1928 https://github.com/MicrosoftDocs/winrt-api/issues/1928.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/winrt-api/issues/1928#event-4559756125,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADVCNM4SRWJ52PDHOVPAQ3LTHOZVVANCNFSM4WSHNKAA
.

@tony-gutierrez, I've added more details to the Fields section of the topic.
-Steve

Thanks! I am not sure this jives with the behavior I am seeing, but I'll investigate further.
@stevewhims If you have any voice in the UWP Bluetooth world, the scanning timings result in very poor scan performance compared to other platforms. You can get the jist of it here: https://docs.microsoft.com/en-us/answers/questions/39802/windows-ble-scanning-interval-issue.html
Would love to see the ability to change the timings in a future api.

Thanks, @tony-gutierrez!
It looks like the issue you link to there is in the right place. Microsoft Q&A is the right place to lobby for a change or a new feature, to make a suggestion, or to ask a product question. From that thread, I see that the question/issue has made its way to Microsoft engineers, so the channel is working.
Btw, you can visit a topic in the documentation and for questions (or requests for help) regarding a specific scenario related to that topic, you can click Send feedback about > This product, and you'll be taken to Microsoft Q&A.
Threads such as this one (created through Send feedback about > This page, instead of This product) are for feedback about the documentation in the topic. For example, if the documentation in the topic is incorrect, or if info needed to succeed with a common scenario is missing.
Thanks again!
Steve


Remove global window registration of Collection and Model

Remove the documentation which suggests registering Collection and Model as window globals.
Closes #5

Will merge this after #6


Providing different color instead of which I enter as Primary color

I am trying brand new Material Theme Builder i.e.: https://m3.material.io/theme-builder#/custom
I am choosing primary color as red i.e.: #FF0000 and theme get generated by self but my primary color get changed to some different varient of Red color i.e.: #c00100
I am not sure if this is bug or a functionality. but This changes my brand color to different and theme got wrong accordingly.

Hi @hpcreator, this is currently by design to align with the M3 color system.
The Color's hue entered stays the same while the chroma and tone is adjusted to fit with the color system's guidelines and create an accessible color scheme.
The Figma plugin does allow for manual color entry into the style properties that will export, but there are algorithm changes coming soon that allows for input color fidelity.
Please check out M3 color guidance to learn more about dynamic color, also available in video form: https://youtu.be/vv3lOGZrcdk.
For more Android specific guidance and break down of color adjustments: https://developer.android.com/design/ui/mobile/guides/styles/color


Need to add set style feature to Google Layer

We are trying to use Google Map custom styles feature to style the map. It is possible to have from OL google layer since it create map instance using google api.
How can I make changes and contribute here?
Thank you for your interest in making OpenLayers better!
To keep this project manageable for maintainers, we ask you to please check all boxes below before submitting an issue.

[ ] I am submitting a bug or feature request, not a usage question. Go to https://stackoverflow.com/questions/tagged/openlayers for questions.
[ ] I have searched GitHub to see if a similar bug or feature request has already been reported.
[ ] I have verified that the issue is present in the latest version of OpenLayers (see 'LATEST' on https://openlayers.org/).
[ ] If reporting a bug, I have created a CodePen or prepared a stack trace (using the latest version and unminified code, so e.g. ol-debug.js, not ol.js) that shows the issue.

Code snippet on this issue.
OpenLayers.Layer.Google.prototype.setstyle = function (mapLayer) {
this.type = 'styled';
var mapStyle = [];
var styledMapType = new window.google.maps.StyledMapType(mapStyle, { name: "styled" });
var googleMapObj = mapLayer.mapObject;
googleMapObj.mapTypes.set('styled', styledMapType);
googleMapObj.setMapTypeId('styled');
}

Can I contribute on this issue?

Wrong issue tracker, this is for OpenLayers 3+.


Changed linter from standard to standard-ts

To lint the .d.ts files this change is necessary

My doubt is, since the source code is JS, can ts-standard lint these files as well? or should we use both (standard and ts-standard) linters?

I am guessing ts-standard can lint both
From their website

ts-standard is the officially supported variant for TypeScript. ts-standard supports all the same rules and options as standard and includes additional TypeScript specific rules. ts-standard will even lint regular javascript files by setting the configuration in tsconfig.json.

Nice, just we want to wait until https://github.com/standard/ts-standard/issues/209 is resolved ☹

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 93.996%

Totals

Change from base Build<PHONE_NUMBER>:
0.4%

Covered Lines:
318

Relevant Lines:
325

💛  - Coveralls


SessionEventArgs manually disposed

I would like to keep track of the WebSession objects (got from SessionEventArgs) during a particular http dialog.
I keep a reference of them in a list, but unfortunately, the SessionEventArgs are manually disposed in HandleHttpSessionRequest and this by consequence diposes the WebSession as well.
Is there any reason for doing that (calling args.Dispose()) ?
Why don't let he GC do the job ?

Some methods inside WebSession will become dead after the end of connection since those are meant to be called when the session is still active.
See below discussion.
https://github.com/justcoding121/Titanium-Web-Proxy/issues/280
Having said that it should be possible to keep the objects separately using you own data structure. Can you show the code? Take a look at the examples below where we keep the header collection and other details for future reference but not the WebSession itself.
https://github.com/justcoding121/Titanium-Web-Proxy/blob/5517830ea89fca001a42e24e49e033b7fddf887c/Examples/Titanium.Web.Proxy.Examples.Basic/ProxyTestController.cs
https://github.com/justcoding121/Titanium-Web-Proxy/blob/develop/Titanium.Web.Proxy/EventArguments/SessionEventArgs.cs

Unfortunately, that's the WebSession itself which I'd like to keep (more specifically Request and Response fields).
I have commented out all the args.Dispose from HandleHttpSessionRequest and all seems to be ok (I can access the Request/Reponse properties). I understand that once the session is closed, some methods will fail.
See attachment (BTW, would be nice if properties had the ExpandableObjectConverter attributes :-)

)

We may be able to keep the object alive in future, but throw exceptions when accessing methods/functions which are no longer valid.

Could you pleses check my https://github.com/justcoding121/Titanium-Web-Proxy/commit/5d2bf63558e9bef2cd32b29680d6289b5f6e898c commit?
The SessionEventArgs is still disposed, but the WebSession (HttpWebClient) is not.
I made some existing properties to public (get only):
Request.RequestBody and Response.ResponseBody Request.RequestBodyRead and Resposne.ResponseBodyRead
These properties were internal earlier, I've added a check which throws exception when the body is not read, yer or already streamed without storing it.
When the session is finised the body is cleared by default. I've also added 1-1 new properties to keep the body (and body as string): Request.KeepRequestBody and `Response.KeepResponseBody
I'd like to rename some proerties, what do you think about it? Request.RequestBody=>Requet.Body Request.RequestBodyRead=>Request.IsBodyRead Request.KeepRequestBody=>Request.KeepBody`
(and the same for the Respose class)
This would not be a breaking change, since they were internal in the last release.

There are some more properties which could be renamed, but it would be a breaking change:
ResponseStatusCode
ResponseStatusDescription
'ResponseKeepAlive'
ResponseHeaders
ResponseStatus
And the similar properties i nthe Request class.

Hello,
Just curious, why don’t you simply not dispose the SessionEventArgs ?
From: honfika<EMAIL_ADDRESS>Sent: Tuesday, November 14, 2017 1:58 PM
To: justcoding121/Titanium-Web-Proxy
Cc: fabricedupre; Author
Subject: Re: [justcoding121/Titanium-Web-Proxy] SessionEventArgs manually disposed (#326)
Could you pleses check my  https://github.com/justcoding121/Titanium-Web-Proxy/commit/5d2bf63558e9bef2cd32b29680d6289b5f6e898c 5d2bf63 commit?
The SessionEventArgs is still disposed, but the WebSession (HttpWebClient) is not.
I made some existing properties to public (get only):
Request.RequestBody and Response.ResponseBodyRequest.RequestBodyRead and Resposne.ResponseBodyRead
These properties were internal earlier, I've added a check which throws exception when the body is not read, yer or already streamed without storing it.
When the session is finised the body is cleared by default. I've also added 1-1 new properties to keep the body (and body as string): Request.KeepRequestBody and `Response.KeepResponseBody
I'd like to rename some proerties, what do you think about it?Request.RequestBody=>Requet.BodyRequest.RequestBodyRead=>Request.IsBodyReadRequest.KeepRequestBody=>Request.KeepBody`
(and the same for the Respose class)
This would not be a breaking change, since they were internal in the last release.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/justcoding121/Titanium-Web-Proxy/issues/326#issuecomment-344251087 , or mute the thread https://github.com/notifications/unsubscribe-auth/ANORS4OhdV0LjXgiFCqwGqRt77DU5jnxks5s2Y5DgaJpZM4Qbs3A .  https://github.com/notifications/beacon/ANORS-XCB9WZ0eESUkLw8KqBgwVix_zeks5s2Y5DgaJpZM4Qbs3A.gif

I think nobody should save a reference to an EventArgs object (not only in this case). For the user's perspective it is valid only in the event handler.

Makes sense :-)

That makes RequestBody/ResponseBody throw an exception even if it does not have body. What about returning null if HasBody is false ?
Also for RequestBodyString/ResponseBodyString, null could be returned if body is null to prevent an exception being thrown.
From: honfika<EMAIL_ADDRESS>Sent: Tuesday, November 14, 2017 1:58 PM
To: justcoding121/Titanium-Web-Proxy
Cc: fabricedupre; Author
Subject: Re: [justcoding121/Titanium-Web-Proxy] SessionEventArgs manually disposed (#326)
Could you pleses check my  https://github.com/justcoding121/Titanium-Web-Proxy/commit/5d2bf63558e9bef2cd32b29680d6289b5f6e898c 5d2bf63 commit?
The SessionEventArgs is still disposed, but the WebSession (HttpWebClient) is not.
I made some existing properties to public (get only):
Request.RequestBody and Response.ResponseBodyRequest.RequestBodyRead and Resposne.ResponseBodyRead
These properties were internal earlier, I've added a check which throws exception when the body is not read, yer or already streamed without storing it.
When the session is finised the body is cleared by default. I've also added 1-1 new properties to keep the body (and body as string): Request.KeepRequestBody and `Response.KeepResponseBody
I'd like to rename some proerties, what do you think about it?Request.RequestBody=>Requet.BodyRequest.RequestBodyRead=>Request.IsBodyReadRequest.KeepRequestBody=>Request.KeepBody`
(and the same for the Respose class)
This would not be a breaking change, since they were internal in the last release.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/justcoding121/Titanium-Web-Proxy/issues/326#issuecomment-344251087 , or mute the thread https://github.com/notifications/unsubscribe-auth/ANORS4OhdV0LjXgiFCqwGqRt77DU5jnxks5s2Y5DgaJpZM4Qbs3A .  https://github.com/notifications/beacon/ANORS-XCB9WZ0eESUkLw8KqBgwVix_zeks5s2Y5DgaJpZM4Qbs3A.gif

@honfika  I don't mind renaming the properties you mentioned.

I don't mind returning null when HasBody is false. But SessionEventArgs.GetRequestBody/String methods also throws exceptions. I think it should be the same.
However the GetRequestBody throws BodyNotFoundException while Request.RequestBody throws Exception with message "Request body is not read yet. Use SessionEventArgs.GetRequestBody() or SessionEventArgs.GetRequestBodyAsString() method to read the request body.", so it is a bug, it should be the same.
@justcoding121 What do you prefer?

throw BodyNotFoundException as it is implemented now in the SessionEventArgs.Get methods
or return null in this case?

@fabricedupre Why is it a problem if it throws an exception? You can prevent it by checking the HasBody property.

I would throw the exception rather.
Like you said developers can always check for the body and the exception message is self-explanatory. They tried to access the body without knowing that not all HTTP requests/responses have a body.

When displaying the Request/Response in a PropertyGrid control, there is no way to check for the body
PropertyGrid can anyway survive such properties throwing exception, but this is at the cost of performance slowdown.

You can hide the Body property in the PropertyGrid:
https://stackoverflow.com/questions/18926109/c-sharp-net-4-5-propertygrid-how-to-hide-properties
I did not try it, but in this case the getter should not be called.
Displaying a huge body and converting it to a string and displaying that string, too is a performance slowdown, too... so you should hide it. (And I think you don't need them, because it wasn't available earlier)
Or if you want to show the body, I suggest you to wrap the object to another which just returns the "small" properties and the body looks like:
        public byte[] RequestBody
        {
            get
            {
                if (!HasBody)
                {
                    return null;
                }

                var body = webSession.RequestBody;
                if (body.Length > 1000)
                {
                    return body.Take(1000).ToArray();
                }

                return body;
            }
        }

I am more than happy to have the RequestBody (/ResponseBody) marked as non browsable as far as the corresponding string is browsable and does not throw an exception

I think that the corresponting BodyString property will throw an exception, since it just converts the byte[] to string:
public string BodyString => bodyString ?? (bodyString = Encoding.GetString(Body));

That's why I suggested returning a null string in case body is null
Or mark it as non Browsable and let the developper handle the logic ...
Also, still linked to PropertyGrid, would it be possible to mark the HeaderCollection with the attribute [TypeConverter(typeof(ExpandableObjectConverter))] ?

Put it to the class?
    [TypeConverter(typeof(ExpandableObjectConverter))]
    public class HeaderCollection : IEnumerable<HttpHeader>

For me it is ok.

Exactly :-)
And same can be done on class Request / Response

Could you please check this commit?
https://github.com/justcoding121/Titanium-Web-Proxy/commit/250a081ece38f8899f0ac20df2d206c2b92b9648

Works fine :-)

Looks okay.

Then I close this issue.


Delete button not accessible via keyboard

Describe the Bug

The 'delete item ' is not accessible via keyboard if the mouse pointer is not hovering over the respective section
Steps to Reproduce

use the keyboard to create a new execution listener
delete the execution listener (via keyboard navigation

Expected Behavior

delete item is accessible
Environment
Modeler 4.11.1-new-properties-panel
OS: Linux Mint 20.2 Cinnamon

How would the expected behavior look like? The delete action is only visible when hovering over the list item. Shall it be also visible once selected via keyboard?

To fix this, we could show the delete button when any of the actions of the list item is focused (incl. the expand/collapse icon).

Thanks for the report. I can imagine we allow to tab into the button and then it's displayed. The fact that the button is not visible per default should not remove it from the tabbing order.


Preserve Bash History in Multiple Terminal Windows

I consistently have more than one terminal open. Anywhere from two to ten, doing various bits and bobs. Now let's say I restart and open up another set of terminals. Some remember certain things, some forget.
I want a history that:

Remembers everything from every terminal.
Is instantly accessible from every terminal (eg if I ls in one, switch to another already-running terminal and then press up, ls shows up).
Doesn't forget random things if there are spaces at the front of the command.

Taken from this StackExchange question.

+1 this would be great!

To be honest after using this for a while it can get a bit frustrating if you're used to the normal behavior.

Having a large bash history file, what is overhead of rereading file after each command? I hate when I have to wait 1s or more until I can type next command just to execute these commands, git prompt and other things like that.

I haven't had any performance issues since I exclude a lot of commands. Though, the history first updates when you have performed a new command between windows.


imageset: do not use oc adm to get installer and cli images

Instead of using oc adm, load the release image in a container and parse the installer and cli images from the image-references file.

https://issues.redhat.com/browse/CO-843

/lgtm


Add source tracking for FSH entities

Since most of the FSH types can have a direct location in the text, the FSHEntity class is created to serve as a parent to contain that location information. Right now, that's all it has, but if there are any other parsing-related attributes that need to be tracked, they can be placed in FSHEntity.
Tests involving the classes that now track location information are updated in order to satisfy deep equality when comparing with the assert methods.
This pull request addresses https://standardhealthrecord.atlassian.net/browse/CIMPL-126

I've made some updates. Thanks again @cmoesel for the suggestion to make the withLocation function.


foundryup clone for installing supersim

Ideally there is a standard way for installing supersim. We don't need to reinvent the wheel here - `foundryup works great for foundry. The main benefits of this approach over things like homebrew:

Can install any commit, even from forks
Optimized for hackers/contributors - if somebody wants to share a patch, they can give a command that instantly installs it on another machine

This could be called supersimup or something like that

good point! this will be added soon


Problem with 'columnVisibilityButton'!

I tried to use columnVisibilityButton, It show but can't click (disable).
And It have problem when use with titleFn (don't show label title).

titleFn issue is fixed. Not enough info to know what the other issue is.

I will try soon again.


Status issue

I seem to have a issue with the status where it will only show {online}/{max}! now sure how to fix this any ideas?

Hey,
Sorry for the late response, please send me your file structure, any errors.
Thanks


Add hibernate optional parameter for az vmss deallocate

Preconditions

[X] No need to upgrade Python SDK or the Python SDK is ready.

Related command
az vmss deallocate --instance-ids 1 --name MyScaleSet --resource-group MyResourceGroup --hibernate
Resource Provider
Microsoft.Compute/VirtualMachineScaleSets
Description of Feature or Work Requested
We would like support for hibernating a VM while deallocating. The support is already added in the SDK and api. We want to extend this support for CLI by adding this optional parameter - hiberante in the az vmss deallocate.
Minimum API Version Required
2021-03-01
Swagger PR link / SDK link
https://github.com/Azure/azure-rest-api-specs/pull/23409
Request Example
No response
Target Date
2023-08-01
PM Contact
jainan
Engineer Contact
srivmura
Additional context
No response

Thank you for opening this issue, we will look into it.

2023-08-01

@vatsan28 Please note that due to the fact that our work is already fully scheduled for this sprint and the submission time of this request is relatively late, so we plan to develop and release it in the next sprint. Do you think this release time meets your expectations?

Hi @vatsan28 ,
Could you help me with the following error prompt that I encountered when running the command in development.

I have run az vmss create -g qinkai-test -n vmss-test --image Canonical:UbuntuServer:18.04-LTS:latest --orchestration-mode Flexible --admin-username vmtest before the execution.
Or could you please provide an end-to-end user case in the form of Azure CLI command sequence so we can test the effect of the new parameter?

@ReaNAiveD The VMSS needs to be created with the hibernationEnabled. Following is the example:
az vmss create -g qinkai-test -n vmss-test --image Canonical:UbuntuServer:18.04-LTS:latest --orchestration-mode Flexible --admin-username vmtest --enable-hibernation true

@ReaNAiveD The VMSS needs to be created with the hibernationEnabled. Following is the example: az vmss create -g qinkai-test -n vmss-test --image Canonical:UbuntuServer:18.04-LTS:latest --orchestration-mode Flexible --admin-username vmtest --enable-hibernation true

Hi @vatsan28 , at present, there is no argument named --enable-hibernation in command az vmss create. You can create a feature request issue for the argument and then we can move forward.

Created issue: https://github.com/Azure/azure-cli/issues/27070

Can you please confirm if both of these can be done before 09-05 release?


Add "top" level as a 'façon de parler'

This would simplify the syntax quite a bit; judgments would be allowed to mention the 'top' level, but U ltop would not be a type. This would replace the optional-ness of level annotations in some judgments.

Yes we can do this. I did not do this purely because of trying to be excessively pedantic for the first experiment. One difficulty of super-clean-API is that the pretty printer needs to know whether it's omega or not. Or, we can just print "at omega" whenever the level is actually lacking.

Should we forbid (lmax lomega lomega)?

Same question for (+ lomega 0) and whether an exception should be thrown for (++ lomega).

@favonia Would it be weird to have (lmax lomega lomega) = lomega and (++ lomega) = lomega?

(lmax lomega lomega) = lomega and (++ lomega) = lomega

This is known as a 'parler trick'.

@jonsterling I am fine with (lmax lomega lomega) = lomega but not the other, because I wish (U l) is always a member of (U (++ l)).

@favonia Aha, I see! I think that is fair


Navigation

I recently downloaded all the files. First page appears just fine, but whenever I try to click or scroll an exception is caught. I've tried a few things but haven't been able to understand why.

Hello,
Try change this line in Routes.dart:
return new PlanetDetailPage(params["id"]);
To:
return new PlanetDetailPage(params["id"][0]);
Hope it works!


Remove extra search box on plugin dialog

🚀 Feature
Now that plugin dialog is uniform in displaying packages on both installed and non-installed plugins, we could move to use a singe search box.

Motivation
Simplify UI :)

Pitch
No need for two separate boxes if we can filter using a single one #2345

Alternatives

Additional context

Thanks @Carreau for suggesting this on the dev meeting!

we might need to put the box in a different place to convey the right meaning

This was my first thought as well. Is it possible to put it at the very top of the page for now, above the "Installed plugins" label?


update study handling documentation with preparation files and new GUI

The current documentation still says template vs. preparation files and it is based on the old study management GUI, needs to be updated.
Files "suffering" from template's sickness:

[x] getting started
[x] no raw sequences
[x] prepare information files
[x] prepare templates

My guess is that those are the same files that suffer from old GUI-tis.

Solved in https://github.com/biocore/qiita/pull/1815


bg merges ok, but color doesnt

Describe the bug
I am using twmerge and color is not merging right.
To Reproduce
https://github.com/dcastil/tailwind-merge/pull/406
Expected behavior
With twMerge('color-red-500 color-black') I would expect the result to be color-black.
Environment

tailwind-merge version: 2.3.0

Additional context

Hey @pvinis! 👋
That is because color-black is not a Tailwind CSS class, at least not in the default config. I think you meant text-black which modifies the CSS property color, right? https://tailwindcss.com/docs/text-color

OH. MYGOD.
Closing everything, I'm sorry, I was confused and I can't believe I mixed these up 🙈.
Thank you for your work!

Haha no worries! 😁 Happy to help!


feat: Make source_path blocks independent

fe72310 fix: improve sh exec step debug logging
a4fd2e1 feat: allow to don't specify path in source_path and use explicit :zip commands
aa2cb10 fix: make source_path blocks independent

The source_path blocks don't impact anymore to each other by workdir changes

38d6715 fix: unify :zip path normalization

Maybe it makes sense to bump major release version due to changed source_path blocks behavior.

This PR is included in version 7.15.0 :tada:


Update spa-services.md

Wrong word.  No issue number to fix, just needs an update.  Thought I'd contribute.

@rynnova Thank you for your contribution!


Cluster sync: Delete

Signed-off-by: Sherin Varughese<EMAIL_ADDRESS>https://github.com/stolostron/backlog/issues/17973
Retry connecting to Postgres if connection is not available.
Process delete cluster transaction

Delete resources and edges when managedclusteraddon is deleted
Delete cluster node itself when managedcluster is deleted

/retest

/lgtm


Add a feature to the judgement page to quickly access previously highlighted paragraphs.

Please let me know if you have any specific ideas @profsmucker , @claclark.
I have previously implemented a mini-map for each document but I never used it. Do you think it's a good idea to go with the mini-map feature?

Personally, I think the easiest thing to do, and the most useful, would be to display only highlighted material if any is highlighted, and to allow a toggling back to "show full document with highlighting" and then allow switching back to "show highlighted material only".
The reason for this is that we know from our previous work that extractive summarization is enough to make relevance decisions.  Thus, we should train people to have their highlighting be the equivalent of a personalized extractive summary, and then it will be sufficient for them to make a decision.
You want to get rid of all the excess content and only show the relevant material.  Changing the scrollbar location is not the correct solution.  This is especially true when you highlight multiple parts of the document.

@claclark So you mean we want two versions; one with scrolling to the first highlight part and one with what mark said.


Note that you can still use the song yourself, but it won't be public


Restore UI layout after Common Criteria confirmation (bsc#1194279)

Problem
Selecting the "Common Criteria evaluated configuration" System Role    produces a confirmation screen with a different, wider, layout. A missing    CloseDialog means this layout would persist after that confirmation is made.

bsc#1194279
retargetting #7 onto SP4

Solution
+ Wizard.CloseDialog

Testing
Tested manually:

SLE-15-SP4-Online-x86_64-Build148.1-Media1.iso , with ... dud=/system-role-common-criteria.rpm
Select SLES product, ...
Register, ...
Select Common Criteria system role

The screen layout works correctly with the fix
Screenshots
See the bug report, bsc#1194279, for screenshots

@msmeissn @lnussel I have added the changelog and also retargeted the PR onto SP4
I haven't tested it though

Tested now, it works.


World origin shifting

I need help to implement world origin shifting as suggested by @HeadClot.
Please leave a comment below if you want to help me on that :)

they did it in blueprints but it looks like the functionality is already in engine https://answers.unrealengine.com/questions/150481/is-there-a-way-to-zero-world-origin-to-a-new-locat.html

Thanks! Should have googled before creating this issue.


Testing all-contributors

@all-contributors please add @Ifycode for doc

@all-contributors please add @Ifycode for code

@all-contributors please add @all-contributors for infra


[WIP]Distributed execution support of Hive with vineyard

What do these changes do?

Related issue number

Fixes #issue number

I try to add location info for splits and Tez is able to get the information I provided, but failed to request a node resource.
Here is the log:

We can see that yarn task scheduler service get the node hostname of "hadoop-yarn-nm-0" but failed to assign container. When the tez client asks for "/default-rack," it matches the hadoop-yarn-nm-0 node ("not match" in the figure should be " match". It is a mistake). Since the current three nodes are at /default-rack, if the requested resource is "/default-rack", which node the task is assigned to is random. There is no guarantee that the task will be assigned to the node with the corresponding data.
I have no idea why the allocation failed.

Rebase to main please.

Rebase to main please.

Done


fix: remove prototypes from intro

Checklist:

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[X] My pull request targets the main branch of freeCodeCamp.
[X] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

The placeholder strings were brought to my attention by a contributor on Crowdin. Given that these objects are for the new curriculum projects, they are currently not used in production and don't need translation. @RandellDawson brought up a valid point that the projects or curriculum might change between now and when we ship, so IMHO we shouldn't be spending contributor bandwidth on translating these objects at all.
Removing them from the file creates no build errors locally - @scissorsneedfoodtoo and @ojeytonwilliams do you see any potential concerns with removing them?

If these are removed and you attempt to run it locally with show_upcoming_changes=true

This was a concern I had too.

The titles will need to be translated at some point - I suppose they could change, but I would guess those probably wouldn't.

I'm not 100% sure - I know there's been some refactoring under consideration.

Other than that, it's just one word, placeholder - it may not be obvious that it doesn't need translating at the moment, so I can see some confusion. It is just one word - If we translate it once, wouldn't it get applied or suggested across the rest of them?

That's certainly a possibility - the question coming from contributors was the lack of context for the use of placeholder. The alternative would be we could hide these strings manually in Crowdin (or via the upload script).

I'll close this for now - I think hiding the strings might be a better approach, since show_upcoming_changes breaks.

Does an empty string work? Then presumably the translators wouldn't run into the issue.

An empty string wouldn't even show in Crowdin, I think 🤔 That would be ideal. Let me test how it would render on Crowdin's side.

Crowdin renders them as empty strings and automatically hides them, so tranlsators and proofreaders would not be able to touch them


Storybook - Cannot add comment to a function in .ts file

Current Behavior
I have a setup with Storybook, from the official template. When I try to add a comment to my function, storybook throws the following error:

The file has only 43 lines of code, so the second error does not even make sense to me. It only occurs, if the function is exported.
/** This comment is fine */
const foo = () => {}

Empty comment also works, even if exported (either named or default export works)
/** */
export const foo = () => {}

Any comment with export (either named or default) fails.
/** This comment gives the above mentioned error */
const foo = () => {}
export default foo

Expected behavior
Should allow adding comments to functions

Suggested solution(s)
This is a bug I have never come across, so I have no idea how to fix it.
Additional context
I only did one change to the original template, I like to keep related code next to each other, so I changed the folder structure:
src/
    |_Foo/
       |_index.tsx
       |_the_file_with_the_function_that_throws_error_on_comment.ts
       |_index.stories.tsx

The `.storybook/main.js` file:

  module.exports = {
    stories: ['../src/**/*.stories.(ts|tsx)'],
    addons: ['@storybook/addon-actions', '@storybook/addon-links', '@storybook/addon-docs'],
    webpackFinal: async (config) => {
      config.module.rules.push({
        test: /\.(ts|tsx)$/,
        use: [
          {
            loader: require.resolve('ts-loader'),
            options: {
              transpileOnly: true,
            },
          },
          {
            loader: require.resolve('react-docgen-typescript-loader'),
          },
        ],
      });
      config.resolve.extensions.push('.ts', '.tsx');
      return config;
    },
  };

Your environment

Software
Version(s)

TSDX
0.13.1

TypeScript
3.8.3

Browser
83.0.4093.3 dev (64-bit)

npm/Yarn
6.9.0/1.21.1

Node
12.6

Operating System
Linux 19.10

Sorry but as you've said yourself, this is an issue with Storybook, not TSDX, so sounds you've filed this issue in the wrong repo.
TSDX doesn't use Storybook, it just provides a template for you to use it.
Please file an issue with Storybook if you're having an issue with Storybook.


JSON_JQ_TRANSFORM - complete system task development

Complete JSON_JQ_TRANSFORM development

[X] Add to TaskType enum
[X] Create TaskMapper
[X] Add mapper to CoreModule
[X] Add constraint validation to required queryExpression input parameter
[X] Create unit tests for task mapper (based on the other task mappers)
[X] Add new system task to docs

Since this change JSON_JQ_TRANSFORM will no longer require a task definition (optional)

For some reason the build is failing, not sure related to the PR

@elisherer Looks good to me. Can you also add an integration test that verifies this task execution end-to-end please? We've been moving towards Spock based integration tests, and here's one example to get started: https://github.com/Netflix/conductor/blob/master/test-harness/src/test/groovy/com/netflix/counductor/integration/test/DecisionTaskSpec.groovy

@elisherer Looks good to me. Can you also add an integration test that verifies this task execution end-to-end please? We've been moving towards Spock based integration tests, and here's one example to get started: https://github.com/Netflix/conductor/blob/master/test-harness/src/test/groovy/com/netflix/counductor/integration/test/DecisionTaskSpec.groovy

@kishorebanala  Done


CQL Filters patch

This patch add a "Empty Query"  option to the query selector instead of an empty string ""

@Rachael-E Thanks for the suggestions! Please take a look when you get the time.

This still needs the screenshot name change

Thanks for the catch

As discussed, this'll need to revert to the kotlin synthetic pattern for now, so we can get it working in the sample viewer

Removed viewbinding!


Detect WEBM video format

As title, and useful reference is available here.

Just notice that the WebM has the same identification situation as OGG.
It has the same identification approach for video/webm and audio/webm.

Specification

https://www.webmproject.org/docs/container/
https://www.w3.org/2013/12/byte-stream-format-registry/webm-byte-stream-format.html
http://fileformats.archiveteam.org/wiki/Webm

Identification

A WebM file begins with 0x1a 0x45 0xdf 0xa3, as do all EBML files.

It contains an EMBL "DocType" element whose value is the string "webm". This means that the byte sequence 0x42 0x82 0x84 'w' 'e' 'b' 'm' will appear in the file, normally near the beginning.

But the magic bytes of OGG are: 0x4f 0x67 0x67 0x53.


Unable to connect to server with HTTPS

Based on the ReadMe attached to this package, I added the SSL certificates to my server.js file as described, but I'm still not able to connect.  I had everything running locally just fine.  Do you have any tips for troubleshooting  or general suggestions for what might be happening?  I've tried connecting to the following, none of which I've been able to get working.
https://mydomain.com:6001
https://localhost:6001
https://<IP_ADDRESS>:6001
localhost:6001
<IP_ADDRESS>:6001

+1
Everything is working fine on local with homestead, but not working on server over HTTPS.

Are you guys including full paths to your your ssl certificate files?

Yes, I believe so.  Here is what I have.
var options = {
    authHost: 'https://mydomain.com',
    host: 'https://mydomain.com',
    port: '6001',
    sslCertPath: '/etc/nginx/ssl/nginx.crt',
    sslKeyPath: '/etc/nginx/ssl/nginx.key'
};

Hmmm. I haven't fully tested this out on a remote server :) so bear with me. I wonder if this may be a permissions issue. Usually files and directories in the etc directory are only accessible by the root user. Could you try running with sudo or using a root user if you can?

I just tried running
sudo node socket.js
and it still didn't seem to work for me.  I'm seeing "ERR_CONNECTION_REFUSED" in my Chrome console.

Just for a little bit of additional information.  I'm currently running a LEMP setup on Laravel Forge.  I'm using a key from Let's Encrypt which is automatically installed within Forge.  The path that is being referenced for the keys in my nginx config file is '/etc/nginx/ssl/default/{unique_id}/nginx.crt'.  These files, however, are empty and when I tried to use one it threw an error (something about it not being able to find the file) when I went to start the socket.  I assumed there was some sort of connection being made between these files, and the files I'm referencing in my example in one of my previous comments.  When I changed the socket.js file to reference the files above, it would allow me to start the socket just fine.

I made a couple changes and now am start to see the following "ERR_INSECURE_RESPONSE".

Gotcha, yeah if the files are not found, the server will not run properly. In the next release there will be some better error logging to help diagnose these type of things.
On Laravel forge, when certificates are renewed, it looks like the directory of the certificate files change. I understand that it may be cumbersome to keep up with these, but the paths will need to remain updated in order to run the server with SSL. As this falls outside of the actual configuration of the server, and more so the location of the ssl certificates files I'm going to close this and ping @taylorotwell.
A possible solution for Laravel Forge and other instances of this issue may be to perform an "atomic" deployment of the certificates by creating a symlink when the SSL certificates are installed.
So each time the certificate is refreshed, the active directory:
/etc/nginx/ssl/SITE/UNIQUE_ID
would be symlinked to:
/home/forge/ssl/SITE/UNIQUE_ID

Thanks for the suggestions @tlaverdure.  I was able to get it working with a combination of setting the ssl files to the ones located within the unique_id folders, combined with using sudo when running "sudo node socket.js".
I'm not able to get it running however using pm2.  Even when I run sudo pm2 start socket.js it doesn't seem to work.  This seems especially important since I seem to be getting an error "Unexpected token a in JSON at position 0" quite often that crashes the service.

@tlaverdure could you link to any discussion online you're having with Taylor about this issue as this was the reason I decided to abandon using SSL and stick with regular http.
Having the SSL directories change every month when let's encrypt updates is a very big hassle.
Maybe if forge sent a callback to your server after a successful SSL update, containing the new paths to the SSL files we could do our own processing etc?

having the same problem here.
Am able to get it to work over https if I run sudo laravel-echo-server start with the json config file pointing to the /etc/nginx files. But can't get it to run using PM2
Also hadn't even realized that the ssl cert path will be changing on a periodic basis which is a big hassle for sure.
in the past I managed to get something similar to this to work by using supervisor so I'll look into that
if anyone knows how to get this to work with PM2 or finds a clean solution to the changing paths please let me know.
I guess sudo is only needed because the ssl cert is located in a root-owned path so a workaround that might be useful too

I same issue ... pls help me

+1

Try using the .pem files generated by LetsEncrypt instead - Source
Your options (either through JSON or the CLI) should instead look like this:
var options = {
    authHost: 'https://example.org',
    host: 'https://example.org',
    port: '6001',
    sslCertPath: ' /etc/letsencrypt/live/www.example.org/fullchain.pem',
    sslKeyPath: '/etc/letsencrypt/live/www.example.org/privkey.pem'
};

For me in local environment it didn't work with host: "https://example.org". I changed it back to host: null, but this value should be in: authHost: "https://example.org"


consider all addresses on an interface

An interface can have multiple addresses. We should consider all addresses on an interface when we are looking for an interface with a given address.

Ping @bowei @bprashanth

/remove-lifecycle stale
On Sun, Oct 21, 2018, 12:05 PM fejta-bot<EMAIL_ADDRESS>wrote:

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually
close.
If this issue is safe to close now please do so with /close.
Send feedback to sig-testing, kubernetes/test-infra and/or fejta
https://github.com/fejta.
/lifecycle stale
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/contrib/pull/2931#issuecomment-431695077,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALPcHc0119piKP_aibJgxhiOMY0su1gOks5unMVugaJpZM4VXPhA
.

/remove-lifecycle rotten


pull request for e2e test at 01/12/2017 20:56:13

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


Answers to the questions

Where can i find the answers to the linux questions?

In Google and experience
On Thu, Oct 31, 2019, 12:04 AM Ashish Github2<EMAIL_ADDRESS>wrote:

Where can i find the answers to the linux questions?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/chassing/linux-sysadmin-interview-questions/issues/53?email_source=notifications&email_token=AJNTTXKUXVA3D5YD5DZTPF3QRHZERA5CNFSM4JHAKFLKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HVR7V2Q,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJNTTXMDN7HHBSUWZK5RIU3QRHZERANCNFSM4JHAKFLA
.

Do we have answer for these questions?  or this is just a awesome question bank?

hye , where i can share the answer to the questions ? or is it just question bank ?

Is it at all useful to post a bunch of questions without answers, or even suggestions? especially for the "fun" questions, which might be open to interpretation?


Fix HttpContext PathBase combine

Fix for issue #597

I think this is intended behavior in that if block because the "rebase" option is set to true there.

Looks good, one minor request to add a unit test. I have the code for it in #597
Thanks!

:tada: This PR is included in version 6.8.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 6.9.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fixing radio component translate

Making translation for radio component.

[ ] Changes have been thoroughly tested to not break existing functionality.
[ ] New functionality has been documented or existing documentation has been updated to reflect changes.
[ ] Visual changes are explained in the PR description using a screenshot/recording of before and after.

Fixed consistency in https://github.com/filamentphp/filament/commit/3a48d49139c88a249cf5348534414fe6ec303013.


Implement GET_STATUS implementation result

Related: https://github.com/eProsima/Micro-XRCE-DDS-Client/pull/312

Tests here: https://github.com/eProsima/Micro-XRCE-DDS/pull/131

Build status:

Linux
Windows


Add files via upload

add galileimondragone.txt file

@Salvatorepaolella First stage of verification passed, official domain is galileimondragone.edu.it as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@Salvatorepaolella Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Python libraries and Rubocop not working in the retroh/molecule docker container

I use the inspec verifier and rubocop linter.
When trying to run molecule test through the retroh/molecule docker container it complains that rubocop is not installed.
I tried to send a pull request for this Dockerfile myself but couldn't find where it is hosted.
Is it possible to get rubocop added to that containers image?
verifier:
  name: inspec
  lint:
    name: rubocop

I also ran into an issue with docker-py support not working so I had to remove that and install the docker python library.
These are the changes I had to make to the docker file to the Dockerfile to get it working.
FROM retr0h/molecule

RUN sudo pip uninstall -y docker-py
RUN sudo pip install docker

RUN sudo apk update
RUN sudo apk add ruby-rdoc ruby-dev ruby-irb
RUN sudo gem install rubocop

Here is the Dockerfile, can you submit a PR?

https://github.com/metacloud/molecule/pull/1390

Implemented in #1390.

How/when does this image get built and pushed to docker hub?

When 2.17 is released in a couple weeks or so.