mirror of
https://github.com/hwchase17/langchain.git
synced 2025-09-07 22:11:51 +00:00
multiple: Remove unnecessary Ruff suppression comments (#21050)
## Summary I ran `ruff check --extend-select RUF100 -n` to identify `# noqa` comments that weren't having any effect in Ruff, and then `ruff check --extend-select RUF100 -n --fix` on select files to remove all of the unnecessary `# noqa: F401` violations. It's possible that these were needed at some point in the past, but they're not necessary in Ruff v0.1.15 (used by LangChain) or in the latest release. Co-authored-by: Erick Friis <erick@langchain.dev>
This commit is contained in:
@@ -55,7 +55,7 @@ _dataset_dict = {
|
||||
def import_fiddler() -> Any:
|
||||
"""Import the fiddler python package and raise an error if it is not installed."""
|
||||
try:
|
||||
import fiddler # noqa: F401
|
||||
import fiddler
|
||||
except ImportError:
|
||||
raise ImportError(
|
||||
"To use fiddler callback handler you need to have `fiddler-client`"
|
||||
@@ -169,7 +169,7 @@ class FiddlerCallbackHandler(BaseCallbackHandler):
|
||||
)
|
||||
print( # noqa: T201
|
||||
f"adding model {self.model} to project {self.project}."
|
||||
"This only has to be done once." # noqa: T201
|
||||
"This only has to be done once."
|
||||
)
|
||||
try:
|
||||
self.fiddler_client.add_model(
|
||||
@@ -181,7 +181,7 @@ class FiddlerCallbackHandler(BaseCallbackHandler):
|
||||
except Exception as e:
|
||||
print( # noqa: T201
|
||||
f"Error adding model {self.model}: {e}."
|
||||
"Fiddler integration will not work." # noqa: T201
|
||||
"Fiddler integration will not work."
|
||||
)
|
||||
raise e
|
||||
|
||||
|
Reference in New Issue
Block a user