feat: Velocity System [Comments Wanted]

I am going to get started on a VelocitySystem, it will be the same as all of the existing systems where it updates every frame.  Unlike the existing component/system frameworks, you must opt-in by doing something like:
function MyCustomNode() {
    Node.call(this);
    this._velocityID = this.addComponent(new Velocity());
    ...
}

(Meaning if you init the node, the Velocity component will not come by default.)
There are several advantages to this approach, and it should generate very smooth velocity components.  I'm thinking of using the PhysicsEngine and I might try to coerce it into generating the Velocities, but I'm open to other suggestions.
Then you would add logic something along the lines of:
MyCustomNode.prototype.onReceive = function onReceive(type, event) {
    this.getVelocity().addEvent(type, event);
    ...
}

The VelocitySystem would take care of direction changes, pauses, increasing / decreasing speed, etc.
Please share any comments / questions / concerns.

So, updating the VelocitySystem every frame would not work.. There would have to be corresponding touch/mouse events for every frame, which will not be the case.
Thus, it will be better to build this as a completely standalone piece and not as one of the core Systems.

I'm not really sure what the use case for such a system would be/ why it's needed. Can you give an example and propose an API?

@alexanderGugel The purpose of such a system is pretty obvious, calculate velocity of a touchmove for use in the PhysicsEngine.  My original thought of creating some type of system will not work, but having something that calculates the delta/velocity from touch events is very important.

The purpose of such a system is pretty obvious, calculate velocity of a touchmove for use in the PhysicsEngine.

I don't think we really need a system for that. While I agree that knowing the velocity of a touchmove is useful in a lot of cases, I don't think it makes sense to architect this as a separate system. I think adding custom components solves this use case:
node.addComponent({
    onReceive: function(type, ev) {
        switch (type) {
            case 'mousemove':
            case 'touchmove':
                // Calculate velocity here
                break;
        }
    }
});

@jd-carroll do you have issues with @alexanderGugel's proposed solution?

Didn't realize I never responded.. That's actually what I had implemented: https://github.com/jd-carroll/engine-seed/blob/scrollViewSample/src/index.js#L312
But, I think this subject needs to be revisited.  Whether touchmove or mousemove we need something that can calculate the delta.  The little sample I have only works for touchmove and (since it was a copy from the old framework) the manner in which velocity is calculated isn't great.


Karpenter fails to create node object if an instance does not have PrivateDnsName

Version

Karpenter: master

Kubernetes: v1.23.0
Expected Behavior
Karpenter creates node object.

Actual Behavior
Karpenter does launch instance, but no node object. Then it let Pods hanging.
The instance eventually does join the cluster just fine.
I am mostly concerned with the Pod being left hanging.

Steps to Reproduce the Problem

It just happens randomly on scaleup. I suspect the EC2 API simply doesn't set PrivateDNSName sometimes.
Resource Specs and Logs

2022-01-01T11:03:09.387Z        ERROR   controller.provisioning Could not launch node, launching instances, All attempts fail:
#1: got instance i-0759df6abd68fe02c but PrivateDnsName was not set
#2: got instance i-0759df6abd68fe02c but PrivateDnsName was not set
#3: got instance i-0759df6abd68fe02c but PrivateDnsName was not set     {"commit": "75afd60", "provisioner": "nodes"}

I suspect the EC2 API simply doesn't set PrivateDNSName sometimes.

This can happen in rare cases, when it does
It's imperative that the node has the provisioner label in its kubelet config so karpenter can manage it
The reconcile loop should fail, resulting in the pods triggering a new reconcile loop
The node should come online, pods may schedule to it or to another node
The extra node should be scaled down

Then it let Pods hanging.

This doesn't sound right to me. What do you mean by this? For how long?

I suspect the EC2 API simply doesn't set PrivateDNSName sometimes.

This can happen in rare cases, when it does
It's imperative that the node has the provisioner label in its kubelet config so karpenter can manage it
The reconcile loop should fail, resulting in the pods triggering a new reconcile loop
The node should come online, pods may schedule to it or to another node
The extra node should be scaled down

Then it let Pods hanging.

This doesn't sound right to me. What do you mean by this? For how long?

There is no reason not to create the node object. In this case, PrivateDNSName is not used for anything interesting. I think I should be able to fix that part.
The reconcile loop doesn't seem to be failing. Karpenter just stops caring about that Pod. The pod is hanging until the Node object is created by Kubelet and eventually schedules by kube-scheduler. This bit worries me a lot more. I have seen pods being left behind in a number of occasions now.

There is no reason not to create the node object. In this case, PrivateDNSName is not used for anything interesting. I think I should be able to fix that part.
The reconcile loop doesn't seem to be failing. Karpenter just stops caring about that Pod. The pod is hanging until the Node object is created by Kubelet and eventually schedules by kube-scheduler. This bit worries me a lot more. I have seen pods being left behind in a number of occasions now.

It's currently used for the node's name.

It's currently used for the node's name.

I think this should be fixed with https://github.com/aws/karpenter/pull/1070. Let's reopen if we see it after v0.5.4.

I have this issue


chore: add resourceType key when listing available questionnaires

Review Checklist

Summary*

[ ] fix: Bug fix for Add the items tackled here as checklists
[x] feat: Completed task
[ ] chore: Incomplete task

[ ] test: Sub-task 1

Structure*

[ ] The Pull Request has a proper title that conforms to our MR title standards
[ ] The Pull Request has one commit, and if there are more than one, they should be squashed
[ ] The commit should have a proper title and a short description
[ ] The commit must be signed off
[ ] Unused imports are not present
[ ] Dead code is not present
[ ] Ensure dry running library to confirm changes

Tests

[ ] Proper and high quality unit, integration and acceptance(if applicable) tests have been written
[ ] The coverage threshold should not be lowered

Sign off*

[ ] All comments have been resolved by the reviewers
[ ] Approved by Czar {replace_with_czar_name}
[ ] Signed off by second reviewer {replace_with_name}
[ ] Ensure all checks are done before merging :warning:
[ ] All PRs needs to be signed before merging :warning:

N/B:

Add a checklist if more than one item is done.
Add screenshots and/or images where necessary
Indicate any breakages caused in the UI :exclamation:
Where necessary, indicate which issue the Pull Request solves (Closes #)
Any new files are updated in the folder structure in the README

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.197%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6714

Relevant Lines:
8070

💛  - Coveralls


CJM-16883: XDM changes for STO & Test profile

https://jira.corp.adobe.com/browse/CJM-16883

@prabhum2 Please review. Also, I see people making changes to 4 files which include files from docs folder. Is this should be the practice to follow or the docs are autogenerated?

@prabhum2 Please review. Also, I see people making changes to 4 files which include files from docs folder eg. in PR - https://github.com/adobe/xdm/pull/1217/files. Is this should be the practice to follow or the docs are autogenerated?

Docs are autogenerated . You need to make changes to Schema json and example json files if necessary

@cmathis & @prabhum2 Could we please merge this PR?


Avoid JCMediaPlayerListener strong references

JCMediaPlayerListener instances in JCMediaManager held strong references, which was causing memory leaks
Fixes https://github.com/lipangit/JieCaoVideoPlayer/issues/110

Thank you.
I will study and test your code.

JCBuriedPointStandard jcBuriedPointStandard = new JCBuriedPointStandard(){...}
The jcBuriedPointStandard class make it public and static, maybe can sovle the problem , because the JCBuriedPointStandard do not rely on Context Activity and Application.

 Your plan is necessary and perfect, I put it back.

@lipangit No need to be ashamed, thanks for your latest improvements on the library

你好，咱们项目推出数字通证饺子token，https://github.com/lipangit/JZT， 给所有提交过pr的朋友30000个JZT，表示对你过去对项目支持的感谢。
请下载币威钱包https://www.bitcv.app/，把里面的以太坊地址给我，币威的地址站内转账不需要手续费。这东西值不值钱不重要，玩的是信仰。
Hello, let's launch the JiaoZiToken, https://github.com/lipangit/JZT, and give all the friends who submitted pr 30,000 JZT, thank you for your support of the project.
Please download the coin wallet https://www.bitcv.app/?from=bitcvCom and give me the address of the Ethereum. There is no charge for transferring money at the address station, if you do not know Chinese , you can give me other address of the eth. It is not important that the value of this thing is worthless. It is faith.


Do not attempt to kill the process if it has already exited

Closes https://github.com/microsoft/vscode-python/issues/22420
This bugs seems to have existed ever since rawProcessApi.ts was created. proc.killed can be false even after process has exited.

Ping @karthiknadig


FFMPEG VideoCapture performance

Is there a reason for using bicubic resampling in CvCapture_FFMPEG::retrieveFrame? (specifically, https://github.com/Itseez/opencv/blob/master/modules/videoio/src/cap_ffmpeg_impl.hpp#L772). Except color space conversion from RGB24 to BGR24, the method does not actually resample anything. Therefore in my opinion, instead of SWS_BICUBIC the flag should be either SWS_AREA or SWS_FAST_BILINEAR.

The same goes for VideoWriter as well; see https://github.com/Itseez/opencv/blob/master/modules/videoio/src/cap_ffmpeg_impl.hpp#L1459.

Right, I also saw this. But there is src_size = dst_size and there is no scaling.
I hope it is not used at all in ffmpeg.
I found unscaled flag in ffmpeg:
https://github.com/FFmpeg/FFmpeg/blob/26e8895b7395ed180dcb7e30465058c45050b5e2/libswscale/utils.c#L1132
And code that forces SWS_BICUBIC in case of "zero" flag:
https://github.com/FFmpeg/FFmpeg/blob/26e8895b7395ed180dcb7e30465058c45050b5e2/libswscale/utils.c#L1170-L1190
In any case, you are free to change value of this flag and make some performance tests.


SAK-29236 Peer assessment assignments average score misses the last record sometimes

When a student is grading another student submission in a peer assessment assignment the score is recorded in the db but the calculated score average is missing the last commited result.
What it's happening here is that process saves and retrieves rows in the same thread process and in some cases it has no time to store it to the db before the list is gotten.
Flushing must prevent that cases.

+1 looks correct to me

I'm not sure this was the right way to fix this. At the moment the org.sakaiproject.assignment.impl.AssignmentPeerAssessmentServiceImpl doesn't have any transaction code and so although this might mask the problem there may be other issues. The traditional pattern when using hibernate is to use the org.springframework.transaction.interceptor.TransactionProxyFactoryBean to proxy the service and then define which calls require a transaction. This way when the method call returns the the transactions gets committed.
You can see an example of this in the components.xml file where the bean org.sakaiproject.assignment.api.model.AssignmentSupplementItemService is defined.

Absolutely agree. It's just a workaround. When I've got time I'll try to analyze and get a better wide solution like you propose. Thanks.


Investigate options for dumping power cells (balls) back out on to the field for others to pick up

Can we reverse the intake to put the balls back on the field possibly for another robot to pickup?

An option would have to be added to the chooser from #42.

This can be done now by @rsninja722's "jam" mode


TextField Multiline has a border-line around it on Safari

Images & references
--> http://www.material-ui.com/#/components/text-field
Versions

Material-UI: 0.16.0
Browser: Sadari 10.0

@stunaz: can you please provide a screenshot?

Sure

and on the SelectField too

Looks like a duplicate of #5410.


Translation issue for other skills in macro

The skill "Piece By Piece" is not translated in german.

Thanks, I'll take a look.
On Thu, Nov 12, 2015 at 7:37 AM Ley21<EMAIL_ADDRESS>wrote:

The skill "Piece By Piece" is not translated in german.
—
Reply to this email directly or view it on GitHub
https://github.com/doxxx/ffxiv-craft-opt-web/issues/95.


Enhancement/block sync

[x] fix memory issue while block sync
[x] testing

:white_check_mark: Build AElf <IP_ADDRESS>3 completed (commit https://github.com/AElfProject/AElf/commit/edff6301ce by @rosona)

:white_check_mark: Build AElf <IP_ADDRESS>6 completed (commit https://github.com/AElfProject/AElf/commit/74a3114d00 by @loning)


Having the post date in your CSV header is misleading.

I was getting started making use of your module. When I looked at your README and was concerned that perhaps this project may not be compatible with Chase Bank's current statements. I couldn't find any indication of a post date in my PDF. This is because Chase doesn't include a post date in their statements. Line 115 of chase2csv.py populates the post date column with an empty string. Removing that column or stating that it's not populated would reduce confusion.

This script is no longer required because you can download transactions as CSVs now.


No hints while extension is active

Just installed and don't see any hints (but keymaps like 'gg' seems to work)
It says that extension is enabled
Any ideas why I doesn't see any hints?
I'm on Mac mini, macOS Monterey 12.3.1

It seems to it doesn't appear automatically. Solved :)


Transaction subjects

Co-authored-by: Alex Stela
From @xstelea
I refactored watchUserDidConfirm and moved it outside of useTransactionsfunction.
In the confirm transaction modal there are at least 3 components rendered that import and use useTransactions , and each is subscribed to userConfirmation. This causes multiple subscriptions to trigger when the user confirms the transaction, resulting in multiple finalize& submit requests


Training with synthetic data

Dear Deep Learning Pros!
I am trying to refine the sample object-detection network with synthetic data instead of doing it with the KITTI data as in the tutorial. My goal is to compare it to a model refined with the KITTI data on real data.
In the first try I used all of my generated synthetic data (19000 images and labels) for training and the original KITTI data for validation. It is still in progress, but it looks like it's going to suck. Should I use synthetic data for validation instead and only compare the performance on real data? What do you think is the best way to refine networks with synthetic data?

Sorry for asking silly questions.
Thx Bernhard

In the end I used a dataset with synthetic data for training and validation which improved the training process:

After the successful training I tested the resulting model against 100 sample KITTI validation pictures and got 158 moslty right positioned bounding boxes but also some false positives. The KITTI data refined model found on the same pictures 294 bounding boxes, some of which were duplicates on the same object.
Sample result from the synthetic refined model:

Sample result from the KITTI refined model:

We were satisfied with the result of our first try, and there is a lot of room for improvement.


Create markdown demo parser

I have an idea for a new feature for jQuery Terminal
There are a few options:

Native one (that uses regex).
Using a real parser that needs to be patched.
Creating something with Peggy.

Maybe a better idea will be to create a Tutorial on how to create Markdown Parser for jQuery Terminal with Peggy.

Markdown parser added to Wiki


Update async udf example

Here are some proposed changes to https://github.com/apache/arrow-datafusion/pull/6713 that I think make the example stronger.  I still have some more changes I would like to do (update the wording, add some more comments about the rewriting)
Thank you @marshauf  -- this is a non trivial thing to have figured out
Here is an example of what it looks like now
 cargo run --example async_udf

Input:
+-------------------------------+
| url                           |
+-------------------------------+
| http://example.com/index.html |
| http://example.com/data.txt   |
+-------------------------------+
Output
+------------------------------------------------+
| get_url(t.url)                                 |
+------------------------------------------------+
| Remote data from http://example.com/index.html |
| Remote data from http://example.com/data.txt   |
+------------------------------------------------+
Output
+---------------+-------------------------------------------------+
| plan_type     | plan                                            |
+---------------+-------------------------------------------------+
| logical_plan  | pow expr.len(1)                                 |
|               |   TableScan: t projection=[url]                 |
| physical_plan | GetUrlExec                                      |
|               |   MemoryExec: partitions=1, partition_sizes=[1] |
|               |                                                 |
+---------------+-------------------------------------------------+

Hi @alamb, looks much better. The executed example output is also nice.
It would be nice if the example didn't replace the complete Projection. At the moment you can't write SELECT get_url(url), url FROM t. It results in Error: ArrowError(InvalidArgumentError("number of columns(1) must match number of fields(2) in schema")) error. I am still trying to figure out how to replace the UDF Expr inside the Projection with an Extension.

At the moment you can't write SELECT get_url(url), url FROM t. It results in Error: ArrowError(InvalidArgumentError("number of columns(1) must match number of fields(2) in schema")) error. I am still trying to figure out how to replace the UDF Expr inside the Projection with an Extension.

Thanks -- I'll figure out how to do it, but I may not get a chance for the next few days


Cryptographic API Misuse Vulnerability : Missing SSL Certificate Validation

Do not use unverified hostname or certificates in connection
Do not disable HTTPS warnings
Description:
In the HTTPie 3.2.2 , there are sections where SSL certificate validation appears to be missing. Proper SSL certificate validation is a cornerstone of secure communication over HTTPS, and its absence can lead to severe security risks such as Man-In-The-Middle (MITM) attacks. In particular, the code does not enforce hostname verification or certificate validation, and it may also be suppressing HTTPS-related warnings that would typically alert a user to a potential security issue.
Affected Version
v3.2.2
Location:
https://github.com/httpie/cli/blob/master/httpie/internal/update_warnings.py#L44
https://github.com/httpie/cli/blob/master/httpie/client.py#L33
Reference

CWE-295: Improper Certificate Validation
CWE-200: Exposure of Sensitive Information to an Unauthorized Actor
CWE-319: Cleartext Transmission of Sensitive Information

Expected Behavior:
The expected behavior for any HTTPS connection is that the client should validate the SSL certificate provided by the server to ensure it is trusted, not expired, and matches the requested hostname. Additionally, any HTTPS warnings should be displayed to the user, rather than being disabled, to avoid security oversights.
Actual Behavior:
The actual behavior observed in the code indicates that SSL certificate validation may not be properly enforced. Furthermore, HTTPS warnings that are essential for debugging and security awareness are not displayed, potentially causing the users to remain unaware of misconfigured or insecure SSL implementations.
Recommendation

Do not use urllib3.disable_warnings() to close tls warning , it is so dangerous! Please delete it.
Set get request verify=True ,it will do ssl certificate validation.

Due to the sensitive nature of data transmitted over HTTPS, it is imperative to address this vulnerability promptly to maintain the integrity and confidentiality of client-server communication for HTTPie users.

I assume this is based on the bogus CVE-2023-48052.

Do not use urllib3.disable_warnings() to close tls warning , it is so dangerous! Please delete it.

Even though the printing of warnings to the terminal is disabled, SSL verification still does occur — unless the user disables it via --verify=no.  We don’t want SSL warnings in the output. Please see https://urllib3.readthedocs.io/en/stable/advanced-usage.html#tls-warnings

Set get request verify=True ,it will do ssl certificate validation.

verify=False is only used for internal version check requests to a hard-coded URL, not the actual user requests.
(Yes, it was left there unintentionally, and we’ll remove it in the upcoming release, but it poses no danger.)


Return "any" type for properties of type System.Object or dynamic.

This works for me, for this issue: https://github.com/frhagn/Typewriter/issues/43
Won't be offended if you reject this and have a better place to implement this, but figured I'd try to save you the work.

Nice @jonstelly, I'll merge right away...
Thanks for contributing!


Fix .tracked_files changing when updating a single app

When I run apibuilder update --app <something> I see incorrect deleted
lines in .tracked_files. The issue is that we are modifying an array
while iterating over it.

closing, but the bug still exists


Relace the Kotlin DSL - closure compatibility snippet

Issue type
Wrong or misleading information
Problem description
The current snippet is based on the com.jfrog.bintray plugin that is useless since Bintray is no longer in operation and is also not able to resolve since JCenter is going away fully.
Context (optional)
No response
Page with the problem
https://docs.gradle.org/current/userguide/kotlin_dsl.html#groovy_closures_from_kotlin

hey @ljacomet I would like to work on this


Error in test-compiler-hook

The following backtrace occurs when running make CXX=clang++ test-tweedledum on the latest master branch (i.e., e1d1333ad5cb0a0086ba8c3236f8a4ffe9292aae):
  TEST-COMPILER-HOOKA violent error occurred when compressing a subsequence.
The offending subsequence is:
RX(-1.5707963267948966) 1
RZ(1.5707963267948966) 1
RZ(3.141592653589793) 2
RX(1.5707963267948966) 2
RZ(-2.582630569970398) 2
RX(1.5707963267948966) 2
CZ 2 1
RX(1.5707963267948966) 2
RZ(-0.1840956864344647) 1
RX(1.5707963267948966) 1
RZ(2.366425470274423) 1
RX(-1.5707963267948966) 1
RZ(-6.134161453323383) 1
RZ(-1.678475176643765) 0
RZ(1.725164134539095) 0
RX(1.5707963267948966) 0
RZ(2.566493759327286) 0
RX(-1.5707963267948966) 0
RZ(1.8666163417301669) 0
CZ 1 0
RX(1.5707963267948966) 1
RZ(-1.5707963267948966) 1
RX(-1.5707963267948966) 1
RX(1.5707963267948966) 0
RZ(1.5707963267948966) 0
RX(-1.5707963267948966) 0
RZ(0.0) 1
CZ 1 0
RX(1.5707963267948966) 0
RZ(-1.5707963267948966) 0
RX(-1.5707963267948966) 0
RX(1.5707963267948966) 1
RZ(1.5707963267948966) 1
RX(-1.5707963267948966) 1
CZ 1 0
RZ(0.555018078283987) 1
RX(1.5707963267948966) 1
RZ(1.4115589539841191) 1
RX(-1.5707963267948966) 1
RZ(-6.187181616688628) 1
RZ(2.9971403537086156) 0
RX(1.5707963267948966) 0
RZ(0.8064312117265775) 0
RX(-1.5707963267948966) 0
RZ(-2.750326351161305) 0
RZ(-1.5707963267948966) 2
RX(-1.5707963267948966) 2
RZ(-1.5707963267948966) 2
RX(1.5707963267948966) 2
RZ(1.5707963267948966) 2
RX(-1.5707963267948966) 2
RZ(3.141592653589793) 0
CZ 2 1
RX(1.5707963267948966) 0
RZ(0.0) 0
RX(-1.5707963267948966) 0
RZ(0.0) 0
RX(1.5707963267948966) 2
RZ(-0.1840956864344647) 1
RX(1.5707963267948966) 1
RZ(2.366425470274423) 1
RX(-1.5707963267948966) 1
RZ(-6.134161453323383) 1
RZ(1.725164134539095) 0
RX(1.5707963267948966) 0
RZ(2.566493759327286) 0
RX(-1.5707963267948966) 0
RZ(1.8666163417301669) 0
CZ 1 0
RX(1.5707963267948966) 1
RZ(-1.5707963267948966) 1
RX(-1.5707963267948966) 1
RX(1.5707963267948966) 0
RZ(1.5707963267948966) 0
RX(-1.5707963267948966) 0
RZ(0.0) 1
CZ 1 0
RX(1.5707963267948966) 0
RZ(-1.5707963267948966) 0
RX(-1.5707963267948966) 0
RX(1.5707963267948966) 1
RZ(1.5707963267948966) 1
RX(-1.5707963267948966) 1
CZ 1 0
RZ(0.555018078283987) 1
RX(1.5707963267948966) 1
RZ(1.4115589539841191) 1
RX(-1.5707963267948966) 1
RZ(-6.187181616688628) 1
RZ(2.9971403537086156) 0
RX(1.5707963267948966) 0
RZ(0.8064312117265775) 0
RX(-1.5707963267948966) 0
RZ(-2.750326351161305) 0
RZ(-1.5707963267948966) 2
RX(-1.5707963267948966) 2
RX(1.5707963267948966) 2
RZ(1.5707963267948966) 2
RX(-1.5707963267948966) 2
The current compression context is:
#S(CL-QUIL::COMPRESSOR-CONTEXT :AQVM #S(CL-QUIL::ANTISOCIAL-QVM :WFS #(#(#C(0.7071067811865468d0 1.4475911902159847d-16) #C(-0.07599339215309883d0 0.703011382801919d0)) #(#C(0.7071067811865474d0 0.0d0) #C(2.895182380431968d-16 0.7071067811865477d0)) #(#C(0.13945812262284524d0 -0.693218170588821d0) #C(0.6932181705888208d0 0.13945812262284524d0)) #(#C(1.1102230246251565d-16 -0.7071067811865476d0) #C(0.0d0 -0.7071067811865475d0)) #(#C(1.0d0 0.0d0) #C(0.0d0 0.0d0))) :INTERNAL-INDICES #(0 0 0 0 0)) :CHIP-SPECIFICATION #<CHIP-SPECIFICATION of 5:4 objects>)
Unhandled SIMPLE-ERROR in thread #<SB-THREAD:THREAD "main thread" RUNNING
                                    {10005305B3}>:
  During careful checking of instruction compression, the produced wavefunction by state prep reduction was detected to not be collinear with the target wavefunction.

Backtrace for: #<SB-THREAD:THREAD "main thread" RUNNING {10005305B3}>
0: (SB-DEBUG::DEBUGGER-DISABLED-HOOK #<SIMPLE-ERROR #<(SIMPLE-ARRAY CHARACTER (206)) During careful checking of instruction compression, the produced ~
                   wavefunction by state prep reduction was detected to not be ~
                   collinear with the target wavefun... {10016A7B5F}> {10082C7743}> #<unused argument> :QUIT T)
1: (SB-DEBUG::RUN-HOOK SB-EXT:*INVOKE-DEBUGGER-HOOK* #<SIMPLE-ERROR #<(SIMPLE-ARRAY CHARACTER (206)) During careful checking of instruction compression, the produced ~
                   wavefunction by state prep reduction was detected to not be ~
                   collinear with the target wavefun... {10016A7B5F}> {10082C7743}>)
2: (INVOKE-DEBUGGER #<SIMPLE-ERROR #<(SIMPLE-ARRAY CHARACTER (206)) During careful checking of instruction compression, the produced ~
                   wavefunction by state prep reduction was detected to not be ~
                   collinear with the target wavefun... {10016A7B5F}> {10082C7743}>)
3: (ERROR #<SIMPLE-ERROR #<(SIMPLE-ARRAY CHARACTER (206)) During careful checking of instruction compression, the produced ~
                   wavefunction by state prep reduction was detected to not be ~
                   collinear with the target wavefun... {10016A7B5F}> {10082C7743}>)
4: (CL-QUIL::COMPRESS-INSTRUCTIONS-IN-CONTEXT (#<RX(-pi/2) 1> #<RZ(pi/2) 1> #<RZ(pi) 2> #<RX(pi/2) 2> #<RZ(-2.582630569970398) 2> #<RX(pi/2) 2> #<CZ 2 1> #<RX(pi/2) 2> #<RZ(-0.1840956864344647) 1> #<RX(pi/2) 1> #<RZ(2.366425470274423) 1> #<RX(-pi/2) 1> ...) #<CL-QUIL::CHIP-SPECIFICATION of 5:4 objects> #S(CL-QUIL::COMPRESSOR-CONTEXT :AQVM #S(CL-QUIL::ANTISOCIAL-QVM :WFS #(#(#C(0.7071067811865468d0 1.4475911902159847d-16) #C(-0.07599339215309883d0 0.703011382801919d0)) #(#C(0.7071067811865474d0 0.0d0) #C(2.895182380431968d-16 0.7071067811865477d0)) #(#C(0.13945812262284524d0 -0.693218170588821d0) #C(0.6932181705888208d0 0.13945812262284524d0)) #(#C(1.1102230246251565d-16 -0.7071067811865476d0) #C(0.0d0 -0.7071067811865475d0)) #(#C(1.0d0 0.0d0) #C(0.0d0 0.0d0))) :INTERNAL-INDICES #(0 0 0 0 0)) :CHIP-SPECIFICATION #<CL-QUIL::CHIP-SPECIFICATION of 5:4 objects>))
5: (CL-QUIL::COMPRESS-INSTRUCTIONS-WITH-POSSIBLY-UNKNOWN-PARAMS (#<RX(-pi/2) 1> #<RZ(pi/2) 1> #<RZ(pi) 2> #<RX(pi/2) 2> #<RZ(-2.582630569970398) 2> #<RX(pi/2) 2> #<CZ 2 1> #<RX(pi/2) 2> #<RZ(-0.1840956864344647) 1> #<RX(pi/2) 1> #<RZ(2.366425470274423) 1> #<RX(-pi/2) 1> ...) #<CL-QUIL::CHIP-SPECIFICATION of 5:4 objects> #S(CL-QUIL::COMPRESSOR-CONTEXT :AQVM #S(CL-QUIL::ANTISOCIAL-QVM :WFS #(#(#C(0.7071067811865468d0 1.4475911902159847d-16) #C(-0.07599339215309883d0 0.703011382801919d0)) #(#C(0.7071067811865474d0 0.0d0) #C(2.895182380431968d-16 0.7071067811865477d0)) #(#C(0.13945812262284524d0 -0.693218170588821d0) #C(0.6932181705888208d0 0.13945812262284524d0)) #(#C(1.1102230246251565d-16 -0.7071067811865476d0) #C(0.0d0 -0.7071067811865475d0)) #(#C(1.0d0 0.0d0) #C(0.0d0 0.0d0))) :INTERNAL-INDICES #(0 0 0 0 0)) :CHIP-SPECIFICATION #<CL-QUIL::CHIP-SPECIFICATION of 5:4 objects>) NIL)
6: ((LABELS CL-QUIL::TRANSITION-GOVERNOR-STATE :IN CL-QUIL::COMPRESS-INSTRUCTIONS) :GLOBAL :GLOBAL :FLUSHING NIL)
7: ((LABELS CL-QUIL::TRANSITION-GOVERNOR-STATE :IN CL-QUIL::COMPRESS-INSTRUCTIONS) 1 2 :QUEUEING NIL)
8: ((LABELS CL-QUIL::PROCESS-INSTRUCTION :IN CL-QUIL::COMPRESS-INSTRUCTIONS) #<CZ 2 3> 1 2)
9: (CL-QUIL::COMPRESS-INSTRUCTIONS (#<RZ(0.0) 1> #<RZ(0.0) 2> #<RZ(0.0) 3> #<RZ(0.0) 4> #<RZ(pi) 0> #<RX(pi/2) 1> #<RX(pi/2) 2> #<RX(pi/2) 3> #<RX(pi/2) 4> #<RX(pi/2) 0> #<RZ(0.0) 1> #<RZ(0.0) 2> ...) #<CL-QUIL::CHIP-SPECIFICATION of 5:4 objects> :PROTOQUIL T)
10: ((LABELS CL-QUIL::PROCESS-BLOCK :IN COMPILER-HOOK) #<CL-QUIL::BASIC-BLOCK ENTRY-BLK-1325 len:15 in:0 out:term {1007285F03}> NIL)
11: (COMPILER-HOOK #<PARSED-PROGRAM {1007277693}> #<CL-QUIL::CHIP-SPECIFICATION of 5:4 objects> :PROTOQUIL NIL :REWIRING-TYPE NIL)
12: (COMPARE-COMPILED #P"/home/jmbr/rigetti/quilc/tests/compiler-hook-test-files/sohaib.quil" :CZ)
13: ((LABELS TEST-COMPILER-HOOK :IN TEST-COMPILER-HOOK))
14: ((LABELS FIASCO::RUN-TEST-BODY :IN FIASCO::RUN-TEST-BODY-IN-HANDLERS))
15: (FIASCO::CALL-WITH-TEST-HANDLERS #<CLOSURE (LAMBDA NIL :IN FIASCO::RUN-TEST-BODY-IN-HANDLERS) {100672633B}>)
16: (FIASCO::PRETTY-RUN-TEST #<test TEST-COMPILER-HOOK> #<FUNCTION (LABELS TEST-COMPILER-HOOK :IN TEST-COMPILER-HOOK) {53DC771B}>)
17: ((LABELS #:BODY-SYM0 :IN TEST-COMPILER-HOOK))
18: (TEST-COMPILER-HOOK)
19: ((LABELS FIASCO-SUITES::CL-QUIL-TESTS :IN FIASCO-SUITES::CL-QUIL-TESTS))
20: ((LABELS FIASCO::RUN-TEST-BODY :IN FIASCO::RUN-TEST-BODY-IN-HANDLERS))
21: (FIASCO::CALL-WITH-TEST-HANDLERS #<CLOSURE (LAMBDA NIL :IN FIASCO::RUN-TEST-BODY-IN-HANDLERS) {1002BE204B}>)
22: (FIASCO::PRETTY-RUN-TEST #<test FIASCO-SUITES::CL-QUIL-TESTS :tests 152> #<FUNCTION (LABELS FIASCO-SUITES::CL-QUIL-TESTS :IN FIASCO-SUITES::CL-QUIL-TESTS) {53BE090B}>)
23: ((LABELS #:BODY-SYM1 :IN FIASCO-SUITES::CL-QUIL-TESTS))
24: (FIASCO-SUITES::CL-QUIL-TESTS)
25: (RUN-SUITE-TESTS #<test FIASCO-SUITES::CL-QUIL-TESTS :tests 152> :VERBOSE NIL :STREAM #<SYNONYM-STREAM :SYMBOL SB-SYS:*STDOUT* {1000006213}> :INTERACTIVE T)
26: (RUN-TESTS :CL-QUIL-TESTS :DESCRIBE-FAILURES T :VERBOSE NIL :STREAM #<SYNONYM-STREAM :SYMBOL SB-SYS:*STDOUT* {1000006213}> :INTERACTIVE T)
27: (RUN-CL-QUIL-TESTS :VERBOSE NIL :HEADLESS NIL)
28: ((SB-PCL::EMF ASDF/ACTION:PERFORM) #<unused argument> #<unused argument> #<ASDF/LISP-ACTION:TEST-OP > #<ASDF/SYSTEM:SYSTEM "cl-quil/tweedledum-tests">)
29: ((LAMBDA NIL :IN ASDF/ACTION:CALL-WHILE-VISITING-ACTION))
30: ((:METHOD ASDF/ACTION:PERFORM-WITH-RESTARTS :AROUND (T T)) #<ASDF/LISP-ACTION:TEST-OP > #<ASDF/SYSTEM:SYSTEM "cl-quil/tweedledum-tests">) [fast-method]
31: ((:METHOD ASDF/PLAN:PERFORM-PLAN (T)) #<ASDF/PLAN:SEQUENTIAL-PLAN {1007FE33A3}>) [fast-method]
32: ((FLET SB-C::WITH-IT :IN SB-C::%WITH-COMPILATION-UNIT))
33: ((:METHOD ASDF/PLAN:PERFORM-PLAN :AROUND (T)) #<ASDF/PLAN:SEQUENTIAL-PLAN {1007FE33A3}>) [fast-method]
34: ((:METHOD ASDF/OPERATE:OPERATE (ASDF/OPERATION:OPERATION ASDF/COMPONENT:COMPONENT)) #<ASDF/LISP-ACTION:TEST-OP > #<ASDF/SYSTEM:SYSTEM "cl-quil/tweedledum-tests"> :PLAN-CLASS NIL :PLAN-OPTIONS NIL) [fast-method]
35: ((SB-PCL::EMF ASDF/OPERATE:OPERATE) #<unused argument> #<unused argument> #<ASDF/LISP-ACTION:TEST-OP > #<ASDF/SYSTEM:SYSTEM "cl-quil/tweedledum-tests">)
36: ((LAMBDA NIL :IN ASDF/OPERATE:OPERATE))
37: ((:METHOD ASDF/OPERATE:OPERATE :AROUND (T T)) #<ASDF/LISP-ACTION:TEST-OP > #<ASDF/SYSTEM:SYSTEM "cl-quil/tweedledum-tests">) [fast-method]
38: ((SB-PCL::EMF ASDF/OPERATE:OPERATE) #<unused argument> #<unused argument> ASDF/LISP-ACTION:TEST-OP :CL-QUIL/TWEEDLEDUM-TESTS)
39: ((LAMBDA NIL :IN ASDF/OPERATE:OPERATE))
40: ((:METHOD ASDF/OPERATE:OPERATE :AROUND (T T)) ASDF/LISP-ACTION:TEST-OP :CL-QUIL/TWEEDLEDUM-TESTS) [fast-method]
41: (ASDF/SESSION:CALL-WITH-ASDF-SESSION #<CLOSURE (LAMBDA NIL :IN ASDF/OPERATE:OPERATE) {1007F7C48B}> :OVERRIDE T :KEY NIL :OVERRIDE-CACHE T :OVERRIDE-FORCING NIL)
42: ((LAMBDA NIL :IN ASDF/OPERATE:OPERATE))
43: (ASDF/SESSION:CALL-WITH-ASDF-SESSION #<CLOSURE (LAMBDA NIL :IN ASDF/OPERATE:OPERATE) {1007EE6EAB}> :OVERRIDE NIL :KEY NIL :OVERRIDE-CACHE NIL :OVERRIDE-FORCING NIL)
44: ((:METHOD ASDF/OPERATE:OPERATE :AROUND (T T)) ASDF/LISP-ACTION:TEST-OP :CL-QUIL/TWEEDLEDUM-TESTS) [fast-method]
45: (ASDF/OPERATE:TEST-SYSTEM :CL-QUIL/TWEEDLEDUM-TESTS)
46: (SB-INT:SIMPLE-EVAL-IN-LEXENV (ASDF/OPERATE:TEST-SYSTEM :CL-QUIL/TWEEDLEDUM-TESTS) #<NULL-LEXENV>)
47: (EVAL (ASDF/OPERATE:TEST-SYSTEM :CL-QUIL/TWEEDLEDUM-TESTS))
48: (SB-IMPL::PROCESS-EVAL/LOAD-OPTIONS ((:LOAD . "/home/jmbr/quicklisp/setup.lisp") (:EVAL . "(push (truename \".\") asdf:*central-registry*)") (:EVAL . "(push :hunchentoot-no-ssl *features*)") (:EVAL . "(push :drakma-no-ssl *features*)") (:EVAL . "(push (truename \"../\") ql:*local-project-directories*)") (:EVAL . "(ql:quickload :cl-quil/tweedledum-tests)") (:EVAL . "(asdf:test-system :cl-quil/tweedledum-tests)") (:QUIT)))
49: (SB-IMPL::TOPLEVEL-INIT)
50: ((FLET SB-UNIX::BODY :IN SB-EXT:SAVE-LISP-AND-DIE))
51: ((FLET "WITHOUT-INTERRUPTS-BODY-7" :IN SB-EXT:SAVE-LISP-AND-DIE))
52: ((LABELS SB-IMPL::RESTART-LISP :IN SB-EXT:SAVE-LISP-AND-DIE))

@jmbr Does this occur on every test run? There is a very small but non-zero chance that numerical instability leads to a bad matrix somewhere along the way.

That's interesting. I just ran it five times in a row and it happened every time. I've recompiled QVM as well after pulling the latest changes and the error persists.

@jmbr Interesting. I assume you've done make clean? Out of interest, what is the githash of your tweedledum submodule?

Yeah, this is after a make clean. The git hash of my tweedledum submodule is 8de4d5bbf1b96d15c0e7f66a443fdd50f3509c09

Maybe the next step is to blow away the cache. Grasping at straws! make clean-cache

Cleaned the cache and repeated with the same effect. FWIW, the output of
(format t "~a ~a~%" (lisp-implementation-type) (lisp-implementation-version))

is SBCL <IP_ADDRESS>8-d5bbd32

@jmbr if you roll back your to before https://github.com/rigetti/quilc/commit/e1d1333ad5cb0a0086ba8c3236f8a4ffe9292aae do you still see the error?

Reverted to 3a5c1f51c48f8ba4a967ed1f98c166f5076128de but to no avail.
Another thing I did not mention is that I'm on a Linux system with kernel version 4.4.0-28-generic (as reported by uname).
I should be able to debug this problem myself later on when I'm more familiar with our code base.

I could also be due to lapack/blas versions. You can find the lapack/blas libs that are known to work in the sdk download: https://downloads.rigetti.com/qcs-sdk/forest-sdk-linux-deb.tar.bz

That might be it. I use OpenBLAS.

As an additional data point, I'm also on Linux but was unable to reproduce. I only ran the full test-tweedledum target once, but I let it run through test-compiler-hook a handful of times. Here are my relevant package versions.
linux-lts 4.19.37-1
blas 3.8.0-2
lapack 3.8.0-2
clang 8.0.0-4
sbcl 1.4.16-1

I am testing on master at commit ce80998.
As a side note, not sure if it's my imagination but my cpu fans sound a lot less like a helicopter landing in my backyard when running the compiler hook tests with tweedledum enabled. This is a welcome change because my laptop has thermal problems and shuts down if it overheats.
Is libtweedledum mandatory (except for maybe windows platforms) or just a convenience that we use whenever it's available? I think it might pay off to write a CMake (or autoconf or whatever) configuration script to take care of finding the right compiler, installation directories, etc. If that's interesting I can go ahead and do it (I'd vote for CMake in that case).

notmgsk will know better than me, but my understanding is that tweedledum is not required, but is a nice optimization that speeds up operations with "permutation gates" (or whatever you call them) considerably.
I don't have much of an opinion on the CMake/autoconf question, but I guess the main requirement would be to get it to integrate seamlessly with the ASDF build. There is also the minor concern of introducing a new build-time dependency on CMake, but I already have it installed so I am naturally unconcerned :).

We should treat tweedledum as mandatory. Once #276 is merged, it will be mandatory. It is a very useful optimization to the compiler, as @appleby has pointed out.
As to the CMake suggestion, I don't have an opinion either way. I'm not sure what benefit it would provide, but I also haven't used CMake much. If you think it will be useful and is worth the effort, go for it. Can't speak for the others, of course.

@jmbr still seeing this?

Unfortunately, yes (with latest master branch). I don't see it with the Macbook pro, though, and it may very well be that I have a peculiar configuration in this system.

Is this still alive?

Haven't seen it in a long time. But then again, I mostly use quilc in macOS laptop. I think we can close since no other user has raised the problem.


Linker failure, possible version mis-match?

On building https://github.com/CandleCandle/pony-eventbus (but not limited to that project)
OSX El Capitan: 10.11.6
Using locally-built ponyc
$ ponyc --version
0.4.0-8d8c38a [release] (llvm 3.6.2)

works as expected.
Using Brew supplied ponyc
$ ponyc --version
0.4.0 [release] (llvm 3.8.1)

Produces the below output:
$ ponyc --output /Users/--/pony/pony-eventbus/target/test --path /Users/--/pony/pony-eventbus/src/main/pony
Building builtin -> /usr/local/Cellar/ponyc/0.4.0/packages/builtin
Building . -> /Users/--/pony/pony-eventbus/src/test/pony/uk.me.candle/pony-eventbus
Building ponytest -> /usr/local/Cellar/ponyc/0.4.0/packages/ponytest
Building time -> /usr/local/Cellar/ponyc/0.4.0/packages/time
Building collections -> /usr/local/Cellar/ponyc/0.4.0/packages/collections
Building uk.me.candle/pony-eventbus -> /Users/--/pony/pony-eventbus/src/main/pony/uk.me.candle/pony-eventbus
Generating
 Reachability
 Selector painting
 Data prototypes
 Data types
 Function prototypes
 Functions
 Descriptors
Optimising
Writing /Users/--/pony/pony-eventbus/target/test/pony-eventbus.o
Linking /Users/--/pony/pony-eventbus/target/test/pony-eventbus
error: Invalid bitcode version (Producer: '8<IP_ADDRESS>_0' Reader: '703.0.29_0')
Error:
unable to link: ld -execute -no_pie -dead_strip -arch x86_64 -macosx_version_min 10.8 -o /Users/--/pony/pony-eventbus/target/test/pony-eventbus /Users/--/pony/pony-eventbus/target/test/pony-eventbus.o -L"/Users/--/pony/pony-eventbus/src/main/pony" -L"/usr/local/Cellar/ponyc/0.4.0/bin/" -L"/usr/local/Cellar/ponyc/0.4.0/bin/../lib" -L"/usr/local/Cellar/ponyc/0.4.0/bin/../packages" -L"/usr/local/lib"  -lponyrt -lSystem

I think this is related to LTO. With LTO enabled, the Clang linker takes LLVM bitcode files in input instead of object files. The prebuilt ponyc uses LLVM 3.8 and your system has something like LLVM 3.7 (according to Apple versioning). Since LLVM tools in a given version expect bitcode in the same version, it can't work. I'm not sure this one can be solved without disabling LTO in prebuilt packages.

We can make the specific version of LLVM be the requirement with homebrew rather than any llvm. That should work right @praetonus?

I've never used Homebrew but it sounds like it could work. Definitely worth a try.

I opened a PR for this. There's no way to test without going through their build process.
https://github.com/Homebrew/homebrew-core/pull/5343

@CandleCandle thanks for bringing this issue to our attention.

@CandleCandle are you using XCode 7 or 8?

upgrading to xcode 8 fixes this problem for me. @CandleCandle can you verify that works for you and we can note in the directions that its required if it works for you.

Actually, I would really appreciate it if you could try
brew install -s ponyc

first without upgrading your Xcode, and see if that works.
It will help us make the right decision on how to fix this problem on the Homebrew side.

@CandleCandle see above... ^^^

@CandleCandle also see: https://github.com/Homebrew/homebrew-core/issues/5346

Thanks for your help @zmwangx

Building from source with brew install -s ponyc does not reproduce the error above, and generates:
$ ponyc --version
0.4.0 [release] (llvm 3.8.1)

$ xcodebuild -version
Xcode 7.3

Thanks @CandleCandle.
@zmwangx assuming you do an upstream fix, what do you think the timeline would be on that?

@SeanTAllen I have opened https://github.com/Homebrew/homebrew-core/pull/5415.
Could you please give us a day or two to review and get it merged, and in the mean time ask users with Xcode 7.3 to build from source (brew install -s ponyc)? Thanks.

I can do that @zmwangx. Could you drop a note here when that gets merged?

Definitely!

Thank you @zmwangx!

@SeanTAllen I have merged Homebrew/homebrew-core#5415.

This has been fixed on the Homebrew side.

Thanks for reporting and helping get to the bottom of this @CandleCandle.

This is an issue again. If you are on say, Sierra and still have XCode 8, you run into this issue now that XCode 9 is out (using the prebuilt binary). We should have a short talk about:

are we ok with making the latest xcode used by homebrew folks a requirement to use pre-built ponyc
turn off LTO in prebuilt ponyc to avoid #1


Error not properly handled. Powershell will exit - [process exited with code<PHONE_NUMBER> (0xe0434352)]

I don't know if I am making a mistake but when I run " Import-Module -Name Microsoft.WinGet.CommandNotFound" in Powershell 7.4.2 on windows home 11 23h2, Powershell gives the following error:
"An error has occurred that was not properly handled. Additional information is shown below. The PowerShell process will exit.
Unhandled exception. System.Management.Automation.CmdletInvocationException: An error occurred while connecting to the catalog.
---> Microsoft.WinGet.Client.Engine.Exceptions.CatalogConnectException: An error occurred while connecting to the catalog.
at Microsoft.WinGet.Client.Engine.Commands.Common.FinderCommand.GetPackageCatalog(CompositeSearchBehavior behavior)
at Microsoft.WinGet.Client.Engine.Commands.Common.FinderCommand.FindPackages(CompositeSearchBehavior behavior, UInt32 limit, PackageFieldMatchOption match)
at Microsoft.WinGet.Client.Engine.Commands.Common.FinderExtendedCommand.FindPackages(CompositeSearchBehavior behavior, PackageFieldMatchOption match)
at Microsoft.WinGet.Client.Engine.Commands.FinderPackageCommand.<>c__DisplayClass1_0.b__0()
at Microsoft.WinGet.Client.Engine.Commands.Common.ManagementDeploymentCommand.Execute[TResult](Func1 func)    at Microsoft.WinGet.Client.Engine.Commands.FinderPackageCommand.Find(String psPackageFieldMatchOption)    at Microsoft.WinGet.Client.Commands.FindPackageCmdlet.ProcessRecord()    at System.Management.Automation.CommandProcessor.ProcessRecord()    --- End of inner exception stack trace ---    at System.Management.Automation.Runspaces.AsyncResult.EndInvoke()    at System.Management.Automation.PowerShell.EndInvoke(IAsyncResult asyncResult)    at System.Threading.Tasks.TaskFactory1.FromAsyncCoreLogic(IAsyncResult iar, Func2 endFunction, Action1 endAction, Task`1 promise, Boolean requiresSynchronization)
--- End of stack trace from previous location ---
at Microsoft.WinGet.CommandNotFound.WinGetCommandNotFoundFeedbackPredictor.WarmUp()
at System.Threading.Tasks.Task.<>c.b__128_1(Object state)
at System.Threading.QueueUserWorkItemCallback.Execute()
at System.Threading.ThreadPoolWorkQueue.Dispatch()
at System.Threading.PortableThreadPool.WorkerThread.WorkerThreadStart()
[process exited with code<PHONE_NUMBER> (0xe0434352)]
You can now close this terminal with Ctrl+D, or press Enter to restart."
If it's not my mistake I thought you might want to know.
Kind regards, Klaas

Any update to fix this?

Same issue here.

Same Issue here on windows 11

I can NOT use pwsh caused by this issue, after I have installed PowerToys pwsh plugin

Looking to fix this as a part of https://github.com/microsoft/winget-command-not-found/pull/10. After that merges, I'll push out a new release.
In the meantime, if you installed the plugin via PowerToys, you can uninstall it in the PowerToys settings page > Command Not Found

That'll make the CommandNotFound module not load during startup and let you get back to using pwsh 😊

FYI, here's the new release: https://github.com/microsoft/winget-command-not-found/releases/tag/v1.0.3
Enjoy! 😊

As a temporary workaround:
For those that may not have the updated version of Microsoft.WinGet.CommandNotFound, this particular workaround appeared to work for me, and I still have v1.0.2 of "CommandNotFound".
If you have this in your PWSH $PROFILE script...

Import-Module Microsoft.WinGet.CommandNotFound

...Change it to use the Get-Module | Import-Module syntax...
Get-Module Microsoft.WinGet.CommandNotFound | Import-Module

I was then able to do this from the standard CMD shell (not a PS prompt):
C:\Temp> pwsh -c "'{0:yyMMdd.HHmm}' -f [datetime]::UtcNow"
241002.2356

Option # 2: Add the -NoProfile/-nop flag to pwsh...
This prevents loading your $PROFILE script with your CMD-executed command...
C:\Temp> pwsh -nop -c "'{0:yyMMdd.HHmm}' -f [datetime]::UtcNow"
241003.0013

Disclaimers:

I give no guarantees this will work on all systems.
I can only claim that after making the profile change and saving it, my DOS prompt was immediately able to execute it without the "CommandNotFound" errors.  :smile:


Update generator_generation.rb

Fix the syntax so credit_line is recognizable as a variable. #541
I have tested it and worked well. @olleolleolle

Maybe you can author a test-case for this change?

@olleolleolle yes, it is WIP. 😄


feature: Adding an option to filter according to rating

An option to filter according to rating just like filtering by price range can be added .
This can help user to choose a better  dish among the options.

@khushi-joshi-05 @sunny0625 please assign this issue to me under gssoc.


Allow "any kind of letter from any language" instead of [a-zA-Z] in jsonata expressions.

Hi,
First of all I would love to say that I'm big fan of this library and I'm using it's extensively, thank you for all the hard work put into JSONata4Java! :)
Today I was trying to migrate our old jsonata processes from nodejs implementation to yours, and I have found that some of our expressions aren't working because they're using special polish diacritics like [żółćśęą] for example:
json:
    {       "żółw": {         "Tytuł": "Pan Tadeusz"       }     }
with expression:
    { "Title": żółw.Tytuł }
will throw
com.api.jsonata4java.expressions.ParseException with message token recognition error at: 'Ż'
When using try.jsonata this expression is perfectly fine:
https://try.jsonata.org/kNopHVcxh
and will return
{
  "Title": "Pan Tadeusz"
}

I've actually prepared the patch and tests for this case. Link in the next comment :)
I hope that you can review this and include in some release.
Thanks in advance,
Michał Jereczek

Here's my forked solution:
https://github.com/trzye/JSONata4Java/commit/b82b27ac67378d51102e08b835fc3698b9f018a4

Thank you for adding this extension. I have merged the code and will make a new release soon. I'm trying to address an issue #199 before creating a new release.


the instruction to generate ssl.pem is outdated

I am new to ssl, and it took time to figure out this. Please update the doc accordingly. Thanks~~
I bought ssl certificate from namecheap.com, and the files they sent me are：
Attached to this email you should find a .zip file containing:
• Root CA Certificate - AddTrustExternalCARoot.crt
• Intermediate CA Certificate - COMODORSAAddTrustCA.crt
• Intermediate CA Certificate - COMODORSADomainValidationSecureServerCA.crt
• Your PositiveSSL Certificate - www_carllo_us.crt
Not the files mentioned in the given guide:"  Now you need combine SSL certificate(s) with the private key and save it in the mup config directory as ssl.pem. Check this guide to do that.
"
The command to get ssl.pem is:
cat www_example_com.crt COMODORSADomainValidationSecureServerCA.crt COMODORSAAddTrustCA.crt AddTrustExternalCARoot.crt example.key > ssl.pem

And the link to the guide doesn't exists anymore

mupx's SSL support is mentioned here.
https://github.com/arunoda/meteor-up/tree/mupx#ssl-support

i've tried to use SSL with mupx as well and i'm getting errors on mobile.  apparently my intermediate certificates are not found?
i got gd_bundle.crt with my GoDaddy certificate and i got a bundle.crt with a cert i bought from sslcertificates.com
here is my mup.json config:
  "ssl": {
    "certificate": "./ssl/bundle.crt",
    "key": "./ssl/private.key"
  }

i looked in the app dir on the server and these files are being copied into /opt/my-app/config and it works fine on desktop.  is there something wrong with my config?

Is it a problem with all mobiles and browsers? A cordova bundled app.
Maybe you'really using a different host name to identify mobile as in mobile.my domain.top
I also though to try fully qualified URLs but it's just an idea from what you provided. You say it works on desktop etc. so I doubt it would change anything for the better. Anyway from what's in the configuration you'very shown looks okay.
Trace access from various clients or browsers in mobile maybe even curl would reveal something.
Also thinking to clear cookies & certs, run updates on client etc. It could even be a DNS cache that TTL will timeout and work fine tomorrow without clearly that too.
Good luck! As to what more you could provide to help I'll suggest the log of the error you get.


Add DOIs for OPLS atomtypes

Our OPLS force field XML (which we created using atomtype definitions and parameters from GROMACS) features several atomtypes with defined SMARTS strings that lack a DOI reference. This PR adds DOIs to these atomtypes so that the source of the relevant parameters is more transparent.

I'm not finished with this yet, but wanted to go ahead and open the PR so that I can log comments as to issues I run into trying to locate the proper DOI references.

The sulfoxide OPLS parameters do not appear to have a published source. The GROMACS parameters that we have in our XML appear in a recent OPLS paper, where these are compared with parameters derived for the improved OPLS-AAx force field. It seems most papers in the literature that used these parameters link to this paper, where the parameters are listed in Ref. 18, although the charges used here are different than those in our XML and as listed in the 2017 paper. I went ahead and added both DOIs to this atomtype. I do not believe our XML parser is currently designed to take more than a single DOI, so I will have to update that.
Also, it might be useful to add a note attribute to Types within the AtomTypes section (unless we already have the ability to do this), so that messages (like the above) could be added if citations are not straightforward.

Parameters for the α-carbon in alkyl pyrroles do not appear to have a published source. It appears with OPLS that for most α-carbons the epsilon and sigma used are the same as for alkane carbons and the charge is simply adjusted to preserve molecular charge neutrality. As per my above comment, this is something we could probably just add to a note attribute and link the DOI to the alkane carbon parameters.

Parameters for cyclopropanes do not appear to have a published source. However, these parameters are identical to those for alkanes. Bonded parameters are obviously different; however, for now I'm just focusing on nonbonded parameters since we do not yet have the ability to add DOI tags to bonded parameters.

DOIs are still needed for propylene and ethylene carbonate parameters and for alkyne parameters.

And to think there was a time I thought adding SMARTS strings was the hard part!


Reduce grouping in the problems panel

When I look at the problems panel, every grammerly problem is the single item in a group. This clutters the problems panel and makes it hard to read. Since the items are already grouped by file, the file name can probably be dropped from the description.

I would prefer to see items listed in the following format similar to the screenshot below:

Missing comma after introductory phrase: however [87, 507]

Extensions cannot control how diagnostics are displayed.


[Bug] Cannot read property '$$' of undefined (can we please create a kitchen-sink dir?

Describe the bug
I'm having a lot of issues event with this package.json:
"devDependencies": {
    "@storybook/addon-actions": "6.2.9",
    "@storybook/addon-essentials": "6.2.9",
    "@storybook/addon-links": "6.2.9",
    "@storybook/addon-svelte-csf": "1.1.0",
    "@storybook/svelte": "6.2.9",
    "postcss": "8.3.6",
    "svelte-loader": "3.1.2",
    "svelte-preprocess": "4.7.4",
    "tailwindcss": "2.2.7",
    "tailwindcss-cli": "0.1.2"
}

with:
"svelte": "3.42.1",
"svelte-preprocess": "4.7.4",
"typescript": "4.3.5"

Expected behavior
Can you please create a kitchen-sink dir instead of README snippet?
So we can see how to configure everything properly.
Please.
Issues
Cannot read property '$$' of undefined
TypeError: Cannot read property '$$' of undefined
    at init (http://localhost:6006/vendors~main.iframe.bundle.js:1985:50)
    at new Button (http://localhost:6006/main.iframe.bundle.js:846:71)
    at Array.create_default_slot (http://localhost:6006/main.iframe.bundle.js:2000:11)
    at create_slot (http://localhost:6006/vendors~main.iframe.bundle.js:105060:29)
    at create_if_block (http://localhost:6006/vendors~main.iframe.bundle.js:11909:98)
    at create_fragment (http://localhost:6006/vendors~main.iframe.bundle.js:11947:38)
    at init (http://localhost:6006/vendors~main.iframe.bundle.js:106499:37)
    at new Template (http://localhost:6006/vendors~main.iframe.bundle.js:12023:71)
    at create_fragment (http://localhost:6006/main.iframe.bundle.js:2082:13)
    at init (http://localhost:6006/vendors~main.iframe.bundle.js:106499:37)

index.js:54 TypeError: Cannot read property '$$' of undefined
    at init (index.mjs:1779)
    at new Button (Button.svelte:854)
    at Array.create_default_slot (Button.stories.svelte:60)
    at create_slot (index.mjs:61)
    at create_if_block (Template.svelte:29)
    at create_fragment (Template.svelte:67)
    at init (index.mjs:1500)
    at new Template (Template.svelte:143)
    at create_fragment (Button.stories.svelte:142)
    at init (index.mjs:1500)

@shilman can you please help with this?

Closing because of inactivity.


Fix typo in jellyfin config for books and audiobooks directories

Title basically sums it up - fixing a small Jellyfin config typo

Good spot, thanks!


On the web page, don't make the whole yellow area clickable.

If you just miss the servo bar, you can drive the motors unexpectedly.

ok, can make it less large.


Check for overflow when adding new levels in setindex!()

PooledDataArrays have this safety feature. We should probably add it too, since it has a very low performance cost.

It turns out we were already safe, though the error messages were a bit cryptic.


Fix calls to CellectExSelector

No, it doesn't support them. Plus, we never retire subjects in a subject_set, we retire them in a workflow. I don't really know what normal Cellect does with that group parameter.

remove them from an internal data structured key'd to the subject_set id, https://github.com/zooniverse/Cellect/blob/master/lib/cellect/server/grouped_workflow.rb

Right, seems like Cellect just needs to know it because of it's internal structure. With CellectEx, this doesn't actually remove them, it just triggers a full reload. In the future, I'm going to have it maintain a temporary list of items to remove while streaming, that gets emptied upon DB reload.


how to generetae version

hi i want to ask about version, im using flutter for develop the app... i use your script for publish to dropbox and send to email... how to generetate version in app using this code? because on my script always return 1.0.0 ... thank you anway

Hi,
the script takes the version number from a JSON file in the project's release directory.
I don't have a flutter environment yet so I don't know where they put this but you can modify the script:
First, run the script with --release.dir parameter pointing to flutters output directory (or any other directory that contains the version file).
Then in the get_app method, change the file name (line 140) to the file that has the version code.

okay thank you, i have solved it by using git tags and change it using flutter build..


Welcome to the awesome-list for Panel

Hi Davide
I checked out the repository and added it to the awesome-list for Panel.
https://awesome-panel.org/awesome-list?search=sympy

I'm a contributor to Panel. Maintain https://awesome-panel.org/ and post about Panel at https://www.linkedin.com/in/marcskovmadsen/ and https://twitter.com/MarcSkovMadsen.

Hello @MarcSkovMadsen , thank you.
What a wonderful list of panel applications!!! :)


[parse] Recover better from malformed subscript decls for code-completion

Explanation: This fixes a class of code-completion crashes with generic subscript decls where we would try to typecheck with the wrong DeclContext.
Scope: Primarily affects code-completion, but it also touches general parser recovery for invalid subscript decls.
Radar: rdar://35619175
Risk: Low. The code-completion side is very low risk, but there is a possibility that the changes to parser recovery uncover other issues in type checking because we are building AST for incorrect code that we didn't before.
Testing: Regression tests added.

@swift-ci please test


Add API documentation to the core package

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-56/3 is complete.
Docs are uploaded to http://gluon-ts-staging.s3-accelerate.dualstack.amazonaws.com/PR-56/3/index.html

Codecov Report

Merging #56 into master will increase coverage by 0.02%.
The diff coverage is 89.74%.

@@            Coverage Diff             @@
##           master      #56      +/-   ##
==========================================
+ Coverage   77.79%   77.81%   +0.02%
==========================================
  Files         108      108
  Lines        6187     6189       +2
==========================================
+ Hits         4813     4816       +3
+ Misses       1374     1373       -1

Impacted Files
Coverage Δ

gluonts/core/_base.py
100% <ø> (ø)
:arrow_up:

gluonts/core/component.py
85.15% <100%> (+0.11%)
:arrow_up:

gluonts/core/serde.py
86.2% <100%> (ø)
:arrow_up:

gluonts/core/log.py
88.88% <50%> (+1.38%)
:arrow_up:

gluonts/core/exception.py
60.86% <66.66%> (+2.17%)
:arrow_up:


whenAll not fulfilled nor rejected

Hello, I have an issue with whenAll
I have a piece of code like this:
func download(from url: URL) -> Promise<Void> { ... }
...
let promises = downloadURLs.map(download(from:))
Promises.whenAll
  .then { ... }
  .onError { _ in ... }

This code randomly fails. I can find out that each of download promises are fulfilled but neither then nor onError are called.
Then I wrote a test:
    private let concurrentQueue = DispatchQueue(
        label: "then.whenAll.test.concurrent",
        qos: .userInitiated,
        attributes: .concurrent)
    func testWhenAllAllAsynchronous() {
        let values = (1...10).map { $0 }
        let promises: [Promise<Int>] = values.map { value in
            return Promise { fulfill, _ in
                self.concurrentQueue.async {
                    fulfill(value)
                }
            }
        }
        let block = expectation(description: "Block called")
        Promises.whenAll(promises).then { array in
            XCTAssertEqual(Set(array), Set(values))
            block.fulfill()
        }
        waitForExpectations(timeout: 3, handler: nil)
    }

and it fails.
Looks like the temporary variable var ts = [T]() has a thread race issue, and registerFinally is not an atomic operation.
Thank you.

https://github.com/freshOS/then/releases/tag/4.1.0  :)
Thanks again, you rock!


Error when trying to use Tutorial in R

Hello,
I updated R to in order to get the training tab and tutorials.  I was able to update it and proceeded to install learnr.  It appeared to install successfully but when I click on Start Tutorial I receive the below error

Not sure of what file it is looking for.
Thanks,
Rhonda
<EMAIL_ADDRESS>

Try this in RStudio:
Go to Tools > Check for Package Updates and then click Select All and then click Install Updates.
Then try doing the Tutorial again and let me know what happens!
Thanks for dropping a question here, Rhonda! I'll have to be sure I'm getting notified.

I receive the below message when clicking on "Check Package Updates"

There's nothing in the box.  This is a fresh update, I'm using the below

When you go to the Packages tab (bottom right quadrant in RStudio) and search on learnr, what version do you have? And what version of rstudioapi?

learnr <IP_ADDRESS>
rstudioapi 0.10

You might try:
remove.packages("learnr")
and then
install.packages("learnr")
and then do the same thing with rstudioapi

Same error message from above regarding the version.
What is and how do I check this getoption('"repos") to make sure it refers to a CRAN repository.

I don't know, I've never used that. I have asked a friend though if she can help.
In the meantime, maybe try this: https://cran.r-project.org/bin/windows/contrib/4.0/learnr_0.10.1.zip
Download that zip file to somewhere you'll know how to find it.
Go to Tools, Install Packages, in the top droplist, select Package Archive File, then navigate to that zip file and try installing it that way.

Same error

We figured out user had installed R version 3.53 because when you google "install R", a link to that version shows up and it looks real enticing, but users need to click the one that just says "Windows".
Updating R solved the problem.


Failed to resolve import "LOCALE_DATA_PATH"

When adding this module to a Nuxt 3.11 project with no module config, I get the following error message:
[plugin:vite:import-analysis] Failed to resolve import "__LOCALE_DATA_PATH__" from "node_modules/nuxt-monaco-editor/dist/runtime/nls.mjs?v=bbcb84bd". Does the file exist?

<PROJECT_PATH>/node_modules/nuxt-monaco-editor/dist/runtime/nls.mjs:1:25

1  |  import localeData from '__LOCALE_DATA_PATH__'
   |                          ^
2  |
3  |  export function _format (message, args) {

Any clues on what might have gone wrong here?

Closing this again as reinstalling everything worked, so something must have gone wrong during first install that I didn't notice


relative id at first level header

Suppose we have the following
content file
# first
{{parent: f}}
# second
{{parent: f}}
# [.er] error
{{parent: f}}

item file
# some item
{{parent: f.err}}

item file cause the following trouble because of f.err id not found
C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/creq-0.2.11/lib/creq/repository.rb:200:in `block in wrong_parents': undefined method `join' for nil:NilClass (NoMethodError)
        from C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/creq-0.2.11/lib/creq/repository.rb:199:in `each'
        from C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/creq-0.2.11/lib/creq/repository.rb:199:in `inject'
        from C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/creq-0.2.11/lib/creq/repository.rb:199:in `wrong_parents'
        from C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/creq-0.2.11/lib/helper.rb:60:in `block in check_repo'
        from C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/creq-0.2.11/lib/helper.rb:57:in `tap'

The gem is deprecated. Use new gem "clerq"


feat: Preserve restartedAt annotation

Description

Changes proposed in this pull request:

Add istio-operator.kyma-project.io/restartedAt to the list of Pod annotations we preserve while reconciling DaemonSets and Deployments

Related Issues and Documents

Closes:
https://github.com/kyma-project/telemetry-manager/issues/379
Traceability

[x] The PR is linked to a GitHub Issue.
[x] New features have a milestone label set.
[x] New features have defined acceptance criteria in a corresponding GitHub Issue, and all criteria are satisfied with this PR.
[x] The corresponding GitHub Issue has a respective area label.
[ ] The follow-up issues (if any) are linked in the Related Issues section.

Testability
The feature is unit-tested:

[x] Yes.
[ ] No, because unit tests are not needed.
[ ] No, because of ...

The feature is e2e-tested:

[ ] Yes.
[x] No, because e2e-tests are not needed.
[ ] No, because of ...

Tests conducted for the PR:

Codebase

[x] My code follows the Effective Go style guidelines.
[x] The code was planned and designed following the defined architecture and the separation of concerns.
[ ] The code has sufficient comments, particularly for all hard-to-understand areas.
[x] This PR adds value and shows no feature creep.
[x] I have augmented the test suite that proves my fix is effective or that my feature works.
[ ] Adjusted the documentation if the change is user-facing.

/retest

/retest

/retest

/retest


feat(networks): Reddit submission to share panel.

This is an incomplete feature. I have only outlined how to get to the
right url, given the user leaves the options set to the defaults. I
still have a few outlying problems that I would appreciate help with:

Styling the reddit button. I think a dark blue background and an
orange or white snoo would work fine.

Including a reddit icon, which isn't available in the entypo
library, but is available in the font-awesome library.

This feature is available now!


Add Volkan Şengül

Fixes or introduces:
Proposed Changes

@volkansengul - Nice work, thank you! 🙌
Just FYI, you can add env-specific stuff like .idea to a global gitignore for your machine:
https://help.github.com/articles/ignoring-files/#create-a-global-gitignore


AsmJs/WAsm calling convention issues on xplat

When attempting to pass a floating-point argument to an asmjs/wasm function on xplat, the caller and callee don't agree on the calling convention. This causes correctness issues, as we end up often using the wrong value. This occurs in JITted code.
Repro:
Remove these lines from test/runtest.py (link):
not_compile_flags = set(['-simdjs']) \
    if sys.platform != 'win32' else None

Run runtests.py -d AsmJSFloat.
Note: There's a separate bug filed related to removing the lines hiding this issue from runtest.py at https://github.com/Microsoft/ChakraCore/issues/2412.
Repro file:
Here's a separate file by @Cellule that reproduces the issue with -maic:0
function AsmModuleDouble() {
        "use asm";

        function add(x) {
                    x = +x;
                    return +sub(1.5);
                }
        function sub(x) {
                    x = +x;
                    return +(x);
                }
        return {
                    add: add
                };
}

var asmModuleDouble = AsmModuleDouble();     // produces AOT-compiled version
print(asmModuleDouble.add(2, 1))

@Penguinwizzard Indeed, didn't get to complete Simd when I worked on xplat JIT. That's why Simd tests were disabled on xplat by that line.

From what I was able to gather this happens because LowererMDArch::LowerCallArgs calls in LowererMDArch::GetArgSlotOpnd which tries to use xmm registers for float argument when we are supposed to put them on the stack on xplat

@jianchun The problem is almost all asm.js test have that flag even though simd is not used (I am not entirely sure the reason why, but I think there's a good one).

@jianchun If Simd doesn't work on xplat, should we disable building Simd/not recognize the flag on xplat?

@Penguinwizzard @Cellule At the time I felt Simd was close to completion, but I was not exactly sure of the calling conventions. Left Simd code in build so it would not diverge further (Simd code kept introducing build failures at the time).

Is wrong calling convention still an issue?

@MikeHolman AFAIK it hasn't been fixed yet, but maybe @obastemur has more info on this

Then can we increase the priority on this issue? It is essentially blocking our wasm fuzzing and generally is a pretty high impact issue for wasm/asm.js code.

#2716 is merged and should fix this issue.

Problems still occur; remove exclude_xplat on most wasm test cases to repro.

@Penguinwizzard the problem shouldn't be calling conventions. At least it was fixed for asm.js and others. About WASM, we have couple of disable stuff (SIMD) for xplat. Maybe we should close this issue and open a new one : Enable WASM on xplat ?

This issue is also hitting in asm.js not just wasm.
One of the problem is how we make calls in jitted code which is shared by both asm.js and wasm

Closing in favor of #3561


added umd in webpack

npm i flowed

was not working in web projects. this fixes that

Thank you for your collaboration. I'll be reviewing and merging your changes soon.

Hi @saleehk, this was solved in v1.14.5.
I made a different commit instead of merging this one so the code is still compliant with the defined coding standards and the commits with Conventional Commits.
I've also used the Webpack optionoutput.library.type instead of output.libraryTarget, since the last one is deprecated.
Thanks for your contribution!


Add WalletLocator field to the display of SignerLists

The ExpandedSignerList amendment (nearing consensus) adds a new property to a SignerList called WalletLocator. It is an "optional 256-bit (32-byte) field containing arbitrary data".
A signer list is shown on the account and SignerListSet transaction pages.
Account Page
https://livenet.xrpl.org/accounts/radE4Xd2RpQTBAx6YkpWbb7Z3fkcAgsHpK

Transaction Page
https://livenet.xrpl.org/transactions/5988C2790600974A277DE9748BB2D52D3531A8CC6C7C5DB6A1CEF6486FC86A31

+1
I'd want this to exist, I see the value of it considering that the WalletLocator field is essentially a reference to a signer.
@intelliot 😄 Thoughts?

Sounds good. @wojake can you share any info about where/how this will actually be used?

The WalletLocator field will mainly be used by technical users to specify/identify a signer entry. Although there are various benefits to displaying this field, this is a biased request on my part since an Evernode dApp's XRPL account will need to have a signer list in order to be controlled in a decentralized manner and each signer entry may have its WalletLocator field filled, so it helps out to display this field.
Another overlapping factor is XLS38d, they also use the signer list feature and may need to use the WalletLocator field.


Merge Model_CL_Ben and Model into one class, Model, with the difference now living in the respective integrator classes, EulerOpenCL and Euler

This has now been completed, apart from documentation. A result of this is that examples are now much easier to construct and follow at user end, all necessary overhead is done when :class:Model is initiated.


feat: thorchain LP halted pool tag

Description

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

High Risk PRs Require 2 approvals

What protocols, transaction types or contract interactions might be affected by this PR?

Testing

Engineering

Operations

Screenshots (if applicable)

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

develop

PR #6138

PR #6139   👈

This stack of pull requests is managed by Graphite.


[TECH-480] Remove the checkout scm step from the mpyl jenkinsfile to …

…improve performance
branch: feature/TECH-480-improve-mpyl-installation

📕 TECH-480 Improve mpyl installation
Our installation logic in the jenkinsfile’s still doesn’t work entirely as it should. If you pass no version it installs the latest version from test.pypi instead of 1.0.6 (our latest released version at the moment this ticket gets created). We think it might be due to the '--site-packages' tag.
We also think people should always specify which version they want to run. So the easy fix is just to remove the option to leave the version empty, this will ensure the install cmd always has/passes a version (the default is the hardcoded value in the jenkinsfile) and the tag won’t be needed anymore hopefully.
🏗 Build 6 ✅ Successful, started by Jorg Post
🚀 cloudfront-service, job, kong-sync, nodeservice, sbtservice, sparkJob

Seems to work just fine @koeves


Poltergeist selects elements even within template tags

I noticed that querySelectorAll will include matching elements even if those elements are inside a template tag. For example, the following code finds the element:
document.querySelectorAll('element')

from this HTML:
<template><element /></template>

This behavior is different from the firefox driver and is somewhat unexpected. I didn't expect the contents of template tags to be searched. Could this be a bug?
I am also seeing poltergeist submitting inputs even if they are within a template tag.

Poltergeist uses phantomjs, the latest release (2.1.1) of which is based on a version of QtWebkit that is roughly equivalent to a browser from 4-5 years ago and as such most likely does not support the template element.  Therefore it treats it as just a regular custom element and everything inside it is treated normally.  There is nothing poltergeist can do about that.


Removed deprecated qiskit_ibm_runtime.IBMEstimator class

I have removed the deprecated IBMEstimator class as per Issue #358.

It's not a deprecation if you remove the thing < 7 days after the PR initially deprecating and both are included in the same release. That's just a breaking API change. The deprecation policy is there so that users can rely on the code we're shipping and get fair warning before we make changes. Even if you didn't start promoting the library until 0.4.0 that was > 1 month ago and you don't know how many people have started using it already and could be relying on the deprecated class. If you want people to start using the library in earnest and relying on it you need to provide a migration path for changes like this, if you just break things without warning nobody will want to use this as they can't rely on anything in the library (that being said #342 also was done incorrectly and is a breaking API change too).
Doing the deprecation properly is low overhead, you just have to keep the class working as is for another couple of release and at least 3 months. That will give any existing users sufficient time to migrate to whatever the replacement is.

It's not a deprecation if you remove the thing < 7 days after the PR initially deprecating and both are included in the same release. That's just a breaking API change.

Not sure what you are referring to here, we are not deprecating and removing anything in the same release. We are removing something in 0.5.0 three months after it was deprecated in 0.4.0.
But I understand the other pieces of your concern, and will play nice by 3a.

If you want people to start using the library in earnest and relying on it you need to provide a migration path for changes like this, if you just break things without warning nobody will want to use this as they can't rely on anything in the library (that being said #342 also was done incorrectly and is a breaking API change too).

Just to be clear, we are not breaking anything without warning. We have already deprecated in 0.4.0 and have provided warning. We are removing it in 0.5.0 which is planned for three months after. But I understand 3a and we can skip a release and remove this in 0.6.0.


Make numRecordsPerBlock configurable

@isnotinvain
Attempt to make numRecordsPerBlock configurable.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Jiyuan Qian seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Crostic support + Misc features

This has the content we talked about in PR 39, which has gotten a little cumbersome...
I assume you can see the notes on the commits from when I added to my repo. If not, let me know and I can include them here.

This is a test case showing different link positions and including extracts (which propagate correctly from grid [which has both ids], but not between clue and extract [which only update grid]).

        <div class="puzzle-grid" data-mode="crostic-grid"<EMAIL_ADDRESS>data-extracts="2 1 3"
            data-links="1|C 2|A 3|C 4|C 5|C 6|B 7|C 8|B 9|B 10|B 11|A 12|C 13|A">

A. Not new

B. Left, on a map

C. Lights, Camera, ____!

interesting selectors at the bottom of reference/reference-styling.html

I'll add .cell.link. Are there any others? I don't see .extract-code there, so I assume .link-label doesn't get added?

extract-code missing is an oversight on my part. I think the label(s) should all be there.

Thanks a million! I think this PR contains everything that's in your other two PRs, and if you agree, you can close them.

agreed. I'll close them. Thanks for all the help and support for this!


FaceDancer: Implement Initial Design Specification / RFP

Create an initial design document describing a FaceDancer backend for LUNA.
This should:

[x] Include plans for the core backend.
[ ] Accommodate future FPGA acceleration, where possible.
[ ] Accommodate fancy USBProxy enhancements.

Latest version can be found in:
https://github.com/antoinevg/GSG/blob/main/luna/design/rfp.md


New version v1.1.0

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Closed by PR https://github.com/conda-forge/psy-reg-feedstock/pull/3


Implement AbstractTestDistributedQueries for MemSQL connector

Add TestMemSqlDistributedQueries

A stub of the implementation would be
public class TestMemSqlDistributedQueries
        extends AbstractTestDistributedQueries
{
    protected TestingMemSqlServer memSqlServer;

    @Override
    protected QueryRunner createQueryRunner()
            throws Exception
    {
        memSqlServer = new TestingMemSqlServer();
        return createMemSqlQueryRunner(memSqlServer, CUSTOMER, NATION, ORDERS, REGION);
    }

    @AfterClass(alwaysRun = true)
    public final void destroy()
    {
        memSqlServer.close();
    }

...

This shows a number of issues and missing features in the MemSQL connector, which need to be addresed.


Prefetch query on hover from preview card

Once the user hovers with a card on artists or collection,
Let do prefetch of data as there is a high probability it will click there.
This could significantly improve UX and performance while browsing
Ref

#1422
#1474

Hello!
I started working on this issue.

@Phantom20320, are you still working on it?

I guess hover cards are super quick now.


Error while running

Hi, this happens partially through running a match using the 0.2.2 client
[jason@metroid battlecode]$ archon match -lxm src/teamA src/teamB
Running 108 matches on 54 maps between 2 teams
6147.xml [teamA/teamB] teamB 1863
6147.xml [teamB/teamA] teamB 1512
arena.xml [teamA/teamB] teamA 1843
arena.xml [teamB/teamA] teamB 2819
backbencher.xml [teamA/teamB] teamB 1972
backbencher.xml [teamB/teamA] teamB 1961
barred.xml [teamA/teamB] teamB 1742
barred.xml [teamB/teamA] teamB 1433
barricade.xml [teamA/teamB] teamA 1510
barricade.xml [teamB/teamA] teamB 1624
boxy.xml [teamA/teamB] teamA 1811
boxy.xml [teamB/teamA] teamB 2999

/usr/local/lib/node_modules/archon/lib/game.js:39
                round = parseInt(split[2].split(' ')[1]);
                                          ^
TypeError: Cannot call method 'split' of undefined
    at Socket.<anonymous> (/usr/local/lib/node_modules/archon/lib/game.js:39:43)
    at Socket.EventEmitter.emit (events.js:95:17)
    at Socket.<anonymous> (_stream_readable.js:746:14)
    at Socket.EventEmitter.emit (events.js:92:17)
    at emitReadable_ (_stream_readable.js:408:10)
    at emitReadable (_stream_readable.js:404:5)
    at readableAddChunk (_stream_readable.js:165:9)
    at Socket.Readable.push (_stream_readable.js:127:10)
    at Pipe.onread (net.js:526:21)

@JustAPerson archon doesn't like boxy_armageddon if you remove it from the list you should be good


Jenkins ci setup

CI is now failing for the right reasons, everything we need seems to be installed. Keeping the PR open for later rebasing.


Add an interface for subsystems of the engine

Add an interface for subsystems of the engine, so that they are automatically initialized, updated, drawn, and terminated when plugged into the engine.

This is implemented in 38c29d5.


[Redesign] Improved Form Creation

This change is intended to start laying the groundwork for generating forms for the redesign:

Adds JQuery Form Validator to redesign (and tweaks its settings to use proper Bootstrap classes)
Adds HTML form helpers that are accessible and Bootstrap compliant: HasErrorFor, ShowTextBoxFor, ShowValidationMessagesFor
Updates the Forgot Password page to use the new HTML form helpers.

Updated jquery.validate.js and jquery.validate.unobtrusive. Both passed TwC Security Guidance (I had a manual assessment done for jquery.validate.unobtrusive).


Add FICO Xpress to set of solvers

Adds bindings to FICO Xpress to solve linear, quadratic, and second-order cone problems.

@SteveDiamond should be a clean rebase of upstream/master. The "\n\n" spurious output that causes the only error I get with pytest is due to a bug in the xpress module that will be removed with the next patch releases.

Thanks @merraksh ! The tests are failing still because of flake8, which should be an easy fix. Otherwise @rileyjmurray can help you bring the interface in alignment with our interface guidelines.

@merraksh the flake8 message Steven mentioned is
./cvxpy/reductions/solvers/qp_solvers/xpress_qpif.py:30:9: E722 do not use bare 'except'
./cvxpy/reductions/solvers/qp_solvers/xpress_qpif.py:252:13: E722 do not use bare 'except'

Oops, sorry. My flake8 (3.4.1) doesn't detect those and I was convinced it was to avoid the "pass".

Great! Merging in.


Switch to AppCompatButton compound drawables in auth provider buttons

Fixes #1860

@AryanNajafi thank you!  Can you add line to CHANGELOG.md like this:
- [Auth] Remove SupportVectorDrawablesButton (#1860) (contributed by @AryanNajafi)


Video won't play after video quality is changed [202]

Hi there, I'm so far getting my feature in order with your video player, but I'm stuck on changing the video quality.
Problem
When I change the quality of the (running and working!) video, my source for video (as I inspected HTML) updates to an ugly URL which is combined of my localhost URL + my component (as I'm using Vue in this whole process) + function%20String()%20%7B%20[native%20code]%20%7D this chunk of code.
Expected behavior
I'm expecting as I change the video quality, let's say from 720p to 360p, to load my video again, adding a source of my 360p video, to my video element which I specified.
Question
Is there a hook exposed which triggers as I change the video quality so I can track what actually happens as soon as I change the video quality? Or better yet: is there a clear solution for this so I don't go running through hooks and "manually" changing the video source?

Hi,
Is it possible to share the link where u are experiencing this

Can not reproduce with local vue setup. Please reopen if you can produce reproduce code.


Remove 0 /  no-data value for a Google Drive export for areas outside of of the export geometry's bounding box

Is your feature request related to a problem? Please describe.
When exporting the image, I use the bounding box of the geometry.
The no-data values within the bounding box are correct 9999) but I cannot figure out how to set the no-data value of the pixels outside of the bounding box; these seem to default to 0.

Describe the solution you'd like

[ ]  specifying the export projection when calling the bound: see solution proposed in https://groups.google.com/g/google-earth-engine-developers/c/aL0NfsxzMf0
[ ]  no data value has to dependend on the product / band type  ( e.g. VHI in 8 bit with a range from 0-100, nodata set to be eg 255, and different for S2_SR 10m bands with 16 int 8bit bands etc

it is not fixed yet

needs to be fixed in the satromo_publisher step at https://github.com/swisstopo/topo-satromo/blob/30e4e884f27bb841fcaff1a7c0c29c5ffbeb7360/satromo_publish.py#L671, since in the GEE assets it is already masked correctly
The approach is: to get from metafile the asset ID the get the masked /non masked area
``
asset_id = 'projects/satromo-int/assets/COL_S2_SR_HARMONIZED_SWISS/S2-L2A_mosaic_2024-07-05T104021_bands-10m'
Load the asset.
asset = ee.Image(asset_id)
Select the first band of the image and convert it to an integral image.
first_band = asset.select(0).toInt()
Create a binary mask.
binary_mask = first_band.mask()
Invert the mask to get the non-masked areas.
non_masked = binary_mask.Not()
Convert the non-masked areas to vectors.
vectors = non_masked.reduceToVectors(geometryType='polygon', maxPixels=1e9, bestEffort=True)
Convert the vectors to GeoJSON.
geojson = vectors.getInfo()
Extract the features from GeoJSON
features = geojson['features']
Extract geometries and properties
geometries = [shape(feature['geometry']) for feature in features]
properties = [feature['properties'] for feature in features]
Create a GeoDataFrame
gdf = gpd.GeoDataFrame(properties, geometry=geometries)
Define output shapefile path
output_shapefile = 'output.shp'
Define the CRS
crs = {'init': 'epsg:2056'}
Export to Shapefile with specified CRS
gdf.to_file(output_shapefile, driver='ESRI Shapefile', crs=crs)
print(f"Shapefile exported successfully to {output_shapefile}")
``

then cast this as config.BUFFER. / use it a config.BUFFER
Make sur that you check if output.shp exists so it does not need to be claulcated everytime

We solve thsi differently: We precompute for  each orbit a buffer https://code.earthengine.google.com/279232b086bcfb5d456fe26eda6b09c6
the apply this buffer when creating the mosaic in satromo_publisher: Advantage: we save EECU hours, and we alawys have the orginal data masked in GEE assets. Disadvantage: with orbit shifts we will miss some  tiles

fixed on dev


Interactive transactions are not logging DB queries

Bug description
There is a report of interactive transactions not logging anything past BEGIN.
Prisma Version
3.10

Thanks, @garrensmith,


WIP fix command upload default value and help message

Current master tests are broken.

@mouradmourafiq I want to merge it. But latest master tests cannot pass.

@gzcf on your local machine 2 tests will eventually fail because the config is not mocked. On travis the test is passing. so it should be fine, I will try to update the tests for local setup.


Accounting for stock splits?

Hello @PatrickAlphaC ,
Thanks for the library, its great so far. I was wondering how do I work around stock splits, is this accounted for in your wrapper? If not, what recommendation would you have if I wanted to pull and view historical data. For example:
   data, _ = time.get_daily_adjusted(symbol=ticker_string, outputsize="full")

    plt.figure(figsize=(18, 8))
    plt.title(f"{ticker_string} Open Valuation Over Time")
    plt.xlabel("Date")
    plt.ylabel("Value (Dollars)")
    plt.plot(data.index.values, data[["1. open"]])
    plt.grid()
    plt.show()

Gives me:

A quick google search shows that AAPL did a 7/1 split in 2014. Am I using the wrong function methods or is this by design from alpha vantage. How do I work around this? Thanks!

@alejandroarmas, you were getting really close 👍
To plot adjusted time series, please change the following line:
plt.plot(data.index.values, data[["1. open"]])
into
plt.plot(data.index.values, data[["5. adjusted close"]])
You may also find the following documentations useful:

https://www.alphavantage.co/documentation/#dailyadj
https://github.com/RomelTorres/alpha_vantage/issues/115

Oh I see how silly of me, thank you Steve! I had used the data[["1. open"]] from the get_daily' method. Did not realize there was an extra column for get_daily_adjusted'.
I'm wondering now, is there any reason to want to use get_daily' over get_daily_adjusted' if that's the case? Would it create more overhead in the API call if you use the latter?
Another question, say I wanted to use to visualize real time updates of the `get_intraday' call.  Thanks a lot!

@alejandroarmas, since the questions are more about API use cases rather than this Python wrapper, could you please direct your questions to<EMAIL_ADDRESS>

Sure thing, thanks for directing me. Thank you.


Webpack Failed to Compile after Typescript 2.3.1 upgrade

Getting the following error message on ng start or ng test after upgrading Typescript to 2.3.1
webpack: Failed to compile.

[karma]: Error: AppModule is not an NgModule
    at Compiler.compiler.plugin (.../node_modules/@angular/cli/plugins/karma-webpack-throw-error.js:10:23)

Versions.
→ ng --version
    _                      _                 ____ _     ___
   / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
  / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
 / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
/_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
               |___/
@angular/cli: 1.0.1
node: 6.9.5
os: darwin x64
@angular/common: 4.1.0
@angular/compiler: 4.1.0
@angular/core: 4.1.0
@angular/forms: 4.1.0
@angular/http: 4.1.0
@angular/platform-browser: 4.1.0
@angular/platform-browser-dynamic: 4.1.0
@angular/router: 4.1.0
@angular/cli: 1.0.1
@angular/compiler-cli: 4.1.0

Repro steps.
Upgrade Typescript to 2.3.1

TS 2.3 is not yet support.  Version 2.2.x is the latest currently supported version.

But Angular 4.1 clearly states that it supports 2.3 ! It is a quite confusing message that angular-cli is giving if it is not aligned with angular. Very annoying.

Downgrading TypeScript from 2.2.2 to 2.3.1 did the trick here. Using Yarn.
Interestingly, the whole problem only surfaced on Travis (running Linux), not locally on OS X. Nuking node_modules and the Yarn cache had no effect on the issue in either environment.

This is very weird. Will investigate.

On a brand new project, with everything Angular 4.1.1, CLI at 1.0.1 and<EMAIL_ADDRESS>I could not reproduce. I tried with npm and yarn.
Does someone have a repro I can follow to see this error?

Heya, can you try<EMAIL_ADDRESS>It should have TS 2.3.x support.

@filipesilva , I just tried with<EMAIL_ADDRESS>and still getting the same error reported initially.
 But let me create a new project to test on my end.

Turns out @clydin has already fixed this in https://github.com/angular/angular-cli/pull/6115, but it will only be out on the next 1.1.x beta.

@filipesilva I tried testing on a new project created with the latest 1.1.0-beta.0 cli but got the following error after trying start server:
→ yarn start
path.js:1142
          cwd = process.cwd();
                        ^

Error: ENOENT: no such file or directory, uv_cwd
    at Error (native)
    at Object.resolve (path.js:1142:25)
    at Function.Module._resolveLookupPaths (module.js:390:17)
    at Function.Module._resolveFilename (module.js:460:31)
    at Function.Module._load (module.js:417:25)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/usr/local/lib/node_modules/yarn/bin/yarn.js:29:17)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)we

@filipesilva Could you please in the future align typescript version support of angular and angular-cli. The current situation with newest angular and newest angular-cli supporting two different typescript versions is confusing.

@teropa, @omargon, Also I tried to reproduce the issue. I'm using yarn.  On windows, Debian and ubuntu it works for me without a glitch. Are you sure you have the latest typescript installed globally?
@mmc41 As it works in most setups, and it only fails for a few, how are we supposed to know? If it breaks in our own testing, we will not release. We don't have a crystal ball, that files issues for bug that are in the future ;)

In a mac and I was having the same error with<EMAIL_ADDRESS><EMAIL_ADDRESS>and<EMAIL_ADDRESS>Basically, all the latest versions..
What solved it was updating @angular/cli to 1.1.0-beta.1 in both local and global. Hope this helps.

@jancarloviray that worked for me - thanks


RFC: Visual Regression Testing

A pretty powerful way of testing with confidence is visual regression testing. What are thoughts on this? uber/image-diff can be used to diff screenshots of images.

I think you are right that visual regression testing can be a really powerful way to guarantee repeatable graphics. That said, I think we should work on making the rest of the code better tested before we invest in something heavy like screenshot diffing?

Worth noting: image-diff seems like it is either deprecated or about to be deprecated.

I've really enjoyed using phantom-css in the past, it works well and is pretty easy to reason about. I'm open to other suggestions if there are more modern alternatives?

phantom-css seems to be pretty well maintained so i think its a good alternative

We're welcoming PR on this but it's not really blocking so will close for now.

Just FYI - The upcoming deck.gl 5.2 release includes a visual regression testing framework for WebGL layers, using puppeteer and chrome dev tools. Pretty sure many parts could be reused if there was interest.

The basic BrowserDrivers parts are available in probe.gl.
Current plan is to expose image diffing tools as part of a collection of GPGPU tools in luma.gl, but we could discuss other options.
The test scene renderer is harder to share as it would need to be adapted for react-vis, but deck.gl's version could be used as a base.

I know @btford was exploring a similar testing strategy for another
application a couple months back, we should sync our efforts.
On Fri, Mar 9, 2018 at 4:31 PM Ib Green<EMAIL_ADDRESS>wrote:

The basic BrowserDrivers parts are available in probe.gl
https://uber-web.github.io/probe.gl/#/documentation/api-reference-testing/browserdriver
.
Current plan is to expose image diffing tools as part of a collection of
GPGPU tools in luma.gl, but we could discuss other options.
The test scene renderer is harder to share as it would need to be adapted
for react-vis, but deck.gl's version could be used as a base.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/uber/react-vis/issues/224#issuecomment-371983840, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABEzW3Zbgn_bkPaZjfE_VbxxwIjnX4OQks5tcx7ggaJpZM4LgNxJ
.

Sounds good. Just in case, I added the image diffing tools as experimental exports in probe.gl.
I think a skeleton of the code that asynchronously drives multiple react renders and wait for comparisons could also be made reusable. Doing that could save a lot of time for others, but preparing it would take a little more time than I have right now, so let's sync first.


[GOBBLIN-373] Expose task executor auto scale metrics to standard sensor

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-373] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-373

Description

[x] Here are some details about my PR, including screenshots (if applicable):
Expose auto scale metrics to external sensor system.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
Cluster integration unit test.

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

@yukuai518 This has conflicts. Can you please resolve it.

This was closed because the PR is already pulled in by another ticket


Fix scripts

No, why? We want the code compiled prior to publishing to npm, no need to have every user do it locally.

@es128 Ok, but in .gitignore I found lib/. So, after npm i -g brunch there isn't lib/ in /usr/local/lib/node_modules/brunch.

It isn't in npmignore though. Why is gitignore relevant?

@es128  Yes, of course, but there is problems when we want to use edge version from github. ex. sudo npm install -g https://github.com/brunch/brunch/tarball/master.

Ah that's right. I had forgotten why we had it this way. Still, perhaps the postinstall could be set to run preinstall if lib is absent, otherwise noop.

we had the good setup.js script — perphaps we need to bring it back.


the screen turns white

the screen turns white , and it has to be closed with the area manager, about every 30 minutes, I'm running it in an updated vps with windows 10.

#374