Skip to content
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

issue fix - Accelerator input and output memory #14

Closed
wants to merge 1 commit into from

Conversation

mahinlma
Copy link

@mahinlma mahinlma commented Mar 4, 2020

accelerator is attached to bank0 and bank1 for input and output

accelerator is attached to bank0 and bank1 for input and output
@mahinlma
Copy link
Author

mahinlma commented Mar 4, 2020

fixed this issue

---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-16-5ff0c5b7b385> in <module>
      1 in_buffer = allocate((old_height, old_width, 3), np.uint8, 
----> 2                      target=resize_design.DDR0)
      3 out_buffer = allocate((new_height, new_width, 3), np.uint8, 
      4                       target=resize_design.DDR1)

~/.local/lib/python3.5/site-packages/pynq/overlay.py in __getattr__(self, key)
    354         """
    355         if self.is_loaded():
--> 356             return getattr(self._ip_map, key)
    357         else:
    358             raise RuntimeError("Overlay not currently loaded")

~/.local/lib/python3.5/site-packages/pynq/overlay.py in __getattr__(self, key)
    880         else:
    881             raise AttributeError(
--> 882                 "Could not find IP or hierarchy {} in overlay".format(key))
    883 
    884     def _keys(self):

AttributeError: Could not find IP or hierarchy DDR0 in overlay

@yunqu
Copy link

yunqu commented Mar 4, 2020

Hi @mahinlma , I think this issue has to be resolved differently, because on some platforms the original code may just work (like U280 where the memory interfaces are DDR0 and DDR1); so the code has to adapt to available interfaces provided by the platform. This requires a bit more work on the notebook changes. I'd appreciate if you could do that change instead; or I can resolve this issue later.

@yunqu
Copy link

yunqu commented Mar 6, 2020

#16 should solve it. Closing.

@yunqu yunqu closed this Mar 6, 2020
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.

None yet

2 participants