Extracted bundle transfer to do not perform localization

We ship a config.yaml in order to perform transformation on manifest to use specific image location. To should happen everytime  a component is transferred with its underlying resources.
This is not the case when do the following:
maps bootstrap --extract --path /tmp
mpas bootstrap --from-file --registry ghcr.io/souleb/mpas-bootstrap-component

We expect the underlying image resources to be transferred alongside the bundle, but it is not done currently.
For example in the code at https://github.com/open-component-model/MPAS/blob/f73a1054f05d1d9b40fba8062355bcc2358acc33/internal/bootstrap/install_flux.go#L211
image := imagesResources[loc.Resource.Name]

This should retrieve an image that is located in ghcr.io/souleb/mpas-bootstrap-component, but the retrieved image will still points to ghcr.io/fluxcd.
I believe there are some additional steps to be performed when transferring the bundle.

cc @phoban01 cc @Skarlso


SPIKE: Original quote highlighting skips various words

Highlighting is skipping some words sometimes.
Jude 1:2  'kai' should be highlighted
Jude 1:3 'tou'
Jude 1:4 'tou'
====================
Check highlighting on a BPF basis (3John, Titus, Ruth, Jonah)
Document verses that aren't rendering (reference, word)
Determine patterns for broken highlights
Make issues (for each issue write a test to check if the words are highlighted)

Found instances where there are missing highlights in Rut 1:1.

Need to figure out how to determine what should be highlighted.
If all the words are aligned to the UGNT does that mean I can just look at the raw usfm3 to determine this?
@klappy @birchamp @jag3773

Proposed approach:
iteration 1:
Manually check if highlights are correct - line by line, but can't confirm the ULT and UST are correct.

document verses that fail
look for patterns and figure out problems.
create issues for specific bugs.

iteration 2 (needs separate issue):
have Content team check if ULT and UST are correct. They will need to look

3 Jn 1:10 ἐκ

https://create.translationcore.com/

Titus:
1:1 - τῆς`` κατ’ εὐσέβειαν`

@RoyalSix We need to create the associated issues for this so we can add them to this as an Epic/Sprint and estimate them.

κατ, τοῦ and ἐπ- are the most commonly missed highlights. I observed this pattern in other Greek Books like eph.

Should be addressed in beta.6

Verified in beta.6- Titus and 3Jn looks good and highlights in original quotes are fine. Checked Ruth and few other books as well.
Keeping in QA Watch.

Tested in Katalon tCC tN Test Highlighting


Created PR_template

Added missing PR template .

This template is not relevant for this repository. I am closing this PR.
If you would like, you can open another PR and create a PR template that is relevant for this use case
Thank you


Create docs on how to use Qualtrics

I need to write a doc to show how to enable and use Qualtrics for specific or all pages

DOC: https://github.com/SAP/cloud-commerce-spartacus-storefront-docs/issues/325


Removed web auth provider from the release notes

Initially released as a preview but redacted for now.

Docs Build status updates of commit 49f4568:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/release-notes-october-2018.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Simplify readme sample

Just use elastic as a dependency, and leave the more complex example for samples.

Something like this should do:
#[macro_use]
extern crate json_str;
extern crate elastic;

use elastic::prelude::*;

fn main() {
    let client = Client::new(RequestParams::default()).unwrap();

    let req = {
        let body = json_str!({
            query: {
                query_string: {
                    query: "*"
                }
            }
        });

        SearchRequest::for_index("_all", body)
    };

    let res: SearchResponse<Value> = {
        client.request(req)
              .send()
              .and_then(into_response)
              .unwrap()
    };

    for hit in res.hits() {
        println!("{:?}", hit);
    }
}

It'd be nice to find a way around needing that separate json_str dependency. An alternative is to implement IntoBody for serde_json::Map<String, Value> and use that.


grouping api

Are there any API method that i will able to put characters into the group? For example, I am looking to group [0-9] and [~!@#$%^&*()_+] and some custom conditions into one group called # and with the letter I want it to make it to group it like A for Aa, B for Bb and C for Cc and so forth.

you can group your items in
@override public long getHeaderId(int position) { ... }

for example, if you got 2 groups in your list. you can just try
if( somecondition ) return  1;
else return 2;

and then try to draw your group view at this method
@Override public View getHeaderView(int i, View view, ViewGroup viewGroup)


When running Dockstore locally UI sometimes won't load properly

Sometimes when you are running Dockstore locally and you load a page, the page will load but will not show any content. Refreshing the page typically fixes this. I have only seen this on a local instance so far, and not on staging or prod.

This might be a race condition

I think the issue here is that the child directives load before the http request to get workflows and tools (and tokens, etc) returns, thus they are empty.
I think this can be fixed by using $scope.$watch on the array of tools, workflows, etc from the child directives, and update their own content based on changing value of the array of interest.

This fix has been implemented on the search pages as well as the token page.

This issue only occurred for usages of a table (search pages and tokens), which have been fixed.


fix: preserve formatting for JS files

fixes #464
locations need to be stripped in order for node comparisons to work properly, see: https://github.com/svelte-add/svelte-add/pull/379#discussion_r1638361267
rather than stripping the ast locations from the original tree, we'll now create a clone of the tree and strip it from there during comparisons.

notable observation: it doesn't seem to be perfect either, but I'm not sure why.
for instance, here's the before:

and here's the after:

however, if I format it using tabs and run the adder again, the tabs are entirely preserved... so I'm not quite sure what's going on there 😅

cool! but yeah, we should probably see if we can get it to be all tabs vs a mix of spaces and tabs

cool! but yeah, we should probably see if we can get it to be all tabs vs a mix of spaces and tabs

playing with it a bit, it looks like if there's no trailing commas (which is how create-svelte applies it), it goes rogue and starts using a mix n' match 🤔

ehhh, I'm not so sure about this PR anymore. I can't get recastPrint to comply at all, even if I force it to use tabs, it still inserts spaces when updating existing files:
/** @type {import('@sveltejs/kit').Config} */
const config = {
    // Consult https://kit.svelte.dev/docs/integrations#preprocessors
	// for more information about preprocessors
	preprocess: [vitePreprocess(), mdsvex()],

    kit: {
		// adapter-auto only supports some environments, see https://kit.svelte.dev/docs/adapter-auto for a list.
		// If your environment is not supported, or you settled on a specific environment, switch out the adapter.
		// See https://kit.svelte.dev/docs/adapters for more information about adapters.
		adapter: adapter()
	},

    extensions: [".svelte", ".svx"]
};

export default config;

Additionally, if we take into account the formatting for other adders, bringing back the location field makes it even worse for newly created files.
consider the following case:
pnpm dlx https://pkg.pr.new/svelte-add/svelte-add@467 drizzle tailwindcss --path ./temp

if you take a peek at the generated ts modules from this PR, it's pretty bad:
// db/index.ts
import { drizzle } from "drizzle-orm/better-sqlite3";
import Database from "better-sqlite3";
import { env } from "$env/dynamic/private";
if (!env.DATABASE_URL) throw new Error("DATABASE_URL is not set");
const client = new Database(env.DATABASE_URL);
export const db = drizzle(client);

// schema.ts
import { sqliteTable, text, integer } from "drizzle-orm/sqlite-core";

export const user = sqliteTable('user', {
                        id: integer('id').primaryKey(),
                        name: text('name').notNull(),
                        age: integer('age')
                    });

// drizzle.config.ts
import { defineConfig } from "drizzle-kit";
if (!process.env.DATABASE_URL) throw new Error('DATABASE_URL is not set');

export default defineConfig({
    schema: './src/lib/server/db/schema.ts',
    dialect: 'sqlite',

    dbCredentials: {
        url: process.env.DATABASE_URL,

    },
    verbose: true,
    strict: true
});

here's how they currently appear in main:
pnpm dlx svelte-add@latest drizzle tailwindcss --path ./temp

// db/index.ts
import { drizzle } from "drizzle-orm/better-sqlite3";
import Database from "better-sqlite3";
import { env } from "$env/dynamic/private";

if (!env.DATABASE_URL)
    throw new Error("DATABASE_URL is not set");

const client = new Database(env.DATABASE_URL);
export const db = drizzle(client);

// schema.ts
import { sqliteTable, text, integer } from "drizzle-orm/sqlite-core";

export const user = sqliteTable("user", {
    id: integer("id").primaryKey(),
    name: text("name").notNull(),
    age: integer("age")
});

// drizzle.config.ts
import { pathToFileURL } from "node:url";
import { defineConfig } from "drizzle-kit";

if (!process.env.DATABASE_URL)
    throw new Error("DATABASE_URL is not set");

export default defineConfig({
    schema: "./src/lib/server/db/schema.ts",
    dialect: "sqlite",

    dbCredentials: {
        url: process.env.DATABASE_URL
    },

    verbose: true,
    strict: true
});

I could try to account for both by stripping the loc field for files that don't already exist, but I'm not sure if the added complexity is really worth it, especially when keeping the loc field doesn't even fully work.
I feel like if users care about the formatting, they should really be using a formatter

But it must have something to do with our tooling. I made a quick test-script and tried around with the three different files you provided, and alle the tabs / spaces and indentation is preserved as it was before, even if I decided to use some unexpected indentations:

Here is the test script:
#!/usr/bin/env node

import { parseScript, serializeScript } from "@svelte-add/ast-tooling";
import { readFile, writeFile } from "fs/promises";

async function stuff() {
    const inFile = "test-in.js";
    const outFile = "test-out.js";

    const contentBuffer = await readFile(inFile);
    let content = contentBuffer.toString();

    const ast = parseScript(content);
    content = serializeScript(ast);

    await writeFile(outFile, content);
}

stuff();

But it must have something to do with our tooling. I made a quick test-script and tried around with the three different files you provided, and alle the tabs / spaces and indentation is preserved as it was before, even if I decided to use some unexpected indentations:

The indentation issues I was seeing with injected nodes was due to the indentation of those created nodes when parsing from a string to an AST node. The loc property was preserving the extra indents.
For example:
	userSchemaExpression = common.expressionFromString(`pgTable('user', {
		id: serial('id').primaryKey(),
		name: text('name').notNull(),
		age: integer('age'),
	})`);

However, using dedent before parsing fixed it in a separate PR #470, so that's no longer a problem.

The first part of my post regarding injecting property nodes into a preexisting objects is still occurring, where recast's printer refuses to respect the existing indentation. I'll keep messing with it for a bit longer, but the current iteration might be the best we can do for now.
We should also keep in mind that this is only an issue for users who don't have Prettier installed, which a tiny fraction. I'd much rather spend energy elsewhere if we're likely going to replace the recast printer anyway with an acorn alternative

LGTM


Handfunc

Handfunc looks to be working for a general number of cases.

This is the best I can do for now..
Current limitations:

It only can evaluate to local scope, so it will not work inside other functions.
Pluto is not working with @handfunc macro. I am not sure why.


Do not add subviews twice and thus not subscribe to KVO twice

Setup for crashing is following (minimal one I've experienced in my app):

iOS8
One stackView inside another used from inside xib.

Apparently - (instancetype)initWithCoder:(NSCoder *)decoder returns view with subviews added. Even though addSubview: was clever enough to not add subview to stackView twice (removed line), it still had consequences: observers to hidden property (OAStackView+Hiding.h) where added twice which caused crash on master view deallocation.

@m1entus JFYI


⚠ CodePush has degraded performance

In a330a4d, CodePush (https://codepush.appcenter.ms/v0.1/public/codepush/update_check?deployment_key=Fzqu71x4FXhOaKcAiwpqXkFQddjgYl2dIUW2K&app_version=4.4.1) experienced degraded performance:

HTTP code: 200
Response time: 12005 ms

Resolved: CodePush performance has improved in 027b1ae.


How to re read the httprequestdata

I'm using the httprequestdata in my middleware. But if I read the request for once, the http-triggered function just could not read the content. Since the request body is not seekable, how could I reset the request after read it for once so that the function could read it?
My code is like:
using Microsoft.Azure.Functions.Worker;
using Microsoft.Azure.Functions.Worker.Http;
      public async Task<int> FuncName(FunctionContext context)
        {
            var httpRequestData = await context.GetHttpRequestDataAsync();
            await httpRequestData.ReadFromJsonAsync<XXXRequest>();
            httpRequestData.Body.Position = 0;

You might be able to seek with the body:
httpRequestData.Body.Seek(0, SeekOrigin.Begin);

I have the same issue in custom Middleware of a .NET 8 Isolated Azure Function. CanSeek is false on the Body stream of HttpRequestData. Trying to Seek as suggested results in a System.NotSupportedException: Specified method is not supported. Same when trying to set the Position of the stream to 0. Related: #1636

@FilipBruins I ran into this problem when trying to implement HMAC authentication as functions middleware. I could not find a decent solution so I had to get "creative". I hope for a better implementation in the future but this might help you
public static async Task<string> SignRequestAsync(HttpRequestData requestData, string requestTimestamp, string key)
{
    string method = requestData.Method;
    string path = requestData.Url.AbsolutePath;
    string content = await requestData.ReadAsStringAsync();
    var contentBytes = Encoding.UTF8.GetBytes(content);
    var copiedStream = new MemoryStream(contentBytes);

    // Reading from the requestData means that other parts of the processing chain cannot, hence we have to copy it and use reflection to reset it.
    var httpRequestField = requestData.GetType().GetField("_httpRequest", System.Reflection.BindingFlags.Instance | System.Reflection.BindingFlags.NonPublic);
    var httpRequest = httpRequestField.GetValue(requestData) as HttpRequest;
    httpRequest.Body = copiedStream;

    using var hash = SHA256.Create();
    var hashedContentBytes = hash.ComputeHash(contentBytes);
    var hasedContentBase64 = Convert.ToBase64String(hashedContentBytes);

    var hmacSha256 = new HMACSHA256 { Key = Convert.FromBase64String(key) };
    var payload = $"{method}\n${path}\n${requestTimestamp}\n${hasedContentBase64}";
    var sigBytes = hmacSha256.ComputeHash(Encoding.UTF8.GetBytes(payload));
    return Convert.ToBase64String(sigBytes);
}


Creation of unnamed environments

Expected behaviour
Should prevent the creation of unnamed environments.
Actual behaviour
Button to create is greyed, but clicking it still allows the creation of an unnamed environment.
The created unnamed environment cannot be removed through the UI.
Steps to reproduce the behaviour
Create an environment without naming it, ignoring greying of button.
Version of CAIRIS
CAIRIS 1.3.2 commit 8cfca9c9db2a7325aa8b058742ecb9298279983a

This particular issue is fixed, but this problem affects other objects as well.  I'll open a new issue for addressing this problem.


My image and withTintColor

Hello, evgenyneu
Thank you for your pod)
I've got some question here.
I have image in png format. I'm able to change tint of it in casual UIImageView with help of .withTintColor(_:) as it expected, but when I add image to rateView.settings.filledImage or rateView.settings.emptyImage with this method nothing happens. It adds my image with origin tint color. Is that supposed to be like that or kinda bug?
How can I change tint color of added image in your pod?
And one more question:
Is it possible to change alignment of stars? Default statement like on left side. Can I make all stars aimed on center in my view?
Sincerely, Andrew

Library setup method: CocoaPods
Version of the library: 23.0
Xcode version. 12.0
OS version. Example: 13.0

Hi @matsota, thanks for reporting the issues.

It adds my image with origin tint color. Is that supposed to be like that or kinda bug? How can I change tint color of added image in your pod?

Yes, it's a bug in Cosmos library, there is no way to display tinted images for the stars at the moment. It has not been implemented. Cosmos does not use UIImageView, but instead stars images using CALayers from (see code). If we want to implements the tint, we need to do something like this.

Is it possible to change alignment of stars? Default statement like on left side. Can I make all stars aimed on center in my view?

You can use auto layout and center the cosmos view in your screen, as shown here.

Hi @matsota, thanks for reporting the issues.

It adds my image with origin tint color. Is that supposed to be like that or kinda bug? How can I change tint color of added image in your pod?

Yes, it's a bug in Cosmos library, there is no way to display tinted images for the stars at the moment. It has not been implemented. Cosmos does not use UIImageView, but instead stars images using CALayers from (see code). If we want to implements the tint, we need to do something like this.

Is it possible to change alignment of stars? Default statement like on left side. Can I make all stars aimed on center in my view?

You can use auto layout and center the cosmos view in your screen, as shown here.

Oh, thanks a lot)

Oh, thanks a lot)


Fix stopped RUMViewManager from being able to start new views.

What does this PR do?
Stopping a session leaves the session in the list of child scopes until the session is done with all of its work (Resources and user actions are stopped).  If a StartView event happens while that session is stopped but still processing events, it was incorrectly being created in the stopped ViewManagerScope.
Additional Notes
I double checked Resources and Actions in the ViewScope and those should be safe, as they are only created if the view is active (and stopping the session stops the active view)
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #1381 (a4a0770) into develop (c1fb449) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop    #1381      +/-   ##
===========================================
- Coverage    81.93%   81.88%   -0.05%
===========================================
  Files          363      363
  Lines        13014    13015       +1
  Branches      2187     2188       +1
===========================================
- Hits         10663    10657       -6
+ Misses        1686     1685       -1
- Partials       665      673       +8

Impacted Files
Coverage Δ

...d/rum/internal/domain/scope/RumViewManagerScope.kt
95.38% <100.00%> (+5.46%)
:arrow_up:

... and 11 files with indirect coverage changes


AspNet support - IHostBuilder and StartUp

setup the configuration for ASPNET applications
allowing for a consitant look and feel to the API

https://github.com/dbones-labs/auditable/commit/d88e68c6e2a46e7919ef6cc36458f1c36f5f743e#diff-3ff8c907b02169d9ea2497a6bcaa5d7e


remove case that undermines the actual check being done

So for context this was resulting in an error during blockstream which would halt the stream but allow AN to continue running. That seems like a larger issue that should be addressed at some point. The visible results were that AN wasn't updating (obv) and other strange behavior in the UI like no time data being available (the original bug this issue was for).
What was occurring was that in the case the user was up to date but within the handoff block window this code would simply make them fetch just the block they last synced +1. This effectively gets rid of any handoff block window and also allows the sync to start at a block that doesn't yet exist in the case where the user's latest synced block is also the highest block. The later case is what was causing the error  that was manifesting where AN stalls and the UI shows weird data.
Since the handoff blocks number was basically being ignored previously I reduced it back to 5.

This whole section is a patch over "what do we do if there is a re-org beyond where the sync started", such as storing raw logs in DB. This bug has always been there, but 105 slipped through caused it to be commonly hit.


Promise rejection in watcher not captured by Vue.config.errorHandler

Version
2.6.11
Reproduction link
https://github.com/vvanpo/minimal-repros/tree/master/vue2
Steps to reproduce
Finding the same problem as posted here: https://github.com/vuejs/vue/issues/11035
See the minimal repro:
import Vue from 'vue/dist/vue.common.js'

test('errorHandler should capture promise rejections in watchers', () => {
  document.body.innerHTML = '

'

  Vue.config.errorHandler = (err) => {
    console.log(err.toString())
  }

  new Vue({
    template: `<div />`,
    data: () => ({ foo: false, bar: false }),
    mounted() {
      this.foo = true
      this.bar = true
    },
    watch: {
      foo: async () => { throw new Error('foo') },
      bar: () => { throw new Error('bar') },
    },
  }).$mount(document.body.querySelector('#app'))
})

Results in the following:
(node:67048) UnhandledPromiseRejectionWarning: Error: foo
(node:67048) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:67048) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
 PASS  ./test.js
  ✓ errorHandler should capture promise rejections in watchers (19 ms)

  console.log
    Error: bar

      at _vueCommon.default.config.errorHandler (test.js:7:13)

Test Suites: 1 passed, 1 total
Tests:       1 passed, 1 total
Snapshots:   0 total
Time:        1.806 s
Ran all test suites.
✨  Done in 2.71s.

So you can see that it correctly captured the bar watcher's error, but not the promise rejection in foo.
What is expected?
Errors in async watchers can be caught by Vue.config.errorHandler.
What is actually happening?
Promise rejections inside async watchers are not caught by the errorHandler.
I've tested it in Vue 3, and there app.config.errorHandler captures all promise rejections as expected.

Duplicate of https://github.com/vuejs/vue/issues/10009


🐛 kvantum manager starts after login since kvantum-theme-matcha update to 20191119-1

branch
testing
Architecture
x86_64
Description
Yesterday I got an update for kvantum-theme-matcha: upgraded kvantum-theme-matcha (20190810-3 -> 20191119-1)
Now kvantum manager starts after every login (or sway reload).
It appears that theme Matcha-Dark (which is set in matcha-green/theme.conf) does not exist any more. kvantum manager only finds Matchama-Dark.
Overriding $kvantum-theme with set $kvantum-theme Matchama-Dark seems to fix it.
Is something wrong with the kvantum-theme-matcha package or is desktop-settings just not updated yet?
Reproduction steps
1. upgrade kvantum-theme-matcha to current release `20191119-1`
2. logout
3. login

Screenshots
No response
Logs
No response

Yes, seems the package in the manjaro gitlab is not what is in the manjaro repos. Will revert that by hand.

Update has been pushed, I guess its fixed?


OpenSearch-Build Repo Python Migration to 3.9 Version

Description
OpenSearch-Build Repo Python Migration to 3.9 Version
Issues Resolved
#3351
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Seems like windows is breaking with this in python-test:

__________________________ TestRunAssemble.test_main __________________________

self = <tests.test_run_assemble.TestRunAssemble testMethod=test_main>
mock_recorder = <MagicMock name='BundleRecorder' id='1721159340672'>
mock_bundles = <MagicMock name='create' id='1721159748864'>
getcwd = <MagicMock name='getcwd' id='1721159757056'>
makeDirs = <MagicMock name='makedirs' id='1721159773248'>

    @patch("os.makedirs")
    @patch("os.getcwd", return_value="curdir")
    @patch("argparse._sys.argv", ["run_assemble.py", BUILD_MANIFEST])
    @patch("run_assemble.Bundles.create")
    @patch("run_assemble.BundleRecorder", return_value=MagicMock())
    def test_main(self, mock_recorder: Mock, mock_bundles: Mock, getcwd: Mock, makeDirs: Mock) -> None:
        mock_bundle = MagicMock(min_dist=MagicMock(archive_path="path"))
        mock_bundles.return_value.__enter__.return_value = mock_bundle

        main()

        mock_bundle.install_min.assert_called()
        mock_bundle.install_components.assert_called()

        mock_bundle.package.assert_called_with(os.path.join("curdir", "tar", "dist", "opensearch"))

        mock_recorder.return_value.write_manifest.assert_has_calls([
            call("path"),
            call(os.path.join("curdir", "tar", "dist", "opensearch"))
        ])  # manifest included in package

>       self.assertEqual(getcwd.call_count, 2)
E       AssertionError: 3 != 2

tests\test_run_assemble.py:54: AssertionError

Windows can change to 3.9.7 with scoop here, no need to go to 3.9.13, tho need some checks on why this is failing tho:
https://github.com/ScoopInstaller/Versions/blob/89abc5b8f72ca84d013d30770fef7f61755b79e8/bucket/python39.json

With Windows able to run Python3.9 now will debug more before merging this.

Approved to proceed with the upgrade window. Thanks


Add option to config for displaying most recent files first

Add a config option which either reverses the order of the displayed files so the most recent file is the first displayed in the list and the oldest one is the last, or instead of reversing the order, make it so it works like camera roll does and just begin the list view at the bottom of the list instead of at the top.

We have the same issue, we found a way to fix it. We can do a pull request


Merging front end with backend

Added Vis Graph, merged Darwin's codebox, and added navbar, popup for link

closing and recomitting


Please offer a download link for latest release, e.g. ttps://github.com/krallin/tini/releases/download/latest/tini

Please offer a download link for latest release, e.g. https://github.com/krallin/tini/releases/download/latest/tini . For people who don't run a huge production server, the version pinning might not give enough benefits (a tad more stability) with the huge downside of forgetting to update the pinning when running lots of custom Dockerfile images with tini in it. Therefore, being able to simply always use the latest release would be hugely appreciated.

I'm unlikely to do this. Besides the fact that I'm not sure I can easily do that with Github releases to begin with, I'd rather not get in a position where breaking changes have to be held back even across major releases, because some users might have chosen not to pin their versions: I've been bitten by this before.
If you have a large number of images, I would recommend using a base image in which you install Tini,a and building your other images off of that one. That way, if you want to upgrade Tini, you only have one image to update.
Cheers,

Then why not make a tini api version or something? https://github.com/krallin/tini/releases/download/latest-api1/tini
Base image doesn't make much sense unless there is a tini base image on dockerhub to derivate from. I share my Dockerfiles with people sometimes, and requiring a special base image only I have would greatly interfere with that. Maybe an official ubuntu tini image might be an idea?

Then why not make a tini api version or something? https://github.com/krallin/tini/releases/download/latest-api1/tini

Tini simply does not change very often (because it's intentionally minimal as a piece of software), so I don't think it's worth maintaining this kind of complexity in its release process for no clear benefit.
If you absolutely need to upgrade Tini centrally and it's a major pain point for you, then I would recommend simply hosting the binaries for yourself, on AWS S3 or another object store.
Cheers,
(as an aside, there is in fact a base Ubuntu Tini image, but it's intended to simply use the core zombie reaping and signal forwarding functionality of Tini, so for now it's still on an older version since that functionality hasn't changed since)

You're slightly missing the point, so let me elaborate: the problem isn't the pain of an upgrade but rather if you ever do a security upgrade, I know I and a lot of other people will forget to upgrade it in some of their images. That's why I don't pin versions intentionally, because I don't run super important production stuff and I'd rather have a broken image rather than have a literally open door for anyone to hack in.

In that case, I would recommend relying on a distro to package and keep Tini updated for you. It is currently packaged in Alpine and Nix. I think it'd be good to try and get it packaged in Debian and Ubuntu, but that's not done as of yet AFAIK.
In any case, I do understand your point, although I think the threat model is a bit misguided here. Downloading a "latest" binary has its own risks since you can't checksum it. I'm sure you will account for this, but if I provide a latest binary and download instructions for it, others may not.
Taking a step back, you're looking at this from your personal perspective and focusing on how I can fix your problem, but I'm afraid you're not considering how this fix will affect me and what I can do with Tini: you would rather have a broken image than one that's not up to date, but others may not share your opinion. If I set up a "latest" URL, and have to release a breaking change at some point, you won't complain about it, but I can certainly guarantee that others will (which is why I'm not doing it).
So, as I mentioned originally, I'm not planning on delivering this. In any case, I'm a lot more likely to work towards getting this packaged into more distributions (note: this is something others can contribute to as well) than I am to ever offer a "latest" URL.

Yea, a distro package I'd use instantly. That sounds like a perfect solution, so I hope there will be an Ubuntu package eventually. I don't really like Alpine because I'm not that super tight on disk space and I prefer to have a similar environment inside and outside of the containers, but I might use it now for tini alone. Thanks for the suggestion!

Downloading a "latest" binary has its own risks since you can't checksum it.

Maybe, but HTTPS is not as insecure as you give it credit for (of course for a state-level attacker it is, but then you have other problems...).
TL;DR: will consider using Alpine for now, but Ubuntu/Debian packages would be really nice to have!


Bumped shipkit & some tidy-up

Cleaned up travis file
Corrected the previous version
Publications comparator will be enabled by default so no need to explicitly enable it

Friendly ping!


Group changes by area/component rather than by SIG

Breaks down SIG structure in favor of area/component
Creates single action required section
Creates depreciation section

TODO

[ ] edit for structure (@bgrant0607, @shtatfeld)
[ ] edit for content (all)
[ ] make changes (all)
[ ] add go/docker/etcd version information (@bgrant0607, @dchen1107, @xiang90)
[ ] add known issues (all)
[ ] squash commits (someone)
[ ] approve final version (@idvoretskyi)

thanks @calebamiles, looks great!

Oh, in the etcd warning at top we should clarify:

using application/vnd.kubernetes.protobuf as the media storage type for 1.6 is default but not required
the ability to rollback to etcd2 can be preserved by continuing to use application/json as the storage media type. This can be changed to application/vnd.kubernetes.protobuf at a later time.

@ethernetdan thanks, added.
@luxas thank you, already fixed :)

@idvoretskyi Yeah, I reviewed this PR this morning, but I noticed I didn't submit it before now :)

@idvoretskyi Please squash commits when merging.

@bgrant0607 sure, sorry.

@bgrant0607 yeah, I also thought about it but wasn't actually confident enough to speak up earlier (didn't know if it was worth mentioning).
I think though, that generally, the commit information could (I'm absolutely not saying it will) be mis-used so that company xxx could say that they have contributed yy percent to k8s when a lot of the changes are simple docs edits/typos/sentence moves which get one commit (or less) in all other cases. These commits will then show up in https://cncf.biterg.io
I'm just very keen that we don't fall into the trap described in https://blog.heptio.com/the-peril-in-counting-source-lines-on-an-oss-project-c50a943fe6b1

(But I like the PR itself and generally the work that's being done here, don't get me wrong)
Thanks @calebamiles @idvoretskyi @ethernetdan for all the hard work with the release :+1:


Log Error when unable to instantiate a consumer

Contact Details
No response
Version
8.x (develop, pre-release)
On which operating system(s) are you experiencing the issue?
Windows
Using which broker(s) did you encounter the issue?
RabbitMQ
What are the steps required to reproduce the issue?

Add consumer with DI dependency, e.g.:

public MyConsumer(IDependency dependency){ ... }

Do not add IDependency to DI
Configure retry policy for this consumer:

endpointConfigurator.UseDelayedRedelivery(policy=>
                policy.Interval(100, TimeSpan.FromMinutes(1)))

Launch your service and post a message

What is the expected behavior?
I expected it to throw an "Error", because retrying the message won't help when the consumer cannot be instantiated. It can only be fixed by changing the code.
What actually happened?
It throws a warning and resends a message according to retry policy even though the consumer will not be able to handle it when it comes back.
Retrying a message because of an error in Consume method seems like a valid scenario for me, which should generate warnings. But when a consumer itself cannot be constructed due to missing DI dependencies or invalid configuration, I think it should throw "Error" to alert the dev to fix the code/configuration.
Related log output, including any exceptions
warn: MassTransit.ReceiveTransport[0]
      R-RETRY rabbitmq://rabbitmq/MyConsumer 028be2ce-f0fb-425f-b319-37eae11366e9 MassTransit.RetryPolicies.RetryConsumeContext<MyMessage>
      System.InvalidOperationException: Unable to resolve service for type 'IDependency' while attempting to activate 'MyConsumer'

Link to repository that demonstrates/reproduces the issue
No response

It's an InvalidOperationException - those happen for so many different reasons.
You can try to filter them yourself using an exception filter if you want, but MassTransit isn't going to just "know" it isn't a transient condition.

@phatboyg

but MassTransit isn't going to just "know" it isn't a transient condition.

You don't need to know whether it is transient, you can just log exceptions that happen during Consume and those that happen when instantiating Consumer object with different log level or different logger, so that I could easily determine what kind of error it is.
Exception Filter does not solve my problem. I do want those messages to go to retry, but I also want an Error in logs, that would alert me that those retries happen because of non-transient error, so I could fix my code, redeploy the app and process those messages.


An internal error occurred in ../../../src/lib/tree.h(101)

I have posted the protein file (20.pdbqt) and the ligand file(2.pdbqt).
When I run './DOCKING/vina_1.2.5_linux_x86_64 --receptor ./20.pdbqt --ligand ./2.pdbqt --size_x 20 --size_y 20 --size_z 20 --autobox', I encountered an error: An internal error occurred in ../../../src/lib/tree.h(101)
I have tried many groups of protein files and ligand files, and this error occurred too many times.
files.zip

Hi @YvetteJxy, Your upload appears as a HTML file (it looks very like the login page of Jupyter server). Let us know if you reshare the files.
One more thing about --autobox, in this scenario (a docking search), --autobox doesn't actually set the grid size or the grid center (?). --size_x --size_y --size_z are necessary even with --autobox, and if not specified the center seems to be 0,0,0. Not sure if this is related to the error.

Thank you for your help! I am very grateful for that!
I have uploaded the file again and I hope this time it will work!
files.zip

Hi @YvetteJxy, the problem was caused by your ligand PDBQT file, which doesn't have valid 3D coordinates but all 0s. You might want to review your procedure for ligand preparation.
[If I guessed your ligand right] (https://en.wikipedia.org/wiki/Dimethylsulfoniopropionate), here's how you could generate a PDBQT with coordinates and charges (although not necessary for vina) with obabel:
./obabel -:"C[S+](C)CCC(=O)[O-]" -o pdbqt --gen3d
REMARK  Name =
REMARK  5 active torsions:
REMARK  status: ('A' for Active; 'I' for Inactive)
REMARK    1  A    between atoms: C_1  and  S_2
REMARK    2  A    between atoms: S_2  and  C_3
REMARK    3  A    between atoms: S_2  and  C_4
REMARK    4  A    between atoms: C_4  and  C_5
REMARK    5  A    between atoms: C_5  and  C_6
REMARK                            x       y       z     vdW  Elec       q    Type
REMARK                         _______ _______ _______ _____ _____    ______ ____
ROOT
ATOM      1  C   UNL     1       3.420  -1.521   0.625  0.00  0.00    +0.283 C
ENDROOT
BRANCH   1   3
ATOM      2  C   UNL     1       1.153  -0.143   0.096  0.00  0.00    +0.340 C
ATOM      3  S   UNL     1       2.913  -0.014  -0.300  0.00  0.00    -0.016 S
ATOM      4  C   UNL     1       3.321   1.174   1.001  0.00  0.00    +0.340 C
ENDBRANCH   1   3
BRANCH   1   5
ATOM      5  C   UNL     1       4.918  -1.736   0.537  0.00  0.00    +0.106 C
BRANCH   5   6
ATOM      6  C   UNL     1       5.385  -2.974   1.303  0.00  0.00    +0.046 C
ATOM      7  O   UNL     1       6.631  -3.182   1.245  0.00  0.00    -0.550 OA
ATOM      8  O   UNL     1       4.496  -3.633   1.911  0.00  0.00    -0.550 OA
ENDBRANCH   5   6
ENDBRANCH   1   5
TORSDOF 3

@rwxayheee Thank you for your help! I think I know what the problem is! I didn't generate valid 3D coordinates of my ligand and now I have corrected my error! Thank you very much!


Multiple users on PseudoTV / Live TV & DVR?

I have a couple users on my Home. When we both try to play something, it requests to end the current channel to be able to start the new one. Is there any way around this? Is this a PseudoTV limitation, or a Live TV & DVR one, or just me configuring things wrong?
It looks like I'm limited by the number of tuners. Is there a way to add more?

Increasing the "Tuner Count" in the pseudotv settings should get this working. It's been a really long time since I looked at the code or even used the app. But ya, increasing the tuner count in the web settings should allow for multiple viewers of different channels. You may need to reconfigure your Plex/DVR app after making this change. Essentially it just lets Plex know how many streams/tuners it has available to use at any given time. Plex is smart enough not to use more than one tuner for any particular channel. So if one user is watching channel 20, and another user decided to watch channel 20, it'll only use one stream/tuner. Hope that makes sense.
In any case, I highly recommend you try out DizqueTV (a fork of this project) and ErsatzTV. This project hasn't been active in a while.


Does the scheduler persist?

I am looking for a task scheduler which can execute tasks in the future. And Chrono seems pretty amazing! and close to my need.
However, I'd like to know if there is a backing store? And does the schedule tasks persist through restarts as well?
Thanks

Hi, first of all, thanks for your feedback.
Unfortunately, chrono does not store your scheduled tasks. If you don't schedule your tasks dynamically by the user inputs, you can try to schedule your tasks in init method so that even if your program restarts, your task will be scheduled at the startup.
If you have any further questions, please free to ask.

This issue is closed due to inactivity


Site inaccessible because of SSL issue

This is probably due to the year change over, but https://speakerline.io isn't available right now.

I'm not getting any more useful error messages on this...
paging @nodunayo -- figure app down is kind of a biggie!

Same issue on OS X in Chrome, Firefox, Opera and Safari

Hmmm!! Strange. Will look into it. Thank you!!
Nadia Odunayo

On 1 Jan 2018, at 10:55, flyinggrizzly<EMAIL_ADDRESS>wrote:
Same issue on OS X in Chrome, Firefox, Opera and Safari
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Hey, @flyinggrizzly.
It's because you're attempting to access Speakerline via https, but I don't have SSL set up yet. Only http for now!
(It's high up on my list to sort it out.)

Oh dear... now I feel silly

Oh no, please don't! I feel silly for not having sorted SSL yet! 😁


Improvements:  switch account menu, and confirm popup

What does this pull request aim to achieve? 💯
Fix the profile menu not scrollable when having a lot of orgs.
Fix the banner (summit announcement) hiding the profile menu.
Add popup confirmation to stay in the same page or redirect to the help board after switching accounts.
Please be concise and link any related issue(s):
closes #1848
closes #1854

[x] I have checked that no one else is working on similar changes.
[x] I have read the latest README and understand the development workflow.
[x] The name of this branch is: feature/<branch_name>, or hotfix/<branch_name> if this is a hotfix, where <branch_name> briefly describes the issue(s) resolved and is prefixed with the issue number(s) (e.g. 1127-1130-update-git-branching-model). The branch is created in a cloned version of this repo, not a forked repo.
[x] This branch is rebased/merged with the latest staging (or the latest production if it is a hotfix).
[x] There are no merge conflicts.
[x] There are no console warnings and errors.
[x] On the right hand side of this PR, I have linked the appropriate issue(s) under "Issues" and added a project progress under "Projects".
[x] The title describes the issue(s) being addressed using format: <issue_numbers> - <brief_issue_description> (e.g. #1127/#1130 - Update Git Branching Model)

Hi,
I tested the following scenarios:

Tested for a large amount of orgs: Drop down is scrollable - PASS
Open About us page - switch account - Popup is shown Stay on the current page?. Yes - stays at the same page, No - redirected to the Help Board. - PASS
Open Help Board - switch account - Popup is shown Refresh page. Click Ok. Page is refreshed.
@Zaydme would it be possible in the future to press Enter and confirm OK? Or maybe do this automatically?
I would use OK - both capital letters or Confirm?
Banner doesn't cover Profile dropdown - PASS
Create new org - click on the dropdown list and select Add org - PASS

@Zaydme
will this last commit fix Switch accounts - login with the default user #1857

@galeza yes


Selective violation cucumber

Description
Allow selective violation in cucumber
Changes
Add Cucumber step
Write example
Additional notes
Most actual tests not yet written,
Issue
part of #402

Wiki page updated to describe new step:
https://github.com/ScottLogic/datahelix/wiki/Using-the-Cucumber-framework
Would the phrase for the step be better expressed as:

we do not violate constraint {operator}

instead of

we do not violate any {operator} constraint

I know the work is still in progress, looking great but I've added a few comments

@pdaulbyscottlogic as we discussed today I think it would be good to add in some testing for each of the constraints we have as a starting point. Let's have another chat about what other test cases would be valuable.
@cakehurstryan @KhanP-SL  what are your opinions on this?


Removed 2.0.0 tag ?

Was the 2.0.0 tag removed ?

Yes sorry, there was some missing handling
regarding the authorized flow.

Will it be added back soon ?
Should I use RC version in the meantime ?

Yes it will be back soon, I hope next week to finish testing it fully with the bundle and the sylius plugin.
In the meantime you can use the RC version.

@dragosprotung finally I released the v2.0.0.
I hope you did what you need with the RC version.

Excellent, thank you.
The RC version worked for my needs, now i will just upgrade to final v2.


Link to live instance from Github "About" section

For easy access to the live website it is handy to have a link in the Github repositories "About" section.
It can be added via cogwheel icon.

Thanks for the suggestion. Done.


Allow a plugin to bust the cache to force a reload of the page

Currently plugins can cache pages, but they cant be easily force reloading. This commit adds the feature to bust the cache by simply sending the parameter _bust_cache along. This allows your application to request the page directly from prerender, ensuring search crawlers always hit the cache and get served the correct page within a few milliseconds.
Since I'm not very familiar with node (yet), I'd love to hear recommendations to further improve this system.
To explain the rationale of this PR, let me sketch the workflow: In our case clients can update their pages (which happens quite frequently), but Google visits less than once a day. So we cannot just have very short TTL's, but would like to send the prerendered version to Google directly (improving SE). Using this approach, we can let the application fire an event over our message bus to a very small microservice, which will simply call this page with the _bust_cache=true parameter, which will set the new content directly in prerender, ready to be served

@alexnederlof for info!


feature request: string regular expression matching

I'm testing code which uses the common
ok, retval = func( ... )

paradigm, where retval is an error message/object if ok is false.  Just as with error messages tossed via exceptions (and handled by has_error) it would be nice to be able to compare the result with a match rather than an exact equality, e.g.
assert.are.matches( '.*: missing file', retval )

Thanks for a great testing framework!
Diab

It seems that I'm mistaken about has_error After perusing the code I see that it only supports exact substring matching and matching numbers at ends of strings.
A generic pattern match would be nice.  I have errors which include keywords taken from tables, so their order in the error message are non-deterministic.  I need to be able to check for strings like
 a and b are excluded
 b and a are excluded

and ensure that the a and b keywords are in the error message (else I could just look for the string are excluded )
Here's some example code:
https://gist.github.com/djerius/7879cbd6d9ac66257e32
It allows one to write
assert.matches( "[ab] and [ab] are excluded",  retval )

@djerius I think this went into v1.7.7


Added Scripting languages

Hey ! I am adding Scripting languages to the readme
hope it is helpful

Merged.


rename charlength and character_length function names

Signed-off-by: galsalomon66<EMAIL_ADDRESS>

some of the tests which uses string-length functions failed since the s3select-engine was updated (to be align with AWS function names)
the like-tests uses much bigger object (which is not that big ..), but in some of teuthology runs it may cause timeout on boto3.
to avoid those random failures in teuthology, the object size was reduced.

can you please put a link to the PR where the code change is done?

can you please put a link to the PR where the code change is done?

https://github.com/ceph/s3select/pull/31/files#diff-df8fff244ae0355c2313b8bf7c2f6e353d791f6ac939620a4e9c80b877519580R88


Revert Remove sensio fw

Reverts rectorphp/getrector.org#459
Per issue https://github.com/rectorphp/getrector.org/issues/461

@TomasVotruba revert the remove sensio fw seems solve the demo page submit error
https://getrector.org/demo/1ec62bf8-bf08-6e1c-85b1-ad1b9028bf0b
🎉


feat: add target for building sage

When migrating to sage in a repository yesterday I found that quite a lot of time I would go back into the sage directory to regenerate my Makefile. Adding a target for doing this could be useful.

What is the reason for addition? :thinking:
Usually a touch .sage/go.mod && make should cover this use-case

What is the reason for the addition? thinking
Usually a touch .sage/go.mod && make should cover this use-case

I find using a target for this easier.

What is the reason for the addition? thinking
Usually a touch .sage/go.mod && make should cover this use-case

I find using a target for this easier.

I have nothing against it. Have you tried it in your repo? Does it work the way you expect it to?
I'll leave it to @odsod or @viktorvoltaire to approve though

I tried it in this repository. Have not tried it in any other repository.

Even though I can approve this, I think its better if @Edholm and @odsod does it :rofl:

@viktorvoltaire Should we add an update-sage target while we're at it...? Breaking changes and convenience considered

Breaking changes and convenience considered

What do you purpose this target does?

I will merge this, and we can look into the update command later


Compiler bug when trying to use libuv

When trying to compile this code with 4f8c26d2c605f24cdeb1a4c7154662b2552640ef ,  the following errors arise :
/home/pvd/Devel/zigtests/src/c.zig:1:19: error: compiler bug: @cImport generated invalid zig code
pub const libuv = @cImport({
                  ^
?.c:1:1: note: redefinition of 'uv_connect_t'
TODO: remember C source location to display here
^
?.c:1:1: note: previous definition is here
TODO: remember C source location to display here
^
/home/pvd/Devel/zigtests/src/c.zig:1:19: error: compiler bug: @cImport generated invalid zig code
pub const libuv = @cImport({
                  ^
?.c:1:1: note: redefinition of 'uv_connect_t'
TODO: remember C source location to display here
^
?.c:1:1: note: previous definition is here
TODO: remember C source location to display here
^
/home/pvd/Devel/zigtests/src/c.zig:1:19: error: compiler bug: @cImport generated invalid zig code
pub const libuv = @cImport({
                  ^
?.c:1:1: note: redefinition of 'uv_shutdown_t'
TODO: remember C source location to display here
^
?.c:1:1: note: previous definition is here
TODO: remember C source location to display here
^
/home/pvd/Devel/zigtests/src/c.zig:1:19: error: compiler bug: @cImport generated invalid zig code
pub const libuv = @cImport({
                  ^
?.c:1:1: note: redefinition of 'uv_shutdown_t'
TODO: remember C source location to display here
^
?.c:1:1: note: previous definition is here
TODO: remember C source location to display here
^
/home/pvd/Devel/zigtests/src/c.zig:1:19: error: compiler bug: @cImport generated invalid zig code
pub const libuv = @cImport({
                  ^
?.c:1:1: note: redefinition of 'uv_pipe_t'
TODO: remember C source location to display here
^
?.c:1:1: note: previous definition is here
TODO: remember C source location to display here
^
/home/pvd/Devel/zigtests/src/c.zig:1:19: error: compiler bug: @cImport generated invalid zig code
pub const libuv = @cImport({
                  ^
?.c:1:1: note: redefinition of 'uv_pipe_t'
TODO: remember C source location to display here
^
?.c:1:1: note: previous definition is here
TODO: remember C source location to display here
^
/home/pvd/Devel/zigtests/src/c.zig:1:19: error: compiler bug: @cImport generated invalid zig code
pub const libuv = @cImport({
                  ^
?.c:1:1: note: redefinition of 'uv_read2_cb'
TODO: remember C source location to display here
^
?.c:1:1: note: previous definition is here
TODO: remember C source location to display here
^
/home/pvd/Devel/zigtests/src/c.zig:1:19: error: compiler bug: @cImport generated invalid zig code
pub const libuv = @cImport({
                  ^
?.c:1:1: note: redefinition of 'uv_read2_cb'
TODO: remember C source location to display here
^
?.c:1:1: note: previous definition is here
TODO: remember C source location to display here
^
/home/pvd/Devel/zigtests/src/main.zig:6:16: error: no member named 'uv_loop_init' in '(C import)'
    _ = c.libuv.uv_loop_init(loop);
               ^
/home/pvd/Devel/zigtests/src/main.zig:10:16: error: no member named 'uv_loop_close' in '(C import)'
    _ = c.libuv.uv_loop_close(loop);
               ^
The following command exited with error code 1:
zig build-exe /home/pvd/Devel/zigtests/src/main.zig --cache-dir /home/pvd/Devel/zigtests/zig-cache --output /home/pvd/Devel/zigtests/zig-cache/main --name main --library uv

Build failed. Use the following command to reproduce the failure:
./zig-cache/build zig . ./zig-cache

Maybe it has something to do with the fact that libuv uses a lot of 'opaque' structs to hide private members with C macro magic.

$ cat la.h
#include <uv.h>
$ zig parsec la.h

gives me the following output:
https://gist.github.com/aindigo/60d47c904cdfea556bd00f39103b7940

I'm still getting the error compiling that repo, can you check?
another example:
pvd@alphanine:~/Devel/llvm5/debug$ cat la.h
#include <uv.h>

pvd@alphanine:~/Devel/zig/build$ ./zig version
<IP_ADDRESS>29322a
pvd@alphanine:~/Devel/llvm5/debug$ ../bin/zig parsec la.h | grep "uv_connect_t ="
pub const uv_connect_t = struct_uv_connect_s;
pub const uv_connect_t = struct_uv_connect_s;

uv.h has the following declarations, for typedef uv_connect_t:
typedef struct uv_connect_s uv_connect_t;
...
...
struct uv_connect_s {
  UV_REQ_FIELDS
  uv_connect_cb cb;
  uv_stream_t* handle;
  UV_CONNECT_PRIVATE_FIELDS
};

talked on IRC, the former issue is fixed and what is remaining is #144


Fix npm install path

Brief Description
Updates the npm install path in the readme
Pull Request Guidelines

[ ] I have added pull requests labels which describe my contribution.
[ ] All existing tests passed.

[ ] I have added tests to cover my changes, of which pass.

[ ] I have compared the render output of my branch to master.
[ ] My code has passed the ESLint configuration for this project.
[ ] My change requires modifications to the documentation.

[x] I have updated the documentation accordingly.

The Github workflow is stuck (not sure why). Merging.
Thanks for the PR @bnjm !


Yii CVE-2018-7269,  CVE-2018-8073 and CVE-2018-8074

replaces #276.
Changed the version constraints so that the validator is happy with it.
@fabpot this is ready to be merged as far as I see.

travis fails because it can not install php 7.2 :eyes:  validator is green locally.

Thank you

Thank you for fast merging!

Thank you, Fabien


Added TypeScript definitions file and fixed issue with dumping objects

The PR resolves this issue and adds a TypeScript definitions file.

good job


Method to redraw table

Don't you have a redraw method?

@alfligno not sure what you mean by redraw @alfligno the table should reflect the state of your rows/columns as they change. If you really want to re-render the entire component for some reason, you can put a v-if on the component and use a setTimeout to make it unmount and mount again.

For nested object it somehow does not make any changes.
But I already have a solution. forcing the list to be refreshed by making changes on the filters will do

Your talking about vue reactivity.
Use Vue.set() to update your data

Also has a situation where the table does not reflect what's inside my object. We have an input field which users set to for instance -4 and that updates the stockinventory-field. So after POST I want the table to reflect the new values in mainobject which is used to populate tablerows in the first place. i will try the methods suggested here.

Also has a situation where the table does not reflect what's inside my object. We have an input field which users set to for instance -4 and that updates the stockinventory-field. So after POST I want the table to reflect the new values in mainobject which is used to populate tablerows in the first place. i will try the methods suggested here. The field with -4 should go back to 0 or empty.

I suggest to use this.$set as what @p0psicles suggest, the method I use is my own derivation on the issue and it may not work :)

We tested that. Not sure if it was implemented correctly though. Then I myself choose the v-if route and that worked great. A small blink on the table but nothing annoying. Don't ask me what I named the toggling-variable. 😝

We tested that. Not sure if it was implemented correctly though. Then I myself choose the v-if route and that worked great. A small blink on the table but nothing annoying. Don't ask me what I named the toggling-variable. 😝

I guess we are on the same boat. Toggling will do, just use some spare variable XD, which is named change


renode-dts2repl: 0-unstable-2024-08-30 -> 0-unstable-2024-09-05

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for renode-dts2repl is: Tool for converting device tree sources into Renode's .repl files
meta.homepage for renode-dts2repl is: https://github.com/antmicro/dts2repl
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0-unstable-2024-09-05 in filename of file in /nix/store/as74zhgygn18vfpvr44jyva7rzc04xn9-renode-dts2repl-0-unstable-2024-09-05

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

renode-dts2repl

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/as74zhgygn18vfpvr44jyva7rzc04xn9-renode-dts2repl-0-unstable-2024-09-05 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A renode-dts2repl https://github.com/r-ryantm/nixpkgs/archive/d7b100d85f406a6d4c739f9f23126f11807bbd3a.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/d7b100d85f406a6d4c739f9f23126f11807bbd3a#renode-dts2repl

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/as74zhgygn18vfpvr44jyva7rzc04xn9-renode-dts2repl-0-unstable-2024-09-05
ls -la /nix/store/as74zhgygn18vfpvr44jyva7rzc04xn9-renode-dts2repl-0-unstable-2024-09-05/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

renode-dts2repl
renode-dts2repl.dist

Maintainer pings
cc @otavio for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


multiple inline option mistake isnt highlighted (it didnt work ingame(4 month ago))

example of the mistake:
    "slots": [
        ["type", "default", "description"],
        ["my_slot", "my_part", "My Slot", {"nodeOffset":{"x":0,"y":1,"z":2}}, {"coreSlot":true}],
    ],

the nodeOffset and coreSlot is seprated into 2 inline option dict, which the plugin doesnt catch, and the game couldnt handle it properly (understandably)

nvm, it does catch it, the plugin didnt refresh? when i did my replication test? now it notices it
my bad :(

Thanks for reporting a potential issue, glad it's not a bug in the end :)


fix handling of message from login window

It'd probably be best if you tried this out locally yourself, but from my tests this works whether or not you need to interact with the popup window, and whether you are on firefox or chrome.

you goddamn hero. tested locally, works for me — redeploying now


Verify lookups.

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#675  👈
#672 : 1 other dependent PR (#674 )
#671
#669
#667
#664
#662
#646
#644 : 1 other dependent PR (#654 )
#642
#639
#638
#637
#636
#651
dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @alonh5 and the rest of your teammates on  Graphite

Codecov Report
Attention: Patch coverage is 55.55556% with 4 lines in your changes missing coverage. Please review.

Project coverage is 89.75%. Comparing base (f3e308f) to head (1dcb284).

Files
Patch %
Lines

crates/prover/src/core/prover/mod.rs
0.00%
3 Missing and 1 partial :warning:

Additional details and impacted files
@@                               Coverage Diff                                @@
##           06-23-merge_trace_writer_and_trace_generator     #675      +/-   ##
================================================================================
- Coverage                                         89.79%   89.75%   -0.04%
================================================================================
  Files                                                75       75
  Lines                                              9717     9726       +9
  Branches                                           9717     9726       +9
================================================================================
+ Hits                                               8725     8730       +5
- Misses                                              910      913       +3
- Partials                                             82       83       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


✨ [Feature] Add Minutes, hours, days methods for facades.Schedule()

Before feedback (在反馈之前)

[X] There are no features that I want to submit in Issues (当前 Issues 中没有我将要提交的新功能)

Describe Feature (描述功能)
Minutes: Run every N minutes
Hours: Run every N hours
Days: Run every N days

Can I take up this task?, If yes, I am on it.

Sure, please, thanks!

Hi @narad-muni Are you still working on this, please?


Core action to wait for trigger

SUMMARY
Our workflows require us to wait for a specific trigger with specific parameters, sleeping indefinitely until that happens.  There is no obvious way to do this with a Python action, and a Python action would eventually timeout anyway.
ISSUE TYPE
Pick one below and delete the rest:

Feature Idea

STACKSTORM VERSION
3.0.1
STEPS TO REPRODUCE
Create workflow.
Observe there is no way to have the workflow pause until a condition (signaled by a trigger) is met.

To be clear, I am not interested in triggering a new workflow -- the feature request is about suspending a workflow completely (whether through an action or some other way) until a trigger fires, and then I can run an action that can deal with the trigger next.

Went thru the code and looked at the inquirer_runner.py file.  That looks like we would need a new runner that simply pauses until a trigger event is received which matches the spec passed to the runner?  Could it be implemented as an action?  What I want is for the Orquesta workflow running said action to pause the branch executing the action, or heck, even continue running the action indefinitely with no timeout, until an event triggers it (or optionally a timeout has passed).  Sorta like core.pause but monitoring the event queue and unsticking itself when the event happens.
The big problem with an action, specifically, is that when the st2actionrunner running the action dies (machine crashes), the Orquesta workflow just remains there, stuck forever, and in many cases can't even be canceled.
Please advise.  I'd like to collaborate on a solution to this, but I don't know the internals of StackStorm well enough to figure this out.

Sounds like https://docs.stackstorm.com/inquiries.html is the closest to what you want to do with Notifying Users of Inquiries using Rules.
Event-queue like logic fits more into StackStorm sensors https://docs.stackstorm.com/sensors.html#sensors where you potentially can work with st2 direct calls/API.
Can you provide any simplified real-world example that would describe your use case and be valuable for the broader community?
Maybe one workflow that does it all could be divided into lower-level st2 primitives like several workflows/sensors/actions/inquiries instead of new runner type?

The part I'm wondering about is how you connect the new trigger to the existing paused action. How do you map those together? What links them?
Speculating here, but what you could do is something like this:
1/ First workflow runs, gets to point where it pauses (using inquiries).
2/ New event received that should restart workflow.
3/ This event kicks off a workflow that somehow matches up that new event with the paused workflow (or takes a different action if there is no matching paused workflow).
4/ That new workflow finds the existing paused workflow, and responds to the inquiry, allowing it to pick up where it left off.

Sort of.  We have an external system that sends events to our platform (which we consume using sensors, which then send triggers).  I can set up a rule to send those triggers to an action which gets executed on the spot.
The main problem I have is that my executions list gets polluted by all the actions that get triggered by the sensor's triggers.  The secondary problem I have is that the actions cannot easily coalesce this way.
Speaking of how I worked around this problem -- seems like an awful workaround, by the way -- what I'm doing in this specific workflow is put a dead stop in the workflow in the form of an inquiry (exactly what you suggested).  Then I have a separate action which gets executed by the trigger, which then (this is the ugly part) queries all pending inquiries, guesses (based on the trigger payload) if they need to be answered in the affirmative, and then answers in the affirmative if they match the guess.  This, of course, requires an awful lot of code (most of which is using the st2 pack library, because there's no inbuilt functionality in StackStorm to call other packs' actions from your custom pack's action).
So, to sum it up, here's the workaround.

Workflow you execute does A -> B -> C where B is an inquiry.  B gets to state pending after A is done.
Sensor S dispatches trigger payload T.
Action R reacts to payload T, queries existing pending inquiries, finds B, looks into its schema, retrieves data from the schema, compares it with payload T, if it matches then responds to the inquiry B.
This unlocks B with a certain data payload answer to B's question.
B continues to C.
C can then consume data from what B published, and proceed.

This seems excruciatingly painful to me, as well as super hard to compose.  If we just had an action st2.wait_for_trigger that would just wait for the trigger (and was pausable), and then I could write an action that processes the output of the trigger, then I'm game.  That'd be much better.  It would, of course, be far better than cluttering the UI with silly questions that would get answered by the computer, or silly workaround actions that were answering as the computer.

(most of which is using the st2 pack library code, because there's no inbuilt functionality in StackStorm to call other packs' actions from your custom pack's action).

Could you explain a bit more about what you mean here? Normal design pattern is to have a workflow in your custom pack, and that workflow calls actions from other packs. Or are you talking about writing a custom Python action and within that action calling other actions?

Well, we have a long-running workflow (1 month or more) which iterates over a bunch of hosts that need some work done.  This work cannot start until an external system notifies us (via a sensor) that so-and-so hosts are ready to go.  Which is fine, we have the sensor and we can bind it to some action without much fuss using a rule.
In our workflow, for now, we have an inquiry that basically freezes the workflow until someone says "yes".  We've also built a separate "robot finger" action which we have bound to our sensor via a rule, to hit "yes" on the inquiry, and then the workflow can proceed, then return to the inquiry until there are no more hosts in the pending list.  The robot finger scans all workflows that are paused with a pending inquiry, and responds the inquiry -- that's basically how we get the workflow to continue when it's appropriate.
But we're talking thousands of invocations, so the list of actions gets hugely polluted with this action getting called by the sensor / rule.
It would be far better if we had an action we can integrate into our workflow, which pauses the workflow, and then the rule can unpause the workflow.  Thus we don't invoke that "robot finger
action at all.

How does the rule figure out which workflow should be unpaused? How do you connect the two?

The robot finger scans all workflows that are paused with a pending inquiry, and responds the inquiry -- that's basically how we get the workflow to continue when it's appropriate.

How do you plan to get around this? How would you connect the sensor event to the paused action?

It would be far better if we had an action we can integrate into our workflow, which pauses the workflow, and then the rule can unpause the workflow. Thus we don't invoke that "robot finger
action at all.

Are you aware that you use core.pause to pause an action, and st2 execution resume <ID> to resume a paused action? You don't have to use Inquiries.
You might be over-thinking this "robot finger" thing - inquiries are just another action, just another API. They don't have to be human-driven.

In my current -- very suboptimal -- setup, the rule answers the core.ask in the workflow by invoking a separate task.  This works fine.
Further problems:
I tried running in my workflow two parallel tasks (1) a core.ask (2) another task that, if the condition is true, answers the core.ask via the API.  Well, there's a race condition: the core.ask action pauses the entire workflow, so (2) never gets to run...
Aaaargh.  Any ideas?

To illustrate:

caller workflow begins

workflow begins
in parallel:

wait for events from database pertaining object X (using core.ask)
query database to see if object X has already had its event

return to caller workflow

do stuff

If the query returns negative, noop.
If the query returns positive, tell core.ask "yes, you may continue" (via the ST2 inquiry API).
So, if the query returns negative, core.ask remains waiting until the trigger calls the action that tells core.ask "you may continue".  Then the trigger happens sometime in the future.  Then core.ask gets answered, then workflow returns to parent workflow, then continues to "do stuff".
Sensor trigger -> core.ask respond works fine.
The problem is when the query returns positive, and core.ask respond from the same workflow happens.  Sometimes this works.  Sometimes absurd things happen -- the pause workflow code in core.ask pauses the workflow while the respond task is running, which results in a workflow that has a successfully answered core.ask, BUT the parent is paused!  This is illogical.

FYI TTL parameter in core.ask does not do anything.  I set it to 1 minute and 1 minute has passed... nothing.

The TTL parameter does work, I have tested it recently.
You need to enable Garbage Collection for Inquiries, and if you require very short expiries, you may need to change your garbage collection interval

I have a custom action that calls the API.
I fixed the TTL thing.

any updates here?

@Rudd-O what you done for this scenario??

Nothing.  I moved on from the company where we were using ST2 and I don't use it anymore these days.  I still would like the ability for ST2 to be true-HA but I have no cycles to invest on that.

got it

@Rudd-O @LindsayHill Any ideas on how to do this??


Fixed the error 400 being thrown for when emoji are sent via msteams …

…api.

closing this as https://github.com/robusta-dev/robusta/issues/841 was fixed by microsoft itself


zip() throws (swallowed) error when closed synchronously in next() handler

Describe the bug
After enabling logging for notifications on stopped observables with config.onStoppedNotification I noticed that zip() sometimes throws an error which can't be delivered anymore in case the observable gets closed from within the next handler.
Firefox: TypeError: buffers is null
Chrome: TypeError: Cannot read properties of null (reading 'some')
The error originates from here: https://github.com/ReactiveX/rxjs/blob/081ca2ba7290aa3084c1477a6d4bcc573bf478f6/src/internal/observable/zip.ts#L91
Expected behavior
I would not expect this to happen. I think this case should be handled internally.
Reproduction code
import { NEVER, concat, config, first, of, zip } from 'rxjs';

config.onStoppedNotification = (notification) => {
  if (notification.kind === 'E') {
    throw notification.error;
  }
};

const source$ = concat(of(1), NEVER);

zip(source$).pipe(first()).subscribe();

Reproduction URL
https://stackblitz.com/edit/rxjs-qxrbts?devtoolsheight=60&file=index.ts
Version
7.4.0
Environment
I discovered this in an Angular App (v13.1) which also uses NgRx (v13) but I guess it's not relevant for this issue.
Additional context
I think adding a simple check if buffers is already null before calling buffers.some() should fix the issue. I would be happy to provide a PR with this fix.

I just faced this issue in my code too and it would be great to have the PR merged 😉


App activity slows down, by a lot

I have a request to a server each 5minutes with a simple interval function.
When it background mode, the app works, I get the notification, except that rate is not 5minutes, it just very long and random.

@gp187 The plugin tries it best but it also depend on the OS and phone, memory, power, ....


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 971 ms

Resolved: Bofrost.be performance has improved in ba8369a.


SetLang get NullRefrenceException in ctor

if we use following code in ctor we get null refrence exception
 public App()
        {
            GlobalData.Init();
            ConfigHelper.Instance.SetLang(GlobalData.Config.Lang);
        }

also this code get null refrence exception too
 public App()
        {
ResourceHelper.GetTheme("HandyTheme", Resources).Skin = skin;
        }

shouldn't have done this here. the right place is "protected override void OnStartup(StartupEventArgs e)"

@ghost1372
What if i am having a WPF Window (triggered by a button click) from within an Microsoft word add-in. In this scenario, I am getting the same error as above (null reference).
Where should the SetLang be at that point?


Feat: job search bar

Describe the feature
the job search bar is located right above the image which is not looking very presentable it can be moved down a bit
I hope you can assign me ...
Add ScreenShots
it can rather look like this....

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@AKB2102  Its Ok there !!


Validate shape of cacheMap on initialization

Addresses https://github.com/facebook/dataloader/issues/82
Checks to see if custom cache map has the required methods on initialization. The documentation just says it needs to be an object with a similar API to Map but the DataLoader only uses those 4 methods, so I figured it was okay to selectively check for those.

Coverage decreased (-0.4%) to 99.611% when pulling bdf8e0b1a58b69859e673f7d6cd56567b5ac69eb on sidsethupathi:cache-map-shape into c7f047e8c357feccfc581e069434a243a70036f0 on facebook:master.

Hmm, I'm having some trouble figuring out how code coverage dropped. It looks like the new test should exercise the added functionality, but maybe I'm overlooking something simple.

Coverage remained the same at 100.0% when pulling e323ec62559a61e410839beeeb907b1ee9858df7 on sidsethupathi:cache-map-shape into c7f047e8c357feccfc581e069434a243a70036f0 on facebook:master.

LGTM

Coverage remained the same at 100.0% when pulling f020aed52a515e8e89dc5a39ab74265b07d91933 on sidsethupathi:cache-map-shape into c7f047e8c357feccfc581e069434a243a70036f0 on facebook:master.


Remove state from basic dropdown component

Fixes
https://issues.redhat.com/browse/HAC-1537

Description
The basic dropdown component uses an additional state to manage the selected dropdown item, and ignores the prop value. This caused it to not show the correct value unless it was manually changed.

Type of change

[x] Bugfix

Screen shots / Gifs for design review

How to test or reproduce?
Navigate to the review components page, unit dropdown should be preselected with value form CDQ.

Browser conformance:

[ ] Chrome
[x] Firefox
[ ] Safari
[ ] Edge

/retest


I import a IQ_4XS model but get an IQ1_M

What is the issue?
As the title says, I imported a custom gguf model, which is of the IQ_4XS quantization type. But after importing it, ollama show displays it as IQ1_M. Is this behavior expected? Because I saw in previous issues that support for IQ_4XS has been added, so this confuses me.

the gguf file
OS
Windows
GPU
Nvidia
CPU
Intel
Ollama version
0.4.4

How was the GGUF file created?

How was the GGUF file created?

Since I am not the model creator, I am not clear about the details. I have opened an issue(in chinese) in the model creator's GitHub repository to ask about it.
By the way, before raising this issue, I asked in the Discord channel and a user told me that all models quantized with IQ_4XS seem to be displayed as IQ1_M

So I think this is not an isolated case

Oh forgot to mention the point, this gguf file is directly downloaded from huggingface, the link is right in the previous text
Already verified to run normally, only the quantization level is incorrect, everything else is fine
Hash also matches that on huggingface

Looks like the filetype defs in ollama have strayed from those in llama.cpp:
$ sdiff -b <(sed -ne 's/.*LLAMA_FTYPE_MOSTLY_\([^ ]*\).*= \([0-9]*\),.*/ \2 \1/p' llama/llama.h) <(sed -ne 's/^[ \t]*fileType\([^ \t]*\).*$/\1/p' llm/filetype.go | tail +2   | cat -n | sed -e 's/[ \t][ \t]*/ /g')
 1 F16								 1 F16
 2 Q4_0								 2 Q4_0
 3 Q4_1								 3 Q4_1
 4 Q4_1_SOME_F16					      |	 4 Q4_1_F16
 5 Q4_2								 5 Q4_2
 6 Q4_3								 6 Q4_3
 7 Q8_0								 7 Q8_0
 8 Q5_0								 8 Q5_0
 9 Q5_1								 9 Q5_1
 10 Q2_K							 10 Q2_K
 11 Q3_K_S							 11 Q3_K_S
 12 Q3_K_M							 12 Q3_K_M
 13 Q3_K_L							 13 Q3_K_L
 14 Q4_K_S							 14 Q4_K_S
 15 Q4_K_M							 15 Q4_K_M
 16 Q5_K_S							 16 Q5_K_S
 17 Q5_K_M							 17 Q5_K_M
 18 Q6_K							 18 Q6_K
 19 IQ2_XXS							 19 IQ2_XXS
 20 IQ2_XS							 20 IQ2_XS
 21 Q2_K_S							 21 Q2_K_S
 22 IQ3_XS							 22 IQ3_XS
 23 IQ3_XXS							 23 IQ3_XXS
 24 IQ1_S							 24 IQ1_S
 25 IQ4_NL							 25 IQ4_NL
 26 IQ3_S							 26 IQ3_S
 27 IQ3_M						      |	 27 IQ2_S
 28 IQ2_S						      |	 28 IQ4_XS
 29 IQ2_M							 29 IQ2_M
 30 IQ4_XS						      |	 30 IQ1_M
 31 IQ1_M						      |	 31 BF16
 32 BF16						      |	 32 Unknown
 33 Q4_0_4_4						      <
 34 Q4_0_4_8						      <
 35 Q4_0_8_8						      <
 36 TQ1_0						      <
 37 TQ2_0						      <

ollama is missing fileTypeIQ3_M and fileTypeIQ4_XS should be below IQ4_XS.

Note this is purely presentation, the model you've imported is still IQ_4XS, just the display of the model type was incorrect.

@jmorganca @rick-github
This issue still has residual effects, the quantization levels of models uploaded on ollama.com are not displayed correctly.

Do you have an example?

Do you have an example?

https://ollama.com/CBYellowstone/sakura-v1.0

The correct display should be IQ_4XS

The website code is not part of the ollama project repo, so somebody else will have to fix it.  Have you tried re-pushing the model to see if it changes?

The website code is not part of the ollama project repo, so somebody else will have to fix it. Have you tried re-pushing the model to see if it changes?

Okay, I understand the current situation. I will try to re-upload later.

The website code is not part of the ollama project repo, so somebody else will have to fix it. Have you tried re-pushing the model to see if it changes?

This doesn't work, the problem still exists. Should I open a new issue somewhere?

Open a new ticket in this tracker.


Allow println() to print a new line

Currently, if you only want to print a new line, you have to explicitly say system:println("\n");. It would be better if we can support system:println() to print a new line, if possible.

This is not possible since method overloading is not supported in ballerina.


input and output commands don't work

I've tried to use input and output command to change routing or mute outputs from command line on DDRC-24 with firmware version 1.17. Commands do not make effects even I restart and reconnect to minidsp using their app on MAC. Other commands like source, config work like a charm.

small update, it look like it works but the effect is not visible for the app (I'm using it on Mac) even after force closing the app

The official applications do not read any state from the device (most devices don't support reading back their configuration), they read it from local files where the application is installed. The commands sent are the same as the application and affect the runtime state of the devices directly.
Only some global state (master gain, mute, source, preset) will be reflected in the app.

ok, thank you for clarification. Are there any other (unofficial) apps that handle that ?

It's a device limitation really, no apps can read the full current config from a device. I don't know of any apps that maintain the same xml configuration as the official apps in order to interchange state.


Reading in a mesh file in parallel

Is there a way to read in a parallel mesh file on any given number of processors?
I am refining a large mesh, so I start with a single mesh in serial and refine it once on 24 processors. This gives me 24 dummy.000* files. Now I want to read this parallel mesh into 48 processors and refine it further. Is it possible to do that? I am unable to do all the refinements in 1 go because I run out of memory.

Thanks Veselin.. I found this to be helpful. I also realized that my original mesh was a nurbs mesh so refining that was not as memory consuming, and I was able to do that on my machine (went from 448 elements to 14 million elements).


Training with 96 CPU cores is slower than with 48 CPU cores

LightGBM version: 3.2.1
Using LightGBM's Python Scikit-Learn API

Hello, I noticed that the training time is consistently about 50% slower when using a c5.24xlarge instance on AWS (96 cpu cores, 96GB RAM) than when using a c5.12xlarge (48 cpu cores, 192GB RAM).
Model is created with the following settings:
lgb_c = LGBMClassifier(
    max_depth=7,
    n_estimators=1000,
    reg_alpha=0.15,
    reg_lambda=0.15,
    num_leaves=100,
    learning_rate=0.006,
    colsample_bytree=0.8,
    min_child_samples=20,
    objective='multiclass',
    class_weight='balanced',
    importance_type='gain',
    n_jobs=os.cpu_count(),
)

Training is done with these parameters:
lgb_c.fit(
    X_train,
    y_train,
    eval_set=(X_eval, y_eval),
    eval_names=['eval'],
    eval_metric='multiclass',
    early_stopping_rounds=100,
    verbose=50
)

Eval set is about 15k rows.
Training dataset has about 270k rows, with about 18 categorical features (high cardinality, between 200 and 2000 unique elements each) and 2 numeric (integer) features. Target is a categorical feature with about 300 unique categories.

Typical training time with 48 cores: 20 minutes
Typical training time with 96 cores: 30 minutes

In each case, all cores are 100% busy during the whole time of the training.
I would have expected training time to go down with the number of cores. Could this be a bug or is this behaviour normal? Is there any way to fix this using different hyper parameters or settings for the model?

Hello,
As you increase the number of CPU threads, the following requirements increase :

overhead of multithreading
RAM bandwidth used
cores fighting for L1/L2/L3 caches
maintaining turbo boost clocks
maintaining NUMA node memory coherency (for multi CPU setups)
CPU interconnects bandwidth (for multi CPU setups)
and possibly more depending on the hardware/software setup (ex: kernel, heat, power limits, etc.)

In most cases, you are very likely to hit a large multithreading overhead especially with "only" 270k rows and few features (and potentially lower clock rates from using more threads). In this scenario, it will significantly cost more CPU time to dispatch work to threads, and the cost become large enough to supersede the gain of parallelism. It is a normal and expected behavior.
Note that 100% CPU usage reported in task managers / top (etc.) does not mean 100% proper usage of the CPU.
You may want to check the following: https://github.com/szilard/GBM-perf/issues/29#issuecomment-689713624 but also some of my old detailed benchmarks : https://sites.google.com/view/lauraepp/benchmarks/xgb-vs-lgb-oct-2018 (or for instance https://github.com/Laurae2/ml-perf/issues/6#issuecomment-699937489 for simple example on xgboost) for some examples on multithreading scaling.

Update with more data on training speed vs CPU count

CPU Cores
Training time (minutes)

36
22

48
20

72
27

96
27

Thanks @Laurae2 for the clear and useful explanation.

So does it mean that in this case (same dataset, same hyper parameters), 20 minutes in the training time lower bound? There's really no way to get significantly below this without changing the hyper parameters?
I've also tried using GPU and it makes everything much, much slower on this "small" dataset, especially because of the high-cardinality categorical variables.

@JivanRoquet Thank you for using LightGBM. Could you please try force_row_wise and force_col_wise options to see if the same conclusion holds on both choices?

Hi @shiyu1994 I'm going to try this, thanks for the suggestion.


Documentation incorrect

Dear Team
Please correct the documentation, section
Create a number sequence for the simulated general journal transfer
Go to General ledger > Ledger setup > General ledger parameters.
On the Number sequences tab, set the sequence to Simulation general journal transfer.
There is no sequence to  Simulation general journal transfer option in GL parameters. The user needs to create a continuous number sequence from Organization administration> Number sequences> Number sequences and then attach it to a daily journal name.
Thanks!

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

ID: d2e5b9cc-1005-f2c4-6371-5c509166feb1
Version Independent ID: d3529cb4-e0c6-4c69-f081-c4c2ea43f38b
Content: General ledger simulations (Italy) - Finance | Dynamics 365
Content Source: articles/finance/localizations/emea-ita-exil-general-ledger-simulations.md
Service: dynamics-365-finance
Product: ****
Technology: ****
GitHub Login: @Anasyash
Microsoft Alias: ilyako

@Anasyash - Can you take a look? Thanks.

Hello. When you enable the feature General ledger simulations, you should be able to see the line Simulation general journal transfer


add event bianca

Hi Bianca, the event issue template is not completely up-to-date with the website development.
You might want to add more information. Have a look at some of the other events at https://nfdi4plants.github.io/events/
#199
@blaker42

Hi Dominik,
I already wondered about the template. I added a description for the event like most events have. It should be fine now. Thanks!

Can you do me a favor and take over the PR?

I can not do a pull request (PR)? I only find the information here 'Only those with write access to this repository can merge pull requests.'

What is the state of this PR? It is not described as draft, but seems unfinished?

I've updated to latest event schema with a few missing values.
Please merge, so @blaker42 can take over from main. Thanks


Add version: Microsoft.WindowsSDK.10.0.22000 version 10.0.20348.0

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Mounting 1 file to excel.


Add basic state checking

Possible packages/articles to check out

Motor-FSM
quickcheck-state-machine-distributed
State machine testing

Done in 2ed0dbaf1233ec79ed0801b406ae9fbf4e36e8a4


SVHN format 1 'bbox_lefts' fields become 65535 after cast from int to uint16

After fuel-convert, SVHN format 1 'bbox_lefts' field become 65535, because previously in 'digitStruct.mat' they were -1.

@vdumoulin Mind taking a look at this?

I already committed the pull request for that bug.
I already committed the pull request for that bug.


The horizontal alignment of margin notes in framed theorem-like environments when combined with XeLaTeX

Description
An extra indentation appears only when we combine kaobook with XeLaTeX.
Minimal Working Example

\documentclass[
  a4paper,
  fontsize=10pt,
  twoside=false,
  secnumdepth=2,
  numbers=noenddot,
  fontmethod=tex,
]{kaobook}

\usepackage[framed]{kaotheorems}

\begin{document}

Some text.\marginnote{marginnote 1.}

\begin{theorem}

Some text.\marginnote{marginnote 2.}

\end{theorem}

Some text.\marginnote{marginnote 3.}

\end{document}

Expected behavior (what I got with pdfLaTeX):
The horizontal position of all the margin notes should be the same, like this:

Actual behavior (what I got with XeLaTeX):

I found a solution to this issue here: https://tex.stackexchange.com/questions/269426/wrong-marginnote-positions-inside-a-tikzpicture-with-xelatex


Remove dead clients from roundtripper

Fixing an issue where clients that previously error'd at the quic level were not removed from future requests through the RoundTripper.

I signed it!

As described in #765, this is not the correct fix. Closing, also due to age.


feat(ingest/snowflake): Flag to turn off filtering upstream lineage by ingested urns

Would like to test this in some way, but can't get the snowflake integration test to pass locally for me (it's not producing glossary terms)
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Closing as this filtering is only for downstream tables, oops


Create automated annotation for module identification

Description of the issue
From #862:
An annotation is needed that indicates the modules in which a class / object property is defined in. This holds especially for classes from oeo-shared.
To reduce the effort of maintenance, an automated assignement, e.g. when releasing, would be great.
Ideas of solution
Workflow checklist

[ ] I am aware of the workflow for this repository

The next release is planned for Oct 1st, which would be a great time for introducing such a tool... 😉
Is it complex? @adelmemariani @MGlauer @jannahastings

Intermediate result:
Ability to create automatically an OWL-File that contains the belonging to a module (shared, physical, ...) for every IRI based on the schema:  oeo:OEO_belongs_to_module  .
Uploads coming soon.

Not fully finished for release v1.9.
Since it is an important issue for our workflow, maybe release this in a minor version before v1.10 (May 1st).

The script is finished so far:

[ ] Creating the belonging annotations.

[ ] Checks, if run is necessary.

I'm unsure if the OMN files are still correct.

Current Bugs:

[ ] script doesn't adopt the prefixes of the further ontology automatically

I need an ID for the "OEO_belongs_to_module" -Property, so that we have an "OEO_XXXXXXX". Then we could refer to "belongs_to_module".

Now the branch should be updated.
Some things that are new:

You can use now the prefixes in the ontologies in the 'belongs-to-annotated' ontologies of the script
The decision to which module an entity belongs, will be done by a priority-rule of the following scheme:
highest priority -> shared -> physical -> social -> model -> lowest priority

Great! What is the name of the branch? (And just to double-check, did you push your commit?)

Hmm, I thought I pushed it...
never mind.
The name of the branch is: 'feature-870-create-automated-annotation-for-module-identification'

I think the next step on this Issue should be that the script should use the prefixes in the ontologies.
So that the output ontologies contain entities as OEO:XXXXXXXX instead of the long IRIs.
But if you have any ideas for suggestions for improvement, just write them down.

I changed the milestone, because I don't have it implemented until tomorrow.

@stage1407 could you may give an update on the issue's status?
PR #1111 was closed without merging. Is the feature's development still ongoing?

My work on the issue has rested for the past few weeks.
Due to the fact that Janna left, I had lost sight of it.
I will see that the last problems are eliminated and it can be merged.
The biggest problems with it were last:

there are still wrong annotations
it is still inconvenient to call the script

Suspended until decision on distribution of modules has been made

Suspended until decision on distribution of modules has been made

This means no quick solution, therefore I move the milestone.

After the large OEO restructuring, do we still need this annotation? Maybe in oeo-shared-axioms?

We have 'belongs to module', which is outdated, though. Further, the new protege identifies the module automatically.
Therefore, the issue might be obsolete now. We should also consider removing the 'belongs to module' annotation before the next release.

This issue is not needed anymore, since 1) protege is now able to show the module itself, and 2) the restructuring in #1592. We can finally close it. 🎉


Please capture execution date and time in metadata section in the output json file

Is your feature request related to a problem? Please describe.
I have scheduled pipeline run, which runs daily on pre-prod to check the environment is good. The team needs the consolidated historical report. I have stored the daily run report in s3 and retrieved the report before execution every execution starts and upload the consolidated report to s3. The download report updates the date as the current date and time instead of the actual execution date and time.

Describe the solution you'd like
If we store the execution date and time in output json metdata. Then we won't have this issue.
Describe alternatives you've considered
I have used the Device field in metadata to store the execution date and time
Additional context


FilterLogEventsRequest return latest events first

There's no way to make FilterLogEventsRequest return the latest events first. Even with StartTime defined the filter takes only first 50 streams. I want to see the most relevant (latest) events first and if I'm not satisfied I could ask another (older) 50 streams.
Expected Behavior
FilterLogEventsRequest should look through the latest streams bu default or at least as an option
Current Behavior
FilterLogEventsRequest returns 50 oldest streams.
Possible Solution

Steps to Reproduce (for bugs)

Context
I'm trying to get latest events since StartTime to find exact event has been occurred.
Your Environment

AWSSDK.CloudWatchLogs version used: <IP_ADDRESS>
Service assembly and version used:
Operating System and version:
Visual Studio version: 2017
Targeted .NET platform: Core 2.1

.NET Core Info

.NET Core version used for development:  2.1
.NET Core version installed in the environment where application runs:  2.1
Output of dotnet --info:
Contents of project.json/project.csproj:

This looks like a feature request to the service team. I'll pass the request along.


Add --fix to at-use-no-redundant-alias

Add fix to at-use-no-redundant-alias by removing the redundant alias from the node string and parsing / appending the new node.

@kristerkari Any thoughts? :) Also, could we make a new release after this PR is merged?

Yes, sorry for the delay. I've been crazy busy.
Looks good 👍