Skip to content

Commit

Permalink
Disable tracing in Promptflow by default
Browse files Browse the repository at this point in the history
  • Loading branch information
ninghu committed Jan 7, 2025
1 parent 407970e commit bcfd86b
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 8 deletions.
6 changes: 0 additions & 6 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -44,9 +44,3 @@ repos:
# Use black profile for isort to avoid conflicts
# see https://github.com/PyCQA/isort/issues/1518
args: ["--profile", "black", --line-length=120]
- repo: https://github.com/pylint-dev/pylint
rev: v3.0.3
hooks:
- id: pylint
name: pylint
args: [ --rcfile=pylintrc, --output-format=parseable ]
2 changes: 1 addition & 1 deletion src/promptflow-core/promptflow/executor/flow_executor.py
Original file line number Diff line number Diff line change
Expand Up @@ -882,7 +882,7 @@ def _get_node_referenced_flow_inputs(

@staticmethod
def _tracing_disabled():
return os.environ.get("PF_DISABLE_TRACING", "false").lower() == "true"
return os.environ.get("PF_DISABLE_TRACING", "true").lower() == "true"

@contextlib.contextmanager
def _start_flow_span(self, inputs: Mapping[str, Any]):
Expand Down
2 changes: 1 addition & 1 deletion src/promptflow-tracing/promptflow/tracing/_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@


def is_tracing_disabled():
return os.environ.get("PF_DISABLE_TRACING", "false").lower() == "true"
return os.environ.get("PF_DISABLE_TRACING", "true").lower() == "true"


def serialize(value: object, remove_null: bool = False, serialization_funcs: Dict[type, Callable] = None) -> dict:
Expand Down

0 comments on commit bcfd86b

Please sign in to comment.