[vcpkg] Remove start-of-string regex check for overlay triplet name matching.…

This change allows for adding a prefix to android overlay ports, which many other systems support already. The android system relies on regex matching of the triplet name to deduce the ABI, so this change just allows for that string to be anywhere in the overlay triplet string.

What does your PR fix?
Fixes the android system behaving differently than other systems with regards to overlay tripets and naming. Android doesn't allow overlay triplets to have prefixes. At our site we like to prefix our overlay triplets with "overlay-" to completely differentiate them from built-in and community triplets. This is currently not possible with the android system since it's ABI deduction regex patterns match on start of the triplet string (name).

Which triplets are supported/not supported? Have you updated the CI baseline?
This change should not impact triplet support, only what the name of overlay triplets are allowed to be.

Does your PR follow the maintainer guide?
I think so.

I have not.

I think... that's not correct. Let's see what's @talregev @christophe-calmejane @jherico said.

Hmmm, I just tested it...
>cp triplets\community\arm64-android.cmake overlay-arm64-android.cmake
>vcpkg.exe install --x-install-root=./installed --overlay-triplets=. --triplet=overlay-arm64-android libpng
Computing installation plan...
The following packages will be built and installed:
    libpng[core]:overlay-arm64-android -> 1.6.37#15
  * vcpkg-cmake[core]:x64-windows -> 2021-07-30
  * vcpkg-cmake-config[core]:x64-windows -> 2021-05-22#1
  * zlib[core]:overlay-arm64-android -> 1.2.11#12
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x64-windows...
Detecting compiler hash for triplet overlay-arm64-android...
Error: while detecting compiler information:
The log content at C:\Users\geoff.evans\Projects\gorlak-forks-vcpkg\buildtrees\detect_compiler\stdout-overlay-arm64-android.log is:
-- Configuring overlay-arm64-android
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:127 (message):
    Command failed: ninja -v
    Working Directory: C:/Users/geoff.evans/Projects/gorlak-forks-vcpkg/buildtrees/detect_compiler/overlay-arm64-android-rel/vcpkg-parallel-configure
    Error code: 1
    See logs for more information:
      C:\Users\geoff.evans\Projects\gorlak-forks-vcpkg\buildtrees\detect_compiler\config-overlay-arm64-android-out.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_configure_cmake.cmake:356 (vcpkg_execute_required_process)
  scripts/detect_compiler/portfile.cmake:18 (vcpkg_configure_cmake)
  scripts/ports.cmake:140 (include)

Error: vcpkg was unable to detect the active compiler's information. See above for the CMake failure output.

Unknown ABI for target triplet overlay-arm64-android is the error in config-overlay-arm64-android-out.log
with my patch:
>cp triplets\community\arm64-android.cmake overlay-arm64-android.cmake
>vcpkg.exe install --x-install-root=./installed --overlay-triplets=. --triplet=overlay-arm64-android sqlite3
Computing installation plan...
The following packages will be built and installed:
    sqlite3[core]:overlay-arm64-android -> 3.35.5
Detecting compiler hash for triplet overlay-arm64-android...
Could not locate cached archive: C:\Users\geoff.evans\AppData\Local\vcpkg\archives\a6\a65b07f5b417d5faea56ef02cba1b136fcb00ad248ecf548085464e9c14a261c.zip
Starting package 1/1: sqlite3:overlay-arm64-android
Building package sqlite3[core]:overlay-arm64-android...
-- [OVERLAY] Loading triplet configuration from: C:\Users\geoff.evans\Projects\gorlak-forks-vcpkg\overlay-arm64-android.cmake
-- Downloading https://sqlite.org/2021/sqlite-amalgamation-3350500.zip -> sqlite-amalgamation-3350500.zip...
-- Extracting source C:/Users/geoff.evans/Projects/gorlak-forks-vcpkg/downloads/sqlite-amalgamation-3350500.zip
-- Applying patch fix-arm-uwp.patch
-- Using source at C:/Users/geoff.evans/Projects/gorlak-forks-vcpkg/buildtrees/sqlite3/src/3350500-adf155e1e1.clean
-- Configuring overlay-arm64-android
-- Building overlay-arm64-android-dbg
-- Building overlay-arm64-android-rel
-- Performing post-build validation
-- Performing post-build validation done
Stored binary cache: C:\Users\geoff.evans\AppData\Local\vcpkg\archives\a6\a65b07f5b417d5faea56ef02cba1b136fcb00ad248ecf548085464e9c14a261c.zip
Building package sqlite3[core]:overlay-arm64-android... done
Installing package sqlite3[core]:overlay-arm64-android...
Installing package sqlite3[core]:overlay-arm64-android... done
Elapsed time for package sqlite3:overlay-arm64-android: 8.016 s

Total elapsed time: 11.4 s

The package sqlite3:overlay-arm64-android provides CMake targets:

    find_package(unofficial-sqlite3 CONFIG REQUIRED)
    target_link_libraries(main PRIVATE unofficial::sqlite3::sqlite3)

Well, I don't know if the policy changed, but when I added the regex match, I was specifically told to prevent matching something before arch-android: https://github.com/microsoft/vcpkg/pull/12634#pullrequestreview-462027232

Oh are the policies listed somewhere? I don't care to debate 😊... much less get into how things should or should not be named.
The spirit of this PR is to make the system cmakes work in a similar manner to each other (at least in terms of overlay naming). Maybe that need isn't best fulfilled by loosening the naming on the android side. Perhaps we make the naming more stringent on the non-android side? So, some enforcement in the other systems? Or generalized frontend enforcement sourcing/validating the overlay triplets?
Does anyone else see utility in being able to easily identify a triplet as being an overlay triplet (vs. vcpkg-controlled triplet), or the ability to side-by-side build and compare the artifacts of overlay triplets (vs. vcpkg-controlled triplets)? If nobody sees value here, perhaps we all just walk away :D

Please do not remove the start from names.
As different from other port, android port is bound to names. The names represent the architecture. It like to pass parameter, or set parameter inside the triplet.
So there for, like the other port can be any name they can, android should be able do some of the flexible.
People should have suffix that describe their need for their need. So remove the start symbol is actually differentiate more  from the other ports.
So I am suggesting to leave it as it, until we find a parameter to pass to the port that tell which architecture we want to compile.

The names represent the architecture. It like to pass parameter, or set parameter inside the triplet.

Triplet names are really overused for configuring the build. I don't really understand why it is necessary to parse the target triplet name when it would be fairly safe to read the triplet file (e.g. in a function scope, only exporting required variables such as VCPKG_TARGET_ARCHITECTURE to the parent scope).
I also don't see how this change might break reasonable use cases. Do we really need to worry about some custom x64-windows-arm-android not targeting android before this PR?

The toolchain should be set according to the settings in the triplet file instead of using the triplet file name.

The toolchain should be set according to the settings in the triplet file instead of using the triplet file name.

It is only done in script mode (ports.cmake). But there is no example of that in CMake project mode - in particular when using the vcpkg cmake toolchain with a user project.

it would be fairly safe to read the triplet file (e.g. in a function scope, only exporting required variables such as VCPKG_TARGET_ARCHITECTURE to the parent scope).

I have to admit that I didn't think of triplets setting environment variables. But OTOH I wonder if it isn't desirable to configure the user project with the same environment as vcpkg (in awareness of the fact that the changes to the environment variables will be lost after CMake configuration, i.e. for the actual build).
@ras0219 Any background information why reading triplets - or configuration exported from triplets - is avoided in the cmake toolchain (for user projects)?

I hope my colleagues will put forward more points.

It is intended that the name of the triplet is 100% independent from the meaning. The user should be able to make a triplet named strawberry and access all of the functionality of vcpkg, including targeting Android. Renaming a triplet should not change behavior (unless the user explicitly inflicts it upon themselves).
That unfortunately is not the case for Android -- this issue was missed during the initial PR for Android and we don't block orthogonal improvements on fixing this original sin. However, we are strongly opposed to expanding this dependency.
Given the above general concerns, this PR looks fine to me. There is no concern that "x64-windows-arm64-android" will be misinterpreted because the only way this file (android.cmake) is pulled in is if their triplet already says set(VCPKG_CMAKE_SYSTEM_NAME Android).
However, the larger long-term fix is likely to be some combination of:

A general-purpose mechanism for the triplet to add arbitrary flags to CMake builds
A better framework for handling CPU variations that are otherwise treated similarly (neon, v7 vs v6, etc). Today's VCPKG_TARGET_ARCHITECTURE is intended to be entirely orthogonal and drives evaluation of supports expressions.

@dg0yt Any objections?

@dg0yt Any objections?

No, in the context presented by @ras0219-msft: It is okay for now, but vcpkg needs to improve this in the future.


you do need a copyright notice for this.

i just noticed the githubcli.py program is based off DaTa XD

virtualpet.py is a copy of virtual pet:
import subprocess
import os
import sys
import random
def main():
status = "happy"
hunger = 0
tired = 0
print("Welcome to the virtual pet game!")
print("Keep your pet happy by feeding it...")
print("Play with it...")
print("Make it rest...")
print("And buy it toys")
print("Enjoy this simple command line game by Shazhain Khan!")
input("Press Enter to continue...")
for i in range(20):
    print(" ")

while True:
    print("Looks like your dog is", status)
    print("You can FEED, PLAY, REST, BUY TOYS, EXIT")
    if tired > 4:
        print("Your dog is tired.")
    if hunger > 4:
        print("Your dog is hungry.")

    task = input("What would you like to do? ").lower()

    if task == "play":
        hunger += 2
        tired += 2
        print("You go to the park to play with your dog!")
        status = "excited"
        print("Your dog is", status, "to be at the park")
        number = random.randint(1, 100)
        try:
            numberpicked = int(input("Pick a number from 1-100: "))
            if numberpicked < number:
                print("Oh, not quite! The right number was:", number)
                print("You throw the ball about 20 feet!")
                print("Nice throw!")
                print(" ")
                status = "satisfied"
            elif numberpicked > number:
                print("Not quite! Still good! The right number was:", number)
                print("Nice!

        # Path to the kernel script in the KERNEL folder
        kernel_script = os.path.join(os.getcwd(), 'KERNEL', 'kernel.py')

        # Check if the kernel script exists before trying to run it
        if os.path.isfile(kernel_script):
            try:
                # Run the kernel script as a subprocess
                subprocess.run([sys.executable, kernel_script], check=True)
            except subprocess.CalledProcessError as e:
                print(f"Error executing the kernel script: {e}")
            except Exception as e:
                print(f"An unexpected error occurred: {e}")
        else:
            print("yay sucsess!")

        break

    else:
        print("That is not a valid action!")

if name == "main":
main() LMAO

except the Shahzain Khan thing is diffrent and maybe smth else?

compare:
import subprocess
import os
import sys
import random
def main():
status = "happy"
hunger = 0
tired = 0
print("Welcome to the virtual pet game!")
print("Keep your pet happy by feeding it...")
print("Play with it...")
print("Make it rest...")
print("And buy it toys")
print("Enjoy this simple command line game by Alter Net codes!")
input("Press Enter to continue...")
for i in range(20):
    print(" ")

while True:
    print("Looks like your dog is", status)
    print("You can FEED, PLAY, REST, BUY TOYS, EXIT")
    if tired > 4:
        print("Your dog is tired.")
    if hunger > 4:
        print("Your dog is hungry.")

    task = input("What would you like to do? ").lower()

    if task == "play":
        hunger += 2
        tired += 2
        print("You go to the park to play with your dog!")
        status = "excited"
        print("Your dog is", status, "to be at the park")
        number = random.randint(1, 100)
        try:
            numberpicked = int(input("Pick a number from 1-100: "))
            if numberpicked < number:
                print("Oh, not quite! The right number was:", number)
                print("You throw the ball about 20 feet!")
                print("Nice throw!")
                print(" ")
                status = "satisfied"
            elif numberpicked > number:
                print("Not quite! Still good! The right number was:", number)
                print("Nice!

        # Path to the kernel script in the KERNEL folder
        kernel_script = os.path.join(os.getcwd(), 'KERNEL', 'kernel.py')

        # Check if the kernel script exists before trying to run it
        if os.path.isfile(kernel_script):
            try:
                # Run the kernel script as a subprocess
                subprocess.run([sys.executable, kernel_script], check=True)
            except subprocess.CalledProcessError as e:
                print(f"Error executing the kernel script: {e}")
            except Exception as e:
                print(f"An unexpected error occurred: {e}")
        else:
            print("yay sucsess!")

        break

    else:
        print("That is not a valid action!")

if name == "main":
main()

add me to alternetcodes

i'll do it, im also making awebsite in next.js for alternetcodes,

we already have one but ill see if its better.

also the license needs to retain the: Copyright (c) 2024 ArtificialXDev copyright notice as well as the Copyright (c) 2024 Alter Net codes notice in the license. im adding you rn.

its bad ik but ill be making a new one, i just made that one blindfolded lol

ok. btw, make sure its based on SkyOS 2.7 when it comes out, fully bugfixed, with devtools, and new features.

btw, you only need to join the SkyOS team, also dont mess with the original SkyOS source code without permission and testing, ok.


i18n(de): update guides/deploy/edgio.mdx

Description (required)

update guides/deploy/edgio.mdx
add . to all Steps because I think it looks better...

Related issues & labels (optional)

#8167
#9240
Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

de/guides/deploy/edgio.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


@nx/remix | incompatibility with future flag v2_dev:true

Hi,
I just start a new remix project and get this error when the future flag v2_dev: true
error  command not found: remix-serve
┃ `remix dev` did not receive `--command` nor `-c`, defaulting to `remix-serve build/index.js`.
┃ You probably meant to use `-c` for your app server command.
┃ For example: `remix dev -c 'node ./server.js'`
┗
>  NX   Remix app exited with code 1
  Pass --verbose to see the stacktrace.

>  NX   Running target serve for project buscaproducto-api failed

It is strange because by default is activated in the latest version of Remix and the command to start the app has not changed still being remix dev.

I think this can be closed as it is now officially supported and runs on v2.4.x right? @Mat-moran


[NCP] Added support for external NCP vendor hook

This change (requested by @pvanhorn) allows developers to add a hook source file to the NCP build that contains hook for implementing vendor-specific commands. This is done via a new configure script argument, --with-ncp-vendor-hook-source=<C++FILE>, with the argument being a single C++ source file containing the hook.
This isn't a terribly elegant solution to this sort of problem, so long-term we should consider major refactoring of the NCP codebase to enable the flexibility of allowing the vendor to implement a library that can be linked in, instead of a file which must be compiled in.

you will need to change ncp_base.cpp, where VendorCommandHandler() is called as it's #if will also need to be changed from OPENTHREAD_ENABLE_SPINEL_VENDOR_SUPPORT to OPENTHREAD_ENABLE_NCP_VENDOR_HOOK.

you will need to change ncp_base.cpp, where VendorCommandHandler() is called as it's #if will also need to be changed from OPENTHREAD_ENABLE_SPINEL_VENDOR_SUPPORT to OPENTHREAD_ENABLE_NCP_VENDOR_HOOK.

Weird, I must have forgotten to save the file after making that change. Will have a fix in a few moments.


pygame.error: Unable to make GL context current

Thank you so much for your work.But when I run the code using the instruction python continuous_driver.py --exp-name=ppo --train=False, there is an error as follows.
Traceback (most recent call last):
File "/home/**/Autonomous-Driving-in-Carla-using-Deep-Reinforcement-Learning-main/simulation/sensors.py", line 63, in    self.sensor.listen(lambda image: CameraSensorEnv._get_third_person_camera(weak_self, image))  File "/home//Autonomous-Driving-in-Carla-using-Deep-Reinforcement-Learning-main/simulation/sensors.py", line 87, in _get_third_person_camera pygame.display.flip()
pygame.error: Unable to make GL context current
I would be very grateful if you can give me some tips!

pip uninstall pygame
pip install pygame==1.9.6


Add support for Nightwatch v3

Also fixes #5.

@beatfactor @gravityvi Can you please review this as well?

Tests are only failing because of some config issues with BrowserStack, there are working fine locally.


How can I declare derived/calculated fields - ones which only have getters and not setters ?

I have some pojos which set/get 4 datetime fields - and we also have related
fields *and* getters (not setters) which covert those date times into local
time zone date times. Can this be supported ?

e.g.

private Date startSate; // with getter and setter - setter also coverts and
sets startLocalDate
private LocalDateTime startLocalDate; // with just setter

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 7 Oct 2014 at 4:18

#90


Set default Task timeout value

It appears that not all Tasks have a timeout value set, so some Tasks can still hang (e.g. Tasks that are created by HTTPAccessLogAnalysisJob).   This can cause the workers to hang in some occasions and then potentially drop pubsub messages (as in #833).   We should set a default timeout and also make sure we have sane defaults for the other Tasks.

Fixed by #883


Nested templates and Parameters Without Default Must Exist In CreateUIDefinition

Hello, we are in the process of certifying a managed application and we are getting the validation error in the issue title that is forcing us to add default values to all nested templates. Since they are nested templates, we understand that they are not directly connected to the CreateUIDefinition file and they should not be taken into account by that check.
I've built a minimal example to demonstrate the issue. I hope that can help better understand it, but happy to provide more details if needed.
createUiDefinition.json:
{
  "$schema": "https://schema.management.azure.com/schemas/0.1.2-preview/CreateUIDefinition.MultiVm.json#",
  "handler": "Microsoft.Azure.CreateUIDef",
  "version": "0.1.2-preview",
  "parameters": {
    "basics": [
      {}
    ],
    "outputs": {
      "mainParameter": "value-from-ui"
    }
  }
}

main.bicep:
param mainParameter string = 'default-value'

module module 'module.bicep' = {
  name: 'Module'
  params: {
    moduleParameter: mainParameter
  }
}

module.bicep:
param moduleParameter string

When built, the Bicep files produce the following ARM template:
{
  "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
  "contentVersion": "<IP_ADDRESS>",
  "metadata": {
    "_generator": {
      "name": "bicep",
      "version": "<IP_ADDRESS>621",
      "templateHash": "4551264275492999207"
    }
  },
  "parameters": {
    "mainParameter": {
      "type": "string",
      "defaultValue": "default-value"
    }
  },
  "resources": [
    {
      "type": "Microsoft.Resources/deployments",
      "apiVersion": "2020-10-01",
      "name": "Module",
      "properties": {
        "expressionEvaluationOptions": {
          "scope": "inner"
        },
        "mode": "Incremental",
        "parameters": {
          "moduleParameter": {
            "value": "[parameters('mainParameter')]"
          }
        },
        "template": {
          "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
          "contentVersion": "<IP_ADDRESS>",
          "metadata": {
            "_generator": {
              "name": "bicep",
              "version": "<IP_ADDRESS>621",
              "templateHash": "15490559216858461434"
            }
          },
          "parameters": {
            "moduleParameter": {
              "type": "string"
            }
          },
          "resources": []
        }
      }
    }
  ]
}

The Parameters-Without-Default-Must-Exist-In-CreateUIDefinition check fails as follows.
$ Test-AzTemplate . -Tests @('Parameters-Without-Default-Must-Exist-In-CreateUIDefinition')"

Validating validation\createUiDefinition.json
  Parameters Without Default Must Exist In CreateUIDefinition
    [-] Parameters Without Default Must Exist In CreateUIDefinition (20 ms)
        moduleParameter does not have a default value, and is not defined in CreateUIDefinition.outputs

If we add a default value to the module parameter, the validation passes. However, that forces us to set default values for hundreds of parameters across multiple modules, which doesn't look right at all.
param moduleParameter string = ''

$ Test-AzTemplate . -Tests @('Parameters-Without-Default-Must-Exist-In-CreateUIDefinition')"

Validating validation\createUiDefinition.json
  Parameters Without Default Must Exist In CreateUIDefinition
    [+] Parameters Without Default Must Exist In CreateUIDefinition (16 ms)

@adrianmo , just want to offer that something does seem strange here. You should be able to do what you want and the marketplace should go and accept the ARM template that you showed. This is actually what you have to do for a location parameter, which I have gone and submitted successfully myself.
Are you seeing this issue when you upload to the partner center, or just locally? it may be that the partner center does not check this rule.
You can find the exact arm-ttk setup for ARM-TTK that the partner center uses here, https://aka.ms/arm-ttk-marketplace
Try running this over your code and see if the error still pops?
I have an example of an Azure DevOps pipeline where I run this version of ARM-TTK on my template before I submit to partner center, just in case that helps you too. https://github.com/microsoft/azure-gaming-devops/blob/main/marketplace/stage_arm_ttk.yml

@dciborow we've recreated this issue in #680 as well, and have seen the problem with nested templates since we started (1yr+). and yea it occurs in the partner center, as well as locally.

@bmoore-msft This looks related to #680, please assign it to me.


backupclientURL gives a 404

It looks like the backupclientURL has an extra version number appended to the url.  The PR https://github.com/puppet-community/puppet-stash/pull/89 fixes it.

Thank you.


Keep campaign configuration in Couch synchronized with MongoDB

Now that https://github.com/dmwm/WMCore/issues/9228 has been taken care and we're ready to do a one time injection into central CouchDB; we should plan on how to keep the campaign configuration consistent between the two different database backends.
I see two possibilities:
a) check with Ops which script they use to create/write campaign records into MongoDB; then we can adapt the same script to write to both backends; this would also allow them to delete campaigns from central couch;
b) extend one of the ReqMgr2 CP-threads to fetch campaign records from MongoDB, and update campaigns in central couch in a polling cycle manner (every X hours).
Both options have pros and cons, but I think option a) would be the best.
Regardless of which option we choose, we need to use this script to make the conversion from Unified to WMCore campaign schema:
https://github.com/dmwm/WMCore/blob/master/bin/adhoc-scripts/parseUnifiedCampaigns.py
@vkuznet @sharad1126 and @thongonary any thoughts here?

If possible I also in favor of option A. Now we have a schema and should adapt existing scripts to use the schema and push docs into appropriate destinations.

@thongonary and @sharad1126 how would you prefer to handle this case? What I had in mind was to extend your script:
https://github.com/CMSCompOps/WmAgentScripts/blob/master/campaignsConfiguration.py
to first inject campaigns into MongoDB, then inject the same campaign (with a different format) into central couchdb. This has two problems though:
*) we need to update the WMAgent version deployed on the unified nodes (or on the node creating/inserting campaigns)
*) if the couchdb campaign injection fails, we need to make sure it gets retried until it succeeds.
Another option would be to keep two different scripts:
*) the one which is already in use, without any modifications
*) then a second script, which reads all the campaigns from mongodb (or a list of campaign names), parse and inject them into couchdb.
Please let me know your thoughts.

Hi @amaltaro , I think this is the same about which we had a small discussion last week. I'll suggest to go ahead and use our script and extend it as from operations side, we won't like to run two scripts separately. It just increases the manual work for the operations. @thongonary what do you think?

This Ops/WmAgentScripts PR fixes this issue: https://github.com/CMSCompOps/WmAgentScripts/pull/466
Which has been merged 5min ago and it's now live. Every campaign Unified has is also available in central CouchDB (production CMSWEB), even though we're not using those yet.


Markers specified via style are not rendered

This happens because of the ID reassignment: elements have their IDs changed (e.g. <marker id="Arrow1Lend"> becomes <marker id="Arrow1Lend-0">) but the style declarations are not updated to match (e.g. <path style="marker-start:url(#Arrow1Lend)"> will  remain unchanged).
You can see below a simple file that exhibits this problem:
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
<!-- Created with Inkscape (http://www.inkscape.org/) -->

<svg
   xmlns:dc="http://purl.org/dc/elements/1.1/"
   xmlns:cc="http://creativecommons.org/ns#"
   xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
   xmlns:svg="http://www.w3.org/2000/svg"
   xmlns="http://www.w3.org/2000/svg"
   version="1.1"
   width="128"
   height="128"
   id="svg8338">
  <defs
     id="defs8340">
    <marker
       refX="0"
       refY="0"
       orient="auto"
       id="Arrow1Lend"
       style="overflow:visible">
      <path
         d="M 0,0 5,-5 -12.5,0 5,5 0,0 z"
         transform="matrix(-0.8,0,0,-0.8,-10,0)"
         id="path3795"
         style="fill-rule:evenodd;stroke:#000000;stroke-width:1pt;marker-start:none" />
    </marker>
    <marker
       refX="0"
       refY="0"
       orient="auto"
       id="DotL"
       style="overflow:visible">
      <path
         d="m -2.5,-1 c 0,2.76 -2.24,5 -5,5 -2.76,0 -5,-2.24 -5,-5 0,-2.76 2.24,-5 5,-5 2.76,0 5,2.24 5,5 z"
         transform="matrix(0.8,0,0,0.8,5.92,0.8)"
         id="path3854"
         style="fill-rule:evenodd;stroke:#000000;stroke-width:1pt;marker-start:none;marker-end:none" />
    </marker>
  </defs>
  <metadata
     id="metadata8343">
    <rdf:RDF>
      <cc:Work
         rdf:about="">
        <dc:format>image/svg+xml</dc:format>
        <dc:type
           rdf:resource="http://purl.org/dc/dcmitype/StillImage" />
        <dc:title></dc:title>
      </cc:Work>
    </rdf:RDF>
  </metadata>
  <g
     transform="translate(0,-924.3622)"
     id="layer1">
    <path
       d="m 14.285713,989.50504 99.999997,0"
       id="path8346"
       style="fill:none;stroke:#000000;stroke-width:1px;stroke-linecap:butt;stroke-linejoin:miter;stroke-opacity:1;marker-start:url(#DotL);marker-end:url(#Arrow1Lend)" />
  </g>
</svg>

The work-around for me was to use the non-minified version and manually disable the ID remapping.
Would it be an option to make ID remapping configurable by the user?

Sounds a lot like #29 , just that in my case I can't use a class (instead of an id) to style my SVG.

Hey Mihai! You are correct, SVGInjector's ID renumeration doesn't currently parse references contained in style attributes... it's looking for the SVG specific element styling properties. In your case the marker related properties: marker-start and marker-end.
The quick fix for your example SVG is to simply use those properties, like so:
<path
  d="m 14.285713,989.50504 99.999997,0"
  id="path8346"
  style="fill:none;stroke:#000000;stroke-width:1px;stroke-linecap:butt;stroke-linejoin:miter;stroke-opacity:1;"
  marker-start="url(#DotL)"
  marker-end="url(#Arrow1Lend)"
/>

Little background on the reason for ID renumeration feature... When injecting SVG inline into the DOM with SVGs that use IDs for fragment identifiers you start having problems related to duplicate IDs (if you use the same SVG twice on the same page, for example), which isn't allowed by the HTML spec:

The id attribute assigns a unique identifier to an element... This name must be unique in a document.
http://www.w3.org/TR/html401/struct/global.html#id-and-class

The HTML spec doesn't define how to handle duplicate IDs, but in practice browsers just use the first instance in the DOM. Browsers also don't properly implement the visibility rules from the SVG spec, taking a rendering performance shortcut which can break SVG display when using multiple SVGs where the first one on the page might be hidden.
So, I'm hesitant to make it possible to disable that feature... but maybe it can be an option, like you said. Maybe even on a per-SVG basis?
So with all that background, any thoughts on how we can improve SVGInjector for this use case?
Thanks!

Well, ideally, the style declarations would be parsed as well :grin: but I'm aware that's quite a project on its own.
For cases where you're only injecting a single SVG, or non-similar SVGs, I still believe it would be a good idea to be able to disable ID renumeration. In my case, this was the only option, because the authoring tool generated the markers in the style attribute, not in a separate marker element.


Cannot create new project

I go through the process of creating a new project, and when I click save, I see no error but the updated project list is empty.

Looks like name is required and silently fails.


Formatting broken in 2.0.1 - please keep using 2.0.0 until this is fixed in 2.0.2

Seems there is a regression in 2.0.1 that leads to formatting not working on some systems.
I could reproduce it on Windows, where after upgrading the extension you get this error message in the Output for the Prisma extension:
[Info  - 21:18:48] Installed version of Prisma binary `prisma-fmt`: 7d77a7cd570c9dedb4cdf31a1cbec615cf2e1eda
[Info  - 21:18:48] Extension name prisma with version 2.0.1
[Info  - 21:18:48] Prisma CLI version: 2.0.1
prisma-fmt error'd during linting.

Error: spawn UNKNOWN
    at ChildProcess.spawn (internal/child_process.js:394:11)
    at Object.spawn (child_process.js:553:9)
    at Object.exec [as default] (c:\Users\Jan\.vscode\extensions\prisma.prisma-2.0.1\server\out\exec.js:5:33)
    at Object.<anonymous> (c:\Users\Jan\.vscode\extensions\prisma.prisma-2.0.1\server\out\lint.js:19:48)
    at Generator.next (<anonymous>)
    at c:\Users\Jan\.vscode\extensions\prisma.prisma-2.0.1\server\out\lint.js:8:71
    at new Promise (<anonymous>)
    at __awaiter (c:\Users\Jan\.vscode\extensions\prisma.prisma-2.0.1\server\out\lint.js:4:12)
    at Object.lint [as default] (c:\Users\Jan\.vscode\extensions\prisma.prisma-2.0.1\server\out\lint.js:17:12)
    at c:\Users\Jan\.vscode\extensions\prisma.prisma-2.0.1\server\out\server.js:85:41 {
  errno: 'UNKNOWN',
  code: 'UNKNOWN',
  syscall: 'spawn'
}

Please keep using 2.0.0 until this problem is fixed in 2.0.2.

If you have the same problem, can you please answer and leave information what operating system you are on?

Copy the file prisma-fmt.de2bc1cbdb5561ad73d2f08463fa2eec48993f56.exe
Rename it to prisma-fmt.7d77a7cd570c9dedb4cdf31a1cbec615cf2e1eda.exe
Copy it to C:\Users\Jan\.vscode\extensions\prisma.prisma-2.0.1\server\out and overwrite the existing broken file.

Afterwards formatting should work again.

I get a similar error running Linux with Fedora 32:
[Error - 9:32:59 PM] Cannot install prisma-fmt: Not Found
events.js:180
      throw er; // Unhandled 'error' event
      ^

Error: incorrect header check
    at Zlib.zlibOnError [as onerror] (zlib.js:170:17)
Emitted 'error' event at:
    at errorOrDestroy (internal/streams/destroy.js:107:12)
    at Gunzip.onerror (_stream_readable.js:744:7)
    at Gunzip.emit (events.js:203:13)
    at Zlib.zlibOnError [as onerror] (zlib.js:173:8) {
  errno: -3,
  code: 'Z_DATA_ERROR'
}

Some background information: The download URL for the formatter is different than with all previous releases, as this is our first patch release. Turns out we missed that in our testing. Ugh.
A solution might be as simple as https://github.com/prisma/vscode/pull/238 and then we can release a new version. I will keep you posted. Until then, just keep using 2.0.0 - there are not interesting changes in the formatter really.

I same yet

Sorry for commenting on closed issue but is it fixed? I am still facing it in 2.10.2.


html: EscapeString could be made faster using a strings.Replacer

Consider the following benchmark:

package escaping

import (
        "html"
        "strings"
        "testing"
)

const noHTMLString = `Can you hear the people sing, singing the song of
angry men. It is the music of a people who will not be slaves again.`
const withHTMLString = `We're going to demand one million
dollars or we're going to threaten Earth with a "Laser".`

var escaper = strings.NewReplacer(
        `&`, "&",
        `'`, "'",
        `<`, "<",
        `>`, ">",
        `"`, """,
)

func BenchmarkHtml_EscapeString_nochanges(b *testing.B) {
        for i := 0; i < b.N; i++ {
                html.EscapeString(noHTMLString)
        }
}

func BenchmarkStrings_Replacer_nochanges(b *testing.B) {
        for i := 0; i < b.N; i++ {
                escaper.Replace(noHTMLString)
        }
}

func BenchmarkHtml_EscapeString_changes(b *testing.B) {
        for i := 0; i < b.N; i++ {
                html.EscapeString(withHTMLString)
        }
}

func BenchmarkStrings_Replacer_changes(b *testing.B) {
        for i := 0; i < b.N; i++ {
                escaper.Replace(withHTMLString)
        }
}

The performance is as follows:

BenchmarkHtml_EscapeString_nochanges      500000              3912 ns/op
BenchmarkStrings_Replacer_nochanges      5000000               368 ns/op
BenchmarkHtml_EscapeString_changes        500000              4597 ns/op
BenchmarkStrings_Replacer_changes        1000000              1360 ns/op

I've written https://golang.org/cl/141930043/ to change html.EscapeString so
that it uses a replacer internally, but it needs to wait until Go 1.5 development opens.

@kbloom do you plan to migrate your CL to Gerrit (the new codereview system)? If not, would you like one of us to do so for you? Thanks!

I'm not sure the OP is still following this thread. I prepare a similar CL (crediting him, of course).

Thanks. I had lost track of your ping because it was in my personal email
rather than my corp email. (Plus the friction of getting set up on a new
review environment.) I can't even figure out how to LGTM your CL, but it
looks good to me.
On Fri, May 8, 2015 at 1:01 PM, GopherBot<EMAIL_ADDRESS>wrote:

CL https://golang.org/cl/9808 mentions this issue.
—
Reply to this email directly or view it on GitHub
https://github.com/golang/go/issues/8697#issuecomment-100298861.


Sign SkypeRoomProvisioningScript.ps1

Please sign the SkypeRoomProvisioningScript.ps1 file.

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

ID: 05282aa8-ae14-117b-5885-7d851ee66b89
Version Independent ID: 85abe13c-d34c-f42c-e273-a69be8cd991e
Content: Deploy Microsoft Teams Rooms with Office 365 - Microsoft Teams
Content Source: Teams/rooms/with-office-365.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@Scott-som  Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@Scott-som thank you for reaching out and helping improve MS Docs. Please clarify what your request is?

@Scott-som  We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

On the page https://docs.microsoft.com/en-us/microsoftteams/rooms/with-office-365 is a link to a powershell script SkypeRoomProvisioningScript.ps1 which is not signed can causes some run issues.


specifying multiple --engine-opt fails

Hi Folks,
I am trying to specify both cluster-store and cluster-advertise as follows:
--engine-opt "cluster-store=consul://X.X.X.X:8500" –-engine-opt "cluster-advertise=eth0:2376"
However, this results in
Invalid command line. Found extra arguments [cluster-advertise=eth0:2376 vmfoo]
According to documentation, this should be allowed (also see an example in documentation):
--engine-opt [--engine-opt option --engine-opt option]
Any ideas?
Thanks,
Himanshu

Are you passing them after the machine name maybe?
Something like this:
$ docker-machine create -d amazonec2 foo --engine-opt dns=<IP_ADDRESS>

Is invalid syntax today.  foo (the machine name) has to come last.

Yeah, this works for me with the latest RC, so let me know if you continue to encounter issues (include the create command please).

No, I am putting machine name last.
Can you give me your version and OS version pls?
Thanks,
-Himanshu
On Wed, Feb 3, 2016 at 10:19 AM -0800, "Nathan LeClaire"<EMAIL_ADDRESS>wrote:
Yeah, this works for me with the latest RC, so let me know if you continue to encounter issues (include the create command please).
—
Reply to this email directly or view it on GitHub.

I'm on OSX El Cap, Machine 0.6.0-rc4.
How about you?

Same here.
Here is the failing command:
/usr/local/bin/docker-machine -D create --driver amazonec2 --amazonec2-access-key XXX --amazonec2-secret-key XXXX --amazonec2-region us-west-1 --amazonec2-vpc-id vpc-838804e6 --amazonec2-subnet-id subnet-7041f729 --amazonec2-zone b --amazonec2-instance-type t2.micro --amazonec2-ami ami-3f50245f --engine-install-url http://X.X.X.X:8081/pkgs/install.sh --engine-opt cluster-store=consul://<IP_ADDRESS>:8500 –-engine-opt cluster-advertise=eth0:2376 vhost-rhim839
A similar command succeeds with generic driver.

I notice that the last flag you posted (--engine-opt cluster-advertise=eth0:2376) has an "em dash" (– - sometimes automatically inserted by Mac editors) instead of a normal hyphen (-) -- maybe that's it?  If you replace that argument with a real dash does it work?

OMG, that indeed was the case.
Thanks for pointing it out.

:+1: I hate that Mac behavior of replacing with the fancy dashes.  Glad to hear it worked out.

OMG!!! You guys saved me a lot of trouble. I was pulling my hair out


{ unspecified parameter } in Firestore rules in how this extension works section

I just installed your extension and stumbled across this bug
It tells me to add new firestore rules
rules_version = '2';
service cloud.firestore {
  match /databases/{database}/documents {
    match /users/{app_user_id}/purchaser-info/{uid} {
      allow read: if request.auth.uid == uid;
    }

    match /{ unspecified parameter }/{id} {
      allow read: if request.auth.uid == resource.app_user_id
    }
  }
}

Notice the { unspecified parameter }.
I did not enable the events. Maybe this rule should only appear when events are enabled?

Yes :)
Once you will configure the "RevenueCat Webhook Events Firestore collection" this undefined will disappear.


Show font size and colors when possible

This will only apply to some default juce stuff?

Don't bend over backwards (yet) to make this editable — mainly want to show what's in use.
Ideally colors are displayed in RGBA values by default?

I've added Color inspector and small widget for showing component image snapshot. I'm not super happy about it's position though. Where do you think it should be? Above or under properties? Also let me know if you have completely different vision for this "widget"?
Available on develop


Have several agents collaborate in a multi-agent hierarchy 🤖🤝🤖 notebook improvements

What does this PR do?

Fixes #193
Who can review?
@merveenoyan and @stevhliu

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Push delay on iOS 8

I have a strange delay on iOS 8 when using SWRevealViewControllerSeguePushController segue. It takes about 2 seconds before transition animation takes place. Also first call of "reveal toggle" takes about 3-4 seconds. Here's my storyboard:

On iOS 7 and 9 it works fine without any delay.

Hi Moonko. Did you fixed this issue? I'm having the same problem with iOS 9 when pushing a second view controller.

@bordadagua Yes, it was my own fault. I just forgot to add my custom fonts in resources and the system was searching for them first time.


Editor loses reference to BaseBehaviour derived object

I have the following classes:
public class Item : FullInspector.BaseBehavior
{
    public float size;
    public float weight;
    ...
}

public class PlayerAction : BaseObject
{
    public virtual void DoAction(Player p) {}
...
}

public class PlayerInventoryAction : PlayerAction
{
   public enum InventoryAction
    {
        Add,
        Remove
    }

    public InventoryAction action;
    public Item item;
    public int count;

    public override void DoAction(Player p)
    {}
    ...
}

public class Quest : BaseObject
{
  public System.Collections.Generic.List<PlayerAction> OnCompleteActions;
}

My issue is that when I populate the OnCompleteActions list in the editor, the second member of the list fails to store its Item reference.  I have a variety of different Item objects and it does not seem to matter which I use.  For some reason the first member of the list seems to serialize correcty.  Can you help with this?

Update to this, I tried deleting the list and starting again, and this time the Item references are saving correctly, but the count member of the second list entry always saves as zero.  It all seems a bit flaky...

Further update, I checked the serialized scene file, and the count value is correctly stored, however in the editor it always appears as zero.

Are you using a build from Access? I recall fixing some serializing issues with BaseObject references.


Make Definition Clearer

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

@lawrencejon OK, I think both @thecodingaviator and I agree to make the suggested changes which I made above, unless have an issue with my suggestion.  We are open to discussing it further.


Connect Database to The GUI

The Database needs to be connected to the Gui so all data can be visualized in the GUI.
The query.py file will house all the database queries that will be done to pull data.
The Projects need to be returned as a list and the Tasks need to be linked to a Dictionary with the key being the Project and value being a list of all projects.

Waiting on #12 to be completed to do this started by making a temp_query that has data to send to GUI to test.


feat: allow parallel worker builds when worker plugins is a function

Description
Fixes #13367
Background
Currently when there's more than 2 nested workers the Rollup build gets into a deadlock and cannot compile. This was a regression of when worker builds were made sequential (#11218). The reason for sequential building was to solve issues with plugin state pollution in parallel builds
This PR is an attempt at implementing @patak-dev's suggestion in https://github.com/vitejs/vite/pull/13394#issuecomment-157055346. By allowing users to provide plugins as a function we can ensure that each worker build gets a separate instance of plugins and hopefully minimise any state pollution problems.
Suggested Solution
Recommend defining worker.plugins in config as a function. When a function is used we run worker builds in parallel because each build can instantiate a fresh instance of the plugins to avoid pollution.
import myWorkerPlugin from 'my-worker-plugin';

export default {
   worker: {
      plugins: () => [myWorkerPlugin()]
   }
}

This new worker config API is backwards compatible and existing configs that have custom worker plugins as arrays will work the same as before.
Let me know if I've misunderstood the solution or whether there's more to be done. Given the complexity I understand if you want to close this PR until your own core team can work on a robust solution.
As always, thanks for all your hard work on this amazing open source tool! 🙇
Additional context
I am unable to get the integration test suite running locally. I followed the contributing guidelines, but no luck. No discernible errors thrown will have to dive into the vitest setup scripts to debug. This is on a Mac M1 Pro, Ventura with Node 18 and 20. Had to debug tests via CI.

What is the purpose of this pull request?

[x] Bug fix
[x] New Feature
[x] Documentation update

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

Thanks so much for your work on this @jamsinclair. This addresses a huge issue for my app which currently builds in ~1/50 retries in CI.

I have the SB repro in the linked issue working on my local changing the key for workerConfigSemaphore (it seems the config change and there is a pending promise):
async function bundleWorkerEntry(config, id, query) {
    const processing = workerConfigSemaphore.get({ key: 'vite:worker' });
    if (processing) {
        await processing;
        return bundleWorkerEntry(config, id, query);
    }
    const promise = serialBundleWorkerEntry(config, id, query);
    workerConfigSemaphore.set({ key: 'vite:worker' }, promise);
    promise.then(() => workerConfigSemaphore.delete({ key: 'vite:worker' }));
    return promise;
}

Closing this PR as we merged #14685, thanks for your work here @jamsinclair! We added you as a co-author in the PR.


[New game]: Addition of Tic toc Toe game

🎮 Game Request
Adding Tic Toc Toe game with two players playing functionality
Point down the features

This game has a two player feature
If you are single player then other one will be handled by the computer
sound effects on winning or loosing

Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @krutikajichkar !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @krutikajichkar 👋!
@kunjgit here,
Hope you are doing extremely well 😇
It's confirmed 🥳, GameZone is taking part in hacktoberfest and ready to have another 💥
Ready to have some amazing contributions once again also thank you for you support in GSSoC23 you have took us on top 5 ❤🔥
if you still haven't joined us join us on Discord
See you soon 😊

Hello @krutikajichkar, Time's Uppp!⏰
Sorry for closing your issue!
But it's more than a week since we haven't received anything from your side 😢 .
Come up with new ideas, create a new issue and make sure you finish it within a week! 🔥
All the best! 🚀
Happy Hacking! 💗

Hey @krutikajichkar ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Configure sensitive config values for redaction

Overview
This defines a configuration file which specifies sections and fields
for config values that are redacted from logs. Specifically, all
values from the "admins" section and the value of "password" in the
"replicator" section are redacted.

Testing recommendations
This must be tested with its companion PR below.
In a remsh, set an admin password: config:set("admins", "admin1", "secretpassword")., and then reload: config:reload(). and you should see the following redactions in the log:
[notice] 2021-02-20T06:16:30.672944Z<EMAIL_ADDRESS><0.196.0> -------- config: [admins] admin1 set to '****' for reason nil
[notice] 2021-02-20T06:17:13.720003Z<EMAIL_ADDRESS><0.196.0> -------- Reload detected config change admins.admin1 = '****'

Related Issues or Pull Requests
https://github.com/apache/couchdb-config/pull/35

Checklist

[x] Code is written and works correctly
[ ] Changes are covered by tests
[ ] Any new configurable parameters are documented in rel/overlay/etc/default.ini
[ ] A PR for documentation changes has been made in https://github.com/apache/couchdb-documentation

The testing instructions are a bit incorrect. The reload event can be tested using the following steps:

set an admin password: config:set("admins", "admin1", "secretpassword").
manually change one of the digits corresponding to hash of pass for admin1 in dev/lib/node1/etc/local.ini
config:reload().


Rename media to atomic

cc: https://github.com/facebook/draft-js/issues/124
The idea is that media isn't generic enough (and can lead to confusion) for what this DraftBlockType serves as.

I renamed to AtomicBlockUtils, insertAtomicBlock, etc. and left some parts as Media for clarity, such as the media example and parts of the docs.

LGTM. @tasti, are you able to import this?

@facebook-github-bot import

@hellendag, nvm I don't have the special powers

This was merged via https://github.com/facebook/draft-js/commit/0096d9873e5f7e98145c0a835a52a42f54003306.


Fix regression in FileBody constructor

Committed to SVN trunk and 4.5.x branch.
Oleg

Thanks for quick approval!
May I ask when approximately this will be released?
Andrey Pokhilko
On 05/27/2015 05:23 PM, Oleg Kalnichevski wrote:

Committed to SVN trunk and 4.5.x branch.
Oleg
—
Reply to this email directly or view it on GitHub
https://github.com/apache/httpclient/pull/30#issuecomment-105929690.

Likely within two weeks


added dp sum

Added a new dp question and clear solution to that question in memoization and tabulation methods.

Added Algorithm to find Articulation Points in CPP


HDDS-1228. Chunk Scanner Checkpoints

What changes were proposed in this pull request?
Save timestamp of last successful data scan for each container (in the .container file).  After a datanode restart, resume data scanning with the container that was least recently scanned.
Newly closed containers have no timestamp and are thus scanned first during the next iteration.  This will be changed in HDDS-1369, which proposes to scan newly closed containers immediately.
https://issues.apache.org/jira/browse/HDDS-1228
How was this patch tested?
Created and closed containers.  Restarted datanode while scanning was in progress.  Verified that after the restart, scanner resumed from the container where it was interrupted.
datanode_1  | STARTUP_MSG: Starting HddsDatanodeService
datanode_1  | 2019-10-08 19:37:07 DEBUG ContainerDataScanner:148 - Scanning container 1, last scanned never
datanode_1  | 2019-10-08 19:37:07 DEBUG ContainerDataScanner:155 - Completed scan of container 1 at 2019-10-08T19:37:07.570Z
datanode_1  | 2019-10-08 19:37:07 INFO  ContainerDataScanner:122 - Completed an iteration of container data scrubber in 0 minutes. Number of iterations (since the data-node restart) : 1, Number of containers scanned in this iteration : 1, Number of unhealthy containers found in this iteration : 0
datanode_1  | 2019-10-08 19:37:17 DEBUG ContainerDataScanner:148 - Scanning container 2, last scanned never
datanode_1  | 2019-10-08 19:38:57 DEBUG ContainerDataScanner:155 - Completed scan of container 2 at 2019-10-08T19:38:57.402Z
datanode_1  | 2019-10-08 19:38:57 DEBUG ContainerDataScanner:148 - Scanning container 1, last scanned at 2019-10-08T19:37:07.570Z
datanode_1  | 2019-10-08 19:38:57 DEBUG ContainerDataScanner:155 - Completed scan of container 1 at 2019-10-08T19:38:57.443Z
datanode_1  | 2019-10-08 19:38:57 INFO  ContainerDataScanner:122 - Completed an iteration of container data scrubber in 1 minutes. Number of iterations (since the data-node restart) : 2, Number of containers scanned in this iteration : 2, Number of unhealthy containers found in this iteration : 0
datanode_1  | 2019-10-08 19:38:57 DEBUG ContainerDataScanner:148 - Scanning container 3, last scanned never
datanode_1  | 2019-10-08 19:39:02 DEBUG ContainerDataScanner:155 - Completed scan of container 3 at 2019-10-08T19:39:02.402Z
datanode_1  | 2019-10-08 19:39:02 DEBUG ContainerDataScanner:148 - Scanning container 4, last scanned never
datanode_1  | 2019-10-08 19:39:02 DEBUG ContainerDataScanner:155 - Completed scan of container 4 at 2019-10-08T19:39:02.430Z
datanode_1  | 2019-10-08 19:39:02 DEBUG ContainerDataScanner:148 - Scanning container 5, last scanned never
datanode_1  | 2019-10-08 19:39:11 ERROR HddsDatanodeService:75 - RECEIVED SIGNAL 15: SIGTERM
datanode_1  | STARTUP_MSG: Starting HddsDatanodeService
datanode_1  | 2019-10-08 19:39:22 DEBUG ContainerDataScanner:148 - Scanning container 5, last scanned never
datanode_1  | 2019-10-08 19:40:18 DEBUG ContainerDataScanner:155 - Completed scan of container 5 at 2019-10-08T19:40:18.268Z
datanode_1  | 2019-10-08 19:40:18 DEBUG ContainerDataScanner:148 - Scanning container 6, last scanned never
datanode_1  | 2019-10-08 19:40:31 DEBUG ContainerDataScanner:155 - Completed scan of container 6 at 2019-10-08T19:40:31.735Z
datanode_1  | 2019-10-08 19:40:31 DEBUG ContainerDataScanner:148 - Scanning container 2, last scanned at 2019-10-08T19:38:57.402Z
datanode_1  | 2019-10-08 19:42:12 DEBUG ContainerDataScanner:155 - Completed scan of container 2 at 2019-10-08T19:42:12.128Z
datanode_1  | 2019-10-08 19:42:12 DEBUG ContainerDataScanner:148 - Scanning container 1, last scanned at 2019-10-08T19:38:57.443Z
datanode_1  | 2019-10-08 19:42:12 DEBUG ContainerDataScanner:155 - Completed scan of container 1 at 2019-10-08T19:42:12.140Z
datanode_1  | 2019-10-08 19:42:12 DEBUG ContainerDataScanner:148 - Scanning container 3, last scanned at 2019-10-08T19:39:02.402Z
datanode_1  | 2019-10-08 19:42:16 DEBUG ContainerDataScanner:155 - Completed scan of container 3 at 2019-10-08T19:42:16.629Z
datanode_1  | 2019-10-08 19:42:16 DEBUG ContainerDataScanner:148 - Scanning container 4, last scanned at 2019-10-08T19:39:02.430Z
datanode_1  | 2019-10-08 19:42:16 DEBUG ContainerDataScanner:155 - Completed scan of container 4 at 2019-10-08T19:42:16.669Z
datanode_1  | 2019-10-08 19:42:16 INFO  ContainerDataScanner:122 - Completed an iteration of container data scrubber in 2 minutes. Number of iterations (since the data-node restart) : 1, Number of containers scanned in this iteration : 6, Number of unhealthy containers found in this iteration : 0

Also tested upgrade from Ozone 0.4.0.  (Downgrade does not work, see HDDS-2268.)

@arp7 thanks for the comments on the original PR.  I've updated this one to use Optional, and while there, use Instant instead of Long to make it more type-safe.  Unfortunately snakeyaml doesn't work well with either of those, so I kept a Long member in the background for serialization.


Upgrade to Kotlin 1.3

Kotlin 1.3 stabilizes coroutines 🎉

Created a branch to test upgrading: kotlin-1.3 - diff
So far, there's a bunch of warnings for using types marked with @ExperimentalCoroutinesApi.
[INFO] --- kotlin-maven-plugin:1.3.0-rc-198:test-compile (test-compile) @ grpc-kotlin-test ---
[WARNING] Source root doesn't exist: /home/circleci/repo/grpc-kotlin-test/src/test/java
[WARNING] /home/circleci/repo/grpc-kotlin-test/src/test/kotlin/io/rouz/greeter/ClosingStatusExceptionTest.kt: (37, 30) This declaration is experimental and its usage should be marked with<EMAIL_ADDRESS>or '@UseExperimental(kotlinx.coroutines.ExperimentalCoroutinesApi::class)'
[WARNING] /home/circleci/repo/grpc-kotlin-test/src/test/kotlin/io/rouz/greeter/ClosingStatusExceptionTest.kt: (38, 13) This declaration is experimental and its usage should be marked with<EMAIL_ADDRESS>or '@UseExperimental(kotlinx.coroutines.ExperimentalCoroutinesApi::class)'
[WARNING] /home/circleci/repo/grpc-kotlin-test/src/test/kotlin/io/rouz/greeter/ClosingStatusExceptionTest.kt: (45, 30) This declaration is experimental and its usage should be marked with<EMAIL_ADDRESS>or '@UseExperimental(kotlinx.coroutines.ExperimentalCoroutinesApi::class)'
[WARNING] /home/circleci/repo/grpc-kotlin-test/src/test/kotlin/io/rouz/greeter/ClosingStatusExceptionTest.kt: (46, 13) This declaration is experimental and its usage should be marked with<EMAIL_ADDRESS>or '@UseExperimental(kotlinx.coroutines.ExperimentalCoroutinesApi::class)'
[WARNING] /home/circleci/repo/grpc-kotlin-test/src/test/kotlin/io/rouz/greeter/DelayedClosingStatusExceptionTest.kt: (43, 30) This declaration is experimental and its usage should be marked with<EMAIL_ADDRESS>or '@UseExperimental(kotlinx.coroutines.ExperimentalCoroutinesApi::class)'
[WARNING] /home/circleci/repo/grpc-kotlin-test/src/test/kotlin/io/rouz/greeter/DelayedClosingStatusExceptionTest.kt: (46, 17) This declaration is experimental and its usage should be marked with<EMAIL_ADDRESS>or '@UseExperimental(kotlinx.coroutines.ExperimentalCoroutinesApi::class)'
[WARNING] /home/circleci/repo/grpc-kotlin-test/src/test/kotlin/io/rouz/greeter/DelayedClosingStatusExceptionTest.kt: (57, 30) This declaration is experimental and its usage should be marked with<EMAIL_ADDRESS>or '@UseExperimental(kotlinx.coroutines.ExperimentalCoroutinesApi::class)'
[WARNING] /home/circleci/repo/grpc-kotlin-test/src/test/kotlin/io/rouz/greeter/DelayedClosingStatusExceptionTest.kt: (60, 17) This declaration is experimental and its usage should be marked with<EMAIL_ADDRESS>or '@UseExperimental(kotlinx.coroutines.ExperimentalCoroutinesApi::class)'
[WARNING] /home/circleci/repo/grpc-kotlin-test/src/test/kotlin/io/rouz/greeter/DelayedThrowingStatusExceptionTest.kt: (43, 30) This declaration is experimental and its usage should be marked with<EMAIL_ADDRESS>or '@UseExperimental(kotlinx.coroutines.ExperimentalCoroutinesApi::class)'
[WARNING] /home/circleci/repo/grpc-kotlin-test/src/test/kotlin/io/rouz/greeter/DelayedThrowingStatusExceptionTest.kt: (57, 30) This declaration is experimental and its usage should be marked with<EMAIL_ADDRESS>or '@UseExperimental(kotlinx.coroutines.ExperimentalCoroutinesApi::class)'
[WARNING] /home/circleci/repo/grpc-kotlin-test/src/test/kotlin/io/rouz/greeter/ExceptionPropagationTest.kt: (167, 30) This declaration is experimental and its usage should be marked with<EMAIL_ADDRESS>or '@UseExperimental(kotlinx.coroutines.ExperimentalCoroutinesApi::class)'
[WARNING] /home/circleci/repo/grpc-kotlin-test/src/test/kotlin/io/rouz/greeter/ExceptionPropagationTest.kt: (175, 30) This declaration is experimental and its usage should be marked with<EMAIL_ADDRESS>or '@UseExperimental(kotlinx.coroutines.ExperimentalCoroutinesApi::class)'
[WARNING] /home/circleci/repo/grpc-kotlin-test/src/test/kotlin/io/rouz/greeter/NoStatusExceptionPropagationTest.kt: (45, 30) This declaration is experimental and its usage should be marked with<EMAIL_ADDRESS>or '@UseExperimental(kotlinx.coroutines.ExperimentalCoroutinesApi::class)'
[WARNING] /home/circleci/repo/grpc-kotlin-test/src/test/kotlin/io/rouz/greeter/NoStatusExceptionPropagationTest.kt: (53, 30) This declaration is experimental and its usage should be marked with<EMAIL_ADDRESS>or '@UseExperimental(kotlinx.coroutines.ExperimentalCoroutinesApi::class)'
[WARNING] /home/circleci/repo/grpc-kotlin-test/src/test/kotlin/io/rouz/greeter/ThrowingStatusExceptionTest.kt: (37, 30) This declaration is experimental and its usage should be marked with<EMAIL_ADDRESS>or '@UseExperimental(kotlinx.coroutines.ExperimentalCoroutinesApi::class)'
[WARNING] /home/circleci/repo/grpc-kotlin-test/src/test/kotlin/io/rouz/greeter/ThrowingStatusExceptionTest.kt: (45, 30) This declaration is experimental and its usage should be marked with<EMAIL_ADDRESS>or '@UseExperimental(kotlinx.coroutines.ExperimentalCoroutinesApi::class)'

@wfhartford Have you had a chance to look at this? Seems like ProducerScope is marked with @ExperimentalCoroutinesApi even in the final 1.0.0 release. This means that the recent change to implement server streaming rpc handlers as extension functions on ProducerScope causes a bunch or warnings unless annotated with @UseExperimental(ExperimentalCoroutinesApi::class).
I'm leaning towards reverting back to having those handlers return a ReceiveChannel<TReply> as earlier. wdyt?

I hadn't noticed that it is still marked as experimental. I agree about reverting to a ReceiveChannel<TReply> return type. I hadn't really though very much about using the ProducerScope, and already found a case in my code where that is inconvenient.

@rouzwawi @wfhartford I submitted a PR keeping it as-is. Not sure I have enough context to switch the mustache template return type from ProducerScope to ReceiveChannel. The diff for that commit looks extensive.
Thoughts?

@drmaas Thanks, merged. I'll get a PR ready for the ProducerScope changes and will do a new release today.


Carousel: Add option to vary display interval on a per-slide basis

Add the option to set the interval per slide leveraging the data-interval attribute, see also http://stackoverflow.com/questions/27047620/custom-carousel-intervals/27049608

@fat Ping.

Tests passed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED
Commit: 4db2c120dd3cd76a39f8eb565543250ffb4ad9a3
Build details: https://travis-ci.org/twbs-savage/bootstrap/builds/52811662
(Please note that this is a fully automated comment.)

@hnrch02 thanks. You are right there is no reason to save this.duration. Although i used this value in the unit test before. In fact the unit should test the value set for setInterval (replace with setTimeout in my last update). I don't know how to test that value? Now the test seems to test more reading of data attributes instead of durations set.
You also ask if i could use this.$active, i don't think so (or at least not in the current code). The cycle function will be the first called when initialising the plugin without any parameters. In this situation this.$active has not been set yet.
You can also initialise the plugin with .carousel(number) or .carousel('next'). .carousel(number) set  this.$active indeed, but  .carousel(number) set a local value for $active. I also wondered if both calls make sense when the carousal has not been initialized before? But when not setting $active in the slide function let directly fail the first test "should not fire slid when slide is prevented'".
I think this.$active should set on initialisation always (and not per function). I will try to fix this in my next update.
I also found the usage of setInterval as far as in understand, setInterval fires immediately and then after the set interval. The first immediately trigger seems to be blocked by the running transition (i don't know what happens when transitions are not supported, but probably not the right thing), setTimeout seems to better fit?
And then it seems cycle is already called before the transition has ended. The active class move in the transition, so cycle runs on the previous active item instead of the current. I realized now that in my current code the interval become (unexpected) set interval + transition duration.

Tests passed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED
Commit: 0664f602b5232e67b90fca7a97e336387b7d362b
Build details: https://travis-ci.org/twbs-savage/bootstrap/builds/52824540
(Please note that this is a fully automated comment.)

I don't understand why you also have duration stuff in here. The transition duration also involves the CSS and thus can't be easily customized, unlike the interval, which is what this PR is supposed to be about.

@cvrebert do you refer to the .emulateTransitionEnd(Math.min(Carousel.TRANSITION_DURATION,($active.data('duration') || this.interval))). I think you are right about that. But i should possible check the situation that duration < Carousel.TRANSITION_DURATION.
Now (after the PR) it runs: duration first slide + Carousel.TRANSITION_DURATION, and so on.

Yes, that's what I'm referring to, in part.
Also, this.$element.find('.item.active').data('duration') ought to be this.$element.find('.item.active').data('interval') to avoid confusion.

I'd like to punt on this for v3. We're not adding any other features and this falls under that banner. Plus, I don't see a super strong benefit to varying the duration right now. Appreciate the effort though!


Fix CLI runner

Current Behavior
Executing ts-oas from CLI throws the following error:
C:\Users\xxxx\github\someproject\node_modules\ts-oas\bin\ts-oas:3
app.run();
    ^

TypeError: app.run is not a function
    at Object.<anonymous> (C:\Users\xxxx\github\someproject\node_modules\ts-oas\bin\ts-oas:3:5)
    at Module._compile (node:internal/modules/cjs/loader:1254:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1308:10)
    at Module.load (node:internal/modules/cjs/loader:1117:32)
    at Module._load (node:internal/modules/cjs/loader:958:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12)
    at node:internal/main/run_main_module:23:47

The error occurs whether running with module installed local to a project, globally, or using npx.
The tool does complete the work before throwing this error.
This error was reproduced on MacOS and Windows 10.
Expected Behavior
No error is thrown.
Solution
We can remove the line of code that calls "run()" on the cli.js module required. That module doesn't export anything, it just executes everything when required.

Thanks, I'm going to publish it right now. Sorry for the late reply.


Change RCTImageLoader's Cache System to default NSURLRequest's cache system

intro
Before this PR, RCTImageLodaer's Cache was too big(200MB on disk) and It doesn't work with HTTP Cache-Control header. So to provide dynamic image, the users must have to add random value on url( ex. adding current date) to avoid cache.
So I change that cache system to default NSURLRequest's cache system, which is well-working with HTTP specs. As the discussion on this issue #7571 , making custom cache policy processor is not ready yet and useless, over-tech things, I think.
Even we have no plan about image cache system(or would change plan later), before having a nice plan, I think we should let user use image module with common HTTP Specs.
So What did I do?
So I remove custom NSURLCache, and make logic like below,

try fetch image,

on response, get Date on response's header and make cacheKey with Date.

(why? because if NSURLRequest's response was cached, the response's Date header dosen't change.)

find decoded image on _decodedImageCache with cacheKey

if there isn't, decode that image and store to _decodedImageCache with cacheKey.

In addition, I didn't stored local image asset on _decodedImageCache because [UIImage imageNamed:] work with its own cache.
Test Plan
just make one simple web-server which provide static images with cache-control header.
set max-old or anything else and test that image was really caching during max-old time.
I've just tested with simple nodejs express server and checked it was perfectly working.

In addition, I didn't stored local image asset on _decodedImageCache because [UIImage imageNamed:] work with its own cache.

This is already being looked at in #8102, could you not do it here?

@javache

(In addition, I didn't stored local image asset on _decodedImageCache because [UIImage imageNamed:] work with its own cache.)
This is already being looked at in #8102, could you not do it here?
Actually I didn't touch anything about local asset images. And my point is, "because we use [UIImage imageNamed:](and it use ios's cache system), we don't have to cache decoded local asset image's UIImage on _decodedImageCache. Is it something wrong?

edited wrong formatted codes and useless nested block.

Where in this diff do we configure NSURLRequest to actually do caching?

Where in this diff do we configure NSURLRequest to actually do caching?

I found we didn't set any cache-policy about NSURLRequest, and then the default value is NSURLRequestUseProtocolCachePolicy, which is working with this flow,

you can find this on this doc

Can you nest this in the else block and have a shared return statement after the block?

Done.

Use NO instead of false

Done.

Thanks! I'll check this out internally and verify that caching works correctly in our use-cases.

@facebook-github-bot import

Thanks for importing. If you are an FB employee go to Phabricator to review.

@mmmulani @javache I have a question. In general case, how update the commit which is already on pull-request? I generally add commit with --amend commend to make simple the pull-request, but sometime it is confused because there are no history about small changes.

sometime it make me confused because there are no history about small changes

Then you should split your PR into multiple PRs. One PR = one commit = one idea.

Could I ask you how it is going? I just wonder how facebook's engineer work or test or anything.

@skatpgusskat hey we're taking a look at it still, hoping to land it by the end of this week


Redesigning the About us section(Aligning the icons and more)

assign this to me and mention the level

@anuza08  how you going to redesign the page , show some designs

@HimanshuNarware please assign this issue to me. I will make the About Us section look visually appealing and add animation and hovering effects that align to the color template of the project. I have attached the sample below.

hello @HimanshuNarware. I kindly request you to assign this issue to me under GGSoC'24. I have built a blogging website whose link is given below, kindly see it and give me permission to contribute towards your project.
link - https://github.com/pahuja-sahil/Online-Blogging-System.
Thank you.

@anuza08 how you going to redesign the page , show some designs

this is my design , if there are any changes to be made pls let me know

assign me this issue under gssoc24

@HimanshuNarware Please assign this issue to me under GSSoC24. I will do this work very effeciently

Please assign me this issue

@anuza08 Is issue still open.
Please assign this to me I will improve it.

please assign this to me.


Style-attributed set to outdated value after transition

Description:
When an element is transitioned-in (e.g. fade-in) any style-*-directive or style-attribute which was updated after the transition had started, will be reset to its initial value it had right before the transition began.
Versions affected:
0.8.14
Reproduction:
https://jsfiddle.net/lejared/v7m4jxwb/
Click on "Toggle Visible" to start fade-in animation
Click on "Change Color" before animation ends
The Color will change instantly but will be reset to its initial value after the transition has completed.
Note that the colors text-value in the box is not affected.

The issue is that transitions take a snapshot of the style property when they start and then restore it when they finish. Anything that happens in the middle is not accounted for. We can address this by tracking individual property values on transitions and only restoring those that have a value that we expect, but there's a considerable amount of cruft in the transition code that makes it a bit painful.

Naive approach without knowing anything about the internals: Wouldn't it be possible to just reapply the current model on the style-attribute or re-evaluate any style-* expression after the transition has completed?


Bugfix for NoneType Error in SSLyze parser

fixes #9848

I reproduced the error in the first commit and will it fix now.


Hand Pose

Is your feature request related to a problem? Please describe.
Is it possible to add hand pose also?
Describe the solution you'd like
Create hand pose editor, maybe a dropdown menu to select between body pose only and hand pose also

Will it be implemented like this guy?
https://github.com/zackhxn/openpose-hand-editor

Update:
https://github.com/huchenlei/sd-webui-openpose-editor has full integration with ControlNet. (Supports hand/face)


Could we have a page for the "XY Problem"?

For those that don't know what the XY problem is outlined here.

Meh, I don't think having an X/Y problem is worthy of a downvote. In fact, it's the exact opposite.
Someone needed to perform a task. They didn't know the best way to do it, so they attempted with X. X didn't work out well for them, due to one issue or another, so they ask the question. That's when people respond "Don't do X, do Y."
It's worth a downvote when they need to perform a task and they don't make any attempt at all...


Syntax error near unexpected token `(' when trying to pass reports from travis ci

Can someone please tell me if this issue is codecov-bash related or if I should take a look someplace else?
As described here : https://stackoverflow.com/questions/52433448/eval-error-when-publishing-to-codecov-from-travis
.travis.yml
language: java
jdk: oraclejdk10
after_success:
- bash <(curl -s https://codecov.io/bash)

pom.xml
<plugin>
    <groupId>org.jacoco</groupId>
    <artifactId>jacoco-maven-plugin</artifactId>
    <version>0.8.2</version>
    <executions>
        <execution>
        <goals>
            <goal>prepare-agent</goal>
        </goals>
        </execution>
        <execution>
            <id>report</id>
            <phase>test</phase>
            <goals>
                <goal>report</goal>
            </goals>
        </execution>
    </executions>
</plugin>

travis.ci output
[INFO] Results:
[INFO]
[WARNING] Tests run: 174, Failures: 0, Errors: 0, Skipped: 7
[INFO]
[INFO]
[INFO] --- jacoco-maven-plugin:0.8.2:report (report) @ PROJECT ---
[INFO] Loading execution data file /home/travis/build/x/y/target/jacoco.exec
[INFO] Analyzed bundle 'PROJECT' with 14 classes
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 6.121 s
[INFO] Finished at: 2018-09-20T20:25:14Z
[INFO] Final Memory: 18M/512M
[INFO] ------------------------------------------------------------------------
The command "mvn test -B" exited with 0.
after_success
0.00s$ bash < (curl -s https://codecov.io/bash)
/home/travis/.travis/job_stages: eval: line 98: syntax error near unexpected token `('
/home/travis/.travis/job_stages: eval: line 98: `bash < (curl -s https://codecov.io/bash) '
Done. Your build exited with 0.

Apparently now it works without any changes made.


Sort by "Date Added" shows a blank page

URL of the Page
No response
What happened?
Currently, sorting by the date will show a blank page:

Steps to reproduce

Go to the search page
Select "Sort by: Date added"

What browsers are you seeing the problem on?
Chrome
Code of Conduct

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

This problem has been fixed by changing Algolia's index name.
But the problem is that the index name is "dateAdded", but the index record entry is called createdAt


Elasticsearch Models

Need to create the elasticsearch models in Django. Refer to the haystack documentation to determine the best practices for implementation.

e66af28ea2ddd8b2dc986627e7297b75082912b5

#9 Had to dig into dockers environment variables to figure out the container communication link. Little tricky.


"Savings Plan, Reservied Instance, and Out-Of-Cluster" set to false incorrectly

model/pricingSourceStatus endpoint returns "false" status for Savings Plan, Reservied Instance, and Out-Of-Cluster:
{
  "code": 200,
  "status": "success",
  "data": {
    "Savings Plan, Reservied Instance, and Out-Of-Cluster": {
      "name": "Savings Plan, Reservied Instance, and Out-Of-Cluster",
      "available": false,
      "error": "No reserved instances detected"
    },
    "Spot Data Feed": {
      "name": "Spot Data Feed",
      "available": false,
      "error": "MissingRegion: could not find region configuration"
    }
  }
}

even if "Savings Plans" are found via Athena Query, but no "Reservation" found at the moment.
From logs:
I1112 14:47:22.218376       1 awsprovider.go:1799] Fetching RI data...
I1112 14:47:22.218590       1 awsprovider.go:1828] No reserved instance data found
...
I1112 14:47:26.941761       1 awsprovider.go:1727] Fetching SavingsPlan data...
I1112 14:47:26.942131       1 awsprovider.go:1750] Found 866 savings plan applied instances

Thanks for this report @Valiev -- I'll investigate this now.


Fix update operations

We should make update operations nested, meaning, you can provide any nested object to it as an argument and it will try to perform these updates recursively. This was not possible before #77 and #78 because with permission resource paths being deep by default and of an arbitrary length to control the depth makes controlling the depth of any operation (especially UPDATE operations) possible.


📝 Docs - Make this project follow Awesome Documentation

Hi @bradystroud ,
I noticed this repo is not following the SSW awesome documentation rule
https://www.ssw.com.au/rules/awesome-documentation/

Please update it to follow the rule

Acceptance Criteria
All 8 documents exist and are populated

Hey @pierssinclairssw,

Please update it to follow the rule
Done - You can see the documentation in the repository Wiki


No implementation for std::collections::hash_set::Intersection

A std::collections::hash_set::Intersection is produced from two std::collections::HashSets and is composed of elements that exist both in collection a and collection b.
I would like to be able to iterate over this object so I am seeking to implement rayon::iter::IntoParallelIterator, but I am not able to find a good starting point. I am more than happy to contribute code to Rayon, how can I contribute to adding this support?

We can't directly parallelize Intersection, as we have no access to its internals.  But you can approximate this in a couple ways:
// Perform the same comparison that `Intersection` does itself.
// (note: `a.par_iter()` internally has to serially collect all references from `a` first)
a.par_iter().filter(|x| b.contains(x)) ...

// Collect the `Intersection`, then use that in parallel
a.intersection(&b).collect::<Vec<_>>().into_par_iter() ...

Or, you can use rayon-hash's HashSet, which has a par_intersection that doesn't have to do any collection up front!
We could consider adding extensions like par_intersection to the standard HashSet, but that would still have to do a collect() in one of the ways mentioned above.  There's no universal answer which way would be better -- depending on the cost of lookups, how many intersections are expected, and what you plan to do with them -- which makes me hesitate to add this.

Generally I'd point you to plumbing/README.md to learn how to implement, and suggest looking at some of the types we already have done.  But I don't think Intersection is feasible.

@cuviper Thank you so much for your response! Thanks for pointing me in the right direction. You can close this if you'd like.

OK - hopefully we'll see some other contribution from you! :)


Memory usage on large query

Hi,
I am using Streaming Query Rows to fetch a table with 330 million rows, and to insert each row into mongodb. The source (mysql) and destination (mongodb) are on another machine over the networ. The RAM usage of node is getting too big for my PC to handle. It gets at 4GB with only 2700579 inserts. I tried to run the script with --always-compact and --expose-gc, whitout success. I think the driver is not freeing some variables.
Here's the code: https://pastebin.com/4JJKRwGx (online 1 month)

nodejs: v6.10.3-1.fc25

Same thing with v8.1.2

Okay... it's the mongo driver that uses the RAM. Whitout inserting into mongo, nodejs uses ~1% RAM.


Enforcing Parent Class Contracts

Hey guys,
Not sure if I'm missing something here, but I'd like to be able to enforce this following behaviour in some way:
abstract class Vehicle extends Model
{
    use ReturnsChildModels;

    public abstract function start();
}

class Car extends Vehicle
{
    use HasParentModel;

    // HAS to implement parent start() method, won't run otherwise
}

If the parent model is abstract, when I call something like Vehicle::all(), an exception is thrown saying Vehicle can't be instantiated (which I guess, is kind of expected).
I could make the parent implement an interface, but if I can't make the parent abstract the parent will have to implement the method, which will not enforce the contract for its children.

I see. That's an interesting point. Right now, I can't think of any solution that wouldn't be horribly janky. If start() will definitely get called on every child type, then you could let the system fail, or add a start() method to Vehicle and throw an exception in it.


SSL Support For Realm Object Server?

Not sure if this is just a documentation issue, but I think it would be useful to have SSL support for the Realm Object Server as part of the default Getting Started path. I can't imagine anyone not wanting to use a secure connection when transmitting user data.
I'm a devops guys so getting port 9080 working with SSL is unknown territory. Any tips on this front would be useful.
Thanks.

This repository is related to an internal component of Realm. If you need assistance with the Realm Mobile Platform, please file an issue in the realm-mobile-platform repository.


re #889 Update Naming

cc @EricWittmann - Still working on the last 7 blog posts.

OK cool - thanks @kahboom  - should I wait to merge until those last 7 are done?

@EricWittmann - Up to you, I'm guess it will take me about an hour or two to look over everything, make sure it looks okay, and maybe have time to edit the images. Right now I only need about 4 more posts (no images).

OK great - I'll wait until you've got the rest ready I guess. :)

@EricWittmann - I pushed up my changes yesterday, which covers all blog posts, however, I'd like for you to take a look at the 2015-05-19-rest-api-redux.markdown post to make sure everything was updated properly, as I felt the least confident with those changes. I'm just editing some images for the blog posts now and will do a sweep through the blog in case I broke anything.

Fixed the styling, updated the text (see screenshot), and fixed an issue where the newer posts were not displaying the notice block. Working on images and also, I'm not sure what we want to do about duplicate entries in the list of blog posts, if anything at all, as that may be confusing for users.


[10/13/2022]:  ISSUE - would like to improve the web accessibility of the site

@BatoolMM @alswajiab would like to improve the web accesibility of OSCSA website, this would help people with disability to use the platform with ease
i would be working on adding tooltip  to various call to actions or buttons in the site , this would help web assitive technologies e.g screen reader
i would also be adding short descriptions to alt attributes in all image tags  thats found in the html code
lastly : according to WAI(web accessibility initiative) using placeholders only in a form doesnt do justice to users of a certain form of disablity...so to improve this i would be adding labels to the forms

i wont be doing the second one "adding description to alt attribute" because another contributor has worked on it

@BlueDragn i am also an outreachy applicant ,but here is what is going to happen, we can split it since its two issues i discovered, so you can also have something to work on....i would take the last issue i stated i.e adding labels then you can take the other one i.e adding tooltip...thats fair i guess

@jesutobi ofcourse, No issue.


bug fix: check errors

Pull Request Test Coverage Report for Build 147

6 of 11 (54.55%)  changed or added relevant lines in 2 files are covered.
6 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.006%) to 46.575%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/egress/egress_config.go
5
10
50.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/egress/egress_config.go
1
68.0%

resolver/destination.go
2
63.83%

protocol/dubbo/dubbo/request.go
3
0.0%

Totals

Change from base Build 145:
0.006%

Covered Lines:
1020

Relevant Lines:
2190

💛  - Coveralls


Need a way to escape mustache

We need to support tripple mustache to escape the curly braces. It doesn't seem to work.

Justo to double-check; Do you want {{{path}}} to be no-op for liaison?

I would expect it to render {{path}} instead of performing binding logic.

Another double-check: Did you actually mean rendering {{{path}}}?

When I put tripple mustaches in my template I expect double mustaches to be rendered. So it is: {{path}} in the light DOM.

I see, any concrete use case?


Missing requirements.txt

In the installation instructions, it is stated that:
Install python (either 2 or 3) requirements: pip install -r requirements.txt
but there is no requirements.txt in the repo.

Thanks for pointing this out. Looks like the overzealous .gitignore filtered it out. Both are fixed in 1c4fa367bc0270f30d38d59709f2c716a8f2502f.


🐛 Fix regression in checking kcp readiness

Summary
Fix a regression in checking kcp readiness
Related issue(s)
Fixes #

/lgtm
/approve


Fix limited MD rendering

This fixes a few issues with the current solution, all of which are
covered by the accompanied test:

Fixes matching tags whose contents span multiple lines. This is
accomplished with the change from (.+?) to ([\s\S]+?).
Fixes matching tags such as
 with the change from [a-z] to
[a-z0-9].
Fixes matching tags with attributes, such as
 by
adding \s*(?:\s[^>]+)? after the tag name matcher.
Fixes matching more than one tag by adding the g flag.
Fixes matching nested tags by recursively calling the replace
function until it stops having any further effect. Note that this
renders the g flag redundant, but I left the flag in, because it
avoids the recursive call stack getting unnecessarily deep.

The test could probably be improved to not have to require the one
trailing whitespace in and the empty newlines at the end of the expected
string, but I couldn't think of a good way to do this right away.

Out of curiosity: what case made you do that? Do you really use that much html in your code? Technically I agree with this PR... but
? Seriously?

Well, this came about because I was trying to use markdown, similar to that in the test, in a @property description. But that got rendered to faulty HTML, with every other HTML tag in place, but only the first
 tag being removed. I thought there's a bug with the MD rendering, but turns out the bug was with the stripping here. So I fixed it to avoid the confusion.
I will soon submit a PR for allowing full MD in @property description as well, because I don't see any reason not to have this. And the fact that it doesn't support full MD also doesn't seem to be documented. There's another bug with the summary/description detection that has to be fixed first, though.

https://github.com/coffeedoc/codo#github-flavored-markdown - it is documented. It's mainly done to not end up with damaged layout (which can differ from theme to theme). Can I please ask you to provide a useful case where full-featured html could be useful in code-level documentation? Or particular case that rendered wrong for you. Just paste it here please.

This was my actual use case
  class ThingRequest

    # @property {Promise}
    #
    # ### Fulfilled with {Thing}
    # If the request is successful and an the other party accepts the request then
    # this promise will be fulfilled with the resulting {Thing}.
    #
    # ### Rejected with {Error}
    # If the request is unsuccessful for any reason then this promise will be
    # rejected with an {Error}. The {Error} may have one of the following causes:
    #
    # - INVALID_INPUT
    # - DECLINED
    # - UNAVAILABLE
    # - CANCEL
    # - TIMEOUT
    # - INTERNAL_ERROR
    thingPromise: null

As for the link to the README's GitHub Flavored Markdown section, yes I saw that part, but what I meant was that it doesn't list @property.

So I've been thinking on this lately. This PR in general doesn't make sense at all. This kind of "limitation" doesn't really limit anything. Also you mention h1 in your comment but it's not even in the list of allowed tags. So basically this code is still supposed to strip it off. In other words your PR doesn't help the aforementioned comment to be converted directly. Am I missing something?
Basically if we go with supporting all the tags - we can just skip this limiter at all. But right now the UI isn't ready. From one side I understand that removing that would make things more scalable for users and hey, if UI is destroyed by incorrect tag one day - you just go and remove it. On the other side you are the first who actually requested that. What if we, instead, will allow to disable this filter via, say, a command key? I think this should cover all the needs.

In other words your PR doesn't help the aforementioned comment to be converted directly. Am I missing something?

I think you are misunderstanding the purpose of this PR, yes. This PR doesn't allow my example to be converted into full-fledged HTML, what it does is avoid creating faulty HTML. Let me give you a clear example of what this PR changes:
Say a Codo user, who hasn't read the documentation and doesn't know that they can't use full markdown for @property documentation, has the following code (similar to the example I gave above) and generates documentation for it:
class Thing

  # @property {Object}
  # The property can be one of the following
  # - OPTION1
  # - OPTION2
  prop: null

At the moment the generated HTML would include something like the following:
The property can be one of the following

OPTION1

OPTION2

Which I hope you agree is unexpected and broken. Whereas the expected output, and the output with this PR being merged, would include something like the following:
The property can be one of the following
OPTION1
OPTION2

So this PR isn't trying to change any of the current behavior, and I'm not trying to convince you that you should enable unlimited Markdown in the @property tag, for example. What this PR does is make sure that the limitations you currently already have in place behave correctly and don't generate faulty HTML (viz the missing
 tag above).

A-ha. Correct. Sorry, don't have to much time to dig deep into things lately. Very good then, thank you.

No worries :) Thanks for merging.


[xxxx] Amended to alpine 3.20 and ruby 3.2.5 and nodejs 20.15.1

Context
Alpine & Ruby & NodeJS version is a bit old
Changes proposed in this pull request
Updated Alpine & Ruby & NodeJS version
Guidance to review
Nothings breaks and should fix
https://github.com/DFE-Digital/register-trainee-teachers/actions/workflows/build-nocache.yml
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

worth adding a review app?


refactor: simplify pagerduty test onnection

Summary

simplify pagerduty test onnection
remove incorrect migrationscript
fix pagerduty connection table

Does this close any open issues?
Part of #4298
Screenshots
test connection

create connection

@klesh LGTM


Remove preview from aspnetcore.doc 9.0, create 10.0 preview moniker

USER STORY 341247
USER STORY 340177

Associated WorkItem - 341263

Done


Add flight tracker tutorial

I'm having some (OK a LOT) of difficulty getting flight tracker to function w/ the Azure load balancer. Everything deploys but the application just "toilet bowls" when accessed through a public IP. I think Crossplane remote mode ends up creating a basic SKU load balancer instead of standard, and it's not as good.
Take a look at the tutorial content here - I think this is a good start for user engagement.

This is a very detailed tutorial. I did a first pass on it for now. Let me replay this by myself tomorrow to provide more relevant feedback.

Keep in mind when you're doing this that this tutorial is designed for someone totally new to OAM to:

Learn some basic ARM-DSL
Learn some basic OAM concepts
Deploy something basic
Try writing / deploying some components with requirements given to them
Give feedback to us at the end

The key (from my experience with user studies) is to explain enough and give clear enough requirements that users can reasonably complete a task, but not hold their hand entirely.
I'm curious to get someone who doesn't know OAM to try it as well (once we have local mode) so we can assess the length.

I'm having some (OK a LOT) of difficulty getting flight tracker to function w/ the Azure load balancer. Everything deploys but the application just "toilet bowls" when accessed through a public IP. I think Crossplane remote mode ends up creating a basic SKU load balancer instead of standard, and it's not as good

@asouza - if you go through this let me know what your experience is like. I'm having trouble getting the flight tracker to function at all without port-forwarding, and I wonder if it's a bad interaction between the load balancer and the app.
The data retrieval for me also seems to just plain not work.

@alex-frankel - you might find this interesting

I had issues with the LB on Azure too. Add instructions for port-forward instead. I also don't know why all services need an external IP.

Thanks for testing this out. I'm going to run through it one more time and address the non-textual feedback 👍

Updated

updated


Product Item amount charged over 1000 contains ","

for lletourneau
Item amount charged over 1000 contains ","

It may be in load_active_order_from_db where we format the string differently than the other values


fix: Arrow Buttons Visible at Carousel Start and End Positions

Issue : https://github.com/RamakrushnaBiswal/PlayCafe/issues/457
The "Prev" button is now conditionally rendered based on the active index. It will only appear if the active index is greater than 0 (i.e., not on the first item).
The "Next" button is conditionally rendered based on the active index. It will only appear if the active index is less than the last item (i.e., not on the last item).
Screenshots

@RamakrushnaBiswal Can you please review and if this PR is well and good to merge, then merge it ASAP as tommorow is the last day for gssoc-ext


Launchpad issuance graph doesn't state assumptions

The issuance graph on the launchpad shows maximum validator return of 21.6% (at 524,288 ETH staked) and 4.9% return at 10m ETH staked. However, digging into the code that generates these numbers (https://github.com/ethereum/eth2.0-deposit/blob/83f34b82a967ba52532f58e7c28467982429a4a2/src/utils/calculateEth2Rewards.ts) it seems that this is based on average network participation of 95% and individual validator uptime of 99%. These assumptions are important and should be stated clearly on the launchpad.
Also the calculateEth2Rewards function seems to rely on a quadratic approximation of the expected inclusion delay reward using rewardAdjModifier. However the rationale for this approximation is not documented anywhere. How was this approximation derived? It looks as though it assumes a fixed penalty for every slot delay (oneSlotLatePenalty) but this isn't consistent with the spec, which indicates that inclusion delays lead to an inversely proportional scaling of the reward, not linear.
@flamingYawn

To add to this, it looks like this calculation may be based on a previous version of the eth2 spec which used a linear inclusion delay reward (see this version from August 2019).
The mainnet version of the spec scales the delay inclusion reward by the inverse of the delay (i.e. for delay n the reward is scaled by 1/n). The expected reward for below 100% participation can be calculated by summing the geometric series.


I hope you can configure the html singletons

I hope you can configure the html singletons.
Because some html templates have singletons custom...
eg:
"codeformatter_html_options": {
    ...
    "singletons": "extend",
    ...
}

# These are the tags that are currently defined as being void by the HTML5 spec, and should be self-closing (aka singletons)
self.singletons = r '<(area | base | br | col | command | embed | hr | img | input | keygen | link | meta | param | source | track | wbr |' + opts.singletons + ') ([^ >] *)> '

Can you give an example of some custom HTML template singletons and what language they reside in?