Skip to content

Change references from arXiv to conference publications#1

Open
harshakokel wants to merge 1 commit into
ctpelok77:mainfrom
harshakokel:patch-1
Open

Change references from arXiv to conference publications#1
harshakokel wants to merge 1 commit into
ctpelok77:mainfrom
harshakokel:patch-1

Conversation

@harshakokel

Copy link
Copy Markdown

Updated references for Kambhampati et al. and Liu et al. to reflect conference publication.

Updated references for Kambhampati et al. and Liu et al. to reflect conference publication.

@ctpelok77 ctpelok77 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please run
./tests/run-tests.sh
before committing.

Comment thread literatur.bib
}

@Misc{kambhampati-et-al-arxiv2024,
Misc{kambhampati-et-al-arxiv2024, --> use kambhampati-et-al-icml2024

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please keep the citation as is, just add a comment above. This is due some differences in these two citations (authors order, title).

Comment thread literatur.bib
}

@Misc{liu-et-al-arxiv2023b,
{liu-et-al-arxiv2023b, --> liu-et-al-iclr2024

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please remove the Misc citation.

Comment thread literatur.bib
Aohan Zeng and Zhengxiao Du and Chenhui Zhang and Sheng Shen and
Tianjun Zhang and Yu Su and Huan Sun and Minlie Huang and Yuxiao Dong and Jie Tang",
title = "{AgentBench}: Evaluating {LLMs} as Agents",
year = "2024",

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please remove year, it is already in the crossref.
Make sure there is only one line break between citations.

You can use ./tests/run-tests.sh to check validity.

@harshakokel

Copy link
Copy Markdown
Author

I did not setup this package. Just made edits in the GitHub editor.

Keep this open. Will circle back to this when I have more changes around next submission.

@ctpelok77

Copy link
Copy Markdown
Owner

I integrated your changes, you can remove this pr

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.

2 participants