Skip to content

Commit

Permalink
Merge branch 'main' into OPIK-50-add-commands-for-installing-start-an…
Browse files Browse the repository at this point in the history
…d-stop-the-opik-server
  • Loading branch information
alexkuzmik authored Sep 6, 2024
2 parents 3d58352 + 7ab906b commit 41db3ae
Show file tree
Hide file tree
Showing 70 changed files with 796 additions and 587 deletions.
46 changes: 0 additions & 46 deletions .github/workflows/deploy_custom_env.yaml

This file was deleted.

13 changes: 6 additions & 7 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -239,6 +239,9 @@ The Opik frontend is a React application that is located in `apps/opik-frontend`
In order to run the frontend locally, you need to have `npm` installed. Once installed, you can run the frontend locally using the following command:

```bash
# Run the backend locally with the flag "--local-fe"
./build_and_run.sh --local-fe

cd apps/opik-frontend

# Install dependencies - Only needs to be run once
Expand All @@ -248,20 +251,16 @@ npm install
npm run start
```

You can then access the development frontend at `http://localhost:5174/`. Any change you make to the frontend will be updated in real-time.

The dev server is set up to work with Opik BE run on `http://localhost:8080`. All requests made to `http://localhost:5174/api` are proxied to the backend.
The server port can be changed in `vite.config.ts` file section `proxy`.
You can then access the development frontend at `http://localhost:5173/`. Any change you make to the frontend will be updated in real-time.

> [!NOTE]
> You will need to have the backend running locally in order for the frontend to work. For this, we recommend running a local instance of Opik using `opik server install`.
> You will need to open the FE using `http://localhost:5173/` ignoring the output from the `npm run start` command which will recommend to open `http://localhost:5174/`. In case `http://localhost:5174/` is opened, the BE will not be accessible.
Before submitting a PR, please ensure that your code passes the test suite, the linter and the type checker:

```bash
cd apps/opik-frontend

npm run test
npm run e2e
npm run lint
npm run typecheck
```
Expand Down
42 changes: 21 additions & 21 deletions README.md

Large diffs are not rendered by default.

4 changes: 1 addition & 3 deletions apps/opik-backend/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,14 +50,12 @@ health:
distributedLock:
lockTimeoutMS: ${DISTRIBUTED_LOCK_TIME_OUT:-500}

bulkOperations:
size: ${BULK_OPERATION_SIZE:-200}

redis:
singleNodeUrl: ${REDIS_URL:-}

authentication:
enabled: ${AUTH_ENABLED:-false}
apiKeyResolutionCacheTTLInSec: ${AUTH_API_KEY_RESOLUTION_CACHE_TTL_IN_SEC:-5} #0 means no cache
sdk:
url: ${AUTH_SDK_URL:-''}
ui:
Expand Down
7 changes: 6 additions & 1 deletion apps/opik-backend/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
<stringtemplate.version>3.45.2</stringtemplate.version>
<jakarta.annotation.version>2.1.1</jakarta.annotation.version>
<liquibase-clickhouse.version>0.7.2</liquibase-clickhouse.version>
<clickhouse-java.version>0.6.4</clickhouse-java.version>
<clickhouse-java.version>0.6.5</clickhouse-java.version>
<org.mapstruct.version>1.5.5.Final</org.mapstruct.version>
<testcontainers.version>1.20.0</testcontainers.version>
<uuid.java.generator.version>5.1.0</uuid.java.generator.version>
Expand Down Expand Up @@ -53,6 +53,11 @@
</dependencyManagement>

<dependencies>
<dependency>
<groupId>com.newrelic.agent.java</groupId>
<artifactId>newrelic-api</artifactId>
<version>8.14.0</version>
</dependency>
<dependency>
<groupId>io.dropwizard</groupId>
<artifactId>dropwizard-core</artifactId>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -73,4 +73,5 @@ List<Dataset> find(@Bind("limit") int limit,

@SqlQuery("SELECT workspace_id FROM datasets WHERE id = :id")
String getWorkspaceId(@Bind("id") UUID id);

}
Loading

0 comments on commit 41db3ae

Please sign in to comment.