percyliang / sempre

Semantic Parser with Execution
Other
828 stars 301 forks source link

sempre interactive crashes on startup #212

Closed HeikoBecker closed 3 years ago

HeikoBecker commented 3 years ago

Hi all, I was using sempre as part of a bigger project fine until today. Updating the dependencies with ./pull-dependencies core interactive makes the run script in the interactive directory crash. The bug is reproducible with the latest master with the following steps:

git clone https://github.com/percyliang/sempre
cd sempre
./pull-dependencies interactive
ant interactive
./interactive/run @mode=voxelurn -server -interactive

The last command is expected to show the sempre REPL, where it fails with an exception: 1 arguments required for Server.port, but got 0.

Full output:

Creating state/lastExec
Execution directory: state/execs/0.exec
java -ea -Dmodules=core,interactive -Xms8G -Xmx10G -cp libsempre/*:lib/* edu.stanford.nlp.sempre.Main --server --masterType edu.stanford.nlp.sempre.interactive.InteractiveMaster --Executor interactive.DALExecutor --LanguageAnalyzer interactive.DALAnalyzer --DALExecutor.convertNumberValues true --DALExecutor.printStackTrace true --VoxelWorld.maxBlocks 100000 --DALExecutor.worldType VoxelWorld --Grammar.inPaths ./interactive/voxelurn.grammar --Params.initWeightsRandomly false --Grammar.binarizeRules false --Grammar.useApplyFn interactive.ApplyFn --LanguageAnalyzer.lowerCaseTokens true --Parser.pruneErrorValues true --Parser interactive.InteractiveBeamParser --Parser.callSetEvaluation false --Parser.coarsePrune true --Parser.beamSize 50 --InteractiveBeamParser.maxNewTreesPerSpan 5001 --ParserState.customExpectedCounts None --InteractiveBeamParser.floatStrategy Never --InteractiveBeamParser.trackedCats Number Numbers Color Direction Set Sets Action Actions --Derivation.derivComparator AnchorPriorityScoreComparator --Params.l1Reg nonlazy --Params.l1RegCoeff 0.0001 --Params.initStepSize 0.1 --Params.adaptiveStepSize true --FeatureExtractor.featureComputers interactive.DALFeatureComputer --FeatureExtractor.featureDomains :rule :span :stats :scope :social :window --InteractiveMaster.intOutputPath ./int-output/ --InteractiveMaster.onlyInteractive true --InteractiveUtils.citationPath ./int-output/citation/ --InteractiveMaster.useAligner false --InteractiveMaster.maxSequence 20 --InteractiveMaster.maxChars 200 --DefinitionAligner.strategies ExactExclusion --InteractiveServer.numThreads 16 --InteractiveServer.maxCandidates 50 --InteractiveServer.queryLogPath ./int-output/query.log --InteractiveServer.responseLogPath ./int-output/response.log --InteractiveServer.port 8410 --GrammarInducer.useBestPacking true --GrammarInducer.useSimplePacking true --GrammarInducer.maxNonterminals 3 --Derivation.showTypes false --Derivation.showValues false --Derivation.showRules false --Derivation.anchoredBonus 1.0 --NumberFn.allowedRange 0 100 --SimpleLexicon.inPaths ./interactive/csscolors.lexicon    -server -interactive
1 arguments required for Server.port, but got 0
Command failed: fig/bin/qcreate java -ea -Dmodules=core,interactive -Xms8G -Xmx10G -cp libsempre/*:lib/* edu.stanford.nlp.sempre.Main --server --masterType edu.stanford.nlp.sempre.interactive.InteractiveMaster --Executor interactive.DALExecutor --LanguageAnalyzer interactive.DALAnalyzer --DALExecutor.convertNumberValues true --DALExecutor.printStackTrace true --VoxelWorld.maxBlocks 100000 --DALExecutor.worldType VoxelWorld --Grammar.inPaths ./interactive/voxelurn.grammar --Params.initWeightsRandomly false --Grammar.binarizeRules false --Grammar.useApplyFn interactive.ApplyFn --LanguageAnalyzer.lowerCaseTokens true --Parser.pruneErrorValues true --Parser interactive.InteractiveBeamParser --Parser.callSetEvaluation false --Parser.coarsePrune true --Parser.beamSize 50 --InteractiveBeamParser.maxNewTreesPerSpan 5001 --ParserState.customExpectedCounts None --InteractiveBeamParser.floatStrategy Never --InteractiveBeamParser.trackedCats Number Numbers Color Direction Set Sets Action Actions --Derivation.derivComparator AnchorPriorityScoreComparator --Params.l1Reg nonlazy --Params.l1RegCoeff 0.0001 --Params.initStepSize 0.1 --Params.adaptiveStepSize true --FeatureExtractor.featureComputers interactive.DALFeatureComputer --FeatureExtractor.featureDomains :rule :span :stats :scope :social :window --InteractiveMaster.intOutputPath ./int-output/ --InteractiveMaster.onlyInteractive true --InteractiveUtils.citationPath ./int-output/citation/ --InteractiveMaster.useAligner false --InteractiveMaster.maxSequence 20 --InteractiveMaster.maxChars 200 --DefinitionAligner.strategies ExactExclusion --InteractiveServer.numThreads 16 --InteractiveServer.maxCandidates 50 --InteractiveServer.queryLogPath ./int-output/query.log --InteractiveServer.responseLogPath ./int-output/response.log --InteractiveServer.port 8410 --GrammarInducer.useBestPacking true --GrammarInducer.useSimplePacking true --GrammarInducer.maxNonterminals 3 --Derivation.showTypes false --Derivation.showValues false --Derivation.showRules false --Derivation.anchoredBonus 1.0 --NumberFn.allowedRange 0 100 --SimpleLexicon.inPaths ./interactive/csscolors.lexicon    -server -interactive
ppasupat commented 3 years ago

The fig package currently breaks the run script. Please roll back as follows:

cd fig
git checkout d81d89d
ppasupat commented 3 years ago

214 should now fix this. Please pull the latest code and try again.

HeikoBecker commented 3 years ago

Thanks. That worked