Option to show full path in cythonize error output?

Currently cythonize prints error line numbers with relative paths:
Error compiling Cython file:
------------------------------------------------------------
...
------------------------------------------------------------

lib/foo.pyx:123:45: Cannot convert Python object to X

However, some environments do not support recognition/auto-hyperlinking of relative paths in external tool command output (e.g. JetBrains). I've made the following local change to get a full path in error messages (now: /path/to/lib/foo.pyx:123:45: Cannot convert Python object to X).
diff --git a/Compiler/Errors.py b/Compiler/Errors.py
index 9761b52..c2fcf8a 100644
--- a/Compiler/Errors.py
+++ b/Compiler/Errors.py
@@ -42,7 +42,7 @@ def context(position):

 def format_position(position):
     if position:
-        return u"%s:%d:%d: " % (position[0].get_error_description(),
+        return u"%s:%d:%d: " % (position[0].filename,
                                 position[1], position[2])
     return u''

This works for my immediate needs, however, it would be nice to have available as a config option. I can make the change at some point, but wanted (a) get approval first (b) suggest as a potential starter/good-first-issue (if you have a tag like that) until I can get to it.

This is definitely something we could support. It sounds like it'd help out a number of editors.
I wonder if there's a good reason to show relative paths at all? Config options are always more difficult than just doing one thing. I can't immediately think of one (but that doesn't mean there isn't one)

I wonder if there's a good reason to show relative paths at all?

They are much more readable for humans. Depending on how long the working directory path is, there might be a lot to mentally ignore before the path becomes interesting.
It seems more of a bug that relative paths don't work well for (some) editors. They should. I think it's rather something to fix in those editors.
OTOH, maybe we could use relative paths when the output goes to a terminal window and absolute paths when sending it to a pipe? That might be just enough to make the behaviour work well in most cases without adding a new option.

While not entirely related to this issue (this one is error while compiling instead of runtime error), there is https://github.com/cython/cython/issues/2543 / https://github.com/cython/cython/issues/3929 which occasionally makes traceback not display the source code.


Could not load file or assembly 'System.Web.WebPages.Razor, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference.

I'm trying to get Articulate to work on an existing Umbraco installation. When I navigate to the blog URL, I get the following error:
Could not load file or assembly 'System.Web.WebPages.Razor, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference.
The Umbraco version I'm using is 7.2.8. So what am I missing here? Do I need to manually update the Razor dll (which I'm hesitant to do)? Should I just redirect the version 3 to whatever version I have in the bin folder (seems unlikely)?

Hi, Articulate requires 7.3 at a minimum, see release notes fro version 2.0: https://github.com/Shazwazza/Articulate/releases/tag/v2.0.0
I will update the docs to make this more clear.


WMS 11050 and 11052 Update

WMS 11050 and 11052 Update
JMS 7.0 Livelabs Workshop 2 & 3 updates (AJ-11971)
Updated screen captures to JMS 7.0 UI
Added new features/enhancements- Java Migration Analysis, Performance Analysis and LCM Customisation
Added object storage service explanation
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[x] All filenames are lower case (including folders, images, files, etc.)
[x] Filenames are descriptive
[x] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[x] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[x] Image references in markdown contain an alternate text

Looks good


Add reference to note when "error updating note" is thrown during npx dendron publish

Please select if your request is either something new or an enhancement

[x] Enhancement of an existing Feature.
[ ] Request of a new feature.

Please select the area your request applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[x] Workspace - VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup - Dendron's Lookup Command
[ ] Views - Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema - Dendron Schemas
[ ] Pod - Data import from / export to Dendron
[x] Publish - External Site Publish
[ ] Markdown - Markdown features, such as syntax support and features

Is your feature request related to a problem? Please describe
When a note is unable to be updated the error is vague and it doesn't specify which note, so it's impossible to track down.
Eg somehow the tags section in the frontmatter of a note was garbled, but instead of saying why or even which note, it just fails with generic error updating note.
Example frontmatter (which I don't know how ended up like this, another story):
---
id: zcxypocn4k2y8isn3oavrxm
title: TED-01
desc: ''
updated:<PHONE_NUMBER>010
created:<PHONE_NUMBER>751
tags:
  Australian Launch: null
  SA: null
  DART: null
  sub-orbital: null
---

Example error output:
error {"stack":"DendronError: error updating note"\n    at DendronEngineClient.updateNote (C:\\Github\\space-03-astro_01\\dendron\\vault\\node_modules\\@dendronhq\\engine-server\\lib\\engineClient.js:311:19)\n    at runMicrotasks (<anonymous>)\n.........
(The actual error is 100x longer than this, but much the same through the stack)
Describe the solution you'd like
I changed my local version of the above referenced Dendron engine file to this:
throw new common_all_1.DendronError({ message: "error updating note:'" + note.title + "'", payload: resp });
And it at least shows the name of the note, so was able to find and fix it.

@AshSimmonds  - thanks for the report.  I've hit this in the past and it can be pretty frustrating - @kevinslin, I think we should prioritize this one as part of the publishing enhancement.


fix(sdk-logs): avoid map attribute set when count limit exceeded

Which problem is this PR solving?
Make LogRecord.setAttribute don't set map<string, any> typed attribute when logRecordLimit.attributeCountLimit exceeded.
Short description of the changes
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Add test case for map attributes

Checklist:

[x] Followed the style guidelines of this project
[x] Unit tests have been added
[ ] Documentation has been updated

Thank you for working on this!


[Enhancement] Add Single Packet Authorization Support

It would be neat to have the ability to send SPA to fwknop, a more secure port knocking method.
While there is already an android app available it lacks the quality of life features of knock on ports

I thought about it already.
Maybe sometime in the future. I won't promise anything for now.


Napalm x Astronaut In The Ocean by GDtrilogy

Song Name
Napalm x Astronaut In The Ocean
Artist Name
GDtrilogy
Source
Youtube
Youtube Link
QuP4SkX9tps
Direct File Link
No response
Song ID
718171
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 65625974
Downloaded: Yes

reject
mashups/covers should have some popularity to show that people want it (especially if the song doesnt sound great).
Note that you can still use the nong yourself, but it wont be public


Plugin test doesn't seem to clean up after itself (sinon error "Attempted to wrap request which is already wrapped")

Summary
I wrote a plugin which has a test suite consisting of

an integration test for the command which is based on the generated fancy-test test (e.g. test.withOrg etc.)
some unit test for helpers, some of which interact with the sfdx Connection which use sinon to stub that Connection

When I run either the fancy-test based integration test separately, or my helper tests separately, everything seems fine. But when I run the whole test suite, the command test runs first (and passes), but then as soon as my helper tests try to stub something on Connection, I get:
TypeError: Attempted to wrap request which is already wrapped
      at checkWrappedMethod (node_modules/sinon/lib/sinon/util/core/wrap-method.js:48:21)
      at wrapMethod (node_modules/sinon/lib/sinon/util/core/wrap-method.js:111:13)
      at stub (node_modules/sinon/lib/sinon/stub.js:123:44)
      at /Users/frans/workspaces/salesforce/sfdx-ci-plugin/node_modules/sinon/lib/sinon/util/core/walk-object.js:33:17
      at /Users/frans/workspaces/salesforce/sfdx-ci-plugin/node_modules/sinon/lib/sinon/util/core/walk.js:27:22
      at Array.forEach (<anonymous>)
      at walkInternal (node_modules/sinon/lib/sinon/util/core/walk.js:19:5)
      at walkInternal (node_modules/sinon/lib/sinon/util/core/walk.js:33:9)
      at walk (node_modules/sinon/lib/sinon/util/core/walk.js:48:12)
      at walkObject (node_modules/sinon/lib/sinon/util/core/walk-object.js:18:5)
      at stub (node_modules/sinon/lib/sinon/stub.js:96:16)
      at stub.createStubInstance (node_modules/sinon/lib/sinon/stub.js:131:25)
      at Sandbox.createStubInstance (node_modules/sinon/lib/sinon/sandbox.js:55:52)
      at ( my code which does const connectionStub = sinon.createStubInstance(Connection); )
      at processImmediate (node:internal/timers:463:21)
  --------------
  Error: Stack Trace for original
      at wrapMethod (node_modules/sinon/lib/sinon/util/core/wrap-method.js:139:26)
      at Function.stub (node_modules/sinon/lib/sinon/stub.js:123:44)
      at Sandbox.stub (node_modules/sinon/lib/sinon/sandbox.js:369:37)
      at Object.stubMethod (node_modules/@salesforce/ts-sinon/lib/index.js:60:20)
      at Object.stubContext (node_modules/@salesforce/core/lib/testSetup.js:233:16)
      at Context.<anonymous> (node_modules/@salesforce/core/lib/testSetup.js:275:17)
      at processImmediate (node:internal/timers:463:21)

Steps To Reproduce:

Generate a new plugin called sfdx-plugin-sinon-error: sfdx plugins:generate sfdx-plugin-sinon-error --defaults

cd sfdx-plugin-sinon-error

Run npm install

Run yarn remove @types/node && yarn add @types/jsforce -D && yarn prepack as per https://github.com/forcedotcom/cli/issues/1038#issuecomment-856802846

Add @salesforce/ts-sinon because somehow it isn't there: yarn add --dev @salesforce/ts-sinon

Add a trivial test that just stubs Connection:
mkdir -p test/helpers && cat <<EOF >test/helpers/test-connection-stubbing.test.ts
import {Connection} from '@salesforce/core';
import {expect} from 'chai';
import sinon = require('sinon');
describe('Test stubbing Connection', () => {
  it('should be able to stub Connection', async () => {
    const connectionStub = sinon.createStubInstance(Connection);
    expect(connectionStub).to.not.be.null;
  });
});
EOF

Run TZ=Z npm test

Or just paste the following into a file and run it with sh:
#!/bin/sh
sfdx plugins:generate sfdx-plugin-sinon-error --defaults
cd sfdx-plugin-sinon-error
npm install
yarn remove @types/node && yarn add @types/jsforce -D && yarn prepack
yarn add --dev @salesforce/ts-sinon
mkdir -p test/helpers && cat <<EOF >test/helpers/test-connection-stubbing.test.ts
import {Connection} from '@salesforce/core';
import {expect} from 'chai';
import sinon = require('sinon');
describe('Test stubbing Connection', () => {
    it('should be able to stub Connection', async () => {
        const connectionStub = sinon.createStubInstance(Connection);
        expect(connectionStub).to.not.be.null;
    });
});
EOF
TZ=Z npm test

Expected result
Both tests pass.
Actual result
    ><EMAIL_ADDRESS>test
    > nyc --extension .ts mocha --forbid-only "test/**/*.test.ts"

    hello:org
        ✓ runs hello:org --targetusername<EMAIL_ADDRESS>(397ms)

    Test stubbing Connection
    1) should be able to stub Connection

    1 passing (414ms)
    1 failing

    1) Test stubbing Connection
    should be able to stub Connection:
        TypeError: Attempted to wrap request which is already wrapped
    at checkWrappedMethod (node_modules/sinon/lib/sinon/util/core/wrap-method.js:39:21)
    at wrapMethod (node_modules/sinon/lib/sinon/util/core/wrap-method.js:85:13)
    at stub (node_modules/sinon/lib/sinon/stub.js:66:44)
    at /Users/frans/workspaces/salesforce/sfdx-plugin-sinon-error/node_modules/sinon/lib/sinon/stub-entire-object.js:15:13
    at /Users/frans/workspaces/salesforce/sfdx-plugin-sinon-error/node_modules/sinon/lib/sinon/util/core/walk.js:23:22
    at Array.forEach (<anonymous>)
    at walkInternal (node_modules/sinon/lib/sinon/util/core/walk.js:18:37)
    at walkInternal (node_modules/sinon/lib/sinon/util/core/walk.js:29:9)
    at walk (node_modules/sinon/lib/sinon/util/core/walk.js:44:12)
    at stubEntireObject (node_modules/sinon/lib/sinon/stub-entire-object.js:7:5)
    at stub (node_modules/sinon/lib/sinon/stub.js:47:16)
    at Sandbox.stub.createStubInstance (node_modules/sinon/lib/sinon/stub.js:73:12)
    at Context.<anonymous> (test/helpers/test-connection-stubbing.test.ts:6:34)
    at processImmediate (node:internal/timers:463:21)
    --------------
        Error: Stack Trace for original
        at wrapMethod (node_modules/sinon/lib/sinon/util/core/wrap-method.js:110:31)
    at stub (node_modules/sinon/lib/sinon/stub.js:66:44)
    at Sandbox.stub (node_modules/sinon/lib/sinon/sandbox.js:289:33)
    at Object.stubMethod (node_modules/@salesforce/ts-sinon/lib/index.js:60:20)
    at Object.stubContext (node_modules/@salesforce/core/lib/testSetup.js:233:16)
    at Context.<anonymous> (node_modules/@salesforce/core/lib/testSetup.js:275:17)
    at processImmediate (node:internal/timers:463:21)

System Information
{
  "cliVersion": "sfdx-cli/7.108.0",
  "architecture": "darwin-x64",
  "nodeVersion": "node-v14.17.1",
  "pluginVersions": [
    "@oclif/plugin-autocomplete 0.3.0 (core)",
    "@oclif/plugin-commands 1.3.0 (core)",
    "@oclif/plugin-help 3.2.2 (core)",
    "@oclif/plugin-not-found 1.2.4 (core)",
    "@oclif/plugin-plugins 1.10.0 (core)",
    "@oclif/plugin-update 1.4.0-3 (core)",
    "@oclif/plugin-warn-if-update-available 1.7.0 (core)",
    "@oclif/plugin-which 1.0.3 (core)",
    "@salesforce/sfdx-plugin-lwc-test 0.1.7 (core)",
    "@salesforce/sfdx-trust 3.6.0 (core)",
    "alias 1.1.10 (core)",
    "apex 0.2.2 (core)",
    "auth 1.7.0 (core)",
    "config 1.2.9 (core)",
    "custom-metadata 1.0.12 (core)",
    "data 0.4.12 (core)",
    "generator 1.1.7 (core)",
    "limits 1.2.1 (core)",
    "org 1.6.7 (core)",
    "salesforce-alm 52.0.2 (core)",
    "schema 1.0.7 (core)",
    "sfdx-cli 7.108.0 (core)",
    "telemetry 1.2.2 (core)",
    "templates 52.0.0 (core)",
    "user 1.3.0 (core)"
  ],
  "osVersion": "Darwin 18.7.0"
}

Additional information

your tests need to clean up after themselves or the mocks are left in place.   https://sinonjs.org/releases/latest/sandbox/
on trivial tests like yours, you can try connectionStub.restore() but it's easier for more complicated things to hang them off the sandbox and restore that after each
Recommendation: for integration tests (real orgs, real SF) try https://github.com/salesforcecli/cli-plugins-testkit

Hi Shane and thanks for responding.
So https://developer.salesforce.com/docs/atlas.en-us.sfdx_cli_plugins.meta/sfdx_cli_plugins/cli_plugins_test.htm suggests to "include both unit tests and more complex integration, smoke, and end-to-end tests" but then the generated plugin doesn't actually show you how to do that as it doesn't reset the sandbox so any unit test you add to that generated test that mocks Connection would have the problem I showed.
I feel that for this to be properly addressed:

the documentation at https://developer.salesforce.com/docs/atlas.en-us.sfdx_cli_plugins.meta/sfdx_cli_plugins/cli_plugins_test.htm should be expanded to explain how to properly write unit tests as well as integration tests and end-to-end tests for sfdx plugins
the generated test in the plugin generated with sfdx plugins:generate should reset the Sinon sandbox since that appears to be the best practice here

End-to-end tests using a scratch org per test have their place but they can't be the solution for everything since they're slow and are not a workable solution for developers on a developer edition org where only 5 scratch orgs are allowed to be created per day. IMHO end-to-end tests don't replace the need for unit/integration tests that mock out the Org/Connection.

To be clear, I've tried adding the following at the end of my tests but it seems to make absolutely no difference:
  $$.SANDBOX.restore();
  $$.SANDBOXES.CONNECTION.restore();

A concrete working example would be greatly appreciated.

Workaround for now: run two "test suites" as separate mocha runs by using this for the test task in package.json:
"test": "nyc --extension .ts mocha -r ts-node/register --forbid-only \"test/**/*.test.ts\" --exclude \"test/commands/**\" && nyc --exclude \"test/**\" --exclude \"src/helpers/**\" --extension .ts mocha -r ts-node/register --forbid-only \"test/commands/**\""

the code that's running the "basic" test from the plugin lives here: https://github.com/salesforcecli/command/blob/main/src/test/index.ts
it's using oclif/fancy-test.
I have a theory that this is related to mixing sinon versions (9 in fancytest, probably 10/11 if you install it yourself).
Could you try installing sinon@^9 and seeing it that prevents the conflict?
The other good alternative would be to not use fancy-test if you intend to do more sophisticated things.  I agree we should do all kinds of updates to the plugin-generate command.
Also, our more opinionated plugin template is here https://github.com/salesforcecli/plugin-template

we're fixing it this week in @W-10260806@


ngc_rdma isn't compatiable with Socket Direct

Need to implement support for Socket Direct in ngc rdma test/wrapper.
As of now, it is measuring each unit separately.

Resolved by #121


Fix e2e/namespace/install/cli/run_test.go for cli run from gist

Fix https://github.com/apache/camel-k/issues/3952
CI  install-it for 1.10.x branch is constantly failing this test
1.12.x branch is already fixed

Release Note
NONE

It seems this is affected by #4358 - merging.


Automated cherry pick of #3260: fix: The domain and project info between disk and disk's cloudprovider maybe different and the new snapshot synced from cloud should have same domain and project info with disk. Project of snapshot with undeleted disk should always be same with disk's. Project of snapshot whose disk has been deleted should not change in SyncWithCloudSnapshot.

Cherry pick of #3260 on release/2.12.
#3260: fix: The domain and project info between disk and disk's cloudprovider maybe different and the new snapshot synced from cloud should have same domain and project info with disk. Project of snapshot with undeleted disk should always be same with disk's. Project of snapshot whose disk has been deleted should not change in SyncWithCloudSnapshot.

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.12@6a69fe0). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.12   #3313   +/-   ##
==============================================
  Coverage                ?   7.31%
==============================================
  Files                   ?     490
  Lines                   ?   78986
  Branches                ?       0
==============================================
  Hits                    ?    5780
  Misses                  ?   72719
  Partials                ?     487

Flag
Coverage Δ

#aFlag
7.31% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/disks.go
0.64% <0%> (ø)

pkg/compute/models/snapshots.go
1.61% <0%> (ø)

Continue to review full report at Codecov.

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

/lgtm
/approve


refactor: replace regular prints with prints to stderr for warnings

I'll replace those of split_edge methods in another PR

Codecov Report
Attention: Patch coverage is 13.63636% with 19 lines in your changes missing coverage. Please review.

Project coverage is 81.48%. Comparing base (1eb96db) to head (7bcc253).

Files with missing lines
Patch %
Lines

honeycomb-core/src/attributes/manager.rs
0.00%
10 Missing :warning:

honeycomb-core/src/attributes/collections.rs
25.00%
6 Missing :warning:

honeycomb-kernels/src/grisubal/model.rs
0.00%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #192      +/-   ##
==========================================
- Coverage   81.51%   81.48%   -0.04%
==========================================
  Files          36       36
  Lines        5329     5331       +2
==========================================
  Hits         4344     4344
- Misses        985      987       +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
ffeb38a -> 0f1f11e

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.42.2 -> v1.42.3

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.42.3
Compare Source
Bug Fixes

deps: update module github.com/ibm/platform-services-go-sdk to v0.72.0 (#901) (6b14632)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

ℹ Artifact update notice
File name: tests/go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

1 additional dependency was updated

Details:

Package
Change

github.com/IBM/platform-services-go-sdk
v0.71.1 -> v0.72.0

/run pipeline

ℹ Artifact update notice
File name: tests/go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

1 additional dependency was updated

Details:

Package
Change

github.com/IBM/platform-services-go-sdk
v0.71.1 -> v0.72.0

/run pipeline

/run pipeline

ℹ Artifact update notice
File name: tests/go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

1 additional dependency was updated

Details:

Package
Change

github.com/IBM/platform-services-go-sdk
v0.71.1 -> v0.72.0

/run pipeline

/run pipeline

ℹ Artifact update notice
File name: tests/go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

1 additional dependency was updated

Details:

Package
Change

github.com/IBM/platform-services-go-sdk
v0.71.1 -> v0.72.0

/run pipeline

/run pipeline

ℹ Artifact update notice
File name: tests/go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

1 additional dependency was updated

Details:

Package
Change

github.com/IBM/platform-services-go-sdk
v0.71.1 -> v0.72.0

/run pipeline

:tada: This PR is included in version 3.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


move to stac 1.0.0 (update pystac)

https://github.com/radiantearth/stac-spec/releases/tag/v1.0.0

done in #11


Scoped session

Updates to scoped_session

Codecov Report

Merging #5 into master will increase coverage by 0.87%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
+ Coverage   61.71%   62.58%   +0.87%
==========================================
  Files         126      127       +1
  Lines       15560    15640      +80
==========================================
+ Hits         9603     9789     +186
+ Misses       5957     5851     -106

Impacted Files
Coverage Δ

cairis/core/MySQLDatabaseProxy.py
53.69% <100%> (+1.16%)
:arrow_up:

cairis/core/ObstacleFactory.py
100% <0%> (ø)

cairis/core/Steps.py
76.47% <0%> (+5.88%)
:arrow_up:

cairis/core/Step.py
86.11% <0%> (+13.88%)
:arrow_up:

cairis/core/WeaknessTarget.py
85.71% <0%> (+32.14%)
:arrow_up:

Continue to review full report at Codecov.

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


Is there any solution to testing?

I am using Jest to test my React application and I'm getting this error on @uiball library. I have already set a lot of configuration lines and nothing works.

This package is pure ESM, meaning that a CommonJS export isn't available. You can learn about using ESM in the Jest docs: https://jestjs.io/docs/ecmascript-modules


Support modules in JavaScript workers

Is your feature request related to a problem? Please describe.
At a first approach, the JavaScript workers follow the Web Workers API. It requires to subscribe to a specific event and then, process the request and provide a response:
https://github.com/vmware-labs/wasm-workers-server/blob/51da71082e4cd41d787b88354d1e14c69ce22f43/examples/js-params/fixed.js#L27-L30
However, the current JavaScript SDK emulates this behavior, so it doesn't provide any Web API behind it. In addition to that, other platforms like Cloudflare and Vercel started to simplify this syntax in favor of ECMA modules:
export default {
  async fetch(request) {
    const body = `Hello world!`;
    return new Response(body);
  },
};

This syntax is simpler and more convenient.
Describe the solution you'd like
I want to write workers using the ECMA modules approach, while keeping compatibility with existing modules based on the Web Workers API.
It requires to change the current JavaScript SDK to:

[ ] Manage ECMA modules
[ ] Detect the current approach
[ ] Run the worker to get the response

Describe alternatives you've considered
No response
Additional context
No response

Why export an object and not the fetch function directly?
Is it a standard that emerges from other platforms?

Why export an object and not the fetch function directly? Is it a standard that emerges from other platforms?

Yes! One of the goals of wws is to keep compatibility with other platforms. For example, you can take a worker from wws and run it in Cloudflare or Fastly. We're following the WinterCG standard and taking Cloudflare as a reference for the syntax on the JavaScript workers: https://developers.cloudflare.com/workers/examples/return-html/.

I'm really happy to close this task! 👏


Fixed Typos

Hi Aleksandar!
I just found your package and tried to follow the README instructions. There I detected that the package name differs from the instructions. I suppose you renamed the package, that's why I also fixed the title and the import name.
Thanks a lot for this package,
Bernhard

Ah, yes. Thank you Bernhard!
If you have any other issues, please let me know.

Thank you, everything is fine and works without any flaws! 👏


Added Python: 1.5 Brief History of Python

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Make your contribution as mentioned in discord channel.

Make your contribution as mentioned in discord channel.
I have followed all the checkpoints. Can u please tell what more to add?

Add your file in proper folder & update index in readme

Add your file in proper folder & update index in readme

Done. Check and merge @adityabisoi

There are two files with the same content and the index in README hasn't been updated. Double-check your commits before pushing.


Add official pretrained weights for Autoformer

Description
Add subnet and supernet weights from offical repo for autoformer. Add interface for official model space config.
Test Options

[x] fast test
[x] full test - NAS

Checklist

[ ] test case
[ ] doc

How to test
from nni.retiarii.hub.pytorch import AutoformerSpace
for name in ['tiny', 'small', 'base']:
    # check subnet weights load
    AutoformerSpace.load_searched_model(name, pretrained = True, download = True)
    AutoformerSpace.load_strategy_checkpoint(name)

/azp run full test - NAS

/azp run full test - NAS

I'll further improve related features in #5096.


Can u update for NG4

currently this wrapper is not working for ng4 can you please upload

HI dear
if you are using Angular 4 or more the Angular 2.0
please keep in you mind use
select.full.js   not select.min.js

@karangoswami31  Does this mean I have to download ng2-select2 library and manually include select.full.js by myself?

Hi @sh977218
yes are correct

Why is it not available as npm dependency?

please add dependence
On Wed, Aug 9, 2017 at 6:18 PM, Daniel Kiesel<EMAIL_ADDRESS>wrote:

Why is it not available as npm dependency?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NejcZdovc/ng2-select2/issues/77#issuecomment-321245277,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHFRxfLVDuBDx7JEo5Ic8rcM-EehVc-Vks5sWaqwgaJpZM4M927N
.


2.0: Thread Safe Replicator

https://github.com/couchbase/couchbase-lite-core/wiki/Thread-Safety#document-exclusive

https://github.com/couchbase/couchbase-lite-ios/pull/2046

Make the replicator thread-safe by using the lock.
Fix the stop status not getting reported when calling replicator.stop() after the replicator is in the offline state.


Allow user's esbuild config to override platform

This is a simple change that won't be useful for most users of this library. However, I'm trying to use karma-esbuild with some tests that run in an Electron app that has full Node.js access within its renderer.
While running Karma tests that're compiled with karma-esbuild, I get errors like:
ERROR [esbuild]: Build failed with 1 error:
src/helpers/fs.ts:1:40: error: Could not resolve "fs" (use "platform: 'node'" when building for node)

This PR lets the user's config override the default browser platform. I looked into adding tests for this, but the best way to do it would be to setup a whole Electron config within a fixture, which might be an extra dependency you'd rather not have here for a single line getting moved.
Let me know and I can make it happen! Thanks to everyone that has worked on this library and made it so easy to integrate.

I wonder if there's a way to add a test case for this.


Add enum PersonGender to CRMScript (#723)

@SuperOfficeDevNet should we add something back to CRMScript.NetServer.AssociateType now that the enum is available?

@SuperOfficeDevNet should we add something back to CRMScript.NetServer.AssociateType now that the enum is available?

No, it's not referenced in Associate at all - it's a code generation bug..


module orangepi crash

[x] I have read the README.md file and visited the Documentation to see if the information there helps.

Describe the bug
i just want to tweak the GPIO on my orange pi 3 lts. Took an example and it doesn't work.
Error messages and traceback
s6-rc: info: service s6rc-oneshot-runner: starting
s6-rc: info: service s6rc-oneshot-runner successfully started
s6-rc: info: service fix-attrs: starting
s6-rc: info: service fix-attrs successfully started
s6-rc: info: service legacy-cont-init: starting
cont-init: info: running /etc/cont-init.d/00-banner.sh
-----------------------------------------------------------
 Add-on: MQTT IO
 Expose GPIO modules and digital sensors via MQTT for remote control and monitoring.
-----------------------------------------------------------
 Add-on version: 0.1.3
 You are running the latest version of this add-on.
 System: Ubuntu 22.04.1 LTS  (aarch64 / odroid-c2)
 Home Assistant Core: 2022.9.6
 Home Assistant Supervisor: 2022.09.1
-----------------------------------------------------------
 Please, share the above information when looking for help
 or support in, e.g., GitHub, forums or the Discord chat.
-----------------------------------------------------------
cont-init: info: /etc/cont-init.d/00-banner.sh exited 0
cont-init: info: running /etc/cont-init.d/01-log-level.sh
Log level is set to DEBUG
cont-init: info: /etc/cont-init.d/01-log-level.sh exited 0
cont-init: info: running /etc/cont-init.d/mqtt-io.sh
cont-init: info: /etc/cont-init.d/mqtt-io.sh exited 0
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service legacy-services: starting
services-up: info: copying legacy longrun mqtt-io (no readiness notification)
s6-rc: info: service legacy-services successfully started
[20:09:07] INFO: Starting MQTT IO...
2022-09-24 20:09:16 mqtt_io.__main__ [ERROR] MqttIo crashed!
Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/mqtt_io/__main__.py", line 107, in main
    mqtt_gpio.run()
  File "/usr/lib/python3.10/site-packages/mqtt_io/server.py", line 1227, in run
    self._init_gpio_modules()
  File "/usr/lib/python3.10/site-packages/mqtt_io/server.py", line 244, in _init_gpio_modules
    self.gpio_modules[gpio_config["name"]] = _init_module(
  File "/usr/lib/python3.10/site-packages/mqtt_io/server.py", line 122, in _init_module
    module_config = validate_and_normalise_config(module_config, module_schema)
  File "/usr/lib/python3.10/site-packages/mqtt_io/config/__init__.py", line 105, in validate_and_normalise_config
    raise ConfigValidationFailed(
mqtt_io.exceptions.ConfigValidationFailed: Config did not validate:
board:
- required field
Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/usr/lib/python3.10/site-packages/mqtt_io/__main__.py", line 115, in <module>
    main()
  File "/usr/lib/python3.10/site-packages/mqtt_io/__main__.py", line 107, in main
    mqtt_gpio.run()
  File "/usr/lib/python3.10/site-packages/mqtt_io/server.py", line 1227, in run
    self._init_gpio_modules()
  File "/usr/lib/python3.10/site-packages/mqtt_io/server.py", line 244, in _init_gpio_modules
    self.gpio_modules[gpio_config["name"]] = _init_module(
  File "/usr/lib/python3.10/site-packages/mqtt_io/server.py", line 122, in _init_module
    module_config = validate_and_normalise_config(module_config, module_schema)
  File "/usr/lib/python3.10/site-packages/mqtt_io/config/__init__.py", line 105, in validate_and_normalise_config
    raise ConfigValidationFailed(
mqtt_io.exceptions.ConfigValidationFailed: Config did not validate:
board:
- required field
[20:09:16] WARNING: MQTT IO crashed, halting add-on
s6-rc: info: service legacy-services: stopping
[20:09:17] INFO: MQTT IO stopped
s6-rc: info: service legacy-services successfully stopped
s6-rc: info: service legacy-cont-init: stopping
s6-rc: info: service legacy-cont-init successfully stopped
s6-rc: info: service fix-attrs: stopping
s6-rc: info: service fix-attrs successfully stopped
s6-rc: info: service s6rc-oneshot-runner: stopping
s6-rc: info: service s6rc-oneshot-runner successfully stopped

Config
mqtt:
  host: localhost
  port: 1883
  user: "*****"
  password: "*******"
  topic_prefix: home/mqtt_io

gpio_modules:
  - name: orgpio
    module: orangepi

digital_outputs:
  - name: fans
    module: orgpio
    pin: 1
    on_payload: "ON"
    off_payload: "OFF"
    pullup: yes

Hardware
Orange pi 3 lts
System:
Home Assistant 2022.9.6
Supervisor 2022.09.1
Frontend 20220907.2 - latest
Integration MQTT IO
Current version: 0.1.3

I was just trying to set this up with opi zero and it seems there is a few issues with this module.  It adds board and mode to the config schema but also references old OrangePi.GPIO module as requirement - but uses OPi.GPIO.
I can't see any use for the board option as there is no setboard for OPi.GPIO.
mode needs to be one of [BCM, BOARD, SUNXI, CUSTOM]
Below patch removes the board setting and fixes requirement module - this works for me.  I will try and do a PR later.
--- orangepi.py.orig    2022-10-10 09:44:28.139118027 +1100
+++ orangepi.py 2022-10-10 09:51:30.311322016 +1100
@@ -7,28 +7,9 @@
 from ...types import ConfigType, PinType
 from . import GenericGPIO, PinDirection, PinPUD

-ALLOWED_BOARDS = [
-    "zero",
-    "r1",
-    "zeroplus",
-    "zeroplus2h5",
-    "zeroplus2h3",
-    "pcpcplus",
-    "one",
-    "lite",
-    "plus2e",
-    "pc2",
-    "prime",
-]
 ALLOWED_MODES = ["bcm", "board", "mode_soc"]
-REQUIREMENTS = ("OrangePi.GPIO",)
+REQUIREMENTS = ("OPi.GPIO",)
 CONFIG_SCHEMA = {
-    "board": {
-        "type": "string",
-        "required": True,
-        "empty": False,
-        "allowed": ALLOWED_BOARDS + list(map(str.upper, ALLOWED_BOARDS)),
-    },
     "mode": {
         "type": "string",
         "required": True,
@@ -58,12 +39,7 @@
             PinPUD.UP: gpio.PUD_UP,
             PinPUD.DOWN: gpio.PUD_DOWN,
         }
-
-        board = self.config["board"].upper()
         mode = self.config["mode"].upper()
-        if not hasattr(gpio, board):
-            raise AssertionError("%s board not found" % board)
-        gpio.setboard(getattr(gpio, board))
         gpio.setmode(getattr(gpio, mode))

     def setup_pin(

And your config will need to specify mode (which essentially identifies the pin numbering)
gpio_modules:
  - name: orangepi
    module: orangepi
    mode: bcm


Add Pride Cloaks To Loadouts

About the PR

Adds the ability to spawn with a pride cloak
Why / Balance

saiirayn (Synth Enjoyer) asked me to do it and i said yes (i wanted it too) most cloaks are too hard to find anyways
Technical details

i uh, added to the yml
Media

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

Breaking changes

id be reallllll suprised if this broke somethin
Changelog

:cl: BITTERLYNX

tweak: Pride cloaks have been added to crew loadouts

There are a few conceptual issues with this:

People should not be able to spawn with a cloak at will (at least as something without a timegate.)
These items are likely intentionally rare. This massively devalues them for no real reason - we already have pride pins, anyways. Not everything needs to be in loadouts.

Thank you for your contribution. I'll be closing this PR due to the reasons above.


Zjpm: nit - small note on IOMMU and memory accesses

Although it is quite obvious that zjpm spec concerns only with CPU based accesses.
Putting a note that device side accesses for shared virtual memory will be addressed by non-ISA spec will be useful.
Use cases like opencl [1] heavily use shared virtual memory between device side and cpu side.
[1] https://www.intel.com/content/www/us/en/developer/articles/technical/opencl-20-shared-virtual-memory-overview.html

Addressed in fd7220807e1405ba9b5de38d9d4ac26e73312128 (Section 2.6)

Instead of saying this
Pointer masking only applies to accesses generated by instructions on the CPU (including CPU extensions such as an FPU). For example, it does not apply to accesses generated by the IOMMU or devices
It should be on the line
Zjpm specification concerns with pointer masking applied to memory accesses generated by instructions executing on the hart. Pointer masking on memory accesses from devices may be covered by non-ISA specifications like IOMMU specification

Quickly checking in about this item whether you still have this concern about the latest version of the spec. The suggestion from the architecture review committee was to remove this paragraph altogether since it is implied by the fact that pointer masking is an ISA extension.

Yeah it was a nit anyways.
That's fine by me if we close it.


add linearizedproductof to lowprioritycandidates for sample list form constraint

See #340 for an explanation for this PR

Perhaps this PR can also add ProductOf to the list?

ProductOf is covered by :
https://github.com/ReactiveBayes/RxInfer.jl/blob/318bd43a2e686e2150299a0295348a20cec8db21/src/constraints/form/form_sample_list.jl#L72-L76
We can add it to the list if you want, but I don't think it will ever be called.

I also wonder why this error message hasn't been showed in the #340 , perhaps the logic can be improved to guide users towards LeftProposal/RightProposal

But it will be covered since that function is recursive

But it will be covered since that function is recursive

ah, you are right, my bad


Update CI versions and steps

References:

https://peps.python.org/pep-0569/

Closing and reopening to awaken CI.

Extra updates from case-utils's build process have been pulled in.


File menu option vanishes after hover.

Issue Type: Bug
file menu item vanishes after hover
same is case for view, go, run, terminal options as well
edit, selection and help are working normally.
i have the file menu in the sidebar
the working screenshot for Edit menu item

VS Code version: Code - Insiders 1.48.0-insider (20cba51fb305868238eddc69758d259c29b3230d, 2020-08-07T05:35:24.758Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8265U CPU @ 1.60GHz (8 x 1800)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.87GB (7.55GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (21)

Extension
Author (truncated)
Version

ng-template
Ang
0.1000.7

vscode-eslint
dba
2.1.8

gitlens
eam
10.2.2

EditorConfig
Edi
0.15.1

vscode-npm-script
eg2
0.3.13

prettier-vscode
esb
5.1.3

output-colorizer
IBM
0.1.2

remote-containers
ms-
0.132.0

remote-ssh
ms-
0.51.0

remote-wsl
ms-
0.44.4

vscode-remote-extensionpack
ms-
0.20.0

typescript-javascript-grammar
ms-
0.0.53

vscode-typescript-tslint-plugin
ms-
1.2.3

cordova-tools
msj
1.9.5

debugger-for-chrome
msj
4.12.9

vscode-xml
red
0.13.0

vscode-yaml
red
0.9.1

sonarlint-vscode
Son
1.17.0

code-spell-checker
str
1.9.0

vscode-stylelint
sty
0.84.0

vscodeintellicode
Vis
1.2.9

@sbatten , is this fix going to be in the insiders build (August 10)?

At first it seems to work, but hovering around on multiple menus (including deeply nested ones) and then coming back to say the File menu causes the following errors:
bootstrap-window.js:216 [uncaught exception]: NotFoundError: Failed to execute 'removeChild' on 'Node': The node to be removed is no longer a child of this node. Perhaps it was moved in a 'blur' event handler?
onUnexpectedError @ bootstrap-window.js:216
bootstrap-window.js:219 Error: Failed to execute 'removeChild' on 'Node': The node to be removed is no longer a child of this node. Perhaps it was moved in a 'blur' event handler?
    at Object.removeNode (/C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/dom.js:17)
    at Menu.dispose (/C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/ui/actionbar/actionbar.js:352)
    at SubmenuMenuActionViewItem.cleanupExistingSubmenu (/C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/ui/menu/menu.js:592)
    at RunOnceScheduler.runner (/C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/ui/menu/menu.js:523)
    at RunOnceScheduler.doRun (/C:/Users/Eric/code/microsoft/vscode/out/vs/base/common/async.js:525)
    at RunOnceScheduler.onTimeout (/C:/Users/Eric/code/microsoft/vscode/out/vs/base/common/async.js:520)
onUnexpectedError @ bootstrap-window.js:219
/C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/dom.js:17 Uncaught DOMException: Failed to execute 'removeChild' on 'Node': The node to be removed is no longer a child of this node. Perhaps it was moved in a 'blur' event handler?
    at Object.removeNode (file:///C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/dom.js:17:29)
    at Menu.dispose (file:///C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/ui/actionbar/actionbar.js:352:17)
    at SubmenuMenuActionViewItem.cleanupExistingSubmenu (file:///C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/ui/menu/menu.js:592:41)
    at RunOnceScheduler.runner (file:///C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/ui/menu/menu.js:523:26)
    at RunOnceScheduler.doRun (file:///C:/Users/Eric/code/microsoft/vscode/out/vs/base/common/async.js:525:22)
    at RunOnceScheduler.onTimeout (file:///C:/Users/Eric/code/microsoft/vscode/out/vs/base/common/async.js:520:22)
bootstrap-window.js:216 [uncaught exception]: NotFoundError: Failed to execute 'removeChild' on 'Node': The node to be removed is no longer a child of this node. Perhaps it was moved in a 'blur' event handler?
onUnexpectedError @ bootstrap-window.js:216
bootstrap-window.js:219 Error: Failed to execute 'removeChild' on 'Node': The node to be removed is no longer a child of this node. Perhaps it was moved in a 'blur' event handler?
    at Object.removeNode (/C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/dom.js:17)
    at Menu.dispose (/C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/ui/actionbar/actionbar.js:352)
    at SubmenuMenuActionViewItem.cleanupExistingSubmenu (/C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/ui/menu/menu.js:592)
    at RunOnceScheduler.runner (/C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/ui/menu/menu.js:523)
    at RunOnceScheduler.doRun (/C:/Users/Eric/code/microsoft/vscode/out/vs/base/common/async.js:525)
    at RunOnceScheduler.onTimeout (/C:/Users/Eric/code/microsoft/vscode/out/vs/base/common/async.js:520)
onUnexpectedError @ bootstrap-window.js:219
/C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/dom.js:17 Uncaught DOMException: Failed to execute 'removeChild' on 'Node': The node to be removed is no longer a child of this node. Perhaps it was moved in a 'blur' event handler?
    at Object.removeNode (file:///C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/dom.js:17:29)
    at Menu.dispose (file:///C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/ui/actionbar/actionbar.js:352:17)
    at SubmenuMenuActionViewItem.cleanupExistingSubmenu (file:///C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/ui/menu/menu.js:592:41)
    at RunOnceScheduler.runner (file:///C:/Users/Eric/code/microsoft/vscode/out/vs/base/browser/ui/menu/menu.js:523:26)
    at RunOnceScheduler.doRun (file:///C:/Users/Eric/code/microsoft/vscode/out/vs/base/common/async.js:525:22)
    at RunOnceScheduler.onTimeout (file:///C:/Users/Eric/code/microsoft/vscode/out/vs/base/common/async.js:520:22)


dbus errors during VS Code installation

Distributor ID:	Ubuntu
Description:	Ubuntu 20.04.6 LTS
Release:	20.04
Codename:	focal
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>Getting these errors during runTests execution. What causes them?
`Downloading VS Code 1.90.2 from https://update.code.visualstudio.com/1.90.2/linux-x64/stable
Downloading VS Code (143925020B)
Downloading VS Code [------------------------------] 0%Downloading VS Code [------------------------------] 0%Downloading VS Code [------------------------------] 0%Downloading VS Code [------------------------------] 1%Downloading VS Code [------------------------------] 1%Downloading VS Code [------------------------------]
...
[=============================-] 100%Downloading VS Code [=============================-] 100%Downloading VS Code [=============================-] 100%Downloaded VS Code into /data/workspace/XXXXXXXX/extensions/vscode/.vscode-test/vscode-linux-x64-1.90.2
Downloaded VS Code into /data/workspace/XXXXXXXXX/extensions/vscode/.vscode-test/vscode-linux-x64-1.90.2
Downloading VS Code [==============================] 100%[4831:0623/230859.727074:ERROR:bus.cc(407)] Failed to connect to the bus: Failed to connect to socket /var/run/dbus/system_bus_socket: No such file or directory
[4831:0623/230901.665173:ERROR:bus.cc(407)] Failed to connect to the bus: Failed to connect to socket /var/run/dbus/system_bus_socket: No such file or directory
[4831:0623/230901.665222:ERROR:bus.cc(407)] Failed to connect to the bus: Failed to connect to socket /var/run/dbus/system_bus_socket: No such file or directory
[4831:0623/230901.667020:ERROR:bus.cc(407)] Failed to connect to the bus: Could not parse server address: Unknown address type (examples of valid types are "tcp" and on UNIX "unix")
[4831:0623/230901.667161:ERROR:bus.cc(407)] Failed to connect to the bus: Could not parse server address: Unknown address type (examples of valid types are "tcp" and on UNIX "unix")
[4831:0623/230901.667207:ERROR:bus.cc(407)] Failed to connect to the bus: Could not parse server address: Unknown address type (examples of valid types are "tcp" and on UNIX "unix")
[4831:0623/230901.667250:ERROR:bus.cc(407)] Failed to connect to the bus: Could not parse server address: Unknown address type (examples of valid types are "tcp" and on UNIX "unix")
[4831:0623/230901.940150:ERROR:bus.cc(407)] Failed to connect to the bus: Failed to connect to socket /var/run/dbus/system_bus_socket: No such file or directory
[4831:0623/230901.940203:ERROR:bus.cc(407)] Failed to connect to the bus: Failed to connect to socket /var/run/dbus/system_bus_socket: No such file or directory
[4831:0623/230901.940228:ERROR:bus.cc(407)] Failed to connect to the bus: Failed to connect to socket /var/run/dbus/system_bus_socket: No such file or directory
[main 2024-06-24T03:09:02.112Z] update#setState disabled
[main 2024-06-24T03:09:02.113Z] update#ctor - updates are disabled by the environment
[4831:0623/230902.115171:ERROR:bus.cc(407)] Failed to connect to the bus: Could not parse server address: Unknown address type (examples of valid types are "tcp" and on UNIX "unix")
[4855:0623/230902.378894:ERROR:viz_main_impl.cc(196)] Exiting GPU process due to errors during initialization
[4878:0623/230902.493068:ERROR:command_buffer_proxy_impl.cc(131)] ContextResult::kTransientFailure: Failed to send GpuControl.CreateCommandBuffer.
workbench#open()
[4831:0623/230904.215817:ERROR:bus.cc(407)] Failed to connect to the bus: Could not parse server address: Unknown address type (examples of valid types are "tcp" and on UNIX "unix")
Started local extension host with pid 4952.
IExtensionHostStarter.start() took 37 ms.
`

I believe these come from Electron and are generally not something you need to worry about (look like you're running with verbose logging.) dbus is used for all sorts of things to interact with the Linux host, but VS Code should handle a lack of dbus just fine. Are you running into problems?

I believe these come from Electron and are generally not something you need to worry about (look like you're running with verbose logging.) dbus is used for all sorts of things to interact with the Linux host, but VS Code should handle a lack of dbus just fine. Are you running into problems?

Thanks @connor4312 I have no screenshots for my vs code extension tests, so I don't know if there are issues. I thought this was an issue, since these are errors. IF there is some way to disable the GPU and also add a valid address, maybe that will make the errors stop.
But thanks for the tip on simply ignoring this.

You can pass electron/chromium flags like --disable-gpu, but I don't know if all behaviors that are causing those logging are able to be disabled.


Russian language not working with Tera Copy

It seems like that TeraCopy does not accept the files with Russian titles in the file lists.

Changed the file format from UTF-8 to UTF-16 with little endian.


Production can't upload images

Fixed by using basic varchar database type, no blobs or byte arrays


Fix API docs

Proposed changes
The docs won't compile in RTD because of the src structure.
Types of changes
What types of changes does your code introduce to FESTIM?

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Code refactoring
[x] Documentation Update (if none of the other choices apply)
[ ] New tests

The docs now compile correctly:


Callback when command processed

Is there anyway to run some custom code after certain commands have run.
IE. I'd like to kick off another part of my app after a file has been uploaded
Only way so far I can tell would be to create my own commandhandlerfactory and provide my own implementations of the existing commands using inheritance.

You're right that this isn't supported yet. Implementing your own IFtpCommandHandlerFactory and returning your own command (which redirects every property and function to the original handler) is currently the only way to achieve this.
I'm currently developing a rewrite of the FTP server using DotNetty which should also support .NET Standard 1.3 and this could be easily done, because you can modify the DotNetty pipeline and intercept the FTP commands (and their responses).

//-------------------
public class CustomAssemblyFtpCommandHandlerFactory : FubarDev.FtpServer.AssemblyFtpCommandHandlerFactory
{
public CustomAssemblyFtpCommandHandlerFactory([NotNull] Assembly assembly, [ItemNotNull, NotNull] params Assembly[] assemblies) : base(assembly, assemblies)
{
}
    public override IEnumerable<FtpCommandHandlerExtension> CreateCommandHandlerExtensions(FtpConnection connection)
    {
        return base.CreateCommandHandlerExtensions(connection);
    }

    public override IEnumerable<FtpCommandHandler> CreateCommandHandlers(FtpConnection connection)
    {
        var baseHandlers = base.CreateCommandHandlers(connection);
        var ret = baseHandlers.Select(each => new CustomCommandDecorator(each, connection, each.Names.First(), each.Names.Skip(1).ToArray()));
        return ret;
    }

    public class CustomCommandDecorator : FtpCommandHandler
    {
        FtpCommandHandler baseFtpCommandHandler;

        public CustomCommandDecorator([NotNull] FtpCommandHandler baseFtpCommandHandler, [NotNull] FtpConnection connection, [NotNull] string name, [ItemNotNull, NotNull] params string[] alternativeNames) : base(connection, name, alternativeNames)
        {
            this.baseFtpCommandHandler = baseFtpCommandHandler;
        }

        public async override Task<FtpResponse> Process([NotNull] FtpCommand command, CancellationToken cancellationToken)
        {
            try
            {
                System.Diagnostics.Debug.WriteLine("FTP COMMAND: " + command.ToString());
                var ret = await this.baseFtpCommandHandler.Process(command, cancellationToken);
                System.Diagnostics.Debug.WriteLine("FTP COMMAND RESULT: " + ret.ToString());
                return ret;
            }
            catch (Exception ex)
            {
                System.Diagnostics.Debug.Fail(ex.ToString());
                throw;
            }
        }

        public override bool IsAbortable
        {
            get
            {
                return this.baseFtpCommandHandler.IsAbortable;
            }
        }

        public override bool IsLoginRequired
        {
            get
            {
                return this.baseFtpCommandHandler.IsLoginRequired;
            }
        }
    }
}

//-----------------------
    protected virtual void StartFtpServer(FubarDev.FtpServer.FileSystem.IFileSystemClassFactory fileSystemProvider)
    {
        //// Load server certificate
        //// var cert = new X509Certificate2("test.pfx");
        //// AuthTlsCommandHandler.ServerCertificate = cert;

        //// Only allow anonymous login
        var membershipProvider = new AnonymousMembershipProvider(new NoValidation());

        //// Use all commands from the FtpServer assembly and the one(s) from the AuthTls assembly
        var commandFactory = new CustomAssemblyFtpCommandHandlerFactory(typeof(FtpServer).Assembly, typeof(AuthTlsCommandHandler).Assembly);

        //// Initialize the FTP server
        this.ftpServer = new FtpServer(fileSystemProvider, membershipProvider, "<IP_ADDRESS>", (int)this.port, commandFactory)
        {
            DefaultEncoding = Encoding.Default,
            LogManager = new FtpLogManager(),
        };

#if USE_FTPS_IMPLICIT
//// Use an implicit SSL connection (without the AUTHTLS command)
ftpServer.ConfigureConnection += (s, e) =>
{
var sslStream = new FixedSslStream(e.Connection.OriginalStream);
sslStream.AuthenticateAsServer(cert);
e.Connection.SocketStream = sslStream;
};
#endif
        try
        {
            this.ftpServer.Start();
        }
        catch (Exception ex)
        {
            ex.Process();
        }
    }

//------------------------

Hello. Do you have any update regarding this issue? It looks like the code sample provided is not compatible with version 3 any more...

You can intercept a command by implementing either a IFtpMiddleware or an IFtpCommandMiddleware. The difference between the two is described in the upgrade guide.
That way, you can intercept the FTP command (and replace it with your own implementation), but you cannot intercept the response, because those is returned through a different channel. To intercept the FTP servers responses, you would need to implement your own IServerCommandFeature and set this feature during the connection initialization in the IFtpServer.ConfigureConnection event.
The main reason for decoupling command execution and sending the response was that I wanted to streamline communication between the connection and the core server functionality. This feature was extremely important for the AUTH TLS and REIN implementations.

For my case I need to catch the STOR command and get the file path on the server and its size. I was able to get the file name with the IFtpMiddleware/IFtpCommandMiddleware but not the size of an uploaded file.
What is the best way to do it?

The file size isn't part of the FTP protocol and the only way to handle this use case is to implement your own STOR command handler. IFtpCommandHandlerProvider is the new extension point where you return the new FTP commands.
Example
Custom FTP command  handler provider
class MyFtpCommnandHandlerProvider : IFtpCommandHandlerProvider
{
    public MyFtpCommnandHandlerProvider(DefaultFtpCommandHandlerProvider defaultProvider)
    {
        IFtpCommandHandlerInformation yourStorCommandInformation = ... /* needs to be implemented by you */;
        CommandHandlers = defaultProvider.CommandHandlers
                .Where(x => x.Name != "STOR")
                .Concat(new[] { yourStorCommandInformation }
                .ToList();
    }

    /// <inheritdoc />
    public IEnumerable<IFtpCommandHandlerInformation> CommandHandlers { get; }
}

Changes to the registration
services
    .AddFtpServer(... /* yadda, yadda */)
    /* Add your custom FTP command handler provider */
    .AddScoped<IFtpCommandHandlerProvider, MyFtpCommandHandlerProvider>()
    /* Register the default provider to get the default FTP command handlers */
    .AddScoped<DefaultFtpCommandHandlerProvider>();

It looks like InvokeAsync of IFtpCommandMiddleware is called before the file is uploaded. Therefore, I cannot obtain the file size using the path of the uploaded file.
Is there any sample which can be used as a starting point for custom implementation of IFtpCommandHandlerInformation?

You can. Yes, it's called before it's uploaded, but you can await the end of the command execution.
There are two interfaces that you can implement: IFtpCommandHandlerInformation and IFtpCommandHandlerInstanceInformation, but you have to register it always as IFtpCommandHandlerInformation.

A simple implementation might look like:
class MyStorCommandHandlerInformation : IFtpCommandHandlerInformation
{
    /// <inheritdoc />
    public string Name { get; } = "STOR";

    /// <inheritdoc />
    public bool IsLoginRequired { get; } = true;

    /// <inheritdoc />
    public bool IsAbortable { get; } = true;

    /// <inheritdoc />
    public Type Type { get; } = typeof(MyStorCommandHandler);

    /// <inheritdoc />
    public bool IsExtensible { get; } = false;
}

Now you have to implement MyStorCommandHandler. You may copy the original implementation and modify it.

Ok, I took a look at the source code and I was right. You can do the following in your IFtpCommandMiddleware:
class YourMiddleware : IFtpCommandMiddleware
{
    public async Task InvokeAsync(FtpExecutionContext context, FtpCommandExecutionDelegate next)
    {
        // Do some stuff before command gets executed
        await next(context);
        // Do some stuff after the command got executed
    }
}

The above works even when the command is a background command. The code after the await will definitely be executed after the command was processed.

Thanks a lot for your clarifications. I was able to implement my logic with a custom IFtpCommandMiddleware.

The above works even when the command is a background command. The code after the await will definitely be executed after the command was processed.

Yes the commands are all hit before the next line of code, however testing this with the S3 provider and no, the code after the await next(context) continues BEFORE the S3 provider uploads the file (e.g. before CreateAsync is called). Is there something else I'm missing perhaps? Or is the only reliable way to do this with a custom provider?
I really think it would be a common requirement to perform some action after the successfull completion of an upload/delete whatever. Perhaps an additional middleware that occurs afterwards or an addition to the IFtpCommandMiddleware?


Add ANTLR language

Fixes #1290
I did not implement all the styles in that issue, but this does correctly recognize ANTLR, highlight grammar rules, recognize comments, and add ANTLR as a first class citizen for the set of languages that can be processed.

@BillWagner
First of all, truly sorry for the long delay in anyone responding to this PR.
Thank you so much for desiring to contribute and writing a language grammar.  Unfortunately, we currently are no longer accepting new languages into the core repository.  The existing library maintainers don't simply have time to maintain any new languages (fixing bugs, dealing with issues, etc).  We'd still love to host your language as a 3rd party module though - if you're willing to help maintain it.
It's easy to turn your language into a module that anyone who likes can easily integrate with Highlight.js using a single line of code:
<script src="/path/to/highlight.min.js"></script>
<!-- they'd simply add this single line -->
<script src="/path/to/highlightjs-antlr.js"></script>
<script>hljs.initHighlightingOnLoad();</script>

If you'd like to become the maintainer for this language then we're happy to create a new repository in the highlightjs organization and host the repository there.  That assumes you'd be open to reviewing issues and fixing bugs as they arise.  You're also welcome to host the repo yourself and we'll link to it from our README.
There isn't much in the way of how to docs on this yet, but there are sample repositories and it's honestly pretty easy.  We're still fleshing out an official "spec" for 3rd party modules, but even after that's done it should require minimal changes from the examples we have already.  IE, if you wanted to get started now there isn't any big reason to wait.
My own robots.txt grammar repository is a great, simple example:
https://github.com/highlightjs/highlightjs-robots-txt
If you have any questions feel free to ask.

Closing for inactivity; (also finally cleaning all these old PRs out for the new year).
As mentioned above if you're interested in becoming the maintainer of a 3rd party module, please respond [whenever you find time] and we'd be happy to help you get started.


Logic Apps on AKS scenario fails to complete on client VM

Scenario which you are working on
https://azurearcjumpstart.io/azure_arc_jumpstart/azure_arc_app_svc/aks/aks_logic_app_arm_template/
Describe the bug
After logging into the client VM and initialising the AppSvc logon PS, deployment seems to be restarting the log processor/ waiting for log-processor to become available for an abnormal amount of time.
Screenshots

Thanks for submitting this. We've merged a fix to address this. Please try to run the scenario again.
Thanks,
Dale


Add Blast Procesing [cover piano] by Waterfame

Song Name
Blast Procesing [cover piano]
Artist Name
Waterfame
Source
Youtube
Youtube Link
ZGXcaQQDE01t7ktp
Direct File Link
No response
Song ID
16
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 77239595

reject
invalid yt link


chore: migrate AWS SDK for JavaScript v2 APIs to v3

Fixes: https://github.com/awspilot/cli-lambda-deploy/issues/12
AWS SDK for JavaScript v2 will enter maintenance mode on September 8, 2024 and reach end-of-support on September 8, 2025. For more information, check blog post at https://a.co/cUPnyil
This PR migrates the AWS SDK for JavaScript v2 APIs to v3

I used<EMAIL_ADDRESS>since the support for Node.js 10.x was dropped in v3.46.0 in AWS SDK for JavaScript v3.
I couldn't find Node.js version support matrix by the project, but since it hasn't published a new version for the last 3 years I believe it'll have users on older Node.js versions.

cc @adrianpraja Can this get a review?
The JS SDK v2 is in maintenance mode, and is going to be end-of-life in Sept'25.


Add the Deamon file and other NiftyWeb-related files for future challenges

I think it would make sense to upload all related files and documentation that enabled us to link this challenge with the NiftyWeb platform, so that next we do it, we won't start from zero.
@stephaniealley I suggest you create a specific folder, with a README and the necessary files.

@jcohenadad Yes, this was being done elsewhere (GDrive), but I will upload everything here.

This has been resolved with commit 7f859f3.


Template from URL

Hi and thank you for this great tool! I love it :+1:
I was wondering if this tool supports passing in templates or config files from another application. Say for example I have an application which creates the templates and all I have to do is to make docker-gen listen on the docker.sock and grab the template from let's say <IP_ADDRESS>/templates/container1.tmpl
Is this possible? If not how I should I go about implementing it?

Hey @SergiuCip, I've added the PR #192 that could help with what you need regarding this issue. Could you have a look at it and tell me what you think? Thanks.


Not working on windows 10 redstone 2

I updated windows 10 rs2 (Creator Update) today and TwitchLeecher doesn't work anyway.
Hope fix it soon.
and it would be nice TwitchLeecher support multiple streams downloading like idm so download vods faster

See issues #111 and #119


Feature request: Give the user the possibility to generate a DTO class for the result of a query

It should be possible to turn this on and off per query (per sql file).
Use a custom file property on the sql file with a toggle menu item on the sql file.

Default for all new SqlQueryTools sql files should be defined in the options screen.
But give the posibility to override the default per SqlQueryTools sql file.


Potential bug during training?

Is there a reason you calculate the reward the way you do in line 69?
https://github.com/toshikwa/gail-airl-ppo.pytorch/blob/4e13a23454600a16d5aeeeb4c09338308115455e/gail_airl_ppo/algo/airl.py#L69
My models were able to learn after I changed that line to
        with torch.no_grad():
            rewards = self.disc.g(states)

This gives the unshaped rewards

Did that work out for you? I found my actor loss unable to converge.

Yes it did, although I was running it on discrete state and action environments. Which env are you using?

@liubaoryol it is great to hear that you got it working with discrete action space! could you please share your code? i think it will be valuable, as multiple people here already asked about discrete action support. Thanks in advance

Of course! Let me clean it up and I'll share it next week:)

I'm interested to know about the implementation for discrete action support too. :)

reward = -logsigmoid(-logits) = -log[1 - sigmoid(logits)] = -log(1 - D), which corresponds the objective of G is minimize log(1-D).


Pass -fno-it on GHC 8.6

See https://ghc.haskell.org/trac/ghc/ticket/14336, which should allow for smaller memory leaks.

Not harmful on earlier GHC's to pass and forget.

I undid the fix due to GHC bugs

Still relevant?

Yes. GHC introduced -no-it which reduces memory usage in Ghci, which would be great. It's available from GHC 8.6 and above. Unfortunately, this flag also turns on the behaviour of printing out lots of spurious () values. Ghcid isn't expecting those, and gets upset. Because GHC 8.6 has those spurious () values the options are 1) never use -no-it, 2) cope with (), 3) patch GHC to not print out the spurious () and wait a decade until Ghcid no longer supports the last version which prints them. Of those, only 2 seems feasible, but I haven't done the work for it.


Use a more generic type for detecting the API group

The previous solution worked for unstructured and default types but not for the Policy kind. This new type works for all.

/cc @jlpadilla
Sorry about the noise on this one but it's a small one.

/cc @zlayne


Optimizely Hooks API

Are there plans for a first-party API that utilizes React hooks?

Hi Nicholas, thanks for the question. Currently, there are no plans to provide a first-party API that utilizes React hooks. Can you give an example of what you are thinking of?
One reason we have not provided any React hook APIs is because we don't want to accidentally encourage the practice of calling Optimizely APIs in non-ideal places like render functions which may be called many more times than desired. This is especially important for APIs that send impression events for experiment results. We are thinking about separating the decision logic from the tracking logic, which would make this less of a problem; however, we have not made this change to our SDKs.
Hope this helps.


LOGIST-1002 - Fix closeDays

Implements the _TILL_ placeholder to have the correct amount of days until closing the PR.

+1


MacOS web midi support

While trying to launch signls on MacOS, It closes automatically with the following message "no midi drivers" even though I have Enfer opened with webmidi permissions given.

Weird, it should be handled by rtmidi.
What's your macOS version and browser?

Could you try this build as well?
signls_80cb6947fe2f40a917e64a56cb02c4a35180c505_macOS.zip
It exposes a default virtual midi output (RtMidi Output Client: default) that you should be able to see and connect in your midi setup configuration (https://support.apple.com/guide/audio-midi-setup/set-up-midi-devices-ams875bae1e0/mac).

It turns out that Enfer force the use of a specific midi device that does not exist on macos. Everything should be working fine with other webmidi synths.
Closing this for now, but feel free to reopen if needed.


Implement select component

Closes https://github.com/covid-projections/act-now-packages/issues/219

Feedback
The "Select" label should be paragraphSmall according to the mocks https://www.figma.com/file/YN7x6bJmSfRHaovP5ABKu2/ActNow-Design-System?node-id=41%3A1555


Tracking: support Iceberg sink

RisingWave is designed to be both a streaming database and a streaming engine so it also supports external sinks natively. RisingWave will only support the ingestion part but not the DDL commands (CREATE TABLE) or data querying. In short, it's merely a streaming ingestion engine.
This issue tracks the features required for the Iceberg sink.

[ ] Support using Hive Metastore as the catalog
[x] Support HadoopCatalog
[ ] Support using Glue as the catalog (medium-priority)
[ ] Support using REST as the catalog (low-priority)

We may support timestamp data type after https://github.com/risingwavelabs/risingwave/issues/8299.


Add extra docs for grains, take two

In a fit of pique, I deleted my euphoria fork.
That wiped out #63.
This recreates it from the discussion: taking my last suggestion plus Gregory's comments, and removing the 32-bit specific stuff.
Attn: @axtens @petelomax @ghaberek

Ah yes, pique. Unrelated to fique however
Merging. It reads okay and I'm tired of seeing it cluttering up the pull requests


Client: another client TCP socket in the same libuv loop

Hi,
This is more of a question than an issue with the library (which is superb, kudos for that) - would appreciate any hints; apologies in advance if this is a duplicate or is irrelevant (I've found https://github.com/uNetworking/uWebSockets/issues/265 but couldn't figure how to apply it to my case).
If I have a uWS client that is normally spawned via the blocking Hub::run() and runs the libuv event loop, is it possible to integrate another (non-websockets) tcp client socket into the same event loop? I would hate to have to synchronize things manually. Ideally, I'd like to have something like on_message_uws() and on_read_other() being called from one event loop (and also be able to send websockets messages through uws and raw tcp through the other one).

Hub::getLoop gives you the (shared) uv_loop_t pointer. You can use this in any other libuv project on the same thread.

I see, thanks! However (IIUC) a pointer to WebSocket is only available in hub's callbacks, in this case would it be possible to call WebSocket::send from a callback triggered by a different handle?

Yes you can use websockets any time between onConnection and onDisconnection

@alexhultman Thanks a million for your help!
(Closing the issue now -- hopefully it will be helpful to others too might they come across the same question.)


Add mappings for DataStage assets (for lineage)

At least the following:

[ ] dsjob
[ ] stage
[ ] ds_stage_column
[ ] link
[ ] referenced_container (shared_container, local_container)

Note that creating or saving jobs in the DataStage Designer client does not trigger any events into the InfosphereEvents topic... May need to consider an alternative mechanism for polling for changes to jobs (modified_on timestamp?) to trigger events in the EventMapper.

After discussion, now considering writing this instead as a separate proxy that integrates directly with the Data Engine OMAS -- covering both design-time lineage (potentially polled via IGC + modified_on) as well as runtime lineage (via the XML file logs of DataStage itself).
This would more closely fit the paradigm of integrating a Data Engine, and provide an additional example of a pattern (integrating via OMAS rather than via OMRS).

Closing, to be addressed through new DataStage-specific adapter that will call into Data Engine OMAS via a new Data Engine Proxy governance server capability.


_ide_helper.php overrides vendor files when using Go To Declaration action in PHPStorm

Versions:

ide-helper Version: 2.9.3
Laravel Version: 8.36.2
PHP Version: 7.4.16
PHPStorm Version: PhpStorm 2020.3.3 (Build #PS-203.7717.64, built on March 16, 2021)

Description:
For some reason class definitions in _ide_helper.php file override actual class definitions when using the "Go To -> Declaration or Usages" action (CTRL+B hotkey for me). The "Go To -> Type Declaration" action (CTRL+SHIFT+B hotkey for me) shows both class declarations from _ide_helper.php and original vendor file (see screenshot).
This also breaks constructor autocompletion, since classes from _ide_helper.php doesn't have constructor arguments.

Steps To Reproduce:

Create fresh Laravel app;
Install laravel-ide-helper package;
Import some Laravel class (for example, Illuminate\Http\Request) and try to open original file by navigating to declaration;
Declaration in _ide_helper.php opens instead of vendor/laravel/framework/src/Illuminate/Http/Request.php file.

You can check my reproduction repository: https://github.com/tzurbaev/ide-helper-navigation-bug
Clone it, run composer install, open tests/Unit/ExampleTest.php and try to navigate to the Request class.

That's how it works 🤷♀

See also this recent comment https://github.com/barryvdh/laravel-ide-helper/issues/592#issuecomment-816388888

What confused me is a Illuminate\Http\Request class. I believe this class was not in _ide_helper.php in older versions of the package and now it's in. Maybe that's why I thought it's a bug.
I'll close this for now. Thanks for a heads up, @mfn!


[Feature Request] VSC explorer isolate folder

When we have a lot of folders, we need to simplify view for better concentrate and orientation

Did you try https://code.visualstudio.com/docs/getstarted/userinterface#_filtering-the-document-tree?

Did you try https://code.visualstudio.com/docs/getstarted/userinterface#_filtering-the-document-tree?

Can I filter only folders?
When I'm trying to filter folder, files inside needed folder also hides
https://streamable.com/km3h91

You can configure files.exclude to hide the other folders, otherwise duplicate of #41860


add roc and gain for evaluation

add roc and gain for binary classification
update evaluation test case

closed by #317


Unable to find some services.

Hi all :
I ' m a new to istio and jaeger.  I have installed the istio(1.0) on minikube and deployed both bookinfo and helloworld sample. However, I can only find the helloworld service on jaeger. Can someone tell me the reason or where should I check.

If you guys need more information please let me know.
Cheers
RWL

cc @objectiser

@readwritelearn Could you list the pods? Did you use auto or manual injections for the bookinfo example?

I had the same problem， and i used auto injections for the bookinfo example.
I deployed jaeger in the following steps：
kubectl create -f https://raw.githubusercontent.com/jaegertracing/jaeger-kubernetes/master/production-elasticsearch/configmap.yml
kubectl create -f https://raw.githubusercontent.com/jaegertracing/jaeger-kubernetes/master/production-elasticsearch/elasticsearch.yml
kubectl create -f https://raw.githubusercontent.com/jaegertracing/jaeger-kubernetes/master/jaeger-production-template.yml

Closing pending more information.


Group 4 - Solution for Language Detector

Sean, Anubhav, Ben

Hey guys, I've started reviewing your code. It looks pretty good so far. Will post questions as I find them:
create_set_from_text(text)

Not saying what you did is wrong, but why did you decide not to use try and excepts? Is it because you cannot differentiate between different value errors?

Your code is very functional ;) very well done in that regard. Do you mind explaining why you decided to compartmentalize both creating a set from common words and creating a set from the text? Since these are both inputs required for the program to run, I think that it would make sense to validate them in one function. But again, just a suggestion / question.
Done Reviewing.

@gkarp The string inside ValueError will be able to differentiate on what is causing the ValueError. Regarding try/catch vs raise, create_set_from_text(text) is kind of the API we have wrote and detect_language is the code that uses our API. Now my API will raise error for the people using the API, and they will decide how to handle the exceptions. So technically detect_language should be using try/except block to catch error raised by my API. I hope I am clear in my explanation.
And yes you are write,we could have use one function to validate the input, and I wanted to implement something like is_input_valid and is_languages_valid, but just couldn't get the time. I have written test_cases for all my API, so had to write them for these two functions too.
Thanks for the review.

Good stuff guys, closing it now.


SWU 16Aug2018 1320

This issue has been auto-generated to facilitate questions about a paid opportunity that has just been posted on the BCDevExchange.To learn more or apply, visit the opportunity page. The opportunity closes on Thursday, August 16, 2018.

This opportunity has been assigned


Ability to connect to GitHub repos with a token rather than username & password

This would be handy. I've written the necessary amendment in a local version of GitHubManager and is very simple. Be great if you would consider including at some point:
namespace GitReleaseManager.Cli.Options
{
using CommandLine;
using Octokit;
public abstract class BaseGitHubSubOptions : BaseSubOptions
{
    [Option('u', "username", HelpText = "The username to access GitHub with.", Required = false)]
    public string UserName { get; set; }

    [Option('p', "password", HelpText = "The password to access GitHub with.", Required = false)]
    public string Password { get; set; }

    [Option('o', "owner", HelpText = "The owner of the repository.", Required = true)]
    public string RepositoryOwner { get; set; }

    [Option('r', "repository", HelpText = "The name of the repository.", Required = true)]
    public string RepositoryName { get; set; }

    [Option('t', "token", HelpText = "The token instead of username and password", Required = false)]
    public string Token { get; set; }

    public GitHubClient CreateGitHubClient()
    {
        Credentials credentials = null;
        if (this.UserName != null)
        {
            credentials = new Credentials(this.UserName, this.Password);
        }
        else
        {
            credentials = new Credentials(this.Token);
        }

        var github = new GitHubClient(new ProductHeaderValue("GitReleaseManager")) { Credentials = credentials };
        return github;
    }
}

}

This is currently already possible. Where you would put your password, simply put the token and it will work as you expect. This is how I use GitReleaseManager across a number of projects.
Perhaps it could be made a little more clear that it it possible, but it certainly can be done.

Hi,
Many thanks - yes, I'd since discovered the same that password / token were
broadly the same thing.
Cheers
Stewart
On 23 August 2016 at 18:14, Gary Ewan Park<EMAIL_ADDRESS>wrote:

This is currently already possible. Where you would put your password,
simply put the token and it will work as you expect. This is how I use
GitReleaseManager across a number of projects.
Perhaps it could be made a little more clear that it it possible, but it
certainly can be done.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GitTools/GitReleaseManager/issues/83#issuecomment-241805790,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADyOfysdkU_VyySFvhRT9A0gD6KjO8y2ks5qiyqMgaJpZM4JrI-8
.

--
Stewart Snow
(m): + 44 (0)7785 922643
(e<EMAIL_ADDRESS><EMAIL_ADDRESS>

@StewartSnow nice one!
Happy to hear that you got it sorted.  Hope you are enjoying using GitReleaseManager!

It's an excellent tool!!
Using it alongside GitReleaseNotes, as we're not using milestones at the
moment, so using GRM to upload the GRN generated release notes file to the
repo.
It makes me very happy !! ;)
On 24 August 2016 at 13:11, Gary Ewan Park<EMAIL_ADDRESS>wrote:

@StewartSnow https://github.com/StewartSnow nice one!
Happy to hear that you got it sorted. Hope you are enjoying using
GitReleaseManager!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GitTools/GitReleaseManager/issues/83#issuecomment-242040813,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADyOf4JR-LwaZXiVmDRdMpX9HmoPfuVBks5qjDTrgaJpZM4JrI-8
.

--
Stewart Snow
(m): + 44 (0)7785 922643
(e<EMAIL_ADDRESS><EMAIL_ADDRESS>

Just as a reminder: the documentation is lacking the information about tokens: https://gitreleasemanager.readthedocs.io/en/develop/commands/create/

@steven-r actually, the docs themselves have been updated:
https://github.com/GitTools/GitReleaseManager/blob/develop/docs/commands/create.md
However, they haven't as yet pushed to readthedocs.  There is "something" that has to be done to turn this back on, after a change on their end, but I haven't yet dug into this to figure out what needs to be done.

@steven-r actually, the docs themselves have been updated:
https://github.com/GitTools/GitReleaseManager/blob/develop/docs/commands/create.md
However, they haven't as yet pushed to readthedocs. There is "something" that has to be done to turn this back on, after a change on their end, but I haven't yet dug into this to figure out what needs to be done.

I see, actually I was following the link at the link at the project's overview. I'll have a look there.
Thnks for the update


Replace TODOs with GitHub issue

See https://github.com/itchysats/itchysats/issues/1094. This is one of the issues that were mentioned before, @rishflab.

hm, without these TODOs it will be harder to find all places by the person trying to fix this issue - not sure whether this is worth it, especially that there's virtually no risk of this lingering?
Totally agree that there should have been an gh issue raised at the time this got merged to master though.

I understand your point, but the TODO noise is annoying me in the codebase atm. What if I permalink to the 3 places where the fix needs to occur in the GH issue?

hm, without these TODOs it will be harder to find all places by the person trying to fix this issue - not sure whether this is worth it, especially that there's virtually no risk of this lingering?
Totally agree that there should have been an gh issue raised at the time this got merged to master though.

I understand your point, but the TODO noise is annoying me in the codebase atm. What if I permalink to the 3 places where the fix needs to occur in the GH issue?

I think that there problem with the TODOs is happening because a lot of them are "someday/maybe" and these ones should be pruned without any hesitation. However, in this particular case, the task in one of our next tasks, potentially being actioned today (and definitely in the near future) - I don't get why make it harder for the assignee.
If I was assigned, I would be grateful for these TODOs being left, and permalinks would be an annoying overhead as I would need to jump out of my IDE.
There is also the point that as per our latest agreement TODOs should not be in the code - I'm happy with you push this as it's valid from this point of view, but this PR made me think that perhaps we should relax this requirement (I'll raise a discussion at the next opportunity :) )
PS in this case it should be actually a FIXME 😅  (we know we're doing something wrong and it should have been fixed).

There is also the point that as per our latest agreement TODOs should not be in the code - I'm happy with you push this as it's valid from this point of view, but this PR made me think that perhaps we should relax this requirement (I'll raise a discussion at the next opportunity :) )

I just assumed that we had already chosen to relax or ignore that agreement, since the number of TODOs doesn't seem to be going down.
I'll close this PR because it doesn't seem to be adding any value at the moment, but I will keep trying to remove TODOs if I get the chance.

There is also the point that as per our latest agreement TODOs should not be in the code - I'm happy with you push this as it's valid from this point of view, but this PR made me think that perhaps we should relax this requirement (I'll raise a discussion at the next opportunity :) )

I just assumed that we had already chosen to relax or ignore that agreement, since the number of TODOs doesn't seem to be going down.
I'll close this PR because it doesn't seem to be adding any value at the moment, but I will keep trying to remove TODOs if I get the chance.

OK, I'll try to keep vigilant of meaningless TODOs too, so that the signal to noise ratio improves. Found a few now #1098 :)


Change branch settings for main to require a pull request

This must be done after the repo is made public

main now requires pull request


tools/bazel reports error on Mac

tools/bazel script reports the following error when running tools/bazel build @com_google_protobuf//:protoc:
/Volumes/BuildData/tmpfs/src/github/grpc/workspace_objc_macos_opt_native/tools/bazel-1.0.0-darwin-x86_64: line 1: syntax error near unexpected token `newline'
/Volumes/BuildData/tmpfs/src/github/grpc/workspace_objc_macos_opt_native/tools/bazel-1.0.0-darwin-x86_64: line 1: `<html>'

Sample build log: https://source.cloud.google.com/results/invocations/2854f73d-d668-4eea-a44f-d48e1e5b8336/log

To explain the weird error message: I think what happends is that the download of bazel binary fails (and curl stores a HTML document with an error instead) and after that we try to execute that file (which contains garbage).
https://github.com/grpc/grpc/blob/ce93ed85bfb70f3c3357402929dd318427bad628/tools/bazel#L66


[$500] App crashes when opening the second distance request as a reviewer

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Log in to User A's account.
Create a workspace and invite User B.
Open the workspace room from User B.
As User B, create a distance request (workspace chat -> + -> Request money -> Distance -> add waypoints -> Next -> Request
Return to User A's account.
Open the Requested Distance sent from User B. Confirm that the app doesn't crash
Switch back to User B's account and send a distance request again.
As User A, try to open the new distance request sent by User B.

Expected Result:
User A should be able to view the new distance request
Actual Result:
App crashes
Workaround:
N/A
Platforms:

Only tested on web. Not sure if the issue is present on other platforms, too

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: v1.3.64-2
Reproducible in staging?: Yes
Reproducible in production?: Yes
**If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
**Email or phone of affected tester (no customers): N/A
Logs: N/A
Notes/Photos/Videos:
Please refer to this video (I couldn't upload the video)
**Expensify/Expensify Issue URL: N/A
Issue reported by: @ayazhussain79
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1694035685129389
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014b20f43c302788fe
Upwork Job ID:<PHONE_NUMBER>794378240
Last Price Increase: 2023-09-07

Proposal
Please re-state the problem that we are trying to solve in this issue.
App crashes when opening the second distance request as a reviewer.
What is the root cause of that problem?
Attempting to call a method on the suggestionsRef in the onBlur method results in the error when the ref is null.
What changes do you think we should make in order to solve the problem?
Add a null check on suggestionsRef.current to prevent access to the null reference.
What alternative solutions did you explore? (Optional)
None.

Problem Statement:
The application crashes when trying to open the second distance request as a reviewer. The crash can also occur when opening the first distance request on a newly created workspace.
Root Cause:
The crash is triggered within the ReceiptUtils#getThumbnailAndImageURIs method while trying to determine if the file is an image. In certain scenarios, the filename passed as a parameter to the method can be null or undefined. When operations (like split, etc.) are attempted on this null or undefined filename, it leads to a crash.
Proposed Solution:
To prevent the crash, an early return should be added in the getThumbnailAndImageURIs function to check if the filename is null or undefined. If it is, the function should return default values for the thumbnail and image. Here's the modification:
if not filename:
    return {'thumbnail': None, 'image': 'ReceiptGeneric'}

This ensures that when the filename is eventually retrieved, the preview will re-render, and the image (if any) will be correctly displayed.
Test:

For a valid filename:

test_filename = "test.jpg"
test_path = "https://example.com/test"
assert getThumbnailAndImageURIs(test_path, test_filename) == {'thumbnail': 'https://example.com/test.1024.jpg', 'image': 'https://example.com/test'}

For a null or undefined filename:

test_path = "https://example.com/test"
assert getThumbnailAndImageURIs(test_path, None) == {'thumbnail': None, 'image': 'ReceiptGeneric'}

Alternative Solution:
While the current proposed solution focuses on handling null or undefined filenames, an alternative approach could involve a more comprehensive validation mechanism. This mechanism would not only check for null or undefined values but also for empty strings, filenames with invalid characters, and other potential anomalies. However, this might introduce additional complexity and may be unnecessary unless there's a broader issue with how filenames are being managed in the application.

@akinwale How will the money request action component look like until the receipt is loaded?

@akinwale How will the money request action component look like until the receipt is loaded?

It displays the default generic receipt image (the green placeholder) until the image loads. I can make a video to demo this.

@mountiny Demo video. My connection is a bit slow so it takes a while to load.
https://github.com/Expensify/App/assets/4160319/84bc7eae-a29c-4171-bd4a-ea353f7f4f30

@mountiny we already discussed this issue backend should be sent as a filename property in  pusher and API
https://github.com/Expensify/App/issues/26585#issuecomment-1704803718
https://github.com/Expensify/App/issues/26585#issuecomment-1704381989
https://github.com/Expensify/App/issues/26585#issuecomment-1703976018
https://github.com/Expensify/App/issues/26585#issuecomment-1704399604

@ArekChr could you review the proposals?

I'm taking over as requested internally in Slack here

@ArekChr this will help you reproduce this video but now the issue was changed pusher not sending the transcation_<PHONE_NUMBER>105531794   data so its keeps loading
https://github.com/Expensify/App/assets/16595846/36c226ae-8710-4f3f-8d00-ca164016d509

@neil-marcellini  @ArekChr  could you check this on the Backend side.

@pradeepmdk I couldn't load the video. Could you optimize it and send it in a smaller size?

@ArekChr which one

@pradeepmdk I couldn't load the video. Could you optimize it and send it in a smaller size?

@ArekChr You can also check out the demo video I posted here: https://github.com/Expensify/App/issues/26925#issuecomment-1709698115
At the time, I was able to reliably reproduce if I tried to open the request before the image loaded.

@akinwale, @pradeepmdk Thanks for videos! I tested that scenario again and It seems the app is not crashing anymore on the main branch and staging, there is a different issue with no loaded distance request which is not related to this issue.

@ArekChr  @akinwale Do you know which PR has fixed this bug?

@ayazhussain79 this is not yet fixed. previously pusher sent the wrong data so it crashed. now Pusher not sending that whole object so the issue has changed now.

@pradeepmdk, The bug with the crash issue, is not reproducible anymore

@ArekChr Yes I agree, I am just explaining @ayazhussain79 to the issue that has changed now. The crash issue is no more here. Maybe one pusher is fixed and again filename  property is missed means this issue will reproduce. now this issue is closed or hold that is your choice @ArekChr.

I think we can close this issue, for the infinite skeleton loading bug, there is a separate issue created here


[python-bindings] Fix Readme: pytest needs to be run from base dir

Thanks @aniemetz !
I plan to resolve this by reorganizing the Python bindings to live in a separate repository. That will clear up this issue as all the Python related code (source and tests) will be together (rather than having the Python tests with the c/c++ tests as it is now)

#202 and https://github.com/leonardt/pycoreir


Develop

@EricR401S @Emmawang00 @pr-124
Graphs for question two.
I am now going to work on the general document structure and the words.
Fixed a runtime issue

@lornamariak I looked at everything, it looks great!!!!


[NIFI-1904] If open, close hamburger menu on window resize

Reviewing...

Looks good @scottyaslan! This has been merged to master. Thanks!


Fix workspace regression from pull/6538

I broke the workspace page in https://github.com/Expensify/App/pull/6538. @thesahindia reported the issue and @mananjadhav has already spun up a fix.
I'll create an Upwork job to compensate both of them shortly.

Ok offer sent in Upwork to both @thesahindia and @mananjadhav.

Paid!


Opening/closing through loqed app

When opening or closing through loqed app, the statis of the lock is not updated within homey.
Within google home it is updated, so the issue is within the homey app.

This should be fixed with the latest version. https://homey.app/a/com.loqed.touch-smart-lock/test/


particle_in_a_box_simulation

add a python project on particle in a box simulation (physics)

Merging, Star the repo if you like it.