Hey @kevinphillips81! I'm here to help you with any bugs, questions, or contributions. Let's tackle this together!

To address the issue of invalid labels with backticks being generated by the upsert_triplet method in the llama-index-graph-stores-neptune package, you should ensure that the rel parameter is properly sanitized before being used in the query. The current implementation replaces spaces with underscores and converts the relationship type to uppercase, which is correct. However, if backticks or other invalid characters are present, you should further sanitize the rel string to remove or replace these characters.

Here's a snippet of the upsert_triplet method for reference:

def upsert_triplet(self, subj: str, rel: str, obj: str) -> None:
    """Add triplet to the graph."""
    query = """
        MERGE (n1:`%s` {id:$subj})
        MERGE (n2:`%s` {id:$obj})
        MERGE (n1)-[:`%s`]->(n2)
    """

    prepared_statement = query % (
        self.node_label,
        self.node_label,
        rel.replace(" ", "_").upper(),
    )

    self.query(prepared_statement, {"subj": subj, "obj": obj})

If the issue persists, you may need to implement additional logic to sanitize the rel string, ensuring it conforms to the valid label syntax for your specific use case ^[1].

To continue talking to Dosu, mention @dosu.

^{Help Dosu learn! Give it feedback: Great Response | Irrelevant Answer | Incorrect Sources | Too Verbose | Hallucination | Other}

To mitigate, you need to update the upsert_triplet method in the /graph_stores/neptune/base.py file as follows:

def upsert_triplet(self, subj: str, rel: str, obj: str) -> None:
        """Add triplet to the graph."""
        query = """
            MERGE (n1:`%s` {id:$subj})
            MERGE (n2:`%s` {id:$obj})
            MERGE (n1)-[:`%s`]->(n2)
        """

        prepared_statement = query % (
            self.node_label.replace("`",""),
            self.node_label.replace("`",""),
            rel.replace(" ", "_").replace("`","").upper(),
        )

        self.query(prepared_statement, {"subj": subj.replace("`",""), "obj": obj.replace("`","")})

Created a PR to solve this issue using your implemented solution @kevinphillips81