[WIP] nixos/refind: init

Motivation for this change
I wanted to use refind instead of systemd-boot.
I have been running this for almost a year now.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

@AndersonTorres maybe you are capable of giving some review-notes, as you are marked as the maintainer for the refind-package ?

ping (triage)

@disassembler @lheckemann I would love to get this into 19.09

I like the idea, but I'm not very comfortable with having a bootloader option without tests — breaking people's boot sucks! Any chance you could add tests to this?

@lheckemann I'll look into it

looking at the following files i am completly lost on how to write tests for this:

nixos/tests/boot.nix
nixos/tests/installer.nix

If the lack of tests and the implication of that is mentioned in the enable option's documentation, I guess I'd be fine with merging this as is as well.

@lheckemann i think i got it figured out how to write a proper test for this. I'm gonna spend some more time on this on thursday and friday.

@lheckemann so i played around some more but i just get it to work in the virtualmachine that is used for the tests.

So what's the problem?

Any progress?

sorry for not reporting back.
my notebook had an unfortunate encounter with a full cup of black-gold ...
followed by a lot happening in my personal life.
@lheckemann as far as i remember the problems i encounter where something along these lines:

the initial nixos installation (by nixos/tests/installer.nix) seems to be run in bios-mode even tho isEfi is set
refind seems to have a problem installing in that mode (systemd-boot does not)
-> refind will start on restart of test-vm after initial installation but will not find its configuration

that's where i got stuck and then i couldnt work on it any further.
i will try to pick this up as soon as i find the time.
sorry for the delays.

@lheckemann i think this is now done.

The current test driver makes it impossible to sanely test with a writable nvram. I tried adding the ability to do so, but there are just too many assumptions in the current test driver. In addition there is the new test driver in python, so I'm not too keen on improving anything for now :/. Either I need to port the installer test to python, or I have to rework the perl test driver which may be soon deprecated, if not removed.
Basically, the issue is two-fold:

The first boot cannot be configured with qemu flags. The first boot is always as legacy boot.
Then, once fixed, the test driver must use pflash options to setup the EFI.

Something similar to the following, where there is the OVMF code (the EFI), and another writable drive for the NVRAM. Unverified still, but the secondary drive may need to be started from a copy of OVMF_VARS.
-drive if=pflash,format=raw,readonly,file=$args->{efi}
-drive if=pflash,format=raw,file=$tmpDir/vm-state-machine/nvram.fd

i set this PR to WIP as it would be best to first merge the following PRs and adapt the tests here accordingly.

#73237 (port installer-tests to python)
#73768 (support useEFIBoot in installer tests)

@samueldr both done ;)

@betaboon @samueldr this is still mildly interesting to me. what's the status of it nowadays?

I'm using it on a daily basis for ages.
back when i opened this PR i was asked to implement tests (for good reasons).
adding support for efi-boot to the test-infrastructure was quite an endeavour.
sadly soon after i had it working there were major changes to the test-infrastructure which resulted in the efi-boot-support requiring alot of rework which i never got around to.

The only issue is the test, then?

The only issue is the test, then?

i would have to check if the code in this PR is what I'm running. not sure as this PR is old.
If there is interest to get this going i could try to find some time.

This is probably one of the hardest PR I've been involved into reviewing.
On the one hand, this is an amazing feature, and seems to work just fine... And the contribution has great value.
But on the other... Testing that the bootloader generation is not broken is primordial. This is because it would lock people out of their system if it fails.
Though, as we discussed earlier, a simpler test where we use the fallback location, like GRUB2 currently does, would be good to go forward.

i totally agree that this should have rock-solid tests.
It's one thing if i lock myself out of my system, but it's a completly other thing if that happens to someone else.
the hardest thing about the tests was to implement support for booting via EFI on the very first boot (which is what #73768 adressed).
It is a non trivial change.
there are several scenarios to be covered by tests:

initial boot in BIOS-mode -> install EFI -> reboot into EFI-mode
initial boot in EFI-mode -> install EFI -> reboot into EFI-mode

If, just like with GRUB, we only test the "default fallback location" (/EFI/BOOT/bootx64.efi, /EFI/BOOT/bootaa64.efi) install, we should be good to go since it at least tests the generations file format.
Anyway, installing at the fallback location would be a requirement for some platforms, like U-Boot's UEFI on ARM. And also required on some bad UEFI implementations.

Thinking like Ryuuguin Seiya, the Overly Cautious Hero, how could we devise an intermediate phase?
I am thinking in something like that:

Start in GRUB;
Put an option "Boot refind" in GRUB menu

This way, if refind does not boot, then it is just a matter of reboot the system and use the GRUB options.

Thinking like Ryuuguin Seiya, the Overly Cautious Hero, how could we devise an intermediate phase?
I am thinking in something like that:
* Start in GRUB;

* Put an option "Boot refind" in GRUB menu

This way, if refind does not boot, then it is just a matter of reboot the system and use the GRUB options.

@cstrahan does that https://github.com/cstrahan/nixos-config/blob/1ec7e813efdaf934014c191a7cdf021bc5f6cb79/configuration.nix#L84

any update on this?

None. The OP seems to lose interest.

Damn, the only thing missing are the tests.

This is important to me. What is the state of efi-boot in the test infractrusture? Is any trivial patch possible ? I am willing to help but I am missing information here.

This is important to me. What is the state of efi-boot in the test infractrusture? Is any trivial patch possible ? I am willing to help but I am missing information here.

I can't help so much. Maybe open a thread on Matrix and/or another on Discourse.nixos.org can be helpful.

Since this doesn't look like it'll be moving any time soon, I'll close this for clarity. You can re-open if you gain interest again.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/easy-refind-boot-by-booting-into-systemd-boot-from-refind/28507/1


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

 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.:x: Soura2605 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


UIFileDialog don't have "OK" and "Cancel" buttons

his I create a new file dialog but the buttons don't appear.
this is the code that I used for the create:
manager = pygame_gui.UIManager((SCREEN_SIZE[0], SCREEN_SIZE[1]))
file_dialog = pygame_gui.windows.ui_file_dialog.UIFileDialog(rect=pygame.Rect((SCREEN_SIZE[0]//2, SCREEN_SIZE[1]//2), (SCREEN_SIZE[1] * 0.7, SCREEN_SIZE[0] * 0.7)),manager=manager)
Platform and software (please complete the following information):

OS:  Mac
Pygame GUI version 0.6.8
Pygame version 2.3.0

Looks like you have your height and width values the wrong way around.
On Wed, 29 Mar 2023, 07:31 gonensel, @.***> wrote:

his I create a new file dialog but the buttons don't appear.
this is the code that I used for the create:
manager = pygame_gui.UIManager((SCREEN_SIZE[0], SCREEN_SIZE[1]))
file_dialog =
pygame_gui.windows.ui_file_dialog.UIFileDialog(rect=pygame.Rect((SCREEN_SIZE[0]//2,
SCREEN_SIZE[1]//2), (SCREEN_SIZE[1] * 0.7, SCREEN_SIZE[0] *
0.7)),manager=manager)
Platform and software (please complete the following information):

OS: Mac
Pygame GUI version 0.6.8
Pygame version 2.3.0

[image: image]
https://user-images.githubusercontent.com/65337111/228445807-b614fb1f-db1d-42a5-911b-29b18b97432a.png
—
Reply to this email directly, view it on GitHub
https://github.com/MyreMylar/pygame_gui/issues/429, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADGDGGWFKORV4L7SATCPHWDW6PJMXANCNFSM6AAAAAAWLQGASA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Looks like you have your height and width values the wrong way around.
…

ok thanks its working
I have another question there is support in Hebrew fonts?

This was added in #511 and a few other following PRs. Closing this for now in preparation for 0.6.10 release later this week.


Replace key_type with partial_key and full_key to distinguish whether…

… a key has a prefix or not

Change Description

API Changes

[ ] API Changes

Documentation Additions

[ ] Documentation Additions

The unit tests work again on the staging-c branch.

With these latest changes, the tests are all passing locally for me.


Swiper slider gallery, and changing orientation on Iphone

I made an image gallery format. It's responsive (boostrap and some own medias @), when i click an image, a Swiper (http://idangero.us/ plugin) slider opens to show images one by one in a bigger size. I tested on PC, and Android phones, and it's working fine on this 2 devices:
responsive, changing orientation on phone (android), all formats are working fine...
except iphones, when i click an image, it opens fine, but when orientation is changed, it doesnt resize the swiper area.
I allready try this 2 options:
window.addEventListener("orientationchange", function() {
    mySwiper.update(); and thisone: mySwiper.onResize();
})
```;
Also i notice this, after changing orientation (landscape to portrait OR portrait to landscape), and do a small scroll-down, swiper resize the container (the scroll to hide the addres-bar upside) and works fine, but in android, that's not necesary.

@ogutierrezdiario Did you ever find a fix? Having exact same issue with the iPad.

@nolimits4web Any ideas?

I also have a similar issues. When changing orientation, the layout is messed up but works when refreshed.

Any word on this? I'm having this issue via Ionic 2

The same happens to me.
Any solution?

+1

The problem still exists.
Steps to reproduce: Open this URL https://www.welt.de/vermischtes/article167880352/Und-dann-tat-Trump-was-man-auf-keinen-Fall-tun-sollte.html#cs-Sonnenfinsternis-in-den-USA.jpg in Safari (iOS 10.3) and change the orientation. Simulator will show you the bug as well.

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Swiper, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue

I found this issue on my project also.
This happens because there is wrong width/height from window object in resize event handler.
( https://bugs.webkit.org/show_bug.cgi?id=170595 )
I managed to workaround it by giving timeout and call swiper.update() manually.


Added Commits

@sukriti-singh  thnx for contributing, try this year, and make sure you follow me and star this repo for updates about special swags I will be giving.
Contribute in October for Hacktoberfest 2020


Added ellipse command

Canvas supports ellipses, which could be made with two arcs i guess, but
this simplifies things

Hey @CJEnright Thanks for the contribution. Could you add a quick test case so we can catch any regressions, as described in https://github.com/gliffy/canvas2svg#add-an-example-case

@CJEnright Still interested in this?

@spanktar Yeah, guess I forgot to come back to this.  I think I deleted that repo a while ago, so I opened up a new one with test included, just let me know and we can close this request and I'll open one with the new repo.  Sorry about the confusion.

While trying to use your code for rotated ellipses and rotated elliptical arcs, I noticed that your implementation ignores the "rotation" parameter as-written.  I modified your code to (1) calculate the rotated coordinates endX/Y and startX/Y and (2) to feed the rotation parameter in degrees to the SVG xAxisRotation parameter.  As far as I can tell across many test cases this properly mirrors the canvas v5 ellipse behavior.
   ctx.prototype.ellipse = function(x, y, radiusX, radiusY, rotation, startAngle, endAngle, counterClockwise) {
     //ellipse is the same svg command as arc, but with a radiusX and radiusY instead of just radius
       if (startAngle === endAngle) {
           return;
       }
       startAngle = startAngle % (2*Math.PI);
       endAngle = endAngle % (2*Math.PI);
       if(startAngle === endAngle) {
           endAngle = ((endAngle + (2*Math.PI)) - 0.001 * (counterClockwise ? -1 : 1)) % (2*Math.PI);
       }
       var endX = x + Math.cos(-rotation) * radiusX * Math.cos(endAngle)
                    + Math.sin(-rotation) * radiusY * Math.sin(endAngle),
           endY = y - Math.sin(-rotation) * radiusX * Math.cos(endAngle)
                    + Math.cos(-rotation) * radiusY * Math.sin(endAngle),
           startX = x + Math.cos(-rotation) * radiusX * Math.cos(startAngle)
                      + Math.sin(-rotation) * radiusY * Math.sin(startAngle),
           startY = y - Math.sin(-rotation) * radiusX * Math.cos(startAngle)
                      + Math.cos(-rotation) * radiusY * Math.sin(startAngle),
           sweepFlag = counterClockwise ? 0 : 1,
           largeArcFlag = 0,
           diff = endAngle - startAngle;

       if(diff < 0) {
           diff += 2*Math.PI;
       }

       if(counterClockwise) {
           largeArcFlag = diff > Math.PI ? 0 : 1;
       } else {
           largeArcFlag = diff > Math.PI ? 1 : 0;
       }

       this.lineTo(startX, startY);
       this.__addPathCommand(format("A {rx} {ry} {xAxisRotation} {largeArcFlag} {sweepFlag} {endX} {endY}",
           {rx:radiusX, ry:radiusY, xAxisRotation:rotation*(180/Math.PI), largeArcFlag:largeArcFlag, sweepFlag:sweepFlag, endX:endX, endY:endY}));

       this.__currentPosition = {x: endX, y: endY};
   };

Hi @spanktar do you have any plans to merge this to support ellipse ? Anything  is missing ? Just asking because this it would be useful for me and if I can help on anything that is missing.
Thanks.

@spanktar It would be great to have this merged.

Oh, thanks for poking me! Let me get in touch with the correct folks here


mkNeovimPlugin: remove hasConfigAttrs

Get rid of the hasConfigAttrs which doesn't seem necessary.
Please, tell me if I missed the point and that it is (or will be) actually useful.

For reference, it has been introduced in https://github.com/nix-community/nixvim/pull/1876.

This makes rustaceanvim a very "strange" plugin, as you can specify things in luaConfig, but they won't be written anywhere as it has configLocation = null.
I think we should either merge configLocation and hasConfigAttrs (i.e hasConfigAttrs == false is equivalent to configLocation == null).
This also means that if we want to delete them we need to delete both of them.
We may need to refactor a bit rustaceanvim to correctly insert the luaConfig to the right place in the init.lua file

This makes rustaceanvim a very "strange" plugin, as you can specify things in luaConfig, but they won't be written anywhere as it has configLocation = null.
I think we should either merge configLocation and hasConfigAttrs (i.e hasConfigAttrs == false is equivalent to configLocation == null).
This also means that if we want to delete them we need to delete both of them. We may need to refactor a bit rustaceanvim to correctly insert the luaConfig to the right place in the init.lua file

Rustaceanvim just uses the global options for its configuration, now. We just pass it along to our globals module https://github.com/mrcjkb/rustaceanvim?tab=readme-ov-file#gear-advanced-configuration

To be consistent with the other plugins we should in some way handle the fact that rustaceanvim is inserted in the globals table. Maybe we should add the preSection just before the globals and the post section after?
I'm not really sure what luaConfig means for this plugin

To be consistent with the other plugins we should in some way handle the fact that rustaceanvim is inserted in the globals table. Maybe we should add the preSection just before the globals and the post section after?
I'm not really sure what luaConfig means for this plugin

Yeah I see what you mean. I think that makes sense for a pre and post. But, I agree it's confusing for this type of plugin that basically configures like a vim plugin.

To be consistent with the other plugins we should in some way handle the fact that rustaceanvim is inserted in the globals table. Maybe we should add the preSection just before the globals and the post section after?
I'm not really sure what luaConfig means for this plugin

I am not sure to follow there.
Plugins, in their config, will set other options. Most of the time, it will be appending text to extraConfigLua (through luaConfig.content). But it can also (and not exclusively) be setting global variables (using config.globals) or setting any other config option.
For instance, the vim plugins using mkVimPlugin often don't add any lua code, but they set globals.
For me, there is no issue if a neovim plugin does or does not set some lua code.
What I mean is that I don't get why rustaceanvim is a special case. It's just a plugin that doesn't generate lua. That's fine no ?

Do we need to handle globals setting for rustaceanvim differently than what we do for vim plugins (i.e. directly leveraging globals.foo = value ?

I have pushed another commit that now assumes that configLocation is never null. If someone deliberately sets it to null, it would error out.
It looks like our entire test suite evaluates to the same result as before.
Maybe we could argue for keeping those abstractions, but as it appears to me, they don't seem very useful at this time.

I think that we should maybe not use mkNeovimPlugin for rustaceanvim, as it is a plugin that's too special. It required adding both hasConfigAttrs & configLocation

It required adding both hasConfigAttrs & configLocation

Are you sure ? According to me, its behavior has not been affected by this PR. Maybe I'm missing something.

You can search for other occurences of configLocation

You can search for other occurences of configLocation

I did, and it was only present in rustaceanvim (setting it to null which didn't have any effect as callSetup was false) and in neovim-plugin.nix itself.

I may be a bit late to the discussion and have missed some context, but the above quote seems to be conflating lua config and nix module config.
To be explicit: luaConfig.* cannot set module-config values. It can only be used for adding lines of lua to init.lua.
One of the limitations of using the module system to configure globals (and similar) is that we have no control over where the actual lua config ends up. Therefore plugins that use config.globals do not currently makes sense with luaConfig's pre/init/post system.
Most of the time we avoid this issue, because mkVimPlugin does not have a luaConfig option. However it does highlight a potential design flaw...
Perhaps in the short term it is ok to simply say "anything using mkNeovimPlugin should have some lua-based setup-style function, which makes sense to call via luaConfig" ? If so, then removing hasConfigAttrs is fine.

We are on the same page on the understanding of the current situation.


feat(resolve): add support for accessing default conditions using ... in overrideConditions

Description

mimic webpack's behavior, providing users with access to default options.
webpack docs
webpack usage

Actually, both extensions and mainFields should support the ... syntax, but they should be implemented in separate pull requests.

ping

I close this pull request since Vite always inherits the default conditions.


Convert blacklist, theme, utils and Button component to TypeScript

change-type: patch

@LucianBuzzo could we proceed and merge this?

@thgreasi If possible, I want to wait for this PR to got through first --> https://github.com/resin-io-modules/resin-components/pull/193

Would love to see that merged at some point :) no pressure though

@thgreasi Yep this can get merged now


Is this behaviour of System.Console.WriteLine expected?

Godot version
4.0 RC2 official binaries
System information
Windows 10 and Manjaro (Arch) Linux
Issue description
When you run Godot with --quiet 2>/dev/null (or the equivalent on Windows), nothing is supposed to be printed out. But, calls to System.Console.WriteLine(...) still print out.
Is this expected? Can I rely on this behaviour?
I'm in the process of making a game for blind players, and I need a "clean" console - something that doesn't have any text from Godot (e.g. the intro "Vulkan running ..." text), warnings, or errors printed out. I need full, 100% control over the console window output (and I'm using it to accept player input via text).
Steps to reproduce
Create a new Godot C# project with a scene that has this script:
public override void _Ready()
{
    System.Console.WriteLine("Hello, World!");
    GD.PushError("ERROR!");
    GD.PushWarning("Warnings?");
    Godot.GD.Print("HI!!!!");
}

Export it to Windows or Linux
Run the exported versions with the additional command-line parameters: Windows (--quiet 2>out.txt) or Linux (--quiet 2>/dev/null)
Observe: Hello, World! prints on the console, without any other warnings or text (e.g. Godot 4 running with Vulkan)

Minimal reproduction project
poc2.zip

Yes, this is expected behavior.

System.Console.WriteLine writes to stdout.
System.Console.Error.WriteLine writes to stderr.

You are using 2> which silences only stderr. To silence stdout use 1>.

Great, thanks. So to re-iterate, I can expect this behaviour to continue in the foreseeable future, correct?

So to re-iterate, I can expect this behaviour to continue in the foreseeable future, correct?

Yes, although we can't guarantee this.

Yes, although we can't guarantee this. See also #63575.

I read through that and don't see the relevance, other than the fact that I should specify --no-header and --quiet both.
Can you clarify? Are you saying "We can't guarantee [-q will remove the header]" or "We can't guarantee [-q will prevent System.Console.WriteLine from writing to the console]", or something else?

Can you clarify? Are you saying "We can't guarantee [-q will remove the header]" or "We can't guarantee [-q will prevent System.Console.WriteLine from writing to the console]", or something else?

-q is designed to silence stdout but not stderr. You can also disable stderr using a project setting:
https://github.com/godotengine/godot/blob/e930c8d3838280e40baabc4426bd8236f7ac50a3/doc/classes/ProjectSettings.xml#L284-L291

Right. So if my goal is to have total control over the console output, I should:

Disable stderr and stdout via the application/run/disable_* settings
Use --no-header when it comes out
Print to the console via System.Console.WriteLine

And this behaviour is expected to be consistent going forward. Is that correct?

And this behaviour is expected to be consistent going forward. Is that correct?

Yes, hopefully. Note that disabling stdout will already disable the header, so you don't need --no-header in this case.

Thank you very much for the feedback.
One caveat: disabling stdout via the project properties disables it in the editor, but not in the exported project; meaning, you can't easily debug in-editor and expect to see that in the output.
For now, I'm using a wrapper method that calls both System.Console.WriteLine and Godot.Print; the former appears in exported versions, the latter in-editor.


[RFC] Add max available item quantity to checkout line

What I'm trying to achieve
As a user, I should be able to check what is currently max available item quantity
Describe a proposed solution
On CheckoutLine type we should add the field quantityAvailable.
type CheckoutLine implements Node & ObjectWithMetadata {
    ...
    quantityAvailable: Int
}

Description:
Quantity of a variant available for sale in this line. Including reservation and other lines quantity.  The field value will be null when productVariant stocks are not tracked.
Example:

We have 10 items of product in stock.
Checkout with 2 lines (Both lines are assigned to the same product):

Line1 quantity is 2 items.
Line2 quantity is 5 items.

This new field should have

Line1.quantityAvailable = 5 items
Line2.quantityAvailable = 8 items

Example2:

We have 80 items of product in stock.
Max checkout quantity Is setted to 50
Checkout with 2 lines (Both lines are assigned to the same product):

Line1 quantity is 2 items.
Line2 quantity is 5 items.

This new field should have

Line1.quantityAvailable = 50 items
Line2.quantityAvailable = 50 items
Values shouldn't be changed whatever reservations are turned on or turned off.

Other solutions I've tried and won't work
We try to change the behavior of Variant.quantityAvailable depending on whether was called via Product or Checkout type. This solution could break caching.
Screenshots or mockups

In quantityAvailable calculation you must take into consideration the channel, country code if the shipping address is set, and also adjust the value when the collection point is chosen or available as a shipping method.

LGTM 🚀


Invert Axis and Other Axis Options

Vedo seems to lack many of the axis options that make it optimal for publication figures, which is sad because it is far more intuitive for their creation than Matplotlib or Matlab. Some of these options are.

Invert Axis: The ability to have an axis direction follow numerical decrease rather than increase. Matplotlib equivalent: https://matplotlib.org/3.1.1/gallery/subplots_axes_and_figures/invert_axes.html

Placing 3D Axis in other locations like the front of the plotting area (see figures)

Placing Axis Planes in other locations (currently the only other option is a second zx grid I believe)

Having Axis Titles follow the camera (this may be an option I couldn't find)

For reference, I have included a figure I created both in Matlab and Vedo. In Vedo, I had to rotate my data from its natural z-down, y-left, x-forward frame (shown in Matlab figure) to get it to not be behind the axis planes. I can rotate the X-Axis title, but it is then upside down. I cannot rotate any of the axis number labels. Ideally, Vedo would be able to produce the same axes as Matlab. Also below, are my axes code options.

vp.show(interactive=True, axes={'zxGrid': True, 'xtitle': 'X-Axis', 'ytitle': 'Y-Axis', 'ztitle': 'Z-Axis','xTitleRotation': 270, 'xTitleOffset': 0.05, 'yLabelRotation': 90})

Thanks @GeneralGarrett  for your suggestions for improvement.
This seem related to issue #250 and #255 maybe something there can be of help.
About the raised points:

that should be fairly easy to add as a flag. At present one can have access at all the elements of the axes and move them around as in #250 , but it's probably not too intuitive. One can also specify order and position of each label and tick with xValuesAndLabels (see here and here).

I need to think if this is possible.. but i guess it should.

Should also be easy to implement. We could set a variable to define at what fraction (or absolute position?) along the axis the grid should sit.

Not doable. This is only possible in axes=8 mode (which I almost never use as I find it spoils the feeling of spatial orientation).

implemented via xInvert keyword or by inverting the range as in matplotlib
implemented via xyShift as a fractional position along the axis
the axis is shifted along with the plane(s) it belongs to

implemented via xInvert keyword or by inverting the range as in matplotlib
implemented via xyShift as a fractional position along the axis
the axis is shifted along with the plane(s) it belongs to


Error: json: cannot unmarshal number into Go struct field ResultData.result_data.org_id of type string

Good morning,
I get the following error message after starting the addon:
Error: json: cannot unmarshal number into Go struct field ResultData.result_data.org_id of type string
Usage:
GoSungrow api login [flags]
Examples:
GoSungrow api login
Flags: Use "GoSungrow help flags" for more info.
Additional help topics:
ERROR: json: cannot unmarshal number into Go struct field ResultData.result_data.org_id of type string
s6-rc: info: service legacy-services: stopping
s6-rc: info: service legacy-services successfully stopped
s6-rc: info: service legacy-cont-init: stopping
s6-rc: info: service legacy-cont-init successfully stopped
s6-rc: info: service fix-attrs: stopping
s6-rc: info: service fix-attrs successfully stopped
s6-rc: info: service s6rc-oneshot-runner: stopping
s6-rc: info: service s6rc-oneshot-runner successfully stopped

same issue as here...?
https://github.com/MickMake/GoSungrow/issues/74

I have changed the file on my PC but not in the Addon.
Can I upload the file to HomeAssistant?

I have the same issue

Same here

same since yesterday.

I also notice that this add-on stops working very often. Most of the time it starts working again after a reboot. But now it's dead completely. Can't seem to get it working again. Same issue as described above

Mick has commented on the HA forum that the update to iSolarCloud changed a field which broke the add on, and that he's looking into it.

all good with 3.0.7. thank you.

Works now! thanks!


oops, I need to list the hidden page to include it

The redirection page needs an entry in the yml to get built

actually, let's scratch this idea. I'll do the redirection page on www.gluster.org instead.


[tracking] warning: exported type PathsConfiguration should have comment or be unexported (golint)

https://github.com/RedHatInsights/insights-results-aggregator-mock/issues/52

Fixed in https://github.com/RedHatInsights/insights-results-aggregator-mock/issues/52


FEATURE: PCI-4809 Backport of 'Make ec2 auth support v4 signature format'

Make ec2 auth support v4 signature format
Extract the signature and access key via whatever method is needed
for the version of the request (e.g headers for v4), and add the
headers and hashed body, which is required for keystone to calculate
the correct v4 signature when validating the request.
Change-Id: I161eccc4ea48a21a80d689f6a328ca95cace2e6e
Closes-Bug: #1408987
Conflicts:
nova/api/ec2/init.py

ok to test

It's easy to implement this in nova-ec2, but this would also require non-trivial hakcing in keystone :| will check later on


proposal: flag: Introduce functions for defining rune flag

Currently the flag package has functions for parsing string.
In order use a rune flag one has to

define it as string flag
check rune count using utf8.RuneCountInString to know if is has one or more rune.
if rune count is more than one, produce an error or call flag.Usage and exit. ( But the usage information
that flag.Usage gives will contain -flag string for that flag. so it requires to add `rune` to that flag's usage
information to get  flag.Usage show -flag rune for that flag's usage information. )
type cast it to []rune
get the first rune by using index 0

The need for a flag that accepts a single rune seems 1) rare; 2) easy to implement by implementing the flag.Value interface.  What is the argument for putting this in the standard library?  See https://golang.org/doc/faq#x_in_std.

I'm at a loss to remember any program I've ever used that took a flag argument that was itself (the argument) required to be a single letter.
Note that you can implement this yourself using the new flag.Func (will be in Go 1.16) or using flag.Value.

(Retracted by author.)


Does Parse Server 2.2.7 support Parse.Cloud.useMasterKey()?

This is not a bug report. It's a question. The compatibility guide (https://github.com/ParsePlatform/Parse-Server/wiki/Compatibility-with-Hosted-Parse#Cloud-Code) says that Parse.Cloud.useMasterKey() is not available in Parse server cloud code. And I've been changing my code the whole day. I forgot it in one place, and tested it, and it works the way it does on Hosted Parse app on my local parse server.
So is it available in this version of Parse server? I can't find any release notes either. So please let me know and save me the torture of going through all these changes and extensive testing. Thanks!

Parse.Cloud.useMasterKey() won't work in parse-server because it modifies global state and will interfere with other requests that are happening simultaneously. The fact that it worked in one place must have been a coincidence.


added option to skip rendering certain headings

Not sure if this is something you would want, but I'll PR because I think it's a nice feature! If you'd prefer an undefined default, I can change it. I set it to an empty array for minimal code repetition and the performance difference would only be noticable if a page has 100s of headings to put in the TOC.
Example:
You have markdown
# Table of contents
[[toc]]

# Magic

# Dinosaurs

Which has TOC section:
Table of Contents

Table of Contents
Magic
Dinosaurs

But you think putting "Table of Contents" in the Table of Contents is an insult to the reader's intelligence, so you don't want it to be there.
md.use(markdownItTOC, {
skipHeadings: ['Table of Contents']
});

Now your TOC renders as:
Table of Contents

Magic
Dinosaurs

Hi, I understand the use case for this but I'm sorry to say I don't think this particular solution is generic enough. Relying on strict string matching on content is not very flexible. It may work on one document but not another (maybe there's a slight difference in wording).
This solution may work totally fine for you, but I don't think it's generic enough to include in this package. If you can figure out a way to make it more generic and flexible I'd be happy to include it though. One idea, for example... Not sure if it would work... How about adding the possibility to exclude headings appearing above the [[toc]] declaration in documents? That would probably be generic enough.


URL Invalid Property

Description
URL invalid property for Open Graph, Twitter, Schema, Google+.

Issue Checklist

[ ] It is missing its protocol.

Assignees

[ ] Final assign @yairodriguez

I’m working on my way! 😉

Review was done successfully.


Unable to build on nightly

per recent travis CI: trait objects without an explicit dyn are deprecated
it seems this is a warning for cargo build but an error for cargo test

@phlmn speedy! thanks :)

@phlmn any plans to publish to crates.io in the near future?

@phlmn speedy! thanks :)

You're welcome!

@phlmn any plans to publish to crates.io in the near future?

Yeah for sure! (At least we thought) We were missing rights to do so, but seems like we do have them.
I think I will finish https://github.com/iron/iron/issues/579, which makes releasing easier, and at least release a 0.7-pre version very soon. For 0.7 we should be sure about the breaking changes.


Overriding templates does not work anymore w/ colocated addon templates

In #1221 we refactored the ember-bootstrap components internally to use template colocation.
While this first seemed like a non-breaking change (all tests passing), it seems that the usual way of overriding template of an addon in an app does not work anymore. Idk exactly how colocation is implemented, but looking at the RFC, specifically the build-time section, I read this as a way to tightly bundle class and template together. So putting a template with the same name as the addon's template into the app does not work anymore to override it. Seems the only way to do it is by explicitly calling setComponentTemplate as in this example:
import BaseAlert from 'ember-bootstrap/components/bs-alert';
import layout from './template';

export default setComponentTemplate(layout, class Alert extends BaseAlert {});

In ember-bootstrap we explicitly told users that extending from classes is not supported anymore (to better allow internal refactorings, e.g. to Glimmer components), but we did not mention anything specific with regards to templates. So not sure if this can/must be considered an (accidental) breaking change? 🤔
/cc @jelhan

To be honest I would consider overwriting a template of a component as a bad practice. It's even worse than extending it's class. I wouldn't consider either as supported. I have used that pattern myself in the past but it was a maintenance nightmare. I would love to support all needed customization through public API so that a consumer can customize as they need by a wrapping component instead.
Regarding the public API I would only consider invoking the component in a template as our public API. So neither extending a class nor overwriting a template.
And I would also push for that pattern in our internal composition of components. This is what I had in mind here: https://github.com/kaliber5/ember-bootstrap/pull/1022#discussion_r483985828

Regarding the public API I would only consider invoking the component in a template as our public API. So neither extending a class nor overwriting a template.

Yeah, I agree. Everything else makes maintenance extremely difficult. I just wanted to bring this up, as people (including me and you) have done so in the past, and we haven't explicitly stated anything about the template part. Maybe we should mention that somewhere, maybe alongside the don't-extend-classes warning!?

Whilst I totally agree with your conclusions on this, We've just run into this issue as we had overridden the element and label templates to provide the required attribute to the label component so we can include a visual indicator for required fields.
Since the move to html attributes on the control this 'hack' has been necessary and no longer works! Can you suggest an alternative solution to getting 'required' into the label?
You suggest that overriding the class is bad... what's the official way of supporting validation addons as ember-bootstrap-changeset-validations seems to do just this...

Can you suggest an alternative solution to getting 'required' into the label?

That's a valid concern, but I am afraid I have no good answer atm. 🤔
Like you could override the whole label component for example, by basically implementing your own component class and template, without extending or replacing just the template. But for a use case that is so common as yours, this doesn't seem like an acceptable solution really...
We could introduce new arguments to form.element like @labelClass, but this seems on the other hand like a step backwards again, as we wanted to get rid of arguments that just apply HTML attributes. Unfortunately there is no way apply attributes to anything but the "default" element (which hash ...attributes) in Ember.
The form API is really not as modular and customizable as it probably should, just as it had been around for so many years without any substantial changes. Like it hides the way components are combined and rendered, i.e. you cannot do something along the lines of:
<form.element ... as |el|>
  <el.label class="required">Email</el.label>
  <el.control />
</form.element>

There have been ideas floating around to rethink all of this, and probably create something not coupled to Bootstrap as the foundation. @jelhan was interested in this as well I believe. But as always there is just so much time...

what's the official way of supporting validation addons as ember-bootstrap-changeset-validations seems to do just this...

Yes, you are right, they do just that, mostly because we haven't yet come up with an API to integrate validation libraries by something other than extending classes (also related to that general form library idea). For now these two addons (´ember-bootstrap-changeset-validationsandember-bootstrap-cp-validations`) are considered privileged in that they continue to extend from base classes (which is something like using a private API), but we will make sure that once there are changes in ember-bootstrap that would cause things to break (e.g. moving to Glimmer components) we will immediately update those addons to continue working.

The form API is really not as modular and customizable as it probably should, just as it had been around for so many years without any substantial changes. Like it hides the way components are combined and rendered, i.e. you cannot do something along the lines of:
<form.element ... as |el|>
  <el.label class="required">Email</el.label>
  <el.control />
</form.element>

There have been ideas floating around to rethink all of this, and probably create something not coupled to Bootstrap as the foundation. @jelhan was interested in this as well I believe. But as always there is just so much time...

Yieldable named blocks may be a good fit for that scenario. Not having much experience with them yet. But if I got it right, they should allow us to provide a public API like the following:
<form.element>
  <:label as |label|>
    Email
  </:label>
  <:control as |control|>
    <control disabled />
  </:control>
</form.element>

It should also allow you to skip named blocks and use the default implementation instead. So if you don't need to customize the control it should also work like this if I got it right:
<form.element>
  <:label as |label|>
    Email
  </:label>
</form.element>

We should be able to still support the existing API for consumers that do not need that level of customization.
<form.element />

<form.element as |el|>
  <el.control />
</form.element>

The polyfill covers all Ember versions that we need. So we should be able to add that feature without requiring a new major.

I like this idea, but it is still a little 'clunky' for something that IMHO should be easy to do. Having to set required on the control and then write my own label from scratch to put a class on it starts to make my form very long with a greater risk of error (therefore even more tests being required!)...
In theory, if the 'is-required' class remained on the FormGroup (see https://github.com/kaliber5/ember-bootstrap/issues/1261) then I could handle everything via CSS without needing to override the label component at all.
I worry though that without formal documentation,I am essentially relying on another feature which is pending deprecation/ removal...

The other solution would be to allow 'custom' label components like you do for controls. As we use FontAwesome 5, rather than relying on CSS styling, we inject an icon tag into the label:
{{#if @requiredLabel}}
  Required
  <FaIcon @icon="star" @size="xs" data-test-form-element-required-icon />
{{/if}}

Declaring this for every required label in every form is a little unpleasant but if we could instead do <form.element labelType="requiredLabel">  that would be quite a bit cleaner.

I like this idea, but it is still a little 'clunky' for something that IMHO should be easy to do. Having to set required on the control and then write my own label from scratch to put a class on it starts to make my form very long with a greater risk of error (therefore even more tests being required!)...

I can't follow to be honest. The proposed API would look like this:
<BsForm as |form|>
  <form.element @label="username">
    <:label as |label|>

       {{label.text}}
       Required
       <FaIcon @icon="star" @size="xs" data-test-form-element-required-icon />

    </:label>
    <:control as |control|>
      <control required />
    </control>
  </form.element>
</BsForm>

If you want to share the required feature on label you can put it on a component, which you use as labelComponent. On such custom components you can support additional arguments. It would look like these:
<BsForm as |form|>
  <form.element @label="username" @labelComponent={{component "my-label-component"}}>
    <:label as |label|>

    </:label>
    <:control as |control|>
      <control required />
    </:control>
  </form.element>
</BsForm>

I see that this would be easier with a @required argument on <BsForm::element>, which is than passed to the yielded control and label component. We had a @required argument in v3. But we dropped it in v4 as it was only used to set the required HTML attribute on the control.
Wondering if there is any way to allow consumers to add such additional arguments on <BsForm::element>. That would be very powerful for customization.

I'm closing this now. We migrated to colocated templates long time back. Seems as if it is working fine for consumers.


Go major versions change fixed

[x] All tests passed. If this feature is not already covered by the tests, I added new tests.
[x] This pull request is on the dev branch.
[x] I used gofmt for formatting the code before submitting the pull request.

I have read the CLA Document and I hereby sign the CLA


configuration for release notes generation

Description
Enter a description of the changes here
Testing instructions
Add a set up instructions describing how the reviewer should test the code

[ ] Review code
[ ] Check Actions build
[ ] Review changes to test coverage
[ ] {more steps here}

Agile board tracking
connect to {issue number}

An example of what the changelog now looks like with the new config

The rules for the config mean that we need to add some more labels to github, and assign PRs with the correct labels.

This PR is an example for custom changelogs, which are categorized based off the the labels on pull requests. An example file has been included to show a workflow that automatically labels pull requests based off of their branch name. This change would mean making changes to the wiki, the pull request template, and optionally the readMe.
The labels used in this PR are examples, you can pick whatever labels / rules to use for future. But should be included in any documentation created.


The key not found error shows up

I started using this module first it shows yellow underline in vscode under module name and when i print response.meta dict it contains only following:
{'download_timeout': 180.0, 'download_slot': 'golden.com', 'download_latency': 0.34168577194213867}
it doesn't contain screenshot or driver etc.
what's the problem

+1, still same

Same issue here, @JDRanpariya , @boardsofcanada any clue ?

The issue was in my Selenium Contract tests. Testing by default yields normal Scrapy.Request objects. See source code here https://docs.scrapy.org/en/latest/_modules/scrapy/contracts.html
In order to workaround this and still use contracts to test ou spiders we need to create a new contract
class WithSelenium(Contract):
    """ Contract to set the request class to be SeleniumRequest for the current call back method to test

    @with_selenium
    """
    name = 'with_selenium'
    request_cls = SeleniumRequest

Then use the contract @with_selenium in our testing.
Hope it helps

+1 Still the same, any updates!

for some reason when I copied the code for SeleniumRequest, SeleniumMiddelware to my project it all worked just fine!

Had the same issue before, for me, it's solved after install the driver (chromedriver or geckodriver, because the missing of driver made the selenium middleware disabled)

Had the same issue before, for me, it's solved after install the driver (chromedriver or geckodriver, because the missing of driver made the selenium middleware disabled)


diabolification

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.1
logical conflict every: 100
sleep for: 300s
close stale after: 4 hours
[pullrequest]
requests per hour: 30
deps=[d]

/trunk merge


Add protection against directory traversal attacks

This prevents people from making a request such as:
$ curl --path-as-is 'http://<IP_ADDRESS>:8888/../../../../../../../../../etc/passwd'

Looks good. Could you squash the commits into one? Refer to https://github.com/mozilla/pdf.js/wiki/Squashing-Commits.

Sure, I'll take care of that tomorrow. Alternatively, you can merge and squash when you accept the commit.  :)

Done! I'm so used to using merge+squash on GitHub that I don't usually think of doing that anymore.  :)

Sounds good. I'm at the Mozilla All-Hands this week so I'll try to get this updated, but otherwise I will get it fixed first thing next week.

Alright, the requested changes have been made.  :)

/botio-linux preview

Thank you for fixing this!


nixos/systemd/coredump: fix kernel.core_pattern truncation

Description of changes
Fixes https://github.com/NixOS/nixpkgs/issues/213408. Substitutes the path to systemd in 50-coredump.conf with a symlink that has a constant-length path. This brings the value of kernel.core_pattern to a consistent 102 characters.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[x] armv7l-linux (cross)
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Huh, this is wild, nice find 😂

I opened a PR at https://github.com/systemd/systemd/issues/26261 - it would be nice if systemd would show a warning if sysctl.conf contains something that'll get truncated.


x86 built

Nice project! It says "experimental", but I only need basic functionality to read credentials, I think this is working?
Unfortunately, I need this on a server where I don't have go available (and I've never used go). Would it be possible to provide x86 builds? Or is there an incompatibility with x86? I think you just need to enable it in your CI!
I tried running go install github.com/gbernady/git-credential-op/cmd/git-credential-op@latest on my local machine but I am a bit lost where it ended up. Also it was super quick but I think that's just go, right? :) So I am a but stuck.

Hey, thanks! Yes, reading credentials should work fine. Never had any issues with it on my machines. I just never got to implementing other actions.
I can enable 32bit x86 builds if you need them. There isn't anything there that might be incompatible, I just rarely see 32bit machines these days.
As for installing from sources, the binary ended up in $HOME/go/bin. And yeah, it should be super quick. There's only one 3rd party build dependency in the project.


/test endpoint should be replaced by /api/v1/test

This is better not to be done. Since we have implemented token based authorization, all endpoints starting with /api/v1/... requires authorization. So this should stay as it is for testing purposes.


Any gradient checkpointing operation?

Nice work!I am using multi-gpu ddp to train, local_tensorfs = DDP(local_tensorfs, device_ids=[args.local_rank], output_device=args.local_rank, find_unused_parameters=True)
When set find_unused_parameters=True, an error is thrown. Is there any gradient checkpointing operation in the code? This may cause the error.
"RuntimeError: Expected to mark a variable ready only once. This error is caused by one of the following reasons: 1) Use of a module parameter outside the forward function. Please make sure model parameters are not shared across multiple concurrent forward- Backward Passes. Or try to use_Set_Static_graph () as a workaround if this module graph does not change training loop.2) Reused Parameters In Multiple Re entyrt Backward Passes. For Example, if you used multiple Checkpoint Functions to wrap the Same Part of your model, it would result in the same set of parameters been used by different reentrant backward passes multiple times, and hence marking a variable ready multiple times. DDP does not support such use cases in default. You can try to use _set_static_graph() as a workaround if your module graph does not change over iterations.
Parameter at index 8 with name t_c2w.0 has been marked as ready twice. This means that multiple autograd engine hooks have fired for this particular parameter during this iteration.”
What's even more excruciating is that when I set find_unused_parameters=False, it throws another error, which confuses me so much that I really need your help, thanks! “RuntimeError: Expected to have finished reduction in the prior iteration before starting a new one. This error indicates that your module has parameters that were not used in producing loss. You can enable unused parameter detection by passing the keyword argument find_unused_ parameters=True to torch.nn.parallel.DistributedDataParallel, and by
making sure all forward function outputs participate in calculating loss.
If you already have done the above, then the distributed data parallel module wasn't able to locate the output tensors in the return value of your module's forward function. Please include the loss function and the structure of the return value of forward of your module when reporting this issue (e.g. list, dict, iterable). Parameters which did not receive grad for rank 1: exposure.0, exposure.1, exposure.2, exposure.3, exposure.4 Parameter indices which did not receive grad for rank 1: 13 14 15 16 17      if torch.is_grad_enabled() and self.reducer._rebuild_buckets(): RuntimeError: Expected to have finished reduction in the prior iteration before starting a new one. This error indicates that your module has parameters that were not used in producing loss. You can enable unused parameter detection by passing the keyword argumentfind_unused_ parameters=Truetotorch.nn.parallel.DistributedDataParallel, and by making sure all forwardfunction outputs participate in calculating loss. If you already have done the above, then the distributed data parallel module wasn't able to locate the output tensors in the return value of your module'sforwardfunction. Please include the loss function and the structure of the return value offorward  of your module when reporting this issue (e.g. list, dict, iterable).
Parameters which did not receive grad for rank 0: exposure.0, exposure.1, exposure.2, exposure.3, exposure.4
Parameter indices which did not receive grad for rank 0: 13 14 15 16 17"

Hi,
I'm not sure what could be the issue. We have several optimizers running at each iteration and operations outside of the forward function (for regularisation losses).
Do you anticipate multi-gpu to significantly speed up the optimisation? Since GPU load is quite low, I believe we have strong overhead from multiple optimizers and a slow dataloader that should be resolved first.

Hi, I'm not sure what could be the issue. We have several optimizers running at each iteration and operations outside of the forward function (for regularisation losses). Do you anticipate multi-gpu to significantly speed up the optimisation? Since GPU load is quite low, I believe we have strong overhead from multiple optimizers and a slow dataloader that should be resolved first.

Yes, you're right, ddp seems to be a lot slower. . .


Updated to reflect support for .NET 6, 2.X, and 3.X

Updated to reflect:

support for: .NET 6 (changed from just .NET 5)
support for 2.X instead of 2.0 because 2.1 was also supported
support for 3.X instead of 3.0 because 3.1 is also supported

Summary
Describe your changes here.
Fixes #Issue_Number (if available)

Hello. Glancing at this PR I'm assuming that the table actually represents the minimum versions, so there is no need to specify X or calling out 6 specificaly.

Now that you point that out, that makes sense. When I read of the table I thought it was outdated. I'll close this PR.

Now that you point that out, that makes sense. When I read of the table I thought it was outdated. I'll close this PR.


Add api endpoint to list available games, stats etc...

I think since games is an argument you pass to the game api server when you create it, (ie since that data is a part of the api implementation) it would be nice if it had an endpoint to list available games, and possibly stats about the players? Maybe high scores or w/e, I think I’ll issue a pull request for the game list endpoint If it sounds like a good idea?

I tried an implementation for a new 'list' API endpoint in the server, to list game instances:
-> at reception of a GET request with format /games/:name, it returns some metadata about instances for the given game name (for now metadata consists in the gameID and the list of players)
-> I also added a 'Lobby' module in the web examples to demonstrate the API endpoints for 'create' and  'list'
https://github.com/francoijs/boardgame.io/tree/add-game-instances-list-api-endpoint

@francoijs Awosone! Feel free to send me a PR.

PR#256 (still some work to do on coverage and MongoDB support)

Marking this as a duplicate of #354


Build failed with the /node_modules/bugsnag-react-native/cocoa/vendor/bugsnag-cocoa/Source/KSCrash/Source/KSCrash/Recording/Tools/NSDictionary+BSG_Merge.m

I am getting this error while compiling the app.{ /node_modules/bugsnag-react-native/cocoa/vendor/bugsnag-cocoa/Source/KSCrash/Source/KSCrash/Recording/Tools/NSDictionary+BSG_Merge.m}
But it worked as expected previously. Suddenly I got an error with this package. Can anyone suggest how to solve this error while building the app?

Thanks for the report @nihp - can you remove the ios/build directory in your project and retry? It sounds like a clean build will fix the issue.

@kattrali My clarification is I have ios/Build folder(Build not build). It has many folders. Do the Build and build files have derived datas?

I have fixed the above error by cleaning the build folder and also removed the derived data.
I have a clarification here. I have ios**/Build folder not ios/b**uild. The both build has the same files or not.

Thanks, glad it worked. 👍


[GrabToPan] Remove more IE-specific code, and remove unconditional browser sniffing

Remove the remaining IE-specific checks from web/grab_to_pan.js (PR 12328 follow-up)
Given that all versions of Internet Explorer are now explicitly unsupported, we can remove some more dead code in this file.

Remove unconditional browser sniffing in web/grab_to_pan.js
According to https://developer.mozilla.org/en-US/docs/Web/API/MouseEvent/buttons#browser_compatibility, and manual testing in an up-to-date version of Google Chrome, the unconditional browser sniffing in web/grab_to_pan.js is no longer necessary.
In particular, this code is definitely not necessary in MOZCENTRAL-builds.

/botio-linux preview

Nice find!


[WIP] Use UIViewPropertyAnimator for adjusting blurRadius

UIVisualEffectView's effect property allows for animation using
UIViewPropertyAnimator. By leveraging this, we can have adjustments of
"blur radius" without using KVC on private APIs.
This is only available for iOS 10. Behavior is unchanged for iOS 9.
Based on feedback from #168.

Can you please solve a conflict here?

I'm running into some issues with this approach when it comes to setting a different blurType and adjusting the values from there.
Since we're animating over the retained views, our animations are referencing view properties (namely, referencing effect) which contains a longer valid property.
I played around with setting up and tearing down views again, so that the references in the animation are correct, but this leads to some inconsistencies as well...
I'd love it if someone else pulled this down and played around with it before I'd call this ready to go.

Should we re-create the view if we receive nextProps.blurType !== props.blurType?
Maybe I'm wrong, but we can re-create a view with custom blurType at the moment we update this prop. Wdyt, @eliperkins?

I'm not quite happy with how this turned out, so I'm going to close this in favor of #183 and then work on top of that...


Clarify binaryData handling by kubectl create and describe

Addresses the documentation portion of https://github.com/kubernetes/website/issues/25921

/approve


epic: Share

https://github.com/34fame/conflux/issues/31#issue-680629586

https://github.com/34fame/conflux/issues/32

Related: https://github.com/34fame/conflux/issues/31


Search feature

Search bar to search blogposts from posts table. Search on title and description column.
Resolves Search feature #15

Great, will review and merge tomorrow!

Sure 🚀


Mac OS运行提示需要访问照片、通讯录等

我在Mac OS端以普通用户身份打开NeteaseMusicHelper，显示了一堆需要访问照片、通讯录等内容的请求，控制台输出如下：
find: /Users/schin//Pictures/Photos Library.photoslibrary: Operation not permitted
find: /Users/schin//Library/Application Support/MobileSync: Operation not permitted
find: /Users/schin//Library/Application Support/CallHistoryTransactions: Operation not permitted
find: /Users/schin//Library/Application Support/com.apple.TCC: Operation not permitted
find: /Users/schin//Library/Application Support/AddressBook: Operation not permitted
find: /Users/schin//Library/Application Support/CallHistoryDB: Operation not permitted
find: /Users/schin//Library/IdentityServices: Operation not permitted
find: /Users/schin//Library/Calendars: Operation not permitted
find: /Users/schin//Library/Messages: Operation not permitted
find: /Users/schin//Library/HomeKit: Operation not permitted
find: /Users/schin//Library/Python: Permission denied
find: /Users/schin//Library/Mail: Operation not permitted
find: /Users/schin//Library/Safari: Operation not permitted
find: /Users/schin//Library/Suggestions: Operation not permitted
find: /Users/schin//Library/Containers/com.apple.VoiceMemos: Operation not permitted
find: /Users/schin//Library/Containers/com.apple.Home: Operation not permitted
find: /Users/schin//Library/Containers/com.apple.Safari: Operation not permitted
find: /Users/schin//Library/Containers/com.apple.iChat: Operation not permitted
find: /Users/schin//Library/Containers/com.apple.mail: Operation not permitted
find: /Users/schin//Library/Containers/com.apple.news: Operation not permitted
find: /Users/schin//Library/Containers/com.apple.stocks: Operation not permitted
find: /Users/schin//Library/PersonalizationPortrait: Operation not permitted
find: /Users/schin//Library/Metadata/CoreSpotlight: Operation not permitted
find: /Users/schin//Library/Metadata/com.apple.IntelligentSuggestions: Operation not permitted
find: /Users/schin//Library/Cookies: Operation not permitted
find: /Users/schin//Library/Caches/com.apple.safaridavclient: Operation not permitted
find: /Users/schin//Library/Caches/CloudKit/com.apple.Safari: Operation not permitted
find: /Users/schin//Library/Caches/com.apple.Safari: Operation not permitted
find: /Users/schin//Library/Caches/com.apple.Safari.SafeBrowsing: Operation not permitted
请问这是正常情况吗？

我来解释一下，这一堆Operation not permitted是find命令在~/目录下寻找NeteaseMusicHelper时提示的，因为~/目录下面有些地方以普通用户身份连读权限也没有（root创建的）。
当然，这个做法很蠢，现在已经改掉了，不再用find命令。请下载并使用最新的NeteaseMusicHelper


Filter completion client-side even if the LSP returns incomplete results

Using rust-analyzer, the LSP server could return partial results for completions, where typing additional characters could cause more completions to show that were not visible in the first batch.
Currently, the completion popup looks like this:
https://user-images.githubusercontent.com/24739403/200195434-a77e619b-ed20-41a2-959d-fe88b44fcf7f.mov
There would be a discrepancy between the user's expectations and the actual completions shown, since the filter operation is noticeably delayed.
If the completion results were not marked as incomplete, it would look like this:
https://user-images.githubusercontent.com/24739403/200195443-1b9c5c57-9e68-4eb9-9d86-39a4e2f7e332.mov
The client would filter the results directly.
Ideally, the client would filter the results directly even if the server returns an incomplete set, while showing a loading indicator. Then, once the server responds with the next batch of completions after server-side filtering is complete, the completion results would be updated with the final value. This would behave similar to IntelliJ's completion display.

related/same as https://github.com/microsoft/vscode/issues/147830

The difference between #147830 is that even if a response from the server arrives delayed, there should be client-side filtering, instead of applying the intermediate response from the server.


Add configuration file

port (dev or prod SSR Only)
public folder (images, json, etc)
add headers for all pages (SSR Only)
multi format yml, json, cfg (if it's possible, it's help the community to have this possibility)

Good idea, I waas thinking of maybe .env for that, it's simple, easy to use, widely used now and useful to not push sensible config, like API keys or database passwords


changing name to SpatialOS GDK for Unity

Contributions: We are not currently taking public contributions - see our contributions policy. However, we are accepting issues and we do want your feedback.

Description
Changing name from Unity GDK to SpatialOS GDK for Unity and fixed up some code examples.
and apparently a lot of whitespacing fixes...
Tests
How did you test these changes prior to submitting this pull request?
What automated tests are included in this PR?
Documentation
How is this documented (for example: release note, upgrade guide, feature page, in-code documentation)?
Primary reviewers
If your change will take a long time to review, you can name at most two primary reviewers who are ultimately responsible for reviewing this request. @ mention them.

I like that. will change!


Alt+ Enter rendering error

Environment
Windows build number: [[Version 10.0.18363.720]]
Windows Terminal version (if applicable):  0.10.761.0

Any other software?

Steps to reproduce
There are two bugs of Alt+Enter behavior.
Text Disappear && Prompt Duplicate
a.  open terminal with WSL
b. type: ls -la or do whatever will print content over a page
c. mouse scroll up to the top
d. type Alt+Enter to Fullscreen
e. text outside orignal windows do not render.
d. scroll down to the bottom: type Alt+Enter several times
f. prompt duplicate several times.
I used zsh with the right side Prompt

Actual behavior

Expected behavior
We need to scroll the terminal  to force a re-rendering.

@Po-wei could you share your profiles.json?
The second screenshot you've shared is certainly very peculiar. It almost looks like the behavior I'd expect from 0.9, but not 0.10. Like the conpty didn't resize to the newly fullscreen'd size. Especially considering the green from the .tooling line doesn't extend to take the full width of the buffer. I'd bet the buffer didn't actually increase in size.
Oh wow actually I just repro'd that immediately. It's almost as if we don't invalidate the screen or something on a resize.
WE almost certainly don't, because we were repainting the entire screen before on a resize, so this wasn't a problem. Yea okay I know what the fix is.

@zadjii-msft
My profile.json of terminal
https://gist.github.com/Po-wei/5512e0a85ea53ad6075a292c2e9bc77f
It is a pretty normal profile.

@Po-wei Thanks for sharing that. I originally thought you might be hitting the "a really large history size causes weird behavior on a maximize", but I was able to actually figure this out already. The fix is already in PR :)


waikamoi name a lat lon change

done!

name changes to: WAIKAMOI HN-154
new coords:
20.773636  -156.222311


⚠ Samsung has degraded performance

In 4c1023e, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3142 ms

Resolved: Samsung performance has improved in 98d3ddb.


https://github.com/kiwiirc/kiwiirc.wiki.git

Bug report
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
System information

OS: [e.g. macOS, Windows]
Browser (if applies) [e.g. chrome, safari]
Server environment [e.g. Go, Java, Node, PHP, Python, Ruby, TypeScript]

Additional context
Add any other context about the problem here.

@salwa445 please provide context to your issue.

@salwa445 please provide context to your issue.


ZBox claims misleading and inaccurate. Please consider rewording the lack of leakage.

SpiderOak made the same mistake with its misuse of the term "zero knowledge" and since migrated to "no knowledge", which hasn't fixed the semantic issue but it has avoided the overlap of terms - by coining another inaccurate term, which you've adopted.
An extract from https://zbox.io/about/ , emphasis mine.

Only encryption is not enough to protect privacy. In addition to strong encryption, Zbox also stores all data in a number of same-sized blocks, which leaves no details to any underlying storages, including Zbox cloud storage. By using this method, only [the] application itself can hold the key and have access to their data.

"Same-sized blocks" may reduce some leakage but this does not imply that nothing remains leaked.

The server knows which users interact with the service and when.
The server knows which blocks the user reads.
The server knows which blocks the user writes.

To prevent (1) the users must identify to the service using a credential that demonstrates their authority without revealing their identity.
To prevent (2) a Private Information Retrieval (PIR) or Oblivious Transfer (OT) protocol must be used; to prevent the server from learning which blocks the client asks for and to prevent the server from learning which value it responds with. PIR may reveal other blocks, potentially owned by other users. Even if encrypted, this is probably not desired. Limiting (2) to OT.
To prevent (3) the dual to PIR whose name I've forgotten may be used to enable a client to upload a block without revealing to the server the block they've pushed into the set.
In so far that I've read, there have been no mentions for PIR or OT or any form of anonymous credentials in ZBox' documentation. When a user reads your claims, they may be led into a false sense of security by correctly interpreting what you've written and not as intended.
Please consider rewording these claims or invest into the cryptographic protocols that make them true. Obviously there are costs in the latter and the choices of which to use depends on how many "cloud" replicas there exist and who controls them. I.e. If there are several replicas owned by distinct parties then you can use some of the cheaper PIR protocols that rely on a ratio of the servers not colluding. If you've only one server, you'll need CPIR (computationally bound) which builds on Diffie-Hellman, or a computationally bound OT.

BTW: The /about/ page says "ZBos" instead of "ZBox" in the 5th paragraph.

Thanks for your suggestion, @james-darkfox . I think this issues had been discussed since beginning, see #4.
PIR and OT protocols are great, but that's not the purpose of Zbox implementation. Its purpose is not trying to hide and be anonymous to server, like Tor. Instead, its goal is to remove the details of data as much as possible. As you said, server knows where you come from, what blocks reads or writes, and when are you doing that, server even knows your email address (if you signed up). It knows you have secret data, we're not hiding that truth, but it just doesn't know what the content is.
'Zero-details' doesn't mean zero knowledge or no knowledge, it definitely has some knowledge but just don't know what's the details, as the name suggested. In a cryptographer's eyes, 'zero-knowledge' has strict claim and that's why I changed the naming to avoid misleading.
Anyway, I will improve words in the doc to give it more accurate description. Thanks again.

Perhaps "near-zero", with a footnote or link to an enumeration of what the server does know.

Sure, I've now updated the wording on https://zbox.io/about, hope it to be more clear and less misleading.

@burmecia I wouldn't quote the near-zero nor mention the things you're not doing. The list of what is leaked is sufficient.


Pipeline output tab has blank rows

Describe the Issue
The Pipeline output seems to populate the correct number of rows / pages when outputting a variety of objects - however the content of the row is blank!
To Reproduce
See Script - Output Service to Pipeline (have not attempted a wide variety of types yet)
Screenshots
UAD/UA UI

Note, I do have the correct details in the Database

Script Sample

$Services = Get-Service
$Services | ForEach-Object {
     Write-Output $_
}

Version Information

UADesktop and UA
WIndows 10 / Server 2019
5.1 and 7
1.1 and Latest Dev

Looks like PS 7 XML is slightly different than PS 5.1. Looking into it.

Even more strange...the CliXml returned within UA is different than when I run on the command line.


ScreenshotManager::save_screenshot_to_disk saves broken PNG for wasm

Bevy version
0.12
[Optional] Relevant system information
Operating System: Fedora Linux 37
KDE Plasma Version: 5.27.8
KDE Frameworks Version: 5.108.0
Qt Version: 5.15.10
Kernel Version: 6.5.12-100.fc37.x86_64 (64-bit)
Graphics Platform: X11
Processors: 4 × Intel® Core™ i5-2400 CPU @ 3.10GHz
Memory: 15.6 GiB of RAM
Graphics Processor: OLAND

If you cannot get Bevy to build or run on your machine, please include:

the Rust version you're using (you can get this by running cargo --version)

1.72.1

the operating system or browser used, including its version

Fedora 37, Mozilla Firefox Flatpak 120.0.1

If your bug is rendering-related, copy the adapter info that appears when you run Bevy.
AdapterInfo { name: "OLAND (, LLVM 15.0.7, DRM 2.50, 6.5.12-100.fc37.x86_64)", vendor: 4098, device: 0, device_type: Other, driver: "", driver_info: "", backend: Gl }

You should also consider testing the examples of our upstream dependencies to help isolate any setup-specific issue:

wgpu for rendering problems
winit for input and window management
gilrs for gamepad inputs

What you did

Apply official screenshot example.
cargo install wasm-server-runner
cargo run --target wasm32-unknown-unknown
Press space to take screenshot.
Try to open PNG file.

What went wrong
If it's not clear, break this out into:

what were you expecting? For Gwenview to properly display it.
what actually happened? PNG doesn't display.

Additional information
Other information that can be used to further reproduce or isolate the problem.
This commonly includes:

screenshots
logs
theories about what might be going wrong
workarounds that you used
links to related bugs, PRs or discussions

See here: https://github.com/Vrixyz/bevy_jam_4/pull/1#issuecomment-1849745123
Files generated: https://file.io/BUBrn8bmWPA5

I was unable to reproduce on macos / chrome 120.0.6099.109.
INFO crates/bevy_render/src/renderer/mod.rs:140 AdapterInfo { name: "ANGLE (Apple, ANGLE Metal Renderer: Apple M1 Max, Unspecified Version)", vendor: 4203, device: 0, device_type: IntegratedGpu, driver: "", driver_info: "", backend: Gl }

I also have this problem for my application, but only about one in five screenshots are broken.


AVS and Raspotify?

Is there anyway that I can run raspotify and avs. For example can I be playing a song through spotify connect/raspotify and then tell the avs (alexa) to pause the spotify music?

Hi neeraj2314 ,
your question is better placed in the librespot-team - >https://github.com/librespot-org/librespot , raspotify is an installation tool not the actual program


Authenticate

Description
Add Authenticate mechanism while creating new connection.
Add Authenticate request and possible responses to the frame.
Add serializers, deserializers.
Fix
Fixes: #13
Pre-review checklist

[x] I have split my patch into logically separate commits.
[x] All commit messages clearly explain what they change and why.
[ ] I added relevant tests for new features and bug fixes.
[x] All commits compile, pass static checks and pass test.
[x] PR description sums up the changes and reasons why they should be introduced.
[ ] I added appropriate Fixes: annotations to PR description.

I somehow forgot to commit my changes on Friday 🤦♀ But here is updated, working version, I just rebased it. So ready for review!
And I have a question about testing - I thought the best way is to add new scylla container with option --authenticator AllowAllAuthenticator and test on default superuser Cassandra (which I did) - I will add this container to the GitHub CI next thing tomorrow*, but now I am not so sure. Maybe we should create some sample users? Which option is better?
*In this commit I will also add a test that checks how authentication fails - so that all commits would pass checks separately 😅

Does this cassandra superuser have a password we can verify? If so, it's ok as an initial test, but it would be much better to prepare a scylla container with some pre-created users, and then test credentials for these users.

Also, please make sure that if we get an AUTHENTICATE message from the server, but the authenticator is not password authenticator, then we end up with an error message that this authenticator type is not supported yet.

I found a way to run some tests separately in GitHub in a different container - for example with some users specified. I wanted to consult scheme for test I came up before pushing them to this PR:

for each authentication type there would be a separate test
each test would consider of following:
-- create a connection with superuser Cassandra and create sample user
-- create new connection with sample user
-- failed attempt to connect with invalid credentials
each test in GitHub would run as a different check (since we need a different container for them)
Does it sound like a good idea, or there are more test cases we want to cover?

Also - I just added all of Scylla supported authentication types, will push implementation for missing ones in another commit tomorrow 😅

There seems to be a conflict with current main - please rebase on top of it (preferably with git pull --rebase to avoid merges).

The code looks good. After fixing the CI issue:

.github/workflows/authenticate_test.yml#L24
The workflow is not valid. .github/workflows/authenticate_test.yml (Line: 24, Col: 9): Unexpected value 'authenticator' .github/workflows/authenticate_test.yml (Line: 30, Col: 3): 'build' is already defined

it looks good to go.

@cvybhu could you assist with getting CI to work with authenticators?

ping

@psarna
We didn't find any way to enable authentication for scylla service nicely in the CI

There doesn't exist any way to pass arguments to docker run
Mounting files doesn't really work because services start before the checkout action and the files are not there yet
We could set some environment variable but we couldn't find any that would enable authentication

We will just have to create a custom image on Dockerhub :/

That's alright, I was actually sure that a custom docker image is the way we agreed to do it in the first place :smile:

@psarna CI is working now


Add duplicate button for mobile view

Resolves #1142
In the desktop version, the duplicate and delete buttons are provided, and in the mobile version, since the delete button already exists, only the duplicate button is provided.

Desktop
Mobile

Since when do we have the layer actions on the left pane?

Since this merge https://github.com/excalidraw/excalidraw/pull/1140

My bad then.. I didn't notice that it was added there.. we should remove those on desktop.. no?!
Sorry @j-f1.. I missed your comment.

In the context of #1125, I understood it as adding to the desktop and mobile respectively. So, is there any way to distinguish a tablet from the current codebase?

And does anyone know why this check fails? Should we check the SENTRY_PROJECT env variable?

Yes, ignore the Sentry check, we'll figure it out in another PR.

For mobile, can we put the duplicate next to the trash icon? I feel like I'm using it a lot and it is worth being more prominent.

If you do my recommendation, then I think it's good to go. Thanks for helping out!

Currently the “finalize line” button is in that spot @vjeux but we could get rid of that button since you can now tap on the end of the line to finalize it.

We don't need to duplicate when drawing a multi-point (I have no idea what that would even do!). We can use the same spot for both.

Thanks for the feedback!
The 'finalize' and 'duplicate' buttons are branched from the top level because the space becomes narrower when there are 7 default buttons.

can we hide the action section when there are no buttons displayed..

Alternatively, we could show those buttons only for touchscreen-enabled devices, using detection like this:
'ontouchstart' in document.documentElement || navigator.maxTouchPoints

Though this may stop working when anti-fingerprinting becomes more widespread.

I don’t think that’s a good idea @dwelle — there are lots of Windows laptops that support touch, and the iPad supports a full keyboard. I think it’s better UX to provide multiple ways of doing the same thing (keyboard, context menu, visible button).


Standardize server init and cleanup a few other small items

The big part of this PR is standardizing some of how servers are initialized. Importantly, this moves federation discovery and some key creation from the various *_serve.go files into InitServer().
In the process of putting this together, I noticed a few other cleanup items. One was changing a parameter.yaml default value and type to match the description. Another was modifying config_test to use assert.Equal instead of assert.True so that if the test fails, it prints out what the expected and actual values are.

@haoming29, I know we happened to stumble on some of the same stuff. If you think I got any of this wrong, or if you found anything we need to add that I didn't catch, feel free to use this PR as a base and open a new PR from your own branch. If that's the case, I'll close this one and we can switch to yours.

This should also close #256

@bbockelm @haoming29 What do we think the solution to the failing test should look like? It looks the problem is caused by trying to create the TLSCACertificateFile in /etc/pki, which the test runner doesn't have perms to do. If this is the default for Linux, we probably don't want to change the ultimate target location, but this seems to imply we can't create the key in the intended spot unless run with sudo or as root...

@bbockelm @haoming29 What do we think the solution to the failing test should look like? It looks the problem is caused by trying to create the TLSCACertificateFile in /etc/pki, which the test runner doesn't have perms to do. If this is the default for Linux, we probably don't want to change the ultimate target location, but this seems to imply we can't create the key in the intended spot unless run with sudo or as root...
What are the consequences of trying to write to /etc/pki and if unable, using a default of /etc/pelican/certificates?

Why not overwrite the default value for TLSCACertificateFile location for the test to use the t.tempDir? Just for testing purposes?

@jhiemstrawisc Why not overwrite the default value for TLSCACertificateFile location for the test to use the t.tempDir? Just for testing purposes?

👆

@jhiemstrawisc Why not overwrite the default value for TLSCACertificateFile location for the test to use the t.tempDir? Just for testing purposes?

More specifically, this is a parameter that can be set via the config file. If we can't handle that location being different in any way, then why is it something that is alterable in a pelican.yaml file? Also, if the user isn't root, then the location is being set via the InitServer() to be some other location already, I believe. The issue here is that the tests are being run with the assumption that the user is root, but the test driver doesn't have root permissions, hence the need for the temp director.

@bbockelm @haoming29 What do we think the solution to the failing test should look like? It looks the problem is caused by trying to create the TLSCACertificateFile in /etc/pki, which the test runner doesn't have perms to do. If this is the default for Linux, we probably don't want to change the ultimate target location, but this seems to imply we can't create the key in the intended spot unless run with sudo or as root...
What are the consequences of trying to write to /etc/pki and if unable, using a default of /etc/pelican/certificates?

Do we actually set Server.TLSCACertificateDirectory to its default value in the code? Not sure if I missed anything but I can't find any viper.SetDefault("Server.TLSCACertificateDirectory") anywhere

Member

@bbockelm @haoming29 What do we think the solution to the failing test should look like? It looks the problem is caused by trying to create the TLSCACertificateFile in /etc/pki, which the test runner doesn't have perms to do. If this is the default for Linux, we probably don't want to change the ultimate target location, but this seems to imply we can't create the key in the intended spot unless run with sudo or as root...
What are the consequences of trying to write to /etc/pki and if unable, using a default of /etc/pelican/certificates?

Do we actually set Server.TLSCACertificateDirectory to its default value in the code? Not sure if I missed anything but I can't find any viper.SetDefault("Server.TLSCACertificateDirectory") anywhere

@haoming29 https://github.com/PelicanPlatform/pelican/blob/ffbe0d54d63cee7c2a3e3894d3934ca3c3a9d080/config/config.go#L402C1-L403C1
I think you were looking for the wrong name.

@turetske Sorry I shouldn't quote reply that comment, but I meant Server.TLSCACertificateDirectory. It's default value is being changed in this PR.

@haoming29 Ah. I see, I was getting confused. I understand now. Then we don't set it. It's possibly something we want to if that's possible.
@jhiemstrawisc This changes my above comment. If we can't set it, then that's a problem for the reasons you described above, I didn't realize the directory was different than the file.

@haoming29 @turetske, I think this is mostly sorted out. It boiled down to the fact that apparently there are a variety of default CA Certificate paths that should be checked based on which version of linux you're running. Rather than assuming we're on a RHEL derivative (which we did before by hardcoding /etc/pki/tls/cert.pem), the additions here now check all the same spots the x509 package checks for a certificate when the host OS is linux. I figure the folks who wrote the official Golang x509 package probably know a bit more than I do about all of this 😜  If none can be found, we create one in $configDir/certificates, which Pelican guarantees is writable.
On that note, we might eventually take inspiration from the x509 package in setting this up for Darwin and Windows: https://go.dev/src/crypto/x509/


Update docs for feat: add geohash and h3 as built-in functions

A document change request is generated from https://github.com/GreptimeTeam/greptimedb/pull/4656

I will add docs when more h3 functions are added.


Catch warnings

Catches model warnings from parallel processes in _run_cv and _optimize_params.
Adds SGD back into MLP search space

resolves #211

Codecov Report
Attention: Patch coverage is 80.76923% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 90.86%. Comparing base (5bdf08b) to head (bc0c9e5).

Files
Patch %
Lines

autoemulate/compare.py
70.00%
3 Missing :warning:

autoemulate/utils.py
87.50%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #212      +/-   ##
==========================================
+ Coverage   90.83%   90.86%   +0.02%
==========================================
  Files          45       45
  Lines        2281     2299      +18
==========================================
+ Hits         2072     2089      +17
- Misses        209      210       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Core: Use importlib's import for compatibility with patching

This uses importlib.import instead of plain import
The package 'pythonnet' breaks kivy by patching python's global import.
Of course it's not kivy to blame here, but using importlib doesn't hurt and as a side effect is better documented.
https://github.com/pythonnet/pythonnet/issues/1245
https://github.com/pythonnet/pythonnet/issues/547

Thanks for the fix. There are some PEP8 issues though (please see the failed test).

thanks @matham for the quick response and title fix! :)
just pushed the pep8 fix..


Fix: add remittanceText

Correct the format as specified in http://www.europeanpaymentscouncil.eu/index.cfm/knowledge-bank/epc-documents/quick-response-code-guidelines-to-enable-data-capture-for-the-initiation-of-a-sepa-credit-transfer/epc069-12-quick-response-code-guidelines-to-enable-data-capture-for-the-initiation-of-a-sepa-credit-transfer1/

Good catch. Thanks!
The spec says it should be one or the other (reference or text).
I'll add validation for this in the setters.


unocss for react native

Guys Would like possible to make work UnoCSS/webpack with react native ?

I've been trying lately(Demo experiment).
I don't know what will antfu thinks.

I am not familiar with React Native, but if it works with CSS, UnoCSS should work.

Thanks, Iam testing it :)
The only detail I dont understand is that ReactNative use className instead of class, does UnoCSS can know that ?
and for uno preset I don't know how to install ?

Unfortunately: it's not CSS. I demo some work. But a lot of work is expected.

so you would inject the UnoCSS to styleSheet object ? uno work only from class. Its a special UnoCSS for react native it needed for that. Maybe some peaple could port it a day.

I think we need to complete the code replacement and replace the class with the form of style = {}`. The other is how to integrate the attributes according to the stylesheet specification.
However, I have no confidence in performing the operation of code replacement before compiling the source file.
What do you think? Will we support this plugin？
If you plan to support it, you can start by completing the dynamic replacement part of the source code? I'll help you complete the rest. Emmm I don't know about webpack @antfu

There is package named "css-to-react-native". if someone want collab with me, we can work on it


Add gNSI large payload tests

Pending https://github.com/openconfig/featureprofiles/pull/2081 from @haussli

Pull Request Functional Test Report for #2165 / 611c8b65798ed90bf1f987588147048406a925b4
No tests identified for validation.
Help

@haussli to review


Update core

Pull Request Test Coverage Report for Build 101

152 of 198 (76.77%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+4.7%) to 75.248%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

main.go
0
12
0.0%

k8s/client.go
105
120
87.5%

k8s/config.go
47
66
71.21%

Totals

Change from base Build 92:
4.7%

Covered Lines:
152

Relevant Lines:
202

💛  - Coveralls

Seems ok - in the future we should come up with a way to validate that the vendor folder is in sync with the toml/lock  (some kind of verification step as part of the travis build)


Add CSI snapshot controller and webhook to Cinder CSI

What type of PR is this?
/kind feature
What this PR does / why we need it:

Add CSI snapshot controller and webhook to the Cinder CSI driver

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
xref #2051
Does this PR introduce a user-facing change?:
Add CSI snapshot controller and webhook to the Cinder CSI driver

/approve
/lgtm

 pkg/addons/applier.go:278:65: `webhookCerts` - `webhookNamespace` always receives `"kube-system"` (unparam)
func webhookCerts(certs map[string]string, prefix, webhookName, webhookNamespace, serviceDomainName string, kubeCAPrivateKey *rsa.PrivateKey, kubeCACert *x509.Certificate) error {
                                                                ^


dist directory is not cleaned if one of outDir is . (root)

Environment
Node.js v20.15.0
pnpm v9.11.0
unbuild v2.0.0
Reproduction
E.g. if I have multiple entries, but one of the ourDir is root, it will only clean ./ but not in ./dist
entries: [
  'src/index',
  { input: 'src', outDir: '.', pattern: ['**/*.css'], builder: 'mkdist', loaders: ['postcss'] },
],

The file in ./dist folder will only be overwritten, but in case any garbage files, they wouldn't be cleaned.

If I change to another subdirectory, it's going to work
entries: [
  'src/index',
  { input: 'src', outDir: './foo', pattern: ['**/*.css'], builder: 'mkdist', loaders: ['postcss'] },
],

Describe the bug
See Reproduction. This is causing files in ./dist folder is not cleaned, where I want the .css file to be in the root directory to publish to NPM.
Additional context
In CI it's causing it to crash because it's really cleaning up the whole root directory, resulting in

Even if I set cleanDist in one of the entry, it's not going to help, it's still cleaning the root
entries: [
  'src/index',
  { input: 'src', outDir: '.', pattern: ['**/*.css'], builder: 'mkdist', loaders: ['postcss'], cleanDist: false },
],

Logs
No response

Seems not an issue anymore via #343 ?


AWS ECS Service - Pagination Issue

Describe the bug
aws_ecs_service not properly handling pagination.
Steampipe version (steampipe -v)
Example: v0.12.2
Plugin version (steampipe plugin list)
Example: v0.48.0
To reproduce
When there are more than 10 services in the account/region...
select * from aws_ecs_service
Expected behavior
Pagination properly handled.
Additional context
Warning: executeQueries: query 1 of 1 failed: ERROR: rpc error: code = Unknown desc = InvalidParameterException: service names can have at most 10 items. (SQLSTATE HV000)

@twratl Thanks for trying out and raising the issue. We are looking into this issue.


Deeplinks do not open using SDK on android

SDK version: 3.1.1
Describe the issue
I had an integration with Customer.IO without SDK, but i want to monitor events like opened and delivery, so I add the Customer IO SDK for android following the documentation, but my deeplinks was stop working, when I click in notification it redirect me for main activity, if the deeplink is a web page, it opens the page on browser but stay in background, with app in main activity.

OBS: The notification is delivery and monitory correct, only problem is the handle of deeplinks.

Hi @thiaagodev Thanks for sharing the details on the issue. Can you please confirm the following:

targetSdkVersion of your app
The state of your app when testing the notification i.e. open, background or closed
The type of deep links you are trying to open i.e. are the links handled by your app or do you want to open another app using link in notification?
Are the supported links registered in AndroidManifest?

Hi @mrehan27

My targetSdkVersion is 31
Its hapen in all state, open, background or closed
Both, links to my app and links for web browser (chrome opens but stay in background)
Yes, they are registred, previously I was use my own implementation of FirebaseMessagingService and it works, but with Customer IO SDK the deeplinks are not handle correctly, every opens the main activity of my app when i click in the notification.

Thanks @thiaagodev. Unfortunately, due to notification trampoline restrictions from Android 12, links outside your app will be opened with your launcher activity being opened in the background. So the browser over your main activity is desired behavior. If you want to override this behavior, you can provide callback in setNotificationCallback and override createTaskStackFromPayload to open browser only.
For the links supported in your app and not opening, can you please set log level to DEBUG and see if you notice any error or anything unusual in Logcat? Or if convenient for you, you can email your logs at<EMAIL_ADDRESS>with link to this ticket so we can look into them in detail and identify the root cause.

Thank you @mrehan27 , could you send me an example of the setNotificationCallback implementation? I didn't find in the documentation how to do it.
I'm going to test the links to my app with the logs to check for errors.

@thiaagodev I think you can find it in the second point in the deep-link docs
class MainApplication : Application(), CustomerIOPushNotificationCallback {

  override fun onCreate() {
    super.onCreate()
    val builder = CustomerIO.Builder(
      siteId = "YOUR-SITE-ID",
      apiKey = "YOUR-API-KEY",
      appContext = this
    )
    builder.addCustomerIOModule(
      ModuleMessagingPushFCM(
        config = MessagingPushModuleConfig.Builder().apply {
          setNotificationCallback(this)
          setRedirectDeepLinksToOtherApps(true)
        }.build()
      )
    )
    builder.build()
  }

  override fun createTaskStackFromPayload(
    context: Context,
    payload: CustomerIOParsedPushPayload
  ): TaskStackBuilder? {
    // return TaskStackBuilder of your choice if you plan to handle the deep link yourself
    // return null if you want CustomerIO SDK to do it for you
    TODO("Pass the link to your Deep link managers")
  }
}

I am having this error in logcat:
queue task 99cac6bf-eb11-4b3f-855c-805df83bd5e8 run failed UnsuccessfulStatusCode(code=400, apiMessage={       "meta": {         "errors": [           "malformed delivery id: ."         ]       }     }     )

Hey @thiaagodev, I think you sent a test message from CIO workspace and those test messages don't have delivery ids, hence the error.
Regarding the notification, you can customize the notification before CIO displays them or you can even display them using your own service, and also notify our SDK to track them. You can find more information here.

Closing due to inactivity.