ITensor / ITensor/ITensorMPS.jl

Add `firstsiteinds(::MPS)`

Open
#46 1 comment 0 reactions 1 assignee View on GitHub

@mtfishman is already working on this.

Since Oct 7, 2020.

enhancement
Dominant language
Julia
Stars
85
Forks
27
Avg merge
22m
Merged PRs (30d)
1

Description

To help with generic code for MPS/MPO (i.e. code that works for both), it would help to define firstsiteinds(M::MPS) = siteinds(M).

Also, it may be good to think about the assumption that is often used in MPS functions that MPS have only one site index, since sometimes it can be more natural to have more than one (for example a Choi matrix representing a pure quantum process, or the boundary MPS of a double layer PEPS). For that case, it may be useful to define functions like allsiteinds(::MPS/MPO), that searches for all site-like indices (indices that are not shared with any neighbors in the MPS/MPO) and return them as a vector of IndexSets.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.