Skip to content
This repository has been archived by the owner on Feb 8, 2024. It is now read-only.

Commit

Permalink
clean up (#2206)
Browse files Browse the repository at this point in the history
* clean up python

* clean up domain
  • Loading branch information
vabarbosa authored Nov 28, 2023
1 parent d99791b commit 3892dce
Show file tree
Hide file tree
Showing 10 changed files with 18 additions and 36 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,10 +34,10 @@ jobs:

- run: npm ci

- name: Install Python 3.8
uses: actions/setup-python@v2
- name: Install Python 3.10
uses: actions/setup-python@v4
with:
python-version: 3.8
python-version: '3.10'

- name: Load pip cache
uses: actions/cache@v3
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,10 +26,10 @@ jobs:

- run: npm ci

- name: Install Python 3.8
uses: actions/setup-python@v2
- name: Install Python 3.10
uses: actions/setup-python@v4
with:
python-version: 3.8
python-version: '3.10'

- name: Load pip cache
uses: actions/cache@v3
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ The textbook is intended for use as a university quantum algorithms course suppl

## Getting Started
### Pre-requisites
1. Python 3.8 environment with `textbook_converter` requirements installed
1. Python 3.10 environment with `textbook_converter` requirements installed

```
$ cd platypus/converter/textbook-converter
Expand Down
21 changes: 1 addition & 20 deletions converter/textbook-converter/requirements-test.txt
Original file line number Diff line number Diff line change
@@ -1,21 +1,2 @@
cython==0.29.32
cmake==3.24.1
nbqa==1.3.1
pylint==2.14.3
pylint==3.0.2
scour==0.38.2
qiskit==0.39.0
qiskit-nature==0.4.1
qiskit-machine-learning==0.4.0
qiskit-finance==0.3.2
scikit-learn==1.0.1
tensorflow==2.9.3
matplotlib==3.5.2
bokeh==3.0.2
pyscf==2.0.1
pylatexenc==2.10
ipywidgets==8.0.1
qiskit-machine-learning[sparse]
qiskit-experiments==0.4.0
git+https://github.com/qiskit-community/qiskit-textbook.git#subdirectory=qiskit-textbook-src
git+https://github.com/anedumla/quantum_linear_solvers
git+https://github.com/NCCR-SPIN/topological_codes.git
2 changes: 1 addition & 1 deletion converter/textbook-converter/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
install_requires=requirements,
packages=find_packages(include=['textbook_converter']),
url='https://github.com/Qiskit/platypus/tree/main/converter/textbook-converter',
python_requires='>=3.6',
python_requires='>=3.10',
entry_points={
'nbconvert.exporters': [
'textbook = textbook_converter:TextbookExporter',
Expand Down
2 changes: 1 addition & 1 deletion frontend/vue/components/OneXP/OneXPBanner.vue
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
/>
<a
class="one-xp-banner__cta"
href="https://learning.quantum-computing.ibm.com"
href="https://learning.quantum.ibm.com"
rel="noopener"
target="_blank"
@click="onClick"
Expand Down
4 changes: 2 additions & 2 deletions frontend/vue/components/UserAccount/AccountAdmin.vue
Original file line number Diff line number Diff line change
Expand Up @@ -31,13 +31,13 @@
<ul class="account-admin__info__steps">
<li>
{{ $translate("Create an account in") }}
<BasicLink url="https://quantum-computing.ibm.com/">
<BasicLink url="https://quantum.ibm.com/">
IBM Quantum.
</BasicLink>
</li>
<li>
{{ $translate("Access your") }}
<BasicLink url="https://quantum-computing.ibm.com/account">
<BasicLink url="https://quantum.ibm.com/account">
{{ $translate("account page") }}
</BasicLink>
{{ $translate("and copy the value from the API token input.") }}
Expand Down
6 changes: 3 additions & 3 deletions server/app.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ import { LOCALES, translate } from '@mathigon/studio/server/utilities/i18n'
import { generateMockData } from './populate-database'

import {
CONFIG, NOTATIONS, TEXTBOOK_HOME, LATEST_TEXTBOOK_VERSION, TRANSLATIONS, UNIVERSAL_NOTATIONS, LEARNING_REDIRECTS,
CONFIG, NOTATIONS, TEXTBOOK_HOME, LEARNING_HOME, LATEST_TEXTBOOK_VERSION, TRANSLATIONS, UNIVERSAL_NOTATIONS, LEARNING_REDIRECTS,
findNextSection, findPrevSection, getSectionIndex, isLearningPath,
updateGlossary, loadLocaleRawFile, tocFilterByType, removeVersionPrefix
} from './utilities'
Expand Down Expand Up @@ -137,8 +137,8 @@ const start = () => {
// .redirects({'/login': '/signin'})
.accounts()
.redirects({
'/': TEXTBOOK_HOME,
'/textbook': TEXTBOOK_HOME,
'/': LEARNING_HOME,
'/textbook': LEARNING_HOME,
...LEARNING_REDIRECTS
})
.get('/locales/:locale', (req, res) => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import { IS_PRODUCTION } from '../../configuration'

import { QCHttpError } from './quantum-computing-domain'

export const QC_URL = IS_PRODUCTION ? 'https://auth.quantum-computing.ibm.com/api' : 'https://auth-dev.quantum-computing.ibm.com/api'
export const QC_URL = IS_PRODUCTION ? 'https://auth.quantum.ibm.com/api' : 'https://auth-dev.quantum.ibm.com/api'

export const isQCHttpError = (error: any): error is QCHttpError => {
return typeof error?.message === 'string'
Expand Down
3 changes: 2 additions & 1 deletion server/utilities.ts
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ import {
import { IS_PRODUCTION } from './configuration'

const TEXTBOOK_HOME = 'https://qiskit.org/learn'
const LEARNING_HOME = 'https://learning.quantum-computing.ibm.com'
const LEARNING_HOME = 'https://learning.quantum.ibm.com'

const LEARNING_REDIRECTS = {
'/course/algorithm-design': `${LEARNING_HOME}/course/variational-algorithm-design`,
Expand Down Expand Up @@ -188,6 +188,7 @@ export {
NOTATIONS,
GLOSSARY,
TEXTBOOK_HOME,
LEARNING_HOME,
LATEST_TEXTBOOK_VERSION,
TRANSLATIONS,
UNIVERSAL_NOTATIONS,
Expand Down

0 comments on commit 3892dce

Please sign in to comment.