Can't use math expressions using '|' in Markdown tables (remark-math/KaTeX/GFM)

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[ ] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
| $1$       | $2$       |
| --------- | --------- |
| $|3| = 3$ | $|4| = 4$ |

will be rendered like:

The '|' inside the latex expression is mistaken for a separator for the sheet.
Reproducible demo
No response
Steps to reproduce
write
| $1$       | $2$       |
| --------- | --------- |
| $|3| = 3$ | $|4| = 4$ |

inside a doc.
Expected behavior
The '|' in the latex expression is processed correctly.
Actual behavior
The '|' in the latex expression is mistaken for a separator for the sheet.
Your environment

Public source code:
Public site URL:
Docusaurus version used:latest
Environment name and version (e.g. Chrome 89, Node.js 16.4):Microsoft Edge, Node.js 18.12.1
Operating system and version (e.g. Ubuntu 20.04.2 LTS):Windows 11

Self-service

[ ] I'd be willing to fix this bug myself.

Hey
This is already fixed for Docusaurus v3 with MDX v2. You'll have to escape the pipes with \| in math expressions.
| $1$         | $2$         |
| ----------- | ----------- |
| $\|3\| = 3$ | $\|4\| = 4$ |

I'm not sure there's a good way to do that under Docusaurus v2 (MDX v1). My suggestion would be to wait for Docusaurus v3, it should move to beta very soon.

Hey
This is already fixed for Docusaurus v3 with MDX v2. You'll have to escape the pipes with \| in math expressions.
| $1$         | $2$         |
| ----------- | ----------- |
| $\|3\| = 3$ | $\|4\| = 4$ |

I'm not sure there's a good way to do that under Docusaurus v2 (MDX v1). My suggestion would be to wait for Docusaurus v3, it should move to beta very soon.

That will be awesome! Thanks! But why is there double vertical line?

That's a good question 😅 any idea @wooorm ?

Hmmm, not sure if it's a bug 😅
But it seems Katex receives the escaping char \

$\|3\| = 3$

---

$|3| = 3$

---

| $1$                      | $2$                      |
| ------------------------ | ------------------------ |
| $\|3\| = 3$              | $\|4\| = 4$              |
| $\|3\| = 3$ | $\|4\| = 4$ |

Would you be ok to add remark-math to the MDX playground options @wooorm ? That could be useful to debug this kind of issues.

It should work the same as code. Code doesn’t get two pipes.
Normally escapes don’t work in code. But in tables, GH supports it, and does not show the slash.
Same with math.
yeah, having math in the mdx playground is fine!

That's what the MDX playground reports with GFM + math turned on:

To me, the \| is unexpected there, and it looks like a bug in https://github.com/micromark/micromark-extension-math right?

Well, maybe using \vert  instead of | can be a solution?

https://katex.org/docs/supported KaTeX will render \| as \Vert, which is double pipes, and this behavior is different from MathJax does, for MathJax will render \| as a single pipe. GitHub uses MathJax.
| $1$       | $2$       |
| --------- | --------- |
| $|3| = 3$ | $|4| = 4$ |

$1$
$2$

$
3

Well, maybe using \vert instead of | can be a solution?

Yes, except maybe the former is more verbose, they are exactly equivalent.
\| becoming \Vert is consistent with LaTeX; I bet MathJax doesn't implement it this way because it would be too confusing in many JavaScript contexts (such as the case here!)
I don't think there's anything we can help, though—all parsing work is delegated to remark-gfm and remark-math.
So the issue here is this: anything inside math is literally preserved—there's no character escaping because \ is a very meaningful thing in LaTeX. However, when tables get parsed, it doesn't understand anything about other Markdown syntax—it even extracts pipe characters from inline code spans. Therefore the two parsers are in conflict and I don't see a way to reconcile them without fundamentally changing how one of them works. Avoiding the pipe character altogether may be the best solution.

Markdown parsing is delegated entirely to MDX and its plugins. There's nothing we can act on here.

Markdown parsing is delegated entirely to MDX and its plugins. There's nothing we can act on here.

This question could be troublesome now, for MDX v2 doesn't allow {} and <> any more, so now writing KaTeX could be very hard. For example, $\dfrac{4F}{2}$ can cause a compilation error.
Is there a solution? Thank you for your reply.

@dbxxx-oi Works for me in the playground. Are you using the latest version of remark-math instead of what's shown in the docs?

https://github.com/mdx-js/mdx/issues/1462 It seems that it's not a mdx problem

@dbxxx-oi Works for me in the playground. Are you using the latest version of remark-math instead of what's shown in the docs?

Thank you for your reply! No, I'm not using the latest version, sorry for that mistake.
I updated remark-math to 6.0.0 and rehype-katex to 7.0.0, and rewrote docusaurus.config.js according to this.
This is my docusaurus.config.js:
// @ts-check
// Note: type annotations allow type checking and IDEs autocompletion

const {themes} = require('prism-react-renderer');
const lightTheme = themes.github;
const darkTheme = themes.dracula;
import remarkMath from 'remark-math';
import rehypeKatex from 'rehype-katex';

const webpackAssetSourcePlugin = () => ({
	configureWebpack() {
		return {
			module: {
				rules: [{
					test: /\.ggb$/,
					use: [
						{
							loader: 'file-loader',
							options: {name: 'assets/ggb/[name]-[hash].[ext]'}
						}
					]
				}]
			}
		}
	}
});

export default {
	title: 'whk wiki',
	tagline: 'whk wiki 致力于产出优秀的文化课资源',
	favicon: 'img/favicon.ico',

	url: 'https://whk-wiki.github.io',
	baseUrl: '/',

	trailingSlash: false,
	onBrokenLinks: 'throw',
	onBrokenMarkdownLinks: 'warn',

	i18n: {
		defaultLocale: 'zh-Hans',
		locales: ['zh-Hans'],
	},

	presets: [
		[
			'classic',
			({
				docs: {
					routeBasePath: '/',
					sidebarPath: require.resolve('./sidebars.js'),
					// Please change this to your repo.
					// Remove this to remove the "edit this page" links.
					editUrl:
						'https://github.com/facebook/docusaurus/tree/main/packages/create-docusaurus/templates/shared/',
					remarkPlugins: [remarkMath],
					rehypePlugins: [rehypeKatex],
				},
				blog: false,
				theme: {
					customCss: require.resolve('./src/css/custom.css'),
				},
			}),
		],
	],

	stylesheets: [
		{
			href<EMAIL_ADDRESS>			type: 'text/css',
			integrity:
				'sha384-odtC+0UGzzFL/6PNoE8rX/SPcQDXBJ+uRepguP4QkPCm2LBxH3FA3y+fKSiJ+AmM',
			crossorigin: 'anonymous',
		},
	],

	themeConfig:
		/** @type {import('@docusaurus/preset-classic').ThemeConfig} */
		({
			// Replace with your project's social card
			defaultMode: 'dark',
			image: 'img/docusaurus-social-card.jpg',
			navbar: {
				hideOnScroll: true,
				title: 'whk wiki',
				logo: {
					alt: 'whk wiki logo',
					src: 'img/logo.svg',
				},
				items: [
					{
						type: 'docSidebar',
						sidebarId: 'introSidebar',
						position: 'left',
						label: '开始',
					},
					{
						type: 'doc',
						docId: 'home/math',
						position: 'left',
						label: '数学',
					},
					{
						type: 'doc',
						docId: 'home/physics',
						position: 'left',
						label: '物理',
					},
					{
						type: 'doc',
						docId: 'home/chemistry',
						position: 'left',
						label: '化学',
					},
					{
						type: 'docSidebar',
						sidebarId: 'biologySidebar',
						position: 'left',
						label: '生物',
					},
					{
						href: 'https://github.com/whk-wiki/whk-wiki',
						label: 'GitHub',
						position: 'right',
					},
				],
			},
			docs: {
				sidebar: {
					autoCollapseCategories: true,
					hideable: true
				},
			},
			footer: {
				style: 'dark',
				copyright: `本文档由 Docusaurus 搭建．`,
			},
			prism: {
				theme: lightTheme,
				darkTheme: darkTheme,
			},
		}),

	plugins: [
		// @ts-ignore
		webpackAssetSourcePlugin
	],
};

But when I try to build, an error occured:

C:\Users\xbc\Desktop\programming\Projects\whk-wiki\docusaurus.config.js:7
import remarkMath from 'remark-math';
^^^^^^
SyntaxError: Cannot use import statement outside a module

I'm poor in react-js and I don't know too much on ES Modules. I've tried several solutions but they didn't work. Is there anyway to solve it? Appreciate to your reply again!

Try this:
const {themes} = require('prism-react-renderer');
const lightTheme = themes.github;
const darkTheme = themes.dracula;

module.exports = async () => {
  const remarkMath = await import('remark-math');
  const rehypeKatex = await import('rehype-katex');
  return {
    // Your config object
  };
};

Further support should use the discussions board instead of this already-closed issue.

ES config file works now and simplify usage of remark plugins:
import remarkMath from 'remark-math';
import rehypeKatex from 'rehype-katex';

export default {
  presets: [
    [
      '@docusaurus/preset-classic',
      {
        docs: {
          path: 'docs',
          remarkPlugins: [remarkMath],
          rehypePlugins: [rehypeKatex],
        },
      },
    ],
  ],
};

Not sure how newer versions of plugins will work with Docusaurus (this is third-party code and we can't guess future changes will always be compatible), but for us those versions work fine: remark-math@5 rehype-katex@6


Sponsortoken:  x509: certificate has expired

Hi,
there is a slight issue with the sponsortokens ....
[main  ] FATAL 2021/05/30 12:47:00 sponsortoken: rpc error: code = Unavailable desc = connection error: desc = "transport: authentication handshake failed: x509: certificate has expired or is not yet valid: current time2021-05-30T12:47:00+01:00 is after 2021-05-30T07:58:05Z"

Confirmed. I'll check asap.

Should be fixed, sorry for the interruption.


Create A Login System

Build a landing page to login to the site.

Docsify isn't meant to be a full-page application with auth routes and etc. It's recommended to have and authorization and authentification routes on the main https://tekperfect.com page


Feature Request: Add a column for Change/Index

I spent some time trying to understand the transaction window and how that relates to the items in the "UTXOs" and "Transactions" panes. Sparrow is unparalleled here.
I think it would be really useful to allow an optional column that had the "/" for the transactions to aid in finding them. So, for example  "0/4" would identify receiving address 5, and "1/72" would indicate the 73rd change address.
It might even be useful in the "Addresses" pane. especially if "Hide Empty Used Addresses" is on. Saving time from needing to hover and wait for the tooltip.
Btw, is this the correct forum for feature requests? New to this.

I have considered this previously, but feel it it would take up too much space/overwhelm the average user.
That said, I agree it's very useful information, so I have decreased the show delay for these tooltips from 1 sec to 250 milliseconds. 78a6ce4


Update README.md

Add some more info

(note that i currently work over the ui, so I cannot squash. Maintainers should be able though to squash this via the UI, if the repo is setup this way)


eShopOnAbp: Triggering data seed without schema migration

Currently, for EF Core, data seeding works only if there is a pending migration. However, there may be cases where there is no pending migration, but we need to seed database.
Possible solution: Run seed on every startup (optional) and manually trigger it in the admin side.

This is fixed after moving to distributed locking for db migrations. IDataSeeder seeds independent of pending migrations.
https://github.com/abpframework/eShopOnAbp/blob/3a4767c6cd307a3741562b58389d4bde026f450d/shared/EShopOnAbp.Shared.Hosting.Microservices/DbMigrations/EfCore/PendingEfCoreMigrationsChecker.cs#L61-L78


potential memory leak in pipeline::start method (callback overloads)

Required Info

Camera Model
any

Firmware Version
n.a.

Operating System & Version
n.a.

Kernel Version (Linux Only)
n.a.

Platform
n.a.

SDK Version
master

Language
C++

Segment
core

Issue Description
The code inside pipeline::start methods (overloads accepting callback) is dangerous and could be a potential source of memory leak:
        template<class S>
        pipeline_profile start(const config& config, S callback)
        {
            rs2_error* e = nullptr;
            auto p = std::shared_ptr<rs2_pipeline_profile>(
                rs2_pipeline_start_with_config_and_callback_cpp(_pipeline.get(), config.get().get(), new frame_callback<S>(callback), &e),
                rs2_delete_pipeline_profile);

            error::handle(e);
            return pipeline_profile(p);
        }

Problem statement:
The ownership of the memory allocated for the frame_callback object is passed to rs2_pipeline_start_with_callback_cpp function however there are control flows where the memory will not be properly released:

argument evaluation order is unspecified and config.get() may throw an exception while memory is already allocated
there is a case inside the rs2_pipeline_start_with_config_and_callback_cpp implementation when an exception may happen before ownership of the passed memory is accepted.

Hi @tomaszmi
Thanks, your point is correct.


closes #272: Added tooltip to dev-notice icon

also super minor point, but if you make a pr to pull an issue could you include in the description a very short summary on what you have done, and the words 'closes #272' so that it automatically marks the issue as under review

@chrismclarke Incorporated the comments. Please check and let me know your thoughts.

great thanks, looks good, I see it's still interfering a little bit with the rest of the page layout (adding a small bar at the top) but I'll merge this for now and add a minor edit later

(updated 4f73a47702845a7a78cde43ad6d8eb215248bf0f)

thanks


Adds a triage job that triggers if pax MGMN or t5x MGMN tests fail

This is a large addition so the highlights are summarized below:

Adds a new _triage.yaml workflow that gets called if T5x MGMN or Pax MGMN tests fail. It relies on a :latest-verified tag as the last known working image.
The latest-verified tag is added to the vanilla t5x and pax images if their respective tests pass
Adds Dockerfile.ff which creates a new image from a BASE_IMAGE where all (or some) of the dependencies are fast forwarded to the versions in BROKEN_IMAGE. The image layer produced by this is small since no recompilation happens. Thus, this dockerfile is currently only suitable for fast-forwarding libraries with python source only (e.g., t5x, paxml, praxis).

Additionally, a ff-summary script is added to help inspect the source offline.

To signal failure or success, a new job called outcome was added to _test_t5x.yaml and _test_pax.yaml in order to detect if the overall test was a success
Adds some utility functions for workflows in this file: .github/workflows/util.sh; which can be reused on the command line or in other workflows. (Was helpful in debugging the triage workflow)
Removes the endpoint artifact upload in _publish_badge.yaml since it was causing collisions on multiple re-runs of the same workflow in a matrix. The artifact isn't really needed anyway since we upload it as a gist and it's printed in the logs.
Files a github issue if there are failures:

For pax: https://github.com/NVIDIA/JAX-Toolbox/issues/215
For t5x: https://github.com/NVIDIA/JAX-Toolbox/issues/218

Should we update some user doc about this?

@nouiz Added a user doc here: https://github.com/NVIDIA/JAX-Toolbox/blob/triage-tool/docs/triage.md

Here are two "sandbox" runs to demonstrate what example triaging runs look like:

for t5x: https://github.com/NVIDIA/JAX-Toolbox/actions/runs/6087387492
for pax: https://github.com/NVIDIA/JAX-Toolbox/actions/runs/6089563249

I've also updated the triage tool to auto file a github issue if there is a failure. Here are two examples:

For pax: https://github.com/NVIDIA/JAX-Toolbox/issues/215
For t5x: https://github.com/NVIDIA/JAX-Toolbox/issues/218

I have questions about the created github issues:
Does this create issue only for nightly and not for PR?
If the same failures happens for many days, does it creates new issues? Append info to the existing one?

Also, how does it select who to assign the issue to?

Also, how does it select who to assign the issue to?

Right now it's hard-coded: https://github.com/NVIDIA/JAX-Toolbox/pull/208/files#diff-2ad6ab3e3b9d04131794c79a52e5f18dc271a4bcbf5d6c08694a956e1a48e287R51-R54

Does this create issue only for nightly and not for PR?

It only creates it for the nightly. The pre-sumit CI for PRs will not trigger the triage or the github issue filing.

If the same failures happens for many days, does it creates new issues? Append info to the existing one?

If the failure goes on for many days, it will create new issues for each day it fails. Another option is to have one issue, and then it can be closed if an engineer deems it as "fixed", and then the triaging workflow can re-open it and add a comment with the new "summary table".
I thought creating new issues would be preferred to keep conversations self-contained since each issue may be different; but the tradeoff is there's more housekeeping everyone has to do to make sure our issue page is not cluttered with these bot triages.

SG to me as a start. Clearly better then what we have now.
We could just check each day if the bug is the same and close the extra issue as duplicate.
If we are able to fix them fast enough, it shouldn't be too much overhead.

The two failures can be ignored:

CI / test-pax / outcome: is not caused by this PR, but rather surfaced b/c of this PR. So it's result should not block this PR
CI / build-rosetta-t5x / build: is known to be broken on main b/c the TE PR in t5x has a merge conflict which we are resolving as we speak.


Show move index on stones

[Mac OS X] Is it possible to show the move index on every stone? Or is there any switch in the app?

No, it's currently not possible to do that. There are some considerations to make when implementing:

How do we handle custom SGF markers?
How do we handle two moves on the same place?
What about the option to only show the last n move numbers?

Here are my suggestions.

How do we handle custom SGF markers?
You mean triangles, squares, ABCD.. ? We can either write the move number first and draw the markers on top. Marker first and number on top. Lets make the move number font small so that both are visible.

If we want to show only one when both are present, We can put an option on menu with "Toggle Move Number" and "Toggle SGF Markers".

How do we handle two moves on the same place?
The gogui ignores the old move and only shows the last move number at the position. We can either choose that way. Or in settings we can have an option to allow user to choose whether to hide the last move or show at the bottom of the board like traditional kifu. Example,  112 : 39, 83

What about the option to only show the last n move numbers?
For this too, we let up to user to decide in settings, if they want the numbering to be exact move number or starting at 1 since the last move back.

How do we handle custom SGF markers?
You mean triangles, squares, ABCD.. ? We can either write the move number first and draw the markers on top. Marker first and number on top. Lets make the move number font small so that both are visible.

Drawing the markers on top will obscure the move number and vice versa. Especially with the cross marker and custom labels. Not good. I'm interested how to make the move number font small so that both are visible. Please note that a move number can consist of three digits. Furthermore, how to distinguish custom number labels with move numbers?

How do we handle two moves on the same place?
The gogui ignores the old move and only shows the last move number at the position. We can either choose that way. Or in settings we can have an option to allow user to choose whether to hide the last move or show at the bottom of the board like traditional kifu. Example, 112 : 39, 83

To be honest, I don't see a use case for move numbers on stones. Since you have the whole game opened already, you can just navigate through the game to get a feeling of placement order. Tracking stones with move numbers only requires a lot of mental effort and I find it very difficult to concentrate on the play itself. The only reason to use move numbers that I can think of is, if you want to convey the move order on a static medium, like a kifu or image.
Should Sabaki get the feature to print or export to an image in the future, I'll gladly add the option to automatically assign move numbers.

I think one of the workaround solution is to toggle on the game tree which shows the move number (from the first move) at the top right corner.  but I guess the request was about showing numbers for a sequence moves as it is usually used in some books, so that when the author of the book says move 7, we can know exactly which one he is talking about.  if we can have a feature to select/mark a certain stone as #1, and then the subsequent moves automatically show their corresponding numbers, that'd be very useful in reading some of the books that come with SGF files.  I think it'd be best to show the numbers of the remaining stones, and if one needs to he can move back a few moves to see which stones were killed.

显示步数非常的重要。建议参考下multiGo实现的规则
http://ruijiang.com/multigo/chs/

Showing move numbers is very important for users. Is there any plan to implement it?

Care to elaborate?

I think there's a grain of truth in this discussion: the current move number is difficult to see / find.
If you don't have the game tree open, you can't see the move number at all:

If you do have the game tree open, the move number is displayed on the scrollbar of the game tree:

This is difficult because:

By default, move number is not visible
It's a small number as the scrollbar, instead of being highlighted as an important focus item
It moves while moving in the tree, so you have to actually "hunt for it" with your eyes


url() refactored with path() and re_path()

From Django 2.0 path() and re_path() are being used and url() will be soon deprecated. django.conf.urls.url() will be removed will be from Django 4.0.

Thanks for this!
I created another PR to resolve conflicts: https://github.com/st4lk/django-rest-social-auth/pull/140

Thanks for this!
I created another PR to resolve conflicts: https://github.com/st4lk/django-rest-social-auth/pull/140


Typo

Great page but funny typo.  Where it should say "unsecure pages" it says "insecure pages" and I'd hate for my page not to feel comfortable.

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

ID: ef31dc71-3610-6c18-2bda-d0bb70804c92
Version Independent ID: 02f7dc7b-541b-4b82-b33c-4e974fb94d8c
Content: Xamarin.Forms WebView - Xamarin
Content Source: docs/xamarin-forms/user-interface/webview.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Fixed in a PR that will go live later this week. I guess the usage of insecure came from Apple (they have a key used in ATS named NSTemporaryExceptionAllowsInsecureHTTPLoads).


Main to Live sync - Jul22

Org changes
May'22 release doc change

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

File
Status
Preview URL
Details

microsoft-community-training/breadcrumb/toc.yml
:white_check_mark:Succeeded

microsoft-community-training/media/Allusersorg.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/bulkuserorg.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/filterorg.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/orgcontactadd.png
:white_check_mark:Succeeded
View

microsoft-community-training/user-management/organization-management.md
:white_check_mark:Succeeded
View

microsoft-community-training/whats-new-in-microsoft-community-training/2022/whats-new-in-may-2022.md
:white_check_mark:Succeeded
View

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


Provide better error message when service creation fails with default and whitelist registry specified

**Rancher Version:**1.2.0-pre4-rc4
**Docker Version:**1.12.2
**OS and where are the hosts located? (cloud, bare metal, etc):**Cloud
**Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)**Single node
**Environment Type: (Cattle/Kubernetes/Swarm/Mesos)**Cattle
Steps to Reproduce:

Create a cattle environment
Add quay registry in rancher with valid credentials
Add the default and whitelist registries as below:
default registry: quay.io
whitelist registry: quay.io
Create a service test with image "index.docker.io/ubuntu"

Results
Service creation fails with the following message:

A better/relevant error message which is helpful to the users should be provided.
[The service creation fails as expected as the "index.docker.io registry" is not present in the whitelist registry]

Tested with master. An improved message is provided for the above issue. Message below:
Validation failed in API: The provided registry is not whitelisted is not a valid option


network Headers

applying headers to play function
to pass Referer:  key

from my research i found it can be added like this ./vlc.exe --http-referrer='http://example.com'
should i just put it in vlc controller constructor?

i was able to add referrer using this code
tmp=VLCController(args:["-vvv","--http-referrer=${VARIABLE FOR THE REFERRER LINK}"]);
thanks


.env file doesn't get resolved

See https://docs.docker.com/compose/environment-variables/#the-env-file
Example:

what Compose version are you using?

Do you have a test repo that we would be able to test and implement this with? From what I understand, we would only be able to implement this for .env files locally hosted on the developer's computer, but it is something we can add as an improvement

what Compose version are you using?

$ docker-compose --version                                                                     [22:57:27]
docker-compose version 1.21.0, build unknown

Do you have a test repo that we would be able to test and implement this with? From what I understand, we would only be able to implement this for .env files locally hosted on the developer's computer, but it is something we can add as an improvement

Sure I'll set one up and link it here once it's ready. And of-course it's only for local .env file. docker-compose automatically looks for .env files in the same directory as the docker-compose.yml file

Awesome! I may go the route of reading the .env file specified in env_file but will need to do more research.
Thank you for your feedback and welcome any contributions/additional feedback

env_file is entirely different from .env, it's not what this issue is about.
.env is used for variable substitution inside the docker-compose.yml file. This file does not affect the container.
However,  env_file is a per-container-configuration for applying environment variables inside the container itself.

Thanks for the clarification. I see what we have to do now and the fix should be rather quick!
Thank you,
Michael

@omertuc fix is in. please test and let us know if you have any issues.

Will look into optimizing handling env file


Update hypopg extension to version 1.4.0

The v1.4.0 includes changes to make it compile with PostgreSQL 16. The commit log doesn't call it out explicitly, but I tested it manually.
v1.4.0 includes some new functions, but I tested manually that the the v1.3.1 functionality works with the v1.4.0 version of the library. That means that this doesn't break existing installations. Users can do "ALTER EXTENSION hypopg UPDATE" if they want to use the new v1.4.0 functionality, but they don't have to.

Thanks!


fix: fixed subscriptions for token balances

In current implementation, we are creating a query subscriptions in the effect hook of the App component to listen for balance changes. Subscription callback functions are depending on the global redux state which they also modify. Therefore a callback has to be modified when the balance state changes. However, we are not able to modify the callback of existing subscriptions and can only create new query subscription with different callback because of the limitations imposed by the polkadot.js api package.
The problem of current implementation is that we are creating new subscription with modified callback with each state change, but are not closing the previous subscriptions. As a result, ever-increasing amount of subscriptions is created with each run of the effect hook which ends up freezing the whole dApp: with each balance change all of these subscriptions are calling the redux reducers which changes the redux state and triggers creation of new subscriptions.
To solve this issue, I moved the callbacks for unsubscribing into React refs and added a call to unsubscribe each time when new subscription is created (not only when the component is unmounted). Also, I split the hook into 3 data-specific ones to enforce that new subscription is created only when related data change.
Note: Ideal solution would be to create subscription on the first render only and then just modify the callback function, but that is not possible now, because the @polkadot/api does not expose this functionality.

Good solution @peterslany - any idea what caused this to become an issue now? It's clearly not something new, so I'm wondering what caused it to become noticeable over the last couple of days.

@tomjeatt I think it became noticeable just because of the testing. Another reason could be that number of the effect hook triggers increased. However I'm not aware of any recent change that could cause this - it would have to be a change to one of the former hook dependencies. :)


[objc] Add a header (embeddinator.h) that's meant to be included by generated headers.

This way we can limit the API surface of the generated code.

Build and tests passed.


fix: support POST for retry-axios

Retry does not emit when POST request is called in the Node SDK. This is happening because the configuration of retry-axios defaults to the ['GET', 'HEAD', 'OPTIONS', 'DELETE', 'PUT'] verbs (source).
I added a test to make sure retry is happening when POST is called. This test would fail without the augment.
Checklist

[x] npm test passes (tip: npm run lint-fix can correct most style issues)
[x] tests are included
[ ] documentation is changed or added

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

:tada: This PR is included in version 2.14.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Images are bridged to IRC as URLs which force a download

Messages with "msgtype": "m.image" are sent to IRC with an URL such as https://matrix.example.com/_matrix/media/r0/download/matrix.org/xxxxxxxxxxxxxxxxxxxxxxxx/image.png. However, Synapse serves these endpoints with Content-Disposition: attachment; filename=image.png. This instructs web browsers to offer to save the image to disk, instead of simply displaying it.
This makes it annoying for IRC users to see images posted by Matrix users.
#227 looks related.
As to how this could be fixed... first, the spec doesn't seem to require any particular Content-Disposition header, nor does it offer any way for clients to request a particular content-disposition. So, Synapse seems to be sending the header on its own volition. Changing this in Synapse might be an option, but another way would be to try to fix this together with #227, which will probably require Heisenbridge to run its own HTTP server. Integrated setups such as https://github.com/spantaleev/matrix-docker-ansible-deploy could simplify setup and configure routes to Heisenbridge's HTTP server without requiring significant effort from users. I see Heisenbridge already uses asynchronous I/O, so maybe throwing in an HTTP server would not be a significant ordeal?

It's a synapse regression https://github.com/matrix-org/synapse/issues/15885, so I'd hope they fix it there

Unfortunately the consensus in https://github.com/matrix-org/synapse/issues/15885 seems to be that the bridge should figure serve content independently :(
It's not a tragedy (and there are workarounds in the link above if your homeserver is behind a proxy that can do some header massaging...) but I agree it's unfortunate.
Just to play with the idea, would there be any other benefit in heisenbridge serving content directly? It's really not a common IRC bouncer use case...

There is a Synapse PR to revert it for some basic types:
https://github.com/matrix-org/synapse/pull/15988

Just to play with the idea, would there be any other benefit in heisenbridge serving content directly? It's really not a common IRC bouncer use case...

Well, it will need to do that anyway to support encrypted rooms.

Looks like this was resolved, I checked after nginx-proxy was ripped from the ansible playbook and I don't hit it anymore. It may have also been fixed by the referenced synapse bug.

I confirm it was fixed before. I also use the ansible playbook, and I have not updated yet (I have traefik in front, but nginx is still behind, so the default config until yesterday).
I don't have the issue.


Fix website code for view lists.

Diamond operator cannot be used with anonymous inner classes, the third parameter of ButterKnife.apply with View.Alpha has Float type and therefore accepts float and not int.

Yep yep. Thanks!


Support customization of query string serializer in redux-resource-xhr

One of my APIs that I proxy through my webserver actually requires a query string that looks something like this:
http://service/path/to/handler?id=1&id=2&id=3&etc...

If you look at the qs lib they allow for configuration of arrays in a few ways, like so:
qs.stringify({ a: ['b', 'c'] }, { arrayFormat: 'indices' })
// 'a[0]=b&a[1]=c'
qs.stringify({ a: ['b', 'c'] }, { arrayFormat: 'brackets' })
// 'a[]=b&a[]=c'
qs.stringify({ a: ['b', 'c'] }, { arrayFormat: 'repeat' })
// 'a=b&a=c'

It seems the current options I have are:

qs-ify my URI prior to passing it in to the action-reactor
transform it on the webserver layer before handing it off to the server

This is due to limitations of the existing qs options we can patch onto the xhr under-the-hood, like so:
const xhrOptions = {
    json: true,
    url: `/api/my/url`,
    qs: {id: ids},
  };

This produces an XHR to a URI that looks something like this by default:
/api/my/url&id=1,2,3

I spoke with @jmeas offline about this, and there's a few ways we can approach this, but wanted to post that here so we could capture the discussion and then once we figure out the desired changes (if any) we can reference it here.
Thoughts?

I know earlier we talked about both the impacts in 'over heard' for a flexible API from a maintenance and API surface area for users, but one thing we could consider adding is allowing qs here to be either:

Something we can pass to our internal qs method and use to build a URI
A function that can be invoked (possibly with other URI strings?) to handle the qs.stringify however users prefer.

Just some random thoughts on it - and I think you mentioned that approach awhile back too.

The API should mirror what's outlined here:
https://github.com/naugtur/xhr/pull/160#issuecomment-292699118
so that this lib is an easy transition to<EMAIL_ADDRESS>

I'd be alright with merging just about any API that implements this. If it follows what xhr plans to add in v3, then that's awesome, but not a requirement. PRs welcome!


Document the pfs net protocol

One of the more annoying issues in working with HDFS in the early days was that the net protocol was entirely undocumented. Things improved a little bit with Hadoop 2, when the protocol was switched over to Protobuf objects, but protocol semantics are still mostly only documented by code.
It would be great if pfs documented the client<->server network protocol and semantics early on. Thanks!

This is a really good point, thanks for opening this issue. I'll make sure to get it in to the next release (meaning v0.5, v0.4 is technically the next release but the dev cycle for that has basically ended.) Does anyone know if there are really good tools for documenting http apis written in go? Since our server code has such a common structure there might be some nice way to autogenerate the docs and make sure they stay in sync with the actual code.

Even a really rough sketch of verbs, endpoints, and that the protocol uses HTTP would be great. It doesn't have to be the best documentation you've ever seen. Thanks!

Hey @cemeyer, we're redoing the PFS API with protocol buffers v3 and grpc, the current API is here https://github.com/pachyderm/pachyderm/blob/master/src/pfs/pfs.proto, there will be generated HTTP endpoints as well. Would this cover your concerns? Should one of us look into generating documentation based on this if not?

That looks ok. It's really up to you, though :).

I think we will just close this issue for now (I'm all about the
unachievable 0 issues 0 PRs active) and when things are more settled, we
may decide to document this further with some documentation generation
tool. Thanks for the input :)
On Monday, July 20, 2015, Conrad Meyer<EMAIL_ADDRESS>wrote:

That looks ok. It's really up to you, though :).
—
Reply to this email directly or view it on GitHub
https://github.com/pachyderm/pachyderm/issues/38#issuecomment-122745539.


better-manpage follow-up #1

Hi ,
The 1st commit fixes the build issue.
The 2nd commit beatifies the man page by turning off justification and hyphenation.

c30c25559fefa5a176bde89a973e4f6a0c40a2f6 does not work in my case. python -m youtube_dl  still imports the old version if there is one.
b47f82b16c3ef26e69e0a39ff6462910499bb922 looks like a good idea, but could you open another PR if #6852 is merged? I guess it's not a good idea to mix commits from different people in a single PR.
40e3a9a3c0590988b8c6f1b91be8b63da556daab can be handled separately. Could you create a PR for that in the original repo rg3/youtube-dl?

Thanks for the review!
I think c30c255 is not needed anymore, since your PR #6867 has fixed the build issue I encountered.
For b47f82b and 40e3a9a, yes I will open new PR.
By the way, I think b47f82b could be forwarded to pandoc upstream as well, since the man page is generated by pandoc.


Help !  Legosurface Error

My testing voxel  value has 900 zeros and 100 fives.
When I set (vmin=0, vmax=10), the legosurface show me reasonable result.
However, when I set (vmin=1, vmax=10), error occurs as follows.

[vedo.visual:1340] ERROR: in cmap(), cannot find any CellData active array ...skip coloring.
[vedo.visual:1340] ERROR: in cmap(), cannot find any CellData active array ...skip coloring.
2024-07-26 15:14:40.806 (   0.442s) [                ]vtkDemandDrivenPipeline:754    ERR| vtkCompositeDataPipeline (000002045FEB2BA0): Input for connection index 0 on input port index 0 for algorithm vtkImageToPoints (000002045FEC15B0) is of type vtkPolyData, but a vtkImageData is
required.
[vedo.visual:1362] ERROR: in cmap(), nr. of input cells scalars 0 != 1 ...skip coloring.
the source code is as follows:
import numpy as np
from vedo import show, Volume
nx = 11
ny = 11
nz = 11
C0 = np.zeros((nx, ny, nz))
C0[:, :, nz-1] = 5
vol = Volume(C0)
vol.add_scalarbar3d()
lego = vol.legosurface(vmin=1, vmax=10)
lego.cmap('jet', vmin=1, vmax=10).add_scalarbar3d()
show(lego, axes=True).close()

I only wanted to draw voxel with a value of 5.

Hi sorry for the late reply... try
lego = vol.legosurface(vmin=1, vmax=10, boundary=True)

Wow. It is perfect !!
Thanks Marcomusy ~


Create a texture pool to reuse reality mesh textures

This PR is a WIP.
This PR adds a TexturePool class that stores unused textures to reuse them instead of creating new ones. The pool stores textures sorted by dimensions and type, as well as a lru list to efficiently free memory once textures are expired. A maximum limit of bytes used can be set, as well as dimension maximums and minimums.
The pool should be used with the createOrReuseTexture function that behave similarly to System.createTexture and with disposeOrReuseTexture that can be used instead of dispose. Both behave the same way seen from outside whether the texture is reused or not.
For testing only:
For tests, there is a stats class that has a small performance cost but contains useful information. It should be removed before merging. This class allows to know how mean memory usage, how many bytes or textures are reused/created/inserted/purged, how much time we are spending in each case, as well as information for each texture dimension. They are printed in console every 30sec.
Current issues:
For the moment, my tests have shown that using current purge rate of RealityTileTrees, we are only reusing 25% of textures, which could be improved.
The test on my local computer has shown that in every situation, calls to TexSubImage2D seem to take as long as TexImage2D, which is counter intuitive. No major time improvements have been observed. Maybe it varies depending on the driver?

The test on my local computer has shown that in every situation, calls to TexSubImage2D seem to take as long as TexImage2D, which is counter intuitive.

Which browser(s)?


data param attribute does not allow setting key with name submit

If we try to assign the key with the name submit when setting the data-params attribute, clicking the link will do nothing, maybe because it is reserved and set automatically, because as soon as i change the key name from submit to Submit it starts working, but it is not documented anywhere in the docs under the renderTagAttributes()
What steps will reproduce the problem?
<?=Html::a('Submit', ['save', 'id' => $model->id], [
    'class' => 'btn btn-primary',
    'data' => [
        'method' => 'POST',
        'params' => ['submit' => 'my-value'],
    ],
]
)?>

What is the expected result?
clicking the link should submit the page to the desired link
What do you get instead?
nothing happens when clicking on the link
Additional info

Q
A

Yii version
2.0.14?

PHP version
5.6

Operating system
Ubuntu

its actually yii.js issue, for a temporary fix you can override yii.js handleAction method but before doing that i will request you fetch the fresh master release and check. If you still have that issue please create a ticket.

Yii version <IP_ADDRESS>
There is a problem. And it is not at the same time. Rather, it is the expected behavior.
But, there are many questions in general about the appropriateness of using. It would not occur to me to use this way, there are other methods for these purposes.
The matter is that form is transferred from the transferred parameters. Naturally, with the transfer 'submit', the formation of 'input':
<input name="submit">

the solution is simple. Let's check if we have "submit", if there is, then we call the trigger on the first element (This will be our form). otherwise we call as usual.
if (("submit" in params)) {
    $form[0].trigger('submit');
} else {
    $form.trigger('submit');
};

I think that the correction does not make sense and the topic can be closed.

Be careful when naming form elements such as submit buttons. According to the jQuery documentation there are some reserved names that can cause conflicts:
Forms and their child elements should not use input names or ids that conflict with properties of a form, such as submit, length, or method. Name conflicts can cause confusing failures. For a complete list of rules and to check your markup for these problems, see DOMLint.

However, also, I think that it is necessary in such cases it is necessary to output a warning to the console.
console.error("Property 'submit' of object #<HTMLFormElement> is not a function.");


Add GraphQL Relay specifications

TODO:

[x] Object identification
[x] Connections
[ ] schema.js is getting big. Create a types folder. Attempted this but it became very verbose with import/exports everywhere and I think having a single schema file is actually cleaner.
[x] maybe use the Babel Require Hook for the /graphql folder only. It shouldn't affect start up time/performance to much. Node 5 supports quite a few ES6 features but I'm missing destructuring for example. And our code is not consistent with all the other GraphQL/Relay examples across the web which use full ES6 features.

:grinning:


Can not read negative Values

Hello,
So far, thank you very much for that library. It helps me a lot.
But I have a problem reading negative values from DB.
E.g. I know that in the address area DB331.12.0 (DB331.Offset.Bit) is a values stored like -12 from Type "DINT", I get "-2147483648" using:
S7.Net.Plc spsPLC = new S7.Net.Plc(CpuType.S7300, "<IP_ADDRESS>", 0, 2);
object Result = spsPLC.Read(S7.Net.DataType.DataBlock, 331, iOffset, S7.Net.VarType.DInt, 1);
long iNewValueInt = System.Convert.ToInt64(Result);
sValue = "" + iNewValueInt;
return sValue;
If the values in the DB are positve it is working fine.
Do you may have an idea?
Thank you.
Stefan

System.Convert.ToInt64 is not correct.
You have to use the conversions provided by the library.

Sorry but i really cant get over this, i kindly ask for your help.
As far as i know, those are the Siemens Base data Types and the conversion that im tring to implement:
Siemens Int      ( 16bit )  -> DotNet Int16
Siemens UInt    ( 16bit )  -> DotNet UInt16
Siemens DInt    ( 32bit )  -> DotNet Int32
Siemens DUInt  ( 32bit )  -> DotNet UInt32
Siemens real     ( 32bit )  -> DotNet double     ( As i read from other post )
Siemens LReal   ( 64bit )  -> DotNet double    ( As i read from other post )
Im working on a proget based on Dot Net Core, building an asp net MVC Project.
While reading and casting a Siemens Int like this is working for positive and negative numbers:
object o = new object();
o = PLC.Read(S7.Net.DataType.DataBlock, 1, 0, S7.Net.VarType.Int, 1);
Int16 j = Convert.ToInt16(o);

For the double Int Negative Values this is not working :
object o;
o = .PLC.Read(S7.Net.DataType.DataBlock, 1, 4, S7.Net.VarType.DInt, 1);
Int32 j = Convert.ToInt32(o);

I read in some ohter post that i have to use the library methods for conversions, but i can not understand how to use them.
Sorry im a novice, but for the S7.Net.Conversion.ConvertToInt(), used for retrive negative numbers from DB, im expected to pass an uint value and i dont understand how to use it.

https://github.com/killnine/s7netplus/wiki/Value-conversion

Thanks for the answer, sorry im new to programming and to Git hub

Long real is not available. The code should be something like this:

  #region Get/Set 64 bit floating point number (S7 LReal) (Range of Double)
        public static Double GetLRealAt(byte[] Buffer, int Pos)
        {
            UInt64 Value = GetULIntAt(Buffer, Pos);
            byte[] bytes = BitConverter.GetBytes(Value);
            return BitConverter.ToDouble(bytes, 0);
        }
        public static void SetLRealAt(byte[] Buffer, int Pos, Double Value)
        {
            byte[] FloatArray = BitConverter.GetBytes(Value);
            Buffer[Pos] = FloatArray[7];
            Buffer[Pos + 1] = FloatArray[6];
            Buffer[Pos + 2] = FloatArray[5];
            Buffer[Pos + 3] = FloatArray[4];
            Buffer[Pos + 4] = FloatArray[3];
            Buffer[Pos + 5] = FloatArray[2];
            Buffer[Pos + 6] = FloatArray[1];
            Buffer[Pos + 7] = FloatArray[0];
        }
        #endregion


.NET 8 Isolated function do not startup on Azure environment

Description
I'm using an Azure Function .NET 8 Isolated (upgraded from .NET 6 template) and also using the official AzureFunctionApp@2 pipeline deploy task to handle the deployment. Locally everything runs fine without any issue, but when deployed I'm seeing this error:

Microsoft.Azure.WebJobs.Script.ExternalStartupException : Error configuring services in an external startup class. ---> System.IO.FileNotFoundException : Could not load file or assembly 'Microsoft.AspNetCore.Routing, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'. The system cannot find the file specified

This is my deploy stage:
- stage: Deploy
  displayName: Deploy stage
  dependsOn: Build
  condition: succeeded()

  jobs:
  - deployment: Deploy
    displayName: Deploy
    environment: 'development'
    pool:
      name: 'SelfHostedAgentPool'
      vmImage: 'ubuntu-latest'

    strategy:
      runOnce:
        deploy:
          steps:
          - task: AzureFunctionApp@2
            displayName: 'Azure functions app deploy'
            inputs:
              azureSubscription: '$(azureSubscription)'
              appType: functionApp
              appName: $(functionAppName)
              package: '$(Pipeline.Workspace)/drop/$(Build.BuildId).zip'

The thing is, I don't even have that .dll in my output folder, although I have the package as a dependency from Microsoft.Azure.WebJobs.Extensions.Http (3.2.0), which in turn is a dependency of Microsoft.NET.Sdk.Functions (4.0.0).

and guess what, I can't downgrade to a non-deprecated version (what?!) because it is a dependency of a major package, and I can't also see that dll in my output folder as well.
Plus: the template offered by VS (2022) didn't even include the package that is causing the issue here on this topic: Microsoft.Azure.Functions.Worker.Extensions.ServiceBus
All packages I have now are:
<PackageReference Include="Azure.Messaging.ServiceBus" Version="7.17.5" />
<PackageReference Include="Microsoft.AspNet.WebApi.Client" Version="6.0.0" />
<PackageReference Include="Microsoft.Azure.Functions.Extensions" Version="1.1.0" />
<PackageReference Include="Microsoft.Azure.WebJobs.Extensions.ServiceBus" Version="5.16.1" />
<PackageReference Include="Microsoft.Extensions.Configuration.UserSecrets" Version="8.0.0" />
<PackageReference Include="Microsoft.NET.Sdk.Functions" Version="4.4.0" />
<PackageReference Include="Microsoft.VisualStudio.Azure.Containers.Tools.Targets" Version="1.20.1" />
<PackageReference Include="Polly" Version="8.4.0" />
<PackageReference Include="System.Configuration.ConfigurationManager" Version="8.0.0" />
<PackageReference Include="Twilio" Version="7.1.1" />

So, indeed, the root cause here, at least for me, is Microsoft.NET.Sdk.Functions, which I can't just downgrade because all non-deprecated versions still has that same dependency chain that end up needing the same Microsoft.AspNetCore.Routing, Version=7.0.0 package
This issue might be related to https://github.com/Azure/azure-functions-dotnet-worker/issues/2270, but, as apparently no one gave the required attention to that one, I'm also raising this.

Steps to reproduce

Create a .NET 6 Isolated azure function template triggered by Service Bus
Upgrade the version to .NET 8
Upgrade the packages
Deploy (I haven't tested with a publish profile, only pipeline)
See the error

UPDATE:
It was weird to me that I couldn't see any .NET 8 template, so I searched a little bit and found out that on VS Tools > Options > Projects and Solutions > Azure Functions I could update my az func tools and templates included.
After that, I could see the template for .NET 8 and it changes a lot from the .NET 6. So I created a separated project and copied all the csproj property group tag, packages and startup class to my project, deployed, boom, same issue again. Then I decided to MOVE all my implementations to the brand new project, deleted my own project from my solution and then imported the brand new .NET 8 template with my code moved back to my solution and it worked. Now I'm having a different issue.
From AI logs:
0 functions loaded
1 functions found (Custom)

Based on some searches, it might be ArchiveFiles@2 pipeline step faults, but I already tried to deploy with all possible parameter combinations and still the same error. There are tho some difference when I switch includeRootFolder to false.
With it set to true, this is what I see on App files tab:

When it is set to false, there is this raw text:

[{"name":"MyPackage.Functions.PackageNotification.zip","size":8516189,"mtime":"2024-06-14T04:34:08+00:00","crtime":"2024-06-14T04:34:08+00:00","mime":"application/x-zip-compressed","href":"https://mypackage-dev.azurewebsites.net/admin/vfs/publish_output/MyPackage.Functions.PackageNotification.zip?relativePath=1&api-version=2022-03-01","path":"C:\home\site\wwwroot\publish_output\MyPackage.Functions.PackageNotification.zip"}]

At this point the only approach left for me is to abaddon my pipeline and try to deploy through publish profiles.

UPDATE:
It was weird to me that I couldn't see any .NET 8 template, so I searched a little bit and found out that on VS Tools > Options > Projects and Solutions > Azure Functions I could update my az func tools and templates included.
After that, I could see the template for .NET 8 and it changes a lot from the .NET 6. So I created a separated project and copied all the csproj property group tag, packages and startup class to my project, deployed, boom, same issue again. Then I decided to MOVE all my implementations to the brand new project, deleted my own project from my solution and then imported the brand new .NET 8 template with my code moved back to my solution and it worked. Now I'm having a different issue.
From AI logs:
0 functions loaded
1 functions found (Custom)

Based on some searches, it might be ArchiveFiles@2 pipeline step faults, but I already tried to deploy with all possible parameter combinations and still the same error. There are tho some difference when I switch includeRootFolder to false.
With it set to true, this is what I see on App files tab:

When it is set to false, there is this raw text:

[{"name":"MyPackage.Functions.PackageNotification.zip","size":8516189,"mtime":"2024-06-14T04:34:08+00:00","crtime":"2024-06-14T04:34:08+00:00","mime":"application/x-zip-compressed","href":"https://mypackage-dev.azurewebsites.net/admin/vfs/publish_output/MyPackage.Functions.PackageNotification.zip?relativePath=1&api-version=2022-03-01","path":"C:\home\site\wwwroot\publish_output\MyPackage.Functions.PackageNotification.zip"}]

At this point the only approach left for me is to abaddon my pipeline and try to deploy through publish profiles.

Publish profiles is working. Just tested. Very likely the issue is related to one of the pipeline steps, either the zipping or the deploying.

"Microsoft.NET.Sdk.Functions

An isolated function does not need this package. That package was meant to be used by in-process applications. Why do you still have this package reference in your dotnet isolated function app?

and startup class to my project,

What startup class are you referring to? Keep in mind that the startup class you use in the ni-proc model will not work for isolated model.

Publish profiles is working. Just tested. Very likely the issue is related to one of the pipeline steps, either the zipping or the deploying.

Let us know what you find.


Dev/strand guide

@dlasky @shuwen @jcmoore @rpjs
As your time permits, please have a look

LGTM

This may just be a consequence of the test page, but knocking out a circle feels like it also spotlights a bunch of noise:

Would it make more sense to highlight only the target element?

CR feedback has been addressed @jcmoore @dlasky @shuwen
Re: Circle "spotlight" the best (and least unattractive) option IMHO was to switch to an ellipse, which eliminated some of the peripheral 'noise' around the target.

👍

I'll allow it, +1  (though you should check out the CI issues)

👍


ux enhancements re: login experience

handful of changes made as independently pickable commits (at least i hope so  🙏)
final result includes:

quieter output unless NEAR_DEBUG is truthy
more prominent message styling
web browser opens to auth URL automatically ✨
3 error cases handled:

invalid account (fails by matching /Account ID/ in error message)
account not found on current network
public key mismatch for account ID

(note: some minor formatting fixes were applied after recording the video demo and again after taking the screenshots below.  font size difference in screenshots is bc of image width here)
30 second demo includes 3 cases: invalid, not found, success
see here: https://asciinema.org/a/02smCQ2fOhIR0B6xICJegsFyC
invalid account

account not found

public key mismatch

@amgando do you plan to look into CI failure?

Thanks for improvements, please address comments though.
BTW, something that also can be useful – automatically callback from wallet to shell, here's approximate implementation:
https://gist.github.com/vgrichina/cfa393ecb3340779bf42f5a0c02d592f
In that case we'll need manual account ID prompt only as backup.

Thanks for improvements, please address comments though.

sure thing, will try to make time before next week to address.  clear direction in comments, thx!

BTW, something that also can be useful – automatically callback from wallet to shell, here's approximate implementation:
https://gist.github.com/vgrichina/cfa393ecb3340779bf42f5a0c02d592f
In that case we'll need manual account ID prompt only as backup.

won't this likely prompt the user to allow the open connection?  and what about proxy stuff?  this is a little out of my comfort zone, dealing with all the edge cases of opening a local server as a side effect of a CLI command

@amgando do you plan to look into CI failure?

the CI failures i've seen from my PRs are coming from 2 things:

forgetting to lint 🤦♂
masterAccount parameter missing (since recent updates to NEAR Shell actually)

won't this likely prompt the user to allow the open connection? and what about proxy stuff? this is a little out of my comfort zone, dealing with all the edge cases of opening a local server as a side effect of a CLI command

usually you don't need anything special to open connection on localhost (it's doesn't need to be externally facing)

oh, of course, it's just a browser redirect. that makes sense
ok, I think I have everything I need to clean up this PR

tests are still failing due to missing masterAccount param
Missing required argument: masterAccount
./test/test_contract.sh FAIL

otherwise, @vgrichina, how close is this to ready?


Faceting Performance Issue

We're seeing very bad performance in one of the tables in rbk-dev. You can see the behavior by doing the following (Please don't try this multiple time. It will slow the whole server down):

Go to the following link: https://dev.rebuildingakidney.org/chaise/recordset/#2/Vocabulary:Anatomy_terms
select "All records with value" for "Specimen Expression"
Open "Anatomical Source of Specimen" and it will hang there. It will eventually throw the timeout error.

These are the set of improvements that we should apply:

Use array_d instead of array for foreign-key and main entity projection list.
Don't ask for foreign-key data when we're only getting a list of row-names (Use entity api instead of attributegroup).
Switch the foreign-key paths to inner join instead of left outer join.

The first two improvements won't have any visible effects in the client. They will improve the performance on the main page, but still the "show more" page for facet will throw the timeout. So we need to do the third improvement too.
Changing the joins to inner will remove null from the list of possible values for the facet. To fix this issue, for now, we decided to add the No Value option to all the facets. It will be the second option that will always be visible for the facets. In the "show more" modal, it will be a checkbox besides the "All Records With Value". Although, when a facet has "null" filter, we must use left outer join for that path for getting the result on the main page.
Other possible improvements that we talked about for later:

Ask server if "null' is possible and show/hide the "No Value" option.
Instead of always showing the "No Value" on top, we can turn the request in ermrestjs into two different requests. One asking for values that exist (inner), and another for the null. The problem with this approach is the complexity of merging the "null"  row with the other rows. In entity-mode it should be simple. It's either at the bottom of the list or first. But in scalar, since we might not be sorting based on the value of that column, it could be anywhere in the list of results (This requires more investigation).


🐛 ListView does not work properly under NavigationView/NavigationPane/PaneItem.

Describe the bug
I don't know if this is a real bug or just me forgetting to read something, but anyway, The ListView does not work properly when its parent is a PaneItem (which is in the items array of NavigationPane, which is a pane for the NavigationView), At this point, I don't even know what I am doing.
To Reproduce
Steps to reproduce the behavior:

Get the code from this example code
add a main function and do a runApp(MyApp)
Run it.
See the error

Expected behavior
The ListView should work and display properly.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
It's likely a widget sizing issue.

¹ you can use the ListView.shrinkWrap property to size the children
² you can try to wrap the ListView in a SizedBox and give it a specific size

Somehow shrinkWrap does not solve the problem for me, but SizedBox did.

closing this as resolved


[ISSUE #4708] Fix potential NPE

Make sure set the target branch to develop
What is the purpose of the change
Close #4708
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #4711 (c52510b) into develop (a58b244) will decrease coverage by 0.06%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             develop    #4711      +/-   ##
=============================================
- Coverage      44.51%   44.44%   -0.07%
- Complexity      7502     7506       +4
=============================================
  Files            977      977
  Lines          67597    67600       +3
  Branches        8928     8929       +1
=============================================
- Hits           30091    30048      -43
- Misses         33778    33834      +56
+ Partials        3728     3718      -10

Impacted Files
Coverage Δ

.../apache/rocketmq/broker/failover/EscapeBridge.java
9.92% <0.00%> (-0.24%)
:arrow_down:

.../broker/subscription/SubscriptionGroupManager.java
68.79% <0.00%> (-14.19%)
:arrow_down:

...n/java/org/apache/rocketmq/test/util/StatUtil.java
16.19% <0.00%> (-14.09%)
:arrow_down:

...va/org/apache/rocketmq/logging/inner/Appender.java
29.21% <0.00%> (-7.87%)
:arrow_down:

...org/apache/rocketmq/common/stats/StatsItemSet.java
41.79% <0.00%> (-7.47%)
:arrow_down:

...ache/rocketmq/broker/topic/TopicConfigManager.java
59.03% <0.00%> (-3.06%)
:arrow_down:

...a/org/apache/rocketmq/filter/util/BloomFilter.java
60.43% <0.00%> (-2.20%)
:arrow_down:

...tmq/namesrv/processor/DefaultRequestProcessor.java
60.33% <0.00%> (-1.99%)
:arrow_down:

...he/rocketmq/client/trace/AsyncTraceDispatcher.java
81.42% <0.00%> (-1.43%)
:arrow_down:

...a/org/apache/rocketmq/store/StoreStatsService.java
49.17% <0.00%> (-1.11%)
:arrow_down:

... and 25 more

Help us with your feedback. Take ten seconds to tell us how you rate us.

Coverage increased (+0.05%) to 48.669% when pulling c52510bbf1ec66f00cf69cae9151b64b8be82a46 on Oliverwqcwrw:develop-fix-npe-39 into a58b2443a4d32e4a168caee5365bf59d73a50ca2 on apache:develop.


Error During skill setup within Alexa app

Hi,
After Linking kodi with Alexa device nothing Happens.
The Alexa App still Shows no Service connected within Setup.

But does it work if you request to play something?
There is stated in the tutorial, that Alexa app has a bug, and doesn't show any linked devices ...

No, it does mit work. Alexa teils me there is no Video Skill Activated. The skill i have activatesd is Named kodi. Is there a specific kodi video skill ?

What content country do you have set up in your Amazon account?

Hi,
i've checkd multiple devices.
It does not work on iphone xr and huawei p20.
On my old Galaxy Tab 10.1 2017 Edition it is working fine.
is it possible, that video skills are not supported on mobile devices ?

I was also having the same issue as described.  On restarting Kodi device was discoved

Still having the same issue on my end. Fire stick, skill is enabled, I liked it even though it shows no devices link but I still can’t control anything.

I had the same problem, I had to uninstall the plug in without deleting the data ( email and secret ) then reinstalled and it appeared in Alexa app.


GraphQL Endpoints shadowed by RequestMappings

I want to integrate GraphQL into an application that already provides a rest api.
The application contains a RequestMapping that is used as fallback, if no Controller method exists for handling the request:
@Controller
@RequestMapping
public class FallbackController {

    @RequestMapping("/**")
    public void unmappedRequest(HttpServletRequest request) {
        final String uri = request.getRequestURI();
        throw new ResourceNotFoundException("There is no resource for path " + uri);
    }
}

A corresponding ExceptionHandler exists, that converts the ResourceNotFoundException into a 404 JSON ResponseEntity.
Now having this RequestMapping in place, I get a 404 if I call any of the GraphQL Endpoints.

Side note: 404 is the perfect issue number for this issue. ;)

Indeed, the GraphQL endpoint is mapped as a RouterFunction and @RequestMapping methods are considered first. We'll consider an improvement to ensure the GraphQL endpoint is more stable for such scenarios, but such a change may be on the Spring Boot starter side.
In the meantime, one option is to create this catch-all handler as a RouterFunction:
@Bean
public RouterFunction<?> routerFunction() {
	return RouterFunctions.route()
			.route(RequestPredicates.all(), request -> ServerResponse.notFound().build())
			.build();
}

Also, if you didn't have it all, you should still get a 404 by default.

In my opinion, the Spring Boot auto-configuration is consistent with the Spring MVC opinion here. We're merely using the functional endpoints as a way to contribute controller handlers; this is probably the best choice when it comes to programmatic contributions like this.
Any @RequestMapping("/**") definition is bound to shadow all other defined mappings, including static resources handling.

@rstoyanchev Thanks for the RouterFunction workaround, I'll give it a try.
Regarding the following part of your comment:

Also, if you didn't have it all, you should still get a 404 by default.

You are right, that I get a 404 if I don't have a catch-all RequestMapping or RouterFunction.
However this is still not an ideal solution in our case. Our WebApp consists of a UI which is implemented using Apache Wicket and some REST-Endpoints that are implemented with Spring Web MVC. A user that get's a 404 inside the wicket application should get a nice Error-HTML-Page. A WebService API user should get a JSON Response instead.
Currently we are able to provide nice Error Pages via ErrorPageRegistrar:
@Bean
public ErrorPageRegistrar errorPageRegistrar() {
    return errorPageRegistrar -> {
        final ErrorPage errorPage500 = new ErrorPage(HttpStatus.INTERNAL_SERVER_ERROR, "/error.html");
        final ErrorPage errorPage404 = new ErrorPage(HttpStatus.NOT_FOUND, "/notFound.html");
        final ErrorPage errorPage403 = new ErrorPage(HttpStatus.FORBIDDEN, "/forbidden.html");
        ...
        errorPageRegistrar.addErrorPages(errorPage400, errorPage401, errorPage403, errorPage404, errorPage500);
    };
}

If we don't handle the 404 in the WebService part, then all /api requests that are not handled, return a 404-HTML-Page instead of the 404-JSON.

It works, but I have to explicitly ignore /graphql and /graphiql.

It worked in my case, but it looks like the order is not deterministic. We've discussed this and created related issues https://github.com/spring-projects/spring-framework/issues/28595 and https://github.com/spring-projects/spring-boot/issues/31314 to ensure RouterFunctions are ordered and the one for GraphQL is at 0. Then you'll be able to do:
@Bean
@Order(1)
public RouterFunction<?> routerFunction() {
    // ...
}

Aside from that, @RequestMapping("/**") is indeed something to be avoided, or otherwise if you really want it mapped that, way you'd have to explicitly exclude anything that might come after it.

Closing for now in favor of the two related issues.


Pathfinder-necros: Updated style sheet for labels and implemented three new features

Finished implementing conditions that the original Sam had started.
Implemented energy drain
Implemented wound threshold
Implemented nested inline rolling.
Added tabination to skills, spell classes and spells
Updated the footer to include myself, Vince and Nibrodooh as maintainers as well as updating the last updated date.

Howdy, there have been multiple people including Vince and others working on this sheet recently. Because of that there is a merge conflict. You'll need to pull from the repository to get our most up to date version with other people's changes and then reapply your changes and make a new pull request.
Let me know if you have questions.

Hi Steve, I've redone the pull request and hopefully this one will be good
to go through.
On Mon, May 11, 2015 at 8:51 PM SteveKoontz<EMAIL_ADDRESS>wrote:

Closed #817 https://github.com/Roll20/roll20-character-sheets/pull/817.
—
Reply to this email directly or view it on GitHub
https://github.com/Roll20/roll20-character-sheets/pull/817#event-302376031
.


[FLINK-11947] Support MapState value schema evolution for RocksDB

What is the purpose of the change
Currently, we do not attempt to perform state schema evolution if the key or value's schema of a user MapState has changed when using RocksDB, with this commit we support value's schema evolution for RocksDBMapState.
Brief change log
add value's schema evolution for RocksDBMapState when migration
Verifying this change
This change added tests and can be verified as follows:

StateBackendMigrationTestBase#{testKeyedMapStateAsIs, testKeyedMapStateStateMigration, testKeyedMapStateSerializerReconfiguration, testKeyedMapStateRegistrationFailsIfNewStateSerializerIsIncompatible}

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

@flinkbot attention @tzulitai

@flinkbot approve description
@flinkbot approve consensus

@tzulitai thanks for the review, I've addressed the comments, please have another look when you're free.

+1, LGTM, merging ..
Thanks for the contribution @klion26!

@tzulitai thanks for the review and merging.


I was wondering if you would be interested in making this work on Windows.

I was able to get it up and running on Windows by installing the requirements. It fails when I click import addons with a charmap error which seems like it would be simple to fix. My coding is limited though so i'm not sure how to fix it. Also, if I remove some addon it seems to work but I think it would be related to how many are in the addons folder.
pip install bs4
pip -m install "C:\Path\to\file\lxml-3.6.4-cp35-cp35m-win32.whl" lxml-3.6.4-cp35-cp35m-win32.whl
pip install PyQt5
python lcurse

I am guessing this being closed means no to a windows version?  The curse client + twitch abomination makes this a really attractive way of managing addons cross-platform.

Have you tried it since the close? ... Except the git part, it should work. At least it worked here at my wifes machine.

I tried installing python and running the requirements.txt with the main instructions, I ran into issues when it attempted to compile a dependency.    I'll try it again on a fresh machine.  If you're willing to support, I'll post issues that I find and help test.

I'll help as far as I can.


luci-mod-admin-full: Fix display problem of wifi_add

Fixed an issue where the target network name is displayed as "%q" on wlan
connection destination network settings page.
Signed-off-by: INAGAKI Hiroshi<EMAIL_ADDRESS>

Good catch!


Single Dockerfile.template

Mainly this PR consolidates the three Dockerfiles into a single one by introducing an extended version of the uname modification. There is a separate shell script file in the scripts folder for this purpose, which is invoked from the Dockerfile.
There were a couple of bugs that I fixed as well:

A positional argument was needed in the Rust part instead of an option one. This made the application always produce Hello, world! output, instead of showing the device type.
On first launch the fake application (used for dependency cache purposes) was executed. Fixing this needed removing target/release/deps/hello-* as well.

I did some small adjustments in the Dockerfile - moving above some WORKDIR definitions so that following statements were more concise.
I bumped the Rust version to latest.
Change-type: patch
Signed-off-by: Zahari Petkov<EMAIL_ADDRESS>

looks good to me, thank you :) will test on rpi and x86 and let you know

No changes on the local-push.rs scripts were needed although RESIN_ARCH is used. No need to substitute this one.

Yes, please check, and let me know if you encounter any issues.

Heyo @majorz !
This tests okay onx86 and rpi3 :tada:  - we did not test on rpi2 before for the lack of hardware, same problem now.
Quick question - in the logs I'm seeing
[Info]     Starting build for raspi, user gh_cyplo
[Info]     Dashboard link: https://dashboard.resin.io/apps/1277499/devices
[Info]     Building on arm02
[Info]     Pulling previous images for caching purposes...
[Success]  Successfully pulled cache images
[main]     Step 1/27 : ARG rust_revision="1.29.1"

1.29.1 is the previous version - is this expected ?
Then in the logs much later:

curl -sSf https://static.rust-lang.org/rustup.sh | sh -s -- -y --revision=${rust_revision}
[main]      ---> Running in 279ebc163fe4
[main]     rustup: gpg available. signatures will be verified
[main]     rustup: downloading manifest for 'stable'
[main]     rustup: downloading toolchain for 'stable'

it should be 1.29.2 instead of stable - it seems to be ignoring the version number altogether maybe ?
I'm not sure if this is introduced by this PR or not so don't worry :) Let me know if you want to take a look into the version problem or shall we merge as is and tackle it independently.

@cyplo I checked the earlier commit and indeed the stable is picked there as well. Let's handle it in a separate PR.
I tested on RPi 1 and things are working there correctly.
I will merge this one now ;)


Add extraction support for pid, label, and endtime for redshift-logs …

…connector.
xid to pid is one-to-many, and pid to queryid is one-to-many, so adding pid doesn't change total number of records or grouping.
However, changed how STL_QUERY is joined with STL_QUERYTEXT and now they are joined additionally on userid, xid, pid.
Have confirmed in group discussion that redshift-logs connector is currently not used by any downstream services so the changes shouldn't cause any breakage.
Tested manually with a real Redshift instance and the extracted results are as expected.

This connector is deprecated because it does not, and I think can not work. What is the reason we are pursuing this change, please?

Because the non-working part is the query text concatenation, where as the other "log metadata" fields are intact. The redshift rawlogs-connector's output is un-aggregated - true that the metadata without query text can also be extracted from the rawlogs connector output just by deduplication, but it would greatly help with prototyping if we can save effort from that additional implementation for now.


Add ability for fetch --keep-keys and --drop-keys flags to do wildcard matching

Thanks for the feedback, Jay! I've incorporated your suggestions.


Allow expectError directives to detect parameter count mismatches

This PR extends the errors swallowed by usage of expectError for the following 2 errors:

function argument count mismatch for overloaded functions
function type argument count mismatch for overloaded functions

@BendingBender Can you fix the merge conflict?

@sindresorhus done

@BendingBender It's probably because I just merged the other PR. Can you resolve them again 🙏 .

@SamVerschueren done


Remove unused files

Quick little PR to remove some of the cruft.

android/ folder and index.android.js
.flowconfig
.watchmanconfig
.travis.yml

Resolves #85

Closing to reopen because of Git stuff.


Logging basicConfig() breaks my loggers

I've recently encountered a problem caused by the usage of logging.basicConfig() in this library. Normally, this call should be made at the entry point to the application, but it was randomly called halfway during my application's initialization, by this library. It should be removed.

Here's one of the two commits adding it: https://github.com/dtsong/sleeper-api-wrapper/commit/b24c8840a5124a8cdd6068a493101fdcbaf30344
This lines up with my experience - I did not have problems with pypi version 1.0.7 but 1.1.0 did break me. I had to roll back.


Provide minimal privileges

Thanks so much for your work on this project!  As a "DevOps-ey" guy, the expansive permissions in this doc are going to be a sore spot with any security minded folks.  This smaller subset of privileges works for us.

[ ] I've added tests (if it's a bug, feature or enhancement)
[ ] I've adjusted the documentation (if it's a feature or enhancement)
[ ] The test suite passes (run bundle exec rspec to verify this)

Summary

Context

Version Changes

Let’s add a section to the same doc with the 2nd example. So have both examples.

spliced in the original section.


Modify murmur function to always use UTF-8 bytes

Murmur partition function picks up the default charset, which can be different
based on the system environment. This forces murmur partition function to use
UTF-8 bytes.

Codecov Report

Merging #2431 into master will decrease coverage by 0.17%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2431      +/-   ##
==========================================
- Coverage    69.4%   69.23%   -0.18%
==========================================
  Files         848      848
  Lines       39516    39517       +1
  Branches     5309     5309
==========================================
- Hits        27428    27358      -70
- Misses      10307    10387      +80
+ Partials     1781     1772       -9

Impacted Files
Coverage Δ

...t/core/data/partition/MurmurPartitionFunction.java
77.77% <100%> (+2.77%)
:arrow_up:

...t/common/exception/PermanentDownloadException.java
0% <0%> (-100%)
:arrow_down:

...not/common/exception/HttpErrorStatusException.java
0% <0%> (-100%)
:arrow_down:

...ommon/utils/retry/RetriableOperationException.java
0% <0%> (-100%)
:arrow_down:

...ot/common/utils/retry/AttemptFailureException.java
0% <0%> (-50%)
:arrow_down:

...core/operator/dociditerators/AndDocIdIterator.java
51.72% <0%> (-31.04%)
:arrow_down:

...not/common/segment/fetcher/HttpSegmentFetcher.java
58.33% <0%> (-16.67%)
:arrow_down:

...edin/pinot/common/utils/retry/BaseRetryPolicy.java
69.23% <0%> (-15.39%)
:arrow_down:

...impl/dictionary/FloatOffHeapMutableDictionary.java
83.63% <0%> (-10.91%)
:arrow_down:

...n/pinot/common/utils/FileUploadDownloadClient.java
67.01% <0%> (-10.31%)
:arrow_down:

... and 19 more

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 4680dc2...dc9d179. Read the comment docs.


Feature/disabled fetching country

fix disabledFetchingCountry

@fabiosal This PR changes all the lines of vue-tel-input.vue, probably because of the conflicts from editors.
Can you take a look and update it?

I fixed it, included some other fixes, v2.0.6 is out!
Thanks for your help.


feat: validation migration function

Describe your changes
Ticket: https://pins-ds.atlassian.net/browse/ASB-2428
Adds a validate migration endpoint called via function app endpoint. It is called with a caseReference, and returns all entities (application, timetable, advice, representation, documents) differences for the application. The response omits any values which are identical and returns only the differences. It will traverse through nested objects and arrays.
Note: Existing endpoint services are used so any mandatory filters such as excluding draft documents will apply here.
JSON Diff library: https://www.npmjs.com/package/json-diff
Useful information to review or test
Dev endpoint:
https://pins-app-applications-service-applications-api-dev-ukw-001.azurewebsites.net/validate-migration/BC0110001
Function endpoint:
Endpoint can be retrieved from Azure - not listed here as contains auth code.
Call with POST request with body:
{
    "caseReference": "BC0110001"
}

Type of change 🧩

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[x] I have performed a self-review of my own code
[x] I have double checked this work does not include any hardcoded secrets or passwords
[x] I have made corresponding changes to the documentation
[x] I have provided details on how I have tested my code
[x] I have referenced the ticket number above
[x] I have provided a description of the ticket
[x] I have included unit tests to cover any testable code changes

:tada: This PR is included in version 1.11.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.22.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.28.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


KAFKA-14534: Reduce flakiness in TransactionsExpirationTest

This test asserts that after a producerId expires and before a transactionId expires, the producerId is reused in a subsequent epoch.
The transactionId expiration time used in this test was too short, and a race condition between the two expirations was occasionally causing a new producerId to be returned without an epoch bump.
Signed-off-by: Greg Harris<EMAIL_ADDRESS>Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@jolshan Could you take a look at this test stabilization?

after your change, it failed other tests:

Thanks, fixed. I was running only the first test locally and didn't check to see what the timeouts in the other test were.

After all, the producerID expiration doesn't cause any test failure here, we shouldn't change it. WDYT?

Yes, the CI failures I was seeing were all about the race condition between the test and transactionId expiration. However, while I was running these tests under a local CPU slowdown to 25-30%, I was seeing race conditions happening between the test and the producerId expiration. The test would poll for the producer state, only for it to have already been expired by the broker. I'm lengthening both here so that I don't have to follow up when this test flakes out in the future :)

No failed tests in TransactionsExpirationTest for 2 CI build in a row.


stream settings: Fix subscribers not showing on private stream creation.

Fixes:

[X] Fix subscribers not showing on private stream creation.

Previously, stream subscribers were not rendering correctly. Due to
following reasons:

Subscribers list didn't get initialized
Subscriber members template add subscriber-list-table DOM element
conditionally, which doesn't handle case if user have access to
subscribers later.
[X] Fix sub-unsub btn not showing on private stream creation.
[X] Fix display of stream subscribers count

Fix this by cleaning subscriber member template to hide the
elements, instead of conditionally adding DOM elements and
initialize subscribers list even if user can't access subscribers.
GIFs or Screenshots:

This PR is still  WIP, cause it will cover subscriber count bug fix and subscribe/unsubscribe button not showing up in private stream creation.

The final look is:

This looks great, merged, thanks @YJDave!
For the handlebars templates, I think there's two valid approaches:

Have the handlebars template completely re-render when new information comes in (in which case we can use whatever conditionals we like inside the template).
Use the hide/show approach that you use here.

So we should make sure everything that depends on state that changes has one of the those two approaches.

Thanks Tim, for reviewing!
I think second approach seems better, cause we can't render whole template to just change display of buttons in it, in case of stream settings. Or we can make it separate template for buttons only, which isn't worth it in this case. I will go through all handlebars files for stream settings at least, and check which approach is better in particular case and if any changes required I will submit PR.


Configuration of authentification timeout for session more flexible

Is your feature request related to a problem? Please describe.
Problem 1 :
Actually the authentification timeout is only every hour, every day, every week ...
But if we want a authentification timeout of 12 hours or 2 days it's impossible to set.
Problem 2 :
Actually the session duration is for every users.
But we cannot define a specific authentification timeout for an user.
Describe the solution you'd like
Problem 1 : A more flexible configuration for session duration ti be able to set it more precisely.
Problem 2 : Add the possibility to define a specific authentification timeout for an user or group (if it's implemented in next release)

With 1.0 the session timeout will be configured in your IdP, and we'll simply replicate this for the end-user session lifetime in the client apps.

See https://learn.microsoft.com/en-us/azure/active-directory/develop/configurable-token-lifetimes for an example


Reducing the number of NEA sampling points

https://github.com/lucerlab/GMM-NEA
Questions that I thought about during Štěpán's talk:

Is estimation based on the semiempirical method reliable for cases where spectra widths differ from the reference?
ML for NEA - preferable to optimal sampling or vice versa?
ML for NEA - how to deal with multiple excited states?
How does it differ from the approach published by Barbatti?

Note: stda apparently works well (used at BASF)

https://github.com/grimme-lab/stda
https://github.com/grimme-lab/xtb4stda/


Autocomplete with Multiselect

If this is a multliselect autocomplete: show chips ABOVE the input field (not inside). When chips are created, and the field is reopened, the first selection in alphabetical order should be visible as the second from the bottom (unless it's the first one in the list or the last one in the list).

Do we make this configurable, such that the table could still have it inside?  Or do we need to account for tables separately?  Would be good to have updated figma to show the design/spacing once the chips are moved.

Designs:
https://www.figma.com/design/wyfVJUHWRMkeCfdB38HFEE/Modus---Web?node-id=48550-101347&t=Vzah26EJaaS539l8-4
Workflow:
https://www.figma.com/design/wyfVJUHWRMkeCfdB38HFEE/Modus---Web?node-id=40560-30944&t=Vzah26EJaaS539l8-4

Identify the gaps with the Figma designs

Adding more context to this:
The requirements for this request are here:
Request
We have two options for how we approach this:

update individual atoms to make this component (recommended)
create a new component

The atoms required are:

List
List item
Chip
Dropdown? (Should be menu)

Additional concerns / needs:

need to support keeping the menu container open while interacting for things like search/autocomplete
might need to separate menu from dropdown component (see MUI)
need to add logic for "chip bar" which allows a large set of chips to be navigable via clicking arrows left/right
need to support X button on chips to clear current selection
list atom has rounded corners which causes multiple issues, ideally remove but may need to make it a config option if it breaks existing usage

The tasks can be split into multiple issues as required

[ ] Implement clear icon
[ ] Keep dropdown open in multi select mode (also applicable for selection via keyboard)
[ ] Search text highlighting
[ ] Allow user to delete a chip using tab and delete


Reduce size of redis image for pets using alpine:3.4

gcr.io/google_containers/redis-install  0.2  50c8a6c1d914  About a minute ago   108.5 MB
gcr.io/google_containers/redis-install  0.1  e23c2ffde782  3 weeks ago          272.1 MB

ping @bprashanth

Great thanks! I'll merge when I'm at keyboard and I can push the new image

Merging


[resolution] - How to show only resolutions

I am trying to show only resolutions like in the screenshot is it possible? if yes , then how?

You'll need to use API. See https://github.com/rg3/youtube-dl#embedding-youtube-dl for a reference.

You'll need to write some Python scripts first and then call them from PHP.

$cmd_image = 'youtube-dl --get-thumbnail '. escapeshellarg($url);
$process_img = shell_exec($cmd_image);
echo $process_img;
$cmd = 'youtube-dl --get-title '. escapeshellarg($url);
$title = shell_exec($cmd);
echo $title;
I could print the thumbnail and title on my webpage. The only problem is with formats .. Help me to overcome this issue.

Only resolutions does not provide much sense, so youtube-dl does not provide it. If you really need it, use the python API:
$ python -c "
import youtube_dl

with youtube_dl.YoutubeDL({'quiet': True}) as ydl:
    info = ydl.extract_info('https://www.youtube.com/watch?v=HxIct0i-CcA', download=False)
    for i in info['formats']:
        if i['vcodec'] != 'none':
            print('{0}x{1}'.format(i['width'], i['height']))
"
256x144
256x144
426x240
426x240
640x360
640x360
854x480
854x480
1280x720
1280x720
1920x1080
1920x1080
176x144
320x180
640x360
640x360
1280x720

Hello @yan12125  Can't i take the data from --dump-json using PHP only and then show resolutions from there?

Apparently you already know another way to do it ;)

Apparently you already know another way to do it ;)

@yan12125  I am trying to get the exact  formats "(multiple - Quality Selection URL)" Like in the screenshot but i got only the below formats. Can you help me with that?
Please Do it with json and php. Or you can call @phihag  .

$data = json_decode(`youtube-dl -j test:cctv`, true);
foreach($data['formats'] as $f)
{
    print $f['format'] . "\n";
}

The result is:
meta - multiple (Quality selection URL)
204 - 320x180
460 - 480x270
870 - 640x360
1228 - 1280x720
2048 - 1280x720

I am getting the ERR  when i try to download from Adult websites .. Here is the result i get using "youtube-dl -v [URL] " command.
Can you help me with that?
By the way my ISP has blocked Adult websites.

By the way my ISP has blocked Adult websites.

youtube-dl can't escape ISP restrictions automatically. If you can watch them with proxies or VPNs, add --proxy to the command line.


Help page with ?

Is your feature request related to a problem? Please describe.
I use some kind of vim binding in every browser i use and there are subtle differences between all of them. A help page would be nice, esp as vimari uses some quite un-vim like bindings i could have never figured out without reading the docs (q, w)...
Describe the solution you'd like
A popup with the keybindings when pressing ?. If not possible, at least in the desktop app a readonly list of the bindings would be nice, not everyone is online all the time...
Describe alternatives you've considered
n/a
Additional context
n/a

hi, Im working on that.


Adding validator for files to upload on SDK

Can you instead use pathvalidate https://github.com/thombashi/pathvalidate/tree/master library?
Avoid returning tuples from functions. Also check for the filte path (directory name on QFieldCloud) validity.

Great, I didn't know about this library!


Enable generated service test targets

Describe the bug
Currently we have disabled the test targets because swift CLI fails to run them and errors out.
Expected Behavior
swift test must run service test targets.
Current Behavior
Fatal fail.
Reproduction Steps

in Package.swift, uncomment testTargets for services
swift test

Possible Solution
No response
Additional Information/Context
No response
AWS SWIFT SDK version used
man
Compiler and Version used
5.7.2
Operating System and version
macOS 12.6.1

We can try using xcodebuild for just on macs to atleast run on one OS. But we would like to stick to swift cli.

This is done now.


added lodash memoize function

PR Checklist
Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[x] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Fixes
Issue Number: 67
What is the current behavior?
Every time that the getTargetZoom function is called with the same scale compute the result
What is the new behavior?
Now the result of getTargetZoom is memoized on the scale value, that means that the result is cached and doesn't have to compute it every time.
Note
I think that splitting the code would be really useful to improve maintainability. After that, you could also think to move to functional components and start using hooks, that are pretty awesome for that kind of improvements!

@damiano-carradori I'm sure this value changes when the viewer is resized or on fullscreen change:
https://github.com/btzr-io/Villain/blob/448b5594186876886c3ff9f654a40b82df3e6949/src/components/render.js#L161-L163

Maybe there are other functions that can benefit from this?
I think react functional components has a memo option.
I'll close this for now