multi waypoint not working

Hello,
Whenever i try to launch the multi_waypoint_vicon demo, i keep having this issue
Log: 42
[ INFO] [1449255005.156678767]: Ready...
[ INFO] [1449255009.122393262]: Takeoff requested!
terminate called after throwing an instance of 'tf2::InvalidArgumentException'
what():  Quaternion malformed, magnitude: 0 should be 1.0
[crazyflie1/controller-6] process has died [pid 26478, exit code -6, cmd /home/maverick/indigo_ws/devel/lib/crazyflie_controller/crazyflie_controller __name:=controller __log:=/home/maverick/.ros/log/d329d03a-9ab7-11e5-b21a-0025647bb69f/crazyflie1-controller-6.log].
log file: /home/maverick/.ros/log/d329d03a-9ab7-11e5-b21a-0025647bb69f/crazyflie1-controller-6*.log
i changed the launch file to make it run for just one crazyflie. Any help
Thanks

I just pushed a change which should fix your issue. Please verify and let me know!

Hello,
I don't get that error and the program runs. But for some reason it just starts to fly and in 2-3 seconds it stops and doesn't continue. The 2 crazyflies are not able to take off more than 1-2 inches above the ground. I don't get any errors. Im not sure whats wrong.
Any idea ? I really am in a bind and would appreciate any help.
Thank you,

Does it work with a single Crazyflie (i.e. hover_vicon.launch)?

Hello,
Yes , I can get single and multiple(2) hover working. But run into problems on multi waypoint (single and double).
I tried using a single crazyradio for both as well as a two radios for both. It just jumps for a second nd lands.

I just pushed another change to the launch file which hopefully fixes the issue you are seeing. Note that I don't have access to a motion capture space right now to try it out though. Please let me know how it goes!

Sure,
I will check it out tomorrow morning and get back to you. What do you
suspect the issue is ?.
Thank you,
On Dec 14, 2015 9:30 PM, "whoenig"<EMAIL_ADDRESS>wrote:

I just pushed another change to the launch file which hopefully fixes the
issue you are seeing. Note that I don't have access to a motion capture
space right now to try it out though. Please let me know how it goes!
—
Reply to this email directly or view it on GitHub
https://github.com/whoenig/crazyflie_ros/issues/20#issuecomment-164633931
.

This launch file was just not updated in a while. I changed the way on how to use the controller slightly and the launch file was not updated. At the end, it didn't load the PID parameters from the YAML file, causing the sudden drop after the take-off sequence.

Hello,
The new launch file worked great. Thanks a lot.
There is one small error. With respect to the frame of crazyflie :
<include file="$(find crazyflie_controller)/launch/crazyflie2.launch">
  <arg name="frame" value="$(arg frame)" />
</include>

-> Arg frame should be arg frame1.
Thank you so much for all your help.
Cheers

Thanks for the feedback. I fixed those arguments!


Q: How to document CallExpression/CallSignature with TypeDoc?

Search terms
CallSignature, CallExpression, CucumberJS, Function Calls, Custom Reflections
Question
TL;DR

Is there a generic way to convert a ts.JSDoc into a list of TypeDoc tokens that I can put into a reflection?
Is there a way to register a custom reflection explicitly dedicated to function CallExpressions that is placed in it's own folder?

Related to a thread on Discord.
Details
I'm using the CucumberJS framework for some integration tests, and we distribute some framework-specific code to other people in the form of step definitions. These step definitions are function calls that are registered at runtime to provide BDD assertions in the form of Gherkins. The following is an example of a step definition (taken straight from the docs with an added TSDoc):
import { Given } from '@cucumber/cucumber';

/**
 * Prepare the actor with a specific number of cucumbers having already being eaten.
 *
 * @param cucumberCount The number of cucumbers to eat
 *
 * @example
 * ```
 * ...
 * ```
 */
Given('I have {int} cucumbers in my belly', function (cucumberCount: number) {
  assert.equal(this.responseStatus, cucumberCount)
});

I want to write a plugin that is able to convert those function calls into readable documentation. So far, it's been going quite nicely in the conversion stage of things, but I'm getting stuck after having extracted the function details and JSDoc from the function. Is there a generic method that can convert a ts.JSDoc to a typedoc.Comment or some form of tokens that I can place directly into a reflection? I was looking at lexBlockComment as that solution, but it's not publicly available through the exports. It might be a little bit too low-level for my needs anyway, but I'm not seeing much else available for my inputs.
Additionally, I haven't quite figured out how to register a reflection that goes into it's own folder. I notice that each part of a reflection can get its own folder (ex. "classes", "enums", "functions", "types", "modules") based on the ReflectionKind that's used in the DeclarationReflection. There doesn't seem to be a way to create a custom ReflectionKind and have it create a unique folder. This is a very niche use-case, so I might be the first person to try something like this.
Any help I can get with these questions would be greatly appreciated! 🙏
Current Demo Plugin
Here's what I have implemented so far. It's fairly basic, but you can see the part in readSourceFile() where it extracts the functions that I'm attempting to convert to docs.

Demo Plugin Code
import { glob } from 'glob';
import {
	Application,
	Context,
	Converter,
	DeclarationReflection,
	ParameterType,
	ReflectionKind,
	Comment,
	TypeScript as ts,
} from 'typedoc';

const CUCUMBER_FUNCTION_NAMES: CucumberFunctionNames[] = [
	'Given',
	'When',
	'Then',
	'Before',
	'After',
	'AfterAll',
	'BeforeAll',
	'BeforeEach',
	'AfterEach',
];

type CucumberFunctionNames =
	| 'Given'
	| 'When'
	| 'Then'
	| 'Before'
	| 'After'
	| 'BeforeAll'
	| 'AfterAll'
	| 'BeforeEach'
	| 'AfterEach';

function getUniquePrograms(programs: readonly ts.Program[]): readonly ts.Program[] {
	const uniqueMap: Record<string, ts.Program[]> = programs.reduce((acc, p) => {
		const root = p.getCurrentDirectory();
		acc[root] = acc[root] ?? [];
		acc[root].push(p);
		return acc;
	}, {});
	return Object.values(uniqueMap).map((pArray) => pArray[0]);
}

/**
 * Reads a TypeScript file and extracts any documentation on Step Definitions.
 * @param sourceFile The TypeScript source file
 */
function readSourceFile(sourceFile: ts.SourceFile) {
	// These are what store the JSDocs
	const toplevelExpressions = sourceFile.statements.filter(
		(f) => f.kind == ts.SyntaxKind.ExpressionStatement
	) as ts.ExpressionStatement[];

	const sources: {
		sourceFile: ts.SourceFile;
		jsDoc: ts.JSDoc;
		functionName: CucumberFunctionNames;
	}[] = [];

	for (const expression of toplevelExpressions) {
		const cucumberFunctionName = isCucumberFunctionCall(expression);
		if (cucumberFunctionName === false) {
			// Not a cucumber function
			continue;
		}

		if (expression['jsDoc'] === undefined) {
			continue;
		}

		const jsDoc = expression['jsDoc'] as [ts.JSDoc];
		// const docComment = sourceFile.getFullText().slice(jsDoc[0].pos, jsDoc[0].end);

		sources.push({
			functionName: cucumberFunctionName,
			sourceFile,
			jsDoc: jsDoc[0],
		});
	}
}

/**
 * Determines if the expression is a cucumber support code function call.
 * @param expression
 * @returns the name of the cucumber function that was called, or false in a situation where the expression is not a function call or a cucumber function.
 */
function isCucumberFunctionCall(expression: ts.ExpressionStatement): CucumberFunctionNames | false {
	const innerExpression = expression.expression as ts.CallExpression;
	if (innerExpression.kind !== ts.SyntaxKind.CallExpression) {
		return false;
	}

	if (innerExpression.expression.kind !== ts.SyntaxKind.Identifier) {
		return false;
	}

	if (
		!CUCUMBER_FUNCTION_NAMES.includes(
			(innerExpression.expression as ts.Identifier).escapedText.toString() as any
		)
	) {
		return false;
	}

	return (
		innerExpression.expression as ts.Identifier
	).escapedText.toString() as CucumberFunctionNames;
}

export function load(app: Application) {
	app.options.addDeclaration({
		name: 'stepDefinitions',
		help:
			'Extract TSDocs from cucumber step definitions listed at the specified paths. ' +
			'Should be a list of files relative to the project root, represented as a relative path or glob.',
		type: ParameterType.GlobArray,
		defaultValue: [],
	});

	app.converter.on(Converter.EVENT_RESOLVE_BEGIN, async (ctx: Context) => {
		const paths = ctx.converter.application.options.getValue('stepDefinitions') as string[];

		if (paths.length == 0) {
			return;
		}

		const resolvedPaths = (await Promise.all(paths.map((p) => glob(p)))).flat();

		for (const program of getUniquePrograms(ctx.programs)) {
			for (const path of resolvedPaths) {
				const sourceFile = program.getSourceFile(path);

				if (!sourceFile) {
					continue;
				}

				readSourceFile(sourceFile);
			}
		}

		const reflection = new DeclarationReflection('customThingy', ReflectionKind.Module);
		reflection.comment = new Comment(
			[
				{
					kind: 'text',
					text: 'this is a custom comment',
				},
			],
			[]
		);

		reflection.children = [
                         // attempting to see if I can register a custom module with a "function" declaration, where the function
                         // is just the information I extract from the step definitions
			new DeclarationReflection('fakeCallSignature', ReflectionKind.Function, reflection),
		];

		ctx.project.children?.push(reflection);
	});
}

I think it might be necessary to monkey-patch the typedoc.DefaultTheme.getUrls() and typedoc-plugin-markdown.MarkdownTheme.getUrls() in order to get the structure we want. I wanted to avoid having dedicated themes to make this work if possible.

There doesn't seem to be a way to create a custom ReflectionKind and have it create a unique folder

There is not, kinds are intended to map to TS types of symbols, and there's logic in TypeDoc that depends on having covered all of the kinds to be sure things work correctly. Surprisingly, to me, you're at least the fourth person to have tried something like this.
I want to eventually make it possible for people to create custom folder structures in output, but have yet to come up with something I'm happy with... not concretely tied to any release target yet. You're going to be stuck with patching or a custom theme for now.
It's worth considering if adding a @group tag to show the organization on a website might be enough.

Is there a generic method that can convert a ts.JSDoc to a typedoc.Comment or some form of tokens that I can place directly into a reflection?

For this specific use case, no. There are a couple comments on Context which let you get a comment for a symbol, but you don't have a symbol for the Given call, just a Node. TypeDoc doesn't need this, but exposing a method for getting a Node's comment seems reasonable, I'll expose a Context.getNodeComment method in the next release.
Using that, I made a couple tweaks to your plugin to do this:

Plugin source
import { globSync as glob } from "glob";
import { Application, Context, Converter, DeclarationReflection, ParameterType, TypeScript as ts } from "typedoc";

declare module "typescript" {
    interface Node {
        symbol?: Symbol;
    }
}

const CUCUMBER_FUNCTION_NAMES = [
    "Given",
    "When",
    "Then",
    "Before",
    "After",
    "AfterAll",
    "BeforeAll",
    "BeforeEach",
    "AfterEach",
] as const;

type CucumberFunctionNames = (typeof CUCUMBER_FUNCTION_NAMES)[number];

/**
 * Reads a TypeScript file and extracts any documentation on Step Definitions.
 * @param sourceFile The TypeScript source file
 */
function readSourceFile(sourceFile: ts.SourceFile, context: Context) {
    // These are what store the JSDocs
    const toplevelExpressions = sourceFile.statements.filter(ts.isExpressionStatement);

    for (const expression of toplevelExpressions) {
        const cucumberFunctionName = isCucumberFunctionCall(expression);
        if (cucumberFunctionName === false) {
            // Not a cucumber function
            continue;
        }

        const expr = expression.expression as ts.CallExpression;
        const template = expr.arguments[0] as ts.Identifier;
        const implementation = expr.arguments[1];

        const symbol = implementation.symbol;
        if (!symbol) return; // Something broke. Should always have one.

        context.converter.convertSymbol(context, symbol);
        const reflection = context.project.getReflectionFromSymbol(symbol) as DeclarationReflection | undefined;
        if (!reflection) continue; // Excluded, probably because of an @ignore

        reflection.name = `${cucumberFunctionName} ${template.text}`;
        for (const sig of reflection.signatures || []) {
            sig.comment ||= context.getNodeComment(expression, sig.kind);
            sig.name = cucumberFunctionName;
        }
    }
}

/**
 * Determines if the expression is a cucumber support code function call.
 * @param expression
 * @returns the name of the cucumber function that was called, or false in a situation where the expression is not a function call or a cucumber function.
 */
function isCucumberFunctionCall(expression: ts.ExpressionStatement): CucumberFunctionNames | false {
    const innerExpression = expression.expression as ts.CallExpression;
    if (!ts.isCallExpression(innerExpression)) {
        return false;
    }

    if (!ts.isIdentifier(innerExpression.expression)) {
        return false;
    }

    if (!CUCUMBER_FUNCTION_NAMES.includes(innerExpression.expression.text as any)) {
        return false;
    }

    return innerExpression.expression.text as CucumberFunctionNames;
}

export function load(app: Application) {
    app.options.addDeclaration({
        name: "stepDefinitions",
        help:
            "Extract TSDocs from cucumber step definitions listed at the specified paths. " +
            "Should be a list of files relative to the project root, represented as a relative path or glob.",
        type: ParameterType.GlobArray,
        defaultValue: [],
    });

    app.converter.on(
        Converter.EVENT_RESOLVE_BEGIN,
        (ctx: Context) => {
            let paths = app.options.getValue("stepDefinitions") as string[];

            if (paths.length == 0) {
                paths = app.options.getValue("entryPoints");
            }

            const resolvedPaths = paths.map((p) => glob(p)).flat();

            for (const program of ctx.programs) {
                for (const path of resolvedPaths) {
                    const sourceFile = program.getSourceFile(path);

                    if (!sourceFile) {
                        continue;
                    }

                    const context = ctx.withScope(ctx.project);
                    context.setActiveProgram(program);
                    readSourceFile(sourceFile, context);
                }
            }
        },
        100,
    );
}

A couple notes on the changes:

Event handlers may not be async.
context.programs containing duplicates was a bug, fixed now.
Adding additional reflections to the project should be done with a high priority so that if the user includes @hidden in a comment, the comment plugin can remove it. A priority of 1 is sufficient, but 100 makes it possible to insert additional listeners in between more easily.
(FYI, not doing manual creation above) If creating reflections manually, they should be created with a reference to their parent, and context.project.registerReflection should be called. If it isn't then some things like link resolution will mysteriously break. context.createDeclarationReflection will do this for you.

@Gerrit0 This is a FANTASTIC example that you provided. This is absolutely the piece of information that I was missing to make this plugin work. I do believe I can use custom reflections/themes with that generated function signature in order to build out the plugin the way I was hoping. I cannot express how much I appreciate this

Adding additional reflections to the project should be done with a high priority so that if the user includes @hidden in a comment, the comment plugin can remove it. A priority of 1 is sufficient, but 100 makes it possible to insert additional listeners in between more easily.

Could you elaborate on the priority a bit further? Are you saying that we should recommend that the plugin is loaded early to prevent changes from this plugin being ignored by other plugins?

Surprisingly, to me, you're at least the fourth person to have tried something like this.

Also regarding this, I can imagine it being useful for more than just Cucumber to be able to document framework-level reflections for specific things. Mainly, these would only really apply to things that are governed by the following properties:

Generated/registered at runtime (which is the case for Cucumber)
Governed by folder structure (not totally sure if/why you'd want to document routes with NextJS through TypeDoc)
Not traditionally exported by the program through any listed entry points, but still expected to be required/imported for any such reason (which is also the case for Cucumber)

I think it's probably important to consider the scope of TypeDoc when making decisions on how well to support these use-cases, because it really makes your life infinitely harder to deal with the edge-cases that arise from that. Like you mentioned, TypeDoc is bound to TS symbols, so it maybe shouldn't do more than that anyway. Just thinking out loud here.

I want to eventually make it possible for people to create custom folder structures in output, but have yet to come up with something I'm happy with... not concretely tied to any release target yet.

Both this repo and typedoc-plugin-markdown do their folder/type mapping almost the exact same way. You've already provided the types to be able to make a generic interface for the mappings. It's certainly possible to make a standardized equivalent to DefaultTheme.mappings extensible for plugins, depending on how easily you can abstract renderer implementations over the mapping template. I can see how you'd have much trouble creating a solution for this, because there are downsides either way. I might look into it this weekend to see if I could come up with a decent solution.

My POC was successful. I will open any issues if I come across any other problems


Post photo

Background
We gonna certainly support a capability to upload Photo with relevant information, such as Author, tag, time.
Ideally, this endpoint only primarily serves for our Internal Dashboard.
Acceptance Criteria

POST /photo
Store photo data appropriately in Postgresql database.
Upload image to S3

Close.
We've already implement the spider man bot to fetch the info and author data directly from Wikiart. Then posting to creating Photo Endpoint.
I believe that we don't need this ticket anymore.


adding view.on() breaks mousewheel zooming in IE

ol3 v3.6.0 and IE 9  ( possibly other versions of IE ? )
When adding view.on('change:resolution', function() { some_actions })
the mousewheel zoom does no longer function.
Some debugging shows that the this.delta_  is not correctly reset. ( It keeps on cumulating with every zoom request in the same direction )
Firefox and Chrome behave correctly.

After some more digging, it turns out that adding a "console.log" statement in the view.on() function causes this !
view.on('change:resolution', function() {
var z = view.getZoom();
console.log('zoomlevel='+z);
});

Maybe not related to ol3 ... ( however, the this.delta_ definitely is not reset to 0 as would be expected )


Document legacy upload API endpoint

Followup to https://github.com/pypa/warehouse/issues/2930#issuecomment-367155334 .
Also improve some headers and tables of contents organization and display.

Fixed.


Duplicate Imports

There are duplicate imports in constants.dart. I am a GSSoC'24 contributor.

Assign this issue to me i want to contribute into this section

Will you please assign this issue to me? I would love to work on this project issue.


[Help] Gui Error

static void drawSmokeCrosshair(ImDrawList* drawList, const ImVec2& pos, ImU32 color, float thickness) noexcept
{
    drawList->Flags &= ~ImDrawListFlags_AntiAliasedLines;

    // left
    drawList->AddRectFilled(ImVec2{ pos.x - 11 - 1000, pos.y - 1 }, ImVec2{ pos.x - 3, pos.y + 2 + config->backtrack.something3 }, color & IM_COL32_A_MASK);
    drawList->AddRectFilled(ImVec2{ pos.x - 10 - 1000, pos.y }, ImVec2{ pos.x - 4, pos.y + 1 + config->backtrack.something3 }, color);

    // right
    drawList->AddRectFilled(ImVec2{ pos.x + 4, pos.y - 1 }, ImVec2{ pos.x + 12 + 1000, pos.y + 2 }, color & IM_COL32_A_MASK);
    drawList->AddRectFilled(ImVec2{ pos.x + 5, pos.y }, ImVec2{ pos.x + 11 + 1000, pos.y + 1 }, color);

    // top (left with swapped x/y offsets)
    drawList->AddRectFilled(ImVec2{ pos.x - 1, pos.y - 11 - 700 }, ImVec2{ pos.x + 2 , pos.y - 3 }, color & IM_COL32_A_MASK);
    drawList->AddRectFilled(ImVec2{ pos.x, pos.y - 10 - 700 }, ImVec2{ pos.x + 1 , pos.y - 4 }, color);

    // bottom (right with swapped x/y offsets)
    drawList->AddRectFilled(ImVec2{ pos.x - 1, pos.y + 4 }, ImVec2{ pos.x + 2, pos.y + 12 + 700 }, color & IM_COL32_A_MASK);
    drawList->AddRectFilled(ImVec2{ pos.x, pos.y + 5 }, ImVec2{ pos.x + 1, pos.y + 11 + 700 }, color);

    drawList->Flags |= ImDrawListFlags_AntiAliasedLines;
}

Above is a function similar to drawCrosshair (slightly modified), I've been having trouble using it to draw a costume crosshair, below is how I'm calling the function, my potato brain doesn't compute. beep boop!
void Misc::SomeFunction(GameEvent& event) noexcept
{
    if (config->SomeFunction.enable) {
        ImDrawList* draw_list = ImGui::GetWindowDrawList();
        if (Somestuff Somestuff) {
            drawSmokeCrosshair(draw_list, ImGui::GetIO().DisplaySize / 2, IM_COL32(255, 0, 255, 255), config->misc.some.thickness);
    }
}

what's the error?

Oh snap my bad my bad, It was "g.CurrentWindow was nullptr."

try this
void Misc::SomeFunction(GameEvent& event) noexcept
{
    if (config->SomeFunction.enable && event) {
        if (Somestuff Somestuff) {
            drawSmokeCrosshair(ImGui::GetBackgroundDrawList(), ImGui::GetIO().DisplaySize / 2, IM_COL32(255, 0, 255, 255), config->misc.some.thickness);
    }
}

hmm it fixed the instant crash when c++ if (Somestuff Somestuff)   is true but it doesn't seem to draw the crosshair T,T

u may add this after ImGui::NewFrame(); in hooks.cpp
Misc::SomeFunction(nullptr, ImGui::GetBackgroundDrawList());
misc.cpp
void Misc::SomeFunction(GameEvent* event, ImDrawList* drawList) noexcept
{
    if (config->SomeFunction.enable && event) {
        if (Somestuff Somestuff) {
            drawSmokeCrosshair(drawList, ImGui::GetIO().DisplaySize / 2, IM_COL32(255, 0, 255, 255), config->misc.some.thickness);
    }
}

misc.h
void SomeFunction(GameEvent* event, ImDrawList* drawList = nullptr) noexcept;
eventListener
Misc::SomeFunction(event);

Hmmm
New error, and instant crash on inject T.T
'csgo.exe' (Win32): Loaded 'A:\STEAM\steamapps\common\Counter-Strike Global Offensive\bin\serverbrowser.dll'.
'csgo.exe' (Win32): Loaded 'C:\Windows\SysWOW64\dhcpcsvc6.dll'.
'csgo.exe' (Win32): Loaded 'C:\Windows\SysWOW64\wlanapi.dll'.
'csgo.exe' (Win32): Loaded 'C:\Windows\SysWOW64\winhttp.dll'.
'csgo.exe' (Win32): Loaded 'A:\STEAM\steamapps\common\Counter-Strike Global Offensive\bin\vaudio_celt.dll'.
The thread 0xc18 has exited with code 0 (0x0).
'csgo.exe' (Win32): Loaded 'E:\Games\Opera\New folder\Osiris\Debug\Osiris.dll'. Symbols loaded.
The thread 0x570c has exited with code<PHONE_NUMBER> (0x7e6a0000).
Exception thrown at 0x7E9F6F2A (Osiris.dll) in csgo.exe: 0xC0000005: Access violation reading location 0x00000000.

'csgo.exe' (Win32): Loaded 'C:\Windows\SysWOW64\dbgcore.dll'.
The thread 0x3c8 has exited with code 1 (0x1).
The thread 0x1f00 has exited with code 1 (0x1).
The thread 0x3990 has exited with code 1 (0x1).

I uploaded the repository in the Experimental branch in my fork: https://github.com/grey-stack/Osiris/tree/Experimental
I'm really lost right now and the error is too advanced for noobies like me ,,, rip

here > if (config->misc.smokeCrosshair && event && event->getInt("weptype") == 9)

tried your suggestion, instant crash no longer happening butttt crashes when the suggested statement is true drawSmokeCrosshair hates me big time, Oh and Happy New Year!


[Storage] Retries exceed MaxRetries if server sends partial response

When using<EMAIL_ADDRESS>and<EMAIL_ADDRESS>BlobClient.DownloadToAsync() appears to retry indefinitely if the server sends a partial body, even though MaxRetries = 1.
Repro Steps

Start a server which can inject HTTP faults, like https://github.com/mikeharder/HttpFaultInjector.

Create a BlobClient with the following options:

var httpClient = new HttpClient(httpClientHandler)
{
    Timeout = TimeSpan.FromSeconds(5)
};

var blobClientOptions = new BlobClientOptions()
{
    Transport = new ChangeUriTransport(new HttpClientTransport(httpClient), "localhost", 7778),
    Retry = {
        MaxRetries = 1,
        NetworkTimeout = TimeSpan.FromSeconds(10)
    },
};

var blobClient = new BlobClient(connectionString, "testcontainer", "testblob", blobClientOptions);
var response = await blobClient.DownloadToAsync(new MemoryStream());

private class ChangeUriTransport : HttpPipelineTransport
{
    private readonly HttpPipelineTransport _transport;
    private readonly string _host;
    private readonly int? _port;

    public ChangeUriTransport(HttpPipelineTransport transport, string host, int? port)
    {
        _transport = transport;
        _host = host;
        _port = port;
    }

    public override Request CreateRequest()
    {
        return _transport.CreateRequest();
    }

    public override void Process(HttpMessage message)
    {
        ChangeUri(message);
        _transport.Process(message);
    }

    public override ValueTask ProcessAsync(HttpMessage message)
    {
        ChangeUri(message);
        return _transport.ProcessAsync(message);
    }

    private void ChangeUri(HttpMessage message)
    {
        // Ensure Host header is only set once, since the same HttpMessage will be reused on retries
        if (!message.Request.Headers.Contains("Host"))
        {
            message.Request.Headers.Add("Host", message.Request.Uri.Host);
        }

        message.Request.Uri.Host = _host;
        if (_port.HasValue)
        {
            message.Request.Uri.Port = _port.Value;
        }
    }
}

Run the client app, and instruct the server to send a partial response (say 50% of the body).

The client will read the partial response, block for 10 seconds (NetworkTimeout), then retry but only requesting the remaining 50% of bytes (with a Content-Range header).

Again instruct the server to send a partial response (50% of the remaining body == 25% of the original body).

The client will again read the partial response, block for 10 seconds (NetworkTimeout), then retry but only requesting the remaining 25% of original bytes.

This can be repeated indefinitely, despite MaxRetries=1.  I'm not sure if this is a bug or by design.

Might also repro if service sends headers but no body (compared to 50% of body).

I'm not sure how to best approach this without making customer code less reliable.
It's true that we're applying MaxRetries only to the attempt to re-establish connection when it fails but if connection resumed succesfully we "reset" the retry counter.
But looking at our current default of 3 retries it's insufficient to be able to successfully download a reasonably large file.
I'm tempted to backlog this until we get reports from customers that we are trying too hard.
@tg-msft what are your thoughts here?

I agree with @pakrym for our current design.  Otherwise we'll have to add more configurable knobs allowing users to explicitly control this scenario if and when we hear customer feedback.  I would also welcome input from @JeffreyRichter as he probably knows of any serious flaws with this approach in practice.

I'm not familiar with your exact implementation but why would you reset the retry counter if something is successful? My mental model (and what I've implemented in the past) was that each operation (XxxClient method) starts out with the customer's configured max retry count and then this decrements with each try and nothing resets this. This works great for everything.
Now, blob download has an additional issue in that after you've left the pipeline, you return the network stream back to customer code and it continues to read the blob. Since you've left the pipeline, the retry count no longer applies. What I did in this case, was a created a ReliableStream class and had an instance of this wrap the NetworkStream. The ReliableStream ctor took a max retry count (which was unrelated to the retry count used by the pipeline policy) and no retry delay options - just a count. Now, while reading from the ReliableStream, if it detected a network failure, it could retry calling DownloadBlob which would then use the pipeline policy's retry. The end result is that that you had one set of retries to initiate downloading a blob and another (very simple) retry that only kicked in once you started reading from the stream.

We have very similar implementation but our ReliableStream currently doesn't limit retry count.

So my understanding of this issue is that we'll do the following:

Start downloading with the customer's MaxRetries via the pipeline
Wrap in a RetriableStream
If downloading the stream fails after we've left the pipeline
a. Try to start downloading from where we left off with the customer's MaxRetries via the pipeline
b. If that succeeds, go back to step 2
c. If that fails, loop back to step 3, up to 3 times
Return the downloaded blob

We don't adjust MaxRetries in Step 3a to account for the number of attempts in Steps 1 or previous iterations of Step 3a.  We also don't keep a running number of attempts in Step 3c so every stream failure kicks off a whole new cycle.  This could take a very long time under adverse circumstances (absolute worst case being 3 * MaxRetries * BlobSize requests if Mike sets up a test server that fails after sending back a single byte).
Mike's point in raising the issue is number of actual retries that could happen in a flaky blob download doesn't correspond to how the customer envisions MaxRetries working.  Pavel and I are okay with this until we hear from customers at which point we could add a few more RetryPolicy knobs to allow configuring what happens in Step 3.

Customers never reported problems with this behavior.


SSRS

Will Managed Instances provide a license to run a separate SSRS server?
We really want to move our app to Azure Sql, but still requiring a stand alone Sql license in order to run SSRS has prevented us.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 28ca1f7f-5a34-0f23-10ce-3ff24c76db97
Version Independent ID: 5c4a22c4-78fc-911d-9fa0-8dfab1653b6e
Content: Azure SQL Database Managed Instance Overview
Content Source: articles/sql-database/sql-database-managed-instance.md
Service: sql-database
GitHub Login: @bonova
Microsoft Alias: bonova

@pmiddleton Thanks for your feedback! We will investigate and update as appropriate.

@pmiddleton Please refer to SQL Server feature support in Azure SQL Database, for SSRS it says 'No - Use PowerBi' for both Azure SQL Database and Managed Instance.

@pmiddleton We will now proceed to close this thread. If there are further questions regarding this matter, please open a new issue and we will gladly continue the discussion.


v3.4: revert bootstrap script before final release

The v3.4 release currently depends on some Invenio modules that have been released as pre-releases (alpha, a1).
Pipenv is not very happy about that and there is no easy way to install such packages pre-releases only.
The /scripts/bootstrap has been changed to use pip instead.
Check the comments in the script and revert the changes.

These are the lines:
# START: PUT ME BACK
if [ ! -f $pipfile_lock_path ]; then
    echo "'Pipfile.lock' not found. Generating via 'pipenv lock --dev'..."
    pipenv lock --dev --pre
fi

# Installs all packages specified in Pipfile.lock
pipenv sync --dev
# /END: PUT ME BACK

# START: REMOVE ME
# pipenv run pip install -e git+https://github.com/inveniosoftware/invenio.git#egg=invenio["base","auth","metadata","files","postgresql","elasticsearch7"]
# /END: REMOVE ME

The changes have already been put back. What's missing is removing the commented code and markers.

Should not be closed, the --pre should be removed, not just the comment??


fix: repository summary on mobile

This PR fixes the repository summary on mobile. Most of it is vertically centering things and some spacing.
Before:

After:

make lgtm work


Create issue template for Flank

Ask to always test on latest snapshot
https://help.github.com/en/github/building-a-strong-community/configuring-issue-templates-for-your-repository

@bootstraponline I need settings permission to resolve this ticket

@jan-gogo you should be able to manage all the issues in this repo. I verified you have permission. Are you getting an error message? Have you tried logging out/logging back in?

@jan-gogo I upgraded you to maintainer, try now.

Fixed in https://github.com/Flank/flank/commit/0d8cfa43c6663977059ada54a4da1447eb3967c2


[Enhancement] Optimize the mem usage of collect stats for delta lake

Why I'm doing:
What I'm doing:
No need to cache the stats in ScanFileTask, just drop it after adding up.
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

@Mergifyio backport branch-3.3


appleseedapp.net still has latest release as <IP_ADDRESS>5

Hi Folks,
Just a heads up that the official site still says latest is <IP_ADDRESS>5 from 12/2016...almost wandered away from the project due to lack of activity, but stumbled across the repo and saw it has been being coded recently.
Dave

Dave,
Thanks for letting us know! We'll have the website updated tomorrow. We've been so focused on client work we haven't kept up with our open-source software development. I'll take care of it tomorrow :)

We've updated the website and we should be releasing a new release in about 1 month.
https://www.appleseedapp.net/Download.aspx
:)


DreamVar is not supported, probably not needed

Due to extra complexity in supporting synaptic-level random variables, this has not been implemented for a while and is now officially gone.  Can reinstate at some later point as we scale up..

//	DreamVar float32     `viewif:"On" def:"0,0.01,0.02" desc:"extra random variability to add to LWts after every SWt update, which theoretically happens at night -- hence the association with dreaming.  0.01 is max for a small network that still allows learning, 0.02 works well for larger networks that can benefit more.  generally avoid adding to projections to output layers."`


B 17975 customer submitted final ppm email

Agility ticket
This PR is for merging #11793 into main
Link to old PR
PR 11737 was made right before the new dev process was released, read it for relevant comments and reviews:
#11737
Summary
This branch sends an email to a service member once they've uploaded their final PPM documentation (i.e. after they have completed their move).
Since #11729 has been merged, but the full epic is not complete, the email template conditionally uses the City and State from the PPM if it is available, and falls back to using the ZIPs if that is the only data provided.
I added and marked the new pickup/destination_address_id fields in the PPMShipmentModel as optional, since work is not yet fully complete on that epic.
How to test

Run the server with the following command to activate sending emails through local dev:
EMAIL_BACKEND=ses AWS_REGION=us-gov-west-1  aws-vault exec transcom-gov-dev -- make server_run

Access the "testharness scenarios" page from milmovelocal:3000

Select the MoveWithPPMShipmentReadyForFinalCloseout option to create a move at the final stage with all needed receipts/docs already uploaded.

Login as that new user (it will always be the top result on the user list if you haven't done anything else after creating).

Before uploading, click "edit" next to step 1 "Profile". In the first section, edit your email to be your CACI email (if you're in the local dev env, you can have as many users pointing to the same email as you want).Screenshot 2024-01-11 at 5.43.42 PM

Now click "Upload PPM Documentation", you should see all fields already filled out.

Once you proceed and sign your name, you should get an email shortly.

This scenario create an ARMY service member. If you want to test the other templates for air/space force, and the naval branches of service, you'll have to go through the entire process manually. (Submit PPM Move -> Login as SC and approve -> Login as TOO and approve if needed -> Login as user and complete step 5)
Backend

[x] Code follows the guidelines for Logging.
[x] The requirements listed in Querying the Database Safely have been satisfied.

issue:  "Next Steps" is bolded
Per test step 7 in the attached screenshot, "Next Steps" is not bolded.  But, in the email received, it is bolded.

issue: "You must complete..." is bolded
"You must complete this step to receive final settlement of your PPM."
Per test step 7 in the attached screenshot, this text is not bolded.  But, in the email I received, it was.

email screenshot needs updating.
It doesn't match the template attached in agility.

testing issue:  "Next Steps" is bolded in received email.  Template does not have it bolded.

Testing notes:

army email: failed

issue:  "Next Steps"

air force email: failed

issue:  "Next Steps"

Note:
Email template:
See:  agility item, attached doc, PPM Closeout Complete EMail 15 Dec 2023.docx

@JamesHawks224

Note: Email template: See: agility item, attached doc, PPM Closeout Complete EMail 15 Dec 2023.docx

Yeah, I'll ask Gary about this in the Slack, I assumed it was an error in the template given since every other existing template bolds the "Next Step" header, but they do seem pretty strict about the template so it'll be good to get confirmation.

Putting in a note:
Commit hash for the merge into integration was:
eb99a49544c79a8325567eae986bbfd164c09bc4
comparison of the file lists between the merge and the branch:
lists were the same except for the following exception:
Merge had 1 extra file.  .circleci/config.yml


Initial refactor to support packr to bundle templates

This PR removes the templates directory and introduces a dependency on Packr2 in order to bundle those things into the Porter binary directly.
Previously, porter create would read those files from the PORTER_HOME and put them into the bundle. Now they are easy bake oven'ed into pkg/config and are fetched that way, rather than reading the file system.
The PR updates the existing tests that read those and write into FS for things like build tests. Now they are fetched from the config and written to the FS via a new test helper method.
A Test as also added to config to validate that what we get from the generated box matches what's in pkg/config/templates directory.
Fixes #118

If you incorporate the changes from my PR, the build will pass (or at least I think that's what is missing)
https://github.com/deislabs/porter/pull/146/files#diff-836546cc53507f6b2d581088903b1785
https://github.com/deislabs/porter/pull/146/files#diff-b67911656ef5d18c4ae36cb6741b7965

Hey @carolynvs and @vdice take a look. I think I got all the doc changes.


Add admin API key for the docker-compose example

Currently, the configuration does not include an Admin API key.

yes, you are right. Welcome PR :)


add meta section and license badge

GitHub shows the license. Also, readme is what generates the website home page so I'd prefer not add another section


Update to rails 5.2.6

Upgrade to address CVE-2021-22904 - a possible information disclosure / unintended method execution vulnerability in
Action Pack when using the redirect_to or polymorphic_url helper with untrusted user input.

I'll look into why this build and https://github.com/ausaccessfed/status-service/pull/136 is failing

I'll look into why this build and ausaccessfed/status-service#136 is failing

Was brakeman @waldofouche I've fixed it across the board.

Ahh okay yep. Did you want me to test these all locally?

No, CI proof is better/enough.


add Flux.jl downstream test

with #111 we can run downstream tests labeling PRs with run downstream test


Add Enrichment instance caching to Enricher

Creates a cache of enrichment objects for the current instance of Krikri::Enricher. This allows individual enrichment instances to maintain their own caches/persistence/optimizations for the life of (e.g.) a queued job.

I'm :+1: - anyone else want to take a look?

Sure, that makes sense.  The frequent instantiation of that class must have been expensive.  I'm happy to merge it.

:rocket: It's not crucial, but it wouldn't hurt to have this in when the UW enrichment starts.


Minor error in the wiki page Host_management

Problem statement
The wiki page Host_management has following:
Example config (~/.config/batchd/controllers/aws1.yaml):
The controllers within the path should be hosts
Steps to reproduce

browse the wiki Host_management page

Expected result
Example config (~/.config/batchd/hosts/aws1.yaml):
Actual result
Example config (~/.config/batchd/controllers/aws1.yaml):
batchd version
Github master
Please specify whether you are using installation from hackage, or building source from github.
Built from github

fixed


Fixed relative links in generated docs

Relative links did not have 'docs/' prefix, so all of them lead to 404 page.
Fixes #5964

waiting for manual merge to build https://circleci.com/gh/facebook/react-native/2746 current docs before shiping this to master

@facebook-github-bot shipit

@facebook-github-bot shipit


Write proper modding instructions on the wiki

Right now there's just some rough notes from me: https://wiki.revolutionarygamesstudio.com/wiki/Modding
those need to be rewritten to proper instructions.

Finished the first overall draft of a basic guide really shortly going over the required main points on the wiki.


[BUG] Error with something

群组发送消息失败
[OpenIM-App] ** message send failed e :PlatformException(1001, 1001 ArgsError, null, null)  #0      StandardMethodCodec.decodeEnvelope (package:flutter/src/services/message_codecs.dart:652:7)
#1      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:310:18)

@yuantianxiang Refer to the open source demo.


[PL-52028] Platform 1.43.6 release notes

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: Platform 1.43.6 release notes
Jira/GitHub Issue numbers (if any): PL-52028
Preview links/images (Internal contributors only): Platform 1.43.6 release notes

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[x] Successful preview build.
[x] Code owner review.
[x] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6679dad939fd8c338ffa47d1--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://667c1f8e7bb2cc5573999876--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://667c24e4498501ed6b245a6a--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://667d69fff61a71007b48d59f--harness-developer.netlify.app


ADR001

Closes: #1
Description
ADR 001 describing the repository structure.

For contributor use:

[ ] Unit tests written
[ ] Added test to CI if applicable
[ ] Updated CHANGELOG_PENDING.md
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Updated relevant documentation (docs/) and code comments
[ ] Re-reviewed Files changed in the Github PR explorer

My initial thought was to use this issue to propose a layout, to go a bit more into detail and explain the layout of modules and relayer, e.g. typical files in ics0X like msgs.rs, query.rs, what is chain specific and what is generic, what goes in `relayer/cli/src, etc. At this point we may leave it like this or still describe it in more detail and review with everyone and make any changes now if needed. Thoughts?

The details of the layout are not very clear to me at this point, so describing this would be very valuable to clarify! I'm in favor of providing more details (as much as we can, it's ok if some parts are debatable still) + reviewing with everyone (perhaps during the call next Tuesday).

Lets just merge for now


Iss499: author name confused with protein family

Add logic to remove bibliographic references in parentheses.

@hickst: this is way to aggressive: you are removing any statements within ().
We need more informed patterns, to guess if what's inside looks like a citation. For example, does it contain "et al."? Does it contain a 4-digit year? Does it have that XREF_BIB tag?

That is incorrect. Please read the code again.

I see. I missed the stringHasBibRef() call. But why isn't that functionality inside the pattern anyway? Seems overkill the way it looks now.

It could be but the test is a logically separate function; especially in case we want to complicate the test in the future.


Function.getValueOpt is Not Fully Implemented

The Function.getValueOpt method is not implemented for the following interpolation methods:

polynomial
shepard

Furthermore, it can be made faster by making used of Numba's just in time compiling functions.

Working on this in the function_arithmetic_operations branch.

The new implementation will include the following features:

Support for callable, spline, akima, linear, polynomial and shepard.
Separate method for spline, akima, linear, polynomial and shepard.
Numba JIT for spline, akima, linear and polynomial.

The use of Numba's JIT make's it much harder to implement the current "remember the last interpolation interval" functionality. However, even without this feature, it can be considerably faster, as can be seen in the following test, where .getValueOpt is the new implementation and .getValueOpt2 is the old one:
>> xData = np.linspace(0, 1, 100)
>> yData = xData - xData**2 + xData**3
>> G = Function(np.array([xData, yData]).T, interpolation='spline')

Comparison evaluating at a single point:
>> %timeit G(0.501)
19.6 µs ± 928 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)

>> %timeit G.getValue(0.501)
19.1 µs ± 1.15 µs per loop (mean ± std. dev. of 7 runs, 10000 loops each)

>> %timeit G.getValueOpt(0.501)
The slowest run took 15.00 times longer than the fastest. This could mean that an intermediate result is being cached.
1.12 µs ± 1.39 µs per loop (mean ± std. dev. of 7 runs, 1 loop each)

>> %timeit G.getValueOpt2(0.501)
7.38 µs ± 87.5 ns per loop (mean ± std. dev. of 7 runs, 100000 loops each)

Comparison taking advantage of "remember the last interpolation interval" functionality:
>> %timeit [G(i) for i in np.linspace(-1, 2, 1000)]
15.1 ms ± 67.2 µs per loop (mean ± std. dev. of 7 runs, 100 loops each)

>> %timeit [G.getValue(i) for i in np.linspace(-1, 2, 1000)]
14.7 ms ± 438 µs per loop (mean ± std. dev. of 7 runs, 100 loops each)

>> %timeit [G.getValueOpt(i) for i in np.linspace(-1, 2, 1000)]
437 µs ± 4.45 µs per loop (mean ± std. dev. of 7 runs, 1000 loops each)

>> %timeit [G.getValueOpt2(i) for i in np.linspace(-1, 2, 1000)]
8.82 ms ± 46.2 µs per loop (mean ± std. dev. of 7 runs, 100 loops each)

In summary:

new Function.getValueOpt is 20x faster than the old implementation when successive points are evaluated.
new Function.getValueOpt is 7x faster than the old implementation when a single point is evaluated many times.

In practice, the new implementation resulted in a 25% speed up in the Flight simulation.


Current thread settings being dropped

https://tweetinvi.codeplex.com/workitem/2659
Verify if this is the case in current version.

NOT AN ISSUE IN <IP_ADDRESS>. The bug was fixed in a previous version of the library


Add count-down for votes

Display to the user's when the proposal ends

This one needed an API update

API is done


Analyzer return exit code 0 even if errors were found

Example of project to recreate issue: https://github.com/ugolova/rembrandt
Steps to recreate issue:

I already have an error in file:
List<String1> brushes = new List<String>(); //String1 classname is error

I run analyzer in strong mode on my project.

Output of analyzer:
C:\Program Files\Dart\dart-sdk\bin\dartanalyzer.bat --strong --format=machine lib/ ERROR|STATIC_TYPE_WARNING|NON_TYPE_AS_TYPE_ARGUMENT|C:\\Users\\yulii.lyubeznikov\\WebstormProjects\\rembrandt\\lib\\painter_rembrandt.dart|8|10|7|The name 'String1' is not a type and cannot be used as a parameterized type

Analyzer error code is 0

It is not correct, because if analyzer find error, it should exits with code 3.
Because of that there is no exception generated. So, I couldn't detect on my continious integration server in my other big project while building process, that there are an analyzer errors. So my code may become not so good and pure. It is a really important for me, please fix it ASAP.
Thanks.
P.S. I added grind task to project, so I could run analyzer easily, just write into console 'pub run grinder analyze'

I can (no longer) reproduce an exit code of 0 from dartanalyzer.bat when there are failures.


Increase the Number of Posts Per Page to 20 (or more)

It would be best if the user didn't have to click on a button to see the next 5 posts. Ideally, if they just continue scrolling down to see older posts, that's best. Otherwise, if there were a simple button to "see more" or "more posts" that would then load the next 10 or 20 posts. This is the ideal because people don't want to click onto a new page and scrolling down for more is how Facebook has programmed us to behave online.

Fixed: https://github.com/zkSNACKs/WasabiBlog/commit/4e6a56c1549257e6120ddbdf2bab240d360524e3

modified


Add test-utils module

Summary:

ported Marathon integration test classes. Most notably MesosTest, ZookeeperServer and MesosFacade and their dependencies
ported MesosClientIntegrationTest into the mesos-client project. It depends on the above-mentioned test classes and starts Mesos/Zookeeper to run its tests.

JIRA: DCOS-47483

I tried to merge master and apply scalafmtAll but it did not work. Any ideas why @alenkacz?


Feature/variable monitoring messages

Describe your changes
Handling of monitoring messages with dispatch to the device model.
Issue ticket number and link
Depends on: https://github.com/EVerest/libocpp/pull/653
Issue: https://github.com/EVerest/libocpp/issues/640
Checklist before requesting a review

[x] I have performed a self-review of my code
[ ] I have made corresponding changes to the documentation
[ ] If OCPP 2.0.1: I have updated the OCPP 2.0.1 status document
[x] I read the contribution documentation and made sure that my changes meet its requirements

All comments have been implemented.
As a note, due to heavy modifications, almost all the modifications in the production-preparation branch probably have to be re-done (a rebase would be too difficult with so many divergent implementation parts): https://github.com/EVerest/libocpp/tree/feature/variable-monitoring-messages-notifications
My suggestion would be to freeze other spec-based fixes, and move them to the final production branch.


Tibia 11 and Tibialyzer 1.4.0-beta

Hello, I've got a problem using Tibia 11 and the newest tibialyzer. So the problem is that it works for awhile(usually 5-10 mins) and then the scanning bar turns gray and tibialyzer stops scanning. I tried it with Tibia 10 and worked properly.

I've same problem, it seems to stops randomly, sometimes it works 5min, sometimes even 1h. It will stop to scan sooner or later anyway.

This was actually already fixed in 3a683fee6d14271fe6d3777f6d4674d0a7fd22cf; I updated the release to include this fix now. You can find the new release here.


iOS (click) firing twice

Ionic version:  (check one with "x")
[ ] 1.x
[X] 2.x
I'm submitting a ...  (check one with "x")
[X] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
As title.
Expected behavior:
Single click...
Steps to reproduce:
Run on iOS device
-->
Related code:
<button ion-button (click)="click()">click</button>
  click(){
    console.log("click")
  }

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):

Cordova CLI: 6.5.0
Ionic Framework Version: 2.0.1
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.1.2
ios-deploy version: 1.9.0
ios-sim version: 5.0.8
OS: macOS Sierra
Node Version: v7.2.0

Xcode version: Xcode 8.2.1 Build version 8C1002

Hello! While we appreciate you opening an issue with us i am going to close this issue and ask that you open a new one that has the issue template filled out please. Thanks for using Ionic!


Update Italian Translation

Missing strings (null) were added and incorrect strings were corrected (syntactically and orthographically)

@pmazz can you take a look?

@pmazz can you take a look?

Hi @MrBartusek, translated content looks good to me (apart from a couple I found unclear).
Pls consider I don't use the card anymore (sorry) so that's all I can do.
:smiley:

Thank you @SiriosDev @pmazz


I was able to bypass the app's security system, but the app eventually crashed.

Describe the bug
Esteem mobile requires an internet connection for it to launch properly. Without an internet connection, the app won't start up. I'd always get the no internet connection message, and the app would just be stuck on the boot screen until I turn on my internet connection or exit the app. The security system also works with an internet connection.  However, I discovered that there was a way to beat this system. It took some level of precision but I could do it anytime I wanted. Kudos, to the dev team though. Although I was able to bypass the lock system, the security of my account wasn't compromised. I got into the app and my account wasn't even loaded. However, I could change some settings. I don't know, but, someone with a higher level of expertise might find a way to get into my account, but, I can't say for certain.
Well, after I got into the app, and navigated around some menus, the app crashed. It took about 1-3 minutes for the app to crash.
To Reproduce
Like I described above, it takes a certain level of precision to bypass the security system.

The first thing to do is; access the app with your internet connection active.
When the app is starting up, there's a small space inbetween where I can access the app. (Like as though the app was open). In that space of time, disable your internet connection.
Since the security system of the app works with internet, turning of the internet means the protocol won't come up.
So, now you have access to the app, but, nothing would work, and I expected that behaviour.
But after some time navigating through menus on the app that can be accessed and trying to perform some actions, the app would crash. I don't know if it is a failsafe system.

Please you have to be on the app for at least 3 minutes for the crash to occur
Expected behavior
I was unable to use the app, even though I had bypassed the security system. But, a crash wasn't expected. I expected that the app would be redundant until I close it manually.
Screenshots or Video
https://youtu.be/oDJuV_C_8lc
Test Environment

Device: Redmi Note 5 Pro
OS: Android 8.1
Software version: Esteem mobile v2.0.9

Logcat
	at com.facebook.react.bridge.ReadableNativeMap.checkInstance(ReadableNativeMap.java:148)
	at com.facebook.react.bridge.ReadableNativeMap.getNullableValue(ReadableNativeMap.java:140)
	at com.facebook.react.bridge.ReadableNativeMap.getString(ReadableNativeMap.java:190)
	at com.facebook.react.modules.dialog.DialogModule.showAlert(DialogModule.java:232)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.facebook.react.bridge.JavaMethodWrapper.invoke(JavaMethodWrapper.java:372)
	at com.facebook.react.bridge.JavaModuleWrapper.invoke(JavaModuleWrapper.java:160)
	at com.facebook.react.bridge.queue.NativeRunnable.run(Native Method)
	at android.os.Handler.handleCallback(Handler.java:794)
	at android.os.Handler.dispatchMessage(Handler.java:99)
	at com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage(MessageQueueThreadHandler.java:29)
	at android.os.Looper.loop(Looper.java:176)
	at com.facebook.react.bridge.queue.MessageQueueThreadImpl$3.run(MessageQueueThreadImpl.java:192)
	at java.lang.Thread.run(Thread.java:764)

Additional context
I have often talked about scrapping the boot screen. It could be there after the first launch, but at subsequent times, it would be great to just get through to the security system of the app. The security system also, is a thing that needs to be optional.

The internet connectivity feature temporarily disabled. This neither a hack neither an unexpected behavior your account already dos not loaded as I saw the record. Also our pincode system not working with internet or something when you logged in successfully the app will ask your pincode until that time you can not share any item of that. if you share somthing from your account with out pincode in first opening that will be a hack. Also the hack not kind ot thing.

Oh okay

The crash was expected too?


The 'docker-machine active' command should have a --timeout flag

I sometimes get "No active host found" when running the command docker-machine active, even though there is an active host. I believe that is because of a timeout after 10 s. I think that the active command needs a --timeout flag, just like the one for ls.
How to reproduce:
I have 6 machines on AWS and one virtualbox, I can reproduce this issue quite easily. When the command take less than 10 seconds, it is successful. When it takes more than 10 seconds, it sometimes fail. (I have changed line breaks below, and removed "user" and "sys" time for readability)
# eval $(docker-machine env aws-swarm-node-3)
#for i in {1..10}; do time docker-machine active; done
No active host found
real	0m10.127s

aws-swarm-node-3
real	0m5.710s

aws-swarm-node-3
real	0m5.749s

aws-swarm-node-3
real	0m5.802s

aws-swarm-node-3
real	0m5.564s

No active host found
real	0m10.125s

aws-swarm-node-3
real	0m6.753s

aws-swarm-node-3
real	0m10.134s

aws-swarm-node-3
real	0m5.773s

aws-swarm-node-3
real	0m10.139s

ls has --timeout, maybe it should be a global-level flag instead


Automate website staging of in-progress PRs

Currently travis.sh deploys automatically to flutter.io when a branch is pushed to master:
if [ "$TRAVIS_EVENT_TYPE" = "push" ] && [ "$TRAVIS_BRANCH" = "master" ]; then
  # Deploy pushes to master to Firebase hosting.
  # TODO: deploy to a personal staging site, based on github ID, when not
  #       merging into master
  echo "Deploying to Firebase."
  npm install --global<EMAIL_ADDRESS>  firebase -P sweltering-fire-2088 --token "$FIREBASE_TOKEN" deploy

(which is great, BTW)
This issue describes and tracks the TODO, which has become a high priority. This feature will allow us to greatly scale our doc efforts internally and externally as we push toward I/O.
We'd like to:

Automatically stage for each branch where a PR has commits and has not yet been merged into master.
Stage on each successful Travis run and display a URL (on success). If that is too often, we can discuss a plan B.
Could we create a new instance for each PR/branch, rather than for each user? When a user deletes their branch, that would be the trigger to automatically clean up and delete the staging deployment. Also, this means the user would not be overwriting their own staged builds when multiple PRs are in-progress...

Thanks! This is very helpful.
I see: I was using "staging" mean "a place before production", and it looks like you were using "staging" as "deployed". Ah, pesky semantics :)
yes, we can help improve this for you :)

We were on the same page semantically - today's workflow was not typical :-)
Sorry for the turbulence.
Happy to help test/debug this feature, if needed.

How many staging areas do we need? Would one per developer be enough (i.e., on every PR, we deploy to something like https://flutter-website-staging.firebaseapp.com/mit-mit/). That would mean that you can always only see the staging content for the most recent PR you have uploaded.
I'm asking as if not we would need to find some way of cleaning up older staging builds, and then it quickly becomes complicated.

I'm asking as if not we would need to find some way of cleaning up older staging builds, and then it quickly becomes complicated.

Yeah, we were talking about this offline. It's a shared concern.

I can take on this task -
Let me know any specific requirements, what I can gather is basically

Someone sends a PR
Create a firebase staging based on that PR, deploy the PR if the build passes
Update the PR on github with a URL to the staging link

Thanks! I added a few items.

Great sounds good, I'll start on this tonight or tomorrow and have something shortly. Will keep this updated

Very cool. Let me know what permissions you need, if any.

@mit-mit @sethladd One instance per developer means when I'm working on multiple PRs at once, I'm overwriting my own changes.
In the per-branch or -PR model, we can know to delete the staging build when the branch gets deleted. That's the idea there. No old builds are hanging around, and there's no ambiguity on when it should be cleaned up.

I think having a one per branch model makes sense vs a one per dev model. Def we dont want to override the previous PR stuff

One branch per PR helps developers who work on multiple, concurrent patches to the website.
Branches don't get auto-deleted when a PR is merged, so I think we can look at multiple signals for when to delete/remove the staged code. Some signals: too old (> 15 days is a strawman), branch is deleted, PR is merged.

Agreed

@LarkAscending what's the code name for the customer who is requesting this? (Or should this be in "no milestone necessary"?)

Happy Presidents Day or Family Day!
I've started work on this now. Will update soon

hope everyone had a good day off!
Here is what I'm proposing for this. There are a few issues with what we oriignally wanted, mainly because firebase hosting does not support "diff" or partial updates. If we had a staging firebase deployment it would get overriden each time when someone sends a pr.
https://stackoverflow.com/questions/30677351/upload-single-file-to-firebase-hosting-via-cli-or-other-without-deleting-existin
Which led me to come up with another solution which is to use Heroku for the stage builds.
Thoughts?
Currently i have a simple flow setup with Travis, that creates a folder based on the branch thats in pull request and deploys it when it runs ./doc.sh
See https://github.com/flutter/flutter/pull/8294

I might have confused myself with the issue being created in this repo and being meant for flutter/website.
I thought it was for API docs website which is generated through docs.sh. Either case the change is very small to get it working for the main website, just looking for feedback on the heroku approach and I'll make the change.

No, we don't need this for API docs as the formatting rarely changes there. The real need is for PRs for https://github.com/flutter/website.

Okay got it! So this repo is used for issues for both website and SDK etc?
Will make the changes shortly and open the PR at website.
Thanks!

Apologies for the confusion. Thanks for making this work for the website.

If we can make this work with Google Cloud Storage and their hosting of static docs, that would be preferable. But, we're flexible.

Hey, that's a good idea, okay let me investigate Google cloud storage

@FaisalAbid Not your fault - it's confusing. All the issues are with flutter/flutter, and this started w/an issue.
It's the Label dev: docs- website that differentiates.

Hey guys, update on this issue. I got it working with firebase deploy, we create a few firebase projects and cycle over them, deploy, check old branches, all that fun stuff!
We even have a PR Bot that updates the pull request with the link to the staged project.
I'm cleaning up and testing, hope to send a PR by end of day today if all goes well.

We have turned this on. Woohoo!

Travis website builds are failing for #431 and #432.
Error: Command requires authentication, please run firebase login

upstream error:
Unauthorized request.
#0      FirebaseClient.send (package:firebase/firebase_io.dart:95:11)

Could this be a dependency issue? Neither @cbracken nor @LarkAscending has the dependencies installed for FB CLI.

Taking a look, thats bizzare

Declaring success!
Thanks @FaisalAbid !!


[INFRA] Transpose the entity table and link to text anchors describing each entity

closes #200

transposing done via: http://harttle.land/invert-markdown-table/
linking will probably work via html code

html code is ignored during the rendering
I can link to any word using this syntax: this is my anchor
... and then I need to refer to that anchor with this syntax: this is my link
see example in pure html

--> linking does not work as easily as expected with the strikethrough html code I proposed above. Perhaps trying to go the "expected" way and link to headings is easier. We could make a low-in-the-hierarchy heading for each entity once it's being introduced in text.
EDIT: The situation is a bit more complicated than I anticipated. For the first time in quite a while, I read the specification consecutively, and it turns out that the key-value pairs (like sub-)are never introduced explicitly or in their own section. They are fed to the reader bit by bit and it is assumed that understanding of the key-value pairs somehow "emerges".
Without a central place describing the key-value-pairs (or "entities" as we also seem to call them), it will become hard to link to specific "entities". For example, the acq parameter is mentioned several times ... and takes on a special meaning depending on the modality.
Suggestion
I think we could

add an explicit section introducing the key-value pairs (and relating them to the word "entities" and the "entity table" in the appendix)
add another section where each entity has a list of items ... and each item links to an in-text description (within the spec), where it is described what meaning that specific entity takes on in that context.

Example for point 2.

Meaning and usage of aqc

aqc in Anatomy Imaging Data
acq in Task Imaging Data
acq in ...

... and in the entity table, we would link from an entity to the list of items (as shown in the example)

I am not sure that the transposed format of the entity table looks better, see here: https://777-150465237-gh.circle-artifacts.com/0/home/circleci/project/site/99-appendices/04-entity-table.html
Any opinions? cc @choldgraf @effigies @franklin-feingold

FWIW
both are equivalently too big ATM, we need to figure it a way to freeze leading column/row ...
Ability to link to specific modality would be a useful addition IMHO. Also I like that in transposed version all key-value pairs are listed horizontally, as if they appeared in a file name - easier to relate.

we need to figure it a way to freeze leading column/row ...

I absolutely agree ... I spent some minutes trying to find something: No luck so far.
But I take your comment as a vote that

transposing
links from entities table to their first in-text reference

would be an improvement.

@yarikoptic it seems like we'd have to write custom CSS to get the sticky first row and column, see: https://github.com/squidfunk/mkdocs-material/issues/1171

Thanks! Sticky header seems indeed simple https://codepen.io/tfzvang/pen/WQBwVo
With transposition it will make it easy to align desired to of the file to the key-value pairs

seems good to me!

Thank you @sappelhoff !
+1 the transposed is looking good and as Yarik said I think it makes it easier to relate and map onto the filenames. Linking from the modality in the entity table to where it is described in the text should help connect the two. Linking to the modality will bring the user to the section they are interested in, perhaps not needing to go into each key (e.g. acq)
As you mentioned Stefan - we don't explicitly define the key-value pairs. The closest is in the context of describing the json file format (https://bids-specification.readthedocs.io/en/latest/02-common-principles.html#keyvalue-files-dictionaries). The common principles section would be the best location. Perhaps right around the section I linked?
What do we think?

I agree with @franklin-feingold  and I have prepared a draft with a short intro.

short section: https://828-150465237-gh.circle-artifacts.com/0/home/circleci/project/site/02-common-principles.html#file-names-consist-of-key-value-pairs-entities
entity table: https://828-150465237-gh.circle-artifacts.com/0/home/circleci/project/site/99-appendices/04-entity-table.html

A new round of reviews would be appreciated. I think this is an acceptable improvement

minor comment on sub-label
but I wonder -- so there is no any kind of ids for the modalities in the table -- it doesn't work at all?

@yarikoptic Could you reword this? I don't understand what you're trying to say.

Two very small cleanups, and then I'm good with this.

cool thanks.
Then once this is merged, I'll open a new issue to track the next enhancements:

Make the first row and first column "sticky" using custom CSS code
Potentially separate the entity table into several sub-tables (one per modality)
this would also make it easier to insert links from an entity in the table to an in-text description of what that entity means in the context of a certain modality
Address some inconsistencies in labelling in the entity table, see https://github.com/bids-standard/bids-specification/pull/272#discussion_r307433924 for reference

I would keep all of these separate issues, because that makes them more attractive to tackle. WDYT?
Finally, @effigies could you open an issue related to "each sentence on its own line" (see: https://github.com/bids-standard/bids-specification/pull/272#discussion_r307318622), please?

I think we can merge this. @yarikoptic's concerns will be addressed in a future PR (I'll open an issue upon merge of this PR). @franklin-feingold ... perhaps you want to give it one more review pass?

@yarikoptic Any remaining qualms?


Unable to edit routes using Firefox 128.0 on MacOS

If I create a new file in Firefox 128.0 on MacOS, the pencil tool is automatically enabled. However, the cursor on the map is still a hand and clicking on the map can only scroll it. Trying to add nodes to the route just doesn't work.
If I do the exact same thing on Safari, editing routes works just fine.

Can you check if there are any errors in the Javascript console?
I am changing things frequently at the moment so it could cause errors from one version to another.
You could verify this by clearing all browser data for gpx.studio and reloading.

It seems to be working today.

Thanks, I am closing the issue but don't hesitate to comment on this issue if you encounter this problem again.

Error: The layer 'gpx-1-direction' does not exist in the map's style.

That should be fixed now.


Pages are Sorting by Themselves

Major issue. Not sure when it started but most everything is now out of order. It seems like BSA has decided that I don't know how I want my pages sorted, and they're being sorted by some sort of haphazard last updated / created date. In some books / chapters they remain sorted as I originally did them, but on others they're being sorted in reverse order ( so Ch 10 is at the top, then 9, then 8, etc ). Still others are a completely random order, but I'm willing to bet it's not so much 'random' as 'last updated' but without an easy way to see when each item ( instead of just the last item ) was updated I cannot confirm that ( One of these book's pages is currently sorted 5,6,4,3,1,2,0 - that makes no sense unless it's being sorted by last update maybe? I can't tell ).
I don't see any option in settings to do an automatic resort. They were definitely correct before, but somehow / someway have gotten fubar'd.

So took a look in the database. A lot of items seem to have the same priority. Is this how bookstack determines sorting? So if I never manually sorted something, it won't have a priority and instead will use the created / updated dates to sort the pages in a chapter / book?

Here's a picture btw. I started fixing everything manually in the meantime ( as noticed by the first one being correct ). You can see how they're sorted; originally they were sorted "correctly" top down 0->1.

Hi @armouredking, Sorry to hear that you're having issues with your BookStack instance.
Your issue did point point out an issue with pages that are created within chapters which I have fixed on commit afc3583 which will be in the next release (Most likely published this weekend).
I'm sorry about the change in ordering, I have absolutely no idea what could have caused this. You are correct that everything is sorted via the priority (Except for draft pages which are forced a higher priority). Unfortunately, Due to the bug mentioned above, most of your pages would have likely had the same priority due to being inside chapters.  There is no secondary sort so they should then go in database order (Which would match order of creation). Do you know if any changes were made during this time? Perhaps a system database update? or did you update your version of BookStack recently?
Instead of manually fixing things you could fix this via MySQL using this command:
SET @i := 0;
UPDATE bookstack.pages SET priority = @i:=@i+1 ORDER BY name;

This will set the order of all pages in the system to be ordered alphabetically. You will have to update bookstack to your database name if different and set a where clause if you want to restrict the update to a single book. Try it on a test table first just in case of problems but I ran it on my test BookStack instance without any issues.

Since a large part of what caused this issue has now been fixed in v0.9 I will close this issue. I will look further if you're able to answer any of the questions in my previous comment.

The only recent update was from 0.8.1 to 0.8.2. I don't recall it saying it needed to make any changes to the database though when I ran the update line. I haven't updated to 0.9 yet but I'll give it a go on the next off work period.
Based on how I looked at it comparing the front end to the database, anything I had manually sorted was in correct order whenever this happened. I had to do manual sorting in the past because sometimes things would 'save' to the wrong spot. Like I copy paste into a page, then the next one, and somehow the more recent one would wind up above it in the order. Quick manual sort fixed that issue in the past and it was only a minor thing and I couldn't make it occur it just happened on and off thus never reported.
A manual MySQL command was not acceptable; while the picture shows everything with nice neat titles a majority of the stories aren't named this way. They have actual titles and thus need to be in a preserved arbitrary / manual order; the sort of setup as pictured is basically reserved for plot work - in drafting stories privately ( I run two BSA, a private and a public because initially it didn't have the different viewing options / user preference list and could only be public or private; I might combine them later if I can find an easy way to set it up ) I use BSA like an online version of Scrivener. It works well in that kind of scenario and Scrivener still doesn't have an acceptable mobile version or something I can use at work on breaks.
With regards to chapters by title, this was initially accomplished by creation date - whenever the resorting error occurred, these stories were still in order ( for the most part they didn't have edits as they'd already gone through that process ) they were just reversed. IE: The most recent page would be at the top instead of the bottom of the list. Not good for sequential story reading.
Anything that wasn't manually sorted previously, IE the sorting was just done based on when I created it, then they were resorted whenever this happened. They were sorted based on last update date, as far as I could tell; and the picture above attests to this - most things were in reverse order, but some items are all mixed up if additional proofreading was required because then they were updated more recently than the original sections were posted.


Add an if-condition to RViz delay

This prevents RViz from spawning twice if the moveit startup is used.
Closes #17

@Mergifyio backport humble


Define Typal Universal properties for CT Structures

See here for how to define a typal universal property should work for products and exponentials.

[ ] Product and unit types (easiest)
[ ] Function types
[ ] Sum and empty types (most complex)

No longer a priority


syncthing: 1.16.0 -> 1.17.0

Motivation for this change
https://github.com/syncthing/syncthing/releases/tag/v1.17.0
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 125439 at a2a06c24 run on aarch64-linux 1

5 packages built successfully:

qsyncthingtray
syncthing
syncthing-discovery
syncthing-gtk
syncthing-relay


[S001162] Rep. Allyson Schwartz

Rep. Allyson Schwartz:
http://www.house.gov/schwartz
Likely contact form URL to start with:
http://schwartz.house.gov/serving-you/email-me

YAML file lost. I am working on it.


Implement Hand Post Processes

In the Hand Pool, you'll now see a Unity Event called "Hand Post Processes" where you can add functions that will modify that hand group right before it gets displayed, without modifying the frame or generating garbage.
How is it implemented?   Sneakily...
An Example Hand Post Process function looks like this:
 public void setHeldHand(Hand inHand) {
    inHand.SetTransform(transform.position, transform.rotation);
  }

[x] Verify that it doesn't generate garbage
[x] Verify that there aren't side-effects
[x] Verify that its totally awesome

I'm not going to list it as a blocker to merging this PR, but just making a note that we are absolutely going to want a way to do a post process on multiple models at the same time without doing the post process multiple times, since copying a hand is so expensive.

Alex's comments are addressed in this next PR I have up.


Language-specific guides request

ruby guide

Thanks for requesting; let me close this as duplicate of https://github.com/docker/docker.github.io/issues/13052


Add buttons to set framework agreements as 'on hold' or 'countersigned'

For this story on Pivotal.
Also for this bug on Pivotal.
This PR adds couple of POST routes to the admin frontend which hit the API and timestamp the relevant framework_agreement to show when they were countersigned or put on hold.
If countersigned some details are also stored about who legally countersigned as well as who actually hit the button.
STILL TO DO:
Currently, the post routes return the user to the framework agreement they were working on. This is not the expected behaviour. It should take the user to the next framework agreement that needs countersigning/putting on hold. The logic for getting this next agreement is a separate piece of work outside the scope of this story.
Will need this PR on the toolkit to be merged before the tests will go green

@TheDoubleK All done!

@TheDoubleK All those changes should be in!

Good stuff.  Let's get this in once Travis is happy (needs the frontend toolkit to get merged first I think).
:+1:


Update github.com/creachadair/jrpc2 to v0.19.0.

In preparation for committing to a stable v1 release for jrpc2 package, I have
made a number of updates to the library to simplify the API a bit. These
changes are discussed in https://github.com/creachadair/jrpc2/issues/46 if you
would like to offer any input.
This commit updates the package version used here to the latest tag as of this
writing, v0.19.0, and updates the existing use to match the changes. These
changes should not produce any functional differences.

Hey!
Thank you so much 🙏 🎉 I like the new API, if I have any observations or suggestions I'll make sure to let you know!


#43 Updating dependencies and adding utility meth…

…ods.

Codecov Report

Merging #44 (0a99c5c) into main (14c4743) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #44   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           15        16    +1
  Lines          883       921   +38
  Branches       149       159   +10
=========================================
+ Hits           883       921   +38

Flag
Coverage Δ

data
100.00% <100.00%> (ø)

data-backend
100.00% <ø> (ø)

data-frontend
100.00% <ø> (ø)

metadata-jsonschema
100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

data/src/methods.ts
100.00% <100.00%> (ø)

metadata-jsonschema/src/transform.ts
100.00% <100.00%> (ø)


Configuration to specify download URL

Is your feature request related to a problem? Please describe.
It is a feature request not related to any existing problems.
nvim-lsp-installer tries to download a LSP server always from https://github.com. However, the access to github.com may be slow in some region due to various reasons. As a result, a lot of people there take advantage of a mirror site (e.g. https://hub.fastgit.org/) to get around the issue.
Describe the solution you'd like
It would be nice to include a feature in the plugin so that we could customize the download URL.
Describe potential alternatives you've considered
No response
Additional context
Similar feature that has been implemented for wbthomason/packer.nvim:
https://github.com/wbthomason/packer.nvim/issues/433

If you  open nvim-lsp-installer/lua/nvim-lsp-installer/core/managers/github/, and there will be init.lua and client.lua two files.
If you want to use the mirror for downloading, just editing init.lua in line 62:
local download_url = ("https://github.com/%s/releases/download/%s/%s"):format(opts.repo, release, asset_file)

Hope it is helpful to you.

This definitely makes sense to support! Does branch feat/github-download-url-setting (https://github.com/williamboman/nvim-lsp-installer/pull/711) fix this?

I think it would be better to add such a function.
Also, you can write a simple guide in README.md to tell people how to use the mirror sites lol.

I think it would be better to add such a function. Also, you can write a simple guide in README.md to tell people how to use the mirror sites lol.

I just see the branch, and I think using a template string is definitely ok.

This definitely makes sense to support! Does branch feat/github-download-url-setting (#711) fix this?
I'm not sure if using a template string for this is too restrictive, what do you think? Should it be a function instead perhaps?

A template string would be enough to start with. If people find it too restrictive, we can try to figure out a better way later on.

:+1:

Also, you can write a simple guide in README.md to tell people how to use the mirror sites lol.

I added a helptag :h nvim-lsp-installer-download-mirrors


Update Go version to 1.18

Changes
Finishing touch to align with Knative components.
/cc @lance @salaboy

:broom: Update Go version to 1.18

/kind

Fixes #

Release Note

NONE

Docs

/lgtm


Issues with Economy

Hi, I'm using InfoBoard for the first time and I'm having some problems.
The plugin is working fine and looks very nice, that is until I try displaying player balance.
I'm using MineConomy with Vault (yes, I have tried other economy plugins as well) and every time I try to display the player's balance with %vault_eco_balance% the InfoBoard simply dissappears.
Here's what my board.yml looks like:
InfoBoard:
  '1':
    ShowTime: 20
    global:
      default:
        Title: '&B&lThe Enchanted Forest'
        Rows:
        - '&6----[InfoBoard]----'
        - ' '
        - '&e&lWelcome!'
        - '&l%player_name%'
        - ' '
        - '&a&lHealth:'
        - '&e%player_health%&a/&c%player_max_health%'
        - ' '
        - '&2&lBalance:'
        - '%vault_eco_balance%'
        - ' '
        - '&e&lWebsite:'
        - '<website>'
        - '&6----[InfoBoard]----'
  '2':
    ShowTime: 10
    global:
      default:
        Title: '&e&lPlayer Stats'
        Rows:
        - '&6----[InfoBoard]----'
        - ' '
        - '&e&lDate Joined:'
        - '&l%player_first_join_date%'
        - ' '
        - '&a&lWorld:'
        - '&e%player_world%'
        - ' '
        - '&5&lLocation:'
        - '&a%player_x%&7, &a%player_y%&7, &a%player_z%'
        - ' '
        - '&e&lWebsite:'
        - '<website>'
        - '&6----[InfoBoard]----'
      Admin:
        Title: '&e&lServer Stats'
        Rows:
        - '&6----[InfoBoard]----'
        - ' '
        - '&a&lMemory:'
        - '&e%server_ram_used%&7/&e%server_ram_max%'
        - ' '
        - '&d&lUptime'
        - '&f%server_uptime%'
        - ' '
        - '&6----[InfoBoard]----'

I'm not quite sure what the problem is, as I've tried redownloading PAPI and I have all the correct extensions downloaded.
Help is greatly appreciated.
Thanks!

[FIXED] I was trying to replace EssentialsEco with another plugin and now I swapped, the problem is fixed.

Ok.
By the things you're telling me, it looks like you had multiple plugins doing economy, and vault was getting conflicts because of that. And IBR just displays what it get's and a conflict in vault is a conflict in IBR.
But it's good that you fixed it! hope you enjoy the plugin!


Add home page component

Add home page component
Add pictures index
Update taliwind config
Add hero section
Add html style
Update hero section
Update Hero Image
Add cards section
Add providers logo section
Add CTA section
Add marquee section

Current dependencies on/for this PR:

main

PR #3

PR #4   👈

This stack of pull requests is managed by Graphite.

Merge activity

Nov 11, 1:23 PM: @williamrolando88 started a stack merge that includes this pull request via Graphite.


We can add custom tags to generated OPS

Codecov Report

Merging #25 (107790d) into main (cb99686) will decrease coverage by 0.20%.
The diff coverage is 97.14%.

@@            Coverage Diff             @@
##             main      #25      +/-   ##
==========================================
- Coverage   92.54%   92.34%   -0.21%
==========================================
  Files          11       11
  Lines         684      705      +21
==========================================
+ Hits          633      651      +18
- Misses         51       54       +3

Impacted Files
Coverage Δ

aiohttp_pydantic/oas/struct.py
90.49% <90.00%> (-0.11%)
:arrow_down:

aiohttp_pydantic/oas/docstring_parser.py
97.75% <100.00%> (-2.25%)
:arrow_down:

aiohttp_pydantic/oas/view.py
94.54% <100.00%> (+0.05%)
:arrow_up:

aiohttp_pydantic/view.py
94.36% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cb99686...107790d. Read the comment docs.


implement createLink for windows

Bump Win32 to first version providing createHardLink
closes https://github.com/jystic/unix-compat/issues/20

👍 looks good


CPLP-968 change tenant-related user endpoints to owncompany/users

implements: https://jira.catena-x.net/browse/CPLP-968

I tested all refactored endpoints successfully except the one for the ownUser deletion where I get an 'Object reference not set to an instance of an object.' exception. Let's have a look at that tomorrow together.

thanks for testing and discovering a bug (which is fixed now) :+1:


symbolator not working with sphinx 4.x

Symbolator is not working with Sphinx 4.x.
>>> import symbolator_sphinx
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.8/site-packages/symbolator_sphinx/__init__.py", line 1, in <module>
    from .symbolator_sphinx import *
  File "/usr/local/lib/python3.8/site-packages/symbolator_sphinx/symbolator_sphinx.py", line 33, in <module>
    from sphinx.util.osutil import ensuredir, ENOENT, EPIPE, EINVAL
ImportError: cannot import name 'ENOENT' from 'sphinx.util.osutil' (/usr/local/lib/python3.8/site-packages/sphinx/util/osutil.py)

It should be possible to fix this error.
The line in: https://github.com/kevinpt/symbolator/blob/master/symbolator_sphinx/symbolator_sphinx.py#L33
must be changed to: from errno import ENOENT, EPIPE, EINVAL
https://github.com/missinglinkelectronics/sphinxcontrib-svg2pdfconverter/commit/252a0b0fa097686dae84e8d7c39f901dc010a531

Maybe the fork https://github.com/hdl/symbolator can help you.


Fix for links

Docs Build status updates of commit e41b446:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-nav/Deploying-Microsoft-Dynamics-NAV-Using-ClickOnce.md
:bulb:Suggestion

Details

dynamics-nav/Walkthrough--Implementing-Security-Certificates-in-a-Test-Environment.md
:bulb:Suggestion

Details

dynamics-nav/deprecated-fields.md
:white_check_mark:Succeeded

dynamics-nav/fin-graph/api/dynamics_account_get.md
:white_check_mark:Succeeded

dynamics-nav/how-to-view-synchronization-job-queue-status.md
:white_check_mark:Succeeded

dynamics-nav/Deploying-Microsoft-Dynamics-NAV-Using-ClickOnce.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

dynamics-nav/Walkthrough--Implementing-Security-Certificates-in-a-Test-Environment.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


HDDS-6342. EC: Fix large write with multiple stripes upon stripe failure.

What changes were proposed in this pull request?
Fix large write with multiple stripes upon stripe failure.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6342
How was this patch tested?
a new ut covers this case.
manual test with ockg.

Thanks a lot @guihecheng for the contribution.


Adjust Regex Changes also to the Part2 Classes

As discussed in https://github.com/admin-shell-io/aas-specs/issues/362, the 'old' regex pattern ("^[\\x09\\x0A\\x0D\\x20-\\uD7FF\\uE000-\\uFFFD\\U00010000-\\U0010FFFF]*$") shall not be used in the schema files but replaced with one that is better suited to UTF-16-compliant validators (^([\\t\\n\\r -\ud7ff\ue000-\ufffd]|\\ud800[\\udc00-\\udfff]|[\\ud801-\\udbfe][\\udc00-\\udfff]|\\udbff[\\udc00-\\udfff])*$).

@g1zzm0 I am not sure if I can follow you... The Part 1 Domain has this regex pattern already since quite some time, e.g. https://github.com/admin-shell-io/aas-specs-api/blob/ed902c9f9da93c3417b291d3cb425334ddae5b76/Part1-MetaModel-Schemas/openapi.yaml#L60
Therefore, I don't think that Part 1 has to implement anything before...


Allow turning off Tracing in FodyWeavers.xml (assembly level)

This two-line change allows  a user to turn off tracing in
FodyWeavers.xml by setting <TraceOn class="none"...
but still enabling trace weaving for specific methods with TraceOn. This
is important for performance-critical assemblies that need a couple of
methods traced. This doesn't affect existing behaviors.

doH!  Will resubmit.


irc3 and celery

hi, I'm executing an irc3 bot inside a celery task. The bot correctly connects to the server but it doesn't auto connect to the channels. And it behaves in a strange way with my plugin (a plugin to print received messages): it works for the first few messages received, then nothing happens.
And it doesn't answer to pings.
If I run my code directly and not inside a celery task it works perfectly.

I don't really have a clue without more informations.  Is your code public ?

Got an idea. Maybe the bot is intanciated in a thread and run in another.
You can try to debug this with:
print(asyncio.get_event_loop() is bot.loop)
Should print True

I think I solved the problem, it was my fault. Thanks for your help!


⚠ Bofrost.be has degraded performance

In f797205, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1600 ms

Resolved: Bofrost.be performance has improved in ca314bb after 13 minutes.


Configurable server address

Allow specifying a custom server address and port.

I'd especially be interested in loading scripts from Github Pages.

Hi @schmijos this feature was now released as part of v2.6.0. You should be able to change the server address via the popup when you click the witch icon in the toolbar. Can be either local or remote.
I am closing the ticket, but please let me know if you have any problems.


Gradle pipeline demo

Hi, we're familiar with the maven pipelines but would love to see a Gradle demo pipeline as well especially utilizing the new ibmint commands :)

Excellent idea :) I shall see how easy it turns out to be . . .


Feature parity between xml vs api vs annotations

Currently have 3 methods of configuring dozer:

xml
api
annotations

Validate that all 3 are equal in terms of features.

Should they be equal? I don't think that annotations can support all variations for example. Because API would become quite messy. I would think in the direction of refactoring Dozer core API in order to support more convenient mappings approaches. Lambda based and Kotlin-based for example. XML API is a pain and no one wants to use it. Annotations are not usable in all cases, because you have to change at least one side of beans being converted.
I think there is a need for Dozer 2018 API out there.

The @Mapping annotation can be enhanced to support the <fields> tag properties like key etc.  I mean configuring the contents of DozerField class.

how about add a convert support for Mapping annotation.
such as
@Mapping(converter=StringToUUIDConverter.class, value="id")
private String uuid;

class StringToUUIDConverter implements Converter {
@Override
Object convert(Object value, Object source, Object target){}
}


catkin_make_isolated error

Hello, thank you for sharing. I ran the project on ubuntu18.04，When I was compiling this project, the following error occurred  :error: no matching function for call to ‘google::protobuf::internal::ArenaStringPtr::GetNoArena(const string*) const’，May I ask how to solve it

Hi
Which branch of the repo are you trying to install?

I had the same problem with python 3.11.
Switch back to python 3.8 solves it.


fix: add explicit imports for useRuntimeConfig

Types of changes

[ ] Chore (a non-breaking change which is related to package maintenance)
[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
This adds some explicit imports for useRuntimeConfig to the plugins.
Checklist:

[ ] My change requires an update to the official documentation.
[x] All TSDoc comments are up-to-date and new ones have been added where necessary.
[x] All new and existing tests are passing.

~> https://github.com/nuxt-modules/prismic/commit/9e9af4b67f5572724e214fa3cb28eb95fed24789

Thanks for helping! ☺