"Icon URL Unreachable" error but URL is reachable (while validating manifest.xml)

Expected behavior
I expect the "Icon URL Unreachable" error to not be shown because the URL is reachable.
Current behavior
Running npm run validate -- -p fails with the following error:
~/Desktop/WinShared/Saas Cap Data main*
❯ npm run validate -- -p

><EMAIL_ADDRESS>validate
> office-addin-manifest validate manifest.xml -p

Validation Information:
Package Type Identified: Package of your add-in was parsed successfully.

Correct Package: Your package matches the submission type.

Valid Manifest Schema: Your manifest does adhere to the current set of XML schema definitions for Add-in manifests.

Manifest Version Correct Structure: The manifest version number has the correct structure for the platform that it supports.

Manifest Version Correct Value: The manifest version number is greater or equal to 1.0.

Manifest ID Valid Prefix: The product ID in the manifest has a valid prefix
  - Details: d348c7c2-36c5-49cc-a8a2-ea185b8427ec

Manifest ID Correct Structure: The structure of the product ID is correct.
  - Details: d348c7c2-36c5-49cc-a8a2-ea185b8427ec

Desktop Source Location Present: A desktop or default source location URL is found.

Secure Desktop Source Location: The manifest desktop source location URLs use HTTPS.

The manifest source location URLs are valid.: The manifest source location URLs are valid.

Supported Office Identified: Supported Office products were successfully determined.

Support URL Present: The manifest support URL is present.
  - Details: https://crcastle.github.io/saascap-data/assets/help/

Valid and reachabe Support URL: The manifest support URL is valid and reachable.
  - Details: https://crcastle.github.io/saascap-data/assets/help/

Valid OnlineMeetingCommandSurface ExtensionPoint.: OnlineMeetingCommandSurface ExtensionPoint extracted from manifest is found to be valid.

Valid Manifest Resource Urls: The manifest Resource URLs are valid.

Valid Manifest AppDomain Urls: The manifest AppDomain URLs are valid.

High Resolution Icon Present: A high resolution icon element was expected and is present.
  - Details: https://crcastle.github.io/saascap-data/assets/saascap-icon-64.png

Supported High Resolution Icon URL File Extension: The manifest high resolution icon URL has a valid image file extension.
  - Details: png

Secure High Resolution Icon URL: The manifest high resolution icon URL uses HTTPS.
  - Details: https://crcastle.github.io/saascap-data/assets/saascap-icon-64.png

Correct High Resolution Icon Dimensions: The manifest high resolution icon dimensions are correct.

Icon Present: A icon element was expected and is present.
  - Details: https://crcastle.github.io/saascap-data/assets/saascap-icon-32.png

Supported Icon URL File Extension: The manifest icon URL has a valid image file extension.
  - Details: png

The manifest icon URL uses HTTPS.: Secure Icon URL
  - Details: https://crcastle.github.io/saascap-data/assets/saascap-icon-32.png

All GetStarted strings are present in Resources: All GetStarted strings are present in Resources

Acceptance Test Completed: Acceptance test service has finished checking provided add-in.

Error #1:
Icon URL Unreachable: Unable to retrieve an image from the icon URL.
  - Details: The IconURL supplied in the app manifest is not reachable, IconURL : https://crcastle.github.io/saascap-data/assets/saascap-icon-32.png.

The manifest is not valid.

~/Desktop/WinShared/Saas Cap Data main* 16s
❯❯

However, accessing https://crcastle.github.io/saascap-data/assets/saascap-icon-32.png from both curl or a browser show that the PNG file is in fact reachable.
Here it is in-line in this GH issue pulled from that URL that the validation says is unreachable:

curl:
~/Desktop/WinShared/Saas Cap Data main*
❯ curl -v https://crcastle.github.io/saascap-data/assets/saascap-icon-32.png
*   Trying <IP_ADDRESS>:443...
* Connected to crcastle.github.io (<IP_ADDRESS>) port 443
* ALPN: curl offers h2,http/1.1
* (304) (OUT), TLS handshake, Client hello (1):
*  CAfile: /etc/ssl/cert.pem
*  CApath: none
* (304) (IN), TLS handshake, Server hello (2):
* (304) (IN), TLS handshake, Unknown (8):
* (304) (IN), TLS handshake, Certificate (11):
* (304) (IN), TLS handshake, CERT verify (15):
* (304) (IN), TLS handshake, Finished (20):
* (304) (OUT), TLS handshake, Finished (20):
* SSL connection using TLSv1.3 / AEAD-CHACHA20-POLY1305-SHA256
* ALPN: server accepted h2
* Server certificate:
*  subject: C=US; ST=California; L=San Francisco; O=GitHub, Inc.; CN=*.github.io
*  start date: Feb 21 00:00:00 2023 GMT
*  expire date: Mar 20 23:59:59 2024 GMT
*  subjectAltName: host "crcastle.github.io" matched cert's "*.github.io"
*  issuer: C=US; O=DigiCert Inc; CN=DigiCert TLS RSA SHA256 2020 CA1
*  SSL certificate verify ok.
* using HTTP/2
* [HTTP/2] [1] OPENED stream for https://crcastle.github.io/saascap-data/assets/saascap-icon-32.png
* [HTTP/2] [1] [:method: GET]
* [HTTP/2] [1] [:scheme: https]
* [HTTP/2] [1] [:authority: crcastle.github.io]
* [HTTP/2] [1] [:path: /saascap-data/assets/saascap-icon-32.png]
* [HTTP/2] [1] [user-agent: curl/8.4.0]
* [HTTP/2] [1] [accept: */*]
> GET /saascap-data/assets/saascap-icon-32.png HTTP/2
> Host: crcastle.github.io
> User-Agent: curl/8.4.0
> Accept: */*
>
< HTTP/2 200
< server: GitHub.com
< content-type: image/png
< permissions-policy: interest-cohort=()
< last-modified: Wed, 14 Feb 2024 22:19:51 GMT
< access-control-allow-origin: *
< etag: "65cd3c87-14c"
< expires: Wed, 14 Feb 2024 22:53:12 GMT
< cache-control: max-age=600
< x-proxy-cache: MISS
< x-github-request-id: 0E06:37212D:FD2A0:128BE7:65CD41FE
< accept-ranges: bytes
< date: Wed, 14 Feb 2024 22:43:53 GMT
< via: 1.1 varnish
< age: 41
< x-served-by: cache-bfi-krnt7300101-BFI
< x-cache: HIT
< x-cache-hits: 1
< x-timer: S1707950634.505313,VS0,VE2
< vary: Accept-Encoding
< x-fastly-request-id: e196149c71e2f208dd1c6b0aed3afd5866d61f0d
< content-length: 332
<
Warning: Binary output can mess up your terminal. Use "--output -" to tell
Warning: curl to output it to your terminal anyway, or consider "--output
Warning: <FILE>" to save to a file.
* Failure writing output to destination
* Connection #0 to host crcastle.github.io left intact

~/Desktop/WinShared/Saas Cap Data main*
❯❯

Note that npm run validate does say the manifest is valid. However, I need npm run validate -- -p to be successful in order to submit my add-in to the Partner Center for listing in AppSource.
Also, I receive the same error when I upload this manifest.xml file to the Partner Center (screenshot) to submit the add-in to be published in AppSource.
Steps to Reproduce
Please provide detailed steps for reproducing the issue.

Run npm run validate -- -p with the this manifest.xml file in the current directory
Open a browser and paste in https://crcastle.github.io/saascap-data/assets/saascap-icon-32.png to see that the PNG is in fact reachable.

Context
Please provide any relevant information about your setup. This is important in case the issue is not reproducible except for under certain conditions.

Operating System: MacOS Sonoma 14.3.1
Node version: v20.10.0
Office version: 16.82 (24021116)
Tool version: From package.json: "office-addin-manifest": "^1.12.3",

Here is a link to the manifest.xml file I'm using: https://gist.github.com/crcastle/a0a241b2ce87b5067cf288549b6be879
Failure Logs
See above.

Did some research into how the manifest is validated when the -p flag is used. From these lines in the code for the office-addin-manifest package, I see that a web service at https://validationgateway.omex.office.net/package/api/check is being used to validate the manifest.
I replicated the request using curl and get the same error, so it seems that the issue here is with this service. Can you ask the team or person responsible for this web service if the error I'm seeing is the intended behavior?
Here is the output of the request using curl:
❯ curl -s -X POST 'https://validationgateway.omex.office.net/package/api/check?clientId=Default' \
      -H "Content-Type: application/xml" \
      --data-binary<EMAIL_ADDRESS>{"utcTimestamp":"2024-02-15T03:18:48.0571929+00:00","status":"Rejected","errors":[{"id":"1f9a991b-be34-49c5-939a-46b5170d7dbf","title":"Icon URL Unreachable","content":"Unable to retrieve an image from the icon URL.","code":"The IconURL supplied in the app manifest is not reachable, IconURL : https://crcastle.github.io/saascap-data/assets/saascap-icon-32.png."}],"warnings":[],"notes":[{"id":"c5506d85-f984-41dd-ab09-78fd9f584de9","title":"Package Type Identified","content":"Package of your add-in was parsed successfully."},{"id":"0bf7e171-217e-40a1-98ad-1f0eaed9098b","title":"Correct Package","content":"Your package matches the submission type."},{"id":"8cf46fb0-8901-49c0-b3c2-15c3114425b4","title":"Valid Manifest Schema","content":"Your manifest does adhere to the current set of XML schema definitions for Add-in manifests."},{"id":"a95552d7-e476-4dd9-a58d-dcf4a246e7af","title":"Manifest Version Correct Structure","content":"The manifest version number has the correct structure for the platform that it supports."},{"id":"587a65ee-44a8-4412-9ccb-73201a8bf5a1","title":"Manifest Version Correct Value","content":"The manifest version number is greater or equal to 1.0."},{"id":"f7bca1c6-051e-405a-88bf-5df8d263f3f0","title":"Manifest ID Valid Prefix","content":"The product ID in the manifest has a valid prefix","code":"d348c7c2-36c5-49cc-a8a2-ea185b8427ec"},{"id":"69e8d286-0a0b-44af-8a41-0193391478f0","title":"Manifest ID Correct Structure","content":"The structure of the product ID is correct.","code":"d348c7c2-36c5-49cc-a8a2-ea185b8427ec"},{"id":"8968ae09-0f70-49b5-a839-5410639f65e5","title":"Desktop Source Location Present","content":"A desktop or default source location URL is found."},{"id":"19192530-7680-4f18-a68a-8bd07b656422","title":"Secure Desktop Source Location","content":"The manifest desktop source location URLs use HTTPS."},{"id":"cec06602-b4a7-40e4-aa50-a13a29a90584","title":"The manifest source location URLs are valid.","content":"The manifest source location URLs are valid."},{"id":"69150951-4963-4c4b-ae6b-58c07160e673","title":"Supported Office Identified","content":"Supported Office products were successfully determined."},{"id":"64f19c25-db3f-445a-8af8-a8dc02701d14","title":"Support URL Present","content":"The manifest support URL is present.","code":"https://crcastle.github.io/saascap-data/assets/help/"},{"id":"abd58ba2-327b-48b7-ad18-c567f963d859","title":"Valid and reachabe Support URL","content":"The manifest support URL is valid and reachable.","code":"https://crcastle.github.io/saascap-data/assets/help/"},{"id":"38e76e9c-314f-4759-b3fb-3afa252ce0c6","title":"Valid OnlineMeetingCommandSurface ExtensionPoint.","content":"OnlineMeetingCommandSurface ExtensionPoint extracted from manifest is found to be valid."},{"id":"987201dd-ea13-4dc3-9ead-190f6ea3fb36","title":"Valid Manifest Resource Urls","content":"The manifest Resource URLs are valid."},{"id":"145e9494-a40d-43aa-98fd-bbb76b430d2f","title":"Valid Manifest AppDomain Urls","content":"The manifest AppDomain URLs are valid."},{"id":"c658985f-8366-4495-9b82-0a534450c406","title":"High Resolution Icon Present","content":"A high resolution icon element was expected and is present.","code":"https://crcastle.github.io/saascap-data/assets/saascap-icon-64.png"},{"id":"718c518d-b110-4d7e-b9a5-d8430e94c96b","title":"Supported High Resolution Icon URL File Extension","content":"The manifest high resolution icon URL has a valid image file extension.","code":"png"},{"id":"79b5a5c9-cb79-40c2-b584-436ab550f030","title":"Secure High Resolution Icon URL","content":"The manifest high resolution icon URL uses HTTPS.","code":"https://crcastle.github.io/saascap-data/assets/saascap-icon-64.png"},{"id":"1ecd4300-6e81-4fbb-9437-499a4c725df9","title":"Correct High Resolution Icon Dimensions","content":"The manifest high resolution icon dimensions are correct."},{"id":"1ba2f086-5f1b-401d-8ad6-482cb6526616","title":"Icon Present","content":"A icon element was expected and is present.","code":"https://crcastle.github.io/saascap-data/assets/saascap-icon-32.png"},{"id":"94330b32-277b-4292-9655-8acbc3ba6fd9","title":"Supported Icon URL File Extension","content":"The manifest icon URL has a valid image file extension.","code":"png"},{"id":"07780e16-8979-4fb8-b1c2-da8cfce1ed0c","title":"The manifest icon URL uses HTTPS.","content":"Secure Icon URL","code":"https://crcastle.github.io/saascap-data/assets/saascap-icon-32.png"},{"id":"99e2d71a-d2cb-41d1-949e-6cf7eeaede99","title":"All GetStarted strings are present in Resources","content":"All GetStarted strings are present in Resources"},{"id":"906eba01-3ef7-44d9-8b44-c606b4fb9f55","title":"Acceptance Test Completed","content":"Acceptance test service has finished checking provided add-in."}],"addInDetails":{"adminInstallOnly":false,"capabilities":[{"code":"ReadWriteDocument","title":"Can read and make changes to your document"},{"code":"SendReceiveData","title":"Can send data over the Internet"}],"defaultLocale":"en-US","description":"SaaS Capital Excel Tools","displayName":"SaaS Cap Data","highResolutionIconUrl":"https://crcastle.github.io/saascap-data/assets/saascap-icon-64.png","hosts":["Workbook"],"iconUrl":"https://crcastle.github.io/saascap-data/assets/saascap-icon-32.png","localizedDescriptions":{"en-US":"SaaS Capital Excel Tools"},"localizedDisplayNames":{"en-US":"SaaS Cap Data"},"localizedHighResolutionIconUrls":{"en-US":"https://crcastle.github.io/saascap-data/assets/saascap-icon-64.png"},"localizedIconUrls":{"en-US":"https://crcastle.github.io/saascap-data/assets/saascap-icon-32.png"},"localizedRootSourceUrls":{"en-US":"https://crcastle.github.io/saascap-data/taskpane.html"},"productId":"d348c7c2-36c5-49cc-a8a2-ea185b8427ec","providerName":"SaaS Capital","requirements":"<Requirements xmlns=\"http://schemas.microsoft.com/office/appforoffice/1.1\"><Sets DefaultMinVersion=\"1.1\"><Set Name=\"SharedRuntime\" MinVersion=\"1.1\" /></Sets></Requirements>","rootSourceUrl":"https://crcastle.github.io/saascap-data/taskpane.html","subtype":"TaskPane","supportedProducts":[{"code":"Mac_Excel","title":"Excel 2019 or later on Mac","version":"<IP_ADDRESS>"},{"code":"Win32_Excel","title":"Excel on Windows (Microsoft 365)","version":"16.0.12527.20092"},{"code":"WAC_Excel","title":"Excel on the web","version":"16.0"},{"code":"Mac_Excel","title":"Excel on Mac (Microsoft 365)","version":"<IP_ADDRESS>"}],"supportedLanguages":["en-US"],"type":"ClientInstalled","version":"<IP_ADDRESS>","autorunLaunchEvents":[]}}⏎
~/Desktop/WinShared/Saas Cap Data main*
❯

And if I format that JSON response in a more human-readable way, I get this. Note the error at the top of the JSON matches the error I reported when I opened this issue. But again, I don't understand the error because the URL is accessible, returns a 200 response, and returns a 32x32 PNG image.
{
  "utcTimestamp": "2024-02-15T03:17:39.9340609+00:00",
  "status": "Rejected",
  "errors": [
    {
      "id": "1f9a991b-be34-49c5-939a-46b5170d7dbf",
      "title": "Icon URL Unreachable",
      "content": "Unable to retrieve an image from the icon URL.",
      "code": "The IconURL supplied in the app manifest is not reachable, IconURL : https://crcastle.github.io/saascap-data/assets/saascap-icon-32.png."
    }
  ],
  "warnings": [],
  "notes": [
    {
      "id": "c5506d85-f984-41dd-ab09-78fd9f584de9",
      "title": "Package Type Identified",
      "content": "Package of your add-in was parsed successfully."
    },
    {
      "id": "0bf7e171-217e-40a1-98ad-1f0eaed9098b",
      "title": "Correct Package",
      "content": "Your package matches the submission type."
    },
    {
      "id": "8cf46fb0-8901-49c0-b3c2-15c3114425b4",
      "title": "Valid Manifest Schema",
      "content": "Your manifest does adhere to the current set of XML schema definitions for Add-in manifests."
    },
    {
      "id": "a95552d7-e476-4dd9-a58d-dcf4a246e7af",
      "title": "Manifest Version Correct Structure",
      "content": "The manifest version number has the correct structure for the platform that it supports."
    },
    {
      "id": "587a65ee-44a8-4412-9ccb-73201a8bf5a1",
      "title": "Manifest Version Correct Value",
      "content": "The manifest version number is greater or equal to 1.0."
    },
    {
      "id": "f7bca1c6-051e-405a-88bf-5df8d263f3f0",
      "title": "Manifest ID Valid Prefix",
      "content": "The product ID in the manifest has a valid prefix",
      "code": "d348c7c2-36c5-49cc-a8a2-ea185b8427ec"
    },
    {
      "id": "69e8d286-0a0b-44af-8a41-0193391478f0",
      "title": "Manifest ID Correct Structure",
      "content": "The structure of the product ID is correct.",
      "code": "d348c7c2-36c5-49cc-a8a2-ea185b8427ec"
    },
    {
      "id": "8968ae09-0f70-49b5-a839-5410639f65e5",
      "title": "Desktop Source Location Present",
      "content": "A desktop or default source location URL is found."
    },
    {
      "id": "19192530-7680-4f18-a68a-8bd07b656422",
      "title": "Secure Desktop Source Location",
      "content": "The manifest desktop source location URLs use HTTPS."
    },
    {
      "id": "cec06602-b4a7-40e4-aa50-a13a29a90584",
      "title": "The manifest source location URLs are valid.",
      "content": "The manifest source location URLs are valid."
    },
    {
      "id": "69150951-4963-4c4b-ae6b-58c07160e673",
      "title": "Supported Office Identified",
      "content": "Supported Office products were successfully determined."
    },
    {
      "id": "64f19c25-db3f-445a-8af8-a8dc02701d14",
      "title": "Support URL Present",
      "content": "The manifest support URL is present.",
      "code": "https://crcastle.github.io/saascap-data/assets/help/"
    },
    {
      "id": "abd58ba2-327b-48b7-ad18-c567f963d859",
      "title": "Valid and reachabe Support URL",
      "content": "The manifest support URL is valid and reachable.",
      "code": "https://crcastle.github.io/saascap-data/assets/help/"
    },
    {
      "id": "38e76e9c-314f-4759-b3fb-3afa252ce0c6",
      "title": "Valid OnlineMeetingCommandSurface ExtensionPoint.",
      "content": "OnlineMeetingCommandSurface ExtensionPoint extracted from manifest is found to be valid."
    },
    {
      "id": "987201dd-ea13-4dc3-9ead-190f6ea3fb36",
      "title": "Valid Manifest Resource Urls",
      "content": "The manifest Resource URLs are valid."
    },
    {
      "id": "145e9494-a40d-43aa-98fd-bbb76b430d2f",
      "title": "Valid Manifest AppDomain Urls",
      "content": "The manifest AppDomain URLs are valid."
    },
    {
      "id": "c658985f-8366-4495-9b82-0a534450c406",
      "title": "High Resolution Icon Present",
      "content": "A high resolution icon element was expected and is present.",
      "code": "https://crcastle.github.io/saascap-data/assets/saascap-icon-64.png"
    },
    {
      "id": "718c518d-b110-4d7e-b9a5-d8430e94c96b",
      "title": "Supported High Resolution Icon URL File Extension",
      "content": "The manifest high resolution icon URL has a valid image file extension.",
      "code": "png"
    },
    {
      "id": "79b5a5c9-cb79-40c2-b584-436ab550f030",
      "title": "Secure High Resolution Icon URL",
      "content": "The manifest high resolution icon URL uses HTTPS.",
      "code": "https://crcastle.github.io/saascap-data/assets/saascap-icon-64.png"
    },
    {
      "id": "1ecd4300-6e81-4fbb-9437-499a4c725df9",
      "title": "Correct High Resolution Icon Dimensions",
      "content": "The manifest high resolution icon dimensions are correct."
    },
    {
      "id": "1ba2f086-5f1b-401d-8ad6-482cb6526616",
      "title": "Icon Present",
      "content": "A icon element was expected and is present.",
      "code": "https://crcastle.github.io/saascap-data/assets/saascap-icon-32.png"
    },
    {
      "id": "94330b32-277b-4292-9655-8acbc3ba6fd9",
      "title": "Supported Icon URL File Extension",
      "content": "The manifest icon URL has a valid image file extension.",
      "code": "png"
    },
    {
      "id": "07780e16-8979-4fb8-b1c2-da8cfce1ed0c",
      "title": "The manifest icon URL uses HTTPS.",
      "content": "Secure Icon URL",
      "code": "https://crcastle.github.io/saascap-data/assets/saascap-icon-32.png"
    },
    {
      "id": "99e2d71a-d2cb-41d1-949e-6cf7eeaede99",
      "title": "All GetStarted strings are present in Resources",
      "content": "All GetStarted strings are present in Resources"
    },
    {
      "id": "906eba01-3ef7-44d9-8b44-c606b4fb9f55",
      "title": "Acceptance Test Completed",
      "content": "Acceptance test service has finished checking provided add-in."
    }
  ],
  "addInDetails": {
    "adminInstallOnly": false,
    "capabilities": [
      {
        "code": "ReadWriteDocument",
        "title": "Can read and make changes to your document"
      },
      {
        "code": "SendReceiveData",
        "title": "Can send data over the Internet"
      }
    ],
    "defaultLocale": "en-US",
    "description": "SaaS Capital Excel Tools",
    "displayName": "SaaS Cap Data",
    "highResolutionIconUrl": "https://crcastle.github.io/saascap-data/assets/saascap-icon-64.png",
    "hosts": [
      "Workbook"
    ],
    "iconUrl": "https://crcastle.github.io/saascap-data/assets/saascap-icon-32.png",
    "localizedDescriptions": {
      "en-US": "SaaS Capital Excel Tools"
    },
    "localizedDisplayNames": {
      "en-US": "SaaS Cap Data"
    },
    "localizedHighResolutionIconUrls": {
      "en-US": "https://crcastle.github.io/saascap-data/assets/saascap-icon-64.png"
    },
    "localizedIconUrls": {
      "en-US": "https://crcastle.github.io/saascap-data/assets/saascap-icon-32.png"
    },
    "localizedRootSourceUrls": {
      "en-US": "https://crcastle.github.io/saascap-data/taskpane.html"
    },
    "productId": "d348c7c2-36c5-49cc-a8a2-ea185b8427ec",
    "providerName": "SaaS Capital",
    "requirements": "<Requirements xmlns=\"http://schemas.microsoft.com/office/appforoffice/1.1\"><Sets DefaultMinVersion=\"1.1\"><Set Name=\"SharedRuntime\" MinVersion=\"1.1\" /></Sets></Requirements>",
    "rootSourceUrl": "https://crcastle.github.io/saascap-data/taskpane.html",
    "subtype": "TaskPane",
    "supportedProducts": [
      {
        "code": "Mac_Excel",
        "title": "Excel 2019 or later on Mac",
        "version": "<IP_ADDRESS>"
      },
      {
        "code": "Win32_Excel",
        "title": "Excel on Windows (Microsoft 365)",
        "version": "16.0.12527.20092"
      },
      {
        "code": "WAC_Excel",
        "title": "Excel on the web",
        "version": "16.0"
      },
      {
        "code": "Mac_Excel",
        "title": "Excel on Mac (Microsoft 365)",
        "version": "<IP_ADDRESS>"
      }
    ],
    "supportedLanguages": [
      "en-US"
    ],
    "type": "ClientInstalled",
    "version": "<IP_ADDRESS>",
    "autorunLaunchEvents": []
  }
}

Hello? Anyone watching this repo's issues?

We do look at the issues . . . just not on a daily basis.
As pointed out, the tool just calls the web service that is supposed to be doing the same validation that is done when submitting to Partner Source.  The information provided by the service call is what we report in the output.
Have you tried asking Partner Center support for clarification? https://learn.microsoft.com/en-us/partner-center/report-problems-with-partner-center

Hi @millerds. Thanks and apologizes for my slightly snarky message. I have reported the issue following the directions at the link you provided.

@crcastle Did you hear back from Partner Center?

Hey @millerds- Thanks for the follow-up. I did hear from them. It seems that the error message provided from the certification web service was a bit misleading. The URL was reachable, and the file extension was png, but the file was actually in WebP format. I converted it to png and the manifest certification was successful.
I replied to the thread with Partner Center suggesting they tweak the error message to say "the URL is unreachable or file is in the wrong format" instead of just "URL unreachable," which sent me on a bit of a wild goose chase. Alternatively separate out the unreachable error condition from the unreadable file error condition (including associated error messages).
Feel free to close this issue.

Glad you were able to get it resolved.  Hopefully they take your feedback.


check log level to errror while encoutering key not in region

Signed-off-by: iosmanthus<EMAIL_ADDRESS>

LGTM


Plugin settings added to .hyper.js uniformly causes "Error loading config: SyntaxError..."

MacOS Version: Sierra 10.12.6
Hyper Version: 1.4.8
Gist of hyper.js: https://gist.github.com/silverjerk/c6709df5a21f7a7fa6880334adda1883
DevTools error: Unknown OSC code: "1"
Whenever I attempt to add config settings to my hyper.js file (no matter which plugin settings I'm adding), the application produces an "Error loading config: SyntaxError" notification and any existing settings -- like themes -- are disabled and the window reloads without any styling or core settings applied.
After a fresh install (using App Cleaner to remove all system files; and doing a rm -rf .hyper.js and .hyper_plugins), I receive the same error when I first load the app, but on subsequent loads it does not appear to happen.
I've installed plugins by manually editing the hyper.js file, and have also used hpm-cli; I've also attempted to load plugins individually and add config settings one at a time. Even the simplest settings (like the summon settings found in my gist) produce this error.
I also receive errors frequently when trying to install plugins (see the attached screenshot) via hpm-cli. Rather than open a separate issue, I assumed these could be related.

Edit: in a strange turn of events, I am now getting an infinitely looping "bell" chime once I load the app; after uninstalling and removing all the files from the file system, navigating to hyper.io's website produces the same infinitely looping "bell" chime.

@silverjerk Causes conflict with the stable version and Canary in macOS. This problem does not occur in Windows and Linux versions. Possibly some plugins are cached and are causing a conflict when installing new plugins.
Please delete Hyper and reinstall it. If the problem persists let me know 😄

Thanks for the reply. Unfortunately, I've already gone through the process of reinstalling, and even did the diligence to clean up all the files the app left behind to ensure there weren't any stragglers in the file system that were causing the problem. The issue persists...

Okay, I'll start investigating your problem. When you have relevant information, I will let you know 😸
I'm sorry if Hyper is causing you problems, we're working hard to solve these bugs.

No apology necessary. I want to report bugs to help improve the app, not out of frustration. I know you guys are working hard, and the effort is appreciated! :D

Experiencing the same issue here with 1.4.8 after I downgraded from 2.0.4. Clean install results in the same error as well.

@Chinese1904 Delete Hyper and reinstall. We are still working to solve the update problem 😄
if the problem persists let me know.

It persisted. Used appcleaner to remove as well, deleted .hyper.js and the plugins folder. So I ended up going back to 2.0.4 and it was gone there then haha
Thanks for your work on this! Love using Hyper!

@Chinese1904 Great 😺


Integration requires unnecessary work

We migrated from @storyblok/react to use richtext in our project - because of the deprecation warning suggesting to use @storyblok/richtext.
Migration brought several surprises, that some things do not work (e.g. styles passed as string).
Biggest one is - it does not work out of the box, we had to find out that we need to reimplement stuff from the playground.
For DX purposes why does package not contain code in playgrounds by default in the package?
It could e.g. export ReactRichtext, which already has whole basics to make it work.
Or just at least state directly in the readme:
"To make it work with react, implement the code in the playground.", so it could save time for people wondering why things do not work.

Hi @WavyWalk thanks for reaching out and providing feedback, you are right, I'm sorry about the confusion, let me help clear things out:
I'm currently working on integrating the @storyblok/richtext package directly on the @storyblok/react SDK https://github.com/storyblok/storyblok-react/pull/1243. This will export 2 high-level DX APIs:

A hook useStoryblokRichtextResolver
A component StoryblokRichtextResolver

Which under the hood, formats the HTML attributes into JSX like the playground does.

Or just at least state directly in the readme:
"To make it work with react, implement the code in the playground.", so it could save time for people wondering why things do not work.

I'm going to update the docs with this info.


QuickOpen: Prefill search-text-field with last entered text (Feature Request)

VSCode Version: 1.17.2
OS Version: ubuntu16.04

I working with angular and therefore I have similar files with different file-extensions (component.ts, component.html, component.css). Using QuickOpen I have to type the filename for each of them. In IntelliJ the QuickOpen-Dialog (ctrl+shift+n) is prefilled with the last entered text. That's really an ease of use.
Steps to Reproduce:

Opens quick open dialog
Enter your search and press enter
Reopens the quick open dialog --> search-text-field is empty, but should be prefilled with the text from step 2.

Reproduces without extensions: Yes

Need to reconcile with "workbench.commandPalette.preserveInput".

@dbaeumer I looked into making this the default for all QuickOpen prefixes. The problem is that this immediately filters and ranks the available choices which basically removes the 'recently used' feature. Due to that we cannot make it the default without removing functionality for some users. For the same reason I'm not sure having a single flag for all prefixes will be good enough, users might rely on the 'recently used' feature more in one case than in the other.
Renaming the setting from "workbench.quickOpen.prefill" to "workbench.quickOpen.preserveInput" to better align with "workbench.commandPalette.preserveInput".

@JimiC Why should this be off by default?

@JimiC Why should this be off by default?

@Michel73 When it opens without anything prefilled it shows the 'recently opened' files with the most recent ones first. Having a value prefilled hides that feature, which makes me think we should not enable it by default. I use the 'recently opened' files a lot, so I might be biased by that.
One variation I would like to try (and we discussed in the team) is to remember the last value for each prefix and use the setting to enable that. Let me know if this is something you want to work on.

@chrmarti Ok, now I understand the question about the default.
I'm afraid I have no time for further contributions now, but i would appreciate it very much if the prefill feature gets into the release soon.


Unable to make 3rd party modules / plugins use local forked version of React Native

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[x] Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment
React Native Environment Info:
    System:
      OS: macOS High Sierra 10.13.6
      CPU: x64 Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz
      Memory: 4.87 GB / 32.00 GB
      Shell: 5.3 - /bin/zsh
    Binaries:
      Node: 10.12.0 - /usr/local/bin/node
      npm: 6.4.1 - /usr/local/bin/npm
    SDKs:
      iOS SDK:
        Platforms: iOS 12.0, macOS 10.14, tvOS 12.0, watchOS 5.0
      Android SDK:
        Build Tools: 23.0.1, 27.0.3, 28.0.2
        API Levels: 23, 26, 27, 28
    IDEs:
      Xcode: 10.0/10A255 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.6.0-alpha.8af6728 => 16.6.0-alpha.8af6728
      react-native: github:MarkOSullivan94/react-native#develop => 0.57.3
    npmGlobalPackages:
      react-native-cli: 2.0.1

Description
Following the steps of the Building React Native from source documentation and I'm stuck on step 4.
It looked like everything was working fine until I noticed this in Android Studio

Unfortunately this is preventing me from running a local version of React Native so I can test my changes.
My version of React Native can be found here: https://github.com/MarkOSullivan94/react-native/tree/develop
Reproducible Demo
My project containing the error: https://github.com/MarkOSullivan94/ReactNativeShareExample
Clone the project, install the node modules and then try and run it in Android Studio and you'll see warning issues.
If you use the command react-native run-android it'll produce the following error.
 error: cannot find symbol import com.facebook.react.modules.core.PermissionAwareActivity;

Looks like this issue affect this: https://github.com/facebook/react-native/issues/13094.

Contain build android aar.

Build android aar for the share example project  or my local version of React Native ?
What would I do after that?

@MarkOSullivan94 Have you followed this step from the documentation?

@gengjiawen could you provide a guide for what you want me to do?

@MarkOSullivan94
Try the workaround posted here, basically it replaces the react native dependency of all third party moduels with your own.

@MarkOSullivan94 After yarn install,  change node_modules\react-native-image-crop-picker\android\build.gradle to
dependencies {
    implementation project(':ReactAndroid')
    compile 'com.github.yalantis:ucrop:2.2.2-native'
    compile 'id.zelory:compressor:2.1.0'
}

You may want to change this in postinstall hook in package.json for clearity.

@gengjiawen just tried your changes and I've some how got the app running again! Thank you!
Red error screens but at least the app's running now.


C# way naming convention

https://github.com/Bubuhubu87/bc-jokes/blob/28f5380d823112e6014773dbe4c4d93695f31daa/jokes/src/app/modules/jokes/interfaces/jokes.interface.ts#L12

That's true, to fix.

Done


Expose sci and emissions endpoint for prometheus

This PR is responsible for the following :

Exposing sci and emissions endpoint for Prometheus to fetch data.
Refactor to conditionally create beans for EnergyConsumptionProvider, MarginalEmissionsProvider and EmbodiedEmissionsProvider only when an implementation is not provided.
Added an implementation of EmbodiedEmissionsProvider in hello-service for testing point 2.
Update playground.sh to refer to tag v1.0.0 of carbon-aware-sdk

@mtthwcmpbll I am merging this PR for now as its mainly for exposing the endpoints for prometheus and some small updates to the starter.


Update status tag accessibility guidance to reflect new role behaviour

What is the purpose of the work?
The behavior of status tags has been updated in ui-kit issue 20. It now has enhanced accessibility features that needs to be reflected in guidance.
Why do we need it?
To describe the new functionality so that users can understand and benefit from it.

Related to #ui-kit issue 20

Question as to whether if/what further guidance is actually required from the devs.


wire protocol tests

I am currently utilising go-msql toolset  as proxy to provide SQL interface for my datastores like (ES, CH...).
My Goal is to ensure this proxy to behave like standard SQL server so that any MySQL client can connect.
To validate this setup, could you please let me know if there is a test suite available that can be used to verify that the proxy  is behaving like standard SQL server.

I'm not aware of any test suite that does this. There is the MySQL Test, but that is testing the syntax rather than the protocol.
The Client/Server Protocol docs might be useful together with Wireshark, but that mostly useful if you already have identified a possible issue.

Thank you for the info


Bug: No stream-type value supplied warning, when on-demand specified

Which Mux Elements/Packages has a bug?
mux-player
Which browser(s) are you using?
Chrome
Which operating system(s) are you using?
MacOS
How are you using Mux Elements?
Svelte Kit
Describe the bug
I'm seeing the warning [mux-player 1.5.1] No stream-type value supplied. Defaulting to on-demand. Please provide stream-type as either: on-demand, live, ll-live, live:dvr, or ll-live:dvr Read more: Even thought the attribute exists in my code as stream-type="on-demand" as well as the attribute shows up correctly in the DOM.

We thought that we had this fixed, but it could have to do with the Svelte. We'll have to investigate.
The previous workaround is to make sure that stream-type is set on the element prior to the playback-id. Hopefully, it still works here.

Hi @stolinski , as @gkatsev mentioned there used to be a bug where this showed up based on the order of attributes.
We fixed that in 1.4, but maybe in Svelte something weird is going on. Can you just confirm that you're on the latest release?

closing for now, let us know if this issue persists.


Submariner 0.12.1 release

This is the 0.12.1 tracker. Any issues which we want to fix in 0.12.1 (through a backport from the development branch, ideally) should be added to the dependency list.
Depends on https://github.com/submariner-io/shipyard/pull/787
Depends on https://github.com/submariner-io/submariner/pull/1810
Depends on https://github.com/submariner-io/submariner/pull/1783
Depends on https://github.com/submariner-io/submariner/pull/1764
Depends on https://github.com/submariner-io/submariner/pull/1765
Depends on https://github.com/submariner-io/submariner-operator/pull/1986
Depends on https://github.com/submariner-io/cloud-prepare/pull/245
Depends on https://github.com/submariner-io/submariner-operator/issues/1977
Depends on https://github.com/submariner-io/lighthouse/pull/737
Depends on https://github.com/submariner-io/submariner/issues/1774

This was released last week


Keyboard overlaps bottomSheet

Thanks for creating this library
I am trying to use TextEditor inside .bottomSheet. UI works as expected

However, as soon as I tap inside TextEditor, keyboard pops up and covers the bottom sheet

What can be done to readjust the bottom sheet?

@manasv do you happen to have a code snippet of this? Running into the same issue within a current project.

One thing, kinda hacky that you can do is listen for NotificationCenter notifications that tells if the keyboard will show or hide and change the height passed to the sheet based on that.
I've implemented it that way In a project of my own, but you have to maybe place an spacer at the bottom of your view for the content to remain in the upper part.
Will try to assess this internally in bottom sheet so its automatic 🤔

Sounds good, but it didn't work well on my end. Can you share any sample code for that?

I have the same issue but really would like to use your code.
Could you please help with a fix?

I found a fix for this. Using the Binging Bool that determines if the modal is open, add this line to the bottom of the view: .padding(.bottom, {Your-Binding-Bool-Goes-Here} ? 0.1 : 0) . For some reason adding this padding causes the view to lift. If you dont check if the view is open though, then it will show a little bit even when it is not open.

I found a fix for this. Using the Binding Bool that determines if the modal is open, add this line to the bottom of the view: .padding(.bottom, {Your-Binding-Bool-Goes-Here} ? 0.1 : 0) . For some reason adding this padding causes the view to lift. If you dont check if the view is open though, then it will show a little bit even when it is not open.

Great! Mind creating a PR for this? Thanks


What is the length and width of the Mill 19- Rubber scene?

Your work is excellent. I want to know the area of the scene
In the paper, I saw:
Datasets. We evaluate Mega-NeRF against multiple varied datasets. Our Mill 19 dataset consists of two scenes we recorded firsthand near a former industrial complex. Mill 19 - Building consists of footage captured in a grid pattern across a large 500 × 250 m2 area around an industrial building.
Is the Mill 19- Rubber scene also 500 x 250 m2?
Thank you very much

This has been a while, but I think Rubble was smaller - something like 250mx250m. I'd need to dig up some old captures to be sure.

This has been a while, but I think Rubble was smaller - something like 250mx250m. I'd need to dig up some old captures to be sure.

Thank you for your reply,
May I ask if you have found the correct answer?

This has been a while, but I think Rubble was smaller - something like 250mx250m. I'd need to dig up some old captures to be sure.

Thank you for your reply, May I ask if you have found the correct answer?

I measured the area on the map by referencing the COLMAP pose range. The measurements are as follows: Rubble covers an area of approximately 220x150 m², and Building spans about 120x430 m².


refactor(admob): use Jetpack's Navigation Component

Kicking Off step 2 of #1217 : Navigation Component ✨
This PR should replace the 2 activities in Admob with 2 fragments (FirstFragment and SecondFragment) and link them using the Navigation Component.
The navigation graph is simple:

~Some code changes that I should point out:~
~1. There were snippets in the MainActivity.java which were moved to FirstFragment.java~
~2. Snippets in MainActivity.kt were moved to FirstFragment.kt~
~3. Snippets in activity_main.xml were moved to fragment_first.xml~
~4. The method beginSecondActivity was renamed to goToNextFragment~
We're moving the snippets from this repo to firebase/snippets-android. See firebase/snippets-android#272

I was unsure of what to do with the tests, since they make use of Activities.
I tried replacing it with the code I found in Test your Fragments, but I'm not sure if it even works.

@rosariopfernandes thank you for doing this and for looking out for the snippet changes.  I'll need to do some internal work to make sure we don't break the docs when we submit this, so that will delay things a bit.

@rosariopfernandes thank you for doing this and for looking out for the snippet changes.  I'll need to do some internal work to make sure we don't break the docs when we submit this, so that will delay things a bit.

@samtstern I noticed the snippets-android has an admob module. What if we move these snippets there? I can open a PR for this change.

@samtstern I noticed the snippets-android has an admob module. What if we move these snippets there? I can open a PR for this change.

@rosariopfernandes yeah eventually I want to get all of the snippets out of this repo and into snippets-android (unless they're special ones like AndroidManifest.xml snippets).
If you're willing to make that PR that would be great!  You can drop anything in a [START_EXCLUDE]/[END_EXCLUDE] block and just replace it with // ... (or nothing) since that code is not shown in the docs anyway.

@rosariopfernandes yeah eventually I want to get all of the snippets out of this repo and into snippets-android (unless they're special ones like AndroidManifest.xml snippets).
If you're willing to make that PR that would be great!  You can drop anything in a [START_EXCLUDE]/[END_EXCLUDE] block and just replace it with // ... (or nothing) since that code is not shown in the docs anyway.

@samtstern What's special about AndroidManifest.xml snippets? I think I saw a snippet in the admob's Manifest. I wonder if maybe I shouldn't move it?
One more question: When moving the snippets, should we keep using Activities? Or do we switch to Fragments?
In the admob snippets specifically, the main difference is that instead of passing the Activity as context to the AdMob SDK, we use Fragment.getContext().

@samtstern What's special about AndroidManifest.xml snippets? I think I saw a snippet in the admob's Manifest. I wonder if maybe I shouldn't move it?
One more question: When moving the snippets, should we keep using Activities? Or do we switch to Fragments?
In the admob snippets specifically, the main difference is that instead of passing the Activity as context to the AdMob SDK, we use Fragment.getContext().

@rosariopfernandes there's nothing really special about them but they're sometimes hard to "compile" in the snippets repo, having them in a real app makes them more likely to be correct.  If they're tied to the Java code in the snippets repo though then it's totally fine to move them there as well.
As for the Fragment thing I'd suggest just separating out a variable, so if you previously had this in an activity:
doAThing(this)

Instead we could do:
Context context = this.getContext();
doAThing(context)

Which should make it clear to the developer that any Context is what they need and our use of a Fragment in this app is an implementation detail.

@rosariopfernandes there's nothing really special about them but they're sometimes hard to "compile" in the snippets repo, having them in a real app makes them more likely to be correct.  If they're tied to the Java code in the snippets repo though then it's totally fine to move them there as well.
As for the Fragment thing I'd suggest just separating out a variable, so if you previously had this in an activity:
doAThing(this)

Instead we could do:
Context context = this.getContext();
doAThing(context)

Which should make it clear to the developer that any Context is what they need and our use of a Fragment in this app is an implementation detail.

@samtstern Ah, I see. I'll be sure to watch out for that. Thanks
That Context solution sounds great! I'll do that.

@samtstern Ah, I see. I'll be sure to watch out for that. Thanks
That Context solution sounds great! I'll do that.

Internal change merged so now I can merge this one as well!

Internal change merged so now I can merge this one as well!


Upper frame crash-start sunk into next BB

Test source code:
void crash(int* ptr) {
  *ptr = 3;
}

int main(){
  int* p;
  int cond = 1;
  p = 0x0;
  if (cond)
    crash(p);
  cond += 2;
  return cond - *p;
}

Compiled with:
$ clang -g -O0 m.c -o m
In the decompiled MIR, for the presented test case, in the main() frame,
call instruction (corresponding to crash(p)) is the last in it's Basic block.
Since Decompiler marks with crash-start flag, the instruction after call,
it means that crash-start instruction for main() frame is first in it's Basic block.
For this reason, Taint Analysis is skipped for function main() and
any upper frames in similar cases.

It does not make sense why main() is skipped. Is there an issue in starting analysis at the start of a basic-block? The function main() should be analyzed from call-site( crash())  back to entry point of main.

It does not make sense why main() is skipped. Is there an issue in starting analysis at the start of a basic-block? The function main() should be analyzed from call-site( crash()) back to entry point of main.

The current Taint Analysis implementation expects the following sequence of instructions for any upper frame:

First analyzed instruction for that frame
Call instruction to the lower frame
Crash-start instruction after the call
Whole sequence needs to be in the same Basic Block.

The code which prevents analysis if the sequence is not as expected is at crash_analyzer::TaintAnalysis::runOnBlameMF.
There are two possible solutions that come to my mind:

Add artificial NOP after Call instruction to ensure there is expected sequence
Make TA resilient to such cases, and look for the first analyzed instruction in the previous BB.

For this reason, Taint Analysis is skipped for function main() and
any upper frames in similar cases.
My mistake, the TA would be performed for upper frames, but the corrupted one (without expected instruction sequence) would be skipped.

Thanks,
Nikola


config.exe in zip archive

Hi, want to try this little app but that config.exe file makes me nervous. What's that for? Looks like malware. I downloaded a zip file from https://github.com/tjackenpacken/taskbar-groups/releases.
Thanks!

The config.exe file is the one that opens the app group when you click on a extension. What happens when you create a group is that the program generates the necessary config files and then copy the config.exe and creates a shortcut. It isn't malware and is part of the software. In the latest release this .exe has been removed as to me merged into the main program. You will have to wait for a future release for that to push through or you will have to build it from source.

Thanks a lot for explanation!


x button changes his position according to the length of the filename

VSCode Version: Version: 1.47.3 (user setup)
OS Version: Windows_NT x64 10.0.18363

Steps to Reproduce:

just name a file with more than ten characters, and the top bar in the tab navigator (the one containing the name of the file) adapts its length to show its full name.
try to close rapidly many tabs, repeatedly pressing x button
ISSUE: notice that? the x button changes his position according to the length of the filename!
SOLUTION_1: fix a max-length prop to those tab navigator and partially show longer filenames
SOLUTION_2: let the user press ctrl+click (or shift+click) to select more tabs and act simultaneously on all of them ( just like on windows and chrome)

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


Add time-dependent iris_response function.

Adding fit_response function to iris_tools.py

Hi @asainz-solarphysics.  I just merged the Slit Meta PR (#101) we reviewed this morning.  This means you will have to do a git pull origin master on your my_response branch to make sure your branch keeps up to date and avoids conflicts with the sunpy/irispy master version.

Hi @asainz-solarphysics, I had a quick look through fit_iris_xput.  It looks fairly good!  I've left a couple comments about how to make it "better Python".  I also see that this function really needs a docstring, particularly to explain the format and meaning of the coefficient inputs and outputs to make the code itself easier to follow.  I know you said that's on your to-do list so I look forward to seeing it :)

Hi @asainz-solarphysics.  How are you getting on with this?  Do you need any further help pushing new changes?

Hi @asainz-solarphysics.  What news on this PR?

Hi @asainz-solarphysics.  I haven't seen any further activity on this PR for a while.  This is understandable considering other pressing priorities I know you have.  However, since this functionality is still very important to IRISpy, I would like to make the next stages of this PR available for a student to take over if they get funded.  Would you have any objection to this?  If you would like to stay involved, you could act as IRIS team advisor while I can be the Python advisor.  However, there's no obligation and I can work with the student myself.


looping

GO:0090579    dsDNA loop formation
Definition (GO:0090579 GONUTS page)
The formation and maintenance of DNA loops that juxtapose separated regions on the same dsDNA molecule. PMID:15950878
Is used for 2  different processes:
I) chromatin loop formation
Cohesin capture and extrusion  of DNA in a processive manner.
ii) Local loop formation as  part of transcription (i.e. promoter enhancer looping)
which is represented by the child term "GO:0090202    gene looping"
and grand kids
GO:0071733    transcriptional activation by promoter-enhancer looping | is_a
GO:0090203    transcriptional activation by promoter-terminator looping
I think we need a specific term for "chromatin loop formation" as these are currently not separable from promoter -enhancer loops

@colinlog  is it correct that looping can refer to different processes?

Obsolete in BP:

'dsDNA loop formation'
'gene looping'
'transcriptional activation by promoter-enhancer looping' -> obsolete and replace by NTR promoter-enhancer loop anchoring activity : Bridging together promoter DNA and enhancer DNA sequences on the chromatin, holding two loop anchors together, maintaining a chromatin loop.
has_part loop anchor DNA binding
'transcriptional activation by promoter-terminator looping'-> obsolete and replace by NTR promoter-terminator loop anchoring activity
Bridging together promoter DNA and terminator DNA sequences on the chromatin, holding two loop anchors together, maintaining a chromatin loop.
has_part loop anchor DNA binding

NTR parent class: chromatin loop anchoring activity
part_of some chromatin organization
NTR BP: chromatin looping (Exact Syn: chromatin loop assembly)
A chromatin organization process that starts with the loading of an extrusion motor (SMC family complex) onto the chromatin, followed by chromatin extrusion that stops at loop anchoring sites on the chromosome.
ADD: chromatin extrusion motor activity
part of 'chromatin loop assembly'
GO:0140564 DNA-DNA tethering activity

[x] move up under 'molecular adaptor activity'

New terms:
+[Term]
+id: GO:0140585
+name: promoter-enhancer loop anchoring activity
+namespace: molecular_function
+def: "Bridging together promoter DNA and enhancer DNA sequences on the chromatin, holding two loop anchors together, maintaining a chromatin loop." [PMID:32213323]
+synonym: "enhancer-promoter loop anchoring activity" EXACT []
+is_a: GO:0140587 ! chromatin loop anchoring activity
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/20204 xsd:anyURI
+created_by: pg
+creation_date: 2021-02-04T14:50:04Z
+
+[Term]
+id: GO:0140586
+name: promoter-terminator loop anchoring activity
+namespace: molecular_function
+def: "Bridging together promoter DNA and terminator DNA sequences on the chromatin, holding two loop anchors together, maintaining a chromatin loop." [PMID:19933151]
+synonym: "terminator-promoter loop anchoring activity" EXACT []
+is_a: GO:0140587 ! chromatin loop anchoring activity
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/20204 xsd:anyURI
+created_by: pg
+creation_date: 2021-02-04T14:51:44Z
+
+[Term]
+id: GO:0140587
+name: chromatin loop anchoring activity
+namespace: molecular_function
+def: "Bridging together two DNA loop anchors together, maintaining a chromatin loop." [PMID:32213323]
+is_a: GO:0106260 ! DNA-DNA tethering activity
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/20204 xsd:anyURI
+created_by: pg
+creation_date: 2021-02-04T14:53:13Z
+
+[Term]
+id: GO:0140588
+name: chromatin looping
+def: "A chromatin organization process that starts with the loading of an extrusion motor (by an SMC family complex) onto the chromatin, followed by chromatin extrusion that stops at loop anchoring sites on the chromosome." []
+synonym: "chromatin folding" RELATED []
+synonym: "chromatin loop assembly" EXACT []
+is_a: GO:0006325 ! chromatin organization
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/20204 xsd:anyURI
+created_by: pg
+creation_date: 2021-02-04T14:55:20Z
Thanks, Pascale

Dear all,
The proposal has been made to obsolete

GO:0090579 dsDNA loop formation
GO:0090202 gene looping
GO:0071733 transcriptional activation by promoter-enhancer looping
GO:0090203 transcriptional activation by promoter-terminator looping

The reason for obsoletion is that the way those terms are defined, they are better represented by molecular functions. We have created 3 functions to replace those terms:
GO:0140587  chromatin loop anchoring activity
GO:0140585  promoter-enhancer loop anchoring activity
GO:0140586  promoter-terminator loop anchoring activity
and a more general function: GO:0140588 chromatin looping
The concepts of transcription regulation should be captured with 'GO:0006355 regulation of transcription, DNA-templated' or a child term.
There are 25 EXP annotations to those 4 terms. @ValWood and @colinlog have made suggestions for reannotation, see https://github.com/geneontology/go-annotation/issues/3610 (it may be a couple of days until the new terms are available).
We are opening a comment period for this proposed obsoletion. We’d like to proceed and obsolete this term on February 18th, leaving some time to make the corrections. Unless objections are received by February 18th, 2021, we will assume that you agree to this change.
You can comment on the ticket here: https://github.com/geneontology/go-ontology/issues/20204
Thanks, Pascale


Testrunner setting fails when set to jasmine1

Do you want to request a feature or report a bug?
Bug: Setting the testRunner to "jasmine1" as described here fails. Setting to "jasmine2" works fine.
What is the current behavior?
 ● Test suite failed to run

    Cannot find module '/Users/coryhouse/Desktop/projects/fusion-starter/jasmine1'

      at Function.Module._resolveFilename (../module.js:339:15)
      at Function.Module._load (../module.js:290:25)
      at Module.require (../module.js:367:17)

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal repository on GitHub that we can npm install and npm test.
Set package.json to:
  "jest": {
    "testRunner": "jasmine1"
  },

What is the expected behavior?
Should use the jasmine1 runner as specified.
Run Jest again with --debug and provide the full configuration it prints. Please mention your node and npm version and operating system.
Node 5.6 on OSX.

It seems like the docs are outdated. We don't actually support jasmine1 in Jest any more and dropped it with version 14 or 15.

publishing a new version of the website now.


Use latest version of debian in Dockerfile.template

This also fixes versioning since the last commit should have left the version at 0.1.0
Change-type: minor

@balena-ci I self certify!


Could you add bark.cpp as a text-to-speech add-on, please?

https://github.com/PABannier/bark.cpp

let's have a try~


Support writer owns the schema

Currently, the writer needs to take a lifetime param and this will make it hard to include in struct sometimes. E.g.
struct OutWriter<'a> {
  avro_schema: avro::Schema
  inner_writer: avro::Writer<'a,Vec<u8>>
}

Even though OutWriter owns the avro_schema, it still needs to define the lifetime param explicitly. And this lifetime will continue to propagate to other types. Self-reference seems a complicated problem https://users.rust-lang.org/t/how-can-we-teach-people-about-self-referential-types/65362/2. So could we just provide a writer who owns the schema and avoids the lifetime param?🤔

Do you need to keep the inner_writer as a field ?
You can create an instance when you need to write something and then discard it, e.g. in OutWriter::write() method.

Do you need to keep the inner_writer as a field ? You can create an instance when you need to write something and then discard it, e.g. in OutWriter::write() method.

For example, I want user to use the writer like following:
outside_writer.write()
outside_writer.write() // user can call write multiple times.
outside_writer.close()

In this case, I have to cache the content of each write. When the user calls close, create an Avro writer and write all cached content at once.
But if I can keep the inner_writer as a field, I don't need to cache the content. I think this will cause more memory consumption, at most twice I think.

You can experiment and propose a PR but I think it will lead to too many API breaks and probably many clone()s of the Schema.


Remove New-UMIPGateway Support in Exchange Online

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: yinzh798 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/unified-messaging/New-UMIPGateway.md
:white_check_mark:Succeeded
View (exchange-ps)

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.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/unified-messaging/New-UMIPGateway.md
:white_check_mark:Succeeded
View (exchange-ps)

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.


⚠ Bofrost.be has degraded performance

In c0dabca, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1809 ms

Resolved: Bofrost.be performance has improved in 70c449d.


Errors while syncing -  File "" - UnicodeDecodeError: 'utf-8' codec can't decode byte 0xa0

Here is the error message I get:
C:\Users\Tom>adbsync push "F:\1-Music\3-Music Phone\Music" "/sdcard/"
* daemon not running; starting now at tcp:5037
* daemon started successfully
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "C:\Users\Tom\AppData\Roaming\Python\Python312\Scripts\adbsync.exe\__main__.py", line 7, in <module>
  File "C:\Users\Tom\AppData\Roaming\Python\Python312\site-packages\BetterADBSync\__init__.py", line 379, in main
    files_tree_destination = fs_destination.get_files_tree(path_destination, follow_links = args.copy_links)
                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\Tom\AppData\Roaming\Python\Python312\site-packages\BetterADBSync\FileSystems\Base.py", line 45, in get_files_tree
    return self._get_files_tree(tree_path, statObject, follow_links = follow_links)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\Tom\AppData\Roaming\Python\Python312\site-packages\BetterADBSync\FileSystems\Base.py", line 33, in _get_files_tree
    tree[filename] = self._get_files_tree(
                     ^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\Tom\AppData\Roaming\Python\Python312\site-packages\BetterADBSync\FileSystems\Base.py", line 33, in _get_files_tree
    tree[filename] = self._get_files_tree(
                     ^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\Tom\AppData\Roaming\Python\Python312\site-packages\BetterADBSync\FileSystems\Base.py", line 33, in _get_files_tree
    tree[filename] = self._get_files_tree(
                     ^^^^^^^^^^^^^^^^^^^^^
  [Previous line repeated 2 more times]
  File "C:\Users\Tom\AppData\Roaming\Python\Python312\site-packages\BetterADBSync\FileSystems\Base.py", line 30, in _get_files_tree
    for filename, stat_object_child, in self.lstat_in_dir(tree_path):
  File "C:\Users\Tom\AppData\Roaming\Python\Python312\site-packages\BetterADBSync\FileSystems\Android.py", line 176, in lstat_in_dir
    for line in self.adb_shell(["ls", "-la", path]):
  File "C:\Users\Tom\AppData\Roaming\Python\Python312\site-packages\BetterADBSync\FileSystems\Android.py", line 87, in adb_shell
    adb_line = adb_line.decode(self.adb_encoding).rstrip("\r\n")
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xa0 in position 71: invalid start byte

Additional info: when using "adb push", I get errors when my files' path is too long, so maybe there's a similar issue with better-adb-sync?
I'm not familar with Python debugging so let me know if you need anything else.
Thank you!

looks like a duplicate of #44 which has an input data to work with and suggested workaround


incrementCounter returns 'undefined' after upgrade to Meteor 1.4

incrementCounter returns 'undefined' after upgrade to Meteor 1.4.

https://github.com/Konecty/meteor-mongo-counter/commit/cca56926365e634e4cac0894685bcf7832054efe

Thank you for report! Sorry for long delay, I fix in 2 days.

I am also getting this error.  Any info on when this will be fixed?

Hey Paul.
I personally replicated the code (to JavaScript), fixed it (1 line) and put
it in directly in my project.
Regards,
Dominique.
2016-08-09 0:11 GMT+02:00 Paul Drummond<EMAIL_ADDRESS>
I am also getting this error. Any info on when this will be fixed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/osv/meteor-mongo-counter/issues/5#issuecomment-238393661,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAx-3rFStHlmdAOEDje_s5JsMJQiR2dpks5qd6mAgaJpZM4JXfjD
.

Fixed. You can bump to<EMAIL_ADDRESS>Sorry for delay again. I close issue.


ip is not updatet on login

when login and ip need be updatet the ip is not updatet

Is now fiexed in V0.1R1


Processing .const statements

Processing for .const statements, dot removed from beginning and semicolon added to end of statement.
Closes #3

@intoinside That was the only issue I found. Thank you for maintaining this project.


native base issue

Hello I am using
"native-base": "^2.2.1",
"react-native": "^0.59.1",
I am getting the following error
error: bundling failed: Error: Unable to resolve module react-native/Libraries/Renderer/shims/ReactNativePropRegistry from path\node_modules\native-base\dist\src\Utils\computeProps.js: Module react-native/Libraries/Renderer/shims/ReactNativePropRegistry does not exist in the Haste module map
Please help me in this...
@beausmith

I'm also facing the same issue,
Can any one please help me.

Sorry, I'm unable to help as I'm not currently working in any react-native projects.

Please check with the latest version.


Not supporting FFMPEG v7 is a big issue

In every app we are using latest version of FFMPEG
but the app fails since it is forcing FFMPEG version 6 and it complicates to make installers for different platforms
now i am going to make auto downloader for version 6 and put into accurate folder

That's a valid point.  Let me keep this issue open and see if I can get rid of it.

Fixed 0e73e45389a07ee4be6b5e72e24638358c59d3f8

@hkchengrex i have spent hours to fix ffmpeg errors haha :D


Combination: Components with different 1-4 scaling factors

Is this even possible? I feel like it should error out.


Basic types

Integer and Strings
(no more details, transferred from YouTrack a posteriori)

I changed the issue to keep the same issues number as YouTrack


Add logging for disco_acm

DiscoACM doesn't log anything when a certificate isn't found. Adding some logging statements to help debug why it can't find certs in staging/prod.

👍


Fix floats in scientific notation not being parsed correctly

Floats using Scientific Notation are not being parsed correctly (eg, 6.022E24)
Results in attributes mysteriously not being parsed since all attributes after the float get silently swallowed.
Might need more tests but i did add two.

Thank you @blackhathedgehog


[Feature Request] Disable Two-Step Hotkey Feature

I want a way to using JUST ONE key to instant send message or command. It will much more convenient in some cases.
Also, an option to disable display command in actionbar
Bonus: I'm ok with dependence, like cloth-config.

It is true that a single-step hotkey would be more convenient in some cases, but there is a good reason that this mod uses a two-step system. My keyboard has ~74 bindable keys, and 67 of those are already bound to other functions. I also currently have 21 quick messages loaded. So, if I were to implement a single-step hotkey, I would have to lose 14 functions/messages.
Disabling the command display is doable, though I'm curious to know why you want it?

Actually, I think the mod shuold have an OPTION, mean if player think two-step hotkey is not nessessory, they can freely disable them.
About actionbar, most of message or command have the own response in the chat, mean you can still know actual message or command you just sent. And disable that can make the play-flow more consistent. And once again it's just an option and can be enable/disable in the setting.

A quick search reveals that are multiple alternative mods implementing a single-hotkey approach. Perhaps not yet on the latest MC version, but you can get in touch with their developers about updating.
Your suggestion is considered, but I will not give an ETA for a final decision or change at this time.


YAML indent is wrong on parser multipath and tail_files is not valid for filestream

Indent is wrong in parser section of filestream.yml template -- extra spaces cause filebeat to not load.
Also tail_files is used in the deprecated log input, not in filestream.

Are you seeing this in a release or when you pull from the repository?


Setup github pages with github actions

As a user, I need a website with updated code documentation.
Acceptance criteria:
1 - GitHub pages website published through github actions

not applicable


Use latest azure-pipelines-tasks-webdeployment-common in AzureRmWebAppDeploymentV4

Task name:
Description:
Documentation changes required: (Y/N)
Added unit tests: (Y/N)
Attached related issue: (Y/N)
Checklist:

[ ] Task version was bumped - please check instruction how to do it
[ ] Checked that applied changes work as expected

/azp run


React-Router createTransitionManager() silently fails in packaged application

Line 96 of Router.js:
this.transitionManager = this.createTransitionManager();
all my inputs are defined at runtime I've confirmed, and this.createTransitionManager() is also defined.
After passing that line though, this.transitionManager is not defined, and the packaged application fails with:
Uncaught (in promise) TypeError: Cannot read property 'getCurrentLocation' of undefined(…) on line 99 of Router.js
Any ideas on why this might be the case?

What is the full stacktrace?

I am creating my store with an existing state, but I have tried removing my createStoreFunction from the promise it is relying on, and also have tried not importing any existing state and just creating an empty one. I get the same results in all configurations.

I've tried now removing it from the function and running it almost identically to the default.
This is my production store config. It is slightly different than the stock, but like all the above, worked fine ~3 months ago, and I can't find any changes to the boilerplate that I would expect to create issues.
import { applyMiddleware, createStore, compose } from 'redux';
import { responsiveStoreEnhancer } from 'redux-responsive';
import thunk from 'redux-thunk';
import { routerMiddleware, push } from 'react-router-redux';
import rootReducer from '../reducers';
import storage from '../utils/storage';
import onExitStorage from '../utils/onExitStorage';
import * as noteActions from '../actions/notes_actions';

var hashHistory = require('react-router').hashHistory;

const actionCreators = {
  ...noteActions,
  push,
};

const router = routerMiddleware(hashHistory);

const enhancer = compose(
  responsiveStoreEnhancer,
  applyMiddleware(thunk, router),
  storage(),
  onExitStorage(),
);

export default function configureStore(initialState: Object | void) {
  return createStore(rootReducer, initialState, enhancer);
}

Within react-router, this.props.history is undefined
I have no idea what could cause this.

Should process.env.NODE_ENV be defined in the packaged version? It is undefined, which seems proper, but since I had to package it myself I'm wondering if it has anything to do with NODE_ENV being wrong.

I believe it should be production because of the define plugin. Also did you resolve this? If so, please close. Makes it easier for me to keep track of issues. Thanks!

@amilajack I have not been able to resolve this one yet. It may be due to the workaround I had to use in #816

So in my index.js store and history are both defined when I run this in the index.js
	store = configureStore(myAppState);
	history = syncHistoryWithStore(hashHistory, store);
	render(
	  <AppContainer>
	    <Root store={store} history={history} />
	  </AppContainer>,
	  document.getElementById('root')
	);

When I examine the script running in the root.js file moments later, they are both undefined there.
Only happens in production. I confirmed process.env.NODE_ENV is supposed to be undefined in a working perfect copy of the repo, so that's not the issue.
I don't know if this is somehow related to my not being able to build via NPM scripts or entirely unrelated. Any thoughts?
All that changed was the boilerplate, and honestly not very much. The change that must have broken it is the addition of the root.js file which I was not using before.
Absolutely no idea why the values are suddenly undefined when it passed off to Root.

I will try to create a small demo using createTransitionManager() and see if I run into the same issue.

I think the root of the issue is really in passing values into root.js and I just didn't realize prior because I was unable to debug in that file by setting a breakpoint. After repackaging it with a debugger in root.js I find the values to be undefined there, which I believe all occurs prior to createTransitionManager() being run

What method are you using to debug the production build?

I've enabled the devTools window to open and changed the sourcemaps mode to inline instead of cheap-eval
Not sure how well that answers your question, but that's what I've been doing to try and pinpoint the cause.

After syncing with upstream again (commit f4dc6f2917f21bfdc7d58d2830421af9c2260201), I now see the error in both production and development and the error is that I am passing the wrong version of history.
In reality, history is not making it into my Root element, even if I move Root to the index.js.
What does work is moving Root to the index.js and not passing store and history, but instead pulling them in from outside the Root scope.
const Root = () => {
  return (
    <Provider store={store}>
      <Router key={Math.random()} history={history} routes={myRoutes} />
    </Provider>
  );
}

store = configureStore(appState);
history = syncHistoryWithStore(hashHistory, store);
render(
  <AppContainer>
    <Root store={store} history={history} />
  </AppContainer>,
  document.getElementById('root')
);

Unfortunately this completely breaks hot reloading of course. Any ideas why this may be? I'm thinking possibly a bug in AppContainer? I don't know what to make of it.

Anyone have any thoughts on this? It really breaks the whole boilerplate for me since I have to fully reload any time I make a typo in my code that causes a compile error.

Do you have a minimal boilerplate that I can look through? I remember asking you for this before but I may have missed your response.

I may have missed you asking. I'll try to put something together. I supposed it shouldn't be terribly difficult to slice apart my current repo since it's at the top level.

@Slapbox it would be easier for me if you could clone the repo and make the minimal amount of changes to the fresh clone in order to reproduce the error

@amilajack it amazes me you are not even out of school.
I managed to resolve the issue, though I didn't exactly manage to figure out what the issue was. It was something in my store I think? I'm really not sure unfortunately.

Okay so fixing this issue, here's the code and how it works:
index.js
import _ from 'lodash';
import React from 'react';
import { Provider } from 'react-redux';
import { render } from 'react-dom';
import { syncHistoryWithStore } from 'react-router-redux';
import { Router, hashHistory } from 'react-router';
import { AppContainer } from 'react-hot-loader';

import configureStore from './store/configureStore';
import myRoutes from './routes';
import Root from './containers/Root';

function ProductionRoot() {
  return (
    <Provider store={store}>
      <Router key={Math.random()} history={history} routes={myRoutes} />
    </Provider>
  );
}

function createStoreFunction() {
	store = configureStore();
	history = syncHistoryWithStore(hashHistory, store);
	render(
	  <AppContainer>
	    {(process.env.NODE_ENV != 'development') ? <ProductionRoot store={store} history={history} />  :  <Root store={store} history={history} /> }
	  </AppContainer>,
	  document.getElementById('root')
	);

	if (module.hot) {
	  module.hot.accept('./containers/Root', () => {
	    const NextRoot = require('./containers/Root'); // eslint-disable-line global-require
	    render(
	      <AppContainer>
	        <NextRoot store={store} history={history} />
	      </AppContainer>,
	      document.getElementById('root')
	    );
	  });
	}

In my particular case, in production, store and history are undefined, unless root is within the same file, and the store and history are not explicitly passed in. This is a workaround for that issue if anyone else runs into it.


Geopackage features

Coverage decreased (-27.1%) to 59.11% when pulling 77dd381449b308a98d4f46acde3c67f0218e2363 on geopackage-features into 9d1df7abacf538ad4eabf5cabcf0172476cdeb51 on master.

Coverage decreased (-27.1%) to 59.11% when pulling 6992968a5cabd997d68fd3ef07d6d9eff64a969d on geopackage-features into 9d1df7abacf538ad4eabf5cabcf0172476cdeb51 on master.

In TestUtility.java%getRandomFile(), you can set testFile.deleteOnExit() and just get rid of TestUtility.deleteFile() altogether, along with all the calls to that function in the Geopackage tests.  See here for more info:
https://docs.oracle.com/javase/7/docs/api/java/io/File.html#deleteOnExit()

In TestUtility, the package imports are unnecessary.

In WkbCurvePolygonZM fix type-o pending implementaiton

What is NetworkUtility.java for?  Testing networks?  Does it need to remain in source?

Code has been reviewed as best as I could.  Some test files were ignored since I assume the build will check if they are correct or not.
How do you plan on merging this?  Have you merged master into this branch in a while?  I will have to disable test coverage thresholds on Coveralls for a while until your tests are done and we are back up to ~86% coverage again.

Added testFile.deleteOnExit()

Removed unnecessary TestUtility package imports

Fixed implementaiton typos

Removed NetworkUtility

Coverage decreased (-25.8%) to 60.464% when pulling 679bcffc4a77045cfc9e71517c31c899535accc7 on geopackage-features into a346e274a1d378cda25ea53bde0b230814c19941 on master.


🎁: Handover filters

Expected outcome
   Filters = Filter name i workspace

[ ] MC Discipline = Disciplines
[ ] Supplier
[ ] Handover Status = Comm pkg Status
[ ] Comm Status
[ ] MC Status
[ ] Comm Responsible
[ ] Comm Location = Area
[ ] Comm Phase = Phase
[ ] Comm System = System
[ ] Comm SubSystem
[ ] Comm Priority 1
[ ] Comm Priority 2
[ ] Comm Priority 3
[ ] Facility
[ ] Project
[ ] Comm Identifier

Add these to workspace

[x] PBI
[ ] Workspace

I have added the filters for PBI in test:
https://webserver-fusion-project-portal-test.radix.equinor.com/apps/handover/ce31b83a-b6cd-4267-89f3-db308edf721e?tab=powerbi
@JoneUE With longer filter names, the whole filters name is not visible.

@espenkalle @Gustav-Eikaas are these filters available for the workspace? if yes, can you please close the issue.

Ser på det etter lunsj :)

@atlewee

If Comm Area is renamed to comm Location, it is good to go. This is due to commonLib using the name Location for areas

@JoneUE We have this file to try and standardize naming across applications.
https://github.com/equinor/fusion-data-gateway/blob/main/src/CCApplications.Applications/Common/Domain/DomainNames.cs

@Gustav-Eikaas I can look at it, but if you're in doubt about someone, I'm trying to standardize with what is in ProCoSys. so you can also look what names they use. For example ChecklistStatus would be Form Status

But also the easy answer would be that all columns that belong to the checklist table, should start with Form, Tag with Tag, SWCR with SWCR, Punch with PL, Work Order with WO, Mcpkg with MC and Commpkg with Comm


Laravel cors are not returned

// Handle the request
$response = $next($request); ///Laravel pass to next Middleware without setting cors
    if ($request->getMethod() === 'OPTIONS') {
        $this->cors->varyHeader($response, 'Access-Control-Request-Method');
    }

    return $this->addHeaders($request, $response); // Cors are added here

Since two days i've a bug with cors bc 'im in virtual machine for back office dev machine and database but on the host for front office dev from vue.js
And tu update auth user profile the browser always throw a Cors error bc missing AllowAccessControlOrigin, and true they are never set by middleware cors package.
Laravel: 8.40
FruitCake: 2.0
PHP: 7.3.27
Chrome DevsTools

this solution is not perfect but you can add headers in the routes web.php file on the top the file
I had the same problem with the chrome extension this is how I resolved it.
the laravel dd will not work with vue.js so use postman for debugging
hope this help
routes/web.php
<?php

header('Access-Control-Allow-Origin: *');
header('Access-Control-Allow-Methods: POST, GET, OPTIONS, PUT, DELETE');
header('Access-Control-Allow-Headers: Content-Type, X-Auth-Token, Origin, Authorization');

use Illuminate\Support\Facades\Route;

    /*
    |--------------------------------------------------------------------------
    | Web Routes
    |--------------------------------------------------------------------------
    |
    | Here is where you can register web routes for your application. These
    | routes are loaded by the RouteServiceProvider within a group which
    | contains the "web" middleware group. Now create something great!
    |
    */
Route::get('/', function () {
    return "hello world";
});

+1 had the same problem


Add helpful error message when attempting to run brew cask

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?

I may have missed some, as there are 26 pages of results for `

[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

I couldn't think of any general enough tests for this that don't basically just duplicate the logic, but I'm open to writing one if needed.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

I get some nondeterministic errors both with my code and with the current master code, that differ between runs. This may relate at least in part to me running this on an M1 mac. Some errors were timeouts. Wasn't able to check/compare many times because it takes about 5 minutes to run the full suite.
EDIT: But the CI tests pass! 🎉

This adds a simple check: if the command being used is brew cask, it exits early with a helpful error message instructing them to use brew <command> --cask instead.
Fixes #12707
See https://github.com/Homebrew/discussions/discussions/902 for related discussion & motivation for this issue

Getting back to this today.

Updated it to be much simpler as requested.

Ah, but I missed some style things. Fixing.

Looks like it's failing due to code coverage going down. I'm not sure how to add a good test for this; any tips on where that would go / what tests to mimic, if you want me to add one?

wait, the coverage report changed after the fact?

The coverage from Linux gets reported first (before the macOS coverage report), so there's a short delay before the coverage status is updated.


Trim whitespace from base64 encoded credentials (username, password, token)

Probably should do this in both the config-provided and secret-store-retrieved ones, so perhaps at use, rather than
at config load?

fixed in 92b5315a1f2cb51dab4f25f03785e07148a791c0


[Q] How to suppress cookies-files when using python3-bindings and HTTP connections

Hi,
I am working on an application /w the tsduck python-bindings to transmuxing http-streams into multicast.
There are some 'cookies'-files created in the /tmp directory.

Is there a way to get ride of these files?

On last check, after hours of running, there were thousends of files named like:  /tmp/tstmp-< hexcode >.cookies
hexcode:  some random hexcode, aka '000B553DAD550000'
Content of those files, are all the same, probably the will be some cookie data when available:
# Netscape HTTP Cookie File
# https://curl.se/docs/http-cookies.html
# This file was generated by libcurl! Edit at your own risk.

Any suggestion to point me into a direction ?

Hi @dutchboontje
Those files are normally created by the hls plugin, not the http plugin. Could you confirm that this is the case?
Having cookie files is required in HLS because the HTTP request on the playlist sometimes returns cookies which are required to access the segments. This is not the case with the HTTP plugin.
I just made a test on macOS with a tsp -I hls command. I can see the cookies files but it is automatically deleted when the plugin is closed. However, in some cases, I have noticed that the cookies file is not deleted, even though the debug message "deleting cookies file /tmp/tstmp-00221ABAADF70000.cookies" is displayed. Looking in the source code, this debug message is immediately followed by the deletion of file. In case of deletion error, an error message is displayed. Since I see no such error message, having the cookie file not deleted is really mysterious. I have to investigate.
In the meantime, could you indicate:

The operating system name and version
The TSDuck version
The exact chain of plugins
How is the command terminated (end of input, some condition such as plugin until or interruption).
If you use the Python bindings, do you call the method waitForTermination of the TSProcessor object? This ensures that all plugins are correctly terminated, including the cleanup of the cookies file in the case of the hls plugin (even though I have noticed cases where the cookies files is mysteriously not deleted as described above).

Hi Ielegard,  some answers...
I will have to check /w you patch, still. I hope this week.
Problem found on version 3.31.2722:
edwin@streaming-server-tvm01:~$ tsversion --version=all
tsversion: TSDuck - The MPEG Transport Stream Toolkit - version 3.31-2722
Built May 17 2022 - 06:34:31
Using GCC 11.2.0, C++ std 2011.03
System: Ubuntu (Ubuntu 22.04 LTS), on Intel x86-64, 64-bit, little-endian, page size: 4096 bytes
Bitrate: 64-bit floating-point value
Dektec: This version of TSDuck was compiled without Dektec support
Web library: libcurl: 7.81.0, ssl: OpenSSL/3.0.2, libz: 1.2.11
SRT library: libsrt version 1.4.4
RIST library: This version of TSDuck was compiled without RIST support

System/OS info:
edwin@streaming-server-tvm01:~$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 22.04 LTS
Release:	22.04
Codename:	jammy

Related libCurl version:
edwin@streaming-server-tvm01:~$ apt show libcurl4
Package: libcurl4
Version: 7.81.0-1ubuntu1.2
Priority: optional
Section: libs
Source: curl
Origin: Ubuntu

Issue/files concerned:
edwin@streaming-server-tvm01:~$ uptime; echo; echo -n "#of tstmp-* files..."; ll /tmp/tstmp-*| wc -l
 11:37:59 up 4 days, 17:09,  1 user,  load average: 0.22, 0.21, 0.23
# of tstmp-* files...4101

Implementation TSduck /w Python binding, config:
# Channel channel1 config
tsp_options = {
	'app_name': 'channel1',
	'bitrate_adjust_interval': 1000,
	'buffer_size': 64
	}
input = ['http', '--receive-timeout', '5000', '--user-agent', 'tsduck/3.x', 'http://<masked host>:80/<masked path>']
plugins = [
	['count', '--pid', '0-31', '--negate', '--brief', '--interval', '16620'],
	['continuity'],
	['pcrbitrate'],
	['regulate', '--pcr-synchronous'], ['sdt', '--increment-version', '--provider', 'live', '--service-id', '1', '--name', 'NPO 1']
	]
output = ['ip', '--packet-burst', '7', '--ttl', '5', '--force-local-multicast-outgoing', '--local-port', '1234', '<IP_ADDRESS>:1234']

Count-plugin is added to monitor traffic of the source.
Implementation TSduck /w Python binding, snip of code:
# Create a TS processor using the report.
tsp = tsduck.TSProcessor(rep)

# Set some global TS processing options.
if hasattr(config, 'tsp_options'):
    # just for safety
    if 'app_name' not in config.tsp_options:
        tsp.app_name = "send-origin1-service1"

    for key in config.tsp_options:
        print( f"setting option {key} = {config.tsp_options[key]}")
        tsp.key = config.tsp_options[key]

tsp.input = config.input
tsp.plugins = config.plugins
tsp.output = config.output

# Run the TS processing and wait until completion.
tsp.start()
tsp.waitForTermination()

# At this stage, there should be some thing going wrong, abondon to restart service
tsp.delete()
rep.terminate()
rep.delete()

It's running as an embedded application. A few streams are fired off all /w differend sources and destinations.
No user interaction will be done.  Any actions to be expected are source dis-continuities, although these are supposed to be live-streams.  :{
As you can see it configures on http-inputs module, how come to hls-input ?
Let me know if you needed additional info.

Hi @dutchboontje
You are right, there was another (and deeply hidden) issue with the http plugin. In fact it was in the handling of common command line options for web requests. I fixed it too.
To be more specific, the spurious cookies file remained in the hls plugin when it was interrupted and in the http plugin in all cases.


Speed-up CI builds by enabling caching

Still need to verify if cache is shared across threads (preffered in our case) or being built by thread


Expose pod name in PipelineExecutionStepState details

Failures in the "deploy" step are hard to debug because often the error message is only visible in the service's log. To be able to filter the environment log for relevant entries one needs to know the pod name. That is unfortunately not exposed in https://github.com/AdobeDocs/cloudmanager-api-docs/blob/b6c0678a414f112dba41e3bc8ecbf324385cd20f/swagger-specs/api.yaml#L1671
Although in one step there might be different pods involved usually only the pod where the error occurred is of interest. Instead of just the pod name also the complete podId could be exposed.

The documentation is correct here.

@justinedelson Do you want me to report that issue through some other channel then?

@kwin This project should only be used for documentation issues. In this case, the PipelineExecutionStepState documentation is correct. Feature requests for the API (or CM in general) should flow through the normal channels.


problems with the orjson import package

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
Bug Report
I think it may be due to certain problems with the orjson import package.
There should be a circuit breaker mechanism to load the json module to support operation when orjson cannot be loaded normally.
File "G:\pro_personal\LLMServer\text_splitter\__init__.py", line 1, in <module>
    from .chinese_text_splitter import ChineseTextSplitter
  File "G:\pro_personal\LLMServer\text_splitter\chinese_text_splitter.py", line 1, in <module>
    from langchain.text_splitter import CharacterTextSplitter
  File "G:\pro_personal\LLMServer\.venv\lib\site-packages\langchain\text_splitter.py", line 2, in <module>
    from langchain_text_splitters import (
  File "G:\pro_personal\LLMServer\.venv\lib\site-packages\langchain_text_splitters\__init__.py", line 22, in <module>
    from langchain_text_splitters.base import (
  File "G:\pro_personal\LLMServer\.venv\lib\site-packages\langchain_text_splitters\base.py", line 23, in <module>
    from langchain_core.documents import BaseDocumentTransformer, Document
  File "G:\pro_personal\LLMServer\.venv\lib\site-packages\langchain_core\documents\__init__.py", line 6, in <module>
    from langchain_core.documents.compressor import BaseDocumentCompressor
  File "G:\pro_personal\LLMServer\.venv\lib\site-packages\langchain_core\documents\compressor.py", line 6, in <module>
    from langchain_core.callbacks import Callbacks
  File "G:\pro_personal\LLMServer\.venv\lib\site-packages\langchain_core\callbacks\__init__.py", line 21, in <module>
    from langchain_core.callbacks.manager import (
  File "G:\pro_personal\LLMServer\.venv\lib\site-packages\langchain_core\callbacks\manager.py", line 29, in <module>
    from langsmith.run_helpers import get_run_tree_context
  File "G:\pro_personal\LLMServer\.venv\lib\site-packages\langsmith\__init__.py", line 10, in <module>
    from langsmith.client import Client
  File "G:\pro_personal\LLMServer\.venv\lib\site-packages\langsmith\client.py", line 43, in <module>
    import orjson
  File "G:\pro_personal\LLMServer\.venv\lib\site-packages\orjson\__init__.py", line 3, in <module>
    from .orjson import *
ModuleNotFoundError: No module named 'orjson.orjson'

Error Message and Stack Trace (if applicable)
File "G:\pro_personal\LLMServer\text_splitter_init_.py", line 1, in
from .chinese_text_splitter import ChineseTextSplitter
File "G:\pro_personal\LLMServer\text_splitter\chinese_text_splitter.py", line 1, in
from langchain.text_splitter import CharacterTextSplitter
File "G:\pro_personal\LLMServer.venv\lib\site-packages\langchain\text_splitter.py", line 2, in
from langchain_text_splitters import (
File "G:\pro_personal\LLMServer.venv\lib\site-packages\langchain_text_splitters_init_.py", line 22, in
from langchain_text_splitters.base import (
File "G:\pro_personal\LLMServer.venv\lib\site-packages\langchain_text_splitters\base.py", line 23, in
from langchain_core.documents import BaseDocumentTransformer, Document
File "G:\pro_personal\LLMServer.venv\lib\site-packages\langchain_core\documents_init_.py", line 6, in
from langchain_core.documents.compressor import BaseDocumentCompressor
File "G:\pro_personal\LLMServer.venv\lib\site-packages\langchain_core\documents\compressor.py", line 6, in
from langchain_core.callbacks import Callbacks
File "G:\pro_personal\LLMServer.venv\lib\site-packages\langchain_core\callbacks_init_.py", line 21, in
from langchain_core.callbacks.manager import (
File "G:\pro_personal\LLMServer.venv\lib\site-packages\langchain_core\callbacks\manager.py", line 29, in
from langsmith.run_helpers import get_run_tree_context
File "G:\pro_personal\LLMServer.venv\lib\site-packages\langsmith_init_.py", line 10, in
from langsmith.client import Client
File "G:\pro_personal\LLMServer.venv\lib\site-packages\langsmith\client.py", line 43, in
import orjson
File "G:\pro_personal\LLMServer.venv\lib\site-packages\orjson_init_.py", line 3, in
from .orjson import *
ModuleNotFoundError: No module named 'orjson.orjson'
Description
I think it may be due to certain problems with the orjson import package.
There should be a circuit breaker mechanism to load the json module to support operation when orjson cannot be loaded normally.
System Info
platform windows
python 3.8.9
langchain version 0.1.12

It should be a bug in langchainsmith. After reading it, langchainsmith seems to have fixed this problem.


Package Docs in Download

For Bootstrap v1.4 you were very kind and packaged all the documentation along with the framework in one easy zip file. I thought you might do the same for v2.0, so downloaded the zip file again but this time it contained no documentation. Could you please package it again? I can't be the only one that works offline enough that they need to be able to download it with the rest of the resources.

If you download the ZIP from GitHub directly, you'll get the docs. If you click the download button on our docs or use the Customize page, you'll get just the assets.
I'll talk to @fat about handling this better in 2.1.

Fantastic, thanks for the swift reply! Even just adding a link to "download with documentation" to the homepage and a link to the github download url would be good for other people


Fix truncated body issue

This PR is closed and replaced with https://github.com/nblockchain/conventions/pull/40 because some commits needed to be squashed.


[Feature request] Make fields important

It would be good a feature like:
const schema = z.object({   id: z.number().positive().important(),   name: z.string().min(5),   ... });
That would stop/break all the validation if has an error in the chain order.
It could also be used like:
const schema = z.object({   id: z.number().positive(),   name: z.string().min(5),   ... }).important({   id: true });
So for example if the field id has an error the field name would not be validated, is a good idea when you have for example refines that depend on top data, hope this can be implemented, Thanks. :)

If anyone wants this feature in the meanwhile I made this modification in my fork (it also has labels feature):
https://github.com/ElYaiko/zod/blob/master/index.ts
Can be used like:
const schema = z.object({
  id: z.number().positive(),
  name: z.string().min(5),
  ...
}).important({ id: true });

Code is not 100% good but it works well :)


Patch eventlet under Sentry SDK

This patches the eventlet library to allow it to run without issues under the Sentry SDK when using greenlet>=0.5. Eventlet doesn't pin the greenlet dependency and greenlet has been supporting contextvars for some time now (that's why the sdk runs nicely under Gevent for example). So, the eventlet library itself is not a good marker to whatever the contextvars are broken or not.
This should fix #1036

Sorry for stepping in without invitation, I wish they removed "contextvars don't work with eventlet" warning from docs.

Sorry for stepping in without invitation, I wish they removed "contextvars don't work with eventlet" warning from docs.
Everyone is welcome haha!
Yes, I think if this patch gets approved they should remove the warning since the default behavior right now is to flag the contextvars broken on the sdk and use locals.

@rhcarvalho @lobsterkatie @iker-barriocanal @AbhiPrasad would you please look at this? Awesome patch. The problem worked around with _is_contextvars_broken function has long been fixed in greenlet lib.

Hey @guiscaranse and @temoto !
Thanks for the great work.
I think this is a important contribution to improve Sentry with async Python. I will try to review and test this

Any updates on this? We would love to have this functionality in so we can move our deployment to eventlets!

Hey @guiscaranse
Sorry for taking soo long. I think it looks good.
Could you please rebase your branch on master again (or merge master into it)
So we have an up to date PR and can let the tests run properly? Would be amazing!

Any updates on this ?

Hey @guiscaranse, @temoto and @ouss1919 !
Sorry again that this one fell through the cracks. Could one of you rebase again on master, so the CI checks ran run and then the changes will be merged automatically!
Thanks!

Hey @guiscaranse I need to be annoying again. Could you merge master again into this branch? Then the tests should be all green. Thanks!

Hey @guiscaranse, would it be possible to set the PR as open to edits from maintainers? Then we could do the missing rebasing/linting changes ourselves without having to bother you all the time. :)

We have opened a new PR (#2464), which contains the same commits as this PR. We are updating the new PR with the latest changes from master so that these changes can get merged.

Closing, since these changes have been merged in #2464