Feature Request: support applying saved theme to individual slides?

Hi @linzhong, can you describe your use case for us?

Sure,
I would like to create a powerpoint file using your libraray. We have some
format themes already defined, so instead of recreate all the format in the
code, I would like to apply the corresponding theme settings to the slides
that I produce. (Just like in the powerpoint application, choose the
corresponding theme and apply to the selected slide)
Ideally, I would like to be able to create some template slides and just
insert certain data into the slide templates, I don't see this is supported
from what I learned in the document so far. The template support may be too
much to ask. But if I can apply the theme to the generated slides, it will
work in my scenario as well.
Thanks,
Lin
On Fri, Feb 20, 2015 at 1:49 PM, Steve Canny<EMAIL_ADDRESS>wrote:

Hi @linzhong https://github.com/linzhong, can you describe your use
case for us?
—
Reply to this email directly or view it on GitHub
https://github.com/scanny/python-pptx/issues/146#issuecomment-75325397.

--
*Lin Zhong  *

Applause gets Three Straight Years as one of Forbes’ Most Promising
Companies http://www.applause.com/awards

As far as the themes go, the approach would be to create a starting .pptx file, a "template" of sorts, that you use as the starting point for the presentation you generate.
http://python-pptx.readthedocs.org/en/latest/user/presentations.html
In that starting .pptx file you would include your customized themes. You can do this by creating a presentation that uses all the themes you would like to have available and then deleting all the slides (but not the slide masters or slide layouts). You will be left with an empty presentation having multiple slide masters, each of which has its own set of slide layouts.
Then you pick the particular layout you want to generate each slide, perhaps something like this:
from pptx import Presentation
prs = Presentation('my-template.pptx')
slide_layout = prs.slide_masters[2].slide_layouts[3]
slide = prs.slides.add_slide(slide_layout)

You'll need to know in advance what the (master_idx, layout_idx) combination is to get the theme/layout combination you're after as they are only accessible by index at the moment.
Does that give you what you need?

Thanks a lot, Steve.
This indeed serve the need, I read the page you refer to earlier and knew
that I can open existing slide. But I didn't realize that I can retrieve
the layout of existing one to create the new one. Thanks a lot for pointing
it out. This is great! :)
Lin
On Fri, Feb 20, 2015 at 4:03 PM, Steve Canny<EMAIL_ADDRESS>wrote:

As far as the themes go, the approach would be to create a starting .pptx
file, a "template" of sorts, that you use as the starting point for the
presentation you generate.
http://python-pptx.readthedocs.org/en/latest/user/presentations.html
In that starting .pptx file you would include your customized themes. You
can do this by creating a presentation that uses all the themes you would
like to have available and then deleting all the slides (but not the slide
masters or slide layouts). You will be left with an empty presentation
having multiple slide masters, each of which has its own set of slide
layouts.
Then you pick the particular layout you want to generate each slide,
perhaps something like this:
from pptx import Presentation
prs = Presentation('my-template.pptx')
slide_layout = prs.slide_masters[2].slide_layouts[3]
slide = prs.slides.add_slide(slide_layout)
You'll need to know in advance what the (master_idx, layout_idx)
combination is to get the theme/layout combination you're after as they are
only accessible by index at the moment.
Does that give you what you need?
—
Reply to this email directly or view it on GitHub
https://github.com/scanny/python-pptx/issues/146#issuecomment-75342055.

--
*Lin Zhong  *

Applause gets Three Straight Years as one of Forbes’ Most Promising
Companies http://www.applause.com/awards

Steve,
I manage to add the slide with the template fine. Could you please point me
to the document on how to delete the slides?
From your document (
http://python-pptx.readthedocs.org/en/latest/user/slides.html), it says
that the support for this feature is in the backlog. Does this feature
become available now, or when do you expect it to be available?
Thanks,
Lin
On Mon, Feb 23, 2015 at 11:04 AM, Lin Zhong<EMAIL_ADDRESS>wrote:

Thanks a lot, Steve.
This indeed serve the need, I read the page you refer to earlier and knew
that I can open existing slide. But I didn't realize that I can retrieve
the layout of existing one to create the new one. Thanks a lot for pointing
it out. This is great! :)
Lin
On Fri, Feb 20, 2015 at 4:03 PM, Steve Canny<EMAIL_ADDRESS>wrote:

As far as the themes go, the approach would be to create a starting .pptx
file, a "template" of sorts, that you use as the starting point for the
presentation you generate.
http://python-pptx.readthedocs.org/en/latest/user/presentations.html
In that starting .pptx file you would include your customized themes. You
can do this by creating a presentation that uses all the themes you would
like to have available and then deleting all the slides (but not the slide
masters or slide layouts). You will be left with an empty presentation
having multiple slide masters, each of which has its own set of slide
layouts.
Then you pick the particular layout you want to generate each slide,
perhaps something like this:
from pptx import Presentation
prs = Presentation('my-template.pptx')
slide_layout = prs.slide_masters[2].slide_layouts[3]
slide = prs.slides.add_slide(slide_layout)
You'll need to know in advance what the (master_idx, layout_idx)
combination is to get the theme/layout combination you're after as they are
only accessible by index at the moment.
Does that give you what you need?
—
Reply to this email directly or view it on GitHub
https://github.com/scanny/python-pptx/issues/146#issuecomment-75342055.

--
*Lin Zhong  *

Applause gets Three Straight Years as one of Forbes’ Most Promising
Companies http://www.applause.com/awards

--
*Lin Zhong  *

Applause gets Three Straight Years as one of Forbes’ Most Promising
Companies http://www.applause.com/awards

FYI,
If I open the template and use the template layout to add slide to a bland
new presentation, I will get an error. The error require repair operation
when open the new slide, though the repair result looks correct.
template = Presentation(TEMPLATE_PATH)
prs = Presentation()
#title slide
slide_layout = template.slide_masters[0].slide_layouts[0]
slide = prs.slides.add_slide(slide_layout)
...
On Mon, Feb 23, 2015 at 3:19 PM, Lin Zhong<EMAIL_ADDRESS>wrote:

Steve,
I manage to add the slide with the template fine. Could you please point
me to the document on how to delete the slides?
From your document (
http://python-pptx.readthedocs.org/en/latest/user/slides.html), it says
that the support for this feature is in the backlog. Does this feature
become available now, or when do you expect it to be available?
Thanks,
Lin
On Mon, Feb 23, 2015 at 11:04 AM, Lin Zhong<EMAIL_ADDRESS>wrote:

Thanks a lot, Steve.
This indeed serve the need, I read the page you refer to earlier and knew
that I can open existing slide. But I didn't realize that I can retrieve
the layout of existing one to create the new one. Thanks a lot for pointing
it out. This is great! :)
Lin
On Fri, Feb 20, 2015 at 4:03 PM, Steve Canny<EMAIL_ADDRESS>wrote:

As far as the themes go, the approach would be to create a starting
.pptx file, a "template" of sorts, that you use as the starting point for
the presentation you generate.
http://python-pptx.readthedocs.org/en/latest/user/presentations.html
In that starting .pptx file you would include your customized themes.
You can do this by creating a presentation that uses all the themes you
would like to have available and then deleting all the slides (but not the
slide masters or slide layouts). You will be left with an empty
presentation having multiple slide masters, each of which has its own set
of slide layouts.
Then you pick the particular layout you want to generate each slide,
perhaps something like this:
from pptx import Presentation
prs = Presentation('my-template.pptx')
slide_layout = prs.slide_masters[2].slide_layouts[3]
slide = prs.slides.add_slide(slide_layout)
You'll need to know in advance what the (master_idx, layout_idx)
combination is to get the theme/layout combination you're after as they are
only accessible by index at the moment.
Does that give you what you need?
—
Reply to this email directly or view it on GitHub
https://github.com/scanny/python-pptx/issues/146#issuecomment-75342055
.

--
*Lin Zhong  *

Applause gets Three Straight Years as one of Forbes’ Most Promising
Companies http://www.applause.com/awards

--
*Lin Zhong  *

Applause gets Three Straight Years as one of Forbes’ Most Promising
Companies http://www.applause.com/awards

--
*Lin Zhong  *

Applause gets Three Straight Years as one of Forbes’ Most Promising
Companies http://www.applause.com/awards

Sorry, Never mind,
Read your email again, and found my misunderstanding. You mean delete the
slide directly from template, not from the code.
Please ignore my earlier questions.
Thanks,
Lin
On Mon, Feb 23, 2015 at 3:28 PM, Lin Zhong<EMAIL_ADDRESS>wrote:

FYI,
If I open the template and use the template layout to add slide to a bland
new presentation, I will get an error. The error require repair operation
when open the new slide, though the repair result looks correct.
template = Presentation(TEMPLATE_PATH)
prs = Presentation()
#title slide
slide_layout = template.slide_masters[0].slide_layouts[0]
slide = prs.slides.add_slide(slide_layout)
...
On Mon, Feb 23, 2015 at 3:19 PM, Lin Zhong<EMAIL_ADDRESS>wrote:

Steve,
I manage to add the slide with the template fine. Could you please point
me to the document on how to delete the slides?
From your document (
http://python-pptx.readthedocs.org/en/latest/user/slides.html), it says
that the support for this feature is in the backlog. Does this feature
become available now, or when do you expect it to be available?
Thanks,
Lin
On Mon, Feb 23, 2015 at 11:04 AM, Lin Zhong<EMAIL_ADDRESS>wrote:

Thanks a lot, Steve.
This indeed serve the need, I read the page you refer to earlier and
knew that I can open existing slide. But I didn't realize that I can
retrieve the layout of existing one to create the new one. Thanks a lot for
pointing it out. This is great! :)
Lin
On Fri, Feb 20, 2015 at 4:03 PM, Steve Canny<EMAIL_ADDRESS>wrote:

As far as the themes go, the approach would be to create a starting
.pptx file, a "template" of sorts, that you use as the starting point for
the presentation you generate.
http://python-pptx.readthedocs.org/en/latest/user/presentations.html
In that starting .pptx file you would include your customized themes.
You can do this by creating a presentation that uses all the themes you
would like to have available and then deleting all the slides (but not the
slide masters or slide layouts). You will be left with an empty
presentation having multiple slide masters, each of which has its own set
of slide layouts.
Then you pick the particular layout you want to generate each slide,
perhaps something like this:
from pptx import Presentation
prs = Presentation('my-template.pptx')
slide_layout = prs.slide_masters[2].slide_layouts[3]
slide = prs.slides.add_slide(slide_layout)
You'll need to know in advance what the (master_idx, layout_idx)
combination is to get the theme/layout combination you're after as they are
only accessible by index at the moment.
Does that give you what you need?
—
Reply to this email directly or view it on GitHub
https://github.com/scanny/python-pptx/issues/146#issuecomment-75342055
.

--
*Lin Zhong  *

Applause gets Three Straight Years as one of Forbes’ Most Promising
Companies http://www.applause.com/awards

--
*Lin Zhong  *

Applause gets Three Straight Years as one of Forbes’ Most Promising
Companies http://www.applause.com/awards

--
*Lin Zhong  *

Applause gets Three Straight Years as one of Forbes’ Most Promising
Companies http://www.applause.com/awards

--
*Lin Zhong  *

Applause gets Three Straight Years as one of Forbes’ Most Promising
Companies http://www.applause.com/awards

Yes. The key thing here is what slide masters and slide layouts you have available to you at run time.
When you add the slides having a different theme, their slide master and slide layouts are automatically added to the file too. You just don't see them unless you go looking for them in the Slide Master view. Once that's done, you get rid of the slides you added so you can start with an empty presentation. Deleting the slides does not remove the new masters and layouts. Once they're in they're there to stay unless you delete them manually. All this is done "by hand" using PowerPoint, just to get your template file configured correctly.
There are other ways to do it--any method that gets the masters and layouts in there is fine--but this way is probably the quickest.
What you're really looking for is how the starting presentation looks in Slide Master view. In regular Slide view mode, it should be empty when you're done. Slide Master view is also where you can figure out what the index of a particular master or layout is; just remember to start counting at 0 instead of 1.
Note there is at least one other part involved, the theme{n}.xml files. Those specify the palette and fonts used by the master and layouts for that theme, and perhaps a few other details. Those get added automatically by this process. I just mention them here for anyone who might take a more direct approach :)

This are awesome tips. Thanks!
On Mon, Feb 23, 2015 at 4:34 PM, Steve Canny<EMAIL_ADDRESS>wrote:

Yes. The key thing here is what slide masters and slide layouts you have
available to you at run time.
When you add the slides having a different theme, their slide master and
slide layouts are automatically added to the file too. You just don't see
them unless you go looking for them in the Slide Master view. Once that's
done, you get rid of the slides you added so you can start with an empty
presentation. Deleting the slides does not remove the new masters and
layouts. Once they're in they're there to stay unless you delete them
manually. All this is done "by hand" using PowerPoint, just to get your
template file configured correctly.
There are other ways to do it--any method that gets the masters and
layouts in there is fine--but this way is probably the quickest.
What you're really looking for is how the starting presentation looks in
Slide Master view. In regular Slide view mode, it should be empty when
you're done. Slide Master view is also where you can figure out what the
index of a particular master or layout is; just remember to start counting
at 0 instead of 1.
Note there is at least one other part involved, the theme{n}.xml files.
Those specify the palette and fonts used by the master and layouts for that
theme, and perhaps a few other details. Those get added automatically by
this process. I just mention them here for anyone who might take a more
direct approach :)
—
Reply to this email directly or view it on GitHub
https://github.com/scanny/python-pptx/issues/146#issuecomment-75673910.

--
*Lin Zhong  *

Applause gets Three Straight Years as one of Forbes’ Most Promising
Companies http://www.applause.com/awards


supplement

LucidDreamer is also an excellent work which is CVPR2024 spotlight by Yingcong Chen.
Recommend it!

Thank you for the recommendation! I will definitely check out ‘LucidDreamer’ by Yingcong Chen, especially since it’s a CVPR2024 spotlight work.

I just found this work, actually it won the CVPR 2024 Highlight (11.9%). But that doesn’t take away from the fact that it’s an excellent piece of work. I'll add it to my collection.


Unsure

Unsure if this or wowmapper but the sticks work when setting up but then i try to click on use recommended for the buttons and it doesnt do it correctly? Doing this on ascension and unsure how to proceed.

image shows this, the sticks work correctly, wowmapper is detecting ascension now but it wont let me click this button

alternively i get to this screen but cannot input anything

I am also having this issue.

Ascension is a custom client, it has major changes on the standard WoW API and it's expected to be problematic, however I did some workarounds to make it work on the 1.1.0 release.


Print Server Nexx wt3020

Hi all.
Is it possible use the router nexx wt3020 like a print server? i have a old HP printer with usb port and i want connect in nexx wt3020 connected wi-fi. I see a little tutorial http://www.fischl.de/ql_printserver/ but i cannot finish the installation.
Many thanks,.

You're probably better off trying to ask this in a user support forum for openwrt, not the bug tracker for one particular user interface.


Add -rdynamic linker flag

Adds -rdynamic flag to linker settings. This is needed for backtraces to work properly.

We should add -ldl as well. We might be adding functionality into the hal using that in the future, and it might be a good argument to have for teams if they want to do some creative stuff.

For a team to make use of that, they're probably going to want to just disable the auto library management at which point they could add it themselves.


Add reusable form components and types

Description
This PR introduces reusable components and types for form management:

Reusable Components:

Input: Custom input field component.
Button: Consistent button styling component.
ContainerForm: Wrapper component for form elements.

Custom Input:

Utilized label from shadcn/ui to enhance custom input fields.

Types:

Added types.ts in utils with a type definition for actionFunction, used for form actions.

Type of Change

[x] New feature
[ ] Bug fix
[ ] Refactor
[ ] Documentation update
[ ] Other (please describe):

Checklist

[x] My code follows the project’s coding style
[x] I have performed a self-review of my code
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have updated the documentation as necessary
[x] My changes generate no new warnings or errors
[x] I have checked my code and corrected any misspellings
[ ] Any dependent changes have been merged and published in downstream modules

Related Issues
No related issues.
Additional Information
None.

:tada: This PR is included in version 1.12.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Reflect new disks size after resize

Description
This only apply to TM without LN and CLONE targets.
The new value of the size of the disks after a resize is not shown till the VM is deleted.

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

Will you create fix for this issue?


Add ArrayPrimitiveTypeExpressionFactory

When cloning Primitives all that needs to be done is copy the source
array to the target array.  Instead of calling the
PrimitiveTypeExpressionFactory call for every element you can use the
static Array.Copy method to assist.
Depending upon the primitive type, the performance gain is different.
In the tests I have, which are coded against 4.6.1, will have a
performance gain of any where from about 98% to almost 225%.  Byte
Array, for whatever reason, has a performance gain of about 6800% ... I
have no idea why byte arrays are so much different.
Int Arrays go from about 2,386,634 ops per sec to 4,727,659 (98% increase)
String arrays go from about 1,926,782 ops per sec to 5,730,086 (197% increase)
TimeSpan arrays go from about 1,988,071 ops per sec to 4,650,697 (133% increase)
DateTime arrays go from about 1,930,501 ops per sec to 4,210,105 (118% increase)
Delegate arrays go from about 1,659,751 ops per sec to 5,404,864 (225% increase)
Byte arrays go from about 58,083 ops per sec to 4,053,648 (6879% increase)
I would like supply a unit test to easily display the difference in
performance in the master project, but given the interface, it is not
exactly straight forward.  I generated the numbers above from running
the changes suggested from a local release build in my current test
setup.

For the polymorphism support, I should have likely created a new fork and made my changes there.  My apologies, I am fairly new to github and am still learning its nuances.

I'm happy to merge the perf improvements for arrays of primitive types, but for the other change related to polymorphism I would need more time to look into more closely because it has much greater implications on the library.
Would you mind reverting your last commit so I can accept this PR sending another one for the rest of the changes?

Done.  There is a similar performance increase for supplying an ExpressionFactory for a List<> of primitives (actually, the difference is more pronounced).  I will add that soon.  For the polymorphism support, I will for a new branch and put my work there soon.
Have you considered a mechanism that would allow users to specify their own expression factories so they may override default behavior by supplying their own implementations?

Hmm ... apparently I can have only one fork of the branch at a time.  Which pretty prevents parallel development since a pull request grab all difference from the base branch at a time (no way to isolate a change set like there is in TFS).

You should not be developing in your master branch, but instead create separate branches for each of the features/changes you're working on. Once they are ready you can request a Pull Request from a non-master branch of yours.


Publish to Marketplace action failing

The PublishVSIX action is failing with:
Error: Error: spawn VsixPublisher.exe ENOENT

There's also a warning:
Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16: mrluje/vs-marketplace-publisher@v2

I don't know if that's the cause of the problem.
I regenerated the PAT and secret for publishing, but that didn't help.
ENOENT indicates a missing file. Is it VsixPublisher.exe itself that's missing?

Still failing with the same error.

I think I fixed this by moving to a different action (CalvinAllen/action-vs-marketplace-publish@v1) in https://github.com/pharring/EditorGuidelines/commit/920537a28acb894c2700362004d45edc961228ed


fix for empty selection in vaadin-select-flow

fixes https://github.com/vaadin/vaadin-select-flow/issues/13

This change is

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you for the contribution, please include a test case also

Hey, in my haste for solving https://github.com/vaadin/vaadin-select-flow/issues/13
I didn't notice that this commit : https://github.com/vaadin/vaadin-list-mixin/pull/52
Already solved the issue. Can close this.


Can i also send GCM to ios devices?

Hey,
I was wondering if i can use this plugin in order to recieve push notifications (through GCM) to an ios device..
It takes me alot of time to actually set & maintain all of ther certificates required for APNS services.
And i saw some discussions about that issue..
So my question is.. Is this even possible?
And if so.. How do i manage to do that?
Thank you :)

@guyCactus Yes, yes you can. Look at the senderId and gcmSandbox parts of this:
https://github.com/phonegap/phonegap-plugin-push/blob/master/docs/API.md#ios
Right now you can send GCM messages to iOS devices but you still need to maintain your APNS certificates. You just end up uploading them to Google.
https://developers.google.com/cloud-messaging/ios/client

@guyCactus Yes, yes you can. Look at the senderId and gcmSandbox parts of this:
https://github.com/phonegap/phonegap-plugin-push/blob/master/docs/API.md#ios
Right now you can send GCM messages to iOS devices but you still need to maintain your APNS certificates. You just end up uploading them to Google.
https://developers.google.com/cloud-messaging/ios/client


fix a nil context getting passed to OutputSignatureSuccessNonKeybase

The original issue (CORE-6440) was that keybase pgp verify wasn't
printing anything for valid messages signed by revoked keys. This turned
out to be caused by a nil context (ctx.NetContext) that was getting
passed into an RPC call. That caused the rpc library to short-circuit
immediately with errors.New("No Context provided for this call"),
however we were dropping the error. As part of fixing the nil context,
make sure we propagate errors here.
TODO?: There are many other places in the codebase referencing
ctx.NetContext directly, as this code did. Are all of them suspect?
r? @patrickxb

I can repro this TestHome failure on master on my own machine. @maxtaco it bisects to 430486a193eaa79f271f4f45c2007485da4d4aa3. Known problem?


inline styles should end with semicolon

This PR should close pugjs/pug#2537
As per this blog post:
http://freshinbox.com/blog/outlook-com-alters-styles-of-elements-next-to-checkboxes/

... if you leave off the final semicolon (;) from the inline style in the checkbox, instead of copying the style to the next element, Outlook merely…. removes the style!

<input type=checkbox style="display:none !important">

CONTENT

becomes:

CONTENT

Coverage decreased (-0.07%) to 90.0% when pulling b9f8cfb08d8775b4de9db730d9b0afee04c91c1a on goenning:master into af8f8bf99ed0857e86c5064a7eced0df41bd90a5 on pugjs:master.

Thanks!

Coverage remained the same at 90.071% when pulling 167077510f3ad096d3a9ef498753ffca9b6baace on goenning:master into af8f8bf99ed0857e86c5064a7eced0df41bd90a5 on pugjs:master.

Coverage remained the same at 90.071% when pulling 167077510f3ad096d3a9ef498753ffca9b6baace on goenning:master into af8f8bf99ed0857e86c5064a7eced0df41bd90a5 on pugjs:master.


Evenly distributed tagset (all counts the same) cause div/0 error in defscale

The default scaling function has this expression:
1.0 / (maxcount - mincount)
When maxcount == mincount, things blow up.  As a workaround I've used a hacky alternative that avoids this well enough for my needs...
def tagscale(count, mincount, maxcount, minsize, maxsize):
if maxcount == mincount:
return defscale(count, mincount, maxcount + 1, minsize, maxsize)
return defscale(count, mincount, maxcount, minsize, maxsize)
There's probably a better way though

Thanks for the hint. It will be fixed in 0.3.5.


[Rolling] rclrs unable to build - cannot find headers for wrapper

rclrs is failing to build on the latest version of Rolling - bindgen is unable to find the includes for rcl during compilation.
It is possible that this issue is related to https://github.com/ros2/rcl/pull/959. If so, the build script for rclrs will need to be tweaked to point to the new location of the headers.

https://github.com/ros2-rust/ros2_rust/blob/dc454890e7cc26adb4dba1cc28703cfb00fb0b89/rclrs/build.rs#L32

Changing that to -I{}/include/rcl seems like it would update it to the new install directory.
Is there a way to tell bindgen to grab the include directory from CMake? Can it invoke a CMake script to grab the include directories off the rcl::rcl target?

Yeah, I tried changing that to grab the inner install directory, but then it complains that rcl/context.h cannot find rmw/init.h... See https://github.com/jhdcs/ros2_rust/blob/bug/fix_missing_includes/rclrs/build.rs for the changed build script.
I am not sure how to pass information from CMake into the build script... That's where the information would get passed to bindgen. Specifically, the part of bindgen we're working with here is the Builder: https://docs.rs/bindgen/latest/bindgen/struct.Builder.html... I'll keep digging through that API to see what abilities we have to grab stuff from CMake, but I'm not sure there's all that much...
I've also looked for Rust crates to interact with CMake, but I was only able to find https://github.com/corrosion-rs/corrosion (which I wasn't quite  able to get working last time I poked at it), and https://crates.io/crates/cmake (which looks like bindings to allow Rust to call CMake - whereas we're trying to go the other direction here).

Yeah, I've also been looking into this the last few days. Unfortunately everything I can find is indicating that cmake doesn't like to leak information outside of its own build system pipeline. There doesn't seem to be any way to just query cmake for the include directories of the target in a cmake package.
The most promising lead I could find for this was the cmake --find-package command, but when I tested it out on a very simple package structure it did not provide any of the package target's transitive dependency headers, which is what we would need in this case.
As far as I can tell the corrosion-rs package that @jhdcs linked to is meant to do the inverse of what we want.. it allows Cargo Crates to be built as part of a cmake project, but it doesn't seem to provide a way to push information from cmake into Cargo.

@jhdcs correct me if I'm wrong, but the problem is that we need to pass info from CMake to cargo, right?
In that case, we can set environment variables from CMake and when we invoke cargo, use std::env::var in the build script to read and add them to the include path for clang. We can obtain the include dirs for rcl with $rcl_INCLUDE_DIRS after calling find_package, or use get_target_property on the rcl::rcl target.

@esteve That's part of the problem, yes. We get around it by digging through the install directory to find folders we need to include, but CMake doesn't really pass anything to us (apart from the AMENT_PREFIX_PATH environment variable, which is what we're currently using to find directories to include)
The weird thing is, even when we're including all these files (via -I/path/to/install/directory/..., it STILL doesn't build. Like, the included headers don't seem to be able to find each other...

I was also considering feeding info to Rust from cmake using environment variables. My only hesitation with that is it locks ros2_rust into being a cmake package at its top level... indefinitely, and I don't find that palatable. I'm hoping to migrate away from ros2_rust needing cmake as a build tool as soon as feasible.

@jhdcs what I meant is that we set the variables ourselves from CMake (e.g. set(ENV{ROS2_RUST_RCL_INCLUDE_DIRS} path)) and then call cargo, which will read them

@mxgrey using CMake for rclrs is fine IMHO, it's what rclpy (https://github.com/ros2/rclpy/blob/master/rclpy/CMakeLists.txt) and the other bindings do, precisely because it's easier to query the ament info from CMake than from another language. However, programs that use rclrs should be buildable via cargo only (the same way that programs that use rclpy use standard Python build tools i.e. setup.py)

Yeah, that's a fair outlook.
Even so, I don't know of a mechanism in cmake where you could say "Hey, let's pretend I have a target that depends on this imported target. Please tell me all the include flags that we would give the compiler." It would be great if that exists, but I haven't found it yet.

I'm hoping to migrate away from ros2_rust needing cmake as a build tool as soon as feasible.

If rcl exported something like a pkg-config file, would using that be an acceptable solution?

"Hey, let's pretend I have a target that depends on this imported target. Please tell me all the include flags that we would give the compiler."

I don't know a good way, but if you want an ugly way:
$ cat /home/sloretz/ws/ros2/install/rcl/share/rcl/cmake/rclExport.cmake | grep INTERFACE_INCLUDE_DIRECTORIES | awk -F '"' '{print $2}'
${_IMPORT_PREFIX}/include/rcl

Yeah, having a pkg-config for rcl would be great. I only hesitate to ask for that because I know how much trouble it is after implementing it for ign-cmake. Without using a tool like ign-cmake the maintainers of rcl would need to make sure they update both the cmake information and the pkg-config information any time a dependency changes.
Maybe for rcl that's a worthwhile endeavor since it's meant to be accessible for bindings across many languages. And as long as the dependencies of rcl don't change very often, the maintenance shouldn't be a big deal.

if you want an ugly way:

This is roughly the same thing that doing cmake --find-package ... would provide. It doesn't include the transitive dependencies of imported targets, e.g. rmw, which is the troublesome part that we want cmake to provide.


[EventEngine] Fix use-after-free in the IOCP read callback

In rare cases across gRPC's end2end tests (only in Windows RBE with debug builds, interestingly enough), the endpoint may be destroyed before the final IOCP read callback has run. This edge case is only triggered when all the following are true:

the previous read operation received data (not an error, not 0-length), so more data is expected in the stream.
the socket has not yet shut down
the application destroyed its endpoint before (or during) the IOCP callback execution.
the Read operation has not yet called the client's on_read callback.

This is a valid scenario, and it is expected that the engine implementation should call the application callbacks with error statuses when this occurs. This PR fixes two associated bugs.
TODO: attempt to make this ordering part of the test suite, possibly as a stress test.

I confirmed that the event_engine_client experiments in the Windows RBE debug tests are now passing with this PR patched in: https://source.cloud.google.com/results/invocations/9f62b2c0-3101-4d23-905e-a54c9bc997c7/targets

Confirmed that the extra Windows build checks all pass.


add ci tests

@vncntprvst, @chyumin, @JuliaSprenger do you think you could use this structure to help put together functioning tests for each non-GPU singularity container?

yes definitely! thank you


Db Platform refactoring

This is a follow-up to #7059 against the latest develop branch. A common base class (AbstractPlatform) with some common implementations is already present there. This PR moves more methods to the base class, reducing the amount of duplicated code.
I double-checked that the critical quoteIdentifierChain(), quoteValue() and quoteTrustedValue() methods are inherited the right way. A second set of eyeballs should verify this, since the unit tests don't cover all possible characters that need escaping. The other quoting methods had already been ported.

@Ocramius Can you look over this, please?

@hschletz This needs a rebase; I cannot merge this as-is.
@Ocramius Thoughts?

Pinging @Ocramius — just need a :+1: or :-1: from you; I can do the merge.

Rebased against latest HEAD.

@weierophinney :+1:


Provide more detailed documentation for pyspark install & implementation

Is your feature request related to a problem? Please describe.
Trying to get mmlspark running in pyspark has been more challenging than it should be.  I'm able to run the latest version when launching a pyspark shell or spark-submit job but had some trouble getting the most up to date version of the package to my local machine to use in local development and importing into our testing environment.
Here are some of the issues I ran into:

Installing directly pypi using pip installed an outdated version of the python library. https://pypi.org/project/mmlspark/ currently points to mmlspark 0.0.11111111. Ideally, this library would be up to date.

Not great
pip install mmlspark

It is still possible to install a somewhat recent version via the wheel file but I don't see this documented anywhere and had to hunt for it.

Better
pip install https://mmlspark.azureedge.net/pip/mmlspark-0.17-py2.py3-none-any.whl

I'm unable to build directly from the git source as it looks like I'm missing some environment variables. Might be able to get this to work but don't have the time to dive in.

Errors out
pip install -e "git+https://github.com/Azure/mmlspark.git#egg=mmlspark&subdirectory=src/main/python"

Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/path/to/module/src/mmlspark/src/main/python/setup.py", line 9, in <module>
        version=os.environ["MML_PY_VERSION"],
      File<EMAIL_ADDRESS>line 40, in __getitem__
        raise KeyError(key)
    KeyError: 'MML_PY_VERSION'

Describe the solution you'd like
Easier to use pyspark install documentation. If it exists would love a link to it.

@njgerner mmlspark is a scala library, packed into jar archive. It contains python/R code, which is mostly wrappers/facades to native code, and never intended to use standalone.
For maintainers: This actually causes issues while developing in IDE - python syntax/dependency analysis fails (understandably) to resolve references to mmlspark. I'd propose to generate and distribute stub files library for mmlspark - this has been in practice for pyspark  for some time.


Better solution for passing data to another state?

Now I use localStorage, any other solution?

Hi @yosmelvin
Thanks for the question!
Since we're including ui-router amongst the starter modules, you can use the $stateParams variable to send data between states.
For example, you can send the name variable while going to the profile state like so:
$state.go('profile', {name: this.name});

and in the profile component you can use it like so:
this.name = $stateParams.name

That way you can send and receive parameters between the different components your states navigate to
Hope this help!

I am sorry, but its not working when I try your solution

My bad, I forgot to mention that when you want to add a state parameter to your states you need to declare it in the state definition.
So going back to my profile example, after the doing the above mentioned, one last step would be to declare/edit the profile state to have a variable in the url, like so
$state('profile, {
  url: '/profile/:name', //The name parameter you pass in the route
  data: {},
  views: {
  'main@': {
      templateUrl: getView('login')
   }
}

Hi @yosmelvin
Can you please confirm if this is working for you?
Thanks!

It's work Thank you, but one more question how to send with post method ?

Sounds great! 😄
Please open a new issue for the new question with a lot more details, it's unclear
Cheers


build: release candidate

:tada: This PR is included in version 1.5.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add sasmodels package

This is a dependency of the sasview package (PR will follow), which should be build first. Tested locally, builds fine. Ready for review/merge.
Also, some minor improvement to the building script, so that it can be executed from any location, not just from scripts/.

LGTM.

Thank you @licode!


Merge latest changes into 'Step 9'

Merge latest fixes from final into step-9 branch.
(This will make it easier to merge changes from final through to all branches in the future.)

LGTM, but looks like something went wrong in the last round of cherry picks. Going forward we need to cherry pick to final and then merge changes down into step branches. It looks like we cherry picked the changes into each branch last time which obviously creates new commits. Anyways after merging this down well be in a clean state again...


Improve badge padding and colors

I noticed the badge padding were uneven so I tried to improve that, plus I tried to make some of the colors less saturated to make it more consistent across the board.
Light mode
Before:

After:

Dark mode
Before:

After:

I've merged the color changes locally and will publish shortly. Thanks!
To me the padding changes make it worse... maybe it's browser / OS rendering differences. I'm on Mac Chrome.


Download folder

It should be possible to assign a separate volume for the download folder, allowing a clear segregation between data and config.
Otherwise nice app,

Volume mappings are configured while creating the container.  So users already have the ability to use a different volume for download purpose.

I realize it a little bit late but thanks. It works like a charm.

You can mount a different share in /downloads for example, and then set the preference to point to that folder.
docker run -d \
…
-v /local/path/to/downloads:/downloads:rw \
-e "FF_PREF_DOWNLOADS=browser.download.dir=\"\/downloads\"" \
…
jlesage/firefox

The only problem though: the container freaks out on it:
[cont-init.d] firefox-set-prefs-from-env.sh: Setting preference 'browser.download.dir'...
sed: bad option in substitution expression

As a workaround, mount the folder to /config/downloads after mounting a locale share to /config:
docker run -d \
…
-v /local/path/to/config:/config:rw \
-v /local/path/to/downloads:/config/downloads:rw \
…
jlesage/firefox

As a workaround, mount the folder to /config/downloads after mounting a locale share to /config:

The problem with this is that a mount point within another mount point has caused problems on some systems.

Volume mappings are configured while creating the container. So users already have the ability to use a different volume for download purpose.

I feel like I search for 20h now, to configure the location of a different download folder. Could you please give me a hint?

You can mount a different share in /downloads for example, and then set the preference to point to that folder.
docker run -d \
…
-v /local/path/to/downloads:/downloads:rw \
-e "FF_PREF_DOWNLOADS=browser.download.dir=\"\/downloads\"" \
…
jlesage/firefox

The only problem though: the container freaks out on it:
[cont-init.d] firefox-set-prefs-from-env.sh: Setting preference 'browser.download.dir'...
sed: bad option in substitution expression

As a workaround, mount the folder to /config/downloads after mounting a locale share to /config:
docker run -d \
…
-v /local/path/to/config:/config:rw \
-v /local/path/to/downloads:/config/downloads:rw \
…
jlesage/firefox

@jlesage  first of all, love all your containers...a
this workaround works, mapping a local folder location to the container downloads location. then I can ssh vIa filezilla and pull the file, but this is cumbersome. If I do a an external volume mappIng like I would for any other contaIner I get the following:

Oddly, when I have an external volume mapped to "/data/downloads" in the container I can ssh into the server, the container can see the volume contents and I can cp a file between that volume and the default downloads location of "/config/downloads", so I'm not sure where the permission issue lies.
I've tried the following mappings and neither work:
external_volume:/config/downloads:rw
external_volume:/data/downloads
the top one fails differently than the bottom one. the bottom one gives the pictured image, while the top method gives a "download failed" text from the download context section in the top menu bar of firefox

Looks like the user you configured for the container (USER_ID/GROUP_ID) doesn't have the permission to write to the mounted folder. Double check the owner and permissions of the mounted folder.

The only problem though: the container freaks out on it:
[cont-init.d] firefox-set-prefs-from-env.sh: Setting preference 'browser.download.dir'...
sed: bad option in substitution expression

-e "FF_PREF_DOWNLOADS=browser.download.dir=\"/downloads\"" is working for me.

Looks like the user you configured for the container (USER_ID/GROUP_ID) doesn't have the permission to write to the mounted folder. Double check the owner and permissions of the mounted folder.

yeah that's why I first thought too, so I reverted to 0 from 1000
same error from both ID#s

How did you create the external volume ?
Also, can you share the output of docker exec <container name> ls -ld /data/downloads ?

How did you create the external volume ? Also, can you share the output of docker exec <container name> ls -ld /data/downloads ?

drwxr-xr-x    2 root     root             0 Dec 28 17:08 /data/downloads
I created the external volume from the Volume section in Portainer. From my experience, that hasn't behaved differently from creating a volume via the 'docker create' command. I have dozens of volumes created in the same fashion without issue.

here's my compose while I"m at it:
firefox:
container_name: firefox
image: jlesage/firefox:v24.11.1
restart: unless-stopped
ports:
- 5801:5800
environment:
- PUID=0
- PGID=0
- UMASK=022
- TZ=America/Bromington
volumes:
- /data/compose/sourcing/firefox/config:/config:rw
#- /data/compose/sourcing/firefox/downloads:/config/downloads:rw
- external_volume:/data/downloads

drwxr-xr-x 2 root root 0 Dec 28 17:08 /data/downloads

Only the owner of the folder (i.e. root) have write access. If this is not expected, you can change the owner of the folder to match the one of the container:
docker exec <container name> chown -R <uid:gid> /data/downloads

Also:

- PUID=0
- PGID=0

Not sure where there are coming from, but these variables are not used. USER_ID and GROUP_ID should be used instead.

drwxr-xr-x 2 root root 0 Dec 28 17:08 /data/downloads

Only the owner of the folder (i.e. root) have write access. If this is not expected, you can change the owner of the folder to match the one of the container:
docker exec <container name> chown -R <uid:gid> /data/downloads

Also:

- PUID=0
- PGID=0

Not sure where there are coming from, but these variables are not used. USER_ID and GROUP_ID should be used instead.

Thank you! ok yeah I had the wrong variables. PUID and PGID are from other containers. Bad assumption on my part

Perfect, I think we can close this issue if it's working for everybody.


Create a discussion thread with meta resources for finding possible datasets

Add some initial ideas for finding datasets that might be suitable in particular:

overview papers that list other datasets
repositories that hold data that is likely to be of interest
Awesome lists

Initial thread here: https://github.com/bigscience-workshop/lam/discussions/31


LNBits support?

Howdy, just curious if LNBits (https://github.com/lnbits/lnbits-legend#lnbits) support is still on the table. The link in the docs seems to be dead.
Thanks!

Probably worth asking on the LNBits repo if you haven't yet, I'm not involved with the project myself.

My understanding is LNBits has full support for Lightning Addresses now.

I wasn't aware you can generate and receive to an LNBits wallet via a LNAddress provided by LNBits itself. Anyone have a link to some docs?

Honestly I don't use LNBits that often so wouldn't be the best person to give input here. Maybe @fiatjaf can help?

The docs are: you must enter the jungle Telegram group and ask. Or create an issue on the LNbits repository.
LNbits itself doesn't offer addresses because its accounts don't have names, but the last time I looked there was an extension that allowed you to point an external domain to an account, much like https://github.com/nbd-wtf/satdress does too (see https://payaddress.co/, for example). Or maybe I am missing something.


First run. PHP notice. Whats wrong?

I did everything according to instructions - https://luya.io/en/guide/install

Hi

Have you run the console commands? ./vendor/bin/luya migrate ./vendor/bin/luya import ./vendor/bin/luya setup
Can you login to your Administration area? yourdomain.com/admin / http://localhost/luya-kickstarter/admin
The errors means, that you have not yet set a Homepage/Default Page. (The Exception is improved in the next beta2 release.) So if you can login to your administration area (with your credentials you have provided during the setup process), create a page and click the "is home" icon.

If you do have further problems, let me know.

yes

yes

so empty sidebar. it should be?

@Emericanec yes, this looks good! Now click on "Gruppen" -> "Berechtigungen". An overlay will popup where you can select "Alle markieren", scroll to the bottom and click "Speichern". Now your group (where you are in) does have all privileges and now you will see all modules.
btw. we are going the deliver translations for the administration area in the next release! So you could help us translate everything in your native language, if you like to!

Done! You can close the task.

Yes, I'm going to be contributor. I are pleased to help with the translation into Russian.

Maybe should turn on default main page?

good point! creating a default page when running the setup command. You could create a new issue for this?


Cant Load LUTs V2

Hi guys, I've read all this thread https://github.com/fat-tire/resolve/issues/47
and I tell you? the most curious thing was that I was solving this problem as the author quoted in his final paragraph:

A final thought-- you can also add your own LUT Locations in the settings: From a project, in the top menu bar, choose DaVinci Resolve->Preferences. Make sure the System tab is active, then under General, you can select LUT Locations. Then select Add or Remove.

It happened that the disk where I saved the presets got burned (it was an old NTFS HDD), so today I went to try to put my LUTS backups in its place to replace it.
Apparently davinci allows me to import the place in the external path that I had created, but the problem is that it never loads them, they don't appear anywhere, neither the folder names.

All my files of docker (no images) are in /home/noe/Documentos/resolve/
I was using a normal rute in RESOLVE_BIND_TARGETS, but now trying solve this also I've used /home/resolve/.local/share/DaVinciResolve/.LUT/
RESOLVE_BIND_SOURCES=(... "/home/noe/Documentos/resolve/Davinci-Presets/MyLUTS/")
RESOLVE_BIND_TARGETS=("... " "/home/resolve/.local/share/DaVinciResolve/.LUT/")

But nothings works, also try from my real machine use chmod -R 777 over Luts folder, chown -R noe:noe, and the permissions inside of the podman dont allow chage it.
Could be it a permissions error?
Note, add information (try mount the LUTS in home/resolve/Davinci-Presets/):
From inside podman
./resolve.sh /usr/bin/bash
bash-4.4$ cd home/resolve/Davinci-Presets/
bash-4.4$ ls -l *

Mios:
total 1272
-rwxrwxrwx 1 resolve resolve 431272 Aug 12 08:50 LUT-Dehancer-Rec.709-Kodak_Vision3_250D-33.cube_shaperlut
-rwxrwxrwx 1 resolve resolve 431272 Aug 12 08:50 Ludwig-Dehancer-Rec.709-Kodak_Vision3_250D-33.cube_shaperlut
-rwxrwxrwx 1 resolve resolve 431272 Aug 12 08:50 ludwig-LUT-Dehancer-Rec.709-Kodak_Vision3_250D-33.cube_shaperlut

I am not sure if it error is somthing of permission but while i keep in the resolver script rute, in my case: /home/noe/Documentos/resolve/ I cant mount files from here
Now I've put my luts of my real machine  outside from resolve folder
Real machine other folder:
/home/noe/Documentos/LUTs/
And now can map it
RESOLVE_BIND_SOURCES=(... "/home/noe/Documentos/LUTs/")
RESOLVE_BIND_TARGETS=("... " "/home/resolve/Davinci-Presets/")
and add /home/resolve/Davinci-Presets/ to :

A final thought-- you can also add your own LUT Locations in the settings: From a project, in the top menu bar, choose DaVinci Resolve->Preferences. Make sure the System tab is active, then under General, you can select LUT Locations. Then select Add or Remove.

Now works without problem.
Surely it is a permission problem, why? Because when i tried modify files from resolve forder of my realmachine (the folder that have resolve.sh), it always ask for root permissions.
With it has been resolved for now.
Thank u very much !

I'm glad it's working...  podman is a little strange with the permissions since you're running rootless, but as long as you've got it working, that's great to hear!

I'm gonna close this.  If you want to re-open it or think it should be moved to a discussion, let me know!  Thanks.

I'm gonna close this. If you want to re-open it or think it should be moved to a discussion, let me know! Thanks.

Hahah its true, man you very much bro !!


Split GuardDuty Checks for Foundational Security into Standard GuardDuty Enabled and S3 Logging Enabled

Is your feature request related to a problem? Please describe.
Today the check is reporting in alarm if the S3 logging is not enabled. I think it would be better to split this into 2 checks. 1 for regular GuardDuty enabled and one for S3 Logging enabled.
Describe the solution you'd like
Split 1 check into 2.
Describe alternatives you've considered
No real alternatives exist except ignoring the finding if S3 logging is not enabled but regular GuardDuty is.
Additional context
Existing logic is here.
(https://github.com/turbot/steampipe-mod-aws-compliance/blob/main/query/guardduty/guardduty_enabled.sql)

Hey @twratl , thanks for suggestion! Looking at the control for GuardDuty.1, it recommends enabling GuardDuty in the region, but I'm not sure if it specifically mentions S3.
@rajlearner17 Do you recall why we check for S3 logging? Is it based on a detail in the Config rule associated with that control?

@twratl Thanks for the suggestion
@cbruno10 Investigating this and the thought process behind the same. Will keep posted

@cbruno10 @twratl After checking in detail, as suggested by @twratl, there is no need of checking S3 logging, which seems to be an additional unwanted check. I can't recall the circumstances during that time. While working 100s of compliance controls, which differ slightly in some places, we might have used this one. We will be fixing this soon for release.


Fix copying/applying font fallback with package

Fixes the duplicate package prefix in the fontFamilyFallback when copying or applying a TextStyle.
Fixes https://github.com/flutter/flutter/issues/118392
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

Thanks for the fix, @IchordeDionysos !

@jason-simmons @mdebbar can we merge the PR now? :)


how to install

Hello,
Can you give docker image ?

simply run python3 main.py and meet its requirements such as aiotg may meet your need.
If you persist in running on docker, pulling a alpine docker image with python3 and run it is a solution.


Question: Why the Handler Wrappers?

Hi,
Like the titles says, this is not really an issue, but just a question. I was looking around the code (kudos by the way :+1: ) and I notices the double Generic caches of ConcurrentDictionaries to hold the generic types of both the Generic Handler and a default Handler Wrapper.
While I do understand the first, because the type of the generic IRequestHandler<> is not know and you want to save up the MakeGeneric call, I did not get to find a reason for the Wrappers, since all they do is to call the inner handler. I did not find any hook on them or usages besides the inner call.
Did I over looked something?
Thanks in advance, and kudos again for this awesome library. It has already become a standard on both my personal and my team's professional projects.

@kappy, as far as I can tell, they facilitate calling the correct method on the implemented handler at runtime. They are used to convert from the request type to the closed generic interface that the concrete handler implements.
I tried writing my own version of a simple IAsyncRequestHandler<TRequest, TResponse> : where TRequest : IAsyncRequest<TResponse> and found that the runtime could not call the Handle method on my handler with the given parameter.
AsyncHandler:
public interface IAsyncCommandHandler<TRequest, TResponse>
    where TRequest : ICommand<TResponse>
{
    Task<TResponse> Handle(TRequest request);
}

Mediator:
public Task<TResponse> ExecuteAsync<TResponse>(ICommand<TResponse> request)
{
    var requestType = request.GetType();
    var handlerType = typeof (IAsyncCommandHandler<,>).MakeGenericType(requestType, typeof (TResponse));

    dynamic handler = _lifetimeScope.Resolve(handlerType);

    return handler.Handle(request); // Throws runtime exception as it can't find the correct overload
}

The parameter passed to Handle is ICommand<TResponse> rather than the concrete type of my request. The wrapper that MediatR uses somehow binds the call correctly but I can't help much more than that without @jbogard stepping in :)

It's funny generics stuff. Basically the only way I could get the compiler to be happy.

@jameshulse did you come up with a solution for this? @jbogard 's generic-fu is pretty hard to fathom, especially if you have a much simpler implementation (e.g. no pipeline). All those request handler wrappers are black magic as far as I am concerned!

@davidkeaveny I'm afraid not. After failing to repeat the black the magic for myself I believe I gave up. Jimmy is a magician - it might be best just using MediatR. Good luck!


RSA Algorithm

Fixes #
Short description of what this resolves:

What Changes proposed in this pull request:

Checklist

[x] Added description of change
[x] Added file name matches File name guidelines
[x] Added code examples, test must pass
[x] Added documentation so that the program is self-explanatory and educational.
[x] Relevant documentation/comments is changed or added
[x] PR title follows semantic commit guidelines
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] I acknowledge that all my contributions will be made under the project's license.

Notes:

@shrejal99 we don't accept this kind of contribution(On this site, we are just adding notes of DSA-JAVA-Bootcamp), before making a PR, please raise an issue, and if an issue gets assigned to you, then you can start working on that.
Thank you!


jaqal code generation

We should implement a compiler that enables translation of openqasm to jaqal via https://gitlab.com/jaqal/jaqalpaq-extras/-/blob/master/src/jaqalpaq/transpilers/qiskit/frontend.py#L40
i think we can really only consider a JaqalCompiler::translate implementation that delegates to the StaqCompiler to map IR to OpenQasm, then delegate to the above qiskit transpiler frontend to generate a jaqal circuit.

https://gitlab.com/jaqal/jaqalpaq/-/blob/master/src/jaqalpaq/generator/generator.py
this will generate jaqal as a string

https://gitlab.com/jaqal/jaqalpaq/-/blob/master/src/jaqalpaq/generator/generator.py
this will generate jaqal as a string


HDDS-5009. Set the default output format based on the newest version …

What changes were proposed in this pull request?
1.Set the default output format.
2.Add a note about the output format reminder of the new and old versions of async profiler.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-5009
How was this patch tested?
Local test.

new ozone-runner (with updated async-profile) is available. Please change the suggested variable in the pom.xml to 20210329-1

Thanks for the reminder from @elek , I will modify it as soon as possible.

@cku328 When squashing PR with only one commit, its message is used as is by default.  In this case one might need to tweak it to include Jira number etc.


Simple changes in a scala multi project build take very long to compile preventing quick iteration

There's  no compiler avoidance for Scala multi project builds.
Making a one line change such as adding a comment in a project that depend on it to be rebuilt.
There's no incremental compilation for the upstream projects either making it painful to  do a simple change and quickly run some tests.
Expected Behavior
Scala uses the @CompilerClasspath similarly to Java to detect ABI compatible changes and skips re-building upstream projects.
Current Behavior
There's no compiler avoidance for Scala.
Context
We switched to building a large multi project Scala project with Gradle and have been mostly happy with it, but ran into some challenges for local development in small iterations of changing a few lines and running tests. We are working around these by using bloop and the bloop Gradle integration, but we'd rather use a single build tool with a single IDE setup. We use IntelliJ so our IDE builds default to Gradle too.

Could you pls. provide us with a reproducer? Will make this issue much simpler to investigate.

@jbartok excited to see this get some traction this is still super important for us!
Please see this sample project, it uses Java classes instead of Scala but compiles them with the Scala plugin, so they behave like Scala classes for the purposes of this issue:
scala-sample.tar.gz
Here's the first build of the project: https://scans.gradle.com/s/v4l2vtpvshlr4/timeline  I then edited Greeter.java to change the String value it returns from "Hello Mars" to "Hello Earth" and built again https://scans.gradle.com/s/cq7vt3iiiz2qw/timeline?details=rblabgyu6onns . Note that on the second run the root project was re-compiled even trough there were no ABI changes in :dependency.  Thus the feature documented here for Java https://docs.gradle.org/current/userguide/java_plugin.html#sec:java_compile_avoidance is not implemented for Scala, even trough Scala compilation times are much longer and Scala projects would benefit more.

A bit late to the party. @alpar-t still impacted by this issue?
We faced similar issues with multiple project builds and an issue in zinc was the cause. See https://github.com/sbt/zinc/issues/1140#issuecomment-2200255982


[Integration] Switch to quay or fedora registry instead docker

Docker recently limited the pulls for anonymous user and something the integration tests fails because of that.
21:52:06     command 'sudo podman build -t hello:test -f Dockerfile', expected to succeed, exited with exit code: 125
21:52:06 Command stdout: STEP 1: FROM centos:latest
21:52:06
21:52:06 Command stderr: time="2020-11-26T16:21:36Z" level=error msg="HEADER map[Cache-Control:[no-cache] Content-Type:[application/json]]"
21:52:06 time="2020-11-26T16:21:38Z" level=error msg="HEADER map[Cache-Control:[no-cache] Content-Type:[application/json]]"
21:52:06 time="2020-11-26T16:21:42Z" level=error msg="HEADER map[Cache-Control:[no-cache] Content-Type:[application/json]]"
21:52:06 time="2020-11-26T16:21:50Z" level=error msg="HEADER map[Cache-Control:[no-cache] Content-Type:[application/json]]"
21:52:06 Error: error creating build container: The following failures happened while trying to pull image specified by "centos:latest" based on search registries in /etc/containers/registries.conf:
21:52:06 * "localhost/centos:latest": Error initializing source docker://localhost/centos:latest: error pinging docker registry localhost: Get https://localhost/v2/: dial tcp [::1]:443: connect: connection refused
21:52:06 * "registry.access.redhat.com/centos:latest": Error initializing source docker://registry.access.redhat.com/centos:latest: Error reading manifest latest in registry.access.redhat.com/centos: name unknown: Repo not found
21:52:06 * "registry.redhat.io/centos:latest": Error initializing source docker://registry.redhat.io/centos:latest: unable to retrieve auth token: invalid username/password: unauthorized: Please login to the Red Hat Registry using your Customer Portal credentials. Further instructions can be found here: https://access.redhat.com/RegistryAuthentication
21:52:06 * "docker.io/library/centos:latest": Error initializing source docker://centos:latest: Error reading manifest latest in docker.io/library/centos: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit

As you can see multiple registries are specified, but the can't satisfy the requested build.
This might be a specfic change that needs to be done to only the integration tests dockerfile? Like specifying the registry in the FROM statement or redefine the test to use an image that exists on registry.redhat.io

I'll take a look. Thanks for spotting it.

Done in #1726


implemented tap-macro-release

@6gk wanted a feature that allows him to tap a button on keypress and another on keyrelease. So i made a small patch for him with a new button called tap-macro-release which works just like tap-macro, but in contrast it taps the last button on keyrelease. I don't know if this is actually needed in kmonad or if it wouldn't bloat it, but i'll just drop the patch off.

Wait, i just realized that i forgot to fix the documentation :S

oh nice! I'm gonna study this and see if I can understand the code.

So, I actually agree with a lot of @slotThe's comments about layouting and how this maybe should be an option instead of its own button. However, I merged this as is because:
A. The code is really ugly right now, because I am also just flying through like a hurricane. Once all the new stuff is in and all the old stuff is gone, I'm going to go through and make sure everything is tidy and documented.  I'm actually hoping to really redo the whole way we specify buttons, so although this button is very similar to tap-macro, and I can't really see a use-case difference, I think in a few months the whole way we specify buttons might be redone anyways, so I find it hard to be very insistent on parsimony.
So, congrats on your Haskell contribution! :-)

It seems like that is soon to replace KMonad.Args, but the latter is simply not removed yet. I suppose @david-janssen will know more :)

Well, not really remove, just move. Changing a little bit at the fringes of those modules, bup reorganizing a lot of our code into some more distinct categories. After the move to the organization I thought I should just keep commiting non-breaking changes, but then I realized it's better to just work on my own tinkering branch, and merge it all in when done. So that was one of the last changes I made to master. I've been tinkering over here: https://github.com/david-janssen/kmonad/tree/tinker


problem with svg validation test

PR #784 test failed. Specifically the reference image reportedly failed validation:

apple_podcasts.svg
should be validated by the w3c validator:
AssertionError: expected ':5.54-5.81: error: SVG element “stop”…' to be empty
at /home/runner/work/SuperTinyIcons/SuperTinyIcons/test/test.js:34:38

The problem appears to be that the reference image uses <stop></stop> instead of <stop/>. But those two constructs should be treated as equivalent. The W3C validator at https://validator.w3.org/ reports that the svg is, in fact, valid.
I checked test.js. It appears to use the W3C's nu validator:
it("should be validated by the w3c validator", async () => {
            await fetch("https://validator.w3.org/nu/?out=gnu", {
                    method: "POST",

I note that the NU validator advertises itself as "an ongoing experiment in better HTML checking." So maybe it's not parsing the xml/svg correctly?

Reference images aren't being validated, so the issue is in your apple_podcasts.svg file. Line 5, columns 54-81 are the first stop tag in your file, so something's wrong there. I believe the issue is that the offset property is not required in SVG 2, but it is required in SVG 1.1 (assumed default).

Thanks for the reply. The validation message was rather unclear to me. After reading your reply, I now realize the numbers in the message are a reference to the line and character.
Further confusing things, I had trouble with the w3c validator. If one submits my code via file upload or uri -- and uri is the method that SuperTInyIcons' test script uses -- it is validated as SVG 1.1 + URL + HTML + MathML 3.0, and produces the missing attribute error. But I used the direct input method, and in that case, the service validates it as xml and it passes. It took some time to notice the different parsing/validation.
I'm not sure if that is by design or necessity, but I've filed an issue in the w3c validator repo.


Send notification if somebody added me as reviewer

It should speed up the review process a little bit.
Related event:
https://gerrit-review.googlesource.com/Documentation/cmd-stream-events.html#_reviewer_added

Neat idea. Implemented, testing right now.


Null in data filter

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Tests pass?
✔

Fixed issues
-

This enables providing null value in filters when there is no option to use literal null (like through URLs). I have seen SO questions asking for this several times.
I have put this on 2.0.40 to discuss it more before releasing (if at all). Automatic change from 'NULL' to null might be messing apps using 'NULL' strings currently so maybe it should be '$NULL'? I hope this change fills the gap in filters usability but maybe I forgot about something so please take a look if you are interested. All comments are welcomed.

Overall the implementation is OK. I don't see where it may break existing apps.
Any refereces to where it was asked?

https://stackoverflow.com/questions/64630624/yii2-how-to-get-null-and-not-null-filtered-results-using-yii2-rest-web-service
https://stackoverflow.com/questions/51876668/how-to-find-records-with-null-values-in-a-column-in-yii2-datafilters-or-rest-api
The current solution right now is to make custom handling so it's sometimes inconvenient.

Alright. Let's have it in the next release (current one has enough changes already).

👍


[HOLD for #19007][$1000] Primary login account is added to split bill participant list and amount also gets divided changing default contact method

HOLD for https://github.com/Expensify/App/issues/19007
If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Precondition: account should have primary login account A, and secondary login account B

Click on 'FAB' icon > click on 'Split bill' link > enter amount > click on 'Next' button
Select two users
Click on 'Split' button
Click on IOU preview component & verify that total of 3 participants are displayed in the details section
Click on 'Profile' icon > click on 'Contact methods' link > click on secondary login account B > click on 'Set as default' button
Close RHN & refresh the page for the changes to take effect
Click on the IOU preview component

Expected Result:
Primary login account A shouldn't be added to the split bill participant list
Actual Result:
Primary login account A is added to the split bill participant list
amount is also divided between primary login account and secondary login account on changing default contact method.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

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

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: 1.3.22
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/66585e83-865d-4891-93b0-b98950edfe32
https://github.com/Expensify/App/assets/43996225/165d8a92-0bfb-4fb0-8ef9-ffb955d46e55
Expensify/Expensify Issue URL:
Issue reported by: @natnael-guchima
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1685434411591889
https://expensify.slack.com/archives/C049HHMV9SM/p1685430101536709
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01b4c1bac53442a0dd
Upwork Job ID:<PHONE_NUMBER>030091264
Last Price Increase: 2023-06-02

<EMAIL_ADDRESS>created the split, then changed the default contact method to<EMAIL_ADDRESS>Before the default contact method change:

After the default contact method change:

Proposal
Please re-state the problem that we are trying to solve in this issue.
The primary login account is added to the split bill participant list and the amount also gets divided changing the default contact method
What is the root cause of that problem?
reportAction.originalMessage.participants not updated to the recent default contact method.

We started to use participants data here:
https://github.com/Expensify/App/blob/0813e2ca1476635a8324cbd9f998424663a33128/src/pages/iou/SplitBillDetailsPage.js#L67
That makes users' old default contact added to split participants.
What changes do you think we should make in order to solve the problem?
Update the originalMessage participants to the current default contact method in the backend.
What alternative solutions did you explore? (Optional)
N/A

@thesahindia have you had a chance to review the proposals?

Okay will ask internally to see if anyone knows about this.

@tienifr I think using the accountId makes sense and we are actively working now to make sure we are re-keying the personal details with accountIDs so I believe this should work well

Actually we discussed this internally and we're refactoring personalDetails by accountID already. So let's hold on that so that we don't duplicate work

Still on hold for https://github.com/Expensify/App/issues/19007

@madmax330 I see that https://github.com/Expensify/App/issues/19007 is now on Production, so I think we can take this off hold, re-test and fix. Do you agree?

Yeah seems like we can take this off hold

Off-hold, ready to re-test and fix (if it still exists as an issue)

Removing from the manual request project since this is a wave2 issue.

@tienifr What are you asking for as compensation? We can certainly discuss it!

Ok, I agree with that! I'll queue up contracts via Upwork.com now (the job post is closed so I need to do that manually)

[ ] External issue reporter - @Natnael-Guchima: $250 in this offer, and contract TBC
[ ] Contributor that fixed the issue: @tienifr: $1000 in this offer, and contract TBC
[ ] Contributor+ that helped on the issue and/or PR: @thesahindia: $1000 - please send Manual Request via New Expensify
[ ] Urgency bonus: N/A
Original Upwork job: https://www.upwork.com/jobs/~01b4c1bac53442a0dd

Accepted the offer. Thanks @conorpendergrast

Paid @tienifr 👍 Closing, as @thesahindia will send a Manual Request.

$1,000 payment approved for @thesahindia based on BZ summary.


Allow footer configuration

Introduced some new configuration options. These are the default values:
params:
  footer:
    enable: true
    navigation:
      enable: true
    contactMe:
      enable: true
    credentials:
      enable: true
  newsletter: # not new, but similar to new options
    enable: false

Where should the documentation go?
Please note that I am technically a web developer, but not familiar with hugo in the slightest. I am very happy about constructive feedback.
Btw it might be easier to review the commits one by one because the indentation makes the diff look more crazy than it is.
Issue
https://github.com/hugo-toha/toha/issues/224
Doesn't solve that issue completly as it is about the footer and the navbar. The navbar was a bit over my head at this moment.
Description
Allows to disable/enable parts of the footer (or all of the footer).
Test Evidence
Footer disabled

Navigation disabled

Footer enabled, but all parts of it disabled

Copyright notice disabled

@Kaligule  Thank you for the PR. I think footer should be just another section so that user can use custom template for it.

@hossainemruz I don't think I can help there. Do I understand you correctly that you do not want to merge this PR but work on your own solution? (I this case I will have to point my site to my own fork until you succeded.)

Cool, thanks :)


ctx.portfolio is undefined

I created a details button to show project's details, but when I clicked, the info about project doesn't appears and in console shows this error message:
ERROR TypeError: ctx.portfolio is undefined
To Reproduce
Steps to reproduce the behavior:

Go to 'Experiência'
Click on 'Detalhes'
See error

Expected behavior
The details about project appears

This issue was resolved and you can view the solution here: #19


Input simulation tutorial uses MRTKv2 terminology

Pointers do not exist in MRTK3. Some terminology:

Ray interactor: the interactor that shoots a ray out into the scene to provide far interaction
Reticle: the visual indicator at the end of the ray
Select: the generic term for the primary action from an interactor used to manipulate an object or click a button

Terminology has been updated to reflect MRTK3 changes.


Ability to enter in hex color codes w/ rounding

Enables users to enter hex codes instead of color names for a color parameter. Instead of preventing invalid hex codes, we round them to the nearest Pebble color.

Other than the formatting comments, this is otherwise a nice approach. My only concern is if the mobile JS interpreters are able to understand that some functions and arrays are actually static and don't need to be reallocated every invocation.

Looks good! I'm going to test it out with the demo app, and a larger test app.


AttributeError: module 'qtpy.QtCore' has no attribute 'pyqtBoundSignal'

Hello,
When running the PyRPL GUI, when I try to load any module I get the error below, and nothing else happens:
AttributeError: module 'qtpy.QtCore' has no attribute 'pyqtBoundSignal'.
I installed PyRPL via the conda command on the documentation - I'm running it on MacOS Catalina. I checked on python command line and I have PyQT5, and it has the class PyQt5.QtCore.pyqtBoundSignal, however qtpy.QtCore.pyqtBoundSignal does not exist. I'm saying this in case it helps, I thought pyqt would wrap PyQT5.

I took a look at the my installed version of pyqt (using version 1.11.2) and it seems it renames the pyqtBoundSignal object:
from PyQt5.QtCore import pyqtBoundSignal as SignalInstance.
So I looked at older versions of qtpy, and I installed version 1.9.0 which doesn't rename the pyqtBoundSignal, so now I can load the modules I tried, IQ and SpectrumAnalyzer.
Is it simply the case that pyRPL is meant to run with version 1.9.0 of qtpy or did I miss something along the way ?

I believe this bug has been fixed in the branch python3-only

Thanks for the reply, that seems to be the case. Is there one package version that I could install with pip that compiles that branch ? Or I should just install it manually from source ?

Nope, I don't think so... also @lneuhaus, I think now python3-only was merged into master... Probably we should just remerge the few new commits that were made on python3only into master and delete it definitively ?

I got the has no attribute 'pyqtBoundSignal' error when using pyrpl from master branch with Python 3.7, but the python3-only branch seems to work fine.


Feature: Disable unused args check for stub functions

Keywords: ARG, ARG001, unused argument
Feature
It'd be great if we could have some settings to disable the unused argument check for stub and abstract functions, such as one that only throws NotImplementedError and does nothing else. The Flake8 plugin has a relevant setting for this.
To Reproduce
This code:
def foo(a, b):
    raise NotImplementedError

Command: ruff check --isolated --select ARG repro.py
Output:
repro.py:1:9: ARG001 Unused function argument: `a`
  |
1 | def foo(a, b):
  |         ^ ARG001
2 |     raise NotImplementedError
  |

repro.py:1:12: ARG001 Unused function argument: `b`
  |
1 | def foo(a, b):
  |            ^ ARG001
2 |     raise NotImplementedError
  |

Found 2 errors.

Configuration
The output is the same if I don't run it in isolated mode. However, in case you may find it useful, this is  the relevant part of my Ruff configuration in pyproject.toml.
[tool.ruff.lint]
extend-ignore = [
    "COM812",
    "D105",
    "EM",
    "FBT",
    "PLR2004",
    "S311",
    "TRY003",
]
select = ["ALL"]

Platform

Python: 3.12.3
Ruff: 0.6.1

For some reason we already do this for all of the ARG rules except ARG001.


[BACKLOG-19589] Fix for dialog message issue found by UX

@pentaho/r2d2
:)

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -Dsurefire.runOrder=alphabetical -B -fn -f 'pom.xml' -pl 'ui' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl 'ui' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'ui' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  ui/src/main/java/org/pentaho/di/ui/repository/repositoryexplorer/controllers/BrowseController.java
  ui/src/main/resources/org/pentaho/di/ui/repository/repositoryexplorer/messages/messages_en_US.properties

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes: <0.1% (click to expand)
org.pentaho.di.trans.steps.salesforceupsert.SalesforceUpsertMeta

Instruction Change: -0.1% :small_red_triangle_down:


Optimize comparison kernels

This avoids the conditional jumps, which is faster when the pattern is not super predictable (which is the case for %10).
I also "fixed" the benchmarks, as they currently run on two arrays with equal elements as they use the same seed and introduced a lt version.
For arrays with a high "matching percentage" this has a huge performance impact (see lt benchmark, this is 5-10x faster), but a low matching percentage (eq on random floats) it  is slower to compare two arrays. The eq benchmark will now match probably never, so the CPU can predict those branches perfectly in the version on main.
Benchmarking eq Float32: Collecting 100 samples in estimated 5.1909 s (121k ite                                                                               eq Float32              time:   [41.836 us 41.890 us 41.947 us]
                        change: [+65.172% +65.623% +66.045%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 2 outliers among 100 measurements (2.00%)
  2 (2.00%) high mild

Benchmarking eq scalar Float32: Collecting 100 samples in estimated 5.0676 s (2                                                                               eq scalar Float32       time:   [20.758 us 20.789 us 20.826 us]
                        change: [-7.0110% -6.7414% -6.5092%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 7 outliers among 100 measurements (7.00%)
  2 (2.00%) low mild
  3 (3.00%) high mild
  2 (2.00%) high severe

Benchmarking lt Float32: Collecting 100 samples in estimated 5.1891 s (136k ite                                                                               lt Float32              time:   [37.666 us 37.725 us 37.789 us]
                        change: [-82.416% -82.241% -82.070%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 2 outliers among 100 measurements (2.00%)
  2 (2.00%) high mild

Benchmarking lt scalar Float32: Collecting 100 samples in estimated 5.0062 s (2                                                                               lt scalar Float32       time:   [21.252 us 21.272 us 21.290 us]
                        change: [-90.458% -90.437% -90.415%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 6 outliers among 100 measurements (6.00%)
  2 (2.00%) low mild
  2 (2.00%) high mild
  2 (2.00%) high severe

Codecov Report

Merging #17 (0e4cba3) into main (65d434c) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #17      +/-   ##
==========================================
- Coverage   63.31%   63.31%   -0.01%
==========================================
  Files         150      150
  Lines       14424    14422       -2
==========================================
- Hits         9133     9131       -2
  Misses       5291     5291

Impacted Files
Coverage Δ

src/compute/comparison/primitive.rs
94.97% <100.00%> (-0.06%)
:arrow_down:

Continue to review full report at Codecov.

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

There is also a strange part in the benchmark here.
    let arr_a = create_primitive_array::<f32>(size, DataType::Float32, 0.0);
    let arr_b = create_primitive_array::<f32>(size, DataType::Float32, 0.0);

As the left & right arrays are using the same seed, they have exactly the same elements!

Thanks @Dandandan . It was a lot of fun to go through this with you. :)


Using BrowserSync?

Im trying to add BrowserSync but nothing shows up, I was wondering if anyone has ever tried this before?
I both those solutions I found:
1-
// frontendMiddleware.js
const browserSync = require('browser-sync');
browserSync({
server: {
baseDir: path.resolve(process.cwd(), 'build'),
middleware: [middleware, webpackHotMiddleware(bundler)],
},
 files: [ /\.html/, /\.css/, ],
 });
2-
//webpack.dev.babel.js
const BrowserSyncPlugin = require('browser-sync-webpack-plugin');
new BrowserSyncPlugin(
{
// browse to http://localhost:3000/ during development
host: 'localhost',
port: 3000,
// proxy the Webpack Dev Server endpoint
// (which should be serving on http://localhost:3100/)
// through BrowserSync
proxy: 'http://localhost:3100/',
},
// plugin options
{ reload: false })
Thanks!

Closing due to inactivity. I have never personally used BrowserSync, so I can't help you here. Hope you can work it out!

See https://github.com/mxstbr/react-boilerplate/pull/1024


Device type field

Minor problem: the protocol returns device type "diy_plug" for all devices.  Should it instead return the real type, i.e. "BASICR3"?  In case the protocol will be different for different devices, and also to be able to display the correct device type in the client app?

@ZZLinvec  If you need any help - let me know

@ZZLinvec why is it closed - is it fixed?


Add TypeScript declaration for SLD object created by xml2js

We should add a declaration for the JS objects generated by xml2js when reading an SLD.

Actually it is not need as it never gets outside of the library.


feat: add gitopia [web-gitopia]

Description
Closes: BDU-956

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] ran linting via yarn lint
[ ] wrote tests where necessary
[ ] included the correct type prefix in the PR title
[ ] targeted the correct branch
[ ] provided a link to the relevant issue or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed
[ ] added a changeset via yarn && yarn changeset

@rachelhox I think we have the same issue with validators moniker not showing correctly on the list, could you update it like you did with other chains pls? Thanks

I think this issue has been occurring for more than 2 chains now, so will rewrite the current logic we have for processing all validators. Let's merge this first, then I'll fix it in the future.


Ignore error when killing, if error is 'process does not exist'

Currently when killing a container, if the container process does not exist (already deleted) or when killing all processes in the container, a process exits between we getting its pid and sending the kill signal, kill command will produce error.
However, in this specific case, it is ok that killing failed, as the process to kill does not exist. So we ignore the error.
Note, I'd like the check of error code to be more idiomatic, but not sure how

May I ask you to add this test case into our integration test?

I'm not sure what you mean by this. If we want to add a new test to validate this, I'm not sure how would I go about doing this, because this is basically a race condition. The problem here was that between we getting list of processes and we sending the kill signal, a process from list exits on its own, making the kill call fail. As we want to kill the process anyways, and it has exited, here we just ignore that specific failure. In my opinion, it would be difficult to consistently recreate this race condition in a test.
Let me know if I'm getting this wrong

May I ask you to add this test case into our integration test?

I'm not sure what you mean by this. If we want to add a new test to validate this, I'm not sure how would I go about doing this, because this is basically a race condition. The problem here was that between we getting list of processes and we sending the kill signal, a process from list exits on its own, making the kill call fail. As we want to kill the process anyways, and it has exited, here we just ignore that specific failure. In my opinion, it would be difficult to consistently recreate this race condition in a test.
Let me know if I'm getting this wrong

You are right!


Fix V2 filters leak

by passing on the literal class-level filters object to an instance of an V2 class, those instances are never able to get GC'ed.
Essentially every single V2 model instance would be stuck in memory.
This passes in a copy of the filters content and should behave the same way but allow the objects to get cleaned up.

fwiw: heroku's dashboard saw a 50% reduction in test memory leakage with this


Add IoT Operations extension v0.3.0b2

Notes for this release can be found here:
https://github.com/Azure/azure-iot-ops-cli-extension/releases/tag/v0.3.0b2

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[ ] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.


chore: make the LiteralContent enum public

Makes the LiteralContent enum public to enable pattern matching.

Would it be enough to cover your use case?

It does indeed - though a minor annoyance with that method is that the type is a little bit looser i.e. one needs to handle cases like (value, Some(...), Some(...)) which never occur in practice.

It does indeed - though a minor annoyance with that method is that the type is a little bit looser i.e. one needs to handle cases like (value, Some(...), Some(...)) which never occur in practice.

Indeed, that's true. However I have to say I never used the destruct method myself and always manage to just use the value, language and datatype accessors.

To make it work you need to do 3 things:

make LiteralContent pub (what your patch was doing)
make the inner field of Literal pub (what your patch is doing now)
because the literal module is not public, reexport Literal content in the root of the library by adding pub use crate::literal::LiteralContent; in the lib.rs file.

Anyway, I am not 100% sure it's a good change to make because it leaks the Literal implementation internals.

To make it work you need to do 3 things:

Makes sense now - thankyou for explaining!

Anyway, I am not 100% sure it's a good change to make because it leaks the Literal implementation internals.

Gotcha - closing the PR now


Fix serde text serialization for XOutpoint

Alternative to #215 preserving serialization for non-YAML types

It seems a better approach than https://github.com/RGB-WG/rgb-core/pull/215 but I'm not sure it works because to try it we need to fix usages of XOutpoint. Could you please provide an updated branch for rgb-std (and possibly others)?

Will work on that tomorrow morning

@zoedberg all downstream crates are updated and do compile. You can test now

It seems this hasn't fixed the yaml conversion issue. Inspecting a consignment created with updated libs (from latest rgb-wallet) produces the usual issue:
rgb inspect --format yaml rcv_compose.rgbc
RGB: command-line wallet for RGB smart contracts
     by LNP/BP Standards Association

Unable to find or parse config file; using config defaults
thread 'main' panicked at cli/src/command.rs:789:56:
unable to present as YAML: Error("serializing nested enums in YAML is not supported yet")


Bump csi-sanity to 5.3.1

5.3.1 is the latest version.
NONE

Not sure why we are getting the CI failure.  It happens with every PR here.

/retest

/lgtm


Insufficient Documentation of Session ID

The session ID used to index ValidationEngine instances in the ValidationService is not clearly documented.
This would have greatly helped with PR #81

Fixed in #132


"Jidvei" is listed in the drop-down menu

Description:
"Jidvei" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Jidvei" in the search bar.

Expected results
The location is listed in the drop-down menu as "Jidvei, Alba (Jidvei)".


⚠ Guya.moe Proxied has degraded performance

In f1c149d, Guya.moe Proxied (https://ice.guya.moe/) experienced degraded performance:

HTTP code: 200
Response time: 11951 ms

Resolved: Guya.moe Proxied performance has improved in 64b36d1 after 11 minutes.


Typescript typings

Would be sooo great to have typings for Typescript. You could use dts-gen and publish it under @types/swarm-numberformat. :)

I've never used Typescript, so I'm afraid this will not be a high priority for me. Happy to accept outside contributions, though!

I'll see what I can do!

So, I started to work on the typings, and after looking more at the docs it looks like there some major options missing. The Parser class is not well documented, as well some methods like format, parse, requiredDecimal, ...


Implement a three-dimensional Convex Hull algorithm to process voxels at the surface of voxel models from Goxel imports

Some 3D models may have internal voxels which are not seen and are thus a waste of memory, computation-wise.
A 3D Convex Hull algorithm will process the external voxels which are visible alone.
This will reduce the time complexities of rendering and dynamic manipulation of 3D objects with "n" lengths from O(n^3) to O(n^2) for cases where internal voxels are present.
While a Jarvis algorithm wraps all points by considering the most counterclockwise trios:

A 3-D Convex Hull algorithm will generate a hull from an initial triangle plane of 3 voxels and pivot around the external points in 3-D space until the hull closes:

Some useful references:
https://www.cs.jhu.edu/~misha/Spring16/09.pdf

It is currently being developed as **MeshWrap** in voxelmap versions >= 2.1.0
see source-code

Current issues with MeshWrap

graphical bugs Will not do triangulation on smooth surfaces. This is intrinsic to the 3-D Convex Hull algorithm; must either set up a constraint on smooth sites or make smooth sections rougher with an image manipulation technique
automation MeshWrap is far from automated. Must manually define some constraints such as for 3-D spacing [ for wrapping ] rel_depth and 3d resizing voxelmap.resize3d to make adequate Triangulated objects.
documentation It currently lacks documentation.

This issue has been resolved with the MarchingMesh() function; applying a marching cubes function from scikit-image.


Run builds in parallel

@aucampia now only one build per image should run regardless of the number of files changed within an image.

Thanks, very good to have this.


i have problem with the console, csrf validation.

I'm trying to run a background process.
my idea is, execute a command in php which in turn run the console Yii2.
$result = exec('php yii controller/action param param > result.log &');
In Localhost everything works great, but the server does not work :(
shows me the message: "Unable to verify the data sent", this should not make sense since it is running from the console.
When I run the command directly from the console, everything is going well, but when the command is launched at runtime does not work.
Friends, thank you very much collaboration.
"This was translated from Spanish"

To run this action from console you should disable CSRF validation (in your controller declare public $enableCsrfValidation = false; or redeclare public function beforeAction). On the other hand, the best solution is to create command for this action (console controller, extends yii\console\Controller)


Error in training code

Hi, I've just read your paper and am trying to run the code but the training code in train_flow.py doesn't appear to be working. I notice a break command here before the loss is computed which triggers an error when removed. Is there a known reason for this? Do you have any suggestions to succesfully train the model? Thanks!

For the record, the error was a torch.cuda.OutOfMemoryError: CUDA out of memory error which was fixed by lowering the batch size. Not sure if the break command there is an accident but removing it lets the model compute the losses now. Thanks!

Keeping this open to ask if there's a reason for the break command

Thanks for pointing this out, it’s for debugging and I have removed it away.