Unable to deploy pre-built JS function using azure/static-web-apps-deploy and github actions

Describe the bug
I have successfully deployed a static web app using the Azure/static-web-apps-deploy.  The code deployment process succeeds correctly, says the API was found and the artifact was uploaded but the function is not active.
To Reproduce
Steps to reproduce the behavior:

create nx repository for angular: npx create-nx-workspace@latest angular-monorepo --preset=angular-monorepo
add the azure function plugin: npm install -D @ziacik/azure-func, nx generate @ziacik/azure-func:application --name=my-func-app --directory=packages/my-func-app --projectNameAndRootFormat=as-provided, and nx generate @ziacik/azure-func:function --name=hello-world --project=my-func-app
Create a github action to pre-build both the angular app and the azure typescript function.  disable the build process with skip_api_build, skip_app_build and set the app_location and api_location to the build folder.
Watch the build succeed

Expected behavior
Both the angular static app and the javascript based function are deployed and running
Actual behavior
The angular static app is successfully deployed and running, the azure function is not active
Screenshots
Here is how the API section of the static site looks like in the azure portal:

I logged the directory contents of the prebuilt function in the GitHub action:
Run ls -lah /home/runner/work/n-notes/n-notes/nnotes-client/dist/apps/n-notes-func
total 28K
drwxr-xr-x 2 runner docker 4.0K Mar  7 01:26 .
drwxr-xr-x 4 runner docker 4.0K Mar  7 01:26 ..
-rw-r--r-- 1 runner docker  289 Mar  7 01:26 host.json
-rw-r--r-- 1 runner docker  568 Mar  7 01:26 index.js
-rw-r--r-- 1 runner docker  172 Mar  7 01:26 local.settings.json
-rw-r--r-- 1 runner docker 2.0K Mar  7 01:26 package-lock.json
-rw-r--r-- 1 runner docker  [15](https://github.com/***/n-notes/actions/runs/8181425269/job/22371179279#step:10:16)4 Mar  7 01:26 package.json

And the results of the azure/static-web-apps-deploy that shows the success of the deployment and the configuration for the function from my staticwebapp.config.json:
Verbose logging enabled
Try to validate location at: '/github/workspace/github/workspace/nnotes-client/dist/apps/n-notes'.
Build timeout not specified, defaulting to 15 minutes
App Directory Location: '/github/workspace/nnotes-client/dist/apps/n-notes' was found.
Try to validate location at: '/github/workspace/swa-db-connections'.
Looking for event info
Event info parsed from GitHub event json.
Skipping step to build /github/workspace/nnotes-client/dist/apps/n-notes with Oryx
Found staticwebapp.config.json file: 'nnotes-client/dist/apps/n-notes/staticwebapp.config.json'
Didn't find Oryx manifest file under location: /bin/staticsites/65286476-b1fc-4bfb-8efc-77c58fa0f1bf-swa-oryx/app-manifest/oryx-manifest.toml
Determined default file to be: index.html
Using 'staticwebapp.config.json' file for configuration information, 'routes.json' will be ignored.
Calculating the size of app artifacts: 269678 B
Try to validate location at: '/github/workspace/github/workspace/nnotes-client/dist/apps/n-notes-func'.
Api Directory Location: '/github/workspace/nnotes-client/dist/apps/n-notes-func' was found.
Skipping step to build /github/workspace/nnotes-client/dist/apps/n-notes-func with Oryx
Function Runtime Information. OS: linux, Functions Runtime: ~4, node version: 18
Zipping Api Artifacts
Api Zip will be created from directory: /github/workspace/nnotes-client/dist/apps/n-notes-func
Api Content Hash: 4894e7bb38a16776d8b8749b6379e931
Done Zipping Api Artifacts
Zipping App Artifacts
App Zip will be created from directory: /github/workspace/nnotes-client/dist/apps/n-notes
Done Zipping App Artifacts
Uploading build artifacts.
Finished Upload. Polling on deployment.
Status: InProgress. Time: 0.0970312(s)
Status: InProgress. Time: 15.1840[23](https://github.com/***/n-notes/actions/runs/8181425269/job/22371179279#step:11:24)(s)
Status: Succeeded. Time: 30.[27](https://github.com/***/n-notes/actions/runs/8181425269/job/22371179279#step:11:28)76[49](https://github.com/***/n-notes/actions/runs/8181425269/job/22371179279#step:11:50)9(s)
Deployment Complete :)
Visit your site at: https://***.4.azurestaticapps.net/
Thanks for using Azure Static Web Apps!
Exiting

Notice that the logs says:

Function Runtime Information. OS: linux, Functions Runtime: ~4, node version: 18
Api Zip will be created from directory: /github/workspace/nnotes-client/dist/apps/n-notes-func

Here are the contents of index.js
"use strict";

// apps/n-notes-func/src/hello/hello.function.ts
var import_functions = require("@azure/functions");
async function hello(request, context) {
  context.log(`Http function processed request for url "${request.url}"`);
  const name = request.query.get("name") || await request.text() || "hello";
  return { body: `Hello, ${name}!` };
}
function registerHelloFunction() {
  import_functions.app.http("hello", {
    methods: ["GET", "POST"],
    authLevel: "anonymous",
    handler: hello
  });
}

// apps/n-notes-func/src/index.ts
registerHelloFunction();

and package.json
{
  "name": "n-notes-func",
  "version": "0.0.1",
  "dependencies": {
    "@azure/functions": "4.3.0"
  },
  "main": "./index.js",
  "type": "commonjs"
}

my github action
name: nNotes PWA

env:
  NODE_VERSION: "18"
  VERBOSE: 'true'
  APP_LOCATION: "${{ github.workspace }}/nnotes-client/dist/apps/n-notes" # location of your client code
  API_LOCATION: "${{ github.workspace }}/nnotes-client/dist/apps/n-notes-func" # location of your api source code - optional
  OUTPUT_LOCATION: "" # location of client code build output

on:
  workflow_dispatch:
  push:
    branches:
      - main
    paths:
      - .github/workflows/nnotes-pwa.yml
      - nnotes-client/**

  pull_request:
    types: [opened, synchronize, reopened, closed]
    branches:
      - main
    paths:
      - .github/workflows/nnotes-pwa.yml
      - nnotes-client/apps/**
      - nnotes-client/libs/**

permissions:
  issues: write
  contents: read
  pull-requests: write

jobs:
  dev:
    # if: github.event_name == 'push' || (github.event_name == 'pull_request' && github.event.action != 'closed')
    runs-on: ubuntu-latest
    defaults:
      run:
        working-directory: ./nnotes-client
    permissions:
      contents: read
      actions: read
    name: Build and deploy to the dev environments
    steps:
      - uses: actions/checkout@v4
        with:
          fetch-depth: 0

      - uses: actions/setup-node@v4
        with:
          node-version: "${{ env.NODE_VERSION }}"
          cache: 'npm'
          cache-dependency-path: ./nnotes-client/package-lock.json

      - name: Install CI Dependencies
        run: |
          npm ci
          npx playwright install --with-deps

      - name: Derive appropriate SHAs for base and head fo `nx affected` commands
        uses: nrwl/nx-set-shas@v4

      - name: Ensure tracking main branch for the `nx affected` command.
        # This line is needed for nx affected to work when CI is running on a PR
        run: git branch -u origin/main

      - name: Run Lint, Test and Build
        run: npx nx run-many -t lint,test,build --parallel=3 --projects=n-notes,n-notes-e2e,n-notes-func

      - name: Run E2E tests
        run: npx nx affected -t e2e

      - name: Show function directory
        run: |
          ls -lah ${{ env.API_LOCATION }}

      - name: Deploy
        uses: Azure/static-web-apps-deploy@v1
        with:
          azure_static_web_apps_api_token: ${{ secrets.NNOTES_PWA_DEPLOY_KEY }}
          repo_token: ${{ secrets.GITHUB_TOKEN }}
          action: "upload"
          skip_api_build: true
          skip_app_build: true
          app_location: ${{ env.APP_LOCATION }}
          api_location: ${{ env.API_LOCATION }}
          # output_location: ${{ env.OUTPUT_LOCATION }}

  close_pull_request:
    if: github.event_name == 'pull_request' && github.event.action == 'closed'
    runs-on: ubuntu-latest
    defaults:
      run:
        working-directory: ./nnotes-client
    name: Close Pull Request
    steps:
      - name: Close Pull Request
        uses: Azure/static-web-apps-deploy@v1
        with:
          azure_static_web_apps_api_token: ${{ secrets.NNOTES_PWA_DEPLOY_KEY }}
          app_location: ${{ env.APP_LOCATION }}
          api_location: ${{ env.API_LOCATION }}
          action: "close"

staticwebapp.config.json
{
  "navigationFallback": {
    "rewrite": "/index.html",
    "exclude": ["*.{css,scss,js,png,gif,ico,jpg,svg}"]
  },
  "platform": {
    "apiRuntime": "node:18"
  }
}

It dawned on me (literally this morning as I awoke) that the problem could be caused by the missing node_module and dependencies in my pre-built function.  Why would static sites run npm i?
🥁
It was.
Since I don't have another developer working on this with me, this issue became my second pair of eyes.
I will close this issue because it is my oversight and not an actual issue.


Docusaurs: Update the footer UI & mobile menus. Closes #4611

Closes #4611
The POC has been updated to include this change: https://jwaegebaert.github.io/cli-microsoft365-docusaurus/
Note: I've also updated the custom breadcrumb code to make use of TypeScript instead of JavaScript 😄
Sneak preview of the changes.
Mobile navigation menu

The footer

Also something I noticed, but is maybe not 100% related to this PR, the GitHub hyperlink at the top right corner is pointing at the Docusaurus GitHub repo.

Merged manually, thank you! 👏


[ONBOARDING] Léllé Demertzi - RPM

Welcome, Léllé! :sun_with_face:
Onboarding check-list for TPS: https://github.com/alan-turing-institute/tps-project-management
Notes/Turing Specific for Léllé
Project board: https://github.com/orgs/the-turing-way/projects/3/views/2

Onboarding notes

General IT: https://hackmd.io/@cm-team/onboarding (taken from Malvika)

Turing Acronyms: https://thealanturininstitute.sharepoint.com/:w:/s/rid/ESfeWchNrgVHnUdqFlNoFtEBcsCJqHya-oOjN1ixCVkNBQ?e=xBilWc
TPS RACI Matrix: https://thealanturininstitute.sharepoint.com/:x:/s/rid/EUnS3TZcbCtOhX6RiqcZSuMB_1e8gNtYUvs_pLaOa3vHrg?e=YddofX

TTW Core Team Onboarding checklist
This was copied from the broader core team onboarding packet, for your own reference: https://hackmd.io/@turingway/ttw-core-team-onboarding :sunflower:
Accounts and Permissions

[x] Add to TTW Github repository with "Writer" permissions (change as needed for role)
[x] Add to Ways of Working page on TTW Github repository
[ ] ~Add to Core Team Google group (currently dormant)~
[ ] (If applicable) Give Accounts/Password Access on Keybase
[x] (If applicable) Add directly to The Turing Way Core Google Drive

[x] Share promotion pack folder
[x] Share conference talks/workshop applications folder

[x] #47
[ ] ~(If applicable) Add to Canva team~ @LelleDemertzi will talk to IT about getting access to Keybase
[ ] (If applicable) Add to Penpot team
[x] (If applicable) Add to HackMD team
[x] (If applicable) Add to Keybase group @LelleDemertzi will talk to IT about getting access to Keybase

About yourself

[ ] Add information to TTW Team slides
[x] (If applicable) add yourself to TTW project page on Alan Turing website
[x] (If applicable) add yourself to TPS/Whitaker Lab slide deck
[ ] Introduce yourself on the Turing Way slack (in #introductions)

Events & Calendars

[ ] Give links to TTW Community Calendars

[ ] The Turing Way Community (public-facing)
[x] The Turing Way Core Team (internal)

[x] Add to calendar invites for The Turing Way core team

[x] Coworking Calls
[x] Collaboration Cafes
[x] Fireside Chat
[x] Community Chats

Communication Channels + Platforms

[x] Share links

[x] Slack
[x] Twitter
[x] Mastodon
[x] Newsletter

[x] Useful Links/Information Hub

[x] Core Team note of notes

If applicable, see TPS Project Management Repository about being onboarded onto the Tools, Practices, and Systems programme more broadly at the Alan Turing Institute. Other research teams and/or groups may have their own onboarding practices and/or repositories (i.e. the Research Application Manager (RAM) team or Community Management (CM) team).
Useful links

The Turing Way Core Google Drive
Community Handbook
TTW Guide to Using Slack
TTW Github repository guide
TTW Translation Github team
TTW Core Team Google Drive Guide

TTW-PM specific tasks:

Existing Project Management

Github: https://github.com/orgs/alan-turing-institute/projects/27/views/1 - Is this project board out of date?
Trello: https://trello.com/b/F1nERwsT/community-management
Note of notes: https://hackmd.io/@turingway/ttw-team-note-of-notes

Turing Way Core: https://drive.google.com/drive/folders/1bdjA-fBotMi9CnjDqu84d5hPZWwjgi-x
Templates & Walk-through

Newsletter
Fireside chat planning

Invitation emails
Planning document
Follow-up emails (reminder week before)

Workshop/Conference Planning

Workshop document (with timings)
Exhibit planning (for gear)

Communications slide deck
Partners & Community

Onboarding tasks: https://hackmd.io/@cm-team/onboarding
Checklists

Before/During/After Fireside
Before/During/After Book Dash
Before/During/After Collab Cafe
Before/During/After Coworking call
Before/During/After Core Team Meeting

16 September

@Arielle-Bennett - I've sent @LelleDemertzi the link for the TPS Project Management repository, wasn't sure if she should do any things there, but I know it was used for onboarding @AlexandraAAJ!

Join us in welcoming our New Research Project Manager, Léllé Demertzi!
We are delighted to announce that Léllé Demertzi has joined our team as the Research Project Manager of The Turing Way!
Léllé, a performance artist by training, brings a wealth of experience to The Turing Way community through her diverse roles and accomplishments.
Before joining our team, she served as a Programme Administrator for the Living with Machines project, where she helped organise various events, exhibitions, and outreach activities within the Arts and Humanities field.
Léllé also worked as the Programme Coordinator for Data Science for Science and Humanities and co-organised the AI & Arts Interest Group at The Alan Turing Institute.
We are excited to welcome her to our team and look forward to introducing her to our community!

Newsletter ready to send out by EOD
Introduction to Slack channels ready
For next meeting:

Zenodo upload

Question RE: Turing reporting for Apr-Oct 2024
Where is information captured? Is there a process in place for formatting for the reporting template?
deadline 18/10/24

@aleesteele the repo we've moved to is tps-team, Léllé should have access to that now. Thanks!

Closing this - woohoo! 🙏

[celebrate]     Alexandra Araujo Alvarez reacted to your message:

From: Anne Lee Steele @.>
Sent: Thursday, December 5, 2024 3:44:05 PM
To: the-turing-way/2024-rcm-rpm-tasks @.>
Cc: Alexandra Araujo Alvarez @.>; Mention @.>
Subject: Re: [the-turing-way/2024-rcm-rpm-tasks] [ONBOARDING] Léllé Demertzi - RPM (Issue #46)
Closing this - woohoo! 🙏
—
Reply to this email directly, view it on GitHubhttps://github.com/the-turing-way/2024-rcm-rpm-tasks/issues/46#issuecomment-2520669028, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A5TNXPQ2GGTLT5L4N4SLZ4L2EBYELAVCNFSM6AAAAABOJK5ZSSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRQGY3DSMBSHA.
You are receiving this because you were mentioned.Message ID: @.***>


Can you disable game status?

I want to disable game status if a new song is playing. How to do that?

In bot.py, in async def update_now_playing change for this:
    async def update_now_playing(self, entry=None, is_paused=False):
        pass

I don't find bot.py

Musicbot/bot.py


CAN Readings

Hi,
First of all, I think what you have done with the UI looks awesome! Way more impressive than Leaf Spy.
I want to know how you actually got to interpret the readings from the CAN messages onto the app? Please excuse my ignorance, but I can't see where in the code the interpretation takes place using the canmsgs codes from xls. that you have used. How did you do this?
I am working on a similar project for my university project. I want to get readings from the Leaf and send it to a server where I can use it for estimating final SoC using the initial SOC at the beginning of a trip. Also, taking into consideration if headlights are on, air conditioning, current speed etc.
Kind regards
Martin

Hi, sorry for the delay.
Codes sent to the OBD are defined here: https://github.com/blandman/HappyLeaf/blob/master/app/services/flow-manager.js
Then are parsed here: https://github.com/blandman/HappyLeaf/blob/master/app/services/data-manager.js

Thanks so much for your reply. I really appreciate it.


CB-16420 Make sure the varint is not null in the Core module. The Clo…

…udPlatformConnector#getDefault couldn't be deleted because in Environment service the resources doesn't have variant
See detailed description in the commit message.

retest this please


Feature: Implement external url warnings and whitelists for proposal descriptions

This PR brings forward the external URL warning system for proposal descriptions as implemented on the former TerraStation platform, while modifying behavior slightly to accommodate interchain (updating whitelists to reflect common discussion platforms for current chains, and changing text verbiage to reflect URLs external to Station rather than external to the Terra ecosystem).
Specifically, this PR introduces the following changes:

For any URL (actual or potentially obfuscated) in a proposal description, instead display [external link] with a tooltip explaining why the URL was not displayed
For proposals with detected external URLs, allow users to view the original proposal description selecting a "view original" checkbox displayed below the description
For proposals with detected external URLs, display reminders about the importance of securing mnemonics and private keys
Displays live warning message when URLs are detected on the SubmitProposalForm indicating that URLs will not be displayed as written when viewed via Station

This PR supersedes PR #159 and fixes #198

Converting to draft pending outcome of URL refinement conversation.

@alecande11 this PR is ready for consideration, and attempts to navigate between transparent governance, and limiting users from spam proposals.
Motivation
Terra Station has always provided a frictionless user experience by linking external URLs as clickable hyperlinks within governance proposals. This behavior has become a target of opportunity for spammers. This behavior has plagued Classic since early summer 2022, and following the launch of Station, many similar spam proposals have been posted to multiple different chains. These all seem to follow a common pattern of attempting to socially engineer users to visit an off-site platform designed to harvest mnemonics or private keys. Implementing this PR on Station would help to disincentivize these spam proposals on any chain integrated with Station by limiting the ease of acquiring traffic to spam sites.
When Classic proposal 7101 passed, a system was developed to mask the display of external links within governance proposal descriptions and integrated into Terra Station. User interface represents one facet of the many tools that exist to limit spam proposals on a Cosmos chain. The system proposed in this PR is designed to remind end-users of the importance of securing private keys and mnemonics, while still allowing a proposal to be viewed in an unmasked original form.
URL detection in free-form text is a complex problem to solve, as many non-URL string patterns, especially in a technical ecosystem such as cryptocurrency, are subject to false positive matches. In developing, and later improving upon, the regex backing the proposed system, a number of safeguards have been implemented, including the ability to whitelist domains, as well as the ability to exempt certain matches from being considered for detection or linking (this exemption is designed to allow for matches to common library names, e.g. terra.py or similar, be exempted)
Implementation
There are two primary behavior changes introduced by this PR: rendering of proposal descriptions, and warning users when a potential external resource is detected prior to submitting a new proposal.
Proposal description
A proposal with external URL references that are not present on either the whitelist or exempted list is masked with the text "[external link ?]" which also includes a tooltip explainer

Users are presented with warnings below the proposal description explaining why URLs were masked. In addition, a checkbox is provided allowing the original proposal to be presented (complete with external linking) if desired.

New proposal
During submission of a new proposal on Station, if a URL reference not in the whitelist or exempted list is included, a real-time indication is displayed. This provides the user an opportunity to review prior to spending coins on submitting a proposal, and adjust if so desired.

Risks/Caveats
No significant risks to implementation have been identified. Overmatching is the most notable caveat, but given the controls and safeguards provided, including the ability for users to view the proposal as originally written, it is my belief that the benefits of such a system to fostering a wallet environment friendly to users of all technical levels outweighs the occasional potential overmatch.
Acknowledgments
Thanks to @letorbi for countless suggestions on improving the regex backing this system.

Note: this PR has been rebased, and an issue with conditional hooks in ProposalDescription.tsx fixed (hooks became conditional based on original placement after fix in 4fa2b55 was introduced)

@letorbi, can you please complete your pending review? I inadvertently re-requested a review from you a couple days ago, and cannot clear it. Because it is outstanding, I am unable to request review from TFL. Sorry for the hassle.

No worries - just had  to find out how to mange reviews on GutHub ;) The regex looks good to me. Regarding the rest of the code I cannot say much, because I don't have not enough knowledge about the station code.

Rebased in response to feb04ca. @alecande11, is this PR something TFL would consider as a reminder to users to exercise caution when visiting external URLs, and hopefully serve as a deterrent to spam proposals?

I would like to ask, at least for the Terra v1 portion of Interchain Station, based on Terra v1 proposal 7101 that this PR be considered (and if there are any areas that would not work that you let the author know so he can consider adjustments that will work within the coding standards for the overall project).
Thank you :)

Duplicate of #251

GÜLERHOLDİNG AŞ. PATORUNU SMH HOLDING AŞ. Şirketi Yurt dışı İŞ Ortaklarına Teşekkür eder Birlikte Başlarılarımızın Devamını Dilerim.


Fill

sent everything over to main

This is ready to merge.  I need you to merge master first please

git fetch --all
git checkout master
git pull
git checkout your_branch
git merge master
npm run build

If the tests still pass, update the branch after that and I'll merge

$ git checkout master
error: pathspec 'master' did not match any file(s) known to
git
On Sat, Feb 13, 2021 at 12:20 PM John Haugeland<EMAIL_ADDRESS>wrote:

git fetch --all
git checkout master
git pull
git checkout your_branch
git merge master
npm run build
If the tests still pass, update the branch after that and I'll merge
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/StoneCypher/circular_buffer_js/pull/26#issuecomment-778648418,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVHMGOUNIUATSJRBBK3TFDS62YHJANCNFSM4XRADYQQ
.

merge main*

look up how to do it.  don't just try once and wait for me to tell you how

You didn't update the results in dist/ after the build, so they're still in conflict
Please add the post-builds

oh, you're still saying master -> main
master is the old branch.  i renamed it main a couple months ago because of the desire to eliminate things called master
if you merged something called master it's probably historic and probably had no impact

i was mistaken to say master earlier; it's main
i just call them master by habit.  trying to change

i could be talking you through this if you were on chat

✔ Appears to support Array.fill/1 API
✔ Tests are adequate
✔ Don't see any bugs
✔ Merged with main

In it goes


hashcat is listed twice

The entry for hashcat is list twice in the default profile

We are undergoing an overhaul and revamping package installations, thank you letting us know.


sidecar: Handle intermediate restarts of sidecar gracefully.

Fixes https://github.com/improbable-eng/thanos/issues/934
CC @SuperQ
Signed-off-by: Bartek Plotka<EMAIL_ADDRESS>

Can you try this out @SuperQ ?


Test runner fails

I get a traceback while running this:
$ python3.7 test_runner.py
Traceback (most recent call last):
  File "test_runner.py", line 2, in <module>
    from tests.test_argument_parser import TestArgumentParser
ModuleNotFoundError: No module named 'tests.test_argument_parser'

The test works as usual. You just can not start it. The module itself is in place?

In another Mac, I couldn't reproduce the issue. In the computer that produces the error above, if I run it with Python 3.6, it's runs as expected. The file test_argument_parser.py is in the tests folder. It's a bit strange.

Tested again in with pipenv (removing old Pipfile and Pipfile.lock, that I think shouldn't be needed at this time) and all tests pass. Maybe its something with this specific machine.


Setting Task Description: new line carriage return ignored.

When setting task description text, vbCrLf is ignored.
How can we specify that a new line should start?
Originally posted: 2015-12-24T09:28:03

I'm not sure what vbCrLf translates to in a string. However, I do know that in C# one can add new lines by putting in the "\r\n" combination. For example, the following code works:
td.RegistrationInfo.Description = "1) Line 1\r\n2) Line 2";

Originally posted: 2015-12-24T15:35:23

in vb.Net vbCrLf is the equivalent to  \r\n  (Carriage Return + LineFeed).
Did you test the effect of using \r\n ?
Originally posted: 2015-12-24T16:01:11

Tried environment.newline instead of vbCrLf.  But it still doesn't result in a new line inside the description.
Originally posted: 2015-12-25T06:23:29

Thanks for pointing out how this works as it has helped me find a bug. For some reason, the native Microsoft 2.0 library will strip out the CR value from the string after registering a RegistrationInfo.Description or a RegistrationInfo.Documentation value with a CRLF value in it. The native windows multi-line textbox, expects a CRLF to display multiple lines. I have added code to correct for the native error for those two items and it will appear in the next release.
Originally posted: 2015-12-26T14:05:43

I have the same issue in v2.8.6 (2018/12/19).  Does anyone know when the fix will be included?
Thx

Can you tell me if you are just using code to create the task of if this happens when using the UI editor?

I am using the code:
TaskDefinition taskDef = ts.NewTask();
taskDef.RegistrationInfo.Description = "Line1" + Environment.NewLine + Environment.NewLine + (char)10 + (char)13 + "Line2";
Thx

What value are you getting after registering (i.e. what exactly is getting lost)? Thanks for the detail.

Here's the code and results I get with the current NuGet package:
ts.RootFolder.DeleteTask("Test", false);
var td = ts.NewTask();
var txt = "Line1" + Environment.NewLine + Environment.NewLine + (char)10 + (char)13 + "Line2";
output.WriteLine(System.Text.RegularExpressions.Regex.Escape(txt));
td.RegistrationInfo.Description = txt;
output.WriteLine(System.Text.RegularExpressions.Regex.Escape(td.RegistrationInfo.Description));
td.Actions.Add("cmd", null, null);
var t = ts.RootFolder.RegisterTaskDefinition("Test", td);
output.WriteLine(System.Text.RegularExpressions.Regex.Escape(t.Definition.RegistrationInfo.Description));

Results:
Line1\r\n\r\n\n\rLine2
Line1\r\n\r\n\r\n\rLine2
Line1\r\n\r\n\r\n\r\nLine2

Will you confirm this is what you're seeing?
I'll work on this for the next release. I don't have a time-frame for it yet.

I have updated the code so that it will consistently convert all standalone LF to CRLF, all standalone CR to CRFL, and leave all CRLF as they are. Effectively, your string above would translate into "Line1\r\n\r\n\r\n\r\nLine2" as it would leave the first two NewLine values alone and the convert both the \n and the following \r to NewLine values as well. I cannot find an instance, using the current NuGet package on any Windows version, where NewLine is replaced with nothing. My goal was to ensure that the value pre-registration and post-registration matched. Given that the base library converts either \n or \r standalones to \n, it is the best I can do.


Expose routes, matchers in API

For a full replacement for tlsrouter, I need more flexible SNI matching than just string equality (dns wildcard and regex).
Adding this stuff to AddSNIRoute feels a bit icky, the functions are going to become unwieldy. How about making the route interface public, with appropriate changes? I think I have sort of a plan for how to make that work. @bradfitz thoughts?
Some things that would need to change in the API:

Use a Peeker interface in signatures instead of bufio.Reader
type Route interface { Match(Peeker) Target }. Gives you full power to examine input and decide on a Target.
type Matcher func(Peeker) bool and func StaticRoute(Matcher, Target) Route enables reuse/recomposition of matching code, and plumbs together the simpler cases.
Proxy.Add* replaced by a single Proxy.AddRoute(addr string, r Route). Maaybe make Proxy.AddStaticRoute(addr string, m Matcher, t Target) instead of a standalone StaticRoute constructor.

Tentative, poorly thought out usage example that I'm not 100% happy with but hopefully gives an idea:
var p tcpproxy.Proxy
// ACME is a complex implementation of the Route interface.
var a tcpproxy.ACME
// Handle *.acme.invalid
p.AddRoute(":443", &a)

// ACMETarget is passthrough for its arguments, so you can inline it elsewhere if you want.
// Registers the target as a candidate for acme challenges.
target := a.ACMETarget(tcpproxy.To("<IP_ADDRESS>:443"))
p.AddStaticRoute(":443", tcpproxy.MatchSNI("foo.com"), target)
// Inlined form
p.AddStaticRoute(":443", tcpproxy.MatchSNI("bar.com"), a.ACMETarget(tcpproxy.To("<IP_ADDRESS>:443")))
// Third-party matcher
p.AddStaticRoute(":70", retro.MatchGopher("/Plushie"), tcpproxy.To("<IP_ADDRESS>:70"))
// Dynamic router
p.AddRoute(":80", backpain.Yxorp()) // because it's a backwards proxy, see?

Rather than exporting all the Peeker complexity, why don't we maintain parsing of hostnames out of protocols (HTTP Host and SNI) and give the user a Match func(ctx context.Context, hostname string) bool instead where they can do the wildcards or database lookups, etc?

s/maintain/retain responsibility for/

I'm torn. What you describe clearly gives me less of a headache. OTOH I kinda like the notion of being able to implement complex things like backpain in a different package.
Let's go with the less exposed API until it becomes untenable.


Error when specifying URL in file for --track-config

I'm using a Singularity image of igv-reports v.1.0.4, and get this error when I add --track-config.
Traceback (most recent call last):
  File "/usr/local/bin/create_report", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.9/dist-packages/igv_reports/report.py", line 271, in main
    create_report(args)
  File "/usr/local/lib/python3.9/dist-packages/igv_reports/report.py", line 63, in create_report
    reader = utils.getreader(config["url"])
TypeError: string indices must be integers

This is what the file I pass to --track-config looks like (validated with JSONLint):
{
      "name": "Genes",
      "type": "annotation",
      "format": "gff3",
      "sourceType": "file",
      "url": "data/NC_000962.3.gff3.gz",
      "indexURL": "data/NC_000962.3.gff3.gz.tbi",
      "displayMode": "EXPANDED"
}

This is the command that generated the error:
singularity exec -B $PWD /singularity/igv-reports-1.0.4.sif create_report output/sample/sample_normalized.vcf.gz data/GCF_000195955.2_ASM19595v2_genomic.fna --flanking 1000 --info-columns AF DP MQ QD ANN --tracks data/NC_000962.3.gff3.gz --track-config configs/igv-tracks.config --output output/sample/igv.html

As far as I can tell my JSON config file looks similar to the example given in the igv.js Tracks wiki, except that I added double quotes around the property names, because otherwise, I get a JSONDecodeError.
I also have pysam 0.18.0 installed. Please let me know what further info is needed to help me with this issue, thank you!

I don't know anything about singularity.    The track-config should be an array of configurations, as noted in the readme,  in can have more than one.  I suspect that is the issue.   Are you able to run our examples?


Create master pliers transformer list

Since transformers often have to parameterized, a master list would be useful. Within datasets, specific transformers can be disabled or the set can be entirely overwritten.

Maybe instead extract this from feature schema? And can remove add on individual level?


Smart Selection

Hey there!
I write a lot of copy in XML, and am trying to only select the text after "<textevent timeoffset="0" message=" and in between the quotation marks. Like this:
        <textevent timeoffset="320" message="We're going to be breaking that 20min up by doing 5, 4min efforts, separated by 2min of recovery."/>
        <textevent timeoffset="330" message="We'll also be changing up the cadence with each interval, and on-bike positions."/>

Is there an extension that allows me to do that? I am trying to copy and paste the content into Grammarly.
Thanks!

Install my extension DOMQS https://marketplace.visualstudio.com/items?itemName=whizsid.domqs With this extension you can find your dom objects with a query selector

@whizsid thanks for your help! Would you mind running through what I'd need to do to only select the text between the quotations marks? What would the query be? Sorry, I have zero programming skills...

Use regex instead.

Type Ctrl+F to find. Click * (regex) button on the find widget.
Paste this <textevent\stimeoffset="(\d+)"\smessage=" into find box.
Click on your document.
Hit Ctrl+Shift+P
Type "Select all occurrences of Find Match" and click it.
Press the Right arrow key once.
Hold Shift key and press the "End" button.

@whizsid you're awesome thanks so much! I get this error though on Step 5:

Press right arrow key.

Ah-ha! Got it! Sorry about that, I wasn't searching in the document itself facepalm
Thanks so much for all of your help! I really appreciate it.


fix: Error on using select_next_entry and select_prev_entry

Fixes #1988
get_selected_index was checking self.active before returning an index, which is actually false when select_next_entry is first called in typical usage using a mapping. Switched to how cursor was calculated before this PR #1986

Sorry, might be totally out of place adding this here, but:
This fixes next and prev, but for some reason my autocomplete prompt is not by default selecting the first item in the list, to get to it I need to select_next then select_prev, any idea why?

@samuelpx link to your prompt? Or maybe post it here?

It is the same fix?
https://github.com/hrsh7th/nvim-cmp/pull/1993

Fixed by #1993, this can be closed.


add Crema filter

please get this updated with master and if you can, please rebase your commits into one!

I'm sorry -- the code for this filter isn't really aligning with the test images: :disappointed:


HttpResponse::tryEnd relies on evaluation order

which is unspecified - https://en.cppreference.com/w/cpp/language/eval_order
thus hasResponded() may come from before or after calling internalEnd(...)

Can you link where in code?
Den mån 30 dec. 2024 08:30Rafal W @.***> skrev:

which is unspecified -
https://en.cppreference.com/w/cpp/language/eval_order
thus hasResponded() may come from before or after calling internalEnd(...)
—
Reply to this email directly, view it on GitHub
https://github.com/uNetworking/uWebSockets/issues/1827, or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2NMOMN4M5GYBZW5GZBN5332IDY73AVCNFSM6AAAAABULUEKUGVHI2DSMVQWIX3LMV43ASLTON2WKOZSG43DENZYGEZTGOI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

https://github.com/uNetworking/uWebSockets/blob/d31eb40c98e1db71357702abf809329d4adbe76b/src/HttpResponse.h#L439

Great, post more if you find


Full flow coverage

Adds remaining flow coverage.
This is a pretty big change. Thank goodness for train rides to and from HQ 😅. Happy to break this down into smaller pieces. I'll make a few notes along the way too.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@5ba4e12). Click here to learn what that means.
The diff coverage is 97.05%.

@@            Coverage Diff            @@
##             master     #354   +/-   ##
=========================================
  Coverage          ?   99.54%
=========================================
  Files             ?       25
  Lines             ?      874
  Branches          ?        0
=========================================
  Hits              ?      870
  Misses            ?        4
  Partials          ?        0

Impacted Files
Coverage Δ

src/addons/with-send.js
100% <ø> (ø)

src/addons/action-form.js
100% <ø> (ø)

src/addons/action-button.js
100% <ø> (ø)

src/registration.js
100% <ø> (ø)

src/addons/model.js
100% <ø> (ø)

src/default-update-strategy.js
100% <ø> (ø)

src/meta-domain.js
70% <0%> (ø)

src/lifecycle.js
100% <100%> (ø)

src/addons/presenter.js
100% <100%> (ø)

src/utils.js
98.5% <100%> (ø)

... and 11 more

Continue to review full report at Codecov.

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

Neat. I'll add eslint-plugin-flowtype-errors to the project. I wonder how that interacts with standard flow plugins for editors (like nuclide for atom and flycheck for emacs).


Support DuckDB as computation backend

If I understand Kotlin dataframes correctly, computations are done directly in the JVM.
For many use-cases, the optimized data storage and vectorized computations of DuckDB could be very useful in terms of performance and memory consumption.
So maybe there could be a way to move computations to DuckDB via Apache Arrow, which offers a columnar memory format, zero-copy reads, streaming, and an extensive Java API.
The goal would be to perform analytical computations against DuckDB directly from Kotlin in a way similar to the one described here for Python and R:
https://arrow.apache.org/blog/2021/12/03/arrow-duckdb/
Looking forward to hear what you think about the feasibility of such an approach.

Hi!

If I understand Kotlin dataframes correctly, computations are done directly in the JVM.
Yes
Looking forward to hear what you think about the feasibility of such an approach.
First of all, thank you for sharing the post about DuckDB. I didn't know about it. I'm aware of Apache Arrow and we indeed consider implementing other backend as a long term goal of the library, so this information should be useful :)
Although, i wouldn't expect other backends to be implemented in near future as we are more focused on improving experience for in-memory computation use case and experimenting with more advanced tooling

@koperagen,
Thank you for the super-fast feedback!

Hi again. I was exploring posibilities for different computational backends and couldn't fully understand purpose and implemenetation of integration between Apache Arrow and duckDB. I assume scope of these libraries intersect greatly, but providing zero-copy integration enables some convenience..? If you use Apache Arrow <-> duckDB integration and call something like filter, result will be computed by Arrow?
Also, i couldn't find if this integration is available in Java API, only Python. Is that true?

If I am not mistaken, Arrow is more about the storage layout/format (a columnar format optimized for computations) than about the computations themselves. This format is the same for storing in memory and transmitting over the network.
So in case of huge data sets, a lot of time is saved by not (de)serializing data when crossing the border between applications and even between local memory and network buffers.

So while arrow greatly speeds up transferring the data around the computation is still done in the respective environment (e.g., DuckDB in your original example).
Hope this helps.
For more solid advice, it might be helpful to contact the Apache Arrow people.
I could imagine they would be interested in more integrations and able to give some pointers.

Your explanations made things a lot clearer, thank you. I'll continue researching this topic :)

I found that DuckDB Java API also offers export / import of data in Apache Arrow format. So, if we manage to fit DataFrame model into Apache Arrow format, it should be possible to move your data between duckdb / kotlin code without copying.
This way one can compute something in duckdb, then move to Kotlin and perform some operations, then again offload heavy computation to duckdb. You're proposing this kind of integration or something else? Maybe you have some specific use cases that can be enabled with it?


softhsm: 2.4.0 -> 2.5.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/softhsm/versions.
meta.description for softhsm is: '"Cryptographic store accessible through a PKCS #11 interface"'.

Checks done (click to expand)

built on NixOS
/nix/store/pavia0rjkfnx04q154s0isajjr4k3j8r-softhsm-2.5.0/bin/softhsm2-keyconv passed the binary check.
/nix/store/pavia0rjkfnx04q154s0isajjr4k3j8r-softhsm-2.5.0/bin/softhsm2-util passed the binary check.
/nix/store/pavia0rjkfnx04q154s0isajjr4k3j8r-softhsm-2.5.0/bin/softhsm2-dump-file passed the binary check.
3 of 3 passed binary check by having a zero exit code.
2 of 3 passed binary check by having the new version present in output.
found 2.5.0 with grep in /nix/store/pavia0rjkfnx04q154s0isajjr4k3j8r-softhsm-2.5.0
directory tree listing: https://gist.github.com/ac300ec697f0e66d66e875a1385f2329
du listing: https://gist.github.com/8706a76ec5b4c7e372b9847e8a2723ef

Instructions to test this update (click to expand)

One-time optional setup to skip building using Cachix:

Install cachix from https://cachix.org/
Use r-ryantm's cache:cachix use r-ryantm

Test this update by entering a nix shell, seeing what is inside the
result, and if applicable, running some binaries:
nix-shell --pure -I nixpkgs=https://github.com/r-ryantm/nixpkgs/archive/7cab2950958ebcfd25c7c124254a57cd53e2f72e.tar.gz -p softhsm
ls -la /nix/store/pavia0rjkfnx04q154s0isajjr4k3j8r-softhsm-2.5.0

cc @leenaars for testing.

Success on x86_64-linux (full log)
Attempted: softhsm
Partial log (click to expand)
shrinking RPATHs of ELF executables and libraries in /nix/store/pavia0rjkfnx04q154s0isajjr4k3j8r-softhsm-2.5.0
shrinking /nix/store/pavia0rjkfnx04q154s0isajjr4k3j8r-softhsm-2.5.0/lib/softhsm/libsofthsm2.so
shrinking /nix/store/pavia0rjkfnx04q154s0isajjr4k3j8r-softhsm-2.5.0/bin/softhsm2-dump-file
shrinking /nix/store/pavia0rjkfnx04q154s0isajjr4k3j8r-softhsm-2.5.0/bin/softhsm2-keyconv
shrinking /nix/store/pavia0rjkfnx04q154s0isajjr4k3j8r-softhsm-2.5.0/bin/softhsm2-util
gzipping man pages under /nix/store/pavia0rjkfnx04q154s0isajjr4k3j8r-softhsm-2.5.0/share/man/
strip is /nix/store/dxf1m7dhc4qb655bdljc1fsd74v1nag3-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/pavia0rjkfnx04q154s0isajjr4k3j8r-softhsm-2.5.0/lib  /nix/store/pavia0rjkfnx04q154s0isajjr4k3j8r-softhsm-2.5.0/bin
patching script interpreter paths in /nix/store/pavia0rjkfnx04q154s0isajjr4k3j8r-softhsm-2.5.0
checking for references to /build in /nix/store/pavia0rjkfnx04q154s0isajjr4k3j8r-softhsm-2.5.0...

Success on x86_64-darwin (full log)
Attempted: softhsm
Partial log (click to expand)
make[2]: Entering directory '/private/tmp/nix-build-softhsm-2.5.0.drv-0/softhsm-2.5.0'
make[2]: Nothing to be done for 'install-exec-am'.
make[2]: Leaving directory '/private/tmp/nix-build-softhsm-2.5.0.drv-0/softhsm-2.5.0'
make[1]: Leaving directory '/private/tmp/nix-build-softhsm-2.5.0.drv-0/softhsm-2.5.0'
post-installation fixup
gzipping man pages under /nix/store/zx2zjgdqbhai55zb36bl4m1q2gsh0gx3-softhsm-2.5.0/share/man/
strip is /nix/store/9xjkb4xz0b5lmizij9ppxy7lkxdxhx6b-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/zx2zjgdqbhai55zb36bl4m1q2gsh0gx3-softhsm-2.5.0/lib  /nix/store/zx2zjgdqbhai55zb36bl4m1q2gsh0gx3-softhsm-2.5.0/bin
patching script interpreter paths in /nix/store/zx2zjgdqbhai55zb36bl4m1q2gsh0gx3-softhsm-2.5.0
/nix/store/zx2zjgdqbhai55zb36bl4m1q2gsh0gx3-softhsm-2.5.0


return type of long-form refinement

As reported by @soundlogic2236, this code should be rejected by the typechecker:
interface Foo<out T> {
    shared formal {T+} val;
}
interface Bar<out T> satisfies Foo<T> {
    shared actual formal [T+] val;
}
class Bat() satisfies Foo<Integer>&Bar<Object> {
    shared actual [Object+] val => ["bob"];
}

Since Ceylon 1.3.0 it has been accepted. (Previous versions of the typechecker correctly rejected this code.)

OK, so this bug was introduced by @209b5afe68e678b55e7093ad28a8553b8ac28223 which was part of the fix for #6349. Commenting out the continue; statement fixes this bug but also, I assume, breaks the fix for #6349.

Fixed. Thanks @soundlogic2236.


Positioning of loading spinner is off

Issue Summary
When the internet connection is not super fast, you'll actually get the chance to see the loading spinner in the admin client. The spinner seems to be off (way to far on top of the page) for most of the pages in Ghost admin. See gif below.
Steps to Reproduce

Use Ghost not local!
Go to Design, Team or Subscribers
See the loading spinner spinning along on top of the page rather than centered

Technical details:

Ghost Version: Ghost 1.0 beta
Node Version: 4.5.0
Browser/OS: Chrome/Safari

Use Ghost not local!

This should also be testable locally by using the network throttling option in Chrome's Web Inspector.


Advanced cache system

automatic event listeners (via boot)
cache to memory (and optionally to cache layer)

Also provided AbstractGetCacheAction that ibjects CachMeService in __constructor


Implement robust dynamic memory

This is the core logic for robust dynamic memory. There are changes to both shaders and the driver logic.
On the shader side, failure information is more useful and fine grained. In particular, it now reports which stage failed and how much memory would have been required to make that stage succeed.
On the driver side, there is a new RenderDriver abstraction which owns command buffers (and associated query pools) and runs the logic to retry and reallocate buffers when necessary. There's also a fairly significant rework of the logic to produce the config block, as that overlaps the robust memory.
The RenderDriver abstraction may not stay. It was done this way to minimize code disruption, but arguably it should just be combined with Renderer.
Another change: the GLSL length() method on a buffer requires additional infrastructure (at least on Metal, where it needs a binding of its own), so we now pass that in as a field in the config.
This also moves blend memory to its own buffer. This worked out well because coarse rasterization can simply report the size of the blend buffer and it can be reallocated without needing to rerun the pipeline. In the previous state, blend allocations and ptcl writes were interleaved in coarse rasterization, so a failure of the former would require rerunning coarse. This should fix #83 (finally!)
There are a few loose ends. The binaries haven't (yet) been updated (I've been testing using a hand-written test program). Gradients weren't touched so still have a fixed size allocation. And the logic to calculate the new buffer size on allocation failure could be smarter.
Closes #175

Also note: this commit doesn't include generated shaders. We should land #178 and #179 then PR this against dev. But I wanted to get a draft out there to look at.

Marking this as ready for review, as the command line drivers are updated. No changes have been made to pgpu_render, so hopefully that'll continue as before (requiring pre-allocation of adequate buffers).

Was shader/image.png intentionally checked in? I couldn't find a reference to it in the PR.

Nope, that was by mistake. We'll remove it in a cleanup, thanks.


convert to kotlin multiplatform

TODO:

[x]  sonatype publishing (javadoc is missing)

closing in favor of #259


Revert "Revert "Parse EXIF metadata from HEIC files properly (#262)" (#274)"

This reverts commit 8a17e202c08b9f29d7c78e2eb6aadd66d1e2c58d.
Update snapshots with yarn test -u
Finish awaiting FileType promise before reading property

screenshot of the backend DB as "proof". I also downloaded the file and compared it to the one I uploaded to make sure its an exact match

It looks like https://github.com/yakovenkodenis/eslint-plugin-parentheses-around-await would have caught this, maybe add it in

Just did another round of manual verification, confirming that the EXIF metadata was extracted for both JPEG and HEIC images, as well as uploaded to Parse/back4app intact:


Added Let's Encrypt

They don’t have a careers section however they are looking
https://twitter.com/letsencrypt/status/653981879754821632

Thank you! :)


Document importance to order of paths

acl url_acl_service-rwe33l path_beg /
acl host_acl_service-rwe33l hdr(host) -i domain.com
use_backend https-service-rwe33l if url_acl_service-rwe33l host_acl_service-rwe33l

acl url_acl_service-u4nlas path_beg /ws
acl host_acl_service-u4nlas hdr(host) -i domain.com
use_backend https-service-u4nlas if url_acl_service-u4nlas host_acl_service-u4nlas

This does not work. Reversing /ws and / paths will make it work as expected.

This is not required anymore, since we sort paths now.
https://github.com/appscode/voyager/blob/f2aecc9e5cdef21b15b0f66ee68ae4d3a3bbf47b/pkg/haproxy/renderer.go#L43


A few regex examples, please!

Thank you, a regex cleanup facility is something I've waited a long time for in the Auto Link Title plugin. But a few examples in the documentation would really help to get going with this.

Added some examples from my config in 6e1eb8b.


Add missing rimraf dev dependency

Also, fix format of "author" in package.json - use angle brackets for email address, parentheses for website.

Thanks @MartinRosenberg !


HMR issue: createProxy is not a function

I have had an issue with HMR.
Current Behavior
I've tried setting up HMR with the Automatic setup, Manual setup and even the pre-0.49.0 setup. An error is thrown TypeError: createProxy is not a function, in node_modules/react-hot-loader/lib/patch.dev.js, which is fatal.
While writing this issue I've fixed it, I'm not sure if it's because of my setup but I think the stuff below is what is happening. If this is because of my setup, just close this issue so it could be used as reference for people having the same problem.
Haul has a dependency to<EMAIL_ADDRESS>Somehow (my) NPM seems to resolve this to react-proxy and not the zamotany fork. At this repo, 3.0.0-alpha.4 isn't available, and I guess, resolves to 1.1.8? The 1.x branch doesn't seem to have the default export (see here) (and breaks a lot of other stuff). Which ultimately throws the createProxy is not a function.
When I npm install<EMAIL_ADDRESS>myself, everything works :)
Expected Behavior
HMR should work.
Haul Configuration (webpack.haul.js)
module.exports = ({platform}, defaults) => {
    return {
        entry: `./index.js`,
        module: {
            ...defaults.module,
            rules: [
                ...defaults.module.rules,
                {
                    test: /\.js?$/,
                    include: ['react-native-linear-gradient'],
                    use: defaults.module.rules[1].use,
                },
            ],
        },
        resolve: {
            ...defaults.resolve,
            plugins: [...defaults.resolve.plugins],
            modules: [/* A list of internal 'root' dirs, we force resolving in a specific order because of white labeling */],
            extensions: ['.js', '.jsx', 'json', `.${platform}.js`, ...defaults.resolve.extensions],
            alias: {
                /* A list of internal aliasses */
            },

        },
    };
};

Environment
We use a weird/specific set of modules and aliasses in webpack (same as on web). Because the build can use additional source bundles/use custom stuff from optional swappable source bundles (white-label stuff).
libs
Haul: 1.0.0-rc.1
React Native: 0.53.3
Node: 8.11.1
Npm: 5.6.0

Closing this myself.


Smoke test

Docs Build status updates of commit f7b57c5:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

:x:Error

Details

Line 81: [Error] W:\zklh-s.optemp\packages\opbuild.scripts.1.72.0\tools\opbuild\plugin.executor.ps1 failed to execute customized script in the plugin step docset_prebuild in the docset node-api
Line 138: [Error] W:\zklh-s.optemp\packages\opbuild.scripts.1.72.0\tools\opbuild\docset.prebuild.ps1 failed to execute
Line 215: [Error] Error happened to execute the task _dependentPackages/CommonPlugins/tools/JoinTOC.ps1
Line 87: [Warning] Failed to run script W:\zklh-s_dependentPackages\CommonPlugins\tools\JoinTOC.ps1 with error message: Container cannot be copied onto existing leaf item..Exception.Message.

Line 81: [Error] W:\zklh-s.optemp\packages\opbuild.scripts.1.72.0\tools\opbuild\plugin.executor.ps1 failed to execute customized script in the plugin step docset_prebuild in the docset node-api
Line 138: [Error] W:\zklh-s.optemp\packages\opbuild.scripts.1.72.0\tools\opbuild\docset.prebuild.ps1 failed to execute
Line 215: [Error] Error happened to execute the task _dependentPackages/CommonPlugins/tools/JoinTOC.ps1
Line 87: [Warning] Failed to run script W:\zklh-s_dependentPackages\CommonPlugins\tools\JoinTOC.ps1 with error message: Container cannot be copied onto existing leaf item..Exception.Message.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Please consider using Hydra

Hi,
I am the author of OmegaConf and Hydra.
Happy to see you are using OmegaConf, please take a look at Hydra. It's specifically designed with research in mind and I think it can help your project a lot.
Hydra is building on top of OmegaConf, which you are already using so it should be easy to give it a shot.

Hi,
Thank you for the feedback. I checked out Hydra and it looks good. I decided to go with OmegaConf cause I was familiar with it and in the interest of time. Once I test Hydra with my framework, I can migrate to it. That will probably take a while though.

Sounds good.
as a heads up, OmegaConf 2.0.0 will be coming soon and there are some subtle incompatibilities.
See details in the RC release notes.
It would be great if you can test it ahead of time to make sure you are prepared.

Thanks for the info. I will then set the dependency to the current version till I finish testing out either the new one or Hydra.

@omry Is there a way to define a YACS like default.py using Hydra. I am having some issues defining a default.yaml. I tried following the steps (https://hydra.cc/docs/tutorial/defaults) but if I place it outside a config group and run my script like python main.py, it doesn't load (just shows a blank dict).
eg.

|---default.yaml
|   |
|   |---conf1.yaml
|   |---conf2.yaml
.
.
Just consider the config.yaml for this project and imagine the above hierarchy/ I want the default.py to be loaded if no config is provided else be overwritten/merged with another config passed through the command line. This should work if I understand the documentation correctly.
I also want the integrate my python logger with the hydra logger. Currently I log the output both to console and file but with different formatters. In such a case, how should I define the job logging parameters?

@tridivb, can you join the chat? it will be easier to communicate there in real time.


RT:678_reference range tooltip

Overview
Add a custom tooltip plugin for showing the reference range.

How it was tested
Tested using run showcase,cardio-patient, and cardio-health app locally.
Checklist

[x] The title contains a short meaningful summary
[x] I have added a link to this PR in the Jira issue
[x] I have described how this was tested
[x] I have included screen shots for changes that affect the user interface
[x] I have updated unit tests
[x] I have run unit tests locally
[ ] I have updated documentation (including README)

@semicolin can you please help me to test a buildConfiguration function
I have written the following but it giving an error to me can you suggest how to test this
describe('buildConfiguration', () => {
let datasets: Dataset[];
let scales: ChartScales;
let annotations: ChartAnnotations;
let layers: ManagedDataLayer[];
let configService: FhirChartConfigurationService;

    beforeEach(() => {
      layers = [
        {
          name: 'a',
          id: 'a',
          enabled: true,
          datasets: [
            {
              data: [
                { x: 10, y: 100 },
                { x: 20, y: 200 },
              ],
            },
          ],
          scale: { id: 'a' },
        },
      ];
      const layerManager: any = { selectedLayers$: of(layers) };
      configService = new FhirChartConfigurationService(layerManager, timeScaleOptions, ngZone);
      configService.chart = jasmine.createSpyObj<Chart>('Chart', ['zoomScale']);
    });

    it('should return the correct tooltip footer text when reference range is defined', () => {
      const config = configService.buildConfiguration(datasets, scales, annotations);

      const tooltipItem = {
        datasetIndex: 0,
        index: 1,
        x:<PHONE_NUMBER>000, // January 14, 2022
        y: 25,
      };
      const tooltipContext = [
        {
          datasetIndex: 0,
          index: 1,
          x:<PHONE_NUMBER>000,
          y: 25,
          label: 'Dataset 1',
        },
      ];

      const footerText = config?.options?.plugins?.tooltip?.callbacks?.beforeFooter(tooltipContext);

      expect(footerText).toEqual('Reference Range 10 - 20');
    });

    it('should return an empty string when reference range is not defined', () => {
      const config = configService.buildConfiguration(datasets, scales, annotations);

      const tooltipItem = {
        datasetIndex: 1,
        index: 2,
        x:<PHONE_NUMBER>000, // January 21, 2022
        y: 25,
      };
      const tooltipContext = [
        {
          datasetIndex: 1,
          index: 2,
          x:<PHONE_NUMBER>000,
          y: 25,
          label: 'Dataset 2',
        },
      ];

      const footerText = config.options?.plugins?.tooltip?.callbacks?.beforeFooter?.(tooltipContext) || '';

      expect(footerText).toEqual('');
    });
  });

Try something like this:
    it('should return the correct tooltip footer text when reference range is defined', () => {
      const dataset: Dataset = {
        label: 'Test',
        yAxisID: 'scale',
        data: [
          { x: 10, y: 100 },
          { x: 20, y: 200 },
        ],
      };
      const scale = { id: 'scale', type: 'linear' } as const;
      const annotations: ChartAnnotations = [
        {
          label: { content: 'Test Reference Range' },
          yScaleID: 'scale',
          yMax: 20,
          yMin: 10,
        },
      ];
      const layerManager: any = { selectedLayers$: EMPTY }
      const configService = new FhirChartConfigurationService(layerManager, timeScaleOptions, ngZone);
      const config = configService.buildConfiguration([dataset], { scale }, annotations);
      const tooltipItems = [{ dataset }] as TooltipItem<TimelineChartType>[];
      const beforeFooter = config.options!.plugins!.tooltip!.callbacks!.beforeFooter!.bind({} as any);
      const footerText = beforeFooter(tooltipItems);
      expect(footerText).toEqual('Reference Range 10 - 20');
    });

Notes:

You don't need to emit anything from layerManager if you're calling buildConfiguration directly.
I'm not sure why the beforeFooter callback has a this type. Callbacks typically use arrow functions which do not have a this. I used bind to get around the type error.
Your test cases were passing the wrong type of object into the callback. It should be an array of TooltipItem. Our implementation only uses the dataset property.
I used ! non-null assertions instead of ? to get around the "possibly undefined" type-check on function call. I usually avoid doing this, but I think it's ok here in unit test as long we make sure the test will fail if the function is undefined/null.
I would take out the  || '' that you used in your other test because that will make the test pass if the callback or its return value is undefined.


Bumping the version of webpack from 4.29.3 to 4.29.4

Webpack version 4.29.4 fixes the issue described here --> https://github.com/webpack/webpack/issues/8781

Closing this one because Dependabot fixed it. 👍


Participation tokens learn more needs a link

Waiting for Peter V to finish this section of the help centre first

Can link here:
https://app.gitbook.com/@augur/s/help-center/reporting-or-disputing-faq#what-are-participation-tokens

links in


PLANNER-613 Workbench: Display info label with default phases when no phases are explicitly specified in solver configuration

Depends on https://github.com/droolsjbpm/optaplanner-wb/pull/84

@mcimbora there's an additional commit in this pr which is already part of another PR. Can you please remove that commit (PLANNER-618). Thanks.

@csadilek The PR is built on the top of the additional commit (e.g. PhaseConfigFormTest.displayEmptyPhaseConfigurationLabel() uses ManagedInstance). I'd wait until PLANNER-618 gets integrated & rebase the PR.

+1


Fix binaries pipeline

I think we have to unify the when.paths options as we need all systems (server, agent, cli) to build to release them even if only the server for example changed. https://ci.woodpecker-ci.org/woodpecker-ci/woodpecker/build/1909

@anbraten did add it to the changelog pull as it's only adding two lines, rest is format ...  #736

->  #737


Actually use project variable in download script

fixes #308
@phant0m2290 if you want me to add you as co-author i would need your commit signature

fixes #308
@phant0m2290 if you want me to add you as co-author i would need your commit signature

how do i do that?

fixes #308

@phant0m2290 if you want me to add you as co-author i would need your commit signature

how do i do that? or what is that? @Leguan16

The email and user you provide to commit something to a git repo.

fixes #308

@phant0m2290 if you want me to add you as co-author i would need your commit signature

how do i do that? or what is that? @Leguan16

The email and user you provide to commit something to a git repo.

Ahh alr
user: phant0m2290
email<EMAIL_ADDRESS>

done.

fixes #308

@phant0m2290 if you want me to add you as co-author i would need your commit signature

how do i do that? or what is that? @Leguan16

The email and user you provide to commit something to a git repo.

Ahh alr user: phant0m2290 email<EMAIL_ADDRESS>
done


Update item_effect.mes minor fixes

i wanted to do statue so it will look like i also did something...
jk, its fine.


Kafka consumer support subscribe by topic,partition,offset

please merge into https://github.com/Kyligence/ClickHouse/tree/rebase_ch/20241224


NewTodoInput Autofocus future Added.

Hi Ryota, I decided to save you some time and add the autofocus #719 future for you.

@PaoloJN Thank you for your contribution!

@all-contributors please add @PaoloJN  for code


CPO 10 - Implement microservice based authorisation

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CPO-10
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

Closing as this has been implemented on a new branch and  accompanying PR
https://github.com/hmcts/cpo-case-payment-orders-api/pull/68


Picker doesn't open on iOS 8

The demos on http://amsul.ca/pickadate.js/ do not work properly on iOS 8. I am only testing on an iPad but am aware of similar issues on iPhone.
The opaque background shows but the position of the calendar is way down at the bottom, so it can mean a lot of scrolling.
iOS 8 possibly changed how it handles CSS position: fixed?

$('.myinput').pickadate({ onOpen: function() { $('.myinput').blur(); } });
also for me, this fixed the problem on iOS 8 and iPad 2

@timcreatewell, @izifortune I believe that's related to #609. As long as the picker opens, I believe this issue has been resolved...

Hi. Great work on reduced this to a fix. The CHANGELOG.md file states that this has been fixed in 3.5.5, but this version has not been tagged or published. Does it look like 3.5.5 will be released soon? Or is it waiting for other work to be completed?

@samblinkmobile @benbarclay

@jokeyrhyme the remaining issues for the 3.5.5 milestone: https://github.com/amsul/pickadate.js/milestones/3.5.5
Cheers.

Hi there @amsul, thanks so much for releasing 3.5.5!  I updated my project with the new files this morning but unfortunately (on the iOS simulator at least - iOS 8.1) I'm still seeing the same behaviour: when you choose a year/month from the drop downs (selectYears / selectMonths) the picker (using default style) still scoots to the bottom of the screen.  Or could I be missing something?

@timcreatewell I'm not sure if this is what you're experiencing, but I have noticed there are still a few layout issues with v3.5.5 on iOS. I've made an unlisted YouTube video here: http://youtu.be/2-Ha45DP2gw
@amsul it seems as though dismissing the pop-up by tapping outside it, instead of pressing the "close" button, makes it more likely (but not 100% likely) to experience weird layout issues.

I confirm I have the same issue. Ipad 3 IOS 8.2 datepicker not opening at all it opens just sometimes randomply. android all ok. I tried also the master and also the blur fix but nothing changed.

Hi @amsul, it looks like this thing was never actually fixed. This solution still seems to work on v3.5.6 as well. Not sure what the solution is here but for now I'm still just commenting out those lines.

On iPad Mini with iOS 8.0.2 using v3.5.6, I still got this issue when I try to change date.
Suggested fixes didn't work.
Inspecting the mobile DOM using a MacBook, HTML code looks fine as usual.
The picker nearly behaves like it got a wrong z-index, it's invisible but the dropdowns are touchable.
Tuning CSS was useless.
Any hints, please?

@riccardobevilacqua is the device you're testing fixed to that version of iOS?  Apple fixed some quirks with iOS 8's version of Safari in later updates.  It's probably worth updating to 8.3 and then testing.

Same behaviour on iOS 8.3


zip to hdf5 error

I got the following error while tring to convert the .zip to .hdf5 using
python -m shift_dev.io.to_hdf5 "/home/tim/AD_models/SHIFT_Dataset/SHIFT/*.zip" -j 1

The error says:
Traceback (most recent call last):
  File "/home/tim/anaconda3/envs/SHIFT/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/home/tim/anaconda3/envs/SHIFT/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/tim/AD_models/SHIFT_Dataset/shift-dev/shift_dev/io/to_hdf5.py", line 12, in <module>
    from ..utils.storage import ZipArchiveReader
  File "/home/tim/AD_models/SHIFT_Dataset/shift-dev/shift_dev/utils/storage.py", line 8, in <module>
    from pycocotools import mask as mask_utils  # type: ignore
  File "/home/tim/anaconda3/envs/SHIFT/lib/python3.8/site-packages/pycocotools/mask.py", line 3, in <module>
    import pycocotools._mask as _mask
  File "pycocotools/_mask.pyx", line 1, in init pycocotools._mask
ValueError: numpy.ndarray size changed, may indicate binary incompatibility. Expected 96 from C header, got 88 from PyObject

Hey @Alexbeast-CN! I have tried but I could not reproduce this error on my side. The problem looks like a version compatibility issue with Numpy and Pycocotools. Did you have any issue when running import pycocotools._mask alone? If so, perhaps you need to install pycocotools manually. This post (https://github.com/cocodataset/cocoapi/issues/248) may be helpful.

Thank you for your reply.  After reinstall pycocotools using the following command, it works!
pip install git+https://github.com/philferriere/cocoapi.git#subdirectory=PythonAPI


⚠ Bofrost.at has degraded performance

In 99bddbc, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2693 ms

Resolved: Bofrost.at performance has improved in 81772a1 after 313 days, 10 hours, 7 minutes.


Сonvert error

I can't convert go file to C# with error at screenshot. Please help! What I do wrong?

Can you provide the Go source file?

@ritchiecarroll I receive the same error while trying to convert the following file: https://github.com/ory/keto/blob/master/internal/relationtuple/definitions.go

Thanks for the source file and the debugging - should be relatively easy to fix.


Fixed int * float multiplication error

Fixed len call throwing error when attribute "sample_weights" is provided. Since "num_transitions" is an int value, multiplying it with a float threw an error. I simply switched it around, since "sample_weights" is either int / float and "num_transitions" is always int.

Further added the possibility to load OXE data from arbitrary data_paths. Simply add a "data_path" field to the OXE_DATASET_CONFIGS dataset.
Can also be a local path ("~" will be converted to home_dir), to allow self made tfds builds.


Added support for @DecimalMin, @DecimalMax annotations

Fix for https://github.com/mbknor/mbknor-jackson-jsonSchema/issues/79
Now @DecimalMin and @DecimalMax is supported. And seems like BigDecimal already processed.
Update
Added support for for specifying default value for decimal numbers.

tx for the PR.
It looks ok, but can you please add some tests?
Also, it would be better if you squashed all the commits into one single commit.

I wrote the tests myself.. thanks.. merged

@mbknor thanks I'm very appreciated!


New Issue -- Issue 7 not resolved - WindowsStore

I'm pretty new to this myself, but I'm finding some things that just aren't working the way they are written.
When searching for AppxProvisionedPackages, there is no attribute called "name".
There is "DisplayName" and "PackageName", but not just "Name".
So lines 22 and 23 will fail to exclude "Microsoft.WindowsStore" or "Microsoft.Photos", causing them to still be deleted.
Suggest changing them like lines 20 and 21:
    Where-Object {$_.packagename -notlike "*Microsoft.WindowsStore*"} |
    Where-Object {$_.packagename -notlike "*Microsoft.Windows.Photos*"} |

This is now fixed in the latest commit. Thank you!


Don't re-generate images of days that were already fully completed

So this generates ALL images for all previous days/years every time it runs, which is pretty slow (especially if you have a lot of previous years). And after both parts are completed, the image doesn't change anymore (unless you add more solutions), so there is no need to generate it each time. Only the new days or days where the solutions changed needs to be generated.

Good addition, thank you!  I assume sometimes you still might want to recreate them if you change some configuration in the script, but in that case you'll have to remove the images manually, which is fine by me. I'll add a comment in the readme to explain this however.

You can also just delete the cache files if you want to regenerate everything.

Good point!


[AVCaptureSession startRunning] startRunning may not be called between calls to beginConfiguration and commitConfiguration3

友盟反馈的问题：HXPHPicker  Application threw exception NSGenericException: *** -[AVCaptureSession startRunning] startRunning may not be called between calls to beginConfiguration and commitConfiguration3 HXPHPicker 0x00000001041258d4 0x1040e4000 + 268500

#76

谢谢


Typings stop working inside slugs

Describe the Bug
After bootstraping a fresh Astro project using the blog template and installing the astro-vtbot integration the typings of the slug are set to any like in the following example.
Before installing:

After installing

I've tried reloading the window on VSCode and even restarting the ts-server after installing the package but for some reason the issue persists.
What's the expected result?
The expected result is to not have typing issues 😁
Link to Minimal Reproducible Example
I don't a repro link
Participation

[ ] I am willing to submit a pull request for this issue.

Hi @dreyfus92, thank you for opening this issue. I agree with the expected result and am able to reproduce it. ;-)

I managed to let Astro forget about the types in .astro/types.d.ts that were created for the content collection.

Should be fine with 1.7.11

Confirmed by original user (@henriqueamaral)


not loading with julia 0.5

using Lasso
ERROR: LoadError: UndefVarError: wrkwt! not defined
in include_from_node1(::String) at ./loading.jl:488
in eval(::Module, ::Any) at ./boot.jl:234
in require(::Symbol) at ./loading.jl:415
while loading /home/jakub/.julia/v0.5/Lasso/src/Lasso.jl, in expression starting on line 18

GLM.jl was changed and this package needs to be tweaked to handle that change. I'm pretty busy right now but will try to take a look this weekend. Alternatively, you can roll back GLM.jl to a previous version.

Finally fixed.


[SPARK-23329][SQL] Fix documentation of trigonometric functions

What changes were proposed in this pull request?
Provide more details in trigonometric function documentations. Referenced java.lang.Math for further details in the descriptions.
How was this patch tested?
Ran full build, checked generated documentation manually

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Thanks @srowen and @HyukjinKwon for your comments so far ...

ok to test

Test build #87546 has started for PR 20618 at commit 168d7b3.

Test build #87547 has started for PR 20618 at commit 2586b0f.

Test build #87546 has finished for PR 20618 at commit 168d7b3.

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

Merged build finished. Test PASSed.

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

Test build #87547 has finished for PR 20618 at commit 2586b0f.

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

Merged build finished. Test PASSed.

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

Test build #87551 has started for PR 20618 at commit 8550a27.

Test build #87554 has started for PR 20618 at commit 40da998.

@misutoth, I am sorry. I missed https://github.com/apache/spark/pull/20618#discussion_r169291756 and discussion in the JIRA.
@srowen, I was thinking about having the same description part in other languages for now. For example,
  * Computes the hyperbolic tangent of the given value.

->
  * Computes the hyperbolic tangent of the given value.
  *
  * @param e hyperbolic angle
  * @return hyperbolic tangent computed by [[java.lang.Math#tanh]]

What do you think about not touching functions.py and functions.R with the format above even though it might duplicate a bit for now?
My only concern is to have different descriptions in other languages APIs and I am a little bit worried of when we come to fix Python's and R's.

Test build #87551 has finished for PR 20618 at commit 8550a27.

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

Consistency is good. You're suggesting updating Python, R separately? If it's a big change, I could see breaking it up, but could also imagine adjusting that here. I am OK with getting this in first as I know we've iterated on this a lot.

Merged build finished. Test PASSed.

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

Maybe I am too much caring about this and It guess it wouldn't be a big change; however, I was thinking there might be subtle nits, like, referring java.lang.Math* in Python side and R side which is not preferred up to my knowledge.
Just wanted to avoid another iteration for finding a better description for both Python and R again here. (Just noticed it's his very first contribution to Spark too.).
Seems fine for SQL / Scala / Java as they already use a fully qualified class name, for example, in case of SQL:
spark-sql> DESCRIBE FUNCTION EXTENDED cos;
...
Function: cos
Class: org.apache.spark.sql.catalyst.expressions.Cos
...

Test build #87554 has finished for PR 20618 at commit 40da998.

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

Merged build finished. Test PASSed.

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

Looking back I guess we can expect a couple of comments on R and Python side too, though I will target a lower number of them. :) So I am a little bit favoring moving functions.* update into a separate ticket.

@HyukjinKwon I think it's fine to refer to java.lang.Math even from Python, R documents. It's just expressing the API's behavior, and that contract is the same regardless of the language. (Or is there a subtlety I'm missing, like, it's computed different in Python or the behavior of things like NaN isn't the same?) (Or is your point just that you can't easily write hyperlinks to the JDK docs in Python? that's fine, no need for a link.)

I just thought about this again and yup I am fine. @misutoth shall we just fix R and Python ones too here? I think we could just target to fix the current functions in other language's API.

Sure, lets do that, no problem.

Was the result to add more changes to this PR or add them in another PR?

Also cc @srinathshankar

Sorry, I missed these comments. As I understood we fix all of them here. I am just struggling with the R documentation: it seems the generated doc is incorrect even if I just take the latest commit as it is. I thought about writing to the dev list to see if others have experienced this.

cc @felixcheung (I saw you and Felix in dev mailing list). So, https://github.com/apache/spark/tree/master/R#generating-documentation does not work?

@misutoth what exactly is the problem you are running into?

@felixcheung, I have started a mail thread on<EMAIL_ADDRESS>with title Help needed in R documentation generation because I did not feel it is directly related to this PR. Thanks for your reply on this thread already.

As discussed in email R documentation is reorganized and math functions are grouped as part of SPARK-20889. Because of this grouping I dont think this change is really applicable on R. @srowen, @HyukjinKwon, @felixcheung  what do you think about it?

Test build #87796 has started for PR 20618 at commit 0e4e15a.

Test build #87796 has finished for PR 20618 at commit 0e4e15a.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

I included java.lang.Math references in functions.R

Test build #87798 has started for PR 20618 at commit fcf1338.

Test build #87798 has finished for PR 20618 at commit fcf1338.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #87803 has started for PR 20618 at commit 2ea1f18.

Test build #87803 has finished for PR 20618 at commit 2ea1f18.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #87860 has started for PR 20618 at commit 10afda5.

Test build #87860 has finished for PR 20618 at commit 10afda5.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #87883 has started for PR 20618 at commit 10afda5.

Test build #87883 has finished for PR 20618 at commit 10afda5.

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

Merged build finished. Test PASSed.

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

Test build #87914 has started for PR 20618 at commit 627e204.

Test build #87914 has finished for PR 20618 at commit 627e204.

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

Merged build finished. Test PASSed.

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

Merged to master and branch-2.3.


Plasticity sometimes converges to zero displacement

This can be reproduced by running a plastic calculation on https://contact.engineering/go/867nv with a hardness of 0.05

This is a bug in the implementation on the TopoBank end, not in ContactMechanics. The pipeline function for contact (see #44) has been adapted from TopoBank and the bug has been fixed there.


Windows ?

This is a really nice apps but a version for Windows is possible pls ???
Thaks a lot ;)

Hi @Benjiiben53.
Unfortunately, it would be pretty difficult to port this to Windows.
The current Linux implementation relies on a module built into PulseAudio to create a virtual audio source. Doing something similar on Windows would require writing an audio driver and getting it signed by Microsoft.
I didn't find any free and open-source equivalents on Windows. The closest proprietary alternative I found is WO Mic, but I have no idea whether it works as advertised. Because it is closed-source, I cannot guarantee it isn't malicious.
Best regards.

Tried via WSL, unsuccessful so far.

Hi @Benjiiben53.
Unfortunately, it would be pretty difficult to port this to Windows.
The current Linux implementation relies on a module built into PulseAudio to create a virtual audio source. Doing something similar on Windows would require writing an audio driver and getting it signed by Microsoft.
I didn't find any free and open-source equivalents for Windows. The closest proprietary alternative I could find is WO Mic, but I have no idea whether it works as advertised. And because it is closed-source, I cannot guarantee it doesn't do anything malicious.
Best regards.

Of course, thanks anyway ;)

I found pgaskin/pulseaudio-win32, didn't try it yet though.


How can I build with Xcode12 carthage error

version: Moya 14.0
macOs: 10.15.6
Task failed with exit code 1:

hey @WUYUEZONG, did you solve the issue?

@sunshinejr
I can build it now.
by
Carthage issue
BTW,
And a easy way
but, it also happens swift complier  error

@WUYUEZONG awesome, glad you got it resolved - thanks for the update!

I have tried the workaround page, and that worked for me.


Non pass holders can have parking passes associated with them

should be an easy fix. Parking pass's foreign key needs to reference pass_holder instead of customer.

I am closing this because I already created an issue for this.


feat(telemetry): add server version

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Can we also add these for the events that are sent by react and proxied through gms?

Yeah I can add these to the tracking controller stuff too


Layout spacing changes design specs

These changes apply to the AEM Learn template, but will also apply to the Solution template.
Additional Information
Summary of changes:

Update spacing below section headers from 48px to 32px.
Update 2nd-tier spacing from 96px to 64px.

AEM provided documentation:
Visual reference of spacing updates
ExperienceGuild-Learn Research.pptx (slide 3)
ExperienceGuild-Learn Template User Testing (slide 6)
Acceptance Criteria

[ ] A Sketch file is created or updated with the Layout spacing design and specs changes
[ ] The design specs and pdf version are added to the Box folder
[ ] Update the Agile Board epic with the spec link
[ ] Add Box folder link to the website content if it exists

I forgot this issue was created and made a new one 😬 #5376. So this can be closed as it's duplicated.


Don't preview binary files

Maybe the preview window just indicate that the file is binary instead.

Sometimes it leaves a floating artifact

.

https://github.com/ibhagwan/fzf-lua/commit/28f3bcac27b6dbab37102336a22a38b81a6c4089
Should work on every system where the file binary exists.


ALPN documentation could use some clarifications

Title: ALPN documentation could use some clarifications
Description:
The current documentation for https://www.envoyproxy.io/docs/envoy/latest/api-v3/extensions/transport_sockets/tls/v3/tls.proto.html?highlight=upstreamtlscontext#extensions-transport-sockets-tls-v3-commontlscontext alpn_protocols seems to be missing some information
The doc mentions:

Supplies the list of ALPN protocols that the listener should expose.

However, this field is also present for UpstreamTlsContext in cluster, where the behavior isn't explained.
Additionally, as far as I can tell Envoy is not actually doing an negotiation of application protocol to use in the cluster (and instead uses ProtocolSelection). Per https://tools.ietf.org/html/rfc7301#section-3.2, I would have expected a mismatch of UpstreamTlsContext and DownstreamTlsContext alpn_protocols to result in an error while it seems envoy does not. If this is intentionally, it would be useful to have a note in the docs about this.

cc. @lizan

Isn't this a dup of https://github.com/envoyproxy/envoy/issues/3431? In short, upstream ALPN isn't supported now and there's ongoing work #14239

Additionally, as far as I can tell Envoy is not actually doing an negotiation of application protocol to use in the cluster (and instead uses ProtocolSelection). Per https://tools.ietf.org/html/rfc7301#section-3.2, I would have expected a mismatch of UpstreamTlsContext and DownstreamTlsContext alpn_protocols to result in an error while it seems envoy does not. If this is intentionally, it would be useful to have a note in the docs about this.

I didn't fully get this part, are you saying Envoy listener isn't sending  a fatal "no_application_protocol" alert. when client advertises protocol it doesn't support?

So the field doesn't do anything whatsoever in the upstreamtlscontext?
What I have seen is my upstream and downstream alpn do not match but I don't get
a fatal "no_application_protocol" alert, but if upstream tls context does nothing maybe that explains it.


API for OpenSSL

@jhass , better ?

I'll be more harsh: drop the security tuning examples completely (ciphers, options, verify mode), except for specifying the keys; document that we do our best to define good defaults (Mozilla Intermediate) in the stdlib with a link to the Mozilla wiki page for more details, along with encouragements to notify us / send a pull request if we get out-of-date, have ciphers enabled that shouldn't be, etc.
Note that we don't exactly follow the Mozilla Intermediate configuration, for example we don't configure the secp384r1 and secp521r1 curves. If someone knows how to enable/initialize them, please step in!

@ysbaddaden I just did , removed all security tuning.

@jhass and @ysbaddaden .
I added more info regarding the Crystal supplying the sane defaults, also that if any issues are found in the choices we made, an issue should be opened

I would encourage opening a crystal-security mailing list so that security issues could be notified secretly and remain unpublished until they are fixed.

@ysbaddaden cool ?

@ysbaddaden @jhass
Is this ok from your end ? or more changes needed ?

@Sija , done.

@ysbaddaden @jhass @Sija
So ... are we done ? can this be merged ?

Looks OK to me.

LGTM

Build has failed on x86-linux. Restarted.


Can you customize error messages?

My test framework is set up to mostly handle log messages so that they make it clear what failed. However, the one failure that always make me go hunting is when any element cannot be found.
So I am sending a POST like:
{"using":"accessibility id","value":"MainLogo"}
Since that element was errantly taken out, it does not exist on the page currently, so I get back:
{"status":7,"value":{"error":"no such element","message":"An element could not be located on the page using the given search parameters."}
What I would like is for the value of my POST to be a part of my error message, so that I know WHICH element could not be found, and don't have to spend time debugging to find out what broke.

I second this request.  It would be great if the existing message was improved to indicate what property and value were searched for.


findbugs: deal with all the encoding issues in a unified way

further getBytes() calls can
getBytes(StringUtils.getPrefferedCharset()) instead

since its a try-with-resources Statement, it will get closed

I seem to have messed up here and put two unrelated commits in one pr. @karuturi, you want me to clean up my mess or can we merge it like this?

@DaanHoogland looks like you split the commit :) (the combined one was also ok. )
I see some checkstyle errors in the jenkins build
[INFO] --- maven-checkstyle-plugin:2.11:check (cloudstack-checkstyle) @ cloud-utils ---
[INFO] Starting audit...
/home/jenkins/jenkins-slave/workspace/cloudstack-pull-requests/utils/src/com/cloud/utils/StringUtils.java:36:28: Name 'acsCharset' must match pattern '^(s)?[a-z][a-zA-Z0-9]*$'.
Audit done.

@DaanHoogland was this committed to master?

yes this is on master. is there doubt?

@DaanHoogland some weird confusion, nevermind


Provide a smart entrypoint to automatize most operations

In order to easily use this docker image in a production environment, it would need to have generic entrypoint that could manage the most (if not all) the operations of the README automatically.
I might be able to provide a PR for that.

Sounds great, a PR for that would be nice!

For info, I developed a docker image for my company that provide this feature: https://github.com/Monogramm/docker-nominatim
We've been testing it for some weeks now and it works pretty well.
As soon as I find the time.... I will try to send a PR that match this official image.

We've managed to provide that now with https://github.com/mediagis/nominatim-docker/pull/183, see https://github.com/mediagis/nominatim-docker/blob/master/3.6/README.md. Thanks for your input


Why am I being ostracized?

If you are experienced in writing specifications, you're also welcome to join the effort.

So if I have no experience, I am not welcomed in this project and barred from getting the experience?

regarding the license: the current license does not actually imply
open-ness at all; because it is a no-derivatives license - that is a
real problem and most want to see it changed to something truly open;
but that has not happened yet
regarding participation: it actually was discussed, whether or not to
close this github issue tracker; but it was decided to leave it open
and the github README clearly states:
Non-members may use the GitHub issue tracker or send us a mail ...

that sentence was added specifically to allow everyone to participate in
the discussions, both on github and on the mailing list; so absolutely
no one is being "ostracized" as this issue title so bluntly claims -
such confrontational insinuations are not productive - "you
catch more flies with honey than with vinegar", as the saying goes -
there are an open issues on this github issue tracker each corresponding
directly to each thread in the formal technical discussions -
absolutely every person on the planet is "encouraged" to express their
opinion; and there is nothing in the github README that would indicate
to the contrary
that README was written before the discussion began over six months ago

for the most part, the discussion topics were exhausted some
months ago - it is quite clear now which features the spec needs to
accommodate; and the work remaining to be done is to codify the spec
and to work on implementations - the first reference implementation is
nearly completed now

regarding the specific language of the README itself: there is an
alternate repo on notabug that includes a revised README - i expect that
you will find the language in that one to be more plain and less
emotionally charged; but because of the current licensing situation,
the github repo has not been updated in some time
https://notabug.org/peers/forgefed/

oh, as an update for the benefit of github subscribers, i should also
add that a new IRC channel was created this week for the work related
to the reference implementation (#vervis on freenode) - that too, is
open to everyone

You mentioned several discussions and implied there are other issue trackers, repositories, etc. Can you provide links to useful, active, and open tools being used by this project? In addition to the one on notabug, I also found related information on dokk.org and framagit.org
Also, I can't tell or not, because the internet never does a good job and conveying feelings, but if you're peeved by my wording I apologize. I know you feel that it was confrontational and counter-productive, and from your perspective, I can respect that. However, from my perspective, as someone who has only known of this project for less than 48 hours, your response has been a treasure trove of information. So thank you for that. I look forward to participating further if you'll have me.

np bud - i have no desire to harp on any personal concerns on the
internet - i will just say that is was the specific words you used that
came across as inflammatory or more provocative than necessary -
the word "ostracized" is not even applicable in this context - what you
really meant was "why are inexperienced people excluded from the formal
technical discussion" (or better yet: "why are the formal technical
discussions conducted only by experienced people") - when stated that
way, the reason should be obvious; and is less likely to be interpreted
as an accusation
as i mentioned, most of the important technical discussions have run
their course; so the lack of activity on the mailing list is not an
indication of a lack of interest or progress - for the most part, there
is just not much to discuss at this stage, thats why the project may
appear to be dormant - the previous discussions are readable on the
framalistes website - that would be a great place to begin, catching up
on what was discussed already
https://framalistes.org/sympa/arc/git-federation
the most activity and discussion recently have been in the #peers IRC
channel on freenode and has been more general than technical - thats
essentially the only "community" channel for now - not for any
particular reason; but because folks in the peers group have been
interested in something like even before this github repo was created -
perhaps someday, forge-fed will have it's own website,, forum, or
fediverse instance, or whatever; but if you want to "connect" with an
interested community, thats probably all i can suggest for now
the #vervis channel i mentioned is more specifically for working on the
vervis forge, which we are treating as a reference implementation to
work the bugs out of the activity-pub messaging logistics - once it is
clear what the form of that looks like, some documentation can start
being written in tandem with the reference implementation
so, for now, neither the github repo or the notabug repo have much
in them; but that is where the written documentation and any support
libraries will eventually go - the two repos are out of sync now mostly
for the reason that there is not much concrete to show - it is hoped
that both repos will be mirrors in sync; but that is not very
important until once the artifacts begin taking shape
https://notabug.org/peers/forgefed/


Add formatted date from timestamp

@tgilan @heylookltsme

My only thought is maybe it should be optional? But honestly, I can't imagine anyone would prefer timestamps to dates...
:+1:


Performance improvement: directly return IQueryable<TUser/TRole>

Performance improvement: directly return IQueryable and IQueryable from the respective method thus avoiding the need to download all the entities each time.

Hi Matteo! Thanks a lot for this useful and unique lib. Wanted to offer some help to make it even better. Thanks!