trix values does not get saved

My Model is like this:
class Form extends Model
{
    use HasTrixRichText;

    protected $fillable = [
        'form_id',
        'title',
        'description',
    ];

    protected $guarded = [];
}

blade like this:

  {{ trans('cruds.form.fields.content') }}
  @trix(\App\Form::class, 'content')
  @if($errors->has('content'))
    {{ $errors->first('content') }}
  @endif
  {{ trans('cruds.form.fields.content_helper') }}

Controller like this:
$data = $request->all();
$data['form_id'] = NumberSequence::getNextNum('forms');
$form = Form::create($data);

nothing get save to the DB

@rabol can you try to remove the $fillable from your model

its is not in my fillable
but... I managed to get it working by adding
form-trixFields to the $fillable

@rabol  ok cool i will close the issue since it's fixed

No, it's not fixed, it's a work-around.
According to the documentation one should not add the field to the $fillable

@rabol if you removed $fillable totally and set $guarded as [] it should work

hm... I'll find another solution as $fillable is a 'Laravel' think I like for serurity

@rabol ok got it i think we will need to add this to the documentation

idk why but i have this problem.. ive set my models with $guarded[] but they didn't want to detect it as 'content' column
here is my model
`<?php
namespace App;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;
use Te7aHoudini\LaravelTrix\Traits\HasTrixRichText;
class Blog extends Model
{
use HasFactory;
use HasTrixRichText;
protected $guarded = [];

// protected $fillable = ['tag', 'title', 'content',];

}`
And here's my form
`
Konten untuk Visi :
                        @trix(\App\Blog::class, 'content');

                    <input type="submit" class="btn btn-primary float-right"/>`

the models is supposed to be in /App/Models/Blog, but i've moved it too /App.. I thought that was the problem, but apparently I still can't create any blog. Please help me
Thanks

idk why but i have this problem.. ive set my models with $guarded[] but they didn't want to detect it as 'content' column
here is my model
`<?php
namespace App;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;
use Te7aHoudini\LaravelTrix\Traits\HasTrixRichText;
class Blog extends Model
{
use HasFactory;
use HasTrixRichText;
protected $guarded = [];

// protected $fillable = ['tag', 'title', 'content',];

}`
And here's my form
`
Konten untuk Visi :
                        @trix(\App\Blog::class, 'content');

                    <input type="submit" class="btn btn-primary float-right"/>`

the models is supposed to be in /App/Models/Blog, but i've moved it too /App.. I thought that was the problem, but apparently I still can't create any blog. Please help me
Thanks


Failure when installing Add-Onn

I get the following message when trying to install this add-on. Tried to refresh the repository (deleted and added), but same behaviour
Can't install davidusb/image-armv7-emhass:0.2.19: 404 Client Error for http+docker://localhost/v1.40/images/create?tag=0.2.19&fromImage=davidusb%2Fimage-armv7-emhass&platform=linux%2Farm%2Fv7: Not Found ("manifest for davidusb/image-armv7-emhass:0.2.19 not found: manifest unknown: manifest unknown")

Hi, yes this is normal. The images are being built. The ARM images take very long time to compile. I unsuccessfully tried different thing to reduced compiling times. I'm still looking for solutions to this.
As you can see here:

The last built timeout after 6h of compiling. I relaunched it 2h ago to see if it compiles correctly this time...

Ok, it's done you should be able to install with no problems.
Found a solution for faster compiles on arm archs, now it compiled on just 1h:


feat(telescope): add <leader>sS for :Telescope lsp_workspace_symbols

I found myself often wanting to search trough the workspace symbols and i think this keymap makes the most sense, since there is already the same thing for document symbols with <leader>ss.

Makes sense!


No devices discovered

The problem
After successful authorization, no devices are detected through the integration. The devices are visible though the Miele Swagger-UI with the same Client_ID and Client_Secret. No errors are generated in the log, but the diagnostics file display the devices and its attributes correctly (see attachment:
config_entry-miele-448c31f5a9b474550c6833e12ade258d.json.txt
It is however not added as a device / entity.
Version of miele integration having the issue?
0.0.19
Version of Home Assistant Core having the issue?
2022.5.3
Anything in the logs that might be useful for us?
No response
Additional information
No response

@arief1986  It is a simple task to just add the device with some generic status sensors. However, you are the first integration user that has access to an induction hob. I will need some help with ideas/tests/verifications when implementing control entities for the cooker plates.

Many thanks for your quick response! Yes, that's indeed the induction model with an integrated extraction fan. Happy to do some more scenarios and extract the diagnostic dumps if that helps.
I've put a kettle on various plates and played with the induction power and fan speed a bit. It seems that many states are read correctly.

For induction power (plateStep), "value_localized" is indeed the set induction power (0 to 9; and "Boost\u00a01" for boost); and are read for each zones correctly.
For fan speed, "value_localized" is the correct speed (0 to 3; and "B" for boost)
Hob status is displayed in "state|status|value_localized", either "Off" or "In Use"

I have attached the diagnostics dumps. Thanks for your help!
config_entry-miele-448c31f5a9b474550c6833e12ade258d.json-2.txt
config_entry-miele-448c31f5a9b474550c6833e12ade258d.json-3.txt
config_entry-miele-448c31f5a9b474550c6833e12ade258d.json-4.txt
config_entry-miele-448c31f5a9b474550c6833e12ade258d.json-5.txt
config_entry-miele-448c31f5a9b474550c6833e12ade258d.json-6.txt

Thank you. Can you please list the pairs of value_raw and value_localized for plateSteps. I use value_raw in the integration
in all other device types since it is more reliable across the supported languages.
I am not very hopeful that it is possible to control the settings through the API. The actions response indicates what actions and settings that are available at each specific moment. All parameters are empty or false which marks that nothing can be altered just now. Is there a setting on the physical device to enable remote control?

Thanks! See the list of pairs below:
value_localized: 0  / value_raw: 0
value_localized: 1  / value_raw: 1
value_localized: 2  / value_raw: 3
value_localized: 3  / value_raw: 5
value_localized: 4  / value_raw: 7
value_localized: 5  / value_raw: 9
value_localized: 6  / value_raw: 11
value_localized: 7  / value_raw: 13
value_localized: 8  / value_raw: 15
value_localized: 9  / value_raw: 17
value_localized: Boost\u00a01  / value_raw: 117
I think you are right with regards to the "actions". This does not seem to be possible in the official Miele app either, so it's probably read only for this device (which makes sense for a hood).
Let me know if you need more tests / diagnostics.


Probestück 4 versions

The second version of PS4 is four bars longer than the first version (apart from other differences). How should we encode this?

Very good question – it might also concern the general question of how we can put text, score and additional material into well-defined relations …
(BTW, since it might be related: now that verovio supports selecting score portions using EMA, we might (re)consider using it for encoding references from text to score …?)


Bug 5537 rename command

[X] Bug fix (non-breaking change, fixes #5537)

Purpose
renaming a command

thanks again! so sorry, i missed on the last one, that it should not be DbaTools but rather Dbatools. If it's not too burdensome, can you also rename Install-DbaWatchUpdate and Uninstall-DbaWatchUpdate
Renaming the file itself to change the casing may require a delete, add, recommit
🙇


Please document how to use the font tme-fa-icons in StoryMenu declarations

Please explicitly document how to use the font tme-fa-icons in StoryMenu declarations.
I spent most of today researching and getting help in how to add "fancy" icons to my own StoryMenu entries, like those used by SugarCube's default UI menu entries. I think it would be very helpful for other novices like myself, saving them a lot of time, if there were an explicit description of the procedure in the SugarCube documentation. It might be there by implication, but, if so, it was not at all obvious to me.
For example, it might be appropriate to

Mention the name of the font that's used in SugarCube's default menu entries (tme-fa-icons)
Mention how to find out what characters it provides and their codes.
If it exists, mention the name of SugarCube's  "i." (italics) class which is used to specify the font
Otherwise, give an example of creating such a specification
Perhaps mention the history why 'i." is used instead of some other HTML markup prefix.
Show an example of code that one can use to prefix an icon to the menu entry.

FWIW, I finally managed to implement the icons in the story that I'm currently working on by using the following procedure:
I created this Story Stylesheet entry
i.fa
{ font-family: 'tme-fa-icons';
  font-style: normal;
}

and invoked it using these StoryMenu entries
<<link ' Navigation Help' "Navigation Help">><</link>>
<<link ' Report a Problem'>> <<bugreport "Bug Report Instructions">><</link>>

While this isn't necessarily an explicit document, there is a reference to tme-fa-icons that you may find helpful.
The convention of using i is actually derived from FontAwesome. In this case, you actually only need the class name; the ::before pseudo-element which contains the code is defined in the css.

@TwelveNights
Was that directed to me or @tmedwards ?
I think I managed to work out the essentials, it's just that it took a lot of time. My comments above weren't a request for information, but mostly were an attempt to suggest some of the items which might be included if that information gets added to the documentation.
Thanks for the pointers, though. I'll be looking through them.

Resolved by e776abaa07e2350241d0420df351033ecdbb4406.


-Wincompatible-pointer-types warning in remote.c

Noticed the following -Wincompatible-pointer-types warning when packaging 4.6.1 for Gentoo:
x86_64-pc-linux-gnu-gcc -I. -O2 -pipe -march=native -fdiagnostics-color=always -frecord-gcc-switches -Wreturn-type     -ggdb3 -Werror=format-security -Werror=implicit-function-declaration -Wimplicit-int -Wformat -c remote.c
remote.c: In function ‘do_stats’:
remote.c:1216:28: warning: passing argument 1 of ‘ssl_printf’ from incompatible pointer type [-Wincompatible-pointer-types]
 1216 |         (void)ssl_printf(rs->ssl, "error no stats enabled at compile time\n");
      |                          ~~^~~~~
      |                            |
      |                            SSL * {aka struct ssl_st *}
remote.c:766:17: note: expected ‘RES *’ {aka ‘struct remote_stream *’} but argument is of type ‘SSL *’ {aka ‘struct ssl_st *’}
  766 | ssl_printf(RES* ssl, const char* format, ...)
      |            ~~~~~^~~

$ gcc --version
gcc (Gentoo Hardened 12.2.1_p20221105 p1) 12.2.1 20221104
Copyright (C) 2022 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Clang gives a similar warning:
remote.c:1216:19: warning: incompatible pointer types passing 'SSL *' (aka 'struct ssl_st *') to parameter of type 'RES *' (aka 'struct remote_stream *') [-Wincompatible-pointer-types]
        (void)ssl_printf(rs->ssl, "error no stats enabled at compile time\n");
                         ^~~~~~~
remote.c:766:17: note: passing argument to parameter 'ssl' here
ssl_printf(RES* ssl, const char* format, ...)
                ^
1 warning generated.

Full build log: build.log

Wow, thanks for the quick fix (and for nsd)!

The --disable-bind8-stats option is what causes this code to get compiled, the bug is likely since the introduction of named pipes for nsd-control in 4.1.24, 13 aug 2018. It fails to print the error that the stats command is not compiled in.
This happens when nsd-control stats is performed. Good to fix because the wrong pointer could not result in the correct message printed!


[HUDI-8670][DOCS] Blog post for non-blocking concurrency control

Change Logs
A new blog for NBCC.
Impact
none
Risk level (write none, low medium or high below)
none
Documentation Update
none
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

Will land once CI passes.


feat: Add a Market Manipulations Widget

The task was to implement a widget using real Market Health data from this RapidAPI endpoint.


Makes pbdirect cross scala versioned with 2.11.11

This PR brings cross scala compatibility with 2.11.11. It was already compatible, I'm just enabling it ;)
Hope it makes sense. Thanks.

Thanks for doing this, just found this lib and would love to use it, we're on 2.11 as well.

Hi @btlines , we've started to evolution our fork since we need to make progress in the freestyle-rpc library where we're working on. Let us know if, in the future, you keep updating this great library and we'll be happy to remove our fork. Thanks!

Thanks for the patch @juanpedromoreno


Hololens2 Not responding to SDP offer and no Video Rendering

Describe the bug
I have setup my unity in accordance to the tutorial located : https://microsoft.github.io/MixedReality-WebRTC/versions/release/2.0/manual/unity/helloworld-unity.html . I am able to get local video on my unity player, i am also able to see the exchange of GET datas 404 between the hololens and my laptop in the dss.
i have created a button in my unity project which essentially just calls peerconnection.StartConnection() , when i call this the SDP offer message can be seen but then the hololens drops out of the dss and nothing more happens.
The problem is i am unable to see any video on my hololens2.
SDP messages
first picture is before the button is pressed and the second two are after the button is pressed. The button is pressed on the laptop and contains the code in the following screenshots
Dss before button click

button click code

Dss After Button click

Screenshots
here are screenshots of what has been configured in unity

Environment
Please fill the information for each peer if different

MR-WebRTC version 2.0.2
Platform: [e.g. Windows Desktop, Unity Editor, and Hololens 2
Unity version (if applicable): 2020.3.11.f1
Target device: Hololens2

This could be a firewall issue with your PC - try (temporarily) disabling your Windows Firewall and any third party firewall software.
If this works, you may want to investigate running a TURN server, which provides an alternative path when peers are unable to directly communicate.
Another thing to check: if you have enforced H264 video codec for HoloLens performance, then try temporarily disabling that. The unity editor/windows desktop versions cannot handle H264 video (they only support VP8).

i have indeed disabled the firewalls on my PC (temporarily) and notice the same result, in terms of the video codec, i just went and had a look within the webcam source script and found the script i feel may be responsible (which is commented out) would you be able to provide some guidance as to what is supposed to be here? if its supposed to change. Thank you

Hi @cherrysodalady, where you able to solve this problem? I have actually the same problem as you.
I am trying to send local video from unity to the hololens.
If you have any advice please let me know.
Thank you

How do I change Videocodec? I am looking everywhere but cannot find anything.
I also decommented those lines of code but I get a lot of errors.

Maybe the Problem is that the VideoFrameCodec is different from the one which is attached to the VideoRenderers. If your video source delivers VideoFrames with codec I420, the renderer which displays the video has to use YUVFeedMaterial. If it delivers ARGB32 you need to use the ARGBFeedMaterial to render. Unfortunately i didn´t find any possibility to make ARGB32-based  VideoSources work in MR WebRTC. It seems like the native peers always use I420 codec and I didn´t find any way to change that behaviour. But since you are using WebcamSource it should already be I420 so your Renderer should all use YUVFeedMaterial. Maybe this is your problem. Tell me if it was :)

Thanks for the answer. Yes I am using WebcamSource and my Renderer uses YUVFeedMaterial but nothing works.
The problem appears when I press the button "CreatOffer" in unity to start the connection. When I press that button my hololens disconnect from node dss and I do not know why. Also another problem is that on the local screen on the hololens I should see what my hololens camera is seeing but instead I get a black screen.
Do not know how to proceed I really tried everything.

I tried it myself and unfortunately have the same problem. No Webcam-Stream at all.. only black screens.. So far I also have no idea about the reason for that.

hi @Matteo-0  - i have not solved the problem yet, i have also not used "CreateOffer" but instead "StartConnection()" as can be seen in my picture, is this different to what you  have? and yes i dont see any video from my hololens either its black like yours
Hi @SilverLive i am still unsure how to change the video codecs, would you be able to explain how to do this? or @spacecheeserocks also suggested this but im not sure how to go about doing it. Thank you :)

Could it be that I am building my application for hololens 2 with ARM64 and recently I read that WebRTC is not compatible with ARM64? Today I will try building it with ARM, hope it will work, if it works I will lwt you know.

@cherrysodalady you have to call StartConnection from the peer which should set up the offer. You can only call it after both client peers were Initialized. StartConnection triggers the whole pairing/signaling process. So here you are right.
Unfortunately also didn´t find any code which allows changing the videoEncoding. I only mentioned VideoFrameCodecs and in this case you are limited to I420 or ARGB32.
@Matteo-0 You are totally right. I tested it and with ARM it works! Thank you, I already read that a view days ago but i didn´t have that in mind ;)

Could it be that I am building my application for HoloLens 2 with ARM64 and recently I read that WebRTC is not compatible with ARM64?

@Matteo-0 Correct, unfortunately there is no ARM64 version of this library currently - you can track any progress at #414. As this repo is currently unmaintained (#773), I expect that an ARM64 version won't be appearing any time soon, unless someone makes a PR for it of course :)

Ok, It worked for me too, I am so happy and happy for you too @SilverLive!
Can anyone please explain me what is the difference between building an application with ARM or ARM64? @SilverLive @spacecheeserocks @cherrysodalady

hi @Matteo-0 and @SilverLive  i changed to ARM and theres definately progress! i now see the hololens's video on the hololens, i still however dont see the webcam feed from my laptop on the hololens (so i can see only one video not two) are you guys able to see the two?

Can anyone please explain me what is the difference between building an application with ARM or ARM64?
In this case, it's non trivial.

You need to rebuild the underlying native library (libwebrtc), which requires a lot of patience (and disk space!). It requires a slightly obscure dev environment, and some customised C++ build tools.
Additionally, the code as-is just fails to build for ARM64. I'm not 100% sure why, but I think some of the dependencies fail the build process because there are no ARM64 versions available. It may also be that there is hardware-specific ARM-specific code blocks, which would need to be rewritten for ARM64. There has already been some discussion about this in other issues, but I don't remember where exactly (probably #414).
The docs do give some instructions for building the native library from scratch, and there are some other instructions to follow for building webrtc-uwp underneath.
TL;DR, if you are comfortable with low level C++, as well as writing assembly code for ARM/ARM64, then you could probably give it a try.

hi @Matteo-0 and @SilverLive i changed to ARM and theres definately progress! i now see the hololens's video on the hololens, i still however dont see the webcam feed from my laptop on the hololens (so i can see only one video not two) are you guys able to see the two?
it may be something wrong with my remote video component (as it differs from the tutorial as the components have changed) is this what you have put or should it be different?

I put in Video Stream Started: VideoRenderer -> StartRendering()
While in Video Stream Stopped: VideoRenderer -> StopRendering()
Try with this, hope it will work!

Hi so does that mean U have the webcam source in the other box too? Instead
of Remotevideoplayer?
On Fri, 23 Jul 2021, 9:41 pm Matteo-0, @.***> wrote:

hi @Matteo-0 https://github.com/Matteo-0 and @SilverLive
https://github.com/SilverLive i changed to ARM and theres definately
progress! i now see the hololens's video on the hololens, i still however
dont see the webcam feed from my laptop on the hololens (so i can see only
one video not two) are you guys able to see the two?
it may be something wrong with my remote video component (as it differs
from the tutorial as the components have changed) is this what you have put
or should it be different?
[image: image]
https://user-images.githubusercontent.com/41712449/126717667-64ba3eac-0587-418f-aaec-1de5a1c9856f.png
I put in Video Stream Started: VideoRenderer -> StartRendering()
While in Video Stream Stopped: VideoRenderer -> StopRendering()
Try with this, hope it will work!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/MixedReality-WebRTC/issues/790#issuecomment-885580833,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ6HWQOAMCMPACA5YGSM4ZLTZFIPLANCNFSM5AOY3QDQ
.

omg its all working! thank you so much everyone for your help, really appreciate it :)


fix: refactor code and add tests to catch invalid output issues

Refactored to use the dynamic blocks instead of the count logic with multiple resources. Made for cleaner output variable logic and added more robust tests that actually grab the resource from azure after it is applied and checks the settings to make sure it was set properly.

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


Update hyper and serde_json

Updated hyper to 0.10 and serde_json to 1.0.

Instead of using hyper, would it be better to use reqwest?

Thank you @sb89.
@zeyla I have removed darksky from my application pending the merge of this PR (OpenSSL conflicts). Could you please merge it?

Thanks for the PR @sb89, and thanks for the re-ping @matthew-piziak!
Sorry I didn't get to this sooner, I saw it when it was opened but got carried away with work. It's unfortunate GitHub doesn't have a better dashboard to handle open PRs and issues.
I think reqwest is a bit more than what darksky needs; hyper + hyper-native-tls should get the job done for what we need.
I also apologize for this repo being a bit behind on, well, everything, and generally being dead. I've got a list down of things I wanted to do sometime, but other things just kept coming up. I'll take some time this evening to bring this library up-to-date and release a new version.

No worries @zeyla, we're all grateful for your efforts. :) Looking forward
to the next release!
On Mon, May 22, 2017, 1:00 PM Zeyla Hellyer<EMAIL_ADDRESS>wrote:

Thanks for the PR @sb89 https://github.com/sb89, and thanks for the
re-ping @matthew-piziak https://github.com/matthew-piziak!
Sorry I didn't get to this sooner, I saw it when it was opened but got
carried away with work. It's unfortunate GitHub doesn't have a better
dashboard to handle open PRs and issues.
I think reqwest is a bit more than what darksky needs; hyper +
hyper-native-tls should get the job done for what we need.
I also apologize for this repo being a bit behind on, well, everything,
and generally being dead. I've got a list down of things I wanted to do
sometime, but other things just kept coming up. I'll take some time this
evening to bring this library up-to-date and release a new version.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zeyla/darksky.rs/pull/1#issuecomment-303159494, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGfGaQjy0PZE0cXme7EikCNomqosMZRoks5r8b8cgaJpZM4NWbNE
.


Server routes redirects to matched page route on errors

Environment
Nuxt project info:

Operating System: Linux
Node Version:     v18.18.0
Nuxt Version:     3.9.3
CLI Version:      3.10.0
Nitro Version:    2.8.1
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      app, modules, i18n, devtools
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-fskkmz?file=server%2Froutes%2Fsitemap%2Fnews.xml.js
Describe the bug
i18n Strategy: "prefix"
Routes;

page route -> /{seller_slug}/[...page].vue
eg:
/en/{seller1} -> no page
/en/{seller2}/products

server route -> /sitemap/news.xml.js -> for /sitemap/news.xml without prefix

If the server route manages to respond successfully, it's okay, and there is no problem. However, when an error occurs on the server route, I am redirected to the page route that matches, instead of just showing the actual error on the server route.
Additional context
No response
Logs
No response

Hi @BobbieGoede, thanks for the triage. I know its minor bug for specific use case but I am planning to use a server route for custom sitemap and since search engines does not like redirections (especially on sitemaps), do you have any advice for workaround while waiting for it to be fixed? I would love to come up with a PR but i am not much of a JS/TS guy myself. Thanks.


TaxonomyPicker Style

Category
[ ] Enhancement
[ ] Bug
[X] Question
Version
Please specify what version of the library you are using: [  3.2.1  ]
Expected / Desired Behavior / Question
Since you are using "office-ui-fabric-react": "7.156.0" I expected to be able to style it the same as controls from that.
Observed Behavior
there is no property for styling (the label and such)
Steps to Reproduce
						<TaxonomyPicker allowMultipleSelections={false} 							termsetNameOrID="Document Type" panelTitle="Select Document Type" label="Document Type" 							context={props.context} placeholder={'Start typing or click the label'} 							onChange={onTaxPickerChange} style={textFieldStyles} //this errors 							isTermSetSelectable={false} />
Submission Guidelines
Is there another way to style the control and the lable?
Thanks!

I have the same question regarding PeoplePicker.
It was a pain to find out, how to change the width of the suggestion callout (it is fixed to 180px in in office-ui-fabric-react).
This post / answer helped me out: https://stackoverflow.com/questions/64510173/how-to-change-the-width-of-normalpeoplepicker-dropdown

@Martin-Leonhartsberger-Habau there is a new styles property that was added to the PeoplePicker control recently (around 2 months ago) that possibly does what you need. Could you please check and open a separate issue if still required (to avoid switching the context of this item)?

@Ofer-Gal is this still required? Have you tried out the modern-looking taxonomy picker in case it suits your requirements?
If still required, would something like the styles property of the PeoplePicker work for you?

I got this working long time ago.  Sorry I did not close then.


Docker improvement advices

This issue for all advices about improving docker usage
While working on issue #212 , i plan to add py-checker tool in cita image.
Meanwhile, I want to collect more greatful suggestions about our cita-run image.
Those tools you think should be in our image, but not include now, like vim supporting. please let me know.
Or which tools in our image you think is junk and should be deleted, please me know too.
We will consider your opinion and puting it together, then add it to our new image.
Versions
Latest v0.21+(develop)
@cryptape/citaman @keroro520 @driftluo @yangby-cryptape @u2

sysstat, vi

According to the current situation，The only job of cita_run image is to run the cita.
I don't think it's a good idea to put other tools in the image.
Does the tool(like vim) is must-have to run cita?
#202 is another question, stil, i think the image should be small and simple

I agree that keeps docker-image tiny. "sysstat, vi" is not must-have.

No tools for development or devops
For example, if we provide vi / vim, maybe some guys will complain:

vi? Why do not provide nano!
There is a vim? D**n it, I am a emacser!

(Further) Docker should NOT be our first choice to publish the binaries
I read a lot manuals of server application. Most of them provide a detailed document as  the first choice to show how to run the server step by step. Docker just is optional.
Ubuntu, and maybe only Bionic Beaver? Docker? K8s? Nobody like software family buckets!
Suppose I was a normal user, I really want to say: just give me a bootstrap for setup a minimum available environment!

We use RUST, that so cool, But our usage is so unfriendly.
I am not clear enough to know about why we use docker in the last long time.
Compiling problem in mac os is one of the biggest problems i know about this. What if we solve this problem and add step-by-step docs in different os environments. Can we dicard docker usage?
In my mind, the perfect situation must be cd cita, then,  cargo build that all.

Compiling problem in mac os is one of the biggest problems i know about this.

As a server software, maybe we should consider that let cita supports the top 3 most popular linux server distribution at first.


Add shebang to launch_gripper.py

Description
launch_gripper.py is not guaranteed to pick up the right version of Python without this line.
Type of change
Please check the options that are relevant.

[x] Bug fix (non-breaking change that fixes an issue)
[ ] Proposes a change (non-breaking change that isn't necessarily a bug)
[ ] Refactor
[ ] New feature (non-breaking change that adds a new functionality)
[ ] Breaking change (fix or feature that would break some existing functionality downstream)
[ ] This is a unit test
[ ] Documentation only change
[ ] Datasets Release
[ ] Models Release

Type of requested review

[x] I want a thorough review of the implementation.
[ ] I want a high level review.
[ ] I want a deep design review.

Before and After
@girifb has encountered an issue where it attempts to launch with the default system Python 2; this fixes that issue.
Testing
Please describe the tests that you ran to verify your changes and how we can reproduce.
Checklist:

[ ] I have performed manual end-to-end testing of the feature in my environment.
[ ] I have added Docstrings and comments to the code.
[ ] I have made changes to existing documentation where needed.
[ ] I have added tests that show that the PR is functional.
[x] New and existing unit tests pass locally with my changes.
[x] I have added relevant collaborators to review the PR before merge.
[ ] [Polymetis only] I have verified that all scripts in tests/scripts runs on hardware.

Codecov Report

Merging #888 (03142b3) into main (14d0192) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #888      +/-   ##
==========================================
- Coverage   32.75%   32.67%   -0.08%
==========================================
  Files         341      340       -1
  Lines       33363    33123     -240
==========================================
- Hits        10929    10824     -105
+ Misses      22434    22299     -135

Impacted Files
Coverage Δ

agents/droidlet_agent.py

interpreter/craftassist/tasks.py
75.00% <0.00%> (+0.36%)
:arrow_up:


Fix stop words

Thanks for the fix! I think we could still have some cases of left stop tokens since we get the start length by batch but this should make it better for datasets with very different prompts sizes.


[Chore] As a user, I can see the coin image fit circular and crop in center

Why
The coin images return from API in different styles (circular, square, transparent background, etc.), the coin images are displaying inconsistency on my coins and trending coins.

Acceptance Criteria

Display the coin image fit circular and crop in center
Default background color if the background is transparent: #f3f4fa

Design
https://www.figma.com/file/HioYp71LllKVy9qaiLauKn/Crypto-App-(Community)?node-id=3%3A181
Resources
N/A

@hoangnguyen92dn Sorry but I still don't see the issue, could you please clarify more with example 🤔 What I'm seeing is all the icons are in the same vertical lines 🤔


KAFKA-6576: Configurable Quota Management (KIP-257)

Enable quota calculation to be customized using a configurable callback. See KIP-257 for details.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

retest this please

@junrao Thanks for the review. I have updated the code and also made some changes to CustomQuotaCallbackTest to make it complete faster (the tests check only the metrics on the broker-side to avoid waiting for throttled requests to complete). Client-side metrics are verified by the other quota tests anyway.
I also added a new method to the callback interface to check if quotas need resetting. Since we now don't get quota limits from the callback handler on each request, we need to check if limits may have changed (e.g. with a callback that uses an external database to store quotas). Thoughts?

@junrao Thanks for the reviews, merging to trunk.


Don't set appearListener if null is sent in.

Handle the case where the webapp sends in null

lgtm


Update version of Istio system chart

Istio currently has two system chart versions:

0.0.1
0.0.2

We should remove version 0.0.1 and rename 0.0.2, so we are left with one chart: version 0.1.0

rancher/rancher:v2.3.0-rc11

System charts defaults to 2.3 release branch
When enabling Istio, only one chart is shown: 0.1.0 - Istio 1.3.1
Istio components deploy successfully
Istio P1 cases pass


How many satellites are simulated?

Hi, can someone please explain, how many satellites are simulated by this SW? Is it simulating just those which could be seen from fictive location or is it just some static number of satellites?
Thank you!

MAX_CHAN defines the number of channels. The default value is 16.

Sooo... if i don´t change this thing, it will always generate singal from 16 satellites independently on selected location?

That's true you will have 16 channels but the number of satellites will depend on the location because the simulator picks them based on their elevation angles

Great guys! Thank you so much


Visual Studio Offline Setup failed to download some components.

While doing the setup with the --layout parameter, it was mentioned that the components failed to download.
Will those components get downloaded by the end of process?
I am currently trying to download Visual Studio Enterprise 2017. (not the RC release)
And I really mean for those red-marked components:

We appreciate your contacting us! So that we can investigate further, we invite you to post this issue to our Developer Community website at https://developercommunity.visualstudio.com/.
Note: You might be asked to provide a setup log file; here's how:

Download collect.exe from http://aka.ms/vscollect, and run it
Attach %temp%\vslogs.zip to the problem report on https://developercommunity.visualstudio.com/
Thanks,
Terry

Can you please help me how to report the issue there?
It only redirects me to "Report an issue" tool inside Visual Studio.
I haven't installed it yet!

Thanks for the follow-up. Here's an answer to your initial inquiry; perhaps it will help you complete your installation so that you can then post the issue to the Developer Community.
When you create a layout and a download fails, Setup will try to download the file(s) several times. However, if Setup still can’t successfully acquire the file(s) after several retries, Setup displays an error message, as shown in your screenshot, where the missing files are indicated in red text. You can try re-running the same --layout command later to try to get the missing files. When you re-run the command, it will not re-download the files that already exist locally in the layout folder. So, if only a few files failed to download the first time, it shouldn’t take as long the second time.
Thanks,
~ Terry


Code errors on Xarray pages

The latest builds have code errors in several places in our Xarray tutorials, e.g.
https://foundations.projectpythia.org/core/xarray/computation-masking.html#groupby-split-apply-combine
and
https://foundations.projectpythia.org/core/xarray/computation-masking.html#compute-anomaly
and
https://foundations.projectpythia.org/core/xarray/enso-xarray.html#visualize-the-computed-nino-3-4-index
Several of these throw an error
TypeError: float() argument must be a string or a number, not 'cftime._cftime.DatetimeNoLeap'

I don't know when this started. These types of Python code errors don't get flagged by our CI tests. Is it due to version changes in Xarray or one of its dependencies?

I'm taking the liberty of labeling this as "high priority" because we should not have ugly errors in our featured tutorials!

The nightly build for the CMIP6 Cookbook is also failing with the same error.

I did some Xarray version testing. It looks like this behavior begins with the most recent (2022.11.0) xarray release. The immediate predecessor (2022.10.0) works without error.
I can put together a PR with an environment.yml that specifies xarray<2022.11.0 as a quick fix, although it would be worth digging deeper to see if this is a bug in Xarray or if the lines that are throwing the error need to be rewritten.

suspect it stems from https://github.com/pydata/xarray/pull/7052

And it looks like the error goes away if we add
import nc_time_axis

Looks like this problem will go away with the next release of Xarray, as it's been fixed with https://github.com/pydata/xarray/pull/7276


Section stylesheet 'scss' failing to compile

Hi, it seems that the section {% stylesheet 'scss' %} is failing to compile nested elements:
{% stylesheet 'scss' %}
  .footer {
    color: red;
    &__main {
      border: 1px solid red;
    }
  }
{% endstylesheet %}

I couldn't get any error log. Perhaps it's the same issue as stated in #370, if so, is there a way to compile these locally?

@eduardobbrito Liquid is not compiled by Slate and done on Shopify servers. The problem you are having is not with Slate.
Why do these styles need to live in Liquid? If you can include them in a .scss file, Slate can compile them locally.

Hi @t-kelly yes, thanks for your reply, it's what I'm currently doing. I intended to use the stylesheet tag so that critical portions of the page could be styled faster.
I guess there's still this limitation on what Shopify's Sass compiler can do, so I'll just use regular css on these few sections!

@eduardobbrito Makes sense! Would love to improve this workflow for themes. Related https://github.com/Shopify/slate/issues/267
@NathanPJF I always seem to forget the limitations of Shopify's SCSS compilation. Does it include nesting?

@eduardobbrito Even on Shopify's servers, the SCSS you wrote will not compile.  Shopify uses a forked version of Sass v3.2 that removes the @import directives (security reasons).
The BEM style SCSS you're writing is something that was introduced in Sass v3.3, and unfortunately, Shopify's servers aren't going to be able to flatten that out.
In the end both #267 (loading critical CSS) and Shopify/theme-scripts/issues/61 (dynamically loading section resources) should help solve a lot of the issues.

Hello @jonathankwok, Is there any chance of getting a newer version of Sass soon? Should I just forget entirely about that beautiful feature?

Think you mean @jonathanmoore 😃

Need to upgrade SASS to 3.3 or higher to support BEM style. http://alwaystwisted.com/articles/2014-02-27-even-easier-bem-ing-with-sass-33
I appreciate for this upgrade!


[HOLD for payment 2023-11-01] Improve the IOUTest with Edit, delete and payment with ACH

Problem
Stemming from https://github.com/Expensify/App/issues/23961
We are actively working on new features for the money request flows, where we have now added ability to delete the request, edit it or we have an option to pay the request with ACH payment type which puts the report into slightly different state than when paying the report with different payment type. The state of the report is 3 BILLING whereas report paid with different type has state 2. Both reports have status 4.
But these new features have not been captured in IOUTest. Lets add it.
Solution
Lets beef up our automated tests, think of many various solutions and really try to capture as much of the optimistic flow there as possible. In the end what we can se can be translated to the Onyx state so we can test if the onyx state is correctly updated during the flow and we are showing a report is settled when its settled for example.

cc @koko57 or @allroundexperts might be interested.

All yours

@allroundexperts how is the PR coming along?

It looks like we're waiting to resolve a few items on the PR then we should be good to merge it.

@allroundexperts I have shared with you some tips for the next steps in the PR, we should be close to completing this

Merged but I think we would actually want to add more tests for all the expense fields

@tjferriss this should be $500 to @allroundexperts nothing else, thank you!

@mountiny @allroundexperts should that payment be through Upworks?

@allroundexperts requested on newdot so we can close now

$500 payment approved for @allroundexperts based on this comment.


Introducing ReactPlay section to improve.

Visit https://react-play-docs.netlify.app/intro and an introduction to react play page will be open and need to improve that page by adding more details about react play.

Please refer these links to know more about react play

https://github.com/reactplay/react-play#readme
https://blog.reactplay.io/introducing-reactplay-learn-create-share-reactjs-projects

Hi @Sachin-chaurasiya I would like to take this up.

Hello @supminn, I have assigned a few issues to you, so once those finish, you can take it p.

Hello @Sachin-chaurasiya , I know I have skipped a step and already worked on this issue (by creating a duplicate issue and the subsequent PR), could you assign me to this issue?

Hello @Sachin-chaurasiya , I know I have skipped a step and already worked on this issue (by creating a duplicate issue and the subsequent PR), could you assign me to this issue?

Hey @joypoddar , I have assigned the issue to you.

@Sachin-chaurasiya I have added a subheading in the intro doc.

What should the next steps be? Should I reopen the issue ?


Move GitHub logic in the cache package into a custom HTTP transport

Related to #120 - this and the API discovery of private release artefacts could be rolled into a custom HTTP transport. For precedent, go-github does this transparent auth internally.

This is the round-tripper you'd use for GH App Installation tokens: https://github.com/bradleyfalzon/ghinstallation

Yeah this is already done.

Sorry, to be more specific - the logic described in this issue is already implemented - the stuff in ghinstallation, we should look at for the private auto-version+GH release downloads.


Nil the finalizers on Close

This stops the extra (harmless) closes being called after already being closed once.
Signed-off-by: Darren Stahl<EMAIL_ADDRESS>/cc @jhowardmsft @jstarks

Seems reasonable, but ping @jstarks to confirm.

LGTM


function passed to onChange prop of ColorPicker doesn't get executed

Description
Hi, thank you for the nice library.
I am trying to use the Panel5 component to allow users to pick colors, but the function passed to the onChange prop of the ColorPicker wrapper doesn't execute.
I am using react-native-reanimated version 2.8.0, and have found out that when I upgrade react-native-reanimated to a version higher than 3.0.0,  the passed function gets executed as expected.
However, due to other libraries depending on react-native-reanimated, it is difficult for me to upgrade the version react-native-reanimated.
additional information
In the ColorPicker component in your library, the below changes do make the component work as I have expected. But I have no deep understanding of the code. Just adding this in case it might help :)
      // onGestureChange function inside ColorPicker.tsx
    const onGestureChange = (color?: SupportedColorFormats) => {
      'worklet';

      if (!onChange) return;
      const colorObject = returnedResults(color);

      // executing runOnJS makes it work, but I have no understanding of why..
      runOnJS(onChange)(colorObject)

      // below is the original code,
      // try {
      //   // run on the UI thread
      //   onChange(colorObject);
      // } catch (error) {
      //   // run on the JS thread
      //   runOnJS(onChange)(colorObject);
      // }
    };

Steps to reproduce

install react-native-reanimated version 2.8.0, react-native-gesture-handler version 2.10.1 with reanimated-color-picker version 3.0.3
pass a function to the onChange prop of the ColorPicker wrapper component
add Panel5 as a child to the Colorpicker wrapper.
press any color in the Panel5 component -> doesn't trigger function passed to the onChange prop, making it impossible to get the pressed color value.

Snack or a link to a repository
No response
Reanimated Color Picker version
3.0.3
React Native version
0.66.4
React Native Reanimated version
2.8.0
React Native Gesture Handler version
2.10.1
Platforms
iOS
JavaScript runtime
JSC
Workflow
React Native (without Expo)
Architecture
Paper (Old Architecture)
Build type
Debug mode

Hey, thanks for reporting this. The code in the library is designed to make the onChange property accept both worklet and non-worklet functions. Since Reanimated doesn't provide a clear way to detect a worklet, I used a try/catch block as a workaround. Please try passing a worklet function instead, and inside it, you can use runOnJS if needed. If that doesn't work, you might consider using patch-package because I can't think of a solution that works for every version of Reanimated.

Hey,
I have tried to pass a  worklet function to the onChange prop and it works! Thank you for your help.
for people who might see this, the function i have implemented is as follows:
// function passed to the `onChange` prop of `ColorPicker` wrapper component
 const onSelectColor = ({ hex }) => {
    'worklet'; // making this function a worklet function did the job for me.
    console.log('pressed color', hex);
  };


KAFKA-2633: Default logging from tools to Stderr

LGTM. Not waiting for Jenkins since we don't have any relevant tests.

Just going to throw my 2 compatibility czar 2¢ in here -- this is clearly the right thing to do long term, but potentially breaks any scripts that have been built on the existing tools with loggers sending output to stdout. tools-log4j.properties is pretty far-reaching. Are you comfortable that every log message from any of our tools that doesn't override the log4j settings is non-critical?
One simple example that I imagine could cause breakage for a lot of people: from a quick scan, the partition reassignment tool prints lots of useful info to stdout.

At a quick glance, the partition reassignment tool is using println() to print out most of its information. This change will not effect that. This change only effects calls to the logger. It also does not prevent the information from being printed, but allows a user to redirect it in case they do not want it to effect the output (with 2>). I don't suspect a user is writing their scripts to depend on logged output, however if they are they should still be able to. I feel most would be relieved to have a good default behavior.
Since we are just before a major release, I think now is a good time to make a change like this. Especially when the public cli has been broken already. Here are some some notable potentially backwards incompatible changes (not an exhaustive list, sorted by potential impact):

KAFKA-2205 - No longer able to alter topic config from the topics command
KAFKA-2514 - default JVM options changed for everything that uses kafka-run-class.sh
KAFKA-2198 - topics commands now exits with non-zero exit code on failure
KAFKA-2337 - added error and warning output in topic command to avoid topic colision

From my perspective, the current situation (errors written to stdout) is a very annoying bug which should be fixed. I don't think we want to maintain buggy behavior as part of the compatibility guarantees.
I think separating output from errors is in the best interests of everyone who runs tools with automation. For example, if your script parses output of ConsumerOffset tool, being able to route errors somewhere else will be very beneficial.
Same for being able to bump log level of tools while maintaining readable output.

I am happy to send a docs patch listing a potentially breaking change. It can be a new jira or a follow up patch. We should likely enumerate the breaking changes I listed in my PR comment somewhere as well.
Obligatory XKCD comic: https://xkcd.com/1172/

@gwenshap I agree, we obviously have to be able to make progress on improving the CLI tools. This was just a particularly broad change since it affects every tool that does any logging via log4j, although as @granthenke points out that may not be that big a change since a lot of the tools just write directly to stdout.
@granthenke Pointing out those other potential compatibility issues is very valuable, but 5 wrongs don't make a right :) Just because we may have missed other compatibility issues isn't an argument that adding more is ok.
I'm fine with this change as long as we're convinced it doesn't affect anything that's likely to be critical. I just like to raise the compatibility issue whenever I notice it so we continue to pay attention to it. CLI is an especially difficult area because it's harder to warn users that may be using the CLI in automated ways. Unlike the Java code where you can add @Deprecated and they should see the warnings, the CLI changes can't be "staged" as easily.
I think it'd be great to add a changelog for compatibility breaking changes. Kafka's current "release notes" (i.e. https://archive.apache.org/dist/kafka/<IP_ADDRESS>/RELEASE_NOTES.html) is not all that helpful to users.

@ewencp completely agree about 5 wrongs making a right. What I was suggesting was that if we have made script breaking changes for 0.9.0, (I assumed they were know to be potentially breaking) then lets do this one now too.
I will open a jira send a PR adding these breaking changes to the release notes.

@ewencp Since KAFKA-2205 has a relative big breaking change...do we need to open a jira to fix that? I can send a PR fixing that today too.
We can add the ability to change topic configs back, and log that its deprecated.

AFAIK, we don't write release notes. They are generated automatically from JIRA before a release. @junrao can correct me on this.
I suggest, since <IP_ADDRESS> will be a big upgrade, to open an umbrella JIRA for missing docs. IMO we need a new doc section called "Upgrade Guide" with both instructions on how to upgrade and long list of behaviors that changed (ISR max lag as a popular example). Responsible admins will read the guide before upgrading and modify scripts accordingly.
New features should be added to existing docs, although Security probably deserves its own section too.

@gwenshap there is an upgrade section here. Perhaps we just need to fill that out more.


Initial Scope 3 support aligned with OECM 2.0

Signed-off-by: MichaelTiemann<EMAIL_ADDRESS>

@kmarinushkin you will likely want to merge the develop upstream changes into your fork once this PR is approved.  The Python 3.10 test failed due to a network error (an upstream repo machine is down), so I'll re-run that job in a few minutes.

As explained, the unit tests will need major work to align with the implications of Scope 3 support.  But sectors such as Autos and Oil and Gas simply cannot be analyzed without Scope 3 data (which becomes attributed to the S1+S2 emissions).
There are also many other bugs fixed in the code.  Happy to reply to line-by-line questions in the review process.  It's a lot to take in, but there's a lot more that's credibly working now.

The first commit is "merge develop to develop". Looks like a mistake, should't we rebase 2 commits on top of develop, instead of merge?
This PR might accidentally merge " develop" branch in the wrong direction, which may create problems in future

I raise my hand with respect to this error.  What I should have done was push to my fork, review the PR there, then merge that to OS-Climate.  I'm still rudderless when it comes to proper git management (things have changed A LOT since the late 1990s).  But the develop-> develop is from my fork to OS climate, so I don't think it will go the wrong way.

It just occurred to me that we can address the question of "one scope" vs many scopes by recognizing that in the producer-centric case, S3 is defined to be zero, which means that S1S2 and S1S2S3 values will be identical.  But we can still exercise multiple scopes by asking the various tools to analyze both.  The analysis should be identical.  In this way we can prepare the tool for separate S3 handling without needing to wait for S3-separating benchmark data.
Who wants to tackle?

As of now this Pull Request would unblock further work on S3 scope handling.  But it also doesn't pass the unit tests (because many unit tests are affected by changes to scope handling rules, such as pushing S3 emissions into S1S2 as per producer-centric rules).
@kmarinushkin can you clone from MichaelTiemannOSC:develop and work from there until we sort out the test cases?
@LeylaJavadova if you have some interns looking to ramp up, getting their help writing/rewriting unit tests is a great way to start...

Sure, I will branch from your fork

Going for the big merge.  I triaged the test cases, fixing discrepancies due to benchmark drift between version 1 and version 2 of OECM, disabling some tests that appeared completely wrong to me, etc.  Help with further test case cleanup would be appreciated!


self_test.h: fix the C++ wrapping

Fixes #17557

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

This pull request is ready to merge

Merged to both.  Thanks for the review.


Dynamic subsystem on bookmark

Is your feature request related to a problem? Please describe.
To day sub system is only set on status and the subsystem is set to the same as name. this can be used to give a better name for apps sins the bookmark component when showing apps uses a clean up app key to display the app name.
Describe the solution you'd like
When creating a bookmark the subsystem  field should be set with the application name.
Describe alternatives you've considered
An alternaternative is to leave as is.

@Noggling should we put this into the Fusion issue tracker - is this still a thing?


Duplicate phet ZIM files

These two phet ZIM files are same size , same content and ISO code which is return by zim file eng.
https://mirrors.dotsrc.org/kiwix/zim/phet/phet_en-ca_2023-01.zim
https://mirrors.dotsrc.org/kiwix/zim/phet/phet_en-gb_2023-01.zim
Only difference is the file name in my opinion we should not rely on file name rather than metadata and seems like both file are returning same metadata.

@pavel-karatsiuba To me it sounds logic that they are really similar, because they use almost the same language. But we have something fishy around the Language metadata.

I have made PR with changes. But I think we need to make it in a different way.
This is my proposition: #207


Adding topic for add-in requirements

@jasonjoh,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/add-ins/add-in-requirements.md
:white_check_mark:Succeeded

docs/add-ins/toc.yml
:white_check_mark:Succeeded

docs/breadcrumb/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


EEP: Logging and SLF4J

This is a "retrospective" EEP about logging in the Embulk eco-system.
Read the rendered version at:
https://github.com/embulk/embulk/blob/eep-logging-slf4j/docs/eeps/draft-logging-slf4j.md

Thanks! Merging this as EEP-13 as Final.


WW-4900 Makes BackgroundProcess transient

Currently we won't support exec and wait from de-serialized session and maybe add this support some day on user demand. Why I think to drop such support? It's not a good practice to try serializing such large or variant unpredictable objects like action and invocation (CWE-579: J2EE Bad Practices: Non-serializable Object Stored in Session).

Changes Unknown when pulling dda3facc265d4e1c3b1c60fa2bcc4240953ee35c on yasserzamani:WW-4900 into ** on apache:master**.

Changes Unknown when pulling 6b131550a649106766f45c83279b036544afc58c on yasserzamani:WW-4900 into ** on apache:master**.

testSerializeDeserialize of BackgroundProcessTest is failing ...

Was a good catch by junit 👍 I should have wait for background thread. fixed. thanks!

Changes Unknown when pulling a8ecd9bd297670b048dd3d12ee7211d4984db664 on yasserzamani:WW-4900 into ** on apache:master**.

Let's rock 👍

I think so , such huge object in session is too bad
It seems that,TokenSession needs some changes too.


Add vue header

Fixes

Fixes #193  by @dhruvkb
Description

This PR adds the vue version of Header component.
Technical details

Tests

Screenshots

Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the master branch of the repository.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@dhruvkb I have made the required changes. Please check.

Hi @jahnvigupta thank you for this PR but we are going to be making some significant changes to vue-vocabulary and will be making the header component work differently. Thank you for your PR, we will let you know when we're ready with the new version if you'd like to work on it.


scheduler agent runs 1 hour later

Since the daylight saving change, my scheduler agent runs 1 hour later. So 1pm runs 2pm

Running in digitalocean Ubuntu 14. Ubuntu shows EST correct time, and .env also has Eastern time.

Does restarting your Huginn instance help? I could imagine that the scheduler did not pick up the DST time change.

It still does that. I have restarted.

I did some tests. My scheduler is set up like this:
{
  "action": "run",
  "schedule": "0 7 * * *"
}

In .env I have TIMEZONE="Eastern Time (US & Canada)" and the scheduler seems to have run at the correct time: 2016-03-18 07:00:08 -0400
Not sure if it is related but c48ff63e7a96260540f9399201cbd7a23d65c9fa might have fixed to issue you are having.

I just changed the time on my scheduler. It must be something with my system.


Feature : Gitops configuration optional in Existing CICD Integration.

Summary
Gitops is mandatory for devtron apps, make this an optional. without gitops configuration user can create and deploy application.

CD pipeline type - (deployment via helm, deployment via acd)
On CD pipeline create api,  for ACD type pipeline, skip git repo creation and acd repo registry.
On CD pipeline trigger, check pipeline type, for helm type - create helm with some identifier label and merged values.
Checklist api modification, also require UI side changes
Checks required for PRE/POST CD to skip in case of helm type pipeline.
DB operation for devtron chart ref instead in GITOPS.

Motivation
Make devtron easy for users who are facing difficulties in configuring GITOPS.
Proposal
Add this is as feature in same existing CICD integration, not required new integration for this.
Testing Guideline [Optional]
How this feature will need to be tested. What all areas should be covered

Hey team! Please add your planning poker estimate with ZenHub @kartik-579 @manish-agrawal-ai @nishant-d @pghildiyal


Define Authentication

You're required to work on auth.js and sendOtp.js files which are present in the middlewares folder

Hey @Abhinav-Bansal751, You can start working on this!
Make sure you read the user controller PR


Multiple tagsymbols

Is it possible to define multiple tagsymbols? I would like to use both + and @ as tagsymbols, as I am using a task tracking system in todo.txt that uses both projects (+) and contexts (@), and would like to be able to refer to both in jrnl using the same symbology, so that I can take quick notes in either tool without having to consider which software I am working in and write accordingly.
If so, how would I modify the relevant line in ~/.jrnl_config? I've tried to edit this to define two tagsymbols, but without success.

Sure is, but you should be using Jrnl 2.0... Not sure this works in v1. I have the following in my ~/.config/jrnl/jrnl.yaml file:
tagsymbols: '@#^%'
# @ is for People
# # is for Tags
# ^ is for Places
# % is for Moods/Emotions

In Jrnl 1.9.8 homebrew, this format works.
"tagsymbols": "@,#,^,%",


Fix fitter selection logic in process tomography

Summary
This commit fixes an issue in the process tomography module. When cvxpy
was not installed the module would fail to import. That's because the
selection logic was reworked in #422 but never updated in process
tomography to reflect that change. Since no CI job runs without cvxpy
installed we never caught this edge case. This commit fixes the
underlying issue to rework the process tomography fitter selection logic
to mirror the changes to state tomography in #422 and then also try and
add ci coverage it removes cvxpy from the docs tox job. With warnings
set to fatal this should ensure we are always able to import everything
and build the docs if cvxpy is not installed.
Details and comments
Fixes #429

It looks like travis is just stuck here, it successfully ran here: https://travis-ci.com/github/Qiskit/qiskit-ignis/builds/172273948


Guest nickname autoban

I realised how the bot_example.py was set up to automatically issue a ban for anyone who set's their nickname as guest or keeps it as this. Maybe there should be a switch to turn this feature on or off?
Line: https://github.com/nortxort/pinylib/blob/master/bot_example.py#L138

I guess i forgot to add a option/command for this. It will be added in the next update

Sure, nice work Nort.


Show vpn & registry services in service list

at least make it visible somewhere - services could be argued to show only user defined services.
And, why not show etcd etc.
getting to see if vpn is available, now done via:

docker ps - bit low level, and hey, there are even more containers that I have as services, and needs to be done per node
kontena vpn create - error if its already existing
kontena vpn config - way more info as needed
kontena vpn delete - oops


Adding apollo client issue

I tried to add the apollo client by using this way,  but It gave me an error, (Invariant violation: Could not find "client" in the context or passed in as an option.) Please,  I really need some help

I spent a lot of time,  google it, still can't find a solution.

Please follow their official guidelines: https://www.apollographql.com/docs/react for implementation details and any issue you can check under their Github repository. This project is using gql-query-builder.


Refactor test utility functions that loop through all cells/edges

We have some duplicated functionality in src/apps/applib/lib/utility.c in the helper functions to loop through all cells or edges at a given resolution:

mapSumAllCells_double
iterateAllIndexesAtRes
iterateAllUnidirectionalEdgesAtRes

See, for example, the discussion in https://github.com/uber/h3/pull/377#discussion_r494657719

Some more thoughts on using an iterator pattern: https://gist.github.com/ajfriend/1afc9d063405e3d2482044ea26caf58a

Hint: There is a parallel between the iterator that "zips down" to the list of children, and the one that "zips up" to the encompassing parent (compact).

Done with #413


Bug: Codecov is unable to find unit tests coverage report file

coverage.cobertura.xml file is not generated by unit tests:
Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
Data collector 'XPlat code coverage' message: [coverlet]System.TypeLoadException: Could not load type 'Microsoft.Extensions.DependencyInjection.ServiceCollection' from assembly 'Microsoft.Extensions.DependencyInjection, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'.
   at Coverlet.Collector.DataCollection.CoverletCoverageCollector.GetDefaultServiceCollection(TestPlatformEqtTrace eqtTrace, TestPlatformLogger logger, String testModule)
   at Coverlet.Collector.DataCollection.CoverletCoverageCollector.OnSessionStart(Object sender, SessionStartEventArgs sessionStartEventArgs) in /_/src/coverlet.collector/DataCollection/CoverletCoverageCollector.cs:line 132.
Results File: /home/runner/work/IoT-Hub-Portal/IoT-Hub-Portal/src/AzureIoTHub.Portal.Tests.Unit/TestResults/_fv-az186-168_2022-10-[12](https://github.com/CGI-FR/IoT-Hub-Portal/actions/runs/3235549877/jobs/5300135754#step:9:13)_[14](https://github.com/CGI-FR/IoT-Hub-Portal/actions/runs/3235549877/jobs/5300135754#step:9:15)_38_17.trx

_____          _
    / ____|        | |
   | |     ___   __| | ___  ___ _____   __
   | |    / _ \ / _` |/ _ \/ __/ _ \ \ / /
   | |___| (_) | (_| |  __/ (_| (_) \ V /
    \_____\___/ \__,_|\___|\___\___/ \_/

  Codecov report uploader 0.3.2
[2022-10-13T06:21:38.429Z] ['info'] => Project root located at: /home/runner/work/IoT-Hub-Portal/IoT-Hub-Portal
[2022-10-13T06:21:38.432Z] ['info'] ->  Token found by environment variables
[2022-10-13T06:21:38.434Z] ['verbose'] Start of network processing...
[2022-10-13T06:21:38.434Z] ['verbose'] Searching for files in /home/runner/work/IoT-Hub-Portal/IoT-Hub-Portal
[2022-10-13T06:21:38.443Z] ['verbose'] coveragepy is not installed
[2022-10-13T06:21:38.443Z] ['info'] Searching for coverage files...
[2022-10-13T06:21:38.[50](https://github.com/CGI-FR/IoT-Hub-Portal/actions/runs/3240097003/jobs/5310301335#step:11:51)0Z] ['info'] Warning: Some files located via search were excluded from upload.
[2022-10-13T06:21:38.500Z] ['info'] If Codecov did not locate your files, please review https://docs.codecov.com/docs/supported-report-formats
[2022-10-13T06:21:38.500Z] ['error'] There was an error running the uploader: No coverage files located, please try use `-f`, or change the project root with `-R`
[2022-10-13T06:21:38.501Z] ['verbose'] The error stack is: Error: No coverage files located, please try use `-f`, or change the project root with `-R`
    at main (/snapshot/repo/dist/src/index.js)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
[2022-10-13T06:21:38.501Z] ['verbose'] End of uploader: 81 milliseconds

Related to https://github.com/coverlet-coverage/coverlet/issues/1390

I'm on version 6.0.13 of Microsoft.EntityFrameworkCore and is being deployed by a 3rd party vendor.
Is there a resolution to this other than Downgrading, this is not an option for me.


Add JWT support

Changes proposal:
These changes allow JWT svids and JWK bundles getting by specifying an audience, JWT file name and JWK file name.
fixes #43

config.go is still requiring svid_file_name and related values be set, which aren't necessary when just using the JWT functionality.

config.go is still requiring svid_file_name and related values be set, which aren't necessary when just using the JWT functionality.

Good catch @keeganwitt

config.go is still requiring svid_file_name and related values be set, which aren't necessary when just using the JWT functionality.

Good catch @keeganwitt

Should we validate that x509, JWT or JWT bundles are going to be fetched? Or simply delete this validation?

config.go is still requiring svid_file_name and related values be set, which aren't necessary when just using the JWT functionality.

Good catch @keeganwitt

Should we validate that x509, JWT or JWT bundles are going to be fetched? Or simply delete this validation?

I was thinking
if c.SvidFileName == "" && c.JWTFilename == "" {
    return errors.New("svid_file_name and/or jwt_file_name is required")
}
if c.SvidFileName != "" && c.SvidKeyFileName == "" {
    return errors.New("svid_key_file_name is required when using svid_file_name")
}
if c.SvidFileName != "" && c.SvidBundleFileName == "" {
    return errors.New("svid_bundle_file_name is required when using svid_file_name")
}
if c.JWTFilename != "" && c.JWKFilename == "" {
    return errors.New("jwk_file_name is required when using jwt_file_name")
}

config.go is still requiring svid_file_name and related values be set, which aren't necessary when just using the JWT functionality.

Good catch @keeganwitt

Should we validate that x509, JWT or JWT bundles are going to be fetched? Or simply delete this validation?

I was thinking
if c.SvidFileName == "" && c.JWTFilename == "" {
    return errors.New("svid_file_name and/or jwt_file_name is required")
}
if c.SvidFileName != "" && c.SvidKeyFileName == "" {
    return errors.New("svid_key_file_name is required when using svid_file_name")
}
if c.SvidFileName != "" && c.SvidBundleFileName == "" {
    return errors.New("svid_bundle_file_name is required when using svid_file_name")
}
if c.JWTFilename != "" && c.JWKFilename == "" {
    return errors.New("jwk_file_name is required when using jwt_file_name")
}

Well I think there are three options, fetch X509 SVIDs, fetch JWT SVID or fetch JWT Bundle. So first and fourth conditions should be:
if c.SvidFileName == "" && c.JWTFIlename == "" && c.JWKFilename == "" {
// error message
}
if c.JWTFileName != "" && c.JWTAudience == "" {
// error message
}

Besides of that, in util_posix.go and util_windows.go, if some of this conditions aren't met no routines will be running. So maybe is better to delete these validations.

TestParseConfig() in config_test.go is missing new options.

LGTM. @maxlambrecht what do you think?

Looking good, I left a few more nit comments. I'm ready to approve.

LGTM. @maxlambrecht what do you think?

Looking good, I left a few more nit comments. I'm ready to approve.

@JU4N98 There are 5 open comments left. Please let us know when you have updated.

@JU4N98 There are 5 open comments left. Please let us know when you have updated.

I think is ready for a last review @faisal-memon , @maxlambrecht and @MarcosDY


ContentScript at MAIN world: browser-polyfill.js: "Error: This script should only be loaded in a browser extension."

Describe the bug
I´m trying a simple content script in the main context, however it fails by importing the polyfill which should only be included in the extension contexts.
Maybe it´s just me doing something wrong, I´m quite new to extensions. But it seems that world: 'MAIN' should not bundle the Polyfill?
Reproduction
content.ts:
import {createIntegratedUi} from 'wxt/client'

export default defineContentScript({
  matches: ['<all_urls>'],
  runAt: 'document_start',
  world: 'MAIN',
  matchAboutBlank: true,
  main(ctx) {
    console.log('*** main', ctx)
    const ui = createIntegratedUi(ctx, {
      position: 'inline',
      onMount: (container) => {
        // Append children to the container
        console.log('** ** createIntegratedUi-main')
        const app = document.createElement('p');
        app.textContent = '...';
        container.append(app);
      },
    });

    // Call mount to add the UI to the DOM
    ui.mount();
  },
});

Error (browser console):
browser-polyfill.js:26 Uncaught Error: This script should only be loaded in a browser extension.
    at browser-polyfill.js:26:11
    at browser-polyfill.js:5:5
    at browser-polyfill.js:13:1
    at browser-polyfill.js:1268:2
    at content.ts:24:2

Steps to reproduce
Add the above to a default project and run
System Info
System:
    OS: macOS 14.4
    CPU: (8) arm64 Apple M1
    Memory: 137.05 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 21.7.3 - /opt/homebrew/bin/node
    Yarn: 1.22.22 - /opt/homebrew/bin/yarn
    npm: 10.8.1 - /opt/homebrew/bin/npm
    pnpm: 9.2.0 - /opt/homebrew/bin/pnpm
  Browsers:
    Chrome: 126.0.6478.114
    Chrome Canary: 128.0.6555.0
    Safari: 17.4

Used Package Manager
pnpm
Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Main world content scripts can't use any extension APIs, so they can only import WXT APIs from wxt/sandbox. Hard to tell with auto-imports, but createIntegratedUi uses the extension APIs.
You can add the UI to the DOM yourself if you're in a main world acript


introduce links to CDEvents

Changes
This commits introduces the concept of linking events to CDEvents. We
take inspiration from Eiffel for certain components, and also outline
new APIs and use cases.
resolves #151
Signed-off-by: benjamin-j-powell<EMAIL_ADDRESS>Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has the primer doc been updated if a design decision is involved
[x] Have the JSON schemas been updated if the specification changed
[ ] Has spec version and event versions been updated according to the versioning policy
[x] Meets the CDEvents contributor standards

Has spec version and event versions been updated according to the versioning policy

@afrittoli One thing that wasn't clear to me was the versioning. Can you explain the process? Since the link looks to 404

@xibz , looks like the title of this PR has been copied from another PR. Could you please correct that?

Has spec version and event versions been updated according to the versioning policy

@afrittoli @e-backmark-ericsson One thing that wasn't clear to me was the versioning. Can you explain the process? Since the link looks to 404. We should fix that as well and document the process.

The versioning of cdevents is documented here: https://cdevents.dev/docs/primer/#versioning
You should step the minor version of the change.created event from 0.1.2 to 0.2.0 and keep the spec version as 0.4.0-draft. It's correct that the $id field in the schema will not be possible to resolve until the new spec version is released (and then all files will be updated to 0.4.0 as part of the release as well)

The only thing missing on this PR is that the commit needs to be signed.
https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

The only thing missing on this PR is that the commit needs to be signed.
docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

@e-backmark-ericsson done!


Add nightly builds of eventing CRDs to rabbitmq

Add nightly builds of eventing to eventing-rabbitmq

/approve


Setting up acsp-api service

Setting up acsp-api service... have created some placeholder classes and files.

LGTM


CMIP6: ensembe approach, regionalization, and bias statistics

Hi @malmans2
I send you a new notebook that include some additional functionalities like:
-the creation of an ensemble of models
-the regionalization of the results
-some statistical analysis on the bias distribution
-plotting of taylor diagram for multimodel comparison
We discussed internally about interpolation and we decided to make the interpolation of the datasets after the postprocessing (temporal aggregation in this case) and just before the plotting. In our opinion this is the most scientifically appropriate procedure. That's why in this the notebook we put the regridding after the postprocessing.
I wait for your feedback, thank you!
Paolo
scripts_v2.zip

Hi @paololanteri ,
I haven't looked into all details of your notebooks yet, but I have a quick comment about interpolation and the use of ensambles.
I was going to implement some code to download and open multiple models using a single request chunked along the parameter "model". That would return a single dataset with an additional dimension (e.g., dimension "model":  {"model1", "model2", ...})
However, I realised that this is not possible without interpolating because different models have different coordinates.
Therefore, if interpolating each model before doing any processing is not accurate, the only solution I can see is that users write code using explicitly for loops (i.e., the same way you are doing it in your notebook).

Hi @malmans2,
thank you,
I agree with you, I can't see other ways to do it.

Hi @paololanteri,
In the next couple of days I'm going to work on your code and see if I can add more diagnostics for your specific use case.
Is the zip file up-to-date, or did you make any progress with your notebook?
If the latter, could you please send me your latest notebook?
Thanks!

Hi @malmans2 ,
I will update you in the next hours
Than you,
Paolo

OK, no rush.
FYI, inspired by your notebook I'm improving a little the caching of the download_and_transform function.
The previous version only allowed to apply transformations to each chunks.
That's not very useful when you need to perform operations on the whole dataset (e.g., time weighted means, ...), or when the final result is much smaller because there have been reductions along the chunked dimensions.
So I'm adding the argument transform_chunks.

When True,trasform_func is applied to each chunk. That means that we cache the original data and all transformed chunks.
When False,  trasform_func is applied to the dataset after concatenating all chunks, and it will store in the cache the original data and the final dataset.

If this sounds confusing, it should be much more clear once I send you some example (hopefully by tomorrow).

Hi @paololanteri ,
I haven't worked on the latest notebook yet, but I've updated the old wp4 template: https://github.com/bopen/c3s-eqc-toolbox-template/blob/main/notebooks/wp4/clima_t2m.ipynb
You'll see that it's using the argument transform_chunks=False.
Could you please test it when you get a chance?
Thanks!

Hi @paololanteri ,
I made some progress. If you want to follow along, I'm working on this notebook: https://github.com/bopen/c3s-eqc-toolbox-template/blob/wp4-paolo/notebooks/wp4/wip.ipynb
I didn't get to the regionalization/analysis part yet, but I've revised the way you downloaded, resampled, and interpolated the results. If you try it, let me know how it goes.
I'll push more commits tomorrow.

Hi @malmans2,
perfect, thank you,
tomorrow I will try it.

Hi @malmans2,
after downloading the branch "wp4-paolo" and creating a new environment the notebook wip.ipynb works correctly!

Hi @paololanteri,
I merged both notebook and code.
The notebook is ready for review. Please have a look and carefully check that all results are correct.
Here is the clima_and_bias_pr_cmip6_regionalised.ipynb.
A couple of comments:

You need to use the latest c3s-eqc-automatic-quality-control as I added/changed some diagnostics
I haven't added skillmetrics to the environment yet. Let's wait and see if anyone else will need it. I will install it on the wp4 VM environment though. If you are working locally, you have to install it yourself.
I'm not useing seaborn so we don't add another dependency. The distribution plots with matplotlib look OK to me, but let me know if you'd like to use seaborn instead.

Let me know!

I forgot to mention, it has been decided that the final result should be a notebook rather than a set of images.
Therefore, we don't need to save each figure anymore.

Hi Paolo,
I'm closing this as there's a template for this under wp4, but feel free to re-open if anything is not right.


chore: Update types to be consistent with documentation

Closes #243
Documentation reference:

Auth Token Response
Shared Labels
Project
Create new task
Update task
Create new comment

@pawelgrimm, I don't think I have permission to assign you as a reviewer. I hope you don't mind the direct tag, thank you!


itemClicks(), itemClickEvents()...for RxRecyclerView?

Will there be a plan to add those bindings for RecyclerView? It would be a very useful binding for recycler views to act like traditional adapter views.

RecyclerView does not offer those callbacks.


Support for postgres version upgrade using Bitnami chart

I'm attempting to execute a major postgres version upgrade (from 14 to 15) for the containerized image from Bitnami.
Up to now, there doesn't exist an official documentation on how to upgrade those images:
https://github.com/bitnami/charts/issues/8025
https://github.com/bitnami/charts/issues/14926
Therefore, I used a combination of the following two comments as guidance:
https://github.com/bitnami/charts/issues/8025#issuecomment-1669896258
https://github.com/bitnami/charts/issues/8025#issuecomment-1704813690
Unfortunately, in the end, I'm always facing this error in step at step 8 of the list below:
The source cluster was not shut down cleanly.
I assume because I'm not doing the following step 2 (from https://github.com/bitnami/charts/issues/8025#issuecomment-1704813690) correctly:
"POSTGRES_DB must be unset in the Helm Release, otherwise migration script will complain about the present database."
I'd next look into https://github.com/bitnami/charts/issues/8025#issuecomment-1708879837
Here the step that I'm executing:

enable diagnosticMode for postgres dependency

unset POSTGRES_DATABASE

unset POSTGRES_DATABASE

Download binaries of your old installation.

cd /tmp/

perl -e 'use IO::Socket::INET; my $s = IO::Socket::INET->new(PeerAddr=>"downloads.bitnami.com", PeerPort=>80, Proto=>"tcp"); print $s "GET /files/stacksmith/postgresql-14.2.0-0-linux-amd64-debian-11.tar.gz HTTP/1.0\r\nHost: downloads.bitnami.com\r\n\r\n"; while(<$s>) { last if $_ eq "\r\n" }; open(my $f, ">", "postgresql-14.2.0-0-linux-amd64-debian-11.tar.gz") or die "Could not open file: $!"; print $f $_ while <$s>; close $f; close $s;'

mkdir /bitnami/postgresql/oldbin/

tar --extract --directory /bitnami/postgresql/oldbin/ --file postgresql-14.2.0-0-linux-amd64-debian-11.tar.gz --strip-components=4 postgresql-14.2.0-linux-amd64-debian-11/files/postgresql/bin

copy old datadir and create postgres.conf and trusted access file for old

/opt/bitnami/scripts/postgresql/entrypoint.sh mv /bitnami/postgresql/data /bitnami/postgresql/olddata
cp /bitnami/postgresql/olddata/postgresql.auto.conf /bitnami/postgresql/olddata/postgresql.conf
echo "local all postgres trust" > /bitnami/postgresql/olddata/pg_hba.conf

Initialize new cluster

/opt/bitnami/scripts/postgresql/entrypoint.sh /opt/bitnami/scripts/postgresql/setup.sh
/opt/bitnami/scripts/postgresql/entrypoint.sh chmod 700 /bitnami/postgresql/olddata
/opt/bitnami/scripts/postgresql/entrypoint.sh chmod 700 /bitnami/postgresql/data

Create postgres.conf and trusted access file

cp /bitnami/postgresql/data/postgresql.auto.conf /bitnami/postgresql/data/postgresql.conf
echo "local all postgres trust" > /bitnami/postgresql/data/pg_hba.conf

Avoid running into 'FATAL:  lock file "postmaster.pid" already exists' at step 8

rm /bitnami/postgresql/olddata/postmaster.pid

Check consistency fist

/opt/bitnami/scripts/postgresql/entrypoint.sh pg_upgrade  -b /bitnami/postgresql/oldbin -B /opt/bitnami/postgresql/bin -d /bitnami/postgresql/olddata -D /bitnami/postgresql/data

If it says the versions are compatible, run the same command without -c and it will copy the data

/opt/bitnami/scripts/postgresql/entrypoint.sh pg_upgrade  -b /bitnami/postgresql/oldbin -B /opt/bitnami/postgresql/bin -d /bitnami/postgresql/olddata -D /bitnami/postgresql/data -c

Delete pg_hba.conf and postgresql.conf so the container versions are used on reboot

rm /bitnami/postgresql/data/postgresql.conf
rm /bitnami/postgresql/data/pg_hba.conf

Can you please support me for this issue?
Figuring out how to execute a postgres version upgrade for the containerized image from Bitnami could be beneficial to multiple products, not just for portal, as multiple products are using it as dependency.

FYI @evegufy  we moved this to our general tasks for system team to find together a way for updates

@FaGru3n next week we'll test a blue green deployment approach, just fyi as it may be affecting this issue.

This comment worked for me.
According to the Postgresql documentation:
Because pg_dump is used to transfer data to newer versions of PostgreSQL, the output of pg_dump can be expected to load into PostgreSQL server versions newer than pg_dump's version. pg_dump can also dump from PostgreSQL servers older than its own version. (Currently, servers back to version 8.0 are supported.) However, pg_dump cannot dump from PostgreSQL servers newer than its own major version; it will refuse to even try, rather than risk making an invalid dump. Also, it is not guaranteed that pg_dump's output can be loaded into a server of an older major version — not even if the dump was taken from a server of that version. Loading a dump file into an older server may require manual editing of the dump file to remove syntax not understood by the older server. Use of the --quote-all-identifiers option is recommended in cross-version cases, as it can prevent problems arising from varying reserved-word lists in different PostgreSQL versions.

I've created a chart with 2 dependencies (bitnami charts with postgres versions 14 and 15).
Created a table and 1 row in the pod for version 14.
Ran this command: kubectl exec -it portal-postgres-new-0  -n portal-postgres -- /opt/bitnami/scripts/postgresql/entrypoint.sh /bin/bash -c 'export PGPASSWORD="asd"; echo "local all postgres trust" > /opt/bitnami/postgresql/conf/pg_hba.conf; pg_ctl reload; time pg_dumpall -h <IP_ADDRESS> -U postgres | psql -U postgres' where <IP_ADDRESS> was the cluster IP of the old version database.
Disabled the old postgres version in the chart
Scaled the version 15 pod to 0 and back to 1 to restore the pg_hba.conf file to its original state.
Queried the new database and got my table with the inserted row.

# Chart.yaml
apiVersion: v2
name: portal-postgres
description: A Helm chart for Kubernetes
type: application

version: 0.1.0
appVersion: "1.16.0"

dependencies:
  - condition: postgresql.enabled
    name: postgresql
    repository: https://charts.bitnami.com/bitnami
    version: 11.9.13 # postgres 14
  - condition: newpostgresql.enabled
    name: postgresql
    repository: https://charts.bitnami.com/bitnami
    alias: newpostgresql
    version: 12.12.10 # postgres 15

# values.yaml
postgresql:
  enabled: false
  auth:
    database: "postgres"
    port: 5432
    postgresPassword: "asd"
newpostgresql:
  enabled: true
  fullnameOverride: portal-postgres-new
  auth:
    database: "postgres"
    port: 5432
    postgresPassword: "asd"

Let's try this tomorrow in a session.

A new follow-up issue is created to document the process so I would close this one.


Maximum property in the ProgressBar in Fluent Theme

Seems changing Maximum property in ProgressBar does not work in the Fluent Theme:
<ProgressBar  IsIndeterminate="False" Minimum="0" Maximum="1" Value="{Binding ProgressValue}"  />
since Maximum is set to 100:
https://github.com/AvaloniaUI/Avalonia/blob/9b8d4d7367548f51cbca2e5500d6942b9503042e/src/Avalonia.Themes.Fluent/ProgressBar.xaml#L15

Setting it explicitly as you have done, should override the style... are you certain this is the case...
if not the I think this would be a bug, @grokys

<ProgressBar IsIndeterminate="False" Value="50" Minimum="0" Maximum="1" Height="20" Width="200" />
How it looks in Default Theme:

How it looks in Fluent Theme:

also, i tried to set the Maximum property in a style, but it applies to all progress bars:
  <StackPanel Spacing="8">
    <ProgressBar IsIndeterminate="False" Value="50" Minimum="0" Maximum="100" Height="20" Width="200" Classes="mmm" />
    <ProgressBar IsIndeterminate="False" Value="50" Minimum="0" Maximum="100" Height="20" Width="200" />
  </StackPanel>

<Styles xmlns="https://github.com/avaloniaui" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml">
    <Style Selector="ProgressBar.mmm">
        <Setter Property="Maximum" Value="1" />
        <Setter Property="Foreground" Value="Red" />
    </Style>
</Styles>

The result:

The Foreground property applies only to "mmm" class, but the Maximum applies to all progress bars. Not sure, probably it's another issue.


F0, Failsafe F0 and wavelet F0 inaccurate results and limited range

Hi,
I noticed the none of the fundamental frequency features seems to work properly, I'm testing the function in 2 different ways:

Classical music: I would expect it to be quite on spot with this kind of music, F0 simply doesn't detect anything it's always 0, the failsafe keep bouncing between 0 and a very large number which is definitely not even close to the actual value. Wavelet also rapidly bounce between 0 and an unrealistic value.

Test audio track to swipe across a range: F0 only starts picking up the frequency near ~1700hz, the value then is accurate. Wavelet seem to be working fine with the test track, the value is accurate till ~1500hz, for higher frequencies it loose it dropping the value. Failsafe f0 as expected it reflects F0 or shows a huge number

Hi, sorry it has been so long to reply, but I'm finally getting around to some LibXtract stuff. Do you still have the audio example you were testing with? If so can you post it somewhere? Thanks


MGMT-16785 Allow getting telemetry for IBI adoption

Add openshift-install-manifests configmap with:
invoker=image-based-install

/hold

/unhold

/lgtm


Rendering property with allOf ignores title and description

I have an object with an allOf listing a $ref.  In this case, the title and description fields are ignored and only the schema name is used in those locations.
{
  "openapi": "3.1.0",
  "info": {
    "title": "Some API",
    "summary": "",
    "description": "",
    "version": "alpha"
  },
  "paths": {
    "/api/v1/whatever/": {
      "post": {
        "parameters": [],
        "responses": {
          "200": {
            "description": "A Response",
            "content": {
              "application/json": {
                "schema": {
                  "$ref": "#/components/schemas/Response"
                }
              }
            }
          }
        }
      }
    }
  },
  "components": {
    "schemas": {
      "NumericValue_str_": {
        "properties": {
          "nv": {
            "type": "string"
          }
        }
      },
      "Response": {
        "properties": {
          "value": {
            "allOf": [
              {
                "$ref": "#/components/schemas/NumericValue_str_"
              }
            ],
            "title": "My Title",
            "description": "My Description"
          }
        }
      }
    }
  }
}

Result:

Wanted:

(also, I'm not clear on the what is happening with the font size there, but honoring the title and description content is what I am after here).
If the $ref is moved outside an allOf or if its an anyOf instead, then it works as expected.  Having more values in the allOf does not help.

Yes, I agree that the type ought to remain as the actual schema name.  The description is what is needed more.  Cases like this, where there is ambiguity is where its most important to be able to manually control this area.  I will admit that I am unclear on just why my schema generator is choosing to use allOf here, especially with only one value, but so far as I can tell, it is a legitimate use case in general.  For this specific instance, I think my schema generator could do a better job at what I am asking it to do, but I don't think that problem is as relevant so long as I can get the overriding title and description to work.

Yeah, that part is definitely seems a bug. Thanks for finding this.

Fixed in 2.2.719.


ctx.db.mutation → ctx.db.types.Mutation?

Using prisma/1.20.7, it appears the mutations are no longer accessible inside a resolver using ctx.db.mutation, but instead are now accessible via ctx.db.types.Mutation. Is that correct? If so, how come there’s no documentation about that? Or am I doing something incorrectly?

Looks like I upgraded from Prisma Bindings to Prisma Client... love how quickly Prisma is developing its tech — one downside is that I can barely keep up! 🎉


Create report outline

The goal of this ticket is to have a structure for the report that identifies the types of sections that will be expanded on in the future.
Acceptance Criteria:

[x] Identify segments of report
[x] Get feedback on segments from project team
[x] Write high-level bullets for segments

It's coming together! I'm so excited for this part :) Way to go @jaimiebutton!


HOSTEDCP-1566: Update go.mod for /hack/tools

What this PR does / why we need it:
Update go.mod for /hack/tools
Which issue(s) this PR fixes:
Fixes HOSTEDCP-1566
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/test all

/area ci-tooling

/test verify

/test verify
/test images

/test verify
/test images

/test verify
/test images

/test verify
/test images

/close
We can't bump this yet because it needs a new Golang update and we are not ready to move to 1.22 yet.

/reopen

/close