Unsafe aliasing in rust after adding first federation

recursive use of an object detected which would lead to unsafe aliasing in rust

I think this is because how I'm swapping in GlueDB after it inits a federation for the first time if none existed. Doesn't seem to hurt anything but probably not safe. A better approach would be to empty init GlueDB but do not start it until a federation is added.

This is happening somewhere in FederationClient::new but only when adding for the first time. But it's happening either kind of inconsistently or the error log is showing up in different places sometimes. Hard to tell but nothing important I think. Postponing for later, nothing is presenting itself as a problem.


hygiene regression with renamed functions

Describe the bug
See repro; Foo gets renamed to Foo1 as in this issue: https://github.com/swc-project/swc/issues/1600#issuecomment-893864913 (credit to jdb8 for figuring out the repro here too)
Input code
var Foo = function () {
  function Foo() {}
  return Foo;
}();

Config
{
  "jsc": {
    "parser": {
      "syntax": "ecmascript",
      "jsx": false
    },
    "target": "es5",
    "loose": false,
    "minify": {
      "compress": false,
      "mangle": false
    }
  },
  "module": {
    "type": "commonjs"
  },
  "minify": false,
  "isModule": true
}

Playground link
https://play.swc.rs/?version=1.2.186&code=H4sIAAAAAAAAAytLLFJwy89XsFVIK81LLsnMz1PQ0FSo5lJA8IHSIKFaoFhRaklpEVjEmqtWQ9MaAMEniZs%2BAAAA&config=H4sIAAAAAAAAA0WMSwrDMAxE76K1t93kDj2EcJXg4B8aBWqM714bUrKSmPdmOp3wtHWqrBBdH1o2%2FtJG4hPDa6hGbmoz2jlChiNjPcSWgtdksRTITR2lkMPe1pIvqaoAD%2BJ8xL855lAqn2sFnazVeVcllXyCxjN0lwPet216yfgBCKaI9boAAAA%3D
Expected behavior
Function should not be renamed
Actual behavior
var Foo1 = function() {
    var Foo = function Foo() {
    };
    return Foo;
}();

Version
1.2.186
Additional context
On the playground, v1.2.101 works while v1.2.102 breaks, so it seems likely to be this PR: https://github.com/swc-project/swc/pull/2408
We were previously on v1.2.81 with experiments.optimizeHygiene set (which no longer exists - does this option still exist in a different form or is it meant to be enabled by default?).
Note also that with mangling on there's no bug.

var Foo1 = function() {
    var Foo = function Foo() {
    };
    return Foo;
}();

This does not seem like a bug

@kdy1 thanks for taking a look! I'm curious to understand how this differs from the original case in https://github.com/swc-project/swc/issues/1600#issuecomment-893864913: as mentioned there, I realise that the renaming of functions and variables like this shouldn't break the code directly (as all references in scope should be updated to the new name), but that in cases where function names are expected to stay consistent with the input code it can break things (React enzyme tests is one example, on components that lack an explicit displayName attribute).
It also differs from babel's output for the same input, which leaves Foo unchanged.
Do you have more context on why this wouldn't be considered a bug, or at least something that could be improved? Especially given that this output seems to have changed back after 1.2.101, and was previously working fine under the optimizeHygiene flag in 1.2.81.

Because input code has clearly two bindings.
// first one
var Foo = function () {
  // second one
  function Foo() {}
  return Foo;
}();

and those two are different from each other


SOLR-17140 - provide extensible query limit concept.

https://issues.apache.org/jira/browse/SOLR-17140
This replaces SolrQueryTimeoutImpl.java with a much simpler SolrTimeLimit and provides a QueryLimits class to allow addition of additional types of limits. (see also https://issues.apache.org/jira/browse/SOLR-17138)

At Salesforce, we've extended this concept to also apply to indexing and one could imagine maybe someone using it in other cases. The "Query" in the name is unfortunate... can we call this RequestLimits instead of QueryLimits?

This hinges on and (and implements) QueryTimeout so for the moment it's pretty solidly query focused. When we expand the functionality the name can change then?

Feedback addressed, tests pass locally and crave runs in past that have failed on crave have been ZK or recovery related... This is holding up other work so I'm merging it.


Can you please create typescript type definitions?

It would really help my IDE autocomplete and check for errors.
Great work on this project btw! ❤

Sorry, was blind

Sorry, was blind

What did you mean?

What did you mean?

found an index.d.ts in the node_module folder but just came back to the project and turned out it was the wrong chatgpt package


feat: python I/O functions + BiGG compliance flag

Main improvements in this PR:

Features python functions for loading and saving the model (read_yeast_model and write_yeast_model) without needing to specify the model path. Usage, as indicated now in the README file, is as follows:import ComplementaryScripts.io as io
model = io.read_yeast_model() # loading
io.write_yeast_model(model)   # saving

For this to work, a .env is required in the repo's root (as also indicated in the README file).
Features a requirements.txt file for ensuring a reproducible environment when performing simulations in Python.
To fully address #172, read_yeast_model features a make_bigg_compliant flag for loading the model with BiGG ids instead of regular ids. If chosen, the BiGG annotation in the model + the BiGG dictionaries in ./ComplementaryData/databases are used for the conversion. Some additional info:

ids will have a _copyN suffix if they are repeated in the model (as ids should be unique).
ids not found will be kept the same as in the model (but in the case of metabolites, converted to the met_comp standard).
Original ids are moved to notes.
Compartments are made BiGG-compliant with the dict {"er":"r", "erm":"rm", "p":"x"}
Compartment info is removed from metabolite names.

NOTE that in each commit I saved the model with write_yeast_model to confirm changes were as expected, but in the last commit I reverted the model to be as the original one in devel, so no model changes are performed here.
NOTE ALSO that although the model can be saved in cobrapy format keeping all fields, it changes considerably the structure of the .xml file and hence should not be done if contributing to this repo until issue #218 is solved.
I hereby confirm that I have:

[x] Tested my code with all requirements for running the model
[x] Selected devel as a target branch (top left drop-down menu)

@BenjaSanchez I am not sure whether errors exist when load model using python, i have used the followed self-function to correct some errors:
def correctSomeWrongFormat(model0):
"""
This function is used to correct some wrong format when read yeastGEM model from cobratoolbox
"""
Correct metabolite ids:
for met in model0.metabolites:
met.id = met.id.replace('91', '_')
met.id = met.id.replace('93', '')
#for reaction in model0.reactions:
reaction.gene_reaction_rule = reaction.gene_reaction_rule.replace('45','-')
for gene in model0.genes:
gene.id = gene.id.replace('45', '-')
return model0

@hongzhonglu the changes you are performing are not needed anymore with the latest versions of cobrapy, as they convert the non-ASCII characters by default. Are you testing the PR with the proper environment? please update your cobrapy version, or even better, pip install -r requirements.txt in a fresh environment to make sure we have the same setup :)

Hi @BenjaSanchez, Thanks for your information. I will test it.

@hongzhonglu is the setup working for you? :)

hi Ben, could you replace this package named "dotenv" as i found it is can't be installed by pycharm?

@hongzhonglu the package is called python-dotenv, as you can see in requirements.txt (no package called dotenv). Pycharm should not have any problem with python-dotenv.

Hi Ben, thanks for your remind!
I meet new issue:
find .env + define paths:
dotenv_path = find_dotenv()
REPO_PATH = dirname(dotenv_path)
when run the above code, REPO_PATH is "", thus the followed commond can not be run.  Why not directly set the whole path of model as the input for  function "read_yeast_model"?

@hongzhonglu did you follow the new installation instructions as the README now states? You should create an empty .env file in the repo root:
type nul > .env                     # creates a .env file for locating the root

The idea of using dotenv is precisely to avoid specifying the path depending of where in the repo we are, as that (absolute) path never changes. This makes it easier & safer for the user to read/write, as it requires less typing and avoids mistaking the path and saving the model somewhere undesired.


question about preserveWhitespace

Description:
The preserveWhitespace option collapses or compresses multiple whitespace characters into a single one.  While this is what HTML does under some circumstances, it can turn make the formatting of templated HTML inadvertently significant.  Was this the intent of the default value false for preserveWhitespace?
Versions affected:
edge (and presumably earlier)
Platforms affected:
Windows 10, but assuming all platforms
Reproduction:
https://jsfiddle.net/jfsnwac7/
HTML

    text 3

    text 4

JavaScript
var r = new Ractive({
  target: '#target',
  template: '#template'
});

Output

Desired Output

To get the desired output with straight HTML, it is of course the author's problem.  Solvable in several ways, ranging from "format the HTML without any intervening whitespace (either by putting it all on one line or by using HTML comments to separate the lines)" to "use CSS on the container div (I've seen font-size:0, display:flex, and even a contrived web font which as an empty space character)".
One could argue that a template under Ractive control has the same choices available.   But I would have thought, given the preserveWhitespace option, which defaults to false, that Ractive would allow me to format my template with whitespace between elements and have an option to thoroughly ignore this whitespace.
Ought the current preserveWhitespace option already do this, and simply has a bug?
Or is this issue merely a feature request?
If the latter, I would point out that other JavaScript libraries, like Polymer, seem to offer this convenience:
https://www.polymer-project.org/2.0/docs/devguide/dom-template

Remove empty text nodes
Add the strip-whitespace boolean attribute to a template to remove any empty text nodes from the template's contents. This can result in a minor performance improvement.

What's an empty node? strip-whitespace removes only text nodes that occur between elements in the template and are empty (that is, they only contain whitespace characters). These nodes are created when two elements in the template are separated by whitespace (such as spaces or line breaks). It doesn't remove any whitespace from inside elements.

[In my example the container's nodelist is
NodeList(3) [div, text, div]
And since that text node is empty by Polymer's definition, the Polymer option removes it.  That's exactly the functionality I wish I had in Ractive.
I know.  Just cuz the other kids are doing it doesn't make it right.  😄

Quick rundown: preserveWhitespace init option is essentially Ractive.parse()'s preserveWhitespace option. What this does is tell whether or not the parser should preserve whitespace information on the AST.

preserveWhitespace: false - Default. Collapse sequential whitespaces  to one space, trim ends.
preserveWhitespace: true - Preserve the whitespaces during parsing (i.e. leave \n, \t etc. alone).

Ractive just collapses multiple sequential whitespaces with preserveWhitespace: false and I think it's by design. It's best to follow how the browser behaves to avoid surprises. Stripping the whitespace is just convenience but breaks expectations. Imagine how many hours and hair are wasted by developers trying to figure out what's happening, only to find out it's documented in some dark corner of the documentation.
As far as I know, flexbox and CSS grids are the official features that address this and several other layout deficiencies in CSS. Best to use the right APIs instead of working around the issue.
But if this is a popular request, we could expand whitespace treatment. I'd think the following would make sense, a new option to expand whitespace treatment:
{
  whitespace: "collapse", // collapse whitespace
  whitespace: "preserve", // preserve whitespace
  whitespace: "strip" // remove whitespaces where applicable
}

I agree with your reasoning. It is definitely a nice-to-have.
That said, I cited the Polymer example to show that other templating
libraries offer this convenience option.
I'd love to see Ractive offer the same level of convenience, but defer to
others as to how to prioritize it.
Best,
Jeff
On Mon, Dec 18, 2017, 9:39 AM Joseph<EMAIL_ADDRESS>wrote:

Quick rundown: preserveWhitespace
https://ractive.js.org/api/#preservewhitespace init option is
essentially Ractive.parse() https://ractive.js.org/api/#ractiveparse's
preserveWhitespace option. What this does is tell whether or not the
parser should preserve whitespace information on the AST.
https://ractive.js.org/playground/#N4IgFiBcoE5SAbAhgFwKYGcUgL4BoQN4A3JGAAhiQGMUBLYtARnIF5yAlG+xgCmAA6AOwHoEkcgBMA9tQCuAWzRCUAOgBG0yQE88w0UgAOh5ZIkoYctHqHly6BYeToJAA30pRAHkMA+D6KiABJoCAjSeOQA6tIwCJIAhAEoXgD0fsk+-iKeuQCCYeTa0nIU6kgYaORkVeqh0kIA5vbS5HIYmenZoq42doYwmGgwjFFgdOgYhjRoEgBmSAiVwjgAlPpC1A0Y0ghoquGNvADkA0MjaGMTmNPUs+QLS9b2aI7OaMeRVLQMzKoOTlQaHWm22u32hxOZ0qFyuk1u90elUiYBQCgQn0o3F+TH+0iCABUALIAGV4qxBwlIFG+PDQACY2JxsXxBDlQhIZPIlCoNFpdB4jCYhGZ7JZrMI7AD3m5MhkcoEUCEwhForF4kkFSkunLurlRAUEEUSmUKlUauQ6uEmi02h0tWl5T0+uRocNRuN4TNzOKVpTQUIdnsDtIjqdBjCPdcpt6xVZItKgZjab96f9XoD0CCtoHwSGw27YZ6brGLPHyKj0cmWQy8YTSeTVrggA

preserveWhitespace: false - Default. Collapse sequential whitespaces
to one space, trim ends.
preserveWhitespace: true - Preserve the whitespaces during parsing
(i.e. leave \n, \t etc. alone).

Ractive just collapses multiple sequential whitespaces with preserveWhitespace:
false and I think it's by design. It's best to follow how the browser
behaves to avoid surprises. Stripping the whitespace is just convenience
but breaks expectations. Imagine how many hours and hair are wasted by
developers trying to figure out what's happening, only to find out it's
documented in some dark corner of the documentation.
As far as I know, flexbox and CSS grids are the official features that
address this and several other layout deficiencies in CSS. Best to use the
right APIs instead of working around the issue.
But if this is a popular request, we could expand whitespace treatment.
I'd think the following would make sense, a new option to expand whitespace
treatment:
{
whitespace: "collapse", // collapse whitespace
whitespace: "preserve", // preserve whitespace
whitespace: "strip" // remove whitespaces where applicable
}
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ractivejs/ractive/issues/3165#issuecomment-352443594,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACAWhwL5GvbFi8eY1M9DxoauEEtwvoV6ks5tBnkVgaJpZM4RFedO
.

Polymer's docs also say that stripping out the whitespace has "a minor performance improvement," which sounds nice.


[feat] Add option to disable window show/hide transitions

Describe the problem
Currently, on windows, showing / hiding a window isn't instantenious, there's a short fade-in / fade-out transition. This is very slightly inconvenient for my current use case (an always-on-top pseudo-fullscreen overlay window, toggled by global hotkeys).
Describe the solution you'd like
I'd like both the window builder and the window API to include an additional method to enable / disable such transitions.
Alternatives considered
none
Additional context
FWIW (and forgive my terrible rust), I was able to achieve this by directly calling the windows API:
#[cfg(target_os = "windows")]
use windows::Win32::{
    Foundation::{BOOL, HWND},
    Graphics::Dwm::{DwmSetWindowAttribute, DWMWA_TRANSITIONS_FORCEDISABLED},
};

...

if cfg!(target_os = "windows") {
    if let Ok(hwnd) = window.hwnd() {
        unsafe {
            let _ = DwmSetWindowAttribute(
                std::mem::transmute::<*mut c_void, HWND>(hwnd),
                DWMWA_TRANSITIONS_FORCEDISABLED,
                &mut BOOL::from(true) as *mut _ as *mut c_void,
                std::mem::size_of::<BOOL>() as u32,
            );
        }
    }
}

/upsrream tauri-apps/tao

/upstream tauri-apps/tao

I don't think this should be provided directly by us. This is a rare case and I think it makes sense that you do it in your code instead.


Update iata-code-decoder to v1.0.1

What's Changed

Bump minimist from 1.2.5 to 1.2.6 by @dependabot in https://github.com/timrogers/raycast-iata-code-decoder/pull/1
Bump ansi-regex from 3.0.0 to 3.0.1 by @dependabot in https://github.com/timrogers/raycast-iata-code-decoder/pull/2
Use pre-packaged Raycast GitHub action by @timrogers in https://github.com/timrogers/raycast-iata-code-decoder/pull/3

New Contributors

@dependabot made their first contribution in https://github.com/timrogers/raycast-iata-code-decoder/pull/1
@timrogers made their first contribution in https://github.com/timrogers/raycast-iata-code-decoder/pull/3

Full Changelog: https://github.com/timrogers/raycast-iata-code-decoder/commits/v1.0.1

Working on some automation! Closing this and making one last one...


Add some environment variables on issues reported from Linux to help identify distro and desktop environment

Advice from Canonical was to add these environment variables in issues reported from Linux.
DESKTOP_SESSION
XDG_SESSION_DESKTOP
XDG_CURRENT_DESKTOP

Example values:
DESKTOP_SESSION=ubuntu
XDG_SESSION_DESKTOP=ubuntu
XDG_CURRENT_DESKTOP=ubuntu:GNOME

Probably XDG_SESSION_TYPE should be reported too, since some issues can be related to running on Wayland with XWayland


🚀 Feature: Ability to search allowedOwners in RepoUrlPicker

🔖 Feature description
Expected Behavior
First of all, amazing work you guys are doing!
When multiple allowedOwners are in the template, the user has to scroll through a long list before selecting one while filling out the rest of the template form. The user would like to search and easily pick one from the long list.
Actual Behavior
Searching for one value of GitLab allowedOwners for the RepoUrlPicker
I have a very long list of GitLab allowedOwners for the RepoUrlPicker and requesting if a search feature can be added to quicken picking of GitLab groups.
Steps to Reproduce
Use a template with RepoUrlPicker containing multiple allowedOwners

- title: Choose a location
  required:
    - repoUrl
  properties:
    repoUrl:
      title: Repository Location
      type: string
      ui:field: RepoUrlPicker
      ui:options:
        allowedHosts:
          - gitlab.com
        allowedOwners:
          - some-gitlab-owner-A
          - some-gitlab-owner-B
          - some-gitlab-owner-C
          - some-gitlab-owner-D
          - some-gitlab-owner-E
          - some-gitlab-owner-F
          - some-gitlab-owner-G
          - some-gitlab-owner-H
          - some-gitlab-owner-I
          - some-gitlab-owner-J
          - some-gitlab-owner-K
        requestUserCredentials:
          secretsKey: USER_OAUTH_TOKEN

Ability to search the allowedOwners during creation of template
🎤 Context
In case there is a very long list of allowedOwners, it is cumbersome to scroll theough the number of options or list applied on the Template's RepoUrlPicker allowedOwners. A filter feature would really be useful to easily pick out one or two from the long drop-down list.
Your Environment
This behaviour happens in Safari and Firefox, though I mainly work in Firefox.
Output of yarn backstage-cli info:
yarn run v1.22.19
OS:   Darwin 22.1.0 - darwin/arm64
node: v16.17.1
yarn: 1.22.19
cli:  0.21.1 (installed)
backstage:  1.8.2
Dependencies:
@backstage/app-defaults                          1.0.8
@backstage/backend-common                        0.16.0
@backstage/backend-plugin-api                    0.1.4
@backstage/backend-tasks                         0.3.7
@backstage/catalog-client                        1.1.2
@backstage/catalog-model                         1.1.3
@backstage/cli-common                            0.1.10
@backstage/cli                                   0.21.1
@backstage/config-loader                         1.1.6
@backstage/config                                1.0.4
@backstage/core-app-api                          1.2.0
@backstage/core-components                       0.12.0, 0.9.5
@backstage/core-plugin-api                       1.1.0
@backstage/errors                                1.1.3
@backstage/integration-react                     1.1.6
@backstage/integration                           1.4.0
@backstage/plugin-api-docs                       0.8.11
@backstage/plugin-app-backend                    0.3.38
@backstage/plugin-auth-backend                   0.17.1
@backstage/plugin-auth-node                      0.2.7
@backstage/plugin-catalog-backend-module-gitlab  0.1.9
@backstage/plugin-catalog-backend                1.5.1
@backstage/plugin-catalog-common                 1.0.8
@backstage/plugin-catalog-graph                  0.2.23
@backstage/plugin-catalog-import                 0.9.1
@backstage/plugin-catalog-node                   1.2.1
@backstage/plugin-catalog-react                  1.2.1
@backstage/plugin-catalog                        1.6.1
@backstage/plugin-github-actions                 0.5.11
@backstage/plugin-kubernetes-backend             0.8.0
@backstage/plugin-kubernetes-common              0.4.4
@backstage/plugin-kubernetes                     0.7.4
@backstage/plugin-newrelic                       0.3.29
@backstage/plugin-org                            0.6.0
@backstage/plugin-permission-common              0.7.1
@backstage/plugin-permission-node                0.7.1
@backstage/plugin-permission-react               0.4.7
@backstage/plugin-proxy-backend                  0.2.32
@backstage/plugin-scaffolder-backend             1.8.0
@backstage/plugin-scaffolder-common              1.2.2
@backstage/plugin-scaffolder                     1.8.0
@backstage/plugin-search-backend-module-pg       0.4.2
@backstage/plugin-search-backend-node            1.0.4
@backstage/plugin-search-backend                 1.1.1
@backstage/plugin-search-common                  1.1.1
@backstage/plugin-search-react                   1.2.1
@backstage/plugin-search                         1.0.4
@backstage/plugin-sonarqube-backend              0.1.3
@backstage/plugin-sonarqube                      0.5.0
@backstage/plugin-tech-radar                     0.5.18
@backstage/plugin-techdocs-backend               1.4.1
@backstage/plugin-techdocs-module-addons-contrib 1.0.6
@backstage/plugin-techdocs-node                  1.4.2
@backstage/plugin-techdocs-react                 1.0.6
@backstage/plugin-techdocs                       1.4.0
@backstage/plugin-user-settings                  0.5.1
@backstage/release-manifests                     0.0.7
@backstage/test-utils                            1.2.2
@backstage/theme                                 0.2.16
@backstage/types                                 1.0.1
@backstage/version-bridge                        1.0.2
✌ Possible Implementation
Enable ability to search a word from the drop-down list generated
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

Am I understanding it right, that you refer primarily to essentially a little edit box that when you start typing, it filters the possible hits to those that match what's entered? That makes sense! Would you be able to contribute that? 🙏

Yes, that's right.. kindly point me in the right direction please..

Could this be potentially related or fixed by #15063?

That is absolutely right @benjdlambert ..

Seems like a nice addition, is this something you would like to contribute @cell-kibet?

Let me go through the code and docs and give it a try. Any input and direction is welcome

I think this is the component that you want to use instead: https://v4.mui.com/components/autocomplete
And I think for GitLab you'd want to edit this file: https://github.com/backstage/backstage/blob/master/plugins/scaffolder/src/components/fields/RepoUrlPicker/GitlabRepoPicker.tsx#L46-L57 and replace that Select component with a Autocomplete component.
It might be possible to always render an Autocomplete but turn off freeSolo if there is allowedOwners so that you can freely type something like the Input does underneath. But feel free to experiment.
Once you have something working, the allowedOwners prop is also used in the other files too that you could probably update pretty safely, like the GitHub Azure and Bitbucket ones too. Or wherever allowedOwners is used. :)

Seems GitlabRepoPicker.tsx imports "Select" from core-components file : https://github.com/backstage/backstage/blob/master/packages/core-components/src/components/Select/Select.tsx which uses the @material-ui/lab  Select component.
After swapping "Select" with "Autocomplete" in file: https://github.com/backstage/backstage/blob/master/packages/core-components/src/components/Select/Select.tsx,  am stuck here:
`
import Box from '@material-ui/core/Box';
import ClickAwayListener from '@material-ui/core/ClickAwayListener';
import FormControl from '@material-ui/core/FormControl';
import InputBase from '@material-ui/core/InputBase';
import InputLabel from '@material-ui/core/InputLabel';
import Autocomplete from '@material-ui/lab/Autocomplete';
import TextField from '@material-ui/core/TextField';
import {
createStyles,
makeStyles,
Theme,
withStyles,
} from '@material-ui/core/styles';
import React, { useEffect, useState } from 'react';
/** @public */
export type SelectInputBaseClassKey = 'root' | 'input';
const BootstrapInput = withStyles(
(theme: Theme) =>
createStyles({
root: {
'label + &': {
marginTop: theme.spacing(3),
},
},
input: {
borderRadius: theme.shape.borderRadius,
position: 'relative',
backgroundColor: theme.palette.background.paper,
border: '1px solid #ced4da',
fontSize: theme.typography.body1.fontSize,
padding: theme.spacing(1.25, 3.25, 1.25, 1.5),
transition: theme.transitions.create(['border-color', 'box-shadow']),
fontFamily: 'Helvetica Neue',
'&:focus': {
background: theme.palette.background.paper,
borderRadius: theme.shape.borderRadius,
},
},
}),
{ name: 'BackstageSelectInputBase' },
)(InputBase);
/** @public */
export type SelectClassKey =
| 'formControl'
| 'label'
| 'chips'
| 'chip'
| 'checkbox'
| 'root';
const useStyles = makeStyles(
(theme: Theme) =>
createStyles({
formControl: {
margin: ${theme.spacing(1)} 0px,
maxWidth: 300,
},
label: {
transform: 'initial',
fontWeight: 'bold',
fontSize: theme.typography.body2.fontSize,
fontFamily: theme.typography.fontFamily,
color: theme.palette.text.primary,
'&.Mui-focused': {
color: theme.palette.text.primary,
},
},
formLabel: {
transform: 'initial',
fontWeight: 'bold',
fontSize: theme.typography.body2.fontSize,
fontFamily: theme.typography.fontFamily,
color: theme.palette.text.primary,
'&.Mui-focused': {
color: theme.palette.text.primary,
},
},
chips: {
display: 'flex',
flexWrap: 'wrap',
},
chip: {
margin: 2,
},
checkbox: {},
  root: {
    display: 'flex',
    flexDirection: 'column',
  },
}),

{ name: 'BackstageSelect' },
);
/** @public */
export type SelectItem = {
label: string;
value: string | number;
};
/** @public */
export type SelectedItems = string | string[] | number | number[];
export type SelectProps = {
multiple?: boolean;
items: SelectItem[];
label: string;
placeholder?: string;
selected?: SelectedItems;
onChange: (arg: SelectedItems) => void;
triggerReset?: boolean;
native?: boolean;
disabled?: boolean;
margin?: 'dense' | 'none';
};
/** @public */
export function SelectComponent(props: SelectProps) {
const {
multiple,
items,
label,
placeholder,
selected,
onChange,
triggerReset,
native = false,
disabled = false,
margin,
} = props;
const classes = useStyles();
const [value, setValue] = React.useState(
selected || (multiple ? [] : ''),
);
const [isOpen, setOpen] = useState(false);
useEffect(() => {
setValue(multiple ? [] : '');
}, [triggerReset, multiple]);
useEffect(() => {
if (selected !== undefined) {
setValue(selected);
}
}, [selected]);
const handleChange = (event: React.ChangeEvent<{ value: unknown }>) => {
setValue(event.target.value as SelectedItems);
onChange(event.target.value as SelectedItems);
};
const handleClick = (event: React.ChangeEvent) => {
if (disabled) {
event.preventDefault();
return;
}
setOpen(previous => {
if (multiple && !(event.target instanceof HTMLElement)) {
return true;
}
return !previous;
});
};
const handleClickAway = () => {
setOpen(false);
};
const handleDelete = (selectedValue: string | number) => () => {
const newValue = (value as any[]).filter(chip => chip !== selectedValue);
setValue(newValue);
onChange(newValue);
};
return (

{label}
<Autocomplete
aria-label={label}
disabled={disabled}
data-testid="select"
multiple={multiple}
options={items}
onClick={handleClick}
open={isOpen}
tabIndex={0}
renderInput={params => (
<TextField
{...params}
label={label}
margin="dense"
FormHelperTextProps={{ margin: 'dense', style: { marginLeft: 0 } }}
variant="outlined"
InputProps={params.InputProps}
          />
        )}
        />
    </FormControl>
  </ClickAwayListener>
</Box>

);
}
`

@cell-kibet so I actually think that we might not want to adjust the @backstage/core-components/Select to be an Autocomplete I think it's fine to continue using that Select component. I think that we should remove the usage of that Select component inside the picker itself and use an Autocomplete instead.

Thank you @benjdlambert . Managed to get the Interface working by editing https://github.com/backstage/backstage/blob/master/plugins/scaffolder/src/components/fields/RepoUrlPicker/GitlabRepoPicker.tsx to below but I get an error once I click on the Owner text-field to pick the owner values:
import React from 'react';
import FormControl from '@material-ui/core/FormControl';
import FormHelperText from '@material-ui/core/FormHelperText';
import Input from '@material-ui/core/Input';
import InputLabel from '@material-ui/core/InputLabel';
import { Select, SelectItem } from '@backstage/core-components';
import Autocomplete from '@material-ui/lab/Autocomplete';
import { RepoUrlPickerState } from './types';
import TextField from '@material-ui/core/TextField';

export const GitlabRepoPicker = (props: {
  allowedOwners?: string[];
  allowedRepos?: string[];
  state: RepoUrlPickerState;
  onChange: (state: RepoUrlPickerState) => void;
  rawErrors: string[];
}) => {
  const { allowedOwners = [], state, onChange, rawErrors } = props;
  const ownerItems: SelectItem[] = allowedOwners
    ? allowedOwners.map(i => ({ label: i, value: i }))
    : [{ label: 'Loading...', value: 'loading' }];

  const { owner } = state;

  return (
    <>
      <FormControl
        margin="normal"
        required
        error={rawErrors?.length > 0 && !owner}
      >
        {allowedOwners?.length ? (
          <Autocomplete
          aria-label="Owner Available"
          options={ownerItems}
          disabled={allowedOwners.length === 1}
          data-testid="select"
          onChange={selected =>
            onChange({
              owner: String(Array.isArray(selected) ? selected[0] : selected),
            })
          }
          tabIndex={0}
          renderInput={params => (
              <TextField
              {...params}
              placeholder="Owner Available"
              margin="dense"
              FormHelperTextProps={{ margin: 'dense', style: { marginLeft: 0 } }}
              variant="outlined"
              InputProps={params.InputProps}
            />
          )}
          />
        ) : (
          <>
            <InputLabel htmlFor="ownerInput">Owner</InputLabel>
            <Input
              id="ownerInput"
              onChange={e => onChange({ owner: e.target.value })}
              value={owner}
            />
          </>
        )}
        <FormHelperText>
          GitLab namespace where this repository will belong to. It can be the
          name of organization, group, subgroup, user, or the project.
        </FormHelperText>
      </FormControl>
    </>
  );
};

The error
TypeError
candidate.toLowerCase is not a function
Call Stack
 eval
  plugins/scaffolder/node_modules/@material-ui/lab/esm/useAutocomplete/useAutocomplete.js:56:31
 eval
  plugins/scaffolder/node_modules/@material-ui/lab/esm/useAutocomplete/useAutocomplete.js:52:35
 useAutocomplete
  plugins/scaffolder/node_modules/@material-ui/lab/esm/useAutocomplete/useAutocomplete.js:238:37
 Autocomplete
  plugins/scaffolder/node_modules/@material-ui/lab/esm/Autocomplete/Autocomplete.js:407:86
 renderWithHooks
  node_modules/react-dom/cjs/react-dom.development.js:14985:18
 updateForwardRef
  node_modules/react-dom/cjs/react-dom.development.js:17044:20
 beginWork
  node_modules/react-dom/cjs/react-dom.development.js:19098:16
 HTMLUnknownElement.callCallback
  node_modules/react-dom/cjs/react-dom.development.js:3945:14
 Object.invokeGuardedCallbackDev
  node_modules/react-dom/cjs/react-dom.development.js:3994:16

@cell-kibet nice - do you want to raise a PR with these changes in the other places too? Like the GithubRepoUrlPicker and the other pickers that also live alongside it for the other providers too and we can do a bigger test then on the branch?
Nice work so far! :pray:

@cell-kibet nice - do you want to raise a PR with these changes in the other places too? Like the GithubRepoUrlPicker and the other pickers that also live alongside it for the other providers too and we can do a bigger test then on the branch?
Nice work so far! 🙏

Hey @benjdlambert kindly add my account to the repo to push to a new branch

@cell-kibet we encourage you to fork the project, and push to your own branches there and then raise a PR back to the original project :)


vesta: Fix pre_install and checkver properties

Closes #9617

[x] I have read the Contributing Guide.

/verify


MATLAB, Octave, Julia and FORTRAN icons

Hi,
Can you please consider providing icon fonts for the MATLAB, Octave, Julia and FORTRAN programming languages?
These are scientific computing tools and there is a highly likelihood of large overlap of the users of these languages with the LaTeX community.

Hi man,
sounds like an excellent suggestion. The mfizz font itself is not my work, however. You'll have to go over to https://github.com/fizzed/font-mfizz and ask Joe to put them in. Then, I can update this package accordingly.
Cheers!

@kdungs thanks! But the README.md for that repo clearly says that new icons nead pull-requests due to time commitments needed from the original package author.  An outstanding issue for the julia language icon already exists to which the project owner asks the community to make the icon and submit a PR. A recipe for doing this has been given.
I am too new to this sort of thing and scared of doing stuff incorrectly.   Can you perhaps give a go at making the matlab icon, and I can follow your lead shortly thereafter?

Maybe my original reply was not clear enough. I don't make the icons. :) All I did was write a little script that builds them into a LaTeX package.

I am too new to this sort of thing and scared of doing stuff incorrectly. Can you perhaps give a go at making the matlab icon, and I can follow your lead shortly thereafter?

Don't be scared to do it incorrectly. In the worst case, people will give you feedback on your PR and you can improve.
If nobody responds or your PR doesn't get in for other reasons that you disagree with, you can always make your own fork.


restrictToOwner unexpected behaviour

There seems to be some inconsistency between restrictToOwner hook documentation and behaviour.
Readme states it can be used with any method
Looking at source code, there is an error thrown when method is not get / patch / update / remove
Not sure whether that's just an outdated readme or code that was supposed to be removed after some changes.
Current conditions don't allow usage on create method, and therefore on all as well.

Well, it makes sense not to be on create, right? With find / get / patch / update / remove, there are resources to check against. With create, there's no resource to check against (not yet).

Sure, but it might as well, underneath, use associateCurrentUser on create. Just making sure if that's an overlook in docs or some unexpected behaviour.


fixed-width font in the example editor?

Heyo,
Have you considered using a fixed-width font in the examples editor?
I just modified the CSS locally and it seems to work well
textarea {
    font-family: monospace;
}

I hadn't considered it before but I added it


Add SetRequestedOrientation to Activity service

Fixes #743

I'm unclear as to what the two UNSPECIFIED and NOSENSOR values do.  Would someone ever want to pass NOSENSOR to this api?

lgtm

Would you like me to link to the android documentation?  NOSENSOR means ignore the sensor and use the factory default orientation for the screen.

Yes, a doc link sounds useful.

Where did the "Title" class go? I don't see it in the diff.

Nevermind, I see title.dart was checkin in separately.


Add pc and sourcemap for return constant;?

Hi, @dop251 . When testing the following code, I could not reach the line 4 with printing pc. Different with line 8, line 4 is a return constant; statement.
  1 var a = false;
  2 function f1() {
  3   let b = 1;
  4   return 1;
  5 }
  6 function f2() {
  7   let c = 1;
  8   return c;
  9 }
 10
 11 f1();
 12 f2();

I am not sure whether it is caused by lack of pc to line mapping. At least, it seems so:
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc000713140, dbg.vm.pc: 0, offset: 0, filename: test.js, line: 1
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc000713140, dbg.vm.pc: 1, offset: 0, filename: test.js, line: 1
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc000713140, dbg.vm.pc: 2, offset: 0, filename: test.js, line: 1
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc000713140, dbg.vm.pc: 3, offset: 0, filename: test.js, line: 1
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc000713140, dbg.vm.pc: 4, offset: 0, filename: test.js, line: 1
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc000713140, dbg.vm.pc: 5, offset: 4, filename: test.js, line: 1
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc000713140, dbg.vm.pc: 6, offset: 97, filename: test.js, line: 11
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc000713140, dbg.vm.pc: 7, offset: 99, filename: test.js, line: 11
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc0007131a0, dbg.vm.pc: 0, offset: 0, filename: test.js, line: 1
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc0007131a0, dbg.vm.pc: 1, offset: 0, filename: test.js, line: 1
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc0007131a0, dbg.vm.pc: 2, offset: 36, filename: test.js, line: 3
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc0007131a0, dbg.vm.pc: 3, offset: 36, filename: test.js, line: 3
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc0007131a0, dbg.vm.pc: 4, offset: 36, filename: test.js, line: 3
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc000713140, dbg.vm.pc: 8, offset: 99, filename: test.js, line: 11
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc000713140, dbg.vm.pc: 9, offset: 103, filename: test.js, line: 12
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc000713140, dbg.vm.pc: 10, offset: 105, filename: test.js, line: 12
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc000713200, dbg.vm.pc: 0, offset: 0, filename: test.js, line: 1
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc000713200, dbg.vm.pc: 1, offset: 0, filename: test.js, line: 1
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc000713200, dbg.vm.pc: 2, offset: 76, filename: test.js, line: 7
dbg.vm: 0xc0003ba120, dbg.vm.prg: 0xc000713200, dbg.vm.pc: 3, offset: 91, filename: test.js, line: 8

This cause an unexpected action when we use the debugger: the debugger won't break on this line when we set a breakpoint on it, as described in https://github.com/mostafa/goja/issues/16.

Ok, this is getting too far I think... Goja wasn't written with debugger in mind, the primary focus was on performance and standards compliance. In this sense srcMap is only used to provide accurate stacktrace locations, so it only makes sense to add locations that can throw, which does not include return, break, continue or pretty much any statement (you'll probably want to break on things likeif (true) as well?).
The solution I believe is having a special compilation mode for debugger which would enable things like forced local variable name maps (without which #356 is not doable) and more locations added to srcMap. Since you're already have a separate branch for debugger, you could implement it there and submit a PR when it's ready.

Got it. Thank you all the same. I'll close these two issues and try to work more on the debugger branch.


[FLINK-17541][table] Support inline structured types

What is the purpose of the change
This enables inline structured types in the Blink planner. Inline structured types are extracted (e.g. in UDFs) and don't need to be registered in a catalog. This PR finalizes FLIP-65 for scalar and table functions because existing functions can be migrated with a replacement to the new type system.
It also aims to add structured type support to all switch/case statements based on ROW. Structured type support should still be declared as experimental until we have more tests and can also deal with structured types in sources and sinks.
Brief change log
See commit messages.
Verifying this change
This change added tests and can be verified as follows: FunctionITCase
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): yes
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? JavaDocs

@dawidwys I updated the PR.


Broken link to display color

Hi @Spoutnik97 ;)
It looks like the website placehold.it you use to display colors is replaced by place-hold.it
I tried to create a PR with this fix but write permissions on your repo are restricted 🥺

Hi,
placehold.it redirects to placeholder.com now.
They may have taken the domain but don't have a proper TLS certificate yet
Seems to work as well
https://via.placeholder.com/15/red/000000/?text=+

Thanks a lot for the info @christophemenager !
I deploy a fix now 😉
P.S: I open write permissions too for futur contribution

Fixed in the version 1.1.7


V1 fixes 2407 toplevel fix

The problem with the existing code with using a Window do add a menu and a status bar was because they are added directly to it, instead Window handling them by adding to the superview. With this system we can have any number of nested toplevels with menus and scroll bar with adjustments on top and bottom. See the unit test for the nested toplevels. I also fixed all the unit tests failures. If you agree you can change the xml documents because now it's not only the Application.Top can handle the menu and status bar adjustments.

I don't think we should take this for v1.

I think it should take this for v1.

It adds new public APIs and fixes a functionality problem that is not really impacting any external developers.

The only new public API is the GrabMouseEventArgs class that will ensuring more drag control over different views, which will not impacting to much and will reflect when the documentation is regenerated.

Can you explain why you think we should do this for v1?

Because I would like to prove that the v1 really can have nested toplevels with their own MenuBar and StatusBar and not with the prior right, but now wrong information that you ended up to classify on the internal View EnsureVisibleBounds.

@tig this PR isn't easy to merge for the v2, but let it to me and I'll apply all the changes in the v2 as well. So, you don't need to worry about merge it on v2 if you don't want have that work,


Diarization Inconsistencies with Nova-2 and Limited Language Support in Other Models

The diarization feature in Deepgram has the following issues:
Inconsistent Diarization with Nova-2:
Speaker detection is unreliable when using the nova-2 model with diarization enabled for supported languages.
Sometimes speakers are detected, but the diarization accuracy is low.
Diarization Limitations in Other Models:
Models like nova-phonecall and nova-medical do not support diarization for languages other than English.
Attempting diarization with non-English languages on these models results in a 400 Bad Request error.

Hi @rizz-official this is unfortunately not an issue with the SDK so I will close it here. You're welcome to raise it on our community Discord or GitHub discussions.


creating build

Could not able to create a build using 5.1.2.tar.gz using "build/clean_build.sh"
[root@ops trafficcontrol]# build/clean_build.sh
++ readlink build/clean_build.sh
+ script_path=
+ script_path=build/clean_build.sh
++ dirname build/clean_build.sh
+ cd build
++ git rev-parse --show-toplevel
+ tc_volume=
+ tc_volume=/trafficcontrol
+ export GOPATH=/tmp/go GOOS=linux
+ GOPATH=/tmp/go
+ GOOS=linux
+ tc_dir=/tmp/go/src/github.com/apache/trafficcontrol
+ which cygpath
+ mkdir -p /tmp/go
+ cd /tmp/go
++ dirname /tmp/go/src/github.com/apache/trafficcontrol
+ mkdir -p src pkg bin /tmp/go/src/github.com/apache
+ rsync -a --exclude=/dist --exclude=/.m2 /trafficcontrol/ /tmp/go/src/github.com/apache/trafficcontrol
rsync: change_dir "/trafficcontrol" failed: No such file or directory (2)
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1179) [sender=3.1.2]
+ exit_code=23
+ '[' 23 -ne 0 ']'
+ echo 'Error on line 1 of build/clean_build.sh'
Error on line 1 of build/clean_build.sh
+ cleanup
+ setowner /trafficcontrol /trafficcontrol/dist
++ stat -c%u:%g /trafficcontrol
++ stat -f%u:%g /trafficcontrol
stat: invalid option -- '%'
Try 'stat --help' for more information.
+ own=
[root@ops trafficcontrol]#

Where as if I clone https://github.com/apache/trafficcontrol , the same is working and version 6 is getting built.

Also, postinstall is giving the following error
/opt/traffic_ops/install/bin/postinstall -a --cfile /opt/traffic_ops/install/bin/input.json
OK<PHONE_NUMBER>000002_add_max_request_header_size_delivery_service.sql
2021/09/26 14:28:11 FAIL<PHONE_NUMBER>000003_server_interface_ip_address_cascade.sql (pq: constraint "ip_address_interface_fkey" of relation "ip_address" does not exist), quitting migration.
Can't run goose: exit status 1
Database migrate failed
Died at /opt/traffic_ops/install/lib/InstallUtils.pm line 62.
Error on line 57 of /opt/traffic_ops/install/bin/postinstall
This is first time installation.

Could not able to create a build using 5.1.2.tar.gz using "build/clean_build.sh"

The official, supported way of build ATC components is by running ./pkg. If you wanted to try building ATC 5.1.2 using build/clean_build.sh anyway, you would need to clone the repo using git and checkout the RELEASE-5.1.2 tag.

Also, postinstall is giving the following error

Looks like you opened #6237 for this.
I noticed that you created #6236 describing a totally separate bug:

hashed = hashlib.scrypt(passwd.encode(), salt=salt, n=n, r=r_val, p=p_val, dklen=dklen)
AttributeError: module 'hashlib' has no attribute 'scrypt'

Please do not edit Issues to describe a different bug than the bug described when it was created.

[root@ops dist]# /opt/traffic_ops/install/bin/postinstall -a --cfile /opt/traffic_ops/install/bin/input.json
INFO:root:Starting postinstall
INFO:root:Debug is on
INFO:root:Running in automatic mode
INFO:root:Using input file /opt/traffic_ops/install/bin/input.json
INFO:root:Adding question 'Port to serve on?' with default answer 443
INFO:root:Adding question 'ldap.conf location?' with default answer /opt/traffic_ops/app/conf/ldap.conf
INFO:root:Adding question 'LDAP server hostname' with default answer
INFO:root:Adding question 'LDAP Admin Password' with default answer
INFO:root:Adding question 'LDAP Search Query' with default answer
INFO:root:Adding question 'LDAP Skip TLS verify' with default answer
INFO:root:Adding question 'LDAP Timeout Seconds' with default answer
INFO:root:File sanity check complete - found 7 differences
INFO:root:===========/opt/traffic_ops/app/conf/production/database.conf===========
INFO:root:Database configuration has been saved
INFO:root:===========/opt/traffic_ops/app/conf/production/tv.conf===========
INFO:root:Database configuration has been saved
INFO:root:Not setting up ldap
INFO:root:===========/opt/traffic_ops/install/data/json/users.json===========
Traceback (most recent call last):
File "/opt/traffic_ops/install/bin/_postinstall.py", line 1528, in
ARGS.no_database
File "/opt/traffic_ops/install/bin/_postinstall.py", line 1304, in main
root_dir
File "/opt/traffic_ops/install/bin/_postinstall.py", line 580, in generate_users_conf
hashed_pass = hash_pass(config["tmAdminPw"])
File "/opt/traffic_ops/install/bin/_postinstall.py", line 433, in hash_pass
hashed = hashlib.scrypt(passwd.encode(), salt=salt, n=n, r=r_val, p=p_val, dklen=dklen)
AttributeError: module 'hashlib' has no attribute 'scrypt'
Error on line 50 of /opt/traffic_ops/install/bin/postinstall
I have installed all the dependencies and still getting this error. I am using CentOS 7.

@rsidhaarth This was an edge-case caused by the fact that you have Python 3.6 installed, in addition to the default Python 2.7 installation on CentOS 7. The fix, #6285, is included in Traffic Control 6.0.1.


Fix ObjectPool refill termination condition

This PR fixes the broken ObjectPool refilling subroutine – previously, the break statement would not break out of the loop since it is used inside a select block.

Coverage decreased (-0.09%) to 78.133% when pulling 21c6f211e315d43872f62e57410988db72a4a16e on kobolog/object-pool-refill-termination into a05864b30490092059fc6b8cc630d561069529f9 on master.


Fix: Endless Icon Insertion at the Top of the File's Window

Closes #22

What is changed?
The condition is trying to match an a element with a viewActions class and aria-label="Actions". However, this does not match, since the elements in the viewActions are of type button. At least, in my version of Obsidian (v1.6.7).

Changing a to button seems to fix the problem for me. To verify that, I made the change in .obsidian\plugins\typing\main.js on line 163363 of the installed plugin.
Before change:

After change:

hi, this is great, i am ready to merge it as soon as you undraft this pull


UX Problems using the table

Hello there,
we integrated the plugin into the Quill Editor and it works as expected. But there are some major UX Problems while using the table. You can delete partial contents of the table if you, for example, double click a cell and press "del" and if you double click inside a cell and try to enter values. Currently the table can be destroyed easily and even "undone" cannot fix it when you once destroyed it. You can reproduce those issues on the demo page.
Is there a way to address these issues?
regards

+1
We are facing the same issues.

Same issue even till now.


cri  log-uri shuld support log-uri=file

What is the problem you're trying to solve
cri default  log-uri is fifo, containerd will use io.copy to generate logs for every container,
why not change to  log-uri=file which will reduce a lot of io.copy for containerd,will save some memory for containerd?
Describe the solution you'd like
cri   log-uri support   log-uri=file
Additional context
No response

pod.yaml
apiVersion: apps/v1
kind: Deployment
metadata:
  name: busybox
spec:
  replicas: 50
  selector:
    matchLabels:
      app: nginx
  template:
    metadata:
      labels:
        app: nginx
    spec:
      containers:
      - name: nginx-web
        image: busybox:latest

default
USER       PID  %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
root      3571  3.4  2.2 2811904 114668 ?      Ssl  05:15   1:10 /usr/bin/containerd

use  log-uri=file
USER       PID  %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
root     26816  9.4  1.7 1447880 89384 ?       Ssl  05:52   0:14 /usr/bin/containerd

will save about  22% memory of containerd
（114668-89384）/114668 = 22%


Direct access to change state

Currently, to change the state of a task, we have to click and change it inside the modal window.
It would be great to be able to change the state from the main page, just clicking on the task box, or in a button shown in this box

This looks difficult, as adding more options to the task will either make each clickable area of the task too small or will make the task "card" too big. I'll look into it when revamping the card style though (so not closing this now).
Thanks for the suggestion!


Stackblitz Angular projects ignore proxy.conf.json

Describe the bug
When creating a proxy.conf.json file, angular (really Webpack dev server) handles re-writing the urls so that, for example, if I navigate to /api/... I can setup the proxy.conf.json file to rewrite that to a different url. Angular projects in Stackblitz seem to ignore the proxy.conf.json file
Link to the blitz that caused the error
https://stackblitz.com/edit/angular-fundamentals-exploration-lpkxzw?file=src%2Fproxy.conf.json
Steps to reproduce

Run the attached stackblitz and notice the 404 errors in the console. They are not re-written according to the proxy.conf.json rules.

Expected behavior
URL rewriting should occur according to the proxy.conf.json.
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots
No response
Platform
Browser name  = Chrome
Full version  = 100.0.4896.127
Major version = 100
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/100.0.4896.127 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 107616659,
  "usedJSHeapSize": 101251999,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = 49c6d2ee58ce7c1528881cc6d46ebf056b3b42bf

Additional context
No response

This was my failure. I hadn't updated the angular.json file to point to my proxy.conf.json.


Fixed #33622 -- Allow overriding FormSet's error messages for too few and too many forms

This PR tries to add the changes needed to fulfill the requirements in https://code.djangoproject.com/ticket/33622.
The code updated moves the messages for too_many_forms and too_few_forms errors to default_error_messages in the BaseFormSet class. This allows overriding the message value for those keys when using the constructor for the formsets.
The code mimics the behavior used for the already customizable messages related to missing_management_form errors.
This is my first time contributing to Django so any suggestions (regarding my code, tests, docs or format used) are appreciated! 🚀

Original author of the patch here…
@Metarizard thanks for adding the tests!
As it is, the documentation should say "the error message must include %(num)" since ngettext_lazy() uses ... % number to format the exception message.
I found no way around this (would like to use a translated message that doesn't include a reference to the number), but got stuck, since this is a fundamental problem in gettext itself as well.

Original author of the patch here…
@Metarizard thanks for adding the tests!
As it is, the documentation should say "the error message must include %(num)" since ngettext_lazy() uses ... % number to format the exception message.

As far as I'm aware this is not true. This patch allows customizing error messages so they don't need to use ngettext_lazy() anymore and don't need to contain  %(num)s, e.g.
FormSet(
    error_messages={
        "too_many_forms": gettext_lazy("Custom message without num.")
    },
)

Original author of the patch here…
@Metarizard thanks for adding the tests!
As it is, the documentation should say "the error message must include %(num)" since ngettext_lazy() uses ... % number to format the exception message.

As far as I'm aware this is not true. This patch allows customizing error messages so they don't need to use ngettext_lazy() anymore and don't need to contain  %(num)s, e.g.
FormSet(
    error_messages={
        "too_many_forms": gettext_lazy("Custom message without num.")
    },
)

Yeah you are right (in fact I tested that in the first commits I pushed). I think that what @hop meant is that to format the string it must contain the %(num)d part (i.e. we cannot format with another key like %(number)d, i will raise a KeyError). This doesn't affect plain string literals like the one in your example. It's explained here too https://docs.python.org/3/library/stdtypes.html#printf-style-string-formatting:

When the right argument is a dictionary (or other mapping type), then the formats in the string must include a parenthesised mapping key into that dictionary inserted immediately after the '%' character. The mapping key selects the value to be formatted from the mapping

If you're fine with it, we should rephrase it in a way that this is clear in the docs. Also, in the same link I sent there's info about %s and %d. I used %d in my code as we're dealing with integers, but the doc changes you suggested use %s. We should also agree on which one to use and use the same conversion type across the patch for consistency.

I used %d in my code as we're dealing with integers, but the doc changes you suggested use %s.

Updated, it was a typo.

@Metarizard Thanks for updates :+1: Welcome aboard :boat:

Sorry for the confusion, I remembered poorly.
I think I mixed up some  draft version of mine and the original code ( % self.min_num instead of  % {...}).
But, as I said, my main fight was with gettext that night, anyway.
Thanks again for polishing up my patch, even though it's now useless to me :-D


Write permissions

When I launch python ./macro_safe.py ./demmsec.bat I had this error:
'---VBA arch detection by "T"---
Error writing file.
Please check permissions and try again.
Exiting...
both macro_safe.py and demmsec.bat are chmod 777, I tried in kali-roll and ubuntu 14.04.
Thank you for your support

It worked giving output file!


Add configurable delays before requests

This is to prevent hammering Yahoo to the point of getting lots of 500s.

Merging since we're having issues with Yahoo groups getting overloaded.


i18n Update Dutch translation.json

Added all new translations and changed a lot to better represent Dutch language rules
Pull Request Checklist
Note to first-time contributors: Please open a discussion post in Discussions and describe your changes before submitting a pull request.
Before submitting, make sure you've checked the following:

[x] Target branch: Please verify that the pull request targets the dev branch.
[x] Description: Provide a concise description of the changes made in this pull request.
[ ] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[ ] Documentation: Have you updated relevant documentation Open WebUI Docs, or other documentation sources?
[ ] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?
[ ] Testing: Have you written and run sufficient tests for validating the changes?
[ ] Code review: Have you performed a self-review of your code, addressing any coding standard issues and ensuring adherence to the project's coding standards?
[x] Prefix: To cleary categorize this pull request, prefix the pull request title, using one of the following:

BREAKING CHANGE: Significant changes that may affect compatibility
build: Changes that affect the build system or external dependencies
ci: Changes to our continuous integration processes or workflows
chore: Refactor, cleanup, or other non-functional code changes
docs: Documentation update or addition
feat: Introduces a new feature or enhancement to the codebase
fix: Bug fix or error correction
i18n: Internationalization or localization changes
perf: Performance improvement
refactor: Code restructuring for better maintainability, readability, or scalability
style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc.)
test: Adding missing tests or correcting existing tests
WIP: Work in progress, a temporary label for incomplete or ongoing work

Changelog Entry
Description

Updated Dutch translations

Added

Added all missing translations

Changed

Updated a lot of existing translations to better Dutch

Deprecated

[List any deprecated functionality or features that have been removed]

Removed

[List any removed features, files, or functionalities]

Fixed

[List any fixes, corrections, or bug fixes]

Security

[List any new or updated security-related changes, including vulnerability fixes]

Breaking Changes

BREAKING CHANGE: [List any breaking changes affecting compatibility or functionality]

Additional Information

[Insert any additional context, notes, or explanations for the changes]

[Reference any related issues, commits, or other relevant information]

Screenshots or Videos

[Attach any relevant screenshots or videos demonstrating the changes]

Thanks!


Add discord specific webhook

I use discord webhooks, but every other application I use has schemas for discord that can talk directly to the webhook. I don't really want to sign up for another site (discordnotifier), invite bots, and all that. And this pattern would make it a bit easier to implement others like slack, telegram, etc...
This isn't all that challenging either, as the data unpackerr needs to send is relatively basic.
https://discord.com/developers/docs/resources/webhook
This is the same as the arr apps, with a discord specific webhook trigger. There would be a config option for discord under webhooks or something like that.
Happy to open a PR for this as it's a side issue and not of major importance.

I've been putting a lot of time into the discordnotifier-client application, but will get back to unpackerr to add these new features soon. Going to combine this with #73.


feature: MDX support

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Similar to improving markdown files visually, it would be great if this plugin worked for mdx files as well.
Describe alternatives you've considered
I tried the following, but it didn't work for mdx files:
return {
  {
    "MeanderingProgrammer/render-markdown.nvim",
    opts = {},
    dependencies = { "nvim-treesitter/nvim-treesitter", "echasnovski/mini.icons" }, -- if you use standalone mini plugins
    config = function()
      require("render-markdown").setup {
        file_types = { "markdown", "mdx" },
      }
    end,
  },
}

I've got the mdx-analyzer and marksman LSPs installed as well.
Perhaps custom handlers could work?
Additional information
No response

Hmm, I think how I support this would depend on how users configure treesitter to work with mdx files. In your case how do you setup highlighting for mdx files?
Do you download a specific parser, I don't see one in nvim-treesitter?
Or do you configure a filetype with something like vim.filetype.add?

To get this plugin to work in mdx files you need to get markdown to be recognized in the file.
One way to do this is to add the following somewhere in your config so that mdx files are recognized and parsed as markdown:
vim.filetype.add({
    extension = { mdx = 'mdx' },
})
vim.treesitter.language.register('markdown', 'mdx')

Then configure the plugin like in your example:
require('render-markdown').setup({
    file_types = { 'markdown', 'mdx' },
})

However this will not look great as all the TS/JS will fail to be highlighted correctly since the entire buffer is read as markdown.
You'll need to do language injections somehow, maybe there's a plugin that does a better job of this but I could not find one. Hope this helps!

for the people that wanted this, you can follow this article
https://phelipetls.github.io/posts/mdx-syntax-highlight-treesitter-nvim/
and there's also the plugin if you don't want to set it up manually
https://github.com/davidmh/mdx.nvim


Add missing comma to search_fields

as per this issue: https://github.com/b12io/orchestra/issues/818#issuecomment-1004935674
Note I found this issue while integration testing some new checks we've written. I'm a GitHub code review bot and part of my integration testing of new checks includes running against 1000 codebases (this one included).
If you're interested: you can add me to GitHub so I review PRs automatically and prevent issues like this being merged in the first place :)

Thank you @code-review-doctor!


jQuery/TinyMCE error on fresh install

Hi,
Actual Behavior: on a fresh install of BookStack (v0.11.1) with Google Chrome on Mac, I created a new book and a new page. On this page edition, I can set some styles with keyboard shortcuts (cmd+B for bold, etc) but when I want to add a link by selecting a text and clicking on the "link" TinyMCE icon, I got an error. Idem on clicking all the other buttons : images, tables, colors, etc.
Uncaught TypeError: Cannot read property 'ownerDocument' of undefined tinymce.min.js?ver=4.3.7:11
Expected Behavior: clicking on TinyMCE buttons should work properly on Google Chrome. Note that it works on Firefox 41.0.2 on Mac.
OS: Debian GNU/Linux 8.5 (jessie)
PHP Version: PHP 7.0.9-1~dotdeb+8.1
MySQL Version: 15.1 Distrib 10.0.26-MariaDB
BookStack Version: v0.11.1
Browser Version: Google Chrome Version 51.0.2704.106 (64-bit) / OSX

Hi @JulienRamel,
Can you login to the BookStack demo instance and let me know if you can re-produce the issue in the editor there aswell?
If that does not work could you also test on this TinyMCE example page here and let me know the results.

Hi @ssddanbrown, I also have the issue on the BookStack demo instance but not on the TinyMCE example page.

Hi @JulienRamel, For BookStack v0.12.1 I have updated TinyMCE to a newer version. Could you give it another go? I've updated the demo instance to the latest version so you can try on there again to test the updated version.
If it still doesn't work in chrome can you try in an incognito window (So any/most plugins are disabled) just to rule those out? I've given v0.12.1 a quick test using BrowserStack (Don't have a mac to hand) and all appeared fine from my quick testing.

Hi @ssddanbrown, I tested on Chrome on the demo application with BookStack v0.12.1 and it works great 👍 But I'm now on Chrome 52...

Okay, Good news, I'll mark this as closed for now then. Just shout if you have any other issues on Chrome/Mac.


How to refer to species in JBrowse URLs

This follows on from https://github.com/alliance-genome/agr_schemas/issues/14
With the changes to the schema, each gene page will "know" the taxon id and location of the feature.  What I'd like to know is this: should I use the taxon id for building JBrowse URLs (which will be somewhat unattractive, but pretty future proof) or should I use the something like species name (eg "C_elegans" or "C%20elegans") or use the human readable name ("worm").  Presumably, the gene page will "know" these things to and so would be available for building the URL (right?)

I think this has been worked out. The full species name is what the gene page gets for basic gene info, and I believe JBrowse has been updated to use it.


codecs/zip/verbose ignored by encode

Setting codecs/zip/verbose: false works for codecs/zip/decode but not codecs/zip/encode.

The question is, if there should be the verbose setting, because preferred way how to control verbosity is using system/options/log. Setting it to zero for the zip should silent it (except error level messages).

Yes, log setting makes more sense now.


Fix problems with nim check

Multi file errors are not shown in the same buffer
Fixes parsing of error type that contain ':'

Cheers! :beers:

:beers:

:beers:


[feature request]improve configuration of concurrent servicetopology endpointslice workers

What would you like to be added:
please refer to pull request https://github.com/openyurtio/openyurt/pull/1906 and improve servicetopology-endpointslice-workers configuration for servicetopology-endpointslice controller in yurt-manager component.
https://github.com/openyurtio/openyurt/blob/ed209553bdb8ba128771f5fb2c53d74e0dabcd8e/pkg/yurtmanager/controller/servicetopology/endpointslice/endpointslice_controller.go#L42
others
/kind feature

Hey @rambohe-ch I can take on this but I need some more details and steps to get started as it will be my first issue

@Bharadwajshivam28 Do you have any progress about this issue?

@rambohe-ch I think we can close it because https://github.com/openyurtio/openyurt/pull/2086 fix it.

@rambohe-ch I think we can close it because #2086 fix it.

@YTGhost Thank you for your notice, i will close this issue.


dwarf-fortress: add dfhack

I added dfhack as an option to dwarf-fortress via the enableDFHack option.
At the same I did do some refactoring in order to keep the code clean, adding template files, etc. Otherwise I feel the addition of dfhack would have made things quite ugly.

Did you see https://github.com/NixOS/nixpkgs/pull/9972 ?

No, unfortunately I didn't. That must have happened as I was working on this. I did try to ask around on IRC a few times, and @roconnor said it would be fine for me to work on this, but I never caught you @the-kenny.
I haven't had a chance to try that new package yet, but from reading it seems that it would require two steps: run dfhack_install_dir and then run dfhack.
My approach I am hoping is cleaner than that. If I'm wrong about how to run dfhack from #9972 then problem solved for me.

If this PR implements a cleaner solution, I'm all for it. Unfortunately I'm quite busy right now and can't say when I'll be able to compare both implementations.

I've attempted to contact @robbinch who created the other pull request. I'll try and get their feedback. Perhaps we can merge our efforts somehow. Anyone else you would trust to review the implementations in lieu of yourself, @the-kenny?

Hi, my opinion follows:
Pros

Integrates better with existing dwarf fortress package.
DFHack binary includes stonesense. (#10082 doesn't)

Cons

Does not build DFHack from source. (#10082 does)

Is it too much to ask to build DFHack from source in this PR? I think doing this will make an even cleaner solution.

@robbinch Sure, I'll change it to build from source. Thanks for taking a look.

Since it was quite a bit of difference, and I wanted to be able to see this version as a reference, I did the work to build dfhack from source in a new branch and put it into #10800. I'll close this PR if you end up liking #10800.

Closed in favor of #10800


Add autoFocus to

onBlur usually do not fired because element do not receive focus after starting editing, demo:
https://reactabular.js.org/#/examples/all-features

Current coverage is 100% (diff: 100%)

Merging #1 into master will not change coverage

@@           master    #1   diff @@
===================================
  Files           4     4
  Lines          49    49
  Methods        13    13
  Messages        0     0
  Branches       19    19
===================================
  Hits           49    49
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update b4dd2d9...a47ea8f

Thanks. Included in 6.2.0.


TLSRollingFileOutputStream in each run creates a new file. Why not append in last?

Hi, if restart the app frequently most of logs are lost. Perhaps it would be better to log in last file?

Please refer to Issue #8

Feel free to reopen if your question is not answered.


Fail to start up with: Failed to probe tracepoint

Running parca-agent in Docker on Unraid 7.0.0-beta.2.
# uname -a
Linux Magda 6.8.12-Unraid #3 SMP PREEMPT_DYNAMIC Tue Jun 18 07:52:57 PDT 2024 x86_64 Intel(R) Xeon(R) CPU E5-2620 v4 @ 2.10GHz GenuineIntel GNU/Linux

The container exits soon after startup due to it not finding /sys/kernel/debug/tracing/events/syscalls/sys_enter_mmap/id.
parca-agent-1  | time="2024-09-03T07:36:39Z" level=error msg="Failed to probe tracepoint: failed to get id for tracepoint: failed to read tracepoint ID for sys_enter_mmap: open /sys/kernel/debug/tracing/events/syscalls/sys_enter_mmap/id: no such file or directory"
parca-agent-1  | time="2024-09-03T07:36:39Z" level=error msg="======================= unexpected error ======================="
parca-agent-1  | time="2024-09-03T07:36:39Z" level=error msg="time=\"2024-09-03T07:36:39Z\" level=error msg=\"Failed to probe tracepoint: failed to get id for tracepoint: failed to read tracepoint ID for sys_enter_mmap: open /sys/kernel/debug/tracing/events/syscalls/sys_enter_mmap/id: no such file or directory\"\n"
parca-agent-1  | time="2024-09-03T07:36:39Z" level=error msg="================================================================"
parca-agent-1  | time="2024-09-03T07:36:39Z" level=error msg="about to report error to server"
parca-agent-1  | time="2024-09-03T07:36:39Z" level=info msg="report sent successfully"
parca-agent-1 exited with code 2

Looking at the host, the kernel debugfs is mounted, but the syscalls directory does not exist.
# ls /sys/kernel/debug/tracing/events/
9p/            cgroup/      devlink/       filelock/     huge_memory/  iommu/        kvmmmu/      migrate/    netlink/         percpu/        rpm/     sock/     tlb/        writeback/
alarmtimer/    clk/         dma_fence/     filemap/      hwmon/        ipi/          kyber/       mmap/       nmi/             power/         rseq/    swiotlb/  tls/        x86_fpu/
amd_cpu/       cma/         drm/           fs_dax/       hyperv/       irq/          libata/      mmap_lock/  notifier/        printk/        rtc/     target/   udp/        xdp/
block/         compaction/  enable         ftrace/       i2c/          irq_matrix/   lock/        module/     nvme/            qdisc/         sched/   task/     vmalloc/    xfs/
bpf_test_run/  cpuhp/       error_report/  gpio/         initcall/     irq_vectors/  maple_tree/  msr/        oom/             ras/           scsi/    tcp/      vmscan/     xhci-hcd/
bpf_trace/     csd/         exceptions/    handshake/    intel_iommu/  kmem/         mce/         napi/       page_isolation/  raw_syscalls/  signal/  thermal/  vsyscall/
bridge/        dev/         fib/           header_event  io_uring/     ksm/          mdio/        neigh/      page_pool/       rcu/           skb/     thp/      wbt/
btrfs/         devfreq/     fib6/          header_page   iomap/        kvm/          mei/         net/        pagemap/         regmap/        smbus/   timer/    workqueue/

Hi @jpmeijers ,
Can you please check if your kernel was compiled with CONFIG_HAVE_SYSCALL_TRACEPOINTS=y ?

Same question for CONFIG_FTRACE_SYSCALLS

Possibly, this is fixed by https://github.com/parca-dev/parca-agent/pull/2990

Presumably related: https://github.com/evilsocket/opensnitch/issues/774

@jpmeijers , can you please try again with the most recent image? ghcr.io/parca-dev/parca-agent:main-48f7888d

I tried the new container version on two different hosts. A physical Dell server running Unraid, and a VM running Ubuntu 20.04. Both fail with the same error:
Failed to load eBPF tracer: failed to read kernel symbols: unable to open /proc/kallsyms: open /proc/kallsyms: no such file or directory

Unraid:
# grep FTRACE /boot/config-$(uname -r)
grep: /boot/config-6.8.12-Unraid: No such file or directory

VM + Ubuntu:
# grep FTRACE /boot/config-$(uname -r)
CONFIG_HAVE_KPROBES_ON_FTRACE=y
CONFIG_HAVE_DYNAMIC_FTRACE=y
CONFIG_HAVE_DYNAMIC_FTRACE_WITH_REGS=y
CONFIG_HAVE_FTRACE_MCOUNT_RECORD=y
CONFIG_FTRACE=y
# CONFIG_FTRACE_SYSCALLS is not set

The tracepoint error has however been replaced by a warning, so this issue can be closed. I'll have to open a new one about eBPF.
parca-agent-1  | 2024/10/25 13:36:42 INFO memory is not limited, skipping package=github.com/KimMachineGun/automemlimit/memlimit
parca-agent-1  | 2024/10/25 13:36:42 INFO memory is not limited, skipping package=github.com/KimMachineGun/automemlimit/memlimit
parca-agent-1  | ooooooooo.                                                  .o.                                            .
parca-agent-1  | `888   `Y88.                                               .888.                                         .o8
parca-agent-1  |  888   .d88'  .oooo.   oooo d8b  .ooooo.   .oooo.         .8"888.      .oooooooo  .ooooo.  ooo. .oo.   .o888oo
parca-agent-1  |  888ooo88P'  `P  )88b  `888""8P d88' `"Y8 `P  )88b       .8' `888.    888' `88b  d88' `88b `888P"Y88b    888
parca-agent-1  |  888          .oP"888   888     888        .oP"888      .88ooo8888.   888   888  888ooo888  888   888    888
parca-agent-1  |  888         d8(  888   888     888   .o8 d8(  888     .8'     `888.  `88bod8P'  888    .o  888   888    888 .
parca-agent-1  | o888o        `Y888""8o d888b    `Y8bod8P' `Y888""8o   o88o     o8888o `8oooooo.  `Y8bod8P' o888o o888o   "888"
parca-agent-1  |                                                                       d"     YD
parca-agent-1  |                                                                       "Y88888P'
parca-agent-1  |
parca-agent-1  | time="2024-10-25T13:36:42Z" level=warning msg="Failed to probe tracepoint: failed to get id for tracepoint: failed to read tracepoint ID for sys_enter_mmap: open /sys/kernel/debug/tracing/events/syscalls/sys_enter_mmap/id: no such file or directory. Parca-agent may fail to run on some kernel versions."
parca-agent-1  | time="2024-10-25T13:36:42Z" level=info msg="External labels: "
parca-agent-1  | time="2024-10-25T13:36:42Z" level=info msg="Interpreter tracers: perl,php,python,hotspot,ruby,v8,dotnet"
parca-agent-1  | time="2024-10-25T13:36:42Z" level=info msg="no config file provided, using default config"
parca-agent-1  | time="2024-10-25T13:36:42Z" level=info msg="Can't connect Docker client to [/var/run/docker.sock]"
parca-agent-1  | time="2024-10-25T13:36:42Z" level=info msg="Environment variable KUBERNETES_SERVICE_HOST not set"
parca-agent-1  | time="2024-10-25T13:36:42Z" level=error msg="Failed to load eBPF tracer: failed to read kernel symbols: unable to open /proc/kallsyms: open /proc/kallsyms: no such file or directory"
parca-agent-1  | time="2024-10-25T13:36:42Z" level=error msg="======================= unexpected error ======================="
parca-agent-1  | time="2024-10-25T13:36:42Z" level=error msg="2024/10/25 13:36:42 INFO memory is not limited, skipping package=github.com/KimMachineGun/automemlimit/memlimit\ntime=\"2024-10-25T13:36:42Z\" level=warning msg=\"Failed to probe tracepoint: failed to get id for tracepoint: failed to read tracepoint ID for sys_enter_mmap: open /sys/kernel/debug/tracing/events/syscalls/sys_enter_mmap/id: no such file or directory. Parca-agent may fail to run on some kernel versions.\"\ntime=\"2024-10-25T13:36:42Z\" level=info msg=\"External labels: \"\ntime=\"2024-10-25T13:36:42Z\" level=info msg=\"Interpreter tracers: perl,php,python,hotspot,ruby,v8,dotnet\"\ntime=\"2024-10-25T13:36:42Z\" level=info msg=\"no config file provided, using default config\"\ntime=\"2024-10-25T13:36:42Z\" level=info msg=\"Can't connect Docker client to [/var/run/docker.sock]\"\ntime=\"2024-10-25T13:36:42Z\" level=info msg=\"Environment variable KUBERNETES_SERVICE_HOST not set\"\ntime=\"2024-10-25T13:36:42Z\" level=error msg=\"Failed to load eBPF tracer: failed to read kernel symbols: unable to open /proc/kallsyms: open /proc/kallsyms: no such file or directory\"\n"
parca-agent-1  | time="2024-10-25T13:36:42Z" level=error msg="================================================================"
parca-agent-1  | time="2024-10-25T13:36:42Z" level=error msg="about to report error to server"
parca-agent-1  | time="2024-10-25T13:36:43Z" level=warning msg="finished callprotocolgrpcgrpc.componentclientgrpc.serviceparca.telemetry.v1alpha1.TelemetryServicegrpc.methodReportPanicgrpc.method_typeunarygrpc.start_time2024-10-25T13:36:42Zgrpc.request.deadline2024-10-25T13:38:42Zgrpc.codeUnimplementedgrpc.errorrpc error: code = Unimplemented desc = unexpected HTTP status code received from server: 404 (Not Found); transport: received unexpected content-type \"text/plain; charset=utf-8\"grpc.time_ms152.119"
parca-agent-1  | time="2024-10-25T13:36:43Z" level=error msg="failed to call ReportPanic(): rpc error: code = Unimplemented desc = unexpected HTTP status code received from server: 404 (Not Found); transport: received unexpected content-type \"text/plain; charset=utf-8\""
parca-agent-1 exited with code 1

Can you say more about how this VM was installed/built and how you're running parca-agent? In particular are you running it as root? (It needs to be run as root.)
I just installed a fresh copy of Ubuntu 20.04 Desktop on a VirtualBox VM and can't repro, so there must be some weird difference between our setups.


📃: Amazon Clone frontend

:red_circle: Title :  Amazon clone frontend
:red_circle: Aim :  To showcase first project one can make using html css only
:red_circle: Tech Stack Used : HTML CSS
Screenshots 📷

:white_check_mark: To be Mentioned while taking the issue :

Full name : Shelly Garg
What is your participant role? (Mention the Open Source Program name. Eg. GSSOC, SSOC, JWOC, etc.)
GSSOC 24

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@Avdhesh-Varshney please assign this issue to me.

@Avdhesh-Varshney I believe this issue requires me to clone the amazon home page by using HTML and CSS. I can work on this and make the web page responsive as well.
Can I use JavaScript as well?
Can you assign this issue to me? I am willing to work on this. Thank you.


[WIP] Updated cgroups oci-spec-rs to 0.5.1

ref: #299
Apply builder pattern.
Getter
- let y = linux_xxx.y
+ let y = linux_xxx.y()

Builder
- let x = LinuxXXX { y: y };
+ let x = LinuxXXXBuilder::default().y(y).build().unwrap();

@utam0k
It seems that if you use a different version of oci-spec-rs, you can't build cargo.
Therefore, can I update youki's oci-spec-rs in this PR?
error[E0308]: mismatched types
   --> src/container/builder_impl.rs:161:28
    |
161 |                 resources: linux.resources.clone().unwrap(),
    |                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected struct `oci_spec::runtime::linux::LinuxResources`, found struct `LinuxResources`
    |
    = note: perhaps two different versions of crate `oci_spec` are being used?

@utam0k
It seems that if you use a different version of oci-spec-rs, you can't build cargo.
Therefore, can I update youki's oci-spec-rs in this PR?
error[E0308]: mismatched types
   --> src/container/builder_impl.rs:161:28
    |
161 |                 resources: linux.resources.clone().unwrap(),
    |                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected struct `oci_spec::runtime::linux::LinuxResources`, found struct `LinuxResources`
    |
    = note: perhaps two different versions of crate `oci_spec` are being used?

Oh... Can I ask you to implement it in this PR?

@utam0k I see. I'll try it.

Codecov Report

Merging #303 (234f490) into main (3bb01d2) will decrease coverage by 0.48%.
The diff coverage is 74.39%.

@@            Coverage Diff             @@
##             main     #303      +/-   ##
==========================================
- Coverage   70.92%   70.44%   -0.49%
==========================================
  Files          46       46
  Lines        6811     6685     -126
==========================================
- Hits         4831     4709     -122
+ Misses       1980     1976       -4


No permission for option requests

At the moment the admin has no permissions for option requests for motion and assignment but for customslide. The reason is the new permission system. For an option request self.action is None, so check_view_permission returns False when self.action is tested for specific strings.
It could be a bug of the django rest framework that self.action is None for an option request. But I did not find the documentation for self.action in a django rest framework viewset, maybe it is not part of the public API.
We could treat self.action == None as option request, but I am not sure if there are other cases where self.action is None.
As long as this is solved, the assignment page does not work because it needs the option request

Opened an issue here: https://github.com/tomchristie/django-rest-framework/issues/3115
At the moment we can solve this inspecting self.request.method.lower().

The fix is marked for Django RESt framework 3.1.4.

fixed by #1602


Swashbuckle.AspNetCore.SwaggerGen.TypeExtensions.FullNameSansTypeArguments(Type type) in TypeExtensions.cs, line 20

My abstract class has this function
void Delete(ICollection<KeyType> input);
There will be a mistake
I tried to modify the following code to solve my problem.
 internal static string FullNameSansTypeArguments(this Type type)
        {
                var fullName = type.FullName??type.ToString();
                var chopIndex = fullName.IndexOf("[[");
                return (chopIndex == -1) ? fullName : fullName.Substring(0, chopIndex);
        }

Are there any other better things to do, or my use is wrong?

Hi, I am facing a similar issue.
My application source code
https://github.com/aspnetlabs/Dynamic-WebAPI-Controllers
This is the stack trace:

Microsoft.AspNetCore.Server.Kestrel:Error: Connection id "0HLE3ADDEN75D": An unhandled exception was thrown by the application.
System.NullReferenceException: Object reference not set to an instance of an object.
at Swashbuckle.AspNetCore.SwaggerGen.TypeExtensions.FullNameSansTypeArguments(Type type)
at Swashbuckle.AspNetCore.SwaggerGen.XmlCommentsMemberNameHelper.BuildTypeToken(Type type)
at System.Linq.Enumerable.SelectArrayIterator2.MoveNext()    at System.String.Join(String separator, IEnumerable1 values)
at Swashbuckle.AspNetCore.SwaggerGen.XmlCommentsMemberNameHelper.GetMemberNameForMethod(MethodInfo method)
at Swashbuckle.AspNetCore.SwaggerGen.XmlCommentsOperationFilter.Apply(Operation operation, OperationFilterContext context)
at Swashbuckle.AspNetCore.SwaggerGen.SwaggerGenerator.CreateOperation(ApiDescription apiDescription, ISchemaRegistry schemaRegistry)
at Swashbuckle.AspNetCore.SwaggerGen.SwaggerGenerator.CreatePathItem(IEnumerable1 apiDescriptions, ISchemaRegistry schemaRegistry)    at System.Linq.Enumerable.ToDictionary[TSource,TKey,TElement](IEnumerable1 source, Func2 keySelector, Func2 elementSelector, IEqualityComparer`1 comparer)
at Swashbuckle.AspNetCore.SwaggerGen.SwaggerGenerator.GetSwagger(String documentName, String host, String basePath, String[] schemes)
at Swashbuckle.AspNetCore.Swagger.SwaggerMiddleware.d__6.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.AspNetCore.Server.IISIntegration.IISMiddleware.d__8.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.AspNetCore.Hosting.Internal.RequestServicesContainerMiddleware.d__3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at

Microsoft.AspNetCore.Server.Kestrel.Internal.Http.Frame`1.d__2.MoveNext()

Just faced the same issue on a generic class, any news?

Also, see the link below - I've come up with a test to repro but have it commented out because I haven't had the time to find a solution. Those XML comments member names are extremely cryptic, particularly when generics come into play and I haven't had the time to give it the focus it needs:
https://github.com/domaindrivendev/Swashbuckle.AspNetCore/blob/master/test/Swashbuckle.AspNetCore.SwaggerGen.Test/Annotations/XmlCommentsMemberNameHelperTests.cs#L29

If anyone else wants to pull it down and take a stab that would be awesome?

@cnlizhipeng, @aspnetlabs, @PatrickTetz I believe I've implemented a fix for this, pushed with the following commit - 0b6e0b2
It should be available with the following preview package on myget.org:
Swashbuckle.AspNetCore.2.5.0-preview-0510.nupkg
Could you please pull it down, test it out and provide feedback? Thanks


How do I pass variables?

I'm trying to use mustache-loader with html-webpack-plugin.
How can I pass the variables to use to render the template?  There's a reference to arguments at https://github.com/deepsweet/mustache-loader/blob/fef82c2c307e1a66c75c05827c93a99a65a99bfb/index.js#L44 but I don't see how to actually pass those arguments in.

Markwoon, are you just looking for how to supply data to your templates?
In your code after you require a template then you can just call the returning function passing in the data.

As mentioned, I'm using mustache-loader to send the result directly to html-webpack-plugin.
There is no "my code".  I'd like a way to return the rendered result directly.  Is this possible?

Thank you for clarifying. That is not something that this loader currently supports. Interesting enough doing something similar is what I was originally looking for when I found this loader. I would accept a PR for that kind of functionality.

Done.

Thank you very much for the PR. I've merged it in, and I'll do a release.


Write back tagged faces to photo files

Hey, I am still impressed by this piece of of software :-)
Though, even with using LibrePhotos I would still no see an easy way of having faces tagged in pictures over software bounderies, i.e. on mobile and on PC. The interface for tagging faces and of course the supporting algorithm of derived faces would be a perfect base to just write them back to the file, which then could be used on my mobile app again (synched by Syncthing)

Was wondering if I should add another issue for this, but decided to piggyback off this; it'd also be fantastic of LibrePhotos could read existing face tags in images. I have a collection of about 100k photos with probably about 20k tagged faces, and it'd be fantastic if this could be transferred over to LibrePhotos.

Is there a general format for saving faces to images? I could support reading from the future LibrePhotos format of saving faces.

If I'm understanding it correctly Image Region Metadata is part of both the XMP and IPTC spec (https://web.archive.org/web/20171214222138/http://www.metadataworkinggroup.com/pdf/mwg_guidance.pdf, section 5.9 and https://iptc.org/news/announcing-the-iptc-photo-metadata-standard-2019-1-introducing-iptc-image-regions/ respectively), one application of which is saving faces to images.

I'm adding to this request.  There are so many tools that have some form of face detection now.  I've found that librephotos face detection is one of the best!  BUT, I have an existing gallery of 28k photos that have (mostly) all been tagged through picasa/digikam and the faces have been written into the exif metadata of the pictures.  Not being able to leverage that existing data is a complete show stopper for me.  I can't justify dedicating the time to go back through and re-create all that data that already exists in my photos (even though the face tagging process works great in librephotos).  I've settled on using pigallery2 for now as it reads all my existing tags and leverages the existing data (it doesn't have any face tagging ability on its own).
Secondarily I decided a while back that writing the metadata to the images (or a sidecar file) is a hard requirement for me to use any sort of metadata creation software (face tagging/GPS tagging/keyword creation).  Having been previously a user of what is now called "Gallery the Revival" all the way back to gallery2 and having moved through multiple gallery softwares over time, I've come to understand that gallery softwares come and go, but the pictures stay forever.  Therefore given the time investment I've put into creating that metadata I have a hard requirement for the software to leverage that metadata (and update it if I'm going to use the tool to generate new metadata).
I would encourage the librephotos project to focus on leveraging (and storing) metadata in the image (or in a sidecar).  So many tools today are trying to build new ways of generating metadata but don't focus on the long term curation/leverage of the existing data.
Thanks for the excellent work!

https://github.com/python-xmp-toolkit/python-xmp-toolkit

I see that the ability to write ratings back to photos has recently been added.  Will some of that similar workflow eventually be able to read and write faces to image metadata?

yep I agree with all of the above, any software that does not leverage already existing data is a show stopper for me. I have already most of my data tagged in one way or another and I cannot use that it's bit pointless doing the same job again.
This project looks very promising. Well done.


Account plugin: disable sync for certain plugins

Is your feature request related to a problem? Please describe.
I have a problem on my laptop where the GPU plugin crashes the client. This is an issue with the laptop, not the plugin. However, this also means I can't sync my account onto that laptop, as this causes the GPU plugin to be re-enabled every time, since I use it on other computers.
Describe the solution you'd like
I'd like it if I could disable certain plugins from being sync'ed by the account plugin.

+1


[Performance] Strategy de ban words

For now we only use caching and it seems to have fixed the performance issues.


AttributeError: type object 'ParallelPolicy' has no attribute 'SUCCESS_ON_ONE'

When I run scenario runner on ubuntu 16.04 with python3 I get this or similar error with all scenarios:
Traceback (most recent call last):
File "scenario_runner.py", line 160, in
main(ARGUMENTS)
File "scenario_runner.py", line 97, in main
scenario = scenario_class(world, args.debug)
File "/usr/local/scenario_runner/Scenarios/object_crash_vehicle.py", line 152, in init
debug_mode=debug_mode)
File "/usr/local/scenario_runner/Scenarios/basic_scenario.py", line 79, in init
behavior = self._create_behavior()
File "/usr/local/scenario_runner/Scenarios/object_crash_vehicle.py", line 191, in _create_behavior
policy=py_trees.common.ParallelPolicy.SUCCESS_ON_ONE)
AttributeError: type object 'ParallelPolicy' has no attribute 'SUCCESS_ON_ONE'

@rguptatx  hello,Have you created a successful scenario by referring to the example of the official scenario creation?(https://carla-scenariorunner.readthedocs.io/en/latest/creating_new_scenario.html)


Feature, Shiny, Add File Builder

Similar to BCGcalc Shiny app add File Builder features.
https://github.com/leppott/BCGcalc
File Builder

[ ] Match taxa list with official taxa list (taxa translator)
[ ] Match COMID to get inputs (predictors) for MMI model calculator
[x] Ability to merge two files

Split in to 3 issues.


BodyOutputType.Component not working

See the error console message here: http://plnkr.co/edit/picmkmZhsnVxg7hicc2q?p=info

Your component is not wrapped or included in an NgModule.  Angular no longer allows you to render orphaned components dynamically.  You need to wrap your component in a module as demonstrated by the tests: https://github.com/Stabzs/Angular2-Toaster/blob/master/src/toaster-container.component.spec.ts#L41

I've updated your plunker with a working example: http://plnkr.co/edit/xn4sZkrf6dCeHwLHMlCV?p=preview


⚠ Bofrost.fr has degraded performance

In 16d63f4, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1322 ms

Resolved: Bofrost.fr performance has improved in c32484f.