build(deps): Updated deps to the latest version

Drop standard-version as it is now deprecated
Update mocha binaray name in test scripts

After these changes running npm audit returns:
$ npm audit
# npm audit report

ssh2  <1.4.0
Severity: high
OS Command Injection in ssh2 - https://github.com/advisories/GHSA-652h-xwhf-q4h6
fix available via `npm audit fix --force`
Will install<EMAIL_ADDRESS>which is a breaking change
node_modules/ssh2

1 high severity vulnerability

To address all issues (including breaking changes), run:
  npm audit fix --force

Still need to investigate possible breaking changes in ssh2 1.x.x in #338

Bumping to newer version of eslint-config-airbnb-base introduced some new rules and lint errors: https://github.com/IBM/nodejs-itoolkit/actions/runs/3317036527/jobs/5479472077

For the changes in https://github.com/IBM/nodejs-itoolkit/pull/347/commits/0eb4b89124c8bc339f78cd9f7a4275da600e723b I ran:
./node_modules/eslint/bin/eslint.js lib/ test/ --fix

Then manually disabled the following rules in the appropriate files
js /* eslint-disable default-param-last */ /* eslint-disable import/no-unresolved */

Ran integration tests manually:
> ./node_modules/mocha/bin/mocha.js --timeout 5s test/functional/*.js

  CommandCall Functional Tests
-----------------------
transport: ssh
verbose: off
-----------------------
    CL command tests
      ✔ calls CL command (1451ms)
    SH command tests
      ✔ calls PASE shell command (2974ms)
    QSH command tests
      ✔ calls QSH command (3686ms)

  DataQueue Functional Tests
-----------------------
transport: ssh
verbose: off
-----------------------
executing system 'CHKOBJ OBJ(QSYS/NODETKTEST) OBJTYPE(*LIB)'
executing system 'CHKOBJ OBJ(QSYS/NODETKTEST) OBJTYPE(*LIB)'
    sendToDataQueue
iDataQueue deprecated As of v1.0, class 'iDataQueue' is deprecated and will be removed at a later time. test/functional/iDataQueueFunctional.js:46:18
iDataQueue deprecated As of v1.0, 'iDataQueue.sendToDataQueue()' is deprecated and will be removed at a later time. test/functional/iDataQueueFunctional.js:48:10
      ✔ sends data to specified DQ (1124ms)
    receiveFromDataQueue
iDataQueue deprecated As of v1.0, class 'iDataQueue' is deprecated and will be removed at a later time. test/functional/iDataQueueFunctional.js:60:18
iDataQueue deprecated As of v1.0, 'iDataQueue.receiveFromDataQueue()' is deprecated and will be removed at a later time. test/functional/iDataQueueFunctional.js:62:10
      ✔ receives data from specfied DQ (1108ms)
    clearDataQueue
iDataQueue deprecated As of v1.0, class 'iDataQueue' is deprecated and will be removed at a later time. test/functional/iDataQueueFunctional.js:74:18
iDataQueue deprecated As of v1.0, 'iDataQueue.clearDataQueue()' is deprecated and will be removed at a later time. test/functional/iDataQueueFunctional.js:76:10
      ✔ clears the specifed DQ (1182ms)

  iNetwork Functional Tests
-----------------------
transport: ssh
verbose: off
-----------------------
    getTCPIPAttr
iNetwork deprecated As of v1.0, class 'iNetwork' is deprecated and will be removed at a later time. test/functional/iNetworkFunctional.js:33:19
iNetwork deprecated As of v1.0, 'iNetwork.getTCPIPAttr()' is deprecated and will be removed at a later time. test/functional/iNetworkFunctional.js:35:11
      ✔ retrieves TCP/IP Attributes (1084ms)
    getNetInterfaceData
iNetwork deprecated As of v1.0, class 'iNetwork' is deprecated and will be removed at a later time. test/functional/iNetworkFunctional.js:76:19
iNetwork deprecated As of v1.0, 'iNetwork.getNetInterfaceData()' is deprecated and will be removed at a later time. test/functional/iNetworkFunctional.js:78:11
      ✔ retrieves IPv4 network interface info (1138ms)

  iObj Functional Tests
-----------------------
transport: ssh
verbose: off
-----------------------
    retrUsrAuth
iObj deprecated As of v1.0, class 'iObj' is deprecated and will be removed at a later time. test/functional/iObjFunctional.js:33:19
iObj deprecated As of v1.0, 'iObj.retrUsrAuth()' is deprecated and will be removed at a later time. test/functional/iObjFunctional.js:35:11
      ✔ returns uses's authority for an object  (1071ms)
    rtrCmdInfo
iObj deprecated As of v1.0, class 'iObj' is deprecated and will be removed at a later time. test/functional/iObjFunctional.js:78:19
iObj deprecated As of v1.0, 'iObj.retrCmdInfo()' is deprecated and will be removed at a later time. test/functional/iObjFunctional.js:80:11
      ✔ returns command info (1157ms)
    retrPgmInfo
iObj deprecated As of v1.0, class 'iObj' is deprecated and will be removed at a later time. test/functional/iObjFunctional.js:131:19
iObj deprecated As of v1.0, 'iObj.retrPgmInfo()' is deprecated and will be removed at a later time. test/functional/iObjFunctional.js:133:11
      ✔ returns program info (1059ms)
    retrSrvPgmInfo
iObj deprecated As of v1.0, class 'iObj' is deprecated and will be removed at a later time. test/functional/iObjFunctional.js:206:19
iObj deprecated As of v1.0, 'iObj.retrSrvPgmInfo()' is deprecated and will be removed at a later time. test/functional/iObjFunctional.js:208:11
      ✔ returns service program info (1431ms)
    retrUserInfo
iObj deprecated As of v1.0, class 'iObj' is deprecated and will be removed at a later time. test/functional/iObjFunctional.js:262:19
iObj deprecated As of v1.0, 'iObj.retrUserInfo()' is deprecated and will be removed at a later time. test/functional/iObjFunctional.js:264:11
      ✔ returns specified user profile info (1095ms)
    retrUsrAuthToObj
iObj deprecated As of v1.0, class 'iObj' is deprecated and will be removed at a later time. test/functional/iObjFunctional.js:289:19
iObj deprecated As of v1.0, 'iObj.retrUserAuthToObj()' is deprecated and will be removed at a later time. test/functional/iObjFunctional.js:291:11
      ✔ retrieves info for users who are authorized to an object using ssh transpsort (1118ms)
    addToLibraryList
iObj deprecated As of v1.0, class 'iObj' is deprecated and will be removed at a later time. test/functional/iObjFunctional.js:317:19
iObj deprecated As of v1.0, 'iObj.addToLibraryList()' is deprecated and will be removed at a later time. test/functional/iObjFunctional.js:319:11
      ✔ appends lib to user's lib list (1044ms)

  iProd Functional Tests
-----------------------
transport: ssh
verbose: off
-----------------------
    getPTFInfo
iProd deprecated As of v1.0, class 'iProd' is deprecated and will be removed at a later time. test/functional/iProdFunctional.js:33:20
iProd deprecated As of v1.0, 'iProd.getPTFInfo()' is deprecated and will be removed at a later time test/functional/iProdFunctional.js:35:12
      ✔ returns info for specified ptf (1115ms)
    getProductInfo
iProd deprecated As of v1.0, class 'iProd' is deprecated and will be removed at a later time. test/functional/iProdFunctional.js:76:20
iProd deprecated As of v1.0, 'iProd.getProductInfo()' is deprecated and will be removed at a later time test/functional/iProdFunctional.js:78:12
      ✔ returns info for specified product (1117ms)
    getInstalledProducts
iProd deprecated As of v1.0, class 'iProd' is deprecated and will be removed at a later time. test/functional/iProdFunctional.js:109:20
iProd deprecated As of v1.0, 'iProd.getInstalledProducts()' is deprecated and will be removed at a later time test/functional/iProdFunctional.js:111:12
      ✔ returns info for installed products (1996ms)

  iSql Functional Tests
-----------------------
transport: ssh
verbose: off
-----------------------
    prepare & execute
iSql deprecated As of v1.0, class 'iSql' is deprecated. Use odbc, idb-connector, or idb-pconnector npm package instead. test/functional/iSqlFunctional.js:34:19
      ✔ prepares & executes stored procedure then fetch results (2432ms)
    addQuery & fetch
iSql deprecated As of v1.0, class 'iSql' is deprecated. Use odbc, idb-connector, or idb-pconnector npm package instead. test/functional/iSqlFunctional.js:68:19
      ✔ runs a query and fetches results (1330ms)
    added test to ensure issue #11 was resolved
iSql deprecated As of v1.0, class 'iSql' is deprecated. Use odbc, idb-connector, or idb-pconnector npm package instead. test/functional/iSqlFunctional.js:94:19
      ✔ should parse SQL result set empty data tags correctly (1330ms)
    tables
iSql deprecated As of v1.0, class 'iSql' is deprecated. Use odbc, idb-connector, or idb-pconnector npm package instead. test/functional/iSqlFunctional.js:121:19
      ✔ returns meta data for specified table (1433ms)
    tablePriv
iSql deprecated As of v1.0, class 'iSql' is deprecated. Use odbc, idb-connector, or idb-pconnector npm package instead. test/functional/iSqlFunctional.js:146:19
      ✔ returns privilege data for a table (1460ms)
    columns
iSql deprecated As of v1.0, class 'iSql' is deprecated. Use odbc, idb-connector, or idb-pconnector npm package instead. test/functional/iSqlFunctional.js:173:19
      ✔ returns meta data for a column (1590ms)
    columnPriv
iSql deprecated As of v1.0, class 'iSql' is deprecated. Use odbc, idb-connector, or idb-pconnector npm package instead. test/functional/iSqlFunctional.js:211:19
      ✔ returns privilege data for a column (1469ms)
    procedures
iSql deprecated As of v1.0, class 'iSql' is deprecated. Use odbc, idb-connector, or idb-pconnector npm package instead. test/functional/iSqlFunctional.js:240:19
      ✔ returns meta data on for a procedure (1424ms)
    pColumns
iSql deprecated As of v1.0, class 'iSql' is deprecated. Use odbc, idb-connector, or idb-pconnector npm package instead. test/functional/iSqlFunctional.js:268:19
      ✔ returns meta data for procedure column (1648ms)
    primaryKeys
iSql deprecated As of v1.0, class 'iSql' is deprecated. Use odbc, idb-connector, or idb-pconnector npm package instead. test/functional/iSqlFunctional.js:307:19
      ✔ returns meta data for a primary key (1517ms)
    foreignKeys
iSql deprecated As of v1.0, class 'iSql' is deprecated. Use odbc, idb-connector, or idb-pconnector npm package instead. test/functional/iSqlFunctional.js:333:19
      ✔ returns meta data for a foreign key (1435ms)
    statistics
iSql deprecated As of v1.0, class 'iSql' is deprecated. Use odbc, idb-connector, or idb-pconnector npm package instead. test/functional/iSqlFunctional.js:368:19
      ✔ returns stats info for table (2964ms)
    special
      - returns meta data for special columns
    rowCount
      - returns the number of rows affected by statement

  UserSpace Functional Tests
-----------------------
transport: ssh
verbose: off
-----------------------
    createUserSpace
iUserSpace deprecated As of v1.0, class 'iUserSpace' is deprecated and will be removed at a later time. test/functional/iUserSpaceFunctional.js:51:25
iUserSpace deprecated As of v1.0, 'iUserSpace.createUserSpace()' is deprecated and will be removed at a later time. test/functional/iUserSpaceFunctional.js:57:17
      ✔ creates a user space (1218ms)
    setUserSpaceData
iUserSpace deprecated As of v1.0, class 'iUserSpace' is deprecated and will be removed at a later time. test/functional/iUserSpaceFunctional.js:82:25
iUserSpace deprecated As of v1.0, 'iUserSpace.setUserSpaceData()' is deprecated and will be removed at a later time. test/functional/iUserSpaceFunctional.js:86:17
      ✔ sets data within the user space (1127ms)
    getUserSpaceData
iUserSpace deprecated As of v1.0, class 'iUserSpace' is deprecated and will be removed at a later time. test/functional/iUserSpaceFunctional.js:108:25
iUserSpace deprecated As of v1.0, 'iUserSpace.getUserSpaceData()' is deprecated ad will be removed at a later time. test/functional/iUserSpaceFunctional.js:110:17
      ✔ returns specified length of data (1205ms)
    deleteUserSpace
iUserSpace deprecated As of v1.0, class 'iUserSpace' is deprecated and will be removed at a later time. test/functional/iUserSpaceFunctional.js:125:25
iUserSpace deprecated As of v1.0, 'iUserSpace.deleteUserSpace()' is deprecated and will be removed at a later time test/functional/iUserSpaceFunctional.js:127:17
      ✔ removes a user space (1058ms)

  iWork Functional Tests
-----------------------
transport: ssh
verbose: off
-----------------------
    getSysValue
iWork deprecated As of v1.0, class 'iWork' is deprecated and will be removed at a later time. test/functional/iWorkFunctional.js:34:20
iWork deprecated As of v1.0, 'iWork.getSysValue()' is deprecated and will be removed at a later time. test/functional/iWorkFunctional.js:36:12
      ✔ returns the value of system variable (1086ms)
iWork deprecated As of v1.0, class 'iWork' is deprecated and will be removed at a later time. test/functional/iWorkFunctional.js:46:20
iWork deprecated As of v1.0, 'iWork.getSysValue()' is deprecated and will be removed at a later time. test/functional/iWorkFunctional.js:48:12
      ✔ returns an error when the specified system value is invalid
    getSysStatus
iWork deprecated As of v1.0, class 'iWork' is deprecated and will be removed at a later time. test/functional/iWorkFunctional.js:58:20
iWork deprecated As of v1.0, 'iWork.getSysStatus()' is deprecated and will be removed at a later time. test/functional/iWorkFunctional.js:60:12
      ✔ returns basic system status information about the signed-on users and batch jobs (1873ms)
    getSysStatusExt
iWork deprecated As of v1.0, class 'iWork' is deprecated and will be removed at a later time. test/functional/iWorkFunctional.js:88:20
iWork deprecated As of v1.0, 'iWork.getSysStatusExt()' is deprecated and will be removed at a later time. test/functional/iWorkFunctional.js:90:12
      ✔ returns more detailed system status info (2253ms)
    getJobStatus
iWork deprecated As of v1.0, class 'iWork' is deprecated and will be removed at a later time. test/functional/iWorkFunctional.js:133:20
iWork deprecated As of v1.0, 'iWork.getJobStatus()' is deprecated and will be removed at a later time. test/functional/iWorkFunctional.js:135:12
      ✔ returns status of specified job (1191ms)
    getJobInfo
iWork deprecated As of v1.0, class 'iWork' is deprecated and will be removed at a later time. test/functional/iWorkFunctional.js:149:20
iWork deprecated As of v1.0, 'iWork.getJobInfo()' is deprecated and will be removed at a later time. test/functional/iWorkFunctional.js:151:12
      ✔ returns info on specfed job (1151ms)
    getDataArea
executing system 'CHKOBJ OBJ(QSYS/NODETKTEST) OBJTYPE(*LIB)'
iWork deprecated As of v1.0, class 'iWork' is deprecated and will be removed at a later time. test/functional/iWorkFunctional.js:200:22
iWork deprecated As of v1.0, 'iWork.getDataArea()' is deprecated and will be removed at a later time. test/functional/iWorkFunctional.js:202:14
      ✔ returns contents of a data area (2636ms)

  ProgramCall Functional Tests
-----------------------
transport: ssh
verbose: off
-----------------------
    addParam
      ✔ calls QWCRSVAL program checks if it ran successfully (1126ms)
    addReturn
      - calls ZZVARY4 and checks the return value

  42 passing (1m)
  3 pending


SIGNINRESTRICTED blocks Teams 2.0 installation

You are missing a case where although the toggle should appear it doesn't.
Both below values appear fine:
enggCompleteFlag: true,
isAboveWin10Vibranium: true
But we are getting a
code: SIGNINRESTRICTED, reason: Device is Sign-in restricted.
We need to figure out which policy applies and does this.
Tue Mar 28 2023 11:03:14 GMT+0300 (Eastern European Summer Time) <26516> -- event -- eventpdclevel: 1, code: SIGNINRESTRICTED, reason: Device is Sign-in restricted, policyValue: false, status: success, scenario: 7203ddf0-8d13-43e3-8d5e-ee23c448763d, scenarioName: appswitcher_appstateservice_check, name: appswitcher_appstateservice_check, step: unknown, sequence: 1, delta: 1, scenarioDelta: 1, elapsed: 291, stepDelta: 1, Scenario.Mode: 1, AppInfo.Language: en-us, complianceEnvironmentType: 0, isDataCategorizationEnabled: true, userpdclevel: 0, processMemory: 23955764, freeMemory:<PHONE_NUMBER>, clientType: desktop, AppInfo.ClientType: desktop, Window.Focus: background, windowIsVisible: false, Window.Status: systray, UserInfo.TimeZone: +03:00, vdiMode: 0, previousVDIOptimizationValue: undefined, ppChannel: Production::MEC, distSrc: PROPLUS_O365ProPlusRetail,VisioProRetail,OneNoteFreeRetail, ppInstallMode: UPDATE, autoStartPolicy: 1, Scenario.Name: appswitcher_appstateservice_check, Scenario.Step: unknown, Scenario.Status: success,
Tue Mar 28 2023 11:03:14 GMT+0300 (Eastern European Summer Time) <26516> -- event -- eventpdclevel: 1, code: PEAR_APP_DISABLED, enggCompleteFlag: true, isAboveWin10Vibranium: true, policyCode: SIGNINRESTRICTED, returnValue: undefined, status: abandoned, scenario: 7203ddf0-8d13-43e3-8d5e-ee23c448763d, scenarioName: appswitcher_appstateservice_check, name: appswitcher_appstateservice_check, step: stop, sequence: 2, delta: 1, scenarioDelta: 1, elapsed: 291, stepDelta: 0, Scenario.Mode: 1, AppInfo.Language: en-us, complianceEnvironmentType: 0, isDataCategorizationEnabled: true, userpdclevel: 0, processMemory: 24001444, freeMemory:<PHONE_NUMBER>, clientType: desktop, AppInfo.ClientType: desktop, Window.Focus: background, windowIsVisible: false, Window.Status: systray, UserInfo.TimeZone: +03:00, vdiMode: 0, previousVDIOptimizationValue: undefined, ppChannel: Production::MEC, distSrc: PROPLUS_O365ProPlusRetail,VisioProRetail,OneNoteFreeRetail, ppInstallMode: UPDATE, autoStartPolicy: 1, Scenario.Name: appswitcher_appstateservice_check, Scenario.Step: stop, Scenario.Status: abandoned,

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

ID: 1d50c2e0-d6d3-ec3f-5e99-acaa2a6868d5
Version Independent ID: 1d50c2e0-d6d3-ec3f-5e99-acaa2a6868d5
Content: The new Microsoft Teams desktop client - Microsoft Teams
Content Source: Teams/new-teams-desktop-admin.md
Service: msteams
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

I found the root cause. Please update the document accordingly.
Tue Mar 28 2023 11:26:42 GMT+0300 (Eastern European Summer Time) <42452> -- event -- eventpdclevel: 1, code:
PEAR_APP_ENABLED, enggCompleteFlag: true, isAboveWin10Vibranium: true, policyCode: NON_BLOCKING, returnValue: true,
status: success, scenario: e6481b9a-4b9b-4209-96f9-da5eb00556b5, scenarioName: appswitcher_appstateservice_check, name:
appswitcher_appstateservice_check, step: stop, sequence: 2, delta: 1, scenarioDelta: 1, elapsed: 113, stepDelta: 0,
Scenario.Mode: 1, AppInfo.Language: en-US, complianceEnvironmentType: 0, isDataCategorizationEnabled: true, userpdclevel:
0, processMemory: 27654132, freeMemory:<PHONE_NUMBER>, clientType: desktop, AppInfo.ClientType: desktop, UserInfo.TimeZone:
+03:00, vdiMode: 0, previousVDIOptimizationValue: undefined, ppChannel: Production::MEC, distSrc:
PROPLUS_O365ProPlusRetail,VisioProRetail,OneNoteFreeRetail, ppInstallMode: UPDATE, autoStartPolicy: undefined,
Scenario.Name: appswitcher_appstateservice_check, Scenario.Step: stop, Scenario.Status: success,
We are using a Group Policy as per below:
Restrict sign in to Teams to accounts in specific tenants
https://admx.help/?Category=Office2016&Policy=Teams.Office.Microsoft.Policies.Windows::String_Teams_SignInRestriction_Policy
This blocks the upgrade. Is it by design or should it obey this setting?
I believe it should work even with this GPO in place.

I'm also seeing the same. It would be good to know if this is by design or not intended/will be fixed. Thanks!

I'm seeing the same. Here's my entries from logs.txt:
Wed Mar 29 2023 14:38:03 GMT+1100 (Australian Eastern Daylight Time) <13132> -- event -- eventpdclevel: 1, code: PEAR_APP_DISABLED, enggCompleteFlag: true, isAboveWin10Vibranium: true, policyCode: SIGNINRESTRICTED, returnValue: undefined, status: abandoned, scenario: [GUID REMOVED BY AUTHOR], scenarioName: appswitcher_appstateservice_check, name: appswitcher_appstateservice_check, step: stop, sequence: 2, delta: 1, scenarioDelta: 1, elapsed: 392, stepDelta: 0, Scenario.Mode: 1, AppInfo.Language: en-us, complianceEnvironmentType: 0, isDataCategorizationEnabled: true, userpdclevel: 1, processMemory: 23572004, freeMemory:<PHONE_NUMBER>, clientType: desktop, AppInfo.ClientType: desktop, Window.Focus: background, windowIsVisible: false, Window.Status: systray, UserInfo.TimeZone: +11:00, vdiMode: 0, previousVDIOptimizationValue: undefined, Scenario.Name: appswitcher_appstateservice_check, Scenario.Step: stop, Scenario.Status: abandoned,

It is the Policy restrictteamssignintoaccountsfromtenantlist.
The restriction to Teams to only allow a specific mandant

Confirming that root cause is the Policy restrictteamssignintoaccountsfromtenantlis which classifies the users as sign-in restricted users. Microsoft confirmed rollout excludes: EDU, VDI, Special Clouds, Mac, Web, Consumer, unsupported OS, and users with sign-in restrictions. Estimated time for sign-in enabled users is late May 2023.
More on the RestrictTeamsSignInToAccountsFromTenantList policy: https://learn.microsoft.com/en-us/microsoftteams/sign-in-teams#how-to-restrict-teams-sign-in-on-desktop-devices

This is addressed here: https://learn.microsoft.com/en-us/microsoftteams/new-teams-desktop-admin?tabs=teams-admin-center#policies-that-could-block-the-user-from-seeing-the-app-switcher-toggle
Policies that could block the user from seeing the App switcher toggle
The following list of policies can block users from seeing the app switcher toggle.

Your admin has sign in restrictions set up Learn more about how to restrict Teams sign in on desktop devices
If the user is on a VDI computer (Citrix, VMware etc.).
If the user is signed in to classic Teams with a Teams for Life account and a work account.
If the user is signed in to classic Teams with a Teams for Life account.
If you have an MSIX client.


Webview Api Prototype

#41047
Proposes a new Api for webviews. This API aims to make the current html preview api more usable, safer, and more extensible for the future.
Besides the new API, the new webviews also differ in several important ways:

They are no longer loaded in the file:// origin so they can't read files on disk (you have to use the new protocols for this)
Scripts are disabled by default
Command uris are disabled by default
postMessage inside of webviews now posts back to vscode directly (no more weird click faking required)

TODO

[ ] Discuss and finalize API
[ ] Add onFocus and onBlur to the webview object
[ ] See if implementation can be cleaned up
[ ] Add dispose method on webview
[ ] Add editorColumn property on webview
[ ] Make enableCommandUris option actually enable command uris
[ ] Add a new protocol for accessing a resource inside of a workspace folder (markdown loading a local image for example)
[ ] Finish moving markdown extension to use the new API

Moved work onto branch on master: #42690


Fix network ip drift

Description
This MR solves a drift issue introduced by backend changes which lead to an empty IP configuration for networks when polling the VM immediately after creation. To solve this we introduced a retry time-boxed mechanism which fetches the VM until the network config contains IP informations. To avoid additional drift due to link-local or VM internally provisioned IPs the provider now ignores IPs which were not expected by the plan.
Acceptance tests

[ ] Have you added an acceptance test for the functionality being added?

Output from acceptance testing:

$ make testacc TESTARGS='-run=TestAccAnxCloudVirtualServer'
...

Release Note
Release note for CHANGELOG:

* Handle incomplete network informations to avoid drift

References

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

/ok-to-test sha=9bb9800

/ok-to-test sha=d69fbc5

/ok-to-test sha=d69fbc5


crash of spyder and anaconda navigator when importing a project

Problem Description
What steps will reproduce the problem?

What is the expected output? What do you see instead?
Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE

Package Versions

Spyder:
Python:
Qt:
PyQt:
Operating System:

Dependencies

PASTE DEPENDENCIES HERE

There's no info at all in this issue, so I'm closing it, sorry. Please read our
https://github.com/spyder-ide/spyder/wiki/Troubleshooting-Guide-and-FAQ
to see if that helps.


function mostDeviant

The function mostDeviant is missing/not implemented. The implementation of this function in the original graphite seems to be very slow.

Should be easy to implement.

Fixed by https://github.com/dgryski/carbonapi/pull/70


Mdc with Cloud Gateway

Does MDC work with Cloud Gateway? I see in some stack overflow that we have to follow a different approach to use MDC for logging with Cloud Gateway. Please let us know if there a pattern to implement it.
@bclozel

I'm unsure what logging has to do with gateway specifically. Can you please describe the problem you are having?

@spencergibb , we are facing problem similar to this.
https://www.novatec-gmbh.de/en/blog/how-can-the-mdc-context-be-used-in-the-reactive-spring-applications/

Ok. I'm going to close it since it is not specific to gateway

It seems to work for me when MDC is populated in WebFilter and there's a lift operator applied like in novatec blog post


Oce/jincy

Added Bible dropdown in pageHeader.js file

Closing this to merge to oce/navigation instead of main...


Add support for include/exclude paths on schedule trigger

Having schedules trigger on a repo with more than one VS solution would be good to have the ability to configure the trigger including/excluding repo paths.
Right now only filter by branches is supported:
https://docs.microsoft.com/en-us/azure/devops/pipelines/process/scheduled-triggers?view=azure-devops&tabs=yaml

Hi, this doesn't look like an issue with a specific task. For general issues/questions about the service that don't have to do with a specific task, please open a ticket here - https://developercommunity.visualstudio.com/spaces/21/index.html


cbb: Improve spritemap.dat export speed

Also, don't use accessibility text of GtkPictures to save/restore data.
I haven't done much testing of this - but hopefully it works!

Tested it and it works great!


index_1->index

Received, but the content needs to be changed, and I have already made the changes,tks


[Draft PR] Adds Stream::le

Blocked by #268
For issue #129
Based on : https://github.com/rust-lang/rust/blob/master/src/libcore/iter/traits/iterator.rs#L2807

This was added in another PR.


Dismiss Parent Directories in Search Context Menu

In "Search", I can write click and Dismiss one file, but it would be much better If can choose parent directories of that file to be dismissed in the context menu as well
Right now

Feature Request (parents, to some configurable level)

This might not be the best UI for this, since the list would

Be growing by how nested the file could be.
Easily look like a wall of text.

To easily collapse things that are under the same directory, I'd advise to browse in tree mode from the search action buttons:

Otherwise, I'm open if there's a better way (UI-wise) to dismiss by directory.

I completely understand your concerns. I frequently find myself needing to utilize this missing feature repeatedly throughout the day.
One possible solution is to restrict the number of parent levels to either 2 or 3, as this is typically the situation in these scenarios.

That's a reasonable suggestion, but it feels a little like a small patch solution to a bigger problem where people want to dismiss whole directories. I still believe that viewing results in the tree view and dismissing the folders from there is the best experience if you want to do something like this currently.

I think what you proposed (Tree) makes sense so I will close this comment


Error occurred when executing BrushNetLoader

Hi!
The node installed without problems and the models are places as your instructions.
But when loading and running the BrushNet_SDXL_basic.json workflow gives me this error. Any tips? I'm running on a 3060 with 12gb VRAM.
Error occurred when executing BrushNetLoader:
CUDA error: operation not supported
CUDA kernel errors might be asynchronously reported at some other API call, so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
Compile with TORCH_USE_CUDA_DSA to enable device-side assertions.
File "F:\StabilityMatrix\Data\Packages\ComfyUI-dev\execution.py", line 151, in recursive_execute
output_data, output_ui = get_output_data(obj, input_data_all)
File "F:\StabilityMatrix\Data\Packages\ComfyUI-dev\execution.py", line 81, in get_output_data
return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
File "F:\StabilityMatrix\Data\Packages\ComfyUI-dev\execution.py", line 74, in map_node_over_list
results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
File "F:\StabilityMatrix\Data\Packages\ComfyUI-dev\custom_nodes\ComfyUI-BrushNet\brushnet_nodes.py", line 105, in brushnet_loading
brushnet_model = load_checkpoint_and_dispatch(
File "F:\StabilityMatrix\Data\Packages\ComfyUI-dev\venv\lib\site-packages\accelerate\big_modeling.py", line 598, in load_checkpoint_and_dispatch
device_map = infer_auto_device_map(
File "F:\StabilityMatrix\Data\Packages\ComfyUI-dev\venv\lib\site-packages\accelerate\utils\modeling.py", line 1121, in infer_auto_device_map
max_memory = get_max_memory(max_memory)
File "F:\StabilityMatrix\Data\Packages\ComfyUI-dev\venv\lib\site-packages\accelerate\utils\modeling.py", line 825, in get_max_memory
_ = torch.tensor([0], device=i)

Try to pass the --disable-cuda-malloc parameter to comfy when you start it.

--disable-cuda-malloc worked. Thanks!

I added "--disable-cuda-malloc" and it still didn't work, but I was able to use it yesterday and not this morning. Above isa screenshot of my console and interface log

I added "--disable-cuda-malloc" and it still didn't work, but I was able to use it yesterday and not this morning. Below isa screenshot of my console and interface log

My video card has more than 40 gigabytes of memory, and I don't think it's a problem with my video card

My graphics card has over 40GB of VRAM, so I don't think the issue is with my graphics card.

My graphics card has 80GB of VRAM, so I don't think it's an issue with my graphics card. I'm running it on GPU 0

No worries, a restart can solve my weird problem.


v2: Sitemaps are not always generated

🐛 Bug Report
sitemap.xml is not always generated. It might be caused by #2443 and #2496 does not fix the issue for sitemap generation.
Here is the code that writes the sitemap file.
https://github.com/facebook/docusaurus/blob/6935b3315e72503ebcac6925e038d195fa6979d1/packages/docusaurus-plugin-sitemap/src/index.ts#L38-L43
Writing the file is async, and the process might already be killed before it was able to write the file. To confirm my hypothesis, I changed writeFile to writeFileSync and the problem seems to go away with my blog repo.
Have you read the Contributing Guidelines on issues?
Yes.
To Reproduce
The bug is not reproducible on every repo. I cannot reproduce it on the docusaurus v2 project, but I managed to reliably reproduce it for my blog repo (at least on my machine).

Clone https://github.com/SamChou19815/website
yarn
yarn workspace blog build
Observe packages/blog/build/sitemap.xml The file is empty.

Expected behavior

sitemap.xml is reliably generated.
Actual Behavior

sitemap.xml is not reliably generated.
Your Environment

Docusaurus version used: v2-alpha.50
Environment name and version (e.g. Chrome 78.0.3904.108, Node.js 10.17.0): Node 12
Operating system and version (desktop or mobile): macOS 10.15.4

Reproducible Demo
(Paste the link to an example repo, including a siteConfig.js, and exact instructions to reproduce the issue.)

@yangshun so do we use synchronous write to sitemap file?

Let's do that if we aren't already
On Sat, Apr 4, 2020, 3:08 PM Alexey Pyltsyn<EMAIL_ADDRESS>wrote:

@yangshun https://github.com/yangshun so do we use synchronous write to
sitemap file?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/facebook/docusaurus/issues/2516#issuecomment-608987154,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKBCHILOAV5QJNUHNQQAPTRK3MFHANCNFSM4L3RLBLA
.

@SamChou19815 you don't want to do it?
сб, 4 апр. 2020 г. в 11:42, Yangshun Tay<EMAIL_ADDRESS>
Let's do that if we aren't already
On Sat, Apr 4, 2020, 3:08 PM Alexey Pyltsyn<EMAIL_ADDRESS>wrote:

@yangshun https://github.com/yangshun so do we use synchronous write
to
sitemap file?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/facebook/docusaurus/issues/2516#issuecomment-608987154
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAKBCHILOAV5QJNUHNQQAPTRK3MFHANCNFSM4L3RLBLA
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/facebook/docusaurus/issues/2516#issuecomment-608996740,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBUIOZ3BEVCN56ZI4FARDDRK3XGLANCNFSM4L3RLBLA
.

My concern is that the implementation detail of terminating the process has been leaked into the userland.
Let's imagine an example: suppose a user creates a plugin that generates a robots.txt file that includes all paths of docs but excludes everything else. If the user implements it in the same way we did currently (use writeFile with callback), then it will run into the same situation that the generation sometimes works and sometimes not.
I would be down to quickly switch writeFile to writeFileSync, but I think we need to think deeper about its broader implications.


Dropdown closes and reopens on keypress with remote dataset

You can see this with the best pictures example here
Start typing 'broadway' (pre 1960 example that is not prefetched). At 'bro' it will be isolated from prefetched data. then each additional letter ('a', 'd', ...) will close + reopen dropdown with same value.

The fix for this appears to have been lost in v 0.11. Can anyone confirm one way or the other?

I'm experiencing the same issue and would like an option to fix it very much. It looks bad when the dropdown is flickering between each keystroke.

@iMoses check out my fork here if you're in need of a quick fix: https://github.com/reverbdotcom/typeahead.js/commit/79b4f088e713e7d5f5156f5b8054164b87fd89af
I was hoping to get an answer here and get another pull merged in (one of #1224, #1233, or #1212) before opening a pull for this.

@joekur, Thanks! I was just starting to debug it, you saved me quite a lot of time :+1:
Please update the official code ASAP, it's the only true problematic behavior I came across of an otherwise excellent library which I really want to keep using :)

I'm having the same problem. Any hope for a fix?

I'm facing the same problem. Anyone found the solution yet?

Same issue here.

https://github.com/gaborbernat/typeahead.js/commit/6e8a91b05d7fa875101af09f26c71e9adf313a79

Any update on this. I tried a few of the unofficial fixes (which did not help), but would like an official fix\response, as this issue makes the control unusable in my opinion.

This project has apparently been abandoned. See https://github.com/corejavascript/typeahead.js where I believe many of these issues have already been resolved.

Thanks. I was unaware.


Route specific locals typings

Describe the problem
I have an app that has several areas such as /user, /(public), /admin etc.
For each of these areas I set different locals as the needs are different. However the type of locals, as far as i can tell, has to be as general as possible
I would like to tell Typescript "if you are on this route, then you can expect locals to hold this data".
That would reduce my boilerplate a lot, and make it easier to share data between both actions and load functions since It would be easier to populate certain data in the hooks.server.ts (if on a particular route/subtree). Currently I can choose between re-fetching data in actions (data that is loaded in a layout load but not available in actions), or I could load data in a function called by the hooks file (if matching a route) but then I would get a very general type with a lot of optional properties since the type of the object has to match all possible routes.
Describe the proposed solution
I am not sure as I am no typescript wizard, but I imagine that since route.id is already very well typed (i.e. in page file it is a literal string / const, and in layouts it is a union of such), it should be possible to create a route dependet Locals type that could also be part of the $types-family somehow. It cannot just match the current route id however, but routes beginning with /user etc.
I'm not sure where and how it should be configured, I imagine that there are people more suited for that. But perhaps just use the app.d.ts file, or we could also perhaps have a locals.d.ts file in such subfolders. I am not sure what would be best. But I guess that I would like to have a function in hooks.server handle that would make sure that all possibilities are exhausted and that I uphold the contract.
I understand that I still have to uphold my part of the contract in the hooks.server.ts file, making sure that I supply the right data so that the Locals are not lying. But that is already the case (if I type Locals in one way and populate it in hooks.server another way).
All of this should of course be opt in, so that a user that is completely fine with same locals everywhere do not have to bother, but it would be nice if you like me have a application with very well defined areas/subroutes.
Alternatives considered
I don't know, perhaps this is already supported but i have just not been able to create the right Locals type. I mean, it would be nicest if { locals } object in the load / actions would be just typed out of the box, if I have configured that way. But I could also use a type narrowing function which asserts the locals, if I could just figure out how to type it.
Importance
would make my life easier
Additional Information
I DID do a search for this, but I didn't find anything at first. But after looking some more, I found
Allow locals to be typed specifically for subpages of layouts #10194
 Make App.d.ts route id aware #11925
(and also https://github.com/sveltejs/kit/issues/9411)

closing as duplicate of https://github.com/sveltejs/kit/issues/11925


Add Product Showcase to Homepage

Based on previous discussion in #2625. Specifically, this issue covers this feature request:

Instead of the 10 items in "Jest's Testing Features" we should have screenshots and make it a product showcase. If we hold off on the website launch for another week, I can help do that.

These are the existing items under Jest's Testing Features:
[ ] Fast interactive mode with --watch.
[ ] Create coverage reports with --coverage. No additional setup or libraries needed!
[ ] Automatically find tests related to changed files to execute in your project with --onlyChanged.
[ ] Error messages are helpful and color coded. Stack traces point to the source of problems quickly.
[ ] Jest runs previously failed tests first. Together with --bail it provides useful signal quickly.
[ ] Sandboxed test files and automatic global state resets for every test.
[ ] Integrated support for testing with promises and async/await
[ ] Run your tests within a fake DOM implementation (via jsdom) on the command line.
[ ] Run tests in parallel processes to minimize test runtime.
[ ] Jest works with any compile-to-JS language and integrates seamlessly with Babel.
[ ] Integrated manual mocking library.
[ ] Can run asynchronous code synchronously.
Let's aim for 7 features to highlight in the homepage.

Ok, here is my shortlist after the three pillars that already mention important features (like the watch mode):

Inbuilt code coverage
"zero-config"
Powerful mocking library for functions and modules
Sandboxed test files
Browser mocks through jsdom
Works with any compile-to-JS language

Here's a preview of what I have so far. Copy still needs to be worked on, and we'll need screenshots to go along with each feature. At this point I'm focusing on getting the homepage layout to look right.

We could mix light and dark-themed terminals, just to show that it looks great on both worlds, what do you think?
Here's a screenshot for Snapshot example:

I think this one could also land somewhere, e.g. in "Sandboxed and Fast"

Great idea @thymikee. What I care about is having the same font and terminal size for those windows. If @hramos tells me exactly what screenshots he wants to have, I can provide :)

Consistency wins!

This is now in master, though we need to update the screenshots as these are currently placeholders. @cpojer can you create screenshots for the following? The placeholder I am using is at https://github.com/facebook/jest/blob/master/website/src/jest/img/content/feature-snapshot-tests.png - if you can make sure they're at least as big as the placeholder they should work great.

[ ] Sandboxed and fast: Sandboxed test files and automatic global state resets for every test. Jest parallelizes test runs across workers to maximize performance. Console messages are buffered and printed together with test results.

[ ] Built-in code coverage reports: Easily create code coverage reports using --coverage. No additional setup or libraries needed! Jest can collect code coverage information from entire projects, including untested files.

[ ] Zero configuration: Jest is already configured when you use create-react-app or react-native init to create your React and React Native projects. Place your tests in a tests folder, or name your test files with a .spec.js or .test.js extension. Whatever you prefer, Jest will find and run your tests.

[ ] Powerful mocking library: Powerful mocking library for functions and modules. Mock React Native components using jest-react-native.

[ ] Works with TypeScript: Jest works with any compile-to-JavaScript language and integrates seamlessly with Babel.

[ ] Simulated DOM environment: Run your tests within a fake DOM implementation on the command line. Every DOM API that you call can be observed in the same way it would be observed in a browser.


Add form to the app

This milestone implements the following

Use preprocessed HTML file with embedded Ruby code.
Use layouts and templates for shared content.
Create a method called current_user in your ApplicationController to make current user data available to all controllers.
It will return the first user from the database.
Don't worry about proper authentication at this time. You will add it in the upcoming projects.
Create forms to perform the following functions:
Creates a Post on behalf of the current_user (use the method that you created in your ApplicationController).
Create a comment on behalf of the current_user (use the method that you created in your ApplicationController).
Allow Users to add likes to Posts.

Thank
@.*** Implemented the changes you requested sir. I need your
response. thank you*
On Sun, 12 Nov 2023 at 00:12, Tazoh Yanick @.***> wrote:

@.**** requested changes on this pull request.
STATUS: CHANGES REQUIRED
Hi @mumbereh https://github.com/mumbereh ,
Good job so far! To Highlight! 👏 🟢
✔ Good commit messages
✔ Descriptive Readme file
✔ All methods have been added
✔ All form views added
There are some issues that you still need to work on to go to the next
project but you are almost there!
Required Changes ♻
Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop
the approval of this PR. However, I strongly recommend you to take them
into account as they can make your code better.
You can also consider:

N/A

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, remember to tag me in your question so I can receive the
notification.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
In app/controllers/posts_controller.rb
https://github.com/mumbereh/Blog_App/pull/7#discussion_r1390286239:

@@ -20,9 +19,29 @@ def like
end
end

def new

@post = @user.posts.build

end

def create

Great job working on this milestone. In our config/routes.rb, kindly
add a route for creating post.

In app/controllers/comments_controller.rb
https://github.com/mumbereh/Blog_App/pull/7#discussion_r1390286534:

end

private

def set_post

def new

In your config/routes.rb file, you should have a route for new
comments.
GET /posts/:post_id/comments/new(.:format) comments#new

—
Reply to this email directly, view it on GitHub
https://github.com/mumbereh/Blog_App/pull/7#pullrequestreview-1726126460,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZTLCE5IACIS6OBO3ZXJRLDYD7S2BAVCNFSM6AAAAAA7HP2HBKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTOMRWGEZDMNBWGA
.
You are receiving this because you were mentioned.Message ID:
@.***>


WIP: Making Alerter class more modular (Discord Intergration +  others)

Tried to make it more modular. Found it to work nicely with discord/slack.
I tried to make it as compatible with existing code as possible.
Extremely open to feedback.

Example CLI Usage
python.py run -a discord -w "https://webhook.url" --config=...
python.py run -a slack -w "https://webhook.url" --config=...
python.py run -a email -r="Relay IP Address -e="Email Recipient" --config=...

Just tried to test but running into a local issue. I git pull origin pull/34/head and docker build -t inventory-hunter . no errors.
I then try and run it using ./docker_run.bash -a discord -w "https://discord.com/api/webhooks/..." -c "config/rtx_3080.yaml" and get the error:
./docker_run.bash: illegal option -- a error: missing config argument usage: ./docker_run.bash -c CONFIG -e EMAIL -r RELAY
I'm seeing your updated code though so -a or --alerter= should be valid. First time I've used Docker and Python so I must be doing something wrong here.

This is really awesome! I'll pull in your branch later today and give it a try.

@alexgraham
Yeah, Looks like docker isn't really working. Didn't touch much on that one. But it worked great for me when replicating the environment in a normal python way.
Not too familiar with docker so hopefully someone can assist in this.

Issue was with docker_run.bash, hacked at it so it accepts the new arguments for -a -w
This works wonderfully, near instant alert with Discord. Thanks @ms7m

@ms7m Thanks for your work on this!
Just a heads up that I committed a small fix to a typo in the EmailAlerter (self.relat -> self.relay).


Indentation issue.

Hi!
I tried your plugin and everything was working pretty fine until I started lurking on the Yesod framework.
There is a problem with the very first example program given here.
The code indentation breaks after the line instance Yesod HelloWorld
Is there any chance to fix that?
Thanks in advance!

I tried to fix but I still know that auto indentation does not work well with Quasiquotations. In general auto indentation for Haskell is of much pain, I think this plugin can support while your coding.


Add a user template directory

Also check for templates in ~/.ano/templates/ so users can set up templates
of their own

Sorry but I'm ending development and support for Arturo as platformio is everything I wanted accomplish and more.


initial fastlane structure

As discussed here, this brings you an initial fastlane structure. Be welcome to update the description, replace screenshots, add a featureGraphic – and maybe even per-release changelogs :smile: Some (hopefully) helpful pointers:

my snippet describing the fastlane format
my wiki page on formatting the full_description.txt (using "Markdown lite" has it compatible with all places I know – and looks best with my repo which even understand full Markdown).

Enjoy – and thanks for providing your app as FOSS! Wish at least one of the apps it depends on would be maintained and available the same way…

As discussed here, this brings you an initial fastlane structure. Be welcome to update the description, replace screenshots, add a featureGraphic – and maybe even per-release changelogs 😄 Some (hopefully) helpful pointers:
* my snippet describing the [fastlane format](https://gitlab.com/-/snippets/1895688)

* my wiki page on [formatting the `full_description.txt`](https://gitlab.com/IzzyOnDroid/repo/-/wikis/Fastlane) (using "Markdown lite" has it compatible with all places I know – and looks best with my repo which even understand full Markdown).

Thanks! I'll check them out

Enjoy – and thanks for providing your app as FOSS!

❤

Wish at least one of the apps it depends on would be maintained and available the same way…

Yeah, it seems like the open source version won't won't get more updates in favor of the lite version.But I get it, theming and android modding in general isn't as popular as it used to. But I still wanted to get some of my Quick Settings space back 😊

Yeah. I also remember those days where modding was a thing. Using Android now for about 12 years. That first time unlocking a bootloader to flash a custom ROM, and the sweat drops on my forehead that I might brick the device :laughing: Those were the days. And now we see the end of repo.xposed.info (which is unreachable 99% of the time), sad.
So thanks for merging! With my repo, I've now enabled: shortdesc,fulldescMD,icon,screenshots. Should you add a new feature set (like a featureGraphic, video.txt or per-release changelogs) please let me know so I can check and enable them as well.No need to tell me if you replaced or added screenshots or a new locale, those things get picked up automatically.

So thanks for merging! With my repo, I've now enabled: shortdesc,fulldescMD,icon,screenshots. Should you add a new feature set (like a featureGraphic, video.txt or per-release changelogs) please let me know so I can check and enable them as well.No need to tell me if you replaced or added screenshots or a new locale, those things get picked up automatically.

Sure, thank you for taking the time and adding the app to your repo!


Broken links in README

Under the flake-parts section of the README, the "detailed walkthrough" link in the README is broken, and the linked real-world example no longer uses treefmt (as of https://github.com/srid/haskell-template/commit/85b7437665ebaa55911ccdddef576957ac1011b7).

Thanks!


[1.2] Settings are not being stored

Latest release has some issues:

The Close button doesn't work when trying to close the new Application Settings

Settings are not being stored correctly

APIs screen isn't populated with the default settings

The Close button doesn't work when trying to close the new Application Settings

Already fixed, I just need to push the changes.

Settings are not being stored correctly
APIs screen isn't populated with the default settings

The problem is with loading, because they are stored without problems. I forgot to remove a comment prior to pushing the merge on master.

All of this got fixed on 1.2.1, sorry for not closing the issue.


Installation instructions for the public repo

Hi, it seems like the installation instructions here reference installing with a wheel file that's not included.  Could you give some instructions on how to install specific to this repo?
Thanks!

I am sure there are inconsistencies in the documentation that we need to sort through so please let us know where you were reading where it wasn't clear. We will address this for sure.
Under the README.rst of the repository it has a section titled "Using a Binary Distribution". There are directions there for downloading a binary distribution and using. You can find the binary distribution under "Releases" on the right side of the repo github.com page. From there you can click through to the dragon-0.61-e8e93dd3.tar.gz file which can be downloaded and unzipped and untarrred using the commands in the "Using a Binary Distribution" section of the documentation. Please let us know if there is something more that we can do to help.

The README.rst file provides directions for building and installing the program. If you would prefer, you can download a binary distribution and then follow the alternative instructions for installing a binary. Both sets of instructions are pointed to in the README.rst file you find at the root of this distribution.


Upgrade  react-styleguidist@11

Supports TS out of the box, so we might not need anymore react-docgen-typescript.
See  👀
https://github.com/styleguidist/react-styleguidist/pull/1500
https://github.com/styleguidist/react-styleguidist/pull/1514/files

It will be present in the upcoming major version react-styleguidist@11

:tada: This issue has been resolved in version 2.12.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Effect editor overlay blocks the rest of the level when painting

The effect overlay becomes opaque and blocks the level view completely when painting the effect. The overlay appears to increase in opacity every time the mouse is moved from one geometry tile to another while holding LMB, eventually completely blocking the level view.

fixed with new rendering system in next 2.1 version


No attribute 'domain' in response

sudo letsencrypt certonly --agree-tos -a letsencrypt-route53:auth -d vpn.rededucation.com
Saving debug log to /var/log/letsencrypt/letsencrypt.log
Obtaining a new certificate
Performing the following challenges:
dns-01 challenge for vpn.rededucation.com
Found credentials in shared credentials file: ~/.aws/credentials
Cleaning up challenges
An unexpected error occurred:
AttributeError: 'DNS01Response' object has no attribute 'domain'
Please see the logfiles in /var/log/letsencrypt for more details... partial gist below
https://gist.github.com/rtfmoz/085400e4a33f8b488dd3f8fb0d7c00b4

Migrated to newer software


npm claims 0.3.8 is the newest version

See https://www.npmjs.com/package/grunt-scss-lint
My guess would be, it's because the package.json says the version 0.3.8.

Nope, it was me being a retard

Heh, I didn't even consider the tag could be wrong. Thank you for the quick response!


Binary size

Hello,
I was looking at the different binary size and I can't figure out why between 2 node version or 2 arch there is such a big size difference (10mb).
Is there some optimization put in place? Is it even possible to optimize anything here?

That's a known thing, seems nodejs versions are bigger on every major, you can try to add compression to optimize it

Yes I added compression from pkg but my final builds are never going under the 50mb, it's not a big deal I would say nowadays but I was looking to understand more on how it works.
Do you think there could be a way to make slim versions of node by excluding some uneeded parts of it, I imagine that it's a full featured node build but let's say (it's a crazy idea i know) you don't need access to the native crypto module, could it be removed from the final build?

@Mikescops I also use esbuild to create a super slim bundle, check https://github.com/zwave-js/zwave-js-ui/blob/master/esbuild.js

I know and I took inspiration from your nice work there, my esbuild outputs something like 1.3mb but after pkg i get 53mb and that's mostly because the nodejs binary something like 50mb itself.

Ok let's close this one.


[Linaro:ARM_CI] Update to new wheel target

Update the scripts to use the new target for building a wheel

@nitins17 Is there any action needed from me before this will be merged please?


Component Dependencies - Service Updates - Poor English

Hi,
I believe that the English on the Service Updates section in the Component Dependencies section is incorrect.
I think the wording is actually the wrong way round compared to the examples given
Current sentence and example
Commerce headquarters and finance and operations apps must be on the same version as, or an earlier version than, Commerce Scale Unit (both cloud and self-hosted).
For example, if Commerce headquarters and finance and operations apps are on version 10.0.29, Commerce Scale Unit must be on version 10.0.29 or earlier (for example, 10.0.28 or 10.0.27).
**Potential Correction **
If the logic is that F&O must be on the same or higher version that CSU (i.e., F&O version >= CSU) then the English should be:
"Commerce headquarters and finance and operations apps must be on the same or newer version than the Commerce Scale Unit (both cloud and self-hosted).
could use later rather than higher
Second Current sentence and example
Commerce Scale Unit must be on the same version as, or an earlier version than, Modern Point of Sale (POS), Hardware Station, and the Commerce software development kit (SDK) and associated local site configurations (such as modules, data actions, and themes).
For example, if Commerce Scale Unit is on version 10.0.29, Modern POS, Hardware Station, and the Commerce storefront must be on version 10.0.29 or earlier (for example, 10.0.28 or 10.0.27).
Potential Correction 2
If the logic is that CSU must be on the same or higher version than MPOS, Hardware Station, SDK (i.e., CSU >= MPOS, Hardware Station, SDK) then the English should be:
"Commerce Scale Unit must be on the same or a newer version than Modern Point of Sale (POS), Hardware Station, and the Commerce software development kit (SDK) and associated local site configurations (such as modules, data actions, and themes)."
could use later rather than higher

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

ID: 48746126-f146-6c6d-d8da-307f819896c6
Version Independent ID: 73cdd013-3569-b678-26f3-75679d84e4f6
Content: Dynamics 365 Commerce component versioning requirements - Commerce | Dynamics 365
Content Source: articles/commerce/arch-component-versioning.md
Service: dynamics-365-commerce
Product: ****
Technology: ****
GitHub Login: @Reza-Assadi
Microsoft Alias: rassadi

Bug 751968 has been logged to address this issue.

Thank you for your feedback. @scott-tucker can you please review the wording in the topic?

@josaw1, am I reviewing the wording in the comment from segibson77?  If so, then both potential corrections sound good to me.  Jarad owns deployment now, I'm not sure who owns the SDK now. When I view the content source it is still the original. That's why I am assuming I should review the comment.
Potential Correction 1
"Commerce headquarters and finance and operations apps must be on the same or newer version than the Commerce Scale Unit (both cloud and self-hosted)."
Potential Correction 2
"Commerce Scale Unit must be on the same or a newer version than Modern Point of Sale (POS), Hardware Station, and the Commerce software development kit (SDK) and associated local site configurations (such as modules, data actions, and themes)."

The corrections were made and will be published shortly.


Participants should be able to add/remove a media

Linked to https://github.com/kalisio/kEvent/issues/49

Closed by https://github.com/kalisio/kEvent/commit/9c983c867fc8bc3cba288249e3250f992f2cd217


Adding UART2 for CLI/BLE

The code will not compile since I changed the header file input parameters. I can change every call to my function to have the correct parameters since they will all be using USART 3. I am not sure if I should since the calls will be in other files.

I was wondering, should we configure the BSP and drivers so we can use them in both RTOS and bare metal code? Like putting #ifdef statements over RTOS specific code. If we're planning on having the RTOS as our final version, then any place where we use the fifo like the one in UART can be using the Micrium fifo library @ClarkPoon @manthanand

When you say bare-metal code, does that mean just without an RTOS? I thought our only RTOS specific code was the actual engine running the RTOS (and other stuff like semaphores). Is the micrium fifo library being used elsewhere in our code (like is it something that we are already using and should start using everywhere or is it something that should be used in an RTOS for FIFO's)?

The code will not compile since I changed the header file input parameters. I can change every call to my function to have the correct parameters since they will all be using USART 3. I am not sure if I should since the calls will be in other files.

Can you change this so it builds?

When you say bare-metal code, does that mean just without an RTOS? I thought our only RTOS specific code was the actual engine running the RTOS (and other stuff like semaphores). Is the micrium fifo library being used elsewhere in our code (like is it something that we are already using and should start using everywhere or is it something that should be used in an RTOS for FIFO's)?

Yes, bare-metal code is when it is running without an RTOS. I don't think we are using the micrium fifo library anywhere right now, but I think the plan was for CAN to use it. @SijWoo correct me if I'm wrong on this.

Since the purpose of the BSP is to make the application code hardware independent, would it make more sense to have an enum that lists out the different UART ports? Also we should do the same for BSP_SPI since the LTC and the new amperes board IC are on separate spi ports.
typedef enum {
    uart_usb,
    uart_ble
} uart_port_t;

typedef enum {
    spi_ltc6811,
    spi_as8510
} spi_port_t

Yee that's correct, the CAN task is going to use the the micrium queue. To make the BSP_UART functions thread safe, we either need to make sure the fifo functions have mutexes or just use the micrium fifo. I think it would be easier to use the micrium fifo (queue) in the BSP_UART as well and not worry about testing the fifo functions for multi-thread safety.

Also we need to be careful of reentrant functions with these. Using only local variables/input parameters or putting mutexes to global variables would prevent re-entrant issues.

I will try using the micrium fifo in the UART code. I will also try updating the simulator code to work with the the new header functions.

Since I removed the #define at the start of the file, all of the code is commented out and that is causing the errors. If it is compiled with RTOS or BAREMETAL, it will work.


new issue

adssdsa

Hi, I have some suggestions to help solving this issue!
Here is a list of similar issues:

1. [test new issue](https://github.com/fjsj/python-social-auth/issues/40)

2. [test new issue again](https://github.com/fjsj/python-social-auth/issues/47)

3. [test new issue 123](https://github.com/fjsj/python-social-auth/issues/42)

4. [test new issue 123](https://github.com/fjsj/python-social-auth/issues/41)

5. [test new issue](https://github.com/fjsj/python-social-auth/issues/46)

Hope this is useful!
Cheers,
Cinnabot.


include getStreamResponse(...) in the interface

The getStreamResponse(...) function was not included in the interface class...

Closes #{put_issue_number_here}
Subject

This cannot be done on the stable branch, that's why I changed the branch for you. Please rebase your branch against the 4.0.x branch @sylvery

Already done with #172


fix: templ CLI panic with --proxy option

fixes #358

Watching this one in hopes it resolves an issue we are having 🤞

Thanks for that!

Thank you for giving me the opportunity to contribute @a-h 😃
Out of curiosity, could you tell me why you preferred not to have the inline function ?

Only reason was so that the functionality can be followed by reading from top to bottom, rather than reading the function, and holding it in your mind until you see where and how it's used.
It was clear, but I thought it could be a tiny bit more direct without having the function variable.


pythonPackages.tpm2-pytss: init at 0.2.4

Motivation for this change
tpm2-pytss is a python binding on tpm2-tss library
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 114071 at c4b2992b run on x86_64-linux 1

3 packages built:

python37Packages.tpm2-pytss
python38Packages.tpm2-pytss
python39Packages.tpm2-pytss

something is not right with this closure.


UpdateSubmissions has unused courseID argument

func (db *GormDB) UpdateSubmissions(courseID uint64, query *qf.Submission) error {

The method does not use the courseID argument in the update query. Similarly, the
func (s *QuickFeedService) updateSubmissions(request *qf.UpdateSubmissionsRequest) error {

and the qf.UpdateSubmissionsRequest probably does not need courseID, since AssignmentID should be unique.
Double check why this field is there.

qf.UpdateSubmissionsRequest got courseID to enable access control check without extra database lookups.


Change album cover and colour for local files

It would be nice if you are able to get the local album art and colours or just use a default art instead of the one from the previous song.

Depending on the player it should revert to the white Monstercat logo, maybe your player sends some info which can't be processed by the plugin for Rainmeter.
Can you tell me which Rainmeter version, Media Player and Skin Version?


xSQLServerDatabase: Should Be Cluster Aware

Details of the scenario you tried and the problem that is occurring:
When the xSQLServerDatabase resource is applied to all of the nodes in a Failover Cluster Instance (FCI), all of the nodes will connect to the instance and ensure the desired state is true even if the node is not currently hosting the instance.
The DSC configuration that is using the resource (as detailed as possible):
When the following configuration is applied to two or more nodes in a FCI.
xSQLServerDatabase Create_Database
{
    Ensure          = 'Present'
    SQLServer       = 'SQLServer'
    SQLInstanceName = 'DSC'
    Name            = 'Contoso'
}

Version of the Operating System, SQL Server and PowerShell the DSC Target Node is running:
Any
What module (SqlServer or SQLPS) and which version of the module the DSC Target Node is running:
Any
Version of the DSC module you're using, or 'dev' if you're using current dev branch:
Any

The resource should evaluate if the current node is actively hosting the SQL instance. If it is not hosting the instance, it should write a verbose message stating that it does not own the instance, and then silently exit.

To resolve this issue, a parameter called ProcessOnlyOnActiveNode should be created on the Set-TargetResource and Test-TargetResource functions.
The Get-TargetResource function will execute the helper function Test-ActiveNode to determine if the current node is actively hosting the SQL instance and return the result via the read-only property IsActiveNode.
The Test-TargetResource function will use the value of ProcessOnlyOnActiveNode and IsActiveNode to determine if it should continue. If ProcessOnlyOnActiveNode is $true and IsActiveNode is $false, Test-TargetResource will return $true.
I put together this gist as a guide to updating the resource.
Also, see the xSQLServerAlwaysOnAvailabilityGroup resource for a completed example.


Fix readme to post to docs website

Coverage remained the same at 71.968% when pulling cb601b211cf0ebb8c71e53489b5af7fb29a24abc on readme-2.0 into 91e24ac8936e7a1d437e2dad0f75f36ec893ac3b on master.


Release for psr/container v2

Hello @rosstuck, could we have a release with https://github.com/thephpleague/tactician-container/pull/15? It is blocking installation of symfony/contracts v3.
Thanks

Tagged as v3.0 :heart:


Added preventing slot highlight

This pull request makes it so the slot hover highlight doesn't happen when you're hovering over slots that don't contain an item, or contain a blank icon.
Without feature

With feature

LGTM


Close PIL images created when loading image files

This PR makes use of contextlib.closing to ensure that PIL images opened in face_recognition.load_image_file are closed.

@ageitgey I work on Windows, and I've not been able to setup properly (using python setup.py develop). This is why I haven't included at least a test for this change.
If someone who can setup properly wants to create a more complete PR based on this one, I'd be happy for them to.


Fix word HL after setting cursor directly

Fixes #159
Fixes #488

Big thanks to @huejim for pointing to the correct comamnd


Live preview not working with rmd using bookdown::render_book

Consider the following rmd using bookdown::render_book provided in #855:
---
output: bookdown::gitbook
knit: bookdown::render_book
---

# Test

Para.

---

# Next

Live preview does not work with such document:
processing file: main.Rmd

  |
  |                                                                      |   0%

  |
  |......................................................................| 100%
  ordinary text without R code

output file: /Users/ken/.vscode-R/tmp/main.knit.md

Error in relativeTo(basepath, dir) :
  The path /Users/ken/Workspaces/test/rbook/libs/header-attrs-2.11 does not appear to be a descendant of /Users/ken/.vscode-R/tmp/
Calls: cat ... html_dependencies_as_string -> lapply -> FUN -> relativeTo

Backtrace:

1: stop("The path ", file, " does not appear to be a descendant of ",
2: relativeTo(basepath, dir)
3: FUN(X[[i]], ...)
4: lapply(dependencies, makeDependencyRelative, basepath = output_dir,
5: html_dependencies_as_string(dependencies, lib_dir, output_dir)

6: as_tmpfile(html_dependencies_as_string(dependencies, lib_dir,
7: pandoc_html_extras_args(extras, self_contained, lib_dir, output_dir)
8: base(...)

9: output_format$pre_processor(front_matter, input, runtime, knit_meta,
10: rmarkdown::render(file_path, output_format = set_html, output_file = output_file_loc,

11: cat(lim, rmarkdown::render(file_path, output_format = set_html,

Execution halted

[VSC-R] main.Rmd process exited with exit code 1

Looks like rendering to ~/.vscode-R/tmp makes the paths inconsistent.

I meet the same problem .
shows :
'Error in relativeTo(basepath, dir) :
The path $WORKDIR/libs/header-attrs-2.9 does not appear to be a descendant of $HOME/.vscode-R/tmp/
Calls: cat ... html_dependencies_as_string -> lapply -> FUN -> relativeTo'

It looks like this is also an issue with the bookdown format and outputs that use libs in general. We can get around this by using output_options = list(self_contained = TRUE. However, we run into another problem -- bookdown is not respecting the output_dir argument. So no matter what we specify, the files will always be knit to the working directory, leading to another error

I am also running into this issue as well trying to use the revealjs package which is a Rmd wrapper around the reveal.js presentation framework. The output is rendered as if it were a plain Markdown file. However, it does work correctly if I directly invoke rmarkdown::render from the console.
I tried to add self_contained = true as per https://github.com/rstudio/revealjs#document-dependencies, to no avail.

Is there any workaround without touching the extension itself to modify output_options?

this is also an issue with the bookdown format and outputs that use libs in general

What is the blocker for this, if rmarkdown::render just works?

I am very new to R and Rmd, and sorry if any of these questions sounds nonsensical.

Hi @SuibianP ,
Thank you for the comment. It's been a little while since I looked at this so I may need a refresher.
I haven't used revealjs before so I'll have to look into the specific reason it doesn't work. If it's a lib folder problem like bookdown, it's because our preview mechanism generates an html file in a temporary folder, and expects the file to work as-is. An issue I ran into with bookdown was that there was no way of telling it where to put the lib folder that worked at the time.
Custom knit and rmarkdown::render don't run into this problem because we don't try to hide the knit files in a temporary folder.

Hi @ElianHugh,
Thanks for the reply. I tried to diff the pandoc command lines and here is what I think makes the difference,
 --to
-html4
+revealjs
 --from
-markdown+autolink_bare_uris+tex_math_single_backslash
+markdown+autolink_bare_uris+tex_math_single_backslash-implicit_figures

Seems that the output key in the front matter (revealjs::revealjs_presentation) is not respected?


Login required

fixes #26

Coverage decreased (-0.02%) to 96.93% when pulling da4c3a6e12bd750266c25dd0834afc4498fba8ef on login_required into 85e21501054a66e42daa324e5a69d91e7de3e506 on master.


TP2000-486 Failure "... multiple values for argument 'record_group'" when importing envelope created by dump_transactions

TP2000-486 Failure "... multiple values for argument 'record_group'" when importing envelope created by dump_transactions

Why

We can't currently import envelopes
What

Add missing self arg to process_file and form.save() test

Codecov Report

Merging #666 (f61e5ae) into master (01264eb) will increase coverage by 0.03%.
The diff coverage is 100.00%.

:exclamation: Current head f61e5ae differs from pull request most recent head d23396f. Consider uploading reports for the commit d23396f to get more accurate results

@@            Coverage Diff             @@
##           master     #666      +/-   ##
==========================================
+ Coverage   93.63%   93.66%   +0.03%
==========================================
  Files         337      337
  Lines       21684    21698      +14
  Branches     1679     1679
==========================================
+ Hits        20303    20323      +20
+ Misses       1063     1056       -7
- Partials      318      319       +1

Impacted Files
Coverage Δ

importer/forms.py
88.57% <ø> (+8.57%)
:arrow_up:

importer/tests/test_forms.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


aRIA.conf

Bug in archive aRIA.conf
continuous deconfiguration.

you can add nodes manually in conf file though for permanent connectivity.


Updating ViewModel cache.

In some situation i get this:
temp1.get().data.items[4].name.value
>>> "Member Name!!!"
temp1.get("data.items[4].name.value")
>>> "Member Name"

How i can manually update cache?

Ractive has two update methods, ractive.update() and ractive.updateModel(). update is useful if you update data without using the built in setter methods, while updateModel is useful if you programmatically manipulate inputs and other elements that have two‐way binding set up.

@Madgvox This methods doesn't updates data.items[4].name.value value.
temp1.get("data.items[4].name.value")
"Member Name"
temp1.get().data.items[4].name.value
"Member Name!"
temp1.update()
temp1.get("data.items[4].name.value")
"Member Name"
temp1.updateModel()
temp1.get("data.items[4].name.value")
"Member Name"

What version of ractive are you on & can you post a jsfiddle repro? That could be a bug.

@Madgvox 0.7.3 http://jsfiddle.net/ddgamw8L/4/ . I know, code is very unusual :-)

The bugs in the ArrayAdaptor. It works if you use the array api method instead:
this.push("context.data.value.items", {});

@martypdx https://github.com/ractivejs/ractive/issues/2275 "Use the array methods instead of api method" :-|

@martypdx but thanks, anyway...

@martypdx #2275 "Use the array methods instead of api method" :-|

:cry: indeed. Marked as a bug.  #2275 probably shouldn't have been closed, but we'll leave it in favor of this one.

I closed #2275 because it had been fixed in 0.8-edge. This is a different bug.

It looks like the binding side of this issue is fixed in edge, so I'm going to close this and link to #2420 for the observer bit.


Update and fix GPT-4 models, prices

Add GPT-4o, remove "-preview" for GPT-4 Turbo, and fix pricing. GPT-4 pricing was wrong since the begining, fixed now.
Reference:

https://openai.com/api/pricing/

Add GPT-4o, remove "-preview" for GPT-4 Turbo, and fix pricing. GPT-4 pricing was wrong since the begining, fixed now.
Reference:

https://openai.com/api/pricing/

Thanks for your contribution! @PeterDaveHello


Allow data classes to be branded with an additional type parameter

cc @gcanti we should make a list of APIs that benefit from branding, basically all APIs that produce class instances


feat(acvm)!: Add CommonReferenceString backend trait

Related issue(s)
Needed for https://github.com/noir-lang/acvm-backend-barretenberg/issues/160
Description
Summary of changes
This requires a CommonReferenceString trait for ACVM Backends.
The reason some functions in other traits were modified to take the common_reference_string is because backends can have a circular dependency, such that they need to instantiate themselves before they can get the circuit size for a given circuit and the CRS often needs the circuit size to the data it is using. This means we can't do Backend::new(CRS::new(circuit_size)) and resulted the functions that need a CRS taking it as an argument.
Dependency additions / changes
(If applicable.)
Test additions / changes
(If applicable.)
Checklist

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt with default settings.
[ ] I have linked this PR to the issue(s) that it resolves.
[x] I have reviewed the changes on GitHub, line by line.
[x] I have ensured all changes are covered in the description.

Additional context
(If applicable.)

Opened as a draft to discuss

This is a significant change to Backends, so we should hold it (even after approvals) until the CRS changes all prepared for the Backend and Nargo.

Will take a look at this in the morning (singapore time) and can merge after.

Looks good, modified one comment.


Can't pull docker images from private repositories

IT runner is unable to pull docker images from private repositories requiring credentials.
We'll need to be able to read docker config.json files as well as take in arguments.

itrunner now supports environment variables to be provided to configuration and dependency files.
With this addition you can use dockerauth to pull private docker images.


cmd/compile: Unclear guidelines on what files/directories are ignored

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
1.8.1
What operating system and processor architecture are you using (go env)?
darwin x86_64
What did you do?
underscore-test ❯ pwd
/Users/ai/src/go/src/scratch/underscore-test
underscore-test ❯ tree -a
.
├── .dot
│   └── dot.go
├── _underscore
│   └── underscore.go
└── main.go

2 directories, 3 files
underscore-test ❯ cat main.go _underscore/underscore.go .dot/dot.go
package main

import (
        "fmt"
        "scratch/underscore-test/_underscore"

        "scratch/underscore-test/.dot"
)

func main() {
        fmt.Println(underscore.Underscore)
        fmt.Println(dot.Dot)
}
package underscore

// Underscore is the best yo
const Underscore = "_"
package dot

// Dot is a dot
const Dot = "."

What did you expect to see?
I expected a compilation error, given that documentation states that "." and "_" prefixed files will be ignored, however this rule does not seem to apply to directories. I ran into this yesterday with @davecheney, who suggested that I modify golang/dep to skip scanning directories which start with _ or .. Currently we're in a weird situation where files are ignored but directories are not, and I can't find any documentation stating what the source of truth should be. Does golang/dep have the correct behavior, or broken behavior? Will the golang toolchain eventually stop compiling directories that start with . or _?
What did you see instead?
It worked:
underscore-test ❯ go build
underscore-test ❯ echo $?
0
underscore-test ❯ ./underscore-test
_
.

https://golang.org/cmd/go/#hdr-Description_of_package_lists

Directory and file names that begin with "." or "_" are ignored by the go tool, as are directories named "testdata".

Ok, so is it a bug that my example code compiles?

I guess the underlying question is "is there a difference between things that are ignored by the gotool, and what are valid import paths?". Is that correct?

I guess the underlying question is "is there a difference between things that are ignored by the go tool, and what are valid import paths?".

yes, there is a difference. Those paths will (probably) not show up in go list ./... but can be imported.

Ok, so we can close this issue, and I should change golang/dep to recurse into .-prefixed dirs when scanning what source packages are available.
Thanks!

@sectioneight I think you should do the opposite
deadwood(~/src/scratch/underscore-test) % go list ./...
scratch/underscore-test
deadwood(~/src/scratch/underscore-test) % find . -type d
.
./.dot
./_underscore

Sorry, what exactly do you mean by "the opposite"? Change dep or change go?

You should not change dep to recruse into paths starting with a period or
an underscore.
On Fri, May 12, 2017 at 1:51 PM, Aiden Scandella<EMAIL_ADDRESS>wrote:

Sorry, what exactly do you mean by "the opposite"? Change dep or change go
?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/20337#issuecomment-300977087, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAcA91TBk-2k4RS5Fr8-odE5vpkN4Ygks5r49dIgaJpZM4NYyyU
.

Ok. In that case dep will fail if a dependency has a package named ".foo" since it doesn't see the source code... We have a real world example of this today with uber/cherami-client-go/.generated.
Suggestions? Or am I missing something really obvious.

You can import that package in another package, but it will not match the
globbing rules of the go tool.
It sounds like dep is conflating two different operations

for a tree of directories; figure out which contain valid Go packages
for a directory of source files; figure out which ones are are in scope
according to build rules.

On Fri, May 12, 2017 at 1:55 PM, Aiden Scandella<EMAIL_ADDRESS>wrote:

Ok. In that case dep will fail if a dependency has a package named ".foo"
since it doesn't see the source code... We have a real world example of
this today with uber/cherami-client-go/.generated.
Suggestions? Or am I missing something really obvious.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/20337#issuecomment-300977503, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAcA2-GXNFtBRQs6sZ0MdQg9R7xf1qYks5r49gzgaJpZM4NYyyU
.

(we can probably close this and discuss over in the dep issue)


Revise version numbering description in read-me

Ok, I think we'll just stick with the three numbers we have now, but they're not exactly semantic versioning then. So for x.y.z we have

increment of z when there's a hotfix (bugfix) pushed to main
increment of y when develop is merged into main
increment of x when we've been working on something extraordinary, when there's a major change in interface (configs or outputs) ... or when y is becoming too big (26 is industry standard I believe).

We need to

review the read-me and make sure that this is what it says
set up a system where each pull request has a headline and description and that these go into the change log, together with the version number.

Originally posted by @einola in https://github.com/nextsimdg/nextsimdg/issues/121#issuecomment-1222109945

This issue is to track point two in the above comment: "review the read-me and make sure that this is what it says"

I've been investigating a bit the tools to achieve point 2 of your comment, the most promising is https://github.com/github-changelog-generator/github-changelog-generator provided we can choose the degree of details we put in the description of each version. I will try to deploy it in branch issue125_version_changelog

This looks very good 👍
Looking forward to see it in action in issue125_version_changelog.

First attempt of using the github changelog generator is visible here : https://github.com/nextsimdg/nextsimdg/blob/issue125_version_changelog/CHANGELOG.md.
I'm disappointed that the comments associated to every release do not appear, I will try to make it so but not sure if it's possible, possibly I will have to choose another tool ...

I see what you mean ... but it's not too bad I think. We just need to do better in writing descriptive titles (which is a good idea, regardless of the change-log tool).
So, I'm ok with this, but you should of course keep trying to improve it if you think that's feasible (either by a better configuration or by using a different system).

I've done some more investigations and managed to add extra informations about releases into the changelog : see https://github.com/nextsimdg/nextsimdg/blob/issue125_version_changelog/CHANGELOG.md.
For each release we now have a description, which is the comment that came with the tag generation. If no comments were provided during the creation of the tag, it will say Merge pull request #XX from nextsimdg/branchYY.
So a good practice would be to provided nice comments when we create the tag, should I add this recommendation in the README ?
Besides, the description provided when doing the release will appear here, so it is nice to have it also.

This is starting to look very nice. I wonder why the change log for version 0.2.0 doesn't include a list over merged pull requests. Any idea?
Please add the recommendation to the README - thanks :)


fix: revert data module serialization (#216)

Fixes #398

I reproduced the issue and the fix using a-vs-b.


Clarification on python tflite-runtime.interpreter.load_delegate for Opaque Delegate in Arm NN

Hello Arm NN Team,
I am currently using tflite-runtime in Python and utilizing Arm NN tflite delegate. I have a question regarding the use of tflite.load_delegate when the library parameter is set to an opaque delegate. Specifically, I would like to know how to set the options parameter in this case. Should the options parameter be configured the same way as when the library parameter is set to a classic delegate?
I have reviewed the Arm NN documentation but couldn't find the information I was looking for. Could you please provide guidance?
Thank you for your assistance!

Hiya,
Yes, the options parameter is the same for both of our delegates. They both use the options defined under armnn/delegate/common/include/DelegateOptions.hpp
I hope this answers your question.

Hiya, Yes, the options parameter is the same for both of our delegates. They both use the options defined under armnn/delegate/common/include/DelegateOptions.hpp I hope this answers your question.

Thanks for the help 😄


syz-manager, dashboard: support collection of kdump cores

Linux kernel supports kdump:
https://www.kernel.org/doc/Documentation/kdump/kdump.txt
which can be used to do memory dump and send it over network. Manager could collect kdump dumps for crashes, esp for bugs without reproducers.
We need to figure out how to set it all up, build kdump kernel, pack it into image, make kdump kernel send code back to manager. Dumping must also be somehow limited, because saving cores for all crashes will take unreasonable amount of space.
That this needs to be integrated with dashboard as well
Should start with making it work on gce first.

FTR, reply from mailing list (captures main things that need to be done):
It's just that usually fully automating something is much larger
amount of work than doing it manually as a one-off thing. I also need
to figure out how much time and space it takes to reboot into kdump
kernel and extract the dump. I don't think that it's feasible to
persistently store all kdumps, because we are getting ~1 crash/sec.
Then, the web server that serves syzbot UI and sends emails is an
Appengine web app which does not have direct access to test machines
and/or git, but it seems that only it can decide when we need to store
dumps persistently. In the current architecture test machines are
disposable and are long gone by the time crash it uploaded to
dashboard. So machines needs to be preserved until after dashboard
says if we need dump or not. Or maybe extract dumps always and store
them locally temporary until we know if we need to persist it or not.
I don't know yet what will work better. This also needs to be
carefully treated through crash reproduction process which has
different logic from main testing loop. And at the end interfaces
between multiple systems need to be extended, database format needs to
be extended, lots of testing done, and we need to figure out what is a
good config for kdump kernel and image build process needs to be
extended to package kdump kernel, configs of multiple systems need to
be extended and probably a bunch of other small things here and there.
Then we also need vmlinux to make dumps actionable, right? And vmlinux
is nice in itself because it allows to do objdump -d. So it probably
makes sense to separate vmlinux uploading and persistance from dumps,
because vmlinux'es probably better be uploaded once per kernel build
(which is like once per day). So that will be separate paths through
the system.
Also probably makes sense to consider if
https://github.com/google/syzkaller/issues/466 can be bundled with
this work (at least data paths, what exactly is captured can of course
be extended later).
We also need to figure out if at least part of all this can be
unit-tested and write tests.
So, yes, nothing extraordinary. But I feel this is not doable within a
day and will preferably require several uninterrupted days with
nothing else urgent, but I am having troubles with such days lately...

Another possibility is using CONFIG_PVPANIC=y to dump as much info as possible on every crash (e.g. traceback of all CPUs, etc):
https://groups.google.com/d/msg/syzkaller/mbUS_75ExMI/pztWbmxICAAJ

The kdump-tools or linux-crashdump packages may be useful for this:
https://www.thegeekstuff.com/2014/05/kdump/
https://packages.ubuntu.com/xenial/devel/linux-crashdump
Maybe we don't need to build own kdump kernel and just use the package-provided one. However, the question is if kdump kernel version needs to match the main kernel or not (e.g. old distro provided kernel won't be able to properly dump everything about the newest kernel).
kdump-tools supports several locations to save the dump: local disk, nfs, ftp (?). Need to figure out what to use and how to glue it all together.

For a while now I've been using an instance where FreeBSD guests are configured to dump core over the network when they crash. This has proven extremely useful for rare crashes that don't get a reproducer. It also permits automated analysis of crash dumps. There are two issues that I've observed, neither of which I have fully solved:

The VM manager needs to wait for the netdump to complete before restarting the VM. For now I've just bumped waitForOutputTimeout to one minute. Ideally we would have an OS-dependent function to monitor console output following a panic and return once the dump is complete.

The agent which receives a kernel dump has no good way to tie it back to a syzkaller report. We can't use the panic string as an identifier since syzkaller transforms the panic string in various ways, and of course one can have distinct bugs with the same panic message. This is not a dealbreaker since one can debug without a reproducer, but it would be quite nice to be able to package kernel crash dumps and syzkaller reproducers together whenever possible.

I've done some work towards 1), but any thoughts or suggestions on 2) would be appreciated.

The agent which receives a kernel dump has no good way to tie it back to a syzkaller report.

I would say this is very important. If we will provide a wrong dump from time to time, it will suck. According to Murphy's law, we will sure get the mismatch when syzbot users will first resort to debugging the dump for a tricky bug.
I don't know what are the means and controls available, and what is the protocol.
Is it possible to give a kernel a unique ID? Is it possible to make different machines send dumps to different addresses? E.g. if we have 10 VMs, we create 10 ports and give each machine own unique one.

I think the two requirements I listed are probably universal. From your notes above we also have:

Need a way to limit space consumed by crash dumps. (As a point of reference, in my setup the kernel creates zstd-compressed dumps that average 16MB each. This is with VMs that run with 1GB of RAM.)
Need a way to package the kernel image and debug info together with a crash.

Is it possible to configure FreeBSD to dump the core only if asked to?

Yes. Basically, from the kernel debugger you can write, "netdump -s <IP_ADDRESS>" to transmit a dump to an agent (netdumpd) listening on <IP_ADDRESS>. This could be done from vmimpl.DiagnoseFreeBSD(). I can't think of a good way to request this from the VMM.

I don't know what are the means and controls available, and what is the protocol.

We use a TFTP-like protocol called netdump, over UDP. It is very simple and not really extensible. We include a kernel dump header which contains various metadata (e.g., the panic string).

Is it possible to give a kernel a unique ID? Is it possible to make different machines send dumps to different addresses?

We have a notion of the "host ID", which is a UUID. I believe we can configure the VMs such that a new host ID is generated on every boot, and after a crash we can recover the host ID from the kernel dump. With this, would it be necessary to send dumps to a unique address? That would be a bit complicated to set up.

But if we can do "netdump -s <IP_ADDRESS>", this seems to be everything we need (assuming it's possible to specify a port as well). Then we just dump each VM to a unique port. Anything I am missing?

Actually it is currently not possible to specify the port, but I can change that pretty easily if needed.
However, I don't think even that is necessary: netdumpd can be configured to run a script after a kernel dump completes, and one of the parameters to that script is the source IP address. So, it should be easy to use that to notify syz-manager that the dump is completed.

If we can figure out IP addresses for the VMs, then it sounds like a plan.

I am now waiting for #2005 to merge because I also need to change that part of the file.

A thread on how to setup KDUMP kernel:
Help needed in getting kernel dump in QEMU VM
https://groups.google.com/g/syzkaller/c/djTdBlMrxi0

@dvyukov Some further information from my side: if you installed a Debian system on the disk other than the debootstrap system, it is easy to collect the kernel dump when you configure kdump following my configuration. However, in this situation, you will lose the "-kernel" option to start the kernel version arbitrarily. If I make mistakes or miss something, please let me know.

@markjdb Can you collect kernel dump in the Freebsd instance? If yes, which environment are you in? QEMU VM, GCE or others?

@mudongliang I am not quite sure what you are asking. In general it is pretty trivial to set up kernel dumps for FreeBSD, but our mechanism works differently from KDUMP. I've been able to collect kernel dumps in various virtualized environments, including QEMU and GCE, sure.

@markjdb thanks for your reply. I have already found a way to generate kernel dump in QEMU.

Regarding kdump collection for GCE.
One other option (in my view, most secure and robust) is to create for each VM an additional persistent disk with the size equal to the RAM size of the VM. During crash, the kexec kernel will just dd all its memory to the disk as raw bytes, no file system is needed in this case. Then, if the dump turns out to be necessary (e.g. the dashboard was asked and replied yes), syz-manager can download the content of the disk and minimize it via makedumpfile.
But there's a problem -- it seems that downloading a persistent disk is not so straightforward in GCP. One has to convert it to an image first and then download the image. The conversion is possible only once in 10 minutes.

Maybe we could boot another VM with that disk to extract contents? Wonder if there is disk hot plug... then we could just attach it to the syz-manager machine.
However, sending the dump to the manager with netcat looks comparable in complexity. Manager could open a separate port for each VM, then we can match dumps to VMs based on the port number. Manager can also reject the dump by closing the connection.