The bot is easily bypassed thanks to Discord.

I got interested in trying the bot so I tried it for a day with friends. 80% of the attempts, it only reacted by the time the server was entirely nuked. And sometimes, it didn't react at all.
It can be helpful for normal excessive actions but not actual nukes.
Im not here trying to put the maker and his work down (I like the verification bot by the way) or anything but it actually is not his fault. It's Discord's.

Discord's Audit Log has inaccuracy: alot of false positives especially when you request that endpoint a lot in a very short period of time. And also if the server is having A LOT of entries being logged regardless of the action type (a large server for an example or a very active server in general). Discord has a habit of returning false log entries when requested a lot. And sometimes.... none (until after few hours which is not very helpful).
There are unfortunately 3 easy ways to break the audit log in a server for at least 3 hours (null entries). Nukers already do it to hide their 'trace' before nuking. This renders the bot pretty much hopeless thanks to Discord.

There are a lot of bypasses that I would like to mention (that are related to the bot), but I won't do that here so that people don't make use of them.
I can dm you them via Discord if you want.
One more thing, Discord will sometimes throw an error regarding this line https://github.com/TrueXPixels/discord-guardian/blob/e20b540b044caa452b52a21740020f16743f4c13/structures/Guild.js#L185. Roles should first be filtered.

There are ways we can do this differently. Which in theory should work, and that would be to rely on the separate events we get from discord. rather than audit logs.
This would be better for lots of cases as there isn't that awful delay when fetching audit log entries, It would also be super good as we aren't making useless api calls.

@ZoroSC That is even worse. You don't know who did what relying on events. And there are for sure innocent stuff that can be caught. I can just kick hundred people and all you would get is member guild leave events which are honestly useless because then, you would not know if these people left on their own or were actually kicked. Apply that on every other event.
From what I've heard, Discord is working on it's own system regarding nukes (and the v8 s supposed to eliminate it and it will be a thing very soon). I recommend not bothering much and not putting more time into this.

Yeah, I'm not a developer of this bot. But development is going to come to a stop due to this.

Good work with the verification bot!

Thanks!

You can now delete the integration of a bot without caring about your position. Someone at the very bottom with Manage Server can kick the anti nuke by deleting its integration
Told you it was not worth it to waste any more time on this :D


Add support for AWS Graviton 2 to AWS Lambda Functions and Lambda Layers

The docker container issue here needs to be investigated separately as it runs locally:
Outputs:
  + imageUri: "894850187425.dkr.ecr.us-west-2.amazonaws.com/basic-container-c9e456c:e3e3522e7e8bffc21b81af0e088a42c6ab7f18860a6e6551f50a2728edd649e9"

Resources:
    + 1 created
    8 unchanged

Duration: 2m31s


code improvements in flex.dart

I don't understand why so many unnecessary switch cases are used here. Maybe they wanted to ensure uniformity, but this made the code difficult to understand. I wanted to update this.
Pre-launch Checklist

[X] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[X] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[X] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[X] I signed the [CLA].
[X] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[X] I added new tests to check the change I am making, or this PR is [test-exempt].
[X] I followed the [breaking change policy] and added [Data Driven Fixes] where supported.
[X] All existing and new tests are passing.

@nate-thegrate Thanks for your comment! I think I learned something, I also reviewed the docs and you are right!

Thanks for the contribution @bedirhanssaglam, based on the feedback above I am going to close this PR.


Does not reconnect after factorio server restart (500 Server Error)

Hey,
the api service does not reestablish the rcon connection after a factorio server restart.
[Fx] HOOK OnStart		github.com/nekomeowww/factorio-rcon-api/v2/internal/grpc/servers.Modules.NewGatewayServer.func2.2() called by github.com/nekomeowww/factorio-rcon-api/v2/internal/grpc/servers.Modules.NewGatewayServer.func2 ran successfully in 383.852µs
[Fx] RUNNING
2024-12-07T08:42:04.743064559Z [error] [/app/api-server/internal/grpc/servers/interceptors/error_handler.go:50] unimplemented error error="rpc error: code = NotFound desc = Not Found"
2024-12-07T08:54:44.813992829Z [error] [/app/api-server/internal/grpc/servers/interceptors/error_handler.go:42] internal error error="rpc error: code = Internal desc = write tcp <IP_ADDRESS>:56668-><IP_ADDRESS>:30100: write: connection timed out" file="/app/api-server/internal/grpc/services/factorioapi/v2/console/console.go:482" function="github.com/nekomeowww/factorio-rcon-api/v2/internal/grpc/services/factorioapi/v2/console.(*ConsoleService).CommandPlayers"

the liveness probe does also continues to think everything is okay.
in a best case the liveness probe will not succeed and the service would retry to connect on its own.
depending on the configuration kubernetes would restart the pod after a few failures.
      livenessProbe:
        failureThreshold: 3
        httpGet:
          path: /healthz
          port: 24180
          scheme: HTTP
        initialDelaySeconds: 10
        periodSeconds: 10
        successThreshold: 1
        timeoutSeconds: 5

in my case the api container would be terminated after 30 seconds when the livenessprobe indicates an issue.
or it would reconnect within that time and everything is okay.
but nether happens and the api is stuck with 500 Server Error

Thanks for reporting, let me reproduce it on my side, the code might not be correctly implemented.

dont know how those scenarios differ - but here is a different log:
2024-12-08T21:15:01.630170461Z [error] [/app/api-server/internal/grpc/servers/interceptors/error_handler.go:42] internal error error="rpc error: code = Internal desc = write tcp <IP_ADDRESS>:56668-><IP_ADDRESS>:30100: write: broken pipe" file="/app/api-server/internal/grpc/services/factorioapi/v2/console/console.go:482" function="github.com/nekomeowww/factorio-rcon-api/v2/internal/grpc/services/factorioapi/v2/console.(*ConsoleService).CommandPlayers"

also after restarting the factorio server - but this time i have looked at the api log way later and not right after the server restarted.
both logs are the result of the same triggered command.

were you able to reproduce this issue?
i utilize the API to figure out if someone is online and rollout restart the factorio-server deployment to apply mod updates on a schedule. after restarting the server the script stops working because the api keeps failing.
(api is its own deployment)
need to handle that manually for now.

Will be fixed in https://github.com/nekomeowww/factorio-rcon-api/pull/19

Sorry for the late reply, I was too rush and busy recent days. And finally got a day to work on side projects.

@nekomeowww Hey,
even though i have updated my deployment i still cannot use the api after restarting the factorio server.
2024-12-26T19:16:59.090181345Z [error] [/app/api-server/internal/grpc/servers/interceptors/error_handler.go:42] internal error error="rpc error: code = Internal desc = write tcp <IP_ADDRESS>:60752-><IP_ADDRESS>:30100: write: broken pipe" file="/app/api-server/internal/grpc/services/factorioapi/v2/console/console.go:482" function="github.com/nekomeowww/factorio-rcon-api/v2/internal/grpc/services/factorioapi/v2/console.(*ConsoleService).CommandPlayers"

here are some details:
cronjob to trigger rollout restart.
apiVersion: v1
kind: ServiceAccount
metadata:
  name: factorio-rollout-restart
  namespace: factorio
---
apiVersion: rbac.authorization.k8s.io/v1
kind: Role
metadata:
  name: factorio-rollout-restart-role
  namespace: factorio
rules:
  - apiGroups: [""]
    resources: ["pods"]
    verbs: ["list", "get"]
  - apiGroups: ["apps"]
    resources: ["deployments"]
    verbs: ["patch", "get"]
---
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
  name: factorio-rollout-restart-rolebinding
  namespace: factorio
subjects:
  - kind: ServiceAccount
    name: factorio-rollout-restart
    namespace: factorio
roleRef:
  kind: Role
  name: factorio-rollout-restart-role
  apiGroup: rbac.authorization.k8s.io
---
apiVersion: batch/v1
kind: CronJob
metadata:
  name: factorio-rollout-restart
  namespace: factorio
spec:
  schedule: "*/15 * * * *"
  suspend: true
  jobTemplate:
    spec:
      completions: 1
      template:
        spec:
          serviceAccountName: factorio-rollout-restart
          containers:
            - name: factorio-rollout-restart
              image: REGISTRY/library/factorio-rollout-restart:1.0.0
              imagePullPolicy: Always
              securityContext:
                runAsNonRoot: true
                readOnlyRootFilesystem: true
                runAsUser: 1001
                capabilities:
                  drop:
                    - ALL
              resources:
                limits:
                  memory: "128Mi"
                  cpu: "200m"
                requests:
                  memory: "64Mi"
                  cpu: "100m"
          restartPolicy: OnFailure

factorio-rollout-restart Dockerfile:
FROM docker.io/library/python:3.13-alpine3.21@sha256:657dbdb20479a6523b46c06114c8fec7db448232f956a429d3cc0606d30c1b59

RUN apk add --no-cache curl \
       && apk add --no-cache --virtual .build-deps gcc libffi-dev musl-dev python3-dev \
       && pip install --no-cache-dir --upgrade pip

RUN addgroup --gid 1001 appuser \
       && adduser --uid 1001 --ingroup appuser --home /app --shell /sbin/nologin --disabled-password appuser \
       && chown --recursive appuser:appuser /app

WORKDIR /app

COPY src/requirements.txt /app/requirements.txt
COPY src/rollout_restart.py /app/rollout_restart.py

RUN pip install --no-cache-dir --requirement /app/requirements.txt \
       && apk del .build-deps

USER appuser

ENTRYPOINT ["python", "rollout_restart.py"]

requirements.txt
kubernetes
requests

rollout_restart.py
#!/usr/bin/env python

"""
This script restarts Factorio server deployments in a Kubernetes cluster.
It checks the online status of players via the rcon-API
and restarts the server's deployment if no players are online
and the pod has exceeded a specified age threshold.
"""

import os
import argparse
import json
import logging
import sys
from datetime import datetime, timedelta, timezone
import requests
from kubernetes import client, config

def parse_options():
    """
    Parse command-line arguments and environment variables.

    Returns:
        argparse: The parsed arguments.
    """
    parser = argparse.ArgumentParser(
        description="Restart Factorio server deployments."
    )
    parser.add_argument(
        "--namespace",
        default=os.getenv("NAMESPACE", "factorio"),
        help="Kubernetes namespace",
    )
    parser.add_argument(
        "--deployment",
        default=os.getenv("DEPLOYMENT", "factorio-server"),
        help="Kubernetes deployment name",
    )
    parser.add_argument(
        "--app",
        default=os.getenv("APP", "factorio-server"),
        help="Kubernetes app name",
    )
    parser.add_argument(
        "--threshold",
        type=int,
        default=int(os.getenv("THRESHOLD", "4")),
        help="Age threshold in hours for restarting the pod",
    )
    parser.add_argument(
        "--api-url",
        default=os.getenv(
            "API_URL", "http://factorio-rcon-api.factorio:30110"
        ),
        help="Base API URL for checking player status",
    )
    parser.add_argument(
        "--debug",
        type=bool,
        default=os.getenv("DEBUG", "False").lower() in ["true", "1"],
        help="Enable debug mode for verbose logging",
    )
    return parser.parse_args()

def setup_logging(debug):
    """
    Set up logging configuration.

    Args:
        debug (bool): If True, set logging level to DEBUG, otherwise INFO.
    """
    level = logging.DEBUG if debug else logging.INFO
    logging.basicConfig(
        level=level,
        format="%(asctime)s [%(levelname)s] %(message)s",
    )

def is_anyone_online(api_url):
    """
    Check if any player is online via the Factorio API.

    Args:
        api_url (str): The base URL for the Factorio API.

    Returns:
        bool: True if any player is online, False otherwise.
        dict: The response data from the API.
    """
    try:
        logging.debug(
            "Sending request to %s/api/v2/factorio/console/command/players",
            api_url,
        )
        response = requests.get(
            f"{api_url}/api/v2/factorio/console/command/players", timeout=30
        )
        response.raise_for_status()
        data = response.json()

        for player in data.get("players", []):
            if player["online"]:
                return True, data
        return False, data
    except requests.RequestException as e:
        logging.error("Error checking player status: %s", e)
        logging.debug("Detailed traceback:", exc_info=True)
        sys.exit(1)

def check_container_age_and_restart_if_needed(
    namespace, app, deployment, threshold
):
    """
    Check the age of the container and restart it if necessary.

    Args:
        namespace (str): The Kubernetes namespace.
        app (str): The Kubernetes app name.
        deployment (str): The deployment name.
        threshold (int): The age threshold (in hours) for restarting the pod.
    """
    try:
        config.load_incluster_config()
        v1 = client.CoreV1Api()
        apps_v1 = client.AppsV1Api()

        label_selector = (
            f"app.kubernetes.io/name={app}" if app else f"app={deployment}"
        )
        logging.debug(
            "Fetching pods in namespace %s with label: %s",
            namespace,
            label_selector,
        )
        pods = v1.list_namespaced_pod(namespace, label_selector=label_selector)

        if not pods.items:
            logging.warning(
                "No pods found for deployment %s in namespace %s.",
                deployment,
                namespace,
            )
            return

        pod = pods.items[0]
        start_time = pod.status.start_time

        if not start_time:
            logging.warning("Failed to retrieve pod start time.")
            return

        current_time = datetime.now(timezone.utc)
        age = current_time - start_time

        if age > timedelta(hours=threshold):
            logging.info(
                "Pod is older than %d hours (Age: %s). "
                "Triggering a rollout restart...",
                threshold,
                age,
            )
            apps_v1.patch_namespaced_deployment(
                name=deployment,
                namespace=namespace,
                body={
                    "spec": {
                        "template": {
                            "metadata": {
                                "annotations": {
                                    "rollout-restarted-at": datetime.now(
                                        timezone.utc
                                    ).isoformat()
                                }
                            }
                        }
                    }
                },
            )
            logging.info("Rollout restart triggered successfully.")
        else:
            logging.info(
                "Pod age is %s, which is within the acceptable limit "
                "(less than %d hours).",
                age,
                threshold,
            )

    except client.exceptions.ApiException as e:
        logging.error("Error interacting with Kubernetes API: %s", e)
        logging.debug("Detailed traceback:", exc_info=True)
        sys.exit(1)

def main():
    """
    Main function rollout restart the Factorio server Kubernetes deployment.
    """
    options = parse_options()
    setup_logging(options.debug)

    logging.info("Starting Factorio server rollout restart script...")

    online_status, data = is_anyone_online(options.api_url)

    if online_status:
        logging.info("Players online.")
        logging.debug(json.dumps(data, indent=4))
    else:
        logging.info("No players online. Checking container deployment age...")
        check_container_age_and_restart_if_needed(
            options.namespace,
            options.app,
            options.deployment,
            options.threshold,
        )

if __name__ == "__main__":
    main()

feel free to include this as usage example or whatever. ;)


fix: getIpFamily not detecting IPv6 support [ROAD-604]

Fixes #116

@pavel-snyk it's little tricky with tests here, since we rely on OS network stack supporting IPv6. I've tricked it by stubbing the function the code-client depends to decide whether IPv6 is supported or not. Let me know if you have any better ideas here 🤔
Thanks for poking on testing here 👍


Function app won't deploy due to Storage Account not existing.

0.3.539
Having a function app resource that depends on a storage module fails with "ResourceNotFound" when deploying the function app. The storage account is always created but the function app always fails. When re running the script, the function app will then deploy.
To Reproduce
var storageName = 'sa12jkhsdfjghsd'

module storage './storage.bicep' = {
  name: 'storage'
  params: {
    storageName: storageName
  }
}

module appinsights './appinsights.bicep' = {
  name: 'appInsightsDeploy'
  params: {
    appInsightsName: 'testappinsight'
  }
}

module appService './appservicefunction.bicep' = {
  name: 'appServiceDeploy'
  params: {
    appServiceName: 'testServiceName'
  }
}

var functionAppName = 'testfuncappscott'
resource functionApp 'Microsoft.Web/sites@2020-06-01' = {
  name: functionAppName
  location: resourceGroup().location
  kind: 'functionapp'
  properties: {
    enabled: true
    hostNameSslStates: [
      {
        name: '${functionAppName}.azurewebsites.net'
        sslState: 'Disabled'
        hostType: 'Standard'
      }
      {
        name: '${functionAppName}.scm.azurewebsites.net'
        sslState: 'Disabled'
        hostType: 'Standard'
      }
    ]
    serverFarmId: appService.outputs.appServiceId
    reserved: false
    isXenon: false
    hyperV: false
    siteConfig: {
      appSettings: [
        {
          name: 'AzureWebJobsStorage'
          value: 'DefaultEndpointsProtocol=https;AccountName=${storageName};EndpointSuffix=${environment().suffixes.storage};AccountKey=${listkeys(resourceId('Microsoft.Storage/storageAccounts', storageName), '2019-04-01').keys[0].value}'
        }
        {
          name: 'APPINSIGHTS_INSTRUMENTATIONKEY'
          value: appinsights.outputs.instrumentationKey
        }
        {
          name: 'FUNCTIONS_WORKER_RUNTIME'
          value: 'dotnet'
        }
        {
          name: 'FUNCTIONS_EXTENSION_VERSION'
          value: '~3'
        }
      ]
    }
    scmSiteAlsoStopped: false
    clientAffinityEnabled: false
    clientCertEnabled: false
    hostNamesDisabled: false
    dailyMemoryTimeQuota: 0
    httpsOnly: true
    redundancyMode: 'None'
  }
  dependsOn: [
    storage
  ]
}

Error Message:
Deployment failed. Correlation ID: 6902c6da-dcdf-4fb9-90db-e1cdf3524138. {
  "error": {
    "code": "ResourceNotFound",
    "message": "The Resource 'Microsoft.Storage/storageAccounts/sa12jkhsdfjghsd' under resource group 'MyResourceGroup2' was not found. For more details please go to https://aka.ms/ARMResourceNotFoundFix"
to https://aka.ms/ARMResourceNotFoundFix"
  }
}

Hmm - can you share the module code as well? particularly for storage.bicep
Since you have the dependsOn with storage, it looks like your dependencies are set correctly

@minLength(3)
@maxLength(24)
param storageName string = 'sa12jkhsdfjghsd'

@allowed([
  'Standard_LRS'
  'Standard_GRS'
  'Standard_RAGRS'
  'Standard_ZRS'
  'Premium_LRS'
  'Premium_ZRS'
  'Standard_GZRS'
  'Standard_RAGZRS'
])
param storageSKU string = 'Standard_LRS'

param location string = 'westeurope'
param resourceTags object = {}

resource stg 'Microsoft.Storage/storageAccounts@2019-04-01' = {
  name: storageName
  location: location
  tags: resourceTags
  sku: {
    name: storageSKU
  }
  kind: 'StorageV2'
  properties: {
    supportsHttpsTrafficOnly: true

  }
}

output storageObject string = stg.id

for your storage account definition try adding: scope: storage where storage = the symbolic name used for the module.

The issue is that the listKeys() function is greedy and the deployments engine attempts to resolve that function at the beginning of the deployment before any resources have been deployed. We plan to fix this, but in the meantime if the listKeys() call is inside a module we won't do the greedy check. In this case, deploying the website in a module should solve the problem.

Putting it in a module did the trick.
Thanks for the quick response guys


Update supervisor to v6.0.1

Change-Type: patch

Created automatically using hub and this script :)

good point @agherzan - I'm modifying the script to use the correct format


Many methods on Parser::Source::Range from Offense.highlighted_area raise NoMethodError

The highlighted_area method on offense is public API. However, many of its methods will result in an error because it is not instantiated right: https://github.com/rubocop/rubocop/blob/dea3b768caa6b3b8742516a52f34325110364e76/lib/rubocop/cop/offense.rb#L138-L145
source_line will be a string here, while Parser::Source::Range expects a Parser::Source::Buffer
Steps to reproduce the problem
# frozen_string_literal: true

require "rubocop"

source_buffer = Parser::Source::Buffer.new("test.rb")
source_buffer.read
location = Parser::Source::Range.new(source_buffer, 0, 2)

offense = RuboCop::Cop::Offense.new(:warning, location, "unused var", "Lint/UselessAssignment", :corrected)

puts offense.highlighted_area.column
# => .../parser-<IP_ADDRESS>/lib/parser/source/range.rb:93:in `column': undefined method `column_for_position' for an instance of String (NoMethodError)
#
#         @source_buffer.column_for_position(@begin_pos)
#                       ^^^^^^^^^^^^^^^^^^^^

RuboCop version
$ [bundle exec] rubocop -V
1.61.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.31.1, running on ruby 3.3.0) [x86_64-linux]
  - rubocop-minitest 0.34.5
  - rubocop-rake 0.6.0
  - rubocop-sorbet 0.7.7

Hi, I had the same conclusion on Range initialization few weeks ago: https://github.com/rubocop/rubocop/issues/12527
Feel free to close my issue in the same time if this one get fixed 👍

Oh, I see. Not sure why I didn't find this while I was searching for duplicates. Thanks for pointing me there instead.


XDG-Shell: update to 1.6.0

Allow the OZONE_WAYLAND_USE_XDG_SHELL environment variable
to work with the latest Weston 1.6.0 release. It will not
work with Weston 1.5.0 anymore. Functionality is unchanged.
(we define a general "wl_array_for_each_type()" macro to
handle "wl_array" types nicely in the future)

Obsoletes https://github.com/01org/ozone-wayland/pull/331.

@kalyankondapally, @baleboy

Merged

I suppose we'll need this in the branch we are using in Crosswalk (thanksgiving?)


404 when running composer require

Hey! I'm getting this error when running composer require to install the package:
❯ composer require tomatophp/filament-media-manager

In CurlDownloader.php line 641:

  The "https://filament-gallery.composer.sh/packages.json" file could not be downloaded (HTTP/2 404 )

I had
    "repositories": [
        {
            "type": "composer",
            "url": "https://filament-gallery.composer.sh"
        }
    ],

in my composer.json, removing it fixed this! Sorry to waste your time.


Big problem with v6.16.2 (MUST REVERT)

I think we got a big problem with the v6.16.2 update.
Here is the scenario:
Testing a CRUD sequence. POST successfully creates an object, but some assertion fails (maybe some property is not quite correct, or the Location header is missing, something not essential, but causing the test to fail). There is a number of tests depending on the POST: GET, PATCH, GET after PATCH, and finally DELETE. Most of them have a @ref to the original POST. But since the response variable gets cleared, for every test with the POST test dependency, POST will be called again and again. This is really bad.
Try this sample:
### Test A
# @name a
POST https://httpbin.org/anything

?? status == 200
?? body json exists

### Test B
# @name b
# @ref a
{{
  exports.$cancel = !(aResponse.statusCode == 200);
}}
GET https://httpbin.org/anything

?? status == 200

### Test C
# @name c
# @ref a
{{
  exports.$cancel = !(aResponse.statusCode == 200);
}}
PATCH https://httpbin.org/anything

?? status == 200

### Test Z
# @name z
# @ref a
{{
  exports.$cancel = !(aResponse.statusCode == 200);
}}
DELETE https://httpbin.org/anything

?? status == 200

You see how a failed assertion in referenced tests essentially turns @ref into @forcedRef?
Not only that. Now there is absolutely no way to reference a response from a request with a failed assertion. The request could've been good and an assertion could be minor, but since assertion failed, any test referencing it will be automatically skipped. Meaning, any failed assertion after a successful POST will not allow to DELETE the object.
We definitely do not want that. And I do not think there is a way to fix it.
I think the original behavior made more sense. A request with  @ref or @forceRef dependency should be skipped if the call fails and the response variable does not get set, but it should not be automatically skipped if the call completes but an assertion fails (or multiple assertion fail). If an assertion fails, we at least can detect the condition programmatically and use the $cancel variable to submit or cancel a request, but if we automatically skip it on a failed assertion, there is no way around.
Does this make sense?

What is your expected behaviour for this use case?

the POST response variables are always set, but the depending requests are only executed if no test fails
negative test results does not stop depending requests from executing
...?

These sound like behaviors (what the system does) vs use cases (what I want to do), so if you're asking about the proposed system behaviors (and there seems to be two), here is my take.

the POST response variables are always set, but the depending requests are only executed if no test fails

This solves the problem of turning @ref into @forcedRef, but unless I'm missing something, the response variables will not be usable for anything (other than not calling the referenced request repeatedly), since the dependent requests will be skipped. I still need to use @ref to get the response variable, right? But if the test with @ref is automatically skipped, the response variable in the @ref is not accessible, right? I suppose, I can save the object ID from the POST in a global (in the POST post-request script), or do some other hack, but it would be a hassle. So, unless I use some globals or whatever, I will not be able to delete an object successfully created by POST (if a POST assertion failed) by referencing the POST's response variable. No good, IMHO.

negative test results does not stop depending requests from executing

I think this makes most sense and that's how the system should behave. The depending request can always check a condition and set $cancel programmatically. I suppose, the only case when you may want to automatically skip a request is when the depending request causes an exception (like, when not being able to make a call at all due to an exception), but it would be fine if the it was not automatically skipped even in this case because the pre-request script should be always checking whether to run the test after a failed dependency anyway, so automatic skipping would not make anything easier. If it's easier to automatically skip a request on a dependency exception (but not on failed assertion!), it's fine, if it's easier not to skip, it's fine too.
Another thing I want to mention is that if a referenced request fails due to an exception (not due to an assertion), it should not be repeatedly called from all @ref instances pointing to it in a single test run. Meaning, under no circumstances should @ref turn into @forcedRef. For example, if I have tests A, B, C, ... Z, where A calls POST to create an object and B, C, ...Z use @ref A, and A fails due to an exception, in a single test run (whether I execute all A-Z test, or some test with an A dependency), A should be attempted only once. If A fails (due to network issue, or whatever) and the A response variable does not get set, there is no point to repeatedly call A again and again for every dependency from B to Z.
What do you think?

I updated the behaviour:

negative test results from reference does not stop dependent request
errored reference is not executed multiple times if it is referenced multiple times

@AnWeber You are the best!


Marketplace VM deployment - Data Producers comma delimitted

If you enter a comma delimited list in the Data Producers field, it should be added to the Istio Authorization policy as
["user<EMAIL_ADDRESS>Currently it is set as user<EMAIL_ADDRESS>which is incorrect.
The VM script should parse the data producers text as shown above or the user could enter that value directly.

Parse the comma delimited string OR
ask the user to enter the string as shown above.

https://istio.io/latest/docs/reference/config/security/authorization-policy/


incorrect version information in container images

The output of headscale version in linux/amd64 images is incorrect:
➜  ~ docker run --rm -ti headscale/headscale:0.10.3 headscale version
dev
➜  ~ docker run --rm -ti headscale/headscale:0.10.4 headscale version
dev
➜  ~ docker run --rm -ti headscale/headscale:0.10.5 headscale version
dev

The would expect the output to match the container tag.

We need to pass the variable into the Dockerfile, this is a good first issue for someone with (or without) docker experience.


stack overflow in task CHIP while adding M5stack CHIP accessories to home

ERROR A stack overflow in task CHIP has been detected.
Backtrace:0x40090a8e:0x3ffddb30 0x40091059:0x3ffddb50 0x40091306:0x3ffddb70 0x40091edc:0x3ffddbf0 0x400913fc:0x3ffddc10 0x400913b2:0x0df15cce |<-CORRUPTED
0x40090a8e: panic_abort at /Users/apple/tools/esp-idf/components/esp_system/panic.c:341
0x40091059: esp_system_abort at /Users/apple/tools/esp-idf/components/esp_system/system_api.c:106
0x40091306: vApplicationStackOverflowHook at /Users/apple/tools/esp-idf/components/freertos/xtensa/port.c:474
0x40091edc: vTaskSwitchContext at /Users/apple/tools/esp-idf/components/freertos/tasks.c:2770
0x400913fc: _frxt_dispatch at /Users/apple/tools/esp-idf/components/freertos/xtensa/portasm.S:431
0x400913b2: _frxt_int_exit at /Users/apple/tools/esp-idf/components/freertos/xtensa/portasm.S:231
ELF file SHA256: 08a43fa0300f1773
Rebooting...
ets Jul 29 2019 12:21:46
ESP-IDF Setup:
M5Stack connected to Mac mini
git clone https://github.com/espressif/esp-idf.git
cd esp-idf
git checkout release/v4.2
git fetch --all
git reset --hard d9ec7df39
git submodule update --init
pip install virtualenv
full logs attached here:
M5Stack_CHIP_StackOverflow_Logs.pdf

Also experienced this issue with a devkitc, identical error message

Also experienced this issue with a devkitc, identical error message

@shah-kartik @MarkMk1 CHIP uses ESP-IDF v4.3 release tag. Please refer to the instructions in the all-clusters-app.
If you are using IDF v4.3 with latest CHIP code and facing the issue, please share the console logs. Is this reproducible everytime at your end with the all-clusters-app? Can you please share any other steps/deviations from the default example along with the sdkconfig file?

You're correct in my case. Other issues with 4.3 (I was running 4.2), but there's no stack overflow at least

We're in progress of reviewing older issues. Is this still an issue given how old the SDK this was tested against? Please re-open if so. Thanks!


[Identity] Mark German cloud as deprecated

Closes #31276

[x] Mark German cloud as deprecated
[x] Fix links in Changelog entry

API change check
API changes are not detected in this pull request.


Improve inline rendering performance

Inline rendering doesn't require load the full controller and simulate a
request. Instead, the controller for async loads and the helper for
inline loads, use the Display class that all the components controller
should implement.
Only one get action is available per controller, by default it is the
display action.

Discarded, this approach introduces too many complexities.


Add unit tests for sriov, traffic control and netlink event handler

This adds appropriate unit tests for modules related to sriov functions, traffic control and netlink event handler. For testing sriov module, it uses sriov mocking framework from sriov-cni and also needed adaption in implementation to either use real or mock netlink manager.

/lgtm


Manifest file is missing

Hello !
Whenever I try to unload the package on google chrome, it says manifest file is missing. maybe it's a JSON file issue ?

Could you please send me more details about your problem? Like the manifest file or the file tree hierarchy
Is this build still affected by the issue?

Thank you soo much ! the manifestation file that you provided solved the issue. I just unloaded the files in main quizletbypass folder. Now it;s working perfectly


Unhandled exception

Hello!
When running:
./t2md --output-folder output --no-numbering --single-file --remove-empty-folders

during phase 1, after many "Finished xxx", it crashes with this error:
Unhandled exception. System.IO.IOException: The process cannot access the file '/Users/alranel/Downloads/output/osx-x64/output/t2md/xxx.json' because it is being used by another process.
   at Microsoft.Win32.SafeHandles.SafeFileHandle.Init(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize)
   at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String fullPath, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize)
   at System.IO.Strategies.OSFileStreamStrategy..ctor(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize)
   at System.IO.Strategies.FileStreamHelpers.ChooseStrategy(FileStream fileStream, String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, Int64 preallocationSize)
   at System.IO.File.Create(String path)
   at GoldenSyrupGames.T2MD.Cli.ProcessTrelloBoardAsync(TrelloApiBoardModel trelloApiBoard, CliOptions options)
   at GoldenSyrupGames.T2MD.Cli.RunAsync(CliOptions options)
   at GoldenSyrupGames.T2MD.Cli.Main(String[] args)
   at GoldenSyrupGames.T2MD.Cli.<Main>(String[] args)

Thanks for the report!

Did you have the /Users/alranel/Downloads/output/osx-x64/output/t2md/ folder or /Users/alranel/Downloads/output/osx-x64/output/t2md/xxx.json path open in any other application at the same time?
Does it happen every time you run that, or just intermittently?
Does it throw on the same .json file every time, or different?
I don't have a Mac to test on. Do you have a linux box you can run it on, and does it occur there too?

Thank you for the quick reply :)
Here's the answers to your question:

Yes! I do have two boards with the same name.
No, I did not have the folder or any other file open in other applications.
It was happening consistently.
I can't reproduce it anymore because now the process is failing with the error reported in #42, but I seem to remember it was happening with the same board which may be the duplicated one. But my memory could fail here. I don't know how to reproduce it now :)
I don't have a Linux box at the moment but I can do it hopefully in the next days

Could you please try again now with v4.0.3 and let me know if it's fixed.

I confirm this issue is fixed in v4.0.3. Thank you! :)


ServiceAccount Volume compatible with Bound ServiceAccount tokens

Is your feature request related to a problem? Please describe:
Yes, this request should be addressing the need to adapt to the Bound ServiceAccount tokens and projected volumes change.
Describe the solution you'd like:
Due to bound service account tokens, pod level tokens are expiring. Toda KubeVirt generates disk images for a token (http://kubevirt.io/api-reference/main/definitions.html#_v1_serviceaccountvolumesource), thus if it expires, then it's still visible inside the guest, because the token is not getting updated on the virtual disk.
We need to fix this.

Fix how we can do this with disks (i.e. unplug, replug)?
Deprecate the disk approach, and use a file-system (virtiofs) based approach which will always pass through the  current pod level token. Here we need to check that virtiofs can be used in a restricted pod.

I'd personally prefer #2104
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.

/reopen
/remove-lifecycle rotten

This issue should have been addressed in https://github.com/kubevirt/kubevirt/pull/9432 since it allows to expose ServiceAccounts through virtiofs. Therefore, ServiceAccounts changes could be reflected in vmis without requiring to be rebooted. This PR does not deprecate the current ISO based systems, it just add this possibility.

@jcanocan While the above is true, an explicit feature gate is needed. We need to have a look if we can run virtiofs as non-root and without capabilities for these cases.

I have run some test without non-root and without capabilities in the virtiofs container, and the VMI is capable of reading the content of ConfigMaps, Secrets, DownwardAPI and ServiceAccounts.
Regarding the feature gate, are you suggesting to expose a separate feature gate from the VirtIOFSGate specifically for config volumes  i.e. ConfigMaps, Secrets, DownwardAPI and ServiceAccounts shared with virtiofs? e.g. VirtiofsConfigVolumes. If yes, should the user enable both (VirtIOFSGate and VirtiofsConfigVolumes) in order to share their volumes with virtiofs?

@jcanocan

I have run some test without non-root and without capabilities in the virtiofs container, and the VMI is capable of reading the content of ConfigMaps, Secrets, DownwardAPI and ServiceAccounts.

Sounds like upstream worth! Lets get this in.
For the latter we do not need new feature gate but any feature that requires virtiofs should be protected right now unless we get the virtiofs working without capabilites and root.

@jcanocan

I have run some test without non-root and without capabilities in the virtiofs container, and the VMI is capable of reading the content of ConfigMaps, Secrets, DownwardAPI and ServiceAccounts.

Sounds like upstream worth! Lets get this in.
For the latter we do not need new feature gate but any feature that requires virtiofs should be protected right now unless we get the virtiofs working without capabilites and root.

We could implement a mechanism to discriminate when a user tries to share a configVolume with virtiofs. In that case, we can deploy a non-root virtiofs container without capabilities. Moreover, AFAIK --sandbox option needs to be none.  I'm agree that this should be protected with a feature gate, is the existing VirtIOFSGate feature gate good enough in your opinion?

is the existing VirtIOFSGate feature gate good enough in your opinion?
Yes that is enough

/reopen

I think  the final bit would be to support live migration.


can't access static html file under NUXT

The Vue.js project converts to a Nuxt.js project and displays 404 when accessing static resource HTML files. How do I parse static HTML files under static?

This question is available on Nuxt community (#c9793)

Did you find a solution to this @jiandian ?
Trying to verify ownership of a website to some advertisers, but their system demands a HTML Verification File.
No idea how to place it. A simpler website would only need it in the root folder.

Did you find a solution to this @jiandian ?
Trying to verify ownership of a website to some advertisers, but their system demands a HTML Verification File.
No idea how to place it. A simpler website would only need it in the root folder.

It has been proposed that the Nginx intercept and the back-end handle the static HTML file redirection.


Start adding documentation

What kind of change does this PR introduce?
Adds documentation for the main behaviors of the SuperbaseClient Object. Also adds usage, installation and other documentation to the README. I'll add more docs like that soon.
What is the current behavior?
There's little documentation.
What is the new behavior?
There's docs 🎉

Any updates on this? There's almost no code changes and just updates the README and comments in the code


Error reading temporary file of Diamond

Hi,
When running "CAT contigs" I get this error:
No such file or directory Error: Error reading file /CAT_outout_directory/diamond-tmp-J1k18Q terminate called after throwing an instance of 'File_read_exception'   what():  Error reading file /CAT_outout_directory/diamond-tmp-J1k18Q [2019-04-03 17:03:41.921512] ERROR: Diamond finished abnormally. [2019-04-03 17:04:51.988192] ERROR: input file /CAT_outout_directory/sample.CAT.ORF2LCA.txt does not exist.
And my script is:
`#$ -pe smp 16
#$ -l h_vmem=8G
CAT contigs -n 16 -c ${Scaffold} -d ${Dir_DB} -t ${Dir_taxon} -o ${Dir_out}/${Sample}
--proteins_fasta ${Protein}
--path_to_prodigal /usr/.pyenv/shims/prodigal
--path_to_diamond /usr/.pyenv/shims/diamond`
Version of dependencies are:
DIAMOND: 0.9.14
Prodigal: V2.6.3
Python 3:  3.6.8
Best,
Mika

Hi Mika,
Could you send me the entire CAT log?
Bastiaan

Hi Bastiaan,
Thank you for your help.
This is the entire CAT log.
# CAT v4.3.3.

CAT is running. Since a predicted protein fasta is supplied, only alignment and contig classification are carried out.
Rarw!

Supplied command: /home/usr/Analysis/scaf/scaffoldSeq.fasta -d /home/usr/CAT/CAT_prepare_20181212/2018-12-12_CAT_database -t /home/usr/CAT/CAT_prepare_20181212/2018-12-12_taxonomy -o /home/usr/Analysis/cat/sample --proteins_fasta /home/usr/Analysis/cat/sample.predicted_proteins.faa --path_to_prodigal /home/usr/.pyenv/shims/prodigal --path_to_diamond /home/usr/.pyenv/shims/diamond

Contigs fasta: /home/usr/Analysis/scaf/scaffoldSeq.fasta
Taxonomy folder: /home/usr/CAT/CAT_prepare_20181212/2018-12-12_taxonomy/
Database folder: /home/usr/CAT/CAT_prepare_20181212/2018-12-12_CAT_database/
Parameter r: 10
Parameter f: 0.5
Log file: 0.5

-----------------

Doing some pre-flight checks first.
[2019-04-03 18:12:02.154792] Diamond found: diamond version 0.9.14.
Ready to fly!

-----------------

[2019-04-03 18:12:02.457721] Importing contig names from /home/usr/Analysis/scaf/scaffoldSeq.fasta.
[2019-04-03 18:12:04.839788] Parsing ORF file /home/usr/Analysis/cat/sample.predicted_proteins.faa
[2019-04-03 18:12:07.458773] Homology search with Diamond is starting. Please be patient. Do not forget to cite Diamond when using CAT or BAT in your publication!
				query: /home/usr/Analysis/cat/sample.predicted_proteins.faa
				database: /home/usr/CAT/CAT_prepare_20181212/2018-12-12_CAT_database/2018-12-12.nr.dmnd
[2019-04-03 22:27:04.436939] ERROR: Diamond finished abnormally.

This problem seems to be the same as #15 .
#15 questioner said this:
I've successfully increased the memory usage of Diamond search step by adding '--block-size' and '--index-chunks' options to 'shared.py' file.
But I can't find 'shared.py' file.
Thanks,
Mika

Hi Mika,
Since the error is in the Diamond alignment step, it's a little harder to debug this.
If you use a small subset of your data, does CAT finish? You can for instance make a .fasta file that only contains the first 10 scaffolds. I suspect this will run but could you confirm this for me?
If the problem is indeed similar to #15, you're machine has run out of disk space, and I would have expected a different error message from Diamond, but let's test this as well! You can check available disk space in Linux with the command df -h. The temporary solution mentioned in #15 is by changing the shared.py file within the CAT directory: CAT_pack/shared.py. Line 79 to 88 of the code generates and calls the Diamond command:
        subprocess.check_call([path_to_diamond,
                               'blastp',
                               '-d', diamond_database,
                               '-q', predicted_proteins_fasta,
                               '--top', '50',
                               '--matrix', 'BLOSUM62',
                               '--evalue', '0.001',
                               '-o', diamond_file,
                               '-p', str(nproc),
                               '--quiet'])

This would translate into a shell command like this: diamond blastp -d nr.dmnd -q predicted_proteins.fasta --top 50 --matrix BLOSUM62 --evalue 0.001 -o out.alignment.diamond -p 24 --quiet. If you would for instance remove the '--quiet' in line 88 Diamond will be much more focal, or as another example you can add '--block-size', '1', '--index-chunks', '1', to get temporary disk space down.
Having said all this (for reference), I will try to put a new code online soon (within days) that will add these Diamond options. So you can either do this yourself or wait a little more...
Let me know if this helps!
Bastiaan

Hi Mika,
I have just published a new release (v4.4) where you can tune these DIAMOND specific parameters! Let me know if that solved your issue!
Bastiaan

Hi Bastiaan,
Thank you for your advice and the release of new version!
I tried to :
(1)  Use a small subset of the data
The file containing the 10 long scaffold (5.8 M) was stopped, but the 10 short scaffolds (1.1 K) worked well.
(2) Use a new version (v4.4)
I got this error:
Traceback (most recent call last):
  File "/home/usr/tools/CAT/CAT-master/CAT_pack/CAT", line 72, in <module>
    main()
  File "/home/usr/tools/CAT/CAT-master/CAT_pack/CAT", line 56, in main
    contigs.run()
  File "/home/usr/tools/CAT/CAT-master/CAT_pack/contigs.py", line 574, in run
    contigs(args)
  File "/home/usr/tools/CAT/CAT-master/CAT_pack/contigs.py", line 236, in contigs
    tmpdir) = check.convert_arguments(args)
  File "/home/usr/tools/CAT/CAT-master/CAT_pack/check.py", line 46, in convert_arguments
    tmpdir = out_prefix.rsplit('/', 1)[0]
NameError: name 'out_prefix' is not defined

My script is:
#$ -pe smp 16
#$ -l h_vmem=4G

CAT contigs -n ${Thread} -c ${Scaffold} -d ${Dir_DB} -t ${Dir_taxon} -o ${Dir_out}/${Sample} \
	--proteins_fasta ${Protein} \
	--path_to_prodigal /home/usr/.pyenv/shims/prodigal \
	--path_to_diamond /home/usr/.pyenv/shims/diamond \
	--sensitive \
	--block_size 1 \
	--nproc ${Thread}

I installed CAT v4.4 and suppled the absolute path.
Thanks,
Mika

Woops that error is on me, I'm sorry! I have pulled the release from yesterday and put one with a bugfix online (still named v4.4, just downloaded the latest). Could you try again with this version?
Also, what kind of machine are you running this on? Am I correct that you are using 16 cores and 4GB of RAM? How many free disk space do you have?
Sorry for Yesterday's bug, it's already confusing enough with one bug. :)
Best,
Bastiaan

Dear Bastiaan,
Thank you for your reply.
I tried today's version but got this error:
Traceback (most recent call last):
  File "/home/usr/tools/CAT/CAT-master/CAT_pack/CAT", line 72, in <module>
    main()
  File "/home/usr/tools/CAT/CAT-master/CAT_pack/CAT", line 56, in main
    contigs.run()
  File "/home/usr/tools/CAT/CAT-master/CAT_pack/contigs.py", line 574, in run
    contigs(args)
  File "/home/usr/tools/CAT/CAT-master/CAT_pack/contigs.py", line 394, in contigs
    if not check.check_fasta_file(predicted_proteins_fasta):
AttributeError: module 'check' has no attribute 'check_fasta_file'

"CAT -v" return:
CAT v4.4 (April 9, 2019) by F. A. Bastiaan von Meijenfeldt.

Also, what kind of machine are you running this on? Am I correct that you are using 16 cores and 4GB of RAM? How many free disk space do you have?

-> I'm running on Linux. I can use 32 core CPU and 187.6G RAM.
Best,
Mika

Ah I see I was a little too eager to push the fix to you without running the code at least once. I have now done that and the latest version runs on our systems (again just download the latest release, v4.4).
We have run CAT on systems with much less cores and RAM, so that shouldn't be an issue. So let's see if the --block_size set to 1 will make the difference! If that does not work out we will start debugging DIAMOND.
Running DIAMOND in sensitive mode will usually take a lot longer by the way, and we generally do not notice great improvements in performance. Do let us know if it makes a difference for you!
Best wishes,
Bastiaan

Thank you for your help.
The latest version could also be run on my system.
Although I tried to use options of few combination such as --block_size set to 1 and 0.2 and sensitive mode, I received the same error as before.
I’ll try to use DIAMOND of the latest version.
I have two questions:

I'll make the database files myself to use the latest version DIAMOND. (Because CAT_prepare_20190108.tar.gz is generated by DIAMOND v0.9.14)
What is default database files on tbb.bio.uu.nl/bastiaan/CAT_prepare/ downloaded from?  Is the  bacterial complete genomes RefSeq data?
Please tell me about how to download NCBI taxonomy files and nr database files.

I ran CAT using a small scaffolding file.
After I got a named CAT classification file, "CAT Summary" gave this error:

[2019-04-10 21:46:53.353757] ERROR: /home/proj/usr/cat/Sample.ORF2LCA_names.txt is not a CAT classification file.

My script is:

CAT add_names -i ${Dir_out}/${Sample}.ORF2LCA.txt -o ${Dir_out}/${Sample}.ORF2LCA_names.txt -t ${Dir_taxon}
CAT summarise -c ${Scaffold} -i ${Dir_out}/${Sample}.ORF2LCA_names.txt -o ${Dir_out}/${Sample}.summary.txt

The contents of "${Dir_out}/${Sample}.ORF2LCA_names.txt" is:

'# ORF	lineage	bit-score	full lineage names
contig1_1	1;131567;2	135.2	root (no rank)	cellular organisms (no rank)	Bacteria (superkingdom)
contig2_1	ORF has no hit to database.
contig3_1	1;131567;2;1783270;68336;976;200643;171549	135.6	root (no rank)	cellular organisms (no rank)	Bacteria (superkingdom)	FCB group (no rank)	Bacteroidetes/Chlorobi group (no rank)	Bacteroidetes (phylum)	Bacteroidia (class)	Bacteroidales (order)
contig4_1	1;131567;2	116.7	root (no rank)	cellular organisms (no rank)	Bacteria (superkingdom)
contig5_1	ORF has no hit to database.
contig5_2	ORF has no hit to database.

Best,
Mika

Hi Mika,
Could you send me by email your contigs fasta? You can find my email on my webpage. Than I'll debug this error myself.

The files on tbb.bio.uu.nl are generated in exactly the same way as when you would run ./CAT prepare --fresh. Inside those database files is a log file as well, you can see there where all the downloads come from. If you want to make the database files yourself I would suggest to just run ./CAT prepare --fresh. The version of DIAMOND is not that important as long as you run the same version for contig classification.
You can currently not run CAT summarise an ORF2LCA file, but only the contigs2classification file, that's why CAT gives you this error. Moreover you can only summarise a file that is named with the --oficial_names tag enabled. We're still evaluating what the summarise mode should do depending on how people use it. I for one think a mode that gives you back a community profile (based on read mappings) would be useful, summarise now only gives you a profile per sequence length. So we are thinking about this. :)

So if you send me you contig file I'll fix this error and report back here for future reference.
Best wishes,
Bastiaan

I close this issue now as the contigs could be classified on our systems.


no response from localhost:8000

I really need this cool project. But after I installed it, there is nothing return from localhost:8000(or <IP_ADDRESS>:8000).   Is there a log where i can check the status ?

@JayMiao looks like you don't have the permissions to run it. You can either
Change the ownership of node_modules folder to jaymiao like this:
sudo chown -R jaymiao /usr/local/lib/node_modules
OR
Run crontab-ui with sudo (Not recommended unless you want to run crontab-ui as root)


Add window fullscreen option

Add GFX.Window fullscreen support on desktop platforms

hmm I can fullscreen on mac (borderless even), how about windows?


[Bug]: The sql check module does not match the create table xxx like statement

ODC version
any
OB version
unrelated
What happened?
When the ‘create table xxx like’ statement is executed, no primary key error is displayed.

What did you expect to happen?
Restrict the create table like statement in sql check or adapt it.
How can we reproduce it (as minimally and precisely as possible)?
We can try the create table like statement to repeat.
Anything else we need to know?
No response

That bug cannot be reproduced


Add drupal-with-postgresql service template

Thanks, really nice idea to have a Drupal template, but please let me point out that the current directory structure used does not use the Drupal standards:
See https://www.drupal.org/download > https://packagist.org/packages/drupal/recommended-project > https://github.com/drupal/recommended-project/blob/6b99c737444f91c14e4b5e9034fd6844415989ba/composer.json#L48
        "drupal-scaffold": {
            "locations": {
                "web-root": "web/"
            }
        },
        "installer-paths": {
            "web/core": ["type:drupal-core"],
            "web/libraries/{$name}": ["type:drupal-library"],
            "web/modules/contrib/{$name}": ["type:drupal-module"],
            "web/profiles/contrib/{$name}": ["type:drupal-profile"],
            "web/themes/contrib/{$name}": ["type:drupal-theme"],
            "drush/Commands/contrib/{$name}": ["type:drupal-drush"],
            "web/modules/custom/{$name}": ["type:drupal-custom-module"],
            "web/profiles/custom/{$name}": ["type:drupal-custom-profile"],
            "web/themes/custom/{$name}": ["type:drupal-custom-theme"]
        },

We should please stick to Drupal standards, when preparing such a service template.

@JPustkuchen Could I ask you to elaborate on that?
Are you referring that the main folder is not /var/www/html/** but rather /opt/drupal/web/**?

@Sandros94 I'm not using coolify yet. I can take a look at it, once I have the time. But to answer the question and unblock the issue here perhaps, I can tell the following:
Drupal creates a certain folder structure when following the proposed composer installation (see link) and the coolify structure should follow that one.
Update: Just saw your EDIT link. That's strange and I think it seems to be kind of inconsistency I can't understand (on the Drupal side). Some years ago Drupal used that /html/ structure, but that's long ago.
So I posted a comment at https://github.com/docker-library/drupal/issues/3#issuecomment-2205824267
So the implementation here seems correct so far, based on the docker documentation, but maybe it's outdated upstream.

That's strange and I think it seems to be kind of inconsistency I can't understand (on the Drupal side)

Yeah, it is an inconsistency. Like for example if you want to seed a Drupal site you must first do it with another container or full build a custom image. Easily doable, but still not as intuitive as other projects.

So I posted a comment at https://github.com/docker-library/drupal/issues/3#issuecomment-2205824267

Just saw you comment popping up as I was reading that issue. To me it does seem like an upstream refactoring is required.

Hi @JPustkuchen I just followed drupal docker instructions while creating that template.
Please see in volumes section https://hub.docker.com/_/drupal

Okay, my assumption and confusion seems to have been wrong, as this comment points out: https://github.com/docker-library/drupal/issues/3#issuecomment-2206569532
I didn't see the link mapping this to the /html folder! Sorry for the confusion, so I think this is looking good, but as written above I didn't try it myself yet, so further feedback is welcome.

Thank you for the PR!


CLI: Show source context by default in ruff check

The --show-source flag can be used to show source information alongside violations during ruff check. This should be the default behavior.
--no-show-source can be used to return to the previous output. In the future, this option will be removed; see #7350.

How does this align with our long-term goal of removing --show-source in favor of different formats?

This is already the case in preview, right?

IIUC --output-fomat=full as default is stabilized in v0.5, why keeping this issue for v0.6 milestone?


[BT] Improve docs

What does this PR do?
Improves the BT docs - draft for now

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


MimeType & CSV Upload Issue

Describe the Bug
I'm generating CSV file and than uploading it to the media library using the code below:
the same code is working with pdf and image extensions but with CSV the record is not saving the MIME TYPE and the file is not uploaded.
I'm using @payloadcms/storage-s3 to upload to my S3.
to be more clear:

on uploading PDF/image/video:
1.1. the file uploaded to s3
1.2. a record with the right mimeType

on uploading csv file from the code below
2.1 the file is not uploaded to S3
2.2. a record is added to the media collection but with null in mimeType field

on uploading the same generated file from the admin panel into the media libraray, it's added and uploaded

Reproduction Steps
generate a csv file and try to add to media libarary:
      const csvFilePath = generateCSVfile()
      const csvMedia = await req.payload.create({
        collection: 'media',
        data: {
          alt: 'generated csv',
          // mimeType: 'text/csv',
        },
        filePath: csvFilePath,
        req,
      })

Which area(s) are affected? (Select all that apply)
plugin: cloud-storage, db-postgres
Environment Info
Binaries:
Node: 20.12.2
npm: N/A
Yarn: N/A
pnpm: N/A
Relevant Packages:
payload: 3.0.0-beta.96
next: 15.0.0-canary.104
@payloadcms/db-mongodb: 3.0.0-beta.96
@payloadcms/db-postgres: 3.0.0-beta.96
@payloadcms/email-nodemailer: 3.0.0-beta.96
@payloadcms/graphql: 3.0.0-beta.96
@payloadcms/live-preview: 3.0.0-beta.96
@payloadcms/live-preview-react: 3.0.0-beta.96
@payloadcms/next/utilities: 3.0.0-beta.96
@payloadcms/plugin-cloud: 3.0.0-beta.96
@payloadcms/plugin-cloud-storage: 3.0.0-beta.96
@payloadcms/plugin-form-builder: 3.0.0-beta.96
@payloadcms/plugin-nested-docs: 3.0.0-beta.96
@payloadcms/plugin-redirects: 3.0.0-beta.96
@payloadcms/plugin-search: 3.0.0-beta.96
@payloadcms/plugin-seo: 3.0.0-beta.96
@payloadcms/richtext-lexical: 3.0.0-beta.96
@payloadcms/storage-s3: 3.0.0-beta.96
@payloadcms/translations: 3.0.0-beta.96
@payloadcms/ui/shared: 3.0.0-beta.96
react: 19.0.0-rc-06d0b89e-20240801
react-dom: 19.0.0-rc-06d0b89e-20240801
Operating System:
Platform: win32
Arch: x64
Version: Windows 11 Home
Available memory (MB): 12102
Available CPU cores: 8

@ythoumaOfficial The example you gave is so minimal it will be hard to reproduce. If you would like to add more information, ie, the function code for both the working example and the non-working example that would be great! Otherwise we will likely close this issue.
A reproduction would be ideal in the situation. Something we can spin up and reach the error quickly.

Yeah, just piggy-backing off Jarrod's answer here:
I just tried to reproduce your issue using the monorepo - no luck. My CSV files upload correctly and are saved with the appropriate mimetype. I also noticed that you are generating your own CSV files. I'd double check to make sure your generation function works as expected. You may also consider updating your Payload version as it seems you're on beta.96 while the latest is beta.114 (Read the releases tab for breaking changes if you do this).

@akhrarovsaid thanks for giving this a go on the reproduction side! Very much appreciated.
@ythoumaOfficial Any updates on your end?

Hey @ythoumaOfficial - going to close this issue for now.
If @akhrarovsaid suggestions do not work for you and you can provide a reproduction repo, I'll be happy to re-open the issue!


Zombie TS-server processes

VSCode Version: 1.8 insider
OS Version: macOS

Steps to Reproduce:
(note that I am running out of sources, did not try to reproduce from a real build)

open the vscode source code folder
run a search for "copyright"
replace all with "copyright2"
wait for all the heavy work to be done
ensure a ts file opens as dirty (it should automatically)
save all dirty buffers
wait for 5 seconds for the dust to settle
reload the window

=> at this point the activity manager shows left over TS server processes that seem to have no parent

The process in question seems to be the typings installer:

Update: I think this is easy to reproduce by just reloading the window with TS files open. The typings installer seems to cause the TS server to stay alive.

I am not able to reproduce this on Windows however Windows has a more aggressive strategy to kill processes when parent processes die.
@mjbvz can you have a look on Mac please. The way how that should work is as follows: the tsserver listen on a close event on the stream it is reading / writing to and exits if this happens. See https://github.com/Microsoft/TypeScript/blob/master/src/server/server.ts#L396
So either the streams don't close or the tsserver can't process the event since it is busy.
@bpasero the ATA process is in since quite some time. Have you seen this before.
Might be a change caused by adopting 2.1.x ?

I never actually looked for this issue so I am not sure if it is new or not.

I'm also getting this. Not sure if it's exactly the same but towards the end of the day I'll have between 1-5 extra "Code Helper" processes, each taking around 100% CPU each.
I start to notice when my projects take AGES to build, so I look at Activity Monitor and see a load of processes hanging.
Was working last night and then upon opening my laptop this morning I notice there's another process still hanging which I have to manually kill. I've no idea what to look for or how to help you guys find the issue, but I've taken screenshots of activity monitor and process dumps etc in hopes that it helps.
Ping me if you want me to do anything, cause it happens all the time to me

@intellix To find out more about this issue, we need to get at the process that is causing the high load. VS Code creates multiple processes, each with different tasks. Depending on your OS you can get at the full command line of the process and paste it here. On Mac, the Activity Monitor allows to find the PID of the process and from a terminal ps aux | grep <pid> reveals its arguments.

On Windows a tool like Process Explorer helps:

Once you have the information please update this issue with it. Thanks!

ok so it seems to be caused by this: https://github.com/angular/vscode-ng-language-service
/Applications/Visual Studio Code.app/Contents/Frameworks/Code Helper.app/Contents/MacOS/Code Helper /Users/dominic/.vscode/extensions/Google.ng-template-0.0.10/node_modules/vscode-languageclient/lib/utils/electronForkStart /Users/dominic/.vscode/extensions/Google.ng-template-0.0.10/server/server.js --node-ipc

During last month's TS performance investigation, we tracked down many cases of these zombie processes to this TS file watcher bug: https://github.com/Microsoft/TypeScript/issues/14636 One possible workaround is to make sure that all of your jsconfig and tsconfig files specify an include configuration to limit the number of files that TypeScript scans. TS is looking into a fix with the next release
If you are still seeing either tsserver or typingsinstaller processes hanging around in the VSCode 1.11 even after you have properly setup your jsconfig or tsconfig, please open a new issue so that we can continue investigating this bug


Add a right click option for classic file explorer

Maybe it would be nice to use, if there would be a right-click shortcut in normal file explorer to open the folder in Files UWP.

In order to do this, we would need to modify the registry. We would like to stay away from doing that but if enough users are interested we can definitely reevaluate. I will keep this issue open for the time being in order to see if it picks up interest.

I think I would like it 👍

It would be great to see this feature! By such it would always remind you about the app

Scott Hanselman created a right click option for the windows terminal link.
I think we could change the inf-file to our needs, but i have zero experience with this.
Maybe someone can have a look.

This will launch Files UWP but not with the correct path. Feel free to play around and try to get it to do that.
; "Windows Terminal" PowerToy

[version]
signature="$CHICAGO$"

[FilesUWPHereInstall]
CopyFiles = FilesUWP.Files.Inf
AddReg    = FilesUWPHere.Reg

[DefaultInstall]
CopyFiles = FilesUWP.Files.Inf
AddReg    = FilesUWPHere.Reg

[DefaultUnInstall]
DelFiles  = FilesUWP.Files.Inf
DelReg    = FilesUWPHere.Reg

[SourceDisksNames]
55="Files UWP Here","",1

[SourceDisksFiles]
FilesUWPHere.INF=55

[DestinationDirs]
FilesUWP.Files.Inf = 17

[FilesUWP.Files.Inf]
FilesUWPHere.INF

[FilesUWPHere.Reg]
HKLM,%UDHERE%,DisplayName,,"%FilesUWPHereName%"
HKLM,%UDHERE%,UninstallString,,"rundll32.exe syssetup.dll,SetupInfObjectInstallAction DefaultUninstall 132 %17%\FilesUWPHere.INF"
HKCR,Directory\Shell\FilesUWPHere,,,"%FilesUWPHereAccel%"
HKCR,Directory\Shell\FilesUWPHere,Icon,,"%FilesUWPHereIcon%"
HKCR,Directory\Shell\FilesUWPHere\command,,0x00020000,""%%LOCALAPPDATA%%\Microsoft\WindowsApps\files.exe -d " ""%1"""
HKCR,Directory\Background\Shell\FilesUWPHere,,,"%FilesUWPHereAccel%"
HKCR,Directory\Background\Shell\FilesUWPHere,Icon,,"%FilesUWPHereIcon%"
HKCR,Directory\Background\Shell\FilesUWPHere\command,,0x00020000,""%%LOCALAPPDATA%%\Microsoft\WindowsApps\files.exe -d " ""%V"""
HKCR,Drive\Shell\FilesUWPHere,,,"%FilesUWPHereAccel%"
HKCR,Drive\Shell\FilesUWPHere,Icon,,"%FilesUWPHereIcon%"
HKCR,Drive\Shell\FilesUWPHere\command,,0x00020000,""%%LOCALAPPDATA%%\Microsoft\WindowsApps\files.exe -d " ""%1\"""

[Strings]
FilesUWPHereName="Files UWP Here PowerToy"
FilesUWPHereAccel="Open Files UWP Here"
UDHERE="Software\Microsoft\Windows\CurrentVersion\Uninstall\FilesUWPHere"
FilesUWPHereIcon=""%%LOCALAPPDATA%%\Microsoft\WindowsApps\files.exe",0"

The correct commandline argument for opening a folder is -WorkingDirectory and not -d.

@lampenlampen That helped somewhat, the app is still not navigating to the right directory but it looks like we are closer than before.

I got it working correctly.
; "Windows Terminal" PowerToy

[version]
signature="$CHICAGO$"

[FilesUWPHereInstall]
CopyFiles = FilesUWP.Files.Inf
AddReg    = FilesUWPHere.Reg

[DefaultInstall]
CopyFiles = FilesUWP.Files.Inf
AddReg    = FilesUWPHere.Reg

[DefaultUnInstall]
DelFiles  = FilesUWP.Files.Inf
DelReg    = FilesUWPHere.Reg

[SourceDisksNames]
55="Files UWP Here","",1

[SourceDisksFiles]
FilesUWPHere.INF=55

[DestinationDirs]
FilesUWP.Files.Inf = 17

[FilesUWP.Files.Inf]
FilesUWPHere.INF

[FilesUWPHere.Reg]
HKLM,%UDHERE%,DisplayName,,"%FilesUWPHereName%"
HKLM,%UDHERE%,UninstallString,,"rundll32.exe syssetup.dll,SetupInfObjectInstallAction DefaultUninstall 132 %17%\FilesUWPHere.INF"
HKCR,Directory\Shell\FilesUWPHere,,,"%FilesUWPHereAccel%"
HKCR,Directory\Shell\FilesUWPHere,Icon,,"%FilesUWPHereIcon%"
HKCR,Directory\Shell\FilesUWPHere\command,,0x00020000, %%LOCALAPPDATA%%\Microsoft\WindowsApps\files.exe -Directory ""%1""
HKCR,Directory\Background\Shell\FilesUWPHere,,,"%FilesUWPHereAccel%"
HKCR,Directory\Background\Shell\FilesUWPHere,Icon,,"%FilesUWPHereIcon%"
HKCR,Directory\Background\Shell\FilesUWPHere\command,,0x00020000, %%LOCALAPPDATA%%\Microsoft\WindowsApps\files.exe -Directory ""%V""
HKCR,Drive\Shell\FilesUWPHere,,,"%FilesUWPHereAccel%"
HKCR,Drive\Shell\FilesUWPHere,Icon,,"%FilesUWPHereIcon%"
HKCR,Drive\Shell\FilesUWPHere\command,,0x00020000, %%LOCALAPPDATA%%\Microsoft\WindowsApps\files.exe -Directory ""%1\""

[Strings]
FilesUWPHereName="Files UWP Here PowerToy"
FilesUWPHereAccel="Open Files UWP Here"
UDHERE="Software\Microsoft\Windows\CurrentVersion\Uninstall\FilesUWPHere"
FilesUWPHereIcon=""%%LOCALAPPDATA%%\Microsoft\WindowsApps\files.exe",0"

Yeah this is a bug in https://github.com/duke7553/files-uwp/blob/1f81dc3a1555d08afc5a3d18eb0deec0f6fbd729/Files/CommandLine/CommandLineParser.cs#L45
We must not split the whitespace, if it is in a sequence surrounded by quotation marks.

Good to know the issue is not in the inf file.

I also found this repo:
https://github.com/kerol2r20/Windows-terminal-context-menu
It uses a powershell script instead of a inf file.

I started adding pages to the Files UWP wiki, would you be interested in adding this to the page that explains how to use the inf file?

@lorisobi I hope the inf file helps you. I am going to be closing this issue now, but feel free to open a new one if you have any other feedback.


Document precedence of configurations.

Currently, spanner-cli has various ways of configuration.

command line options -p, -i, -d
Environment variables #55
.spanner_cli.cnf in $HOME and current directory #51

Although, precedence is not well documented.
It should be documented to prevent unexpected behaviors.

Currently, .spanner_cli.cnf is read from the either of $HOME or $PWD but reading from the both of them may be more intuitive.

What precedence order do you think is natural?
I think natural order would be command line option > environment variable > config file, but current order looks like command line option > config file > environment variable.

I agree with your opinion because config files are expected to be a overridable default.
Current order seems a bit strange.


improves getting started doc by explaining where to save yml file

When reading the getting started guidance on my first ever exposure to DropWizard, I was confused about where the YML file was supposed to go. At first I put it in the src/main/resources directory expecting the far jar execution to pick it up when I named it on the command line execution arguments. This tiny pull request seeks to clarify that the YML file can be saved anywhere and referenced later.

Thanks!

Awesome, thanks Justin!

Coverage remained the same at 84.664% when pulling a179fdcdfa74d0799ee4546a191beeead9d74122 on mrice:improve_doc_getting_started_yml into e8d7e8935483855d25e5622e0467c475d8529425 on dropwizard:master.


Default sources source

Where does the task take default sources from? I assumed it will authentificate against nuget.config, but its not. In my case when calling:
      - task: NuGetAuthenticate@1
        displayName: NuGet Authenticate

is always does:
Starting: NuGet Authenticate
==============================================================================
Task         : NuGet authenticate
Description  : Configure NuGet tools to authenticate with Azure Artifacts and other NuGet repositories. Requires NuGet >= 4.8.5385, dotnet >= 6, or MSBuild >= <IP_ADDRESS>604
Version      : 1.213.0
Author       : Microsoft Corporation
Help         : https://aka.ms/NuGetAuthenticateTask
==============================================================================
Installing the Azure Artifacts Credential Provider (.NET Framework) to 'C:\Users\AzDevOps\.nuget\plugins\netfx\CredentialProvider.Microsoft'. This credential provider is compatible with nuget.exe <IP_ADDRESS>85 or later, and MSBuild <IP_ADDRESS>604 or later.

Installing the Azure Artifacts Credential Provider (.NET Core) to 'C:\Users\AzDevOps\.nuget\plugins\netcore\CredentialProvider.Microsoft'. This credential provider is compatible with .NET SDK 6 or later.

Setting up the credential provider to use the identity 'Dabacon Products Build Service (COMPANY)' for feeds in your organization/collection starting with:
  https://pkgs.dev.azure.com/COMPANY/
  https://COMPANY.pkgs.visualstudio.com/

ignoring nuget.config content in root folder.

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

ID: 5c258bb7-f024-0301-d62e-9d7f8c854a44
Version Independent ID: 5c258bb7-f024-0301-d62e-9d7f8c854a44
Content: NuGetAuthenticate@1 - NuGet authenticate v1 task
Content Source: task-reference/nuget-authenticate-v1.md
Service: azure-devops-pipelines
Sub-service: azure-devops-pipelines-tasks
GitHub Login: @steved0x
Microsoft Alias: sdanie

@quasarea -- thank you for your continued interest.  You may have seen these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@steved0x -- please look into this issue.

Ah, it looks like it authenticates against organisation/project, not particular feeds

Thank you for your feedback. We are migrating this issue to our new documentation feedback system, and closing this issue.
As we phase out GitHub issues for this repository, we encourage you to propose changes using public PRs or submit feedback through our updated control at the top of the page. For more information on our feedback systems, see https://aka.ms/contentuserfeedback.


test: add e2e test for card payment for Adyen connector successful flow

Type of Change

[ ] Bugfix
[x] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description
Closes: hyperswitch/issues/6082

Added command getIframeElement as cy.iframe as it can only be applied to exactly one iframe at a time
edit: using cy.iframe see comment #issuecomment-2436324828
Added e2e test cases for successful card payment for the Adyen connector

How did you test it?

Checklist

[x] I ran npm run re:build
[x] I reviewed submitted code
[ ] I added unit tests for my changes where possible

@Sanskar2001 please review

I wanna highlight one thing previously I used get because cy.iframe sometimes gives error with iframe selector .adyen-checkout__iframe, I could not find out any specific reason for it but with iframe selector .adyen-checkout__iframe--threeDSIframe it is working fine. I have made the necessary changes.
@Sanskar2001 @seekshiva please look at it, thank you!
Error while using .adyen-checkout__iframe

I will look requested changes in a few days.

@Sanskar2001 can you please respond to my comments, I wanna close this PR soon

@Sanskar2001 can you please respond to my comments, I wanna close this PR soon

Hey @shubhamchasing, your task involved testing a normal card payment i.e. no 3DS flow meaning you don't have to test the redirection flow to Adyen Simulator. This task is assigned to someone else.
Could you please resolve the comments and take a pull from main, so that we can proceed?
Thanks!

@Sanskar2001 is there any issue from the backend because I am getting 403 error, I also tried new creds too but issues persit

@Sanskar2001 is there any issue from the backend because I am getting 403 error, I also tried new creds too but issue persit

@shubhamchasing Yes, there is some maintenance going on. Please try again in some time.

@shubhamchasing The  maintenance is over. You can start working on it now.

@Sanskar2001 @seekshiva hey guys please review

Requested changes are done

Hey guys @Sanskar2001 @seekshiva, I understand you guys must be busy and dealing with a lot of stuff but can you please take a moment to review this PR again? Apologies if it seems like spam but in good spirit of Open Source Contribution I like to make sure whatever contribution I make should reach its goal.


NetCDF-CF support by Ribasim_python

Ribasim_python should get NetCDF-CF support,
i.e. you provide a netcdf which is converted to arrow or you ask for arrow results in NetCDF

It's not yet fully clear to me if something is needed inside ribasim-python for that. Internally we use pandas dataframes. So if users can convert their netcdf to dataframes, with xarray for instance, then ribasim-python doesn't need to depend on xarray or xugrid, like the example in #183.


[nrf noup] ci: add linux label to jenkins agent

Required now that we have windows agents.
convert stray tabs to spaces.
Signed-off-by: Stilwell, Thomas<EMAIL_ADDRESS>

Jenkins will use the old pipeline to test this PR - so it has a 25% chance of failing. (if it gets a windows agent)

Found the following issues, please fix and resubmit:
Gitlint issues
1: UC2 Signed-off-by: must have a full name
Identity/Emails issues
a01b67eee2b2e08480218d9d01eac176c459f1eb: author email (Stilwell, Thomas<EMAIL_ADDRESS>does not follow the syntax: First Last .


Start tunnel at startup

Perhaps it makes sense for the "Run automatically at startup" checkbox to actually start the tunnel.
Or this can be split into two checkboxes:

Launch app at startup
Start tunnel at startup (depends on the first)

+1 please add auto-start tunnel on startup

+1 пожалуйста добавьте автозапуск туннеля при запуске

+1 please add auto-start tunnel on startup)))


fix typo in sequence.md

fix typo in sequence.md

Job d2l-zh/PR-766/1 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-766/


Using htmlwidgets graphics with servr::jekyll()

I'm trying to use metricsgraphics with my jekyll blog but I can't find where I'm missing something.
It seems that the div is created but the graphic never shows up.
Take a look at this: http://dfalbel.github.io/knitr-jekyll/2014/09/jekyll-with-knitr.html
However when I just knit the file in RStudio, evrything works great. See this file:
https://github.com/dfalbel/knitr-jekyll/blob/gh-pages/_source/2014-09-28-jekyll-with-knitr.html
I also tried this with others htmlwidgets like recharts and chartist.
Thanks for the help!

HTML widgets only work in these cases:

R console
RStudio viewer
R Markdown v2

What servr::jekyll() does is simply knitting .Rmd to .md. Then Jekyll renders .md to .html, and Jekyll knows nothing about HTML widgets. Your best bet is not to use Jekyll, but to use R Markdown v2, which is what we use to build several widget package websites, e.g. https://github.com/rstudio/DT/tree/gh-pages https://github.com/rstudio/dygraphs/tree/gh-pages https://github.com/rstudio/leaflet/tree/gh-pages In these cases we use Makefile and server::make().

Alright! I tought htmlwidgets could print raw html, just like the googleVis package.
Thanks Yihui!
I'll try to move my blog to this format!

@dfalbel I wanted to do the same thing with the leaflet package.  Whole site is served as a gh-pages jekyll site.  Hack that I got to work was to include the required js and css in a folder at the root of the site then make sure the links get added to the header.  Just added those links to my default.html layout.
Examples:

Folder at the root
_layouts/default.html

I am certain @yihui 's solution is the better one, but we didn't have time to re-work the site so this hack did the trick.

@jhollist Thanks for an alternative solution. Yeah, that will also work. The only problem is that if the widgets packages are updated, you will have to manually update the JS/CSS files accordingly (unless you don't care about the new versions).

@yihui I really like all the graphs and tables to be interactive in a blog. dygraphs and DT are my favorite packages for writing reports. Do you have any tutorial how to make the sites?https://github.com/rstudio/leaflet/tree/gh-pages
Thanks!
Richard

@semiexpert The websites of DT, leaflet, and dygraphs are all made from R Markdown v2. DT and leaflet websites are compiled through a Makefile. To learn more about R Markdown v2, see http://rmarkdown.rstudio.com; to learn Makefile, see http://kbroman.org/minimal_make/ servr::jekyll() is not what you want to use in these cases; servr::make() is.

Firstly, thanks so much for this repo!
I'm using it with pandoc as my Jekyll markdown renderer. It seems a shame to lose the nice features of servr::jekyll() in order to use htmlwidgets.
Would htmlwidgets work if the --include-in-header /tmp/... argument (as generated by rmarkdown) was passed to pandoc when called by Jeykll, or is there something else to it?
Based on my shaky understanding, it seems like it might be possible to wrap servr::jekyll() in a function which pre-emptively:

parses the .Rmd file for html dependencies
copies necessary  js/css files from the R packages to a Jekyll dependencies dir
produces a file pointing to them, for the --include-in-header argument passed to pandoc

The last two look like they could be achieved by passing an rmarkdown  dependencies object to rmarkdown:::html_dependencies_as_string. However, after spending a while reading through the internals in rmarkdown, I can't see how to derive a dependencies object for arbitrary .Rmd files (it looks like knitr::knit_meta is involved, but I can't piece together how).
My questions are:

Would the above (theoretically work)?
Do you have any hints on how to derive an rmardown dependencies list, for an arbitrary .Rmd file?

@brendan-R I think you are on the right track, but unfortunately I don't know the exact details of rendering dependencies, either. My gut feeling is that Jekyll may not be the right tool to render R Markdwon v2 documents. Actually it might be easier to design a new blog system, build HTML locally, and push to Github directly instead of relying on Jekyll to render Markdown to HTML.

@Yihui In reference to the websites (Leaflet and DT) you mentioned: is there an accessible tutorial for creating a website structure similar to what you did there from Rmarkdown? Are the index.html scripts created in Rmd or by hand? I.e. would I need to create it by hand as well? And lastly, is there a way to include JS scripts to my created html files using render() directly from R?

@Nicktz Please see the links in my very first reply, e.g. you can see there is an index.Rmd, so index.html was generated from it instead of being manually edited. There is a section "Creating a website" at http://rmarkdown.rstudio.com/html_document_format.html

Thanks @yihui will do!

I cobbled together a solution for my knitr-jekyll blog a while ago. It can all be handled with a small addition to build.R, and a short addition to one of the html template files. Write-up here: http://brendanrocks.com/htmlwidgets-knitr-jekyll/

Thanks @brendan-R

@brendan-R I just want to say this is very impressive! The more I think about these issues, the more I want to reinvent Jekyll in pure R, and I believe we can do much much better than Jekyll with the power of R Markdown and Pandoc. That said, I'm afraid I cannot commit any time to this before the summer this year.


hardcoded path to serialized data

Shouldnt we remove this hardcoded path to a personal scratch ?
https://github.com/MeteoSwiss-APN/spack-mch/blob/f19516a0563944ea07b98030cb1ccf94c660d05a/sysconfigs/daint/packages.yaml#L38

Yes, the problem is, the serialization is not working on Daint so I have no other way than to use the serialization data that Ben did once, I know of course that it is not the right way...

test

@BenWeber42 FYI I am closing this issue - the path to your data was remove.
@kosterried for info there is a default path for the unittest for daint, no idea if there is a plan or something that update data there - this would be something to look at in relation with the dycore issue on daint (but could be postpone for now)

This was fixed 5 months ago: https://github.com/MeteoSwiss-APN/spack-mch/commit/d0293f05818aa4dacaf6d300e7a3fa976603fc1b


[stable/hubot] Add OWNERS file

What this PR does / why we need it:
The hubot chart has no OWNERS file
Checklist

[x] DCO signed
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/lgtm

/assign @cpanato
Can you please accept my changes? I will set myself as an owner of this chart (I am the sole contributor to it so far). Thanks!

/approve

/lgtm
if you could find someone to pair with on this chart that would be amazing :)


Local Runtime Upgrade Documentation and Scripts

As a developer I want to be able to run a local command to update the running chain's wasm.
Acceptance Criteria

[x] Documentation on how to do this to the Readme
[x] Add/update scripts to trigger the upgrade make upgrade-forkless-local see scripts/onboard.js for example call methods

Help
Reach out to @enddynayn and/or @shannonwells

resources:
docs
stackexchange


Request for position on an emerging web specification

Request for position on an emerging web specification

WebKittens who can provide input: @annevk, @heycam, @jensimmons

Information about the spec

Spec Title: The Popover API
Spec URL: https://github.com/whatwg/html/pull/8221
GitHub repository: N/A
Issue Tracker (if not the repository's issue tracker): https://crbug.com (https://crbug.com/1307772)
Explainer: https://open-ui.org/components/popup.research.explainer
Incubation: OpenUI. See issues discussed/resolved.

Design reviews and vendor positions

TAG Design Review: #599 (closed waiting for anchor positioning), #680 (closed recommending change from  to popup attribute), #743 (open).
Mozilla standards-positions issue: https://github.com/mozilla/standards-positions/issues/698

Bugs tracking this feature

WebKit Bugzilla:
Radar:

Brief summary of the issue or the subject of the spec.
A very common UI pattern on the Web, for which there is no native API, is "pop up UI" or "pop-ups". Pop-ups are a general class of UI that have three common behaviors:

Pop-ups always appear on top of other page content.
Pop-ups are ephemeral. When the user "moves on" to another part of the page (e.g. by clicking elsewhere, or hitting ESC), the pop-up hides.
Pop-ups (of a particular type) are generally "one at a time" - opening one pop-up closes others.

The Pop-Up API aims to make this type of UI easy to build.
Originally posted by @mfreed7 in https://github.com/WebKit/standards-positions/issues/74

Request for position on an emerging web specification

WebKittens who can provide input: @annevk, @heycam, @jensimmons

Information about the spec

Spec Title: The Popover API

Spec URL: https://github.com/whatwg/html/pull/8221

GitHub repository: N/A

Issue Tracker (if not the repository's issue tracker): https://crbug.com (https://crbug.com/1307772)

Explainer: https://open-ui.org/components/popup.research.explainer

Incubation: OpenUI. See issues discussed/resolved.

Design reviews and vendor positions

TAG Design Review: #599 (closed waiting for anchor positioning), #680 (closed recommending change from  to popup attribute), #743 (open).

Mozilla standards-positions issue: https://github.com/mozilla/standards-positions/issues/698

Bugs tracking this feature

WebKit Bugzilla:

Radar:

Brief summary of the issue or the subject of the spec.
A very common UI pattern on the Web, for which there is no native API, is "pop up UI" or "pop-ups". Pop-ups are a general class of UI that have three common behaviors:

Pop-ups always appear on top of other page content.

Pop-ups are ephemeral. When the user "moves on" to another part of the page (e.g. by clicking elsewhere, or hitting ESC), the pop-up hides.

Pop-ups (of a particular type) are generally "one at a time" - opening one pop-up closes others.

The Pop-Up API aims to make this type of UI easy to build.
Originally posted by @mfreed7 in https://github.com/WebKit/standards-positions/issues/74


YAML code in Actions (f.e. triggered alarm) does not work properly

Alarmo Version
v1.9.7
HA Version
2023.3.1
Bug description
Within the 'triggered' action a task can be defined.
When this task is altered to YAML code and custom code is inserted, it can be tested and works great.
When the 'action' is saved, reopened, it defaults to 'entity' and cannot be tested.
Next when the alarm gets triggered, the YAML code is not executed and so no Siren is 'enabled' with high volume.
It only works when the task is set up with an entity/device and not with YAML code.
Steps to reproduce
Create an action, set it to trigger on alarm.
Change ACTION to Yaml and include the yaml to call a service with some extra input to get the Zwave Siren activated with the correct settings.
Save the action, open de the newly created action and it shows an empty entity field and does not fall back to the custom Yaml code.
Relevant log output
No response

I did some test with the following action:

If I save it and edit it again, the code is still there.
Can you share your action configuration (YAML code) so I could try the same as you do?

Hi Niels,
When I open the action it first displays:

Next when I switch it to YAML code i do see my service-call:

This service call works perfectly when being called from the 'Developer Tools' -> 'Services' tab.
It changes the volume, the duration time and next switches the alarm on.
The script that is called via this service call is:

Thanks in advance.

I'm still having trouble reproducing the issue.
I set up an action like this:

And configured the action to run when the alarm is armed.
I configured the script like this:
sequence:
  - service: notify.persistent_notification
    data:
      title: Notification title
      message: >-
        Script was triggered with entity '{{ entity }}' and duration '{{ duration }}'
alias: Send notification
mode: single

When arming the alarm, the notification is triggered with the correct parameters:

Perhaps you could try clicking the 'try it' button to see if the action is performed?
If that works, maybe the action is not triggered correctly.
For completeness, these are the conditions I configured for the action:

Could you troubleshoot if this is maybe the problem?

Good morning Niels,
When 'try it' is used, it works flawlessly, just like it can be executed from the DEV-tools->Services part of HASS.
When I replace the with the action to just turn on the siren, without YAML call to the script, it does make a lot of noice in the garage when the PIR or door-contacts are triggered in any armed mode.
This is the standard entity call with standard action:

When 'try it' is used, it works flawlessly, just like it can be executed from the DEV-tools->Services part of HASS.

The 'try it' button is intended exactly for the purpose of testing whether the configuration is OK.
Alarmo will (should) execute the action in the same way as when clicking that button.
If it works OK when clicking the test button, this suggests that the action is configured correctly.
Could you check whether the script is executed when it should?
You can see this under Settings -> Automations and Scenes -> Scripts

There should be the the time displayed when the script was last executed.
If you click the dots you can check the traces and see the parameters which were sent by alarmo:

In my testing, all looks as it should.
At this point I have no clue what is happing on your side.
Did you also try to delete the action and create it again?

I have checked it too over and over and still nothing happens at the Siren object.
I have done a test by activating the PIRs once more and I get the following feedback from the 'Automations & Scenes' script debug:

The device itself shows this:

But when I trigger is manually by 'Try it' I get the following:

Good evening Niels, have you looked at it?
I went through the code but have not found anything useful that links to a valuable error during the 'trigger'.

I don't really know how to help at this point.
In both the screenshots you shared, you can find in the variables section:
entity: switch.sirene_garage
duration: 3

So I would expect the script to run OK.
I think it's good to check if this data is also present when the alarm is triggered and the automation is executed from HA.
As said in https://github.com/nielsfaber/alarmo/issues/675#issuecomment-1475108656, in my testing this worked properly.
You could try to perform this test on your own configuration as well.


MIM Sync Service 4.6.258 and Windows Server 2019

Hi,
When i run the SHarePointSync.psm1, it throw an error like;

SharePoint Sync requires MIM PowerShell version 4.3.2064 or greater (this version is currently installed: 4.6.258.0. Please install the latest MIM hotfix

MY Server OS is: Windows Server 2019 Standard Version 1809
What i have to do?

Quickest way is to update SharePointSync.psm1 (at line 89) to accommodate the later builds of MIM:
if ($MimPowerShellModuleAssembly.VersionInfo.ProductMajorPart -eq 4 -and
(
(
$MimPowerShellModuleAssembly.VersionInfo.ProductMinorPart -eq 3 -and
$MimPowerShellModuleAssembly.VersionInfo.ProductBuildPart -ge 2064
) -or
(
$MimPowerShellModuleAssembly.VersionInfo.ProductMinorPart -eq 4 -and
$MimPowerShellModuleAssembly.VersionInfo.ProductBuildPart -ge 1237
)
)
)
}
Becomes
if ($MimPowerShellModuleAssembly.VersionInfo.ProductMajorPart -eq 4 -and
(
(
$MimPowerShellModuleAssembly.VersionInfo.ProductMinorPart -eq 3 -and
$MimPowerShellModuleAssembly.VersionInfo.ProductBuildPart -ge 2064
) -or
(
$MimPowerShellModuleAssembly.VersionInfo.ProductMinorPart -eq 4 -and
$MimPowerShellModuleAssembly.VersionInfo.ProductBuildPart -ge 1237
) -or
(
$MimPowerShellModuleAssembly.VersionInfo.ProductMinorPart -eq 6
)
)
)
)
(the build of Windows Server is irrelevant to this particular check)
For info, this build check is a really bad idea in it’s current form as it will need updated every time a new build of MIM ships.
Originally, a couple years ago – there were early builds which did not work with the SP Connector and other aspects of the toolkit, so it made sense then – but now it needs reworked alongside many other aspects of the toolkit’s approach.
Most people using it in it’s current guise will simply remove the version checking completely and rely on having an upto date install of MIM.
hth
s.
From: Hasan KÖROĞLU<EMAIL_ADDRESS>Sent: 31 July 2020 19:08
To: pnp/PnP-Tools<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [pnp/PnP-Tools] MIM Sync Service 4.6.258 and Windows Server 2019 (#255)
Hi,
When i run the SHarePointSync.psm1, it throw an error like;
SharePoint Sync requires MIM PowerShell version 4.3.2064 or greater (this version is currently installed: 4.6.258.0. Please install the latest MIM hotfix
MY Server OS is: Windows Server 2019 Standard Version 1809
What i have to do?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/pnp/PnP-Tools/issues/255, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADCOKCPEC3KM5OURH3ML5VLR6MCALANCNFSM4PQVGZDA.

Thanks Mr. Harbar.


Implemented tower data load, UI drag and drop and creation

Closes #3 and closes #4
Created tower prefabs
Added tower JSON data and serializable classes
Load tower data
Added tower UI elements
Added tower drag & drop with placeable and non-placeable tiles
Added Text Mesh Pro and essentials

No reviewers so will go ahead and merge pull request to main


Fix/issue 1384

fix #1384
Add time.Time type to Scan method.
PAY ATTENTION: there is a questionable part. I've added function level nolint: gocyclo cause with +1 branch limit is broken.
I can split the switch inside to 2 parts and move them to separate functions. And remove nolint of course. But IMHO the function will lose current readability.
WDYT?

And remove nolint of course. But IMHO the function will lose current readability. WDYT?

I am fine with nolint.

Thanks, merging.

I had to move package to proto_test. Otherwise I had circular imports.

I will move the test to redis_test.go.


Default Team Tweaks

[x] Update words
[x] Remove Default label from about the place

@toolmantim feel free to change the words and merge when you're happy!

That screen shot is slightly out of date (I fixed the "via by" typo)

@toolmantim how's this?

Whoops, accidentally merged. But it's fine. I added some commas to master.


Automatic network management

This PR adds automatic discovery / handling of IP addresses that we currently have to configure for each field / environment we run in. With these changes, manually specifying addresses / interfaces is longer required in normal circumstances.
This includes these features:

MulticastListener gains the ability to listen on all interfaces at once if the interface address argument is left empty.
IP address launch arguments are now blank to default to auto-discovery. They are kept so they can be used as command line overrides when needed.
The team client reconnect service can now optionally change the game controller IP address.
The GC multicast bridge node will now call the team client reconnect service when multicast packets are received and the team client is not connected.

🎉 Do we want any buttons in the UI to call the reconnect service or anything?

Yes. On my mental TODO list has been a widget for the UI that shows the GC connection status, referee message rate, and has a reconnect button.


[Request] Orange Pi Prime support

Orange Pi Prime is released for 5 months but still don't have RetrOrangePi.

Actually not a RetrOrangePI issue.
OrangePi Prime still lacks GPU Mali drivers, what we don't develop.


fix: l2 claim issue due to gas estimate normalisation

odd that this change is necessary, this is a departure from previous functionality and normally having a small gas limit is no trouble for block limits

also just disabled here actually https://github.com/Synthetixio/js-monorepo/commit/c1085e277952a37954049cc00f01e36fef5ab5de


Sync dev-es branch with main

Describe your changes
Sync dev-es branch with main
Related issue number or link (ex: resolves #issue-number)
Related to https://github.com/cncf/glossary/issues/2575

@gamba47 Thank you for your work, but we can sync dev-es with main by directly merging cncf:main into cncf:dev-es, without creating a feature branch, given that there is no merge conflict.
Could you modify this PR by changing the source branch to cncf:main, or open a new one using this link?

Ok @jihoon-seo I will close it and follow your steps.
Thanks!