Gabby/AddOwnerCommand

The  Add Owner command is used in the chatroom to make a Jabbr user an Owner of that room.. To test you should be able to click into light meow and in the textbox type /addowner [username] and it should return "[username] is now an owner of light_meow". If you put a username that does not exist, it should return " Unable to find [username]"

Is this ready for testing? Needs to be tested before we can merge :)

Tested UI:

[x] entered room
[x] "/addowner Jack" -> "Jack is now an owner of light_meow"
[x] "/addowner Jane" -> "Jane is now an owner of light_meow"
[x] "/addowner Adam" -> HubException Hit -> UI prints to chat "Unable to find Adam"

I was able to run back to back "/addowner Jack" with the same results. Should this be able to repeatedly do? I know we don't have real data. If this is expected, then :shipit:

This totally does what gabby says and adds the user as a room owner. Owners are not added to database so you can add a person as an owner more than once, which shouldn't happen once we actually get the database and caching to work. :shipit:

@ashanhol @gcrev93 why are you able to add the Owner more than once? Even using the InMemoryRepository it should still be able to detect the user already exists as an owner right?

Can you verify old Jabbr behavior here? Looking at the schema real quick, looks like this is enforced by referential integrity in the database, which is fine, but also enforced in the command. Be nice to verify what happens in the old Jabbr as far as error message though. The repository shouldn't check this, but the AddOwner command should. Looking at the old code, it looks like it does? See "  // If the target user is already an owner, then throw"
//this single line of code from AddOwnerCommand.cs
context.Service.AddOwner(callingUser, targetUser, targetRoom);
..
//ChatService.cs
public void AddOwner(ChatUser ownerOrCreator, ChatUser targetUser, ChatRoom targetRoom)
{
    // Ensure the user is owner of the target room
    EnsureOwnerOrAdmin(ownerOrCreator, targetRoom);

    if (targetRoom.Owners.Contains(targetUser))
    {
        // If the target user is already an owner, then throw
        throw new HubException(String.Format(LanguageResources.RoomUserAlreadyOwner, targetUser.Name, targetRoom.Name));
    }

    // Make the user an owner
    targetRoom.Owners.Add(targetUser);
    targetUser.OwnedRooms.Add(targetRoom);

    if (targetRoom.Private)
    {
        if (!targetRoom.AllowedUsers.Contains(targetUser))
        {
            // If the room is private make this user allowed
            targetRoom.AllowedUsers.Add(targetUser);
            targetUser.AllowedRooms.Add(targetRoom);
        }
    }
}

So I looked into running it twice and it should give an error that the person is already an owner. But if its hardcoded in the InMemoryRepo, then will it ever really show the change? Changes we make on the site arent actually saved to the InMemoryRepo.. so we wouldnt see that error correct?


Make "About" link point to current website and not open in a new tab

When I was working on https://github.com/rust-lang/docs.rs/pull/2218, I realized (after some time...) that it was actually redirecting me to docs.rs every time and not simply use my local instance.
So this PR changes two things:

Use a relative URL instead of pointing to docs.rs.
Since it's the same website, not open this link into a new tab.

I like the change, will approve after testing when I'm back next week


Feature/day17

Codecov Report

Merging #21 (6eea4f3) into master (d827af2) will increase coverage by 0.09%.
The diff coverage is 94.44%.

@@            Coverage Diff             @@
##           master      #21      +/-   ##
==========================================
+ Coverage   92.00%   92.10%   +0.09%
==========================================
  Files          17       18       +1
  Lines        1289     1343      +54
==========================================
+ Hits         1186     1237      +51
- Misses        103      106       +3

Impacted Files
Coverage Δ

day17/src/main.rs
94.44% <94.44%> (ø)

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 d827af2...6eea4f3. Read the comment docs.


DTSPO-22117 - add db single server recreation

Jira link
DTSPO-22117
Change description

Testing done

Checklist

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)
[ ] Does this PR introduce a breaking change

Plan Result (nonprod_general - TerraformPlanApply)
Plan: 1 to add, 1 to change, 0 to destroy.

Create

azurerm_postgresql_server.atlassian-preprod-server

Update

azurerm_application_gateway.ag

Change Result (Click me)

  # azurerm_application_gateway.ag will be updated in-place
  ~ resource "azurerm_application_gateway" "ag" {
        id                                = "/subscriptions/b7d2bd5f-b744-4acc-9c73-e068cec2e8d8/resourceGroups/atlassian-nonprod-rg/providers/Microsoft.Network/applicationGateways/atlasssian-nonprod-app-gateway"
        name                              = "atlasssian-nonprod-app-gateway"
        tags                              = {
            "application"  = "atlassian"
            "builtFrom"    = "hmcts/atlassian-infrastructure"
            "businessArea" = "Cross-Cutting"
            "criticality"  = "Medium"
            "environment"  = "staging"
            "startupMode"  = "always"
        }
        # (8 unchanged attributes hidden)

      - probe {
          - id                                        = "/subscriptions/b7d2bd5f-b744-4acc-9c73-e068cec2e8d8/resourceGroups/atlassian-nonprod-rg/providers/Microsoft.Network/applicationGateways/atlasssian-nonprod-app-gateway/probes/appgw-probe-cnf" -> null
          - interval                                  = 30 -> null
          - minimum_servers                           = 0 -> null
          - name                                      = "appgw-probe-cnf" -> null
          - path                                      = "/confluence" -> null
          - pick_host_name_from_backend_http_settings = true -> null
          - port                                      = 0 -> null
          - protocol                                  = "Http" -> null
          - timeout                                   = 30 -> null
          - unhealthy_threshold                       = 3 -> null
            # (1 unchanged attribute hidden)

          - match {
              - status_code = [
                  - "200-399",
                ] -> null
                # (1 unchanged attribute hidden)
            }
        }
      - probe {
          - id                                        = "/subscriptions/b7d2bd5f-b744-4acc-9c73-e068cec2e8d8/resourceGroups/atlassian-nonprod-rg/providers/Microsoft.Network/applicationGateways/atlasssian-nonprod-app-gateway/probes/appgw-probe-crd" -> null
          - interval                                  = 30 -> null
          - minimum_servers                           = 0 -> null
          - name                                      = "appgw-probe-crd" -> null
          - path                                      = "/crowd" -> null
          - pick_host_name_from_backend_http_settings = true -> null
          - port                                      = 0 -> null
          - protocol                                  = "Http" -> null
          - timeout                                   = 30 -> null
          - unhealthy_threshold                       = 3 -> null
            # (1 unchanged attribute hidden)

          - match {
              - status_code = [
                  - "200-399",
                ] -> null
                # (1 unchanged attribute hidden)
            }
        }
      - probe {
          - id                                        = "/subscriptions/b7d2bd5f-b744-4acc-9c73-e068cec2e8d8/resourceGroups/atlassian-nonprod-rg/providers/Microsoft.Network/applicationGateways/atlasssian-nonprod-app-gateway/probes/appgw-probe-jira" -> null
          - interval                                  = 30 -> null
          - minimum_servers                           = 0 -> null
          - name                                      = "appgw-probe-jira" -> null
          - path                                      = "/jira/status" -> null
          - pick_host_name_from_backend_http_settings = true -> null
          - port                                      = 0 -> null
          - protocol                                  = "Http" -> null
          - timeout                                   = 30 -> null
          - unhealthy_threshold                       = 3 -> null
            # (1 unchanged attribute hidden)

          - match {
              - status_code = [
                  - "200-399",
                ] -> null
                # (1 unchanged attribute hidden)
            }
        }
      + probe {
          + id                                        = (known after apply)
          + interval                                  = 30
          + minimum_servers                           = 0
          + name                                      = "appgw-probe-cnf"
          + path                                      = "/confluence"
          + pick_host_name_from_backend_http_settings = true
          + protocol                                  = "Http"
          + timeout                                   = 30
          + unhealthy_threshold                       = 3
            # (1 unchanged attribute hidden)
        }
      + probe {
          + id                                        = (known after apply)
          + interval                                  = 30
          + minimum_servers                           = 0
          + name                                      = "appgw-probe-crd"
          + path                                      = "/crowd"
          + pick_host_name_from_backend_http_settings = true
          + protocol                                  = "Http"
          + timeout                                   = 30
          + unhealthy_threshold                       = 3
            # (1 unchanged attribute hidden)
        }
      + probe {
          + id                                        = (known after apply)
          + interval                                  = 30
          + minimum_servers                           = 0
          + name                                      = "appgw-probe-jira"
          + path                                      = "/jira/status"
          + pick_host_name_from_backend_http_settings = true
          + protocol                                  = "Http"
          + timeout                                   = 30
          + unhealthy_threshold                       = 3
            # (1 unchanged attribute hidden)
        }

      - request_routing_rule {
          - http_listener_id            = "/subscriptions/b7d2bd5f-b744-4acc-9c73-e068cec2e8d8/resourceGroups/atlassian-nonprod-rg/providers/Microsoft.Network/applicationGateways/atlasssian-nonprod-app-gateway/httpListeners/appgw-http-listener" -> null
          - http_listener_name          = "appgw-http-listener" -> null
          - id                          = "/subscriptions/b7d2bd5f-b744-4acc-9c73-e068cec2e8d8/resourceGroups/atlassian-nonprod-rg/providers/Microsoft.Network/applicationGateways/atlasssian-nonprod-app-gateway/requestRoutingRules/appgw-routing-rule" -> null
          - name                        = "appgw-routing-rule" -> null
          - priority                    = 1 -> null
          - rule_type                   = "PathBasedRouting" -> null
          - url_path_map_id             = "/subscriptions/b7d2bd5f-b744-4acc-9c73-e068cec2e8d8/resourceGroups/atlassian-nonprod-rg/providers/Microsoft.Network/applicationGateways/atlasssian-nonprod-app-gateway/urlPathMaps/appgw-url-map-path" -> null
          - url_path_map_name           = "appgw-url-map-path" -> null
            # (8 unchanged attributes hidden)
        }
      + request_routing_rule {
          + backend_address_pool_id     = (known after apply)
          + backend_address_pool_name   = "appgw-backend-pool-cnf"
          + backend_http_settings_id    = (known after apply)
          + backend_http_settings_name  = "appgw-backend-settings-cnf"
          + http_listener_id            = (known after apply)
          + http_listener_name          = "appgw-http-listener"
          + id                          = (known after apply)
          + name                        = "appgw-routing-rule"
          + priority                    = 1
          + redirect_configuration_id   = (known after apply)
          + rewrite_rule_set_id         = (known after apply)
          + rule_type                   = "PathBasedRouting"
          + url_path_map_id             = (known after apply)
          + url_path_map_name           = "appgw-url-map-path"
            # (2 unchanged attributes hidden)
        }

        # (18 unchanged blocks hidden)
    }

  # azurerm_postgresql_server.atlassian-preprod-server will be created
  + resource "azurerm_postgresql_server" "atlassian-preprod-server" {
      + administrator_login              = (sensitive value)
      + administrator_login_password     = (sensitive value)
      + auto_grow_enabled                = true
      + backup_retention_days            = (known after apply)
      + create_mode                      = "Default"
      + fqdn                             = (known after apply)
      + geo_redundant_backup_enabled     = false
      + id                               = (known after apply)
      + location                         = "uksouth"
      + name                             = "atlassian-preprod-server"
      + public_network_access_enabled    = true
      + resource_group_name              = "atlassian-nonprod-rg"
      + sku_name                         = "B_Gen5_2"
      + ssl_enforcement_enabled          = true
      + ssl_minimal_tls_version_enforced = "TLS1_2"
      + storage_mb                       = 51200
      + version                          = "11"
    }

Plan: 1 to add, 1 to change, 0 to destroy.


Ability to redirect http traffic to https

We should allow the user to direct all http traffic to https. However, we must also make sure we dont end up screwing up the letsencrypt http challenge handler.  So we should probably add a middleware after we add one for it.
Maybe we should also allow the user to select the port and host to redirect to.

This is fairly tricky since we don't know which domain / port to direct the request to. Any ideas?


Update create-retention-policies.md

Added more information about retention limitations for private channels

@JohanFreelancer9 Please copy review

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/4851

@msbemba

#hold-off


fix(rust): Bump base rust image

Fixes https://github.com/firezone/firezone/actions/runs/9650523419

Sorry I did not see the PR and already did exactly the same fix https://github.com/firezone/firezone/commit/74e10b512afe9b0e12e570c22f2c320307eaeafa


[Fix][dolphinscheduler-spi] json parse support LocalDateTime

Purpose of the pull request
Fixed #10403.
Brief change log
Add json serializer and deserializer for LocalDateTime, and register them to Jackson ObjectMapper.
Verify this pull request
This pull request is already covered by existing tests, such as:
method org.apache.dolphinscheduler.spi.utils.JSONUtilsTest#localDateTimeToString
method org.apache.dolphinscheduler.spi.utils.JSONUtilsTest#stringToLocalDateTime
This change added tests and can be verified as follows:
Run mvn clean test in terminal.

The reason for #10403  is that after mysql-connector-java version 8.0.22, datatime type in mysql will be convert to the LocalDateTime type in Java. In most places in dolphinscheduler, LocalDateTime will be handled by mybatis. But in SqlTask, LocalDateTime will be handled by JSONUtils in spi module and JSONUtils does not support parse LocalDateTime.
From the principle of the SPI should have only minimal dependencies, I just add LocalDateTime serializer and not add dependencies jackson-datatype-jsr310.
BTW, JSONUtils in spi module still not support LocalDate and LocalTime, and no use of them has been found so far.

Codecov Report

Merging #10440 (a69d66d) into dev (190f253) will increase coverage by 0.49%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                dev   #10440      +/-   ##
============================================
+ Coverage     40.55%   41.04%   +0.49%
- Complexity     4774     4871      +97
============================================
  Files           878      883       +5
  Lines         35741    35981     +240
  Branches       3969     3991      +22
============================================
+ Hits          14493    14767     +274
+ Misses        19805    19761      -44
- Partials       1443     1453      +10

Impacted Files
Coverage Δ

...g/apache/dolphinscheduler/spi/utils/JSONUtils.java
73.41% <100.00%> (+55.77%)
:arrow_up:

.../org/apache/dolphinscheduler/api/enums/Status.java
100.00% <0.00%> (ø)

...er/api/service/impl/TaskDefinitionServiceImpl.java
24.01% <0.00%> (ø)

...scheduler/api/service/impl/ClusterServiceImpl.java
88.28% <0.00%> (ø)

...rg/apache/dolphinscheduler/api/dto/ClusterDto.java
92.85% <0.00%> (ø)

...hinscheduler/api/controller/ClusterController.java
89.47% <0.00%> (ø)

...olphinscheduler/common/utils/ClusterConfUtils.java
0.00% <0.00%> (ø)

...rg/apache/dolphinscheduler/dao/entity/Cluster.java
100.00% <0.00%> (ø)

...inscheduler/api/service/impl/UsersServiceImpl.java
72.50% <0.00%> (+0.24%)
:arrow_up:

...apache/dolphinscheduler/spi/utils/StringUtils.java
27.27% <0.00%> (+1.01%)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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


Add support for IQueryable.Min and .Max

Min and Max extension methods for IQueryable were not available.

Thank you for the pull.
We will look if there is a reason why Min and Max is still not here in the DynamicQueryableExtensions class.
We will look at your pull next week
Best Regards,
Jon

Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Entity Framework Classic • Bulk Operations •  Dapper Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

This was addressed before #404

@StefH I did notice #404 - as far as I could tell it specifically addressed the usage of Min/Max nested within a dynamic expression (e.g. the example ion the dotnetfiddle). It didn't account for the extension methods exposed for the IQueryable interface.

Hello @gregfullman ,
The v1.2.3 has been released. I also agree on this addresses a different problem.
@StefH if you believe otherwise, let me know what should be done instead


PTVS fails to load file for other frames in mixed mode debugging

Debugging https://github.com/microsoft/PTVS/tree/main/Examples/PythonNative you cannot open other frames in the callstack. Debugger complains that the file is already in use. That's likely not the real problem.

Step out doesn't seem to work either


ValueError: not enough values to unpack (expected 2, got 1)

I got an error when I was running the test
sgpt "nginx default config file location"
The error  I got
Traceback (most recent call last):
  File "/home/flipped/.local/bin/sgpt", line 5, in <module>
    from sgpt import cli
  File "/home/flipped/.local/lib/python3.8/site-packages/sgpt/__init__.py", line 1, in <module>
    from .app import main as main
  File "/home/flipped/.local/lib/python3.8/site-packages/sgpt/app.py", line 9, in <module>
    from sgpt.config import cfg
  File "/home/flipped/.local/lib/python3.8/site-packages/sgpt/config.py", line 83, in <module>
    cfg = Config(SHELL_GPT_CONFIG_PATH, **DEFAULT_CONFIG)
  File "/home/flipped/.local/lib/python3.8/site-packages/sgpt/config.py", line 40, in __init__
    self._read()
  File "/home/flipped/.local/lib/python3.8/site-packages/sgpt/config.py", line 72, in _read
    key, value = line.strip().split("=")
ValueError: not enough values to unpack (expected 2, got 1)

I created the .sgptrc configuration file manually myself, probably in the wrong format


FCM onMessage, onLaunch, and onResume not working after upgrading to Flutter 0.2.3-pre.46 • channel master

Steps to Reproduce
- Upgrade Flutter to last version:
Flutter 0.2.3-pre.46 • channel master
Framework • revision d6bac5fd4a
Engine • revision 1348ab5b63
Tools • Dart 2.0.0-dev.39.0.flutter-f1ebe2bd5c

- Run the Firebase messages plugin example

The notification delivered successfully if the app was closed, but when I try to open the delivered notification or send a notification while the app is open nothing happened.
Flutter Doctor
[✓] Flutter (Channel master, v0.2.3-pre.46, on Mac OS X 10.13.3 17D102, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[!] iOS toolchain - develop for iOS devices (Xcode 9.2)
    ✗ Xcode requires additional components to be installed in order to run.
      Launch Xcode and install additional required components when prompted.
[✓] Android Studio (version 3.0)
[✓] IntelliJ IDEA Community Edition (version 2017.3.5)
[✓] VS Code (version 1.20.1)
[✓] Connected devices (3 available)

This is new too , when I launch Xcode, it doesn't show anything.

#15697

All above issues gone after changing channel to beta and upgrading Flutter.

Is this the same as https://github.com/flutter/flutter/issues/15735?

The easy fix is the change the method signature of MessageHandler:
typedef Future<dynamic> MessageHandler(Map<String, dynamic> message);
// to
typedef Future<dynamic> MessageHandler(Map<dynamic, dynamic> message);

This seems to be a Dart 2 only issue.

@salehahmedZ
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/882. Any further collaboration will be done there.


draft guidance for partnerships

Fix #61
Are there other things we should be adding? Feedback welcome from any R-Lady who might see this!

Thanks a ton @jenniferthompson! :rocket:


Sub-sketches

[ ] try using namespace instead of #define
[ ] provide direct support in a new source file. Call the class SubSketch.

Done


fix: 🐛 LeetCodeV2 detection mechanism

Content

Update LeetCodeV2 detection mechanism;
Rehandle submit by keyboard shortcuts;
Use MutationObserver to determine when the submit button elements are loaded

Note

Submit by Keyboard Shortcuts only support on LeetCode v2
Detecting LeetCodeV2 by the presence of submit button is really make sense

BREAKING CHANGE: 🧨 Detecting LeetCodeV2 by the presence of submit button
✅ Closes: #51

Perhaps you will wonder about the lines 1037~1038 in the script/leetcode.js file.
In the default mode of LeetCodeV2, we have three textarea element and the main textarea element is 2nd (index = 1).
When use raw testcase editor (below image), we have four textarea element and the main textarea element is 3rd (index = 2).
Line 1037~1038:
const textareaList = document.getElementsByTagName('textarea');
const textarea = textareaList.length === 4 ? textareaList[2] : textareaList[1];

Use raw testcase editor:

Merged in PR #52


Error

Running the commands and leaving the default options im getting the following errors:
Azure:/
PS Azure:> Add-AzKeyVaultKey -VaultName $keyVaultName >>     -Name "myKey"
>>     -Destination "Software"
Add-AzKeyVaultKey : Operation returned an invalid status code 'Forbidden'
At line:1 char:1

Add-AzKeyVaultKey -VaultName $keyVaultName `

CategoryInfo          : CloseError: (:) [Add-AzKeyVaultKey], KeyVaultErrorException
FullyQualifiedErrorId : Microsoft.Azure.Commands.KeyVault.AddAzureKeyVaultKey

Azure:/
PS Azure:> Add-AzKeyVaultKey -VaultName $keyVault625288001 >>     -Name "myKey"
>>     -Destination "Software"
Add-AzKeyVaultKey : Cannot validate argument on parameter 'VaultName'. The argument is null or empty. Provide an argument that is not null or empty, and then try the command again.
At line:1 char:30

Add-AzKeyVaultKey -VaultName $keyVault625288001 `

                         ~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidData: (:) [Add-AzKeyVaultKey], ParameterBindingValidationException
FullyQualifiedErrorId : ParameterArgumentValidationError,Microsoft.Azure.Commands.KeyVault.AddAzureKeyVaultKey

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

ID: 86b44bac-c760-a06d-e618-149f275e0d0e
Version Independent ID: bcaaf53e-87c1-59aa-c51b-d3359ebb9046
Content: Encrypt disks on a Windows VM in Azure
Content Source: articles/virtual-machines/windows/encrypt-disks.md
Service: virtual-machines-windows
GitHub Login: @cynthn
Microsoft Alias: cynthn

Hi @chi-ltd Thank you for your feedback! We will review and update as appropriate.

@chi-ltd The second script under thins link is setting the value for $keyVaultName  .
I hope you are executing that script as well in the same shell.
You can verify the value of the $keyVaultName  by giving the variable name. It should have a value something like this(myKeyVault1451418150). If it's empty, Then the above script which sets that variable is not executed properly.
PS Azure:\> $keyVaultName = "myKeyVault$(Get-Random)"
Azure:/
PS Azure:\> $keyVaultName
myKeyVault1451418150
Azure:/
PS Azure:\>

try that and let me know

OK i managed to get this working


doc generation is broken because of extra help param --ids for vmss

Doc build process encountered the following error when running make xml

Exception occurred:
File "d:\msdn\azure\my-azure-cli\src\azure\cli_help.py", line 311, in _load_from_data
raise HelpAuthoringException('Extra help param {0}'.format(extra_param['name']))
azure.cli._help.HelpAuthoringException: Extra help param --ids

It seems the error is caused by the following code in /src/command_modules/azure-cli-vm/azure/cli/command_modules/vm/_help.py
helps['vmss get-instance-view'] = """
    type: command
    parameters:
        - name: --ids
          short-summary: "One or more scale set or specific VM instance IDs. If provided, no other 'Resource Id' arguments should be specified."
"""

helps['vmss reimage'] = """
    type: command
    parameters:
        - name: --ids
          short-summary: "One or more scale set or specific VM instance IDs. If provided, no other 'Resource Id' arguments should be specified."
"""

helps['vmss show'] = """
    type: command
    parameters:
        - name: --ids
          short-summary: "One or more scale set or specific VM instance IDs. If provided, no other 'Resource Id' arguments should be specified."
"""

The ids parameter is in fact part of the argument for this command. The fact that it was previously omitted was in fact a bug. @BurtBiel it seems to be complaining at line 311 of _help.py. However, this does not happen when you run a command (az vmss show -h) or during the tests. It might be something in azhelpgen.py where ids is not attached in the command table and thus ends up being seen as extra?

Fixed by #832


When does "finished" callback get called?

Hi there,
I'm enjoying using this plugin but don't fully understand it yet and perhaps you can help me with an issue I'm having.  Essentially I'm using a fairly default dropzone:
{{> dropzone contentType='images' fileTypes='.jpg' multiple=false formData=specificFormData callbacks=myCallbacks }}
and a "finished" callback that inserts some of the fileInfo/formFields data into an Uploads mongo table.
Unfortunately what I'm seeing is the Uploads table getting populated in "triangular" sequence - on my first upload one entry is added, on my second the new entry is added, plus the previous entry is added - and then thereafter, every time finished is called it not only adds a new entry, but re-adds all the previous entries in the Uploads table!
I assume I'm doing something really silly, but can you cut through the confusion and give me any clue why the "finished" callback would be called one time for each upload that has been done to date?
Many thanks.

Incidentally I can't get either {{> upload_bootstrap }} or {{> upload_semanticUI }} to work for me at all (503 errors every time), which may be completely a side issue or may be an insight into the fact that I've done something terribly wrong which is breaking my whole experience...

Hmm, if I wipe the database then I upload one new file, all files I have uploaded this session get reuploaded. Should I be resetting the contents of the dropzone in some way that I'm not doing I wonder...?

@BeggarsMattM I have updated the plugin so that queue is now wiped before any file is added to the upload control. Please let me know if this solved your issue. Version 2.1.4 is the newest one.
Sorry for the late response, but I was moving between Australia and Europ back and forth ... crazy time. If ll good, please close this issue.


Support for rxjava 3

Hey! Are you going to support rxjava 3.x?

yep, i don't start working on it bu it is on the roadmap

On the end I think that for the 6.0.0 I completly remove the clojure part.
I don't have a good uses cases for rxjava on jvm (having core.async and manifold). Handling backpressure in rxjava and clojure is... not idiomatic. It is not worh the effort of maintaining it.
On the cljs part i using it a lot on many projects and i want to focus on the real use case.

I would be sorry to lose support for beicon on the JVM, but I would not be sorry to loose rxjava.  If it was all implemented with core.async, that would suit me fine.  The expressiveness of reactive streams is superior to plain core.async IMO.  And I think beicon.core/to-atom is superior to approaches that define "cells" with "automatic" updates.  Just my 2 cents.  Beicon is one of my favorite libraries.


Using webpack 5 in gulp via webpack-stream creates empty output files

We are using webpack via gulp / webpack-stream in our project, and are checking how to make the transition to the recently released webpack 5. I assume this is still a very common usage scenario for webpack.
Luckily, webpack-stream allows to use a webpack version different from the one that comes as a dependency (which currently is 4.26.1), by specifying a reference as an optional second parameter.
Unfortunately, doing so creates empty output files.
I've created a small reproducer repo that showcases the issue.
Any ideas as of what changed in webpack 5 that could result in this behaviour?
Any suggestions as of how to fix/ work around the issue?
Note: I've also raised this as an issue in the webpack-stream repository.

The mistake was on my side. The imported component was actually empty, so webpack 5 obviously creates the superior output.


Seems to not be running

after doing the initial config, I am running the job, but is just sits at the below text, and I don't see anything happening in top.
python3 dtdd_api.py
Established memcache client
 * Serving Flask app "dtdd_api" (lazy loading)
 * Environment: production
   WARNING: Do not use the development server in a production environment.
   Use a production WSGI server instead.
 * Debug mode: off
 * Running on http://<IP_ADDRESS>:5000/ (Press CTRL+C to quit)

EDIT: I found the problem.   I was not using the correct token to connect to plex.

I found the problem. I was not using the correct token to connect to plex.


Archi 4.7 - Ubuntu - Cursor not displaying in model view correctly

Version of Archi

4.7 Build<PHONE_NUMBER>30
see installation log attached
Archi Plug-ins

See installation log attached
Operating System

Linux Unbuntu
Expected Behaviour

When selecting a view or folder in the model the cursor should highlight the text indicating that it is being changed and the text when entered show
Actual Behaviour

When selecting a view or folder in the model the cursor does not show and you do not see the text being entered, however hit return and the text is auctioned
Steps to Reproduce the Behaviour

Consistently there

Archi Installation Log.txt

Which cursor do you mean? Please also - steps to reproduce.

The cursor where you rename the folders or elements in the model.
Happy to show you using zoom, etc
M
From: Phil Beauvoir<EMAIL_ADDRESS>Sent: 05 May 2020 17:20
To: archimatetool/archi<EMAIL_ADDRESS>Cc: Matthew Dickie<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [archimatetool/archi] Archi 4.7 - Ubuntu - Cursor not displaying in model view correctly (#617)
Which cursor do you mean? Please also - steps to reproduce.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/archimatetool/archi/issues/617#issuecomment-624119311, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMNK3HQJAWF4LJN4KQEZMPLRQAVCZANCNFSM4MZVF5UA.

Which version of Ubuntu?
Do you have Wayland on?
Do you mean the text I beam cursor?

Linux matt-VirtualBox-GTCprod 4.15.0-99-generic #100-Ubuntu SMP Wed Apr 22 20:32:56 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
No idea re Wayland
Yes I think I mean that cursor.. when you want to change a name F2 then type etc
M
From: Phil Beauvoir<EMAIL_ADDRESS>Sent: 05 May 2020 17:24
To: archimatetool/archi<EMAIL_ADDRESS>Cc: Matthew Dickie<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [archimatetool/archi] Archi 4.7 - Ubuntu - Cursor not displaying in model view correctly (#617)
Which version of Ubuntu?
Do you have Wayland on?
Do you mean the text I beam cursor?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/archimatetool/archi/issues/617#issuecomment-624121527, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMNK3HQT4VKCOXKET6BGDU3RQAVQTANCNFSM4MZVF5UA.

Linux matt-VirtualBox-GTCprod 4.15.0-99-generic #100-Ubuntu SMP Wed Apr 22 20:32:56 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

I can't tell from that what version of Ubuntu it is.

Sorry was doing three things at once - Ubuntu 18.04.4 LTS
From: Phil Beauvoir<EMAIL_ADDRESS>Sent: 05 May 2020 17:30
To: archimatetool/archi<EMAIL_ADDRESS>Cc: Matthew Dickie<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [archimatetool/archi] Archi 4.7 - Ubuntu - Cursor not displaying in model view correctly (#617)
Linux matt-VirtualBox-GTCprod 4.15.0-99-generic #100https://github.com/archimatetool/archi/pull/100-Ubuntu SMP Wed Apr 22 20:32:56 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
I can't tell from that what version of Ubuntu it is.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/archimatetool/archi/issues/617#issuecomment-624125385, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMNK3HVQ7OOPYBWMZ7ZSO6TRQAWIHANCNFSM4MZVF5UA.

Perhaps a screen shot might help...

The screen shot won’t show anything (that’s the bug …)  .. that’s why I suggest a zoom desktop share to show you ..
From: Phil Beauvoir<EMAIL_ADDRESS>Sent: 05 May 2020 17:34
To: archimatetool/archi<EMAIL_ADDRESS>Cc: Matthew Dickie<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [archimatetool/archi] Archi 4.7 - Ubuntu - Cursor not displaying in model view correctly (#617)
Perhaps a screen shot might help...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/archimatetool/archi/issues/617#issuecomment-624127422, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMNK3HSABK7GZUXAEC7ZQJDRQAWVBANCNFSM4MZVF5UA.

I can reproduce this on Ubuntu 18.04 with Archi 4.6 and 4.7
The issue is that under some circumstances the text editor control does not appear in the Models Tree on Rename action.

Launch Archi with an empty Models Tree (this stage is important)
New Model
Try to rename the Default View or add an element and try to rename it
The text edit control does not appear

To get it to work launch Archi with a model already open in the Models Tree or close the Models Tree and re-open it (ensure that it contains a model).
Some observations:

This does not happen on Archi 4.5.1 (that uses Eclipse 4.11)
This does not happen on Ubuntu 19.10
The same thing happens in the jArchi Scripts Manager
It is only happening if the Models Tree (or Scripts Manager) is initially empty. If there are tree nodes when these are opened the text editor works.

And I can reproduce this on Linux Mint (based on Ubuntu 18.04).
The key to reproducing this bug is to ensure that the Models Tree is initially empty. Add a new model and the tree nodes do not show a text editor control. Close and re-open the Models Tree (with at least one model in it) and it works.
Right now I have no idea if this is an Eclipse or a GTK issue. I'll need to test with Eclipse 4.15.

Ok , let me know if I can test a release etc when your ready ..
M
From: Phil Beauvoir<EMAIL_ADDRESS>Sent: 05 May 2020 19:36
To: archimatetool/archi<EMAIL_ADDRESS>Cc: Matthew Dickie<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [archimatetool/archi] Archi 4.7 - Ubuntu - Cursor not displaying in model view correctly (#617)
And I can reproduce this on Linux Mint (based on Ubuntu 18.04).
The key to reproducing this bug is to ensure that the Models Tree is initially empty. Add a new model and the tree nodes do not show a text editor control. Close and re-open the Models Tree (with at least one model in it) and it works.
Right now I have no idea if this is an Eclipse or a GTK issue. I'll need to test with Eclipse 4.15.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/archimatetool/archi/issues/617#issuecomment-624201352, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMNK3HVYCRVTT6JIU3LBBIDRQBFBPANCNFSM4MZVF5UA.

Ok , let me know if I can test a release etc when your ready ..

I don't think there's anything I can fix if it is an Eclipse bug.

I've tested this against Eclipse 4.15 (Archi 4.6 and 4.7 EA are built against Eclipse 4.13) and it seems to be fixed.
I know that the Eclipse people "fixed" some GTK issues in later versions of Eclipse for Linux. But with any Eclipse "fixes", something else is broken (a.k.a "Fix one, break one).
I don't like this bug because a brand new Archi installation on Linux, followed by a "New Model" leads to the user not being able to rename things in the Models Tree.
@jbsarrodie I'm going to build Archi 4.7 on top of Eclipse 4.15 - I know there are other fixes in that version that we could benefit from apart from this one.

I can reproduce this bug. FWIW, when it happens, toggling the model tree off and on solve the issue, but that's really annoying.

@jbsarrodie I'm going to build Archi 4.7 on top of Eclipse 4.15 - I know there are other fixes in that version that we could benefit from apart from this one.

Ok, let's see what gets broken ;-)

Ok, let's see what gets broken ;-)

Well...something like this bug, which got broken somewhere between 4.11 and 4.13...

Well...something like this bug, which got broken somewhere between 4.11 and 4.13...

Or something like setting the clipping area on connexions...

Or something like setting the clipping area on connexions...

Yes, that one too.

Even though this seems to be fixed in Eclipse 4.15 I opened a bug report just in case there are underlying issues:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=562872

Even though this seems to be fixed in Eclipse 4.15 I opened a bug report just in case there are underlying issues:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=562872

It's this one https://bugs.eclipse.org/bugs/show_bug.cgi?id=550606

@gtc-bigyin This should be fixed in Archi 4.7 beta 1 - https://www.archimatetool.com/beta/

Tested, seems to work ok ..
M
From: Phil Beauvoir<EMAIL_ADDRESS>Sent: 11 May 2020 10:04
To: archimatetool/archi<EMAIL_ADDRESS>Cc: Matthew Dickie<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [archimatetool/archi] [Linux] Tree text cell editor not working when starting with no tree nodes (#617)
@gtc-bigyinhttps://github.com/gtc-bigyin This should be fixed in Archi 4.7 beta 1 - https://www.archimatetool.com/beta/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/archimatetool/archi/issues/617#issuecomment-626541361, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMNK3HUXKSXX6MI6DT73CF3RQ6WOHANCNFSM4MZVF5UA.

Tested, seems to work ok ..

Thanks. It was an Eclipse bug (https://bugs.eclipse.org/bugs/show_bug.cgi?id=550606) and Archi 4.7 is now built on Eclipse 4.15 which has the fix.
I'll close this one, thanks for reporting it.


how-to link in README broken

From breakup of how-tos daea29d1e1217832b8a64bec8214e463ab8f3160
how-to link in README:
https://github.com/nix-community/nixos-anywhere/blob/73a6d3fef4c5b4ab9e4ac868f468ec8f9436afa7/README.md?plain=1#L57
should be ./docs/howtos/no-os.md
and
https://github.com/nix-community/nixos-anywhere/blob/73a6d3fef4c5b4ab9e4ac868f468ec8f9436afa7/README.md?plain=1#L79
./docs/howtos/custom-kexec.md
and
https://github.com/nix-community/nixos-anywhere/blob/73a6d3fef4c5b4ab9e4ac868f468ec8f9436afa7/README.md?plain=1#L90
should be:
./docs/howtos/INDEX.md

fixed in https://github.com/nix-community/nixos-anywhere/commit/90894fe52998513229f268249ffedce90169431f

fixed in https://github.com/nix-community/nixos-anywhere/commit/90894fe52998513229f268249ffedce90169431f


Bad name of the template

In chapter Bootstrapping Rango / Quick Style Change there's line:
“Let’s apply these classes to our rango/categories.html template.”
There's no such a file, and the file path should be rango/cats as is here:
https://github.com/leifos/tango_with_django_2/tree/master/code/tango_with_django_project/templates/rango

Thanks for your message!
I’m aware of this issue. Currently working through a revision of the book to address this inconsistency amongst others. The file that it is referring to is cats.html in the sample codebase.
Cheers,
David

On 18 Jun 2019, at 15:30, piotrsynowiec<EMAIL_ADDRESS>wrote:
In chapter Bootstrapping Rango / Quick Style Change there's line:
“Let’s apply these classes to our rango/categories.html template.”
There's no such a file, and the file path should be rango/category.html as is here:
https://github.com/leifos/tango_with_django_2/tree/master/code/tango_with_django_project/templates/rango https://github.com/leifos/tango_with_django_2/tree/master/code/tango_with_django_project/templates/rango
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/leifos/tango_with_django_2/issues/4?email_source=notifications&email_token=ABD23HJC7HREQ4WZHAWGUWLP3DWQDA5CNFSM4HZAV7EKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G2FFDUQ, or mute the thread https://github.com/notifications/unsubscribe-auth/ABD23HMLV5LH6GD63YL72S3P3DWQDANCNFSM4HZAV7EA.

Problem fixed. Code and narrative now match up. Closing issue.


Error update Atom v1.9.9

When I will update language-babe packages, happens the error below:

Looks like an atom issue of some sort and nothing to do with language-babel. I suggest closing all atom editors and manually removing /Users/tclestino/.atom/language-babel, restarting atom and reinstall language-babel using either atom via the GUI for via apm install language-babel


Docker for Mac: add page on disk space

Proposed changes
The disk space usage of Docker on Mac has been difficult to manage and has confused lots of people, see
https://github.com/docker/for-mac/issues/371
This patch adds a top-level section to the Docker for Mac docs alongside networking which explains

where the space has gone
how to reclaim some of it

In future I'd like to add more about why some systems use Docker.raw and some Docker.qcow2 but we're in a transitional state at the moment. It'll be simpler to explain after the next stable version has been released.
Related issues (optional)
Related to https://github.com/docker/for-mac/issues/371

Thanks for your work! I think this will help a lot.
Do you think it would be useful to communicate this in the docker mac UI?
Something like this on the disk tab (the learn more link would lead to the docs in this PR):
optimal ✅
your system supports raw mode which is optimal for docker mac.
[learn more]
compatibility mode ⚠
your version of macos does not support raw mode so docker mac is using qcow.
upgrade to the latest macos to make docker faster.
[learn more]
compatibility mode ⚠
your image is on a disk not formatted as APFS so docker mac is using qcow.
put your image on an APFS disk or upgrade your hardware to make docker faster.
[learn more]
compatibility mode ⚠
your version of macos supports raw mode but you are running qcow from a prior install.
[remove all data] to make docker faster.
[learn more]

@gondalez I think that's a good idea. What do you think, @gtardif ?

@gtardif - PTAL - thank you!

Merging this - will open up a separate PR for TOC update. Thank you @djs55 and @thaJeztah !


move example's setup to setup.py

move the list of dependences for examples to setup.py

NOTE: This resolves #515


[Feature] convert the oneOf field in api schema to ui schema

Is your feature request related to a problem? Please describe.
One parameter has a multiple value structure, such as the container-image trait:
 parameter: *#PatchParams | close({
        	// +usage=Specify the container image for multiple containers
        	containers: [...#PatchParams]
        })

This cue script will generate an API schema with the oneOf type:
schema: {
  oneOf: [{properties: {containerName: {default: "",…},…},…}, {,…}]
  type: "object"
}

For this case, currently can not generate UI schema, users can not use this trait in UI. refer to #3578.
Describe the solution you'd like
We should support converting the oneOf field in api schema to the ui schema. uses could select one structure to set values in UI.

I want to fix it and learn more about that converts api schema to the ui schema

Hi @fourierr Are you still working on this?

Hi @fourierr Are you still working on this?

I haven't been working on this lately


Update Virtual Network Service Endpoint Policies Portal

In section 'Create a service endpoint policy', step 5 should say 'Tags' instead of 'Tabs'. #17114

@PrasadHonrao : Thanks for your contribution! The author, @anithaa, has been notified to review your proposed change.

#sign-off

@KumudD: I'm sorry - only the author of this article, @anithaa, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@rjagiewich : Thanks for your contribution! The author, @anithaa, has been notified to review your proposed change.

#sign-off

@AnithaAdusumilli: I'm sorry - only the author of this article, @anithaa, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Jds 1.0 discord chat data

Related to issue: https://github.com/b21phiro/jds-website/issues/17

I can not find any problems with this P.R

Suggestion:
An folder named img is created in root.
In this folder an image is placed for the avatar of each user. Example RobertMiliAvatar.png.
The avatar url is added to chat data.
{
  "comments": [
    {
      "author": "RobertMili",
      "comment": "Najs, tack för tips. Jag kan fixa idag, jag verkligen uppskattar göt eran tips \uD83D\uDCAA",
      "date": "2024-09-02",
      "avatarUrl": "/img/RobertMiliAvatar.png"
    }
}

This should enable dynamic handling of the avatar image for each author.


KMEMLEAK disabled

Issues are for bugs in syzkaller itself, for everything else please contact<EMAIL_ADDRESS>mailing list.
Before filing an issue please check the existing issues and the mailing list (https://groups.google.com/forum/#!forum/syzkaller).
Please provide syzkaller revision, Go version (output of go version), relevant information about host and target systems.
If syz-manager is not working overall, please provide output of syz-manager with -debug flag.
the KMEMLEAK is diabled.
the .config is below:
CONFIG_HAVE_DEBUG_KMEMLEAK=y
CONFIG_DEBUG_KMEMLEAK=y
CONFIG_DEBUG_KMEMLEAK_EARLY_LOG_SIZE=20000
# CONFIG_DEBUG_KMEMLEAK_TEST is not set
# CONFIG_DEBUG_KMEMLEAK_DEFAULT_OFF is not set
CONFIG_DEBUG_KMEMLEAK_AUTO_SCAN=y

when run the syz-manager, the log is :
2019/04/15 03:06:16 leak checking           : KMEMLEAK disabled: increase CONFIG_DEBUG_KMEMLEAK_EARLY_LOG_SIZE or unset CONFIG_DEBUG_KMEMLEAK_DEFAULT_OF

the /sys/kernel/debug/kmemleak existed, what happned ? how to solve it ?
Thanks.

Try increasing CONFIG_DEBUG_KMEMLEAK_EARLY_LOG_SIZE.


feat: deploy / aws lambda / sst v3 waku stack

This is a first draft for a sst stack for WAKU.
Based on the fact, that there can be more than 25 folders (AWS Cloudfront routing limit) on the first level of the public folder only the folder public/images and public/assets are copied to s3 and forward by CloudFront.
A possible, but more complex solution could handle this by adding a Cloudfunction to Cloudfront which knows all routes and can forward all existing static root level routes.

@rmarscher this is based as suggested on the sst tanstack-start stack

@rmarscher this is based as suggested on the sst tanstack-start stack

Thank you for creating this! I hope it helps it get added upstream to SST. 🙏


max_satisfying is not returning max

This looks like a bug in max_satisfying to me.
from semver import max_satisfying
versions = [ '<IP_ADDRESS>', '<IP_ADDRESS>', '<IP_ADDRESS>' ]
range_ = '>=49'
print(max_satisfying(versions, range_, loose=True, include_prerelease=True))

$ python max_satisfying.py
<IP_ADDRESS>

I would expect it to return <IP_ADDRESS>, the highest (or max) version that satisfies.
Changing this
            if max_ is None or max_sv.compare(v) == -1:  # compare(max, v, true)

to this
            if (max_ is None or max_sv.compare(v) == -1) or (max_sv.compare(v) == 0):  # compare(max, v, true)

fixes it.
$ python max_satisfying.py
<IP_ADDRESS>

Ah OK, if send me PR I will merging it.


remove scipy and torch from requirements

You're missing import pytest.

Is test_winograd.py worth keeping at all? It looks from the comments like it wasn't even a correct test to begin with. It would be nicer to fix the test (perhaps by validating the values offline and then committing it as the golden output), but we could also just delete it.

I would like to keep the test. It's at least compliant with what Tian had implemented for his job.

Although I do agree that it can be optimized more.

And, it is not that "it was not the correct test, to begin with". It's just that we can expand the matrix thus can delete zero coefficients so that it becomes more optimal.

What does # TODO: finish test, validate output (returns NaN right now) mean, then?

The test needs to be validated, but removing the torch dependency is a different issue. Python 3.10 doesn't support torch yet (and torch is a huge dependency), and not all the users need to run winograd test.

The test needs to be validated, but removing the torch dependency is a different issue. Python 3.10 doesn't support torch yet (and torch is a huge dependency)

It's not that different. This way is somewhat vacuous. We could also "remove" all of our dependencies by skipping every test and turning our whole API surface into one big no-op. Torch is a huge dependency we probably shouldn't have even if it supported Python 3.10.

and not all the users need to run winograd test.

None of our users need to run any test. They need to use the library. The developers and CI need to run the tests.

None of our users need to run any test. They need to use the library. The developers and CI need to run the tests.

Sure. Ideally, we probably want to separate EXO dependency and EXO test dependency.

Torch is a huge dependency we probably shouldn't have even if it supported Python 3.10.

I'm on board with that

This PR is ready for review @gilbo @alexreinking


Convert socket adaptor to socket

for issue #144

Codecov Report

Merging #160 into dev will decrease coverage by -0.13%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##                dev     #160      +/-   ##
============================================
- Coverage     29.77%   29.65%   -0.13%
+ Complexity     1251     1247       -4
============================================
  Files            97       97
  Lines         23303    23305       +2
  Branches       3871     3871
============================================
- Hits           6939     6910      -29
- Misses        15014    15043      +29
- Partials       1350     1352       +2

Flag
Coverage Δ
Complexity Δ

#JDBC41
29.58% <0%> (-0.03%)
1244 <0> (-3)

#JDBC42
29.57% <0%> (-0.14%)
1245 <0> (-2)

Impacted Files
Coverage Δ
Complexity Δ

...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java
32% <0%> (-0.69%)
0 <0> (ø)

src/main/java/microsoft/sql/DateTimeOffset.java
37.14% <0%> (-2.86%)
8% <0%> (-2%)

...c/main/java/com/microsoft/sqlserver/jdbc/Util.java
38.24% <0%> (-0.72%)
50% <0%> (-1%)

...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java
45.67% <0%> (-0.37%)
182% <0%> (-1%)

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 6e8dc59...c086691. Read the comment docs.

I tried out our test case and it fixes the cancellation issue when multiSubnetFailover is enabled. It no longer hangs and the cancellation happens correctly. Very nice.
Skimming through the implementation, it looks like the approach is to close the socket channel and reconnect it as a regular socket. That should be be fine though it does lead to the situation that all multi subnet enabled connections will create two socket connections to the server (the first being closed). It'd be marginally slower than directly using the channel but it shouldn't be a big deal as it's just the establishment of the TCP socket. Also, I don't think it's an issue on the server to have the first connection closed as the driver is already doing that to the rest of the resolved addresses anyway.
So looks good to me.
FYI, the commit message has a typo: Convert socket adaptor to socket #160. Adaptor should be spelled adapter). Looks like it's already been merged though ...

@sehrope Thank you for getting back to us. We are glad to know you are satisfied with this fix.
Forgive me about the typo. English isn't my first language, Java is :)


Updates no longer display while Airplane Mode activated and off

Updates are hidden while Airplane Mode is activated and off. If I deactivate Airplane Mode the updates now show.
I still have to dig further. Below are some screenshots to explain.

Airplane Mode deactivated.

Airplane Mode activated and on

Airplane Mode activated and off

i think this is actually due to #44 which I am about to push a fix for

about to bump to 0.0.6 across the board to fix this. my bad.


deep link not operational on SIOPV2 presentaton

At some point I forced deeplink presence to false. Need rework to handle SIOPV2 presentation with deep link.

I wrote somewhere deepLink : false... which is bad :D
short complexity: 1


Add CSS variables for customization

Adds CSS overridable CSS variables to customize the colors of the text (neutrals), background and the primary.
It works, but I set it as a draft as this is a breaking change as with this users need to provide a whole palette to customize the primary color.
Im not sure about the best way to handle that. Probably allowing both would work the best.

What are your thoughts on this?

It works, but I set it as a draft as this is a breaking change as with this users need to provide a whole palette to customize the primary color.

@Visual-Dawg thanks, like you said, asking users to provide a whole color palette which might be too much work. it also increases the complexity for maintaining.
i think what we can do is adding several custom utility classes in tailwind config to allow overriding at least backgrounds plus some other visual elements via CSS variables first.

Sorry for answering so late.
I think waiting for Tailwind v4 might make sense, as this will ship with variables for the purpose of themeing.
Which should be much easier


ingress-shim: allow to impersonate ingress-creator instead of using cert-manager serviceaccount

Is your feature request related to a problem? Please describe.
I am trying to implement CertificateRequestPolicies with approver-policy. The goal is to limit different tenants to their own ClusterIssuer. This works successfully by whitelisting their Namespaces. However, for "infrastructure" ingresses I would like to have an additional ClusterIssuer that none of the tenants should be able to use.
This is currently not supported, I think.
Describe the solution you'd like
I would like to be able to use the RBAC system for allowing usage of the infrastructure ClusterIssuer. AS ingress-shim is always creating the CertificateRequests using the cert-manager ServiceAccount, there is no way to distinguish tenant Ingresses from infrastructure ones.
I would like to configure cert-manager ingress-shim to impersonate the creator of the Ingress when generating the CertificateRequest. This would allow to use RBAC as in https://github.com/cert-manager/approver-policy/blob/main/docs/examples/example.com.yaml#L48-L53.
Describe alternatives you've considered
From my understanding I can currently only reach this goal by whitelisting the infrastructure Namespaces. This is pretty cumbersome and error-prone in a changing environment.
Environment details:

Kubernetes version: v1.26.9+k3s1
cert-manager version: v1.13.1
Install method: helm

/kind feature

/remove-lifecycle stale

/remove-lifecycle stale

Sounds like an interesting idea. The main challenge that I see is backwards compatibility.
We might also want to investigate if there are any alternative solutions.
Maybe a design document would be useful here.

/remove-lifecycle stale


What does "north-south" and "east-west" traffic mean? (Request to add that clarification to the document.)

Is it possible to add this information to the document, to provide clarification to persons unfamiliar with setting up the firewall?
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 474ba3cb-afa1-4ec5-37df-6553a5523dbb
Version Independent ID: 56096558-1943-3f6b-9609-7722200272fb
Content: What is Azure Firewall?
Content Source: articles/firewall/overview.md
Service: firewall
GitHub Login: @vhorne
Microsoft Alias: victorh

@RichardSmith1208 , thank you for your feedback. We'll review this and get back to you shortly!

@RichardSmith1208 ,
Traffic between Azure resources and the internet is called North-South traffic.
Traffic between subnets is referred to as east-west traffic.
For more information, refer: https://learn.microsoft.com/en-us/azure/architecture/framework/security/design-network-flow#east-west-and-north-south-traffic
I've added the above link in this doc for clarity. It will be updated soon.


 For now it makes sense to run away from it.
Later, I'll change the code to make them an obstacle. This will allow the bot to maneuver around them without running into them. The same thing can be done for enemies that can't split on me.

sounds great.
it was only when you're small enough that it would not be worthy enough for big ones to should mass to kill you a "much smaller ball"

@Apostolique I've started on some of the work on this, and think I have it most of the way to requiring less distance between a blob and a virus. Do you mind if I refactor some of the findDestination() and getAllThreats() functions to differentiate between viruses and players more easily?

I used to have a function that was called getAllViruses() then I merged it in getAllThreats().
You can separate them again if you want.
interNodes[element].d is the property that will tell you if something is a virus. d is a boolean value.

Ah, I had noticed that interNodes[element].W was only set to 1 if it was a virus. I'll take a look at everything for .d and see if that simplifies my code.

.w is just the last time it was drawn.

.W (uppercase) is 0 or 1, and seems to only be 1 for viruses. The timestamp was pretty obvious on (lower) .w

Latest version handles this.


Output "may loose settings" warning on mysql/mariadb for more impacted change types

Impact

[ ] Bug fix (non-breaking change which fixes expected existing functionality)
[X] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
MySQL/MariaDB's alter table ... modify column syntax requires a complete redefinition of the column when modifying a column, such as setting a comment, changing a datatype, etc.
The the setColumnRemarks, addDefaultValue, etc change does not support all possible column features, only the data type so anything else (like nullability etc) will be lost.
This PR expands the existing warning we had for modifyDataType, since that is all we can do for this.
Things to be aware of

Only impacts Mysql/MariaDB.
Adds warnings to addDefaultValue, setColumnRemarks, addNotNull, and dropNotNull changes.

Things to worry about

Nothing

For this PR tested if every change type listed before had a warning msg on MySQL and MariaDB:
addDefaultValue
setColumnRemarks
addNotNull, and dropNotNull


Full rewrite

Thanks to @trasher to initiate improvements
This rewrite prepare the integration for the home assistant core:

Move API handling to a dedicated python package https://github.com/Aohzan/pyecodevices
Add Config Flow for configuration from the UI
Add authentication (credits for @trasher)

I took a quick look, I have nothing to say right now; but I'm not a python expert :)
Maybe could you drop a line on migration from older release? I guess one should drop existing "hard-coded" config and do it again from the UI; that's what I've read on other integrations readme.
Please let me know when this is "stable" enough to be tested; and I'll try to test (maybe not before next week-end).

I took a quick look, I have nothing to say right now; but I'm not a python expert :)
Maybe could you drop a line on migration from older release? I guess one should drop existing "hard-coded" config and do it again from the UI; that's what I've read on other integrations readme.
Please let me know when this is "stable" enough to be tested; and I'll try to test (maybe not before next week-end).

You can test in right now
I put a tag for the last version in 0.1, I will let a message for the migration when I will merge and add a new tag
I opened a PR on core : https://github.com/home-assistant/core/pull/45499

You can test in right now
I put a tag for the last version in 0.1, I will let a message for the migration when I will merge and add a new tag
I opened a PR on core : https://github.com/home-assistant/core/pull/45499


prepare for cargo for mips and powerpc

with these changes I can cross compile Cargo for these targets:

mips-unknown-linux-gnu
mipsel-unknown-linux-gnu
powerpc-unknown-linux-gnu
powerpc64-unknown-linux-gnu
powerpc64el-unknown-linux-gnu

using these commands:
$ ./.travis.install.deps.sh
$ ./configure --local-rust-root=$(pwd)/rustc --enable-nightly --target=$TARGET
$ make

in Ubuntu 16.04 using the standard cross toolchains
r? @alexcrichton
cross compiling Cargo for the mips targets requires rust-lang/libc#378

@bors: r+ d581274c957f751876f25c790fcf881559b667e0
Oh how I wish we could just build all this on Travis...

:hourglass: Testing commit d581274 with merge 9419e3f...

Oh how I wish we could just build all this on Travis...

We can't? Is the time limit a problem? or is the problem something else? the cross toolchain? we are using Ubuntu's cross toolchain on the buildbots as well.

:broken_heart: Test failed - cargo-win-gnu-32

Nah unfortunately the reason is much more lame than that. We haven't figure out the best way to build binaries on Travis but still cryptographically sign them. Once we solve that problem it should be trivial to move over!
@bors: retry

:zap: Previous build results for cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64 are reusable. Rebuilding only cargo-cross-linux, cargo-win-gnu-32...

Oh, hmm. How about this solution (see step 3) of storing an encryted version of secring.gpg in this GitHub repository and then decrypting it in the travis worker? Gotta admit that handing over your secret key (even if encrypted -- it has to be decrypted at some point) to someone else sounds scary to me.

:sunny: Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64
Approved by: alexcrichton
Pushing 9419e3f37279740b5848ca6a016d6b5d0c3e1621 to master...

Yeah that's what we'd likely end up doing if we were to do it. Just need to decide to do it at this point!
We'd probably have a signing subkey and such with a revocation cert and various bells and whistles. We'd also want to think through about landing code on master in terms of what implications it has as to what CI runs have the key available to look at.


MessageEmbed VideoInfo not supported

General Troubleshooting

[ ] I have checked for similar issues.
[ ] I have updated to the latest JDA version.
[ ] I have checked the branches or the maintainers' PRs for upcoming features/bug fixes.

Issue
Issue Type

[ ] Bug Report
[x] Feature Request

Description
In the MessageEmbed class, there is no way to set the VideoInfo object or utilize video embeds at all, the class doesn't offer any way to do this, yet the VideoInfo class exists. I am requesting a means to attach a VideoInfo object to a MessageEmbed, without having to manually call the API myself.
https://ci.dv8tion.net/job/JDA/javadoc/net/dv8tion/jda/core/entities/MessageEmbed.VideoInfo.html

You cannot set the video field on embeds as described in the Discord documentation

Thanks, I missed that. Really odd that they even describe this object in their docs to begin with.


How can we output topk best generation from BART's sample function?

❓ Questions and Help
Before asking:

search the issues.
search the docs.
I have searched the question but didn't see any solution.

What is your question?
When we use BART for a sequence generation task, for example, here in the instruction:
https://github.com/pytorch/fairseq/tree/master/examples/bart#evaluating-the-bartlargecnn-model
how can we generate the top K best generations from the sample function here? I know there is an argument sampling_topk but it seems not to be designed for this situation.

I guess this is the relevant part of code for processing the predictions?
https://github.com/pytorch/fairseq/blob/f0a61a2774aff2efbc1adb0b5daee346a8401605/fairseq/models/bart/hub_interface.py#L133
        # Process top predictions
        hypos = [x[0] for x in translations]
        hypos = [v for _, v in sorted(zip(sample['id'].tolist(), hypos))]
        return hypos

Check out this simple fix in this PR: https://github.com/pytorch/fairseq/pull/2869


[Backport] [2.x] Ensure support of the transport-nio by security plugin (HTTP) (#16474)

Backport of https://github.com/opensearch-project/OpenSearch/pull/16474 to 2.x

@andrross if you don't mind this backport please, thank you!


Support for removing the table name from the column name in the workbench query results

Is your feature request related to a problem? Please describe.
When downloading the query result as 'csv', the column name is 'table name.column name', so it must be rename every time in R or Python
Describe the solution you'd like
Support for removing the table name from the column name in the workbench query results
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@ufoscw  hmm. I think there is a way to solve it by connection options, please check.


Simplify generated build

This was significantly more involved than anticipated.
This PR:

Addresses the 5 linked issues
Consolidates Gradle Plugin configuration to buildscript{}, plugins{} and apply plugin: in build.gradle
Changes Gradle buildSrc/build.gradle and Settings File to non-default, optional features
When the Gradle Settings File feature is selected, it only contains one line defining the root project
When the Gradle buildSrc feature is selected buildscript{} is not generated, they are equivalent
Works around https://github.com/grails/grails-gradle-plugin/issues/351 by using "apply plugin:" for Gradle plugin ids that are not published to the Gradle Plugin repo and are not release versions.  Release versions work fine with id and version.
significant updates to fix the tests, including one that was not cross platform compatible
adds classpath scope
updates asset-pipeline to 4.5.1
standardizes on double quotes with no parenthesis in build.gradle, did not update web-driver/selenium since it has been removed in 7.0.x
dependencies set to pom will still have parenthesis, since they are required for platform()

Merging up to 7.0.x is going to be fun.

This required significant changes to get the desired outcome. We need to debate whether this is completed in 6.2.2 or 7.0.0.
https://github.com/grails/gorm-docs/issues/24 - ticket to update documentation.

I'm not sure it's even going to help anyone out, but rather cause more confusion. If a user is on Grails 5 or lower, is there any need to upgrade to Grails 6 before upgrading to Grails 7? If a user is on Grails 6 it should be an easy operation to remove buildSrc when upgrading to Grails 7 (if that is what the user wants).

@matrei we need to think from the perspective of what is create-app for?  Creating new apps.

Why create an overcomplicated build for a new app only for someone to have to dramatically change it in an upcoming release?
Grails <= 6.2.0 was broken.  We were just cleaning up an inherited mess and are putting our names behind the release.  Let's finish with a more polished product we can be proud o.
This has to be done either way for Grails 7.
This can continue to evolve independently as it is only forge.
The current behavior is inconsistent with Grails shell.  Does Grails shell create-app create a buildSrc?

I've seen this test fail several times.  Deleting build manually gets it working again.
> Task :grails-forge-core:test FAILED

GradleSpec > test buildSrc/build.gradle FAILED
    Condition not satisfied:

    buildSrcGradle
    |
    ""
        at org.grails.forge.build.gradle.GradleSpec.test buildSrc/build.gradle(GradleSpec.groovy:50)


User-facing APIs for _request_for_retry and _normalize_etag

In transformers, we are currently leveraging three private methods (used here):

_raise_for_status
_request_for_retry
_normalize_etag

The first of these is being formalized in https://github.com/huggingface/huggingface_hub/pull/1019 with a hf_raise_for_status public-facing method. This is great as from transformers' side we're certain that this method will be stable and won't have breaking changes in the coming months.
I wonder if the same could be done with _request_for_retry and _normalize_etag, so that transformers could 100% rely on huggingface_hub without needing to juggle with releases when APIs break.
WDYT @Wauplin?
If that is not possible, maybe we could go one step further and implement the method we're using in transformers (get_hub_metadata, which returns the commit hash and associated etag for a given url) directly in huggingface_hub.
cc @sgugger

Hi @LysandreJik , thanks for opening this ticket ! I also wanted to bring up this subject recently but didn't know where to start :grin:
So yes, I'm definitely in favor of reducing as much as possible the use of private methods by making some of them "officially supported". About the get_hub_metadata I'm fine with having it in huggingface_hub so that "_normalize_etag" and "_request_for_retry" can stay private. Do you think that's all the occurrences of private methods ?

Yes, _normalize_etag and _raise_for_status are only used in get_hub_metadata. So implementing this directly in huggingface_hub would remove the need to use those two private methods. Plus it's probably useful for other users.


jscad-web minor updates

Nothing too major here:

add scroll bar to error panel
rename EngineState to ViewState
Unify checkPrimary and checkSecondary into just filesToCheck. This is a first step toward refactoring that code out of main. If there's a reason to have the primary/secondary be seperate, I couldn't see it.
Fix import of @jscadui/fs-provider
Remove old unused file check
Remove unused initScript

Note: I got your commits @hrgdavor, and then rebased. FYI, I've been deleting the branch jscad-web in between PRs in order to avoid merge commits. You should probably delete your local branch before the next time you pull.

If there's a reason to have the primary/secondary be seperate, I couldn't see it.

Yes, intention is to have a primary list that is always checked, versus secondary list that is checked less often. Primary list should always have the initial script and then files that changed. Files that are discovered with require should go in secondary list until changed.

FYI, I've been deleting the branch jscad-web in between PRs in order to avoid merge commits. You should probably delete your local branch before the next time you pull.

ok, it seemed strange that we are using the same branch over and over. But it is ok, I just need to remember to delete local branch after PR merge.


add new test case and new enum mappings

We've discovered some new undocumented enumerations

@kevbite @Blundell89  please review

Looks good ill get it in to master. Good work!


Remove not-longer-necessary run global in .estlintrc

Sindre already merged it.

Coverage remained the same at 88.288% when pulling 56a28d4a9ba2e7ca915deb7db514e3791dab5bc2 on bye-run into fc3569114f795801920ad0728638d1ac017b3809 on master.


update entiled registry verifcation example

Dave asked for this one right away but let me know when the others are ready and I'll review


UPDATE_CAPTURE in Animation's Track has some strange behavior

Godot version:
3.2.2, 4.0.dev

OS/device including version:
MacOS 10, Win 10

Issue description:
In AnimationPlayer, tracks with UPDATE_CAPTURE will make a mess of loop and loop_wrap option.

Same as linear interpolation, but also interpolates from the current value (i.e. dynamically at runtime) if the first key isn't at 0 seconds.

https://docs.godotengine.org/en/stable/classes/class_animation.html
The documentation says above, but it behaves differently than linear interpolation when looping. Also, it is not consistent in behavior between AnimationPlayer and AnimationTree. Moreover, the behavior is also inconsistent between normal and reverse playback. So, it is hard to know what is a bug and what is correct behavior.
And... it may be an exaggeration, but I think that if all the behaviors of UPDATE_CAPTURE are not yet decided, we should consider discontinuing it.

Steps to reproduce:
Try looping the track with the UPDATE_CAPTURE option set, and try switching the loop_wrap option.
Minimal reproduction project:
capture_test.zip

I've never though that playing it backwards not showing anything was a bug, as the "current value" of the track has actually changed to the new value the animation track set, so I always assumed it didn't remember what the previous value was.
But I also had some trouble today with animations with capture tracks played by an AnimationTreeNode. It seems to just take the first keyframe it finds in the track an play it immediately.

But I also had some trouble today with animations with capture tracks played by an AnimationTreeNode. It seems to just take the first keyframe it finds in the track an play it immediately.

I've had the exact same problem! It works in just an animationPlayer version, but when put in an animationTree, it doesn't work!

I've had the exact same problem! It works in just an animationPlayer version, but when put in an animationTree, it doesn't work!

This is being tracked in https://github.com/godotengine/godot/issues/55110 already. Please keep the discussion about AnimationTree there, so we can dedicate this issue to the capture mode not working reliably with looping and backwards playback in AnimationPlayer.


Random port for locator in unit tests

Instead of hard coding a port, select one at random.

Fixed via 14ce40c0603ec0649556775503bc7a56ca288401


UnderlineNav items collapses into the more menu then it is wrapped with a flex

Description
When UnderlineNav is placed into a flex container, even though there are enough space, items are collapsed into the menu. I suspect that is happening due to the overflow behaviour. When there is any "autonomy" given to the underlinenav container using like max-content, auto etc, it simply renders the item in less space as possible i.e. collapses them under more menu.
This is problematic when UnderlineNav is used along with other elements in the same row when styling with flex. An example is reported here in Slack (Hubber only link )
Seems like this was working fine on the v35.21.0 based on the case shared above in slack🤔
Steps to reproduce

Go to any stories on underlinenav storybook
Wrap the Underlinenav with a flex container like below

<Box sx={{display: 'flex'}}>
    <UnderlineNav aria-label="Repository">
      {children.map((child: string, index: number) => (
        <UnderlineNav.Item key={index} icon={child.icon} href="#" aria-current={index === 0 ? 'page' : undefined}>
          {child.name}
        </UnderlineNav.Item>
      ))}
    </UnderlineNav>
</Box>

Observe the items are pulled into the menu even though there is enough space

Version
v36.9.0
Browser
No response

👋🏻 @broccolinisoup I saw this got reopened. Is this something you're actively working on, or do we need to run it through a more focused triage session?

Hi @lesliecdubs 👋🏻 I am not actively working on it however I'd like to address this issue whenever I get a chance. I believe it is a limitation in the component though it is a very low priority. Only one team reported so far (as far as I know) and we found a workaround for them - let me know how you want to move forward with this issue - whether or not keeping it open.

@broccolinisoup thanks for the update! I'm comfortable leaving this issue open in the Primer backlog to circle back to when you have some time. Let's move this back to the "Inbox" if other reports come through about it.


[release-1.22] k3s kubectl logs pod failed behind proxy

Backport to branch release-1.23:

#5712

this will be worked on so no need to move to the next milestone. Will be closed soon before release

This has already been fixed; QA is working to validate it for the k3s2 release.

Yes I am working on closing it out

Validated on v1.22.11-rc1+k3s1
Environment Details

Infrastructure

[ ] Cloud
[ ] Hosted

Node(s) CPU architecture, OS, and Version:

Linux 5.4.0-1009-aws #9-Ubuntu SMP Sun Apr 12 19:46:01 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
NAME="Ubuntu"
VERSION="20.04 LTS (Focal Fossa)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 20.04 LTS"
VERSION_ID="20.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=focal
UBUNTU_CODENAME=focal

Cluster Configuration:

Single node

Config.yaml:

token: summer
write-kubeconfig-mode: 644
bind-address: <redacted_node_private_ip>

Additional files

Update AFTER installing

$ sudo cat /etc/systemd/system/k3s.service.env
HTTPS_PROXY="http://proxy.example.com"
HTTP_PROXY="http://proxy.example.com"
NO_PROXY="<IP_ADDRESS>/16,<IP_ADDRESS>,localhost"

Testing Steps

Setup 1: Without passing bind-address

Install k3s: curl -fL https://get.k3s.io | INSTALL_K3S_VERSION=v1.22.10+k3s1 sh -s - server --write-kubeconfig-mode 644
Once the cluster is up, update /etc/systemd/system/k3s.service.env with the proxy setup
Restart k3s: sudo systemctl restart k3s

Setup 2: With passing bind-address

Copy config.yaml

$ sudo mkdir -p /etc/rancher/k3s && sudo cp config.yaml /etc/rancher/k3s

Install k3s: curl -fL https://get.k3s.io | INSTALL_K3S_VERSION=v1.22.10+k3s1 sh -s - server --write-kubeconfig-mode 644
Once the cluster is up, update /etc/systemd/system/k3s.service.env with the proxy setup
Restart k3s: sudo systemctl restart k3s

Replication Results:

k3s version used for replication:

v1.22.10+k3s1

Check the core-dns pod
Observe the error

kubectl logs -n kube-system coredns-7796b77cd4-rdnsl
Error from server: Get "https://ip-172-31-11-20:10250/containerLogs/kube-system/coredns-7796b77cd4-rdnsl/coredns": proxyconnect tcp: dial tcp <IP_ADDRESS>:6443: connect: connection refused

Validation Results:

k3s version used for validation:

v1.22.11-rc1+k3s2

Follow the same steps as mentioned above with v1.22.11-rc1+k3s1

Check the core-dns pod
Observed there are no errors

$ kubectl logs -n kube-system coredns-7796b77cd4-4s55j
[WARNING] No files matching import glob pattern: /etc/coredns/custom/*.server
.:53
[WARNING] No files matching import glob pattern: /etc/coredns/custom/*.server
[INFO] plugin/reload: Running configuration SHA512 = b941b080e5322f6519009bb49349462c7ddb6317425b0f6a83e5451175b720703949e3f3b454a24e77f3ffe57fd5e9c6130e528a5a1dd00d9000e4afd6c1108d
CoreDNS-1.9.1
linux/amd64, go1.17.8, 4b597f8
...


Avoid triggering UBSan with memcpy(,NULL,0).

This is triggered running t/modules/http2.t from the perl framework when built under
gcc -fsanitize=undefined if it's surprising to get here with len=0 lmk and I can debug further.

Good change. Pesky NULL pointers...


Bad link in Readme

https://blog.continuation.io/time-to-celebrate/ instead of
https://blog.continuation.io/time-to-clelebrate/ in readme

Thanks


Fix Enum#to_s for flag enums to join with pipe

See #3961

This overrides #3961, right? I don't know why there were commas in the first and i didn't stop in questioning them to avoid the ambiguity the parens solved.

I've just read the 3961 thread. Unless something comes up i would also go with pipes :-)

Pipe wins 💯

NOTE: Comma separation is corresponding to Enum.flag macro argument. It may be important.

Good note. I believe Enum.flag to a macro to quickly a flags Enum, I'm not sure it should dictate an Enum representation.

Pipes are good for to_s. I originally copied the comma from to_s from how C# shows these enums, but using pipes is better.


[WIP ]Update to run on CircleCI 2.0

Failing due to token mismatch, how do you want to handle that?

Would be good to separate into two PRs maybe (fixing tests by itself).
Fixes https://github.com/codecov/codecov-bash/issues/82 then.
But it seems like this repo is not well maintained unfortunately.

I'm trying to improve that (not well maintained). We shall see how much time I can devote to that.

Ah, I see now that we had contact in https://codecov.freshdesk.com/helpdesk/tickets/1990 already.
Thanks!

@blueyed This is mainly to setup the tests on CircleCI, not to fix them.
The changes to the test are because I want to try and get coverage reports on the uploader. Which, to be fair, probably should be a separate PR. I'm very bad at scope creep.

What I'm finding I really don't like here is that the tests are failing like crazy and think they are passing. Not cool :(

This should be good to merge once the repo is added back to CircleCI
I have, in theory, tested this here: https://github.com/drazisil/kcov-bats-circleci-codecov

There is still circle.yml in the root.

Waiting to have project added back to CircleCI to actually test before merging

I guess I have to merge something before CircleCI will start building. :crossed_fingers:


Can't get profiler-based dependency tracking to work anymore

I've been struggling for the past few weeks trying to get profiler-based dependency tracking working again on my local dev box, and can't seem to figure out what is going on. This broke at some point after updating to 2.4.x, as it was definitely working before!
OS: Windows 10 - (10.0.15063.0)
Most recent release of Status Monitor installed (have also tried uninstalling and reinstalling).
From what I can see in the event logs, the profiler attached correctly to the IIS worker processes:
.NET Runtime version 4.0.30319.1 - The profiler was loaded successfully.  Profiler CLSID: '{324F817A-7420-4E6D-B3C1-143FBED6D855}'.  Process ID (decimal): 8420.  Message ID: [0x2507].

Profiler key in the registry clearly points to the right location:

In code, I can see that Decorator.IsHostEnabled() returns true (but then again, it seems that's the default value, so it's probably not very useful), however, any call to attempt to decorate a method fails saying the profiler is not active.
Here's output from an ETW trace showing the issue:

System.InvalidOperationException: Instrumentation Agent is not active
         at Microsoft.Diagnostics.Instrumentation.Extensions.Base.Callbacks.Add(Int32 methodId, Func`3 onBegin, Func`5 onEnd, Action`4 onException)
         at Microsoft.Diagnostics.Instrumentation.Extensions.Intercept.Decorator.Decorate(String assemblyName, String moduleName, String methodName, UInt32 argsCount, Func`3 beginCallback, Func`5 endCallback, Action`4 exceptionCallback)
         at Microsoft.ApplicationInsights.DependencyCollector.Implementation.ProfilerRuntimeInstrumentation.DecorateProfilerForHttp(ProfilerHttpProcessing& httpCallbacks)
         at Microsoft.ApplicationInsights.DependencyCollector.DependencyTrackingTelemetryModule.InitializeForRuntimeProfiler()
         at Microsoft.ApplicationInsights.DependencyCollector.DependencyTrackingTelemetryModule.InitializeForRuntimeInstrumentationOrFramework()

I suspect there must be some issue when the profiler is attaching to the process and it's not correctly binding the agent assembly methods, but I do not know if there is some other ETW provider or something similar I can enable to get data out of it. Is there any way to log what the native profiler is doing?

Closing old issue without repros.


Invalid Link

The proper Link needs to be https://admin.teams.microsoft.com/ not ... microsoft.net
[Enter feedback here]

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

ID: 470478ed-3b4e-ced8-0e28-6cda545b8400
Version Independent ID: 95b4044b-3a47-a511-c726-b5ee75863441
Content: Manage meeting settings - Microsoft Teams
Content Source: Teams/meeting-settings-in-teams.md
Service: msteams
GitHub Login: @HowlinWolf-92
Microsoft Alias: v-mahoffman

@scanum The customer is right. It should be (https://admin.teams.microsoft.com/)
Can you fix this?

See this pull request, which will go live tomorrow morning:

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness-pr/pull/8615

OK, it's live now.


any 1.18 port?

says it all in the title, are you planning to port to create 0.5.1b and c for 1.18.2 forge?

feel free to port it yourself, I can't manage 3 versions I am the only dev here and I don't have much free time.


Cannot login

Hi, please help, now i cant login using link provided when start, before it is work, is it from Line that change something?
Traceback (most recent call last):
File "E:/SHARE/scripts/line-py-master/singletrace.py", line 4, in
client = LineClient()
File "E:\SHARE\scripts\line-py-master\linepy\client.py", line 24, in init
self.qrLogin(keepLoggedIn=keepLoggedIn, systemName=systemName, appName=appName, showQr=showQr)
File "E:\SHARE\scripts\line-py-master\linepy\api.py", line 121, in qrLogin
self.tokenLogin(result.authToken, appName)
File "E:\SHARE\scripts\line-py-master\linepy\api.py", line 133, in tokenLogin
self.loadSession()
File "E:\SHARE\scripts\line-py-master\linepy\api.py", line 30, in loadSession
self.revision = self.poll.getLastOpRevision()
File "build\bdist.win32\egg\akad\TalkService.py", line 3440, in getLastOpRevision
File "build\bdist.win32\egg\akad\TalkService.py", line 3463, in recv_getLastOpRevision
akad.ttypes.TalkException: TalkException(parameterMap=None, reason=u'LOG_OUT', code=8)

We will have an update with a fix for this very soon..

Ah i c

Apa sekarang sudah bisa digunakan?

Yes, we have updated to 1.9.0
Please update your linepy with command
pip install linepy --upgrade


CLA missing section #7

The CLA (https://docs.google.com/a/hired.com/forms/d/e/1FAIpQLSf9cbouWptIpMgukAKZZOIAhafvjFCV8hS00XJLWQnWDFtwtA/viewform) is missing a section between 6 and 8. What happened?

Thanks for pointing this out! It looks like there was a typo in the CLA. We've fixed the error.


Working with Flexslider

Hi, great plugin! I got this working with Flexslider using $(window).load but it still only works for the first slide - any suggestions on how to get it to work for text in all slides? Thanks!

Hi Zach, I love this plugin. Great job.
Can we set a character limit for the text line 1  to 10 characters and auto wrap the as you type in?
Thanks


randomNonce in SCEP messages are not allowed per RFC

Describe the bug
Version 13 of the SCEP RFC specifies in Section 3.2.1 which attributes are allowed and/or required in a SCEP message. I see a senderNonce (2 16 840 1 113733 1 9 5) and recipientNonce (2 16 840 1 113733 1 9 6), but no randomNonce (1 2 840 113549 1 9 25 3). However, the sample messages seem to include a randomNonce in both the request as well as the reply.
To Reproduce

Open sample scep_req2 or scep_resp2.
Search for the string "1 2 840 113549 1 9 25 3"

Expected behavior
The attribute randomNonce should not appear in these messages.

It doesn't define what's allowed, merely the minimum that's required, you can add any extra attributes you like.  In this case the messages were generated by an implementation that always adds a randomNonce (to deal with signing predictable messages if there are weaknesses in the hash function used), which is perfectly valid - think of it as a test of an implementation's ability to handle arbitrary attributes.


Update: Microsoft.dotnetPreview version <IP_ADDRESS>210

Removing old versions to reduce clutter
Microsoft Reviewers: Open in CodeFlow

/azp run


Spelling

Summary of the Pull Request
Misspellings identified by the check-spelling action -- note: the spell checker is not included in this PR.
It reported: https://github.com/jsoref/PowerToys/commit/d3d817ed324ea209ba1610f5b7817515bd178015#commitcomment-39328576
References
PR Checklist

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

I'd be happy to add the test (it's in use by microsoft/terminal), but it's best to merge the spelling fixes first and discuss the feature later.

[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments

This isn't built.
I'm happy to split out changes as requested within reason, e.g. comments v. apis vs localization stuff.
The automated testing is the tool mentioned above/below.
Validation Steps Performed
Merging the spell-check action into this branch: https://github.com/jsoref/PowerToys/commit/d5af5c3deff0dbaf4006e9dff928a4c875f617ba

Hi @jsoref
thank you for the contribution, it's greatly appreciated, but we will not be able to accept one single PR for code and documentation.
Can you remove from this PR all the changes made to code and leave only the changes that don't touch the code?
For the code fixes, some may need to be in their own PRs (haven't gone through all of them yet).

Yes...

I've left the original commits in https://github.com/jsoref/PowerToys/tree/spelling-full (+thr => the and ziped => zipped)

@enricogior: @azure-pipelines-bot indicates that the failures are a general problem. Is there anything else you need from me for now?

@jsoref
the failure is not caused by your commit, we will fix it soon.
We'll let you know if other changes are required, probably early next week since we are currently working on the 0.18.1 hotfix release.
Thanks.

this has a conflict now with fancyzones.h

This still too big to be merged at once and we have more PRs on the way.
It may be better to split this by modules.

@jsoref
Great work but yes, I agree with @enricogior this is a a massive change. Can we break this into smaller PRs?  Something this big, while appreciated is hard to merge in.

Happy to split if that'll help these changes get merged, but I'd rather split based on a proposal.
How small do you need the PRs to be? Is it sufficient for me to split out the launcher module?

Yeah. Like, some are decent misspellings like toggle and hit lots of stuff. Others are pretty centralized.
80 files is just a wee bit too hard to CR.

Is this small enough, or do you want me to split out one more? I can split the remainder of src/modules it's 29 files, which is on par with launcher

@jsoref
I left a few comments, when those are fixed we can merge the PR. Thank you.

@enricogior: thanks. I think these are all set.


Endpoint to delete quarantined media

Fixed by https://github.com/turt2live/matrix-media-repo/commit/f110f67c3cf6a758b104ad448c777c99bde070cf


[SPARK-18497][SS]Make ForeachSink support watermark

What changes were proposed in this pull request?
The issue in ForeachSink is the new created DataSet still uses the old QueryExecution. When foreachPartition is called, QueryExecution.toString will be called and then fail because it doesn't know how to plan EventTimeWatermark.
This PR just replaces the QueryExecution with IncrementalExecution to fix the issue.
How was this patch tested?
test("foreach with watermark").

cc @marmbrus

LGTM

Test build #68873 has started for PR 15934 at commit 0f3e4af.

Test build #68873 has finished for PR 15934 at commit 0f3e4af.

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

Merging to master, 2.1 and 2.0

Merged build finished. Test PASSed.

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

Seems like this is a correctness bug as aggregation will give wrong results when using aggregations.


Declare app_label in model Meta class to work with Django 1.9

Fixes RemovedInDjango19Warning:
Model class db_mutex.models.DBMutex doesn't declare an explicit
app_label and either isn't in an application in INSTALLED_APPS or else
was imported before its application was loaded.  This will no longer be
supported in Django 1.9.
Credit: https://github.com/minervaproject/django-db-mutex/commit/429d701cce7ad2cf8fb77f169c4af6f2f27562fd

@justinsg FYI there is a new package available with your change-set: https://pypi.python.org/pypi/django-db-mutex/0.2.3
Thanks for the contributions.
Many thanks @jaredlewis for taking care of the upload.


Play Position

Is it possible to get the progress of the music currently playing?

If you get the most recent version you can define the avrc information that are requested: e.g. just call set_avrc_metadata_attribut_mask(ESP_AVRC_MD_ATTR_TITLE | ESP_AVRC_MD_ATTR_ARTIST | ESP_AVRC_MD_ATTR_ALBUM | ESP_AVRC_MD_ATTR_TRACK_NUM | ESP_AVRC_MD_ATTR_NUM_TRACKS | ESP_AVRC_MD_ATTR_GENRE | AVRC_MEDIA_ATTR_ID_PLAYING_TIME);
before you call start();
However I never tried this out myself.

Thanks, although I was not able to get it to work, so far I was only able to get the total time with ESP_AVRC_MD_ATTR_PLAYING_TIME, but AVRC_MEDIA_ATTR_ID_PLAYING_TIME would not compile and complain that it is not defined. No luck with getting the current position. I don't know how but I think the Bluetooth on cars is able to get the progress bar, which is why I wanted to do the same. I tried using a timer, but would instantly break when I fast forward.


is this proto-type deprecated ?

no updates or progress for this project found for a long time.
is this proto-type deprecated ?

Duplicate of #108

And for now, only can share the same answer than https://github.com/dotnet/Microsoft.Maui.Graphics.Controls/issues/108#issuecomment-1083109110
Thanks for the interest in the project!


Uber H3 GCC Issue when building in Go

FROM golang:alpine AS build-env
WORKDIR /appname
COPY go.mod .
COPY go.sum .
RUN go mod download
COPY . .
RUN go get -d -v ./...
RUN go build -o appname
FROM golang:alpine
WORKDIR /app
COPY --from=build-env /appname/ ./
EXPOSE 8100
ENTRYPOINT ["./appname"]

Getting this error on build with above docker file. How to build uber h3 go lib with go mod?
github.com/uber/h3-go
exec: "gcc": executable file not found in $PATH
Thanks for the help.

It sounds like GCC is not installed in your Docker environment. I haven't tried these steps myself, but could you try what Alpine recommends for installing GCC? https://wiki.alpinelinux.org/wiki/GCC run:apk add build-base

Thanks @isaacbrodsky  works for me

Thanks @isaacbrodsky  works for me


Enable serde feature on playgrounds

Playground metadata is explicitly inspired by docs.rs so looks very similar, was implemented at rust-lang/rust-playground#326
Example of usage at serde, as well as uuid-rs/uuid#224 for the PR at one of the early adopters / implementers of that system.
@shepmaster looked around a bit, and didn't find any way to test it offline so I assume short of having a bespoke test playground you can only add the metadata and try to remember to check if it works the next time you publish a release?
Fixes #102

any way to test it offline

Yeah, there isn't such a way, unfortunately. On quick visual inspection it seems reasonable.

check if it works the next time you publish a release

The crate has to be published and then the playground has to be updated, but yeah.

any way to test it offline

Yeah, there isn't such a way, unfortunately. On quick visual inspection it seems reasonable.

check if it works the next time you publish a release

The crate has to be published and then the playground has to be updated, but yeah.

Thanks, good to know I didn't miss something obvious (for once).