Using two different Context files

I want to use different context files(FeatureContext.php and WordPressContext.php) for my behat tests.  FeatureContext was present initially, then I defined a new WordPressContext file. I have amended the behat.yml config file and included it under contexts: section in default: profile.  The idea is to keep general functions related to wordpress functionality in WordPressContext and custom plugin related functions in FeatureContext.
FeatureContext Structure:
class FeatureContext extends MinkContext {
}
WordPressContext Structure:
class WordpressContext extends FeatureContext implements Context{
}
In WordPress context I want to call all the methods of MinkContext class in this manner:
MinkContext::fixStepArgument or  otherclassname::fixStepArgument
MinkContext::getSession or otherclassname::getSession;
instead of $this->getSession->find();
I am unable to use two Context files. When I run the feature files in PHPStorm using WordPressContext I get an error saying that Step " xyz" is already defined in WordPressContext::"function name". I have removed some of the functions from FeatureContext and pasted them in WordPressContext. There are no two functions with the same name in FeatureContext and WordpressContext.
How do I resolve this issue? Any assistance is greatly appreciated
Thanks!

Could you find a solution for this problem? Because I have the same.

The answer is not really so clear I think, what's surprising to me is one of the two possibilities

People dont bother testing web apps anymore (doubt it)
Maybe they've moved on from behat/mink? (probably, but i dont come across other projects?)

Maybe this helps
https://web.archive.org/web/20210120104126/http://blog.scur.pl/2012/06/subcontexts-behat-mink/
if you come across a better selenium2/behat/gherkin style BDD test hardness tell me.. I beg you :)


implement basic vote withdrawal

Bounty template:
Enabling to change number of casted votes by sending a withdrawal transaction. Realisation via input field that allows a number between 0 and the previously casted votes on the proposal. Pressing the submit button will send the quadratic votes to the trash bins and return the voice credits to the user.

Scope

input validation (number between 0 and the previously casted votes)
building and sending the transaction
success screen if transaction included
error screen if transaction failed

Deliverables

code
unit tests

Roles
bounty gardener: @MaxKuck   / 10%
bounty worker: @MaxStalker  / 70%
bounty reviewer: @johannbarbie  / 20%

Implemented in #14 and #84


Remove version: StrawberryMusicPlayer.Strawberry version 1.0.21

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/strawberrymusicplayer/strawberry/releases/download/1.0.21/StrawberrySetup-1.0.21-msvc-x64.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 911)


Animated Splash Screen

Hey,I have noticed that this app doesn't have splash screen . If you want I can add animated Splash screen to the app

Hey, I want to work on it.


Fix nettop invoke task path

What does this PR do?
The path to the nettop code in the invoke task was incorrect.
Motivation
Fixes #7747
Checklist

[x] A release note has been added or the changelog/no-changelog label has been applied.

should we get rid of this nettop altogether?

Probably? I've never used it as a debugging tool.


This adds a github issue template.

As title.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2880   +/-   ##
=======================================
  Coverage   86.08%   86.08%
=======================================
  Files         324      324
  Lines       67687    67687
  Branches     7612     7612
=======================================
  Hits        58267    58267
  Misses       8201     8201
  Partials     1219     1219


Feature Request: Auto-close JSX tags when typing </

There are packages for atom that auto-close HTML / XML tags when typing </. This package sounds like a  good place for such a feature for JSX.

Hi! Thanks for your feedback. Adding this is in the plans among other emmet type features. I'll most likely get started during the weekend.

Would love to see this in place! Thanks for the great plugin.

Hi! Had some extra time today so I decided to move forward with this feature.

So instead of auto-closing closing tags I decided to take similar approach that atom takes with curly braces. Everyone okay with it?

Just released a new version that contains it. Closing for now.

It is not working with coffescript (.cjsx files)...

@maxguzenski have you found a package for cjsx? My googling is coming up short

@benrlodge  I dont remember, I've ported everting to es6


UI5 SDK: Change version redirects to 404

OpenUI5 version: 1.96.0
Browser/version (+device/version): Edge (latest)
Any other tested browsers/devices(OK/FAIL): FAIL Firefox (latest)
URL (minimal example if possible): https://openui5.hana.ondemand.com/
Steps to reproduce the problem:

Open https://openui5.hana.ondemand.com/
Change version, select any, press Apply
Page "https://openui5.hana.ondemand.com/null" is opened

What is the expected result?
Page with proper version is opened.
What happens instead?
Page "https://openui5.hana.ondemand.com/null" is opened

Strange, I can't reproduce it, neither in Edge nor in Chrome, nor Safari, nor in Firefox.

Hello @whydrae, @codeworrior,
The issue is reproducible with all browsers. From the "Change Version" dialog press over a version, but don't choose a patch from the nested side menu, then press "Apply".
Thank you for sharing this finding. I've created an internal incident<PHONE_NUMBER>.
The status of the issue will be updated here in GitHub.
Regards,
Boyan


Stage filter by transport removes link

In the stage screen if you filter by transport the matching lines have the transport id highlighted. If you then remove the filter the highlight is removed but the link to navigate to the transport does not reappear. It's just text then.

Maybe this logic doesn't correctly account for the A tag inside of a span?
https://github.com/abapGit/abapGit/blob/3defa2c1654837d9845faaf925c3d8554940753d/src/ui/zabapgit_js_common.w3mi.data.js#L669-L691
https://github.com/abapGit/abapGit/blob/d47c72f429d080f7c6697b4907fd24b45c7f58f8/src/ui/zcl_abapgit_gui_chunk_lib.clas.abap#L950-L962


The display problem of jupyter notebook running results

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.77.3 (user setup)
OS Version: Windows_NT x64 10.0.22621

Steps to Reproduce:

Run a jupyter notebook cell and wait for it to print a line or lines
Slide the jupyter notebook page to other units while no output is complete
After running the output completely, slide the page to view the result and find that the printing is incomplete

I demonstrated this process using a video:
https://user-images.githubusercontent.com/24618122/234637810-de16627e-555e-49a1-ba67-9d7284f24ea0.mp4

@Zolliner thanks for the report, this issue should now be fixed in latest Insiders and be available in next Stable build (shipped next week)


Binary types

Should the Arrow binary types sit under a binary data type category?
Should we add a content encoding property for strings, to specify whether they contain binary data encoded in plain text, and which encoding is used?
Do we need to support binary types at all at this point?
Other considerations?

@calumabarnett Looks like a binary type category is indeed needed? https://github.com/moj-analytical-services/etl_manager/pull/140

@tmpks yes, we've just come across some binary data in Delius


Inconsistent return type widening

Reported on DT.
Why would these two declarations produce different types?
declare function useState1<S>(initialState: (S extends (() => any) ? never : S) | (() => S)): S; // No args
declare function useState2<S>(initialState: (S extends ((...args: any[]) => any) ? never : S) | (() => S)): S; // Any args

const func1 = useState1(() => () => 0); // () => number
const func2 = useState2(() => () => 0); // () => 0

Playground
It seems to have been like this since 3.6, so not urgent - just perplexing.

@DanielRosenwasser offered a tweaked version that behaves more intuitively.

The tweaked version was intended to see if we could "capture" where things are going wrong, I just feel more confused now though 😅

useState1 silently creates a circularity during instantiation that makes the return position contextually typed by any rather than itself, which it otherwise would be typed by; useState2 avoids this because it hits our isAnySignature fast path in compareSignaturesRelated and thus avoids recursively pulling on the return type of the target signature.
It's pretty easy to expand that fast path to avoid pulling on the source signature return when the target signature return is any - in fact, we already do so for void returns. Now, that makes both cases consistent - in fact, it also makes the other two examples from the playground @amcasey linked also fall into line with a literal 0 return type.
Now, is that prefered? Ehhh

I have another change I can layer on top of #47306 if we'd like to fix the broader issue of signatures contextually typing themselves and creating literal return positions; it just has more (imo, intended) fallout in our existing baselines.


evaluate whether we should use a different cluster setup, like k3d or if kind is alright

What is this issue about? Use Kind for ... what?

What is this issue about? Use Kind for ... what?

It is regarding the test setup in the CI workflow (we're installing and testing the lifecycle toolkit on each PR)

This was a placeholder which is not needed anymore. I am closing this.


URL output format

Hi there,
I intend to use this plugin for versioning purposes:
Master
serverless --stage dev
Versions
serverless --stage dev --alias v1
serverless --stage dev --alias v2
Endpoints
https://{{domain}}.execute-api.eu-west-1.amazonaws.com/{{alias}}/resource
My question is, can I customize the alias format so it becomes with the plugin?
https://{{domain}}.execute-api.eu-west-1.amazonaws.com/api/{{alias}}/resource
Note the /api/ before the {{alias}}
@Thanks!

As the alias is technically an API Gateway Stage, the way to achieve your layout is, to add a custom domain mapping to your APIG API.
Select "Custom domain names" in the AWS APIG console. Then add an URL mapping (e.g. custom domain mysvc.domain.com and configure a path like this: /api -> your-api:*.
Then you can access the different aliases as: mysvc.domain.com/api/<alias>/resource...

ok, sounds like a plan.
Do you think its a good idea to use it for versioning?

You can try. We use it currently at work for developer branch deployments, but using it for versioning also sounds good. Please give it a try and let me know if it works for that use case. Maybe I cann add this use case to the README later.

Ok.
I also need something like this, I'd like to hear some opinions :)
https://github.com/serverless/plugins/issues/79
Thanks for your time


[BEAM-3099] Split out BufferedReader and BufferedWriter from gcsio.

Most of the code in filesystemio.py is copied verbatim from gcsio.py.
The Downloader and Uploader classes are new.

Rebased commits into one.
@chamikaramj could you please merge?


Add support for configuring pools in airflow

Signed-off-by: Stijn De Haes<EMAIL_ADDRESS>
What this PR does / why we need it:
This adds support for configuring pools through the airflow charts. This is done similar to airflow variables.
Which issue this PR fixes

none

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

@gsemet If you have time to review this. It would be greatly appreciated.

Looks great ! Thanks !

/lgtm

@gsemet I just pushed a small update to the docs that you can only use this feature from airflow 1.10.2

/lgtm cancel
Ok and can you check to fix the small issue in the minikube example ?

Link to the issue : https://github.com/helm/charts/issues/12042#issuecomment-486991038

/lgtm cancel
Ok and can you check to fix the small issue in the minikube example ?

Not sure what issue you mean. I have also removed the pools config from the minikube example. I could upgrade the minikube example to airflow 1.10.2. Then I can add the pool config again.

@gsemet Should I also upgrade it to use airflow 1.10.2 by default?

Sure you can do it !

@gsemet Ok it's ready for another review :)
I upgraded to 1.10.2
Fixed the default values for minikube by removing the ingress specification
And added support for configuring pools
I tested this locally on minikube

Yes there has been another PR today (often they come in group, weirdly). Can you rebase and change the version please ?

@gsemet I have rebased and resolved conflicts :)

Thanks for your PR. I am approving it but wonder how would you start several pools ? And say n workers for pool A and m for pool B?

/lgtm

@gsemet An example of a config with 2 pools:
{
    "first pool": {
        "description": "This is an example of a first pool with 2 slots",
        "slots": 2
    },
    "second pool": {
        "description": "This is a second pool with 1 slot",
        "slots": 1
    }
}

I hope this makes it clear. The json syntax looks a bit weird I think, but this is how it works in airflow.

I never actually used it. And how do you associate slot with tasks ? Per operator ?

The documentation actually does a pretty good job of explaining it: https://airflow.apache.org/concepts.html#pools
In short you assign a task to a pool. And only as much tasks as the pool allows are run in parallel. You use this to be sure you don't overwhelm a system.

 When you assigned a task to a non existing pool the scheduler used to crash. But this is fixed somewhere in the 1.10.x series. Can't remember the correct version. Now it just never schedules your task which can be confusing. But better then a crashing scheduler ;)


Proxy does not work

Try to download https://github.com/vporton/passport-client-dfinity/tree/43b400d2651376cddbc1fdc43d81e0e940016603 and then run

dfx deploy backend
curl 'http://localhost:3000/api/v2/canister/gf4a7-g4aaa-aaaaa-qaarq-cai/call' -H 'Accept: */*' -H 'Accept-Language: en-GB,en-US;q=0.9,en;q=0.8,ru;q=0.7' -H 'Connection: keep-alive' -H 'Content-Type: application/cbor' -H 'Cookie: next-auth.session-token=eyJhbGciOiJkaXIiLCJlbmMiOiJBMjU2R0NNIn0..sKP0dzSTrnvUjXrg.VMxTpMyD3nLAeZUGjJmpr7IIo7J89EG_YYdM6M7QVr0PazVVvxIBWuefkcMjbQQcbWg0a-adSvgf-SFumUsjIiZu1w9XbTPGSd7FjQJfIWkAExDInPBA3mlTjzdCzeVru37uz-VYFxspkKC9nxzMOyWSspDF_mOKw4teDwEHajuKIiPiDhwjh6xF9UWlk8DhX8h-joylIngq_eQxj71IbGSc_Ya8H2mcg1oeJK81XY6VxLCo6dNjmap8xXCoTIm49iuADl2MS3pY7MItutTF0_LT3wwy90vj2IL5I8K_WRaHfti1NKZ-YTzH5kLz5uJJB4HlFEtOHREP36iCj4iTr97dfP3FqoEyWo7jA34Lt-HC.wtwGsAE0nfnLpXeLP0zQXQ' -H 'Origin: http://localhost:3000' -H 'Referer: http://localhost:3000/' -H 'Sec-Fetch-Dest: empty' -H 'Sec-Fetch-Mode: cors' -H 'Sec-Fetch-Site: same-origin' -H 'User-Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36' -H 'sec-ch-ua: "Not_A Brand";v="8", "Chromium";v="120"' -H 'sec-ch-ua-mobile: ?0' -H 'sec-ch-ua-platform: "Linux"' --data-raw $'ÙÙ÷¡gcontent§cargXÆDIDL\u0001l\u0002øÅ®«\u0001q´ãè\u0009q\u0001\u0000\u0084\u00010x3ee0dc43440e1132aa506de9dfa547ab0f69fc2a6d22d535a1bae3502699f28257410511f4830aec198df2676480ed18e0e1b8d9c5a18a4e5e4c0534c0cabfe91b*0xF321940F3D1599a1336CA33f08015dB52eDB0A14kcanister_idJ\u0080\u0000\u0000\u0000\u0000\u0010\u0000#\u0001\u0001ningress_expiry\u001b\u0017ª¾*Ïõ\u0008\u0000kmethod_namex\u001cscoreBySignedEthereumAddressenoncePXa±õ&\u0015Ts\u0096|>\u0011²¶ïxlrequest_typedcallfsenderA\u0004'

The second command is copied from the browser “Export to cURL”.
<!DOCTYPE html>

<meta charset="utf-8">
<title>Error</title>

Cannot POST /api/v2/canister/gf4a7-g4aaa-aaaaa-qaarq-cai/call

$ dfx --version
dfx 0.15.2+rev15.283413df

Queries through http://<IP_ADDRESS>:4943/?canisterId=bw4dl-smaaa-aaaaa-qaacq-cai&id=bkyz2-fmaaa-aaaaa-qaaaq-cai do work.
But in the browser
Cannot POST /api/v2/canister/bkyz2-fmaaa-aaaaa-qaaaq-cai/call

So, (both in the browser, the first is through using http://<IP_ADDRESS>:4943/?canisterId=bw4dl-smaaa-aaaaa-qaacq-cai&id=bkyz2-fmaaa-aaaaa-qaaaq-cai)
POST http://<IP_ADDRESS>:4943/api/v2/canister/bkyz2-fmaaa-aaaaa-qaaaq-cai/call

works and
POST http://localhost:3000/api/v2/canister/bkyz2-fmaaa-aaaaa-qaaaq-cai/call

doesn’t.
More information about this bug at https://forum.dfinity.org/t/troubleshooting-agenthttpresponseerror/23846

Even worse:
Directly after
...
    backend: http://<IP_ADDRESS>:4943/?canisterId=be2us-64aaa-aaaaa-qaabq-cai&id=bkyz2-fmaaa-aaaaa-qaaaq-cai
...

When I try to open http://<IP_ADDRESS>:4943/?canisterId=be2us-64aaa-aaaaa-qaabq-cai&id=bkyz2-fmaaa-aaaaa-qaaaq-cai I get:
Could not find a canister id to forward to.

Removing ~/.dfinity did solve Could not find a canister id to forward to. but not the proxy 404 bug.

interesting - the localhost known host bug should have been fixed in 0.20.2. I'll have to figure out what's going on

I realized that the above are usual response of a react-scripts app running on port 3000.
My app does not know about Replica running locally.
Now I wonder, why something similar wonder in the past.
@krpeacock How to run npm start in such a way that new HttpAgent would connect to correct port, not to the same 3000 port that is used by npm start?

why something similar wonder in the past

Should read "why something similar worked in the past".

Apparently, if we use identity got from client.getIdentity() from await AuthClient.create(props.options!.createOptions) to initialize HttpAgent (new HttpAgent({identity})), then it works (when frontend run from npm start), otherwise, it doesn't.
Still need to investigate.


Access to the path '<source_file_to_transform>' is denied

Hi!
First of all, thank you for making this open source! I'm using it within TFS 2015 (update 4). Just after "get sources" I've tried to transform my web.config and got this:

##[error]Exception calling "Transform" with "4" argument(s): "Access to the path '<Path_to_my_build_agent_workspce><myproject>\web.config' is denied."

I thought it was an "IsReadOnly" issue, so I cleared it before using PowerShell and still with this error. Any ideas what should be the problem? Thank you in advance!

Sorry guys, my bad. My "IsReadOnly Cleaner" script wasn't doing its job correctly.

@adailsonfilho Thank you for getting us know!


Fix MS level filter

The option parser has a bug if there's a space in the MS level range, so [2, 3] is interpreted as only MS level 3, but [2,3] is levels 2 and 3. This is technically a bug in msconvert but it's easier to fix here.

Being able to update test files with CI would be a big time saver.


Fix broken link in README.md

This updates the sizes link from
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/img#attr-sizes to
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/img#sizes

Thank you @nfriedly.


New style bilinear

New style bilinear function to support double backprop.

Would you resolve the conflicts?

Thank you for pointing out, rebased!

LGTM, will merge after CI passes.

The backport is to be made after #4021 will be merged.


chore(main): release 1.0.1

:robot: I have created a release beep boop
1.0.1 (2024-07-15)
Bug Fixes

readme (this is just to trigger release please) (b9deadd)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/benlubas/neorg-se/releases/tag/v1.0.1 :sunflower:


I hope to provide a new command to update the view of vscode in real time (for example, changing the background image of vscode now requires restarting to take effect)

I hope to provide a new command to update the view of vscode in real time (for example, changing the background image of vscode now requires restarting to take effect)

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Bridge request for Pearltrees

Bridge request for Pearltrees

It seems pearltrees.com has a lot of potentially interesting content, but provides no RSS feed.
General information

Host URI for the bridge (i.e. https://github.com): https://pearltrees.com

Which information would you like to see?

The Title + link should be enough.

How should the information be displayed/formatted?

Title, link, empty content

Which of the following parameters do you expect?

[X] Title
[X] URI (link to the original article)
[ ] Author
[ ] Timestamp
[] Content (the content of the article)
[ ] Enclosures (pictures, videos, etc...)
[ ] Categories (categories, tags, etc...)

Options

[ ] Limit number of returned items

Default limit: 5

[ ] Load full articles

Cache articles (articles are stored in a local cache on first request): yes
Cache timeout (max = 24 hours): 24 hours

[X] Balance requests (RSS-Bridge uses cached versions to reduce bandwith usage)

Timeout (default = 5 minutes, max = 24 hours): 5 minutes

http://www.pearltrees.com/chri.schmitt

@aurelg this still interesting?


Themes support. Bugs

Hello, I found some bugs in themes support (I know, it's experimental, but we are born to experiment with code, isn't it? :smile_cat: )

[x] If Menu bar is hidden, through settings, it doesn't appear after pressing Alt nor F10 key
[x] Filter doesn't work
[x] Ctrl+K doesn't bring focus to filter bar

The filter is functional, but very finicky. If you put something in right after process hacker launches, it should work, but it'll get stuck later and you'll have to relaunch the program again.

@dmex Could the erratic filter behavior please be fixed? I rely on the filter a lot, but I also don't like my eyes hurting.

These issues were fixed a while ago but the ticket wasn't updated 👍


Upload Request

I feel like abstract paint this would fit the app. I own all rights to this image and would be happy to distribute it through Doors.
https://collection.cloudinary.com/ddrp4kzcw/a2da3926a74d352fde3126786c94d6f8

thanks for the request, will get it uploaded once i get the update out tonight

uploaded, should be available now :)


Is dgl needed at runtime?

Comment:
As compared to only needed for fitting

Yes https://github.com/choderalab/espaloma/blob/13af90b8158d50730c2c6888dacce8351021f797/espaloma/graphs/utils/read_homogeneous_graph.py#L123-L128 lol


General jumbo frame support/Network Refactor

Description of changes:
We have jumbo frame support via virtio/proxmox, but no support for other device types like e1000e (etc.pp.) or software defined networking.
This PR adds general case jumbo frame support via netplan. To achieve this, I've refactored the networking code a little to unify software defined network devices with additional network devices.
While doing this, I removed the limitation of mutually exclusive DHCP and static IP. It's nonsensical and not in netplan.
I also gave all routes a metric and made the metric of default network 100 and other network devices 200. This is still buggy, but less buggy (and dangerous) than having all interfaces add default routes with a metric of 0. This only worked because netplan applies rules in order and was subject to breaking any time.
In a nutshell, this part of netplan https://netplan.readthedocs.io/en/stable/netplan-yaml/#properties-for-all-device-types is now unified in our API and our templating, making the adding of more features much easier. The end goal is to unify AdditionalNetworkDevices with DefaultNetworkDevice, but that's an API breaking change, so I plan on doing that in v1alpha2.
I've also fixed a couple of bugs on the way.
Testing performed:
Testing is still WIP

checks and e2e passed, merging


Update link in blueprint

There is not really much more to say about this except thanking you for the work :)

You are welcome sir, and thanks for the PR! :)


please document correct import of this library

The readme shows:
import {BingLayer} from '../src'

This is obviously not going to work for users.
This might not be a big problem as long as
import { BingLayer } from 'react-leaflet-bing-v2'

just works. But due to https://github.com/TA-Geoforce/react-leaflet-bing-v2/issues/1 that is not the case for me. So I am wondering if I am importing the wrong way...

BingLayer is now deprecated what should we be using?

you are not wrong we will point this out better in the Readme thank you for pointing it out @barbalex :)


Spread operator results in incorrect types when used with tuples

Bug Report
🔎 Search Terms
generic spread operator tuple tuples
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ.

⏯ Playground Link
Playground link with relevant code
💻 Code
// ----------------
// PART 1 - CORRECT
// ----------------

type MyTuple = [
    func: (someArg: boolean) => void,
    arg2: number,
];

class MyClass1 {
    myTuples: MyTuple[] = [];

    someMethod() {
        for (const myTuple of this.myTuples) {
            const [callback, ...args] = myTuple;
            // Look at the type of "args".
            // It is correctly narrowed.
        }
    }
}

// ------------------
// PART 2 - INCORRECT
// ------------------

// This is the same as part 1, but we make it slightly more complicated with
// an interface that maps MyEnum to tuples (instead of just using a single tuple).

enum MyEnum {
    Value1,
    Value2,
}

interface MyParameters {
    [MyEnum.Value1]: [
       func: (someArg: boolean) => void,
       arg2: number,
    ];
    [MyEnum.Value2]: [
        func: (someArg: string) => void,
        arg2: symbol,
    ];
}

class MyClass2<T extends MyEnum> {
    myTuples: Array<MyParameters[T]> = [];

    someMethod() {
        for (const myTuple of this.myTuples) {
            const [callback, ...args] = myTuple;
            // Look at the type of "args".
            // It should be "number | symbol",
            // but it instead incorrectly includes the function types.
        }
    }
}

The type of the variable attached to the spread operator is incorrect.
Furthermore, we can see that the bug is with the spread operator specifically, since accessing the tuple by a specific index returns the correct type.
Other Issues
This is possibly related to #49802.

Here's an example without enums or classes that has the same issue:
const test = <N extends number>(singletons: (['a'] | ['b'])[], i: N) => {
  const singleton = singletons[i]
  // const singleton: (["a"] | ["b"])[][N]

  // Expected
  const [c] = singleton
  // const c: "a" | "b"
  // @ts-expect-error: (Property '1' does not exist on type '["a"] | ["b"]'.(2339))
  const [, err] = singleton

  // Slightly unexpected
  const [,] = singleton // type is no longer generic here
  // const singleton: ["a"] | ["b"]
  const ___ = singleton // but here it is generic again
  // const singleton: (["a"] | ["b"])[][N]

  // Unexpected
  const [, ...rest] = singleton
  // const rest: ("a" | "b")[]
  // Expected: const rest: []
}

TypeScript playground

Even teenier of a repro for that last // Unexpected :smile::
const test = <N extends number>(singletons: ['a'][], i: N) => {
  const singleton = singletons[i]
  //    ^?

  // Unexpected
  const [, ...rest] = singleton
  //           ^?
  // Expected 👎: const rest: []
}

TypeScript playground


update roadmap for 0.4.0

The next release we want to focus on stability and improvements to the local providers.  These are the focus points:

Finalize Boot2Docker Migration (OS)
Provisioning Stability (error reporting, re-provisioning existing hosts)
Local Provider Improvements (make VirtualBox, Fusion, Hyper-V extremely stable)

/cc @bfirsh @samalba @nathanleclaire @hairyhenderson @sthulb @efrecon

+1 I can get behind this :)

Nice and focused - LGTM!

:+1:
Some specific callouts I'd like to see (might fill in more as I remember them):

OSX/Windows installer (preferably unified with KM / Compose etc.), like what b2d has today
Improved per-driver logging.  I'd like to see a structure where machine operations log their debug output to the relevant folder on disk, whether or not they are displayed to the terminal when they are happening, and can be queried later with something like a docker-machine logs dev command.

@nathanleclaire +1

Another:

A command (or script) which will generate information about the host system and environment for debugging purposes, similar to the one that Docker has.

This focuses on stability and this is good. If you were to consider new features, I would suggest a way to specify labels to attach to created machines so that swarm can make use of these labels for orchestration.

This focuses on stability and this is good. If you were to consider new features, I would suggest a way to specify labels to attach to created machines so that swarm can make use of these labels for orchestration.

@efrecon I'm not sure if this is exactly what you're after, but in Machine master today you can use --engine-label to do this, e.g.:
$ docker-machine create -d amazonec2 --engine-label disk=ssd --engine-label size=micro foobar

It will set the labels on the Docker daemon, you should be able to see them using docker info.

Thank you @nathanleclaire ! It is what I was looking for, I had missed that option, so disregard my comment...


[question] what is the best way to export database?

Hello! Thank you for this library!
await Share.open({
    url: db.getDbPath(),
    failOnCancel: false,
    type: 'application/x-sqlite3',
});

I'm trying to export db file but .sqlite file does not contain relevant data. Do you have any ideas?
I tried using pragmas but it did not help
await db.executeRawAsync(`PRAGMA synchronous = FULL`);

I’m experiencing a similar issue with an empty database file when attempting to export it, even after closing the database. If I restart the application and try exporting the file again, the database is no longer empty.
It seems that the data is not being persisted to the filesystem until the application is fully closed or unloaded.
Has a solution been identified for this issue?

Seems unlikely that db is not persisted, you probably want to copy it to the documents folder before sharing though, since the file system is sandboxed it might be refusing to just read from the default location.

You guys might be doing something wrong, just tested this:
const shareDb = async () => {
    try {
      const db = open({
        name: 'shareableDb.sqlite',
      });

      await db.execute(
        'CREATE TABLE IF NOT EXISTS test (id INTEGER PRIMARY KEY, name TEXT)',
      );
      await db.execute("INSERT INTO test (name) VALUES ('test')");
      const res = await db.execute('SELECT * FROM test');
      console.log(res);

      await db.close();
      await Share.open({
        url: 'file://' + db.getDbPath(),
        failOnCancel: false,
        type: 'application/x-sqlite3',
      });
    } catch (e) {
      console.log(e);
    }
  };

And the file on my mac

It turns out I was indeed making a mistake. I'm using PRAGMA journal_mode = WAL, and after carefully reviewing the related documentation, I added a manual checkpoint to force synchronization with the following command: PRAGMA wal_checkpoint(FULL).


unable to access /sd

I'm trying to use rshell with my esp32, loaded with MicroPython_ESP32_psRAM_LoBo.
Mounting the SDCard works fine:
rshell -p /dev/ttyS12 -b 115200 -n
Using buffer-size of 32
Connecting to /dev/ttyS12 (buffer-size 32)...
Trying to connect to REPL  connected
Testing if ubinascii.unhexlify exists ... Y
Retrieving root directories ... /flash/
Setting time ... Mar 30, 2019 14:35:19
Evaluating board_name ... pyboard
Retrieving time epoch ... Jan 01, 1970
Welcome to rshell. Use Control-D (or the exit command) to exit rshell.
/home/_me_/MicroPython_ESP32_psRAM_LoBo> repl
Entering REPL. Use Control-X to exit.
>
MicroPython ESP32_LoBo_v3.2.24 - 2018-09-06 on ESP32 board with ESP32
Type "help()" for more information.
>>>
>>> import uos
>>> uos.sdconfig(uos.SDMODE_1LINE)
>>> uos.mountsd()
---------------------
 Mode:  SD (1bit)
     Name: 00000
     Type: SDSC
    Speed: high speed (50 MHz)
     Size: 971 MB
      CSD: ver=0, sector_size=512, capacity=1989632 read_bl_len=9
      SCR: sd_spec=2, bus_width=5

>>> uos.listdir('/sd')
[]
>>> uos.listdir('/flash')
['aenum.mpy', 'boot.py', 'collections2.mpy', 'examples', 'fonts', 'lib', 'linux_metrics.mpy', 'main.py', 'paho.mpy', 'pint.mpy', 'spiffs.info', 'www']
>>>

Then when I return from repl with Ctrl-x, I am able to see /flash, but unable to see /sd:
/home/_me_/MicroPython_ESP32_psRAM_LoBo> ls
Documents/         MicroPython_BUILD/ Tools/             liota/
LICENSE.md         README.md          get-pip.py
/home/_me_/MicroPython_ESP32_psRAM_LoBo> ls /flash
aenum.mpy         collections2.mpy  fonts/            linux_metrics.mpy paho.mpy          spiffs.info
boot.py           examples/         lib/              main.py           pint.mpy          www/
/home/_me_/MicroPython_ESP32_psRAM_LoBo> ls /sd
Cannot access '/sd': No such file or directory

So I see the local content, I see /flash, but I cannot see /sd.
I tried to restart rshell, but then it fails to load:
rshell -p /dev/ttyS12 -b 115200 -n
Using buffer-size of 32
Connecting to /dev/ttyS12 (buffer-size 32)...
Trying to connect to REPL . connected
Testing if ubinascii.unhexlify exists ... b'ets Jun  8 2016 00:22:57\r\n\r\nrst:0xc (SW_CPU_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)\r\nconfigsip: 0, SPIWP:0xee\r\nclk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00\r\nmode:DIO, clock div:2\r\nload:0x3fff0018,len:4\r\nload:0x3fff001c,len:4820\r\nload:0x40078000,len:8764\r\nload:0x40080000,len:5748\r\nentry 0x400802dc\r\n\r\nInternal FS (FatFS): Mounted on partition \'internalfs\' [size: 2162688; Flash address: 0x1F0000]\r\n----------------\r\nFilesystem size: 2101248 B\r\n           Used: 323584 B\r\n           Free: 1777664 B\r\n----------------\r\nMicroPython ESP32_LoBo_v3.2.24 - 2018-09-06 on ESP32 board with ESP32\r\nType "help()" for more information.\r\n>>> '
Traceback (most recent call last):
  File "/home/_me_/.local/lib/python3.5/site-packages/rshell/main.py", line 1255, in connect
    ip_address = socket.gethostbyname(port)
socket.gaierror: [Errno -2] Name or service not known

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/_me_/.local/bin/rshell", line 10, in <module>
    sys.exit(main())
  File "/home/_me_/.local/lib/python3.5/site-packages/rshell/command_line.py", line 4, in main
    rshell.main.main()
  File "/home/_me_/.local/lib/python3.5/site-packages/rshell/main.py", line 2843, in main
    real_main()
  File "/home/_me_/.local/lib/python3.5/site-packages/rshell/main.py", line 2805, in real_main
    connect(args.port, baud=args.baud, wait=args.wait, user=args.user, password=args.password)
  File "/home/_me_/.local/lib/python3.5/site-packages/rshell/main.py", line 1261, in connect
    connect_serial(port, baud=baud, wait=wait)
  File "/home/_me_/.local/lib/python3.5/site-packages/rshell/main.py", line 1285, in connect_serial
    dev = DeviceSerial(port, baud, wait)
  File "/home/_me_/.local/lib/python3.5/site-packages/rshell/main.py", line 1565, in __init__
    Device.__init__(self, pyb)
  File "/home/_me_/.local/lib/python3.5/site-packages/rshell/main.py", line 1339, in __init__
    unhexlify_exists = self.remote_eval(test_unhexlify)
  File "/home/_me_/.local/lib/python3.5/site-packages/rshell/main.py", line 1459, in remote_eval
    return eval(self.remote(func, *args, **kwargs))
  File "/home/_me_/.local/lib/python3.5/site-packages/rshell/main.py", line 1437, in remote
    self.pyb.enter_raw_repl()
  File "/home/_me_/.local/lib/python3.5/site-packages/rshell/pyboard.py", line 187, in enter_raw_repl
    raise PyboardError('could not enter raw repl')
rshell.pyboard.PyboardError: could not enter raw repl

When I try again, it starts but still no /sd:
rshell -p /dev/ttyS12 -b 115200 -n
Using buffer-size of 32
Connecting to /dev/ttyS12 (buffer-size 32)...
Trying to connect to REPL  connected
Testing if ubinascii.unhexlify exists ... Y
Retrieving root directories ... /flash/
Setting time ... Mar 30, 2019 14:52:06
Evaluating board_name ... pyboard
Retrieving time epoch ... Jan 01, 1970
Welcome to rshell. Use Control-D (or the exit command) to exit rshell.
/home/_me_/MicroPython_ESP32_psRAM_LoBo> ls /flash
aenum.mpy         collections2.mpy  fonts/            linux_metrics.mpy paho.mpy          spiffs.info
boot.py           examples/         lib/              main.py           pint.mpy          www/
/home/_me_/MicroPython_ESP32_psRAM_LoBo> ls /sd
Cannot access '/sd': No such file or directory
/home/_me_/MicroPython_ESP32_psRAM_LoBo>

This is happening because rshell only scans the root directories once. I'll see about adding a command to re-scan

I also figured out that this works:
ls /pyboard/sd


add compound with 2019/05 schema

Hacky but works with old schema

Thanks @jzaki for your review!


tests: reduce the poll interval for Eventually and Consistently

It should be perfectly acceptable to poll every 10ms without exerting
too much cpu load on the test systems. This seems to make the tests run
in less than 1/3rd time than before.
Signed-off-by: Raghavendra Talur<EMAIL_ADDRESS>

BEFORE
Using cached envtest tools from /home/rtalur/Code/go/src/github.com/red-hat-storage/ramen/testbin
setting up env vars
?       github.com/ramendr/ramen        [no test files]
?       github.com/ramendr/ramen/api/v1alpha1   [no test files]
ok      github.com/ramendr/ramen/controllers    78.592s coverage: 61.4% of statements
ok      github.com/ramendr/ramen/controllers/util       0.058s  coverage: 16.9% of statements

AFTER
Using cached envtest tools from /home/rtalur/Code/go/src/github.com/red-hat-storage/ramen/testbin
setting up env vars
?       github.com/ramendr/ramen        [no test files]
?       github.com/ramendr/ramen/api/v1alpha1   [no test files]
ok      github.com/ramendr/ramen/controllers    27.917s coverage: 61.5% of statements
ok      github.com/ramendr/ramen/controllers/util       0.055s  coverage: 16.9% of statements


Force process.stdout to be a TTY to support spinners

I tested this PR on Windows and I made several changes in order to make it work: https://github.com/dennisjlee/lint-staged/commit/98d1dbffd7e609f1d42767c2e8b4a9bfc1581e24
@iiroj should I send a PR from my fork?

@dennisjlee yeah, please do! Try to keep the code async if possible, though. I see you replaced some methods with the sync versions.

@iiroj I tried to keep it async but I ran into some blockers - I'll open the PR and explain there

I sent https://github.com/okonet/lint-staged/pull/1181 as a PR targeting this fix-renderer branch

Is this being worked on? This is blocking me from using lint-staged. I appreciate the work you have put down into this! 🌟

Hello, the problem was caused by git and fixed (at least for me) by a later git update.
Since I don't have a Windows environment, I haven't put more effort into this.

Is this being worked on? This is kind of blocking me from using lint-staged. I appreciate the work you have put down into this! 🌟
Here are some results and info:

I found this issue from https://stackoverflow.com/questions/72579519/multiple-run-lint-staged-when-use-with-husky
I have tried with exec >/dev/tty 2>&1 and it works as long as you commit from the terminal, but if you commit from for example VS Code, it doesn't work.

For me it runs crazy many times, this list about 5-10 times longer than what I've screenshot here.

I had a pending modification to this PR in #1181 but unfortunately I haven't been able to revisit this for a long time.
However, I did try upgrading my Git for Windows to the latest (v2.38.1, from v2.33.0) and that did not resolve the issue for me.
@iiroj if I have time to revisit #1181, would you want me to still land that on top of your fix-renderer branch, or would it make sense to combine #1181 and #1163 into a separate PR?

@dennisjlee feel free to try out whatever. 👍 Keeping a single PR open is easiest since I have to manually trigger pipelines.

Thank you @dennisjlee! I appreaciate it a lot! 😊

Is this PR still being worked on?

Closing as the original issue should be already be fixed in more recent git versions.


[Spike] Configs in schema work

This is in reference to issue https://app.zenhub.com/workspace/o/fishtown-analytics/dbt/issues/2401
Goal/Exit Criteria: Take 2-3 days to dig into the issue, identify a plan for what changes need to be made and the time it would take so that we can make a decision on whether to further pursue it. A list of the difficult/unknown items we would need to implement.

I know that #2401 has a lot going on in it, and it reads more as a conversation than a set of requirements. So I'll quickly summarize the thrust of this larger initiative below.

User stories
As a dbt user, I shouldn't have to think about what's a config and what's a property, or whether something can go in dbt_project.yml vs. in-file config() vs. everything_else.yml. I should be able to define these things wherever & however I want.
Beyond a convenience, this becomes crucial if I want to override properties defined in a package, since I can't edit the yaml files directly—I can only override configs in dbt_project.yml.
As the team maintaining dbt, this also gives us the opportunity to significantly clean up the codebase around config inheritance and property patching, which are two very different things today. It also gives us the opportunity to remove some rationalizations from the docs.
Examples
Say I'm creating a model, and I want to add a description. Why not:
# models/my_model.sql

{{ config(description = 'This is my model') }}

select 1 as fun

Alternatively, if I hate the kwarg-y config() syntax, I could set all my model's config-properties in yaml instead:
# models/schema.yml
version: 2

models:
 - name: my_model
   config:
      materialized: view
      bind: false
   meta: ...  # for historical reasons, we'll need to support certain keys here

Or say I'm configuring the location of sources, and I want to override all the sources in my project. Instead of manually overriding each one, setting it to a dynamic value, or using a var, I could simply do this in dbt_project.yml:
models:
  +database: ci_db   # possible today (config)

sources:
  +database: ci_db   # not possible today (property)

If we're already going to go through the trouble of adding config: dictionaries, to make this syntax more consistent, we could also offer it as an alternative to the + prefix, which many newer users find confusing:
models:
  config:
    database: ci_db

sources:
  config:
    database: ci_db

Complexities

This gets into some very thorny code, where I haven't spelunked much personally. There's prior art in the form of tags, which are the only config-property hybrid today.
What to do about overlap? If a user sets the same config multiple times at the same "specificity level," i.e. in a model's config() and also in its yaml properties, I think we should raise an error.
Let's figure out a sensible way to represent this in manifest.json. Configs show up in the manifest within node.configs, whereas properties are node-level keys.
Let's be choosy in what we support as a node selection method (config.kwarg:value). I don't think selecting models based on config.description: makes a lot of sense!

Moving forward


Coordinator: giving up waiting for unused V8 context when under load

my environment running ArangoDB
I'm using the latest ArangoDB of the respective release series:

[x] 3.2

Mode:

[x] Cluster

Storage-Engine:

[x] rocksdb

On this operating system:

[x] Linux

[x] docker - official docker library
[x] other: I've also used a custom self-built Arango docker image (using the official .deb packages under the hood) but can reproduce this issue with both the official version and the custom build

Hello,
I've been running some load tests against a three node (an instance of agency, coordinator and primary on each) cluster. This is done using an application reading and writing into arango (using the Java driver) rather than arangobench. As part of the load test a few operations are performed - to the best of my knowledge it's two documents being created, one of them being read and subsequently also updated. This cycle is performed concurrently by many application threads and the end goal of the test is to get to 1mln documents in two collections (1mln documents in each one of them). So far we've been having much better results with the 3.2 series using RocksDB but we're currently running into an issue every few hours where the coordinator node stops responding.
At the time the coordinator starts acting up we start seeing messages like these in the logs:
arangodb-coordinator_1          | 2017-09-19T23:59:48Z [16] WARNING {memory} process is near the maximum number of memory mappings. current: 64507, maximum: 65530
arangodb-coordinator_1          | 2017-09-20T00:00:08Z [16] WARNING {memory} process is near the maximum number of memory mappings. current: 64551, maximum: 65530
arangodb-coordinator_1          | 2017-09-20T00:00:18Z [16] WARNING {memory} process is near the maximum number of memory mappings. current: 64539, maximum: 65530
arangodb-coordinator_1          | 2017-09-20T00:00:28Z [16] WARNING {memory} process is near the maximum number of memory mappings. current: 64539, maximum: 65530
arangodb-coordinator_1          | 2017-09-20T00:00:38Z [16] WARNING {memory} process is near the maximum number of memory mappings. current: 64539, maximum: 65530
arangodb-coordinator_1          | 2017-09-20T00:00:49Z [16] WARNING {memory} process is near the maximum number of memory mappings. current: 64539, maximum: 65530
arangodb-coordinator_1          | 2017-09-20T00:00:59Z [16] WARNING {memory} process is near the maximum number of memory mappings. current: 64539, maximum: 65530
arangodb-coordinator_1          | 2017-09-20T00:01:08Z [16] WARNING {v8} giving up waiting for unused V8 context after 60.000000 s
arangodb-coordinator_1          | 2017-09-20T00:01:08Z [16] WARNING {v8} giving up waiting for unused V8 context after 60.000000 s
arangodb-coordinator_1          | 2017-09-20T00:01:08Z [16] WARNING {v8} giving up waiting for unused V8 context after 60.000000 s
arangodb-coordinator_1          | 2017-09-20T00:01:08Z [16] WARNING {v8} giving up waiting for unused V8 context after 60.000000 s
arangodb-coordinator_1          | 2017-09-20T00:01:09Z [16] WARNING {memory} process is near the maximum number of memory mappings. current: 64540, maximum: 65530
arangodb-coordinator_1          | 2017-09-20T00:01:09Z [16] WARNING {v8} giving up waiting for unused V8 context after 60.000000 s
arangodb-coordinator_1          | 2017-09-20T00:01:09Z [16] WARNING {v8} giving up waiting for unused V8 context after 60.000000 s
arangodb-coordinator_1          | 2017-09-20T00:01:09Z [16] WARNING {v8} giving up waiting for unused V8 context after 60.000000 s
arangodb-coordinator_1          | 2017-09-20T00:01:09Z [16] WARNING {v8} giving up waiting for unused V8 context after 60.000000 s
arangodb-coordinator_1          | 2017-09-20T00:01:09Z [16] WARNING {v8} giving up waiting for unused V8 context after 60.000000 s
arangodb-coordinator_1          | 2017-09-20T00:01:09Z [16] WARNING {v8} giving up waiting for unused V8 context after 60.000000 s
arangodb-coordinator_1          | 2017-09-20T00:01:09Z [16] WARNING {v8} giving up waiting for unused V8 context after 60.000000 s

This looks very similar to #2324 so I've added some v8 tweaks on the command line:
--javascript.v8-contexts 5 --javascript.v8-max-heap 3072
but unfortunately this doesn't seem to make much difference. This issue usually starts occurring after 5-6 hours of testing and the v8 flags don't seem to influence this timing much.
Here's what the memory usage looks like for this particular coordinator node:

Here's the CPU usage:

(the above are container metrics obtained using cAdvisor)
Here's the HTTP requests served by the server nodes:

(these are obtained from the Arango API - different colours are different HTTP verbs)
As you can see there's a very slight jump in CPU usage at the same time http requests stop being forwarded to the server and afterwards the memory usage starts growing. I'm happy to share the full Grafana dashboard privately if this helps!
After restarting the affected coordinator node the load test goes straight back to normal - the application picks another coordinator node and continues processing.

sorry - the enter key seems to have created an issue instead of a newline! updating the contents now.

We recently found that it may help in this context to increase the value of a kernel parameter, as specified here.
The Linux default value for /proc/sys/vm/max_map_count seems to be rather low for a powerful server with many cores and many parallel threads, so increasing it may allow a process (i.e. arangod) to use more memory mappings than the default 64K.
Additionally, the number of V8 contexts (--javascript.v8-contexts) may need to be increased a bit. It seems you set it 5, but this will mean that if there are more than 5 threads in parallel that require the usage of V8, the other threads will block.
Can you try the adjusted settings to see if that fixes the problem?

I had the same issue. Increasing max_map_count, v8-contexts, threads and the frequency of V8 GC it seems solved.
This doesn't happen to us with 3.1.X or prior versions, it started when we migrated to 3.2. Also happened to us in a clean installation.

It took a lot of time but one of our coordinators get the same issue.
2017-09-23T09:01:13Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255044, maximum: 256000
2017-09-23T09:01:23Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255046, maximum: 256000
2017-09-23T09:05:59Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255001, maximum: 256000
2017-09-23T09:06:09Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255009, maximum: 256000
2017-09-23T09:06:19Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255014, maximum: 256000
2017-09-23T09:06:29Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255017, maximum: 256000
2017-09-23T09:06:39Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255021, maximum: 256000
2017-09-23T09:06:49Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255022, maximum: 256000
2017-09-23T09:06:59Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255022, maximum: 256000
2017-09-23T09:07:11Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255022, maximum: 256000
2017-09-23T09:07:25Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255022, maximum: 256000
2017-09-23T09:07:36Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255024, maximum: 256000
2017-09-23T09:07:49Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255024, maximum: 256000
2017-09-23T09:08:00Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255024, maximum: 256000
2017-09-23T09:08:10Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255021, maximum: 256000
2017-09-23T09:08:20Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255026, maximum: 256000
2017-09-23T09:08:30Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255026, maximum: 256000
2017-09-23T09:09:35Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255026, maximum: 256000
2017-09-23T09:09:45Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255030, maximum: 256000
2017-09-23T09:09:55Z [1] WARNING {memory} process is near the maximum number of memory mappings. current: 255031, maximum: 256000

In this case the config of the coordinator is:
[server]
authentication = true
endpoint = tcp://[::]:8529
threads = 16
jwt-secret = ***
[log]
level = info
[javascript]
v8-contexts = 32
gc-frequency = 10
gc-interval = 128
[database]
index-threads = 4

Thanks @jsteemann - running more tests now after increasing max_map_count (as well as doubling v8 contexts). So far so good but it usually takes about 6hrs before I hit this issue so I'll report once the test is complete later on.
I've read through all the tuning recommendations that you've linked (as well as implemented all of them before re-testing) and I have a couple of questions:

The docs specify a rule of thumb for calculating the max_map_count value - when I follow this the resulting value is lower than the default as I'm running tests on a 2-core system (but with 15G of ram and very fast storage). I've resorted to making it 262144 as this seems to be a very common tweak for Elasticsearch as well and that's the value a lot of articles seem to recommend. Should 256k be the de-facto recommended value regardless of the number of cores?
There seems to be a bit of a discrepancy in the overcommit_memory setting - the docs specify this should be 0 however looking at the logs of the server and coordinator nodes I get this message:
WARNING {memory} /proc/sys/vm/overcommit_memory is set to '0'. It is recommended to set it to a value of 2
Which one is correct?

Thanks for coming back. I have created a PR to change the documentation and to increase the recommended default values.

Thanks @jsteemann! I can confirm now that the load test has finished successfully and I believe max_map_count is what solved the issue here. I'm running a higher throughput test now to verify again but things are looking good.

I can definitely confirm that this problem has been resolved for me now. Thanks for your help @jsteemann!

Ok, Thanks a lot for confirming.

@dothebart in our case logs have disappeared, the process just finish when the map limit is reached (checked with pmap). Should I open other issue? Is the same case :S

You need to increase the available mappings. The current way jemalloc handles this isn't the best way - we're working on analysing whether we find a better way of doing this.
Eventually you may want to start more coordinators to better distribute the mappings across more processes. Please report back if neither of both solves the issue.

@dothebart the mappings allowed in our systems is around 2.5 millions, it take some hours due to the volume of queries we have to reach that value.
The current config is the suggested:
sudo bash -c "echo madvise >/sys/kernel/mm/transparent_hugepage/enabled"
sudo bash -c "echo madvise >/sys/kernel/mm/transparent_hugepage/defrag"
sudo bash -c "echo 0 >/proc/sys/vm/overcommit_memory"
sudo bash -c "echo 0 >/proc/sys/vm/zone_reclaim_mode"
sudo bash -c "sysctl -w 'vm.max_map_count=2500000'"
export GLIBCXX_FORCE_NEW=1

All the config is permanent. Also we have tested with echo 2 >/proc/sys/vm/overcommit_memory and it usually resist more hours. Also we tried with numactl --interleave=all even when our system isn't NUMA enabled, no improvements found with this change.
Currently we have 9 coordinators working, so load balancing between them are the only way to not suffer this bug.
Thanks for all your efforts, I'm pretty sure you will fix it in an elegant way and soon 😃

You should be able to increase the max mapping count even more.
We're currently adding monitoring for this to collectd: https://github.com/collectd/collectd/pull/2454
so we can trace it a bit better throughout loadtests.

That's perfect! I will check it this afternoon :) Thanks

I have a configuration where I have 8 nodes with 8 coordinators and 8 db servers and 3 agents. After reading the above discussion I had set the value of max mapping count = 848000 = 256000.
After 14 days, the memory mapping limit was reached.
I understand that I can increase the memory mappings to a higher value but looks like the problem will then occur at a later time, but definitely occur : O.
Is there a fix for this coming out in 3.3?
Thanks!

Hi,
we recently learned, that the V8 engine included in 3.2 has a memory leak.
We're currently working on 3.2.7 with a fixed version of V8.

3.2.7 is out today, I see that we have increased the number of initial V8 contexts,but I did not see a memory fix for the V8 engine, is that covered? Did I miss anything?

Hi,
Yes the V8 engine has also been upgraded.


move to redis json

would be nice, could also speed up a bunch of stuff such as checking if something is bridged
this would be breaking and require migrations
see https://redis.io/docs/data-types/json and https://github.com/iuioiua/r2d2

also, make prefixes for keys look like <plugin>:<somehting>
(bolt-bridge:id or bolt-guilded:migration-a)


Gantt chart, cannot add data

Currently, in the dev branch, when I navigate to the Gantt Chart view and click 'Add Button'...there is no response.
The console shows these errors:

This behavior still occurs when in an incognito window and after clearing cache.

MT node ID will default to number type if there are no non number characters in the ID. The gantt view was expecting a string type for node id.


Add index routes for 'gf'

At the moment, the gf command in the router will not work if the app uses an index.js rather than .js.
If possible, it would be great if the plugin tested for the index.js variant if the named version does not exist.

I've just pushed a commit that should support this use case as well. Could you try it out and let me know if it works for you?

hey - sorry for the late reply but I've tried the latest commit and this seems to work now. Thanks again for the plugin. Feel free to close.


Refactoring: Improve inconsistent core pickles variable names

As per the Least Authority's audit and subjective experience, it would be a good idea to improve naming of the core Pickles (and Kimchi) variables. This issue is only for the ones that are particularly confusing -- there is much more that can be done.
Examples include (non-exhaustive)

xi/r same as u/v same as polyscale/evalscale -- the u/v can be used in Latex docs, while code can use just polyscale/evalscale. I suggest removing xi/r because it's less descriptive, and also because u/v is already used in the docs + in Plonk paper.
b is just a terribly non-descriptive variable name for a quite a complicated thing (a combined evaluation)
x_hat is the same as public_Inputs and sometimes public_evals. I prefer the longer descriptions in the code, while the latex docs can have \hat{x} if needed.
prev_challenges: PrevChallenges is actually not challenges, but challenges /and/ commitments. It's quite confusing.

Nice ideas


libiscsi: 1.19.0 -> 1.20.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/libiscsi/versions.
meta.description for libiscsi is: iscsi client library and utilities
meta.homepage for libiscsi is: https://github.com/sahlberg/libiscsi
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.20.0 with grep in /nix/store/iqp03wrvsjpmpswhcvyi8ywlqjy99ppq-libiscsi-1.20.0
found 1.20.0 in filename of file in /nix/store/iqp03wrvsjpmpswhcvyi8ywlqjy99ppq-libiscsi-1.20.0

Rebuild report (if merged into master) (click to expand)

79 total rebuild path(s)

79 package rebuild(s)

First fifty rebuilds by attrpath
OVMFFull
colima
cot
diffoscope
guestfs-tools
incus
libguestfs
libiscsi
lima
lima-bin
linuxKernel.packages.linux_4_19.virtualbox
linuxKernel.packages.linux_4_19_hardened.virtualbox
linuxKernel.packages.linux_5_10.virtualbox
linuxKernel.packages.linux_5_10_hardened.virtualbox
linuxKernel.packages.linux_5_15.virtualbox
linuxKernel.packages.linux_5_15_hardened.virtualbox
linuxKernel.packages.linux_5_4.virtualbox
linuxKernel.packages.linux_5_4_hardened.virtualbox
linuxKernel.packages.linux_6_1.virtualbox
linuxKernel.packages.linux_6_1_hardened.virtualbox
linuxKernel.packages.linux_6_6.virtualbox
linuxKernel.packages.linux_6_6_hardened.virtualbox
linuxKernel.packages.linux_6_7.virtualbox
linuxKernel.packages.linux_6_7_hardened.virtualbox
linuxKernel.packages.linux_6_8.virtualbox
linuxKernel.packages.linux_hardened.virtualbox
linuxKernel.packages.linux_latest_libre.virtualbox
linuxKernel.packages.linux_libre.virtualbox
linuxKernel.packages.linux_lqx.virtualbox
linuxKernel.packages.linux_xanmod.virtualbox
linuxKernel.packages.linux_xanmod_latest.virtualbox
linuxKernel.packages.linux_xanmod_stable.virtualbox
linuxKernel.packages.linux_zen.virtualbox
linuxPackages-libre.virtualbox
linuxPackages.virtualbox
linuxPackages_4_19_hardened.virtualbox
linuxPackages_5_10_hardened.virtualbox
linuxPackages_5_15_hardened.virtualbox
linuxPackages_5_4_hardened.virtualbox
linuxPackages_6_1_hardened.virtualbox
linuxPackages_6_6_hardened.virtualbox
linuxPackages_6_7_hardened.virtualbox
linuxPackages_hardened.virtualbox
linuxPackages_latest-libre.virtualbox
linuxPackages_latest.virtualbox
linuxPackages_lqx.virtualbox
linuxPackages_xanmod.virtualbox
linuxPackages_xanmod_latest.virtualbox
linuxPackages_xanmod_stable.virtualbox
linuxPackages_zen.virtualbox

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/iqp03wrvsjpmpswhcvyi8ywlqjy99ppq-libiscsi-1.20.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A libiscsi https://github.com/r-ryantm/nixpkgs/archive/cfbe3ac5418878ad33d7626b852476b105aa748e.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/cfbe3ac5418878ad33d7626b852476b105aa748e#libiscsi

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/iqp03wrvsjpmpswhcvyi8ywlqjy99ppq-libiscsi-1.20.0
ls -la /nix/store/iqp03wrvsjpmpswhcvyi8ywlqjy99ppq-libiscsi-1.20.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 45m and timed out

Maintainer pings
cc @misuzu for testing.

Update with darwin fix: https://github.com/NixOS/nixpkgs/pull/300073


V0.0.3 add a new api methods for searching films/people

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@010ac71). Click here to learn what that means.
The diff coverage is 32.07%.

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   33.33%
  Complexity        ?       65
=========================================
  Files             ?       36
  Lines             ?      258
  Branches          ?        0
=========================================
  Hits              ?       86
  Misses            ?      171
  Partials          ?        1

Impacted Files
Coverage Δ
Complexity Δ

...truedev/kinoposk/api/service/KPApiClientService.kt
66.66% <0%> (ø)
4 <0> (?)

...in/com/truedev/kinoposk/api/model/people/Spouse.kt
11.11% <0%> (ø)
1 <0> (?)

...dev/kinoposk/api/exception/BadResponseException.kt
0% <0%> (ø)
0 <0> (?)

...tlin/com/truedev/kinoposk/api/model/people/Data.kt
11.11% <0%> (ø)
2 <0> (?)

...ruedev/kinoposk/api/service/KinopoiskApiService.kt
100% <100%> (ø)
9 <3> (?)

...v/kinoposk/api/model/search/people/SearchPeople.kt
16.66% <16.66%> (ø)
1 <1> (?)

...m/truedev/kinoposk/api/model/search/people/Data.kt
40% <40%> (ø)
2 <2> (?)

...com/truedev/kinoposk/api/model/search/film/Data.kt
40% <40%> (ø)
2 <2> (?)

...k/api/model/search/people/SearchPeopleResultExt.kt
50% <50%> (ø)
2 <2> (?)

...noposk/api/model/search/film/SearchFimResultExt.kt
50% <50%> (ø)
2 <2> (?)

... and 1 more

Continue to review full report at Codecov.

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


How to update 0.19.0 -> 1.1.0

Hi guys,
What is version 1.x doing?
Can you provide a migration guide or similar?
I was using 0.19.0 before and updated to 1.1.0, but send failed with a 404, so I switched back to 0.19.0.
Cannot find any information.
    send(event.node.req, filePath, {
      cacheControl: true,
      maxAge: '1 year',
      immutable: true
    })
    .pipe(event.node.res)

Thanks!

There's a history file here: https://github.com/pillarjs/send/blob/master/HISTORY.md#100-beta1--2022-02-04
But I don't see anything that can help me debug the snippet you pasted. Do you have the filePath that is now a 404?

/Users/Name/Development/Company Name/website_company/.media-cache/images/w_2000/background_01-8cc5c16c-94b29dfe.jpg
I know there is a space in the path, but I tried to mask it with a backslash \ or encode it, nothing helped.
Also it works in 0.19.0 so I am a bit confused.
Thanks for the quick reply, though!

This is really helpful, it looks like the package could use some updates to the README and release notes. In this case I think it's due to a change in dotfiles:

Remove hidden option -- use dotfiles option

In 0.x it only hid files when the began with a ., but now it hides anything that should be hidden (i.e. if the directory or file starts with a dot). I believe it's the ignore option in the README.
Do you need the ability to access files from hidden directories? Or just hidden files? You could tweak the dotfiles option or I can look at adding the old functionality back as an option.

Oh, i didn't guess that, because its a dot"folder". Yet, the moment i read the dotfiles entry in the docs i understand that this means files and directories the moment they are on the path.
I don't want to ever serve hidden files, the files i serve are located in a hidden directory.
Anyways i am happy to do dotfiles: 'allow'…
…But I saw you just added »The default value is similar to 'ignore', with the exception that this default will not ignore the files within a directory that begins with a dot, for backward-compatibility.«? – That's my best-case, thanks!
On another note: If send throws errors like this, maybe something like a cause could be a nice DX in the error log?
Like: Error, 404, due to dotfile rule.

But I saw you just added

Yeah, that's an outdated note in the README. The default behavior in 1.0 was changed to ignore, and that's why that file started to 404.

If send throws errors like this, maybe something like a cause could be a nice DX in the error log?

There's no error for this, but it was probably printed if you enable debugging: https://github.com/pillarjs/send/blob/dc6b5d4ec29355ffcf1ab122e52c27a98c392c15/index.js#L460

That's my best-case, thanks!

Does that mean you'd want an option like how it behaved in < 1.0? Something like dotfiles: 'allow file'? I'll think on the behavior more but don't see why it couldn't be supported as an official configuration.

dotfiles: 'allow directories'? I am slightly concerned to allow all dotfiles to be potentially served, now that I am thinking about it, yet i am also sure that's it's not a security issue in my setup at the moment, as i have preliminary checks that the file exists and is in the right folder.
On the other side: Is it possible to set my root as
/Users/Name/Development/Company Name/website_company/.media-cache
and get my file from
/images/w_2000/background_01-8cc5c16c-94b29dfe.jpg
That should work in 1.x, right?
send(event.node.req, '/images/w_2000/background_01-8cc5c16c-94b29dfe.jpg', {
    root: '/Users/Name/Development/Company Name/website_company/.media-cache',
    cacheControl: true,
    maxAge: '1 year',
    immutable: true
})
.pipe(event.node.res)

Maybe that's a good solution because I knowingly opt in to serve from a dotdirectory and my path is controlled by dotfiles


[Bug] Hyperspace.explain does not display "Indexes used" if indexes are incrementally refreshed

Describe the issue
If indexes are incrementally refreshed, Hyperspace.explain will not display Indexes Used even if indexes are applied.
The culprit is https://github.com/microsoft/hyperspace/blob/ad4b3af899f402544f04840ae7f4204e0627018a/src/main/scala/com/microsoft/hyperspace/index/plananalysis/PlanAnalyzer.scala#L216 and https://github.com/microsoft/hyperspace/blob/ad4b3af899f402544f04840ae7f4204e0627018a/src/main/scala/com/microsoft/hyperspace/index/plananalysis/PlanAnalyzer.scala#L149
, since indexLocation doesn't include v__ when multiple versions of index are used (incremental refresh scenario).
To Reproduce

Create a incrementally refreshed index and run Hyperspace.explain.
Expected behavior

It should include the incrementally refreshed index under "Indexes used" section.

Is this issue already fixed in the PR above? I can see the index name (without v__=?) in Indexes Used section.


Update for changes in remark

Hi!
I’ve updated remark / remark-html / vfile: this could produce different output HTML, but apparently the tests don’t catch that

Cheers! :)


Temporarily pin werkzeug in CI to avoid test suite hanging

Towards debugging https://github.com/dask/dask/issues/9323

@martindurant do you have any insight into what might be going on here? Have you seen any related issues over in s3fs?

It seems s3fs is also seeing this https://github.com/fsspec/s3fs/runs/7559234571?check_suite_focus=true , and werkzeug is indeed in the env (not sure via what dep)

Yep, this pin seems to solve it. I can only suppose it's within moto.

@martindurant you are right the dependency is on moto and they are downgrading it too https://github.com/spulec/moto/pull/5330


A bit small

VSCode Version:
OS Version:

Steps to Reproduce:

Reproduces without extensions: Yes/No

Closing this issue as there isn't enough information in the title or description to work on.


[ADAM-595] Delete the 'comparisons' code.

This means deleting the comparisons code here; these files (Comparisons,
AvailableComparisons, and GeneratorFilter) are moving into the qc-metrics
package, along with the ComparisonsTraversalEngine.
Also being deleted are the test-suites for each of these classes.
Finally, the CLI commands which use these classes, CompareADAM and
FindReads, are also being deleted.
Each of these classes will be moved into the qc-metrics repository as
part of a separate pull request.
This resolves #595

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

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

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/619/Build result: FAILUREGitHub pull request #596 of commit 4aa7af12ea60f92e160821365d38d57742173a3f automatically merged.Notifying endpoint 'HTTP:https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'[EnvInject] - Loading node environment variables.Building remotely on amp-jenkins-slave-01 (centos) in workspace /home/jenkins/workspace/ADAM-prb > git rev-parse --is-inside-work-tree # timeout=10Fetching changes from the remote Git repository > git config remote.origin.url https://github.com/bigdatagenomics/adam.git # timeout=10Fetching upstream changes from https://github.com/bigdatagenomics/adam.git > git --version # timeout=10 > git fetch --tags --progress https://github.com/bigdatagenomics/adam.git +refs/pull/:refs/remotes/origin/pr/ > git rev-parse origin/pr/596/merge^{commit} # timeout=10Checking out Revision f260cb8edca472db9423a4964cc659f235caddd3 (detached) > git config core.sparsecheckout # timeout=10 > git checkout -f f260cb8edca472db9423a4964cc659f235caddd3 > git rev-list 329a72a0026fd04ee2340e846a3da461dd6e8c9b # timeout=10Triggering ADAM-prb » 2.2.0,centosTriggering ADAM-prb » 2.3.0,centosTriggering ADAM-prb » 1.0.4,centosADAM-prb » 2.2.0,centos completed with result FAILUREADAM-prb » 2.3.0,centos completed with result FAILUREADAM-prb » 1.0.4,centos completed with result FAILURENotifying endpoint 'HTTP:https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'
Test FAILed.

Jenkins, retest this please.

If this builds, Frank, I think it's ready to go.

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

Let's cut it loose! Thanks @tdanford!

Thanks @fnothaft.


Introducing ErsatzTV Guru on Gurubase.io

Hello ErsatzTV team,
I'm the maintainer of Anteon. We have created Gurubase.io with the mission of building a centralized, open-source tool-focused knowledge base. Essentially, each "guru" is equipped with custom knowledge to answer user questions based on collected data related to that tool.
I wanted to update you that I've manually added the ErsatzTV Guru to Gurubase. ErsatzTV Guru uses the data from this repo and data from the docs to answer questions by leveraging the LLM.
In this PR, I showcased the "ErsatzTV Guru" on the README, which highlights that ErsatzTV now has an AI assistant available to help users with their questions. Please let me know your thoughts on this contribution.
Additionally, if you want me to disable ErsatzTV Guru in Gurubase, just let me know that's totally fine.

spam?


updated .travis.yml to specify go 1.7.5

I edited .travis.yml to specify go 1.7.5 since I think this might be needed to actually use Go 1.7.5 in the user app on the Docker images despite https://github.com/microservices-demo/user/blob/master/Dockerfile being built from golang:1.7-alpine which currently already uses Go 1.7.5.  This is in connection with https://github.com/microservices-demo/user/issues/46.
Thanks.

Well, the test failed, but maybe because I ran against my own fork?
Note that the Travis build log does show go 1.7.5 being used which is good.
Also, I see:
Successfully built 74426bc79434 (line 8328)
The command "make test" exited with 0 (line 8404)
Successfully built 7f4a0f31a5a0 (line 8652)
Successfully built e1feec6148ba (line 8845)
The problem seems to be with the dockertest command:
scripts/testcontainer.sh
Error: bad response code from user service 000
make: *** [dockertest] Error 1
The command "make dockertest" exited with 2.
Done. Your build exited with 1.

I think the testcontainer.sh failed because the sleep was not long enough.  On my laptop, when I ran the test it also failed, but I noticed the containers were still running.  So, I tried running the testcontainer.sh script and it was successful.  So, I increased the sleep in that file from 2 to 10 seconds and will close and re-open this pull request so that the test is run again.

Re-opening to see if sleep of 10 seconds helps.

Coverage remained the same at 23.252% when pulling 2e7573177cbd0372dc6c17aef8d76530b8071fe2 on rberlind:master into 1714eca3ba643b9e2b9cd6c7e5a384bf89a20d8b on microservices-demo:master.

Coverage remained the same at 23.252% when pulling 2e7573177cbd0372dc6c17aef8d76530b8071fe2 on rberlind:master into 1714eca3ba643b9e2b9cd6c7e5a384bf89a20d8b on microservices-demo:master.

The build seem to have stalled out eventually, but note that setting the sleep to 10 seconds in testcontainer.sh DID allow that script to succeed.
After that, I saw:
The command "make dockertest" exited with 0. (line 8910)
and eventually:
No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.
I will re-open and force another build.

Coverage remained the same at 23.252% when pulling 2e7573177cbd0372dc6c17aef8d76530b8071fe2 on rberlind:master into 1714eca3ba643b9e2b9cd6c7e5a384bf89a20d8b on microservices-demo:master.

I think the build failure is probably ok and merely reflects that I don't have the DOCKER_PASS variable set.  I presume if you guys run it, you will have that set and the test will complete successfully.

Why is the sleep moved to 10?

Hi Richard,
See my comments above about the sleep.  With the original value of 2 seconds, I think there was not enough time for the containers to be ready for the testcontainer.sh script to succeed.  At least not on the day I was running the tests.  Changing to 10 allowed the test to succeed.  I don't particularly care about the value since it won't affect the runtime aspects of the docker image or app.
Thanks
Roger

Setting go version in travis file did the trick.
Thanks.


Make sure that callback returns void to match WebIDL

Imported for internal review and testing. Created internal issue b/124052479.

Thanks!


Build(deps-dev): Bump @typescript-eslint/parser from 5.10.1 to 5.10.2 (#2616)

Pull request opened by the merge tool on behalf of #2616

Merged in da477d41833bf9a5e419bb0b886cf2edd24b3800


Implementation

This is my vision of how statsd metrics should be stored in InfluxDB.  I think this is in line with what others would expect.
Pretty thoroughly tested; just do npm test.

bump?


object passed for parameter instead of string

Description

Motivation and Context

https://github.com/apache/jmeter/issues/5922
How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] My code follows the code style of this project.
[ ] I have updated the documentation accordingly.

I guess you'll need to add JMeterProperty handling to org.apache.jorphan.util.Converter#getInt(java.lang.Object), getLong, and similar relevant methods.

Like that?
 public static int getInt(JMeterProperty o) {
        Object objectValue = o.getObjectValue();
        return getInt(objectValue, 0);
    }

not really. Something like
... getInt(...){
if (value instanceof JMeterProperty) {
    return value.getIntegerValue();
}

adding :src:core dependency to :src:jorphan for JMeterProperty cause cyclic dependency error.  Is there way to tackle this?

Oh, I did not think about it.
One of the options could be: make intproperty implement Number.
An alternative option could be adding a custom interface to jorphan that could be implemented in jmeterproperty

Sorry, but I couldn't handle the solution. Do you know any resource for interface solution?

One of the ways would be add a converter in :src:core so it would try convert the properties first, and if the element is not a property, then delegate to the converter in jorphan.
WDYT?


Can not load saves from Title screen.

I can not load saves from the title screen, I can use the continue button but hitting the load game button either results in no menu showing up or a blank uninteractable one.  My saves all still work fine if I try and load one from in game as the menu works there it is just the title screen effectively preventing me from hosting a multiplayer game.  The Logs throw the following error whenever I press the button.
NullReferenceException: Object reference not set to an instance of an object
at SolastaUnfinishedBusiness.Patches.LoadPanelPatcher+OnBeginShow_Patch.Prefix (LoadPanel __instance) [0x0002d] in :0
at (wrapper dynamic-method) LoadPanel.LoadPanel.OnBeginShow_Patch1(LoadPanel,bool)
at (wrapper dynamic-method) GuiPanel.GuiPanel.Show_Patch1(GuiPanel,bool)
at MainMenuScreen.RefreshToggles () [0x00180] in <28022e176eab4ca58d372907a8cf1e38>:0
at MainMenuScreen.OnToggleChangedCb (UnityEngine.GameObject obj) [0x00271] in <28022e176eab4ca58d372907a8cf1e38>:0
at UnityEngine.Events.InvokableCall1[T1].Invoke (T1 args0) [0x00010] in <85d1d3e7744a4a47b5f51883bf40bba2>:0    at UnityEngine.Events.CachedInvokableCall1[T].Invoke (System.Object[] args) [0x00001] in <85d1d3e7744a4a47b5f51883bf40bba2>:0
at UnityEngine.Events.UnityEvent1[T0].Invoke (T0 arg0) [0x00086] in <85d1d3e7744a4a47b5f51883bf40bba2>:0    at UnityEngine.UI.Toggle.Set (System.Boolean value, System.Boolean sendCallback) [0x0007a] in <3ac7dc6602734c42a20ab3905339e904>:0    at UnityEngine.UI.Toggle.set_isOn (System.Boolean value) [0x00000] in <3ac7dc6602734c42a20ab3905339e904>:0    at UnityEngine.UI.Toggle.InternalToggle () [0x00018] in <3ac7dc6602734c42a20ab3905339e904>:0    at UnityEngine.UI.Toggle.OnPointerClick (UnityEngine.EventSystems.PointerEventData eventData) [0x00009] in <3ac7dc6602734c42a20ab3905339e904>:0    at UnityEngine.EventSystems.ExecuteEvents.Execute (UnityEngine.EventSystems.IPointerClickHandler handler, UnityEngine.EventSystems.BaseEventData eventData) [0x00007] in <3ac7dc6602734c42a20ab3905339e904>:0    at UnityEngine.EventSystems.ExecuteEvents.Execute[T] (UnityEngine.GameObject target, UnityEngine.EventSystems.BaseEventData eventData, UnityEngine.EventSystems.ExecuteEvents+EventFunction1[T1] functor) [0x00063] in <3ac7dc6602734c42a20ab3905339e904>:0
UnityEngine.DebugLogHandler:Internal_LogException(Exception, Object)
UnityEngine.DebugLogHandler:LogException(Exception, Object)
UnityEngine.Logger:LogException(Exception, Object)
UnityEngine.Debug:LogException(Exception)
UnityEngine.EventSystems.ExecuteEvents:Execute(GameObject, BaseEventData, EventFunction`1)
UnityEngine.EventSystems.StandaloneInputModule:ReleaseMouse(PointerEventData, GameObject)
UnityEngine.EventSystems.StandaloneInputModule:ProcessMousePress(MouseButtonEventData)
UnityEngine.EventSystems.StandaloneInputModule:ProcessMouseEvent(Int32)
UnityEngine.EventSystems.StandaloneInputModule:ProcessMouseEvent()
UnityEngine.EventSystems.StandaloneInputModule:Process()
InputPointerModule:Process()
UnityEngine.EventSystems.EventSystem:Update()
(Filename:  Line: 0)

I am also getting this problem. i am getting a warning about mod instability when i try to load a multiplayer save.

You have one of more saves that are incompatible. Need to delete them from the save folder.

I have checked every save I have all of them load just fine with no instabilities in play, and even with no saves existing but 1 test save made on this patch with the mod installed the error still occurs with the same error handle, this diagnosis is false.

There are 15k unique mod users and only a few report that. Not sure what else to suggest.

Myself and at at least two of my friends are getting this same error. The
only two who play this game. We have tried starting a new campaign (fresh
save file) and still receive the same message.
On Thu, 8 Jun 2023 at 10:12 pm, Paulo Monteiro @.***>
wrote:

Closed #3439
https://github.com/SolastaMods/SolastaUnfinishedBusiness/issues/3439 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/SolastaMods/SolastaUnfinishedBusiness/issues/3439#event-9471809734,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BANKGDK3VZ575I3LQIIGM3LXKG6SRANCNFSM6AAAAAAY646WCY
.
You are receiving this because you commented.Message ID:
<SolastaMods/SolastaUnfinishedBusiness/issue/3439/issue_event/9471809734@
github.com>

--
Tim Hair
0412695424

I'd done some digging and it's caused by the Override the required min and max levels when starting new adventures option, whenever it is enabled the error is thrown.

So not having this option enabled should solve the problem?
On Fri, 9 Jun 2023 at 1:36 pm, Caulderstolo @.***>
wrote:

I'd done some digging and it's caused by the Override the required min and
max levels when starting new adventures option, whenever it is enabled the
error is thrown.
—
Reply to this email directly, view it on GitHub
https://github.com/SolastaMods/SolastaUnfinishedBusiness/issues/3439#issuecomment-1583917426,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BANKGDJ5IUQ257CZRA2KLHLXKKK5PANCNFSM6AAAAAAY646WCY
.
You are receiving this because you commented.Message ID:
@.***>

--
Tim Hair
0412695424

can confirm this solved my problem
On Fri, Jun 9, 2023 at 2:56 PM Tim Hair @.***>
wrote:

So not having this option enabled should solve the problem?
On Fri, 9 Jun 2023 at 1:36 pm, Caulderstolo @.***>
wrote:

I'd done some digging and it's caused by the Override the required min
and max levels when starting new adventures option, whenever it is enabled
the error is thrown.
—
Reply to this email directly, view it on GitHub
https://github.com/SolastaMods/SolastaUnfinishedBusiness/issues/3439#issuecomment-1583917426,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BANKGDJ5IUQ257CZRA2KLHLXKKK5PANCNFSM6AAAAAAY646WCY
.
You are receiving this because you commented.Message ID:
@.***>

--
Tim Hair
0412695424

--
Tim Hair
0412695424

Please check on <IP_ADDRESS> if it will work with this setting enabled. I tweaked that

Updated to 52.1.
Works with no load instability.
On Sat, 10 Jun 2023 at 12:04 pm, Paulo Monteiro @.***>
wrote:

Please check on <IP_ADDRESS> if it will work with this setting enabled. I
tweaked that
—
Reply to this email directly, view it on GitHub
https://github.com/SolastaMods/SolastaUnfinishedBusiness/issues/3439#issuecomment-1585424848,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BANKGDJZBAILA2E2N362HG3XKPI4RANCNFSM6AAAAAAY646WCY
.
You are receiving this because you commented.Message ID:
@.***>

--
Tim Hair
0412695424


Title: Drop shadows in navbar in bright mode and responsive Get in touch section

Is your feature request related to a problem? Please describe.
The Get in touch is not working on clicking on any of the forms.
The navbar hover in bright mode looks a bit dull
the quotes in bright mode is almost not visible

Describe the solution you'd like.
I want to allow users to enter their data, without storing it
and add shadows to the navbar links in bright mode
Add little darker color to the quotes in the testimonial
Describe alternatives you've considered.
Alternatives not as such, but doing these will definitely make the project more appealing
Add any other context or screenshots about the feature request here.
No response

Sure @PoulavBhowmick03


[BSP] Send diagnostics and meaningful error message when reloading fails

Fix #6010 and address https://github.com/scalameta/metals-feature-requests/issues/143#issuecomment-866642593 (@ckipp01)
It should be probably merged after #6553 to avoid conflicts.
Just as #6553, merging this PR depends on whether we decide to break the binary compatibility of PluginData or not. In which case it would probably goes to sbt 1.6.
In order of commits:

Simplify the bspReload command to catch the exceptions and fill the error response with the correct message
Add a BuildServerEvalReporter to send diagnostics when evaluating .sbt files
Add a test on a failing workspace/reload request

In action within Metals, it looks like this:

@adpi2 I feel like locally I'm doing something wrong when testing this out with Metals.

I checkout out this pr
Did a sbt publishLocal
Switched my build to use the snapshot I just made
Generated by bspConfig
Then attached via bsp-switch

I can't seem to get the diagnostics to publish. I do see that it attaches to the new snapshot (I even bumped the version up in sbt to ensure I was really getting a fresh version).

BSP logs

[Trace - 09:51:42 AM] Sending request 'workspace/reload - (14)'
Params: null

[Trace - 09:51:42 AM] Received notification 'build/logMessage'
Params: {
  "type": 4,
  "message": "Processing workspace/reload"
}

[Trace - 09:51:42 AM] Received notification 'build/taskStart'
Params: {
  "taskId": {
    "id": "50",
    "parents": []
  },
  "eventTime":<PHONE_NUMBER>599,
  "message": "Compiling global-plugins",
  "dataKind": "compile-task",
  "data": {
    "target": {
      "uri": "file:/Users/ckipp/.sbt/1.0/plugins/#global-plugins/Compile"
    }
  }
}

[Trace - 09:51:42 AM] Received notification 'build/taskFinish'
Params: {
  "taskId": {
    "id": "50",
    "parents": []
  },
  "eventTime":<PHONE_NUMBER>605,
  "message": "Compiled global-plugins",
  "status": 1,
  "dataKind": "compile-report",
  "data": {
    "target": {
      "uri": "file:/Users/ckipp/.sbt/1.0/plugins/#global-plugins/Compile"
    },
    "errors": 0,
    "warnings": 0,
    "time": 6
  }
}

[Trace - 09:51:42 AM] Received notification 'build/taskStart'
Params: {
  "taskId": {
    "id": "51",
    "parents": []
  },
  "eventTime":<PHONE_NUMBER>967,
  "message": "Compiling sanity-build-build",
  "dataKind": "compile-task",
  "data": {
    "target": {
      "uri": "file:/Users/ckipp/Documents/scala-workspace/sanity/project/project/#sanity-build-build/Compile"
    }
  }
}

[Trace - 09:51:42 AM] Received notification 'build/taskFinish'
Params: {
  "taskId": {
    "id": "51",
    "parents": []
  },
  "eventTime":<PHONE_NUMBER>975,
  "message": "Compiled sanity-build-build",
  "status": 1,
  "dataKind": "compile-report",
  "data": {
    "target": {
      "uri": "file:/Users/ckipp/Documents/scala-workspace/sanity/project/project/#sanity-build-build/Compile"
    },
    "errors": 0,
    "warnings": 0,
    "time": 8
  }
}

[Trace - 09:51:43 AM] Received notification 'build/taskStart'
Params: {
  "taskId": {
    "id": "52",
    "parents": []
  },
  "eventTime":<PHONE_NUMBER>650,
  "message": "Compiling sanity-build",
  "dataKind": "compile-task",
  "data": {
    "target": {
      "uri": "file:/Users/ckipp/Documents/scala-workspace/sanity/project/#sanity-build/Compile"
    }
  }
}

[Trace - 09:51:43 AM] Received notification 'build/taskFinish'
Params: {
  "taskId": {
    "id": "52",
    "parents": []
  },
  "eventTime":<PHONE_NUMBER>659,
  "message": "Compiled sanity-build",
  "status": 1,
  "dataKind": "compile-report",
  "data": {
    "target": {
      "uri": "file:/Users/ckipp/Documents/scala-workspace/sanity/project/#sanity-build/Compile"
    },
    "errors": 0,
    "warnings": 0,
    "time": 9
  }
}

[Trace - 09:51:44 AM] Received response 'workspace/reload - (14)' in 2143ms
Result: null
Error: {
  "code": -32603,
  "message": "reload failed"
}

I do get a whole bunch of
2021.07.01 09:51:41 ERROR java.lang.UnsupportedOperationException: empty.max
        at scala.collection.TraversableOnce.max(TraversableOnce.scala:275)
        at scala.collection.TraversableOnce.max$(TraversableOnce.scala:273)
        at scala.collection.AbstractTraversable.max(Traversable.scala:108)
        at sbt.internal.server.BuildServerProtocol$.$anonfun$globalSettings$40(BuildServerProtocol.scala:164)
        at sbt.internal.server.BuildServerProtocol$.$anonfun$globalSettings$40$adapted(BuildServerProtocol.scala:163)
        at scala.Function1.$anonfun$compose$1(Function1.scala:49)
        at sbt.internal.util.$tilde$greater.$anonfun$$u2219$1(TypeFunctions.scala:62)
        at sbt.std.Transform$$anon$4.work(Transform.scala:68)
        at sbt.Execute.$anonfun$submit$2(Execute.scala:282)
        at sbt.internal.util.ErrorHandling$.wideConvert(ErrorHandling.scala:23)
        at sbt.Execute.work(Execute.scala:291)
        at sbt.Execute.$anonfun$submit$1(Execute.scala:282)
        at sbt.ConcurrentRestrictions$$anon$4.$anonfun$submitValid$1(ConcurrentRestrictions.scala:265)
        at sbt.CompletionService$$anon$2.call(CompletionService.scala:64)
        at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
        at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
        at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
        at java.base/java.lang.Thread.run(Thread.java:829)

So I thought maybe that was causing some issues, but even removing the max call in bspBuildTargetCompile still was giving me this, which makes me feel like I'm not correctly using the snapshot I just published. Am I doing something wrong or missing a step to use this snapshot?

From the logs it seems you don't have the snapshot running. Because the following message does not exist anymore (in this PR):
Error: {
  "code": -32603,
  "message": "reload failed"
}

When you change the sbt version in the build.properties file, you can try this:

rm -Rf ~/.sbt/boot/scala-2.12.14/org.scala-sbt/sbt/1.5.5-SNAPSHOT to remove the cached jars
run the sbt reboot command (reload is not enough) or kill the sbt process and start it again

If it is still not working you can try to clean the project/target directory. But there may be something wrong in this PR as well...

rm -Rf ~/.sbt/boot/scala-2.12.14/org.scala-sbt/sbt/1.5.5-SNAPSHOT to remove the cached jars

Ahh bingo, that's what I was missing. Thanks! Back in business. This is fantastic that this is working.

Could you run scalafmt please?


"fix" excludes with the --redirect flag.

HTML imports are still broken, and will require a more nuanced approach to the
NoopResolver. Since the NoopResolver has no visilbity into redirected URLs,
it will have a hard time matching the custom://protocol/url type links.

Lgtm


Continues software development with HppoRAG

If anyone has achieved or can suggest for this . In this we want to continuously index the updated code base in the HippoRAG index and query on the updated index and then make code changes and continuously do so.  Here consider that we want to do this offline with ollama type models only and dont want to use OpenAI or Claude .
If anyone can suggest how can i do this ?

Hi, thanks for your interest. Continuously building indexes here is an inspiring idea.
HippoRAG is not developed for codes currently, but we are open to any suggestions or discussions.
Combined with frameworks such as Langchain, you should be able to replace model calls with open-source models.

if possible please try to integrate it with ollama's open-webui https://github.com/open-webui/open-webui

您好，感谢您的关注。在这里持续构建索引是一个鼓舞人心的想法。 HippoRAG 目前不是为代码开发的，但我们对任何建议或讨论持开放态度。结合 Langchain 等框架，您应该能够将模型调用替换为开源模型。

可是我尝试将其替换为Ollama的模型，但是出现了报错。https://github.com/OSU-NLP-Group/HippoRAG/issues/53


STO Jira integration DOC-2491

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

@douglas-j-bothwell I'm not sure how to best resolve the naming of Jira Issue (which is what a ticket is called in Jira) vs. the abstract concept of an External Ticket on the Harness platform (which will eventually include additional ticketing system providers). We also need to avoid the confusion of Security Issue with Jira Issue

@chris-harness WRT the use of "Jira issue" vs. "Jira ticket:"  The current UI uses "Ticket" so I'm going use this term in the docs to keep things consistent. If we want to standardize on the term "Jira issue," we should update the UI as well as the doc.

I've implemented @chris-harness edits in a separate pull request. Doc is live: https://developer.harness.io/docs/security-testing-orchestration/use-sto/jira-integrations