add asterisk rotating on scroll & add logo interaction

add asterisk rotating on 'download' page and 'reference' page within scroll.js
add logo interaction on mouse hover

Yeah!

so hot!! one thing, I added the cursor back into the sketch here:
https://github.com/processing/p5.js-website/commit/6023849e0f75d1a0832f43929d863639f2df2d4a
the logo is a shortcut link to the homepage, so I felt like it was important that people felt it was clickable. is this ok with you?

Not a problem at all!
I forgot it's a button ;)
-xy
On Tue, Jun 2, 2015 at 10:23 PM, Lauren McCarthy<EMAIL_ADDRESS>wrote:

so hot!! one thing, I added the cursor back into the sketch here:
6023849
https://github.com/processing/p5.js-website/commit/6023849e0f75d1a0832f43929d863639f2df2d4a
the logo is a shortcut link to the homepage, so I felt like it was
important that people felt it was clickable. is this ok with you?
—
Reply to this email directly or view it on GitHub
https://github.com/processing/p5.js-website/pull/133#issuecomment-108163857
.

--
@xy_feng http://twitter.com/xy_feng
xystudio.cc

hm, so the other thing I notice is that it seems like performance of the homepage sketch takes a hit with this logo. I guess that makes sense, running two sketches at the same time will be slower. so I guess we could have the animated logo on all pages but the homepage, and use the static one there. how do you feel about this? I know it's not ideal, but I'm not sure what to do about performance, I feel like we don't want the first impression to be too crunchy.

Agreed on the performance. A temporary solution:
Display image at the beginning, load and hide logo sketch; use a timer or
callback to swap out of the image. We may want to set the parent div with
fixed width and height in case the content shifts.
The code may get messy. It's kind of nice to see it on all pages, however,
I'm totally ok with displaying it only on other pages as well.
-xy
On Tue, Jun 2, 2015 at 10:35 PM, Lauren McCarthy<EMAIL_ADDRESS>wrote:

hm, so the other thing I notice is that it seems like performance of the
homepage sketch takes a hit with this logo. I guess that makes sense,
running two sketches at the same time will be slower. so I guess we could
have the animated logo on all pages but the homepage, and use the static
one there. how do you feel about this? I know it's not ideal, but I'm not
sure what to do about performance, I feel like we don't want the first
impression to be too crunchy.
—
Reply to this email directly or view it on GitHub
https://github.com/processing/p5.js-website/pull/133#issuecomment-108166153
.

--
@xy_feng http://twitter.com/xy_feng
xystudio.cc

Ok, I've set it up to not display on the homepage. I also added a little swap out with the svg image so we don't see a jump as we wait for sketch to load. I'm not sure I understand what you mean by timer though.. I'm open to any solution for the homepage as long as it doesn't make the featured sketch slow down too much.


[SPARK-40717][CONNECT][TESTS][FOLLOW-UP] Assert analyzed plan in column alias test case

What changes were proposed in this pull request?
This PR is a followup of https://github.com/apache/spark/pull/38174 which mistakenly missed to assert the analyzed plans.
Why are the changes needed?
To make sure the test coverage.
Does this PR introduce any user-facing change?
No, test-only.
How was this patch tested?
Unittest fixed.

Ah I was trying to fix this in https://github.com/apache/spark/pull/38227.
We can get this one in first, either is fine to me.
LGTM

meh that's fine. Let;s fix it there together.


Panel support / N-D labeled arrays

Hello,
Python Pandas provides a nice data structure called Panel for 3 dimensional data.
http://pandas.pydata.org/pandas-docs/stable/dsintro.html#panel
xarray (previously known as xray) http://xarray.pydata.org/ provides N-D labeled arrays.
This kind of data structure are very convenient.
See for example this example in Python (using Pandas DataReader http://pandas-datareader.readthedocs.org/en/latest/ )
from pandas_datareader import data as web
data = web.DataReader("GOOG", "IBM", "2015-04-01", "2015-04-10")

In [7]: web.DataReader("IBM", "google", "2015-04-01", "2015-04-10")
Out[7]:
              Open    High     Low   Close   Volume
Date
2015-04-01  160.23  160.62  158.39  159.18  3700791
2015-04-02  159.52  162.54  158.89  160.45  4671578
2015-04-06  159.69  162.80  158.70  162.04  3465682
2015-04-07  161.67  163.84  161.62  162.07  3147975
2015-04-08  161.72  163.55  161.01  161.85  2524323
2015-04-09  161.70  162.47  160.72  162.34  2263490
2015-04-10  162.34  163.33  161.25  162.86  2515703

In [8]: web.DataReader("MSFT", "google", "2015-04-01", "2015-04-10")
Out[8]:
             Open   High    Low  Close    Volume
Date
2015-04-01  40.60  40.76  40.31  40.72  36865322
2015-04-02  40.66  40.74  40.12  40.29  37487476
2015-04-06  40.34  41.78  40.18  41.54  39223692
2015-04-07  41.61  41.91  41.31  41.53  28809375
2015-04-08  41.46  41.69  41.04  41.42  24753438
2015-04-09  41.25  41.62  41.25  41.48  25723861
2015-04-10  41.63  41.95  41.41  41.72  28022002

In [9]: panel = web.DataReader(["IBM", "MSFT"], "google", "2015-04-01", "2015-04-10")
In [10]: panel
<class 'pandas.core.panel.Panel'>
Dimensions: 5 (items) x 7 (major_axis) x 2 (minor_axis)
Items axis: Open to Volume
Major_axis axis: 2015-04-01 00:00:00 to 2015-04-10 00:00:00
Minor_axis axis: IBM to MSFT

In [13]: panel["Open"]  # or panel.loc['Open',:,:]
Out[13]:
               IBM   MSFT
Date
2015-04-01  160.23  40.60
2015-04-02  159.52  40.66
2015-04-06  159.69  40.34
2015-04-07  161.67  41.61
2015-04-08  161.72  41.46
2015-04-09  161.70  41.25
2015-04-10  162.34  41.63

I wonder if such data structure exists in Julia.
It will be very convenient for my project DataReader.jl.
Kind regards

Am I right that this is basically a list of DataFrames with some convenience functions? If so, that could easily be experimented in a separate package. DataFrames.jl already contains GroupedDataFrame for split-apply-combine operations, which might give some inspiration.
I also wonder whether this isn't closer to TimeSeries and AxisArrays than to DataFrames.

I don't find in DataFrame.jl the concept of index... this is something very important when creating a Panel from dict of DataFrame as it should automatically add NaN for missing values when some index doesn't exist in one of the DataFrame.
Thanks for pointing to TimeSeries and AxisArrays. I will have a look

NDSparseData.jl does have the concept of indexes, and is in very early stages, but is something I think we will pursue. Would be great if you could tell us what operations on indexed data you would like to do.

Wouldn't you simply specify the name of the column to use as an index when converting a DataFrame to a panel object? Why does this information need to be stored in the data frame?

@ViralBShah  Some features
Support for both integer indexing and label indexing.
Slicing
Rename dimension
Rename columns
Groupby operation
Datetime indexing
Resampling
Easy serialization

The separation of indices and columns is one of the strangest things about Pandas. Wes can give lots of good arguments for why you might want to do it, and Hadley will remind you that real databases don't do any of these tricks. It may be useful for financial applications in the same way that big data stacks distinguish partitions and columns because they're not capable of generating indices for tables but need efficient filtering.

Hierarchical indexing is also a great feature (MultiIndex) !

I think this would be best pursued as an independent package.


Step progress bar

A step progress bar is used to track the progress at every step . Initially all the steps are incomplete when ever a task is completed then
it is marked and moved to next step.

What's there uses ?

Sir actually it is used to display the progress of our work as displayed in
the image .
When we complete some task 1 then one tick wil be placed in that way I
guess it is useful
On Sun, 29 Jan 2023, 12:16 Harshit Paneri, @.***> wrote:

What's there uses ?
—
Reply to this email directly, view it on GitHub
https://github.com/TusharKesarwani/Front-End-Projects/issues/507#issuecomment-1407578998,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATCQWHRDDHEFFE4LBEDNZCDWUYG55ANCNFSM6AAAAAAUKBBUUI
.
You are receiving this because you authored the thread.Message ID:
@.***>

okay


Add clear function for bag state types

This allows all the bag type states to clear their state and adds tests/integration tests for them.
Part of #22736

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.


Check splitbutton against DL and VRT

Check splitbutton

[ ] matches DL
[x] check VRT passes
[x] check and update migration guides example

Split button is fine, the positioning is changed by the line breaks and not the component itself.


Redesigned color picker widget

This PR introduces a newly redesigned color picker widget, built on top of the Pickr UI widget as opposed to Spectrum, which has not seen a release since 2016.

Benefits

More clean and streamlined UX - available colors are now shown within the popover as opposed to spanning across the entire row at all times. The UI is very similar to VSCode's color picker, which should hopefully encourage familiarity.
Text input next to the color picker allows full copy+paste capabilities, as well as simply typing out the color code.
Supports touch screen devices.
Full accessibility and internationalisation support.
Full support for other color formats (HEX / RGB / HSL / CMYK) - this PR would replace https://github.com/wintercms/winter/pull/105 and fix https://github.com/wintercms/winter/issues/94
Will also fix https://github.com/wintercms/winter/issues/321
Will properly clean up and dispose of the widget as needed.

New config options

Option
Description

format
Specifies the color format to store. Can be one of cmyk, hex, hsl and rgb. Defaults to hex.

allowCustom
If false, only the colors specified in availableColors will be available for selection. The color picker palette will be disabled. Default true.

Notes

Given that we've dropped support for IE11 in the Backend, I've used some ES6 code (just arrow functions, really) to resolve some scoping issues in the JS, and the picker itself uses CSS Grid Layout. Both features, AFAICT, have been supported by all browser versions within the last 4-5 years, so I don't think we're leaving anyone behind with this.
This should be fully compatible with values from the old color picker, since they would've been stored as hex values. I also believe that it should be able to convert any value to another format if someone opts to change the format in the config, but I have not tested this yet.
I'll post up a video of this tomorrow, but feel free to have a play around with it on Gitpod in the mean time.

Something seems off about the font, is that the default font? Maybe we should use a mono space font instead

@LukeTowers yeah it's the default body font. I'll change it to monospace.

Could we also provide format: any to allow the user to pick which format they want to use?


Looking for a clear fix : UserWarning: NumPy linked against 'Accelerate.framework'

Hi all,
I am using dedupe to obtain a threshold for matches, running the following code, it fails to use multiprocessing:
threshold = deduper.threshold(full_dict, recall_weight=1)
(full_dict is my data)
The warning I get is:
UserWarning: NumPy linked against 'Accelerate.framework'. Multiprocessing will be disabled. http://mail.scipy.org/pipermail/numpy-discussion/2012-August/063589.html
warnings.warn("NumPy linked against 'Accelerate.framework'. "
not using multiprocessing it is painfully slow.  I am testing this on jupyter notebook with anaconda2 on mac. I know it would probably work if I switch to a linux server and not use anaconda but I was wondering if there is a fix for this anyway. There is no clear answer online that actually fixes this.
Thank you.
Closing. it seems that the only solution is explained here: https://dedupe.readthedocs.io/en/latest/OSX-Install-Notes.html

you are welcome, I hope you can get it working for you.


Update tutorial.md

I updated the proteogenomics  tutorial 3. Mostly fixed MVP to its latest version.

thanks @subinamehta! I will have a look ASAP

I thought I did add..but I will remove the old ones.
On Mon, Nov 4, 2019 at 11:56 AM Saskia Hiltemann<EMAIL_ADDRESS>wrote:

@shiltemann commented on this pull request.
In
topics/proteomics/tutorials/proteogenomics-novel-peptide-analysis/tutorial.md
https://github.com/galaxyproject/training-material/pull/1648#discussion_r342158338
:

-> {:width="50%"}
+> {:width="70%"}
don't forget to add these new images to the PR as well (and maybe remove
old ones if they are no longer used?)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/galaxyproject/training-material/pull/1648?email_source=notifications&email_token=AGP3A7IQVZSCDPS7FPYCP4TQSBH3XA5CNFSM4JIVXLCKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCKGL2YY#pullrequestreview-311213411,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGP3A7NZ3AIMXZKQEFLXBUTQSBH3XANCNFSM4JIVXLCA
.

--
Subina Mehta
Bioinformatics Researcher
Dept. of Biochemistry, Molecular Biology and Biophysics
University of Minnesota
7-166 MCB
420 Washington Ave SE
Minneapolis, MN 55455
Lab:<PHONE_NUMBER>
Phone:<PHONE_NUMBER>
Email<EMAIL_ADDRESS><EMAIL_ADDRESS>www.galaxyp.org http://www.galaxyp.org

@subinamehta thanks! (and let me know if you would like me to help with adding/removing the images)


Funnel for remote development

Add support for remote development in the VSCode Tailscale extension.

package.json

Add remote to extensionKind to specify the extension's capabilities for remote environments.

src/extension.ts

Modify the activate function to detect if the extension is running in a remote context by checking the vscode.env.remoteName property and adjust the behavior accordingly.
Update commands and UI elements to handle remote-specific scenarios, such as opening terminals and managing SSH configurations.

src/config-manager.ts

Add support for detecting the remote context and adjusting configurations accordingly in the ConfigManager class.
Update the Config interface to include properties specific to remote environments, such as remoteHost, remotePort, and remoteUser.
Modify the set and setForHost methods to handle remote-specific configurations and ensure that these methods can set and persist configurations for both local and remote environments.
Update the withGlobalStorageUri method to initialize remote-specific configurations if the extension is running in a remote context.

src/config-manager.test.ts

Create automated tests for the extension using vitest to cover different remote environments such as SSH, Dev Containers, and GitHub Codespaces.

.github/workflows/test.yml

Integrate the testing of remote environments into the existing continuous integration (CI) pipeline.
Add steps to the CI pipeline to set up and test the extension in SSH, Dev Containers, and GitHub Codespaces.

Didn't intend to create this merge request from Copilot Workspace pending manual human testing. Here's the link to the snapshot if needed.


9985a9e3-cffb-4013-b2a9-9abd4f3f7675

this is for automated testing

a8f88f6d-46a5-491e-8a36-b374600cf8bf


Rename gem

Q: Should I also rename the lib/health_monitor* files to lib/rt_health_monitor* in here?  Would make this a version 1.x release then (due to breaking change as the requires would be now different)

i guess its ok with the current naming :+1:


Room - When entering a "Space" while writing Room name "-" is not added

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: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Log in to NewDot with a High traffic account that owns a existing Workspace
Click on the green + button
Click on the "Send Message" option
Click the tab 'Room'
Type some text and hit "Space"

Expected Result:
When entering the room name, entering a "Space" adds a "-" to the name
Actual Result:
When entering a "Space" while writing Room name "-" is not added
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/93399543/486e28c9-14db-4dae-af65-ffa07aa4f7ce

MacOS: Desktop

View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

@kowczarz would you be able to take a look at this?

Kamil is OOO, I'm on it @puneetlath 😄

@BhuvaneshPatil we can make this a $250 job. If you're good with that, please share your proposal.

Ah interesting. If it created that many regressions then I agree it's better to revert.

@puneetlath I agree, let's revert and Kamil will be able to fix these once he is back.

We're reverting the offending PR

We can close this PR since we're CPing the revert


Use C# code instead of manually modified DLL.

Now that C# natively supports these access modifiers, we don't need to rely on a checked-in binary anymore.

AccessibilityTests.cs(14,27): error CS8107: Feature 'private protected' is not available in C# 7.0. Please use language version 7.2 or greater.

Good news is that the environment variable worked. Bad news is that either the C# syntax is wrong, the compiler is too old, or both. I don't know enough about the C# syntax to comment on that, but version 2.7.0 of Microsoft.Net.Compilers should be new enough for the added syntax.


[GovTool + PDF] Handle disconnected state of proposals

Context & versions

Steps to reproduce

On home page

Proposal card have a button labeled: Connect to propose - should be View proposals

On proposals page

User disconnects the wallet

User is redirected to home

Actual behavior

Proposal card on home have button Connect to propose
When user disconnects the wallet on proposals list, they are being redirected to home page

Expected behavior

Proposal card on home should have button View Proposals
When user disconnects the wallet on proposals list, they should stay on proposals page

@ValMBO @l-br1 I will also leave a comment here that we have everything we need on the ticket.
How does this affect the PDF, should all the actions that we have triggered the login so far be removed and disabled when the wallet is not connected? This is now important for us to know how to act when the PDF is visited without a wallet

Maybe they don't need to be removed, you could just make them executable only when walletAPI.isEnabled() returns true. If it is false we can allow user to display only the list of proposals without the possibility of submitting them on-chain

@teske00 Before introducing signData, what was the action that happened after you click "Propose GA" button?

I Marked the issue as blocked until this comment is resolved

If the wallet is not connected, all main actions like "add a comment" or "propose GA" should be disabled. Instead of deleting these actions, we should simply disable them. @l-br1 do you agree?

Hi guys, we had this discussion with Lorenzo and Bosco, and seems like there are 2 things here:

@MSzalowski : There is a misunderstanding. We should keep the "Connect to Propose" as it is now and then "Learn more" will be change to "view Poposals" (Ticket 1347)

@teske00 - If the wallet is not connected, User should be able to see all the proposals, all main actions like "add a comment" or "propose GA" should be disabled. However we should keep the option of connect your wallet in all those main actions.

@teske00 this is the example in Figma. Wallet is not connected, "Add a comment" is disabled and user are able to  connect their wallet by clicking the blue button

@ValMBO @MSzalowski I think that currently the opening of the wallet selection modal can only be triggered via govtool, not via PDF. The "Connect wallet" button is always visible in the govtool header, so I don't think there's any need to add it here?

Yes, this is the design from Figma, if the CTA is on the top, then is fine.

@ValMBO @MSzalowski I think that currently the opening of the wallet selection modal can only be triggered via govtool, not via PDF. The "Connect wallet" button is always visible in the govtool header, so I don't think there's any need to add it here?

That is right and I think that we should follow this approach in the PDF pillar as well for the consistency with the rest of the system. Rest of the points are clear meaning that I am removing the blocked label willing to prepare the solution on GovTool side. Thank you @ValMBO, @bosko-m  and @teske00

Based on the discussion on call (26th jun 2024) - I also added a Proposal nav item on top of the home page

Expected behavior
NOT OK Proposal card on home should have button View Proposals - still two buttons

OK When user disconnects the wallet on proposals list, they should stay on proposals page - Jam recording
Note: there is other 404 error on recording trancked here

@pmbinapps In regards to above mentioned NOT OK part, it is done according to the agreement as stated here - https://github.com/IntersectMBO/govtool/issues/1431#issuecomment-2189200591

Thanks @bosko-m for noticing this:
OK NOW - according to  commet "If the wallet is not connected, User should be able to see all the proposals, all main actions like "add a comment" or "propose GA" should be disabled. However we should keep the option of connect your wallet in all those main actions."


Error in detect-reboot script

The script used to detect the last time a system was restarted gives an error on my Windows 10 build (19045.2728 Dutch)
https://github.com/JayRHa/EndpointAnalyticsRemediationScripts/blob/main/Toast-RebootMessage/detect-reboot.ps1
The error is
`
Cannot convert value "1680444208,84456" to type "System.Int32". Error: "De indeling van de invoertekenreeks is onjuist."
At line:1 char:1

$diff = $now - $poweron`

It's triggered by the command  $diff = $now - $poweron, but the problem is probably related to the values of $now and $poweron.
The returned values in the original script are
1680444208,84456 for $now and<PHONE_NUMBER>,71884 for $differ.
If you replace the , with a . by changing the $now and $differ to these commands, it works.
$now = (Get-Date -UFormat "%s" -Date (Get-Date)) -replace (",",".")
$poweron = (Get-Date -UFormat "%s" -Date (Get-Process -Id $pid).StartTime) -replace (",",".")
New values are<PHONE_NUMBER>.84456 for $now and<PHONE_NUMBER>.71884 for $differ and then the calculations work.
I think it has something to do with the region we run in our enviroment Windows (Dutch), so the characters used for numbering might differ.

Considering it's now October 10th, 2023 (more than 6 months since you opened this issue), I'd imagine you've either figured out how to fix the issue or have moved onto something else entirely. Regardless, I just came across this repository a few minutes ago and almost immediately noticed the same issue(s) you're describing.
I rewrote the entire PowerShell script but I'll just share the fix for the specific problem you asked for here:
$ProgressPreference = 'SilentlyContinue'

# Define how often you want your devices to be rebooted
$RebootDeadlineInDays = 7

# Retrieve the Operating System's "Uptime"
$Uptime = Get-ComputerInfo -Property OSUptime | Select-Object OSUptime

if ($Uptime.OSUptime.Days -ge $RebootDeadlineInDays) {
    Write-Host "Operating System has been running for $($Uptime.OSUptime.Days) days! Reboot required."
    Exit 0
} else {
    Write-Host "Operating System was rebooted $($Uptime.OSUptime.Days) days ago! Compliance check: Pass."
    Exit 1
}

Hope that helps! For what it's worth, you can choose to use different measurements of time ("Hours", "Minutes", etc.) by selecting the associated property in the "$Uptime" variable. For example, if you wanted to determine how many hours ago a device rebooted, you could use the following code:
$UptimeInDays = $Uptime.OSUptime.Days
$UptimeInHours = $Uptime.OSUptime.Hours

if ($UptimeInDays -eq 0) {
    Write-Host "Device was rebooted $UptimeInHours hours ago!"
}

Thanks for the code, but I think this does not take Fast Boot into consideration.
With the original script it detects my system was last booted this morning, but when I run your code it tells me my system was last rebooted 12 days ago, so be carefull when you use this code. I use a different code these days, but when I try to paste it here it becomes quite unreadable

Thanks for the code, but I think this does not take Fast Boot into consideration. With the original script it detects my system was last booted this morning, but when I run your code it tells me my system was last rebooted 12 days ago, so be carefull when you use this code. I use a different code these days, but when I try to paste it here it becomes quite unreadable

That's fair. I have Fast Startup disabled across all devices in our environment so that's a use-case I never would've come across.
Out of curiosity, what's your result when using the following?
$DeviceLastBootupTime = Get-CimInstance -Class Win32_OperatingSystem | Select-Object -ExpandProperty LastBootupTime
$TimeDifference = (Get-Date) - $DeviceLastBootupTime
Write-Host "Device has been online for $($TimeDifference.Days) days, $($TimeDifference.Hours) hours and $($TimeDifference.Minutes) minutes."

The output is 'Device has been online for 13 days, 2 hours and 15 minutes.'. I know it technically has not rebooted since 13 days since it goes into hibernation, that's why we also disable fast boot so updates get installed properly.
Tried pasting my script again, but it won't get any better than this. Hope you can get the output from it you need with and without fast boot enabled
$Last_reboot = Get-ciminstance Win32_OperatingSystem | Select -Exp LastBootUpTime $Check_FastBoot = (Get-ItemProperty "HKLM:\SYSTEM\CurrentControlSet\Control\Session Manager\Power" -ea Silentlycontinue).HiberbootEnabled If(($Check_FastBoot -eq $null) -or ($Check_FastBoot -eq 0)){ 		$Boot_Event = Get-WinEvent -ProviderName "Microsoft-Windows-Kernel-Boot"| where {$_.ID -eq 27 -and $_.message -like "*0x0*"} 		If($Boot_Event -ne $null){ 			$Last_boot = $Boot_Event[0].TimeCreated		 			} 	} ElseIf($Check_FastBoot -eq 1){ 		$Boot_Event = Get-WinEvent -ProviderName "Microsoft-Windows-Kernel-Boot"| where {$_.ID -eq 27 -and $_.message -like "*0x1*"} 		If($Boot_Event -ne $null){ 				$Last_boot = $Boot_Event[0].TimeCreated		 			}			 	}	 If($Last_boot -eq $null){ 		$Uptime = $Last_reboot 	}Else	{ 		If($Last_reboot -ge $Last_boot){ 				$Uptime = $Last_reboot 			}Else{ 				$Uptime = $Last_boot 			}	 	}	 $Current_Date = get-date $Diff_boot_time = $Current_Date - $Uptime $Boot_Uptime_Days = $Diff_boot_time.Days	 $Hour = $Diff_boot_time.Hours $Minutes = $Diff_boot_time.Minutes $Reboot_Time = "$Boot_Uptime_Days day(s)" + ": $Hour hour(s)" + " : $minutes minute(s)"						 If($Boot_Uptime_Days -ge $Reboot_Delay){ 		write-host "$Reboot_Time"			 		EXIT 1		 	}Else	{ 		write-output "No Reboot needed"			 		EXIT 0 	}

I was able to reformat your script to make it readable. The trick to posting any multi-line code is to use three of the backticks before your script and three more after it. You were only using 1 backtick which tells MarkDown that you're wanting to display inline code and not a block of code.
I appreciate you posting your script, though! I hope you don't mind, I've taken it and restructured it to better fit my use-case. I'm posting it here in case you (or anyone else) wants to take a look!
$ProgressPreference = 'SilentlyContinue'
$ErrorActionPreference = 'Stop'
Write-Host "[ Get-DeviceLastBootupTimeWithFastStartup.ps1 ]" -ForegroundColor Green
Write-Host "Determining whether Fast Startup is enabled, please wait... " -NoNewline

# Define Variables
$RebootThresholdInDays = 7
$FastStartupRegistryKeyPath = "HKLM:\SYSTEM\CurrentControlSet\Control\Session Manager\Power"
$IsFastStartupEnabled = (Get-ItemProperty $FastStartupRegistryKeyPath -ErrorAction SilentlyContinue).HiberbootEnabled
Write-Host "Done!" -ForegroundColor Green

# Determine the last bootup time based on Fast Startup setting
$LastOSBootupTime = if ($IsFastStartupEnabled -eq 1) {
    Write-Host "[Fast Startup] Enabled"
    $LatestBootupEvent = Get-WinEvent -ProviderName "Microsoft-Windows-Kernel-Boot" | Where-Object { $_.ID -eq 27 -and $_.Message -like "*0x1*" }
    $LatestBootupEvent[0].TimeCreated
}
elseif (($null -eq $IsFastStartupEnabled) -or ($IsFastStartupEnabled -eq 0)) {
    Write-Host "[Fast Startup] Disabled"
    (Get-CimInstance Win32_OperatingSystem).LastBootUpTime
}
else {
    Write-Host "[WARNING] Unable to determine whether Fast Startup is enabled. Script exiting with exit code 1." -ForegroundColor Yellow
    exit 1
}

# Calculate OS Uptime
if ($null -ne $LastOSBootupTime) {
    Write-Host "[Last OS Bootup Time] $LastOSBootupTime"
    $OsUptime = (Get-Date) - $LastOSBootupTime
    $OsUptimeInDays = $OsUptime.Days
    $OsUptimeInHours = $OsUptime.Hours
    $OsUptimeInMinutes = $OsUptime.Minutes
    Write-Host "[Operating System Uptime] $OsUptimeInDays day(s), $OsUptimeInHours hour(s), $OsUptimeInMinutes minute(s)"

    # Check if reboot is needed
    if ($OsUptimeInDays -ge $RebootThresholdInDays) {
        Write-Host "Uptime threshold ($RebootThresholdInDays days) has been met or exceeded."
        Write-Host "[Result] " -NoNewLine
        Write-Host "Reboot required." -ForegroundColor Yellow
        exit 2
    }
    else {
        Write-Host "Uptime threshold ($RebootThresholdInDays days) has not been met."
        Write-Host "[Result] " -NoNewline
        Write-Host "No reboot required." -ForegroundColor Green
        exit 0
    }
}
else {
    Write-Host "[WARNING] Unable to determine how long the Operating System has been running. Script exiting with exit code 1." -ForegroundColor Yellow
    exit 1
}


Tagged test as integration

I found that setting the GITHUB4S_ACCESS_TOKEN allows the test to pass and not having it makes it fail, so I tagged the test as Integration
#300

@duanebester sorry I've been a bit out of this, could you please explain to me why this is working? Are those tests executed?


fix creation of reverse proxy

Reverse proxy creation reuses the httputil.NewSingleReverseProxy method's director to handle additional slashes

Pull Request Test Coverage Report for Build 905

11 of 12 (91.67%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.2%) to 90.588%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

api/osb/osb_controller.go
11
12
91.67%

Files with Coverage Reduction
New Missed Lines
%

api/osb/osb_controller.go
1
75.0%

Totals

Change from base Build 886:
0.2%

Covered Lines:
1819

Relevant Lines:
2008

💛  - Coveralls


mat: relax restrictions on vector orientation

Please take a look.
If I've missed any please let me know.
Fixes #388.

There are additional changes here that are needed to make shadow detection correct; striding is not correctly handled as it stands.
This is a note to myself.

PTAL

Sorry that I've been absent, I've been traveling the last few weeks. I'm
almost caught up on the backlog
On Sun, May 19, 2019, 11:54 AM Dan Kortschak<EMAIL_ADDRESS>wrote:

PTAL
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gonum/gonum/pull/985?email_source=notifications&email_token=AA4CUW66XKNIZBZHOZ3FCGTPWEWVDA5CNFSM4HNJR74KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVW7USA#issuecomment-493746760,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4CUW3QFJQ3XVELS646T4TPWEWVDANCNFSM4HNJR74A
.


Create Roles Admin Panel

Description:

As CLARISA user loged, I want manage roles for CLARISA Services so that I need to create a admin Panel to manage roles into CLARISA
Add Screenshot
Tasks:

[x] Migration
[x] Backend
[x] Frontend

Deliverable or Expected outcome:

[x] Service
[x] Website or section

Additional comments:

We created a mockup here , After aprovation of mockup, we developed a interface with all services with frontend components

after review the funtionality of this section, we proceed to close this issue

The table is divided into two tabs, to identify the deactivated and activated, in addition the functionality of activating the role is created


[MIRROR]  feat: add check for proper snake_case usage

This is a mirror of #13, as I do not have access to push to @simojoe's fork.

Closes #13.


contributor page made responsive

@Palak-137 Nice Addon

@all-contributors please add @Palak-137 for code


Revert PHP 7.1 support

7.1 was causing issues with building the PHAR and is just annoying to support in general.
With the fact that bumping symfony/panther to 7.2 is accepted (https://github.com/symfony/panther/pull/389#issuecomment-744043089), I am reverting the support for 7.1

@laurent35240 I'm sorry to say that I'm reverting #53. It has unfortunately caused issues with building the PHAR, which is essential to distributing this as a stand-alone tool.
The maintainer of symfony/panther has agreed to bump their required php version to 7.2, which means we no longer need to support 7.1.
Still, thanks for your work on it! It is still appreciated.

OK.
Sad that we did not that before.
But great anyway that minimum PHP version is 7.2. It avoids keeping some hacks in the code.


remove unsupported verbs

They aren't coming soon :)

Had some conflicts due to the way I set this up.
However, external to that, be sure to update the:

Summary
Root BXML

A good way to find if there are links is to search the entire project for the file name you deleted.
I've updated the conflicts and removed the links.

Thanks :)
[image: email-signature-logo.jpg]
Jennifer McLaughlin
Director, Product Management - Voice APIs
900 Main Campus Drive, Suite 400
Raleigh, NC 27606
Bandwidth
http://www.bandwidth.comm<PHONE_NUMBER>
e j email<EMAIL_ADDRESS>On Mon, Mar 13, 2017 at 11:09 AM, Daniel Tolbert<EMAIL_ADDRESS>wrote:

Had some conflicts due to the way I set this up.
However, external to that, be sure to update the:

Summary
https://github.com/Bandwidth/ap-docs/blob/bxml-updates/SUMMARY.md
Root BXML
https://github.com/Bandwidth/ap-docs/blob/bxml-updates/bxml/bxml.md

A good way to find if there are links is to search the entire project for
the file name you deleted.
I've updated the conflicts and removed the links.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Bandwidth/ap-docs/pull/19#issuecomment-286136255, or mute
the thread
https://github.com/notifications/unsubscribe-auth/APjV4fCzUwxGELYimAxi-GPWhXiZlvYhks5rlVwlgaJpZM4MZZyJ
.


Cannot set onUploadProgress

I didn't find an option of passing a config of axios request and because of that I'm unable to pass function that I need to fire on upload progress when doing post request.
Did I miss something or redux-axios-middleware can't handle this right now?

Same issue

You might want to take a look at this.
https://github.com/axios/axios#request-config


Feat epi composition

Type
enhancement, documentation

Description

Added a new IAM policy for EKS Pod Identity management and updated IAM policy versions.
Introduced a new Crossplane composition and definition for EKS Pod Identity management.
Updated Crossplane provider versions and added a new provider for AWS EKS.
Configured Flux for syncing the feat_epi_composition branch.

Changes walkthrough
Relevant filesEnhancement
3 files

irsa.tfAdd EKS Pod Identity Management IAM Policy

terraform/eks/irsa.tf
Added a new IAM policy crossplane_eks for managing EKS Pod identities.  Included the new crossplane_eks policy ARN in the role_policy_arns map.

+25/-0

epi-composition.yamlIntroduce EKS Pod Identity Composition

infrastructure/base/crossplane/configuration/epi-composition.yaml
Introduced a new Crossplane composition for EKS Pod Identity management named xepis.cloud.ogenki.io.  Defined steps for patching, transforming, and attaching additional policies in the composition.

+214/-0

epi-definition.yamlDefine Composite Resource for EKS Pod Identity

infrastructure/base/crossplane/configuration/epi-definition.yaml
Created a new CompositeResourceDefinition xepis.cloud.ogenki.io for EKS Pod Identity.  Specified the schema for the composite resource including parameters for cluster name, service account, and policy document.

+96/-0

Documentation
1 files

vault-clusterissuer.yamlUpdate Vault ClusterIssuer RoleId

security/base/cert-manager/vault-clusterissuer.yaml

Updated the roleId for the Vault cluster issuer to a new value.

+1/-1

Configuration changes
4 files

gotk-sync.yamlSetup Flux Sync for EPI Composition Branch

clusters/mycluster-0/flux-system/gotk-sync.yaml
Generated Flux GitRepository and Kustomization resources for syncing the feat_epi_composition branch.

+27/-0

kustomization.yamlUpdate Crossplane Configuration Kustomization

infrastructure/base/crossplane/configuration/kustomization.yaml
Included epi-composition.yaml and epi-definition.yaml in the Crossplane configuration.

+2/-0

kustomization.yamlInclude Provider AWS EKS in Providers Kustomization

infrastructure/base/crossplane/providers/kustomization.yaml
Added provider-eks.yaml to the list of resources in Crossplane providers' kustomization.

+1/-0

kustomization.yamlCreate Flux Kustomization for MyCluster-0

clusters/mycluster-0/flux-system/kustomization.yaml

Created a kustomization file for Flux resources in mycluster-0.

+5/-0

Dependencies
7 files

provider-ec2.yamlUpdate Provider AWS EC2 to v1.1.0

infrastructure/base/crossplane/providers/provider-ec2.yaml

Updated the version of the provider-aws-ec2 package to v1.1.0.

+1/-1

provider-iam.yamlUpdate Provider AWS IAM to v1.1.0

infrastructure/base/crossplane/providers/provider-iam.yaml

Updated the version of the provider-aws-iam package to v1.1.0.

+1/-1

provider-kms.yamlUpdate Provider AWS KMS to v1.1.0

infrastructure/base/crossplane/providers/provider-kms.yaml

Updated the version of the provider-aws-kms package to v1.1.0.

+1/-1

provider-rds.yamlUpdate Provider AWS RDS to v1.1.0

infrastructure/base/crossplane/providers/provider-rds.yaml

Updated the version of the provider-aws-rds package to v1.1.0.

+1/-1

provider-s3.yamlUpdate Provider AWS S3 to v1.1.0

infrastructure/base/crossplane/providers/provider-s3.yaml

Updated the version of the provider-aws-s3 package to v1.1.0.

+1/-1

provider-eks.yamlAdd New Provider AWS EKS

infrastructure/base/crossplane/providers/provider-eks.yaml
Introduced a new provider provider-aws-eks with version v1.1.0.

+10/-0

provider-kubernetes.yamlUpdate Provider Kubernetes to v0.11.4

infrastructure/base/crossplane/providers/provider-kubernetes.yaml
Updated the version of the provider-kubernetes package to v0.11.4.

+1/-1

 ✨ Usage guide:
Overview:
The describe tool scans the PR code changes, and generates a description for the PR - title, type, summary, walkthrough and labels. The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on a PR.
When commenting, to edit configurations related to the describe tool (pr_description section), use the following template:
/describe --pr_description.some_config1=... --pr_description.some_config2=...

With a configuration file, use the following template:
[pr_description]
some_config1=...
some_config2=...

  Enabling\disabling automation

When you first install the app, the default mode for the describe tool is:

pr_commands = ["/describe --pr_description.add_original_user_description=true"
                         "--pr_description.keep_original_user_title=true", ...]

meaning the describe tool will run automatically on every PR, will keep the original title, and will add the original user description above the generated description.

Markers are an alternative way to control the generated description, to give maximal control to the user. If you set:

pr_commands = ["/describe --pr_description.use_description_markers=true", ...]

the tool will replace every marker of the form pr_agent:marker_name in the PR description with the relevant content, where marker_name is one of the following:

type: the PR type.
summary: the PR summary.
walkthrough: the PR walkthrough.

Note that when markers are enabled, if the original PR description does not contain any markers, the tool will not alter the description at all.

  Custom labels
The default labels of the describe tool are quite generic: [Bug fix, Tests, Enhancement, Documentation, Other].
If you specify custom labels in the repo's labels page or via configuration file, you can get tailored labels for your use cases.
Examples for custom labels:

Main topic:performance - pr_agent:The main topic of this PR is performance
New endpoint - pr_agent:A new endpoint was added in this PR
SQL query - pr_agent:A new SQL query was added in this PR
Dockerfile changes - pr_agent:The PR contains changes in the Dockerfile
...

The list above is eclectic, and aims to give an idea of different possibilities. Define custom labels that are relevant for your repo and use cases.
Note that Labels are not mutually exclusive, so you can add multiple label categories.
Make sure to provide proper title, and a detailed and well-phrased description for each label, so the tool will know when to suggest it.

  Inline File Walkthrough 💎
For enhanced user experience, the describe tool can add file summaries directly to the "Files changed" tab in the PR page.
This will enable you to quickly understand the changes in each file, while reviewing the code changes (diffs).
To enable inline file summary, set pr_description.inline_file_summary in the configuration file, possible values are:

'table': File changes walkthrough table will be displayed on the top of the "Files changed" tab, in addition to the "Conversation" tab.
true: A collapsable file comment with changes title and a changes summary for each file in the PR.
false (default): File changes walkthrough will be added only to the "Conversation" tab.

  Utilizing extra instructions
The describe tool can be configured with extra instructions, to guide the model to a feedback tailored to the needs of your project.
Be specific, clear, and concise in the instructions. With extra instructions, you are the prompter. Notice that the general structure of the description is fixed, and cannot be changed. Extra instructions can change the content or style of each sub-section of the PR description.
Examples for extra instructions:
[pr_description]
extra_instructions="""
- The PR title should be in the format: '<PR type>: <title>'
- The title should be short and concise (up to 10 words)
- ...
"""

Use triple quotes to write multi-line instructions. Use bullet points to make the instructions more readable.

  More PR-Agent commands

To invoke the PR-Agent, add a comment using one of the following commands:

/review: Request a review of your Pull Request.
/describe: Update the PR title and description based on the contents of the PR.
/improve [--extended]: Suggest code improvements. Extended mode provides a higher quality feedback.
/ask <QUESTION>: Ask a question about the PR.
/update_changelog: Update the changelog based on the PR's contents.
/add_docs 💎: Generate docstring for new components introduced in the PR.
/generate_labels 💎: Generate labels for the PR based on the PR's contents.
/analyze 💎: Automatically analyzes the PR, and presents changes walkthrough for each component.

See the tools guide for more details.
To list the possible configuration parameters, add a /config comment.

See the describe usage page for a comprehensive guide on using this tool.

/describe


Add PR "Blocked" in QA Slack Notification

Hi there !
QA team would like to have on the daily QA Slack Notification a new info : The PR in status "Blocked".
Just like the PR with labels "Waiting for QA" + "Waiting for author", the PR with labels "Waiting for QA" + "Blocked" should not be counted in the "Ready to test" PR, nor in the "PR 1.7.8.x", "PR develop", "PR Modules" or "PR Specs" counts.
So to summurize, we would like to have :

A new category in the stats : "PR Blocked", with the following filter :

https://github.com/search?q=org%3APrestaShop+archived%3Afalse+is%3Apr+is%3Aopen+label%3A%22waiting+for+QA%22+label%3A%22blocked%22+-label%3A%22waiting+for+dev%22+-label%3A%22waiting+for+PM%22

The other filters should be (correct me if it's wrong :D ) :

PR 1.7.8.x : https://github.com/search?q=repo%3APrestaShop%2FPrestaShop+is%3Apr+is%3Aopen+base%3A1.7.8.x+label%3A%22waiting+for+QA%22+-label%3A%22waiting+for+author%22+-label%3A%22waiting+for+dev%22+-label%3A%22waiting+for+PM%22+-label%3A%22blocked%22
PR develop : https://github.com/search?q=repo%3APrestaShop%2FPrestaShop+is%3Apr+is%3Aopen+base%3Adevelop+label%3A%22waiting+for+QA%22+-label%3A%22waiting+for+author%22+-label%3A%22waiting+for+dev%22+-label%3A%22waiting+for+PM%22+-label%3A%22blocked%22
PR Modules : https://github.com/search?q=org%3APrestaShop+archived%3Afalse+-repo%3APrestaShop%2FPrestaShop+-repo%3APrestaShop%2Fprestashop-specs+is%3Apr+is%3Aopen+label%3A%22waiting+for+QA%22+-label%3A%22waiting+for+author%22+-label%3A%22waiting+for+dev%22+-label%3A%22waiting+for+PM%22+-label%3A%22blocked%22
PR Specs : https://github.com/search?q=repo%3APrestaShop%2Fprestashop-specs+is%3Apr+is%3Aopen+label%3A%22waiting+for+QA%22+-label%3A%22waiting+for+author%22+-label%3A%22waiting+for+dev%22+-label%3A%22waiting+for+PM%22+-label%3A%22blocked%22

And the "Blocked" PR should also not be counted in the "PR Ready to Test" block.

Do you think it's possible ?
Thanks !

@Robin-Fischer-PS Done in #43

Thanks a lot !


Make doclinks use relative URLs

This makes doclinks use relative URLs, which works since we include site-url in _quarto.yml, fixing #696. It's actually only a one-line change in doclinks.py - the rest are tests and modidx. cc @hamelsmu @jph00

Does this impact nbdev-stdlib and the other indexes at all?
Does this impact the ability to use this project's index on other projects?

Ah, I overlooked that! I've made this a draft PR while I look into it. My gut feeling is that it's not actually a quick fix then

Actually I'll close and open a new PR when appropriate


Add support for H-series 2014 TVs

H-Series from 2014 require a PIN to authenticate the remote with the TV, which is not supported by this plugin.
See here for reference info:
https://imkapps.com/mytifi/manual/#Use_with_H-Series_or_J-Series
The plugin needs to be adapted to support authentication for the H-Series
See issue https://github.com/jsiegenthaler/homebridge-samsungtvht/issues/2 where the issue of connecting to an H-series was discovered

Thanks @jsiegenthaler this is great!
Happy to help in any way I can :)

@ChrisTav424
What's your skillset? Are you familiar with coding javascript?
To support the H-series I have to move to a different node module for the remote control, and as I do not have a H-series TV myself, makes it very difficult for me to test it.
One of the easiest ways to find what works is for you to try various other Samsung Homebridge plugins until you find one that works, and then let me know which one it is. I can then research that plugin and see how the PIN code and encryption is handled.

Thanks @jsiegenthaler, I tried using the myTiFi app this week and it doesn't work so I think this is a non starter for my TV, but I do want to say thank you so much for all your help :)

OK. if myTifi doesn't work, then there's no way I can get it to work.
I'll close this issue


raftstore-v2: fix merge block if MsgAvailabilityRequest is dropped

What is changed and how it works?

Issue Number: Close #15651
What's Changed:

Before proposing prepare merge, source region leader requires all peers
of source region and target region responses to MsgAvailabilityRequest
so it can check if all peers are trimmed. However message can be lost,
and this blocks merge in the current implementation.

This commit fixes the issue by retry sending MsgAvailabilityRequest.

Related changes

Need to cherry-pick to the release branch

Check List
Tests

Integration test

Release note
None

Do we need to limit the max retry times?


Script always fails with error "Storybook object not found on window. Open your storybook and check the console for errors."

When running my CLI command (regardless of config), the cli always hangs with the following output:
$ npm run test:a11y -- --headless false --browser firefox

><EMAIL_ADDRESS>test:a11y /Users/karrui/Projects/FormSG/frontend
> axe-storybook "--headless" "false" "--browser" "firefox"

[firefox] accessibility
Error! The suite failed to run
An error was encountered before we started testing stories. Likely this means the browser failed to open.

Error: Storybook object not found on window. Open your storybook and check the console for errors.

0 passing (0ms)
1 failing
0 pending

Awesome, it works now. Once again, thanks for such a quick response!


Correct definition of MultipleOf and ExclusiveMaximum.

Correct definition of MultipleOf and ExclusiveMaximum. Both should be float64 according to http://json-schema.org/draft/2020-12/json-schema-validation.html#name-multipleof
fix: set exclusiveMinimum to float64 as well

Replaces #116 (just a rebased from main and added one missing type conversion)


docs: create-help-for-the-non-technical.md

What/Why/How?
A new guide specifically for non-developers, such as tech writers, who wish to use Redocly CLI.
Reference
Testing
Screenshots (optional)
Check yourself

[ ] Code is linted
[ ] Tested with redoc/reference-docs/workflows
[ ] All new/updated code is covered with tests

Security

[ ] Security impact of change has been considered
[ ] Code follows company security practices and guidelines

Hi team @adamaltman @JodyWinter, I've managed to carve out some time for open-source contributions :blush: so I'd like to try working on this project again. Before I go in, I'd just like to check if you need any feedback on this PR or is it considered stale at this point? Thanks!

Hi @ivana-isadora. Welcome back! This was turned into my three-part blog series that was published recently, so I guess this can be closed...?

Welcome back! This was turned into my three-part blog series that was published recently, so I guess this can be closed...?

Thanks, Jody! Okay, in that case there's nothing for me to do here. I'm sure there's plenty of other stuff I can work on. :smile:


Expand datasets function in side panel

From Rick:
The first element in the sidebar "Datasets Available" doesn't actually do anything.  If you want to view some stars, you have to do one of two things,  you can go to General/Plot Stars and select a loaded dataset and a radius, or you can go to search and query a loaded dataset and radius.  Datasets available doesn't even show you which one you're actually seeing in the plot window, it just shows you what's available. I see two potential solutions: 1) Dump it.  2) Change it so it highlights somehow the currently active dataset.
I will put a highlight on the current dataset for 0.3.That may be bumped to 0.4

I've added more information so that it says the name and how many stars are in the set.
There is also a tooltip that if you hover tells you more about the dataset
and I added a context menu that allows you to plot, display data or do both based on the current query settings
there is also a delete function that i disabled
this will be in 0.5


Upgrade - Improve the way of enabling the "backup/restore" feature on existing clusters.

Rancher server version - Fresh install of rancher server version - v2.17  upgraded to 2.2 master builds.
Currently there is no straight forward way to enable the new "backup/restore" feature on existing cluster in upgraded setup.
The only way this feature can be turned on is , for the  users to  edit the cluster and "Save" , the new backup/restore feature in the UI will be available (There is a message that is presented to the user).
If the users do not edit the cluster , they will continue to be using the legacy backups.
Questions:

Why can this feature not be turned on automatically on  existing clusters as part of rancher server uprgade ? Is it because we may not be able to do exact conversion of the settings ? retention": "72h"/"creation": "12h" in  2.1.7 to "intervalHours": 12, "retention": 6 in 2.2.

If we are not able to turn on this feature automatically on  existing clusters as part of rancher server upgrade , Can the user be provided with an option at the cluster level which when selected would enable this feature( similar to what happens on the edit cluster currently).

We could continue to have current way of making an attempt to turn on this feature on edit cluster , if needed .

Why can this feature not be turned on automatically on existing clusters as part of rancher server upgrade ? Is it because we may not be able to do exact conversion of the settings ?

Yes, it is mostly because it is not easy to convert. Plus we usually try to avoid any auto updates for a cluster.

If we are not able to turn on this feature automatically on existing clusters as part of rancher server upgrade , Can the user be provided with an option at the cluster level which when selected would enable this feature( similar to what happens on the edit cluster currently)

We do have a way to convert from an old snapshot setting to the new ones. @westlywright can you confirm

@alena1108 I misunderstood when we talked offline, we do not offer a way at the cluster level to upgrade just this setting, the only time this setting is changed is when the user opt's into upgrading the cluster. I think what sangeetha is asking is that we showcase some kind of warning and possible action if the user is at the cluster level and just wants to change this particular cluster setting without editing. @sangeethah is that correct?

we just need to document and put in the release notes

closing. the act of releasing noting it is tracked separately


Add renderHeader and renderFooter

This allows users to add captions and style around the image itself. Currently working in branch render-header-footer, expecting to release sometime next week after proper testing is in place.

#25 Closed


No MediaQuery ancestor could be found starting from the context that was passed to MediaQuery.of().

This is breaking whole application.
This kind of updates tested first and release.
please add solution if any one find.
@override
  Widget build(BuildContext context) {
    return DevicePreview(
      enabled: kDebugMode,
      builder: (context) => ScreenUtilInit(
        designSize: const Size(411, 683), // with reference of pixel 2 5.0"
        minTextAdapt: true,
        splitScreenMode: true,
        builder: () => GetMaterialApp(
          useInheritedMediaQuery: true,
          locale: DevicePreview.locale(context),
          builder: DevicePreview.appBuilder,
          debugShowCheckedModeBanner: false,
          initialRoute: AppRoutes.home,
          getPages: AppRoutes.routes,
          theme: lightTheme,
          darkTheme: darkTheme,
          themeMode: ThemeMode.light,
        ),
      ),
    );
  }

Experiencing the same problem on version 5.1.0. Downgrading to version 5.0.4 fixed this issue for me.

@lizhuoyuan please suggest or add working solution as most of developers getting same error after upgrading to 5.1.1
I have tried both solutions mentioned in document but not working.
I have static file which has all Dimensions with static key

Duplicate od https://github.com/OpenFlutter/flutter_screenutil/issues/329

Remove device preview and wrap Screen Util with MaterialApp. That will fix the crash.

// main.dart
void main() {
  runApp(const APP());
}

class APP extends StatelessWidget {
  const APP({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return GetMaterialApp(
      builder: (context, widget) {
        return MediaQuery(
          data: MediaQuery.of(context).copyWith(textScaleFactor: 1.0),
          child: widget!,
        );
      },
      home: ScreenUtilInit(
        designSize: const Size(360, 690),
        minTextAdapt: true,
        splitScreenMode: true,
        builder: () {
          // ScreenUtil.setContext(context);
          return Theme(
            data: ThemeData(
              textTheme: TextTheme(button: TextStyle(fontSize: 45.sp)),
            ),
            child: SignPage(),
          );
        },
      ),
    );
  }
}

//sign_page.dart
class SignPage extends StatelessWidget {
  SignPage({Key? key}) : super(key: key);

  final ctrl = Get.put(SignCtrl());

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(
          "汉字 ABC abc !@#%^&",
          style: TextStyle(fontSize: 10.sp),
        ),
      ),
    );
  }
}

这样可以解决问题,我想可能是(flutter2.8)的问题，更改了某些渲染问题?
This can solve the problem. I think it may be the problem of (fluent 2.8), which has changed some rendering problems?

This was absolutely destroying me tonight and I thought it was something I did.  So glad to find this thread.

Remove device preview and wrap Screen Util with MaterialApp. That will fix the crash.

I want device preview to test multiple device UI behaviour. So I can't remove it.

add     ScreenUtil.setContext(context); in builder
  @override
  Widget build(BuildContext context) {
    return ScreenUtilInit(
      designSize: const Size(360, 690),
      minTextAdapt: true,
      builder: () {
        return GetMaterialApp(
          builder: (context, widget) {
            **ScreenUtil.setContext(context);**
            return MediaQuery(
                data: MediaQuery.of(context).copyWith(textScaleFactor: 1.0),
                child: widget!);
          },
          debugShowCheckedModeBanner: false,
          theme: settingService.getThemeData(context),
          home: SplashPage(),
        );
      },
    );
  }

Found the fix
Main.dart
void main() {
WidgetsFlutterBinding.ensureInitialized();
SystemChrome.setPreferredOrientations([DeviceOrientation.portraitUp, DeviceOrientation.portraitDown])
.then((_) {
runApp(MaterialApp(
debugShowCheckedModeBanner: false,
home: App()));
});
}
App.dart
return   ScreenUtilInit(
builder: (){
return MaterialApp(
theme: ThemeData(
fontFamily: "LatoRegular"
),
debugShowCheckedModeBanner: false,
routes: {
'/splash':(context)=>SplashScreen()
},initialRoute: "/splash",
);
},
//designSize: Size(screenWidth,screenHeight),
);


Failed to initialize (beginners error)

I am trying to get started using buzz and argos an am having a rough time of it. I am able to successfully run the argos tutorials but have run into issues while trying to run the buzz tutorials. I believe it is a library issue but I am not experienced enough to figure out the problem.
Here is my console output while trying to run gradient.bzz ( Ubuntu 16.04 WSL):

Gradient.bzz: (.txt)
gradient_bzz.txt
Gradient.argos: (txt)
gradient_argos.txt
Initially I thought it was because buzz was installed in a different dir than argos, while that did not seem to be the issue, I re-installed so the file tree is now as follows:
rootfs/usr/lib/argos3/argos3-examples; buzz; buzz-examples
Thanks for the help and sorry for such a basic question

Thank you for your interest in Buzz!
You did almost everything right. I had to just add a couple of lines in the .argos file to have your setup working.
The issues that I had to fix are:
As you see in the error message, Argos couldn't initialize the LEDs actuator because it couldn't find the right medium to simulate the LEDs.
These mediums in Argos are somewhat special, they simulate the propagation of forms of energy like radio waves, light and etc. You could basically think about it as some kind of medium like the air that carries radio waves or light from a source to all the other parts of the environment.
So, whenever you use some kind of actuator that dissipates some kind of energy either in the form of light or radio waves you need to add a medium associated with it, in order to simulate the propagation of that entity.
To find out if the actuator you use needs a medium you could try argos3 -q "NAME_OF_THE_ACTUATOR" (I think you know this, I remember seeing this in your comments).
You basically forgot to add these mediums in the .argos file for two of the actuators you were trying to use: 1. range and bearing and 2. LEDs.
I also placed the robots a bit close, so they will not be obstructed by the walls.
I am also attaching the .argos file and the .bzz file for your reference.
gradient.txt
gradient_bzz.txt
Good Luck!
And if you have any further questions feel free to post them, we will get back to you as soon as possible.

Wow, I knew it was something simple that I would never figure out. Thanks for the help!


Various notebook links lead to 403 or 404 pages

From the Gradient interface they result in, e.g.,

The commits pushed by @evawGraphcore yesterday should have fixed this

Sounds good, will check then close

I'm seeing them fixed in the HF runtime but not this one


Allow for the PluginError constructor to be called without new.

This also falls in line with the builtin Error types and most of the other builtins as well.

This will add a new item to the error stack and push other stuff off. You can manipulate the stack to remove PluginError from it

Closing, as work shouldn't be done in this module anymore.


Alpine Linux

Is it possible to use it on alpine linux? On debian its working fine.

If it uses systemd and the packages names still the same, probably yes.
Check the install script and all dependencies script need to install. If the installation codes are the same, probably might work.
Recommend a test machine first.
Regards


must follow fvar axes order when building gvar GlyphDeltas

TupleBuilder was iterating over VariationRegion's axis_tents which is a BTreeMap, but it should instead iterate over the region's tents following the fvar's axis order, that's what the gvar Tuples must follow. Otherwise we store deltas for the wrong axes!

I wonder if VariationRegion should be changed to store tents in axis order

I wondered the same, maybe we can revise this later. I'll merge this for now


Program does not work?

When copy pasting the example from crates.io output is "<< was unexpected at this time."
None of the examples listed work for me....

i was using windows, works on WSL tho.
maybe add windows CMD incompatability to readme.


[B2B3-550] hide empty ad

as related to hiding pencil ad on homepage redesign
https://github.com/weddingwire/weddingwire-ng/pull/7655

@chancedowns can you take a look?

Looking at the docs, there are a few options. It looks like you picked the global option instead of per slot, do you think that's the best? I'm not too familiar with it, so I don't know the reasoning behind one vs the other

no reasoning really ... i'll look through the docs again

closing this ... @americos is working to modify the helper to collapse at the slot level instead of the global level


Feature make spdk optional v2

Based on https://github.com/yoursunny/setup-dpdk/pull/1 I was unaware the line below is connected to SPDK.
  sudo scripts/pkgdep.sh
Moved it to the optional SPDK block.

Please push (or force push) subsequent   revisions to the same branch used in your original Pull Request #1, instead of creating a new branch and new Pull Request for each revision.


st2sensorcontainer is not running.

"st2ctl reload --register-all" lists that "st2sensorcontainer is not running." ... this means sensors won't work right ??
in the logs i can see
AUDIT access [-] Access granted to "sensors_container" with the token set to expire at "2018-01-24T05:04:39.725464Z". (username='sensors_container',token_expiration='2018-01-24T05:04:39.725464Z')

anyway, I can get this into working condition??

What does st2ctl status report?
If st2sensorcontainer is not running, you can restart it with sudo st2ctl restart-component st2sensorcontainer. You may also want to look at the log file for that service (/var/log/st2/st2sensorcontainer.log) to see what is happening with it.

I did that and it restarted the process... but looks like (username='sensors_container',token_expiration='2018-01-24T05:04:39.725464Z') is not letting sensor to run...
any way I can reset this, means expiration date or auth key?

It looks like that the auth token used by sensor container process has expired.
We generate auth token which is used for sensor container purposes on sensor start up so as @LindsayHill has said, restart should do the trick (new token should be generated).
Can you please gist whole sensor container log file so we can see what is going on. The log line and date you posted is a bit confusing / don't make sense since it's almost a month old. Again, hopefully seeing the whole log file will clear things up.


Update PYPI

Hi @kamo-naoyuki , could you please update the latest version of this repo to the Pypi? I'm working on some demonstrations for espnet2 and would like to include some latest models, which unfortunately is not shown in the pip version. Many thanks!

sorry, I forgot it. I'll do

Done

Many thanks!


[infra] Update temple test to allow templefiles without auth services

The way temple-test had been written it was always assumed an auth service existed, so would try and get an auth token before starting, now we skip that logic if there's no auth service generated

Closes #290


refactor: don't nest md-content elements

It would only be suitable for locations where we want to have multiple scrollbars (e.g. in dialogs, where the outer scrollbar is disabled). I quickly checked and the site worked in the same way with this change. I think a md-content without a defined height is basically the same as a div.
Reference: https://material.angularjs.org/latest/api/directive/mdContent
Together with #312 it would also mean that we should only use md-content in md-dialog, if necessary. @D0nPiano what do you think about this?

Codecov Report

Merging #313 into master will decrease coverage by -0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #313      +/-   ##
==========================================
- Coverage   98.68%   98.67%   -0.02%
==========================================
  Files         152      148       -4
  Lines        2213     2185      -28
==========================================
- Hits         2184     2156      -28
  Misses         29       29

Impacted Files
Coverage Δ

client/app/components/signup/signup.controller.js
44.44% <ø> (-5.56%)
:x:

.../group/_groupEditCreateForm/groupEditCreateForm.js
100% <ø> (ø)
:white_check_mark:

client/app/components/signup/signup.js
100% <ø> (ø)
:white_check_mark:

client/app/components/signup/signup.spec.js
100% <ø> (ø)
:white_check_mark:

...mponents/_markdownInput/markdownInput.component.js

...pp/components/_markdownInput/markdownInput.spec.js

...ponents/_markdownInput/markdownInput.controller.js

...ent/app/components/_markdownInput/markdownInput.js

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 730ebf4...a8d7c5b. Read the comment docs.

I think it sounds reasonable - and having nested scrollbars really is a pain in many cases, so I'm for your proposal. I'll look through the changes tonight.

The changes look good too (:


[Feature] Build multiple test images

Is your feature request related to a problem? Please describe.
We build a single kernel64.bin image right now. We might want to build multiple different demo images. (In fact, we already have two images, one for KTF_UNIT_TEST=true and one for KTF_UNIT_TEST=false -- this is hard-coded into test.c right now.)
Describe the solution you'd like
We could have tests/ subdirectories and build separate images per subdirectory. I'm happy with other alternatives, too.

I am not against it per-se, but I am thinking of embedding all the test into a single image and making them selectable. Alternatively, we could make the tests load able as modules (via multiboot for instance) , build KTF core and run it with selected modules.

We should make sure to support automated test setups that run KTF without
user interaction. Selecting tests after boot would make for a nice demo,
but is rather blocking real life usage.
I agree we could still build a single iso that has all the tests selectable
via grub and that could be automatically used by setting the respective
default entry.

We should make sure to support automated test setups that run KTF without
user interaction. Selecting tests after boot would make for a nice demo,
but is rather blocking real life usage.
I agree we could still build a single iso that has all the tests selectable
via grub and that could be automatically used by setting the respective
default entry.

Yup. The latter is what I meant. We already have cmdline options support to enable that.
But, the ability to build different images based on tests/ might still be useful.


Jenkins publisher plugin sonar publish step issue

Hello,
I generated the sonarpublishbuildstep using pipeline syntax generator and using that in my jenkinsfile to push sonar metrics to hygieia but it fails to do and throws the below error
``java.lang.StringIndexOutOfBoundsException: String index out of range: -1
at java.lang.String.substring(String.java:1967)
at hygieia.builder.SonarBuilder.getSonarMetrics(SonarBuilder.java:192)
at jenkins.plugins.hygieia.workflow.HygieiaSonarPublishStep$HygieiaSonarPublishStepExecution.run(HygieiaSonarPublishStep.java:123)
at jenkins.plugins.hygieia.workflow.HygieiaSonarPublishStep$HygieiaSonarPublishStepExecution.run(HygieiaSonarPublishStep.java:81)
at org.jenkinsci.plugins.workflow.steps.AbstractSynchronousNonBlockingStepExecution$1$1.call(AbstractSynchronousNonBlockingStepExecution.java:47)
at hudson.security.ACL.impersonate(ACL.java:290)
at org.jenkinsci.plugins.workflow.steps.AbstractSynchronousNonBlockingStepExecution$1.run(AbstractSynchronousNonBlockingStepExecution.java:44)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at java.lang.Thread.run(Thread.java:748)

@vbottu What version of the plugin are you using?

2.2.0-SNAPSHOT

Can you paste the screenshot of the Hygieia Plugin Settings from Jenkins Global Settings

@vbottu can you get the latest code and retry. I believe this issue should be fixed now

This issue has been resolved.

Hi there, I am having exactly the same issue with version 2.2.3, is there a more recent version I should be using?

Just tried rebuilding the .hpi from source and adding the plugin again, still the same issue :(  @rvema any chance this issue can be re-opened?

@keir-lavelle let me get back to you


Incorrect comparison to global 'nsplt' instead of local ksplt(k) in fv_tracer_2d

I believe I found a bug in fv_tracer2d.F90 when filling the optional dpA in the subroutine tracer_2d. A comparison is made to the global nsplt when it should be compared to the local ksplt(k) value. This leads to dpA not being fully filled if ksplt for that level is less than nsplt
This:
https://github.com/GEOS-ESM/GEOSgcm_GridComp/blob/f5fd1e30f4c6ee376c6f799b32cb71e606335dd4/GEOSagcm_GridComp/GEOSsuperdyn_GridComp/FVdycoreCubed_GridComp/fvdycore/model/fv_tracer2d.F90#L546
Should be:
if ( it /= ksplt(k)) then

because if ksplt(k) == 1 for that level, and nsplt == 2, it will never be 2 because the outer if ( it .le. ksplt(k) ) will fail and therefore dpA won't be assigned.

Assigned to @wmputman .

Kyle, how would this bug manifest itself in a simulation? The version of FV3 in GEOS-Chem High Performance (GCHP) may need the fix but I am uncertain of the priority without assessing impact. Do you have test results you could share?

I ran into this when running the GEOS-CTM at resolutions greater than C48. The code would just crash in offline_tracer_advection with floating invalid when using dpA to calculate PE because it was half-filled with NaNs.

It's the correct change, i don't test this case very often if ever since we most exclusively run GEOS with z_tracer = T so we always use the tracer_2d_1L routine above this.

This fix appears to correct the issue I was having in GEOS-Chem with this version of FV3 where concentrations were blowing up at our default dynamic timestep. I was getting around it by reducing the timestep but now this is no longer necessary. Thank you!!


Memory leak in finalizing GeneratorDataset iterator

System information

OS Linux Ubuntu 18.04
TensorFlow installed from `pipenv install tensorflow~=2.1
TensorFlow version 2.1.0
Python version: 3.6
CUDA/cuDNN version: 10.1
GPU model and memory: 2 x (RTX 2080 Ti 11GB)

Describe the current behavior
tf.keras.preprocessing.image.ImageDataGenerator output warning message Error occurred when finalizing GeneratorDataset iterator: Cancelled: Operation was canceled when I am using tf.keras.model.fit(..., worker = 10) start to train to next epoch. The warning is gone when worker = 1. However, the warning does not seem to affect the result.
In #35100, people tend to mean the warning is superfluous and spurious and the warning has been removed in tf-nightly. But after my usage case, I believe the warning is correct. I have infinite models to be trained one by one (generated by a genetic algorithm). When tf.keras.model.fit(..., worker = 10) the error message print out and the memory does not get freed in the next epoch. After training a few models.  All the memory is occupied and the process gets killed. It does not happen when worker=1 the memory gets freed for each epoch.
I have try tf-nightly, The error message is removed but the memory problem is still there.
Describe the expected behavior
It should free the memory for every epoch.
Code to reproduce the issue
I prepare my data in the following way.
samples is NumPy array with shape (n, 90, 90, 3). n'th image withe 90 by 90 RGB image.
labels is NumPy array with shape (n, 2), which is yes and no. [1, 0] is yes and [0, 1] is no.
I have to make a pipenv file, data set and small script for you to demonstrate the memory problem.
Download link from google drive
(300 MB)
Unzip the link above and run
pipenv install
pipenv shell
python main.py
Then you see the warning
train_generator = data_aug.flow(samples, labels, batch_size = 64)
self.model.fit(train_generator, epochs=70, max_queue_size = 64, workers = 10)

Other info / logs
During these 70 epochs, you can see the memory go up for each epoch and print the error message every time between the epoch.

@Mauhing
Could you please share colab link or simple standalone code with supporting files to reproduce the issue in our environment. It helps in localizing the issue faster. Thanks!

@Mauhing
Could you please share colab link or simple standalone code with supporting files to reproduce the issue in our environment. It helps in localizing the issue faster. Thanks!

I have to make a pipenv file, data set and small script for you to demonstrate the memory problem.
https://drive.google.com/file/d/1oQ1wQkY1ZXimQNBQljlMbNB4VNYjDAp7/view?usp=sharing
(300 MB)
Unzip the link above and run
pipenv install
pipenv shell
python run
Then you see the warning
The problem occurs when you train it on GPU. The warning shown twice when you train on two GPU at each epoch.

Was able to reproduce the issue on colab. Please find the gist here. Thanks!

I want to make a kindly reminder that the colab code does not seem to print out the memory leak in each epoch. But the memory does not get free in each epoch.  Thanks

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


Spark Driver and Executors cannot be configured with less than 1 core

By documentation, spark driver task should be able to be configured with a number smaller than 1:
For example in https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/blob/master/docs/user-guide.md#requesting-gpu-resources it says:
spec:
  driver:
    cores: 0.1
    coreLimit: "200m"
    memory: "512m"
    gpu:
      name: "amd.com/gpu"   # GPU resource name
      quantity: 1           # number of GPUs to request
    labels:
      version: 3.1.1
    serviceAccount: spark
  executor:
    cores: 1
    instances: 1
    memory: "512m"
    serviceAccount: spark
    gpu:
      name: "nvidia.com/gpu"
      quantity: 1

I'm trying to run a spark job with spark driver cores 0.25 and the next error appears:
spec.driver.cores: Invalid value: 0.25: spec.driver.cores in body should be greater than or equal to 1

So, how should we configure the driver/executors cores? Also looks a bit inconsistent that the cores should be configured with an integer value but the coreLimit and the memory with string values.

you can use the coreRequest and coreLimit instead
executor:
coreRequest: "100m"
coreLimit: "1000m"

hey @cchenna, do you know if there is any way to set a request and limit the memory of the executors? I've seen there is only one field called memory but there is no memoryRequest and memoryLimit options. Thx!!

We don’t, it does guaranteed scheduling. Memory Request and limit is set to the same value.


CPUCollector doesn't honor percore config variable for psutil collections

If you don't have /proc/stat present, CPUCollector will fallback to using 'psutil' to get the CPU data.
Unfortunately, only the /proc/stat codepath checks the 'percore' config variable.
Easy fix; will push a one shortly.

This repo has moved to https://github.com/python-diamond/Diamond
Please open a new ticket there if this issue is not resolved with the current code there.
Thanks!


linuxKernel.kernels.linux_zen: 6.9.8-zen1 -> 6.10.1-zen1 ; linuxKernel.kernels.linux_lqx: 6.9.8-lqx1 -> 6.9.11-lqx1

Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

CC @thiagokokada for review

Please ignore the committer name. Need to look into what my git borked again.

Please ignore the committer name. Need to look into what config my git borked again.

Can you fix the commit e-mail? I think merging as-is currently can cause some issues since this looks like a valid address.

FTFY.
You need to set
[user]
	email =<EMAIL_ADDRESS>	name = JerrySM64

Or use your actual email that is also connected with your GH account (though note that git emails are and will forever be public).


[Bug]: If used with sqlcmd 18 encryption error is returned. 17 is fine

What happened?
If i run simple SELECT 1 under sqlcmd 18 i see following:
Sqlcmd: Error: Microsoft ODBC Driver 18 for SQL Server : Encryption not supported on SQL Server..
Sqlcmd: Error: Microsoft ODBC Driver 18 for SQL Server : Client unable to establish connection.
I use following sqlcmd on alpine:
https://download.microsoft.com/download/8/6/8/868e5fc4-7bfe-494d-8f9d-115cbcdb52ae/msodbcsql18_<IP_ADDRESS>-1_amd64.apk
https://download.microsoft.com/download/8/6/8/868e5fc4-7bfe-494d-8f9d-115cbcdb52ae/mssql-tools18_<IP_ADDRESS>-1_amd64.apk
I run sqlcmd with -C flag which trusts any server certificate
Version
2.1.1 from https://hub.docker.com/layers/jonathanpotts/babelfishpg/2.1.1/images/sha256-981aba45e0b805126a66845e72a6b5253daad65ffe1d233fc3237d3cac59cafe?context=explore
Dockerfile: https://github.com/jonathanpotts/docker-babelfishpg/blob/main/Dockerfile
Which flavor of Linux are you using when you see the bug?
Ubuntu (Default)
Relevant log output
Sqlcmd: Error: Microsoft ODBC Driver 18 for SQL Server : Encryption not supported on SQL Server..
Sqlcmd: Error: Microsoft ODBC Driver 18 for SQL Server : Client unable to establish connection.

Code of Conduct

[X] I agree to follow this project's Code of Conduct.

It is true that the Version 18 of the Microsoft ODBC Driver for SQL Server comes with breaking changes:
https://techcommunity.microsoft.com/t5/sql-server-blog/odbc-driver-18-0-for-sql-server-released/ba-p/3169228
However, when I use this new ODBC version with Babelfish for Aurora PostgreSQL for both 2.1.2 and 2.2.0 releases, I am able to successfully connect to the server from the sqlcmd utility.
Could you review this page to see whether the server is setup correctly for encrypted connections?
https://babelfishpg.org/docs/usage/security/

Thanks for quick response. I took latest 2.2.0 docker image from https://hub.docker.com/r/jonathanpotts/babelfishpg and enabled SSL in config, and it failed to start with error
SSL is not supported by this build
Can it be that babelfish in image is compiled without SSL?
https://github.com/jonathanpotts/docker-babelfishpg/blob/main/Dockerfile

Case solved :) Jonathan reworked image a bit to include ssl, and after few commands that generate certificate & enable SSL in config - SQL cmd works perfectly.


Ajax Layered Navigation Price slider not updating on other product attribute filter apply

I am having shellpea/magento-advanced-elasticsuite-catalog:1.2.6 installed along with hyva theme 1.3.7
Steps to reproduce:
Open category page with price filter (as as slider) active
Click on product attribute filter like brand, color, etc. -> Apply filter
Expected result
The price slider should also update  as range of product collection and product count should also applied (like other product attribute)
Actual result
The price slider is not update on ajax request / It update when we do page refresh manually.
Magento version: 2.4.6-p1
PHP version: 8.1
Module version: 1.2.6

Hi @rahul-navigate, the functionality you described was still in the process of implementation, at the moment I cannot offer a quick solution, since this requires state synchronization with Alpine.store, I will mention this issue as soon as the solution is ready, thanks for the feedback

Hi @rahul-navigate, the functionality you described was still in the process of implementation, at the moment I cannot offer a quick solution, since this requires state synchronization with Alpine.store, I will mention this issue as soon as the solution is ready, thanks for the feedback

@AlexDroomer thanks for sharing an update. we will looking forward to it. Thanks !

Hey @AlexDroomer, will the fix mentioned in your previous comment be part of next release? If it is not yet in the process of development could you shed a bit more light on how can this be fixed using Alpine.store?
I can see that currently category object is stored in Alpine.store, so is the idea to have priceSlider property stored within category object and then reinit this with new min/max price and product count on loadActualFilters?


[lit-html] Re-rendering an asyncReplace clears its value.

Which package(s) are affected?
Lit Core (lit / lit-html / lit-element / reactive-element)
Description
When a template using an asyncReplace is re-rendered without changing the iterable or its current value in the asyncReplace, the value is incorrectly cleared.
Reproduction
https://lit.dev/playground/#gist=257917d929b16bd6f3109c81f121b992
Click the requestUpdate button
Expected:  the asyncReplace value should not change
Actual: the asyncReplace value is cleared (it is replaced if the iterable emits another value)
Workaround
None.
Is this a regression?
No or unsure. This never worked, or I haven't tried before.
Affected versions
3.x
Browser/OS/Node environment
All

The issue appears to be that this line needs to return noChange.


Error when training text-line detection model

Hi, thanks for the great repo!

I followed the guide in the README file and want to train the text-line detection model
I also prepared a dataset with COCO format, same as the MMOCR's repo and psenet_r50_fpnf_600e_pubtabnet.py but got the following error. Seem like it occurs with an empty object but I did not sure the error came from
Did you face this issue when training the text-line detection model? Or do you have an idea about how to fix this issue?

Thanks in advance!

Hi, thanks for the great repo!

I followed the guide in the README file and want to train the text-line detection model
I also prepared a dataset with COCO format, same as the MMOCR's repo and psenet_r50_fpnf_600e_pubtabnet.py but got the following error. Seem like it occurs with an empty object but I did not sure the error came from
Did you face this issue when training the text-line detection model? Or do you have an idea about how to fix this issue?

Thanks in advance!

Hi, seem like it is failed to load the training data. And what's your MMDet version?
Maybe you can check the data loading setting in config file, or check the data loading pipeline.

Hi, seem like it is failed to load the training data. And what's your MMDet version?
Maybe you can check the data loading setting in config file, or check the data loading pipeline.

@JiaquanYe
lib version
mmocr: 0.2.0 (installed from repo)
mmdet: 2.11.0 (installed from repo)
mmcv-full: 1.3.4

I checked and just realized that when training the model, it's called the build_dataset function first to building the dataset. But I searched through all the source code and only find this function from mmdet? Is it normal?
Here is my PSE config: https://pikab.in/5127bb5d68
train.json is under COCO format, images is the folder contains the image data

@JiaquanYe
Can you provide the pre-trained PSENet text-line detection model?
Currently, I want to test the pipeline but can not train the text-line detection due to this issue.
Hope to see your response, thank you!

@JiaquanYe
lib version
mmocr: 0.2.0 (installed from this repo)
mmdet: 2.11.0 (installed from this repo)
mmcv-full: 1.3.4

I checked and just realized that when training the model, it's called the build_dataset function first to building the dataset. But I searched through all the source code and only find this function from mmdet? Is it normal?
Here is my PSE config: https://pikab.in/5127bb5d68
train.json is under COCO format, images is the folder contains the image data

Could you paste your train.json here?

@JiaquanYe Here is the example of json file: https://pikab.in/537fbc4566

I found the error. Must set the name of the category to text and turn on with_mask=True in the config file
Closed. Thank you for your support!


Update postal code logic based on patient country

FYSA: @Adrian-Brewster @anshulkumar-usds @mlighdc @emmastephenson @LGFranklin

This isn't needed. Closing out.


Load cpp_server.js through the file option to improve error messages

Loading through the file option results in better error messages from cpp_server.js since nodejs knows which file the script came from. I have tested that this seems to work the same as using the code option but unfortunately don't know if it fully works because of issue #4.

The path inferred from using file is /c9sdk/plugins/c9.ide.language.cpp/cpp_server.js, hence why this change will break the Docker image and c9.io.
Did you try using node v0.12 for the plugin and c9 in regards to #4? I can walk you through it should anything be unclear.


Disable Expandable Image

Presently, in the Vision tab, if you click on an image, it will expand. This is something we had for our previous RTP maps but no longer need. Low priority.

This looks like it works. Thanks!