It is possible to allow multiple domains as options

Something like this:
app.use(helmet.frameguard({
action: "allow-from",
domain: ["https://example1.com", "https://example2.com", "https://example3.com"]
}));

See this part of the README.

I miss it? Where is it?

Because it's not natively supported by the X-Frame-Options header, this is something I'd discourage in favor of Content-Security-Policy's frame-ancestors directive.


Added permanentRedirect option to i18nRouter

There are situations when you could prefer to have permanent redirects with 301 status instead of temp redirects with 307/308 status.
I added optional prop for that.

Hi @samvelArm. I believe a permanent redirect for all redirects would have some problems.
Off the top of my head, this would cause issues with redirecting to a user-specified language. For example if your browser is set to English, you will be immediately permanently be redirected to /en on your first visit. If the user later changes their language to German via a UI button, they will still be redirected to /en the next time they visit the site at /.
I think this is an edge case and will cause problems and confusion since the developer really needs to know what they're doing. You can implement this in your middleware pretty easily if you really want this:
export function middleware(request: NextRequest) {
  const response = i18nRouter(request, i18nConfig);

  if (response.status === 307) {
    response.status = 301;
  }

  return response;
}

That is the option I need, what is the reason for implementing custom logic in the middleware when I am using lib for that?
This PR doesn't break anything, it's an optional change, and I didn't get the reason for declining that.
If someone doesn't need this change he can simply skip the param.
The permanent redirects are needed for SEO proposes as 301 doesn't get cached.
And I really didn't see the reason not including my changes in the library itself.
If the case it that I can implement this in the middleware, well I can implement in the middware the entire library, but the library doesn't give me the opportunity to select the status code of the redirect and cuts the next.js supported stuff.


Feature/harsh daniel/vsr page 2

Tracking Info
Resolves #19
Changes

Added VSR Page 2 backend and frontend
Updated MongoDB, added new validators and controllers, and created second page form

Testing

Ran frontend and went to localhost:3000/vsr.

Can see created VSR form in MongoDB after hitting submit button

TODO

Confirmation of Change

Run the frontend and go to localhost:3000/vsr. This should show both existing pages of the VSR form. Use the Next button after filling out the first page to see the second page.
After hitting Submit on the second page, check MongoDB for newly created VSR object

Thanks @2s2e , I fixed the issues!


(#2674) Subscription page stream filtering results should include streams whose description matches the user query.

The filtering process first obtains the streams whose name matches with
the user search query, then it sorts and displays them. From the remaining
streams, it obtains the ones whose description matches the query and
displays them in sorted order after previous results. The rest streams are
not displayed.
Fixes: #2674.

Automated message from Dropbox CLA bot
@vabs22, it looks like you've already signed the Dropbox CLA. Thanks!

One thing that impacts this change is that we want to move away from data-stream-name and start using data-stream-id for the widgets here.

data-stream-name has been used here to sort the streams alphabetically. Are you suggesting to use data-stream-id for this ?

Does it make sense to just sort the subs here, and then map the subs back to the widgets after the sort?  This would accomplish two things:

We do fewer jQuery lookups O(N) vs. O(NlogN)
It may be easier to unit test the sorting logic without the jQuery interactions.

i am not able to get your point, can you please elaborate

@vabs22 sorry for the slow reply!  When asking a question, I'd encourage you to provide more detail on which part(s) you did and didn't understand; @showell made several points.  That makes it much more likely that you'll get a prompt reply.

@vabs22 I am suggesting this process:

use jQuery to get a list of streams
sort/filter the streams themselves, not the widgets
once you have the right list of streams, use that to create a new ordering of widgets

@showell i had got your point after a while, and i had updated this PR according to that then :) Please give your feedback on it.
@timabbott okay, i will keep this in mind in future, thanks :)

@showell yes, in that case widget is displayed along with other widgets (having stream name match). Also the widgets having stream name match are displayed before widgets with stream description match, and both are displayed in sorted order.

@vabs22 sorry for the delay, can you rebase this?  It seems to have grown some merge conflicts.

Otherwise looks great!

@vabs22 Thanks for the pull request, and thanks for writing a descriptive commit message. Here is a little bit of rewriting to make it clearer and make it fit our guide, specifically, "The first line of commit messages should be written in the imperative and be kept relatively short while concisely explaining what the commit does." and "The rest of the commit message should be written in full prose and explain why and how the change was made."
Include stream description matches in subscription page filter results.

User search for streams will now return results where the stream
description (but not the stream name) include the string in the
user query.

The filtering process first obtains the streams whose names match the
user search query, then sorts and displays them. From the remaining
streams, it obtains streams whose description matches the query and
displays them in sorted order after the name match results. Other
streams are not displayed.

Fixes: #2674.

Hope this helps!
I look forward to this improvement and thank you for working on it. :)

@brainwane Thanks! I will try to make it better from now on :).

@showell feel free to give this another read and then merge if you're happy with it (I'll be offline today).

@vabs22 @timabbott This PR looks pretty solid, but I'm not ready for it to be merged yet, as we have a history of streams setting stuff being kind of buggy, so I want to be really thorough with the automated tests here.  I also added a comment related to _.each().

@showell i have added a couple of test cases which assert the content of the two arrays and checks that a stream belongs to only of them, when the keyword matches both description and title. Would these be enough?

@timabbott Both of my comments have been addressed here, so I am about to merge this.

Oops, looks like we forgot to close this when it was merged :).


Update main.sh

Fixes:
!!!
!!! PLEASE USE 'dev_boot_altfw' INSTEAD OF 'dev_boot_legacy'
!!!

@googlebot I signed it!

The dev_boot_altfw variable was only added very recently, so this code will break most platforms (including old platforms that no longer get updates but we still support according to CROS_MIN_VERS).
This PR (thanks, by the way!) unfortunately will need to detect which is the correct variable to use, and then (preferably) use that.  Something like:
altfw=dev_boot_altfw
if crossystem "$altfw" 2>&1 >&- | grep -q Invalid; then
    altfw=dev_boot_legacy
fi

Yes. I will get busy on this.
(No worries.)


More details needed on network setup required between Hyper-V host and client/guest OS.

Nothing on this page about what network connectivity is needed. What ports need to be open. What kind of Virtual Switch needs to be set up, configured. Nothing. So... There's that....

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

ID: dd7ba3e6-21be-c6f8-b292-723504bb1e48
Version Independent ID: e43fd080-13ab-7c06-4951-2e017b715ed1
Content: Manage Windows Virtual Machines with PowerShell Direct
Content Source: virtualization/hyper-v-on-windows/user-guide/powershell-direct.md
Product: windows-10-hyperv
GitHub Login: @scooley
Microsoft Alias: scooley

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Add support for content being a Buffer

As I understand the story today, I can supply a base64 encoded string for the content property of a Document object that I wish to process where the content will be either from a PDF, a TIFF or a GIF.  This request asks for consideration of other equivalent data structures such as a NodeJS Buffer.  I am reading GCS data and getting Buffer objects and would like to pass in Buffers without explicitly converting to base64 (in user code) first.

@kolban-google For feature request, I would recommend you to open new issue here:
https://issuetracker.google.com
With this note, I am closing the issue.


Resonable sleuth overhead

Understandably enabling sleuth on a spring boot application will add some additional overhead. The question, is what is reasonable overhead. Enabling sleuth (2.1.1) on Spring Cloud Gateway 2.1.1 seems to slow down request processing times by around 20 to 40%. Is this in the expected range? Note, this is much better than Spring Cloud Gateway 2.0.x where the performance was 500% slower (this was a known issue in reactive).
@spencergibb

I wonder if there's a way to get to the same page with this. For example, if you can make a sample app, and perhaps use a tool like wrk to show the before/after? Not sure if this one helps or not https://github.com/openzipkin/sleuth-webmvc-example

Thanks, probably can do that. Will get back to you.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

I will provide repo which shows performance issue. Just hang on a few days :)

@tony-clarke-amdocs any progress on this?

@marcingrzejszczak It's on my todo list. I have not forgotten about it. I plan to do it in coming weeks.

Ok so I'll close it until then. If you have anything to show us, we'll reopen it.

We have an example that may be useful.
We have used Spring Cloud Gateway to check the impact of Sleuth. In our tests the average time with Sleuth is 77% worse than without.
The repository with the test performed is: https://github.com/rgarciapariente/test-perf-sleuth
We simply compared a Spring Cloud Gateway against a test micro. In one case, Spring Cloud Gateway includes Sleuth and another does not.
I hope this can be helpful.

That's a duplicate of https://github.com/spring-cloud/spring-cloud-sleuth/issues/1397


[CELEBORN-1295] Add tm to Celeborn's website.

What changes were proposed in this pull request?
Add trace mark symbol.
Why are the changes needed?
Does this PR introduce any user-facing change?
How was this patch tested?

Should we also change mkdoc.yml and web module?

Should we also change mkdoc.yml and web module?

Celeborn website doesn't use the mkdoc.yaml from this repo. It uses the https://github.com/apache/incubator-celeborn-website repo and I have changed that mkdoc.yaml.

@FMX, does the logo picture in docs/logo directory need to add tm?

@FMX, does the logo picture in docs/logo directory need to add tm?

Referring to other projects, I think it is not necessary.

merge to main(v0.5.0)


7.3 and 7.4 tags not published to Docker Hub

Docs talk about tagged images for 7.3 and 7.4 but they don't exist on Docker hub.

Someone updated the readme with that incorrectly.

Someone updated the readme with that incorrectly.


(v0.8.0-release) Support zos for AQA releases (#3293)

cherry-pick: https://github.com/adoptium/aqa-tests/pull/3293
Signed-off-by: lanxia<EMAIL_ADDRESS>

I will have another PR to update JDK11_OPENJ9_BRANCH in the v0.8.0-release branch once this is merged.

FYI @pshipton


ENH: Update version of DCMTK to 3.6.1_201610.12

@che85 this depends on patched ITK here, correct?
https://github.com/fbudin69500/ITK/tree/update_DCMTK

Yes, correct.

Wait, no I didn't update the revision tag from the External.cmake here. So actually this PR doesn't depend on the patched ITK...

Right, but CTK reused ITK from Slicer, and that ITK is the patched one. CTK does depend on ITK, so if you were to compile it standalone without updating ITK hash in CTK externals, I would expect CTK build to fail.

I will check this out.

@fedorov You were right. It doesn't build with the currently defined ITK revision in https://github.com/commontk/CTK/blob/master/CMakeExternals/ITK.cmake#L28
Should I update this or just wait for further action of Francois?

Additional fixes will need to be done to ITK to fix Slicer issues you identified, so things will change.
I would not spend any more time on this until we hear back from @thewtex and @fbudin69500.

This looks good to me but a decision has to be made on the DCMTK commit to choose for the update:
18e2abbd930f00fb41c1cea42cec04ccf0cf8637 (08/23/2016)
efe1a177eac040b6d41aae0db7714cd970e12bc5 (10/12/2016)

@jcfr  @thewtex  Since Slicer and CTK do not require C++11 features, this patch is good to be merged.

Thanks for the patches.
Next will be the update of Slicer ?

yes :)

Note that we are waiting for some additional merges to DCMTK, and possibly release of the next snapshot early November (@michaelonken is coordinating this). If/when this happens, we will need to go through the update process again.

Nice work all!

@fbudin69500 why does it matter whether Slicer or CTK uses C++ 11 features?

why does it matter whether Slicer or CTK uses C++ 11 features?

@nolden  At that point in time, it doesn't really for CTK and Slicer. That said, some folks that depends on ITK which depends on DCMTK build their project (e.g BRAINSTools maintained by @hjmjohnson) are using C++11, this is why this came up in the discussion. @fbudin69500 was doing some testing build BRAINSTools and found the issue.
The good news is that @michaelonken mentioned that they work working on fixing DCMTK.
Et voila, hope this clarify things.

Ok, thanks @jcfr . MITK already uses C++11 features and passes
CMAKE_CXX_STANDARD:STRING=11 to all external projects in the SuperBuild.
No problems so far but good to know before the next CTK/DCMTK update
On 10/28/2016 06:32 PM, Jean-Christophe Fillion-Robin wrote:

why does it matter whether Slicer or CTK uses C++ 11 features?

@nolden https://github.com/nolden At that point in time, it doesn't
really for CTK and Slicer. That said, some folks that depends on ITK
which depends on DCMTK build their project (e.g BRAINSTools maintained
by @hjmjohnson https://github.com/hjmjohnson) are using C++11, this
is why this came up in the discussion. @fbudin69500
https://github.com/fbudin69500 was doing some testing build
BRAINSTools and found the issue.
The good news is that @michaelonken https://github.com/michaelonken
mentioned that they work working on fixing DCMTK.
Et voila, hope this clarify things.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/commontk/CTK/pull/675#issuecomment-256966895, or
mute the thread
https://github.com/notifications/unsubscribe-auth/AANXobw7WfQXaHIRXk1jDJScMpATbE23ks5q4iOjgaJpZM4Kc5j4.

@nolden DCMTK requires the CMake variable boolean "DCMTK_USE_CXX11_STL" to specify that it needs to be built with C++11 support. My guess is that when it is built with MITK, it is actually not build with C++11 features. Let me know if I am mistaken.

New version of DCMTK still does not compile with C++11
In file included from /scratch/johnsonhj/src/BT-11/DCMTK/ofstd/tests/tvariant.cc:5:
/scratch/johnsonhj/src/BT-11/DCMTK/ofstd/include/dcmtk/ofstd/ofvriant.h:61:9: warning: type definition in a constexpr function
is a C++14 extension [-Wc++14-extensions]
struct test { alignas(OFvariant_traits<Index+1,Alternatives...>::alignment()) first_alternative a; };
^
/scratch/johnsonhj/src/BT-11/DCMTK/ofstd/include/dcmtk/ofstd/ofvriant.h:61:23: error: requested alignment is less than minimum
alignment of 4 for type 'first_alternative' (aka 'float')
struct test { alignas(OFvariant_traits<Index+1,Alternatives...>::alignment()) first_alternative a; };
^
/scratch/johnsonhj/src/BT-11/DCMTK/ofstd/include/dcmtk/ofstd/ofvriant.h:61:16: note: in instantiation of member class 'test'
requested here
struct test { alignas(OFvariant_traits<Index+1,Alternatives...>::alignment()) first_alternative a; };
^
/scratch/johnsonhj/src/BT-11/DCMTK/ofstd/include/dcmtk/ofstd/ofvriant.h:61:74: note: in instantiation of member function
'OFvariant_traits<1, float, bool>::alignment' requested here
struct test { alignas(OFvariant_traits<Index+1,Alternatives...>::alignment()) first_alternative a; };
^
/scratch/johnsonhj/src/BT-11/DCMTK/ofstd/include/dcmtk/ofstd/ofvriant.h:61:16: note: in instantiation of member class 'test'
requested here
struct test { alignas(OFvariant_traits<Index+1,Alternatives...>::alignment()) first_alternative a; };
^
/scratch/johnsonhj/src/BT-11/DCMTK/ofstd/include/dcmtk/ofstd/ofvriant.h:316:21: note: in instantiation of member function
'OFvariant_traits<0, int, float, bool>::alignment' requested here
alignas(traits::alignment()) std::uint8_t m_Content[traits::size()];
^
/scratch/johnsonhj/src/BT-11/DCMTK/ofstd/tests/tvariant.cc:60:31: note: in instantiation of template class
'OFvariant<int, float, bool>' requested here
OFvariant<int,float,bool> v0( true );
^
1 warning and 1 error generated.
From: Jean-Christophe Fillion-Robin<EMAIL_ADDRESS>Reply-To: commontk/CTK<EMAIL_ADDRESS>Date: Friday, October 28, 2016 at 11:32 AM
To: commontk/CTK<EMAIL_ADDRESS>Cc: Hans Johnson<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [commontk/CTK] ENH: Update version of DCMTK to 3.6.1_201610.12 (#675)
why does it matter whether Slicer or CTK uses C++ 11 features?
@nolden At that point in time, it doesn't really for CTK and Slicer. That said, some folks that depends on ITK which depends on DCMTK build their project (e.g BRAINSTools maintained by @hjmjohnson) are using C++11, this is why this came up in the discussion. @fbudin69500 was doing some testing build BRAINSTools and found the issue.
The good news is that @michaelonken mentioned that they work working on fixing DCMTK.
Et voila, hope this clarify things.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


fix warnings from PVS Studio

Hi!
well the PR cannot be accepted in its current form - mainly because an include has been moved from the implementation to the interface header (#include <fstream>) and also because the builds don't pass. Is there a way to suppress PVS studio warnings in the code - with some comment?

I guess removing and recreating a remove dev branch triggered this to get closed - my bad.


[Robust] Sound support

Closes #369
https://github.com/wixoaGit/OpenDream/pull/369 but without conflicts
Untested because I don't have decent headphones with me.

Sounds fine: https://streamable.com/p72n47


In ocp4.9 and above, igc pipeline command fails but doesnot give any error. Tasks and pipelines get copied  and get deleted itself in target namespace.

This issue got resolved by changes by the this igc cli repo: https://github.com/cloud-native-toolkit/ibm-garage-cloud-cli/pull/386


Feature Request: IphoneX

@m0ngr31  - First and foremost we use this package in almost every page on our application so kudos for creating it. I was wondering if the package is still being actively supported? I'd be willing to open a PR for IphoneX now that Apple has that.
Would be similar to https://github.com/ptelad/react-native-iphone-x-helper but I believe the less packages you can bring in the better. Let me know if you are accepting PRs and I'll open one!

Thanks for the kind words. I haven't updated it in a good while, so I'm glad you are getting some mileage out of it. If you want to submit a PR, I'll get it updated on npm.

@m0ngr31 - PR created https://github.com/m0ngr31/react-native-device-detection/pull/3. Let me know if you have any questions or concerns. Thanks!


Fix wrong service name

Motivation and Context
Typo fix in service name on SSM example.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you for the fix! However, examples are maintained at https://github.com/awsdocs/aws-doc-sdk-examples. Please submit this to that PR and I'll merge it there z

https://github.com/awsdocs/aws-doc-sdk-examples/pull/4990 Done @DavidSouther !


Disable and revert unpacked-list-comprehension (UP027)

Summary of changes
In https://github.com/astral-sh/ruff/issues/12754 it was found that the original pyupgrade rule unpacked-list-comprehension (UP027) is based on lacks justification and is actually slower in cases relevant to unpacking (ie: small sizes). The rule is deprecated in https://github.com/astral-sh/ruff/pull/12843 for Ruff 0.6
This PR disables the rule now before there's any usage of it. I also searched for ,.*? = \( to find any code that would've triggered it and revert.
Pull Request Checklist

[ ] Changes have tests
[ ] News fragment added in newsfragments/.
(See documentation for details)

Thank you very much.


z-index issue on "Add Item" button while editing a content in a column

Hi,
Thanks for your work, cockpit rocks 👍
When using a column of content and editing a text, the button "Add Item" in background is visible in foreground. See : https://www.useloom.com/share/ea0640b05f5f4aa4ad315e6ddc0b7699
Cheers,
Charles

This z-index issues are new 🤔 Thanks for reporting, I'll look into that


Request: add support for custom render actions

Hi,
Now, the only actions available to render are ['print', 'csv', 'excel', 'pdf']. The code searchs for the action request variable and if it matches any of those, it calls the function of the same name.
Could it be possible to add support for custom actions? If the action param doesn't matches any of the above, but a user function exists with that name, then call it.

The restrictions was added for security purposes since a user can hijack the request and set the action value to any valid method name.
Anyways, I am open to the idea of allowing custom actions but I think we should explicitly set a property of allowed actions to limit the access or some other ways that you can think of. Please do not hesitate to submit a PR for possible enhancement. Thanks!

Here you go...
https://github.com/yajra/laravel-datatables-buttons/pull/18
It's my first PR, be aware :-)

Thanks a lot. Simple yet elegant PR. 👍


Text resizing based on available space

This issue talks about it a little: https://github.com/flutter/flutter/issues/13455 as do these StackOverflows:
https://stackoverflow.com/questions/47665083/how-do-i-auto-scale-down-a-font-in-a-text-widget-to-fit-the-max-number-of-lines
https://stackoverflow.com/questions/50751226/how-to-dynamically-resize-text-in-flutter
Here's the specific constraints I have (that others might too):

I want to allocate a "maxSize" box of space for my text, e.g. 60 x 100; the text should never take up more than this space but can take up less.
The text is sized 30. If it overflows the space, i.e. hits three lines, scaleDown.
If the text is really short (e.g. 1 character), the box should be instead 30 x 100 instead of 50 x 100.

Things I've tried:
FittedBox
#13455 lets you adjust the text size by doing:
FittedBox(
  child: Text('short or long text'),
  fit: BoxFit.scaleDown,
);

but the text is always 1 line. This means the text is really small. So this only satisfies 1) if my "maxSize" is defined to be 1 line of text.
FittedBox with SizedBox
The first StackOverflow link suggests putting the FittedBox in a SizedBox, e.g.:
SizedBox(
  height: 60.0,
  width: 100.0,
  FittedBox(
    child: Text('short or long text'),
    fit: BoxFit.scaleDown,
  );

This satisfies 1) and 2) but if my text is short, I have all this extra space below my text that is not used. This isn't great from a UI perspective when I have widgets below this Text.
FittedBox with BoxConstraints
I tried replacing the SizedBox with BoxConstraints.
BoxConstraints(
  maxHeight: 60.0,
  maxWidth: 100.0,
  FittedBox(
    child: Text('short or long text'),
    fit: BoxFit.scaleDown,
  );

I think this one also resulted in 1-line text all the time.
Manually calculating font sizes based on String length
This is what I'm doing now. It'd be great if there were a Flutter widget that would satisfy the three constraints above though.
Ideas:

SizedBox could have a property that is "try-to-be-small-if-possible".
BoxConstraints can try to support multiple lines of text

@Hixie

You can measure the size of text with TextPainter, and render it with a CustomPaint (via a CustomPainter). The CustomPainter can scale its Canvas to ensure that the text will fit.
To create a widget that's one of two sizes you could use a LayoutBuilder. The LayoutBuilder's size will match the size of the widget it builds. The layout builder could create a TextPainter (or use the one created by its stateful parent widget), and use it to decide on the overall size (so return a widget wrapped in a SizedBox) and the scale factor. The text painter and scale factor would be handed off to the CustomPaint/CustomPainter.

In my opinion, this is a common use case and there should be a built in solution like Android's Autosize Textview.
Please consider providing a helper / Widget / plugin for this problem 👍

Any solution for this ?

Any package for text resizing ?

There is auto_size_text but have some issues.


chore: update changelog container

[x] Well-formatted commit messages
[x] Rebased/mergeable
[x] Tests pass

hmm - the flaky test looks related to flux timezone features...


Revisit the code used to close Marconi, and add iohk-monitoring logging through a transformer.

The former close code of the coordinators wasn't waiting for the workers to close nicely.
As I ran into issues while implementing it, I add iohk-tracing facility to ease debugging.

Pre-submit checklist:

Branch

[ ] Tests are provided (if possible)
[ ] Commit sequence broadly makes sense and have useful messages
[ ] Important changes are reflected in changelog.d of the affected packages
[ ] Relevant tickets are mentioned in commit messages

PR

[ ] (For external contributions) Corresponding issue exists and is linked in the description
[ ] Targeting main unless this is a cherry-pick backport
[ ] Self-reviewed the diff
[ ] Useful pull request description
[ ] If relevant, reference the ADR in the PR and reference the PR in the ADR
[ ] Reviewer requested

I'll re-review once you fix the conflicts

It will wait next week, I want to chat with you about where the tracer should be declared first (I didn't remember you already reworked the logger declaration in the retry PR).


Form-forwarding feature no longer working in form-builder 1.0.2

I regret to report the Form-Forwarding feature appears no longer to work...

The screen capture shows how the X-FBMS-FormForward HTTP header is being sent with the response, but the form in the page has not changed.

Chrome dev tools says that entries.length is undefined here:  https://github.com/uPortal-contrib/form-builder/blob/46e518401aecdd6f618039c7ece1f9e285db9c5c/src/App.js#L163

@rmathis -- FYI.

This issue was fixed by #49


Build of solution fails

A clone of the CodeGeneration.Roslyn solution failed to build. (I tried this both in Visual Studio 15.5.7 and the current preview of Visual Studio.) The Errors List shows the following error:
Severity	Code	Description	Project	File	Line	Suppression State
Error	MSB4062	The "GenerateCodeFromAttributes" task could not be loaded from the assembly C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src..\bin\CodeGeneration.Roslyn.Tests\Debug....\CodeGeneration.Roslyn.Tasks\Debug\net45\CodeGeneration.Roslyn.Tasks.dll. Could not load file or assembly 'file:///C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\bin\CodeGeneration.Roslyn.Tasks\Debug\net45\CodeGeneration.Roslyn.Tasks.dll' or one of its dependencies. The system cannot find the file specified. Confirm that the  declaration is correct, that the assembly and all its dependencies are available, and that the task contains a public class that implements Microsoft.Build.Framework.ITask.	CodeGeneration.Roslyn.Tests	C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Tasks\build\CodeGeneration.Roslyn.BuildTime.targets	20
Here is the full output of dotnet build:
PM> dotnet build
Microsoft (R) Build Engine version <IP_ADDRESS>948 for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.
C:\Program Files\dotnet\sdk\2.1.300-preview1-008174\Sdks\Microsoft.NET.Sdk\build\Microsoft.NET.Sdk.DefaultItems.targets(198,5): warning : A PackageReference for 'Microsoft.NETCore.App' was included in your project. This package is implicitly referenced by the .NET SDK and you do not typically need to reference it from your project. For more information, see https://aka.ms/sdkimplicitrefs [C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Tool\CodeGeneration.Roslyn.Tool.csproj]
Restore completed in 40.94 ms for C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Tests.RunBuild\CodeGeneration.Roslyn.Tests.RunBuild.csproj.
Restore completed in 43.8 ms for C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Attributes\CodeGeneration.Roslyn.Attributes.csproj.
Restore completed in 45.25 ms for C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Tests.Generators\CodeGeneration.Roslyn.Tests.Generators.csproj.
Restore completed in 45.26 ms for C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Tasks\CodeGeneration.Roslyn.Tasks.csproj.
Restore completed in 3.01 ms for C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Tests\CodeGeneration.Roslyn.Tests.csproj.
Restore completed in 2.06 ms for C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn\CodeGeneration.Roslyn.csproj.
Restore completed in 3.44 ms for C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Tool\CodeGeneration.Roslyn.Tool.csproj.
C:\Program Files\dotnet\sdk\2.1.300-preview1-008174\Microsoft.Common.CurrentVersion.targets(1126,5): error MSB3644: The reference assemblies for framework ".NETFramework,Version=v2.0" were not found. To resolve this, install the SDK or Targeting Pack for this framework version or retarget your application to a version of the framework for which you have the SDK or Targeting Pack installed. Note that assemblies will be resolved from the Global Assembly Cache (GAC) and will be used in place of reference assemblies. Therefore your assembly may not be correctly targeted for the framework you intend. [C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Attributes\CodeGeneration.Roslyn.Attributes.csproj]
C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Attributes\CodeGeneration.Roslyn.Attributes.csproj : error MSB4057: The target "Build" does not exist in the project.
C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Attributes\CodeGeneration.Roslyn.Attributes.csproj : error MSB4057: The target "Build" does not exist in the project.
CodeGeneration.Roslyn.Attributes -> C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\bin\CodeGeneration.Roslyn.Attributes\Debug\net40\CodeGeneration.Roslyn.Attributes.dll
CodeGeneration.Roslyn.Tasks -> C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\bin\CodeGeneration.Roslyn.Tasks\Debug\net45\CodeGeneration.Roslyn.Tasks.dll
CodeGeneration.Roslyn.Attributes -> C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\bin\CodeGeneration.Roslyn.Attributes\Debug\netstandard1.0\CodeGeneration.Roslyn.Attributes.dll
CodeGeneration.Roslyn.Tasks -> C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\bin\CodeGeneration.Roslyn.Tasks\Debug\netstandard1.5\CodeGeneration.Roslyn.Tasks.dll
C:\Program Files\dotnet\sdk\2.1.300-preview1-008174\Sdks\Microsoft.NET.Sdk\build\Microsoft.NET.Sdk.DefaultItems.targets(198,5): warning : A PackageReference for 'Microsoft.NETCore.App' was included in your project. This package is implicitly referenced by the .NET SDK and you do not typically need to reference it from your project. For more information, see https://aka.ms/sdkimplicitrefs [C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Tool\CodeGeneration.Roslyn.Tool.csproj]
CodeGeneration.Roslyn.Tests.RunBuild -> C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\bin\CodeGeneration.Roslyn.Tests.RunBuild\Debug\net462\CodeGeneration.Roslyn.Tests.RunBuild.dll
CodeGeneration.Roslyn -> C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\bin\CodeGeneration.Roslyn\Debug\netstandard1.5\CodeGeneration.Roslyn.dll
CodeGeneration.Roslyn.Tests.Generators -> C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\bin\CodeGeneration.Roslyn.Tests.Generators\Debug\netstandard1.5\CodeGeneration.Roslyn.Tests.Generators.dll
CodeGeneration.Roslyn.Tool -> C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\bin\CodeGeneration.Roslyn.Tool\Debug\netcoreapp1.0\dotnet-codegen.dll
C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Tasks\build\CodeGeneration.Roslyn.BuildTime.targets(20,5): error MSB6006: "dotnet" exited with code -2147450730. [C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Tests\CodeGeneration.Roslyn.Tests.csproj]
Build FAILED.
C:\Program Files\dotnet\sdk\2.1.300-preview1-008174\Sdks\Microsoft.NET.Sdk\build\Microsoft.NET.Sdk.DefaultItems.targets(198,5): warning : A PackageReference for 'Microsoft.NETCore.App' was included in your project. This package is implicitly referenced by the .NET SDK and you do not typically need to reference it from your project. For more information, see https://aka.ms/sdkimplicitrefs [C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Tool\CodeGeneration.Roslyn.Tool.csproj]
C:\Program Files\dotnet\sdk\2.1.300-preview1-008174\Sdks\Microsoft.NET.Sdk\build\Microsoft.NET.Sdk.DefaultItems.targets(198,5): warning : A PackageReference for 'Microsoft.NETCore.App' was included in your project. This package is implicitly referenced by the .NET SDK and you do not typically need to reference it from your project. For more information, see https://aka.ms/sdkimplicitrefs [C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Tool\CodeGeneration.Roslyn.Tool.csproj]
C:\Program Files\dotnet\sdk\2.1.300-preview1-008174\Microsoft.Common.CurrentVersion.targets(1126,5): error MSB3644: The reference assemblies for framework ".NETFramework,Version=v2.0" were not found. To resolve this, install the SDK or Targeting Pack for this framework version or retarget your application to a version of the framework for which you have the SDK or Targeting Pack installed. Note that assemblies will be resolved from the Global Assembly Cache (GAC) and will be used in place of reference assemblies. Therefore your assembly may not be correctly targeted for the framework you intend. [C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Attributes\CodeGeneration.Roslyn.Attributes.csproj]
C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Attributes\CodeGeneration.Roslyn.Attributes.csproj : error MSB4057: The target "Build" does not exist in the project.
C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Attributes\CodeGeneration.Roslyn.Attributes.csproj : error MSB4057: The target "Build" does not exist in the project.
C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Tasks\build\CodeGeneration.Roslyn.BuildTime.targets(20,5): error MSB6006: "dotnet" exited with code -2147450730. [C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\src\CodeGeneration.Roslyn.Tests\CodeGeneration.Roslyn.Tests.csproj]
2 Warning(s)
4 Error(s)

I also tried running against sdk 2.1.4, and I received a similar error. I decided to try to go back to 1.1.8, but then sensibly received the error, "The version of Microsoft.NET.Sdk used by this project is insufficient to support references to libraries targeting .NET Standard 1.5 or higher.  Please install version 2.0 or higher of the .NET Core SDK."
Finally, I thought I found the sweetspot with sdk 2.0.2. This at least discovers the project tests and avoids the errors above. But then I received another error: "The process cannot access the file 'C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\bin\CodeGeneration.Roslyn.Tasks\Debug\net45\CodeGeneration.Roslyn.Tasks.dll' because it is being used by another process."

@mbabramo  The first error:

C:\Program Files\dotnet\sdk\2.1.300-preview1-008174\Microsoft.Common.CurrentVersion.targets(1126,5): error MSB3644: The reference assemblies for framework ".NETFramework,Version=v2.0" were not found. To resolve this, install the SDK or Targeting Pack for this framework version or retarget your application to a version of the framework for which you have the SDK or Targeting Pack installed. Note that assemblies will be resolved from the Global Assembly Cache (GAC) and will be used in place of reference assemblies. Therefore your assembly may not be correctly targeted for the framework you intend.

is expected if you didn't have .NET Framework 2.0 SDK installed - the CodeGeneration.Roslyn.Attributes project has net20 target among multiple others, and so to build it you need corresponding SDK.
The second error:

"The process cannot access the file 'C:\Users\mbabr\source\repos\CodeGeneration.Roslyn\bin\CodeGeneration.Roslyn.Tasks\Debug\net45\CodeGeneration.Roslyn.Tasks.dll' because it is being used by another process."

is actually a common problem that I don't know how to permanently resolve (@AArnott ?), but a typical workaround is to kill all MSBuild.exe processes:

For some reason one MSBuild instance locks said dll and other instances can't access it.


Meeting notes for Jan 25 Office Hours

We kicked off our first office hours today! 🥳
I wasn't 100% sure how to log the minutes from this meeting, so here's a pull request that sticks them in the same meetings/2024 folder as our "regular" meetings, but I can also move them to a sub-folder, etc. too if that's better.
I also proposed here using the naming convention of YYYYMMDD.md because then the files will auto-sort chronologically, but that is a pedantic nerd thing. 🤓

I'm not in the habit of merging my own pull requests, but @Cheukting said it was ok in Slack. :D


pkg/profile: skip missing upper profile

This fixes a generator bug showing up with a missing next-profile,
by skipping upper profile alltogether in such a case.

I'm currently working on a ftest to cover this, it will come in a followup PR.


deepEqual(undefined, null) === true, even though undefined !== null

Example: https://runkit.com/591a7863d76ac2001216d505/591a7863d76ac2001216d506
var deepEqual = require("deep-equal") // v1.0.1

console.log(undefined === null); // false (expected)
console.log(deepEqual(undefined, null)); // true (unexpected)

Have you tried passing in {strict: true} as the third argument?

Thanks. Should have read the docs more carefully. I appreciate the suggestion!


Use sysctls available in macOS 12 / iOS 15 for hardware feature support.

@malfet ping for review.

Thank you!


No 'Update geodata' message showing

I was on NAM
instance ID was "giqerute-536"
version 1.2.12-103-g2f96742

There was updated geodata available, but it didn't direct @TNgidi to update it.
@Nicolaidavies can you confirm you bumped the geodata version in the config?

Could be the internet which is very slow today. I am getting 51 kb/s from fast.com

I got it this time

@onlyjsmith @mosaic141688 @TNgidi
We have a property called spatial_hierarchy.data_version that we need to bump every time we make a change to the geodata.
I just bumped nam staging and it forced me to update geodata.
This is far from ideal, but we can solve the new nam geodata not updating by bumping this number.
We should version each layer of geodata separately. Probably want to create a new card for a new feature.

Ok, great. I think I was saying the same yesterday!.. (https://github.com/locational/douma-app/issues/476#issuecomment-410613485)
So every time we change any geodata we need to bump that number. I’m fine with this approach for now.


what is the meaning of TRAINING???

what is the meaning of the stage 1 and stage 2 training?
i know that GAN is not needed to train.
is there anybody know the meaning of stage1 and stage2 training?
can you explain that to me?
help me guyz :)

Read the paper: https://arxiv.org/pdf/1612.03242v2.pdf
This is also not the forum for a question like this.

Imagine you want to run a marathon which is about 26 miles (~40 km). You have to practice or "train" in order to run that distance. Similarly, the GAN needs to be "trained" --- practice with hundreds of pictures-- in order to properly work and create images.
In this particular instance, the StackGAN is broken up into two parts: Stage 1 and Stage 2. Each part needs to be trained in order for the whole StackGAN to work properly


Added box shadow to places cards

Related Issue

Issue No. #154

Changes you made :

Added box shadow and on hover box shadows to places cards on home and tours pages

Check list :

[x] I have followed code style of this project.
[x] I have read the CONTRIBUTING document.
[x] All the aspects mentioned in the issue are covered.
[x] The website is properly working after my changes.
[x] 🌟 ed the repo

Type of change :

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Output Screenshots :

Home page:

On hover:

@goldenPheonix02 Good to go
LGTM 🤩


Telemetry coverage for InfraServer Roles Tab

Signed-off-by: Venkatesh-rengasamy<EMAIL_ADDRESS>:nut_and_bolt: Description: What code changed, and why?
Track the Infra-server roles actions using telemetry track event. The following actions should be tracked:-  search, create, delete and edit roles.

:chains: Related Resources
https://chefio.atlassian.net/browse/KINETICS-23
:+1: Definition of Done
Infra-server roles actions are tracked using telemetry track event.
:athletic_shoe: How to Build and Test the Change
inside the hab studio
[default:/src:0]# build components/automate-ui-devproxy/
[default:/src:0]# start_automate_ui_background
[default:/src:0]# start_all_services
STEP 2
open new window
go to automate UI path
$ cd components/automate-ui
and run the command
npm run serve:hab
Steps for preview screen:
Go To Infrastructure -> Chef Servers -> Click on any Chef Infra Server -> Click on Orgs name -> select Roles tab -> you can search, create, edit and delete roles -> For each actions, you can observe the telemetry track event in Network tab(inspect the element window).
API URL - https://telemetry-acceptance.chef.io/events
:white_check_mark: Checklist
All PRs must tick these:

[x] I have read the CONTRIBUTING document.
[x] All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

[x] Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
[x] Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
[x] Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
[x] Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
[x] Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

[ ] Tests added/updated? (all new code needs new tests)
[ ] Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.
:camera: Screenshots, if applicable


Improve python debugger client.

Replace DisplayData to DebuggerAction. The new class has only four type options. Furthermore several functions returning with DisplayData is changed to return with string.

Any more comments?

Thank you for the review. Patch updated.

Why did you changed the file permissions of jerry-debugger/jerry_client_ws.py?

It is a library, should never be executed directly.


SENRAN KAGURA Bon Appetit! (Hanzo x Crimson Squad) [PCSB00602]

App summary

App name: SENRAN KAGURA Bon Appetit! (Hanzo x Crimson Squad)
App serial: PCSB00602
App version: 1.00

Vita3K summary

Version: v0.1.2
Build number: 1942
Commit hash: https://github.com/vita3k/vita3k/commit/befe4e39

Test environment summary

Tested by: Zangetsu38
OS: Windows 10
CPU: AMD Ryzen 9 3900X
GPU: NVIDIA RTX 2070 Super
RAM: 16 GB

Issues

get one unicorn error on boot, can explain stop boot
Boot with blackscreen and fps freeze

Log
vita3k.log

Vita3K summary

Version: v0.1.4
Build number: 2397
Commit hash: https://github.com/vita3k/vita3k/commit/ee75b7c9

Issues
Some glitch on graphics:

texture get broken after switch on cloth in dressing mode

get strange explosion on render, no idea what is it, shinovis versus hafve same in more big

Sound can little crackely

Log

sound crackely on all, also on video intro, is ngs player issue

strange shadow element is like explose vertex, very visible in dressing move with rotate charachter

![image](https://user-images.githubusercontent.com/5261759/149113665-c5578e76-e328-484e-abfe-3064205adb1a.png

Log
PCSB00602 - [SENRAN KAGURA Bon Appetit! (Hanzo x Crimson Squad)].log

Doesnt work whit VULKAN backend.
vita3k.log

RETESTED ON PC - Confirm this is fully working for me now.

App summary
App name: SENRAN KAGURA Bon Appetit! (Hanzo x Crimson Squad)
App serial: PCSB00602
App version: 1.00
Vita3K summary

Version: v0.2.0
Build number: 3690
Commit hash: https://github.com/vita3k/vita3k/commit/[f0285143]
CPU backend: Dynarmic
GPU backend: Vulkan

Test environment summary

Tested by: JackMonty
OS: Windows 10
CPU: AMD Ryzen 5 Pro 3500U
GPU: Nvidia Geforce GT1030
RAM: 16 GB

Issues
Possible regression - Game fails to boot - crashes instantly - this happens on both OpenGL and Vulkan. Fials with message in logs: Unhandled C++ exception. vk::Device::allocateMemory: ErrorOutOfDeviceMemory
vita3k.log
Recommended labels

Remove Playable
Remove Graphics bug
Remove Audio Bug
Add Bootable


Fix: ensure buffer is loaded in buf_enabled

This PR adds a check at the top of buf_enabled to ensure that a buffer is loaded and returns false if not since it seems to me that any other check isn't worth it if the buffer isn't loaded. This fixes #21 . I wasn't sure though if there was a subtlety I was missing so another possible fix is adding a pcall at the specific error line (176)
  local ok_ft, ft = pcall(api.nvim_buf_get_option(bufnr, 'filetype'))
  if ok_ft and excluded_filetypes[ft] then
    return false
  end

thanks!


Wrong energy readings when there is more than one device

I have one outdoor unit with two indoor devices.
In the Panasonic Comfort Cloud app there are three energy readings for each device ("Heat", "Cool" and "Other"), where "Other" seems to be the energy used by the other (second) device.
Your integration seems to add all readings for the daily_enegy sensor.
That way both devices are reporting twice the energy actually consumed, as the devices' own entry is added to the "other" energy reading of the second device.

@xyz667 Can you enable debug logging and provide an energy log entry, they start with the following:
DEBUG (MainThread) [custom_components.panasonic_cc.pcomfortcloud.panasonicsession] POST url: https://accsmart.panasonic.com/deviceHistoryData

@xyz667 Can you enable debug logging and provide an energy log entry

@sockless-coding lat me explain the issue with a little more detail, maybe is just a misconception on my side.
As mentioned I have two indoor units, both connected to the same single outdoor unit.
I wanted to show energy readings for both indoor units in the HA energy dashboard, and thus added both daily_energy sensors (one for each indoor unit). daily_energy and not daily_cooling_energy because I want to see total energy consumption of each device, both heating and cooling.
As it turns out both daily_energy readings are the same (only minor differences, maybe rounding related).
Why? Because there‘s the „Other“ category in the Comfort Cloud app, and there is no HA sensor for that. „Other“ seems to be the energy used by the rest of the system (outdoor unit and all other attached indoor units).
See two screenshots for both indoor units of the same day (from Panasonic app):

One device was not in use on that day, still the energy used is (almost) the same, because of „other“ for the inoperative device.
Thinking about it, maybe it does not make sense to track the consumption of each indoor unit, as probably most power is used in the single outdoor unit driving both indoor units combined.
Thats why I have deleted one of the indoor units from the enegry dashboard, and still get total consumption numbers for both indoor units although only the sensor of one of them is included.
It‘s like the separate sensors for outdoor temperature for each indoor unit. They are the same, as temp is measured only once in the single outdoor unit, having separate sensors for each indoor unit is redundant. But I guess I get why you set it up like that, probably this is a design decision forced upon us by the Panasonic cloud.
So, maybe this issue can be closed.

The energy tracking isn't very accurate, I think the heating/cooling is just takes all energy used by the system while in heating/cooling mode... if in another mode "other" will be used.
I'll close the issue for now.

Thanks for the suggestion of disabling all but one sensor, at least that cleans up the energy dashboard.
As we know which indoor units are active, is there a way we can allocate the total consumption to the individual indoor units?

@Bouke does the indoor units show up as individual devices in HA? If so, each device should have their own energy sensor.
But if the values are identical on both, Panasonic is only reporting a combined value and it would probably be impossible to get any accurate per device reading from that; since the energy values only gets updated every 15 or so min.


Require vm UUID confirmation to delete the vm

Can one of the admins verify this patch?

Pull Request Test Coverage Report for Build 1871

19 of 19 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 53.305%

Totals

Change from base Build 1870:
0.06%

Covered Lines:
9625

Relevant Lines:
16021

💛  - Coveralls


Cramer vs. Theil

Hi, I'm searching for correlations in a very small dataset (26 samples):

When calculating the associations I'm getting very different results for the Cramer and Theil tests.
print('Cramer correlations')
cramer_plot = associations(df_cat, nom_nom_assoc='cramer',filename= 'complete_correlation.png',\
                                    figsize=(4,4))
print('Theil correlations')
theil_plot = associations(df_cat, nom_nom_assoc='theil', filename= 'theil_correlation.png', figsize=(4,4))

Is the difference in their absolute values to be expected? Which test is more valid? I would tend to use Cramer since it's built on the chi-square test which is broadly accepted but wonder if I should use Theil due to the (slight) features asymmetry. I've read the post explaining these methods and wonder what would you recommend in such cases

Hey @ronyarmon - I recommend to read the documentation about these two measures. They're different and cannot be compared.
If you ask me, I wouldn't choose Cramer's V on such a small dataset. See the method's documentation for more on this.

Thanks for your advice, I didn't see a reference to dataset size but will keep exploring


Test -- Corrected the validations.py to show output.

Closes: #1
updated validations.py python script.
Fixed the behavior of validate_user function in validations.py
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

Testing Testing


Support custom predicates for attestations

Description
The command below sets the predicateType to cosign.sigstore.dev/attestation/v1 and uses a predicate format custom to cosign.
cosign attest --predicate code-review.json --type custom ...

Would be great to have support for user specified custom predicates as discussed here: https://github.com/in-toto/attestation#custom-type-examples.

/assign @JimBugwadia


Subscriptions Text and Email Notifications

Email
We have been collecting emails when people "subscribe" but not emailing them.
We should queue an email to all subscribers for the phone number in the CheckedInViaSms::handle() method.
SMS
We should allow people to subscribe to updates from another number via SMS with SUBSCRIBE {phoneNumber}
We should queue an sms notification to go out to each subscriber for a phone number in the CheckedInViaSms::handle() method.
We also need to add SUBSCRIBE to the INFO message if/when that is working.

Email
Opinion piece
Bulk style email with the last update from any numbers they have subscribed to. Ex:
Recently Updated
--------------------
800-867-5309 - [30 min ago] I'm doing fine
202-456-1111 - [2 days ago] Got plenty of water and food, doing well

Could run on a scheduled job every X minutes. Since we offer a one->many subscription relationship this could cut down on extraneous emails for the user and your email provider.
We will need to add a notified_at column to the subscriptions table - either that or transition to proper Laravel notifications.
SMS

We should allow people to subscribe to updates from another number via SMS with SUBSCRIBE {phoneNumber}

Sounds great to me. The SMS commands are easily extensible.

We should queue an sms notification to go out to each subscriber for a phone number in the CheckedInViaSms::handle() method.

Same thought as 'bulk email' from above. Would we be better off aggregating subscriptions into singular text messages (as size allows)? Would save on transactions and hopefully increase discoverability to people with limited service.

We also need to add SUBSCRIBE to the INFO message if/when that is working.

INFO should work in the sense that it triggers the "unknown command" path - which returns a "here's the help menu" style response.
Source: TwilioWebhookController:36


Bold formatting cleared after applying italic to the same selection

Describe the bug
In markdown mode, if I apply bold formatting to a selection, then italic, the bold gets cleared. If I do it reverse (italic, then bold) it works. Also no reproducable in wysiwyg mode.
To Reproduce
Steps to reproduce the behavior:

Type any text in markdown mode
Select part or all of the text
Choose "Bold" formatting
Choose "Italic" formatting

Expected behavior
Both formatting gets applied.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Linux Mint 20
Browser Chrome
Version 99

Thank you for the issue.
We need to check the markdown specification over this issue.


Revert "Adding npm authenticate task, npm-common, and refactoring npm…

… task (#5027)"
This reverts commit d93ba1d4a41bd084dce0582dcb4f2bfd55abc6c0.

@t-jakurl,
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


Preview feature feedback: extendedIndexes

Please share your feedback about the referentialIntegrity preview feature released in v3.1.1 in this issue.

If you encounter a bug, please open a bug report in this repo.
If the feature is working well for you, please share this in a comment below or leave a :+1: on this issue.

If you have any questions, don't hesitate to ask them in the #product-feedback channel in the Prisma Slack.

Hey @janpio, after I upgrade to v3.6 from v3.2 and enable extendedIndexes, I got all my index drop and recreated every time I run prisma migrate dev. I'm on postgres 14.1.

That sounds terrible, sorry about that @leoyli-headsup. We have a kinda similar report in https://github.com/prisma/prisma/issues/10405, but there the further migrations even do not work - so this might be slightly different than what you are describing. Can you please open a new bug report issue and provide all the information it asks for? We should really understand what is going on there. Thanks!

That sounds terrible, sorry about that @leoyli-headsup. We have a kinda similar report in #10405, but there the further migrations even do not work - so this might be slightly different than what you are describing. Can you please open a new bug report issue and provide all the information it asks for? We should really understand what is going on there. Thanks!

This exactly the bug we're having right now as well. It just drops and recreates all indexes, including unique indexes, for every migration.

Please make sure to leave a comment on that issue please as well so we can connect the dots.

Saw this locally last night as well. Adding extendedIndexes and then migrate resulted in the index being dropped and recreated. Aside from that, it seemed to work as intended.

Can you create a more detailed issues @blankey1337 with the before and after state etc? We would be very interested in reproducing and hopefully fixing this.

@hugbubby @blankey1337 @leoyli-headsup Hey, is this still happening for you? The issue we initially linked to https://github.com/prisma/prisma/issues/10405 turned out to be specific for MySQL and connected to a specific configuration, so it would be weirf if that would apply to the 3 of you. Can you please take another look at that issue and open your own issue if this does not describe exactly your situation? Thanks.


CI: Enable bundler cache

Which issue(s) this PR fixes:
N/A
What this PR does / why we need it:
ref: https://github.com/ruby/setup-ruby#caching-bundle-install-automatically
Docs Changes:
N/A
Release Note:
N/A

It seems that some tests always fail on Ruby 2.7 on Linux & macOS.
They doesn't occur when we install gems by ourself.

It seems that some tests always fail on Ruby 2.7 on Linux & macOS.

===============================================================================
Failure: test: invalid values are set to RUBYOPT(TestFluentdCommand::configured to run 2 workers):
  log doesn't match.
  <"bundler: failed to load command: /home/runner/work/fluentd/fluentd/bin/fluentd (/home/runner/work/fluentd/fluentd/bin/fluentd)\n" +
  "/opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/runtime.rb:304:in `check_for_activated_spec!': You have already activated bundler 2.1.4, but your Gemfile requires bundler 2.4.13. Since bundler is a default gem, you can either remove your dependency on it or try updating to a newer version of bundler that supports bundler as a default gem. (Gem::LoadError)\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/runtime.rb:25:in `block in setup'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/spec_set.rb:165:in `each'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/spec_set.rb:165:in `each'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/runtime.rb:24:in `map'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/runtime.rb:24:in `setup'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler.rb:162:in `setup'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/setup.rb:23:in `block in <top (required)>'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/ui/shell.rb:159:in `with_level'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/ui/shell.rb:111:in `silence'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/setup.rb:23:in `<top (required)>'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/cli/exec.rb:56:in `require_relative'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/cli/exec.rb:56:in `kernel_load'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/cli/exec.rb:23:in `run'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/cli.rb:492:in `exec'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/vendor/thor/lib/thor/invocation.rb:127:in `invoke_command'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/vendor/thor/lib/thor.rb:392:in `dispatch'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/cli.rb:34:in `dispatch'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/vendor/thor/lib/thor/base.rb:485:in `start'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/cli.rb:28:in `start'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.1.4/libexec/bundle:46:in `block in <top (required)>'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/friendly_errors.rb:117:in `with_friendly_errors'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.1.4/libexec/bundle:34:in `<top (required)>'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/bin/bundle:23:in `load'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/bin/bundle:23:in `<main>'\n">
  was expected to include:
  <["Invalid option is passed to RUBYOPT"]>
/home/runner/work/fluentd/fluentd/test/command/test_fluentd.rb:209:in `assert_log_matches'
/home/runner/work/fluentd/fluentd/test/command/test_fluentd.rb:994:in `block (2 levels) in <class:TestFluentdCommand>'
     991: </match>
     992: CONF
     993:       conf_path = create_conf_file('rubyopt_invalid_test.conf', conf)
  => 994:       assert_log_matches(
     995:         create_cmdline(conf_path),
     996:         'Invalid option is passed to RUBYOPT',
     997:         env: { 'RUBYOPT' => 'a' },
===============================================================================
.........F
===============================================================================
Failure: test: without RUBYOPT(TestFluentdCommand::configured to run 2 workers):
  log doesn't match.
  <"bundler: failed to load command: /home/runner/work/fluentd/fluentd/bin/fluentd (/home/runner/work/fluentd/fluentd/bin/fluentd)\n" +
  "/opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/runtime.rb:304:in `check_for_activated_spec!': You have already activated bundler 2.1.4, but your Gemfile requires bundler 2.4.13. Since bundler is a default gem, you can either remove your dependency on it or try updating to a newer version of bundler that supports bundler as a default gem. (Gem::LoadError)\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/runtime.rb:25:in `block in setup'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/spec_set.rb:165:in `each'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/spec_set.rb:165:in `each'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/runtime.rb:24:in `map'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/runtime.rb:24:in `setup'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler.rb:162:in `setup'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/setup.rb:23:in `block in <top (required)>'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/ui/shell.rb:159:in `with_level'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/ui/shell.rb:111:in `silence'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/setup.rb:23:in `<top (required)>'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/cli/exec.rb:56:in `require_relative'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/cli/exec.rb:56:in `kernel_load'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/cli/exec.rb:23:in `run'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/cli.rb:492:in `exec'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/vendor/thor/lib/thor/invocation.rb:127:in `invoke_command'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/vendor/thor/lib/thor.rb:392:in `dispatch'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/cli.rb:34:in `dispatch'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/vendor/thor/lib/thor/base.rb:485:in `start'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/cli.rb:28:in `start'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.1.4/libexec/bundle:46:in `block in <top (required)>'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.4.13/lib/bundler/friendly_errors.rb:117:in `with_friendly_errors'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/lib/ruby/gems/2.7.0/gems/bundler-2.1.4/libexec/bundle:34:in `<top (required)>'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/bin/bundle:23:in `load'\n" +
  "\tfrom /opt/hostedtoolcache/Ruby/2.7.8/x64/bin/bundle:23:in `<main>'\n">
  was expected to include:
  <["-Eascii-8bit:ascii-8bit"]>
/home/runner/work/fluentd/fluentd/test/command/test_fluentd.rb:209:in `assert_log_matches'
/home/runner/work/fluentd/fluentd/test/command/test_fluentd.rb:977:in `block (2 levels) in <class:TestFluentdCommand>'
     974: </match>
     975: CONF
     976:       conf_path = create_conf_file('rubyopt_test.conf', conf)
  => 977:       assert_log_matches(create_cmdline(conf_path), '-Eascii-8bit:ascii-8bit')
     978:     ensure
     979:       ENV["RUBYOPT"] = saved_ruby_opt
     980:     end
===============================================================================

It seems that some tests always fail on Ruby 2.7 on Linux & macOS.

This is because the tests override RUBYOPTS.
(Although there are some succeeded tests that override RUBYOPTS, they aren't tested correctly because they expect patterns_not_match)
Although we can resolve it by adding work around, I don't think there is any merit to introduce this with such complexity.


Fixed project structure

Hello! This PR is fixing many issues in current POMing

the parent pom had to be manually installed [showstopper of previous structure]
the project was not possible to split to subprojects [showstopper of previous structure]
the parent pom was in subfolder (!!)
there were three poms for project of two packaghes and 12 files
redundant artefacts weregenerated
bad versioning

New structure

just one pom
just  one mvn clean install
allows split to subproject (as I would really like to split api and impl in future)
assembly, test, verify and isntall doen by guidelines
added exempalt test
added spotbugs
fixed versioning

In addition the package layout and name of the artifact was changed.
old:

jc as project name  is  to short and very misleading
jc as main package  is  to short and very misleading
new:
project renamed to "classpathless-compiler"
main package changed to org.terminusbrut.classpathless , which  correspond to guideline of org.minorauthor.minorproject

As I really need to move release of java-runtime-decompiler, which already is using the apis in this project, the poms and naming really had to go to shape. (just FYI: https://github.com/pmikova/java-runtime-decompiler/pull/70)


PowerShell takes a long time to load profile

Environment

Item
Value

OS, Version / Build
Microsoft Windows NT 10.0.19041.0

Processor Architecture
AMD64

Processor Type & Model
Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz, 2112 Mhz, 4 Core(s), 8 Logical Processor(s)

Memory
16.0 GB

Storage Type, free / capacity (e.g. C: SSD 128GB / 512GB)
SSD 512GB

Relevant apps installed
Windows Terminal

Description
When opening PowerShell, it takes quite some time before I get the first prompt. With Command Prompt it is nearly instant. Anything that can be done to improve load times can help all developers.

In this screenshot it took 14 seconds to load the profile.
Steps to reproduce

Use PowerShell or Windows Terminal or Visual Studio Code Terminal to open a PowerShell command prompt.

I do not have any PowerShell extensions installed.
Expected behavior

It should not take as long to open PowerShell.

Actual behavior
I have attached a process monitor log of PowerShell.exe.
LogFile.zip
It seems that there is a lot of opening and closing of registry keys. Maybe that is one area that could be investigated. There is also one thread that appears to takes a few seconds to run.

Hey @nmoinvaz Thanks for filing the issue, but could you file it over in the PowerShell repo where @joeyaiello @SteveL-MSFT can triage and address?


explain same component in multiple routes

wanted to clarify the behavior seen here: https://codesandbox.io/s/sleepy-hopper-bgntr

Thanks!

Thanks @timdorr for all your work!


Update Lisk SDK to v6.0.0-beta.0

What was the problem?
This PR resolves #105
How was it solved?

[x] Update Lisk SDK to v6.0.0-beta.0
[x] Update the banner image

How was it tested?
Local against testnet

Codecov Report

Merging #106 (8633b73) into feature/43-prepare-lisk-migrator-v2-0-0-for-beta (e7670f1) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@                                Coverage Diff                                @@
##           feature/43-prepare-lisk-migrator-v2-0-0-for-beta     #106   +/-   ##
=================================================================================
  Coverage                                             84.51%   84.51%
=================================================================================
  Files                                                    17       17
  Lines                                                   465      465
  Branches                                                 50       50
=================================================================================
  Hits                                                    393      393
  Misses                                                   72       72


adjust groupAwareClient and schemaResolver

solves #94

Note: merge to dev will fail until 0.0.5 is visible on central


dan horsley - DS-Unit-1-Sprint-3-Data-Storytelling

Your GitHub pages looks good, ready to add in some great blogs/projects. Nice start for your portfolio.
Your Tuesday assignment looks good, you did a great job data cleaning and the chart looks good, although the data is kinda clunky to chart with so much information. Maybe binning or sorting the data differently may help?

Your assignments from Wednesday and Thursday look fantastic. Your data wrangling and charting are great and your animated chart looks amazing. You are more than ready to fly through the sprint challenge this week and move onto next weeks work. Its been a joy looking through your assignments each day. You do wonderful work!


Investigate (& maybe implement) upgrade to manifest v3

Mozilla & Google have released v3 of their addon/extension manifest. It would be good to see what exactly needs doing to this repo to make it compatible on v3 for both Chrome & Firefox.
https://blog.mozilla.org/addons/2021/05/27/manifest-v3-update
Goal is to maintain cross-browser support for this extension. Currently, Checka11y.css browser extension is compatible on both Firefox & Chrome using the same code (not all extensions can do this and they have to create 2 separate repos for each browser) - we want to try and maintain this if we can so there is less code to maintain.

If this is a Chrome extension (which I think it is from the file structure with code for the popup UI etc), I'm happy to be assigned this and migrate things to MV3. I've recently been spending alot of time on developer.chrome.com studying extension development (and doing open source stuff) so most of the MV3 migration guide is still really fresh.
Nice job on breaking this out into an extension. While working on the stylesheet project I thought how useful this would be as a chrome extension so kudos!


Collection Log Completed Sound Not Playing After Changing

I have recently changed the sounds source files for many of the sounds within the plugin. Everything other than the collection log sound seems to be playing correctly.
I've taken the following troubleshooting steps:

Verified the plugin is on while receiving a collection log notification.
Verified the source file is a .wav file.
Verified the audio of the file.
Reinstalled Runelite and the C Engineer Plugin as well.
Verified chat notifications are enabled in Settings

If there are any other troubleshooting steps I should be taking, please let me know. Thanks.

Is there anything relevant in the logs? client.log in C:\Users\<username>\.runelite\logs
The plugin is on, but is the collection log specific setting within the plugin settings on?

What is the exact name of the file you are trying to use that isn't playing?

I checked the logs, and don't see anything that specifically mentions failing to load the plugin. (There is a lot of info here, so I may have overlooked something.)
I have everything in the plugin checked, and the "New collection log entry" box is checked as well.
The file name in the folder is: ColLogSlotCompleted_r1
I copied the previous file name exactly as it was prior.

Are you comfortable sharing the sound file and log file with me? I am spoody. on discord if so

Discussed on discord, I believe this was a case of an unsupported file (despite being .wav, there are nuances that sometimes java clip does not like).
For anyone finding this in the future - if you are sure your settings are correct, it's probably an issue with the sound file itself. It's not feasible for me to manually resolve invalid file issues, so for reference: I use 'ardour' to export 16 bit wav files.


awesome: Allows awesome to load svg images.

Awesome can load SVG images, just like it can do PNG, through gdk.
The support for SVG images through GDK needs librvsg.
This commits adds the plumbing necessary in the wrapper that makes
awesome be able to load SVG images.
Without this, awesome will not load SVG images. The related error message is:

Couldn't recognize the image file format for file

Which comes from gdk.

Motivation for this change
My awesome configuration uses SVG images. Awesome does support SVG images. Adding librsvg as an input and the GDK thing to the wrapper makes it work.
Things done

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

[X] NixOS
[ ] macOS
[ ] Linux

[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Thanks!


Show community-list link in context menu once a user has add/write/admin rights to a community

give community add/write rights to a group(user)
the context menu link to the community listing is not shown for the user

/xmlui/community-list shows all the communities and collections should that be displayed?
each community displays its immediate children e.g. https://lindat.mff.cuni.cz/repository/xmlui/handle/11234/3430 shows two subcommunities (and those then show collections).


Support relative URLs as an option in Remarkise, eg for images

This is a frequent issue for me.
When using images in my presentations, I would like to have the option to reference them relative to the current URL: when using Remarkise, that URL should be the one from where Remarkise has loaded the presentation, not the one where Remarkise itself is located. This will enable truly portable presentations (at least in the way I use Remark).
Sometimes I work on a presentation that sits in a GitHub branch (images included), and I can render it via github.io with Remarkise. When that gets merged, all images are broken, and URLs for images need to be updated.
(There might be already a way to do this, that I'm not aware of…?)

This should be possible by just adding the <base> html tag to remarkise.

Ah, thank you for the suggestion @bkeepers! I didn't think of that, and it'll probably work for what I need…
@gnab, can you assign this one to me, please, so that I don't forget about it?


Refactor to render SVG element with React

Resolves #214
Resolves #212
Resolves #183

Can we improve this?

1 click of a svg node triggers:
* Providers() - 1x

  * getDefaultStore() - 1x

    * setupMetricsCache() foreach "blob" (72 blobs) - 72x

      * setDominanceColor() - 72x

        * unionAuthors() - 72x
      * setDominantAuthorColor() - 72x

        * unionAuthors() - 72x

both setDominanceColor and SetDominantAuthorColor calls unionAuthors()
1 browser reload triggers 2x Providers() calls - 2 * (72 + 72) ≃ 278
1 click is sometimes a double click, triggering 4x Providers() calls - 4 * (72 + 72) = 576

Search animations are different between bubble and tree map, is this intended?

I have made this more consistent.

When changing from tree to bubble map, there is a short delay before the folder names are shown.

Issue was created for looking into this issue.

Search field seems inconsistent. Sometimes very quick and smooth, other times laggy

I have readded the debounce.


remove commercial paper references

Signed-off-by: Josh Horton<EMAIL_ADDRESS>Remove all "Commercial Paper" tutorials and references from 2.4 doc.
Type of change

New feature
Documentation update

Description
Remove commercial paper tutorial - because it uses the prior dev apps method, not the new 2.4 Fabric Gateway method.
Release Note
Should add a comment to next release notes to use the Asset Transfer tutorial for 2.4 Fabric instead of Commercial Paper.

@Mergifyio backport release-2.4


Added Basic Java Highlighting

Highlights most (all?) Java keywords and comments. Most of the boilerplate code was copied from the C++ highlighter. Keywords retrieved here.

@ouuan You might need to pull this to your QCodeEditor, It will be good to have Actual Java Syntaxhighlighting in 6.0 for us. Or maybe @tay10r could you also open this PR at github.com/ouuan/QCodeEditor

@coder3101 It looks like @ouuan has made some commits that have not been merged to upstream. I think it's best in this case for any forks to pull the changes if they want them, either from my fork or from upstream if this gets merged.

These commits add some specific features to the editor that we need in our project, which might not be needed here. If your PR was possibly showing a conflict, It would still be when he tries to pull upstream.
@ouuan will merge these changes directly because it might take more time for your PR to get merged here and even more time for us to pull upstream changes. I asked you to open a PR because there because it would have eased his job of visiting your fork to get changes!

@coder3101 I see what you're saying. If it takes more than a few days to merge this PR, I'll just start sending this and further PRs to your branch (if they're useful to what you're using it for.)
In the mean time, the feature isn't really complete anyway so you'll probably be better off waiting. There's still missing language features that haven't been added, such as annotation highlighting and Javadoc highlighting.

I forked and not send PR basically because:

The upstream author seldom updates.
I don't know the contribution rules, so I just ignore everything and do what I like. For example, I use the .clang-format file for CP Editor, so I don't need to change the settings when I have turned "format on save" on.

I think the numbers regex should be modified? I don't know Java much, but I think it should be different from C++17.

Sure, let's adjust this for Java:
(?<=\b|\s|^)(?i)(?:(?:(?:(?:(?:\d+(?:'\d+)*)?\.(?:\d+(?:'\d+)*)(?:e[+-]?(?:\d+(?:'\d+)*))?)|(?:(?:\d+(?:'\d+)*)\.(?:e[+-]?(?:\d+(?:'\d+)*))?)|(?:(?:\d+(?:'\d+)*)(?:e[+-]?(?:\d+(?:'\d+)*)))|(?:0x(?:[0-9a-f]+(?:'[0-9a-f]+)*)?\.(?:[0-9a-f]+(?:'[0-9a-f]+)*)(?:p[+-]?(?:\d+(?:'\d+)*)))|(?:0x(?:[0-9a-f]+(?:'[0-9a-f]+)*)\.?(?:p[+-]?(?:\d+(?:'\d+)*))))[lf]?)|(?:(?:(?:[1-9]\d*(?:'\d+)*)|(?:0[0-7]*(?:'[0-7]+)*)|(?:0x[0-9a-f]+(?:'[0-9a-f]+)*)|(?:0b[01]+(?:'[01]+)*))(?:u?l{0,2}|l{0,2}u?)))(?=\b|\s|$)

All jokes aside, there's no way I'm touching that. Java supports a very similar set of a rules for numeric literals, as described here. If there's someone who is smarter with regex who can patch this, they can go ahead and give it a go. Otherwise, they're similar enough that I wouldn't care too much.
Is it binary literals? If it is, I can't really tell towards the end what part of the pattern I can remove. Feel free to suggest something.

I just couldn't find a good reference for number literals in Java, now I can work on it.

See 11f5647502e2295bd93c48c79da0e767ec16f389

@ouuan thank you that's really helpful! I'll commit that here shortly. Thanks for the help


Different animation speed on Android and iOS

Description
Animations runs slowly on React Native iOS, but normally everywhere else like React Native Android and web.
Provide a Repro
https://github.com/Waltari10/riveAnimSpeedBug
Source .riv/.rev file
program.riv.zip
Expected behavior
Would expect the animation to work the same way on all platforms, when using same code and same .riv file.
Screenshots
https://github.com/rive-app/rive-react-native/assets/8919286/a6f3bc6a-a0ba-4432-a1f8-9a3ec94885a0
Device & Versions (please complete the following information)

Device: All iOS devices including real and simulator, most recently replicated on iPhone SE (3rd generation) 17.2 simulator
OS: iOS 17.2
NPM Version: 10.1.0

Additional context
Other .riv files seem to work fine, but this program.riv has these animation speed issues. Our designer has attempted recreating the .riv file and modifying it several times to no avail.

This was fixed by upgrading rive-runtime on iOS to 5.7.1. Just used patch-package to do it and required at least one swift code change I'm not so sure about if it's safe.


Add automation for setup with ghpai

Current papercuts that can be addressed:

[x] Syncing the description text with settings.ini
[ ] #654
[x] Updating the description for pages
[ ] Update the tutorial when we are done

We need to possibly add ghapi as a dependency.  cc: @jph00

@hamelsmu is this done? (there's already a separate issue for gh-pages)

Yes this can be closed now


WSS4J WSS-622 and  WSS-623

WSS4J WSS-622 from SAMLCallback no possible setting SAML2 SubjectConfirmation.NameID
Missing the Envelope/Header/Security/Assertion/Subject/SubjectConfirmation/NameID settings possibility.
WSS4J WSS-623 NameIdBean SPNameQualifier support
Pull request for https://issues.apache.org/jira/browse/WSS-622 issue and https://issues.apache.org/jira/browse/WSS-623

I'm merging the PR for WSS-622 only, please submit a separate PR for WSS-623.


Would it be convenient to swap out warning.js with fbjs utils?

Discussion
As Redux is a part of the reactjs community repo, would it convenient to swap out scr/utils/warning.js with the warning utility the fbjs repo provides? This might seem like unnecessary work, but it would be alongside with the structure of the  other Facebook/React libraries.
What do you think?

Thanks for the suggestion! I would prefer not to depend on fbjs so we have more control here. Redux doesn’t really any other utilities from it so I don’t see the point.


long tc program names are not truncated, causing a netlink error

when this error message reported, what is the real reason?
I can't attach this https://github.com/Sherlock-Holo/mahiro/commit/fe85104164ade7fd7fec944cfb2eae2f7efbb73e#diff-b47e6e512f3e362859296f1fca2c484e0699ca24db91fcf4813a04af127701a4R20 bpf program, however, I can attach https://github.com/Sherlock-Holo/mahiro/commit/fe85104164ade7fd7fec944cfb2eae2f7efbb73e#diff-b47e6e512f3e362859296f1fca2c484e0699ca24db91fcf4813a04af127701a4R16

oh... I change #[classifier(name = "dnat_ingress_with_redirect_route")] to #[classifier(name = "dnat_ingress_with_redirect")] and it works
that makes me more confuse...

I would guess the name is too long. Would be interesting where the ENOSPC comes from though.
Either running with strace or Aya's debug logging enabled would be helpful.

it seems the error reported by this https://github.com/aya-rs/aya/blob/58f1ecbf0089194d729327692adca6391fc24932/aya/src/sys/netlink.rs#L491

Ah yeah here's the issue. Name should probably be truncated here:
https://github.com/aya-rs/aya/blob/58f1ecbf0089194d729327692adca6391fc24932/aya/src/sys/netlink.rs#L134

Hi, I'd like to work on this issue.
As I understand the length of the name should not exceed 32 bytes (31 chars + null byte).
Is it ok to just truncate anything beyond this size ? also maybe show a warning ?

According to the kernel code, the name can be up to 256 bytes long:
https://github.com/torvalds/linux/blob/master/net/sched/cls_bpf.c#L28
#define CLS_BPF_NAME_LEN	256

And realised you have set the total length of attributes to 64 bytes:
https://github.com/aya-rs/aya/blob/main/aya/src/sys/netlink.rs#L254
#[repr(C)]
struct TcRequest {
    header: nlmsghdr,
    tc_info: tcmsg,
    attrs: [u8; 64],
}

I increased the length of the attributes and it works fine until the name reaches 256 bytes limit enforced by kernel and this error appears:
Error: netlink error while attaching ebpf program to tc

Caused by:
    Invalid argument (os error 22)

So we can conclude that the limit is enforced by Aya not the kernel.
Are there any particular reasons behind choosing 64 as the length of the attributes ?

@pooladkhay could you send a patch that increases this limit with a test? I'd be happy to guide you through writing the test.

@pooladkhay could you send a patch that increases this limit with a test? I'd be happy to guide you through writing the test.

Yeah I'd love to do that,
For the actual size, from what I saw it always requires 33 bytes for values other than name (289 bytes in total) but it just sounds like a magic number. I'll try to figure out the reason behind that 33 and if it can ever increase and will send a patch.
In the meantime I'd really appreciate it if you tell me more about the test.

Ah, sorry I missed your reply. The tests are in the integration-test directory. I think the smoke test is closest to what you're looking for, but have a look around. You can run these tests locally using cargo xtask integration-test -- <my-test-name>.

Actually, this may be the test you're looking for: https://github.com/aya-rs/aya/blob/445cb8b46318a13a94e10e11000232d4bd5b23af/test/integration-test/src/tests/load.rs#L17.

@tamird No worries,
Thank you, I'll update the comment and add the test tomorrow.


Fix ignored source attribute

Currently it isn't possible to use a custom Source with the Logos derive macro. Part of #222.

Thanks @kmicklas! Running the tests, and will come back to you when I have some time!


"mod.json cannot find accesses to the Module folder, which prevents mods from being opened."

"If you don't mind, could you explain more specifically where to place the Modules folder, because when they are inside the CheatModFramework folder, they don't work, neither the menu nor the modules.

I'm dump, nvm

You're not the first to installed the mod incorrectly, nor will you be the last.
Glad you figured it out though.


Incorrect path on Windows for input

First off, thank you for this crate. I am new to Rust and saw some different patterns to what I was using in my CLI I have been writing.
Not only that but your thoughtful use of info and debug console writing allowed me to diagnose the issue.
I tried installing the binary on Windows and using it and I received the following error:

Error: HeadlessChromeError("Method call error -32000: Cannot navigate to invalid URL")

Looking at the console output the input file seemed a bit weird:

INFO  html2pdf > Input file: file://\?\C:\Users\devon\GitHub\cv\index.html

It seems that using that with the "file://" for headless_chrome is not valid, at least headless_chrome doesn't see it that way.

I have a fix for this. Hopefully, you accept PRs?

Pulled latest from cargo and works on Windows. Closing.


[AIRFLOW-475] make the segment granularity in Druid hook configurable

Dear Airflow Maintainers,
Please accept this PR that addresses the following issues:

https://issues.apache.org/jira/browse/AIRFLOW-475

The Druid hook now has hardcoded segmentGranularity - "DAY", we need it configurable for different use cases.
@mistercrunch @aoen @plypaul

Current coverage is 65.43% (diff: 0.00%)

Merging #1771 into master will decrease coverage by 0.05%

@@             master      #1771   diff @@
==========================================
  Files           142        142
  Lines          9883       9886     +3
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           6472       6469     -3
- Misses         3411       3417     +6
  Partials          0          0

Powered by Codecov. Last update 6b9ad4b...48689aa

LGTM, let me know when you want this merged

@aoen fixed your comments, please merge this as Greg's anomaly detection depends on this, thanks!

Thanks @hongbozeng and @aoen !


Incompatible with YouTube TV

YTC is incompatible with YouTube TV.
you can test it using: https://www.youtube.com/tv/
If I start a video you get a question first whether you want to play it on your TV or not.
YTC already opens the video in another "object" and places it wrong.
Both screens are visible at the same time then and are partially on top of each other.
I'm able to "repair" it by choosing the device in the screen below
Two Screenshots showing what I'm talking about:
First it looks like this:

And a second later it looks like this:

Hopefully you can fix it ;)
Regards

It's been 2 full years and I cannot find any sign of an attempt to address this bug which isn't typical of the development effort so can I ask is there something specific making this unfixable that the rest of us can possibly assist in figuring out?  With Chromecast and the TV function of youtube becoming so widely used this is now to the point where to cast successfully and use the queue, we now have to totally disable YTC all together and I then miss all the functionality with it off.

@CollinChaffin The author has not been active for quite some time. Read "Status of YouTube Center" https://github.com/YePpHa/YouTubeCenter/wiki

Okay perhaps after 2 years of no comment we can hope for a "future surprise", LOL.  Last year there were still updates and I just figured since other Chromecast issues had been addressed in the past since it appeared the primary author had a Casting device that we'd luck out and at least get pointed in the direction of the bug so I don't have to start from scratch in digging through the code to try to fix it.
Thanks for pointing me to that line in the Wiki though, not sure how I missed it since I check it often these days!