Daemon lockup on concurrent containers

Ok, so I'm using my docker-stress tool found here to stress test the daemon.  You can download and build it here:
https://github.com/crosbymichael/docker-stress
If you run with:
docker-stress -c 4 --containers 500

Here is the stress.json:
[
    {"name":"busybox", "args": ["true"]},
    {"name":"busybox", "args": ["sleep", "2"]},
    {"name":"busybox", "flags": ["--volumes-from", "redis", "--link", "redis:redis"], "args": ["true"]},
    {"name":"busybox", "flags": ["-v", "/data"], "args": ["true"]},
    {"name":"busybox", "flags": ["--link" , "redis:redis"], "args": ["ping", "-c", "5", "google.com"]}
]

More repeatable if you change -c 4 to -c 6
Anyways we will hit a kernel bug on unregister net device.
[62903.496161] unregister_netdevice: waiting for vetha59dbae to become free. Usage count = 1
[62913.736257] unregister_netdevice: waiting for vetha59dbae to become free. Usage count = 1
[62923.976168] unregister_netdevice: waiting for vetha59dbae to become free. Usage count = 1
[62934.220242] unregister_netdevice: waiting for vetha59dbae to become free. Usage count = 1
[62944.460258] unregister_netdevice: waiting for vetha59dbae to become free. Usage count = 1

We cannot do much about that, however, it locks up the entire daemon, which should not happen.  You have to restart to get it to a usable state.
Here is the stack of the stuck goroutine:
goroutine 24645 [syscall, 92 minutes, locked to thread]:
syscall.Syscall6(0x2c, 0xc, 0xc208784ec0, 0x20, 0x0, 0xc2087
Oct 27 16:31:47 michael-stress-test docker[2798]: 84d34, 0xc, 0x1, 0x1, 0xc208784ec0)

/usr/local/go/src/syscall/asm_linux_amd64.s:46 +0x5
syscall.sendto(0xc, 0xc208784ec0, 0x20, 0x20, 0x0, 0xc208784d34, 0xc20000000c, 0x0, 0x0)

/usr/local/go/src/syscall/zsyscall_linux_amd64.go:1923 +0x9a
syscall.Sendto(0xc, 0xc208784ec0, 0x20, 0x20, 0x0, 0x7fcbf96f3660, 0xc208784d28, 0x0, 0x0)

/usr/local/go/src/syscall/syscall_unix.go:234 +0xb6
github.com/vishvanandanetlinknl.(*NetlinkSocket).Send(0xc208784d20, 0xc209062300, 0x0, 0x0)

/go/src/github.com/docker/docker/vendor/src/github.com/vishvanandanetlinknlnl_linux.go:327 +0xdd
github.com/vishvanandanetlinknl.(*NetlinkRequest).Execute(0xc209062300, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0)

/go/src/github.com/docker/docker/vendor/src/github.com/vishvanandanetlinknlnl_linux.go:213 +0x126
github.com/vishvanandanetlink.LinkDel(0x7fcbf9715d28, 0xc2091ca280, 0x0, 0x0)

/go/src/github.com/docker/docker/vendor/src/github.com/vishvanandanetlink/link_linux.go:392 +0x251
github.com/docker/libnetwork/drivers/bridge.(*driver).DeleteEndpoint(0xc208556580, 0xc208557040, 0x40, 0xc208c27900, 0x40, 0x0, 0x0)

/go/src/github.com/docker/docker/vendor/src/github.com/docker/libnetwork/drivers/bridge/bridge.go:1039 +0x5c8
github.com/docker/libnetwork.(*endpoint).deleteEndpoint(0xc20856a480, 0x0, 0x0)

/go/src/github.com/docker/docker/vendor/src/github.com/docker/libnetwork/endpoint.go:595 +0x22d
github.com/docker/libnetwork.(*endpoint).Delete(0xc20856a480, 0x0, 0x0)

/go/src/github.com/docker/docker/vendor/src/github.com/docker/libnetwork/endpoint.go:574 +0x71b
github.com/docker/libnetwork.(*sandbox).Delete(0xc208ac51e0, 0x0, 0x0)

/go/src/github.com/docker/docker/vendor/src/github.com/docker/libnetwork/sandbox.go:180 +0x73a
github.com/docker/docker/daemon.(*Container).releaseNetwork(0xc208d50e00)

/go/src/github.com/docker/docker/daemon/container_unix.go:1145 +0x380
github.com/docker/docker/daemon.(*Container).cleanup(0xc208d50e00)

/go/src/github.com/docker/docker/daemon/container.go:342 +0x33
github.co
Oct 27 16:31:47 michael-stress-test docker[2798]: m/docker/docker/daemon.(*containerMonitor).Close(0xc209099650, 0x0, 0x0)

Kernel:
Linux michael-stress-test 4.2.0-16-generic #19-Ubuntu SMP Thu Oct 8 15:35:06 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
4 cores

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Ping @mrjana @sanimej

also see https://github.com/docker/docker/issues/5618 ?

this kernel bug seems to be related when the hairpin nat is enabled, which is not the docker default

This might be a silly question, but is there actually a bug report on bugzilla.kernel.org yet?
I can see various attempts to work around this in docker in the tickets above, but nothing that indicates the kernel developers were informed about a potential bug so they can get a chance to address this issue to fix it properly in the long run..

I am not able to reproduce this, as detailed in related issue: https://github.com/moby/moby/issues/5618#issuecomment-1537076724
No activity on this issue since 2016, and quite a bit has happened since within Docker and Kernel dev since then.
I tried to reproduce with the above example config. Seems some instructions were missing regarding redis, I just ran a redis image with a random volume attached. Not sure if that affects the reproduction. No time or other context was given for when the kernel event message should start appearing by either.
Output after a while from the Go program:
INFO[0307] ran 500 containers in 307.49 seconds (1.63 container per sec. or 0.61 sec. per container)
INFO[0307] failures 0

I believe this is a duplicate of #5618. Kernel devs fixed a number of bugs that would trigger this error in the past 10 years, so I think there's nothing we can do here. Thus, let me close it as stale.


Misc cleanup

Some very minor misc cleanup, separate from the lint fixes in #1726

PTAL @johnpryan

PTAL @johnpryan


[bug]: managed to completely kill webui, even after reboot with nodes

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
cuda
VRAM
8GB
What version did you experience this issue on?
3.0.0
What happened?
I was playing with txt-to-img and decided to try out nodes, I had it enabled, but to my dismay the nodes window wasn't auto-populated with the current working project (txt-to-img) so I figured ok, let me try to load the config from txt-to-img into nodes...
I went back to txt-to-img clicked the info button for the image, jump to the nodes json in the image and just save that to a JSON file and loaded the saved jSON over in the nodes view... and the window went grey, like everything... refreshing didn't resolve the issue.
I restarted the entire invokeai and i'm still getting a blank grey screen, in dev console i'm seeing errors, thought due to obfuscation not much it really tells me on what is actually wrong.
Screenshots

Additional context
The json i saved from txt-to-img that i loaded into nodes that broke everything :) https://pastebin.com/raw/XBMEWQTC
Contact Details
No response

I did manage to get the webui to load, I had to force clear the local-storage for the nodes in the browser for the local url, and that got me back into the UI... I then attempted to reproduce by loading the json again, and sure enough it killed the interface, so apparently somewhere in the json import i think someone forgot to sanitize the imported json, and when it fails the whole interface fails.
What i cleared...

The JSON in the image isn't the JSON that is loaded to load a graph. You need to use the save button after creating a graph to save it.
We haven't built the logic to parse the graph just yet. We'll be sure to provide a smooth experience loading an image's graph - like a simple "Use this image's graph" button - when we polish the node editor.

I'm closing this as non-issue but happy to continue the conversation if you have any questions

How’s this not an issue? It’s not just that importing the json from the image info that doesn’t work it’s that theirs no sanity check in the imported json in the node import…
My issue wasn’t that it didn’t support the json I imported the bug report is that importing invalid json breaks the whole web interface and requires diving into development console to restore any part of the dashboard.
I know it’s experimental so getting changed a lot before going to full release, but still feel their should be a tracked bug or outstanding issue of the fact that invalid json imported crashes the full ui.
It also feels odd to dump the json in the image info that doesn’t match the json that would be used in the nodejs having json displayed and saved differently seems… confusing to the user either the json in the image info should note it’s not for node usage or it should just be properly formatted per node usage.
Also some form of sanity check on importing a node json should be cut out of it doesn’t match the expected format.

This is an experimental feature meant to enable early exploration - the guardrails are off.
Everything in the node editor tab is "at your own risk", insofar as it's very easy to footgun yourself and crash things right now. For example, the connection validation logic is intentionally disabled.
It will always be possible to provide invalid, app-crashing JSON for this particular implementation of graph saving and loading.
Generally, of course I agree 100% that you shouldn't be able to bork the app so thoroughly. A lot of care has been taken to prevent issues like this - in the areas of the app that are not marked as experimental. I hope that's clear as you use the app.
Given that...

this is an experimental implementation of graph saving and loading
it's not possible to provide full validation with this implementation, so it will always be able to crash the app fully
it will be rewritten for release in a more sophisticated way with full validation
there is a fix (to clear local storage)

... the solution to this issue is "don't do the thing that we know so break the app, and if you do, reset local storage".
I'm sorry you had the very unpleasant experience of a fully and fully borked ui.

Closing as the "borked UI" issue is fixed in main and will be issued on next release


JDK 9 might ship with a more space efficient String implementation

I stumbled across this issue on the openjdk bug tracker https://bugs.openjdk.java.net/browse/JDK-8054307 . It seems like JDK9 might ship with a more space efficient implementation for Strings, similar to what ByteString / AsciiString are trying to achieve. This is just meant as an FYI :-).

Thanks for the heads up.  Perhaps we can utilize this some day :)


Cannot resolve 'file' or 'directory' ./style

I'm using the React-Starter-Kit and I'm attempting to add React-Toolbox to it for the Navigation, but I keep running into this error whenever I attempt to import something in.
ERROR in ./~/react-toolbox/lib/button/Button.js     Module not found: Error: Cannot resolve 'file' or 'directory' ./style in C:\Users\light1c3\Documents\GitHub\ResumeSite\node_modules\react-toolbox\lib\button      @ ./~/react-toolbox/lib/button/Button.js 28:13-31

do you have a scss loader enabled?

Hi @felipeleusin, I am also getting a similar error and I have a SCSS loader enabled. Here is how my webpack module looks like in my webpack.config.js file:
  module: {
    loaders: [
      {
        test: /(\.js|\.jsx)$/,
        loaders: ['react-hot', 'babel'],
        exclude: /node_modules/,
      }, {
        test: /(\.scss|\.css)$/,
        loader: ["style", "css", "sass"]
      }
    ]
  },

Can you try using ?modules to the css loader?
loader: ["style", "css?modules", "sass"]

should do the trick

I'll give that a shot and update the post once I've tried it.

I can't find that section in my webpack.config.js. Where should I insert the code?

This is how styles are imported in components, e.g. in IconButton.js
import style from './style';
Webpack can't find proper loader for file without extension, that cause an error

Hi, is there any fix for this yet? I am trying to check if it's possible to import relative files and resolve their extension.

Sorry @simofacc I was mistaken. It's not a bug of React Toolbox.
You need add a few lines to your Webpack config:
0. install dependencies npm install --save-dev<EMAIL_ADDRESS>sass-loader (it's important to install version 3.4.2 of node-sass—look here for more details);

add .scss extension to resolve section like here;
then, add new loader (toolbox-loader isn't necessary).

I hope this will help.

@xomyaq That is what I am trying to do. I installed<EMAIL_ADDRESS>but I got a different error
ERROR in ./~/react-toolbox/lib/dropdown/style.scss
Module build failed: ReferenceError: window is not defined

I also keep on getting
    ERROR in ./~/css-loader?sourceMap&modules!./~/react-toolbox/lib/input/style.scss     Module not found: Error: Cannot resolve 'file' or 'directory' ../base in /Applications/AMPPS/www/iGame/node_modules/react-toolbox/lib/input      @ ./~/css-loader?sourceMap&modules!./~/react-toolbox/lib/input/style.scss 3:10-79

thanks. will follow

@simofacc and @Light1c3 Show us your full Webpack config or at least the part around resolve. Your errors may be caused by (missing) resolve.modulesDirectories.

@bernhardw I managed to fix my issues. Had to remove node-sass first and then reinstall<EMAIL_ADDRESS>then I had to install some other packages like sass-loader and toolbox-loader.
I also added the resolve to the configuration too.

Got the same error.
An idea:
Why not release this module with the scss files already after being processed into css?
If this isn't possible - can we add that into the readme?

It´s not possible because in that fancy you are not able to customize the style with SASS variables.

@javivelasco understandable.
From last night's experience - I've ran into too many issues installing the react-toolbox (all of them regarding the styling) causing me to drop it just for the sake of delivering what I needed.
Will keep track :+1:

@aviadhahami Note - one thing that caused an issue for me was excluding the node_modules folder in my css/sass loader.

@aviadhahami it will be solved with #498 hopefully. Check please

@simofacc can you please share your webpack.config.js ..
How you made resolve entry and module entry .. in react-starter-kit

**"You need add a few lines to your Webpack config:
0. install dependencies npm install --save-dev<EMAIL_ADDRESS>sass-loader (it's important to install version 3.4.2 of node-sass—look here for more details);

add .scss extension to resolve section like here;
then, add new loader (toolbox-loader isn't necessary)."**

It must be --save-dev<EMAIL_ADDRESS>When I do it a lot of errors includind an ask to instal Python2 and VisualStudio2013...

So, im running into this issue as well, pretty sure its my webpack config.
This is that I already had a CSS loader but not sure how to integrate with this one. This is my current file:
resolve: {         extensions: ['', '.js', '.jsx','.scss'],     },     module: {         preLoaders: [             {                 test: /\.jsx?$/,                 exclude: /(node_modules|bower_components)/,                 loaders: ['eslint-loader'],             },         ],         loaders: [             {                 test: /\.jsx?$/,                 exclude: /(node_modules|bower_components)/,                 loader: 'babel-loader',                 query: {                     presets: ['react', 'es2015', 'stage-0'],                     plugins: [                         'react-html-attrs',                         'transform-class-properties',                         'transform-decorators-legacy',                     ],                 },             },             {                 test: /\.css$/,                 loaders: [                     'style?sourceMap',                     'css?modules&importLoaders=1&localIdentName=[path]___[name]__[local]___[hash:base64:5]',                 ],             },             {                 test: /\.json$/,                 loader: 'json-loader',             },             {                 test: /\.scss$/,                 loaders: ["style", "css", "sass"]             }         ],
Thanks in advance!

Check the SCSS loader configuration. You are not including modules option to enable CSS Modules!


PP-5023: Emit time to expiry metric

Emit the Apple Pay merchant identity certificate's time-to-expiry (in
milliseconds) metric on server startup.
There are few logger.debug lines which I will delete when I've verified it all works in the test environment.
WHAT
A brief description of the pull request:
HOW
Steps to test or reproduce:

@sfount fixed


Error adding Kaltura resource

Caused by the great plugin merge of 2018
Template not found for name<EMAIL_ADDRESS>The name was interpreted by this TemplateLoader: com.tle.web.freemarker.CustomTemplateLoader
Steps to reproduce:

Open a contribution wizard containing a kaltura attachment control
Search for a kaltura video, select it and press the green add button
You will get an error message

Fixed on the Kaltura repo


Font functions change the text selection

When using the highlight or any font related function (except fixed width) on marked text, the new settings is applied, but it expands the mark by 1 character on each click, which should not happen.

Yeah, I had not noticed that but must have seen it a hundred times !
Trivial fix, done.


Shouldn't be able to try to create a new folder in read only directories

Fixed by removing the 'Create Folder' option in 54a40c634c2e62a60a4e66eb7b76d223f994500a.


Unable to recognize tab page YouTube-dl and can't download videos from YouTube shorts

Checklist

[x ] I'm reporting a broken site support
[x ] I've verified that I'm running youtube-dl version 2021.12.17
[x ] I've checked that all provided URLs are alive and playable in a browser
[ x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ x] I've searched the bugtracker for similar issues including closed ones

Verbose log

youtube-dl --verbose 'https://www.youtube.com/shorts/o38ilgL25lc'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.youtube.com/shorts/o38ilgL25lc']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Git HEAD: fed93bb93
[debug] Python version 3.9.10 (CPython) - macOS-12.3-arm64-arm-64bit
[debug] exe versions: none
[debug] Proxy map: {}
[youtube:tab] shorts: Downloading webpage
ERROR: Unable to recognize tab page; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/opt/homebrew/lib/python3.9/site-packages/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/opt/homebrew/lib/python3.9/site-packages/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/opt/homebrew/lib/python3.9/site-packages/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/opt/homebrew/lib/python3.9/site-packages/youtube_dl/extractor/youtube.py", line 2862, in _real_extract
    raise ExtractorError('Unable to recognize tab page')
youtube_dl.utils.ExtractorError: Unable to recognize tab page; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.```

## Description

<!--
Provide an explanation of your issue in an arbitrary form. Provide any additional information, suggested solution and as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
-->

Unable to recognize tab page YouTube-dl and can't download videos from YouTube shorts

Just use the ID:
PS C:\> youtube-dl o38ilgL25lc
[youtube] o38ilgL25lc: Downloading webpage
[youtube] o38ilgL25lc: Downloading player 293baa5d
[download] Destination: When u sit in the emergency exit row 😂-o38ilgL25lc.f398.mp4
[download]   3.4% of 5.95MiB at 80.93KiB/s ETA 01:12

Also, duplicate of #30269; fixed in git master.

https://github.com/ytdl-org/youtube-dl/issues/30813

https://github.com/ytdl-org/youtube-dl/issues/30834

https://github.com/ytdl-org/youtube-dl/issues/31202

Somebody on here got back to me a while ago and you don’t put the entire URL, just the video ID at the end of the URL, and it works perfect.
Wow while everyone’s here I know this probably goes against forum decorum,but I  have a css landing page template for a website that I gotta get done today and I can’t get the page to format/render correctly… I’m using concrete CMS and of course they’ve taken all of the necessary steps to make it as difficult as possible for customers to get what they want.  I had some code that worked as an override to the CMS but maybe I just put it in the wrong location. Any tips greatly appreciated.
photograph Michael Patmon Marketing Designer M<PHONE_NUMBER> E @.*** My Company Street, City Zip Code, Country www.my-company.com facebook icon twitter icon youtube icon linkedin icon instagram icon pinterest icon Banner
On Oct 5, 2022 at 1:21 AM -0700, Littlemac123 @.***>, wrote:

(base) ***@***.***:~/Downloads/linux$ mpv https://www.youtube.com/shorts/SG8409ppUE8 [ytdl_hook] ERROR: Unable to recognize tab page; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type youtube-dl -U to update. Be sure to call youtube-dl with the --verbose flag and include its complete output. [ytdl_hook] youtube-dl failed: unexpected error occurred [ytdl_hook] It appears that your youtube-dl version is severely out of date. Failed to recognize file format. Exiting... (Errors when loading file) (base) ***@***.***:~/Downloads/linux$ yt-dlp --verbose [debug] Command-line config: ['--verbose'] [debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8 [debug] yt-dlp version 2022.10.04 [4e0511f] (zip) [debug] Python 3.9.1 (CPython 64bit) - Linux-5.4.0-126-generic-x86_64-with-glibc2.31 (glibc 2.31) [debug] Checking exe version: ffmpeg -bsfs [debug] Checking exe version: ffprobe -bsfs [debug] exe versions: ffmpeg 4.2.7, ffprobe 4.2.7, rtmpdump 2.4 [debug] Optional libraries: brotli-None, certifi-2020.12.05, sqlite3-2.6.0 [debug] Proxy map: {} [debug] Loaded 1690 extractors Usage: yt-dlp [OPTIONS] URL [URL...] yt-dlp: error: You must provide at least one URL. Type yt-dlp --help to see a list of all options.
its still not fixed
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

just the video ID at the end of the URL

As I said at https://github.com/ytdl-org/youtube-dl/pull/30577#issuecomment-1292927326 , I'm still impressed that this is possible in YTDL but not YT itself lol


Remove reliance on classes

Look at href to programmatically determine link type and action
Provide an optional class to override this behavior

Added: commit


Watch mode unhandled promise rejection (failed to parse JSON)

Running estrella in the watch mode in some cases produces the following error:
Wrote dist/app.js (129.9kB, 72.89ms) Watching files for changes... Unhandled promise rejection: Error: failed to parse dist\.esbuild.7czfvj1r7mzo41vo8xvg1utpi4512zaw2z.meta.json: Unexpected token ','     at Be (d:\temp\estrella-test-1\sample\node_modules\estrella\<estrella>\util.js:133:11)     at y (d:\temp\estrella-test-1\sample\node_modules\estrella\<estrella>\estrella.js:754:23)     at i (d:\temp\estrella-test-1\sample\node_modules\estrella\<estrella>\watch\watch.ts:52:25)     at Object.nl (d:\temp\estrella-test-1\sample\node_modules\estrella\<estrella>\watch\watch.ts:99:3)     at rn (d:\temp\estrella-test-1\sample\node_modules\estrella\<estrella>\estrella.js:768:23)     at processTicksAndRejections (internal/process/task_queues.js:93:5)
If I run build without the watch mode it completes without errors with a valid JS and CSS files.
I've attached the sample project that reproduces the error.
sample.zip
Environment:
Windows 10
NodeJS - v14.15.0

This might be due to https://github.com/evanw/esbuild/pull/504, which was fixed
In your package.json, if you set this:
"resolutions": {
  "esbuild": "0.8.5",
}

it seems to work

Thanks, resolution to the latest esbuild fixes the issue.
So I think the issue may be closed when estrella will update dependency to esbuild version 0.8.5+.


Spyder freeze after code execution

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[ ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[x] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder
[x] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[x] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
Spyder seems to freeze on IPython execution of any command for 10 seconds. That includes variable assignment (x = 10) and the import of libraries (import math as m). Scripts run fine but I still get the freeze after submitting the request. I have limited possiblities to look into the backend as this is at work in an environment provided at goodwill from our IT department and I can only access the program via a Citrix environment, one colleague however agreed to help me look into the issue. He can reproduce it on the VM the IDE runs on.
What steps reproduce the problem?

Open Spyder (default setup)
Execute any command in IPython
Execution seems to succeed, then ten seconds of unresponsive state from the IDE in which nothing reacts.

What is the expected output? What do you see instead?
I went and looked into a logfile of the above process(Opening spyder, executing x = 10, closing it again.
I suspect either kite or pyls as the problem driver, I found some issues regarding pyls, however these should have been fixed with the provided version (4.1.2). Looking into the section of the logfile where I execute a command, one can see the freeze:
2020-06-26 11:59:07,169 [DEBUG] [spyder.plugins.completion.languageserver.client] -> python response: textDocument/publishDiagnostics
2020-06-26 11:59:07,169 [DEBUG] [spyder.plugins.editor.plugin] -> python completion server request: 'textDocument/foldingRange'
2020-06-26 11:59:07,182 [DEBUG] [spyder.plugins.completion.languageserver.client] -> python request: textDocument/foldingRange
2020-06-26 11:59:07,182 [DEBUG] [spyder.plugins.completion.plugin] -> Completion plugin: Request 8 Got response from kite
2020-06-26 11:59:07,183 [DEBUG] [spyder.plugins.editor.plugin] -> python completion server request: 'textDocument/documentSymbol'
2020-06-26 11:59:07,183 [DEBUG] [spyder.plugins.completion.languageserver.client] -> python request: textDocument/documentSymbol
2020-06-26 11:59:07,183 [DEBUG] [spyder.plugins.completion.plugin] -> Completion plugin: Request 9 Got response from kite
2020-06-26 11:59:07,212 [DEBUG] [spyder.plugins.completion.plugin] -> Completion plugin: Request 8 Got response from lsp
2020-06-26 11:59:07,213 [DEBUG] [spyder.plugins.completion.plugin] -> Completion plugin: Request 8 removed
2020-06-26 11:59:07,213 [DEBUG] [spyder.plugins.completion.plugin] -> Gather responses for textDocument/foldingRange
2020-06-26 11:59:07,383 [DEBUG] [spyder.plugins.completion.plugin] -> Completion plugin: Request 9 timed out
2020-06-26 11:59:07,539 [DEBUG] [spyder.plugins.completion.plugin] -> Completion plugin: Request 9 Got response from lsp
2020-06-26 11:59:07,539 [DEBUG] [spyder.plugins.completion.plugin] -> Completion plugin: Request 9 removed
2020-06-26 11:59:07,539 [DEBUG] [spyder.plugins.completion.plugin] -> Gather responses for textDocument/documentSymbol
2020-06-26 11:59:10,282 [DEBUG] [traitlets] -> execute_input: {'code': 'x = 10', 'execution_count': 1}
2020-06-26 11:59:11,435 [DEBUG] [traitlets] -> execute_reply: {'status': 'ok', 'execution_count': 1, 'user_expressions': {}, 'payload': []}
2020-06-26 11:59:11,451 [DEBUG] [traitlets] -> handle_msg[9c1662d8b79311eab213005056be56e6]({'header': {'msg_id': 'f31622c4-94110c2fdea6112588bd9c3d_43', 'msg_type': 'comm_msg', 'username': 'username', 'session': 'f31622c4-94110c2fdea6112588bd9c3d', 'date': datetime.datetime(2020, 6, 26, 9, 59, 11, 451128, tzinfo=tzutc()), 'version': '5.3'}, 'msg_id': 'f31622c4-94110c2fdea6112588bd9c3d_43', 'msg_type': 'comm_msg', 'parent_header': {'msg_id': '8789e027-adccd8715cfcc768070580c9_16', 'msg_type': 'execute_request', 'username': 'username', 'session': '8789e027-adccd8715cfcc768070580c9', 'date': datetime.datetime(2020, 6, 26, 9, 59, 10, 282647, tzinfo=tzutc()), 'version': '5.3'}, 'metadata': {}, 'content': {'data': {'spyder_msg_type': 'remote_call_reply', 'content': {'is_error': False, 'call_id': '664c42e75e634dafbbf34cfd182477fd', 'call_name': 'get_namespace_view'}, 'pickle_protocol': 4, 'python_version': '3.7.7 (default, May  6 2020, 11:45:54) [MSC v.1916 64 bit (AMD64)]'}, 'comm_id': '9c1662d8b79311eab213005056be56e6'}, 'buffers': [<memory at 0x0000000AD3E32648>]})
2020-06-26 11:59:11,451 [DEBUG] [traitlets] -> handle_msg[9c1662d8b79311eab213005056be56e6]({'header': {'msg_id': 'f31622c4-94110c2fdea6112588bd9c3d_45', 'msg_type': 'comm_msg', 'username': 'username', 'session': 'f31622c4-94110c2fdea6112588bd9c3d', 'date': datetime.datetime(2020, 6, 26, 9, 59, 11, 451128, tzinfo=tzutc()), 'version': '5.3'}, 'msg_id': 'f31622c4-94110c2fdea6112588bd9c3d_45', 'msg_type': 'comm_msg', 'parent_header': {'msg_id': '8789e027-adccd8715cfcc768070580c9_19', 'msg_type': 'comm_msg', 'username': 'username', 'session': '8789e027-adccd8715cfcc768070580c9', 'date': datetime.datetime(2020, 6, 26, 9, 59, 11, 451128, tzinfo=tzutc()), 'version': '5.3'}, 'metadata': {}, 'content': {'data': {'spyder_msg_type': 'remote_call_reply', 'content': {'is_error': False, 'call_id': '4bf04aed8d1f4060b5e718bc5f5b2632', 'call_name': 'get_cwd'}, 'pickle_protocol': 4, 'python_version': '3.7.7 (default, May  6 2020, 11:45:54) [MSC v.1916 64 bit (AMD64)]'}, 'comm_id': '9c1662d8b79311eab213005056be56e6'}, 'buffers': [<memory at 0x0000000AD5A72D08>]})
2020-06-26 11:59:21,092 [DEBUG] [traitlets] -> handle_msg[9c1662d8b79311eab213005056be56e6]({'header': {'msg_id': 'f31622c4-94110c2fdea6112588bd9c3d_47', 'msg_type': 'comm_msg', 'username': 'username', 'session': 'f31622c4-94110c2fdea6112588bd9c3d', 'date': datetime.datetime(2020, 6, 26, 9, 59, 11, 451128, tzinfo=tzutc()), 'version': '5.3'}, 'msg_id': 'f31622c4-94110c2fdea6112588bd9c3d_47', 'msg_type': 'comm_msg', 'parent_header': {'msg_id': '8789e027-adccd8715cfcc768070580c9_19', 'msg_type': 'comm_msg', 'username': 'username', 'session': '8789e027-adccd8715cfcc768070580c9', 'date': datetime.datetime(2020, 6, 26, 9, 59, 11, 451128, tzinfo=tzutc()), 'version': '5.3'}, 'metadata': {}, 'content': {'data': {'spyder_msg_type': 'remote_call_reply', 'content': {'is_error': False, 'call_id': '8f74d8cbfc8345fa88cce17381fb1c72', 'call_name': 'get_var_properties'}, 'pickle_protocol': 4, 'python_version': '3.7.7 (default, May  6 2020, 11:45:54) [MSC v.1916 64 bit (AMD64)]'}, 'comm_id': '9c1662d8b79311eab213005056be56e6'}, 'buffers': [<memory at 0x0000000AD5A727C8>]})
2020-06-26 11:59:21,233 [DEBUG] [spyder.plugins.editor.plugin] -> python completion server request: 'textDocument/cursorEvent'
2020-06-26 11:59:21,238 [DEBUG] [spyder.plugins.completion.plugin] -> Completion plugin: Request 10 Got response from kite
2020-06-26 11:59:21,433 [DEBUG] [spyder.plugins.completion.plugin] -> Completion plugin: Request 10 timed out

Execution happens at 11:59:10, short response at 11:59:11, then the IDE freezes and only reacts after 11:59:21. Is the last line significant here?
 Please update spyder to our latest version. In your anaconda prompt run:
conda update anaconda
conda install spyder=4.1.4
Please let me know if the error persists after this.

Hi @juanis2112, we did this this morning, unfortunately the error persisted.

Can you please post here the output of conda list in your working environment?

We found the issue! It's gladly nothing with the specific version of Spyder. Sorry for the delays by the way, corporate environments are slower :)
When a command is executed, a temporary file is created in the .spyder-py3 folder, example name is tmp4ayti0fz. From what we have seen, that file contains folders, similar to
C:\xxx\yyy\.spyder-py3
C:\xxx\yyy
P:\xxx\yyy\Spyder

The virtual machine hosting Spyder needs to run an IT management tool similar to AppSense which, at execution, detects the file, tries to see if it is registered, fails to do so and deletes the file, leading to bespoken freeze. What is that file? I think that wasn't the issue with Spyder 3 IIRC.

@impact27, I think those file are created to temporarily save the contents of unsaved files before execution, right?

Pinging @impact27 again about this one.

Hey @dbischof90 , thanks for the further investigation. I'd personally suggest letting your IT people know to whitelist the files in that directory, or otherwise ask how they can mitigate the problem, since the issue appears to be, as you say, with your particular security software and not Spyder and I haven't seen this reported elsewhere, so I'm not sure how much sense it makes to do something about this from the Spyder side, or how effective that would be.

@impact27, I think those file are created to temporarily save the contents of unsaved files before execution, right?

The home directory is C:\xxx\yyy and the spyder executable is probably P:\xxx\yyy\Spyder.  The folder C:\xxx\yyy\.spyder-py3 is in the home directory and contains a bunch of things, among which:

The autosave of the files
The config files of spyder
The shell history
The temp.py file

It is important that spyder can access those files.

This looks like a file created by the tempfile module. Looking through the spyder code I can see for example: https://github.com/spyder-ide/spyder/blob/2515fa78e8d8c20848d3fddc911e4f04f28d21d4/spyder/plugins/help/utils/sphinxify.py#L189

Hey all, thanks for chipping in.
@CAM-Gerlach, yes, the problem "luckily" seems to be independent of Spyder and has to do with our IT management tools. It's now for me to understand what I have to tell IT to explain the problem and provide a viable solution, I'm not too certain right now that "just by default whitelist all these files in that folder" will go well.
@impact27 I see. Is it possible to modify the folder location of where these files are created and stored? Maybe that would open up some additional options for IT to work on a solution. From what I have seen, the personal user folders are set to some hashed location and it seems like it's at least not trivial to generalize a sane whitelisting rule. All that happens in the .spyder-py3 folder, right? Can the location of that folder be changed somewhere or is that more or less fixed? Can the location of the files you mentioned above be modified? The location you linked seems to call mkdtemp with default arguments, is there a way to parametrize this?
Still open question (but rather out of personal interest): We did not have this issue with Spyder 3. I can only guess how the environment looked before but what changed with regards of these temp files between release 3 and 4?
Thanks a lot for your help all by the way, really appreciated.

@impact27 I see. Is it possible to modify the folder location of where these files are created and stored? Maybe that would open up some additional options for IT to work on a solution. From what I have seen, the personal user folders are set to some hashed location and it seems like it's at least not trivial to generalize a sane whitelisting rule. All that happens in the .spyder-py3 folder, right? Can the location of that folder be changed somewhere or is that more or less fixed? Can the location of the files you mentioned above be modified? The location you linked seems to call mkdtemp with default arguments, is there a way to parametrize this?

I am not sure if this is the correct line in the code. This would need to be investigated. The problem with moving the .spyder folder is that this folder contains the settings, so where do you store the setting of where the settings are?

That's true - what about a possibility in the settings to modify the location those files get created? I didn't find any by browsing the config files I could find in the folder. Is there some non-default setting that could be changed here?

There's no way to adjust the location of our preferences directory, sorry. Given that this is a very narrow use case, please consider to tell your company to fund that work through a Quansight Labs Community Work Order. That's basically a contract between a company and Spyder (mediated by Quansight Labs) to fund specific features or developments in the project.

@dbischof90 There is currently work being done to enable setting a custom config path in issue #4150 , which is useful for a variety of users beyond just your specific situation. Given Spyder is developed by and for the community thanks to uses like you, we'd welcome your contributions there!
Alternately, I imagine the way to do it would be overriding the config directory location via an environment variable that Spyder could read on startup, though I'd suggest just going along with the approach already in work on the existing issue.

@dbischof90 There is currently work being done to enable setting a custom config path in issue #4150 , which is useful for a variety of users beyond just your specific situation. Given Spyder is developed by and for the community thanks to uses like you, we'd welcome your contributions there!
Alternately, I imagine the way to do it would be overriding the config directory location via an environment variable that Spyder could read on startup, though I'd suggest just going along with the approach already in work on the existing issue. You could have a look at my past PR, #8837, for some background on how this can be set during early startup.

I'd like to clarify that I mentioned the option of funding this work through Quansight Labs because we don't have plans to implement it soon ourselves. As @CAM-Gerlach said, it'd be useful to have but it's very low priority for us.

Just to clarify as well, by "work being done" I meant by community volunteers and not the core Spyder team. Funding it is an option (and would probably not involve too much of an outlay, especially compared to the benefit for your specific case), and so is contributing the feature yourself; as you presumably are relatively experienced with Python it shouldn't be too difficult following the examples given in the linked issues/PRs, if you're so inclined. Thanks!

Okay, understood - so there is currently no way to modify the location (or at least parametrize the file itself) for these temporary files. I think the chances are quite slim for this to be honest but I'll check.
Were those temporary files also created under Spyder 3.x? We work with Spyder for over three years and only had troubles after the upgrade to Spyder 4.

I can confirm this bug on Archlinux with spyder 4.1.3 (https://termbin.com/exl9)

Were those temporary files also created under Spyder 3.x?

No, they weren't. That's a new feature in Spyder 4.

Closing because this is not caused by Spyder but by an external software and we don't have control over it.


Support EXTERNAL TABLEs for deltalake

          > This looks good — just a little comment.

Should add the EXTERNAL TABLE support as well.

I opted to not do that in this PR since I'm now sure how we want to handle this. Delta files are able to be accessed without a catalog (since they're just files in object storage). If we added EXTERNAL TABLE support, what would we do? Require that a catalog is provided, or just have the user specify the path to the file?
It's not clear to me what the best solution is here. I figured we'll learn more while we continue to flesh this out.

You also mentioned connecting without "catalog". How does that work? A catalog is just a database, right?

The catalog in the case of deltalake just provides us the location of objects in some object store. The catalog isn't needed if the location of an object is known ahead of time, since all that's needed to read/modify a delta file is self-contained.
For example, the databricks deployment I have set up on AWS stores delta files in s3. Making the GET request for one of the tables will just return the table location in s3. I then use the credentials provided when actually access those objects in s3.
Originally posted by @scsmithr in https://github.com/GlareDB/glaredb/issues/1119#issuecomment-1591138471

delta_scan ?


Height of box-content not resizing to highcharts graphs

I am using the content class section to hold a box class with a highcharts graph inside of it.

When using the chart without the adminLTE template, I can set the height to 100% like as seen in this jfiddle: http://jsfiddle.net/wkkAd/149/
When I place the chart in the adminLTE template, the height attributes are not resizing the graph to fit 100% of the box.
The only way I can change the height is by manually setting the height of the graph by pixels and not by percentage.
The highcharts documentation states that the height of the graph is dynamically as long as you set a height on the chart's containing element. It can be a fixed number or a even a percent if position is absolute.
The box class already has a default height of 100%, and setting the box-body to 100% doesn't work either.
Any ideas on how to make the content within the box or box-body class all 100%?

Hello,
This is a bit of tough one. In order to get all of the containers of the graph to have a specified width then you need to set all of it's parent components individually.
I would try doing this in JS before initializing the graph:
$('#graph").parents().each(function(){
  $(this).css('height', '100%');
})

I have a feeling this will cause all sorts of other issues, but deserves a try.
I am sorry we can't help you any further here since this wouldn't be an issue that is directly related to an AdminLTE feature. Sorry for the inconvenience.
Thanks!


Things we need to do

Change Background
create and change sprite
Add a crisis when you fall
Extend the bottom platform

You can use * [ ]  before each item to create a check list.
For example:

[ ] item 1
[ ] item 2


Typescript typing inconsistencies for the logout function

Checklist

[X] The issue can be reproduced in the auth0-react sample app (or N/A).
[X] I have looked into the Readme, Examples, and FAQ and have not found a suitable solution or answer.
[X] I have looked into the API documentation and have not found a suitable solution or answer.
[X] I have searched the issues and have not found a suitable solution or answer.
[X] I have searched the Auth0 Community forums and have not found a suitable solution or answer.
[X] I agree to the terms within the Auth0 Code of Conduct.

Description
https://github.com/auth0/auth0-react/blob/bcfd2199b616ca50c20c2701acbd6e7e2e06f92b/src/auth0-provider.tsx#L199-L200
logout here is a promise , while when we use the hook to retrieve it is not typed as such , and we will end up with type inconsistencies
https://github.com/auth0/auth0-react/blob/bcfd2199b616ca50c20c2701acbd6e7e2e06f92b/src/auth0-context.tsx#L130
why they are different is it intentional ?
Reproduction

Check the interface typing
Check the code that is run with the actual provider

Additional context
No response
auth0-react version
2.1.0
React version
18.2.0
Which browsers have you tested in?
Other

Thanks, I opened a PR to address this.


Documentation for migration from aad-pod-identity to azure-workload-identity

Document steps for migration from aad-pod-identity to azure-workload-identity
Dependencies:

https://github.com/Azure/azure-workload-identity/issues/325

One thing that's not clear to me is this: If I enable Workload Identity on an existing AKS cluster that's currently using AAD Pod Identity, will this interfere with each other or is it safe to enable WI and gradually enable the actual WI logic per workload (by adding the annotation(s) to the relevant components). All of this without using the sidecar container.


Added GitLab

#1077

@kgizdov "accepting pull requests" not the same as "actively working on it".

@Carlgo11 perhaps you were a bit premature closing this. @kgizdov can easily update his pull request to remove that status line.

Ah, sorry. Should have read through it more carefully.

status removed


Link Series to homepage

As it's sort of our premium feature right now, we should not miss it out on homepage

I would like to work on this issue.

Where should the Series be in the list of premium features?


aws staging: traefik fails to generate certificate

time="2023-10-17T12:33:54Z" level=error msg="Error renewing certificate from LE: {testing.osparc-staging.io []}" providerName=myresolver.acme error="error: one or more domains had a problem:\n[testing.staging.sim4life.io] [testing.staging.sim4life.io] acme: error presenting token: route53: failed to determine hosted zone ID: zone sim4life.io. not found for domain _acme-challenge.testing.staging.sim4life.io.\n" ACME CA="https://acme-v02.api.letsencrypt.org/directory"
time="2023-10-17T12:33:55Z" level=error msg="Error renewing certificate from LE: {testing.api.osparc-staging.io [api.testing.staging.osparc.io]}" ACME CA="https://acme-v02.api.letsencrypt.org/directory" providerName=myresolver.acme error="error: one or more domains had a problem:\n[api.testing.staging.sim4life.io] [api.testing.staging.sim4life.io] acme: error presenting token: route53: failed to determine hosted zone ID: zone sim4life.io. not found for domain _acme-challenge.api.testing.staging.sim4life.io.\n"
time="2023-10-17T12:33:56Z" level=error msg="Error renewing certificate from LE: {api.osparc-staging.io []}" ACME CA="https://acme-v02.api.letsencrypt.org/directory" providerName=myresolver.acme error="error: one or more domains had a problem:\n[api.staging.sim4life.io] [api.staging.sim4life.io] acme: error presenting token: route53: failed to determine hosted zone ID: zone sim4life.io. not found for domain _acme-challenge.api.staging.sim4life.io.\n"
time="2023-10-17T12:33:58Z" level=error msg="Error renewing certificate from LE: {osparc-staging.io [invitations.osparc-staging.io traefikdashboard.osparc-staging.io]}" error="error: one or more domains had a problem:\n[invitations.staging.sim4life.io] [invitations.staging.sim4life.io] acme: error presenting token: route53: failed to determine hosted zone ID: zone sim4life.io. not found for domain _acme-challenge.invitations.staging.sim4life.io.\n[staging.sim4life.io] [staging.sim4life.io] acme: error presenting token: route53: failed to determine hosted zone ID: zone sim4life.io. not found for domain _acme-challenge.staging.sim4life.io.\n" ACME CA="https://acme-v02.api.letsencrypt.org/directory" providerName=myresolver.acme

I fixed it manually by removing the outdated sim4life.io entries from acme.json. This domain was no longer in our DNS and thus LE DNS Challenges wre failing leading to the errors presented above


Fix CI/presubmit for observability

Fixes https://github.com/knative/test-infra/issues/749

fixed/updated e2e-tests.sh script
fixed/updated presubmit-tests.sh script
fixed/updated verify-codegen.sh
added hack/update-deps.sh for easier dep update (aligned with the other repos)
updated dependencies
vendored test-infra instead of submoduling it (looks like Prow doesn't like it)
updated test-infra

/assign @dprotaso

/hold
Looks like submodules confuse Prow.

/hold cancel
All issues resolved now, all tests are passing.

/assign @Benjamintf1 @wfernandes

/lgtm

/approve

/approve

@adrcunha Ty for the contribution!


etc: Add Debian init.d script for mptcpd.

Add a Debian init.d script for mptcpd for the case where start of
mptcpd through systemd is not available or undesirable.

@matttbe FYI

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 53.266%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1060

Relevant Lines:
1990

💛  - Coveralls

The etc/init.d/mptcpd script generated by this change is installed even when systemd is detected by the configure script.  I'm not sure that is desirable.

I don't yet know if other platforms such as Fedora support this Debian style init.d script.

The etc/init.d/mptcpd script generated by this change is installed even when systemd is detected by the configure script. I'm not sure that is desirable.

Yeah, that doesn't seem good - but I don't know how packagers usually handle installation of sysv-init vs systemd scripts

The etc/init.d/mptcpd script generated by this change is installed even when systemd is detected by the configure script. I'm not sure that is desirable.

Yeah, that doesn't seem good - but I don't know how packagers usually handle installation of sysv-init vs systemd scripts

If I'm not mistaken, you can install both the init.d script and systemd service. If you use systemd and there is a service, the init.d script will be ignored I guess. e.g. for iwd: https://salsa.debian.org/debian/iwd/-/commit/65a0232b9715d9002d69d55b217ae6e3f3c75ffb
If it eases stuff, only having the file in the tarball and not installing it by default (or only with a configure option) would help packagers.
If the init.d script you have is Debian-specific, we can add it in the Debian package directly.

(nit) the hashbang doesn't need to go through env, that's just a limitation of FreeBSD that's irrelevant here (because 1. if FreeBSD grows MPTCP support, it won't use Linux-specific internal management interfaces thus this daemon, 2. it was fixed by making /lib/init/init-d-script a binary stub)
description: "Debian" is merely the first distro to support, any other that uses init scripts (Slack, Alpine, ...) can use the script as-is
description: "init script" is redundant here
messing with LD_LIBRARY_PATH is indeed unneeded and harmful


DOC: Clarify integration plans with NiBabel in the README

Codecov Report

Merging #128 (d982d6a) into master (d91f9ae) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head d982d6a differs from pull request most recent head 5c295c4. Consider uploading reports for the commit 5c295c4 to get more accurate results

@@           Coverage Diff           @@
##           master     #128   +/-   ##
=======================================
  Coverage   98.89%   98.89%
=======================================
  Files          12       12
  Lines        1084     1084
  Branches      138      138
=======================================
  Hits         1072     1072
  Misses          6        6
  Partials        6        6

Flag
Coverage Δ

travis
98.89% <ø> (ø)

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

Continue to review full report at Codecov.

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


ci: Add .resinci.yml

Signed-off-by: Jack Brown<EMAIL_ADDRESS>Change-Type: patch

@jhermsmeier My understanding is that we didn't need alpine support on drivelist. This .resinci.yml file will restrict what we run on.

@brownjohnf the issue was that pip was missing from the alpine builds, which makes them fail on drivelist, as it uses the cpplint python package to lint the native binding code. While testing this on Alpine might not be necessary, I'd think that might impact other repos as well

@resin-ci retest


Fix matchhighlight vim cursor when background-color is set for cm-matchhighlight

This makes the cursor z-index in vim the same as other keymaps.

Does this solve a specific problem? I can't immediately identify an issue with the patch, but the layering of the selection relative to other stuff is so subtle and fragile that I'm hesitant to make changes to it.

Sorry- I was mistaken that it affects midnight theme. If an opaque background-color style is set for cm-matchhighlight, then the cursor becomes invisible with vim keymapping.
(demo: https://stackblitz.com/edit/js-yu9slo ; look at the CSS file)
Relatedly, it also looks to me like the midnight theme has a typo, btw (it's specifying CodeMirror-matchhighlight instead of cm-matchhighlight).
I suspect we can work around this issue for our usage by making the background not opaque, but it seems like it would be a good idea to address the inconsistency of the cursor z-index values between vim and non-vim.

The matchhighlight theme was indeed using the wrong class name. Attached patch fixes that.
I would propose leaving things as they are until an actual problem is identified—I should have written more comments around this, since there's a lot going on with the layering and I don't remember the details. But as it is, I really don't want to touch it without good reason.

Fair enough; I'll drop this then; but then, shouldn't the midnight theme use a translucent background color, rather than an opaque one, to avoid running into the issue I've noted here?

(or maybe that style should just be dropped, if nobody has complained about it being missing :D )

(hoping that closing the PR doesn't prevent further responses)

Good point. Attached patch drops the styles.


feat(a11y): alternative text for Avatar component

Allow to add an alternative text to the icon or image. It is related with #1100 and #1101 and #225.

Cool! Thanks!


Change config directory

:shipit:

Comment accepted. Will treat it ASAP

Squashing and checks in progress.


Remove CLI daemon?

The CLI daemon may be useless since response time problems have been fixed. Maybe, we can try to remove this component and make the CLI store tokens in ~/.cosy.

See  #103


Add overlaps keyword

Fix https://github.com/scriptcoded/sql-highlight/issues/26

I see the checks didn't run immediately. I'll squash the commit into a semantic commit message. For future reference I'd like to keep commit messages in line with the semantic commit message format. :)
https://github.com/scriptcoded/sql-highlight/blob/master/CONTRIBUTING.md#commit-message-guidelines


Implement tests

Test some implemented functions
Points

Test comment toggle function
Test function of trimming trailing white spaces
Check dependencies to vim versions

whether or not a builtin function or a command used in scripts work in older version (e.g. vim v7.0 ~)

Implement: 99c82a0f47f5eae2b8844ae587e447c4d316d8f3

Note: implements this issue on issue-3 and ci_test

Test script for toggle comment may fail in vim v7.3 or older.
This problem have been handled in #9 .

Implements for test trim trailing spaces in #10

So far, all necessary tests have been conducted (maybe...).


[Feature Request] Add gui picker button/mapping inside floating pane

Sometimes I want to pick colors outside Neovim. I'm using https://github.com/KabbAmine/vCoolor.vim
BTW, add an option to increase the color panel, too small for now.

Can you elaborate? I created the plugin so I don't have to use outside color pickers when I'm in Neovim.
You mean you'd like the pop up window to be bigger? Like when you press t to bring up the transparency mode?

Pick a color from the external GUI when tuning color theme with Neovim.
Not popup window, but the color bg.

I'll look into it. No promises tho :)
I don't know how to make it bigger without making it ugly. Any suggestions?


External resource names should be configurable

What problem are you facing?

As a cloud administrator I would like control over the names of the external resources created by Crossplane. Crossplane does not currently offer much control or consistency in this area. For example at least the following are used to name external resources:

A prefix plus the managed resource's .metadata.UID
A prefix plus a hash of the managed resource's .metadata.UID
The managed resource's .spec.Name
The managed resource's .metadata.Name
The managed resource's .spec.NameFormat, which is passed along with the managed resource's .metadata.UID to util.ConditionalStringFormat

How could Crossplane help solve your problem?

If Crossplane moves toward high fidelity managed resources per #621 it would make sense for any external resource that typically has the caller configure its name via an API Object field have a corresponding field under the managed resource's spec. For example in the CloudSQL API the field is simply named Name, and thus .spec.Name is appropriate.  In the CloudMemoryStore API the field is named InstanceId, and thus .spec.InstanceId is appropriate.
One consideration is that managed resources are frequently dynamically provisioned to satisfy a resource claim, using a resource class. If the above .spec.InstanceId were simply read from a resource class we'd end up with two external resources attempting to use the same name. ConditionalStringFormat is one approach to address this.
I suggest that external resource names always be static and explicitly specified as part of the managed resource .spec. In cases where managed resources are dynamically provisioned it should be the responsibility of the ManagedConfigurator to set the managed resource's external resource name in a fashion that will avoid collisions.

Relates to #211 and #458.

This ticket sounds like it is related to having a common API pattern for specifying resource names. In a past conversation, the idea came up of having a reusable mechanism for setting the name, or at least for generating a default name if nothing was in the resource. I want to make a note of that idea in this ticket, since they seem related, and there isn't a ticket (yet) tracking the concept of a reusable default naming mechanism.

I suggest that external resource names always be static and explicitly specified as part of the managed resource .spec

Names are often optional fields which are fulfilled by the providers' own name generation.
As a user of IaC tools, I'm often confused when confronted with multiple name and label fields, where it is not clear how the two fields will be distinguished, especially if that convention is not upheld between resources.

Kubernetes generateName and Kustomize namePrefix and nameSuffix are prior art worth investigating.
https://github.com/eBay/Kubernetes/blob/master/docs/devel/api-conventions.md#idempotency
https://kubectl.docs.kubernetes.io/pages/reference/kustomize.html#nameprefix
Here's a little more on generateName (see the idempotency link first).
$ kubectl explain job.metadata.generateName
KIND:     Job
VERSION:  batch/v1

FIELD:    generateName <string>

DESCRIPTION:
     GenerateName is an optional prefix, used by the server, to generate a
     unique name ONLY IF the Name field has not been provided. If this field is
     used, the name returned to the client will be different than the name
     passed. This value will also be combined with a unique suffix. The provided
     value has the same validation rules as the Name field, and may be truncated
     by the length of the suffix required to make the value unique on the
     server. If this field is specified and the generated name exists, the
     server will NOT return a 409 - instead, it will either return 201 Created
     or 500 with Reason ServerTimeout indicating a unique name could not be
     found in the time allotted, and the client should retry (optionally after
     the time indicated in the Retry-After header). Applied only if Name is not
     specified. More info:
     https://git.k8s.io/community/contributors/devel/api-conventions.md#idempotency

Names are often optional fields which are fulfilled by the providers' own name generation.

We're seeing something like this for the AWS network types where the caller has no control over the resource names, which are randomly generated by the cloud provider.


Implement policy metrics

Calculate packet and session numbers of each NetworkPolicy rules.

/test-all

Codecov Report

Merging #1140 into master will increase coverage by 0.13%.
The diff coverage is 52.72%.

@@            Coverage Diff             @@
##           master    #1140      +/-   ##
==========================================
+ Coverage   44.06%   44.20%   +0.13%
==========================================
  Files         138      138
  Lines       15043    15188     +145
==========================================
+ Hits         6629     6714      +85
- Misses       7827     7883      +56
- Partials      587      591       +4

Flag
Coverage Δ

#integration-tests
31.54% <30.30%> (+0.05%)
:arrow_up:

#unit-tests
40.36% <46.40%> (+0.08%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/agent/openflow/client.go
35.60% <ø> (ø)

pkg/agent/openflow/testing/mock_openflow.go
5.71% <0.00%> (-0.13%)
:arrow_down:

pkg/agent/types/networkpolicy.go
16.66% <0.00%> (-8.34%)
:arrow_down:

pkg/ovs/openflow/ofctrl_action.go
83.79% <ø> (+1.58%)
:arrow_up:

pkg/ovs/openflow/ofctrl_builder.go
70.56% <50.00%> (-1.75%)
:arrow_down:

pkg/agent/openflow/pipeline.go
57.11% <54.09%> (+0.31%)
:arrow_up:

pkg/agent/openflow/network_policy.go
73.12% <61.42%> (-1.33%)
:arrow_down:

pkg/ovs/openflow/ofctrl_bridge.go
72.24% <0.00%> (+0.66%)
:arrow_up:

@weiqiangt @tnqn Apologies. I didn't realize that this PR was part of https://github.com/vmware-tanzu/antrea/pull/1172. I left some relevant comments in that other PR.

@weiqiangt @tnqn Apologies. I didn't realize that this PR was part of #1172. I left some relevant comments in that other PR.

Thanks for reviewing, I have updated the code according to your comment.

/test-all

/test-all

I understand that this is merged. Trying to use this patch for adding netpol info in flow records. Have a couple of questions.
What happens to ct_label if both cluster network policy rule and k8s rule are applied on to one flow? Do we support this scenario?

For now, only the ID of the last rule which takes effect will be calculated.

I understand that this is merged. Trying to use this patch for adding netpol info in flow records. Have a couple of questions.
What happens to ct_label if both cluster network policy rule and k8s rule are applied on to one flow? Do we support this scenario?

For now, only the ID of the last rule which takes effect will be calculated.

Okay. Maybe using the rest of 64bits is probably the right way I guess, right?

I understand that this is merged. Trying to use this patch for adding netpol info in flow records. Have a couple of questions.
What happens to ct_label if both cluster network policy rule and k8s rule are applied on to one flow? Do we support this scenario?

For now, only the ID of the last rule which takes effect will be calculated.

Okay. Maybe using the rest of 64bits is probably the right way I guess, right?

Please ignore the above comment. I confirmed with @abhiraut that only one rule either from cluster network policy or K8s network policy is being applied.


Update breaks panel and walllpaper settings.

I just updated my flake and I noticed that plasma-manager no longer installs the scripts to update the panel or wallpapers. As a result, it no longer applies the settings in my Flake that change the wallpaper and panel.
Is this a deliberate change? If so, what's the correct way to configure the panel and wallpaper now, as the way documented in the examples doesn't work for me anymore.

This isn't deliberate. There was a recent change which merged the panel and wallpaper application into one script. Could you share the config. It works fine on my end, but I may have missed some edge case.

Sure thing. I made a branch of my Flake. Feel free to look.
https://github.com/YaroKasear/flakes/tree/plasma-update
Specifically, files important to Plasma are:
https://github.com/YaroKasear/flakes/blob/plasma-update/modules/nixos/plasma/default.nix <-- This is for sddm, mostly. No issue here and doesn't use plasma-manager.
https://github.com/YaroKasear/flakes/blob/plasma-update/modules/home/gui/plasma/default.nix <-- This is my home-manager module for Plasma, and where the defaults are set.  I have workspace.wallpaper defaulting to a setting in the user module, but it is no longer working as of latest plasma-manager update.
https://github.com/YaroKasear/flakes/blob/plasma-update/homes/x86_64-linux/yaro%40loki/default.nix <-- My standard user, does some overriding and customization of the panel.

I don't really see anything wrong there. Are the files at
~/.local/share/plasma-manager/scripts/2_desktop_script_panels_and_wallpaper.sh
and
~/.local/share/plasma-manager/data/desktop_script_panels_and_wallpaper.js
not generated?

~/.local/share/plasma-manager is empty.
A comparison between a profile with latest plasma-manager and one on the last known good pinned below. It shows that pretty much all the plasma manager scripts get removed (Along with another user's wallpaper.):
<<< /nix/var/nix/profiles/system-1098-link
>>> /nix/var/nix/profiles/system-1099-link
Removed packages:
[R.]  #1  bri.png                                                  <none>
[R.]  #2  hm_autostartplasmamanagerautostart.desktop               <none> x2
[R.]  #3  hm_plasmamanagerdatadesktop_script_apply_panels.js       <none> x2
[R.]  #4  hm_plasmamanagerrun_all.sh                               <none> x2
[R.]  #5  hm_plasmamanagerscripts2_desktop_script_apply_panels.sh  <none> x2
[R.]  #6  hm_plasmamanagerscripts3_set_wallpaper.sh                <none> x2
Closure size: 3006 -> 2995 (25 paths added, 36 paths removed, delta -11, disk usage -47.7KiB).

That's weird, I'm having trouble trying to reproduce this. Have you tried to change some value in the panel and try to regenerate the config. Are you also on the very last version? The script should be created when any kind of wallpaper is set or a panel is present

Do you mean for me to make a change in Plasma's settings manager or in my Flake?

In one of the panels for example

No, it still just removes the scripts entirely.

I may have found the problem. If you set workspace.colorScheme = "BreezeDark" I'm assuming the scripts are generated.

Indeed, this seems to have restored the scripts. I've not yet tested if they run, but I assume they do.

Yeah I'm pretty sure that's the problem. I'll look into if I can fix this immediately.

Thanks for the help.

Would be great if you could test with flake url github:magnouvean/plasma-manager/bugfix-script-disappearing to verify that the pr linked fixes the issue. If this works I'll merge right away. From my little testing it seems to do the trick.

Hmmm, commenting out the colorscheme and using your fork seems to show the scripts being removed again.

Hmm, I'm a little stuck on this. It might take a little bit of time before I can resolve this. I'll notify again when I seemingly get it working. Until then you can at the very least use the old commit.

I'll just set the colorscheme for now and keep my eye on this issue. Let me know if you need anything more from me.

Looks like that worked. The only script not appearing is applythemes, but that's because I commented out the colorscheme to test your fix.

Great. The applythemes script indeed shouldn't appear unless any themes are set so that's all good. I'll merge right away. Thanks for notifying me about this and doing the testing, it's pretty hard to test all sorts of different combinations of configurations :)


[WIP] Docs refresh

Opening early as a draft...

Codecov Report

Merging #830 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #830      +/-   ##
==========================================
- Coverage   53.64%   53.63%   -0.02%
==========================================
  Files         284      284
  Lines       16821    16821
==========================================
- Hits         9024     9022       -2
- Misses       7092     7094       +2
  Partials      705      705

Impacted Files
Coverage Δ

lib/auto_resolver.go
89.55% <0%> (-1.5%)
:arrow_down:

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 182e12b...ee1f809. Read the comment docs.


Add Windows tests

Runs the test suite in Windows as well as Ubuntu.  Hopefully will catch Windows-specific bugs like #1183

Codecov Report
Patch and project coverage have no change.

Comparison is base (c7002e3) 38.40% compared to head (97ea9ac) 38.40%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1191   +/-   ##
=======================================
  Coverage   38.40%   38.40%
=======================================
  Files          38       38
  Lines        1940     1940
=======================================
  Hits          745      745
  Misses       1107     1107
  Partials       88       88

Flag
Coverage Δ

unittests
38.40% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


webhooks/intercom: Implement support for all events.

@timabbott: FYI, still working on this, should be able to finish in the next day or two! :)

@timabbott This is ready for review! :)

Looks great, merged, thanks @eeshangarg!


Fix readme to correct rendering problem in npmjs

The same readme file is used in npmjs to provide a description.
In npmjs, readme is rendered wrong because of errors in markdown
formatting.
Addresses issue #303

As stated in the issue, previously it was like.

I've tried this in a markdown editor but not in npmjs. Still I'm pretty sure that it'll fix the problem


Collate pip package so that it picks up from main repo.

Actually, there is a pip package but it is based out of a fork of this repo. I think it would make sense to collate this effort to the main repo.
Originally posted by @sarthakpati in https://github.com/lessw2020/Ranger-Deep-Learning-Optimizer/issues/33#issuecomment-821314754

Pinging @mpariente

I don't think it's the first time I say it, but yes, I'm fully open to make @lessw2020 the owner of the pip package


Feature/x documentation url

What do you guys think about this flow?

End-user makes request
readme-node generates logId and URL

E.g. "https://dash.readme.io/logs/[GENERATED ID]" (we would need to create a new route)

End-user goes to said URL
End-user is routed to the hub of the associated project ([HUB_URL]/logs/:id)

Thoughts @mjcuva?

I want to revisit this now!
I think this looks great, the only change I would make is that I don't think we can have dash.readme.io be the url (since we are in people's api responses it needs to be white labeled). I think maybe the best way to do this is have a configuration option to provide the docs url (this way we don't need to make a call to readme which can be slow or down). For now if no url is provided we can fallback to dash.readme.io (logs.readme.io to keep everything on the hub).
Since people will just copy and paste any code sample we send them the initial code sample for setting this up can just have their custom domain in it!

I made a change to just surface the log ID for now. We should add the full URL, but alas...
It doesn't seem to actually save the custom-made id, though. So we need to fix this!

We're consolidating our Metrics SDK repos down to help build up a unified testing framework so I'll pull this work into https://github.com/readmeio/metrics-sdks.

Update! Evidently this should now be a uuid, not ObjectId(), and I also assume it's not set at _id? So, in theory I still like this change but just wanted to ⚠ warn not to merge or anything!

Cherry picked all these commits over to https://github.com/readmeio/metrics-sdks/pull/6 and will wrap this up there.


Covert a policy to JSON representation

How can I convert the policy to a JSON representation and vice-versa.
I'm looking to dynamically author a policy and came across this documentation but I couldn't achieve this through cedar-go.
Can someone help me please?
See this snippet from cedar docs:
Example

A “standard” Cedar policy looks like the following:
permit (
    principal == User::"12UA45",
    action == Action::"view",
    resource in Folder::"abc"
) when {
    context.tls_version == "1.3"
};

When you retrieve the JSON representation of this policy, it looks like the following:
{
    "effect": "permit",
    "principal": {
        "op": "==",
        "entity": { "type": "User", "id": "12UA45" }
    },
    "action": {
        "op": "==",
        "entity": { "type": "Action", "id": "view" }
    },
    "resource": {
        "op": "in",
        "entity": { "type": "Folder", "id": "abc" }
    },
    "conditions": [
        {
            "kind": "when",
            "body": {
                "==": {
                    "left": {
                        ".": {
                            "left": {
                                "Var": "context"
                            },
                            "attr": "tls_version"
                        }
                    },
                    "right": {
                        "Literal": "1.3"
                    }
                }
            }
        }
    ]
}

They talk about two functions to convert a policy to/from json, which I couldn't figure in Go implementation

You can use the Cedar Policy::to_json() method to convert the specified policy into a JSON document.
You can also use the Policy::from_json() method to convert a JSON document into a new Cedar policy. This gives you another option for programmatically constructing or parsing your policies.

Hi, thanks for opening this issue.  We do have this on our roadmap, but don't have a timeline set.
Cheers
-Phil

This would be an excellent enhancement. I have been exploring this feature to establish basic constraints on policies using tools such as conftest, aiming to prevent users from creating policies that permit everything, for example:
permit(
  principal,
  action,
  resource
)

Is there anything I can do to assist in implementing this feature?

This would be an excellent enhancement. I have been exploring this feature to establish basic constraints on policies using tools such as conftest, aiming to prevent users from creating policies that permit everything, for example:
permit(
  principal,
  action,
  resource
)

Is there anything I can do to assist in implementing this feature?

@caiorcferreira  - I had a similar thought, perhaps can collaborate or atleast give some usecases.. I'll try to upload my repo soon to see if it helps.
@philhassey - You mind sharing the roadmap please? Been comparing policy libraries and this will help. Thanks!

The approximate roadmap is:

Add in JSON support
Add in JSON Schema support
Add in partial evaluation
Add in human readable Schema support

If you have ideas as to a Go idiomatic shape for the Go version of the AST (for programmatic building), please share those ideas, as that'll help us figuring what shape we want to use.
I don't have an approximate timeline yet.

The approximate roadmap is:

Add in JSON support
Add in JSON Schema support
Add in partial evaluation
Add in human readable Schema support

If you have ideas as to a Go idiomatic shape for the Go version of the AST (for programmatic building), please share those ideas, as that'll help us figuring what shape we want to use.
I don't have an approximate timeline yet.

Hi @philhassey,
Regarding 1, I raised a PR to add the feature of 'Convert a JSON-Policy to a cedar Text-Policy' ==> https://github.com/cedar-policy/cedar-go/pull/15
I'll add @philhassey  & @caiorcferreira as reviewers, anybody else you suggest?
Thanks!

We are considering using Cedar, and this seems like critical functionality to enable its usage.
Looking forward to this feature. Hopefully, it lands soon.

I took a quick stab at implementing the JSON-Text conversion and per Phil's recommendation, here is the fork.
I intend to drop/retire the fork when the official feature arrives.

We may skip over the current JSON representation and go right to the next version that is being rolled out:
https://github.com/cedar-policy/cedar-docs/pull/109

As of v0.2.0 we have JSON marshaling and unmarshaling.
https://github.com/cedar-policy/cedar-go/releases


popovers: "dismiss on next click" should still toggle

To complement https://github.com/twbs/bootstrap/issues/31088, I think popovers dismissed on next click outside should still toggle on click inside, which is not the case now.

Use data-trigger="focus click". That will do it.

Use data-trigger="focus click". That will do it.

It doesn't work.

@mdo Is this a bug? As per the docs mentioned Use the focus trigger to dismiss popovers on the user’s next click of a different element than the toggle element, and currently it is occurring like that only.
So currently if the focus goes from the button, popover gets dismissed.

@mdo Is this a bug? As per the docs mentioned Use the focus trigger to dismiss popovers on the user’s next click of a different element than the toggle element, and currently it is occurring like that only.
So currently if the focus goes from the button, popover gets dismissed.

Since the trigger for the popover is the 'focus', the popover can not be hidden until the focus is removed from the button. And clicking again on the button can not remove the focus from the button.
So if you click on a different element than the toggle element, the focus will be removed from the button and the popover will be hidden. It is not an issue IMO :thinking:
/CC @XhmikosR

Since the trigger for the popover is the 'focus', the popover can not be hidden until the focus is removed from the button. And clicking again on the button can not remove the focus from the button.
So if you click on a different element than the toggle element, the focus will be removed from the button and the popover will be hidden. It is not an issue IMO :thinking:
/CC @XhmikosR

Closing as a won't fix / not a bug with those last two comments. Thanks folks!

Closing as a won't fix / not a bug with those last two comments. Thanks folks!

I understand that when the trigger is "focus", then it should not toggle on click.
But when the trigger is "click" or "focus click", in my opinion for a coherent UX the toggle should toggle, no matter where is the focus, which is not the case now (with trigger click).

I understand that when the trigger is "focus", then it should not toggle on click.
But when the trigger is "click" or "focus click", in my opinion for a coherent UX the toggle should toggle, no matter where is the focus, which is not the case now (with trigger click).

Using the 'click' trigger with 'focus' trigger is not required (also should not be required). Clicking the button will set the focus and the popover will be shown.
And if you use only 'click' trigger, popover is toggled on clicking again.

Using the 'click' trigger with 'focus' trigger is not required (also should not be required). Clicking the button will set the focus and the popover will be shown.
And if you use only 'click' trigger, popover is toggled on clicking again.

Is there something I'm missing?
How can I have the popover both a) toggle on click (always) and b) dismiss on next click?
The user should be able to toggle on click (always) + dismiss on next click outside the element.

Is there something I'm missing?
How can I have the popover both a) toggle on click (always) and b) dismiss on next click?
The user should be able to toggle on click (always) + dismiss on next click outside the element.

For those looking for a solution to using both 'click' AND 'focus' together which does not work with bootstrap's JS, you can manually watch the focusout event to achieve the intuitive result of the full set of click to open, click to close AND click away to close:
const popoverTriggerList = document.querySelectorAll('[data-bs-toggle="popover"]')
        popoverTriggerList.forEach(
            popoverTriggerEl => {
                const popover = new bootstrap.Popover(popoverTriggerEl, {
                    trigger: "click",
                })
                popoverTriggerEl.addEventListener('focusout', (e) => {
                    popover.hide()
                })
            }
        )


Mpo edit wiki

Added partial support for GrCh38 for get_gene_list.
Please ignore the make_pretty_igv edits, I switched to another branch thinking the commit wouldn't be tracked.
I don't think I can directly change the wiki unless I am an official collaborator.
https://help.github.com/articles/inviting-collaborators-to-a-personal-repository/

Merged minus make_pretty_igv edits. Thanks!


Myriad 220

This PR encompasses code changes that address MYRIAD-220 and is branched off of MYRIAD-200.

@hokiegeek2 could I possibly get you to squash your commits and rebase this PR afterward?  It will make the merge much cleaner.

Sure thing!
--John
On Tue, Jul 12, 2016 at 2:53 PM, Darin<EMAIL_ADDRESS>wrote:

@hokiegeek2 https://github.com/hokiegeek2 could I possibly get you to
squash your commits and rebase this PR afterward? It will make the merge
much cleaner.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-myriad/pull/83#issuecomment-232143899,
or mute the thread
https://github.com/notifications/unsubscribe/AKSRga-R2P3I41Qxzv2tw5bGVvbl_ZiDks5qU-K9gaJpZM4JDFAB
.

Done, pull request in
On Tue, Jul 12, 2016 at 3:35 PM, John Yost<EMAIL_ADDRESS>wrote:

Sure thing!
--John
On Tue, Jul 12, 2016 at 2:53 PM, Darin<EMAIL_ADDRESS>wrote:

@hokiegeek2 https://github.com/hokiegeek2 could I possibly get you to
squash your commits and rebase this PR afterward? It will make the merge
much cleaner.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-myriad/pull/83#issuecomment-232143899,
or mute the thread
https://github.com/notifications/unsubscribe/AKSRga-R2P3I41Qxzv2tw5bGVvbl_ZiDks5qU-K9gaJpZM4JDFAB
.


Update log message to show correct image tag

@bparees @mfojtik ptal

Today a build log reads:
I0310 02:31:48.727816       1 util.go:186] Pulling image "<IP_ADDRESS>:5000/haowang/ruby-hello-world:latest" ...
I0310 02:31:48.727825       1 docker.go:308] Pulling Docker image <IP_ADDRESS>:5000/haowang/ruby-hello-world:latest ...
I0310 02:31:49.302541       1 sti.go:146] Existing image for tag haowang/ruby-hello-world-6:a9d2d308 detected for incremental build
I0310 02:31:49.302616       1 sti.go:151] Performing source build from file:///tmp/s2i-build547406922/upload/src
I0310 02:31:49.302697       1 sti.go:377] Saving build artifacts from image <IP_ADDRESS>:5000/haowang/ruby-hello-world:latest to path /tmp/s2i-build547406922/upload/artifacts
I0310 02:31:49.308702       1 sti.go:388] The assemble user is not set, defaulting to "1001" user
I0310 02:31:49.322838       1 docker.go:410] Image contains io.openshift.s2i.scripts-url set to 'image:///usr/libexec/s2i'
I0310 02:31:49.322853       1 docker.go:465] Base directory for STI scripts is '/usr/libexec/s2i'. Untarring destination is '/tmp'.
I0310 02:31:49.322861       1 docker.go:479] Running "/usr/libexec/s2i/save-artifacts" command in container ...

Instead of:
Existing image for tag haowang/ruby-hello-world-6:a9d2d308 detected for incremental build

We should get:
Existing image for tag <IP_ADDRESS>:5000/haowang/ruby-hello-world:latest detected for incremental build

That's what this change is about.

Lgtm [merge]


Source fontawesome from npm packages

Current self-hosted fontawesome is

too big and heavy
hard to maintain

so it's better to turn it to be a dependency.
See more at

https://fontawesome.com/docs/web/setup/packages
https://www.npmjs.com/package/@fortawesome/fontawesome-free

Fontawesome moved from self-hosted to the @fortawesome/fontawesome-free dependency.


[TIR] Bugfix for zero number arguments tir functions.

Thanks for contributing to TVM!   Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.

Hi @areusch ! The motivation of the PR is here: https://discuss.tvm.apache.org/t/tvm-primfunc-returning-a-constant-cannot-compile/10577/2
It is mainly because we cannot distinguish to pack/unpack in the case of num_args == num_unpacked_args == 0 with previous logic.


ucentral-event: wifi-12324 5g band is showing as unknown

ucentral-event: wifi-12324 5g band is showing as unknown

merged, Thanks !


Fix native build on Windows

Fixes native build on Windows. The fixes are related to the fact that the separator is \ on Windows:

@andriy-dmytruk Don't forget to add this to a Project, or it may get missed from the release train 🚂

@timyates Thanks for ping. I added it to 4.2.0 release. Is it right? It is first time I am adding anything in projects🤨

🤔 if it's a bugfix with no breaking changes, I guess it could go in 4.0.6?

Cannot find it there for some reason 🤨


[New Port Request] <BehaviorTree.CPP>

Library name: BehaviorTree.CPP
Library description: This C++ 14 library provides a framework to create BehaviorTrees. It was designed to be flexible, easy to use, reactive and fast.
Source repository URL: https://github.com/BehaviorTree/BehaviorTree.CPP
Project homepage (if different from the source repository):
Anything else that is useful to know when adding (such as optional features the library may have that should be included):

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.


lint: Add lint rule to warn people making changes to static/locale.

We occasionally get PRs with translations or other fixes to the files in static/locale, due to a reasonable misunderstanding of how our translation system works. It would be great to add some sort of warning that points people either to transifex or to our translation documentation.

I wish .gitattributes supported controlling git grep, but it doesn't (just checked), since that's what we really want here.  We could try adding a static/locale/README.md.  I think the next most natural thing after that would be to have a zulipbot feature to complain about diffs touching that directory.
@synicalsyntax would that be hard to do?

@timabbott Personally I think it might be better to have file diffs checked in a testing suite, but it should be possible with zulipbot; can you open an issue?

The thing which is tricky here is that we don't really have test suites that look at the Git history (as opposed to the current state of the repo).  I guess we could look at using gitlint for this if it supports inspecting the commit file list?  @tommyip do you know if that's possible?

@timabbott gitlint only checks the commit messages by default. There is a command to inspect the commit file list, git diff-tree --no-commit-id --name-only -r upstream/master...HEAD^, we could add this as a custom gitlint rule or just put it somewhere in tools/lint.


Host tunnel as background process

Hi, can someone tell is there any option to launch devtunnel host
In detached (background mode)?

Hi @Dufran, thank you for reaching out to us! We currently do not support the option to launch devtunnel host in background mode. I'm going to tag this issue as a feature request and keep it open.
We'd love to hear more from the community if this is something ya'll want. Folks, please up-vote this issue if you'd like the ability to run devtunnels as a service on your machine, so you can host tunnels as a background process.

Token expirations vary. There is some documentation on this at https://learn.microsoft.com/en-us/azure/developer/dev-tunnels/security#tunnel-access.
Unfortunately, we don't have plans to add the ability to have devtunnel to run in the background so I'll be closing this issue.
However, depending on exactly what you're looking to do, it may be possible to integrate with one of our SDKs. These can be found in the source code in this repo.


Trouble updating Agda and agda-language-server at the same time

While I am able to use
cabal install --overwrite-policy=always Agda
cabal install --overwrite-policy=always agda-language-server

when running
cabal install --overwrite-policy=always Agda agda-language-server

I get
Resolving dependencies...
Build profile: -w ghc-9.4.8 -O1
In order, the following will be built (use -v for more details):
 - agda-language-server-<IP_ADDRESS> (lib) (requires build)
 - agda-language-server-<IP_ADDRESS> (exe:als) (requires build)
Starting     agda-language-server-<IP_ADDRESS> (lib)
Building     agda-language-server-<IP_ADDRESS> (lib)

Failed to build agda-language-server-<IP_ADDRESS>.
Build log (
/home/foxy/.cache/cabal/logs/ghc-9.4.8/agda-language-server-<IP_ADDRESS>-a4c9e30592f7f6ffa836aff71ad7eceafe5da87251e3e8966c3329a7a1520b76.log
):
Configuring library for agda-language-server-<IP_ADDRESS>..
Preprocessing library for agda-language-server-<IP_ADDRESS>..
Building library for agda-language-server-<IP_ADDRESS>..
[ 1 of 25] Compiling Agda.Position    ( src/Agda/Position.hs, dist/build/Agda/Position.o, dist/build/Agda/Position.dyn_o )

src/Agda/Position.hs:21:1: error:
    Could not load module ‘Language.LSP.Types’
    It is a member of the hidden package ‘lsp-types-<IP_ADDRESS>’.
    Perhaps you need to add ‘lsp-types’ to the build-depends in your .cabal file.
    It is a member of the hidden package ‘lsp-types-<IP_ADDRESS>’.
    Perhaps you need to add ‘lsp-types’ to the build-depends in your .cabal file.
    Use -v (or `:set -v` in ghci) to see a list of the files searched for.
   |
21 | import qualified Language.LSP.Types as LSP
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[ 2 of 25] Compiling Control.Concurrent.SizedChan ( src/Control/Concurrent/SizedChan.hs, dist/build/Control/Concurrent/SizedChan.o, dist/build/Control/Concurrent/SizedChan.dyn_o )
[ 3 of 25] Compiling Paths_agda_language_server ( dist/build/autogen/Paths_agda_language_server.hs, dist/build/Paths_agda_language_server.o, dist/build/Paths_agda_language_server.dyn_o )
[ 4 of 25] Compiling Render.RichText  ( src/Render/RichText.hs, dist/build/Render/RichText.o, dist/build/Render/RichText.dyn_o )

src/Render/RichText.hs:41:1: error:
    Could not find module ‘Agda.Interaction.Options.IORefs’
    Perhaps you meant
      Agda.Interaction.Options.Base
      Agda.Interaction.Options.Base
      Agda.Interaction.Options.Base
    Use -v (or `:set -v` in ghci) to see a list of the files searched for.
   |
41 | import qualified Agda.Interaction.Options.IORefs as Agda
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

src/Render/RichText.hs:45:27: error:
    Module ‘Agda.Utils.Suffix’ does not export ‘subscriptAllowed’
   |
45 | import Agda.Utils.Suffix (subscriptAllowed, toSubscriptDigit)
   |                           ^^^^^^^^^^^^^^^^
[18 of 25] Compiling Server.CommandController ( src/Server/CommandController.hs, dist/build/Server/CommandController.o, dist/build/Server/CommandController.dyn_o )
[19 of 25] Compiling Server.ResponseController ( src/Server/ResponseController.hs, dist/build/Server/ResponseController.o, dist/build/Server/ResponseController.dyn_o )
Error: cabal: Failed to build agda-language-server-<IP_ADDRESS> (which is required
by exe:als from agda-language-server-<IP_ADDRESS>). See the build log above for
details.

Am I perhaps not using the proper cabal command/flag to upgrade packages?

I'm not sure what's your goal of upgrades. But I've managed to upgrade the lsp packages to >= 2 and now it builds with various versions of Agda (still working on 2.7.0 though).
Here's a Stack lockfile with some versions pinned down
https://github.com/agda/agda-language-server/blob/af389aa74dc1d96d5866215d912c793405a7db32/stack.yaml.lock
Not sure if this will help.

"Goal of upgrades" as in, what versions of Agda and ALS I am upgrading to?
If so, it is <IP_ADDRESS> for Agda and <IP_ADDRESS>.0 for ALS.

(still working on 2.7.0 though)

I see! Is this why cabal can successfully install the two as binaries separately, but not together?
Unrelated: even though I am installing ALS <IP_ADDRESS>.0, invoking als --version output v<IP_ADDRESS> instead. Am I doing something wrong?


i have scrpy v2.5 on samsung device android 13, SLING APP, audio, no video

i have scrpy v2.5 on samsung device android 13. i installed the sling app, and it wont show their streaming video, their audio work on my computer speakers, but doesnt show the video, any ideas, thanks
can i modify a file to show the video ?

They probably set the "secure flag" to prevent capture, so it cannot be captured (#36, #2129).

that makes sense, im new to scrpy
so sling allows non-members recording of local,
but not screaming, unless u have a sling acct..
thanks for ur help
.
 On Friday, August 2, 2024 at 01:40:43 PM MST, Romain Vimont<EMAIL_ADDRESS>wrote:

They probably set the "secure flag" to prevent capture, so it cannot be captured (#36, #2129).
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


"No such file or directory" while executing pysh command withing withPythonEnv block

Trying to use pyenv-plugin for the first time. Can't make it work with either ShiningPanda or python binary path, always get the same error:
[workspace] Running shell script
+ . /var/lib/jenkins/jobs/test-jenkins-pipeline-2/branches/feature-new-.moo0ks.ins-pipeline/workspace/.pyenv-usr-bin-python2.7/bin/activate
<EMAIL_ADDRESS>line 2: /var/lib/jenkins/jobs/test-jenkins-pipeline-2/branches/feature-new-.moo0ks.ins-pipeline/workspace/.pyenv-usr-bin-python2.7/bin/activate: No such file or directory
script returned exit code 1

I've checked workspace directory and .pyenv-usr-bin-python2.7 is not there, but /usr/bin/python2.7 definitely exists.
Here is the code that I'm running(part of it):
pipeline {
  agent any
  stages {
    stage('Install requirements') {
      steps {
        withPythonEnv('/usr/bin/python2.7'){
            pysh 'pip install pylint'
        }
      }
    }
    ...

Also, the logs that I've provided are the only logs that I'm getting. So I don't see anything like this - https://github.com/jenkinsci/pyenv-pipeline-plugin/blob/master/src/main/java/com/github/pyenvpipeline/jenkins/steps/WithPythonEnvStep.java#L169 in the output, which leads me to the conclusion that the environment hasn't been created in the first place. Any thoughts on this?
I would I appreciate any help,
Thanks

That is very odd, and I have had no success replicating the issue on my side of things.
What version of the plugin and Jenkins are you using?

Sorry, I'll close this one - I've found the issue and it wasn't fault of pyenv plugin but rather ShiningPanda and python configuration on Jenkins machine. As a result - virtualenv was not created by ShiningPanda.


Permission denied when trying to build and pulling a npm package from a private Github repo

App Id: private
Region: us-west-2
Step: Build
Status:
Issue/question
When trying to deploy a static react application, AWS amplify is failing on the build step because it cannot access a private repository (that is used for a npm package host). Is there some way to bypass this?
The Github account that is used for pulling has access to this private org.
Error message
2020-07-15T23:48:20.590Z [WARNING]: error Command failed.
                                    Exit code: 128
                                    Command: git
                                    Arguments: ls-remote --tags --heads git@************
                                    Directory: /codebuild/output/********
                                    Output:
                                    Permission denied (publickey).
                                    fatal: Could not read from remote repository.
                                    Please make sure you have the correct access rights
                                    and the repository exists.

Hi @scotttball,
Is it possible to share your appId?

Hi @scotttball,
Is it possible to share your appId?

App Id: dnv3f4j5dx5cz

Two things I wanted to check here:
(1) Is it possible to move the NPM package to a public repo?
(2) Have you looked at this: https://docs.npmjs.com/using-private-packages-in-a-ci-cd-workflow
Amplify wouldn't explicitly have access to the private NPM repo.

@ganipcanot

(1) Is it possible to move the NPM package to a public repo?
No, I don't believe so.

(2) Have you looked at this: https://docs.npmjs.com/using-private-packages-in-a-ci-cd-workflow
Yes, I have. Sadly, we can't move to private registries at this point in time.

The only path was to use private git repos for npm packages (which although I know is unorthodox), it's my companies decision to do it like this.

@ganipcanot do you want to keep this open or should I mark this a non-issue since it doesn't seem like amplify supports it?

@scotttball I'll mark it as a "feature request" for now and can discuss with the team internally if this is something we want to support in the future.

We have a similar setup in my company as well. Amplify console has read access to a private repo. Would be nice if Amplify console pulls the package from GitHub.

In case anyone else has the same problem, you can workaround it as explained here.


Can't be used as a route.data

Not sure why, something throws when route.start() is called.

Can you share your route code? I'm not seeing this issue. https://codepen.io/kphillips86/pen/OebbLV?editors=0011.

Closing since we can't reproduce. Will re-open if someone else runs into issues.