Fixes incorrect order of calculate_iou_partly args and its outputs.

This issue might be helpful.
https://github.com/open-mmlab/OpenPCDet/issues/139

Yes, I know that overall, it's correct. But the way that wrong variables are passed as arguments and also unpack in the wrong order is confusing for someone new who is reading the code. This fixes the problem w/o making any changes to overlaps or parted_overlaps tensors.

Thanks for the PR! Have you checked the final evaluation performance? If so, we will merge this in.

Thanks. I just made another commit fixing the order of overlap_part and overlaps. To verify, I ran the evaluation on KITTI with the pre-trained PV-RCNN model and the following setup. The results before and after this PR are now exactly the same as follows:
Setup:

Ubuntu 22.04

CUDA 11.7

python==3.9.13

cudatoolkit==11.3.1

pytorch==1.10.1 py3.9_cuda11.3_cudnn8.2.0_0

spconv-cu113==2.1.21

pcdet==0.5.2+7e8bbe2

Results before and after this PR:
Car<EMAIL_ADDRESS>0.70, 0.70:
bbox AP:96.2470, 89.4992, 89.2430
bev  AP:90.0894, 87.9004, 87.4072
3d   AP:89.3476, 83.6901, 78.7028
aos  AP:96.22, 89.39, 89.07
Car<EMAIL_ADDRESS>0.70, 0.70:
bbox AP:98.2662, 94.4208, 92.2765
bev  AP:93.0239, 90.3255, 88.5319
3d   AP:92.1047, 84.3604, 82.4830
aos  AP:98.25, 94.26, 92.07

Pedestrian<EMAIL_ADDRESS>0.50, 0.50:
bbox AP:73.1477, 68.0799, 64.3542
bev  AP:65.1821, 59.4169, 54.5101
3d   AP:63.1230, 54.8428, 51.7816
aos  AP:67.84, 62.49, 58.73
Pedestrian<EMAIL_ADDRESS>0.50, 0.50:
bbox AP:73.6837, 68.2715, 64.3622
bev  AP:65.9365, 58.5166, 54.1258
3d   AP:62.7110, 54.4902, 49.8798
aos  AP:67.82, 62.17, 58.07

Cyclist<EMAIL_ADDRESS>0.50, 0.50:
bbox AP:96.1222, 81.3613, 76.4936
bev  AP:88.5292, 73.3251, 70.3690
3d   AP:86.0637, 69.4789, 64.5046
aos  AP:95.98, 81.07, 76.17
Cyclist<EMAIL_ADDRESS>0.50, 0.50:
bbox AP:97.1514, 82.4180, 78.2196
bev  AP:93.4584, 74.5322, 70.1025
3d   AP:89.1011, 70.3809, 66.0168
aos  AP:97.04, 82.12, 77.88

I notice that the performance is not exactly the same with the performance in README.md, like:
Car: 83.69 vs. 83.61
Pedestrian: 54.84 vs. 57.90
Cyclist: 69.47 vs. 70.47
Given this context, we still think it is risky to change this function since it has been utilized for a long time by lots of researchers.
If you think it is necessary, we prefer to add a comment in this function to say that the order will not affect the final performance. And we appreciate if you can add such a comment and change back this function to its original logic.

Yes, the results do not match the README.md but this has nothing to do with my changes. I got this results before my changes, and the results after my changes match with that. It might be due to the setup or other params that I got slightly different results before my changes. Anyway, I leave the code as it is and close the PR. Thanks.

Anyway, thanks for your contribution! I believe this PR will be very useful for the following users who have the same confusion as you (actually I was also confused when I saw this function at the first time).

Hi
@fnozarian
We are grateful for your efforts in helping improve this open-source project during your personal time.
Welcome to join OpenMMLab Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/UjgXkPWNqA
If you have a WeChat account，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤


First week player feedback

Lines we've received complaints about with suggested fixes.

Ported to https://github.com/Tsukihimates/Tsukihime-Translation/pull/238


"goose: not found" in migrations service when running separately from metadata service

I am trying to get the metadata service and migrations service set up as separate services. I built two separate images using Dockerfile.metadata_service and Dockerfile.migration_service on the 2.0.3 git tag. On doing a GET of /db_schema_status of the migrations service, I get
{"detail": "Exception('unable to get db version via goose: /bin/sh: 1: goose: not found\\n')"}

A cursory examination of the three dockerfiles (including the one that combines the two services) shows that the combined dockerfile installs goose but the individual dockerfiles do not. I also noticed libpq-dev in the combined dockerfile but not in the individual ones.

hey @LHCGreg thanks for bringing this up. The individual docker files are mostly meant for development purposes and not used as part of our prod deployment (publishing image to dockerhub). But I agree it does make sense to update Dockerfile.migration_service to include goose

the way we use goose changed quite a bit since this, closing the issue


bugfix: change payment method for PayPal plus

Without this option, users see the following in the payment method selection list:
http://dl3.joxi.net/drive/2019/07/09/0026/1778/1763058/58/24eb6ea214.jpg
https://www.paypalobjects.com/webstatic/ppplus/public/pages/en_US/notavailableerror.html    ( iframe URL )
The problem was that the "js file"   https://www.paypalobjects.com/webstatic/ppplus/ppplus.min.js
was tried to use "ApprovalUrl"  parameter.
http://dl3.joxi.net/drive/2019/07/09/0026/1778/1763058/58/25402b15b3.jpg
And since the parameter is missing, I received an error...
Аfter adding the missing parameter, everything worked.
http://dl4.joxi.net/drive/2019/07/09/0026/1778/1763058/58/5d7041931c.jpg
Shopware: 5.5.8
SwagPaymentPayPalUnified: 2.2.4

hey @bogusbd
thanks for your PR :+1:
That the parameter is missing there, was an intended change. It was introduced with this commit https://github.com/shopwareLabs/SwagPaymentPayPalUnified/commit/424b124c76422e21f5635498d0ee3f54006b94f3 and this ticket
The problem here was, if you have changed, e.g the shipping method, the page gets a partial reload and the iFrame is rendered new. But the iFrame got not the correct payment, which was created again for PayPal, due to the wrong approval-URL. Might not have effect on first sight, but you will see the problem, if you have different shipping costs.
So you should check your template and DOM and have look, if this line is present there: https://github.com/shopwareLabs/SwagPaymentPayPalUnified/blob/master/Resources/views/frontend/checkout/shipping_payment_core.tpl#L14
This might be the problem, that you got this error.
Kind regards
Michael Telgmann

Hello @mitelg
Yes, you are right, the original reason was in our template =/
Thanks for the help.


[WIP] Add 5.11

... because this is one of the more useful documents, still, evaaaaaah :)

Hi @chrisarcand! Thanks! I think ... I think these numbers should be right, but I spose you'll find out eventually if they're not :)

@d-m-u  👋 You gotta change the application code along with the README change you've already committed.

Whoa, a stray Jason was watching this.

🙇


Setup dialog / Windows spawn on wrong screen

When creating a new tile map, the setup dialog should be on the same screen as the application

These windows should spawn on the same screen where the application is positioned

Doesn't make sense, not needed


Spelling of Approval Reminder Email

Documentation inaccuracy
The Approval Reminder email has a misspelling of Provisioning:

Additional Info
That's it :)

@swallace74 I've fixed this in the latest V1 and V2 releases


Add support for custom footer text

Such that the user can change it to whatever they want, not just "Powered by Hugo and the Notepadium".
I'm thinking something like this:

    {{- $copyright := .Site.Copyright -}}
    {{- if not $copyright -}}
    {{- $copyright = .Site.Title | safeHTML -}}
    {{- end -}}

{{- $copyright | safeHTML -}}

{{- .Site.Params.footer | safeHTML -}}

{{- partial "math.html" . -}}
{{- partial "syntax.html" . -}}
{{- template "_internal/google_analytics_async.html" . -}}

Where  .Site.Params.footer defaults to
Powered by Hugo and the Notepadium

Good idea.
I think we should create user-side partial, named footer-bottom.html.

    {{- $copyright := .Site.Copyright -}}
    {{- if not $copyright -}}
    {{- $copyright = .Site.Title | safeHTML -}}
    {{- end -}}

{{- $copyright | safeHTML -}}

{{- partial "footer-bottom.html" . -}}

content of builtin footer-bottom.html :
Powered by Hugo and the Notepadium

Users can create their footer-bottom.html at YOURSITE/layouts/partials/footer-bottom.html

That's an even better idea. I can PR this if you want

Sure, thanks!

1

{{- partial "footer-bottom.html" . -}}

2

{{- partial "footer-bottom.html" . -}}

3
{{- partial "footer-bottom.html" . -}}

I don't know which is more flexible

@Benjamin-Lee
I made some changes, maybe more flexble.
https://github.com/cntrump/hugo-notepadium#custom-footer-copyright-layout

Tried it on https://lvv.me


[Windows] Unable to use raw string literal in cdef extern from statement

The path separator for Windows (\) needs escaping in the string notation, which we worked around using raw string literals r"<dir>\<file>".
A small example is at https://gist.github.com/tusharpm/ac0eef89796ca9c67ca92fe6c6071f6b.
In mypxd.pxd, there are two variants of cdef extern from:

(commented) works - using the escape sequence (\\).
(not commented) uses the r"<file_path>" notation which results in an error (attached Output.txt).

According to the code in
https://github.com/cython/cython/blob/0.27.x/Cython/Compiler/Parsing.py#L3069
and
https://github.com/cython/cython/blob/0.27.x/Cython/Compiler/Parsing.py#L870
this should actually be supported, although it's probably untested. Coming up with a cross-platform test for this doesn't seem easy...
Could you try to debug that code to see where it goes wrong for you? It should enable is_raw in your case and override kind with 'u', and that should be it.

In my code, I didn't specify the language_level=3 directive.
Attaching the debugger showed is_python3_source was False, and it went to _append_escape_sequence (from line 941) even when is_raw was set.
The problem I faced is resolved, so I'll close the issue.
Thanks!


Ignore default branch with 'push-to-registries' and 'push-to-app-catalog'

Towards https://github.com/giantswarm/giantswarm/issues/31824 -- see details there. Note: indentation and quoting changes may occur due to the automated handling. Sorry about that!

{APP-NAME} indicates that the repo hasn't been initialized correctly


allow middle clickable links

similar to https://github.com/filebrowser/filebrowser/issues/3374

added in this release https://github.com/gtsteffaniak/filebrowser/releases/tag/v0.2.9


[BUG] No search results for 'dialog.type'

Seems like the same cause or issue like in #48

Seems like the same cause or issue like in #48


ENH improve install of julia deps

This allows to easily install julia dependencies from a github repo, using:
class Solver(JuliaSolver):

    julia_requirements = ["pkg_name::https://url.to/package#branch"]

tested for the following benchmark: benchopt/benchmark_logreg_l2#29

Codecov Report

Merging #343 (937d2d4) into main (7683e15) will increase coverage by 0.01%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             main     #343      +/-   ##
==========================================
+ Coverage   55.55%   55.57%   +0.01%
==========================================
  Files          42       42
  Lines        2403     2404       +1
  Branches      443      443
==========================================
+ Hits         1335     1336       +1
  Misses        981      981
  Partials       87       87


more tests

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

FYI @bgruening as this was pushed to a branch on the feedstock this has already deployed a package. Would recommend closing it out and making a new PR from a branch on your fork of the feedstock. Will need an additional build number bump to get a fresh package build.

Even if the branch was never build successfully? Didn't know that. Sorry.

Mh, I don't see it in the channel.

Good point, if it was always failing, it wouldn't have deployed.


Extending snmpfwd into a caching proxy?

SNMP polling is great - until every man and his canine companion is polling your devices every 30 seconds.
That makes your devices very sad and some devices get much sadder than others (especially network switches, which may spend so much time handling snmp requests that they forget to do their primary job - switching network packets)
It would be great to have snmpfwd  able to cache results for a preset (configurable?) period so that when there are a bunch of programs polling a device for the same information in a short period of time it can feed the cached answers back to the pollers instead of killing the device.

Thanks for rising this! Caching may be very reasonable.
In fact, the required infrastructure is hopefully already in place. If you consider working on this, I imagine you could come up with a plugin Python module which would define two functions to be called on passing SNMP request and response. This is where you could read from / populate cache.
I'd happily merge your caching module if you PR it.
I may be able to work on this, however the priority of this work is currently not at the very top. ;-)


 Could you describe what's exactly your problem?
Are the commands you posted not working? Id so, what's the issue you're faced with?

Thanks, Carlos.
I was able to perform the update by trying
different ideas not provided in the video.
I have resolved the "Python update" issue.
Carlos, I have no Python experience other
than the class of "Python for Engineers"
and therefore, I have one question:
"How do you 'print' two parameters as
two separate columns to allow me to take
the output information as 2 columns into
Excel for plotting y versus x. For instance,
here are two temperatures columns data in
format ###.# where the symbol # stands
for "decimal place":
T_C          T_F
0.0        32.0
100.0      212.0
I will guess that the Python code would look
as follows:

print()
print("T_C          T_F")
print.format(###.#      ###.#) T_C_Data,T_F_Data
print()

Carlos, the symbol "__" is three instances of the
single symbol "" intended to indicate that it is a
blank line not intended to be part of the Python
statrements being sough as the "guessed response"
to my inquiry.
Carlos, I have experience in Fortran and Mathcad Prime
software which may serve you in the preparation of your
response this "Python print" commands inquiry. Also, I
have a Master of Science in Mechanical Engineering with
interest in thermo-structural analysis and desgn of turbo-
machineries.
Thanks,
Julio C. Banks, BSMET, MSME, PE
Solutions Consultant
"Doing Business by the Book"
Port Saint Lucie, Florida USA
e-mail<EMAIL_ADDRESS>P Please consider the environment before printing this email.
"Education is a better safeguard of liberty than a standing army"
Edward Everett

 From: Carlos Cordoba < ***<EMAIL_ADDRESS>>
  Sent: Monday, May 27, 2024, 9:32 AM
     To: spyder-ide/spyder < ***<EMAIL_ADDRESS>>
     Cc: Saint-Julio < ***<EMAIL_ADDRESS>>; Mention < ***<EMAIL_ADDRESS>>

Subject: [spyder-ide/spyder] Update to spyder=5.5.1 (Issue #22122)
Hey @Saint-Juliohttps://github.com/Saint-Julio, thanks for reporting.
Could you describe what's exactly your problem?
Are the commands you posted not working?
If so, what's the issue you're faced with?

Reply to this email directly, view it on GitHubhttps://github.com/spyder-ide/spyder/issues/22122#issuecomment-2133495641, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BIW6BZDBQKKOAJCUSSJIWELZEMYW3AVCNFSM6AAAAABIIPGRKWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZTGQ4TKNRUGE.
You are receiving this because you were mentioned.
Message ID<EMAIL_ADDRESS>

I have resolved the "Python update" issue.

Ok, that's great!

Carlos, I have no Python experience other
than the class of "Python for Engineers"
and therefore, I have one question:

I'd love to help you with that but, unfortunately, I'm very busy with other things, sorry.


add missing github token for the release

Signed-off-by: Augustin Husson<EMAIL_ADDRESS>

:+1:


Removed data redundancy from features list.

Thanks!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


document of lib_custom_op support gpu without rebuilding mxnet

Description
I want to create a lib_custom_op in c++ mxnet-backend without rebuilding the whole mxnet. In example/extensions/lib_custom_op/makefile, I didn't see the nvcc compiler and the source code sets the CPU device. Does it support GPU?

Can we update the document of custom op.

References

https://github.com/apache/incubator-mxnet/tree/master/example/extensions/lib_custom_op
https://mxnet.apache.org/api/python/docs/api/mxnet/operator/index.html#mxnet.operator.CustomOp

@a550461053 The GPU support for customOps is finally ready thanks to all the hard work from @rondogency !!! Yay! You can get a nightly build or build from source with the latest master to try out the functionality.
@rondogency is also working on the documentation updates for GPU support in #17486 too, so you can review the latest there an leave comments if there is something we can clarify.

@a550461053 The GPU support for customOps is finally ready thanks to all the hard work from @rondogency !!! Yay! You can get a nightly build or build from source with the latest master to try out the functionality.
@rondogency is also working on the documentation updates for GPU support in #17486 too, so you can review the latest there an leave comments if there is something we can clarify.

Ok, I will try it and this issue can be closed.


Fix notifysend check when "which" is unavailable

Recently ran into this problem when building my application into a
docker container with a minimal environment.  The which utility
wasn't available to my application, so the return value of Sheller's
stdout call was nil, which caused the empty? check to blow up.
This adds in a nil check as well.

Awesome, thanks!

Released as https://github.com/guard/notiffany/releases/tag/v0.1.1


Fix for ALS-362 / ALS-427

scope filtering and performance improvement on data download tree.
ALS-427
ALS-362

missed something checking for undefined


added render method to ArcBuffer

ArcBuffer also missing a render method that GridBuffer has that I though was very useful

:+1:


[25] Start work on issue #100

Start work on the reverse nodes in k-group

Thanks, glad to have helped!


Incorrect variable name in Step 6 - substep: edit the Windows 10 task sequence

In Step 6 "Create the deployment task sequence", under substep "edit the Windows 10 task sequence", the text instructions in Step 2-a-iii specify to set DriverGroup001 to Windows 10 x64%Manufacturer%%Model%
%Manufacturer% is the incorrect variable name.  The variable name for the manufacturer name is %Make%.  This is shown correctly farther down the page in the figure "The task sequence for production deployment".  However, I was very frustrated for 2 days trying to get this to work before I realized it.
Hope someone can correct this for the next poor knucklehead to try to learn MDT.
I guess this error is understandable because the command find the manufacturer name is wmic computersystem get manufacturer.

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

ID: 1c66f2a3-5be3-e3cf-b90f-c9ce2f6ecf9d
Version Independent ID: c9ed7f55-3770-73a4-5ee4-deee16eb7600
Content: Deploy a Windows 10 image using MDT (Windows 10) - Windows Deployment
Content Source: windows/deployment/deploy-windows-mdt/deploy-a-windows-10-image-using-mdt.md
Product: w10
Technology: windows
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@dcdorsey777 . Thanks for the information. I will correct those mistakes

This does not appear to be corrected.  What am I missing?  The error still appears in the document at: https://docs.microsoft.com/en-us/windows/deployment/deploy-windows-mdt/deploy-a-windows-10-image-using-mdt#edit-the-windows-10-task-sequence
I have cleared my browser cache.  Is there another step I need to take to see the updated document?
@.***
Dave Dorsey
PCFD
From: Denise Vangel-MSFT @.>
Sent: Friday, September 17, 2021 12:02 PM
To: MicrosoftDocs/windows-itpro-docs @.>
Cc: David Dorsey @.>; Mention @.>
Subject: Re: [MicrosoftDocs/windows-itpro-docs] Incorrect variable name in Step 6 - substep: edit the Windows 10 task sequence (#9894)
Closed #9894https://github.com/MicrosoftDocs/windows-itpro-docs/issues/9894.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/issues/9894#event-5320851649, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AU5K55YX2EAVGFDFXTNUOX3UCN7BDANCNFSM5CCQC3IA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Revert "Created GitHub action to update site pagespeed svg"

Reverts alshedivat/al-folio#1529

I was going though PRs and noticed that we reverted #1529 but never added it back. what was the issue with it?

I created #1529 to test the GitHub action because I could not find a way to test it in isolation. But then I realized that the heroku app it uses is no longer online. I detailed a possible solution here.


Add links to UserGuide

Added links to tables sections.
See heyzec.github.io/tp/UserGuide.html to preview.

Codecov Report
Base: 54.80% // Head: 54.80% // No change to project coverage :thumbsup:

Coverage data is based on head (3c84dcb) compared to base (72377ef).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #200   +/-   ##
=========================================
  Coverage     54.80%   54.80%
  Complexity      730      730
=========================================
  Files           159      159
  Lines          3202     3202
  Branches        394      394
=========================================
  Hits           1755     1755
  Misses         1292     1292
  Partials        155      155

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


"Bunești" commune is listed in the drop-down menu

Description:
The commune is listed in the drop-down menu according to the law
Preconditions: the website (https://peviitor-ro.github.io/ui.orase/) is up and runing.
After every step delete the text you typed from the search bar.

Step 1
Type "Bunești" in the search bar.
Expected results
The location is listed in the drop-down menu as "Bunești, Brașov"


We should be able to publish without packing

When using only lint + publish and a custom "orb-dir" like :
orb-dir: ./src
File can never be found. Like other jobs we should let the user interact with the code and checkout it.
At the moment having the command with "orb-dir: ./src" produces:
  Error: Could not load config file at ./src/orb.yml: open ./src/orb.yml: no such file or directory

+1 for this. There are enough cases when orb is simple enough and doesn't require splitting it into complex structure. A plain orb.yml is enough.
What I suggest is that orb-tools/publish when detects orb.yml (instead of @orb.yml) fall back to orb-tools/validate with the corresponding log message.

Hey folks,
Could we see your config? The orb wasn't technically designed for this use-case, but it should still work. Happy to take a look.
If you look here: https://github.com/CircleCI-Public/orb-tools-orb/blob/c14c72a5d7ec98ac0d868b551ebbb64966cb2c03/src/scripts/publish.sh#L24
The publish job is only looking for an orb.yml file within a directory, where it comes from should not be relevant. So I believe this should work, with the proper config.

closing as abandoned


World height of Town 01 has incorrect scale

It seems that simulator sends height in centimeters for Town01.
According to Town01.txt world shift for Z axis is -0.3811 (in meters, I suppose), and for Town02.txt it is -0.22. For Town02 the actual height of the objects is the same as in the corresponding text file (without minus sign), but for Town01 all objects are located about 38 meters above the ground.
Related to #537.

I changed to 0.38 in the files after we changed the reference to meters.
Are objects really located at 38 meters above the ground  @nsubiron  @TheNihilisticRobot  ?

Hey @danil-tolkachev.
Did these answers solved your question?

I also think is inconsistent. I will try to solve it and I will check your PR @danil-tolkachev
Thanks a lot !

@felipecode Oh, my PR is only related to this issue and not fixes it.

This was corrected.


Added ColorPicker unit tests to CI and moved the unit test project

Summary of the Pull Request
What is this about?
This PR moves the ColorPickerUI unit test project to src\modules\colorPicker and enables the tests on CI.
PR Checklist

[X] Applies to #7463
[X] CLA signed. If not, go over here and sign the CLA
[X] Tests passed

Info on Pull Request
What does this include?
Validation Steps Performed
How does someone test & validate?
Run tests locally and on CI. Validated that tests run and pass on CI https://dev.azure.com/ms/PowerToys/_build/results?buildId=117703&view=results

associated CI: https://dev.azure.com/ms/PowerToys/_build/results?buildId=117703&view=results

@arjunbalgovind this passed. Want me to force merge?

Yeah that would be good @crutkas. Might have to resolve merge conflicts on #7483 after this

I merged in the other first ...  can you resolve?

Made a change to the output path since it was set to the default path (within the project folder). Can merge after validating tests are run on https://dev.azure.com/ms/PowerToys/_build/results?buildId=117762&view=results

Successful

@arjunbalgovind @crutkas
the ColorPickerUI tests are not built by default, is there a reason for that?
From the VS log:
56>------ Skipped Build: Project: UnitTest-ColorPickerUI ------


istio-cni - privilege container

istio-cni - add privilege to allow for selinux with istio-cni.
The istio-cni provider will fail to write to /host/etc/cni/net.d/10-aws.conflist.tmp: Permission denied.
k -n kube-system logs istio-cni-node-hlzzd -c install-cni
/host/opt/cni/bin is non-writable, skipping
/host/secondary-bin-dir is non-writable, skipping
/install-cni.sh: line 185: /host/etc/cni/net.d/istio-cni.conf.tmp: Permission denied
Cleaning up and exiting.
Removing istio-cni config from CNI chain config in /host/etc/cni/net.d/10-aws.conflist
/install-cni.sh: line 110: /host/etc/cni/net.d/10-aws.conflist.tmp: Permission denied

# ---

k -n kube-system logs istio-cni-node-hlzzd -c repair-cni
{"level":"info","time":"2021-04-21T15:26:09.172568Z","msg":"Controller Option: Running as a Daemon."}
{"level":"info","time":"2021-04-21T15:26:09.172613Z","msg":"Controller Option: Deleting broken pods. Pod Labeling deactivated."}
{"level":"info","time":"2021-04-21T15:26:09.172620Z","msg":"Filter option: Only managing pods with an annotation with key sidecar.istio.io/status"}
{"level":"info","time":"2021-04-21T15:26:09.172625Z","msg":"Filter option: Only managing pods with field selector spec.nodeName=ip-10-0-49-241.eu-west-1.compute.internal,"}
{"level":"info","time":"2021-04-21T15:26:09.172629Z","msg":"Filter option: Only managing pods where init container is named istio-validation"}
{"level":"info","time":"2021-04-21T15:26:09.172633Z","msg":"Filter option: Only managing pods where init container exit status is 126"}

[X] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

Hi @ajohnstone. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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.

@howardjohn thanks updated the description with the details.
k -n kube-system logs istio-cni-node-hlzzd -c install-cni
/host/opt/cni/bin is non-writable, skipping
/host/secondary-bin-dir is non-writable, skipping
/install-cni.sh: line 185: /host/etc/cni/net.d/istio-cni.conf.tmp: Permission denied
Cleaning up and exiting.
Removing istio-cni config from CNI chain config in /host/etc/cni/net.d/10-aws.conflist
/install-cni.sh: line 110: /host/etc/cni/net.d/10-aws.conflist.tmp: Permission denied

@bianpengyuan would it make sense to integrate that overlay here: https://github.com/GoogleCloudPlatform/anthos-service-mesh-packages/blob/master/asm/istio/options/cni-gcp.yaml
??
I also wanted to ask, is there any case where it would make sense to configure in the values privileged: false in this proposed patch?
I would rather suggest to patch the DaemonSet with a default value for securityContext that does not need a new value option.
The securityContext we had in the istio-init container when not using istio-cni was like this:
    securityContext:
      allowPrivilegeEscalation: false
      capabilities:
        add:
        - NET_ADMIN
        - NET_RAW
        drop:
        - ALL
      privileged: false
      readOnlyRootFilesystem: false
      runAsGroup: 0
      runAsNonRoot: false
      runAsUser: 0

Would this be enough to write those files that fail now with permissions errors ?

@bianpengyuan Unfortunately using istio asm 1.6. That works great in 1.9 and will be upgrading to soon. Wasn't aware of overlays in istio :)
@zioproto
To get it to work with selinux need to modify the securityOptions.

Make privileged by default without it being an option
Can instruct users in docs to patch as per @bianpengyuan advice, no change.
Unsure how https://github.com/GoogleCloudPlatform/anthos-service-mesh-packages/blob/master/asm/istio/options/cni-gcp.yaml or where this fits into istio asm?

I solely made an option if people needed to disable, let me know either way and happy to update.

Since it works within most environment as unprivileged container, I would prefer to keep CNI default configuration as is, treat SELinux as a special case, and document how to run CNI on it with overlays configuration (we are working on CNI beta graduation at Istio 1.11 release and one of the major tasks is better document).

Since it works within most environment as unprivileged container, I would prefer to keep CNI default configuration as is, treat SELinux as a special case, and document how to run CNI on it with overlays configuration (we are working on CNI beta graduation at Istio 1.11 release and one of the major tasks is better document).

I just wanted to clarify:

SELinux requires privileged
Everything else does not

is that accurate? If so, +1 to what you suggested

Since it works within most environment as unprivileged container, I would prefer to keep CNI default configuration as is, treat SELinux as a special case, and document how to run CNI on it with overlays configuration (we are working on CNI beta graduation at Istio 1.11 release and one of the major tasks is better document).

I just wanted to clarify:

SELinux requires privileged
Everything else does not

is that accurate? If so, +1 to what you suggested

Yeah that is what I am thinking, since so far people only request for this when running on SELinux. I can do a bit more research on this but at least for now I don't think this worth a value option.

The following patch resolves the issue with a overlay working in istio 1.6 and greater.
  components:
    cni:
      enabled: true
      namespace: kube-system
      k8s:
        overlays:
        - kind: DaemonSet
          name: istio-cni-node
          patches:
            - path: spec.template.spec.containers.[name:install-cni].securityContext
              value: {"privileged": true}


unable to create Orgnasitation or logout

Iall done without any Error , successfully login to webhook , URL : https://mouqaa.com/
but not able to unable to create Orgnasitation or logout ...
it give error following error : -

[web.1:2296] [2019-03-05T19:11:35.616] INFO -- : Started POST "/organizations" for <IP_ADDRESS> at 2019-03-05 19:11:35 +0000
[web.1:2296] [2019-03-05T19:11:35.617] INFO -- : Processing by OrganizationsController#create as HTML
[web.1:2296] [2019-03-05T19:11:35.617] INFO -- :   Parameters: {"utf8"=>"✓", "organization"=>{"name"=>"a1", "permalink"=>"e"}, "commit"=>"Create organization"}
[web.1:2296] [2019-03-05T19:11:35.626] WARN -- : Can't verify CSRF token authenticity.
[web.1:2296] [2019-03-05T19:11:35.629] INFO -- : Completed 422 Unprocessable Entity in 11ms (ActiveRecord: 2.8ms)
[web.1:2296] [2019-03-05T19:11:35.631] FATAL -- :
[web.1:2296] [2019-03-05T19:11:35.632] FATAL -- : ActionController::InvalidAuthenticityToken (ActionController::InvalidAuthenticityToken):
[web.1:2296] [2019-03-05T19:11:35.632] FATAL -- :
[web.1:2296] [2019-03-05T19:11:35.632] FATAL -- : vendor/bundle/ruby/2.3.0/gems/actionpack-5.2.1/lib/action_controller/metal/request_forgery_protection.rb:211:in handle_unverified_request' vendor/bundle/ruby/2.3.0/gems/actionpack-5.2.1/lib/action_controller/metal/request_forgery_protection.rb:243:in handle_unverified_request'
vendor/bundle/ruby/2.3.0/gems/actionpack-5.2.1/lib/action_controller/metal/request_forgery_protection.rb:238:in verify_authenticity_token' vendor/bundle/ruby/2.3.0/gems/activesupport-5.2.1/lib/active_support/callbacks.rb:426:in block in make_lambda'
vendor/bundle/ruby/2.3.0/gems/activesupport-5.2.1/lib/active_support/callbacks.rb:198:in `block (2 levels) in halting'
i follow you previous post #158 but nothing help ...
Please suggest !!
Thank You.

Hello,
Firstly, your Postal login screen shouldn't be naked. That means something is wrong with your installation or config file or DB or some files there.
Thanks

What do you meant about naked ?
i am able to login this ?

For some reason, your Postal installation is not loading its page assets like CSS, images, and javascript.
This implies your install of Postal did not succeed fully or something has since broken and it is unable to load various pages/files.
Are you able to reinstall it?

strange ... for whole installation i did not any error !
BTW is it might be due to nginx reconfiguration ?
Yes , i can reinstall it , even i want to install it with my all 15 servers .... so me try for another one !

Is it work for apache instead of nginx ?

It is possible your nginx server block is incorrect which might explain the missing assets but the error you initially encountered suggests something is wrong in the Ruby.
You should be able to backup and restore or reuse your database without remaking all your servers.

i follow all steps of link https://github.com/atech/postal/wiki/Installation ...
without any error ... not sure where i am wrong ..
just installed another server but its same result ....

@martandya
Try this installation first https://github.com/atech/postal/wiki/Quick-Install
Let's see if it would work fine.
Just try it, if same error then we would know what next step to take.
Thanks

@saporus
its working now ...perfect !
Many Thanks


Added description of updates contained in release 4.2.c to the 5.0 documentation.

Pretty self-explanatory if you look at the code changes.  Only affects docs/Development/ReleaseNotes.md.
closes #351

just one comment, fix then I'll merge.

OK, I've updated the release dates - can you have a look?

Removed tabs...

Okay, looks good.  Merging now.


suggestion: document workspaces functionality

This is important as the Deno Standard Library will become a workspaces-enabled codebase once it is fully ported to JSR. Documenting how workspaces work will be important to those maintaining and contributing to it.
Some documentation on workspaces functionality can be seen in #245.

Closed by https://github.com/denoland/deno-docs/pull/567


plugin

i could not able to find the   plugin" add coordinates to point"

@Chandumouli21, where did you find a reference to such "add coordinates to point" plugin? Have you tried to search in the QGIS Python Plugins Repository at https://plugins.qgis.org/? What should this plugin actually do in details?

I think this issue report should be closed.


How to change mage level?

How to change mage level???

There aren't really levels in Magic- have you read through the docs?
https://github.com/elBukkit/MagicPlugin/wiki/Progression


Remove whitespaces as default text

The line break after the <textarea> opening tag in the HTML seems to introduce the indent before the closing tag as default text for the request message.
Before:

After:

Outdated


upstream/newversion

You might have pressed the wrong button in VSCode to create this pull request. You want to merge this locally, and push to your own fork of the repository.


Tally doctests alongside ordinary unit tests.  Fixes #109.

With @travs:

Created a proper test suite from README doctests
The total number of tests does not change, but this is a step towards exposing individual tests with the new breakdown of test.py

@travs is the problem here that the Readme file contains tests that we are expecting to fail, and now that they are part of the test suite, they are causing the build to fail?

Looks like it.  Add an expected failure tag somehow and create a new issue for removing it once they are fixed?

Hmmm. this looks tricky.  You can add a skip or expectedFailure to an individual test, but it's not clear to me how to do that for an entire test suite.

Hmm. this might be tricky to do since the doctests are an entire suite, and the individual tests are not accessible.  Will write back once I've figured it out.  Some relevant reading for anyone interested:
http://stackoverflow.com/questions/12487532/how-do-i-skip-a-whole-python-unittest-module-at-run-time

Interestingly, the way it was done before, we could have simply added @unittest.expectedFailure above the definition for the doctests.

Great!


--testFailureExitCode

is there a way to restart AppVeyor build?

Codecov Report

Merging #4067 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4067      +/-   ##
==========================================
+ Coverage   60.41%   60.42%   +<.01%
==========================================
  Files         196      196
  Lines        6766     6767       +1
  Branches        6        6
==========================================
+ Hits         4088     4089       +1
  Misses       2675     2675
  Partials        3        3

Impacted Files
Coverage Δ

packages/jest-config/src/index.js
0% <ø> (ø)
:arrow_up:

packages/jest-config/src/valid_config.js
100% <ø> (ø)
:arrow_up:

packages/jest-config/src/defaults.js
85.71% <ø> (ø)
:arrow_up:

packages/jest-config/src/normalize.js
81.08% <100%> (+0.1%)
:arrow_up:

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 22f6235...5ba5f3b. Read the comment docs.


Eth contract: bug with hooks, but not with the other syntax (ethContract.approve(...) for example)

  const { data:approveData, approveLoading, error:approveError, reset:approveReset, invoke:approve } = useStarknetInvoke({ ethContract, method: 'approve'})

  // It doesn't work
  useEffect(() => {
    ethContract && approve({ args: ['0x6806c42960e739918af543b733e76eb4f52a99402ec00e57794cb26cb3a6723', [100000000, 0]] })
  }, [ethContract, approve])
  // It doesn't work
  ethContract && approve({ args: ['0x6806c42960e739918af543b733e76eb4f52a99402ec00e57794cb26cb3a6723', [100000000, 0]] })

  // It works
  ethContract && ethContract.approve('0x6806c42960e739918af543b733e76eb4f52a99402ec00e57794cb26cb3a6723', [100000000, 0])

(when it doesn't work, absolutely nothing happens, no error)
Here is the eth contract : https://goerli.voyager.online/contract/0x4bb0f7af15a5469be4a571c8660d9aaf39e7e67f27f774702d75e77141ef90#writeContract

Here is the eth contract ABI :
https://pastebin.com/YxyfivFr
I also tried :

This should be fixed in v2. If not, feel free to reopen.


New funcs place market/limit order returns order filled events.

Description
Describe the changes or additions included in this PR.
Added new functions for place market/limit/swap. New function returns vector of OrderFilled events which provides on-chain information about which orders were matched.
Test Plan
How did you test the new or updated feature?

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)

[ ] protocol change
[ ] 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

Hi, thank you for your efforts. We are pausing development on DeepBook and will include these new features in DeepBook v3: https://github.com/MystenLabs/deepbookv3


File Encoding

Hi,
thank you very much, that you have been able to publish this firmware.
I just had a look into it, it compiled without any problems!
Is there any plan, that the file encoding is translated into the standard UTF-8 format?
Greets from Austria,
Karl

PS: I could do the translation, if I know, the pull request gets merged.

Hi Karl,
please make the changes. I will accept your pull requests, of Course.
kind regards
Mathias


Update maintainers.yml

Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/india/blob/main/CONTRIBUTING.md.

Which change are you proposing?

[ ] Adding/updating open source project/s
[x] Adding/updating maintainer/s

Adding/updating open source project/s

[x] The project was built in or receives significant contributions from India
[x] The project has at least 100 stargazers

Adding/updating maintainer/s

[x] Maintainer's location on their GitHub profile indicates as India
[x] Maintainer has approved this PR to be included on the website

What open source project/s does the individual maintain?

https://github.com/powerexploit/Ashok
https://github.com/powerexploit/Powerexploit
https://github.com/powerexploit/Awesome-Python-Scripts
https://github.com/powerexploit/Awesome-Penetration-Scripts

@mvkaran can you review this PR?

Thanks for your contribution @powerexploit 💙
It's great to see all the work you are doing for the security community and the open source community! For this listing, we are providing an opportunity to maintainers of open source projects with significant traction in the community. I am confident I can see you soon there, at which point we'd be happy to feature you on this list.
Until then, all the best in your open source journey!


Lazy sequences and printing seem to get mixed up

I've been seeing :out values in my lazy-sequence :ret if I (prn ...) from inside them. I think this is a Clojure bug but I've got to prove it first. I don't think this is a Conjure bug but I'm raising it here for visibility.

It's a bug in Clojure, might give patching it a go at some point.


Use ARM to Deploy Managed VNet

Can you please confirm if ARM can be used to deploy managed vnet?
Do you a example or document?
Looks like rest API is documented https://docs.microsoft.com/en-us/rest/api/datafactory/managedvirtualnetworks/createorupdate
But ARM manual, I don't see this resource type,  https://docs.microsoft.com/en-us/azure/templates/microsoft.datafactory/allversions
[Enter feedback here]

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

ID: 0b7d5707-9f14-db2b-00fb-5edd7bbcd06f
Version Independent ID: 2a0f8297-bdf7-3056-121e-963140ec0703
Content: Managed virtual network & managed private endpoints - Azure Data Factory
Content Source: articles/data-factory/managed-virtual-network-private-endpoint.md
Service: data-factory
GitHub Login: @nabhishek
Microsoft Alias: abnarain

There is one incomplete arm example https://docs.microsoft.com/en-us/azure/data-factory/continuous-integration-deployment#example-parameterizing-an-existing-azure-databricks-interactive-cluster-id
showing resource type  "Microsoft.DataFactory/factories/managedVirtualNetworks/managedPrivateEndpoints"
so wondering if you have a document or example for Microsoft.DataFactory/factories/managedVirtualNetworks resource type

@RonnieQuan - Thanks for posting the query. I am looking into it and will update you shortly.

@nabhishek - Can you please take a look and share your thoughts.

Really appreciate your guidance, please respond at your earliest convenience.

any update on this?

any update on this?

Hi @nabhishek - Could you please have a look at this ask from Cx and do the helpful needed. Thank you

Hi @nabhishek - Could you please have a look at this ask from Cx and do the helpful needed. Thank you

I've been using bicep to create an ADF endpoint so that I can automate the creation of a private endpoint for accessing a data lake. Part of that requires creating an ADF managed network. The following is the compiled output which is working fine for me.
{
    "type": "Microsoft.DataFactory/factories",
    "apiVersion": "2018-06-01",
    "name": "[variables('adfName')]",
    "location": "[variables('location')]",
    "identity": {
        "type": "SystemAssigned"
    }
},
{
    "type": "Microsoft.DataFactory/factories/managedVirtualNetworks",
    "apiVersion": "2018-06-01",
    "name": "[format('{0}/{1}', variables('adfName'), 'default')]",
    "properties": {},
    "dependsOn": [
        "[resourceId('Microsoft.DataFactory/factories', variables('adfName'))]"
    ]
}

Thanks for reporting this @RonnieQuan, and your contribution to improve our docs.  This is a current work-in-progress, so it may be included in the future.  For now I will close out this issue but if you have additional questions please feel free to continue the discussion here.  #please-close


ISPN-1632 use xsddoc for schema doc

This is part 1, to actually generate the doc. Part 2 is to fill in more of the xsd doc, which is to come. But want to get this reviewed and merged in in parallel :-D

Thanks!


[Question] Efficient shortcut for unstacking only parts of dimension?

Hi all, is there an efficient way to unstack only parts of a MultiIndex?
Consider for example the following array:
<xarray.Dataset>
Dimensions:                      (observations: 17525)
Coordinates:
  * observations                 (observations) MultiIndex
  - subtissue                    (observations) object 'Skin_Sun_Exposed_Lower_leg' ... 'Thyroid'
  - individual                   (observations) object 'GTEX-111FC' ... 'GTEX-ZZPU'
  - gene                         (observations) object 'ENSG00000140400' ... 'ENSG00000174233'
  - end                          (observations) object '5' '5' '5' ... '3' '3'
Data variables:
    fraser_min_pval              (observations) float64 dask.array<chunksize=(17525,), meta=np.ndarray>
    fraser_min_minus_log10_pval  (observations) float64 dask.array<chunksize=(17525,), meta=np.ndarray>

Here, I have a MultiIndex observations=["subtissue", "individual", "gene", "end"].
However, I would like to have end in its own dimension.
Currently, I have to do the following to solve this issue:
xrds.unstack("observations").stack(observations=["subtissue", "individual", "gene",])

However, this seems quite inefficient and introduces NaN's.
 We'd take a PR though.


Don't skip the handling and forwarding of 404 responses with content …

…to downstream WebServer filters (oracle#390)

Only skip the handling and forwarding of 404 responses in io.helidon.webserver.jersey.ResponseWriter if there is no content

@Holubow You will to sign the OCA for us to accept your contribution. See https://www.oracle.com/technetwork/community/oca-486395.html

@romain-grecourt, I completed and submitted the OCA


Add 2 DPS users

Adding 2 more DPS users to test the integration.

LGTM!  Found the new users. . .


SPFx webparts on OnPrem 2016 intermittently fails with Error "Failed to render client side web part"

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Issue: In OnPrem SP 2016, SPFx webparts developed using SPFx-1.4.1, intermittently fails with below error:
ULS Log Errror:
03/07/2019 17:37:38.03 w3wp.exe (0x12B0) 0x1FA4 SharePoint Foundation General blde6 Medium Manifest {44DD20AD-349C-4394-B306-BEED5093E55C} not returned because feature 44dd20ad-349c-4394-b306-beed5093e55c not enabled aec7c69e-810d-5031-4dc4-1fddd88a8c70
Component '{44DD20AD-349C-4394-B306-BEED5093E55C}' doesn't have manifest. Skipping.
Failed to render client side web part. Id: [44dd20ad-349c-4394-b306-beed5093e55c], Manifest: [], Error Details:: [Microsoft.SharePoint.ClientSideComponent.SPClientSideFrameworkException: Web Part 44dd20ad-349c-4394-b306-beed5093e55c failed to render as manifest is not found
Removing the app from the site and re-adding it again from corp catalog or from the recycle bin fixes the issue and the page will work for couple of weeks and it will break with same error/stack.
Found similar issue on SPO - https://github.com/SharePoint/sp-dev-docs/issues/2764 which seems to be fix. Is this fix in latest SPFx or in SP 2016 CU?
Expected or Desired Behavior
Once the app is installed, the SPFx parts added to the Page. Those webparts should render without any exception.
Observed Behavior
We see below error in developer tools. The SPFx webparts are not rendered on the page
Failed to render client side web part. Id: [44dd20ad-349c-4394-b306-beed5093e55c], Manifest: [], Error Details:: [Microsoft.SharePoint.ClientSideComponent.SPClientSideFrameworkException: Web Part 44dd20ad-349c-4394-b306-beed5093e55c failed to render as manifest is not found
at Microsoft.SharePoint.WebPartPages.ClientSideWebPart.GetManifest[TManifest](Guid componentId, String& manifestJson)
at Microsoft.SharePoint.WebPartPages.ClientSideWebPart.<>c__DisplayClass4.b__0()
at Microsoft.SharePoint.ClientSideComponent.SPClientSideFrameworkReliabilityHelper.ExecuteInReliabilityMonitoredScope[TException](Action userCodeDelegate, SPReliabilityMonitor monitor, UInt32 expectedFailureTag, Action`1 customErrorHandlerDelegate, Boolean reThrow)]
Web Part framework is not loaded.
Please assist. Thanks.

Hi I am also getting intermittent error on SP2016 classic
I can fix by uninstalling and reinstalling in the site collection.
This is the 4th time it has happened and I need to get this fixed!
Failed to render client side web part. Id: [xxxxxx-xxxx-xxxx-xxxxxx], Manifest: [], Error Details:: [Microsoft.SharePoint.ClientSideComponent.SPClientSideFrameworkException: Web Part xxxxxx-xxxx-xxxx-xxxxxx failed to render as manifest is not found
at Microsoft.SharePoint.WebPartPages.ClientSideWebPart.GetManifest[TManifest](Guid componentId, String& manifestJson)
at Microsoft.SharePoint.WebPartPages.ClientSideWebPart.<>c__DisplayClass4.b__0()
at Microsoft.SharePoint.ClientSideComponent.SPClientSideFrameworkReliabilityHelper.ExecuteInReliabilityMonitoredScope[TException](Action userCodeDelegate, SPReliabilityMonitor monitor, UInt32 expectedFailureTag, Action`1 customErrorHandlerDelegate, Boolean reThrow)] default.aspx:635:29
cswp_WPQ7 https://intranet/sites/asite/Pages/default.aspx:635
 https://intranet/sites/asite/Pages/default.aspx:628
Web Part framework is not loaded. default.aspx:636:29
cswp_WPQ7 https://intranet/sites/asite/Pages/default.aspx:636
 https://intranet/sites/asite/Pages/default.aspx:628

We are also experiencing this issue

@VesaJuvonen This issue was fixed in SPO but still there in 2016 FP2. We may need a port.

@westleyMS - if you have internal insights on what was exactly fix and tracking numbers, those would absolutely help. Based on this comment, I can't really perform additional actions as such.

@VesaJuvonen looks like it was fixed for SPO in issue #2764

The original message has the following environmental description: In OnPrem SP 2016, SPFx web parts developed using SPFx-1.4.1. Can you please double check the used version as SharePoint Framework 1.4.1 is NOT supported with SharePoint 2016 Feature Pack 2. Only supported version is SharePoint Framework 1.1.
There are mentions of Premier support, but unfortunately, I do not have access to what has reported there, so can't comment on the progress of that. Obviously, an issue reported with premier support should not be waiting for triage in this issue list as Premier customers are absolutely getting premier support for their issues.

Also - if someone still has this issue, can you please check the situation of your deployed solutions.
Failed to render client side web part. Id: [xxxxxx-xxxx-xxxx-xxxxxx], Manifest: [], Error Details:: [Microsoft.SharePoint.ClientSideComponent.SPClientSideFrameworkException: Web Part xxxxxx-xxxx-xxxx-xxxxxx failed to render as manifest is not found

'Would mean that the solution or components are not anymore served from the app catalog. Now - You can actually check this by moving to ComponentManifests folder in the app catalog site collections, so moving to for example to address of contoso.sharepoint.com/sites/apps/Lists/ComponentManifests. This is where the installed solutions are available.
One possible cause of the issue is that if you are using old old old version of the Yeoman to generate your solution, feature or component IDs might get changed when you do an update. This would then cause challenges whenever you update the solution package in app catalog as the manifest ID is changed and the existing instances in the pages can no longer be found.
This issue was fixed in the later version of the Yeoman packages, which means that in the best possible scenario - you'd be using 1.8.1 version and then you have targeted the environment to be SP2016... which will result packages.json to reference to 1.1 versions... when you do however do package solution with the later versions, we use new versions of the packaging code... which will then ensure that your identifiers will remain the same.
How to detect this possible issue? Check from the ComponentManifest library what's the ID of the web part which is missing based on the name. If the ID has been changed after a new version of the solution package has been deployed, you are using a problematic version of the SPFx solution packaging.


-m is a new flag to escape the files validation phase.

problem:
Quick outputs nothing if the two files (as described with their XML validation profile) contain the same information.
a user may define an erroneous matchAgainst columns, and as a result Quick will output nothing, because none of the two files rows were used for comparison, giving the user a false result.
The solution:
Quick should support a flag "-m" that makes it skip the validation phase and goes right to the matching phase, this means that even if the two files contain the same information, all the rows described in the XML validation profile will be matched and displayed.
The user can then use this option to ensure that her validation profile is doing what is intended, or debug the validation profile in case she needs to.

implemented here https://github.com/MouslihAbdelhakim/Quick/pull/8/commits/fbae9475ad6d7c3978043662cf1b7fd364cf1c90


Add test for final N being prime

This test would catch the case where the sieve algorithm only goes up to N-1, which is a fairly natural fence post error.  You can see an example of this problem happen here and corrected version here.  I think adding a third test to this very simple problem is worth helping programmers to carefully consider how they're iterating to stop at the exact number desired.

Poignantly enough, the example.py has calculates that sieve(7) should be [2,3,5] and not [2,3,5,7].  Nonetheless, if you ask for primes up to the number 7, it should return that 7 is prime.  I'll fix the example.py and resubmit.


librelp: 1.3.0 -> 1.4.0

Motivation for this change
Fixing package faillures by @r-ryantm
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Notify maintainers
cc @
[2 built, 14 copied (9.5 MiB), 4.5 MiB DL]
2 package were build:
librelp rsyslog

$ nix path-info -Sh ./results/librelp/nix/store/771ihbv1a1y4h3xgm7dd0b8bsihg17pq-librelp-1.4.0         44.2M

@GrahamcOfBorg build librelp

@GrahamcOfBorg build librelp

@jonringer this doesn't seem to build (I did fix the wrong sha256 in the push).

@GrahamcOfBorg build rsyslog

@mmahut going from the pre-packaged tar to source required an autoreconfHook, and looks like they added openssl as a dependency

@jonringer thank you, this looks good.


[JAMES-3805] fixes more flaky pulsar tests

supercedes https://github.com/apache/james-project/pull/1236
without this patch repeat run of pulsar mailqueue sometimes fails within a dozen runs with a missing public tenant. With the patch I couldn't make the run fail on my laptop (stopped a bit over 40 runs)

Little issue at the compilation first though, but that should be easily solved ;)
20:56:34,289 [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:testCompile (default-testCompile) on project scaling-pulsar-smtp: Compilation failure
20:56:34,289 [ERROR] /home/jenkins/workspace/james_ApacheJames_PR-1239/server/apps/scaling-pulsar-smtp/src/test/java/org/apache/james/PulsarExtension.java:[40,25] unreported exception org.apache.pulsar.client.admin.PulsarAdminException; must be caught or declared to be thrown

Hmm sorry about that.
Unfortunately I'm traveling for work this week not sure I can fix it before
next weekend
Feel free to take over if you want
Le lun. 10 oct. 2022 à 04:26, Rene Cordier @.***> a
écrit :

Little issue at the compilation first though, but that should be easily
solved ;)
20:56:34,289 [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:testCompile (default-testCompile) on project scaling-pulsar-smtp: Compilation failure
20:56:34,289 [ERROR] /home/jenkins/workspace/james_ApacheJames_PR-1239/server/apps/scaling-pulsar-smtp/src/test/java/org/apache/james/PulsarExtension.java:[40,25] unreported exception org.apache.pulsar.client.admin.PulsarAdminException; must be caught or declared to be thrown
—
Reply to this email directly, view it on GitHub
https://github.com/apache/james-project/pull/1239#issuecomment-1272717927,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAFTQYLJVLWIYMQP75BK4DWCN5EVANCNFSM6AAAAAARAMUCOA
.
You are receiving this because you were assigned.Message ID:
@.***>

Unfortunately I'm traveling for work this week not sure I can fix it before
next weekend
Feel free to take over if you want

Don't worry, I merged the hotfix (@Tag(Unstable) so that you have all your time fixing this ;-)

BTW I did not had the right to push on your branch despite it being opened against an ASF repo :-(

Hmmm   I didn't change permissions defaulted by GitHub on the fork but the
branch  is on my personal fork so isn't that normal? (Also I don't think I
forked directly from the apache repo but from a fork of the repo or even a
fork of a fork maybe that's why ? No idea really)
Le lun. 10 oct. 2022 à 10:23, Benoit TELLIER @.***> a
écrit :

BTW I did not had the right to push on your branch despite it being opened
against an ASF repo :-(
—
Reply to this email directly, view it on GitHub
https://github.com/apache/james-project/pull/1239#issuecomment-1272951779,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAFTQYDW53NTXY5KNCDX3DWCPG7BANCNFSM6AAAAAARAMUCOA
.
You are receiving this because you were assigned.Message ID:
@.***>

:green_apple: rebase and merge incoming


several body tracking APIs missing const qualifier, prevents const objects/variables

Multiple C++ APIs for body tracking are missing the const qualifier. By not having const on these c++ member functions, it prevents the use of const objects. Such objects are desired by style, coders, and compliers.
I have so far found the following. There may be others. These all return POD. I don't readily see any reason to prevent them being const.
k4abt::frame
  get_num_bodies()
  get_body_skeleton()     both signatures
  get_body_id()
  get_body()

Setup
Microsoft Windows [Version 10.0.19042.546]
Azure Kinect SDK 1.4.1
Body Tracking SDK 1.0.1
Visual Studio Community 16.7.2
RGB camera firmware:      1.6.110
Depth camera firmware:    1.6.79
Repro
An example of the issue can be reproduced in two lines...
const k4abt::frame btframe(nullptr);
const auto numBod = btframe.get_num_bodies();

Result
Compiler errors and failed compile.
'uint32_t k4abt::frame::get_num_bodies(void) noexcept': cannot convert 'this' pointer from 'const k4abt::frame' to 'k4abt::frame &'
'numBod': an object of const-qualified type must be initialized

Expected
Clean compile. No errors.
Notes
Lets look at one of the APIs k4abt::frame::get_num_bodies()

not qualified const
returns POD of uint32_t
no need to add another reference on to the frame handle since the k4abt::frame of that member function
already holds a reference
the code in that member function is a single line return k4abt_frame_get_num_bodies(m_handle)
that C API's param is non-const

Somewhat related: the C APIs that implement the functionality in these member functions can likely have const params themselves. They are not going to be changing the frame handle itself.

Additionally
k4abt::tracker
  pop_result()             both signatures
  set_temporal_smoothing()
  shutdown()

I do hesitate slightly in one area. That last one shutdown() can technically be const.  The tracker c++ object doesn't change since it contains m_handle and that is used to call the C api k4abt_tracker_shutdown(m_handle) and no part of the C++ object changes. The thing to which the handle points does change. But not the handle itself, and not tracker. So technically, it can be const. However since shutdown() causes a drastic logical change to the logical tracker (not the technical k4abt::tracker), I can imagine a viewpoint that making it non-const conveys such a logical change.
As a rigorous coder, I suggest make them consistently const in all cases possible. Simultaneously, I acknowledge there is another possible viewpoint.

This issue is partially fixed in Body Tracking SDK v1.1.0.
The apis still capable of const are
enqueue_capture()
pop_result()             both signatures
set_temporal_smoothing()
shutdown()

Fixed


Add snapshotter guts

for this one not sure too, but driver overcomplain wihout :man_shrugging:

Hi.
That won't work, unfortunately. The code itself does not support snapshotting yet. I'll see what can be done on the Go side (this week), and get back to you.


Update default memory request for server-config-init container (#150)

What this PR does:
Changes default memory request for server-config-init container from 256M to 300M.
Which issue(s) this PR fixes:
Fixes #150
Checklist

[x] Changes manually tested
[ ] Automated Tests added/updated
[ ] Documentation added/updated
[ ] CHANGELOG.md updated (not required for documentation PRs)
[x] CLA Signed:  DataStax CLA

@xunzhou from discussion in #150 it looks like you folks have developed a workaround. I will close this PR for now, but please let us know if you need to discuss this issue further and we will look at what we can do to assist.


[Security Hotspots] Review this potentially hardcoded credential.

On File config/passport.js Line 56
Review this potentially hardcoded credential.
See https://sonarcloud.io/project/security_hotspots?id=ricardocarva_MortiSmart for more details

The specified usernameField and passwordField values in the code snippet don't represent actual login credentials; they are simply field names expected in the request body during the authentication process. Attempting to use arbitrary values in these fields for login won't be successful, as they are not hardcoded credentials that grant access. The authentication logic is designed to check the user's provided email and password against existing records in the database, and without a valid match, login attempts will fail.


SAP UI5 Application View loading twice

Hi,
My UI5 application view is being instantiated twice.
Please find the below masnifest.json file.
{
"_version": "1.12.0",
"sap.app": {
"id": "com.hersheys_productmix",
"type": "application",
"i18n": "i18n/i18n.properties",
"applicationVersion": {
"version": "1.0.0"
},
"title": "{{appTitle}}",
"description": "{{appDescription}}",
"resources": "resources.json",
"ach": "ach",
"sourceTemplate": {
"id": "html5moduletemplates.basicSAPUI5ApplicationProjectModule",
"version": "1.40.12"
}
},
"sap.cloud": {
	"public": true,
	"service": "hersheys_productmix"
},

"sap.ui": {
	"technology": "UI5",
	"icons": {
		"icon": "",
		"favIcon": "",
		"phone": "",
		"phone@2": "",
		"tablet": "",
		"tablet@2": ""
	},
	"deviceTypes": {
		"desktop": true,
		"tablet": true,
		"phone": true
	}
},

"sap.ui5": {
	"flexEnabled": false,
	"rootView": {
		"viewName": "com.hersheys_productmix.view.App",
		"type": "XML",
		"async": true,
		"id": "app"
	},
	"dependencies": {
		"minUI5Version": "1.60.1",
		"libs": {
			"sap.ui.core": {},
			"sap.m": {},
			"sap.ui.layout": {}
		}
	},
	"contentDensities": {
		"compact": true,
		"cozy": true
	},
	"models": {
		"i18n": {
			"type": "sap.ui.model.resource.ResourceModel",
			"settings": {
				"bundleName": "com.hersheys_productmix.i18n.i18n"
			}
        },
        "oDetailViewModel": {
			"type": "sap.ui.model.json.JSONModel"
		}
	},
	"resources": {
		"css": [{
			"uri": "css/style.css"
		}],
        "js": [
            {
                "uri": "./util/formatter.js"
            }
        ]
	},
	"routing": {
		"config": {
			"routerClass": "sap.m.routing.Router",
			"viewType": "XML",
			"async": true,
			"viewPath": "com.hersheys_productmix.view",
			"controlAggregation": "pages",
			"controlId": "app",
			"clearControlAggregation": false
		},
		"routes": [
            {
                "name": "dashboard",
                "pattern": "",
                "target": [
                    "dashboard"
                ]
            },
            {
                "name": "detailView",
                "pattern": "detailView/{contextPath}",
                "target": [
                    "detailView"
                ]
            },
            {
                "name": "replace",
                "pattern": "replace",
                "target": [
                    "replace"
                ]
            }
        ],
		 "targets": {
            "dashboard": {
                "viewType": "XML",
                "transition": "slide",
                "clearControlAggregation": false,
                "viewId": "dashboard",
                "viewName": "dashboard"
            },
            "detailView": {
                "viewType": "XML",
                "transition": "slide",
                "clearControlAggregation": false,
                "viewId": "detailView",
                "viewName": "detailView"
            },
            "replace": {
                "viewType": "XML",
                "transition": "slide",
                "clearControlAggregation": false,
                "viewId": "replace",
                "viewName": "replace"
            }
        }
	}
}

}
Please help me solve the issue.
Thanks.

Hi
Check the following links  :
(1) https://github.com/SAP/openui5/issues/1746
(2)https://sapui5.hana.ondemand.com/sdk/#/topic/26ba6a5c1e5c417f8b21cce1411dba2c
Hope this will be usefull.
Regards
Prashant

Hi @guptha460,
thanks a lot for sharing your finding with us.
Could you please provide further details, like which view is instantiated twice or even better provide a sample so we can analyse your finding further.

Since there is no response from the author for more than 4 weeks I'm closing the ticket.

The problem appears when target view is the same as the specified root view. When the component is loaded the UI5-Framework creates an instance of the root view and all target views. Moreover, if target und root view have the same ID, an duplicate ID exception will be thrown. Otherwise, two difference instances will be created.
I would also be interested if there is a way to solve this issue without creating a separate root view. I would have expected, that the router checks if the target view with a given Id have been already created (eg. as rootView ) and in this cases reuses it's reference.


Allow storing logs in Elasticsearch

Add interface to either store logs in the database or in a different store, such as Elasticsearch
Make Wharf default to store logs in the database.
Add migrations to transfer logs to Elasticsearch. This is perhaps best suited as a manually triggered job. Such as via CLI: #142

Elasticsearch supports logs streaming, so this could let the wharf-api be scalable as the logs is currently the big thing making the wharf-api unscalable. Currently if we were to have 2 wharf-api pods, then the SSE log streaming would only send half of the logs due to the load balancing on log insertion.

The scaling shouldn't be the main reason for this change. Not to say the scaling isn't an issue, but that the scaling issue should result in a new MQ component instead.
Maybe we could introduce a separate component instead that deals with the messaging, inspired by how Mastodon has a separate "streaming" component just for the updates.
To begin with just extracting the streaming into a non-scalable separate component would make the wharf-api scalable at least. And then we could extend further if need be to add support for a full fletched MQ like RabbitMQ to even scale the streaming component.

Idea: what about Loki? Is Loki and/or Pro tail a better fit than Elasticsearch?
Loki is known for their elaborate logs handling algorithms in both storage and indexing/searching.
Needs further investigation.


[travis-ci] make log detailed, and dump fail log

also break some long lines.
Reference: https://www.bazel.io/docs/command-line-reference.html

close since I merged #1629 with this. It is a trivial change and we already did this in internal CI months ago.


Ruby: Add updater-proc as a constructor param.

This will allow users to specify an updater proc to transform metadata for requests. There has been requests for this in order to allow users to add authentication parameters to their requests.

@jmuk was there a specific reason the scopes parameter was not being documented?

Codecov Report

Merging #1123 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1123   +/-   ##
=========================================
  Coverage     85.13%   85.13%
  Complexity     3624     3624
=========================================
  Files           368      368
  Lines         14380    14380
  Branches       1649     1649
=========================================
  Hits          12242    12242
  Misses         1700     1700
  Partials        438      438

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 6b3f3d7...72d3433. Read the comment docs.

I don't remember why scopes hasn't been documented -- probably I missed that with no reasons. LGTM.


Raw value missing in generated constant file.

Generated constant file doesn't contain raw String Value identifiers.
example:
public enum TableViewCellReuseIdentifier : String {
    case MessageCell
}

@orta Support for storyboard identifiers with spaces was removed in d935c6dace92ca4cebfd35eccc1e647623f6b0d4

Closing: #41 Removed this as this is not necessary.


[feature request] Preview customization

Being able to customize the previews is an amazing feature in Decap.  I use it to import my global css so that the content in the preview looks just like the page the content is on!
Any plans to support this feature?  Or perhaps I missed it?

Hi, thank you for checking Sveltia CMS! Custom Previews are not supported yet, as mentioned in the README. I’ll figure out how to do it hopefully this month.

I have used NetlifyCMS on a couple of hugo projects. I created PreviewTemplates for the collection types and found the process a bit of a torture. I am not a React developer. I have dreamed more than once of a svelte port of the CMS.
To allow clients to build their own page layouts from a menu of sections we implemented the 'Advanced_Page' collection type.
The client can add a new page and choose the components for the page in the order they require. I never managed to implement a satisfactory PreviewTemplate for this Advanced page type.I have attached the section for the advanced page type from our config.yml (it is inside the zip file.)
So my request is a Preview template made up of sub templates displayed in the order passed in as a collection.
advanced_page.zip

I'm so keen for this.
My initial thought would an onChange event, and a way to subscribe to partial updates of content.
If I was editing in a directory collection .[5].reviewer, it would send a payload with:
{
  segments: [5, 'reviewer'],
  value: {
    from: 3,
    to: 5
  }
}


Adding git tags for when each blog post is released

Hi,
First I'd like to say thanks for taking the time to make this blog series.
I'd like to see the code at the time of writing a specific post, but that requires me to search through the repo's history and make a guess as to which commit corresponds with when that blog post was posted. Would it be possible to use git tags to tag the commit of the repo when you post a new blog post? That way, myself and others could just checkout the repo at that tag to browse the code.

Great idea! I’ll add them now.


Added CocoaPods Support

So I wanted to install this library through CocoaPods, along with all of my other libraries, so I decided to go and make the changes myself and submit a pull request.
This is kind of a confusing pull request (had to figure out how to use CommonCrypto without Xcode getting all kinds of mad :persevere: )
So I suppose a little bit of explanation is in order.
I read a whole lot of things online, but the one that actually ended up helping the most was this article.
Module Maps
CocoaPods builds your library as a Framework. Frameworks end up being Modules. Modules aren't allowed import non-modular things into their code. That's why there's a ton of hurt trying to link to CommonCrypto.
I solved this by making individual Module Maps for CommonCrypto for each platform. This makes it so I can kind of "privately" import CommonCrypto without exposing it to the outside, which I think is what makes the compiler so angry.
I added it to the project targets themselves, so the default project still builds like normal. I also added it to the Podspec, so when people import the project, it will still be able to use the right module map.
injectXcodePath.sh is a neat little script from the blog post up above. It looks into the module maps, where it is importing the CommonCrypto header, and changes it to be in the correct location for the user's machine.
Needs Changing
The podspec points to my fork. The reason for this, is that the last release tag is from a long time ago, and wouldn't build on my machine. So I made the branch master, and pointed it towards my fork so it has the module maps. Otherwise it wouldn't build.
After merging (or just before), we'll have to change it back to mattdonnelly/Swifter for consistency, and then we can push it to the main Pods repo.

This is great, thanks!
We can have a new release (1.7, about time :sweat_smile: ) after merging #131 :
once that is done, you'll be able to update the git source ( https://github.com/mattdonnelly/Swifter/pull/132/files#diff-6a7d817b08dfa9a5a74bfc48ede7f96dR9 ) and then we can merge this PR.
Thank you for the awesome work!

Actually, you know what, I woke up this morning and I'm not sure that the default project will run for people without CocoaPods that don't have my same setup (i.e. the Mac OSX SDK in the same directory, etc).
I think it should be possible to keep #import <CommonCrypto/CommonCrypto.h> in the standard project, and just use the module map when building with CocoaPods. Let me see if I can get it fixed up so it works anywhere.

No worries, I would have test it out before merging!
Anyway, I'm waiting for @lewis-smith's patch for #103 to release a new version :+1: (no rush Lewis, thanks for the awesome job!).
This way we will have a complete (and future proof!) new release :sunglasses:

+1 to this PR. Love the library, wish it were a pod!

Rebased on top of master 👍

Sorry this took so long to get to, I've now added CocoaPods support in 7ddeabb01ce00c4cc793a42218fca951b1cb94ef


Why there still has Chinese i18n strings in bundle assets when use enUS in ConfigProvider

[x] I'm sure this does not appear in the issue list of the repository

Basic Info

Package Name And Version<EMAIL_ADDRESS>Browser: chrome101.0.4951.64

Extra info

import enUS from '@arco-design/web-react/es/locale/en-US';

 <ConfigProvider locale={enUS}>

</ConfigProvider>

Hi, could you please provide a minimal reproducible code sample? (either as a github repo or codesandbox link)

Because Chinese is the default language, in locale/default.ts, it will be bundled into the component library by default.
For now, you can use https://github.com/arco-design/arco-plugins/tree/main/packages/plugin-webpack-react to change default language, options.defaultLanguage.

Because Chinese is the default language, in locale/default.ts, it will be bundled into the component library by default. For now, you can use https://github.com/arco-design/arco-plugins/tree/main/packages/plugin-webpack-react to change default language, options.defaultLanguage.

Is there a solution in vite.js?

Because Chinese is the default language, in locale/default.ts, it will be bundled into the component library by default. For now, you can use https://github.com/arco-design/arco-plugins/tree/main/packages/plugin-webpack-react to change default language, options.defaultLanguage.

update:
Thanks,
I refer to the webpack plugin and write a vite plugin.


Cow: Watch coinography.com

Cow requests the watch of the watch_keyword coinography\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Mast in Charcoal HQ


Parsing code needs to support the new forecast format

Pull request #233 introduces a new data layout for forecasts.  The parsing code and System constructors that take a ps_dict as a parameter rely on the old layout and need to be updated.

We should fix this issue in #233 before merging.

I agree that that is ideal.  However, it involves a lot more changes.  When do you want it #233 merged?

We need both issues addressed since I have to parse the CDM data to make tests in PowerSimulations.  I will review the code today and we can merge soon.

Not necessarily.  There is a workaround.
system = System(PowerSystems._System(old_constructor_inputs))

ok, then I will review it and we can merge

Actually, this is such a big mess now that I will try to fix the parsing code today.

Nope.  I take it back.  :(


Feature/bluetooth support for ios tested on ql820nwb

This is to address https://github.com/MenelicSoftware/cordova-plugin-brother-label-printer/issues/43


loadDrawingFromUrl  vs  loadVolumes

Thanks for your awesome work!
When I use loadDrawingFromUrl and loadVolumes , I have a few questions.
When I use loadDrawingFromUrl to import a file that records tags, nv.loadDrawingFromUrl("/api/image/202302/0b2be9e0-886b-4144-99f0-8bb4c6eaa848-label.nii.gz") , the result is this:

When I use loadVolumes to import the same, the result is this (without color):

When I use both stacked together, the result is as follows:

We can see that overlaying the two gives a more three-dimensional and clear color effect, but this will cause some problems when we use the brush (because visually it is possible, but when we try to delete some blocks with the eraser, the part of the loadVolume display will be exposed), I would like to achieve this more three-dimensional and clear effect just by using loadDrawingFromUrl. I would like to use loadDrawingFromUrl only to achieve this more three-dimensional and clear effect, because loadVolume can not provide the function of a brush, so I would like to ask what kind of settings can be used only with loadDrawingFromUrl to achieve the last picture display effect? It also allows us to edit the colors with the brush
Looking forward to your reply!

@johnren-code I can provide more feedback if you provide a sample that allows me to replicate your setup . However, you describe the intended behavior.  Drawings are discrete: a voxel is or is not a specific color (so nearest neighbor interpolation is required). In contrast, scalar volumes can use trilinear interpolation to reduce aliasing. Put another way, the boundary of Brodmann Area 6 and 8 is not Area 7: the classification is discrete. In contrast, for scalar MRI T1 images, 7 is a reasonable partial volume estimate for a mixture of 6 and 8.

Here is my code:
<template>
<div>
<canvas id="gl" height="480" width="640"></canvas>
</div>
</template>
const nv = new Niivue({
  logging: true,
  dragAndDropEnabled: true,
  backColor: [0, 0, 0, 1],
  show3Dcrosshair: true,
  // onLocationChange: this.locationChangeHandler,
})
export default {
  name: 'Segmentation',
  data() {
    return {
      volumeList: [
        {
          url: "/api/image/202302/0b2be9e0-886b-4144-99f0-8bb4c6eaa848.nii.gz",
        }
      ],
    }
  },
mounted() {
    nv.setInterpolation(false)
    nv.setRadiologicalConvention(false)
    nv.opts.multiplanarForceRender = true
    nv.attachTo('gl')
    nv.loadVolumes(this.volumeList)
    nv.setSliceType(nv.sliceTypeMultiplanar)
    nv.setClipPlane([0.3, 270, 0]);
    nv.setRenderAzimuthElevation(120, 10);
    nv.setSliceMM(true);
    nv.graph.autoSizeMultiplanar = true
    // nv.loadDrawingFromUrl(this.testUrl)
    nv.loadDrawingFromUrl("/api/image/202302/0b2be9e0-886b-4144-99f0-8bb4c6eaa848-label.nii.gz")
    nv.drawMeasurementTool()
    // nv.overlayRGBA()
  }

I would be grateful if you could suggest something to achieve the above effect.

The files I use is available at . 0b2be9e0-886b-4144-99f0-8bb4c6eaa848.nii.gz is the original file, 0b2be9e0-886b-4144-99f0-8bb4c6eaa848-label.nii.gz is the label file after 3D segmentation of it

@johnren-code this is the intended behavior. The drawing colors are categorical, not scalar. The images below show the effect of using trilinear (top) versus nearest-neighbor (bottom) interpolation for your drawing.

As you have found, one solution would be to load the drawing as both a categorical drawing as well as a scalar grayscale image.
Future versions of NiiVue will introduce optional shaders that will apply gradient-based lighting shaders, as we demonstrate in this live demo that will also provide hinting. However, this method requires substantially more resources, and our focus has been to provide interactive graphics even on low-end devices.

Ok, thanks for the explanation, I understand, how is the trilinear interpolation shown in your picture above achieved? I think the effect achieved by trilinear interpolation meets my needs.
In addition, for solution 1, the superimposed effect to achieve a visual three-dimensional sense, which will make the user in the eraser to delete a certain area, resulting in categorical drawing and scalar grayscale image both out of sync, so I do not have a better measure for the superimposed method.
I'm also thinking about some possible solutions to this problem, and I'll be happy to share them with you if I can finally come up with a workable solution.

In fact, the overlay effect I show in the image is only achieved by loading the same file as loadDrawingFromUrl and loadVolume.

You can make your own fork of NiiVue and update the function drawImage3D() to support linear interpolation:
    if (this.drawBitmap && this.drawBitmap.length > 8) {
      let interp = this.gl.LINEAR;
      this.gl.activeTexture(this.gl.TEXTURE7); //overlay
      this.gl.texParameteri(this.gl.TEXTURE_3D, this.gl.TEXTURE_MIN_FILTER, interp);
      this.gl.texParameteri(this.gl.TEXTURE_3D, this.gl.TEXTURE_MAG_FILTER, interp);
      gl.uniform1f(shader.drawOpacityLoc, this.drawOpacity);
    } else gl.uniform1f(shader.drawOpacityLoc, 0.0);

Precisely, you can get the effect by loading the same volume as both a scalar and indexed volume. Note that this doubles the resources required on your GPU.

Ok, thanks for the answer, I will try this method later. In fact, I am currently using niivue library to make a complete 3D medical data presentation platform, if it goes well, I will open source my code on top of github when I finish, hopefully it will bring some help to niivue library.
Thanks again for your awesome work!

I noticed that the color of the 2d slice is fine, the color of the 3d model obtained by trilinear interpolation is a bit problematic, what is the reason for this?

These are classic partial volume interpolation artifacts. The 2D slices show large contiguous regions of the same category, with artifacts only seen along the edge of the object. The 3D rendering is explicitly displaying the edges of the objects. The off-axes view of the rendering creates more partial volume voxels than sampling with the grid.

Is there any way to solve this problem? For example, the corresponding code changes

@johnren-code I have added the ability to create custom labeled colormaps. You can try this out with this live demo. Note that the bottom left corner can provide the label name (e.g. bladder). It also provides a check box to toggle interpolation on and off. I do think for your situation you will want to use labeled images (as demonstrated) rather than drawings. Drawings are limited to 8-bits (256 levels) and are restricted (to allow rapid data transfer to the GPU).
git clone --branch issue616LiveDemo git@github.com:niivue/niivue.git
cd niivue
npm install
npm run dev

Ok, thank you very much for the example, it was very helpful for me! In fact, the reason I've been obsessed with using loadDrawingFromUrl is that I not only want to use colorMap to change the color, but I also want to use an eraser or brush etc. to do something editable with volume[1], similar to loadDrawingFromUrl, is this possible?

@johnren-code I have added a new variable renderDrawAmbientOcclusion. The live demo now includes a slider that allows you to adjust this value between 0.0...1.0 - the default is 0.4.
git clone --branch issue616LiveDemo git@github.com:niivue/niivue.git
cd niivue
npm install
npm run dev

I see, this is very helpful, thank you! Also, how do I add your changes to the nivue library I'm using for my current project, I'm using version 0.32.0 and the installation command in the vue framework is
npm install<EMAIL_ADDRESS>

I think @hanayik can provide a better answer. In the short term, you can compile your own copy of niivue.umd.js that you can use with your pages using the command:
git clone --branch issue616LiveDemo git@github.com:niivue/niivue.git
cd niivue
npm install
npm run demo

Longer term, if you are satisfied with this solution, please close this issue. At that stage, I will generate a pull request to the master branch. The PR will need to be reviewed and accepted to become part of the master branch. Once the PR is reviewed and approved, it will become part of the master branch.  At that stage, you can call it with:
<script src="https://niivue.github.io/niivue/features/niivue.umd.js"></script>

New releases are generated from the master branch, on an irregular schedule.

OK, I will close this issue. Thank you very much for your help!


WaitReady () -> TIMEOUT (when PN532 reader is selected)

The PN532.cpp library contains the following snippet:
bool PN532::WaitReady()
{
    uint16_t timer = 0;
    while (!IsReady())
    {
        if (timer >= PN532_TIMEOUT)
        {
            Utils::Print("WaitReady() -> TIMEOUT\r\n");
            return false;
        }
        Utils::DelayMilli(10);
        timer += 10;
    }
    return true;
}

As a result, the WaitReady () -> TIMEOUT message is very often listed in the serial monitor.
This is very annoying, and I propose the corresponding part (one single line) in the library to comment.

I have PN532 lying around somewhere, actually it need to be tested more.

[ ] Test PN532


Generate lunr search index

This pull request introduces a metalsmith plugin which can create a lunr compatible search index along with a document store.
Only documents that are in a navigation section with 'includeInSearch' will be included in the index.
See commits for more details.
The file generated is ~2.2kB gzipped
Paired with @nickcolley (with semi-regular hydration breaks)
As part of https://trello.com/c/YfpJzpDf/1270-5-generate-a-search-index-as-part-of-the-build-process

You can preview this change here:
Built with commit 31658888b4f86cb0ca3bccd2e10112d467fcc330
https://deploy-preview-466--govuk-design-system-preview.netlify.com


Remove machineconfigs and machineconfigspools

We are already override the machine config operator so removing those
configs shouldn't affect the cluster and those are regenerated with
updated content as long as user enable it.

/cherry-pick 4.7


feat(module:notification): support open at differencet places

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[ ] No

Other information

ping @hsuanxyz Please re-review this~