Skip to content

Commit 108b9b0

Browse files
committed
Refactored names of methods and variables
1 parent e7d4343 commit 108b9b0

6 files changed

Lines changed: 24 additions & 25 deletions

File tree

migration/requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,4 +11,4 @@ wikirender>=0.0.23
1111
# https://github.com/somnathrakshit/geograpy3
1212
geograpy3>=0.1.29
1313
# https://github.com/SmartDataAnalytics/OpenResearch
14-
ConferenceCorpus>=0.0.4
14+
ConferenceCorpus>=0.0.7

migration/tests/corpusfortesting.py

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,15 +3,14 @@
33
44
@author: wf
55
'''
6-
from corpus.eventcorpus import EventDataSource
76
from corpus.lookup import CorpusLookup
8-
from datasources.openresearch import OREventCorpus, OREventManager
97
from lodstorage.storageconfig import StorageConfig
10-
118
from ormigrate.toolbox import HelperFunctions as hf
129
from wikifile.wikiFileManager import WikiFileManager
1310
from os import path
1411
import os
12+
13+
1514
class CorpusForTesting(object):
1615
'''
1716
Simplify initializing an EventCorpus for tests (singleton)
@@ -48,7 +47,7 @@ def getWikiFileManager(cls,wikiId=None,debug=False):
4847
return wikiFileManager
4948

5049
@classmethod
51-
def getEventCorpusFromWikiAPI(cls, lookupId:str="orclone", forceUpdate:bool=False, debug:bool=False):
50+
def getEventDataSourceFromWikiAPI(cls, lookupId:str= "orclone", forceUpdate:bool=False, debug:bool=False):
5251
'''
5352
get events with series by knitting / linking the entities together
5453
@@ -64,7 +63,7 @@ def getEventCorpusFromWikiAPI(cls, lookupId:str="orclone", forceUpdate:bool=Fals
6463
return eventDataSource
6564

6665
@classmethod
67-
def getEventCorpusFromWikiText(cls, lookupId:str="orclone-backup", forceUpdate:bool=False, debug=False):
66+
def getEventDataSourceFromWikiText(cls, lookupId:str= "orclone-backup", forceUpdate:bool=False, debug=False):
6867
"""
6968
get events with series by knitting/linking entities from a WikiFileManager
7069

migration/tests/testIssue168_ratingCallBack.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ def testPageRatingList(self):
6666
# test the rating call back
6767
# '''
6868
# profile=Profiler("test rating call back")
69-
# eventCorpus=Corpus.getEventCorpusFromWikiAPI(debug=self.debug)
69+
# eventCorpus=Corpus.getEventDataSourceFromWikiAPI(debug=self.debug)
7070
# lod,errors=eventCorpus.eventList.getRatedLod(Event.rateMigration)
7171
# self.checkRatedLod(lod, errors)
7272
# lod,errors=eventCorpus.eventSeriesList.getRatedLod(EventSeries.rateMigration)

migration/tests/testIssue170_Curation.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,8 +47,8 @@ def testCurationQualityCheck170(self):
4747

4848
def testUserCount(self):
4949
# only needed to setup userrating yaml file
50-
eventCorpus=Corpus.getEventCorpusFromWikiAPI(debug=self.debug, force=True)
51-
userLookup=eventCorpus.eventManager.getLookup("lastEditor",withDuplicates=True)
50+
eventDataSource=Corpus.getEventDataSourceFromWikiAPI(debug=self.debug, force=True)
51+
userLookup=eventDataSource.eventManager.getLookup("lastEditor",withDuplicates=True)
5252
if self.debug:
5353
print (f"{len(userLookup)} users")
5454
expected=1 if hf.inPublicCI() else 140

migration/tests/test_EventCorpus.py

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,8 @@ class TestEventCorpus(unittest.TestCase):
5252
def setUp(self):
5353
self.debug = False
5454
self.profile=True
55-
self.eventCorpusAPI=None
56-
self.eventCorpusWikiText=None
55+
self.eventDataSourceAPI=None
56+
self.eventDataSourceWikiText=None
5757
pass
5858

5959
def tearDown(self):
@@ -85,9 +85,9 @@ def testEventCorpusFromWikiUser(self):
8585
'''
8686
self.debug=True
8787
profile=Profiler("getting EventCorpus from WikiUser")
88-
eventCorpus=Corpus.getEventCorpusFromWikiAPI(forceUpdate=True, debug=self.debug)
88+
eventDataSource=Corpus.getEventDataSourceFromWikiAPI(forceUpdate=True, debug=self.debug)
8989
profile.time()
90-
self.checkEventCorpus(eventCorpus)
90+
self.checkEventCorpus(eventDataSource)
9191

9292

9393
def testEventCorpusFromWikiUserCache(self):
@@ -97,19 +97,19 @@ def testEventCorpusFromWikiUserCache(self):
9797
debug = True
9898
if Corpus.hasCache():
9999
profile=Profiler(f"getting EventCorpus for {Corpus.wikiId} from WikiUser Cache",self.profile)
100-
self.eventCorpusAPI=Corpus.getEventCorpusFromWikiAPI(debug=debug, forceUpdate=False)
100+
self.eventDataSourceAPI=Corpus.getEventDataSourceFromWikiAPI(debug=debug, forceUpdate=False)
101101
profile.time()
102-
self.checkEventCorpus(self.eventCorpusAPI)
102+
self.checkEventCorpus(self.eventDataSourceAPI)
103103

104104

105105
def testEventCorpusFromWikiFileManager(self):
106106
"""
107107
test the Event Corpus from the wiki file manager(wikiFiles).
108108
"""
109109
profile=Profiler(f"getting EventCorpus from wikiText files for {Corpus.wikiId}")
110-
self.eventCorpusWikiText = Corpus.getEventCorpusFromWikiText(forceUpdate=True, debug=self.debug)
110+
self.eventDataSourceWikiText = Corpus.getEventDataSourceFromWikiText(forceUpdate=True, debug=self.debug)
111111
profile.time()
112-
self.checkEventCorpus(self.eventCorpusWikiText,['pageTitle'])
112+
self.checkEventCorpus(self.eventDataSourceWikiText, ['pageTitle'])
113113

114114

115115
def testMatchingSetsForEventCorpus(self):
@@ -119,15 +119,15 @@ def testMatchingSetsForEventCorpus(self):
119119
if not Corpus.hasCache():
120120
return
121121
profile=Profiler(f"getting EventCorpora from wikiAPI and wikiText files for {Corpus.wikiId}")
122-
if self.eventCorpusAPI is None:
123-
self.eventCorpusAPI=Corpus.getEventCorpusFromWikiAPI(debug=self.debug, forceUpdate=False)
124-
if self.eventCorpusWikiText is None:
125-
self.eventCorpusWikiText = Corpus.getEventCorpusFromWikiText(debug=self.debug)
122+
if self.eventDataSourceAPI is None:
123+
self.eventDataSourceAPI=Corpus.getEventDataSourceFromWikiAPI(debug=self.debug, forceUpdate=False)
124+
if self.eventDataSourceWikiText is None:
125+
self.eventDataSourceWikiText = Corpus.getEventDataSourceFromWikiText(debug=self.debug)
126126
profile.time()
127127
profile=Profiler(f"finding common events and series for {Corpus.wikiId}")
128128
keys=["acronym","pageTitle"]
129-
eventSet=MatchingSet("Events","api",self.eventCorpusAPI.eventManager,"wikiText",self.eventCorpusWikiText.eventManager,keys)
130-
eventSeriesSet=MatchingSet("EventSeries","api",self.eventCorpusAPI.eventSeriesManager,"wikiText",self.eventCorpusWikiText.eventSeriesManager,keys)
129+
eventSet=MatchingSet("Events","api", self.eventDataSourceAPI.eventManager, "wikiText", self.eventDataSourceWikiText.eventManager, keys)
130+
eventSeriesSet=MatchingSet("EventSeries","api", self.eventDataSourceAPI.eventSeriesManager, "wikiText", self.eventDataSourceWikiText.eventSeriesManager, keys)
131131
profile.time()
132132
eventSet.showStats()
133133
eventSeriesSet.showStats()

migration/tests/test_EventLocationContext.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ def setUp(self) -> None:
1717
targetWikiTextPath = f"{home}/.or/generated/Location"
1818
self.wikiFileManager = CorpusForTesting.getWikiFileManager()
1919
self.wikiFileManager.targetPath=targetWikiTextPath
20-
self.eventCorpus=CorpusForTesting.getEventCorpusFromWikiText()
20+
self.eventDataSource=CorpusForTesting.getEventDataSourceFromWikiText()
2121
self.eventLocationContext=EventLocationHandler(wikiFileManager=self.wikiFileManager)
2222

2323
def test_generateLocationPages(self):
@@ -37,7 +37,7 @@ def test_generateORLocationPages(self):
3737
Note: Not running in CI since it generates a lot of pages and uses functionalities which are test in the other tests
3838
"""
3939
profile = Profiler("Generate OPENRESEARCH location pages (Limited to 100 events)", self.profile)
40-
self.eventLocationContext.generateORLocationPages(self.eventCorpus.eventManager.events[:100], overwrite=True)
40+
self.eventLocationContext.generateORLocationPages(self.eventDataSource.eventManager.events[:100], overwrite=True)
4141
profile.time()
4242
# ToDo: test if generated correctly
4343

0 commit comments

Comments
 (0)