Issue deployment of Azure resources

Hello,
I am trying to follow the procedure described in scripts/README.md in order to provision the Azure resources.
I logged in to my Azure account using CLI and tryied terraform init, which failed because resource group dagx-infrastructure was missing.
I created it manually and I got another error saying that storage account dagxtstate was missing.
Again I created it manually and now I am getting this error:
Error: Failed to get existing workspaces: containers.Client#ListBlobs: Failure responding to request: StatusCode=404 -- Original Error: autorest/azure: Service returned an error. Status=404 Code="ContainerNotFound" Message="The specified container does not exist.\nRequestId:7a8a8c1f-b01e-0025-3341-89fc75000000\nTime:2021-08-04T15:03:40.1040244Z"
Am I missing something in the procedure? Are we suppose to build these resources manually? If yes could you please advise on how to do it?
Thanks in advance for your help.
Benjamin.

Hello @bscholtes1A ,
can you please give me a bit more context? Why and how are you using the IDS Metadata Broker with Azure?
Best,
Sebastian

@bscholtes1A  Could you please provide us with some information on this otherwise, probably we will have to close this ticket.

@bscholtes1A  For now I am closing this issue, feel free to reopen it in the future. Also please provide us with more information on this issue.


[Help wanted] I want options to place comments like XCTAssertEqual(A, B, "Comment") on a separate line

Hi giginet!
I have a ideas to improve this great tools.
Details
In XCTAssertEqual, it is possible to pass a comment as the third argument, but after converting to SwiftTesting, you can achieve the same display on error by placing the comment on the line above #expect.
This method is more readable and reduces the amount of code on a single line, which is why it is highly valued in our project.
Would it be possible to provide an option to convert in this way?
Sample code
original code
func test_is_success() {
    vm.successs()
    XCTAssertEqual(vm.state.state, "success", "成功していること")
}

after run tool
@Test
func is_success() {
    vm.successs()
    // 成功していること
    #expect(vm.state.state == "success")
}

This conversion is not common so it should not be implemented.
However, we can take the second argument of #expect and I thought I implemented this conversion 🤔
https://developer.apple.com/documentation/testing/expect(_:_:sourcelocation:)
I'll check the situation!

I confirmed this conversion works well. We already have a test case, but I added comments to the fixtures.

I'll close this issue, but if you find a situation in which this conversion doesn't work well, feel free to open a new issue.


Added MSR support and updated .gitignore

Hello @KevinAlavik ,

At this point you should give me commit rights

So I added the support to read and write MSRs (of course with checking if they are available) and Updated you .gitignore to include configs for my Neovim setup

Not interested atm


Android: maps blank in release apk

Hey, I just packaged my app up for release, and for some reason maps are not loading. They work fine in debug mode, but are just grey in the release version, with the google logo in the bottom left corner.
Anyone know what's going on?

your google map api key?

ya definitely. the key I was using wasn't signed with my release key.

@allthetime Can you elaborate? I'm having same problem. Fine in debug, but in release api, I see all white screen, and a little Google logo at bottom left. Not clear what steps I need to take. thanks!

I'm facing same problem,
map work on debug but it's not working on Google Play Store
please help
thanks

Hi
solved by replacing SHA1 key from Google play store

Debug apk  install and  app work properly and google map show, too
but  release generated apk  install when map not showing in please solution
All process are done, but i'm facing same problem
Could you please guide me for the same on how to resolve it asap? I really need to get this done. Help me.

same problem here, done adding SHA 1 key using keytool, but not working. app works in debug build but not in release.. please help

My simple insight on those still struggling with this.  use the solution given by @wkw  it will work ,
the secret is here https://cloud.githubusercontent.com/assets/1691220/23273776/5418ae8e-f9ce-11e6-8bb2-d0ef5b13dc5b.png

is this the keystore that is in the /android/app folder?


Release 0.6.0

:tada:

r? @nastevens
(rust-highfive has picked a reviewer for you, use r? to override)

bors try

bors cancel

bors try-

bors try

bors r+

bors r+


Add license for Scala

Adding the Scala license

Where is Scala used?

On Wed, Mar 30, 2016 at 5:00 PM, Dan Debrunner<EMAIL_ADDRESS>wrote:

Where is Scala used?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/apache/incubator-quarks/pull/57#issuecomment-203690269

It is in the Kafka distribution.  See
https://git-wip-us.apache.org/repos/asf?p=incubator-quarks.git;a=tree;f=connectors/kafka/ext;h=d8379cff925e830ddc6878caf7e136b9352d4011;hb=HEAD

Closing this as I think we should revisit when we have a new build system and working on a release.


Infinite retries

Expected Behaviour
The client should only retry up to the specified retry number.
Actual Behaviour
The client retries to infinite and beyond with the error Connection closed while having active streams.
Debug Information
I´m only able to reproduce this problem on my application that I can´t share but the issue lays here https://github.com/enisdenjo/graphql-sse/blob/master/src/client.ts#L476
Before getting the results we clear the number of retries but if the client fails to get the results back (which is the problem in my case) it will keep retrying over and over.
I think the number of the retries should only be cleared after successfully getting the results back.

Hey there, thanks for reporting! Your bug feels possible, I am just thinking of how to create a failing test so that a proper fix can be placed. Any suggestions?

I was able to pinpoint the problem in my server.
I will try and create a small poc

@esfomeado any luck with the repro?

I was not able to create a perfect replication of my original problem but I created something close.
Bug.zip
Run the server and the client.
Navigate to http://localhost:4200/sse if you check the console you will see the data being logged.
Refresh the page (try a few times if doesn't work at first)
You will see that no data is longer logged and you will see the client reconnecting over and over.

Hi! this is also happening to me, is there a workaround for this?

Just wanted to ping you guys that I've managed to assemble a failing test in #57 and am working on a fix. Thank you guys for reporting and for your patience!

Thanks @enisdenjo , timing couldn't be better. If you need any help testing this, I am happy to help. Thank you!

Thanks for the fix @enisdenjo

This is fixed and merged. However, GitHub is having issues currently - so I'll delay releasing a new version until issues are resolved. Thanks for understanding!

Thank you very much!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

GitHub's still shaky, but I've pushed a new version nevertheless to unblock you guys. Try it out!

GitHub's still shaky, but I've pushed a new version nevertheless to unblock you guys. Try it out!

It works great! Thanks!!

Oh, sad to hear that @2snEM6. Are you sure you updated the lib to 2.1.2? Is it the same problem? I would need your support to get it fixed, a failing test or a concrete repro would be of great help - otherwise I am in the dark.

Yes, on 2.1.2. I will try to create a small server to reproduce it, but it's hard. I saw that if the NodeJS subscriptions server crashes abruptly, I get a 404 error on the frontend, meaning the stream cannot be found. I will share more information when I have it available

@enisdenjo just happened to me again and I was able to see the error thrown on the server:
Error: Command timed out
2023-05-11 14:39:37     at Timeout._onTimeout (/code/node_modules/ioredis/built/Command.js:192:33)
2023-05-11 14:39:37     at listOnTimeout (node:internal/timers:557:17)
2023-05-11 14:39:37     at processTimers (node:internal/timers:500:7)

I am running a NodeJS GraphQL Server (GraphQL Yoga) with subscriptions enabled with https://www.npmjs.com/package/graphql-redis-subscriptions pub sub package and what is timing out is the subscription client to the pub sub right in the subscription definition.
For instance:
onPositionInterviewStatsUpdated: {
      type: Models.Position,
      args: {
        positionId: {
          type: GraphQLID,
        },
      },
      pubsub.asyncIterator([
        'an-event-name'
      ]),
      resolve: (eventData, _, context) => {
        // resolver
      },
},

A command timeout to the Redis pub sub server is making my server crash abruptly.
Of course the server should not crash this way but the frontend shouldn't be hitting the server with very high frequency calls. It should stop retrying at some point.
I guess the way to reproduce this is creating a simple GraphQL Yoga server running on NodeJS + redis pubsub. Subscribing and then turning redis off

@enisdenjo I forgot to say Im using single connection mode, it's crashing there

Thanks for sharing, this is honestly too much of a setup to debug only graphql-sse. I'd really appreciate a repro.
Furthermore, can you share your Yoga setup? Are you using the @graphql-yoga/plugin-graphql-sse plugin with Yoga?
In the meantime, I'll see whether I can get a similar failing test to #57 for single connection mode.

Hey @2snEM6, just wanted to ping you that I've managed a failing test in #59. Will be working on a fix.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Thanks for sharing, this is honestly too much of a setup to debug only graphql-sse. I'd really appreciate a repro.
Furthermore, can you share your Yoga setup? Are you using the @graphql-yoga/plugin-graphql-sse plugin with Yoga?
In the meantime, I'll see whether I can get a similar failing test to #57 for single connection mode.

That is correct! I've tried to set up a simple repro repo but I fail to reproduce the issue.

Hey @2snEM6, I've just pushed v2.1.3 containing a fix for single connection mode - please give it a try!

Thank you very much! I will give it a try and let you know 🙏🙏


What's the Norm function?

What's the Norm function in Eq.(3)? LayerNorm?

What's the Norm function in Eq.(3)? LayerNorm?

We use batch norm for the experiments in the paper.

Thanks a lot.

Another question: for the input BxHxNxN, BN is performed along which dim?

nn.BatchNorm2d(num_features=H) or nn.BatchNorm2d(num_features=N)?

Good work！What about the initialization of the HxH matrix？Is torch.eye or torch.randn?

Good work！What about the initialization of the HxH matrix？Is torch.eye or torch.randn?

Hi,
Thanks for your interest. This is a good question, I try both the eye init and random init and the results are similar. For the experiments in the paper, the results are based on random init. I am considering add in a set of experiments regarding the initialization into the paper also.
Best regards,
Zhou Daquan

Thank you very much.


urgent: ML performance regression

@acoh64 take a look at the following
https://www.deeplearningbook.org/
Can we implement some of the ideas in the above paper?

yes

@acoh64 as an PEE1 dev, you should NOT be closing issues like this. reopening

please call me master pisser


Problem with Example Code

Configuration:

Redis version: 6.0.1
RedisJSON version 10007
ioredis: 4.27.3
iorejson: 0.1.1

Redis and RedisJSON are running on standard ports from within a Docker container.

I am running Redis (with RedisJSON) from a Docker container.  I am able to access both standard Redis and RedisJSON from other applications.
Here is my simple package.json
{
"name": "iodredis-test",
"version": "1.0.0",
"description": "",
"main": "app.js",
"scripts": {
"test": "echo "Error: no test specified" && exit 1",
"start": "node app.js",
"redis": "node redis.js",
"redisjson": "node redisjson.js"
},
"author": "",
"license": "ISC",
"dependencies": {
}
}
However, when I attempt to run the sample iorejson code, I get an error.
var Rejson = require('iorejson');

var instance = new Rejson();
yield instance.connect()

yield instance.set('foo', '.', {
  bar: {
    hello: 'world'
  }
});

value = yield instance.get('foo', '.');
console.log(value);

Here is the error.
anthony@anthony-desktop:/e-drive/projects/iodredis-test$ node redisjson
/e-drive/projects/iodredis-test/redisjson.js:4
yield instance.connect()
      ^^^^^^^^

SyntaxError: Unexpected identifier
    at wrapSafe (node:internal/modules/cjs/loader:1018:16)
    at Module._compile (node:internal/modules/cjs/loader:1066:27)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1131:10)
    at Module.load (node:internal/modules/cjs/loader:967:32)
    at Function.Module._load (node:internal/modules/cjs/loader:807:14)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:76:12)
    at node:internal/main/run_main_module:17:47

What do I need to be able to test out and use RedisJSON?
Thank you for your help!

I don't know why yield is used in this context. It should be await as those methods return Promise objects.


feat: Added all files for Fashion Trends Prediction app

Description
Thank you for opening a Pull Request!
Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Follow the CONTRIBUTING Guide.
[ ] You are listed as the author in your notebook or README file.

[ ] Your account is listed in CODEOWNERS for the file(s).

[ ] Make your Pull Request title in the https://www.conventionalcommits.org/ specification.
[ ] Ensure the tests and linter pass (Run nox -s format from the repository root to format).
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

Note: I updated links in config and README.md, but other code that uses the removed files will need to be refactored slightly to download/upload the files from GCS where needed.

Hi, this PR has been inactive for 2 weeks. I'm going to close it out for now. Please re-open if you would like to continue.


GEODE-1793 LocatorDUnitTest.testStartTwoLocatorsOneWithSSLAndTheOther…

This was a product issue.  When the locator using plain-text sockets is
contacted by a TcpClient using SSL the locator often just closes the socket.
On some platforms this causes a SSLHandshakeException but on others it
just causes a "SocketException: connection reset".  Writing some text to
the socket forces the TcpClient to get a SSLException (which is the superclass
of SSLHandshakeException).
The test class is still marked as Flaky due to GEODE-2542.
I deleted one of the tests in LocatorDUnitTest as it wasn't doing any useful validation and really served no purpose.
I also increased the joinTimeout in this test.  The original 1-second timeout was intended to make the tests run faster but I think it's probably the source of some of the flaky-ness in this set of tests.  Some of them were also overriding the joinTimeout established by the DUnit framework, so that was actually a bad thing to be doing.  The tests all run in a few seconds with the default joinTimeout setting anyway.

There are "spotless" problems I'm cleaning up, and I'm removing the commented out code from the test.

remote: geode git commit: GEODE_1793 spotless fixes and removal of dead code
remote: geode git commit: GEODE-1793 LocatorDUnitTest.testStartTwoLocatorsOneWithSSLAndTheOtherNonSSL
To https://git-wip-us.apache.org/repos/asf/geode.git
c09a856..4112204  develop -> develop


Fix orphaned browser processes due to uncaught exceptions in the tests

If uncaught exceptions occur in the tests (which happened in #17962 and can be triggered manually by throwing an error in integration-boot.js) the teardown logic of the tests doesn't get to run and thus spawned browser processes are not closed properly. Given that test.mjs is the only process that has a reference to them they will become orphaned and keep running if test.mjs exits without explicitly closing them.
This commit fixes the issue by always closing the browsers if uncaught exceptions occur, and we make sure to log them for debugging purposes.
Unblocks #17962.
This patch is ever so slightly easier to review with the ?w=1 flag: https://github.com/mozilla/pdf.js/pull/18401/files?w=1.

I have applied the following diff:
diff --git a/test/integration-boot.mjs b/test/integration-boot.mjs
index 4e04bb6f1..332c97a50 100644
--- a/test/integration-boot.mjs
+++ b/test/integration-boot.mjs
@@ -20,6 +20,8 @@ async function runTests(results) {
   jasmine.exitOnCompletion = false;
   jasmine.jasmine.DEFAULT_TIMEOUT_INTERVAL = 30000;

+  throw new Error("Yikes, something unexpected happened");
+
   jasmine.loadConfig({
     random: false,
     spec_dir: "integration"

Before this patch npx gulp integrationtest would crash, keep the Chrome process open and log this:
file:///home/timvandermeij/Documenten/Ontwikkeling/pdf.js/Code/test/integration-boot.mjs:23
  throw new Error("Yikes, something unexpected happened");
        ^

Error: Yikes, something unexpected happened
    at runTests (file:///home/timvandermeij/Documenten/Ontwikkeling/pdf.js/Code/test/integration-boot.mjs:23:9)
    at startIntegrationTest (file:///home/timvandermeij/Documenten/Ontwikkeling/pdf.js/Code/test/test.mjs:818:9)

Node.js v20.15.0
[12:46:24] The following tasks did not complete: integrationtest, runIntegrationTest
Did you forget to signal async completion?
file:///home/timvandermeij/Documenten/Ontwikkeling/pdf.js/Code/gulpfile.mjs:723
        throw new Error(`Running ${testsName} tests failed.`);
        ^

Error: Running integration tests failed.
    at ChildProcess.<anonymous> (file:///home/timvandermeij/Documenten/Ontwikkeling/pdf.js/Code/gulpfile.mjs:723:15)
    at ChildProcess.emit (node:events:519:28)
    at ChildProcess.emit (node:domain:551:15)
    at maybeClose (node:internal/child_process:1105:16)
    at ChildProcess._handle.onexit (node:internal/child_process:305:5)
    at Process.callbackTrampoline (node:internal/async_hooks:130:17)

Node.js v20.15.0

After this patch npx gulp integrationtest exits normally, closes all browser processes and logs this:
Error: Yikes, something unexpected happened
    at runTests (file:///home/timvandermeij/Documenten/Ontwikkeling/pdf.js/Code/test/integration-boot.mjs:23:9)
    at startIntegrationTest (file:///home/timvandermeij/Documenten/Ontwikkeling/pdf.js/Code/test/test.mjs:818:9)
    at async main (file:///home/timvandermeij/Documenten/Ontwikkeling/pdf.js/Code/test/test.mjs:1093:7)

Run 0 tests
OHNOES!  No integration tests ran!
integration tests runtime was 3.3 seconds

Not only does this therefore fix the problem of hanging browser windows, but it also makes the logs more concise/actionable by removing noise from the uncaught exception propagating.

/botio unittest

/botio integrationtest


Replicate useful StatBot functionality

[ ] Number of messages per channel
[ ] Roles that are most active (esp. year roles)
[ ] Recording numbers of people who leave and join and their roles at the time
[ ] Other statistical data collecting

Should these stats commands be available to all users, or just committee members?

I don't see why it shouldn't be for everyone tbh


"The field 'images' does not allow null values."

I am attempting to create new products.
When calling
var x = client.Products.Create(p);
I get an error response of "The field 'images' does not allow null values."
I have create a new ProductImage and attached it, that doesn't work.
I have queried the ProductImages and attached one of them, that doesn't work.
I have created an empty collection and set it onto the product, that does work.
I reviewed the BigCommerce API documentation, it reads that images is readonly.
Any thoughts?
Thanks

Try something like this, there is no support for uploading of the image files in this api.
Hope this helps
var product = new {
    Name = "Product Name.....",
    Sku = "Sku.....",
    Description = "Description....."
    ...
};
var response_product = Client.Products.Create(product);
var new_product = response_product.Data;

if (response_product.ResponseErrors == null && new_product != null) {

    var product_image = new {
    image_file = "i/866/HydromaticSPD100_w__41534.jpg",
    Description = "Description....."
    ...
    };
    var response_product_image = Client.ProductsImages.Create(new_product.Id, product_image);

    if (response_product_image.ResponseErrors == null && response_product_image.Data != null) {
                    //You now have added a new product_image
    }
}

Thanks for your reply.
Yeah, I'm just trying to get something into the database (doing a spike).  I don't care if it has images or not.
Do nothing with the product's Images Collection (Images Property) yields the response of ""The field 'images' does not allow null values.""  at line Client.Products.Create(product).

Did you in include the image_file property?
The BigCommerce Documentation says the product image (image_file) is required
BigCommerce Documentation- https://developer.bigcommerce.com/api/stores/v2/products/images
I really have not tested the ProductsImages.Create, maybe the file needs to be on the server before creating the record, not sure.

Product image (image_file) is just the path to the image not the image file.


🍒 Cherry pick PR #18735 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/18735 to staging 🍒

🚀 Cherry-picked to staging by https://github.com/mountiny in version: 1.3.15-11 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/yuwenmemon in version: 1.3.15-12 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 1.3.28-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.28-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


thoughts on the presence validator doing a trim()?

more of a discussion topic than an issue. i noticed the other day that the presence validator treats whitespace as present. this is probably not desired but i haven't seen anyone else report this type of problem. i'll probably write a custom nowhitespace-presence validator for the time being. thanks!

d'oh, it's a configuration option. thanks!


Update OS disk selection when creating a VMSS resource through CLI using a non-premium VM

Related command
az vmss create
--resource-group lobichar-rg
--name lorraine-vmss-test-4
--orchestration-mode Flexible
--image UbuntuLTS
--upgrade-policy-mode automatic
--instance-count 2
--admin-username azureuser
--generate-ssh-keys
--vm-sku Standard_D2_v4
Resource Provider
N/A
Description of Feature or Work Requested
When creating a virtual machine scale set (VMSS) through the Azure Portal, the default OS disk selection when selecting a VM that does not support premium storage, is a Standard SSD disk (exact disk SKU name is StandardSSD_LRS). When creating a VMSS resource using CLI, using the same VM (Standard_D2_v4, for example), the OS disk gets defaulted to Standard HDD (Standard_LRS).
The ask is to update the creation template so when creating a VMSS through CLI using a VM that doesn't support premium storage, the OS disk selection defaults to Standard SDD as well. This will provide consistency with the portal experience, and Standard SDD disks are better in terms of performance and availability.
Minimum API Version Required
N/A
Swagger PR link / SDK link
N/A
Request Example
No response
Target Date
N/A - the sooner the better as Standard HDD disks aren't as reliable as Standard SSD.
Additional context
No response

compute feature

@lorbichara Could you please specifically describe the changes in the interface of CLI command?

@lorbichara Yes, the az vmss create command uses the ARM template to deploy VMSS.
However, I found that when creating VMSS, if the --vm-sku is Standard_D2_v4, then in the ARM template built by CLI the storageAccountType is null
"storageProfile": {
    "osDisk": {
        "createOption": "FromImage",
        "caching": "ReadWrite",
        "managedDisk": {
            "storageAccountType": null
        }
    },
    "imageReference": {
        "publisher": "Canonical",
        "offer": "UbuntuServer",
        "sku": "18.04-LTS",
        "version": "latest"
    }
}

But from the response returned by the service, the storageAccountType becomes Standard_LRS
"storageProfile": {
    "imageReference": {
        "offer": "UbuntuServer",
        "publisher": "Canonical",
        "sku": "18.04-LTS",
        "version": "latest"
    },
    "osDisk": {
        "caching": "ReadWrite",
        "createOption": "FromImage",
        "diskSizeGB": 30,
        "managedDisk": {
        "storageAccountType": "Standard_LRS"
        },
        "osType": "Linux"
    }
}

'x-ms-correlation-request-id': '32e3e286-6e9b-40a4-9c16-fb09ffa4dce3'
So the default value is Standard_LRS not from the CLI side, but from the REST service.

Thanks for pointing this out @zhoxing-ms. I'm checking if it's VMSS team or DiskRP that owns the REST service and will ask them to update. I will close this issue for now, assuming that once they update, no change is needed from CLI side. Thank you!


Puzzleshark/add driver t4 template

Added the T4 template Entrypoint cpp file generation.

Closing this PR, since I believe https://github.com/microsoft/qsharp-compiler/pull/929 supersedes this PR.


basic concepts: Add a remote document that describes its differences from external dependencies

In External Dependencies there is a line

Note that remote storage is a different
feature.

but also in remote command ref

What is a "local remote" ?

While the term may seem contradictory, it doesn't have to be. The "local" part
refers to the type of location where the storage is: another directory in the
same file system. "Remote" is how we call storage for DVC projects.
It's essentially a local backup for data tracked by DVC.

AFAICT the difference between remotes and dependencies is this: Remotes are actually containers that can store external dependencies and external outputs. A file can be a dependency to a stage or an output from a stage, can may be stored in a remote or in .dvc/cache. Remote is a location designation, it means not stored in .dvc/cache. Dependency/output is a role designation, a file can be in dependency role for a stage, in output role in another stage, both if a stage reads and writes to the same file or neither if the file is not involved in any pipeline.
Is there any error in my understanding? @jorgeorpinel @shcheklein

@iesahin you got it mostly right. It's a confusion with terminology: The terms "remote" and "external" can have general meanings that are pretty similar. See #608 and #1127
A DVC Remote or what we most often call remote storage in DVC is a storage unit you configure for the project where (parts of) the cache can be pushed/pulled to/from.
But then on top of that we support external data in different ways (mainly as dependencies or outputs) and to indicate its location you can use the some of same URLs that remote storage configuration supports. There's even a special format like remote://myremote/path to locate external.
See also #520

both if a stage reads and writes to the same file

p.s. on this that's more of a Q about deps and out. I don't think you can define the same file as dep and out of a single state. There's other constraints like no overlapping outputs among stages.

That said I think we definitely want a concept page (and probably some guides) that extracts all the info on configuring remotes. But not sure we need to focus on a separate page to disambiguate external deps. Let's just keep that in mind as we address #520?


Proposal: Update the welcome message of dtm cli-tool

Description
Devstream is a very interesting program, I just followed it and learned about it through quick start. Could we set the welcome message "DEVSTREAM" to the same height?
Describe the Proposal
I guess the goal of the original design was to highlight the letters D and S. But in this case, all the letters are capitalized, could we adjust all the letters to the same height.
I have conducted relevant experiments and the comparative results are as follows.

Before

Update

Looking forward to your consideration

@IronCore864 Please take a look at this proposal.

@hxcGit we tried to avoid using all caps because it would feel like yelling.
We also considered using "DevStream" (a mix of upper and lower case) but smaller characters don't look so nice as ASCII art.
That's why we used the current form: it doesn't look as "yelling" as all caps, and the smaller characters look nicer than what lower case characters would look like.
As long as we don't use ALL CAPS, we are open to a change. Please make some proposals.
Thanks again! @hxcGit
@daniel-hutao please also have a look; see if it's reasonable.

@hxcGit we tried to avoid using all caps because it would feel like yelling.
We also considered using "DevStream" (a mix of upper and lower case) but smaller characters don't look so nice as ASCII art.
That's why we used the current form: it doesn't look as "yelling" as all caps, and the smaller characters look nicer than what lower case characters would look like.
As long as we don't use ALL CAPS, we are open to a change. Please make some proposals.
Thanks again! @hxcGit
@daniel-hutao please also have a look; see if it's reasonable.

Got it. I'll think about it later.


invalid request block size: 4243 (max 4096)...skip

Installed at Unraid but can't open WebUI. This is the log:
Operations to perform:
Apply all migrations: admin, auth, bookmarks, contenttypes, sessions
Running migrations:
No migrations to apply.
There is already a secret key in `secretkey.txt`
[uWSGI] getting INI configuration from uwsgi.ini
[uwsgi-static] added mapping for /static => static
*** Starting uWSGI 2.0.18 (64bit) on [Sun Sep 6 07:36:41 2020] ***
compiled with version: 6.3.0 20170516 on 07 June 2020 12:18:45

os: Linux-4.19.107-Unraid #1 SMP Sun Mar 8 14:34:03 CDT 2020
nodename: 86ee98549238
machine: x86_64
clock source: unix
detected number of CPU cores: 8
current working directory: /etc/linkding
writing pidfile to /tmp/linkding.pid
detected binary path: /usr/local/bin/uwsgi
!!! no internal routing support, rebuild with pcre support !!!
setgid() to 33
setuid() to 33
chdir() to /etc/linkding
your processes number limit is 123542
your memory page size is 4096 bytes
detected max file descriptor number: 40960
building mime-types dictionary from file /etc/mime.types...554 entry found
lock engine: pthread robust mutexes
thunder lock: disabled (you can enable it with --thunder-lock)
uWSGI http bound on :9090 fd 4
uwsgi socket 0 bound to TCP address <IP_ADDRESS>:34187 (port auto-assigned) fd 3
Python version: 3.7.7 (default, May 20 2020, 21:22:20) [GCC 6.3.0 20170516]

Python main interpreter initialized at 0x55a85049e140
python threads support enabled
your server socket listen backlog is limited to 100 connections
your mercy for graceful operations on workers is 60 seconds
mapped 416720 bytes (406 KB) for 8 cores
*** Operational MODE: preforking+threaded ***
WSGI app 0 (mountpoint='') ready in 0 seconds on interpreter 0x55a85049e140 pid: 10 (default app)
*** uWSGI is running in multiple interpreter mode ***
spawned uWSGI master process (pid: 10)
spawned uWSGI worker 1 (pid: 11, cores: 2)
spawned uWSGI worker 2 (pid: 12, cores: 2)
spawned uWSGI worker 3 (pid: 14, cores: 2)
spawned uWSGI worker 4 (pid: 16, cores: 2)
*** Stats server enabled on <IP_ADDRESS>:9191 fd: 18 ***
spawned uWSGI http 1 (pid: 18)
invalid request block size: 4211 (max 4096)...skip
[uwsgi-http key: <IP_ADDRESS>:9090 client_addr: <IP_ADDRESS> client_port: 3026] hr_instance_read(): Connection reset by peer [plugins/http/http.c line 647]

@edalcin Did you use the Docker image? Other than that this is not much to go on, I don't use Unraid, so I can't really tell what the issue might be.

Closing due to inactivity.

@sissbruecker I had the same issue using docker-compose on debian. In fact this had nothing to do with the host but with the client.
Using firefox in private mode (no addons) it worked. However with all my addons I got the same error message.
"invalid request block size: 4388 (max 4096)"
According to this:
https://topic.alibabacloud.com/a/when-uwsgi-is-deployed-to-nginx-invalid-request-block-size-4161-max-4096-appears_8_8_20029027.html
The header size of uwsgi by default is only 4k.
Upping it to 8k by adding this line in uwsgi.ini:
buffer-size = 8192
solved the issue.

@elmodor Thanks for debugging this, I guess increasing the header size in uwsgi should be no problem. I'll reopen this.


Drop Original Feature when Encoding

Can we set an option to drop the original feature feat_0 for the numeric encoders?
See this code below:
import category_encoders
import pandas as pd
import numpy as np
dd = {'feat1':['a','b','c','d','e'],'feat2':np.random.random(size=(5))}
df = pd.DataFrame(dd)
encoder = category_encoders.BackwardDifferenceEncoder(cols=['feat1'])
encoder.fit_transform(df)
this returns feat_0 as the first feaure and every row element is the number 1. Is this a desired property of should it be removed?

Thats a good idea.  I use the Statsmodels Contrasts functionality as closely as possible, and using your snippet modified to use different numbers of categories (up to 1000), it does seem that feat_0 has no obvious use.
I think rather than adding an option specific to this particular case (index 0 feature after backwards difference), adding an option to all encoders called something like 'drop_invariant', that dropped any columns with variance of 0 (or perhaps below some configurable threshold) would be more useful.
Do you think that would make sense?  If you're interested in helping with that let me know, or I can work on it.

Thanks for the prompt response Will! First ,I think the "drop invariant" idea you propose would be ideal actually. Since I last messaged you I came with this work around code, so not having the option right now is not hurting. See my example adding to my code from the earlier post:
dd = {'feat1':['a','b','c','d','e'],'feat2':np.random.random(size=(5))}
df = pd.DataFrame(dd)
encoder = category_encoders.BackwardDifferenceEncoder(cols=['feat1'])
new_df = encoder.fit_transform(df)
#Delete Feats with Bad behavior for certain encoders
for col in df.columns:
q = '%s_0'%(col)
if q in new_df.columns:
del new_df[q]
It appears that Helmert, Sum, and Polynomial have the same issue but this is easily remedied with the simple for loop below so this technically is not an "issue."
I also looked at your code and I am nowhere near the programmer you are. I would probably get in your way more than I could help with the invariant feature you talked about but if you really wanted me to I could take a stab at something...not sure if it would scale or fit your programming style though! :D

I'd be happy to help you out along the way, we can do it in a simple way.
In the transform methods of all of these encoders, we make sure that the X variable is always a dataframe, so you can use pandas tools before returning anything to drop low variance cols:
df = df.loc[:, df.var()>some_threshold]

So if you just add a parameter to the class for something like 'drop_invariant' as true or false and then 'variance_threshold' as some float, but defaulting to 0.0, then it should be a pretty straight forward addition.
Try it out and send a pull request, we can get it working.

This:

df = df.loc[:, df.var()>some_threshold]
actually makes A LOT of sense.

What I am scratching my head on is A) How to add the parameter to the Class (or any class for that matter--Classes confuse the crap out of me) and B) How this whole "pull request" thing actually works even if I were able to add the parameter.
Im such a newbie. I'll do my best though and try and send the pull request as soon as I can!

Github's docs on pull requests are pretty good: https://help.github.com/articles/using-pull-requests/, if you have trouble just let me know.
As for parameters, look at cols/self.cols in BackwardDifferenceEncoder, that is an example of one.  So you can add another one called 'drop_invariant' defaulting to false in line 49, set it to be an instance variable (i.e. self.x) in line 58ish, then use it as self.drop_invariant in the transform method.

Here is a pretty good overview on classes and OOP in general in python: https://jeffknupp.com/blog/2014/06/18/improve-your-python-python-classes-and-object-oriented-programming/

Thanks for the resources Will. I'll definitely give it a go as soon as I can!

As a side, for BackDiff,Sum,Polynomial, and Helmert encoder should the dependent variable (y) be specified in the .fit() call since the docs state the comparisons are made based on the dependent variable at different levels of the categorical?
So something like this new_df =BackDifferenceEncoder(cols=['feat1']).fit_transform(df,y) . What is interesting is that even if the code above is run without specifying y, a new dataframe with different dimensionality is created. I wonder what is going on internally here.

My understanding is that the regression of the dependent variable against the encoded matrix is the comparison of the dependent against the different levels of the categorical.  So to match the context of the statsmodels docs, we would be using the category encoders in this library as preprocessing before running an OLS regression or something like that on the now-encoded X and y.
So they'll use:
ols("write ~ C(race, Helmert)", data=hsb2)

Where we are just doing the C(race, Helmert) part here.

Ah, ok makes perfect sense. Thanks for all your help!

@pGit1 just checking in, are you still planning to work on this?


Reddit strategy doesn't redirect to failure URL on declined auth request

To reproduce:

Go to https://login-with.com/login
Login with Reddit
Hit the "decline" button on the reddit auth request page

You'll get redirected to https://auth.login-with.com/reddit/callback with {"error":null,"user":false}
This is happening because in routes.js here, both error and user are falsy and so it falls through to the res.json at the end.
A solution here would probably be to change line 34 to if (error || !user) - would you accept a PR for that? I'm not sure if this is happening for other providers, I've only tested with reddit.

👍  Sure, PR is very welcomed! I think if (error || !user) totally makes sense. Should have  a meaningful error of course...

https://github.com/lipp/login-with/pull/34

Hey, any chance we can get a version bump and updated docker image pushed out with this?

Sure... tomorrow... promise


Be smart concerning "matchingVariants" flag.

The latest version shorthandedly deactivated the usage of the "matchingVariants" flag in the search results listing and offers only a comparably undifferentiated alternative that activates it all the time.
Since it's a necessary feature that leads to "broken" results for the user  (filter for green or size XL, see blue or XS in the listing) I suggest that a smarter variant is implemented as default:

automatic:  if a filter (not full text search) is set (by the user) on an attribute that does not have the constraint "same for all", matching variants are calculated
manual:  it's made part of the faceting configuration which attribute filters trigger "matchingVariant" calculation.

it can still lead to idiosyncraties e.g. when searching for a color name in  the full text search, but at least it qualifies as "acceptable default" to me.

It is a good idea. Probably the manual version would be the less complex and yet most complete option to implement first.
But having it disabled is a perfect valid option, it all depends on how your data is modelled, hence the two options Sunrise offers. As an example, our Sunrise data doesn't require matching variants enabled.

Agree that manual is more coherent as long as the facet selection is "hardcoded" in the frontend anyways.
The demo data set is not a reference for default behavior though - It's fake so e.g. all sizes are always available.
But even there: e.g.  i could configure a product discount rule on XS clothing on "small peoples day" and boom, the search results would show the wrong price after having set a filter on "XS"


chore: absolute imports (embed)

Ticket: TA-2936 [Embed] Set up absolute imports and replace relative paths

@brunodesde1987 getting this when running in dev mode (yarn dev), I can help debug later in case

@brunodesde1987 getting this when running in dev mode (yarn dev), I can help debug later in case

@luca-gr4vy
Nice catch.
This seems to be an old issue. It's happening on the main branch too.

:rocket: PR was released in v2.15.0 :rocket:


Integrate GPTQ with Quantization Infra (Keras only)

Integrating quantization infrastructure into GPTQ (Keras) to allow simple use and development of GPTQ quantizers:

Added BaseKerasGPTQTrainableQuantizer base class for all GPTQ Keras quantizers.
Organized quantizers classes methods and parameters.
Removed GPTQ model builder and weights config and replaced them with QI's quantization_builder template and trainable quantizer configs.
Additional changes:

Moved quantizers' class getters to QI (instead of duplications in QAT and Exporter) and split them to "trainable quantizer getter" and "inferable quantizer getter".

Modified and extended Keras GPTQ tests.

Only small typos. small reminder.. did you check that tutorial 'example_keras_mobilenet_gptq.py' still ok?

Ran it now and actually found a bug. It fixed now. Thanks for the advice.


gpg: keyserver receive failed: Server indicated a failure

root@61f86a7af7e2:/etc# sudo apt-key adv --keyserver hkp://ha.pool.sks-keyservers.net:80 --recv-key 8AB59D3A2BD7B692
Executing: /tmp/apt-key-gpghome.K5OWoHWrnC/gpg.1.sh --keyserver hkp://ha.pool.sks-keyservers.net:80 --recv-key 8AB59D3A2BD7B692
gpg: keyserver receive failed: Server indicated a failure

Hi @Lyk77,
Could you please give us more details about what you are trying to do here? It is unclear to me whether this error message is related to ifm. Thanks!

@lola-masson I believe it's related to the keyserver not responding. I'm facing the same issue as well.
Executing: /tmp/apt-key-gpghome.an4Ls3y7Ab/gpg.1.sh --keyserver hkp://ha.pool.sks-keyservers.net:80 --recv-key 8AB59D3A2BD7B692 gpg: failed to receive from keyserver: Server indicated a failure
Is there an upated key/keyserver?


Saving profile changes now works correctly with irods user account

@pkdash @mjstealey This simple one-line change will fix the bug @mjstealey found regarding saving profile changes resetting iRODS user account setting. @pkdash Please give a quick review so that it can be merged.

@pkdash @mjstealey Also just checked in the changes made to fix the issue 1322 which involves addition of a conditional statement on profile page. So this PR will fix both issue #1339 and #1322. @pkdash Please review to see if you can give a +1 to have it merged for release 1.7.3.

+1


Extend semantics of HTTP endpoints

When trying to write actual application code, I'm finding the simple mapping from Function, Supplier and Consumer to HTTP/REST actions a bit limiting. Some ideas...

Tighten up the content negotiation, so that HTTP clients know what to expect. E.g. actually test what happens when you POST to a Consumer.

Make it easy to express the common idiom "POST an entity" get back a 202 (Accepted), or a 201 (Created) or 204 (No Content)? Make it possible for the developer to control these patterns.

Maybe allow users to "replay" content that was streamed into a Consumer (automatically generate a GET endpoint that mirrors a POST for instance).

More nuanced handling of infinite streams. Pagination as an alternative to SSEs, for instance.

Pagination idea: use Link headers (https://tools.ietf.org/html/rfc5988).

Example where "/" separators would be useful: the Commander pattern. Commander has 2 groups of endpoints (/commands and /events).

/commands accepts POST and publishes the entities (after maybe some decoration) to a topic/Supplier. It also allows commands to be listed and viewed by id, e.g. at GET /commands/{id}.

/events accepts POST from consumers of the commands (each event is a reaction to a command or another event), and publishes the entities to a topic/Supplier. It also allows commands to be listed and viewed by id, e.g. at GET /events/{id}.

You can implement the basic POST with 2 Functions called "commands" and "events", but the GET by ID and the "replay" endpoints cannot be implemented with "natural" URIs yet.


CCE-541 add SmartEdit icons

Some small issues:

New icons don't convert when running "gulp iconfont"... I checked the svg code but I dunno what's the problem. Maybe is the size, they are 32x32, other ones are 133.2x133.2..
We have already an "add" icon, let's use the old one

Changed the icon filenames to be only one word.
SmartEdit icons are hidden on the page, since the style is different.

looks good.


Flow claims reachable code is unreachable

Flow thinks the return statement is unreachable:
/* @flow */

function foo(x: ?number): string {
  switch (x) {
    default: {
      if (x) {
        break;
      }

      throw new Error('hmmm');
    }
  }

  return 'yup'
}

TryFlow error:
14:   return 'yup'      ^ unreachable code

But the return statement is in fact reachable when x is a truthy number. Flow doesn't seem to understand the effect of break in this context, and behaves as if the throw statement is guaranteed to occur in all cases.
Note: Usually it's easy to work around these kinds of issues with invariant, but in this case it's more difficult because the invariant(...) statement is also deemed unreachable.

You can get around this with an else statement, though it does definitely seem like a bug:
function foo(x: ?number): string {
  switch (x) {
    default: {
      if (x) {
        break;
      } else {
         throw new Error('hmmm');
      }

    }
  }

  return 'yup'
}

Issue still exists, I believe its a bug and needs to be resolved.


onClusterMouseover onClusterMouseout not working

the mouse event in MarkerClusterGroup not working

"react-leaflet-markercluster": "^3.0.0-rc1",
"leaflet.markercluster": "^1.4.1",
"leaflet": "^1.7.1",
"react-leaflet": "^3.1.0",

its working with the props   onMouseover onMouseout onClick  ...


Any ALT+... combination opens menu

I use two languages and switch with alt+shift combination. But if I try to switch language in app it also opens menu and I cannot type anything, until I press ESC. This is VERY annoying, because I need to use both languages.

I'm afraid I am no longer supporting this project, so I will not be looking into this issue. Please see the updated README for more information. I will leave the issue open, in case of a new maintainer who is able to look into it.


Popover stopped working in v2.5.0

The issues forum is NOT for support requests. It is for bugs and feature requests only.
Please read https://github.com/angular-ui/bootstrap/blob/master/CONTRIBUTING.md and search
existing issues (both open and closed) prior to opening any new issue and ensure you follow the instructions therein.
Bug description: Popover is not working in version 2.5.0, It's working in version 2.4
Link to minimally-working plunker that reproduces the issue:
Steps to reproduce the issue:
Version of Angular, UIBS, and Bootstrap: v2.5.0
Angular: v1.6.4
UIBS: v2.5.0
Bootstrap: v3.3.7

can not reproduce.


Fix filename comment in TaskDelegate header

The file was likely renamed without the comment being updated. Comment used to report:
//  Error.swift

Good catch! Thanks for the fix. 🍻


Collect appconnect_time in CurlFactory TransferStats

Context
For some historical reason the original implementation of curl_getinfo($ch) did not include the appconect_time metric and wasn't available until PHP 5.5.0, where it can be retrieved separatedly by passing the CURLINFO_APPCONNECT_TIME constant as the second parameter of curl_getinfo($ch, $opt). appconnect_time is the time it takes cURL to negotiate the TLS handshake with the server after it establishes a TCP connection.
This PR records appconnect_time into the TransferStats::$handlerStats array, so it can be read and used in on_stats callables.
References
https://www.php.net/manual/en/function.curl-getinfo.php#refsect1-function.curl-getinfo-changelog
https://curl.haxx.se/libcurl/c/CURLINFO_APPCONNECT_TIME.html
http://docs.guzzlephp.org/en/stable/request-options.html#on-stats

Well done @1ma

@1ma thanks! Can you please rebase your branch?

Yup, done.


Pacu advanced reorg

all credit to @pacu . i rebased some of his test framework on our reorganized dev branch

If you don't keep the commits I did then the following PRs will be  as hard as this one because they are based on each other


WIP: Registration page tweaks.

So far I have worked on these. These are mostly Javascript changes to the Registration page.

[x]  The School select field updates dynamically based on the university the user selected.

[x]  Now the username is checked asynchronously after the user has finished typing the username.

[x]  All the fields can we traversed by pressing enter.

[ ] University specific email ( Though I think having users give their personal email will be better than their university email )

Fixes #48

@digi0ps Let's make university email mandatory during registration, since it's needed for verification.
Or what about this?

When the user clicks on register, we can ask him/her to enter his university email id. If it's valid and non-flagged, he can proceed to the registration form? (However, his uni's email will be attached to his user object)

@jeremyphilemon Okay but I still think asking them there Uni email is not a good idea.
I will list out the pros and cons.
Pros:

The email be unique for each user.
We can blacklist their email and make sure that user doesn't return.

Cons:

The user will feel the lack of privacy as they know that we have their university identity. This will suppress their opinion and won't give them complete anonymity.
We can't notify them or send digest emails as students rarely check their uni emails.
What about the existing user database?

The user will feel the lack of privacy as they know that we have their university identity. This will suppress their opinion and won't give them complete anonymity.

We can mention that the email is used for verification purpose only and the said information will be protected no matter what happens. Regardless of the flagging system, this verification can prevent impersonation of other students too (think about the consequences of that too).

We can't notify them or send digest emails as students rarely check their uni emails.

Exactly, the uni emails are for verification only while the other email id they provide during the actual registration can be used for sending them emails.

What about the existing user database?

They're verified beta testers by default ¯\(ツ)/¯

Okay what you are saying seems good to me!
But won't it be irritating for users to enter two emails?

I don't think we should compromise on the user's identity and safety for just another email field...

Okay! Where will we be asking the uni email?

@digi0ps Like first we could show one field (centre of the screen) to enter their email id, then if it's valid the field moves to the left and the registration form enters from the right?

@digi0ps Alright, this looks good. We can work on the university specific email by opening another branch!


Is there a way to prevent ads on videos?

From what I know, if the youtube video you are playing has monetization turned off then I shouldn't see ads when playing the video with any library. Can someone confirm that to be true?
Thanks!

Should be correct. Can't guarantee though, ads on the embedded player seem to have strange behaviors sometimes.


Fixing like 250 ESLint Problems (Code Style)

I fixed a lot of code style problems according to our ESLint checker. Some of the files with more problems to fix (and potential merge conflicts if I tried to change them), are now just prefixed with /* eslint-disable */. Now, no more ESLint warnings spam the terminal & DevTools.
I suggest installing the ESLint VS Code extension to discover these problems earlier in the future.

When we are disabling eslint almost in every line why just not delete the eslint file completely?

When we are disabling eslint almost in every line why just not delete the eslint file completely?

I fixed them everywhere in our teams' code files, but I didn't want to interfere with the stuff the other teams did. I think it is beneficial to have a common code style, so I think leaving the warnings is good, but with those files the warnings were just too much. On the other hand I wanted to get warned about my own code style problems. Therefore I left the files from outside our team untouched, just disabled the warnings for them. I contacted the people who wrote the code which had those warnings and they said they would look into changing the code style to be adapted to the ESLint config.
But maybe we shouldn't be so strict about the code style, I don't know. If we don't care about the code looking uniform everywhere, we could of course also just disable linting completely.


Framless windows?

Is it possible to add a frameless window in Eel? Like in electron we can just do frame: false. How about Eel?

This is a feature of the browser. The latest version of Eel supports Electron much better than before so I would just say use Electron if you want frameless window


Can't do anything in the settings

I can't click or move anything in the settings window. Help

Hold the CTRL key while right-clicking to open rainmeter skin context menu. Be sure to disable the "cluck through" option by unticking it.


Fix PHPUnit deprecation warning

assertRegExp is deprecated in PHPUnit 9.4 but assertMatchesRegularExpression is not available in PHPUnit 8.1
This should fix travis build after #9

Sorry, I didn't see this and I just fix this issue (plus others changes) to provide support to PHP 8.
Thanks anyway!


Pass connection profile to extension contributed actions in dashboard toolbar

The connection profile of the server/database dashboard wasn't getting passed to the extension contributed actions

Coverage remained the same at 36.02% when pulling 9a46184d030f05e463b8cfd6acb4ee206119822c on kisantia/toolbarPassConnection into 6d94d795272a01266a88c6b7d250bceb02de5adc on main.

which bug are you trying to fix?

@alanrenmsft created issue #11219. I noticed this when trying to add "Import New Database Project" to the dashboard toolbar


Changing GitHub Personal Access Token

When I started using the plugin, the personal access token I assigned did not have correct privileges.  This was reported by the plugin when I tried to perform any actions with the plugin such as accessing PRs. The error messages did not report any way to ix the credential issue and I could not find any fields in the plugin settings to set or update the PAT.
My fix was to remove and re-install the plugin to get prompted for my GitHub credentials. When my credentials expire in a few weeks, I'm not sure how I can avoid another remove/re-install cycle - a plugin option to do this would be great.
Thanks for this super useful plugin! :)

Extension version:
VSCode Version: v0.28.0
OS: Windows

Steps to Reproduce:

Create PAT with basic repo-only access permissions
Install plugin and use that token as your credential
Try to perform any operations and view errors messages about lack of access to read email settings etc.

You should be able to logout using the account icon


Update to blazor 3.2 preview 1 and netstandard21

Some packages are on 3.1.1 now.
Closes #21, #40, #41
Review please.
Had run problems so I basically re-created the test projects and copied the files in, after finding something about build target being defined on project creation preventing me from running the application. So that messed with the solution file.
Also, can I add an editorconfig file?

Whoops, forgot ot actually fixup the fixup-commit.
Fixed the indenting and bumped the version.

Right, I totally forgot that. I also updated the aspentcore version in the Readme.

Thanks for this. I'll roll a NuGet package soon.

Hold on a sec.. Is this now obsolete as in blazor 3.2 preview 1 - microsoft signal r client is not officially working: https://devblogs.microsoft.com/aspnet/blazor-webassembly-3-2-0-preview-1-release-now-available/

It seems this should be obsolete now. Although I can't "just make it work" right now using this chat as a basis, not sure why. Maybe you can see what's wrong:
https://github.com/Regenhardt/BlazorSignalR/tree/feature/obsolescense


Fork options package with ability to add global scopes

No response on https://github.com/appstract/laravel-options/issues/56 after 10 days, so forking seems wise in the short term.


WIP: Add a mechanism to reuse (bot) user email addresses.

Fixes: #1183.

Automated message from Dropbox CLA bot
@HarshitOnGitHub, it looks like you've already signed the Dropbox CLA. Thanks!

Hi @HarshitOnGitHub, can you give us an update on where this PR stands?   Have you had a chance to discuss this with anybody on https://chat.zulip.org/ ?

@showell Backend related work is almost complete. Some frontend changes are left to do. I am finding very less free time due to my exams. I will get free by next week. I will complete it then. And yes I had discussions regarding this PR with @timabbott.

@timabbott @showell This is also ready for a review!! :tada:

@HarshitOnGitHub can you rebase?

@timabbott Doing!!

I merged the first 3 commits, since they merged cleanly and looked good.  So this will be a bit smaller of a PR once you rebase :)

@timabbott Done!!

This is looking great @HarshitOnGitHub!  I merged several pieces of it.
The reason I didn't merge the rest is that the UI pieces will merge conflict pretty strongly with, https://github.com/zulip/zulip/pull/3784.  I think it makes sense to merge #3784 first, because it has more involved changes to the actual code (whereas my read is most of your changes are just putting things inside an if statement), so I think it'll be less work to do the rebases in that order.

#3784 is merged, so I think you should be able to rebase this now.

@timabbott Rebased!!

@HarshitOnGitHub did you test this after rebasing?  I don't see the reactivate UI on the /#your-bots page...

Oops, my git state wasn't what I thought it was.

Looks good, merged, thanks @HarshitOnGitHub!
I think it'd be a good follow-up project to this to clean up the "bot creation" form, since now that looks really bad in the "inactive bots" tab.  @brockwhittaker did you end up doing work on that which @HarshitOnGitHub should pick up?

Closing this one since it has been merged.

Thanks @HarshitOnGitHub!


[move-compiler] Allow implicit copies on all copyable types

Allow implicit copy on any value whose type has the copy ability
Switch last usage of every variable to a move
Refuse/error to infer copy/move if the local is borrowed during its last usage

Motivation

It has been a source of confusion for a while
The original motivation of only allowing primitives was to save gas. But it doesn't really help with that in a general setting. Sure, it helps in this one narrow case, but there are so many potential gas issues that I don't think accidental duplication is at the top of the list. Hopefully this new assumption isn't wrong

Test Plan

new test

:broken_heart: Test Failed - ci-test

There seems to be non-determinism in the yul baseline tests. Looking at a fix.

There seems to be non-determinism in the yul baseline tests. Looking at a fix.

Once #86 lands and after rebase, this should be gone.

:lock: Merge Conflict

/land


Added send email check when changing usernames

Fix for error produced when changing usernames with the USER_ENABLE_EMAIL flag set to False
[2019-07-11 10:34:59,531] ERROR in app: Exception on /user/change-username [POST]
Traceback (most recent call last):
  File "/home/ubuntu/github/my_app/venv/lib/python3.5/site-packages/flask/app.py", line 2311, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/ubuntu/github/my_app/venv/lib/python3.5/site-packages/flask/app.py", line 1834, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/ubuntu/github/my_app/venv/lib/python3.5/site-packages/flask/app.py", line 1737, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/home/ubuntu/github/my_app/venv/lib/python3.5/site-packages/flask/_compat.py", line 36, in reraise
    raise value
  File "/home/ubuntu/github/my_app/venv/lib/python3.5/site-packages/flask/app.py", line 1832, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/ubuntu/github/my_app/venv/lib/python3.5/site-packages/flask/app.py", line 1818, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/home/ubuntu/github/my_app/venv/lib/python3.5/site-packages/flask_user/user_manager.py", line 390, in change_username_stub
    return self.change_username_view()
  File "/home/ubuntu/github/my_app/venv/lib/python3.5/site-packages/flask_user/decorators.py", line 58, in decorator
    return view_function(*args, **kwargs)
  File "/home/ubuntu/github/my_app/venv/lib/python3.5/site-packages/flask_user/user_manager__views.py", line 141, in change_username_view
    self.email_manager.send_username_changed_email(current_user)
AttributeError: 'UserManager' object has no attribute 'email_manager'

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@14d30bb). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master     #267   +/-   ##
=========================================
  Coverage          ?   88.55%
=========================================
  Files             ?       25
  Lines             ?     1451
  Branches          ?        0
=========================================
  Hits              ?     1285
  Misses            ?      166
  Partials          ?        0

Impacted Files
Coverage Δ

flask_user/user_manager__views.py
95.73% <100%> (ø)

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 14d30bb...4d01986. Read the comment docs.


Preserve bundle-size; backward-compatible with unpadded placeholder.

🚧 Do not merge. Still validating solution. 🚧

Verified that error traces are now mapping to the correct code points in Chrome & Safari.


Make eval("arguments") work inside of lambdas.

Fixes #1209
This is fairly close to the solution @aneeshdk @ianwjhalliday and I discussed. There's one minor difference: in order for the new location of AddVarsToScope to be as clear as possible, we planned to move it above all of the paramScope code. Sadly, the call the pushScope(paramScope) needs to happen prior to AddVarsToScope to avoid an assertion failure in ByteCodeGenerator.cpp:1659 due to currentScope not being updated.
I couldn't come up with a less repetitive way to test this than to just copy/paste the bulk of the "arguments in lambdas" test, and replace the use of arguments with eval("arguments"). Definitely open to suggestions here.

Hi @jordonwii, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Jordon Wing). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

It turns out that in my first push of the test, I mistakenly included an eval call inside of the new test body in lambda1.js, and that buried some failures which I'm currently trying to figure out. In the debug builds, there's an assertion failure I don't understand yet, and in the test builds, arguments.callee gets set to assert.areEqual rather than the lambda's non-lambda parent.

I can't seem to reproduce either the test failure or the assertion failure outside of running runtests.cmd. The failures only happen in the "dynapogo" variant, and if I copy/paste the command it reports that it's running and run ch.exe, there are no failures. Any ideas as to why this might happen?

Contacted @jordonwii to follow up on this PR.

Progress, at last. :)
It looks like the errors I mentioned in my last comment were the result of the jit trying to inline the outer function, when it shouldn't have been able to because the inner Lambda used eval. The code added in these last few commits fixes that by marking the outer function arguments object as referenceable. I haven't figured out how to verify that this isn't over compensating somehow, but right now it appears to work.
If this looks good, I'll flatten the commits tonight and add a couple comments, especially to the tests.

@pleath: I contacted @jordonwii offline to discuss this PR and he said he was under the impression that this PR would be superceded by other refactoring work or another change. Is that the case? If so (regardless of whether that work is in progress or not), we should close this PR.

Per offline discussion with @pleath I'm closing this issue.


Add release tags

Please maintain consistent releases by also adding Git tags and pushing them to this repository.
Please consider using semantic versioning for release tags.
$ git checkout {some-release-commit}
$ git tag -a "v.1.0.0" -m "Short summary of the release."
$ git push -tags origin master

Done, thanks

Nice. Thank you.


OSSL_DESERIALIZER_from_bio fails to read EC .pem encoded key files

OSSL_DESERIALIZER_from_bio fails to load EC key files (unencrypted).
-----BEGIN EC PARAMETERS-----
BggqhkjOPQMBBw==
-----END EC PARAMETERS-----
-----BEGIN EC PRIVATE KEY-----
MHcCAQEEIJaxcqfc/0EVHu9MYZRoMW09rIW9KcszL5F2OSU7L2SYoAoGCCqGSM49
AwEHoUQDQgAEZcaJuVZdivcCLFnzsUguRxmo9HHbg9IJagpQ5QNTQkYDHzNB0jBe
gdklfiqxjQFYour0Sibux1qU3DtpmgtRAQ==
-----END EC PRIVATE KEY-----

Does it work if you remove the parameters section?

Yes, it works when the EC PARAMETERS section is removed.

Uhm, ok!
It requires further inspection, as if this was working before it should keep working.
On a side note, the parameters sections is superfluous, as it is repeated inside the EC key structure. I actually wonder what happens if the 2 sections were in conflict!

I actually wonder what happens if the 2 sections were in conflict!

Nothing, as far as I know.  They are two separate sections, and should return ass two separate objects (EVP_PKEYs).  That is, unless there's higher level code that checks for such combinations, but I doubt we have that (we have such a mechanism for certificate trust of some sort, I forget the details).

@matbech, please try #12569.

Something to be noted, @romen, is that EC key generation generates that kind of output.  Example:
$ ./util/wrap.pl ./apps/openssl ecparam -genkey -name prime256v1
-----BEGIN EC PARAMETERS-----
BggqhkjOPQMBBw==
-----END EC PARAMETERS-----
-----BEGIN EC PRIVATE KEY-----
MHcCAQEEIO9NY/EAxP2L39o4DfTs7TPzt3EvlQ6/X/Y40YtiniMroAoGCCqGSM49
AwEHoUQDQgAEMB8OqdnovZGeNlzAaGxBssxs7kMaiBrWZrY0i1B7cZAYCTzo+a6d
S9vJB9D/0Ukhd4uDJRIxACEddM8zOPqAtA==
-----END EC PRIVATE KEY-----

@levitte With the patch, OSSL_DESERIALIZER_from_bio succeeds, but the returned pkey only seem to contain the EC PARAMETERS, e.g. EVP_PKEY_get_bn_param with OSSL_PKEY_PARAM_PRIV_KEY, and OSSL_PKEY_PARAM_PRIV_KEY are failing.

The deserializer only processes one object at a time, and this is two objects. That essentially means that you have to call OSSL_DESERIALIZER_from_bio() twice, using the same OSSL_DESERIALIZER_CTX.
Essentially, OSSL_DESERIALIZER isn't aware of multiple connected objects. That kind of connection has to be made at a higher level.

The EC parameters are also in the key, right?  If so, how does one write an application that reads one or both parts?

For the moment, we have no public function to find out exactly what an EVP_PKEY contains (parameters, public key, private key). We do have the internal functionality for it, but only for provider-native keys; EVP_PKEY_ASN1_METHOD is lacking in this regard.
OSSL_STORE, however, returns information on what kind of object was just loaded, and it does distinguish parameters from key material EVP_PKEYs.
A side suggestion is to stop using openssl ecparam - genkey and switch to openssl genpkey


Flip the tests around

I had a little epiphany the other day as I was working on the unit tests for the form labels rule
I think we've been using the testing tools the wrong way around. I'll try and explain.
The business rules that we need to use examples to illustrate, are the standards themselves. So what we need are features with several scenarios that give examples of good and bad content. We can use these features to test the tool itself using Cucumber, but the tool does not necessarily need to then use Cucumber to actually test people's pages. In fact RSpec would be a much better fit, I think.
So for example, rather than having a scenario like this:
  Scenario: Validate use of labels
    When I visit the page
    Then the following form elements must have a title attribute or associated label tag:
      | select                     |
      | textarea                   |
      | input[type=checkbox]       |
      | input[type=color]          |
      | input[type=date]           |
      | input[type=datetime]       |
      | input[type=datetime-local] |
      | input[type=email]          |
      | input[type=file]           |
      | input[type=month]          |
      | input[type=number]         |
      | input[type=password]       |
      | input[type=radio]          |
      | input[type=range]          |
      | input[type=search]         |
      | input[type=tel]            |
      | input[type=text]           |
      | input[type=time]           |
      | input[type=url]            |
      | input[type=week]           |

We could have scenarios like this instead:
Scenario: Single-field form label associated via `for` attribute
  Given a page with this html in the body:
    """
    <label for="search">Search the BBC</label>
    <input type="text" id="search" name="q" />
    """
  When the page is checked for compliance with the accessibility standards
  Then the form labels check should pass

Scenario: Field with no label or title attribute
  Given a page with this html in the body:
    """
    <input type="text" name="q" value="Search the BBC" />
    """
  When the page is checked for compliance with the accessibility standards
  Then the form labels check should fail with:
    """
    'input' tag at /html/body/input has no title attribute or associated label
    """

Fixed in 7664d4ee4b5aa2c67c73d816f442bf8bb759c959


'cannot list resource "helmrepositories" in API group "source.toolkit.fluxcd.io" at the cluster scope'

This error is shown in the wego logs which leads to an error on the UI.
We are possibly not scoping a call to a namespace, so the call is going cluster-wide, thus asking for a cluster scope.

The errors
cannot list resource "helmrepositories" in API group "source.toolkit.fluxcd.io" at the cluster scope

and
{"error": "list resource: deployments.apps is forbidden: User \"system:serviceaccount:flux-system:wego-app-service-account\" cannot list resource \"deployments\" in API group \"apps\" in the namespace \"flux-system\""}

are caused by this bit around profiles
we are not sure why profiles are in core, so for now i am going to leave this and come back round once i have finished the service account work

Update for those playing along at home:
So this profiles (helm) watcher is start on wego-app launch. This means that it will use whatever powers the wego-app SA has to look up resources. It does not impersonate at all, and I am not sure it can, since queries do not come from the user; the watcher is just "on" all the time. This gets in the way of our "impersonate all the things all the time plan". To maintain the current profiles feature, the wego SA would need to have Cluster scope access to the objects that profiles cares about.

It can run with its own ServiceAccount?

And for this, I think it's fine to not impersonate all the time, as you say, it's not the user that's doing the queries.
It does raise some interesting questions around which ones we can show data from tho'.
So, maybe we query for the HelmReleases with impersonation, and use that to look in the cache?

It does raise some interesting questions around which ones we can show data from tho'.

Yeh that's an interesting thought.
I am going to close this ticket, as the error is solved, and will open a new ticket to do the "make sure people see profiles they are meant to" bit.

@bigkevmcd by "cache" are you talking about the disk-based profiles cache which I just spotted?

@Callisto13 yes, there's a cache of the parsed data extracted from the Helm charts repositories.

👍  coolio


API doesn't fetch sprint status data instantly - Feature

When a sprint is started with the current date and time given for starting date/time in JIRA, the collector updates the issues in database including sprint start and end dates, but the API does not return that data. But when I start a sprint with a past date/time, it returns the data instantly. I assume that it's an issue with the API's date/time comparison..

I got the same error. in API  use the UTC time how to fix that. it's working after changing the time but its not the correct way.how to fix it?

Found the reason, the JIRA API always sends UTC time, when the JIRA collector stores data in the DB it converts the time to local system's time. You can change that from the collector.


Add missing dependencies to setup.py

I needed additional two requirements to install it in clean virtualenv.

LGTM ;)


Windowed mode out of makefile

I got tired of us always changing the screen resolution in the makefile. This PR adds the following:

If running "make" then emerald will default to your current screen size
If setting DEVELOP (export DEVELOP=1) in shell before running make (once per shell) emerald will compile with 1024x600 resolution
When running emerald there are new command line arguments:

-f, --fullscreen        Run emerald in fullscreen
-sf, --scene-file       Choose scene file for emerald

Nice one.
But might get annoying to export the environment variable instead?
What about having different make targets? You can have common flags then dev flags and non dev flags like their respective resolution. Just a thought.

One could just set the line in .bashrc / .bash_profile (like I have) if you don't wan't to set it for every shell. You can also set the resolution as shell variable there.

I should probably name the variables something like "emerald_develop" or something like that.

I've thought about this and I think environment variables is the best way to go here unless we want to rewrite our shader pre-compile-stage to run during Emerald runtime instead of at compile time.
Until then i think this will have to do. Changed the variables. I removed "DEVELOP" and are now only using this within my ~/.bash_profile:
export EMERALD_SCREEN_WIDTH="SCREEN_WIDTH=1280"
export EMERALD_SCREEN_HEIGHT="SCREEN_HEIGHT=800"

I suggest we merge this so that we can have user-independent config - and a default which compiles to screen res.

The window size gets the size of both my monitors together when I have not exported the env variables. Do you think this can be fixed easily?
Wouldn't it be nice to set the width with a command line argument?
Like
./emerald -w 1280 800

"Eyefinity" support 😂 I'll look into this - I assume the problem is on Mac OS since I have dual-monitors on My Linux system.
We use screen size information in shaders which are prepended in pre-compile stage running when you type make. I could rebuild this so that the pre-compile stage for shaders is done at emerald run-time. There's however some ammount of work for this to be Done - bot not impossible at all.

3 dec. 2016 kl. 20:45 skrev Isakwiberg<EMAIL_ADDRESS>The window size gets the size of both my monitors together when I have not exported the env variables. Do you think this can be fixed easily?
Wouldn't it be nice to set the width with a command line argument?
Like
./emerald -w 1280 800
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@Isakwiberg Non fix on multi-monitor problems on OS X due to this bug: https://bugzilla.libsdl.org/show_bug.cgi?id=2479


chore Added order by create_time for show routine load

Proposed changes
Issue Number: close #xxx

run buildall

run P0

run P0 Regression

run P0 Regression (Doris Regression)

run p0

反向顺序是不是更好？

reverse order is much better?

No, I think the average user would want to see information about the newly created task, for example I just created a task and if there are 100 historical tasks I need to scroll up many pages

reverse order is much better?

@dataroaring In addition, I will add the order by and limit syntax in the future to give users more choices

反向顺序是不是更好？

reverse order is much better?

No, I think the average user would want to see information about the newly created task, for example I just created a task and if there are 100 historical tasks I need to scroll up many pages

according to your example in commit msg, the newest is put at the end.


add new lur-cache in secondary-cache to bench

This PR got some whitespace issues, so I added it manually.
Average results I get on my machine: https://github.com/medikoo/memoize#benchmarks

after replace "now" to "performance-now"(hrtime), I found this scene is still a little slower than lru-cache indeed.
Fibonacci 3000 x10:

1: 25.04ms  Underscore
2: 34.17ms  lru-cache                 LRU (max: 1000)
3: 36.99ms  secondary-cache           LRU (max: 1000)
4: 37.70ms  Memoizee (primitive mode)
5: 66.78ms  Lo-dash
6: 108.92ms  Memoizee (primitive mode) LRU (max: 1000)
7: 264.58ms  Memoizee (object mode)    LRU (max: 1000)
8: 300.27ms  Memoizee (object mode)

@snowyu this benchmark is not perfectly reliable, and I assume that going down to microseconds won't bring better accuracy. To test it well, each utility will have to be run alone in separate process, at some point I'll try to fix it.


Error trying to create managed stack

Trying to follow the readme and getting stuck with creating the managed stack.  Getting an error that the github application installation is not found even though the application installation must have been working enough to create the other resources.

I believe that you've already solved this shorty after posting it going by an internal chat thread
github_enterprise {
    namespace = "yourNamespaceGoesHere" # The GitHub organization / user the repository belongs to
  }

This would be needed on every stack (if using github + other examples for bitbucket / gitlab in the link).
If you don’t add the block, we’re assuming that you’re using the Spacelift-managed GitHub app.

Hi, you can now set TF_VAR_github_app_namespace={your_github_org_name_or_username} as an environment variable in spacelift to fix this issue.


Remove creation of infra_rg variable

The variable was used in line 71 before https://github.com/MicrosoftDocs/azure-docs/commit/63caa9789bf01240817016993e031982645ed114.

#label:"aq-pr-triaged"

#sign-off


Both AddScreenCaptureFromBase64String and AddScreenCaptureFromPath do nothing.

I try to add a screenshot to the report, both: AddScreenCaptureFromBase64String and AddScreenCaptureFromPath do nothing, don't throw any exception either.
_extentScenario.CreateNode(new GherkinKeyword(stepType), scenarioStepContext.StepInfo.Text);
var screenshot = ReportingUtils.TakeScreenshot(driver);
_extentScenario.AddScreenCaptureFromBase64String(screenshot.AsBase64EncodedString, "aTitle");
var aPath = "picture.jpg";
Assert.That(File.Exists(aPath)); //it does
_extentScenario.AddScreenCaptureFromPath(aPath, "aTitle");

No screenshot is being added to the report, the step itself is added correctly besides the keywords as reported in the previous issue.
As I inspected the code, adding screenshot causes this to execute:
    internal void AddScreenCapture(Test test, ScreenCapture screenCapture)
    {
        StarterReporterList.ForEach(x => x.OnScreenCaptureAdded(test, screenCapture));
    }

but OnScreenCaptureAdded is empty for the ExtentHtmlReporter so does the community version support screenshots at all?

@Duke482132 The lines below:
Assert.That(File.Exists(aPath)); //it does
_extentScenario.AddScreenCaptureFromPath(aPath, "aTitle");

do not represent the path correctly in relation to the report.  Even though the image may be available relatively to your project, it may not be present relatively to the report.  I would recommend saving the image relative to the report instead of the project.

@anshooarora I'll do it however that's not the point, it's just second example of AddScreenshot type method that does nothing(they later invoke the same code). It seems that OnScreenCaptureAdded is empty for ExtentHtmlReporter it should at least thrown NotImplementedException. So the screenshots never worked for ExtentHtmlReporter, do I imiss something and inserting the image is implemented in different place?

I do not think you have gone through the code in its entirety as your statement is invalid.  Each reporter can implement the hooks differently, for example, KlovReporter uses this hook:
See here for example:  https://github.com/extent-framework/extentreports-csharp/blob/master/ExtentReports/Reporter/ExtentKlovReporter.cs#L300
But FileReporters such as Html do not.  They only care for aggregates as when the file is written, it requires the entire domain to be passed to it.  See here:  https://github.com/extent-framework/extentreports-csharp/blob/master/ExtentReports/Reporter/ExtentHtmlReporter.cs#L28
Regarding the issue, logically, a scenario on its own does nothing other than bind and implement steps.  So instead of attaching a screencapture to a scenario, try attaching it to a step (given, when, then, but etc.) and see if it shows up then>

I do not think you have gone through the code in its entirety as your statement is invalid. Each reporter can implement the hooks differently, for example, KlovReporter uses this hook:
See here for example: https://github.com/extent-framework/extentreports-csharp/blob/master/ExtentReports/Reporter/ExtentKlovReporter.cs#L300
But FileReporters such as Html do not. They only care for aggregates as when the file is written, it requires the entire domain to be passed to it. See here: https://github.com/extent-framework/extentreports-csharp/blob/master/ExtentReports/Reporter/ExtentHtmlReporter.cs#L28
Regarding the issue, logically, a scenario on its own does nothing other than bind and implement steps. So instead of attaching a screencapture to a scenario, try attaching it to a step (given, when, then, but etc.) and see if it shows up then?

I've seen KlovReporter code but I marked in the first post that I ask about ExtentHtmlReporter. See last line, again I changed code to add the screenshot to the step but nothing happens.... so to make it clear.
Does HTML reporter supports adding screenshots or not? (Yes/No) If yes where is the code for that cause it doesn't do the job? If not, then the proper methods shouldn't be empty but throw the not implemented exception or unsupported exception not just do nothing.

See here:
https://github.com/extent-framework/extentreports-csharp/blob/master/ExtentReports/Views/Html/Partials/SparkStepDetails.cshtml#L15
This line is invoked from:
https://github.com/extent-framework/extentreports-csharp/blob/master/ExtentReports/Views/Html/Partials/SparkBDD.cshtml#L42
Screen capture is used for the lowest level here so changing it to step should work. In your HTML, does the screen shot link not show up in devtools even though the image is not being displayed?

https://github.com/extent-framework/extentreports-csharp/blob/master/ExtentReports/Views/Html/Partials/SparkStepDetails.cshtml#L15
it iterates through Model.LogContext.All() I don't create logger anywhere or add anything to logs, I added a screenshot to an object of ExtentTest class which represents a step -> I have edited first post.
Screenshots are not in the DOM at all and I think they're lost in this method:
        private void CopyContextInfoToObservable(Test test)
    {
        //missing check for screenshot
        /*if (test.HasScreenCapture)
           do something
        }
        */
        if (test.HasAuthor)
        {
            test.AuthorContext.All().ToList()
                .ForEach(x => AuthorContext.AddAttributeContext((Author)x, test));
        }
        if (test.HasCategory)
        {
            test.CategoryContext.All().ToList()
                .ForEach(x => CategoryContext.AddAttributeContext((Category)x, test));
        }
        if (test.HasDevice)
        {
            test.DeviceContext.All().ToList()
                .ForEach(x => DeviceContext.AddAttributeContext((Device)x, test));
        }
        if (test.HasException)
        {
            test.ExceptionInfoContext.All().ToList()
                .ForEach(x => ExceptionInfoContext.AddAttributeContext((ExceptionInfo)x, test));
        }
        if (test.HasChildren)
        {
            test.NodeContext.All().ToList()
                .ForEach(x => CopyContextInfoToObservable(x));
        }
    }

Can you share a barebones snippet that reproduces this issue?  I will check today.

I have added support for screenshots. I am not sure if I am allowed to contribute to an external Open Source project. If yes, would implementation that produces such a result would be considered in a PR?


ERROR:Path to downloaded Udacity course/Nanodegree doesn't exist.

Nevermind i solved it

@Charul090
Hi, how u solve it, plz post a reply here so other user also follow ur steps
:-)


Fixed #26747 -- Used more specific assertions in the Django test suite.

https://code.djangoproject.com/ticket/26747

I'm not convinced about the merits of assertTrue/False since such assertions pass if bool(val) is True/False.

Makes sense.
Do you think these should be changed to assertIs() as recommended by the documentation, or left as is?

I did find the pattern:
self.assertEqual(bool(p.mugshot), False)

Asserts that follow this pattern have been modified to use assertFalse().

Yea, I guess assertIs is preferred.

Made discussed changes.

Made changes. All additional feedback welcome. Thanks.

assertIs here and below?

Fixed.


Cassandra log file missing on cassandra-mesos-2.1.2-1

Cassandra-mesos-2.1.2 build from master are working great, except the is no log file form Cassandra. Cassandra will emit the warning below.
SLF4J: Class path contains multiple SLF4J bindings.
SLF4J: Found binding in [jar:file:/tmp/mesos/slaves/20141216-201216-1694607552-5050-3917-S4/frameworks/20141216-215902-1694607552-5050-6890-0000/executors/cassandra.58031e09-460a-4df6-8ff2-fbcf7a1da6bd/runs/17479497-99fb-432a-a947-77702c8200e8/cassandra-mesos-2.1.2-1/lib/cassandra-mesos-2.1.2-1-deps.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: Found binding in [jar:file:/tmp/mesos/slaves/20141216-201216-1694607552-5050-3917-S4/frameworks/20141216-215902-1694607552-5050-6890-0000/executors/cassandra.58031e09-460a-4df6-8ff2-fbcf7a1da6bd/runs/17479497-99fb-432a-a947-77702c8200e8/cassandra-mesos-2.1.2-1/lib/logback-classic-1.1.2.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
SLF4J: Actual binding is of type [org.slf4j.impl.Log4jLoggerFactory]
log4j:WARN No appenders could be found for logger     (org.apache.cassandra.service.CassandraDaemon).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.

After deleting cassandra-mesos-2.1.2-1-deps.jar, cassandra binary works well and does emit log. However this will cause problem for cassandra-mesos.
Cassandra-mesos-2.0.5 does not have the same problem.
Thankyou
Wish everyone merry Christmas!

Obsolete framework has been rewritten.


Dad Jokes Generator

Project title
(Write project title over here)
Dad Jokes Generator
Project Description
(Describe the project over here)
I would like to create a Dad Jokes Generator using Html , Css and Javascript. In this , it will generate random dad jokes and everytime it generates different jokes .Please assign it to me.
Doc Domain

[] Solidity , Html, Css , JavaScript , Web3.js / Ether.js
[] Solidity , ReactJs /NextJs , Web3.js /Ether.js
[] Solidity , Python3 , Web3.py
[] Other , feel free to mention below

Are you contributing under any open-source program ?

@Tusharjoshi1703 closing this issue as we do not accept  front-end projects anymore from now on.


build error: Cannot open include file: 'CL/opencl.hpp': No such file or directory

Hi Stephane @StRigaud ,
I'm just trying to build CLIc as described here and retrieve this error:
11>------ Build started: Project: CLIc (clic\CLIc), Configuration: Release x64 ------
11>cleGPU.cpp
11>C:\structure\code\CLIc_prototype\clic\include\core\clic.hpp(29,10): fatal error C1083: Cannot open include file: 'CL/opencl.hpp': No such file or directory

Can you remind me, where I have to enter the path to this file? It's located here on my computer:
C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v11.2/include/

Thanks!
Best,
Robert

Wait! I can't locate the file CL/opencl.hpp in any place on my PC...

These configuration entries appear to be the problem. These folders don't exist when I clone the project... I think it's my inabilty to deal with git-submodules

From the root folder
git submodule update --init --recursive

Solved the problem.
Thanks for reading :-D


Add support for Github enterprise

TL;DR
Support scanning GHES for misconfigurations
Detailed design
No response
Additional information
No response

Glad I looked as I was going to open an issue requesting support for GitHub Enterprise.
This is what I get when I try against our IBM GHE:
 go run main.go analyze --org https://github.ibm.com/CICD-CPP-Ops --namespace repository
 ___      _______  _______  ___   _______  ___   _______  __   __
|   |    |       ||       ||   | |       ||   | |       ||  | |  |
|   |    |    ___||    ___||   | |_     _||   | |    ___||  |_|  |
|   |    |   |___ |   | __ |   |   |   |  |   | |   |___ |       |
|   |___ |    ___||   ||  ||   |   |   |  |   | |    ___||_     _|
|       ||   |___ |   |_| ||   |   |   |  |   | |   |      |   |
|_______||_______||_______||___|   |___|  |___| |___|      |___|
By Legit Security

Error: GitHub token seems invalid (should start with "ghp_"
2022/11/10 14:03:19 error executing command: GitHub token seems invalid (should start with "ghp_"
exit status 1

Thanks

Hey @carltonmason
What kind of GitHub token are you using?

A GHE personal access token.

@carltonmason is that a fine-grained token?
If this is a classic token that doesn't have the ghp_ prefix, then we can remove this check for GHES (which was just merged https://github.com/Legit-Labs/legitify/pull/40)

I am not sure if its a fine-grain token or how to tell if it is.  I have several GHE tokens and think the one I am using is:

@carltonmason looks like your token is a classic one.
I'm gonna open a separate issue to remove the prefix limitation for GHES


Need to add the warning below

This is a warning I would like to add to the Public document because the VM is not accessible after Generalization.
Warning to be added:
The machine will not be accessible after Capturing image.
If you generalize the virtual machine, no further actions on it are allowed. For example, a generalized virtual machine can't be started or modified. You can select this checkbox to have the virtual machine deleted automatically once the image is created. If you do not generalize the virtual machine, you can continue to use it as normal.

@kangupt : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@cynthn - If you accept these changes, a few edits will need to be made to the text before it's merged.
#label:"aq-pr-triaged"

You have to generalize the VM ahead of time (the portal doesn't generalize the VM or anything in this context) and generalizing is covered by a different topic, which has a warning about not restarting the machine etc. I don't want to imply that generalizing happens as part of "capture" in the portal. Maybe take a look at this article and see if you think the warning needs changes: https://docs.microsoft.com/en-us/azure/virtual-machines/generalize
#please-close

@cynthn,
I understand while capturing the VM is not being generalized but the VM all I want to add is the VM will be unusable after preforming the capture. Can you please check if we can add this

Hello Team,
Please reopen this ask as its important message missing from public documents

@cynthn do you want to take another look?


⚠ Samsung has degraded performance

In 876bf3f, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2451 ms

Resolved: Samsung performance has improved in ff7a0d7.


Failed To Init IOURing: iouring_setup: cannot allocate memory

OS: Linux (Ubuntu 22.04 LTS)
Kernel Version: 6.5.0-26-generic #26~22.04.1-Ubuntu
Go Version: 1.21
code segment:
IOURing, err = iouring.New(10000)
if IOURing == nil {

	return false, err
}

while executing the above statement, many a times I encounter the IOURing object as nil and the err as: iouring_setup: cannot allocate memory. Can you help me find the RCA for this error.

@Iceber Please, help me with this issue.


Boot takes ~200ms on pageload, can it be deferred?

You mention in another issue that you don't need boot to complete before using the other methods, but as far as I understand from your comment you should at least kick off the boot before calling the other methods:
https://github.com/johnnynotsolucky/vue-intercom/issues/2#issuecomment-342275826
It would be nice to remove that restriction. Or maybe have an additional option in boot so that it only starts up on demand.

Hi @d1manson, apologies for the delay getting back to you. It should be trivial to add that functionality, however, I'd prefer to keep the API as close to Intercom's as possible. If there was a bug with the Intercom script, for example, it would be a lot easier for users to trace the function calls.


Subscriber using grpc client fails on decode

When a Subscriber is constructed using grcp client
PubSubSubscriber
  .grpc[IO]
  .projectId(ProjectId("..."))
  .subscription(Subscription("..."))
  .defaultUri
  .httpClient(client)
  .defaultRetry
  .noErrorHandling
  .withDefaults
  .decodeTo[String]
  .subscribe
  .evalMap { message => IO.print(message.value) *> message.ack }
  .compile
  .drain

It's then going to fail on following error. Same constructor with http, is fine.
java.lang.IllegalArgumentException: Illegal base64 character 7b
        at java.base/java.util.Base64$Decoder.decode0(Base64.java:852)
        at java.base/java.util.Base64$Decoder.decode(Base64.java:570)
        at fs2.pubsub.grpc.GrpcConstructors$.fs2$pubsub$grpc$GrpcConstructors$Client$$anon$1$$_$$anonfun$2$$anonfun$2(GrpcConstructors.scala:143)
        at scala.Option.map(Option.scala:242)
        at fs2.pubsub.grpc.GrpcConstructors$Client$$anon$1.$anonfun$2(GrpcConstructors.scala:143)
        at scala.collection.immutable.Vector2.map(Vector.scala:2140)
        at scala.collection.immutable.Vector2.map(Vector.scala:443)
        at fs2.pubsub.grpc.GrpcConstructors$.fs2$pubsub$grpc$GrpcConstructors$Client$$anon$1$$_$read$$anonfun$1(GrpcConstructors.scala:155)
        at modify @ fs2.internal.Scope.close(Scope.scala:262)
        at flatMap @ fs2.Compiler$Target.flatMap(Compiler.scala:163)
        at uncancelable @ fs2.Compiler$Target.uncancelable(Compiler.scala:165)
        at run$ @ Main$.run(Subscribe.scala:10)
        at >>$extension @ org.typelevel.keypool.KeyPool$Builder.keepRunning$1(KeyPool.scala:370)
        at run$ @ Main$.run(Subscribe.scala:10)
        at update @ fs2.internal.Scope.releaseChildScope(Scope.scala:227)
        at >>$extension @ org.typelevel.keypool.KeyPool$Builder.keepRunning$1(KeyPool.scala:370)
        at run$ @ Main$.run(Subscribe.scala:10)
        at void @ org.typelevel.keypool.KeyPool$.reap(KeyPool.scala:187)
        at main$ @ Main$.main(Subscribe.scala:10)

I suspect that PubSubMessage response in rpc protocol isn't base64 encoded, as it's not stated in rpc api docs https://cloud.google.com/pubsub/docs/reference/rpc/google.pubsub.v1#google.pubsub.v1.PubsubMessage. Unlike REST response, which is base64 encoded https://cloud.google.com/pubsub/docs/reference/rest/v1/PubsubMessage
Following line probably shouldn't contain base64 decoding https://github.com/permutive-engineering/fs2-pubsub/blob/main/modules/fs2-pubsub/src/main/scala-2.13%2B/fs2/pubsub/grpc/GrpcConstructors.scala#L143

Hey @martintupy, great catch! It seems we didn't catch this on testing since both subscriber & publisher were using the base 64 encoding/decoding. This would have failed if data would have been published from another source.

Hey @martintupy, this has been released in v1.1.0. Thanks for raising it!


feat(plugin-https): patch https requests

Which problem is this PR solving?

closes #375

Short description of the changes

remove static for component property since it's used with https plugin.

@OlivierAlbertini Could you please resolve the conflicts and remove "private": true tag.