Use blueprintjs HTMLTable

closes #583

Replace "Title" with "Header" to make the bordered property work in the HTMLTable header
Add the color prop for StyledTable
Remove Controls story (duplicate of Control)

Codecov Report
Attention: 128 lines in your changes are missing coverage. Please review.

Comparison is base (fa300f8) 47.14% compared to head (098bc28) 24.44%.
Report is 17 commits behind head on main.

Files
Patch %
Lines

stories/components/table.stories.tsx
0.00%
79 Missing :warning:

src/components/table/Table.tsx
21.56%
40 Missing :warning:

stories/components/drop-zone.stories.tsx
0.00%
4 Missing :warning:

src/app/panels/SignalProcessingPanel.tsx
0.00%
3 Missing :warning:

src/components/info-panel/InfoPanel.tsx
0.00%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #611       +/-   ##
===========================================
- Coverage   47.14%   24.44%   -22.71%
===========================================
  Files         143      223       +80
  Lines        7502    13011     +5509
  Branches      146      234       +88
===========================================
- Hits         3537     3180      -357
- Misses       3965     9743     +5778
- Partials        0       88       +88

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

@stropitek an error occurs when saving the file Table.tsx (since before blueprint integration):

Error: invalid Table child
at splitChildren

It disappears after refreshing the page

@stropitek an error occurs when saving the file Table.tsx

@moonayyur what do you mean? Can you provide a link / screenshot / video to show it?

@moonayyur what do you mean? Can you provide a link / screenshot / video to show it?

@stropitek

Is Table's color prop used anywhere? It does not seem very useful.

It is only use in StyledTable story, the text color doesn't change when modifying it using style due to the blueprint component

@moonayyur are there any other breaking changes?

I think this one is important :
Replace "ValueRenderers.Title" with "ValueRenderers.Header" to make the bordered property work in the HTMLTable header


⚠ Bofrost.de has degraded performance

In 94ec209, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 868 ms

Resolved: Bofrost.de performance has improved in 3aa67ab after 6 minutes.


make node manager tolerate disk and memory pressure

Coverage decreased (-0.01%) to 51.762% when pulling 7ee9c7388b5e7c2c50c81b35c0e42f2863789950 on zimiao/node_manager_toleration into 92975d7da2fca0e2695136a6d9a058774d57959a on master.


isinstance check used on optional fields throws AssertionError

https://github.com/turettn/fastapi_offline/blob/4e400ab27e36c219eb1604c9c67a1626cc3494e2/fastapi_offline/core.py#L28-L29
In the main FastAPI class, these are optional, so they are allowed to be None:
openapi_url: Optional[str] = "/openapi.json",
swagger_ui_oauth2_redirect_url: Optional[str] = "/docs/oauth2-redirect",

If a user calls the class like this:
app = FastAPIOffline(openapi_url=None)

They get an assertion error:
  File "main.py", line 19, in <module>
    app = FastAPIOffline(openapi_url=None)
  File "venv/lib/python3.9/site-packages/fastapi_offline/core.py", line 28, in FastAPIOffline
    assert isinstance(app.openapi_url, str)

This is needed in situations where you want to nest a FastAPI instance beneath another instance that you wish to disable the OpenAPI docs on:
app = FastAPIOffline(openapi_url=None)

api_app = FastAPIOffline()
app.mount("/my_api", api_app)

I believe that FastAPIOffline should handle the case that these fields are None in a nested context.

I think for sub-applications to work properly within fastapi-offline, FastAPIOffline needs to check/respect root_path: https://fastapi.tiangolo.com/advanced/sub-applications/#technical-details-root_path

I'm not sure if this is the best method, but one way I got mounted sub-apps to work is with this method:
from fastapi_offline import FastAPIOffline

app1 = FastAPIOffline(title="App 1")

openapi_mount = "/app2"
app2 = FastAPIOffline(title="App 2", openapi_mount=openapi_mount)
app1.mount(openapi_mount, app2)

@app1.get("/")
async def app_1():
    return {"message": "Hello World"}

@app2.get("/")
async def app_2():
    return {"message": "Hello World 2"}

Add a new openapi_mount parameter.
"Provide non-CDN-dependent Swagger & Redoc pages to FastAPI"
from pathlib import Path
from typing import Any, TYPE_CHECKING
from fastapi import FastAPI
from fastapi.openapi.docs import (
    get_redoc_html,
    get_swagger_ui_html,
    get_swagger_ui_oauth2_redirect_html,
)
from fastapi.staticfiles import StaticFiles

if TYPE_CHECKING:
    from fastapi import Response

_STATIC_PATH = Path(__file__).parent / "static"

def FastAPIOffline(openapi_mount: str = "", *args: Any, **kwargs: Any) -> FastAPI:
    "Return a FastAPI obj that doesn't rely on CDN for the documentation page"
    # Disable the normal doc & redoc pages
    kwargs["docs_url"] = None
    kwargs["redoc_url"] = None

    # Create the FastAPI object
    app = FastAPI(*args, **kwargs)

    # This mostly just keeps mypy happy
    assert isinstance(app.openapi_url, str)
    assert isinstance(app.swagger_ui_oauth2_redirect_url, str)
    openapi_url = f"{openapi_mount}{app.openapi_url}"
    swagger_ui_oauth2_redirect_url = app.swagger_ui_oauth2_redirect_url

    # Set up static file mount
    app.mount(
        "/static-offline-docs",
        StaticFiles(directory=_STATIC_PATH.as_posix()),
        name="static-offline-docs",
    )

    # Define the doc and redoc pages, pointing at the right files
    @app.get("/docs", include_in_schema=False)
    async def custom_swagger_ui_html() -> "Response":
        return get_swagger_ui_html(
            openapi_url=openapi_url,
            title=app.title + " - Swagger UI",
            oauth2_redirect_url=swagger_ui_oauth2_redirect_url,
            swagger_js_url="/static-offline-docs/swagger-ui-bundle.js",
            swagger_css_url="/static-offline-docs/swagger-ui.css",
        )

    @app.get(swagger_ui_oauth2_redirect_url, include_in_schema=False)
    async def swagger_ui_redirect() -> "Response":
        return get_swagger_ui_oauth2_redirect_html()

    @app.get("/redoc", include_in_schema=False)
    async def redoc_html() -> "Response":
        return get_redoc_html(
            openapi_url=openapi_url,
            title=app.title + " - ReDoc",
            redoc_js_url="/static-offline-docs/redoc.standalone.js",
        )

    # Return the FastAPI object
    return app

I think fundamentally, we want a way for sub-apps to respect being mounted, so this could be done automatically rather than manually?

Here is an improved version that ensures that each app is using the proper URLs:
from fastapi import FastAPI
from fastapi_offline import FastAPIOffline

# We disable OpenAPI on the top level, using a normal FastAPI App.
app1 = FastAPI(title="App 1", openapi_url=None)

# Mount an entire second app under a prefix.
openapi_mount = "/app2"
app2 = FastAPIOffline(title="App 2", openapi_mount=openapi_mount)
app1.mount(openapi_mount, app2)

@app1.get("/")
async def app_1():
    return {"message": "Hello World"}

@app2.get("/")
async def app_2():
    return {"message": "Hello World 2"}

Upgrade fastapi-offline to reference the correct mount position of an app:
"Provide non-CDN-dependent Swagger & Redoc pages to FastAPI"
from pathlib import Path
from typing import Any, TYPE_CHECKING
from fastapi import FastAPI
from fastapi.openapi.docs import (
    get_redoc_html,
    get_swagger_ui_html,
    get_swagger_ui_oauth2_redirect_html,
)
from fastapi.staticfiles import StaticFiles

if TYPE_CHECKING:
    from fastapi import Response

_STATIC_PATH = Path(__file__).parent / "static"

def FastAPIOffline(openapi_mount: str = "", *args: Any, **kwargs: Any) -> FastAPI:
    "Return a FastAPI obj that doesn't rely on CDN for the documentation page"
    # Disable the normal doc & redoc pages
    kwargs["docs_url"] = None
    kwargs["redoc_url"] = None

    # Create the FastAPI object
    app = FastAPI(*args, **kwargs)

    # This mostly just keeps mypy happy
    assert isinstance(app.openapi_url, str)
    assert isinstance(app.swagger_ui_oauth2_redirect_url, str)
    openapi_url = f"{openapi_mount}{app.openapi_url}"
    swagger_ui_oauth2_redirect_url = app.swagger_ui_oauth2_redirect_url

    # Set up static file mount
    app.mount(
        f"/static-offline-docs",
        StaticFiles(directory=_STATIC_PATH.as_posix()),
        name="static-offline-docs",
    )

    # Define the doc and redoc pages, pointing at the right files
    @app.get("/docs", include_in_schema=False)
    async def custom_swagger_ui_html() -> "Response":
        return get_swagger_ui_html(
            openapi_url=openapi_url,
            title=app.title + " - Swagger UI",
            oauth2_redirect_url=swagger_ui_oauth2_redirect_url,
            swagger_js_url=f"{openapi_mount}/static-offline-docs/swagger-ui-bundle.js",
            swagger_css_url=f"{openapi_mount}/static-offline-docs/swagger-ui.css",
        )

    @app.get(swagger_ui_oauth2_redirect_url, include_in_schema=False)
    async def swagger_ui_redirect() -> "Response":
        return get_swagger_ui_oauth2_redirect_html()

    @app.get("/redoc", include_in_schema=False)
    async def redoc_html() -> "Response":
        return get_redoc_html(
            openapi_url=openapi_url,
            title=app.title + " - ReDoc",
            redoc_js_url=f"{openapi_mount}/static-offline-docs/redoc.standalone.js",
        )

    # Return the FastAPI object
    return app

The parameter name maybe should be made unique since we are trying to insert ourselves into the FastAPI interface. Perhaps offline_openapi_mount would be more unique.
Let me know what you think about this potential enhancement.

Good catch on the sub-app stuff.
I think there are 2 separate issues here:

The case where you want to disable docs
The case where you want to have sub-apps

For the case where you don't want docs, I assumed people would just use FastAPI - you can mount a FastAPI to a FastAPIOffline, and it works just fine.
For the second case, you definitely found a bug.  The best way to handle this that I found is to actually check the FastAPI root_path and use that when handling the help page requests.  It's less thinking on the part of the user, feels more natural, and should let people do things like have sub-apps of sub-apps without a ton of manual manipulation.
Would you mind looking at merge request #9?  There are two additional test cases - for one, I start with a FastAPIOffline object and mount FastAPI and FastAPIOffline objects.  The other tests the opposite.
Would this solve these problems?

For the case where you don't want docs, I assumed people would just use FastAPI - you can mount a FastAPI to a FastAPIOffline, and it works just fine.

I agree: #10


non-developer testing question

I have a Roku 3 and I've installed the code, seems to work, except I cannot login. I just stumbled across this and gave it a shot, is it complete or still a work in progress?

It’s technically still a work in Progress. Are you using 10.4.1? Did you have an install before?
It could be the baseurl thing. I don’t think this client has been updated for that yet.
You could try adding /Jellyfin at the end of your server entry.

I wonder if this was archived before the password changes.

OH. I totally missed that this is original. @caj411  you want to head over to Jellyfin/Jellyfin-Roku
This is the old archived code.

OH. I totally missed that this is original. @caj411 you want to head over to https://github.com/jellyfin/jellyfin-roku
This is the old archived code.

Thank you. Your response is greatly appreciated!


Android mic and webcam

Hi there,
we are using openuri to connect but mic and webcam are not redirecting, can you help me to get proper format to pass it.
here the example which we are using.
freerdp://user@ip:port/connect?sound=&microphone=sys:alsa&rfx=&p=password&clipboard=%2b&themes=-

webcam is not supported on android, so you will not have much luck
just drop the microphone=sys:alsa and use microphone (there is no alsa on android...)


⚠ Bofrost.fr has degraded performance

In 8990984, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1434 ms

Resolved: Bofrost.fr performance has improved in c5d4806 after 9 minutes.


[toml syntax] Looks like netlify doesnt like our toml config's syntax

In logs we got lots of
Incorrect TOML configuration format: Escape sequence \: is reserved
which is caused by https://github.com/endoflife-date/endoflife.date/blob/0d4399f82bcc223d2f1b5fb7495f13546a93bc45/netlify.toml#L10

@captn3m0

Will be fixed by #2859 (I moved the sed to deploy.sh).

The errors were visible in Netlify Post-processing logs, such as https://app.netlify.com/sites/endoflife-date/deploys/64e0a47ca0fe2f0007129028:
1:18:50 PM: Starting post processing
1:18:50 PM: Incorrect TOML configuration format: Escape sequence \: is reserved
1:18:51 PM: Skipping HTML post processing
1:18:51 PM: Incorrect TOML configuration format: Escape sequence \: is reserved
1:18:51 PM: Incorrect TOML configuration format: Escape sequence \: is reserved
1:18:51 PM: Post processing - header rules
1:18:52 PM: Incorrect TOML configuration format: Escape sequence \: is reserved
1:18:52 PM: Post processing - redirect rules
1:18:52 PM: Incorrect TOML configuration format: Escape sequence \: is reserved
1:18:52 PM: Post processing done
1:18:53 PM: Section completed: postprocessing
1:18:54 PM: Site is live ✨

In recent version (post #2859 merge), such as https://app.netlify.com/sites/endoflife-date/deploys/64e0a4148c837b000894a329, those errors cannot be seen anymore:
1:17:42 PM: Starting post processing
1:17:42 PM: Skipping HTML post processing
1:17:43 PM: Post processing - header rules
1:17:43 PM: Post processing - redirect rules
1:17:43 PM: Post processing done
1:17:43 PM: Section completed: postprocessing
1:17:44 PM: Site is live ✨


dependency name (and alias) clashes should be reported as errors

While trying to fix #9150 I found another class of issues with aliases and name clashes of dependencies.
I am proposing to report the following situation as errors on linting (and template/installing/upgrading) a chart:
dependencies:
- name: foo
  alias: baz # ← baz used twice
  version: 1.0.0
- name: bar
  alias: baz # ← baz used twice
  version: 1.0.0

dependencies:
- name: foo
  alias: bar # ← shadows chart below
  version: 1.0.0
- name: bar
  version: 1.0.0

dependencies:
- name: foo
  version: 1.0.0
- name: foo # ← chart with same name as above (although version or repo will be different, this will not work currently)
  version: 1.2.3

While the last example may be working in the future (see #7413), I can't see any valid use case for the first two examples.
Currently Helm renders templates, resulting in more or less undefined behaviour. Some templates will override each other, others won't, depending on file names in the actual charts and on the chart names itself.
Output of helm version:
version.BuildInfo{Version:"v3.4.2", GitCommit:"23dd3af5e19a02d4f4baa5b2f242645a1a3af629", GitTreeState:"clean", GoVersion:"go1.14.13"}

Thank you for the report, @dastrobu . I agree (1) and (2) can't have any practical use-cases.  (3) may have a valid use case and could be supported in future.
@mattfarina , is the following worth marking for v4 ?
dependencies:
- name: foo
  version: 1.0.0
- name: foo # ← chart with same name as above (although version or repo will be different, this will not work currently)
  version: 1.2.3

Thank you for the report, @dastrobu . I agree (1) and (2) can't have any practical use-cases.  (3) may have a valid use case and could be supported in future.
@mattfarina , is the following worth marking for v4 ?
dependencies:
- name: foo
  version: 1.0.0
- name: foo # ← chart with same name as above (although version or repo will be different, this will not work currently)
  version: 1.2.3

Remove stale.
There is an open PR waiting for approval.

Remove stale.
There is an open PR waiting for approval.

Remove stale.
There is an open PR waiting for approval.

Remove stale.
There is an open PR waiting for approval.

Remove stale.
There is an open PR waiting for approval.

Remove stale.
There is an open PR waiting for approval.

Remove stale.
There is an open PR waiting for approval.

Remove stale.
There is an open PR waiting for approval.


[Bug] can't install on balzor webassembly

version:
13.5.1

.NET Core version

5.0, visual studio 2019 16.11.0

blazor webassembly:

Steps to Reproduce:

Create a new blazor webassembly
Attempt to install latest Electron.NET nuget package

when install on in blarzor webassembly then window error show message "The project references the ASP.NET Core shared framework, which is not supported by Blazor WebAssembly apps. Remove the framework reference if directly referenced, or the package reference that adds the framework reference."

🎉🚀 New Electron.NET version 23.6.1 released 🚀🎉
With native Electron 23 and .NET 6 support. Your problem should be fixed here. If you continue to have the problem, please let us know. Please note the correct updating of your API & CLI. Info in the README. Have fun!


Spring Security OAuth2 Rejects Non-Spring Clients

The Spring OAuth2 sample app on GitHub treats non-Spring clients differently than it treats the Spring ui client in its sample configuration. This means that the sample is non-compliant with the OAuth2 spec in its out-of-the-box form. The problem might be in how the filter chain in front of the token endpoint is configured.  Here are the specifics:

The Spring OAuth2 `ui` client was carefully translated into Node.js to avoid any confusion in re-creating the conditions under which the Spring OAuth2 `ui` sample app works. The new node.js client (whose code can be downloaded from a file sharing site at this link), is able to initiate Step A of the flow by redirecting to the Spring OAuth2 `authserver` app. The `authserver` app is able to authenticate the user and redirect back to the Node.js client in Step C of the flow. However, then the client app then initiates Step D of the flow, the `authserver` app rejects the request as anonymous in what would otherwise be Step E of the flow. Here are the logs in the `nodemon` console:
Finished Step C of Authorization Code Flow. Welcome back from authserver
queryData.code is: Z3B1mb
About to start Step D of Authorization Code Flow.
The POST is finished. Waiting for response.
Step D. Token Request Sent To AuthServer.
JSON data response is: [object Object]
access token is: undefined
error is: unauthorized

To confirm that this is not a client problem, I re-wrote the code shown below using two separate npm packages, including passport.js and client-oauth2. However, both of these well-tested, production-ready npm packages also failed to connect to the Spring OAuth2 sample app's authserver /token endpoint at the same point in the flow. Code for two separate clients that independently reproduce the authserver problem is included in file sharing links below.  However, in addition to the links below, I am also posting here the version of the routing code that isolates at the lowest level the steps that the client takes to try to connect with Spring OAuth2 authserver.
Here is the route.js from the zip file in the link above:
var url = require('url');
var request = require('request');

// expose the routes to our app with module.exports
module.exports = function(app) {

    // application --------------------------------
    app.get('/resource/**', function(req, res) {
        console.log("You Hit The Resource Route TOP. ");
        request.get('http://localhost:9000/resource', function (error, response, body) {
            if(error) {console.log('ERROR with resource request.')}
            if (!error){// && response.statusCode == 200) {
                console.log(response.statusCode);
                console.log(body);
            };
        });
        console.log("You Hit The Resource Route BOTTOM. ");
    });

    app.get('/user**', function(req, res) {
        console.log("You Hit The User Route TOP");
        request.get('http://localhost:9999/uaa/user', function (error, response, body) {
            if(error) {console.log('ERROR with user request.')}
            if (!error){// && response.statusCode == 200)
            {
                console.log(response.statusCode);
                console.log(body);
                console.log("response is: ");
                console.log(response);
                console.log("end of response.");
            };
        });
         console.log("You Hit The User Route BOTTOM");
    });

    app.get('/login', function(req, res) {
        console.log("You Hit The Login Route ");
        console.log("headers are: "); console.log(req.headers);
        console.log("cookies are: "); console.log(req.cookies);
        console.log("params are: "); console.log(req.params);
        console.log("body is: "); console.log(req.body);
        console.log("method is: "); console.log(req.method);
        console.log("query is: "); console.log(req.query);
        console.log("route is: "); console.log(req.route);
        console.log("signedCookies are: "); console.log(req.signedCookies);
        var queryData = url.parse(req.url, true).query;

        if (queryData.code) {
            //This is the end of Step C in the flow
            // "code" was in the GET request, ex: http://localhost:8080/login?code=w6zGJO&state=IuaO63
            console.log('Finished Step C of Authorization Code Flow. Welcome back from authserver');
            console.log('queryData.code is: ' + queryData.code);
            //Now start Step D in the flow
            console.log('About to start Step D of Authorization Code Flow. ');
            request.post('http://localhost:9999/uaa/oauth/token?grant_type=authorization_code&code='queryData.code'&client_id=acme&redirect_uri=http://localhost:8080/login', function(err, response, body){
                // Step E is the response received as follows:
                if(err) {console.log('ERROR with token request.')}

                    var data = JSON.parse(body);
                    console.log('JSON data response is: ' + data);
                    var variable1 = data.access_token;
                    var variable2 = data.token_type;
                    var variable3 = data.error;
                    console.log('access token is: ' + variable1);
                    console.log('error is: ' + variable3);
                    //Do more processing here
                });
            console.log('The POST is finished. Waiting for response.');
            console.log('Step D. Token Request Sent To AuthServer.');
        } else {
            //This is Step A console.log('Step A of Authorization Code flow about to start. ');
          res.redirect('http://localhost:9999/uaa/oauth/authorize?client_id=acme&redirect_uri=http://localhost:8080/login&response_type=code&state=IuaO63'); }
    });

    app.get('*', function(req, res) {
        res.sendfile('./public/index.html'); // load single view file (angular will handle the front-end) }
    );
    };

To recreate the problem in only a few minutes, simply:
1.) download the zipped node.js project, unzip it, type nodemon server.js in its root directory to start it.
2.) Then separately start the Spring Boot authserver app and the resource app which you alreay have.
3.) From that point, you can simply type http://localhost:8080in your web browser, click the /login link in the client app, then follow the redirect and enter username= user and password=password into theauthserver` app and click confirm to recreate the problem.
The authserver debug logs will indicate that the user was authenticated in Step B, but then forgotten and rejected in Step D.  And the client nodemon logs will indicate that the token request was rejected.
This has also been posted as a Stack Overflow question at this link.
**Validating the problem with a second client app:**
The Passport.js version of the code that generates the same identical problem (**proving that this is a Spring OAuth2 `authserver` app problem**) is summarized in a different Stack Overflow Posting, and is also available in complete form in a link below to independently validate the problem on a second app.  But again, I posted the more granular code above so that you can more clearly see what is going on.
To recreate the problem with the second client app that was written with passport.js to confirm for yourself that the problem is with the Spring OAuth2 authserver sample app, follow these steps:
1.) Download the passport.js version of the client at this link, unzip it, and type nodemon server.js in its root folder to launch it.
2.) Point your browser to http://localhost:8080 to launch the client.
3.) Click the link in the browser to "Login with OAuth 2.0 Provider".  This will redirect to the authserver app, where you give username and password, then click approve.  You will then be re-directed back to the client, which will then make another call to authserver, this time to the token endpoint, which will result in the same error as above.
The Spring Boot authserver logs will show that the user authenticated during the Steps A, B, C sequence, but that the user was subsequently treated as anonymous during the Steps D and E sequence.  And the nodemon logs for the client during these transactions will be:
GET /user 200 4.412 ms - 905
GET /resource/ 200 1.634 ms - 905
GET /auth/provider 302 1.573 ms - 0
GET /auth/provider/callback?code=ca0Jhn 500 34.359 ms - 1185
TokenError: Full authentication is required to access this resource
    at OAuth2Strategy.parseErrorResponse (/home/user/nodejs_apps/oauth_client_passport/node_modules/passport-oauth2/lib/strategy.js:320:12)
    at OAuth2Strategy._createOAuthError (/home/user/nodejs_apps/oauth_client_passport/node_modules/passport-oauth2/lib/strategy.js:367:16)
    at /home/user/nodejs_apps/oauth_client_passport/node_modules/passport-oauth2/lib/strategy.js:166:45
    at /home/user/nodejs_apps/oauth_client_passport/node_modules/oauth/lib/oauth2.js:177:18
    at passBackControl (/home/user/nodejs_apps/oauth_client_passport/node_modules/oauth/lib/oauth2.js:123:9)
    at IncomingMessage.<anonymous> (/home/user/nodejs_apps/oauth_client_passport/node_modules/oauth/lib/oauth2.js:143:7)
    at emitNone (events.js:85:20)
    at IncomingMessage.emit (events.js:179:7)
    at endReadableNT (_stream_readable.js:913:12)
    at _combinedTickCallback (internal/process/next_tick.js:74:11)
    at process._tickCallback (internal/process/next_tick.js:98:9)

Both client apps failing prove the problem with with authserver not being able to serve non-Spring clients, and thus authserver not being OAuth2-compliant out of the box.  So how do we fix this?

I can't see anywhere where you authenticate the request to the /token endpoint (so it's unsurprising it comes back as a 401). In the spec it says that clients must authenticate with the auth server (https://tools.ietf.org/html/rfc6749#section-3.2.1). The default in Spring OAuth is HTTP Basic using the client id and secret (it's all in the user guide).

Thank you.  I have been carefully studying both the entire official spec, and your entire written documentation, for some time now.  I think what is missing are cook-book-like instructions for common tasks.  Like, for example, simply showing the contents of a valid HTTP request and response for each step of a flow, in digest form so that it is easier for developers to extract the salient points when they study 50 to 100 page manuals.
I was able to resolve the current problem by using your suggestion.  I hope that this posting helps others who encounter the same problem.


Fix some typos/grammar in dalloc description

Please do squash merge, I used github to edit these, don't need to be 3 commits.


saving player settings crashed when exiting inventory

perhaps this should only be called when formname=rc_form
however I couldn't reproduce this error yet

this is reacreatable if you create a new world and hit "i" and then "i" before doing anything else.

this is a bad sign for testing because it means to properly test a new empty world needs to be created each time.


⚠ Garbage Collector has degraded performance

In c406344, Garbage Collector (https://gc.fef.moe/login) experienced degraded performance:

HTTP code: 200
Response time: 20978 ms

Resolved: Garbage Collector performance has improved in 3ae58e0.


Fixed schemaTool problems.

The develop branch is currently broken after a recent merge. This PR fixes it.

Grumble grumble github merge model grumble grumble

Yeah, it's a pain. Feel free to +1 and merge this once travis finishes.


FakeOwner.registry.resolve is deprecated

Calling FakeOwner.resolveRegistration hits a deprecation warning
Using ApplicationInstance.registry.resolve is deprecated. Please use ApplicationInstance.resolveRegistration instead.

@kamal - Thanks for reporting, can you provide a demo ember-twiddle (they have addon support now)?

Ping @kamal?

Closing for now, please share a reproduction and I'm happy to jump on this....


kubernetes nightly test fails

Description
kubernetes automated test failed.
test can not determine the number on ready nodes
TestK8s/kubernetes 2023-11-16T01:00:09Z logger.go:66: "\n[Interface]\nAddress = <IP_ADDRESS>\nPrivateKey = 8MGbYxFxU06JHKBbkRmtEaYzCBeWO/vAUGfuQ6AFOVg=\n[Peer]\nPublicKey = 0coWX2G3QClsViRpVRxUy+0rNoL52XbNhovUvRLcHjw=\nAllowedIPs = <IP_ADDRESS>/16, <IP_ADDRESS>/16\nPersistentKeepalive = 25\nEndpoint = <IP_ADDRESS>:5061\n\t"
TestK8s/kubernetes 2023-11-16T01:00:11Z retry.go:91: terraform [output -no-color -json ygg_ip]
TestK8s/kubernetes 2023-11-16T01:00:11Z logger.go:66: Running command terraform with args [output -no-color -json ygg_ip]
TestK8s/kubernetes 2023-11-16T01:00:11Z logger.go:66: "300:487a:7ea1:5b4e:a0d1:83d1:3f8d:69"
    k8s_test.go:77:
        	Error Trace:	/home/runner/work/terraform-provider-grid/terraform-provider-grid/integrationtests/k8s_test.go:23
        	            				/home/runner/work/terraform-provider-grid/terraform-provider-grid/integrationtests/k8s_test.go:77
        	Error:      	Should be empty, but was could not excute command on remote with output W1116 01:00:12.837132     251 loader.go:222] Config not found: /etc/rancher/k3s/k3s.yaml
        	            	E1116 01:00:12.848956     251 memcache.go:238] couldn't get current server API group list: Get "http://localhost:8080/api?timeout=32s": dial tcp <IP_ADDRESS>:8080: connect: connection refused
        	            	E1116 01:00:12.850910     251 memcache.go:238] couldn't get current server API group list: Get "http://localhost:8080/api?timeout=32s": dial tcp <IP_ADDRESS>:8080: connect: connection refused
        	            	E1116 01:00:12.851863     251 memcache.go:238] couldn't get current server API group list: Get "http://localhost:8080/api?timeout=32s": dial tcp <IP_ADDRESS>:8080: connect: connection refused
        	            	E1116 01:00:12.852827     251 memcache.go:238] couldn't get current server API group list: Get "http://localhost:8080/api?timeout=32s": dial tcp <IP_ADDRESS>:8080: connect: connection refused
        	            	E1116 01:00:12.854653     251 memcache.go:238] couldn't get current server API group list: Get "http://localhost:8080/api?timeout=32s": dial tcp <IP_ADDRESS>:8080: connect: connection refused
        	            	The connection to the server localhost:8080 was refused - did you specify the right host or port?
        	            	: Process exited with status 1
        	Test:       	TestK8s/kubernetes
    k8s_test.go:77:
        	Error Trace:	/home/runner/work/terraform-provider-grid/terraform-provider-grid/integrationtests/k8s_test.go:27
        	            				/home/runner/work/terraform-provider-grid/terraform-provider-grid/integrationtests/k8s_test.go:77
        	Error:      	Should be true
        	Test:       	TestK8s/kubernetes
        	Messages:   	number of ready nodes is not equal to number of nodes only %!s(int=0) nodes are ready

working on this

Investigation and Solution:
The problem now is that there's a problem with Yggdrasil network, and we can't use it to check the connection with the deployed cluster, and I couldn't get it to work and others had the same problem, so this issue is blocked until Yggdrasil issue is done
TestK8s/kubernetes 2023-12-13T01:16:14Z retry.go:91: terraform [output -no-color -json wg_config]
TestK8s/kubernetes 2023-12-13T01:16:14Z logger.go:66: Running command terraform with args [output -no-color -json wg_config]
TestK8s/kubernetes 2023-12-13T01:16:14Z logger.go:66: "\n[Interface]\nAddress = <IP_ADDRESS>\nPrivateKey = ePw4ffcRYGHKDlldAmOKugBMQVYTFqb9LAywBQXVW1o=\n[Peer]\nPublicKey = 5YvWQkRVs1qV0JQ4T9FK4dlsLFO7NYzr3xis+zirkgo=\nAllowedIPs = <IP_ADDRESS>/16, <IP_ADDRESS>/16\nPersistentKeepalive = 25\nEndpoint = <IP_ADDRESS>:7234\n\t"
    k8s_test.go:71:
        	Error Trace:	/home/runner/work/terraform-provider-grid/terraform-provider-grid/integrationtests/k8s_test.go:71
        	Error:      	Should be true
        	Test:       	TestK8s/kubernetes
    k8s_test.go:74:
        	Error Trace:	/home/runner/work/terraform-provider-grid/terraform-provider-grid/integrationtests/k8s_test.go:74
        	Error:      	Should be true
        	Test:       	TestK8s/kubernetes
TestK8s/kubernetes 2023-12-13T01:22:15Z retry.go:91: terraform [output -no-color -json ygg_ip]
TestK8s/kubernetes 2023-12-13T01:22:15Z logger.go:66: Running command terraform with args [output -no-color -json ygg_ip]
TestK8s/kubernetes 2023-12-13T01:22:15Z logger.go:66: "300:68f9:dc9f:d10e:c0a2:ceeb:8ac1:e137"
    k8s_test.go:77:
        	Error Trace:	/home/runner/work/terraform-provider-grid/terraform-provider-grid/integrationtests/k8s_test.go:23
        	            				/home/runner/work/terraform-provider-grid/terraform-provider-grid/integrationtests/k8s_test.go:77
        	Error:      	Should be empty, but was could not start ssh connection: dial tcp [300:68f9:dc9f:d10e:c0a2:ceeb:8ac1:e137]:22: connect: connection timed out
        	Test:       	TestK8s/kubernetes
    k8s_test.go:77:
        	Error Trace:	/home/runner/work/terraform-provider-grid/terraform-provider-grid/integrationtests/k8s_test.go:27
        	            				/home/runner/work/terraform-provider-grid/terraform-provider-grid/integrationtests/k8s_test.go:77
        	Error:      	Should be true
        	Test:       	TestK8s/kubernetes
        	Messages:   	number of ready nodes is not equal to number of nodes only %!s(int=0) nodes are ready

Blocked by scheduler issue
Details: https://github.com/threefoldtech/terraform-provider-grid/pull/808#issuecomment-1859136734

Work Done:

updated the tests to run in freefarm and fixed them, all is passing

--- PASS: TestK8s (169.15s)
    --- PASS: TestK8s/kubernetes (121.03s)
    --- PASS: TestK8s/k8s_invalid_names (48.11s)
    --- SKIP: TestK8s/k8s_using_module (0.00s)
PASS
ok      github.com/threefoldtech/terraform-provider-grid/integrationtests       169.187s
➜  terraform-provider-grid git:(development_fix_k8s_nightly_test) ✗

Verification steps

tried TestK8s test and it passed (k8s_using_module skipped https://github.com/threefoldtech/terraform-provider-grid/issues/770)

--- PASS: TestK8s (173.97s)
    --- PASS: TestK8s/kubernetes (125.88s)
    --- PASS: TestK8s/k8s_invalid_names (48.06s)
    --- SKIP: TestK8s/k8s_using_module (0.00s)
PASS
ok      github.com/threefoldtech/terraform-provider-grid/integrationtests


ENH: increment testing ds version + hash

Reference issue
ENH: Follow up to the PR in mne-testing-repo and the new testing dataset release.
What does this implement/fix?
Incrementing testing dataset version to 0.150 and setting the new hash.

Huh, oh right, maybe I messed up the order. Shall I PR the read_raw_neuralynx() first or rebase on top of this branch?

You can just push commits here to fix the reader and increment the testing version

That would work too

Closing for #11969


Fix spacy_stanza.md example

Change Description
The example was missing the super call in the constructor. Thus the example will fail as the super constructor initialized the ner config.
Issue reference
This PR fixes issue #XX
Checklist

[ ] I have reviewed the contribution guidelines
[ ] I have signed the CLA (if required)
[ ] My code includes unit tests
[ ] All unit tests and lint checks pass locally
[X] My PR contains documentation updates / additions if required

Hi, I'm not sure that passing the loaded spacy model into the parent class init is the right way to go.
Looking at the init for SpacyNlpEngine, it accepts a models dict and an ner_model_configuration
https://github.com/microsoft/presidio/blob/a6e253d0e7846c761144aa172844ddd6016a3970/presidio-analyzer/presidio_analyzer/nlp_engine/spacy_nlp_engine.py#L29
The models are already loaded, so no need to pass them. If a user requires a different NerModelConfiguration, then this should be passed to the super.

Closing, please re-open if you'd like to continue working on this.

@omri374 don't have rights to re-open PRs in this repo.
Did remove the argument in my origin branch. But I think it is not updating closed MRs.

Reopened


Fix bcrypt hard limit on passwords to 71 bytes

Despite the original Bcrypt paper claiming passwords have a maximum of 56 bytes, some implementations are compatible to up to 72 bytes; Ruby's BCrypt gem doesn't even enforce any limit.
I don't have enough knowledge to verify the claim, yet, increasing the limit to 72 —including the leading null byte, thus allowing a 71 bytes password— doesn't break the current digests. Please test and report!
closes #5353

I think it would be worth adding a note about the length limit in the docs as the gist from this StackExchange  answer.

Can we merge this or add a comment?


option to skip the first copy (only watch)

Hi, nice package.
For the cpx.watch, it would be nice to be able to skip the first copy (if files are already in sync.) Maybe add an option to the watch command (nocopy)?

Thank you for this issue. I'm sorry for delay.
Interesting.
A new option of watch command sounds reasonable.
Maybe:

CLI: --no-initial (which set false to options.initialCopy)
Node API: options.initialCopy (default: true)

Thanks, sounds great.
I would personally implement the default the other way:

CLI: --initial (which set true to options.initialCopy )
Node API: options.initialCopy (default: false)
but that is a matter of taste and I can image you maybe do not want to break backwards compatibility.

I released v1.5.0 with --no-initial option.
Sorry, I didn't want to make a breaking change for now.

Great. no problem.
Many thanks.


The code sandbox example for Analytics is down.

Specfically this link: https://codesandbox.io/s/solid-create-analytics?file=/src/index.tsx

I think the analytics package isn't finished yet.

Yes I saw now, sorry about that! Maybe a quick comment about "stages" would help?

"Stages" are also a work in progress :D
But I understand it that way: Packages marked as 0 or 1, aren't actually done yet, maybe they are published on npm, maybe not. 2 is finished and published, but is still pretty experimental, might change, be removed, etc. And the 3 up are the packages that have proven themselves, are stable, well tested, etc. At least in theory.


VS2 +Eureka incompatible with connector extras

Describe the bug
mixin conflict with reach entity attributes
Steps to reproduce
1.add the mods required to boot vs2, eureka, connector + extras
2. throw in create big cannons just to be safe
3. crash
...
Logs
https://gist.github.com/Xplodin/40305731d05955764ce79db7768ad308
Additional context
I believe both mods have a conflict with reach entity attributes
Looked at both mod's code

Actually this is VS2 and Eureka with connector.
Mod could just be broken rn. not connector or extras fault here

seems to be embeddium I believe


improve log output format

Ⅰ. Describe what this PR did
Just Improve log output format of file registry/nacos/address/address_discovery.go
Ⅱ. Does this pull request fix one issue?

None
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

LGTM


Inspect container log spam

While testing on GCE today I saw the worker spam the logs with this message:
Apr 17 20:51:39 funnel-worker dockerd[2359]: time="2017-04-17T20:51:39.763366662Z" level=error msg="Handler for GET /v1.24/containers/b3qijhvntf6gcu8gcd2g-0/json returned error: No such container: b3qijhvntf6gcu8gcd2g-0"

The task did run successfully, but I'm guessing that while the container was starting these messages were spamming.

Leaving this open to remind me to check this manually on GCE

Confirmed on exastack.


Docker attestation generation fails

Docker job fails on attestation generation, see job - https://github.com/UpstreamDataInc/goosebit/actions/runs/10710434164/job/29697391423
Might be unrelated, but trying to pull the image after this failed run results in the error
Error response from daemon: manifest for upstreamdata/goosebit:latest not found: manifest unknown: manifest unknown

Now the part that we didn’t/couldn‘t test is failing 😞

Don't worry about it lol, we can release a bunch of bug fix versions for now, just wanted to get versions out to test this stuff.


Explicitly define resource namespaces in Helm chart

This is very useful in combination with Helm template.

/retest


docker2fl stack overflow

got this error when try to convert  from docker to .fl on Linux
#./target/debug/docker2fl  -i  redis -s "dir:///tmp/store0"
2024-10-16 03:56:18,173 INFO [docker2fl::docker2fl] get docker info
2024-10-16 03:56:18,173 INFO [docker2fl::docker2fl] docker image 'redis:latest' is being extracted
2024-10-16 03:56:18,173 INFO [docker2fl::docker2fl] pulling docker image redis:latest
2024-10-16 03:56:27,520 INFO [docker2fl::docker2fl] Creating '.startup.toml' file from container 0c7db154-b52a-4256-9499-753139694eb3 contains [startup.entry]
name = "core.system"

[startup.entry.args]
args = [""]
dir = "/data"
name = "docker-entrypoint.sh"

[startup.entry.args.env]
GOSU_VERSION = "1.17"
PATH = "/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"
REDIS_DOWNLOAD_SHA = "bc34b878eb89421bbfca6fa78752343bf37af312a09eb0fae47c9575977dfaa2"
REDIS_DOWNLOAD_URL = "http://download.redis.io/releases/redis-7.4.1.tar.gz"
REDIS_VERSION = "7.4.1"

2024-10-16 03:56:27,520 INFO [docker2fl::docker2fl] docker image 'redis:latest' is extracted successfully
2024-10-16 03:56:27,520 INFO [docker2fl::docker2fl] packing flist
2024-10-16 03:56:27,520 INFO [rfs::pack] packing root:
2024-10-16 03:56:27,531 INFO [rfs::pack] create metadata
2024-10-16 03:56:27,531 INFO [rfs::pack] create pack list
2024-10-16 03:56:27,531 INFO [rfs::pack] create failures list
2024-10-16 03:56:27,531 INFO [rfs::pack] create uploader
2024-10-16 03:56:27,532 INFO [rfs::pack] create worker pool

thread 'main' has overflowed its stack
fatal runtime error: stack overflow
Aborted

notes:

some logs above are added by me
i've tried to increase the stack size using RUST_MIN_STACK=536870912  (modified the value until 4GB)
tried to increase stack size using ulimit -s until 256MB
using latest master, commit 4611ed98dfb1acf54d9f2733e1591e9ed845829e

fixed by #80


Verify secrets and EnvFrom in addition to Env

Based off of: https://github.com/1Password/onepassword-operator/pull/74
Resolves: #73
Adds support for provisioning variables through kubernetes secrets set in the EnvFrom field of a container.

Tested manually and it works as expected. Example deployment spec:
apiVersion: apps/v1
kind: Deployment
spec:
  template:
    spec:
      containers:
      - name: [...]
        image: [...]
        envFrom:
          - secretRef:
              name: example-secret

Closing because of the unnecessary checks this PR introduces, merging the base MR #74  instead.


Add system dependencies for new counterpart signature PDF script

https://trello.com/c/4Su1kLUN/397-update-apt-get-dependencies-for-e-signature-counterpart-agreement-script
We need the official build of wkhtmltopdf with patched QT to be able to get page numbers working in the latest agreement PDF format. Also, to make sure the fonts are consistent across the document we need Microsoft core fonts installed.

I think we may need to do some extra steps to accept the Microsoft EULA i.e. https://askubuntu.com/a/25614/234184 ?

I think we may need to do some extra steps to accept the Microsoft EULA i.e. https://askubuntu.com/a/25614/234184 ?

Ah good point.
There's an example of how to do it in ansible here https://github.com/pludoni/ansible-wkhtmltox/blob/master/tasks/font.yml

I think we may need to do some extra steps to accept the Microsoft EULA i.e. https://askubuntu.com/a/25614/234184 ?

Added.


IRGen: disable-llvm-optzns should disable the objc-arc-contract pass

Don't run the objc-arc-contract pass when disable-llvm-optzns is passed.
In pipeline setups where the swift-frontend is called twice;

the first time to create bitcode
the second time to generate object code from the bitcode (using
disable-llvm-optzns)
we don't want to run the objc-arc-contract pass twice.

rdar://91908312

@swift-ci Please test

@swift-ci Please test


Export PageViewport class?

I've noticed that the PageViewport class is only exported as a type.
I was wondering if it would be possible to export the class as a value, so consumers of the library can use it
Why? I have a PageManager class that handles which pages should be rendered, etc.
It stores a viewport for the current page width/height.
In my tests, I'd like to pass in a fake viewport, but currently I cannot do this because    I cannot use PageViewport as a value.

I was wondering if it would be possible to export the class as a value, so consumers of the library can use it

While it'd be technically possible, it's unfortunately not a desirable change from the PDF.js library perspective.
Note that while PageViewport-instances are returned by the PDFPageProxy.getViewport-method in the API, that class is not intended to be initialized manually by users. We purposely don't want to expose everything in the public API, since that makes both maintenance and any future changes of the affected code so much more difficult.

In my tests, I'd like to pass in a fake viewport, but currently I cannot do this because I cannot use PageViewport as a value.

It's possible that there's ways to work-around that, but given that you've not provided any details about exactly what you're trying to do we cannot really help here.
For future reference: When opening an issue, please provide all of the information requested in https://github.com/mozilla/pdf.js/blob/master/.github/ISSUE_TEMPLATE.md such that the issue becomes easily actionable.
Furthermore, since it seems very relevant here, please see https://github.com/mozilla/pdf.js/blob/master/.github/CONTRIBUTING.md (emphasis mine):

If you are developing a custom solution, first check the examples at https://github.com/mozilla/pdf.js#learning and search existing issues. If this does not help, please prepare a short well-documented example that demonstrates the problem and make it accessible online on your website, JS Bin, GitHub, etc. before opening a new issue or contacting us in the Matrix room -- keep in mind that just code snippets won't help us troubleshoot the problem.


changes based on engg feedback.

changes based on engg feedback.

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks


Bug in R square calculation fixed

When results are sorted, prediction array and data array are not identical anymore, so R square was not being calculating exactly. I modified script to keep data before sorting and use that
unsorted list in R2 calculation.

If Y value of chart is 0(zero) it was not included in R square calculation while it should. This is fixed in https://github.com/canyildiz/highcharts-regression/commit/0d12ce1314d6a82456e673a23dd23c3cfb2ff7df.


WIP : added firebase logger

Description
This PR adds firebase-logger to the project. The firebase project was added to BearStudio's  set of FB project
Addresses #21
Resources

https://github.com/BearStudio/firebase-logger#firebase-sdk

Even after upgrading to firebase-logger version 2.0.0 I am still getting the same error upon running yarn android

I've tried solutions discussed on following topics, but it didn't fix them :

https://github.com/facebook/react-native/issues/26687
https://nkaushik.com/javascript/react-native-invariant-violation-appregistry-not-registered/

I've yet to try the one discussed here : https://stackoverflow.com/questions/34969858/react-native-module-appregistry-is-not-a-registered-callable-module


Emmet 2.0: wrap lines with list broken

VSCode Version: Code - Insiders 1.15.0-insider (f39385b8326671dc6389f21f8c2e2f6afdb352a1, 2017-08-01T05:07:59.267Z)
OS Version: Darwin x64 16.7.0
Extensions:

Extension
Author (truncated)
Version

ng-template
Ang
0.1.4

code-settings-sync
Sha
2.8.2

sort-lines
Tyr
1.3.0

html-snippets
abu
0.1.0

Bookmarks
ale
0.15.2

project-manager
ale
0.18.1

path-intellisense
chr
1.4.2

vscode-eslint
dba
1.2.11

tslint
eg2
0.17.0

auto-rename-tag
for
0.0.14

Angular2
joh
2.4.0

ecdc
mit
0.10.3

HTMLHint
mka
0.3.3

vscode-autoprefixer
mrm
2.1.1

vscode-stylefmt
mrm
2.5.0

angular2-inline
nat
0.0.17

stylelint
shi
0.28.0

change-case
wma
1.0.0

(4 theme extensions excluded)

Steps to Reproduce:

create a bunch of lines that you want to be a bulleted list, or paste a bulleted text list like I did, in this format:

• lorem ipsum
• lorem ipsum
• lorem ipsum

select the block and use the "Emmet: Wrap with abbreviation" command
Issue the command ul>li*|t
you get an error:

Failed to expand abbreviation
e.doShow @ /Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:28

This command, in Emmet 1.0, would wrap the lines with a list and remove the bullets, as in

lorem ipsum

lorem ipsum

lorem ipsum

What filter is the |t at the end of your abbreviation? Is there any documentation on that?

Ah! Never mind. Found it here: https://docs.emmet.io/actions/wrap-with-abbreviation/

wow.. I just scrolled down that page (https://docs.emmet.io/actions/wrap-with-abbreviation/) didn't know the wrapping feature has so many more features. I just thought the first demo was the one!
It sure is complicated to implement all that in the current model.
@sergeche The wrap with abbreviation feature is not yet in the new atom plugin in.... Do you plan to implement it keeping parity with current set of features?

Will implement it for sure but first I have to experiment with better UI/UX for it to provide instant feedback on wrapped result

All the Wrap with Abbreviation features are already implemented in Emmet 2

I know you have the support via the text option in the expand method from the expand-abbreviation module, but its not surfaced as a command in the atom beta plugin correct?

No command yet because of absent UI/UX concept

The main issue here is that the Wrap command doesn't make use of implicit repeaters (* without the number) and so individual lines do not get wrapped.
Logged https://github.com/Microsoft/vscode/issues/31893 for the trimming part of this issue.
With https://github.com/Microsoft/vscode/commit/211ffdf184329179ddca9c1fe993f5f11f3c4dda there will be a new command Emmet: Wrap individual lines with abbreviation that can be used for https://docs.emmet.io/actions/wrap-with-abbreviation/#wrapping-individual-lines
Reason for creating new command instead of re-using existing wrap command is that the Wrap with Abbreviation command makes use of $TM_SELECTED_TEXT and does not really pass the individual lines to the expand module.
We make use $TM_SELECTED_TEXT so as to take advantage of all the indenting support we get out of the box for snippets.

And @smlombardi, nice find!

I use that Emmet feature all the time to take lists from manuscript and make html lists out of them quickly.  Without this feature working, I have to open the file in Sublime just to do this.  Still faster than any other way of making a list.

It works, except the |t does not work.

https://github.com/Microsoft/vscode/issues/31893 has been logged for the |t part of the issue

Note to verifier:
Use the new command Emmet: Wrap individual lines with abbreviation and verify it works as described in https://docs.emmet.io/actions/wrap-with-abbreviation/#wrapping-individual-lines

As I mentioned earlier, filters are deprecated in favour of expand options. In this case, you have to simply remove list markers in lines before sending it to expand abbreviation function


CDB_TableMetadataTouch wrong parameter?

This table code seems to be invoking CDB_TableMetadataTouch with an id instead of a name:
  def update_cdb_tablemetadata
    owner.in_database(as: :superuser).run(%Q{
      SELECT<EMAIL_ADDRESS>    })
  end

I'm assigning this to you, @rafatower , because I think you have a better view on this, and maybe you can tell what this might imply (performance? invalidation?, and if it's wrong at all), but please let me know if you want me to fix it.
PD: spotted with this staging error, although, since number is inside quotes, it looks more like a DB extension problem than the problem of being an id instead of a name. cc @zenitraM

dunno if table_id is an oid, but if that is the case the conversion is right:
select 'tm_world_borders_s_11'::regclass::oid::regclass
-[ RECORD 1 ]-------------------
regclass | tm_world_borders_s_11

Closing after talking to @rochoa about this.

Just in case I added some tests in https://github.com/CartoDB/cartodb-postgresql/commit/7f58e1f690bdb2e83a58662338ed6a685fae1c17


Migrate DepositFeed spec to forge test contract

This is nice because:

it's consistent with the existing contract testing
smaller test spec - easier to read and maintain
no need for the DummyContract to simulate contract calls

Codecov Report

Merging #304 (1f02e0a) into main (95834ed) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #304   +/-   ##
=======================================
  Coverage   55.51%   55.51%
=======================================
  Files          43       43
  Lines        3116     3116
=======================================
  Hits         1730     1730
  Misses       1196     1196
  Partials      190      190

Continue to review full report at Codecov.

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


Check out shell scripts with unix line endings, even on windows.

This allows running ./hack/update-deps.sh on Windows if you have dep and go on your PATH, and you have GOPATH set.

/lgtm
/approve


Add right click popup options in Connections to Method dialog.

Connections to method is quite baron in it’s current read-only form, lacking quick access to useful connection functionality. In this pull request I have added the following right click options:

Edit…
Show Node in Tree
Disconnect

I decided to preserve the ordering found in the signals tab for consistency. However, I did switch out “Go to Method” in lieu of the “Show Node in Tree” option because we had to come from the method to open this dialog.

Show Node in Tree goes to the appropriate node for both source and target, and signal defaults to using the source node.
Closes godotengine/godot-proposals#1849
Closes godotengine/godot-proposals#10840

I think it would be better to have 2 separate "Go to Source Node" and "Go to Target Node" options instead of the same item changing behavior depending on where the user clicks. The discover ability would be better that way IMO.
Great addition anyways!

@JoNax97 Thank you for the comment. I hadn't thought of that, but it is definitely the more intuitive approach. I went ahead pushed those changes.

@Lazy-Rabbit-2001 That's a good idea! I just pushed those changes. Thanks for the suggestion!


FontSelector should also impact the openFontCategory.

The openFontCategory should be exclusive to a radio font number (FontSelector) to have a better UI experience. When a user opens a font category and then changes the font number of choice, the open category should of the previous font number should close. If for the new font number, the user already had opened a category, this category should open back.
The same should be happening when a user is using the search bar. Its content should be tied to the font number so that when the user changes the font number, the search goes back to empty (or its previous state if it was already used before).

Done #6.


Tile width

Hi
We have been looking to add this to our application, but the tiles are showing very narrow. See attached.
Are you able to advise on what may be causing this?

We have been using this plugin for a time now.
This week we just updated Apex from 19.x to 21.x and this problem started happening here too.

We managed to work around it by hard coding the width in pixels into the dashboard CSS (in the plug-in config page), but it's not ideal and doesn't work with all screen widths. It would be good to get a proper fix, but unfortunately, I'm not a front-end developer...

We have been using this plugin for a time now. This week we just updated Apex from 19.x to 21.x and this problem started happening here too.

Did you manage to fix this? I have another issue now where I want to remove the "Click Here>" at the bottom of each tile. Seems like this isn't being supported any longer...

Same issue here

We managed to work around it by hard coding the width in pixels into the dashboard CSS (in the plug-in config page), but it's not ideal and doesn't work with all screen widths. It would be good to get a proper fix, but unfortunately, I'm not a front-end developer...

Could you show where you did this, as I would really like to use this in a project I am working on

That's the best I could figure out, sorry. I haven't been able to do any more than that.

Note:-In the Latest Verison Please use below CSS code in your page.
.apex-row {
     display: block;
     flex-wrap: wrap;
}


update supervisord image to use v0.2.0

What is the purpose of this change? What does it change?
this pr will update supervisord image to use v0.2.0
which will fix issue #840
Was the change discussed in an issue?
#840
How to test changes?
build this pr or download binary from the  AppVeyor build artifacts here and try deploying the example code mentioned in issue #840
and validate if the stderr logs are being shown on ./odo logs

Let's merge this :tada:


Add emscripten support

Related issue: https://github.com/CartoDB/mobile-sdk/issues/510

Install Emscripten (3.1.17)
Change src/embind/embind.js with https://github.com/bugra9/emscripten/blob/embind-overloading-support/src/embind/embind.js
Build mobile swig https://github.com/bugra9/mobile-swig/tree/emscripten-support
cd scripts
python swigpp-emscripten.py
python build-emscripten.py
cd ..
./scripts/emscripten-dev/server/server-localhost
Visit http://localhost:8000 with your browser

@mtehver, I implemented your suggestions and updated the demo site. Additionally, the listeners now work.


Crash when user dictionary contains Date

The experiment client crashes encoding JSON (on this line https://github.com/amplitude/experiment-ios-client/blob/main/Sources/Experiment/ExperimentClient.swift#L482) if the user dictionary contains a date:
*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: 'Invalid type in JSON write (__NSTaggedDate)'
*** First throw call stack:
(0x194afa69c 0x18cd8fc80 0x1939f2c70 0x1939f25c8 0x18e1251b0 0x18dea0ba0 0x1939f20c4 0x1939f25c8 0x18e1251b0 0x18dea0ba0 0x1939f20c4 0x193a26074 0x193a25f1c 0x10162c928 0x10162a1d4 0x101625e88 0x1015f8258 0x10cb9cf50 0x10cb9eb34 0x10cba698c 0x10cba7728 0x10cbb45f8 0x10cbb3c2c 0x1fea33964 0x1fea33a04)
libc++abi: terminating due to uncaught exception of type NSException

@JanGorman, thanks for raising this issue. The date property should be passed as a string to avoid this issue.

That's what we ended up doing of course but it comes rather surprising and it's never cool if a third party crashes your code.

My team is experiencing this issue as well. Has a fix for this been prioritized within Amplitude @tyiuhc?

@JanGorman @daltonclaybrook, thank you for your patience on this. A fix for this is currently being worked on.

@JanGorman @daltonclaybrook, this issue has been addressed in v1.13.5, please let us know if it is not behaving as expected, thanks!


Fix intro to emphasize multiple devices and contributing your own

Hi @ivorb, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


Problem with subject change in Reported Content

When you chose subject in Reported Content - first page is OK but when you click load more then you will have reports from all subjects.
Video:
https://user-images.githubusercontent.com/61503032/104201239-d2aede80-5429-11eb-8ccc-03f5460e25c4.mp4

Thanks, good catch!

Thanks, good catch!


Allow migration of specific organization (not all)

Current work is being done to provide a mechanism to migrate all organizations from a single package into one project, as multiple products. I would like the option to migrate ONLY a specific organization.
This would facilitate leaving information contained in unnecessary organizations behind, streamlining the migration.

It might be easier to handle this outside of the tool by simply renaming the files for the organizations that are to be ignored.


Refactor sqlsync react

This PR refactors sqlsync-react and sqlsync-worker in order to move most of the generic code out of sqlsync-react and into sqlsync-worker. The goal of this PR is to make it easier to build more framework specific libraries such as support for SolidJS (see #37).

@matthewgapp can you take a look at this PR and let me know if it looks ok to you? Ideally it dramatically simplifies the SolidJS PR.
I haven't finished integrating this with demo/frontend yet so it's still marked as a draft. Will finish that up tomorrow along with any comments you have.

Ok fixing the frontend wasn't very difficult. Will finish this up tomorrow.

@matthewgapp can you take a look at this PR and let me know if it looks ok to you? Ideally it dramatically simplifies the SolidJS PR.
I haven't finished integrating this with demo/frontend yet so it's still marked as a draft. Will finish that up tomorrow along with any comments you have.

Looks gr8! I rebased my PR onto this branch and tested it. Seems to be working!


Get more than 10 parameters

It seems aws-ssm get only 10 parameters using directory when it syncs, and I can't find any configuration to make it get more than 10 parameters for a single secret with directory configuration.
I want to request one of

Get many params at once using MaxItems
Get all params using pagination.

and this may not be needed

Add an annotation to set parameter count to get.

It seems https://github.com/cmattoon/aws-ssm/pull/30 this PR solves this issue.


Different theme location in Linux

In Linux it should be ${XDG_DATA_HOME:-$HOME/.local/share}/warp-terminal/themes/ and not ~/.warp/themes/ according to https://docs.warp.dev/appearance/custom-themes

Thanks, I gave this a try yesterday and meant to update the instructions, but wasn't able to update the README yet, i'd appreciate a PR.

created https://github.com/catppuccin/warp/pull/6

Forgot to close via #6


ActiveSetQuadratic with weights and atoms passed directly

The constructor of ActiveSetQuadratic is a bit of a hassle, there should be a way to separate the preparation of weights, atoms, and x, and all the rest of the quantities tracked down. Something like reset_quadratic_terms!

Absolutely true, will fix this! Btw, I never really understood the necessity of having two constructors for ActiveSet, namely, one with and one without types. I'd be tempted to have one call the other, anything speaking against this?


Sysinternals as example/community repo

Hi Crescendo Team !
Every admin in the windows world loves sysinternals. If we would have Powershell Crescendo commandlets around sysinternal command, this may give Crescendo a massive push in adaption.
Would be great if the powershell team could host a repo with a few examples and the community may contribute to a PSsysinternals module based on Crescendo.
To be discussed/R.

Thank you @RStadlmair for the suggestion!  Crescendo is a framework to help accelerate the process of creating cmdlets for cli commands. It is my hope that the community will use Crescendo for Sysinternals or any commands that they want a cmdlet experience with structured data (Objects).  While we aren't focused on releasing a specific module for SysInternals (which we all love), I would certainly encourage you to use Crescendo to build one.  Maybe even share it on the PowerShell Gallery with the rest of the community.

@adamdriscoll has started one https://github.com/adamdriscoll/sysinternals


[Feature request] Add a toggle for vertical tab

I think it would be nice if users can choose which tab design they like the most, and add a vertical version tab like the Notes app on Mac Os

I completely agree, please take a look at this issue I created.
https://github.com/JasonStein/Notepads/issues/688

I also agree


Some PDF documents not correctly rendered

Hi!
Here you are a short list of some documents that are not correctly rendered:

Stressed characters not rendered: http://serverdedicati.aruba.it/contratti/propContr_WebfarServices_005.pdf
Unreadable document: http://serverdedicati.aruba.it/contratti/CGC_Webfarm_Services.pdf

I also have on my PC a PDF that was generated with LateX, in a very standard environment (using a standard template for scientific papers) which is poorely rendered and also crashes my Firefox when I turn the pages. I have no idea about how to attach it here. Can I send it to you by email? (in case you're interested!)

Crashing Firefox??? What version of Firefox/Operation system is that?

Firefox 7.0.1 on Windows 7 Professional 64bit (but Firefox is 32bit).
I send you the document by email right now ;-)

I've sent you the email. I report here the text, so other developers can see:
HOW TO CRASH FIREFOX:

copy the attached PDF to the "web" folder and rename it to "compressed.tracemonkey-pldi-09" (I'm too lazy to change the kDefaultURL variable :D )
Just open web/viewer.html and you'll see that the document is loaded (although the font is badly rendered).
Turn some pages, waiting about 1 second between one and the other....at about page 5, my Firefox crashes. If I turn pages waiting more (for example 3 seconds) it doesn't crash... So it seems related to the turning speed....but you know, users often press "page down" quite fast, so it should not crash ;-)

I think that it's related to memory allocation. I opened the task manager and monitored RAM usage....after every page turning, it increases by several tens of megabytes and then, after few seconds, it decreases. But if I turn page fast, it increases faster than it's able to decrease (after loading the PDF, Firefox was at 90MB....I started turning page quite fast and at page 6 it got to 400MB and then crashed!).

@lucaferrario, are those documents still rendered incorrectly?

The issue should be resolved with the latest version.  If I overlooked anything please leave a comment and we'll re-open the issue.


Added var<mat> implementation of log_determinant_spd (Issue #2101)

Summary
Added var<mat> implementation of log_determinant_spd
Release notes
Added var<mat> implementation of log_determinant_spd
Checklist

[x] Math issue #2101

[x] Copyright holder: Columbia University
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[x] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

Ready for review

@SteveBronder this is ready when tests passes. Took me two tries to get the make_callback_var right :/.

@SteveBronder this should be good to go. Only changes I did since your last review was swapping to make_callback_var


Order Orgs in filter alphabetically

The org list in the filter is now unordered, making it hard to find anything without searching and generally looks a bit messy.
Organize these alphabetically in the BFF

Should we close this or set it to test @allinox ?

Nah I checked it now after prod and works really well, so closing it seems fine 😊


Support libraries in decomp.me

This MR introduces libraries in decomp.me. This is a new section under "Options" that allow adding libraries to the scratch, which allows the user to #include the headers provided by that lib.
Fixes #842
Example:

This is ready for review.

There's one mypy error that I just can't figure out for the life of me. If someone can find a fix, I'd be grateful.
Otherwise, this feature is ready for review.

I think the library flag(s) that get appended should always start with a space

I think the library flag(s) that get appended should always start with a space

Nice catch, I added a space, and also moved the libraries to be within sandbox.quote_options


Useful debugging features

OK this PR adds a bash and a powershell script for replace dotnet core nugget cache dlls with
the ones built by the ControlCatalog.NetCore project
just run ReplaceNugetCache.ps1 or ReplaceNugetCache.sh with 1 argument of the version
I.e.
ReplaceNugetCache.ps1 0.5.1-build3397-alpha
this allow easy debugging of avalonia from applications like Avalon studio or Core2d.
The second change is courtesy of @kekekeks
A DevTools.ForceOpen() method that can be called from immediate window to attach dev tools to a popup root or other control.

I know I say to merge this, but looking again - are you sure that samples/ControlCatalog.NetCore is the best place for them? It seems rather strange to put them here.


validate enum with their string representation

i'm use enum in proto files and generate swagger definition. In swagger enum represented as strings. Does it possible to validate automatic with string representation of enum (not only int32)?

Validation only occurs against the in-memory representation of the protobuf message. As such, there is never a time where we'd (even be able to) validate against a string representation of it.

ok, thanks


Change morden conf so it is easy to run

after this change you can start morden with
./bin/mantis -Dconfig.file=conf/morden.conf

not needed


queued_max_messages default value(2000) cause out of memory

when I use balancedconsumer class to consumse the kafka message queue, I get the out of memory problem with  queued_max_messages.
To make it simple, I simplly modify the pykafka-demo which worte by cscheffler
and as follow code see, just add a line:
time.sleep(10)
to simulate the real application.
from pykafka import KafkaClient
import time
from threading import current_thread
import uuid
try:
    import configparser  # Python 3
except ImportError:
    import ConfigParser as configparser  # Python 2

config = configparser.ConfigParser()
config.read('settings.conf')

kafka_client = KafkaClient(hosts=config.get('kafka_demo', 'kafka_hosts'))  # Create Kafka client
topic = kafka_client.topics[config.get('kafka_demo', 'topic')]  # This will create the topic if it does not exist
consumer = topic.get_balanced_consumer(
    consumer_group="test_group",
    auto_commit_enable=True,
    zookeeper_connect=config.get('kafka_demo', 'zookeeper_hosts'),
    #queued_max_messages=2000,
    )
display_interval = int(config.get('kafka_demo', 'display_interval'))

print 'Consuming messagse from topic %r. Press Ctrl-C to interrupt.' % topic.name
display_iteration = 0
message_count = 0
partitions = set()  # Track which partitions got consumed by this consumer
process_size = 0
start_time = time.time()
while True:
    message = consumer.consume()  # Read one message from Kafka
    #identifier = uuid.UUID(message.value)  # Decode the message
    process_size += len(message.value)
    message_count += 1
    time.sleep(10)
    print("total_message_count: %d, total_process_size(KB): %d" % (message_count, process_size/1024))

PID USER      PR  NI    VIRT    RES    SHR S  %CPU %MEM     TIME+ COMMAND
23252 root      20   0  9.845g 9.072g   4076 S  20.6 58.5   3:40.80 python demo-consumer.py

when i use top -c command, the memory consumption(nearly 10GB)  makes me surprised.
Per message length is below 500KB, althrough queued_max_messages=2000, the total memory consumption may below 2000 * 500KB=1GB, why 10GB memory has been used?

Thanks for filing this, @ajaxhe. It's clear between this and #470 that there is some memory issue in the consumer. Nothing about your code looks wrong - it's possible that this problem gets worse when the messages being consumed are larger.

@ajaxhe One thing to note here is that the queued_max_messages kwarg is per partition, not total. Thus 10GB of memory might make more sense here if you were testing on a topic with ten partitions. How many partitions does your test topic have?


Test to Replicate Issue ODK-X missing directory

cc @wbrunette @elmps2018

@wbrunette tried to refactor the test method to involve the concept of coping a test file. Possibly some more things I need to  do to make it look better

Test is complete, waiting for appropriate time to merge.

@permission-error you can consider this successfully complete


Change default logleve for reconnect and transact operation

V(5) heavily used for ovsdb cache sync, it may flood log.
In custom logger we usually set log threshold less than 5.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 73.878%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4545

Relevant Lines:
6152

💛  - Coveralls


Make SoftwareComponentInternal finalizable for v8.1

Fixes #20581
Context
Changes made in pull request #22399 were reverted to target milestone 8.1 instead of 8.0.

No problem, I will squash the changes into a single commit.
Ok for it not to be merged until release 8.1.

It is not only about squashing, it is about using the revert feature of Git directly.

@ljacomet should be good now.

@bot-gradle

Sorry I don't understand what you said.
Currently, the following commands are supported:
@bot-gradle test <BuildTrigger1> <BuildTrigger2> ... <BuildTriggerN>

A trigger is a special build for this PR on TeamCity, common triggers are:

SanityCheck/CompileAll/QuickFeedbackLinux/QuickFeedback/PullRequestFeedback/ReadyForNightly/ReadyForRelease
Shortcuts: SC/CA/QFL/QF/PRF/RFN/RFR
Specific builds:

BD: BuildDistributions/BuildDocs so that you can preview the generated docs/distribution.
PT: PerformanceTest, all performance tests for Ready For Nightly stage.
APT: AllPerformanceTest, all performance tests, including slow performance tests.
AST: AllSmokeTestsPullRequestFeedback
AFT: AllFunctionalTestsPullRequestFeedback
ASB: AllSpecificBuildsPullRequestFeedback
ACC: AllConfigCacheTestsPullRequestFeedback
ACT: AllCrossVersionTestsReadyForNightly
AFTN: AllFunctionalTestsReadyForNightly
ACTR: AllCrossVersionTestsReadyForRelease
AFTR: AllFunctionalTestsReadyForRelease

@bot-gradle test and merge

queues this PR for testing and merges if all tests pass by:

Creating a merge commit from your PR branch HEAD and the target branch
Running a ReadyForNightly build against the merge commit
When it passes, fast-forward the target branch to this merge commit (i.e. merge the PR)

The merge commit is called a pre-tested commit, which means that it fully tests the integration
of your branch HEAD and latest master, instead of only testing your branch HEAD.
@bot-gradle cancel

cancel a running pre-tested commit build or remove it from queue

@bot-gradle clean

clear the conversation history

@bot-gradle help

display this message

To run a command, simply submit a comment.  For detailed instructions see here.

@bot-gradle BD

Sorry I don't understand what you said.
Currently, the following commands are supported:
@bot-gradle test <BuildTrigger1> <BuildTrigger2> ... <BuildTriggerN>

A trigger is a special build for this PR on TeamCity, common triggers are:

SanityCheck/CompileAll/QuickFeedbackLinux/QuickFeedback/PullRequestFeedback/ReadyForNightly/ReadyForRelease
Shortcuts: SC/CA/QFL/QF/PRF/RFN/RFR
Specific builds:

BD: BuildDistributions/BuildDocs so that you can preview the generated docs/distribution.
PT: PerformanceTest, all performance tests for Ready For Nightly stage.
APT: AllPerformanceTest, all performance tests, including slow performance tests.
AST: AllSmokeTestsPullRequestFeedback
AFT: AllFunctionalTestsPullRequestFeedback
ASB: AllSpecificBuildsPullRequestFeedback
ACC: AllConfigCacheTestsPullRequestFeedback
ACT: AllCrossVersionTestsReadyForNightly
AFTN: AllFunctionalTestsReadyForNightly
ACTR: AllCrossVersionTestsReadyForRelease
AFTR: AllFunctionalTestsReadyForRelease

@bot-gradle test and merge

queues this PR for testing and merges if all tests pass by:

Creating a merge commit from your PR branch HEAD and the target branch
Running a ReadyForNightly build against the merge commit
When it passes, fast-forward the target branch to this merge commit (i.e. merge the PR)

The merge commit is called a pre-tested commit, which means that it fully tests the integration
of your branch HEAD and latest master, instead of only testing your branch HEAD.
@bot-gradle cancel

cancel a running pre-tested commit build or remove it from queue

@bot-gradle clean

clear the conversation history

@bot-gradle help

display this message

To run a command, simply submit a comment.  For detailed instructions see here.

@bot-gradle test BD

I've triggered the following builds for you:

BuildDistributions build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

I've triggered a build for you.


inetd like deamon?

It would be really nice with some kind of a inetd like deamon, that would not only make sure cbridge is up and running, but also start whatever services on demand.

Having services-on-demand could be implemented similarly to TOPS-20, where the CHARFC server program opens CHA:. (i.e. with empty host and contact components). See the Amber document.
For cbridge, it could be done with a new LSN * operation, which would then send all non-matched RFCs to that socket as RFC nnnn contact args. I thought about this for the Multics implementation, where listening for incoming connections could use something like it. See dps8m.
Making sure cbridge is up and running is a different problem, probably best solved by systemd/launchd etc?

I didn't have an exact idea in mind, what I had sorta in mind initially was a "Chaosnet NCP Super Server" which it would keep cbridge alive, and a way to say that "if RFC is XYZ, then start in.xyz with the following switches".  I always liked inetd, since it provided a very nice little protocol for implementing new services.  launchd/systemd all have their own issues ... notably, might require root to have fun.
But, given that .. CHARFC from TOPS-20 seems just as nice.  All I wanted was really to get rid of the "cbridge-starter-services.sh" script, which often needs tweaking on a per site basis (i.e., I cannot use it .. since it assumes cbridge.conf to be in CWD and other minor stuff like that).

Given an RFC for FOO BAR, CHARFC looks for SYSTEM:CHAOS.FOO, and if it exists, starts it. For the corresponding thing with cbridge, of course switches etc can be added.
You would still have to configure the "cbridge charfc" of course, so you won't get rid of having to edit your local configuration. (So not all that much of a difference from cbridge-starter-services.sh, except the "on demand" perspective?)

Well, and a central place to state what you want to run on your bridge, which I think is nice -- but it sorta is complicating the config I guess?


First PR of initial commit is not showing up

I am trying to setup my first blog using fastpages. I am having some issues right now.
I can't see the following setting:

If you are not seeing a PR, please make sure you have third party actions enabled in your organization: Settings -> Actions -> Actions Permissions -> Enable local and third party Actions for this repository

So, I checked the following-

In some cases because of permissions step 2 may fail to create a Pull Request. If this occurs, go to your repository settings and in the Actions section, grant Read and Write permissions and check Allow GitHub Actions to create and approve pull requests.

The required options are already checked.
What am I missing?

NB: I am writing this comment on all open issues
Thank you for posting this issue, and I'm really sorry I haven't gotten around to fixing this.  All of my time has been focused on nbdev.   In the new version of nbdev, we knew that we wanted to include blogging rather than it being a separate project.  We realized that Quarto, which the new version of nbdev is built on, is much better than fastpages.    Rather than keep this project going, we are recommending that people migrate to Quarto as we think that will be more powerful for users and is a much better tool for blogging.
I've written a detailed migration guide with some tools that will hopefully make the process of switching much easier.  Overall I think it will be worth it for users.
My apologies again for not getting to this issue sooner.  Will close this issue with this PR https://github.com/fastai/fastpages/pull/657, where we are letting people know that we are deprecating this project.  Thanks again for your support and for using fastpages!  🙇🏽


Open proposal summary: using all the votes for and against

bring back the vote summary and use the details on each pending vote to build up the total for/ against. These are not yet provided in API for open proposals.

show 2 decimal places

Participation: Not Met 0 <- no + yes
Participation: Not Met 0 0.0000% <- tokens votes percentage of total tokens fixed total supply
query {
assets {
totalSupply
name
}
}

This has been done and signed off in Staging


Enable schemaDescription & inputTypeDeprecation

Not sure why schemaDescription wasn't already enabled for GraphiQL introspection queries?
Either way, this fixes the DocExplorer schema.description issue in #2027 when fetcher is provided instead of schema prop.
Also added support for input type deprecationReason

Took me a few minutes, but this is where the problem started.
Starting 1.5.9, including https://unpkg.com/graphiql/graphiql.min.js is very broken.
The errors showing up in 1.5.9:
{
  "errors": [
    "{\n  \"errors\": [\n    {\n      \"message\": \"Cannot query field \\\"description\\\" on type \\\"__Schema\\\".\",\n      \"extensions\": {\n        \"category\": \"graphql\"\n      },\n      \"locations\": [\n        {\n          \"line\": 4,\n          \"column\": 9\n        }\n      ]\n    },\n    {\n      \"message\": \"Unknown argument \\\"includeDeprecated\\\" on field \\\"args\\\" of type \\\"__Directive\\\".\",\n      \"extensions\": {\n        \"category\": \"graphql\"\n      },\n      \"locations\": [\n        {\n          \"line\": 16,\n          \"column\": 16\n        }\n      ]\n    },\n    {\n      \"message\": \"Cannot query field \\\"specifiedByURL\\\" on type \\\"__Type\\\".\",\n      \"extensions\": {\n        \"category\": \"graphql\"\n      },\n      \"locations\": [\n        {\n          \"line\": 27,\n          \"column\": 7\n        }\n      ]\n    },\n    {\n      \"message\": \"Unknown argument \\\"includeDeprecated\\\" on field \\\"args\\\" of type \\\"__Field\\\".\",\n      \"extensions\": {\n        \"category\": \"graphql\"\n      },\n      \"locations\": [\n        {\n          \"line\": 31,\n          \"column\": 14\n        }\n      ]\n    },\n    {\n      \"message\": \"Unknown argument \\\"includeDeprecated\\\" on field \\\"inputFields\\\" of type \\\"__Type\\\".\",\n      \"extensions\": {\n        \"category\": \"graphql\"\n      },\n      \"locations\": [\n        {\n          \"line\": 40,\n          \"column\": 19\n        }\n      ]\n    },\n    {\n      \"message\": \"Cannot query field \\\"isDeprecated\\\" on type \\\"__InputValue\\\".\",\n      \"extensions\": {\n        \"category\": \"graphql\"\n      },\n      \"locations\": [\n        {\n          \"line\": 62,\n          \"column\": 7\n        }\n      ]\n    },\n    {\n      \"message\": \"Cannot query field \\\"deprecationReason\\\" on type \\\"__InputValue\\\".\",\n      \"extensions\": {\n        \"category\": \"graphql\"\n      },\n      \"locations\": [\n        {\n          \"line\": 63,\n          \"column\": 7\n        }\n      ]\n    }\n  ]\n}"
  ]
}

1.5.10 does not seem to exist, and the errors for 1.5.11 and 1.5.12 (which is the last one at the time of writing this):
{
"errors": [
"{\n  "errors": [\n    {\n      "message": "Cannot query field \"description\" on type \"__Schema\".",\n      "extensions": {\n        "category": "graphql"\n      },\n      "locations": [\n        {\n          "line": 4,\n          "column": 9\n        }\n      ]\n    }\n  ]\n}"
]
}


Questions about the model FLOP, GPU memory cost, and speed

Hello,
very impressed with your work.
But I have a few small questions about Table1 and Table2, specifically about the model FLOP, GPU memory cost, and speed.
As far as I know, the size of each slide is different, which causes the efficiency of each image calculation to be different.
So my question is, what size slice are you computing on, and how many patches are there?
Looking forward to your reply.
Yours.
pzSuen.

Hi,
Very thanks for your attention to our work.
These results were calculated under a consistent size of patches for each dataset because we need to pad the data to a fixed tensor size for batch training. We assigned a max patch size of 2048 for the gastric dataset since we found most of the patch number in this dataset not exceed 2048. Similarly, we assigned the max patch size as 4096 for our endometrial dataset. So, the GPU memory, speed, and FLOPs for the gastric dataset are the results for 2048 patches and the results for the endometrial dataset are 4096 patches.
Please feel free to contact me if you have other questions.
我看你头像是567，是在哈尔滨高校的同僚吗？方便的话我们也可以用中文交流。
Sincerely,
Yushan Zheng
郑钰山
发件人: @.*** @.> 代表 pzSuen
发送时间: 2022年11月13日 13:13
收件人: Zhengyushan/kat @.>
抄送: Subscribed @.***>
主题: [Zhengyushan/kat] Questions about the model FLOP, GPU memory cost, and speed (Issue #1)
Hello,
very impressed with your work.
But I have a few small questions about Table1 and Table2, specifically about the model FLOP, GPU memory cost, and speed.
As far as I know, the size of each slide is different, which causes the efficiency of each image calculation to be different.
So my question is, what size slice are you computing on, and how many patches are there?
Looking forward to your reply.
Yours.
pzSuen.
—
Reply to this email directly, view it on GitHub https://github.com/Zhengyushan/kat/issues/1 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AG6JCPATZ5GMTT4QTNMLXTTWIB2FZANCNFSM6AAAAAAR6U7PGY .
You are receiving this because you are subscribed to this thread.  https://github.com/notifications/beacon/AG6JCPAVVIF3XQ2DWH5DKX3WIB2FZA5CNFSM6AAAAAAR6U7PG2WGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHFMO34ME.gif Message ID: @.*** @.***> >

谢谢您的快速回复。
是的，我是哈工大计算学部的一名在读博士生，目前研究方向是病理图像分析。
我在各大顶会、顶刊上都看到过您的工作，真是久仰大名！
您的回答解决了我的疑问。
那接下来我有下面几个小问题：

请问您的2048/4096的Patch是从20倍放大的切片上裁剪的吗？是否进行背景区域的过滤？以及如果保证切片标签赋值给Patch保证标签不变化？
您是否在全切片上测试您的模型？比如图像尺寸在100000×100000数量级。
您的结果中只展示了KAT在各种参数下的计算量和显存，那您是否对比与TransMIL等方法的计算量和显存？

期待您的再次回复！


Consider upgrading built-in Hive version to 2.3.x

I would like to bring out the discussion about upgrading the built-in Hive support version from 1.2.1 to 2.3.x. The reason for this upgrading is:

The community is working on Hive Iceberg support, and there's some potential version problems (https://github.com/apache/incubator-iceberg/issues/170#issuecomment-594122562).
The version 1.2.1 is quite old, most of the frameworks now move on to 2.3.x, like Spark.

Please share your comments.
CC @rdblue @rdsr .

The reason for using 1.2.1 is that's the version Spark uses. When we update to Spark that depends on something other than 1.2.1-spark2, I think we all agree we should update.
I also thought that it was safer to depend on 1.2.1, so that we don't use any new APIs that are incompatible with that version. But it should be safe to update to 2.3.x as long as we test the spark-runtime Jar with 2.4.x versions of Spark where 1.2.1-spark2 is the provided version.
Since the Hive catalog is fairly stable at this point, I think it probably makes sense to update to 2.3.x to enable Hive/Iceberg support.

I think it should be worked when upgrading to Hive 2.3.x, I will take a try on this.


Accept list of sources with markers in uv.tool.sources

tool.uv.sources should accept a list of dicts instead of a single dict, too, to support splitting for different python versions (detailed design TBD):
[project]
dependencies = [
    "idna>=2.8; python_version > '3.11'",
    "idna<2.8; python_version <= '3.11'"
]

[tool.uv.sources]
idna = [
  { url = "https://example.org/idna-2.8.zip, python = "<3.11" },
  { url = "https://example.org/idna-2.7.zip, python = ">=3.11" },
]

In review here: https://github.com/astral-sh/uv/pull/7745


Conversation out of frame, when width is less than 640 pixels

When the window is scale down to less than 640 pixels in width, friend's chat jumps out of frame.
OS: Ubuntu 14.04.
AutohideSideBar = true

The auto-hide sidebar isnt working very well. I will take a look at it.
Please see https://github.com/Aluxian/Facebook-Messenger-Desktop/issues/458 meanwhile


Exception when debug function is not defined

If you use this library setting debug=null (I don't want a debug function at all), reconnections will throw an Exception and your application may become unstable:
Line:
@debug("STOMP: scheduling reconnection in #{@reconnect_delay}ms")
Should become something like:
@debug?("STOMP: scheduling reconnection in #{@reconnect_delay}ms")
This bug also affects the JS and minified version, of course.
If you could, please, fix it.
Thanks for your dedication and work.

Planning for next version of stompjs is on the way. Your participation will be really appreciated, please join at https://github.com/stomp-js/stompjs/issues/1


Set actual value to information_shcema.tables.table_comment

Current information_schema.tables.table_comment column always returns NULL. This commit sets actual comment value to the column.
@findepi I tried to modify io.prestosql.metadata.MetadataListing#listTables from Set<SchemaTableName> to Map<SchemaTableName, TableMetadata> first. However, some connector may throw an exception during getTableHandle. Therefore, I put the logic on InformationSchemaPageSourceProvider.
https://github.com/prestosql/presto/issues/170#issuecomment-475170357

I don't think we want to add this, or at least it needs to have a config flag which is disabled by default. This column was added in #192 specifically to fix Hue. Fetching the comment will make listing tables extremely expensive for connectors like Hive.
We want to remove these column in the future after the Hue fix is generally available, since they are not part of the SQL standard for information_schema.

I don't think we want to add this, or at least it needs to have a config flag which is disabled by default. This column was added in #192 specifically to fix Hue. Fetching the comment will make listing tables extremely expensive for connectors like Hive.
We want to remove these column in the future after the Hue fix is generally available, since they are not part of the SQL standard for information_schema.

This is also why the columns are hidden. They are just dummy null columns so that the SQL generated by Hue does not fail.

I see, my concern is that there's no way to get the comment except for parsing SHOW CREATE TABLE result. Do you have any idea to improve that? Many RDBMS offer to get it from their dictionary table.

I agree it's a problem we should solve. Having it in information_schema.tables is problematic due to performance and because it's not in the standard (though we have other non-standard columns).
We could potentially solve the performance issue by only fetching the column data if the column is selected by the query. This would require extending SystemTable but would be a useful extension. However, that brings up the issue of if it should remain hidden. It seems strange to be hidden if it's a supported interface, though making it non-hidden means SELECT * FROM ... will have bad performance -- including any existing queries which do this.
The system.jdbc.tables table used for java.sql.DatabaseMetaData#getTables has a remarks field which should be supported per the JDBC specification, but is not implemented for the same performance reason. Unfortunately, the projection trick wouldn't work since there's no way to choose columns with that JDBC call.
Another option is to add a system.metadata.table_comments table. This is probably the easiest and least contentious solution.

Thank you for providing the detail. It's so helpful to understand the context.
Let me close this PR. I'll send new PR to add system.metadata.table_comments table.


is there any pattern or example that i can follow to create master-slave program using etcd's concurrency package ?

Is there any pattern or example that i can follow to create master-slave program using etcd's concurrency package ? And what about this one?
func main() {
	cli, err := clientv3.New(
		clientv3.Config{
			Endpoints: []string{"<IP_ADDRESS>:2379"},
			DialTimeout: 3* time.Second,
			})

	if err != nil {
		logrus.Error("NewClient: ", err.Error())
	}

	for {

		session, err := concurrency.NewSession(cli, concurrency.WithTTL(10))

		if err != nil {
			logrus.Info("NewSession: ", err.Error())
		}

		election := concurrency.NewElection(session, "/dist-election")

		err = election.Campaign(context.Background(), "e1")

		if err != nil {
			logrus.Error("Campaign: ", err.Error())
		}

		logrus.Info("Start Program")

		select {
		case <-session.Done():
			logrus.Error("Session Done")
			logrus.Info("Stop Program")
		}
	}

}

Here's a good example of master election https://github.com/PaddlePaddle/Paddle/blob/master/go/master/etcd_client.go

It's good. but it did not use election campaign interface from concurrency.So what should i do ?


fix: ensure we execute tests only once per run

fix #113
It ensures if the current run is active, we don't schedule new runs even invoked via CLI or another programmatic run

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: [Pull request #114 opened]

Start Time: 2020-10-29T17:06:05.722+0000

Duration: 14 min 1 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
42

Skipped
0

Total
42

@andrewvc Added the test.


feat: document and contract query verification

Issue being fixed or feature implemented
We had no way to verify document or contract proofs, now we do.
What was done?
Added methods to verify document query proofs.
Added methods to verify contract query proofs.
Fixed historical contracts.
Added methods to verify historical contract query proofs.
How Has This Been Tested?

Breaking Changes

Checklist:

[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have added or updated relevant unit/integration/functional/e2e tests
[X] I have made corresponding changes to the documentation

For repository code-owners and collaborators only

[ ] I have assigned this pull request to a milestone

@markin-io @shumkov @antondash I'm going to merge this one without a review as it will make life a lot simpler for the next task. This will save a lot of time. If any of you have an issue with this PR tomorrow we can revert it.


Cuba/Suncrypt/Everest always think its the first scrape

Describe the bug
Darknet sites for Cuba, Suncrypt and Everest always return as "This is the first scrape for x, no victim notifications will be sent." I feel like scraping for these specific sites might currently be faulty as they definitely currently have victims on the site which should be being captured and archives as part of the scrapes.
To Reproduce
Run the tool with Cuba/Suncrypt/Everest TOR addresses inputted in config.
Expected behavior
For the Cub/Suncrypt/Everest site scrapes to return victims that are successfully populated into the SQL database.
Screenshots
N/A
Logs
N/A
Environment

OS: Ubuntu 18.04.5 LTS
How you are running it: Docker via cronjob

Additional context
I should probably raise as a separate bug report but unsure if this is actually a bug or just the nature of TOR - does anyone else have exceptionally shaky up time with the darknet sites? I can run multiple concurrent scans and find that on one scan certain sites will be scraped fine while an immediate subsequent scan will find that the same site is down? Has anyone else experienced this flakiness of the scrapes? I know TOR sites are up and down like yoyos at the best of times, but this feels like it might be mroe bug based...

The scrape may not be working properly for those, I'll take a look when I can and get those updated. Thanks for flagging the issue!
Some sites are definitely up more than others, a few of them have terrible uptime (Avaddon was one that was up maybe half the time). It could be that the timeout value is too aggressive for your network connection, adjusting that may be beneficial. You can change the timeout value in the config.