-
Notifications
You must be signed in to change notification settings - Fork 153
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
Experimental Kintex7 xc7k325 support (alternative) #1844
base: master
Are you sure you want to change the base?
Conversation
Looks like you need to rebase before I can test CI. |
c2472db
to
d395907
Compare
e2fc2f2
to
44d1ee8
Compare
Indeed. Sorry about the hold-up. I finally managed the intended rebase. Please note that this commit has been tested with DSP disabled for Kintex7. DSPs were enabled for all devices in #1816. If this turns out to be an issue for this PR, we could try deactivating the DSPs temporarily. |
Are there any updates on the end of CI? Still waiting for this PR (or equivalently #1794) to be confirmed for merge. |
@unbtorsten - Can you rebase this onto master so that the GitHub Action CI (which replaces kokoro) runs here? |
Signed-off-by: Torsten Reuschel <[email protected]>
Signed-off-by: Torsten Reuschel <[email protected]>
Signed-off-by: Torsten Reuschel <[email protected]>
…, since test returns empty list Signed-off-by: Torsten Reuschel <[email protected]>
…ase of empty list of IOI tiles Signed-off-by: Torsten Reuschel <[email protected]>
Signed-off-by: Torsten Reuschel <[email protected]>
Signed-off-by: Torsten Reuschel <[email protected]>
Done |
Is CI still broken? How can we move forward with this? |
@unbtorsten They currently are working on getting the license working in the new GitHub actions based workflow |
This is an attempt alternative to #1794. Differences are:
This pull request is to determine whether changes pass automatic testing.