fix: npm scripts on Windows

Closes #5220
Single quotes weren't resolved correctly on Windows. The single quotes don't seem necessary on any OS as they were never wrapping a space character.
Knip also reported prettier as unused, so I've added prettier to Knip's ignore list.
npm test is now passing on a clean install from on Windows, where it was failing before.
Ref https://github.com/mark-wiemer/mocha/pull/1 for status checks

This is a bit unintuitive to me – my impression is that the quotes should be there so that the js-based glob resolves rather than the shell glob 🤔

@voxpelli I'm not a glob expert, in my book if it works it works, but obviously if you don't feel comfortable merging that's fine. I'll dig deeper as obviously the tests are now failing on other operation systems

Investigating remaining issues now, ref https://github.com/mark-wiemer/mocha/pull/1:

https://github.com/mark-wiemer/mocha/actions/runs/11094433059/job/30821980668?pr=1
https://github.com/mark-wiemer/mocha/actions/runs/11094433059/job/30821980743?pr=1

I'm able to reproduce this issue locally with Node 22 on Windows, looks like the same err message. Going to see if different versions of 22.x work:
Full output
~/repos/mocha (mark-wiemer/lint)
$ fnm use 22
2024-09-29 10:35:59
Using Node v22.9.0
[last: 0s]
~/repos/mocha (mark-wiemer/lint)
$ npm run test-node:integration
2024-09-29 10:36:06

><EMAIL_ADDRESS>test-node:integration
> run-s clean build && npm run -s test-node-run -- --parallel --timeout 10000 --slow 3750 "test/integration/plugins/root-hooks.spec.js"

><EMAIL_ADDRESS>clean
> rimraf mocha.js mocha.js.map

><EMAIL_ADDRESS>build
> rollup -c ./rollup.config.js

./browser-entry.js → ./mocha.js...
created ./mocha.js in 1.9s

  root hooks
    when mocha run in serial mode
      ✔ should run root hooks when provided via mochaHooks object export
      ✔ should run root hooks when provided via mochaHooks function export
      support ESM when type=module or .mjs extension
        ✔ should run root hooks when provided via mochaHooks
      support ESM via .js extension w/o type=module
        1) should fail due to ambiguous file type
    when mocha in parallel mode
      ✔ should run root hooks when provided via mochaHooks object exports
      ✔ should run root hooks when provided via mochaHooks function export
      when running multiple jobs
        ✔ should run root hooks when provided via mochaHooks object exports for each job

  6 passing (9s)
  1 failing

  1) root hooks
       when mocha run in serial mode
         support ESM via .js extension w/o type=module
           should fail due to ambiguous file type:

expected Promise when fulfilled to contain output /SyntaxError: Unexpected token/
  expected
  {
    output: '\n✖ \x1b[31mERROR:\x1b[39m Error [ERR_REQUIRE_CYCLE_MODULE]: Cannot require() ES Module C:\\Users\\markw\\repos\\mocha\\test\\integration\\fixtures\\plugins\\root-hooks\\root-hook-defs-esm-broken.fixture.js in a cycle.\n    at ModuleLoader.importSyncForRequire (node:internal/modules/esm/loader:303:15)\n    at loadESMFromCJS (node:internal/modules/cjs/loader:1381:24)\n    at Module._compile (node:internal/modules/cjs/loader:1503:5)\n    at Module.replacementCompile (C:\\Users\\markw\\repos\\mocha\\node_modules\\append-transform\\index.js:60:13)\n    at Module._extensions..js (node:internal/modules/cjs/loader:1691:10)\n    at Object.<anonymous> (C:\\Users\\markw\\repos\\mocha\\node_modules\\append-transform\\index.js:64:4)\n    at Module.load (node:internal/modules/cjs/loader:1317:32)\n    at Module._load (node:internal/modules/cjs/loader:1127:12)\n    at TracingChannel.traceSync (node:diagnostics_channel:315:14)\n    at wrapModuleLoad (node:internal/modules/cjs/loader:217:24)\n    at cjsLoader (node:internal/modules/esm/translators:329:5)\n    at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:260:7)\n    at ModuleJob.run (node:internal/modules/esm/module_job:262:25)\n    at async onImport.tracePromise.__proto__ (node:internal/modules/esm/loader:483:26)\n    at async formattedImport (C:\\Users\\markw\\repos\\mocha\\lib\\nodejs\\esm-utils.js:2:567)\n    at async exports.requireOrImport (C:\\Users\\markw\\repos\\mocha\\lib\\nodejs\\esm-utils.js:7:1135)\n    at async exports.handleRequires (C:\\Users\\markw\\repos\\mocha\\lib\\cli\\run-helpers.js:38:364)\n    at async C:\\Users\\markw\\repos\\mocha\\lib\\cli\\run.js:18:40 {\n  code: \'ERR_REQUIRE_CYCLE_MODULE\'\n}\n',
    code: 1,
    args: [
      'C:\\Users\\markw\\repos\\mocha\\bin\\mocha.js',
      '--require=C:\\Users\\markw\\repos\\mocha\\test\\integration\\fixtures\\plugins\\root-hooks\\root-hook-defs-esm-broken.fixture.js',
      '--no-color',
      '--no-bail',
      '--no-parallel'
    ],
    command: 'C:\\Users\\markw\\repos\\mocha\\bin\\mocha.js --require=C:\\Users\\markw\\repos\\mocha\\test\\integration\\fixtures\\plugins\\root-hooks\\root-hook-defs-esm-broken.fixture.js --no-color --no-bail --no-parallel'
  }
  to contain output /SyntaxError: Unexpected token/

  expected Promise when fulfilled to contain output /SyntaxError: Unexpected token/
    expected
    {
      output: '\n✖ \x1b[31mERROR:\x1b[39m Error [ERR_REQUIRE_CYCLE_MODULE]: Cannot require() ES Module C:\\Users\\markw\\repos\\mocha\\test\\integration\\fixtures\\plugins\\root-hooks\\root-hook-defs-esm-broken.fixture.js in a cycle.\n    at ModuleLoader.importSyncForRequire (node:internal/modules/esm/loader:303:15)\n    at loadESMFromCJS (node:internal/modules/cjs/loader:1381:24)\n    at Module._compile (node:internal/modules/cjs/loader:1503:5)\n    at Module.replacementCompile (C:\\Users\\markw\\repos\\mocha\\node_modules\\append-transform\\index.js:60:13)\n    at Module._extensions..js (node:internal/modules/cjs/loader:1691:10)\n    at Object.<anonymous> (C:\\Users\\markw\\repos\\mocha\\node_modules\\append-transform\\index.js:64:4)\n    at Module.load (node:internal/modules/cjs/loader:1317:32)\n    at Module._load (node:internal/modules/cjs/loader:1127:12)\n    at TracingChannel.traceSync (node:diagnostics_channel:315:14)\n    at wrapModuleLoad (node:internal/modules/cjs/loader:217:24)\n    at cjsLoader (node:internal/modules/esm/translators:329:5)\n    at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:260:7)\n    at ModuleJob.run (node:internal/modules/esm/module_job:262:25)\n    at async onImport.tracePromise.__proto__ (node:internal/modules/esm/loader:483:26)\n    at async formattedImport (C:\\Users\\markw\\repos\\mocha\\lib\\nodejs\\esm-utils.js:2:567)\n    at async exports.requireOrImport (C:\\Users\\markw\\repos\\mocha\\lib\\nodejs\\esm-utils.js:7:1135)\n    at async exports.handleRequires (C:\\Users\\markw\\repos\\mocha\\lib\\cli\\run-helpers.js:38:364)\n    at async C:\\Users\\markw\\repos\\mocha\\lib\\cli\\run.js:18:40 {\n  code: \'ERR_REQUIRE_CYCLE_MODULE\'\n}\n',
      code: 1,
      args: [
        '--require=C:\\Users\\markw\\repos\\mocha\\test\\integration\\fixtures\\plugins\\root-hooks\\root-hook-defs-esm-broken.fixture.js',
        '--no-color',
        '--no-bail',
        '--no-parallel'
      ],
    }
    to contain output /SyntaxError: Unexpected token/
      at Array.forEach (<anonymous>)
  From previous event:
      at Context.<anonymous> (test\integration\plugins\root-hooks.spec.js:140:16)
      at callFn (lib\runnable.js:110:584)
      at Runnable.run (lib\runnable.js:110:57)
      at Runner.runTest (lib\runner.js:182:1054)
      at C:\Users\markw\repos\mocha\lib\runner.js:210:654
      at next (lib\runner.js:159:422)
      at C:\Users\markw\repos\mocha\lib\runner.js:159:809
      at next (lib\runner.js:147:579)
      at Immediate.<anonymous> (lib\runner.js:151:1253)
      at process.processImmediate (node:internal/timers:491:21)
      set UNEXPECTED_FULL_TRACE=true to see the full stack trace

[last: 27s]

Last good node: 22.6.0
First bad node: 22.7.0

Added tests to distinguish between --no-experimental-module-detection and --experimental-module-detection as the latter is the default in Node 22.7 and later


Flyout links incorrect on Mkdocs projects

GitHub links in the sidebar footer contain an extraneous path. For example, on http://tracks.readthedocs.org/en/latest/upgrading/ the "View" link points to https://github.com/TracksApp/tracks/blob/master/home/docs/checkouts/readthedocs.org/user_builds/tracks/checkouts/latest/doc/Upgrading.md.

#1480 was closed even though the root cause seems to be the same. Yet, the flyout links are still incorrect.

I was trying to fix this and believe I found the cause, I was able to get a working version for a MkDocs based project.
More info here: https://github.com/rtfd/readthedocs.org/commit/98b5d34316917e0bdb26404c6f3816108ce7f42e

@agjohnson any thoughts on my comment in https://github.com/rtfd/readthedocs.org/commit/98b5d34316917e0bdb26404c6f3816108ce7f42e
I have tested the change locally using the directions here: https://docs.readthedocs.org/en/latest/install.html#installing-read-the-docs and it works for a MkDocs and Sphinx based projects.

@ericholscher do you have any thoughts on this? If I have a fix that is tested, so I can create the PR. Want to get this fixed as we just went live with our ReadTheDocs integration.

@sahilTakiar Definitely send the PR over!

The same here.
Steps to reproduce:

Go to this page.
Click Read the Docs in the bottom, then View or Edit.

The links are like:
https://github.com/geerlingguy/drupal-vm/blob/master/home/docs/checkouts/readthedocs.org/user_builds/drupal-vm/checkouts/latest/docs/index.md
https://github.com/geerlingguy/drupal-vm/edit/master/home/docs/checkouts/readthedocs.org/user_builds/drupal-vm/checkouts/latest/docs/index.md

Same here: http://mathquill.readthedocs.io/en/latest/

#2066 was merged hours ago and thus this should now be closed I guess.

Thanks @marcelstoer. I just rebuilt and the issue persists so I don't believe this should be closed until the change is deployed.

Just deployed this, so it should hopefully be fixed.

It appears to be. Thanks @sahilTakiar and @ericholscher!

It works for the OP's project at http://tracks.readthedocs.org but it doesn't work for any other project listed here in the comments: http://docs.drupalvm.com,  http://mathquill.readthedocs.io and http://nodemcu.readthedocs.io (mine) still have this odd ".../docs/checkouts/readthedocs.org/..." URLs. What's the difference between them? I even just built mine again to be sure.

same for me. The view and edit URLs have the same '.../docs/checkouts/readthedocs.org/...' in them
I thought maybe it was related to having edit_uri set in mckdocs.yml, but removing it and rebuilding seems to make no difference. @marcelstoer doesn't even have the set in their mkdcos.yml.

https://psraw.readthedocs.io/en/latest/
https://psmsgraph.readthedocs.io/en/latest/
https://autodocumentsexample.readthedocs.io/en/latest/

edit_uri is a feature that appeared in mkdocs 0.16 and that would help in solving that I think. But it seems RTFD is still using mkdocs 0.15 according to my build logs.
Is an upgrade to mkdocs 0.16 planed ? Or is it something that could be contributed by someone relatively easily ?

I didn't know that this issue was already reported, I'm fixing this in https://github.com/rtfd/readthedocs.org/pull/3525

Sorry to ping in a closed issue, but I'm seeing the same broken "View/Edit on GitHub" links in this project, which is using Mkdocs for the build: http://fieldguide.opennews.org/en/latest/. Both URLs have the ".../docs/checkouts/readthedocs.org/..." path stuff appended.
I just triggered a fresh build to see if that would clear things up (it had been a while), but no luck. Is there a setting I need to update?

Links are correct for me

Hmmm. I hoped maybe it was just a caching thing so I cleared that out, but the links are still broken for me. (See URL at bottom of screenshot)

Links are correct for me too. Might be CDN related cache?

Yeah, I think so, CDN or ISP caching


channel: update benchmarks

Closes #231

bors r+

This also contains a bug fix, does that have a test somewhere?

@Thomasdezeeuw Good idea - I've added a test in https://github.com/crossbeam-rs/crossbeam/pull/284


Standard Lookup field references are not retrieved on custom profile using sf project retrieve start --manifest

Summary
Using the command sf project retrieve start --manifest results in standard lookup fields not being retrieved in the profile definition. Other standard fields are retrieved, but those where data type = 'lookup' are not retrieved along with the profile.
If the entire customobject is retrieved then the lookup fields appear in the profile definition, but retrieving with only the fields specified results in the FLS for the field not being retrieved in the profile.
For example, the following manifest will result in the FLS for the field not being retrieved;

Test STND Retrieve
Profile

Contact.AccountId
CustomField

60.0

However if the manifest is changed to reference the custom object, then the FLS is returned in the profile definition.

Test STND Retrieve
Profile

Contact
CustomObject

60.0

Steps To Reproduce
Is reproducible in any project

Modify FLS for Contact.AccountId on a custom profile
Create manifest in local project which contains the profile and Contact.AccountId field
Run command sf project retrieve start --manifest  from the local project
The profile returned does not have any reference to the field Contact.AccountId

Expected result
FLS for the Contact.AccountId field is returned in the profile definition
Actual result
FLS is missing for the field
System Information
Which shell or terminal are you using?
zsh
Paste the full output of the sf version --verbose --json command below -->
{
"architecture": "darwin-arm64",
"cliVersion"<EMAIL_ADDRESS>"nodeVersion": "node-v20.12.2",
"osVersion": "Darwin 23.4.0",
"rootPath": "/Users/leecunningham/.local/share/sf/client/2.42.6-2b3c407",
"shell": "zsh",
"pluginVersions": [
"@oclif/plugin-autocomplete 3.0.18 (core)",
"@oclif/plugin-commands 3.3.4 (core)",
"@oclif/plugin-help 6.0.22 (core)",
"@oclif/plugin-not-found 3.1.9 (core)",
"@oclif/plugin-plugins 5.0.21 (core)",
"@oclif/plugin-search 1.0.24 (core)",
"@oclif/plugin-update 4.2.12 (core)",
"@oclif/plugin-version 2.1.2 (core)",
"@oclif/plugin-warn-if-update-available 3.0.19 (core)",
"@oclif/plugin-which 3.1.10 (core)",
"@salesforce/cli 2.42.6 (core)",
"apex 3.1.11 (core)",
"auth 3.6.11 (core)",
"data 3.3.5 (core)",
"deploy-retrieve 3.6.14 (core)",
"info 3.2.8 (core)",
"limits 3.3.7 (core)",
"marketplace 1.2.5 (core)",
"org 4.1.10 (core)",
"packaging 2.4.4 (core)",
"schema 3.3.6 (core)",
"settings 2.2.6 (core)",
"sobject 1.4.2 (core)",
"source 3.3.12 (core)",
"telemetry 3.3.7 (core)",
"templates 56.2.7 (core)",
"trust 3.6.11 (core)",
"user 3.5.8 (core)"

Confirmed that it is still happening after updating version
{
"architecture": "darwin-arm64",
"cliVersion"<EMAIL_ADDRESS>"nodeVersion": "node-v20.13.1",
"osVersion": "Darwin 23.4.0",
"rootPath": "/Users/leecunningham/.local/share/sf/client/2.44.8-c60512e",
"shell": "zsh",
"pluginVersions": [
"@oclif/plugin-autocomplete 3.0.18 (core)",
"@oclif/plugin-commands 3.3.4 (core)",
"@oclif/plugin-help 6.0.22 (core)",
"@oclif/plugin-not-found 3.1.10 (core)",
"@oclif/plugin-plugins 5.1.3 (core)",
"@oclif/plugin-search 1.0.24 (core)",
"@oclif/plugin-update 4.2.14 (core)",
"@oclif/plugin-version 2.1.2 (core)",
"@oclif/plugin-warn-if-update-available 3.0.19 (core)",
"@oclif/plugin-which 3.1.10 (core)",
"@salesforce/cli 2.44.8 (core)",
"apex 3.1.13 (core)",
"auth 3.6.17 (core)",
"data 3.4.3 (core)",
"deploy-retrieve 3.8.3 (core)",
"info 3.3.3 (core)",
"limits 3.3.9 (core)",
"marketplace 1.2.8 (core)",
"org 4.1.14 (core)",
"packaging 2.4.5 (core)",
"schema 3.3.9 (core)",
"settings 2.2.7 (core)",
"sobject 1.4.6 (core)",
"source 3.3.18 (core)",
"telemetry 3.3.11 (core)",
"templates 56.2.9 (core)",
"trust 3.7.3 (core)",
"user 3.5.11 (core)"
]
}

this is an issue with how the API works, see https://github.com/forcedotcom/cli/issues/1396#issuecomment-1033792989

Hi - thank you for the confirmation but I do not see where this is documented. As per the MDAPI documentation referenced in the linked article
"Metadata API intentionally makes it difficult to include standard fields in retrieve() calls to prevent unexpected profile changes. However, you can still retrieve and deploy profile permissions for custom and standard fields in standard objects, such as Account."
However it does not state that in order to retrieve standard lookup fields FLS the entire object definition must be retrieved. Including as a  type will result in the FLS not being retrieved onto the profile. Is this documented somewhere?

Hi @cristiand391 , I think the issue here is different. The one you point out: https://github.com/forcedotcom/cli/issues/1396#issuecomment-1033792989 is related to source-tracking and the usage of pull.
This issue is different. Here we are retrieving not pulling. We are using the following command:
sf project retrieve start --manifest package.xml
And inside package.xml we have the profile and the field. We are not using source tracking here, so ... how is this same issue? Thanks for the help here.

And inside package.xml we have the profile and the field. We are not using source tracking here, so ... how is this same issue? Thanks for the help here.

It's been a while since I played with profiles but from what I remember (and MDAPI docs/Shane's comment), your package.xml should have the object and field for it to work.
@LeeCunningham mentions it above:

If the entire customobject is retrieved then the lookup fields appear in the profile definition, but retrieving with only the fields specified results in the FLS for the field not being retrieved in the profile.

See also: https://github.com/forcedotcom/cli/discussions/1785


NEXT-0000 - Improve write exception message to include property, that has faulty content

1. Why is this change necessary?
When you do a bad API request, you do not understand why it is bad.
2. What does this change do, exactly?
Add the field name to the path mentioned in the exception.
3. Describe each step to reproduce the issue or behaviour.

Use the API and create data
Send request:

POST /api/seo-url-template
Content-Type: application/json
Accept: application/json

{
    "id": "118f8e56b0da7d81a4392f876b3b5ee6",
    "salesChannel": "foo",
    "entityName": "product",
    "routeName": "foobar",
    "template": "foo"
}

Get error response with message Expected data at /0 to be an array

4. Checklist

[x] I have rebased my changes to remove merge conflicts
[x] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hey @JoshuaBehrens thanks for your contribution! Your PR just got merged :tada: :blue_heart:

Thank you for merging :)


[scan] add possibility for copying pre-built .xctestrun file into test_output directory

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context

Resolves #18056
Description

Please read #18056
If this parameter is set to true, the .xctestrun file will be copied into test_output directory and will be called settings.xctestrun.
Testing Steps

You need an iOS example project with a scheme for UI tests.
Configure a scan lane with parameter additional_xctestrun: true.
Use this fastlane version as described here.
Note:
If you get a timeout, increase the timeout and the number of retries with the variables:
ENV["FASTLANE_XCODEBUILD_SETTINGS_TIMEOUT"]
ENV["FASTLANE_XCODE_LIST_TIMEOUT"]
ENV["FASTLANE_XCODEBUILD_SETTINGS_RETRIES"]
If you get a lint error, please remove lint from the project, because Fastlane.swift  has some lines with more than 120 characters.
I have already tested, but please review :)

@googlebot I signed it!

@googlebot I signed it!

I don't think you should push and update the Swift files, as they're auto generated when the fastlane release process is done.

I don't think you should push and update the Swift files, as they're auto generated when the fastlane release process is done.

I removed the changes in .swift files.
Thanks!

Make sure you run locally the lane generate_swift_api to see if the changes are well passed into the Swift project as well as the SPM package 🙏

Make sure you run locally the lane generate_swift_api to see if the changes are well passed into the Swift project as well as the SPM package 🙏

Done.
The changes look good!
What do you mean with "SPM package"? (Brewfile.lock.json and Gemfile.lock have no problem)
Thanks a lot for the review!

There's a step in that lane where the fastlane's SPM package gets build, so if you could run it without issues it's fine!

@googlebot I consent.

@Kukurijek Heyyyy 👋  I fixed up the unit tests and also renamed the variable from additional_xctestrun to output_xctestrun. Hopefully that’s okay 😇  Do you mind taking a look at this before it gets merged? 🙏

@joshdholtz Hi!  👋  👋 That is really kind of you 🥇
Sorry for keeping the PR open so long, due to lack of time and zero knowledge of Ruby I did not get to finish it 😅
That is great, thank you really much 👍
Yes, of course, I'll test it and get back to you!

@joshdholtz It looks good! I updated just the name of parameter in the unit test description and merged master.
Thanks again 👍


Hide attributes not working

Neither configurations is currently working to hide attributes:
item-flags:
  - HIDE_ATTRIBUTES

OR
hide_attributes: true
version: 1.14.0
server: Pufferfish version 1.21.1-21-HEAD@cfa3c61 (2024-10-22T23:08:19Z) (Implementing API version 1.21.1-R0.1-SNAPSHOT)

Change the option to item_flags. It should contain an underscore


Example for adding outgoing request span

Hi!
Could you provide an example how to add an outgoing Reqwest request to the trace chain?
I followed https://www.dynatrace.com/support/help/extend-dynatrace/opentelemetry/opentelemetry-traces/opentelemetry-ingest/opent-rust but I wasn't able to make that outgoing request appear in the trace, even the target app is fully instrumented.
fn get_parent_context(req: &Request<Body>) -> opentelemetry::Context {
    global::get_text_map_propagator(|propagator| {
        propagator.extract(&HeaderExtractor(req.headers()))
    })
}

#[instrument(skip(state))]
pub async fn proxy_handler(
    Path((service, path)): Path<(String, String)>,
    State(state): State<ProxyApiState>,
    mut req: Request<Body>,
) -> Result<Response<Body>, UserError> {

    let uri = .. // new uri was constructed here

    *req.uri_mut() = Uri::try_from(&uri)
        .wrap_err_with(|| eyre::eyre!("Proxy service constructed invalid uri: {}", &uri))?;

    let span = global::tracer("manual-client")
        .start_with_context("my-client-span", &context); //TODO Replace with the name of your span
    let cx = Context::current_with_span(span);

    //Method to inject the current context in the request
    global::get_text_map_propagator(|propagator| {
        propagator.inject_context(&cx, &mut HeaderInjector(&mut req.headers_mut().unwrap()))
    });

    cx.span()
        .set_attribute(KeyValue::new("my-client-key-1", "my-client-value-1")); //TODO Add attributes

    match state.client.request(req).await {
        Ok(v) => Ok(v),
        Err(e) => {
            Err(eyre::eyre!("Failed to make proxy request to {}.", &uri))?
        }
    }
}

Highly appreciated!
Thanks

Until, I provide a sample into this crate, you can look at the sample I linked into the README.
It's a more advanced sample put into an other repo with a Rust application that call itself with reqwest middleware https://github.com/davidB/sandbox_axum_observability/blob/development/app/src/main.rs

To retrieve your current context you look at the code of find_current_trace_id():
/// Search the current opentelemetry trace id into the Context from the current tracing'span.
/// This function can be used to report the trace id into the error message send back to user.
///
/// ```rust
/// let trace_id = axum_tracing_opentelemetry::find_current_trace_id();
/// // json!({ "error" :  "xxxxxx", "trace_id": trace_id})
///
/// ```
pub fn find_current_trace_id() -> Option<String> {
    use opentelemetry::trace::TraceContextExt;
    use tracing_opentelemetry::OpenTelemetrySpanExt;
    // let context = opentelemetry::Context::current();
    // OpenTelemetry Context propagation inside code is done via tracing crate
    let context = tracing::Span::current().context();
    let span = context.span();
    let span_context = span.span_context();
    span_context
        .is_valid()
        .then(|| span_context.trace_id().to_string())
}

Thanks for the hint @davidB . Using et cx = tracing::Span::current().context(); solved the problem. Awesome!!!

For reference. I was using hyper:
/// Proxies requests to endpoints configured in `routes`
#[instrument(skip(state))]
pub async fn proxy_handler(
    Path((service, path)): Path<(String, String)>,
    State(state): State<ProxyApiState>,
    mut req: Request<Body>,
) -> Result<Response<Body>, UserError> {

     // Skipping the code to update the uri according to a config

    // Trace trace trace
    let cx = tracing::Span::current().context();
    global::get_text_map_propagator(|propagator| {
        propagator.inject_context(&cx, &mut HeaderInjector(req.headers_mut()))
    });

    match state.client.request(req).await {
        Ok(v) => Ok(v),
        Err(e) => {
            error!("Proxy request failed. Error: {}", e);
            Err(eyre::eyre!("Failed to make proxy request to {}.", &uri))?
        }
    }

Can I use your code fragment as sample (for test or documentation) ?

Sure. Just the code from above did not make our request span appear as child of the span issued by the remote target system. To wrap async code within a span one should use tokio's instrument trait. See here for details https://docs.rs/tracing-futures/latest/tracing_futures/trait.Instrument.html.
So here is a better example:
// I think a better example would be without a proxy/without reusing the incoming request
// ..

// Create span that we will use later to wrap the outgoing request
let span = span!(
    Level::INFO,
    "Outgoing hyper request",
    otel.name = format!("{} {}", req.method(), new_url)
);

// Enrich headers of the outgoing request with current tracing context
let cx = span.context();
global::get_text_map_propagator(|propagator| {
    propagator.inject_context(&cx, &mut HeaderInjector(req.headers_mut()))
});

// Build request future
// Wrap future into the outgoing request span
// Execute request
match state.client.request(req).instrument(span).await {
    Ok(v) => Ok(v),
    Err(e) => {
        // Log full error
        error!("Proxy request failed. Error: {}", e);
        // Return error without details to frontend
        Err(eyre::eyre!("Failed to make proxy request to {}.", &uri))?
    }
}


java.lang.RuntimeException: Unable to start activity ComponentInfo{com.example.myapplication/com.example.myapplication.MainActivity}: android.view.InflateException: Binary XML file line #25: Binary XML file line #25: Error inflating class com.synnapps.carouselview.CarouselView

I solved it by adding an android.enableJetifier=true to the gradle.properties(Project Properties) directory, I hope it will help you

I was also having the same issue and enabling Jetifier resolved it. Thanks.


Non-decodable tokens

The are significant numbers of non-decodable tokens in the cl100k-base BPE. These tokens don't decode back to strings using the utf-8 encoding.
Nether do the models gpt-3.5-turbo or gpt-4 generate these tokens, nor can they be sent to the API because it only accepts utf-8 encoded text.
If the model is forced to generate these tokens using logit_bias set to 100 for the token, the completion comes empty.
{
  "choices": [
    {
      "finish_reason": "length",
      "index": 0,
      "message": {
        "content": "",
        "role": "assistant"
      }
    }
  ],
  "created":<PHONE_NUMBER>,
  "id": "chatcmpl-************************VciB",
  "model": "gpt-3.5-turbo-0613",
  "object": "chat.completion",
  "usage": {
    "completion_tokens": 0,
    "prompt_tokens": 8,
    "total_tokens": 8
  }
}

Note how the finish_reason is length even though, max_tokens for this request was set to 1000.
What's the utility of having these tokens in the BPE?
Take these tokens for example: [94, 95, 57923]
The above list is just a tiny fraction of the non-decodable tokens present in the BPE.

BPE is a "byte" pair encoder. 94 in cl100k is how we represent the single byte \xa2. Fundamentally our transformers are trained on bytes, and we want all possible sequences of bytes to be representable.
Note of course that non-decodable can be concatenated to form valid UTF-8. For example, you mentioned 57923, which will appear in enc.encode(' 馬關')
As for why your logit_bias experiment refuses to sample, I'm not sure. Seems like a bug to me.

Thanks for the detailed reply @hauntsaninja
Here's the code I'm using to sample:
import openai
openai.api_key = "Valid API Key"  # supply your API key however you choose

conversation = [{
    "role": "user",
    "content": input("Input:")
}]

bias = {57923: 100}

print("GPT: ")
reply = openai.ChatCompletion.create(model="gpt-3.5-turbo",
                                     messages=conversation,
                                     max_tokens=1000,
                                     logit_bias=bias)
print(reply)


[AOT] Fix rounded corners black edges

Enable antialiasing for render target

Summary of the Pull Request

PR Checklist

[x] Closes: #19296
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

Run AlwaysOnTop on Win11 with rounded corners enabled
Observe that corners don't have black edges anymore

Yes!🔥🔥🔥


Correct 2.0 functions doc

Addresses #https://github.com/cockroachdb/cockroach/issues/24917

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/7d7d919c4c07ab90c78779e9abf82e11dc0b2bc2/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/9aaf8bb7670d2c439d96e885f82fa061d30f8b8f/


Exploring vit features_only via new forward_intermediates() API, inspired by #2131

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

design changed a bit,
making the focal method forward_intermediates() ... it's a variant of forward_features() that focuses on collecting relevant intermediates and processing them for consisten NCHW spatial feature output.
Instead of trying to build this in to foward_features() I opted to keep it separate for now, to lower risk of regressions and prevent some complexities with torchscript / typing, etc.
FeatureGetterNet is a new feature extraction wrapper that uses forward_intermediates to return feature maps using the features_only=True attribute. This wrapper calls forward_intermediates() with the features_only=True arg to just return the intermediate features.
Models that have forward_intermediates() can also be used as below, splitting forward to extract both the intermediates and get the class predictions.
final_feat, intermediates = model.forward_intermediates(input)
output = model.forward_head(final_feat)  # pooling + classifier head

Looking forward to testing this!


[Bug] Intro Screen

Steps to Reproduce
Just enter your name with spaces like 'Abdullah Ahmed'
It will show you a validation message that name should only contain alphabetic character
Space should be accepted
So You have to just change Regex:
You can find that text field in IntroScreen [lib/presentation/screen/intro_screen.dart].
Also, change the message of validation:

name should be less than 15 characters to Name should be less than 15 characters.
name should only contain alphabetic characters to Use alphabets only, no symbols allowed.

You can find the validation message in the form_validator file [li/presentation/shared/form_validator.dart]
Function Name: nameValidator.

Hi, I would like to work on this issue.


example spa: relative modules were not found

Tried to follow the quick start section of README.md. Step npm start exitted with:
...
[1] <s> [webpack.Progress] 94% after seal
[1] <s> [webpack.Progress] 95% emitting
[1] <s> [webpack.Progress] 95% emitting HtmlWebpackPlugin
[1] <s> [webpack.Progress] 95% emitting CopyPlugin
[1] <s> [webpack.Progress] 98% after emitting
[1] <s> [webpack.Progress] 98% after emitting CopyPlugin
[1]  ERROR  Failed to compile with 2 errors18:06:26
[1]
[1] These relative modules were not found:
[1]
[1] * ../../cfg.json in ./node_modules/cache-loader/dist/cjs.js??ref--12-0!./node_modules/babel-loader/lib!./node_modules/vuetify-loader/lib/loader.js!./node_modules/cache-loader/dist/cjs.js??ref--0-0!./node_modules/vue-loader/lib??vue-loader-options!./src/pages/SignIn.vue?vue&type=script&lang=js&
[1] * ../cfg.json in ./src/firebase.js
[1] <s> [webpack.Progress] 100%
[1]

... and the top of the build log says:
$ npm start

><EMAIL_ADDRESS>start /home/dmoebius/src/vue-crud-x/example-spa
> concurrently "npm run rest" "npm run serve"

[0]
[0] ><EMAIL_ADDRESS>rest /home/dmoebius/src/vue-crud-x/example-spa
[0] > cd ../backend&&npm run-script dev
[0]
[1]
[1] ><EMAIL_ADDRESS>serve /home/dmoebius/src/vue-crud-x/example-spa
[1] > vue-cli-service serve --port 8080
[1]
[0]
[0] > functions@ dev /home/dmoebius/src/vue-crud-x/backend
[0] > set NODE_ENV=development&&nodemon index.js
[0]
[1]  INFO  Starting development server...
[0] [nodemon] 1.19.1
[0] [nodemon] to restart at any time, enter `rs`
[0] [nodemon] watching: *.*
[0] [nodemon] starting `node index.js`
[0] Environment:  undefined
[0] HTTPS:  No
[0] env development
[0] KEYV_CACHE undefined
[0] use map
[0] internal/modules/cjs/loader.js:583
[0]     throw err;
[0]     ^
[0]
[0] Error: Cannot find module '../models/User'
[0]     at Function.Module._resolveFilename (internal/modules/cjs/loader.js:581:15)
[0]     at Function.Module._load (internal/modules/cjs/loader.js:507:25)
[0]     at Module.require (internal/modules/cjs/loader.js:637:17)
[0]     at require (internal/modules/cjs/helpers.js:22:18)
[0]     at Object.<anonymous> (/home/dmoebius/src/vue-crud-x/backend/services/index.js:9:14)
[0]     at Module._compile (internal/modules/cjs/loader.js:689:30)
[0]     at Object.Module._extensions..js (internal/modules/cjs/loader.js:700:10)
[0]     at Module.load (internal/modules/cjs/loader.js:599:32)
[0]     at tryModuleLoad (internal/modules/cjs/loader.js:538:12)
[0]     at Function.Module._load (internal/modules/cjs/loader.js:530:3)
[0]     at Module.require (internal/modules/cjs/loader.js:637:17)
[0]     at require (internal/modules/cjs/helpers.js:22:18)
[0]     at Object.<anonymous> (/home/dmoebius/src/vue-crud-x/backend/middleware/auth.js:1:100)
[0]     at Module._compile (internal/modules/cjs/loader.js:689:30)
[0]     at Object.Module._extensions..js (internal/modules/cjs/loader.js:700:10)
[0]     at Module.load (internal/modules/cjs/loader.js:599:32)
[0] [nodemon] app crashed - waiting for file changes before starting...
[1] <s> [webpack.Progress] 0% compiling
[1] <s> [webpack.Progress] 10% building 0/0 modules 0 active
...

I see. cp example-app/cfg.sample.json example-app/cfg.json helped a bit, but I need to fill in projectId and apiKey, or else I will get "projectId must be a string in FirebaseApp.options" or "Your API key is invalid, please check you have copied it correctly." Seems that example-spa cannot run without firebase db, or can it?

Hi,
models/User cannot be found because it is models/user
I am developing on windows so this casing may have caused a problem...
You can rename the file from user.js to User.js to get it working...
I will fix it in next release, sometime early next month...

Ok, renaming user.js to User.js helped removing the first error. But the demo cannot run without a proper cfg.json with valid projectId and apiKey, can it?

Hi...
You are right, let me work on a fix... If you need to get it working quick...
Search for all @/firebase, comment it out...
And comment out anything using the firebase stuff...
You may need to include @/mongo also...

Will add in environment variables to enable or disable mongo, firebase, target to finish by end of the week.

I pushed to develop branch... I have made fixes...
You can use the cfg.json file in there (with empty keys), I will make further improvement on this.

https://github.com/ais-one/vue-crud-x/pull/73


Clean up v1alpha1 usages from tests

How to categorize this PR?

/kind cleanup
What this PR does / why we need it:
This PR is a preparation for the v1alpha1 API removal.
Which issue(s) this PR fixes:
Part of #3
Special notes for your reviewer:
N/A
Release note:

NONE

/approve


vapor 1.3 support

The package will need to be updated as well. Supporting mustache is not a priority for the team since we have Leaf now.


[nanosvg] update port and use upstream cmake instead of vendored cmake

Will update as soon as possible.

Fixed by https://github.com/microsoft/vcpkg/pull/29998.


How to change the person detection model to any coustom object detection model

I need to change the person detection example tyo my own coustom object detection. how to train the model and get a tflite model that should move forward in the code generation.

hello, i have the same issue too. Have you found a solution to it?


build reference error

Hi,
I followed your tutorial to download and build references for other species.
My command is this:
python build_reference.py --species Other --other_species_TE TE_reference.csv --other_species_GTF Gene_reference.gtf
However, it shows the following error:

Traceback (most recent call last):
File "/Users/xinghua/Downloads/build_reference.py", line 154, in
main()
File "/Users/xinghua/Downloads/build_reference.py", line 58, in main
genes = genes[genes['Feature'] == 'gene']
~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/xinghua/miniconda3/lib/python3.12/site-packages/pyranges/pyranges_main.py", line 436, in getitem
return _getitem(self, val)
^^^^^^^^^^^^^^^^^^^
File "/Users/xinghua/miniconda3/lib/python3.12/site-packages/pyranges/methods/getitem.py", line 33, in _getitem
raise Exception("Not a valid subsetter: {}".format(str(val)))
Exception: Not a valid subsetter: False

Can you help figure this out? Or could you share the format of the expected references?
Thanks!

Hi @lxinghua ,
It looks like the format of GTF you are using is not Ensembl format, which has 'gene' keyword in the gtf file. If you are using NCBI refseq format GTF, the keyword should be 'transcript'. As you can in the attached screenshot, NCBI has several format of gene annotation file (The track tab). If the species you want to process has the Ensembl format, you can download it and re-process. But if you can only find the NCBI Refseq or other format, please share the format of track with us.
I will also update the build_reference.py file to support NCBI Refseq format today.
Thanks,
Yumin

Hi @lxinghua ,
I also quickly update the build_reference.py file. If you are using the ensemble or GENOCDE GTF format, the updated code may solve your issue.
Thanks,
Yumin

Hi @lxinghua ,
Thanks for your feedback. We realized the previous tutorial might be misleading and code is not user friendly enough. So we have enhanced the step by step tutorial of reference genome building tutorial and the script.
To avoid the confusion, can you check our latest reference genome building tutorial and update the code to the latest version (v0.1.4)?
Thanks,
Yumin


Customize the section title generated by apxproof

Hi @PierreSenellart,
Thanks for the great package!
I would like to know if it is possible to create something like the screenshot below, which has more fine-grained control over the titles auto-generated by apxproof package. You will notice that

The proofs in the appendix have a shared "upper-level" section title `Deferred Proofs``.
The title for each subsection in the appendix actually corresponds to the subsection title in the main text. For example, Proofs for  Section 1.1: Analysis for Case I in the appendix versus Analysis for Case I in the main text.

However, using apxproof package, I can only now generate something like the screenshot below. Specifically,

I don't know how to create the shared main title `Deferred Proofs``.
I don't have fine-grained control over the generated title.

I tried to play around with \appendixsectionformat option; however, from the code it seems that the package only uses the title Information up to the section level instead of subsection or even subsubsection levels.
Thanks,
YD

The latex code (compiled on Overleaf for the desired output, i.e., the first screenshot) is attached below.
\documentclass{article}
\usepackage[appendix=append,forwardlinking=no]{apxproof}
\newtheoremrep{theorem}{Theorem}
\date{}
\title{apxproof: Flexible Title}

\begin{document}

\maketitle

\section{Analysis}
\subsection{Analysis for Case I}
\begin{theorem}\label{thm:1}
We assert that
$$
1+1 = 2
$$
\end{theorem}

\subsection{Analysis for Case II}
\begin{theorem}\label{thm:2}
We assert that
$$
2+2 = 4
$$
\end{theorem}

\appendix
\newpage
\section{Deferred Proofs}
\subsection{Proofs for  Section 1.1: Analysis for Case I}
\paragraph{Theorem~\ref{thm:1}
We assert that
$$
1+1 = 2
$$
}
\begin{proof}
This is trivial(1).
\end{proof}
\subsection{Proofs for  Section 1.2: Analysis for Case II}
\paragraph{Theorem~\ref{thm:2}
We assert that
$$
2+2 = 4
$$
}
\begin{proof}
This is trivial(2).
\end{proof}
\end{document}

The latex code for generating the second screenshot is also attached.
\documentclass{article}
\usepackage[appendix=append,forwardlinking=no]{apxproof}
\newtheoremrep{theorem}{Theorem}
\date{}
\title{apxproof test}

\begin{document}

\maketitle

\section{Analysis}
\subsection{Analysis for Case I}
\begin{theoremrep}
$$
1+1 = 2
$$
\end{theoremrep}
\begin{proof}
This is trivial(1).
\end{proof}

\subsection{Analysis for Case II}
\begin{theoremrep}
$$
2+2 = 4
$$
\end{theoremrep}
\begin{proof}
This is trivial(2).
\end{proof}

\end{document}

Hi,
Thanks for your comment and interest. There is no automatic way to do this, though you can do it semi-manually easily enough:
\documentclass{article}
\usepackage[appendix=append]{apxproof}
\newtheoremrep{theorem}{Theorem}
\date{}
\title{apxproof test}

\begin{document}

\begin{toappendix}
  \section{Deferred Proofs}
\end{toappendix}

\maketitle

\section{Analysis}
\nosectionappendix
\subsection{Analysis for Case I}
\label{case1}
\begin{toappendix}
  \subsection{Proofs for Section~\ref{case1}: Analysis for Case I}
\end{toappendix}

\begin{theoremrep}
$$
1+1 = 2
$$
\end{theoremrep}
\begin{proof}
This is trivial(1).
\end{proof}

\subsection{Analysis for Case II}
\label{case2}
\begin{toappendix}
  \subsection{Proofs for Section~\ref{case1}: Analysis for Case II}
\end{toappendix}

\begin{theoremrep}
$$
2+2 = 4
$$
\end{theoremrep}
\begin{proof}
This is trivial(2).
\end{proof}

\end{document}

Is this what you want? I could try improve the customization options so that it can be done automatically, but I am not sure what a good interface would be in the general case.

Thanks, this is exactly what I want. I'm not sure how to automate this part.
\begin{toappendix}
  \section{Deferred Proofs}
\end{toappendix}

But I think it might be possible to automate the following by providing something like to \tocdepth{(a number)}.
\begin{toappendix}
  \subsection{Proofs for Section~\ref{case1}: Analysis for Case I}
\end{toappendix}

\begin{toappendix}
  \subsection{Proofs for Section~\ref{case2}: Analysis for Case II}
\end{toappendix}

Specifically, \tocdepth{1} has the default behavior of apxproof, where the section title in the appendix inherits the corresponding section title in the main text and \tocdepth{2} makes it inherits the corresponding subsection title.
Again, thanks for the timely response.

Thanks for the suggestion, I'll see if I have the time to implement something like this.


no effect when hovering over left edge of zoomed in chrome

i guess some invisible object is blocking the left side, but only in chrome. no messages get through.
this is just a demo and i don't really care, but tracking the issue anyway.

Also affects all mobile browsers. I guess I care,  although it doesn't affect the main purpose of this library.


Ability to sort and filter routers when adding them

Ability to sort routers alphabetically when adding them to the routers tab makes easy to check if all your routers are added. If by any reason, the first router is not added (e.g. vmx501) when adding all the rest (e.g. vmx502, vmx503, ..., vmx507), vmx501 will be shown the last one.

Ability to filter routers from the list by tags. So this will include a tagging ability when adding routers. For example, if adding routers and saying they are ACXs, or MXs, or they belong to East or West, you can easily track if all your expectations are eventually there (e.g. 4 devices from East and 5 from West).

will be supported with version 1.1.0

will be supported with version 1.1.0


Apply multiple compaction job completions in a single state store update

Background
Split from:

https://github.com/gchq/sleeper/issues/2101

Description
We'd like to add an operation to the state store to be able to apply multiple compaction job completions in the state store all at once.
We'll follow this up in other issues with code to batch up sending these completions to the state store.
Analysis
This can just be adding an operation to the state store.
We could split out issues for the different state stores if this is too big.

On hold because we might not do this if the transaction log state store solves the problem of contention applying many compactions.


The refactor 22 days ago fixes a bug in current npm version.

This is the fixed code in your unreleased refactor.
https://github.com/rtfeldman/elm-webpack-loader/blob/master/index.js#L53
Before refactor this is the fault - 'exitCode' does not exist as a variable.
https://github.com/rtfeldman/elm-webpack-loader/blob/ec48ffd2e150f78094fab916595289fbb95c32d9/index.js#L53
If I fix this by hand locally webpack on windows 7 64bit, will retry compiling on errors, with watch or the webpack-dev-server. Without it any error will stop -watch or the dev server and they both will not try to compile again.

@rluiten Thanks! Released 1.1.2, which includes the fix.
Please feel free to reopen if this is still happening in 1.1.2.


Please add support for ".pls" extension

Can you please add support for .pls file format (which is the default extension when Oracle SQLDeveloper tool exports file. I've done locally by myself by adding it to package.json
			"extensions": [
				".sql",
				".ddl",
				".dml",
				".pkb",
				".pkh",
				".pks",
				".pck",
				".pls"
				],

and to
<array>
    <string>sql</string>
    <string>ddl</string>
    <string>dml</string>
    <string>pks</string>
    <string>pkb</string>
    <string>pck</string>
<string>pls</string>
</array>

But it would be much better to have it supported in the standard.
Thanks a lot!

Fix in version 1.1.0


Cannot Debug any .Net Core application

VSCode Version: 1.7.2
OS Version: Windows 8.1 Enterprise
.Net Core 1.1 Version: 1.0.0-preview2-003177
OmniSharp C# extension Version: 1.5.3 (no other extensions installed)

Steps to Reproduce:

dotnet new
dotnet restore
open folder in VS code
click debug
click cog
select ".Net Core"
click Start
click Configure Task Runner
select ".Net Core"
click Start

messageService.ts:128` spawn UNKNOWN: Error: spawn UNKNOWN
at Object.f [as create] (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:29:20675)
at file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:121:27301
at Object.v [as _notify] (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:31:10797)
at Object.enter (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:31:14302)
at _run (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:31:15623)
at _chainedError (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:31:14982)
at n (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:31:12805)
at then (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:31:16432)
at Object.enter (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:31:12874)
at _run (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:31:15623)
at _setCompleteValue (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:31:15317)

Please file this against https://github.com/OmniSharp/omnisharp-vscode


Integration broken FW <IP_ADDRESS>

Fist of all many thanks for this beautiful integraiton.
Sadly I updated my Firmware Version to <IP_ADDRESS> on my Delta MAX and now the Integration with Home-Assistant is broken. I tried to remove and re-add my Delta MAX in Home-Assistant with no success.
Do you have some advises or tests I can to to provide further logs?
Hopefully EcoFlow hasn't disabled there local API. Anyone else facing similar problems?
Cheers e-bits

I have Delta MAX with FW <IP_ADDRESS> and everything is working to me.
Sometimes is reset needed - try to disconnect everything (input+output) from your unit. turn off you Delta MAX (10s long press of main button) and wait 30m before power on again.

I have Delta MAX with FW <IP_ADDRESS> and everything is working to me. Sometimes is reset needed - try to disconnect everything (input+output) from your unit. turn off you Delta MAX (10s long press of main button) and wait 30m before power on again.

Thanks for feedback. I tried your workflow but without any success.
Delta MAX is connected to WiFi and responding to ping on a static IP served by DHCP. App is working as well but the integration is stuck somehow.
Anything else I can try?

Can you try to check if port 8055 is opened on DELTA Max? E.g. by running following command (xx.xx.xx.xx is IP address of your DELTA Max) in some linux terminal
nmap -p 8055 xx.xx.xx.xx

Sure, here we go.
Starting Nmap 7.80 ( https://nmap.org ) at 2022-09-21 18:14 CEST
Nmap scan report for <IP_ADDRESS>
Host is up (0.0064s latency).

PORT     STATE  SERVICE
8055/tcp closed senomix04

Nmap done: 1 IP address (1 host up) scanned in 0.17 seconds

I did a fresh setup with linking my Delta MAX to the App again. Therefore I unlinked my existing Delta MAX and did the Setup again. During the Setup I did a Port-Scan and realized shortly after adding my WiFi Credentials the Port 8055 is getting closed.

related #58

I just opened a request at EcoFlow Zendesk and described my problem. I asked if the Port 8055 will be open again on newer Firmware Version or if it would be possible to do a Firmware Downgrade.
I will keep you posted.
Cheers

I opened request also (two days ago) but it seems EcoFlow is focusing only on use of their official API (https://api.ecoflow.com/iot-service/open/api/device/queryDeviceQuota/...)

Hello, I have DELTA Pro, DELTA Max and River RIVER Pro. All units are on latest FW, But only DELTA Pro has all TCP ports closed.
DELTA Max and RIVER Pro still works correctly with this integration. They have also port 80 opened.
So curl -X POST -d '{"action":"getTCPServer"}' http://ip-address/ return following:
DELTA Max: {"action":"getTCPServer","data":{"port":8055},"Msg":"success"}
RIVER Pro: {"action":"getTCPServer","data":{"port":8055},"Msg":"success"}
DELTA Pro: curl: (7) Failed to connect to xx.xx.xx.xx port 80: Connection refused

That is strange as the unit must have a port open to communicate with the external server. Have you made a deep scan with nmap?
nmap -Pn -sS -sV --version-all --reason -v -A -O --osscan-guess -p0-65535 IP Address

I will try full scan. So far I tried only to 0-30000

I try a full scan -p0-65535 on Delta2, all are closed.

Here is my result ...
DELTA Max:
Not shown: 65533 closed ports
Reason: 65533 resets
PORT     STATE SERVICE    REASON          VERSION
80/tcp   open  http?      syn-ack ttl 255
6093/tcp open  unknown    syn-ack ttl 255
8055/tcp open  senomix04? syn-ack ttl 255
RIVER Pro:
Not shown: 65533 closed ports
Reason: 65533 resets
PORT     STATE SERVICE    REASON          VERSION
80/tcp   open  http?      syn-ack ttl 255
6093/tcp open  unknown    syn-ack ttl 255
8055/tcp open  senomix04? syn-ack ttl 255
DELTA Pro:
All 65536 scanned ports on xx.xx.xx.xx are closed because of 65536 resets

That is interesting. So what we need is a capture between the device and the server from Ecoflow.
If you have a Fritzbox , it is quite easy with the internal http://fritz.box/html/capture.html to see the ports that they are using.
But do not place a capture here, as it will contain a lot of private information. You should open the capture with wireshark and look at the traffic between your device and the ecoflow server by filtering the stream with ip.addr == IP Address of your device
Without a Fritzbox it is depending on your router if a capture feature is build in or not

I think that Ecoflow unit is connecting to Ecoflow cloud and act as a sender of data. Capture of this communication will not help because Ecoflow unit has probably hardcoded connection to Ecoflow cloud in FW (probably by some URL). If no port is open then it cannot serve as local server which provides some data.
In theory can be created local 'Ecoflow cloud' server, which could be used instead of official - e.g. if local DNS server will return your local IP of this local 'Ecoflow cloud' intead of official IP of Ecoflow cloud. But I am skeptical with it because communication between Ecoflow unit and Ecoflow cloud will be probably protected with some certificates which cannot be easily replaced.
In case of taking data from Ecoflow cloud is better to use official API or MQTT as it was described here https://github.com/vwt12eh8/hassio-ecoflow/discussions/58
I like this integration because it didn't require connection to Ecoflow cloud to get data. For me it's strange that such units which at least partially designed for emergency use will require internet connection over the world or unit must be started in IoT mode which is not secure. If somebody has more units then it's real nightmare to have all units in IoT mode with limited WiFi range and which isn't secured...

Hi I received an answer to my opened support ticket.

From the perspective of device security and data privacy, we will not open this port to the public, please you understand.

What should I say, I'm really disappointed.

https://github.com/v1ckxy/ecoflow-withoutflow/issues/1#issuecomment-1129501342

That is the mobile app, but does the device now really connect using MQTT to the ecoflow server? Would be great if so.

Yes it probably connects via MQTT, using the log from the android device I found out the name and password to ecoflow MQTT server. And using the MQTTX app I was able to read the information through the MQTT from the ecoflow server. The information which I got is shown here But I am unable to get the data into home assistant(I am new to MQTT).


Ruby scss-lint gem required to lint scss

Find a node-only alternative or update documentation to include instructions for installing gem

Closing since this is not an issue with 2.0 and later


Bug in OMT for dimensions > 10

This keeps partitioning the points in high dimensions (so it builds a tree with less height):
    let mut entries = Vec::new();
    while !points.is_empty() {
        let remaining = points.len().saturating_sub(partition_size);
        points.select_nth_unstable_by(remaining, |a, b| {
            a.coord_at(split_dim)
                .partial_cmp(&b.coord_at(split_dim))
                .unwrap()
        });
        let slice = points.split_off(remaining);
        entries.extend(partition_points(slice, split_dim + 1, num_slices, params));
    }

With this current implementation of OMT, the bulk-loading of points forms a tree with height = 3 only for dimensions > 10, so its query performance is close to the linear scan (https://github.com/aicers/srtree/tree/main/bench#uniform-dataset).


[GLUTEN-5039][VL] Add support for AppleClang compiler

What changes were proposed in this pull request?
Fixes: #5039
How was this patch tested?
Manual tested on Apple M1.

does velox support clang compiler?

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_5053_time.csv
log/native_master_03_21_2024_9d859d8e4_time.csv
difference
percentage

q1
35.67
36.78
1.106
103.10%

q2
23.43
23.82
0.387
101.65%

q3
36.81
36.97
0.166
100.45%

q4
38.25
38.29
0.033
100.09%

q5
69.31
68.82
-0.487
99.30%

q6
5.75
7.79
2.043
135.52%

q7
81.76
82.16
0.404
100.49%

q8
86.35
85.93
-0.423
99.51%

q9
120.75
120.31
-0.447
99.63%

q10
45.91
43.15
-2.753
94.00%

q11
20.40
20.44
0.042
100.21%

q12
26.78
26.75
-0.034
99.87%

q13
47.36
48.78
1.421
103.00%

q14
18.65
21.16
2.513
113.48%

q15
30.11
28.73
-1.385
95.40%

q16
15.70
14.42
-1.285
91.81%

q17
102.10
99.19
-2.915
97.14%

q18
140.78
139.46
-1.317
99.06%

q19
13.60
15.07
1.466
110.78%

q20
27.41
27.86
0.446
101.63%

q21
225.89
226.99
1.096
100.49%

q22
15.38
14.13
-1.248
91.89%

total
1228.16
1226.99
-1.170
99.90%


Quadratic and Sawtooth waves

Hello! Tell me, please, is it possible to create quadratic and sawtooth waves? I found only a sine wave in the documentation

Not part of the official library, but if you'd like an example to work from (or copy verbatim 😂) I have a sawtooth wave for a toy project I'm working on.
https://github.com/smangelsdorf/rtttlsynth/blob/fcd7be5908599553b528eaf401731bcf42c02914/src/synth/oscillator.rs
If there's any interest from the project maintainers I'd be happy to tidy this up and PR it as a library feature.

I was curious, so I looked, but I couldn't find a standard definition for a quadratic wave. What do you have in mind for that?
I imagine it being something like this, but I could be way off 😅

I was curious, so I looked, but I couldn't find a standard definition for a quadratic wave. What do you have in mind for that?
I imagine it being something like this, but I could be way off 😅

I mostly asked the AI 😅

Ah, that's interesting. I asked GPT too, but I confused it by asking the wrong question. I was expecting another fundamental wave.
Adjusting the pitch over time based on a quadratic function is trickier. Both the inbuilt SineWave and my SawWave implementation have a fixed frequency, so won't work properly for that. As far as I know, the oscillators would need to support being modulated by some function, or else accept a pitch envelope describing the pitch over time.
As a random internet guy, I can't say whether that's something Rodio wants to support or whether it should be in a separate library, but it sounds like it would be fun to build. 😅

Hi!
I know a little about Quadratic Waves.
They are related Bézier curves.

Take two non-Parallel lines AB and BC (i.e they share a point B).
Trace a point on starting at A to B (called V). At the same time, a second point from B to C (called W).
Lerp between the line created by WV.
The midpoint of WV traces a quadratic curve.

I'm not sure if this is what @anti7447 meant.
It does come up in computer music, usually used as envelopes.
Rodio is open source, so If people want to throw it in, awesome.
I would consider quadratic enveloping outside the scope of Rodio.
I really liked this animation.

Hello! Tell me, please, is it possible to create quadratic and sawtooth waves? I found only a sine wave in the documentation

Did you figure it out? If not let me know! And if you did ill close this issue.

Rodio is open source, so If people want to throw it in, awesome.
I would consider quadratic enveloping outside the scope of Rodio.

It would be a great fit for a third party crate extending Rodio by providing more Source's. If something like that grows and becomes popular it could even be taken over by the RustAudio group. Feel free to begin something like that and add all the waves you need :) It think we would happily link to it from the Readme.

closing, rodio now has signal generator. PR's extending that with more waveforms (including sawtooth) are appreciated :)


Fix CodeQL initial scan findings

Background
The LGTM code check is going away and being replaced by CodeQL. Our baseline scan has some issues that need to be addressed before the check will start passing on PRs.
Action requested
Fix the issues identified by CodeQL scanning. See here for a baseline scan.
Acceptance Criteria
CodeQL passes when run against a clean main.
It seems this will require changes to the following files:

PatientExperienceLoggingInterceptor.java
ReminderServiceTest.java
TextMessageeStatusService.java
DataHubClientConfiguration.java
TestEventExportIntegrationTest.java
TextMessageStatusServiceTest.java

Each of these alerts has more details that can be found under the Security section of Github, ex: https://github.com/CDCgov/prime-simplereport/security/code-scanning/59

Branch with fixes to the above issues: https://github.com/CDCgov/prime-simplereport/tree/emma/4865-codeql
Will convert to a PR once I'm back in office.


Screen reader optimized in top left corner

OS X, insiders

Turn on voice over
Start vscode insiders -> notice screen reader optimized popup in the top left corner (instead of the bottom right)

I feel like I already filed this issue in the past

Fixed by using a notification instead of custom UI in https://github.com/Microsoft/vscode/commit/1433e180d1ca25fa2a33b900a17b633123120631


didn't work with latest version of web3(1.0.0-beta.38 and later)

do
const thorify = require("thorify").thorify;
const Web3 = require("web3");

const web3 = thorify(new Web3(), "http://localhost:8669");

"dependencies": {
    "thorify": "^1.0.4",
    "web3": "^1.0.0-beta.38"
  }

Keep throwing Error: Missing parameter: provider from the path of ~/node_modules/web3-core/dist/web3-core.cjs.js:163
From web3 version 1.0.0 docs, when we init the Web3, the parameter cannot be empty.

@libotony Maybe it's time to update web3. They have bee doing a lot of fixes https://github.com/ethereum/web3.js/ For my code i ned to have v 58 but thorify requires . 37.

any update on upgrading? there is high severity issue with a web3 dependency (tar)

Web3 has decided to publish version 1.0 based on the codebase of 1.0.0-beta.37 for the compatibility with most packages, this is described in this release note. Thorify relies on the codebase of beta.37 so I have published version 1.3.0 which works with<EMAIL_ADDRESS>


Update CI node versions

Drop node 6 and 7, and add node 11 and 12
Closes #409

@knownasilya do you think that this should go to v5.0?
In that case, we should merge this but not release then I can do all the proposed changes for v5.0 and then release.
Wdyt?

Coverage remained the same at 100.0% when pulling a074ec977788e6757492bf603ff431488b26d140 on juanjoDiaz:update_ci_node_versions into b57715b300eb17e50fdc332fb969860f96d31321 on zemirco:master.

Yeah


Trollop module method tests

Add tests around the Trollop module:

options
die
educate
with_standard_exceptions
(version)

make assertions more similar to minitest's suggestions

Coverage increased (+2.5%) to 94.67% when pulling c397efd57b00410b55aff2dd6943f8cafc032f03 on kbrock:splitup_tests into d78a8882fc944589cc70a6f0f83b5821777d151c on ManageIQ:master.

Coverage increased (+0.48%) to 92.65% when pulling 7b06be577970898edcb11312d28e7b94a7ad7d30 on kbrock:splitup_tests into d78a8882fc944589cc70a6f0f83b5821777d151c on ManageIQ:master.

Coverage increased (+0.48%) to 92.65% when pulling 7b06be577970898edcb11312d28e7b94a7ad7d30 on kbrock:splitup_tests into d78a8882fc944589cc70a6f0f83b5821777d151c on ManageIQ:master.

Coverage increased (+0.74%) to 92.91% when pulling dad24e91f10097798e454ace60909d302a938525 on kbrock:splitup_tests into d78a8882fc944589cc70a6f0f83b5821777d151c on ManageIQ:master.


/node_modules/weex-ui/packages/wxc-button/index.vue    Module build failed: SyntaxError: Unexpected token

1、question：
/node_modules/weex-ui/packages/wxc-button/index.vue
Module build failed: SyntaxError: Unexpected token (43:8)
41 |
42 |       const mrBtnStyle = {
> 43 |         ...STYLE_MAP[type],
|         ^
44 |         ...btnStyle,
45 |         ...BUTTON_STYLE_MAP[size]
46 |       };
2、development environment
centos 6.8 ,
node(6.9.2 ~ 10.+)
"weex-ui": "^0.6.4-beta1"
"webpack": "^3.12.0"
"babel-plugin-component": "^1.1.1"
"babel-preset-stage-0": "^6.24.1"
3、describe
use this command
npm i babel-plugin-component babel-preset-stage-0 -D
and modify .babelrc
but it's still a mistake.
seek the answer of the God.

If babel-loader in webpack.config.js has a exclude for node_modules, please turn on for week-ui as exclude: /node_modules(?!(/|\).(weex).)/.

@tw93
Have been used exclude: /node_modules(?!(/|).(weex).)/ , but no solution to the problem.


InvalidStateError: Failed to execute 'transaction' on 'IDBDatabase': The database connection is closing with ios safari

I am getting error on log when this get executed:-
import React, { useEffect } from "react";
import db from "../config/db";

useEffect(() => {
    db.reads
      .get(props.match.params.id)
      .then((data) => {})
      .catch((e) => {
        console.log(e);
      });
  }, []);

import Dexie from "dexie";
const db = new Dexie("bookDB");
db.version(2).stores({
  reads: "hash,token,sa,ss",
  books: "hash,token",
});

export default db;

I am getting many errors :-

InvalidStateError: Failed to execute 'transaction' on 'IDBDatabase': The database connection is closing.

UserAgent:-

Mozilla/5.0 (iPhone; CPU iPhone OS 13_7 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/13.1.2 Mobile/15E148 Safari/604.1

I could not find steps that produce the following error, some iPhone's i've tested does not return any error.

This happens when database is closing down when the component is mounted. The reason could be if the application at another place calls db.close(), or if the database is forcibly deleted in devtools, or if there is a newer version of the db loaded in another tab or window, so the db has to close down to resume an upgrade.
It could also be a iOS Safari related issue. Do you only see the issue on Safari or have you just tested it there?

This happens when database is closing down when the component is mounting. The reason could be if the application at another place calls db.close(), or if the database is forcibly deleted in devtools, or if there is a newer version of the db loaded in another tab or window, so the db has to close down to resume an upgrade.
It could also be a iOS Safari related issue. Do you only see the issue on Safari or have you just tested it there?

All the other components files using the same, import db then use .get or .put method, this issue, regarding the version upgade, currently we have version 2 since 4 months ago, and users who got this error are first time they visit our website.
This error only occurred with ios users.

S
![Uploading 20201007_114721.jpg…]
Screenshot showing the user-agent that got the error

This happens when database is closing down when the component is mounting. The reason could be if the application at another place calls db.close(), or if the database is forcibly deleted in devtools, or if there is a newer version of the db loaded in another tab or window, so the db has to close down to resume an upgrade.
It could also be a iOS Safari related issue. Do you only see the issue on Safari or have you just tested it there?

So how should we catch and fix this issue?

So how should we catch and fix this issue?

Too little info about the cause of this issue to tell. Try make a repro.

Is the reason that your own application is closing down the db? Fix it in your app.
Is the reason that another tab/window is loading a newer version of the app, making the old close down? Listen to db.on('versionchange').
Is the reason due to a bug in Safari? - File an issue in webkit bugzilla.

I never close the dexie db.
I can't reproduce but I know it happened for someone using Firefox 88 on windows. Also I know that once I changed from storing blobs to storing ArrayBuffer that this issue happens a lot more frequently.
I have never changed versions of my db - so that should never be called right?

right. Pay attention to errors occurring before this error happens. Are you also getting quotaexceeded errors? I don't see the reason why the browser closes the db for anything else than an unexpected internal error, but i could be wrong. Would be interesting to know.

I wasn’t getting quotaexceeded until I did the ArrayBuffer but still I’m get those error far less frequently.
The one Firefox user didn’t have any issues when saving blob without doing the ArrayBuffer. I wouldn’t know there was an issue until I saw only 15 bytes retrieved from the db.

So how should we catch and fix this issue?

Too little info about the cause of this issue to tell. Try make a repro.

Is the reason that your own application is closing down the db? Fix it in your app.
Is the reason that another tab/window is loading a newer version of the app, making the old close down? Listen to db.on('versionchange') to customize how your app should act when this happens.
Is the reason due to a bug in Safari? - File an issue in webkit bugzilla.

I begin to think that it's a bug in Safari, because i noticed not only indexedDB data get cleared from those users, also the localStorage get cleared.

Yep, it is a safari bug - https://bugs.webkit.org/show_bug.cgi?id=197050 . It may happen when you switch the site between foreground and background.

Thanks @quolpr! Yes, that would be great if we could workaround this. I posted an idea on how to fix it Based on mikekreeki's workaround but without having to test IDB state for every request.

This is ocurring on latest chrome randomly as well, I noticed that it happens more often when browser was put in the background for a while

https://github.com/dexie/Dexie.js/pull/1398
Might fix that?

#1398
Might fix that?

Yes, #1398 works around the exact same problem - catches exception thrown when executing db.transaction and reopens the connection. It's released in version 3.2.1.

Thank you, I'll test test that!

It seems indeed to workaround the issue pretty well! Thank you!


CIRC event had an unrecognized remote_reason

Please handle

CHANNEL_CLOSED
HS_CLIENT_HSDIR
HS_SERVICE_HSDIR
HS_VANGUARDS

Caught several of these:
INFO:stem:CIRC event had an unrecognized remote_reason (CHANNEL_CLOSED). Maybe a new addition to the control protocol? Full Event: 'CIRC 347 FAILED $relay,$relay,NEED_CAPACITY PURPOSE=TESTING TIME_CREATED=2018-08-25T02:48:19.546319 REASON=DESTROYED REMOTE_REASON=CHANNEL_CLOSED'
INFO:stem:CIRC event had an unrecognized purpose (HS_CLIENT_HSDIR). Maybe a new addition to the control protocol? Full Event: 'CIRC 472 LAUNCHED BUILD_FLAGS=IS_INTERNAL,NEED_CAPACITY PURPOSE=HS_CLIENT_HSDIR HS_STATE=HSCI_CONNECTING TIME_CREATED=2018-08-25T02:50:03.529166'
INFO:stem:CIRC_MINOR event had an unrecognized old_purpose (HS_SERVICE_HSDIR). Maybe a new addition to the control protocol? Full Event: 'CIRC_MINOR 150 PURPOSE_CHANGED $relay,$relay,$relay,NEED_CAPACITY PURPOSE=MEASURE_TIMEOUT TIME_CREATED=2018-08-25T03:10:53.505273 OLD_PURPOSE=HS_SERVICE_HSDIR OLD_HS_STATE=HSSI_CONNECTING'
INFO:stem:CIRC_MINOR event had an unrecognized old_purpose (HS_VANGUARDS). Maybe a new addition to the control protocol? Full Event: 'CIRC_MINOR 175 PURPOSE_CHANGED $relay,$relay,$relay,$relay,NEED_CAPACITY,NEED_UPTIME PURPOSE=HS_SERVICE_HSDIR HS_STATE=HSSI_CONNECTING TIME_CREATED=2018-08-25T03:10:58.927978 OLD_PURPOSE=HS_VANGUARDS'

These log messages are from stem. They are harmless, since we do not use keywords for these values, we use the direct string.
Out of curiosity, what is your stem version? I just pushed 7d79823f4e4cc4e157e75954492f80b2c9172212 to aid in debugging any issues that might arise from this.
Also, with your stem version, I can pin that in the unit tests and look for any issues (currently tox pins stem at 1.5.4).
You can also try one of the last two testing methods outside of Tox to ensure that the system-wide stem passes tests against either the git checkout or a system-wide vanguards install:
https://github.com/mikeperry-tor/vanguards/blob/master/README_TESTS.md

I tested Stem 1.6.0 against the unit tests and there are no ill effects due to these info messages.
If you notice any, please open specific bugs.


[LLM Runtime] Unify KV_cache and Support Batch-dim Process in Beam Search

Type of Change
feature or bug fix or documentation or others: Feature
API changed or not: YES (model_eval)
add a new struct for batch inference:
struct model_input {
  // embd or next token
  model_token* tokens = nullptr;
  // tokens length
  uint32_t n_tokens = 0;
  // prompt length
  uint32_t n_prompt_tokens = 0;
  // kv cache n_past
  uint32_t n_past = 0;
  // text tokens length (prompt + all next tokens)
  // the number of tokens evaluated so far (including evicted tokens if there is any)
  uint32_t n_total = 0;
  // request id
  int request_idx = -1;
  // beam id in beam search
  int beam_idx = 0;
  // padding related, attention mask
  // (0: left, 1: right)
  // only support padding left in decoder only model
  int padding_side = 0;
  // padding length
  uint32_t n_padding = 0;
};

MODEL_API int model_eval(struct model_context* ctx, const model_input* inputs, const int n_input, int n_threads););

cc @Zhenzhong1 , we can co-work and discuss about how to support static batching generation since you are doing ChatGLM multi-batch inference (pad, mask MHA, return, etc.)
Description
detail description
JIRA ticket: 829
TODO:

[x] kv_cache_seq_cpy and kc_cache_seq_concat prototype and refact beam search
[x] beam search for static batching prompt
[x] deal with padding
[x] pybind and python ut example
[x] test

Expected Behavior & Potential Risk
the expected behavior that triggered by this PR
How has this PR been tested?
how to reproduce the test (including hardware information)
transformers batch_decode example (padding_side='left' for right next_token generation)
from transformers import pipeline, set_seed, AutoModelForCausalLM, AutoTokenizer
model_dir = "/home/zhentao/gpt-j-6b"
tokenizer = AutoTokenizer.from_pretrained(model_dir, padding_side="left")
tokenizer.pad_token = tokenizer.eos_token # to avoid an error
model = AutoModelForCausalLM.from_pretrained(model_dir)
model.eval()

prompts = ["she opened the door and see",
           "tell me 10 things about jazz music",
           "What is the meaning of life?",
           "To be, or not to be, that is the question: Whether 'tis nobler in the mind to suffer"\
            " The slings and arrows of outrageous fortune, "\
            "Or to take arms against a sea of troubles."\
            "And by opposing end them. To die—to sleep,",]
inputs = tokenizer(prompts, padding=True, return_tensors='pt')
print(inputs)
out = model(**inputs)

generate_ids = model.generate(**inputs, max_new_tokens=128, min_new_tokens=30, early_stopping=True, num_beams=4)
ans = tokenizer.batch_decode(generate_ids, skip_special_tokens=True, clean_up_tokenization_spaces=False)
print(ans)

"""
ans[0]
'she opened the door and see me standing there.\n\n"What are you doing here?" she asked.\n\n"I came to see you," I said.\n\n"I don\'t want to see you," she said.\n\n"Why not?" I asked.\n\n"Because I don\'t want to see you," she said.\n\n"Why not?" I asked.\n\n"Because I don\'t want to see you," she said.\n\n"Why not?" I asked.\n\n"Because I don\'t want to see you," she said.\n\n"Why not?" I asked.\n\n"Because I'
ans[1]
'tell me 10 things about jazz music that i don’t know\n\nI’m not a jazz musician. I’m not a jazz fan. I don’t listen to jazz music. I don’t know anything about jazz music. I don’t know anything about jazz musicians. I don’t know anything about jazz fans. I don’t know anything about jazz. I don’t know anything about jazz musicians. I don’t know anything about jazz fans. I don’t know anything about jazz. I don’t know anything about jazz musicians. I don’t'
ans[2]
'What is the meaning of life?\n\nWhat is the meaning of life?\n\nWhat is the meaning of life?\n\nWhat is the meaning of life?\n\nWhat is the meaning of life?\n\nWhat is the meaning of life?\n\nWhat is the meaning of life?\n\nWhat is the meaning of life?\n\nWhat is the meaning of life?\n\nWhat is the meaning of life?\n\nWhat is the meaning of life?\n\nWhat is the meaning of life?\n\nWhat is the meaning of life?\n\nWhat is the meaning of life?\n\nWhat is the meaning of life?\n\n'
ans[3]
"To be, or not to be, that is the question: Whether 'tis nobler in the mind to suffer The slings and arrows of outrageous fortune, Or to take arms against a sea of troubles.And by opposing end them. To die—to sleep, No more; and by a sleep to say we end The heart-ache and the thousand natural shocks That flesh is heir to—'tis a consummation Devoutly to be wish'd. To die, to sleep; To sleep: perchance to dream: ay, there's the rub; For in that sleep of death what dreams may come, When we have shuffled off this mortal coil, Must give us pause: there's the respect That makes calamity of so long life; For who would bear the whips and scorns of time, The oppressor's wrong, the proud man's contumely, The"
"""

Dependency Change?
any library dependency introduced or removed

gpt-j 6b static batching prompts inference successfully without padding.
padding prompt is still under development
prompt = [ "she opened the door and see me", "tell me 10 things about jazz music" ]

inputs:
{'input_ids': tensor([[ 7091,  4721,   262,  3420,   290,   766,   502],
        [33331,   502,   838,  1243,   546, 21274,  2647]]), 'attention_mask': tensor([[1, 1, 1, 1, 1, 1, 1],
        [1, 1, 1, 1, 1, 1, 1]])}

results are as same as transformers
prompt 1:
she opened the door and see me standing there.

"What are you doing here?" she asked.

"I came to see you," I said.

"I don't want to see you," she said.

"Why not?" I asked.

"Because I don't want to see you," she said.

"Why not?" I asked.

"Because I don't want to see you," she said.

"Why not?" I asked.

"Because I don't want to see you," she said.

"Why not?" I asked.

"Because I don

prompt 2:
tell me 10 things about jazz music that i don’t know

I’m not a jazz musician. I’m not a jazz fan. I don’t listen to jazz music. I don’t know anything about jazz music. I don’t know anything about jazz musicians. I don’t know anything about jazz fans. I don’t know anything about jazz. I don’t know anything about jazz musicians. I don’t know anything about jazz fans. I don’t know anything about jazz. I don’t know anything about jazz musicians. I don’t

q4-j results from python api
from transformers import AutoTokenizer, TextStreamer
from intel_extension_for_transformers.transformers import AutoModelForCausalLM, WeightOnlyQuantConfig

model_name = "gpt-j-6b"
woq_config = WeightOnlyQuantConfig(compute_dtype="int8", weight_dtype="int4")
model = AutoModelForCausalLM.from_pretrained(model_name, quantization_config=woq_config, trust_remote_code=True)

prompts = [
           "she opened the door and see",
           "tell me 10 things about jazz music",
           "What is the meaning of life?",
           "To be, or not to be, that is the question: Whether 'tis nobler in the mind to suffer"\
            " The slings and arrows of outrageous fortune, "\
            "Or to take arms against a sea of troubles."\
            "And by opposing end them. To die—to sleep,"
            ]

tokenizer = AutoTokenizer.from_pretrained(model_name, trust_remote_code=True, padding_side="left")
tokenizer.pad_token = tokenizer.eos_token # gpt-j has no pad_token
pad_token = tokenizer(tokenizer.pad_token)['input_ids'][0]
inputs = tokenizer(prompts, padding=True, return_tensors='pt').input_ids

# pass pad_token if set it
outputs = model.generate(inputs, num_beams=4, max_new_tokens=128, min_new_tokens=30, early_stopping=True, pad_token=pad_token)
ans = tokenizer.batch_decode(outputs, skip_special_tokens=True, clean_up_tokenization_spaces=False)
for a in ans:
    print(a)

results:
1.
she opened the door and see me standing there.

"What do you want?" she asked.

"I want to talk to you," I said.

"I don't want to talk to you," she said.

"I want to talk to you," I said.

"I don't want to talk to you," she said.

"I want to talk to you," I said.

"I don't want to talk to you," she said.

"I want to talk to you," I said.

"I don't want to talk to you," she said.

2.
tell me 10 things about jazz music that i don't know.

1. jazz music is the most popular music in the world.
2. jazz music is the most popular music in the world.
3. jazz music is the most popular music in the world.
4. jazz music is the most popular music in the world.
5. jazz music is the most popular music in the world.
6. jazz music is the most popular music in the world.
7. jazz music is the most popular music in the world.
8. jazz music is the most popular music in the world.
9. jazz music is the most popular

3.
What is the meaning of life?

What is the meaning of life?

What is the meaning of life?

What is the meaning of life?

What is the meaning of life?

What is the meaning of life?

What is the meaning of life?

What is the meaning of life?

What is the meaning of life?

What is the meaning of life?

What is the meaning of life?

What is the meaning of life?

What is the meaning of life?

What is the meaning of life?

What is the meaning of life?

4.
To be, or not to be, that is the question: Whether 'tis nobler in the mind to suffer The slings and arrows of outrageous fortune, Or to take arms against a sea of troubles.And by opposing end them. To die—to sleep, No more; and by a sleep to say we end The heart-ache and the thousand natural shocks That flesh is heir to—'tis a consummation Devoutly to be wish'd. To die, to sleep; To sleep, perchance to dream: ay, there's the rub; For in that sleep of death what dreams may come, When we have shuffled off this mortal coil, Must give us pause: there's the respect That makes calamity of so long life; For who would bear the whips and scorns of time, The oppressor's wrong, the proud man's contumely, The

Can we compare the results of our multi-batches with those of a single batch?

@intellinjun @Zhenzhong1 @CeciliaWwq I add the model_input for model_eval. It may affect your model enabling PRs

Can we compare the results of our multi-batches with those of a single batch?

It should be the same. I have compared the batch 0. I will compare the rest later.

https://inteltf-jenk.sh.intel.com/view/nlp-toolkit-validation/job/ITREX-cpp-graph-extension/173/

Does all models support beam search after this pr ?

Does all models support beam search after this pr ?

If the model supports multi-batch inference && stores permuted kv tensor && uses single cache tensor

what kinds of test the PR will involve?

chatglm, chatglm2, dolly, mpt, starcoder: https://inteltf-jenk.sh.intel.com/view/nlp-toolkit-validation/job/ITREX-cpp-graph-extension/177/

what kinds of test the PR will involve?

1.ext test for all mode-archs due to the new model_eval API
2. beam search python ut (it will be removed into graph folder by @zhenwei-intel )

beam search python ut (it will be removed into graph folder by @zhenwei-intel )
in this pr?

beam search python ut (it will be removed into graph folder by @zhenwei-intel )

in this pr?

another PR (He will enhance llm runtime python test which would also add beam search feature)


the example in the readme

I saw this example in the readme and I really like it, is this just a gif for display? Or do you have the source code to achieve this effect through this project? Can you provide it? Thank you for your help.

Sure! You can achieve this effect like this:
let view = ProgressRing(to: .constant(0.8), color: .green)
    .overlay {
        Text("80%")
            .font(.caption)
            .foregroundStyle(.secondary)
    }

let dynamicNotch = DynamicNotchInfo(
    iconView: view,
    title: title,
    description: description
)

dynamicNotch?.show(for: seconds)

I haven't documented ProgressRing in depth yet, but it is included in the project :)
Let me know if you have any other questions!


What is the object fusion algorithm or architecture ? (radar + mobileye camera)

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Apollo installed from (source):
Apollo version (3.0):

You provided the code and the system architecture as open source. Thank you for that. However, I could not find sufficient information about object fusion (especially, radar + camera). Since I could not understand the object fusion logic from the codes, It is hard to grasp the architecture of radar + camera fusion. Can you provide a hint or documentation or something else which describes the radar+camera fusion of Apollo ?
Any information from anybody would be awesome.

@earcz Thank you for using Apollo! We do have document explaining general fusion algorithm but we don't have documents explaining all details at this moment.

Thank you for the response. I am actually curious about object fusion logic. Let’s say the camera does not see the object, but the radar sees the same object at a moment. This object cannot be fused. So, does the radar assigns an ID to this object and system takes an action according to the radar output? Or, is this object ignored because camera does not see? An answer to this question

We do not currently recommend this, and suggest porting the algorithm as much as possible


NuGet package outdated?

Hi,
I ran into an issue where setting ClassName would not do anything. Then I came here to see if it's a bug, and found it was fixed in August 2022 already (#53).
But the latest NuGet package (TinyMCE.Blazor) seems to be 1.0.4 from May 2022. Is it not getting built automatically and can you please provide an updated package?

We released a 2.0 version that should include that fix.


sbt 1.1.6

To validate this pull request against other sbt modules, please visit this link.

A validation involving this pull request is in progress...

The validator has checked the following projects against Scala 2.12, 2.11,
tested using dbuild, projects built on top of each other.

Project
Reference
Commit

sbt
1.x
sbt/sbt@e3131936090a8522d384362941cd0fd7977b38e9

zinc
pull/555/head
sbt/zinc@83feb94dca4913d232f47ef9667470bd787e7281

io
1.x
sbt/io@7cdc5f0f6d1989fbef577bd8ca476129f888d78f

librarymanagement
1.x
sbt/librarymanagement@a500d0e0a3ceee7f0e9172bb44d6f9b5aa15683c

util
1.x
sbt/util@b643c98f29458cca3e8452604d09f14b5705026b

website
1.x

:x: The result is: FAILED
(restart)

Closing this in favor of https://github.com/sbt/zinc/pull/551


filtering on sim.people.age + another field using '&' fails

This is a curious one. When the population changes due to births or deaths, filtering by sim.people.uid first in a sequence of conditions causes a shape mismatch ValueError. If the uid condition occurs after the other condition, it seems to work correctly.
import starsim as ss
import pandas as pd

ppl = ss.People(10000)
disease = ss.SIR()
pregnancy = ss.Pregnancy(pars={'fertility_rate': pd.read_csv(ss.root / 'tests/test_data/nigeria_asfr.csv')})
n_years = 3
sim = ss.Sim(dt=1 / 52, n_years=n_years, people=ppl, diseases=disease, demographics=pregnancy)
sim.run()

newborns = ((sim.people.age > 0) & (sim.people.uid > 10000)).uids # Works
newborns = ((sim.people.uid > 10000) & (sim.people.age > 0)).uids # Errors

@RHull-IDM - I think this is related to what Robyn mentioned here - https://starsim-idm.slack.com/archives/C059G2HE57X/p1717649148257889? It seems like almost the same thing. I think Cliff and/or Romesh are looking at this behavior and https://github.com/starsimhub/starsim/issues/508?

Closed by #561


Better nopush choices

if (pCmd->forwardmove == 0 && pCmd->sidemove == 0 && g_EntityCache.GetLocal()->OnSolid())
This works fine; however, I've noticed that the movement is slower than if nopush was off normally.
pCmd->buttons & (IN_FORWARD | IN_BACK | IN_MOVERIGHT | IN_MOVELEFT)
This is more accurate.
And truth be told, I love the slower version.
I don't want people pushing me out of spawn if possible, but I also don't want to be kicked if I stand still and someone nopush-checks me. So "Slower Idle" is a compromise between nopush and yespush- Players can push you while you're not moving, however the pushing is slower.  You'll be tougher to push out of spawn than normal players, but you will be pushed, which prevents hackusation.
this also fixes the issue where people not using WASD were fucked
+moved some things around (stop, during, start -> start, during, stop)

Considering f388c7c I'm guessing no slow push

idk maybe i really dislike the afk check tho

I had in-air check because the entire point of idle push (for me adding it at least) was to prevent teammates from pushing you while you're bhopping around, but considering you labeled the entire commit to 'fixing' nopush despite more code being spent on backtrack, I feel like this whole thing annoyed you so I'll just leave it like it is and fix it in my local build.

Baan is a meanie head I think it was a fine idea

it was called fixing it cause it didnt compile

the code was also messy with how it was checking the movement thing, and im pretty sure it didnt work cause if u failed the check the hook would just end and u would have no push


Added new clustering tools and reorganize existing ones

Implements #67, consider adding additional algos.
Proposed Changes

create new subpackage to contain clustering tools
add DBSCAN clustering algorithm
add Spectral clustering algorithm

Thanks for doing this. Do we want to add any other clustering methods or provide the option to fit a transformation kernel first before applying? Do we want to add an option to normalize the data before applying a clustering, either as a parameter or as a separate function?

Good idea, I'll look into it.

Do we even want to add normalization? It's a simple numpy call with std and mean, no? It would basically be a one-line wrapper call.
On Jul 19, 2021 6:26 PM, Arthur Lu @.***> wrote:
Thanks for doing this. Do we want to add any other clustering methods or provide the option to fit a transformation kernel first before applying? Do we want to add an option to normalize the data before applying a clustering, either as a parameter or as a separate function?
Good idea, I'll look into it.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://github.com/titanscouting/tra-analysis/pull/82#issuecomment-882927529, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEO4JMQLFNA6B5STHFEDYYLTYSYDVANCNFSM5AOPNZRA.

Do we even want to add normalization? It's a simple numpy call with std and mean, no? It would basically be a one-line wrapper call.

Might as well add the option


Try to fix a race condition that only shows up on TeamCity code coverage build…

…, by not removing temp files associated with Skyline Immediate window until that window is Disposed. Originally we didn't try to delete them at all, my recent commit deleted them when the tool writing to the window exited but that apparently introduced a race condition. So now they will accumulate during a Skyline run but should not remain afterward, which is still an improvement.

Note that this won't truly be tested until merged to master, since that's the only branch on which TC runs the code coverage test. But first let's make sure nothing breaks on this branch.

I added this PR to the branch filter for TC.

I added this PR to the branch filter for TC and manually triggered a build.

Thanks!

Fixed the broken test but broke two new ones?

Maybe? Though I don't see how, flow of control doesn't go into the area I changed. (Verified with breakpoints on my local machine)

The pwiz core was clearly a sporadic error that resolved with a rerun. I'll rerun the code coverage build to see if those were sporadic too.

The pwiz core was clearly a sporadic error that resolved with a rerun. I'll rerun the code coverage build to see if those were sporadic too.

I'd done the same, can't hurt to have two looks at the ball though.

@chambm They both passed, I'm going to go ahead and merge to master if you'll sign off on the changes.

We seem to be getting a lot more intermittent failures on TC lately. I wonder if it has to do with using different instance types and/or spot fleet.


What coin am I mining here?

Total n00b question, but I dont see anything in the read me.  Does nicehash mine zcash, or does the system automatically switch me to whatever network is most profitable?

NiceHash is an algorithm. And actually is just that, a hash algorithm.
ZCash uses that algorithm, and then to mine ZCash you need to use a program that implements that algorithm.
You are not using nicehash per se, you are using the miner nheqminer.
So you can use a miner, with the specific algorithm to solve a specific kind of cryptocoin.
Nicehash does not mine anything, and the nheqminer is used to solve nicehash.
If you want to use a system that chooses the coin you need a pool that do that.

The algorithm is Equihash, nicehash is just a company for rent hashing power on different algorithms, but they have build one of the better equihash miner: nhequiminer (NiceHash Equihash Miner), it's why a lot of other equihash miner are based on this one.
For the moment, i think ZCash is the only coin using Equihash, you can learn more on the website: https://z.cash/

Thanks ya'll.  this answers my question.

If you use nheqminer with NiceHash service with your Bitcoin address like so -l equihash.eu.nicehash.com -u BTC_ADDRESS you will be payed in Bitcoin for renting your hashing power.
But if you want to mine ZCash with it you can mine with your ZCash addres on our pool like so -l stratum.eu.zcash.nicehash.com -u ZCASH_ADDRESS. The eu location can be replaced with usa,jp and ch depending depending on your location


Add condition for runAqa config Json file

Added condition checks for runAqa Json file. Skip if there's no such file.
Fixes: #219
Signed-off-by: Jerome Ju<EMAIL_ADDRESS>

Tested with respective workflow in aqa_test_repo workflow and in openj9_repo workflow


enable_error_lens is configured to display diagnostic messages but …

enable_error_lens is configured to display diagnostic messages but does not include the styling for diagnostics.

We probably need a different config for this.