CRAB/python/Creator.py

from Actor import Actor
from WorkSpace import WorkSpace
from JobList import JobList
from JobDB import JobDB
from ScriptWriter import ScriptWriter
from Scheduler import Scheduler
from crab_logger import Logger
from crab_exceptions import *
from crab_util import *
import common

import os, string, math

class Creator(Actor):
    def __init__(self, job_type_name, cfg_params, ncjobs):
        self.job_type_name = job_type_name
        self.job_type = None
        self.cfg_params = cfg_params
        self.total_njobs = 0
        self.total_number_of_events = 0
        self.job_number_of_events = 0
        self.first_event = 0

        self.createJobTypeObject()
        common.logger.debug(5, __name__+": JobType "+self.job_type.name()+" created")

        self.job_type.prepareSteeringCards()
        common.logger.debug(5, __name__+": Steering cards prepared")

        self.defineTotalNumberOfJobs_()
        common.logger.debug(5, __name__+": total # of jobs = "+`self.total_njobs`)

        # Set number of jobs to be created

        self.ncjobs = ncjobs
        if ncjobs == 'all' : self.ncjobs = self.total_njobs
        if ncjobs > self.total_njobs : self.ncjobs = self.total_njobs
        
        fileCODE1 = open(common.work_space.shareDir()+"/.code","a")
        ### commented for FAMOS  
        if self.job_type.name() == 'ORCA':
            self.owner = cfg_params['ORCA.owner']
            self.dataset = cfg_params['ORCA.dataset']
            fileCODE1.write('::'+str(self.job_type.name())+'::'+str(self.ncjobs)+'::'+str(self.dataset)+'::'+str(self.owner))
            pass
        elif self.job_type.name() == 'FAMOS':
            pass
        fileCODE1.close()


        #TODO: deprecated code, not needed,
        # will be eliminated when WorkSpace.saveConfiguration()
        # will be improved.
        #
        # Set/Save Job Type name

        jt_fname = common.work_space.shareDir() + 'jobtype'
        if os.path.exists(jt_fname):
            # Read stored job type name
            jt_file = open(jt_fname, 'r')
            jt = jt_file.read()
            if self.job_type_name:
                if ( jt != self.job_type_name+'\n' ):
                    msg = 'Job Type mismatch: requested <' + self.job_type_name
                    msg += '>, found <' + jt[:-1] + '>.'
                    raise CrabException(msg)
                pass
            else:
                self.job_type_name = jt[:-1]
                pass
            jt_file.close()
            pass
        else:
            # Save job type name
            jt_file = open(jt_fname, 'w')
            jt_file.write(self.job_type_name+'\n')
            jt_file.close()
            pass
        #end of deprecated code

        common.logger.debug(5, "Creator constructor finished")
        return

    def defineTotalNumberOfJobs_(self):
        """
        Calculates the total number of jobs to be created.
        """

        try:
            self.first_event = int(self.cfg_params['USER.first_event'])
        except KeyError:
            self.first_event = 0
        common.logger.debug(1,"First event ot be analyzed: "+str(self.first_event))

        # TODO Could we find a better way to get this number?
        maxAvailableEvents = int(self.job_type.maxEvents)
        common.logger.debug(1,"Available events: "+str(maxAvailableEvents))

        # some sanity check
        if self.first_event>=maxAvailableEvents:
            raise CrabException('First event is bigger than maximum number of available events!')

        # the total number of events to be analyzed
        try:
            n = self.cfg_params['USER.total_number_of_events']
            if n == 'all': n = '-1'
            if n == '-1':
                self.total_number_of_events = (maxAvailableEvents - self.first_event)
                common.logger.debug(1,"Analysing all available events "+str(self.total_number_of_events))
            else:
                if maxAvailableEvents<(int(n)+self.first_event):
                    raise CrabException('(First event + total events)='+str(int(n)+self.first_event)+' is bigger than maximum number of available events '+str(maxAvailableEvents)+' !! Use "total_number_of_events=-1" to analyze to whole dataset')
                self.total_number_of_events = int(n)
        except KeyError:
            common.logger.message("total_number_of_events not defined, set it to maximum available")
            self.total_number_of_events = (maxAvailableEvents - self.first_event)
            pass
        common.logger.message("Total number of events to be analyzed: "+str(self.total_number_of_events))


        # read user directives
        eventPerJob=0
        try:
            eventPerJob = self.cfg_params['USER.job_number_of_events']
        except KeyError:
            pass
        
        jobsPerTask=0
        try:
            jobsPerTask = int(self.cfg_params['USER.total_number_of_jobs'])
        except KeyError:
            pass

        # If both the above set, complain and use event per jobs
        if eventPerJob>0 and jobsPerTask>0:
            msg = 'Warning. '
            msg += 'job_number_of_events and total_number_of_jobs are both defined '
            msg += 'Using job_number_of_events.'
            common.logger.message(msg)
            jobsPerTask = 0 
        if eventPerJob==0 and jobsPerTask==0:
            msg = 'Warning. '
            msg += 'job_number_of_events and total_number_of_jobs are not defined '
            msg += 'Creating just one job for all events.'
            common.logger.message(msg)
            jobsPerTask = 1 

        # first case: events per job defined
        if eventPerJob>0:
            n=eventPerJob
            if n == 'all' or n == '-1' or (int(n)>self.total_number_of_events and self.total_number_of_events>0):
                common.logger.message("Asking more events than available: set it to maximum available")
                self.job_number_of_events = self.total_number_of_events
                self.total_njobs = 1
            else:
                self.job_number_of_events = int(n)
                self.total_njobs = int((self.total_number_of_events-1)/self.job_number_of_events)+1
        # second case: jobs per task defined
        elif jobsPerTask>0:
            common.logger.debug(2,"total number of events: "+str(self.total_number_of_events)+" JobPerTask "+str(jobsPerTask))
            self.job_number_of_events = int(math.floor((self.total_number_of_events)/jobsPerTask))
            self.total_njobs = jobsPerTask
        # should not happen...
        else:
            raise CrabException('Somthing wrong with splitting')

        common.logger.debug(2,"total number of events: "+str(self.total_number_of_events)+
            " events per job: "+str(self.job_number_of_events))

        return

    def writeJobsSpecsToDB(self):
        """
        Write firstEvent and maxEvents in the DB for future use
        """

        common.jobDB.load()
        # case one: write first and max events
        nJobs=self.nJobs()

        firstEvent=self.first_event
        ### fede for famos
        lastJobsNumberOfEvents = self.job_number_of_events
        print "lastjobsnumberofevents", lastJobsNumberOfEvents
        ###
        # last jobs is different...
        for job in range(nJobs-1):
            common.jobDB.setFirstEvent(job, firstEvent)
            common.jobDB.setMaxEvents(job, self.job_number_of_events)
        ### fede checks parameter for famos and orca
            try:
                self.events_management = common.analisys_common_info['events_management']
                #print "self.events_management = ", self.events_management
            except KeyError: self.events_management = 0
            if int(self.events_management) == 1:
                firstEvent=firstEvent+self.job_number_of_events
                #print "firstEvent", firstEvent
            ###

###            firstEvent=firstEvent+self.job_number_of_events

        # this is the last job
        common.jobDB.setFirstEvent(nJobs-1, firstEvent)
        #### FEDE
        #print "firstEvent", firstEvent
        if int(self.events_management) == 1:
            lastJobsNumberOfEvents= (self.total_number_of_events+self.first_event)-firstEvent
        ####

###        lastJobsNumberOfEvents= (self.total_number_of_events+self.first_event)-firstEvent
        #print 'lastJobsNumberOfEvents :', lastJobsNumberOfEvents
        common.jobDB.setMaxEvents(nJobs-1, lastJobsNumberOfEvents)
    
        if (lastJobsNumberOfEvents!=self.job_number_of_events):
            common.logger.message(str(self.total_njobs-1)+' jobs will be created for '+str(self.job_number_of_events)+' events each plus 1 for '+str(lastJobsNumberOfEvents)+' events for a total of '+str(self.job_number_of_events*(self.total_njobs-1)+lastJobsNumberOfEvents)+' events')
        else:
            common.logger.message(str(self.total_njobs)+' jobs will be created for '+str(self.job_number_of_events)+' events each for a total of '+str(self.job_number_of_events*(self.total_njobs-1)+lastJobsNumberOfEvents)+' events')

        # case two (to be implemented) write eventCollections for each jobs

        # save the DB
        common.jobDB.save()
        return

    def nJobs(self):
        return self.total_njobs
    
    def createJobTypeObject(self):
        file_name = 'cms_'+ string.lower(self.job_type_name)
        klass_name = string.capitalize(self.job_type_name)

        try:
            klass = importName(file_name, klass_name)
        except KeyError:
            msg = 'No `class '+klass_name+'` found in file `'+file_name+'.py`'
            raise CrabException(msg)
        except ImportError, e:
            msg = 'Cannot create job type '+self.job_type_name
            msg += ' (file: '+file_name+', class '+klass_name+'):\n'
            msg += str(e)
            raise CrabException(msg)

        self.job_type = klass(self.cfg_params)
        return

    def jobType(self):
        return self.job_type

    def run(self):
        """
        The main method of the class.
        """

        common.logger.debug(5, "Creator::run() called")

        # Instantiate ScriptWriter

        script_writer = ScriptWriter('crab_template.sh')

        # Loop over jobs

        njc = 0
        for nj in range(self.total_njobs):
            if njc == self.ncjobs : break
            st = common.jobDB.status(nj)
            if st != 'X': continue

            common.logger.message("Creating job # "+`(nj+1)`)

            # Prepare configuration file

            self.job_type.modifySteeringCards(nj)
            # Create script (sh)
            #print "nel for prima del modifyTemplateScript, nj vale", nj
            script_writer.modifyTemplateScript(nj)
            os.chmod(common.job_list[nj].scriptFilename(), 0744)

            # Create scheduler scripts (jdl)
            common.scheduler.createSchScript(nj)

            common.jobDB.setStatus(nj, 'C')
            # common: write input and output sandbox
            common.jobDB.setInputSandbox(nj, self.job_type.inputSandbox(nj))

            outputSandbox=self.job_type.outputSandbox(nj)
            stdout=common.job_list[nj].stdout()
            stderr=common.job_list[nj].stderr()
            outputSandbox.append(common.job_list[nj].stdout())
            # check if out!=err
            if stdout != stderr:
                outputSandbox.append(common.job_list[nj].stderr())
            common.jobDB.setOutputSandbox(nj, outputSandbox)
            common.jobDB.setTaskId(nj, self.cfg_params['taskId'])

            njc = njc + 1
            pass

        ####

        common.jobDB.save()

        msg = '\nTotal of %d jobs created'%njc
        if njc != self.ncjobs: msg = msg + ' from %d requested'%self.ncjobs
        msg = msg + '.\n'
        common.logger.message(msg)
        return
Revision:	1.21
Committed:	Wed Mar 8 16:50:21 2006 UTC (19 years, 1 month ago) by fanzago
Content type:	text/x-python
Branch:	MAIN
Changes since 1.20:	+22 -4 lines
Log Message:	changes for famos
#	User	Rev	Content
1	nsmirnov	1.1	from Actor import Actor
2			from WorkSpace import WorkSpace
3			from JobList import JobList
4			from JobDB import JobDB
5			from ScriptWriter import ScriptWriter
6			from Scheduler import Scheduler
7			from crab_logger import Logger
8			from crab_exceptions import *
9			from crab_util import *
10			import common
11
12	slacapra	1.8	import os, string, math
13	nsmirnov	1.1
14			class Creator(Actor):
15			def __init__(self, job_type_name, cfg_params, ncjobs):
16			self.job_type_name = job_type_name
17			self.job_type = None
18			self.cfg_params = cfg_params
19	nsmirnov	1.2	self.total_njobs = 0
20			self.total_number_of_events = 0
21			self.job_number_of_events = 0
22	slacapra	1.7	self.first_event = 0
23	slacapra	1.14
24	nsmirnov	1.1	self.createJobTypeObject()
25	nsmirnov	1.3	common.logger.debug(5, __name__+": JobType "+self.job_type.name()+" created")
26	nsmirnov	1.1
27			self.job_type.prepareSteeringCards()
28	nsmirnov	1.3	common.logger.debug(5, __name__+": Steering cards prepared")
29	nsmirnov	1.1
30	nsmirnov	1.2	self.defineTotalNumberOfJobs_()
31	nsmirnov	1.3	common.logger.debug(5, __name__+": total # of jobs = "+`self.total_njobs`)
32	nsmirnov	1.1
33			# Set number of jobs to be created
34
35			self.ncjobs = ncjobs
36			if ncjobs == 'all' : self.ncjobs = self.total_njobs
37			if ncjobs > self.total_njobs : self.ncjobs = self.total_njobs
38	spiga	1.11
39	spiga	1.20	fileCODE1 = open(common.work_space.shareDir()+"/.code","a")
40			### commented for FAMOS
41			if self.job_type.name() == 'ORCA':
42			self.owner = cfg_params['ORCA.owner']
43			self.dataset = cfg_params['ORCA.dataset']
44			fileCODE1.write('::'+str(self.job_type.name())+'::'+str(self.ncjobs)+'::'+str(self.dataset)+'::'+str(self.owner))
45			pass
46			elif self.job_type.name() == 'FAMOS':
47			pass
48	spiga	1.11	fileCODE1.close()
49
50	nsmirnov	1.1
51			#TODO: deprecated code, not needed,
52			# will be eliminated when WorkSpace.saveConfiguration()
53			# will be improved.
54			#
55			# Set/Save Job Type name
56
57			jt_fname = common.work_space.shareDir() + 'jobtype'
58			if os.path.exists(jt_fname):
59			# Read stored job type name
60			jt_file = open(jt_fname, 'r')
61			jt = jt_file.read()
62			if self.job_type_name:
63			if ( jt != self.job_type_name+'\n' ):
64			msg = 'Job Type mismatch: requested <' + self.job_type_name
65			msg += '>, found <' + jt[:-1] + '>.'
66			raise CrabException(msg)
67			pass
68			else:
69			self.job_type_name = jt[:-1]
70			pass
71			jt_file.close()
72			pass
73			else:
74			# Save job type name
75			jt_file = open(jt_fname, 'w')
76			jt_file.write(self.job_type_name+'\n')
77			jt_file.close()
78			pass
79			#end of deprecated code
80
81	nsmirnov	1.3	common.logger.debug(5, "Creator constructor finished")
82	nsmirnov	1.1	return
83
84	nsmirnov	1.2	def defineTotalNumberOfJobs_(self):
85			"""
86			Calculates the total number of jobs to be created.
87			"""
88
89			try:
90	slacapra	1.8	self.first_event = int(self.cfg_params['USER.first_event'])
91	slacapra	1.7	except KeyError:
92			self.first_event = 0
93	slacapra	1.8	common.logger.debug(1,"First event ot be analyzed: "+str(self.first_event))
94	slacapra	1.7
95	slacapra	1.10	# TODO Could we find a better way to get this number?
96	slacapra	1.8	maxAvailableEvents = int(self.job_type.maxEvents)
97			common.logger.debug(1,"Available events: "+str(maxAvailableEvents))
98	slacapra	1.7
99	slacapra	1.8	# some sanity check
100			if self.first_event>=maxAvailableEvents:
101	slacapra	1.7	raise CrabException('First event is bigger than maximum number of available events!')
102
103	slacapra	1.8	# the total number of events to be analyzed
104	slacapra	1.7	try:
105	nsmirnov	1.2	n = self.cfg_params['USER.total_number_of_events']
106			if n == 'all': n = '-1'
107	slacapra	1.8	if n == '-1':
108			self.total_number_of_events = (maxAvailableEvents - self.first_event)
109			common.logger.debug(1,"Analysing all available events "+str(self.total_number_of_events))
110			else:
111			if maxAvailableEvents<(int(n)+self.first_event):
112	slacapra	1.17	raise CrabException('(First event + total events)='+str(int(n)+self.first_event)+' is bigger than maximum number of available events '+str(maxAvailableEvents)+' !! Use "total_number_of_events=-1" to analyze to whole dataset')
113	slacapra	1.8	self.total_number_of_events = int(n)
114	slacapra	1.7	except KeyError:
115			common.logger.message("total_number_of_events not defined, set it to maximum available")
116	slacapra	1.8	self.total_number_of_events = (maxAvailableEvents - self.first_event)
117	slacapra	1.7	pass
118	slacapra	1.8	common.logger.message("Total number of events to be analyzed: "+str(self.total_number_of_events))
119
120	slacapra	1.7
121	slacapra	1.8	# read user directives
122	slacapra	1.7	eventPerJob=0
123			try:
124			eventPerJob = self.cfg_params['USER.job_number_of_events']
125	nsmirnov	1.2	except KeyError:
126			pass
127
128	slacapra	1.7	jobsPerTask=0
129	nsmirnov	1.2	try:
130	slacapra	1.7	jobsPerTask = int(self.cfg_params['USER.total_number_of_jobs'])
131	nsmirnov	1.2	except KeyError:
132	slacapra	1.7	pass
133
134			# If both the above set, complain and use event per jobs
135			if eventPerJob>0 and jobsPerTask>0:
136	nsmirnov	1.2	msg = 'Warning. '
137	slacapra	1.7	msg += 'job_number_of_events and total_number_of_jobs are both defined '
138			msg += 'Using job_number_of_events.'
139	nsmirnov	1.2	common.logger.message(msg)
140	slacapra	1.7	jobsPerTask = 0
141			if eventPerJob==0 and jobsPerTask==0:
142			msg = 'Warning. '
143			msg += 'job_number_of_events and total_number_of_jobs are not defined '
144			msg += 'Creating just one job for all events.'
145			common.logger.message(msg)
146			jobsPerTask = 1
147
148			# first case: events per job defined
149			if eventPerJob>0:
150			n=eventPerJob
151			if n == 'all' or n == '-1' or (int(n)>self.total_number_of_events and self.total_number_of_events>0):
152			common.logger.message("Asking more events than available: set it to maximum available")
153			self.job_number_of_events = self.total_number_of_events
154	slacapra	1.8	self.total_njobs = 1
155	slacapra	1.7	else:
156			self.job_number_of_events = int(n)
157	slacapra	1.8	self.total_njobs = int((self.total_number_of_events-1)/self.job_number_of_events)+1
158	slacapra	1.7	# second case: jobs per task defined
159			elif jobsPerTask>0:
160	slacapra	1.8	common.logger.debug(2,"total number of events: "+str(self.total_number_of_events)+" JobPerTask "+str(jobsPerTask))
161			self.job_number_of_events = int(math.floor((self.total_number_of_events)/jobsPerTask))
162			self.total_njobs = jobsPerTask
163	slacapra	1.7	# should not happen...
164			else:
165			raise CrabException('Somthing wrong with splitting')
166
167	slacapra	1.8	common.logger.debug(2,"total number of events: "+str(self.total_number_of_events)+
168	slacapra	1.7	" events per job: "+str(self.job_number_of_events))
169	slacapra	1.8
170	slacapra	1.7	return
171
172			def writeJobsSpecsToDB(self):
173			"""
174			Write firstEvent and maxEvents in the DB for future use
175			"""
176
177			common.jobDB.load()
178			# case one: write first and max events
179			nJobs=self.nJobs()
180
181			firstEvent=self.first_event
182	fanzago	1.21	### fede for famos
183			lastJobsNumberOfEvents = self.job_number_of_events
184			print "lastjobsnumberofevents", lastJobsNumberOfEvents
185			###
186	slacapra	1.7	# last jobs is different...
187			for job in range(nJobs-1):
188			common.jobDB.setFirstEvent(job, firstEvent)
189			common.jobDB.setMaxEvents(job, self.job_number_of_events)
190	fanzago	1.21	### fede checks parameter for famos and orca
191			try:
192			self.events_management = common.analisys_common_info['events_management']
193			#print "self.events_management = ", self.events_management
194			except KeyError: self.events_management = 0
195			if int(self.events_management) == 1:
196			firstEvent=firstEvent+self.job_number_of_events
197			#print "firstEvent", firstEvent
198			###
199
200			### firstEvent=firstEvent+self.job_number_of_events
201	slacapra	1.7
202			# this is the last job
203			common.jobDB.setFirstEvent(nJobs-1, firstEvent)
204	fanzago	1.21	#### FEDE
205			#print "firstEvent", firstEvent
206			if int(self.events_management) == 1:
207			lastJobsNumberOfEvents= (self.total_number_of_events+self.first_event)-firstEvent
208			####
209
210			### lastJobsNumberOfEvents= (self.total_number_of_events+self.first_event)-firstEvent
211	fanzago	1.19	#print 'lastJobsNumberOfEvents :', lastJobsNumberOfEvents
212	slacapra	1.8	common.jobDB.setMaxEvents(nJobs-1, lastJobsNumberOfEvents)
213	slacapra	1.7
214	slacapra	1.15	if (lastJobsNumberOfEvents!=self.job_number_of_events):
215			common.logger.message(str(self.total_njobs-1)+' jobs will be created for '+str(self.job_number_of_events)+' events each plus 1 for '+str(lastJobsNumberOfEvents)+' events for a total of '+str(self.job_number_of_events*(self.total_njobs-1)+lastJobsNumberOfEvents)+' events')
216			else:
217			common.logger.message(str(self.total_njobs)+' jobs will be created for '+str(self.job_number_of_events)+' events each for a total of '+str(self.job_number_of_events*(self.total_njobs-1)+lastJobsNumberOfEvents)+' events')
218	slacapra	1.7
219			# case two (to be implemented) write eventCollections for each jobs
220	nsmirnov	1.2
221	slacapra	1.9	# save the DB
222			common.jobDB.save()
223	nsmirnov	1.2	return
224
225	nsmirnov	1.1	def nJobs(self):
226			return self.total_njobs
227
228			def createJobTypeObject(self):
229			file_name = 'cms_'+ string.lower(self.job_type_name)
230			klass_name = string.capitalize(self.job_type_name)
231
232			try:
233			klass = importName(file_name, klass_name)
234			except KeyError:
235			msg = 'No `class '+klass_name+'` found in file `'+file_name+'.py`'
236			raise CrabException(msg)
237			except ImportError, e:
238			msg = 'Cannot create job type '+self.job_type_name
239			msg += ' (file: '+file_name+', class '+klass_name+'):\n'
240			msg += str(e)
241			raise CrabException(msg)
242
243			self.job_type = klass(self.cfg_params)
244			return
245	nsmirnov	1.5
246			def jobType(self):
247			return self.job_type
248	nsmirnov	1.1
249			def run(self):
250	nsmirnov	1.4	"""
251			The main method of the class.
252			"""
253	nsmirnov	1.1
254	nsmirnov	1.3	common.logger.debug(5, "Creator::run() called")
255
256	nsmirnov	1.1	# Instantiate ScriptWriter
257
258			script_writer = ScriptWriter('crab_template.sh')
259
260			# Loop over jobs
261
262			njc = 0
263	nsmirnov	1.4	for nj in range(self.total_njobs):
264	nsmirnov	1.1	if njc == self.ncjobs : break
265	nsmirnov	1.4	st = common.jobDB.status(nj)
266	nsmirnov	1.1	if st != 'X': continue
267
268	nsmirnov	1.6	common.logger.message("Creating job # "+`(nj+1)`)
269	nsmirnov	1.3
270	nsmirnov	1.1	# Prepare configuration file
271
272	nsmirnov	1.4	self.job_type.modifySteeringCards(nj)
273	fanzago	1.12	# Create script (sh)
274	fanzago	1.13	#print "nel for prima del modifyTemplateScript, nj vale", nj
275	nsmirnov	1.4	script_writer.modifyTemplateScript(nj)
276			os.chmod(common.job_list[nj].scriptFilename(), 0744)
277	nsmirnov	1.1
278	fanzago	1.12	# Create scheduler scripts (jdl)
279			common.scheduler.createSchScript(nj)
280
281	nsmirnov	1.4	common.jobDB.setStatus(nj, 'C')
282	slacapra	1.9	# common: write input and output sandbox
283			common.jobDB.setInputSandbox(nj, self.job_type.inputSandbox(nj))
284
285			outputSandbox=self.job_type.outputSandbox(nj)
286			stdout=common.job_list[nj].stdout()
287			stderr=common.job_list[nj].stderr()
288			outputSandbox.append(common.job_list[nj].stdout())
289			# check if out!=err
290			if stdout != stderr:
291			outputSandbox.append(common.job_list[nj].stderr())
292			common.jobDB.setOutputSandbox(nj, outputSandbox)
293	corvo	1.18	common.jobDB.setTaskId(nj, self.cfg_params['taskId'])
294	slacapra	1.9
295	nsmirnov	1.1	njc = njc + 1
296			pass
297
298			####
299	fanzago	1.12
300	nsmirnov	1.1	common.jobDB.save()
301
302			msg = '\nTotal of %d jobs created'%njc
303			if njc != self.ncjobs: msg = msg + ' from %d requested'%self.ncjobs
304			msg = msg + '.\n'
305			common.logger.message(msg)
306			return