ValueError: On entry to SGESDD parameter number 12 had an illegal value

Hello,
I'm getting a ValueError from scipy/linalg/decomp_svd.py, line 104 when tring to run sklearn's pca.fit() function.
Traceback:
File "train.py", line 269, in
pca.fit(X)
File "/homes/pp305/.local/lib/python2.7/site-packages/sklearn/decomposition/pca.py", line 221, in fit
self._fit(X)
File "/homes/pp305/.local/lib/python2.7/site-packages/sklearn/decomposition/pca.py", line 271, in _fit
U, S, V = linalg.svd(X, full_matrices=False)
File "/homes/pp305/.local/lib/python2.7/site-packages/scipy/linalg/decomp_svd.py", line 104, in svd
full_matrices=full_matrices, overwrite_a=overwrite_a)
ValueError: On entry to SGESDD parameter number 12 had an illegal value
Versions I'm using:
Python 2.7.5
scipy 0.16.0
sklearn 0.16.1
numpy 1.9.2
print scipy.config.show()
atlas_3_10_blas_threads_info:
libraries = ['tatlas']
library_dirs = ['/usr/lib64/atlas']
define_macros = [('HAVE_CBLAS', None), ('ATLAS_INFO', '"\"3.10.1\""')]
language = c
include_dirs = ['/usr/include']
lapack_opt_info:
libraries = ['tatlas', 'tatlas', 'tatlas']
library_dirs = ['/usr/lib64/atlas']
define_macros = [('ATLAS_INFO', '"\"3.10.1\""')]
language = f77
include_dirs = ['/usr/include']
blas_opt_info:
libraries = ['tatlas']
library_dirs = ['/usr/lib64/atlas']
define_macros = [('HAVE_CBLAS', None), ('ATLAS_INFO', '"\"3.10.1\""')]
language = c
include_dirs = ['/usr/include']
openblas_info:
NOT AVAILABLE
openblas_lapack_info:
NOT AVAILABLE
atlas_3_10_threads_info:
libraries = ['tatlas', 'tatlas', 'tatlas']
library_dirs = ['/usr/lib64/atlas']
define_macros = [('ATLAS_INFO', '"\"3.10.1\""')]
language = f77
include_dirs = ['/usr/include']
lapack_mkl_info:
NOT AVAILABLE
blas_mkl_info:
NOT AVAILABLE
mkl_info:
NOT AVAILABLE
None
I found an issue like this here already:
https://github.com/scipy/scipy/issues/3895
Could you let me know how I can go about getting this to work?
Thank you!

(i) What is your LAPACK version? If older than 3.2.2, GESDD work space
query was known to have bugs.
(ii) What is the size of the matrix?

Hello,
LAPACK version is 3.4.2 and the size of the matrix was 9537x46080.
Now I tried using matrices with other dimensions and I'm getting the same error when I try with matrices of sizes 9537x20000, 9537x10000, 9537x9538, 9537x9537.
If I use a matrix of size 9537x9536 it's working fine.

LAPACK uses 32-bit integers and will start failing if the work space size becomes larger than INT_MAX. You can maybe check this via
import numpy as np
from scipy.linalg import lapack
intmax = np.iinfo(np.intc).max
n, m = 9537, 9537
lwork = lapack.sgesdd_lwork(n, n, True, True)[0]
print(lwork / intmax) # should be < 1

# Try to decompose:
a = np.random.rand(n,n)
lapack.sgesdd(a, True, True, lwork=int(lwork))

n=9537 is however not yet so big it should be problematic, at least with lapack 3.5.0, but it is not very far from the limit either. Maybe it is different on 3.4.2?

OK, I ran the code you wrote and got the same error:
python test.py
0.127092868149
Traceback (most recent call last):
File "test.py", line 10, in
lapack.sgesdd(a, True, True, lwork=int(lwork))
ValueError: On entry to SGESDD parameter number 12 had an illegal value
with n = 9536 it works.
What do you suggest? Should I just try upgrading lapack, or is there something else I can try?
Thank you!

Just to update, I tried your code again with n=9538, n=9600 and with n=10000 and it worked each time. I also tried running with n=20000 and n=40000 and it seems to be working both times (I didn't wait until it finished).
n=9537 still resulting in error (). Is there something special about 9537 or lwork being 272929856?

These numbers come directly from LAPACK. What I think is happening is the following: LAPACK sgesdd lwork query returns the optimal lwork as a single-precision float value. However, the next possible float32 number bigger than 272929856 is actually 272929888. If the value of lwork LAPACK computed falls between these two, it gets rounded down when returned in a float32 variable.
So I think:

This is a LAPACK bug (should be reproduced with a small fortran program and reported to LAPACK authors).
Scipy should work around it, by taking the next largest floating point value (lwork = int(np.nextafter(lwork, np.float32(np.inf))) instead of lwork = int(lwork)).

In the meantime, you can work around it

Thank you very much for your replies! I will try to use your workaround.

LAPACK bug report: http://icl.cs.utk.edu/lapack-forum/viewtopic.php?f=2&t=4826
Workaround in gh-5413

Appears to be fixed in the next lapack version http://www.netlib.org/lapack/Errata/index2.html#_strong_span_class_green_bug120_span_strong_workspace_query_return_too_small_value_for_xgesdd


Update version

What does this PR do?
Update current example to appwrite 8.2.2
Test Plan
Manual
Related PRs and Issues
(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)
Have you read the Contributing Guidelines on issues?
(Write your answer here.)

@codercatdev sorry, I actually have this change pending along with an update to make it match the other todos: https://github.com/appwrite/demo-todo-with-flutter/pull/13

Also, this is the related issue for this: https://github.com/appwrite/appwrite/issues/4276

@codercatdev sorry, I actually have this change pending along with an update to make it match the other todos: #13.
Also, this is the related issue for this: appwrite/appwrite#4276.

Yes this is to get current working though. @stnguyen90 I would like to get this merged and then work on #13

@lohanidamodar added update

Closing as #13 is our new setup.


Add watermark support

Thank you for developing Gallery Deluxe! (it was the only gallery that "just worked" for me).
Would be great if during/after generation, gallery supported applying a .png watermark on top of images.
Why it has to be done on the gallery level and not at originals level is that tiny previews should not have a watermark, while when user clicks to see the larger picture - it should have one.

Yea, that sounds like a good idea.

Wow, I couldn't imagine that it can be added so fast. Thank you! Already migrated from my bash script with composite from ImageMagick and it works!


5 load new provider form data from json schema

This solution currently read from the git repository the file to use. It should be better to have a local version of that file and read it when needed.

Load json schema from path


WIP: Cfs/rh cf rp dev env working sessions

Work-in-progress: Do not merge. Using to track working sessions between RH and my org.
Which issue this PR addresses:

Fixes
What this PR does / why we need it:

Test plan for issue:

Is there any documentation that needs to be updated for this PR?

Closing this PR, it was only used for experimentation purposes.


The result of running the python code is empty

Hello, sorry to bother you. I want to ask if the code in this code base is complete. The result of running the python code I tried is empty.

It seems that the result might be empty depending the feasibility and solution of the CVXPY problem. CVXPY is less robust to SDP constraints respect to CVX in Matlab, which can lead to infeasibility or suboptimal solutions.
For example, In Python, K = np.linalg.solve(P.value, U0 @ Q.value) requires P.value to be invertible. If P.value is close to singular or non-invertible, np.linalg.solve will fail.  Therefore the empty result.
Switching to different solver (such as MOSEK?) might lead to better results.
Besides the solver, there could be some workaround to make the python script work more similar to matlab.

ensure P.value is invertible before calculating K:

if P.value is not None and np.linalg.cond(P.value) < 1 / np.finfo(P.value.dtype).eps:
    K = np.linalg.solve(P.value, U0 @ Q.value)
else:
    K = old_K
    print("Using old K due to singular or undefined P")

regarding numerical stability of the code, you might need to add regularization:
P = cp.Variable((n, n), symmetric=True) + 1e-5 * np.eye(n)
you could also check the convergence status of the solve by printing out prob.status to ensure that the optimization problem is feasible.

Let me know whether these steps are helpful.

Hello, I ran it successfully. There was something wrong with my previous operation. Now I can run it normally and get two pictures.


JDA#shutdown gets blocked on shard death.

When doing JDA#shutdown(false) [Attempting to drop a dead shard] or even JDA#shutdown() [when trying to shutdown the whole bot with a dead shard], the shutdown method gets blocked and it never calls the subsequent statements or shuts down the bot at all.
After a shard is dead the only option left is to eval a system exit call and restart the whole bot all over again.
http://i.imgur.com/TEQKzd8.png -> After restarting a shard, it should say "dropped shard x successfully", but it never does. Also weird status changes in between.

Not getting this on 3.0.0_168. I get a lot of dead shards but I detect and restart them automatically without further problems.

3.0.0_169 here.

Does this still happen on 3.0.0_176?
Can you provide a thread dump?

I have never gotten this, just putting the info out there.

I will update, deploy and check. If it hapoens again I will provide the dump
Thanks.

Has this occurred again or am I safe to assume this is a no-issue?

Happened once more a while ago. Haven't gotten a shard death since though, so cannot check either.

It turned out that jda-nas has a bug that caused the thread that calls AudioManager#closeAudioConnection() to lock for 1000 seconds (around 16 minutes). As this method is often called from the event thread it locked the reading thread of jda's main websocket.
Please update jda-nas and report back weather this resolved the issue the issue or not.

I removed it for now. I'll report back

It turned out that this behaviour can also be caused by #338 . Thus I will keep this open until that issue is resolved and further investigations show this being fixed.

Does this still happen with build 193?

Gonna check.

As I haven't received reports of this for 5 days I now close this issue.


Enabling pg_stat_statements in Docker Compose

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
There are multiple slow downs over time which have been adressed in different issues / PRs by adding Indexes or rewriting Queries.
Expected Behavior
To be able to monitor Query Performance proactively it would be great to have pg_stat_statements enabled at least in instances of contributors, maybe in all instances.
This allows getting Query Count & Average Execution Times for Queries to take action where needed and ensure optimal Instance Performance.
These Stats could also be be used in Custom Teslamate Dashboards (Database Info).
The change would be a rather simple addition to the docker compose file (adding the command) and a database migration (to enable the views required to retrieve the stats) or even better, we could incorporate that dashboard and add pg_stat_statement derived statistics.
While it might sounds like beeing overwhelming for the average users as most users are operating their instance themselves i think it's great to make it easier / more transparent on how their instances are performing and what issues they are facing.
database:
    image: postgres:17
    restart: always
    command: postgres -c default_toast_compression=lz4 -c shared_preload_libraries=pg_stat_statements
    environment:
      - POSTGRES_USER=teslamate
      - POSTGRES_PASSWORD=****
      - POSTGRES_DB=teslamate
      - TZ=Europe/Berlin
    volumes:
      - teslamate-db:/var/lib/postgresql/data

CREATE EXTENSION IF NOT EXISTS pg_stat_statements;

Steps To Reproduce
No response
Relevant log output
-

Screenshots
No response
Additional data
No response
Type of installation
Docker
Version
v1.30.1

@JakobLichterfeld - should i open a change for the docs (contribution / general install instructions)?

should i open a change for the docs (contribution / general install instructions)?

That would be awesome, ty!

will do, what do you prefer, within contribution guide or as addition to the general docker based install?

will do, what do you prefer, within contribution guide or as addition to the general docker based install?

For now I would start with contributing quide


Siren Bloodpact

Record damage from Hysteric Assault.
Credit for finding the message id to NassirAlawar.
https://github.com/Windower/Lua/pull/1829/commits/edf900c66e4da636875787d10e47b017e0bc0d4d

Sorry, I haven't been playing the past few months; is this an ability or it should be counted as melee damage from Siren? If it is an ability, it might be better to wait for the Window devs to add it to their res files so it categorises it under the specific ability (should automatically work once it is added, but if it isn't already, I'll take a closer look).

It's a new bloodpact that is already in the resources. The issue is that it uses a brand new message id (802) because it shows up as a drain in game instead of direct damage. As far as I can tell, all the damaging bloodpacts are counted as melee damage in the current version.

Ah okay. I think I vaguely remember something about this, but I've never played SMN :(
I'll also open an issue for this to remind myself to look into it if I ever get the chance.


Add code for temperature sensor on COBC PCB

Fixes #297

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.68%. Comparing base (1dcf911) to head (5bb8dd1).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #334   +/-   ##
=======================================
  Coverage   79.68%   79.68%
=======================================
  Files          47       47
  Lines        1580     1580
  Branches       57       57
=======================================
  Hits         1259     1259
  Misses        321      321

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

HW test ran successfully on the FlatSat today.


[REQUEST] Add drag and drop option to wordlist and add a option to add multiple wordlists in both upload and filesystem

[REQUEST] Add drag and drop option to wordlist and add a option to add multiple wordlists in both upload and filesystem

You literally opened a similar issues a few days ago https://github.com/openbullet/OpenBullet2/issues/31

please add that option as soon as possible

please add that option as soon as possible

Everything takes time, I don't see this as a priority right now, bugfixes need to be addressed before anything else.


Fixing a typo in a 'select' test

it('should scroll top the top when pressing HOME',
chganged to
it('should scroll to the top when pressing HOME'

I signed it!

I have added the email I used to create the PR to my google account.


[9.x] Add isWithParameters method and an alias named isWith

With using Route::isWith() we can extend Route::is() functionality to check named routes with parameters.
Usage :
Route::isWith(
    'without.parameter',
    ['parameters.removed'],
    ['with.parameter', 'john'],
    ['with.parameter', ['name' => 'john']]
) {
    //
}

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


ABNF char-val sensitivity inverted on generate API

@Timothepou made a demo using pap, but it generated a case-sensitive string for a case-insensitive rule.
The sensitivity check at https://github.com/pandatix/go-abnf/blob/main/generate.go#L83 might be the problem.

Fixed in v0.1.3


IDX20803 Error when using Identity Server 4 on Linux Docker containers

Question
I am encountering an IDX20803 error when I try to deploy our .NET 5 Angular application to a docker container. We have been able to successfully deploy the code locally and on Azure without any authentication issues. However, when we deploy that same code inside a docker container we get the following error:
fail: Microsoft.AspNetCore.Authentication.JwtBearer.JwtBearerHandler[3]
Exception occurred while processing message.
System.InvalidOperationException: IDX20803: Unable to obtain configuration from: 'https://localhost/.well-known/openid-configuration'.
at Microsoft.IdentityModel.Protocols.ConfigurationManager`1.GetConfigurationAsync(CancellationToken cancel)
at Microsoft.AspNetCore.Authentication.JwtBearer.JwtBearerHandler.HandleAuthenticateAsync()
fail: IdentityServer4.AccessTokenValidation.IdentityServerAuthenticationHandler[0]
IDX20803: Unable to obtain configuration from: 'https://localhost/.well-known/openid-configuration'.
System.InvalidOperationException: IDX20803: Unable to obtain configuration from: 'https://localhost/.well-known/openid-configuration'.
at Microsoft.IdentityModel.Protocols.ConfigurationManager`1.GetConfigurationAsync(CancellationToken cancel)
at Microsoft.AspNetCore.Authentication.JwtBearer.JwtBearerHandler.HandleAuthenticateAsync()
at Microsoft.AspNetCore.Authentication.JwtBearer.JwtBearerHandler.HandleAuthenticateAsync()
at Microsoft.AspNetCore.Authentication.AuthenticationHandler`1.AuthenticateAsync()
at Microsoft.AspNetCore.Authentication.AuthenticationService.AuthenticateAsync(HttpContext context, String scheme)
at IdentityServer4.AccessTokenValidation.IdentityServerAuthenticationHandler.HandleAuthenticateAsync()
info: IdentityServer4.AccessTokenValidation.IdentityServerAuthenticationHandler[7]
Bearer was not authenticated. Failure message: IDX20803: Unable to obtain configuration from: 'https://localhost/.well-known/openid-configuration'.
info: Microsoft.AspNetCore.Authorization.DefaultAuthorizationService[2]
Authorization failed. These requirements were not met:
DenyAnonymousAuthorizationRequirement: Requires an authenticated user.
ClaimsAuthorizationRequirement:Claim.Type=permission and Claim.Value is one of the following values: (roles.view)
trce: IdentityServer4.AccessTokenValidation.IdentityServerAuthenticationHandler[0]
Forwarding challenge to scheme: BearerIdentityServerAuthenticationJwt
info: Microsoft.AspNetCore.Authentication.JwtBearer.JwtBearerHandler[12]
AuthenticationScheme: BearerIdentityServerAuthenticationJwt was challenged.
info: IdentityServer4.AccessTokenValidation.IdentityServerAuthenticationHandler[12]

When the container is running we are able to resolve the website "https://localhost/.well-known/openid-configuration" and view the openid-configuration.
My current idea is that "options.authority" is being set such that I am able to access the openid-configuration externally, but that same configuration is somehow blocking access internally.
Any Ideas?
Minimal working example
Dockerfile
FROM mcr.microsoft.com/dotnet/aspnet:5.0 AS base
RUN apt-get update && \
    apt-get install -y wget && \
    apt-get install -y gnupg2 && \
    wget -qO- https://deb.nodesource.com/setup_10.x | bash - && \
    apt-get install -y build-essential nodejs
WORKDIR /app
EXPOSE 80
EXPOSE 443

FROM mcr.microsoft.com/dotnet/sdk:5.0 AS build
RUN apt-get update && \
    apt-get install -y wget && \
    apt-get install -y gnupg2 && \
    wget -qO- https://deb.nodesource.com/setup_10.x | bash - && \
    apt-get install -y build-essential nodejs
WORKDIR /src

COPY ["DAL/DAL.csproj", "DAL/"]
COPY ["QuickApp/QuickApp.csproj", "QuickApp/"]
RUN dotnet restore "QuickApp/QuickApp.csproj"
COPY . .
WORKDIR "/src/QuickApp"
RUN dotnet build "QuickApp.csproj" -c Release -o /app/build

FROM build AS publish
RUN dotnet publish "QuickApp.csproj" -c Release -o /app/publish

FROM base AS final
WORKDIR /app

COPY --from=publish /app/publish .
COPY ["QuickApp/Assets", "Assets"]
COPY ["QuickApp/ClientApp", "ClientApp"]
RUN cd ClientApp && npm install

ADD conf.d/localhost.crt /usr/local/share/ca-certificates/localhost.crt
RUN chmod 644 /usr/local/share/ca-certificates/localhost.crt && update-ca-certificates

ENTRYPOINT ["dotnet", "QuickApp.dll"]

Docker-Compose
services:
  quickapp:
    user: root
    image: quickapp
    build:
      context: .
      dockerfile: QuickApp/Dockerfile
    environment:
      - WEB_ENVIRONMENT=Development
      - ASPNETCORE_ENVIRONMENT=Development
      - ASPNETCORE_URLS=https://+:443;http://+:80;
      - ASPNETCORE_HTTPS_PORT=443
      - ASPNETCORE_Kestrel__Certificates__Default__KeyPath=/https/localhost.key
      - ASPNETCORE_Kestrel__Certificates__Default__Path=/https/localhost.crt
    ports:
      - 80:80
      - 443:443
    volumes:
      - ./conf.d:/https:ro

Authentication Configuration
services.AddAuthentication(IdentityServerAuthenticationDefaults.AuthenticationScheme)
                .AddIdentityServerAuthentication(options =>
                {
                    options.Authority = applicationUrl; // Set to "https://localhost"
                    options.SupportedTokens = SupportedTokens.Jwt;
                    options.RequireHttpsMetadata = false;
                    options.ApiName = IdentityServerConfig.ApiName;
                });

Note: I am creating this issue here per direction of the aspnetcore issue I created.
https://github.com/dotnet/aspnetcore/issues/30942

Figured out the issue.
I had to add this to my .AddIdentityServerAuthentication
options.JwtBackChannelHandler = GetHandler();

private static HttpClientHandler GetHandler()
        {
            var handler = new HttpClientHandler();
            handler.ClientCertificateOptions = ClientCertificateOption.Manual;
            handler.SslProtocols = SslProtocols.Tls12;
            handler.ServerCertificateCustomValidationCallback = (message, cert, chain, errors) => true;
            return handler;
        }


Notification Library

Implement Notification Library to define interfaces

Need to save:

INotyfier
IChannel
Mapping on INotyfier with IChannels

from NotificationService


Adding switch telemetry to default poller.

snmp txrx counters to default pollers

*** BUILD #1059 ***

test this please

*** BUILD #1060 ***

test this please

👍 after the minor comments from @yyscamper and I. Looks good otherwise!

@benbp and @yyscamper updated with comments

This will fail tests until the constant is add to on-core: https://github.com/RackHD/on-core/pull/169

@RackHD/corecommitters @benbp @VulpesArtificem

👍

👍


Validation guide not working

Hi team,
I moved this from here: https://github.com/MicrosoftDocs/windowsserverdocs.pt-br/issues/2
The broken link is also present in the en-us page.
@ingd commented 30 minutes ago — with docs.microsoft.com
The link for validation guide is not working. Please, could you fix it?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7db91f6a-373f-70d5-7e5e-455e08a1d9e1
Version Independent ID: 2f44b38f-dadd-689b-c90e-2d2ea2c93528
Content: Insider preview for Windows Time Service features in Windows Server 2019
Content Source: WindowsServerDocs/networking/windows-time-service/insider-preview.md
Product: windows-server
Technology: networking
GitHub Login: @shortpatti
Microsoft Alias: pashort

Thanks for your dedication to our documentation. If we have been unable to review your issue in a timely manner, we sincerely apologize for the delayed response. We have captured the informtion you have provided for review. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Mouse position (without click) affects what action takes place in Task Editor

As ever, it is interesting to watch someone else interact with software, they do things that I don't and you realize the little gremlins that creep in.  In this case, it has to do with the mouse position when you are adding masses in the Task Editor.  Even if you click the cursor on the line and add a mass, it is overridden by the position of the mouse highligher (even if it just drifts there - no click) when you press enter.  This can mean that masses don't get added as intended, or that masses get restored when you meant to add a different one.  See video:
https://youtu.be/eV6OdGHL8JE

Improved UI so button only allows adding masses; restore requires re-entry using add; and background mass index is set with a spinner.  Available in next release.


i am working on SQL Langchain to ask natural english questions and get answers from postgress sql but not getting any answers

Discussed in https://github.com/langchain-ai/langchain/discussions/15928

Originally posted by BalajiPolisetty2207 January 11, 2024
i am using this langchain to query my postgres sql with natural language.
with sql server i am successful in asking and getting answers .
I am using azure open ai key and endpoint and gpt 4 model and deployment .
Can some one help what is missing here to query postgressql and how i can set the schema before querying.
Below is the code
Importing necessary libraries
from sqlalchemy import create_engine, text
from langchain.chat_models import AzureChatOpenAI
from langchain.prompts.chat import ChatPromptTemplate
from langchain.agents import AgentType, create_sql_agent
from langchain.sql_database import SQLDatabase
from langchain.agents.agent_toolkits.sql.toolkit import SQLDatabaseToolkit
import os
db_url = 'postgresql://asdas:e?Yasdad@asdasd-nonprod.postgres.database.azure.com:5432/asdad'  # PostgreSQL connection
openai_api_key = 'asdsadad'  # Azure OpenAI key
openai_model = 'gpt-4'  # OpenAI model, like GPT-4
openai_deployment = 'opeanaigpt4'  # OpenAI deployment name
os.environ["OPENAI_API_KEY"]  =  'xxxx'
os.environ["OPENAI_API_BASE"]="https://asdasd.openai.azure.com/"
os.environ["OPENAI_API_TYPE"]="azure"
os.environ["OPENAI_API_VERSION"]="2023-07-01-preview"
Creating the database engine
engine = create_engine(db_url)
Setting up the Azure OpenAI for LangChain
llm = AzureChatOpenAI(
model=openai_model,
deployment_name=openai_deployment,
temperature=0
)
Setting up the prompt template
final_prompt = ChatPromptTemplate.from_messages(
[
("system",
"""
You are an AI assistant expert in querying the PostgreSQL database.
you are going to respond the question on the students schema.
"""
),
("user", "{question}\n ai: "),
]
)
Creating the SQL Database object for LangChain
db = SQLDatabase(engine)
Setting up the SQL Toolkit for LangChain
sql_toolkit = SQLDatabaseToolkit(db=db, llm=llm)
sql_toolkit.get_tools()
Creating the SQL Agent for LangChain
sqldb_agent = create_sql_agent(
llm=llm,
toolkit=sql_toolkit,
agent_type=AgentType.ZERO_SHOT_REACT_DESCRIPTION,
verbose=True,
handle_parsing_errors=True,
)
Example usage of the agent to answer a question
question = "get me list of tables in the students schema ?"
final_prompt.messages[-1] = ("user", f"{question}\n ai: ")
response = sqldb_agent.run(final_prompt)
print(response)

its working now.. thanks .


DefaultCryptoMaterialsManager docstring incorrectly indicates that algorithm is a configurable parameter

DefaultCrytoMaterialsManager docstring incorrectly indicates that algorithm is configurable.  Behavior is as intended, but I missed clearing out this docstring.
https://github.com/awslabs/aws-encryption-sdk-python/blob/b6a2bbca4db6ab46d65fe7e2eacfffe25abff3a2/aws_encryption_sdk/materials_managers/default.py#L26

This is addressed in #12


[Cowswap] API Price Strategy check endpoint

Hi guys - as we mentioned briefly some weeks back, would be great to have an endpoint returning the best price strategy for querying prices. Something we could use below that returns which endpoint to use: COWSWAP or LEGACY (names can be whatever)

where we can check and in case of a change, fallback and continue offering service to the frontend:

@fleupold moved here

Discussing our options on Slack. Either:

Host static file on Github
Host static file on our Nginx ingress

Closing since, AFAICT, this will be configured with our Github configuration.

This would have come in handy today.

AFAIK this was implemented in v1.8.0. Specifically: https://github.com/gnosis/cowswap/pull/1835


Configure diego overcommit

We've been talking about overcommitting diego, and we're currently overcommitting memory on staging, but we should do more research before configuring production to overcommit. Some questions to answer:

Should we overcommit at all?
If so, by how much?
How will we get alerted if we overprovision a box and tenant performance start to suffer?

@sharms @cnelson

How much is DEA overcomitted in east?  Probably a good place to start.

Should we overcommit at all?
I think so, most people don't use all the RAM on their apps

If so, by how much?
10x might be too much but 5x IMO is about right

How will we get alerted if we overprovision a box and tenant performance start to suffer?
We should get alerts if a box is running out of disk or ram regardless of this so we should get something...


[Benchmark] Support Video MCQ with TaskMeAnything-v1-video-random as an example

Hi,
For this PR

I added TaskMeAnything-v1-video-random video benchmark which includes 2700 video mcq questions.
Along with the benchmark, I found that unlike image mcq, video datasets don’t have a video_mcq.py file, which might hard for adding other mcq video benchmark. Therefore, I implemented video_mcq.py following the logic of image_mcq.py.

The usage of video_mcq.py is the same as image_mcq.py:
Just convert the benchmark to a TSV file, and encode the MP4 video to base64. I have provided the function named mp4_to_base64 in vlmeval/dataset/utils/video_mcq_utils.py.
I added the TaskMeAnything-v1-video-random video benchmark as an example for video_mcq.py and tested it on Paligemma (ImageQA model) and Video-LLaVA (VideoQA model), and it works well.

Hi, @weikaih04. We found that TaskMeAnything_v1_imageqa_random has a minor typo. The field of tsv file spelled category incorrectly, resulting in the final calculation result can only output the overall score, but not the category.

@junming-yang  Hi, thank you for pointing this out! I’ve uploaded new versions of ImageQA and VideoQA and changed the md5, it should work now.

hey @junming-yang do you think it's ready to merge?

Hi, @JieyuZ2 , I may need to take a look at this PR and run the evaluation. I will try to complete this in this week.

Thanks!


ArmorType "Monk" leads to errors

See for example: https://www.aonprd.com/NPCDisplay.aspx?ItemName=Spry Ambusher
Add Monk to the valid armor types, but handle it like "rage" or other types that are not represented in the system.

ArmorType "wis" as well. Resolved in 3.1.1


Client authentication with JWT assertion

Ready for preliminary review, yet still there are several things to update plus a bunch of questions.
Question: I've factored out PKCE verification part from OAuth2ClientAuthenticationProviderTests and reused it in client assertion authentication provider. Should it be merged as part of this change or separately ?
Question: decision on how to map (decoded and verified) client assertion onto registered client is delegated to RegisterClientResolver interface. While there is default (trivial) implementation, I'm wondering if it is 'battery included' solution ? Should be extended somehow ?
For now PR consists of several commits, will be merged into a single big commit after review+corrections cycles.
Issue gh-59
Note: new client authentication methods constants will need to be aligned with what was implemented in spring-security oauth2 client recently (v. 5.5).

I've updated PR, so now PKCE verification is hidden from client code. Not sure if there is more that can be remove from public APIs at the moment. Yet, there are two issues: adapting to new customization API (#139), implementing client_secret_jwt and private_key_jwt instead of currently implemented one generic jwt client authentication method (with batteries not included). So @jgrandja, for now this PR is probably not worth your time when it comes to reviewing, I'm waiting until customization parts settle down.

Hi @rlewczuk. I'm finally circling back to this PR as we're looking to add this feature in 0.2.1. Are you still available to work on this?

Hi,
Sorry for not reacting on it, I was involved in significant project
recently.
I'll try to look at it over weekend and get back with some code next week.
Regards,
rle
On Thu, Sep 23, 2021 at 9:45 PM Joe Grandja @.***>
wrote:

Hi @rlewczuk https://github.com/rlewczuk. I'm finally circling back to
this PR as we're looking to add this feature in 0.2.1. Are you still
available to work on this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spring-projects/spring-authorization-server/pull/293#issuecomment-926103590,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMPMH4ERCLWJFEBUZ2XZEDUDN7VFANCNFSM4466ISJA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@jgrandja  I've implemented converter and provider parts of this in a way that full client assertion verification (signature and against registered client repository) is delegated to a specialized JwtDecoder instance. This keeps authentication provider minimal and delegates most of work to decoder. Decoder which will validate assertion against registered client repository still needs to be implemented but I'm not sure this is valid way of thinking, So I'm stopping here for a moment and await for feedback. Can I ask for preliminary review and green light to go with decoder as described above (or if this is bad - hint which way to go) ?
Thanks and Regards,
rle

@jgrandja Implemented as per review comments with some remarks below.
I wasn't able to use standard JwtDecoderFactory interface in authentication provider to create JWT decoders as two parameters are needed (registered client and authentication method); with only registered client as parameter two token validation methods will exclude themselves (shared secret and private key).
Question regarding validation: RFC states that aud and iss claims are required. Should we strictly validate those ? Now I'm only checking iss if token issuer is configured in registered client settings. Where can I get external server URL (or should audience field also be part of registered client settings) ?
Sorry for noob questions, I'm still learning logic behind design of Spring projects :)
Can I ask for review ?
Regards,
rle

While we cannot trust in principle JWT tokens prior to verification, I wonder if we can use JWT headers. Digging through nimbus implementation and NimbusJwetDecoder I see that it always parses header before verification. It always uses alg field in com.nimbusds.jwt.JWTParser.parse() method. I wonder if we can do the same in converter to determine exact client authentication method. In certain configurations kid fields is also used.
Currently we declare a synthetic client authentication method (in fact twice - in converter and authentication provider in order to not expose it), then we add a bunch of clumsy logic OAuth2ClientAuthenticationProvider.authenticate to dispatch it and on top of that we have to work around fact that JwtDecoder has single argument and this we pack arguments into context object, which further complicates things (mostly in unit tests, plus we are forced to cram two separate pieces of logic and switch into single decoder factory).
I'm writing this because - frankly - I suspect that  currently we are adding lot of complexity only to work around non-existent issue. Determining correct authentication method earlier would make things straight and as we already trust JWTParse.parse() prior to verification we are free to use it to know exact algorithm early enough (determine authentication method knowing it).
@jgrandja what do you think ?

We are ready for next round of review. Design and implementation seems to be complete (i.e. won't change without feedback from review), unit tests are here. One missing thing are integration tests. I see some implemented in configurers.oauth2.server.authorization tree. Is this particular change suitable for integration tests ? (I'm thinking of register client -> use client scenario)

Now it's complete as far as I can tell. Please review. Sorry for that chaos :)

Thanks for the updates @rlewczuk.
FYI, I'm on PTO Nov 19-24 (inclusive) and the release is Dec 1, so ideally I'd like to merge this week.
I'll start the review tomorrow and will add a polish commit (if needed) to get it merged this week.
Thanks.

Ok, I'll be available tomorrow, so if there is still something you want me to do I can react quickly. Now I'll rebase it against latest commits on main branch and push (i.e. no more changes here, only rebase and tests). Thanks.

@jgrandja all changes from review are done plus I discovered some missing parts in oauth2 discovery and oidc discovery endpoints. Amount of changes grew, it becomes a monster patch of more than 2000 LOC.
One of recent changes for resolving issuer from current request did wreak a bit of havoc: I've resigned from using statically configured issuer URL from ProviderSettings and using value resolved in OAuth2AuthorizationServerMetadataEndpointFilter. This had (potential) impact on RegisteredClientJwtAssertionDecoderFactory which would require not only RegisteredClient but also proper issuer URL. I decided against changing it as it caused unnecessary complications in both caching and subclassing OAuth2AuthenticationContext. Instead I'm verifying aud claim directly in OAuth2ClientAuthenticationProvider. For reference implementation with aud verification in decoder is here: https://github.com/rlewczuk/spring-authorization-server/tree/jwt-client-auth-aud-verifier. Also, I'm verifying URI against endpoint client is calling (/oauth2/token, /oauth2/introspect or /oauth2/revoke). I didn't find anything in RFCs clarifying this, so I'm guessing aud should point to appropriate endpoint, not just blindly on /oauth2/token.
I'm marking PR as ready for review. I'll probably also look myself at it in coming days, so some small patches/cleanups/fixes/tests may appear but it won't cause any major changes.
Thanks for passing those fixes to me, not just proceeding with polishing commit. In terms of climbing up the learning curve that last round was probably the biggest uplift for me.
Note: I'll be absent on Monday, 29th, will be available on all remaining days this month.
Thanks and Regards,
rle

Thanks for all the updates @rlewczuk.
We're definitely getting close but we'll have to continue this after 0.2.1 is released.
I don't want to rush this as there are quite a few changes.
Also, I'm thinking about breaking up this PR into smaller PR's so it's easier to consume and more efficient to review.
But please hold off for now until I come up with a plan.
I'll get back to you later next week after the release is out.
Thanks for all your work!

Yep, at some point I also started thinking if one big commit is valid approach. Split into authentication (on preconfigured client), OIDC registration and discovery parts seems to be natural.

Hi @jgrandja. I've rebased code onto newly released version. Now one thing has to be decided: do we split this PR into parts or try to proceed as it is. As we are at the beginning of iteration, there is time to fix flaws in subsequent PRs (granted that there are no serious ones).

@rlewczuk We're going to continue with this PR as we're pretty close and it's not needed to split up at this point.
Before I continue with next review, please apply these changes:

Remove token_endpoint_auth_signing_alg_values_supported, revocation_endpoint_auth_signing_alg_values_supported and introspection_endpoint_auth_signing_alg_values_supported. These provider configuration metadata are OPTIONAL and is not needed in this PR. We can consider adding it in a subsequent PR.
Make RegisteredClientJwtAssertionDecoderFactory as an inner private static class in OAuth2ClientAuthenticationProvider.

I should be available to review within the next couple of days. Thank you.

Both changes are applied.

Thanks for all the updates @rlewczuk.
After reviewing, I found a few more changes were required (especially in tests) and my goal was to have this merged before end of week, as I'm off for the following 2 weeks. So I decided to add a polish commit to get this merged.
Please let me know if you have any questions in regards to the changes applied in the polish commit.
Thanks again for all your time in contributing to this very important feature !

Quite extensive changes, I have some questions (of "how things work here" kind) but I need to dig into this polish commit a bit deeper first.
I have two general questions:

I've set aside patch with additional server metadata as per your previous request; can I post PR with this change now ?

Can I start work on client assertion grants (second part of this RFC) ?

Regards,
rle

Regarding JwtClientAssertionDecoderFactory caching: I see that cache refresh logic has been removed and replaced with simple .computeIfAbsent() call. What is rationale behind it ? Is it assumed that client registrations are immutable once created ?

@rlewczuk

I've set aside patch with additional server metadata as per your previous request; can I post PR with this change now ?

The patch would not work as you specified all the JWA algorithms and that would not be the case for all deployments. It depends what keys are configured in JWKSource<SecurityContext> per deployment and likely it would be a very small set of the superset. This would differ is each deployment. Given this is OPTIONAL and IMO is not really needed, we'll leave it out for now.

Can I start work on client assertion grants (second part of this RFC) ?

Please log a new issue and we'll see if this is a priority issue based on upvotes. There are other features that are higher in priority compared to this one so we'll leave this on hold for now.

Regarding JwtClientAssertionDecoderFactory caching...

I consider the "refresh cache" logic a potential future enhancement. It's not needed at this time and I'd rather keep things as simple as possible with the initial implementation. We might introduce this at a later point if it becomes an issue.


Cant Open a new path with epic games launcher

hello, I have a problem where I tried to put the path to the epic games launcher but... it didn't do anything,
so please add one for the epic games launcher ty.

I have the same issue.  Clicking browse button returns error "The path is not of a legal form"
I'm on latest version of Windows 11

Same here

same on epic

Same

same

sane

the issue got fixed just update the program and everything will be fine


Add fuzzing infrastructure

This adds two fuzzers targeting trillium_http::ReceivedBody. One is particularly targeted at an infinite loop problem, across multiple polls, and the other is more generic, just looking for crashes. I added one input that triggers the infinite loop issue by choosing chunked encoding and then immediately returning an EOF after the headers.
.../trillium/fuzz$ cargo afl run --bin received_body_hang < in/received_body_hang/chunked_body_immediate_eof
    Finished dev [unoptimized + debuginfo] target(s) in 0.02s
     Running `target/debug/received_body_hang`
Input: FuzzInput { content_length: None, socket_reads: SocketReads { reads: [] } }
[]
did not finish reading body after 1000 polls
woke up 1000 times

I tried to suppress crashes due to OOM allocation failures in the infinite loop fuzzer, by providing a global allocator that exits the program with success on allocation failure. This isn't entirely effective, because some paths in the standard library perform fallible allocations, and then panic upon failure. (as opposed to the allocator itself aborting) Configuration from afl or cargo-afl turns this panic into an abort. These OOM issues ought to be addressed for DoS reasons anyway, and that will let the fuzzers progress past shallow bugs.
FWIW I am slightly regretting my choice to use AFL++ instead of libfuzzer, because I recalled late in the game that the Rust crates for the latter have nicer support for structured fuzzing, and would have let me remove dumping Debug representations from the fuzzing target itself.

Thank you so much for this!


Figure out how to force cargo fix to run

Moved over from https://github.com/rust-lang-nursery/rustfix/issues/79, I hear @killercup has a fix in mind!

Bumping this to the RC1 milestone

I sort of forget the original rationale for this, but one articulation of a use case is:

First, run cargo fix --edition
Realize that you have a ton of warnings, and it turns out that cargo fix has a bug in it. A big message was printed at the top indicating so, and you'd like to see it again.
Rerun cargo fix --edition and this time redirect the output to a file so you can read over it later

The second fix doesn't actually do anything!

Looks like #5944 might be doing the trick.


The same person appears twice as author

Onefetch shows the same person twice as author (I'm guessing it's because it's seeing different e-mail address?) It might be a good idea to just addition those and displaying them as being from the same author.
Repository: https://github.com/iv-org/invidious
Onefetch version 2.11.0
Authors: 39% Omar Roth 1402
              13% Omar Roth 457
              9% Samantaz Fox 335

@spenserblack Interesting, thanks!
Solved then.


How to get the large-scale dataset of Tweets？ Thanks

Sorry, due to the copyright. We can not share the dataset. You can collect that following collect_script


Add missing JS instrumentations to registry

All Node.js contrib instrumentations are now in. Thanks @Rauno56


Animation status callbacks should be re-entrant

We should mutate the status of the AnimationController before calling
the status callbacks so that we're prepared to be re-entered.
Fixes #1911

@Hixie

lgtm


feat(Editor): onInit callback

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe the problem.
Hi i mentioned earlier that i wanted to change <p> to <br> onEnter and it seemed to work great, but I also need onInit to be exposed too. I dont really know how to create a PR to be completely honest
Describe the solution you'd like
here is my implementation of what the change would look like
Editor.razor.js.zip
Additional context
here is a sample project of it in action
BootstrapBlazor.OnlyServer1.zip

@celadaris bump version 9.0.0-beta06

After further testing I noticed there was a memory leak, I asked claude.ai to work its magic and its working well for me so far, please carefully examine the code to make sure it works okay.
files that need to be changed:
LeakFix.zip
sample project of it in action:
BootstrapBlazor.OnlyServer1.zip

quest 1
const editorClickHandler = async () => {
    editor.tooltip.hide()
    option.placeholder = editor.editorElement.getAttribute('placeholder')
    const toolbar = await invoker.invokeMethodAsync('GetToolBar')
    // ...
}

there is no invoker define.
quest 2
can you explain in detail how to cause a memory leak and how it is caused in the code?
qest 3
how to fixed this leak?

quest 1:
i really honestly have no idea how summernote really works, ive been using claude for just about all of this, check out the last responses i hope it helps
Claude-Identifying and Fixing Memory Leaks.md
quest 2:
i was only having problems on a server at work, which was why it took time between the first post and my suggested second post to see anything wrong, I dont know how to reproduce the memory leak on my work pc or my home pc, only on the work server when its deployed. i can give it a shot but it would have to be over the weekend because im too busy during the week. i can try to give you code.
the problem was something like i had a summernote Editor being launched in a Dialog and for some reason loading it would make mjml.net render inconsistent results, it felt like a memory leak, and when i asked claude.ai to fix memory leaks they were gone.
quest 3:
i have no idea, but the code from the LeakFix.zip sure does work for me! Lmao, im at home right now i cant access my other code right now. but i didnt really change anything from LeakFix.zip, its been working well so far. it might have problems im not sure...

okay, after reviewing the code i was using in my project i forgot to use the changed Editor.razor.cs file! And everything seems to be working so i guess there's no problems!
So because i am using an unchanged file and im not seeing any problems i guess im good 😅
I personally need light mode for my project because my users request it, so i will be using the summernote editor from the source code (just like how i have it in the project below) and will be exposing onEnter and onInit myself, however it would still be cool to see those callbacks exposed in the official bootstrapblazor.summernote project, i think they were removed from the latest src files, anyway i think that's all i dont know if this answers questions? please let me know if you need anything, if not you can close this...
BootstrapBlazor.OnlyServer1.zip

my apologies i didnt see this line of code yesterday:
const reloadCallbacks = (id, option) => {
    const events = ['Blur', 'BlurCodeview', 'Change', 'ChangeCodeview', 'DialogShown', 'Enter', 'Focus', 'ImageLinkInsert', 'ImageUploadError', 'Init', 'Keydown', 'Keyup', 'Mousedown', 'Mouseup', 'Paste', 'Scroll'];

    events.forEach(event => {
        option.callbacks[`on${event}`] = function () {
            const callbacks = window.BootstrapBlazor?.SummerNote?.callbacks;
            const cb = callbacks?.find(i => i.id === id);
            cb?.[`on${event}`]?.apply(this, arguments);
        };
    });
}

ive tested how it works with my project and it works great! if there are no questions than this can be closed.

i think there is a small problem in the latest 9.0.0 when ShowSubmit="false" the value associated with @bind-Value isnt being changed and the OnValueChanged callback isn't being triggered.
SummernoteTest.zip

@celadaris Thanks for submitting the issue. I will reproduce the issue later and try to fix it.

@celadaris bump version 9.0.1
https://www.nuget.org/packages/BootstrapBlazor.SummerNote/9.0.1
please let me know if you have any questions.


remove padding and change note

*Issue:*https://github.com/bcgov/entity/issues/22109
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

/gcbrun

Temporary Url for review: https://bcregistry-btr-dev--pr-200-wvebzjw0.web.app


wip add julia 1.7 support

not yet ready, contains debug code. My attempt to support julia 1.7

@dfdx can you approve running CI?

CI checks are all green and I see no more action items here for now. Are we ready to merge?

Yes we are ready!


Files to use this project

Hey, FYI your instructions say use the files from NDParallaxIntroView, should be use the files from NDParallaxIntroViewExample.
Good project by the way.
thanks

The files in NDParallaxIntroViewExample/NDParallaxIntroView/ and in /NDParallaxIntroView should be the the same ones - so it shouldn't matter which ones to integrate.


New notebook for using the LLM

The LLM can be any of the open source LLM.

I can add a new LLM-trained notebook which is trained on WhatsApp chats.

Feel free to do PR.


Added setSpawnLocation command

The PlayerSystem receives the RespawnRequestEvent and finds the location from the world generator.
This behavior has been modified such that it now checks for the existence of a SpawnLocationComponent attached to the ClientInfo Entity. If the component exists, the location is not found from the world generator and instead used directly.
The command that allows the player to set the spawn location is setSpawnLocation.
 Notice that the spawn location changes every time.

Hooray Jenkins reported success with all tests good!

Hooray Jenkins reported success with all tests good!


ImportError: cannot import name 'build_sampler' from 'mmdet.core.bbox.assign_sampling'

I have successful install the mmdetection by the command "pip install -v -e .". But I have the problem in the test. Would anyone help me ?
(lab) gpuserver@ubuntu:~/ht/labs/mmdetection-master$ python
Python 3.7.3 (default, Mar 27 2019, 22:11:17)
[GCC 7.3.0] :: Anaconda, Inc. on linux
Type "help", "copyright", "credits" or "license" for more information.

from mmdet.apis import init_detector
Traceback (most recent call last):
File "", line 1, in
File "/home/gpuserver/ht/labs/mmdetection-master/mmdet/apis/init.py", line 2, in
from .inference import inference_detector, init_detector, show_result
File "/home/gpuserver/ht/labs/mmdetection-master/mmdet/apis/inference.py", line 9, in
from mmdet.core import get_classes
File "/home/gpuserver/ht/labs/mmdetection-master/mmdet/core/init.py", line 1, in
from .anchor import *  # noqa: F401, F403
File "/home/gpuserver/ht/labs/mmdetection-master/mmdet/core/anchor/init.py", line 2, in
from .anchor_target import anchor_inside_flags, anchor_target
File "/home/gpuserver/ht/labs/mmdetection-master/mmdet/core/anchor/anchor_target.py", line 3, in
from ..bbox import PseudoSampler, assign_and_sample, bbox2delta, build_assigner
File "/home/gpuserver/ht/labs/mmdetection-master/mmdet/core/bbox/init.py", line 1, in
from .assign_sampling import assign_and_sample, build_assigner, build_sampler
File "/home/gpuserver/ht/labs/mmdetection-master/mmdet/core/bbox/assign_sampling.py", line 3, in
from . import assigners, samplers
File "/home/gpuserver/ht/labs/mmdetection-master/mmdet/core/bbox/samplers/init.py", line 2, in
from .combined_sampler import CombinedSampler
File "/home/gpuserver/ht/labs/mmdetection-master/mmdet/core/bbox/samplers/combined_sampler.py", line 1, in
from ..assign_sampling import build_sampler
ImportError: cannot import name 'build_sampler' from 'mmdet.core.bbox.assign_sampling' (/home/gpuserver/ht/labs/mmdetection-master/mmdet/core/bbox/assign_sampling.py)

I encountered the same error today.

same problem here.

@hjinlee88 @hellock Thank you . Above  problem has been solved. But I have another problem.
Welcome to Ubuntu 16.04.6 LTS (GNU/Linux 4.4.0-122-generic x86_64)

Documentation:  https://help.ubuntu.com
Management:     https://landscape.canonical.com
Support:        https://ubuntu.com/advantage

35 packages can be updated.
16 updates are security updates.
New release '18.04.2 LTS' available.
Run 'do-release-upgrade' to upgrade to it.
*** System restart required ***
Last login: Thu Aug  1 16:20:57 2019 from <IP_ADDRESS>
/usr/bin/xauth:  /home/gpuserver/.Xauthority not writable, changes will be ignored
gpuserver@ubuntu:~$ source activate open-mmlab
(open-mmlab) gpuserver@ubuntu:~$ cd ht/labs/mmdetection-master/
(open-mmlab) gpuserver@ubuntu:~/ht/labs/mmdetection-master$ python
Python 3.7.3 (default, Mar 27 2019, 22:11:17)
[GCC 7.3.0] :: Anaconda, Inc. on linux
Type "help", "copyright", "credits" or "license" for more information.

from mmdet.apis import init_detector
Traceback (most recent call last):
File "", line 1, in
File "/home/gpuserver/ht/labs/mmdetection-master/mmdet/apis/init.py", line 2, in
from .inference import inference_detector, init_detector, show_result
File "/home/gpuserver/ht/labs/mmdetection-master/mmdet/apis/inference.py", line 9, in
from mmdet.core import get_classes
File "/home/gpuserver/ht/labs/mmdetection-master/mmdet/core/init.py", line 6, in
from .post_processing import *  # noqa: F401, F403
File "/home/gpuserver/ht/labs/mmdetection-master/mmdet/core/post_processing/init.py", line 1, in
from .bbox_nms import multiclass_nms
File "/home/gpuserver/ht/labs/mmdetection-master/mmdet/core/post_processing/bbox_nms.py", line 3, in
from mmdet.ops.nms import nms_wrapper
File "/home/gpuserver/ht/labs/mmdetection-master/mmdet/ops/init.py", line 2, in
from .dcn import (DeformConv, DeformConvPack, DeformRoIPooling,
File "/home/gpuserver/ht/labs/mmdetection-master/mmdet/ops/dcn/init.py", line 1, in
from .deform_conv import (DeformConv, DeformConvPack, ModulatedDeformConv,
File "/home/gpuserver/ht/labs/mmdetection-master/mmdet/ops/dcn/deform_conv.py", line 9, in
from . import deform_conv_cuda
ImportError: /home/gpuserver/ht/labs/mmdetection-master/mmdet/ops/dcn/deform_conv_cuda.cpython-37m-x86_64-linux-gnu.so: undefined symbol: __cudaPopCallConfiguration

The another problem has been solved.
"export CUDA_HOME=$CUDA_HOME:/usr/local/cuda"  modify  to
"export CUDA_HOME=/usr/local/cuda"
, and reinstall the mmdetection.


✨ add REST support for Training

Adds REST support for Training APIs.
Depends on https://github.com/caikit/caikit/pull/312 for internal code refactoring

Issues with DataStream
A typical DataStreamSourceSampleTrainingType object gives as json:
'{"jsondata": {"data": [{"number": 1}, {"number": 2}]}}'

vs the pydantic model gives:
'{"data_stream":{"data":[{"number":1}]}}'

One way to get around this is to add the ability to use the oneof field name while constructing from_json, for example:

@dataobject
class BazObj(DataObjectBase):
    @dataobject
    class Foo(DataObjectBase):
        data: List[str]

    @dataobject
    class Bar(DataObjectBase):
        data: str

    data_stream: Union[
        Annotated[Foo, FieldNumber(1), OneofField("foo")],
        Annotated[Bar, FieldNumber(2), OneofField("bar")],
    ]

foo1 = BazObj(foo=BazObj.Foo(data=["hello"]))

This works:
BazObj.from_json({"foo": {"data": ["hello"]}})

but ideally we want this to work as well:
BazObj.from_json({"data_stream": {"data": ["hello"]}})

This is turning out to be harder than I thought. I need to implement some sort of _infer_which_oneof but in json, which can correctly identify which oneof field the data actually corresponds to.
So in the example above, {"data": ["hello"]} can fit in foo, so data_stream has to be replaced with foo in the from_json call.

superseded by https://github.com/caikit/caikit/pull/331


Missing auto import suggestions for UMD modules such as classnames

VSCode Version: 1.51.1
OS Version: 11.0.1

Steps to Reproduce:
import * as React from 'react';

const el =
foo
;

yarn add @types/classnames
Erase REPLACE_ME and begin typing classNames

Does this issue occur when all extensions are disabled?: Yes
Expected: VS Code suggests an auto import of classNames from module 'classnames'.
Actual: no suggested auto import.

I believe this is something to do with the fact that classnames is defined as a UMD module. If I change the type definitions for classnames to remove the UMD definition, the auto import does work correctly:

Note that VS Code does correctly provide import suggestions after we've finished typing the identifier, when we move our cursor.

This issue is specifically about auto imports inside the suggestions that appear as you're typing.

Would someone guide me how I can easily get to a point to reproduce this locally?
Not very familiar how code fixes work for<EMAIL_ADDRESS>and requesting a code completion at a specific point.

It turns out this case happens cause the completion ends up within a {..} and current implementation looks up for destructuring of an object.
Trying to find a good spot to add the completion logic...

@OliverJAsh is the issue occurs within JSX/TSX language variant or JS?

@OliverJAsh is the issue occurs within JSX/TSX language variant or JS?

I think I only tested it with TSX. Not sure if it occurs in other languages.

@OliverJAsh would you also provide a simplified version of your TS config? specially how you configured the modules.
there are settings that I'm interested to see:
https://www.typescriptlang.org/tsconfig#module
https://www.typescriptlang.org/tsconfig#esModuleInterop
https://www.typescriptlang.org/tsconfig#allowSyntheticDefaultImports

To be honest at this point I'm not sure if this is a bug or the correct behaviour. The code fix works though I'm not sure if auto import should be provided or even how auto import works in TypeScript/VSCode.
I would need some help to investigate more.
Added a new fourslash test here:
https://github.com/microsoft/TypeScript/pull/4214

@aminpaks I am using the default settings as I have no tsconfig.json. Repo with reduced test case: https://github.com/OliverJAsh/ts-umd-auto-import-suggestions-bug
.

@OliverJAsh yes this is exactly what I observed by looking at the code and writing that unit-test.
If you try to write class VSCode will offer you classNames as a completion, after you write full classNames TypeScript will offer you a code fix to add classnames as a new import.
Here I'm not sure if auto import should (if yes what is the default behaviour) be provided when you complete writing classNames and press enter.

Here I'm not sure if auto import should (if yes what is the default behaviour) be provided when you complete writing classNames and press enter.

Most users nowadays choose to import UMD modules rather than relying on the globals provided by the UMD (that's the best practice now) so I think the majority of users would find this useful.
Just to make sure we're on the same page, this is what I'm asking for: the import should be suggested in the list as the user types, in the same way that all other non-UMD modules appear in the list.

@OliverJAsh that's exactly what I expect to happen. As you can see in the test that I wrote it checks for both. Auto import by completion and code fixes.
I don't know why in provided project auto import stops working though!

@RyanCavanaugh I would need some help to continue working on this issue. If someone on the team would clarify how TypeScript determines auto imports from node_modules/@types in code completion would really help.


Could not load the video: AbortError: BodyStreamBuffer was aborted

What is the reason for the error "Could not load the video: AbortError: BodyStreamBuffer was aborted" when uploading a video to the web page? How to solve it?

Hi @zhousheng97，I should have just fixed this #182 in version 2.0.4. Could you please double check the version you were using? If you're using the latest one, please send me one example video and may I know which browser are you using?

Hi @zhousheng97，I should have just fixed this #182 in version 2.0.4. Could you please double check the version you were using? If you're using the latest one, please send me one example video and may I know which browser are you using?

Sorry, I did not notice the update of the version, I have changed the new version, there is no problem at present, thank you very much!


From private repo

@huypub : Thanks for your contribution! The author, @PatAltimore, has been notified to review your proposed change.


[Event Hubs] Fix unit test: round-tripping AMQP encoding/decoding

Debugging https://github.com/Azure/azure-sdk-for-js/issues/19547

/azp run js - event-hubs - tests

/azp run js - event-hubs - tests

/azp run js - event-hubs - tests

/azp run js - event-hubs - tests


YARN-10520. Deprecated the residual nested class for the LCEResourceHandler

https://issues.apache.org/jira/browse/YARN-10520

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 32s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:

0m  0s
test4tests
The patch appears to include 2 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
33m 11s

trunk passed

+1 :green_heart:
compile
1m 15s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
compile
1m 10s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
checkstyle
0m 32s

trunk passed

+1 :green_heart:
mvnsite
0m 46s

trunk passed

+1 :green_heart:
shadedclient
16m 25s

branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 38s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
0m 33s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+0 :ok:
spotbugs
1m 24s

Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
1m 22s

trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 39s

the patch passed

+1 :green_heart:
compile
1m  6s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javac
1m  6s

hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-nodemanager-jdkUbuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04 with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04 generated 0 new + 55 unchanged - 1 fixed = 55 total (was 56)

+1 :green_heart:
compile
1m  1s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
javac
1m  1s

hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-nodemanager-jdkPrivateBuild-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01 with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01 generated 0 new + 34 unchanged - 1 fixed = 34 total (was 35)

+1 :green_heart:
checkstyle
0m 24s

the patch passed

+1 :green_heart:
mvnsite
0m 35s

the patch passed

+1 :green_heart:
whitespace
0m  0s

The patch has no whitespace issues.

+1 :green_heart:
shadedclient
14m 34s

patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 33s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
0m 30s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
findbugs
1m 21s

the patch passed

_ Other Tests _

+1 :green_heart:
unit
22m 35s

hadoop-yarn-server-nodemanager in the patch passed.

+1 :green_heart:
asflicense
0m 33s

The patch does not generate ASF License warnings.

102m 29s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2527/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2527

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux e2d215c6d117 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / da1ea2530fa

Default Java
Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2527/1/testReport/

Max. process+thread count
609 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2527/1/console

versions
git=2.17.1 maven=3.6.0 findbugs=4.0.6

Powered by
Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Thanks @adamantal for the review.

Thanks for working on this, committed to trunk!


Good Tables gives confusing error message re: headers

Running
goodtables schema --schema https://raw.githubusercontent.com/okfn/tabular-validator/master/examples/hmt/spend-publishing-schema.json  https://www.gov.uk/government/uploads/system/uploads/attachment_data/file/214660/Wales-Office-spend-over-500-June-2011.csv

Returns

There is a mismatch between the headers according to the schema, and those found in the data. The schema says the headers should be: Department family, Entity, Date, Expense type, Expense area, Supplier, Transaction number, Amount, Description, Supplier postcode, Supplier type, Contract number, Project code, Expenditure type, VAT registration number.

This is a confusing message given that only some of those listed headers are actually required according to the schema.

Duplicates https://github.com/frictionlessdata/goodtables/issues/70


import statements in bug-introducing patches

Hi @rjust and @Greg4cr,
I've been analyzing the buggy lines of the projects/bugs introduced in v1.5.0 and v2.0.0 and I got confused regarding buggy/fixed import statements and even more confused after reading the Patch Minimization Guide, in particular Section 2.1.
I've just analyzed all bug-introducing patches that:

remove import statements from the fixed version to create the buggy version
modify import statements
add new import statements to the buggy version

and here is the breakdown.

Bug-introducing patch
Action
Defects4J version

Math-12
remove imports
< v1.5.0

Mockito-2
remove imports
< v1.5.0

Mockito-10
remove imports
< v1.5.0

Mockito-14
remove imports
< v1.5.0

Mockito-15
remove imports
< v1.5.0

Mockito-17
remove imports
< v1.5.0

Mockito-19
remove imports
< v1.5.0

Mockito-25
remove imports
< v1.5.0

Mockito-31
remove imports
< v1.5.0

Mockito-32
remove imports
< v1.5.0

Mockito-36
remove imports
< v1.5.0

Mockito-39
remove imports
< v1.5.0

Closure-158
remove imports
= v1.5.0

Cli-10
remove imports
= v2.0.0

Codec-13
remove imports
= v2.0.0

Collections-21
remove imports
= v2.0.0

Collections-24
remove imports
= v2.0.0

Csv-7
remove imports
= v2.0.0

Gson-3
remove imports
= v2.0.0

Gson-8
remove imports
= v2.0.0

JacksonDatabind-103
remove imports
= v2.0.0

JacksonDatabind-109
remove imports
= v2.0.0

JacksonDatabind-20
remove imports
= v2.0.0

JacksonDatabind-21
remove imports
= v2.0.0

JacksonDatabind-38
remove imports
= v2.0.0

JacksonDatabind-43
remove imports
= v2.0.0

JacksonDatabind-50
remove imports
= v2.0.0

JacksonDatabind-55
remove imports
= v2.0.0

JacksonDatabind-60
remove imports
= v2.0.0

JacksonDatabind-66
remove imports
= v2.0.0

JacksonDatabind-73
remove imports
= v2.0.0

Jsoup-7
remove imports
= v2.0.0

Jsoup-16
remove imports
= v2.0.0

Jsoup-28
remove imports
= v2.0.0

Jsoup-29
remove imports
= v2.0.0

Jsoup-30
remove imports
= v2.0.0

Jsoup-36
remove imports
= v2.0.0

Jsoup-52
remove imports
= v2.0.0

Jsoup-58
remove imports
= v2.0.0

Jsoup-71
remove imports
= v2.0.0

Jsoup-73
remove imports
= v2.0.0

Jsoup-78
remove imports
= v2.0.0

Jsoup-79
remove imports
= v2.0.0

Jsoup-81
remove imports
= v2.0.0

Jsoup-87
remove imports
= v2.0.0

Jsoup-92
remove imports
= v2.0.0

JxPath-11
remove imports
= v2.0.0

JxPath-2
remove imports
= v2.0.0

JxPath-3
remove imports
= v2.0.0

JxPath-7
remove imports
= v2.0.0

JxPath-13
remove imports
= v2.0.0

JxPath-15
remove imports
= v2.0.0

JxPath-18
remove imports
= v2.0.0

Math-61
modify imports
< v1.5.0

Closure-137
modify imports
= v1.5.0

Cli-1
modify imports
= v2.0.0

Cli-7
modify imports
= v2.0.0

Cli-36
modify imports
= v2.0.0

Codec-14
modify imports
= v2.0.0

Closure-153
add imports
= v1.5.0

Closure-158
add imports
= v1.5.0

Cli-3
add imports
= v2.0.0

Cli-36
add imports
= v2.0.0

JacksonDatabind-63
add imports
= v2.0.0

JxPath-7
add imports
= v2.0.0

JxPath-9
add imports
= v2.0.0

Regarding (1) bug-introducing patches that remove import statements from the fixed version to create the buggy version, to the best of my knowledge, those bug-introducing patches avoid including ingredients in the buggy version that are part of the bugfix mostly because they would not communicate anything about the bug or the bugfix.
However, the documentation, in particular the example "Collections-19 Non-minimized vs. Minimized" described here, seems to point that the import statements originally added to fix the bug should remain in the buggy version created by Defects4J.
Am I missing something here or does the documentation need to be fixed?

Regarding (2) bug-introducing patches that modify import statements, e.g., an import foo.Bar; in the fixed version that needs to be modified to import bar.Foo; in the buggy version so that the buggy code compiles, I have no questions.

Regarding (3) bug-introducing patches that add new import statements to the buggy version, it seems we followed two different approaches.
a) In Defects4J < v1.5.0, we discarded new import statements and used the fully qualified classes names, e.g, Mockito-5 Non-minimized vs. Minimized.
b) In Defects4J >= v1.5.0, we left new import statements in the bug-introducing patches: Closure-153, Closure-158, Cli-3, Cli-36, JacksonDatabind-63, JxPath-7, and JxPath-9.
Should or should not the bug-introducing patches of these 7 bugs follow the patch minimization procedure we used in Defects4J < v1.5.0, i.e., discard new import statements and used fully qualified classes names?

As far as I remember, the Patch Minimization Guide was written after version 2.0.0, so I'm not too surprised to see some inconsistency from merging our interpretations of "patch minimization".
Regarding (1), the policy that I followed was that import statements added in a bug fix that are relevant to the bug fix should be removed from the buggy version.
I took a quick glance at the Collections-19 example, and I do not believe the import statements relate to the buggy behavior. Therefore, they were retained in the buggy version.
To me, this is consistent with the idea that the differences between the buggy and fixed versions should only relate to the buggy behavior, and not refactoring or other unrelated changes. Is there something different we should be doing here?
Regarding (3), using fully-qualified class names is arguably more "minimal" as measured by number of lines. It makes some sense to me to update the newer patches to follow the earlier policy. If we do this, we should also document this decision in the Patch Minimization Guide.

I took a quick glance at the Collections-19 example, and I do not believe the import statements relate to the buggy behavior. Therefore, they were retained in the buggy version.

Got it.

To me, this is consistent with the idea that the differences between the buggy and fixed versions should only relate to the buggy behavior, and not refactoring or other unrelated changes. Is there something different we should be doing here?

Perhaps we could improve Section 2.1 as
- 1. Changes made to import statements should be removed
+ 1. Changes made to import statements that are not relevant to the bug fix should be removed

Regarding (3), I could go ahead and do it on a separate branch.

I agree with your suggestion for improving Section 2.1.
This is a really interesting analysis. Thanks for doing it!


Connect with Metadata 2020

Recommendation 3. COPIM keeps a watching brief on this project and makes its findings available to Metadata 2020.

As of Q3 2023, Thoth now is a signatory of the Metadata 2020 pledge https://metadata2020.org/do-more/sign-the-pledge/
To Do:

[ ] Draft a writeup to compare Thoth's model against the Metadata2020 Best Practices https://metadata2020.org/resources/metadata-practices/
[ ] engage the Metadata2020 core group via email<EMAIL_ADDRESS>[ ] ...


Minor versions cannot be skipped

The page clearly states:

When you upgrade an AKS cluster, Kubernetes minor versions cannot be skipped.

However, when you come to upgrade from v1.13 you don't get the option to upgrade to v1.14; so presumably minor versions must be skipped?
# az aks get-upgrades --subscription $azSub --resource-group $azRG --name $azClus --output table
Name     ResourceGroup          MasterVersion    Upgrades
-------  ---------------------  ---------------  ----------------
default  myResourceGroup     1.13.12          1.15.11, 1.15.12

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7010bd58-42b1-5c90-3253-05deed855be8
Version Independent ID: c890efb2-cdaf-a3eb-d54f-0e7ccbd4314d
Content: Upgrade an Azure Kubernetes Service (AKS) cluster - Azure Kubernetes Service
Content Source: articles/aks/upgrade-cluster.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@JohnLBevan
Thanks for your Question! We are checking on this and will get back on this.

We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.


Fixes issue 2278

Description

Supplied missing arguments to iteration over files to 'diff', and added a logging statement to make it clear what is being compared.
Motivation and Context

Fixes #2278
How Has This Been Tested?

Tested interactively
Checklist:

[x] I have added a Signed-off-by.
[x] I have either added documentation to cover my changes or this change requires no new documentation.
[x] I have either added unit tests to cover my changes or this change requires no new tests.
[x] I have run golint and have fixed valid warnings in code I have added or modified. This tool generates false positives so you may choose to ignore some warnings. The goal is clean, consistent, and readable code.

Signed-off-by: Bishop Brock<EMAIL_ADDRESS>

Odd failure in travis/behave, though I am guessing it's not related to this patch.  Anyone know what that is?

I thought so as well. behave worked locally for me.

reverify Jenkins

@ghaskins @christo4ferris The Travis issue is documented as #2266

LGTM, thanks


Is there an initialized or ready event?

Hello,
I'm curious if there's an initialized or ready event or something I can listen to when instantiating an editor?
Would be useful in the below case for example:
<script>
    require.config({ paths: { "vs": "node_modules/monaco-editor/min/vs" } });
    require(["vs/editor/editor.main"], function() {
        monaco.languages.json.jsonDefaults.setDiagnosticsOptions({
            validate: false,
            allowComments: false
        });

        var editor = monaco.editor.create(document.getElementById("container"), {
            value: '{ "prop": "Hello world" }',
            language: "json",
            theme: "vs-dark",
            contextmenu: false,
            quickSuggestions: false,
            automaticLayout: true
        });

        editor.getAction("editor.action.format").run();
    });
</script>

Here the line editor.getAction("editor.action.format").run(); doesn't have any effect. The JSON doesn't get formatted and I assume it's because the editor is not fully initialized at that point.
I know I can solve it with setTimeout, but I was wondering if there's any other option.
Thanks in advance, keep up the good work :)
UPDATE:
Adding the below solves it, not the most beautiful solution though :)
var didScrollChangeDisposable = editor.onDidScrollChange(function(event) {
    didScrollChangeDisposable.dispose();
    editor.getAction("editor.action.format").run();
});

Perhaps we could expose the language registries (https://github.com/Microsoft/vscode/blob/66c13c81fd8d844fa2cba38e03bd719e285a1c7e/src/vs/editor/common/modes.ts#L1026), they fire events when language providers are attached.

Hi,
first, many thanks to @dezsiszabi for pointing out one (dirty) solution. It helped me with my approach to automatically trigger a AngularJS function after the monaco editor is fully loaded and displaying some code. Works fine.
I think there should be some "ready" events when the editor is up and running. Or are they already here and I'm missing something out?

I'd like to up-vote on this one.. I ended up sing the onDidScrollChange too with a checker on if (!(editor.getModel() as any)._tokenizationSupport) return; (and then a this.didInit check)... awful code - but that works.. But before that one is hit, I also do a require(["vs/editor/common/modes/languageConfigurationRegistry"], (mod: any) => { this._disposables.push(mod.LanguageConfigurationRegistry.onDidChange(() => { ... to modify loaded language :-S
The reason above is that, when language HTML has been async loaded, I do a extend of that (e.g. showing written css colors as colors in html styles and css blocks etc.) so it end up being an unknown/random race to what has been loaded and initialised.
Thanks

I have the same struggle too. Thanks for show me a solution but I like to bump this requested feature. onReady event would be nice.

OnReady or onLoaded Event would be better.

@dezsiszabi hello~ why I can not use the action 'editor.action.format'.The action is not exist .

maybe 'editor.action.formatDocument' is work

Perhaps we could expose the language registries (https://github.com/Microsoft/vscode/blob/66c13c81fd8d844fa2cba38e03bd719e285a1c7e/src/vs/editor/common/modes.ts#L1026), they fire events when language providers are attached.

has this event been added?
It's really needed because onDidScrollChange will be fired before language providers are attached.

@sunzhengxin92 window.codeEditor._actions

onDidScrollChange not working in the newest version

Hello,
I'm curious if there's an initialized or ready event or something I can listen to when instantiating an editor?
Would be useful in the below case for example:
<script>
    require.config({ paths: { "vs": "node_modules/monaco-editor/min/vs" } });
    require(["vs/editor/editor.main"], function() {
        monaco.languages.json.jsonDefaults.setDiagnosticsOptions({
            validate: false,
            allowComments: false
        });

        var editor = monaco.editor.create(document.getElementById("container"), {
            value: '{ "prop": "Hello world" }',
            language: "json",
            theme: "vs-dark",
            contextmenu: false,
            quickSuggestions: false,
            automaticLayout: true
        });

        editor.getAction("editor.action.format").run();
    });
</script>

Here the line editor.getAction("editor.action.format").run(); doesn't have any effect. The JSON doesn't get formatted and I assume it's because the editor is not fully initialized at that point.
I know I can solve it with setTimeout, but I was wondering if there's any other option.
Thanks in advance, keep up the good work :)
UPDATE:
Adding the below solves it, not the most beautiful solution though :)
var didScrollChangeDisposable = editor.onDidScrollChange(function(event) {
    didScrollChangeDisposable.dispose();
    editor.getAction("editor.action.format").run();
});

it's not work when codestring is short

use onDidCreateEditor
  monaco.editor.onDidCreateEditor(function(event) {
    console.log('editor crated');
  });

  var editor = monaco.editor.create(document.getElementById("container"), {
    value: '{ "prop": "Hello world" }',
    language: "json",
    theme: "vs-dark",
    contextmenu: false,
    quickSuggestions: false,
    automaticLayout: true
  });

if someone needs to really check, if model has been loaded, one may use mutation observer. You should only look for childList of the container, where you loaded monaco editor to keep costs of mutation observing to minimum;
let observer = new MutationObserver((mutations, o) => {

            if (Array.from(mutations).some(mutation => {
                return Array.from(mutation.addedNodes).some(node => node.classList.contains('monaco-editor'));
            }))
           {
                console.log('editor loaded');

            }
        }).observe(container, { childList: true, })

I am trying to add an event listener to the editor, and when using the above, the event listener is gone when the editor is fully loaded.

6 years later,  has an initialized or ready event supported?

6 years later, has an initialized or ready event supported?

still waiting

try this hook
Instance.onDidLayoutChange((event:any)=>{
      // console.log('onDidLayoutChange')
      if(MEInstance){
        MEInstance.trigger(
          "anyString",
          "editor.action.formatDocument",
          null
        ); // format code
      }
 })

@alexdima I want to help push this feature forward. Any pointers as to where to start contributing?

I have this monaco editor which gets mounted and unmounted by the user. this is how I could do this:
function formatDocument() {
  editor.getAction('editor.action.formatDocument').run();
}

/// on first initialization
editor.onDidChangeModelLanguageConfiguration(formatDocument);

/// on every initialization after the first one (also the first one)
editor.onDidLayoutChange(formatDocument);

@dezsiszabi hello~ why I can not use the action 'editor.action.format'.The action is not exist .

const handleEditorDidMount = (editor: any,monaco:any) => {
editorRef.current = editor;
editor.onDidLayoutChange()
setTimeout(
() => editor.getAction('editor.action.formatDocument').run(),
100,
);
};
I've used it like this. It worked for json.

6 years later, has an initialized or ready event supported?

still waiting

Now it's seven years.

My 2 cents: use retry with exponential backoff, something like this:
export async function promiseRetry<T>(
  promise: () => Promise<T>,
  initial_retry_count: number,
  retry_delay: number,
): Promise<T> {
  async function __promiseRetry(promise: () => Promise<T>, remaining_retry_count: number): Promise<T> {
    try {
      const res = await promise();
      return res;
    } catch (e) {
      if (remaining_retry_count <= 0) {
        throw e;
      }
      setTimeout(() => {
        __promiseRetry(promise, remaining_retry_count - 1);
      }, retry_delay ** (1 + 0.1 * (initial_retry_count - remaining_retry_count + 1)));
    }
  }

  return __promiseRetry(promise, initial_retry_count);
}

Then, when you want to get a worker ref (or anything else that requires it to be loaded):
// So.. there is no way to check when the model is initalized apparently!
// https://github.com/microsoft/monaco-editor/issues/115

// If we accidentally call the `getTypeScriptWorker()` function to early, it throws.
//  Just use retry with exponential backoff to get it!
await promiseRetry(
  async () => {
    const getWorker = await monaco.languages.typescript.getTypeScriptWorker();
    const tsWorker = await getWorker();
    dispatch('fullyLoaded', { model: editor.getModel(), worker: tsWorker });
  },
  5,
  10,
);

6 years later, has an initialized or ready event supported?

still waiting

Now it's seven years.

Happy to celebrate 8 years here 🎉

is there a workaround for this? All the methods mentioned above don't seem to work.

6 years later, has an initialized or ready event supported?

still waiting

Now it's seven years.

Happy to celebrate 8 years here 🎉

already 9


iOS quality-selector improvements

Ticket:  Closes #2705
Test:  Salt
Details:  See commits
Will test it further on all platform permutations.
Best if someone can confirm on actual device too.

-- Draft --
Popup not quite right

This expands to livestreams also I guess? I wasn't thinking about that, but good idea. Original would be the highest quality in that case...

-- ready for tests ---

Ok, I see "original" is removed for livestreams in non-IOS.  Also tried it and it just moves to the next video.  No such thing as "original stream", I guess.  Disabled this thing for livestreams + iOS too.
The popup highlight will not be present ... I couldn't figure out why.  I think we can live with it, as I want to keep the hacks easily removable.  What's important is that the button string is correct, and the right files are being fetched.

Looks good on physical iphone and non-iphone scenarios here.


Enhancement: Exposing TICK Vars to Alert template expansion

Currently there is no way to access TICK variable from within a alert template, so pre expansion needs to be done.  As an example the following will not work as 'host_count_warn' and 'host_count_critical' are not present in the template expansion variable scope.
var host_count_warn = 2
var host_count_critical = 1

.... Stuff ....
|alert()
.message('Healty Host Count {{ .TaskName }} is {{ .Level }}: Got a value of {{ index .Fields "_host_count" }} expecting > {{ if .Level eq "WARN" }} {{ .host_count_warn }} {{ else }} {{ .host_count_critical }} {{ end }}')

The current solution is to pre-expanded using the "string" function the variable before the template is executed, i.e.
'{{ if eq .Level "WARN" }} ' + string(host_count_warn) + '{{ else }}' + string(host_count_critical) + '{{ end }}'

It would be nice and much cleaner if we could simply access the variables from a tickscript directly in the alert message as part of the template expansion.
I would propose a new map "TaskVars" or "TickTaskVars" etc that will be populated with the tick script variables, and referenced as such:
Healthy Host Count {{.TaskName}} is {{ .Level }}: Got a value of {{ index .Fields "_host_count" }} expecting > {{ if eq .Level "WARN" }} {{ index .TaskVars "host_count_warn"}} {{ else }} {{ index TaskVars "host_count_critical"}} {{ end }}

In addition, another use case is accessing fields or tags by a variable key, for instance:
var field string = "\"some_field\""
|alert()
    .message('Some alert on field:  {{ index .Fields field }}')

I have not seen a workaround that can accomplish this yet, other than writing individual templates for each field of interest being alerted on.

@nathanielc Any updadate :)

If you're going to extend this for the message property, then extending for id and log would be ideal too.

May be it is assumption that with flux this is not getting probably a priority but this is super important or us as we need to send alerts in a format which servicenow itom can accept and therefore must be able to customize HTTP post bodies.
Wwe should be able to send something like this
Expected
Must be able to access  .Tags outside .Message
{
"id": "{{.ID}}",
"message": "{{.Message}}",
"node": "{{index .Tags "host"}}", ---------> Currently I cannot access .Tags outside .Message
"details": "{{.Details}}",
"time": "{{.Time}}",
"duration": "{{ .Duration}}",
"level": "{{.Level}}",
"recoverable": {{.Recoverable}},
"data": {{ .Data }}
"hardcoded": "test"
}
#2
We should also be able to translate .Level to an integer value
Example:
Critical = 5
Warning = 4
Info = 3


Adding a div to ng-scrollable doesn't allow the scroll

Can you be more specific and provide an example, best on plnkr?

closing due to inactivity, guess things work as expected


One cannot set TextArea to empty

Steps:

Load the textarea with some text.
Delete text in the textarea. (Maybe you want it to be empty)
Focus on another element

Uncaught TypeError: value.split is not a function
    at RIETextArea._this.renderNormalComponent (RIETextArea.js:63)
    at RIETextArea.RIEStatefulBase._this.render (RIEStatefulBase.js:142)
    at ReactCompositeComponent.js:796

My bad.. was a code issue


A structure to test release staging

Signed-off-by: Scott M Stark<EMAIL_ADDRESS>

Want a new EE4J_8 upstream branch


Update visibility inside JsonRPC/Server class

Hi there,
I would like to make a PR in order to change the visibility of some elements in this class.
In fact, it is really difficult to extend the class in order to update the way that the query is parsed.
Furthermore I would like to make the properties protected. I understand that your goal was to protected the class' attributes but I am not sure it is necessary to not be able to manipulate it in a sub class.
Does it seams good to you? If you are ok with this approach I can make the PR.

no problem

Hello all.
@dlevel and I updated Server class altogether. You can find the PR below:
https://github.com/fguillot/JsonRPC/pull/47

I merged your PR, if you see something else to improve, feel free to send a PR.


added Java code editor extension

Description
This pull request introduces a new Chrome extension named "Java Code Editor Extension" that provides users with quick and easy access to an online Java code editor. The extension includes a welcome page with a typewriter effect that greets users and invites them to start coding in Java. Upon clicking the button, users are directed to the Java code editor.
Fixes:  #1109

Type of change

[ ] New feature (non-breaking change which adds functionality)

Checklist:

[X ] I have made this from my own
[ ] I have taken help from some online resourses
[ X] My code follows the style guidelines of this project
[ 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 made corresponding changes to the documentation
[ X] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

@Sulagna-Dutta-Roy added new extension please check


⚠ PrivateHD has degraded performance

In 0adbe84, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 1910 ms

Resolved: PrivateHD performance has improved in 1e1424d.


replace an error for an exception : why ?

Laravel Version: 5.4
PHP Version: all
Database Driver & Version: n/a

Description:
https://github.com/laravel/framework/blob/5.4/src/Illuminate/Routing/Controller.php#L80
Steps To Reproduce:

instantiate a controller
call a random method name

Nvm, I don't want to know.


config: Increase the auxiliary descriptor limit again

Complex configurations still easily hit the descriptor limit.
See (#876)

This should be fixed in  0cec5ce80


Separate JSON-LD from MetaTags component

fix: #7

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Enhance specifying the mandatory "platform"  arg for multi arch images

Syft version after 0.75 fails to generate SBOM for multi-arch images and requires explicitly specifying the --platform option even when using the SHA specific to architecture.
Enhance specifying the  --platform option when scanning images as an input arg to the Syft arguments.
New Feature PR on Syft that is causing the problem: https://github.com/anchore/syft/pull/1801
How to reproduce:
Syft packages -v image:tag@sha256:<any-non-amd64-arch> -o spdx-json > sbom.spdx.json
Error: * failed to construct source from user input image:tag could not fetch image: unable to use DockerDaemon source: image has unexpected architecture "arm64", which differs from the user-specified architecture "amd64

Thanks for opening this issue! The underlying issue in stereoscope has been fixed, see https://github.com/anchore/stereoscope/issues/188. Once we do a syft release that pulls in the stereoscope fix, probably early next week, this issue should be fixed as well.


fix:rocketmq funtion.definition config

Describe what this PR does / why we need it
fix spring.cloud.stream.function.definition configuration item error
Does this pull request fix one issue?
#3727

Describe how you did it
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.zhangqian seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Support tokenized input

Feature request
The OpenAI API /embedding endpoint accepts input for both text (list of strings) and tokenized input (list of integers). text-embeddings-inference should also support list of integers (tokens) as input to create embeddings from.
Motivation
Without this feature, tools like langchain-openai do not work out of the box. The OpenAIEmbeddings object tokenizes text before sending to the embedding model; the following code throws errors:
from langchain_openai import OpenAIEmbeddings
base_url = "<IP_ADDRESS>"
api_key = "sk-xxx"
embeddings = CustomOpenAIEmbeddings( # Use the custom class that doesn't tokenize the input
        model="jina-v2-base",
        base_url=base_url,
        openai_api_key=api_key
)
text = "This is a test query."
query_result = embeddings.embed_query(text)
print(query_result)

A workaround is to force langchain to not tokenize, although it would be cleaner if TEI supported tokens as input:
import os
from typing import List, Tuple, Iterable, Union
from langchain_openai import OpenAIEmbeddings

class CustomOpenAIEmbeddings(OpenAIEmbeddings):
    def _tokenize(
            self, texts: List[str], chunk_size: int
        ) -> Tuple[Iterable[int], List[Union[List[int], str]], List[int]]:
        _iter = range(0, len(texts), chunk_size)
        tokens = texts
        indices = list(range(len(texts)))
        return _iter, tokens, indices

base_url = "<IP_ADDRESS>"
api_key = "sk-xxx"
embeddings = CustomOpenAIEmbeddings( # Use the custom class that doesn't tokenize the input
        model="jina-v2-base",
        base_url=base_url,
        openai_api_key=api_key
)
text = "This is a test query."
query_result = embeddings.embed_query(text)
print(query_result)

Your contribution
I'm not familiar with rust so I don't think I'm comfortable implementing this, but I can help test and document the new functionality.

We already accept token ids for the open ai route.
What errors do you see?

It is accepted, but it does not seem to work correctly in the context of LangChain.
Using the following example, TEI as the backend, and tokenized and non-tokenized clients.
Example code: https://python.langchain.com/v0.1/docs/modules/data_connection/vectorstores/
The pre-tokenized client provides incorrect results.
Custom Client:
text: What did the president say about Ketanji Brown Jackson
Similarity search:[ And I did that 4 days ago, when I nominated Circuit Court of Appeals Judge Ketanji Brown Jackson. One of our nation’s top legal minds, who will continue Justice Breyer’s legacy of excellence. ]
Similarity search by vector:[ And I did that 4 days ago, when I nominated Circuit Court of Appeals Judge Ketanji Brown Jackson. One of our nation’s top legal minds, who will continue Justice Breyer’s legacy of excellence. ]
OpenAIEmbeddings Client
text: What did the president say about Ketanji Brown Jackson
Similarity search:[ It won’t look like much, but if you stop and look closely, you’ll see a “Field of dreams,” the ground on which America’s future will be built. ]
Similarity search by vector:[ It won’t look like much, but if you stop and look closely, you’ll see a “Field of dreams,” the ground on which America’s future will be built. ]


external storage capacity

Issue description:external storage capacity when attached to storage drawers is showing capacity far exceeding max
What happens:external storage capacity when attached to storage drawers is showing capacity far exceeding max
What you expected to happen:external storage to behave as normal
Steps to reproduce:
1.attach external storage to storage drawer controller
2.
3.
...
Version (make sure you are on the latest version before reporting):

Minecraft:1.12
Forge:<IP_ADDRESS>38
Refined Storage:1.6.15

Does this issue occur on a server? [yes/no]no
If a (crash)log is relevant for this issue, link it here:
[pastebin/gist/etc link here]

This issue is with storage drawers reporting wrong numbers. The numbers are purely visual and as such can be safely ignored.