Issues related to keypad in Vachan Go1.3.8-alpha.1

Not able to type in SignUp & sign in
Not able to type in Notes
Not able to write to search
Not able to type in feedback form.

@JesniJ @JesniJ please change the version to 1.3.8_alpha dot 1 as this is the format we are following

Version number is corrected.

Testing completed and verified that issues with the keypad has fixed. Thus moving this issue to done.


Puffin_egui v0.27 does not display the flamegraph correctly

Describe the bug
A clear and concise description of what the bug is.
The puffin_egui flamegraph doesn't display correctly.
To Reproduce
Steps to reproduce the behavior:

Make an egui native app, that has puffin scopes and allows you to interact with the puffin_egui ui.
Run the app and open the debugger.

Expected behavior
A clear and concise description of what you expected to happen.
I expected the flamegraph to show, like in previous versions.
Screenshots
If applicable, add screenshots to help explain your problem.
Device:

OS: [e.g. iOS] NixOs
Browser [e.g. chrome, safari] Using Native App, not browser.
Version [e.g. 22]

Additional context
Add any other context about the problem here.

app code here: https://github.com/C0D3-M4513R/unnamed-artnet-orion-vrsl-app/commit/41bb2cf38b82a001a9e549b6ee3841a8695c85cc (needs feature puffin, to display debugging menu)
(call to draw the window here: https://github.com/C0D3-M4513R/unnamed-artnet-orion-vrsl-app/blob/41bb2cf38b82a001a9e549b6ee3841a8695c85cc/app/src/app.rs#L405
)

I narrowed it down to puffin not creating any scopes in non-release mode.

Please check out the repo and run cargo r --example eframe. It should show you some profile scopes even without --release.


feat: implement cargo-toml impure translator for rust subsystem

This implements a new impure cargo-toml translator that can translate Cargo projects that don't have a Cargo.lock file. In addition, it makes the cargo-lock translator not put absolute paths for fixing up path dependency paths in a Cargo.toml, instead relative paths are used now, because absolute paths break the generated dream-lock. Rust discoverer is also updated to support the new translator.
This is a draft because it doesn't seem to be able to build stuff properly (ripgrep doesn't build). ripgrep builds fine now.

I'm marking as draft again because this branch breaks nix-cargo-integration's tests. Will see to that soon.

Turns out the issue was related to path sources with nix store paths being removed from the dream-lock which makes sense, but I'm not sure why it hasn't popped up before. Anyhow to (properly) fix this #143 is needed, because otherwise path source specs with nix store paths will be removed, which breaks Rust vendoring. It will also break for crates outside of project source, #143 allows fixing that too.

Could you rebase this on main to see if all tests succeed?
Also please add an example under ./examples which uses this translator.

Could you rebase this on main to see if all tests succeed?
Also please add an example under ./examples which uses this translator.


Admin API Security: Handle errors gracefully in Admin UI when making unauthorized GraphQL requests

Sub-Issue of #2218

We are not using hydra anymore


[Integration][Snyk] - Migrate API from v1 to REST

Description
What - Snyk has announced that they will burn down their v1 API by end of Q3, and all subsequent calls to their v1 APIs will return 410 (gone). This PR migrates all instances of our Snyk exporter that relies on the v1 API, and moves them to the REST API. The PR contains the following:

Migration of v1 API to REST API
Added additional kind called vulnerabilities. and this will be used to ingest Snyk vulnerabilities for new customers. To support backwards compatibility, the existing issue kind is maintained, and will be switched for the vulnerabilities after Q3 when we have communicated the deprecation notice to client
Moved calculation of low|medium|high|critical open vulnerabilities on the Target blueprint to Aggregation property to reduce making calls to Snyk.
Other blueprint and mapping changes

Why -
How - Get issues, get organizations, get users etc are now using REST API
Type of change
Please leave one option from the following and delete the rest:

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] New Integration (non-breaking change which adds a new integration)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Non-breaking change (fix of existing functionality that will not change current behavior)
[ ] Documentation (added/updated documentation)

Screenshots
Include screenshots from your environment showing how the resources of the integration will look.
API Documentation
Issues API
Users API
API SAMPLE RESPONSE
Issues from REST API
{
  "id": "0ecc9091-c1cc-4107-a6cb-d66bb7935ede",
  "type": "issue",
  "attributes": {
    "classes": [
      {
        "id": "CWE-94",
        "source": "CWE",
        "type": "weakness"
      }
    ],
    "coordinates": [
      {
        "representations": [
          {
            "dependency": {
              "package_name": "vm2",
              "package_version": "3.9.19"
            }
          }
        ]
      }
    ],
    "created_at": "2023-09-03T15:02:41.211Z",
    "effective_severity_level": "critical",
    "ignored": false,
    "key": "SNYK-JS-VM2-5772825",
    "problems": [
      {
        "id": "CVE-2023-37466",
        "source": "SNYK",
        "type": "vulnerability",
        "updated_at": "2024-01-12T07:35:26.623212Z"
      },
      {
        "id": "SNYK-JS-VM2-5772825",
        "source": "SNYK",
        "type": "vulnerability",
        "updated_at": "2024-01-12T07:35:26.623212Z"
      }
    ],
    "risk": {
      "factors": [],
      "score": {
        "model": "v1",
        "value": 597
      }
    },
    "status": "open",
    "title": "Remote Code Execution (RCE)",
    "type": "package_vulnerability",
    "updated_at": "2023-09-03T15:02:41.211Z"
  },
  "relationships": {
    "organization": {
      "data": {
        "id": "7c5908e4d4d2",
        "type": "organization"
      },
      "links": {
        "related": "/orgs/7c5908e4d4d2"
      }
    },
    "scan_item": {
      "data": {
        "id": "36f9fbb0-caf2-48a7-bbac-e7c1dc093f37",
        "type": "project"
      },
      "links": {
        "related": "/orgs/7c5908e4d4d2/projects/36f9fbb0-caf2-48a7-bbac-e7c1dc093f37"
      }
    }
  }
}

Organizations from REST API
    {
      "id": "7c5908e4d4d2",
      "type": "org",
      "attributes": {
        "group_id": "49270cfc54b5",
        "is_personal": false,
        "name": "Port NFR - Shared",
        "slug": "port-nfr-shared"
      },
      "relationships": {
        "member_role": {
          "data": {
            "id": "9409d7b4f883",
            "type": "org_role"
          }
        }
      }
    }

Looks great, 1 code NIT and 1 small Q: Existing integrations with the old blue print and mapping, they'd work fine?

Yes, existing integrations with old config should work fine with this new data


Working with h5 file

Hi,
I'm saving a model to disk:
from keras.models import Sequential
from keras.layers import Dense
model = Sequential()
model.add(Dense(12, input_dim=8, activation='relu'))
model.add(Dense(8, activation='relu'))
model.add(Dense(1, activation='sigmoid'))
model.compile(loss='binary_crossentropy', optimizer='adam', metrics=['accuracy'])
model.save("model.h5")
print("Saved model to disk")

Then, removing one layer
import h5py
h = h5py.File("model.h5")
del h["model_weights/dense_2"]
h.close()

and re-importing by name:
from keras.models import Sequential
from keras.layers import Dense
model = Sequential()
model.add(Dense(12, input_dim=8, activation='relu'))
model.add(Dense(8, activation='relu'))
model.add(Dense(1, activation='sigmoid'))
model.compile(loss='binary_crossentropy', optimizer='adam', metrics=['accuracy'])
model.load_weights("model.h5", by_name=True)

gives me:

KeyError: "Unable to open object (object 'dense_2' doesn't exist)"

What is the reason for this ? The option by_name=True should accept that some layers are absent ?

The answer is:
h["model_weights"].attrs["layer_names"]=[b'dense_1', b'dense_3']


⚠ Punto de Fuga has degraded performance

In e5f09eb, Punto de Fuga (http://puntodefugapodcast.com) experienced degraded performance:

HTTP code: 200
Response time: 10519 ms

Resolved: Punto de Fuga performance has improved in cd6ccf9.


chore: synchronised configuration with TypeScript-Demo-Lib

Description
This synchronises with the changes in CI/CD:

test load balancing
native build fixes coming from js-db
npm configuration
ts-node now using swc
linting for javascript, c++ and shell scripts

Note that I've found that parent pipelines cannot depend on child pipeline artifacts due to: https://gitlab.com/gitlab-org/gitlab/-/issues/285100
So for now all prebuild tasks are occurring before the tests and platform-specific tests in the parent pipeline.
Tasks

[x] 1. Updated to 22.05 revision
[x] 2. Introduced sharded load balancing
[x] 3. Removed prebuildify in favour of ./prebuild.js
[x] 4. Added lint-native and lint-shell
[x] 5. CI/CD runs prebuilds in parent pipeline first, and child pipeline can dependt on the parent pipeline artifacts
[x] 6. Container image deployment to Gitlab container registry
[x] 7. Chocolatey caching and Homebrew caching
[x] 8. Removed typescript-cached-transpile in favour of swc
[x] 9. Container images have timestamps

Final checklist

[ ] Domain specific tests
[ ] Full tests
[ ] Updated inline-comment documentation
[ ] Lint fixed
[ ] Squash and rebased
[ ] Sanity check the final build

Some changes here can be propagated to js-db. There's a bunch of repos that require updating.
I imagine moving all repos towards a staging process might be complex.


Update Changelog to 2.6.0 version

Describe the bug
The Changelog is not aligned with the 2.6.0 release
Expected behavior
I expect to see latest changes in the file Changelog
Actual behavior
The latest changes in the Changelog are stuck to version 2.5.1
Steps to reproduce
Open the file a read
https://github.com/locustio/locust/blob/master/CHANGELOG.md

github_changelog_generator is broken, hence this comment:

Github changelog generator is broken, so the PR:s will only become visible on the next release

But I did it now...

Thanks!


node production mode

<Have you checked issues section which may have your similar issue?>

Issue Overview
If You are running npm install --production
You don`t have all needed node module to make production build on linux agent.
If You make only npm install. You are installing all packages not needed for production. Like:
browser-sync
connect-history-api-fallback
Issue Description
gulp.config strongly banded to  gulp.config.js

var historyApiFallback = require('connect-history-api-fallback');

And no production logic is separated. If You are following best practice installing only node mode that are required for each environment.
Reproducables
npm install --production
gulp build --exclude-source
Information

Operating System
Debian

Node version
4.x/5.x/orYourNodeVersionHere

Environment
Browser/Mobile/WebWorker

Thanks, I'd love to accept a PR for this.

I did node production mode. For angular 1.X examples can be taken from https://github.com/darxx/AngularWorkshop/tree/step5

@darxx , can you send a PR? I'll help where I can

Yes. After work I will try to figure it out. Now I`m thinking to use global.env and separate gulp.config.js part that is using
if (global.env === 'dev') {
var historyApiFallback = require('connect-history-api-fallback');
}
module.exports = function () {
And
if (global.env === 'dev')
{
var browserSync = {
dev: {
port: 3000,
server: {
baseDir: './src/',
middleware: [historyApiFallback()],
routes: {
"/node_modules": "node_modules",
"/src": "src"
}
},
files: [
src + "index.html",
src + "systemjs.conf.js",
src + "assets/styles/main.css",
tmpApp + "/*.js",
app + "/.css",
app + "**/.html"
]
},
prod: {
port: 3001,
server: {
baseDir: './' + build.path,
middleware: [historyApiFallback()]
}
}
};
}
and
var config = {
root: root,
translations: translations,
src: src,
app: app,
test: test,
tmp: tmp,
tmpApp: tmpApp,
tmpTest: tmpTest,
tmpE2E: tmpE2E,
testHelper: testHelper,
e2e: e2e,
e2eConfig: e2eConfig,
assets: assets,
index: index,
build: build,
report: report,
assetsPath: assetsPath,
tsFiles: tsFiles,
tsTestFiles: tsTestFiles,
systemJs: systemJs
};
if (global.env === 'dev')
{
    config.browserSync = browserSync;

}

And now I`m getting
07 06 2016 15:34:03.353:INFO [karma]: Karma v0.13.22 server started at http://localhost:9876/
07 06 2016 15:34:03.363:INFO [launcher]: Starting browser PhantomJS
07 06 2016 15:34:04.918:INFO [PhantomJS 2.1.1 (Windows 8 0.0.0)]: Connected on socket /#2elfkEsL-DS8sZ26AAAA with id 62765646
07 06 2016 15:34:07.171:WARN [web-server]: 404: /base/node_modules/karma-ie-launcher/test/launcher.spec
PhantomJS 2.1.1 (Windows 8 0.0.0) ERROR
Error: XHR error (404 Not Found) loading http://localhost:9876/node_modules/karma-ie-launcher/test/launcher.spec
Error loading http://localhost:9876/node_modules/karma-ie-launcher/test/launcher.spec
Remapping coverage to TypeScript format...
Test Done with exit code: 1
[15:34:07] 'unit-test' errored after 26 s
[15:34:07] Error: 1
at formatError (C:\Users\darlau\AppData\Roaming\npm\node_modules\gulp\bin\gulp.js:169:10)
at Gulp. (C:\Users\darlau\AppData\Roaming\npm\node_modules\gulp\bin\gulp.js:195:15)
at emitOne (events.js:95:20)
at Gulp.emit (events.js:182:7)
at Gulp.Orchestrator._emitTaskDone (C:\dev\coreUI\node_modules\orchestrator\index.js:264:8)
at C:\dev\coreUI\node_modules\orchestrator\index.js:275:23
at finish (C:\dev\coreUI\node_modules\orchestrator\lib\runTask.js:21:8)
at cb (C:\dev\coreUI\node_modules\orchestrator\lib\runTask.js:29:3)
at DestroyableTransform. (C:\dev\coreUI\tasks\test.js:59:13)
at emitNone (events.js:85:20)
at DestroyableTransform.emit (events.js:179:7)
at finishMaybe (C:\dev\coreUI\node_modules\through2\node_modules\readable-stream\lib_stream_writable.js:475:14)
at endWritable (C:\dev\coreUI\node_modules\through2\node_modules\readable-stream\lib_stream_writable.js:485:3)
at DestroyableTransform.Writable.end (C:\dev\coreUI\node_modules\through2\node_modules\readable-stream\lib_stream_writable.js:455:41)
at DestroyableTransform.onend (C:\dev\coreUI\node_modules\vinyl-fs\node_modules\readable-stream\lib_stream_readable.js:523:10)
at DestroyableTransform.g (events.js:273:16)
at emitNone (events.js:85:20)
at DestroyableTransform.emit (events.js:179:7)
at C:\dev\coreUI\node_modules\vinyl-fs\node_modules\readable-stream\lib_stream_readable.js:965:16
at _combinedTickCallback (node.js:376:9)
at process._tickCallback (node.js:407:11)
[15:34:07] 'build' errored after 31 s
[15:34:07] Error in plugin 'run-sequence(unit-test)'
Message:
unit-test callback
Remapping done! View the result in report/remap/html-report
At home I will have more time to figure it out.

gulp build --exclude-source --prod


Query: Live Channels App

Hi,
There is an app called "Live Channels" on Android TV. But how can I integrate JioTV channels on that app??? Is there any way to extract m3u8 file from your addon to work properly on "Live Channels" or "PVR Live" app?

Tl;dr  NO
Kodi doesn't support this kind of integration. Regarding the m3u playlist which is required some tweaks for headers to work, and this is not possible directly via m3u.


Unable to access props inside component

This is my component
import React, { Component } from 'react';
import PropTypes from 'prop-types';
import { bindActionCreators } from 'redux';
import { connect } from 'react-redux';
import * as actions from './redux/actions';

export class InstrumentsSearch extends Component {
  static propTypes = {
    home: PropTypes.object.isRequired,
    actions: PropTypes.object.isRequired,
  };

  render() {
    console.log(this.props) // --> it is undefined
    return (
      <div className="home-instruments-search" style={{ width: 'auto' }}>
          <AutoComplete
            className="global-search"
            size="large"
            style={{ width: '100%' }}
            placeholder="Search here"
            optionLabelProp="text"
          >
          <Input suffix={<Icon type="search" className="certain-category-icon" />} />
          </AutoComplete>
      </div>
    );
  }
}

/* istanbul ignore next */
function mapStateToProps(state) {
  return {
    home: state.home,
  };
}

/* istanbul ignore next */
function mapDispatchToProps(dispatch) {
  return {
    actions: bindActionCreators({ ...actions }, dispatch)
  };
}

export default connect(
  mapStateToProps,
  mapDispatchToProps
)(InstrumentsSearch);

My initial state
const initialState = {
  getInstrumentsPending: false,
  getInstrumentsError: null,
  instruments: [],
  fetchedInstruments: []
};

export default initialState;

basically i had just modified your example redditList component with my actions and reducers.

Hi, as a note i am not able to dispatch actions nor able to access props from components which don't have a route.
The components with route works fine.
Is this by design or i am missing something.

How do you import the component in your route.js? It looks like redux connect didn't work. Maybe you used named import like:
import { InstrumentsSearch } from './home/InstrumentsSearch';

It should be:
import { InstrumentsSearch } from './home';

Hey thanks.

I'll try this tomorrow and let you know.
I think this were i am wrong..


Introduced ZSETs for testcases and testcases with component

Comparison between XX and YY for metric: OverallQueryRates.Total. Time Period from 1 moth ago to now

Test Case
Baseline (median obs. +- std.dev)
Comparison (median obs. +- std.dev)
% change (higher-better)
Note

ENTITY_COUNT
33329 +- 0.0%  (17 datapoints)
33329 +- 0.0%  (6 datapoints)
-0.0%
-- no change --

EXPLICIT-EDGE-DELETION
100 +- 0.0%  (17 datapoints)
100 +- 0.0%  (6 datapoints)
0.0%
-- no change --

GRAPH500-SCALE_18-EF_16-1_HOP
19999 +- 0.0%  (17 datapoints)
19999 +- 0.0%  (6 datapoints)
0.0%
-- no change --

GRAPH500-SCALE_18-EF_16-1_HOP_MIXED_READ_65perc_WRITE_25perc_DEL_10perc
48 +- 2.0%  (17 datapoints)
49 +- 2.7%  (6 datapoints)
2.5%
-- no change --

GRAPH500-SCALE_18-EF_16-1_HOP_MIXED_READ_75perc_WRITE_25perc
56 +- 1.4%  (17 datapoints)
56 +- 0.0%  (6 datapoints)
-0.0%
-- no change --

GRAPH500-SCALE_18-EF_16-2_HOP
12499 +- 8.1%  (17 datapoints)
16665 +- 0.0%  (6 datapoints)
33.3%
waterline=8.1%.


Image could not be loaded in SA1641 documentation page

I've noticed that in https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1641.md the image /documentation/Images/CompanyInformationSettings.JPG is missing.

This page was copied over from StyleCop "classic". The new instructions for configuring this information are available in Configuration.md, but we should update this page to be accurate too.


Add a test case that exibits over-capacity out-of-memory error

This is exactly the same class of error that has hit bincode recently.
https://github.com/TyOverby/bincode/issues/41
I have an issue with a solution open on rustc-serialize.

Nice find!


Add copyrights to Cholesky examples

@hcq9102 can you add copyrights as well? Thanks.

Sure, will do.


[Suggestion] Support library

Hi. I have a suggestion for you. Could you please use
implementation 'com.android.support:exifinterface:<your version>
or
compile ('com.android.support:exifinterface:<your version>) {
    transitive = false
}```
to avoid support library version conflicts in following versions?  Thank you.

Can you link some documentation explaining what it does?

I suspect this is related:

Error:Execution failed for task ':app:processDebugManifest'.
Manifest merger failed : Attribute meta-data#android.support.VERSION@value value=(26.1.0) from [com.android.support:design:26.1.0] AndroidManifest.xml:28:13-35
is also present at [com.android.support:exifinterface:26.0.2] AndroidManifest.xml:25:13-35 value=(26.0.2).
Suggestion: add 'tools:replace="android:value"' to  element at AndroidManifest.xml:26:9-28:38 to override.

To fix it, I have to put in my own build.gradle:
implementation 'com.theartofdev.edmodo:android-image-cropper:2.5.0', {
    exclude group: 'com.android.support', module: 'exifinterface'
}

I'm no gradle expert but I hope that helps understand what's going on!

@lisawray exactly
@ArthurHub transitive libraries will be available in modules, which depend on your library. This can cause issues when people use other library versions than you. @lisawray gave you a good example just above. If you set transitive to false, such conflicts will no longer occur even without that workaround.
More info on that here: official gradle docs

added in 2.5.1, hope it solves the merge issue, thx @j-garin


API Documentation Error

The call api.getThreadList(start, end, type, callback) claims to return an array with objects which contain field 'name,' which should be: "Name of the thread. Usually the name of the user. In group chats, this will be empty if the name of the group chat is unset." However, unless the thread is a group chat with a user set name, the field 'name' is an empty string, even if the thread is with a single individual.

I think you're confusing this with facebook-chat-api's getThreadList. fbchat doesn't have anything called getThreadList, and it definitely doesn't have callbacks

Also, see facebook-chat-api#476


Alert shows after game Over

After Game is over an alert still shows up after and indicates whether you gained or lost a point want to remove this

Resolved after modifying code order and returning earlier

Resolved after modifying code order and returning earlier


Missing WiFi101.h error

Following the starter kit instructions for remote monitoring with the Adafruit Huzzah ESP8266 works until I compile the sketch at step 1.8 and the following error results:
C:\Users<name>\Documents\Arduino\libraries\AzureIoT\src\util/HTTPSClient.h:4:21: fatal error: WiFi101.h: No such file or directory
#include <WiFi101.h>
Is there something missing from the instructions i.e. something else that needs to be installed as the file is not present on my PC. Adding the WiFi101 library to Arduino IDE (which is not mentioned in the instructions) causes more compilation errors.

Hey Richard, are you using the m0 sample (this repo) with the Huzzah, or the Huzzah sample/instructions?

I'm using the https://github.com/Azure-Samples/iot-hub-c-huzzah-getstartedkit sample and instructions.

Great.  Also verify you have the Huzzah selected from the board selection UI in your Arduino IDE - that is the most likely reason it would try to use the incorrect headers as they are selected in httpsclient.h based on the type of board you are compiling for.

Oh - Sorry Richard, I suspect these instructions were incorrect and led you to install the Azure IoT Library from the Arduino UI - this version looks to be not refreshed with the latest for the Huzzah yet.  You should remove the existing AzureIoT library with the library manager, and download the library from https://github.com/stefangordon/AzureIoT by cloning the repo, or just downloading it as zip from https://github.com/stefangordon/AzureIoT/archive/master.zip and placing the AzureIot folder in C:\Program Files (x86)\Arduino\libraries and restarting the IDE.
I'm working on getting this documentation updated right now.

The library manager doesn't appear to allow the uninstall of the AzureIoT library that's already there so I have copied the new AzureIoT folder into Program Files (x86)\Arduino\libraries and there are a load of compile errors starting with;
C:\Program Files (x86)\Arduino\libraries\AzureIoT\src\esp8266\libc_esp.c: In function 'fprintf':
C:\Program Files (x86)\Arduino\libraries\AzureIoT\src\esp8266\libc_esp.c:19:5: error: incompatible type for argument 2 of 'ets_vprintf'
ret = ets_vprintf(format, arglist);
It ends by saying;
Multiple libraries were found for "AzureIoT.h"
Used: C:\Program Files (x86)\Arduino\libraries\AzureIoT
Not used: C:\Users<name>\Documents\Arduino\libraries\AzureIoT
Unless you have any other suggestions I'll remove Arduino IDE and start again when I have time.

Next issue you might run into is the DHT22 remote monitoring sample in those instructions being outdated - I'm verifying that now.

Instructions and remote monitoring sample are updated in the quick start at https://github.com/Azure-Samples/iot-hub-c-huzzah-getstartedkit to reflect using the latest library and some tweaks to the .ino to be compatible.

I've updated the files and its all working fine now and I can see data from the sensors being displayed in the Azure IoT Suite dashboard! Huzzah! Thank you Stefan.

Perfect, thanks Richard.


load generator not given enough time to finish

Running a big load on a big server, the output of the tail end of the load generator/agent looks like the following; note the ERROR at 05.324  It seems like the coordinator isn't  being patient enough shutting things down.
[07:48:57.190] Processing job 'load' (3) in state Running
[07:48:58.189] Processing job 'load' (3) in state Running
[07:48:59.189] Processing job 'load' (3) in state Running
[07:48:59.804] Driver stopping job '3'
[07:49:00.190] Processing job 'load' (3) in state Stopping
[07:49:00.190] Stopping job 'load' (3)
[07:49:00.190] Stopping heartbeat (load)... Success!
[07:49:00.190] Stopping counters event pipes for job 'load' (3)
[07:49:00.190] Stopping event pipe session (load)...
[07:49:00.323] Event pipe session stopped (load)
[07:49:05.324] [ERROR] Counters could not be stopped in time for job 'load' (3)
[07:49:05.324] Counters stopped for job 'load' (3)
[07:49:05.324] Invoking SIGINT ...
[07:49:05.524] Process has stopped
[07:49:05.524] Process stopped (Stopped)
[07:49:05.860] Driver stopping job '3'
[07:49:06.524] Processing job 'load' (3) in state Stopped
[07:49:06.525] Job 'load' (3) has stopped, waiting for the driver to delete it
[07:49:06.886] Driver deleting job '3'
[07:49:07.525] Processing job 'load' (3) in state Deleting
[07:49:07.525] Deleting job 'load' (3)
[07:49:07.525] Deleting -> Deleted
[07:49:08.525] All jobs in group are finished
And the stdout from the coordinator job for the load looks like this, with the useful "requests/sec" missing.

load

CPU Usage (%)
10

Cores usage (%)
1,537

Working Set (MB)
47

Start Time (ms)
106

First Request (ms)
197

Can you check/paste the output log of the load job?

Looks like bombardier doesn't exit. Can you try to run it as is, outside of the agent.
bombardier  -d 30s -c 512 --insecure -l --fasthttp --header "Accept: application/json,text/html;q=0.9,application/xhtml+xml;q=0.9,application/xml;q=0.8,*/*;q=0.7" --header "Connection: keep-alive" http://<IP_ADDRESS>:5000/db --print r --format json

The bombardier process appears to be hung, and doesn't respond to kill -9.
It turns into a zombie process, leaving ~200  kworker (kernel worker)
processes running and stuck. Time for another reboot.
I did get one complete clean crank run in today (even with the bombardier
load tool writing json with requests/second) in it.  But that's only 10% of
the total number of runs.
On Tue, Feb 23, 2021 at 9:07 AM Sébastien Ros<EMAIL_ADDRESS>wrote:

Looks like bombardier doesn't exit. Can you try to run it as is, outside
of the agent.
bombardier  -d 30s -c 512 --insecure -l --fasthttp --header "Accept: application/json,text/html;q=0.9,application/xhtml+xml;q=0.9,application/xml;q=0.8,/;q=0.7" --header "Connection: keep-alive" http://<IP_ADDRESS>:5000/db --print r --format json
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dotnet/crank/issues/241#issuecomment-784355677, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBLLY44OQWAXJT42RUBM2DTAPODPANCNFSM4YBEWIXQ
.

Fixed by removing the timeout calculation from bombardier and wrk


New project won't run

After running:
mix scenic.new example
cd example
mix deps.get
iex -S mix

After some normal compilation errors I get this output:
Generated scenic_driver_glfw app
==> example
Compiling 2 files (.ex)
Generated example app
Interactive Elixir (1.8.1) - press Ctrl+C to exit (type h() ENTER for help)
sh: /tmp/example/_build/dev/lib/scenic_driver_glfw/priv/prod/scenic_driver_glfw: No such file or directory

08:47:17.330 [error] dirty close

And a scenic window never opens. I don't think it's a system dependency issue because I have another scenic application that's still working. I'll try and update all the deps on that later to narrow down possible issues.
Elixir version: Elixir 1.8.1 (compiled with Erlang/OTP 21)
Erlang version: 21.2.4
And this is with the latest scenic.new task

I have noticed the same issue. I wouldn't know how to diagnose it as I am an absolute beginner with Scenic and Elixir in general.
I noticed, though, that you can work around this issue pretty easily (I'd still like to understand it!):
mix do deps.get, scenic.run

Let it fail... abort.
mkdir _build/dev/lib/scenic_driver_glfw/priv/prod
ln -s \
  $(pwd)/deps/scenic_driver_glfw/priv/prod/scenic_driver_glfw \
  _build/dev/lib/scenic_driver_glfw/priv/prod/scenic_driver_glfw

Build again. Profit.

@Fusion what OS version are you using? And what version of Elixir and Erlang?

This is a not a very standard configuration:
It's Ubuntu 18.04, running in WSL.
Erlang/OTP 22 [erts-10.4.3] [source] [64-bit] [smp:4:4] [ds:4:4:10] [async-threads:1]
Elixir 1.9.0 (compiled with Erlang/OTP 21)

If this boils down to a command line fix - that sounds like an addition to the readme.

@boydm I get this error on Mac 10.14.5 as well so I think there's something going on. It seems odd that the scenic_driver_glfw is being built into the priv directory of scenic in the deps folder. I would think it would only be built into _build and not deps.

@axelson Yes that is weird. I'll try to repo.

I have a repo. Working on it.

@boydm I get this error on Mac 10.14.5 as well so I think there's something going on. It seems odd that the scenic_driver_glfw is being built into the priv directory of scenic in the deps folder. I would think it would only be built into _build and not deps.

I noticed that the Makefile for scenic uses $(MIX_COMPILE_PATH)/../priv while this Makefile uses priv.  The former compiles in _build while this one compiles in deps.
I also read through elixir_make which has this comment:
# IF there was no priv before and now there is one, we assume
# the user wants to copy it. If priv already existed and was
# written to it, then it won't be copied if build_embedded is
# set to true.

That's why priv/prod/scenic_driver_glfw isn't being copied the first time, but is once it already exists.
I have a patch which I'll be submitting a PR for.

Oh good! I am swamped with something else have haven't been able to focus on it properly.
I suspected it was something like that.

I have noticed the same issue. I wouldn't know how to diagnose it as I am an absolute beginner with Scenic and Elixir in general.
I noticed, though, that you can work around this issue pretty easily (I'd still like to understand it!):
mix do deps.get, scenic.run

Let it fail... abort.
mkdir _build/dev/lib/scenic_driver_glfw/priv/prod
ln -s \
  $(pwd)/deps/scenic_driver_glfw/priv/prod/scenic_driver_glfw \
  _build/dev/lib/scenic_driver_glfw/priv/prod/scenic_driver_glfw

Build again. Profit.

Worked for me as well, on OSX Mojave 10.14.5.

I'm using WSL too but in my case this file seemingly got created fine:
$ ls -l _build/dev/lib/scenic_driver_glfw/priv/prod
total 232
-rwxrwxrwx 1 trejkaz trejkaz 333864 Aug 29 22:25 scenic_driver_glfw

But still:
$ mix scenic.run

22:26:03.326 [error] dirty close

Ubuntu 18.04.4 LTS on WSL
Erlang/OTP 23 [erts-11.0.3] [source] [64-bit] [smp:8:8] [ds:8:8:10] [async-threads:1]
Elixir 1.10.4 (compiled with Erlang/OTP 22)
Different issue?


ext-mongo

Your requirements could not be resolved to an installable set of packages.
Problem 1
- mongodb/mongodb 1.0.0 requires ext-mongodb ^1.1.0 -> the requested PHP extension mongodb is missing from your system.
- jenssegers/mongodb v3.0.0 requires mongodb/mongodb ^1.0.0 -> satisfiable by mongodb/mongodb[1.0.0].
- Installation request for jenssegers/mongodb ^3.0 -> satisfiable by jenssegers/mongodb[v3.0.0].
To enable extensions, verify that they are enabled in those .ini files:
- D:\php-5.6.11-nts-Win32-VC11-x64\php.ini
You can also run php --ini inside terminal to see which files are used by PHP in CLI mode.
Installation failed, reverting ./composer.json to its original content.

Looks like you are missing ext-mongodb. Check the readme for the installation link.

same problem
I can find ext-mongodb in my computer system(osx  /usr/lib/php/extensions/no-debug-non-zts-20121212),   and I and extension=mongo.so to php.ini  . I don't know what's the reason@jenssegers


Document minification might break client

Minifiers changes reference names which might break the client library due to dynamic lookups in decorator implementation. This should be documented more clearly.
It can be solved with configuration like this (for example) :
  bundling: {
      sourceMap: true,
      minify: true,
      keepNames: true
  }

Fixed since decorators have been removed in version 8.0.0.


Remove bad labels, update naming

minor cleanup thing: can you also list the chart in the readme as well? https://github.com/lightstep/otel-collector-charts/blob/main/README.md


Poor examples which compare boolean parameter and 'true'

There are examples in the code which show a comparison of a boolean parameter with a string, for example here: https://learn.microsoft.com/en-us/azure/devops/pipelines/process/templates?view=azure-devops
parameters:
- name: doThing
  default: true
  type: boolean

steps:
- script: echo I did a thing
  condition: and(succeeded(), eq('${{ parameters.doThing }}', 'true'))

This works as expected but only because 'true' resolves to a truthy value. The following line would have the same effect:
condition: and(succeeded(), eq('${{ parameters.doThing }}', 'false'))
I don't think this would be expected by someone reading the docs. A better example can be found on this page: https://learn.microsoft.com/en-us/azure/devops/pipelines/process/templates?view=azure-devops
#azure-pipeline.yml
parameters:
- name: experimentalTemplate
  displayName: 'Use experimental build process?'
  type: boolean
  default: false

steps:
- ${{ if eq(parameters.experimentalTemplate, true) }}:
  - template: experimental.yml
- ${{ if not(eq(parameters.experimentalTemplate, true)) }}:
  - template: stable.yml

Note how true is not enclosed in quotes.

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

ID: 21e5cee4-eaae-3a96-db91-540ac759e83a
Version Independent ID: 9bdc837c-ffe0-d999-f922-f3a5debc7f92
Content: Conditions - Azure Pipelines
Content Source: docs/pipelines/process/conditions.md
Service: azure-devops-pipelines
Sub-service: azure-devops-pipelines-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

@SBFrancies -- Simon, thank you for your well-researched feedback.  There is a Note under Job, stage, and step templates with parameters that explains this distinction.
@juliakm -- Julia, please look into this possible documentation clarification.

@SBFrancies Thank you for the feedback! I think there may be some variations depending on the agent because my condition processes correctly with and without quotes. I'll remove the quotes and add a more detailed explanation to make it clearer that two functions get evaluated and succeeded() always returns true. Thank you for helping improve our docs!

Hi, I came across a similar problem in the documentation for conditions today.
The first example states this condition:
condition: and(succeeded(), eq('${{ parameters.doThing }}', true))
The example right afterwards, states the following condition:
condition: ${{ eq(parameters.doThing, true) }}
The intention behind the current statements is confusing to me (why compare 'true' to true?). If there is some reason for doing this (quoting VS not quoting) then that should be highlighted. If not, the conditions should match IMO, to make the example clearer.

@juliakm -- Julia, please review this update in private PR 6396.


Run CI tests on branches as well

We're only running the CI checks on the main branch right now.
It'd be helpful to run the CI checks on all branches, so it's easier to review PRs.  This will hopefully make it easier to avoid issues like this one.

If we use it by default on: pull_request we may start facing problems with the limit of GHA (3000 minutes/month as I remember). Maybe we should try to find a way to run it on_dispatch but with fork support? Because the problem with this PR was that I wasn't able to run action on fork and it's very hard to run tests on all versions locally (I only run on 3.3)...

@SemyonSinchenko - good point.  Would it be possible to run one set of CI tests (for a single Spark version) on the branches and the entire CI suite for all the Spark versions when merging with main?  You think that'd be a good compromise?

For me the perfect scenario will be to be able to run all spark tests manually on PR with a button. Looks like it is a tricky task but ChatGPT told me it is possible

So, we will have lightweight CI with linters that is triggered on every PR and also an option to run Unit tests on specific PR when we see it is necessary. Because linters requires 19 seconds but tests requires about 3 minutes and there is no reason to run it on PRs that, for example, change only a comma in docs

Another option is to set-up triggering only on a specific tag. So, for example, we can trigger tests only on PRs with tag bug or feature (or we can create our own tag like tests-required)

@MrPowers What do you think about all this options? Which one do you prefer?

@SemyonSinchenko - I like this suggestion: "So, we will have lightweight CI with linters that is triggered on every PR and also an option to run Unit tests on specific PR when we see it is necessary".  I suck at GitHub Actions, so hopefully we can provide some level of protection, but keep it simple.
Breaking main isn't great, but at least we have really solid checks setup now, so even if we break main, we can make sure to fix it before making a release.

Hey, ideally, there should be template(very easy to set up) for each kind of issue (bug, feature, docs) and PR template.
Each issue is automatically labeled by its usecase(enhancement, docs, bug etc..) and also starts with given title like (DOCS: IssuenName, FEATURE:IssueName, BUG:IssueName).
It gives u much more transparency and readability especially for people wanting to contribute.
And then u could trigger your CI workflows based on given label. So for new feature request and bugs ofcourse u want to run your spark versions etc..
For docs or something similar, not really.
You can assign this to me if u wish.

Thanks for offering to help @dinjazelena.  Let us know if you need any help.

Close this one for #203
Bceuase #203 is about some kind of a simple solution, this one may be reopened at any moment.


If is is a bug report, to help us reproducing this bug, please provide information below:

Your Operation System version (uname -a): Linux  5.10.0-23-cloud-amd64
The version of vineyard you use (vineyard.__version__): 0.14.5
Versions of crucial packages, such as gcc, numpy, pandas, etc.: gcc (Debian 10.2.1-6) 10.2.1 20210110
Full stack of the error (if there are a crash):

Traceback (most recent call last):
  File "/home/divyansh19464/vin-write.py", line 12, in <module>
    remote_blob_id = vineyard_rpc_client.create_remote_blob(remote_buffer_builder)
vineyard._C.IOErrorException: IOError: Send message failed: Broken pipe

Minimized code to reproduce the error:

import os
import vineyard

vineyard_rpc_client = vineyard.connect("<IP_ADDRESS>", 9600)

payload = os.urandom(1024 * 1024)
remote_buffer_builder = vineyard.RemoteBlobBuilder(len(payload))
remote_buffer_builder.copy(0, payload)
remote_blob_id = vineyard_rpc_client.create_remote_blob(remote_buffer_builder)
print(remote_blob_id)

I can produce the failure. Working on it.

Yep, the issue seems fixed now! Thank you for such an immediate response!


Error - System.OutOfMemoryException: Array dimensions exceeded supported range.

Describe the bug
When using ConcatenateIndexFiles with NoGaps rendering mode to create FCS for 672 1-min files representing 1 week of recording, with a framelength of 16,384 and acoustic indices calculated over 15 seconds, I get the following error:
System.OutOfMemoryException: Array dimensions exceeded supported range. (see screenshot)
Expected behavior
A FCS for the duration of one week. False Color Spectrograms were successfully derived using the same data with a lower framelength (2048, 4096 and 8192).
Diagnostics

[x] I have followed the instructions for reporting a bug

I have attached a log:

log_20190724T134204Z.txt

The Operating system I'm using:               Windows 10 Home Version 1803

The version of AnalysisPrograms.exe:          v<IP_ADDRESS>

The data used can be found:  Data consists 672 1-min files, representing 1-min recording every 15-min for 24h/day for 1 week, at a sampling rate of 384 kHz. A framelength of 16,384 was used, and acoustic indices were calculated over 15 seconds.

The command I used:

D:/User/Software/AP/AnalysisPrograms.exe ConcatenateIndexFiles E:/Acoustic_Data/AM1_AIs/BatchIndicesOutput7/Week_1 -o E:/Acoustic_Data/AM1_AIs/BatchIndicesOutput7/Week_1/Output_Concatenate -d *.WAV* -ip D:/User/Software/AP/ConfigFiles/IndexPropertiesConfig.yml -fcs D:/User/Software/AP/ConfigFiles/SpectrogramFalseColourConfig.yml -f Acoustic_Test -z 0 -g NoGaps --draw-images -n --concatenate-everything --log-level 7

The config file I used:

Screenshots

Additional Details
Add any other context about the problem here.

Thanks for the report (and for the attention to detail 😄). We've never processed data this large, in that particular dimension, before.
We have an internal re-design scheduled for the concatenate function (see #168) which should allow it to scale much better. Fixing this bug will be part of that.
In the meantime, I suggest running the concatenate in two parts. It is suboptimal, but it will work for now. The images can be placed next to each other for correct alignment. If you overlap the analysis a small bit temporally, then you paste the second image under the right edge of the first to hide the second y-axis produced.


Minor Changes to README.md

Related Issue

#1088

Proposed Changes

Gramatical error fixes and minor changes

Checklist

[ ] ✅ My code follows the code style of this project.
[ ] 📝 My change requires a change to the README.
[ ] 🎀 I have updated the README accordingly.
[ ] 👀 I have read the CONTRIBUTING document.
[ ] ✨ I have added tests to cover my changes.
[ ] 🚩 All new and existing tests passed.
[ ] 🌟 ed the repo

[ ] 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 change)

Starr the repo complete the checks .

@all-contributors please add @SarthakSKumar  doc


KAFKA-6711: GlobalStateManagerImpl should not write offsets of in-memory stores in checkpoint file

More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

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

This is a follow up to #4782
\cc @bbejeck @vvcephei

Updated this.

@mjsax Thanks for the PR, merging to trunk and 2.0

@rajinisivaram @mjsax the cherry-picking to 1.1 seems causing compilation to break:
/home/jenkins/jenkins-slave/workspace/kafka-pr-jdk8-scala2.11/streams/src/main/java/org/apache/kafka/streams/processor/internals/ProcessorStateManager.java:84: error: incompatible types: HashMap<Object,Object> cannot be converted to Map<String,StateRestoreCallback>

EDIT: not this PR, but the other one I think: 37978548b74521ed41db944dc45c41d761a4b068

Cherry-picked to 1.1, 1.0, 0.11.0 and 0.10.2.


Allow Java 8 language level for Standard if runtime in appengine-web.xml is java8

We should disable the suggestion to enforce language level 7.
We should remove blacklist.
We should allow servlet 3.1, along with 2.5, and expression language 2.2.

java 7 is already only enforced for != runtime java 8.
I'm not sure about the whitelist.  I thought java8 kept some sandbox constraints but we should check with @ludoch
+1 to not enforcing servlet 2.5 for runtime java 8.  Ideally an inspection/quickfix that checks for this.  I think we may have an issue open already. Probably worth opening an issue specifically for that. @etanshaul talked about this before, so it may already exist.

Just realised the common library uses AppEngineDescriptor. https://github.com/GoogleCloudPlatform/appengine-plugins-core/blob/b36e2b313c332d2664f8f8c715e1e94172d60eb2/src/main/java/com/google/cloud/tools/appengine/AppEngineDescriptor.java
We should probably start making use of it.

@ludoch is telling me there aren't constraints, and it should be removed

I am at the desk next to João. Where are you? :-)
On Tue, Mar 7, 2017 at 11:37 AM, João Martins<EMAIL_ADDRESS>wrote:

@ludoch https://github.com/ludoch is telling me there aren't
constraints, and it should be removed
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-intellij/issues/1291#issuecomment-284834798,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE4zXu_6X4tkCrNFBIBFaJssxq_XeZLks5rjbINgaJpZM4MV6_e
.

related: https://github.com/GoogleCloudPlatform/google-cloud-intellij/issues/1231

and yes, as @patflynn it is on my radar (but not yet started) to enhance support for the servlet version + java runtime version compatibility.

Just as a note that we should prefer our IJ native AppEngineStandardWebApp
descriptor over the common library unless there's some issue that makes it
inadequate.
On Tue, Mar 7, 2017 at 6:13 PM, Etan Shaul<EMAIL_ADDRESS>wrote:

and yes, as @patflynn https://github.com/patflynn it is on my radar
(but not yet started) to enhance support for the servlet version + java
runtime version compatibility.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-intellij/issues/1291#issuecomment-284891538,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHf5HaRafBzgYAi1HbGS1xviecCNQYNPks5rjeSFgaJpZM4MV6_e
.

Is the blacklist removed for Java 8?

I'm not sure. I think it may be gone or at least different. Did you on Ludo have a chance to discuss  it?


Dino webpage issue #59

Fixes #59
Description
The front end part of the webpage is done, only the animations and some javascript part is left.
Describe your changes in detail
Screenshots (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x ] I have read the CONTRIBUTING document.

SWOC Participation

Name: Manav Lohia
Discord Username: manav lohia#9433
[ x] I have included the issue number in my PR

Great!! I enjoyed building this webpage. Thanks for the opportunity Diganta. Some Javascript part is remaining and also I am left to add some animations should I create a new issue for that?? I will try and complete the new issues within this week so that you can review and merge it before the event ends ie. before 15th march

@ManavLohia945  Sure! Go ahead


Add more details on bumping catalog deps

Changes
Looking at https://github.com/tektoncd/catalog/pull/508 I realized there
is more to update than just the e2e scripts.
I'm actually wondering if we want to bump all of these - I think if
we're doing to do that, we should bump the version of the Task as well
(doesn't seem like it makes sense to have different copies of the same
Task at the same version behave differently). So I'm wondering if we
want to be more selective about when we bump these?
Would like you hear your thoughts @vdemeester @vinamra28
/hold
/kind documentation
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[n/a] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices
[x] Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
NONE

So… what would we do, if those images where not "tied" to the tektoncd/pipeline releases ?
Those images are there because they are still used as part of the PipelineResource. Depending on the future of the resources, they could be maintained elsewhere, at their own pace.
That said, I tend to agree that we may not want to bump those each time but this require us to go through the changelog of release and see if the behaviour changed or not ;and sometimes, it changes for some reason that do not appear evident just by looking at the changelog. Bumping blindly is the "easiest" path in that matter.

Note, that from the TEP-003, an update is required if we bump the image. If we follow it blindly, we should be those images at each new release of tektoncd/pipeline (not for bugfix release). One less "constraining" rules would be bump the version if the behavior changed between two version — but this is more involved because we would need to ensure no behavior changed between two tektoncd/pipeline release.

Should we merge this and have release managers update the dependencies in catalog? Release manager can a PR and the PR can be updated based on the feedback from catalog expert, similar to https://github.com/tektoncd/catalog/pull/576.

Sounds good to me 🙂
LGTM

Should we merge this and have release managers update the dependencies in catalog? Release manager can create a PR and the PR can be updated based on the feedback from catalog expert, similar to tektoncd/catalog#576.

There's still the question of whether or not to bump the Task version: https://github.com/tektoncd/catalog/pull/597/files#r539321005

I've created https://github.com/tektoncd/catalog/issues/644 to continue this discussion.


Duplicates appear in generated scripts.

Description
Duplicate #r lines appear in generated script files. Is this necessary? Can it cause problems?
Repro steps
Use the Entity Framework nuget package as a test case.
Expected behavior
I would expect to see the following in the generated script file:
#r "System.ComponentModel.DataAnnotations"
#r "../../../packages/EntityFramework/lib/net45/EntityFramework.dll"
#r "../../../packages/EntityFramework/lib/net45/EntityFramework.SqlServer.dll"

Actual behavior
Paket generates a file as follows:
#r "System.ComponentModel.DataAnnotations"
#r "System.ComponentModel.DataAnnotations"
#r "../../../packages/EntityFramework/lib/net45/EntityFramework.dll"
#r "../../../packages/EntityFramework/lib/net45/EntityFramework.SqlServer.dll"

Known workarounds
Create your own references file.

Oh we should dedupe these like we do in csproj references. PRs welcome.
Just filter it with a hashset or something
Am 04.09.2017 10:08 vorm. schrieb "Isaac Abraham"<EMAIL_ADDRESS>
:

Description
Duplicate #r lines appear in generated script files. Is this necessary?
Can it cause problems?
Repro steps
Use the Entity Framework nuget package as a test case.
Expected behavior
I would expect to see the following in the generated script file:
#r "System.ComponentModel.DataAnnotations"
#r "../../../packages/EntityFramework/lib/net45/EntityFramework.dll"
#r "../../../packages/EntityFramework/lib/net45/EntityFramework.SqlServer.dll"
Actual behavior
Paket generates a file as follows:
#r "System.ComponentModel.DataAnnotations"
#r "System.ComponentModel.DataAnnotations"
#r "../../../packages/EntityFramework/lib/net45/EntityFramework.dll"
#r "../../../packages/EntityFramework/lib/net45/EntityFramework.SqlServer.dll"
Known workarounds
Create your own references file.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/issues/2716, or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNNDhpRh1sSsO6hM3rVZ1QNO7wZJqks5se6_jgaJpZM4PLlZj
.

Maybe just a distinct rather than a hashset - need to maintain ordering I would imagine?

distinct is using a hashset ;-)

https://github.com/Microsoft/visualfsharp/blob/master/src/fsharp/FSharp.Core/seq.fs#L1084-L1089

Yes but it maintains order? Does Hashset?

look at the code. it just checks if the the thing is already in the set. It doesn't retrieve things from the set. So yes order is maintained

ok sorry.

Yes use distinct  @forki is only trying to confuse you ;)

It shouldn't actually cause any harm so up-for-grabs

Very kind of you When I can get the solution to open on my PC maybe I'll do it otherwise yes up for grabs please


Bump recursion limit to 200

While I'm not able to reproduce this problem locally @overvenus had this issue as well in https://github.com/pingcap/tikv/commit/641dfd1a48cb024779e45c40a43278585c13c727#diff-b4aea3e418ccdb71239b96952d9cddb6R26.
Fixes #2947.

LGTM

It occurs when I build fuzz targets cargo fuzz run codec_bytes.
error: recursion limit reached while expanding the macro `opts`
    --> /home/pingcap/stn/tikv/src/util/rocksdb/engine_metrics.rs:1305:9
     |
1305 | /         register_int_gauge_vec!(
1306 | |             "tikv_engine_oldest_snapshot_duration",
1307 | |             "Oldest unreleased snapshot duration in seconds",
1308 | |             &["db"]
1309 | |         ).unwrap();
     | |_________^
     |
     = help: consider adding a `#![recursion_limit="200"]` attribute to your crate
     = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)


setup.sh: substitute uses only valid bash names

bash variable names may only contain alphanumeric ASCII-symbols and _,
and must not start with a number. Nix expression attribute names however
might contain nearly every character (in particular spaces and dashes).
Previously, a substitution that was not a valid bash name would be
expanded to an empty string. This commit introduce a check that throws
a (hopefully) helpful error when a wrong name is used in a substitution.
That was discovered by chexxor when it happened to me in a substitution.
Additionally, substituteAll previously filtered out all environment
variables starting with an uppercase letter. That increased the number
of silently(!) ignored envvars (and nix attributes) even further.
Because it was quite arbitrary, it is removed.
This will trigger a mass rebuild.

I'm convinced the filtering is meant to reduce the risk of accidental substitution. It's a convention that standard global env vars are all-uppercase, and typically you have large amount of those in the env. On the other hand, the main intended purpose of substituteAll seems to act on converted nix variables which rarely start with an uppercase character.

The first change seems good to me. It's a bit unfortunate for subsituteAll that we prefer to use dashes in nix bindings. For example, I now tested passing a dashed attribute to a derivation and it's just not added to the env silently (or invisible to bash).

I'm convinced the filtering is meant to reduce the risk of accidental substitution.

@aszlig brought the same objection. However in my opinion the worst an automatic computing system can to is silently ignore user input.
So unless you can think of a way to throw an error for these cases I’m in strong favor of risking wrong substitutions.
Even better would be to have a clean namespace for these substitutions in the first place. Maybe that can be done, too.

If we do not change this filtering, we should at least document it in http://nixos.org/nixpkgs/manual/#fun-substituteAll
To me there seems a more pressing "silent" problem: if you forget to put $var into env or make a typo somewhere, any @var@ occurrences just won't be substituted. We might make substituteAll check afterwards that no such references have remained.

if you forget to put $var into env or make a typo somewhere, any @var@ occurrences just won't be substituted.
We might make substituteAll check afterwards that no such references have remained.

How about files where @something@ is intended to be left as-is? It’s the classic “symbol-space used by multiple layers” problem.

I don't think you should use substituteAll in such cases, as anyone can define $something for some other reason.

@vcunat I added the filter again and included documentation.
The first fallout of the changes is already visible, the documentation travis build is failing. If I did everything right, all of these failures should have been bugs from the get-go, though.

Are you sure about the failures? It seems like stdenv won't even bootstrap.
building path(s) ‘/nix/store/r6p6ipkqaf7ivbszwf25nnwf16fvr604-bootstrap-gcc-wrapper’

substitution variables must be valid bash names, "ccLDFlags+" isn't.

OMG, the implementation has been crappy all along – it uses env call to get all env variables, but that isn't one-var-per-line if there are multi-line strings in the vars...

Let me properly test a fix for that.

It got to building the final gcc so far, so it's probably OK. @Profpatsch: checkout https://github.com/Profpatsch/nixpkgs/pull/1 ;-)

@vcunat I changed it, should we make Travis build everything? :)

@vcunat What’s the next steps here? Should we merge into staging?

@Profpatsch sounds like the next logical step

Closed by https://github.com/NixOS/nixpkgs/commit/62616ec5e211d8cfbafe31c0876a1d8cdab9e98e. Thanks!

I think this PR is what causes that mass-breakage on darwin: http://hydra.nixos.org/build/34945843/nixlog/1/raw I assume the bash used at that point behaves differently, or something similar, as there seem to be no larger problems on the two linux platforms.

Unfortunately, this leaves me in kind-of stuck situation, as I can't well test darwin stuff and I don't like bringing that mass failure to master either...

Hm, do you think the set does not work the same on darwin bash? That’s nearly all we changed. Maybe someone on Mac can post an output of set?

And maybe an output of
for e in $(set); do
  if ! [[ "$varName" =~ ^[a-zA-Z_]+[a-zA-Z0-9_]*$ ]]; then
    echo "$varName"
  fi
done

I think using set instead of env is very dangerous, because it not only shows exported variables but also shell-internal variables. In addition to that it handles different types with different syntax.
What I think is the problem on Darwin is that the contents of a function is recognized as a variable name here. For example if I run set within bash on my system I get the following along the lines:
command_not_found_handle ()
{
    local p=/run/current-system/sw/bin/command-not-found;
    if [ -x $p -a -f /nix/var/nix/profiles/per-user/root/channels/nixos/programs.sqlite ]; then
        $p "$@";
        if [ $? = 126 ]; then
            "$@";
        else
            return 127;
        fi;
    else
        echo "$1: command not found" 1>&2;
        return 127;
    fi
}

While command_not_found_handle isn't usually set inside stdenv, other functions may have been set and whenever there is a line within that function without prior whitespace it might end up getting included in substitute.

Aah... thanks for finding this case, Aszlig. Unfortunately, as noted above, env instead handles multi-line vars badly which we use heavily (e.g. for phases).

We could force set to display only variables:
set -o posix
# use set to list vars
set +o posix

Any other/better ideas?

@vcunat: We can still use env -0 and cut -z to split off the name from the content.

an example of a var even with a newline:
while IFS= read -r -d '' var val; do
  echo ",,$var''"
done < <(env $'foo\nbar=blah\nxxx' env -0 | cut -d= -f1 -z)

prints along with others the following:
,,foo
bar''

I don't have time right now to implement this by myself, but the args for the substitution function can be gathered like this:
args=()
while IFS= read -r -d '' varname; do
  args+=("--subst-var" "$varname")
done < <(env -0 | cut -d= -f1 -z)

A small test yields the correct values even with newlines:
foo() { for i in "$@"; do echo ">$i<"; done; }
args=()
while IFS= read -r -d '' varname; do
  args+=("--subst-var" "$varname")
done < <(env $'foo\nbar=bar\nfoo' env -0 | cut -d= -f1 -z)
foo "${args[@]}"

The problem with set -o posix is that it still contains internal variables (but no functions) and they're still printed in bash syntax rather than the raw contents, which env -0 provides.

@vcunat: We can still use env -0 and cut -z to split off the name from the content.

Nice idea! I expect to try that properly tomorrow morning.

I think I would filter unusual variable names anyway, probably like this:
-    # Select all environment variables that start with a lowercase character.
-    # Will not work with nix attribute names (and thus env variables) containing '\n'.
-    for envVar in $(set | sed -e $'s/^\([a-z][^=]*\)=.*/\\1/; t \n d'); do
+    # Select all environment variables that start with a lowercase character
+    # and only contain reasonable characters (see grep below).
+    # We need to be careful due to vars with multi-line contents, etc.
+    for envVar in $(env -0 | cut -z -d= -f1 | grep -z '^[a-z][a-zA-Z0-9_]*$' | tr '\000' ' '); do

Stdenv is written in bash, so considering variables that don't have a valid bash varname probably wouldn't be good.

@vcunat Err nopenopenope.
 That’s the whole point. Everything that is not a bash variable is filtered out at https://github.com/NixOS/nixpkgs/pull/14907/files#diff-7a1764865f30c6cb4c22c2654ebe9a38R412 and leads to an explicit crash.

I somehow thought the PR was meant to address the case where those names are passed explicitly and ignored, not substituteAll.

No matter what this PR is about, but I think it certainly makes more sense to have this check once rather than twice. So if I didn't miss something, using a bash array variable to enumerate over the env variables is still the safest option, because no matter how "garbled" the variable is, it still gets passed correctly to substitute, even with spaces.
However, I did miss something, which are variables like _, which we probably don't want to be substituted. Unfortunately, having a check against ^[a-zA-Z_]+[a-zA-Z0-9_]*$ will substitute the variable (the previous regex in substituteAll did only match against [a-z]).

which are variables like _, which we probably don't want to be substituted

Why? I’m waiting for the first person losing an hour or two because it doesn’t work.

We really should fix the mass breakage on staging: so what... only filter ^[A-Z] silently, I suppose?

Or perhaps ^[_A-Z].

^([A-Z]|_$) maybe?

Why keep _FOO when we filter-out BAR?

Indeed. I thought of _foo cases but I don't even know if they are used by anyone.

I think not in nixpkgs, by reading output of git grep '@_.*@'.

Ouch, still sleepy and haven't thought of this regex. I think we should just go with it to fix breakage...

Why not revert the commit until we implement it correctly? Fixes breakage and we can find a good solution. :)

I didn't want to undo this whole PR, given that we've invested so much energy into it and it doesn't seem anyone really wants to implement it soonish.

So I opted to make an iteration. We can improve later.

Hm, the filtering of _ab and Ab was sensible from the beginning I think. But we definitely need to document it in the manual.

Ah, I already documented uppercase, why did we add underscores as well?

That filtering was there on master already. I thought that underscore-starting variables are considered "hidden", i.e. not to be used normally.
Personally, I think that using the shell substituteAll function isn't good, and we should mostly replace it by something less error prone, perhaps improved pkgs.substituteAll.

There is a „nix“ substituteAll, but it just calls the shell function.

Yes, and I believe we should change that to call substitute directly with variable list explicitly passed into shell (by listing the nix-passed bindings).

I’m preparing a patch for the manual.

No, darwin again. The bootstrapping shell apparently doesn't support process substitution. http://hydra.nixos.org/build/35382446/nixlog/1/raw @copumpkin: any ideas? It would be nice if we used recent bash even during bootstrapping...

@vcunat: A very ugly way of handling this would be to use something like:
varNames="$(mktemp)"
env -0 | cut ... > "$varNames"
while IFS= read -r -d '' varName; do
    ...
done < "$varNames"
rm -f "$varNames"
...

@vcunat: Another variant with eval:
declare -a args=()
eval "$(env -0 | cut -z -d= -f1 | while IFS= read -r -d '' varName; do
    echo 'args+=("--subst-var" '\'"${varName//\'/\'\\\'\'}"\'')'
done)"
echo "${args[@]}"
```bash

I'd personally prefer the latter... simply because I don't like temporary files :-D

I also dislike temporary files, but the escaping here... I can't really parse it :-)
Hmm, null character can't be used in $IFS, but = is also disallowed in variable names (on system/C level IIRC), so this should work:
  local IFS==
  local varNames=$(env -0 | cut -z -d= -f1 | tr '\000' '=')
  for varName in $varNames; do
    args+=("--subst-var" "$varName")
  done

+ the things around, e.g. filtering vars according to the first character, etc.

... except that even local IFS will be inherited to the child, but it's easy to override it back to the default.

So, pushed 62fc8859c1. Let's see if darwin's fine this time.

No luck. And this time it even isn't clear to me why the substitutions didn't happen: http://hydra.nixos.org/build/35421166/nixlog/1/raw I suppose we'll have to completely revert all this for now.

@vcunat: Using tr with octal escapes probably could be the problem here.
The escaping boils down to s/'/'\\''/g (escapeing ' in bash) and I think it should work on Darwin as well.
@abbradar or somebody else on Darwin, can you please test whether the following one-liner works?
env $'new\nline=something' bash -c 'declare -a args=(); eval "$(env -0 | cut -z -d= -f1 | while IFS= read -r -d '\'\'' varName; do echo '\''args+=('\''\'\'\''env: '\''"${varName//\'\''/\'\''\\\'\''\'\''}"\'\'\'')'\''; done)"; for var in "${args[@]}"; do echo "$var"; done'

The output should be the current environment variables along with this in between:
env: new
line

Sadly I'm not on Darwin, but an acquaintance is:
bash-3.2$ env $'new\nline=something' bash -c 'declare -a args=(); eval "$(env -0 | cut -z -d= -f1 | while IFS= read -r -d '\'\'' varName; do echo '\''args+=('\''\'\'\''env: '\''"${varName//\'\''/\'\''\\\'\''\'\''}"\'\'\'')'\''; done)"; for var in "${args[@]}"; do echo "$var"; done'
env: illegal option -- 0
usage: env [-i] [name=value ...] [utility [argument ...]]
cut: illegal option -- z
usage: cut -b list [-n] [file ...]
       cut -c list [file ...]
       cut -f list [-s] [-d delim] [file ...]

...

If it's bash-3*, we also won't have arrays IIRC.

Hm, indeed. How did it work before? (I suppose that usually we have proper Bash on Darwin, and this... thing... is used only for bootstrapping).

Yes, all tools should be GNU ones, the same versions as on Linux, except that I don't know how exactly bootstrapping is done.

сс @copumpkin I guess. Old code just haven't used Bash arrays. I'm out of ideas at this point again besides to bootstrap Bash on Darwin...

From more reading, Bash 3 doesn't have associative arrays. This, however, doesn't fix crippled env and cut...

I understand that this is very ugly, but seems like a plausible workaround for now: we can check $BASH_VERSION and if it starts from 3, use the old code for env (but we can still use arrays and such, so only one line needs to be guarded I guess).

I suppose we'll have to completely revert all this for now.

OK, so I think I shall push 577fefe1d36. Feel free to find some better way later. Let me reopen as it won't be fully resolved after that commit.

First there was an out-of-disk failure, but after restart we have all Hydra-built stdenvs succeeding again :-)

@abbradar, @vcunat: What's the minimum bash version we should aim for if we want to support Darwin as well?

I don't know, but in this case the largest stumbling block seems that env doesn't support -0 at some point during the bootstrap.

@vcunat: Hm, what about doing that portion in C instead... like this (untested):
#include <stdio.h>

int main(int argc, char **argv, char **envp) {
    char *tmp = NULL;
    fputs("local -a args=(", stdout);
    while (*envp != NULL) {
        if (tmp != NULL) putchar(' ');
        fputs("'--subst-var' '", stdout);
        tmp = *envp++;
        while (*tmp != 0 && *tmp != '=') {
            if (*tmp == '\'')
                fputs("'\\''", stdout);
            else
              putchar(*tmp);
            tmp++;
        }
        putchar('\'');
    }
    puts(")");
    return 0;
}

This could then be simply evaled and we directly get out the args variable, although it feels a bit like breaking a fly on the wheel :-D

OTOH... what about replacing the whole set of substitute* functions by a single small C program (substitute with the same functionality as before and substitute -i as an alias for substituteInPlace and substitute -a as an alias for substituteAll)? I mean the functionality is very simple and we don't need to make it ugly to read by even uglier shell workarounds.

Yeah, writing a C utility for all those substitute* functions seems a good longer-term goal. (Github won't allow to reopen this PR anymore.)

Finally we might make it an error when substituteInPlace does no changes :-)

@vcunat: Mhm, that might be a good idea for a lot of these sed calls as well. But let's first find if there is already something that does what we want before we start writing our own.

We really need a good DSL like Elm wraps JavaScript, but for Shell. Something simple, but well typed.

http://hpux.connect.org.uk/hppd/hpux/Users/replace-2.24/man.html

If you know a good typed shell-suitable language, please, tell me. (Getting further from the topic, I'm afraid.)

Scala with https://github.com/lihaoyi/Ammonite  ;-)

lol scala.
Let’s put everything in a JVM and call it Noracle.

Please, let's move this discussion elsewhere (sorry for bringing it up). Let's focus on the goal of the PR. Thank you!

I'm getting  a bunch of
building path(s) ‘/nix/store/cx3az6zxzzzyxzrk14bda2x9k6vb8v20-clang-wrapper-3.8.0’
WARNING: substitution variables should be valid bash names,
  "ccLDFlags+" isn't and therefore was skipped; it might be caused
  by multi-line phases in variables - see #14907 for details.
WARNING: substitution variables should be valid bash names,
  "ccCFlags+" isn't and therefore was skipped; it might be caused
  by multi-line phases in variables - see #14907 for details.
WARNING: substitution variables should be valid bash names,
  "ccLDFlags+" isn't and therefore was skipped; it might be caused
  by multi-line phases in variables - see #14907 for details.
WARNING: substitution variables should be valid bash names,
  "ccCFlags+" isn't and therefore was skipped; it might be caused
  by multi-line phases in variables - see #14907 for details.
WARNING: substitution variables should be valid bash names,
  "ccLDFlags+" isn't and therefore was skipped; it might be caused
  by multi-line phases in variables - see #14907 for details.
WARNING: substitution variables should be valid bash names,
  "ccCFlags+" isn't and therefore was skipped; it might be caused
  by multi-line phases in variables - see #14907 for details.
WARNING: substitution variables should be valid bash names,
  "ccLDFlags+" isn't and therefore was skipped; it might be caused
  by multi-line phases in variables - see #14907 for details.
WARNING: substitution variables should be valid bash names,
  "ccCFlags+" isn't and therefore was skipped; it might be caused
  by multi-line phases in variables - see #14907 for details.
WARNING: substitution variables should be valid bash names,
  "ccLDFlags+" isn't and therefore was skipped; it might be caused
  by multi-line phases in variables - see #14907 for details.
WARNING: substitution variables should be valid bash names,
  "ccCFlags+" isn't and therefore was skipped; it might be caused
  by multi-line phases in variables - see #14907 for details.
WARNING: substitution variables should be valid bash names,
  "ccLDFlags+" isn't and therefore was skipped; it might be caused
  by multi-line phases in variables - see #14907 for details.
WARNING: substitution variables should be valid bash names,
  "ccCFlags+" isn't and therefore was skipped; it might be caused
  by multi-line phases in variables - see #14907 for details.

Seemingly from:
      ccCFlags+=" -B${cc_solib}/lib"

in cc-wrapper.nix.

@copumpkin Seeing the same thing.

Yes, explained in 577fefe1d36.

@vcunat I'm not sure I understand what's actually happening here. Does that mean someone's trying to pass in a key called "ccCFlags+" from the Nix side? Isn't my example above just using the += bash notation?

@copumpkin: it's a false positive. The problem is that the string is in a multi-line variable (a build phase) and env separates the listing by newlines. As seen above, I wanted to fix that by using env -0 but that didn't work out as the parameter wasn't supported early in bootstrap (for Darwin, IIRC).

Ah yes, if you use /usr/bin/env, we have no control (literally, the OS wouldn't let us overwrite it even if we wanted to) over where that points in Darwin 😦

In this case env was just searched on current $PATH.

Hmm, then I'm confused, because we use the usual GNU coreutils everywhere inside of Nix:
[nix-shell:~]$ uname -a
Darwin truffles 16.0.0 Darwin Kernel Version 16.0.0: Mon Aug 29 17:56:20 PDT 2016; root:xnu-3789.1.32~3/RELEASE_X86_64 x86_64 i386 MacBookPro11,3 Darwin

[nix-shell:~]$ env -0 | head -n 1
GHC_DOT_APP=/Applications/ghc-7.10.2.appTERM_PROGRAM=Apple_TerminalSSL_CERT_FILE=/Users/copumpkin/.nix-profile/etc/ca-bundle.crtNIX_CC=/nix/store/gppkhlphb093bq35l4836s378fra84w5-clang-wrapper-3.7.1__sandboxProfile=(allow file-read* (literal "/usr/lib/libncurses.5.4.dylib"))

@copumpkin: yes, but it wasn't so at some point during bootstrapping, at least according to my comment on 81df035. You can try before that commit yourself.


Plugin displays <?> "Character not found" character in Avogadro2 menu entries in Windows 10

Source: The menu entries "Configure ..." and "Get xtb ..." use the single unicode character "…"
Problem: Menues in Windows 10 show this character as "�"
Solution: Replace "…" with "..." three consecutive dots in config.py : 201 and install.py : 155

This is done to match the other menu items in Avogadro, which have been standardized to use the ellipsis character rather than three full stops/periods. So I'll be keeping it as is.
Obviously the fact that it is displayed incorrectly is a problem that needs fixing, though.
I assume that other menu options with … display correctly for you?
In which case there is some problem in the way that Avogadro gets the options json from plugins. Presumably down to the character encoding used at various stages.

Indeed, Avogadro's native menu items show correctly.
But I was able to fix the problem: Inserting the following code in config.py line 41 should do the trick.
'''sys.stdout.reconfigure(encoding='utf-8')'''

Thanks for digging on this. Seems like it should be an easy fix.
It would be better to fix this on Avogadro's side of things though, as all plugins are going to be behaving the same on Windows.

Are you going to open the forum thread or shall I do it ?

I opened an issue but you're welcome to start the forum thread!

True. I added the fix for now.
@Ineluki80 I would appreciate you testing the plugin again. Then I'll make a 0.2.1 release.

Works beautifully now.


gomod: Update to 1.23, add types.Alias case

Changed

Build with 1.23.2
This brings a change to use the alias, rather than the underlying type (which 1.22 did)
Regenerated test files

Strange that it says that the coverage is only 50% because I thought all of it would be covered when we were going through this.


fix: Can't see image in webui v1.3.0

BUG  https://github.com/jexom/sd-webui-depth-lib/issues/28
Fix it by assign bg color.

you can check out the flies sort  or reset your webui

Hi @chace20
I've implemented your change but in a different way in my fork https://github.com/wywywywy/sd-webui-depth-lib so that the images can actually be seen.


Stop subscribing to the participant if we don't display it's camera

We should unsubscribe to the participants that aren't being shown on screen, also we should re-subscribe once the scren size is big enough to fit the camera or  the participants speaks gains priority.

Okay so after a few days of investigation I couldn't find anything to solve this issue here are the approaches I took and how they didn't turned out to work:
Approach number 1:
First thing I tried is(of course) to try to unsubscribe to the videoTrack of the users that weren't being shown. Let's say my screen could fit 5 cameras of the 10 participants that were actually on the call, so I tried to unsubscribe from the videoTrack but keep suscribed to the audioTrack(bc I still want to hear them).
Turns out I couldn't find a way to unscubscribe to a single track. The only way I found to subscribe/unsubscribe a track is through unsubscribing the participant(https://spaces-js-docs.mux.dev/classes/RemoteParticipant.html#unsubscribe).The problem with this method is that it unsubscribes the participant from BOTH tracks: audio and video so this approach is no longer valid since I still want to .
Approach number 2:
So after accepting the fact that there's no way to unsubscribe myself from the track I said: okay, why don't we just unsubscribe from both tracks and when the participant starts speaking we subscribe to his tracks back again?
I implemented this and I ran some tests. The result was it FAILED, when the remote participant started speaking I couldn't hear him for like 1-2 seconds and then I could start hearing him so this ain't the right solution.
@marcovidonis aiutooooo

Hi Felipe,
yes, you want to remain subscribed to the participant, otherwise re-subscribing will introduce a delay, as you experienced!
I think a possible solution would be to stop the video track. Our Mux spaces wrapper exposes the MediaStreamTrack object, so you could just add a stop() method to the Track class:
https://github.com/montevideo-tech/hybrid-meetup-platform/blob/develop/frontend/src/lib/webrtc.js#L16
You could also try detaching the track before or after stopping it, but I'm not sure if it's needed...
Then, when you want to show the video again, you could call Participant.getTracks(), select the video track and add it to a <video> element.
Hope this helps, let me know how it goes!


(1.19.2) Piercing on Tridents only hits up to 2 targets

I came across an issue where no matter how clumped a group of monsters are and no matter how high the Piercing enchantment is the trident doesn't seem to damage more than 2 targets. I don't know if that's intentional, but I assumed every level past 1 would increase the amount of targets hit by the projectile, just like with crossbows.
versions used is
Forge 43.2.3
Apotheosis 6.1.5
Placebo 7.1.7
the steps I used to reproduce this is enchanting a Trident with Piercing greater than 1, then test the trident on a 3 by 3, 2 block deep pit filled densely with creepers.

What is happening here is that due to how tridents track if they have hit an entity (for purposes of returning with the Loyalty enchantment), they cannot hit more than one entity in a given tick.  This means that clusters of enemies will not be pierced correctly, but a spread-out chain of enemies will work properly.
I'll explore how to keep the flag set to false.  I already reset it at the end of onHitEntity but clearly that isn't good enough.


Add timeout for proxy

@business-factory/pythonistas Or do you prefer some other value? Maybe even separate values for connection timeout and read timeout?

@harpener I would prefer at least 20 secs for read timeout. So (10, 20) because sometimes even Lisa is not faster than 10 secs :(

@miso-belica Tomeš came here that cpt. Hook was down probably because of Selena test which try to download unavailable image from some FTP. It consumes all connections apparently.


Fix Multi-module coverage for modules with 'dot .' in the name

Description:
Muti-module coverage does not work with modules that have a dot in the name.
The ISourceFileCoverage class used in CodeCoverage report returns a package name which has the dot . replaced with an underscore _.
  String sourceFileModule = sourceFileCoverage.getPackageName().split("/")[1];

ModuleCoverage.getInstance().updateSourceFileCoverage(jsonCachePath, sourceFileModule,
                                    sourceFileCoverage.getName(), coveredLines, missedLines);

This causes the path to the source files to be resolved incorrectly, thus resulting in a NoSuchFileException.
This can be seen for modules such as java.jdbc which inturn becomes java_jdbc.
Simply replacing the _ with . in sourceFileModule will not work as this can interfere with modules that contain _ originally in their name.
Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

https://github.com/ballerina-platform/ballerina-lang/issues/26477
https://github.com/ballerina-platform/ballerina-lang/issues/26431
Suggested Labels (optional):

Suggested Assignees (optional):

It is worth noting that since _foo and .foo can exist simultaneously in a project, the .foo is converted to _foo and can result in some conflicts.

@Ibaqu I did a bit of debugging and the issue is from the jar files generated during ballerina build that get added to "target/caches/jar_cache".
For example path for the main.class is as follows for "diffmod.dil" module when I extracted and check the above jar,
dilhashanazeer/diffmod_dil/0_1_0/main.class

Once the "Improve code gen methods to reduce clashes" has been finalized :
https://github.com/ballerina-platform/ballerina-lang/issues/26346
We should be able to use the following function decodeIdentifier : https://github.com/ballerina-platform/ballerina-lang/blob/70c5342829156da8525211726b1548d2ad018271/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/IdentifierUtils.java#L88 to get the original module name.

https://github.com/ballerina-platform/ballerina-lang/pull/26683


nl-nl, nl-be or just nl?

I made sure to weed out most sentences that were too northern Dutch (nl-nl) or too Flemish (nl-be).
However, the name nl-nl seems to be too specific. Shouldn't it be just nl if we want to use this for both language variants?

This is a good point. I need to be careful about naming the voice, etc. derived from these prompts too. But for locales, I usually only see nl_NL or nl_BE. Is there a third more generic one?

Well the voices should definitely have nl_BE in their name if the speakers are Flemish, but I think the prompts should just use the generic locale nl, as they can be spoken both by Dutch and Flemish speakers.
However, thinking about these differences I just found some inconsistencies in the dictionary. What's the source of the nl.dict.gz file? Because this is Northern Dutch pronunciation:
politie p o ˈl i t s i
But this is Flemish pronunctiation:
politiebediende p o ˈl i s i b ə ˌd i n d ə
Notice the difference: politie is pronounced p o ˈl i t s i in Northern Dutch and p o ˈl i s i in Flemish.

The pronunciations are coming from the Dutch wiktionary. For "politie", the IPA is / poˈli(t)si / which contains the optional (t). It looks like I need to make the parser generate both forms (with and with the t) when it encounters optional pieces.


Testing

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@johnomotani I am trying to enable windows builds - do you know how to do that?
I think I have a patch to get it building on windows ...

I don't know - I'd have expected the change you made to do it!

I had to run conda-smithy.
Rather then spending so much time to get it running with python3.12 I could have just use a magic comment as mentioned in the template :see_no_evil:

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender


Build scripts are difficult to run in a debugger

The output of Cargo has a path to the actual binary that is invoked every time a build script is run. That's useful, but there's not output showing all of the environment variables that are set that the build script may expect to be present. This makes it very difficult to launch the build script under a debugger and replicate the conditions of the Cargo build in order to investigate something that isn't working correctly.

As of #12829, we now show the env variables as part of -vv so I think this is resolved.  If there is something I'm missing, let us know!


PTI Calculator method added to User

Linked Issue
https://github.com/autotrader-plus/autotrader-plus/issues/55

Context
Didn't have PTI calculator before -- now added as it's often used to help determine whether or not someone can get the loan.

Your Changes
Description:

Added a method getPTI which returns the PTI of the User. If the User is a basic User, it returns 0.00, otherwise it'll return the calculation of monthly budget / monthly income. Where monthly budget represents the max monthly payments of a potential loan.
Type of change (select all that apply):

[x] New feature (additional functionality separate from existing code)

Testing
Added test case for basic and advanced user. Both ran successfully.

Checklist

[x] Code is up to date with main branch (no merge conflicts)
[x] Code has been properly tested
[x] Code has been properly documented
[ ] One or two people have reviewed this pull request

Why does it return the String instead of Double, but it looks good regardless

Just follows the type returned for the other getter methods. I figured it'd be easier passing it back into the frontend not having to deal with parsing anywhere else


Recurse references in validation

JSON schema references can be nested in complex schemas.
Recurse in dereferencing logic until the returned item is no
longer a reference.
Also: Added a recursion to test data and fixed a typo
in test_shortcuts.py.

@evahala thank you for the contribution.


Added --name argument to set window title

First stab at doing anything in Rust, so I may have butchered the String vs &str stuff, but this seems to work. Should close #65.

Thanks for the PR! This looks pretty good overall. Would you mind changing name to something more descriptive like title?

Sure. Amended the commit with that change.

This looks good; thanks!


Centralizes Toon data, reducing API calls

Fixes #21825
Signed-off-by: Franck Nijhof<EMAIL_ADDRESS>

Hmz, VScode just ignores the templates when creating a PR on GitHub... sorry about that, will update OP in a second.

Blocked by https://github.com/home-assistant/home-assistant/pull/23899

For your next PR, please keep styling changes separate from functional changes. Makes it hard to see what is going on

You are absolutely right, sorry about that.

Since the blocking PR has been merged:
Will resolve merge conflicts and resolve last review comment tonight.

Blocked by #25027

Rebased onto origin/dev, addressed review comment.
Ready for review.

I have a followup PR ready that enhances this a bit more and improves the use of Climate 1.0. However, I did not want to mix it up into this PR.


Read the divisor from the API

Right now, the divisor for the values from the API is taken as a fixed number. but the API actually deliverst that value under the "CfgFactor" key. Read that value, fall back to 1, and apply it during the parsing phase.

Closed by #191


MQTT brightness_scale conversion results in inaccurate values

Home Assistant release (hass --version):
Latest release, 0.62.1
Python release (python3 --version):
Hass.io default install, updated this week
Component/platform:
/components/light/mqtt.py
Description of problem:
Issue with mapping MQTT dimmer intensity value from Homebridge ->Home assistant -> MQTT dimmable light
Relevant components:

MQTT dimmer light accepts light intensity levels 1-100 (Itead LED strip with Tasmota firmware)
HA MQTT light is configured with brightness_scale: 100
Homebridge (via hass.io plugin) is installed and configured on iPhone

Problem: when setting dimmer value to say 14%, the actual MQTT value sent to dimmer will almost always be lower, e.g. 13%. This results in light responding back with status MQTT message that the value was set to (lower) value (e.g. 13%), which in turn causes an update to HA and then Homebridge/ Homekit resulting in sub-optimal user experience (jerky and slow updates, changing values, and unable to set, say 1% since the light will be turned off instead of set at 1%)
The issue lies in the way HA converts Homebridge values (0-100%) to Dimmer values 0-255 and then back to MQTT light as per brightness_scale value (100).
It seems to me that the offending lines are at least these (could be others, in other components maybe?):

/components/light/mqtt.py:https://github.com/home-assistant/home-assistant/blob/2ae0c5653ed32c8b7309686d9b897f7d7c6fc0fb/homeassistant/components/light/mqtt.py#L249-L250
/components/light/mqtt.py:https://github.com/home-assistant/home-assistant/blob/2ae0c5653ed32c8b7309686d9b897f7d7c6fc0fb/homeassistant/components/light/mqtt.py#L452-L453

I.e. the percent_bright float value always gets rounded down when converting to int. Inserting round() before conversion should improve/solve the problem. As far as my testing goes this seems to work, but there might be better/more elegant solutions and same fixes maybe need to be applied elsewhere.

I have tested the following changes and it seems to work perfectly! Can somebody more skilled than me update the official repository?
/components/light/mqtt.py:

self._brightness = int(round(percent_bright * 255))

device_brightness = int(round(percent_bright * self._brightness_scale))
Thanks!  @TD22057 maybe?
P.S. I noticed that every time the brightness is changed, a redundant POWER ON command is sent via MQTT. If changing brightness via Homekit slider it updates frequently sending multiple brightness values and the light struggles to keep up with all the commands thus visible delays can be observed.
Changing this line: https://github.com/home-assistant/home-assistant/blob/2ae0c5653ed32c8b7309686d9b897f7d7c6fc0fb/homeassistant/components/light/mqtt.py#L508
to only send POWER "ON" command when brightness is not changed at the same time seems to fix the problem (my light will power on whenever it receives brightness command anyway):
if self._on_command_type == 'last' and ATTR_BRIGHTNESS not in kwargs:

Still an issue - I'll try and look at this in a week or so.


Add new endpoint to BFF to get actors to a userId

Codecov Report

Merging #1241 (b22d3fd) into main (7d29f71) will increase coverage by 11.52%.
The diff coverage is 11.76%.

@@             Coverage Diff             @@
##             main    #1241       +/-   ##
===========================================
+ Coverage   66.12%   77.64%   +11.52%
===========================================
  Files          21       35       +14
  Lines         307      595      +288
  Branches       27        0       -27
===========================================
+ Hits          203      462      +259
- Misses         94      115       +21
- Partials       10       18        +8

Impacted Files
Coverage Δ

...Api/Controllers/MarketParticipantControllerBase.cs
0.00% <ø> (ø)

...Api/Controllers/MarketParticipantUserController.cs
0.00% <0.00%> (ø)

...s/MarketParticipantUserRoleAssignmentController.cs
0.00% <0.00%> (ø)

...ers/MarketParticipantUserRoleTemplateController.cs
0.00% <0.00%> (ø)

...urce/DataHub.WebApi/Controllers/TokenController.cs
0.00% <0.00%> (ø)

apps/dh/api-dh/source/DataHub.WebApi/Startup.cs
96.38% <100.00%> (+2.40%)
:arrow_up:

...taHub.WebApi/Controllers/Wholesale/Dto/BatchDto.cs
0.00% <0.00%> (-100.00%)
:arrow_down:

...gistration/HealthEndpointRegistrationExtensions.cs
36.84% <0.00%> (-42.11%)
:arrow_down:

...WebApi/RequireNonNullablePropertiesSchemaFilter.cs
80.00% <0.00%> (-8.00%)
:arrow_down:

... and 12 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fonts can't load all fonts from .ttc files with FreeType engine

Sometimes fonts are found in .ttc archive files which contain multiple related fonts.  In these cases you need to pass the index of the font in the file to free type as well as the file, I think here:
https://github.com/celiagg/celiagg/blob/1af9287ea193ca3e355526c4481cf6e34fa57bae/celiagg/font.cpp#L60
But that means that the Celiagg Font needs to have the index information available (and assume 0 if not provided).

Sure enough, that's exactly where we can do this. The Freetype docs have more detail. (face_index is what that 0 gets passed as)


Error response from daemon: ttrpc

@lichao060103 Could you please provide some details if you are running any recipe from docker cookbook.
Closing it for now, you can open a new issue with enough details.


Cannot run on IE or EDGE

when I download this demo;  I can run it on chrome ;but not send message on ie or edge ; always say io is not defined

Closed due to inactivity, please reopen if needed.

Closed due to inactivity, please reopen if needed.


Intermittent test failure in io.helidon.webserver.PrematureConnectionCutTest.cutConnectionBefore100Continue

Stack seen in a recent pipeline build:
io.helidon.webserver.PrematureConnectionCutTest.cutConnectionBefore100Continue  Time elapsed: 3.012 s  <<< FAILURE!
java.lang.AssertionError:

Expected: is <5>
     but: was <4>
	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:8)
	at io.helidon.webserver.PrematureConnectionCutTest.cutConnectionBefore100Continue(PrematureConnectionCutTest.java:94)

Looks like https://github.com/oracle/helidon/pull/3365 is maybe? supposed to fix this?

Happened to me today on local machine


Add EVP_PKEY_public_check_quick.

Adding the EVP_PKEY_param_check_quick() reminded me that there are also
partial checks for public keys as part of SP800-56A for FFC (DH named safe
prime groups) and ECC. The code was mainly already there and just needed
to be plumbed into the validate methods.

Checklist

[x] documentation is added or updated
[ ] tests are added or updated

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests.  Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.

rebased with no changes.. Should not need reapproval.

Thanks for reviewing. Merged to master.


Access subfolder

Laravel Version
10.4
PHP Version
8.1
Database Driver & Version
Mysql 8
Description
I need to release a subfolder within the public (public/api) where there is another system (index.php) but Laravel does not allow access, giving a 405 error. How can I release access without going through Laravel? grateful
Steps To Reproduce
release subfolder within public for free access without going through laravel routes

Hi there,
Such usage is not supported by Laravel: https://laravel.com/docs/10.x/installation#directory-configuration