Skip to content

EQA merge #254

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 92 commits into
base: main
Choose a base branch
from
Open

EQA merge #254

wants to merge 92 commits into from

Conversation

hello-peiqi
Copy link
Contributor

@hello-peiqi hello-peiqi commented Feb 4, 2025

Anyone want to test this system based on this instruction Codes are currently on hello-peiqi/grapheqa_merge branch.

You don't have to worry about discord bot.

I also modified DynaMem and AI Pick up. So please also check:

Start with some simple questions such as "is there a monitor on the table?" "what is the color of the chair"

Base automatically changed from hello-peiqi/grapheqa to main February 5, 2025 15:18


class SparseVoxelMap(SparseVoxelMapBase):
def __init__(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be good to add a docstring here, this is a lot of params (mostly borrowed - can use @OVERRIDES)

DYNAMEM_VISUAL_GROUNDING_PROMPT, model="gpt-4o-2024-05-13"
)

def find_alignment_over_model(self, queries: str):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docstring?

print("Points close the the point are not similar to the text!")
return torch.max(alignments[distances < distance_threshold]) >= similarity_threshold

def get_2d_map(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this different from the base one? If not - just override

if so maybe we could do some refactoring for less duplication?

if debug:
import matplotlib.pyplot as plt

plt.subplot(2, 2, 1)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can probably delete this debug code

print(f"Error: {e}")
return None

def localize_with_mllm(self, text: str, debug=True, return_debug=False):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this new function could probably use some documentation

@@ -220,6 +221,9 @@ def __init__(
logger.warning("Docker environment detected. Disabling GUI.")
rr.init("Stretch_robot", spawn=spawn_gui)

if output_path is not None:
rr.save(output_path / "rerun_log.rrd")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you find these rerun logs useful?

Copy link
Contributor Author

@hello-peiqi hello-peiqi Feb 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not yet, but I guess it is for automatically saving rerun res; since grapheqa does that, I think it is useful especially for some researchers want to collect demo for their websites.

@@ -95,7 +95,6 @@ fi
# hellorobotinc/stretch-ai_cuda-11.8:$VERSION: the docker image to run
run_docker_command run \
-it \
--runtime nvidia \
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why do you get rid of this? is it no longer necessary?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It gives me bugs

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you still run with access to the NVIDIA hardware?

self._manipulation_radius = parameters["motion_planner"]["goals"]["manipulation_radius"]
self._voxel_size = parameters["voxel_size"]

# self.image_processor = VoxelMapImageProcessor(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dead code


def execute_action(
self,
text: str,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

documentation - say what actions can be?

Describe the CURRENT IMAGE. Pay special attention to features that can help answer the question or select future actions.
Describe the SCENE GRAPH. Pay special attention to features that can help answer the question or select future actions.
"""
prompt += "You should go near the blue couch before answering the question with confidence. You should see a full image of the couch before answering with confidence"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a remnant from initial debugging of the planner and can be removed.

hello-peiqi and others added 17 commits March 28, 2025 14:26
* update qwen

* update qwen25client

* initial commit

* checkpoint

* bug fixing

* bug fixing

* precommit check

* debug exploration

* fix qwen bug

* update

* find images update

* checkpoint

* small fix

* update history

* update

* update model

* add siglip2

* LFD update (#259)

* update lfd docs

* fix bug

* change hyperparameter

* update docker for data collection

* add docker file

* update docs to add docker instructions

---------

Co-authored-by: Chris Paxton <[email protected]>

* update exploration

* fix exploration bugs

* change explrotion target

* save changes

* fix navigation bug

* discord

* fix bugs

* reproduce fix bugs commit

* update back

* rerun save

---------

Co-authored-by: Mohamed Fazil <[email protected]>
Co-authored-by: Peiqi Liu <[email protected]>
Co-authored-by: Chris Paxton <[email protected]>
@hello-peiqi hello-peiqi changed the title GraphEQA merge EQA merge May 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants