[FEATURE] Installation instructions

Duplicate issue

[X] I've searched for any related issues and avoided creating a duplicate issue.

Description
Working through the installation instructions, it's not 100% clear which config files I should be copying/editing/using.
I've copied the config-template.json file in /src/assets into a config.json file, with a change in the URL for the OCPP server from localhost to the URL of an actual OCPP server. Should I also be copying or changing other template files? It's not clear.
When I try to npm start after doing that (windows 10), I get the following outputs.
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
PS D:\hughs\e-mobility-charging-stations-simulator-main> npm start
npm WARN config global --global, --local are deprecated. Use --location=global instead.

<EMAIL_ADDRESS>prestart
npm run build

npm WARN config global --global, --local are deprecated. Use --location=global instead.

<EMAIL_ADDRESS>build
npm run rollup

npm WARN config global --global, --local are deprecated. Use --location=global instead.

<EMAIL_ADDRESS>rollup
rollup --config

'rollup' is not recognized as an internal or external command,
operable program or batch file.
PS D:\hughs\e-mobility-charging-stations-simulator-main>
Attachments
No response

You first have to run npm install. It's not documented because it's a implicit when you want to run node.js code.

npm install

I try to follow the exact instructions. Jumping from language to language means that I don't know what should be implicit.

A PR to enhance the README will be welcome

I'm very reluctant to do PRs on well run projects.
But another question on the readme. Should the "npm start:server" instruction be "npm run start:server"
H


mouseMove ignoring offset

Bug report

Node Version: 10+
Protractor Version: 5.3.2
Angular Version: 1.6+
Browser(s): chrome
Operating System and Version Ubuntu

I got the following code
`
    const centerX = dimensions.width / 2;
    const centerY = dimensions.height / 2;

    return browser.actions()
        .mouseMove(startingPoint, {
            x: centerX + x, y: centerY - y
        })
        .click()
        .perform();

`
About 3 months ago this was working just fine. We recently did an update and it seems that the offset is completely ignored alltogether.  The offsets are valid numbers. I having a hard time trying to understand where the change is happening but overall this is an issue that is now blocking our testing of a map where we draw squares. Do we know why the offset would be ignored?
On another note I tried to just give it an offset
`
    const centerX = dimensions.width / 2;
    const centerY = dimensions.height / 2;

    return browser.actions()
        .mouseMove({
            x: centerX + x, y: centerY - y
        })
        .click()
        .perform();

`
The above yields an error WebDriverError: unknown error: at least an element or offset should be set. I am a bit confused as I did pass an offset.
The last thing I will say is that the doc urls on your official sites are broken for alot of things. You can start by looking at the actions page for your API and try clicking anything.

I also see this example. https://github.com/angular/protractor/blob/ed955e56a839d7f69da43acb6755763220d3681d/lib/selenium-webdriver/webdriver.js#L69

Ok After many hours of trying the answer was to ensure they are int. please update the docs to ensure that people know they cannot use floats.

pbrain19, thanks! I've spent hour trying to figure out what's the problem, it was working and then not working, and then working again, depending on location.

I also faced this issue with coordinates, then find the mistake i was doing:
In chrome.conf.js i was doing w3c: false under chromeOptions:
I removed that property from chrome.conf.js then it start working fine for me.


nga-card 100% height

I'm trying to have an nga-card take as much space as possible, but it's not working :(
How can I achieve this?

The ba-card will naturally be set to fill the entire width of the container, but you can mess with the default height of the card, by going into /theme/sass/bootstrap-overrides/_card.scss and messing with the pre-sets. Though this will affect the card height across your entire application.

.my-container {
display: block;
width: 100%;
height: 100vh;
}
Add class="my-container" to your nga-card tag.

I believe @zbagley solution should work, closing now.


ec2.images and ec2.snapshots don't return all results

ec2.images are limited to self-owned images only, because otherwise, the query would never finish. ec2.snapshots has a default limit of 100 results set for the same reason.
To address this, it is required to:

add predicate pushdown to allow fetching a limited number of results
add limit and topN pushdown
return an Iterator instead of a List of rows, since it's lazy, to allow to cancel queries; same as in https://github.com/nineinchnick/trino-rest/blob/master/trino-rest-github/src/main/java/pl/net/was/rest/github/GithubRest.java#L1905

Predicate pushdown support has been added in 47e8d0e030cb65b03ee0c837ff962512c8262159


Consider generating Kotlin files for Kotlin projects.

Very long term goal, but there's probably a lot that we can do here.
http://www.reddit.com/r/java/comments/4an7xb/kotlin_a_new_jvm_language_you_should_try/d12tf2z

I'm the one who wrote that comment!
It sort of works out of the box on Kotlin, actually. There are a few snags. I can't make a Mapper instance, for example, because Kotlin doesn't like the protected constructor (even though I'm making it as a companion object to a subclass of that model).
The ColumnAdapter interface also doesn't seem to be present when the plugin is applied to a Kotlin project. That one took me a little longer to notice because I wasn't using custom classes in my .sq files initially.
Generating Kotlin files sounds great (though probably annoying, given that you guys already have JavaPoet for doing these things), but I think that the generated Java classes could also be adapted slightly to work in Kotlin.

Thanks for the clarifications.
...I'm surprised it works at all since the Kotlin plugin can't see generated sources from Gradle plugins (#80).

@kevinmost on reddit you've mentioned that you use a pure java module. I tried it and can not get it to work, because generated files are not being put on the class path by the 'java' plugin. But they are when using 'android.library' or 'android' plugins.
I fail to find a way to correctly supply a generated path to 'java' plugin, do you have a hint maybe? Or did you use 'android.library' too? I'd rather use 'java' because I don't need any android features - want to have a simple models-only java module while whole app lives in 'kotlin'-enabled module.

@dimsuz Sorry for the ambiguity! Yes, it is an Android library plugin, in actuality. I don't think a pure Java module would work, actually, because you need to reference classes like android.database.Cursor in your Entity class.

@kevinmost oh, right! Thanks for clarification. Also I didn't figure out how to use generated classes from Kotlin. I'd wish I could use data class instead of AutoValue, but having the problems with Mapper which you described above. Kotlin compiler says it is unable to call constructor with supplied arguments (which I seem to get right) - I guess this is because, as you say, Mapper is generated to be package local.
So in the end I went with using AutoValue for now, although I feel it's not strictly needed for Kotlin.

This may or may not be a problem depending on the goals for the API for consuming code, but the static constants in the generated interface are not inherited by the model, so you must call e.g. HockerPlayerModel.TABLE_NAME in Kotlin, while you can use HockeyPlayer.TABLE_NAME in Java.
Not sure if this can be worked around while only Java code is generated.

working-kotlin branch tracking this


Invariant Violation: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.  Check the render method of App.

Where's the problem?

Project Files
https://github.com/spencercarli/getting-started-react-navigation

router.js - Update React-Navigation v3
import React from 'react';
import { createBottomTabNavigator, createStackNavigator } from 'react-navigation';
import { Icon } from 'react-native-elements';

import Feed from '../screens/Feed';
import Settings from '../screens/Settings';
import UserDetail from '../screens/UserDetail';
import Me from '../screens/Me';

const FeedStack = createStackNavigator({
Feed: {
screen: Feed,
navigationOptions: {
title: 'Feed',
},
},
Details: {
screen: UserDetail,
navigationOptions: ({ navigation }) => ({
title: ${navigation.state.params.name.first.toUpperCase()} ${navigation.state.params.name.last.toUpperCase()},
}),
},
});

export const Tabs = createBottomTabNavigator({
Feed: {
screen: FeedStack,
navigationOptions: {
tabBarLabel: 'Feed',
tabBarIcon: ({ tintColor }) => ,
},
},
Me: {
screen: Me,
navigationOptions: {
tabBarLabel: 'Me',
tabBarIcon: ({ tintColor }) =>
},
},
});

const SettingsStack = createStackNavigator({
Settings: {
screen: Settings,
navigationOptions: {
title: 'Settings',
},
},
});

const Root = createStackNavigator({
Tabs: {
screen: Tabs,
},
Settings: {
screen: SettingsStack,
},
}, {
mode: 'modal',
headerMode: 'none',
});

index.js
import React, { Component } from 'react';
import { Root, Tabs } from './config/router';

export class App extends Component {
  render() {
    return <Root />;
  }
}

export default App;

software
version

react-navigation
^3.0.0"

react-native
0.59.3

node
10.15.3

npm or yarn
1.13.0

@akisplays You are using Root component but it is actually undefined because it is not exported from your router.js file


Include estimate of bandwidth leaving a data centre

Feature Suggestion
Currently the methodology says:

We also have chosen to ignore traffic that leaves a data center to provide end-users with services, because this traffic is usually handled by CDN providers

In my use case, that traffic goes straight to end users, and not to a CDN. Would it be possible to include the end-user traffic using a configuration option please? The option could be disabled (i.e. ignore traffic to end-users) by default.
It would be even more helpful if the bar for (say) S3 in the ‘Emissions Breakdown’ could be split into several stacked sub-bars, one for storage emissions, one for bandwidth emissions to other data centres, one for bandwidth emissions to end users, etc. That would give users of CCF a bit more guidance about which where the main contributions to emissions for each service come from. But that could be a separate feature request (and I’m happy to split it out into a separate issue if that would be more helpful). Thanks.

Hey @pwithnall -- re: networking usage from cloud providers to end users, this is definitely something that should be possible, but the main blocker from us doing this so far is that  whatever co-efficient we use for kilowatt hours per Gigabyte transferred is dependent on the type of end user device (phone, tablet, laptop, etc) and the version (old, new, etc). Given that we don't know the type of user devices from the cloud provider APIs, it's a little challenging to settle on a co-efficient. So it's less about there being a CDN in your stack or not, and more about knowing the types of devices end users are using to request your data. That said, it's definitely possible to implement - we'd just need to do some research to settle on a reasonable coefficient that would make sense for the majority of users of the software as a default.
Re: your 2nd request, it would be good to separate the request for additional breakdowns of the emissions data based on the type of usage considered when estimating energy/CO2e, if you're able to. If you can include some example use cases and possibly a mockup/image of the stacked bar chart you mention, even better! Thank you!

Hey @pwithnall -- re: networking usage from cloud providers to end users, this is definitely something that should be possible, but the main blocker from us doing this so far is that whatever co-efficient we use for kilowatt hours per Gigabyte transferred is dependent on the type of end user device (phone, tablet, laptop, etc) and the version (old, new, etc).

Are you planning on including the power consumption of the end user device in the CCF figures, then, or are you thinking of just using this as a proxy for the type of network infrastructure the traffic goes over (fibre, ethernet, satellite, Wi-Fi, 4G, etc.) which, aiui, has a significant impact on the network emissions?
I suggest that including the power consumption of the end user device itself could easily lead to double counting of emissions (depending on where people are drawing their system boundaries) and, as you’ve said, it introduces a significant number of additional dimensions to model.

Given that we don't know the type of user devices from the cloud provider APIs, it's a little challenging to settle on a co-efficient. So it's less about there being a CDN in your stack or not, and more about knowing the types of devices end users are using to request your data. That said, it's definitely possible to implement - we'd just need to do some research to settle on a reasonable coefficient that would make sense for the majority of users of the software as a default.

That makes sense. I assume you’d pick a general-purpose default, but allow the coefficients to be customised in the configuration, since site owners are likely to know the rough breakdown of types of network infrastructure their users use, either through having an idea of their target audience, or through web analytics?

Re: your 2nd request, it would be good to separate the request for additional breakdowns of the emissions data based on the type of usage considered when estimating energy/CO2e, if you're able to. If you can include some example use cases and possibly a mockup/image of the stacked bar chart you mention, even better! Thank you!

Opened #354 :slightly_smiling_face:

Hey @pwithnall -- I seem to have a pattern of apologizing for taking a while to get back to you. Sorry for my tardiness.
You are correct -- it's the "type of network infrastructure the traffic goes over" that would impact the chosen co-efficient. Maybe the user device could affect it too, but it's much more a factor of the network infrastructure (fibre, Wi-Fi, etc). I'm guessing the coefficient could vary a fair bit by the location of the data center (known) and the location of end user (maybe known, but likely not via the cloud provider APIs).
So yes, we should add the ability to include traffic leaving the data center, we just need to do a bit of research to settle on a co-efficient, or multiple co-efficients for different regions.

Not sure where to put it but I wanted to suggest a little (uncomplete) litterature review, if that helps : https://github.com/cloud-carbon-footprint/cloud-carbon-footprint/issues/379#issuecomment-1291612358


When a person is selected the Map is not showing up

When a person is selected the Map is not showing up

https://github.com/matt8707/ha-fusion/blob/main/static/documentation/Map.md


Type Annotations

Is it possible to add type annotations to Actors functions in Python SDK:
async def set_my_data(self, data: object) -> None: #from this
async def set_my_data(self, data: MoreSpecificClassName) -> None: #to this
########
await proxy.SetMyData({'data': 'new_data'}) #from this
await proxy.SetMyData(MoreSpecificClassName(data: 'new_data')) #to this

And will VS Code (or any other editor) intellisense work with it?

I think you can, the type annotations have no effect at runtime.

@omidb yes you can definitely use any types you want for the actor interfaces and then implementation in your actor.


fix(canary): make default provider optional and remove default type

Description

depends on https://github.com/armory-io/deploy-engine/pull/417
Motivation and Context

How has this been tested? How can a reviewer test these changes?

Have your performed the following tests?
Please confirm you have done the following tests to ensure your pull request is ready to be merged..

[x] If this change modifies the deployment flow have triggered a deployment using this branch (i.e. build/dist/darwin_amd64/armory deploy start -f <path to file>)?
[x] If this change modifes the login flow have you tried logging in and out with this branch (i.e. build/dist/darwin_amd64/armory login) and verified the credentials work?
[x] Have you verified the specific change made in this PR?

Over all looks good with some comments about error message and field names.


Configurable Clone

Yossi requested the ability to configure what elements of a workspace are carried over on a clone. This is most relevant to him because data and method configs cannot be deleted (currently).
Specifically, he wants the following options:
[ ] clone data
[ ] clone configurations
[ ] clone attributes
[ ] clone notebooks
These should all be checked by default. These options should be a detail on demand. This behavior will require backend work

Keep this in new issues for now. It's kind of an icebox thing


TypeError in /tsm-editor/

Error in TSM Editor
TypeError in /tsm-editor/
e is null
View on Bugsnag
Stacktrace
src/components/editor/Editor.vue:172 - onEditorCaretPositionChange
src/components/editor/EditorEvent.vue:78 - onInput/<

View full stacktrace
Created automatically via Bugsnag

An error linked to this issue has been reopened in Bugsnag
TypeError in /tsm-editor/

An error linked to this issue has been marked as fixed in Bugsnag
TypeError in /tsm-editor/

An error linked to this issue has been marked as fixed in Bugsnag
TypeError in /tsm-editor/


About Colorgrade and the all-body colour effect of the silhouette

Request Description
As the title: About Colorgrade and the all-body colour effect of the silhouette, they‘s effect see to a bit conflict
So I want to propose a function:
can through give in-SkinConfig.yaml's "ColorGrade" a "Special-Value", to get Silhouette's that effect
(Assuming that Special-Value is "HairColors", then it's usage be like "ColorGrade: HairColors")
Target Date
No response

If you agrees, and thinks that the Special-Value should can be other values, you can also say that idea


Add support for layer 3 VPC to collectors.nxos_get_port_channel_data & collectors.get_vpc_state

In progress:

Changed 'nxos_get_vpc_state' to use the words before ':' as the column header

Need to test to see if Pandas inserts it correctly, since the column headers will have spaces

Next:

Change 'nxos_get_port_channel_data' to use JSON for the output

This output is easier to parse when returned with JSON
There is a possibility that some devices will be so old that JSON isn't supported, but we can cross that bridge when we come to it

Verified that Pandas can read and write to the table as long as the column names are surrounded by quotation marks. Need to test other functions to make sure that it doesn't break them, although I can't imagine why it would.


Enable Assertions

Addition of the enabling of enableAssertions to true in the build.gradle file.

Codecov Report

Merging #115 (a0a7c7b) into master (d286a64) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #115      +/-   ##
============================================
- Coverage     61.75%   61.71%   -0.04%
  Complexity      462      462
============================================
  Files            87       87
  Lines          1668     1669       +1
  Branches        185      186       +1
============================================
  Hits           1030     1030
- Misses          574      575       +1
  Partials         64       64

Impacted Files
Coverage Δ

.../seedu/address/logic/parser/AddressBookParser.java
50.00% <0.00%> (-1.52%)
:arrow_down:

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 d286a64...a0a7c7b. Read the comment docs.


Where and how to get the certificate for AWS KMS? So, I can enable the verifySSL all the time.

What platform/OS are you using?
Centos 7.7
Which version of the SDK?
aws-cpp-sdk 1.7.252

I have read the cpp-sdk. It said that verifySSL should be true in production scenario.
However, when I set verifySSL to true, then I get error to connect with KMS.
So, I just want to know which certificate I should use for communicating with AWS KMS.

openssl s_client -connect kms.us-west-1.amazonaws.com:443.
I have use above command to verify the connection. And it seems the verification is passed.
But when I use verifySSL=true, then I got error to connect with KMS.

I have tried to set ca_path to "/etc/ssl/certs/", it still not working.

Thanks,

Will investigate more on this. close this for now.


Enable rummager message queue in production

This commit enables the Rummager message queue consumer in production and staging.
Trello: https://trello.com/c/lRnxI2x5

:+1: Merge when green.


fix(ci): renamed labeler config file name with .yml.

Issue link
Relates: #14
Changes in this PR

Renamed .github/labeler.yaml to .github/labeler.yml since job would be failed to detect .yml file

Changes not included in this PR
N/A

Note that labeler could accept input with configuration-path, which is specified in the file under .github/workflows, but we would simply change config filename in this PR.


Footer missing

Absence of a footer in the codebox.

Thanks for your contribution Raghav, but currently wasn't looking for a footer. Please make sure to lookup CONTRIBUTING.md before making PRs so your hard work doesn't get wasted.


ISPN-15400 Support collection parameter serialization

https://issues.redhat.com/browse/ISPN-15400
Draft => I want to test more! done

We need a test in the integration module that uses the container as well

Thank you for the review. I added it

I added a commit to fix the infinispan-client-hotrod-graalvm tests.

Thanks @fax4ever

Thank you for the review @ryanemerson


[BUG] Cannot edit an application that has a credential/identity assigned

Is there an existing issue for this?

[X] I have searched the existing issues

Konveyor version
0.5-beta.1
Priority
Critical
Current Behavior
If an application has a source credentials or a maven settings assigned, the application cannot be edited.
The edit modal opens and changes can be made, but the Save action will fail with an error 400.
Expected Behavior
Save should complete successfully.
How Reproducible
Always (Default)
Steps To Reproduce

Create an application
Create a credential
Manage the credentials for the application created in 1 and select the credential created in 2
Edit the application
Make a change anywhere and click 'Save'

Environment
Any

Anything else?
No response

This is a regression from #1985


Support span tags with empty values

PR: https://github.com/wavefrontHQ/wavefront-sdk-python/pull/41

Move to: https://github.com/wavefrontHQ/wavefront-opentracing-sdk-python/issues/27


Test coverage is better, ish

Before

### After (Auth has tests, but is disabled because auth is unfinished atm)


support mac os build documentation

hello,
i was trying to build ecal for mac os , considering support to build is there. however i ended up solving build errors and couldn't succeed.
could there be steps how to build for mac and minimal hardware and software requirements?

Hey Rakesh,
Take a look at our CI build script here:
https://github.com/eclipse-ecal/ecal/blob/master/.github/workflows/build-macos.yml
You don't really need Cap'n'Proto, just set -DHAS_CAPNPROTO=OFF in the cmake call.
Keep in mind that the MacOS Support is experimental. We are compiling on some MacOS x86_64 version on GitHub's CI (not even sure which one), so we know that it compiles. Runtime-wise it is pretty much untested, as Apple forces developers to buy proprietary hardware. And we don't have money for that at the moment ;)
Keep us updated on what you had to do to get it working 😊
Regards
Florian


TRY IT YOURSELF sections of the book doesn't render well using the online reader

Hello I have buyed the kindle version of the book because I wanted to work with the editor and the book on the same machine but sadly the TRY IT YOURSELF sections does not render well on the cloud reader

This doesn't happend on the android app so I have bean working with my phone and the pc but this is not the experience I wanted to have in the first place.
Enjoying the book so far (hour 5) but if you can fix this problem I will enjoy it even more :)

Hey! Thanks for the compliment, but I'm afraid I can't do much about this. As an author, I don't have control over the final product. The most I can do is try to reach them about this. It's better if you can complain directly to Amazon or to the publishing company (Sams Publishing / Pearson).

For instance, in the Amazon store page for the book there's a box below where you can report formatting issues:


Ability to add friends, view existing ones and see pending friend requests

Some commands to manage your friends and friends list from the bot would be super handy.
Something like:

friends to view a list of pending friend requests, and your existing friends and their online status
addfriend to send a friend request

Unfortunately this isn't a feature of discord.js due to user tokens being against discords ToS

So, soru tried implementing this and in the process of doing so her test account got a soft-lockdown (somehow can only accept/invite friends from the webbrowser, not the bot. It keeps asking for email verification codes) and when trying to make new test accounts they get a hard lockdown (it is asking for phone verification).
Is it worth implementing this? Soru thinks she has a working version atm but it may lock down your account, so she isn't sure if she should publish it

Added it behind an enabling wall. Readme is updated with instructions


Tetgen binary not available for github actions

The 3d meshing step cannot be tested if the tetgen binaries are not available.
Options:

Compile and host tetgen executables
Download source and compile in the CI

This bug would have been caught by the CI:

https://github.com/hpgem/nanomesh/issues/117

Added installation instructions for tetgen to the documentation.

Binaries available here:
https://github.com/hpgem/tetgen/releases/tag/1.6.0


Add GraphSpace dependencies to conda environment

Closes #30
I listed the new packages separately so that we can remove them later if we decide to Dockerize GraphSpace.
@annaritz do you know whether GraphSpace needs version 2.10.0 of the requests package specifically?  Only that version is allowed in https://github.com/adbharadwaj/graphspace-python/blob/42283556eb3a52212cfa9f75dfc694c70069add7/requirements.txt#L3.  That prevents us from installing it through conda.  It is so old it conflicts with other packages in our environment.  It may also be throwing warnings in our test code
test/OmicsIntegrator1/test_oi1.py::TestOmicsIntegrator1::test_oi1_required
  /usr/share/miniconda3/envs/spras/lib/python3.8/site-packages/requests/models.py:170: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated since Python 3.3, and in 3.10 it will stop working
    if isinstance(hook, collections.Callable):

I also split the monolithic test workflow into three jobs that can run in parallel.  Building the Docker images is tested independently from testing the conda environment and running the test code.  The conda environment job also runs in Linux, macOS, and Windows so we can confirm SPRAS can be installed on any platform.  The tests only run in Linux.

I forked my own version of GraphSpace that specifies a minimum version of requests instead of a specific version (https://github.com/adbharadwaj/graphspace-python/pull/37).  Now requests can be installed through conda and not uninstalled/reinstalled by pip.


Issue with k8s.io/docs/setup/production-environment/tools/kubeadm/create-cluster-kubeadm/

This is a Bug Report

Problem:
can not see section under "Installing a Pod network add-on" that has tabs of "Calico, cilium,conitv,kube-router" so I dont know the download links for
calico.yaml "kubectl apply -f https://docs.projectcalico.org/v3.11/manifests/calico.yaml"
Proposed Solution:
please add missing section back
correct contents are in https://v1-18.docs.kubernetes.io/docs/setup/production-environment/tools/kubeadm/create-cluster-kubeadm/
Page to Update:
https://kubernetes.io/...
https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/create-cluster-kubeadm/

latest 1.18

Issue is https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/create-cluster-kubeadm/ is the one used in exam and not all people will find the correct missing info

this was changed. see https://github.com/kubernetes/website/issues/22961#issue-673047152
/triage support
/sig cluster-lifecycle
/close


fix readme import

Hi, thank you for the great library.
I found an import statement in README is different from its code.

Thanks


[Accessibility] Date mask: VoiceOver doesn't read out the digit right before the separator

Problem with VoiceOver not reading out the digits followed by a forward slash.
Example 1

Example 2

Reproducible on both jsfiddle:

Vanilla JS component: https://jsfiddle.net/nosir/kbaxx64s/
React JS component: https://jsfiddle.net/nosir/gLLsrxxf/

@luish Any chance you were able to find a workaround for this? Still seems to be an issue in the latest version of Cleave.

@luish @rogorman9 I also faced the same issue but in Vue. I have a clean solution.
Use cleave programmatically.
Use a regular input and bind the state var directly to input's value. Then on the input event call the below function.
const cleavingFunction = (value) => { const myElem = document.getElementById('someid') const cleave = new Cleave(myElem, {   numeral: true,  ..... }) cleave.setRawValue(223) const cleavedVal = cleave.getFormattedValue() this.inputVal = cleavedVal }
This should work as expected.


Prisma error when creating new secret

The server is returning the following error when creating a new secret while I'm logged in (using latest version v5.15.4):

{"level":50,"time":1703876192116,"pid":18,"hostname":"secrets","reqId":"req-28","req":{"method":"POST","url":"/api/secret","hostname":"XXX","remoteAddress":"XXX","remotePort":48242},"res":{"statusCode":500},"err":{"type":"PrismaClientKnownRequestError","message":"\nInvalid prisma.secret.create() invocation:\n\n\nForeign key constraint failed on the field: foreign key","stack":"Error: \nInvalid prisma.secret.create() invocation:\n\n\nForeign key constraint failed on the field: foreign key\n    at fn.handleRequestError (/home/node/hemmelig/node_modules/@prisma/client/runtime/library.js:174:6477)\n    at fn.handleAndLogRequestError (/home/node/hemmelig/node_modules/@prisma/client/runtime/library.js:174:5907)\n    at fn.request (/home/node/hemmelig/node_modules/@prisma/client/runtime/library.js:174:5786)\n    at async t._request (/home/node/hemmelig/node_modules/@prisma/client/runtime/library.js:177:10477)\n    at async Object. (file:///home/node/hemmelig/src/server/controllers/secret.js:144:28)","code":"P2003","clientVersion":"4.12.0","meta":{"field_name":"foreign key"}},"msg":"\nInvalid prisma.secret.create() invocation:\n\n\nForeign key constraint failed on the field: foreign key"}

It seems that the issue is coming from this fragment:
https://github.com/HemmeligOrg/Hemmelig.app/blob/1df56be5143629bc1cdc6c01c3b32e876785b855/src/server/controllers/secret.js#L142
I tried printing the req?.user?.user_id and it was the current logged user's id:
e626fc45-8368-4d6f-9907-4671b09761bb
Any idea?
PS: I can make the PR myself, however, I'm not understanding the exact cause of this error.

Hm, strange error. Not sure why this is happening. What version did you come from?

To me, the database migration / or the database here is having an issue. Meaning it is not the application itself. Hard to verify without the database.

https://github.com/HemmeligOrg/Hemmelig.app/issues/246#issuecomment-1913275736


Add libxml2 2.9.12

@jsharpe The module file is missing from the source archive. You can add it with a patch.

This was generated with the add_module.py tooling; it should be updated to add the module file patch automatically.

@meteorcloudy I stopped iterating on this because I don't have a local windows box setup to iterate the tests and having to wait for BCR maintainer approver for buildkite each time was becoming tedious. Are there some conditions under which we can persist the approval between CI runs on the same PR?

Are there some conditions under which we can persist the approval between CI runs on the same PR?

I think if a new commit is pushed to the PR which was previously approved by a registry maintainer, and the commit didn't touch the presubmit.yml file, we could persist the approval. I'm not sure how feasible that is (and whether that actually reduces the churn in your case?)

Or maybe we can add a label to indicate a PR is sent by a trusted contributor, then we by pass the BCR reviewer approval for CI run?

GitHub actions has a nice default, where a contribution from someone with no commits on the project requires approval
https://docs.github.com/en/actions/managing-workflow-runs/approving-workflow-runs-from-public-forks
but once they've merged, they get the "contributor" status on the repo and approvals are no longer required for them.

@Wyverald @meteorcloudy what is the policy regarding having to have linux, macos and windows support working in presubmit.yml. I don't have bandwidth nor a windows setup to work out how to fix this on windows; can I just remove the windows platform from the test matrix?

Yes, you can leave a TODO in the presubmit to enable windows in the matrix, like https://github.com/bazelbuild/bazel-central-registry/blob/e7b901a1784b9f630bab512d0bfa197ed77100de/modules/aspect_rules_js/0.11.1/presubmit.yml#L4
However I'm concerned that no one will ever pick up the TODO, so any module that depends on libxml won't work on Windows either. I think we should show this sort of compatibility warning on the registry.bazel.build to make it discoverable, though we'd have to do some parsing to figure it out.

Yes, you can leave a TODO in the presubmit to enable windows in the matrix, like
https://github.com/bazelbuild/bazel-central-registry/blob/e7b901a1784b9f630bab512d0bfa197ed77100de/modules/aspect_rules_js/0.11.1/presubmit.yml#L4
However I'm concerned that no one will ever pick up the TODO, so any module that depends on libxml won't work on Windows either. I think we should show this sort of compatibility warning on the registry.bazel.build to make it discoverable, though we'd have to do some parsing to figure it out.

This is also a higher barrier to entry for module maintainers - they need to be fluent in building on all three major platforms in general to be able to publish high quality modules. I suspect that the number of people familiar with getting bazel builds working on windows is relatively low and may mean that generally windows becomes a second class citizen in the registry?

The problem here is that the BCR has accidentally become a C/C++ package manager, so we have contributors who are not the rule author nor the library author. libxml2 probably does work on Windows, but you're just the messenger and don't know why unistd.h isn't on the windows machine used for CI.
Windows is already a second class experience in Bazel but some policy here would be important to prevent that getting worse.

Becoming a C/C++ package manager probably wasn't an accident: If we don't at least centralize the effort to improve Windows support for Bazel projects, it will never become better.
Maybe some companies that use Bazel on Windows could contribute a bit of funding to motivate module maintainers to figure out Windows builds?

Hello! I'm doing some routine cleanup of stale PRs. If you're still working on this and planning to make progress soon, please let me know and we can re-open it.


Prepare website repository

@ahmedidrissi  created the Repo

set two teams for the repo
Devs : for all contributors
maintainers : for peoples who will verify pull requests before merging

Done


Update README.md: Added documentation for csv_validation.py

@yhenon Could you please take a look at this?


'enter' key or move with click doesn't work

I have the following configurations:

[ vnc4server  4.1.1+xorg4.3.0-37ubuntu5.0.2 ] VNC server and version
other vncservers installed on my system:
-->  xtightvncviewer 1.3.9-6.4ubuntu1
--> remmina-plugin-vnc  1.0.0-4ubuntu3
[ 0.12.0 ] vncdotool version
Run a python script consisting vncdo commands as

sudo python3 py-scripts/automation.py xxx.xxx.xx.xxx  password testuser testpass
for script:
`#!/usr/bin/python
import sys
import os
saddress = sys.argv[1]
spassword =  sys.argv[2]
susername =  sys.argv[3]
spasw = sys.argv[4]
os.system('vncserver -kill :1')
os.system('sudo vncdo -v -s  ' + saddress + ' -p ' + spassword + ' --delay=1500 ' +
' type ' + susername +
' key tab type ' + spasw +
' move 1045 525 click 1 click 2 click 3 key enter key enter pause 60 capture mousepointer.jpg '
)
os.system('vncserver -kill :1')`
In the above script, I tried to login on a Mac 10.x but even after providing enter key or click event on submit button,It still stuck there.

version 0.12.0 is too old; please give the version in git branch main a try as it supports macOS since #200:

pip install git+https://github.com/sibson/vncdotool.git@main

If this still does not work feel free to re-open this issue or to file a new issue.


Add field descriptions for demographic data?

Add a "horizontal" row lined up with the data that gives the definition of the field.  Would need to be comma/special-char escaped.

Another option is to create a separate, stand-alone data dictionary file specifically for the demographics data. This file could be chock full of details that we don't have room for on the webpage or in the CSV with the data itself.
Right next to the "Download as CSV" button, we could have a "Download data dictionary" button which would serve this file.


initialize config with  LocalSearchConfig and GlobalSearchConfig

Description
initialize config with  LocalSearchConfig and GlobalSearchConfig
Proposed Changes
settings.yaml: graphrag/index/init_content.py
Checklist

[X] I have tested these changes locally.

[X] I have reviewed the code changes.

[X] I have added appropriate unit tests (if applicable).

tests.unit.config.test_default_config.TestDefaultConfig.test_create_parameters_from_env_vars

Moving branch to graphrag repo to perform tests! Thanks!

Added to #533

Added to #533

I have closed this PR.


apps: separate capacity alert limits for pv

What this PR does / why we need it:
This PR separates limit configurations for node disk and PVC capacity alerts, as well as allowing PVC capacity alerts to be disabled.
Reason: When using NFS and the nfs-provisioner for provisioning Persistent Volumes, the metrics used for measuring PVCs shows the total capacity & usage of the entire NFS volume instead of the capacity of each PVC. As a result, we'd like to be able to turn off PVC capacity notifications and handle them individually.

Which issue this PR fixes: fixes #
Public facing documentation PR (if applicable)

Special notes for reviewer:
Add a screenshot or an example to illustrate the proposed solution:
Checklist:

[X] Added relevant notes to WIP-CHANGELOG.md
[X] Proper commit message prefix on all commits
[ ] Updated the public facing documentation
Metrics checks:

[X] The metrics are still exposed and present in Grafana after the change
[X] The metrics names didn't change (Grafana dashboards and Prometheus alerts are not affected)
[X] The metrics names did change (Grafana dashboards and Prometheus alerts were fixed)

Logs checks:

[X] The pods logs do not show any errors

Network Policy checks:

[X] The NetworkPolicy Dashboard doesn't show any dropped packages

Gatekeeper PSPs checks:

[X] Gatekeeper PSPs do not block the pods from starting after the change

Falco checks:

[X] No Falco alerts are generated by the change

Is this changeset backwards compatible for existing clusters? Applying:

[X] Is completely transparent, will not impact the workload in any way.
[X] Requires running a migration script.
[ ] Will create noticeable cluster degradation.
E.g. logs or metrics are not being collected or Kubernetes API server
will not be responding while upgrading.
[ ] Will change any APIs.
E.g. removes or changes any CK8S config options or Kubernetes APIs.

Chart checklist (pick exactly one):

[X] I upgraded no Chart.
[ ] I upgraded a Chart and determined that no migration steps are needed.
[ ] I upgraded a Chart and added migration steps.

Pipeline config (if applicable)
If you change some config options (e.g. add/rename variable or change the default value) you may need to update the config used by the pipeline in pipeline/config.

@lunkan93 Added a migration script and updated the migration README to run the script manually.


Add Lisp introduction for the book, part 1

Hello @mattwparas,
First of all, I'd like to thank you for your efforts in making embedded Lisp for Rust, as well as advocating for it through the actual implementation of a plugin system for Helix.
I've been observing responses from individuals who are resistant to the idea of Lisp as an embedded language for the editor, and in response I've taken it upon myself to write an introductory Lisp chapter for the book. My hope is that this will help broaden the reach of the project and make it more approachable.
While my work is not yet finished, I wanted to share my progress early. If this idea resonates with you, I am eager to continue my efforts.

Thanks for the PR - I will take a look in the next few days

Would it be possible to make the distinction between Scheme and Lisp? I don't mind referring to steel as a member of the lisp family (since it is) but in general its origin bears much closer resemblance to scheme, and I'd like that to be reflected in the documentation

hey, I tried to frame it more as a descendant of Scheme

Just the one nit, otherwise I think this looks okay - did you want to get this merged as is? Or continue adding things?

Just the one nit, otherwise I think this looks okay - did you want to get this merged as is? Or continue adding things?

Thanks for the review, I planned to do more PRs to make them more manageable as I'm not quite sure where is the point when I find it "done", so this one could be merged


ci: add npm cache

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


解决UnicodeEncodeError  'gbk' codec can't encode character

从谷歌浏览器下载一个har运行后报：
UnicodeEncodeError: 'gbk' codec can't encode character '\x86' in position 2: illegal multibyte sequence 的错误
相关解决方案：
http://m.jb51.net/article/64816.htm

Coverage decreased (-2.06%) to 71.134% when pulling 0af374362752fd551bfec34e7d9fb35f68958f62 on wushenghang:master into c51f6390499f540d0a0e6afeba916e147069fcf9 on HttpRunner:master.


Extension causes high cpu load

Issue Type: Performance
Extension Name: vscode-graphql
Extension Version: 0.1.7
OS Version: Darwin x64 18.5.0
VSCode version: 1.33.1

:warning: Make sure to attach this file from your home-directory:
:warning:/Users/john.mica/Prisma.vscode-graphql-unresponsive.cpuprofile.txt
Find more details here: https://github.com/Microsoft/vscode/wiki/Explain:-extension-causes-high-cpu-load

cannot seem to find the actual file required for upload


What hardware do you need for the experiment?

What is your experimental machine ? The number of GPUs, memory size of GPUs, CPU and its memory size. Hoping for solution~

We use one machine with 8 TITAN X GPUs (12GB Memory).

When I run the code, the "model" folder is always empty . I want to know what went wrong?


Revert changes related to spdx package component

Issue number #798
As per recent discussion with @sebasgomez238 on sbom-tool Repository, It is confirmed that We don't want to display information from within the SPDX file. SBOMs are representative of an individual piece of software. We just list the presence of an SPDX file in the externalDocumentRefs section. No additional details are required. Therefore, spdx related changes should be removed.
Reference PR -> https://github.com/microsoft/sbom-tool/pull/319

Closing it base on comment on the issue


fix: Remove boolean NotImplemented usage

NotImplemented is not falsy, and using in a boolean context is also deprecated:
DeprecationWarning: NotImplemented should not be used in a boolean context

Issue #, if available:
Description of changes:
Testing done:
Merge Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.
General

[x] I have read the CONTRIBUTING doc
[x] I used the commit message format described in CONTRIBUTING
[x] I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

[x] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[x] I have checked that my tests are not configured for a specific region or account (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

nit: do we prefer this to raising NotImplemented? I think so, but just checking

Yeah, because NotImplemented actually acts as a "truthy" value, which leads to stuff like
>>> if Observable(...) == "I'm not an observable":
...     print("equal")
...
equal

nit: do we prefer this to raising NotImplemented? I think so, but just checking

Yeah, because NotImplemented actually acts as a "truthy" value, which leads to stuff like
>>> if Observable(...) == "I'm not an observable":
...     print("equal")
...
equal

I'm talking about raising the exception, not returning it

Closing PR since there has been no activity on this for a while. Please feel free to reopen if you wish to continue working on this. Thanks!


How can I create an instance using only ephemeral storage? (No EBS volumes)

I tried looking at the source, but it's not obvious to me how to achieve this.

Nevermind me. This is not something that AWS allows. I could have sworn I've seen it, tho.
Sorry for the noise! 👋


v0.11.0 Locally installed wasm-opt not found

🐛 Bug description
When I run the command wasm-pack build --target web, I get an error that says wasm-opt is not found, even though wasm-opt exists in PATH.
The situation is similar to #1062, but the version (and possibly the cause) is different, so I have started a new issue.
See also: https://github.com/rustwasm/wasm-pack/issues/1062#issuecomment-1477077150 https://github.com/rustwasm/wasm-pack/issues/1062#issuecomment-1477181213
🤔 Expected Behavior
The installed wasm-opt will be used
👟 Steps to reproduce

run cargo new --lib hello-wasm and implement a simple project hello-wasm following MDN tutorial

run wasm-pack build --target web and get the following error

> wasm-pack build --target web
[INFO]: Checking for the Wasm target...
[INFO]: Compiling to Wasm...
  Finished release [optimized] target(s) in 0.03s
[WARN]: :-) origin crate has no README
[INFO]: Installing wasm-bindgen...
[INFO]: found wasm-opt at "C:\\Users\\user\\scoop\\shims\\wasm-opt.exe"
Error: C:\Users\user\scoop\shims\bin/wasm-opt.exe binary does not exist
To disable `wasm-opt`, add `wasm-opt = false` to your package metadata in your `Cargo.toml`.
Caused by: C:\Users\user\scoop\shims\bin/wasm-opt.exe binary does not exist
To disable `wasm-opt`, add `wasm-opt = false` to your package metadata in your `Cargo.toml`.

🌍 Your environment
on Windows 10:
> rustc --version
rustc 1.68.0 (2c8cc3432 2023-03-06)

> wasm-pack --version
wasm-pack 0.11.0

> wasm-opt --version
wasm-opt version 112 (version_112)

on Fedora 37:
$ rustc --version
rustc 1.67.1 (d5a82bbd2 2023-02-07)

$ wasm-pack --version
wasm-pack 0.11.0

$ wasm-opt --version
wasm-opt version 110

ad hoc fix (still passes all cargo tests in my Windows 10 environment):
(src/wasm_opt.rs)
//! Support for downloading and executing `wasm-opt`

use crate::child;
use crate::install;
use crate::PBAR;
use anyhow::Result;
use binary_install::Cache;
use std::path::Path;
use std::process::Command;

/// Execute `wasm-opt` over wasm binaries found in `out_dir`, downloading if
/// necessary into `cache`. Passes `args` to each invocation of `wasm-opt`.
pub fn run(cache: &Cache, out_dir: &Path, args: &[String], install_permitted: bool) -> Result<()> {
    let wasm_opt_path = match find_wasm_opt(cache, install_permitted)? {
        Status::Found(path) => path,
        Status::Install(status) => match status {
            install::Status::Found(dl) => dl.binary("bin/wasm-opt")?,
            install::Status::CannotInstall => {
                PBAR.info("Skipping wasm-opt as no downloading was requested");
                return Ok(());
            }
            install::Status::PlatformNotSupported => {
                PBAR.info("Skipping wasm-opt because it is not supported on this platform");
                return Ok(());
            }
        },
    };

    PBAR.info("Optimizing wasm binaries with `wasm-opt`...");

    for file in out_dir.read_dir()? {
        let file = file?;
        let path = file.path();
        if path.extension().and_then(|s| s.to_str()) != Some("wasm") {
            continue;
        }

        let tmp = path.with_extension("wasm-opt.wasm");
        let mut cmd = Command::new(&wasm_opt_path);
        cmd.arg(&path).arg("-o").arg(&tmp).args(args);
        child::run(cmd, "wasm-opt")?;
        std::fs::rename(&tmp, &path)?;
    }

    Ok(())
}

/// Possible outcomes of attempting to find/install `wasm-opt`
pub enum Status {
    /// We found `wasm-opt` at the specified path (in PATH)
    Found(std::path::PathBuf),
    /// Installation results
    Install(install::Status),
}

/// Attempts to find `wasm-opt` in `PATH` locally, or failing that downloads a
/// precompiled binary.
///
/// Returns `Some` if a binary was found or it was successfully downloaded.
/// Returns `None` if a binary wasn't found in `PATH` and this platform doesn't
/// have precompiled binaries. Returns an error if we failed to download the
/// binary.
pub fn find_wasm_opt(cache: &Cache, install_permitted: bool) -> Result<Status> {
    // First attempt to look up in PATH. If found assume it works.
    if let Ok(path) = which::which("wasm-opt") {
        PBAR.info(&format!("found wasm-opt at {:?}", path));
        return Ok(Status::Found(path));
    }

    Ok(Status::Install(install::download_prebuilt(
        &install::Tool::WasmOpt,
        cache,
        "latest",
        install_permitted,
    )?))
}

Same issue here on MacOS with 0.11:
[INFO]: found wasm-opt at "/opt/homebrew/bin/wasm-opt"
Error: /opt/homebrew/bin/bin/wasm-opt binary does not exist

Looks like it is adding an extra bin to the path for some reason.
Going to downgrade back to 0.10.3 for now.

Temporary fix is to symlink ~/bin/wasm-opt to ~/bin/bin/wasm-opt

This issue still persists on Windows to this day, @drager

This issue still persists on Windows to this day, @drager

Still an issue in 0.13.0? If so, please open a new issue.


ERC721 Standard Should use felt* instead of felt for tokenURI

Raising this discussion again as it's been discussed a few times but OpenZeppelin has decided to use felt instead of felt* for tokenURI. Their reasoning is discussed here. This is problematic for the community as it means that no reasonable NFT with metadata can be used as the tokenURI limit of 31 characters is too small for even an IPFS URI
Other entities such as Aspect have also diverged from the OpenZeppelin Standard. Having discussed with the team at LootRealms, they will also face this issue.
Immutable X, out of necessity, has also diverged and created an ERC721 standard inspired by OpenZeppelin which a key divergence being the tokenURI. Immutable X provides this as a preset to the community: https://github.com/immutable/imx-starknet/blob/main/immutablex/starknet/token/erc721/presets/ERC721_Full.cairo
Having discussed with Martín Triay from OZ, his view was that they would rather wait for StarkNet to release Strings. This unknown date of string support causes issues for organizations wishing to move close to productionizing their apps.
Immutable X will be supporting the long-form Token URI represented using felt* as used in their preset. Immutable X also presents encoding and decoding implementation for the tokenURI:
We believe the community should align on enabling organizations to be successful as these organizations will drive users to StarkNet. For NFTs on StarkNet, this means enabling metadata through an extended tokenURI.
If this gets more than 10 Rocket reacts, I'll make a SIMP for it.

If the proposal to return felt* from tokenURI goes forward I propose to include a strict requirement on how strings should be encoded (should that be another SIMP?) into felts to avoid future issues. For example, a string with 94 characters can be serialized in 4 felts but also in 3 depending on the implementation. Also, should strings be zero terminated or not?

If the proposal to return felt* from tokenURI goes forward I propose to include a strict requirement on how strings should be encoded (should that be another SIMP?) into felts to avoid future issues. For example, a string with 94 characters can be serialized in 4 felts but also in 3 depending on the implementation. Also, should strings be zero terminated or not?

Agreed that the proposal should be accompanied with a strict standard on the encoding. The nuances can be discussed if this moves ahead. Let me know if there is anything else you would like to see in the encoding standard.

Another option is to introduce a new method for StarkNet NFTs (like metadataURI) so that tokens are still ERC721 complaint (by not implementing the optional ERC721Metadata interface). I can see this being very confusing for developers so I don't really like it even if it checks all boxes.

Another option is to introduce a new method for StarkNet NFTs (like metadataURI) so that tokens are still ERC721 complaint (by not implementing the optional ERC721Metadata interface). I can see this being very confusing for developers so I don't really like it even if it checks all boxes.

The developer experience of this specialized knowledge on ERC721 encoding is not ideal, but the alternative is complete unusability. Furthermore, this. specializing knowledge can be paired with the ERC721 SIMP.
It's better that it gets used by those who know how than be unusable by everyone.


Bug: codgen error display when url is empty

Describe the bug/problem
When the url is empty , every javascript and node js related cogen generates an error
Steps to Reproduce the bug/problem
Keep the url empty and gen code for js (axios) , js (fetch) , nodejs (axios), nodejs (fetch) and cUrl

Expected behavior
Instead of showing error , it should indicate a message that the url is empty and you should fill it to generate code
Device Info (The device where you encountered this issue):

OS: Windows
Version: Windows 11

Duplicate of #157


Add GVK matcher

Adds a matcher that can be used to contruct an CustomResourceDefinition Object, to check existence of cds using a GVK:
gvk := schema.GroupVersionKind{
  Group:   "helm.cattle.io",
  Version: "v1alpha1",
  Kind:    "HelmChart",
}

lgtm :shipit:


To open or to not open public ports

It make sense to open grafana port to the public.
But I'm not not sure to understand why prometheus and alertmanager have their port public. Any particular reason for this behaviour?
Many cheers!

I'm guessing you want to view alerts and be able to silence them, as for Prometheus you can monitor the targets, write queries and more. I will update the project and I'll include a reverse proxy with basic auth for those two.

@pascalandy I've added the reverse proxy with basic auth, can you please test it out?

Cannot start compose.
root@vps01:~/dockprom# docker-compose up -d
ERROR: Invalid interpolation format for "environment" option in service "caddy": "ADMIN_USER=${ADMIN_USER:-admin}"

Looks like inline defaults are supported in compose 2.1, should work now.

humm
ERROR: Version in "./docker-compose.yml" is unsupported. You might be seeing this error because you're using the wrong Compose file version. Either specify a version of "2" (or "2.0") and place your service definitions under the services key, or omit the version key and place your service definitions at the root of the file to use version 1.
For more on the Compose file format versions, see https://docs.docker.com/compose/compose-file/
Aborting due to errexit on line 26. Exit code: 1

can you please post docker-compose -v

On 1.8.0 and I see we are now at 1.16.1 !
https://github.com/docker/compose/releases

No, compose comes only with Docker for Mac, on linux you have to update via:
curl -L https://github.com/docker/compose/releases/download/1.16.1/docker-compose-`uname -s`-`uname -m` > /usr/local/bin/docker-compose
chmod +x /usr/local/bin/docker-compose

Updated. Compose is running.
ERROR: for caddy  Cannot start service caddy: driver failed programming external connectivity on endpoint caddy (68b28af87Creating grafana ... done

ERROR: for caddy  Cannot start service caddy: driver failed programming external connectivity on endpoint caddy (68b28af8726c3582076d5a26fe40746b46ec61361202aa23705cff77d00f144b): Bind for <IP_ADDRESS>:9093 failed: port is already allocated

That's because you are running the older version with prometheus exposing port 9093.
You have to remove the stack with:
docker-compose down -v

And run the new one with:
ADMIN_USER=admin ADMIN_PASSWORD=admin docker-compose up -d

After that your prometheus and alermanager services will require user/pass.

Ok, thanks for testing, I've updated the readme with the minimum required Docker Compose version.

Sorry for the confusion earlier. I meant, compose is now installed properly.
All the container behave normally. But there is nothing on those 3 ports :-/
Prometheus (metrics database) http://:9090
AlertManager (alerts management) http://:9093
Grafana (visualize metrics) http://:3000

It was  problem with the mount point for Caddy, should work now.

Yes! This is solid now :)


Enabling Natural Language Graph Queries in GraphScope with chatGPT

Is your feature request related to a problem? Please describe.
Graph queries are a critical workload in GraphScope, currently submitted through query languages including Gremlin and Cypher. Although these graph query languages greatly reduce the user barrier compared to programming languages like Java or C++, GraphScope users still frequently struggle with query writing. This not only severely limits user graph querying capabilities, but also consumes valuable developer time in answering user questions. With the emergence of chatGPT, people have realized the incredible ability of large language models to handle natural language-related tasks including writing articles, answering questions, and even writing code. We wonder if we can leverage the ability of large language models to further reduce the barrier to using graph queries in GraphScope.
In this task, you will explore this question with us. Specifically, we expect that given the context of graph queries, such as query samples, graph schema information, and reference query implementations (based on Cypher), can we allow users to automatically generate Cypher queries expressed in natural language?
Describe the solution you'd like

Explore the use of chatGPT to generate additional test samples (including manual labeling) based on existing query samples.
Use the above test samples to explore the improvement of Cypher query generation accuracy through the prompt method.
Add natural language querying capabilities to the existing Python interface in GraphScope.

wanna join🙋

wanna join🙋

Hi, please sign in as candidate in: https://m.summer-ospp.ac.cn/org/prodetail/2313e0395

I wonder what's the next step for this feature.


New user/gift 56

Screen Preview

Mobile

Back end not yet implemented

Branch atnnn/56 fixes the lint and deduplication errors


Material UI themeing

Note
Mistake with this branch. Closed the pull request without merging and delete the branch.


Failed to convert from kitti to voc: Expected subdirectory training/image_2 ?????

Hi
I am trying to convert txt to pascal voc, but I couldn't I tried python2 and puthon3.6, both give same error
(py3.6) abdu@grasp-uoit:~/1vod-converter$ python3.6 vod_converter/main.py --from kitti --from-path /home/abdu/datasets/mydata-kitti --to voc --to-path /home/abdu/datasets/mydata-voc
INFO:root:Namespace(filter_images_without_labels=False, from_key='kitti', from_path='/home/abdu/datasets/mydata-kitti', select_only_known_labels=False, to_key='voc', to_path='/home/abdu/datasets/mydata-voc')
Failed to convert from kitti to voc: Expected subdirectory training/image_2 within /home/abdu/datasets/mydata-kitti

Any suggestions please ???

Check that your "/home/abdu/datasets/mydata-kitti" has that subdirectory structure. When you download the kitti data, the default directory structure should match what the conversion tool is expecting.

Check that your "/home/abdu/datasets/mydata-kitti" has that subdirectory structure. When you download the kitti data, the default directory structure should match what the conversion tool is expecting.

Hey. I also got this error. I donwloaded data_tracking_image_2.zip.  Could you tell me which kitti is right?
Thank you.

I also got the error.
what is wrong?
python vod_converter/main.py --from kitti --from-path ../data/label_2 --to voc --to-path ../data/label_voc INFO:root:Namespace(filter_images_without_labels=False, from_key='kitti', from_path='../data/label_2', select_only_known_labels=False, to_key='voc', to_path='../data/label_voc') Failed to convert from kitti to voc: Expected subdirectory training/image_2 within ../data/label_2

You need to have the PASCAL VOC data in a directory structure expected by this package (I think it's the same as is used for the VOC2102). When I use this package to convert my PASCAL VOC files I first create the directory structure and copy the files into the expected locations, like so:
from vod_converter.converter import convert
from vod_converter.kitti import KITTIEgestor
from vod_converter.voc import VOCIngestor

def pascal_to_kitti(
        pascal_dir: str,
        images_dir: str,
        kitti_dir: str,
) -> int:
    """
    Builds KITTI annotation files from PASCAL VOC annotation XML files.

    :param pascal_dir: directory containing input PASCAL VOC annotation
        XML files, all XML files in this directory matching to corresponding JPG
        files in the images directory will be converted to KITTI format
    :param images_dir: directory containing image files corresponding to the
        PASCAL VOC annotation files to be converted to KITTI format
    :param kitti_dir: directory where output KITTI annotation files will
        be written, will be created if it does not yes exist
    :return: the number of PASCAL VOC annotation files converted to KITTI format
    """

    # get list of file IDs of the PASCAL VOC annotations and corresponding images
    file_ids = _matching_ids(pascal_dir, images_dir, ".xml", ".jpg")

    # create a temporary directory to contain everything in the structure
    # (VOC2012) expected by the vod_converter package we'll use for conversion
    with tempfile.TemporaryDirectory() as base_dir:

        # recreate the VOC2012 directory structure
        voc2012_dir = os.path.join(base_dir, "VOC2012")
        os.mkdir(voc2012_dir)
        jpegimages_dir = os.path.join(voc2012_dir, "JPEGImages")
        os.mkdir(jpegimages_dir)
        imagesets_dir = os.path.join(voc2012_dir, "ImageSets")
        os.mkdir(imagesets_dir)
        main_dir = os.path.join(imagesets_dir, "Main")
        os.mkdir(main_dir)
        voc_annotations_dir = os.path.join(voc2012_dir, "Annotations")
        os.mkdir(voc_annotations_dir)

        # link annotation and image files into the expected locations
        for file_id in file_ids:
            pascal_file_name = file_id + ".xml"
            image_file_name = file_id + ".jpg"
            pascal_file_path = os.path.join(pascal_dir, pascal_file_name)
            image_file_path = os.path.join(images_dir, image_file_name)
            voc_link = os.path.join(voc_annotations_dir, pascal_file_name)
            image_link = os.path.join(jpegimages_dir, image_file_name)
            os.symlink(pascal_file_path, voc_link)
            os.symlink(image_file_path, image_link)

        # create the "trainval.txt" file
        trainval_path = os.path.join(main_dir, "trainval.txt")
        with open(trainval_path, "w+") as trainval_file:
            for file_id in file_ids:
                trainval_file.write(f"{file_id}\n")

        # utilize the vod_converter to convert PASCAL VOC to KITTI
        success, msg = convert(
            from_path=base_dir,
            ingestor=VOCIngestor(),
            to_path=kitti_dir,
            egestor=KITTIEgestor(),
            select_only_known_labels=False,
            filter_images_without_labels=True,
        )
        if not success:
            raise ValueError(f"Failed to convert from PASCAL to KITTI: {msg}")

    # return the number of annotations converted
    return len(file_ids)

@krosaen please colse this issue as it is solved and inactive


Test Nix extraction

Currently, the Nix extraction process and mostly nixpkgs-graph.nix are not tested.
This issue gathers suggestions and tracks improvements.

Not in project as this needs to be turned into actionables.
First idea is to use the core model introduced by #56 to check the shape of the output

A model and schema was created, it is not ideal but enough for now.

Would introducing a nightly pipeline that runs extraction help here? You would catch hey we need to update something errors, right?


Refactor account config

The configure command is too vague, and leads to confusion. There are 3 things to consider:

Configuration can operate at file level (configuration of accounts)
Configuration can operate at app level (keyring set up, folder creation etc)
Configuration can be debugged (like a doctor command)

Implemented by https://github.com/pimalaya/himalaya/commit/f84146bd1bee68fb3740cbab1484031cca3945a4.


Adding the pjit in the comparison

Some people are more familiar with using model parallel via pjit. What about adding one more rows here?

Pjit will be exactly the same as xmap in this table.  I can add it later


Deploy a live labflow instance.

GCP might be a good target.
Consider how to gate user signups for this.

Closing because this issue is duplicated in the lab-grid/flow project.


No sound - dolby with headset connected - when switch from DVB playback to OTT (ExoPlayer)

Media3 Version
ExoPlayer 2.18.6
Devices that reproduce the issue
Amino hybrid box
Devices that do not reproduce the issue
No response
Reproducible in the demo app?
Not tested
Reproduction steps
Precondition
There is dolby audio stream (codec: ac-3). Device has audio decoder, There is connected headset by Bluetooth
1 Step) Play OTT stream - select dolby audio track, track is down-mixed, playback could be heard.
2 Step) Play DVB stream
3 Step) Stop DVB stream
4 Step) Play OTT stream - select dolby audio track, there is error in logs like

06-01 10:09:22.301  7551  9033 I BTAudioProviderStub: streamStarted - SessionType=A2DP_SOFTWARE_ENCODING_DATAPATH, status=FAILURE
06-01 10:09:22.301  7551  9033 I BTAudioProviderSession: ReportControlStatus - status=FAILURE for SessionType=A2DP_SOFTWARE_ENCODING_DATAPATH, bluetooth_audio=0x0100 started
06-01 10:09:22.301  7551  9033 D A2DPHW  : BluetoothAudioPortOut::Start: state=1, ret=1
06-01 10:09:22.303  7551  9033 D a2dp_hal: a2dp_out_resume: state=1

Expected result
if stream can not be down-mixed there should be reject when selecting dolby track, or supported flag should be to false
Also this looks like regression between versions as issue is not reproducible at version 2.12.3
It's reproducible at 2.17.1 ( which we currently use, as also as we tested it's still reproducible at version 2.18.6 )
Actual result
There is no sound, there is no error notification from ExoPlayer
Attached bug report from version 2.18.6
Media
no specific media
Bug Report

[X] You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

Hi, thank you for reporting the issue. In order for us to reproduce it, would you mind sharing the media you were using for it?
Also, when you say "DVB playback", do you mean an RF signal carrying DVB? And is that arriving on the device via TV Input Framework?

Yes, signal DVB is from RF input. I do not have knowledge about solution over DVB as it's delivered by device manufacturer as third party library (we have support of manufacturer but communication pipe is long ).
Used media do not looks like important in this case.
From my perspective it looks like as there in an issue with using device decoder to down-mix Dolby track ( Manufacturer reported that not down-mixed stream were tried to pushed by Bluetooth )

It will be hard to investigate without a way to reproduce locally. I understand this is specific to the device and perhaps to the content or the way the app is using ExoPlayer.

From my perspective it looks like as there in an issue with using device decoder to down-mix Dolby track ( Manufacturer reported that not down-mixed stream were tried to pushed by Bluetooth )

ExoPlayer does not know whether the Dolby decoder will down-mix surround audio. On Android 12L+, an the app can ask the decoder to not downmix. ExoPlayer does ask the decoder to not downmix as of 2.18.0. However, you observe issues with prior ExoPlayer versions and I don't think the Amino hybrid box is on Android 12L or later. In addition, even if the decoder outputs multi-channel PCM, the platform (=> AudioTrack) should down-mix and transmit stereo to the BT headset.
Without a way to repro, I can only ask for more information to narrow-down the conditions the issue occurs:

Is the error happening only when you switch between OTT -> DVB -> OTT?
What is the audio of the DVB stream? AAC stereo?
Can the error be reproduced in the ExoPlayer demo app, e.g. switching between AC3 -> [the same audio as the DVB stream] -> AC3 streams?
If yes, can you please repro on the ExoPlayer demo app and send us a bug-report after the error occurs? It is critical to use the demo app because we know how the app uses ExoPlayer and it produces additional logs.
If you cannot use the demo app, can you please attach an EventLogger on the ExoPlayer instances (with Player.addAnalyticsListener) on your app and get another bug-report? Note that the bug-report we received has no logs coming out from ExoPlayer. If you've disabled ExoPlayer's logging, please re-enable them.

Thanks

I've send another bug report with EventLogger enabled, I've also attached logs gathered by adb logcat.
Ad1 ) Maybe my description were misleading
When OTT is started directly bug is not reproduced, to trigger it before OTT there is need to do DVB playback
Ad2) DVB audio is MPEG 1 - Layer II
Ad3,4 ) Establish such environment would be a bit difficult for us & imho this would not reproduce issue, we will try it if other analysis will fail.
About device specification I think it could be read from bug report, If you will have some issue with that, please let me know I'll get it for you

Thanks for sending the BR with the additional logs. May I ask, at what point in your setup do you connect to BT headset? Is it always connected or do you connect it after already having played the OTT the first time?
TLDR: I believe we have a fix for this issue but it's not yet released. It will be in the upcoming media3 1.1.0 (or ExoPlayer 2.19.0). To verify the fix addressed your issue, can you build you app using ExoPlayer from the dev-v2 branch (or media3 from main branch)? You need to make your app depends on the library locally.
More details: There is a subtle detail in the player logs. While the device can play ac-3, the platform is not exposing an ac-3 decoder, rather the device reports it can play ac-3 directly via passthrough. I'm not sure if the device has an ac-3 hardware decoder that is only exposed via passthrough, or the device is plugged over HDMI to a TV (or AV receiver) that supports ac-3, but that's a detail.  We have a fix on the player to detect that when the device is connected to a BT headset, the player should not play surround formats that are supported via passthrough and either play stereo or use a track that is decoded locally to PCM (the platform will downminx to stereo before transmitting to the BT headset). The player fix also detects when a BT headset is connected on the device mid-playback and the player should automatically switch from ac-3 to a compatible audio track.
I noticed however the following: on your device, when the player attempts to play ac-3 over BT, there's a native crash. We expect to get an error at the Java-layer and to act upon it. This is going to impact the case where the player is already playing an ac-3 and the user connects the BT headset midstream. The platform should not crash, but we expect to receive an error and  switch to stereo. This is not something ExoPlayer can control.

For reference, I think this issue is likely a duplicate of either https://github.com/google/ExoPlayer/issues/9588 or https://github.com/google/ExoPlayer/issues/10227.

I can confirm that using build from dev branch resolves our client issues

Glad to hear. media3-1.1.0 is already in beta, and we expect to release media3-1.0.0 stable sometime in July. ExoPlayer 2.19.0 will be released with media3-1.0.0 stable.
I'm closing this issue as the underlying issue is fixed.


pull request for e2e test at 01/15/2017 09:26:14

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


Mobile support on iOS/Android?

Is there anything preventing this plugin from working on iOS & Android? Draw.io, Diagrams.net does appear to work on iOS & iPad. Can this plugin be enabled for iOS/iPad mobile Obsidian?

I'm also interested to know

The Excalidraw plugin works on mobile and I'm using that for now. But I'd love to be able to use this one.


feat(rest): add thumbnailDetails to search2 response

Checklist

[x] the contributor license agreement is signed
[x] commit message follows commit guidelines
[x] tests are included
[x] screenshots are included showing significant UI changes
[ ] documentation is changed or added

Description of change
To assist with the display of suitable thumbnails (especially if includeAttachments is false) a new property has been added to the search2 response.
Included a slight refactor to push some code into AttachmentHelper for reuse.

Thanks @PenghaiZhang . All feedback addressed, but let me know if you see an issues. Otherwise, I'll merge on green.

Failed on known flaky LDAPTest. merging.


Soften language for "A patch has been attached on this bug, which was already closed" alert

I saw a new category of bot-comment today:

A patch has been attached on this bug, which was already closed. Filing a separate bug will ensure better tracking. If this was not by mistake and further action is needed, please alert the appropriate party.

https://bugzilla.mozilla.org/show_bug.cgi?id=1763431#c14
This language seems to suggest that this scenario is always concerning (either a mistake or something where a party needs to be alerted).
But in fact, we do legitimately post patches on already-closed bugs from time to time, for several trivial/non-concerning scenarios:

landing testcases after a bug becomes fixed (either closed as WFM, or duped against a related bug that turns out to fix it, or is a security bug and can't have testcases land right away)
simple/trivial followup patches (e.g. fixing a typo in a code-comment)

From my reading, the bot is requesting that a new bug be filed for all of these scenarios, which feels unnecessarily complex/heavier-in-terms-of-overhead.  Could we adjust the language of this bot comment to note that these scenarios are OK?  E.g. maybe add:
 "If the patch changes behavior and [...]"

(In all of the trivial scenarios that I'm currently thinking about, the patch in question would be something that doesn't change behavior.)

The suggested wording sounds fine to me.
We can also implement something to avoid adding a comment in case the patch is only touching test files or documentation files or (more complex) comments.

Thanks! For now, I think just improving the wording would be fine.  The legit scenarios that I brought up aren't super-frequent, and it's nice to attempt to e.g. catch cases when people posting on the wrong bug due to a typo in their commit message (maybe leaving out a digit from the bug number), regardless of whether their commit is test-only/comment-only/etc.


Please add AV1 (libdav1d?) in VLC

IDK if this is the right place to ask this, but please consider building VLC with an AV1 decoder, preferably libdav1d. At present, VLC works alright on non-AV1 video (and audio) files but a growing portion of my video library is AV1, so many videos are unplayable.
PS: I know that VLC can be build with a decoder, as the Flatpak version has one.
Edit: The AV1 decoder IS present and works great. It seems the *actual* a problem lies with the OpenGL video output, and happens regardless of which codec you use. (Tested with AV1, VP9, and MPEG-4 part 2.) I will use the bug reporting template:
Describe the bug
OpenGL video output doesn't seem to work. VLC tries to play the video, but fails. It flickers for a few seconds, then just shows the VLC logo as if no file has been opened, with the audio playing just fine. Sometimes VLC will keep changing window sizes, making pausing more difficult.
Steps To Reproduce
Steps to reproduce the behavior:

Go into the preferences, set the video output to OpenGL video output.
Open any video

Expected behavior
OpenGL-accelerated video playback.
Additional context
I am running Ubuntu 22.04, and installed the nix package manager using these instructions. Also, I will attach a console log of VLC, in case something there helps.
vlc-output.txt
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result. - system: "x86_64-linux"

host os: Linux 5.15.0-47-generic, Ubuntu, 22.04.1 LTS (Jammy Jellyfish), nobuild
multi-user?: no
sandbox: yes
version: nix-env (Nix) 2.10.3
channels(tyler): "nixpkgs"
nixpkgs: /home/tyler/.nix-defexpr/channels/nixpkgs

Turns out, there is a wrapper script, nixGL, which allows nix programs to use OpenGL on non-NixOS systems. I just needed to follow the instructions in the link above. With that, I'm closing this bug.


Fix 2>&1 redirects

2>&1 comes after 1>, not before, if you want to combine outputs.
The way this works internally in bash is that when you do 2>&1, bash copies fd 1 and pastes it on fd 2. When you later change fd 1 to the file, fd 2 is still stdout.
But if you put the fd 1 redirect before, then fd 1 is changed first and fd 2 copies this change.

@AstroSnail yup good catch, someone in another chat room pointed this out to us as well. i just addressed this in https://github.com/Xe/site/pull/400/commits/48dfb429eaf548d1795a9d67743d713bb90b4e95 but also added verbiage to point out the footgun and the advantage of using &> to avoid it

understood, closing this pr


Please support deepseek coder greater-ly.

Issue
deepseek is much cheaper and useful for certain scenario but i seemed to have hit some limits:
Model deepseek/deepseek-coder has hit a token limit!
Token counts below are approximate.

Input tokens: ~18,596 of 128,000
Output tokens: ~3,223 of 4,096 -- possibly exceeded output limit!
Total tokens: ~21,819 of 128,000

For more info: https://aider.chat/docs/troubleshooting/token-limits.html

please make it "smarter" by allowing us to break down it's output generation in piecemeals or something.
how to make this work?
split the router.rs file into smaller files for easier management. (do for the first 2 files first)

optionally i did "do the first 2 files first" when the initial prompt didnt work but it still shows token limit hit etc.
pls help suggest ways to work around. thx
Version and model info
aider latest
ubuntu 24.04

DeepSeek now offers a bete API with 8k output tokens. This might ease the pain. https://platform.deepseek.com/api-docs/pricing/

@olafgeibig how do i do the "beta" that offers the 8k version?

Thanks for trying aider and filing this issue.
I'm waiting on litellm support for prefill with deepseek.
https://github.com/BerriAI/litellm/issues/4881

Are the 8K tokens and the prefill functionality actually referring to the same thing? Or would we benefit from using both prefill and the 8K api endpoint instead of just one of them?

The main branch has 2 new features which I think solve this issue:

Infinite output for DeepSeek Coder, Mistral models in addition to Anthropic's models.
New --deepseek switch to use DeepSeek Coder.

The change is available in the main branch. You can get it by installing the latest version from github:
python -m pip install --upgrade git+https://github.com/paul-gauthier/aider.git

If you have a chance to try it, let me know if it works well for you.

Is this right?
python -m aider.main --model deepseek --browser

It runs but I get this error with any prompt
BadRequestError: litellm.BadRequestError: GetLLMProvider Exception - list index out of range

original model: deepseek

You need to use --model deepseek/deepseek-coder.
I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.


[Minor] [ML] Fix naming conventions of AFTSurvivalRegression coefficients

#9311 has rename coefficients to regressionCoefficients, I think it's

Merged build triggered.

Merged build started.

Test build #44905 has started for PR 9431 at commit ff1f9e4.

Test build #44905 has finished for PR 9431 at commit ff1f9e4.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * case class UnresolvedStar(target: Option[Seq[String]]) extends Star with Unevaluable \n

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

Merged build finished. Test PASSed.

LGTM. This makes more sense.

Merged into master. Thanks!


fix autofocus

Fixes https://github.com/koca/vue-prism-editor/issues/63

@koca Any chance this can get merged in?

Hi guys! thanks for the PR :) LGTM

Released! 🚀


Fix column width for minus i

In the table telling the composition of standard ket states (|+>, |->, |i>, |-i>) in zero/one basis, the "|-i>" formula is wrapped on two lines, hurting readability (tested with both Firefox and Chromium with Ubuntu 20.04).
In this pull request, I suggest to widen the table columns a little bit, so that each formula will fit on a single line.

 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.:x: abienvenu sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Hi,
Looks good, but maybe we could get rid of the fixed width all together with default markdown tables like this:
|   |   |
|:---:|:---:|
|$|+\rangle = \frac{1}{\sqrt{2}}\big(|0\rangle + |1\rangle\big)$|$|-\rangle = \frac{1}{\sqrt{2}}\big(|0\rangle - |1\rangle\big)$|
|$|i\rangle = \frac{1}{\sqrt{2}}\big(|0\rangle + i|1\rangle\big)$|$|-i\rangle = \frac{1}{\sqrt{2}}\big(|0\rangle - i|1\rangle\big)$|

Not sure if this would also work on your setup.

@vivanwin I don't really like how default Markdown tables look when they don't have a header, and in this case in particular I like to have some extra space between formulas in the same row to keep them visually separate:

In some cases they look nicer, but in this case I think we'll be better using the HTML formatting.


Replace deprecated linters

Background
The make lint complains about these warnings.
The following linters have been deprecated.
WARN [runner] The linter 'interfacer' is deprecated (since v1.38.0) due to: The repository of the linter has been archived by the owner.
WARN [runner] The linter 'scopelint' is deprecated (since v1.39.0) due to: The repository of the linter has been deprecated by the owner.  Replaced by exportloopref.
WARN [runner] The linter 'golint' is deprecated (since v1.41.0) due to: The repository of the linter has been archived by the owner.  Replaced by revive.

Update these linters to the ones that have been recommended.

Fixed by https://github.com/lightningnetwork/lnd/pull/6244.


[Bug]: MySQL queries dependent on temporary tables has stopped working

Is there an existing issue for this?

[X] I have searched the existing issues

Description
https://app.frontapp.com/inboxes/teams/folders/15690316/archived/69384087244
Steps To Reproduce
https://app.frontapp.com/inboxes/teams/folders/15690316/archived/69384087244
Public Sample App
No response
Issue video log
No response
Version
cloud, self hosted

Please note that this was never an intended feature - i.e. multiple separate queries being able to access a temporary table. This was supported as a side effect of a bug that was long pending - i.e. we used to run queries on a single connection as opposed to a connection pool. Now that we have a connection pool each subsequent query may get allocated a different connection. As per the MySQL documentation:
You can use the TEMPORARY keyword when creating a table. A TEMPORARY table is visible only within the current session, and is dropped automatically when the session is closed.

Ref: https://dev.mysql.com/doc/refman/8.0/en/create-temporary-table.html
In order to support this again we would have to introduce a new feature - i.e. something equivalent of pinning of single connection for a datasource or we allow users to create a connection pool with exaclty one connection in the pool.
cc: @rohan-arthur @Nikhil-Nandagopal

@felix-appsmith

https://mixpanel.com/project/2032485/view/178891/app/boards#id=4172284&edited-bookmark=53K8d9PeM9ky

Closing since this is the intended behaviour and we don't want to alter this any time soone.


NOT TRY ERROR

rocketmq/client/src/main/java/org/apache/rocketmq/client/impl/producer/DefaultMQProducerImpl.java
private SendResult sendKernelImpl(final Message msg,
final MessageQueue mq,
final CommunicationMode communicationMode,
final SendCallback sendCallback,
final TopicPublishInfo topicPublishInfo,
final long timeout) throws MQClientException, RemotingException, MQBrokerException, InterruptedException {}
MessageClientIDSetter.setUniqID(msg);
Error thrown when commons-validator-1.7.jar is Missing ,However, the error is not captured, so the SendResult cannot be returned.
java.lang.NoClassDefFoundError: org/apache/commons/validator/routines/InetAddressValidator
at org.apache.rocketmq.common.UtilAll.ipV6Check(UtilAll.java:474)
at org.apache.rocketmq.common.UtilAll.getIP(UtilAll.java:530)
at org.apache.rocketmq.common.message.MessageClientIDSetter.(MessageClientIDSetter.java:36)
at org.apache.rocketmq.client.impl.producer.DefaultMQProducerImpl.sendKernelImpl(DefaultMQProducerImpl.java:709)

Please check in your application why this exception NoClassDefFoundError is thrown back

I use rocketmq-flume, which introduces the client, but does not throw an exception, and is always waiting.


⚠ Bofrost.es has degraded performance

In 3c37575, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1471 ms

Resolved: Bofrost.es performance has improved in 0b70549 after 9 minutes.


remove bc dependency since TrueNAS Scale doesn't include it.

Hey @StraightfaceStudios !
Thx for opening a PR 🤓
Is the script outputting the exact same text to the console after this change? Can you please post examples of old and new to compare?

progress_percent=$((current_index * 100 / file_count))
In line 72 fixes this problem. Let me know if you want a new PR.
`
Progress -- Files: 7/130564 (0%)
Copying '/mnt/excitement/FileName.me' to '/mnt/excitement/FileName.me.balance'...
`

Not sure why this wasn't merged, but it's still open.

Because the tests are failing.
Looks like the decimals are gone and shellcheck isn't happy.

Closing this in favor of #24


Revert "Upgrade FBOS dependencies to 2<IP_ADDRESS> (#11018)"

This reverts commit 6d1fbf0576c2d188832c0237709a32f69f5b92e6.
I validated locally that reverting this commit fixes the velox_functions_remote_client_test failure.
Resolves https://github.com/facebookincubator/velox/issues/11071

@czentgr I feel we must revert this PR to unblock CI. The investigation of the sigsegv is taking longer than I anticipated.


iBoot Panic on next macOS startup after using asahi-bless while macOS was hibernated

If I use asahi-bless to change the default boot disk to macOS while macOS is hibernated, macOS shows an error "SOCD report detected: (iBoot Panic)" on the next startup (this is a follow-up to #15).
PS. While repeatedly retesting these steps, I didn't want to hibernate by actually running out of battery, so I temporarily used pmset -a hibernatemode 25 to manipulate sleep mode.
To change the setting back to normal after testing, pmset -a hibernatemode 3 is the default for laptops and 0 for desktops, I believe.
Steps to reproduce:

startup disk -> Asahi Linux
in macOS, hibernate (I used sudo pmset -a hibernatemode 25 && pmset sleepnow)
wait a couple of seconds and then press the power button to start up the laptop (loads Linux because of step 0)
in Linux: shutdown -h now
long press the power button, boot from "Macintosh HD"

As expected, no error in macOS (yet).
Let's do the same thing again but now with asahi-bless:

(startup disk is still Asahi Linux)
in macOS, hibernate (I used sudo pmset -a hibernatemode 25 && pmset sleepnow)
wait a couple of seconds and then press the power button to start up the laptop (loads Linux because of step 0)
in Linux: use sudo asahi-bless to set the startup disk -> "Macintosh HD"
reboot

What happens: when starting up, macOS shows an error, see screen shots below (I used TinkerTool to "Display crash report" when applications crash)

Expected: don't show an error
I'm using an M1 MacBook Air, macOS 14.3.1 (23D60), the latest version of Fedora Asahi Remix and the latest Git tip of asahi-bless

Unfortunately, for now the solution is "well, do not do that then", reliably detecting if there is a hibernated macos requires parsing too many apfs structures to be practical.


Pass userData to the setProfileVariables method

From HelperMethods.constructAndStoreUserTags method, pass userData to the BusinessRules.setProfileVariables method so it can be used in setting the profile data.
Fixed location of call to callback method in setProfileVariables.

Redoing based on current master branch code line.


Support for handling primitives/builtin functions, and language settings

This change adds basic infrastructure to use annotations to handle primitives in Truffle languages.
It supports parse-time and eager run-time specialization. During specialization, the normal method call can be replaced with a specific operation. This means, there is no overhead for a method call anymore in the interpreter.
How to use this code is shown for TruffleSOM in this PR: https://github.com/smarr/TruffleSOM/pull/13
However, the code is still work-in-progress, because it is not yet directly useable for SOMns.
Open issues:

[x] different constructor arguments for nodes
[x] VmSettings handling

@charig @richard-roberts this is a preview. I'd be curious about what you think.
I just wanted to get things started and actually have some code going into the project.
If you look a the code, please check the JavaDoc, I tried to actually write some documentation.
VmSettings
This PR also introduces a basic framework for language-specific settings.
A language can implement the Settings interface.
The name of the class implementing it, needs to be provided via the bd.settings Java property.
From that property, the settings are loaded and used to initialize static fields of VmSettings.
Interfaces
The PR now introduces the following interfaces:

EagerlySpecializable, provides basic function needed to specialize a node eagerly
EagerPrimitive, is used for nodes that replace the original function calls for instance and provides basic functions to execute the node correctly after replacing the old one in the AST
Operation, is used by tools and marks a node to be an operation that has arguments and a name
PreevaluatedExpression provides a uniform method to execute a node with already evaluated subexpressions
WithContext marks a node as requiring access to a language's context object, which typically holds some global state or behavior nodes might need access to.

Grace uses a prototype-based object model. Would it be possible to decouple support for primitives from class instances, so that the VM can support primitive expression on any object?
If so, then how significant are the changes that would be required? One option for the longer term Grace implementation would be to support a single class that distributes primitives.

I worked a bit further on this, and made SOMns work with it.
I needed to introduce a configuration mechanism, too.
The idea is that the language can implement the Settings interface.
The name of the class implementing it, needs to be provided via the bd.settings Java property.
From that property, the settings are loaded and used to initialize static fields of VmSettings.

Ok, so, this is the first 'diamond' ready to merge, I think.
It has everything I need to make TruffleSOM and SOMns work.
@charig if I would want to apply it to TruffleMate, which branch would I be looking at?
SOMns PR to use it is here: https://github.com/smarr/SOMns/pull/185
The TruffleSOM PR is here: https://github.com/smarr/TruffleSOM/pull/13

@smarr ReflectiveCompiler is the latest dev branch. I could apply it if you prefer, but I do not know how to create a PR to another repo...


Fix Reader::read

With the stream refactor being developed and tested alongside, some buggy behaviours are being discovered.
This PR fixes one of such, where calling read on a dead stream caused the program to segfault.

OK, the failure reports make no sense.
Toaster, do your job, and test this please

C'mon Jenkins. test this please


[Refactor] Collecting Feedback + Bug reports

We have a revamp of the Evaluation Harness library internals staged on the big-refactor branch! It is far along in progress, but before we start to move the master branch of the repository over to this new design, we'd like to ensure that it's been tested by outside users and there are no glaring bugs!
We’d like your help to test it out! you can help by:
1.Trying out your current workloads on the big-refactor branch, and seeing if anything breaks or is counterintuitive,
2. Porting tasks supported in the previous version of the harness to the new YAML configuration format described in https://github.com/EleutherAI/lm-evaluation-harness/blob/big-refactor/docs/new_task_guide.md !
If you choose to port a task not yet completed according to https://github.com/EleutherAI/lm-evaluation-harness/blob/big-refactor/lm_eval/tasks/README.md , then you can contribute it by opening a PR containing [Refactor] in the name with:

A shell command to run the task in the master branch, and what the score is
A shell command to run the task in your PR branch to big-refactor, and what the resulting score is, to show that we achieve equality between the two implementations!

Lastly, we'll no longer be accepting new feature requests beyond those that are already open to the master branch as we carry out this switch to the new version over the next week, though we will be accepting bugfixes to master branch and PRs to big-refactor!
Happy to answer any questions you may have!
This issue is intended to collate the problems encountered by users when trying out the big-refactor branch. Please feel free to reply down here with any confusions , errors, bugs, or otherwise encountered when using the big-refactor branch!

Closing as this is a bit stale, please get in touch with us at our discord in the #lm-thunderdome channel if you have any feedback on the big-refactor branch or are interested in helping port the last of the unported tasks to that new version.


Autosave indicator

I'm sure we used to have this but one of our larger customers has just sent some feedback that they expected to see, and missed, an autosave indicator in our editor:
Particularly on draft posts, it would be useful if this said Draft — Saving... and Draft — Saved

😕 we still have an autosave indicator but it looks like there's been a regression somewhere

Fix for the missing save indicator is now in master. I haven't made any changes to the text as it wasn't exactly clear in the issue what the design is for when different text states are displayed.
Draft — Saving... and Draft — Saved - should the "Saved" part also be on a timer or is this a permanent state after the first save?

I think permanent state is nice/reassuring - when in draft?
I've forgotten how it used to work 😅

Currently (after regression fix 😬) it changes from Draft, Published, Scheduled to Saving... and displays that for 3 seconds any time a save happens then reverts back to the post status. When scheduled/published we also show additional info.
If we're changing to Draft - Saved would we also need a way to reset to Draft any time a change has been made and a save hasn't occurred for whatever reason? ie. you're typing and so the auto-save keeps getting deferred, something caused an error when saving, etc

Hmm, I think it might get distracting if it flickers too much? Not 100% sure — might need a demo/example

Can we prioritise this for next cleanup weeks please