support cuda 10.1

Doesn't quite work yet, I'm getting compilations errors:
/home/rou/src/xgboost/src/common/host_device_vector.cu: In instantiation of ‘void xgboost::HostDeviceVectorImpl<T>::DeviceShard::Init(xgboost::HostDeviceVectorImpl<T>*, int) [with T = long unsigned int]’:
/home/rou/src/xgboost/src/common/host_device_vector.cu:205:1:   required from ‘xgboost::HostDeviceVectorImpl<T>::InitShards()::<lambda(int, xgboost::HostDeviceVectorImpl<T>::DeviceShard&)> [with T = long unsigned int]’
/home/rou/src/xgboost/src/common/host_device_vector.cu:205:13:   required from ‘struct xgboost::HostDeviceVectorImpl<T>::InitShards() [with T = long unsigned int]::<lambda(int, struct xgboost::HostDeviceVectorImpl<long unsigned int>::DeviceShard&)>’
/home/rou/src/xgboost/src/common/host_device_vector.cu:204:23:   required from ‘void xgboost::HostDeviceVectorImpl<T>::InitShards() [with T = long unsigned int]’
/home/rou/src/xgboost/src/common/host_device_vector.cu:171:11:   required from ‘xgboost::HostDeviceVectorImpl<T>::HostDeviceVectorImpl(size_t, T, xgboost::GPUDistribution) [with T = long unsigned int; size_t = long unsigned int]’
/home/rou/src/xgboost/src/common/host_device_vector.cu:430:12:   required from ‘xgboost::HostDeviceVector<T>::HostDeviceVector(size_t, T, xgboost::GPUDistribution) [with T = long unsigned int; size_t = long unsigned int]’
/home/rou/src/xgboost/include/xgboost/data.h:186:14:   required from here
/home/rou/src/xgboost/src/common/host_device_vector.cu:56:16: error: cannot call member function ‘size_t xgboost::HostDeviceVectorImpl<T>::Size() const [with T = long unsigned int; size_t = long unsigned int]’ without object
       LazyResize(vec_->Size());
            ~~~~^~

The error is similar to https://github.com/protocolbuffers/protobuf/issues/5805 and https://github.com/tensorflow/tensorflow/issues/26155.

Can you please add a new entry to the Jenkinsfile build matrix for cuda 10.1.
I think we can also remove the second entry from the build matrix, we should have sufficient coverage without it and it will speed up CI:
    [ "enabled": true,  "os" : "linux", "withGpu": true, "withNccl": true,  "withOmp": true, "pythonVersion": "2.7", "cudaVersion": "9.2", "multiGpu": true],
    [ "enabled": true,  "os" : "linux", "withGpu": true, "withNccl": true,  "withOmp": true, "pythonVersion": "2.7", "cudaVersion": "9.2" ],
    [ "enabled": true,  "os" : "linux", "withGpu": true, "withNccl": true,  "withOmp": true, "pythonVersion": "2.7", "cudaVersion": "8.0" ],
    [ "enabled": true,  "os" : "linux", "withGpu": true, "withNccl": false, "withOmp": true, "pythonVersion": "2.7", "cudaVersion": "8.0" ],
]```

Thought this was a compiler bug, I built gcc 8.3 from source, but still getting the same error.

Wow, it is a compiler bug. Got everything built under gcc 6.

@RAMitchell added 10.1 to the build matrix.
This should work now as long as we stick with gcc 6.

@rongou That's problematic.  We might need to workaround that.  Is it possible to file an issue at GCC with host code only?

Fixes #4220

Wait, is 10.1 container available already?

@hcho3 yeah just got pushed to docker hub.
@trivialfis agreed. Maybe we should be less ambitious and stick with cuda 10.0 for now. Let me try to figure out if I can reproduce the bug easily.

Hmm, maybe it's actually a cuda sdk bug. In the generated host_device_vector.cudafe1.cpp, that line is changed to LazyResize(Size());, which is obviously incorrect.

@rongou Any way I worked around it by de-referencing the pointer first:
(*vec_).size();

@trivialfis cool, that's a good workaround for now. I made the change and added a TODO.

Codecov Report

Merging #4223 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4223      +/-   ##
==========================================
- Coverage   67.28%   67.27%   -0.01%
==========================================
  Files         132      132
  Lines       12219    12220       +1
==========================================
  Hits         8221     8221
- Misses       3998     3999       +1

Impacted Files
Coverage Δ

src/tree/updater_colmaker.cc
39.6% <0%> (-0.08%)
:arrow_down:

Continue to review full report at Codecov.

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


QBD - Tags toggle is not enabled

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.58-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5184319
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:
Precondition: User is connected to QBD

Navigate to a Workspace connected to QBD
Navigate to "More features" in workspace settings

Expected Result:
Tags toggle is Enabled and locked. Shown in Workspace settings
Actual Result:
Tags toggle is disabled and locked. Not Shown in Workspace settings
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/9fa67f00-febf-42a4-894b-75e791534fa5
View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
The Tags feature is disabled and not active  on the WorkspaceMoreFeaturesPage when a user has QuickBooks Online (QBO) connected.
What is the root cause of that problem?
I think this line is causing the issue:
https://github.com/Expensify/App/blob/580bf69714fcb90fbf96026b3e2fc12f4b556eeb/src/pages/workspace/WorkspaceMoreFeaturesPage.tsx#L213
What changes do you think we should make in order to solve the problem?
I think we can change it to :
isActive: (policy?.areTagsEnabled || hasAccountingConnection) ?? false,

Confirmed this seems buggy

@stephanieelliott any reason this is a QBD specific bug? The reason I am asking is if Tags toggle should be enabled for the other connections?

@mananjadhav are you guys still considering proposals for the issue?

Yes open for proposals while I still have my outstanding question.

@stephanieelliott any reason this is a QBD specific bug? The reason I am asking is if Tags toggle should be enabled for the other connections?

No, I bet this got labeled QBD because it was one of the QBD regression tests. But in general - if Tags toggle is enabled, it should show in Workspace settings. If disabled, it should not. That goes for all integrations.

Still waiting for better proposals. I don't think adding accounting integration check is the correct solution.

@mananjadhav, what are your thoughts on my updated proposal?

Assigning an engineer here. I think this needs to be fixed on the backend.
🎀 👀 🎀 C+ reviewed.

@stephanieelliott to clarify my understanding - for a workspace that is connected to any accounting integration (whether that's QBD or QBO or any other), the tags feature will be disabled and locked in More features. And so, in the Accounting settings, the tags feature should also be disabled and locked. Is my understanding correct?
@mananjadhav could you outline why you think this requires backend changes?

Waiting on clarification on the above and then we can decide how to proceed here

for a workspace that is connected to any accounting integration (whether that's QBD or QBO or any other), the tags feature will be disabled and locked in More features.

AFAIK it would depend on the accounting integration. For instance if Xero the tags are enabled then the More Features will show them enabled and locked.

could you outline why you think this requires backend changes?

The areTagsEnabled flag is supposed to be synced from Policy connection is populated from the backend. It could be backend or IS sync issue. May be I jumped too early but I feel this is something to be fixed from backend.

@MariaHCD did you get a chance to look at my comment?

I’m still unclear on the expected behavior in this scenario and would appreciate some clarification.
For the basic case, my understanding is:

If the Tags toggle is enabled, it should appear in the Workspace settings. If disabled, it should not. This behavior applies to all integrations.

However, I’m uncertain about how this works when a policy is connected to an accounting integration. Specifically, does this mean that tags are automatically enabled? If so, should they be displayed as locked and enabled in the Workspace Settings page? Is there any integration-specific logic here? (I'm assuming not but wanted to check anyway)
cc: @stephanieelliott and @aldo-expensify (I understand you’ve worked on some of the backend logic related to enabling tags)

cc: @stephanieelliott and @aldo-expensify (I understand you’ve worked on some of the backend logic related to enabling tags)

From testing manually with QBO (which I think should be the same), I see that Tags is enabled by default after connecting:

In QBO the defaults settings are set in the IS server here:
https://github.com/Expensify/Integration-Server/blob/9fbb26613c260101be30ef4c045c399dbb73cb25/src/expensify/quickbooksonline/QuickbooksOnlineImporter.java#L212-L213
I'm guessing that for QBD there should be something similar (should verify).
Then, in PHP, when we are storing the connection data for the first time we enable tags here:
https://github.com/Expensify/Web-Expensify/blob/b096b163a1313f07454405c8f508512befcea951/lib/PolicyAPI.php#L3890-L3891
This piece in PHP is only doing it for XERO and QBO, should it do it for all connections? or should we just add QBD there?

Sorry for the delay guys, I was out for Thanksgiving. I think we are snowballing on this issue though -- whether Tags are auto-enabled when you first connect is not really an issue, I think we can just assume that behavior is working as intended.
The Locked behavior is also not a factor here -- Tags will always show the lock icon whether enabled or disabled. This is just because you can't modify the toggle from the More Features tab, Tags must be enabled/disabled from the Accounting tab.
The actual bug is that even when Tags are enabled, Tags doesn’t appear in the LHN and the More Features > Tags setting shows as Disabled. This is a problem because it prevents users from viewing or modifying the Tags that were imported through the integration.
These images may help explain it a little better, but let me know if I can clarify this any further!
Current behavior:
Import as Tags is selected under Accounting > Import, but More Features > Tags shows as Disabled and Tags does not show in the LHN.

Expected behavior:
if Import as Tags is selected under Accounting > Import, then More Features > Tags should show as Enabled and Tags should show in the LHN.

Thanks @stephanieelliott. This comment makes sense. Will you be able to update the GH issue body? I think the important step is that the Workspace > Import > Any of the options have imported as tags, yet the More Features > Tag shows disabled with lock. This should've been shown as enabled with lock.
I'll investigate this but also open for proposals now.

I think the important step is that the Workspace > Import > Any of the options have imported as tags, yet the More Features > Tag shows disabled with lock. This should've been shown as enabled with lock.

Yep, exactly @mananjadhav! I will update the issue body to make this more clear

Just spot checked and it looks like this is still occurring. It is isolated to QBD though, not seeing it with other integrations.

Yes because I checked Netsuite and Xero. It worked fine. I don't have QBD setup and I am setting it up.
@stephanieelliott Would it be possible for you to share the policy JSON from your local browser? To verify the data + conditions we have.

Still waiting for proposals.

Still waiting for proposals.

Not overdue - still awaiting proposals

Oooh I think this is fixed! Spot checked a handful customer accounts who just connected today, and all are showingTags in the menu as intended:


[bug?] cannot resolve global components from "@vue/runtime-core".

I need add the code to my .dts file to get my global components' hint:
import { GlobalComponents as GCS } from "@vue/runtime-core";

declare module "vue" {
  interface GlobalComponents extends GCS {}
}

before:

after:

my extensions version: v0.34.10
my vue version: ^3.2.31
my @vue/runtime-core version: ^3.2.31

Hi @JuerGenie, can you reproduce this problem in https://github.com/johnsoncodehk/volar-starter?

Hi @JuerGenie, can you reproduce this problem in https://github.com/johnsoncodehk/volar-starter?

ok, I'm trying to reproduce the problem now, but cannot reproduce it... so weird😮💨. I'll try it again, if I reproduced the problem, I'll update this issue and let you know, thanks for your reply 🙂.

It seems a TS + pnpm issue, see https://github.com/antfu/unplugin-vue-components/issues/355.

It seems a TS + pnpm issue, see antfu/unplugin-vue-components#355.

yes its! thank you very much!


Add support to deactivate Subscription via CLI

Is your feature request related to a problem? Please describe.
Since it is possible to create a subscription via the Azure CLI:
https://docs.microsoft.com/en-us/cli/azure/ext/subscription/account?view=azure-cli-latest
I would like to see the same to deactivate a subscription.
Describe the solution you'd like
Similar to az account create:
az account deactivate --name
[--force]

Thank you for the feedback. We will consider this as a feature request.


How to set playlist for background media in JavaScript?

I'm trying to rewrite the demo code in JavaScript and setting the source for 1 item in the backgroundmediaplayer works, but the following code
// Assign the list to the player
BackgroundMediaPlayer.Current.Source = playbackList;
is undoable in JavaScript so it seems.
Setting
mediaPlayer.source = mediaPlaybackList
with mediaPlaybackList being a Windows.Media.Playback.MediaPlaybackList always results in a 'No such interface supported' error. the old syntax using setMediaSource gives the same error

Hi, you need to use URL.createObjectURL(). Please see the VideoPlayback sample Scenario 5 here:
https://github.com/Microsoft/Windows-universal-samples/blob/master/Samples/VideoPlayback/js/js/scenario5.js

mediaPlayer.src = URL.createObjectURL(mediaPlaybackList, { oneTimeOnly: true });
with mediaPlayer being a Windows.Media.Playback.BackgroundMediaPlayer.current.
src is not a known property; after setting the src and calling .play() the player does nothing.
mediaPlaybackList.moveNext() results in an error: 'the request is invalid in the current state'
as if the mediaplayer is not running / the playlist is inactive; please advice

it looks like you aren't asking a question about the samples. You can open a support case with our Developer Support team who can further help with the specific problem you are experiencing. Please use these steps below for further help:

Visit http://aka.ms/storesupport and sign-in with your Microsoft account
Under the “App Development” section choose the appropriate type – such as Building apps and games, and then choose from the Problem type/Category to get engaged through a formal support channel

lucienimmink--
I forgot you were trying to get this working in the background whereas the video sample shows how to do it in the foreground using an HTML5 video element (which has a src property).
You do need to use BackgroundMediaPlayer.Current.Source = playlist but you can only set this from your background process, not your foreground process.
Have you rewritten the entire background process in JS? It is possible to do that as far as I know, but we do not have a sample for this since it is not commonly done.

Think I found out how to handle this: All Background related functions have to be in the item declared as start page for the background audio in the manifest.
Thanks for all the hints Aaron. Without you I had given up already since the documentation is quite vague on this subject and no demos exist.

I am glad to hear with that you were able to get it working! I appreciate the feedback and we'll revisit whether to extend our doc and sample coverage for this case.


wrong handling of cpu binding on more than 1 node

fixed with https://github.com/vecma-project/QCG-PilotJob/commit/45d232f5eecce6bf387648e5e3fef22d59f85319


dep: bump containerd version to v1.6.6

To fix https://github.com/advisories/GHSA-5ffw-gxpp-mxpf
Signed-off-by: Yan Song<EMAIL_ADDRESS>

@imeoer , a new test job has been submitted. Please wait in patience.

@imeoer , The CI test is completed, please check result:

Test CaseTest Result

merge-target-branch:white_check_mark:SUCCESS

build-docker-image:white_check_mark:SUCCESS

compile-nydus:white_check_mark:SUCCESS

compile-ctr-remote:white_check_mark:SUCCESS

compile-nydus-snapshotter:white_check_mark:SUCCESS

start-nydus-snapshotter-config-containerd:white_check_mark:SUCCESS

run-container-with-nydus-image:white_check_mark:SUCCESS

Congratulations, your test job passed!


Update Composer dependencies (2024-12-03-00-13)

Visual regression test passed!
View the visual regression test report


Polyline outline needs its own interactive parameter

Currently outline will be marked interactive if its parent line style is marked interactive. But its perfectly legal to set interactive: false for an outline explicitly, which will not work in current implementation.

do i have option to set outline/stroke to Polygon? In Android sdk if i add a Marker like map.addMarker() and setStyle as setPolygon. Can i set its outline/stroke, stroke color and width?

Hey Nouman
Currently a marker styling has a limitation that a marker can only be drawn with a single style. So it is not possible to style a "polygon" geometry marker with both polygon and line styles.
However a workaround this issue you can draw 2 markers for the same geometry, one with polygons style and another with lines style to get the outline/stroke of that marker.
Also request you to open a different issue if you still have problem with polygon marker strokes, since its unrelated to the original issue this is addressing.


[EN] Convert Yerevan budget documents from PDF to CSV/XML/JSON machine readable data

Goal
The goal is to create a dataset with the Yerevan city budget for further analysis and visualization. It could be done now since the budget is being published as a set of PDF documents.
Tasks
The Yerevan city budget for the 2023 year and report about the budget execution of the 2022 budget will be published on the city website https://www.yerevan.am/hy/finance/ as a set of archives with PDF documents insight.
These documents have a text layer that can be processed to extract tables.
These tables look like this

Page 49 from the budget report of the 2022 year https://www.yerevan.am/uploads/media/default/0002/19/b257858f7a9940c75efc4a98acb88e949dd6e554.pdf

These tables should be extracted and converted as Excel, CSV, or JSON files—one file per table.
It would be great if table headers were in English and headers were translated from Armenian to English. For example, Եկամտատեսակները in the Excel or CSV file should be written as "income".
It would be even better if you could convert past budgets too, city budgets 2018-2022 available as sub-pages at the same link

Context
The Yerevan budget was published as a set of Armenian-only text/pdf documents without any machine-readable or at least Excel file.
To convert PDF files to Excel or CSV/JSON, you could use ABBYY Finereader, Tabula, or any other tool that could help.
Requirements

create a public GitHub repository to store code and data under one of the free and open licenses like Creative Commons license or MIT license

Wishes
Please write your code as reusable code that could be launched by someone else later since we could need to update this dataset later.
Resources

Yerevan city budget page https://www.yerevan.am/hy/finance/

Prepared by
The Open Data Armenia team prepared this task

I can help with converting pdf files to csv/json/excel but after 1.5 weeks. I need to complete several study projects before exams

@ivbeg Hi! I processed one pdf document. Everything okay? If not, please tell me where there is a mistake. Tables are available on google drive and on my repository

@dkagramanyan Hi! Yes, it looks great!
P.S. repository looks private, so I've checked only google drive documents

@dkagramanyan thanks a lot! If you could possibly share the code that did the trick it would be just perfect (the repo you referred to above is private, as @ivbeg pointed out).
Meanwhile, best of luck with your exams.

Hi! I have successfully completed parsing of Yerevan budget 2023. The data is available in my  repository and on gdrive. Links are in my previous comments

@dkagramanyan Thanks a lot, David, this was really useful. The data look fine to me. Ivan has been away on business, so he could not respond promptly, but he will as soon as possible.

@dkagramanyan Hi David! Sorry, I was on business trip for some time and was unable to answer. Yes, it looks great! Thanks a lot.


Avatar getting

So someone has one of my old avatars so im trying to rip it back

We don't support ripping of any kind this is the wrong place to ask for that. reverse engineering of any kind is also completely against VRChat's terms of service.


Fix the logic when validating the creation of webhooks

The logic should be inverted. We need to search for the webhook names against the list of webhooks. If one is not found, then return false.
Currently this test fails if the cluster already has another webhook previously deployed.

@jwendell: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
152fcf9ffb2c79aa07ca3a72880166937003ff49
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jwendell: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio
7c9ac515d25cd155aac5e3502840d13c34d05680
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Fix opensearch throttling and add modal to show api reqs

Issue #, if available:
Closes #302
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

skipping code review since this code has been reviewed before as part of the content analysis solution


[Bug Report] el-table expand-row-keys not in sync

Element UI version
2.12.0
OS/Browsers version
Chrome 76.0.3809.100
Vue version
2.6.10
Reproduction Link
https://jsfiddle.net/afvdghy1/1/
Steps to reproduce
In the table, click one of the links. On click, it will add the key of the current row to the expand-row-keys  array. The row expands as expected.
Click again, and the key will be removed from the array. But - the row is still expanded.
Furthermore, if you use the arrow icons to expand/contract a row, it has no effect on the expand-row-keys array.
What is Expected?
I would expect the array given through expand-row-keys to be the single source of truth for expanded items.
If a key is in the array, the row should be expanded. If a key is not in the array, the row should not be expanded (therefore contracted)
What is actually happening?
By that, if you expand/contract an item via the arrow keys, I would expect the array to change values accordingly.
But using the arrow icons has no effect on the expand-row-keys array.
By adding a key to the array, I would expect the row to expand (that happens correctly).
By removing an item from the array, I would expect the row to contract (does not happen).

What I want to achieve is to have my own link in one of the columns that can expand/contract the rows.

Same problem here, any solution?

+1


crypto/arm_arch.h: add a variable declaration

Add this variable declaration to prevent
"-Werror,-Wmissing-variable-declarations" error from compiler.
This error currently only happens on clang.

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

This pull request is ready to merge

Merged to master, thanks for the fix.


log SSM command ID in e2e test run outcome

Description of changes:
bubble up the SSM command id from the execution of each test run, and include it in the final log message for completed tests. This is useful for indexing various things include the CW log streams generated by each command
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: danbudris
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [danbudris]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: danbudris, g-gaston
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [danbudris,g-gaston]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Removed One Expired Link

Removed an expired link.

@karentamrazyan, should I remove others or add new ones, too?

hey @oldphones68 thanks A TON for the contribution! And yes, feel free to update anything you find broken, outdated or add new stuff ❤ 🙏


Work-around problematic adb behavior during screenshot pull

Addresses #4915
adb creates a different directory structure during pull commands depending on whether or not the destination directory exists already.
Work around this by ensuring that the destination directory exists before doing the pull.

LGTM 👍

This change was effectively reverted by #5567


Running into assertion failure while applying relocation for macho-x86_64

Running into assertion the following failure while ld links fine:
Assertion failed: (r.size == 4), function apply_reloc, file /Users/tapanth/sold/macho/arch-x86-64.cc, line 200.

I added a few logs to see where we are running into this:
mold: fatal: Assertion failure, r.size != 4, Archive:./PPRiskMagnes.framework/PPRiskMagnes File:MagnesSystemConfigUtils.o
mold: fatal: Assertion failure, r.size != 4, Archive:./PPRiskMagnes.framework/PPRiskMagnes File:DynamicDataCollector.o
mold: fatal: Assertion failure, r.size != 4, Archive:./PPRiskMagnes.framework/PPRiskMagnes File:TelemetryDataCollector.o
mold: fatal: Assertion failure, r.size != 4, Archive:./PPRiskMagnes.framework/PPRiskMagnes File:MagnesSensorDataCollector.o
mold: fatal: Assertion failure, r.size != 4, Archive:./PPRiskMagnes.framework/PPRiskMagnes File:AuditDataCollector.o
mold: fatal: Assertion failure, r.size != 4, Archive:./PPRiskMagnes.framework/PPRiskMagnes File:MagnesConstants.o
mold: fatal: Assertion failure, r.size != 4, Archive:./PPRiskMagnes.framework/PPRiskMagnes File:MagnesSensorDataCollector.o
mold: fatal: Assertion failure, r.size != 4, Archive:./PPRiskMagnes.framework/PPRiskMagnes File:MagnesNetworkManager.o
mold: fatal: Assertion failure, r.size != 4, Archive:./PPRiskMagnes.framework/PPRiskMagnes File:DynamicDataCollector.o
mold: fatal: Assertion failure, r.size != 4, Archive:./PPRiskMagnes.framework/PPRiskMagnes File:MagnesSensorDataCollector.o
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Here's a reproducer; just execute the mold_link.sh script
mold_repro_assertion_failure_while_applying_relocation.tar.gz

We've tightened error checks recently which caused this error. Please try again with the above change.


bug - keepkey btc legacy send fail

@tshifty tried to send legacy to segwit and it failed with this console error

this has been narrowed down to specifically keepkey btc legacy sends with a new version of hdwallet, this doesn't appear in beta, and must have been this way for some months.

@0xdef1cafe - to try and move this forward next week

@pastaghost / @bithighlander - can you take a look at this one at some point.

ended up being this 1 line change to prevent converting already sats amount to sats for utxo referenced prevTx output value causing hash mismatch in firmware.
https://github.com/shapeshift/hdwallet/pull/554/commits/2eecaefe67166e532e52f31b363f5e514896e2f2

closed by https://github.com/shapeshift/web/pull/2238
💩


The more you are zoomed out the more off are the tile highlights in fixed mode

Zoomed out more:

Zoomed out less:

Side

see https://github.com/runelite/runelite/issues/2420

Oh so it actually was an issue.


1.16.4 BioFuel Generator acts like Creative infinite power block

Industrial Foregoing Version:
3.2.5 according to my mods screen
When using the Biofuel Generator, it never loses power... goes to full power and stops...
appears to function like an infinite power source
I used a bioreactor to fill it with the bio fuel until power reached Max Value
then I removed the boo reactor to make sure nothing else was still powering it
I attached an RF Tools builder to it to see if I could drain it....
RF tool Builder would only fill at 10k per tick...and it can power much faster than that
and biofuel generator never went down in power or biofuel

What is your titanium version?

What is your titanium version?


Change link for metrics

What's changing
Fix metrics link
How to test it
README
Related Jira Ticket
Additional notes for reviewers
I already...

[ ] added some tests for any new functionality;
[ ] updated the documentation.

😎 @veekaybee


[android] New language: Romanian

Ticket #265

please git pull the changes on the ams tool so you can get any left over strings out before we merge
https://github.com/gubatron/android-missing-strings

@gubatron Done?


Support max number of profile files output in -loop feature

I've become a fan of the new -loop mode feature for continuous profiling. However I am worried that it will continue writing profiles until the disk runs out.
It would be great if we have a feature that lets us keep at most N number of profiles, and once we've reached that the profiler will overwrite the oldest file.
Implementation wise, I imagine that simplest way is to introduce a new filename template %c which can be used with the filename flag -f /path/to/profile-%c.jfr and what it does is keeps a count (i.e. from 0-9, and then back to 0). So the output filename will be profile-0.jfr, profile-1.jfr ... all the way to profile-9.jfr and then go back and overwrite profile-0.jfr. This is similar to log-rotate naming.
This way I don't have to worry about disk running out of space. At the same time if I use -loop=60m I can retain up to 10 hours of profiling data but only download the one I am interested in.

It's an interesting idea. I'd only add the possibility to control the maximum number of files, either directly in the filename pattern (e.g. profile-%10c) or with a separate option: --maxfiles 10

I am interested in having this capability as well. My use case is that I want to continuously profile some applications running in k8s, where I have limited disk space. Would you be willing to accept a PR that adds this functionality?

Added %n pattern, which is expanded to a sequence number.
It can be optionally followed by {max}.
Examples:
profile-%n.jfr is expanded to profile-0.jfr, profile-1.jfr, ..., profile-999999.jfr;
p%n{4}.jfr is expanded to p0.jfr, p1.jfr, p2.jfr, p3.jfr, p0.jfr, p1.jfr, ...


Fix cargo clippy and rustfmt for Rust 1.79

Summary
After the recently merged #12543 when working with Rust 1.79 cargo fmt makes a small formatting change that rust 1.70 wouldn't and clippy makes flags a &Vec<_> that should be a slice &[_] instead. This commit makes these two small chagnes so they're not an issue for people building with the latest stable version of rust, not just our MSRV.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

2 of 3 (66.67%)  changed or added relevant lines in 1 file are covered.
18 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 89.777%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

crates/accelerate/src/synthesis/permutation/utils.rs
2
3
66.67%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
6
92.11%

crates/qasm2/src/parse.rs
12
96.23%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
64520

Relevant Lines:
71867

💛  - Coveralls


Accept experiments submissions (bug 1220097)

Closing (wrong bug number, branch name, commit bug number)


[wallet-ext] show only 2 ledger accounts in the import flow

Description
https://mysten-labs.slack.com/archives/C04R8B45QDS/p1681502281725109?thread_ts=1681493666.781589&cid=C04R8B45QDS
TLDR; Ledger wants us to make users confirm every request to getPublicKey, so we're only going to show 2 accounts until we work out some improvements to the Sui Ledger app and figure out the best pagination / load more / infinite scrolling UX.
Test Plan

Manual testing

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[X] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes
N/A

Actually nevermind on this, we don't need to make any urgent UX changes after all


.NET Core 3 Targeting and Demo Project.

I updated the project with the following changes:

Added .NET Core 3 and .NET Framework 4.6.1 targets to the library.
Added a .NET Core 3 WPF Demo project that is identical to the original demo project.

I left a target in the library for .NET 4.5.2 also.  The new project type for the library allows for a NuGet package to be created automatically as well.

Thanks for your PR, @blakepell ! Everything looks alright so I'm going to merge it. Sorry for such a long delay.


Wrong incremental build

Environment

Gradle: 8.2.1
Android Gradle Plugin: 8.2

What happened

Build JNI libraries written in Rust and assemble AAR
Update Rust source code
Build and assemble once again
Updates are not contained in AAR

Expectation
JNI libraries in AAR must be updated after the Rust code is updated.

fixed by https://github.com/emakryo/rust-android-gradle/pull/15/files

fixed by https://github.com/emakryo/rust-android-gradle/pull/15


Unable to prove Reddit identity

Keybase GUI Version: 1.0.18-20161116112520+95f8303
Windows 10 Pro
Click "Prove your Reddit"
Enter username
Click "Reddit form"
Nothing

Same issue when trying to verify on iPhone both when launching Reddit app and in safari.

@CapraRoyale  - can you confirm this is fixed now? (and close issue if so). thanks!

It seems to have resolved on it's own.

No. it is still there and it wont allow to prove on reddit on iOS. Bug. From 2017 not fixed today. laughable.


check for field "state" always fails

Description
Scenario: To enabled threat_detection_policy for sql db.
Given I have azurerm_mssql_database defined
Then it must contain threat_detection_policy
Then it must contain state
And its value must be Enabled
To Reproduce

Feature File:

        Given I have azurerm_mssql_database defined
        Then it must contain threat_detection_policy
        Then it must contain state
        And its value must be Enabled

Plan File:
 "address": "module.sql_server.azurerm_mssql_database.db[\"exampledb1\"]",
                            "mode": "managed",
                            "type": "azurerm_mssql_database",
                            "name": "db",
                            "index": "exampledb1",
                            "provider_name": "registry.terraform.io/hashicorp/azurerm",
                            "schema_version": 1,
                            "values": {
                                "create_mode": "Default",
                                "geo_backup_enabled": true,
                                "name": "exampledb1",
                                "recover_database_id": null,
                                "restore_dropped_database_id": null,
                                "storage_account_type": "GRS",
                                "tags": {
                                    "CostCenter": "1234",
                                    "ProductOwner": "ANY"
                                },
                                "threat_detection_policy": [
                                    {
                                        "disabled_alerts": null,
                                        "email_account_admins": "Disabled",
                                        "email_addresses": null,
                                        "retention_days": null,
                                        "state": "Enabled",
                                        "use_server_default": "Disabled"
                                    }
                                ],
                                "timeouts": null
                       }

Error Output:
Failure: Resource module.sql_server.azurerm_mssql_database.db["exampledb1"] does not have its value property enabled (its value=[]).
Failure:
Expected Behavior:
Passed. Works fine when other values within the same block like use_server_default are used.
Tested Versions:

terraform-compliance version: 1.3.22
terraform version: 1.0.3
python version: 3.9

Within the terraform-compliance  "then" directive the term 'enabled' is a reserved word - adding quotes around the value (i.e. "Enabled") should resolve the issue.


add emacs support

basic emacs support

Thanks for starting to contribute some Emacs support!
One issue we'll run into at least on the Mac, and maybe on Linux, is that Meta is not always Alt. For example in my preferred Emacs Mac port Command ("cmd-") is Meta. Probably we'll need to define an action that takes a placeholder like "meta-" or "M-" and maps it, perhaps based on a setting? I could even see some people would want to be able to use Esc as Meta. (Esc as Meta may actually be a reasonable default given it'll work everywhere.)

ok, I've added a setting for how meta should be done: esc, alt, or cmd.
and spruced up a few other things.

I need to test this locally, I've made a bunch of edits without running the result now.

I'm not sure it's a good idea to bind all of these talon commands directly to key bindings, as users may have customized the bindings to perform different things. I think we should use M-x <command-name> where possible.

I think another approach for implementing this would be:

Having a dictionary mapping Talon command to Emacs command, i.e.

command_map = {
    "file open": "find-file",
    ...
}
ctx.lists["user.emacs_commands"] = command_map

The Talon file could look like:

[emacs] {user.emacs_commands}: user.do_emacs_command(emacs_command)

do_emacs_command would do the following:

Look up what key to use for the given command in settings/emacs_commands.csv, i.e. 'find-file' would return 'ctrl-x ctrl-f'. A user could customize this CSV to change any of the bindings, and we can create an elisp script for users to generate it themselves, until we have something better like voicemacs
If there's no binding in the file, we fall back to M-x, i,e. M-x find-file

The benefits of this are it's more maintainable and extendable (it's easier to modify a dictionary or CSV file instead of a Python or Talon file), it's easier for users to use their own keybindings (with the elisp script), and when Voicemacs or some other server is ready for use, it'll be easier to integrate.
Let me know what you think.

I like the idea of a CSV file to record metadata about commands (eg their keybindings; could also use this for "short forms" - M-x will accept abbreviations of commands as long as they're sufficiently unambiguous, which can be faster than typing the whole command if there's no direct keybindings).
I'm less sure about whether spoken forms should go in that file vs just using a .talon file. Not all voice commands will correspond to a single Emacs command, for example. But I guess we can put the names for single commands into the CSV file and the rest into a talon file.

I have updated this PR heavily, bringing it in-line with my current configuration, which is based on @gsingh93's suggestion to use a CSV file mapping commands to metadata about them, namely their key-binding and/or short-form. Almost all uses of key(...) have been replaced with user.emacs(command_name), which performs the given command by whatever method is appropriate - by default, it uses a keybinding if known, otherwise M-x.

@nriley if you'd be willing to give this a spin it would be greatly appreciated.

ok, I think this should be ready for merging.

oh, wait, still don't have a cancel/quit command for ctrl-g/keyboard-quit

ok, I've added a command cancel for keyboard-quit/ctrl-g. I don't have a strong opinion what this should be called; I use cancel for this but have it bound across multiple applications (it does esc by default, ctrl-c in terminal, and presses ctrl-g three times in emacs with sleeps in between), so I'm just gonna disable it. I would be concerned that quit is, like, super short in a chaining context? but it's probably fine I guess?

@rntz I can't think I've ever had a misrecognition with "quit", but if others feel strongly I am happy to leave "cancel" as the default. You don't feel like there is a conflict with "cancel cancel"? I admit I never actually use that knausj feature...

ok, hopefully this is good enough now, because I don't have much more energy to work on this!


Update HexColor.m

Off topic question: Why are you reversing the equality checks? (e.g. 7 == hexString.length instead of hexString.length == 7)


Cleanup non standard  token prefix

Please provide a description of this PR:

@costinm Do you know when is this prefix used?


Add a Transactional Outbox design pattern to the application

Description: https://microservices.io/patterns/data/transactional-outbox.html
An idea to implement:
Currently, the application of the recipe sends a message to the delivery application as soon as a new recipe is created. In case if the message broker is not available, the message is not delivered. To avoid this situation the transactional outbox design template is used.
To use this template it is necessary to create records in the outbox table and use a poller to read unprocessed messages and send messages based on them.

Implemented in #34


Update correct volume group name in ReadMe

Pull Request template
Please, go through these steps before you submit a PR.
Why is this PR required? What issue does it fix?:
Fix this error while create PVC in K8s:
Warning  ProvisioningFailed    64s (x8 over 3m13s)   local.csi.openebs.io_openebs-lvm-controller-0_c6baa0a4-8d08-43d5-8521-8fb877620fd9  failed to provision volume with StorageClass "openebs-lvmpv": rpc error: code = ResourceExhausted desc = no vg available to serve volume request having regex="^lvmpv-vg$" & capacity="4294967296"
What this PR does?:
Correct document text
Does this PR require any upgrade changes?:
No
If the changes in this PR are manually verified, list down the scenarios covered::
Correct group name when setup volume group in node
Any additional information for your reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs

Sorry, it's a mistake. The volgroup of StorageClass must match this volume group name.


Can't debug windows build?

It's my understanding that the debugger should be openable on a windows electron build via shift+ctrl+I, but this (or other shortcuts) seem to do nothing and AFAIK, the devTools: false in the config should only prevent a debugger from auto-opening rather than prevent it from opening at all... I'm not familiar enough with electron/electron-forge/etc. to know if this is expected behavior or not though.

Apologies. Documentation says devTools: false should result in this behavior.


Some ERROR about Encode of transcription in Video Understanding task.

We identified the issue and completed a hotfix in main branch. It should work after updating the latest code in main branch :).

We identified the issue and completed a hotfix in main branch. It should work after updating the latest code in main branch :).

Could you please tell me which python file you modify for fix this probelm?


Trailing spaces should contribute textWidth

In Flash, a long line of spaces at the end of a word-wrapped line will be cut off*. However, sadly, these spaces still contribute to textWidth and can be sized by getCharBoundaries(). textWidth can exceed the width of the field without any scrollHing, and getCharBoundaries() can get the bounds of offscreen and inaccessible characters.
Currently, trailing spaces are truly deleted and causes getCharBoundaries() to report null on valid characters in certain cases.
*unjustified, it will fill the line with spaces up until they would cause a wrap. Justified, no trailing spaces are shown
Repro:
var textField = new TextField();
textField.wordWrap = true;
textField.multiline = true;
textField.x = 20;
textField.y = 20;
textField.width = 360;
textField.height = 300;
textField.text = "here's some           here's some          here's some          here's some                                                           here's some          ";
// textField.setTextFormat(new openfl.text.TextFormat(null, null, null, null, null, null, null, null, TextFormatAlign.JUSTIFY), -1, -1);
trace(textField.getCharBoundaries(111)); // FLASH: (x=423,y=2,w=3,h=15) NEKO: null
trace(textField.textWidth); // FLASH: 490 NEKO: 355
trace(textField.maxScrollH); // 0, the spaces cannot be scrolled to

Seems like spaces sometimes do wrap and show up at the beginning of the next line, which is wrong. Only the first line of a paragraph can start with spaces. I'm curious what a full initial line of spaces before any real text looks like...

Spaces that are placed beyond width still exist. maxScrollH is always 0 during wordWrap, so they can't be scrolled to. But you can navigate through them with LRUD and even highlight them. The highlight box stops at the gutter, though.
Trailing spaces don't contribute to any align, but leading spaces do (within these rules, leading spaces can only occur at the beginning of a paragraph). The spaces will be placed after the aligned text.
There's definitely a lineWidthWithoutSpaces going on, which the aligners use. realLineWidth is used in getLineMetrics and textWidth but ONLY in LEFT align. I realize this is confusing.


Parallel cross building

Attempt at https://github.com/prometheus/promu/issues/176

Thanks for looking at this! See https://github.com/prometheus/promu/issues/176#issuecomment-586230373, I'm working on something that would enable the improvements I've described there.

Any update here @simonpasquier?

Is there any interest in resuming work on this?

Not from my end, anyone is welcome to takeover from where I left it if interested.

Closing in favor of https://github.com/prometheus/promu/pull/208.


Issue with using extended query protocol in DBeaver (Received resultset tuples, but no field structure for them)

CrateDB version
5.4.2 / Master
CrateDB setup information
No response
Problem description
It seems like DBeaver with PG JDBC (42.5.2) currently can lead to issues, when scrolling in a result set that breaks the connection with the following:
Caused by: java.lang.IllegalStateException: Received resultset tuples, but no field structure for them
	at org.postgresql.core.v3.QueryExecutorImpl.processResults(QueryExecutorImpl.java:2301)
	at org.postgresql.core.v3.QueryExecutorImpl.execute(QueryExecutorImpl.java:356)
	at org.postgresql.jdbc.PgStatement.executeInternal(PgStatement.java:496)
	at org.postgresql.jdbc.PgStatement.execute(PgStatement.java:413)
	at org.postgresql.jdbc.PgStatement.executeWithFlags(PgStatement.java:333)
	at org.postgresql.jdbc.PgStatement.executeCachedSql(PgStatement.java:319)
	at org.postgresql.jdbc.PgStatement.executeWithFlags(PgStatement.java:295)
	at org.postgresql.jdbc.PgStatement.execute(PgStatement.java:290)
	at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.execute(JDBCStatementImpl.java:330)

Steps to Reproduce
It is easy to replicate in DBeaver, however not so sure how to replicate it externally.
create table a as select * from generate_series(1,1000,1) g(a);

select * from a;

DBeaver intially fetches 200 records.
Scrolling past 200 records triggers another:
select * from a;

CrateDB - WireShark

PG16 - WireShark

Actual Result
it doesn't work
Expected Result
it works

♻ Workaround
Only use extended query protocol for prepared statements:

@proddata can you check if https://github.com/crate/crate/issues/14778 resolves the issue?

@proddata can you check if https://github.com/crate/crate/pull/14955 resolves the issue?
Seems to resolve it ✅

Tested with
DBeaver Version <IP_ADDRESS>310221533
master -> reproducible
j/fetch-interleaved -> not reproducible

Thx @proddata !


support for LDAP user authentication

Added support for LDAP user authentication.

#2179 has direct dependencies with https://github.com/captncraig/easyauth/pull/1, So can't be merged
Test Error:
cmd/bosun/web/middlewares.go:103: unknown field 'UserBaseDn' in struct literal of type ldap.LdapProvider

This pr seems a bit odd. It has commits from other branches that it really shouldn't. I can't really dig in and see exactly what the relevant changes are. Can you rebase onto the current master just your changes? I merged the easyauth pr, so that can also be re-vendored now.

@captncraig  sure ...


No module named 'realsense_device_manager

Required Info

Camera Model
L515

Firmware Version
<IP_ADDRESS>)

Operating System & Version
{Win (10)

Platform
PC

Language
python

Hey Everyone
I would like to take 2 pictures and then merge them later based on the internal values. To get an idea how to do that, I saw this file.
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/python/examples/box_dimensioner_multicam/box_dimensioner_multicam_demo.py, which I tried to run but i encounter in this error :
No module named 'realsense_device_manager'
I saw then #8184, but that did not help me any furhter. My pip has the version  22.0.2 and python has 3.8, im currently working with pycharm and pyrealsense2 is already installed.
I do know where the problem is, hopefully somebody got any idea ?

Hi @TheMarsupilami  The script box_dimensioner_multicam.py script is part of a larger project structure that includes the realsense_device_manager.py script.
https://github.com/IntelRealSense/librealsense/tree/master/wrappers/python/examples/box_dimensioner_multicam
It sounds though as though point cloud 'stitching' may be more compatible with what you have in mind.  This subject is discussed in regard to Python at https://github.com/IntelRealSense/librealsense/issues/9832#issuecomment-938701928

Hi @TheMarsupilami  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


CLFloor

Would it be possible to extend this library such that it supports access to the floor attribute in CoreLocation. Here are the docs for the functionality I am asking about : https://developer.apple.com/documentation/corelocation/clfloor
Thank you.

Hi @VladIacobIonut! I will try to iterate this issue tonight, and expose all possible properties. :)

Hey @VladIacobIonut! Finally had some time to check this issue.
Sadly, as stated on Apple Documentation, CLFloor is only available on iOS 8.0+, tvOS 9.0+, and watchOS 2.0+. Are you trying to use this library on any of these platforms?
I just pushed 0.2.0 to NPM, exposing altitude, horizontalAccuracy, and verticalAccuracy. :)

Hi Victor ,
Could it be the case that the library is not supported by Meteor/Cordova ?
Thank you !

Oh, I see.
Well, I used this on an Electron app, but I'll take a look at meteor and check if I can somehow integrate it there. :)


How to render on standalone html file

I have .excalidraw file available as url like this hosted on github.
How to display this file in standalone static html file which doesn't use any backend?
Something like using iframe.

do you mean you want to integrate Excalidraw into your app?

I have a blog (GitHub pages). I want to embed the drawing in my blog

You can use iframe to render Excalidraw drawing as shown here using shareable links, for more functionality you can integrate Excalidraw in browser directly

Neat. Thanks!!

closing this as its resolved


LS based errors

Very much a work in progress. For discussion/demos only at this point.
Demo prep

[x] Disentangle the runtime errors work from this (and #214) and merge that separately.

Now https://github.com/microbit-foundation/python-editor-next/pull/224. This needs rebasing on main once that's merged.

[x] Remove TygerPython cruft from this PR. I expected to fail quickly but instead I've ended up with a successful mess!

Investigation

How viable is it to override common error messages to improve them? They're in resource bundles etc. but how general are the error codes?

The type-related messages are pretty specific.
The syntax related ones are dominated by 40+ expectedFoo message codes, e.g. expectedSliceIndex and a few lower-level parse issues.
I think we need to experiment with this with an idea of the messages we care about.

Minimal mergeable PR

[ ] Sort out the language server client

I think we should replace with something based on vscode-jsonrpc that supports multiple files and perhaps workspaces from the start. It can drop features, we can add them back incrementally. This means it needs an API beyond just being a CM extension and integration with the rest of the app (file create, delete, open, close, edit). We should also do incremental edit support.
It would be good to improve typeshed initialization. Can we make it part of the initialize message?
We should probably open all Python files in the project (there won't be many). That way we'll own them and the language server won't expect them to be in the file system.
We can tell the language server when file operations occur. Not sure if Pyright supports this
We should remove (non-error) hovers for now

[ ] Acceptable quality stubs

Matt to chat to Carlos about approach

Sufficient feature quality (to be discussed) for:

[ ] error messages (but no text customisation)
[ ] autocomplete (can be split if sensible)

[ ] Licensing. As the built code is in our repo we should include really clear licensing info with a copy of Microsoft's license. I think ideally we'd build it and package it but not sure we're ready for that quite yet.

[ ] Test Pyright with Monaco, perhaps inside the Pyright fork. A testing environment for Pyright in the browser should be using Monaco to uncover problems rather than a minimal language server that's also undergoing change.

Later

[ ] Customise common error messages in educational scenarios.
[ ] Improve stub docs and discuss exposing hover information. The hover UX might not be what we want but the information has value. Might need lots of edu team info on quality of text for code stubs though.
[ ] Parameter hovers. VS Code shows completion when you're between the brackets. It shows the current parameter in bold etc.

Having a quick play with this alongside #214, there are definitely some cases that I think the TigerPython errors make much more clear
This error:

Compared to this one:

@microbit-matt-hillsdon - I know we've discussed it generally, but thought I'd add here: is it possible to get down to the nice specificity of TigerPython with Pyright?

Work will slowly move to #251, but for now this branch has more features and is a useful reference.
I'll move the comments too before closing this.

@microbit-matt-hillsdon - I know we've discussed it generally, but thought I'd add here: is it possible to get down to the nice specificity of TigerPython with Pyright?

This particular case was a bug in the language server integration. Pyright now reports Expected ":" at the same point as Tiger Python. It does also have some knock-on errors that seem less than ideal, including that one about the expected expression. I suspect this could be a bug. I'll take a look soon.

This particular case was a bug in the language server integration [1]. Pyright now reports Expected ":" at the same point as Tiger Python. It does also have some knock-on errors that seem less than ideal, including that one about the expected expression. I suspect this could be a bug. I'll take a look soon.

Raised #257 to track "soon" 🙂

Closing as we're going with #251.


Non inclusion proof?

It says that you can generate non inclusion proofs, but there doesn't seem to be any method provided for doing that?

Sorry for heavily late reply and thanks for your interest.
As you pointed out, there's no explicit method provided like check_non_inclusion().
However, the monotree already has that intrinsic feature. Remember that the monotree is a Sparse Merkle Tree :)
Just follow the key path you want to read.
(using the monotree's get() method: https://github.com/thyeem/monotree/blob/master/src/tree.rs#L105)
If there's nothing (None), it means there's really nothing (non-inclusion).  This easy non-inclusion proving is one of the main reasons we use sparse merkle trees.
This requires lots of hashes initially stored, but luckily as they are mostly hahses of

None
hash(None)
hash(hash(None))
hash(hash(hash(None)))
... (# of tree depth)

This observation allowed me to optimize monotree like this:

Initially cache those hash values above.
Keep compressed every path as much as possible when inserting so that it can reduce the number of db works in both reads and wrties.

I'm sure you've got already your own answer, but I'm writing for someone with the same question.


Slightly lighten the shade of grey for uncategorized alters.

node background color (grey) and the alter name color (black) might not contrast enough for people with visual impairment.
Change the shade of grey to be slightly lighter to help with this issue. At the moment, I believe this color is actually 'grey' in the css, so it should be relatively easy to track down instances of this color and change it.

I'm replacing all instances of "grey" with a new lighter color #acacac/rgb(172, 172, 172). Hopefully that will help to make the text more readable.
Note: This change will not affect existing participants in the database, since the color that is being referenced is stored as "grey" in the database. Should be good to go for new users in future waves though.


DLR loss implementation in TF2

Hi authors,
The DLR loss in TF2 may be incomplete. Misclassified cases are not be implemented:
https://github.com/fra31/auto-attack/blob/42189aab794672131f7f7584f15710e41f5c540d/autoattack/utils_tf2.py#L196-L202
I tried to implement DLR loss from the original paper's description in Section 4.1. Please verify the following code's correctness.
def dir_loss(x, y, num_classes=10):

    # logit
    logit = x
    logit_max = tf.reduce_max(logit, axis=1)
    logit_sort = tf.sort(logit, axis=1)

    # onthot_y
    #argmax_y = tf.argmax(y, axis = 1)
    argmax_y = y
    y_onehot = tf.one_hot(argmax_y , num_classes, dtype=tf.float32)
    logit_y = tf.reduce_sum(y_onehot * logit, axis=1)

    # z_i
    cond = (logit_max == logit_y)
    z_i = tf.where(cond, logit_sort[:, -2], logit_sort[:, -1])

    # loss
    z_y = logit_y
    z_p1 =  logit_sort[:, -1]
    z_p3 = logit_sort[:, -3]

    loss = - (z_y - z_i) / (z_p1 - z_p3 + 1e-12)

    return loss

If the code is acceptable, I will open a new PR.

Hi,
thanks for the contribution! It looks good to me, feel free to open a PR.
As a minor comment, is argmax_y = y necessary or would tf.one_hot(y, ...) work as well?

As a minor comment, is argmax_y = y necessary or would tf.one_hot(y, ...) work as well?

It dependent on what shape of y in the code.
In my own code, the term y is a one-hot vector, whose shape is [batch_size, n_class]; the argument y in this project is scalar, whose shape is [batch_size, 1].
The operation tf.reduce_sum(y_onehot * logit, axis=1) is to get the logit belongs to y class. Because y_onehot has been a one-hot vector already, the element-wise multiplication ensures that other classes are masked.  Next, we do the reduce_sum along axis=1 and the variable's shape is from [batch_size, n_class] to [batch_size, 1] and its logit belongs to y class.


init: order dynamic resource initialization to make RTDS always be first. Take 2.

Commit Message:
Order dynamic resource initialization such that RTDS is completely initialized before starting to load rest of xDS resources.
Additional Description:
This PR puts back the original commit with xDS ordering and fixes the bug where cluster manager did not handle the case of asynchronous completion of primary cluster initialization.
Risk Level: High (changes to xDS and cluster initialization)
Testing: Unit Tests, Integration Tests and repro case
Docs Changes: N/A
Release Notes: N/A
Fixes #9709

Giving it a try now.

@yanavlasov so far so good 👍


docs:policylibrary: more examples on privileges escalation and execution

policylibrary: Privileges Escalation, unlinked and fileless binary execution

LGTM, thank you! Please move the configuration details out from the "Policy Library" and just reference the corresponding page.

Thanks @sharlns for the review, I did that. If it feels not perfect then let's discuss that in separate issue or PR please

Thanks, I haven't checked the content just some nits on the form.
If you could also wrap consistently the text around 80 chars that would be nice. Some paragraphs are in your PR while others aren't.

Thank you @mtardy I did and fixed all others ;-)

On another topic I find https://tetragon.io/docs/policy-library/observability/ really hard to read, maybe we could present that differently but that's another topic :)!

Agree, need to not look at it for a moment then read it again ;-D !


Record Edit Post Submission Page

With the new changes in ermerstjs, now create and edit will return a object that contains successful and failure. so chaise doesn't need to check for the failure cases and can show these two pages.
Also the returned page is contextualize and chaise doesn't need to contextualize it again.
This PR also changes the bind-html-unbind that has been used in multiple apps to ng-bind-html.

@jrchudy I made the changes and addressed your comments.

I verified that the tests pass locally from start to finish for me using your ermrestJS branch and Chaise branch for this issue.


Adding support for slack icon_emoji in slack receiver config.

As it may be useful to quickly identify notifications in a channel stream, we needed support for "icon_emoji" in the slack receiver configuration.
Configuration look like this :
slack_configs:
  - send_resolved: true
    username: 'Prometheus'
    channel: '#monitoring'
    icon_emoji: ':prometheus:'
    api_url: 'https://hooks.slack.com/services/<token>'

@fabxc, is this small addition ok for you ? Thanks in advance for your feedback.

Thanks!

Merged too early. Template changes need a manual run of make assets.
(It's not run by the default target since it rewrites timestamps aggressively every time)


Fix typo ( SongsRepresentable )

Just fix a small typo. For parsing without dedicated array representable, you need to change SongRepresentable, not SongsRepresentable.

Thanks, please PR against the branch f6.


about questions in the dataset

Hi,
I find a lot of questions in the PopQA such as "What genre is Organisation?" is very vague and almost can not be answered correctly by human. When using Wikipedia to retrieve documents related to the question, I get the following top-10 docs:
knowledge  0 : Nature and Organisation | Nature And Organisation is the creative musical works of British musician Michael Cashmore. The music of Nature And Organisation is characterized by a mix of acoustic, classical and folk structures met with abrasive electronic instrumentation.
knowledge  1 : Organisation (album) | Organisation is the second album by Orchestral Manoeuvres in the Dark (OMD), released in 1980. " Enola Gay" was the only single taken from the record. Malcolm Holmes had played drums with OMD before, notably on "Julia's Song" which was featured on the band's debut album, and for "Organisation" he was recruited as a full-time member replacing the TEAC tape recorder affectionately named "Winston". " The More I See You" is a cover of a song written by Mack Gordon and Harry Warren in 1945, and popularised by Chris Montez in 1966. The record was remastered and re-released in 2003, with several bonus tracks.
knowledge  2 : Organised Sound | Organised Sound is an international peer-reviewed academic journal which focuses on the rapidly developing methods and issues arising from the use of technology in music today. Published three times a year, it concentrates upon the impact which the application of technology is having upon music in a variety of genres, including sound art, sound sculpture and music ranging from popular idioms to experimental electroacoustic composition. It thus provides a forum for those interested in electroacoustic music studies, its creation and related developments to share the results of their research as they affect musical issues. Whilst an accompanying CD/CD-ROM/DVD is sent to subscribers annually all media content is or will soon also be available online.
knowledge  3 : Organisation (band) | Organisation zur Verwirklichung gemeinsamer Musikkonzepte (German: "Organisation for the Realization of Common Music Concepts"), was an experimental krautrock band, that was the immediate predecessor of the band Kraftwerk. In addition to the founding members of Kraftwerk, Ralf Hütter and Florian Schneider-Esleben, Organisation included Basil Hammoudi, Butch Hauf, and Fred Monicks. The band was assisted by Paul Lorenz, Peter Martini, and Charly Weiss during their career.
knowledge  4 : The Organization (band) | The Organization was a heavy metal band formed in 1991 after the breakup of the thrash metal band Death Angel. The band incorporated elements of metal, funk, and alt rock, releasing two albums in the early 1990s on Metal Blade Records. The band included the original line-up of musicians from Death Angel minus singer Mark Osegueda who left the music business to go to college. The Organization disbanded after bassist Dennis Pepa left the band in 1995.
knowledge  5 : Organization (journal) | Organization is a peer-reviewed academic journal that covers the field of management and organization studies. The editors-in-chief are Craig Prichard (Massey University) and Robyn Thomas (Cardiff Business School). It was established in 1994 and is published by Sage Publications.
knowledge  6 : The Organization (TV Series) | The Organization is a British television drama series, produced by Yorkshire Television for the ITV network in 1972.
knowledge  7 : Organizations (book) | Organizations is a book by James G. March and Herbert A. Simon.
knowledge  8 : Making Music (organisation) | Making Music (formerly the National Federation of Music Societies) is a UK organisation for voluntary music, with around 3,000 member groups. Its members include choirs, orchestras, music promoters, jazz and wind bands, community festivals, and samba groups, among others.
knowledge  9 : Come Organisation | Come Organisation was a record label started by William Bennett in 1979 as a way to release albums by his own band, Come, when he was unable to find a label willing to release them. It is best known for releasing the work of Bennett's subsequent band, Whitehouse.
We can find Organisation corresponds to multiple different items and the question does not describe the specific item it is related to. Therefore, both model and human can not get a correct answer. Would you like to make the questions in the dataset more clear?

Hi, there will inevitably be some ambiguity in datasets like this. Adding clarifying details inherently trades off against making a controlled comparison between entities, as less popular entities need more clarification. (Further, there is no way to change this now that the dataset has already been released.) The ambiguity is also somewhat realistic to real-world scenarios where users don't add clarifying details, and the most likely interpretation of the query is expected to be used.
If you'd like, you can resolve these ambiguities for your use-case using the wikipedia entity IDs given in the dataset.


mgmt/mcumgr/lib: SMP processing should be always able to at least return "rc" code

Is your enhancement proposal related to a problem? Please describe.
The mcumgr is able to process SMP frame only when it is able to allocate buffer for it. The allocation happens at transport level, when transport wants receives SMP data and needs buffer to store them.
The problem is that, when there are no more free buffers, it is left for transport to respond with an error and often happens that the the transport does not have such capability, for example serial will just drop the frame. This causes some problems as for example a client will not be informed what a problem was and may retry sending causing more frame drops, instead of just lowering the pace the frames are send at.
Describe the solution you'd like
Every time when transport is not able to obtain "request" frame for received data, from mcumgr, it should be given by SMP a rc=MGMT_ERR_ENOMEM frame that it can send back to the client.
The frame should have proper response header (group,command id, operation opposite to request and sequence number) and only payload of the rc code.
Describe alternatives you've considered
Adding some special SMP header to respond; modifying transport (not practical, not possible everywhere); leaving as is.
Additional context
BT dropping SMP frame: https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/mgmt/mcumgr/smp_bt.c#L232-L236
Serial dropping SMP frame when allocation fails, https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/mgmt/mcumgr/smp_uart.c#L41-L52, due to https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/mgmt/mcumgr/serial_util.c#L76-L81

I parked my changes here https://github.com/nordicjm/zephyr/tree/mcu_err_buf
This is a bit more complex that first anticipated, because depending upon the data received, which cannot be analysed due to the lack of a net buffer, there might or might not be sufficient information to construct a valid error response - the header information from the request is needed, group, ID, etc. and so if there is a problem with this information or if it is lacking is unknown as e.g. the point in the serial transport where this code is needed will not have been able to verify any of the incoming data, so a valid error response might not be able to be constructed.


Add DNS command line option

Update to add capability as specified in Issue #2.

Merged pull request to include addition of -d --dns option submitted in issue #2


docker container fails to start the jupyter notebook without --allow-root option

Dear Zipline Maintainers,
Before I tell you about my issue, let me describe my environment:
Environment
This issue is about running the docker container built from the latest Dockerfile on master: https://github.com/quantopian/zipline/blob/master/Dockerfile

Operating System: (Windows Version or $ uname --all) - the non-developer docker container
Python Version: $ python --version - the non-developer docker container
Python Bitness: $ python -c 'import math, sys;print(int(math.log(sys.maxsize + 1, 2) + 1))' - the non-developer docker container
How did you install Zipline: (pip, conda, or other (please explain)) - the non-developer docker container
Python packages: $ pip freeze or $ conda list - the non-developer docker container

Now that you know a little about me, let me tell you about the issue I am
having:
Description of Issue

What did you expect to happen?
What happened instead?

Here is how you can reproduce this issue on your machine:
I built the non-developer docker container and it fails to start the notebook without the --allow-root option:
root@zipline:/projects# jupyter notebook -y --no-browser --notebook-dir=${PROJECT_DIR} --certfile=${SSL_CERT_PEM} --keyfile=${SSL_CERT_KEY} --ip='*' --config=${CONFIG_PATH}
[C 19:42:34.067 NotebookApp] Running as root is not recommended. Use --allow-root to bypass.

Confirm it's not running somewhere in the background:
root@zipline:/projects# ps auwwx | grep notebook
root       205  0.0  0.0  12820  1028 ?        S+   19:42   0:00 grep notebook

Add the --allow-root option and retry...it works:
root@zipline:/projects# jupyter notebook -y --no-browser --notebook-dir=${PROJECT_DIR} --certfile=${SSL_CERT_PEM} --keyfile=${SSL_CERT_KEY} --ip='*' --config=${CONFIG_PATH} --allow-root
[I 19:52:01.869 NotebookApp] Serving notebooks from local directory: /projects
[I 19:52:01.869 NotebookApp] 0 active kernels
[I 19:52:01.869 NotebookApp] The Jupyter Notebook is running at: https://[all ip addresses on your system]:8888/
[I 19:52:01.869 NotebookApp] Use Control-C to stop this server and shut down all kernels (twice to skip confirmation).

can this be added to the docker_cmd.sh?
jupyter notebook -y --no-browser --notebook-dir=${PROJECT_DIR} \
    --certfile=${SSL_CERT_PEM} --keyfile=${SSL_CERT_KEY} --ip='*' \
    --config=${CONFIG_PATH} --allow-root

Reproduction Steps

...
What steps have you taken to resolve this already?
-v ./fixed-docker_cmd.sh:/docker_cmd.sh
...
Anything else?
...
Sincerely,
$ whoami

I ran in to the same issue - I couldn't start the Docker container as per the instructions (error log below).  I made the change suggested by @jay-johnson in PR #1871.
...
writing new private key to '/root/.jupyter/jupyter.key'
/root/.jupyter/jupyter.key: No such file or directory
139911180314256:error:02001002:system library:fopen:No such file or directory:bss_file.c:398:fopen('/root/.jupyter/jupyter.key','w')
...
ValueError: certfile "/root/.jupyter/jupyter.pem" does not exist
...

Looks fixed in #1871.


Parallel evaluation

Parallel minor fix
issues fixed for reduction

Jenkins Linux s390x Build #13886 [push] Parallel evaluation (#26... started at 10:20

Jenkins Linux amd64 Build #13859 [push] Parallel evaluation (#26... started at 09:20

Jenkins Linux ppc64le Build #12883 [push] Parallel evaluation (#26... started at 10:28

Jenkins Linux amd64 Build #13859 [push] Parallel evaluation (#26... passed after 1 hr 21 min

Jenkins Linux s390x Build #13886 [push] Parallel evaluation (#26... passed after 1 hr 33 min

Jenkins Linux ppc64le Build #12883 [push] Parallel evaluation (#26... passed after 2 hr 0 min


get an error when call createSplitZipFileFromFolder function

zip4j version:2.6.1
code as follow：
public static void main(String[] args) throws Exception {
ZipFile zipFile = new ZipFile("/out.zip");
// 编码格式
Charset gbk = Charset.forName("GBK");
zipFile.setCharset(gbk);
ZipParameters parameters = new ZipParameters();
parameters.setCompressionMethod(CompressionMethod.DEFLATE);
parameters.setCompressionLevel(CompressionLevel.NORMAL);
zipFile.createSplitZipFileFromFolder(new File("/黄燕-数据整理"), parameters, true, 65536L);
}

When zip4j creates split zip files, it first creates a file with a temporary name, and then renames the file (to example: out.z01, etc) when the split limit has been reached. Unfortunately, zip4j was not able to rename the file in your case, because it might be that a file with that name already exists in that output directory, or because the process that zip4j is running in does not have enough permissions to rename a file. Can you check if any of them is valid?

i don't think the reason about  permissions
i can rename manually.   please check

the above code can be renamed successfully

This obviously does not happen in all cases. I have tested this feature on my machine, and also there are Integration Tests for split zip. I have a Windows 7 machine. I will try on it to reproduce on it.

I tried this on a Windows 7 machine as well, and it worked perfectly fine for me. This might be something to do with your machine than zip4j. The only difference is that I tried with the latest version of zip4j (2.6.4). Maybe try upgrading to this version and see if that works.

Closing this issue, as this issue was not reproducible as mentioned above. All tests on different OS worked fine. Please feel free to reopen if you are able to let me know a way to reproduce this issue.


add configuration for Caipi vtol with reversed elevon servos

adds a new mixer for reversed elevon servos, a new rotation in lib/conversion
and sets MAV_TYPE in the init script to avoid the hidden dependency in rcS

I hate to do this: there is a new PWM_REV param set (look in the default or PWM out group) to reverse individual outputs. That should make special configs unnecessary.

I can't do without the new rotation value though, and I've burned 3 days trying unsuccessfully to merge this with the pwm_invert branch. You did notice that I tested that branch on a quad?
Unless I can get a working config under master for my Caipi VTOL, I'm not likely to make any further progress with it. Advice would be appreciated.

there is a new PWM_REV param set (look in the default or PWM out group) to reverse individual outputs.

QGC really needs a way to set all this up (or test it) for fixed wing. I'm just converting a fixed wing from ardupilot board to pixhawk and the fact that I can reverse output in my radio, reverse the rc channel with respect to the RC#_REV parameter and also reverse essentially the mixer output (right?) with these new PWM_REV params really makes you head spin!

Then just copy the small rotation changes over to an up to date release_v1.0.0 branch, which has all that already. Sorry to hear about the pain. I can merge your rotation changes pretty quickly if they are in a single PR.

Yes Don - it's not hard to do. Each step can be verified separately and QGC can set all the params for you. It's only confusing if you only look at inputs and outputs.


Feat: Reduce time when fetching masp

Issue
Link: Fetching shielded balance via SDK nor extension working
Solution

First, Make fetches by block ranges instead of all blocks.
Second, Use Promise pool from @supercharge/promise-pool library to process multiple fetch block ranges in parallel. It significantly reduced time when fetching block.
Then save the context of the rage block to a new temporary Indexed DB with key as startBlock-EndBlock and value as context after serializing this ranges block.It void loss data if the process is down. Only need fetching again missed ranges block.
Once finished fetching and saving all ranges blocks to temporary Indexed DB . Will call a function to get all the data from temporary Indexed DB  and load it into the context.
The last step is saved to the root db.
Also the solution help get the progress of process and display in to user.

Image

Button to naviagte shielded sync page

Shielded sync page

Note:

I have another change for Namada source. And change depency in cargo.toml to my change. PR: https://github.com/anoma/namada/pull/2898

Ping me on Discord so we can chat more easily, I want to help ! @phychain

@pnam1609 I really appriciate the effort. I think it will be hard to merge this soon. The reason is you worked under some assumptions about how things should work and look like. The proper flow would be:

create an issue
discuss solution in the created issue
settle on requirements
work on functionality
create PR

Cool, thank you for your hard work. We need it ASAP

Hi @mateuszjasiuk , Thanks for your response. I hope it's available as soon as possible and i can contribute more for project.

@pnam1609 Really nice ! I appreciate your comprehensive solution, which outlines a systematic approach to optimize block fetching by implementing block ranges, utilizing Promise pools for parallel processing, and ensuring data integrity through temporary IndexedDB storage. Your solution not only enhances efficiency by reducing fetch times but also provides a robust mechanism for handling interruptions and displaying progress to the user. Awesome valuable contribution.

Hello,
I think if we want to merge it quicker we have to chage few things :):

UI - We did not have a plan to create Shielded Sync button. User should not need to remember to do shielded sync every time they want to, for example, see up to date balance. So for the interface we call the "sync" each time we query for the shielded balance. So in general I would move all the UI changes to the separate issue so we can discuss it further :)

Changes to Namada repo - I don't think your changes will be merged in near future or at all. On the upside, there is a recent change in the Namada SDK(0.32.0) that allows you to specify the starting index of the fetch function, which should help with your solution.

WDYT?

@pnam1609
Hello, I think if we want to merge this quicker we have to chage few things :):

UI - We did not have a plan to create Shielded Sync button. User should not need to remember to do shielded sync every time they want to, for example, see up to date balance. So for the namada.me we call the "sync" each time we query for the shielded balance. So in general I would move all the UI changes to the separate issue so we can discuss it further :)
Changes to Namada repo - I don't think your changes will be merged in near future or at all. On the upside, there is a recent change in the Namada SDK(0.32.0) that allows you to specify the starting index of the fetch function, which should help with your solution.

WDYT?

Hi @mateuszjasiuk ,

For this shielded sync button and UI for that. Let me remove this.
I will change the dependency and will check to logic code again.


fix: reduce frequency of iframe removed error

Description
We're seeing rare cases when we see Detected iframe close before second render has a chance to finish in slow browsers with poor connection. With this change, we're adding an additional check before we decide to throw the error.
Here is the example:
https://github.com/krakenjs/zoid/blob/main/src/parent/parent.js#L866

@oscarleonnogales @dtjones404 @westeezy is it cool if we merge this and cut a new release of zoid?


removing todo comment from query docs

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes
Fixes #7024

Nice, thanks!


PySCF - maintain as optional install?

When chemistry was part of Aqua, and Aqua installed by default with qiskit, PySCF was changed to an optional install since it would otherwise install for everyone and some people had problems with their installation and did not even want to use chemistry function.
Given that chemistry is part of nature here, and you still might want to install it for physics or other aspects, maybe leaving PySCF as optional is fine. On the other hand today its mostly chemistry and our tests and tutorials etc use this extensively. So I figured I'd raise this for discussion as to whether we want to go back to having it installed as a dependent, like we had in the past before making it optional for reasons given above, or leave it be as optional....

I'm generally in favor of making it a requirement, except in pyscf's case specifically there have been several packaging issues over the years which have made it particularly problematic. The biggest of these is that there are no windows wheels available (and no convenient source based installation option for windows users) which means users on windows will not be able to install pyscf unless we specifically exclude windows on the requirements list (which is basically the same as optional). There is also macOS version compatibility, I believe all the recent macOS wheel files depend on >=10.13 which precludes users with older macs who can't install 10.13.
There are also issues with newer releases in the 1.7.x series where using pyscf will break the time stdlib modules: https://github.com/pyscf/pyscf/issues/814 (which is why everywhere we install pyscf typically pins it to avoid that issue).

we specifically exclude windows on the requirements list (which is basically the same as optional).

This was always the case and what we did do, not much choice here. The chemistry programs like PySCF and PSI4 don't do Windows; there is only PyQuante, which is quite limited and only has a conda pkg, or the commercial product Gaussian.
Given that Nature is predominately chemistry, the rationale of installing it was that you could do chemistry out of the box. Otherwise you have to also do an optional install of one of the drivers no matter the platform.
I agree there are limitations - question is it a positive for most people, given that you choosing now to install chemistry as its not installed by default. In the past I'd agree it was the right choice, just wondering if it still is.
I must admit I was not aware of the time issue. It seems like there is a fix in the works once PySCF goes to 2.0

Thinking about this and informing the users, in the past we used to have more around install. I note that, the docs here, we have an API ref, release notes and tutorials but no installation material specific to this that would note/reference to driver installation material etc. So at least I think we need this, and maybe this can be referenced from the main install of qiskit, as it makes sense, like the other docs here, to keep it separate but at least inform users installing optional aspects of qiskit, like this, optimization etc, that there is specific additional install information.

We have had some more discussion offline and, if I recall correctly, we have come to the conclusion to keep PySCF as an optional install (for the time being), correct @woodsp-ibm?

Yes. I had left this open as a reminder to augment the new documentation around installation - we used to have top level text or around optional element with links down to the install information. The latter text still exists in the drivers around installation what it's currently missing is some top level 'getting started' information around the optional install, that 'adds' to the getting started information for qiskit for the application specifics  - in this case Nature. In other words the fix for this issue was documentation improvement. If you want to close this and re-open a new one, or maybe rename the title and give it documentation label thats fine by me.


Release 1.0.0

Codecov Report

Merging #42 (204fa20) into master (6d46354) will decrease coverage by 5.47%.
The diff coverage is 42.41%.

:exclamation: Current head 204fa20 differs from pull request most recent head 92cf59e.  Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Implement extract operation

This change will introduce a new operation called extract. This operation is designed for transforming FHIR data into a flattened form, for use within other tools, such as statistical and machine learning models.
Still to do:

[ ] Fix problem relating to cartesian product
[x] Look into Spring Boot caching
[x] Allow for extract of Coding values
[x] Add support for async response
[ ] Add tests

I think I have finally cracked the problem of how to join the column expressions, taking into account element identity (thanks to an algorithm from the brain of @piotrszul).
I just need to implement a filtering solution now, which I think is going to be something very similar to what is used within the SearchExecutor (get a list of filtered IDs, and use a left semi-join).

@piotrszul I think this is ready for you to have a look at - thanks.

There are two problems remaining with this implementation:

Async and authentication don't work together, specifically in the retrieval of the async result via the $job operation. I think this is related to AOP tripping over itself. I need to add an integration test that reproduces this.
In order for this to work with Power BI (and perhaps other real-world integrations that have a problem with talking to multiple endpoints/protocols), we need to proxy the retrieval of the result. This adds complexity but I think it is necessary. I'm proposing to create a new $result operation that takes an ID. This operation will look up an in-memory structure which will contain the actual URL and the MIME type and stream it back to the client.

@piotrszul I've got async and authorisation working together, but now these two tests are failing for reasons that I can't fathom. It seems to be working as intended when tested manually.

SecurityEnabledOperationsTest.testForbiddenIfAggregateWithoutAuthority
SecurityEnabledOperationsTest.testForbiddenIfImportWithoutAuthority

It might be related to the fact that I needed to separate out the methods that were targeted by AOP in order to get it all working.

I'm going to merge this into release/5.0.0 now, I think this branch has lived long enough :slightly_smiling_face:


Enable Build from Vmware which use a proxy & Fix download Issue cause…

Fix of Downloading the Prebuild Compressed Folder from Github, for Vmware user who have proxy installed (Block downloads).

Thanks for your contribution. This feature would help others who work on offline systems.


Fix handling of unary operator in FinalizePrivateField

The recipe should distinguish between reading and modifying unary operators, as this may allow a few more fields to be made final.

@kunli2 I took the liberty to make one small improvement to your recipe with regards to the handling of unary operators.

Thanks @knutwannheden !

Thanks @knutwannheden !
good point, I just learned that -i is a J.Unary.


Fix command args parsing

Fixes issue #65 where the command line args were not being used correctly.
https://github.com/kimmobrunfeldt/git-hours/issues/65

@kimmobrunfeldt any chance to merge this one anytime soon? thanks!


Pasting Twice

Issue Type: Bug
Hi Team,
If I copy any of the variables in the jupyter notebook, it is getting pasted twice. For example:
I copied df_new, I got "df_new", It is pasting:
"df_newdf_new"

Extension version: 2020.7.1
VS Code version: Code 1.47.0 (d5e9aa0227e057a60c82568bf31c04730dc15dcd, 2020-07-09T08:02:06.629Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8350U CPU @ 1.70GHz (8 x 1896)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.85GB (0.63GB free)

Process Argv

Screen Reader
no

VM
0%

Thanks for the bug report! Closing as this is a duplicate of https://github.com/microsoft/vscode-python/issues/11796, please feel free to track that issue for updates. 😊 JFYI, the VS Code team has been working on fixes for this issue, and I believe they are tracking it at https://github.com/microsoft/vscode/issues/101946


Hide data lower than... on Y axis

Hello all !
I try to hide some data lower than 50, but my chart keep going down...
Is there any solution ?

Hi @FrSenpai 👋
There was a bug due to which noOfSectionsBelowXAxis={0} was not working. This has been fixed from version 1.4.4 onwards.
Please use the latest version of react-native-gifted-charts.


Fix technical issues with README.md

Please make sure that this is a documentation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:doc_template
System information

TensorFlow version:
Doc Link:

Describe the documentation issue
We welcome contributions by users. Will you be able to update submit a PR (use the doc style guide) to fix the doc Issue?

Nagging Assignee @MarkDaoust: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Closing this issue since its obsolete and README.md  has been updated. Thanks!

Closing this issue since its obsolete and README.md  has been updated. Thanks!


Styled Components - Reorder on save

Hey,
I've been using this awesome plugin to order styles across our entire organization. It works really well, and everyone is acquainted with the setup.
We are moving from Sass to the very amazing Styled Components. Unfortunately, this config doesn't reorder the styles written in a template literal.
Here is my .stylelintrc
{
    "defaultSeverity": "warning",
    "extends": [
        "stylelint-config-recommended",
        "stylelint-config-styled-components",
        "stylelint-order",
        "stylelint-config-rational-order"
    ],
    "plugins": [
        "stylelint-prettier",
        "stylelint-order",
        "stylelint-config-rational-order/plugin"
    ],
    "syntax": "scss",
    "processors": ["stylelint-processor-styled-components"],
    "rules": {
        "order/properties-order": [],
        "plugin/rational-order": true,
        "at-rule-empty-line-before": [
            "always",
            {
                "except": ["first-nested"],
                "ignoreAtRules": ["import"]
            }
        ],
        "block-closing-brace-newline-after": "always",
        "block-opening-brace-space-before": "always",
        "prettier/prettier": true,
        "property-no-vendor-prefix": true,
        "rule-empty-line-before": [
            "always",
            {
                "except": "first-nested"
            }
        ],
        "selector-no-vendor-prefix": true,
        "unit-case": "lower",
        "value-no-vendor-prefix": true
    }
}

Am I missing something?
Cheers!

Let me help...

First, you must abandon stylelint-processor-styled-components.
Autofix is new but that won't work with a processor. The Stylelint team intends to eventually depreciate the processor API now that the (built in) postcss-syntax is the preferred way to go.
Stylelint now has "css-in-js" syntax , so setting the syntax to "scss" is incorrect. In fact, you shouldn't explicitly set it in your config file... Stylelint will detect the syntax for you.
Autofix is still experimental. It is not something you can safely do on save. If you are using VSCode, the vscode-stylelint extension does not support this do to a list of issues with stylelint core.

So to be safe, prefer to save first and then run stylelint --fix. It will function if you remove syntax config and the processor.

Hey @brandonkal,
Thanks a lot for the detailed reply.
I did whatever you mentioned. Now everytime there is some error in my CSS-in-JS, the vscode-stylelint extension just throws an error like so:

If I remove stylelint-config-rational-order, this error goes away.
Here is my updated .stylelintrc
{
    "defaultSeverity": "warning",
    "extends": [
        "stylelint-config-recommended",
        "stylelint-order",
        "stylelint-config-rational-order"
    ],
    "plugins": [
        "stylelint-prettier",
        "stylelint-order",
        "stylelint-config-rational-order/plugin"
    ],
    "rules": {
        "order/properties-order": [],
        "plugin/rational-order": true,
        "at-rule-empty-line-before": [
            "always",
            {
                "except": ["first-nested"],
                "ignoreAtRules": ["import"]
            }
        ],
        "block-closing-brace-newline-after": "always",
        "block-opening-brace-space-before": "always",
        "prettier/prettier": true,
        "property-no-vendor-prefix": true,
        "rule-empty-line-before": [
            "always",
            {
                "except": "first-nested"
            }
        ],
        "selector-no-vendor-prefix": true,
        "unit-case": "lower",
        "value-no-vendor-prefix": true
    }
}

Yes. I ran into that as well. It is a bug in stylelint-config-rational-order. Removing that and configuring the plugin directly worked for me.
Also, the errors reported by rational-order contain an empty line. This is annoying but I have the to search for a fix. I'd also prefer these as warnings rather than errors. Hope that helps.

Hey @brandonkal,
Thanks a lot for the detailed comment. It is very annoying indeed.
Also, can you probably help me out, on how to configure stylelint-order without using stylelint-config-rational-order?

@samrith-s there is a vscode plugin may help you https://github.com/stuartZhang/vscode-stylelint

@samrith-s Just in case you are still looking for a solution, here is the relevant config that won't cause vscode-stylelint to throw. Note I am also setting the severity to warning as the css is still valid when out of order.
module.exports = {
  extends: ['stylelint-config-standard', 'stylelint-config-prettier'],
  plugins: ['stylelint-order', 'stylelint-config-rational-order/plugin'],
  rules: {
    // Property Order
    'order/properties-order': [[], { severity: 'warning' }],
    'plugin/rational-order': [
      true,
      {
        'border-in-box-model': false,
        'empty-line-between-groups': false,
        severity: 'warning',
      },
    ],
  },
}

That's exactly what I needed @brandonkal, huge thanks !

Thanks @brandonkal, linting works with your solution. However, I'm still not able to get css files to reorder on save in VSCode.

Thanks @brandonkal, linting works with your solution. However, I'm still not able to get css files to reorder on save in VSCode. Is this possible?

@kubradeck Reorder on save is possible with the forked stylelint plugin. I had to switch to it due to some other issues so I would recommend doing that. However, I personally don't run stylelint --fix on save and prefer to run it manually after staging changes in git.