Windows/ Powershell error: Expand-Archive :  is not a supported archive file format. .zip is the only supported archive file format.

Hi,
i get the following error using

name: Install CMake and Ninja
uses<EMAIL_ADDRESS>with:
useLocalCache: true
useCloudCache: false
cmakeVersion: "~3.27.0"
ninjaVersion: latest

log:
useCloudCache:false
useLocalCache:true
user defined cmake version:~3.27.0
user defined ninja version:latest
cmake version:~3.27.0
ninja version:latest
::group::Computing cache key from the downloads' URLs
Cache key: '507572093'.
Local install root: 'C:\Users\User.cache\act\8c275a1a4fda57b3\tmp\507572093''.
::endgroup::
::group::Restoring from local GitHub runner cache using key '507572093'
Local cache miss.
::endgroup::
::group::Downloading and extracting CMake
[command]C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe -NoLogo -Sta -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command "$ErrorActionPreference = 'Stop' ; try { Add-Type -AssemblyName System.IO.Compression.FileSystem } catch { } ; if ((Get-Command -Name Expand-Archive -Module Microsoft.PowerShell.Archive -ErrorAction Ignore)) { Expand-Archive -LiteralPath 'C:\Users\User.cache\act\8c275a1a4fda57b3\tmp\ad76d5b6-c455-4ab6-810f-4a4c64dce49b' -DestinationPath 'C:\Users\User.cache\act\8c275a1a4fda57b3\tmp\507572093' -Force } else {[System.IO.Compression.ZipFile]::ExtractToDirectory('C:\Users\User.cache\act\8c275a1a4fda57b3\tmp\ad76d5b6-c455-4ab6-810f-4a4c64dce49b', 'C:\Users\User.cache\act\8c275a1a4fda57b3\tmp\507572093', $true) }"
Expand-Archive :  is not a supported archive file format. .zip is the only supported archive file format.
At line:1 char:210

...  Ignore)) { Expand-Archive -LiteralPath 'C:\Users\User.cache\act\8c2 ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidArgument: (:String) [Expand-Archive], IOException
FullyQualifiedErrorId : NotSupportedArchiveFileExtension,Expand-Archive

::endgroup::
Error: The process 'C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe' failed with exit code 1
at ExecState._setResult (C:\Users\User.cache\act\8c275a1a4fda57b3\act\actions\lukka-get-cmake@v3.27.7\dist\index.js:5801:25)
at ExecState.CheckComplete (C:\Users\User.cache\act\8c275a1a4fda57b3\act\actions\lukka-get-cmake@v3.27.7\dist\index.js:5784:18)
at ChildProcess. (C:\Users\User.cache\act\8c275a1a4fda57b3\act\actions\lukka-get-cmake@v3.27.7\dist\index.js:5678:27)
at ChildProcess.emit (node:events:514:28)
at maybeClose (node:internal/child_process:1105:16)
at ChildProcess._handle.onexit (node:internal/child_process:305:5)
::error::get-cmake action execution failed: 'Error: The process 'C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe' failed with exit code 1'

@irnbrux this looks like identical to this problem: https://github.com/lukka/get-cmake/issues/74
Are you using a self-hosted action runner? Looks like there is a difference in the environment between the GH hosted runner and the self-hosted runner, and it would be nice if we find out what it is, and how to insert the fix in the action itself.

Hi,
it is identical to #74 indeed. The cause is to use the expand cmdlet on a file that does not have a .zip file extension. When you rename the file first (as part of the action) to have a .zip file extension, it will work I guess. At least this is the behavior if I invoke the commands manually.
Cheers
Brian

@irnbrux this should be the root cause: https://github.com/actions/toolkit/issues/1179

@irnbrux you may try this version and report back: uses: lukka/get-cmake@dev/fix91

So with fix91 I can see the zip gets extracted correctly, however the process just keeps running after extraction and never completes. I need to investigate further:
useCloudCache:false useLocalCache:false user defined cmake version:latest user defined ninja version:latest cmake version:latest ninja version:latest ::group::Computing cache key from the downloads' URLs Cache key: '507572093'. Local install root: 'C:\Users\User\.cache\act\679324afd465c947\tmp\507572093''. ::endgroup:: ::group::Downloading and extracting CMake [command]C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe -NoLogo -Sta -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command "$ErrorActionPreference = 'Stop' ; try { Add-Type -AssemblyName System.IO.Compression.FileSystem } catch { } ; if ((Get-Command -Name Expand-Archive -Module Microsoft.PowerShell.Archive -ErrorAction Ignore)) { Expand-Archive -LiteralPath 'C:\Users\User\.cache\act\679324afd465c947\tmp\0d05c39a-3b7c-4607-93ac-54ef847dbfbc.zip' -DestinationPath 'C:\Users\User\.cache\act\679324afd465c947\tmp\507572093' -Force } else {[System.IO.Compression.ZipFile]::ExtractToDirectory('C:\Users\User\.cache\act\679324afd465c947\tmp\0d05c39a-3b7c-4607-93ac-54ef847dbfbc.zip', 'C:\Users\User\.cache\act\679324afd465c947\tmp\507572093', $true) }"
There is nothing more after the last line ...

OK, the cmake extract does now work, however we get the same error for the ninja download as well I think:
useCloudCache:false
useLocalCache:false
user defined cmake version:latest
user defined ninja version:latest
cmake version:latest
ninja version:latest
::group::Computing cache key from the downloads' URLs
Cache key: '507572093'.
Local install root: 'C:\Users\User.cache\act\679324afd465c947\tmp\507572093''.
::endgroup::
::group::Downloading and extracting CMake
[command]C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe -NoLogo -Sta -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command "$ErrorActionPreference = 'Stop' ; try { Add-Type -AssemblyName System.IO.Compression.FileSystem } catch { } ; if ((Get-Command -Name Expand-Archive -Module Microsoft.PowerShell.Archive -ErrorAction Ignore)) { Expand-Archive -LiteralPath 'C:\Users\User.cache\act\679324afd465c947\tmp\0d05c39a-3b7c-4607-93ac-54ef847dbfbc.zip' -DestinationPath 'C:\Users\User.cache\act\679324afd465c947\tmp\507572093' -Force } else {[System.IO.Compression.ZipFile]::ExtractToDirectory('C:\Users\User.cache\act\679324afd465c947\tmp\0d05c39a-3b7c-4607-93ac-54ef847dbfbc.zip', 'C:\Users\User.cache\act\679324afd465c947\tmp\507572093', $true) }"
::endgroup::
::group::Downloading and extracting Ninja
[command]C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe -NoLogo -Sta -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command "$ErrorActionPreference = 'Stop' ; try { Add-Type -AssemblyName System.IO.Compression.FileSystem } catch { } ; if ((Get-Command -Name Expand-Archive -Module Microsoft.PowerShell.Archive -ErrorAction Ignore)) { Expand-Archive -LiteralPath 'C:\Users\User.cache\act\679324afd465c947\tmp\a01294d2-2a5b-49f3-9a93-abca76fc5550' -DestinationPath 'C:\Users\User.cache\act\679324afd465c947\tmp\507572093' -Force } else {[System.IO.Compression.ZipFile]::ExtractToDirectory('C:\Users\User.cache\act\679324afd465c947\tmp\a01294d2-2a5b-49f3-9a93-abca76fc5550', 'C:\Users\User.cache\act\679324afd465c947\tmp\507572093', $true) }"
Expand-Archive :  is not a supported archive file format. .zip is the only supported archive file format.
At line:1 char:210

...  Ignore)) { Expand-Archive -LiteralPath 'C:\Users\User.cache\act\679 ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidArgument: (:String) [Expand-Archive], IOException
FullyQualifiedErrorId : NotSupportedArchiveFileExtension,Expand-Archive

::endgroup::
Error: The process 'C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe' failed with exit code 1
at ExecState._setResult (C:\Users\User.cache\act\679324afd465c947\act\actions\lukka-get-cmake@dev-fix91\dist\index.js:5811:25)
at ExecState.CheckComplete (C:\Users\User.cache\act\679324afd465c947\act\actions\lukka-get-cmake@dev-fix91\dist\index.js:5794:18)
at ChildProcess. (C:\Users\User.cache\act\679324afd465c947\act\actions\lukka-get-cmake@dev-fix91\dist\index.js:5688:27)
at ChildProcess.emit (node:events:514:28)
at maybeClose (node:internal/child_process:1105:16)
at ChildProcess._handle.onexit (node:internal/child_process:305:5)
::error::get-cmake action execution failed: 'Error: The process 'C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe' failed with exit code 1'

@irnbrux thank you for reporting back! Now the same manipulation is applied to the ninja build tool archive, you may try again, thank you!

Hi @lukka ,
that works for me now for cmake and ninja under Windows 11.
Thank you for the work you are doing.
Cheers
Brian

@irnbrux thank you for testing the PR #92 ! I thought to rework the code in order to apply the fix only if strictly needed, and I'd be happy if you could still confirm this is working correctly. Then I'll merge, thank you.

I also hit this issue on a self-hosted windows runner. After installing powershell 7 with
winget install --id Microsoft.Powershell --source winget

I still had the same issue (apparently the newer version doesn't replace the old powershell, it's just installed alongside), but switching to uses: lukka/get-cmake@dev/fix91 did seem to work for me.

Fixed in main and latest branches.


Merge Dev

Copy download link to clipboard

Current coverage is 86.60% (diff: 100%)

Merging #182 into master will increase coverage by 0.13%

@@             master       #182   diff @@
==========================================
  Files            10         10
  Lines           739        739
  Methods         131        131
  Messages          0          0
  Branches        133        133
==========================================
+ Hits            639        640     +1
+ Misses          100         99     -1
  Partials          0          0

Powered by Codecov. Last update b53b4fc...f19749d


Intellisense shows foreach loop snippet after typing 'fun'

VSCode Version: 1.0.0-alpha
OS Version: Windows 10/Ubuntu 15.10

Steps to Reproduce:

Open a javascript file
Type "fun"
Scroll to the bottom of the intellisense suggestions to see snippets

Actual:
Shows the foreach snippet suggestion even though isn't relevant after typing 'fun'

This is the actual design behaviour.

You type fun and trigger intellisense (discard the suggest on type box) -> no 'foreach' shows
You type f and intellisense comes up cos of typing -> It shows all suggestions for f keeps selecting the best match as you continue typing


vscode, vscodium: 1.45.1 -> 1.46.0

Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Does it make sense to introduce a small script to automatically update versions? If so, what tech is considered OK for this task (such that perhaps later on it is possible to automate on a bigger scale in NixPkgs)? ping @Mic92 .

Does it make sense to introduce a small script to automatically update versions? If so, what tech is considered OK for this task (such that perhaps later on it is possible to automate on a bigger scale in NixPkgs)? ping @Mic92 .

I don't think there is standardised tech for this; perhaps something to integrate it with nixpkgs-auto-upgrade would be nice. An "update-script" meta field?

Everything should be ready, please merge.

Now, also includes a bash script to fetch latest versions for both VSCode and VSCodium. Please review and merge.

Probably need to rebase this to drop the update script commits entirely rather than reverting.

Probably need to rebase this to drop the update script commits entirely rather than reverting.

Done

@ofborg eval

@GrahamcOfBorg build vscode
@GrahamcOfBorg build vscodium


Replace default monospace font with SourceCodePro

The PR forces the electron app to use the SourceCodePro font instead of the default browser monosapce font. This is a temporary fix that will be deprecated when non-default fonts are implemented in JupyterLab proper.

Lets use Source Code Pro instead please ;-) That is what we will be using
in the main JupyterLab
On Fri, Aug 4, 2017 at 12:01 PM, Luc Bouchard<EMAIL_ADDRESS>wrote:

The PR forces the electron app to use the Cousine font instead of the
default browser monosapce font. This is a temporary fix that will be
deprecated when non-default fonts are implemented in JupyterLab proper.
You can view, comment on, or merge this pull request online at:
https://github.com/jupyterlab/jupyterlab_app/pull/102
Commit Summary

Fix font in CobeMirror blocks

File Changes

M src/browser/style/main.css
https://github.com/jupyterlab/jupyterlab_app/pull/102/files#diff-0
(12)

Patch Links:

https://github.com/jupyterlab/jupyterlab_app/pull/102.patch
https://github.com/jupyterlab/jupyterlab_app/pull/102.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jupyterlab/jupyterlab_app/pull/102, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AABr0BFNknq092nNhXqkcVZutp5WYeFtks5sU2p-gaJpZM4OuE3Z
.

--
Brian E. Granger
Associate Professor of Physics and Data Science
Cal Poly State University, San Luis Obispo
@ellisonbg on Twitter and GitHub
<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

Fixed!


The latest version problem on npm

Hello @StefanKovac,
I am the member of cdnjs project.
We want to host this library.
I downloaded the latest version v1.0.2, but I found the version description of files such as flex-layout-attribute.min.css, flex-layout-attribute.css  which downloaded from npm were wrong, and the version description is still v1.0.1. Could you update the latest version description on npm?
Thanks very much!
cdnjs/cdnjs#8174

Hi @nolsherry,
ahh, thanks for letting me know.
I've bumped version to 1.0.3 on npm & here on github — hopefully it will work for you now. Please let know.
Thank you!

Hi @StefanKovac,
thanks for your help, and it works for me, thank you very much!


Use topological sort in type inference

Currently in Hydra, the type of every element must be given along with the data (term) of the element. This is a holdover from Hydra's initial type system, which was based on STLC rather than Hindley-Milner (which permits type inference). While it is possible now to infer the type of many elements rather than relying on type annotations (which are labor intensive and error-prone for the developer), this must be done by ordering the elements in a graph to take into account the dependencies introduced by element references. Only in those cases where the reference structure forms a cycle will type annotations be required.
Currently, types are inferred on a term-by-term or element-by-element basis, but the change described above will require types to be inferred for an entire graph at once.

Done.


use QueryRange but given the uncorrect data

Hello, we push data with value 0 or 1 to prometheus at with regular interval, the 1 or 0 represent something occur and recover. But when we use function QueryRange(ctx context.Context, base *url.URL, query string, startTime, endTime, step int64, opts QueryOptions) (model.Matrix, []string, error) to query data, and the step is 60 second, smaller than the push data interval,we get the data that's like 0,0,1,1,1, it's not meets the data we pushed to prometheus. As we expect, we should get the data like 0,1,0,1,0,1.
To get the original data,we use the function QueryInstantWithTimeRange(ctx context.Context, base *url.URL, query string, t time.Time, opts QueryOptions) (model.Matrix, []string, error), the query in this function  with time durations and  fffset modifier, we get the data we expect.
the query string just like this :{job="",entity=",name=}[16m] offset 1371m

Sorry, could you please be more specifc? Are you using Thanos or Prometheus?
For QueryRange and QueryInstantWithTimeRange, they are functions in Thanos, right? How are you using them? If you just need a Prometheus API client then probably use the Prometheus go client instead.


refactor core password prompting, file exist and terminal checks

Refactor the common utilities for password prompting, checking for file existence as well as determining terminal status, per #1078
Signed-off-by: Ivan Wallis<EMAIL_ADDRESS>Summary
Ticket Link
Fixes
Release Note

On a fire!


Create basic security measures

[ ] CSFR
[ ] XSS
[x] SQL injection

SQL injection solved by PDO prepared statements and basic sanitization of input inside request class


InterSystems Objectsript Contest

Hi @jakcpto,
Here’s Iryna from the InterSystems team!
We noticed you participating in the InterSystems Objectsript Contest:
image
InterSystems prepared $$$ prizes for the winners. So we welcome you to our competition! Details here.
In order to win, you need to commit the ObjectScript repo to the github and show it to us (admins of InterSystems) + be registered on the InterSystems Developer Community.
Please let me know if you need any help with onboarding to our ObjectScript Contest and Community.
Happy coding!
Best regards,
Iryna from InterSystems Team

Ok, thanks


[CLI]: Opening runs leaks file pointers and semaphores

Describe the bug
Reopening #1447 which was closed in a broad issue cleanup.
Initializing wandb runs appears to leak file pointers, as after opening a few runs, the process crashes with "too many open files." Quitting the process with Control-C leads to the common leaked semaphore message (and thus may be unrelated?)
My use case: I intend to sequentially launch a group of runs for K-folds cross-validation. Each group must be launched from the same Python file because that file is my sweeps entry point. However, I can't get through all the folds I need without crashing on "too many open files."
I'm on wandb=0.12.21, Python 3.9.13, MacOS Big Sur 11.6.
Code to reproduce:
import wandb
for i in range(100):
    with wandb.init(entity='exr0nprojects', project='snap', group='useless'):
        print(f"run number {i}")

The "leaked semaphores" appears during the first run, but this may be happening because I've been testing in the same shell. The full error:
/usr/local/Cellar/python@3.9/3.9.13_1/Frameworks/Python.framework/Versions/3.9/lib/python3.9/multiprocessing/resource_tracker.py:216: UserWarning: resource_tracker: There appear to be 54 leaked semaphore objects to clean up at shutdown
  warnings.warn('resource_tracker: There appear to be %d '

I got the "too many open files" error after the 18th run. The error message is quite long, but appears to just repeat the following segment:
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/wandb/sdk/wandb_init.py", line 1043, in init
    run = wi.init()
  File "/usr/local/lib/python3.9/site-packages/wandb/sdk/wandb_init.py", line 556, in init
    backend.ensure_launched()
  File "/usr/local/lib/python3.9/site-packages/wandb/sdk/backend/backend.py", line 220, in ensure_launched
    self.wandb_process.start()
  File<EMAIL_ADDRESS>line 121, in start
    self._popen = self._Popen(self)
  File<EMAIL_ADDRESS>line 284, in _Popen
    return Popen(process_obj)
  File<EMAIL_ADDRESS>line 32, in __init__
    super().__init__(process_obj)
  File<EMAIL_ADDRESS>line 19, in __init__
    self._launch(process_obj)
  File<EMAIL_ADDRESS>line 58, in _launch
    self.pid = util.spawnv_passfds(spawn.get_executable(),
  File<EMAIL_ADDRESS>line 450, in spawnv_passfds
    errpipe_read, errpipe_write = os.pipe()
OSError: [Errno 24] Too many open files

The above exception was the direct cause of the following exception: [etc]

Additional Files
No response
Environment
WandB version: 0.12.21
OS: MacOS Big Sur 11.6
Python version: 3.9.13
Versions of relevant libraries: No other libraries are needed for reproduction.
Additional Context
No response

Hi @Exr0n, thank you for reporting this. I was unsuccessful in replicating the issue using the following code:
import wandb
import psutil

def print_file_info()::
    proc = psutil.Process()
    print('Num open files: %s' % len(proc.open_files()))

for i in range(100):
    with wandb.init(project='snap', group='useless'):
        print(f"run number {i}")
        print_file_info()

The print_file_info() function never printed that it had more than 4 files open and I was able to finish all 100 iterations.
Do you happen to be using the WANDB_START_METHOD environment variable or any other custom way of running wandb that I may not be using in my replication?
Thank you,
Nate

Hi @nate-wandb thanks for the swift response! I got the error after the 19th run again using your version of the script.
Curiously, print_file_info() always showed the number of open files as 5 in Alacritty (the terminal emulator that I use), and this never increased as more runs were created. In Terminal.app, the number of open files stays at a constant 2, but the script still crashes after the 19th run. Maybe the open files are owned by a spawned process or thread?
It's also possible that I have a lower-than-default process limit because this laptop is provided by my employer and has management profiles installed. It still seems like somethings leaking though...

With the following code, I get  OSError: [Errno 24] Too many open files after the 202nd model is created:
import wandb
while True:
    # Create new model here and pass its config to wandb.init
    wandb.init(project="project-bug-testing")
    for x in range(100):
        wandb.log({"foo": 5})
        # Log data for each of 100 batches
    wandb.finish()
    # Save data for this model before moving on to next model

The time until failure probably depends on how many open files the computer allows. When I run ulimit -n, it tells me that my limit for open files is 1024.

Hi @Exr0n, sorry for the delay here. I was finally able to replicate this after setting my default number of files allowed to be opened with ulimit -n 100. My defaults were set to 1048575 originally which was why I could not replicate. I also can confirm what you are seeing that it appears that these files are being held by an external process that wandb spins up. I'll report this to our engineering team.
In the meantime you should be able to set your limit higher with ulimit -n <number_of_file_handles_you_need> to get you unblocked for the time being.
I'll follow up once we are able to come up with a fix on this and figure out what the issue is.
Thank you,
Nate

I also encounter the error "Too many open files" on mac. Could you let me know if there are any updates? besides changing the limit of open files

Any update on this?

Any update on this?

I rolled back to wandb 0.12.21 and it seems to work fine.

I see, thank you @nate-wandb. I haven't tried the latest version yet.

@nate-wandb I was able to replicate the error with wandb 0.13.9 in a hope that updating would fix the issue, and with a larger ulimit setting.
However, for me I think it was actually an issue with a PyTorch DataLoader that was somehow crashing my code and causing WandB workers to also crash. Following https://github.com/pytorch/pytorch/issues/11201#issuecomment-586007563, I added the following two lines to the start of my code appears to have fixed my issue:
import torch.multiprocessing
torch.multiprocessing.set_sharing_strategy("file_system")

Hi! I am using AWS SageMaker -- so I can't set ulimit. Pytorch Lightning 1.8.6 & Pytorch 1.12.1 & wandb 0.13.9.
I still getting this error OSError: [Errno 24] Too many open files.
I tried to set sharing strategy to "file_system" and even for each worker in dataloader suing worker_init_fn. Tried to wandb==0.12.21 and 0.13.9.
This is hard to debug. Just want to bump this issue.

Thank you @tristanengst and @n-kasatkin for testing. It looks like we have this fixed in the wandb module but the issue still exists in the our PyTorch integration.
Could either of you provide a minimal reproduction of the issue when using PyTorch or PTL?
Thank you,
Nate

The problem in my case was not related to wandb!
I had an internal pytorchvideo bug, that prevented videos from closing.

This is still an issue, but I'm following on #2825

Problem still exists


Add spacy-transformer wrapper model to load danish BERT form botxo

Hey @AmaliePauli, I see you're using the BotXO weights for your BertTone model. Is that the version 1 or version 2 representations?
https://github.com/botxo/nordic_bert

Hi @lewisbails , I have used version 1 weights

It is hard to see exactly what this issue is on, but you might find this spacy-extension useful.


Provider: Vendure

E-commerce Provider: https://www.vendure.io
Regards and Thanks
for this great project.

Hi!
Is there a possibility to use Next.js Commerce with Vendure provider?

Many thanks to @michaelbromley for working on vendure provider for nextjs commerce.
Running PR in https://github.com/vercel/commerce/pull/223

Would love to see vendure officially supported by nextjs commerce!

Can't wait to see this through


Add source set for GDSL

Possibly auto-map
need to make an issue to add a demo/documentation for how to use it

FWIW, one can be added from one's $JENKINS_URL/pipeline-syntax/gdsl
I did not figure out why there are two different copies of the file in the example project in jenkinsResources/ and in resources/ which IDEA recognizes as a "resources root" and probably that is the one helping code-completion etc.

I managed to get as far as IDEA recognising the file, but all the declared classes for all properties seem to be off by one.
e.g. in the gdsl served by Jenkins, it says that currentBuild is a RunWrapperBinder. IDEA then shows currentBuild.rawBuild as an unknown property. The actual type should be RunWrapper, which does have the rawBuild property. It seems you can call getValue on the binder object to get the actual object, but IDEA doesn’t do this. I have no idea whether IDEA is handling the file incorrectly or whether Jenkins is serving the file with the wrong types.

Well, IDEA's integration with Groovy generally is helpful, but with Jenkins effective dialects it struggles :)
I had posted a loosely related issue which so far got little traction: https://youtrack.jetbrains.com/issue/IDEA-320924/IDEA-does-not-handle-Groovy-call-class-methods-call-operator-for-context-help-and-auto-completion and one replier pointed to https://github.com/lppedd/idea-jenkins-pipeline - which might be a useful point to collaborate with regarding IDE support.


support igv reports for each variant

https://github.com/igvteam/igv-reports
TBD

this is implemented


Limit the amount of output a terminal gets sent

The 'logs' and 'shell' buttons open a terminal; everything the backend can provide is then piped up to that terminal.
Suppose there are 10MB of logs: this will scroll in the terminal as fast as it can and you will be left with the bottom few hundred lines.  The first 9.5MB is just a waste.  I find it's often hard to get a control-C through the connection while it's scrolling, so the user may end up just closing the window.
I think we could use a rate-limiter to allow up to some reasonable burst through, then drop the bulk.
Set a timer so when (if!) it goes quiet we can deliver the last buffer-full.

As noted in #2908 we could also limit the amount of logs we ask for - both Docker and Kubernetes have a 'tail' option.
This feature would still be valid for the 'shell' case.


remove oq3_program_result

Issue #, if available:
Description of changes:
This model is vestigial and not used by the final simulator
Testing done:
tox still passes
Merge Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.
General

[x] I have read the CONTRIBUTING doc
[x] I used the commit message format described in CONTRIBUTING
[x] I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

[x] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[x] I have checked that my tests are not configured for a specific region or account (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Though it's not currently used in the service, electing to keep the model as part of our schemas.


[Suggestion] Support for other keyboard layout

Oh right, totally forgot about that, will try working on it soon

update: latest commits now kinda allow for this, unfortunately, it is still in its early phase and currently requires lots of manual work on the player's side.

Isn't there a way to know what character is mapped to which key code on the os??

Vanilla fixed the issue while switching to LWJGL 3 in 1.13 🤔.

yes and no, see if you have the specific keyboard layout, you can easily track down the keycodes per key, unfortunately, it seems like the keycodes are bound to the keyboard's internal layout rather than the selected kb locale from the OS, which means, even if I change my kb layout, it will still return the original keycodes for my kb, so until I figure out how I can try out different layout without changing kb, I might not be able to solve this properly.
That said, the keybinding part of the mod still works regardless of the kb layout, just that the visual keymapper won't show the proper layout for the current kb locale.

working on smth rn, no ETA but next release should finally address this issue


Option to highlight gutter background

It is very... odd looking when the background of the gutter is not highlighted.

Is there api for that? Have you seen any other extension highlight gutter?

I don't do vscode extensions, that'd be your job to look up no?

AFAIK there's no api for that.

I'm going to close this since there is no VSCode api. No action can be taken.


[Bug]: Women products section

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Text seen on women products section.
Add Screenshots / Video Clip

What browsers are you seeing the problem on?
Microsoft Edge
Agreements

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC contributor
[ ] I'm a Hacktoberfest contributor
[X] I want to work on this issue [Optional]
[X] I'm willing to provide further clarification or assistance if needed.

Already fixed


Week 17 review ticket

My contribution to the CPT is a history page where you can view all past games played. Here is my first ideas of making that:

Responsibilities and tasks for CPT: https://github.com/GeneChang1/TGDKP_reunion_fort_batman/issues/2
CPT scoring issue: https://github.com/dashpen/blog/issues/17

[x] Team. View and Frontend Hacks +++ Teachers looking to see beginnings of frontend coding in CPT project.
[ ] Team. Model and OOP Hacks +++ Teachers looking to see beginnings of class definitions and object usage in CPT project.
[ ] Individual. Perform Create Performance Task Scoring blog +++ Teachers want to see understanding of criteria and evaluations of CPT. Also, we want to see how understanding scoring of these examples will help you exceed CPT requirements.


Clinical Notes Guideline Tests does not include patient id in url

Subject of the issue
When running a Clinical Notes Guideline Test, the patient ID that is specified as an input does not get included in the url, causing all tests to fail.
Your environment

Edition of inferno (Community or Program): Community
Version of inferno: 2.11.0
Which browser and version(s) is the bug present on?: Chrome version 89.0.4389.128

Steps to reproduce

Go to  https://inferno.healthit.gov/community
Click US Core v3.1.1
Enter https://inferno.healthit.gov/reference-server/r4 as the server and click begin
Click Run for Clinical Notes Guideline Tests
Enter SAMPLE_TOKEN as auth token and 85,355 for patient ids. Click Execute
Tests will fail. Click 'results...' for the first test. Notice that the patient parameter in the url is just 'patient' and not 'patient=85' or 'patient=355'
Click the Inputs tab and confirm that patients 85 and 355 are included in the Patient Ids input
Optionally run DocumentReference Tests and compare how the patient parameter in the url should be structured

See https://inferno.healthit.gov/community/f8N9c40PxRb/ for my specific testing results.
Expected behavior
A patient ID should be included in the url. The url for the first test should look something like "https://inferno.healthit.gov/reference-server/r4/DocumentReference?patient=85&type=http://loinc.org|11488-4"
Actual behavior
The "=<patientid>" portion of the url is missing. So the url looks like this: "https://inferno.healthit.gov/reference-server/r4/DocumentReference?patient&type=http://loinc.org|11488-4"

Hello @AGobron : I followed the link you provided. ( https://inferno.healthit.gov/community/f8N9c40PxRb/) The result shows all tests passed and the HTTP Requests logs queries with patient= (see attached screen shot).

Hi @yunwwang, thanks for looking into this. it looks like your screenshot is of "DocumentReference Tests", which I included in the link for reference. If you scroll down the page you will see the failed "Clinical Notes Guideline Tests".

Sorry, I looked the wrong sequence. Thank you for reporting the problem to us. We will fix that asap.

Thanks!


Multi-project gradle setup does not allow module's to configure different repository sources, leading to coupled projects.

------------------------------------------------------------
Gradle 4.4
------------------------------------------------------------

Build time:   2017-12-06 09:05:06 UTC
Revision:     cf7821a6f79f8e2a598df21780e3ff7ce8db2b82

Groovy:       2.4.12
Ant:          Apache Ant(TM) version 1.9.9 compiled on February 2 2017
JVM:          1.8.0_92 (Oracle Corporation 25.92-b14)
OS:           Mac OS X 10.12.6 x86_64

Steps to Reproduce:

Take two modules, 'app' and 'lib'
Have lib setup with 'google' repository and dependency of 'com.google.android.gms:play-services-location:11.4.2' in module's build.gradle
Attempt to build app with gradle app:assembleDebug without defining 'google' repository inside the app's build.gradle (or using an allprojects block in the root build.gradle)

This doesn't matter which repository you are defining or which dependency. Above I'm using a concrete example. There is nothing specific about the values used. I'm not using Android Studio when running this build. I'm using the gradle CLI and running the commands manually.No behavior changes if I'm using a gradle wrapper or not.
* What went wrong:
Could not resolve all files for configuration ':app:debugCompileClasspath'.
> Could not find com.google.android.gms:play-services-location:11.4.2.
  Searched in the following locations:
      file:/Users/me/Library/Android/sdk/extras/m2repository/com/google/android/gms/play-services-location/11.4.2/play-services-location-11.4.2.pom
      file:/Users/me/Library/Android/sdk/extras/m2repository/com/google/android/gms/play-services-location/11.4.2/play-services-location-11.4.2.jar
      file:/Users/me/Library/Android/sdk/extras/google/m2repository/com/google/android/gms/play-services-location/11.4.2/play-services-location-11.4.2.pom
      file:/Users/me/Library/Android/sdk/extras/google/m2repository/com/google/android/gms/play-services-location/11.4.2/play-services-location-11.4.2.jar
      file:/Users/me/Library/Android/sdk/extras/android/m2repository/com/google/android/gms/play-services-location/11.4.2/play-services-location-11.4.2.pom
      file:/Users/me/Library/Android/sdk/extras/android/m2repository/com/google/android/gms/play-services-location/11.4.2/play-services-location-11.4.2.jar
  Required by:
      project :app > project :lib

I've attached a sample project that contains build files only. There is no source but there are defined AndroidManifest.xml files in each module as the android plugin requires this. I believe this is the smallest example I could build that still displays the issue.
test-gradle.zip
I would expect that the lib module would be able to define repositories to pull its dependencies from. When the app is compiled using the lib project as a dependency, it should not need to redefine repositories used by transitive dependencies it gets from the lib module.
I have already opened an issue in Google's Issue Tracker it was deemed 'Infeasible' as an issue with Gradle's handling of repositories.

 I'd consider the need to use an allprojects block an explicit act of leaking repository sources into other projects.

It's the opposite of a leak. A is in full control where its transitive dependencies come from.

While it's indeed not a leak, it creates tight coupling that's by now discouraged. Instead, you should share build logic / configuraion via buildSrc or convention plugins.
In any case, to me it's a recurring source of confusion that custom repositories need to be re-declared in all consumers 😞


Cost Optimization Workbook - Azure Firewall - "Is Intrusion Detection enabled" not detecting Deny

🐛 Problem
In the Cost Optimization Workbook under Usage Optimization -> Networking -> Azure Firewall - "Is Intrusion Detection enabled?" query, it is showing false when IDPS has been set to "Alert and Deny"
👣 Repro steps

Need a firewall with firewall policy that you can toggle IDPS mode between Disabled/Alert/Alert and Deny
Set to Alert and Deny first (which means IDPS is enabled and blocking suspicious traffic
Go to Cost Optimization Workbook under Usage Optimization -> Networking -> Azure Firewall - "Is Intrusion Detection enabled?" query
It will say False as it is looking for "Alert" in the query

🤔 Expected
It should display True when set to Alert and Deny.  However, this one may be better off than just a True/False.  Alerting, while it is utilizing the setting to get started with IDPS is not blocking traffic.  So maybe False when not using with setting and True (Alert) or True (Alert and Deny) for the other 2 options.
📷 Screenshots

ℹ Additional context
Fix would be to add an -or with Deny, see below.  Tested in last screenshot above, however this is not an environment where I have write access to test the firewall with both Alert and Alert and Deny.
resources
| where type =~ 'Microsoft.Network/azureFirewalls'
| project FWID=id, firewallName = name, SkuTier = tostring(properties.sku.tier), resourceGroup, location
| join kind=inner (
resources
| where type =~ 'microsoft.network/firewallpolicies'
| mv-expand properties.firewalls
| extend intrusionDetection = tostring(properties.intrusionDetection contains "Deny" or properties.intrusionDetection contains "Alert"), transportSecurity = tostring(properties.transportSecurity contains "keyVaultSecretId")
| extend FWID=tostring(properties_firewalls.id)
| project PolicyName = name, PolicySKU=tostring(properties.sku.tier), intrusionDetection, transportSecurity, FWID
) on FWID
🙋♀ Ask for the community

Please vote this issue up (👍) to prioritize it.

Thank you for that @MS-mikeo. I'm adding this fix to the next month release.
@sebassem FYI.

[like]  Mike O'Shaughnessy reacted to your message:

From: Arthur Clares @.>
Sent: Friday, February 23, 2024 10:46:01 AM
To: microsoft/finops-toolkit @.>
Cc: Mention @.>; Author @.>
Subject: Re: [microsoft/finops-toolkit] Cost Optimization Workbook - Azure Firewall - "Is Intrusion Detection enabled" not detecting Deny (Issue #613)
Thank you for that @MS-mikeohttps://github.com/MS-mikeo. I'm adding this fix to the next month release.
@sebassemhttps://github.com/sebassem FYI.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/finops-toolkit/issues/613#issuecomment-1961102143 or unsubscribehttps://github.com/notifications/unsubscribe-auth/A673SREF6GRSF343AWFED2DYVBXOVBFKMF2HI4TJMJ2XIZLTSWBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLAVFOZQWY5LFVI2DSOJXGU3DEOJQG2SG4YLNMWUWQYLTL5WGCYTFNSBKK5TBNR2WLKRVGAYDEMRUGU4TGMVENZQW2ZNJNBQXGX3MMFRGK3FMON2WE2TFMN2F65DZOBS2YSLTON2WKQ3PNVWWK3TUUZ2G64DJMNZZJAVEOR4XAZNKOJSXA33TNF2G64TZUV3GC3DVMWUTKOBXHE3DKOBWGKBKI5DZOBS2K2LTON2WLJLWMFWHKZNKGIYTIOJXGM3TGOBUQKSHI6LQMWSWYYLCMVWKK5TBNR2WLKRUHE4TONJWGI4TANUCUR2HS4DFUVWGCYTFNSSXMYLMOVS2UNJQGAZDENBVHEZTFJ3UOJUWOZ3FOKTGG4TFMF2GK.
You are receiving this email because you were mentioned.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@MS-mikeo Mike, I made a very small change to the code as the intent of this query is to find Premium Firewall with Premium Firewall policies that does not use premium features, such as TLS or IDPS.
resources
| where type =~ 'Microsoft.Network/azureFirewalls' and properties.sku.tier=="Premium"
| project FWID=id, firewallName = name, SkuTier = tostring(properties.sku.tier), resourceGroup, location
| join kind=inner (
resources
| where type =~ 'microsoft.network/firewallpolicies'
| mv-expand properties.firewalls
| extend intrusionDetection = tostring(properties.intrusionDetection contains "Alert" or properties.intrusionDetection contains "Deny"), transportSecurity = tostring(properties.transportSecurity contains "keyVaultSecretId")
| extend FWID=tostring(properties_firewalls.id)
| where intrusionDetection == "False" and transportSecurity == "False"
| project PolicyName = name, PolicySKU=tostring(properties.sku.tier), intrusionDetection, transportSecurity, FWID
) on FWID
The Advisor workbook should be updated in 2 weeks and the Toolkit workbook should be updated in about ~3 weeks. Let me know if you need access to the JSON before that and I'm happy to share it with you.

hey @MS-mikeo Mike, we have released the new version of the workbook with this fix.
Let me know if you have any questions, otherwise I will close this bug 😃

[heart]         Mike O'Shaughnessy reacted to your message:

From: Arthur Clares @.>
Sent: Thursday, March 28, 2024 2:08:58 PM
To: microsoft/finops-toolkit @.>
Cc: Mention @.>; Author @.>; Comment @.***>
Subject: Re: [microsoft/finops-toolkit] Cost Optimization Workbook - Azure Firewall - "Is Intrusion Detection enabled" not detecting Deny (Issue #613)
hey @MS-mikeohttps://github.com/MS-mikeo Mike, we have released the new version of the workbook with this fix.
Let me know if you have any questions, otherwise I will close this bug 😃
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/finops-toolkit/issues/613#issuecomment-2025274690 or unsubscribehttps://github.com/notifications/unsubscribe-auth/A673SRD3CLWRJTBWSM6KV2LY2QP7XBFKMF2HI4TJMJ2XIZLTS6BKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLAVFOZQWY5LFVI2DSOJXGU3DEOJQG2SG4YLNMWUWQYLTL5WGCYTFNSBKK5TBNR2WLKRVGAYDEMRUGI4TEOFENZQW2ZNJNBQXGX3MMFRGK3ECUV3GC3DVMWVDKMJUG4ZDKNRTHE3KI3TBNVS2S2DBONPWYYLCMVWIFJLWMFWHKZNKGU2DKMZZHAZDIMZZURXGC3LFVFUGC427NRQWEZLMVRZXKYTKMVRXIX3UPFYGLLCJONZXKZKDN5WW2ZLOOSTHI33QNFRXHFUCUR2HS4DFVJZGK4DPONUXI33SPGSXMYLMOVS2SNJYG44TMNJYGYZIFJDUPFYGLJLJONZXKZNFOZQWY5LFVIZDCNBZG4ZTOMZYGSBKI5DZOBS2K3DBMJSWZJLWMFWHKZNKGQ4TSNZVGYZDSMBWQKSHI6LQMWSWYYLCMVWKK5TBNR2WLKRVGAYDEMRUGI4TEOECUR2HS4DFUVWGCYTFNSSXMYLMOVS2UNJRGQ3TENJWGM4TNAVEOR4XAZNFNRQWEZLMUV3GC3DVMWVDKNBVGM4TQMRUGM42O5DSNFTWOZLSUZRXEZLBORSQ.
You are receiving this email because you were mentioned.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Awesome Linux is broken

Awesome Linux https://awesomerank.github.io/lists/aleksandar-todorovic/awesome-linux.html build is broken.
Reason described here: https://github.com/aleksandar-todorovic/awesome-linux/pull/40

Fixed.


Create and Update Documentation for the project Milestone 3

This has been documented at the wiki:
https://github.com/airavata-courses/airavata-nextcoud/wiki/Project-Milestone-3


Followup for ZEPPELIN-1785

What is this PR for?
After #1745, zeppelin fail to launch in windows. https://issues.apache.org/jira/browse/ZEPPELIN-1785?focusedCommentId=15754688&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15754688
This due to the shell script difference between windows and *nix. This PR just remove the checking for ZEPPELIN_NOTEBOOK_DIR as it would be created in JVM side if the dir is not existed. I also remove it from  zeppelin.sh for *nix as well.
What type of PR is it?
[Hot Fix |]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-1785

How should this be tested?
Tested it manually in windows and mac. Zeppelin can launch successfully on windows/mac even when the notebook dir doesn't exist.
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@Leemoonsoo @jongyoul

Thanks @zjffdu for this fix.
LGTM and i'm merging this guy to master as a Hotfix.


We Can Go Deeper

This pull request simplifies the JSON machinery and adds the ability to subscript deeper into nested structures using the new JSONKeypath Monoid.

I hope this makes it easier that much easier to knock out #58
Ready for review.

This will take a while.  :P

This looks fine to me, now!  I wouldn't be surprised if there were spelling errors somewhere, though; I couldn't hold the whole thing in my head at once as I usually can with pull requests.  :P

Welp, now I'm paranoid.  I'll deal with typos before the next point release.  There's still more I want to do with this before 1.4.

Nevertheless, :boat:


chore: make addUserPolicies protected to be overridden in ee code

Description

Make addUserPolicies protected to be overridden on EE codebase.

PR fixes following issue(s)
Fixes # (issue number)
Type of change

Chore (housekeeping or task changes that don't impact user perception)

Testing

No testing required as we are widening the scope of a method.

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test

/ok-to-test

UI test case failing: OtherUIFeatures/ApplicationURL_spec.js


Suggestion: Growth Times

A simple suggestion for the addition of average growth time for each seed.

Big fan of this idea, I'll add this to my to-do list!

I have created a Pull Request for this feature to be merged into RuneLite.
Its status can be monitored here: https://github.com/runelite/plugin-hub/pull/2128
I will close this issue once it is successfully merged in.

The update including this feature has been merged into RuneLite, thank you for the suggestion!