Unexpected colours from BlendFoo()

I'd expect all these to print #1a1a46, but they don't. I does matter what colour c2 is, so I think this would take some broader testing initially. Also, for example c1 as #1a1a00 gives yields expected result.
func main() {
	c1, _ := colorful.Hex("#1a1a46")
	c2, _ := colorful.Hex("#666666")

	fmt.Printf("%s\n", c1.BlendHsv(c2, 0).Hex())
	fmt.Printf("%s\n", c1.BlendLuv(c2, 0).Hex())
	fmt.Printf("%s\n", c1.BlendRgb(c2, 0).Hex())
	fmt.Printf("%s\n", c1.BlendLab(c2, 0).Hex())
	fmt.Printf("%s\n", c1.BlendHcl(c2, 0).Hex())
}

#461a1a
#1a1a46
#1a1a46
#1a1a46
#440320

Thank you, great report! Since only Hsv (you don't want to use that) and Hcl are wrong, and only for some second colors, it indicates the error probably lies in one of these angular branches. I'll have a look.

I found what I think is a better (and hopefully correct) formula for angular interpolation. I don't remember where I got the original from or if I made it up myself, but it was wrong for some pairs of angles.
I added unit-tests based on your snippet, but it only tests that both extremes of the interpolation are correct, which they now are. I'm not sure where I could get values from for testing intermediate values during the interpolation, if you know a trustworthy source we could add another table-based test for that.
But anyways, please give the new code a try and let me know. Feel free to reopen this issue should you find more interpolation problems!

Added some more tests in a follow-up commit so we can be a little more confident that it's correct now :smile:


merge

Codecov Report

Merging #1891 (c4c9094) into alainmarcel-patch-1 (dedb993) will increase coverage by 0.02%.
The diff coverage is 19.27%.

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

@@                   Coverage Diff                   @@
##           alainmarcel-patch-1    #1891      +/-   ##
=======================================================
+ Coverage                24.16%   24.19%   +0.02%
=======================================================
  Files                      187      191       +4
  Lines                    31336    31359      +23
=======================================================
+ Hits                      7573     7586      +13
- Misses                   23763    23773      +10

Impacted Files
Coverage Δ

src/Design/Design.cpp
43.75% <0.00%> (ø)

src/Design/DummyType.h
0.00% <0.00%> (ø)

src/Design/Enum.h
0.00% <0.00%> (ø)

src/Design/Function.h
36.84% <0.00%> (-6.91%)
:arrow_down:

src/Design/Parameter.h
66.66% <0.00%> (-6.07%)
:arrow_down:

src/Design/Scope.h
25.00% <0.00%> (-3.58%)
:arrow_down:

src/Design/SimpleType.h
0.00% <0.00%> (ø)

src/Design/Statement.h
6.25% <0.00%> (-0.73%)
:arrow_down:

src/Design/Task.h
0.00% <0.00%> (ø)

src/Design/Union.h
0.00% <0.00%> (ø)

... and 30 more

Continue to review full report at Codecov.

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


Rewrite addons loader as a webpack virtual module?

When I wrote the addons loader I did not knew about the existence of https://www.npmjs.com/package/webpack-virtual-modules
On loading the addons, Volto creates a temporary file that includes all the addon loading configuration as "static code". This is needed because I did not (and I still don't) know of a better way to make webpack discover code and include it in bundles.
Now, this temporary file can be virtual, if webpack-virtual-modules would be used.  The disadvantage of the existing mechanism is that it seems that the temporary file is not cleaned up properly in some cases.

@tiberiuichim nice! I guess it is always better to rely on a lib that is maintained by someone else than creating our own stuff. So I am +1 to investigate this further...


Update requirements-licenses.md

Office 365 E3 and Microsoft 365 E3 are different types of licenses, as described in the content below.  Product names and service plan identifiers for licensing - Microsoft Entra ID | Microsoft Learn
Customer made misunderstanding that the Office 365 E3 and the Office 365 E5 do not meet the requirements for cloud updates cased on the current license description,

Signing off this change


update

think i did it right

yall should do a step by  step guide on how to update they bot on yalls discord


show tagOverlay on the user selected node

When the user clicks on the model a tag overlay should appear with the selected model information.
if no tag information is provided vi can do it like this.

Agreed to close.


The term 'Get-AzureStackLogs' is not recognized as the name of a cmdlet

It's 'Get-AzureStackLog', not 'Get-AzureStackLogs'

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

ID: 9791d946-d79a-6929-4e48-56eb37021c6f
Version Independent ID: 22b2c392-7cc5-c21a-fae5-2f717ce477c2
Content: Use the privileged endpoint (PEP) to collect diagnostic logs - Azure Stack Hub
Content Source: azure-stack/operator/azure-stack-get-azurestacklog.md
Service: azure-stack
GitHub Login: @Justinha
Microsoft Alias: justinha

@Donnie-Zhang  Thank you for bringing this to our notice, we will review the document and update it accordingly.

@Donnie-Zhang This is already listed in #816, closing this a dupicate and following up with that issue.


_translate_event removes information about channel and overrides it with inconsistent channel name OR user name

Description
It's impossible to reply in channel where the event appeared.
When event generated
_translate_event is called
which calls
_lookup_channel_by_id
which returns dict with 'channel_name' and 'channel_type'.
Two issues here:

_lookup_channel_by_id returns user name as channel name, when it's not a channel name.
_lookup_channel_by_name will fail this channel_name, so it's not a channel name
but at least it returns the channel_type, so the caller can still know that and use channel name in get_im_channel
_translate_event will completely ignore channel_type returned by channel_name.
As a result receiver of event has completely no info about which channel to use to reply to it.

What type of issue is this? (place an x in one of the [ ])

[x] bug

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
slackclient version: 1.0.9
python version: 2.7

wrong project


[Cosmos] further conversation on api_version parameter

Follow up to https://github.com/Azure/azure-sdk-for-python/pull/24008/.
Cosmos is currently ignoring the passing in of this option to the client init methods, but further conversation on the subject is requested to ensure this decision makes sense. Using this issue to mark this topic as a follow-up conversation to be had with the central SDK team.

PR: https://github.com/Azure/azure-sdk-for-python/pull/24008


New save and sample

This currently is tested to work for both unconditional and upsampler. However, the coloring of the original images for the upsampler look off. I'm not sure what the issue is, so please have a look.
Note that this moves the generator kwargs arguments into a separate definition in the GigaGan class so it can be called in the save and sample, too.

Nevermind on the color issue. I was using some custom resizing with cv2 in the dataloader and forgot to convert the bgr to rgb after loading.

One other issue. It seems that just for the Generator, the image outputs are all identical on each individual save and sample. Upsampler doesn't have this problem.

@CerebralSeed thank you for getting the ball rolling here! that is good news that the upsampler works, as that unblocks Nicholas
let us merge this and i'll figure out what's wrong with the generator (should be really close to being functional!)

@lucidrains was retesting and it seems I messed up on the upsampler when copying over code or something. Easy fix though:
1.
import torchvision # needed for the Resize after the interpolation, so the low res images are the same physical size as the upsampled image

Change line 2,000 to:

G_kwargs, maybe_text_kwargs, low_res_image = self.generate_kwargs(dl_iter, batch_size, sample=True)

@CerebralSeed try the updated version! i simplified some of the code, and also the base generator should be working now :partying_face: thank you for letting me know that it was only generating one image

It might have been that I set the learning rate too high and was overshooting. Retesting with a lower learning rate and higher gradient accumulation. Also, might try clipping the Generator loss, and see if that improves anything.


S3 Bucket removal failure

Noticed that cloud-nuke fails to remove S3 buckets in some cases:
[cloud-nuke] INFO[2022-10-27T15:57:19Z] Emptying bucket cloudfront-example-hon37l.abc-logs
[cloud-nuke] INFO[2022-10-27T15:57:19Z] [OK] - successfully emptied bucket cloudfront-example-hon37l.abc-logs
[cloud-nuke] ERRO[2022-10-27T15:57:19Z] [Failed] - 1/1 - Bucket: cloudfront-example-hon37l.abc-logs - bucket deletion error - BucketNotEmpty: The bucket you tried to delete is not empty. You must delete all versions in the bucket.
	status code: 409, request id: 123, host id: XYZ
[cloud-nuke] ERRO[2022-10-27T15:57:19Z] [Failed] - 1/1 - S3 bucket(s) failed deletion in us-east-1
ERROR: 1 error occurred:
	* BucketNotEmpty: The bucket you tried to delete is not empty. You must delete all versions in the bucket.
	status code: 409, request id: 123, host id: XYZ

Hey @denis256, can you describe how to reproduce this error message?
I tried creating a test s3 bucket with versioning enabled and uploaded files multiple times with different contents (to produce different versions of the same file). Then, I ran the following command to delete the test s3 files:
gruntwork vault exec rasbox-shared -- cloud-nuke inspect-aws --resource-type s3
gruntwork vault exec rasbox-shared -- cloud-nuke aws --resource-type s3

The command above successfully deleted the s3 file.

Haven't heard back on this thread for a while. Closing this issue. Feel free to reopen the ticket if you face similar error again.


Update docs

Description
Starting to cleanup/update docs for the new caikit+tgis manifests. Marking this as a draft for now to start some discussions.
closes #109

@heyselbi I think we should discuss the future of the docs/manifests/script in demo/kserve


weird Sanskrit highlighting

It's probably a bug in lucene-sa but I don't have time to confirm that now so I'm just opening it here:
http://library.bdrc.io/search?q="nāmasaṅgīti"&lg=sa-x-iast&t=Work
shows some odd highlighting such as

another example:


Test files outside src fail to load Svelte files

Describe the bug
When I use the typescript.config option to change the path of the tests folder to a folder in my root the Svelte files I have used in this tests fail to load.
This is for tests that use a Svelte wrapper file. I need these wrapper files to setup slot tests.
Reproduction
https://github.com/seanlail/kit-issue-tests
There are 2 commits:

Sets up the tests inside the src folder and the svelte imports work (test passes)
Moved the tests folder outside of src. (test fails)

Logs
FAIL  __tests__/+layout.test.ts [ __tests__/+layout.test.ts ]
Error: Failed to load url /__tests__/LayoutWrapper.svelte (resolved id: /Users/seanlail/kit-issue-tests/__tests__/LayoutWrapper.svelte) in /Users/seanlail/kit-issue-tests/__tests__/+layout.test.ts. Does the file exist?
 ❯ loadAndTransform node_modules/vite/dist/node/chunks/dep-ca21228b.js:39470:21

### System Info

```Shell
System:
    OS: macOS 13.1
    CPU: (12) x64 Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
    Memory: 1.38 GB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 18.14.0 - ~/.nvm/versions/node/v18.14.0/bin/node
    npm: 9.3.1 - ~/.nvm/versions/node/v18.14.0/bin/npm
  Browsers:
    Chrome: 110.0.5481.100
    Firefox: 99.0.1
    Safari: 16.2
  npmPackages:
    @sveltejs/adapter-node: 1.2.0 => 1.2.0
    @sveltejs/kit: 1.8.4 => 1.8.4
    svelte: 3.55.1 => 3.55.1
    vite: 4.1.4 => 4.1.4

Severity
serious, but I can work around it
Additional Information
I thought this may be a Vitest / Vite issue, but it's part of the Kit setup that we can change the folder for the tests, so I think it's valid to create the issue here.

I just updated the dependencies to latest (to see if anything had changed) and the issue still exists.

https://github.com/vitest-dev/vitest/issues/3953#issuecomment-1679134767

Closing this as it's fixed without any changes in Vitest 0.34.x


Refactor migrate commands to use migrate v3

https://github.com/mattes/migrate
PROS

migrate v3 now support other locals where get migrations files ex: github repository, s3 and many others

CONS

Migrate v3 have a lot of changes that will break our cli api
Create command is removed

cc/ @avelino @crgimenes @marioidival  @cassiobotaro

close because is deprecated https://github.com/mattes/migrate/issues/311


Add util for TDD instantiation

This PR is more of a proposal and tries to make TaskDeploymentDescriptor instantiation in tests less tedious.
With a total of 17/18 constructor arguments calling the constructor naturally becomes an unreadable affair. As such i added a TaskDeploymentDescriptorBuilder with a set method for every argument; should an argument not be set when build() is called a default value is used.
I recently tried adding the job name to the TDD and when i saw the number of occurrences i would have to change I came up with this idea.

I like this Chesnay. Definitely makes the tests more readable. Feel free to merge it if more people think this is useful.

Still like the idea, but we didn't get any feedback and the TDD changed in the mean time. The number of arguments also decreased to 10. I think it's best to close this PR.


Investigate whether we can build ASTContext from high-level representation.

This might help us to integrate with CIR. Also see: https://github.com/llvm/clangir/issues/5

Might make a good intern project. Similar to Rellic.


Passing additional params

Discord recently added a new prompt parameter for the authorization URL that allows you to skip re-approval if the user is already authorized.
How would I go about adding that to my application. Does it require a change to this strategy?

Sorry forgot to add more details:
Here's the https://github.com/discordapp/discord-api-docs/issues/264 with more details and the commit related to the changes

Looks like the "authorize querystring" can contain a prompt parameter now.

Apologies for the delay, I've added support for the prompt parameter settable through query string as well as config!

Awesome thanks @schwarz


add --keepDecorators compile option

As #22980 has pointed out. There is a sound logic for lossing decorators in declaration file ".d.ts".
But in my opinion, some decorators do provide useful information for readers. For example:
@expose({name: "sockFile", since: 2.0})
domainSocketFile: string

It tells me that when serialize to JSON, it will be renamed to "sockFile", and it will only be serialized after version 2.0
Suggestion
I'd suggest
keepDecorators: never = default
keepDecorators: declaration = put decorator in .d.ts files. Perhaps in comments?

I think "design time decorator"  #2900 can fit my need.


Don't Merge! PR real time test for target branch full build

Docs Build status updates of commit 5be83ba:
:clock10: Full build: the average full build time is 5 min(s) 58 sec(s), this is based on the last 629 build(s) for this repository.

Docs Build status updates of commit 5be83ba:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

docs-ref-conceptual/testing.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

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


SharedInstance<> not working

An exception is thrown when deserialising a SharedInstance in version 2.6.3 on Unity 5.4.1f1.
The same problem occurs even in the latest build from the github repository.
Following the guide here is enough to reproduce the issue:
http://jacobdufault.github.io/fullinspector/guide/#docs/workflow_shared_instance
Shared Instances in version 2.6.2 on Unity 5.0.2 work perfectly.
Here is the exception:
ToString is not allowed to be called during serialization, call it from Awake or Start instead. Called from MonoBehaviour 'MyDemoBehavior1' on game object 'GameObject'.
See "Script Serialization" page in the Unity Manual for further details.
System.String:Concat(Object[])
FullInspector.InspectedProperty:Write(Object, Object) (at Assets/FullInspector2/Core/Utility/InspectedProperty.cs:139)
FullInspector.Internal.fiISerializedObjectUtility:RestoreState(ISerializedObject) (at Assets/FullInspector2/Core/fiISerializedObjectUtility.cs:251)
FullInspector.BaseBehavior`1:RestoreState() (at Assets/FullInspector2/Core/BaseBehavior.cs:65)
FullInspector.Internal.fiSerializationManager:DoDeserialize(ISerializedObject) (at Assets/FullInspector2/Core/fiSerializationManager.cs:210)
FullInspector.Internal.fiSerializationManager:OnUnityObjectDeserialize(ISerializedObject) (at Assets/FullInspector2/Core/fiSerializationManager.cs:117)
FullInspector.BaseBehavior`1:UnityEngine.ISerializationCallbackReceiver.OnAfterDeserialize() (at Assets/FullInspector2/Core/BaseBehavior.cs:132)

I've landed ff08d3e290c8128834a913f763a62eaad645066c which should fix the ToString call.
SharedInstance still worked for me though on Unity 5.5b8, but there is a generated exception when you change the SharedInstance type, ie, from SharedInstance<int> to SharedInstance<bool>.

I've pushed a build for this commit, DLLs should be available shortly here.


Release notes can never be opened after you try to open them when offline

Issue Type: Bug
If you try to open the release notes while offline, then if a release notes url is configured then vscode will open it in a browser, otherwise it will do nothing, not even show an error.
Later, if you go online and try opening the release notes again, the same will happen - vscode will never attempt to open the release notes webview again.
This state is preserved until you reload the window.
Steps to Reproduce:

Go offline (you can also open devtools in vscode and go to the network tab and change the "online" at the top to "offline")

Try to open the release notes using Show Release Notes from the command palette.

Depending on whether you are on an official vscode build or a local build, the release notes page will be opened in your default browser. On a local build nothing happens.

Go back online

Try to open the release notes again

The same thing will happen

Reload the window using Reload Window from the command palette

Try to open the release notes again

The release notes open in a webview inside vscode

VS Code version: Code - Insiders 1.47.0-insider (6c235c723cb2d8defb840f732eac620f54aa8b9d, 2020-06-26T06:20:06.931Z)
OS version: Windows_NT x64 10.0.18363
Does this issue occur when all extensions are disabled?: Yes

/duplicate of #101132


Unable to verify response if event-stream

Problem Description
When I redirect a specific url to localhost, if it is an event-stream, it comes out as content missing in mitmproxy response tab.
def responseheaders(flow):
    flow.response.stream = flow.response.headers.get('content-type', '').startswith('text/event-stream')

Steps to reproduce the behavior:

setup responseheaders in my script
redirect www.example.com --> localhost:8081 in my script

System Information
9.0.1

9.0.1

Please try with the latest version

same problem in 10.3.1
I resolved it with a temporary workaround.
def is_stream(flow):
    return "text/event-stream" in flow.response.headers.get("content-type", "")

def responseheaders(flow):
    def stream_handler(data: bytes):
        if not hasattr(flow, "buffer"):
            flow.buffer = b""
        flow.buffer += data
        flow.response.content = flow.buffer
        return data

    if is_stream(flow):
        flow.response.stream = stream_handler

I am running a development server locally. When accessing www.example.com, I redirect only specific parts to localhost for testing.

redirect

Via 301? 302? Where's the relevant script?
Please provide full steps to reproduce this with every detail. You also didn't answer what you are trying to achieve.

it comes out as content missing in mitmproxy response tab.

I'm tempted to close this as a dupicate of https://github.com/mitmproxy/mitmproxy/issues/6126, feel free to re-open if you disagree and can provide necessary details.

I am handling the redirect as follows:
def request(flow: http.HTTPFlow) -> None:
    if flow.request.url.startswith("https://www.example.com/api"):
        flow.request.host = "localhost"
        flow.request.scheme = "http"
        flow.request.port = 8081

This implementation works fine except for http://www.example.com.
For response content-type: text/event-stream:
When I set flow.response.stream = True:

The stream is processed correctly on the screen.
The flow response shows "content missing."

When flow.response.stream = False:

The entire data comes at once on the screen instead of streaming.
The flow response shows the data.

I want the stream to be processed correctly on the screen while the flow response also shows the data.

Can you clarify what "on the screen" means? In the console UI? Maybe you can share a screenshot.

When I set flow.response.stream = True:
* The stream is processed correctly on the screen.

* The flow response shows "content missing."

When flow.response.stream = False:
* The entire data comes at once on the screen instead of streaming.

* The flow response shows the data.

I think what you're seeing is expected behavior and a combination of #6126 and https://github.com/mitmproxy/mitmproxy/discussions/6996 (@mhils am I correct?)

@Prinzhorn
For example, in a chat service like ChatGPT where the response is output as a stream, the behavior is similar to cases #6126 and #6996.


NO LAUNCH FILE PROVIDED +  POOR DOCUMENTATION

Hello I am trying to get this package to run but it seems the attached Application Note file doesn't reflect the content on this repo. First there's no launch provided. And there's little or no documentation at all as to what parameters are needed to run the python file.

Hi,
It's true that there isn't any ros launch file and that the application note does not make any sense yet.
But there is a way to test the ros driver. Run like this:
rosrun ros_openimu openimu_driver.py
I have debugged some bugs that I pointed out in the previous issue.
https://github.com/ROS-Aceinna/ros_openimu/issues/1
The driver finds autonomously the Aceinna device, so you have not to pass any parameter in the first test. I have tested this with an openIMU300ZI. The magnetometer topic is not working, but the accelerometer is reporting data.
This is a beta version and we will have to wait for Aceinna's colleagues to debug and release it.
Bye,
Guillermo

Hi @GilenTell ,
Thanks for the insight and quick response. I have the  openIMU300RI which arrived yesterday but even though it gets detected by running the "openimu_driver.py" the call to read IMU data seems to return "None" which causes the program to terminate unexpectedly. My question is, do I have to program(st-link way) the sensor before I can data from it??

Hi Leocorp96, we have updated the repo with a launch file and also the app note to reflect the changes made.
The default RS232 rate for the Openimu300RI is 0 when shipped from the factory. You will have to set this to a higher value (like 100Hz) for use with the ros driver. We will add the capability to set parameters in a future update to this repository.
At the moment you can set the packet rate and save permanently with the standard evaluation software.

Download and run the server .exe at https://developers.aceinna.com/devices/connect
With the device connected and server running click the devices->openimu tab
click settings, change the packet rate and save permanently (otherwise change will be lost after a power cycle)
There is a video on our YouTube site with further information - skip to 39:30 for changing settings https://www.youtube.com/watch?v=szHDnQdDAXw

Hi Leocorp96, we have updated the repo with a launch file and also the app note to reflect the changes made.
The default RS232 rate for the Openimu300RI is 0 when shipped from the factory. You will have to set this to a higher value (like 100Hz) for use with the ros driver. We will add the capability to set parameters in a future update to this repository.
At the moment you can set the packet rate and save permanently with the standard evaluation software.

Download and run the server .exe at https://developers.aceinna.com/devices/connect
With the device connected and server running click the devices->openimu tab
click settings, change the packet rate and save permanently (otherwise change will be lost after a power cycle)
There is a video on our YouTube site with further information - skip to 39:30 for changing settings https://www.youtube.com/watch?v=szHDnQdDAXw

Hello @ROS-Aceinna, I followed the outlined instructions "Save Permanently": I get the message "Saved successfully" but then it keeps resetting to 0 on power cycle. My software version is "IMU_J1939 03.00".
QUICK FIX: Is there a way to configure this variable on each connection?

Hi Leocorp96, if you update the app on the openimu to the attached you should be able to save permanently. To do this go to https://developers.aceinna.com/code/upload and unzip and then add the attached application (openimu300Ri IMU app with packet rate of 100Hz). You can then write it to the part.
OpenIMU300RI app.zip

Works perfect. Thanks for taking the time @ROS-Aceinna


Can enter to own wools, The latest version

We need more details

Me happens the same
El dia 16/02/2015 20.51, "Kevin Phoenix"<EMAIL_ADDRESS>va
escriure:

We need more details
—
Reply to this email directly or view it on GitHub
https://github.com/twizmwazin/CardinalPGM/issues/183#issuecomment-74563239
.

I was playing an scrim and everyone can enter our own wool

I need to know what map, and if you could have a pastebin to your XML, that would be helpful.

Its working fine for me
Tested on dynamo

Wich map?
El dia 16/02/2015 20.52, "ItzWid13"<EMAIL_ADDRESS>va escriure:

I was playing an scrim and everyone can enter our own wool
—
Reply to this email directly or view it on GitHub
https://github.com/twizmwazin/CardinalPGM/issues/183#issuecomment-74563364
.

On Dynamo,Deepwind Jungle, xml maps.oc.tc
before it works fine

was there any errors when this was happening?
and what is the result of /cardinal -v

is the latest version of cardinal :cactus:

Is my error sorry


Commandline updates

Minor fixup to the cube-cfg help text.
Add bash-completions for 'c3' commands.

merged


Inefficient representation when doing custom controlled gates verses native implimentations

Information

Qiskit Terra version: master
Python version:
Operating system:

What is the current behavior?
Consider a CCZ gate made by controlling a Z gate:

qc = QuantumCircuit(2)
qc.z(1)
ctr_gate = qc.to_gate().control(2)

This has a decomposition of
circ1 = QuantumCircuit(4)
circ1.append(ctr_gate, range(4))
circ1.decompose().draw('mpl')

that uses P, U, and CX gates, where as writing the same expression written in terms of predefined gates yields:

circ2 = QuantumCircuit(4)
circ2.h(3)
circ2.ccx(0,1,3)
circ2.h(3)
circ2.decompose().draw('mpl')

that is decomposed into an combination of U2(0,pi) (which is an H), H, T, and CX.  Importantly, there are only 6 CX gates in this latter decomposition and 8 in the former.  These gates cannot be transpiled away at any optimization level.  So in this very simple case there are 33% more CX gates when implementing the exact same thing using custom controlled gates, and a 40% increase in the final transpiled depth (14 vs 10).
Steps to reproduce the problem
What is the expected behavior?
Suggested solutions

I believe this could be fixed by adding 'z' to the basis list https://github.com/Qiskit/qiskit-terra/blob/2d11db2/qiskit/circuit/add_control.py#L114 and a corresponding rule below for the case where gate.name == 'z'.


contrib/verbs causes compile error on master

Recently merged RDMA-supporting feature is based on r1.0, and incompatible with the latest master.

ERROR: /data/chuangchen/workspace/tensorflow/tensorflow/contrib/verbs/BUILD:104:1: C++ compilation of rule '//tensorflow/contrib/verbs:rdma_rendezvous_mgr' failed: crosstool_wrapper_driver_is_not_gcc failed: error executing command external/local_config_cuda/crosstool/clang/bin/crosstool_wrapper_driver_is_not_gcc -U_FORTIFY_SOURCE '-D_FORTIFY_SOURCE=1' -fstack-protector -fPIE -Wall -Wunused-but-set-parameter ... (remaining 151 argument(s) skipped): com.google.devtools.build.lib.shell.BadExitStatusException: Process exited with status 1.
tensorflow/contrib/verbs/rdma_rendezvous_mgr.cc: In constructor 'tensorflow::RdmaRemoteRendezvous::RdmaRemoteRendezvous(const tensorflow::WorkerEnv*, tensorflow::int64, tensorflow::RdmaMgr*)':
tensorflow/contrib/verbs/rdma_rendezvous_mgr.cc:34:35: error: 'worker_name' was not declared in this scope
: BaseRemoteRendezvous(env, worker_name, step_id, true),
^
tensorflow/contrib/verbs/rdma_rendezvous_mgr.cc: In constructor 'tensorflow::RdmaRendezvousMgr::RdmaRendezvousMgr(const tensorflow::WorkerEnv*, const string&, tensorflow::WorkerCacheInterface*)':
tensorflow/contrib/verbs/rdma_rendezvous_mgr.cc:139:41: error: no matching function for call to 'tensorflow::BaseRendezvousMgr::BaseRendezvousMgr(const tensorflow::WorkerEnv*&, const string&)'
: BaseRendezvousMgr(env, worker_name) {}
^
tensorflow/contrib/verbs/rdma_rendezvous_mgr.cc:139:41: note: candidate is:
In file included from ./tensorflow/contrib/verbs/rdma_rendezvous_mgr.h:22:0,
from tensorflow/contrib/verbs/rdma_rendezvous_mgr.cc:18:
./tensorflow/core/distributed_runtime/base_rendezvous_mgr.h:62:12: note: tensorflow::BaseRendezvousMgr::BaseRendezvousMgr(const tensorflow::WorkerEnv*)
explicit BaseRendezvousMgr(const WorkerEnv* worker_env);
^
./tensorflow/core/distributed_runtime/base_rendezvous_mgr.h:62:12: note:   candidate expects 1 argument, 2 provided
Target //tensorflow/tools/pip_package:build_pip_package failed to build
Use --verbose_failures to see the command lines of failed build steps.

It's broken by this commit.
This is at least the second time of compile error. There should be a pre-commit check?

@yifeif should we modify one of our configurations to configure with rdma?

Is someone working on this?

Is it solved, I meet the same issue

@DogNick I made a local fix. I can provide a PR.

@llhe Nice!

Thanks @llhe
+1 on one configuration with RDMA check (we usually check the master with VERBS support but not every day).


Automated Builds with Limited Access

File: docker-hub/builds.md, CC @johndmulhausen
I can't seem to find the way to add GitHub webhooks for DockerHub manually.

I'm not sure what you're trying to achieve, but have you tried checking in Docker Cloud
https://docs.docker.com/docker-cloud/builds/automated-build/

@costa As documented here: https://docs.docker.com/docker-hub/webhooks/
Go to: https://hub.docker.com/r///~/settings/webhooks/
:)


Blank Page

If i go to setting > terminal terminus application is blank
note:
windows 7 Profesional x64

Are you running from source (if so, what commit)?
If prebuilt Terminus.Setup.1.0.0-alpha.*.exe from GitHub Releases, what version?
Are you able to reproduce it reliably? If so, try starting Terminus with --debug (i.e terminus.exe --debug) so it opens with the Chrome dev tools and you can see what the error is in the console.

Terminus shouldn't crash anymore due to plugin problems.


Auto height wrong on Ipad

Hi, great work with this package 👏
I face an issue with auto height on ipad. I tried to remove all the logic and it becomes clear that the behavior differs from iphone.
Here is the code related to the screenshots bellow:
<TrueSheet
        dimmed={true}
        initialIndexAnimated={true}
        initialIndex={0}
        sizes={["auto"]}
        grabber={false}
      >
        <Box>
          <Box p="6">
            <Box h="48" bg="red100"></Box>
          </Box>
        </Box>
      </TrueSheet>

As you can see, auto height works great on iphone (there is no safe area on purpose) but on ipad the content is cropped.
If I try with a larger height, same result the content is cropped at the bottom.
Actually it's same with a scrollview too, it always makes it scroll a bit.

I would love some help from on this point, thanks !

Ah, might be due to this code:
https://github.com/lodev09/react-native-true-sheet/blob/d73a39743ce902e6f72773872701c4233f6eb8bb/ios/TrueSheetView.swift#L195-L207

Thanks for the quick answer ! Is it something easily fixable in the next release ?
I noticed other height issues, would you like me to list them ?
I feel that fixing these issues might make react-native-true-sheet the reference package in terms of modal/bottom-sheet for react-native.
It's really a pain to manage modals on react-native and it gives me hope 🙏

The biggest pain I have right now with react-native-true-sheet are:

the auto height not always working (sometimes it's wrong or the modal disappears.
scrollable={true} with a fixed modal header, I know the floatable header hack but it does not feed intuitive, sometimes the header size can be unpredictable and when it works, the scroll indicator goes under the fixed header.

Other found issue:
When rnts is used with a ScrollView inside and auto height, only the first rendered height is used. Upcoming height changes inside the scrollView are ignored which makes in my case a small Modal that scrolls a lot to display the content.
Here is the code that reproduces the issue:
In this exemple, the red View displays with 1000ms delay. The modal does not grows when it appears.
const App = () => {
  const ready = useDelayed(true, 1000);
  const scrollRef = useRef();

  return <TrueSheet
        dimmed={true}
        initialIndexAnimated={true}
        initialIndex={0}
        sizes={["auto"]}
        grabber={false}
        scrollRef={scrollRef}
      >
        <ScrollView ref={scrollRef}>
          <View style={{ background: 'red', height: 50 }}  />
          {ready && <View style={{ background: 'red', height: 60 }} />}
        </ScrollView>
   </TrueSheet>
}


Apple M1 编译报错

Question
环境信息( 工具版本均符合文档要求)：
➜  miniob git:(main) uname -a
Darwin MacBook-Pro-3.local 22.1.0 Darwin Kernel Version 22.1.0: Sun Oct  9 20:15:52 PDT 2022; root:xnu-8792.41.9~2/RELEASE_ARM64_T8112 arm64

➜  miniob git:(main) cmake --version
cmake version 3.25.2
CMake suite maintained and supported by Kitware (kitware.com/cmake).

➜  miniob git:(main) clang --version
Apple clang version 14.0.0 (clang-14<IP_ADDRESS>)
Target: arm64-apple-darwin22.1.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

➜  miniob git:(main) g++ --version
Apple clang version 14.0.0 (clang-14<IP_ADDRESS>)
Target: arm64-apple-darwin22.1.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

➜  miniob git:(main) flex --version
flex 2.6.4 Apple(flex-34)

➜  miniob git:(main) bison --version
bison (GNU Bison) 3.8.2
Written by Robert Corbett and Richard Stallman.

Copyright (C) 2021 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Copyright (c) 2021 OceanBase and/or its affiliates.

Learn more about OceanBase at https://github.com/oceanbase/oceanbase
Learn more about MiniOB at https://github.com/oceanbase/miniob

Successfully load ../etc/observer.ini
Not set log file name, use default observer.log
Output configuration
Begin dump configuration
[]

Finish dump configuration

Db already exists: sys
Extend one pool, this->size:128, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:256, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:384, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:512, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:640, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:768, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:896, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:1024, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:1152, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:1280, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:1408, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:1536, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:1664, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:1792, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:1920, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:2048, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:2176, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:2304, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:2432, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:2560, item_num_per_pool:128, this->name:BufPool.
Extend one pool, this->size:2560, item_num_per_pool:128, this->name:BufPool.
buffer pool manager init with memory size 20971520, page num: 2560, pool num: 20
double write buffer load pages done. page num=0
Successfully read db meta file. db=sys, file=miniob/db/sys/sys.db, check_point_lsn=0
All table have been opened. num=0
Successfully recover db. db=sys checkpoint_lsn=0
Default handler init with miniob success
Successfully init utility
Listen on port 6789
Observer start success

启动客户端 ./obclient， 执行基本的show/select报错，不确定是否为编译的问题

➜  bin git:(main) ✗ ./obclient

Welcome to the OceanBase database implementation course.

Copyright (c) 2021 OceanBase and/or its affiliates.

Learn more about OceanBase at https://github.com/oceanbase/oceanbase
Learn more about MiniOB at https://github.com/oceanbase/miniob

miniob > show databases;
SQL_SYNTAX > Failed to parse sql
miniob > show tables;
Tables_in_SYS
miniob > select * from Tables_in_SYS
FAILURE
miniob >

编译过程中打印信息如下：
执行sudo bash build.sh init 的输出（看起来没有error）：
output.md
执行bash build.sh报fatal error: too many errors emitted, stopping now [-ferror-limit=]

fatal error: too many errors emitted, stopping now [-ferror-limit=]
20 errors generated.
make[2]: *** [unittest/CMakeFiles/buffer_pool_log_test.dir/buffer_pool_log_test.cpp.o] Error 1
make[1]: *** [unittest/CMakeFiles/buffer_pool_log_test.dir/all] Error 2
make: *** [all] Error 2

再另外的一台m1 电脑上编译，也没有此问题，忽略掉好了。感谢回复～

llvm的clang18报错，是需要降低版本吗？
/Users/koschei/Project/miniob/src/observer/sql/operator/aggregate_vec_physical_operator.cpp:26:11: error: no member named 'for_each' in namespace 'std::ranges'
  ranges::for_each(aggregate_expressions_, [this](Expression *expr) {
  ~~~~~~~~^

➜  miniob git:(main) ✗ clang -v
Homebrew clang version 18.1.8
Target: arm64-apple-darwin21.6.0
Thread model: posix
InstalledDir: /opt/homebrew/opt/llvm/bin

我用clang16编译没有遇到你说的问题，clang18反而不兼容这个语法了？

llvm的clang18报错，是需要降低版本吗？
/Users/koschei/Project/miniob/src/observer/sql/operator/aggregate_vec_physical_operator.cpp:26:11: error: no member named 'for_each' in namespace 'std::ranges'
  ranges::for_each(aggregate_expressions_, [this](Expression *expr) {
  ~~~~~~~~^

➜  miniob git:(main) ✗ clang -v
Homebrew clang version 18.1.8
Target: arm64-apple-darwin21.6.0
Thread model: posix
InstalledDir: /opt/homebrew/opt/llvm/bin

我在一个环境上也重现了你的问题，但与Mac或clang无关，除了需要头文件，还需要引用  头文件，真是不可思议。


chore: add typos config

:tada: This PR is included in version 0.6.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


what is password

Try to follow the documentation from http://benedmunds.com/ion_auth/.
The default login is:
Email<EMAIL_ADDRESS>Password: password


Setup Travis to run an integration test when a PR is made

It could be nice if Travis would try and build all the packages which we have confirmed to pass the sanity check.

Good resource on how to Conda with Travis:
https://github.com/astrofrog/example-travis-conda/blob/master/.travis.yml

Travis is now up and running, but i haven't tested it on a PR.

Travis will now run it's tests whenever a PR is opened.


Transparent NameID checkbox

Many Attribute Manipulation scripts map the original NameID to the new NameID using this code:
$response['']['CustomNameId'] = $response['']['OriginalResponse']['__']['OriginalNameId'];
It would be nice to have a Service Registry checkbox for that.

+1

I see where you come from. However, I believe the cases where this is used is always to support some legacy scenario. We don't gain any new SP's that require this. So I'm not sure if we should prioritize it currently.

I think we should remove it instead.


Allows better resolving of some NIDs by fixing reloading of eboot.bin…

… and UnityPlayer by changing the file paths to app0: and using syscall estimation for a couple of functions.

For resolve.c, can you make it exploit independent by just looking for "ux0:/patch" and stripping the ux0:/patch/*/"?

The latest commit now does that.


insert_into and set

Most of the code/syntax is easy to read, and imagine what it would look like as SQL.  Except for insert_into and set, which can look awkward in some cases.
On the frontpage, there is this example:
// A sample insert
db(insert_into(foo).set(foo.id = 17, foo.name = "bar", foo.hasFun = true));

Above is clear and simple, foo is passed to insert_into, and only members of foo is used inside of set.  Most of the examples are like this.  Except for /examples/sample.cpp
examples/sample.cpp : line 37
 db(insert_into(f).set(f.name = "Loves C++", p.fatal = false));

above looks strange because f is passed to insert_into, but in set both f and p are used.
I did not find much info on insert or set in the wiki, it would be good to document how they both work.
https://github.com/rbock/sqlpp11/wiki/Insert

actually, i think p.fatal=false is a typo bug in /examples/sample.cpp.  because the person table doesn't have a member fatal.
error: ‘struct mydatabase::TabPerson’ has no member named ‘fatal’
	errors: 1
385		file: /tmp/mytest.cpp
			  insert_into(f).set(f.name="Loves C++",p.fatal=false);

i guess that is a typo in sample.cpp, because when i try this:
	auto p = mydatabase::TabPerson();
	auto f = mydatabase::TabFeature();
	insert_into(f).set(f.name="Loves C++", f.fatal=false, p.name="Roland", p.feature=1);

i get this error:  set() arguments contain assignments from more than one table
portable_static_assert.h:40:7: error: static assertion failed: set() arguments contain assignments from more than one table

Hi,
Actually, the commented out examples where used to demonstrate nice error messages.
I should make this clear with a comment...
Best,
Roland


Defining more enzymes in OpenMS Comet adaptor

Description of feature
Currently, @timosachsenberg @jpfeuffer comet only support 'Asp-N,Chymotrypsin,CNBr,no cleavage,unspecific cleavage,Trypsin,Arg-C,Lys-C,Lys-N,PepsinA,Trypsin/P,glutamyl endopeptidase' However comet has a way to pass a definition of more enzymes https://uwpr.github.io/Comet/parameters/parameters_202301/search_enzyme_number.html using a parameter file. How can we use that possibility to define for example Lys-C/P currently Lys-C will not work because msgf+  processor change it to Lys-C/P and comet do not supported it.

quick fix:
adding the comet id here https://github.com/OpenMS/OpenMS/blob/develop/share/OpenMS/CHEMISTRY/Enzymes.xml#L142-L163
and adding the output here
https://github.com/OpenMS/OpenMS/blob/develop/src/topp/CometAdapter.cpp#L567-L579

@ypriverol this would actually be a good entry level task for a student that wants to get into OpenMS/C++

For both lysc and multi enzymes you will need to give up consensus id compatibility then.
Lysc is just a simple if-case logic in the workflow.
Multi enzymes is a large change in both openms and the workflow.
Openms needs to support it in both the data structures and things like indexing. You don't only need support for multiple enzymes but also logic if they were applied at the same time or after each other.
It will probably also not be compatible with an own or a workflow generated decoy databases unless you run multiple searches with different enzymes.
You will need to use comet's decoy generation.
Therefore it is probably easiest to run comet without the adapter and convert to idxml later on.

I agree with Julianus that properly modelling multienzyme digestion is adding a lot of complexity.
One note: you often see Lys-C/Trypsin combination because it improves cutting after K.
From a search engine perspective the combination can just be treated as Trypsin (or even Trypsin/P).
So maybe such complexity is not needed?

Im trying to tackle here the first use case which is quite common, the use of another enzyme and not multi-enzyme. Then, it should be easy to extend OpenMS to extend enzymes and support them.

We could make this workaround on the workflow level by allowing multi enzymes on workflow level only.
Then you would see trypsin/lys-c in the workflow reports and trypsin as far as  OpenMS is concerned.
Or we start by adding this special case to OpenMS. (Introducing a new mix enzyme).
This would be mainly for reporting reasons then.

I don't know why you want to do the mix enzyme. The problem is actually much simpler. We have Lys-C/P which in fact is supported by comet but the Adapter in OpenMS doesn't support it. I want to support it in OpenMS in order to be able to process the dataset that used only Lys-C/P with msgf+ and comet. No mix enzymes.

Ah ok I completely misread the issue then haha

Doing Arg-C and Lys-C before trypsin is not an issue.

but Glu-C as listen on: https://www.ebi.ac.uk/pride/archive/projects/PXD005200
is an issue. It cleaves mainly after E but also after D

I don't know why you want to do the mix enzyme. The problem is actually much simpler. We have Lys-C/P which in fact is supported by comet but the Adapter in OpenMS doesn't support it. I want to support it in OpenMS in order to be able to process the dataset that used only Lys-C/P with msgf+ and comet. No mix enzymes.

Ok that should be easy. You mean similar to https://github.com/OpenMS/OpenMS/pull/7422/files

PR request in OpenMS https://github.com/OpenMS/OpenMS/pull/7584


ReactNative stuck at Running script '[CP] Prepare Artifacts'

It seems that when you add the precompiled sdk the above step freezes.
Might be a cocoapods issue, but I wonder fi you have any idea?
In the Pods/Target Support Files/Pods-{target}-artifacts.sh the below line will freeze (no idea why 😅)
ARTIFACT_LIST_FILE="${BUILT_PRODUCTS_DIR}/cocoapods-artifacts-${CONFIGURATION}.txt"
cat > $ARTIFACT_LIST_FILE

OSX Mojave
Xcode 11.2.1
Latest RNFirebase.
RN: 0.61.4
Thank you 😻

This doesn't happen to me but I'm on up to date Xcode, I wonder if that's a thing?

I'm afraid to upgrade 😅

RIp the bandaid off @rborn  ;-)
But have the first air ready haha

I had the same issue with a Cordova/AngularJS project. I think it's a CocoaPods issue. It was happening consistently with pod version 1.9.3. I upgraded to their latest beta, 1.10.0.beta.2 with sudo gem install cocoapods --pre and the issue was solved. One of their scripts was doing cat > $ARTIFACT_LIST_FILE and cat with no arguments would just sit there.

@tzappia thnx for the update, will test later this week and let you know

@mikehardy @tzappia finally upgraded xcode and cocoapods to 1.10.0.rc.1 and all goes smooth. Thanks for all the help 😻

sudo gem install cocoapods solved my problem.
Cheers guys.

sudo gem install cocoapods solved my problem.
Cheers guys.


Special Characters Excluded

The list of special characters appears to be incomplete. For example, square and curly braces are ignored in the search as well.

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

ID: e83b77e5-e3ec-6a5f-1f09-e232e2dda483
Version Independent ID: 0b263c72-95a2-d111-20a7-68857f288369
Content: Keyword queries and search conditions for Content Search - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/keyword-queries-and-search-conditions.md
Service: o365-seccomp
GitHub Login: @markjjo
Microsoft Alias: markjjo

@terrogin Thanks for the heads up. We'll look into this and update doc accordingly.


tvOS target?

Hi
Would it be possible to get a tvOS target? (I've forked a copy and added the new target for tvOS, and it seems to compiles fine, and even builds a framework but I couldn't get Carthage to see the tvOS scheme...  I'll be happy to issue a PR for my (utterly trivial) changes but might be faster for you to just add the target).
best,
David

Yeah, I'd be happy to - I'll try to get around to it some time tomorrow. I had already added tvOS and watchOS support with CocoaPods, but I had forgotten to add an actual target in the project itself.

Awesome - thanks! (btw, is there a trick to getting Carthage to pick up all the build targets? I matter what I tried I couldn't get it to see the SWXMLHashtvOS target I'd added to my copy, and the Carthage docs didn't offer any useful clues..)
best,
Dave

It took me a little longer than expected, but it should be out there now. I added a watchOS target, too.

Thank you very much!  And, by the way, thank you very much for creating such a great utility - it's saving me a lot of time grinding through really large and complex XML files that would have otherwise eaten up a lot of dev time to handle with NSXMLParser.
best,
Dave

I'm happy it is helping out - working with NSXMLParser can be a pain!


CSI: Should report success instead of error if attach call asking to attach an attached volume

Warning  FailedAttachVolume      13m (x2 over 15m)  attachdetach-controller                                      AttachVolume.Attach failed for volume "pvc-a37147de-969a-11e8-96b5-42010a80010c" : rpc error: code = Aborted desc = The volume pvc-a37147de-969a-11e8-96b5-42010a80010c is attached
Warning  FailedAttachVolume      5m (x9 over 7m)    attachdetach-controller                                      AttachVolume.Attach failed for volume "pvc-a37147de-969a-11e8-96b5-42010a80010c" : rpc error: code = Aborted desc = The volume pvc-a37147de-969a-11e8-96b5-42010a80010c is attached

This bug result in an attached volume cannot be used by a pod.
According to https://github.com/container-storage-interface/spec/blob/master/spec.md#controllerpublishvolume

This operation MUST be idempotent. If the volume corresponding to the volume_id has already been published at the node corresponding to the node_id, and is compatible with the specified volume_capability and readonly flag, the Plugin MUST reply 0 OK.

Please double check all the error code reported by the CSI plugin vs the spec carefully.

Fixed.


error while testing on TRAVIS-CI (Ubuntu)

Hi,
I saw the following dump pass by while testing
...
t/dumper/01-basic.t ............................. ok

WARNING: unhandled Failure detected in DESTROY:

Type Int does not support associative indexing.

  in sub pwd at /home/travis/build/MARTIMM/mongo-perl6-driver/.panda-work/1475147112_6/lib/Config/TOML/Parser/Actions.pm (Config::TOML::Parser::Actions) line 753

  in sub pwd at /home/travis/build/MARTIMM/mongo-perl6-driver/.panda-work/1475147112_6/lib/Config/TOML/Parser/Actions.pm (Config::TOML::Parser::Actions) line 725

  in method table:hoh at /home/travis/build/MARTIMM/mongo-perl6-driver/.panda-work/1475147112_6/lib/Config/TOML/Parser/Actions.pm (Config::TOML::Parser::Actions) line 561

  in regex table:hoh at /home/travis/build/MARTIMM/mongo-perl6-driver/.panda-work/1475147112_6/lib/Config/TOML/Parser/Grammar.pm (Config::TOML::Parser::Grammar) line 564

  in regex table at /home/travis/build/MARTIMM/mongo-perl6-driver/.panda-work/1475147112_6/lib/Config/TOML/Parser/Grammar.pm (Config::TOML::Parser::Grammar) line 560

  in regex segment:table at /home/travis/build/MARTIMM/mongo-perl6-driver/.panda-work/1475147112_6/lib/Config/TOML/Parser/Grammar.pm (Config::TOML::Parser::Grammar) line 615

  in regex segment at /home/travis/build/MARTIMM/mongo-perl6-driver/.panda-work/1475147112_6/lib/Config/TOML/Parser/Grammar.pm (Config::TOML::Parser::Grammar) line 597

  in regex document at /home/travis/build/MARTIMM/mongo-perl6-driver/.panda-work/1475147112_6/lib/Config/TOML/Parser/Grammar.pm (Config::TOML::Parser::Grammar) line 620

  in regex TOP at /home/travis/build/MARTIMM/mongo-perl6-driver/.panda-work/1475147112_6/lib/Config/TOML/Parser/Grammar.pm (Config::TOML::Parser::Grammar) line 631

  in block  at t/exceptions/01-parser.t line 288

  in sub subtest at /home/travis/.rakudobrew/moar-nom/install/share/perl6/sources/C712FE6969F786C9380D643DF17E85D06868219E (Test) line 346

  in block <unit> at t/exceptions/01-parser.t line 276

t/exceptions/01-parser.t ........................ ok
...

It didn't fail to install but perhaps needs some attention.
Perl6 version used in test is Rakudo version 2016.09-85-ged0ced2 built on MoarVM version 16.09-3-g9b39aa5 implementing Perl 6.c.
Greetings,
Marcel

Looks like it's being triggered by this bit of invalid TOML:
# DO NOT DO THIS
a = 1

[a.b]
c = 2

The most likely culprit is the pwd subroutine in Actions.pm. I could add a try/catch block there wherein an X::Config::TOML exception is raised when invalid associative indexing is detected. Such a thing should only happen if TOML keys are being overwritten, which makes for invalid TOML, as in this example.

I'm not seeing this come up in testing anymore. I will close #1 unless your experience installing Config::TOML with zef differs from my own.


Request: </ and /> ligatures for HTML tags.

Fira code has these, and I find they make reading HTML much easier.

Also, thanks for your work on this project. It is very much appreciated.

Yeah, I've actually redone all the ligatures and added a few more. It also supports more advanced OpenType features so it works more like Fira Code. I've just been so busy, I haven't had time to finish and update the docs.

Awesome. If I knew literally anything about fonts, I'd offer to help. Maybe I'll fiddle around with Glyphs this weekend, it seems pretty interesting.

Here's work in progress:

Done in 3ac6faed0c71802aa9887bc0c70f15496031df8c


Give the option to auto-retry failed jobs

What is the expected feature or enhancement?
Jobs sometimes fail because of temporary server / networking error, and a retry would work. Unfortunately for many iterative algorithms, this means the user would need to restart the entire workload from the beginning. For this reason QuantumInstance has built-in retry. Since we are encouraging people to use Qiskit Runtime / primitives and not QuantumInstance, it'd be nice to have the same capability.
Acceptance criteria
Have the options for users to specify max retry (0 being no retry) when running a QRT job.

I am curious as to why this is a runtime issue, and not an application issue.  Namely, a well-written iterative algorithm would save state in between calls so that if one job failed, the routine can be restarted in-place.

Two reasons - one being it makes it easier for algorithm developers so they don't need to each implement their own thing (unless they choose to), and we want algorithm dev to use our service. The other being many of these failures are caused by the current instability of the runtime service. Auto-retry is just an attempt to mitigate that a bit.

@nonhermitian, I agree with @jyu00. We already implement several flavors of "retry-primitives" to run our algorithms, you can check out this issue for reference, @luciacuervovalor keeps it updated with the latest retry features. It would be totally fine to rely on these kind of enhancements if the runtime failures were an exceptional thing, but our current situation is that we need them in almost every experiment we run. At least until the service is stabilized, I think this would be a great feature for the users.

I'm interested in this as well. I have had some issues with VQE calculations returning "Internal server errors" and it would be good to have the job retry a few times to check if the error was a just a fluke before the Session deactivates.

I've been messing around trying to see if I can solve this.
I tried creating a custom Estimator that is a child class of the runtime Estimator and inserts retries in the _run() method. This does't seem to work, as soon as a job fails all other tries immediately fail.
Seems like it would make more sense to add a max_job_retries option to Session(). I tried doing it by using the same method as above and creating a child CustomSession() class.
Unfortunately I get the following error
Backend <__main__.CustomSession object at 0x29adc3c70> cannot be found in any hub/group/project for this account. <__main__.CustomSession object at 0x29adc3c70> is of type <class '__main__.CustomSession'> but should instead be initialized through the <QiskitRuntimeService>."

Are we not allowed to build custom Sessions? If yes then I guess we need to wait for this to be implemented.

Hey @MarcoBarroca! Please look at @ElePT's message above, there's a link to the code we are using for this and hopefully it fixes your problem

Didn't see you had an implementation already!

Can we increase the priority of this. I continue to see people posting problems where a job has failed and has stopped things. It affects many algorithms, and users in general if they use the primitives themselves where the task can have several/many steps they are doing. In general the primitive has the best knowledge of whether an error is recoverable - and while it might be possible to signal that in the failure, this would require every primitive user to deal with this aspect. Having recovery in one place will make these more usable/consumable, and if logic needs updating due to new sources of failure possibilities it just has to be done in one place. Its why QuantumInstance had the equivalent function so it was one place, and all algorithms/users did not need to do this.


As a learner, I want the most robust and interactive course, so that I can stay engaged throughout my learning experience.

[ ] amplitude demo
[x] binary demo
[x] add 1 example of a video integrated

@korgan00 @vabarbosa Is the amplitude demo live and deployed? If so, this story can be closed.

the amplitude components are all available and ready in various forms:
https://learn.qiskit.org/course/examples/widgets#widgets-2-0
i am just not aware of them currently being used in any specific chapter/page?
CC: @korgan00 @frankharkins

The amplitude widgets are used here in the QML course here and I've added a PR to add them to the intro course too (https://github.com/Qiskit/platypus/pull/348)

Any blockers on that pull request on the intro course? I think that's the last thing before closing this.

it has been merged

Great. thank you! Closing.


minizip-ng: 3.0.10 -> 4.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/zlib-ng/minizip-ng/releases.
meta.description for minizip-ng is: Fork of the popular zip manipulation library found in the zlib distribution
meta.homepage for minizip-ng is: https://github.com/zlib-ng/minizip-ng
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.0.0 with grep in /nix/store/8vdjq71pnys26gjlpbcb4q2nq603p095-minizip-ng-4.0.0
found 4.0.0 in filename of file in /nix/store/8vdjq71pnys26gjlpbcb4q2nq603p095-minizip-ng-4.0.0

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

12 total rebuild path(s)

12 package rebuild(s)

First fifty rebuilds by attrpath
blender
blender-hip
dolphin-emu
embree
embree2
linphone
luxcorerender
minizip-ng
olive-editor
opencolorio
openimageio
osl

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8vdjq71pnys26gjlpbcb4q2nq603p095-minizip-ng-4.0.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8vdjq71pnys26gjlpbcb4q2nq603p095-minizip-ng-4.0.0
ls -la /nix/store/8vdjq71pnys26gjlpbcb4q2nq603p095-minizip-ng-4.0.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

8 packages failed to build:

blender
blender-hip
embree
embree2
olive-editor
opencolorio
openimageio
osl

3 packages built:

dolphin-emu
linphone
minizip-ng

Maintainer pings
cc @gebner @risicle for testing.

Fixed by #238978


Start at specified line/column number instead default of line 1, column 1.

Would it be possible to implement this in micro? Its implemented in other text-editors as the following:
vim "+call cursor(<LINE>, <COLUMN>)"
emacs +LINE:COLUMN
nano +LINE,COLUMN

There is no way to choose the starting line number from the command line right now. However, you can press CtrlL to jump to a given line number, and if you enable the savecursor option, micro will put the cursor at the line and column number it was at when you last closed the specific file (perhaps this is the behavior you are trying to achieve).

I actually don't want to start where I left off, but at specified positions from the get-go (usually at the end actually).
Your code is well written, and I see how I could implement it pretty easily. I can fork your code and send a pull request if I get a nice solution!

Sure go ahead!


Camera  plugin  Issue

CameraDeviceState: Legacy camera service transitioning to state ERROR

java.lang.NullPointerException: Attempt to invoke virtual method 'android.view.Surface android.media.ImageReader.getSurface()' on a null object reference
        at io.flutter.plugins.camera.Camera.startPreview(Camera.java:424)
        at io.flutter.plugins.camera.Camera$2.onOpened(Camera.java:160)
        at android.hardware.camera2.impl.CameraDeviceImpl$1.run(CameraDeviceImpl.java:143)

Hi @jackzhangkun
Can you please provide your flutter doctor -v,
your flutter run  --verbose
and your pubspec.yaml.
Thank you

@jackzhangkun
closing since additional information isn't provided


Secure connection to yahoo.bintray.com

Vespa already uses sign-artifacts for security.
But some security policy may require HTTPS.
Thanks,

Fixed in #4075


Adding Alibaba platform

Adding Alibaba cloud platform.

Agreed, LGTM, thanks @kwoodson
/lgtm

@deads2k @sttts Would either of you mind taking a look at this PR? Thanks in advance!

/approve


How to set current volume

How can I set the current volume for my template media_player?
I tried current_volume_template but that is not allowed.
I don't see it in the example config.

https://community.home-assistant.io/t/media-player-template/203062/69


Webview lost some css variables

Issue Type: Bug
In the latest insiders, it looks like some base css variables are missing (--color, --font-size, etc)
/cc @mjbvz
VS Code version: Code - Insiders 1.32.0-insider (1c50a87561055485d93a3961160902308497f1e3, 2019-02-08T06:16:17.233Z)
OS version: Windows_NT x64 10.0.18329

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz (8 x 4008)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
31.93GB (20.10GB free)

Process Argv

Screen Reader
no

VM
0%

This was an intentional break early in this iteration. Please switch over to using the --vscode-* prefixed variables instead as they are part of the official API: https://code.visualstudio.com/api/extension-guides/webview#theming-webview-content.
Example variables: --vscode-editor-font-family and --vscode-editor-font-size

@mjbvz is there any place extension devs can find out about intentional breaks right when they happen? I spent a lot of time figuring out the problem, fixing it, writing up a bug report, etc. Also now until we get a new release out, CodeStream is basically completely busted and GitLens' settings and welcome ui are useless as well.
I have no issue with breaks, it would just be nice to get a heads up somewhere -- and ideally even before it lands -- even if its a short window.

Sorry but these old css variables were never official api; they were more of an implementation detail. We do document changes to our real apis and generally provide at least one release to alert extension authors if we are intentionally breaking them

That's a pretty unsatisfying answer, FWIW. "Intentional break" was your phrase, not Eric's.


Is it possible to add trackers programmatically?

Hi,
I'm looking for the ability to add trackers to a torrent that is already downloading, is this possible.
I'm looking for something along the lines of:
torrent.addTracker('wss://tracker.com')

Sorry -- this is not supported. You have to add trackers upfront:
client.add(torrentId, { announce: ''wss://tracker.com' })

What is the use case for this?

I'm creating a private tracker to allow the download of software from my site.
I was thinking that the tracker URL will contain a key to identify a user and their access to the torrent. Which would change every so often to prevent it from being shared. So it would need to delete the old tracker URL and replace it with a new one containing a new key.
I'm probably completely over thinking this so please let me know if you know of a simpler way to achieve this.
Sent from my Samsung Galaxy smartphone.
-------- Original message --------From: Feross Aboukhadijeh<EMAIL_ADDRESS>Date: 24/04/2016  9:43 a.m.  (GMT+00:00) To: feross/webtorrent<EMAIL_ADDRESS>Cc: Chris Harvey<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [feross/webtorrent] Is it possible to add trackers programmatically? (#770)
Sorry -- this is not supported. You have to add trackers upfront:
client.add(torrentId, { announce: ''wss://tracker.com' })
What is the use case for this?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

You might consider using the getAnnounceOpts property of the client.add function. It's documented here: https://github.com/feross/bittorrent-tracker#client
You could use it like this:
var token = 1

client.add(torrentId, {
   // include a token parameter in announce requests to the server
  getAnnounceOpts: function () {
    return {
      token: token
    }
  }
})

// update the token every 10s
setInterval(function () {
  token += 1
}, 10000)


Build assets with webpack

connects #492

remove any bower deps
build all assets with webpack

Change-Type: minor

ping @pimterry @emirotin if you get a chance today could you take a look?