Ability to rearrange the analysis screen

This includes one or several of:

a way to collapse/expand the left side
make content fill the screen (move list should expand to the end of the screen or dragged area and also expand vertically to fill the screen)
allow for independent resize of the analysis move list
hide the board completely
unpin the analysis move list into another window

Implemented in 2.1.35

Still to implement a page toggle (not a general one for all windows) to hide the board completely.

Implemented as the 'board' command in v2.1.37


Ingested uniformed data sources for final report work

Hi @mklasby @bgulseren @KBehairy @hasnil please use csv files from bucket=fs as dataset for the final report work

lgtm


[SPARK-37455][SQL] Replace hash with sort aggregate if child is already sorted

What changes were proposed in this pull request?

In the query plan, if the child of hash aggregate is already sorted on group-by columns, we can replace hash aggregate with sort aggregate for better performance, as sort aggregate does not have hashing overhead of hash aggregate. Add a physical plan rule ReplaceHashWithSortAgg here, and can be disabled by config spark.sql.execution.replaceHashWithSortAgg.
Why are the changes needed?

To get better query performance by leveraging sort ordering in query plan.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Added unit test in ReplaceHashWithSortAggSuite.scala.

cc @cloud-fan could you help take a look when you have time? Thanks!

Test build #145594 has started for PR 34702 at commit 6448864.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50066/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50066/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50066/

Test build #145594 has finished for PR 34702 at commit 6448864.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145594/

Test build #145649 has started for PR 34702 at commit a683137.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50120/

Test build #145649 has finished for PR 34702 at commit a683137.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145649/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50120/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50120/

Test build #145672 has started for PR 34702 at commit e8609fd.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50142/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50142/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50142/

Test build #145672 has finished for PR 34702 at commit e8609fd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145672/

This PR is ready for review again thanks @cloud-fan.

Test build #145825 has started for PR 34702 at commit cff1424.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50300/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50300/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50300/

Test build #145825 has finished for PR 34702 at commit cff1424.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145825/

The code change LGTM. Since there are quite some TPCDS queries that get plan changes, can we run a TPCDS benchmark to verify performance improvement?

Since there are quite some TPCDS queries that get plan changes, can we run a TPCDS benchmark to verify performance improvement?

@cloud-fan - sure. Today I ran the TPCDS benchmark (sf=1) on one AWS r3.xlarge (same as https://github.com/apache/spark/pull/26049). I don't see much performance difference compared enabling and disabling this rule:

Benchmark result with enabling this rule
Benchmark result with disabling this rule

Then I tried with sf=5, but the benchmark has task failure with no space left on device, so the benchmark cannot be conducted on single machine.
Do you recommend disabling this rule by default? After adding sort aggregate code-gen, we can do more large scale testing to enable it. WDYT?

Discussed offline with @cloud-fan, we decide to disable the rule by default now. After adding sort aggregate code-gen, a large-scale TPCDS benchmark can be done later.

Test build #145880 has started for PR 34702 at commit 8ce7d27.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50355/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50355/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50355/

Test build #145880 has finished for PR 34702 at commit 8ce7d27.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145880/

thanks, merging to master!

Thank you @cloud-fan and @tanelk for review!


HP SoftPaq Download Manager discontinued

Under Step 5, The section titled "For the HP EliteBook 8560w"
https://docs.microsoft.com/en-us/windows/deployment/deploy-windows-mdt/deploy-a-windows-10-image-using-mdt#for-the-hp-elitebook-8560w
HP SoftPaq Download Manager is no longer available. It has been replaced by HP Image Assistant available at https://www.hp.com/us-en/ads/clientmanagement/overview.html
[Enter feedback here]

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

ID: 1c66f2a3-5be3-e3cf-b90f-c9ce2f6ecf9d
Version Independent ID: c9ed7f55-3770-73a4-5ee4-deee16eb7600
Content: Deploy a Windows 10 image using MDT (Windows 10) - Windows Deployment
Content Source: windows/deployment/deploy-windows-mdt/deploy-a-windows-10-image-using-mdt.md
Product: w10
Technology: windows
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@korzynski . Thanks for pointing out. I will add new link ..

Hello @korzynski ,
please have a look at https://github.com/MicrosoftDocs/windows-itpro-docs/pull/9899
Thank you


Add Maskable icons

Add maskable icons to the web app manifest.
This would make icons on Android home screens look way better.
If you want me to I will open a PR

Thanks for your interest, but to be honest, I'm not really maintaining this project anymore. At this point your best bet may be to fork and deploy to another domain.

is there a fork that someone else is maintaining?
I am looking to improve open source PWAs but I don't know enough about Pokémon to be a good maintainer for a project like this


[Web Components | SEO] Investigate rendering with shadyDOM renderer

User Story

As a [user role below]:
IBM.com pre-render service

I need to:
Pre-render an application using Carbon for IBM.com Web Components

so that I can:
Render and serve to search crawlers for SEO purposes

Additional information

After some research, the rendering of the components using ShadyDOM polyfill may be necessary for the prerender service to load properly
This could hopefully be something that can be leveraged via feature flag, so this is only used by pre-render services and not regular users
https://lit-html.polymer-project.org/guide/styling-templates

Acceptance criteria

[ ] Determine best and most efficient way to implement using the conditional ShadyDOM rendering
[ ] Create issues for work necessary

Closing this out in favor of #6083


Update result for the librispeech recipe using vocab size 500 and att rate 0.8

The best WERs for the librispeech test datasets are

test-clean
test-other

WER
2.42
5.73

ngram_lm_scale
attention_scale

2.0
2.0

using HLG decoding + n-gram LM rescoring + attention decoder rescoring.

Note  that

vocab size is changed from 5000 to 500
att rate is changed from 0.7 to 0.8
The best WER occurs at epoch-77.pt, with --avg=55.

By the way, the pre-trained model is uploaded to
https://huggingface.co/csukuangfj/icefall-asr-librispeech-conformer-ctc-jit-bpe-500-2021-11-09
You can also find instructions there for how to deploy it in C++ with k2.

I just verified that the posted WER can be produced using the pre-trained model from hugging face: https://huggingface.co/csukuangfj/icefall-asr-librispeech-conformer-ctc-jit-bpe-500-2021-11-09
See the documentation https://icefall.readthedocs.io/en/latest/recipes/librispeech/conformer_ctc.html#pre-trained-model
for how to reproduce the WER from the pre-trained model.


fix two syntax errors in python

missing  :  at the end of an else

comparision with two equal signs instead of one

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@af28579). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #7   +/-   ##
=========================================
  Coverage          ?   67.32%
=========================================
  Files             ?       44
  Lines             ?     1723
  Branches          ?        0
=========================================
  Hits              ?     1160
  Misses            ?      563
  Partials          ?        0

Continue to review full report at Codecov.

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


Nested recording rule generation for different SLO time spans?

I have a use case where we have a ton of metrics data. We want to do SLOs up to a 4 week period, but having those queries regularly query 4 weeks worth of raw data is very compute heavy. It would be nice if the generated recording rules for longer time span SLO queries were based on shorter term recording rule time spans. For example, making the generated 4 week SLO recording rule based on the 1 day SLO recording rule instead of the raw data.
Is this possible with the current implementation of Pyrra?

This sounds like Thanos downsampling might be worth looking into.
Right now, Pyrra's recording rules take into account all of the 4w of data. That's how the recording rule works.
I think there was a discussion at some point to split the 4W queries up, but I can't find it right now, and I don't think it went anywhere.


[Low Detail] Low detail mode no longer works for most of Hallowed Sepulchre

Type
Incorrect behavior
Operating System
Linux
OS Version
Arch Linux
Bug description
Low detail mode no longer removes floor decorations, ground objects, etc. for any of the Hallowed Sepulchre except for small parts of the first floor. I can reproduce this with RuneLite running in safe mode.
It used to remove all of them - see he box jonge's video for comparison.
Screenshots or videos
Floor 1, Low detail setting disabled

Floor 1, Low detail setting enabled

Floor 2, Low detail setting disabled

Floor 2, Low detail setting enabled

RuneLite version
RuneLite version: 1.10.39
Launcher version: 2.6.10
Logs
No response

Jagex changed the floors here and in Verzik's room, so low detail no longer works in these places.
Using Ground Object Hider from the hub has been the workaround people have been doing since this was changed a month ago.
cc: https://github.com/runelite/runelite/discussions/18250#discussioncomment-10593711

Cheers. Here's a list of Sepulchre ground object IDs in case anyone else finds it useful (doesn't include the lightning tiles). The first floor is weird in that some sections will look how they used to with low detail mode and some will look pure black, which is how all the other floors look.

List
38142,38141,38135,38149,38150,38146,38143,38145,38132,38133,38136,38137,38131,38129,38138,38139,38140,38147,38391,38389,38390,38148,38163,38164,38151,38169,38167,38168,38165,38154,38155,38170,38159,38153,38161,38162,38160,38158,38156,38171,38172,38388,38386,38387,38157,38185,38197,38202,38395,38396,38201,38397,38198,38199,38205,38206,38253,38248,38252,38240,38254,38247,38250,38244,38246,38245,38225,38221,38222,38394,38392,38393,38229,38230,38223,38226,38217,38211,38209,38213,38216,38215,38212,38219,38218,38187,38189,38188,38203,38251,38227,38653,38228,38220,38242,38236,38233,38235,38234,38214,38283,38296,38295,38406,38407,38300,38297,38285,38405,38299,38291,38307,38319,38327,38316,38309,38320,38324,38321,38402,38323,38404,38403,38312,38313,38314,38343,38347,38344,38333,38334,38331,38350,38346,38398,38399,38400,38351,38345,38337,38335,38332,38341,38338,38339,38134,38194,38196,38204,38328


High CPU usage

Issue
If aider use remote LLM, why is so high CPU usage?
Version and model info
No response

When reporting problems, it is very helpful if you can provide:

Aider version
LLM model you are using
Any stack traces or error messages
A description of what you were doing when the error happened.

Including the “announcement” lines that aider prints at startup is an easy way to share some of this helpful info.
Aider v0.37.1-dev
Models: gpt-4o with diff edit format, weak model gpt-3.5-turbo
Git repo: .git with 243 files
Repo-map: using 1024 tokens

Aider v0.69.1
Main model: o1-mini with whole edit format
Weak model: gpt-4o-mini


Allow specify relative path in transform.conf.json

Is this related to a new or existing Amplify category?
api
Is this related to another service?
No response
Describe the feature you'd like to request
currently if I want to use a custom transformer, I have to create a npm package or specify a absolute path from my local machine:
{
    "transformers": [
        "npm-package-name",
        "file:///Users/XX/Documents/YY/src/infrastructures/amplify/backend/api/PP/transformers/myTransformer.js"
    ]
}

I don't want to publish a custom npm package for my private project, but if I use a absolute path, other team members won't be able to run the amplify push because the absolute path is bound to my local machine.
So the third option could be using "workspace" in package.json, but I don't want to do that for some reasons.
Describe the solution you'd like
use ../ to represent the parent directory(like how npm works) or use <root> to represent the project root.
{
    "transformers": [
        "file://../transformers/myTransformer.js"
        "<root>/transformers/myTransformer.js"
    ]
}

Additional context
Related code:
https://github.com/aws-amplify/amplify-cli/blob/master/packages/amplify-provider-awscloudformation/src/transform-graphql-schema.ts#L102
Is this something that you'd be interested in working on?

[ ] 👋 I may be able to implement this feature request
[ ] ⚠ This feature might incur a breaking change

@yaquawa in this re:Invent 2019 workshop there is a sample on how to build and reference a custom transformer with a relative path, check if that works for you and get back to us.

Thanks @attilah ! Could you point out where is the sample located at?

At the link above ^

@attilah Do you mean this? https://github.com/amazon-archives/aws-reinvent-2019-mobile-workshops/blob/master/MOB405/01 - Finished/validatortest/amplify/backend/api/validatortest/transform.conf.json
but this is not a relative path relative to the file right?

No, it is this line: https://github.com/amazon-archives/aws-reinvent-2019-mobile-workshops/blob/master/MOB405/01 - Finished/validatortest/package.json#L13 that adds the custom transformer - unpublished - package to the project's dependency and since this way it gets installed to the amplify project's node_modules, the relative path ./<package-name> will resolve to that when transformer is looking for the package.
This is the transformer loading code: https://github.com/aws-amplify/amplify-cli/blob/5645c711844b5f9a9af248a31e5e264e0ba1bcdf/packages/amplify-provider-awscloudformation/src/graphql-transformer/transform-graphql-schema.ts#L67-L102 if you want to see how it is resolving the transform.conf.json referenced custom transformers.

Thanks @attilah , but I have already read the source code, and found that it could not locate a transformer js file relative to the file transform.conf.json, that why I created this issue(feature request).

What I want to do is keep the custom transformers in the same directory of transform.conf.json, I don't want to create a lots of directories under my project root just for such a reason.

I leave it as an enhancement then, that is not a use case that we support Today.

Got it, thanks @attilah


Resize Font

Some aspects of the game, may be resized whenever the game window is resized.
Text / Font is usually not one of those things that should be resized. At least not on just one axis, but with the aspect ratio in place.
Perhaps other things should also be resized with aspect ratio preserved?

Is this something we should let the user/programmer handle? Because resizing with aspect ratio preserved isn't something we can do (we can resize it, but how would we move / align it?

I think this is something we should leave to the user.


Toggle sidebar false when navbar dropdown opens in mobile / tablet mode.

Please accept my minor contribution. :+1:

+1
Nitpick: I would maybe change the mutation type to something like CLOSE_MENU, since the action is never really toggling the menu, but always closing it.

@raymondmuller makes sense. Thanks


Proxies

Is it possible to use different proxies per browser instance?
I want to create something so that every instance has a different proxy through which the browser will retrieve information.

you can pass in browser configs via puppeteerOptions.
    const cluster = await Cluster.launch({
        concurrency: Cluster.CONCURRENCY_PAGE,
        maxConcurrency: 2,
        puppeteerOptions: {
            headless: false,
            devTools: true,
            ignoreHTTPSErrors: true,
            timeout: 0,
            args:  [
                    '--no-sandbox',
                    '--disable-setuid-sandbox',
                  '--window-size=1920,1080',
               '--proxy-server=http://localhost:8888'
            ],
            ignoreDefaultArgs: ['--enable-automation']
        }
    });

+1, I also would like to use different proxy to each browser instance.

I found the solution literally a few searches later after I posted the comment.
Anyway anyone else looking for a solution use this: proxy-per-page

I tried puppeteerOptions and perBrowserOptions individually and at the same time and the proxy is completely ignored.

I tried puppeteerOptions and perBrowserOptions individually and together at the same time and the proxy is completely ignored.

I experience the behavior. Have you found a solution to this yet?

@cedricdsc I'm sorry for the delayed response, just got a chance to reply to your question. I did something similar to you but a little different, here's my solution:
I created a proxyServer variable with the proxy server for this instance: const proxyServer = chrome.process()?.spawnargs.find(it => it.startsWith("--proxy-server"))?.split("=")[1] || undefined;
and i changed context to: context = await chrome.createIncognitoBrowserContext({ proxyServer });

That's another way to do it. Good you found it too 👍

Hi all,
I have found a solution for those who may have been struggling with a lack of proxy support per request or per browser in puppeteer-cluster. I was able to achieve this by utilising the proxy-per-page package.
I hope this solution helps others in a similar situation. Please see the example code below for implementation details.
proxies.json
[
    "xxx.xxx.xxx.xxx:xxxxx",
    "xxx.xxx.xxx.xxx:xxxxx",
    "xxx.xxx.xxx.xxx:xxxxx",
    "xxx.xxx.xxx.xxx:xxxxx",
    "xxx.xxx.xxx.xxx:xxxxx",
    "xxx.xxx.xxx.xxx:xxxxx"
]

index.ts
import { Cluster } from 'puppeteer-cluster';
import ProxyList from '../proxies.json';
import useProxy from 'puppeteer-page-proxy';

(async () => {

    const cluster = await Cluster.launch({
        concurrency: Cluster.CONCURRENCY_CONTEXT,
        maxConcurrency: 2,
        monitor: true,
        puppeteerOptions: {
            headless: false
        }
    });

    await cluster.task(async ({page, data: url}) => {

        await useProxy(page, `direct://${getProxy()}`);

        await page.goto(url);
    });

    cluster.queue('https://ipinfo.io');
    cluster.queue('https://ipinfo.io');

    await cluster.idle();
    await cluster.close();
})();

function getProxy() {
    return ProxyList[Math.floor(Math.random() * ProxyList.length)];
}

The end result can be seen in the screenshot:

I have implemented proxy support in my forked version, available at: https://github.com/joone/headless-cluster.


Add support for PATCH  HTTP method

Had a look at the On class:
https://github.com/rapidoid/rapidoid/blob/4893a8f7f8d55af80a996b26558884ff4150fceb/rapidoid-http-fast/src/main/java/org/rapidoid/http/fast/On.java
and it seems to only have support for initial HTTP methods, in particular PUT for updates.
Please add support for the newer PATCH HTTP method as well. Thanks
http://restcookbook.com/HTTP Methods/patch/

Thanks for pointing this out, I will add the missing HTTP verbs (PATCH, HEAD and TRACE).

Resolved and already available in the release 5.0.5.


How to set epoch ?

How can I set the number of epoch ?
I cannot find it in  hparams.py.
Thanks

@mightmay  did you find it?

hi @mightmay . I don't see the epoch
Best regards
PeterPham


Fixed Bug: Food Flavours now get properly ignored

About the PR
This PR fixes a bug in how Flavour Profiles are handled, previously flavours in the toIgnore List were not properly ignored, with this PR they should get ignored.
Why / Balance

I found this out when trying to add a flavour profile to a dead mouse, it always added the bitter flavour note  even though i didnt add this flavour profile.
Turns out the bitter flavour note comes from the uncokkedmouseprotein that should have been ignored.
Technical details

Media

Pictures show that flavour message still work properly:

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

fix: Flavor Profiles now properly ignore undesired flavors.

Changelogs don't support any media, so you shouldn't duplicate it under Changelog. And you can just uncomment the changelog example instead of creating one by yourself that has already led to a missing 🆑 which means your changelog won't be displayed ingame

Hi @LuposX, are you coming back to this PR?


Add Workflow class.

Includes base functionality, allowing you to add steps, and run workflow in serial.
var wf = new Workflow("legal")
    .addStep("eula", showEula)
    .addStep("whatsNew", showWhatsNew);

wf.run().done();

Looks good to me!

:+1:

Awesome! :+1:


npx command error

I ran the npx command in a repo with no eslint configs, it produces an error
not sure if it's expected to error out, or unexpected but thought i'd let you know :)
using yarn v1
%  npx create-sheriff-config
Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y)
verbose: ▪ 'eslint.config.js' file not found. Generating and configuring 'eslint.config.js' file...
warn:    ▪ ⚠  If you have other Eslint configs in your project, remove them.
verbose: ┏ Setting Sheriff with options:
verbose: ┃ [1] {
verbose: ┃ [2]   react: false,
verbose: ┃ [3]   lodash: false,
verbose: ┃ [4]   next: false,
verbose: ┃ [5]   playwright: false,
verbose: ┃ [6]   jest: false,
verbose: ┃ [7]   vitest: false
verbose: ┗ [8] }
info:    ▪ ✔  Successfully generated eslint.config.js file.
verbose: ▪ An already present 'prettier' configuration was found in the project. Skipping '.prettierrc.json' file generation and configuration.
verbose: ▪ No '.prettierignore' file was found in the project. Generating and configuring '.prettierignore' file...
verbose: ▪ Installing 'eslint'...
verbose: ▪ Installing 'eslint-define-config'...
verbose: ▪ Installing 'eslint-config-sheriff'...
silly:   ▪ Detected package manager: yarn
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning "eslint-config-sheriff ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "eslint-config-sheriff > eslint-plugin-etc ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "eslint-config-sheriff > eslint-plugin-etc ><EMAIL_ADDRESS>has incorrect peer dependency "typescript@^3 || ^4".
warning "eslint-config-sheriff > eslint-plugin-etc > eslint-etc ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>error An unexpected error occurred: "could not find a copy of eslint to link in /home/cdiesh/4site_stuff/mailer_watcher/node_modules/eslint-config-sheriff/node_modules".
error:   ▪ ❌  Couldn't auto-install the required packages.
    You have to install them manually yourself.
    Please try to run: yarn add -D eslint@latest eslint-define-config@latest eslint-config-sheriff@latest. Error: Command failed: yarn add -D eslint@latest eslint-define-config@latest eslint-config-sheriff@latest
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning "eslint-config-sheriff ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "eslint-config-sheriff > eslint-plugin-etc ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "eslint-config-sheriff > eslint-plugin-etc ><EMAIL_ADDRESS>has incorrect peer dependency "typescript@^3 || ^4".
warning "eslint-config-sheriff > eslint-plugin-etc > eslint-etc ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>error An unexpected error occurred: "could not find a copy of eslint to link in /home/cdiesh/4site_stuff/mailer_watcher/node_modules/eslint-config-sheriff/node_modules".

error:   ▪ ❌  Couldn't walk up the filesystem. Error: Error: Command failed: yarn add -D eslint@latest eslint-define-config@latest eslint-config-sheriff@latest
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning "eslint-config-sheriff ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "eslint-config-sheriff > eslint-plugin-etc ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "eslint-config-sheriff > eslint-plugin-etc ><EMAIL_ADDRESS>has incorrect peer dependency "typescript@^3 || ^4".
warning "eslint-config-sheriff > eslint-plugin-etc > eslint-etc ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>error An unexpected error occurred: "could not find a copy of eslint to link in /home/cdiesh/4site_stuff/mailer_watcher/node_modules/eslint-config-sheriff/node_modules".

file:///home/cdiesh/.npm/_npx/1cf4107118993e1f/node_modules/create-sheriff-config/dist/index.js:84
  throw new Error(`${isString(error) ? error : String(error)}`);
        ^

Error: Error: Error: Command failed: yarn add -D eslint@latest eslint-define-config@latest eslint-config-sheriff@latest
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning "eslint-config-sheriff ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "eslint-config-sheriff > eslint-plugin-etc ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "eslint-config-sheriff > eslint-plugin-etc ><EMAIL_ADDRESS>has incorrect peer dependency "typescript@^3 || ^4".
warning "eslint-config-sheriff > eslint-plugin-etc > eslint-etc ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>error An unexpected error occurred: "could not find a copy of eslint to link in /home/cdiesh/4site_stuff/mailer_watcher/node_modules/eslint-config-sheriff/node_modules".

    at printError (file:///home/cdiesh/.npm/_npx/1cf4107118993e1f/node_modules/create-sheriff-config/dist/index.js:84:9)
    at autoInstallPackages (file:///home/cdiesh/.npm/_npx/1cf4107118993e1f/node_modules/create-sheriff-config/dist/index.js:125:5)
    at async setDependencies (file:///home/cdiesh/.npm/_npx/1cf4107118993e1f/node_modules/create-sheriff-config/dist/index.js:144:3)
    at async main (file:///home/cdiesh/.npm/_npx/1cf4107118993e1f/node_modules/create-sheriff-config/dist/index.js:477:3)

Node.js v18.14.2

It's ok, i didn't expect to cover every possible usecase for every package manager in existence. I mainly support pnpm because that's what i use, so the experience with it should be smooth.
I implemented the fallback that alerts you to install the dependencies yourself, in cases like this where the automatic installation fails.
I think you can just copy/paste the command that the CLI suggests you and it should work. Or tweak it as you see fit.

awesome, thanks for confirming!

@cmdcolin  maybe the issue could be caused by having in pnpm in the engines field. See the message: The engine "pnpm" appears to be invalid.
I'll look into this this weekend.
I'll reopen this issue until i can confirm what the issue is.

As i thought the issue was with the "engines: pnpm" field.
It should now be fixed with this 4aba8c92a7efdde46a46ee09ca14cc070a9ee14d.
Just update to the latest release and please let me know if it fixed it for you!


Alert to know when service is down?

So I have had a couple of times when one of the containers was stopped for some reason and I only realized this at the end of the day when wanting to review the logs and the drives. The fix was simple - restarted everything.
I was wondering - would there be any benefit to send an alert out when some service is down - could be as simple as email, or via IFTTT or something. I don't know if this is a edge case but would we all benefit from it,
Thoughts?
Thanks.

Actually I'd love to have some kind of status port that I could query and that sends back a simple http 200 or something. Probably not that easy to actively send out a signal because when something crashes there's probably nothing left to send out a signal. But being able to use some kind of server monitoring... sounds interesting.

Some sort of healthcheck route could  be useful especially when running with docker / kubernetes. However, sending an alert via IFTT etc. should not be part of TeslaMate in my opinion. This task should be performed by specialized third-party tools.
I'm not sure if all that is worth the effort though. In the last 5 months I had to restart TeslaMate once. That was because Tesla decided to change the IDs of some cars (https://github.com/adriankumpf/teslamate/issues/90#issuecomment-521005302). That shouldn't happen that regularly ;) In general, TeslaMate is pretty fault-tolerant – in case of an error specific parts of the application are automatically restarted. In the unlikely event of a crash, docker restarts the whole container.

https://serverfault.com/questions/806934/bash-script-for-checking-remote-webserver-health-status
Try this idea. You can run it on the docker host or on any other computer with access to TeslaMate endpoint.

Thanks @virtualm2000 - will have a look. So this happened again - here is what I see. I don't know why.

Exit 1 means that the container wasn't able to start successfully. To get more detailed information about the problem that caused this to happen check out the logs. As mentioned in the comments of your other threads, you can do this with e.g. docker-compose logs grafana.


Revolutionise the Images!

Thrown a few ideas down here in a haphazard way... This doesn't start to address the broader CPG questions around how to structure multi-layered docker builds across the images portfolio

AIP is currently built on top of the CPG driver image (for ease), and runs all jobs in that container. There are several issues with this approach:

the base image is HUGE (over a GB for the Google Cloud SDK alone)
the image would need further expansion to be multi-cloud
substantial reliance on CPG libraries

This is done for convenience, and could be finessed out so that each job uses the minimum container.

Benefits?

Delegating cloud interactions to workflow generation script provides a single location for platform related changes (auth for writes, etc.)

except for code executed inside cpg-workflows?
cpg-workflows reliance will (hopefully) end once the Hail team gets VEP working hail-natively

skinny lil images will speed up workflows
the heavy lifting is done in a X-platform image, making debugging simpler

Issues?

multi-imaging: this approach could result in different containers for:

the workflow generation (needs Hail, cpg-utils, cpg-workflows, google cloud)
querying panelapp (minimal)
annotation (optional | GATK for genomic intervals, BCFtools/GATK for generating sites-only, VEP for actual annotation, driver/Hail for combining all data)
AIP Hail stage (needs Hail, google cloud)
AIP post-Hail (needs AIP installed)

That seems like a lot of complexity, and requires quite a few images to be built for a new deployment. The core issue here is that a lot of the 'heavy lifting' in AIP is done in Hail, which is in turn reliant on the GCP/Azure SDK, so that is a chunky image. It's not really possible to make an end-to-end AIP image that's lightweight and cloud-independent. There's probably some combination that caters to all needs better than the current implementation though.

reading/writing files

not a huge issue, but reads & writes are coded lazily, under the assumption that all containers will have the google cloud SDK
redraft this to have relevant data copied into & out of each job
cloud writes done using hail batch itself
this would also affect the config file - requires manual setting of config path to a local file

The Hail library itself doesn't need the GCP / Azure SDK, so that could maybe serve as a common thinner base image.

not a huge issue, but reads & writes are coded lazily

Do you have an example of how this is implemented currently? Is it using gcloud storage cp or similar explicitly?

Not using console commands explicitly, but a lot of reliance on CloudPath to read/write directly to cloud locations - It wouldn't be many LOC to change this behaviour, but I'd just have to track down all those usages in code and swap them over to expecting a local file.
e.g. here - a direct write to the input path, which in this case is a GCP path, but could just as easily be a {batch.output}

If I understand correctly, cloudpathlib should still work just fine without the SDKs, as it's either using the cloud provider's client libraries or direct REST APIs in the implementation.
That Hail team just recently removed the SDK from their images (https://github.com/hail-is/hail/pull/12526), so maybe we should try to do the same at least for a smaller base image that e.g. the AIP could build on?
It would break scripts that use gsutil directly (like https://cs.github.com/populationgenomics/ancestry/blob/0a25b549c670f6fe06f407852b10c22ff71ad483/scripts/copy_1kg_hgdp_tob_wgs_pca.sh), so maybe we'd still want it as a convenience for the standard analysis-runner driver.
@illusional You might be interested in this as well.

Oh, spicy. So cloudpathlib operates exclusively from env CREDENTIALS, not the gcloud library. That's useful.

DONE


Error while install

After pip install dalle-pytorch i got:  ModuleNotFoundError: No module named 'Cython'
I tried pip install cython, but it didn't help(

Make sure you have version Cython-3.0.11
It worked for me


How to call a function after an element is rendered(mounted event)?

I'm interested in an analogue of mounted in vue. Any ideas on how to implement this?

https://github.com/vanjs-org/van/issues/245
or
https://github.com/vanjs-org/van/discussions/376

#245
or
#376

I want to avoid using setTimeout
And how to track the mounted event in the second example (376)?

Thanks!
vanjs-element / vanE This is something new🤔, I didn't find documentation for vanE

van-element it's just wrapper for customElements.define (code)

Exactly, I found the documentation


Non-whitespace before first tag

I'm consistently getting this error thrown in my debug console shortly after startup when running the extension hosting.
Extension Version: 0.9.1 (using en language)
VSCode Version: 1.8.1
Operating System: Windows 10 x64
Error message:
Error: Non-whitespace before first tag.
Line: 0
Column: 1
Char: A
    at error (C:\Users\[user]\.vscode\extensions\seanmcbreen.Spell-0.9.1\node_modules\sax\lib\sax.js:667:10)
    at strictFail (C:\Users\[user]\.vscode\extensions\seanmcbreen.Spell-0.9.1\node_modules\sax\lib\sax.js:693:7)
    at beginWhiteSpace (C:\Users\[user]\.vscode\extensions\seanmcbreen.Spell-0.9.1\node_modules\sax\lib\sax.js:967:7)
    at Object.write (C:\Users\[user]\.vscode\extensions\seanmcbreen.Spell-0.9.1\node_modules\sax\lib\sax.js:1019:11)
    at Parser.exports.Parser.Parser.parseString (C:\Users\[user]\.vscode\extensions\seanmcbreen.Spell-0.9.1\node_modules\xml2js\lib\xml2js.js:145:29)
    at Parser.__bind [as parseString] (C:\Users\[user]\.vscode\extensions\seanmcbreen.Spell-0.9.1\node_modules\xml2js\lib\xml2js.js:3:63)
    at Request._callback (C:\Users\[user]\.vscode\extensions\seanmcbreen.Spell-0.9.1\out\features\callATD.js:22:16)
    at Request.init.self.callback (C:\Users\[user]\.vscode\extensions\seanmcbreen.Spell-0.9.1\node_modules\request\main.js:118:22)
    at Request.<anonymous> (C:\Users\[user]\.vscode\extensions\seanmcbreen.Spell-0.9.1\node_modules\request\main.js:459:16)
    at emitOne (events.js:96:13)

Repro Steps:

Be in an extension project
Start the extension from the debug pane, wait for extension hosting environment to start
Open the debug console, find error message from above

I'm getting the exact same error. Also using 0.9.1 on VSCode 1.8.1.
My spell.json:
{     "language": "en",     "ignoreWordsList": [],     "mistakeTypeToStatus": {         "Spelling": "Error",         "Passive voice": "Hint",         "Complex Expression": "Disable",         "Hidden Verbs": "Information",         "Hyphen Required": "Disable",         "Redundant Expression": "Disable",         "Did you mean...": "Disable",         "Repeated Word": "Warning",         "Missing apostrophe": "Warning",         "Cliches": "Disable",         "Missing Word": "Disable",         "Make I uppercase": "Warning"     },     "languageIDs": [         "markdown",         "plaintext"     ],     "ignoreRegExp": [         "/\\(.*\\.(jpg|jpeg|png|md|gif|JPG|JPEG|PNG|MD|GIF)\\)/g",         "/((http|https|ftp|git)\\S*)/g"     ] }

To add to it - it also seems as if it is no longer spellchecking either. That's probably obvious, but didn't see it mentioned above.

This extension has had a large number of stability issues based on the availability of the underlying service.  Today I published an update which points users to another extension for Spell Checking.
I'm closing the majority of issues as development on the extension has stopped. Sorry for any inconvenience this caused anyone, the good news is that there are better options out there now that don't require the service.
Sean


Make Fluvio.connect throw exception on rather than Rust panic

Let's make Fluvio.connect return a Result. This will make fail on connect throw an exception that we can catch rather than just a panic.

I think this is implemented with my check.


Ability to get/set eco mode

Ability to get/set eco mode.
Let me know if you have any comments or suggestions.

I'm fine with the PR. One question: Have you tested this on both version of the Purei9 Robot? I'm asking because the Purei9.2 has 3 Power Modes (Eco/Normal/Power) while the old one only has 2. I only have the old one to test.

I can test this on the newer model later today. I have them both nowadays.

I reworked the code because it did not work with the newer model (i got access to one to test). It should now work with both versions, feel free to test. Also, if you want to include this functionality in the homeassistant integration and want me to make a release to pypy, ping me and i'll try to do it.

Appreciate it!
Yes, a release to pypip would be awesome.

Yes, a release to pypip would be awesome.

I've pushed version 0.0.4.

Thank you


Contributor JSON Updated

We are not entertaining this kind of PR, please open a relevant PR. You are just changing JS to JavaScript.  Please re-open it, if you have to really push some important changes.


[R20-1856] - Allow single select dropdowns to be 'deselected'

Issue: https://digital-vic.atlassian.net/browse/R20-1856
What I did

adds an option with the placeholder as the label, mimicking a native select input

How to test

Checklist

For all PR's

[ ] I've added relevant changes to the project Readme if needed.
[ ] I've updated the documentation site as needed.
[ ] I have added unit tests to cover my changes (if not applicable, please state why in a comment)

For new components only

[ ] I have added a story covering all variants
[ ] I have checked a11y tab in storybook passes
[ ] Any events are emitted on the event bus

Can we add a way to opt out of this behaviour, for selects that require a option selected?

I've added a preventDeselect prop to the dropdown that defaults to false.


Atkey validation & parsing

Is your feature request related to a problem? Please describe.
Right now what makes a key valid is not clearly defined. Idea is to come up with the definition and then implement it in the code.
Describe the solution you'd like
The below document captures the thoughts:
https://docs.google.com/document/d/16wQTDo2Lb9G9Pti_LnTc726FlUKO_aMcuIAnkT7iqew/edit?usp=sharing
** Subtasks

[ ] Define a "Valid Key"
[ ] Implement the validation of the AtKey
[ ] Scan every single key in all secondaries to find the complete set of characters currently used, identify all characters which aren't in our allow list and their frequency of use, and then decide for each of those characters whether we want to add to the allow list or not.
[ ] If we end up with existing characters that we do not wish to add to our allow list, then we need to decide what to do. It is certainly possible for us to implement code to allow existing keys which break the rules to continue to exist, be retrieved, and even be updated, while preventing creation of any new keys, but this would be a lot of work.
[ ] Plug the validation into SDK and the server. (Question: Is there a staged rollout required here? e.g. first prevent any new 'bad' keys being created at the server side)

** Sub tasks

[ ] Define a "Valid Key"
[ ] Implement the validation of the AtKey
[ ] Plug the validation into SDK and the server

I added some subtasks to the main issue description following team discussion today

Definition of a valid @key is defined here:
https://docs.google.com/document/d/16wQTDo2Lb9G9Pti_LnTc726FlUKO_aMcuIAnkT7iqew/edit?usp=sharing

[

]

Above diagram should give us an idea into where to plug-in the key validations

@gkc IMO action plan for the implementation.

Implement toString on AtKey and its subclasses to get on the wire representation
Implement validators for various key types. This should be in at_commons.
AtKey should be validated before and after creation
3.1. New builder of AtKey should validate the ATKey before the creation
3.2 Introduce validate() on AtKey to validate after creation
Based on the state diagram plug in the key validations on SDK first then on the server

Agreed

Analysis of the key types and their behaviour:
https://docs.google.com/spreadsheets/d/1EOcF_vznBoKWXxRT8dbeG47RP7wnF30ubwXghganTlk/edit?usp=sharing

Burned down an 8 still a 5 to 8 is remaining. Will carry over to this sprint too.

The AtKey validation implementation is tracked in #394. Closing this issue.


Domain name changed, breaks script

The domain name for ChatGPT has changed (or is in the process of changing) from chat.openai.com to chatgpt.com.
The login screen is still on openai but once logged in, you are redirected to chatgpt.com.

I think you got ab tested, I can still use chat.openai.com. 🤔
And chatgpt.com will redirect me to chat.openai.com. Do you have the source of this news?

I am experiencing the same issue. There is a redirect, but from chat.openai.com to chatgpt.com, and Tampermonkey will then not be able to identify the page. I tried to copy/paste/add/adapt all the match statements to include chatgpt.com, but that does not seem to be sufficient. I got a HTML export menu entry, but no follow-up action.

I see. Then I think this is something really happening.

I just updated the script to make it work on chatgpt.com. Thanks @geocell for testing out the changes needed. 🙏


Update article for updated metrics

There us no metrics like "FileShareQuota" it is "File share capacity Quota". Please update the segment:

Metrics we have:

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

ID: a0a586c4-71fd-e60b-9c4c-1a3eb2443c74
Version Independent ID: 2f3706cc-073b-a310-8da4-2a5a90da6473
Content: Azure Monitor supported metrics by resource type - Azure Monitor
Content Source: articles/azure-monitor/platform/metrics-supported.md
Service: azure-monitor
Sub-service: metrics
GitHub Login: @rboucher
Microsoft Alias: robb

@ashitasharma Thank you for bringing this to our notice, assigning this issue to our document author @rboucher to further look into the feedback and update the document accordingly.

This was fixed some time ago.
#please-close


Caching of FastClass does not work

This issue was found by inspecting a memory leak in Google Guice:
https://github.com/google/guice/issues/1042
These are my findings after digging into the problem:

FastClass.Generator is a sub class of AbstractClassGenerator
AbstractClassGenerator supports caching of generated classes:
https://github.com/cglib/cglib/blob/master/cglib/src/main/java/net/sf/cglib/core/AbstractClassGenerator.java#L116
Maybe I am wrong but the primary lookup key for the generated classes is the FastClass.Generator instance itself
FastClass.Generator does not implement hashCode() and equals()
This effectively means that CGLIB always creates a new FastClass instead of using a cached one

Proposed fix:

Implement  hashCode() and equals() for FastClass.Generator

AFAIK, FastClass.Generator uses type name as a cache key: https://github.com/cglib/cglib/blob/28aa2484f1c4a85ac61eb11cb477040f217b7b0e/cglib/src/main/java/net/sf/cglib/reflect/FastClass.java#L65

OK, I've overlooked that LoadingCache uses a helper for mapping of objects to concrete keys.


mud build commands (tablegen, worldgen) should use config path as cwd

mud tablegen, etc can fail if run outside the contracts dir but still specifying a config path inside the contracts dir
https://discord.com/channels/865335009915961364/1181576857611681812/1181588845691678810

I think this is fixed with latest refactors


feat(daterange): allow clear on selected daterange

What - Added enableClear prop as optional and handleClear function to clear the state.
Why - If we want to clear the selected date range. enableClear prop will help to clear the selection.
How - Added enableClear prop if its true than handleClear function will set the state to null.
Testing - set enableClear prop to true and than select daterange. Click on the cancel button to clear the selection.

#321

Great Job 🥳

Great Job partying_face

Thanks excited to contribute more to this project


add auth support to marathon namer

Summary
linkerd's Marathon namer only supports unauthenticated access to the Marathon API. Marathon may be configured to require authentication, and is always the case in DC/OS Enterprise Edition.
We should modify the Marathon namer to read from a DCOS_SERVICE_ACCOUNT_CREDENTIAL environment variable. This environment variable contains a private key, giving the Marathon namer access to the DC/OS API.
DC/OS Setup
This will require the user to create a DC/OS service account and a DC/OS secret, all prior to linkerd deployment. Full details:
https://docs.mesosphere.com/1.8/administration/id-and-access-mgt/service-auth/custom-service-auth/
Service account and secret creation
# generate a keypair
openssl genrsa -out private-key.pem 2048
openssl rsa -in private-key.pem -pubout -out public-key.pem

# modify keypair files for use in curl
tr -d '\n' < public-key.pem | tee public-key.pem
cat private-key.pem | sed -e ':a' -e 'N' -e '$!ba' -e 's/\n/\\n/g' > private_key.pem

# create service account
# Note that <PUBLIC-KEY> is the contents of public-key.pem, with "\n" added after "-----BEGIN PUBLIC KEY-----" and before "-----END PUBLIC KEY-----"
curl -X PUT -H "Content-Type: application/json" -H "Authorization: token=$(dcos config show core.dcos_acs_token)" -d "{\"description\":\"linkerd test service\",\"public_key\":\"<PUBLIC-KEY>\"}" $(dcos config show core.dcos_url)/acs/api/v1/users/linkerd-service-acct

# add perms to service account
curl -X PUT -H "Authorization: token=$(dcos config show core.dcos_acs_token)" $(dcos config show core.dcos_url)/acs/api/v1/acls/dcos:superuser/users/linkerd-service-acct/full

# create secret
curl -v -X PUT -H "Content-Type: application/json" -H "Authorization: token=$(dcos config show core.dcos_acs_token)" -d "{\"value\":\"{\\\"scheme\\\": \\\"RS256\\\",\\\"uid\\\": \\\"linkerd-service-acct\\\",\\\"private_key\\\": \\\"$(cat private-key.pem | sed -e ':a' -e 'N' -e '$!ba' -e 's/\n/\\n/g')\\\",\\\"login_endpoint\\\": \\\"leader.mesos/acs/api/v1/auth/login\\\"}\"}" $(dcos config show core.dcos_url)/secrets/v1/secret/default/linkerd-secret

secret format:
{
  "scheme": "RS256",
  "uid": "service-acct",
  "private_key": "<private-key-value>",
  "login_endpoint": "https://leader.mesos/acs/api/v1/auth/login"
}

linkerd Marathon config
"env": {
  "DCOS_SERVICE_ACCOUNT_CREDENTIAL": { "secret": "serviceCredential" }
},
"secrets": {
  "serviceCredential": {
    "source": "linkerd-secret"
  }
}

Authenticate to Marathon API from linkerd
JWT Generation
From inside docker/linkerd environment, generate a JWT token
Payload:
{"uid": "my-new-service-acct"}

echo "{\"uid\":\"$(echo $DCOS_SERVICE_ACCOUNT_CREDENTIAL | jq --raw-output .uid)\"}"

Private key
echo $DCOS_SERVICE_ACCOUNT_CREDENTIAL | jq --raw-output .private_key

Headers
{
  "alg": "RS256",
  "typ": "JWT"
}

DCOS_TOKEN Acquisition
Login endpoint
LOGIN_ENDPOINT=$(echo $DCOS_SERVICE_ACCOUNT_CREDENTIAL | jq --raw-output .login_endpoint)

Get temporary DCOS_TOKEN using JWT_TOKEN:
DCOS_TOKEN=$(curl -s -X POST -H "Content-Type: application/json" -d "{\"uid\":\"$UID\",\"token\":\"$JWT_TOKEN\"}" $LOGIN_ENDPOINT | jq --raw-output .token)

Authenticate
linkerd can now connect to authenticated endpoints:
curl -H "Authorization: token=$DCOS_TOKEN" leader.mesos/service/marathon/v2/apps

Open Issues

Determine how to handle HTTPS. Enabling SSL_ENABLED causes a cert failure. The other Universe frameworks set SSL_CERT_FILE, SSL_KEY_FILE, SSL_CA_FILE. SSL_CA_FILE seems to be available on mesos leader but not on the agent nodes. More details at https://docs.mesosphere.com/1.8/administration/tls-ssl/#get-dcos-cert.

$ ls /run/dcos/pki/tls/private/
bouncer.key  cosmos.key  minuteman.key  navstar.key  networking_api.key

$ ls /run/dcos/pki/tls/certs
cosmos.crt  minuteman.crt  navstar.crt  networking_api.crt

$ ls -R /run/dcos/pki/CA/
/run/dcos/pki/CA/:
certs  private
/run/dcos/pki/CA/certs:
ca.crt  cacerts.jks
/run/dcos/pki/CA/private:
ca.key

Is it really common practice to provide secrets via environment variables?  As far as I understand, this isn't really safe since users on the parent OS can read these without appropriate access control (i.e. via ps -E).

This is the recommended process for all DC/OS packages.
code:
https://github.com/mesosphere/universe/search?utf8=✓&q=DCOS_SERVICE_ACCOUNT_CREDENTIAL
docs:
https://docs.mesosphere.com/1.8/administration/id-and-access-mgt/service-auth/universe-service-auth/
https://docs.mesosphere.com/1.8/administration/id-and-access-mgt/service-auth/custom-service-auth/


Color for header description anchor element

Hi, love the theme! Hope you'll consider my PR (or a similar one with a different colour; I must stress that I'm no designer). Merry Christmas! 🎄
Before:

After:

Currently links in the header description are the same colour as the background. Here I've darkened the header description background colour (which is the link colour everywhere else in the theme) to the point at which it meets the WCAG AAA requirements for contrast ratio in large text:

Thanks for your PR. Looks great 👍


Now playing shows up with cached data when Spotify is closed

Hi @paced, I encountered a strange behavior that seems to be a bug.
Steps To Reproduce
I repeatedly tried to toggle now-playing through the macOS menu bar. Also, I made sure Spotify is closed completely then a scrollbar with cached data (the last song played) appeared in now-playing.
https://user-images.githubusercontent.com/11756815/166101347-eb92d001-7b69-428f-952c-ae475cdfc8ba.mov
Expected Behavior
Warn end-user there's no active player or still show loading

Thanks for the report @msudgh. I think the cached/previous song appearing is an Electron quirk. It seems like this was intended behaviour from the previous maintainer (e.g., if the current song can't be determined, just show as "loading") but I am escalating to a bug since it is bad UX as it currently is.
Please note that things are a bit slow on this project due to real-life commitments, but we will get to this when we can.

Indeed, It brought bad UX and a bit confusion.
I understand how an open source project takes time and dont worry about that.
There's no rush for a patch/fix as the bug have low priority
Thanks

Can I suggest a different expected behaviour: no notification

@tgrrr Could you please elaborate what you mean?

In my opinion, the expected behavior when Spotify is closed (or maybe even paused) should be to display no song title at all in the menu bar.

We will keep that in mind for user experience @Wever1985, thanks for the report.


Table Component Selected Row not updating

We are using the Table Component, and on 2 of our screens, it is not updating the Selected Row Highlight.  Please see the screen capture below.  Link is provided.
The Foundset selected Record is updated, but the highlighted row is not.
Servoy Version: 8.1.4  - build 3035
Servoy Extra Components 1.1.4
https://www.screencast.com/t/THt1pwil

Linking this to issue in Servoy Jira
https://support.servoy.com/browse/SVY-11585

Case https://support.servoy.com/browse/SVY-11585 is closed as awaiting input.
I am closing this one as well; if new info on how this can be reproduced appears, please reopen both this Github issue and the JIRA case.


feat: support proxy bridge export

Description

add listener when moduleInfo is ready
support proxy bridge export in onload hook

Related Issue

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Checklist

[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] I have updated the documentation.

@squadronai review

@squadronai review

@squadronai review

@squadronai review


Question faster RCNN anchor and sliding windows

Hello,
I have some misunderstanding regarding faster RCNN and the sliding windows. In fact, the sliding window (3x3x512 kernel for VGG16) is applied on the feature map, but I do not understand how to get the anchor boxes. There are 3 scales : 128², 256², 512². But, does that mean we rescale the sliding window so as to get the different scales? Or we rescale the input image of the convolutional neural network ?
thank you for the help !

Hey @Shiro-LK, I am new to the faster RCNN and i am also having the same confusion. Have you figured it out? I was confused by how a single 3X3 sliding window at each position on the feature map represents 9 different anchor boxes. It seems that, for each position on the final feature map, a 3X3 convolution operation is performed to generate a 512 vector, and this 512 vector is then mapped to a 18 vector representing objectness score for each anchor by 1X1 conv. How does this single 512 represent 9 different anchors?

n

@jqJordan  From what I understood : k (9 anchors) are created for each images. The anchors are resized taking in account the downscale of our network. For example, if we use VGG16, the downscale is 16 so it will not be an anchor of 512 but 512/16.  It is not our 3x3 sliding windows which is our anchors. The 512 vector is a feature map.


Add example showing patterns_dir usage

Describe the feature:
Please add an example showing how to include custom patterns via patterns_dir
Describe a specific use case for the feature:
Complex logstash configs often share custom patterns across multiple filters/groks. It's not clear what the best way to do this is currently. Would be great to have an example.

Hi @prbsparx,
I never used patterns_dir but looking quickly at the doc, it seems to read some pattern config files in a specific directory.
I don't think this is something that you can do easily with Logstash chart.
I would suggest adding your pattern config files in a ConfigMap that you will manage externally, then use extraVolumeMounts value to mount this ConfigMap in your pod.

Also looking for this example/feature. If can't be done currently, maybe this is something that can be added via a PR?


add wildcard to log access

At the moment we assume they're all called tomcat-access-log

resolved in #49


backup,import: count system table rows as rows

PR: https://github.com/cockroachdb/cockroach/pull/44965
From release notes:

Row counts in BACKUP and RESTORE now include rows in system tables. [#44965][#44965] {% comment %}doc{% endcomment %}

No docs needed.


MongoDB Error when Upgrade kubedb-community from v0.15.1  to v0.16.0-rc.0

kubedb-community operator error log:
I0107 02:43:35.889281       1 service.go:77] Patching Service abm-production/abm-mongo with {"spec":{"externalTrafficPolicy":null}}.
E0107 02:43:35.921766       1 workqueue.go:169] intended secret "abm-production/abm-mongokey" already exists
E0107 02:43:35.921839       1 worker.go:92] Failed to process key abm-production/abm-mongo. Reason: intended secret "abm-production/abm-mongokey" already exists
I0107 02:43:35.921859       1 worker.go:96] Error syncing key abm-production/abm-mongo: intended secret "abm-production/abm-mongokey" already exists

I deleted mongodb with spec.terminationPolicy is set to Halt, then deleted Secret "abm-production/abm-mongokey"
Recreated same MongoDB object with same Spec, the error disappeared.

I deleted mongodb with spec.terminationPolicy is set to Halt, then deleted Secret "abm-production/abm-mongokey"
Recreated same MongoDB object with same Spec, the error disappeared.


Server internal error

I've been trying to get the data processing to work, and I've tried on several different computers. Below is the most recent error. It occurs when trying to "Create subset gpkg" for wb-3315280, if that at all matters.
From WSL (Ubuntu):
Internal Server Error
The server encountered an internal error and was unable to complete your request. Either the server is overloaded or there is an error in the application.

In the output directory a folder is made for wb-3315280, and cfe_noahowp_attributes.csv gets created, and that looks correct, but the sub-directory config is empty, and no other sub-directories or files are made.
Any ideas?

It looks like the web server has unhelpfully hidden the actual python error, but it's probably one of two things.

part of the subsetting process calls the ogr2ogr command line tool, so if that's not available on the wsl cli then that could break it
There are some issues creating subsets for catchments that flow into a coastal nexus. It's hard to say without the exact conus.gpkg you're using, but for me, wb-3315280 is not coastal and the subset worked.

So to trouble shoot I'd:

Check the ogr2ogr command works in wsl, if not sudo apt install gdal-bin should install it
Delete conus.gpickle in the data sources folder and restart the app (this file is the network graph and needs to be regenerated if the conus.gpkg is ever changed
Try reinstalling the tool from scratch. I recently updated it and published it to pypi, so you can just do

pip install ngiab_data_preprocess
python -m map_app

and the first time it's run, it will automatically re-download the conus gpkg


ran create-react-app and installed needed dependency

[ ] Replace the Readme with the readme template that is in the frontend repo - Readme

[ ] Add the Code of Conduct - Code of Conduct

[ ] Add the pull request template - Pull Request Template

[ ] Move the CODE_OF_CONDUCT.md and pull_request_template.md into a folder called .github in the root of the project


Stub for CFE_ES_RunLoop default return is false, should it be?

Describe the bug
CFE_ES_RunLoop uses UT_DEFAULT_IMPL which typically returns CFE_SUCCESS, which happens to also equal 0.  CFE_ES_RunLoop returns UT_DEFAULT_IMPL !=0, which causes the default return to be false (0 != 0).  Is that the desired default behavior?
To Reproduce
Steps to reproduce the behavior:

Write a unit test that expects CFE_ES_RunLoop to succeed by default.
Run test, see that is not the behavior.

Code snips
https://github.com/nasa/cFE/blob/8a7dc8f2e434889287973c41d4a5cb4a48f48b44/fsw/cfe-core/ut-stubs/ut_es_stubs.c#L1085-L1089
To run a single loop for a unit test, this is required:
UT_SetForceFail(UT_KEY(CFE_ES_RunLoop), true);
UT_SetDeferredRetcode(UT_KEY(CFE_ES_RunLoop), 2, false);
The main reason this requirement does not make sense is UT_SetForceFail makes one think it should fail, not succeed.
System observed on:
RHEL 7.6
Additional context
If this is the desired behavior, close and disregard this issue.
Reporter Info
Alan Gibson  NASA GSFC/587

Yes, to default to false is indeed desired behavior, because in the typical use-case this function is the conditional on a while loop.  So if defaulted to true one would have an infinite loop if no return code was configured.
Defaulting to false is just an easy way to keep it simple and avoid an infinite loop if the test case forgets to explicitly configure something.  If you want one loop you should be able to just do:
UT_SetDeferredRetcode(UT_KEY(CFE_ES_RunLoop), 1, true)
And nothing else, don't need to bother with "force fail" unless you want many iterations.

Also related to nasa/OSAL#559 (UT_SetForceFail misnomer).

The single loop was an example, so yes many iterations will require the force fail.
Where is the behavior for CFE_ES_RunLoop stub defined outside of the code?  Do all bool return functions for stubs have this same behavior or just this one because of the loop usage?


Some gallery examples are broken

Six of the gallery examples show a "broken" thumbnail.
https://terrapower.github.io/armi/gallery/index.html
Related to #394; some of these examples are calling a sefl._log() with arguments that are consistent with the ARMI logger where a standard python logger is expected.
Traceback (most recent call last):
  File "/home/runner/work/armi/armi/doc/gallery-src/framework/run_fuelManagement.py", line 31, in <module>
    o, reactor = test_reactors.loadTestReactor(inputFileName="refTestCartesian.yaml")
  File "/home/runner/work/armi/armi/armi/reactor/tests/test_reactors.py", line 176, in loadTestReactor
    o = operators.factory(cs)
  File "/home/runner/work/armi/armi/armi/operators/__init__.py", line 52, in factory
    return getOperatorClassFromSettings(cs)(cs)
  File "/home/runner/work/armi/armi/armi/operators/operator.py", line 110, in __init__
    reportingUtils.writeWelcomeHeaders(self, cs)
  File "/home/runner/work/armi/armi/armi/bookkeeping/report/reportingUtils.py", line 201, in writeWelcomeHeaders
    _writeCaseInformation(o, cs)
  File "/home/runner/work/armi/armi/armi/bookkeeping/report/reportingUtils.py", line 85, in _writeCaseInformation
    runLog.header("=========== Case Information ===========")
  File "/home/runner/work/armi/armi/armi/runLog.py", line 384, in header
    LOG.log("header", msg, single=single, label=label)
  File "/home/runner/work/armi/armi/armi/runLog.py", line 163, in log
    self.logger.log(msgLevel, msg, single=single, label=label)
  File "/opt/hostedtoolcache/Python/3.8.11/x64/lib/python3.8/logging/__init__.py", line 1512, in log
    self._log(level, msg, args, **kwargs)
TypeError: _log() got an unexpected keyword argument 'single'

PR #418 resolved this ticket.


Fix issue #4127: [Bug]: Disable cache on web application's index.html

This pull request fixes #4127.
This PR addresses the caching issue for the root path '/' and '/index.html' by modifying the openhands/server/listen.py file. Two new routes have been added for these paths, serving the index.html file from the ./frontend/dist directory. The Cache-Control header has been set to 'no-store, max-age=0' for both routes, effectively preventing caching. This should resolve the problem of the frontend breaking on updates due to caching. The original static file mounting remains intact for serving other static assets. These changes ensure that the frontend will update properly on each refresh, addressing the reported issue without affecting other functionality.
Automatic fix generated by OpenHands 🙌

@rbren what're we thinking for this PR?

I think I fixed this elsewhere


Request: Would love to see an integration of this script as a WP plugin

This script is awesome and I would love to see it as a plugin or any way to integrate it into WP. Unfortunately I don't understand enough WP to do it myself.

Thanks! I don't have any plans to turn this into a Wordpress plugin no.


Apip-609 Host header replacement

Added filter to allow replacement of the Request's header with the value of the "host" element in the "Forwarded" header.
This is not configurable because we expect all the requests to be sent with the actual host value and not the service name that L5D uses for lookup.

@fiore-adrian and @erdody, please review this, I don't have permissions to assign code reviewers.

@erdody, I've made all the requested changes.

Look now @erdody


Pylint error

Issue Type: Bug
Pylint marks as an error the import of functions from other archives. It returns pylint(no-name-in-module). My code works fine, but it marks it as not, and it's very annoying to see that you have an error. Thanks a lot
Pylint marca como un error la importación de funciones desde otro archivo. Retorna pylint(no-name-in-module). Mi código funciona perfectamente, pero lo marca como si no, lo cual es bastante molesto el ver el error eternamente. Gracias
Extension version: 2020.3.71659
VS Code version: Code 1.44.0 (2aae1f26c72891c399f860409176fe435a154b13, 2020-04-07T23:31:18.860Z)
OS version: Windows_NT x64 10.0.18362

Hello @EllyptiumOfficial, thank you for reaching out.
Can you provide the following information:

Python version (& distribution if applicable, e.g. Anaconda)
Type of virtual environment used (N/A | venv | virtualenv | conda | ...)
Jedi or Language Server? (i.e. what is "python.jediEnabled" set to; more info #3977)
Value of the python.languageServer setting
The output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
Some sample code that I could use to try and reproduce your issue.

Does pylint display the same errors if you run it in the terminal directly?
Thanks!

Hello!
Python 3.8.1 32 bit version, no virtual environment. Jedi is enabled. I can't reach the python.languageServer setting. If it helps, I have not touched the default values of the extension. The error is at the pylint precheck, not at the compilation, so at the terminal doesn't spit the error. If you want to test it, do this:

Create the first file

At my case was errors.py and it contained this:
def launch_error(*args):
    try:
        raise Exception

    except Exception:
        lines = ""
        for a in args:
            lines += a
            lines += "\n"

        print(lines)

It only retruns a custom error message.

AT THE SAME FOLDER- Create the second file

At my case I was trying errors.py so its name is test.py and contains this:
from errors import launch_error

def error_test:
    if True:
        launch_error("W U DOING?", "This is an error")

NOW is when pylint says that is an error. NOTE: In the same folder is a __init__py file and the problem dissapears once you delete de init file, so...

I tried reproducing your issue, and the only pylint error I get is because def error_test: should have parentheses since it's a function.
However, once that's fixed I did not have any issues, even with an __init__.py file:

What is the content of the Python panel in the Output tab?

I think I have not explained myself well, and I am sorry. The error occurs in the live analysis of the program, when you import a function from another file in the same folder with an init.py file in it. The error disappears when you delete the init file. Tomorrow morning (here is the 1 AM and you got me on the bed XD) I will pass you some screenshots for you to see it. Thanks a lot!

Here ya go!

Hi @EllyptiumOfficial,
I couldn't reproduce your error with the steps you listed. What I suspect is happening is that when you add an __init__.py file the folder becomes a module, and you have to use relative imports: from .errors import X (notice the . here).
If you don't have an __init__.py file then you're not in a module, so  you can do from errors import X.
Do the following resources help?

https://riptutorial.com/es/python/example/9561/importaciones-absolutas---relativas
http://elornitorrincoenmascarado.blogspot.com/2006/09/python-25-importaciones-relativas-y.html

Also, it is not related to the Python extension: if you run python -m pylint pascal_triangle.py in the terminal directly you would get the same error.

The problem is that if you add the init file with the code that you said to me, is when you get the error. In the init file is all. Later I'll post it to you.

@EllyptiumOfficial At this point this looks like an issue with Pylint and not the extension. If you can show how this is a bug in the extension then we would be happy to re-open this issue and keep helping (i.e. you can show us how running Pylint manually doesn't show this error), otherwise I'm afraid at this point this seems to be a problem between you and Pylint and you will need to ask for help from them or maybe Stack Overflow.


feat(config): Move config to /private mount

Move both the private drupal files and configuration sync into the "/private" mount which will link to Azure Files.
The only question I have is knowing the PVC will mount to /private can I for the config-sync directory with emptydir set to /private/config/sync ?
This was done as a request so that can follow Drupal best practice of placement of configuration files.
https://www.drupal.org/docs/configuration-management/changing-the-storage-location-of-the-sync-directory

There is also a problem that this P.R. fixes against master which is a result of the pgbouncer move as it wasn't lowered enough in the spec so was taking the mounts meant for Drupal.


Upstream some of the backends here

Really excited to see y'all move to ibis 5.1.0!
Thoughts on moving some of the other backends like Teradata and DB2 into the main ibis repo?
We'd love to bring those backends into the main fold!

That would be great! Happy to help with that if needed!


Announcements content

Perform the following tasks for the Announcements and events page:

[x] Review for content design principles
[x] Get final accuracy approval from Christina/Alex
[x] Airtable fields confirmed
Title
Slug
Menu
Page description (short text that appears in bookmarks and social media page shares)
Body content

Loaded content into Airtable with required information.


NetworkCurrencyMosaic namespaceId is wrong

Please fix NetworkCurrencyMosaic namespace Id, it is suppose to be prx.xpx.
Currently still following NEM cat.currency

I'll change it to match proximax mainnet, but I am not convinced it is the right action - IMO that class doesn't belong to the sdk and should not be exported at all. Proper place for this would be the application configuration. Network currency mosaic is chain specific and thus sdk should be agnostic to it - even if we agree that default namespace for network currency mosaic in sirius chain variants is "prx.xpx", other properties (divisibility, mutability, transferability, supply) can be different for other chains.

yes, I agree. I think this class properties should be configurable in some way.
Well, it is really urgent to update it as people starting to use the SDK and it will be huge inconvenient for them to convert the amount manually without createRelative and createAbsolute method
Thanks


Phase 2 Migration of Existing Production JPL AWS Infra to MCP

💡 Description
Phase 2 includes move of production web presence to MCP
⚔ Sub-tasks

https://github.com/NASA-PDS/portal-tasks/issues/89
https://github.com/NASA-PDS/planetary-data-cloud/issues/95
https://github.com/NASA-PDS/portal-tasks/issues/89
https://github.com/NASA-PDS/planetary-data-cloud/issues/97
https://github.com/NASA-PDS/planetary-data-cloud/issues/98

📆 05/2024 status: Pushed to later in build plan due to delays in https://github.com/NASA-PDS/registry/issues/185. This is an internal cost cutting/consolidation measure so no impact on delivery to customers.

📆 06/2024 status: Pushed again to later in build plan due to delays in https://github.com/NASA-PDS/registry/issues/185. This is an internal cost cutting/consolidation measure so no impact on delivery to customers.

📆 07/2024 status: Start delayed an additional couple sprints. This is an internal cost cutting/consolidation measure so no impact on delivery to customers.

@viviant100 @sjoshi-jpl FYI, this is next very high priority task for us. if this is something the SAs can help with let's do it. Let me know if there is any testing or anything you need from me.

📆 08/2024 status: Operations task deferring to B15.1. No impact on delivery to customers.

@sjoshi-jpl FYI, per our schedule, this task is up. we can talk about this next week, but we should start with moving the node data backup accounts to MCP (ping @viviant100 for more info on this) and the web logs bucket. we are currently testing upgrades for the EC2s that should be complete in the next week or so and ready for a migration.

📆 11/2024 status: Started. On Schedule.


Enable CORS

Cross-domain support would allow using API from a browser client.

Note for myself: https://github.com/mschae/cors_plug


On guild_create, calling chunk_guilds offsets heartbeats and doesn't update cache

I'm trying to get the offline members of a large guild. See the code (also below).
  fn guild_create(&self, ctx: Context, guild: Guild, new: bool) {
    if new || !guild.is_large() {
      return;
    }

    info!("Asking for offline members of {} ({})", guild.name, guild.id);
    ctx.shard.chunk_guilds(vec![guild.id], None, None);
  }

Is calling chunk_guilds on guild_create not valid? Otherwise, the cache is not updated with offline members, and guild.members does not contain offline members. I also get this warning:
[13:59:59] [INFO] lalafell_bot::listeners::guilds_ext – Asking for offline members of lala world<PHONE_NUMBER>96185345)
[13:59:59] [WARN] serenity::gateway::shard – [Shard [0, 1]] Heartbeat off; them: 5, us: 3

This is on 0.5.1 and rustc 1.26.0-nightly (2789b067d 2018-03-06).

Any news on this? This is a very, very annoying bug that breaks functionality in my bot. Unless what I'm doing is wrong, this should be looked at.

I apologize for the (very) late reply.
fd77a91 should fix this.


Add chat mute status and command

Adds a chat mute command and teaches inbox.go how to handle muted. Now there's both chat hide and chat mute, but let's clean that up later.
This is the client side of https://github.com/keybase/gregor/pull/276
NAME:
   keybase chat mute - Mute or unmute a conversation.

USAGE:
   keybase chat mute [command options] [<conversation>]

OPTIONS:
   --topic-type "chat"	Specify topic name of the conversation. Has to be chat or dev
   --topic-name 	Specify topic name of the conversation.
   --public		Only select public conversations. Exclusive to --private
   --private		Only select private conversations. Exclusive to --public. If both --public and --private are present, --private takes priority.
   -u, --unhide		Unhide/unblock the conversation

r? @mmaxim
cc @patrickxb I cherry-picked just MessageType_ATTACHMENTUPLOADED. So that I could vendor into my local gregor without it blowing up (because it has code that depends on vendoring from the future). Hopefully this won't get in your way too much if it gets merged first?
cc @cjb Here's this. But it depends on future gregor, so I'll ping you when that gets deployed.

re: cherrypick...looks fine, but I'd really like to get that chat attachment branch in master soon.  Waiting on desktop.
Is this chat mute command urgent?
The protocol change to bump versions is uncharted territory, and would prefer not to add parts of it ahead of time.  But if this is urgent, we can.  I'd like to check with desktop on their status before doing so.

Is this chat mute command urgent?

Having muting working from the GUI (so via RPC, not command line) is a blocker for Chat Blog Post so I'd like to get our side merged in this week, which depends on getting your side merged in first.  Does that answer the question?  I don't personally care about a chat mute command in the CLI.

Ok, well if possible, I'd prefer to merge my branch before this PR (waiting on DESKTOP-2672, which I've been told it is close, so I'm hoping that happens today).
If it doesn't happen today, we can go ahead and do this one first and figure the rest out.

Ok, this will wait for https://github.com/keybase/client/pull/5546 then.
chat mute is not urgent. I put it in to have any to try this stuff manually, and figured it would be useful for @cjb. It's pretty straightforward. Is it getting in the way of something? Too gross CLI design?

I thought you were adding a service endpoint for GUI?  This is solely in client/go/client?

No new service endpoint. This mute kerfuffle has n parts:

Chat db change to add enum value 'muted' for msg_reader.status: (https://github.com/keybase/keybase/pull/1145, needs to deploy first)
Gregor needs to understand how to deal with 'muted' (https://github.com/keybase/gregor/pull/276, needs to deploy second)
This PR which tells the client how to handle muted, and adds chat mute, but most importantly contains the protocol changes.
And the client PR should be merged before the gregor one because protocol lives in client 😢 . To prevent ATTACHMENTUPLOADED-type problems happening to the next person who tries to update the protocol in gregor.


Feature/add js

Did this incorrectly. Ran into an error on CLI when doing pull origin main. Had to run a fast forward? What exactly did I Do?
Not sure, but I was able to "Compare & Pull" like normal afterward!

The commit -m message was not there anymore though


Generate report of key marker expression for GABAergic clusters

e.g. Is LAMP5 expressed in all subclusters of the LAMP5 node (CS202002013_125), and a what level (some stats?  % cells in node, mean expression level?)
Highest priority = genes named in grouping clusters of GABAergic neurons
LAMP5
alpha7
Pvalb
VIP
Sst
Scng
But we should cover all genes in names attached to clusters in the dendrogram

(Can you clarify what the specific ask is here?  I don't think alpha7 is a gene...)

Took this from alpha7 Lamp5 GABAergic cortical interneuron -  text def has "strongly labelled for α7 nAChRs" - but I guess may be antibody / multiple genes.

@jeremymiller - have updated spec a bit to clarify.  I think you mentioned being able to provide a heat map of expression of key marker genes for all nodes.  That would work nicely.  The aim here is to judge whether transcriptomics grouping nodes like the one named Lamp5 should be Lamp5-like in the ontology.

If you or @BAevermann provide me with a list of markers, then I can provide the heatmap.  My guess is a list of these NSForest markers (or other markers) already exist somewhere.  Thanks!

These are not NSForest markers, they are genes in names of clusters/dendrogram nodes.
Could you start with
LAMP5, Pvalb, Vip, Sst & Scng.  If you need ensembl IDs, we have them here: https://github.com/obophenotype/brain_data_standards_ontologies/blob/master/src/templates/ensmusg.tsv

@shawntanzk  - can you provide a complete list of genes names in cell-set preferred aliases?

whats the best format? I already have it sort of written out when looking at developmental nature of genes, but thats in a word document

Any text-based format (including a word document) is fine.
From: Shawn Tan @.>
Sent: Tuesday, March 23, 2021 9:03 AM
To: obophenotype/brain_data_standards_ontologies @.>
Cc: Jeremy Miller @.>; Mention @.>
Subject: Re: [obophenotype/brain_data_standards_ontologies] Generate report of key marker expression for GABAergic clusters (#54)
whats the best format? I already have it sort of written out when looking at developmental nature of genes, but thats in a word document

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fobophenotype%2Fbrain_data_standards_ontologies%2Fissues%2F54%23issuecomment-805025583&data=04|01||f1efa500a8264d1b0dff08d8ee151c7f|32669cd6737f4b398bddd6951120d3fc|0|0|637521121739628816|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=%2Bve8NPyDlAqEc8PLOc8nF6Q%2BnVLgtXx7%2FLT3XpyW7tg%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGCOKV5TAR7C3SUBLMI5DP3TFC3SVANCNFSM4Y6QO3EQ&data=04|01||f1efa500a8264d1b0dff08d8ee151c7f|32669cd6737f4b398bddd6951120d3fc|0|0|637521121739638769|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Dl7eqeq96gXumnyLUEZNCQOT3WtRkgJU84nnPPVvIFc%3D&reserved=0.

Gene Markers.docx
Sure thing, see attached for word document with list of markers

Thanks!  So - for the GABAergics it looks like there's a reasonable case for making VIP-like and Lamp5-like grouping classes, as they both subsume a couple of leaf nodes where mean expression and fire plot have expression at or barely above level seen in other GABAergics.   We could still potentially have CL GABAergic classes defined by Vip and Lamp5 expression, populated based on assertions of expression on leaf nodes.  OTOH, sst looks safe &  sncg also probably OK (overall expression is much lower).  @jeremymiller - will heatmaps for key markers such as these be available in cell type cards?  I'll be more comfortable with crude qualitative assertions if users can see the real data too.

We will be including marker gene plots as part of the cell type cards, although the specifics of which genes will be included could be adjusted to address these kinds of issues.  Nik Jorstad is the content owner for this.
From: David Osumi-Sutherland @.>
Sent: Tuesday, March 23, 2021 11:19 AM
To: obophenotype/brain_data_standards_ontologies @.>
Cc: Jeremy Miller @.>; Mention @.>
Subject: Re: [obophenotype/brain_data_standards_ontologies] Generate report of key marker expression for GABAergic clusters (#54)
Thanks! So - for the GABAergics it looks like there's a reasonable case for making VIP-like and Lamp5-like grouping classes, as they both subsume a couple of leaf nodes where mean expression and fire plot have expression at or barely above level seen in other GABAergics. We could still potentially have CL GABAergic classes defined by Vip and Lamp5 expression, populated based on assertions of expression on leaf nodes. OTOH, sst looks safe & sncg also probably OK (overall expression is much lower). @jeremymillerhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjeremymiller&data=04|01||4b37cb8ff38a47eac7d208d8ee282e72|32669cd6737f4b398bddd6951120d3fc|0|0|637521203631743194|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=IIXnuZmtJYBnIWDGl7V1DqLzXpx8OWQqwk9KQ7Ijj48%3D&reserved=0 - will heatmaps for key markers such as these be available in cell type cards? I'll be more comfortable with crude qualitative assertions if users can see the real data too.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fobophenotype%2Fbrain_data_standards_ontologies%2Fissues%2F54%23issuecomment-805125403&data=04|01||4b37cb8ff38a47eac7d208d8ee282e72|32669cd6737f4b398bddd6951120d3fc|0|0|637521203631753147|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=K6rvxolzr4q%2BwiOXx41SwSsMX8LBhGqTcR%2BOCnJVpqQ%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGCOKV6DDF3SCM6VSYICHKTTFDLSTANCNFSM4Y6QO3EQ&data=04|01||4b37cb8ff38a47eac7d208d8ee282e72|32669cd6737f4b398bddd6951120d3fc|0|0|637521203631763102|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=mJRyLiysUFRkc3mK2N%2FTYlT8BRK53l20yj%2FvxZGDF6g%3D&reserved=0.

@dosumis have just gone through the fireplot and trimmed mean heatplot and honestly only lamp5 really has an issue with Lamp5 Egln3_1 expressing Lamp5 at a level that is really close to baseline.
VIP has a couple that have low VIP expression which get wiped out in the trimmed means plot, but you can see the difference in the fireplot.
Interesting thing from this for me is that there are nodes that express other classes marker - eg Sncg Col14a1 clearly expresses Vip.
On the contrary, nodes like Sst Pvalb Etv1 barely expresses Pvalb (though it is higher than the other Sst, just really low amounts - might need to clarify what the values on the plots are actually to resolve this)

but you can see the difference in the fireplot.

But that fire plot shows that very few cells express above baseline in the cluster for those 2 cases.  I'd say these are at least borderline worth getting more opinions on - or coming up with some way to derive cutt-offs from the data.

Interesting thing from this for me is that there are nodes that express other classes marker - eg Sncg Col14a1 clearly expresses Vip.

Another good reason to make a VIP-like class with clear tagging that it is defined by transcriptomics.  A property based classification + some automated call based on tresholds might reasonably pull this additional node under a node defined as VIP expressing GABAergic.

@shawntanzk  - might be worth talking with mouse phenomics group at EBI.  They have lots of experience with designing pipelines for automating extraction ontology assertions from data.

I do like the idea of finding some cut-offs from the data and doing automated call based on the threshold - we should get some experts input on this, will add it to my notes and see if we can figure this out.

might be worth talking with mouse phenomics group at EBI.

Happy to, would you mind helping connect me with them? Thanks


Housecleaning

Remove deprecated functions

from mtl, Control.Monad.Error has been deprecated in favor of Control.Monad.Except. We replace all the Error's with Except's and bump mtl to version 2.2.2, as Except was added in mtl-2.2.1, and mtl-2.2.2 works for any base>4, so bumping it is a formality anyway.
just have time>=1.5 in the cabal file, as support for 7.10 has been dropped anyway.
parseTime has been deprecated, so it is replaced with parseTimeM True
remove old-locale as it is unneeded.

Drop support for ghc<8.4

require base>4.11
test for ghc 8.4-8.10
test for stack lts 16.16 and 14.21
bump cabal used to 2.4
drop alex and happy from travis because they don't seem to do anything
using ghc>=8.4 also allows us to drop a bunch of imports that no longer do anything.

@iyefrat This looks good! Can you update the changelog too ?

@psibi How's this? I only wrote the user facing changes, is there anything else you think I should add?

Thanks!


Feature requested.

Can you add the cloud storage space.

Total space(capacity) storage of the cloud service.
How much space has been taken already.
Available space.

This would require rclone to know the total storage of the remote which it doesn't at the moment.
You can do 2 with rclone size remote:

thanks didn't know that.

I'll re-open this so we can do this some time!

OK here is probably the final version of about before I merge it - comments appreciated!
https://beta.rclone.org/v1.40-089-gef3bcec7-about/ (uploaded in 15-30 mins)
See the docs here: https://github.com/ncw/rclone/blob/about/cmd/about/about.go#L44
This is supported for local, crypt, cache, drive, swift, hubic, onedrive, pcloud, dropbox

Nick, worked perfectly:
Gdrive:
Total:   30G
Used:    20M
Free:    28.440G
Trashed: 0
Other:   1.541G

Dropbox:
Total:   1.009T
Used:    398.448G
Free:    635.177G

Onedrive:
Total:   12G
Used:    89.062M
Free:    11.913G
Trashed: 0

Thanks for this feature!

rclone about is merged to master now:
https://beta.rclone.org/v1.40-090-g2b855751/ (uploaded in 15-30 mins)
Thank you for testing.


DOC: polars cheat sheet

Documentation problem
Make a polars cheat sheets (pdf). In addition, improve the user experience with a streamlit and a google colab.
Suggested fix for documentation
I have developed a repository with several cheat sheets in pdf, streamlit and google colab format, in case you are interested in improving the learning experience of this library.

Topic
PDF
Streamlit
Google Colab

polars

Note: The PDF format cheat sheets included here are authored by other contributors and have been used as sources of inspiration for the content presented.

Hi @fralfaro ,
That's a nice cheatsheet, but I don't see how it would fit in with our documentation right now.
Furthermore, we're in the process of moving this repo into the main Polars repo, and merging this would add additional complexity.
I do encourage you to share your cheatsheets with others and hope it helps other people work with Polars. Maybe we can include a link to your repo somewhere.

Hi @stinodego ,
Great, if you need anything, you can use my material to spread Polars content. On the other hand, if you need help with anything, please contact me :)


Add test for search_anchor.0.dart

Contributes to https://github.com/flutter/flutter/issues/130459
It adds a test for

examples/api/lib/material/search_anchor/search_anchor.0.dart

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] I followed the breaking change policy and added Data Driven Fixes where supported.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@piedcipher
Thanks for reviewing this.
Gentle reminder that reviewers should be careful about details: in this PR there are 3 occurrences of the following format error: widget.color ==const Color(0xff6750a4) (missing space).
Also consider letting the author adding the autosubmit label:

It will give time for a last check
For many contributors it is a pleasure to add the autosubmit label because it is the last step of their work.

@bleroux Sorry for (once again) a formatting issue 😥
I can make a PR to fix that

@piedcipher Thanks for reviewing this. Gentle reminder that reviewers should be careful about details: in this PR there are 3 occurrences of the following format error: widget.color ==const Color(0xff6750a4) (missing space).
Also consider letting the author adding the autosubmit label:

It will give time for a last check
For many contributors it is a pleasure to add the autosubmit label because it is the last step of their work.

I noticed this but I was unsure if we should've space. I'll be mindful about this. Thank you

@ValentinVignal @piedcipher
No worries, we all make mistakes and those ones were very very minor. My main point was about not applying the autosubmit label as it might help the author or someone else to give a last look.
Thanks for all your contributions :pray:


Remove * from valid version characters

:page_facing_up: Rendered version
We recommend to remove * from valid Version characters, due to confusing intersection with
VersionSpec glob expressions.
We suggest different possibilities on how the character could be reinterpreted in VersionSpec
with an analysis on backward compatibility.

Hi all,
How can I move this forward? Is there a meeting I should attend to champion this?


stack-controller should watch TriggerBinding and TriggerTemplate

Watch and reconcile changes/deletions to owned TriggerBinding and TriggerTemplate, as is done for Pipeline & Task to ensure integrity.
Granted, #483 needs to be fixed

We've already discussed this internally, but I'll just write it down for other's awareness.  We can't fix this issue until we have a cluster scoped operator.  This will probably happen when we implement #284 .

I am going to lump this into #284 for now.  However I'd like to start by keeping the stack controller namespace-scoped, and just resolving the issue as it's described above, by watching objects in the other namespace.  We can continue to have a discussion about whether the stack controller should be namespace scoped or cluster scoped.    My thinking on this right now is as follows:

@dacleyra discovered that it is possible to have the manager cache for multiple namespaces (see https://github.com/operator-framework/operator-sdk/blob/v0.17.x/doc/user-guide.md#manager).  The Kabanero/Team controller should be able to set an env var in the stack controller that tells it what to watch.  In theory this should be enough to resolve this issue.
Having a cluster scoped Stack controller would make it impossible to start a version of the stack controller that differs from the version of the Kabanero operator, or at least would make it impossible to have two different Kabanero/Team CR instances at different versions of Kabanero.  There are downstream requirements that the operator be capable of installing a certain number of previous versions of Kabanero, to allow users to upgrade at their own pace.
The stack controller interacts with some of the other versioned components.  For example as-is you can say spec.version: "0.9.0" in your Kabanero CR instance, and you get the "0.9.0" versions of the CLI service, landing page, events operator, and stack controller.  The stack controller works with these other components.  If we forced the stack controller to be the version of the kabanero operator, it might not be compatible with version "0.9.0" of the events operator.  This is a somewhat arbitrary line since one could say the kabanero/team controller might not be compatible with the stack controller, but that same incompatibility would apply to every versioned component and in the event it was not compatible, the kabanero/team controller would just not allow you to install that version.

So to fix this issue, the kabanero/team controller needs to set an environment variable in the stack controller deployment/pod, telling it what additional namespaces need to be watched.  For the time being it's the tekton-pipelines namespace, since that's where the trigger bindings and trigger templates are.  In the future it will also be the pipelinesNamespace but we're not there yet (#230).


Incompatible types in assignment

zo_back/authentication/models.py:35: error: "UserManager" has no attribute "_db"
zo_back/authentication/models.py:65: error: Incompatible types in assignment (expression has type "BooleanField", base class "AbstractBaseUser" defined the type as "bool")
zo_back/authentication/models.py:87: error: Incompatible types in assignment (expression has type "str", variable has type "EmailField")
zo_back/authentication/models.py:87: error: Argument 1 to "normalize_email" of "BaseUserManager" has incompatible type "EmailField"; expected "Optional[str]"
zo_back/authentication/models.py:98: error: "CharField" has no attribute "upper"
zo_back/authentication/models.py:99: error: "CharField" has no attribute "upper"
zo_back/authentication/models.py:101: error: "CharField" has no attribute "upper"

I have bunch of issues related to models. Mypy expect CharField instead str, don't found .upper() method of EmailField and lot of other issues related to models fields, because it expect field instances instead it's python representation (stored value).
Can you give me advice why its happen (basically I understood but I interested in your opinion) and what to do.

Any suggestions?

Hey @Skorpyon, sorry for no replies. Have you also tried asking on https://gitter.im/mypy-django/Lobby?
There isn't so many people working on this right now.


Fix circle.xml, square.xml

Configure circle and square to pass CI.

Mea culpa -- I left an error in the XML that caused the build to fail, and didn't catch while CI wasn't available.
THIS PR fixes it once and for all, and returns us to a passing baseline: behold the Travis green checkmark of approval.
This should also mean that the site can now be re-deployed, so that all these changes are live.

I've deployed. I notice that the circle reference doesn't have an example. Is that intended?

Not intended, no -- slipped through when I was restoring the build. The xml was invoking tests that don't exist. I will correct the example and add the image generation back in, sans test generation.