feat: minor topological improvements

Prove that a set is a Gdelta if and only if it is an intersection of open sets indexed by ℕ.
Cleanup porting todos in the Gdelta file
Rename cluster_point_of_compact to exists_clusterPt_of_compactSpace
Generalize the class T2Space to T2OrLocallyCompactRegularSpace in the file Support.lean

Prerequisite of #9909

Thanks!
bors d+


Convert from YAML to JSON

Right now I use YAML for everything. While this works great, I think I should consider using JSON instead (at least for the generated files).
This should make it easier for other scripts outside of this tool to modify files if that's ever needed.

I do expect developers to read these entries in git, so I would like to make it prettified JSON if possible.
One thing I also want to benchmark before/after change: I think that because YAML is a superset of JSON, it should be slower to read and write. So there is hopefully a performance boost from this change, however that's not the main goal.

After changes
8.52 real        35.95 user        19.82 sys
            64073728  maximum resident set size
                   0  average shared memory size
                   0  average unshared data size
                   0  average unshared stack size
               38685  page reclaims
                   0  page faults
                   0  swaps
                   0  block input operations
                   0  block output operations
                   0  messages sent
                   0  messages received
                   0  signals received
                  55  voluntary context switches
              989654  involuntary context switches
<PHONE_NUMBER>55  instructions retired
<PHONE_NUMBER>05  cycles elapsed
            57688064  peak memory footprint

So in terms of performance...not that different. JSONDecoder is known to be pretty slow in general so I guess this shouldn't surprise me

Benchmark code
import Files
import Foundation
import Yams
import Version
struct Temp: ParsableCommand {
  static var configuration = CommandConfiguration(
    commandName: "temp",
    abstract: ""
  )

  func run() throws {
    let loadedConfig = try ConfigurationLoader().load()
    guard let workingFolder = try File(path: loadedConfig.path).parent else {
      throw ChangesError("Could not find folder of changes config.")
    }

    let releasesFolder = try workingFolder.createSubfolderIfNeeded(
      at: ".changes/releases"
    )
    let encoder = JSONEncoder()
    for i in 0..<500 {
      let releaseFolder = try releasesFolder.createSubfolderIfNeeded(at: "1.0.\(i)")
      let info = ReleaseInfo(version: try! Version("1.0.\(i)"), createdAtDate: Date())
      let infoData = try encoder.encode(info)
      try releaseFolder.createFile(named: "info.json").write(infoData)
      let subfolder = try releaseFolder.createSubfolderIfNeeded(at: "entries")
      for _ in 0..<100 {
        let entry = ChangelogEntry(
          tags: ["Added", "Removed"],
          description: "This is a description",
          createdAtDate: Date()
        )

        let output = try encoder.encode(entry)
        try subfolder.createFile(named: "\(UUID().uuidString).json").write(output)
      }
    }
  }
}

Tried to use ikigajson for fun to see if it'd speed it up but it just crashed.


Add JSON classification route

It would be useful to have an API route which classifies an image and returns the classifications as a JSON object.

Closed in b4d1658


Docs.MS Release Updates for Azure.Storage.Blobs.ChangeFeed

Update docs metadata and targeting for release of Azure.Storage.Blobs.ChangeFeed

Docs Build status updates of commit 32cbb3f:
:warning: Validation status: warnings

File
Status
Preview URL
Details

api/overview/azure/storage.blobs.changefeed-readme-pre.md
:warning:Warning
View
Details

api/overview/azure/storage.blobs.changefeed-readme-pre.md

Line 2, Column 1: [Warning-ms-prod-and-service] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.
Line 9, Column 16: [Warning-ms-prod-technology-invalid] Invalid value for 'ms.technology': 'azure' is not valid with 'ms.prod' value 'azure'.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.
Line 72, Column 1: [Suggestion-table-syntax-invalid] Table syntax is invalid. Ensure your table includes a header and is surrounded by empty lines. NOTE: This Suggestion will become a Warning on 1/29/21.

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


[feat] draft block verification tests based on block data in tests/blocks

[x] issue #49
[x] follows contribution guide
[x] code change includes tests

Test validate_and_apply on block 0 and block 170.
Added tests on generated next_chain_state
Solved a few small bugs
Wait for more implementation for more tests
Block 170 need txid() to generate merkle root, it currently fails because merkle root of tx is set to [0,0,...] so it reverts to non-duplication of hashes

@m-kus

Waiting #86 to rebase and push


Opening keepass with keyfile does not work: Composite Key is invalid although it is correct

My environment:

KeePass 2.43
Powershell 7
SecretManagement.Keepass <IP_ADDRESS>

I found an issue when opening a keepass file with keyfile.
The private function Connect-KeePassDatabase tests a variable $KeyFile but that is not set anywhere in the function.
Example keepass & keyfile added
dbwithkeyfile.zip

Haven't actually tested the keyfile and usewindowsaccount fully yet, not surprised. I'll look into this.

Haven't actually tested the keyfile and usewindowsaccount fully yet, not surprised. I'll look into this.

got it working, was just a typo: keyfile should be keypath

got it working, was just a typo: keyfile should be keypath

@trir262 I plan to provide a Get-KeePassParams or something like that to produce an object to clearly structure the keyvault options since there's basically zero feedback.
@paulhigin another real-world user example of what I'm talking about :)

@trir262 I plan to provide a Get-KeePassParams or something like that to produce an object to clearly structure the keyvault options since there's basically zero feedback.
@paulhigin another real-world user example of what I'm talking about :)

I would think that the VaultParameters in Register-SecretVault would need to have the necessary information, like 'MasterPassword=$true' or 'KeyFileLocation=path', no?
That way, both test-secretvault and get-secret can rely on that data to do their correct handling.
Test-SecretVault in itself can also - like you did - store a module script variable to store sensitive data like the credential password, required key, ....
Or am I missing something else?

I would think that the VaultParameters in Register-SecretVault would need to have the necessary information, like 'MasterPassword=$true' or 'KeyFileLocation=path', no?
That way, both test-secretvault and get-secret can rely on that data to do their correct handling.
Test-SecretVault in itself can also - like you did - store a module script variable to store sensitive data like the credential password, required key, ....
Or am I missing something else?


Let's create the Stage 4?

Or just a new milestone "Future"?

You can move the priority issues of future to the Stage 4, and the others to the Future.

It will take at least 5 months to finish stage 1-3 so that might not be a good idea

Stage 4 could have: #41, #26, #22, #20, #21, #40.


feat: #184 Expose intersections rust

Expose the intersect method, similar to numpy's intersect1d that gives the count of intersection between 2 integer vectors.

Looks good! I think we should also add tests and examples to the docstrings.
Also, a while ago I wrote a small toolkit to compare the operations I frequently use in StringZilla against memchr crate. Would be great to extend with a TF-IDF implementation in vanilla Rust vs the one accelerated with those operations in SimSIMD. This is more of a creative challenge, than some of the other issues in this repo, but let me know if you'd like to try such a challenge?

Yes and yes!


Is it the responsability of Arcus to enable adding correlation-id's to dependency calls ?

Is it the responsability of Arcus to provide functionality for developers to easy set 'dependencyIds' when calling dependencies ?
For instance:

is it the responsability of Arcus to provide a method (extension method on HttpClient which easily allows developers to set the correlation-info ?
Is it the responsability of Arcus to provide functionality to easily set correlation-info in servicebus messages ?

I would say: no, this is not the responsability of Arcus.  (At least, not in a first phase).
I think we should focusing on the 'accepting' end.
That is, we should be able to log requests, and correlate incoming requests with the dependency call that triggered that request.
To do that, we would need methods that would allow developers to specify a delegate on how to retrieve the correlation-information from the incoming request (http request, servicebus message, .... )
That is, when configuring the Correlation Middelware in API / ASP.NET projects, developers must be able to specify how the correlation information must be retrieved
Same is true when registering a message-pump that processes servicebus messages

Yep, I think that would be a good first step. I think it will be very valuable to also document this in a cross-repo way. Or some sort. That we can show people how to set this up. Even when in the first phase we don't provide built-in functionality that does the 'add' of correlation information in HTTP and/or Azure Service Bus messages.

This is more of a discussion, really, than an issue 😄 .

That is, when configuring the Correlation Middelware in API / ASP.NET projects, developers must be able to specify how the correlation information must be retrieved

This is already the case I think as the HTTP correlation middleware in Web API has several options to how the correlation should be extracted from the request, how the response should be structure, and many more... https://webapi.arcus-azure.net/features/correlation

Creating the 'next' dependency ID is probably a good thing to be included in Arcus, otherwise the service-to-service functionality is not at all user-friendly.
Talking about this line for Service Bus dependencies:
https://github.com/arcus-azure/arcus-service-to-service-correlation-poc/blob/dad99894d7080cf44e5f7e1e58ded7421b9a4826/src/Arcus.API.Market/Repositories/OrderRepository.cs#L46
We could provide an easier way to come up with this stuff, as it's not only obscure of new people, it's hard to get right.

Something like this or so?
var orderRequest = new EatBaconRequestMessage
{
    Amount = amount
};

using (var serviceBusDependencyMeasurement = DurationMeasurement.Start())
{
    bool isSuccessful = false;
    CorrelationInfo currentCorrelation = _correlationInfoAccessor.GetCorrelationInfo();
    CorrelationInfo newDependencyCorrelation = currentCorrelation.CreateForNextDependency();

    try
    {
        var serviceBusMessage = ServiceBusMessageBuilder.CreateForBody(orderRequest)
                                                    .WithCorrelationInfo(newDependencyCorrelation)
                                                    .Build();

        await _serviceBusOrderSender.SendMessageAsync(serviceBusMessage);

        isSuccessful = true;
    }
    finally
    {

        var serviceBusEndpoint = _serviceBusOrderSender.FullyQualifiedNamespace;
        _logger.LogServiceBusQueueDependency(_serviceBusOrderSender.EntityPath, isSuccessful, serviceBusDependencyMeasurement, dependencyId: newDependencyCorrelation.OperationParentId);
    }
}

I think it is mandatory to not put a burden on the developer to define / generate those Id's.
If possible, I'd like to make sure that the Arcus user doesn't even need to think about generating a dependencyId, even if it just means that a certain function needs to be called.
Can't we include the generation of that Id in the LogDependency extension method ?

Can't we include the generation of that Id in the LogDependency extension method ?

Well, no, bc the measurement of the call to the dependency should come before that, so then it's already too late.

I think it is mandatory to not put a burden on the developer to define / generate those Id's.

Yes, isn't then the second example the thing that you want? Where the tracking is completely hidden?

Can't we include the generation of that Id in the LogDependency extension method ?

Well, no, bc the measurement of the call to the dependency should come before that, so then it's already too late.

You can only call the LogDependency method when the dependency call is finished no ?  Because you need to know whether the call was succesfull and you also need to know how long the dependency call took.

You can only call the LogDependency method when the dependency call is finished no ? Because you need to know whether the call was succesfull and you also need to know how long the dependency call took.

Yes, exactly, so that's why the dependency ID can't be generated by the LogDependency.

So, to come back: we can opt for a way to hide the dependency ID completely in a separated overload (2th code sample) or via simpler methods on something else (1th code sample).

You can only call the LogDependency method when the dependency call is finished no ? Because you need to know whether the call was succesfull and you also need to know how long the dependency call took.

Yes, exactly, so that's why the dependency ID can't be generated by the LogDependency.

Yes, true.  I might have been confused as I was focusing on 'parent Ids' yesterday.

So, to come back: we can opt for a way to hide the dependency ID completely in a separated overload (2th code sample) or via simpler methods on something else (1th code sample).

I like the 1st sample best, where the correlation information is specified when creating the message.


Broker Log Truncation

Update the broker to truncate the log based on the highest index persisted across data nodes.

All done.


MGDCTRS-2205, MGDCTRS-2270: remove strimzi CRDs and change RBAC names

Removes the Strimzi CRDs from the stage overlay.
Changes Strimzi RBAC names to be unique to RHOC.

I'm waiting on a different fix before merging this to avoid mixing issues. Please don't merge for now. I'll merge when the other issue is fixed.


Verification

Will this auto verify accounts? Or will they all be locked out after 2 days?

This does not do verification. See #7 for other options

thanks


DatePicker dialog doesn't work on iPhone 12 or greater

With iPhone 12 or higher, the DatePicker window blocks its execution when the showAndWait method is called, without showing exceptions and without opening its dialog. In addition, it is no longer possible to go back but you must necessarily close the app.
https://docs.gluonhq.com/charm/javadoc/6.0.6/com.gluonhq.charm.glisten/com/gluonhq/charm/glisten/control/DatePicker.html
I have attached the log of the run made on an iPhone 12. By clicking on the central button that should open the calendar for choosing the date, the app freezes and I can do nothing but close it.
Run_Iphone12.txt
this is the example project:
project_test.zip

I don't have iPhone 12 or higher to test. Do you mean that the same app works fine on iPhone 11 or lower?

Yes, the problem starts with iPhone 12 and later

Is the com.gluonhq.charm.glisten.control.DatePicker component to be used only in a Dialog or is there a way to use it as a node in a view?
Is DatePicker the only component to display a calendar and select a date or is there another component?

FYI: There was a similar issue reported on Attach/BarcodeScanner service and we have identified the problem.
All classes that extend from GluonMobile's Dialog use enterNestedEventLoop, so that needs to be fixed.


Hang Status is not handled

Describe the bug
Discord rolled out new feature called Hang Status for some users. Unfortunately, I wasn't able to find official docs for it, but there are a lot of articles in Google about it.
Apparently, this new status is not handled by ActivityType and whenever someone sets it, an exception is thrown. Now here comes another problem. After trying many different options to catch this exception, none of them actually worked (I tried adding onError callbacks, handleError calls, try/catch in different places, runZoneGuarded, etc., all this with and without ignoreExceptions plugin). Eventually, once this error occurs, bot stops listening to other commands and the only way to fix it is to restart the bot.
To Reproduce
Steps to reproduce the behavior:

If you have Hang Status feature enabled for your account:

Join voice channel and set Hang status;
Start bot with Nyxx.conntectGateway call, use GatewayIntents.all;
Observe thrown exception - FormatException: Unknown activity type

If you don't have new Hang status feature enabled for your account:

Remove dnd key from UserStatus;
Set Do not disturb status on the account;
Start bot with Nyxx.conntectGateway call, use GatewayIntents.all;
Observe thrown exception - FormatException: Unknown user status (which essentially leads to the same issue);

Expected behavior
Library correctly handles Hang status for the users.
Desktop (please complete the following information):

OS: Any;
Dart version: 3.3.4
Nyxx version: 6.2.1

Thanks for reporting this! This is an issue we are aware of and will be fixed in the next version, but this gives us a nice reminder to fix it.
For now, you can use a patched version of nyxx with support for hang status by adding this to your pubspec.yaml:
dependency_overrides:
  nyxx:
    git:
      url: https://github.com/abitofevrything/nyxx.git
      ref: patch/hang-status

Works like a charm, thanks a lot for quick reply and awesome library! :)
I'll leave it to you to decide whether to close the ticket now (since there's a solution to the problem) or you'll do it when this fix is released.

We'll leave it open to keep track of it.

Hello @dmbaranov the issue is now fixed on the dev branch, you can switch to the main repo in your dependency_overrides :)

Thank you so much, appreciate it! :)


[C] Normalization Refactor + Adding CUDNN backend

Description

cuDNN normalization was integrated into TE. Users can use this backend via env vars as:

export NVTE_NORM_FWD_USE_CUDNN=1
export NVTE_NORM_BWD_USE_CUDNN=1

CuDNN 9.6 is needed.
By default, TE Kernels are still used for normalization. Further performance evaluation on different supported GPUs is needed to make cuDNN backend as default.

Major changes were introduced in the TE normalization C APIs:

Merging barrier, dgamma_part, and dbeta_part into the workspace. Thus those args are no longer needed in the APIs.
Unifying APIs with and without zero centering.

These are the updated C APIs:
void nvte_layernorm_fwd(const NVTETensor x, const NVTETensor gamma, const NVTETensor beta,
                        const float epsilon, NVTETensor z, NVTETensor mu, NVTETensor rsigma,
                        NVTETensor workspace, const int multiprocessorCount,
                        const bool zero_centered_gamma, cudaStream_t stream);

void nvte_layernorm_bwd(const NVTETensor dz, const NVTETensor x,  const NVTETensor mu,
                        const NVTETensor rsigma, const NVTETensor gamma,  NVTETensor dx,
                        NVTETensor dgamma, NVTETensor dbeta, NVTETensor workspace,
                        const int multiprocessorCount, const bool zero_centered_gamma,
                        cudaStream_t stream);

void nvte_rmsnorm_fwd(const NVTETensor x, const NVTETensor gamma, const float epsilon, NVTETensor z,
                      NVTETensor rsigma, NVTETensor workspace, const int multiprocessorCount,
                      const bool zero_centered_gamma, cudaStream_t stream);

void nvte_rmsnorm_bwd(const NVTETensor dz, const NVTETensor x, const NVTETensor rsigma,
                      const NVTETensor gamma, NVTETensor dx, NVTETensor dgamma,
                      NVTETensor workspace, const int multiprocessorCount,
                      const bool zero_centered_gamma, cudaStream_t stream);

The following APIs are deprecated:
nvte_layernorm1p_fwd
nvte_layernorm1p_bwd
nvte_rmsnorm1p_fwd
nvte_rmsnorm1p_bwd

TE normalization implementations were refactored to be more friendly for adding cuDNN backend.

Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Infra/Build change
[x] Code refactor

Checklist:

[ ] I have read and followed the contributing guidelines
[x] The functionality is complete
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/te-ci

/te-ci

/te-ci

/te-ci

/te-ci


Unable to deploy template using inline json

Hi all,
There's probably something wrong with my code, but as there's no documentation around, hopefully someone can help me. Im trying to deploy a template using local jsons - i don't want to go the "templatelink" route of publishing content to a storage group first. Here's my code:
resource_client = azure.mgmt.resource.ResourceManagementClient(creds)

GROUP_NAME = "arm-python"
REGION = 'westeurope'
result = resource_client.resource_groups.create_or_update(
    GROUP_NAME,
    azure.mgmt.resource.ResourceGroup(
        location=REGION,
    ),
)

template = """{
  "$schema": "https://schema.management.azure.com/schemas/2015-01-01/deploymentTemplate.json#",
  "contentVersion": "<IP_ADDRESS>",
  "parameters": {
    "location": {
      "type": "string",
      "allowedValues": [
        "East US",
        "West US"
      ],
      "metadata": {
        "description":"The location"
      }
    }
  },
  "resources": [
    {
      "type": "Microsoft.Compute/availabilitySets",
      "name": "availabilitySet1",
      "apiVersion": "2015-05-01-preview",
      "location": "West US",
      "properties": {}
    }
  ]
}
"""

parameters = """
{
    "$schema": "http://schema.management.azure.com/schemas/2015-01-01/deploymentParameters.json#",
    "contentVersion": "<IP_ADDRESS>",
    "parameters": {
        "location":{
            "value": "West US"
        }
    }
}
"""

properties = azure.mgmt.resource.resourcemanagement.DeploymentProperties(
        mode="incremental",
        template=template,
        parameters=parameters
    )

deploy_parameter = azure.mgmt.resource.Deployment()
deploy_parameter.properties=properties

result = resource_client.deployments.create_or_update(
    resource_group_name="arm-python",
    deployment_name="depname",
     parameters=deploy_parameter
)

The credentials part work, because the first call to create the resource group works fine. THe error I'm getting is:
b'{"error":{"code":"InvalidRequestContent","message":"The request content was invalid and could not be deserialized: 'Error converting value "<IP_ADDRESS>" to type 'Microsoft.WindowsAzure.ResourceStack.Frontdoor.Data.Definitions.DeploymentParameterDefinition'. Path 'properties.parameters.contentVersion', line 1, position 520.'."}}'
The SDK seems unable to correctly parse my json, but the same template/parameter content works fine if using powershell to create the deployment. Any pointers appreciated! (and to maintainers: Please provide samples!)

I was able to get your code working by changing the parameters json.  Apparently, when it's inline, the top level objects aren't needed.
https://msdn.microsoft.com/en-us/library/azure/dn790564.aspx
parameters = """
{
    "location":{
        "value": "West US"
    }
}

Nice, that works perfectly. Thanks!

Added test and documentation for this. Thanks for the feedback.

Hey folks,
I have the same issue using C#. I also tried to do what you recommended in this thread but did not work. Any thoughts?
Here is the full exception:
Error: System.AggregateException: One or more errors occurred. ---> Microsoft.Rest.Azure.CloudException: The request content was invalid and could not be deserialized: 'Error converting value "
}" to type 'Microsoft.WindowsAzure.ResourceStack.Frontdoor.Templates.Schema.Template'. Path 'properties.template', line 3, position 23171.'.
at Microsoft.Azure.Management.ResourceManager.DeploymentsOperations.d__9.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Azure.Management.ResourceManager.DeploymentsOperations.d__8.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Azure.Management.ResourceManager.DeploymentsOperationsExtensions.d__7.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
at ConsoleApplication1.Program.d__3.MoveNext() in

@huguesv you are a saver, my couple of days of searching ended here with your link to MSDN. (i was trying an inlined template and paramater from the nodejs sdk)

Added test and documentation for this. Thanks for the feedback.
@huguesv could you provide a link to said documentation please?

Hi @arcotek-ltd
I don't find the documentation, but this unitest might help:
https://github.com/Azure/azure-sdk-for-python/blob/c04531af1ef6d76384c9f3e61eead7ace04fcc03/azure-mgmt/tests/test_mgmt_resource.py#L201-L319

But isn't this then incompatible with what New-AzureRmResourceGroupDeployment in Powershell?
That expects the full json with outer elements.

How was this issue resolved. Hitting same issue with C#

To clarify the workaround:
The parameters file from the portal looks something like this:
{
  "$schema": "https://schema.management.azure.com/schemas/2015-01-01/deploymentParameters.json#",
  "contentVersion": "<IP_ADDRESS>",
  "parameters": {
    "name": {
      "value": "ImagingFunctionsd49b41"
    },
    "location": {
      "value": "eastus"
    }
  }
}

To work-around the problem, remove the header elements to make the parameters file look like this:
{
    "name": {
      "value": "ImagingFunctionsd49b41"
    },
    "location": {
      "value": "eastus"
    }
}

Hi Friends,
Getting the same error while deploying the template in deploy if not exists Azure policy action
error: '{"error":{"code":"InvalidTemplate","message":"Deployment template parse failed: 'Error converting value \"Standard_LRS\" to type 'Microsoft.WindowsAzure.ResourceStack.Common.Core.Definitions.Resources.ResourceSku'. Path ''.'."}}'.",


Jumps back to fancy graph when it really shouldn't

When I look at a fast graph and increase the range to, say, 1000 days, the tab freezes for something like a minute and then presents the fancy graph. I'm assuming the page would have frozen a significantly shorter time using the fast graph.

It probably jumps back because a few thousand datapoints isn't that much (I assume you didn't select all too many dimensions).
The graph rendering seems to be fine as well, but it waits a few seconds until it even starts fetching new data. I couldn't reproduce that on our instance, but a quick profile on yours showed it spent around 90% of the time inside Vue, the UI framework, calculating and updating dependencies between the displayed state and the internal one.
The page would probably be frozen just as long if it picked the fast graph, the graph computation and rendering time seemed to be quite normal.
I will have a look at reproducing it locally and tracking down where exactly it updates way too many dependent bindings.

I partially retract my statement. Echarts doesn't seem to like big data ranges, it can handle more points when the date range is small and there are a few series apparently.
I lowered the threshold by around 20.000, you can always manually switch back to the fancy graph if you want to, after all.
The other big problem was that it updated its data, waited for the current graph to fully render and then selected the more fitting graph.
I will see if that's enough, but it looks like quite a nice improvement with our dummy data (a few hundred thousand points of perlin noise).

I think that might've actually been enough. Loading https://velcom.aaaaaaah.de/repo-detail/2ff64fb6-5996-4bfd-a99e-86ebcd6441c7?dimensions=Beat this:cats in the cradle:magic:oracle:size::BenchMe:cats in the cradle:magic:oracle:size completes in around 8 seconds, but only 1.4 are spent on drawing graphs. The rest is waiting for the backend to respond (and that should get faster once the listener branch is merged as well).
We can probably merge the UI changes to stable ourselves in the next few days, once #135 landed so you can try it out on the lean instance.

Yep, sounds good!


Matrix Determinant

I did not see any determinant function. Any objection to my adding one?

Yes, there is no determinant function in the Matrix module and a pure OCaml contribution is warmly welcome.
But if you're looking to get something working now, there is a driver over lacaml in the determinant_symmetric and more general determinant method in the Lacaml_util.ml module.

@fourtytwo I'm curious, what did you end up using? What is your use case?

@rleonid  I used the one you pointed to... but i intend to write a pure ocaml version once i have more time.


Add more info to docs regarding multiproc mode

Make it more clear how the multiproc mode works and explicitly point out and link to the respective section in the prometheus client docs

https://github.com/trallnag/prometheus-fastapi-instrumentator/discussions/24#discussion-1559548
https://github.com/trallnag/prometheus-fastapi-instrumentator/discussions/24#discussioncomment-331307


The border disappears on the workspace and the flyout overlaps the backpack when you drag blocks

Expected Behavior
Borders and layering shouldn't change when you drag blocks
Actual Behavior
The border disappears, and the flyout overlaps the backpack

If you open and close the backpack, it does this instead

Steps to Reproduce
Drag a block out from the flyout
Operating System and Browser
Mac Chrome

I'm pretty sure this is a duplicate of these two issues: #1855 for the first gif, LLK/scratch-gui#953 for the second one. I've had a hard time repro'ing #1855 which is why I closed it, but what you're seeing in your gif is the exact same thing I used to. There's an old pull request I made, #1895, which definitely helps with the second and might help with the first.


Add details on AWS deployment

Including some Sonatype specific links and additional details for new AWS users (like me!).
@bhamail can you please review.

PS: Build may fail due to unrelated OSSIndex issues, so don't spend cycles worrying about that. Merge away if you like.


how i can upload image and video to server i try header multipart/form-data  any help please

I'm submitting a ...

[ ] bug report
[ ] feature request
[ ] other (Please do not submit support requests here (see above))

Current behavior:

Expected / new behavior:

Minimal reproduction of the problem with instructions:

Angular version: 1.x.y

Browser: [all | Chrome XX | Firefox XX | Edge XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Anything else:

@henrik @expilo
any one help me please
?

This sounds like a general support question. Please, use one of the appropriate support channels for these types of questions.
GitHub issues are reserved for bug reports and feature requests.
Thx!


[Question] Why not Vue 3?

As the heading suggests, this is more of a question than an issue.
Why not use vue 3?
Is it because of Vuetify(since it doesnt have a vue 3 integration yet)?

That's a fair question, thanks! I think we mainly focused on Getting Things Done™, instead of checking whether all the dependencies (vuetify, chartjs) would work with Vue 3.
But we could totally migrate to Vue 3 in the future :+1:

Oh, ok, cool then..
When i get time, i was thinking of creating the Vue 3 version from you code(refactor it basically).
Would that be okay?
I' ll make a branch and a PR.
Cant guarantee when though.

@AnoRebel that would be awesome :+1:

Just FYI: Vuetify3 is not getting released soon. We have to hold our breath for months, if it continues like this with Vuetify 3 😉
Also note that Vuetify v3.1 will take even longer and only that version is feature-complete...


504 error with Chrome, Edge, Opera No error with Firefox, Brave or app

Discussed in https://github.com/jc21/nginx-proxy-manager/discussions/1637

Originally posted by Philmo67 December  7, 2021
Hello,
I have a confusing problem: redirects to my services only work when using Firefox, or the dedicated Android apps (in LAN, WLAN and WAN, so the network/firewall configuration can be considered correct). I can't connect with Chrome or Edge, I get a 504 error.
I've purged the caches, including HSTS.
Latest docker image.
The services are set up as follows:
{
  "id": 1,
  "created_on": "2021-08-23 23:48:41",
  "modified_on": "2021-11-29 12:24:09",
  "owner_user_id": 1,
  "domain_names": [
    "service.domain.org"
  ],
  "forward_host": "<IP_ADDRESS>",
  "forward_port": 80,
  "access_list_id": 0,
  "certificate_id": 19,
  "ssl_forced": true,
  "caching_enabled": false,
  "block_exploits": true,
  "advanced_config": "",
  "meta": {
    "letsencrypt_agree": false,
    "dns_challenge": false
  },
  "allow_websocket_upgrade": true,
  "http2_support": false,
  "forward_scheme": "http",
  "enabled": 1,
  "locations": [],
  "hsts_enabled": false,
  "hsts_subdomains": false
}

Error log shows :
2021/12/07 21:40:31 [error] 13228#13228: *114081 upstream prematurely closed connection while reading response header from upstream, client: <IP_ADDRESS>, server: service.domain.org, request: "GET / HTTP/2.0", upstream: "http://<IP_ADDRESS>:80/", host: "service.domain.org"
Access log shows :
[07/Dec/2021:21:39:40 +0000] - - 499 - GET https service.domain.org "/" [Client <IP_ADDRESS>] [Length 0] [Gzip -] [Sent-to <IP_ADDRESS>] "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.55 Safari/537.36 Edg/96.0.1054.43" "-" [07/Dec/2021:21:40:31 +0000] - 502 502 - GET https service.domain.org "/" [Client <IP_ADDRESS>] [Length 556] [Gzip -] [Sent-to <IP_ADDRESS>] "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.55 Safari/537.36 Edg/96.0.1054.43" "-" [07/Dec/2021:21:41:23 +0000] - 502 502 - GET https service.domain.org "/favicon.ico" [Client <IP_ADDRESS>] [Length 556] [Gzip -] [Sent-to <IP_ADDRESS>] "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.55 Safari/537.36 Edg/96.0.1054.43" "https://service.domain.org/"
I tested using Brave and services work fine, however, no success using Opera.
So the same services are OK using Firefox and Brave but sending a 504 error using Chrome, Edge and Opera ...
Where to start investigating?

What tools do you use to see what is going on ?
What do Chrome, Edge or Opera have in common that could cause timeout/bad gateway error when establishing connection (that Firefox, Brave or an android app do not have) ?


Add missing Gerber extensions and heuristics

Description
There are a few additional extensions for Gerber (.cmp, .crc, .crs, .ncl). See: http://cirqwizard.org/docs/gerber.html
We had a Gerber sample in samples/Text/LIDARLite.ncl which is now moved to samples/Gerber Image.
Updated heuristic for .ncl, and added new ones for .cmp, .crc and .crs, with a catch-all fallback to Text.
.cmp is certainly common, but .crc. .crs and .ncl not so much. An alternative to this PR would be removing the LIDARLite.ncl sample and adding just the .crs extension Gerber Image.
It's quite hard to find any sample with a liberal license, so tests are just reusing all Gerber samples using the alt_name trick to simulate the extension.
Checklist:

[x] I am associating a language with a new file extension.

The new extension is used in hundreds of repositories on GitHub.com

[x] .cmp: https://github.com/search?q=extension%3Acmp+(Gerber+OR+Ultiboard+OR+LPD+OR+MOIN+OR+M02)+NOT+nothack&type=Code
[ ] .crc: https://github.com/search?q=extension%3Acrc+(Gerber+OR+Ultiboard+OR+LPD+OR+MOIN+OR+M02)+NOT+nothack&type=Code
[ ] .crs: https://github.com/search?q=extension%3Acrs+(Gerber+OR+Ultiboard+OR+LPD+OR+MOIN+OR+M02)+NOT+nothack&type=Code
[ ] .ncl: https://github.com/search?q=extension%3Ancl+(Gerber+OR+Ultiboard+OR+LPD+OR+MOIN+OR+M02)+NOT+nothack&type=Code

I have included a real-world usage sample for all extensions added in this PR:

[x] .ncl: samples/Gerber Image/LIDARLite.ncl, previously present as samples/Text/LIDARLite.ncl
[ ] .cmp, .crc, .crs: not found so far, simulated in tests

[x] I have included a change to the heuristics to distinguish my language from others using the same extension.

cc @Alhadis per #3744

with a catch-all fallback to Text.

Wouldn't it be better to mark said extensions as generic instead?

with a catch-all fallback to Text.

Wouldn't it be better to mark said extensions as generic instead?

Sure! I missed that feature.

Done. We still need to decide whether any of these extensions belong to Linguist (re. popularity).

162 users is high enough to qualify as a plural w.r.t. @lildude's "hundreds of users" requirement. 😉

To remain consistent, it's a "not yet".

Rebased and removed .crs and .crc extensions.


[sailfish-secrets] Put the P2P DBus socket in a sub-directory of /run.

The DBus dialog is currently happening on /run/user/100000/sailfishsecretsd-p2pSocket. For instance when you need a confirmation to access the secret storage, or you require a password.
But for jailed application, /run is no readable. So like QMF, this MR is moving the socket to a subdirectory to allow to whitelist it.
@chriadam do you agree ?

Technically user services should not really directly have services in /run but only in /run/user. What stops us from allowing writing to to this socket with a permissions?

Sorry, it seems I'm still not clear.
When you're using gpgme library in your code, your code will dialog with the gpg-agent, using dedicated sockets allowed in GnuPG.permission file, as you mentioned.
Then, the gpg-agent may launch a pinentry program to get the passphrase from the user, for signature handling or decryption for instance.
Sailfish secret is providing a pinentry program (like Gnome is doing for instance) that is interfaced with Lipstick. To work, this pinentry program is discussing with the secret daemon over DBus to get a user input interaction. So the pinentry requires to access the DBus p2p socket established by the secret daemon.
At the moment, a gpg-agent is not permanently running in the user session. So one is launched on demand by a program using gpgme for instance. If the program using gpgme is running inside a jail, the gpg-agent will also run inside the jail. Thus it won't be able to see the secret daemon socket. Except if this socket is whitelisted. But as far as I now, we can only white list directories, and whitelisting /run/user/100000 would be a breach in isolation.
The solution that was proposed in QMF for instance, was to move the socket from /run/user/100000 to a dedicated subdirectroy, so this subdirectory only can be whitelisted for Email permission. I'm applying the same solution here for the secret daemon.
Actually the PR here has nothing to do with GnuPG. This PR is more about allowing permission for an application to talk to the secret daemon. At the moment, not any jailed application can (and particularly the gpg-agent from a jail), thus this PR.
Another completely independent solution to my problem of email signing not working would have been to run a gpg-agent continuously in the user session, or having it started by systemd, so it could always access the secret daemon socket.

Yes, it needs to be in its own directory, so that the sailjail whitelisting can work.  Thank you.  @Tomin1 can you check this one also, please?

Yes, it needs to be in its own directory, so that the sailjail whitelisting can work.

That's right.

Thank you. @Tomin1 can you check this one also, please?

Looks fine to me too. :+1:


[staging-next] ocaml-ng.ocamlPackages_4_09.ocaml: fix with GCC 12

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Why does this target staging-next (rather than, say, master)?

Because in staging-next, GCC 12 is the default, so that's where the failure really becomes noticeable.  But I suppose it would be fine to target master and get the fix in right away.

@GrahamcOfBorg build coq_8_10


Update contribute section under community

Update the contribute section to:

Update the number of contributors (and organizations) for each GitHub repo
Missing repos: delta-examples, delta-docs (??)

Hey can u please elaborate little bit to it. Like to update number of contributors for each github repo delta have is that correct?

Hi @tiwariParth  - Yes, the numbers are not up to date.  For example, on the Community page, delta has 192 contributors, but currently there are 266 contributors per Delta GitHub


Styled Charsheet not rendering correctly

Environment Details

Foundry Core Version: 0.7.9
System & Version: KULT 2E 1.1.0 (https://git.lynxcore.org/foundry/kult2e/system)
OS & Browser: Chrome, Vivaldi
Hosting: Self Hosted
(Forge, Self Hosted, Electron App, etc might be relevant)

Describe the bug
On displaying a character sheet of my game system in the GM screen, I only get a very basic non-styled and non-layouted version of the sheet.
To Reproduce
Steps to reproduce the behavior:

Add character sheet to GM Screen
View the Character sheet in GM Screen

Expected behavior
The GM Screen should display my styled and layouted character sheet
Screenshots

Now, I think this might 100% be my fault, but I didn't find any resources on this and I compared with two other systems where it works (CyberPunk RED and Dark Heresy 2E), and I can't find the place where I might influence this behaviour.

What system is this? I should probably clarify I don't mean "linux" by "system" in the template :p

Happens :) It's my own system, KULT2E. If you're on the Foundry Discord, you can hit me up there if you want a live demonstration or something, but the source is viewable and checkoutable here: https://git.lynxcore.org/foundry/kult2e/system

Interesting, I'll take a look and see if I can tell what's going on.
Some gut feelings I'm going to have to investigate:

the entity doesn't have the right sheet class assigned to it
my CompactGenericDisplay is overriding something you do differently in your sheet module that most other systems do not do.
somehow editable or popout is what's driving your stylesheet.

huh... some thoughts: my sheet is indeed editable, and I have the popout! module installed. A quick check shows PopOut! is not the culprit. As for editable I'll have to look into that.
I don't think I do anything different. I basically followed the tutorials and started with boilerplate. The only thing I do is adding two classes to the sheet in defaultOptions()
Which sheet class should be assigned? If it's something like that I can guarantee that I don't have the correct sheetclass :)

If all of your styles are gated behind editable, they won't appear on GM Screen. At the moment I'm forcing all sheets displayed on the grid into editable: false to avoid potential issues.

ok, I can work with that. I wanted to make a compact sheet for the GM screens anyways, and that doesn't need to be editable. Let me check that and get back to you.

Hmmm. Nope. Here's the complete sheet js in question:
import { Kult2eCharacterSheet } from './character-sheet.js';

export class Kult2eMonsterSheetCompact extends Kult2eCharacterSheet {
  /** @override */
  static get defaultOptions() {
    return mergeObject(super.defaultOptions, {
      classes: ['kult2e', 'sheet', 'actor', 'dx', 'monster'],
      template: 'systems/kult2e/templates/actor/monster-sheet.html',
      width: 800,
      height: 750,
      editable: false,
      popout: false,
      tabs: [
        { navSelector: '.info-tabs', contentSelector: '.info-body', initial: 'bio' },
        { navSelector: '.gear-tabs', contentSelector: '.gear-body', initial: 'skills' },
        { navSelector: '.combat-tabs', contentSelector: '.combat-body', initial: 'wounds' },
        {
          navSelector: '.mentalbalance-tabs',
          contentSelector: '.mentalbalance-body',
          initial: 'advantages',
        },
      ],
    });
  }

  getData() {
    const data = super.getData();

    return data;
  }
}

I'm using the same template.html as before and changed nothing else, just set editable and popout to false. The sheet still renders fine, and no changes in GM screen.

Took a look, this is happening because of your (LESS) CSS:
.kult2e.window-app.sheet.actor.dx.character .window-content

window-app and window-content are not wrapping sheets inside the Grid as the Grid itself can be inside an Application.
Similarly, your .kult2e.window-app .window-content which contains the background for all your windows isn't going to apply to the Grid cells. You might not want it to though, and might instead prefer the whole grid drawer get this texture. If that's the case, you'd need to change this background to apply simply to .window-content
Basically your CSS makes some assumptions about the elements wrapping sheets which I break with GM Screen. I think if you format your styling in such a way that the contents of the sheet are targetted without the assumption that they'll be wrapped in a .window-content or .window-app, you'll be golden!
I hope this helps, if you get stuck on this endeavor I might be able to provide some specific pointers.

Gotcha. So, just to make sure I'll just drop .window-app and .window-content and basically start at form, at least for the compact sheet? I'll try it out and let you know how it works out

Keep me posted about that compact sheet, I think #6 will make it much more useful, I just haven't gotten around to implementing that as there's not been much demand.

Yep, that was exactly it. Background in the GM screen does not get set, but that I'll fix tomorrow.
I'll design the compact sheet tomorrow and will let you know once it's merged and can be viewed. Should I notify you via this issue or close this and notify via #6?

Nice! I'll close this and feel free to leave a comment on #6


Trunk

Sqoop-trunk

Closing conflicting PRs older than a year. Please reopen if it is still applicable.


load data Read timed out

when load a file TabSeparated has 1001000 rows,exception happens as below with java jdbc driver:
ru.yandex.clickhouse.except.ClickHouseException: ClickHouse exception, code: 159, host: <IP_ADDRESS>, port: 12123; Read timed out

Try to increase receive_timeout and send_timeout options. By default it's 300 sec.

@569934390 have you tried it?


Could extend the original function sortby , like sortBy (fn, order)?

the sortBy ,default is to sort in ascending order (+), but i  want specify descending order (-), can you provide a function that sorted according to  the comparator function like sort,.
To achieve this function data in descending order ,my idea is  use reverse base on the sortby
// sort by asc or desc
export const sortBy = (arr, key, order) => {
  const sortByKey = R.sortBy(R.prop(key));
  const newArr = order === false ? sortByKey(arr) : R.reverse(sortByKey(arr));
  return newArr;
};

hmm, you can achieve the descending thing just by invert your comparator.

@juanha:
I wouldn't mind visiting this, but I think we'd have to look carefully at signatures, to ensure that they matched Ramda's notions of simplicity.  We're not much on boolean flags, and the parameter order above is not well-designed for easy currying.  Would it be simply to have sortBy and reverseSortBy or sortBy / sortByDescending ?
If we do this, I would rather write a version that passes an appropriate comparator to sort up front rather than one which sorts then reverses the result.  But that wouldn't be hard to do.  Are you interested in creating a PR for this, understanding that there is a real chance, but no guarantees that it would be accepted?
@syaiful6:
The problem is the sortBy does not return a comparator.  It creates and uses one internally, but it's not exposed.

@CrossEye hmm, correct. Looking on the sortBy, we have two options for descending:

Invert / negate it. (only work for number)

var pairs = [[-1, 1], [-2, 2], [-3, 3]]
var sortByFirstItem = sortBy(compose(negate, prop(0)))
sortByFirstItem(pairs) // [[-1, 1], [-2, 2], [-3, 3]]

Reverse. We can achieve it just by compose the current sortBy and reverse.

...or we can rewrite slightly to allow the code that is responsible for deciding direction to have access to the comparator, and then we can simply apply lift(negate) against it as necessary.  This doesn't sound too hard.  The hard part here is deciding on the correct API for this.

@CrossEye
I reconsider the issue, Could we rewrite the sortBy like the following code to sort list by prop ( both ascending and descending)
module.exports = _curry2(function sortBy( fn, order,  list) { return _slice(list).sort(function(a, b) {      var aa = fn(a);      var bb = fn(b);   return order === 1 ?  aa - bb : bb - aa; }); }); R.sortBy(R.prop(prop), order)(list)
Look forward to your reply

@juanha:
I see several issues with this.
First, I think the signature is probably in the wrong order for easiest currying.  I would expect that a user would more likely want to fix the sort order/direction than sorting function itself.  So the parameter order should probably be (order, fn, list).  (I might also use "direction" rather than "order", but that's minor.)
Second, the order parameter is unintuitive and not at all descriptive.  Why should I pass a 1 to sort ascending?  And what should I pass to sort descending?  @buzzdecafe has taken to calling this FOP, or "flag-oriented programming."  This is just as bad with a boolean, of course, although one called isAscending might be at least somewhat more clear.  (This would not be an issue for an internal function, by the way.  Internally, we can take such shortcuts.  This is one of the reasons that I was suggesting a public interface with two different functions.  They could simply act the same, except that they passed opposite booleans to such an internal function.)
Finally, this code would only allow sorting by numbers (or more precisely by elements with appropriate valueOf semantics), but the current code is more flexible than that, sorting on anything that responds to <.  You can see this in the test suite.
If all these issues were fixed, my answer would still be, "that's nice, but I'd prefer to make this an internal function, and publicly expose partially applied versions with the 'ascending' / 'descending' parameter partially applied.  That still feels like the cleanest public API.

Doesn't ascend and descend in 0.23.0 make this moot?

thanks @ahstro


Dependency Relation Types

I've noticed that the dependency relation types that I get from the dependency parser in spaCy are a bit different than the Stanford dependencies.  Is there some kind of mapping between the these dependency types and the Stanford dependencies?

SpaCy currently uses the Yamada and Matsumoto (2003) dependency scheme, one of several different schemes in use by researchers. I opted for this scheme because some research has found it more accurate for down-stream applications than Stanford, and the smaller number of dependency types can make hard-coding logic easier.
You can see more details about the scheme and the tool used to produce it here: http://stp.lingfil.uu.se/~nivre/research/Penn2Malt.html


17004 proxier.go:1600] "can't open port, skipping it

https://github.com/kubernetes/kubernetes/issues/107619
I am not sure if it is a k3s or a kubernetes issue. Neither how to resolve it.

Are you sure that these errors are actually preventing you from starting pods? When I've seen these warnings in the past, it seems like it is actually trying to bind the ports twice.

Are you sure that these errors are actually preventing you from starting pods? When I've seen these warnings in the past, it seems like it is actually trying to bind the ports twice.

I see following issues/symptoms:
=> lot of bots are in restart/often fail
=> traefik shows the default example.com (even when cert-manager is installed and working)
=> these errors are showed more then once.
I had a feeling there would be some kind of ghost pods since the mounting also failed, but i really can't figure it out why this is happening

The behavior went away and can't reproduce so this can be closed.


Message sizes flags are not in Readme

This commit is merged:
https://github.com/Mellanox/ngc_multinode_perf/commit/a9a322a4a2a5f87713d2fe8d983603f270e73abb
But Readme doesn't show the new flags.

Fixed in e1a3bc8ebbce80fd0103afc5029134e350824b30. Closing. Thanks for the catch!


Web UI doesn't respect protocol

Web UI API calls don't seem to respect the protocol in use.
Example browser console output when running beets-flask behind reverse proxy with TLS:
Mixed Content: The page at 'https://beets.mydomain.com/' was loaded over HTTPS, but requested an insecure resource 'http://beets.mydomain.com/api_v1/config/'. This request has been blocked; the content must be served over HTTPS.

I reproduced the issue and tracked it down.
It essentially comes down to this  (cf this reddit):
https works, but we are missing trailing slashes backend urls in some places of our frontend code.  (e.g. /config instead of /config/). This causes flask to do a redirect, which is not https.
@semohr
Any Ideas for a good fix? I feel like trailing slashes is a small detail our code should be able to tolerate. But adding it for all requests in the customizeFetch might cause other problems with our more involved routes for searching etc.

can we configure this in flask?
or should we just be strict and update all occurences of fetch?
other option would be to try with whatever the user provides to fetch, and retry with a trailing slash if the response fails (plus a decent warning to migrate code?)

Thanks for having a look already!
In my opinion, any fix of this behavior should not influence client-side code as the issue stems from how flask resolves redirects. I haven't tested it, but setting up flask ProxyFix or writing something similar ourselves should fix this issue. I.e. detecting X-forwarded-* headers and rewriting the URLs properly (if that even is the root cause of the problem).
Any such fix should not be enabled by default tho, as anyone might set the headers if you are not running the app behind a reverse proxy which rewrites the headers.


How to add documentation on resource identifiers or properties?

Hello,
I cannot find a proper way to define documentation for a resource identifiers or properties.
From the smithy-examples repository, how do you recommend to define documentation for the City resource identifiers or properties?
resource City {
    identifiers: { cityId: CityId }
    properties: { coordinates: CityCoordinates }
    read: GetCity
    list: ListCities
    resources: [
        Forecast
    ]
}

Both the @documentation trait and /// doesn't work inside the definition of the resource (either in identifiers or properties). Here are two invalid examples:
resource City {
    identifiers: {
    	@documentation("City ID")
    	cityId: CityId
	}
    properties: { coordinates: CityCoordinates }
    read: GetCity
    list: ListCities
    resources: [
        Forecast
    ]
}

resource City {
    identifiers: {
    	/// City ID
    	cityId: CityId
	}
    properties: { coordinates: CityCoordinates }
    read: GetCity
    list: ListCities
    resources: [
        Forecast
    ]
}

It is possible to document when using the elision syntax. However, this leads to a lot of duplication, so it is more convenient to document it once at the resource level.

Resource properties and identifiers aren’t members so they can’t bear traits. They point to root level shapes. You can document those root level shapes. This turns out to be a very good thing. It encourages shape reuse and consistency across resource operations because members of operation inputs and outputs have to target the same shapes as the resource. It also makes automatic identifier and resource property binding work.


Missing "Mapping file" option in Build Pipelines

In the documentation, it says that we can specify the mapping.txt file for an Android build, however, this option is not available on the App Center Distribute task. The documentation and the actual App Center Distribute task is inconsistent.

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

ID: da6643ac-ea32-1761-a7e9-0f17a3c0b8ee
Version Independent ID: 2442af0f-3f01-a91c-a98a-29c2f32d35b9
Content: App Center Distribute task - Azure Pipelines
Content Source: docs/pipelines/tasks/deploy/app-center-distribute.md
Product: devops
GitHub Login: @erickson-doug
Microsoft Alias: dastahel

any update on this. AppCenter has much less VSTS integration support and this is just one of reason why we don't want to migrate to it

@botatoes or @dimazaid can we do a doc update for this?

@jwargo created a pr to the vsts docs, for for it to get approved.

@botatoes thanks!

@patniko can you close this?

Not a collaborator on this repo @jwargo. @RohanChhipa or one of the repo collaborators will have to do it.


TypeScript intellisense auto import can't use

VSCode Version: 1.28
OS Version: Windows 10 1803 17134.345
Language: Chinese

Steps to Reproduce:

Cocos Creator : Create a new ts script
Can be prompted in other scripts, but not auto import the new script.

Console: err出现未知错误。有关详细信息,请参阅日志。
but i can't find log :)
Thanks Google Translate :)

Does this issue occur when all extensions are disabled?: Yes

Duplicate of #60314
Should be fixed in VS Code insiders


instance: stop API races while creating/deleting snapshots

closes #11538
Here is an test script :
package main

import (
	"fmt"
	"sync"
	"time"

	lxd "github.com/lxc/lxd/client"
	"github.com/lxc/lxd/shared/api"
)

const LxdSock = "/var/snap/lxd/common/lxd/unix.socket"

func main() {
	fmt.Println("Trying to connect to LXD...")
	if srv, err := lxd.ConnectLXDUnix(LxdSock, nil); err == nil {
		fmt.Println("LXD server connection acquired")
		imageServer, err := lxd.ConnectSimpleStreams("https://images.linuxcontainers.org/", nil)
		if err != nil {
			panic(err)
		}

		fmt.Println("Connected to image server")

		alias, _, err := imageServer.GetImageAlias("alpine/3.17/cloud/amd64")
		if err != nil {
			panic(err)
		}

		fmt.Println("Got image alias")

		image, _, err := imageServer.GetImage(alias.Target)
		if err != nil {
			panic(err)
		}

		fmt.Println("Got image")

		req := api.InstancesPost{
			Name: "snap-del-api-race",
			Type: api.InstanceType("container"),
			Source: api.InstanceSource{
				Type:        "image",
				Fingerprint: image.Fingerprint,
				Project:     "default",
			},
		}

		op, err := srv.CreateInstanceFromImage(imageServer, *image, req)
		if err != nil {
			panic(err)
		}
		op.Wait()

		fmt.Println("Instance created")

		start := api.InstanceStatePut{
			Action:  "start",
			Timeout: 5,
			Force:   true,
		}

		upd, err := srv.UpdateInstanceState("snap-del-api-race", start, "")
		if err != nil {
			panic(err)
		}
		upd.Wait()

		fmt.Println("Instance started")

		// Create a lot of snapshots for an instance
		var wg sync.WaitGroup
		for i := 0; i < 100; i++ {
			go func(i int) {
				wg.Add(1)
				fmt.Println(fmt.Sprintf("Creating snapshot snap-%d", i))
				snap := api.InstanceSnapshotsPost{
					Name: fmt.Sprintf("snap-%d", i),
				}

				opCreate, err := srv.CreateInstanceSnapshot("snap-del-api-race", snap)
				if err != nil {
					panic(err)
				}

				err = opCreate.Wait()
				if err != nil {
					panic(err)
				}

				wg.Done()
				fmt.Println(fmt.Sprintf("snap-%d created", i))
			}(i)
		}

		wg.Wait()
		fmt.Println("All snapshots created")

		// Trigger 50 parallel delete snapshot requests on that instance via the HTTP API
		tasks := make([]lxd.Operation, 0)
		tasksMu := sync.Mutex{}
		for i := 0; i < 50; i++ {
			go func(i int) {
				opDel, err := srv.DeleteInstanceSnapshot("snap-del-api-race", fmt.Sprintf("snap-%d", i))
				if err != nil {
					panic(err)
				}

				fmt.Println(fmt.Sprintf("Deleting snapshot snap-%d", i))
				tasksMu.Lock()
				tasks = append(tasks, opDel)
				tasksMu.Unlock()
			}(i)
		}

		// created a ticker of 100ms
		// every 100ms, we check if the instance is running
		ticker := time.NewTicker(100 * time.Millisecond)
		longTicker := time.NewTicker(20 * time.Second)
		wait := make(chan struct{})
		// ...And fetch the operations for all 50 delete requests via the HTTP API at the same time
		// and check that we don't have "Instance snapshot record count doesn't match instance snapshot volume record count"
		go func() {
			for {
				select {
				case <-ticker.C:
					tasksMu.Lock()
					for i, task := range tasks {
						status := task.Get()
						fmt.Println(fmt.Sprintf("Snapshot snap-%d status: %s", i, status.Status))
					}

					tasksMu.Unlock()
				case <-longTicker.C:
					fmt.Println("Timeout")
					ticker.Stop()
					longTicker.Stop()
					wait <- struct{}{}
					return
				}
			}
		}()

		<-wait
		fmt.Println("Done")
	} else {
		fmt.Println(err)
	}
}

@tomponline this scenario also causes problems that I think are an extension of the PR https://github.com/lxc/lxd/pull/11586#:~:text=err %3D opCreate.Wait()
				if err !%3D nil {
					panic(err)
				}

@tomponline the following script gives no error on my side. Does it reproduce the needed scenario on your side ?

@tomponline yep, tested this solution and it seems to work too ;) thanks

@gabrielmougard tests are failing pretty badly though :(

yeah looking at it

@gabrielmougard its probably a deadlock with the startup snapshot functionality. As Start will have taken a lock and then it probably calls Snapshot to take a start up time snapshot and blocks.

@gabrielmougard look at where startupSnapshot is called.

@gabrielmougard suggest creating an unexported func (d *lxc) snapshot(name string, expiry time.Time, stateful bool) error { function that doesn't lock and then calling that instead of inst.Snapshot() from startupSnapshot then update Snapshot() to lock and then call snapshot().

@tomponline tests should be ok now


Image resizing not working when the origin point is in the top-left corner

Bug Description

When the origin of the resizing is top-left corner AND the image is rotated 90 degrees, resizing fails and the image "disappears".
Similar issues occur with text boxes and many different rotations (when rotated more than 90 degrees from 0 in either direction)
Expected Behaviour
Resizing should work despite of the rotation / start position

Steps to Reproduce

Add an image
Assign rotation 90 degrees
Position it to the top-left corner
Resize from the bottom-right corner as it is visible currently without considering rotation
See the odd stuff happening.

Screenshots

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria
QA Instructions

Add an image
Set rotation to 90 degrees
Now drag the image to the upper left corner
Resize it from the corners
Verify the resizing works as expected and that the image doesn't "jump around".

After spending some time debugging and making clear under which conditions the resizing fails, recreated the situation in CodeSandbox and added an issue to Moveable repo instead: https://github.com/daybrush/moveable/issues/251
For information, the bug happens when keepRatio is true and snapping is enabled -- also happens only when the element is in one of the snapping points, e.g. in the left edge of the Page (doesn't have to be a corner).
Will wait for information from Moveable and then see how to proceed.

@barklund FI: I'm resizing this to 1 since once it's fixed in Moveable, all we need to do is update the module.
Let me know if you have any objections.

Verified in QA (Master)

UAT - looks good. Closing ticket


signed responses?

Maybe i don't see it in the code, but it looks like that responses are not signed at all?
How would this prevents a MITM to temper a response?

Yep, requests are signed whereas the responses depend on the security of TLS/X509 certificates currently.


How to train to make Yolo differ face and person?

Hi,
The application need to detect the faces in the video, then censor them.
Alturos.Yolo is detecting face picture and person picture as 'person' label.
How can I make the Alturos.Yolo differ face and person?
Thanks,

I don’t think it’s that easy.
But you can check this project.
https://github.com/abars/YoloKerasFaceDetection

Thank you for information.
I will check.

I am using haar cascade classifier to detect person faces using opencvsharp or other opencv wrapper the emgucv to detect faces. Now how do we train this faces and compare or recognize using Alturos as I understand this needs to be train the names and cfg I think.


(Android) Doesn't load images from gallery when launch mode is in singleInstance

Hi there. Congrats for this excellent plugin, it has been a most have in all of my project. However, I have encountered an issue in Android devices when your declare in your main activity android:launchMode="singleInstance" instead of singleTop like the default config. Images from the default files app have no problems, however, if you switch to your gallery app, then images are not loaded after selection. ImagePicker.pickImage() just returns null.
Steps to Reproduce
Just go to your android/app/src/main/AndroidManifest.xml and change the lauchmode options. Should be something like this:
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
   ...
   <application
        ...
        <activity
          android:name="com.yalantis.ucrop.UCropActivity"
          android:screenOrientation="portrait"
<EMAIL_ADDRESS>        />
        <activity
            android:name=".MainActivity"
            android:launchMode="singleInstance"
            android:theme="@style/LaunchTheme"
            android:configChanges="orientation|keyboardHidden|keyboard|screenSize|smallestScreenSize|locale|layoutDirection|fontScale|screenLayout|density|uiMode"
            android:hardwareAccelerated="true"
            android:windowSoftInputMode="adjustResize">

Would be great to give some alternative to this feature...maybe it's something that we must include in the activity and I'm missing.

Hi @yarn-rp, it looks like there's a similar issue already open describing what you're experiencing in https://github.com/flutter/flutter/issues/76856. It looks like it's not just an issue with the gallery, picking from google photos doesn't work either.
Kindly follow up there for more information.
Closing from here as duplicate.
Thank you


Issue: Add hovering effects over the jounral cards make it more interactive

What feature would you like to see?
when the use hover over the journal card,the card scale will change a liitle bit giving it effect of floating

Why is this feature useful?
make the interface more engaging to the user
Have you seen this feature in other projects?
Yes, I've seen something similar elsewhere
Your experience level with this project:
I'm new here (first-time contributor)
Would you like to work on this feature?
Yes, I'd like to try implementing this feature

Hey @Hh440 that's a good one ! Im assigning it to you

Please also add hacktoberfest label


Fix default value for Ephemeral in documentation

What type of PR is this:
/kind documentation
What does this PR do / why we need it:
Since #5795, the default value for 'Ephemeral' is now False, but the documentation was outdated.
Which issue(s) this PR fixes:
No issue related
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[x] Documentation

How to test changes / Special notes to the reviewer:

/approve
/lgtm

/override windows-integration-test/Windows-test
Doc only


Add Auth screens.

Description of changes:

Add Sign in, Signup, forgot password, and reset password screens
Add PrivateRoute HOC
replace Id usage with  {entity}Id
Fix postAuth trigger bug

Please check out Auth demo here
Updated Postman collection here
Checklist

[ ] Tests have been added and are passing
[ ] Documentation has been updated

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

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


Minor typo

On "How Billing Works" section, just fix Aure to become Azure. thanks

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

ID: 18f26025-af57-17c6-0b51-b64d46b450fe
Version Independent ID: d60c32f2-f86e-7773-db4b-8f59dbcabf5d
Content: Choose a pricing tier or SKU for Azure Search service - Azure Search
Content Source: articles/search/search-sku-tier.md
Service: search
GitHub Login: @HeidiSteen
Microsoft Alias: heidist

@amjad-Alhusseini, Thanks for bringing this to our attention.We have updated the doc accordingly.
The changes should be live EOD or tomorrow.

@amjad-Alhusseini, The requested changes now reflects publicly in the document,  so we will close this out, but if you feel you need more information please just let us know. Thanks again for your feedback.


Update object to 0.9.0

Also use the Object trait instead of File (see https://github.com/gimli-rs/object/issues/53).
This requires https://github.com/gimli-rs/gimli/pull/305.
I played a bit with using Rc/Arc<Cow<'input, [u8]>> instead of EndianRcSlice, but wasn't sure it was worth the API complexity. So I think Context::new should be the simple high level interface, and consumers with more specific needs can use Context::from_sections.
Before:
test context_new_and_query_location       ... bench:     951,031 ns/iter (+/- 79,277)
test context_new_and_query_with_functions ... bench:   1,760,225 ns/iter (+/- 54,934)
test context_new_location                 ... bench:     760,223 ns/iter (+/- 57,231)
test context_new_with_functions           ... bench:     751,183 ns/iter (+/- 55,055)
test context_query_location               ... bench:  10,378,538 ns/iter (+/- 92,496)
test context_query_with_functions         ... bench:  11,578,429 ns/iter (+/- 142,557)

After:
test context_new_and_query_location       ... bench:   5,527,469 ns/iter (+/- 285,657)
test context_new_and_query_with_functions ... bench:   6,809,893 ns/iter (+/- 490,899)
test context_new_location                 ... bench:   5,230,294 ns/iter (+/- 307,209)
test context_new_with_functions           ... bench:   5,213,000 ns/iter (+/- 266,617)
test context_query_location               ... bench:  15,353,989 ns/iter (+/- 309,998)
test context_query_with_functions         ... bench:  16,591,755 ns/iter (+/- 402,449)

For reference, Arc<Cow<[u8]>>:
test context_new_and_query_location       ... bench:   1,063,159 ns/iter (+/- 57,010)
test context_new_and_query_with_functions ... bench:   2,483,267 ns/iter (+/- 180,353)
test context_new_location                 ... bench:     758,812 ns/iter (+/- 16,336)
test context_new_with_functions           ... bench:     755,755 ns/iter (+/- 31,252)
test context_query_location               ... bench:  18,362,107 ns/iter (+/- 417,995)
test context_query_with_functions         ... bench:  19,264,415 ns/iter (+/- 260,564)

The query numbers for Arc<Cow<[u8]>> are surprising to me, but I didn't look into it.

@philipc would you want to make a hacky experimental PR that builds on this one which has no_std support? Then I can start working on a prototype for libstd integration :).

Coverage increased (+0.8%) to 82.201% when pulling fc4763003225bbdd2f23844acfe5f4f618f5b64b on philipc:object into c1203c63e8b551b7ede91d7450a381121b358bec on gimli-rs:master.

NVM made it here https://github.com/est31/addr2line/tree/no_std_prototype


Update README.md

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=ru
Description of changes:

{"killed":false,"code":1,"signal":null,"cmd":"node /tmp/yfm/build/index.js -- -i -i /tmp/source/input -o /tmp/output","stdout":"","stderr":"internal/modules/cjs/loader.js:896\n  throw err;\n  ^\n\nError: Cannot find module '/tmp/yfm/build/index.js'\n    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:893:15)\n    at Function.Module._load (internal/modules/cjs/loader.js:743:27)\n    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:72:12)\n    at internal/main/run_main_module.js:17:47 {\n  code: 'MODULE_NOT_FOUND',\n  requireStack: []\n}\n"}

WARN Variable specified_key not found in /en/compute/concepts/instance-groups/variables-in-the-template.md
WARN Variable unknown_key not found in /en/compute/concepts/instance-groups/variables-in-the-template.md
WARN Variable unknown_key not found in /en/compute/concepts/instance-groups/variables-in-the-template.md
WARN Variable specified_key not found in /en/compute/concepts/instance-groups/variables-in-the-template.md
WARN Variable unknown_key not found in /en/compute/concepts/instance-groups/variables-in-the-template.md
WARN Variable unknown_key not found in /en/compute/concepts/instance-groups/variables-in-the-template.md
WARN Variable label not found in /en/monitoring/concepts/querying.md
WARN Variable host not found in /en/monitoring/concepts/visualization/dashboard.md
WARN Variable myparm not found in /en/monitoring/concepts/visualization/dashboard.md
WARN Variable label not found in /en/monitoring/concepts/querying.md
WARN Variable isotime | clean_resource_name not found in /en/solutions/infrastructure-management/packer-quickstart.md
WARN Variable isotime | clean_resource_name not found in /en/solutions/infrastructure-management/packer-quickstart.md
WARN Variable .TablePathPrefix not found in /en/ydb/sdk/index.md
WARN Variable ydb-cli not found in /en/ydb/quickstart/examples-ydb-cli.md
WARN Variable ydb-cli not found in /en/ydb/quickstart/examples-ydb-cli.md
WARN Variable ydb-cli not found in /en/ydb/quickstart/examples-ydb-cli.md
WARN Variable ydb-cli not found in /en/ydb/quickstart/examples-ydb-cli.md
WARN Variable ydb-cli not found in /en/ydb/quickstart/examples-ydb-cli.md
WARN Variable .TablePathPrefix not found in /en/ydb/sdk/index.md
WARN Variable ydb-cli not found in /en/ydb/maintenance/backup_and_recovery.md
WARN Variable label not found in /ru/monitoring/concepts/querying.md
WARN Variable host not found in /ru/monitoring/concepts/visualization/dashboard.md
WARN Variable myparm not found in /ru/monitoring/concepts/visualization/dashboard.md
WARN Variable label not found in /ru/monitoring/concepts/querying.md
WARN Variable isotime | clean_resource_name not found in /ru/solutions/infrastructure-management/packer-quickstart.md
WARN Variable isotime | clean_resource_name not found in /ru/solutions/infrastructure-management/packer-quickstart.md
WARN Variable .TablePathPrefix not found in /ru/ydb/sdk/index.md
WARN Variable .TablePathPrefix not found in /ru/ydb/sdk/index.md
Build time: 27.422s


Can this package update to vcpkg repo?

I am using vcpkg to manage pkg dependencies. It's a burden to mantain 3rd part pkg compilation.
So hope it can be updated to vcpkg repo.

Emm... as fmtlog has been stable for a while and no planned changes in the future, you just need to copy the two header files to your project and done.


Seamless scrolling of website

Due to large content on the rooms, tracks, there is a bit laggy behaviour of scrolling leading to poor UX. I want to have everyone's inputs. :)

@sumedh123 Okay. What is your suggestion ?

I am a little confused regarding this. Could you guys please upload a video or a gif showing the problem, please? Thanks!!


Make Survey submit

Functionality for taking surveys was done. The only thing left is to display my surveys in 'Manage Profile' same as 'My Exams'.


feat: Support binary files

I before using Microbin is awesome because it support share the binary file like image. But the project is seems no long time maintain. Do you have any plan support upload the binary file?
Upload feature:

Enable by optional argument. Example: --allow-binary, more advanced will support --allow-binary .png.jpeg.mp4, means only allow png, jpeg/jpg and mp4 extension files. value split by ., empty or * means match all.
Have extra argument can define the file size maximum limit. Example: --binary-limit, or merge with enable argument like --allow-binary 1mb.jpg/1gb.mp4/png, value is split by /,  then if contain ., split again for binary size limit, if not unlimited binary size. In this example is allow maximum 1mb for jpg, 1gb for mp4, unlimited for png.
Web UI can preview the file if it is image, video. More advanced will auto generate the preview file instead use original for preview, this will save the network bandwidth and system performance. For example, image file will generate maximum width and height 512px from original image to be preview image.

And I wanna say, your web ui is look nice!

I am OP, comment for subscribe issue.


Allow the current state to be saved and resumed

This is good for a burn-in phase and being able to continue running chains from a starting point.

I'm not sure this is required. Yes it is needed for feature parity, but surely the user can just discard the first few iterations?

Things that have to happen for this change to be possible:

[x] return the finishing state from sample and run_mcmc
[ ] ensure when we allocate new chain and lnprob objects, we may want to extend them rather than overwrite them
[ ] allow a State object to be passed in as arguments to sample and run_mcmc

This last one Rust is not very good at. Without default arguments or function overloading we cannot use the same method, meaning a new method must be added for each. If we provide too many options, we get an explosion of possibilities. It would be nice to think of a better way to do this, perhaps with the Builder pattern.

I believe the emcee code does not extend chain or lnprob objects. It is expected that the user calls reset to reset the sampler and remove that state.


Moloch StatefulSet, changes across directories, general improvements

Added Moloch directory for container + Helm Chart
Fixed Elasticsearch StatefulSet
Updated some documentation
Some bug fixes

Not all commits are shown, old fork was orphaned when the repo went public.

Closing in favor of updated Moloch deployment that has more feature/improvements


:wrench: chore: mobile-apps - create base guidelines for Baloise native apps

Summary

[ ] App Icon Rules
[ ] Splashscreen
[ ] IOs / Android Store Screens

Draft
https://www.figma.com/file/w6bGlt9F7pTMwfgUFLTpAD/🔵-Brand-Assets?node-id=1560%3A3146&viewport=1929%2C1255%2C0.24
Additional Information
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

In Progress

We close this issue due to inactivity


Call for PR: use Pulldown-CMark to extract match expressions

Context
Currently, mdBook-KaTeX uses simple heuristics I hand-rolled to identify and extract math expressions. Various Markdown markups are not handled, e.g., indented code blocks and HTML.
Recently, Pulldown-CMark started supporting math. This is some great news, as Pulldown-CMark should be much more robust and feature-rich in this regard.
To reliably convert Pulldown-CMark events back to Markdown text, it would also be nice to wait until Pulldown-CMark-to-Cmark supports preserving escape characters.
Pushback
The reasons I am not very interested in implementing this myself are as follows:

I am fine with the current heuristics for my use cases.
A proper implementation shall take quite some efforts, and will likely even introduce new bugs.
I am sick of messing with Pulldown-CMark as of recently, from making the FMTM formatter.

If anyone is interested in this, please try and make a PR!

I think this and #72 (created by me) can be merged

Another regression would be Pulldown-CMark does not support custom math delimiters like we do now.


[Issue #854] Fixed uniquie ID issue on Vote and Curriculum

Fixes #854.
The issue was caused due to the new poll/category/skill taking it's id as the length of the corresponding array of that entity. This caused 2 entities to have the same ID.
Fixed the issue by finding the max id from the array and setting maxId + 1 as the new ID for the entity. This makes sure the new id is unique. Fixed the issue in both Vote and Curriculum.

Nice. Thanks a lot to be so reactive.


Jenkins build fails when applying patch

I've been trying to integrate Phabricator with our Jenkins server without success. The output (in Jenkins) before the build fails is this:
[hidden] $ /home/jenkins/phabricator/arcanist/bin/arc patch --nobranch --diff 204 ********
Exception
 Select a Default Commit Range

You're running a command which operates on a range of revisions (usually,
from some revision to HEAD) but have not specified the revision that should
determine the start of the range.

Previously, arc assumed you meant 'HEAD^' when you did not specify a start
revision, but this behavior does not make much sense in most workflows
outside of Facebook's historic git-svn workflow.

arc no longer assumes 'HEAD^'. You must specify a relative commit explicitly
when you invoke a command (e.g., `arc diff HEAD^`, not just `arc diff`) or
select a default for this working copy.

In most cases, the best default is 'origin/master'. You can also select
'HEAD^' to preserve the old behavior, or some other remote or branch. But you
almost certainly want to select 'origin/master'.

(Technically: the merge-base of the selected revision and HEAD is used to
determine the start of the commit range.)

    What default do you want to use? [origin/master] The program is attempting to read user input, but stdin is being piped from some other source (not a TTY).
(Run with `--trace` for a full exception trace.)
[phabricator:arcanist] Error applying arc patch; got non-zero exit code FAILURE
[phabricator:process-build-result] No unit results available.
[phabricator:process-build-result] No coverage provider available.

I am running the latest version of anarcist, so that should not be the issue.
Any ideas?

Hmm... I feel like I've seen that prompt from arcanist locally, but never or jenkins. What happens if you run the same command (without the *** part) on your own working copy?
It also doesn't really make sense that arc would need a commit range for patch, which should apply the downloaded patch to the working copy.
Do you have any other steps to reproduce?

Are you having issues with this still? Any thoughts on my questions above?

I never got it to work correctly with your plugin, so we're applying the patches with a bash script instead.

@zegl have you set git-plugin correctly?
I follow the steps on jenkins plugins page and setup this plugin to work successfully.

Closing due to inability to reproduce.

I am also seeing this on Jenkins. Any thougths?

Discussed offline, this is a job misconfiguration not an issue with the plugin.

Hi I am trying to configure Phabricator with Jenkins getting below error message on Jenkins log any help?
[phabricator:uberalls] No cobertura results found
com.uber.jenkins.phabricator.conduit.ConduitAPIException: Unable to find 'R3:169' key in 'result': (null object) null
at com.uber.jenkins.phabricator.conduit.DifferentialClient.fetchDiff(DifferentialClient.java:85)
at com.uber.jenkins.phabricator.PhabricatorNotifier.perform(PhabricatorNotifier.java:165)
at hudson.tasks.BuildStepMonitor$1.perform(BuildStepMonitor.java:20)
at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:779)
at hudson.model.AbstractBuild$AbstractBuildExecution.performAllBuildSteps(AbstractBuild.java:720)
at hudson.model.Build$BuildExecution.post2(Build.java:185)
at hudson.model.AbstractBuild$AbstractBuildExecution.post(AbstractBuild.java:665)
at hudson.model.Run.execute(Run.java:1745)
at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
at hudson.model.ResourceController.execute(ResourceController.java:98)
at hudson.model.Executor.run(Executor.java:404)
[phabricator:conduit] Unable to fetch differential from Conduit API
[phabricator:conduit] Unable to find 'R3:169' key in 'result': (null object) null

Arcanist is mandatory to install on build server?


allow execution of function to generate matches

I like this idea. But I think it requires some thought before something like this is merged. All of a sudden this.currentMatch will be part of the API and the way matches are pushed onto it must be a locked API.
If you're still interested in doing this here's a few pointers to help you move forward:

I think the match function should return an array of matches rather than "pushing on this".
The object which is returned in above should only have keys which are described in the error matcher documentation (file, col, col_end, etc).
How to use a function as an error matcher must be documented in the README
Specs must be added

Yea, this was simply a proof of concept impl
Returning a list sounds sensible, but I don't know much about javascript, it's all stackoverflowed together
Wrt only having the described fields, i don't see the point in checking.
I'll add docs once the design is finalized

I think the match function should return an array of matches rather than "pushing on this".

done

The object which is returned in above should only have keys which are described in the error matcher documentation (file, col, col_end, etc).

This isn't true for regex now, and I don't see why it would be an issue to have more fields

How to use a function as an error matcher must be documented in the README

done

Specs must be added

I have no clue what Specs are in this context.

Specs are tests. See the spec folder. I suggest you create a new file, copy the topmost beforeEach and afterEach from another file to get the correct testing env, then write some specs to guarantee the functionality over coming releases

I'm not sure what I'm doing wrong, a third of the specs are failing by default on my machine and the new specs I added are timing out. Is there a way to run just one spec file?

Yes, if you change it to fit it will focus that spec.

thanks, that helped a lot. Specs are passing now.

Just one minor thing, then I'm super happy with this change!

done

Thank you so much for this. Really great addition!


Introduce merge() expr built-in function

This pull request adds merge() built-in expr function which is equivalent to the samber/lo's Assign() function.
example
test: |
  compare(
    merge({"a": 1, "b": 2}, {"b": 3, "c": 4}),
    {"a": 1, "b": 3, "c": 4}
  )

FYI: This PR is the 3rd one that I implemented yesterday.

~~https://github.com/h6ah4i/runn/tree/feature/add-built-in-function-pick~~ (Thx, already merged :pray:)
~~https://github.com/h6ah4i/runn/tree/feature/add-built-in-function-omit~~ (Thx, already merged :pray:)
https://github.com/h6ah4i/runn/tree/feature/add-built-in-function-merge 👈  This pull request
https://github.com/h6ah4i/runn/tree/feature/support-yaml-anchor-alias-handling

@h6ah4i
Could you tell us what use case you are using merge() for? (I could imagine pick() and omit()) .

@k1LoW

Could you tell us what use case you are using merge() for? (I could imagine pick() and omit()) .

For instance, I want to construct an API request body like the following using expr.
steps:
  get_data_1:
    /get_data_1:
      get:
        body: null

  get_data_2:
    /get_data_2:
      get:
        body: null

  post_data:
    req:
      /post_merged_data:
        post:
          body:
            application/json: `{{ merge(pick(steps.get_data_1.res.body, 'a', 'b', 'c'), omit(steps.get_data_2.res.body, 'created_at')) }}`

Oh, it is quite complex.
Certainly useful when creating a request body.

@h6ah4i Thank you!!!


Blacklist (or allow us to blacklist) Tinkers' Construct tools for research item consuming

I just lost my really good Tinker's pickaxe because it had Fortune 3 on it after unlocking the Break focus. I had enchanted a Stone pick for it to use, I didn't think that it would take the TiCon pickaxe at all. I'm really upset by this because I spent a lot of time and effort to make that pickaxe, and now it's gone and I can't just spawn it back in.
If we could blacklist items that get removed, that would stop this from happening again in the future. Or maybe allow us to choose which specific items to use instead of just taking them from out inventory.

Upvoted.  I've had this happen to me too, fortunately I was able to cheat in the pick I lost (but lost the levels associated with it).


[Identity] DefaultAzureCredential should use InteractiveBrowserCredential

While reviewing the Identity package for JS, we realized that DefaultAzureCredential has different behaviors in JS and .Net. In .Net, if no credentials are available, it uses InteractiveBrowserCredential. We should align in JS!
Here's .Net's code: https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/identity/Azure.Identity/src/DefaultAzureCredential.cs#L24
Keep in mind that in .Net they use: ExcludeInteractiveBrowserCredential to control whether to use this credential.
Also, if we haven't GAd this, it might make more sense to use DeviceCodeCredential instead of InteractiveBrowserCredential.

Have we checked with our friends over at Java and Python SDKs as well?

@ramya-rao-a Python also uses InteractiveBrowserCredential as the last credential:
https://github.com/Azure/azure-sdk-for-python/blob/master/sdk/identity/azure-identity/azure/identity/_credentials/default.py#L123 they also have the exclude_interactive_browser_credential parameter.

They have many "exclude" parameters! That's interesting.

Also, if we haven't GAd this, it might make more sense to use DeviceCodeCredential instead of InteractiveBrowserCredential.

All users of auto generated JS packages for both data plane and mgmt plane packages so far have been using the Device Code auth as the getting started experience. This can be seen in the code snippet in the generated readme of these files.
But I can see how InteractiveBrowserCredential can be a smoother experience given that I dont have to click on a url and copy paste in the code.
So, what would be the benefit of choosing DeviceCodeCredential over InteractiveBrowserCredential?
cc @schaabs

@ramya-rao-a The only advantage of DeviceCodeCredential over InteractiveBrowserCredential is that DeviceCode doesn't need a browser, thus can be used in applications without graphical server or browsers. Let's say somebody deploys a project with Identity in a Heroku instance, they could see the logs and authenticate the remote app manually even though they might have nothing in the environment to authenticate with.

This is an area where we should definitely align across languages so that expectation from DefaultAzureCredential is same for a customer when they move from one language to another. Please share an update after discussing with the rest of the Identity feature crew!

After we talked internally, we decided not to add DeviceCodeCredential to the DefaultAzureCredential pool. We will nonetheless add InteractiveBrowserCredential, and align with the optional parameters.

Here's the summary of what we talked internally:

InteractiveBrowserCredential offers a better experience for new users than DeviceCodeCredential.
People deploying to an environment without a browser would probably not mind to use DeviceCodeCredential directly.
This means that if we were to add any of these two to DefaultAzureCredential, we would add InteractiveBrowserCredential.
However, DefaultAzureCredential should never hang waiting for the user's input. So the way we've decided to add it is by adding includeInteractiveCredentials as an option parameter on DefaultAzureCredential to opt-in to InteractiveBrowserCredential.
There are many differences on the optional parameters of InteractiveBrowserCredential across languages. In particular, several languages have options to drop credentials from the DefaultAzureCredential chain. They have GAd with these credentials, however the Arch board would rather avoid making all languages consistent on those options. Their preference (and I'm favorable to it too) is that if users wanted to pick specific credentials, they would use ChainedTokenCredential directly.

https://github.com/Azure/azure-sdk-for-js/pull/14561 adds InteractiveBrowserCredential to DefaultAzureCredential. This is enabled only via enabling an option includeInteractiveCredentials which is disabled by default.
This issue originally came up while reviewing #14356 where we are updating the management plane auto generated packages to support credentials from the @azure/identity package. Before this PR, the code snippet for getting started made use of device code authentication which needed no set up on a user's machine. Moving to DefaultAzureCredential requires users to either login via Azure CLI/VS Code or set up service principal.
https://github.com/Azure/azure-sdk-for-js/pull/14561 helps this matter, but requires us to document the option as well as recommend users not to use this in production. This does not add any value to the alternative of showing the use of DeviceCodeCredential and then recommending the use of DefaultAzureCredential in further development needs and in production.
Since aligning across languages regarding adding InteractiveBrowserCredential  to DefaultAzureCredential was also a non-goal, we are not gaining much with this change. So, my recommendation would be to revert the changes in https://github.com/Azure/azure-sdk-for-js/pull/14561 and not rush this in

@ramya-rao-a thank you!


[UI] Fix #63: Wrong URL in comments

@timotiusnc

I tried it by using the same text from https://hacker-news.firebaseio.com/v0/item/9910742.json?print=pretty, but this error came up:
07-28 04:36:08.205    2156-2156/io.pumpkinz.pumpkinreader E/InputEventReceiver﹕ Exception dispatching input event.
07-28 04:36:08.206    2156-2156/io.pumpkinz.pumpkinreader E/MessageQueue-JNI﹕ Exception in MessageQueue callback: handleReceiveCallback
07-28 04:36:08.208    2156-2156/io.pumpkinz.pumpkinreader E/MessageQueue-JNI﹕ android.content.ActivityNotFoundException: No Activity found to handle Intent { act=android.intent.action.VIEW dat=\"https://web.archive.org/web/20070221033032/http://news.ycombinator.com/user?id=sam\" (has extras) }
            at android.app.Instrumentation.checkStartActivityResult(Instrumentation.java:1765)
...

I think that's out of this PR scope.
Please create new issue.


style(menu member.scss): changed style for member and menu admin header

changed the size of the provider logo and change the background of the member and help dropdown on
hover and open
Proposed Changes

Potential Breaking Changes

Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

Codecov Report

Merging #1512 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1512   +/-   ##
=======================================
  Coverage   98.65%   98.65%
=======================================
  Files         504      504
  Lines       10401    10401
  Branches     2405     2405
=======================================
  Hits        10261    10261
  Misses        140      140

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 4a24f04...b548e50. Read the comment docs.


Allow childprocess 5

I think it only brings improvements: https://github.com/enkessler/childprocess/releases/tag/v5.0.0

Thanks!


Update wabt package

Update wabt from 0.2 to 0.9 and with it wabt-sys. Old versions of wabt-sys failed to compile with newer versions of gcc.
The change also requires and update to the WebAssembly text format.

thanks, changes included in #139


remove docs about @obj and @deriving jsConverter/abstract

these decorators are not really needed or even supported (for jsConverter) in v11.

Thanks!


support router and ps resource isolation by resource name

Router and PS can only be used with the same resource_name space
step1: set global config
[global]
# specify which resources to use to create space
resource_name = "default"
step2: create space
curl -XPUT -H "content-type: application/json" -d'
{
"name": "space1",
"partition_num": 1,
"replica_num": 1,
"resource_name":"default",
"engine": {
...
},
"properties": {
...
}
}
' http://master_server/space/$db_name/_create

https://github.com/vearch/vearch/issues/496