aws / aws-eks-best-practices

A best practices guide for day 2 operations, including operational excellence, security, reliability, performance efficiency, and cost optimization.
https://aws.github.io/aws-eks-best-practices/
Other
2.04k stars 498 forks source link

Spreadsheet subnet-calc is not compatible with Microsoft Excel for Mac #255

Open davivcgarcia opened 1 year ago

davivcgarcia commented 1 year ago

Describe the problem

An _xlfn. prefix is displayed in front of a function in a formula for the .../content/networking/subnet-calc/subnet-calc.xlsx. When the formula is calculated, it displays the #NAME? error value for Microsoft Excel for Mac (version 16.66.1).

Cause

The Excel workbook contains a function that is not supported in the version of Excel that you are currently running. For example, you may have opened a workbook that contains the IFERROR function, which is not supported in versions of Excel earlier than Excel 2007. Functions that are not supported in versions of Excel earlier than Excel 2007 include: AVERAGEIF, AVERAGEIFS, MAXIFS, MINIFS, IFS, CUBEKPIMEMBER, CUBEMEMBER, CUBEMEMBERPROPERTY, CUBERANKEDMEMBER, CUBESET, CUBESETCOUNT, CUBEVALUE, COUNTIFS, IFERROR, and SUMIFS.

Resolution

Remove the unsupported functions, or if possible, replace the unsupported functions with widely supported functions.

davivcgarcia commented 1 year ago

I'm trying to figure out how to fix it, but my MS Excel skills are very limited... =S

jicowan commented 1 year ago

@geoffcline can you have a look at this issue?

geoffcline commented 1 year ago

Will do!

geoffcline commented 1 year ago

Thanks for the issue @davivcgarcia !

The issue is not Mac/Win versions of Excel.

The issue is the XLOOKUP function. This function is only available in recent versions of Excel for Mac (~2021 I believe). It has existed for longer in the Windows version.

The sheet uses this function to convert instance names (M1.large) to the the number of Max Pods.

I'm considering two options

  1. Revising the spreadsheet to avoid this function.
  2. Publishing the spreadsheet as a Google sheet (which supports XLOOKUP).

Let me know your thoughts. Thanks, Geoffrey.

davivcgarcia commented 1 year ago

I'd recommend keep it as a standalone spreadsheet, but make it portable if possible.

davide-romanini-milkman commented 1 year ago

Please consider compatibility with LibreOffice too, it seems to have the same issue.