PauliList([]) confusion

Environment

Qiskit Terra version: 0.23.2
Python version: 3.9.7
Operating system: Linux

What is happening?
I find the following code example puzzling:
>>> from qiskit.quantum_info import PauliList
>>> PauliList(["XX", "YY"])[0:0]
PauliList([])
>>> PauliList([])
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/garrison/serverless/.direnv/python-3.9.7/lib/python3.9/site-packages/qiskit/quantum_info/operators/symplectic/pauli_list.py", line 140, in __init__
    base_z, base_x, base_phase = self._from_paulis(data)
  File "/home/garrison/serverless/.direnv/python-3.9.7/lib/python3.9/site-packages/qiskit/quantum_info/operators/symplectic/pauli_list.py", line 181, in _from_paulis
    raise QiskitError("Input Pauli list is empty.")
qiskit.exceptions.QiskitError: 'Input Pauli list is empty.'

As in #9720, the first expression evaluates to something whose repr cannot be instantiated directly.
How can we reproduce the issue?
Code sample above.
What should happen?
Either both lines should error or it should be possible to evaluate the repr string.
I should note that I've noticed at least one method that assumes that the PauliList is not empty:
>>> PauliList(["XX", "YY"])[0:0].unique()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/garrison/serverless/.direnv/python-3.9.7/lib/python3.9/site-packages/qiskit/quantum_info/operators/symplectic/pauli_list.py", line 620, in unique
    if np.any(self._phase != self._phase[0]):
IndexError: index 0 is out of bounds for axis 0 with size 0

There may be others.  EDIT: _commutes_with_all is another.
Any suggestions?
It would be nice for PauliList to support the empty list.  However, while num_paulis would be 0, num_qubits would be undefined in this case, and perhaps this will lead to complications elsewhere.
Or, alternatively, there could be a way to construct a PauliList with 0 elements but n qubits.  And then the repr for PauliList(["XX"])[0:0] could be something like PauliList([], num_qubits=2).
Perhaps even better would be to make the __repr__ for PauliList(["-XYZ"])[0:0] be PauliList(["III"])[:0].  That way, the number of qubits can be encoded in its repr without introducing any additional API surface area (and standardized to be n identities).  This is my favorite option at the moment.

I saw this after the PR. I commented the main bits on the PR - I'd be ok with adding an optional num_qubits field to the constructor, but not to making the repr some compound expression. That's not really what repr is for.

After thinking about this more, I essentially agree with Jake.  It makes much more sense to add an optional num_qubits field to the constructor, such that repr(PauliList(["XXX"])[:0]) would be "PauliList([], num_qubits=3)", instead of pursuing my proposed change in #9729.  This better path will achieve the following goals:

The meaning of the repr PauliList([], num_qubits=3) is immediately understandable to a human.  Even though a repr of PauliList(["III"])[:0] is technically correct, is it entirely unclear what is meant by it.  You have to understand slicing and think for a moment before hopefully realizing that it's an empty PauliList on 3 qubits.  The optional num_qubits field makes this explicit, and per the Zen of Python, "Explicit is better than implicit."
The repr can be evaled to construct the object.  I agree that one shouldn't be calling eval on a repr programmatically, but it is incredibly helpful to the user if the REPL outputs something that the user could type in to get the same result.  Indeed, Python's documentation on repr mentions this as a desirable trait:

Return a string containing a printable representation of an object. For many types, this function makes an attempt to return a string that would yield an object with the same value when passed to eval(); otherwise, the representation is a string enclosed in angle brackets that contains the name of the type of the object together with additional information often including the name and address of the object.

If one passes num_qubits, the code can succeed whether the list contains some elements or zero elements.  In fact, this was the original motivation behind me filing this issue: I was calling PauliList(c) where c was some list comprehension which could be empty in a corner case, and I needed that code to succeed in that corner case.

By contrast, my proposal in #9729 achieves only the second goal.

However, while num_paulis would be 0, num_qubits would be undefined in this case, and perhaps this will lead to complications elsewhere

Why can't we also set the number of qubits to zero?

Why can't we also set the number of qubits to zero?

Do you mean, for any empty list, why not just set num_qubits to zero by definition?
This could work as an alternative to num_qubits in the constructor, but has some drawbacks.

It is already possible to create an empty PauliList with a non-zero number of qubits through two different methods: slicing, and from_symplectic.  These objects are only considered equal to each other if they have the same number of qubits.
>>> PauliList(["II"])[:0] == PauliList(["II"])[:0]
True
>>> PauliList(["II"])[:0] == PauliList(["III"])[:0]
False

The goal of #9770 is to make it possible to construct these states via list construction as well.  If an empty PauliList were to always have num_qubits == 0, these other construction methods would need to be modified, too, to enforce that, and the above equalities would both evaluate to True.

There is the PauliList.insert() method, which requires that the Pauli being inserted has the same number of qubits as the PauliList.  If an empty PauliList were to have zero qubits by definition, then would would be no way to insert a Pauli with non-zero num_qubits in the list.

Ikko: your suggestion solves the representation problem, but means that for logical support, we'd violate some of the current invariants of PauliList.  For example, given the structure of the collection and paulis being some PauliList then for all valid x, paulis[:x] + paulis[x:] == paulis.  Right now, "all valid x" doesn't include those that create empty lists (since the list quickly reveals itself to be in an invalid state when trying to do operations on it), but if we inferred "num_qubits == 0 if unspecified", then we're violate this (very logical) invariant: paulis[:0] + paulis[0:] would be a value error due to mismatched num_qubits unless we specifically added catch clauses into every operation.
It's simpler to not infer something non-inferrable - in the vast majority of cases, including most of those that are natural ways an empty list might appear - num_qubits is inferrable, so users will see no change.

Sorry, I think now the number of qubits should be Optional[int].
For example, let's consider a list of strings of the same length.
[""]

The length should be 0.
[]

The length should be None.
["II", "XX"][:0]

This case is length is None.
I believe PauliList is similar to this example.
Another question: what is SparsePauliOp with empty PauliList? Is it buggy without the new validation? In particular, I think it would be a change of the representative element of 0.

I think we roughly agree that your second example has an unknowable number of qubits.  In the third example, I disagree that PauliList needs to be None in this case; PauliList contains more context than a list, so we can safely store that the empty list has two qubits in that case.
I think that allowing PauliList.num_qubits to be None is a problem (specifically the exposed attribute: I'm fine using PauliList.__init__(..., num_qubits=None) to indicate "must infer, error if not possible"):

it's a breaking change in the existing API because the return type changes
it forces all consumers (both us and users) to now handle the case that num_qubits is None when inspecting that attribute

What we're proposing is something that looks approximately like:
class PauliList:
    def __init__(self, data, num_qubits=None):
        self._data = ...
        if num_qubits is None:
            if len(data) == 0:
                raise ValueError("couldn't infer number of qubits, set 'num_qubits' explicitly")
            self._num_qubits = _infer_shape(self._data)
        elif num_qubits != _infer_shape(self._data):
            raise ValueError("explicitly given 'num_qubits' doesn't match data")
        else:
            self._num_qubits = num_qubits

    def __getitem__(self, val):
        if isinstance(val, int):
            return Pauli(self._data[val])
        elif isinstance(val, slice):
            return PauliList(self._data[val], num_qubits=self.num_qubits)
        ...

    ...

so the only operation that ever needs to do inference is the initial construction of a PauliList; for every other operation, num_qubits is already known, so even if we get an empty list of of __getitem__, the resulting object is still usable.
We don't want PauliList.num_qubits to be None, because that makes every binary consumer of PauliList start with something like:
def add(self, other):
    if self.num_qubits is None and other.num_qubits is None:
        out_num_qubits = None
    elif self.num_qubits is None:
        out_num_qubits = other.num_qubits
    elif other.num_qubits is None:
        out_num_qubits = self.num_qubits
    elif self.num_qubits != other.num_qubits:
        raise ValueError("cannot add together PauliLists on different qubits")
    else:
        out_num_qubits = self.num_qubits

instead of just
def add(self, other):
    if self.num_qubits != other.num_qubits:
        raise ValueError("cannot add together PauliLists on different qubits")
    out_num_qubits = self.num_qubits

The advantage of letting num_qubits be None is that the empty PauliList can interact in binary operations with a PauliList of any number of qubits, but I don't really see how this is especially desirable - there are other ways of denoting "broadcasting zero" than compromising the whole class.
For SparsePauliOp: I haven't tested it, but I imagine it should be fine for most inputs because it sets its num_qubits by doing LinearOp.num_qubits = self._pauli_list.num_qubits, which in our scheme would always be set.  Because of the amount of typing inference all over quantum_info, I imagine we'd need to add a similar num_qubits option to SparsePauliOp if it can also accept [], but the only place that would matter would be in __init__.

@jakelishman Thank ynu. I understand that handling optional types is difficult. (In your addition example, if one is just an int and the other is None, I think it's an error, so existing is enough.) However, I still can't find a positive reason to allow empty PauliList. If optional is difficult, empty can be an error. As you know, the original design did not allow empty PauliList and I don't want to change the API as possible.
If you could point me to a useful example of PauliList, I would be convinced.

I can let @garrison speak more to his uses, but from my perspective, it's that empty collections tend to arise fairly naturally whenever we allow and use slicing, and it's rather annoying as a user to need to defensively code against this, or have to insert special cases to handle the "zero" case when mathematically it makes sense for it to behave the same. But sure, I don't have a concrete use in mind, more just a ~general feeling, and your point about not wanting to relax the current deliberate constraint without good reason is a solid point.  I don't have one right now, but Jim might.

If you think of PauliList in terms of its real data structure, z,x boolean arrays, it makes sense that you can have a zero element op, or a zero qubit op, or both, and these are already valid Pauli's you can construct via from_symplectic:
0-qubits, 0-length
z = x = np.zeros((0, 0), dtype=bool)
op = PauliList.from_symplectic(z, x)
print(f"op = {op}, array shape = {op.z.shape}, len = {len(op)}, num_qubits = {op.num_qubits}")
#op = [], array shape = (0, 0), len = 0, num_qubits = 0

3-qubits, 0-length
z = x = np.zeros((0, 3), dtype=bool)
op = PauliList.from_symplectic(z, x)
print(f"op = {op}, array shape = {op.z.shape}, len = {len(op)}, num_qubits = {op.num_qubits}")
# op = [], array shape = (0, 3), len = 0, num_qubits = 3

0-qubits, 3-length
z = x = np.zeros((3, 0), dtype=bool)
op = PauliList.from_symplectic(z, x)
print(f"op = {op}, array shape = {op.z.shape}, len = {len(op)}, num_qubits = {op.num_qubits}")
op = ['', '', ''], array shape = (3, 0), len = 3, num_qubits = 0

So imo you can already represent zero length or zero qubit paulis lists, and pauli[:0] is valid (maybe there are some internal methods that fail for empty paulis but thats a separate issue).
The problem is that there is no way to initialize some of these from the __init__ constructor which assumes an iterable (and hence the repr being used as sugar for string label iterable not working). One solution would be to add a num_qubits kwarg to init, which is only needed for empty paulis (it should only be used for validation to throw an expection for non-empty paulis if it is specified and doesnt match the number of qubits of paulis in the list). It doesn't need to be stored, its would just convenience to call
Pauli([], num_qubits=nq) -> PauliList.from_symplectic(np.zeros((0, nq), np.zeros((0, nq)))

and then you could update the repr to show this kwarg only if the length is 0.

The problem is that there is no way to initialize the length-0 cases from the init constructor which assumes an iterable (and hence the repr being used as sugar for string label iterable not working). One solution would be to add a num_qubits kwarg to init, which is only needed for empty paulis (it should only be used for validation to throw an expection for non-empty paulis if it is specified and doesnt match the number of qubits of paulis in the list).

The first sentence is the crux of how this issue arose, but it's also important to note that the same method of attempting the inference in the __init__ is reused all over the place.  The Pauli class correctly inferred things from the shape of the symplectic matrices given, but PauliList doesn't do it right.  The second sentence is the solution Jim and I agree with.

It [PauliList.num_qubits] doesn't need to be stored

This is true, though it does need to be exposed on the class to fulfil the interface.  It can be a computed property using the correct inference, though.

One solution would be to add a num_qubits kwarg to init, which is only needed for empty paulis (it should only be used for validation to throw an exception for non-empty paulis if it is specified and doesnt match the number of qubits of paulis in the list). It doesn't need to be stored, its would just convenience to call
Pauli([], num_qubits=nq) -> PauliList.from_symplectic(np.zeros((0, nq), np.zeros((0, nq)))

and then you could update the repr to show this kwarg only if the length is 0.

Thanks.  This is what https://github.com/Qiskit/qiskit-terra/pull/9770 proposes.

@chriseclectic Thank you.Indeed, my understanding of the current situation was insufficient. And your suggestion (using from_symplectic) is smart! I support this direction.


JAX WS builtin Server implementation is very slow on Linux

Previously tracked at: https://bugs.openjdk.java.net/browse/JDK-8193236
FULL PRODUCT VERSION :
openjdk version "1.8.0_151"
OpenJDK Runtime Environment (build 1.8.0_151-8u151-b12-1~deb9u1-b12)
OpenJDK 64-Bit Server VM (build 25.151-b12, mixed mode)
ADDITIONAL OS VERSION INFORMATION :
Linux c0ab5270bb70 4.9.49-moby #1 SMP Wed Sep 27 23:17:17 UTC 2017 x86_64 GNU/Linux
A DESCRIPTION OF THE PROBLEM :
The built-in JAX WS server implementation works 100 times slower on linux machines then on Mac OS X or Windows. JMH test is published here: https://github.com/Andremoniy/linuxjvmjaxwstest
Basically it does the following:

starts a JAX WS with one SOAP method:
endpoint = Endpoint.publish("http://localhost:8888/", new FooServiceImpl());
performs client requests to this method:
String result = state.client.foo(state.foo);

On Mac OS X with 2 Cores Intel Core i7 it gives:
JMH version: 1.19
VM version: JDK 1.8.0_151, VM 25.151-b12
VM invoker: /Library/Java/JavaVirtualMachines/jdk1.8.0_151.jdk/Contents/Home/jre/bin/java
...
Result "test.openjdk.MyBenchmark.testMethod":
3077.813 ±(99.9%) 51.818 ops/s [Average]
(min, avg, max) = (1718.347, 3077.813, 3315.271), stdev = 219.400
CI (99.9%): [3025.995, 3129.630] (assumes normal distribution)
Run complete. Total time: 00:06:49
Benchmark Mode Cnt Score Error Units
MyBenchmark.testMethod thrpt 200 3077.813 ± 51.818 ops/s
The same machine, docker without CPU limitations:
JMH version: 1.19
VM version: JDK 1.8.0_151, VM 25.151-b12
VM invoker: /usr/java/jdk1.8.0_151/jre/bin/java
...
Result "test.openjdk.MyBenchmark.testMethod":
19.882 ?(99.9%) 0.169 ops/s [Average]
(min, avg, max) = (10.031, 19.882, 20.104), stdev = 0.715
CI (99.9%): [19.714, 20.051] (assumes normal distribution)
Run complete. Total time: 00:07:01
Benchmark Mode Cnt Score Error Units
MyBenchmark.testMethod thrpt 200 19.882 ? 0.169 ops/s
The situation changes if JAX WS server is started with Jetty (there is no performance issues):
ContextHandlerCollection contextHandlerCollection = new ContextHandlerCollection();
server.setHandler(contextHandlerCollection);
JettyHttpServerProvider.setServer(server);
HttpContext context = new JettyHttpServerProvider().createHttpServer(new InetSocketAddress(8888), 5).createContext("/");

Endpoint endpoint = Endpoint.create(new MyBenchmark.FooServiceImpl());
endpoint.publish(context);

server.start();

Please, refer also: https://stackoverflow.com/questions/47645817/jax-ws-server-implementation-performance-issue-for-linux-jvm
STEPS TO FOLLOW TO REPRODUCE THE PROBLEM :
On linux machine (for instance, docker pull openjdk:8u151)
git init
git clone https://github.com/Andremoniy/openjdkjaxwstest
mvn package
cd target
java -jar benchmark.jar
EXPECTED VERSUS ACTUAL BEHAVIOR :
EXPECTED -
The performance should be the about 3000-5000 ops/s.
ACTUAL -
The performance is about 20 ops/s.
REPRODUCIBILITY :
This bug can be reproduced always.
---------- BEGIN SOURCE ----------
https://github.com/Andremoniy/linuxjvmjaxwstest
---------- END SOURCE ----------
CUSTOMER SUBMITTED WORKAROUND :
Use Jetty server for publishing WS.
Source: https://github.com/javaee/metro-jax-ws/issues/1219
Author: LanceAndersen

Running Java VM with -Dsun.net.httpserver.nodelay=true seems to solve the issue on Linux:
# Run complete. Total time: 00:00:13

Benchmark                Mode  Cnt     Score     Error  Units
MyBenchmark.testMethod  thrpt   10  6152.077 ± 831.090  ops/s


Parse in operator before comparisons

Hi,
Inspired by SQL where clauses, I'm used to the in operator having precedence over others like and and or.
For example, I'm used to MainCompanyId in (1, 2) and Name in ("A") to be parsed as (((x.MainCompanyId == 1) OrElse (x.MainCompanyId == 2)) AndAlso (x.Name == "A")).
I think currently the parser attempts to parse it to something like ((((x.MainCompanyId == 1) OrElse (x.MainCompanyId == 2)) AndAlso x.Name) == "A"), which is invalid syntax.
A current workaround is to add additional parentheses and write the clause as (MainCompanyId in (1, 2)) and (Name in ("A")). This parses successfully, but it feels quite verbose to me.
This PR fixes the issue by first parsing for ins before comparisons. It was a simple fix and I'm almost surprised that it didn't break any existing tests (as far as I can see). I added a unit test for this PR's change.
Hope I didn't miss anything and that you agree with change in operator precedence in principle.

@igitur
Thank you.
Can you also create a unit-test which really tests this functionality ? Maybe EntitiesTests ?

Can you also create a unit-test which really tests this functionality ? Maybe EntitiesTests ?

Done. Added new partial class in EntitiesTests.In.cs.

@StefH Any chance of a new release to include the changes?

@igitur
It should already be included:

Oh, thanks!
Where do you see that?
I just kept looking at the Released entry on the repo's main page sidebar:

And then I assumed that my PR wasn't released yet.

https://github.com/zzzprojects/System.Linq.Dynamic.Core/blob/master/CHANGELOG.md

I have to find a way to automatically convert a Tag into a Release here in GitHub...

https://github.com/zzzprojects/System.Linq.Dynamic.Core/blob/master/CHANGELOG.md

Thanks. I wasn't monitoring that.

I have to find a way to automatically convert a Tag into a Release here in GitHub...

Github already creates a draft release when you tag. To convert that to a final release isn't much extra effort. But I'm sure there will be some GitHub Actions script to do that.

I found a github action which can do this:
https://github.com/softprops/action-gh-release
I've added this action on this project, so in the future, when I add a tag, this will be converted into a release.

Great, I might use that in my own projects.

In case you are interested in generating the release notes in the same I do: take a look at https://github.com/StefH/GitHubReleaseNotes


Update customize-the-office-fluent-ribbon-by-using-an-open-xml-format…

…s-file.md

Docs Build status updates of commit 83b9352:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Library-Reference/Concepts/customize-the-office-fluent-ribbon-by-using-an-open-xml-formats-file.md
:bulb:Suggestion
View
Details

Library-Reference/Concepts/customize-the-office-fluent-ribbon-by-using-an-open-xml-formats-file.md

Line 89, Column 3: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/previous-versions/office/developer/office-2010/ff863131(v=office.14)' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add Azure active directory connector

Description

Provide description here

Type of change

[ ] Bug fix
[ ] New feature
[ ] Breaking change

Related issues

Fix #1

Migration notes

Describe migration notes if any

Extra info

Add any additional information

Already test e2e

Already test e2e

There are some issues. Can we fix them? First, when run acceptance-test, it throws: AssertionError: Some of defined cursor fields ['renewedDateTime'] are not specified in discover schema properties for groups stream E assert {}
Second, while writing geo_Location to Faros in users converter

ok I fixed

@cjwooo we might need to bump the version


Add flox blogpost v2

[x] fix bokeh image
[x] add banner
[ ] add social share card

Anderson I tried to add a bokeh image. Can you help get that to work

OK I'm done here.


ListOfFolders wiki page is broken on Safari on MacOS Mojave

https://github.com/vscode-icons/vscode-icons/wiki/ListOfFolders#html

Look like an issue with our generation script. Thanks for bringing it to our attention.

Fixed


Another Translation Fix

Sorry for my mistake...

If you want to write something like "don't", please write "don't". The "" is necessary to pass compiling

And use … instead of ...
They are different.


[nrf fromlist] mgmt/mcumgr/lib: Fix image list command crashing

... when no device
It is possible, in case of two application images, to have no access
to one of devices of the secondary image.
When asserts are enabled, such situation causes crash even though the
image list command can handle it with no problem.
The commit removes the assert and adds additional swap type:
IMG_MGMT_SWAP_TYPE_UNKNOWN to indicate situations where it was
not possible to obtain swap type from boot_util.
Upstream PR: https://github.com/zephyrproject-rtos/zephyr/pull/45636
Signed-off-by: Dominik Ermel<EMAIL_ADDRESS>(cherry picked from commit e209b59c4c2aea3d3805326843b9984fc5ad2994)

Superseded by #811.


Argument encoding cleanup

Unfortunately, a breaking, since the old set_buffers appear to be incorrect (please confirm!).

@mjadczak great points spotted, thank you for the review!
bors r=mjadczak


removed files with duplicate content

I removed the app0.py and the LICENSE2 because:

the LICENSE2 was the exact same as LICENSE
the code from app0.py is already contained in app.py

Thanks!

Thank you for the helpful suggestion


Show count for Object

Show count for Object

We need object field count
so, I count one by one field...
I add counting function object filed

@tulios
check my pr
thx


Preview Database Update dialog height exceeds screen and cannot be scrolled

Type: Bug
When changes should be done on a complex table with many indexes and/or foreign keys, the Preview Database Update dialogs height exceeds screen available space and buttons are out of screen, the dialog is not scrollable making it hard to understand which button will be activated by pressing Enter.
Attached screen is taken on a reduced window for convenience, same problem occurs when window is maximized.

Azure Data Studio version: azuredatastudio 1.49.1 (2f696aa95a13ccabd7e0a4431d1d759e3a7b8cfd, 2024-08-13T18:12:38.831Z)
OS version: Windows_NT x64 10.0.19045
Restricted Mode: No
Preview Features: Enabled
Modes:

System Info

Item
Value

CPUs
AMD Ryzen 7 4800H with Radeon Graphics          (16 x 2894)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
23.37GB (10.13GB free)

Process Argv

Screen Reader
no

VM
40%

Extensions (7)

Extension
Author (truncated)
Version

admin-pack
Mic
0.0.2

admin-tool-ext-win
Mic
0.1.3

agent
Mic
0.49.0

dacpac
Mic
1.14.0

import
Mic
1.6.1

profiler
Mic
0.14.0

schema-compare
Mic
1.21.0

Duplicate of #25479

Come on, this is a serious problem.


Ensure video variants update in sync with a color scheme change

Bug/issue #, if applicable: 124102221
Summary
This fixes an issue where a video does not automatically switch to the expected variant (if multiple are available) when a user toggles from "Light" to "Dark" mode or vice-versa. After refreshing the page, the initial variant is always the correct one, but it does not dynamically switch as expected when using the color scheme toggle.
I'd love to hear if there is a better way to fix this or if I'm missing something obvious, because I don't think a key should strictly be needed in this situation, but I may be overlooking something. I tried a couple of other things that seemed like more appropriate fixes, but neither worked:

add a new normalizedPath variable to data and update it using a watcher on the videoAttributes computed property—it does get updated when toggle is changed, but the DOM doesn't update
explicitly call this.$forceUpdate() as part of the above approach—still doesn't update

The key approach works since it forces Vue to not re-use DOM elements for videos with different URLs, but I feel like it shouldn't be necessary...so open to other ideas if I'm missing the true root issue :)
Testing
Steps:

Add a video directive (example: @Video(source: test-video)) to a DocC article and make sure that both light/dark variants of the asset are provided in the catalog (example: test-video.mp4 and test-video~dark.mp4).
Build this branch and use it when previewing the catalog with DocC (example: DOCC_HTML_DIR=/path/to/swift-docc-render/dist docc preview Example.docc)
Verify that the video switches from the light variant to the dark variant when switching from light to dark using the color scheme toggle (and vice-versa)
Verify that there are no other video-related regressions introduced by this change

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[ ] Added tests (not really a good/useful way to test the key attribute)
[X] Ran npm test, and it succeeded
[X] Updated documentation if necessary

@swift-ci test


MTA-3416: Adding requirements for Java analyzer and discovery task

JIRA

MTA-3416

PREVIEW

Adding minimum requirements for Java analyzer and discovery

I would make it clear that this is also the default, that you don't have to set it.


Add support for Django 2.2, DRF 3.9, and Python 3.7; drop support for Python 3.4 and DRF < 3.6.3

Python 3.4 will reach its EOL in March.
Only django rest framework 3.6.3+ supports Django 1.11+.

I had to switch to multiple jobs on travis.  Otherwise the tests would not pass after the switch to Python 3.7 (for details see the message of commit https://github.com/percipient/django-querysetsequence/commit/f5aa145cc2b4e44e9b068f3933b797980c79f036).
Multiple jobs also allow to run tests against Django master while allowing them to fail.  This is relevant because Django master no longer includes django.utils.six.  The simplest fix would be to drop support for Python 2.7 (https://pythonclock.org/, https://python3statement.org/).

Thanks for this PR! I'll take a look at it soon, hopefully.

Looks good! Thanks.


Fix github workflow to run unit tests

Currently the github workflow in .github/workflows/maven.yaml for running tests does not actually run the tests.

Also involves fixing all unit tests.


Web canvaskit renderer version doesn't work

Running a web based version with flutter build web --web-renderer canvaskit doesn't work.
Error is: Permission denied NoSuchMethodError: method not found: "BrowserMultiFormatReader" on null.
It works fine with flutter build web --web-renderer html, but I need the canvaskit renderer.
Using flutter 3.10.6 and mobile_scanner 3.3.0. Issue may be related to #222
Code snippet
late final MobileScannerController _controller = MobileScannerController(formats: [BarcodeFormat.qrCode]);

...
...
MobileScanner(
                  controller: _controller,
                  errorBuilder: (context, error, child) {
                    return CameraErrorWidget(error: error);
                  },
                  onDetect: _onDetect,
                ))
...
...

Seems to be a duplicate of https://github.com/juliansteenbakker/mobile_scanner/issues/622

As mentioned, this is a duplicate of #622 , which has since been fixed.


chore(release): elements-core v7.1.5

Needed for: https://github.com/stoplightio/platform-internal/issues/7052
Also bumps markdown-viewer to 5.1.0(this changes the MarkdownViewer components class, which affects the styling) - for details see: https://github.com/stoplightio/markdown-viewer/pull/100
Releases:

elements-core: 7.1.4 -> 7.1.5
elements: 7.0.2 -> 7.0.3
elements-dev-portal: 1.0.4 -> 1.0.5

I didn't mean to tell you to release an elements / dev-portal version. 😅  Only that the dep needs updating. But I guess, it's OK, we wanted to do a release soon anyway.

I've the width of sl-markdown-viewer to width: 0 in the stylesheet due to the bug mentioned here: https://bugs.chromium.org/p/chromium/issues/detail?id=464210


Support fort Opel/Vauxhall ?

Hi,
I just bought an open/Vauxhall astra k 2017 full options and I was wondering if open pilot is compatible ? I'm getting mixed messages from the info online!
Cheers

Might be known as Holden in the US


⚠ ClamAV has degraded performance

In 2067841, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 74 ms

Resolved: ClamAV performance has improved in e5076ba.


Add --network option to update and encrypt gce

Allows user to specify a non-default network
to launch encryptor instances in
Testing: encrypted and updated gce images

Where are we validating if the network entered is valid? Or are we just letting GCE APIs error out if that happens to be the case?

we don't check that the network is valid. It will fail when launching the encryptor if the network is invalid

Bobby's approach should be ok, as long as we fail quickly, we don't show a traceback, and the error message is clear.  If this isn't the case, we should be proactive about validating user input.

lgtm


Outlines

Isn't there a polyline branch that this depends on? I think we should merge that first

I thought we already did that

Nope, there hasn't been a PR made for the polylines branch yet

Master was merged on Polyline. Polyline is on PR. Polyline was merged on outline.
In Polygon Style I have left commented a water outline example for testing


VSecm Sentinel should have the ability to generate random passwords and assign it to workloads upon initialization (based on a env-var setting)

When bound via an environment variable sentinel should register auto-generated secrets to workload upon initialization (if such a secret does not exist)
This will allow the cluster components to self-initialize without having any operator intervention.

Related to #167 .

Closing this.
I’m not sure what I meant when I opened the ticket.
We have the ability to generate randomized secrets; that should be good enough.
The rest can be scriptable anyway.


Clean up boundbox

There are several implementations of bounding box/ or extents. consolidate and clean these various implementations and the code the depends on them. Renaming it to Extents by the end.
Part of #269

Going through the various bounding box (and extent) implementations I come to the conclusion that the way the clipping algorithms is using bounding box is wrong. They are assuming that a bounding box is made up of two points the (upper left) and (lower right) which in mvt happen to be the minx, miny and maxx, maxy points. I thinking it makes more sense to have bbox be [4]float64 as it's not really two points as I was previously thinking.

@gdey I'm not quite sure I follow. What's the difference between [2][2]float64 and [4]float64? Seem like you need to know what is the min / max is if you're going to access the values directly or you will need to leverage MinX/Y() MaxX/Y() methods. If we stick with [2][2]float64 we can leverage the geom package. Maybe a New([2]float64, [2]float64) would be helpful and it could figure out what the min / max points should be?


Type Error in tabBar={(props) => <AnimatedTabBar {...props} tabs={tabs} />

Bug
Environment info
"@react-navigation/bottom-tabs": "^5.11.2",
"@gorhom/animated-tabbar": "^2.1.0",
"react-native": "0.63.4",
"react-native-gesture-handler": "^1.9.0",
"react-native-reanimated": "^1.13.2",
Steps To Reproduce
When I add tabBar={(props) => <AnimatedTabBar {...props} tabs={tabs} /> in BottomTab.Navigator from README
TS says

Type '{ tabs: TabsConfig<BubbleTabBarItemConfig, { [key: string]: BubbleTabBarItemConfig; }>; keyboardHidesTabBar?: boolean | undefined; ... 15 more ...; navigation: NavigationHelpers<...>; }' is not assignable to type '{ preset?: "bubble" | "flashy" | "material" | undefined; tabs: TabsConfig<BubbleTabBarItemConfig | FlashyTabBarItemConfig | MaterialTabBarItemConfig, { ...; }>; ... 6 more ...; safeAreaInsets?: Insets | undefined; }'.
Types of property 'style' are incompatible.
Type 'false | RegisteredStyle | Value | AnimatedInterpolation | WithAnimatedObject | WithAnimatedArray<...> | null | undefined' is not assignable to type 'StyleProp'.
Type 'Value' is not assignable to type 'StyleProp'.
Type 'Value' is missing the following properties from type 'RecursiveArray<false | ViewStyle | RegisteredStyle | null | undefined>': length, pop, push, concat, and 26 more.

Describe what you expected to happen:
No type error with this awesome package
Reproducible sample code
    <BottomTab.Navigator
      // screenOptions={TabBarVisibleOnRootScreenOptions}
      initialRouteName={initialTabRoute}
      tabBar={(props) => <AnimatedTabBar {...props} tabs={tabs} />}
    >
      <BottomTab.Screen name={APP_SCREEN.TAB_PRODUCTS} component={ProductsNavigator} />
      <BottomTab.Screen name={APP_SCREEN.SETTINGS} component={SettingsScreen} />
    </BottomTab.Navigator>
  );

Also having this issue. Here's my full code: https://pastebin.com/raw/mgq1s8fn (most of it copied directly from the readme).
Is this project still being maintained?

Also having this issue. Here's my full code: https://pastebin.com/raw/mgq1s8fn (most of it copied directly from the readme).
Is this project still being maintained?

hi @JaeWangL , @r0hin , thanks fo submitting this issue , i will look into to it later today 👍

hi @JaeWangL , @r0hin , thanks fo submitting this issue , i will look into to it later today 👍

@JaeWangL could you share your tabs object ?

@JaeWangL could you share your tabs object ?

@r0hin you need to provide a function component
in your pastebin
...
      component: props => <Feather {...props} name="home" size={24} color="black" />,
...

@r0hin you need to provide a function component
in your pastebin
...
      component: props => <Feather {...props} name="home" size={24} color="black" />,
...

My apologies if I am doing this completely wrong but after replacing the icon components to what you have pasted above, there is still an error.
Full Code: https://pastebin.com/raw/JTPqun02
Error: https://pastebin.com/raw/VGUYfFFD
Dependencies: https://pastebin.com/raw/36wjCeJW
Thanks for your help!

My apologies if I am doing this completely wrong but after replacing the icon components to what you have pasted above, there is still an error.
Full Code: https://pastebin.com/raw/JTPqun02
Error: https://pastebin.com/raw/VGUYfFFD
Dependencies: https://pastebin.com/raw/36wjCeJW
Thanks for your help!

Confronting the same error.
This might be a type error in AniamtedTabBarProps where
    /**
     * Root container style.
     */
    style?: StyleProp<ViewStyle>;

should be replaced with
    /**
     * Root container style.
     */
    style?: Animated.WithAnimatedValue<StyleProp<ViewStyle>>;

as seen inside in react-navigation:
https://github.com/react-navigation/react-navigation/blob/main/packages/bottom-tabs/src/types.tsx#L256
I'm not sure if it affects other types.

Confronting the same error.
This might be a type error in AniamtedTabBarProps where
    /**
     * Root container style.
     */
    style?: StyleProp<ViewStyle>;

should be replaced with
    /**
     * Root container style.
     */
    style?: Animated.WithAnimatedValue<StyleProp<ViewStyle>>;

as seen inside in react-navigation:
https://github.com/react-navigation/react-navigation/blob/main/packages/bottom-tabs/src/types.tsx#L256
I'm not sure if it affects other types.

Thanks @maximiliankaske for investigating it, ill work on a fix asap 👍

Thanks @maximiliankaske for investigating it, ill work on a fix asap 👍

fixed with v2.1.1  🤟

fixed with v2.1.1  🤟


Is it possible to support images in jfif format?

Hello.
This is my first time asking a question, so I apologize if this is weird.
I hired another engineer to use tweepy, and he had a problem where he couldn't download some images.
The reason seems to be that it doesn't support jfif, is it possible to fix it?
Sorry. I'm not an engineer, so I don't know the details.
It would be very helpful if you could help.

What exactly is the issue that he has with downloading the images? If they are showing in the entities object of the status, he could try downloading those images from those URL directly

I'm not sure what support for JFIF images means here.
I don't think the Twitter API uses JFIF images anywhere, Tweepy itself does not provide a specific means of downloading images (you can do so using the URLs provided by the Twitter API / Tweepy), and generally, there doesn't need to be "support" for specific file formats when downloading them.


handle pending minion tasks properly when getting the task progress status

bugfix: when subtasks are waiting in Helix, the get task progress endpoint failed with NPE as those pending subtasks don't have assigned workers yet. This PR handles pending tasks properly.
Validated that the Pinot UI tab Minion Task Manager could show the task progress properly now even with pending subtasks.

Codecov Report

Merging #9911 (a0446b3) into master (ecf41be) will decrease coverage by 48.44%.
The diff coverage is 70.83%.

@@              Coverage Diff              @@
##             master    #9911       +/-   ##
=============================================
- Coverage     64.30%   15.86%   -48.45%
+ Complexity     5034      175     -4859
=============================================
  Files          1928     1928
  Lines        103874   103882        +8
  Branches      15823    15826        +3
=============================================
- Hits          66793    16477    -50316
- Misses        32254    86190    +53936
+ Partials       4827     1215     -3612

Flag
Coverage Δ

unittests1
?

unittests2
15.86% <70.83%> (+0.05%)
:arrow_up:

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

Impacted Files
Coverage Δ

.../pinot/common/function/scalar/StringFunctions.java
0.00% <0.00%> (-56.25%)
:arrow_down:

...lix/core/minion/PinotHelixTaskResourceManager.java
14.35% <77.27%> (+1.03%)
:arrow_up:

...src/main/java/org/apache/pinot/sql/FilterKind.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ain/java/org/apache/pinot/core/data/table/Key.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...in/java/org/apache/pinot/spi/utils/BytesUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/java/org/apache/pinot/core/data/table/Table.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../java/org/apache/pinot/core/data/table/Record.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../java/org/apache/pinot/core/util/GroupByUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/org/apache/pinot/spi/trace/BaseRecording.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/org/apache/pinot/spi/trace/NoOpRecording.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 1223 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Extract git-utils.ts

This is a refactoring PR that addresses the TODO item left from https://github.com/github/codeql-action/pull/2559.
Merge / deployment checklist

[x] Confirm this change is backwards compatible with existing workflows.
[x] Confirm the readme has been updated if necessary.
[x] Confirm the changelog has been updated if necessary.

Thanks for this! Are there any other changes beyond reorganizing the files? I didn't see anything obvious, but wanted to double check.

This is a pure refactoring PR, and there are no changes beyond reorganizing the files.


Create a Navbar-4

One need to make this component using HTML and Tailwind CSS. I would suggest to use Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.

After building the component please raise a PR with a screenshot of the component and add the component in Navbar/navbar-nav4.html.
If you need to use any icon please use it from Hero Icons
Good luck.

I want to contribute to this.

You are already assigned to an issue. @20RitikSingh

@Rajdip019 @rupsha014 please assign me

You are already assigned to two issue. @rohan9024

@Rajdip019 Created PR for both of them😎....... please assign this issue as well

Ha ha slow down. @rohan9024 . By the way assigning you.

@Rajdip019 Thank you!


Add maven libraries

Issue #58
Sample:
keycloak = new KeycloakContainer()
    .withProviderClassesFrom("target/classes");
    .withMavenDependencies("org.apache.commons:commons-dbcp2:2.9.0",
				"G:A:");

This copy transients dependencies too.

Same as with your other PR. Please don't merge other project branches into your PR branch, just rebase your branch onto the main branch. This keeps the Git history clear.
If it's possible to you, please reset your PR branch to the commit before the merge commit, thanks.

I rebased my branch with another branch in my repository, I don't know why Github doesn't understand that.
I will take a step back, understand better your proposal in this library before proposing a PR.


Static assets having duplicate file revision when generating Service Worker precache

Hello, I'm currently trying to configure the Service Worker precache so that I can cache all my files when installing the application but I noticed a strange behavior when trying different approaches using the Vite PWA docs.

Scenario 1: don't include assets nor use workbox glob patterns

If I configure the VitePWA plugin using the following configuration:
VitePWA({
  // Public resources
  // includeAssets: [],

  // Manifest configuration.
  // https://vite-pwa-org.netlify.app/guide/pwa-minimal-requirements.html
  manifest: {
    ...,
    icons: [
      {
        src: 'pwa-64x64.png',
        sizes: '64x64',
        type: 'image/png',
      },
      {
        src: 'pwa-192x192.png',
        sizes: '192x192',
        type: 'image/png',
      },
      {
        src: 'pwa-512x512.png',
        sizes: '512x512',
        type: 'image/png',
        purpose: 'any',
      },
      {
        src: 'maskable-icon-512x512.png',
        sizes: '512x512',
        type: 'image/png',
        purpose: 'maskable',
      },
    ],
  },

  // Workbox configuration.
  // workbox: {
  //   globPatterns: ['**/*.{js,css,html,ico,png,svg}'],
  // },
}),

Then, it would result in the following Service Worker precache after build. Please note that public/ files are not being included as part of the precache array (e.g. apple-touch-icon, favicon.ico, logo.svg, robots.txt etc.), at least not like the Static assets handling page statement.
e.precacheAndRoute(
  [
    { url: 'assets/index.0fb3162c.js', revision: null },
    { url: 'assets/index.f0b36889.css', revision: null },
    { url: 'assets/workbox-window.prod.es5.e0cc53cf.js', revision: null },
    { url: 'index.html', revision: 'ecfa72ef5ede33cddcab4b29ddfff4ff' },
    { url: 'pwa-64x64.png', revision: '160c4d06af493fa1cb62ae9f4258003b' },
    {
      url: 'pwa-192x192.png',
      revision: '9397e91d5e3a42bfa03ce2d07f7e3171',
    },
    {
      url: 'pwa-512x512.png',
      revision: '49e40aedc6b5cd06157f00d8c564e379',
    },
    {
      url: 'maskable-icon-512x512.png',
      revision: 'a51a4f0cc05f269554afef971b7f658d',
    },
    {
      url: 'manifest.webmanifest',
      revision: 'f9addcd4a864ed3a97c677e5274540c8',
    },
  ],
  {}
),

Scenario 2: include assets but don't use workbox glob patterns

Moreover, if I configure the VitePWA plugin using the following configuration instead:
VitePWA({
  // Public resources
  includeAssets: [
    'apple-touch-icon-180x180',
    'favicon.ico',
    'logo.svg',
    'robots.txt',
  ],

  // Manifest configuration.
  // https://vite-pwa-org.netlify.app/guide/pwa-minimal-requirements.html
  manifest: {
    ...,
    icons: [
      {
        src: 'pwa-64x64.png',
        sizes: '64x64',
        type: 'image/png',
      },
      {
        src: 'pwa-192x192.png',
        sizes: '192x192',
        type: 'image/png',
      },
      {
        src: 'pwa-512x512.png',
        sizes: '512x512',
        type: 'image/png',
        purpose: 'any',
      },
      {
        src: 'maskable-icon-512x512.png',
        sizes: '512x512',
        type: 'image/png',
        purpose: 'maskable',
      },
    ],
  },

  // Workbox configuration.
  // workbox: {
  //   globPatterns: ['**/*.{js,css,html,ico,png,svg}'],
  // },
}),

Then, it would result in the following Service Worker precache after build, which is more accurate to what I'm looking for. However, I'm still missing other valuable assets that were auto-generated by Vite using a hash in their name.
e.precacheAndRoute(
  [
    { url: 'assets/index.0fb3162c.js', revision: null },
    { url: 'assets/index.f0b36889.css', revision: null },
    { url: 'assets/workbox-window.prod.es5.e0cc53cf.js', revision: null },
    { url: 'index.html', revision: 'ecfa72ef5ede33cddcab4b29ddfff4ff' },
    { url: 'favicon.ico', revision: '547587c645ecf0155439c8e0edb6cb3d' },
    { url: 'logo.svg', revision: '40dcf9f191738beb353d7a53edb8abd9' },
    { url: 'robots.txt', revision: 'f77c87f977e0fcce05a6df46c885a129' },
    { url: 'pwa-64x64.png', revision: '160c4d06af493fa1cb62ae9f4258003b' },
    {
      url: 'pwa-192x192.png',
      revision: '9397e91d5e3a42bfa03ce2d07f7e3171',
    },
    {
      url: 'pwa-512x512.png',
      revision: '49e40aedc6b5cd06157f00d8c564e379',
    },
    {
      url: 'maskable-icon-512x512.png',
      revision: 'a51a4f0cc05f269554afef971b7f658d',
    },
    {
      url: 'manifest.webmanifest',
      revision: 'f9addcd4a864ed3a97c677e5274540c8',
    },
  ],
  {}
),

Scenario 3: don't include assets but use workbox glob patterns

Furthermore, if I configure the VitePWA plugin using the following configuration:
VitePWA({
  // Public resources
  // includeAssets: [],

  // Manifest configuration.
  // https://vite-pwa-org.netlify.app/guide/pwa-minimal-requirements.html
  manifest: {
    ...,
    icons: [
      {
        src: 'pwa-64x64.png',
        sizes: '64x64',
        type: 'image/png',
      },
      {
        src: 'pwa-192x192.png',
        sizes: '192x192',
        type: 'image/png',
      },
      {
        src: 'pwa-512x512.png',
        sizes: '512x512',
        type: 'image/png',
        purpose: 'any',
      },
      {
        src: 'maskable-icon-512x512.png',
        sizes: '512x512',
        type: 'image/png',
        purpose: 'maskable',
      },
    ],
  },

  // Workbox configuration.
  workbox: {
    globPatterns: ['**/*.{js,css,html,ico,png,svg}'],
  },
}),

Then, it would result in the following Service Worker precache after build. Which is exactly what I'm looking for, but then you may notice that there are duplicates in the output from the manifest itself (e.g. pwa-{size} images).
e.precacheAndRoute(
      [
        {
          url: 'apple-touch-icon-180x180.png',
          revision: '9b7a985648e65a01c130fa6e22b2459b',
        },
        { url: 'assets/app-logo.007611f1.svg', revision: null },
        { url: 'assets/close.915249f6.svg', revision: null },
        { url: 'assets/image1.47eb2262.png', revision: null },
        { url: 'assets/image2.82abb677.png', revision: null },
        { url: 'assets/image3.0727d240.png', revision: null },
        { url: 'assets/image4.c21696be.png', revision: null },
        { url: 'assets/image5.976b0ba1.png', revision: null },
        { url: 'assets/index.0fb3162c.js', revision: null },
        { url: 'assets/index.f0b36889.css', revision: null },
        { url: 'assets/settings.e67704e2.svg', revision: null },
        { url: 'assets/workbox-window.prod.es5.e0cc53cf.js', revision: null },
        { url: 'favicon.ico', revision: '547587c645ecf0155439c8e0edb6cb3d' },
        { url: 'index.html', revision: 'ecfa72ef5ede33cddcab4b29ddfff4ff' },
        { url: 'logo.svg', revision: '40dcf9f191738beb353d7a53edb8abd9' },
        {
          url: 'maskable-icon-512x512.png',
          revision: 'a51a4f0cc05f269554afef971b7f658d',
        },
        {
          url: 'pwa-192x192.png',
          revision: '9397e91d5e3a42bfa03ce2d07f7e3171',
        },
        {
          url: 'pwa-512x512.png',
          revision: '49e40aedc6b5cd06157f00d8c564e379',
        },
        { url: 'pwa-64x64.png', revision: '160c4d06af493fa1cb62ae9f4258003b' },
        { url: 'pwa-64x64.png', revision: '160c4d06af493fa1cb62ae9f4258003b' },
        {
          url: 'pwa-192x192.png',
          revision: '9397e91d5e3a42bfa03ce2d07f7e3171',
        },
        {
          url: 'pwa-512x512.png',
          revision: '49e40aedc6b5cd06157f00d8c564e379',
        },
        {
          url: 'maskable-icon-512x512.png',
          revision: 'a51a4f0cc05f269554afef971b7f658d',
        },
        {
          url: 'manifest.webmanifest',
          revision: 'f9addcd4a864ed3a97c677e5274540c8',
        },
      ],
      {}
    ),

I don't think having duplicates is such a problem since they have the same revision, but still it's unclear to me what approach should I take considering:
a. It looks like the Static assets handling page is not accurate based on my tests
b. Including assets manually through includeAssets helps on adding public files into the precache, but it won't help for other files
c. The most accurate approach, which is using globPatterns is causing duplicate file revisions
Last but not least, I was hoping to develop my own Service Worker using injestManifest strategy instead of generateSW, which only requires me to change from workbox.globPatterns to injectManifest.globaPatterns. However, I'm facing the exact same issue having duplicate file revisions, so it's not up to the strategy I'm using.
Thanks in advance!

EDIT: I forgot to share the Dev Tools output that shows up all files just once (which is good!), so the issue would remain on the precache generation only, which seems like a minor thing.

Omitted custom images and icons from the screenshot, leaving just manifest files.

@lgaspari weird, I have publicsh the new docs for my vuetifnu nuxt module (VitePress) and the sw is fine:
https://vuetify-nuxt-module.netlify.app/
I'll check running the examples in the vite-plugin-pwa repo.
are you using Vite or some meta framework (SvelteKit, Nuxt...)?

@lgaspari includeManifestIcons: false when including png extension, all pwa icons are included by default, I'll try to fix it, can you confirm it?

are you using Vite or some meta framework (SvelteKit, Nuxt...)?

Just plain Vite for now.

@lgaspari includeManifestIcons: false when including png extension, all pwa icons are included by default, I'll try to fix it, can you confirm it?

Actually, this made the trick! It looks like using both features together causes the duplicate file revision, so I could just do:
{
  ...

  // Don't include manifest icons in favor of `injectManifest.globPatterns`.
  includeManifestIcons: false,

  // Inject manifest options.
  injectManifest: {
    globPatterns: ['**/*.{js,css,html,ico,png,svg}'],
  },

  ...
}

I guess we can improve this function https://github.com/vite-pwa/vite-plugin-pwa/blob/main/src/assets.ts#L47, called to include pwa icons and shorcuts icons

That would be good, like either use just globPatterns or fallback to includeManifestIcons + includeAssets.

Yeah, I've been doing some extra runs on my end and that seems exactly what I needed for adding not only public/ files (from includeAssets option) but also any other asset found in the project (from workbox.globPatterns option) whilst avoiding duplicate file revision adding includeManifestIcons: false.
@userquin could you confirm this behavior below?
// meant for public/* files only
includeAssets: [
  // works
  'apple-touch-icon-180x180.png',
  'favicon.ico',
  'logo.svg',
  'robots.txt',

  // won't match src/assets/* files
  '**/*.png',
],

// meant for the whole project, including public/* files
workbox: {
  // will match most files
  globPatterns: ['**/*.{js,css,html,ico,png,svg}'],
},

No, workbox will use glogPatterns in the dist folder only. The options included in the pwa will generate the revision from the public folder, Vite will copy public folder after workbox build.

No, workbox will use glogPatterns in the dist folder only.

Right, I meant the dist/ folder, my bad.

The options included in the pwa will generate the revision from the public folder, Vite will copy public folder after workbox build.

Hmm, if Vite copies the public/ folder after workbox build, how is it possible to have those files listed in the precache then?
These are my runs:
Run 1

config:
includeAssets: [
  'apple-touch-icon-180x180.png',
],

output:
e.precacheAndRoute(
  [
    {
      url: 'apple-touch-icon-180x180.png',
      revision: '9b7a985648e65a01c130fa6e22b2459b',
    },
  ]

Run 2

config:
includeAssets: [
  'apple-touch-icon-180x180.png',
],
workbox: {
  globPatterns: ['**/*.{js,css,html,ico,png,svg}'],
},

output:
e.precacheAndRoute(
  [
    {
      url: 'apple-touch-icon-180x180.png',
      revision: '9b7a985648e65a01c130fa6e22b2459b',
    },
    {
      url: 'apple-touch-icon-180x180.png',
      revision: '9b7a985648e65a01c130fa6e22b2459b',
    },
  ]

Run 3

config:
workbox: {
  globPatterns: ['**/*.{js,css,html,ico,png,svg}'],
},

output:
e.precacheAndRoute(
  [
    {
      url: 'apple-touch-icon-180x180.png',
      revision: '9b7a985648e65a01c130fa6e22b2459b',
    },
  ]

The same would happen to every file listed in the includeAssets config (e.g. favicon.ico, logo.svg) that matches the globPatterns. In case of robots.txt, I'd keep it in the includeAssets array, but I could def add .txt to the glob pattern.
Let me know if you need a further example, I'm just trying to wrap my head around how it works to fine-tune the configuration based on that.

We include them using manifestEntries when listed in includeAssets and includeManifestIcon enabled

Check the assets module

Gotcha! I thought they were kind of exclusive at moment of writing my above comment without having double-checked the code as you pointed out.
The issue can be closed since the solution to my problem was given in https://github.com/vite-pwa/docs/issues/67#issuecomment-1644155534 by @userquin. Thanks a lot dude! 💯


tf.placeholder don't exist in tf 2.0

Describe the Bug
On tensorflow 2.0, there is no tf.placeholder
Traceback (most recent call last):
  File "load_and_extract.py", line 18, in <module>
    model = load_trained_model_from_checkpoint(config_path, checkpoint_path)
  File "/usr/local/lib/python3.5/dist-packages/keras_bert/loader.py", line 43, in load_trained_model_from_checkpoint
    training=training,
  File "/usr/local/lib/python3.5/dist-packages/keras_bert/bert.py", line 58, in get_model
    inputs = get_inputs(seq_len=seq_len)
  File "/usr/local/lib/python3.5/dist-packages/keras_bert/layers/inputs.py", line 15, in get_inputs
    ) for name in names]
  File "/usr/local/lib/python3.5/dist-packages/keras_bert/layers/inputs.py", line 15, in <listcomp>
    ) for name in names]
  File "/usr/local/lib/python3.5/dist-packages/keras/engine/input_layer.py", line 178, in Input
    input_tensor=tensor)
  File "/usr/local/lib/python3.5/dist-packages/keras/legacy/interfaces.py", line 91, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/keras/engine/input_layer.py", line 87, in __init__
    name=self.name)
  File "/usr/local/lib/python3.5/dist-packages/keras/backend/tensorflow_backend.py", line 517, in placeholder
    x = tf.placeholder(dtype, shape=shape, name=name)
AttributeError: module 'tensorflow' has no attribute 'placeholder'

Version Info
tf-nightly-gpu-2.0-preview       2.0.0.dev20190304
Keras                            2.2.4
python 3.5.2

There is nothing I can do because the incompatibility happens between Keras and Tensorflow. Keras has a branch named "tf-2" that solves the compatibility problem with Tensorflow 2.0. I don't think you should use a preview version of Tensorflow while Keras has no stable release for it.

   else:         tf.compat.v1.disable_eager_execution()         x = tf.compat.v1.placeholder(dtype, shape=shape, name=name)
File "/usr/local/lib/python3.5/dist-packages/keras/backend/tensorflow_backend.py", line 517
In here I change like this...
For similar error, I have to change it manually.

There is nothing I can do because the incompatibility happens between Keras and Tensorflow. Keras has a branch named "tf-2" that solves the compatibility problem with Tensorflow 2.0. I don't think you should use a preview version of Tensorflow while Keras has no stable release for it.

oh, thank you very much


how do i actually install this on my 3ds?

how do i actually install this on my 3ds? i have no idea how to do that but I really want to.

@EanimeiCCC There's 2 ways to run homebrew applications like this: either using a flashcartridge (R4 and all kinds of copies/clones/variants) or by hacking the 3DS so you can put it on the SD-card.
There is a 'flashcarts' reddit which has a nice overview of cartridges you can get (the AliExpress ones are cheap and will do the job), but if you really want to get the full potential of your 3DS, you should search for '3DS Hacks Guide'.

on flashcart: put the .nds application in the root of the card along with the uxn folder. When you turn on your 3DS, you'll be albe to see your flashcart just like you would see the icon of a game that is inserted. When opening that, you'll see the uxn application and you can open that up.
on a hacked 3DS: you'll need to install 'Twilight ++', this is an applicaton which can run DS applications and games. After completing the hacking guide, you'll have 1 or more applications that are sort of stores for homebrew applications. They will have it listed so you can simply click and install it.

I recommend installing nds-hb-menu over TWLMenu++ at this time, as the latter includes DRM circumvention game patches and other features which may be legally dubious in some jurisdictions, and are besides only useful for running pirated games. It's also a 40MB+ install versus, like, a few hundred kilobytes, and takes longer to boot...

@EanimeiCCC There's 2 ways to run homebrew applications like this: either using a flashcartridge (R4 and all kinds of copies/clones/variants) or by hacking the 3DS so you can put it on the SD-card. There is a 'flashcarts' reddit which has a nice overview of cartridges you can get (the AliExpress ones are cheap and will do the job), but if you really want to get the full potential of your 3DS (eg. install other homebrew apps or games), you should search for '3DS Hacks Guide' and follow that. Takes more time though, and has some risk to it.

on flashcart: put the .nds application in the root of the card along with the uxn folder. When you turn on your 3DS, you'll be able to see your flashcart just like you would see the icon of a game that is inserted. When opening that, you'll see the uxn application and you can open that up.
on a hacked 3DS: you'll need to install 'Twilight ++', this is an application which can run DS applications and games. After completing the hacking guide, you'll have 1 or more applications that are sort of stores for homebrew applications. They will have it listed so you can simply click and install it.

which store is it in? i couldn't seem to find it.

@EanimeiCCC You should be able to find both HBmenu and TWiLight Menu++ using the Universal Updater application.

@EanimeiCCC You should be able to find both HBmenu and TWiLight Menu++ using the Universal Updater application.

so I managed to get the thing on it put I still can't seem to get it to work. i put the uxnds file in and it opens but it's just blank

@EanimeiCCC When using uxnds_debug.nds or uxnds_profile.nds, what error or warning do you get?

@EanimeiCCC When using uxnds_debug.nds or uxnds_profile.nds, what error or warning do you get?

nothing, I think I got it mostly working now though. the screen on the top is cutting off some and it won't let me go back to the menu properly for some reason


Random Forest: m_random and parallel RNG

Hello, I would first like to thank you for making such an excellent and accessible repo. We're getting quite a bit of use out of it in multiple projects.
I've noticed that despite setting the seed in the Random Forest learner, I get slightly varying results from run-to-run given identical inputs. I suspect the problem is in the following code block:
Parallel.ForEach(rangePartitioner, (work, loopState) =>
{
   results.Add(CreateTree(observations, targets, indices, new Random(m_random.Next())));
});

The parallel random number generation is a problem; the same random numbers are generated because the seed is set in the constructor, but there is a "race" between the threads to grab the next random number. Locking m_random would not help, I think. This should be fixed by generating a random number for each tree prior to entering the Parallel.ForEach loop, like so:
int[] randomNumbers = new int[m_trees];
for(int i = 0; i < randomNumbers.Length; i++)
{
   randomNumbers[i] = m_random.Next();
}
Parallel.ForEach(rangePartitioner, (work, loopState) =>
{
   results.Add(CreateTree(observations, targets, indices, randomNumbers[work]));
});

Let me know if I'm overlooking something. I'd be happy to fork and make a pull request.
Rob

Hi @rmillikin,
Thanks for using SharpLearning. I am glad you find it useful!
I think you are correct about the seeding and the "race" between threads. It is something I have been wanting to fix for a while.
I believe your solution should work with a small change, you still need to create a Random for each tree, but the seeding should be created prior to entering the Parral.ForEach loop as in your suggestion:
var randomSeeds = new int[m_trees];
for(int i = 0; i < randomSeeds.Length; i++)
{
   randomSeeds[i] = m_random.Next();
}
Parallel.ForEach(rangePartitioner, (work, loopState) =>
{
   results.Add(CreateTree(observations, targets, indices, new Random(randomSeeds[work])));
});

You are very welcome to fork and make a pull request - that would be a great help.
best regards
Mads

Good catch! I didn't realize each tree needed a new Random. I'll make a pull request soon.
Rob

@rmillikin The issue should now be fixed with PR #67. The new version is available on nuget (v. 0.26.9).

Great to hear! Thanks for putting in the work to figure this out


multi files in player

dropdown or button to select which file, like you uploaded 2 files one in english and another in french, so in the player menu you can select wich one you want to watch

Have a look at my enhancement on enabling multiple audio streams #225 :)


1 convert docx to pdf

Convert docx and word doc files to pdf.
Using LibreOffice headless. Run as python subprocess.
Another subprocess is used to rename the output file in desired format.

Got a build warning
 1 warning found (use --debug to expand):
 - WorkdirRelativePath: Relative workdir "./" can have unexpected results if the base image changes (line 10)

Got a build warning
 1 warning found (use --debug to expand):
 - WorkdirRelativePath: Relative workdir "./" can have unexpected results if the base image changes (line 10)

removed workdir from dockerfile

Error when pushing to hub.ncsa.illinois.edu
ubuntu@minum:~/extractors-soffice$ docker image push hub.ncsa.illinois.edu/clowder/soffice-extractor:0.1.1
The push refers to repository [hub.ncsa.illinois.edu/clowder/soffice-extractor]
e428be4ee250: Pushed
5f9a2e6a496a: Pushed
155b5094e0bf: Pushed
b12195f958d1: Pushing [==================================================>]  1.794GB/1.794GB
037d144dc18b: Mounted from clowder/rct-transparency-extractor
439d9e503eb9: Mounted from clowder/rct-transparency-extractor
8d280ca3a828: Mounted from clowder/rct-transparency-extractor
3a8081ce85fa: Mounted from clowder/rct-transparency-extractor
045d8b74bf0d: Mounted from clowder/rct-transparency-extractor
25879f85bbb0: Mounted from clowder/rct-transparency-extractor
6abe10f2f601: Mounted from clowder/rct-transparency-extractor
unknown: Client Closed Request

Deployed

Error when pushing to hub.ncsa.illinois.edu
ubuntu@minum:~/extractors-soffice$ docker image push hub.ncsa.illinois.edu/clowder/soffice-extractor:0.1.1
The push refers to repository [hub.ncsa.illinois.edu/clowder/soffice-extractor]
e428be4ee250: Pushed
5f9a2e6a496a: Pushed
155b5094e0bf: Pushed
b12195f958d1: Pushing [==================================================>]  1.794GB/1.794GB
037d144dc18b: Mounted from clowder/rct-transparency-extractor
439d9e503eb9: Mounted from clowder/rct-transparency-extractor
8d280ca3a828: Mounted from clowder/rct-transparency-extractor
3a8081ce85fa: Mounted from clowder/rct-transparency-extractor
045d8b74bf0d: Mounted from clowder/rct-transparency-extractor
25879f85bbb0: Mounted from clowder/rct-transparency-extractor
6abe10f2f601: Mounted from clowder/rct-transparency-extractor
unknown: Client Closed Request

Resolved error

Deployed to hub.ncsa.illinois.edu/clowder/extractors-soffice:0.1.0

There is an issue with rendering these converted pdf documents. All highlights are being displayed in the first pages. (sort of overwritten). The grobid outputs (json file with sentence coordinates) seem to be correct. Investigating this issue,.

There is an issue with rendering these converted pdf documents. All highlights are being displayed in the first pages. (sort of overwritten). The grobid outputs (json file with sentence coordinates) seem to be correct. Investigating this issue,.

This issue was because of the file-previewer extractor giving the output preview of the pdf file as type 'thumbnail'. See this issue for details.

Deployed. Tested. works fine. Merging to main.


a61eb89 changes selection nodes functionality

Hi, I have found out that https://github.com/CESNET/netopeer2/commit/a61eb8983e5253bed6c770752cbc924b508509f9 changes how selection nodes work.
This is my schema:
module netopeer2-change-list {
  yang-version 1.1;
  prefix aha;
  namespace "http://example.com";

  list ports {
    key 'name';
    leaf name {
      type string;
    }

    leaf type {
      type string;
    }
  }
}

And this is my data:
<ports xmlns="http://example.com">
  <name>AHOJ</name>
  <type>AHOJ</type>
</ports>

Now if I connect to Netopeer via netopeer2-cli and use this command:
get --filter-subtree

And use this filter:
<ports xmlns="http://example.com">
  <name/>
</ports>

then, if I understand https://tools.ietf.org/html/rfc6241#section-6.2.4 correctly, I should only get this output:
<ports xmlns="http://example.com">
  <name>AHOJ</name>
</ports>

This is the way it worked until https://github.com/CESNET/netopeer2/commit/a61eb8983e5253bed6c770752cbc924b508509f9. After this commit I get this data:
<ports xmlns="http://example.com">
  <name>AHOJ</name>
  <type>AHOJ</type>
</ports>

What's your opinion on this?

Yes, I believe you are right, please follow a duplicate issue #824.

Sorry, I didn't check for duplicate issues. Thank you.

Sorry, it was not really the exact same issue, so it is not a duplicate. Should be fixed now.


Switches to net:http from net:http:persistent. Bumps version to 2.10.2."

closes #69

This is now the same as dor-services-client: https://github.com/sul-dlss/dor-services-client/blob/master/lib/dor/services/client.rb#L89


[Hub Generated] Review request for Microsoft.Billing to add version preview/2018-11-01-preview

If you are a MSFT employee you can view your work branch via this link.
Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-js

Automation for azure-sdk-for-ruby

Can one of the admins verify this patch?

Automation for azure-sdk-for-go

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/6403
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/6403/commits/f9cbc296845932be678ed9a95adce1b39d765b8f

Automation for azure-sdk-for-java

@sarang-amondikar has this API been deployed?

@jhendrixMSFT the api  is not yet deployed, we want to test it in ppe first

Sounds good, please update when the API is live so we can merge.

@AutorestCI regenerate azure-sdk-for-go

the api is now deployed to prod, could you please approve this PR?


Manual load Grpc.Core.dll then GetTypes() throws, wrong version of System.Interactive.Async

What version of gRPC and what language are you using?
1.12.0 C#
What operating system (Linux, Windows, …) and version?
Windows 10
What runtime / compiler are you using (e.g. python version or version of gcc)
Visual Studio 2017 Community
What did you do?
var assy = Assembly.UnsafeLoadFrom("Grpc.Core.dll");
var types = assy.GetTypes(); // this line throws ReflectionTypeLoadException
What did you expect to see?
No exception
What did you see instead?
ReflectionTypeLoadException with 3 LoaderExceptions, all:
"Could not load file or assembly 'System.Interactive.Async, Version=3.0.1000.0, Culture=neutral, PublicKeyToken=94bc3704cddfc263' or one of its dependencies. The system cannot find the file specified."
Anything else we should know about your project / environment?
Everything I see in the nuspec etc. says the version used is <IP_ADDRESS>, and the VS project uses <IP_ADDRESS>.
But I'm in an environment where I need to manually load the assembly. I don't know why it is looking
for 3.0.1000.0, nor is that version available anywhere that I've found.

This failure doesn't occur in the project directory where associated files are available. Closing this issue until I do some more research.


Fix Dependencies in Recipe

[x] I read the conda-forge documentation and could not find the solution for my problem there.

Issue: Dependencies install perfectly fine, but won't build when added to conda-forge feedstock (See #7 for addition)
I have tested just installed the dependencies according to feedstock manually through the following docker image, but both build perfectly fine whereas the tests in #7 keep failing for Python 3.6 and Python 3.7.
$ docker run -it centos:7 bash
$ yum install wget
$ wget https://repo.anaconda.com/miniconda/Miniconda3-latest-Linux-x86_64.sh
$ chmod +x ./Miniconda3-latest-Linux-x86_64.sh
$ ./Miniconda3-latest-Linux-x86_64.sh
$ source ~/.bashrc
$ conda create -c conda-forge -n pytkdocs-36 python=3.6 pytkdocs "astunparse>=1.6.3" "dataclasses>=0.7,<0.9" "typing_extensions>=<IP_ADDRESS>" "cached-property>=1.5.2"
$ conda create -c conda-forge -n pytkdocs-37 python=3.7 pytkdocs "astunparse>=1.6.3" "dataclasses>=0.7,<0.9" "typing_extensions>=<IP_ADDRESS>" "cached-property>=1.5.2"

@conda-forge/help-python - Could you help me with this? Thank you!


Add an uncompressed sha256 to release and streams

Moving this from
https://github.com/coreos/coreos-assembler/pull/2000#issuecomment-763152177
I'm trying to port openshift-install to streams, and it currently
is using the uncompressed sha256 - and further it's exposed in
some cases to users.
Since compression isn't always easily reproducible, I think what
cosa is doing in having both checksums is a good thing; might
as well do belt-and-suspenders.
To emphasize we already have this data from the cosa
metadata, this is just adding support for it to the stream
and release metadata.

xref https://github.com/coreos/fedora-coreos-tracker/pull/720

xref https://github.com/coreos/fedora-coreos-tracker/pull/720

xref https://github.com/coreos/coreos-assembler/pull/2018

xref https://github.com/coreos/coreos-assembler/pull/2018


ColumnSource simulator

@CristianAntochi just pointed out that ColumnSource has no simulate method. How would we implement one given that the differential rate of events in a ColumnSource are precomputed and passed in via data.

As @JelleAalbers said out during the review of #51 we could do this in the meantime:

For columnsource perhaps we can add a default simulate that returns no events and emits a warning to tell the user they should provide the method?

Implemented @JelleAalbers suggestion in #54. SourceBase now has an 'abstract' simulate method. A test is added.

Closing, since a generic ColumnSource simulate method is not meaningful / not defined. Users should provide this if using a ColumnSource. You added the warning, that's the best we can do.
We could someday add more advanced ColumnSource-like things, e.g. HistogramSource, which would have a simulate method.


Fix Issue #36 : sonic_vpp_cfg.sh should inject non-contiguous numbere…

…d interfaces
Changed the sonic_vpp_cfg.sh to pass an env variable to generate non-contiguous port names.

Please pass LANES=4 env variable to sonic_vpp_cfg.sh command to generate the required port names
sudo LANES=4 sonic_vpp_cfg -p -i linux_interface_names_comma_seperated


RCTCurrentThreadName crashing

dispatch_get_current_queue() should not be used and has been known to crash.  It seems to do so frequently on my iPhone 5 when combined with dispatch_queue_get_label().  There's no availability markup on DISPATCH_CURRENT_QUEUE_LABEL, but it should be used instead.

Fixed a couple releases back.


fix(docs): present useForm when copy form

When I tried it after the attached issue, I realized;
For someone who's absolutely copying and pasting form the guide, this would give an error, as useForm is not imported in the guide. So I fixed that.
Related issue: https://github.com/shadcn-ui/ui/issues/1482

@SoyDiego hi, any updates here?

@SoyDiego hi, any updates here?

Hi @onurhan1337 I'm not part of this project only a contributor that sometimes review code but I cannot merge or do anything.
It's difficult here, the projects need more maintainers

@SoyDiego Yes I understand you. Please don't get me wrong for tagging, it's just that since this is my first contribution to this project I'm not sure if I'm doing everything right and I wanted to ask you. Thanks for review

@SoyDiego Yes I understand you. Please don't get me wrong for tagging, it's just that since this is my first contribution to this project I'm not sure if I'm doing everything right and I wanted to ask you. Thanks for review

No problem. Looks good for me.
You can edit in your first message and write Fix #1482. Doing this you will link the PR to the issue and when they approve it and merge it, will close the issue automatically.
You have more info here https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword


Add support for MSPop

Proof-of-Possession (PoP) provides a mechanism to bind (sender-constrain) access_tokens at the application layer. PoP provides mitigation as access_token are only accepted inside a signed envelop. An access_token obtained by an attacker can only be used if the attacker has also obtained private the key associated with the access_token.
A SignedHttpRequest provides a mechanism to bind an http request to a (PoP) token by signing specific parts of the request.
Proof-of-Possession (PoP) represents a concept that can be implemented in many different ways. This implementation of PoP is based on the SignedHttpRequest spec and it's named MSPop to make a clear distinction between other existing and future PoP implementations.
SignedHttpRequest will be released as another IdentityModel assembly (Microsoft.IdentityModel.Protocols.MSPop).

Other changes:

Mark JsonWebTokenHandler.DecryptToken as public.
Add Default CallContext to ease SignedHttpRequestData instantiation.
Add basic roundtrip test.

Will be adding more tests. Pushing this PR out for an early review.
--
Resolves: #1232

Closing this PR in favor of:
https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/pull/1260


[cluster-test] added new reboot cluster experiment

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

Can you please add some more description about this experiment to the commit message and the PR?
It is important to preserve some context.
Also please provide test plan as - this is how I run experiment and this is the output of cti

Can you please add some more description about this experiment to the commit message and the PR?
It is important to preserve some context.
Also please provide test plan as - this is how I run experiment and this is the output of cti

done. Sorry forgot this part since this pr was draft

Can you also add output of the experiment in the test plan? (Just a meat of it, not build related stuff and cti links)

Also for the description - can you add it into commit message, not just PR message?

@bors-libra r=andll

:pushpin: Commit 3c214e4 has been approved by andll

:hourglass: Testing commit 3c214e4a9523e3d31f713df588d33ce970e2799a with merge b1af5a3ce6ec1514b626d50ad1a6e8cd4c6fdfad...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: andll
Pushing b1af5a3ce6ec1514b626d50ad1a6e8cd4c6fdfad to master...


How to notHotReload an entire directory

For example, if I make an update to /src/pages/Content/modules/print.js, it will hot reload (and cause any tabs running the content script to enter a refresh loop).
Has anyone found a solution to this?

No idea, frustrating bug for a while now https://github.com/lxieyang/chrome-extension-boilerplate-react/issues/151


Split a string to a list of substrings

Added snippet to split a long string to a list of substrings.
No issues. Works Perfectly.
Hope it helps.
Savio Martin

I believe string.split is already a builtin function in python
There are issues with formatting, especially with example section, please follow snippet template
Name of snippet must be a snake_case

So, What should I do now.
Please help


add docs for environment variables configuration

also update discarding_keys/index.md and link to the page.

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


⚠ Mobitopup has degraded performance

In a4e85cb, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 3012 ms

Resolved: Mobitopup performance has improved in 313f06d after 29 minutes.


GUVNOR-2341 - Remove z-index from context menu

+1 to merge.

https://github.com/uberfire/uberfire/commit/a3a5207fb6d0fe4633e0d6c5d914aeb6c9e48949


Icons are scaled weirdly when there's less space for the full name of a definition in the sidebar

See screenshot for an example.

Perhaps we should truncate the text better and allow a hover to show the full name (it'd be nice if a hover showed signatures too).

related: https://github.com/unisonweb/codebase-ui/issues/133

Design update:
Menu items should shrink such that first the definition category qualifier is removed then the text is truncated (fully visible in tooltip on hover). The icon should never scale down.

The same thing can happen with tests:


[BugFix] Restore logging for cluster load statistics

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

The logging for cluster load statistics is mistakenly removed by previous pr.
Signed-off-by: Dejun Xia<EMAIL_ADDRESS>Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_admit_test

run starrocks_fe_unittest

@Mergifyio backport branch-2.5


API simplification?

Hey! Here is a little feedback:
// Shared service
container.share("foo", () => {
    return /* object */;
});
// New service every time
container.factory("foo", (c) => {
    return /* object */;
});
// Register function
container.share("foo", container.protect(/* function */));
// Parameter
container.define("foo", /* anything */);
// Extend
container.extend("foo", (previous, c) => {
    return /* anything */;
});

Wouldn't it be possible to simplify the API? It's a lot of different use cases and to learn/remember them I think it might be challenging. Especially since there might not be a difference between registering a function and registering a parameter for example? (both are simply stored and returned)
So how about using define to register functions? That would leave share, factory, define and extend.
Or you can go the other way and keep container.proctect and merge share and define together?
// Shared service
container.share("foo", () => {
    return /* object */;
});
// New service every time
container.factory("foo", (c) => {
    return /* object */;
});
// Register function
container.share("foo", container.protect(/* function */));
// Parameter
container.share("foo", /* anything */);
// Extend
container.extend("foo", (previous, c) => {
    return /* anything */;
});

Also I'm not sure of how much useful tags are? They do complicate the API where one could simply use extend? I.e. it seems you are reusing both Symfony's and Silex's solution to solve the same problem (tags and "extend").
Before:
container.share("app.static", container.protect(express.static("public")), ["middleware"]);
container.share("app.static", container.protect(express.static("files")), ["middleware"]);

container.extend("app", (app, c) => {
    c.tagged("middleware").forEach(middleware => app.use(c.get(middleware)));

    return app;
});

After:
// Module 1
container.extend("app", (app) => {
    app.use(express.static("public"));
    return app;
});
// Module 2
container.extend("app", (app) => {
    app.use(express.static("files"));
    return app;
});

@mnapoli Thanks for your feedback! I'm starting some work on the point you mentionned.

Or you can go the other way and keep container.proctect and merge share and define together?

I chose to make .define a private API method and allow .share to receive scalar and objects values.

Also I'm not sure of how much useful tags are?

To me tags are really useful to make things as much dynamic as possible: you define some middlewares (for example in an express app), tag them and the get injected in the app service thanks to the loop on tagged services. Doing so, there is a single modification required: declaring the middleware as a service.
With extend, we have to declare the middleware as a service (not required but if it needs configuration or depends on other services) and then, extend the app service to add the middleware injection.
.extend was, to me, meant for other use-cases: you have the static middleware in a service and later, you want to add some caching feature to it for example:
container.share('static', container.protect(express.static("files")));

// ... later

container.share('static', (static) => {
    var cache = /* wathever */;

    return cache(static);
});

:+1: makes sense now that you explain it!


Fixed: Prevent exception if nodes is empty string

Reference https://github.com/Sonarr/Sonarr/commit/3bc6bf9e992b0c2e0f82591608f2037fdd313297

@Qstick My only question is why is this a problem in the first place? If the data contains an empty BEncodedList then is some other part of the code hitting an IndexOutOfRangeException by assuming it's non-empty, or is the issue that some application is (incorrectly?) putting in a BEncodedString instead of a BEnodedList and you're getting an InvalidCastException ?

Possibly fixed by https://github.com/mono/monotorrent/commit/b1f976a18fd42bef1c9c54a6bae654a3221283f6 . If not, let me know what the issue is :)

This should be fixed with the patch in master. I was linked to this as the probable original issue ( https://forums.sonarr.tv/t/sonarr-fails-to-download-torrent-files-from-rss-feed/9700 ) and it indicates an InvalidCastException was the cause. That makes me believe the issue is that some implementors of this feature use a BEncodedList of BEncodedLists and others used a BEncodedString with compact encoding. The spec doesn't really say what you should do, but it heavily implies BEncodedList of BEncodedLists.
If there are problematic torrents do let me know and we can figure out the next step! For now i think things should be OK.

Thanks for the patch, even though I didn't take it exactly as-is this time :)


add monorepo cicd

Status: Passed in https://github.com/switchupcb/disgo/actions/runs/2512023700. Testing cache by re-running all jobs.


[SKY-271] Don't hard code compression and encryption

null
SKY-271

hi could you add me to linear group<EMAIL_ADDRESS>


NET Assembly analyser Linux

Hi,
Does NET Assembly analyser run on linux? i have dotnet core installed but DependencyCheck doesn't pick it up...
dotnet --version
3.1.100

I've also read https://github.com/jeremylong/DependencyCheck/issues/1464
and tried with v2 but it still didn't pick it up

Have you tried adding --dotnet <path> to tell ODC where the executable is?

I did, i tried to point it to both the SDK's paths, i tried with both runtime's as well.
What path does it expect to have? the only example i saw had a .exe extension but this is linux so that was part of the reason for asking.

Could you check what happens when you pass the following parameter: -Danalyzer.assembly.dotnet.path=/usr/bin/dotnet (to the correct dotnet path).

@JStyle21 try adding the --log odc.log and reviewing this for clues as to what is going on.

Thanks @vdotjansen  and @jeremylong
The log file helped understand the situation, i used the wrong path :(
Now it works, although as mentioned in #1464 it only works with the 2.2 runtime and not with the 3.1


Testing

add variable for persistence directory
reset testing state automatically
reset staging state manually
use prisma deploy instead of dev
remove winston from backend
fix typo acknoledgment
acknowledge message hashed instead of timestamps

todo:

[ ] check why session in getDatabase not removed
[ ] remove pendi

ng message schema


Resolving conflicts between SQLPS and SqlServer modules

Some command names exist in both SQLPS and SqlServer modules.
This means that if you have one module already installed an error is raised which can be prevented by using the -AllowClobber parameter when installing or updating the other module.
Also, when importing one module you need to remove the other module first or use the -DisableNameChecking parameter to prevent an error.

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

ID: cdc5826c-d266-6f61-e1eb-b4f236d03dd9
Version Independent ID: 4ab93d92-6a23-a0c0-2911-ce06e99dc5c5
Content: Download SQL Server PowerShell Module - SQL Server
Content Source: docs/powershell/download-sql-server-ps-module.md
Service: sql
Sub-service: sql-server-powershell
GitHub Login: @markingmyname
Microsoft Alias: maghan

@renbud -- Renato, thank you for your clear feedback.
@markingmyname -- Mark, please look into this issue.

@renbud
Thank you for bringing this to our attention.
That is a valid point.
We will work on adding some sort of context to help others avoid this in the future.

Updates to the article should be live early next week.
Thank you.


Add EJS support

What is EJS : EJS website
File extension:
.ejs
Icon:
EJS Icon

here's one i think would fit nicely as well


Websocket implementation for pay / receive methods in onchain and lightning modules

What does this mean? Sending updates about state transitions?

Currently the await- methods are blocking until the transaction completes successfully, big one is for lightning. Websocket implementation should send back updates so the wrapper can consume them async like when an invoice gets created but paid a half hour later


Prevent different users from using the same name

Currently multiple accounts can have the exact same name. We require the email to be unique but not the name.
This can be problematic given that the domain used by albatross is the name of the account.

What about using the users uuid for domain purposes? This can be a quick fix in this case.
But I assume the other frontends (grafana), may not like the uuid values (if it's displayed somewhere in the UI?)

For now, esp. since we've a running albatross with existing setup (and there's no UUID prefix for the unikernels), and the question about where such a mapping (user name and uuid) should be persisted (if needed) -- see https://github.com/robur-coop/albatross/issues/173 -- I'd prefer to check that the user name is unique.

Thank you for the additional context on the albatross issue. I totally agree it's a bit more complex to decide. I think we can easily just check the names and prevent duplicate ones for the time being.


nginx seems crash when more than one rtmp stream push

when use one rtmp stream push like /live/0 . nginx works above 12 hours , but when add 2 rtmp stream push /live/0  and /live/1 . nginx will crash and no response of /rtmp_stat at 20-30 minutes. how do i check the problem ?

Does it still have this problem with the latest pingos code? And show your nginx.conf?

when use one rtmp stream push like /live/0 . nginx works above 12 hours , but when add 2 rtmp stream push /live/0 and /live/1 . nginx will crash and no response of /rtmp_stat at 20-30 minutes. how do i check the problem ?

Does it still have this problem with the latest pingos code? And show your nginx.conf?


If it is possible to output the domain in the format of constraints or generator?

as titled, If it is possible to output the domain in the format of constraints or generator?
for example,
--crab --crab-domain=int(or pk_apron)
if i want to get the polyhedra domain of newpolka or interval, is that possible?
thanks a lot.

There is a method to_linear_constraint_system that does that.
On Jun 27, 2017 7:29 AM, "gaobo89"<EMAIL_ADDRESS>wrote:

as titled, If it is possible to output the domain in the format of
constraints or generator?
for example,
--crab --crab-domain=int(or pk_apron)
if i want to get the polyhedra domain of newpolka or interval, is that
possible?
thanks a lot.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/caballa/crab-llvm/issues/15, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEyv9Gf5rUDHB8n6KXXK6lJRd-vY9u0fks5sIRHjgaJpZM4OGtvO
.

@gaobo89: can I close the issue?


Allow users to access IP/TCP/UDP headers

This will allow users to access packet metadata without needing to parse
the "parsedPacket" string, and will also allow them to access the packet
data present after the headers for further computation, because the
header addresses are now exposed.
The IPHeader, TCPHeader, UDPHeader structs have been moved outside
PacketFragmentArgs to allow definition of properties with the same name.
They have been converted from structs to class to make them into
reference types and therefore have the homonymic properties return null
if the corresponding header was not successfully parsed by
ParsePacket().
I am proposing this change because I need to parse DNS request/response
in packets, and it is almost impossible to do it with the current class
structure without duplicating the code of FIndIpHeader(), and I think
that it's much cleaner to expose the headers themselves rather than just
exposing FindIpHeader() and having the users duplicate the parsing
logic.

@brianrob could you please review? This is the last bit I need for my own ETL parsing tool which interprets DNS data. Thanks! :)

@lupino3 thanks for submitting.  I will need a bit of time to look through this, but consider this an ack.


detect improper use of t.throws

Protects against a common misuse of t.throws (Like that seen in #739).
This required the creation of a custom babel plugin.
https://github.com/jamestalmage/babel-plugin-ava-throws-helper
Sample output:
Improper usage of t.throws detected at /Users/jamestalmage/dev/ava/test/fixture/improper-t-throws.js (4:10).

You should wrap the following expression in a function:

  throwSync()

Like this:

  function() {
    throwSync()
  }

See https://github.com/sindresorhus/ava#throwsfunctionpromise-error-message for more details.

The linked documentation isn't that helpful. We should probably make a short recipe about this. (maybe a common-mistakes recipe? This seems like too small a topic for a whole recipe). We can update the URL when that recipe is written.

The linked documentation isn't that helpful. We should probably make a short recipe about this

:+1:

(maybe a common-mistakes recipe?

#404?

/Users/jamestalmage/dev/ava/test/fixture/improper-t-throws.js (4:10)

Perhaps displaying relative path would be better and would align with our "clean stack traces" feature.

Updated with relative file paths and colors:

I know it is an overkill, but would be incredibly cool to have syntax highlight in errors (not just this one).

I know it is an overkill, but would be incredibly cool to have syntax highlight in errors (not just this one).

Like this?:

I think maybe the middle section:
The following expression:
  t.throws()

Is kinda useless with the codeframe right above.

Super awesome!

Marvelous. :shipit:

Cool!
Should update the default plugin sections in the readme and babelrc recipe.

Should update the default plugin sections in the readme and babelrc recipe.

I went with just the babelrc recipe. I feel like most user's don't care, so I don't want to pollute the readme with every side-effect free plugin we install. Really, we could just update the readme to say. "AVA always includes a few internal plugins regardless of configuration, but they should not impact the behavior of hour code", and then link to the Notes section of the babelrc recipe.

"AVA always includes a few internal plugins regardless of configuration, but they should not impact the behavior of hour code", and then link to the Notes section of the babelrc recipe.

:+1:

:tada:

🍾✨


support twisted.trial unittest runner

Even with aysncio people still need it: https://youtrack.jetbrains.com/issue/PY-2410

supported in https://github.com/JetBrains/teamcity-messages/blob/master/twisted/plugins/teamcity_plugin.py
please test it a little and file issues

+1

+1

@whatalokation @ciarancourtney What exactly is not working for you? twisted support is implemented and automatically tested (to a some degree)

I'm just voting for a trial test runner support in Pycharm as per Ilya's comment in https://youtrack.jetbrains.com/issue/PY-2410 , I've no idea what teamcity is.

@shalupov No test count provided. Please take a look:
(env) C:\temp\early_print>trial --reporter=teamcity foo_test.test_test.TestFooTest.test_test
##teamcity[testStarted timestamp='2017-07-17T23:36:25.712' captureStandardOutput='true' flowId='foo_test.test_test.TestFooTest.test_test' name='foo_test.test_test.TestFooTest.test_test']
1
##teamcity[testFinished timestamp='2017-07-17T23:36:25.712' duration='0' flowId='foo_test.test_test.TestFooTest.test_test' name='foo_test.test_test.TestFooTest.test_test']

-------------------------------------------------------------------------------
Ran 1 tests in 0.020s

PASSED (successes=1)