Unexpected MenuItem visibility when setting display: flex on MenuContent

Description
Hello Ark UI team,
I am using your headless UI library and I have encountered an issue regarding the visibility of MenuItems when changing the display style of MenuContent.
Here's my original working code:
<template>
  <Menu>
    <MenuTrigger>
      <button>Open menu</button>
    </MenuTrigger>
    <Teleport to="body">
      <MenuPositioner>
        <MenuContent>
          <MenuItem id="edit">Edit</MenuItem>
          <MenuItem id="delete">Delete</MenuItem>
          <MenuItem id="export">Export</MenuItem>
          <MenuItem id="duplicate">Duplicate</MenuItem>
        </MenuContent>
      </MenuPositioner>
    </Teleport>
  </Menu>
</template>

And the related CSS:
[data-scope='menu'][data-part='content'] {
  margin: 0;
  width: 160px;
  background-color: white;
  border-radius: 0.25rem;
  padding: 0.25rem;
  border: 1px solid lightgray;
  list-style-type: none;
}

With the above setup, the menu behaves as expected: the MenuItems are hidden until the Open menu button is clicked.
However, when I add display: flex; to the MenuContent style:
[data-scope='menu'][data-part='content'] {
  ...
  display: flex;
  ...
}

The MenuItems are always visible, regardless of whether the Open menu button has been clicked or not. I wanted to use display: flex to create a column layout for my MenuItems, but I am unsure why this affects the visibility of the MenuItems. I didn't find any related information in the documentation.
Could you kindly explain why this is happening, and suggest any potential solutions if I would like to keep the column layout?
Thank you for your assistance.
Link to Reproduction
none
Steps to reproduce
Here's my original working code:
<template>
  <Menu>
    <MenuTrigger>
      <button>Open menu</button>
    </MenuTrigger>
    <Teleport to="body">
      <MenuPositioner>
        <MenuContent>
          <MenuItem id="edit">Edit</MenuItem>
          <MenuItem id="delete">Delete</MenuItem>
          <MenuItem id="export">Export</MenuItem>
          <MenuItem id="duplicate">Duplicate</MenuItem>
        </MenuContent>
      </MenuPositioner>
    </Teleport>
  </Menu>
</template>

And the related CSS:
[data-scope='menu'][data-part='content'] {
  margin: 0;
  width: 160px;
  background-color: white;
  border-radius: 0.25rem;
  padding: 0.25rem;
  border: 1px solid lightgray;
  list-style-type: none;
}

With the above setup, the menu behaves as expected: the MenuItems are hidden until the Open menu button is clicked.
However, when I add display: flex; to the MenuContent style:
[data-scope='menu'][data-part='content'] {
  ...
  display: flex;
  ...
}

The MenuItems are always visible, regardless of whether the Open menu button has been clicked or not. I wanted to use display: flex to create a column layout for my MenuItems, but I am unsure why this affects the visibility of the MenuItems. I didn't find any related information in the documentation.
Ark UI Version
<EMAIL_ADDRESS>Framework

[ ] React
[ ] Solid
[X] Vue

Browser
No response
Additional Information
No response

@dwzkit Thanks for taking your time to create this issue.
The MenuPositioner as the name indicates is responsible for positioning the content of the menu.
The MenuContent holds the - well - content of the menu. If the menu is closed the content is hidden -> display: none
If you now force display: flex on the menu content - things will fall apart as you described in your issue.
Solultions:
a) Wrap your content:
<MenuPositioner>
  <MenuContent>

 ...

  </MenuContent>
</MenuPositoner>

b) Use CSS:
<MenuPositioner>
  <MenuContent>
     ...
  </MenuContent>
</MenuPositoner>

[data-scope='menu'][data-part='content']:not([hidden]) {
  display: flex;
}

Hope that helps.


Test lakefsfs with mock server

Translate almost all tests in LakeFSFileSystemTest to use a Java MockClient.  The result is called LakeFSFileSystemServerTest.
The existing Mockito-based code is horribly brittle; see the underlying issue #6554 for  why this manages to block our move to a forwards-compatible Java SDK.  So this is blocking #6424 .
Fixes #6554 for the most part.  Some tests are still missing, but I do not want to block the Java client review on fixing them.  I will open a new issue to uncomment these tests, and copy over any tests from LakeFSFileSystemTest that are still missing.

Tests that are no longer there:
testRename_existingDirToExistingNonEmptyDirName
testRename_fallbackStageAPI
testRename_nonExistingSrcFile
testRename_srcAndDstOnDifferentBranch
testRename_srcEqualsDst

Are we cool with that?

THANKS!
Sorry for the mega-review
If it's any consolation, #6656 is even worse!
Pulling once tests pass.


make user on synology nas

I'm installing the docker container on synology. There is no make on synology. How can i make the first user?

just copy docker commands from makefile to CLI


Chore: Use folder path in current context

Changelog Description
Use folder_path instead of asset_name in current context and context data.
Additional info
This is minimalistic PR changing just the keys used when data for context are read or defined.
Breaks backwards compatibility on current context data and launch hooks. Launch hooks don't have access to asset_name in data.
Testing notes:

Application can launch on a context.
Change of context with workfiles too triggers correct callbacks without issues.

I faced two things:

Publisher tab looks like this:

Sometimes context change works and sometimes it fails.

Sometimes context change works and sometimes it fails.

Any traceback? Is the fail related to how the context is changed?

For anyone reading this PR in future. The issues with publisher did not happen again, probably was caused by lost connection to server, and the context change issue in houdini is not related to this PR.


Sheet locks scroll permanently if not closed

To Reproduce

Go to https://design.swedbankpay.com/v/4.5.0/docs/getting-started/contributing (e.g.)
Scroll down and back up (scroll works)
Select Components > Sheet in sidebar
Click Open sheet
Click back in browser
Contributing page (or any other page) is now unscrollable

I believe there is another unwanted issue here too:
If you do the steps to reproduce and then open a sheet: the body does not get the .has-vscroll class.
Now if you close the sheet and open it again: the body get .has-vscroll (in addition to sheet-open)
I suppose this happens because opening a sheet toggles class has-vscroll, it should rather set it explicitly and remove it explicitly on close

Some more insight:
body.sheet-open {     overflow: hidden; }
overflow: hidden actually prevents scroll event to trigger, I thought maybe ...
window.addEventListener('scroll', function(){ console.log("scroll"); }, false);
...could test if a sheet is actually open, but it doesn't work. The event isn't triggered because of overflow: hidden on body.
The reason for this logic is to prevent scrolling of background body when a sheet is open, and that it should do. Doing that without overflow rule sounds hacky.
An option would be to try to remove body.has-vscroll and body.sheet-open on back-/forward button events, but I didnt like what you have to do to get that working in the examples I have found

Could a hashchange event listener suffice?

I can probably look into this throughout the day unless you have time, @pthoang ?

@VikingTristan would be nice if you have time. If not, then I will look into it tomorrow

Could a hashchange event listener suffice?

No, there isn't necessarily a # in the url, and it looks like hashchange only fires if you set location.hash = "123" in js.


Vector Store: Comparing float_vector column creation with other implementations

Problem Statement
About
While working on integrating LangChain with CrateDB, we discovered a little anomaly compared to the corresponding pgvector implementation.
pgvector
pgvector accepts creating a table like that. Here, the first inserted vector will actually define the vector size.
create table testdrive (embedding vector);

CrateDB
With CrateDB, when creating a table without specifying the vector size, it will be implicitly defined with a size of 1.
create table testdrive (embedding float_vector);

LangChain
In LangChain, the pgvector adapter model looks like this:
embedding: Vector = sqlalchemy.Column(Vector(None))

-- https://github.com/langchain-ai/langchain/blob/52bf62354ec5cce030a74a68970df1a0f0b17962/libs/langchain/langchain/vectorstores/_pgvector_data_models.py#L66
In CrateDB, it is difficult to implement, because hardcoding the value here is not appropriate, because the user may want to work with models of different dimensions.
embedding = sqlalchemy.Column(FloatVector(1024))

-- https://github.com/langchain-ai/langchain/blob/52bf62354ec5cce030a74a68970df1a0f0b17962/libs/langchain/langchain/vectorstores/cratedb/model.py#L68-L72
Possible Solutions
No response
Considered Alternatives
No response

Walkthroughs
pgvector
docker run --rm -it --publish=5432:5432 \
  --env "POSTGRES_HOST_AUTH_METHOD=trust" \
  ankane/pgvector postgres -c log_statement=all

create extension vector;
create table testdrive (embedding vector);
insert into testdrive (embedding) values ('[1, 2, 3]');
insert into testdrive (embedding) values ('[1, 2, 3, 4]');
insert into testdrive (embedding) values ('[1, 2, 3, 4, 5]');

CrateDB
docker run --rm -it --name=cratedb --publish=4200:4200 --publish=5432:5432 \
    --env=CRATE_HEAP_SIZE=4g crate/crate:nightly \
    -Cdiscovery.type=single-node

create table testdrive (embedding float_vector);
insert into testdrive (embedding) values ([1]);

insert into testdrive (embedding) values ([1, 2, 3]);
SQLParseException[The number of vector dimensions does not match the field type]

Thanks for the inputs, but the fixed length is a requirement of the used Lucene type.

Thanks. I will investigate how the corresponding Elasticsearch and OpenSearch adapters work in this regard, and report back.

Thanks. I will investigate how the corresponding Elasticsearch and OpenSearch adapters work in this regard, and report back.

https://www.elastic.co/guide/en/elasticsearch/reference/current/dense-vector.html#dense-vector-params
dims is mandatory param


Updating title from +layout.svelte fails after 404

Describe the bug
If you have a dynamic title in your +layout.svelte file. Where you might have something like
<svelte:head>
	<title>{$page.data.title || "Default"}
</svelte:head>

where title is returned from the page load function.
After 404'ing unless you navigate to a page that explicitly sets it title via it's load function or svelte:head. The title will still stay as 404 when navigating to subsequent pages that do not return title. So it seems like || "Default"} never gets hit after 404'ing. It works as expected before a 404.
Reproduction
https://www.sveltelab.dev/g9belf05056nzaf
At / you see that the title is Repro - Home when you navigate to en-CA or en-CA/category/1 you see that the title is Default however from the route en-CA when you click on invalid Category we get the expected 404 but when clicking back on category 1 from the 404 page. We do not get the Default title. If you navigate back with arrow keys in browser and then click home which has an explicit title it updates title back to Repro - Home
Logs
No response
System Info
System:
    OS: Linux 5.15 Ubuntu 22.04.3 LTS 22.04.3 LTS (Jammy Jellyfish)
    CPU: (12) x64 Intel(R) Core(TM) i7-10750H CPU @ 2.60GHz
    Memory: 2.44 GB / 15.50 GB
    Container: Yes
    Shell: 3.6.0 - /usr/bin/fish
  Binaries:
    Node: 20.6.1 - /usr/local/bin/node
    Yarn: 1.22.19 - /home/linuxbrew/.linuxbrew/bin/yarn
    npm: 10.1.0 - /usr/local/bin/npm
    pnpm: 8.7.6 - ~/.local/share/pnpm/pnpm
  Browsers:
    Chrome: 117.0.5938.132
  npmPackages:
    @sveltejs/adapter-node: ^1.3.1 => 1.3.1
    @sveltejs/kit: ^1.25.1 => 1.25.1
    svelte: ^4.2.1 => 4.2.1
    vite: ^4.4.9 => 4.4.9

Severity
annoyance
Additional Information
To my knowledge this has SEO implications for pages that are valid but have a 404 title

This is more of an issue with Svelte than it is SvelteKit. In the 404 page, you've replaced the <title> element from the layout. It will remain as 404 until the layout <title> element is updated. When navigating from /404 to /category/1 there is no update because the evaluation inside the layout <title> element is "Default" for both pages. However, navigating to / updates the <title> element  because the value changes from "Default" to $page.data.title.
Closing this issue as it cannot be fixed from SvelteKit's side.

@s3812497 Ah okay it just clicked for me to use the $page.error object to check to see if there any errors and if so display the title as $page.status otherwise $page.data.title or "Default" if that is undefined. Is that the recommended approach so we don't have to manipulate title in +error.svelte ?


[FIX] flexChangesBundler: Merge existing with new flexibility-bundle.json

Merge any existing level zero changes into the flexibility-bundle.json
of UI5 adaptation projects
Cherry-pick of https://github.com/SAP/ui5-builder/pull/974

coverage: 95.171% (+0.04%) from 95.128%
when pulling 7212f26b32808bac10b17e40eb0465d2ef9e6be1 on cherry-pick-merge-flex-bundle
into 34cb087c1a81d676ef8556ce836d3516ca786e7d on v2.


chore: update Deno logo to the latest version

Since Deno 2 released

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

Oh, my bad; I did not see this PR. A PR landed earlier today that does this, so apologies about that. I will close this PR


Typing errors on main.go

Some typing errors on the main.go file

Codecov Report

Merging #19 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #19   +/-   ##
=======================================
  Coverage   97.16%   97.16%
=======================================
  Files           5        5
  Lines         141      141
=======================================
  Hits          137      137
  Misses          2        2
  Partials        2        2

Continue to review full report at Codecov.

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

Thanks!!


Adding Extra lib execution to fast for key naming convention

monaco-editor version: 0.9.0
Browser: Google Chrome Version 61.0.3154.0 (Official Build) canary (64-bit)
OS: Windows 10 Enterprise
Steps assume monaco editor is already installed and createEditor has already been called.
Steps or JS usage snippet reproducing the issue:
Load libraries statically:
// this._typescriptExtraLibs is used to dispose later
this._typescriptExtraLibs = monaco.languages.typescript.typescriptDefaults.addExtraLib(person.toString());
this._typescriptExtraLibs = monaco.languages.typescript.typescriptDefaults.addExtraLib(animal.toString());
this._typescriptExtraLibs = monaco.languages.typescript.typescriptDefaults.addExtraLib(flee.toString());
this._typescriptExtraLibs = monaco.languages.typescript.typescriptDefaults.addExtraLib(color.toString());

Receive error at:
line 39, column 1220 (minified script)
Error:
Uncaught Error: ts:extralib-1499848020086 already a extra lib
My conclusion:
I stepped through the code and notice that due to the way that the keys in the this._extraLibs are generated using the date it has conflicts due to "too fast" processing.

Yes you get the same value from Date.now() as it's too fast ... Maybe we can add another Math.random() to further reduce the possibility of collision

Yes that would relieve this issue, but since that check is executed on the creation and there would be a random value involved i think that check would be redundant, if you wanted to check that the item is not replicated would it not be best to hash the lib being added then use that as the key instead, then the duplicate check would be more applicable accurate.

@jrieken Is there any progess on this? as currently I have to have some ugly hacks in place to provide properly.
Thanks in advance.

Is there an update on this? I'm getting this issue as well

@jrieken I'm also running into this issue. Is there a fix planned?
@JohannWNel What hacks are you using to make it work?

Hi @Klustre
So I have noticed in their code that when you specify the file name when using that function then they use that as the key of that lib, thus there are no confilcts then, so I used a GUID as file names and works great for me.
I hope this helps.

@JohannWNel That indeed solved it. Thanks a lot!

Closing as this will not be fixed.


Upload WARC support

Users should be able to upload any WARCs to Webrecorder. This will work as follows:

If the WARC is a full collection, downloaded from Webrecorder, the entire collection will be recreated (description + recordings)
If the WARC is a single recording, downloaded from Webrecorder, the recording will be recreated (bookmarks list)
For any other WARC, a new recording will be created for that WARC. Users can then rename the recording or move to any collection.

This will use the new metadata structure specified in #204

Done! Users can upload into a specific collection or allow Webrecorder to create new collections
Temp users are not allowed to upload for now, though can enable or make it an option later if needed.


CLOUDSTACK-10247: L2 network not shared on projects

When trying to deploy a vm providing a project id and a L2 network id, this error is logged:
2018-01-18 09:04:31,749 INFO  [c.c.a.ApiServer] (qtp1310540333-17:ctx-7ff91c56 ctx-8c37c46b ctx-d58064ad) (logid:74e86028) PermissionDenied: Unable to use network with id= 5bee486a-ff20-4db2-b62e-4b4f3485cfff, permission denied on objs: []

LGTM, merging based on reviews and test results.


Enhance MySQL client implementation

Description:
Currently, generic JDBC client code is used for MySQL client. Along with the refactoring done to provide a richer feature set focusing on vendor specific areas, need to enhance the MySQL client.

Refer the comment https://github.com/ballerina-platform/ballerina-lang/issues/13818


[lints] table reason

Problem
I was playing around with the new rust 1.81 lint reason feature and I noticed that the reason cannot be set in the Cargo.toml [lints] section like it can in rust code:
#![warn(clippy::shadow_unrelated, reason = "I'm afraid of shadows")]
fn main() {
    let x = 1 + 2;
    println!("{x}");

    let x = "foo bar";
    println!("{x}");
}

Proposed Solution
I would expect that I would be able to set the lint reason in the Cargo.toml like so:
[package]
name = "lint-reason-check"
version = "0.1.0"
edition = "2021"

[dependencies]

[lints.clippy]
shadow_unrelated = { level = "warn", reason = "I'm afraid of shadows" }

I'm unsure what behavior makes sense if the reason is set in both the Cargo.toml file and the rust source file. I'm also not sure if this should be a cargo issue, and rust issue, or a clippy issue.
Notes
No response

Cargo is the appropriate place for this.
The way we currently pass [lints] to rustc, clippy, and rustdoc is insufficient to handle this.  That is also true for #12534 and #13565.  We need a new way to pass this information to rustc on the command line.

Speaking of rustc's command-line, there isn't a flag for expect.  I wonder if that should also be exposed within this new CLI.


[ceph-dashboard] Make standby behaviour and status code configurable

Allow to configure the ceph dashboard's options standby_behaviour ([1]) and standby_error_status_code ([2]).
While keeping the current default to redirect, this allows to adapt the behaviour to be more compatible with deployment behind a loadbalancer, where a redirect to a possibly internal URL is less useful than an error.
Especially in the case of a deployment behind haproxy one may set
ceph_dashboard_standby_behaviour: error
ceph_dashboard_standby_error_status_code: 404

and configure haproxy to
http-check expect status 200,404
http-check disable-on-404

where the former will set the backend "UP" on status codes 200 and 404, while latter will immediately transition the server to the "NOLB" status, thus not marking passive instances as down, while they are actually functioning correctly.
[1]
https://docs.ceph.com/en/quincy/mgr/dashboard/#disable-the-redirection
[2]
https://docs.ceph.com/en/quincy/mgr/dashboard/#configure-the-error-status-code

Part of https://github.com/osism/issues/issues/1013


Building wheel from source and publishing source on pypi

Python version
3.11
Problem description
We try to build wheels from source (and not use provided wheels).
pip uninstall redshift-connector -y && pip install --no-binary :all: redshift-connector==2.1.3  --index-url https://pypi.python.org/simple
ERROR: Could not find a version that satisfies the requirement redshift-connector==2.1.3 (from versions: none)
ERROR: No matching distribution found for redshift-connector==2.1.3

First problem is that https://pypi.org/project/redshift-connector/#files does not contain source distribution.
When trying to build it locally (the source distribution from git:
python -m build --sdist
pip wheel dist/redshift_connector-2.1.3.tar.gz

we get:
(.venv) ➜  amazon-redshift-python-driver git:(master) ✗ pip wheel dist/redshift_connector-2.1.3.tar.gz
Processing ./dist/redshift_connector-2.1.3.tar.gz
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [7 lines of output]
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "/private/var/folders/sj/1swnwb8n6tb_3cnl8zq6wgjh0000gn/T/pip-req-build-jhbghmqn/setup.py", line 107, in <module>
          install_requires=open("requirements.txt").read().strip().split("\n"),
                           ^^^^^^^^^^^^^^^^^^^^^^^^
      FileNotFoundError: [Errno 2] No such file or directory: 'requirements.txt'
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

Adding to MANIFEST.in:
include requirements.txt

Solve the problem of building from source tar.gz.
Reproduction code
git clone https://github.com/aws/amazon-redshift-python-driver.git
cd amazon-redshift-python-driver
python3.11 -m venv .venv
source .venv/bin/activate
pip install wheel build
python -m build --sdist
pip wheel dist/redshift_connector-2.1.3.tar.gz

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Hi @matejsp , thank you for raising this issue. My apologies for the delay in response. I will discuss this request with the team and provide an update here when available.


Embed built files

The project must be runnable without having to compile the assets:

It needs Node and NPM that are no always easy to install depending architecture and rights
Installing webpack download half the internet

Closed with #118


Circular element references inside wsdl - assistance needed

Hello.
This fix https://github.com/vpulim/node-soap/pull/1101 fixed one bug and introduced another. Now parsing wsdl with recursive ref elements will cause stack overflow. I created 2 test cases which demonstrate this bug, but I need advice of someone who knows this project better than me.
I can quick fix stack overflow with one kludge - just like https://github.com/vpulim/node-soap/pull/1101, something like this, or maybe a little more complex - because second test will fail with this:
          if (!this.$ref) {
            definitions.descriptions.types[typeName] = elem;
          } else {
            const refType = splitQName((typeElement as ElementElement).$type);
            if(refType.name != typeName)
              definitions.descriptions.types[typeName] = elem;
          }

As pedantic person, I think kludge is not the way to do. In my opinion main problem is more deep in internals: both element names (with references) and type names go to the same object definitions.descriptions.types, they are mixed, and this introduces all these bugs with elements and types, but I'm not sure in this, because I'm new to node-soap internals, I just use it as a regular user.
So I need advice from someone, I can try to fix this by splitting definitions.descriptions.types for elements and for types, but it seems this is core internal feature, I'm afraid to break something and waste my time making PR which will not be accepted.

Coverage remained the same at 95.019% when pulling 9bf98d1c2ae2b75b2aec5e626c124fd8784f496e on doubleaxe:recursive_wsdl_ref_bug into bf3b07d7c81f9e159d239f414c2e347d71d01695 on vpulim:master.

I updated PR, this is my vision on how this bug should be fixed. All tests seems passing, I only doubt in backward compatibility.


Add an gender column on user registration

In the users model add an extra column for gender
define an enum with male and female

Why do you create issues on your own repo, and then assign them to yourself? Also, if you're going to solve them yourself just remove the good first issue tag, it may be confusing to new open source peeps that search for issues to solve :)

Hey @Frenzoid thanks for mention it. Yes you are right good first issue is confusion for new comers. I'll remove them from the issues.
I am just exploring the issues in github.

Awesome, thanks mate :)


Frigate v0.12

It's simulating the merge and running on that, so running on the branch isn't the same. I think you can checkout exactly what happens in actions with:
git checkout --progress --force refs/remotes/pull/458/merge


Add the example code of how to accelerate the data sharing with vineyard on fluid platform.

What do these changes do?
As titled.

@dashanji  Remember to take a look at the comment of codacy: https://github.com/v6d-io/v6d/pull/1776/files and remove these trailing whitespaces the next time.


NumberOfLines of text has bug in ios

NumberOfLines Bug:
1、Show not accurate in IOS
2、 Truncation not accurate in Android

https://github.com/facebook/react-native/pull/24975 probably it is related.

Truncation not accurate in Android

Truncation not accurate in Android

@chenhuaxian Hi, please separate issues of iOS and Android, and fill in the template issue. After that, we can try to investigate.


Full-Text Search highlight matched keyword results from query

Problem

Right now there's no way to highlight the keywords you matched on. For example, querying "great" matches "greatest" in the corpus, but how do you highlight "greatest" in the result set?
const results = await prisma.book.findMany({
  where: {
    content: {
      search: query
    }
  },
  select: {
    id: true,
    title: true,
    content: true
  }
})

// Trim content and highlight query. Use word boundaries (\b) and capture
// the query
//
// TODO: For the query "great", how do we highlight "greatest", which is matched by "great"?
const highlighter = new RegExp(`\\b(${query})\\b`, 'g')
for (let result of results) {
  const i = result.content.indexOf(query)
  result.content = result.content.slice(i - 100, i + 100)
  result.content = result.content.replace(
    highlighter,
    '<strong>$1</strong>'
  )
}

Suggested solution

The full-text search API should expose the matches, so you can highlight them as results for your users.

PostgreSQL provides the ts_headline function to highlight fragments of the document.
PostgreSQL Docs - Highlighting results

Hello everyone 👋
We are starting design work to improve Prisma's support for Full Text Search (FTS)!
If you would like to help, please help us learn about your needs via a short FTS user research survey.
Thank you!


Replace Windows special characters with empty string

Some Windows users report syncing not working properly.
> str = "\asdf|*\"\\/<>:?|asdf"
'asdf|*"\\/<>:?|asdf'
> str.replace(/[\\/<>?|:*"]/gm, '')
'asdfasdf'

plugin:instapaper:725 [instapaper] sync failure: Error: File name cannot contain any of the following characters: * " \ / < > : | ?
    at Lb (app.js:1:726521)
    at t.checkPath (app.js:1:733744)
    at t.<anonymous> (app.js:1:735629)
    at app.js:1:237056
    at Object.next (app.js:1:237161)
    at app.js:1:236077
    at new Promise (<anonymous>)
    at v (app.js:1:235822)
    at t.create (app.js:1:735517)
    at fileForArticle (plugin:instapaper:644:24)

Potentially a fix for #48


[ArPow] Remove patches related to sourcelink and aspnetcore consumption

As part of the PR to add offline CI, a sourcelink patch was needed as well as an aspnetcore patch which worked around the issue.  The sourcelink patch should be removed once the fix is picked up be sourcebuild.  The aspnetcore patch should be integrated into the aspnetcore repo and the patch should be removed from source build.

Closing in lieu of https://github.com/dotnet/source-build/issues/2708.


[Feature]  i want to add french

General

[X] My feature does not exist in the latest version of MySpeed.
[X] I have checked that my feature has not been suggested by anyone else.

Your Idea
I wanted to add french to the project.i'm french and i discovered your project.

Thank you! I've added French to our Crowdin page. I don't speak French myself, so I can't really help with translating, but if you need any help with the Crowdin interface, feel free to ask.
Again, thanks for the effort :)


Copy files instead of linking and add binary file support

I like the idea of this package, so I improved it a bit in this PR:
In the current version only the main file is linked. In order to test the package as realistic as possible, it makes sense to have the files copied into node_modules that would be published with npm publish. npm-packlist does this. The rest is copying the files async.
The other improvement is binary files. Now binaries declared in the bin section of package.json are linked into node_modules/.bin.

Interesting stuff, thanks I'll take a look at this tomorrow :)

Looks great, I'll get something published tonight :)

Great thanks man!

@jamesadarich when are you planning to release it on npm?


fontfor: 0.4.1 -> 0.4.2

This update fixes a build failure with rustc >= 1.75 that caused Hydra build failures on unstable.
See: https://github.com/7sDream/fontfor/pull/69
Failing Hydra job: https://hydra.nixos.org/build/245712787
Hydra log: https://hydra.nixos.org/build/245712787/nixlog/5
Description of changes
Updated package to version 0.4.2

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

Apologies that this was missed earlier; closing as it was superseded by #310691.


ER diagram syntax highlighting

Cutting and pasting the ER diagram example from mermaid-js.github.io/mermaid-live-editor results in syntax errors being reported.  The live preview diagram is generated correctly however.
The syntax errors appear as follows:
Parse error on line 1:

^
Expecting 'NEWLINE', 'SPACE', 'GRAPH', got 'EOF'

============================

I have the same issue with sequenceDiagram on an mmd file that used to work before

Thank you for reporting. I think parsing error should be resolved in v0.9.0. If you still faced to the symptom, please let me know. I'm still working on syntax highlight problem.

It works fine now - however, mermaid configuration doesn't take effect from the mermaid-editor.preview.defaultMermaidConfig

It works fine now - however, mermaid configuration doesn't take effect from the mermaid-editor.preview.defaultMermaidConfig

Thank you for your reporting. It could't be reproduced at my local I'm sorry to say. If you have default config file with default_config.json at workspace top for example, it'll be resolved like "mermaid-editor.preview.defaultMermaidConfig": "./default_config.json". It worked fine for me.
I'm really appreciate if I could get more detail of your situation at your convenience.

@tomoyukim  thanks for responding - I have an absolute path to the config file - something like
    "mermaid-editor.preview.defaultMermaidConfig": "/Users/naseer/dotfiles/mermaid-config.json",

I can file a separate ticket for this if needed

I can file a separate ticket for this if needed
@naseer Yes, thank you. Please put the issue again as new one if you still face to the problem around default mermaid config. I'm sorry for my late reply.


addTransactions arguments

Hi, thank you for this.
I'd appreciate if you can add the arguments to addTransactions as documented such as runTransfers and learnCategories.
Thank you.

Hey Jim, sure thing

Thank you. :)


Behavior velocity planner delete param

Description
Removed default values defined in declare_parameter function.
The following parameters need to be added
stop_line_extend_length: 5.0
detection_area.hold_stop_margin_distance: 2.0
intersection.walkway.external_input_timeout: 1.0
intersection.merge_from_private.merge_from_private_area.stop_duration_sec: 1.0
stop_line.hold_stop_margin_distance: 2.0
stop_line.use_initialization_stop_line_state: false
stop_line.debug.show_stopline_collision_check: false
virtual_traffic_light.hold_stop_margin_distance: 0.0
Behavior velocity planner delete param.webm
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@yamazakiTasuku Additionally, please fix conflicts :pray:


Simple e2e test. Header text checks

Test checks header text from all pages
also on all pages verified appearing/disappearing of the left panel with differnt window sizes
Problem on page CAROUSEL, test failed with tag "Time out",
but if we use option ignore synchonization = true - test is passed

@SergeyKuryatnick Can you fix those lint errors?: https://travis-ci.org/valor-software/ng2-bootstrap/builds/144451979#L2092-L2100

@Dinistro Done


ngx_pagespeed_beacon ERR_BLOCKED_BY_CLIENT

I have included location ~ "^/ngx_pagespeed_beacon$ps_dollar" { } in my server { } block but get this error.
ngx_pagespeed_beacon ERR_BLOCKED_BY_CLIENT

Do you use any extensions in Chrome that may block requests?


test(gsr): Fix translate query test

Just a simple fix. old query did not return a result with translationPronunciation so tests were failing.

Tests are still failing due to https://github.com/typicalninja/google-sr/pull/29#issuecomment-2334394238


Bump the copyright year

Should we?

sure

I think we only should if we're making changes to the code.


fix: DNS TTL not respected

Description
Fixes #1425
Testing
Testing DNS change is not really possible in unit tests so @smuu agreed to help me test manually

This all looks good. The only hesitation I have is that reconnectPeer is relentless. It's designed for persistent peers, not for normal peers that you want to connect to. Thus, afaik, it ignores errors and continuously redials.
When trying to reconnect, if this fails we should give up trying to connect the peer. I would suggest we add some additional input to the reconnectToPeer method which will distinguish between retry forever and retry once and if failed log an error

For context here the peer won't try to reconnect forever. It first tries to connect 20 times with 5 second intervals(20*5=100s), then another 10 times with exponential backoff which ends in 16h(3**10 = 16hrs). it will give up after that so no change needed.

yeah, we should try it out


Add option to download the response data in csv/excel format

Currently the response is downloadable only in json format. It would be good if we were able to download the response into csv / Excel format.
The view response documentation does not mention the above format.
https://support.insomnia.rest/article/63-viewing-responses

The view response documentation does not mention the above format.
https://support.insomnia.rest/article/63-viewing-responses

Thank you for pointing this out. In fact, Insomnia does preview CSV files, but not XLS files. If there is an easy way to add XLS support, I would rather add that as an officially-supported format.


[DOCS] Create a step-by-step installation docs with images

The context of images here means, while creating the docs make sure to only add images related to the installation of the project.

Hello Jatin, I would love to contribute to this project and I want to get started. Can you assign this issue to me? Thank you.

👋 Hello! I'd be happy to help with this issue. Could you please assign it to me? Thank you! 🚀
I have just now have created a beautiful Learn.md Link with Video as well

👋 Hello! I'd be happy to help with this issue. Could you please assign it to me? Thank you! 🚀 I have just now have created a beautiful Learn.md Link with Video as well

Thankyou for your interest.
But this issue is already assigned to @pankildoshi.

Hi @jatiinyadav , I have raised the PR. Can you review it and also let me know if there are any changes required. Thank you!


Fix initialize_beacon_state_from_eth1 previous_version

Thank @tbenr for finding this error!

On second thought, I’m not convinced what correct behavior for this versioning is.

compute_domain() in process_deposit is “fork agnostic” and is always based on GENESIS_FORK_VERSION
There is no prior fork when initializing a chain so the use of ALTAIR does not immediately seem correct
The primary usecase of previous_version is to verify signatures from the previous fork, but there is no previous history prior to the genesis in this case

Because of the above, I’d argue to either initialize it to GENESIS_FORK_VERSION because deposits use this OR just initialize it to MERGE_FORK_VERSION which I think is a bit more semantically correct when looking at how the state in initialized in phase 0.
The altair analogue of this function uses GENESIS_FORK_VERSION so double MERGE_FORK_VERSION isn’t consistent unless we change the function in altair as well.

my two cents (just because I raised the problem).
the second option seems preferable to me:
current_version=MERGE_FORK_VERSION,  # [Modified in Merge]
previous_version=current_version

I was also exploring the idea about introducing a non-configurable constant NO_FORK_VERSION=Version('0x00000000'). Using this should maintain phase0-MainNet compatibility but I guess brakes pahse0-devnets.

My institution was making it more like a mainnet real case, but you convinced me that MERGE_FORK_VERSION is better for the testing context.
If possible, I'd like to also change Altair initialize_beacon_state_from_eth1 to using previous_version=ALTAIR_FORK_VERSION.
Note: it has nothing to do with the mainnet consensus, just for testing.

@djrtwo @tbenr
I changed it to previous_version=current_version.

Agreed that it's reasonable to adjust the altair function as well.
This affects not only testing, but also testnets/networks that deploy their genesis from altair logic. I don't think we have any such nets today, so it's okay to introduce this change


added Archlinux and Debian support for install_ansible_rulebook

I added Archlinux and Debian support for the install_ansible_rulebook role.

Thanks for the PR @Haennetz. Could you please rebase and resolve the conflict?

@Haennetz on second thoughts theres no need for a rebase. We are in the process of refactoring the installer so we'll incorporate your additions into that refactor. Thanks.


Type of change
New feature or functionality (change which adds functionality)
Anyone using our LATEST_VERSION symbol will be on TS 5.0 now
Test plan

Covered by existing test cases

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


Use Coffee-Script for swagger-n.js

Looking at https://github.com/paulhill/swagger-n/blob/master/lib/swagger-n.js is really nice and simple
And it would be even better if written in Coffee-Script
That is an accepted idea I'm happy to send a PR with it :)

It was specifically written in JS to be very simple and vanilla. I borrowed
ideas from swagger-jack which is coffee-script.
That said if you want to make a coffee-script version that passes the same
(yet to be written) test suite, that would be cool.
On Sunday, January 18, 2015, Dinis Cruz<EMAIL_ADDRESS>wrote:

Looking at
https://github.com/paulhill/swagger-n/blob/master/lib/swagger-n.js is
really nice and simple
And it would be even better if written in Coffee-Script
That is an accepted idea I'm happy to send a PR with it :)
—
Reply to this email directly or view it on GitHub
https://github.com/paulhill/swagger-n/issues/6.


fix: add missing dependency (pylibjuju)

The tests use importlib.metadata to get the installed version of pylibjuju (tests/conftest.py):
    if juju_version := os.environ.get("LIBJUJU_VERSION_SPECIFIER"):
        juju_version.replace("==", "")
        juju_version = juju_version[2:].split(".")[0]
    else:
        juju_version = version("juju")

However, pylibjuju is not installed as part of the requirements. This means that all tests fail with an error like:
ERROR tests/unit/test_data_interfaces.py::TestDatabaseProvides::test_database_requested_event - importlib.metadata.PackageNotFoundError: No package metadata was found for juju

This is fixed by installing pylibjuju for the unit environment as well as integration. Alternatively, the version("juju") could be wrapped in a try/except, and you could fall back to a hard-coded version?

Looks good! I would just update the reference to the juju to something more recent than 2.9.x

It's the same as you have for your integration tests (I assumed you'd want it to match), but happy to change it to whatever you'd prefer.

We tests a number of versions, among which <IP_ADDRESS> which is the oldest. I would just keep on testing on CI on that one, but I would move the default reference to latest, e.g. <IP_ADDRESS>. Otherwise the reference would already seem out of date.
I don't think that should break anything, transitive dependencies included in tox.ini are not resolved in charmlib anyway... :D


Flux point not connecting to phantom energyface

Hello, im trying to connect the flux point to a phantom energyface to make insane amount of power just like in SSundee's skyfactory 3 video [35]. I'll show an example of what is happening in the photo.
Link: https://imgur.com/a/vFutP
I have updated mods to the latest [1.10.2]

This interaction will be disabled by default in the next version (1.12.2 only for now)


Task 12.1 Create a virtual machine to run the API so it is always running and accessible.

EC2 AWS Linux


SOCKS example code from wiki fail to run

Hi!
I took code from here: https://urllib3.readthedocs.org/en/latest/contrib.html#socks
And here is my result: http://prntscr.com/9njs5b
I'm using Windows 7, Python 3.5, urllib3 newest version.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/29703735-socks-example-code-from-wiki-fail-to-run?utm_campaign=plugin&utm_content=tracker%2F192525&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F192525&utm_medium=issues&utm_source=github).

That error looks like your PySocks is too old. What version do you have installed?

Thanks very much, so problem is my pySocks is too old. Sorry for this inconvenient :)


[Docs] Ruby Version Error Fix for make docker

Notes for Reviewers

This PR fixes #12952
Added a dockerfile to build a custom Jekyll environment having ruby version 3.2.2 as mentioned in the gemfile.
Now the error your ruby version is x.x.x but your Gemfile specified 2.7.x shall no longer occur.
Tested it by running make docker successfully.

Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit e86b4c3e53c70270f2743b94ac7a08096fbba39e at: https://6772aba863703b9f640d2192--meshery-docs-preview.netlify.app


Update to idiomatic rust

Dependent on #25

Resolved some issues that clippy was giving.
renamed some functions to not make them collide with functions from std

Also fixed some small stuff from indy-utils, just to make clippy be quiet.

I converted all the Into<T> to TryInto<T> but a downside now is that if we request Option<impl Into<SchemaId>, and we want to supply None, we MUST do it like this: None::<SchemaId> which I do not like. We only do this for the rev_reg_def_id, but it is a weird inconsistency. Not too sure what the right direction is here.


revert "virtualSize" name change

Commit b717de5153db503fae61c11e30f04f400f66a6fd (https://github.com/docker/docker/pull/26108/commits/b717de5153db503fae61c11e30f04f400f66a6fd#diff-4adce79060ec85cddf1545ebadcc0d5eR244) changed the name of the "size" argument to "virtualSize", as the "VirtualSize" field was re-used for calculating the size of all layers not used by other images.
be20dc15af0cb281bd6d11586cfcc96bd50d12ca (https://github.com/docker/docker/pull/30061) reverted the change in calculation, but did not change the argument name back to "size".
This changes the name back to its original name, because since the introduction of the content-addressable store in docker 1.10, there no longer is a "virtual" size, so "size" is a better name for this argument.
/cc @vdemeester @mlaventure

Note that this is just a "nit", no real reason to cherry-pick this into the release branch


AUX Serial Throttling

 Update note-arduino with AUX throttling

I missed the review before this was merged -  just noting that I've looked over the code. 👍


Class is not applied to component extending abstract component

Vue.js version
2.1.8
Reproduction Link
Codepen
outer class is not applied to a component`s $el node.
Try to comment absract: true line and you will see that class is applying.
What is Expected?
Outer class must be applied to the root component element.
What is actually happening?
Outer class is not applied if the component extends a base component with abstract flag.

Extending an abstract component, the component test becomes abstract too. IMO abstract components can have only bindings defined in props.

Abstract components are not designed to render templates of their own - they are like "decorators" that simply wrap other components/elements via composition. e.g. <transition>:
<transition>
  <component></component>
</transition>

So abstract components do not have DOM elements of its own - it simply enhances its slot children. Extending an abstract component and give it some concrete DOM doesn't make sense.
Also, abstract mode is not publicly documented and may change without notice, you should avoid using it unless you are really sure you know what you are doing ;)

Any plan to include abstract components in the doc?


install error

pip install -r requirements.txt
Processing /home/conda/feedstock_root/build_artifacts/ase_1602722750108/work
ERROR: Could not install packages due to an OSError: [Errno 2] No such file or directory: '/home/conda/feedstock_root/build_artifacts/ase_1602722750108/work'
do u have a clean requirements.txt?

@usccolumbia I'm not affiliated, but this seems to work (generated via pipreqs). You just need to deal with the protobuf dependency ad-hoc per the comment or https://github.com/tensorflow/tensorflow/issues/6341.
https://github.com/sparks-baird/AMDNet/blob/main/requirements.txt


Add RISC-V Compliance v0.2 check

RISC-V defines a Compliance Framework which is available in v0.2 as RISCOF.
It is still marked as beta but is now at a point where it can be utilised.
It allows for a more flexible way to check for compliance.
I hacked together a first test which runs some checks successfully. This is based on the current dv/riscv-compliance work as it will also compare the generated signatures.
It sill needs some work for an initial version.
This issue should keep track of the open points and questions.

[ ] Implement Ibex model plugin according to riscof-plugins

[ ] Make it variable so different configurations of Ibex can be checked

[ ] Can / should ibex_config.py be extended to generate a YAML specification which is needed by riscv-config which serves as an input checker for RISCOF
[ ] Add it to CI?
[ ] Choose the reference model to which Ibex is compared
[ ] Document the usage
[ ] Remove riscv-compliance? I am not yet sure how RISCOF is integrated into riscv-compliance as mentioned in https://github.com/riscv/riscv-compliance#issuecomment-747524231. Maybe it is only a way to run the test created in the new framework?

Documentation for reference RISCOF, riscv-config, riscv-isac

Add it to CI?

Absolutely.

Add it to CI?

Absolutely.


The generated code is invalid when the library name has a separator

When instantiating a new project with a separator in the library name like this:
plule@LYOPLULE1:~$ ansys-templates new pyansys
product_name [Product]: platform
library_name [Library]: some_lib
version [0.1.dev0]:
short_description [A Python wrapper for Ansys platform some_lib]:
repository_url [https://github.com/pyansys/pyplatform-some-lib]:
Select requires_python:
1 - 3.7
2 - 3.8
3 - 3.9
4 - 3.10
Choose from 1, 2, 3, 4 [1]:
max_linelength [100]:

The source is structured with a folder named "some-lib" (I believe this is invalid in python):
plule@LYOPLULE1:~$ ls pyplatform-some-lib/src/ansys/platform/some-lib/
__init__.py  __pycache__

but the test assumes that the package is "some.lib":
plule@LYOPLULE1:~$ cat pyplatform-some-lib/tests/test_metadata.py
from ansys.platform.some.lib import __version__

def test_pkg_version():
    assert __version__ == "0.1.dev0"

Thanks for discovering this @plule-ansys. This should be addressed in the cookiecutter.json files of each template. Let me open a PR to solve this.


⚠ ClamAV has degraded performance

In 9af5470, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 74 ms

Resolved: ClamAV performance has improved in 29bca50.


AmazonReport uses private variables, limiting possiblities to extend

I doubt it's the only place private variables are used, but in my case I want to save the report to a variable, and not to a file, and then manipulate it, but as it stands I can't extend and access the $rawReport variable.
In general, it would be useful for most private variables to be made protected instead, to allow users to extend to fit their own use cases.

That's a good point. When I first wrote the class, the only requirement I had was that it could save to a file, so I didn't even think to add a getter for the raw data.
I'll work on improving it when I am able to. I want to put some thought into whether or not to make the variables protected, but at the very least I want to add a getter for the data.

Taken care of.

Superb! Thank you :-)
I'm just coming to use the library for a whole new project, and it looks like it's had some great improvements since I last used it!
Great work!


fix(tornado.py): support concurrent requests in tornado

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


Document how to manage channels via the System Console

Context
Details on how to manage Mattermost channels as a system admin using the System Console are missing from the Mattermost Product Documentation.
Docs Request
Develop admin documentation for the following topics:

Find channels
Filter channel searches
Review channel configuration
Advanced access controls

Create messages
Message reactions
Manage members
Channel mentions
Manage bookmarks

Channel management

Sync group members
Public channel or private channel

Groups (Skipped)
Members

Add members
Remove members

Note: The ability to add groups to a channel also needs to be documented, but requires additional internal investigation/testing first.
How you can help

Create a single docs PR that contains all of these changes across as many commits as necessary
Use the latest Mattermost mobile app and the standard Mattermost Denim theme to capture all workflows.
Use existing admin docs content as a starting point for voice and tone guidance. The Docs team will review and provide support for publication.

@cwarnermm Please assign this issue to me. Thanks.


"Invalid argument" executing symbolic link on DrvFs mount with absolute DrvFs Windows binary target

Your Windows build number:
Microsoft Windows [Version 10.0.17763.379] (myself & 3rd party)
Did not affect 17134.648 (reporter by another 3rd party)

What you're doing and what's happening:
Executing symbolic link on a drvfs mount with absolute drvfs Windows binary target returns ./linkNameHere: Invalid argument.
Executing symbolic link on a drvfs mount with relative drvfs Windows binary target functions as expected.
Executing symbolic link on a drvfs mount with Linux binary target functions as expected.

Affects both Ubuntu-16.04 & Ubuntu-18.04 distributions.
Feels related to #3889
Repro:
(#!/bin/bash)
$ cd /mnt/c/Users/fred
$ #Setup
$ cp /bin/echo localEcho
$ ln -s /bin/echo lxfsLinuxAbsTarget
$ ln -s localEcho drvfsLinuxRelTarget
$ ln -s $(pwd)/localEcho drvfsLinuxAbsTarget
$ cp /mnt/c/Windows/System32/cmd.exe localCmd.exe
$ ln -s localCmd.exe drvfsWinRelTarget
$ ln -s $(pwd)/localCmd.exe drvfsWinAbsTarget

$ #Test
$ ./lxfsLinuxAbsTarget HELLO
HELLO
$ ./drvfsLinuxRelTarget HELLO
HELLO
$ ./drvfsWinRelTarget /C echo HELLO
HELLO
$ ./drvfsWinAbsTarget /C echo HELLO
./drvfsWinAbsTarget: Invalid argument
                                     $

$ #Cleanup
$ rm -f drvfsLinuxAbsTarget  drvfsLinuxRelTarget  drvfsWinAbsTarget  drvfsWinRelTarget  localCmd.exe  localEcho  lxfsLinuxAbsTarget

What's wrong / what should be happening instead:
Failing to run windows .exe. Should run.

Strace of the failing command, if applicable:
Strace between working & failing examples provided no major insight.
Standard ETL logs (https://github.com/Microsoft/WSL/blob/master/CONTRIBUTING.md#8-detailed-logs) provided not major insight.

For WSL launch issues, please collect detailed logs.
I don't believe this is WSL launch, as much as
WSL runs fine, just a specific process launch approach

See our contributing instructions for assistance.

@benhillis Would this use the same path resolution code that is likely behind #3889 ?


Update station to 1.7.0

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Sorry but PR #44423 was posted first.

Closing as duplicate. Thank you anyway for the submission, @naotone.


Failure in DPOTRF() in SKYLAKEX AVX512 build but works fine in HASWELL build.

I've built the development version (downloaded June 25) for TARGET=HASWELL and TARGET=SKYLAKEX.  Unfortunately, the attached test program fails using the version of the library built with TARGET=SKYLAKEX and works correctly using the TARGET=HASWELL library.
Other flags used were CC=gcc-6, FC=gfortran-6, and USE_OPENMP=1.  I had to add -march=skylake-avx512 to COMMON_OPT on SKYLAKEX build to get AVX512.
The results for HASWELL were:

./testdpotrf 10000
10000   2.07    161.1  6.74e-13

The results for SKYLAKEX were:

./testdpotrf 10000
dpotrf info=33

This indicates that the Cholesky factorization failed at A(33,33).  This particular matrix is known to be positive definite and the Cholesky factorization proceeds without error using MKL and the reference BLAS/LAPACK.  Of course it also works correctly on the development version of OpenBLAS with TARGET=HASWELL, so this would seem to indicate a bug in the AVX512 support for SKYLAKEX.
testdpotrf.zip

AVX512 support is just SGEMM/DGEMM right now. (Wonder if the corresponding spotrf testcase would succeed, as DGEMM was an add-on to the original PR - unfortunately I do not have the hardware)
@fenrus75 ?

I'll try to take a look.
it's... curious how this can happen; if dgemm was completely broken a lot more things would be failing

The testspotrf test case works correctly with this version of the library, so the problem is specific to double precision.

I've rebuilt the libraries and tried to make everything as identical as possible except for the TARGET.  The gcc-6 compiler used here is:
gcc version 6.4.0 20180424 (Ubuntu 6.4.0-17ubuntu1~16.04)

make CC=gcc-6 FC=gfortran-6 USE_OPENMP=1 TARGET=HASWELL

The build completed with reported test failures.  Both testspotrf and testdpotrf produce correct results.

make CC=gcc-6 FC=gfortran-6 USE_OPENMP=1 TARGET=SKYLAKEX

All files were compiled with march=skylake-avx512.  I did not have to modify COMMON_OPT
Several tests failed:
 ******* FATAL ERROR - COMPUTED RESULT IS LESS THAN HALF ACCURATE *******
           EXPECTED RESULT   COMPUTED RESULT
       1      0.262339         -0.849231E-01
      THESE ARE THE RESULTS FOR COLUMN   1
 ******* DSYMM  FAILED ON CALL NUMBER:
    382: DSYMM ('R','U',  1,  7, 1.0, A,  8, B,  2, 0.0, C,  2)    .

 DTRMM  PASSED THE TESTS OF ERROR-EXITS

 ******* FATAL ERROR - COMPUTED RESULT IS LESS THAN HALF ACCURATE *******
           EXPECTED RESULT   COMPUTED RESULT
       1      0.485095          0.186814
      THESE ARE THE RESULTS FOR COLUMN   2
 ******* DTRMM  FAILED ON CALL NUMBER:
    758: DTRMM ('R','U','N','U',  1,  7, 1.0, A,  8, B,  2)        .

 DTRSM  PASSED THE TESTS OF ERROR-EXITS

 ******* FATAL ERROR - COMPUTED RESULT IS LESS THAN HALF ACCURATE *******
           EXPECTED RESULT   COMPUTED RESULT
       1      0.186813         -0.166775E-01
      THESE ARE THE RESULTS FOR COLUMN   1
 ******* DTRSM  FAILED ON CALL NUMBER:
    764: DTRSM ('R','U','T','U',  1,  7, 1.0, A,  8, B,  2)        .

With this version of the library, testspotrf produced correct results, but testdpotrf produced incorrect results as described before.

./testdpotrf 10000
dpotrf info=33

Comments:

1. I suppose that there could be a compiler bug that shows up only with -march=skylake-avx512.

2. The only calls to BLAS/LAPACK in the test program are to dgemv, dptorf, dpotrs, and dnrm2.

I've attached the test programs and output from the SKYAKEX build
[testproblems.zip](https://github.com/xianyi/OpenBLAS/files/2138970/testproblems.zip)

Please let me know if there's anything further that I can do to help isolate this bug.

Setting OMP_NUM_THREADS leads to a slightly different failure:

./testdpotrf 10000
dpotrf info=19

I will spend my evening on tihs later today
if I can't find it we may need to disable dgemm/avx512 for now
On Tue, Jun 26, 2018 at 2:35 PM Brian Borchers<EMAIL_ADDRESS>wrote:

Setting OMP_NUM_THREADS leads to a slightly different failure:
./testdpotrf 10000
dpotrf info=19
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xianyi/OpenBLAS/issues/1643#issuecomment-400470639,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPeFWUIo7q38PJSHVL3vo7R_DfD0oc8ks5uAqk4gaJpZM4U3NxM
.

I noticed that DPOTRF calls DTRSM, which is one of the routines that failed its tests during the build.   Given that DTRSM is failing its test at build time, this should probably be the priority in debugging the problem.    I'd just assume that DPOTRF is failing because DTRSM is broken.
Haven't there been other reported problems with DTRSM recently?
This could be a bug with gcc-6 in combination with -march-skylake-avx512.  Perhaps applying the march=skylake-avx512 flag only to routines needed by dgemm/sgemm would resolve the problem in dtrsm and point to a compiler bug.

trsm.c calls gemm_thread_m and gemm_thread_n, so there could well be an issue with dgemm that is breaking dtrsm and then dpotrf.

avx512 changed the shape of the kernel. there is a pre-existing haswell
version of that shape I used as base
my first test will be to swap to the hsw version
if that fails it's the shape not the actual avx code
On Tue, Jun 26, 2018, 16:49 Brian Borchers<EMAIL_ADDRESS>wrote:

trsm.c calls gemm_thread_m and gemm_thread_n, so there could well be an
issue with dgemm that is breaking dtrsm and then dpotrf.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xianyi/OpenBLAS/issues/1643#issuecomment-400498101,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPeFYywxUlZ22RzFlBuB_UDULF_C0H8ks5uAsh4gaJpZM4U3NxM
.

OK- let me know if you'd like me to run a test with this.

Not all of the pre-existing kernel files may be usable without prior inspection unfortunately, if nothing uses them currently they may have been retired due to unexplained errors but never removed. Others including the generic trmmkernel_16x2 have no apparent usage history at all.
I have now tried a Haswell build with the kernels used on Skylake (obviously using the pre-existing 16x2 dgemm kernel rather than the new avx512) and see DSYMM,DTRMM and DTRSM failing as well.

It appears to be the dgemmkernel_16x2 itself that causes the failures. Looking through the git history I am not convinced it was ever used, although there was some work done on it in 2013. (Checking now if it is just the GEMM_DEFAULT_UNROLL_ parameters in param.h that must be adjusted to go with the change in kernel)

if the 16x2 will never work I might be able to get the piledriver 8x2 to
work for dgemm... but it's narrow and that will hurt performance a bit.
On Wed, Jun 27, 2018 at 1:45 AM Martin Kroeker<EMAIL_ADDRESS>wrote:

It appears to be the dgemmkernel_16x2 itself that causes the failures.
Looking through the git history I am not convinced it was ever used,
although there was some work done on it in 2013. (Checking now if it is
just the GEMM_DEFAULT_UNROLL_ parameters in param.h that must be adjusted
to go with the change in kernel)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xianyi/OpenBLAS/issues/1643#issuecomment-400592248,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPeFZHNUPWC9MC1tgLHVEZbllJ0JQ-Dks5uA0YdgaJpZM4U3NxM
.

Tried an earler version of dgemm_16x2_haswell (the one labeled "corrected and tested" that preceded the "optimized" commit), but that even fails on DGEMM itself (and DSYRK, while interestingly DTRMM passes)

curious but...
now what?
I suspect the 16x2 just needs to be retired since clearly it never worked
at all.
I'm pondering writing a whole new dgemm kernel since I kind of want a 32x4
(or 16x8 but 32x4 is nicer) just it is a much steeper ramp than just
replacing the kernel of a supposedly-working wrapper framework.
On Wed, Jun 27, 2018 at 7:38 AM Martin Kroeker<EMAIL_ADDRESS>wrote:

Tried an earler version of dgemm_16x2_haswell (the one labeled "corrected
and tested" that preceded the "optimized" commit), but that even fails on
DGEMM itself (and DSYRK, while interestingly DTRMM passes)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xianyi/OpenBLAS/issues/1643#issuecomment-400696194,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPeFdsjWVVfHVt_AnNClIJOfc68un8Wks5uA5jKgaJpZM4U3NxM
.

I have disabled that DGEMM kernel for now (after fumbling aimlessly both with the 16x2 haswell kernel and my attempt of adapting its 8x2 piledriver counterpart) and moved the milestones on the two related tickets to 0.3.2. This will allow 0.3.1 to still get the initial SkylakeX support with the AVX512 SGEMM kernel

The latest development version still fails on one of the tests at build time.  With
make TARGET=SKYLAKEX CC=gcc-6 FC=gfortran-6 USE_OPENMP=1
I get:
DTRMM  PASSED THE TESTS OF ERROR-EXITS
******* FATAL ERROR - COMPUTED RESULT IS LESS THAN HALF ACCURATE *******
EXPECTED RESULT   COMPUTED RESULT
1      0.485095          0.186814
THESE ARE THE RESULTS FOR COLUMN   2
******* DTRMM  FAILED ON CALL NUMBER:
758: DTRMM ('R','U','N','U',  1,  7, 1.0, A,  8, B,  2)        .
This is exactly the same failure that we were seeing before the DGEMM kernel was removed.

Sorry - I had not disabled the 16x2 TRMMKERNEL as well with the latest PR, thinking it was harmless. Thanks for catching this, I will create a new PR in a minute.

Now the SKYLAKEX build works without errors.  The SKYLAKEX version is about 10% faster on SGEMM (360 gigflops vs. 330) than the HASWELL build.  So, no significant performance improvement yet, but everything does function.

sgemm also still needs more performance tuning; avx512 really wants bigger
kernels to hit the 32 FMAs per cycle that the cpu can do.
On Sat, Jun 30, 2018 at 9:34 AM Brian Borchers<EMAIL_ADDRESS>wrote:

Now the SKYLAKEX build works without errors. The SKYLAKEX version is about
10% faster on SGEMM (360 gigflops vs. 330) than the HASWELL build. So, no
significant performance improvement yet, but everything does function.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xianyi/OpenBLAS/issues/1643#issuecomment-401551740,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPeFemAmqVO6f19hoyDi-s22k0JO9dNks5uB6iVgaJpZM4U3NxM
.

Still I guess it would be labeled significant if we ever saw a ten percent performance loss somewhere...

absolutely. some companies (ahem) spend a LOT of effort making a cpu a few
percent faster in architecture.
performance comes by foot, but leaves by horse.
anyway current code tops out at 18 multiplies per cycle.... I have kernels
in my test bench that get over 29....
but they're bigger and I need to then change a lot more "glue" in openblas
to make them fit, and I know I need to learn a lot more of said "glue" first
On Sat, Jun 30, 2018 at 9:59 AM Martin Kroeker<EMAIL_ADDRESS>wrote:

Still I guess it would be labeled significant if we ever saw a ten percent
performance loss somewhere...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xianyi/OpenBLAS/issues/1643#issuecomment-401553272,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPeFYlhUPzImhfomqeLIiCdj9BnCB7Bks5uB65agaJpZM4U3NxM
.

FYI, some SKL-X CPUs come with 1 AVX-512 execution unit per core (under 10 cores, so i7 7820X and under), but the 10 core (i9 7900X) and higher core count CPUs have 2 AVX-512 units per core
not sure if this matters, just thought to put it down here, because it is not widely known

Somehow I suspect fenrus75 knows this :smile:

Assuming fixed by the new DGEMM kernel now in 0.3.4


req.ReadFormAsync() throws exception with Microsoft.Extensions.Primitives v 3.0 with Azure Function

Describe the bug
Using Azure Functions version 2
The following line throws an exception with Microsoft.Extensions.Primitives v3.0/1
var formdata = await req.ReadFormAsync();
Exception "System.MissingMethodException: 'Method not found: 'Microsoft.Extensions.Primitives.StringValues Microsoft.AspNetCore.Http.IFormCollection.get_Item(System.String)'.'
If I downgrade to version 2.0 it works fine.
To Reproduce
Steps to reproduce the behavior:

Create a new Azure Function (using VS 2019 16.3.9)

Add the following line: var formdata = await req.ReadFormAsync();

Test function, everything should work properly.

Add Nuget Package Microsoft.Extensions.Primitives (3.0.1)

Run the function again, you  will get exception:

System.MissingMethodException: 'Method not found: 'Microsoft.Extensions.Primitives.StringValues Microsoft.AspNetCore.Http.IQueryCollection.get_Item(System.String
Expected behavior
No exception should be thrown, with req.ReadFormAxync();
Screenshots

I believe this is expected.  Azure Functions runs on the 2.2 version of ASP.NET Core, so it probably already has the 2.2 version of this assembly loaded.

I ran into a very similar issue. I had tried adding the Logging extensions package version 3.0.x and couldn't figure out what the problem was (tons of MissingMethodExceptions). Turns out that like Pilchie mentions, the 2.x version is already loaded by the runtime. In my case removing the reference fixed the issue.


Raml 1.0?

Hi,
I stumbled upon this project when looking for a way to generate a flask server specified by a raml file.
Sadly I'm going to use raml 1.0 while this project is using raml 0.8..
Are there any plans on upgrading to 1.0?
Cheers,
Theo

Would be interesting to know indeed. What's the overall status?


Return table columns in order it was defined in meta data rcp message

GetTableMetaResponse currently sorts the columns
Need to return the columns in the order it was defined in the TableDef to enable driving of the table column order from server side. This remove the need for UI configuration to set the column order in default cases.

duplicate https://github.com/finos/vuu/issues/1308


ReferenceError: __WC_CONFIG__ is not defined

Describe the bug
Freshly created tutorials without any modifications are running into __WC_CONFIG__ is not defined errors. This is coming up randomly and looks like possible race condition. When page is refreshed with cache clearing the error disappears, but normal refreshes run into this error immediatelly.
This only affects dev mode.
Link to a StackBlitz project which shows the error
No response
Steps to reproduce

pnpm create tutorial
Start in dev mode
Open page in browser
Keep refreshing page and check JS console

Expected behavior
Error should not happen.
Screenshots

Platform

TutorialKit version: 0.2.0

Additional context
No response

Could this be a bug with Astro? I wonder if we also see this issue with an older version of Astro.
I'm wondering because this is literally the first thing we set in the vite config and I don't think this has ever really changed:
https://github.com/stackblitz/tutorialkit/blob/3e7e6cb59f7f6ffd4551f483d71f36f51fcf0ef9/packages/astro/src/index.ts#L77-L94
(also if that helps bisecting the combo that breaks, I've never encountered that issue when I was doing demos back in June)

This issue isn't present on 0.1.2 but appears first in 0.1.3.
https://github.com/stackblitz/tutorialkit/compare/0.1.2...0.1.3

This is likely introduced in https://github.com/stackblitz/tutorialkit/pull/219.
When import of './OpenInStackblitzLink.astro' is removed here, the issue disappears:
https://github.com/stackblitz/tutorialkit/blob/3e7e6cb59f7f6ffd4551f483d71f36f51fcf0ef9/packages/astro/src/default/components/TopBarWrapper.astro#L6-L8
There's two depedency chains for the setup.ts:
TopBarWrapper.astro > setup
TopBarWrapper.astro > OpenInStackblitzLink.astro > webcontainer > setup

The first one works fine. But the second on comes from

And we get:

Looks like Vite's dev-time define is added loaded after the Astro component is. Looks like bug to me.
I'll try to find quick solution for us here so that we can release fix as soon as possible. Maybe convert the component into React one. 🤔

https://github.com/withastro/astro/issues/11874


Introduce the timeout operator modifier

... and remove the one-off timeout option that we added to the batch operator.

@jachris and I discussed this and given our internal use cases for this being still a bit out we decided to close this for now and to revisit it later. Ultimately, having timeout primitives like this is really valuable, e.g., deduplicate can be expressed as group … timeout … head …, and we also will need this for the Feather store in the future, but then we probably want to allow counting both from the first and the last event, respectively.


iOS error on gradle sync "Could not resolve com.apollographql.apollo:apollo-compiler:4.0.0-rc.1"

Version
4.0.0-rc.1
Summary
Trying to convert our existing apollo module to KMP and getting this error in every iOS variant

tried 4.0.0, same issue.
Kotlin 1.9.23
AS Koala 2024.1.2 Beta 2
plugins {
    alias(libs.plugins.kotlinMultiplatform)
    alias(libs.plugins.myapp.base.plugin)
    alias(libs.plugins.apolloKotlin)
}

kotlin {
    jvm()
    iosX64()
    iosArm64()
    iosSimulatorArm64()

    sourceSets {
        commonMain.dependencies {
            api(project(":libs:apollo:api"))
            api(libs.apollo.api)
            api(libs.apollo.runtime)
            api(libs.coroutines)

            implementation(project(":libs:apollo:schema"))
            implementation(project(":libs:logger"))
            implementation(libs.apollo.normalizedcache)
            implementation(libs.apollo.normalizedcache.api)
        }
    }
}

apollo {
    service("service") {
        packageName.set("apollo.impl")
        dependsOn(project(":libs:apollo:schema"))
    }
}

Definitely possible that I'm doing something wrong but I can't figure out what

The compiler is not multiplatform (yet!), it's JVM only. You can't add it to a multiplatform project.

That would do it, thanks for the quick reply! Up to you if you want to leave this open or not

I'll close it, KMP is not for any time soon.
We'd need KMP javapoet and kotlinpoet before we start tackling this.


Geocode derives to wrong values due to catx

Hi Al,
The catx to derive geocode gives the concatenated value of county,state,tract instead of state, county tract. To get the correct value cats or || can be used in place of catx. Please suggest.
Thanks!
Kanchana

Thanks for drawing my attention here! I've made the recommended replacement from catx to cats.


Datagrid ExpandedRowContentComponent component unable to use hooks

Datagrid ExpandedRowContentComponent component unable to use hooks
What package(s) are you using?

[x] Carbon for IBM Products (or Carbon for IBM Cloud & Cognitive)

Detailed description
The <Datagrid /> expanded row component is unable to utilize hooks freely. If 2 or more hook dependent ExpandedRowContentComponent components are rendered, an error is thrown: Rendered more hooks than during the previous render.. Furthermore, since the component is treated internally as a function, it would not support a class based component workaround: https://github.com/carbon-design-system/ibm-products/blob/main/packages/ibm-products/src/components/Datagrid/Datagrid/DatagridExpandedRow.js#L42C12-L42C39.
Expected behavior: I would expect expanded rows to support modern hook based components, but I would also expect that since the name is capitalized ExpandedRowContentComponent, and contains the word Component, that it be treated as a component internally and rendered via JSX, facilitating the use of class based components
Here is a codesandbox reproduction: https://codesandbox.io/s/nervous-wright-hc6fdd?file=/src/Table.js
@carbon/ibm-products version: 2.15.1
@carbon/react version: 1.41.2

https://github.com/carbon-design-system/ibm-products/pull/3787

Hey @lluisrojass, thanks for the detailed issue and joining office hours yesterday to discuss further. This fix should be included in our next weekly release (Nov 14).

Thank you, the fix is working properly. Cheers!


Cannot set channel description

Is there a option call channelDescription to replace "rn-push-notification-channel-description"?
Seem only have channelName now.

or just disable it

@lauming1111 Right now it's not supported but I can add it when I have time this weekend. May I ask why you want to disable it?

Thanks,
Because I think it is an optional function now.

thank you for the update.
is there a parameter that can handle badge numbers in the main theme in android?
how to make notification.setNumber(Integer.parseInt(numberString)); can dynamic change from react-native?
so, I can pass the real unread message to the app icon badges in android.

@lauming1111 Try adding "number" in your notification payload:
https://github.com/CatalystCode/react-native-azurenotificationhub/blob/7275f6c1d5753ea7f50deb037aa010fcc6d74584/android/src/main/java/com/azure/reactnative/notificationhub/ReactNativeNotificationsHandler.java#L130


Silent problem when loading or displaying DAE files

Hi all. Thanks again for the contribution you are bringing with MeshCat.
I have a problem when loading some DAE files, for example the forearm of the UR5 as available here:
https://gepgitlab.laas.fr/gepetto/example-robot-data/-/raw/master/robots/ur_description/meshes/ur5/visual/upperarm.dae
The DAE seems to be properly parsed, and no warning or error is displayed. Yet it does not show in the viewer. Trying the same code on the corresponding STL file (for example obtained by converting it with osgconv) works like a charm.
Am I doing something incorrect? If not, would you have some directions for me to investigate the problem?
Thanks in advance.
import meshcat
viz=meshcat.Visualizer()
viz.open()

# I used the DAE packaged in APT (deb [arch=amd64] http://robotpkg.openrobots.org/packages/debian/pub bionic robotpkg)  robotpkg-py36-example-robot-data
# Change the path if you are using the url above to obtain it.
dae=meshcat.geometry.DaeMeshGeometry.from_file('/opt/openrobots/share/example-robot-data/robots/ur_description/meshes/ur5/visual/upperarm.dae')
material = meshcat.geometry.MeshPhongMaterial()
material.color = 255
material.transparent=False
viz['upperarm'].set_object(dae,material)

Are you still having trouble?
I'm having different problems with .dae files, as described here:
https://github.com/stack-of-tasks/pinocchio/issues/1475
I haven't had any problem loading the .dae files for the ur5 from the ur_description package from the Universal Robots repository, but I'll try loading all the UR5 meshes from example-robot-data later today and post the results here.

The MeshCat direct loads from example-robot-data work fine for me (red parts):

Having problems with the Pinocchio viewer versions, still trying to track that down.
Code:

import example_robot_data
import pinocchio as pin
import numpy as np
import meshcat

robot = example_robot_data.load('ur5')

# Currently, MeshCat is not able to retrieve the scaling from DAE files. Set it manually.
for geom in robot.visual_model.geometryObjects:
    s = geom.meshScale
    s *= 1.0
    geom.meshScale = s
robot.setVisualizer(pin.visualize.MeshcatVisualizer(robot.model, robot.collision_model, robot.visual_model))
robot.initViewer(open=True)
robot.loadViewerModel('ur5_model')
robot.display(robot.q0)

# -- extract the concrete .dae paths from the text repr of robot.visual_model --
s = repr(robot.visual_model)
meshes = {} #key, value like wrist2:path/to/wrist2.dae
blocks = s.split('Absolute path to mesh file:')[1:]
for block in blocks:
    meshpath = block.split('Scale')[0].strip()
    meshkey = meshpath.split('/')[-1].replace('.dae', '')
    meshes[meshkey] = meshpath

# -- add wrist2.dae as a comparison
material = meshcat.geometry.MeshPhongMaterial()
material.color = 0x802317
dae_file = meshes['wrist2']
for n, (mk, mp) in enumerate(meshes.items()):
    mn = mk + '_comparison'
    robot.viz.viewer[mn].set_object(meshcat.geometry.DaeMeshGeometry.from_file(mp), material)
    print(f'Adding {dae_file}')
    T = np.array([[-1.,  0.,  0.,  0.8+0.3*n],
                  [ 0.,  1.,  0.,  0.3],
                  [-0.,  0., -1.,  0.1],
                  [ 0.,  0.,  0.,  1.]])

    robot.viz.viewer[mn].set_transform(T)

The above issue is fixed in Pinocchio 2.9.1 and is not a MeshCat issue.

What is the status of this issue?
I cannot display ANYmal (using https://github.com/Gepetto/example-robot-data) properly.
See the picture for more details:

I am using Pinocchio v2.6.7


Migrating to use Django >= 4.x

This package does not work with Django >=4.x.

I have noticed the same issue. Starting the app leads to
Exception in thread django-main-thread:
Traceback (most recent call last):
  File "/Users/awacker/.pyenv/versions/3.8.8/lib/python3.8/threading.py", line 932, in _bootstrap_inner
    self.run()
  File "/Users/awacker/.pyenv/versions/3.8.8/lib/python3.8/threading.py", line 870, in run
    self._target(*self._args, **self._kwargs)
  File "/Users/awacker/.pyenv/versions/lms-3-8-8/lib/python3.8/site-packages/django/utils/autoreload.py", line 64, in wrapper
    fn(*args, **kwargs)
  File "/Users/awacker/.pyenv/versions/lms-3-8-8/lib/python3.8/site-packages/django/core/management/commands/runserver.py", line 125, in inner_run
    autoreload.raise_last_exception()
  File "/Users/awacker/.pyenv/versions/lms-3-8-8/lib/python3.8/site-packages/django/utils/autoreload.py", line 87, in raise_last_exception
    raise _exception[1]
  File "/Users/awacker/.pyenv/versions/lms-3-8-8/lib/python3.8/site-packages/django/core/management/__init__.py", line 398, in execute
    autoreload.check_errors(django.setup)()
  File "/Users/awacker/.pyenv/versions/lms-3-8-8/lib/python3.8/site-packages/django/utils/autoreload.py", line 64, in wrapper
    fn(*args, **kwargs)
  File "/Users/awacker/.pyenv/versions/lms-3-8-8/lib/python3.8/site-packages/django/__init__.py", line 24, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/Users/awacker/.pyenv/versions/lms-3-8-8/lib/python3.8/site-packages/django/apps/registry.py", line 116, in populate
    app_config.import_models()
  File "/Users/awacker/.pyenv/versions/lms-3-8-8/lib/python3.8/site-packages/django/apps/config.py", line 304, in import_models
    self.models_module = import_module(models_module_name)
  File "/Users/awacker/.pyenv/versions/3.8.8/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 783, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/Users/awacker/.pyenv/versions/lms-3-8-8/lib/python3.8/site-packages/useraudit/models.py", line 6, in <module>
    from .signals import password_has_expired, account_has_expired, login_failure_limit_reached
  File "/Users/awacker/.pyenv/versions/lms-3-8-8/lib/python3.8/site-packages/useraudit/signals.py", line 3, in <module>
    password_will_expire_warning = Signal(providing_args=["user", "days_left"])
TypeError: __init__() got an unexpected keyword argument 'providing_args'```

https://docs.djangoproject.com/en/4.0/releases/4.0/#features-removed-in-4-0 mentions that providing_args named argument for Signal is no longer supported.

I have the same issue @mouneyrac

Hello @muccg any idea when this will be fixed?

Hello @muccg any idea when this will be fixed?

As soon as possible I hope.

I made this awhile ago, but feel free to use my pull request / branch for any of you that need it working until it gets merged in. https://github.com/muccg/django-useraudit/pull/45 . I've been running it since I made it without issue.


PR always goes to main

Describe the bug
gitops app add --branch foo .
The PR created on GitHub wants to merge to main not foo
Environment

gitops: [e.g. v0.1.0]
kubernetes:  [e.g. 1.20.4]

[X ] KinD - version]
[X ] k3s - version
[ X] cloud [e.g., EKS, AKS]  version
[ ] other - name version

Browser + version: [e.g. chrome 74, safari 12, firefox 87]

Affects versions
To Reproduce
Expected behavior
The PR on GitHub should merge to the specified branch e.g. foo.
Config and Logs
If applicable, add logs to help explain your problem. please compress the output before attaching

[ ] gitops config
[ ] gitops cli call output (and parameters)
[ ] gitops-controller logs
[ ] Events from wego-* namespaces
[ ] kubectl cluster-info dump
[ ] Prometheus alerts

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

This is working as intended. The --branch flag is for the source repository, not the destination "Config" repo.

--branch string            Branch to watch within git repository (default "main")

So if I want gitops to watch my branch foo I pass it --branch foo. It creates a PR to merge its changes into main. Why not foo? Does it matter that the .wego files are in a different branch to the watched branch?
If it always uses the main branch, is it possible to use one repo with two branches for staging and prod?

So if I want gitops to watch my branch foo I pass it --branch foo

@steveww Correct. The --branch argument only applies to the Source of the manifests. All config files get written to the default branch of the configuration repo.
We do have this issue: #380 , but we are waiting for a larger refactor around the config repo directory structure.

It creates a PR to merge its changes into main. Why not foo?

The configuration repo is separate from the Source repo.

Does it matter that the .wego files are in a different branch to the watched branch?

No, they are (logically) separate repositories in the system.

If it always uses the main branch, is it possible to use one repo with two branches for staging and prod?

They will be treated as two different Applications.
Happy to hop on a call to discuss.

Thanks for the tips. I think I'm getting the hang of it now.
IMHO the docs could be improved around how it works with the repos and to detail some more advanced use cases.


Showing only certain landmarks on android FaceLandmarker App

Hello, I am playing around with your Face Landmarker app for android.
I am very inexperienced with Koltin, so my apologies. I would like to display only certain landmarks on the face when running the app and I'm not sure where to start. For example, if I only wanted to show the left eye landmarks on the app, how would I do that?
In python, I would find the variable storing the facelandmark coordinates, and I would index it by these values to get the left eyelandmarks: [85, 246, 161, 160, 159, 158, 157, 173, 155, 154, 153, 145, 144, 163]. How would I do that in this koltin project? What variable contains the landmarks and how would I index it like this?
It looks like OverlayView.kt setResults function might be the place to start, but I'm really not sure. Any advise would be helpful. Thank you!
https://github.com/googlesamples/mediapipe/tree/main/examples/face_landmarker/android

Yup the OverlayView setResults function is where the landmarks to display are set! :) You can add a filter before sending those results to limit it to only the landmarks that you want.

Thank you for your reply. That is really helpful. Can you critique this specific section? My approach is to modify setResults() to:

create index of the only landmarks I want to keep
filter results by these indices
rewrite results with the landmark data for only the previously specified indices

`    fun setResults(
faceLandmarkerResults: FaceLandmarkerResult,
imageHeight: Int,
imageWidth: Int,
runningMode: RunningMode = RunningMode.IMAGE
) {
results = faceLandmarkerResults
val indexList = listOf(85, 246, 161, 160, 159, 158, 157, 173, 155, 154, 153, 145, 144, 163)
val filteredNumbers = results.filterIndexed { index, value ->
index in indexList
}
results = filteredNumbers
this.imageHeight = imageHeight
this.imageWidth = imageWidth
    scaleFactor = when (runningMode) {
        RunningMode.IMAGE,
        RunningMode.VIDEO -> {
            min(width * 1f / imageWidth, height * 1f / imageHeight)
        }
        RunningMode.LIVE_STREAM -> {
            // PreviewView is in FILL_START mode. So we need to scale up the
            // landmarks to match with the size that the captured images will be
            // displayed.
            max(width * 1f / imageWidth, height * 1f / imageHeight)
        }
    }
    invalidate()
}`

However, I keep getting the error that the code cannot infer the type for this parameter (point to index and value). How should can I properly type these parameters, or, is there another way I should do this?
Thank you!
The full error is below.
Unresolved reference. None of the following candidates is applicable because of receiver type mismatch:public inline fun <T> Array<out TypeVariable(T)>.filterIndexed(predicate: (index: Int, TypeVariable(T)) -> Boolean): List<TypeVariable(T)> defined in kotlin.collectionspublic inline fun BooleanArray.filterIndexed(predicate: (index: Int, Boolean) -> Boolean): List<Boolean> defined in kotlin.collectionspublic inline fun ByteArray.filterIndexed(predicate: (index: Int, Byte) -> Boolean): List<Byte> defined in kotlin.collectionspublic inline fun CharArray.filterIndexed(predicate: (index: Int, Char) -> Boolean): List<Char> defined in kotlin.collectionspublic inline fun CharSequence.filterIndexed(predicate: (index: Int, Char) -> Boolean): CharSequence defined in kotlin.textpublic inline fun DoubleArray.filterIndexed(predicate: (index: Int, Double) -> Boolean): List<Double> defined in kotlin.collectionspublic inline fun FloatArray.filterIndexed(predicate: (index: Int, Float) -> Boolean): List<Float> defined in kotlin.collectionspublic inline fun IntArray.filterIndexed(predicate: (index: Int, Int) -> Boolean): List<Int> defined in kotlin.collectionspublic inline fun LongArray.filterIndexed(predicate: (index: Int, Long) -> Boolean): List<Long> defined in kotlin.collectionspublic inline fun ShortArray.filterIndexed(predicate: (index: Int, Short) -> Boolean): List<Short> defined in kotlin.collectionspublic inline fun String.filterIndexed(predicate: (index: Int, Char) -> Boolean): String defined in kotlin.textpublic inline fun UByteArray.filterIndexed(predicate: (index: Int, UByte) -> Boolean): List<UByte> defined in kotlin.collectionspublic inline fun UIntArray.filterIndexed(predicate: (index: Int, UInt) -> Boolean): List<UInt> defined in kotlin.collectionspublic inline fun ULongArray.filterIndexed(predicate: (index: Int, ULong) -> Boolean): List<ULong> defined in kotlin.collectionspublic inline fun UShortArray.filterIndexed(predicate: (index: Int, UShort) -> Boolean): List<UShort> defined in kotlin.collectionspublic inline fun <T> Iterable<TypeVariable(T)>.filterIndexed(predicate: (index: Int, TypeVariable(T)) -> Boolean): List<TypeVariable(T)> defined in kotlin.collectionspublic fun <T> Sequence<TypeVariable(T)>.filterIndexed(predicate: (index: Int, TypeVariable(T)) -> Boolean): Sequence<TypeVariable(T)> defined in kotlin.sequences


EventListener not fire

the event listener seem not firing. Any suggestions?

@Ogek what was the mistake and how did you fix it ?


Event code

This is the bare outline of the post loops for the events.

I've closed this one, because there's newer stuff to be shared.


I found this: http://opensourcedesign.net/ — should we try to contact someone there?

http://opensourcedesign.net/
Also, they have a resources section, would we be better off trying to add or contribute or help organize their resources?

There are also these:
http://opendesign.foundation/resources/
How are we not just duplicating/further replicating the same lists? How are we really helping with all this?

Oh, and FSF tries to keep a list of everything...
https://directory.fsf.org/wiki/Main_Page


HTML renderer should export the markdown renderer

I think we discussed this, and some of the plumbing is there, but the processMarkdown function is not exposed to the main API surface.

@danmarshall Can you refresh my memory - what is the ask exactly? Is it to have a way to invoke the exact same markdown implementation as the renderer from the host? If so, I was actually thinking of doing it differently: I wanted to strip markdown processing out of the renderer and instead provide an event that would let the host do it. This way, the host can use whichever library it wants. Thoughts?

This way, the host can use whichever library it wants

That pretty much sums it up :-) So the event-based way would be fine.

In the event, can you provide the element context? This way we could do selective rendering. The use case in WebChat is that we do not want to support markdown in every element (to maintain consistency with other chat clients)

Not sure what you're asking. If a string contains markdown markup, then not processing it will result in bad stuff. And there is no way to distinguish between two TextBlock elements, so how would you make the decision to process some but not others?

Also, on second thought, I think it is preferable that the renderer has full control over how it processes markdown. So as you originally asked, I will instead export processMarkdown.

We should ensure every platform supports this extensibility. Please keep this issue open to track


Add support for condition-based scheduling

mdf is updated to match the latest spec
EvaluableGraphWithConditions is added to run a Graph with conditions using the psyneulink scheduler and conditions underneath
a simple three-node example abc_conditions is added to demonstrate usage of conditions
README is updated to latest spec and show a conditions snippet from an actually working example (real-time conditions are of unknown completion date)

@kmantel Which branch of Psyneulink does this require? A recent PR here sets pytorch>=1.8.0, but Psyneulink seems to need <1.8.0..?
Is it possible to have a dedicated MDF branch on Psyneulink, that we can install from, that we know will always have the latest changes, until they're all stable in master?

The <1.8 isn't a true requirement, but is restricted for management by dependabot. Until the >=1.8.0 requirement here, there wasn't an issue. That restriction will be lifted shortly, but for now, there is a branch mdf that will mostly track the latest development branch. We can install from that branch on github ci instead of a requirement for now (I'm not sure if this can be done for travis also)

@kmantel I have a branch here: https://github.com/ModECI/MDF/tree/condition_test I was using to test this. Works locally but some isues with install of PNL on CI, see https://github.com/ModECI/MDF/actions/runs/790280283. Does that branch of PNL support py3.9?

@pgleeson There are some issues with 3.9 at the moment, one of which is from the winpty on windows. We discussed this today, but there may be some others. With that branch specifically, the GPy install step should go before PNL. That step can be removed when they update their pypi package.

The Travis runs are failing because PsyNeuLink is not installed (it is not a dependency in setup.cfg). It looks like the github actions workflow has been modified to install psyneulink:
python -m pip install git+https://github.com/PrincetonUniversity/PsyNeuLink.git@mdf
This needs to be added in Travis as well.
PsyNeuLink seems to be working on Windows Python 3.9 now, I think it was a missing wheel for pywinpty that has since been built.