Proxy support for HTTP client

Effectively covers #182.
How to use:
(def pool (http/connection-pool {:connection-options {:proxy-options {:host "localhost"}}}))
(http/get "http://google.com" {:pool pool})

With more advanced settings:
(def pool (http/connection-pool
           {:connection-options {:proxy-options {:host "localhost"
                                                 :port 8080
                                                 :protocol :http
                                                 :tunnel? true
                                                 :user "username"
                                                 :password "123456"
                                                 :keep-alive? true
                                                 :http-headers {"X-Via" "Proxy!"}}}}))

(http/get "http://google.com" {:pool pool})

Right now covers next protocols:

HTTP proxy w/ tunneling (using CONNECT)
HTTP proxy w/o tunneling
Socks4
Socks5

Proxy configuration is set as an additional (optional) key in :connection-options when creating connections pool. Minimal configuration is :host, all other settings are optional. Proxy connection timeout is suppressed in preference to :connect-timeout handling. "Proxy-Connection" is set to "Keep-Alive" if otherwise is not specified explicitly.
The default protocol is HTTP w/o tunneling (same as curl). The client falls back to CONNECT if one the following true:

tunnel? is set to true explicitly
auth params are provided
additional http headers are provided
attempting to connect to HTTPS destination throw HTTP proxy

I'm still working on this one to add:

[ ] per-request configuration (to override connection pool settings) to provide friendlier API
[x] SSL support for proxy connection
[x] documentation

Placing this earlier to get feedback, thoughts and potential questions (as this field is pretty wide and might be a bit unpredictable in practice despite relatively good RFCs coverage).

@ztellman Updated PR with SSL handling.


Garage door opener - sliding gate opener

Hello everybody. First I’d like to congratulate for the very good job. I am amazed with the library.
I have a slide gate that open automatic with a remote control. I was wondering if I could use the Garage example to make it work with my hardware. (https://www.amazon.com/CO-Z-Automatic-Hardware-Driveway-Security/dp/B0197V3OD2?ref_=fsclp_pl_dp_3)
In its board, have 2 relays that when triggered open and close the gate and use 2 reed switches to manage when to stop.
My doubt is that in the code, I only see one relay and one reed switch. On the other hand, I saw YouTube video that it was able to make it work with a Sonoff.
Have you guys stumbled in a similar situation or know an alternative?
Thank you very much

Ok, I agree, current garage door opener is a bit weird. Looks like all it does is rely on existing door opener controller to do opening/closing and ESP just flicks relay that is connected to a button that controls stock door controller (which kind of surprising, it might have been just a pin, but who knows, maybe it's a 12V button). Anyways, I agree, it would be nice to have an another controller, that controls up/down commands individually.

I can design the controller, but can not code :(

This is the circuit I design for using one relay to control two switches with state sensor. I original design for homebridge plugin homebridge-gpio-garage-door. It also prevent bounce when relay switch its state. Using 2 or 4-channel relay module. I test it on bread broad with no issue but I'm not sure it can work perfect with real door.


Small quality of life improvements

When console=ttyS0,1152800n8 is passed in to the kernel it does not directly translate to a /dev entry. Simply strip the ,value off of the console=x,y parameter and set that as our /dev entry.
When a pool import fails for some reason, log the text from the zpool import process at the debug log level.

This'll need to be rebased once #176 is merged.


Get react-native-mapbox-gl working

Hey I saw you added react-native-mapbox-gl, awesome! I put together a quick demo but I'm getting the following error:

Do I need to do anything special to enable it? The only way I've been to reproduce this locally is by removing react-native-mapbox-gl from the node_modules folder.

Thanks for the report! I'm not sure why this is happening actually. The packager has the module installed,  but it's throwing this error: Unable to resolve module react-native-mapbox-gl. I'll look into it some more.

This is also happening with react-timer-mixin. Still not sure what's up, will keep looking! I've tested this module locally without issue.

Interesting....

@jsierles any findings?

Not yet - been swamped with other work lately. if you have any time to check out the iOS app for possible misuse of the plugin, it's appreciated! Otherwise next week I will have time to loop back to this. Sorry for the delay.

Sorry for the accidental closure. Been tracking this down but still no conclusion. It appears even with a full js bundle, this error occurs when running from the app. Going to continue investigating.

This is related to bugs in the 0.8 packager dependency resolver update. 0.10 has a bunch of fixes, so going to start upgrading everything to use that. However, mapbox-gl crashes on 0.10:
2015-08-22 20:43:06.156 Playground[27741:1264492] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'customDirectEventTypes must return an array, but RCTMapboxGLManager returned __NSDictionaryI'

Interesting, I'll take a look this weekend.

related
https://github.com/facebook/react-native/issues/2347

I fixed the issue with the packager! There's a working sample here:
https://rnplay.org/apps/IFLnWQ
However, the map tiles are missing. Perhaps I messed up when including the mapbox assets?

Perhaps some more details: after applying some fixes for 0.10, I get this error with your demo:
 <Error>: CGContextDrawImage: invalid context 0x0. This is a serious error. This application, or a library it uses, is using an invalid context  and is thereby contributing to an overall degradation of system stability and reliability. This notice is a courtesy: please fix this problem. It will become a fatal error in an upcoming update.
2015-08-23 02:41:41.592 [info][tid:com.facebook.React.JavaScript] { zoom: 0, longitude: 0, latitude: 0 }
libc++abi.dylib: terminating with uncaught exception of type mbgl::util::SpriteImageException: Sprite image dimensions may not be zero

I'll post my PR shortly.

Regarding However, the map tiles are missing. Perhaps I messed up when including the mapbox assets?
Looking at the project, it seems like you added and are properly linking to MapboxGL.bundle but the bundle was never checked in in git.

I can put together a PR and add the bundle.

OK - FYI I see the bundle was installed in the build I uploaded to Appetize.

Yeah, it's a tile loading issue, otherwise the background color would not load if the styles were not there. Is there anyway we can see if there are any errors in the console?

The missing tiles was on 0.8 - and the new error on 0.10. Which should we look at first?

The new error is still happening when I load your example with the 0.10 release. I'd like to push to the app store today. I'll still include it even if we can't solve this, but it would be good to know what's up.

@jsierles: having the same issue "SpriteImageException: Sprite image dimensions may not be zero"

I'll take a look again at this soon. At the moment I don't have time, but any help is appreciated!

Hey could you try updating to<EMAIL_ADDRESS>(just released) and see if the problem still persist?

Sure thing - will be reviewing all the modules after we get android support out on the site. Unfortunately many are not working properly in the latest build! But we can push them out faster now.

This problem has gone away with the latest release - but I still am not seeing map tiles. Can you check the code here? https://rnplay.org/apps/YmkqXg

It looks like your loading a non exists at style. In the update, all styles have been updated so Mapbox-streets-v7 is no longer valid. You will need to add the new Mapbox.bundle and also try streets-v8 instead.

I added the latest Mapbox.bundle, and tried styleURL={'asset://styles/mapbox-streets-v8.json'}
But still not seeing the map. Is that asset path correct?

Little confusing, dropped the Mapbox prefix on the style. It's just streets-v8. You can also view all the styles that ship with the sdk by opening up the Mapbox.bundle.

It works! Looks like we're good - the next iOS release will include the latest Mapbox, and it's available today on the site. I've also added it to the homepage. If you'd like to setup a more complete example or want this one assigned to your account, let me know!

Awesome!

The app was approved yesteday, and Mapbox finally works! I added your example app to the homepage.

:100: !!!


[CU-8688g2vxx] Create required collections when fetching system collections

Task linked: CU-8688g2vxx Users without a bookmark collection get an error when going to their favorited models


Customize default request body when executing function

When clicking "Execute Function Now", the request body is set to { "name": "Azure" } by default.

Is it possible to customize this default value?

No that is not possible today. Any thoughts on how you would prefer to customize it? Also what language are you using? For example

We could add a VS Code setting, but that might get complex
We could read the contents of "sample.dat" (depending on your language, that file exists by default in the function's folder) or some other file
We could remember the last "request body" entered and just use that

I'm using a Powershell azure function. I think it makes sense to read from the sample.dat file because that file starts out as
{
    "name": "Azure"
}

Since this matches the default request body, my initial thought was that this file was being used to set it.
However, if not all languages start out with a sample.dat file, maybe it's better to just remember the last request body entered.
Alternatively, a VS Code setting could be added to switch between reading from the sample.dat or just using the last request body.

This issue has become stale and is at risk of being closed. The community has 60 days to upvote the issue. If it receives 5 upvotes we will keep it open and take another look. If not, we will close it. To learn more about how we handle issues, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this issue has received less than 5 community upvotes and we closed it. Still a big Thank You to you for taking the time to create it! To learn more about how we handle issues, please see our documentation.
Happy Coding!


refactor(ast, transformer): remove StringLiteral::new method

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#6788  👈
#6787
#6786
#6785
#6784
#6783
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite


Form Submit button styling broken

When viewing the form on mobile, the styling is broken.
Browsers tested:

Safari
Chrome
Firefox
Brave

Seems as though input elements with button classes applied don't get their appearance reset, the submit button needs -webkit-appearance: none to remove the odd looking iOS Safari styles

Would I use something like this?
input[type=submit] {
  -webkit-appearance: none;
}

Or use a class?
.button {
  -webkit-appearance: none;
}

I guess I could just try both, lol.

It’s the input one, but there’s no harm applying it to both. I would’ve made a pull request but I don’t know where to put these extra styles

It's in _includes/head.html! Maybe I should break that out into its own file now... lol

Probably should create a assets/css/theme-overrides.scss file

Hm, seems to be still broken here. Completely clueless here on how to fix it...
:(

Yay! I fixed it with an inline style tag in the head... wtf tho man. lol


Updated DINE FW Diag policy

Overview/Summary
Updated Azure Firewall deploy diagnostic settings policy to be parameterized and to check the destination type of existing diag setting profiles, so it will flag non-compliant diag profiles/settings if they do not have the specified destination type.
Changed defaultValue to resource-specific instead of legacy logs:
https://learn.microsoft.com/en-us/azure/firewall/logs-and-metrics#structured-logs
This PR fixes/adds/changes

Changed defaultValue from AzureDiagnostics to Dedicated
Parameterize existenceCondition
Added property logAnalyticsDestinationType to existenceCondition
Update template deployment apiVersion
Use "categoryGroup": "allLogs" instead of adding all categories one by one
Added output for easier troubleshoot and/or transparency in the deployment

Breaking Changes
Testing Evidence
Please provide any testing evidence to show that your Pull Request works/fixes as described and planned (include screenshots, if appropriate).

Testing URLs
The below URLs can be updated where the placeholders are, look for {YOUR GITHUB BRANCH NAME HERE - Remove Curly Brackets Also} & {YOUR GITHUB BRANCH NAME HERE - Remove Curly Brackets Also}, to allow you to test your portal deployment experience.

Please also replace the curly brackets on the placeholders {}

Azure Public

Azure US Gov (Fairfax)
[](https://portal.azure.us/#blade/Microsoft_Azure_CreateUIDef/CustomDeploymentBlade/uri/https%3A%2F%2Fraw.githubusercontent.com%2F{YOUR GITHUB ORG/ACCOUNT HERE - Remove Curly Brackets Also}%2FEnterprise-Scale%2F{YOUR GITHUB BRANCH NAME HERE - Remove Curly Brackets Also}%2FeslzArm%2FeslzArm.json/uiFormDefinitionUri/https%3A%2F%2Fraw.githubusercontent.com%2F{YOUR GITHUB ORG/ACCOUNT HERE - Remove Curly Brackets Also}%2FEnterprise-Scale%2F{YOUR GITHUB BRANCH NAME HERE - Remove Curly Brackets Also}%2FeslzArm%2Ffairfaxeslz-portal.json)
As part of this Pull Request I have

[X] Checked for duplicate Pull Requests
[X] Associated it with relevant issues, for tracking and closure.
[X] Ensured my code/branch is up-to-date with the latest changes in the main branch
[X] Performed testing and provided evidence.
[X] Ensured contribution guidance is followed.
[ ] Updated relevant and associated documentation.
[ ] Updated the "What's New?" wiki page (located: /docs/wiki/whats-new.md)

Thanks for the PR here @vegazbabz
However, we are very close to making a big change to ALZ and we will be moving from the custom diagnostic settings policies to the built-in initiative for diagnostic settings for resources: https://www.azadvertizer.net/azpolicyinitiativesadvertizer/f5b29bc4-feca-4cc6-a58a-772dd5e290a5.html which is about to get a lot more resource coverage also, as we've all been working together behind the scenes as this is the right approach for all customers going forward.
We went into this on our recent external community call here: https://youtu.be/E3Pz_VcefZ4?si=to88-8W9Oy8DrxU1&t=1576
So at this time I wouldn't want to waste your time and efforts on this PR as we will soon be replacing it as stated above 👍

Also we cannot accept this PR as it is anyway as there is a git submodule added in it?

Thanks for the PR here @vegazbabz
However, we are very close to making a big change to ALZ and we will be moving from the custom diagnostic settings policies to the built-in initiative for diagnostic settings for resources: https://www.azadvertizer.net/azpolicyinitiativesadvertizer/f5b29bc4-feca-4cc6-a58a-772dd5e290a5.html which is about to get a lot more resource coverage also, as we've all been working together behind the scenes as this is the right approach for all customers going forward.

Also we cannot accept this PR as it is anyway as there is a git submodule added in it?

Alright that sounds good; I completely missed the December call 👍.
I have already asked some of your colleagues to fix the built-in policies for the resource types that has support for resource-specific logs, so this was just an alternative while we wait for that.
Appreciate that we are moving more and more to a built-in setup.


Exam live statistics: Chrome issue (Error:  attribute d: Expected number)

Describe the bug
The Exam Live Statistics system causes errors when using the Chrome browser (these errors do not occur with Firefox).

Possible reasons:

Decimals with multiple decimal places (precision)
Updating data during re-rendering
Malformed data: https://stackoverflow.com/questions/62245818/error-path-attribute-d-expected-number-521739130434781-nanl60-739130434
...

To Reproduce

Go to Exam Live Statistics Page with the Chrome browser
Participate with multiple students
Open the console
See errors

Expected behavior
Similar to other browsers, the exam live statistics system should not produce any issues or errors in Chrome.
Screenshots
No response
What browsers are you seeing the problem on?
Chrome
Additional context
No response
Relevant log output
No response

Finding no. 22 in exam mode testing.


Password encryption in registry/config.yml for proxy?

Hello, I am using proxy for remote mirror, so I have to config proxy username and password in common/templates/registry/config.yml. However, the password is in plain code. Is there any way to have encrypted password rather than plain password in the configuration files for security?
E.g.
proxy:
remoteurl:  [https://remoteurl]
username: [username]
password: [encrypted password rather than plain password]
I have read https://github.com/vmware/harbor/issues/1329 and https://github.com/vmware/harbor/issues/1329 but still not sure how it supports.
In short, Is there any way to store the password in encrypted format? Or I missed any documents?
Thank you!
If you are reporting a problem, please make sure the following information are provided:
1)Version of docker engine and docker-compose.
no need.
2)Config files of harbor, you can get them by packaging "harbor.cfg" and files in the same directory, including subdirectory.
no need.
3)Log files, you can get them by package the /var/log/harbor/ .
no need.

Strictly speaking as the docker registry's code read plain text from the config.yml I don't think there's a chance to avoid leaking if the docker host is compromised.

Closing as won't fix.


[8.x] Delay pushing jobs to queue until database transactions are committed

DB::transaction(function(){
    $user = User::create(...);

    SendWelcomeEmail::dispatch($user);
});

When running the code above, the SendWelcomeEmail job may get dispatched and picked by a worker before the transaction is committed. This will lead to errors since the user model won't exist when the job runs.
This PR is an attempt to capture queue dispatches, store them in a local cache, and only perform the dispatch when all transactions has been committed. Given the example above, the SendWelcomeEmail won't get dispatched to the queue until the transaction is committed.
To enable this behaviour, you need to set the after_commit configuration value to true in the connection settings inside the queue.php config file. You can also use the afterCommit() method when dispatching the job:
DB::transaction(function(){
    $user = User::create(...);

    SendWelcomeEmail::dispatch($user)->afterCommit();
});

Or if the default is to delay dispatches, you can override that using the beforeCommit() method:
DB::transaction(function(){
    $user = User::create(...);

    SendWelcomeEmail::dispatch($user)->beforeCommit();
});

In the case of rollbacks, the jobs will get discarded.
You can add a dispatchAfterCommit public property to mailables, notifications, listeners, and broadcastable events to achieve the same behaviour.
This PR is an alternative to https://github.com/laravel/framework/pull/35266

I'm very glad to see this as built-in functionality, thank you. One question: will the "Queueable" be dispatched in this case?

@LastDragon-ru no it won't get dispatched. That commit is translated by laravel to a save point.

no it won't get dispatched.

Great.
Meanwhile, seems it doesn't work with Queue:fake()
    public function testJobDispatchedAfterTransactionCommitFakeQueue() {
        Queue::fake();

        DB::transaction(function () {
            QueueWithinTransactionJob::dispatch();

            Queue::assertNothingPushed();
            // Jobs were pushed unexpectedly.
            // Failed asserting that an array is empty.
        });

        Queue::assertPushed(QueueWithinTransactionJob::class);
    }

Full TestCase
<?php

namespace Illuminate\Tests\Integration\Queue;

use Exception;
use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Foundation\Bus\Dispatchable;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Facades\Queue;
use Orchestra\Testbench\TestCase;

/**
 * @group integration
 */
class QueueWithinTransactionTest extends TestCase {
    protected function getEnvironmentSetUp($app) {
        $app['config']->set('app.debug', 'true');
        $app['config']->set('database.default', 'mysql');
        $app['config']->set('queue.default', 'redis');
        $app['config']->set('queue.connections.redis.after_commit', true);
    }

    public function testJobDispatchedAfterTransactionCommit() {
        $size = Queue::size();

        DB::transaction(function () use ($size) {
            QueueWithinTransactionJob::dispatch();

            $this->assertEquals($size, Queue::size());
        });

        $this->assertEquals($size + 1, Queue::size());
    }

    public function testJobNotDispatchedAfterRootTransactionRollback() {
        $size = Queue::size();

        try {
            DB::transaction(function () use ($size) {
                DB::transaction(function () use ($size) {
                    QueueWithinTransactionJob::dispatch();

                    $this->assertEquals($size, Queue::size());
                });

                $this->assertEquals($size, Queue::size());

                throw new RollbackException('rollback');
            });
        } catch (RollbackException $exception) {
            // empty
        }

        $this->assertEquals($size, Queue::size());
    }

    public function testJobDispatchedAfterTransactionCommitFakeQueue() {
        Queue::fake();

        DB::transaction(function () {
            QueueWithinTransactionJob::dispatch();

            Queue::assertNothingPushed();
            // Jobs were pushed unexpectedly.
            // Failed asserting that an array is empty.
        });

        Queue::assertPushed(QueueWithinTransactionJob::class);
    }

    public function testJobNotDispatchedAfterRootTransactionRollbackFakeQueue() {
        Queue::fake();

        try {
            DB::transaction(function () {
                QueueWithinTransactionJob::dispatch();

                throw new RollbackException('rollback');
            });
        } catch (RollbackException $exception) {
            // empty
        }

        Queue::assertNothingPushed();
        // Jobs were pushed unexpectedly.
        // Failed asserting that an array is empty.
    }
}

class QueueWithinTransactionJob implements ShouldQueue {
    use Dispatchable, Queueable;

    public function handle() {
    }
}

class RollbackException extends Exception {

}

Yes it doesn’t have to work when faking the queue, because in your test you are testing the framework not your code.

In real app Queue::fake(); also doesn't work :(
<?php declare(strict_types = 1);

namespace Tests\Feature;

use Illuminate\Support\Facades\Queue;
use Tests\TestCase;

class ExampleTest extends TestCase {
    public function testQueueReal() {
        $size = Queue::size();

        $this->get('/test')->assertStatus(503);

        $this->assertEquals($size, Queue::size()); // Passed
    }

    public function testQueueFake() {
        Queue::fake();

        $this->get('/test')->assertStatus(503);

        Queue::assertNothingPushed();
        // Jobs were pushed unexpectedly.
        // Failed asserting that an array is empty.
    }
}

// routes/web.php
Route::get('/test', function () {
    DB::beginTransaction();

    // Doing something ...

    QueueWithinTransactionJob::dispatch();

    // Uops something went wrong...
    DB::rollBack();

    abort(503);
});

// QueueWithinTransactionJob.php
class QueueWithinTransactionJob implements ShouldQueue {
    use Dispatchable, Queueable;

    public $dispatchAfterCommit = true;

    public function handle() {
    }
}

@LastDragon-ru  I'm not sure exactly what your test is supposed to be doing but feel free to open a pull request after/if this one is merged.
Let's keep the discussion focused on the added functionality for now.

I'm not sure exactly what your test is supposed to be doing

The test trying to test the added functionality with Sync::fake(), and as I see it is not possible now :(

@themsaid will this work for batched jobs? I just ran into a bug using SQS and batches.
DB::transaction(function () use ($import, $iobs) {
    $batch = Bus::batch($jobs)
        ->name('Process Import ' . $import->id)
        ->dispatch();

    $import = $batch->id;
    $import->save();
});

If we want to persiste the batch_id on a related entity within a transaction, SQS jobs start to get processed before the main transaction is committed. Each job atomically moves a couple of columns on job_batches that keep track of the progress.
Each SQS job means an HTTP call (they can be sent in groups of 10 using SendMessageBatch but that's for another PR).
I think the DatabaseBatchRepository::updateAtomicValues won't fail if the batch does not exists. Final scenario is that you get an almost completed job_batch because event if all the jobs ran successfully, some couldn't persist their success/failure anywhere.

Needs rebasing on latest 8.x. ;)

Thanks for adding this, it isn't listed in the docs however? :)

Would the community be interested in a solution that simply gets rid of transactions in the RefreshDatabase trait?
These transactions seem to be creating all kinds of issues. Symfony has the same problem. CakePHP has a solution to refresh the test database that is fast, but without dropping the database at the start of the test suite, thus does not require to run the migrations every time, and cleans up the database based on SQL triggers, without transactions.
I posted this in a discussion https://github.com/laravel/framework/discussions/36019#discussioncomment-313627 here, but I am not sure on how to get people of Laravel aware, that cleaning the DB with transactions is not ideal, and that there are other solutions for that.
Here is the repo to the solution for CakePHP https://github.com/vierge-noire/cakephp-test-suite-light
I worked on an adapter for Laravel, which worked like a charm.
I am aware that this PR is not the ideal place to discuss that. I already created a discussion here for that: https://github.com/laravel/framework/discussions/36019#discussioncomment-313627

Would the community be interested in a solution that simply gets rid of transactions in the RefreshDatabase trait?
These transactions seem to be creating all kinds of issues. Symfony has the same problem. CakePHP has a solution to refresh the test database that is fast, but without dropping the database at the start of the test suite, thus does not require to run the migrations every time, and cleans up the database based on SQL triggers, without transactions.
I posted this in a discussion https://github.com/laravel/framework/discussions/36019#discussioncomment-313627 here, but I am not sure on how to get people of Laravel aware, that cleaning the DB with transactions is not ideal, and that there are other solutions for that.
Here is the repo to the solution for CakePHP https://github.com/vierge-noire/cakephp-test-suite-light
I worked on an adapter for Laravel, which worked like a charm.
I am aware that this PR is not the ideal place to discuss that. I already created a discussion here for that: https://github.com/laravel/framework/discussions/36019#discussioncomment-313627


Support Dart 3

Description
https://medium.com/dartlang/the-road-to-dart-3-afdd580fbefa
I believe we just need to increase the versioning in the pubspec file, but it has to be tested.

Likely a 7.0.0 milestone.
Let's watch https://github.com/dart-lang/sdk/issues/49530 before going ahead.

Dart v3 is already part of the master channel so I am going to make it part of the v7 release.


⚠ Bofrost.be has degraded performance

In 981cd0c, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1153 ms

Resolved: Bofrost.be performance has improved in 9fca291 after 31 minutes.


crc start fails saying LIBVIRT_3.4.0

General information

OS: Linux / macOS / Windows  - Linux
Hypervisor: KVM / Hyper-V / VirtualBox / hyperkit   KVM
Did you run crc setup before starting it (Yes/No)?  Yes

CRC version
# Put the output of `crc version`

version: 1.0.0-beta.3-4.1.11+e97bf65
Host Operating System
cat /etc/redhat-release
Red Hat Enterprise Linux Server release 7.3 (Maipo)
Steps to reproduce
crc start fails with error saying
FATA exit status 1 : /root/.crc/bin/crc-driver-libvirt: /lib64/libvirt.so.0: version `LIBVIRT_3.4.0' not found (required by /root/.crc/bin/crc-driver-libvirt)
Is libvirt-3.4 a requirement ?
I have libvirt installed but the version is not 3.4.
The doc. needs an update saying libvirt needs to be 3.4 :
https://code-ready.github.io/crc/#required-software-packages_gsg
Logs
crc start fails as below:
FATA exit status 1 : /root/.crc/bin/crc-driver-libvirt: /lib64/libvirt.so.0: version `LIBVIRT_3.4.0' not found (required by /root/.crc/bin/crc-driver-libvirt)
You can start crc with crc start --log-level debug to collect logs.
Please consider posting this on http://gist.github.com/ and post the link in the issue.

General information
* OS: Linux / macOS / Windows  - Linux

* Hypervisor: KVM / Hyper-V / VirtualBox / hyperkit   KVM

* Did you run `crc setup` before starting it (Yes/No)?  Yes

CRC version
# Put the output of `crc version`

version: 1.0.0-beta.3-4.1.11+e97bf65
Host Operating System
cat /etc/redhat-release
Red Hat Enterprise Linux Server release 7.3 (Maipo)

We indeed expect a more recent RHEL release to be used  (7.5 or newer)

The doc. needs an update saying libvirt needs to be 3.4 :
https://code-ready.github.io/crc/#required-software-packages_gsg

Yes, we could have a preflight check about this as well.

We need to add a pre-flight check for this.

This was fixed.


unable to fix problems

Describe the bug
While trying to create a simple tasks mgt app, I've spent 6m tokens and hours of my time trying to get rid of this fetch issue. I am really disappointed by the value provided after hours spent and most of my credit consumed without satisfaction on simple tasks. Kindly let me know how to fix this

Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-leklqz
Steps to reproduce
just run the draft app as is
Expected behavior
you will get errors notification
Screen Recording / Screenshot

Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

Appreciate the feedback! At times there are programming challenges that are too complex for the AI to solve. Sometimes pasting any error messages from the terminal into the chat is successful in resolving issues. At other times, manual intervention is required to prevent excessive token usage. This page here has a number of helpful insights on using Bolt effectively while maximizing token efficiency. Appreciate your patience as the team continues to make improvements!


how to run YCSB simultanausly in two nodes of cassandra cluster

I plan to use two nodes for co-ordinator job in a Cassandra cluster So, that i need to run YCSB on both the nodes. Did anyone try this setup before. please share how you did and also the steps you follow

closing as stale in favor of #255


dart2js: isolates in minified mode

Users have reported that messages sent between isolates appear to break if both isolates are compiled in minified mode.
We are still working on getting a repro to clarify the issue. We believe the message only contains the serializable types, but I'm not certain at this time that we are validating this in the isolate library (It may be that we are not complaining if Dart objects are sent on the message).

update: it appears their issue was that they were using --trust-type-annotations --trust-primitives. We should test and make sure isolates don't break when using those flags.

Thanks for digging in!!


Move Dockerfile to test/ dir

Because the Dockerfile is for testing and is not a Dockerfile to
build an image for aptly_cli.
Someday you might want to create a Dockerfile for aptly_cli itself,
so nice to move the test Dockerfile out of the way to make room.

Good call! Makes sense to me. Thank you!


Add yandex prebuilt image to dockerhub

Any chance what Yandex browser will appear among prebuilt images in Dockerhub? #140
Can I help with automation of that process?

@zarom I would wait for official webdriver binary from Yandex. According to my knowledge it planned to be published during next months.

Sad news that we need to wait.
Good news what Yandex finally making their webdriver. Thank you.

@zarom It exists for a long time. The problem is that it was never published to the outside.


Stopping zookeeper

I want to know, how to explicitly stop the zookeeper. In your code I do not find any way of stopping the zookeeper.

@MohammedShoaib  Sorry for noticing late. Do you need any help around stopping zookeeper ?


Support for Typescript 2.1

There were a few breaking changes in TS 2.1 in typings (NodeFlags split into NodeFlags and ModifierFlags, ScriptTarget.ES6 renamed to ScriptTarget.ES2016, some changes in JSX definitions)
Plus, the compiler still has bug (?) regarding callback referencing this inside and passed to super()
This PR, hopefully, fixes all of them

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

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

Signed CLA

Great work, thank you very much.
This is available on our "releases" branch now. You can point your npm dependency there if you need the latest files.

@HamletDRC Glad to help. After this changes, tslint-microsoft-contrib requires TS2.1:
...
### Fast Compile >>./src/utils/implicitRoles/tr.ts
### Fast Compile >>./src/utils/implicitRoles/ul.ts
### Fast Compile >>./src/validTypeofRule.ts
Using tsc v2.0.10
src/utils/AstUtils.ts(110,22): error TS2339: Property 'getCombinedModifierFlags' does not exist on type 'typeof ts'.
src/utils/AstUtils.ts(110,58): error TS2339: Property 'ModifierFlags' does not exist on type 'typeof ts'.
...

So I'm not sure about 2.0.14 milestone :/


Can you generate a new release?

It is better than use the gem with the git url 😄

Done! 0.5.0 is on rubygems now.


Updated Documentation / Help Center

Update documentation to support widespread adoption and allow support tickets for issues.

Estimated at $2000 (90% confidence)


Add MJPEG server example using h/w MJPEG encoder

Otherwise identical to mjpeg_server.py.
Signed-off-by: David Plowman<EMAIL_ADDRESS>

Note: flake8 bugbear has released its 23.1.14 version that includes the new B028 error.

@CrenoEno Thanks for the info. Perhaps I'll just go and fix those up first!


chore(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2831

Questions ?
Please refer to the Backport tool documentation


Clarify relabelings doc string

cc @squat Let me know if you want to clarify this further, happy to make any further changes that might be useful. Thanks!

Seems like the job failure is unrelated to this PR, but I can't retrigger that.

we need to bump minikube, we've recently seen this in kube-state-metrics as well https://github.com/kubernetes/kube-state-metrics/pull/846

lgtm


placement of "[Exported]"

MichaelHatherly: (Only nitpick would be the placement of "[Exported]" and "[Internal]" labels (they're controlled by the :md_subheader setting right?), but that's really a stylistic issue and won't hold up merging of this.)  https://github.com/MichaelHatherly/Lexicon.jl/pull/89#issuecomment-96392918

(they're controlled by the :md_subheader setting right?)

I thought for bigger projects it might be better to group by category headers and not simple by 'Exported' / 'Internal'.

Do you prefer to have all (exported and internal) below one catecory header and  "[Exported]" and "[Internal]" next to the catecory removed?

I thought for bigger projects it might be better to group by category headers

Yeah, that sounds better.

Do you prefer to have all (exported and internal) below one catecory header and "[Exported]" and "[Internal]" next to the catecory removed?

I've not got a preference at the moment. Once these have been around for a bit I should have a better idea.

md_subheader           (default: :simple): Valid options are ":skip, :simple, :category"

md_subheader=:simple   adds documentation and API-Index subheaders "Exported" / "Internal".
md_subheader=:category adds documentation and API-Index subheaders per category.
md_subheader=:skip     adds no subheaders to the documentation and API-Index and can be used
for documentation which has only few entries.

added a new md_subheader option split_category

md_subheader=:simple   adds documentation and API-Index subheaders "Exported" / "Internal".

md_subheader=:category adds documentation and API-Index subheaders per category.

md_subheader=:split_category adds documentation and API-Index subheaders per category. Each categort is split into: category ["Exported"] and category ["Internal"]

md_subheader=:skip     adds no subheaders to the documentation and API-Index and can be used for documentation which has only few entries.

To be discussed: md_subheader=:split_category would it be better to put the 'Exported' in front.
Exported Methods
Exported Types
Internal Methods
Internal Types
instead of current
Methods [Exported]
Types [Exported]
Methods [Internal]
Types [Internal]
Any preference? Not sure if it worthy to add for this a new Config field

I guess this should be changed: just use simple types without anything in fron or after and insert the "External" - "Internal" subheaders. Probably one wants  to allow separate styling in such case: e.g. "External" - "Internal" H3  and the Category subheader something like H4

Exported
Methods
Types
Internal
Methods
Types

I will do that.

:+1: to that last example of yours @peter1000. Much clearer looking.

I nearly finished this : but than started cleaning up the Config and had the idea with the signature -- Well today I start looking into Docile - not easy as an outsider ;)
But I think I'm getting around and a bit of your help I should be able to finishe most of the planned things in the next couple of days. I will send tomorrow morning a Docile PR for the adding of the signature to entry.data

not easy as an outsider ;)

Especially with a mutually recursive call across different files :) (the loopdocs code, don't worry too much about it though, it's off by default and I haven't found much use for it actually.)

a bit of your help

Feel free to ask about anything regarding Docile, I should be around more from now through the weekend.

I think this could be closed with the new document(...)


Support for invert logic

A node can be configured to invert boolean logic:
Input true leads to an output false.

Available with v1.3.1


Issue with missing "Windows Remote Management (Beta)" in @site/docs/reference/integrations/

Describe the problem
In market place, but missing in https://xsoar.pan.dev/docs/reference/integrations/
Screenshots
n/a
Environment
n/a
Suggested fix
add page - describing how to use 'scriptname' is how we ran into this issue of missing documentation.

Fixed at: https://github.com/demisto/content/pull/12131


[Feature Request] Namespace overview stats

Hi 👋
I think it would be super handy to display some stats for the selected namespace in the Web UI for a quick overview of:

total running workflows
total failed / timed out workflows
longest running workflows
workflow throughput

all split by workflow names if possible

For visibility, this will be taken care of in post-v1, will keep you updated! Thanks for opening the ticket

still important to us, just low pri, appreciate the feature request


Could not load /__uno.css when vite build with watch.

When the vite build option turns on watch, the second build after modifying the file will prompt the following error, and the build will not continue until you modify the file imported into uno again:
ERROR  [vite:load-fallback] Could not load /__uno_components.css (imported by src/main.ts): ENOENT: no such file or directory, open '/__uno_components.css'
or
ERROR  [vite:load-fallback] Could not load /__uno.css (imported by src/main.ts): ENOENT: no such file or directory, open '/__uno.css'                                             09:12:59

#244


A sample Redline collection in a Win8 VM.

Could you move the sample into the specimens subdirectory (for consistency) and maybe add a bit of context, e.g. text file to indicate how the sample was created. Such as version of Redline used, settings enabled/disabled.

I've addressed your comments Jaachim, please take a look.


Add Stats Graph SVG Render View

Adds an endpoint to render SVG files for the website to display, in the style of the current nycmesh.net/stats page
Also does a bit of refactoring and cleanup in meshweb, we should probably spread the views out into multiple files, and we don't need to use the DRF decorators for non-API views

The new graphs aren't identical, but they're really close. The biggest thing that's missing is the little green % diff indicator. I don't think matplotlib can really draw that, but if anyone has tips I would appreciate them

Okay changed it so that we just render the graph in MeshDB. All the header stuff will be done on the website side, but I needed an API in order to expose the total counts publicly (using our standard API for this is a security hazard as the token would need to be public). Having this little JSON API also has the advantage of making it trivial to convert to a frontend graphing framework in the future if desired


Lock icons are not affected by CanvasLayer

Tested on Godot 2.1 and 2.1.1, Windows 10, 64 bits
Repro:

Create a CanvasLayer and set its scale to (3, 3)
Create a Sprite as child of CanvasLayer, with a texture on it
Move the sprite off the origin, then lock it
Observe the lock icon is off, as if it was not affected by the scale.

I expected the lock icon to be shown on the sprite as usual, but its size and rotation should not be affected (just like the rectangular handles of the sprite).
Example project:
LocksAndCanvasLayers.zip

First of all thank you for your report and sorry for the delay.
We released Godot 3.0 in January 2018 after 18 months of work, fixing many old issues either directly, or by obsoleting/replacing the features they were referring to.
We still have hundreds of issues whose relevance/reproducibility needs to be checked against the current stable version, and that's where you can help us.
Could you check if the issue that you described initially is still relevant/reproducible in Godot 3.0 or any newer version, and comment about it here?
For bug reports, please also make sure that the issue contains detailed steps to reproduce the bug and, if possible, a zipped project that can be used to reproduce it right away. This greatly speeds up debugging and bugfixing tasks for our contributors.
Our Bugsquad will review this issue more in-depth in 15 days, and potentially close it if its relevance could not be confirmed.
Thanks in advance.
Note: This message is being copy-pasted to many "stale" issues (90+ days without activity). It might happen that it is not meaningful for this specific issue or appears oblivious of the issue's context, if so please comment to notify the Bugsquad about it.

Still happens in Godot 2.1.5 rc1.
Happens too in Godot 3.0.2.
I also noticed that even when not locked, moving sprites in a scaled CanvasLayer makes them move faster than intented.

Tested in master. The lock now behaves as expected.

Thanks, closing as fixed.


The bg.jpg background image is too big

The file size of the bg.jpg background image is 349 KB which is way too much. For the index page we have a total available payload of 28 KB (including HTML and CSS). So the picture shouldn't be bigger than 20-22 KB (depending on how good the rest can be compressed with gzip).
Too possible solutions:

We find a smaller picture (does the image has to be 1400 pixel wide?)
We change to design of the page so that we don't need a picture.

There is one 3rd but worst case solution: We use an other TCP roundtrip for that image. Which would have a potential payload of 57 KB. But that would increase the loading time of the index page by 50%.

I'll take care of that later. Please use this picture for now (is has 19 KB):


matching default arguments.

Matching default arguments.
Description
I conformed to coding style.(http://libcloud.readthedocs.io/en/latest/development.html#don-t-abuse-kwargs)
Adding, OpenStackDriverMixin Class match default arguments with OpenStackBaseConnection Class.
Status

done, ready for review

Checklist (tick everything that applies)

[x] Code linting (required, can be done after the PR checks)
[ ] Documentation
[ ] Tests
[ ] ICLA (required for bigger changes)

Coverage remained the same at 88.237% when pulling f9615879317362b31dea7f39241f86bb75f1aaee on utam0k:conform_coding_style into a6d11ab7511788c27ce8ba712baf2531f87b15dc on apache:trunk.

this is much cleaner, that openstack driver could really use a spring-clean

thanks for the contribution 👍


[a11y]: Select usage and other guidance updates to support accessibility

This issue captures the high-level considerations from a review of the Usage, Style, storybook and figma documentation on the Select component. More details can be discovered by examing in the comments in the pdf exports of the Usage and Style pages.
It is complementary to the https://github.com/carbon-design-system/carbon/issues/13008 issue covering actual implementations with the Select component in the React storybook.

Various suggested rewrites for clarity and accuracy, commented on in the Usage pdf (linked to above)
Inconsistencies between images and captions
Some of the decisions on illustrations between this and the prior version of the Usage page are not improvements.
Calling the select list a "menu" in the anatomy section is ill-advised
Material in the Content section of the Usage has some challenges.
"Selected" is missing as a state in the Behaviors section
There are several inaccuracies in the Style content.

closing in favor of tracking via https://github.com/carbon-design-system/carbon/issues/13521


Question on using with Media Foundation

Can this be used in an exiting application written for Media Foundation?
Specifically the decoders / encoders?

Yes it is very possible to integrate this with an existing application that is using MediaFoundation. The FFmpegInterop object creates a MediaStreamSource which implements the IMFMediaSource interface and can be connected to the MediaFoundation graph. This will allow to leverage the container format processors from FFmpeg as well as the decoders if you want. At this point, FFmpegInterop is a playback component and therefore would not be usable to get to the encoders, but it is certainly something that can be done in the future.


Display out of stock badge for products on order creation page

Issue: https://github.com/RailsEventStore/ecommerce/issues/376
Client View:

Admin View:

Two things here:

In the client orders read model we need a boolean flag, we dont need integer here, let's adjust
In the admin/sales orders read model, let's keep it as integer, but lets create a new ticket where we display the number - this is a good opportunity to diverge these two read models from each other


Please sync against the lastest version of Flint

"69 commits behind facebookarchive:master."
thanks

We cannot do that, sorry. From the point we branched at we rewrote almost all of the code base. The only part of FPP that is still in line with facebook is the the token parsing. And even the interface to that has changed.


Add initial .travis.yml

Using docker with a more recent version of Ubuntu, so we do not need to
maintain a long list of tasks to setup the environment needed to build
Camera Streaming Daemon.
Next step: create a custom image with the dependencies installed, so we
will be able to remove the time-consuming apt-get step.

The first and third commits can be squashed.
I did some experiments with tests running in more than one environment - Arch Linux besides the Ubuntu committed here. However, I am wondering whether it's useful or not.

+1

I did some experiments with tests running in more than one environment - Arch Linux besides the Ubuntu committed here. However, I am wondering whether it's useful or not.

It's useful to support more systems.  I'd like to support semaphoreci as well.. can you use another docker to run e.g. fedora on semaphoreci?

As discussed this morning with @otaviobp , we'll move to the images approach. This will be updated when we get ACK to publish them.

It's useful to support more systems. I'd like to support semaphoreci as well.. can you use another docker to run e.g. fedora on semaphoreci?

@lucasdemarchi I guess so, long time I don't check semaphoreci. Is there any specific feature of semaphoreci to justify maintaining it besides travis?

See #28

Is there any specific feature of semaphoreci to justify maintaining it besides travis?

Last time I checked it was way faster than Travis. It could be not the case anymore. It's nice to have another CI since it's not uncommon to get failed builds due to network shortage on Travis or unexplainable failures.

Last time I checked it was way faster than Travis. It could be not the case anymore. It's nice to have another CI since it's not uncommon to get failed builds due to network shortage on Travis or unexplainable failures.

Got it. I am not that familiar with Travis, I didn't know about such issues. We can work to have semaphoreci tests running as well.


查看日志提示错误

error from daemon in stream: Error grabbing logs: rpc error: code = Unknown desc = warning: incomplete log stream. some logs could not be retrieved for the following reasons: rpc error: code = Internal desc = grpc: received message length<PHONE_NUMBER> exceeding the max size 4194304

类似这样的都是 Docker daemon 直接返回的错误，你用 docker 的命令行工具查看日志应该也是一样的。


Rename iree-hip- compiler flags to iree-rocm- when they apply to codegen.

The various ROCM codegen flags like iree-hip-target and iree-hip-enable-ukernels are not HIP-specific and need to be renamed back to ROCM. The AMDGPU target uses the same ROCM codegen and we shouldn't have any hip naming in the shared code paths.

Can we alias the flags? Changing the flag names once caused some disruption.

My vote would be for llvm-amdgpu or maybe just amdgpu (but that might be unclear w.r.t. SPIRV) so that it's clear that we're using LLVM's amdgpu backend.

llvm-amdgpu sgtm


Simplify interop CLI for objects

Expose get (list and object)
Upload single object for easier manual error handling

The purpose of the CLI is to make it possible for teams to compete if they don't have the ability to develop a direct API integration. It is not intended to make competing optimal for said teams. Decided this is not worth dev/maintenance cost.


Remove of old styling

This PR removed all old styling the only source of truth is styled-components right now.
Size of package before removing old styling:
File sizes after gzip:

  658.87 KB  build/static/js/index.js
  642.04 KB  build/static/js/middleware.js
  352 B      build/static/css/middleware.css
  348 B      build/static/css/index.css

After removing old styles:
File sizes after gzip:

  582.74 KB  build/static/js/index.js
  569.56 KB  build/static/js/middleware.js
  352 B      build/static/css/middleware.css
  348 B      build/static/css/index.css

@Huvik Thanks a lot for finishing my half-written effort 🙈😅Sorry about leaving it like that. You're awesome :D


global: release checklist

Closes #75.

Labeled as Need: information since we will wait to have all components CHANGES.rst so we know what to add in this one.

Release commit removed, so release is done separately in https://github.com/reanahub/reana/issues/83. PR ready to be re-reviewed and eventually merged.


Maze of cables

PR Description 📜
I have added Maze of Cables game where the player has to connect the cable to the socket keeping in mind the decreasing length of the wire and ticking time to clear each level.
Fixes #1832

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @VSatwika ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Hello @kunjgit
I have already added ReadMe.md file in the game folder.

@kunjgit

Thank you  @VSatwika , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

Hey @VSatwika,
You can't work on a closed issue 👀!
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗


[Quick PR][LocalNet] Turn off minimum-gas-prices

Summary
minimum-gas-prices was increased to 0.000000001upokt on LocalNet. We are not ready for that yet. #794 will address the default fees values, so all existing documentation and processes will continue working with non-zero gas prices.

Supersedes #804 per this discussion:


debug_trace_transaction with Erigon

Version
2.0.0
Platform
Linux sc-devbox-01 5.13.0-52-generic #59~20.04.1-Ubuntu SMP Thu Jun 16 21:21:28 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Description
Erigon StructLog has error as an Object, not a string.
I would suggest substituting a Value for string:
https://github.com/gakonst/ethers-rs/blob/master/ethers-core/src/types/trace/geth.rs#L37
s/String/Value/
pub struct StructLog {
    pub depth: u64,
    #[serde(default, skip_serializing_if = "Option::is_none")]
    pub error: Option<String>,
   ...

Test data:
      {
        "pc": 131,
        "op": "SSTORE",
        "gas": 18441,
        "gasCost": 20000,
        "depth": 1,
        "error": {},
        "stack": [
          "0xa2e62045",
          "0x85",
          "0x1",
          "0x0"
        ],
        "memory": [
          "0000000000000000000000000000000000000000000000000000000000000000",
          "0000000000000000000000000000000000000000000000000000000000000000",
          "0000000000000000000000000000000000000000000000000000000000000060"
        ],
        "storage": {
          "0000000000000000000000000000000000000000000000000000000000000000": "0000000000000000000000000000000000000000000000000000000000000001"
        }
      }

hmm this should serialize as string, right?
https://github.com/ethereum/go-ethereum/blob/ee530c0d5aa70d2c00ab5691a89ab431b73f8165/eth/tracers/logger/logger.go#L77-L77
mind checking the erigon code if that differs and if {} is just null?
because the error in structlog is supposed to be a string:
https://geth.ethereum.org/docs/developers/evm-tracing/built-in-tracers

I have no doubt that it is an Erigon bug. I'll take a look at the source code.
The example I gave contained an out of gas error (following a divergence reproducing the
DAO fork).


Create my own project

Tell me, can someone help me building my own project by using this repo?

Hi @SteevyTheGeek. Stackoverflow would be a better place for this request. Ask your question there with additional details to help narrow down any issues you're having.


Individual page creation: field translation button is confusing

If you add multi languages, such as French, a translation button appears after each text field. The button label is the same as the question for the text field and at the end it says (French). There are tow problems with this:

The label is not very clear and doesn't tell user what will happen if they select it
At 500% zoom, user don't get to see the (French) right in the beginning of the question and becomes confusing why they see the same question again.

Replace the label with something clearer such as: Enter [French] translation for your response here

"French translation of 'Field name'"

Further discussion to use "Add [language] translation" - pending on Alan's feedback on whether this labelling provides enough context to a screenreader

I have updated all the fields that require translation for the question to be short enough for us to use "French translation for ..." That being said, some of them required a bit more copy tweaking and we couldn't just use the field name.
Here's the proposed copy:
Individual
Your bio: [Language] translation of your bio
Skills and strengths: [Language] translation of your skills and strengths
Lived experience: [Language] translation of your lived experience
Community Organization
About your organization:

[Language] translation of about your organization

Other areas of interest:

[Language] translation of your organization’s other areas of interest

FRO
About your organization:

[Language] translation of about your organization

Project
Please indicate the goals for this project:

[Language] translation of project goals

Please identify which communities this project will impact:

[Language] translation of the communities this project will impact

Please indicate how the identified communities will be impacted:

[Language] translation of how the identified communities will be impacted

Please provide an update about the work that has been completed since the previous project:

[Language] translation of project update

Please indicate what is “out of scope” for this project.

[Language] translation for what is out of scope

Who is analyzing the results?

[Language] translation of who is analyzing the results

Please indicate the tangible outcomes of this project.

[Language] translation of tangible outcomes of this project

Approximate response time

[Language] translation of approximate response time

Engagement
What is the name of your engagement?

[Language] translation of engagement name

Please describe this engagement.

[Language] translation of engagement description

Title of meeting

[Language] translation of meeting title

Interview - Accepted format - other

[Language] translation of other accepted format

Also, if the "[language] translation] part could be in bold, that would really help for sighted users to scan.

@michelled made some updates to the project scope questions based on copy related issues from accessibility testing. Please see the previous comment where I've labeled "EDIT" for changes.


Login by another attribute

Hey!
First of all - really great package, it save me a lot of time and work, thank you!
I have one last problem now - I managed to configure my application so I can login through my OpenLDAP server. The users are authenticated by common name (cn=common_name;dc=example;dc=com) but I want them to be able to enter they username (uid) in login form in my application instead of full name.
I need some solution to get uid from login form but send common name (related to that uid) to server, Is this even possible? Can I do it some way by the config or simple method instead of changing original package files?

Hi @Pawlox,
This is definitely possible, all through your config/adldap_auth.php file. You need to configure the username_attribute and login_attribute options. Here's an example (not tested):
    /*
    |--------------------------------------------------------------------------
    | Username Attribute
    |--------------------------------------------------------------------------
    |
    | The username attribute is an array of the html input name and the LDAP
    | attribute to discover the user by. The reason for this is to hide
    | the attribute that you're using to login users.
    |
    | For example, if your HTML input name is `email` and you'd like users
    | to login by their LDAP `mail` attribute, then keep the
    | configuration below. However, if you'd like to login users
    | by their usernames, then change `mail` to `samaccountname`.
    | and `email` to `username`.
    |
    | This must be an array with a key - value pair.
    |
    */

    'username_attribute' => ['username' => 'uid'],

    /*
    |--------------------------------------------------------------------------
    | Login Attribute
    |--------------------------------------------------------------------------
    |
    | The login attribute is the name of the active directory user property
    | that you use to log users in. For example, if your company uses
    | email, then insert `mail`.
    |
    | This option must be a string.
    |
    */

    'login_attribute' => env('ADLDAP_LOGIN_ATTRIBUTE', 'cn'),

Thank you very much for quick answear! It works, I don't know how I could miss this configuration set. Thank you again and sorry for waisting your time with such silly question.

Oh no don't worry about it! No question is silly. Thanks for using Adldap2 :).

Hello,
i cannot find "config/adldap_auth.php".
i am just have this file "config/adldap.php"

Hi @ibenettaleb, you need to make sure you insert the AdldapAuthServiceProvider in your config/app.php and then run the php artisan vendor:publish command.


Malformed XML

It seems that ./source/docs/01-introduction.xml is malformed in the ./release/v5.0 branch, causing ant to fail.
Buildfile: /Users/laurent/libs/mei_official/build.xml

init:
     [echo] xerces available: true
     [echo] saxon available: true
     [echo] prince available: false
     [echo] initialized

init-mei-classpath:
     [echo] mei.classpath set

canonicalize-source:
     [echo] canonicalizing source...
     [java] Error on line 305 column 21 of 01-introduction.xml:
     [java]   SXXP0003   Error reported by XML parser: The element type "p" must be terminated by the
     [java]   matching end-tag "
".: The element type "p" must be terminated by the matching end-tag "
".
     [java] Error on line 113 column 79 of mei-source.xml:
     [java]   SXXP0003   Error reported by XML parser: Error attempting to parse XML file
     [java]   (href='docs/01-introduction.xml').: Error attempting to parse XML file
     [java]   (href='docs/01-introduction.xml').
     [java] org.xml.sax.SAXParseException; systemId: file:/Users/laurent/libs/mei_official/source/mei-source.xml; lineNumber: 113; columnNumber: 79; Error attempting to parse XML file (href='docs/01-introduction.xml').

BUILD FAILED
/Users/laurent/libs/mei_official/build.xml:204: Java returned: 2

That was fixed before in 43acdb2bccc9a8748be2adeac6fafefd5405406a by @musicEnfanthen and seems to be broken again with #1311.

Just testing build of my local fix PR almost there ;-)

first!

did you build it locally?

yes.


The inference process is stuck.

When I use the pre-trained model on NYUD-v2, the inference process is stuck.  Here is my command
CUDA_VISIBLE_DEVICES=0 python -m torch.distributed.launch \
        --nproc_per_node=1  \
        --master_port=$((RANDOM%1000+12000)) \
         main.py \
         --config_exp './configs/nyud/nyud_vitLp16_taskprompter.yml' \
         --run_mode infer  \
         --trained_model  TaskPrompter_nyud_vitLp16.pth.tar

The log is as follows:
Number of dataset images: 654
Use checkpoint TaskPrompter_nyud_vitLp16.pth.tar
Infer at iteration 0
  0%|

Hi, thank you for your interests. I have not heard about this problem. Can you locate at which line does it stop? It seems that the pre-trained model is not successfullly loaded because it says "infer at iteration 0."
Hanrong
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Jiangpengtao @.>
Sent: Wednesday, May 24, 2023 7:03:15 PM
To: prismformore/Multi-Task-Transformer @.>
Cc: Subscribed @.***>
Subject: [prismformore/Multi-Task-Transformer] The inference process is stuck. (Issue #20)
When I use the pre-trained model on NYUD-v2, the inference process is stuck. Here is my command
CUDA_VISIBLE_DEVICES=0 python -m torch.distributed.launch
--nproc_per_node=1
--master_port=$((RANDOM%1000+12000))
main.py
--config_exp './configs/nyud/nyud_vitLp16_taskprompter.yml'
--run_mode infer
--trained_model  TaskPrompter_nyud_vitLp16.pth.tar
The log is as follows:
Number of dataset images: 654
Use checkpoint TaskPrompter_nyud_vitLp16.pth.tar
Infer at iteration 0
0%|
—
Reply to this email directly, view it on GitHubhttps://github.com/prismformore/Multi-Task-Transformer/issues/20, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADLPY6STEWMG3S4SVPHEYVTXH24WHANCNFSM6AAAAAAYOFLO7U.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Hi, at L448 in taskprompter.py, cur_task_fea = self.fea_decode_spa[il][task](cur_task_fea)

Hello, hanrong,
when I use the inference.py file, the inference process is running successfully.  Here is my running command.
CUDA_VISIBLE_DEVICES=0 python inference.py \
    --config_path=./configs/nyud/nyud_vitLp16_taskprompter.yml \
    --image_path=./dataset/NYUDv2/images/0001.png \
    --ckp_path=./TaskPrompter_nyud_vitLp16.pth.tar \
    --save_dir=../TaskPrompter_nyud_vitLp16/results/

Hi, at L448 in taskprompter.py, cur_task_fea = self.fea_decode_spa[il][task](cur_task_fea)

That is quite interesting. Does it result from the ddp?

Hello, I guess the problem occurs when I use DDP with one GPU.

@PengtaoJiang This should not be a problem as I always use it in this way. If you use the recommended environment does this still happen?

@prismformore,
hello, hanrong, I successfully run the evaluation code.  I guess it is my MACHINE problem.


[libhv] update to 1.2.6

Describe the pull request

What does your PR fix?
Fixes #25930, update libhv to 1.2.6.
Feature test pass on x64-windows.

Thanks for the update :)


publish

looks like the published version is a few months old on npm,

Yes, I have published latest version. Thanks.


Calypso: We have strong opinions.

Don't take any options, even when used programatically.

Nice!


Unable to handle notifications on iOS when the app is on screen

When I send a notification through the Firebase console to test the notifications on iOS it prints the following in the console:

Here's my handling of the notification service which is working fine on android.
import 'dart:math' as math;

import 'package:awesome_notifications/awesome_notifications.dart';
import 'package:firebase_auth/firebase_auth.dart';
import 'package:firebase_messaging/firebase_messaging.dart';
import 'package:flutter/material.dart';
import 'package:page_transition/page_transition.dart';
import 'package:persistent_bottom_nav_bar/persistent-tab-view.dart';
import 'package:spike_app/bloc/auth_bloc.dart';
import 'package:spike_app/bloc/incident_bloc.dart';
import 'package:spike_app/bloc/user_bloc.dart';
import 'package:spike_app/helpers/user_helper.dart';
import 'package:spike_app/main.dart';
import 'package:spike_app/screens/home_screen.dart';
import 'package:spike_app/screens/incident_screen.dart';

import 'app_color_helper.dart';
import 'app_utils.dart';

class NotificationHelper {
  static IncidentBloc bloc = IncidentBloc();
  static UserBloc userBloc = UserBloc();
  static Future init(BuildContext context) async {
    AwesomeNotifications().initialize("resource://drawable/launcher_icon", [
      NotificationChannel(
          importance: NotificationImportance.High,
          playSound: true,
          channelKey: 'basic_channel',
          channelName: 'Basic notifications',
          channelDescription: 'Notification channel for basic tests',
          defaultColor: AppColorHelper.blue,
          ledColor: AppColorHelper.white)
    ]);

    AwesomeNotifications().isNotificationAllowed().then((isAllowed) {
      if (!isAllowed) {
        AwesomeNotifications().requestPermissionToSendNotifications();
      }
    });

    AwesomeNotifications().actionStream.listen((ReceivedAction action) async {
      if (action.buttonKeyPressed == 'Acknowledge') {
        print("Acknowledge");
        await bloc.ackSingleIncident(context,
            counterId: int.parse(action.payload?['counterId'] ?? "0"));
      } else if (action.buttonKeyPressed == 'Resolve') {
        print("Resolve");
        await bloc.resSingleIncident(context,
            counterId: int.parse(action.payload?['counterId'] ?? "0"));
      }

      if ((action.payload?['type'] == "incident" ||
              action.payload?['type'] == "incidentAssign") &&
          action.payload?['counterId'] != null) {
        try {
          pushNewScreen(
            navigatorKey.currentContext!,
            screen: IncidentScreen(
              counterId: int.parse(action.payload?['counterId'] ?? "0"),
              index: 0,
            ),
            pageTransitionAnimation: PageTransitionAnimation.fade,
            withNavBar: false,
          );
        } catch (e) {
          print(e.toString());
        }
      } else if (action.payload?['type'] == "oncall") {
        Navigator.push(
          navigatorKey.currentContext!,
          PageTransition(
              child: HomeScreen(
                initialIndex: 2,
              ),
              type: PageTransitionType.fade),
        );
      } else if (action.payload?['type'] == "orgRemove") {
        AuthBloc().signOut(navigatorKey.currentContext!);
      }
    });

    initializeFirebaseService(context);
    AppUtils.log("Notification", "Successfully initialized");
  }

  static Future<void> initializeFirebaseService(BuildContext context) async {
    FirebaseMessaging messaging = FirebaseMessaging.instance;
    messaging.getToken().then((value) {
      AppUtils.log("FCM token", value);

      UserHelper.fcmToken = value!;
      User? user = FirebaseAuth.instance.currentUser;
      userBloc.setUserFCM(context, fcm: value, email: user?.email ?? "");
    });
    messaging.subscribeToTopic("general");
    AppUtils.log("FCM", "Subscribed to general");

    FirebaseMessaging.onMessage.listen((RemoteMessage event) {
      print("message received");

      buildNotification(event);
    });
  }

  static void buildNotification(RemoteMessage event) {
    if (event.data['type'] == "incident") {
      AwesomeNotifications().createNotification(
        actionButtons: [
          NotificationActionButton(
            key: 'Acknowledge',
            label: 'Acknowledge',
          ),
          NotificationActionButton(
            key: 'Resolve',
            label: 'Resolve',
          ),
        ],
        content: NotificationContent(
          payload: {
            "type": event.data['type'],
            "counterId": event.data['counterId'] ?? ""
          },
          id: math.Random().nextInt(10000),
          channelKey: 'basic_channel',
          title: event.data['title'],
          body: event.data['body'],
        ),
      );
    } else {
      AwesomeNotifications().createNotification(
        content: NotificationContent(
          payload: {
            "type": event.data['type'],
            "counterId": event.data['counterId'] ?? ""
          },
          id: math.Random().nextInt(10000),
          channelKey: 'basic_channel',
          title: event.data['title'],
          body: event.data['body'],
        ),
      );
    }
  }

  static Future dispose() async {
    try {
      AwesomeNotifications().actionSink.close();
      AwesomeNotifications().createdSink.close();
      await AwesomeNotifications().dispose();
    } catch (e) {
      print(e.toString());
    }
  }
}

Please help

I have the same problem here. Anyone has a solution ?

Have you guys used another plugin and made some changes in the iOS app source code?

Yeah i use firebase_messaging (10.0.9) for a configuration of push notification.
Even if i don't use the lib awesome_notifications (just let it in pubspec.yaml), i have this problem.
"Was going to present a notification, but no plugin wanted ti handle it."

I have the same issue on iOS. Works all good on Android

Any update on this? After removing code from AppDelegate (from local notifications setup) still does show : "was going..."

I have the same problem here. Anyone has a solution ?

Please, if you did a modification on your iOS swift / objective-C source code, please remove it. These changes can create a loop in the receiving process because its handle twice or even more depending of what changes you did.

I am also facing same issue. do you find any solution

I am Facing the same Issue In my app as well. I am migrating from flutter local notification to this package. so I removed these block of code from appDelegate,
 if #available(iOS 10.0, *) {
   UNUserNotificationCenter.current().delegate = self as? UNUserNotificationCenterDelegate
   }

Now background message is working fine But foreground says "Was going to present a notification, but no plugin wanted to handle." Could you tell me how To fix it
This is my AppDelegate.swift file.
import UIKit
import Flutter
import flutter_downloader
@UIApplicationMain
@objc class AppDelegate: FlutterAppDelegate {
    override func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?) -> Bool {
        GeneratedPluginRegistrant.register(with: self)
        FlutterDownloaderPlugin.setPluginRegistrantCallback(registerPlugins)
        // if #available(iOS 10.0, *) {
        //     UNUserNotificationCenter.current().delegate = self as? UNUserNotificationCenterDelegate
        // }
        return super.application(application, didFinishLaunchingWithOptions: launchOptions)
    }
}
private func registerPlugins(registry: FlutterPluginRegistry) {
    if (!registry.hasPlugin("FlutterDownloaderPlugin")) {
        FlutterDownloaderPlugin.register(with: registry.registrar(forPlugin: "FlutterDownloaderPlugin")!)
    }
}

Any update on this?
When I used only the firebase messaging library, I am getting notifications and callbacks in code but in the same project if I add awesome notifications then firebase messaging stops working.
The default firebase notification appears from the notification payload when the app is running in the background but in case of foreground messages not getting callbacks in onMessage of FirebaseMessaging.
Whenever I send the message from console it prints
was going to present a notification, but no plugin wanted to handle it
I haven't added any native code. Just added firebase_messaging and awesome_notifications libraries in my project.
Flutter version: 2.5.3
Firebase messaging: 11.2.0
Awesome notifications: 0.6.18+2

Hi @rafaelsetragni , Happy holidays, wanted to say that the Issue still remains, once awesome_notifications is removed 0.6.19 onMessage works correctly.

Facing this issue as well just as described by others.
Solution would indeed be great if possible?

Hi, I am sending data in notification like shown below and receiving it properly when app is in background.  But when app goes to foreground I am not able to receive any notification. Actually main issue is that none of the listener gets triggered of FirebaseMessaging.
Above issue happens only in IOS devices while in Android everything works fine using this library.
firebase_messaging: 11.1.0
awesome_notifications: 0.0.6+12
   data = {
        'registration_ids': tokens,
        "notification": {
         body: "",
          title: "",
        },
        'apns': {
          'headers': {'apns-priority': "5"},
          'payload': {
            'aps': {
               body: "",
               title: "",
              'channel': '1000',
              'isUpdateNotification': 'update'
            }
          }
        }
      };

Any update on this, please?
When I used only the firebase messaging library, I am getting notifications and callbacks in code but in the same project if I add awesome notifications then firebase messaging stops working.
The default firebase notification appears from the notification payload when the app is running in the background but in case of foreground messages not getting callbacks in onMessage of FirebaseMessaging. Whenever I send the message from console it prints was going to present a notification, but no plugin wanted to handle it
I haven't added any native code. Just added firebase_messaging and awesome_notifications libraries in my project.
Flutter version: 2.5.3 Firebase messaging: 11.2.0 Awesome notifications: 0.6.18+2

Update:
{
  "message": {
    "token": "...",
    "notification": {
      "body": "notification message body",
      "title": "notification message title"
    },
    "data": {
        // custom data
        // here you can add your title and body
    },
    "apns": {
      "headers": {
        "apns-push-type": "background",
        "apns-priority": "5"
      },
      "payload": {
        "aps": {
          "content-available": 1
        }
      }
    }
  }
}

With this payload, everything is working fine as expected.
For testing the above payload through postman, refer to this link -> https://stackoverflow.com/questions/50399170/what-bearer-token-should-i-be-using-for-firebase-cloud-messaging-testing
@rafaelsetragni Thanks for creating such an amazing package

Hi! Sorry for taking so long to respond to your issue on GitHub. I was focused on releasing the new 0.7.0 version of awesome_notifications and awesome_notifications_fcm and i didn't had time enough to do both. But now I can answer all your questions.
So now i'm asking you to recreate this topic using the new issue template. There's a lot of missing informations that i need to understand your problem, and all the instructions are already in the new issue template. Also, remember to check beforehand if your issue was posted by another user.
So I will automatically close all previous issues so far.
Sorry for the inconvenience and i will be waiting for your new issue request.
Thank you so much for your support!


[WIP][SQL] Read/write ANSI intervals from/to the parquet datasource

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

By running the existing test suites:

$ build/sbt "test:testOnly *ParquetFileFormatV2Suite"
$ build/sbt "sql/test:testOnly *JsonV2Suite"

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47985/

Test build #143474 has finished for PR 34057 at commit 0f1a0fb.

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

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

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47985/

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

Test build #143483 has started for PR 34057 at commit d3b5714.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47994/

Test build #143483 has finished for PR 34057 at commit d3b5714.

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

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

Test build #143485 has started for PR 34057 at commit c0f45d0.

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47994/

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47996/

Test build #143485 has finished for PR 34057 at commit c0f45d0.

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

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47996/

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

Test build #143491 has started for PR 34057 at commit abf79a2.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48002/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48002/

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

Test build #143491 has finished for PR 34057 at commit abf79a2.

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

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

Test build #143501 has started for PR 34057 at commit 801d65c.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48012/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48012/

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

Test build #143507 has started for PR 34057 at commit 62f26d9.

Test build #143501 has finished for PR 34057 at commit 801d65c.

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

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48017/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48017/

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

@sarutak @beliefer @AngersZhuuuu @Peng-Lei @cloud-fan Could you review this PR, please.

Test build #143507 has finished for PR 34057 at commit 62f26d9.

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

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

@mswit-databricks @ala @sadikovi Could you look at this PR, please.

Test build #143566 has started for PR 34057 at commit 32f4eb9.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48073/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48073/

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

Test build #143566 has finished for PR 34057 at commit 32f4eb9.

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

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

Merging to master. Thank you, @sarutak @sunchao and @cloud-fan for review.


NPM Test - No Tests Found

Is this a bug report?
Yes
Did you try recovering your dependencies?
Yes
Which terms did you search for in User Guide?
testing, test configuration, jest, jest configuration, overriding pattern
Environment
System:
OS: macOS 10.14.3
CPU: x64 Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
Binaries:
Node: 10.15.1 - /usr/local/bin/node
npm: 6.8.0 - /usr/local/bin/npm
Browsers:
Chrome: 72.0.3626.121
Safari: 12.0.3
npmPackages:
react: ^16.8.4 => 16.8.4
react-dom: ^16.8.4 => 16.8.4
react-scripts: ^2.1.8 => 2.1.8
npmGlobalPackages:
create-react-app: 2.1.8
Steps to Reproduce

(Write your steps here:)

mkdir node_modules
cd node_modules
create-react-app test
cd test
npm test
(optional) enter "a" after test runner is up.

Expected Behavior
The test in App.test.js should have ran.
Actual Behavior
Step 5 output: No tests found related to files changed since last commit.
Step 6 output:
No tests found
No files found in /Users/svencowart/dev/node_modules/test.
Make sure Jest's configuration does not exclude this directory.
To set up Jest, make sure a package.json file exists.
Jest Documentation: facebook.github.io/jest/docs/configuration.html
Pattern:  - 0 matches

PS:  I totally understand that it cannot find any tests because the project is within the node_modules folder; however, this is something beyond my control.  It seems though if I run npm test within a project regardless of what parent directory is, it should find those tests but ignore the node_modules that are child directories of the project.

This is probably a limitation with Jest: https://github.com/facebook/jest/issues/1995
Regardless, even this can be worked around, I am just not seeing how this feature would be useful to anyone else.  I recommend tweaking your setup, or tweaking the way how your project runs tests.

I was able to resolve this issue by using react-app-rewired and configuring jest inside of package.json.
For anyone with similar issues, here is what I did:
Configuring jest within package.json
"jest": {
  "haste": {
    "providesModuleNodeModules": [
      "@itential/app-json_forms/client"
    ]
  }
}

The contents of my config-overrides.js file:
module.exports = {
  webpack: (config, env) => config,
  jest: config => {
    config.testPathIgnorePatterns = [];
    config.transformIgnorePatterns[0] = `<rootDir>${
      config.transformIgnorePatterns[0]
    }`;
    return config;
  },
  devServer: configFn => configFn,
  paths: (paths, env) => paths,
};

@bugzpodder It seems the real issue is that react-scripts does not support changing the jest configuration for testPathIgnorePatterns and transformIgnorePatterns so I need to rely on react-app-rewired for such changes.  However, this could be very intentional so I'll close this issue since my original ticket is resolved.


Deploy flow to Azure managed online endpoint

Is your feature request related to a problem? Please describe.
Is it possible to use sdk/cli to deploy flow to Azure managed online endpoint? This would be very useful for CI/CD process
Describe the solution you'd like
If possible would be great to have an example in the docs (similar to azure app service deployment example)

@wangchao1230  to follow up as a documentation improvement.

Ack the ask, we have some code improvement plan to make this path work smoothly. It will take some time, we will share the code and samples when that's ready.

Is it possible to deploy Prompt Flow to Azure Functions? Since heavy lifting is done by the GPT endpoints, Azure functions should be enough to handle lightweight callers.

I understand the implementation of this feature will take some time as mentioned by @eniac871 and also that it is possible to manually deploy a flow to an online endpoint.
Eagerly waiting for this feature to implemented and documented in promptflow since I need to automate the deployment in a pipeline.
In the meantime, can anyone indicate if the below path should work?

Build the docker image as referenced in the promptflow deployment guide
Deploy the flow by following along the Azure ML guidelines for using a custom container to deploy a model to an online endpoint

Would be glad about any high-level guidance of how to automate the deployment workflow - if this is not possible or not recommended for some reason at the moment, this would also be a very valuable information.
Thank you!

the process is now documented here:
https://learn.microsoft.com/en-us/azure/machine-learning/prompt-flow/how-to-end-to-end-llmops-with-prompt-flow?view=azureml-api-2


ignore incomplete conditional

Overview

An incomplete Simple Conditional in the form definition such as:
{
  "conditional": {
    "show" : "false"
    // missing "when" and "eq" properties for a Simple Conditional
  }
}

will now be ignored instead of erroring.
Checklist

[ ] tests/ added or updated
[ ] CHANGELOG.md's Unreleased section is updated
[ ] Documentation is updated

Coverage: 92.362% (-0.06%) from 92.422% when pulling 695ce30cd82057e90ab0e11fda09b029350f92e4 on SOAP-355-ignore-incomplete-conditional into c244a9899b4962bb79e3cfc3aa82122029577f25 on develop.


Mat Focus Indicator breaks mat-icon with font awesome

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
14
Description
The following mat-focus-indicator  class is applied to mat-icon in chips. This class sets ::before to display: none. This breaks font libraries such as fontawesome which use the ::before psuedoselector. As it causes the icon to be hidden. Additionally, in other places, this same class appears to remove the content attribute, which is how the icons are displayed.

Reproduction
Steps to reproduce.
Create a mat-icon using font awesome, make it a button for example
<mat-chip-grid #chipGrid aria-label="Fruit selection">
  <mat-chip-row *ngFor="let fruit of fruits" (removed)="remove(fruit)">
    {{fruit}}
    <button matChipRemove [attr.aria-label]="'remove ' + fruit">
      <mat-icon fontSet="fa" fontIcon="fa-times"></mat-icon>
    </button>
  </mat-chip-row>
</mat-chip-grid>

See that the class mat-focus-indicator nukes the icon by setting ::before to display: none;

Expected Behavior
mat-focus-indicator should not set display: none on ::before as it impacts all other ::before psuedoselectors
Actual Behavior
Not overriding icons which use the ::before psuedoselector
Environment

Angular:
CDK/Material: 16 (and 15)
Browser(s): Edge
Operating System (e.g. Windows, macOS, Ubuntu): Windows

There is also a selector which overrides the content attribute thus removing the font...
https://github.com/angular/components/blame/f0a5008b237d4bde3e8d759ca9e793c164656d7a/src/material/chips/chip.scss#L320-L322

I couldn't reproduce the issue based on the provided markup. Here's what it looks like without focus indicators:

And here's what it looks like with focus indicators:

Both appear to be working as expected.

We fixed this by overriding font-awesome to this. And yes the current scss is setting the display on icons inside of chips to "none" when those icons are using the ::before method and there is a mix of legacy and new components.
 I needed to add display: contents here since 'mat-focus-indicator' was nuking the ::before seletor
by setting the display to none. This is a workaround to get the icons to show up. */
@each $name, $icon in $fa-icons {
  .#{$fa-css-prefix}-#{$name}::before { content: unquote("\"#{ $icon }\""); display: contents; }
}

Closing since you've resolved the issue and we were unable to reproduce on our side.


speed up examples

I am quoting @ntolley from here:

I think a very necessary discussion will be which examples we are ok with using a longer dt. Especially since this PR is adding two more simulations to the pile. At the moment it is set to 0.025 ms or 40kHz which is right in line with state-of-the-art recording setups. While a larger dt may result in less accurate simulations, most (if any) of the examples will see no appreciable change. Especially for subthreshold simulations where we're only interested in 1-100 Hz oscillations (slight inaccuracies in the ODE solver may be more impactful for spiking dynamics).

We need to start thinking of strategies to speed up the examples so CircleCI doesn't run forever when we make a pull request.
Another strategy might be to add an argument prune_weights_below=1e-8 or some such thing ... not sure how much speed gains we might get from that.
Finally, we could also consider running the builds in parallel or running only the affected example etc. But these CircleCI tricks should be last resort ...

I really want to agree to increasing dt, but I think we'd regret it. At least in public-facing pages. Even a factor of 2 will lead to visible changes in dipole waveforms. Do we really want to equivocate which small differences are meaningful, and which aren't? Our model outputs are highly weighted towards the spiking dynamics of the cells, not sub-threshold phenomena (this is a little counter-intuitive, and something I think about a lot).
If we want to avoid including MPI in our examples, the only alternative I can see is to have CircleCi only rebuild the examples that have changed. I suppose that would require some sort of persistence of the builds, which is unlikely to be possible...


Add SkyDNS logo

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/202/
Test FAILed.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/203/
Test PASSed.


chore: fix and refine execution plan

What Did
Related PR: https://github.com/pingcap/tidb-dashboard/pull/1561

Fix export statements fail issue which is caused by incorrect gin router middlewares order and incorrect logic to generate binary_plan_text field
Fix the logic to generate binary_plan_text field
Add binary_plan_json field
Support download execution plan text
Support hide and show execution plan table columns
Refactor parse logic to generate execution plan table rows, parse from execution plan text, instead of from execution plan json object
Truncate too long text to prevent the render performance issue, showing too long text for user is nonsense

TODO: check the compatibility for old tidb versions.
Preview
https://github.com/pingcap/tidb-dashboard/assets/1284531/632632e2-ff7b-4027-bc94-eff33221443b

hi @mornyx @shhdgit , this PR is ready to review, PTAL, thanks~
I make this PR as draft still because I don't want it to be merged automatically before I check the compatibility with the old TiDB versions.


Update manage-the-usage-of-a-phone-number.md

Docs Build status updates of commit 378ec6f:
:warning: Validation status: warnings

File
Status
Preview URL
Details

Teams/manage-the-usage-of-a-phone-number.md
:warning:Warning

Details

Teams/manage-the-usage-of-a-phone-number.md

Line 51, Column 44: [Warning: file-not-found - See documentation] Invalid file link: './contact-tns-service-desk.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Variables passed into Badge name only resolve to default values

Hi!
When using parameters for the badge name, e.g. %env.platform%-%env.build_configuration%, looking at UnrealGameSync the badge created will only use the default values for the parameters instead of the actual passed parameters. I have also double checked, that the variables I used here are actually changed and set in the build.
Am I doing something wrong or does something need to be implemented into the plugin to re-resolve the variables?

@markusstephanides I don't think any of our badges use that functionality, so I'll have to look into it. Is it true for all the statuses of your badge? It's possible that commit status publishers runs in a manner where it's not evaluating the variables in the context of the build job itself. I'll take a peek.

Yes, it is. Thanks for looking into this!

I've filed a ticket with TeamCity support about this, as I don't see anything obvious I'm supposed to do to expand them. The values are used verbatim from SBuildFeatureDescriptor.getParameters (as that's what's passed in to the plugin), and so if they're expanding to some value, then I don't even know how I'd go about getting the unexpanded variables and expanding them "correctly".

Ok, thank you very much for the effort and keeping me posted!

Hi, do you have any updates about this?
Thx!


HDXDSYS-691 funding

I only have one thing to flag about this pipeline:
In the schema I says we should aggregate any plans with the same appeal code. However, those are unique per country.  There are some regional plans but I don't feel that we should aggregate those because we wouldn't be able to assign them to a single country.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

47 of 47 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 91.614%

Totals

Change from base Build<PHONE_NUMBER>:
0.4%

Covered Lines:
863

Relevant Lines:
942

💛  - Coveralls