TextInput have a different size depending on platform

TextInput have a different height depending on platform.
On iOS it's smaller, and on Android, it's bigger.
React Native version:
info
React Native Environment Info:
System:
OS: macOS 10.14.5
CPU: (4) x64 Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz
Memory: 177.69 MB / 8.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 10.13.0 - ~/.nvm/versions/node/v10.13.0/bin/node
Yarn: 1.17.3 - ~/.nvm/versions/node/v10.13.0/bin/yarn
npm: 6.10.2 - ~/.nvm/versions/node/v10.13.0/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 12.2, macOS 10.14, tvOS 12.2, watchOS 5.2
Android SDK:
API Levels: 23, 25, 26, 27, 28
Build Tools: 23.0.1, 25.0.2, 26.0.2, 26.0.3, 27.0.0, 27.0.3, 28.0.2, 28.0.3
System Images: android-23 | Intel x86 Atom_64, android-23 | Google APIs Intel x86 Atom_64, android-27 | Google APIs Intel x86 Atom, android-27 | Google Play Intel x86 Atom
IDEs:
Android Studio: 3.4 AI-183.6<IP_ADDRESS>92245
Xcode: 10.2.1/10E1001 - /usr/bin/xcodebuild
npmPackages:
react: ^16.8.6 => 16.8.6
react-native: ^0.60.0 => 0.60.4
npmGlobalPackages:
react-native-cli: 2.0.1
Steps To Reproduce

Place a <TextInput />

Describe what you expected to happen:
The <TextInput /> should have the same height on any platform. I would even say same height as a <Text /> component (when it has the same content inside of course).
Describe what happens:
The <TextInput /> is bigger than a <Text /> on Android, smaller on iOS, and equals on React-Native-Web.
Snack, code example, screenshot, or link to a repository:
https://snack.expo.io/@yairopro/textinput-height-difference-demo

same problem +1


Issue512 benchmark2

Implementation of the Mehlman et al. benchmark into the full model infrastructure
Fixes #512
Task List

[x] Defined the tests that specify a complete and functioning change (It may help to create a design specification & test specification)
[x] Implemented the source code change that satisfies the tests
[x] Documented the feature by providing worked example
[x] Updated the README or other documentation
[x] Completed the pre-Request checklist below

Change Description
Implementation of the dynamics benchmark test case from Mehlmann et al. (2021). This involves

Creating new initial conditions via a python script
Small fixes to the existing "benchmark" forcing classes

Test Description
Run the model using the config_benchmark.cfg config file and visually compare the output with previous results.
A proper integration test is covered by #524

Documentation Impact
Documentation is covered by #520

Other Details
The "benchmark coordinates" now assume that the box is 512 x 512 km (as in the paper). This means that using AtmosphereBoundaryModule = Nextsim::BenchmarkAtmosphere with a different grid (e.g., one of the realistic ones) may yield unexpected results.

Pre-Request Checklist

[x] The requirements of this pull request are fully captured in an issue or design specification and are linked and summarised in the description of this PR
[x] No new warnings are generated
[x] The documentation has been updated (or an issue has been created to track the corresponding change)
[x] Methods and Tests are commented such that they can be understood without having to obtain additional context
[x] This PR/Issue is labelled as a bug/feature/enhancement/breaking change
[x] File dates have been updated to reflect modification date
[x] This change conforms to the conventions described in the README

So, the last merge (81f11e4) accedntely included almost everything in this branch. There's just one line of code left to check.
Can you have a look @timspainNERSC


Update README, add tests

:clap:

Its jsDelivr, not JSDelivr  :)
Brand awareness and everything

I believe its missing this functionality https://github.com/jsdelivr/api/issues/109 ?

@jimaek yes and yes. Will add that.

Also its a good idea to add an "Examples" section to help people understand how it works

"Examples" section

I noticed it was removed also, but I like the shorter README.  Perhaps more clear /tests (API is meant for devs)?  Or separate file?

Updated. I also added tests for the documented stuff.


Add animation in the Home Page

What would you like to share?
Add animation in the Home page while scrolling down so that it looks fabulous.
Additional information

Can I solve this

Can you elaborate a bit like the direction or something or how you want things to be animated

You can add animation effects, transition effects and make the website more responsive

@sidling1 do you want to work on it?

@Sathvik902 assigning it to you

Actually I am having problem while running this website in localhost..
Can you guide me?? @Debajyoti14

@Sathvik902 Sure
Do you have the .env file??

@Debajyoti14  next-env.d.ts file is there

what are the additional dependencies which I need to install inorder to run in localhost?

This is the error I am getting when I npm run dev

@Sathvik902 Mail us at<EMAIL_ADDRESS>for the env file content
Then make a .env.local file in root of your project and add the copy paste the content
Then you can run the project

Okay, I figured it out!
Site is running in localhost.
Thank you @Debajyoti14

@Sathvik902 Nice

Done with the work @Debajyoti14 ..
Added the transition animation to the landing page of the website

Please accept my PR

@Sathvik902 No PR is there. Check once again and make a PR

Created!

@Debajyoti14

Created @Debajyoti14

Made a PR @Debajyoti14

@Sathvik902 Sorry but there aren't any open PR's

How about now? @Debajyoti14

@Sathvik902 merged it

Okay!

Anything else to be added @Debajyoti14 ??
if you describe the type of animation, I'll try to add the same.

Please assign it to me I no completed!

@khushigarg1 It's done only for the Hero section at home screen
You can add it to other screens

@khushigarg1 It's done only for the Hero section at home screen
You can add it to other screens

Okay thank you i will.

I want to work on this issue @Debajyoti14 . Please assign it to me


Solving the "Could not find a declaration file for module" Type Error in JavaScript Projects

can someone merge this?


Better support for dev takeover and testing new ManifestSync yamls

With the addition of the Repo resource #59 we now support building a whole bunch of resources (Images, Manifests) that comprise an application.
One of the friction points right now is doing a takeover. Right now to do a dev takeover the usual pattern is to create a second manifestsync.yaml. That's kind of annoying.
An alternative would be to use kustomize functions to replace all the git references in a bunch of resources to point to a branch or even a local path.
One thing this doesn't solve is how do we selectively enable rebuilding certain images but not others?

One might expect the following to work.

Specify a RepoConfig pointing at a branch; e.g.
apiVersion: hydros.sailplane.ai/v1alpha1

kind: RepoConfig
metadata:
name: repo
namespace: sailplaneai
spec:
repo: https://github.com/sailplaneai/code.git?ref=jlewi/cicd
..
globs:
- "**/*.yaml"
selectors:
- matchLabels:        env: dev
```

Apply it

hydros apply -f repoconfig

This doesn't work though because RepoConfig won't override the repositories referenced in ManifestSync. So they could end up pointing at Main which means the images that do get built will have the commit of the branch but the ManifestSync will look for  the commit corresponding to main so it won't find the images and hydration won't succeed.
This is a perfect use case for a kustomize function.
RepoConfig uses findresources
https://github.com/jlewi/hydros/blob/071b01b535b3f6c2ca718fb0f9edf2721b5211bb/pkg/gitops/repocontroller.go#L157
So we could apply any transformations in memory.

#75 works pretty well for deploying a bunch of microservices.
The current pain point would be that with #75 you have to rebuild all images on every commit which would be very expensive. I think a better approach would be

Create a unique but mutable tag for the "deployment"
Build and tag all images with that mutable tag
When hydrating pin the image to the current version with that mutable tag
Let the user selectively rebuild and retag a given image
When manifests get resync'd if the mutable tag has moved the manifests should be rehydrated

The image pinning already happens with tag strategy mutable. I think the part that might need to change is that when deciding whether a manifest needs to be resync'd we need to take into account whether the mutable tag has moved to a new image. Rather than just looking at the sourceCommit.

A related problem is the developer flow when defining a new manifestsync.yaml for a new service.
How can you use hydros to test that everything is working and deploy it through gitops?
This is effectively a dev-takeover but

You don't want to have to author a separate ManifestSync.yaml from the one that will be auto-deployed
You don't want to have to rebuild the image on every reconcile; so you'd like to use a mutable tag


Can't delete sequence

I can't delete a sequence, it shows this error message:

I have the same problem i.e. for deleting a single photo from a sequence on kartaview.org
I debugged a bit and the reason is that for these POST endpoints the access token should be provided as a normal field in the request body. Instead the application provides it as a url parameter like for GET requests, i.e. /some-endpoint/?access_token=xxxxxx so for the server the user is not authenticated.
When I sent the same request as the application sent but added the access_token in the request body the deletion worked.

Same here. I got this message

As a temporary workaround, you can use the app (though loading the profile sequences takes minutes or longer) to delete the sequences

hey, this has been fixed as of yesterday. please try to see if it works!

Still not working with https://kartaview.org/details/1566873
https://kartaview.org/details/1566193
https://kartaview.org/details/1456702
https://kartaview.org/details/1455626

@brmonaghan - you are trying to access sequences that are processing. The delete function can only be used if recordings are published. However, @zhaitianduo @salabogdan - the recordings mentioned above are from 2019 but can not be accessed.

I am able to delete sequences again for what it's worth. I can leave this issue open for now until we confirm that all sequences are able to be processed again.

I tried to do that @miruna-craciun and it is still not working. This has been a problem when trying to delete sequences. But its been awhile, so let me try again.

Aaand its now working! @joeybab3 you can now close this.
THANK YOU VERY MUCH @zhaitianduo AND @miruna-craciun FOR FIXING THIS!

@brmonaghan - you are trying to access sequences that are processing. The delete function can only be used if recordings are published. However, @zhaitianduo @salabogdan - the recordings mentioned above are from 2019 but can not be accessed. Can you take a look?

I have tracks which show up "processing" but will never process and cannot be deleted, e.g. this one: https://kartaview.org/details/4537689. I don't see a way to either change its status or delete it. Am I missing something?

agree.


[Serious] Keystore wiped with the following

Enabled ropsten
Saw conflicting TokenScript for UEFA
Removed airdropped UEFA file
Killed the app
Wallets have disappeared

Cannot seem to reproduce this on my own phone, happened on Victors...

@James-Sangalli Did it happen once or is it reproducible?

@James-Sangalli Did it happen once or is it reproducible?

Only once, having trouble reproducing it

Might be related to an iOS bug I have seen before, NSUserDefaults sometimes not accessible shortly after startup. But I have never been able to reproduce it. Someone reported something similar with various different conditions, iOS 11, 32-bit, etc.

#1591 helps to mitigate this. There might be other cause, but I can't figure out what else might be wrong by code inspection though.

@hboon fair enough, thanks for that. Will see if it can be reproduced again with the fix but we might never know.


feat(dashboards): add hardware metrics

#392  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


[WIP] Add mongodb-writer

Signed-off-by: Darko Draskovic<EMAIL_ADDRESS>

Codecov Report

Merging #297 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #297   +/-   ##
=======================================
  Coverage   86.68%   86.68%
=======================================
  Files          34       34
  Lines        1307     1307
=======================================
  Hits         1133     1133
  Misses        128      128
  Partials       46       46

Continue to review full report at Codecov.

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

This PR is moved to #307.


20.build-args encrypted_args_file not working

The example is not working as on the build phase the variable has no value. not sure if is the docker version.

@aetheon Are you running this locally with the Jet CLI, or on Codeship? If you are using the CLI, could you report the version, as well as the Docker version you use?

Hey @rheinwein, was running locally, didn't try on codeship.
Docker version 1.13.1, build 092cba3
Jet version 1.14.1

As you can see on the screenshots the args are not being passed correctly. From my reading I think this is not possible with the latest docker. There's a workaround that we are using, let me know if you are interested.

Hey @aetheon You're right -- encrypted build args are not a Docker feature, they're an additional feature from Codeship specifically designed for CI/CD.
Looks like you're using an older version of the CLI that doesn't have full support for build arguments. You can see release notes on our docs page, and grab the new CLI version, which should fix your issue.
We'll remove (parts of) this example from the repo, since it won't work for anyone outside of the Codeship organization -- you don't have access to our key to unencrypt the variable.

Hi @rheinwein, I am also having a issue with encrypted_args_file but it doesn't work when running it on Codeship with our .aes key.
codeship-services.yml:

decrypted deploy/codeship.env:

@tylerbakke could you open a ticket on https://helpdesk.codeship.com with the error message you get and (if possible) a link to the builds on Codeship?
The config looks good, and if you can decrypt the file locally it should work with the automated builds as well, as long as the key you use locally is the same as the one specified on the projects General settings page.

Is there any update on this?
I'm trying to configure encrypted build-time args on my codeship pro build (for my paying customer) and the feature is not working!
I verified that I used the latest version of the JET CLI.

In general, it would be good to update all the samples to include an aes key, so they can run by themselves (e.g. like done in ./11.encrypted-aes)

@thomastoye are you working from a fork, and have you set that fork up as a CodeShip project? If no, the testing script won't pass for you, since the encrypted build args are using CodeShip's key, which you don't have access to.
However, you do seem to have uncovered a bug with encrypted build args passed directly into the codeship-services.yml file (like this and this). The eng team will follow up with that after the holiday.
Passing in a file of encrypted build args works as expected. If you want the tests in the 20.build-args dir of your fork to pass, you can append the build_arg file with ENCRYPTED=some_value and using the AES key provided to your project on CodeShip, reencrypt that file with jet encrypt build_args build_args.encrypted. Running jet steps in that directory (with your AES key present) will show you the encrypted build args in action.

This is not working for me
unecrypted file
(cat .envs/.local/.ssh)
PRIVATE_SSH_KEY=testing

Then running
> jet encrypt .envs/.local/.ssh .envs/.encrypted/.ssh

# Encrypted file
> cat .envs/.encrypted/.ssh
codeship:v2
hvHhdgpF5vVjYu6mJE+u7b+7gXV6N9Tu0e47/0BDr+r6vqlAJGO2xlEug/8C+AV4afaPUGnIDEhm8IqamuB2rWBposL8sCGr%

# Decrypt for verification
> jet decrypt .envs/.encrypted/.ssh decrypted_ssh
> cat decrypted_ssh
PRIVATE_SSH_KEY=testing

so this should be correct. Now doing this in my Dockerfile
FROM python:3.6-alpine
ARG PRIVATE_SSH_KEY
RUN echo "PRIVATE_SSH_KEY: $PRIVATE_SSH_KEY"

should echo  "PRIVATE_SSH_KEY: testing"
but it doesn't. There is nothing displayed in the slot where $PRIVATE_SSH_KEY variable is and therefor my build fails. Anything I'm doing wrong here?

Any update on this or perhaps a workaround?
I have the same issue and the set bash command shows variables on build time.

I finally got it working, but I had to add all the variables present in build_args.encrypted to the Dockerfile as well. Unfortunately I did not find anything in your documentation to specify this.
Example of what I added to my Dockerfile:
ARG AWS_ACCESS_KEY_ID
ARG AWS_SECRET_ACCESS_KEY
ARG AWS_REGION


Cache vagrant boxes in github actions

Seeing log failures like:
An error occurred while downloading the remote file. The error
message, if any, is reproduced below. Please fix this error and try
again.
The requested URL returned error: 429

(https://github.com/pycontribs/python-vagrant/runs/7184689242?check_suite_focus=true#step:9:130)
I'm wondering if setting a cache would avoid this ?
If so, what would be the correct key ? something like:
${{ runner.os }}-${{ matrix.tox_env }}-vagrantbox-${{ hashFiles('tests/tools/create_dummy_box.sh', 'tests/test_vagrant*py', '**/Vagrantfile', 'tests/vagrantfiles/*Vagrantfile') }}

Sounds like a great idea, but will that actually cache .vagrant/machines where the VMs are stored?

ah yes, i actually meant the boxes, not the vms :/
path:  ~/.vagrant.d/boxes was the part i was missing from the first code line above


Enable Czech language

Hi,
translating QWAT Doc is done. Please enable it on doc pages.
Please enable Czech translation of plugin on Transifex.
Thanks, best regards,
Jiri

Hi,
We currently face a bug with transifex documentations but we will publish is as soon as it is solved!
Do you know any Czech folks using QWAT?
Translating QWAT datamodel value lists and QGIS project would then be meaningfull!
Best regards!
Arnaud


Hi All any one can tell me how  i can build sql db agent with langflow thanks

Feature Request
Hi All any one can tell me how  i can build sql db agent with langflow thanks
Motivation
None
Your Contribution
None

@dosu Can you please provide me with the complete  code if you know that shows  connect an LLM (Language Learning Model) to my sql database and chat effectively with my SQL database? I would be very thankful to you. plzz!!!

@dosu. Can you provide me with code that includes an LLM instance? I want to use OpenAI's LLM and return responses properly. I don't want to use any tools right now; I just want to chat directly with the tables in my SQL database in a best way.

@dosu I want to connect to all the tables and data in the database so that if I ask anything from the entire DB, it gives me a proper and accurate answer. This is what I want. plz i dont want to connect a single table i want to connect whole sql db so..

@ dosu i dont need to provide open  ai or other llm key?

from langchain.sql_database import SQLDatabase
from langchain.base_language import BaseLanguageModel
from langflow.components.chains.SQLGenerator import SQLGeneratorComponent
Initialize your SQL database connection
db_uri = "your_database_connection_string"
llm = BaseLanguageModel(your_language_model)
Create an instance of SQLGeneratorComponent
sql_generator = SQLGeneratorComponent(
input_value="Your natural language question here",
llm=llm,
db=SQLDatabase(db_uri),
top_k=5,  # Number of results per select statement
prompt="Your custom prompt with {question}"  # Optional
)
Generate SQL query from natural language
query = sql_generator.invoke_chain()
print(query)
plz add the llm configuration at this entire code i want to use open ai

@dosu
from langchain.sql_database import SQLDatabase
from langchain.base_language import BaseLanguageModel
from langflow.components.chains.SQLGenerator import SQLGeneratorComponent
Initialize your SQL database connection
db_uri = "your_database_connection_string"
llm = BaseLanguageModel(your_language_model)
Create an instance of SQLGeneratorComponent
sql_generator = SQLGeneratorComponent(
input_value="Your natural language question here",
llm=llm,
db=SQLDatabase(db_uri),
top_k=5, # Number of results per select statement
prompt="Your custom prompt with {question}" # Optional
)
Generate SQL query from natural language
query = sql_generator.invoke_chain()
print(query)
plz add the llm configuration at this entire code i want to use open ai

@dosu
i want to build with langflow not with langchain

@dosu can you please give me complete and proper code with proper return plz

@dosu
but yeh error is code me
File "/home/ahmed-siddiqui/python-important/langflow/dbagent.py", line 89, in
from langflow.interface.agents.custom import SQLAgent
ModuleNotFoundError: No module named 'langflow.interface.agents'

@dosu
this  from langflow.interface.agents.custom import SQLAgent
class is decpreted plz update the code

@dosu
from langflow.custom import CustomComponent
from langchain_community.utilities import SQLDatabase
from langchain_community.agent_toolkits import SQLDatabaseToolkit
from langchain_community.agent_toolkits.sql.base import create_sql_agent
from langflow.field_typing import LanguageModel
from langchain.agents import AgentExecutor
from typing import Callable, Union
from langchain_openai import ChatOpenAI
class SQLAgentComponent(CustomComponent):
display_name = "SQLAgent"
description = "Construct an SQL agent from an LLM and tools."
def build_config(self):
    return {
        "llm": {"display_name": "LLM"},
        "database_uri": {"display_name": "Database URI"},
        "verbose": {"display_name": "Verbose", "value": False, "advanced": True},
    }

def build(
    self,
    llm: LanguageModel,
    database_uri: str,
    verbose: bool = False,
) -> Union[AgentExecutor, Callable]:
    db = SQLDatabase.from_uri(database_uri)
    toolkit = SQLDatabaseToolkit(db=db, llm=llm)
    return create_sql_agent(llm=llm, toolkit=toolkit, verbose=verbose)

def create_sql_query_agent(api_key: str, database_uri: str, verbose: bool = False):
"""
Creates a SQL query agent with the given API key and database URI
Parameters:
api_key (str): OpenAI API key
database_uri (str): Database connection string
verbose (bool): Whether to print detailed output

Returns:
tuple: (agent, component)
"""
try:
    # Initialize the OpenAI language model
    llm = ChatOpenAI(
        api_key=api_key,  # Pass the API key directly
        model="gpt-3.5-turbo",
        temperature=0
    )

    # Create an instance of SQLAgentComponent
    sql_agent_component = SQLAgentComponent()

    # Build the SQL agent
    sql_agent = sql_agent_component.build(
        llm=llm,
        database_uri=database_uri,
        verbose=verbose,
        handle_parsing_errors=True

    )

    return sql_agent, sql_agent_component

except Exception as e:
    raise Exception(f"Error creating SQL agent: {str(e)}")

def query_database(agent: AgentExecutor, question: str) -> str:
"""
Queries the database using the SQL agent
Parameters:
agent (AgentExecutor): The SQL agent
question (str): Natural language question

Returns:
str: Agent's response
"""
try:
    response = agent.run(question)
    return response
except Exception as e:
    return f"Error querying database: {str(e)}"

if name == "main":
# Your credentials
openai_api_key = "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"
database_uri = "postgresql://master_user:tekrevol12311@<IP_ADDRESS>:5432/schology_new"
try:
    # Create the SQL agent
    sql_agent, component = create_sql_query_agent(
        api_key=openai_api_key,
        database_uri=database_uri,
        verbose=True  # Set to True to see detailed output
    )

    # Example questions
    questions = [
        "could you tell me about my database in detail which purpose of this db please briefly explain",
    ]

    # Query each question
    for question in questions:
        print(f"\nQuestion: {question}")
        response = query_database(sql_agent, question)
        print(f"Response: {response}")

except Exception as e:
    print(f"An error occurred: {str(e)}")

could you plz solve this type of error in this code and provide me perfect solution for this
Action Input: SELECT title FROM article LIMIT 5;[('How are religiosity and ethnicity navigated by young people in the context of school, and wider societal expectations?',), ('Is Farming the New Rock and Roll?',), ('All eyes on Rafah – the role of social media in raising awareness and bringing about social change',), ('Racing Against the Clock: The Effects of Social Jet Lag and Exercise on Metabolic Health',), ('The backlash on flexible working – how might this impact women? ',)]Response: Error querying database: An output parsing error occurred. In order to pass this error back to the agent and have it try again, pass handle_parsing_errors=True to the AgentExecutor. This is the error: Could not parse LLM output: I now know the final answer. The database contains articles with titles such as "How are religiosity and ethnicity navigated by young people in the context of school, and wider societal expectations?" and courses with names like Economics, Business, and Politics.

@dosu
from langflow.custom import CustomComponent
from langchain_community.utilities import SQLDatabase
from langchain_community.agent_toolkits import SQLDatabaseToolkit
from langchain_community.agent_toolkits.sql.base import create_sql_agent
from langflow.field_typing import LanguageModel
from langchain.agents import AgentExecutor
from typing import Callable, Union
from langchain_openai import ChatOpenAI
from pydantic.v1 import SecretStr
class SQLAgentComponent(CustomComponent):
display_name = "SQLAgent"
description = "Construct an SQL agent from an LLM and tools."
def build_config(self):
    return {
        "llm": {"display_name": "LLM"},
        "database_uri": {"display_name": "Database URI"},
        "verbose": {"display_name": "Verbose", "value": False, "advanced": True},
    }

def build(
    self,
    llm: LanguageModel,
    database_uri: str,
    verbose: bool = False,
) -> Union[AgentExecutor, Callable]:
    db = SQLDatabase.from_uri(database_uri)
    toolkit = SQLDatabaseToolkit(db=db, llm=llm)
    return create_sql_agent(llm=llm, toolkit=toolkit, verbose=verbose, handle_parsing_errors=True)

def create_sql_query_agent(api_key: str, database_uri: str, verbose: bool = False):
try:
llm = ChatOpenAI(
api_key="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",
model="gpt-3.5-turbo",
temperature=0
)
    sql_agent_component = SQLAgentComponent()

    sql_agent = sql_agent_component.build(
        llm=llm,
        database_uri=database_uri,
        verbose=verbose
    )

    return sql_agent, sql_agent_component

except Exception as e:
    raise Exception(f"Error creating SQL agent: {str(e)}")

def query_database(agent: AgentExecutor, question: str) -> str:
try:
response = agent.run(question)
return response
except Exception as e:
return f"Error querying database: {str(e)}"
if name == "main":
openai_api_key = "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"
database_uri = "postgresql://master_user:tekrevol12311@<IP_ADDRESS>:5432/schology_new"
try:
    sql_agent, component = create_sql_query_agent(
        api_key=openai_api_key,
        database_uri=database_uri,
        verbose=True
    )

    questions = [
        "could you tell me about my database in detail which purpose of this db please briefly explain",
    ]

    for question in questions:
        print(f"\nQuestion: {question}")
        response = query_database(sql_agent, question)
        print(f"Response: {response}")

except Exception as e:
    print(f"An error occurred: {str(e)}")

can i use genemi llm instead of open ai plz

@dosu now i need to make a agent that works for no sql db can you plz provide that complete code plz

@dosu aik aisa agent bhi banana hy ju data json ya csv me input aur osse vector db me convert karen aur phir process karen

yar aisa feature hy kiya langflow me ke jab hum input me data json ya csv me de input me tu vector db ka uri nahi dena hu koi bhi vector db use hu bhale but hume hume is tahra uri nahi dena hu

@dosu
yar aisa feature hy kiya langflow me ke jab hum input me data json ya csv me de input me tu vector db ka uri nahi dena hu koi bhi vector db use hu bhale but hume hume is tahra uri nahi dena hu

@dosu but jab hum langflow ke dashboard me vector agent banate hy rag agent tu wahan astra db me tu hume koi vector uri provide kerne ki need nahi huti

@dosu
from langflow.custom import CustomComponent
from langchain_community.utilities import MongoDBAtlasVectorSearch
from langflow.field_typing import LanguageModel
from langchain.agents import AgentExecutor
from typing import Callable, Union
from langchain_google_genai import ChatGoogleGenerativeAI
from pydantic.v1 import SecretStr
class NoSQLAgentComponent(CustomComponent):
display_name = "NoSQLAgent"
description = "Construct a NoSQL agent from an LLM and tools."
def build_config(self):
    return {
        "llm": {"display_name": "LLM"},
        "mongodb_uri": {"display_name": "MongoDB URI"},
        "db_name": {"display_name": "Database Name"},
        "collection_name": {"display_name": "Collection Name"},
        "verbose": {"display_name": "Verbose", "value": False, "advanced": True},
    }

def build(
    self,
    llm: LanguageModel,
    mongodb_uri: str,
    db_name: str,
    collection_name: str,
    verbose: bool = False,
) -> Union[AgentExecutor, Callable]:
    vector_search = MongoDBAtlasVectorSearch(
        mongodb_atlas_cluster_uri=mongodb_uri,
        db_name=db_name,
        collection_name=collection_name,
        embedding=llm
    )
    # Create an agent using the vector search
    return AgentExecutor.from_vector_search(vector_search, llm, verbose=verbose)

def create_nosql_query_agent(api_key: str, mongodb_uri: str, db_name: str, collection_name: str, verbose: bool = False):
try:
llm = ChatGoogleGenerativeAI(
api_key="AIzaSyAEOVGaFTlqhwDVhuw5XuddBOM6ZNoYIdk",
model="gemini-1.5-flash",
temperature=0
)
    nosql_agent_component = NoSQLAgentComponent()

    nosql_agent = nosql_agent_component.build(
        llm=llm,
        mongodb_uri=mongodb_uri,
        db_name=db_name,
        collection_name=collection_name,
        verbose=verbose
    )

    return nosql_agent, nosql_agent_component

except Exception as e:
    raise Exception(f"Error creating NoSQL agent: {str(e)}")

def query_database(agent: AgentExecutor, question: str) -> str:
try:
response = agent.run(question)
return response
except Exception as e:
return f"Error querying database: {str(e)}"
if name == "main":
google_api_key = "AIzaSyAEOVGaFTlqhwDVhuw5XuddBOM6ZNoYIdk"
mongodb_uri = "mongodb://localhost:27017/"
db_name = "mydatabase"
collection_name = "mycollection"
try:
    nosql_agent, component = create_nosql_query_agent(
        api_key=google_api_key,
        mongodb_uri=mongodb_uri,
        db_name=db_name,
        collection_name=collection_name,
        verbose=True
    )

    questions = [
        "What are the most recent entries in the collection?",
    ]

    for question in questions:
        print(f"\nQuestion: {question}")
        response = query_database(nosql_agent, question)
        print(f"Response: {response}")

except Exception as e:
    print(f"An error occurred: {str(e)}")

but apne ju code diya hy is me yeh error araha hy yar
Traceback (most recent call last):
File "/home/ahmed-siddiqui/python-important/langflow/main.py", line 80, in
from langchain_community.utilities import MongoDBAtlasVectorSearch
ImportError: cannot import name 'MongoDBAtlasVectorSearch' from 'langchain_community.utilities' (/home/ahmed-siddiqui/.virtualenvs/langflow-env/lib/python3.10/site-packages/langchain_community/utilities/init.py)


Added vue-cursor-fx to Miscellaneous

An animated custom cursor effects for interactive elements like navigation, galleries and carousels.

fix: git: no conflicts resolved
fix: lists: not the last on the list


upgrade to latest common

change signing to use v4 based signing, which can be used by Unicron
load credentials using the credentials.json file

 IC/ICG : Turn on V2, support uploading existing assets


Claiming the website for business accounts

Hi @seregazhuk ,
I would like to claim my website for my business accounts but I think it is not implemented yet.
Could you please provide me a support to claim websites with the bot?
It should automatically download the HTML file that Pinterest provides.
Thank you for your support !

Hi! Sorry, there is no such functionality in the bot.


Rewrite under the terms of the MIT license

Or ask for a license under MIT (and maybe explain why MIT is needed)

@doctormo There was a request a while ago to change the license. While I don't remember the details, I was left with the impression that licensing is impeding adoption. Then somewhat naively, and admittedly without doing my homework, I changed it to MIT.
I think the most appropriate course of action now is to wait until someone requests more permissive licensing and have them explain their use case and reasoning.

Sounds good. LGPL was chosen for the same reason, it used to be GPLv3, but complaints came in so it was moved to LGPL to help :+1:


Service Now - AAP Job run workflow

Service Now - AAP Job run workflow

/hold until workflow validated

/unhold meeting with SA is done

I think we can merge now, no?
/unhold


COROV difficulty scaling and performance metric

After playing around with a quite a bit of machine learning on datasets collected with the current CORVO implementation, we're concerned that there's some patterns in the performance metric that is hindering our ability to fit it to brain data.

The difficulty metric doesn't appear to be scaling correctly. Currently, it has the tendency to increase shortly after the test begins, peak, and then decline down to 0. Thereafter, the test gets stuck at a low difficulty.

The performance metric, while responsive to missed trials, also seems to have this trend to slowly decrease over time (perhaps as a function of the difficulty). It looked nice at first, as maybe an indicator of fatigue over time, but it shows up in nearly every dataset with almost exactly the same slope. Thus, we don't think it's coming directly from the user, but from the performance metric implementation.

Here's an example of what difficulty (blue) and performance (green) look like over a session:

What we'd love to try is to turn off all scaling of difficulty and performance and see if that improves the accuracy of our regression algorithms. I agree there's something to be gained from having dynamic difficulty, but a constant high difficulty combined with a performance score that's always the same should minimize sources of variance in our labels.

Thanks for dealing with this @cbattista at the last hacknight!


Redis upgrades/QA

Summary

Update redis chart to v17.3.7
Update redis-cluster chart to v8.2.6
Fix redis-cluster recipe names
Update image tags for charts
Define CPU and memory requests
Update vendoring regexp
Bump chart version

Closes ENG-826.
Test Plan

[x] Link and deploy Redis after updating versions
[x] Check resources inside cluster (image below)
[x] Install CLI, auth, check redis_version with info server command (image below)
[x] Verify CPU memory and requests in Console (image below)

Checklist

[ ]  No images hosted from dockerhub
[ ]  Are dashboards present to understand the health of the application.  There must be at least 1 of these

[ ]  all databases should have dashboards
[ ]  ideally also have at least cpu/mem utilization dashboards for webserver tier of the app
[ ]  you can use plural from-grafana to convert a grafana dashboard found via google to our CRD

[ ]  Are scaling runbooks present

[ ]  all databases must have scaling runbooks
[ ]  you can use the charts in pluralsh/module-library to accelerate this

[ ]  do you need to add config overlays?

[ ]  inputing secrets
[ ]  configuring autoscaling

[ ]  If there’s a web-facing component to the app, we need to support OIDC authentication and setting up private networks if no authentication option is viable
[ ]  All major clouds must be supported

[ ]  Azure
[ ]  AWS
[ ]  GCP

@michaeljguarino @DavidSpek Please take a look.

@michaeljguarino @DavidSpek PTAL.


Add GUI tests

Part of #1494.
I finally took the time to wrap something together. So in this case, I took what we do for rustdoc and simply ported it to docs.rs. I added 2 GUI tests so you can take a look at what they look like. The documentation for the framework is here.
Do you think this is the right approach or would you prefer another framework?

I'm a bit stuck with the current docker situation. So to sum up what I tried to and what my issue is:

I want to have a docs.rs environment set up with a "clean" database with only the crates/updates we need for the GUI tests. The other point is that it doesn't "pollute" the existing DB by adding new crates.
It should be reproducible easily.
The docker image doesn't need to be rebuilt every time you make a GUI test update.

So now the problem is that I think putting everything in one docker container isn't a great idea because it makes a big image which needs to be rebuilt every time there is a change on docs.rs source code.
The other problem is that apparently, we can't start docs.rs web server in the container: it exists right away (without an error).
So first: is it the right approach? Secondly: any help would be very appreciated.

So I was finally able to make it work like I wanted. I even added it to the docker-compose and I added a shell script to make it simpler to run.

Maybe ping @Nemo157 ?

@syphar It's "ready" now, we need to discuss about if this looks good to you or if it seems to complicated to maintain and eventually try to pick another solution.

@syphar It's "ready" now, we need to discuss about if this looks good to you or if it seems to complicated to maintain and eventually try to pick another solution.

Thanks for the ping!
This topic was next on my list to dig into.
For my understanding: rustdoc is actively using the same framework?

Yes. You can see it being used in src/test/rustdoc-gui.

I updated to the last version of the framework and added some documentation in case you want to run it locally. So with this, it should be ready.
From here, the plan is then to test as much as possible our UI, especially the top-navbar and all docs.rs pages.

CI finally passed! \o/
PR is now ready. :)

only one last comment from my side. I wonder if we can safe time on the setup, or if caching will help a a little, 20 minutes, are quite long.

Yes, 20 minutes is way too long. If you know how to cache it, I'd be very happy to do it.

I would like to have a short test from @Nemo157 because I can't currently run the tests on my mac

Is it because of this PR or something specific to your mac?

only one last comment from my side. I wonder if we can safe time on the setup, or if caching will help a a little, 20 minutes, are quite long.

Yes, 20 minutes is way too long. If you know how to cache it, I'd be very happy to do it.

do you know where most of the time is spent?

I would like to have a short test from @Nemo157 because I can't currently run the tests on my mac

Is it because of this PR or something specific to your mac?

I mean these GUI tests don't run, the other tests do.
the -- build crate ... things don't run, and I have to use docker compose run web build crate...
but running the test script leads to another error related to the host / container platform (I'm on an M1).
But IMO this doesn't block this PR, I'll figure it out afterwards.

Oh wow. It took only 8 minutes without docker build.

the -- build crate ... things don't run, and I have to use docker compose run web build crate...

I also can't use a local build crate, the rustup that rustwide downloads doesn't like nixos.

Wow, weird. I thought using docker would prevent all these issues. ^^'

rustwide does all the installation outside docker containers for some reason (it should be possible to move that all into the containers, which would probably make it work on macos just fine too).

I see. Do you want it to be fixed before we merge this PR or after?

It's orthogonal to this PR (and given maintenance status of rustwide unlikely to happen soon). I plan to see if I can adapt the script here to run everything inside docker-compose containers, can probably get around to it today if it's not too complicated.

Thanks! :D

I got it working in https://github.com/rust-lang/docs.rs/commit/16d4957acfdad0d0b0fbac88d49b9d6521a9a96a and managed to run them locally, but I think this can be much simpler after the refactoring in #2269, so I'd merge as is and I'll try to finish that up and update the script later.

@Nemo157 so you think we should merge this PR now, and then improve on #2269 ?

Yep

Awesome! \o/


Poorly rendered image quality

Hi all,
I have tried using the latest 1.3.91 version for this. All I did was modify the viewer.js to point to my own PDF. This is the result using the fresh out of the box viewer.html :

The left one is the actual PDF, and the right one is rendered by viewer.html. Same thing in IE and Chrome. I did not try it in FF because my users use either IE or Chrome.
I've tried searching everywhere I could and have been unsuccessful in finding a solution to this. Here's the PDF file from the picture.
.0002.pdf

What exactly is wrong with the PDF from viewer.html?
(the zoom levels are different by the way, the one at the left uses 100% zoom, the one on the right 80%).

Hi Rob,
Thanks for your reply.
The problem is the text quality (I'm not sure if it's strictly font rendering or not) makes it a bit harder to read the words on it. Check out the word Billing at number 12, for example. The one on the right looks like it says "Bllllng". My users need to be able to see at least 8 pages at the same time, as they go through thousands every day. This means they're going to view the pdfs with an even smaller zoom level.
The one on the left is not at 100%. It's actually at a smaller percentage than 80%. Just look at the sizes. The one on the left showed number 1-28 on the same screen, while the right one only showed 25.

Could you paste a screenshot with exactly the same zoom sizes?
The "text rendering" is not a text rendering issue. What you see is an image. The question is whether the quality deteriotated after scaling (and of course, whether it can be improved).

Let me clarify that I've tried playing with the scaling for both. I've tried looking at the PDF at 50% in Adobe Reader and Foxit. The quality doesn't get affected much by scaling.
Here's from the viewer at 100%. Notice that the first 'i' in the word Billing still looks like an 'l'.

And this next one is at 100% in Adobe Reader.

Thanks for your extra info. I don't know what goes wrong, but maybe some of the other PDF.js devs know.

I think the loss of image quality happens in canvas.js#L1895, where the Jpeg image of dimensions 2508 x 3525 is drawn onto a canvas that is much smaller than the image. Images look poor when they are scaled down by a large factor at once, which is fixed in canvas.js#L2081 (from #3312). The same fix was applied to thumbnails in #4924. So I guess it should be fixed here for Jpeg images as well.

The PDFs I'm rendering mostly contain text as majority of the content.
The same PDFs are displayed perfectly in an iframe on IE + Adobe plugin, or IE + Foxit plugin. This is despite the iframe's dimensions were set to 360 x 400 (much smaller than the actual size). While on Chrome, using both the default viewer and PdfViewer extension, they look really distorted.
Adobe and Foxit use their own engine to render the PDF, while most Chrome extensions translate the PDF contents into html elements. Problem is, Adobe & Foxit draw their whole interface on a different layer on top of the html, disabling my application from displaying context menu and other things over the PDFs.
I'm still hoping I can use pdf.js for my app. But right now, I'm forced to try a different solution.

The PDFs I'm rendering mostly contain text as majority of the content.

@nicholassinggih The pdf you provided contains a (scanned?) jpeg image in the background. The text that you can select in the document is an invisible text overlay.
The images are downscaled by the browser (not by pdf.js's javascript code), so image quality may vary with the browser or OS. Firefox and Chrome (maybe others too) do a terrible job of downscaling the big images to small canvases, thus causing the bad image quality.
I pushed a commit to https://github.com/fkaelberer/pdf.js/commit/913d3bcadba828c1f3a73f17b0c029016c244259, which downscales the jpegs in multiple steps, in each step with a factor of <= 2x. As a result, the readability is increased a lot, see images below.
I did not open a pull request, though, because the code is unfinished and not tested much. Anyone, please feel free take and improve the code.

[x] better readabilty / image quality
[ ] figure out if / how it works if image does not fill the whole canvas
[ ] Fix blurry issue1350
[ ] Bonus: deduplicate the downscaling code
[ ] Check out if blurryness of some images can be reduced (red font in issue2642 looks better without this patch)

At 50%:

At 70%:

Hi Felix,
Thank you very much for this. I've tested your new code and it does give a much better result, just as shown in your screenshots. Bravo to you!!
Forgive me if my next questions sound silly, but I just want to clarify if I understand this correctly. About the text vs jpeg image thing, are you saying that the PDF that I provided actually only contain jpeg of a scanned paper with text on it? Therefore, it doesn't actually contain text data and it was rendered as an image which in turn was scaled down by the browser?
Should I close this thread, or let either Yuri or Tim to close this?

About the text vs jpeg image thing, are you saying that the PDF that I provided actually only contain jpeg of a scanned paper with text on it?

Yes. That's why I slapped the jpeg label on this ticket.

Therefore, it doesn't actually contain text data and it was rendered as an image which in turn was scaled down by the browser?

It does contain text data (with a transparent color, probably for text selection), but what you see (and what is printed) is the (scaled) image.

Should I close this thread, or let either Yuri or Tim to close this?

The (legitimate) issue hasn't been resolved yet, so I'd keep the issue open.

It does contain text data (with a transparent color, probably for text selection), but what you see (and what is printed) is the (scaled) image.
I see. They ran OCR on the PDF, thus the transparent/invisible text data. But, if they hadn't ran OCR on it, I'm guessing it would just contain a jpeg image with no text.

Thank you both for your help. Really appreciate it.

That's a first time I see scanned data was packaged as JPEG (vs JBIG2 or CCITT). We probably didn't see this issue early since it's probably a rare case. As mentioned in https://github.com/mozilla/pdf.js/issues/7041#issuecomment-192496865 above, we already do it for most of the images except JPEG (e.g. https://github.com/mozilla/pdf.js/blob/master/src/display/canvas.js#L2081) Trade-off is more memory and CPU is used. If we will decide to move decoding of the JPEG to the worker side then we don't have to worry about code duplication.

That's a first time I see scanned data was packaged as JPEG (vs JBIG2 or CCITT). We probably didn't see this issue early since it's probably a rare case.

It does not only affect text, but also images
From #2739:

http://www.ikea.com/at/de/assembly_instructions/rakke-kleiderschrank__AA-808506-1.pdf (notice the thumbnails)

Hi Felix or anyone,
Is there a chance that you could implement a better downsampling algorithm in the paintJpegXObject methods? Some of my users are still complaining about the quality of the images. Even with the gradual scaling that Felix added, scanned images (JPEG) of handwritten documents are sometimes too faded or blurry too read.
Volume 1 of 4 43_Redacted.pdf
Volume 1 of 4 6_Redacted.pdf
This is all I know about the user's machine:
Windows 7 SP1
Browser: Google Chrome  54.0.2840.71 m
I used Felix's version https://github.com/fkaelberer/pdf.js/commit/913d3bcadba828c1f3a73f17b0c029016c244259
Graphics card: GeForce GT610
Monitor's resolution 1680 x 1050
I am currently trying to implement Lanczos or sinc downscaling algorithm in the paintJpegXObject. But I don't know how bad it will affect the performance, and not sure if it is going to fix the problem.
Really appreciate your help.

But I don't know how bad it will affect the performance, and not sure if it is going to fix the problem.

@nicholassinggih if you will find out algorithm that we can use for downsampling and it works, we will provide pointers on how to improve its performance (e.g via asm.js).

@yurydelendik I've tried implementing bicubic interpolation instead to scale down the jpegs. The result is not as good as @fkaelberer 's solution, not to mention that the performance is also slower.
This problem proves to be the most difficult challenge in the process of developing the current application. I'm currently looking into the possibility of just creating a modified copy of the PDF file with all the pages and images resized to the desired dimensions. As my users don't actually dynamically zoom in and out when viewing the pages, this could work.

var options = options || {
scale: 1
};
increase the scale you can see improved clarity

😀
เมื่อ 21 ธ.ค. 2017 14:11 "Nataraj"<EMAIL_ADDRESS>เขียนว่า

var options = options || {
scale: 1
};
increase the scale you can see improved clarity
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mozilla/pdf.js/issues/7041#issuecomment-353275368,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AeFt7fOCDd67jMPJ8D5BmwZB5Iu4VdIvks5tCgSTgaJpZM4HlCqI
.

https://accounts.firefox.com/signin
may33153

hey guys, not sure if this is applicable in this case, i have been getting blurry text as well from the rendered canvas, and i have been wondering if this is rather html5 canvas issue rather than pdf.js, till i stumbled on:
https://www.html5rocks.com/en/tutorials/canvas/hidpi/
before accounting for window.devicePixelRatio. the pdf looks like this:

after accounting for window.devicePixelRation, the pdf looks like this:

@Wotzhs yes, pdf.js demo viewer relies on devicePixelRatio to increase pixel density per pixel via CSS.

Is it a dup for #2750?

@yurydelendik I am confused.  Are you saying that the existing pdfjs should handle the devicePixelRatio as that article suggested?  If so, you don't happen to know when it started doing that, do you?  I am seeing blurriness with wix where they seem to be using pdfjs-dist version 2.305 from Feb 1, 2018, and wondering if they need to either upgrade or make use of the devicePixelRatio stuff in their use of the library.


update tsukuyomi-codegen for the latest nightly compiler

Coverage remained the same at 66.199% when pulling 03697ffd5a6a154a2544f49bed35995e44efeb4f on codegen-fix into f23a40d7f77b92e51c05973c4933dd8283a35b35 on master.


stdlib/public fixes for Windows support

This pull request contains a few changes to allow the stdlib to compile against the Windows/MSVC toolchain.
Of particular note:

There was a compile issue with the use of __cxa_demangle in Errors.cpp as that's unavailable on Windows. The containing method is actually not used on Windows, since it's conditional on SWIFT_SUPPORTS_BACKTRACE_REPORTING; however, I thought it better to provide an implementation for when Windows eventually does support that.

I'm using the Windows API for thread-local storage, but it's still implemented under the pthread_create method; the functionality should be the same, but a higher level of abstraction might be appropriate. The other caveat with this implementation is that TlsAlloc doesn't support a destructor; from what I can gather, it takes care of disposing its own memory, but if I'm incorrect in that or misunderstanding the usage this could cause a memory leak.

@milseman This touches TLS, so you should probably have a look

libc++ at least has a complete abstraction over the threading model (it uses Win32 threading on Windows, and pthreads on Linux/Darwin).  It would be nice if we could use that instead in the LibcShims.cpp if the TLS data is not exposed to others through the pthread_key_t.

From what I can tell, the only usage of those methods is in ThreadLocalStorage.swift, which uses it to cache the result of __swift_stdlib_ubrk_open per-thread. I don't think there's anything that's specifically tied to a pthread implementation, so if the consensus is that an abstraction over the libc++ implementation should be used (perhaps with names like _swift_stdlib_tls_setvalue or _swift_stdlib_tls_setspecific rather than _swift_stdlib_pthread_setspecific) then I can have a go at refactoring that.

Right, this key is for the standard library internal use only. I think that on Darwin, we will still want to call pthread for now (see https://bugs.swift.org/browse/SR-4915, CC @gparker42 and @atrick). Otherwise, I think @troughton's renaming the shims is a good idea to reflect that it's not literally pthread on all platforms anymore.

I've done the renaming. I also modified my original change to use the Fls* rather than Tls* functions on Windows, which should do the same thing (as per the comment in LibcShims.cpp) and ensures the destructor is correctly called.

The suggestion from @compnerd seems like an excellent one and should not be ignored.  Using the C++ standard library as a portability layer, where applicable, is a technique we want to employ elsewhere, e.g. random number generation.

@dabrahams I am not ignoring his suggestions. I agree with you in general, but there are specific (pun intended!) concerns surrounding project policy and especially surrounding Darwin's implementation of thread local storage. (Also, as this PR was originally written, the semantics were potentially different, but I'm completely unfamiliar with "fibers" on Windows. @troughton's latest commit makes this a moot point.)
@compnerd, are you referring specifically to __libcpp_tls_get/set? If so, that addresses performance concerns at it results in the exact same pthread calls. But, using that would require a dependency on libc++ internals, which may not be something we can always assume (cc @dexonsmith). On the other hand, __threading_support is present in Xcode distributions, so I don't have a strong opinion on this. Do you know of any other places where we pretty much assume libc++ specifically is used and available to build the project?
Another approach would be to copy the definitions (or even the file itself) into the Swift project. I don't think that should block this PR, but if anyone thinks that's worth pursuing, please file an SR.

@swift-ci please smoke test

@milseman yeah, I was referring to the __threading_support. We could pull just that implementation from libc++.  However, the thing is that those are what back the std::thread implementation.  I don't expect that the overhead from the class on top of them would make any substantial performance difference.  But, using the std::thread routines completely makes swift agnostic to the underlying threading semantics, which seems like a positive to me (in terms of portability).

@swift-ci please smoke test

@compnerd, there are two constraints here. First, we want to manage the TLS allocation and initialization on the Swift side for the stdlib. The second is that after ABI stability, we definitely want to be making pthread calls on Darwin for highly-Darwin-specific reasons.
Currently, the TLS only caches an ICU pointer, but we want the ability to add pure-Swift constructs onto it in the future. If std::thread can be used to implement some kind of tls_get|set, then I'm totally on board with that for all non-Darwin platforms.

I've rebased this onto master; the only modification is removing what I'd changed in UnicodeNormalization.cpp.
Is this pull request ready to be merged? I know there was some discussion about possibly moving to std::thread for non-Darwin, but I'd consider that slightly out of scope for this PR and best suited to a separate refactor. If preferred, I could remove the TLS additions for Windows and the rest of this could be merged in.

@swift-ci please smoke test

@troughton sorry, if tests passed I will merge.

This now requires rebasing

I'm sorry, I thought this got merged. @compnerd, you've been generalizing this area as well. Does this PR overlap with your work? I'm going to assign to you to help steward this through.

I think everything here has now been covered by @compnerd’s separate PRs, so I’m going to close this.


resource_control: support manage resource group background metadata

What is changed and how it works?

Issue Number: ref #14900
What's Changed:

- Add a new type `ResourceLimiter` to limit the cpu and io resource usage.
- Add ResourceLimiter to ResourceGroup as an optional field, currently, only the `default` resource group contains this field, we may support other groups later.

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

Unit test

Side effects
Release note
None

@nolouch @Connor1996  PTAL again, thanks~

/test

/test

/merge


Basic fix

Hi @Dylan-yxn. It seems that the base of your branch is quite old. Was this PR opened by mistake ?

Sorry for the trouble caused to you. I opened this PR by mistake. At present, I am only learning full homomorphic encryption. Thank you for reminding me.
------------------ 原始邮件 ------------------
发件人:                                                                                "ldsec/lattigo"                                                                                @.>;
发送时间: 2021年9月29日(星期三) 下午3:31
@.>;
<EMAIL_ADDRESS>主题: Re: [ldsec/lattigo] Basic fix (#147)
Hi @Dylan-yxn. It seems that the base of your branch is quite old. Was this PR opened by mistake ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

No trouble :) Thanks for letting us know.


Remove version: CrystalDewWorld.CrystalDiskInfo version 9.3.2

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: Duplicate;


fix: handle unshare_setns_x missing thread_info, chore: thread_info check cleanups

Signed-off-by: Adnan Ali<EMAIL_ADDRESS>What type of PR is this?
/kind bug
/kind cleanup

Any specific area of the project related to this PR?
/area libsinsp

Does this PR require a change in the driver versions?
no

What this PR does / why we need it:
This PR is made of 2 commits:
Commit1:
Adds a safety check to validate if thread information is available before trying to dereference it when parsing the unshare_setns_exit event. Follows https://github.com/falcosecurity/libs/pull/818 to avoid further segfaults I encountered in specific parsers.
Commit2:
Performs a cleanup and follows a consistent method of thread information validation rather than having multiple different ways to check the same condition within a file making it very confusing and inconsistent.
For More Details , please refer to https://github.com/falcosecurity/libs/issues/817#issuecomment-1372257283 where I have provided code examples of the issue.
Which issue(s) this PR fixes:
Continuation from  https://github.com/falcosecurity/libs/issues/817
See https://github.com/falcosecurity/libs/issues/817#issuecomment-1372382513 as well.
Special notes for your reviewer:
I have gone through the whole parsers.cpp file and specifically taken a look at the different parsers to see where we need to perform fixes and cleanups. Most parsers already have both - the thread information check as well as the comparison to nullptr as a consistent means of comparison.
I would also like to point out the below parsers where m_tinfo is dereferenced directly without a check, but I personally have not faced any segfaults due to the same. I have only seen segfaults due to the parsers I have mentioned(the same ones for which I have created PRs), which makes me believe that either the below parsers are fine or my rules do not make use of the parsers which is why segfaults have not yet been observed. I will leave it to the maintainers to decide on the best course of action for these parsers and will be happy to create follow up PR's if needed for the same.
As an example:
parse_rw_exit: https://github.com/falcosecurity/libs/blob/master/userspace/libsinsp/parsers.cpp#L4336
fcntl_exit: https://github.com/falcosecurity/libs/blob/master/userspace/libsinsp/parsers.cpp#L4960
eventfd_exit: https://github.com/falcosecurity/libs/blob/master/userspace/libsinsp/parsers.cpp#L4448
Does this PR introduce a user-facing change?:
NONE

Apologies for the delay in sending this out.
cc: @loresuso @Andreagit97 @FedeDP

No problem sir :)
We are rather late for the Falco 0.34 release; is this a problem?
/milestone 0.11.0

We are rather late for the Falco 0.34 release; is this a problem?

I am fine with either release....personally not a problem for me as I have applied an internal patch to resolve this issue, if that is what you are referring to.

LGTM label has been added.  Git tree hash: cf9957462c281f36be2ceccbc22aad4ab6008e7a

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: adduali1310, FedeDP, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,LucaGuerra]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Enhance "registry apply" to infer --parent from registry configuration.

This makes --parent optional for registry apply by using the configured registry.project when it is nonempty.
Manually verified... I looked into writing tests for this and didn't see a good way to set the registry configuration in an isolated way. Reviewers, WDYT? I'm thinking maybe the tests should save the configuration, modify it as needed, and put it back afterwards, but that seems risky and messy. Since we're not actually running the registry tool in tests, could there be a better way to do this by writing the configuration data structures directly?

The most straightforward way right now is to set the config dir, create a config, save it and go like is done here. Otherwise, I think we'll have to refactor to add some indirection in front of the global connection.ActiveConfig() function that's used throughout the cmd functions.

@theganyo I had an idea about this that I got working with some hacks -- essentially modify connection.ActiveConfig to cache the configuration when it is initially read, and then modify the cached value so that when it is returned by subsequent calls to connection.ActiveConfig, we get the modified configuration. This was bit harder than I had hoped because configurations are returned by value everywhere so I can't just get it, modify it, and expect the cached value to also be modified.
I'll put it in a followup PR so we can discuss it.


Providing avatar information in response to use it in the frontend

Before we've been providing id, join_date and name only. Now, we want to show avatar of the users on their publisher pages so including avatar_url and gravatar info in the response.

Coverage remained the same at 68.717% when pulling 27cd6fbba20d95372797daf7f90024ea037bac3a on improvements/get-user-profile into 41e0168b41c368c8d0319f558bc359906e9fe739 on master.


Idea: integrate organization data somehow?

One example: https://github.com/PyCQA/pylint/ is maintained by Claudiu Popa by some time now.
I suspect there are other cases, like companies with gh accounts employing Romanians. Eg: https://github.com/iMedicare or https://github.com/gorgias

@ionelmc That's an interesting subject.
The initial idea was to fetch projects made by individuals who live in Romania.
The good part of this strategy is to find out what's happening in the Romanian OSS community (on GitHub, of course).
Of course, there are great projects developed by Romanian people, but who do not live in Romania.
You can take a look at @jilix too—the company I worked for—where I created a lot of OSS stuff. I wouldn't really include these, but I'm open to hear other thoughts.
From me you have a 👍 to include OSS from Romanian organizations. Maybe keeping them into separated lists would be fine.

As far as I know iMedicare hires some people from Romania and Claudiu Popa is in Iași. Gorgias has office in Paris or somewhere outside.


Button focus outline missing in Firefox on Windows

Description of Issue
When tabbing focus to <Button />, the focus outline seems to be missing in Firefox on Windows.
###System Configuration
Windows 7
Component Version
v0.1.0
Browser and Version
Firefox: 52
Additional Details (optional)
Image from Firefox on Windows

Steps to Reproduce the Issue

Navigate to http://engineering.cerner.com/terra-core/#/site/button
Press tab key to move focus to <Button /> components

Expected Outcomes

<Button /> to be styled with focus outline

Fixed in #196


Added api for putting state via cli/rest

Motivation
Currently state can be altered inside the function via the context.setState. There is no other mechanism to alter this state outside the function. This pr adds a rest interface and the corresponding cli handler for putState
Modifications
Describe the modifications you've done.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

run java8 tests

@jerrypeng added tests as well

One thing that I’m using here is functionstate class for getting both the state stored as well as for changing the state. Another approach would be to have an alternate structure for changing state. What do you feel?

run java8 tests
run integration tests

One thing that I’m using here is functionstate class for getting both the state stored as well as for changing the state. Another approach would be to have an alternate structure for changing state. What do you feel?

@srkukarni I think that is fine for now

@jerrypeng ^^^

run integration tests

run integration tests


fix: remove-package from demo flavours used

Only delete the specific package from demo flavours used rather than
depend on a package.json being present.
Closes: #8

@cla-bot check

@NathanWalker Please let me know if any other packages need amending based on this change.


[Suggestion] Support for Metal Barrels

https://www.curseforge.com/minecraft/mc-mods/metal-barrels
This mod does for barrels what Iron Chests does for chests.

When I tested they worked even without my addon, don't they work for you?

Never mind then... I hadn't even tried yet. I was working on building a
pack and found this mod as an add on. Iron chests were on the list for this
mod and assumed (apparently incorrectly) that metal barrels would likewise
need some finesse to work.
On Sun, Nov 21, 2021, 11:50 AM juh9870 @.***> wrote:

When I tested they worked even without my addon, don't they work for you?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/juh9870/MoreMountedStorages/issues/4#issuecomment-974863465,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKQ4EY73YVGXHUHFNKKGHDUNEWPZANCNFSM5IOVF73Q
.
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.

I will close this for now. If you face issues while using Metal Barrels on moving contraptions, feel free to open a new issue


Happened again, even after update.

Happened again, even after update.
Error
An error occurred. Please start Anki while holding down the shift key, which will temporarily disable the add-ons you have installed.
If the issue only occurs when add-ons are enabled, please use the Tools > Add-ons menu item to disable some add-ons and restart Anki, repeating until you discover the add-on that is causing the problem.
When you've discovered the add-on that is causing the problem, please report the issue on the add-on support site.
Debug info:
Anki 2.1.47 (dc156f34) Python 3.8.6 Qt 5.14.2 PyQt 5.14.2
Platform: Windows 10
Flags: frz=True ao=True sv=3
Add-ons, last update check: 2021-09-07 00:54:01
Caught exception:
Traceback (most recent call last):
File "aqt\webview.py", line 41, in cmd
File "aqt\webview.py", line 142, in _onCmd
File "aqt\webview.py", line 589, in onBridgeCmd
File "aqt\hooks_gen.py", line 4000, in call
File "C:\Users\gusta\AppData\Roaming\Anki2\addons21\511710206\webview.py", line 26, in handle_bridge_commands
if handle_autocomplete(cmd, editor):
File "C:\Users\gusta\AppData\Roaming\Anki2\addons21\511710206\webview.py", line 65, in handle_autocomplete
nids = col.find_notes(query)
File "anki\collection.py", line 503, in find_notes
File "anki_backend\generated.py", line 762, in search_notes
File "anki_backend_init.py", line 131, in _run_command
anki.errors.SearchError: Invalid search: an opening double quote " was found, but there was no second one to close it. If you want to search for a literal ", prepend a backslash: ".
I tested it with all addons disabled... This time, the symbol was ' and ".  You can reproduce if you type ';" (single quote and after a double quote).
Also, the add-on really stop working, now it's a real bug
Originally posted by @gustavosmendes in https://github.com/ankipalace/anki-field-autocomplete/issues/3#issuecomment-914755111

It should work now. I made a mistake earlier and the changes I made to fix the error didn't make it to Ankiweb.

Found another one - just tried to type to verify if was everything ok. If you type 'single quotation, backslash, single quotation'

Caught exception:
Traceback (most recent call last):
File "aqt\webview.py", line 41, in cmd
File "aqt\webview.py", line 142, in _onCmd
File "aqt\webview.py", line 589, in onBridgeCmd
File "aqt\hooks_gen.py", line 4000, in call
File "C:\Users\gusta\AppData\Roaming\Anki2\addons21\511710206\webview.py", line 26, in handle_bridge_commands
if handle_autocomplete(cmd, editor):
File "C:\Users\gusta\AppData\Roaming\Anki2\addons21\511710206\webview.py", line 66, in handle_autocomplete
nids = col.find_notes(query)
File "anki\collection.py", line 503, in find_notes
File "anki_backend\generated.py", line 762, in search_notes
File "anki_backend_init.py", line 131, in _run_command
anki.errors.SearchError: Invalid search: the escape sequence ' is not defined. If you want to search for a literal backslash </code>, prepend another one: \.
Again, thank you for all your effort. I'm just doing this for trying to help you. <3

Update the add-on again!

Updated the add-on again!

You are the best!


Convert return format of rf.loglikelihood from list to dict for custom metrics

I recommend converting the return format of rf.loglikelihood() from a tuple to a dictionary. This change is necessary because there may be a need to include additional metrics in the future. For instance, I would like to incorporate a metric called "token-length normalized loglikelihood" alongside the existing "byte-length normalized loglikelihood." However, modifying the code is challenging because it would impact all other tasks.

As Is

log_probs = [rf.loglikelihood(ctx, " {}".format(choice))[0] for choice in doc["choices"]]
is_exact_matches = [rf.loglikelihood(ctx, " {}".format(choice))[1] for choice in doc["choices"]]

To Be

log_probs = [rf.loglikelihood(ctx, " {}".format(choice))['log_prob'] for choice in doc["choices"]]
is_exact_matches = [rf.loglikelihood(ctx, " {}".format(choice))['is_exact_match'] for choice in doc["choices"]]

This has been added.


Change share message base on selected app

Hi, I'm using this package to implement external sharing in my application. I have a question, is it possible to change the share message base on the app that user has chosen ? For example:

If user selects "Messages", "Whatsapp" or "Messenger", the share message will be this is message A
If user selects other apps, the share message will be  this is message B

I guess it is possible, but it is a kind of logic that is on your land.

For more details about how to implement this, I recommend asking on https://stackoverflow.com/

@sonlexqt Did you get any solution?


Is it possible to apply a module and all its Dependants ?

I am currently evaluating Terragrunt for my company and have a question.
When you terragrunt run-all apply  on a module X that is depending on module A, terragrunt will apply A -> X.
My question is whether there there is an option to do the vice-versa? I want to do some changes to A, and when I apply it, Terragrunt would figure out that module X (and possibly Y, Z ) all depends on A, and hence apply X, Y, and Z after applying A.
Possible Solutions/WorkArounds:

Print Out the Dependency Graph, and run Apply Manually at each Dependant.
Use --terragrunt-include-dir "xxxxxx" and use run-all apply ( will still need to evaluate the dependency graph myself and it is subject to human error. )

The only solution I found is running run-all apply at the root of my project, but at our infrastructure size this will:

Takes a lot of time.
Will introduce friction between teammates when a drift is detected, although drift needs to be fixed sooner or later, it might be completely unrelated to the current task an engineer is applying.

Hello,
not sure if it is supported yet, currently Terragrunt can show a warning before destroy of A that it has dependent module X

@denis256 How easy is it to implement a "cascading apply"? If I changed something in my VPC, I need to make sure all dependants has be re-applied?

Hi, good question, it may require additional design since cascading apply may generate "ripple" effects on multiple modules

This duplicates #1509.
Coincidentally, they were posted exactly 1 year apart. Neat!


Update README.md

Update readme.md
Addition of new features to readme file
Fixes Issue #6

Devscript

Update readme.md
Addition of new features to readme file
Fixes Issue #6

Looks good to me @PrathmeshMutke . Let @Sejal2801 review the PR once, if she confirms that no issues are there then I'll merge the PR.


Removing braces for arrow function argument

Hello, I removed braces for arrow function arguments. The reason is that according to airbnb style guide, you should omit the braces if the arrow function has only one argument. I find myself writing much more one argument arrow functions, so I always had to remove the braces. Also, adding braces to multi argument arrow functions is easier than removing them from one argument functions.

Makes sense. I also have a few improvements to add. I'll try to merge them all for the next release, including this. Thanks!


Use pow() from musl in order to make (**) deterministic

What I've done here is to pull musl’s implementation pow(), whose answer agrees with our “working case” on both platforms. This is compiled into the Pact library, and called via C unsafe FFI (since it is pure). This way, we aren’t depending on whatever pow() implementation happens to get pulled in by whatever compiler GHC uses.

We should provide some unit tests coverage for this code. At least some basic sanity checks.

Two comments at first glance:

In Haskell projects I usually expect to see C code in a top level directory called cbits.  This gives me a very easy way to at a glance see anything that involves the C FFI.
We need to make sure that this builds with GHCJS, potentially using conditional compilation.


check.http not work .

when i use the follow code,i can't check http request
#!/usr/bin/env python
from consul.base import CB
import consul
import os
def registerMyService():
    podIP= os.environ.get('POD_IP')
    #qkAppCheck=consul.Check.tcp(host=podIP,port=80,interval=5,deregister=1)
    qkAppCheck=consul.Check.http(url='http://'+podIP+':80',interval=5,timeout=5,deregister=1)
    client=consul.Consul(host='<IP_ADDRESS>',port=8500)
    client.agent.service.register(name='quark-app',service_id='quark-app2',port=80,address=podIP,tags=['urlprefix-/'],check=qkAppCheck,in

registerMyService()

error log.
2017/10/30 21:28:17 [WARN] agent: http request failed 'http://<IP_ADDRESS>:80': Get http://<IP_ADDRESS>:80: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)

why ?

Are you using a container to host consul? I had the exact same error while using a container and I believe it is an issue in the docker networking config

Closing for now. Please re-open if @bagerard wasn't helpful.


Different profile used for versions:set and versions:revert .

I have an aggregate POM situation that does not use inheritance, i.e the aggregated POMs do not have a parent-child relationship with the aggregating POM. The directory structure looks like this:

aggregate/
foo/
bar/

The aggregating POM in the aggregate/ directory specifies the foo module:
	<modules>
		<module>../foo</module>

It also has a profile foobar which brings in the bar module:
	<profiles>
		<profile>
			<id>foobar</id>
			<modules>
				<module>../bar</module>
			</modules>
		</profile>

Now here's the interesting part. If I go into the aggregate/ directory and issue the following command:
mvn versions:set -DnewVersion=1.2.3 -DprocessAllModules

Thanks to the processAllModules option, this updates the aggregate POM version and the aggregated module versions both (both foo and bar, as well as there interdependent dependency references)! Yay!
But if I then try to revert the changes:
mvn versions:revert -DprocessAllModules

This command does not revert the POM for the bar module!!! I have to specify the foobar profile to get it to be reverted too:
mvn versions:revert -DprocessAllModules -P foobar

One can make arguments either way about whether the plugin should automatically process modules that are part of profiles that weren't indicated, and for the purposes of this ticket it really doesn't matter.
The important thing is that: shouldn't both goals versions:set and versions:revert use the same module inclusion rules for the processAllModules option? Put another way, shouldn't I be able to use the same options I used for changing the version to commit or revert that version? I don't understand what different semantics of processAllModules would require me to indicate a profile for one of the goals but not the other.

I get the exact same issue in one of my projects (writing this comment just to "bump" the issue). I think versions:revert and versions:set should have the same behaviour and alter the same modules!

Looking into it. I know it's an old one.

RevertMojo only recovers the old copy of the root pom.xml from the backup in the .versionsBackup file. This could indeed be enhanced.


LepIndex - duplicate Sources

For some sources there are duplicates e.g. ‘Bales 1909’, ‘Barragué, 1954’, ‘Bernardi, 1953’. The first one created on 11/09/2016 and the second record created on 22/09/2016 (the only difference is that the later version is marked up as ‘This source is used in another project.’).

This is (almost completely) an artifact of the sandbox.  We just build the same import against it again and again.  Once we go to production those references will (likely) not be there, so the problem is drastically reduced for these early imports.

Closing this as resolved.


Fix Wifi Info Issue on Connect Method

Problem
When Wifi Connect method is called then it checks for error != null but on iOS 18 on success join error is null and it returns the default NetworkData object which is null and have no information.
Solution
Check for error is null explicitly and fetch the Network Info and return correct data
Note: This issue is easily reproducible on demo app on iOS 18.x versions

Thank you for your contribution, @Arslan007! 🙌
I’ll be testing the changes this week and preparing for a new stable release. This release will also include updates from the feat/use_ne_hotspot_ios branch.


Missing "Pulley Holder Bottom" and "M4 spacer 4.1mmx6" stl files.

updated BOM name M4 spacer 4.1mmx6 to Z axis belt tensioner arm hinge spacer.
Pulley holder was an assembly and it was removed. Thank you for pointing things out!


Feature Request(?): Ability to run the browser support tests explicitly

Hey there! 👋
I was wondering if it was possible to expose a way to run the browser support tests explicitly.  I have things that trigger off the change of focus and ran down a bug today that was happening because of the browser support tests changing the focus.
I'd love some way to manually and explicitly run the support suite so it can build the localStorage item, and anywhere we use Ally functions they can use that cached version.
Thanks!

Hey there!
Browser support is evaluated here this function is called internally by any function that requires the supports data. So the tests are executed and cached (in localStorage) by the first ally method you use on an origin.
To kick off the tests manually you could do something like ally.element.disabled(document.body) at a time that suits you (before any other ally method is used).

That's pretty well what I did, I just wanted to make it more explicit in some way.  So it was clear why I was running an ally function at app start-up.   At the moment it's just wrapped in a runAllySupportTests function wrapper to try and make it clear.

ok, so, basically you'd like to expose src/supports/supports (maybe as initialize()) via src/ally to make things more explicit.

That would be ideal I think, just so it's clear what's going on.  That and I'm calling a function to throw away the result right now which works but just seems silly.


Peak of requests towards sqs queue observed in case of exception

Describe the bug
we are using spring cloud version 2.2.1.RELEASE
We have implemented a simple annotated sqs listener  like this:
@SqsListener(value = "${param.queue}", deletionPolicy = SqsMessageDeletionPolicy.ON_SUCCESS)
    public void handleMessages(String payload) {
        logger.debug("Original payload: {}", payload);
   ....

We noticed that, when the listener raises an exception, a peak of requests towards the Sqs queue are issued, and our instance starts consuming memory and cpu resources
Sample
Unfortunately I can't provide a sample ...

Do you mean exception thrown inside @SqsListener annotated method? Are you able to turn on logging to see what kind of requests are issued?

Yes I mean exactly an exception in the @SqsListener annotated method.
Unfortunately we have problems or "rate exceeded limit" on CloudWatch and we can't raise the log level... for now we have simply placed a sleep in our code when an exception occurs, to alleviate the issue..

Assuming you don't have any specific exception handler for this type of exception, Spring Cloud AWS behaviour is to apply deletion policy when error occurs. If Spring Cloud AWS is meant to delete the message it executes deleteMessageAsync using AmazonSQSAsync from AWS SDK. This call can be retried by the SDK itself if it fails but I don't see a place where Spring Cloud AWS itself could trigger peak of requests. I am also not able to reproduce it.
I will close it for now but you are welcome to come back with more details and ideally code that can reproduce this issue. I would be happy to investigate it further then.


Only run webpack_resolve.py when debugging

Running it in production causes permission issues and is not needed.

We should probably add some docs somewhere some time that this command has to be run before a new production/staging environment is set up (docker containers...).


Check if the new license for Volve is out

In that case, open dedicated issues for making the data available for testing in the Intersection

It seems there has been no update, and that the person I was in contact with is on holiday until next week.
I'm afraid it may take a while, so we  may consider to start working with the current version of Volve (the new license will be more permissive, but the current one is anyway compatible with our usage). If we want to play it safe, we can start by hosting the data we need for testing on a private repo, and we access it in the unit tests and the storybook using tokens (both Github Actions and Radix supports secrets, it should be good enough)
Or, if all of this is too much efforts compared to out actual need, we can just keep waiting

Let's wait a couple more weeks, if nothing changes we will start working using the current version of Volve

Apparently everything is ready for publishing the new version, but they are waiting for a key person to come back from sick leave before proceeding

The dataset with the new license has been published, together with 3 more datasets!


ols: nightly-2023-11-04 -> nightly-2024-02-09

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Why has this PR been closed ?

I'm going to make a new PR in an hour or so

see #288276


feat: vaadin-integer-field

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


Unable to select dropdown in winappdriver

In winappdriver for selection of dropdown combox class is not present which is present in winium desktop automation tool.

Does the combo box has a name or automation id.

@punampatil792 You can have following common method for selecting dropdown values.
public static void SetDropDownValue(WindowsElement windowsElement, string dropDownValue)
{
try
{
            windowsElement.SendKeys(Keys.F4);
            Thread.Sleep(500);
            windowsElement.FindElementByName(dropDownValue).FindElementByClassName("ListBoxItem").Click();
            //Assert.AreEqual(dropDownValue, windowsElement.Text);
        }
        catch (Exception err)
        {
            Assert.Fail($"Unable to select text value {dropDownValue} in the dropdown {windowsElement}" + err.Message);
        }

    }

Thank you Vijay

@punampatil792  Is this worked for you ?

No it is not working do you have any another solution ?

Try below code. It is working for me.

Ok, Thank you.
On Mon, 11 May, 2020, 4:54 PM dillibabu123<EMAIL_ADDRESS>wrote:

Try below code. It is working for me.
[image: image]
https://user-images.githubusercontent.com/55104669/81556302-b8f35e00-93a7-11ea-847a-bdf0ddb06d20.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/WinAppDriver/issues/1067#issuecomment-626643089,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOCFU5WJJHKUBAR37K6WWJ3RQ7N6PANCNFSM4KW7UZJA
.


Dev Blog Post - Replica Signed Queries

Thank you for your contribution to the IC Developer Portal.
Before submitting your Pull Request, please make sure that:

[ ] You have updated the .github/CODEOWNERS file according to your changes.
[ ] You have registered new documents to /sidebars.js if any.

Thanks a lot for drafting the post. Very nice!
I left some comments mostly about removing the rust and javascript code and quoting specification instead.

Did a final pass. It looks great. Thanks a lot.


rename fixtures with * in path to use asterick

fixes #204

@BarryThePenguin Oh thanks so much, I appreciate it!
So, does remark now build on Windows?

I can clone it on windows now. I haven't tried building it, the only script I've run is test-coverage. lerna doesn't bootstrap without admin privilege. That's about as far as I've got...

Oh really? Hmm. I use nvm/n, which are really useful and fix the admin things. Maybe with nvm-windows it works?
But great the tests run now :)

Oh.. nah, even with an environment/permissions setup properly, you still need admin privileges to create symlinks :( lerna/lerna#319 and lerna/lerna#237 explain a bit


Missing the CSI Snapshotter sidecar - blockSnapshots will never work

vSphere Server Info

vSphere version: 7.0.3

Rancher Server Setup

Rancher version: 2.7.3

Information about the Cluster

Kubernetes version: 1.25.7 RKE2

Describe the bug
RKE2 has added the snapshot controllers and vSphere CSI supports VolumeSnapshots. In fact, this chart added a feature flag to add support here however it seems like it was forgotten that the CSI Snapshotter sidecar is required, as per VMware's installation instructions: https://raw.githubusercontent.com/kubernetes-sigs/vsphere-csi-driver/v2.4.0/manifests/vanilla/deploy-csi-snapshot-components.sh
Notice they apply the following patch:
# Update the vSphere CSI driver to add the snapshot side car.
tmpdir=$(mktemp -d)
echo "creating patch file in tmpdir ${tmpdir}"
cat <<EOF >> "${tmpdir}"/patch.yaml
spec:
  template:
    spec:
      containers:
        - name: csi-snapshotter
          image: 'k8s.gcr.io/sig-storage/csi-snapshotter:${release}'
          args:
            - '--v=4'
            - '--timeout=300s'
            - '--csi-address=\$(ADDRESS)'
            - '--leader-election'
          env:
            - name: ADDRESS
              value: /csi/csi.sock
          volumeMounts:
            - mountPath: /csi
              name: socket-dir
EOF

echo -e "Patching vSphere CSI driver.."
kubectl patch deployment vsphere-csi-controller -n vmware-system-csi --patch "$(cat "${tmpdir}"/patch.yaml)"
echo -e "✅ Successfully patched vSphere CSI driver, please wait till deployment is updated.."
echo -e "\n✅ Successfully deployed all components for CSI Snapshot feature.\n"

To Reproduce
Create a VolumeSnapshotDriver and a VolumeSnapshot with blockSnapshots enabled. You will notice that the snapshot will never complete.
Expected Result
The snapshot should complete and the CSI Snapshotter should be added as a sidecar

This chart should also install the other resources listed within the deploy-csi-snapshot-components.shscript

Deploys the VolumeSnapshot CRDs
Creates RBAC rules to support VolumeSnapshot
Deploys snapshot-controller in kube-system namespace
Deploys the snapshot validation webhook
Patches vSphere CSI driver to deploy the csi-snapshotter sidecar

Otherwise the cluster will still lack support for Snapshot, doesn't it?

RKE2 already does those steps I believe. @santimar

I do not see any VolumeSnapshotClass after the CSI is installed.
Do i need to create it by myself? StorageClass is created automatically

I deployed rancher-vsphere-csi:100.4.0+up2.6.2-rancher1 on my RKE1 cluster. Can I get Snapshot support ? Do I need to wait for an update of this chart ?

Any progress on this issue?

Closing in favor of https://github.com/rancher/rancher/issues/41321 which is now assigned


Github command for adding/duplicating issues across repos

Recently, there have been issues that crop up in one repo and need to be propagated across multiple related repos (such as all of our CSharp repos). It would be nice to have some command line tools to create these on the fly for multiple repos, or even clone from an existing one.
Some additional features/implementation details-

The ability to have a filter for the list of repos to create the issue for would be nice (such as AndcultureCode.CSharp.*)
A confirmation prompt with the list of affected repos would be nice (we have a prompt.confirmOrExit function that should make this trivial)
The ability to clone from an existing issue would be nice (Generally, we identify the issue in one place and then end up copy/pasting the issue to other repos)
If it's not too much extra code, it might be nice to have something to look up existing issues and prompt the user if they are attempting to create a duplicate (by name)

Just noting this here - I have started hacking on this based on the 2.0.0 branch, and it is looking like some decent refactoring is involved to better structure + support subcommands so that our github command does not become a bloated mess of options.
The API I am proposing is similar to the gh command line tool, at least for breaking out subcommands based on the entity (topic, issue, etc): https://cli.github.com/manual/gh_issue
and-cli github issue --list --repo AndcultureCode.Cli
and-cli github topic --add test-topic --repo AndcultureCode.Cli
and-cli github topic --list --repo AndcultureCode.Cli


Contributing pshinting_impact check / psautohint_stats condition

I liked the output of the hinting impact check, so I wrote a similar condition and check for psautohinted cff fonts.
Is there any use to include this in the public checks of the repo, and if so, where would I add it in a PR?

I think it may be good to extend the existing check to support CFF hints as well, instead of adding a new one.


Problem with sqlite3_load_extension

Hi,
Im getting this error:

Im using latest stable version with latest macos version.
Thanks.

Try to build with SQLITECPP_INTERNAL_SQLITE e.g.
cmake -DSQLITECPP_INTERNAL_SQLITE=ON ..

Yes, the problem is your where using an external SQLite3 library without the support for load extensions.


[BUG] Corrupted log when using SearchEngine

Hi again,
I'm stuck with a strange behavior, that from my tests seems to be related to the use of the SearchEngine.
I'm using a SingletonLogger that logs everything to stdout and persists that log onto a file.
When the program runs, the index takes a bit of time to be calculated and if I check the logfile, I can correctly see everything printed until this point. After the SearchEngine finishes calculating the index, the first row of the logfile becomes a series of nul values.
Below a sample of code and of the log file.
Can anyone give me pointers to solve this?
_logger = Logger()

[code doing stuff, collecting collection mainly]

_logger.info("Building index...")
SearchEngine("new-index").index(collection, show_progress=False)
_logger.info("Index built.")

logfile.log

Hi @AmenRa ,
Here is a script that reproduces the error. The logger class only has this problem when SearchEngine().index() is run, if commented the logs are correctly printed.
I may have done something wrong, but I can't seem to find the cause for this since every other test I've run didn't reproduce the issue.
binary_log.zip

Hi, i removed from commons.logger import Logger.
Everything seems to work fine on my end.
I get this in both my terminal and the log file:
[2023-09-02 09:16:28,031] {binary_log.py:55} INFO - Building index...
[2023-09-02 09:16:28,303] {binary_log.py:57} INFO - Index built.

Is that what is present in the log file? The commons.logger was just the Logger class implementing a singleton, I'll upload the script using that class. I'll also try to see if the issue is present without importing the commons. Thanks for now.

Hi @AmenRa I did some further testing and I found something interesting. If I change the order of the lines
logger.info("Building index...")  SearchEngine("new-index").index(collection, show_progress=False)  logger.info("Index built.")
to
logger.info("Building index...")  logger.info("Index built.")  SearchEngine("new-index").index(collection, show_progress=False)
the log file becomes empty. So it seems from the tests I conducted that somehow rertiv is still sending something to stdout even when with show_progress=False. If I comment the indexing line, the log correctly is shown in the file. I'm running on windows 11 with python 3.10.0. I don't know if I can give you any more information, but please tell me if you need something else to try to reproduce the error in your environment.

Hi @AmenRa , thanks for your help! I ran the sample code you provided me, sadly (?) everything worked fine, the output was the expected one in both the console and the logfile. Seems tqdm isn't the culprit here. I agree that the problem may lie in the fact that I'm using Windows. As I was saying this kind of behavior seems to be linked on how files are opened under the hood. I'll try to dig deeper and keep you updated if I find anything interesting, since this behavior is strange and not as intended I guess!