Support

Sorry for opening an issue for this.
I just wanted to say thanks for taking this over. Flask security has been a sore spot for me trying to advocate in keeping our stack in python, and due to your work, this is going to help me immensely.
Really appreciate it, thank you so much!
Ok, now what this ticket is specifically about. Seeing a "Pallets" ref in your docs made me think of it.
I support some libs through scheduled donations through Pallets, and considering how valuable you taking this over is to this community, you should set up taking that same kind of "Show Support" option for us to thank you.

You are very welcome - and thanks for the kind words. I would love feedback (you can email be directly if you want) about the features you are using, what is annoying, inconsistent, etc. Of course PRs are also welcome - but if you don't have time for that - dropping me a line or opening an issue (even if it is small or a idea for how to improve docs etc) - please do so.
My next step - hopefully in a week or so is to move this repo to a github organization - it will still be me for now - but that offers a lot more flexibility down the road
3.4 was a huge release with lots of features - 4.0 will be a drop py2.7 and maybe some other old stuff release - probably in a couple months....

I will definitely do that. Thanks again for all the work.


completion: redundant f (field) and m (method) showing for some methods

I'm pretty sure I've seen this a few times.. need to track down an example and figure out what's going on.

pj.SetClass has both for pj = nt.ConnectLayers


fix: minor issues found during test

This looks good to me. I figured out my other problem. fetchUserInfo() supports passing in an access token string as a single argument. If you pass in a user's access token, it tries to hit the /applications/... path instead of /me/.... That's the reason I was getting a 401. See here: https://github.com/rownd/node/pull/1/files#diff-f222a5de97fe90cf5d60d95868361a1b428cd296e5109f2bf809859590702dc4R117
If I call fetchUserInfo({ user_id: decodedJWT['https://auth.rownd.io/app_user_id'] }); it works fine.

Good point. There's maybe not even a reason to allow passing the token here, since it doesn't get you anything the app keys don't already provide.


Plugin does not play nicely with partially staged files

I often commit quite late after playing around and partially stage files to have meaningful commits. The plugin does not play nicely with this workflow, as it stages the whole file (even if it wasn't a java file but e.g. pom.xml).
The plugin should only work on the staged state of a file and not regard a partially staged file as added.

Indeed.
Could you help with a Pull Request?


ESP32-S3 BLE Server Vs. Detection on PC Bluetooth Dongle

Hi!
I need to create a project with bluetooth connection on my PC, but ANY BT library I use show only on my smartphone the connection.
My question:
Is this a caracteristic of ESP32-S3 BLE? Or I need do a extra config to work on my PC in NimBLE?
My dongle is a Baseus BT 5.0, like this:
https://pt.aliexpress.com/item/1005003601253541.html
Another thing, i try to use all examples but... Well, don't works on this situation...

I'm not sure what you mean, does the PC not see the advertisements from the ESP32?

I'm not sure what you mean, does the PC not see the advertisements from the ESP32?

Yes, now i have success with this, but after 2 seconds or less my device disconnect on PC. See Code above:
`#include <NimBLEDevice.h>
#define SERVICE_UUID        "4fafc201-1fb5-459e-8fcc-c5c9c331914b"
#define CHARACTERISTIC_UUID "beb5483e-36e1-4688-b7f5-ea07361b26a8"
static NimBLEServer* pServer;
static NimBLEService* pService;
static NimBLECharacteristic* pCharacteristic;
/**  None of these are required as they will be handled by the library with defaults. **
**                       Remove as you see fit for your needs                        /
class ServerCallbacks: public NimBLEServerCallbacks {
void onConnect(NimBLEServer pServer) {
Serial.println("Client connected");
Serial.println("Multi-connect support: start advertising");
NimBLEDevice::startAdvertising();
};
/** Alternative onConnect() method to extract details of the connection.
*  See: src/ble_gap.h for the details of the ble_gap_conn_desc struct.
/
void onConnect(NimBLEServer pServer, ble_gap_conn_desc* desc) {
Serial.print("Client address: ");
Serial.println(NimBLEAddress(desc->peer_ota_addr).toString().c_str());
/** We can use the connection handle here to ask for different connection parameters.
*  Args: connection handle, min connection interval, max connection interval
*  latency, supervision timeout.
*  Units; Min/Max Intervals: 1.25 millisecond increments.
*  Latency: number of intervals allowed to skip.
*  Timeout: 10 millisecond increments, try for 5x interval time for best results.
/
NimBLEDevice::startAdvertising();
pServer->updateConnParams(desc->conn_handle, 24, 48, 0, 60);
};
void onDisconnect(NimBLEServer pServer) {
Serial.println("Client disconnected - start advertising");
NimBLEDevice::startAdvertising();
};
void onMTUChange(uint16_t MTU, ble_gap_conn_desc* desc) {
Serial.printf("MTU updated: %u for connection ID: %u\n", MTU, desc->conn_handle);
};
/********************* Security handled here **********************
****** Note: these are the same return values as defaults ******/
uint32_t onPassKeyRequest(){
Serial.println("Server Passkey Request");
/ This should return a random 6 digit number for security
*  or make your own static passkey as done here.
*/
return 123456;
};
bool onConfirmPIN(uint32_t pass_key){
    Serial.print("The passkey YES/NO number: ");Serial.println(pass_key);
    /** Return false if passkeys don't match. */
    return true;
};

void onAuthenticationComplete(ble_gap_conn_desc* desc){
    /** Check that encryption was successful, if not we disconnect the client */
    // if(!desc->sec_state.encrypted) {
    //     NimBLEDevice::getServer()->disconnect(desc->conn_handle);
    //     Serial.println("Encrypt connection failed - disconnecting client");
    //     return;
    // }
    Serial.println("Starting BLE work!");
};

};
/** Handler class for characteristic actions /
class CharacteristicCallbacks: public NimBLECharacteristicCallbacks {
void onRead(NimBLECharacteristic pCharacteristic){
Serial.print(pCharacteristic->getUUID().toString().c_str());
Serial.print(": onRead(), value: ");
Serial.println(pCharacteristic->getValue().c_str());
};
void onWrite(NimBLECharacteristic* pCharacteristic) {
    Serial.print(pCharacteristic->getUUID().toString().c_str());
    Serial.print(": onWrite(), value: ");
    Serial.println(pCharacteristic->getValue().c_str());
};
/** Called before notification or indication is sent,
 *  the value can be changed here before sending if desired.
 */
void onNotify(NimBLECharacteristic* pCharacteristic) {
    Serial.println("Sending notification to clients");
};

/** The status returned in status is defined in NimBLECharacteristic.h.
 *  The value returned in code is the NimBLE host return code.
 */
void onStatus(NimBLECharacteristic* pCharacteristic, Status status, int code) {
    String str = ("Notification/Indication status code: ");
    str += status;
    str += ", return code: ";
    str += code;
    str += ", ";
    str += NimBLEUtils::returnCodeToString(code);
    Serial.println(str);
};

void onSubscribe(NimBLECharacteristic* pCharacteristic, ble_gap_conn_desc* desc, uint16_t subValue) {
    String str = "Client ID: ";
    str += desc->conn_handle;
    str += " Address: ";
    str += std::string(NimBLEAddress(desc->peer_ota_addr)).c_str();
    if(subValue == 0) {
        str += " Unsubscribed to ";
    }else if(subValue == 1) {
        str += " Subscribed to notfications for ";
    } else if(subValue == 2) {
        str += " Subscribed to indications for ";
    } else if(subValue == 3) {
        str += " Subscribed to notifications and indications for ";
    }
    str += std::string(pCharacteristic->getUUID()).c_str();

    Serial.println(str);
};

};
/** Handler class for descriptor actions /
class DescriptorCallbacks : public NimBLEDescriptorCallbacks {
void onWrite(NimBLEDescriptor pDescriptor) {
std::string dscVal = pDescriptor->getValue();
Serial.print("Descriptor witten value:");
Serial.println(dscVal.c_str());
};
void onRead(NimBLEDescriptor* pDescriptor) {
    Serial.print(pDescriptor->getUUID().toString().c_str());
    Serial.println(" Descriptor read");
};

};
/** Define callback instances globally to use for multiple Charateristics \ Descriptors */
static DescriptorCallbacks dscCallbacks;
static CharacteristicCallbacks chrCallbacks;
void setup() {
Serial.begin(115200);
Serial.println("Starting NimBLE Server");
/** sets device name */
NimBLEDevice::init("NimBLE-Arduino");

pServer = NimBLEDevice::createServer();
pServer->setCallbacks(new ServerCallbacks());

pService = pServer->createService(SERVICE_UUID);
pCharacteristic = pService->createCharacteristic(
                                           CHARACTERISTIC_UUID,
                                          NIMBLE_PROPERTY::READ |
                                          NIMBLE_PROPERTY::WRITE);

pService->start();

pCharacteristic->setValue("Test");
pCharacteristic->setCallbacks(&chrCallbacks);

NimBLEAdvertising* pAdvertising = NimBLEDevice::getAdvertising();
//pAdvertising->addServiceUUID(pService->getUUID());
pAdvertising->addServiceUUID(SERVICE_UUID);
// pAdvertising->addServiceUUID(pBaadService->getUUID());
/** If your device is battery powered you may consider setting scan response
 *  to false as it will extend battery life at the expense of less data sent.
 */
//pAdvertising->setScanResponse(false);
pAdvertising->start();

Serial.println("Advertising Started");

}
void loop() {
/** Do your thing here, this just spams notifications to all connected clients /
//  if(pServer->getConnectedCount()) {
// //     // NimBLEService pSvc = pServer->getServiceByUUID("BAAD");
// //     // if(pSvc) {
// //     //     NimBLECharacteristic* pChr = pSvc->getCharacteristic("F00D");
// //     //     if(pChr) {
// //     //         pChr->notify(true);
// //     //     }
// //     // }
// }
//pServer->getConnectedCount();
//pCharacteristic->setValue("Test loop");
//pCharacteristic->notify();
delay(100);
}`
I try remove notify to see if i have any diference reaction, but nothing happens...

Any suggestion @h2zero?

I would need to see the logs from the esp32 at debug level to provide much insight with this.

@h2zero how can I enable logs ?

I believe this to be related to bluetooth classic, not this repo. please reopen if still having this issue.


TraceTelemetryConverter sets Severity correctly, but is still recorded as Information in Application Insights.

Description
The TraceTelemetryConverter class sets the TraceTelemetry.Severity correctly yet it is still recorded as "Information" in Azure Application Insights.
I have created the following converter just so I could validate the TraceTelemetryConverter is producing the correct results:
public class AppInsightsTelemetryConverter : TraceTelemetryConverter
{
    public override IEnumerable<ITelemetry> Convert(LogEvent logEvent, IFormatProvider formatProvider)
    {
        logEvent = logEvent ?? throw new ArgumentNullException(nameof(logEvent));

        var converted = base.Convert(logEvent, formatProvider).First();

        var trace = converted as TraceTelemetry;
        if (trace == null)
        {
            return new[] { converted };
        }

        return new[] { converted };
    }
}

Which produces the following watch values:

That is still not recorded correctly in Azure Portal

I'm configuring an Azure v4 function.  My config is the following in Program.cs
var host = new HostBuilder()
    .ConfigureFunctionsWebApplication((context, builder) =>
    {
        builder.Services.AddAuthentication(JwtBearerDefaults.AuthenticationScheme)
            .AddMicrosoftIdentityWebApi(context.Configuration.GetSection(AzureB2CConfigSection));

        builder.Services.AddApplicationInsightsTelemetry(options =>
        {
            options.ConnectionString = context.Configuration[AppInsightsConnectionString];
        });
    })
    .ConfigureServices(services =>
    {
        services.AddSingleton<ILoggingLevelSwitchService, LoggingLevelSwitchService>();
        services.ConfigureFunctionsApplicationInsights();
        services.Configure<JsonSerializerOptions>(options =>
        {
            options.PropertyNamingPolicy = JsonNamingPolicy.CamelCase;
            options.PropertyNameCaseInsensitive = true;
        });

        Log.Logger = new LoggerConfiguration()
            .WriteTo.Console(
                outputTemplate: "{Timestamp:HH:mm} [{Level}] {Message}{NewLine}{Exception}",
                formatProvider: System.Globalization.CultureInfo.InvariantCulture
            )
            .WriteTo.ApplicationInsights(
                telemetryConfiguration: serviceProvider.GetRequiredService<TelemetryConfiguration>(),
                // telemetryConverter: TelemetryConverter.Traces,
                telemetryConverter: new AppInsightsTelemetryConverter())
            .CreateLogger();

        services.AddLogging(configure => {
            configure.ClearProviders();
            configure.AddSerilog(Log.Logger);
        });

    })
    .Build();

host.Run()

Reproduction

Create an Azure Function v4 and configure serilog with the ApplicationInsights sink.
Write a message that is something other than information (such as warning)
Observe the log message recorded in Application insights.

Expected behavior
If the written log message is Warning, expect to see a log message in application insights with a warning severity.
Actual behavior
Regardless of the log level, all entries are recorded as information in application Insights
Relevant package, tooling and runtime versions
.net 8
    <FrameworkReference Include="Microsoft.AspNetCore.App" />
    <PackageReference Include="Microsoft.ApplicationInsights.AspNetCore" Version="2.22.0" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker" Version="1.22.0" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.Http" Version="3.2.0" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.Http.AspNetCore" Version="1.3.0" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Sdk" Version="1.17.2" />
    <PackageReference Include="Microsoft.ApplicationInsights.WorkerService" Version="2.22.0" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker.ApplicationInsights" Version="1.2.0" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.OpenApi" Version="1.5.1" />
    <PackageReference Include="Microsoft.Identity.Web" Version="2.18.2" />
    <PackageReference Include="Serilog.AspNetcore" Version="8.0.1" />
    <PackageReference Include="Serilog.Extensions.Logging" Version="8.0.0" />
    <PackageReference Include="Serilog.Sinks.ApplicationInsights" Version="4.0.0" />

Additional context
It seems that the issue occurs somewhere after the code converts the LogLevel to the ApplicationInsights Severity.

I updated to the latest packages, and the code began to work as expected.

Great, thanks for the follow-up :+1:


Error logs created by Synapse COPY INTO activity cannot be downloaded using the client tool

The files could be loaded using the Azure portal Storage Explorer.
2020/12/16 16:35:41 AzcopyVersion  10.7.0
2020/12/16 16:35:41 OS-Environment  windows
2020/12/16 16:35:41 OS-Architecture  amd64
2020/12/16 16:35:41 Log times are in UTC. Local time is 16 Dec 2020 18:35:41
2020/12/16 16:35:41 Job-Command copy https://dumarkovadlsgen2.blob.core.windows.net/dudi/errorsfolder/_rejectedrows/20201216_104011/QID272_10_C.Error.Txt?se=2021-01-15t16%3A35%3A37z&sig=-REDACTED-&sp=rl&sr=c&sv=2019-12-12 C:\Users\dumarkov\Downloads\QID272_10_C.Error.Txt --output-type=json --cancel-from-stdin --overwrite=prompt --check-md5 FailIfDifferent --from-to=BlobLocal --recursive
2020/12/16 16:35:41 Number of CPUs: 4
2020/12/16 16:35:41 Max file buffer RAM 2.000 GB
2020/12/16 16:35:41 Max concurrent network operations: 32 (Based on number of CPUs. Set AZCOPY_CONCURRENCY_VALUE environment variable to override)
2020/12/16 16:35:41 Check CPU usage when dynamically tuning concurrency: true (Based on hard-coded default. Set AZCOPY_TUNE_TO_CPU environment variable to true or false override)
2020/12/16 16:35:41 Max concurrent transfer initiation routines: 64 (Based on hard-coded default. Set AZCOPY_CONCURRENT_FILES environment variable to override)
2020/12/16 16:35:41 Max enumeration routines: 16 (Based on hard-coded default. Set AZCOPY_CONCURRENT_SCAN environment variable to override)
2020/12/16 16:35:41 Parallelize getting file properties (file.Stat): false (Based on AZCOPY_PARALLEL_STAT_FILES environment variable)
2020/12/16 16:35:41 Max open files when downloading:<PHONE_NUMBER> (auto-computed)
2020/12/16 16:35:41 ISO 8601 START TIME: to copy files that changed after this job started, use the parameter --include-after=2020-12-16T16:35:32Z
2020/12/16 16:35:41 Any empty folders will not be processed, because source and/or destination doesn't have full folder support
2020/12/16 16:35:41 Final job part has been created
2020/12/16 16:35:41 PERF: primary performance constraint is Unknown. States: C:  0, R:  0, W:  0, F:  0, H:  0, B:  0, S:  0, P:  0, Q:  0, D:  0, E:  0, T:  0, GRs:  0
2020/12/16 16:35:41 0.0 %, 0 Done, 0 Failed, 1 Pending, 0 Skipped, 1 Total,
2020/12/16 16:35:41 JobID=e979bd19-10d8-0741-5a63-7122bbabea14, credential type: Anonymous
2020/12/16 16:35:41 scheduling JobID=e979bd19-10d8-0741-5a63-7122bbabea14, Part#=0, Transfer#=0, priority=0
2020/12/16 16:35:41 Final job part has been scheduled
2020/12/16 16:35:41 INFO: [P#0-T#0] has worker 45 which is processing TRANSFER
2020/12/16 16:35:41 ==> REQUEST/RESPONSE (Try=1/318.9916ms, OpTime=634.0021ms) -- RESPONSE SUCCESSFULLY RECEIVED
GET https://dumarkovadlsgen2.blob.core.windows.net/dudi/errorsfolder/_rejectedrows/20201216_104011/QID272_10_C.Error.Txt?se=2021-01-15t16%3A35%3A37z&sig=-REDACTED-&sp=rl&sr=c&sv=2019-12-12&timeout=901
X-Ms-Request-Id: [3b931c53-501e-0061-7fc9-d384b3000000]
2020/12/16 16:35:41 ERR: [P#0-T#0] DOWNLOADFAILED: https://dumarkovadlsgen2.blob.core.windows.net/dudi/errorsfolder/_rejectedrows/20201216_104011/QID272_10_C.Error.Txt?se=2021-01-15t16%3A35%3A37z&sig=-REDACTED-&sp=rl&sr=c&sv=2019-12-12 : 000 : the MD5 hash of the data, as we received it, did not match the expected value, as found in the Blob/File Service. This means that either there is a data integrity error OR another tool has failed to keep the stored hash up to date. (NOTE In the specific case of downloading a Page Blob that has been used as a VM disk, the VM has probably changed the content since the hash was set. That's normal, and in that specific case you can simply disable the MD5 check. See the documentation for the check-md5 parameter.). When Checking MD5 hash. X-Ms-Request-Id:
Dst: %5C%5C?\C:\Users\dumarkov\Downloads\QID272_10_C.Error.Txt
2020/12/16 16:35:41 INFO: [P#0-T#0] https://dumarkovadlsgen2.blob.core.windows.net/dudi/errorsfolder/_rejectedrows/20201216_104011/QID272_10_C.Error.Txt?se=2021-01-15t16%3A35%3A37z&sig=-REDACTED-&sp=rl&sr=c&sv=2019-12-12 Deleting incomplete destination file Dst: %5C%5C?\C:\Users\dumarkov\Downloads\QID272_10_C.Error.Txt
2020/12/16 16:35:41 JobID=e979bd19-10d8-0741-5a63-7122bbabea14, Part#=0, TransfersDone=1 of 1
2020/12/16 16:35:41 all parts of entire Job e979bd19-10d8-0741-5a63-7122bbabea14 successfully completed, cancelled or paused
2020/12/16 16:35:43 PERF: primary performance constraint is Unknown. States: C:  0, R:  0, W:  0, F:  0, H:  0, B:  0, S:  0, P:  0, Q:  0, D:  0, E:  0, T:  0, GRs: 32

@dudu-markovitz If the error is false alarm, you can try disabling the MD5 hash check from "Settings -> Transfers -> AzCopy -> Check MD5" and select No Check or Log only. That should unblock you from this error. If the file does get corrupted, then the error is expected.

Yep. That did the work.
We still need to understand if the MD% hash check is problematic or there is something wrong with the files (which might be related to the ADLS gen 2 API)
David (דודו) Markovitz
Sr. Cloud Solution Architect, Data & AI      Mobile: +972-525-834-304
Microsoft Israel                                              Office: +972-747-119-274
<EMAIL_ADDRESS>From: JasonYeMSFT<EMAIL_ADDRESS>Sent: Thursday, December 17, 2020 00:36
To: microsoft/AzureStorageExplorer<EMAIL_ADDRESS>Cc: David Markovitz<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [microsoft/AzureStorageExplorer] Error logs created by Synapse COPY INTO activity cannot be downloaded using the client tool (#3910)
@dudu-markovitzhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdudu-markovitz&data=04|01|Dudu.Markovitz%40microsoft.com|d684fc46f8c1494373f908d8a212f679|72f988bf86f141af91ab2d7cd011db47|1|0|637437549635934224|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=uupH%2Bcgd8BPtvDHyLmio2YQIL%2BOYPlCunusmDGFrGgQ%3D&reserved=0 If the error is false alarm, you can try disabling the MD5 hash check from "Settings -> Transfers -> AzCopy -> Check MD5" and select No Check or Log only. That should unblock you from this error. If the file does get corrupted, then the error is expected.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FAzureStorageExplorer%2Fissues%2F3910%23issuecomment-747081623&data=04|01|Dudu.Markovitz%40microsoft.com|d684fc46f8c1494373f908d8a212f679|72f988bf86f141af91ab2d7cd011db47|1|0|637437549635944186|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=6W5SRtWO1X1lhU%2BBWdioJGio460rHvE1ISA4r1cJcsE%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGLSYSDR2K3SWDLNUSSY5PLSVEY47ANCNFSM4U6IMLPQ&data=04|01|Dudu.Markovitz%40microsoft.com|d684fc46f8c1494373f908d8a212f679|72f988bf86f141af91ab2d7cd011db47|1|0|637437549635944186|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=tLcq%2B1VuJD%2FhxurAnv2v0fpeP7n0rDYphtD3VSeEcOY%3D&reserved=0.

@dudu-markovitz the MD5 hash is meant to be updated by any tools interacting with the data. So try to think about any tooling that touches your data and check to see if they are all doing MD5 updates. By default, storage explorer (really azcopy), will put/update the MD5 when modifying blobs. So I don't think we're the responsible ones. But if you investigate and think that we are, please let us know! Since this immediate issue is fixed, I'll close it. Feel free to open another if you think there's an issue with Storage Explorer not updating the MD5 hash. Thanks!

@dudu-markovitz the MD5 hash is meant to be updated by any tools interacting with the data. So try to think about any tooling that touches your data and check to see if they are all doing MD5 updates. By default, storage explorer (really azcopy), will put/update the MD5 when modifying blobs. So I don't think we're the responsible ones. But if you investigate and think that we are, please let us know! Since this immediate issue is fixed, I'll close it. Feel free to open another if you think there's an issue with Storage Explorer not updating the MD5 hash. Thanks!


feat: Add DaSCH contact information somewhere in the logged-in part

Is your feature request related to a problem? Please describe.
There is a section with DaSCH contact information in the footer of the logged-out homepage but there is nothing in the logged-in part.
Describe the solution you'd like
Contact information should be added on the dashboard or in the user menu > tab e.g."Need help" or "Contact DaSCH" or something equivalent.
Describe alternatives you've considered
Contact form.
Additional context
N.A.

Good point


mirage enabled even when using --proxy flag

It looks like this PR https://github.com/miragejs/ember-cli-mirage/pull/2495 released 2 weeks ago under 3.0.0-alpha.4 has removed the config/environment.js file
That config/environment.js file had a usingProxy() function that would set ENV['ember-cli-mirage'].usingProxy to true if proxy flag was set
https://github.com/miragejs/ember-cli-mirage/pull/2495/files#diff-77d6e2aab53cbf594475ecab6eac3882cd95a1c3d95085833caa58c39cb0a910L4-L11
When usingProxy is true, that would disable mirage server in the initializer
https://github.com/miragejs/ember-cli-mirage/blob/dc3852505a18fae1abde78ef13dc9add9a30c057/packages/ember-cli-mirage/app/initializers/ember-cli-mirage.js#L54
Right now nothing sets that usingProxy property in the config, so mirage ends up being enabled even when proxies are set
This seems like an unintentional change (wasn't mentioned in 3.0.0 Breaking Changes)

cc @deepfryed

@Techn1x sorry for churn, fix published in v3.0.1

Thanks so much! You do great work! I am just glad to see things moving in mirage land :)

@Techn1x @SergeAstapov I appreciate the report and the quick response, as my team encountered a related issue on 3.0.0. Updating ember-cli-mirage to 3.0.1 fixed our problem, and the update from 2.4.0 to 3.0.0 went otherwise smoothly. ✨
Even after updating to 3.0.1, one of our developers found that the issue exists on their Windows machine. By temporarily updating the Mirage config to passthrough all requests, the requests to the desired server worked:
function routes() {
  this.passthrough();
  // removed all other overrides
}

We're not sure yet if the Windows issue is an existing one, either in ember-cli-mirage or miragejs. If we can come up with a minimal reproducible app (I imagine this will take time, as we would need to create an Ember app as well as a Rails/Node server), we'll let you know.

@ijlee2 thank you for the pull request! This is released as v3.0.2


[BUG] Groups are not refreshed after creation/deletion

Describe the bug
When I create a new group, the subscriptions page is not updated, the new group is not shown
Same thing when I delete an existing group, the deleted group is still visible on the subscriptions page
To Reproduce

(Re)start the app, and go to the subscriptions page
Click on new to add a new group
In the modam, give the group a name, and click OK
The modal closes, but the group is not shown

Screenshots

Page BEFORE creation

Modal

Page AFTER creation

Page AFTER creation and after app kill/reboot

Device
Oneplus 6, LineageOS 18.1, Fritter 2.10

This one was already fixed in v2.11.0 (#163), but it looks like that version still hasn't made its way through to F-Droid (it takes a few days, sometimes a week, for them to pick it up).
You can download the latest version from GitHub Releases for now, but make sure to export your subscriptions beforehand, as you'll need to uninstall the F-Droid version first!


Display colored icons for packaged apps

Summary of the Pull Request
Show colored icons in Packaged apps.
PR Checklist

[x] Applies to #4534 , #3883
[x] CLA signed. If not, go over here and sign the CLA
[x] Tests added/passed

Info on Pull Request
This PR adds the feature to display colored icons if they are available for an app in Light/Dark windows apps. The following changes were made in this PR

Plate icons when displayed in Light/Dark mode. This was done because some apps only have white icons resulting in difficult visibility when displayed in Light theme mode.
HC icons were prioritized to be displayed(if available) in high contrast modes.
Code to plate the icons was added back from Wox. This was modified to use plating color as the current windows accent color similar to the start menu if the background color is set as transparent in the manifest file.

Validation Steps Performed

Manually validated that colored icons are displayed(if available) in dark/light mode
Manually validated that HC icons are displayed(if available) in HC modes.

@royvou When this PR is ready, the code could be interesting for you for the uri plugin.

Code to plate the icons was added back from Wox. This was modified to use plating color as the current windows accent color similar to the start menu if the background color is set as transparent in the manifest file.

@somil55
As far as I know MS changed the background colour for tiles in start menu with 20H2. Do we need to react on this change?

@somil55 I have not run it just yet, will do that tomorrow. But as far as I can tell looking at your code, you are using the accent color as the background, right?
I agree with @htcfreek - with the latest Windows Insider builds the Start Menu is moving away from colored tiles / backgrounds in the app list and using light gray / dark gray. I personally like that look and feel more, especially for the colorful icons.
Could we make these backgrounds rounded (CornerRadius = 4) as well or is that a big undertaking? All UI elements in Run (and WinUI) are rounded so it would fit better.

@somil55 I have not run it just yet, will do that tomorrow. But as far as I can tell looking at your code, you are using the accent color as the background, right?
I agree with @htcfreek - with the latest Windows Insider builds the Start Menu is moving away from colored tiles / backgrounds in the app list and using light gray / dark gray. I personally like that look and feel more, especially for the colorful icons.

@niels9001 @htcfreek  Background color for an icon is decided based on the following conditions:

If the manifest for the program specifies a color, it is used. For example, manifest for Spotify specifies a #1ed760 background. Apps such as mail, maps, etc. specify #0078D7 background color. I believe these would be accordingly updated in 20H2.
If the manifest file specifies color as transparent, we use accent color. The reason we have to use to do this is that apps might have a white icon that becomes invisible with a white pt run theme. The Messaging app is an example of this. I believe this would need to be changed. I could think of two possible ways to handle this:  First would be to use light gray / dark gray background instead of accent color. Second would be to decide this based on the windows version. @ryanbodrug-microsoft @crutkas Any thoughts on this?

Could we make these backgrounds rounded (CornerRadius = 4) as well or is that a big undertaking? All UI elements in Run (and WinUI) are rounded so it would fit better.

I will update this. Thanks for pointing out this!

Actually, I stand corrected. Start menu changes to high contrast icons when having the light theme enabled.

Actually, search is using accent color. So maybe the current implementation is the way to go.


Conda installation fails

I am unable to install riboWaltz from Conda using the instructions in the README file.
Creating a new environment for riboWaltz with the latest release of R version 3 succeeds:
conda create -y -n waltz r=3.6.0
conda activate waltz

However, installing riboWaltz using the command
conda install -c bioconda ribowaltz

prints this error message
Collecting package metadata (current_repodata.json): done
Solving environment: unsuccessful initial attempt using frozen solve. Retrying with flexible solve.
Solving environment: unsuccessful attempt using repodata from current_repodata.json, retrying with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: unsuccessful initial attempt using frozen solve. Retrying with flexible solve.
Solving environment: \
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed                                                                     -    \ -                       -

UnsatisfiableError: The following specifications were found to be incompatible with a past
explicit spec that is not an explicit spec in this operation (r):

  - ribowaltz -> bioconductor-biostrings -> r[version='3.2.2.*|3.3.1.*|>=2.8.0|>=2.10|>=3.1.0']

followed by a long list of incompatible packages.
I also tried to install riboWaltz without first installing R, using this command:
conda create -y -n waltz -c bioconda ribowaltz=1.2.0

which also finds conflicting packages and fails with an UnsatisfiableError.
I am using a 2019 MacBook Pro with an Intel 8-Core i9 processor running macOS 13.5.2. Thank you for your help.

Hi there,
sorry for the inconvenient.
That line of code for the installation was copied-pasted from the Conda wabpage. Now I can see something has changed, so I'll update the ReadMe with a direct link to the Conda instructions.
You can try to follow that. Unfortunately, I'm not really into the Conda environment, so if it doesn't work following what is written there, I'm won't be of much help.
Let me known how it goes.
Best
Fabio

Hi there,
I'm closing this issue due to inactivity. Please re-open if required.
Best
Fabio


Fix wetty import

In the changes from 1.1.4 to 1.3 the src/server/index.ts lost it's getter for wetty.
public static get wetty(): WeTTy {
    return wetty;
  }

this resulted in a undefined return upon import.

Exports are handled more cleanly with version 2


pause issues

i use pauseDownload && resumeDownload
but when pause just pause callbacks like download progress and download not pause
when pause : downloaded percent = 3.07 and after 10 sec call resume see 20 percent downloaded
`
ipcMain.on("download", (event, info) => {
// info.options.onProgress = status => window.webContents.send("download progress", status);
// download(window, info.url, info.options)
//   .then(dl => window.webContents.send("download complete", dl.getSavePath()));
let DownloadID

DownloadID = manager.download({
  window,
  url: info.url,
  directory: info.options.directory,
  callbacks: {
    onDownloadProgress: async ({ id, item, percentCompleted }) => {
      window.webContents.send('download progress', {
        id,
        percentCompleted,
        bytesReceived: item.getReceivedBytes(),
      });
    },
  }
})

ipcMain.on('cancelDownload', () => {
  manager.cancelDownload(DownloadID)
})

ipcMain.on('pauseDownload', () => {
  manager.pauseDownload(DownloadID)
})

ipcMain.on('resumeDownload', () => {
  manager.resumeDownload(DownloadID)
})

});
`

Have you tried debug logging to see if it is actually getting the download item to pause?
Under the hood all it does is just call the download item pause call. I have noticed sometimes where pausing doesn't cause it to stop immediately if you look at my source code for the initiator.
It's possible that this is an Electron-specific bug as all this lib does is call the underlying pause / resume methods on the item.

Here's my test. I'm not able to reproduce, but I do believe you that you are experiencing it. It's hard to know if it's a bug on my side or in electron itself when I can't reproduce the issue:
https://gist.github.com/theogravity/c25fd0250f455ae3a2f6fdee18dd8bb3
I used electron fiddle to test
One really far-fetched theory is IPC is taking longer than expected to reach the main process and that's why pause triggering is delayed, but I don't think it'd take so long that you'd see progress after 10s after the pause.
If you can create a fiddle that reproduces the issue, it would really help! You can use the Load Fiddle in the toolbar against the URL I provided above to load my fiddle code and adjust the sample.

Another theory is that you're pausing before the internal resume happens:
Non-interactive:
https://github.com/theogravity/electron-dl-manager/blob/main/src/DownloadInitiator.ts#L190
Interactive:
https://github.com/theogravity/electron-dl-manager/blob/main/src/DownloadInitiator.ts#L160
What's going on is I do the following when the user initiates the download:

Immediately pause
Set up the listeners / allow the user to select a download location (if using a dialog box)
Resume the download

I bet what's happening is the user-set pause gets called before the resume.
Let me see if I can address this somehow.

I published 2.4.1
Re-open if you still have the issue.

thanks,
after download version 2.4.1 my code has error

Thanks for the report. This was def a situation where unit tests wouldn't have easily discovered the issue. I had to test using a sample app. Found there was a race condition where the item actually might not be populated when you call pauseDownload() / pause().
I've published 3.0.0. The big change is you need to use await download() instead of download(). This will ensure that the item data is properly populated before you use it against pauseDownload()


Allow configurable logger

This fixes #4 by allowing the configuration of the logger function via the options. I didn't add unit tests, but the current tests are still passing. If this is something you think is worth using, and you want it unit tested before merging, just let me know.

What is status of this PR?

Great, thanks!


Give maas rabbit user perms on all vhosts

Previously the maas user only had permisions on the vhost "/" which is
no longer used by the openstack services, as they each have their own
vhost.
No modification to the plugin is necessary, as the same api calls will
return data from all the vhosts the user has access to.
Closes #1113

See my inline comments

On the initial kilo deployment:
15:12:14 ********************** Run Holland ***********************
15:12:14 + run_holland
15:12:14 ++ sudo lxc-ls
15:12:14 ++ grep galera
15:12:14 ++ head -n1
15:12:14 + sudo lxc-attach -n jrpcaioiad-d1a_galera_container-26712479 -- /bin/bash -c 'holland bk'
15:12:14 No handlers could be found for logger "holland.core.util.bootstrap"
15:12:14 + HOLLAND_RC=1

I think the upgrade job failed because of timing, not because its an upgrade.
Timings:

14:41 - swift job started
14:42 - ceph job started (jenkins-rpc clone already done)
14:43 - holland test merged to jenkins-rpc
14:47 - upgrade job started

recheck_swift

recheck_upgrade

hmm, earlier theory incorrect as r e c h e c k_swift succeeded. Will wait and see what happens to the upgrade this time.

recheck_upgrade

recheck_upgrade

The upgrade job has been disabled for master, so the failure will not be removed by recheck_ugprade. This patch is ready for review. It requires manual testing, as the gate didn't catch the original failure (bad notification message count).

Updated commit message to not auto-close parent issue.

recheck_swift
10:08:15 TASK: [Wait for container ssh] ************************************************
10:08:15 skipping: [jrpcaioiad-7dd]
10:28:15 Build timed out (after 20 minutes). Marking the build as aborted.
10:28:15 Build was aborted

👍  tested. Before:
# ./rabbitmq_status.py -H <IP_ADDRESS> -n master1_rabbit_mq_container-9f1a0eaf -U maas_user -p 745a62f3038d2474f29222e9634f2beebc258bcce6a90906b677252fe43d|grep notific
metric rabbitmq_notification_messages int64 0 messages
metric rabbitmq_msgs_excl_notifications int64 360 messages

alarm was generated
After:
# ./rabbitmq_status.py -H <IP_ADDRESS> -n master1_rabbit_mq_container-9f1a0eaf -U maas_user -p 745a62f3038d2474f29222e9634f2beebc258bcce6a90906b677252fe43d|grep notific
metric rabbitmq_notification_messages int64 360 messages
metric rabbitmq_msgs_excl_notifications int64 0 messages

alarm was not generated/or the alarm cleared if it had been previously generated
👍

👍 the output from the the queue API includes the additional vhosts once this patch is run and so looks to be accounting for all the messages.


pc-i440fx: use storage api

Use the superior API to specify the storage request. This does not require to specify the accessmode anymore.

@kbidarkar @acardace can you please have a look?


bbsplit behavior with ambiguous reads : --extra_bbsplit_args possible to add ?

I'm dealing with PDX.
I wanted to remove ambiguous reads that map on mouse genome.
From my understanding, in the pipeline, bbsplit keep ambigous (ambiguous2=all) reads (mapped on both genome reference)  for mapping.
It doesn't remove reads that are mapped on other references from the bbsplit_fasta_list.
From here https://github.com/BioInfoTools/BBMap/blob/master/sh/bbsplit.sh
ambiguous2=<best>   Set behavior only for reads that map ambiguously to multiple different references.
                    Normal 'ambiguous=' controls behavior on all ambiguous reads;
                    Ambiguous2 excludes reads that map ambiguously within a single reference.
                       best   (use the first best site)
                       toss   (consider unmapped)
                       all   (write a copy to the output for each reference to which it maps)

I would like to remove these reads using ambiguous2=toss.
Could be useful to add an extra_bbsplit_args parameter ? or is there another way to change this behavior whereas I'm using a singularity profile ?

The extra_tool_args parameters were never meant to cater for all tools, but to be a convenience solution for the most commonly requested tools. The primary way of customizing a tool's arguments is via a custom configuration.

Sorry but it's not clear to me where to apply the change.
For Nextflow DSL2 nf-core pipelines - parameters defined in the parameter block in custom.config files WILL NOT override defaults in nextflow.config!
So how do you override something ?
From here  I see exactly the 'ambigous2' parameter I want to change in module.config.
if (!params.skip_bbsplit) {
    process {
        withName: 'BBMAP_BBSPLIT' {
            ext.args   = 'build=1 ambiguous2=all maxindel=150000'
            publishDir = [
                [
                    path: { "${params.outdir}/bbsplit" },
                    mode: params.publish_dir_mode,
                    pattern: '*.txt'
                ],
                [
                    path: { "${params.outdir}/bbsplit" },
                    mode: params.publish_dir_mode,
                    pattern: '*.fastq.gz',
                    enabled: params.save_bbsplit_reads
                ]
            ]
        }
    }
}

So should I use the params-file option pointing to a json file with :
`{  " ext.args": "ambiguous2=all"   }
`
I doubt that is the good syntax...

Could be useful to add an extra_bbsplit_args parameter ?

Well, the convention in nf-core is, that in the context of a whole pipeline variables are referred to as parameters and in the context of a single tool as arguments. This is, why your proposed new pipeline parameter would be called extra_bbsplit_args and not extra_bbsplit_params.
So what you need to do to override the arguments is creating a custom config file that looks approximately like this:
process {
        withName: 'BBMAP_BBSPLIT' {
            ext.args   = 'build=1 ambiguous2=toss maxindel=150000'
        }
    }

This file you can provide with nextflow run -c your_config ... thanks to Nextflow's config priority. The parameter block is irrelevant here, since you are only reconfiguring a single process.


fix: inputToggle flx growth issue

:tada: This PR is included in version 1.0.0-next.17 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.0.0-next.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Edges of the floor are causing issues with wheel collision

Is this still an issue? https://github.com/srobo/competition-simulator/pull/230 fixed a bunch of things -- any chance it fixed this too?

The arena itself was fully rebuilt in #260.
I'm going to close this as we don't seem to have a reproduction of the issue. However if it reappears we can open this again.


aws.StaticSite errorPage issue

Version: 0.0.393
Currently it is not possible to use the errorPage field in the StaticSite construct.
const website = new sst.aws.StaticSite('Website', {
      path: 'website',
      indexPage: 'index.html',
      // wont work
      // errorPage: 'index.html',
      // errorPage: '/index.html',
      domain: {
        name: 'example.com',
        redirects: [
          'www.example.com',
        ],
      },
      transform: {
        cdn: {
          customErrorResponses: [
            // works
            {
              errorCode: 404,
              responsePagePath: '/index.html',
              responseCode: 200,
            },
          ],
        }
      }
});

The deploy fails with message:
CloudFront: UpdateDistribution, https response error StatusCode: 400, RequestID: XXX, IllegalUpdate: Your request must specify both ResponsePagePath and ResponseCode together or both should be empty.

I'm also getting the same CloudFront 400 error when adding an errorPage this way:
--- sst.config.ts
+++ sst.config.ts
@@ -21,6 +21,7 @@ export default $config({

     const site = new sst.aws.StaticSite("Extensions", {
       path: "repo",
+      errorPage: "404.html",
       domain: extDomains[$app.stage],
     });

I just ran into this bug. Here's a quick PR to fix it: #635


[enhancement] Cleaner serial management by the kernel

Better distinction between debug and release mode

@arnauldm You should update defconfig files with the newer config if the update change the config list, or the CI will lock each time it runs

CI reloaded. Should work now :)


Min(comparer) emits comparer result value instead of original stream value

RxJS version:
5.0.0-beta.11
Code to reproduce:
Observable.range(1,2).min((a,b) => a > b ? -1 : a < b ? 1 : 0)
Expected behavior:
next(2) (reverse comparer)
Actual behavior:
next(1 | -1) (the comparer result is emitted instead)
Additional information:
existing rxjs 4 unit test showing expected behavior: min with comparer non-empty
existing rxjs 5 unit test showing the wrong behavior: should handle a constant predicate on observable with many values - 42 is emitted instead of some value from original observable.
I could work on a PR, but checking first that it's not an horrible mistake on my side, or an on-purpose change.
Thx.

I'll work on one after vacation (2 weeks from now).


possible feature

in your opinion it's possible to add an external monitoring service that send me a notification as soon as the pi goes offline?
sorry for use this section but i don't know where to ask you this
thanks in advance for responding

Yep, I'm thinking about using AWS for this. I will do this at some point in the near future (next 1-2 months), it's definitely the most requested feature so far, so stay tuned!


Updates to Header, Footer and script consolidation

Updates include:

Mobile-friendly navigation with animated hamburger icon
Web components for reusable elements  and  in ReusableComponents.js. This file can be added to any time a reusable element is needed (I would recommend adding in the Slack widget).  A reference to ReusableComponents.js is needed as well for any page that includes a reusable component.
scripts.js added as a global file to import scripts into. If additional global scripts are added, they can be created in separate files (and exported as a js module), and imported into scripts.js. This will prevent the need to add global script tags on every single page of the site.

I converted the slack link to the web component  and updated references to it on all content pages. I also un-hid the dark-mode toggle and moved it down slightly so it wasn't covering the nav link. Updates were made to both the constitution template and the events template.


Can't save files when running in Windows 10 virtual machine

Problem: Cannot save files when editing with brackets in a windows 10 pro virtual machine running in macOS High Sierra 10.13.2 with Parallels.
Steps to Reproduce

clone a git repo of some text files into the windows virtual machine drive.
Open the repo folder on the windows drive with brackets in windows on the virtual machine.
edit file
save file

Expected behavior: File is saved
Actual behavior: Error modal.  "Error Saving File".  See image to see modal.  Main message is "The permission do not allow you to make modifications."
I can save with other text editors just fine.  I have all the permissions for the files.  I even change the properties of Brackets to 'run as administrator', but still no luck.  Anyone have any ideas?

Versions
Brackets Release 1.12 build 1.12.0-17619
OS:
Host OS: macOS High Sierra 10.13.2
Virtual Machine OS: Windows 10 pro, version 10.0.16299
Parallels Desktop Business Edition 13.2.0

Nevermind.
For some reason the 'hidden' attribute (in windows properties dialog) was checked for the files I wasn't able to save.  I believe it's because the the folder was first created (repo cloned) some of the files lacked the .txt extension.  Even after adding the extension, brackets still didn't save the files until I went into properties dialog and unchecked "hidden" for each one I added the extension to.


run docs nightly too

This PR configures cucim docs builds to also run nightly (not just on PR merges only)

/merge

Thanks all! 🙏


Store local tags in localstorage & add a remove and clear button

Currently user provided annotations are not stored. Thats problematic for the users.

Not high prio. User tags can be exported with the graph export. Postpone


remove US3 log collection note

What does this PR do?
Removes the partial used to implement US3 log collection note. This note no longer is applicable.
Motivation
Jira request
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Resolved the conflict, ready for final review!

Thanks @yzhan289! I merged the fix. Can you merge? Looks like I'm blocked on my end.


Blazor server-side project has incorrect version for Microsoft.AspNetCore.Razor.Design

I created a new Blazor server-side project with the latest dev build from myget, and it looks like the Server project isn't populating the version of the Microsoft.AspNetCore.Razor.Design package correctly:
<Project Sdk="Microsoft.NET.Sdk.Web">

  <PropertyGroup>
    <TargetFramework>netcoreapp2.1</TargetFramework>
    <RestoreAdditionalProjectSources>
      https://dotnet.myget.org/f/blazor-dev/api/v3/index.json;
    </RestoreAdditionalProjectSources>
    <LangVersion>7.3</LangVersion>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore.App" />
    <PackageReference Include="Microsoft.AspNetCore.Blazor.Server" Version="0.6.0-preview1-20180924.3" />
    <PackageReference Include="Microsoft.AspNetCore.Razor.Design" Version="$(TemplateRazorDesignPackageVersion)" />
  </ItemGroup>

  <ItemGroup>
    <ProjectReference Include="..\WebApplication8.App\WebApplication8.App.csproj" />
  </ItemGroup>

</Project>

ON IT


Fix potential NPE from HelixDataAccessor

When calling getProperty method from HelixDataAccessor, it could return null.
This PR handles potential NPE for that method in Pinot codebase.

Codecov Report

Merging #4494 into master will decrease coverage by 0.03%.
The diff coverage is 50%.

@@             Coverage Diff              @@
##             master    #4494      +/-   ##
============================================
- Coverage     65.55%   65.52%   -0.04%
  Complexity       20       20
============================================
  Files          1068     1068
  Lines         55468    55481      +13
  Branches       8065     8070       +5
============================================
- Hits          36364    36354      -10
- Misses        16466    16477      +11
- Partials       2638     2650      +12

Impacted Files
Coverage Δ
Complexity Δ

...ntroller/helix/core/PinotHelixResourceManager.java
62.82% <14.28%> (-0.37%)
0 <0> (ø)

...a/org/apache/pinot/common/utils/ServiceStatus.java
74.02% <50%> (-0.98%)
0 <0> (ø)

.../core/realtime/PinotLLCRealtimeSegmentManager.java
70.13% <55.55%> (-0.12%)
0 <0> (ø)

.../pinot/broker/broker/helix/HelixBrokerStarter.java
72.54% <70%> (-0.61%)
0 <0> (ø)

.../apache/pinot/core/transport/DataTableHandler.java
88% <0%> (-12%)
0% <0%> (ø)

...ore/query/scheduler/resources/ResourceManager.java
85.71% <0%> (-10.72%)
0% <0%> (ø)

...e/operator/dociditerators/BitmapDocIdIterator.java
60.71% <0%> (-3.58%)
0% <0%> (ø)

...ache/pinot/common/metadata/ZKMetadataProvider.java
68.13% <0%> (-3.3%)
0% <0%> (ø)

.../org/apache/pinot/transport/netty/NettyServer.java
80.8% <0%> (-3.04%)
0% <0%> (ø)

...e/operator/dociditerators/SortedDocIdIterator.java
61.11% <0%> (-2.78%)
0% <0%> (ø)

... and 22 more

Continue to review full report at Codecov.

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

@jackjlli We should drop this pr right?

Closing this PR since only live instance can be null. Other znode should never be null.
PR to handle NPE from live instance: https://github.com/apache/incubator-pinot/pull/4528


Increase code coverage & psalm level

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Fixed issues
#33, #34

@kamarton Hi! Could you help with TrustedHostsNetworkResolver middleware coverage? Almost done, but the case of "confusing hosts" remains.


What is Self-Supervised Learning?

Be sure to visit our Resources Page for tools, resources, and example articles to go over.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
We're looking for the first 2-3 paragraphs of the article that appropriately summarize what your article will be about.
NOTE: We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
Proposed title of article
Your title should be descriptive of the article/tutorial. Be Specific. Use keyword research to gain your article higher ranking.
What is Self-Supervised Learning?
Introduction paragraph (2-3 paragraphs):
How do humans learn so quickly?
Humans learn by using both supervised learning and unsupervised learning. We are multifaceted. This multifaced approach to learning has been so easy for humans. But, that has not been the case for deep learning systems. Though we have seen robust deep learning systems that have performed image recognition and natural language processing tasks, performing complex tasks has remained challenging. This is the problem self-supervised learning is trying to address.
Key takeaways:
What are the 3-5 most important things the reader should understand or be able to do after reading this article? Use this area to get your ideas down on the bulk of your article or tutorial.
By the end of this article, the reader should be able to understand:

What is Supervised Learning?
The challenges associated with Supervised Learning
Introduction to Self-Supervised Learning
Applications of Self-Supervised Learning

References:
Please list links to any published content/research that you intend to use to support/guide this article.

A Simple Framework for Contrastive Learning of Visual Representations
Unsupervised Representation Learning by Predicting Image Rotations
Generative Pre-trained Transformer 3 (GPT-3)
Bidirectional Encoder Representations from Transformers (BERT)

Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Thank you for your topic suggestion @Bayler. Can you please confirm that there is not too much overlap with this previously published article? https://www.section.io/engineering-education/supervised-learning-algorithms/

@ninjaginja the article just touches a bit on supervised learning as it is essential to explaining it's challenges but the topic solely focuses on self supervised learning

@ninjaginja Hi, is my topic approved?

nice topic - approved


restic: enable darwin builds

Motivation for this change
I want to use restic on OS X :)
Things done
Please check what applies. Note that these are not hard requirements but merely serve as information for reviewers.

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

[ ] NixOS
[x] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

@mbrgm Is it usual that this messages come?
post-installation fixup
stripping (with flags -S) in /nix/store/p7g7ngdg9dfkgqs7gdh3v6zcwmw9xl90-restic-0.7.1-bin/bin
patching script interpreter paths in /nix/store/p7g7ngdg9dfkgqs7gdh3v6zcwmw9xl90-restic-0.7.1-bin
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
find: '/nix/store/z8qwsca338na6f827jkc71z1npzggflb-restic-0.7.1': No such file or directory
building path(s) ‘/nix/store/009v53dbhv88hfg0pp7p60chma3zw5bm-user-environment’
created 6 symlinks in user environment

@davidak Nope, not seeing any of those when building on Linux.


Could you please support placeholder image?

Hi,
This is amazing image lib for my react native app. It's will be more awsome if you support placeholder image from local when loading image from internet by uri.
Have you plan to implement it?
Thanks

Yes, specially since Glide and SDWebImage both support them natively.

Thank you support @hhunaid.
I have try to set placeholder for default request in GlideModule but It doesn't work.

Is there any api new for setting placeholder image before the reomte source has been downloaded?

Is there any api new for setting placeholder image before the reomte source has been downloaded?

Is there any update for supporting placeholder images. Or if anyone has found something in forked repo , please update here. Need it urgently in efficient way

Any update on this?

This PR adds placeholder prop https://github.com/DylanVann/react-native-fast-image/pull/409

Merged!


Issue #144 The change password servlet operation is unsafe

Please consider the changes from this pull request to resolve issue #144

Thanks for Your contribution.


Add engine API call to set testing/validation data

This is tricky because right now i'm expecting a range of rows to be considered "the testing set" or "the validation set". If the user wishes to upload separate files for training/testing, it would be super helpful if the UI could merge those sets when sending them to the engine, then just specify which rows are to be considered testing and which training. This would vastly reduce the amount of work on my end.
Call should look like:
{
  type: 'setPartition',
  data: {
    label: 'fit' | 'test' | 'validation' | (other)
    start: number  // index of row to begin at
    end: number    // index of row to end at (exclusive)
    ratio: 0 <= number <= 1    // if this is specified, start & end will be ignored
  }
}

Done. c4cc5f151468bd6eec5a46732400b6339ff4ae83


Added Json File for LinkFree

Json File added for Sanjay

All the changes requested has been completed and requested new PR

Made the necessary changes.

@sanjay-kv still 1 comment remaining https://github.com/EddieHubCommunity/LinkFree/pull/408#pullrequestreview-769474595 then we can merge

Why was this closed?


Change insert node output

This is suggested as an extension to #36.
We might consider to change the node output to not contain raw requests, but an aggregate of the msg.ack field.
Suggestion:

Output 1: the aggregated content of msg.ack for data updates on success.
Output 2: the aggregated content of msg.ack for meta data updates on success.

I.e. consider the following messages:
{
  _msgid: 1,
  ack: ["a1", "a1-2"],
  payload: {...}
}
{
  _msgid: 2,
  // ack undefined
  payload: {...},
  signal: {...}
}
{
  _msgid: 3,
  ack: ["a3"],
  payload: {...},
  signal: {...}
}
{
  _msgid: 4,
  ack: ["a4"],
  signal: {...}
}

The aggregated output 1 on success is:

{msgs: [1,2,3], ack:["a1","a1-2","a3"]} // all messages with data.

The aggregated output 2 on success is:

{msgs:[3,4], ack: ["a3", "a4"]} // all messages with meta data.

Errors, are logged, while the detailed response is discarded both on success and on error.

We should definitively adapt, evolve this.
There seam to be some "ack" nodes in node-red:

https://flows.nodered.org/node/node-red-contrib-ack
https://flows.nodered.org/node/node-red-contrib-amqp-ack

We could have a look at what they are doning, and which message fields they are inspecting.

Somehow related (if splitting a message!)

https://nodered.org/docs/user-guide/messages#message-sequences

I think it would be OK to ignore split messages (sequences) in the initial version. That is, for a split message to be acked correctly, it must be joined before it's passed to the input node.


fix:  fall detection issues #279

Hello @bhavikapanara! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file src/ambianic/pipeline/ai/fall_detect.py:

Line 246:80: E501 line too long (99 > 79 characters)
Line 247:80: E501 line too long (114 > 79 characters)
Line 248:80: E501 line too long (117 > 79 characters)
Line 250:37: E222 multiple spaces after operator
Line 251:80: E501 line too long (82 > 79 characters)
Line 254:38: E222 multiple spaces after operator
Line 255:80: E501 line too long (84 > 79 characters)

In the file src/ambianic/pipeline/ai/image_boundingBox_detection.py:

Line 113:80: E501 line too long (86 > 79 characters)
Line 117:80: E501 line too long (84 > 79 characters)

In the file src/ambianic/pipeline/ai/pose_engine.py:

Line 126:80: E501 line too long (90 > 79 characters)
Line 129:80: E501 line too long (98 > 79 characters)

In the file src/ambianic/pipeline/ai/tf_detect.py:

Line 140:80: E501 line too long (80 > 79 characters)
Line 159:80: E501 line too long (86 > 79 characters)
Line 163:80: E501 line too long (84 > 79 characters)
Line 167:5: E303 too many blank lines (2)
Line 180:34: W292 no newline at end of file

Hello @bhavikapanara! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file src/ambianic/pipeline/ai/fall_detect.py:

Line 246:80: E501 line too long (99 > 79 characters)
Line 247:80: E501 line too long (114 > 79 characters)
Line 248:80: E501 line too long (117 > 79 characters)
Line 250:37: E222 multiple spaces after operator
Line 251:80: E501 line too long (82 > 79 characters)
Line 254:38: E222 multiple spaces after operator
Line 255:80: E501 line too long (84 > 79 characters)

In the file src/ambianic/pipeline/ai/image_boundingBox_detection.py:

Line 113:80: E501 line too long (86 > 79 characters)
Line 117:80: E501 line too long (84 > 79 characters)

In the file src/ambianic/pipeline/ai/pose_engine.py:

Line 126:80: E501 line too long (90 > 79 characters)
Line 129:80: E501 line too long (98 > 79 characters)

In the file src/ambianic/pipeline/ai/tf_detect.py:

Line 140:80: E501 line too long (80 > 79 characters)
Line 159:80: E501 line too long (86 > 79 characters)
Line 163:80: E501 line too long (84 > 79 characters)
Line 167:5: E303 too many blank lines (2)
Line 180:34: W292 no newline at end of file

@bhavikapanara please use the user provided images that I uploaded to this PR for regression tests. Otherwise we may be making one step forward and two back.

@bhavikapanara please use the user provided images that I uploaded to this PR for regression tests. Otherwise we may be making one step forward and two back.

@bhavikapanara could you please review the pep8speaks alerts?

@bhavikapanara could you please review the pep8speaks alerts?

I don’t have the original, but you can crop it with an editor. The image
size is the thumbnail which is bigger than or equal to the input tensor so
inference should work fine.
On Wed, Jan 13, 2021 at 4:44 AM bhavika panara<EMAIL_ADDRESS>wrote:

@bhavikapanara commented on this pull request.
In src/ambianic/pipeline/ai/fall_detect.py
https://github.com/ambianic/ambianic-edge/pull/280#discussion_r556426124
:

@@ -243,11 +243,18 @@ def fall_detect(self, image=None):
pose_dix = {}
inference_result = []

           for e_corr in self.fall_detect_corr:

               if pose.keypoints[e_corr].score > 0.6:

                   # detected key-point probability must be greater than 0.6

                   pose_dix[e_corr] = pose.keypoints[e_corr].yx

           # Find leftVectorScore and rightVectorScore --> i.e average score of shoulder & hip

           leftVectorScore = (pose.keypoints['left shoulder'].score + pose.keypoints['left hip'].score) / 2.0

@ivelin https://github.com/ivelin Can you please share only original
person image, not screenshot?
I need to feed similar kind of images as we have already used for other
tests like
[image: fall_img_1]
https://user-images.githubusercontent.com/19815572/104441763-1fc1ba80-55ba-11eb-865b-72b46b498d30.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ambianic/ambianic-edge/pull/280#discussion_r556426124,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARBUFI24IRSZPC4TISAO6LSZV2QJANCNFSM4V7GOLUA
.

I don’t have the original, but you can crop it with an editor. The image
size is the thumbnail which is bigger than or equal to the input tensor so
inference should work fine.
On Wed, Jan 13, 2021 at 4:44 AM bhavika panara<EMAIL_ADDRESS>wrote:

@bhavikapanara commented on this pull request.
In src/ambianic/pipeline/ai/fall_detect.py
https://github.com/ambianic/ambianic-edge/pull/280#discussion_r556426124
:

@@ -243,11 +243,18 @@ def fall_detect(self, image=None):
pose_dix = {}
inference_result = []

           for e_corr in self.fall_detect_corr:

               if pose.keypoints[e_corr].score > 0.6:

                   # detected key-point probability must be greater than 0.6

                   pose_dix[e_corr] = pose.keypoints[e_corr].yx

           # Find leftVectorScore and rightVectorScore --> i.e average score of shoulder & hip

           leftVectorScore = (pose.keypoints['left shoulder'].score + pose.keypoints['left hip'].score) / 2.0

@ivelin https://github.com/ivelin Can you please share only original
person image, not screenshot?
I need to feed similar kind of images as we have already used for other
tests like
[image: fall_img_1]
https://user-images.githubusercontent.com/19815572/104441763-1fc1ba80-55ba-11eb-865b-72b46b498d30.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ambianic/ambianic-edge/pull/280#discussion_r556426124,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARBUFI24IRSZPC4TISAO6LSZV2QJANCNFSM4V7GOLUA
.

@bhavikapanara please use the user provided images that I uploaded to this PR for regression tests. Otherwise we may be making one step forward and two back.

Added test case for the above images.

@bhavikapanara please use the user provided images that I uploaded to this PR for regression tests. Otherwise we may be making one step forward and two back.

Added test case for the above images.

Good work. I only have one more comment, which I should have made earlier, but didn't spot the redundancy until now.
Since we are on different time zones, please feel free to address the last comment and continue on to #282 and #283.

I have already added attached screenshots of #283  in our test_fall_detect.py. And currently working on the #282 .

Good work. I only have one more comment, which I should have made earlier, but didn't spot the redundancy until now.
Since we are on different time zones, please feel free to address the last comment and continue on to #282 and #283.

I have already added attached screenshots of #283  in our test_fall_detect.py. And currently working on the #282 .


determineFontProperties unimplemented on devkit native

This function is unimplemented fully as of now when using a devkit native build. This can cause text to not render on native. The existing non-native implementation should work in theory if this issue is resolved.
The code in question can be found here.

This aspect is now implemented, though the implementation is not adequate.


.FLA files for Retro, Sakuroma, etc. aren't provided.

Not necessarily a bad thing or anything, but I'd be nice to have them publicly released.

I can just imagine the things people would do if they got a hold of saku's files...


[FEATURE] Create hyperlinks in certifications instead of showing distinct redirection links

Currently if I want to link my certifications to my resume, it shows the links distinctly below the certificate description. It wastes a lot of space and looks so cluttery, hence I can not link them.
How about the certification titles themselves become the hyperlinks so that if anyone clicks on them from the pdf, they get redirected to the certification link? It will save a lot of space and will do the job too.
I understand that it might not be so obvious to the reader that those titles are hyperlinks just by the looks so one might not think of clicking on them. So maybe we can colour them differently so they look like hyperlinks or just give the user option to decide if they want hyperlinks or distinct links showing beneath the cert description.

I would love this. I have added a few certifications and went through the pain of gathering all links only to see they look hideous due to their length. :-(
Does anyone have a link to documentation how/where to possibly change the template by myself until this feature is somehow implemented?
For now, I guess I have to remove the links.

I am using markdown to put hyperlinks. This is not the best thing but it works just fine until it is implemented natively. You cannot use markdown in the link section in rxresu so you will have to use it in the body itself tho.

I am using markdown to put hyperlinks. This is not the best thing but it works just fine until it is implemented natively. You cannot use markdown in the link section in rxresu so you will have to use it in the body itself tho.

Thanks! That is a great workaround and will work for me for now.


Selector preview prevent me from clicking on “Accept Cookie” button

Hello:
I have used this tools for a few years now, it worked well.
However, today, I have to write some PuppeteerSharp code to visit one web page, for this web page, there is one button called “Accept” cookie at the middle- bottom part of the page, which you have to click on it, in order to go further.
For old version of this tools, it should be no issue at all.  But the current version 1.0.0 has something, I think it is called “Selector Preview”, which I can see the preview of the element the mouse hovers; but the biggest issue is: this “Selector Preview” just covers the “Accept” cookie button, and I can’t figure out how to move this “Selector Preview” to somewhere else, therefore, I can’t click on the “Accept” cookie button.
Please let me able to configure where to show the “Selector Preview” on the whole screen area, but not fix it in one specific area.  You can imagine that some specific button, like those “Accept Cookie” buttons will usually be placed at the bottom part of one page, and if I can’t move the “Selector Preview” to somewhere else, how I can click on the “Accept Cookie” button?
Hope you understand.

Already working on a toggle feature to solve this.
Duplicate of #144


this midi seems cant play properly on secret sauce

https://github.com/stuerp/foo_midi/assets/11797565/cfccb9cd-75c5-4a59-8e61-7b144cac3714
same midi on bassmidi;
https://github.com/stuerp/foo_midi/assets/11797565/e54c1dac-623d-4057-ba5b-6db1ef413ed9
midi:
DKC3-GBA_RockfaceRumble_v1-1.zip
on x86 1.6.16, tried 2.0 but seems same results

I don't hear any problem with BASS MIDI nor with Secret Sauce on my setup. What exactly do you find wrong with that rendering?

harmonica is played as piano in secret sauce

Secret Sauce uses the GS instruments. This is a GM file.

hmm
gm  flavor didnt help either, and
魔王魂  村05.zip
this seems plays fine though, assuming this is gm

Both are GM and neither of them sends a GM Reset so I assume Secret Sauce defaults to the GS set.

hmm gm flavor didnt help either, and 魔王魂 村05.zip this seems plays fine though, assuming this is gm

GM Flavor was a no-op. Don't know why. I re-enabled it but it does not change the result. Bank 0 is selected before playback starts. I'll have to dive into the byte stream. I'm intrigued.

I can't find anything weird in the MIDI stream.


Write documentation for classes and methods

Hi!
This package is awesome and really great, I need it and I enjoy a lot to use it, so I want to do my bit! I started to write documentation (dartdoc comments) for classes and methods of the package.
I leave this PR as a draft until I think it is enough or maybe everything it's documented! If I see that it's going to take a long time, I will remove the draft status of this PR and in the future I will work in another one.
Please, review the documentation and leave comments if it is necessary, because the best documentation is the one that it is reviewed for more than one! ❤
I will always try to write the docs based on the recommendation of Effective Dart: Documentation, to be clear but also concise about what it is necessary.

About the first line
How about something like this?
An abstract class that must be implemented in a higher layer of software, which must have at least the following methods

I thought about using "abstract class" but I feel a bit redundant because in the generated documentation the signature already indicates it is an abstract class.
Now it is really late but soon I will review your comment and rethinking the abstract class documentation.

As a note, as this PR is going to be a bit huge, it could be really useful if we make the reviews per line using the review system and let the comments for general reviews, in that way we could have a more organized environment with threads per issues/doubts/improvements in the contributions.
What do you think?


Enable ExtensionController in ActiveGate

Description
see K8S-10118
ActiveGate is also created when Extensions are enabled in the DynaKube. No other AG capabilities are needed.
How can this be tested?

Unittests.
Apply dynakubes below. ActiveGate pod should be always created.
Check enabled capabilities on the AG pod using:

kubectl -n dynatrace get -o yaml pod/dynakube-activegate-0 -o jsonpath="{.spec.containers[0].env[0]}"

a)
dynakube:
spec:
  activeGate:
    capabilities:
      - routing
    replicas: 1
  extensions:
    prometheus:
      enabled: true
  oneAgent:
    cloudNativeFullStack: {}

capabilities:
{"name":"DT_CAPABILITIES","value":"MSGrouter,extension_controller"}

b)
dynakube:
spec:
  activeGate:
    replicas: 1
  extensions:
    prometheus:
      enabled: true
  oneAgent:
    cloudNativeFullStack: {}

capabilities:
{"name":"DT_CAPABILITIES","value":"extension_controller"}

c)
dynakube:
spec:
  extensions:
    prometheus:
      enabled: true
  oneAgent:
    cloudNativeFullStack: {}

capabilities:
{"name":"DT_CAPABILITIES","value":"extension_controller"}

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 26.66667% with 11 lines in your changes missing coverage. Please review.

Project coverage is 57.66%. Comparing base (0b2c1f9) to head (ec57a27).

Files
Patch %
Lines

pkg/api/v1beta3/dynakube/properties.go
0.00%
6 Missing :warning:

...llers/dynakube/activegate/capability/capability.go
28.57%
3 Missing and 2 partials :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3468      +/-   ##
==========================================
- Coverage   57.68%   57.66%   -0.02%
==========================================
  Files         354      354
  Lines       20006    20012       +6
==========================================
  Hits        11540    11540
- Misses       7226     7230       +4
- Partials     1240     1242       +2

Flag
Coverage Δ

unittests
57.66% <26.66%> (-0.02%)
:arrow_down:

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

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


Better representation model based on OpenAssetIO

Changelog Description
Using OpenAssetIO to replace represenation model in the future.
Additional info
This is adding OpenAssetIO framework to be used to resolve and publish data imbued with Traits and Specifications to handle
data previously impossible to get from simple representation model. It is not replacing immediately all legacy behaviour. It is adding it's own plugins so workflows can be slowly transitioned with backwards compatibility in mind.
Changes
This is adding support for running unit test directly in the addon context. For that, it provides repository level pyproject.toml with dependencies needed to run the tests. It is also including tests folder.
There is also root folder scripts with generate_traits.ps helper script to help to generate Traits in ayon_core.pipeline.traits.generated from YAML files.
This directory contains the traits and specifications for the OpenAssetIO
used across the pipeline.
They are defined in YAML files and OpenAssetIo-traitgen tool is used for generating the code from the yml files. This code is placed in the generated directory along with the __init__.py file that allows importing traits wherever needed.
You can use these traits in pipeline by importing them and using them like so:
from ayon_core.pipeline.traits.generated.openassetio_mediacreation import traits as mediacreation_traits
from openassetio.trait import TraitsData

data = TraitsData()
frame_ranged = mediacreation_traits.timeDomain.FrameRangedTrait(data)
frame_ranged.setStartFrame(1001)
frame_ranged.setEndFrame(1025)
frame_ranged.setFramesPerSecond(24.0)
frame_ranged.setStep(1)

data.hasTrait(mediacreation_traits.timeDomain.FrameRangedTrait.kId)  # True

# ImageTrait has no properties, it is just a "family" trait
mediacreation_traits.twoDimensional.ImageTrait(data)

pixel_based = mediacreation_traits.twoDimensional.PixelBasedTrait(data)
pixel_based.setDisplayWindowHeight(1080)
pixel_based.setDisplayWindowWidth(1920)
pixel_based.setPixelAspectRatio(1.0)

data.hasTrait(mediacreation_traits.timeDomain.FrameRangedTrait.kId)  # True
data.hasTrait(mediacreation_traits.twoDimensional.ImageTrait.kId)  # True
data.hasTrait(mediacreation_traits.twoDimensional.PixelBasedTrait.kId)  # True
data.hasTrait(mediacreation_traits.color.OCIOColorManagedTrait.kId)  # False

You can regenerate those traits from YML files by running the script
from /scrits/generate_traits.ps1.
It is also adding host.yml file to ayon_core that is needed by OpenAssetIO host interface configuration.
To use OpenAssetIO manager, you need to run code like this:
from ayon_core.pipeline import get_openassetio_manager

manager = get_openassetio_manager()
manager.resolve(...)

[!IMPORTANT]
Be aware, that getting manager is also initializing openassetio host and that can later on take some time. It would be better later on to do it once (per DCC host?)

[!TIP]
More description coming

Closing this on slightly behalf of different approach:

[ ] https://github.com/ynput/ayon-core/issues/909
[ ] https://github.com/ynput/ayon-core/issues/911
[ ] https://github.com/ynput/ayon-core/issues/910


Docs work for V3

This PR adds full documentation to the website for Meta-World
(built on #499 so lots of changes)

@frankroeder
Can you update us on what needs to be done?

@frankroeder Can you update us on what needs to be done?

@reginald-mclean Within 2-3 days, I will push everything that I have and list what still needs to be done.

@reginald-mclean, I have now incorporated all the information I could gather from the paper by Yu et al. and some of the suggestions you provided regarding the reward function variations. My next step will be to dive deeper into the code to verify and extend the documentation with anything that is missing.
It would be great if you could assist me with the section on the state space and provide a snippet for the current initialization of the parametric task variations. Please let me know which branch I should base the documentation updates on.


Update run-hop-in-apache-airflow.adoc

on the front end the underscores being changed don't show up for the first 2 instances but do for the 3rd.
Turns out the variable had double underscores on the first 2 instances and not on the 3rd (which displays correctly on the front end)
Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Run mvn clean install apache-rat:check to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[x] If you have a group of commits related to the same change, please squash your commits into one and force push your branch using git rebase -i.
[x] Mention the appropriate issue in your description (for example: addresses #123), if applicable.

To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

[x] I hereby declare this contribution to be licensed under the Apache License Version 2.0, January 2004
[x] In any other case, please file an Apache Individual Contributor License Agreement.

thanks a lot for the PR, Diego. the double underscores are what Airflow expects, so can't be reduced to a single one...
closing the PR, but appreciate the contribution nevertheless!


[#119379405] Fix bootstrap pipelines

What
In my enthusiasm I have forgotten to define outputs for mostly TF tasks.
That is, I let TF run directly on file that is in the input directory and
update it in place. That works fine, but if you try to 'put' that file
later, that is a different container and it doesn't have access to that
file. So you need to explicitly define it as an output. Fix the pipelines to
do so.
How to review
Deploy a completely new environment from scratch. You can e.g. pick a name that's not used by anyone. All should work fine.
Who can review
not @mtekel
that can't review, because they worked on it.

It looks good. I will let it finish deploying my environment and if it works fine, merge.


Usage about nerf-player model

Hi, thanks for your effort to implement nerfplayer into nerf-studio. But I failed to run nerf-player on DNeRF dataset. Does anyone know which Dynamic scene should I run with nerfplayer and any runnable commands would be appreciate.
The error message is as fllows
[08:15:42] Saving checkpoints to: outputs/unnamed/nerfplayer-ngp/2023-03-14_081541/nerfstudio_models      trainer.py:128
Traceback (most recent call last):
File "/userhome/wangbingxuan/anaconda3/envs/nerfstudio/bin/ns-train", line 8, in
sys.exit(entrypoint())
File "/userhome/wangbingxuan/code/nerfstudio/scripts/train.py", line 247, in entrypoint
main(
File "/userhome/wangbingxuan/code/nerfstudio/scripts/train.py", line 233, in main
launch(
File "/userhome/wangbingxuan/code/nerfstudio/scripts/train.py", line 172, in launch
main_func(local_rank=0, world_size=world_size, config=config)
File "/userhome/wangbingxuan/code/nerfstudio/scripts/train.py", line 86, in train_loop
trainer.setup()
File "/userhome/wangbingxuan/code/nerfstudio/nerfstudio/engine/trainer.py", line 141, in setup
self.pipeline = self.config.pipeline.setup(
File "/userhome/wangbingxuan/code/nerfstudio/nerfstudio/configs/base_config.py", line 59, in setup
return self._target(self, **kwargs)
File "/userhome/wangbingxuan/code/nerfstudio/nerfstudio/pipelines/dynamic_batch.py", line 57, in init
super().init(config, device, test_mode, world_size, local_rank)
File "/userhome/wangbingxuan/code/nerfstudio/nerfstudio/pipelines/base_pipeline.py", line 231, in init
self.datamanager: VanillaDataManager = config.datamanager.setup(
File "/userhome/wangbingxuan/code/nerfstudio/nerfstudio/configs/base_config.py", line 59, in setup
return self._target(self, **kwargs)
File "/userhome/wangbingxuan/code/nerfstudio/nerfstudio/data/datamanagers/base_datamanager.py", line 384, in init
self.train_dataset = self.create_train_dataset()
File "/userhome/wangbingxuan/code/nerfstudio/nerfstudio/data/datamanagers/depth_datamanager.py", line 41, in create_train_dataset
return DepthDataset(
File "/userhome/wangbingxuan/code/nerfstudio/nerfstudio/data/datasets/depth_dataset.py", line 36, in init
assert (
AssertionError

Hi @DOGEwbx , I'm having the same error, did you find a way to run the training?


Add instructions for using when embedded in Spring Boot

This plugin gets extensively used when Camunda
is embedded into the Spring Boot application.
Although the project has existed for quite some time, new
users are still struggling to use it with Spring Boot since
searching forums and GitHub issues is not the optimal way.
This commit adds instructions for new Spring Boot users
so that they can quickly use the plugin, addressing
GitHub issues #30, #24, #15, #8, and #7.

Thanks. I was expecting Camunda to provide some generic UI plugin documentation, which they never did.


refactor: remove dead code

The dispatcher doesn't use the history foldable because this code was reimplemented in the authority-claimer service.

This works, but you can also just change the "root" foldable from RollupState to InputBox. (Then, if you want, you could delete RollupsState and History.)
The RollupsState is just this aggregate foldable. I recommend not using it, and instead setting the InputBox as your foldable.
How can I help you? I can do it if you prefer.


Change "Asset manager" to "Assets"

Change

[x] "Asset manager" to "Assets" (in the nav)
[x] "/account/asset-manager" to "/account/assets"
[x] "Your hosted assets" to "Assets" in the page title

RS additions:

[x] Upload feedback if the user doesn't have the feature
[x] Investigate possibility of moving the cursor as the mouse position changes to allow for accurate position DnD

@remy is there an ETA on the asset management stuff?

Have you got a pro account, if you do, just turn on experiments (in your profile) and assets will work.
I'll likely release to stable in the next week or so.

@remy very cool. I'm going to take a look at this to see if we can switch our Polymer samples from Plunker to JSBin Pro.


changing hover effect of the submit button in contact us section

Improving the hover effect of the submit button in contact us section
please assign me this issue

@subrato3 contact us needs to be worked on as whole


Translate Part 8 into French

Any chance I could get an update on this?

I'm currently studying for my exams, gonna do it afterwards :)

No worries, feel free to contribute if/when you have the time.


Add accordion for groups founded  (from 1992)

Problem Description:
A group that was founded by a person is not linked on the person's page.
Expected Behavior/Solution:
Add an accordion to show groups that were founded by a person on the person page.
Requirements:

[x] Add config

Needed for promotion:
If an item on the list is not needed, it should be crossed off but not removed.

[x] Wireframe/Mockup - Heather
[ ] ~Committee discussions - Sarah~
[ ] ~Feasibility/Team discussion - Sarah~
[x] Backend requirements - Peter is creating a HAL link per the linked ML issue below
[x] Frontend requirements - Kameryn
[ ] ~Questions~

UAT/LUX Examples:

Steve Wozniak

Dependencies/Blocks:

 HAL Link - Agent Founded By Agent (from 123) #16
Old Ticket - 123

Related Github Issues:
N/A
Related links:
Bugherd: https://www.bugherd.com/projects/284041/tasks/1869
Wireframe/Mockup:

Approved per UAT

Looks good in prod, closing.


ingress/controllers/README.md: Fix a link

This change is

CNCF CLA signed.
On 28 September 2016 at 16:38, Kubernetes Submit Queue
<EMAIL_ADDRESS>wrote:

[CLA-PING] @gg7
Thanks for your pull request. It looks like this may be your first contribution to a CNCF open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
Please visit https://identity.linuxfoundation.org/projects/cncf to sign.
Once you've signed, please reply here (e.g. "I signed it!") and we'll verify. Thanks.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
If you signed the CLA as a corporation, please sign in with your organization's credentials at https://identity.linuxfoundation.org/projects/cncf to be authorized.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

thanks, lgtm


Run VOC/VOI scraping every day

Current VOC/VOI scraping jobs run once in the pipeline – they should be configured to run every day.

Consolidate all 5 VOC rules into one rule
This allows to remove all input dependencies for this VOC rule
As one output, have a dummy daily .done file (output in the status folder) so that the rule triggers every day


Deposit contract and genesis handling

Creating of beacon chain genesis state is still WIP as I'm missing some new types and functions but it should be solved with #318

Spec Tests really don't like this pr, otherwise LGTM

LGTM, FYI, I'm working off these changes to bring the types / state transition up to spec


Regression  lvm2: 2.03.23 -> 2.03.25 #312314

Describe the bug
After https://github.com/NixOS/nixpkgs/commit/8e06d10fb6909d12684be5cac7007b264e6ca671 , my systems with LVM2 on crypt on LVM2 on LVM2 no longer boot.
init times out waiting for disks to be online without ever reaching systemd-cryptsetup unlock prompt.
Steps To Reproduce
Have a system with LVM2 on crypt on LVM2 on LVM2. The nested lvm2 will require scan_lvs = 1 in initrd.
Example https://github.com/gauravjuvekar/nixos_common/blob/670b4c39255194a737beb670b481bd084b19e1eb/common/base_boot.nix#L27
Systemd waits for the crypt physical partition to be up and times out after 90s during boot at nixos-unstable https://github.com/NixOS/nixpkgs/commit/e2f08f4d8b3ecb5cf5c9fd9cb2d53bb3c71807da
Reverting https://github.com/NixOS/nixpkgs/commit/8e06d10fb6909d12684be5cac7007b264e6ca671 fixes this showing the unlock prompt during boot.
https://github.com/gauravjuvekar/nixpkgs/commit/de8a1174e38f4c9d7a7e4fcc095933bcab7bec1c works correctly.
Notify maintainers
@ajs124
@raskin
@helsinki-systems
@alyssais
@7c6f434c

Metadata
[gaurav@dt:/nix/var/nix/profiles]$ nix-shell -p nix-info --run "nix-info -m"
this path will be fetched (0.02 MiB download, 0.06 MiB unpacked):
  /nix/store/91d27rjqlhkzx7mhzxrir1jcr40nyc7p-stdenv-linux
copying path '/nix/store/91d27rjqlhkzx7mhzxrir1jcr40nyc7p-stdenv-linux' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 6.11.2, NixOS, 24.11 (Vicuna), 24.11.20241006.de8a117`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.24.8`
 - nixpkgs: `/nix/store/nn8y7pkarzhb5zkzx1cy0src0d1rr2d6-source`

Add a :+1: reaction to issues you find important.

This was a udev change in upstream https://gitlab.com/lvmteam/lvm2/-/commit/ee73875ff0e2e4f93d2acf75e7ec33b216f84c97


"Șotânga" is listed in the drop-down menu

"Șotânga" is listed in the drop-down menu alongside its corresponding county according to the law
Precondition:
The website is up an running.

Step 1
Type "Șotânga" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Șotânga, DÂMBOVIȚA".


Confusing Home Page UI

India is placed on top of the stats for states, on the right, while the numbers for the whole of India are on the left.

solved by someone, by adding 'India Map' instead


Cuelist crossfade briefly dips in brightness at start of crossfade

Describe the bug
When using a cuelist with infinite hold scenes, and the crossfade slider function, the brightness of the scenes being crossfaded briefly dips as the crossfade starts.
To Reproduce
Steps to reproduce the behavior:

Open QLC+ with a blank workspace.
Add a single Generic fixture.
Create a scene called "full on": fixture set to 100%
Create a scene called "off": fixture set to 0%
Create a chaser with the two scenes in the order above, with the hold set to infinite.
Go to Virtual Console and add a cuelist. In the properties, use the chaser from step 5, and set the side fader to Crossfade.
Switch to Operate mode.
Open Fixture monitor.
Click the play button on the cuelist.
The fixture monitor should show the DMX value at 255.
With the crossfade slider selected, press the DOWN arrow on your keyboard to nudge the slider down. (you can also drag with mouse or use an external input, but the down arrow is an easy way to consistently show the problem)
As you press DOWN, observe the DMX output, you'll see that very very briefly the value will drop to 037, and then immediately back to 252.

Expected behavior
The crossfade should be a smooth transition between the two scenes, but instead you have this this "blink" as you begin to crossfade. It doesn't look like much of a problem on the screen, but when you control multiple fixtures with the scene you can see quite a noticable "blink" in the lighting.
Screenshots
This problem is difficult to capture on a screenshot, so I recorded a video of it happening using my phone pointing at the computer monitor in slow motion. I then recorded my phone screen while I slowly played it back to make it even slower. I have uploaded the result of this to YouTube as an unlisted video which shows the problem:
https://youtu.be/PtYDBmUBcbo
You can see the problem happen at 7 seconds into the video. Watch it closely, it drops to 037 and then quickly returns to 252.
Desktop (please complete the following information):

OS: Window 11
QLC+ Version: 4.13.1
High spec laptop.

I've confirmed this issue on Window 11, 4.13.1 GIT. Very Strange.

I just tested in Ubuntu 22.04 with 4.13.2-GIT works as expected.
Missatge de Lachlan Hicks @.***> del dia dg., 22 de
set. 2024 a les 3:28:

I've confirmed this issue on Window 11, 4.13.1 GIT. Very Strange.
—
Reply to this email directly, view it on GitHub
https://github.com/mcallegari/qlcplus/issues/1596#issuecomment-2365402453,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABO5H423KLCQXKS5RGPZ2E3ZXYMLHAVCNFSM6AAAAABKRGESDWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRVGQYDENBVGM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I can confirm this issue with 4.13.1 on Fedora 41.
It takes indeed a camera to spot the 037.

Is this fixed with latest sources or not?

I cannot reproduce with the latest git sources. But I also did not find the fix in the git log.


Update OSDisk.yml

This documentation file for OSDisk contains Unicode characters that don't seem to belong there.
I can't make sense of the original intent, so I'm making this pr to remove them.

Docs Build status updates of commit ed056a0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

<EMAIL_ADDRESS>:white_check_mark:Succeeded
View (azure-node-latest)

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

Since there is no movement on this I will close the PR


Add support for CP_LOAD_STATE6_* packets

This includes showing not only the packet contents, but also any constant data and V#/T#/S# data
Missing support for bindless (no test case!) and showing shader disassembly (probably shouldn't be part of the packet disassembly)

Screenshots, 1st showing constant data, the 2nd one showing UBOs:

Screenshots, 1st showing constant data, the 2nd one showing UBOs:

in the 2nd screenshot, each elem in the array is a ubo (with address and size)? where there are some ones with size of 0x0?

Screenshots, 1st showing constant data, the 2nd one showing UBOs:

in the 2nd screenshot, each elem in the array is a ubo (with address and size)? where there are some ones with size of 0x0?

My (limited) understanding is that it's tied in with how the descriptor sets are setup. So if a descriptor set has a valid V# on the 0th, 2nd, and 3rd entry but none in the 1st entry, then the one in the 1st entry is usually not set. However, CP_LOAD_STATE6_* sets an array of them, which means if they want to setup 3 non-consecutive V#s, then they need to pad some of the in-between V#s they may not care about. This is more of a semi-educated guess.

This includes showing not only the packet contents, but also any constant data and V#/T#/S# data Missing support for bindless (no test case!) and showing shader disassembly (probably shouldn't be part of the packet disassembly)

more general question, does adreno keeps a heap that has all descriptors (V#/T#/S#) and use CP_LOAD_STATE6_ to indicate which descriptor is used by current draw/dispatch?


fix(input): unable to focus input in IE 11

Fixes being unable to focus an input in IE 11 by clicking on the label. This was due to the style for hiding the placeholder which, for some reason, ended up disabling the input altogether if it didn't have a value.

LGTM