1436

pending completion

Build # 1436

Build Type

Pull #378

travis-ci-com

Committed by

web-flow

Commit Message

Merge ed4924a00 into cf078a09c

Pull Request Pull Request #378: Wip/wikidata single turn

Run Details

7230 of 13591 branches covered (53.2%)

Branch coverage included in aggregate %.

290 of 627 new or added lines in 14 files covered. (46.25%)

20 existing lines in 5 files now uncovered.

16111 of 25775 relevant lines covered (62.51%)

1711115.32 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

53.49

/languages/thingtalk/dialogue_acts/coref-actions.js

// -*- mode: js; indent-tabs-mode: nil; js-basic-offset: 4 -*-
//
// This file is part of Genie
//
// Copyright 2020 The Board of Trustees of the Leland Stanford Junior University
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
//    http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
//
// Author: Giovanni Campagna <gcampagn@cs.stanford.edu>
"use strict";

const assert = require('assert');

const ThingTalk = require('thingtalk');
const Ast = ThingTalk.Ast;

const C = require('../ast_manip');

function contextualAction(ctx, action) {
    assert(action instanceof Ast.Invocation);
    const ctxInvocation = C.getInvocation(ctx.next);
    if (!ctxInvocation || ctxInvocation.selector.isBuiltin)
        return null;
    if (!C.isSameFunction(ctxInvocation.schema, action.schema))
        return null;
    if (action.in_params.length === 0) // common case, no new parameters
        return ctxInvocation;

    for (let newParam of action.in_params) {
        if (newParam.value.isUndefined)
            continue;

        // check that we don't change a previous parameter

        // if we're introducing a value for the chain parameter that was not previously provided,
        // it must one of the top 3 results
        if (newParam.name === ctx.nextInfo.chainParameter &&
           !ctx.nextInfo.chainParameterFilled) {
            if (!ctx.results)
                return null;
            const results = ctx.results;
            let good = false;
            for (let i = 0; i < Math.min(results.length, 3); i ++) {
                const id = results[i].value.id;
                if (id && id.equals(newParam.value)) {
                    good = true;
                    break;
                }
            }
            if (!good)
                return null;
        }

        for (let oldParam of ctxInvocation.in_params) {
            if (oldParam.value.isUndefined)
                continue;
            if (newParam.name !== oldParam.name)
                continue;

            if (!newParam.value.equals(oldParam.value))
                return null;
        }
    }

    // semi-shallow clone
    const clone = new Ast.Invocation(null, ctxInvocation.selector, ctxInvocation.channel,
        ctxInvocation.in_params.map((ip) => new Ast.InputParam(null, ip.name, ip.value)),
        ctxInvocation.schema);
    for (let newParam of action.in_params) {
        if (newParam.value.isUndefined)
            continue;

        let found = false;
        for (let oldParam of clone.in_params) {
            if (newParam.name === oldParam.name) {
                if (oldParam.value.isUndefined)
                    oldParam.value = newParam.value;
                found = true;
                break;
            }
        }

        if (!found)
            clone.in_params.push(newParam);
    }

    return clone;
}

module.exports = {
    contextualAction
};

1	// -- mode: js; indent-tabs-mode: nil; js-basic-offset: 4 --
2	//
3	// This file is part of Genie
4	//
5	// Copyright 2020 The Board of Trustees of the Leland Stanford Junior University
6	//
7	// Licensed under the Apache License, Version 2.0 (the "License");
8	// you may not use this file except in compliance with the License.
9	// You may obtain a copy of the License at
10	//
11	// http://www.apache.org/licenses/LICENSE-2.0
12	//
13	// Unless required by applicable law or agreed to in writing, software
14	// distributed under the License is distributed on an "AS IS" BASIS,
15	// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
16	// See the License for the specific language governing permissions and
17	// limitations under the License.
18	//
19	// Author: Giovanni Campagna <gcampagn@cs.stanford.edu>
20	"use strict";
21
22	const assert = require('assert');	3✔
23
24	const ThingTalk = require('thingtalk');	3✔
25	const Ast = ThingTalk.Ast;	3✔
26
27	const C = require('../ast_manip');	3✔
28
29	function contextualAction(ctx, action) {
30	assert(action instanceof Ast.Invocation);	17,898✔
31	const ctxInvocation = C.getInvocation(ctx.next);	17,898✔
32	if (!ctxInvocation \|\| ctxInvocation.selector.isBuiltin)	17,898!
33	return null;	×
34	if (!C.isSameFunction(ctxInvocation.schema, action.schema))	17,898✔
35	return null;	15,928✔
36	if (action.in_params.length === 0) // common case, no new parameters	1,970!
37	return ctxInvocation;	×
38
39	for (let newParam of action.in_params) {	1,970✔
40	if (newParam.value.isUndefined)	7,695✔
41	continue;	7,650✔
42
43	// check that we don't change a previous parameter
44
45	// if we're introducing a value for the chain parameter that was not previously provided,
46	// it must one of the top 3 results
47	if (newParam.name === ctx.nextInfo.chainParameter &&	45!
48	!ctx.nextInfo.chainParameterFilled) {
49	if (!ctx.results)	45!
50	return null;	×
51	const results = ctx.results;	45✔
52	let good = false;	45✔
53	for (let i = 0; i < Math.min(results.length, 3); i ++) {	45✔
54	const id = results[i].value.id;	135✔
55	if (id && id.equals(newParam.value)) {	135!
56	good = true;	×
57	break;	×
58	}
59	}
60	if (!good)	45!
61	return null;	45✔
62	}
63
UNCOV 64	for (let oldParam of ctxInvocation.in_params) {	×
UNCOV 65	if (oldParam.value.isUndefined)	×
UNCOV 66	continue;	×
67	if (newParam.name !== oldParam.name)	×
68	continue;	×
69
70	if (!newParam.value.equals(oldParam.value))	×
71	return null;	×
72	}
73	}
74
75	// semi-shallow clone
76	const clone = new Ast.Invocation(null, ctxInvocation.selector, ctxInvocation.channel,	1,925✔
77	ctxInvocation.in_params.map((ip) => new Ast.InputParam(null, ip.name, ip.value)),	7,655✔
78	ctxInvocation.schema);
79	for (let newParam of action.in_params) {	1,925✔
80	if (newParam.value.isUndefined)	7,650!
81	continue;	7,650✔
82
UNCOV 83	let found = false;	×
UNCOV 84	for (let oldParam of clone.in_params) {	×
UNCOV 85	if (newParam.name === oldParam.name) {	×
UNCOV 86	if (oldParam.value.isUndefined)	×
UNCOV 87	oldParam.value = newParam.value;	×
UNCOV 88	found = true;	×
UNCOV 89	break;	×
90	}
91	}
92
UNCOV 93	if (!found)	×
94	clone.in_params.push(newParam);	×
95	}
96
97	return clone;	1,925✔
98	}
99
100	module.exports = {	3✔
101	contextualAction
102	};

stanford-oval / genie-toolkit / 1436

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous