Disable log files

Ref: https://pm2.keymetrics.io/docs/usage/log-management/#disable-logging

coverage: 88.472% (+0.08%) from 88.394% when pulling 0e5837b6e07767833723d9d8f58fafa9a45305e3 on disable-log-files into 045d786f6f1fba11e43c843c7dc010e433a4c97c on master.

Tested on staging:

uploading original file with replaceMedia --> OK
delete files on GCS and call replaceMedia --> can upload image successfully


Validation loss logger

This is long overdue.
We have the iso.loggers being called in the iso run! functions.
We should have a
struct ValidationLogger{T}
  logevery::Int
  loss::Vector{Float64}
  iter::Vector{Int}
  validationdata::T
end

which computes the loss on validationdata every logevery calls and stores it, together with the length of the current iso.losses (to align them in plots).

implemented in 7ece9d1a183cae7202cdaafb26644ce3183dc9e7


Show a screen when a notification arrives like WhatsApp incoming call feature

Use case
I was developing an application which has video calling feature, everything is working perfectly in flutter except when it comes to handling a notification (which I want to act like WhatsApp's Incoming Call Feature) when the app is terminated. Basically I want to show a "Ringing Screen" when a notification for call arrives and automatically show this screen and require user interaction on it rather than on the notification banner. It works fine when the App is open and navigates to the "Ringing Screen" as required but I have no way to handle it in flutter when the app is terminated. Obviously we can achieve it through writing platform specific codes but then that just doesn't fit right with me as it not only would make the coding part longer but also because I want to handle it though Flutter.
Over the past week I've searched through the internet and found similar problems that required to open an alert screen when a specific high priority notification (notification for call in my case) arrives when the app is terminated but didn't find a single universal and easy solution. I was hoping if there could be a library in flutter that would let developers to easily handle push notifications in however way we want even when the app is terminated.
Yes I've seen Flutter Local Notification and other background process plugins/packages but they just don't fit this use case. Neither is there any other single package that refers to this problem providing a solution for both Android and IOS.
Proposal
I want Flutter to have a package that would allow to handle push notification in a way that we can define our own UI (banner, alert dialog or a whole screen) for a specific notification.
Like for example no matter which state (terminated, open or in background) the app is, when an "on call" notification arrives, a "Ringing Screen" should open and ask for user to take an action (similar to an alert dialog but a screen) - the whole mechanism working like the WhatsApp's Incoming Call Feature.
Similarly for more flexibility if the UI element is user defined then developers can easily define their own notification widgets for both Android and IOS (i.e. a ringing screen for Android and a ringing banner for IOS etc.)

This isn't something we would build a first-party package for, as the use case is highly specific. We encourage any interested community members to collaborate on a third-party package.


feat: scroll fix firefox

Smooth scrolling will be disabled in firefox for internal pages and it is an issue with the nextJS link redirect

Chrome
https://user-images.githubusercontent.com/22061815/216587493-bcefd828-4553-4faf-b97d-cd353b2761e1.mov
Firefox
https://user-images.githubusercontent.com/22061815/216588082-1846855d-f1d6-4fab-a4fa-fe7a7f41fd63.mov

Why are we only applying this on Firefox? It's not desirable to have that scroll on internal pages anywhere


Support for giant graphs

Is there any builtin method that helps do graph convolution on huge graphs? By huge graph I mean that a single GPU would result in a Cuda out of memory error. Any help will be appreciated.

Sure, we provide the NeighborSampler API. You can check the reddit.py script for an example.

@sbagchi12, How many nodes / edges does your graph contain ?

@rusty1s Is there a method for using the NeighborSampler while also using a dataset of multiple very large graphs, that are stored individually on disk?

I believe you could just create multiple NeighborSamplers for each graph individually, and iterate over them in your training loop. Would this work for your case?

Probably not. I do have around 600 graphs, that range in size between 2 and 60.000 nodes and between 1 and 450.00 edges. And my thought was to sample smaller sub graphs out of the bigger ones.

I see. Sadly, this is currently not supported. Eventually, you could pre-save generated subgraphs using NeighborSampler and load them on the fly from disk?

Sounds like a good idea. i'll try this.
Does the size=[25, 10] from the reddit example indicate a range for the size?

size=[25,10] means that we sample at most 25 neighbors in the first layer, and 10 neighbors for each of the 25 neighbors in the second layer.


Managed shutdown

Managed shutdown for client and server using wait groups. Ensure that the udp and tcp server threads do not block in the main thread.

Closed in favor of the threadgroups PR


Provide powershell examples

Current examples tab only shows Portal, Azure Cli,, do not have Powershell examples with use of data share, used in many common scenarios.
Bicep is also starting to be surfaced in examples.
Found examples here.
https://docs.microsoft.com/en-us/azure/data-share/samples-powershell

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

ID: 682a68d1-fef3-02e6-be8c-f31c5be543d3
Version Independent ID: 50f7d2b2-5e6e-a4ae-6c34-84aa9086db43
Content: Tutorial: Accept & receive data - Azure Data Share
Content Source: articles/data-share/subscribe-to-data-share.md
Service: data-share
GitHub Login: @jifems
Microsoft Alias: jife

HI @asears
Thanks for the feedback! We have assigned the issue to the content author to further review this and update the document as appropriate.

@whhender can you please see if this is covered in any of the upcoming docs, or if the existing page should be altered to add PowerShell examples?

Yes! We are working on getting this added to the current documentation within these next weeks. I'll update here once they are included.
All the commands needed are included in the PowerShell reference (https://docs.microsoft.com/en-us/powershell/module/az.datashare/?view=azps-6.6.0) however, we should shortly have them added as steps to the Tutorial page.

@asears - Apologies for the late reply, but we were able to add this tutorial in November.
You can now find PowerShell instructions on the same page.
https://docs.microsoft.com/azure/data-share/share-your-data?tabs=powershell#create-a-data-share-account

#please-close


[cirqflow] Add target_gateset to QuantumRuntimeConfiguration

Optionally run gate compilation on executables before running them.
I took the opportunity to flesh out the tests for QuantumRuntimeConfiguration as well. It's learning more features.

I should say I found the default behavior of cirq.optimize_for_target_gateset when passed None as the target gateset surprising. I think it defers to using each gate's preferred target gateset via decompose() rather than doing nothing or throwing an error.
The behavior here is different: If target_gateset is None, it will not monkey with your circuit.

Otherwise, I was absolutely delighted how easy it was to plug in the wonderfully designed transformers api!

I should say I found the default behavior of cirq.optimize_for_target_gateset when passed None as the target gateset surprising. I think it defers to using each gate's preferred target gateset via decompose() rather than doing nothing or throwing an error.

Just FYI,
By default, if specified as None, the method defaults to using cirq.decompose on all operations, which should decompose (most of) them to Cirq's [default target gateset] (https://github.com/quantumlib/Cirq/blob/fc9a204ffb36a3a1f0dd89288db62d0fbaa00beb/cirq-core/cirq/protocols/decompose_protocol.py#L50).  We also check that all cirq.decompose(op) ends at the default gateset as part of the consistent protocols test suite
But in hindsight, I agree that a good alternative could also be to give a sensible default (like CZTargetGateset) to the gateset parameter of optimize_for_target_gateset method and then either ignore or raise an error if gateset is None. Maybe we can still do this change before Cirq 1.0.

default gateset as part of the consistent protocols test suite

I must have missed the part that we actually have a gateset we target for decompose. That makes things much nicer!
I think for the cg.workflow tools we need a way to turn it off. I don't think the different None behavior between the QuantumRuntimeConfiguration object and optimize_for_target_gateset object will be too confusing, but worth pointing out.


Build HUD pages using JSON

Some progress has already been made on this, in large part thanks to @cooolbros
Just need to polish the output, make it fit all necessary controls and still look nice.

Added in https://github.com/CriticalFlaw/TF2HUD.Editor/commit/7c9f1423ff7b6bba7d360617b0eddfe0d9c57055


No implementation found for method disposeAllPlayers

On debian.  I installed mpv, doesn't work.
ERROR - 2023-10-22 16:01:27.034803
Asynchronous error
MissingPluginException(No implementation found for method disposeAllPlayers on channel com.ryanheise.just_audio.methods)
package:just_audio_platform_interface/method_channel_just_audio.dart 29  MethodChannelJustAudio.disposeAllPlayers
ERROR - 2023-10-22 16:01:27.034974
Asynchronous error
MissingPluginException(No implementation found for method init on channel com.ryanheise.just_audio.methods)
package:just_audio/just_audio.dart 1341  AudioPlayer._setPlatformActive.setPlatform

nevermind


Return highlighted excerpt of result

Addresses issue #100.
If configuration is set to
:using => {
  :tsearch => { :highlight => true }
}

then queries will return records with an additional pg_highlight attribute.
Currently uses the defaults set by the Postgres function ts_headline, as specified here: http://www.postgresql.org/docs/current/static/textsearch-controls.html#TEXTSEARCH-HEADLINE
Also supports custom delimiters
:using => {
  :tsearch => {
    :highlight => {
      :start_sel => "begin",
      :stop_sel => "end"
    }
  }
}

will override the default <b> and </b> query delimiters in the highlighted results.
NOTE: the pg function used (ts_headline()) is not supported on pg versions < 9.0. I know that the gem is intended to support 8.0 and above, but this seemed like a useful enough feature get started. Any way to work around?

So the build is failing because Rubocop wants class definitions to have <= 100 lines of code, and /lib/pg_search/features/tsearch.rb is 107. :/
I could resolve by either:

Extracting into a module

Changing the Rubocop config

Do you have a preference @nertzy?

It would be better to refactor some of the methods into service objects. For instance, tsquery_for_term is way too complicated.
But for now it would be OK with me to add a Rubocop comment that turns off the file size check for just tsearch.rb until we can get a good refactor in. Just add a line to the top of the file that turns off that rule:
# rubocop:disable Metrics/ClassLength

This is awesome!

Thanks, @nertzy. I'd rather do it right the first time, so I'll refactor them out as suggested.

Ok, so here's the deal. I cleaned up this PR, added documentation, and added a workaround to appease Rubocop.
Then, in a separate branch, I did some serious refactoring.
I didn't want to bundle all of it into this one PR, so we can do it in two phases:

If this PR looks good, then you can merge and the new feature will be available.

I'll make another PR with the refactorings so that the code is actually a little cleaner (not perfect, but better).

What do you think, @nertzy? Will that work for you?

I'm sorry that I've taken so long to address this pull request. Unfortunately, the new subquery structure in pg_search 1.0 breaks this pull request. I spent a few minutes trying to refactor what you've built to act similarly to how .with_pg_search_rank works, but to no avail.
So I'm going to close this for now. I'd gladly accept a pull request based off of this that adapts things to the new subquery, and that makes the highlight optional through something like a .with_pg_search_highlight method chained onto the scope.

Hey @nertzy no worries, I got pulled in other directions too. Sadly this is not a high priority for me anymore, so don't think I'll be able to work on the feature anytime soon.
Thanks for the helpful suggestions and feedback earlier though! Hope that #280 is able to make it through by building on this one. :smile:

@nertzy Was documentation for this feature merged into the master branch README by mistake? I keep trying to use with_pg_search_highlight on a search scope and the method always comes up as undefined.

By the way I'm referring to https://github.com/Casecommons/pg_search/blob/ff1af346322ca54808b3f53081149b8cac55bb76/README.md#highlight-postgresql-90-and-newer-only
The recommended configuration causes the following error:
ArgumentError: Unknown key: :start_sel.

It's not an error. It's merged to master but not yet in a released gem version, I believe. I'll try to get a new version cut very soon, sorry for the confusion!

Oh, fantastic. Thank you.

There are still error in documentation (the sample content is:
:using => {
  :tsearch => {
      :start_sel => "begin",
      :stop_sel => "end"
  }
}

but not
:using => {
  :tsearch => {
    :highlight => {
      :start_sel => "begin",
      :stop_sel => "end"
    }
  }
}

)
Also I found that there is error when trying to use with_pg_search_highlight method in combination with associated_against, the error description is something like
table "pg_search_3be97ef13e109a88624534" is absent in FROM-clause

@sintro I fixed the documentation bug in #312


Is the project dead?

Doesn't seem to be any activity in the last months

I too am interested in this project


update transit coverage

updating coverage documentation to reflect current transit coverage

Docs Build status updates of commit 8aedd44:
:warning: Validation status: warnings

File
Status
Preview URL
Details

BingMaps/coverage/transit-coverage/africa.md
:warning:Warning
View
Details

BingMaps/coverage/transit-coverage/asia-australia-pacific-islands.md
:warning:Warning
View
Details

BingMaps/coverage/transit-coverage/europe.md
:warning:Warning
View
Details

BingMaps/coverage/transit-coverage/index.md
:warning:Warning
View
Details

BingMaps/coverage/transit-coverage/north-america.md
:warning:Warning
View
Details

BingMaps/coverage/transit-coverage/africa.md

Line 14, Column 13: [Warning-ms-service-subservice-invalid] Invalid value for 'ms.service': 'bing-maps'.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

BingMaps/coverage/transit-coverage/asia-australia-pacific-islands.md

Line 14, Column 13: [Warning-ms-service-subservice-invalid] Invalid value for 'ms.service': 'bing-maps'.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

BingMaps/coverage/transit-coverage/europe.md

Line 14, Column 13: [Warning-ms-service-subservice-invalid] Invalid value for 'ms.service': 'bing-maps'.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

BingMaps/coverage/transit-coverage/index.md

Line 14, Column 13: [Warning-ms-service-subservice-invalid] Invalid value for 'ms.service': 'bing-maps'.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

BingMaps/coverage/transit-coverage/north-america.md

Line 14, Column 13: [Warning-ms-service-subservice-invalid] Invalid value for 'ms.service': 'bing-maps'.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Generating mipmaps for ImageTexture3D sometimes fails

Tested versions
v4.2.stable.official [46dc27791]
System information
Windows 10 - Mobile
Issue description
I'm trying to generate mipmaps for an ImageTexture3D.  I tried to create my code to match what is happening in the Godot engine source code.  Sometimes it works, but sometimes I get an error saying that the data buffer generated is 4 bytes too long.  The trouble is, I have no control over the size of the buffer.  I just pass in an Array[Image] of the mipmaps an Godot does the rest.  The 4 bytes could corespond to the topmost 1x1x1 image, but if I leave that out I get an error saying I've submitted too few mipmaps.
Steps to reproduce
Run the attached project.  The first image size (which is commented out) will build a texture successfully and the second will not.
Minimal reproduction project (MRP)
image_texture3d_mipmaps.zip

There seems to be some issues with Godot 3D texture mipmap calculations when certain image sizes are used. Possibly related issue: https://github.com/godotengine/godot/issues/56066, but in this MRP the 3D image sizes are pretty reasonable and some similar sizes seem to work.


Why are some extensions under ASPNET whene they dont need to be?

Eg, Microsoft.Extensions.Configuration, DependencyInjection, Logging, etc need to be used by all project types, not just ASPNET.

These will be moved under the new dotnet/runtime monorepo. See this announcement.

ok, thanks

Closing as answered.


Using Set to assert uniqness of iterabile's elements

URL(s)
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Set
Request type

[X] New documentation
[ ] Correction or update

Details
If You want to console.assert that all ids in a document are unique You can querySelectorAll elements that have an id, map to an Array and then compare the length of the  Array with the size of a new Set from it.
It could be a good use case example.

@mattiapontonio great idea. Keep them coming!

Editing done: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Set

@mattiapontonio thanks a lot.


(7zip.install) improved and added shim

Simplified installer
Added shim (closes #549)
Improved nuspec
Removed 0 prefix on version

Notes:

I removed helper because that check seems unnecessary - explorer is either running or not.
This package can now be used as 7zip dependency instead of 7zip.portable

Only thing I ought to mention is that this one isn't a direct replacement of the 7zip.portable when it comes to shims.
I believe the 7zip.portable have a few other binaries it shims on installation.

Yes,, but it is not something of importance in general case. I looked for the 7z vs 7za difference and it seems there is none functional level (different linking).


[Feature Request] Custom Dictionary Delimeter to reduce the text information on context/suggestion menu

Hello @tadashi-aikawa !!!!!!!
My FR is about a new delimeter in the Custom Dictionary to hide/reduce the amount of text showed in the suggestion menu (context menu)
My Workflow
I have some tokens that I use to store some Lorem Ipsum text generator from  https://www.lipsum.com
See below 3 of then:
Lorem Ipsum is simply dummy text of the printing and typesetting industry. Lorem Ipsum has been the industry's standard dummy text ever since the 1500s, when an unknown printer took a galley of type and scrambled it to make a type specimen book. It has survived not only five centuries, but also the leap into electronic typesetting, remaining essentially unchanged. It was popularised in the 1960s with the release of Letraset sheets containing Lorem Ipsum passages, and more recently with desktop publishing software like Aldus PageMaker including versions of Lorem Ipsum.<Lor_G><loi>
Neque porro quisquam est qui dolorem ipsum quia dolor sit amet, consectetur, adipisci velit <Lor_M><loi>
Lorem Ipsum is simply dummy text <Lor_S><loi>
It's useful for me to have this because I'm always trying some CSS snippets and I wanna see/tweak how it looks on different amount of texts
Another usage is from users who needs to write documentation and have a a bunch of definitions of concepts like mean ,standard deviation ... and can create tokens with a definitions to avoid having to write the definition for every new documentation :)
My FR
A new delimeter that could be placed in the note to limit the amount of characters that will be showed in the suggestion menu. Following my example, something like this:
Delimeter= <STOP>
Lorem Ipsum <STOP> is simply dummy text of the printing and typesetting industry. Lorem Ipsum has been the industry's standard dummy text ever since the 1500s, when an unknown printer took a galley of type and scrambled it to make a type specimen book. It has survived not only five centuries, but also the leap into electronic typesetting, remaining essentially unchanged. It was popularised in the 1960s with the release of Letraset sheets containing Lorem Ipsum passages, and more recently with desktop publishing software like Aldus PageMaker including versions of Lorem Ipsum.<Lor_G><loi>
Neque porro<STOP>quisquam est qui dolorem ipsum quia dolor sit amet, consectetur, adipisci velit <Lor_M><loi>
Lorem<STOP> Ipsum is simply dummy text <Lor_S><loi>

Let me know if you need more information :)
Thanks for reading this :)
Have a great day!

Hi, @FelipeRearden. Thank you for your FR!
I understood and thought I needed it 👍

@FelipeRearden
I have updated beta version to obsidian-various-complements-plugin-beta 🚀
Please confirm whether it works as you expected by using BRAT.

I have updated beta version to obsidian-various-complements-plugin-beta 🚀

Hello @tadashi-aikawa !!!!!!!!!
Thank you very very very much !!!!!!!!!!
It's working perfect !!!!!
I loved that you created a text field in the Settings for us to customize the delimeter !!!! It's AMAZING
Working perfect on version 3.3.0.b2 Obsidian 0.12.19
I wish you a fantastic day!!!!!

Released as v3.3.0 🚀

Released as v3.3.0 🚀

Working perfect on version v3.3.0 Obsidian 0.12.19
Thanks @tadashi-aikawa !!!!!!!


Chef Client fails to download when running knife ec2 server create

The command I ran:
knife ec2 server create -x Administrator --node-name WindowsTest2 --identity-file C:\Users\chef.ssh\id_rsa --ssh-key chef_demo_2x --region us-west-2 -g sg-aff696d4 --subnet subnet-31f9016a -I ami-2ad04f4a --flavor m3.medium --tags Name="WindowsTest2" --run-list recipe['chef-client::task'],recipe['fourthcoffee']
The error message output:
<IP_ADDRESS> Attempting to download client package using PowerShell if available...
<IP_ADDRESS> powershell.exe -ExecutionPolicy Unrestricted -InputFormat None -NoProfile -NonInteractive -File  C:\chef\wget.ps1 "https://www.chef.io/chef/download?p=windows&pv=2012&m=x86_64&DownloadContext=PowerShell&v=12" "C:\Users\ADMINI~1\AppData\Local\Temp\chef-client-latest.msi"
<IP_ADDRESS> Exception calling "DownloadFile" with "2" argument(s): "Unable to connect to
<IP_ADDRESS> the remote server"
<IP_ADDRESS> At C:\chef\wget.ps1:14 char:1
<IP_ADDRESS> + $webClient.DownloadFile($remoteUrl, $localPath);
<IP_ADDRESS> + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
<IP_ADDRESS>     + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
<IP_ADDRESS>     + FullyQualifiedErrorId : WebException
<IP_ADDRESS>
<IP_ADDRESS> Failed download: download completed, but downloaded file not found
<IP_ADDRESS> Warning: Failed to download "https://www.chef.io/chef/download?p=windows&pv=2012&m=x86_64&DownloadContext=PowerShell&v=12" to "C:\Users\ADMINI~1\AppData\Local\Temp\chef-client-latest.msi"
<IP_ADDRESS> Warning: Retrying download with cscript ...
<IP_ADDRESS> C:\chef\wget.vbs(47, 1) msxml3.dll: The operation timed out
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS> Failed download: download completed, but downloaded file not found
<IP_ADDRESS> Exiting without bootstrapping due to download failure.
ERROR: Failed to execute command on <IP_ADDRESS> return code 1
ERROR: Bootstrap command returned 1

This was resolved by adding --associate-public-ip to the command.  We might want to mention that in the README to warn future users.

@scarolan I'm not sure what your environment is such that you needed --associate-public-ip. From the logs, it looks like your node had a private address and you were able to connect to it (<IP_ADDRESS>), but it did not have internet access. Adding a public IP could provide that, but adding an internet gateway to the VPC would have worked too.
Maybe we could add an error message like "Verify that you have public internet access from this node" when we fail to download. The "download completed" portion of the existing message is a bit confusing.


vs code IntelliSense have a problem

How to close this prompt，Intelligent Tips for JS Files doesn't work

Folding code is no longer available

/needsMoreInfo

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

How to close this prompt，Intelligent Tips for JS Files doesn't work
image
Folding code is no longer available
319936A9-23AC-4b10-8CFB-E6B66ED9C554

Please follow the issue reporting guidelines to provide steps to reproduce the issue and basic information about the project you are opening

Closing as more information is needed about the what the issue issue is and how to reproduce it


How do I use parceler with an extended class?

@Parcel class BaseClass {}
class ExtendedClass extends BaseClass {}

I would like to wrap ExtendedClass and unwrap it later, but upon unwrapping I got the following error: org.parceler.ParcelerRuntimeException: Unable to find generated Parcelable class
Note that I want it unwrapped as ExtendedClass and not BaseClass. This is how I do the wrapping.
ExtendedClass extendedClass = new ExtendedClass();
Parcels.wrap(ExtendedClass.class, extendedClass));

Is there an easy way to fix this or do I have to implement ParcelConverter?

Turns out I had to add @Parcel to the extended class as well.


chore: bump msw in packages

Signed-off-by: blam<EMAIL_ADDRESS>Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

Going to close this for now and keep a close eye on https://github.com/mswjs/msw/issues/1218


Updated Readme for TDX Release

Updated the Readme based on https://salesforce.quip.com/O2JgAJTD8XIl
What does this PR do?
Updates the Readme to the new version
What issues does this PR fix or reference?
https://salesforce.quip.com/O2JgAJTD8XIl

This all looks great to me!


fix: dropdown item color

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)

Closes #109
Screenshots

Before
After

@huyenltnguyen please deploy this when you get a moment.


added proper country code to phone

Ready for review

@coolwr @csapinoso I have updated this PR.

This PR does not introduce any change to the master branch and should just be discarded


Fixed cypress env for PostgreSQL connection

Summary
After moving from MySQL to PostgreSQL e2e cypress tests are crashing.

Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/15766

Hello @shtelzerartem,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.


pull request for e2e test at 01/28/2017 10:56:14

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


provided out is the wrong size for the reduction

I am trying to run the infercnv function on an anndata object and am receiving an error message: "provided out is the wrong size for the reduction" --

Looks like it is fixed by converting obs_col to numpy array with obs_col.values.


[Bug] Special facilities cannot be reenabled

As the title suggests, once you disable a special facility and close it, it cannot be reenabled ever again. Left click won't open the facility, right click won't open the context menu to edit it or delete it, toggling the sheet's edit state won't work either. The special facility is just stuck there for eternity in its disabled state.
I tested it with zero modules in a new world and I saw no errors in the console. It is as if it is not even a clickable object anymore.

You have to advance a bastion turn, which will cause the facility to be repaired and re-enabled automatically.

Should it be impossible to even delete the bastion facility (perhaps due to destruction by the Ruin card, or some story reason) in that case? Disabling use the facility's activities certainly makes sense, but making it impossible to edit or even delete, when no other kinds of items work this way, feels a little odd.


Raylib.LoadShader(sbyte* vsFileName, sbyte* fsFileName);

Issue description
Hello,
i wanted to use the Raylib.LoadShader(sbyte vsFileName, sbyte* fsFileName) fuction but I don´t know how to pass the sbyste*

You can use the Raylib.LoadShader wrapper function to pass in strings instead which handles that for you. What data are you trying to pass into it?

I´m trying to pass the path to raymarching.fs

Ok I found it. Thanks


If I apply a class to datepicker it stops working

I can add this to my template:
{{jqui-datepicker value=date}}
It works fine, but I'm using twitter bootstrap for styling, so I want to apply a couple of classes to the input, if I change the previous snippet to this:
{{jqui-datepicker value=date class="form-control input-lg"}}
the styles are applied, but the datepicker doesn't work anymore.

Tried the following:
ember new example-app
cd example-app
ember install ember-cli-jquery-ui
ember install ember-bootstrap
Then I appended
{{jqui-datepicker value=date class="form-control input-lg"}} {{date}}

to app/templates/application.js
Worked for me.


chore: fix probes and culling

I still have to add some tests for this.
But I think that the following is the way forward in terms of culling:

do not worry about the active kernels - the cpu usage is a good and more general proxy for this
use a max age culling threshold on top of the idle - set in the JupyterServer resource - if set to 0 then the session will not be culled because it is old
use the /api/status endpoints for probes which does not reset the activity

See here for failing tests when the endpoint for the probes is changed to something that does not exist.
https://github.com/SwissDataScienceCenter/amalthea/pull/128
Here are the changes between the this PR (with working probes) and the one where the probes don't work.
https://github.com/SwissDataScienceCenter/amalthea/compare/fix-culling-and-probe-endpoint...test-culling-tests-failing

Also the run-tests workflow does not exist anymore. I refactored the code from that into 2 separate integration tests that show up in the list above.

@ableuler this is ready for review.

To not hold things up I have moved the code changes for testing in a separate branch and will make those in a separate PR. We were using two operators by doing kopf run ... in the tests and also by installing the full Amalthea helm chart. So basically launching sessions was done by the kopf run ... operator that was a test fixture and the culling by the operator in the helm chart. But weird errors would crop up when the fixture tries to clean up after itself but the other operator is still active too.

@ableuler @pameladelgado this is ready for a final approval. I rolled back the changes in the tests. So here we just have the updates to the probes and culling and the tests are the same as before.
@ableuler when I ran the tests just now they kept crashing on a key error for this: https://github.com/SwissDataScienceCenter/amalthea/pull/127/files#diff-69cb09e867444dc54b48f970c6707f25e2e8f07d14a13fa3286a4ab03da92e64R46. So I decided to deal with the exception there.


Remove fastai as a dependency

Polvo will be restructured into many framework-specific libraries such as: fastpolvo, huggingpolvo.
The polvo library will be a common base and should not depend on fastai, though, it's okay to depend on fastcore.

[ ] classification.baseline.baseline

closed in #d485da03256e5ab69cd59388cd0bb2e6386101a2


How to extend Nuxt I18n translations?

Description
I recently updated to the v3, which now uses Nuxt I18n for its own component translations. Since this update, my existing i18n configuration no longer works as expected. It seems the framework’s new i18n setup conflicts with or overrides my own translations, and I’m unable to figure out how to properly extend its default localization to include my own.
Could you please provide guidance on how to configure or extend the new i18n setup so that my existing translations remain functional?

Have you checked https://ui3.nuxt.dev/getting-started/i18n/nuxt?

Have you checked https://ui3.nuxt.dev/getting-started/i18n/nuxt?

Yes. I had a bit of trouble understanding how to "extend" a language to add new messages or modify an existing message.
@benjamincanac, another thing I had difficulty with is related to this section described in the documentation.
<script setup lang="ts">
import * as locales from '@nuxt/ui/locale'

const { locale } = useI18n()

const lang = computed(() => locales[locale.value].code)
const dir = computed(() => locales[locale.value].dir)

useHead({
  htmlAttrs: {
    lang,
    dir
  }
})
</script>

<template>
  <UApp :locale="locales[locale]">
    <NuxtPage />
  </UApp>
</template>

This doesn't work with languages that contain the country in the code, as the translation file is non-standard, pt_br instead of pt-BR.

To get around this, I'm using the code below, but I don't know if it's recommended.
<script setup>
import * as locales from '@nuxt/ui/locale'

const { locale } = useI18n()

const currentLocale = computed(() =>
  Object.values(locales).find(item => item.code === locale.value)
)
</script>

<template>
  <UApp :locale="currentLocale">
    <NuxtPage />
  </UApp>
</template>


[Feature Request] Request to support proxy auth for azure-sdk-for-cpp

Hi Azure team,
We are need to using proxy auth (username and password). We found your SDK is not support yet.
Could you support this feature for Curl transport?
https://github.com/Azure/azure-sdk-for-cpp/blob/main/sdk/core/azure-core/inc/azure/core/http/curl_transport.hpp
https://github.com/Azure/azure-sdk-for-cpp/blob/main/sdk/core/azure-core/src/http/curl/curl.cpp
https://curl.se/libcurl/c/CURLOPT_PROXYAUTH.html
In addition, we also submit a ticket to azure support portal (Task Number:<PHONE_NUMBER>002220) and they suggest us to creating a issue to this Git repo.
Circle Chen
Trend Micro

Thank you for submitting this request.  We will look into adding this support in a future release.

Hello @anyoiuo, thank you for reaching out.
Have you tried using proxy environment variables to set the full proxy configuration?
Something like
export http_proxy=http://USERNAME:PASSWORD@SERVER:PORT/

See this
Libcurl should honor the env vars as mentioned here: https://curl.se/libcurl/c/CURLOPT_PROXY.html

The options can be easily added to the libcul transport adapter options. However, there's not currently an available strategy for testing that it would work as expected ( No proxy with auth within CI or test env ).  Coverage or correctness support might be missing.  I need to talk to the team about this to see alternatives.
In the mind time, if using env vars for the proxy won't work for you, you can also give it a try to implement a custom transport adapter where you can take control of the libcurl handle to set any configuration to it before performing the request.  See an example of it here: https://github.com/vhvb1989/libcurl-callbacks-transport-adapter
Will be back to you about implementing this.

We already added this options and modified some code. Currently, it is work properly.
We are waiting to the official support for it.

Hello @anyoiuo
We’re very glad that you are unblocked and are successful with the proxy support you added yourself.
At this time, we have higher priorities and there is no timeline for when official support will be introduced.

/unresolve

FWIW, this work item is included in the /feature/websockets.
We're currently vetting the other work in this branch with partners and hope to have this fully merged with main sometime this fall (3Q 2022).


Added the possibility to override the webservice location

Allows one to load the WSDL from one place and to query a server that is different from the one specified in the WSDL

Awesome, the feature looks great and is very much appreciated! :+1:
I understand this could be a bit cumbersome, but have you tried looking into adding a test? :-)
I also like the immutable interface. Should we take #5 into account here?

I will add the test now while working on another feature I need to make your library work in my project, don't worry.
I don't understand what you mean by "take #5" into account, though. My commit doesn't address this. If there is something I can do to help, let me know !
To satisfy the needs of our customer, I need to be able to get the location in the WSDL, append a URL parameter to it and use the overrideLocation() method to send queries to that new location. SoapClient makes it impossible to retrieve the location stored in the WSDL. To get it, I'd have to resort to using a hack. I have several solutions, but they're all hackish. The best I found is this one :

The client constructor calls seekWSDLLocationMode() on ClientEncoder which sets some flag to true.
When the flag is set to true, the __doRequest method takes the location, stores it inside the ClientEncoder and sets the flag to false.
In the client constructor, we send a fake request to make the encoder set its 'WSDLLocation' variable.
We put a method on the ClientEncoder AND the client to retrieve the location, something like getWSDLLocation().

I need that feature. I know it's hackish. As far as I know, the alternatives are no better. My questions are :

Do you know of any better way ?
If I implement this, will you accept to merge my work on your repo ? I would understand if you didn't, but it would help me a great deal, while providing your users more features.

I've pushed tests. Not optimal, but they work !

I will add the test now while working on another feature I need to make your library work in my project, don't worry.

Awesome! :-)

I don't understand what you mean by "take #5" into account, though. My commit doesn't address this.

Indeed, this isn't directly related. I was more referring to how PHP's normal SoapClient API works. Eventually, we should also support non-WSDL-mode and hence offer a way to specify the location and uri attributes. This is why this might be related to this ticket.

SoapClient makes it impossible to retrieve the location stored in the WSDL. To get it, I'd have to resort to using a hack. I have several solutions, but they're all hackish. The best I found is this one […]
If I implement this, will you accept to merge my work on your repo ?

Now this is a very interesting hack :-D
In fact, I consider most of this library to be a huge hack, so I'm not opposed to getting something like this in at all :-)

As far as I know, the alternatives are no better. […] Do you know of any better way ?

Have you looked into using the __setLocation() method already? Untested pseudo code ahead:
public function getLocation()
{
    $ret = $this->soap->__setLocation();
    $this->soap->__setLocation($ret);

    return $ret;
}

Well done sir. It seems to work. I should be reading docs more !

Indeed, this isn't directly related. I was more referring to how PHP's normal SoapClient API works. Eventually, we should also support non-WSDL-mode and hence offer a way to specify the location and  uri attributes. This is why this might be related to this ticket.

OK, so my work here must be thought with the future in mind. If I rename withOverridenTarget to withTarget, which I will do in a minute, you'll be able to offer the same interface to your users when #5 is implemented, while adding more functions later on. Is it OK for you ?

Meh. What are we supposed to do about those weird Travis errors ? Why am I considered responsible for a Travis segfault ? Am I missing something ?

It seems to work […]
UPDATE : No, it doesn't. It returns what you set during the previous setLocation call :(

Indeed :) Initially, this ticket was about only overriding the location setting. This is now about accessing the location setting, so I'd rather like to keep this separate. See https://github.com/clue/php-soap-react/pull/13 for more details.

rename withOverridenTarget to withTarget […]. Is it OK for you ?

Looks good so far :) May I ask you to remove the other changes related to accessing the location so that we can ease the review? Thanks!

I've read all your comments. I'll make the changes them

Looks good so far :) May I ask you to remove the other changes related to accessing the location so > that we can ease the review? Thanks!

Done !
(Tests fails for the same unknown reason as for PR #14)

The changes LGTM and I'd love to get this in :shipit: Can you update the PR / rebase on current master? :+1:

Done!

Is there a timeline for v0.3.0? I need this feature. :-)

This currently has a merge conflict and there are currently no immediate plans to build this from my end (no demand at the moment and more important outstanding issues currently), but I would be really happy to accept any help with this :+1:
(If you need this for a commercial project and you want a quote, please check out my profile and send me an email) :-)

@floriansimon1 Thank you for your patience and thank you for working on this!
I've recently started working on this project again and I agree that it makes sense to have this feature. I see that @pascal-hofmann has picked up your work and added some tests for this in #23, so I will close this PR for now and I will get back to the other PR as soon as time permits :+1:


stockfish: 16.1 -> 17

Description of changes
Upgrades stockfish to latest version 17.

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

Add a :+1: reaction to pull requests you find important.

Let's go, that's a straightforward upgrade :+1:

Result of nixpkgs-review pr 340473 run on aarch64-darwin 1

2 packages built:

stockfish
uchess

Stockfish and uchess work, thanks!


ValidateMailEnabledPublicFolders: Public folders have proxy GUIDs even though the folders are mail-disabled

Provide Version Number
Version <IP_ADDRESS>0
Describe the issue
The script has run and reported some issues I have fixed, but now I have new issues that I cannot resolve.
2 folders should be mail-disabled, either because the MailRecipientGuid does not exist, or because they are system folders.
24 MailPublicFolders are orphans and should be deleted. They exist in Active Directory but are not linked to any public folder.

I cleaned this up with the recommended commands.
Than I ran the ValidateMailEnabledPublicFolders again and now it says:
4 public folders have proxy GUIDs even though the folders are mail-disabled.
These folders should be mail-enabled. They can be mail-disabled again afterwards if desired.

I checked the folders and these are
Public Folders\Internet Newsgroups
System Folders\OFFLINE ADDRESS BOOK\/o=MYDOMAIN/cn=addrlists/cn=oabs/cn=Standard-Offlineadressliste
System Folders\OFFLINE ADDRESS BOOK\EX:/o=MYDOMAIN/ou=Erste administrative Gruppe
System Folders\SCHEDULE+ FREE BUSY\EX:/o=MYDOMAIN/ou=Erste administrative Gruppe

The recommended commands to enable these are failing:
No existing 'PublicFolder' matches the following Identity: 'Public Folders\Internet Newsgroups'. Make sure that you specified the correct 'PublicFolder' Identi
ty and that you have the necessary permissions to view 'PublicFolder'.
    + CategoryInfo          : NotSpecified: (0:Int32) [Enable-MailPublicFolder], MapiOperationException
    + FullyQualifiedErrorId : 97D1B431,Microsoft.Exchange.Management.MapiTasks.EnableMailPublicFolder

No existing 'PublicFolder' matches the following Identity: 'System Folders\OFFLINE ADDRESS BOOK\/o=MYDOMAIN/cn=addrlists/cn=oabs/cn=Standard-Offlineadressliste
'. Make sure that you specified the correct 'PublicFolder' Identity and that you have the necessary permissions to view 'PublicFolder'.
    + CategoryInfo          : NotSpecified: (0:Int32) [Enable-MailPublicFolder], MapiOperationException
    + FullyQualifiedErrorId : 6FD2DBF,Microsoft.Exchange.Management.MapiTasks.EnableMailPublicFolder

No existing 'PublicFolder' matches the following Identity: 'System Folders\OFFLINE ADDRESS BOOK\EX:/o=MYDOMAIN/ou=Erste administrative Gruppe'. Make sure that
you specified the correct 'PublicFolder' Identity and that you have the necessary permissions to view 'PublicFolder'.
    + CategoryInfo          : NotSpecified: (0:Int32) [Enable-MailPublicFolder], MapiOperationException
    + FullyQualifiedErrorId : B25901A9,Microsoft.Exchange.Management.MapiTasks.EnableMailPublicFolder

No existing 'PublicFolder' matches the following Identity: 'System Folders\SCHEDULE+ FREE BUSY\EX:/o=MYDOMAIN/ou=Erste administrative Gruppe'. Make sure that y
ou specified the correct 'PublicFolder' Identity and that you have the necessary permissions to view 'PublicFolder'.
    + CategoryInfo          : NotSpecified: (0:Int32) [Enable-MailPublicFolder], MapiOperationException
    + FullyQualifiedErrorId : EE9CA690,Microsoft.Exchange.Management.MapiTasks.EnableMailPublicFolder

I tried enabling and disabling Public Folders\Internet Newsgroups but this has not resolved the Proxy GUID issue and only bubble up one orphaned again. The system folders cannot fixed or I do not know how.
Can someone help here and explain how to resolve the proxy GUID issue, please?
Expected behavior
No issues at all.
Script Output
Security Warning
Run only scripts that you trust. While scripts from the Internet can be useful, this script can potentially harm your computer. Do you want to run
C:\ExchangeOnlineMigration\Scripts\ValidateMailEnabledPublicFolders.ps1?
[D] Do not run  [R] Run once  [S] Suspend  [?] Help (default is "D"): r
WARNING: Active Directory server settings remained unchanged.
Checking for mail-enabled System folders...
Found 0 mail-enabled System folders.
Getting all public folders. This might take a while...
Found 60169 public folders.
83 of those are mail-enabled.
Generating ExFolders export.
WARNING: NOTE: ExFolders will appear to be Not Responding during the export. That is normal.
Waiting for export to finish...
0 folders are mail-enabled with no AD object.
83 folders are mail-enabled and are properly linked to an existing AD object.
Getting all MailPublicFolder objects...
0 MailPublicFolders are orphaned.
Building EntryId HashSets...
0 orphaned MailPublicFolder objects.
0 of those orphans point to mail-enabled folders that point to some other object.
0 of those orphans point to mail-disabled folders.

No folders need to be mail-disabled.

No orphaned MailPublicFolders were found.

No duplicate MailPublicFolders were found.

4 public folders have proxy GUIDs even though the folders are mail-disabled.
These folders should be mail-enabled. They can be mail-disabled again afterwards if desired.
To mail-enable these folders, run:
Get-Content "C:\ExchangeOnlineMigration\Scripts\MailDisabledWithProxyGuid.txt" | % { Enable-MailPublicFolder $_ }

No disconnected MailPublicFolders were found.

Done!

We need to adjust the folder paths for use the with cmdlets. "Public Folders" can be removed from the path, while "System Folders" should be replaced with "NON_IPM_SUBTREE". The four paths should look like this:
\Internet Newsgroups
\NON_IPM_SUBTREE\OFFLINE ADDRESS BOOK\/o=MYDOMAIN/cn=addrlists/cn=oabs/cn=Standard-Offlineadressliste
\NON_IPM_SUBTREE\OFFLINE ADDRESS BOOK\EX:/o=MYDOMAIN/ou=Erste administrative Gruppe
\NON_IPM_SUBTREE\SCHEDULE+ FREE BUSY\EX:/o=MYDOMAIN/ou=Erste administrative Gruppe

You can try enabling and then disabling using those paths and see if that works.
Also, the problem on these four folders might not prevent migration. I wouldn't expect modern versions of Exchange to attempt to mail-enable a non_ipm_subtree folder due to the presence of a proxy GUID. So you may be able to just ignore these last few errors and proceed with the migration.

I tried manually the Internet Newsgroups and after mail-enableing/disabling them I get again the error with proxy guid.
I tried system folders with NON_IPM_SUBTREE but the CMDlet refused it and says system folders cannot email enabled.

I'm stuck with the 4 errors from the validation script saying "public folders have proxy GUIDs even though the folders are mail-disabled"

A last resort might be to delete those folders, if it will let you. They are no longer used.


Make device_array_like create contiguous arrays (Fixes #4832)

As reported in Issue #4832, device_array_like maintains the strides of source arrays, rather than following the behaviour of Numpy and always creating a contiguous array. This PR changes device_array_like so that it always creates a contiguous array.
Strides for the new array are computing using the same logic as Numpy uses when keeping the order of the source array.
Whilst working on this I noticed that the strides of transposes differs between the CUDA target and Numpy (see Issue #4974) - for the purpose of this PR I've added a test that checks the correctness of the transpose of a view for one particular example. Some further work may follow in another PR depending on the outcome of any discussions in #4974.

Removed the [WIP] tag now that CI is passing.

need a smoke test

Is a smoke test a run through the buildfarm, or is there something else that's required here?

it's a buildfarm run. And unfortunately, the buildfarm has been unstable lately. I could not start the smoketest.

Triggered smoketest: numba_smoketest_cuda_5

Seeing an error in smoke test on these platforms:
test_win_64_py2.7_npy1.16_cuda9.0
test_win_64_py2.7_npy1.16_cuda10.0
test_win_64_py3.7_npy1.16_cuda9.0
test_win_64_py3.7_npy1.16_cuda8.0
test_win_64_py3.7_npy1.16_cuda10.0
test_win_64_py2.7_npy1.16_cuda8.0
The error:
======================================================================
FAIL: test_transpose_view (numba.cuda.tests.cudapy.test_transpose.Test)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\Users\ci\64\Miniconda3\envs\testenv_dda5f874-d1f7-44da-810f-49e634cde40a\lib\site-packages\numba\cuda\tests\cudapy\test_transpose.py", line 37, in test_transpose_view
    self.assertEqual(d_a_view_t.strides, (40, 8))
AssertionError: Tuples differ: (20L, 4) != (40, 8)

First differing element 0:
20L
40

- (20L, 4)
+ (40, 8)

----------------------------------------------------------------------

Looks like a windows only issue. All linux platforms passed.

I guess the default dtype on Windows for arange is int32 (just seeing if I can check this theory) - will fix up shortly if it seems that that is the case.

@sklam Thanks for the heads-up - I've now pushed a change which I hope will resolve that issue.

Passed internal CI

I think everything is addressed, unless there were more tests that you would like to see?

Two more numpy->NumPy else good to go (pending CI) thanks for the fixes.

I'm going to merge the NumPy suggestions and mark this as ready as soon as internal CI catches up.

I'm going to merge the NumPy suggestions and mark this as ready as soon as internal CI catches up.

Sounds good, thanks.

Actually, Github is telling me I don't have permissions...

all CI has passed.  This can be merged after the typo fixes.

I've merged the spelling changes - I see there are now conflicts, but will have to address them in the morning - thanks for blitzing the reviews of all these CUDA PRs today!

I've merged the spelling changes - I see there are now conflicts, but will have to address them in the morning - thanks for blitzing the reviews of all these CUDA PRs today!

Thanks. As they were trivial we did try just merging them in but I think your branch is locked (which is entirely fine, but the reason for you having to click two boxes :)). Once the conflicts are resolved I'll take another look, thanks again!

Seems this doesn't work with recarrays on the simulator or device - looking into fixing now.

The issues that turned up after the merge now seem to be resolved - I've tested with a CUDA device (V100) myself and have no fails after fixing the way contiguity is checked.

also rechecked this on internal CUDA CI


Improve documentation

Hey, I would like to work on this issue.
What changes are needed in documentation?

I also want to work

On this issue


chore(dependencies): bump supported php version to 5.5.0

Travis CI will keep failing until [feat-tests] is merged.


Consolidate logic to wait for dask futures.

Change Description
Reducing code duplication for waiting on dask futures, updating a tqdm task bar, dask task log, raises error, and marks stage done. Also switches from explicit file names to single stage name.
Code duplication was raised as a pylint error in the SOAP PR, so I'm trying to get ahead of it.

I hadn't been thinking too much about it, but now that you're asking me, I guess I will:
I'm in favor of keeping things plain-old-dataclasses for now, and for the foreseeable future. In nearly all cases, the methods that are being added are not modifying the state of the member variables, but are rather helper methods that could live just about anywhere. For the sake of code organization, it makes sense for them to live as class methods.
And I think to your question on slack
does the reduction of boilerplate that dataclass provides imply that maybe more of our traditional classes should be dataclasses

I think that's probably true, though there are certainly limitations and many things that will need to stay full-fledged classes. I have some biases that are showing (coming from years in java persistence layers, and years with protocol buffers) that make me lean toward these kind of container objects.


Add some generated source code

Add a task that simply generates source code to some sourceset.

Milestones

Use Lombok
Code generation in custom annotation processors
Generated code by Thrift, Avro, Jaxb/xjc

Lombok usage in branch https://github.com/gradle/perf-enterprise-large/commit/02e1c30 .

Lombok usage pushed to master branch, M1 complete.


How to get the auto-created file name to admin controller? (If file exists)

Laravel Version: 5.5.22
PHP Version: 7.1.9
Laravel-admin: 1.5.6

Description:

How to get the auto-created(unique) name when uploading file?

Steps To Reproduce:

If image file is already existed. (same file name)
Laravel-admin will create a new unique file name.
In Admin Controller form(), I use getClientOriginalName(), but still get the original file name, not unique.
How to get this unique name and save it to db column when uploading?

Here is code:
    protected function form()
    {
        return Admin::form(Header::class, function (Form $form) {

            $form->image('img', 'PHOTO')->move('path/to/image/');
            $form->hidden('img_small');
            $form->saving(function($form) {
                if ($form->img) {
                    $form->img_small = 'path/to/image/small/' . $form->img->getClientOriginalName();

                    // then save small image
                }
            }

        });
    }

$form->image('img_main', trans('app.img_main'))
->move('image/' . date('Ymd'))->uniqueName();
when save this set auto get a unique name   ;don`t need edit saving
Good luck to you.

@manren Thanks for your solution 🤓
But which I want to ask is:

First time, upload the image , file name is "photo.jpg"
Second time, upload the image, file name is "photo.jpg",  when uploading, the file name is transformed into path/5exivbsi49ekduq9diwfjwisjdu.jpg (No need to use uniqueName())
I hope to get this return value path/5exivbsi49ekduq9diwfjwisjdu.jpg
Is this achievable? 😂

@Haogood why?Talk about the needs？chinese?

@manren
因為想使上傳的圖片維持原有的檔名，存檔的時候產生縮圖，給縮圖和原圖一樣的檔名
發現上傳相同檔名的圖片會自動產生新檔名，還能將路徑保存到DB
想問問看能不能調回這個"新產生"的檔名，可能太過鑽牛角尖了😂
目前用你提供的方法解決這個問題，謝謝🤓


Usage of RBAC for no authentication install

I have kafka-ui on Docker running without authentication since i have an external mechanism for auth.
Is it still possible to set the RBAC without any of the official group providers like oauth, ldap etc?
Example:- I want all the users to have read-only access to ACL and topic with MESSAGES_PRODUCE access.
Any advise or workaround will be appreciated. Thanks!
Version you're running
0.7.0

as a workaround: running keycloak in a separate container, it takes around 50 MB of RAM.


[KYUUBI #1796] Destroy the process after the engine starts

Why are the changes needed?

Destroy the process after the engine starts. #1796
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[X] Add screenshots for manual tests if appropriate
add conf: kyuubi.session.engine.startup.waitCompletion  false

[X] Run test locally before make a pull request

Codecov Report

Merging #1815 (27fab5e) into master (e4bf600) will increase coverage by 0.01%.
The diff coverage is 80.00%.

@@             Coverage Diff              @@
##             master    #1815      +/-   ##
============================================
+ Coverage     59.80%   59.82%   +0.01%
+ Complexity      263      262       -1
============================================
  Files           274      274
  Lines         13655    13669      +14
  Branches       1742     1743       +1
============================================
+ Hits           8167     8177      +10
- Misses         4792     4793       +1
- Partials        696      699       +3

Impacted Files
Coverage Δ

...n/scala/org/apache/kyuubi/engine/ProcBuilder.scala
86.88% <60.00%> (-1.36%)
:arrow_down:

...in/scala/org/apache/kyuubi/config/KyuubiConf.scala
95.61% <100.00%> (+0.03%)
:arrow_up:

.../org/apache/kyuubi/session/KyuubiSessionImpl.scala
82.75% <0.00%> (-1.46%)
:arrow_down:

...rg/apache/kyuubi/engine/trino/TrinoStatement.scala
64.70% <0.00%> (-1.18%)
:arrow_down:

...a/org/apache/kyuubi/metrics/MetricsConstants.scala
100.00% <0.00%> (ø)

.../org/apache/kyuubi/operation/KyuubiOperation.scala
65.33% <0.00%> (+1.95%)
:arrow_up:

Continue to review full report at Codecov.

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

@pan3793, please help to merge.

Thanks, merging to master

I had merged this pr and tested it on our test env.
It seems that it does not work.
I have to found another temporary workaround.
  protected val KYUUBI_PROC_UNIQUE_ID: String = "kyuubi.proc.uniqueId"

  protected lazy val procUniqueId = UUID.randomUUID().toString

  protected def procConf(): Map[String, String] = Map(KYUUBI_PROC_UNIQUE_ID -> procUniqueId)

    var allConf = conf.getAll ++ procConf()

  protected def killProcessByUniqueId(): Unit = {
    val psEf = Runtime.getRuntime.exec("ps -ef")
    val input = new BufferedReader(new InputStreamReader(psEf.getInputStream))
    val regex = s"$KYUUBI_PROC_UNIQUE_ID=$procUniqueId".r
    var psInfo = input.readLine()
    var found = false
    while (!found && psInfo != null) {
      regex findFirstIn (psInfo) match {
        case Some(_) =>
          found = true
          psInfo.trim.split("\\s+").drop(1).headOption.foreach { psId =>
            Runtime.getRuntime.exec(s"kill -9 $psId")
          }
        case None =>
      }
      psInfo = input.readLine()
    }
  }


remove node_modules from version control

this directory is in .gitignore, and AIUI it should never be included in version control, but instead always produced in a local working copy by running 'npm ci' or similar.
this was added in aef0b8bf32ac7c1312ee7478da9ebc223b7602db; the commit message is pretty vague but my read of it is that it must've been accidental.

ty, I guess it's in there forever in the git history, but no this shouldn't have been in there


Use the current URL to determine whether to open a secure websocket connection

I see that an insecure websocket connection is opened from WebSocketUse.js regardless of whether the application is accessed over SSL or not. Instead this is decided by a value returned from the call to /api/settings.
This causes a problem when accessing over a reverse proxy via SSL. Can we use the window.location.protocol to determine whether to use ws:// or wss://?

Hi,
what version are you using?
If you use https (-ssl option) why your connection is not secured?
And why you don't use normal http if in your setup https is not working?

If you use https (-ssl option) why your connection is not secured?

Hi, where can I set this? Perhaps I missed it, but I don't remember seeing this option at all.

And why you don't use normal http if in your setup https is not working?

I want to access it behind a reverse proxy over SSL.

If you don't add any parameter to the command line the frontend is served by a http server that serves also a websocket.
If you add the option -ssl you need to add your certificates and optionally also the CA certificate. In this case both frontend and websocket are served by https. The secure connection must be working if you use the -ssl.

Ah thanks. I am running matterbridge as a docker container, how would I specify this option?

services:
matterbridge:
container_name: matterbridge
image: luligu/matterbridge:latest # Matterbridge image with the dev tag
restart: always # Ensures the container always restarts automatically
network_mode: host # Ensures the Matter mdns works
volumes:
- "${HOME}/Matterbridge:/root/Matterbridge" # Mounts the Matterbridge plugin directory
- "${HOME}/.matterbridge:/root/.matterbridge" # Mounts the Matterbridge storage directory
- "${HOME}/Root:/root" # Mounts the root directory
environment:
- TZ=Europe/Paris
command: [ "matterbridge", "-childbridge", "-docker" ]
You can add command and put what you need. Don't forget -docker if you override the default!
How to use -ssl:
check here https://github.com/Luligu/matterbridge/blob/main/CHANGELOG.md#136---2024-06-28
and matterbridge -help


added visionOS support

change all #if os(iOS) to #if os(iOS) || os(visionOS) to support visionOS
added visionOS as one of the supported destination

Thank you @Ju-Long 🙏🏼


Update scalafmt-core to 3.5.1

Updates org.scalameta:scalafmt-core from 3.5.0 to 3.5.1.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequest = { frequency = "@monthly" },
  dependency = { groupId = "org.scalameta", artifactId = "scalafmt-core" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Codecov Report

Merging #1944 (b580245) into main (8586b42) will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1944      +/-   ##
==========================================
- Coverage   86.17%   86.09%   -0.09%
==========================================
  Files          73       69       -4
  Lines        2532     2503      -29
  Branches      132      132
==========================================
- Hits         2182     2155      -27
+ Misses        350      348       -2

Flag
Coverage Δ

2.12.15
?

2.13.8
86.09% <ø> (-0.20%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...re/shared/src/main/scala/io/circe/MapDecoder.scala
96.07% <0.00%> (-1.97%)
:arrow_down:

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
87.13% <0.00%> (-1.76%)
:arrow_down:

...es/core/shared/src/main/scala/io/circe/Error.scala
61.76% <0.00%> (-1.40%)
:arrow_down:

...core/shared/src/main/scala/io/circe/CursorOp.scala
54.16% <0.00%> (-1.39%)
:arrow_down:

.../core/shared/src/main/scala/io/circe/Decoder.scala
83.37% <0.00%> (-0.30%)
:arrow_down:

...nter/src/main/scala/io/circe/pointer/Pointer.scala
89.34% <0.00%> (-0.13%)
:arrow_down:

.../src/main/scala-2/io/circe/EnumerationCodecs.scala
100.00% <0.00%> (ø)

...rc/main/scala-2/io/circe/EnumerationDecoders.scala
100.00% <0.00%> (ø)

...in/scala/io/circe/pointer/PointerSyntaxError.scala

...rc/main/scala-2.12/io/circe/FactoryInstances.scala

... and 2 more

Continue to review full report at Codecov.

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

Superseded by #1947.


[TEST] Harvester v1.2.2 - Rancher v2.8.4 Integration Test

Prerequisites

Setup 3 nodes Harvester v1.2.2 (QEMU/KVM, 8C/16G/500G per node)
Create a new storage class apart from default one.
Create 2 networks on different NICs: mgmt-vlan1 and bond-vlan1
Create 2 VMs with extra disk and data with md5sum: 1 running and 1 stopped
Take VM backups and snapshots on 2 VMs

Rancher Integration Test
v2.8.4

Setup Rancher v2.8.4 (Helm with v1.28.9+k3s1 on 2C/4G QEMU/KVM)
Import Harvester to Rancher
Create single node RKE2 cluster

harvester_e2e_tests/integrations/test_9_rancher_integration.py
Scale up to 2 nodes
Scale down to 1 node

Create single node RKE1 cluster

harvester_e2e_tests/integrations/test_9_rancher_integration.py
Scale up to 2 nodes
Scale down to 1 node

Create single node RKE2 cluster via Terraform

harvester_e2e_tests/integrations/test_z_terraform_rancher.py

v2.8.3 -> v2.8.4

Setup Rancher v2.8.3 (Helm with v1.28.9+k3s1 on 2C/4G QEMU/KVM)
Import Harvester to Rancher
Create single node RKE2 cluster

harvester_e2e_tests/integrations/test_9_rancher_integration.py
Scale up to 2 nodes
Scale down to 1 node

Upgrade Rancher to v2.8.4

Cluster still Active
Nginx deployment and LB still works

Reference

harvester/tests/issues/1150
harvester/tests/issues/900
https://github.com/rancher/rancher/releases/tag/v2.8.4

Prerequisites

 Setup 3 nodes Harvester v1.2.2 (QEMU/KVM, 8C/16G/500G per node)

 Create a new storage class apart from default one.

 Create 2 networks on different NICs: mgmt-vlan1 and bond-vlan1

Create 2 VMs with extra disk and data with md5sum: 1 Running and 1 Off

 running w/ mgmt-vlan1

 stopped w/ bond-vlan1

 Both with extra data and md5sum

Take VM backups and snapshots on 2 VMs

 Backup

harvester/harvester/issues/5841

 Snapshot

Rancher Integration - v2.8.4

 Setup Rancher v2.8.4 (Helm with v1.28.9+k3s1 on 2C/4G QEMU/KVM)

:yellow_circle: Create single node RKE2 cluster (v1.28.9+rke2r1)

 harvester_e2e_tests/integrations/test_9_rancher_integration.py::TestRKE2

test_import_harvester
test_create_rke2
test_create_pvc
test_csi_deployment
test_delete_deploymen
test_deploy_nginx
test_load_balancer_service[1-dhcp]
test_load_balancer_service[1-pool]

 Scale up to 2 nodes

 Scale down to 1 node :x:

Known harvester/harvester/issues/4358

:green_circle: Create single node RKE1 cluster (v1.28.9)

 harvester_e2e_tests/integrations/test_9_rancher_integration.py::TestRKE1

test_create_rke1
test_create_pvc
test_cloud_provider_chart
test_deploy_nginx
test_load_balancer_service
test_csi_driver_chart
test_csi_deployment
test_delete_deployment

 Scale up to 2 nodes

 Scale down to 1 node

:green_circle: Create single node RKE2 cluster via Terraform

 harvester_e2e_tests/integrations/test_z_terraform_rancher.py

test_import_harvester
test_create_cloud_credential
test_create_machine_config
test_create_rke2_cluster
test_delete_rke2_cluster
test_delete_machine_config
test_delete_cloud_credential

Provide test result on harvester-v1.2.2 with

rancher-v2.8.4 (detail)
rancher-v2.8.3 upgrade to rancher-v2.8.4 (detail)

Issues hit:

harvester/harvester/issues/5841

Only affect offline VM backup

harvester/harvester/issues/4358

Known scale down issue
Source issue is planned to fix in rancher-v2.9
rancher/rancher/issues/42582

cc. @khushboo-rancher

FYR, harvester-cloud-provider upgrade fail.
[BUG] Fail to Upgrade harvester-cloud-provider on RKE2 cluster (rancher-v2.8.4)


Vscode Terminal still black screen

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: latest version(1.64)
OS Version: windows 10

I have used vscode for 2 years and I have never see like this issue.
I am using wsl2 with window. Before the bug, the default terminal of vscode was wsl2, and I turned off the hyper -v option to use linux as vmware without using wsl2 for a while.
After that, even if I turned on the hyper -v option and turned on vscode again, vscode's terminal did not work.
The same is true if you change the default terminal to a shell other than wsl2.
The problem is still there even if you connect to vscode through the code command in wsl2
No text is displayed in the terminal and there is no response
how can I bug this?

Can you share your settings.json file and also do a trace log while creating a terminal? https://github.com/microsoft/vscode/wiki/Terminal-Issues#enabling-trace-logging

Sorry for the late reply
Several files were deleted while updating Windows, and at the same time, the terminal problem of vscode was resolved.
Maybe there is a problem with the path of vscode or some file itself is broken.
thank you for the reply


MQTT not working

can't get MQTT working
`[INFO][MQTTTask] Connecting to MQTT broker: <IP_ADDRESS> on port 1883
[INFO][MQTTTask] Connected to MQTT broker as: OE6PLD-10
[DEBUG][RadiolibTask] [19:33:44] Transmitting packet 'Source: OE6PLD-10, Destination: APLG01, Path: , Type: Error, Data: =4706.08NL01549.48E&LoRa iGATE & Digi, SiniGate'
[DEBUG][RadiolibTask] [19:33:47] TX done`
I have activated the last_will in the setup so I presume there should be a "Sending birth message..." (see TaskMQTT.cpp line 67)
on the mqtt-server I don't see any topics connected to LoraAPRS
73 de OE6PLD

...compiled newest sources: works as expected --> close


Move pageLoadTime to details

While schematizing userale log data in a downstream application, I've noticed that pageLoadTime is the only unique field for a specific event. All other events use the details field, so it would make sense if pageLoadTime was also a detail.

I'm totally cool with this, just want to make sure we aren't breaking any data pipelines for downstream users with this schema change

Ideally we'd have [ or later add ] tests for the downstream dependencies.  Having on-hand, in repo, is an ideal way to facilitate developer velocity, and not having to check with users.
Related a published spec of what the project agrees to adhere to [ per major version, for example ] might be helpful?
Trying to ensure to not be in the business of having to check with downstream users - but to proactively be looking out for them, and establish on what the project can/not change, or at least without 'breaking'[major version] change.

Ideally we'd have [ or later add ] tests for the downstream dependencies. Having on-hand, in repo, is an ideal way to facilitate developer velocity, and not having to check with users.
Related a published spec of what the project agrees to adhere to [ per major version, for example ] might be helpful?
Trying to ensure to not be in the business of having to check with downstream users - but to proactively be looking out for them, and establish on what the project can/not change, or at least without 'breaking'[major version] change.

100%. At this point, I'm not even sure how to verify (maybe @poorejc knows?). I think in the past when we've adjusted the schema it's just been a call out in the release notes. Maybe it's time to put that schema in code and version it?

I could add a json schema, and some testcases to validate log data. I'm assuming the schema file should go in test/ and new test cases could be added to packageLogs_spec.js?

in this case, JSON Schema probably makes sense.  Personally, generally, I try to avoid use of JSON Schema and err towards defining protobuf - but that is less prevalent in the web-focused world [ LOL -- though, at times, I even will send a proto over REST, and not using gRPC or gRPC Web.
Also curious, why add schema checks via an existing test file - rather than creating a new test file focused on tests specifically for schema?
If defining a schema that is being tested against, we might want that very clearly referenced/findable.  That might be in /tests if well defined, but ensuring that is clearly published would potentially be valuable to users.

On second thought, it would be cleaner to put schema tests in a new file. I suggested packageLogs_spec.js because the schema validation would probably test functions in packageLogs.js.
Another benefit of json schema over protobuf, json schema can be used by applications receiving userale data for safe deserialization.

In our case, I would not advocate for proto -- that is a higher learning curve, and engineering bar.  JSON Schema suffices.

IMHO, this makes a lot of sense. Jason is 100% pageload time does break the core schema. Moving to details makes a lot of sense, given the use of details.
Regarding downstream users, there are lots of folks using forks... so I don't feel bad making a smart breaking change. We could also configure the NPM release so that the package.json flag ~2.3.0 wont' grab 2.3.1 or 2.4.0 or however we version. That won't break existing builds.

Update:
Been thinking about this more...

No brainer to push PageLoad time to details
I like the idea of schema validation, but thinking about this more, I think tests should focus on packageLogs and validating the core schema
The thing that would truly break things is if we were too heavy handed in enforcing schemas coming out of the UserALE API--customLog, packageCustomLog are meant to be flexible. <--schema management for these should probably happen in downstream pipelines, as we do with ELK (soon Kafka).

I think the second two bullets are discussions we should have on user@flagon

+1
Sorting out schema management not intimately tied to this PR
On Wed, Mar 15, 2023, 8:08 AM poorejc @.***> wrote:

If nothing else heard, I'm good to merge this PR. I think its gtg (test
looks good, and at least risky scope) then continue discussion on schema
management as per above.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-flagon-useralejs/pull/347#issuecomment-1470177484,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGMTJE64PCJTKRNIO3GW7TW4HLPTANCNFSM6AAAAAAVRN3YKI
.
You are receiving this because you commented.Message ID:
@.***>

Thanks, I'll do some more testing and get merged into Master! Thanks @Jyyjy

@Jyyjy your PR is on master! Again, thanks so much for your thoughtful work here!
BTW, your PR is the very first commit to master for Apache Flagon as a TOP LEVEL PROJECT. Congrats! You're immutably committed into Flagon's project history 👍


[HOLD for payment 2024-11-14] [$250] iOS - Categories - Offline indicator overlaps with home bar in Default spend categories page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.54-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Launch ND app
Go to workspace settings > Categories
Go offline
Tap Settings
Tap Airlines
Scroll down to dismiss the keyboard

Expected Result:
Offline indicator will not overlap with home bar after dismissing the keyboard
Actual Result:
Offline indicator overlaps with home bar briefly after dismissing the keyboard
Workaround:
Unknown
Platforms:

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

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[x] iOS: Standalone
[x] iOS: HybridApp
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/af943413-04fd-4b42-accb-f28ada6f0ded
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021850900545784411884
Upwork Job ID:<PHONE_NUMBER>784411884
Last Price Increase: 2024-10-28
Automatic offers:

dukenv0307 | Reviewer | 104697028

Issue OwnerCurrent Issue Owner: @isabelastisser

Proposal
Please re-state the problem that we are trying to solve in this issue.
Offline indicator overlaps with the iOS home bar briefly after closing keyboard.
What is the root cause of that problem?
The category selector page uses a Modal and inside the modal, we have a padding bottom added to avoid the home bar overlapping. But, if a keyboard is shown, we won't apply the padding-bottom.
https://github.com/Expensify/App/blob/5103b4bd53ae74dcb301b8450f7046e82dc311d3/src/components/Modal/BaseModal.tsx#L196
It was added to fix this issue where there is an extra space at the bottom when the keyboard is shown, so they apply the padding-bottom only if the keyboard is closed. So, in this issue, when the keyboard opens, the padding-bottom isn't applied and when we close the keyboard, the offline indicator will overlap with the home bar for a while until the keyboard state changes to "closed".
What changes do you think we should make in order to solve the problem?

Applying the padding-bottom conditionally based on the keyboard state is only needed when the keyboard-avoiding view is enabled. If it's disabled, then the modal content won't be pushed above the keyboard, so we won't see the padding-bottom either. So, the first solution is to always apply the padding-bottom if the keyboard avoiding view is disabled.
shouldAddBottomSafeAreaPadding: (!avoidKeyboard || !keyboardStateContextValue?.isKeyboardShown) && shouldAddBottomSafeAreaPadding,

avoidKeyboard is disabled by default.
https://github.com/Expensify/App/blob/5103b4bd53ae74dcb301b8450f7046e82dc311d3/src/components/Modal/BaseModal.tsx#L47
And because we don't need to avoid keyboard for the category selector modal, we can just disable it from the ScreenWrapper too.
https://github.com/Expensify/App/blob/5103b4bd53ae74dcb301b8450f7046e82dc311d3/src/pages/workspace/distanceRates/CategorySelector/CategorySelectorModal.tsx#L33-L48
shouldEnableKeyboardAvoidingView={false}

What alternative solutions did you explore? (Optional)
Wrap the children with SafeAreaView and remove the manual safe area padding calculation.
https://github.com/Expensify/App/blob/5103b4bd53ae74dcb301b8450f7046e82dc311d3/src/components/Modal/BaseModal.tsx#L188-L203
https://github.com/Expensify/App/blob/5103b4bd53ae74dcb301b8450f7046e82dc311d3/src/components/Modal/BaseModal.tsx#L272-L277
<SafeAreaView style={styles.flex1}>

@muttmuure, does this issue belong to [#whatsnext] #quality?

PR is ready
cc: @dukenv0307

Bump @dukenv0307 for PR review. Thanks!

Requested in ND.

Payment summary:
@bernhardoj requires payment through NewDot Manual Requests $250 C+
@dukenv0307 requires payment automatic offer (Reviewer) Paid in Upwork.

$250 approved for @bernhardoj


K8SSAND-1339 ⁃ Doc: Tasks > Repair topic needs review and updates for K8ssandra Operator multi-cluster

What happened?
The "Repair Cassandra with Reaper" topic at https://docs-v2.k8ssandra.io/tasks/manage/private-registries/ is just a copy of the v1.4.x topic.  Need a SW engineer to review and update the content for K8ssandra Operator multi-cluster.
How to reproduce it (as minimally and precisely as possible):

https://github.com/k8ssandra/k8ssandra/blob/docs-v2/docs/content/en/tasks/repair/_index.md
https://docs-v2.k8ssandra.io/tasks/repair/

K8ssandra Operator version
1.0.x.
Anything else we need to know?:

Revise the topic on the doc-v2 branch of k8ssandra/k8ssandra GH repo.
When done and PR is merged, please tag and publish the updates. Tag & publish steps: https://github.com/k8ssandra/k8ssandra/tree/docs#github-actions-and-tags

┆Issue is synchronized with this Jira Task by Unito
┆friendlyId: K8SSAND-1339
┆priority: Medium

Handled in the k8ssandra project: https://github.com/k8ssandra/k8ssandra/issues/1368


Reload the workspace state if workspace-state.json changes

Depends on https://github.com/apple/swift-package-manager/pull/4310

Previously, sourcekit-lsp wouldn’t take into account changes to the workspace state (like swift package edit) after it was launched.
Fixes rdar://89551074 [SR-15918]
Fixes rdar://89082936 [SR-15875]

Closed in favor of https://github.com/apple/swift-package-manager/pull/5515


Add Dockerfile for IPython notebook

I moves the jupyter command to pid 1. Also means only one process running in container rather than bash + jupyter.
c.f.
$ docker exec 2da15f24bbbe ps -aux
USER       PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
root         1  0.0  0.2  17964  2880 ?        Ss+  02:25   0:00 bash /root/less.sh
root         8  0.0  0.1   6700  1644 ?        S+   02:25   0:00 less /root/f

$ docker exec 015e79b13d9e  ps -aux
USER       PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
root         1  0.1  0.1   6700  1652 ?        Ss+  02:26   0:00 less /root/f

where bash runs less.

The TF equivalent does not have that behavior, though: https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tools/docker/run_jupyter.sh#L18.


Fixes: #2368

cleans up a couple of possible seg faults when RDBASE is not set.

Fair enough.  This might not be in the static initializer then.


[BUG] YAML file is wrong at chunk_full.yaml and README.md

Describe the bug
At the chunk_full.yaml and README.md, add_file_name is ko or en
Right? @bwook00

Plus, the import is different (do not use beta) at README.md in data creation


Solving the N+1 problem

1- modified the sql queries : merging the two queries into a single query which joins the departements table and the directions table .
2-the query outputs where an array of departements objects with a directionId attribute so i've mad an algorithm to  transform the raw objects into  directions objects containing an array of departements

thank you for your contribution , the issue has been resolved just how it should be.


Implement XlibClient

Xlib is very hard I don't know why init function doesn't work but maybe xlib -> xcb -> x11rb connection available via xlib_xcb and XcbConnection


edb.errors.IdleSessionTimeoutError: closing the connection due to idling

EdgeDB Version: 2.0 (built from master branch)
OS Version: Linux 3.10.0-1127.19.1.el7.x86_64

Steps to Reproduce:
Sorry that I'm using a custom java client and there might be some difference in implemention detail between the official library and my one. So I cannot give a minial reproducible example.
But I think the bug is actually quite obvious that maybe I can just explain it with code.
_reinit_idle_gc_collector returns timeout in seconds.
    def _reinit_idle_gc_collector(self) -> float:
        ...

        session_idle_timeout = config.lookup(
            'session_idle_timeout', self._dbindex.get_sys_config())

        timeout = session_idle_timeout.to_microseconds()
        timeout /= 1_000_000.0  # convert to seconds

        if timeout > 0:
            self._idle_gc_handler = self.__loop.call_later(
                timeout, self._idle_gc_collector)
        return timeout

Thus the idle_timeout in following code will be in seconds。 And the calculation:  expiry_time  = now (microseconds) - idle_timeout (seconds)  will be problematic. Which in some cases, lead to the IdleSessionTimeoutError.

    def _idle_gc_collector(self):
        try:
            self._idle_gc_handler = None
            idle_timeout = self._reinit_idle_gc_collector()

            if idle_timeout <= 0:
                return

            now = time.monotonic()
            expiry_time = now - idle_timeout

@Androidown, do you have a link to the official and/or custom Java clients/libraries?

That would be a nice easy fix, but unfortunately (for this bug!) time.monotonic does return seconds, not microseconds.

How quickly are you timing out?

That would be a nice easy fix, but unfortunately (for this bug!) time.monotonic does return seconds, not microseconds.

Sorry, my mistake.

How quickly are you timing out?

How quickly are you timing out?

Almost immediately after a command issued.  Maybe it's because I maintained a connection pool and didn't set idle_session_timeout to null for each connection. I'll take a look into it. Thanks for your reply.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-30-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4486

Questions ?
Please refer to the Backport tool documentation


Email is blank error after password reset

I'm trying to use the reset password functionality but I seem to be running into some issues. The email sends to the user and I'm able to click the password reset link. When I put in the new password and the confirmation password, it takes me to the password reset initiation page with the error "Email can't be blank."
I'm not sure what to do
Here are the routes.db, devise.rb, and my custom users_controller.rb file. The custom users_controller is not being used for the actual password reset functionality, but the initiation is done through my custom users controller.
https://gist.github.com/dsarhadian/66d86c143627d30026980fe06fe94ea6
Any help is appreciated.

@dsarhadian Just now I checked with latest devise version 4.2 and rails version 4.2.5. It works fine for me. The only diff. I noticed that I used devise controllers for password reset functionality . Could you please provide devise version and the rails version? So that I can cross check?

@sivagollapalli Sure I'm using Rails 4.2.6 and Devise 4.2.0. I'm also using Devise controllers for the actual password reset functionality. The only part that differs is that my custom User's controller just initiates the password reset email.

Can you please provide a sample application that reproduces the error?

Closing this for now. @dsarhadian feel free to send sample app that reproduces this issue and we\ll try to help you. Thanks!


[Bug]: Markdown syntax goes inside brackets

Bug Description
When a link is selected and toggle bold or italics command is used, the asterisks will go inside of the brackets instead of outside even though the brackets were included in the selection.
Relevant Screenshot

Immediately after using the command

Similarily, when the whole line is selected

the brackets on the end are ignored

To Reproduce

Select text
Use smarter bold command

I only tested bold, italics, and comment. Commenting did not have the same issue.
Also, this might be in FR territory, but I think it's related. My intuition would say that using a smarter command in a link when no text is selected would always place the syntax outside the brackets. I can see the case for it not being that way for syntax that is allowed in links like round brackets, but for markdown syntax like asterisk, square bracket etc it will always break the link.
i.e using smarter bold in like this

turns into this

Obsidian Version
0.13.19
Which editor are you using?

[X] New Editor
[ ] Legacy Editor

In which mode(s) does the problem occur?

[X] Source (Edit)
[X] Live Preview

Checklist

[X] I updated to the latest version of the plugin.

Okay, the latest release fixes the first issue, markup is now always out outside the selection. (did this for the punctuation commands as well, since I do not want to make the commands too different from each other).
This also works when you have the cursor inside a one-word wikilink. Wikilinks with more than one word are far more tricky to implement, so it's definitely something for later. Could you open a separate FR for that? Thanks


Python: In kernel.py, we swallow exceptions when they shouldn't be.

Customer report: today in kernel.py, we swallow exceptions when they shouldn't be.

This was fixed in #2948, so they should upgrade to a newer version of SK!


Discussion: transposing images in roiextractors

ROIExtractors read and return image shapes that corresponds to (height x width), whereas NWB expects in (width x height). Currently, we transpose image data in add_two_photon_series as:
https://github.com/catalystneuro/nwb-conversion-tools/blob/a8f224f9611a3b7444964f6d3b8b2e887a18a5f2/src/nwb_conversion_tools/tools/roiextractors/roiextractors.py#L185-L188
wrapped into DataChunkIterator.  While working on  #575, it caused some issues with determining the shape of the buffer, since there was a mismatch between the original image size (height x width) and data shape (transposed to width x height).
I would like to open a discussion about this, because with #575 being in progress I feel like there could be a more appropriate place manipulating the image data than the iterator level. What if the ImagingExtractors would make sure that the images are returned in the appropriate (width x height) format?
Let me know what you think about this, @CodyCBakerPhD @bendichter @h-mayorquin
(While it is connected to roiextractors, the place where the image data is being transposed is in CT, so this why I opened the issue here, but let me know if I should move this issue somewhere else.)

What if the ImagingExtractors would make sure that the images are returned in the appropriate (width x height) format?

Unfortunately that would be a much bigger overhaul - also, it's more natural for ROIExtractors to use height x width since that is what several of the constituent formats use, and seems to be an ad-hoc 'standard' axis convention for image processing.
Basically, NWB is the outlier in that respect, so it makes the most sense to just order the axes when mapping from ROIExtractors to NWB.
It's also not that awkward to perform such axes mappings at the iterator level, we used to do that for the old spikeextractors (when traces returned channels x frames).

While working on https://github.com/catalystneuro/nwb-conversion-tools/pull/575, it caused some issues with determining the shape of the buffer, since there was a mismatch between the original image size (height x width) and data shape (transposed to width x height).

The axis order should not cause problems with size determination - if it's that extra unsqueezed problem you found in https://github.com/catalystneuro/fee-lab-to-nwb/issues/16, it should be unrelated to this. If it's a new problem, what exactly is the issue you encountered?

@CodyCBakerPhD the maximum shape that is determined by the iterator is incorrect, because it considers the original order of axes and not the transposed shape of data, i fixed it here:
https://github.com/catalystneuro/nwb-conversion-tools/blob/a253a5535aaa6123209912468798b788ddfc284a/src/nwb_conversion_tools/tools/hdmf.py#L161


Python 3 Migration

It will be worthwhile to begin the inevitable migration to Python 3.
Will have to review compatibility with the following packages:

[ ] Django>=1.9.6,<1.10
[ ] django-environ>=0.4.0,<0.5
[ ] django-markdown-deux>=1.0.5,<1.1
[ ] django-session-security>=2.3.2,<2.4
[ ] django_redis>=4.4.3,<4.5
[ ] djangorestframework>=3.3.3,<3.4
[ ] djangorestframework-jwt>=1.5.0,<1.6
[ ] ehb-client
[ ] ehb-datasources
[ ] python-ldap>=2.4.25,<2.5
[ ] psycopg2>=2.6.1,<2.7
[ ] python-logstash>=0.4.6,<0.5

As well as our Docker baseimage

python-ldap is py3 incompatible.
review https://pypi.python.org/pypi/ldap3


Added mypy for linter configuration

After adding flake8 and pylint, we also added mypy configuration file for testing.

FYI, related to #1.