Skip to content

Commit

Permalink
refactor: work on getting shell optional
Browse files Browse the repository at this point in the history
  • Loading branch information
EagleoutIce committed Oct 24, 2023
1 parent b4defac commit 1e09ca7
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,8 @@ import {
import { emptySummarizedWithProject, recordFilePath, SummarizedWithProject } from '../../post-processing'
import { array2bag } from '../../../../util/arrays'
import { SourcePosition } from '../../../../util/range'
import { SteppingSlicer, StepResult, StepResults } from '../../../../core'
import { collectAllIds, countChildren, requestFromInput, retrieveXmlFromRCode, RType } from '../../../../r-bridge'
import { SteppingSlicer, StepResults } from '../../../../core'
import { countChildren, requestFromInput, retrieveXmlFromRCode, RType } from '../../../../r-bridge'
import { locationToId } from '../../../../slicing'

interface FunctionDefinitionSummaryInformation<Measurement> {
Expand Down Expand Up @@ -231,7 +231,7 @@ async function processNextLine(data: FunctionDefinitionSummaryInformation<number
}
const [hits, context] = line

const fullPath = context ? path.join(config.sourceBasePath, context) : undefined
const fullPath = context && config.sourceBasePath ? path.join(config.sourceBasePath, context) : undefined
let result: StepResults<'normalize'> | undefined = undefined

if(fullPath) {
Expand Down
2 changes: 1 addition & 1 deletion src/util/summarizer/statistics/summarizer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ export interface StatisticsSummarizerConfiguration extends CommonSummarizerConfi
/**
* If given, the summarizer may use the given path as a base to find and re-read source files (for token lengths :c)
*/
sourceBasePath: string
sourceBasePath: string | undefined
/**
* The r-shell to use for (re-) queries
*/
Expand Down

0 comments on commit 1e09ca7

Please sign in to comment.