From 0772764623983f529513a3ce0486434789d74c49 Mon Sep 17 00:00:00 2001
From: Diwakar Ravichandran <53701742+bundle-adjuster@users.noreply.github.com>
Date: Thu, 2 Mar 2023 10:59:13 +0530
Subject: [PATCH] Including installation for metis in dockerfile

Seems like metis was a hidden dependency that doesn't break the installation. But gives error when trying to execute the tests. Adding metis to the dockerfile seems to have fixed everything. Not sure if some dependency of metis caused this or metis itself. For now this docker build seems to be stable.

The requirement for timezone still exists. Adding TZ env did not seem to solve the issue will raise another PR if the fix seems to work.
---
 scripts/docker/Dockerfile | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/scripts/docker/Dockerfile b/scripts/docker/Dockerfile
index 74ff8b006..7c376e3ea 100644
--- a/scripts/docker/Dockerfile
+++ b/scripts/docker/Dockerfile
@@ -5,6 +5,7 @@ MAINTAINER Antoni Rosinol "arosinol@mit.edu"
 
 # To avoid tzdata asking for geographic location...
 ENV DEBIAN_frontend noninteractive
+ENV TZ Asia/Kolkata
 
 # Set the working directory to /root
 ENV DIRPATH /root
@@ -18,7 +19,7 @@ RUN apt-get update && apt-get install -y git cmake
 RUN apt-get update && apt-get install -y xvfb
 
 # Install GTSAM
-RUN apt-get update && apt-get install -y libboost-all-dev
+RUN apt-get update && apt-get install -y libboost-all-dev libmetis-dev
 ADD https://api.github.com/repos/borglab/gtsam/git/refs/heads/master version.json
 RUN git clone https://github.com/borglab/gtsam.git
 RUN cd gtsam && \