Improve wording for “What is Kubernetes”

In the section titled "Why you need Kubernetes and what it can do" the first bullet point reads:

Service discovery and load balancing Kubernetes can expose a container using the DNS name or using their own IP address. If traffic to a container is high, Kubernetes is able to load balance and distribute the network traffic so that the deployment is stable.

I'm not clear on the meaning here with regards to load balancing being a benefit with one container. I'm new to Kubernetes, but I assume that multiple containers make more sense. Would this be an accurate rewrite?

Service discovery and load balancing Kubernetes can expose a container using the DNS name or using their own IP address. If traffic to one container is high, Kubernetes is able to load balance and distribute some of the network traffic to other available containers so that the deployment is stable.

/retitle Improve working for “What is Kubernetes”
/language en
The original wording is more appropriate. Balancing, whether it's network traffic, a finance ledger, or weights on beam, is something done in aggregate.
Additionally, the thing that Kubernetes directs traffic to is actually a different endpoint - usually a Pod. It's OK to be simple but the more detail we add to this overview, it becomes more problematic not to cover the full detail.
I like having a simple summary that doesn't yet include all the detail. We could look for a different wording that implies that the one overloaded container might be part of a broader set of some kind (without mentioning the actual implementation details: ReplicaSet, Deployment, etc).

I see, "Balancing" implies an aggregate action to me too, so with regards to "a container" I wasn't sure if I was making the right assumption that "there must be other containers" to make that work. I think this would be clearer to me to identify a broader set (change in italics):

Service discovery and load balancing Kubernetes can expose a container using the DNS name or using their own IP address. If traffic to one of the containers is high, Kubernetes is able to load balance and distribute the network traffic so that the deployment is stable.

Would that wording meet your goal of keeping a simple summary that is still accurate?

/retitle Improve wording for “What is Kubernetes”
How about:
Service discovery and load balancing Kubernetes can expose a container using the DNS name or using their own IP address. When there are multiple containers backing a service, Kubernetes is able to load balance and distribute the network traffic. Load balancing helps ensure that a workload can stay available even if one container stops working, or during deployments.

Yes, that reads much clearer to me.

/triage accepted
/kind cleanup

/good-first-issue

/assign

Hey @sftim , I've reviewed the PR for this issue. Kindly look at it and feel free to suggest some changes if necessary.

I've reviewed the PR for this issue. Kindly look at it and feel free to suggest some changes if necessary.

English nit: you've opened a PR and are requesting a review.

@sftim My bad! Sorry, I mean I've made changes to the PR as required so now I'm asking  for review if any further changes are required to be made before merging.

/remove-help
We shouldn't label updates to the What Is Kubernetes page as good first issues, unless the fixes are genuinely trivial. Updates to that page always require extra review.

I think we can close this issue. WDYT @sftim ?

As @sftim commented on Apr 13, does this change when merged? I'm not familiar with your workflow on when https://kubernetes.io/docs/concepts/overview/ will change.

/remove-lifecycle stale
BTW, once someone merges a fix for this issue, we can close the issue. I think there is room to improve the wording in https://kubernetes.io/docs/concepts/overview/

@sftim As far as the last comment by @reylejano (https://github.com/kubernetes/website/pull/34037#issuecomment-1226797408) , Do you think that any change is required?

I still think we should make an improvement (see my previous comment). It needs to be a change that reviewers agree is a benefit to the Kubernetes docs.


Set default log level to WARNING

Fixes:

An issue, where the log format and correct logging level weren't loaded

Improvements:

Resolves #27
More expressive logging format
Formatting in finch script

Dask logging must be configured in the dask configuration. This is tracked by #39.


How do we want to access processed data from simulation trajectories?

We need a way to store processed data (for now rdf's, densities, etc.) and keep that in this repo for further analysis. This needs to make sure that the trajectory information can be easily accessed for each simulation engine, but to make sure that none of the trajectory information itself is pushed to this repository due to data storage concerns.
My first thought was to have the workspace folder be saved to the git repo, but keep all of the trajectory information in the .gitignore so only the processed files would be tracked by git. We could also just have an entire separate directory to copy the information over and be accessible to the analysis routines. That could also be separated by job or by simulation engine, or even by molecule. Anyone have any suggestions?

related to #28

My first thought was to have the workspace folder be saved to the git repo, but keep all of the trajectory information in the .gitignore so only the processed files would be tracked by git.

This seems reasonable to me. The hashed job folder names will be the same across systems (as long as we don't modify the statepoints).

We could also just have an entire separate directory to copy the information over and be accessible to the analysis routines. That could also be separated by job or by simulation engine, or even by molecule. Anyone have any suggestions?

This also seems worable--I would recommend we use something like a bash with signac cli to manage this folder because manually copying could be annoying.

Just to continue this discussion, it looks like all of our workspace trajectory information will be stored via Michigan's dropbox account information (thanks to @kjcoulter). So at least we can all also run the analysis scripts and save output to the workspace folders that you are working with. That at least gives us a starting place to have access to this information.

Closing this issue, seems that the solution is through a combination of rclone and dropbox, it is really easy to store this information remotely.


Exception: EMFILE: too many open files, open 'd:\Desktop\real project\smartyt\package.json'

Code snippet triggering the exception
// INSERT CODE HERE

All Logs
(Please open the output channel for "JS/TS Import/Export Sorter" and copy/paste ALL logs here)
PASTE LOGS HERE

@Mdkamrujjamanshimu
Thanks for the feedback!
Could you provide more info about the issue, e.g. screenshots, demo project, logs, environment?


[Feature][connector][kafka] Supplement the test case in json format

Purpose of this pull request
close #3872

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

cc @Hisoka-X  @hailin0

The test case not passed, please fix it. Thanks.

@Hisoka-X Thanks for your review,I have made the changes.


deeplearning models integerated

can you give some examples to show how to integerate this rank with deeplearning models serving ?

Actually, deep learning models are not supported in gorse.  The challenging things are:

How to fit deep learning models in master node? (It seems that the master node has to call fit procedures written in Python.)
How to use deep learning models in server node? (TVM might be a good option.)


Change move commands to copy commands

Change move command to copy for all files except images and darks when upgrading

@Alex-developer Good idea to cp instead of mv.
Thanks for changing.


Rerender feedstock

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Trying to debug build issues across #8 and #9

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@jeromekelleher, Can you up it to bed-reader 2.7?

The idea was to try out the build to update the external infrastructure but to keep the bedreader version constant. If we update to version 2.7 we'll end up with the same as #8 I think.
Hmm, so this is failing for different reasons now...

[If I’m remembering correctly] that error is the same as the previous Python 3.10/Bed-reader 2.5 error.
The move from Bed-reader 2.5 to 2.7 changed things related to the Rust NDArray library (the source of that error)
From: Jerome @.>
Sent: Monday, November 15, 2021 11:00 AM
To: @.>
Cc: Carl @.>; Review @.>
Subject: Re: [conda-forge/bed-reader-feedstock] Rerender feedstock (PR #10)
The idea was to try out the build to update the external infrastructure but to keep the bedreader version constant. If we update to version 2.7 we'll end up with the same as #8https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fconda-forge%2Fbed-reader-feedstock%2Fpull%2F8&data=04|01||c5da22c8d449475348b008d9a86a1f1c|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637725996085797609|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=geDsM0%2BTAUG1Mputrn0DTRqaANflAZm33xCYec4LqN4%3D&reserved=0 I think.
Hmm, so this is failing for different reasons now...
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fconda-forge%2Fbed-reader-feedstock%2Fpull%2F10%23issuecomment-969219614&data=04|01||c5da22c8d449475348b008d9a86a1f1c|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637725996085807570|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=qV2ZJtrcqmM9W98llBrG00SGCMUgiPRRZjL2b9c%2Fywk%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABR65P7FLSDHTHC4KA47TV3UMFKC5ANCNFSM5ICJ4KCQ&data=04|01||c5da22c8d449475348b008d9a86a1f1c|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637725996085817524|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=ih3VmT1p9xDFvjFIOZm2SM5Qbw0oZotQHCt0%2BCJkJmU%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01||c5da22c8d449475348b008d9a86a1f1c|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637725996085817524|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=df3PZ6v7AuW9Rc0BCUSM2r%2BPHwnqdUd9fEl7hZI7n1A%3D&reserved=0 or Androidhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01||c5da22c8d449475348b008d9a86a1f1c|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637725996085827477|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=cnY1QmO1af9i3he6l5ZYx7TBO4CU6HqHCoUkliTZNbg%3D&reserved=0.

I see - looks like it's better to go back to #8 then, where it's just windows failing. closing this PR


coerceTypes coerces nulls

What version of Ajv are you using? Does the issue happen if you use the latest version?
Latest
Test case
const ajv = require('ajv');

const v = new ajv({ removeAdditional: true, coerceTypes: true });

v.addMetaSchema(require('ajv/lib/refs/json-schema-draft-06.json'));

v.addSchema({
    "$id": "test-schema.json",
    "$schema": "http://json-schema.org/draft-06/schema#",
    "definitions": {
        "test::Test": {
            "type": "object",
            "properties": {
                "id": {"type": "number"},
                "name": {"type": "string"},
                "num": {"type": "number"}
            },
            "required": ["id"]
        }
    }
});

const t = v.compile({ '$ref': 'test-schema.json#/definitions/test::Test' });

const obj = { id: "1", name: null, num: null };

t(obj);

console.log(obj);

Validation result, data AFTER validation, error messages
{ id: 1, name: '', num: 0 }

What results did you expect?
{ id: 1 } or { id: 1, name: null, num: null }

Are you going to resolve the issue?
Maybe

It happens even if I add null option explicitly, like "name": {"anyOf": [{"type": "string"}, {"enum": [null]}]},

I think the behaviour should depend on whether the field is optional. For optional fields it's OK to just remove nulls, for required fields it's probably more correct to coerce them

This works as documented - see the docs. To avoid coercing null, you should either add “null” type (e.g. "type": ["string", "null"]) or use nullable.

This works as documented - see the docs. To avoid coercing null, you should either add “null” type (e.g. "type": ["string", "null"]) or use nullable.

Sorry, but what's "nullable"?
Also another funny thing is that if I modify the above example and change
"name": {"type": "string"},
to
"name": { "oneOf": [ {"type": "string"}, {"type": "null"} ] },
The result becomes
{ id: 1, name: null, num: null }
Not just
{ id: 1, name: null, num: 0 }
:)

Even more: if, in the same example, I change obj to:
const obj = { id: "1", name: null, num: "2zzz" };
It becomes
{ id: 1, name: null, num: "2zzz" }
After the validation. This is definitely a bug, even though not the originally described one :)

I'll re-post this in a separate issue.
About the original problem: don't you think it would be convenient to automatically handle optional fields as nullable, at least under the condition of coerceTypes enabled? Having to add oneOf:[{type: null}] everywhere pollutes the schema...

That would be against the spec.

Is there a spec on type coercion at all?

“to automatically handle optional fields as nullable” would be against the spec. Coercion is not part of the spec.

Ok I mean automatically handle optional fields as nullable, that is make null equivalent to a missing field, when coerceTypes is enabled

Allowing "null" and coercing are two separate things, they should not be mixed together I believe.


Add context menu action to delete all occurrences of a tag/meta

Summary
To make the removal of tags/metas easier, introduce a context menu action on Tag nodes in the tool window. It would remove all occurrences of it from all associated .feature and .story files.

Completed in v1.2.0


UI - Add a Rules counter on the sidebar

On the sidebar, please add a rules counter:

I see the HTML as:

212121

SSW Rules

Than a little CSS:
.rules-counter{text-align:center;}
.rules-counter h2{margin-bottom:0;}

AB#57151

@tiagov8
Done on staging. Any feedback?

Figure: Rule counter has been added

@adamcogan , @christianmorfordwaitessw
As per our conversation,

Format number x,xxx

To: @tiagov8
CC: @adamcogan
Done

Figure: Rule counter has been added

Hi @SebastienBoissiere
Reopening - As per our conversation with @adamcogan ,

Change the formating of the red number to x,xxx

@adamcogan

Change the formating of the red number to x,xxx

Done - See https://ssw.com.au/rules/

Figure: Formatting of the red number now has a comma


Edit tests to check for correct results

Implement a compare functionality to check the exportet results in a CSV file with stored ore computed values.

Maybe use https://github.com/modelica-tools/csv-compare to compare results.

First version of result verification is added with 4e23fe8e0eb47bd7504586910033f7e8cda3db83


Add how to configure IntelliJ IDEA to run RSpec

As I'm fairly new to running IDEA I'm not sure if this is configured correctly, but with this setup I can run the specs inside of IDEA and it reports things correctly.

I'm curious to understand how you figured that out!

@ketan Just trial and error, together with reading the source for the run rspec rake file :stuck_out_tongue_closed_eyes:
I've been using some other Jetbrains IDEs in the past, and I recently switched to IDEA so I could have an easier time with multi-language projects. So I've configured these parts for other projects in the past, so with a rough understanding of what it ought to look like I clicked around until it worked. :smile:


Plugin file format

The .drawio format is natively recognized to be opened by drawio APP by Windows and macOS, it would save users some effort to manually change the name if users want to open the file by drawio APP.
Is there a way to change from .svg to .drawio for the files created by the plugin?
If not, could you please consider to add it to the roadmap?
Thank you.

Appreciate your effort in making such a good and useful plug-in.


Long-term temperature evolution

Reference

Get all six data sets

[x] ERA5: use CDS
[x] GISTEMPv4: use CDS
[x] NOAA GlobalTempv5: link to gridded netCDF
[ ] JRA-55: needs registration (?)
[x] HadCRUT5: link to gridded netCDF
[x] Berkeley Earth: use CDS

Comparison figure

[ ] compare temperature for four regions (global, global-land, Europe, Arctic)

Issues

[ ] cannot reproduce ERA5 mean temperature estimates for reference period (1981-2010); perhaps due to lower resolution?


explicitly say how to migrate to nginx

This PR should clarify that application_server must be set to wsgi-tcp. I imaging if people migrate from apache to nginx this may be missing and cause more community questions about:
    Configuration mismatch omero.web.application_server=wsgi cannot be used with omero web config nginx.

to test:

set omero.web.application_server=wsgi try starting omeroweb. at this point it should fail with misconfiguration error.
set omero.web.application_server=wsgi-tcp try starting omeroweb. started
unset omero.web.application_server try starting omeroweb. started

Built on staging now - https://www.openmicroscopy.org/site/support/omero5.3-staging/sysadmins/server-upgrade.html#omero-web-migrating-from-apache-to-nginx, https://www.openmicroscopy.org/site/support/omero5.3-staging/sysadmins/troubleshooting.html#troubleshooting-omeroweb-migrate-to-nginx etc - looks fine but will list for technical review tomorrow before merging and rebasing to dev_5_2.

using 5.3.0-m5 server on ubuntu 14.04

run OMERO.server/bin/omero config set omero.web.application_server wsgi

if we try to start web w/o any other configuration, it falls back to "apache" and we have the following message
609 static files copied to '/home/omero/OMERO.server/lib/python/omeroweb/static', 2 post-processed. Clearing expired sessions. This may take some time... [OK] [ERROR] You are deploying OMERO.web using Apache and mod_wsgi. OMERO.web does not provide any management for the daemon process which communicates with Apache child processes using UNIX sockets to handle a request. Generate apache config using 'omero web config apache' or 'omero web config apache24' and reload web server.

if the following command is run:
OMERO.server/bin/omero web config nginx --http 80 > OMERO.server/nginx.conf.tmp
The expected error message is displayed
configuration mismatch. omero.web.application_server=wsgi cannot be used with 'omero web config nginx'.

Change wsi to wsi-tcp. Web restart w/o error

unset   omero.web.application_server and restarted web

Possible rebase candidate for 5.2.7 release

@aleksandra-tarkowska: 2 minor changes then we can rebase it and I will start a new 5.2 build

Maybe also look at https://github.com/openmicroscopy/ome-documentation/pull/1579#discussion_r90186628 ?  Though if it reads okay to @hflynn then I'm fine with it.

Looks good thanks @aleksandra-tarkowska
Could you rebase it to 5.2?

--rebased-to #1588

Merging as this is now on the 5.2 branch anyway. We can always do some minor wording fixes before the 5.3 release.


Use parameter for retriever

Description
Use parameter for retriever
Issues
n/a.
Type of change

[x] Breaking change (fix or feature that would break existing design and interface)

Tests
Pass

Please fix conflict and add low-code configuration for data ingestion


Python interactive session not displaying output after VSCode insiders update

Does this issue occur when all extensions are disabled?: Yes (only the jupyter and python extensions enabled)

Version: 1.94.0-insider (user setup)
Commit: e34ecab75b0c379c17921f759a77eebaf24bebf2
Date: 2024-09-16T05:03:40.122Z
Electron: 30.4.0
ElectronBuildId: 10073054
Chromium: 124.0.6367.243
Node.js: 20.15.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19045
Steps to Reproduce:
I restarted VSCode to install the latest update (above), and now python interactive sessions do not print output of cells in the interactive window.
After running a cell in a interactive session:
09:35:59.913 [info] Starting interactive window for resource '~\path\to\script.py' with controller '.jvsc74a57bd0150234034aa1892935e2a97addface6e4e0ed9c60975f6380d8abb8f4340d5b9.c:\Users\~\miniconda3\envs\env\python.exe.c:\Users\~\miniconda3\envs\env\python.exe.-m#ipykernel_launcher (Interactive)'
09:36:00.046 [info] Starting Kernel (Python Path: ~\miniconda3\envs\env\python.exe, Conda, 3.11.9) for 'Interactive-1.interactive' (disableUI=false)
09:36:01.340 [info] Process Execution: ~\miniconda3\envs\env\python.exe -c "import ipykernel; print(ipykernel.__version__); print("5dc3a68c-e34e-4080-9c3e-2a532b2ccb4d"); print(ipykernel.__file__)"
09:36:01.347 [info] Process Execution: ~\miniconda3\envs\env\python.exe -m ipykernel_launcher --f=c:\Users\~\AppData\Roaming\jupyter\runtime\kernel-v38a4de0c2082783f5d3350a60bdaaa3b70f6792ca.json
    > cwd: ~\path\to\wordir
09:36:02.432 [info] Kernel successfully started
09:40:02.970 [info] Generated code for 1 = <ipython-input-1-2c1606276f95> with 8 lines
09:40:08.786 [info] Generated code for 2 = <ipython-input-2-8f12bea4ca2e> with 13 lines
09:40:09.504 [info] Generated code for 3 = <ipython-input-3-afe48573e573> with 40 lines
09:40:10.369 [info] Generated code for 4 = <ipython-input-4-1ec5be4f1b82> with 45 lines
09:40:11.830 [info] Generated code for 5 = <ipython-input-5-93fe69f2fad2> with 12 lines

Using the jupyter extension, one can see that the variables are created in memory, but there is no output in the interactive window.
Tested with a jupyter notebook, output is correctly printed below the cells:

Output of "window" after running a single cell in an interactive session:

Show output (large)
```
2024-09-16 09:42:55.860 [error] Cannot read properties of undefined (reading 'id'): TypeError: Cannot read properties of undefined (reading 'id')
    at dDi (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:332:109820)
    at K3.createActionGroups (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:332:107533)
    at Tte.getActions (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:332:109567)
    at er (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:332:97875)
    at FHe.getCellToolbarActions (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1592:26243)
    at c (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1592:25778)
    at new FHe (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1592:25848)
    at Wot.o (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1248:1586)
    at Wot.createInstance (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1248:1083)
    at Gde.renderTemplate (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1592:50278)
    at vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:244:2068
    at Array.map ()
    at Gxi.renderTemplate (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:244:2059)
    at Pxi.alloc (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:201:5753)
    at ers.Z (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:201:16825)
    at ers.U (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:201:14362)
    at ers.splice (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:201:12904)
    at vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:199:45014
    at Array.forEach ()
    at gxi.splice (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:199:45001)
    at vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:244:7637
    at S9.bufferEvents (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:3811)
    at LHe.splice (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:244:7613)
    at LHe.splice2 (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1397:75721)
    at vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1397:74491
    at Array.forEach ()
    at LHe.lb (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1397:74268)
    at gQ.value (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1397:73687)
    at x.B (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:733)
    at x.C (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:803)
    at x.fire (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:1020)
    at d (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1592:64308)
    at gQ.value (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1592:64889)
    at x.B (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:733)
    at x.C (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:803)
    at x.fire (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:1020)
    at gQ.value (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1254:5154)
    at Xss.B (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:733)
    at Xss.fire (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:951)
    at Xss.resume (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:1985)
    at zWe.applyEdits (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1254:8590)
    at WRe.apply (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:424:6709)
    at async Wze.p (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1980:18215)
    at async Wze.perform (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1980:17673)
    at async Hze.apply (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1980:19301)
2024-09-16 09:42:55.885 [error] Cannot read properties of null (reading 'domNode'): TypeError: Cannot read properties of null (reading 'domNode')
    at ers.bb (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:201:18139)
    at ers.Y (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:201:16435)
    at ers.Y (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1397:65828)
    at ers.updateElementHeight (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:201:12664)
    at LHe.updateElementHeight2 (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1397:84607)
    at D1e.o [as d] (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1738:21565)
    at D1e.execute (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:30:7102)
    at n (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:30:7345)
    at vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:30:7516
2024-09-16 09:43:54.457 [error] Cannot read properties of undefined (reading 'id'): TypeError: Cannot read properties of undefined (reading 'id')
    at dDi (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:332:109820)
    at K3.createActionGroups (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:332:107533)
    at Tte.getActions (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:332:109567)
    at er (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:332:97875)
    at FHe.getCellToolbarActions (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1592:26243)
    at c (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1592:25778)
    at new FHe (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1592:25848)
    at Wot.o (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1248:1586)
    at Wot.createInstance (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1248:1083)
    at Gde.renderTemplate (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1592:50278)
    at vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:244:2068
    at Array.map ()
    at Gxi.renderTemplate (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:244:2059)
    at Pxi.alloc (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:201:5753)
    at ers.Z (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:201:16825)
    at vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:201:16585
    at Pxi.transact (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:201:5957)
    at ers.Y (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:201:16458)
    at ers.Y (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1397:65828)
    at ers.ib (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:201:21456)
    at x.B (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:733)
    at x.C (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:803)
    at x.fire (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:1020)
    at gQ.value (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:69:33191)
    at x.B (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:733)
    at x.fire (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:951)
    at ZN.n (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:69:30421)
    at ZN.setScrollDimensions (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:69:29159)
    at X7.setScrollDimensions (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:69:34571)
    at ers.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:201:16081)
    at LHe.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:244:8769)
    at LHe.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1411:170)
    at Qde.Ic (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1738:14976)
    at Qde.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1738:14634)
    at oje.Lb (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1883:6186)
    at oje.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:1883:5979)
    at vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:560:101597
    at W4e.X (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:560:101713)
    at W4e.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:560:101583)
    at VR.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:672:31597)
    at ju.y (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:560:80824)
    at ju.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:560:80672)
    at zTi.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:335:187757)
    at Tp.U (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:335:199756)
    at Tp.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:335:193328)
    at XY.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:560:74283)
    at mli.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:560:82962)
    at YY.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:560:92450)
    at YY.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:560:96786)
    at _Ji.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:672:44281)
    at WJi.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:672:40473)
    at n5e.vc (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:672:57184)
    at n5e.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:672:57121)
    at ju.y (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:560:80824)
    at ju.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:560:80672)
    at _Ti.layout (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:335:187757)
    at Tp.U (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:335:199756)
    at Tp.M (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:335:195124)
    at x.B (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:733)
    at x.fire (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:951)
    at gQ.value (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:23:6239)
    at x.B (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:733)
    at x.fire (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:951)
    at gQ.f [as value] (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:332:37189)
    at x.B (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:733)
    at x.fire (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:28:951)
    at s (vscode-file://vscode-app/c:/Users/username/AppData/Local/Programs/Microsoft%20VS%20Code%20Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:69:50533)
```

As clarification: nothing is shown in the interactive window, not just output
"input" cells are not shown either

likely a regression from https://github.com/microsoft/vscode/pull/228577/files

Same thing happened to me.
Though the jupyter panel shows the variables created by running the code, the interactive window shows nothing. (see image below)

actually caused by https://github.com/microsoft/vscode/commit/fa28177248a9fa34a16ef44b1bc865f43773cabf, but only because we are creating a menu with an undefined ID by overriding a type restriction here: https://insiders.vscode.dev/github/microsoft/vscode/blob/main/src/vs/workbench/contrib/notebook/browser/view/cellParts/codeCellRunToolbar.ts#L44


best way to implement blocking-with-timeout read

Hi Vinnie,
I am wondering - what is the best way to implement a blocking websocket read (say) that returns either after something is received or a given timeout period elapses? Is this something that should go into Beast, or something that should be written on top of it?

For asynchronous operations, there are many ways to implement a timer - each with its own advantages and disadvantages. For this reason, it makes more sense to push the responsibility for providing timeout facilities on to users of Beast. Asio makes a similar choice - none of its interfaces allow for timeouts, but you can build a timeout facility yourself.
For synchronous operations, I don't know that its possible to implement a timeout feature. How would it be done? ip::tcp::socket::cancel is not thread safe.

Is this resolved?

Looks like this is no longer an issue. If you feel it is still relevant, please re-open it.


Demos don't work on ui-grid.info / stable / tutorial

Hi,
If you go on http://ui-grid.info/docs/#/tutorial/ and select "stable" (and change again to tutorial since it brings you back to the api).
There's many errors and all the demos grids are not working :
Error: [ngRepeat:dupes] Duplicates in a repeater are not allowed. Use 'track by' expression to specify unique keys. Repeater: page in module.directives track by page.url, Duplicate key: #/api/ui.grid.directive:uiGridColumnMenu, Duplicate value: {"section":"api","id":"ui.grid.directive:uiGridColumnMenu","shortName":"uiGridColumnMenu","type":"directive","keywords":"api column directive framework grid leverages menu provides the ui uigridmenu underneath","partialUrl":"partials/api/ui.grid.directive.uiGridColumnMenu.html","url":"#/api/ui.grid.directive:uiGridColumnMenu"}
Same behaviour in Chrome, Firefox, IE.
Thank you for your help.

There is no stable release yet, UI-Grid is still in beta. The "stable" links will work once a stable release is complete.
Sorry for the confusion!


Documentation doesn't mention backend names

I found it hard to figure out how to get the correct URL for the LinkedIn OAuth2 backend. I couldn't find it mentioned anywhere in the documentation but found my answer on this Stackoverflow question http://stackoverflow.com/questions/25732954/python-social-auth-backend-not-found-for-linkedin-oauth2. A comment also mentions you can find the correct name by looking at the backend class's name attribute but I find that not to be the best user experience for developers using this package.
Would you accept a pull request with added examples for how to generate the URL for the different authentication backends?

Yes, I would accept such PR, the documentation location is now at social-docs.
Thanks.


Add variables to the layout in example

Don't use @variable in templates. Raxx.View lets you choose your own variable.
Add a page variable for the layout.
I think a simplish trick with bindings will allows us to have a default value in that page variable.
page = Keyword.get(bindings, :page, :default)

https://github.com/CrowdHailer/raxx/pull/180
Raxx.View directly supports optional arguments now


⚠ AsianCinema has degraded performance

In 244f4cc, AsianCinema ($ACM) experienced degraded performance:

HTTP code: 200
Response time: 2791 ms

Resolved: AsianCinema performance has improved in 61aa4bd after 35 minutes.


Upgrade notification still happening after upgrade

Describe the bug
We updated from 1.31.1 to 1.32.0, but our notifications are still telling us to upgrade. This is running in EKS, and I've verified from the running pod that it is in fact version 1.32.0 that starts up, but it's still telling us to upgrade from 1.31.1.

Hi @V3XATI0N,
I cannot reproduce this. I did not experience this on any of my vms and running the code I checked that the module is able to correctly identify that the version running doesn't require update.
Are you running the binary?

thanks, yeah we are just running the published docker image nttgin/bgpalerter:v1.32.0 , and when I run the same container locally or log in via kubectl it says it's the right version. it may be that we have a ghost container still running somewhere, so maybe this bug doesn't belong here

I'm going to close this then. If you find something let me know.


[BUG] User Milestones card alignment fix (good first issue)

Description
The icon and the milestone description aren't aligned and looks odd
Screenshots
how it is

It should be

Additional information
No response

I can help work on this if nobody is working on it

@Panquesito7 what do you think about this?


🔀 Remove expand box click feedback

Add pointer-events: none css to details tag

@Futinx @billforfuture seems current deployed version, the click feedback still exist

@Futinx @billforfuture seems current deployed version, the click feedback still exist

You have to merge this branch first. I have fixed click feedback in this pr.


WIP: sysinfo: report available hugepages per-NUMA

We will depend on ghw (once it's ready) to report the
NUMA-local memory, if available. To count hugepages,
we need to implement some logic atm.
Signed-off-by: Francesco Romani<EMAIL_ADDRESS>

superceded by U/S change.


Fix wrong CPU count after CPU hot-plugging

- What I did
This fix tries to address issues raised in #23768 where the CPU count is not updated after cpu ho-plugging.
- How I did it
This fix follows the suggestion from #23768 and replace go's runtime.NumCPU() so that correct CPU count could be obtained even after CPU hot-plugging.
- How to verify it
This fix is tested manually, as is suggested in #23768.
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)
This fix fixes #23768.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

Replace the +build by:
// +build !android,linux
// +build cgo

… because Android uses bionic and not the glibc.

A version without CGO:
import (
        "syscall"
        "unsafe"

        "golang.org/x/sys/unix"
)

func popcnt(x uint64) (n byte) {
        x -= (x >> 1) & 0x5555555555555555
        x = (x>>2)&0x3333333333333333 + x&0x3333333333333333
        x += x >> 4
        x &= 0x0f0f0f0f0f0f0f0f
        x *= 0x0101010101010101
        return byte(x >> 56)
}

// AvailableCPU queries the system for the count of threads available
// for use to this process.
//
// Issues two syscalls.
// Returns 0 on errors. Use |runtime.NumCPU| in that case.
func AvailableCPU() uint {
        // Gets the affinity mask for a process. The very one invoking this function.
        pid, _, _ := syscall.RawSyscall(unix.SYS_GETPID, 0, 0, 0)

        var mask [1024 / 64]uintptr
        _, _, err := syscall.RawSyscall(unix.SYS_SCHED_GETAFFINITY, pid, uintptr(len(mask)*8), uintptr(unsafe.Pointer(&mask[0])))
        if err != 0 {
                return 0
        }

        // For every available thread a bit is set in the mask.
        var ncpu uint = 0
        for _, e := range mask {
                if e == 0 {
                        continue
                }
                ncpu += uint(popcnt(uint64(e)))
        }
        return ncpu
}

Thanks @wmark I tried the implementation locally and it works as expected. The PR has been updated.
I also realized that on Windows the same issue may apply as well. I will spend sometime to see if I could get Windows implementation with GetProcessAffinityMask.

@wmark The NumCPU on Windows has been implemented and the PR has been updated. Please let me know if there are any issues.

Thanks, @yongtang! In a nutshell: LGTM
You've vendored-in x/sys/unix, hence the many changed lines. I've only glanced over the first four changed files.
The syscalls work for all posix, not only linux. That is, I cannot test it but expect that it will work on some *BSD variants and Solaris. But yes, this time Android is covered. Other than that I am no neutral reviewer of the code anymore.
There's a subtle difference between your old approach (which has been fine!) and the new one, querying affinity masks:

An operator might have pinned Docker to a disjunct set of CPUs than for the containers.
So the affinity mask does not reflect the count of available »CPUs«.
An affinity mask that had been reduced (does not span all CPUs anymore) does not get expanded after plugging in more CPUs. The count of online CPUs on the other hand will be increased. (Someone messing with the mask will already be aware of that limitation, though.)

Neither of the approaches reliably tells how many »CPUs« we can expect (affinity mask could reduce online ones). But both approaches are more accurate and suitable than just runtime.NumCPU.

LGTM

ping @jhowardmsft PTAL for the windows part

LGTM for the Windows change

all green, so merging, thanks all!


Conversions

Includes

A Conversions module which consists of six functions of type X -> Y which convert each of the three types Maybe, Either and Validation to all other types. The module is private (it is not exported in data). The functions for converting a Maybe to an Either or Validation accept as a second parameter the error value which is to be put in the Either/Validation in case the first parameter is Nothing.

Aliases to all of the module's functions as part of the type definitions (so Either.fromValidation is an alias to validationToEither).

Methods for converting an instance to another one which call the above function (so Left(1).toEither() is leftToEither(Left(1))). This introduces a circular dependency which is resolved by requiring the helper functions after the type definition is exported in the core.js files for all datatypes.

Ah, that's neat :+1:
Instead of putting the requires at the bottom and relying on Node's caching (which I'm not too fond of), we can just move those requires inside of the function. Like:
Maybe.toEither = function(...args) {
  return require('folktale/data/conversions/maybe-to-either')(this, ...args);
};

Since these are not resolved eagerly, mutual recursion just works. Caching solves the loading problem if people call it many times.

Done.

LGTM. Could you move fromNullable to data/conversions and add one for Either as well?

Good idea.

Thanks :)
I suppose this has to wait Validation to be merged now.

Could you rebase this on master?

oops I think I messed it up.
I will pull from master and try to rebase again.
sorry.

I redid the changes in another branch:
https://github.com/origamitower/folktale/tree/conversions-redone

Thanks :)


Upgrade Faraday to 2.X

Hi! After installing this gem I've noticed 2 depreaction warnings when running the app server or rspec.
Top level ::CompositeIO is deprecated, require 'multipart/post' and use `Multipart::Post::CompositeReadIO` instead!
Top level ::Parts is deprecated, require 'multipart/post' and use `Multipart::Post::Parts` instead!

After some research I've reached the conclusion that this is due to the faraday gem that is being forced to stay at 1.X due to the faraday_middleware dependency. The later is currently deprecated and should not be used as stated in the project README.
Would be nice to replace this dependency with the recommentded standalone replacement

By updating the faraday gem with bundle update faraday it updated to 1.10.3 and this version already fixes the warnings, nonetheless it would be cool to upgrade to faraday 2 if you guys agree. I'm willing to submit a PR with that.

Hi @j-monteiro, thanks for raising this! Also had this on my list but haven't started working on the upgrade.  If you would like to submit a PR for this, that would be very welcome.

It's now released as version 1.0.3. Thanks again for the fix!


Save animated gif

How to save frames of animated gif for html/js output?

The only saving functionality available for ascii art gifs is in the form of gifs with --save-gif flag. You will have to split the gif into its frames manually before feeding them to ascii-image-converter if you want to have each frame converted separately.
Doesn't look like an issue with the project, so I'm closing this. Please discuss such matters in the Discussions tab.


Doesn't see any related changes, but embed player loose playlist

Hello @vintikzzz !
I am using next code to integrate with website, but now player load first file, and don't give a ability to select other.
I am look over last changes, but don't see options/settings for this in player js sdk.
<script>
        window.webtor = window.webtor || [];
        window.webtor.push({
            id: 'player',
            magnet: 'magnet:?xt=u',
            on: function(e) {
                if (e.name == window.webtor.TORRENT_FETCHED) {
                    console.log('Torrent fetched!')
                }
                if (e.name == window.webtor.TORRENT_ERROR) {
                    console.log('Torrent error!')
                }
            },
        });
    </script>

It is possible to select specific file by providing pwd and file parameter.
Also it is possbile to use player API to select another file after player initialization.
You can check out full example here

Oh ok, thanks.


Documentation improvement

I think it would be a help to new user if the documentation for the plugin at https://docs.jboss.org/wildfly/plugins/maven/latest/index.html included examples where the server was hosted on another machine other than the localhost or provided a list of the properties that can be set.
I know it took me several hours of research to find what settings where required to deploy my application using the plugin to a wildfly instance on another machine on the network. By default the plugin will always look for an instance on <IP_ADDRESS>:9990.
Graham

The documentation on the goals is pretty clear what the defaults are. Have a look at hostname property and the port property.
Of course contributions on documentation are always welcome :)


SOLR-16996: remove no-longer-applicable solr/licenses/argparse4j-* files

#2602 follow-up
https://issues.apache.org/jira/browse/SOLR-16996

I wonder if this is important enough to bring to branch_9_7?

I wonder if this is important enough to bring to branch_9_7?

I think so, yes, i.e. back porting to both branch_9x and branch_9_7 IMO. Otherwise we'll unnecessarily include/ship license files that are no longer needed in the 9.7 release.


CheerioCrawler - requestTimeoutSecs should also override httpRequest timeoutSecs

I think there is a bug that if you set longer requestTimeoutSecs (which only races the request) the underlying request timeout will fire first.
E.g.
requestTimeoutSecs is set to 90 but you get this timeout error
2019-10-04T13:59:39.833Z   TimeoutError: Timeout awaiting 'request' for 30000ms
2019-10-04T13:59:39.834Z     at ClientRequest.request.once.error (/usr/src/app/node_modules/got/source/request-as-event-emitter.js:176:14)
2019-10-04T13:59:39.835Z     at Object.onceWrapper (events.js:286:20)
2019-10-04T13:59:39.837Z     at ClientRequest.emit (events.js:203:15)
2019-10-04T13:59:39.838Z     at ClientRequest.origin.emit.args (/usr/src/app/node_modules/@szmarczak/http-timer/source/index.js:37:11)
2019-10-04T13:59:39.839Z     at Immediate.timeoutHandler (/usr/src/app/node_modules/got/source/utils/timed-out.js:63:11)
2019-10-04T13:59:39.840Z     at runCallback (timers.js:706:11)
2019-10-04T13:59:39.841Z     at tryOnImmediate (timers.js:676:5)
2019-10-04T13:59:39.843Z     at processImmediate (timers.js:658:5)

Yeah, apparently the value is not passed down to requestAsBrowser and the default is used. @petrpatek


Swagger dosen't work. Only default swagger template is visible.

Describe the bug
Swagger is not available on 8090 port. There is swagger on http://localhost:8080/swagger-ui/index.html but it is default tamplate - with pets endpoint :) I am on the master branch.
Expected behaviour
http://localhost:8090/swagger-ui/index.html will present api documentation
Steps to reproduce the issue
Go to http://localhost:8090/swagger-ui/index.html - doesn't work
go to http://localhost:8080/swagger-ui/index.html - default swagger template is showing

To make it work you need to enter url: localhost:8080/swagger and will be redirected to http://localhost:8080/swagger-ui/index.html?configUrl=/api/docs/swagger-config
Documentation update will be nice.

Thank you @flamaniac for the correct URL!


Cell Renderer

Im trying to use cell renderer.
How could I get more fields from record than only current column value?
IE: from your example:
#[derive(TableComponent, Serialize, Deserialize, Debug, Clone, Default, PartialEq)]
#[table(sortable)]
pub struct Book {
    #[table(key)]
    pub id: u32,
    #[table(skip)]
    pub title: String,
    #[table(skip)]
    pub author: String,
    #[table(renderer = "TitleAuthorTableCellRenderer")]
    pub title_author: String,
    ...
}

I want to concat title and author and show in one column.

You're right, this is not possible yet. I'll implement a solution ASAP!

Just released v0.2.0 which implements getters.
Please check https://github.com/Synphonyte/leptos-struct-table#field-getters for how to use.

Ok, thanks, the getter is good solution, but what if want to return view!?
view! { cx, {self.title} " (" {self.author} ")" }

In that case you would use both, a getter and a custom renderer.
The renderer will get the value that the getter returns


Review maven dependencies

Remove cilyc dependencies
Remove unnecesary files in deb/rpm packaging

Issue migrated to JIRA.


Table builder Browser column overrides parent module field

Description
When using Table builder to add Browser column, its value overrides parent module field.
For example, parent module has a browser to other modules, and when I try to add Browser column to the index table to show related modules, their representation overrides parent modules "title" column.
$table->add(
    Browser::make()->browser('operation_types')->field('title')->title('Operation Types')
);

Steps to reproduce

Create two modules.
Add browser to one of them which allows to add relations to other module.
Add Browser column to the module index table which takes related module "title" field.

Expected result
Table should show Browser column as a separate column.
Actual result
Table shows Browser as a separate column, but at the same time it overrides parent module title field.
Versions
Twill version: 3.0.2
Laravel version: 10.13.0
PHP version: 8.1.0
Database engine: MySQL 8.0.29

Because they have same column name, when calling getIndexTableData, the module's title value will be replaced by browser's title value. There is a easy way to fixed it.
In A17\Twill\Services\Listings\Columns\Browser:
public function getKey(): string
    {
        if ($this->key === null) {
            throw new ColumnMissingPropertyException();
        }
        return "$this->browser.$this->key";
    }


[POC] bind state to props

Hello, this is just a draft proposal of how we could fix https://github.com/acdlite/recompose/issues/259.

Hi,
the main problem of this HOCs that state will be dependent not on data but on render calls.
It's somehow similar to #199 and even with selector it has the same problem.
The other way is to use initialState function with signature like (nextProps, props, state) similar to all that we have in componentWillReceiveProps.
I wrote here https://github.com/acdlite/recompose/issues/259#issuecomment-254812353
that having multiple source of one state is not a good idea, and it's better to avoid such situations, or solve with tools like rx.


[faiss v1.7.1] Impl PreTransformIndex

https://github.com/facebookresearch/faiss/pull/1816
and
https://github.com/facebookresearch/faiss/pull/1869
merged

Maybe we should not use the option when own_fields = = false?

Maybe we should not use the option when own_fields = = false?

I mean, I suppose part of the problem would no longer apply if we require all indexes to be owned (and so own_fields would never be false), but adding a few type/lifetime parameters should do the trick here.

I won't complicate the impl. I like the transfer of ownership.

I suggest starting with the ownership of indexes, quantizers. In the future, it will become clear whether to use own_fields = = false


InferContext.ResultFormat.CLASS for RNN

Currently I am working on sequence prediction model with RNN and the output will be in the shape of [batch_size, seq_len, tags]. After trying with the following request format
ctx.run(
    {input_cfg.name: [np.array([1, 2, 3, 4]).astype("int32")]},
    {output_cfg.name: (InferContext.ResultFormat.CLASS, top_k_classes)},
    batch_size
)

The result is assumed as a single tensor
{u'111': [[(9L, 0.05252315104007721, u'')]]}

Is there a way to make the prediction in the form of [batch_size, seq_len]? Or for now we need to do it manually using RAW output?

Are you asking how to make a request with a batch-size > 1? For your input you just need to supply multiple np.array(...).astype("int32") entries in the list that you pass to ctx.run. Of course set "batch_size" to match the length of that list.
Your result array will then have multiple outputs, one for each entry in the batch.

@deadeyegoodwin nope, I’m asking on how to output multiple class in one request. So for example if I have a request with size of ‘[batch, seq_len]’. I want the output be in the same form as the input ‘[batch, seq_len]’. Imagine problem like named entity recognition or pos tagging where each words has their own prediction instead of only one prediction for the whole sentence

The inference server doesn't have that feature, it can only return one set of class results for each batch entry. You'll need to post-process the raw results yourself if you want something more.

Got it, thanks!


Question: Why "classpath("com.android.tools.build:gradle")" is commented?

On the instructions, it says this:
    dependencies {
        classpath("org.lsposed.lsparanoid:gradle-plugin:0.4.4")
        // classpath("com.android.tools.build:gradle")
    }

Why the comment? It's actually an important part, no?
Is it to show that it's supposed to be before?
Or the order doesn't matter?
Please explain.

order does not matter now.
and we don't want to hard code AGP since we don't want to specify a version name

I don't understand. Why is it commented?
Should it be removed?

commected because theres no verson name
does not mean it should be removed

So I can ignore it, right?
I already know it's needed...
It's just misleading as I never saw such a thing on other repositories.
Including the one that this one is based on:
https://github.com/MichaelRocks/paranoid

Anyway, please consider removing the comment, or change it to being uncommented. It's confusing...

no
if we remove the comment, we need to specify a version name, and this will confuse user that the plugin only works on a specific version of AGP
and the original repo does not even mention that you have to add AGP as dependency, which is more confusing.

Why do you need to specify the version name? You can just remove it completely, and show only the dependency of the repository.
It's already one of the basic things you have when you create an Android app project. This is what I get from the wizard of Android Studio. Today the structure of a new project even changed. It's not with "classpath" anymore, no?
Putting a comment shows as if you want to get rid of it. It has nothing to do with the version. Putting a comment means it won't be included.
Why do you add it? I never saw on any other repository a comment of an existing dependency, with the explanation that it shouldn't be commented. Why comment and say it shouldn't be commented...
I don't understand.

aha. because we need AGP.
I know theres no more classpath now. we will make it morden.

But AGP was already added to every new Android project.
And this library is for Android projects, so developers should know it's supposed to be there.
Yet, instead, I see it being commented, as if this repository replaces this dependency. Commenting a dependency doesn't mean that it's without a version. Commenting it means it's not used. This is confusing...
Repositories that want to mention a dependency without a version will just say it. They won't use a comment of a different dependency.
The dependency of the repository has a version, too, even though it doesn't have to have it.
Here, examples of Android repositories that don't mention AGP, as it makes sense that it's needed already:
https://github.com/MichaelRocks/libphonenumber-android
https://github.com/aws-amplify/aws-sdk-android
https://github.com/hdodenhof/CircleImageView


Change CRF layer dtype

Description
Addresses #2231. Note that mixed precision has already fixed in f429133, but the behavior is different from TensorFlow semantics.
Fixes # (issue)
Type of change

[x] Bug fix
[ ] New Tutorial
[ ] Updated or additional documentation
[ ] Additional Testing
[ ] New Activation and the changes conform to the activation contribution guidelines
[ ] New Callback and the changes conform to the callback contribution guidelines
[ ] New Image addition and the changes conform to the image op contribution guidelines
[ ] New Layer and the changes conform to the layer contribution guidelines
[ ] New Loss and the changes conform to the loss contribution guidelines
[ ] New Metric and the changes conform to the metric contribution guidelines
[ ] New Optimizer and the changes conform to the optimizer contribution guidelines
[ ] New RNN Cell and the changes conform to the rnn contribution guidelines
[ ] New Seq2seq addition and the changes conform to the seq2seq contribution guidelines
[ ] New Text addition and the changes conform to the text op contribution guidelines

Checklist:

[x] I've properly formatted my code according to the guidelines

[ ] By running Black + Flake8
[ ] By running pre-commit hooks

[ ] This PR addresses an already submitted issue for TensorFlow Addons
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] This PR contains modifications to C++ custom-ops

How Has This Been Tested?
Passes original tests.
If you're adding a bugfix or new feature please describe the tests that you ran to verify your changes:
*

Hi @xuxingya, can you review this if time allows? Thank you!


Can't coerce newValue base on oldValue

Hello, every one.
In case, I am using "static get props {} " to define a property that is an object which itself include 2 or more properties.  Then it would be nice if I can do something as below.
Right?
static get props {     myProp: {         default: { a: 1, b: 1},         coerce: (newValue, oldValue) => {             const { a = oldValue.a, b = oldValue.b } = newValue;             return {a, b};         }     } }

Can you provide a concrete use-case for this? It's not immediately clear to my why this would be useful. Thanks :)

Ahh right, so basically you want to merge the previous value with the new value? This could potentially be unexpected for the consumer. To get around this you can do something like elem.prop = { ...elem.prop, ...{ updated: true } }.
I really want to keep each feature in prop definitions as simple as possible and free from side effects. Thanks for raising the issue, though :)


Closes Issue #51: Cannot download some files from the evaluate-contigs qzv

Closes #51
While this solves the issue, I could not generate the same contig.qzv provided in the issue description. The artifact that I could generate allows the user to download the corresponding graphs however they are empty PDFs in contrast with the provided contig.qzv where the corresponding PDFs actually contain graphs. Both artifacts in question were generated with the same parameters but in different operating systems.

Hey @Sann5, could you please merge/rebase using the main branch? That should fix the CI 🔮


[WIP] MAISTRA-2057 Add missing _affinity.tpl to the istio-control helm templates

@dgn @rcernich @knrc
Do I need to update charts for this PR eg:  running make gen  or make update-2.1-charts or is this PR is sufficient to test?

/test all

Closing this and created https://github.com/maistra/istio/pull/300


BulkInsert can receive IDs out of order

We have had a problem where the returned identity Id is sometime set out of order when using BulkInsert.
According to Microsoft's documentation, using OUTPUT does not garantee that the order is preserved.
Please see the following article:
https://docs.microsoft.com/en-us/sql/t-sql/queries/output-clause-transact-sql?redirectedfrom=MSDN&view=sql-server-ver15

However, SQL Server does not guarantee the order in which rows are processed and returned by DML statements using the OUTPUT clause. It is up to the application to include an appropriate WHERE clause that can guarantee the desired semantics, or understand that when multiple rows may qualify for the DML operation, there is no guaranteed order.

This is a major problem for us, is there a way we can solve this problem?

Pwelzien,
The library should be using an internal guid to track the entities from the client to SQL Server and back.
I will look into the code if to see how this issue is being created.

Please re-test this bug and confirm that it has been fixed.

Pwelzien,
I was able to replicate the issue you reported and I can confirm that the latest update fixed the issue.

Is this update in v1.4.6? I have that version and I'm getting the ID values out of order. If this it going to be in 1.4.7 can you tell me when that will be available via NuGet? Thx

The fix would have been in v1.4.6. Can you confirm that you have this version?
I thoroughly tested v1.4.6 after fixing the issue with the ID's being out of order, and I'm confident that it has been fixed.
I am releasing v1.4.7 today but that fixes a different issue.

I'm running 1.4.6 and I have been trying to figure this out for a few hours. Related items were being assigned to the wrong things and I just realized that it's because of the IDs being out of order.

mbakervti,
This issue will be fixed in v1.4.7 which I plan to release tomorrow.

Thanks for the update. I'll test it as soon as I can get my hands on it.

I just released v1.4.7 as a nuget package. Please test it when you have the chance and once you have confirmed that it is fixed, I will close this issue.

I'm still getting the bad behavior and I'd like to send you some console dumps. Do you want them here or to a different email?

mbakervti,
Please try v1.4.8, it should fix your issue.
If it doesn't fix it, then you can send your dump files to this email.

YES!! That appears to have it fixed. I was going to mention that it seemed to work OK with really small lists. A list of 25 worked but a list of 35 didn't. I just tried it with a list of 145 and it worked just fine.

mbakervti,
That is great! If there is an other issues you find or any features you would like to request, please report them as a new issue.


The IPv4 conditional item doesn't work properly

Hi Hannes
When creating a conditional item for ip-address it needs to be preceded by the word 'ANY' to check all interfaces.
This is easy to edit in the text view but it would be awesome if it was inserted automatically

Thank you for reporting. Will fix at some point. The condition editor is also missing some of the newer keys...

Fix for this is finally included in version 1.4.3.


Tag support

It would be great to have tags to post, and have them organized in another page.

Great idea.
If any would like to help develop this feature, I'm happy to code review and merge this feature into the theme.


Add config option

OK, I will add configoptions to set path to PostCSS config :-P.

We don’t need multiple configs anymore https://github.com/postcss/postcss-loader#plugins

Is there a way to specify the postcss config location as even with adding the configuration to webpack 2
{
        test: /\.(css|scss)$/,
        use: [
          { loader: 'style-loader' },
          { loader: 'css-loader' },
          // See postcss.config.js for configuration
          {
            loader: 'postcss-loader',
            options: {
              plugins: function () {
                return [
                  require('autoprefixer')({
                    browsers: [
                      '>1%',
                      'last 3 versions',
                      'Firefox ESR',
                      'not ie < 9', // React doesn't support IE8 anyway
                    ]
                  })
                ];
              }
            }
          },
          {
            loader: 'sass-loader',
            options: {
              importLoaders: true
            }
          }
        ]
      },

Im still getting an error saying Module build failed: Error: No PostCSS Config found in:

@samjbmason please, debug what you have here in params.plugins and loader.options?
https://github.com/postcss/postcss-loader/blob/master/index.js#L48

@ai Im sure Im not I follow what your asking for

@samjbmason open plugin code in node_modules/postcss-loader, put console.log(params.plugins, loader.options) after line 48. Run build. Post here the output.

Ah right yeah sure thing
undefined { devtool: 'cheap-module-source-map',
  entry:
   [ '/Users/samm/Sites/lmn-webpack-config/node_modules/react-dev-utils/webpackHotDevClient.js',
     './src/js/main.js' ],
  output:
   { pathinfo: true,
     path: '/Users/samm/Sites/design-system/build',
     publicPath: '/',
     filename: 'bundle.js',
     chunkFilename: '[id].bundle.js',
     library: '',
     hotUpdateFunction: 'webpackHotUpdate',
     jsonpFunction: 'webpackJsonp',
     libraryTarget: 'var',
     sourceMapFilename: '[file].map[query]',
     hotUpdateChunkFilename: '[id].[hash].hot-update.js',
     hotUpdateMainFilename: '[hash].hot-update.json',
     crossOriginLoading: false,
     hashFunction: 'md5',
     hashDigest: 'hex',
     hashDigestLength: 20,
     devtoolLineToLine: false,
     strictModuleExceptionHandling: false },
  resolve:
   { extensions: [ '.js', '.json', '.scss' ],
     alias:
      { component: '/Users/samm/Sites/design-system/node_modules/@lostmyname/components/dist',
        chameleon: '/Users/samm/Sites/design-system/node_modules/chameleon-sass/assets/stylesheets',
        css: '/Users/samm/Sites/design-system/node_modules/@lostmyname/css/scss',
        fonts: '/Users/samm/Sites/design-system/node_modules/@lostmyname/css/fonts',
        scss: '/Users/samm/Sites/design-system/src/scss',
        helpers: '/Users/samm/Sites/design-system/src/js/helpers' },
     modules:
      [ '/Users/samm/Sites/design-system/src',
        'node_modules',
        '/Users/samm/Sites/design-system/node_modules' ],
     unsafeCache: true,
     aliasFields: [ 'browser' ],
     mainFields: [ 'browser', 'module', 'main' ] },
  resolveLoader:
   { modules: [ '/Users/samm/Sites/lmn-webpack-config/node_modules' ],
     unsafeCache: true,
     mainFields: [ 'loader', 'main' ],
     extensions: [ '.js', '.json' ] },
  module:
   { rules: [ [Object], [Object], [Object], [Object], [Object], [Object] ],
     unknownContextRequest: '.',
     unknownContextRegExp: false,
     unknownContextRecursive: true,
     unknownContextCritical: true,
     exprContextRequest: '.',
     exprContextRegExp: false,
     exprContextRecursive: true,
     exprContextCritical: true,
     wrappedContextRegExp: /.*/,
     wrappedContextRecursive: true,
     wrappedContextCritical: false,
     unsafeCache: true },
  plugins:
   [ OccurrenceOrderPlugin { preferEntry: undefined },
     EnvironmentPlugin { keys: [Object] },
     HotModuleReplacementPlugin { multiStep: undefined, fullBuildTimeout: 200 } ],
  node:
   { fs: 'empty',
     net: 'empty',
     tls: 'empty',
     console: false,
     process: true,
     global: true,
     Buffer: true,
     setImmediate: true,
     __filename: 'mock',
     __dirname: 'mock' },
  cache: true,
  context: '/Users/samm/Sites/design-system',
  target: 'web',
  performance:
   { maxAssetSize: 250000,
     maxEntrypointSize: 250000,
     hints: 'warning' } }

Additional info is that this is running the latest webpack 2 RC

Replace it to console.log(options)

Getting undefined logged out

Did you put it after 48 line? What output will be for console.log(params.plugins || loader.options.postcss)?

@samjbmason Could you try removing the '.scss' ext from webpack.resolve.extensions, since you testing for /\$.(css|scss}/ in webpack.module.rules?

Could you replace line 48 to:
var options  = params.plugins || loader.postcss || loader.options.postcss;

Did it fix build for you?

Yeah sure did here is the code
var loader = this;
    var file   = loader.resourcePath;
    var params = loaderUtils.parseQuery(loader.query);

    var options  = params.plugins || loader.options.postcss;
    console.log(options)
    var pack     = params.pack;
    var callback = loader.async();

@michael-ciniawsky seems like it has postcss in loader by some reason :-/

@samjbmason Could you replace line 48 to:
var options  = params.plugins || loader.postcss || loader.options.postcss;

Did it fix build for you?

@ai Still getting undefined after adding var options  = params.plugins || loader.postcss || loader.options.postcss;

@michael-ciniawsky oops, it was webpack plugins :)

@samjbmason go to webpack team. It is webpack 2 issue or webpack UX issue.
/cc @TheLarkInn

Brilliant thanks, do you know what kind of info they will need when I log the issue?

@samjbmason they will need your full config, full webpack version and that loaderUtils.parseQuery(loader.query) didn’t contains config options.

@samjbmason I think issue is locate somewhere here. Does it build correctly without postcss-loader ?
 resolve:
   { extensions: [ '.js', '.json', '.scss' ],
     alias:
      { component: '/Users/samm/Sites/design-system/node_modules/@lostmyname/components/dist',
        chameleon: '/Users/samm/Sites/design-system/node_modules/chameleon-sass/assets/stylesheets',
        css: '/Users/samm/Sites/design-system/node_modules/@lostmyname/css/scss',
        fonts: '/Users/samm/Sites/design-system/node_modules/@lostmyname/css/fonts',
        scss: '/Users/samm/Sites/design-system/src/scss',
        helpers: '/Users/samm/Sites/design-system/src/js/helpers' },
     modules:
      [ '/Users/samm/Sites/design-system/src',
        'node_modules',
        '/Users/samm/Sites/design-system/node_modules' ],
     unsafeCache: true,
     aliasFields: [ 'browser' ],
     mainFields: [ 'browser', 'module', 'main' ] },
  resolveLoader:
   { modules: [ '/Users/samm/Sites/lmn-webpack-config/node_modules' ],
     unsafeCache: true,
     mainFields: [ 'loader', 'main' ],
     extensions: [ '.js', '.json' ] },

@TheLarkInn maybe I use API in wrong way? But I didn’t found good 2.0 migration guide for loader authors.
We load plugins from:
{
  loader: 'posctss-loader',
  options: {
    plugins: [function () { … }]
  }
}

by loaderUtils.parseQuery(loader.query).plugins

@samjbmason To my knowledge there is no importLoaders: true option for sass-loader

@michael-ciniawsky Yeah it works when i remove postcss-loader

@samjbmason Yep, I can confirm that's a bug, functions/requires inside of rules.use[n].options are not parsed/loaded at all in<EMAIL_ADDRESS>Meanwhile switch to either postcss.config.js (recommended) or try with LoaderOptionsPlugin
Tracking issue is here
/cc @sokra @TheLarkInn

@michael-ciniawsky Unfortunately using a postcss.config.js doesnt work as Im creating the webpack within another npm module similar to how create-react-app works.
However, I can confirm LoaderOptionsPlugin works for now 👍
Thanks for all your help on this

@samjbmason Ok I understand, so with LoaderOptionsPlugin the No PostCSS Config found in: ...vanishes (Definitely no postcss.config.js present somewhere, 'inline' options get parsed (especially functions) correctly)?

Stop. We should not use LoaderOptionsPlugin here.
Seems like webpack 2 has a issue. If we will fix it by a hack, issue will be released version and many users will get it.
So, don't use hack. Create a issue in webpack repo.

@ai Already open

@ai I agree LoaderOPtionsPlugin is not a long term solution but was useful that I could use it, before the bug gets fixed.


How can I fix the error generated by the ListView?

Guys I have a question for you:
I have built a PageView. In this PageView is a ListView and in this ListView is a list of widgets.
But unfortunately my code doesn't work for some reason ... And I have no idea why. I'm still very new to Flutter. I would really appreciate your help.
The code that generates the error is the following:
Widget build(BuildContext context) {
return Scaffold(
backgroundColor: Colors.white,
body: ListView(
children: createHomePage(
controller: _controller,
content: widget.content,
title: widget.title,
menuButtonAction: widget.menuButton,
showShadow: widget.showShadow,
context: context),
),
);
}
}
I would be really grateful if you could help me. The code works fine when I replace the ListView with a Colum, but then unfortunately I can't scroll ... But it has to be scrollable!
I keep getting the following error:
I/flutter ( 4539): ══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
I/flutter ( 4539): The following assertion was thrown building NotificationListener:
I/flutter ( 4539): Incorrect use of ParentDataWidget.
I/flutter ( 4539): Expanded widgets must be placed inside Flex widgets.
I/flutter ( 4539): Expanded(no depth, flex: 1, dirty) has no Flex ancestor at all.
I/flutter ( 4539): The ownership chain for the parent of the offending Expanded was:
I/flutter ( 4539):   RepaintBoundary ← IndexedSemantics ←
NotificationListener ← KeepAlive ←
I/flutter ( 4539): AutomaticKeepAlive ← KeyedSubtree ← SliverList ← MediaQuery ← SliverPadding ← ShrinkWrappingViewport
I/flutter ( 4539): ← ⋯
I/flutter ( 4539):
I/flutter ( 4539): The relevant error-causing widget was:
I/flutter ( 4539):   ListView
How can I solve this problem? I've tried a lot. For example, I put the list of widgets in a column and changed the ListView to a SingleChildListView, but that didn't help either. However, I got another error message. This was:
══╡ EXCEPTION CAUGHT BY RENDERING LIBRARY ╞═════════════════════════════════════════════════════════
I/flutter ( 4539): The following assertion was thrown during performLayout():
I/flutter ( 4539): RenderFlex children have non-zero flex but incoming height constraints are unbounded.
I/flutter ( 4539): When a column is in a parent that does not provide a finite height constraint, for example if it is
I/flutter ( 4539): in a vertical scrollable, it will try to shrink-wrap its children along the vertical axis. Setting a
I/flutter ( 4539): flex on a child (e.g. using Expanded) indicates that the child is to expand to fill the remaining
I/flutter ( 4539): space in the vertical direction.
I/flutter ( 4539): These two directives are mutually exclusive. If a parent is to shrink-wrap its child, the child
I/flutter ( 4539): cannot simultaneously expand to fit its parent.
I/flutter ( 4539): Consider setting mainAxisSize to MainAxisSize.min and using FlexFit.loose fits for the flexible
I/flutter ( 4539): children (using Flexible rather than Expanded). This will allow the flexible children to size
I/flutter ( 4539): themselves to less than the infinite remaining space they would otherwise be forced to take, and
I/flutter ( 4539): then will cause the RenderFlex to shrink-wrap the children rather than expanding to fit the maximum
I/flutter ( 4539): constraints provided by the parent.
I/flutter ( 4539): If this message did not help you determine the problem, consider using debugDumpRenderTree():
I/flutter ( 4539):   https://flutter.dev/debugging/#rendering-layer
I/flutter ( 4539):   http://api.flutter.dev/flutter/rendering/debugDumpRenderTree.html
I/flutter ( 4539): The affected RenderFlex is:
I/flutter ( 4539):   RenderFlex#bfee3 relayoutBoundary=up15 NEEDS-LAYOUT NEEDS-PAINT NEEDS-COMPOSITING-BITS-UPDATE(creator: Column ← RepaintBoundary ← IndexedSemantics ← NotificationListener ← KeepAlive ← AutomaticKeepAlive ← KeyedSubtree ← SliverList ← MediaQuery ← SliverPadding ← ShrinkWrappingViewport ← IgnorePointer-[GlobalKey#665ae] ← ⋯, parentData:  (can use size), constraints: BoxConstraints(w=411.4, 0.0<=h<=Infinity), size: MISSING, direction: vertical, mainAxisAlignment: start, mainAxisSize: max, crossAxisAlignment: center, verticalDirection: down)
I/flutter ( 4539): The creator information is set to:
I/flutter ( 4539):   Column ← RepaintBoundary ← IndexedSemantics ← NotificationListener ←
I/flutter ( 4539):   KeepAlive ← AutomaticKeepAlive ← KeyedSubtree ← SliverList ← MediaQuery ← SliverPadding ←
I/flutter ( 4539):   ShrinkWrappingViewport ← IgnorePointer-[GlobalKey#665ae] ← ⋯
I/flutter ( 4539): See also: https://flutter.dev/layout/
I/flutter ( 4539): If none of the above helps enough to fix this problem, please don't hesitate to file a bug:
I/flutter ( 4539):   https://github.com/flutter/flutter/issues/new?template=BUG.md
I/flutter ( 4539):
I/flutter ( 4539): The relevant error-causing widget was:
I/flutter ( 4539):   Column
In the second example, the screen turns white, but nothing happens and nothing works. At the first error the screen turns red and I get the error that you can see in the picture!
Red Error on the Screen
 If you have found a bug or if our documentation doesn't have an answer
 to what you're looking for, then fill our the template below. Please read
 our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports

-->
Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
...

Expected results:
Actual results:

Logs

Hi @LarsHenning98
can you please provide your flutter doctor -v and flutter run --verbose?
Also, to better address the issue, would be helpful if you could post a minimal code sample to reproduce the problem, your code sample is incomplete
Thank you


Global permissions on UI are in a random order

Setup

Rancher version: v2.7.13-rc4
Rancher UI Extensions: release-2.7 d14964b0d
Browser type & version: Chrome

Describe the bug
Global permissions on the Users and authentication page, have the permissions in a different order.
To Reproduce

Install rancher 2.7.13-rc4
Navigate to users and authentication >> Users
Create a user
Notice the order of the global permissions

Result
Restricted Admin
User base
Administrator
Standard user

Expected Result
Expected the order to be:
Administrator
Restricted Admin
Standard user
User base

Screenshots

Additional context:
Do not see the issue on v2.7.12

I see a slightly different order in 2.9-head that is probably the same issue and should be fixed

Admin
User-Base
Restricted Administrator
Standard User

/forwardport v2.9.0

/forwardport v2.8.next

/forwardport v2.8.next1

For docs (cc: @btat) we need a release note for the current 2.7.next1 (2.7.13) as we move this work to a later version. It is a regression though the UI around this hasn't changed in years so we'll have to look at the data given to us.

Will not fix for 2.7.x. We have tickets for 2.9.x & 2.8.x at this stage though based on timing, we have to sort this out in 2.9.x first (or even 2.10.0 now)


Kestrel  rejects requests with non standard ASCII characters

Description
When sending a request to Kestrel web server with header value which has non standard ASCII characters, the server rejects the response with 400 Bad request: malformed headers.
Reproduction steps
Steps to reproduce the behavior:

create a new web app: dotnet new web
run the application as console (not IISExpress)
send a request (using Fiddler/postman/wget) with non standard ASCII header value

This is reproducible with 2.1.9, 2.1.10 and 2.2.4
wget -Uri http://localhost:5000 -Headers @{"test"="Tk'emlúps te Secwépemc"}
Expected behavior
The request is accepted and the header value is accessible in the Request.Headers
Additional context
.NET Core SDK (reflecting any global.json):
Version:   2.2.203
Commit:    e5bab63eca
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.17763
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\2.2.203\
Host (useful for support):
Version: 2.2.4
Commit:  f95848e524
.NET Core SDKs installed:
2.1.505 [C:\Program Files\dotnet\sdk]
2.1.603 [C:\Program Files\dotnet\sdk]
2.2.106 [C:\Program Files\dotnet\sdk]
2.2.203 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.9 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.1.10 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.2.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.9 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.1.10 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.2.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.9 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.10 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.2.4 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]

Forgot to mention that when using IIS/IISExpress, inproc mode accepts the header while the default out of proc doesn't (as 'expected'...)

@JunTaoLuo / @Tratcher
What is the expected behaviour here? You know I believe rejecting the headers is the correct default, with an option to accept invalid and possibly dangerous input being left as opt-in, for all hosts.

UTF-8 sent directly to Kestrel should work. No other encodings are accepted. We'd want a wireshark trace to verify the encoding.
If you send UTF-8 through IIS out-of-proc the encoding gets messed up.

Note the UTF-8 support was added in 2.2 (https://github.com/aspnet/KestrelHttpServer/issues/1144). That's the only version we need to investigate.

@Tratcher here's the Wireshark trace. IIS is not involved.
ws capture.zip

0050   48 54 54 50 2f 31 2e 31 0d 0a 74 65 73 74 3a 20   HTTP/1.1..test:
0060   54 6b 27 65 6d 6c fa 70 73 20 74 65 20 53 65 63   Tk'eml.ps te Sec
0070   77 e9 70 65 6d 63 0d 0a 55 73 65 72 2d 41 67 65   w.pemc..User-Age
0080   6e 74 3a 20 4d 6f 7a 69 6c 6c 61 2f 35 2e 30 20   nt: Mozilla/5.0

That's not UTF-8, it's ANSII. ú is encoded as fa. In UTF-8 it would be \xC3\xBA.

I tried the following headers, both didn't return 400, but the value read as is from ctx.request.headers:
wget -Uri http://localhost:5000 -Headers @{"test"="\x54\x6b\x27\x65\x6d\x6c\xc3\xba\x70\x73\x20\x74\x65\x20\x53\x65\x63\x77\xc3\xa9\x70\x65\x6d\x63"; "Accept-Charset"="UTF-8"}

wget -Uri http://localhost:5000 -Headers @{"test"="\x54\x6b\x27\x65\x6d\x6c\xc3\xba\x70\x73\x20\x74\x65\x20\x53\x65\x63\x77\xc3\xa9\x70\x65\x6d\x63"}

wget -Uri http://localhost:5000 -Headers @{"test"="Tk'eml\xC3\xBAps te Secw\xC3\xBApemc"; "Accept-Charset"="UTF-8"}

Wireshark? Are you sure it's not sending the literal string \x54 rather than the byte 54?

Any idea how to force sending a byte array instead of a string as the header value?

No, very few clients let you control the header encoding.

A reverse proxy is your best option.

Ok, thanks, I'll get working on that. It would be nice if the server was a little more lenient in this, or at least allows interjection. This exact set up is working with IIS in proc mode as well as other web servers. Unfortunately Kesterl is the only container hosting option ATM.

Closing as per the discussion. Kestrel requires the use of UTF-8 or ASCII, ANSI extensions are not supported. Either use a client that sends UTF-8, or use a reverse proxy that transcodes this.


Error with HotelReservation (bug?)

After attempting to use the command sudo docker-compose up -d, this yields the error

on both local ubuntu based VMs & AWS based ubuntu VM.
Has anyone got a fix for this?

Hi, not sure if this is a good fix or not, but try adding this to the beginning of .../hotelReservation/Dockerfile:
RUN apt update && apt install ca-certificates libgnutls30 -y


Tracking versus the main distribution (WIP)

This is an attempt to add automated testing to this app; it's far from a place ready to review; I just wanted to see how far off from the main app it currently is.

Hi!


Add method for external custom toolbar

Hi @dstansby ,
Would you mind considering the idea suggested in this comment?
https://github.com/matplotlib/napari-matplotlib/pull/202#issuecomment-1708833704
That would allow others to replace the toolbar by a customized version.
Looking forward to your reply!
Best,
Marcelo

Related to this, would there be a way to set the canvas manager like this?
matplotlib.rcParams["toolbar"] = "toolmanager"
I think then some code like in this example would work, but right now I could not make it work (figure.canvas.manager is None)

I think this is beyond scope of napari-matplotlib too, sorry - hopefully you can use the code in https://github.com/matplotlib/napari-matplotlib/pull/214 to achieve the same effect in a third-party library?

Alright, maybe I will do that then, but for now, I think it will be easier to implement these tools outside the matplotlib toolbar. Thanks for your time!