Not compatible with Node 16+

Given I am using VS Code, Windows 11,
And I have downloaded the 'example-carved-rock-fitness-order-tracker-master' code
When I execute 'npm install' in the e2e directory
Then I see the following:
PS C:\PS\Testing_Progressive_Web_Apps\my-carved-rock-fitness-order-tracker\e2e> npm install
npm WARN old lockfile
npm WARN old lockfile The package-lock.json file was created with an old version of npm,
npm WARN old lockfile so supplemental metadata must be fetched from the registry.
npm WARN old lockfile
npm WARN old lockfile This is a one-time fix-up, please be patient...
npm WARN old lockfile
npm WARN deprecated<EMAIL_ADDRESS>this library is no longer supported
npm WARN deprecated<EMAIL_ADDRESS>Fixed a prototype pollution security issue in 4.1.0, please upgrade to ^4.1.1 or ^5.0.1.
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>The querystring API is considered Legacy. new code should use the URLSearchParams API instead.
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained and not recommended for usage due to the number of issues. Because of the V8 engine whims, feature detection in old core-js versions could cause a slowdown up to 100x even if nothing is polyfilled. Some versions have web compatibility issues. Please, upgrade your dependencies to the actual version of core-js-pure.
npm ERR! code 1
npm ERR! path C:\PS\Testing_Progressive_Web_Apps\my-carved-rock-fitness-order-tracker\e2e\node_modules\fibers
npm ERR! command failed
npm ERR! command C:\Windows\system32\cmd.exe /d /s /c node build.js || nodejs build.js
npm ERR! gyp info it worked if it ends with ok
npm ERR! gyp info using<EMAIL_ADDRESS>npm ERR! gyp info using<EMAIL_ADDRESS>| win32 | x64
npm ERR! gyp info find Python using Python version 3.12.0 found at "C:\Users\user\AppData\Local\Programs\Python\Python312\python.exe"
npm ERR! gyp ERR! find VS
npm ERR! gyp ERR! find VS msvs_version not set from command line or npm config
npm ERR! gyp ERR! find VS VCINSTALLDIR not set, not running in VS Command Prompt
npm ERR! gyp ERR! find VS checking VS2022 (17.8.34330.188) found at:
npm ERR! gyp ERR! find VS "C:\Program Files\Microsoft Visual Studio\2022\Community"
npm ERR! gyp ERR! find VS - found "Visual Studio C++ core features"
npm ERR! gyp ERR! find VS - missing any VC++ toolset
npm ERR! gyp ERR! find VS checking VS2019 (16.11.34301.259) found at:
npm ERR! gyp ERR! find VS "C:\Program Files (x86)\Microsoft Visual Studio\2019\Community"
npm ERR! gyp ERR! find VS - found "Visual Studio C++ core features"
npm ERR! gyp ERR! find VS - missing any VC++ toolset
npm ERR! gyp ERR! find VS could not find a version of Visual Studio 2017 or newer to use
npm ERR! gyp ERR! find VS not looking for VS2015 as it is only supported up to Node.js 18
npm ERR! gyp ERR! find VS not looking for VS2013 as it is only supported up to Node.js 8
npm ERR! gyp ERR! find VS
npm ERR! gyp ERR! find VS **************************************************************
npm ERR! gyp ERR! find VS You need to install the latest version of Visual Studio
npm ERR! gyp ERR! find VS including the "Desktop development with C++" workload.
npm ERR! gyp ERR! find VS For more information consult the documentation at:
npm ERR! gyp ERR! find VS https://github.com/nodejs/node-gyp#on-windows
npm ERR! gyp ERR! find VS **************************************************************
npm ERR! gyp ERR! find VS
npm ERR! gyp ERR! configure error
npm ERR! gyp ERR! stack Error: Could not find any Visual Studio installation to use
npm ERR! gyp ERR! stack at VisualStudioFinder.fail (C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\lib\find-visualstudio.js:113:11)
npm ERR! gyp ERR! stack at VisualStudioFinder.findVisualStudio (C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\lib\find-visualstudio.js:69:17)
npm ERR! gyp ERR! stack at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
npm ERR! gyp ERR! stack at async createBuildDir (C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\lib\configure.js:69:26)
npm ERR! gyp ERR! stack at async run (C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js:81:18)
npm ERR! gyp ERR! System Windows_NT 10.0.22631
npm ERR! gyp ERR! command "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" "rebuild" "--release"
npm ERR! gyp ERR! cwd C:\PS\Testing_Progressive_Web_Apps\my-carved-rock-fitness-order-tracker\e2e\node_modules\fibers
npm ERR! gyp ERR! node -v v20.11.1
npm ERR! gyp ERR! node-gyp -v v10.0.1
npm ERR! gyp ERR! not ok
npm ERR! node-gyp exited with code: 1
npm ERR! Please make sure you are using a supported platform and node version. If you
npm ERR! would like to compile fibers on this machine please make sure you have setup your
npm ERR! build environment--
npm ERR! Windows + OS X instructions here: https://github.com/nodejs/node-gyp
npm ERR! Ubuntu users please run: sudo apt-get install g++ build-essential
npm ERR! RHEL users please run: yum install gcc-c++ and yum groupinstall 'Development Tools'
npm ERR! Alpine users please run: sudo apk add python make g++
npm ERR! 'nodejs' is not recognized as an internal or external command,
npm ERR! operable program or batch file.
npm ERR! A complete log of this run can be found in: 2024-04-04T20_05_50_162Z-debug-0.log2024-04-04T20_05_50_162Z-debug-0.log

You'll want to use Node v12.8.3 for now, reflected in the .nvmrc of the repo since it is not yet compatible with latest LTS versions of Node.js. I recommend using nvm to manage versions, which makes this a bit easier.

no thank youSent from my iPhoneOn Apr 22, 2024, at 12:40 PM, Kamran Ayub @.***> wrote:
You'll want to use Node v12.8.3 for now, reflected in the .nvmrc of the repo since it is not yet compatible with latest LTS versions of Node.js. I recommend using nvm to manage versions, which makes this a bit easier.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

This is now fixed and the project is supported on all current Node LTS versions, v20.12.2 and v18.20.2.
However, WDIO tests will not work until the test specs are migrated to the async API, which would be out-of-sync with the course. I will create a separate issue for that.


Postgres storage conflicts if already using sqlx

Since apalis postgres uses sqlx for migrations, it conflicts with the existing sqlx migrations I have in my project. Will it be possible to expose an option which will change the sqlx table used by apalis?

The migration name in the screenshot is mine. I am running sqlx::migrate! before establishing the connection for apalis and running setup.

Looking at the migrate! docs, I don't think it is possible to change the table name.

Usually apalis uses its own tables and schema.
There are possible workarounds in #81

I don't think the workarounds will work for me.

Usually apalis uses its own tables and schema.

What does it use?

Not sure what you are asking.

I am aware that apalis uses the Jobs table. But does it use a different postgres schema? If it does then the error I am getting should not have popped up...

It uses apalis schema.
https://github.com/geofmureithi/apalis/blob/42b1ab297c5b123c6b1c32a462691937c4939094/packages/apalis-sql/migrations/postgres/20220530084123_jobs_workers.sql#L1

It creates this schema for its own tables, not for the _sqlx_migrations table that sqlx creates. That's why it is conflicting.

Yeah, I am thinking of dropping sqlx and going with the native drivers or diesel. Still thinking about it.

While the current implementation is problematic, you might get some success with this trait:
https://docs.rs/sqlx/latest/sqlx/migrate/trait.MigrationSource.html
You could in theory, import apalis migrations as Vec<Migration>, do the same for your migrations, merge and sort.
See also https://github.com/launchbadge/sqlx/issues/1934
I could give it a try and see if I can crack it.

How do I import the apalis migrations?

If I was able to expose apalis's migrations as Vec<Migration> for example apalis::migrations::postgres::migrator()
pub struct ApalisPlusPath(Path);

impl<'s> MigrationSource<'s> for &'s ApalisPlusPath {
    fn resolve(self) -> BoxFuture<'s, Result<Vec<Migration>, BoxDynError>> {
        Box::pin(async move {
            let mut migrations = self.0.resolve().await?;
            migrations.extend(apalis::migrations::postgres::migrator()); // Append apalis migrations
            // ensure that we are sorted by `VERSION ASC`
            migrations.sort_by_key(|m| m.version);

            Ok(migrations)
        })
    }
}

This should work for you right?

Yeah it should.

I would expose it via this without the run.
https://github.com/geofmureithi/apalis/blob/42b1ab297c5b123c6b1c32a462691937c4939094/packages/apalis-sql/src/postgres.rs#L69C9-L69C46

Alright LGTM.

Ok I will sort you out this weekend. On the two issues.

Please check if this helps you. #179

@geofmureithi Yep, it works!


No curses for Business

Signed up for business to get some courses but nothing shows up. Even with the correct business name. Are we broken?

you need to get enrolled to the course first - the listed courses are the ones you have in : https:///home/my-courses/learning/


Documentation pages are blank

Problem description
Most documentation pages on docs.docker.com are blank pages, containing no content.
Reproduction

Navigate to https://docs.docker.com/.
Follow a link, e.g. "Write a Dockerfile" to https://docs.docker.com/engine/getstarted/step_four.md.
The loaded page is blank.

Browser: links2.

@hlandau Can't reproduce; can you tell me what browser you're using?

links2.

@hlandau This is really uncommon since links is not that widely used.
You might check this SSL/TLS error with links and lynx. I have 2.8 version installed. It says "SSL error", as the above link pointed out.
If updated to 2.13, it works, so if this is the only browser you can use, update it and good luck!

Try https://docs.docker.com/engine/getstarted/step_four/, not https://docs.docker.com/engine/getstarted/step_four.md.

That works, but everything links to .md, not /.

If you are in a non-links browser, we actually catch links to .md and change them to /
Regardless of that, I've added logic to 404.md that catches attempts to use a .md URL and forwards you to the equivalent / URL with forwarding logic. Can you try clicking on https://docs.docker.com/engine/getstarted/step_four.md in links and let me know if you get forwarded?

Nope, still blank.

Well, we've done all we can. Any redirect solution we have available to us is going to use  or window.location.replace, and currently we're doing both just so that as many browsers as possible get redirected. I'm afraid we have to say that Links is not supported. In fact I'd suspect that there are many things are are broken besides this issue in text-only browsers that have no JS support. If you need to use a text-based browser, here is a Stack Overflow topic on text-based browsers that have JavaScript support:
http://askubuntu.com/questions/34687/is-there-a-text-mode-browser-which-supports-javascript


UI - token page

[ ] #51
[ ] #52
[ ] #54
[ ] #55

Adding more sub-issues:

[ ] UI - token page: edit/create form validation triggers too early
[ ] UI - token page: transfer action appears only on the token edition or creation
[ ] UI - token page: highlight user tokens when listing tokens


Create Indexer for Sputnik DAO

Description
We need to build an indexer similar to the one used for DevHub proposals, designed to index proposals from Sputnik DAOs. This will enable enhanced functionality for filtering and searching within the Treasury Dashboard.
Scope
Decide Indexing Strategy

Should we index all .sputnik-dao.near accounts, or only the accounts associated with our treasury?
If we choose to index treasury-related accounts only, we need to implement an API to dynamically start indexing new accounts created through the self-create flow.

Data to be Indexed

Proposal data as retrieved from the DAO contract.
Execution transaction details, including the link and timestamp.

Required Endpoints
Sorting

By creation time.
By expiry time.

Filtering

By creator.
By proposal type.
By number of votes.
By proposal status.

Searching

By proposal description.

Notes from our standup call: Ideally create one indexer for all DAOs, leverage sharding ... instead of deploying an indexer per DAO instance.

Game Plan for Sputnik DAO Indexer

Indexing Strategy
We will use a centralized indexer to handle all DAOs rather than deploying a separate instance for each DAO. The endpoints will support filtering and querying by DAO account.
Since our Fly.io machines remain offline unless needed, we’ll implement a "just-in-time" indexing strategy. This means we’ll only index data when the frontend requests it for a specific DAO account. This approach will index all relevant data, but only when an updated table for a DAO account is requested.
To fetch uncached transactions, we’ll index each DAO subaccount separately using the Nearblocks API. This ensures we retrieve all transactions not already stored in our cache.
Given the relatively small volume of data, all transactions can be stored in a dedicated table. From there, we’ll transform this raw data into specific tables optimized for sorting, filtering, and searching.
Additionally, we can index the DAO factory to detect and dynamically start indexing new accounts created through the self-create flow.

Data to Be Indexed
Most of the proposal data is already indexed, including searching by description, which is functional. However, we need to index the following new data:

Execution transaction details, including:

Transaction link (e.g., to Explorer).
Execution timestamp.

New API Endpoint
The new endpoint will allow querying transactions with sorting and filtering options:

/txns/{DAO_ID}/?sort={sorting}&filters={filters}

Sorting Options:

By creation time.
By expiry time.

Filters:

By creator.
By proposal type.
By number of votes.
By proposal status.

This endpoint will integrate seamlessly with the frontend, providing the necessary data for the Treasury Dashboard.

Note that sorting by creation time and expiry time will be the same since the expiry time is defined by the creation time + policy duration time.

Update
I'm currently working on indexing add_proposal and act_proposal functions from the DAO contract.
At first I wanted to use the transaction arguments to indext the state. However these functions change the proposal's inside the contract with a ton of logic.
So instead of recreating that we just get the proposal from contract on the transaction blockheight. I'm now (Friday Dec 20) working on deserializing the string U128. So we can use the RPC response to cache the proposal.
@frol do you have a tip to deserialize this?
JsonRpcError(ServerError(HandlerError(ContractExecutionError { vm_error: "wasm execution failed with error: HostError(GuestPanic { panic_msg: \"panicked at 'ERR_NO_PROPOSAL', sputnikdao2/src/views.rs:102:48\" })", block_height: 114574681, block_hash: CEx1UdXoBRxsBFoHys9m7zaMwQsRP9e7qiVvHZWW1erf })))
DeserializeError(Error("u128 is not supported", line: 1, column: 2807))


Align license_files globs with data_files globs

In #2712, Setuptools is considering adding glob support for declarative config for data_files. In that PR, I observed that license_files already has glob support, but handled in the dist and not the config. In this comment, option A proposes to align the license_files directive with this design, requiring users to use a glob: directive to indicate if a glob is present (and if wildcards are present, presume they're literals). To deal with the backward-incompatibility of this change, I propose the following transition for license_files:

Add support for glob: directive, and if present, expand the values when loading the config (same as data_files).
Deprecate globs without a glob: directive and warn if such a value expands due to globbing.
In the near future, O(months), remove support for globs without the directive.

If I recall, the license_files setting (and its default) is also entangled with license_file. Perhaps as part of this change, support for license_file could also be removed or at least decoupled from license_files, such that the default for license_files would be simply glob: LICEN[CS]E*, COPYING*, NOTICE*, AUTHORS*.
@cdce8p Since you contributed much (all?) of the license_files support, I'd like your opinion. Does this approach sound acceptable?

Thank you for bringing it to my attention. I have to admit that I needed to think about it a bit before writing this.
My goal for the original PR (#2620) was to make it easier to include license files by copying what wheel already does, i.e. using the patterns as globs: https://wheel.readthedocs.io/en/stable/user_guide.html#including-license-files-in-the-generated-wheel-file
It's true that adding the glob: directive would certainly make setuptools more consistent, it does however also introduces a challenge that I fear are not easily overcome:
wheel still depends on the license_files settings to compute which license files to include in the build distribution. Thus adding glob would make setuptools incompatible with wheel. It would require an update to wheel and would put additional requirements on which version of setuptools works with which version of wheel.
In the end it might be possible to do the deprecation, I'm just not sure if there is a true need for it. It should be fair to assume that most users either didn't add the license_files section, in which case the defaults are set by us, or know about it and expect it to work exactly like wheel i.e. the same output for the source and build distributions (at least with respect to the license files).
--
Why handled in dist not config
When I started to working on it I was new to setuptools. dist just had some prior work regarding the license handling and it made sense to add it there. Especially since originally it only modified the filelist. Support for the metadata item came later.

What are your thoughts on this @jaraco?
It sounds like it might be too disruptive to add the glob: directive to license_files if we want to work the same way as wheel. Perhaps the choice of following "the wheel way" can be more explicitly explained in the setuptools documentation to prevent user confusion.

Sorry for not giving feedback sooner. I'm leaning that direction too. It definitely would be unfortunate to have different formats for the same concept in the same tool and preferable to converge across tools.
@darkvertex Would you be willing to adapt the PR to implement implicit glob support?

@darkvertex Would you be willing to adapt the PR to implement implicit glob support?

Sure, no problem. I'll get back to you.

@jaraco Updated my PR: https://github.com/pypa/setuptools/pull/2712 👍
Since we're going for "implicit" globbing, I made a choice to support globby-like filenames literally should they exist, so if someone made such a file and is expecting it to match the real file on disk instead of expanding the pattern, they will get the expected result (I think!) -- At the same time, it's a bit of an edge case, but.. 🤷♂
If you disagree with this decision we can take it out but I think it's what a user would expect.
You can see my updated unit test for an example using a file with a question mark and having it match the file correctly as a literal value.


feature: show open files in fuzzy finder

If any open files are not in the current project, they are not found by the fuzzy finder.
I would like to use the fuzzy-finder to switch to a file that is:

open in the current Atom window
but not inside the project root(s)

cmd/ctrl-b will allow you to switch within open files.  Is that what you're asking for?

Yeah, thanks. Is there an option to have those results appear in the cmd-P prompt?

Not at the moment, no.


bug:

Issue description
Environment

apisix version (cmd: apisix version):
OS: 1.4

Minimal test code / Steps to reproduce the issue
When creating a route, the route is created successfully. If the route is modified through the web page, an error is reported
What's the actual result? (including assertion message & call stack if applicable)

What's the expected result?

@gxthrj please take a look

ok, get it

@juzhiyuan please have a look, when redirect key should not be exist when disable redirection

Please make sure you are using the latest codes from master branch by git pull
It seems you are under development mode from your snapshot, if you are using the latest codes, then IT would be a bug on FE, we may need to check the transformer logic when update.

Thank you for your answer
At present, the latest master branch has been used. This problem has been solved, and a stable version of the front end is expected

OK, it looks like the problem has been resolved, feel free to reopen it if you find the related issue.


chore: where should the proposed addition to NLP rust go

@neoncitylights proposed some functionality such as
Word::new("woman").as_plural() // "women"
Word::new("men").as_singular() // "man"
Word::new("walking").as_morphemes() // Vec["Walk", "-ing"]
Word::new("walking").as_lemma
Should we add this to differ.rs ie ( the monorepo ) or should we put it in a separate directory ?

How about under crates/semantic_diff?

thats fine ill go ahead and get that started


ES6 feature: Modules: Tracking Issue

Syntax:

imports:

ImportDeclaration :
  import ImportClause FromClause ;
  import ModuleSpecifier ;
ImportClause :
  ImportedDefaultBinding
  NameSpaceImport
  NamedImports
  ImportedDefaultBinding , NameSpaceImport
  ImportedDefaultBinding , NamedImports
ImportedDefaultBinding :
  ImportedBinding
NameSpaceImport :
  * as ImportedBinding
NamedImports :
  { }
  { ImportsList }
  { ImportsList , }
FromClause :
  from ModuleSpecifier
ImportsList :
  ImportSpecifier
  ImportsList , ImportSpecifier
ImportSpecifier :
  ImportedBinding
  IdentifierName as ImportedBinding
ModuleSpecifier :
  StringLiteral
ImportedBinding :
  BindingIdentifier

exports:

ExportDeclaration :
  export * FromClause ;
  export ExportClause FromClause ;
  export ExportClause ;
  export VariableStatement
  export Declaration
  export default HoistableDeclaration[Default]
  export default ClassDeclaration[Default]
  export default [lookahead ∉ {function, class}] AssignmentExpression[In] ;
ExportClause :
  { }
  { ExportsList }
  { ExportsList , }
ExportsList :
  ExportSpecifier
  ExportsList , ExportSpecifier
ExportSpecifier :
  IdentifierName
  IdentifierName as IdentifierName

Spec:

imports: https://people.mozilla.org/~jorendorff/es6-draft.html#sec-imports
exports: https://people.mozilla.org/~jorendorff/es6-draft.html#sec-exports

Remaining Tasks:

[x] Land the Classes feature
[ ] Review and Test Token Generation
[ ] Review Error Messages
[ ] Fix #311
[ ] Address https://github.com/estree/estree/issues/11
[ ] Address https://github.com/estree/estree/pulls/38
[ ] Address https://github.com/estree/estree/pulls/39
[ ] Fix #1077
[ ] Fix #1087
[ ] Fix #1088

We need to discuss in Wednesday meeting on whether we need to expose an API to parse using different start symbols.
In ES5, the only start symbol is Script, so parse is unambiguous. In ES6, we have two symbols, Script and Module. Modules are parsed only in strict mode and imports/exports are only allowed in top level of Modules.

For now, Script vs Module can be handled just like in commit 85fc2f4b from the harmony branch.

FWIW We've found it useful already (at FB) to add a nonStrictModule source type for intermediary conversion-of-codebase purposes. I've been meaning to send this upstream, but wanted to try it downstream first to be sure it's worth it.
(@ariya feel free to say you told me so: https://github.com/jquery/esprima/pull/290#issuecomment-59876225)
Worth considering here too. We found it to be super convenient to be able to opt in to disabling implicit-strict in order to switch existing CJS (etc) modules over to ES6 before having a chance to go through and audit for strict-safety.

@jeffmo that's pretty interesting.

I plan to work on this this, at least taking the bulk of the work.

https://github.com/jquery/esprima/pull/1149


Fix Grammar and Spelling Mistakes

Fixed a spelling mistake (singe instead of single) and cleaned up some grammar issues while I was in here.

@TimCorey : Thanks for your contribution! The author, @CarlRabeler, has been notified to review your proposed change.

#sign-off


chore: update sync upstream script

Description

Make possible to specify target branch for sync-upstream script
Periodic execution using cron has been removed

https://github.com/tier4/autoware.universe/actions/workflows/sync-upstream.yaml

Tests performed

Test1: https://github.com/tier4/autoware.universe/actions/runs/6069493855
Test2: https://github.com/tier4/autoware.universe/actions/runs/6069495007
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

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

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

[ ] The PR follows the pull request guidelines.

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

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

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

sync image


pull request for e2e test at 01/18/2017 19:26:14

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


[opentelemetry] Evaluate Audit Logging with OpenTelemetry

Context
Currently we are using another solution (Logshipper?) for shipping audit logs. We need to evaluate if and how OpenTelemetry can handle and ship audit logs.
Acceptance Criteria

[ ] Evaluate Audit Logging based on OpenTelemetry in Greenhouse

References

We decided to use OpenTelemetry for Audit-Logging as a separate Greenhouse Plugin. A new issue has been created to track the status of development here #544


working with erobhal

Moving the logic into the manifest.
Not finished, needs more spec test, docs and style fixes.

@ghoneycutt This patch brings in support for periodic tasks on Suse and RHEL5 that was missing (removed actually) from the vendor versions of crontab.
My question here is if I should activate that functionality by default or not ?
pro: it is a fix for a problem in my eyes
con: it will change the default behaviour of the module
What do you think ?

updated spec & docs

@erobhal From my perspective these commits are ready to get merged. It should fully feature your functional change with some additions from my side. Please give it a test that I haven't missed something you need.

I don't have strong feelings about it. Personally I don't use the periodic entries and I think it was right for the distro's to remove them in favor of using cron.d. Adding them to systems where they don't already exist could cause confusion.
Perhaps if certain platforms use it, then it could be a boolean to be enabled for those platforms.

Which after refreshing the code, seems to be what you did:)

Problem is that several packages depend on them. Like logrotate for example. We had a bunch of SuSE-systems where the logs didn't get rotated.

I don't have strong feelings about it. Personally I don't use the periodic entries and I think it was right for the distro's to remove them in favor of using cron.d. Adding them to systems where they don't already exist could cause confusion.

Perhaps if certain platforms use it, then it could be a boolean to be enabled for those platforms.

I read the OS vendor docs to see what their defaults looks like. Only RHEL5 and Suse are using them and so does the module now by default.

Which after refreshing the code, seems to be what you did:)

And backward compatibility can be reached by setting $periodic_jobs_manage  to false.
Gentlemen, thanks for your help and time !! I will add the missing string support for the content, merge, and release it.

Released in 2.1.0


Add data catalog in Concepts

Add Data Catalog concept in data engineering wiki

Hi @JPHaus, Please help review the above request.


Read me update to explain how to actually produce the components necessary to have a new fully operation section

[ ] Explain how to actually produce the components necessary to have a new fully operation section (whether its local, static, dynamic or configurable)

It lacks the required variable in the data of the view component (for static, dynamic and configurable)

It lacks the required format to get data sent to sections for static section types forms

it lacks the mechanism we defined to customize the display of form elements for configurable section types forms

Basically we need to cover these topics with links to examples (from the various sections we already created, so it's clear for any dev on how to produce new section components
Acceptance testing

[ ] Have Julien validate the read me

https://gl.eweev.com/eweev/geeks/sections/-/wikis/home


fix(playground): loading whitespace and auto scroll on template output

Closes #72

Changes

Contents of playground base always be at the bottom of the screen. Even if the content is still loading.
Auto scrolling to the template output section so that the footer and related components does not overflow.

Related Issues

Issue #72

Author Checklist

[x] Ensure you provide a DCO sign-off for your commits using the --signoff option of git commit.
[ ] Vital features and changes captured in unit and/or integration tests
[x] Commits messages follow AP format
[x] Extend the documentation, if necessary
[ ] Merging to main from fork:branchname

@sanketshevkar good to go now!


How to install

Hi!
Is this node working and is there a way to (manually) install this node? I didn't figure out how...
Thank you for your help!

Hi @BlackAlpha700
It think it was fine when I was using it, although it is quite a while ago.
Let me know if you have any issues, I still have a Danfoss Air unit, so I can test it out if need be.
Cheers.

Hi @Laro88,
thanks for your reply. :-)
Maybe I'm a little stupid, but i didn't figure out how to install your node in my node-red installation because I can't find it via Palette-Manager.
Greetz.

I looked at the code yesterday, something is definently missing, seems like I never pushed it properly.
The underlying node.js IO module is not referenced so something is completely bonkers.
I hope I stille have the computer that I wrote the code on, I will take a look in ond of the upcoming days.
Regards
Laro88

Thank you! It would be great if you find the missing parts. :-)

Hey @BlackAlpha700
Lost some code somewhere, but finally got the underlying driver up and running.
Any chance that you have the time to do a git checkout of my dfair branch, find your DFAir units ip on your lan, and run the testapp against your IP?
You should get an object with the fan speeds and current humidity printed in the console.
Just need to know it the underlying IO is ok before we start messing around in nodered :-)
Cheers

Hi @Laro88 ,
after I'd figured out, what I had to do in detail, I've changed the IP in the demoapp and run a "node demoapp_knownipnumber.js" - it works! :-)
root@dfair-test:~/dfair# node demoapp_knownipnumber.js demoapp_knownipnumber started initialized dfair_io using ip:xxx.xxx.xxx.xxx Connected Sanity passed ArefreshData Refreshing data ArefreshData Refreshing data dfair_callback invoked: payoad:[{"name":"relative humidity measured","unit":"%","endpoint":4,"address":5232,"datatype":"byte","scale":0.39215686274509803,"value":44.705882352941174,"valuetimestamp":0},{"name":"Actual Supply Fan Speed","unit":"rpm","endpoint":4,"address":5200,"datatype":"ushort","scale":1,"value":2778,"valuetimestamp":0},{"name":"Actual Extract Fan Speed","unit":"rpm","endpoint":4,"address":5201,"datatype":"ushort","scale":1,"value":2965,"valuetimestamp":0},{"name":"Total running minutes","unit":"min","endpoint":0,"address":992,"datatype":"uint","scale":1,"value":5230765,"valuetimestamp":0}] ArefreshData Refreshing data dfair_callback invoked: payoad:[{"name":"relative humidity measured","unit":"%","endpoint":4,"address":5232,"datatype":"byte","scale":0.39215686274509803,"value":44.705882352941174,"valuetimestamp":0},{"name":"Actual Supply Fan Speed","unit":"rpm","endpoint":4,"address":5200,"datatype":"ushort","scale":1,"value":2790,"valuetimestamp":0},{"name":"Actual Extract Fan Speed","unit":"rpm","endpoint":4,"address":5201,"datatype":"ushort","scale":1,"value":2937,"valuetimestamp":0},{"name":"Total running minutes","unit":"min","endpoint":0,"address":992,"datatype":"uint","scale":1,"value":5230765,"valuetimestamp":0}] ^C
For me it looks all fine! :-)

Yup - looks good, you are on the right track!
So - which data do you need from the Air unit - what is your usage scenario? Do you only have one unit or many?
I don't think I have time right now to reimplement the set operations (boost, bypass, mode and manual level) but lets see.
Next for me is to dust off the nodered node that I made and find out if I have a branch hiding somewhere. Update time, seems like a couple of things has changed afterall.


Adding new Soundpack

Insert Pack Name(s) Here
Insert a link to the GitHub repository of the pack.
Checklist
Failure to complete this checklist or deleting boxes from the checklist will result in the closing of your pull request unless this is a pack update. Please write any comments regarding this checklist at the bottom of your pull request.
Check every box.

[ ] I am the original author of this pack or have permission from the original author to make this pull request.
[ ] All copyright of this pack's contents belong to the listed author or is cited in the description and repository linked above.
[ ] This theme works properly on the latest versions of SteamOS for Steam Deck, decky-loader and SDH-AudioLoader.
[ ] This pack is under 10MB in size and uses the least disk space possible.
[ ] This pack is safe for work and does not contain any sexual, drug-related, or profane content.
[ ] This pack does not contain loud, distorted, or otherwise bad faith sound files.
[ ] This pack is only for either music or sounds and only adds or modifies the intended sound files.

This pull request is extremely messy and seems to have conflicts. Please create a new fork or rebase your fork from this repo. You should also update from origin through GitHub whenever you add a new JSON to your main branch as it can create more issues like these.


Fix/coverage failures

Fixing coverage reporting

Added CodeCov.io to the mix.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@37a0a63). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #273   +/-   ##
=======================================
  Coverage        ?   97.52%
=======================================
  Files           ?        8
  Lines           ?      445
  Branches        ?       67
=======================================
  Hits            ?      434
  Misses          ?        9
  Partials        ?        2

Flag
Coverage Δ

unittests
97.52% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.


Trident backends remain "online" even when not available

Describe the bug
When a SVM is not available, I would expect backend status in trident not to be online, but rather offline or non available.
However, statues do not change (even after 8hours ... so not a timeout issue)
Environment
LoD Kubernetes v4 with the following upgrades

ONTAP 9.8P1
Trident 21.01.1

To Reproduce
Steps to reproduce the behavior:

Modify all LIF of a SVM to status_admin = down
Create a PVC against this SVM (will fail) or run a tridentctl update backend (will also fail)
Check the status of the backend with tridentctl get backend: still online

Expected behavior
Backend status to reflect the correct status
Additional context
Some customers rely on Trident metrics to monitor backend statuses.
if Trident always report "online", they will not know that something happened on the backend

The issue is fixed with https://github.com/NetApp/trident/commit/458db05e6475ab095351848a849d4a5a8de3f937 and is included in Trident 23.07 release.


[1241] Add support email link to footer

Changes proposed in this pull request
Add support email link to footer
Guidance to review
After

Just noticed a very subtle text colour change between mcui and mcf - dunno if it's deliberate or not:
mcf

mcui

@benilovj it was deliberate as I didn't see any benefit in adding the CSS rule into the rails app to make the text black. Happy to include. CC. @fofr

The new one is better.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-source-version-target/pull/126 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-source-version-target.git

Docs Build status updates of commit 813edad:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

versioningArchiveSourceFolder/TOC.yml
:white_check_mark:Succeeded
View (sql-server-2014 || azuresqldb-current || azuresqldb-mi-current || azure-sqldw-latest || >= aps-pdw-2016 || >= sql-server-2016 || >= sql-server-linux-2017 || sql-server-previousversions || sqlallproducts-allversions)

versioningArchiveSourceFolder/docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


General macro for supporting only a subset of fields in the Backend.

My general stance is to avoid unsafe code if possible so here's my proposal:

Introduce a Columns enum which holds Vec<String, Vec<T>> for all possible T we support
Introduce a ProgramWithFixed enum which holds (Analysis<T>, Vec<String, Vec<T>>) for all possible T we support
Pass this instead of the first two arguments in the BackendFactory trait. The T generic parameter disappears.
 Have the backend implement the ones it supports. This can use a macro:

impl Backend for Plonky3 {
   fn prove(&self, witness: &Columns) -> Result<Proof, Error> {
      call_with_all_fields!(Plonky3FieldElement, p3_prove, self, witness)
   }
}

where the macro expands to
match self.program {
   GoldilocksProgram(p) => Ok(p3_prove(self, witness)?),
   BabybearProgram(p) => Ok(p3_prove(self, witness)?),
   _ => Err(Error::UnsupportedField)
}

where the implemented arms correspond to the fields which implement Plonky3FieldElement. If this is not something we have access to, maybe we can make use of the KnownField enum.
This proposal may need a bit of adjustment but from my perspective:
Cons:

Requires changing the backend API
Pros:
No unsafe code
We'd have to make such a change anyway to allow backends to accept a program which internally uses different fields, think recursion, folding etc

I don't get the Pro number two. A change in this direction might be needed, but it is not sufficient, right? The backend interface would have to change in other aspects, and probably our structs representing the program, too.
Anyway, one pro for the current PR is that it is small and reuses the interfaces we already have. And the unsafe is contained within a small block inside the match choice that guarantees its safety at runtime.


pip3 install marklogic_python_api or pip install marklogic_python_api does not work

Hello i am using Linux Mint and installed pip and pip3 (i don't know what the difference is) when i executing the following command in the terminal "pip3 install marklogic_python_api"
It returns to the following result:
Collecting marklogic_python_api
  Could not find a version that satisfies the requirement marklogic_python_api (from versions: )
No matching distribution found for marklogic_python_api

Can you tell me, what can i change to use python for Marlogic
Manny thanks,
Erik

I'm still uploading it to the test repository until I think it's really stable. Progress has been delayed by other priorities. Try:
/usr/bin/pip3 install marklogic_python_api -i https://testpypi.python.org/pypi

I'm trying to install this package on a windows machine and I've triedpip install marklogic_python_api as well as  pip install marklogic_python_api -i https://testpypi.python.org/pypi and either do not work. Is this package still available?


api: inet_pton

inet_pton converts an IPv4 or IPv6 Internet network address in its standard text presentation form into its numeric binary form
https://learn.microsoft.com/en-us/windows/win32/api/ws2tcpip/nf-ws2tcpip-inet_pton

what namespace would you propose for this? communication/ip::manipulate IP address ?

how about just using communication/ip?


Roll DEPS.xwalk

disable_webmidi.
blink-crosswalk:
22631cd83923ea63e4d68fd0f4c5d0189b55c95f

@PeterWangIntel,  roll DEPS.

LGTM. @halton, it has rolled the DEPS.xwalk to the newest patch about MIDI.


Adds lifetime to IoCmd, init ipmi commands

@ultimaweapon Does it make sense for a each process to have it's own instance of Vm? shouldn't it be inside of the ProcManager?

That is not possible on macOS. It also seems like not a good idea since the amount of process may be hundred.

If it doesn't make sense... why is it that way in the code? :eyes:

Well... I suppose it made sense with only one process

If it doesn't make sense... why is it that way in the code? 👀

What do you mean by that? Our code current have a single VM that will be hosting multiple processes.

I mean the Vm struct

In that case yes, each process will have only one. I believed the reason FreeBSD put this as a top-level is because the VM can live alone without a process. The process is just one of a thing that depend on the VM. So I think it is better to keep it on a top-level.

I'm not sure I can make us run in a VM. Maybe you have to do it


Fixed RemoteOptions Integration With RemoteInfo (#880)

Updated RemoteInfo so that properties are correctly set/retrieved when interacting with the RemoteOptions property
Added some unit test to check for expected functionality
Updated NEXT.md

Thanks a lot, especially for the tests and the release not :+1:


[Nodes] Clean up PFN node docs.

Description
This PR builds on https://github.com/aptos-labs/developer-docs/pull/168 and continues to clean up the node docs. In this PR, we:

Restructure the PFN section, including adding a new section for deployment methods and splitting up the different deployment methods into different files.
Adds a new dedicated page for verifying a PFN.
Takes a pass over all PFN related docs.
Adds explicit hardware, network and storage requirements for PFNs.

Still more work is required on the configuration and validator sections. Will follow up in a separate PR.
Checklist

[x] Do all Lints pass?
[x] Have you ran pnpm spellcheck?
[x] Have you ran pnpm fmt?
[x] Have you ran pnpm lint?

Closing out -- will roll into a giant PR 😄


There is no easy way to resize the window and backbuffer at runtime

At the moment, resizing the SadConsole window requires the user to set the backbuffer and window size, then manually call GraphicsDeviceManager.ApplyChanges() and ResetRendering(). It would be nice for users to not have to duplicate this logic.
The following is a working example of the task from my own code that sets the window size to the screen size (thanks to Thraka for providing the basic outline and helping to debug it):
              var mode = GraphicsAdapter.DefaultAdapter
                    .SupportedDisplayModes
                    .OrderByDescending(dm => dm.Width)
                    .First();

                root = new SadConsole.Console(
                    mode.Width / Global.FontDefault.Size.X,
                    mode.Height / Global.FontDefault.Size.Y);
                Global.CurrentScreen = root;

                Global.GraphicsDeviceManager.PreferredBackBufferWidth = mode.Width;
                Global.GraphicsDeviceManager.PreferredBackBufferHeight = mode.Height;
                Global.GraphicsDeviceManager.ApplyChanges();

                Global.WindowWidth = mode.Width;
                Global.WindowHeight = mode.Height;
                Global.ResetRendering();

Thanks, I got this fixed! It will be in the next release. I added Global.Settings.ResizeWindow


Website with basic authentication

I´m trying to automate a web test in opentest with a public Url which uses basic authentication
When i try to pass credentials in url like this:
<EMAIL_ADDRESS>Steps finish ok, but the next steps of the test dont work properly, and in the end, log show me diferents 401 error.
Failed to load resource: the server responded with a status of 401 (Unauthorized)
How can I include basic authentication in test web browsing?
I trying to test a website with basic authentication

@vmsoriano try this: https://github.com/mcdcorp/opentest/issues/604#issuecomment-1504874522


Restaurant Fetch class

The purpose of this class is to interface with our REST api. When the user wants to view a specific restaurants page this class will fetch the information from the server and then store it in the model on the client.
This class is used when the restaurant needs to be fetched or updated (requery). It will make updates to the data model stored in the passed ArrayListAdapter
See the Volley android api for more details

I am not too sure about this one, but I can work with someone on this, and try to learn


Preliminary scrubber buff

About the PR
Scrubbers have been buffed from 200 L/s to 4000 L/s. This value shouldn't be final (i.e. gas leaks possibly not being a problem), but it's a decent start.
I believe that the value should be anywhere from 1000L/s to 4000L/s for single scrubbers - 1000L/s because of 5 would-be scrubber layers and 4000L/s to mostly preserve atmos setups.
This PR can be closed freely should a more complete atmos balance rework happen.
Why / Balance
Response to #20917.
Technical details
Air alarm presets have been changed. Scrubbers have a higher maxTransferRate defined in their prototype.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
:cl: router

tweak: Scrubbers have been buffed to 4000L/s to account for slow radiators. This change is temporary, more balance feedback is needed for a proper scrubber balance level.

This value would make scrubbers 62 times stronger than their previous value, even stronger than if you were abusing the stacking bug in the first place. How on earth is this value reasonable?

even with 32 stacked scrubbers the scrubbers are the bottleneck, pumps are much faster

ok more pro idea what if plasteel scrubber that scrubs faster intended for gas production instead of station usage

ok more pro idea what if plasteel scrubber that scrubs faster intended for gas production instead of station usage

That would make some more sense, yes.

You know, it's perfectly possible to make tritium using passive vents in the chamber and a big bank of filters. I don't know why people insist that scrubbers are absolutely necessary.
An alternative to plasteel scrubbers is to lean into machine upgrades to upgrade scrubbers.

the best designs use 32x scrubbers and not filters + passive vents

Not 4000, 2500.

You know, it's perfectly possible to make tritium using passive vents in the chamber and a big bank of filters. I don't know why people insist that scrubbers are absolutely necessary.
An alternative to plasteel scrubbers is to lean into machine upgrades to upgrade scrubbers.

using passive vents has terrible efficiency and i don't think it's better than even unstackable scrubbers right now

The number to use is 2500. Not 4000, 2500.

valid

I'm closing this since everything is going to be buffed soon in https://github.com/space-wizards/space-station-14/pull/21954. Thanks for your initial contribution.


chore: upgrade to 0.6

What I did

fixes: APE-528
How I did it
How to verify it
Checklist

[ ] Passes all linting checks (pre-commit and CI jobs)
[ ] New test cases have been added and are passing
[ ] Documentation has been updated
[ ] PR title follows Conventional Commit standard (will be automatically included in the changelog)

see https://github.com/ApeWorX/ape-infura/pull/58


URL to “See the post” should be a direct URL to the comment by u/Mr-Molester

EXPECTED BEHAVIOR: The “See the post” button should take you directly to the Reddit comment where u/Mr-Molester announced he had bought the domain.
ACTUAL BEHAVIOR: Takes the user to the full post, which does not directly talk about this website.
STEPS TO REPRODUCE: Click the “See the post” button on the website.
The URL that should be linked: https://www.reddit.com/r/assholedesign/comments/8fmitr/comment/dy4x2gc?st=JGL9JEW2&sh=2e2fb312

Fixed it

Commited to main website.


Errot: RequestVehicles Error to get data for Vehicles : 'Service unavailable. Please try later from www.bmw-connecteddrive.com'!"

Hi,
I'm using your bmw-adapter (v2.0.0) but I did not get fresh data. in object "bmw.0._LetzerFehler" is the following value: "2021-08-07@14:31:06: "RequestVehicles Error to get data for Vehicles : 'Service unavailable. Please try later from www.bmw-connecteddrive.com'!""
The BMW-account is connected to a F11.
What can i do to solve issue?
Greetings
Thomas

Please delete alle states under bmw.0. and restart the adapter

Please delete alle states under bmw.0. and restart the adapter

no i got other errors in log:

here in plaint text:
{"error":"invalid_request","error_description":"The request is missing a required parameter, includes an unsupported parameter value (other than grant type), repeats a parameter, includes multiple credentials, utilizes more than one mechanism for authenticating the client, or is otherwise malformed"}`

Can you double check your passoword and login via app?

Can you double check your passoword and login via app?

do i have to use app or would website be okay too? App is crashing atm

Wait until the app is working again then the adapter is also working

thx, should it be App "My BMW" or "BMW connected"?

MyBmw

now connection is done :-)
But no any object about fuel any more? Many new with electric/charging
It is a BMW from 2016.
WIll Data like LastTrip will be filled soon?
Here are some errors left:

same problem here.

now connection is done :-)
But no any object about fuel any more? Many new with electric/charging
It is a BMW from 2016.
WIll Data like LastTrip will be filled soon?
Here are some errors left:

Hi, i have the same problen.

I need login credentials to check this contact me in the iobroker forum https://forum.iobroker.net/user/tombox

please install via github to try the new v2 api

please install via github to try the new v2 api

Hi,
I did update/install:
`$ iobroker url https://github.com/TA2k/ioBroker.bmw/tarball/master --host raspberrypi-iob
install TA2k/ioBroker.bmw#master
NPM version: 6.14.13
npm install TA2k/ioBroker.bmw#master --loglevel error --prefix "/opt/iobroker" (System call)
../authenticate_pam.cc: In function ‘void after_doing_auth(uv_work_t*, int)’:
../authenticate_pam.cc:107:87: warning: ‘v8::Localv8::Value Nan::MakeCallback(v8::Localv8::Object, v8::Localv8::Function, int, v8::Localv8::Value)’ is deprecated [-Wdeprecated-declarations] Nan::MakeCallback(Nan::GetCurrentContext()->Global(), Nan::New(m->callback), 1, args); ^In file included from ../authenticate_pam.cc:23:../../nan/nan.h:1026:46: note: declared here NAN_DEPRECATED inline v8::Localv8::Value MakeCallback( ^~~~~~~~~~~~../authenticate_pam.cc:107:87: warning: ‘v8::Localv8::Value Nan::MakeCallback(v8::Localv8::Object, v8::Localv8::Function, int, v8::Localv8::Value)’ is deprecated [-Wdeprecated-declarations] Nan::MakeCallback(Nan::GetCurrentContext()->Global(), Nan::New(m->callback), 1, args); ^
In file included from ../authenticate_pam.cc:23:../../nan/nan.h:1026:46: note: declared here NAN_DEPRECATED inline v8::Localv8::Value MakeCallback( ^~~~~~~~~~~~
../authenticate_pam.cc: In function ‘Nan::NAN_METHOD_RETURN_TYPE Authenticate(Nan::NAN_METHOD_ARGS_TYPE)’:../authenticate_pam.cc:147:83: error: no matching function for call to ‘v8::Array::Get(v8::Localv8::String)’ Local res = options->Get(Nan::New("serviceName").ToLocalChecked()); ^In file included from /home/iobroker/.cache/node-gyp/14.17.0/include/node/node.h:67,
from ../../nan/nan.h:56, from ../authenticate_pam.cc:23:/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3717:43: note: candidate: ‘v8::MaybeLocalv8::Value v8::Object::Get(v8::Localv8::Context, v8::Localv8::Value)’ V8_WARN_UNUSED_RESULT MaybeLocal Get(Local context, ^/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3717:43: note: candidate expects 2 arguments, 1 provided/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3720:43: note: candidate: ‘v8::MaybeLocalv8::Value v8::Object::Get(v8::Localv8::Context, uint32_t)’ V8_WARN_UNUSED_RESULT MaybeLocal Get(Local context, ^/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3720:43: note: candidate expects 2 arguments, 1 provided../authenticate_pam.cc:150:69: error: no matching function for call to ‘v8::String::WriteUtf8(char [128], unsigned int)’ serviceName->WriteUtf8(m->serviceName, sizeof(m->serviceName) - 1); ^
In file included from /home/iobroker/.cache/node-gyp/14.17.0/include/node/node.h:67, from ../../nan/nan.h:56, from ../authenticate_pam.cc:23:/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3037:7: note: candidate: ‘int v8::String::WriteUtf8(v8::Isolate*, char*, int, int*, int) const’ int WriteUtf8(Isolate* isolate, char* buffer, int length = -1, ^~~~~~~~~/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3037:7: note: no known conversion for argument 1 from ‘char [128]’ to ‘v8::Isolate*’../authenticate_pam.cc:152:69: error: no matching function for call to ‘v8::Array::Get(v8::Localv8::String)’ res = options->Get(Nan::New("remoteHost").ToLocalChecked()); ^
In file included from /home/iobroker/.cache/node-gyp/14.17.0/include/node/node.h:67, from ../../nan/nan.h:56, from ../authenticate_pam.cc:23:/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3717:43: note: candidate: ‘v8::MaybeLocalv8::Value v8::Object::Get(v8::Localv8::Context, v8::Localv8::Value)’ V8_WARN_UNUSED_RESULT MaybeLocal Get(Local context, ^/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3717:43: note: candidate expects 2 arguments, 1 provided/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3720:43: note: candidate: ‘v8::MaybeLocalv8::Value v8::Object::Get(v8::Localv8::Context, uint32_t)’ V8_WARN_UNUSED_RESULT MaybeLocal Get(Local context, ^/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3720:43: note: candidate expects 2 arguments, 1 provided../authenticate_pam.cc:155:66: error: no matching function for call to ‘v8::String::WriteUtf8(char [128], unsigned int)’ remoteHost->WriteUtf8(m->remoteHost, sizeof(m->remoteHost) - 1); ^In file included from /home/iobroker/.cache/node-gyp/14.17.0/include/node/node.h:67,
from ../../nan/nan.h:56, from ../authenticate_pam.cc:23:/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3037:7: note: candidate: ‘int v8::String::WriteUtf8(v8::Isolate*, char*, int, int*, int) const’ int WriteUtf8(Isolate* isolate, char* buffer, int length = -1, ^/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3037:7: note: no known conversion for argument 1 from ‘char [128]’ to ‘v8::Isolate*’../authenticate_pam.cc:160:58: error: no matching function for call to ‘v8::String::WriteUtf8(char [128], unsigned int)’ username->WriteUtf8(m->username, sizeof(m->username) - 1); ^In file included from /home/iobroker/.cache/node-gyp/14.17.0/include/node/node.h:67, from ../../nan/nan.h:56, from ../authenticate_pam.cc:23:/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3037:7: note: candidate: ‘int v8::String::WriteUtf8(v8::Isolate*, char*, int, int*, int) const’ int WriteUtf8(Isolate* isolate, char* buffer, int length = -1, ^
/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3037:7: note: no known conversion for argument 1 from ‘char [128]’ to ‘v8::Isolate*’../authenticate_pam.cc:161:58: error: no matching function for call to ‘v8::String::WriteUtf8(char [128], unsigned int)’ password->WriteUtf8(m->password, sizeof(m->password) - 1); ^In file included from /home/iobroker/.cache/node-gyp/14.17.0/include/node/node.h:67, from ../../nan/nan.h:56, from ../authenticate_pam.cc:23:/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3037:7: note: candidate: ‘int v8::String::WriteUtf8(v8::Isolate*, char*, int, int*, int) const’ int WriteUtf8(Isolate* isolate, char* buffer, int length = -1, ^~~~~/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3037:7: note: no known conversion for argument 1 from ‘char [128]’ to ‘v8::Isolate*’../authenticate_pam.cc: At global scope:../authenticate_pam.cc:170:11: error: variable or field ‘init’ declared void void init(Handle exports) { ^~../authenticate_pam.cc:170:11: error: ‘Handle’ was not declared in this scope
../authenticate_pam.cc:170:11: note: suggested alternative: ‘rand_r’ void init(Handle exports) { ^~~~~~ rand_r../authenticate_pam.cc:170:24: error: expected primary-expression before ‘>’ token void init(Handle exports) { ^../authenticate_pam.cc:170:26: error: ‘exports’ was not declared in this scope void init(Handle exports) { ^~~~~~~
In file included from ../../nan/nan.h:56, from ../authenticate_pam.cc:23:../authenticate_pam.cc:175:31: error: ‘init’ was not declared in this scope NODE_MODULE(authenticate_pam, init); ^~/home/iobroker/.cache/node-gyp/14.17.0/include/node/node.h:758:36: note: in definition of macro ‘NODE_MODULE_X’ (node::addon_register_func) (regfunc), \ ^../authenticate_pam.cc:175:1: note: in expansion of macro ‘NODE_MODULE’ NODE_MODULE(authenticate_pam, init); ^~~~~~~
../authenticate_pam.cc:175:31: note: suggested alternative: ‘int’ NODE_MODULE(authenticate_pam, init); ^~/home/iobroker/.cache/node-gyp/14.17.0/include/node/node.h:758:36: note: in definition of macro ‘NODE_MODULE_X’ (node::addon_register_func) (regfunc), \ ^../authenticate_pam.cc:175:1: note: in expansion of macro ‘NODE_MODULE’ NODE_MODULE(authenticate_pam, init); ^~~~~~~
make: *** [authenticate_pam.target.mk:109: Release/obj.target/authenticate_pam/authenticate_pam.o] Error 1
gyp ERR! build error gyp ERR! stack Error: make failed with exit code: 2gyp ERR! stack at ChildProcess.onExit (/usr/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:194:23)gyp ERR! stack at ChildProcess.emit (events.js:376:20)gyp ERR! stack at Process.ChildProcess._handle.onexit (internal/child_process.js:277:12)gyp ERR! System Linux 5.10.17-v7l+gyp ERR! command "/usr/bin/node" "/usr/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /opt/iobroker/node_modules/authenticate-pamgyp ERR! node -v v14.17.0gyp ERR! node-gyp -v v5.1.0gyp ERR! not ok
../src/serialport.cpp: In function ‘Nan::NAN_METHOD_RETURN_TYPE Open(Nan::NAN_METHOD_ARGS_TYPE)’:
../src/serialport.cpp:78:69: warning: cast between incompatible function types from ‘void ()(uv_work_t)’ {aka ‘void ()(uv_work_s)’} to ‘uv_after_work_cb’ {aka ‘void ()(uv_work_s, int)’} [-Wcast-function-type] uv_queue_work(uv_default_loop(), req, EIO_Open, (uv_after_work_cb)EIO_AfterOpen); ^~~~~~~~~~~~~
../src/serialport.cpp: In function ‘Nan::NAN_METHOD_RETURN_TYPE Update(Nan::NAN_METHOD_ARGS_TYPE)’:../src/serialport.cpp:135:71: warning: cast between incompatible function types from ‘void ()(uv_work_t)’ {aka ‘void ()(uv_work_s)’} to ‘uv_after_work_cb’ {aka ‘void ()(uv_work_s, int)’} [-Wcast-function-type] uv_queue_work(uv_default_loop(), req, EIO_Update, (uv_after_work_cb)EIO_AfterUpdate); ^~~~~~~~~~~~~~~
../src/serialport.cpp: In function ‘Nan::NAN_METHOD_RETURN_TYPE Close(Nan::NAN_METHOD_ARGS_TYPE)’:../src/serialport.cpp:175:70: warning: cast between incompatible function types from ‘void ()(uv_work_t)’ {aka ‘void ()(uv_work_s)’} to ‘uv_after_work_cb’ {aka ‘void ()(uv_work_s, int)’} [-Wcast-function-type] uv_queue_work(uv_default_loop(), req, EIO_Close, (uv_after_work_cb)EIO_AfterClose); ^~~~~~~~~~~~~~
../src/serialport.cpp: In function ‘Nan::NAN_METHOD_RETURN_TYPE Flush(Nan::NAN_METHOD_ARGS_TYPE)’:../src/serialport.cpp:215:70: warning: cast between incompatible function types from ‘void ()(uv_work_t)’ {aka ‘void ()(uv_work_s)’} to ‘uv_after_work_cb’ {aka ‘void ()(uv_work_s, int)’} [-Wcast-function-type] uv_queue_work(uv_default_loop(), req, EIO_Flush, (uv_after_work_cb)EIO_AfterFlush); ^~~~~~~~~~~~~~
../src/serialport.cpp: In function ‘Nan::NAN_METHOD_RETURN_TYPE Set(Nan::NAN_METHOD_ARGS_TYPE)’:
../src/serialport.cpp:270:68: warning: cast between incompatible function types from ‘void ()(uv_work_t)’ {aka ‘void ()(uv_work_s)’} to ‘uv_after_work_cb’ {aka ‘void ()(uv_work_s, int)’} [-Wcast-function-type] uv_queue_work(uv_default_loop(), req, EIO_Set, (uv_after_work_cb)EIO_AfterSet); ^~~~~~~~~~~~
../src/serialport.cpp: In function ‘Nan::NAN_METHOD_RETURN_TYPE Get(Nan::NAN_METHOD_ARGS_TYPE)’:../src/serialport.cpp:314:68: warning: cast between incompatible function types from ‘void ()(uv_work_t)’ {aka ‘void ()(uv_work_s)’} to ‘uv_after_work_cb’ {aka ‘void ()(uv_work_s, int)’} [-Wcast-function-type] uv_queue_work(uv_default_loop(), req, EIO_Get, (uv_after_work_cb)EIO_AfterGet); ^~~~~~~~~~~~
../src/serialport.cpp: In function ‘void EIO_AfterGet(uv_work_t*)’:../src/serialport.cpp:329:96: error: no matching function for call to ‘v8::Object::Set(v8::Localv8::String, Nan::imp::FactoryBasev8::Boolean::return_t)’ results->Set(Nan::Newv8::String("cts").ToLocalChecked(), Nan::Newv8::Boolean(data->cts)); ^In file included from /home/iobroker/.cache/node-gyp/14.17.0/include/node/node.h:67, from ../../../../../nan/nan.h:56, from ../src/./serialport.h:6, from ../src/serialport.cpp:1:/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3670:37: note: candidate: ‘v8::Maybe v8::Object::Set(v8::Localv8::Context, v8::Localv8::Value, v8::Localv8::Value)’ V8_WARN_UNUSED_RESULT Maybe Set(Local context, ^/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3670:37: note: candidate expects 3 arguments, 2 provided/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3673:37: note: candidate: ‘v8::Maybe v8::Object::Set(v8::Localv8::Context, uint32_t, v8::Localv8::Value)’ V8_WARN_UNUSED_RESULT Maybe Set(Local context, uint32_t index, ^/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3673:37: note: candidate expects 3 arguments, 2 provided../src/serialport.cpp:330:96: error: no matching function for call to ‘v8::Object::Set(v8::Localv8::String, Nan::imp::FactoryBasev8::Boolean::return_t)’ results->Set(Nan::Newv8::String("dsr").ToLocalChecked(), Nan::Newv8::Boolean(data->dsr)); ^In file included from /home/iobroker/.cache/node-gyp/14.17.0/include/node/node.h:67, from ../../../../../nan/nan.h:56, from ../src/./serialport.h:6, from ../src/serialport.cpp:1:/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3670:37: note: candidate: ‘v8::Maybe v8::Object::Set(v8::Localv8::Context, v8::Localv8::Value, v8::Localv8::Value)’ V8_WARN_UNUSED_RESULT Maybe Set(Local context, ^/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3670:37: note: candidate expects 3 arguments, 2 provided/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3673:37: note: candidate: ‘v8::Maybe v8::Object::Set(v8::Localv8::Context, uint32_t, v8::Localv8::Value)’ V8_WARN_UNUSED_RESULT Maybe Set(Local context, uint32_t index, ^/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3673:37: note: candidate expects 3 arguments, 2 provided../src/serialport.cpp:331:96: error: no matching function for call to ‘v8::Object::Set(v8::Localv8::String, Nan::imp::FactoryBasev8::Boolean::return_t)’ results->Set(Nan::Newv8::String("dcd").ToLocalChecked(), Nan::Newv8::Boolean(data->dcd)); ^In file included from /home/iobroker/.cache/node-gyp/14.17.0/include/node/node.h:67, from ../../../../../nan/nan.h:56, from ../src/./serialport.h:6, from ../src/serialport.cpp:1:/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3670:37: note: candidate: ‘v8::Maybe v8::Object::Set(v8::Localv8::Context, v8::Localv8::Value, v8::Localv8::Value)’ V8_WARN_UNUSED_RESULT Maybe Set(Local context, ^/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3670:37: note: candidate expects 3 arguments, 2 provided/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3673:37: note: candidate: ‘v8::Maybe v8::Object::Set(v8::Localv8::Context, uint32_t, v8::Localv8::Value)’ V8_WARN_UNUSED_RESULT Maybe Set(Local context, uint32_t index, ^/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3673:37: note: candidate expects 3 arguments, 2 provided
../src/serialport.cpp: In function ‘Nan::NAN_METHOD_RETURN_TYPE GetBaudRate(Nan::NAN_METHOD_ARGS_TYPE)’:../src/serialport.cpp:363:76: warning: cast between incompatible function types from ‘void ()(uv_work_t)’ {aka ‘void ()(uv_work_s)’} to ‘uv_after_work_cb’ {aka ‘void ()(uv_work_s, int)’} [-Wcast-function-type] uv_queue_work(uv_default_loop(), req, EIO_GetBaudRate, (uv_after_work_cb)EIO_AfterGetBaudRate); ^~~~~~~~~~~~~~~~~~~~
../src/serialport.cpp: In function ‘void EIO_AfterGetBaudRate(uv_work_t*)’:../src/serialport.cpp:378:106: error: no matching function for call to ‘v8::Object::Set(v8::Localv8::String, Nan::imp::IntegerFactoryv8::Integer::return_t)’ results->Set(Nan::Newv8::String("baudRate").ToLocalChecked(), Nan::Newv8::Integer(data->baudRate)); ^In file included from /home/iobroker/.cache/node-gyp/14.17.0/include/node/node.h:67, from ../../../../../nan/nan.h:56, from ../src/./serialport.h:6, from ../src/serialport.cpp:1:/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3670:37: note: candidate: ‘v8::Maybe v8::Object::Set(v8::Localv8::Context, v8::Localv8::Value, v8::Localv8::Value)’ V8_WARN_UNUSED_RESULT Maybe Set(Local context, ^/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3670:37: note: candidate expects 3 arguments, 2 provided/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3673:37: note: candidate: ‘v8::Maybe v8::Object::Set(v8::Localv8::Context, uint32_t, v8::Localv8::Value)’ V8_WARN_UNUSED_RESULT Maybe Set(Local context, uint32_t index, ^/home/iobroker/.cache/node-gyp/14.17.0/include/node/v8.h:3673:37: note: candidate expects 3 arguments, 2 provided
../src/serialport.cpp: In function ‘Nan::NAN_METHOD_RETURN_TYPE Drain(Nan::NAN_METHOD_ARGS_TYPE)’:../src/serialport.cpp:409:70: warning: cast between incompatible function types from ‘void ()(uv_work_t)’ {aka ‘void ()(uv_work_s)’} to ‘uv_after_work_cb’ {aka ‘void ()(uv_work_s, int)’} [-Wcast-function-type] uv_queue_work(uv_default_loop(), req, EIO_Drain, (uv_after_work_cb)EIO_AfterDrain); ^~~~~~~~~~~~~~
../src/serialport.cpp: At global scope:../src/serialport.cpp:430:28: warning: unnecessary parentheses in declaration of ‘ToParityEnum’ [-Wparentheses] SerialPortParity NAN_INLINE(ToParityEnum(const v8::Localv8::String& v8str)) { ^../src/serialport.cpp:449:30: warning: unnecessary parentheses in declaration of ‘ToStopBitEnum’ [-Wparentheses] SerialPortStopBits NAN_INLINE(ToStopBitEnum(double stopBits)) { ^
In file included from ../../../../../nan/nan.h:56, from ../src/./serialport.h:6, from ../src/serialport.cpp:1:/home/iobroker/.cache/node-gyp/14.17.0/include/node/node.h:758:43: warning: cast between incompatible function types from ‘void ()(Nan::ADDON_REGISTER_FUNCTION_ARGS_TYPE)’ {aka ‘void ()(v8::Localv8::Object)’} to ‘node::addon_register_func’ {aka ‘void ()(v8::Localv8::Object, v8::Localv8::Value, void)’} [-Wcast-function-type] (node::addon_register_func) (regfunc), \ ^/home/iobroker/.cache/node-gyp/14.17.0/include/node/node.h:792:3: note: in expansion of macro ‘NODE_MODULE_X’ NODE_MODULE_X(modname, regfunc, NULL, 0) // NOLINT (readability/null_usage) ^~~../src/serialport.cpp:483:1: note: in expansion of macro ‘NODE_MODULE’ NODE_MODULE(serialport, init); ^
make: *** [bindings.target.mk:112: Release/obj.target/bindings/src/serialport.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 2
gyp ERR! stack at ChildProcess.onExit (/usr/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:194:23)gyp ERR! stack at ChildProcess.emit (events.js:376:20)gyp ERR! stack at Process.ChildProcess._handle.onexit (internal/child_process.js:277:12)
gyp ERR! System Linux 5.10.17-v7l+gyp ERR! command "/usr/bin/node" "/usr/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"gyp ERR! cwd /opt/iobroker/node_modules/iobroker.mysensors/node_modules/@serialport/bindingsgyp ERR! node -v v14.17.0gyp ERR! node-gyp -v v5.1.0gyp ERR! not ok
upload [4] bmw.admin /opt/iobroker/node_modules/iobroker.bmw/admin/words.js words.js application/javascript
upload [3] bmw.admin /opt/iobroker/node_modules/iobroker.bmw/admin/style.css style.css text/css
upload [2] bmw.admin /opt/iobroker/node_modules/iobroker.bmw/admin/index_m.html index_m.html text/html
upload [1] bmw.admin /opt/iobroker/node_modules/iobroker.bmw/admin/bmw.png bmw.png image/png
upload [0] bmw.admin /opt/iobroker/node_modules/iobroker.bmw/admin/admin.d.ts admin.d.ts video/mp2t
Update "system.adapter.bmw.0"
Process exited with code 0`
Here are the logs:
`bmw.0
2021-08-16 14:56:35.368	error	{"error":{"code":2012,"description":"(SmartPhoneUtil-A-2012) Resource not found. No status available for VIN WBA....."}}
bmw.0
2021-08-16 14:56:35.367	error	Error: Request failed with status code 404
bmw.0
2021-08-16 14:56:35.366	error	https://b2vapi.bmwgroup.com/webapi/v1/user/vehicles/WB...../status?deviceTime=2021-08-16T14:56:00&dlat=0&dlon=0
bmw.0
2021-08-16 14:56:28.490	info	starting. Version 2.1.0 in /opt/iobroker/node_modules/iobroker.bmw, node: v14.17.0, js-controller: 3.3.15
bmw.0
2021-08-16 14:56:22.853	info	Terminated (ADAPTER_REQUESTED_TERMINATION): Without reason
bmw.0
2021-08-16 14:56:22.852	info	terminating
bmw.0
2021-08-16 14:56:22.850	info	Got terminate signal TERMINATE_YOURSELF`

With the latest version this should only appears at the start but you should have enough data about the car

With the latest version this should only appears at the start but you should have enough data about the car

yes thx. data is been updated/filled but no info about fuel and last trips so far.

contact me please via iobroker forum
https://forum.iobroker.net/user/tombox


ServiceMixin does not obey never_ options

https://github.com/poise/poise-service/blob/master/lib/poise_service/service_mixin.rb#L105
vs.
https://github.com/poise/poise-service/blob/master/lib/poise_service/service_providers/base.rb#L72
I ran into this, when trying to abuse poise-service by creating instantiated services with systemd. It now tries to start the<EMAIL_ADDRESS>which causes an error.
I'm separately handling the starting (and stopping, and whatever) of<EMAIL_ADDRESS>I tried setting never_start and others to true, but as seen above those aren't checked in the case of the mixin.

ServiceMixin only wraps the underlying poise_service resource. While those two bits of code look similar, in the base provider service_resource is an instance of Chef::Resource::Service while in the mixin it's a poise_service. So the first calls the second.
You would have to show your code for me to say what the actual issue is.

Thanks. I must have misread the trace yesterday.
It was a matter of properly introducing the never_ options.
Sorry for the noise.


Automate the execution of commands with Cake

Cake is a simple build tool that saves time by tracking command dependencies, enforcing static guarantees, and stealing work.
Here is an example of using a cakefile to build Go code. The Redox kernel has a cakefile too.
For this ticket, implement something that will reduce the size of the Quick Start Guide.

Cake looks pretty neat, thanks for sharing! Unfortunately, it's neither widely adopted nor idiomatic to node or React development - two requirements for any library included here. I'm also not sure that there is a problem to solve here with cake. The way this project is set up is pretty typical for node projects and is actually quite minimal already.


Not work in the 'Bootstrap Tabs and Pills'

Not work in the 'Bootstrap Tabs and Pills'
0.0

Hi. If the slider was instantiated inside a hidden tab or pill, it can't resize itself properly; you will need to programatically resize by using $('#my-slider').sliderPro('resize') it once the tab becomes visible.
Cheers

Still having the margin issue stated in #82
The margin is being applied inline by the script. I will check css to see if I can solve this.


fix: kill other running tasks on failure

This kills already running tasks as soon as an error is emitted by another task. Making it possible to interrupt concurrently running tasks on first failure.
Aside from the included unmocked test, you can also try it out by staging a linting error and changing the lint-staged configuration in this repo to .lintstagedrc.mjs with the following content:
export default {
  "*.js": "eslint",
  "*.{js,ts}": () => "npm run test",
}

... and running:
node ./bin/lint-staged.js
I have changed the precedence of the tags, since I think it would be confusing to users to see the signals when we consciously kill tasks. (With the previous order, they would have seen SIGTERM. This way, it is KILLED.)
There are two minor caveats:

It doesn't work with shell: true.
For this to work with npm scripts (e.g. vanilla tsc, jest and probably anything that deals properly with its children is just fine), we need to kill all children ourselves. Which execa currently doesn't do for a good reason. However, I would argue that while this makes sense for an all purpose tool like execa, given lint-staged's use case, I can't imagine danger from killing child processes within milliseconds.

Fixes #594

Hello,
thanks for the PR. Currently we use listr2 for handling task lifecycle — I wonder if it has a built-in way to cancel parallel tasks on failure. Also, execa can use an AbortController signal, would it possible to use that somehow?

I believe we are intentionally disabling this error for parallel tasks, and only enabling it for subtasks (that run serially): https://github.com/okonet/lint-staged/blob/531275cbdb40fa9ec7ee972ee129e6323b9ab9ee/lib/runAll.js#L143-L149

I believe we are intentionally disabling this error for parallel tasks

That was the first thing I tried. It just marks the task as interrupted in the terminal but doesn't actually exit.

I wonder if it has a built-in way to cancel parallel tasks on failure

Like you mention - it's just the task life cycle. We manage what those tasks actually do in resolveTaskFn.js with execa.

Also, execa can use an AbortController signal, would it possible to use that somehow?

That would work as well but...

It is only supported starting from node 15. They claim that it was back ported to 14 but using 14.19.0, I get: ReferenceError: AbortController is not defined
I'm not sure whether it is reliable when it comes to killing children. While it does seem to work with both npm and yarn at the moment, given that neither of them used to pass along signals to their children, I don't know why this works all of a sudden and whether it will continue to work in the future.

It is a more elegant solution though, so if you were to argue that I should check for the existence of AbortController and we will gamble on the fact that it will continue to work with children, I'm happy to make those changes.

Thanks for the explanation! Just to make sure, does this kill all tasks or just the subtasks that we currently set to fail on error? We should still sync those settings.

Looks like Node.js 12 Windows tests failed, random?

That said, I believe this might have to do with the fact that I didn't mock pidtree and maybe it tries to get something like pid 0 and that succeeds on nix systems but not on windows.
I'll obviously mock that asap, wouldn't want to try killing random things when running tests. 😅

Our Windows tests could always use more love and attention, thanks.

Just ran this with #1118 merged in locally in my Windows 11 environment and at least here, everything passes now. 👍

It's probably not related to this, but I'm getting double-output from terminal:
❯ node bin/lint-staged.js
✔ Preparing lint-staged...
✔ Hiding unstaged changes to partially staged files...
⚠ Running tasks for staged files...
  ❯ .lintstagedrc.mjs — 1 file
    ❯ *.mjs — 1 file
      ✖ prettier --list-different [FAILED]
    ❯ *.{mjs,js,ts} — 1 file
      ✖ npm run test
↓ Skipped because of errors from tasks. [SKIPPED]
↓ Skipped because of errors from tasks. [SKIPPED]
✔ Reverting to original state because of errors...
✔ Cleaning up temporary files...

✖ prettier --list-different:
.lintstagedrc.mjs
✔ Preparing lint-staged...
✔ Hiding unstaged changes to partially staged files...
⚠ Running tasks for staged files...
  ❯ .lintstagedrc.mjs — 1 file
    ❯ *.mjs — 1 file
      ✖ prettier --list-different [FAILED]
    ❯ *.{mjs,js,ts} — 1 file
      ✖ npm run test [KILLED]
↓ Skipped because of errors from tasks. [SKIPPED]
↓ Skipped because of errors from tasks. [SKIPPED]
✔ Reverting to original state because of errors...
✔ Cleaning up temporary files...

It's probably not related to this, but I'm getting double-output from terminal

I noticed that too and double-checked with previous master, where I saw it happen as well (no fancy options - just running default concurrently and waiting for tests to finish running). I figured that you might do this on purpose, so that the final output is always the list of tasks, even when there are potentially long error logs.

Great work!

Thanks! It's been a pleasure working with you on this. 🙂

It might be that we print the task error output too soon, and listr2 does some final update after that, producing the duplicate.


Performance Discrepancy in Bi-Directional Sync: Google to Outlook vs Outlook to Google

Hi @MShekow ,
I've set up two flows for bi-directional synchronization between Google Calendar and Outlook Calendar:

Outlook to Google
Google to Outlook

While the setup works as intended, I've noticed a significant performance difference between the two flows:

Outlook to Google: Average runtime is 15 seconds.

Google to Outlook: Average runtime is 1:31 minutes.

Observations

Both flows use the same NG logic for synchronization.
The delay in the Google to Outlook flow occurs consistently, regardless of the number of events being synchronized or the specific conditions applied.

Question
Do you have any idea why the Google to Outlook flow takes so much longer to execute compared to the Outlook to Google flow? Is there something about the API, throttling, or potential processing differences between the platforms that could explain this discrepancy?
Looking forward to your insights!

Well, just looking at the total duration won't allow you to diagnose the problem. Just open one of the runs and look at the durations of the individual PA actions to find out what part of the flow is slow.

I added an extra condition check before creating or updating events. However, I noticed it runs through 73 loops, taking about 0.8–0.9 seconds per loop. This explains why it’s slower compared to the Outlook-to-Google sync. Initially, syncing from Outlook to Google took 1:20 minutes for 84 events. The Outlook-to-Google process seems more efficient, as it only needs 1–6 loops, whereas the Google-to-Outlook process repeatedly goes through 70–80 loops.

In this case, the most likely reason is that Outlook is retroactively changing the description of the event.
Example: a Google event has a description: Some description, the flow syncs this description to Outlook, but then the Outlook platform changes the description internally to Some description.
The only workaround is to set the Setting: ignore description comparison to true. However, you'll have to live with the fact that real updates that are made to the Google events after they have been synchronized will no longer be synced to Outlook.
Believe me, I've tried to make this work in a better way, e.g. by parsing the HTML, converting it to text, and comparing it, but there were more and more edge cases which I couldn't solve, so I gave up and built this simplified solution.

Thank you for the detailed explanation, your approach makes a lot of sense, especially given how tricky it can be to account for all the platform-induced quirks with descriptions. I completely understand why you opted for a simplified solution after encountering so many edge cases.
That said, your response got me thinking though, and I wanted to run an idea by you to see if this was something you’ve already tested before I end up going down another rabbit hole myself.
The idea is to implement a lightweight description normalization process rather than full HTML parsing. This would involve:

Cleanup simple expressions to strip redundant tags (like ,  or inline styles) without diving into deeper HTML structures.
For cases where the HTML differs but the core content is the same, flattening the description to plain text for a more straightforward comparison.
Generating a hash of the cleaned description for both Google and Outlook events and only syncing when the hashes differ.

I realize this approach still comes with challenges, and you may have already explored something similar. But if this idea wasn’t tested, I thought it might offer a middle ground to reduce unnecessary loops without abandoning description updates entirely.
I’d love to hear your thoughts on whether this might be worth exploring further or if it’s likely to hit the same roadblocks you’ve already encountered.

You'd have to think about where to store the hash, so that's one problem. And the other problem, simply put: "concept sounds easy on paper, extremely hard in practice". For instance, just throwing away HTML tags is not an option, you'll have newline and spacing issues.
I won't go into that rabbit hole again, I've just burned too much time in that already. You can always implement your own solution.

I can accept the duration time. It's not like we're talking about a lot. Was just curious, let's close this "issue" 😀


Update the xDS feature list for CSDS Python

As title.

Can we close this in favor of https://github.com/grpc/grpc/pull/26343/. Updated Java patch version too in that PR.

Closing.


Updates needed to workflows

[ ] Update workflow to save deltas here
[ ] Document how we are creating the bulk datasets (Makefile) and live datasets (update script)
[ ] Extra timesteps: are these added by the rolling window cleaner? Possibly related to #30
[ ] Merge six month datasets

Extra timesteps: this does not seem to show up for the live dataset. Need to look again at what is happening for the bulk dataset


How to downlaod a file to specific directory in pyhon?

How to downlaod a file to specific directory in python code
?

url = 'https://drive.google.com/uc?id=0B9P1L--7Wd2vNm9zMTJWOGxobkU'

# 1) pass filename
output = '/tmp/spam/20150428_collected_images.tgz'
gdown.download(url, output, quiet=False)

# 2) pass dirname (filename is retrieved from gdrive)
output = '/tmp/spam/'
gdown.download(url, output, quiet=False)

# 1) pass filename
output = '/tmp/spam/20150428_collected_images.tgz'
gdown.download(url, output, quiet=False)

Yes, I tried the above code.
But It made an error.
So I can't download to specific directory.
I'll try again.

url = 'https://drive.google.com/uc?id=0B9P1L--7Wd2vNm9zMTJWOGxobkU'

# 1) pass filename
output = '/tmp/spam/20150428_collected_images.tgz'
gdown.download(url, output, quiet=False)

# 2) pass dirname (filename is retrieved from gdrive)
output = '/tmp/spam/'
gdown.download(url, output, quiet=False)

@wkentaro this does not work, there is a bug.
steps to reproduce:
in a windows computer:
pip install -U gdown

url = 'https://drive.google.com/uc?id=0B9P1L--7Wd2vNm9zMTJWOGxobkU'

output = '/tmp/spam/'
gdown.download(url, output, quiet=False)

it will give error.
reason:
in windows os.sep corresponds to \\ and this line fails:
https://github.com/wkentaro/gdown/blob/442411be2fe10d9045103212031e51c50a8366cd/gdown/download.py#L169
solution
use os.path.isdir(output) isntead of output.endswith(osp.sep)

This problem hasn't been solved yet. 👀

This problem hasn't been solved yet. 👀

Unfortunately @wkentaro does not seem to mantain this package anymore.


docs: Added Azure Pipelines sample in CI guide

This commit adds sample integration code for Azure Pipelines in the CI guide

Would it be possible @helderco to make the changes that you suggested so that we can get this merged?


new

test test::valid_collections ... FAILED
I can´t find the error.

and no slug in the meta.json file
I help you modify them. @MistaJJ


retry attempt configuration in blob download

When we are performing download, how can we configure retry? In the old versions of SDK I saw max_retry parameter, do we have something like that now?
I need to handle the network fluctuations on the client machine while attempting file download and avoid re-downloading of complete files again.
Azure-storage python SDK version: 0.33.0
python: 2.7
OS: linux

Hi @samrudh,
In v0.33.0, retry parameters for our create and get functions were removed in favor of client-level retry policies. Take a look at this sample which covers retry usage for blobs.


Refactor S2N_BLOB_FROM_HEX to create a s2n_blob on the stack

Issue # (if available):
#1225
Description of changes:
Update S2N_BLOB_FROM_HEX that produces a s2n_blob with a hex string content without the need to use s2n_stuffer and allocation on the heap. This also also removes the need for S2N_BLOB_FREE
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

:exclamation: No coverage uploaded for pull request head (s2n_hex_blob@0bafb5d). Click here to learn what that means.
The diff coverage is n/a.

thanks for the comments @colmmacc. They should all be addressed in my rewrite of s2n_hex_string_to_bytes() and added tests for s2n_blob.


Can not call API to Meteor GraphQL

I setup GraphQL in Meteor server, and I created a simple API. I can not call to this api. Please help.
This is API setup:

This is command line to call api:
curl -X POST http://localhost:3000/api/status
This is result:

I fixed it. The reason is Meteor server and GraphQL are running in the same port (port 3000). We need to change port of GraphQL to other port (for example: 3002, etc). Close this issue.


bugfix: add judge for whether pidfile path is given

Signed-off-by: Ace-Tang<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
save pid into pidfile only when pidfile path is given, make code more reasonable.
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1374   +/-   ##
=======================================
  Coverage   17.29%   17.29%
=======================================
  Files         190      190
  Lines       11913    11913
=======================================
  Hits         2060     2060
  Misses       9706     9706
  Partials      147      147

@Ace-Tang please create PR base on branch 0.5.x, thanks a lot !!!

LGTM


[Intel MKL] Changes in common files to enable MKL Quantized ops with native format

This PR adds the common changes to enable MKL Quantized ops in native format mode. It removes the dependency on the duplicate number of inputs/outputs for quantized ops.
Added by @penpornk for API review:

The ops that are changed in API Golden files all have hidden visibility.
The attribute data_format is added because MKL ops support both TF's native NHWC format and its own blocked format.
The attributes is_weight_const and is_filter_const are added for caching purposes.
The changes should be backward compatible.

Thank you for reviewing the PR. I have made the requested changed.
For adding an attribute to the ops, I thought these ops are not visible in the API as we mark them as hidden, for e.g. here. These ops usually get used by frozen graphs generated by our offline tool.

@mahmoud-abuzaina Can you please check @penpornk's comments and keep us posted ? Thanks!

@penpornk no worries. This PR and other relative pending PRs don't add any new implementation to use the newly added attributes. Rather, the implementation that uses these attributes has been there for long time.
Here is the background: all Quantized* ops are hidden API ops and get generated/added to frozen pb file via offline tool. They were considered dummy ops. At runtime those ops will be rewritten to corresponding _MklQuantized*. The purpose of _MklQuantized* is to handle the extra number of inputs and outputs that we used to require. With native format support, we are getting rid of all extra inputs and outputs. So I thought we can reuse Quantized* ops (that are usable only via oneDNN integration code) instead of creating new ops. While doing that I noticed that these ops are missing few required attributes. These attributes have been used for a while with existing implementation, but the missing attributes were not causing a failure before because Quantized* ops were dummy ops and will be always rewritten to _Mkl ones which have the those attributes. Now since we are using Quantized* named ops, we are adding the missing attributes but they will use existing implementation. As an example of existing attribute usage see here. Sorry for my verbosity and please let me if I am missing something.

@penpornk no worries. This PR and other relative pending PRs don't add any new implementation to use the newly added attributes. Rather, the implementation that uses these attributes has been there for long time.
Here is the background: all Quantized* ops are hidden API ops and get generated/added to frozen pb file via offline tool. They were considered dummy ops. At runtime those ops will be rewritten to corresponding _MklQuantized*. The purpose of _MklQuantized* is to handle the extra number of inputs and outputs that we used to require. With native format support, we are getting rid of all extra inputs and outputs. So I thought we can reuse Quantized* ops (that are usable only via oneDNN integration code) instead of creating new ops. While doing that I noticed that these ops are missing few required attributes. These attributes have been used for a while with existing implementation, but the missing attributes were not causing a failure before because Quantized* ops were dummy ops and will be always rewritten to _Mkl ones which have the those attributes. Now since we are using Quantized* named ops, we are adding the missing attributes but they will use existing implementation. As an example of existing attribute usage see here. Sorry for my verbosity and please let me if I am missing something.

Closing for now. I will do some changes and reopen once ready.

@mahmoud-abuzaina So sorry for my delayed reply and thank you very much for the thorough explanation! I didn't remember that these attributes are already used in the already checked-in kernels.
Re: Hidden ops: The API review committee reminded me that op names not prefixed by _ are still callable. Marking them hidden just puts them in tf.raw_ops, so they are still part of the API golden.


fix #添加账户及id MoYan;test=document_fix

hi,添加账号momozi1996/MoYan,请审核是否有问题

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


Correct link text misspell

Update service-fabric-quickstart-java-spring-boot.md

"preqrequisites" link text should be "prerequisites"

@vedipen : Thanks for your contribution! The author, @suhuruli, has been notified to review your proposed change.

Merging per Angela's approval in email.


Only provide "SPRING_DATASOURCE_URL" environment variable when using username & password as auth type

Only provide "SPRING_DATASOURCE_URL" environment variable when using username & password as auth type.
Here is the screenshot when using service connector:

The  "SPRING_DATASOURCE_URL" environment variable is duplicated.

Hi, @saragluna , @haoozhang .
Please help to review this PR.


[stable/prometheus-operator] Add a Changelog.md

Hi,
I'm using the prometheus-operator and it is great. Thanks for the great work. However, the update process can be a bit painful given there is no Changelog. There is now an upgrade path in the Readme.md and a request for this feature was previously opened here: https://github.com/helm/charts/issues/10669.
Is your feature request related to a problem? Please describe.
The number of releases has increased a bit since and the versions are not specified in the commit message making it hard to filter the features and bug fixes especially when coming from an old version.
Describe the solution you'd like
I'd love to see a Changelog to ease this process and I'd even be willing to contribute one if someone has the time to review it before getting it integrated. I would also need help regarding the integration process since that would be my first time working in this repository.
Describe alternatives you've considered
I am currently trying to implement a local Changelog to keep track of our versions but it is not in sync with the repository and it feels like the community could benefit from that work.
Additional context
None.

so is there any plans to add changelog?


[SPARK-7482][SparkR] Rename some DataFrame API methods in SparkR to match their counterparts in Scala.

Merged build triggered.

Merged build started.

Test build #32225 has started for   PR 6007 at commit e861dc1.

Test build #32225 has finished for   PR 6007 at commit e861dc1.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

@sun-rui - Thanks for the PR. However I am having second thoughts about some of the renames. For some functions like filter and I guess to a certain extent sample its probably fine to overload as they are less often used functions.
However load and save are probably more widely used functions and our functions actually do something different from the load and save in R. One idea might be to rename these to read or read.df as these functions typically load data from external formats (like read.csv or read.table etc.). Any thoughts ?
cc @rxin

@shivaram, I think it is OK to rename APIs to ones that R user are accustomed to, for example, read.df and write.df. It would be better if there are more feedback from R community.

Yeah lets go with read.df and write.dffor now - We can also just alias these to loadDF and saveDF. We should revisit this when we graduate SparkR from experimental status and see if we have any user feedback

Build triggered.

Build started.

Test build #32465 has started for   PR 6007 at commit 9967c0f.

Test build #32465 has finished for   PR 6007 at commit 9967c0f.

This patch fails Spark unit tests.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test FAILed.

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

@sun-rui Could you bring this up to date with master ?

Jenkins, retest this please

Build triggered.

Build started.

Test build #32506 has started for   PR 6007 at commit 9967c0f.

Test build #32506 has finished for   PR 6007 at commit 9967c0f.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds no public classes.

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

Build finished. Test PASSed.

rebased to master.

Merged build triggered.

Merged build started.

Test build #32582 has started for   PR 6007 at commit 5c5cf5e.

Thanks for the update. LGTM. Will wait for Jenkins and then merge

Test build #32582 has finished for   PR 6007 at commit 5c5cf5e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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