Adobe doesn't load

Hi guys,
This package seems just what my project needs!
Unfortunately, adobe fonts do no load.
N.B> Adobe doesn't load in the playground either...
Hope it will be sorted,
kind regards,
Marco

Would you provide a reproduction? Unfortunately Stackblitz does not always work due to CORS.

Hi Daniel, thank you for the swift reply & you great work on Nuxt!
A reproduction (on stackblitz) would probably also cause CORS issues.
I have used the fonts module on my testdomain though. You can find it @ www.trauminfo.de
This is my nuxt.config.ts:
export default defineNuxtConfig({
  app: {
    head: {
      htmlAttrs: {
        lang: 'nl'
      }
    }
  },
  site: {
    url: 'https://www.trauminfo.de',
    name: 'Droominfo.nl',
    defaultLocale: 'nl'
  },
  nitro: {
    moduleSideEffects: [
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>    ],
  },
  modules: [
    ["nuxt-ssr-lit", { litElementPrefix: ["md-"] }],
    '@nuxtjs/tailwindcss',
    '@vueuse/nuxt',
    'nuxt-lodash',
    "@nuxtjs/sitemap",
    "@nuxtjs/google-adsense",
    "@nuxt/fonts"
  ],
  fonts: {
    adobe: {
      id: ['ddx0usu'],
    },
    experimental: {
      processCSSVariables: true
    }
  },
  googleAdsense: {
    id: 'ca-pub-7034225815334900',
    test: true
  },
  tailwindcss: {
    cssPath: '~/assets/css/droominfo.css',
    exposeConfig: false,
  },
  routeRules: {
    '/': { prerender: true },
  }
});

Snippet from my tailwind.config:
  theme: {
    extend: {
      fontFamily: {
        sans: ['ff-tisa-sans-web-pro', 'sans-serif'],
        serif: ['ff-tisa-web-pro', 'serif'],
      },

Example implementation:
<p class="font-sans mb-4 text-blue-600 ...">
N.B. This is the correct project id (ddx0usu), since this works on my main domain @ www.droominfo.nl
kind regards,
Marco Troost

Snippet from my tailwind.config:
  theme: {
    extend: {
      fontFamily: {
        sans: ['ff-tisa-sans-web-pro', 'sans-serif'],
        serif: ['ff-tisa-web-pro', 'serif'],
      },

We only match the names of fonts. ff-tisa-sans-web-pro is a slug, so it won't be matched. You could try using FF Tisa Sans Pro and see how it goes.

Hi Qwerzl,
I've tried FF Tisa Sans Pro, but this doesn't work either.
N.B. The slug is the documented implementation as given by Adobe:

When i look in the .nuxt folder, nuxt-fonts-global.css is empty:

Hope these screenshots are of help.
kind regards, Marco

Ah, I understand where the problem is. Currently we match fonts with its name from the Typekit API to check if it exists, then we fetch the CSS and get the @font-face definitions whose family-name is the font's slug. But for this font, its slug (ff-tisa-web-pro) and its font-family name defined in the CSS (ff-tisa-web-pro) are not the same.


Make default "Move messages" configuration context-dependent

At present, the "Move messages" menu defaults to "Move this and all following messages". When moving the first/last message or the last message in a topic, this is actually equivalent to choosing one of the other options, in terms of which messages get moved. In these cases, we should set a different option as the default, as it will help us better manage automatically shifting which topics a user has followed/unmuted to a new place when appropriate (and not otherwise).
So, in priority order:

When moving the first message in a topic, make the default: "Move all messages in this topic".
When moving the last message in a topic, make the default: "Move only this message".

(So for topics with just one message, (1) will apply.)

@zulipbot claim


Update auth in the token contract to match soroban-auth

@dmkozh I think this was done, is that correct?

Yeah, I think so.


Abandoned dependency

Composer remarks:
Package guzzle/common is abandoned, you should avoid using it. Use guzzle/guzzle instead.
Package guzzle/stream is abandoned, you should avoid using it. Use guzzle/guzzle instead.
Package guzzle/parser is abandoned, you should avoid using it. Use guzzle/guzzle instead.
Package guzzle/http is abandoned, you should avoid using it. Use guzzle/guzzle instead.
Package guzzle/plugin-cookie is abandoned, you should avoid using it. Use guzzle/guzzle instead.

Would it be possible that you'd update your composer.json accordingly?

Thanks a lot!


⚠ Bofrost.at has degraded performance

In 745534a, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1278 ms

Resolved: Bofrost.at performance has improved in 26d5cee after 6 minutes.


feat: flagd flag evaluation metadata

This PR
Related to flagd change at https://github.com/open-feature/flagd/pull/761
Handles evaluation metadata from flagd and fills required OF SDK field

Merging as all conversations and suggestions are applied :)


Add Stremio web support

https://www.reddit.com/r/StremioAddons/s/J63gc8jrld

The url returned by the addon is a private url which requires authentication and redirects the player to the actual public video url via the Location header in the http response. If we want to support web we must extract this public url from the response and return the resulting public url instead.

Important tot note that the inner public stream should be extracted on a per-stream basis. Resolving all of them beforehand (before returning the streams object that is) would take too long.
We likely need to convert from the stremio-addon-sdk to a custom express.js (or similar) setup in order to support this.

web interface support would be awesome


Map IP space for new hub and spokes

In order to create the new hub and spoke environments, we need to decide on how to carve up the CIDRs and document this
Acceptance Criteria

[x] GIVEN that there may be several new hub and spokes in the future, we need to map the TF_VAR_vpc_cidr and TF_VAR_smtp_ingress_cidr_blocks each new hub and spoke will use

Security considerations
This is part of the effort to split hub and spokes accounts and will increase isolation of resources
Implementation sketch

[x] Create a document/spreadsheet of the cidr ranges each hub and spoke will have and reserve ip ranges for future growth

Mapping these with https://docs.google.com/spreadsheets/d/1oF8w6Tme2hYdcZCgW1Y7jq-Okmfu7CXvtCIGjkfFa4Q/edit#gid=0


Release 0.6.12

Specifically to add peek for TcpStream.

This can be closed since 0.6.13 is already out, even.


without any Router, dynamic loading the component base address not start from '/' will cause errors

🐛 Bug Report
   without any Router, dynamic loading the component seems using the last base URL, it cause the address errors to link the code splitting component

To Reproduce
using a gif to explain it : two component, if we navigate the path start from '/'

component: form1    path: '/test1'
component: form2    path:'/test2'
.... then  it work fine.
but if we navigate the path start from '/ab'
then:
component: form1    path: '/ab/test1'
component: form2    path:'/ab/test2'
then: when start from web base address.  the first loading component work fine. the second loading error.   because it start from 'website/ab/test1/codesplit address
we using this way to dynamic load component without Routes
import React from 'react';
import _ from 'lodash';
import memoized from 'nano-memoize';
import MicroApp from '@/components/MicroApp'
import Pageloading from '@/components/PageLoading'
import loadable from '@loadable/component';

const Dashboard = loadable(() => import('../pages/dashboard'), {
  fallback: <Pageloading tip="组件加载中..." />,
});
const Test1 = loadable(() => import('../pages/test1'), {
  fallback: <Pageloading tip="组件加载中..." />,
});
const Test2 = loadable(() => import('../pages/test2'), {
  fallback: <Pageloading tip="组件加载中..." />,
});
const Test3 = loadable(() => import('../pages/test3'), {
  fallback: <Pageloading tip="组件加载中..." />,
});

const getPage = memoized((apageStr) => {
  const pageStr = _.upperFirst(apageStr);
  switch (pageStr) {
    case 'Test3':
      return <Test3 />
    case 'Test2':
      return <Test2 />
    case 'Dashboard':
      return <Dashboard />
    case 'Test1':
      return <Test1 />
    // 默认没有，则用微端的方式加载试试。
    default:
      return <MicroApp entry={apageStr} />
  }
});
export default getPage;

Check your publicPath configuration. It is webpack who loads the script chunk on request, and this is webpack setting.

Check your publicPath configuration. It is webpack who loads the script chunk on request, and this is webpack setting.

no config with this; i'm checking now,thanks a lot！soon feedback

Check your publicPath configuration. It is webpack who loads the script chunk on request, and this is webpack setting.

it works perfect now. thanks a lot!


Simplify all path types to use similar code

🎁 Pull Request

[x] Used a clear / meaningful title for this pull request
[ ] Tested the changes in your own code (on your projects)
[ ] Added / Edited tests to reflect changes (tst folder)
[x] Have read the Contributing part of the Readme
[x] Passed npm test

Fixes

N/A
Why have you made changes?

For easier maintenance of code in the future. All path types now have very similar code, so creating / modifying path types should be easier. For example, this will make implementing #57 across all types in P significantly easier.
What changes have you made?

simplified all path types to use similar code

What tests have you updated?
None.
Is there any breaking changes?
No - no tests or public APIs have been changed.
Anything else worth mentioning?

Pick is quite different to the other P types, and the code reflects that.

thanks @mcpower, this looks great


Update pe-format.md: Forward slash vs. backward slash in .libs.

@jaykrell : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Karl-Bridge-Microsoft : Thanks for your contribution! The author(s) have been notified to review your proposed change.


[cleanup][broker]remove unnecessary variable in OwnershipCache

Motivation
Variable bundleFactory_ in OwnershipCache is not used.
Documentation

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

Matching PR in forked repository
PR in forked repository: https://github.com/lordcheng10/pulsar/pull/16
After opening this PR, the build in apache/pulsar will fail and instructions will
be provided for opening a PR in the PR author's forked repository.
apache/pulsar pull requests should be first tested in your own fork since the
apache/pulsar CI based on GitHub Actions has constrained resources and quota.
GitHub Actions provides separate quota for pull requests that are executed in
a forked repository.
The tests will be run in the forked repository until all PR review comments have
been handled, the tests pass and the PR is approved by a reviewer.
-->

Please combine this to #17768 , it doesn't make sense to submit these one-by-one.

OK


Issue-130: Align OH / IEAM docs

Description
This PR attempts to synchronize as much of the Open Horizon / IEAM production documentation as possible.
Merges were done in both directions. Last Updated fields were synced.
Each modified markdown file is in a separate commit.  That way we can discuss, reject, modify any changes I've proposed, individually.  I won't have to drag along all the other modifications.
This will also prevent a massive merge conflict when the PR is accepted. (Smaller merge conflicts may occur and will be easier to resolve)
Fixes #130
Type of change

[x ] This change requires a documentation update

How Has This Been Tested?
I wrote a edge-doc-compare.sh bash script which scanned across both cloned repos.
Compared the .md files and, if there were differences, loaded them into meld (a file merge/compare tool available on Linux)
I inspected the files and synchronized as much as possible.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] I have checked my code and corrected any misspellings
[x] I have tagged the reviewers in a comment below incase my pull request is ready for a review
[x] I have signed the commit message to agree to Developer Certificate of Origin (DCO) (to certify that you wrote or otherwise have the right to submit your contribution to the project.) by adding "--signoff" to my git commit command.

Too many commits for a single PR 😅 @johnwalicki can you please stash these commits into a single commit?

I'm a strong believer in small, bite size commits which fix one thing.  Each commit in this PR is one file worth of merges between the OH/IEAM documentation.  There are 55+ files changed, so, close to 60 commits.  A jumbo PR would be indigestible.

I'm a strong believer in small, bite size commits which fix one thing. Each commit in this PR is one file worth of merges between the OH/IEAM documentation. There are 55+ files changed, so, close to 60 commits. A jumbo PR would be indigestible.

got it :)

It looks like we only have three comments left to resolve

@joewxboy - Thanks for the comments/suggestions.  I've done some minor fix ups and punted others to track in separate issues.  Not perfect but definitely highlights where the docs need more work.  I recommend we merge this monster boulder and chip away at the smaller rocks.


chore: Clean up paladin function

Fixes #27
Summary

I personally feel mapping the logic into an object of functions would make the code less readable. I have simplified the current logic, but it is in a 'step-by-step' flow (eg functional), you can see that each block of code if attempting. to set a specific header. Would love to here some thoughts on this

yea this file is so small that abstracting out wouldn't make any sense IMO. i dont think it's hard to read as it is now. i like the step by step approach.


ERROR ANALYSIS: A basic breakdown of skills by ESCO's structure and by regions

[brief] A basic breakdown of skills by ESCO's structure and by regions.
Remember: ESCO has some skills in multiple parts of the taxonomy, so might need to make decisions about this/double count?

which skills need manually editing (e.g. teach communication to communication)?
what are % of skills not matched?
what are the unmatched ones?

notes:

are we actually deduplicating extracted_skills?

this was explored in the last sprint, closing this issue


[CT-3274] Remove usage of dbt.contracts.connections in dbt/adapters

Closed by https://github.com/dbt-labs/dbt-core/pull/8906/commits/469a9aca06037a46bcf607ea70b4305d72ee6dd1


Is it possible to update schema once created?

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the SQLModel documentation, with the integrated search.
[X] I already searched in Google "How to X in SQLModel" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to SQLModel but to Pydantic.
[X] I already checked if it is not related to SQLModel but to SQLAlchemy.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
class Xconnect(SQLModel, table=True):
    id: int = Field(primary_key=True)
    lvc: Optional[str] = Field(sa_column_kwargs={"unique": True})
    customer: int = Field(foreign_key="customer.id")
    region: int = Field(foreign_key="attr.id")

Description
Im trying to see if I can update the exisitng generated table Xconnect without deleting and recreating it,
for example, if I wantto make the Region field Optional, adding
region: Optional[int]

will not update the existing schema bcs the table is already created. How can I update the existing schema? Do I need to use Alembic for this?
Operating System
Linux
Operating System Details
linux mint 19
SQLModel Version
0.0.5
Python Version
3.8
Additional Context
No response

As you've mentioned, Alembic would be a good solve.


Deployment

##User Story
Creating a portfolio of work I can use to showcase my skills and talents to employers looking to fill a part-time or full-time position.
##Acceptance Criteria
When I visit the website using the URL.
When I can access my website from any browser.

Access to website using URL and on other browsers


Add Rx, Ry, Rz gates

This PR creates separate Rx, Ry, Rz gates.

Discussed in cirq sync: we should implement __pow__ on these R* gates so they stay as rotation gates when exponentiating.

@maffoo Exponentiating R* gates work as expected because _with_exponent is already overloaded. Added explicit tests for the same.

@balopat Do you want to take another look or should I merge?


Data quality issue with CVE-2022-0685

The CVE ID
CVE-2022-0685 is missing from OSV (it seems like it used to be there but was removed?)
Describe the data quality issue observed
CVE-2022-0685 is a valid CVE ID but it is no longer found in the OSV database.
Suggested changes to record
CVE-2022-0685 should be added back to OSV.

@andrewpollock can you help take a look at this one?

This is very strange.
https://cve-osv-conversion.storage.googleapis.com/osv-output/CVE-2022-0685.json exists, and we have had some historic challenges with vim in particular and analysis of Git commits, but by the looks of running analysis manually on this record, it's not causing a crash.
Based on the modified time of the OSV record, it was last modified on 2024-06-04T04:00:19Z, so we don't have importer logs from that far back, so I'm triggering a manual reimport of this record to see how it gets treated now...
Separately, I'll also do a cross-reference of all CVEs in gs://cve-osv-conversion/osv-output to get a sense of how many aren't importing successfully at present.

Forcing a reimport of this record appears to have been successful: https://api.osv.dev/v1/vulns/CVE-2022-0685

(as a note, because this record was absent from the database, the normal approach of using poetry run tools/datafix/reimport_gcs_record.py --verbose --no-dry-run --project oss-vdb CVE-2022-0685 didn't work, all I had to do was gcloud storage objects update gs://cve-osv-conversion/osv-output/CVE-2022-0685.json to make the importer see the file as new)


EAGLE-530 Fix checkstyle problems on eagle-alert module and enable failOnViolation

https://issues.apache.org/jira/browse/EAGLE-530

Merged into a0fc870849a5fa5d54c0980b21226977eef81920


Add documentation for the code

Fixes #288

LGTM - thanks for putting this together.

Thanks @alex-r-bigelow, this is great!

Let me know if you have any questions / want more details about anything.


Update README.md to center header

This commit changes the header of the README to be center-aligned

Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

This doesn't seem like an improvement to me, we'd prefer to keep minimalistic markdown where possible, not using HTML unnecessarily.

I agree, I prefer the current style.  Thank you for putting time into this though.


Blog post with ATO Team Lineup

published and scheduled on social media http://ncopenpass.com/blog/teams-are-selected-for-next-step-in-datapalooza-open-data-competition/


sbt-native-packager is not being used for now

As long as we are not using sbt-native-packager, we are removing it from the plugin configuration, just to avoid transitive conflict in the libraries using the sbt-microsites plugin.
In contrast, we are using a piece of code that is useful within our scope, hence we extracted it from that project, and, the LICENSE has been included as the license of this project.
Please, @raulraja could you take a look? Thanks!

Makes sense. LGTM


Updates

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Hey there, can you take a look-see at https://github.com/NixOS/nixpkgs/issues/108568#issuecomment-755364223?

Hey there, can you take a look-see at https://github.com/NixOS/nixpkgs/issues/108568#issuecomment-755364223?

Now I think it is done. I need to learn more about wrappers!

Now I think it is done. I need to learn more about wrappers!

Result of nixpkgs-review pr 108589 1

1 package built:

germinal

Result of nixpkgs-review pr 108589 1

1 package built:

germinal

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108589 run on x86_64-linux 1

1 package built:

germinal

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108589 run on x86_64-linux 1

1 package built:

germinal

Result of nixpkgs-review pr 108589 run on x86_64-darwin 1

1 package marked as broken and skipped:

germinal

Result of nixpkgs-review pr 108589 run on x86_64-darwin 1

1 package marked as broken and skipped:

germinal


Allow thickness of underline to be 1

This PR makes a small refactor if DecorationRendere.cpp and allows underline thickness to be equal to 1.
I think that this can close https://github.com/contour-terminal/contour/issues/663
And refs https://github.com/contour-terminal/contour/issues/1242

we agreed to leave it like this, because std::ranges::iota requires a start parameter, which we do not need here.


Scroll issue 1.17.2 OSX

Prerequisites

[ X ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
I can't scroll left pane with trackpad on mac osx v. 10.12.14. I just don't see the scrollbar, but if jump between files with arrows everything is ok.
In previous version 1.16.0 x64 scroll on trackpad works as expected.
Steps to Reproduce

Open atom in full window
Open much projects and open many folder trees
Try to scroll with trackpad (two fingers), and nothing happens

Expected behavior: Scroll left pane content
Actual behavior: No scroll at all
Reproduces how often: on new version 1.17.2 every time
Versions
$ atom --version Atom    : 1.17.2 Electron: 1.3.15 Chrome  : 52.0.2743.82 Node    : 6.5.0
Additional Information
Same issue happens on 1.18-beta.
But works as expected on previous stable release 1.16
$ atom --version Atom    : 1.16.0 Electron: 1.3.13 Chrome  : 52.0.2743.82 Node    : 6.5.0
osx version
Darwin MacBook-zh.local 16.5.0 Darwin Kernel Version 16.5.0: Fri Mar  3 16:52:33 PST 2017; root:xnu-3789.51.2~3/RELEASE_X86_64 x86_64

Unable to reproduce on Windows.

I can't reproduce either on macOS 10.12.15 with Atom 1.17.2.

Open atom in full window

Just to confirm, do you mean fullscreen here?  If so, does that mean scrolling works ok if you're not fullscreen?
Also, are you using the Atom Material theme by any chance @phstoned?

I have the same issue.
Atom 1.17.2 x64
macOS 10.12.4 (16E195)
Theme: Atom Material

Change the theme and it works again

Thanks @SimonFricker, it looks like that issue is fixed in Material 2.0 /cc https://github.com/atom-material/atom-material-ui/issues/420.

@rsese It works neither fullscreen nor in window. I just thought it only fullscreen problem but then I did testing and the issue happens no matter how window is opened.
yes, I'm using Atom material.
so is it theme issue, not atom?

yes, this is definitely theme problem. Atom Dark is somehow broken.
Changed theme from Atom Dark to Atom Light and scrolling works well.

Can you clarify @phstoned, you mentioned both:

yes, I'm using Atom material.

But also:

Atom Dark is somehow broken.

Do you mean Material in this sentence ☝ or did you have a scrolling problem with Atom Dark as well?

@rsese I used these

I think the problem is in UI theme.

Gotcha, thanks :+1:  So it does look like you're hitting https://github.com/atom-material/atom-material-ui/issues/420, and they say it's fixed as of Material 2.0 so we'll go ahead and close this out.


Add git commands and descriptions to README.md

@Rinthm you and I just worked on this, can you please review it!


Failed npm run build by adding code snippet with SQL comments */

Description
The npm run build command fails if you have a code snippet with SQL comments:
hugo v0.101.0-466fa43c16709b4483689930a4f9ac8add5c9f66+extended windows/amd64 BuildDate=2022-06-16T07:09:16Z VendorInfo=gohugoio
Error: Error building site: MINIFY: failed to transform "index.js" (application/javascript): unexpected \u003e\u003c in expression on line 130 and column 866
  130: .../\r\n\r\n\u003c/code\u003e\u003c/pre\u003...

Steps to reproduce
Clone a new site as described:
git clone https://github.com/h-enk/doks-child-theme.git my-doks-site npm install
Check 'npm run build' command works - It does.
Edit any docs page and add a code block with the following:
/* This is a comment */

npm run start will work OK but npm run build will fail.
Expected result
The build command should be successful
Actual result
hugo v0.101.0-466fa43c16709b4483689930a4f9ac8add5c9f66+extended windows/amd64 BuildDate=2022-06-16T07:09:16Z VendorInfo=gohugoio
Error: Error building site: MINIFY: failed to transform "index.js" (application/javascript): unexpected \u003e\u003c in expression on line 130 and column 866
  130: .../\r\n\r\n\u003c/code\u003e\u003c/pre\u003...

Environment
><EMAIL_ADDRESS>precheck
> npm version

{
  'doks-child-theme': '0.5.0',
  npm: '8.15.0',
  node: '16.17.0',
  v8: '<IP_ADDRESS>-node.22',
  uv: '1.43.0',
  zlib: '1.2.11',
  brotli: '1.0.9',
  ares: '1.18.1',
  modules: '93',
  nghttp2: '1.47.0',
  napi: '8',
  llhttp: '6.0.7',
  openssl: '1.1.1q+quic',
  cldr: '41.0',
  icu: '71.1',
  tz: '2022a',
  unicode: '14.0',
  ngtcp2: '0.1.0-DEV',
  nghttp3: '0.1.0-DEV'
}

><EMAIL_ADDRESS>check
> exec-bin node_modules/.bin/hugo/hugo version

hugo v0.101.0-466fa43c16709b4483689930a4f9ac8add5c9f66+extended windows/amd64 BuildDate=2022-06-16T07:09:16Z VendorInfo=gohugoio

Thank you. ❤

This is the same issue as #716.
Adding this to config_default\config.toml fixes the problem
[minify]
  disableJS = true


Use GH actions for lint and unit tests

Running unit and component tests separately leads to a problem combining their coverage reports. This PR should also move unit tests to the GitHub Actions job and make them run together with the component tests. The comment from the codecov bot below confirms that the bug is resolved.

Codecov Report
Base: 0.00% // Head: 87.08% // Increases project coverage by +87.08% :tada:

Coverage data is based on head (3e7ba49) compared to base (133dfa2).
Patch has no changes to coverable lines.

:exclamation: Current head 3e7ba49 differs from pull request most recent head 82fb4fa. Consider uploading reports for the commit 82fb4fa to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           master     #446       +/-   ##
===========================================
+ Coverage        0   87.08%   +87.08%
===========================================
  Files           0       25       +25
  Lines           0     1154     +1154
  Branches        0      442      +442
===========================================
+ Hits            0     1005     +1005
- Misses          0      149      +149

Impacted Files
Coverage Δ

src/Components/Modals/DisableRule.js
94.44% <0.00%> (ø)

src/Components/MessageState/MessageState.js
100.00% <0.00%> (ø)

src/Components/Common/Tables.js
88.72% <0.00%> (ø)

src/Services/Filters.js
100.00% <0.00%> (ø)

src/Utilities/Rule.js
100.00% <0.00%> (ø)

src/Services/SmartProxy.js
40.00% <0.00%> (ø)

src/Components/MessageState/EmptyStates.js
92.59% <0.00%> (ø)

src/Components/Labels/RuleLabels.js
100.00% <0.00%> (ø)

src/Components/ClusterRules/ClusterRules.js
86.22% <0.00%> (ø)

src/Components/RecsListTable/RecsListTable.js
80.28% <0.00%> (ø)

... and 15 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

:tada: This PR is included in version 1.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Java version compatibility

"Quartz 2.4.0 RC1 now requires minimum Java version of Java 8" - from release notes, but I got "class file has wrong version 55.0, should be 52.0" error that means I should use minimum Java 11. Same for Quartz 2.5.0 RC1. Сould you please clarify this point?
https://stackoverflow.com/questions/9170832/list-of-java-class-file-format-major-version-numbers

Also 2.5.0 RC1 does not seem to be present on the Release Page :(  https://github.com/quartz-scheduler/quartz/releases

It is present in maven repositories though: https://central.sonatype.com/artifact/org.quartz-scheduler/quartz/2.5.0-rc1/overview
Here's a comparison: https://github.com/quartz-scheduler/quartz/compare/v2.4.0-rc2...v2.5.0-rc1
Seems that project is heading towards javax -> jakarta for 2.5.x

Quartz 2.5.0-rc1 is indeed compiled with minimal JDK target version 11. It cannot be used anymore in JDK 8 projects.
If that is the intention, the manifest in the jar file should not state

Manifest-Version: 1.0
Implementation-Title: quartz
Implementation-Version: 2.5.0-rc1
Built-By: jenkins-slave
Implementation-Vendor-Id: org.quartz-scheduler
Built-JDK: 1.8.0_322


Improve titles

The original titles only contain a generic reference of the satellite where the image came from and a time stamp much like  Flood Vectors - MODISAqua. It would be better to have something more descriptive so our users can identify what they are interested in downloading quickly.

Dataset titles have been improved to include the description of what the file is about. Here is an example: Tropical Cyclone analysis for Philippines (08 December 2014). I'll be closing this issue for now until better suggestions emerge.


SAK-29545 Removed hard coded block on presence logging.

Wasn't the idea here that you might want to display users present but not actually have SiteStats collect them?

I have no idea. All I know is that there is some ropey hard coded false in there still. That needs refactoring if it's to be switched on in a different way.

I'll refactor, test, and update the PR. I prefer not using spring setters, though.


Develop a mechanism to pass cluster-dependent arguments during a module orchestration

Problem statement:
Manager selects a module based on M4DModule set running on a coordinator cluster. The module arguments include some values that are cluster dependent, such as URL of some instance to connect to in a selected cluster, thus can not be configured in a M4DModule directly.
A possible solution would be to apply a local instance of the module in a remote cluster that will include the specific values, and use the local instance to override the values during the orchestration.

Requirements:
We want to propagate some of the decisions to the local cluster level (Blueprint, Blueprint controller running in the remote cluster).
Another requirement is to enable an easier module development when there is no forced connection between module arguments and the manager knowledge of the module (we do not want to change module arguments structure).
Proposed solution:

M4DModule is deployed only in coordinator cluster.
Optionally, spec of the module will allow attach environment similar to the way it works with pods.

spec:
...
  env:
    - name:
       value:
   envfrom:
      - configMapRef:
             name:
       - secretRef:
             name:

The first part (env) is used for variable defaults - add a variable with its default value for all possible deployments. Names/Values are propagated through blueprint as an addition to module arguments. It is optional.
The second part (envfrom) provides instructions to the BlueprintController where to look for cluster-specific values. Configmap/secret names are propagated to the BlueprintController, the values take precedence over all module arguments including env part. Optional as well.
Example: a module receives jobNumber, url, username and password arguments.
M4DModule spec will look like:
env:
  - name: jobNumber
  - value: 1
envfrom:
   - configMapRef:
      name: module-config
    - secretRef:
      name: module-secret

A local cluster will deploy a module-config configmap with jobNumber and url variables, module-secret with username and password. If the configmap does not include jobNumber, the default value 1 will be taken.

@shlomitk1 how does the BlueprintController will get the instructions on where to look for cluster-specific values? Will the Blueprint contain it? Your suggestion seems to answers the requirements as far as I understand them. Unfortunately, I am not familiar with all Kubernetes features so I do not have an alternative approach for a solution if such exists.

@shlomitk1 how does the BlueprintController will get the instructions on where to look for cluster-specific values? Will the Blueprint contain it? Your suggestion seems to answers the requirements as far as I understand them. Unfortunately, I am not familiar with all Kubernetes features so I do not have an alternative approach for a solution if such exists.

Blueprint will contain the environment directives (name of the configmap, secret). The resources themselves (configmap,secret) will be deployed in advance in remote clusters in m4d-system.

Do we really need something from manager to support this? Can't it be part of the module itself?

Do we really need something from manager to support this? Can't it be part of the module itself?

We need:

A declaration of what a module needs (to avoid mistakes of a module requiring one resource and a cluster admin deploying another).
It would be nice to provide defaults and having an ability to deploy module even when the resources are partially (or not at all) deployed in the remote cluster.
A centralized way to manage the module environment rather than having each module developer address these issues.

I guess that what I don't understand is what functionality is added that would not be achieve by putting these in the existing spec.charts.values (with no code changes). Can you explain please?

I guess that what I don't understand is what functionality is added that would not be achieve by putting these in the existing spec.charts.values (with no code changes). Can you explain please?
Edit: I understand that there could be some special functionality, e.g., around validation. I'm just trying to get it clearer because it's not obvious to me.

If M4DModule is deployed only in the coordinator cluster, we can't use the existing spec.charts.values to reflect settings specific to remote clusters. That's why we need to deploy different environments on different cluster, and let the module specify with what environment it works.

For example
spec:
...
  chart:
    values:
    - "jobNumber": "1"
    - "configMapRef": "module-config"
    - "secretRef": "module-secret"

Ok. My concerns were regarding reading secrets/configmaps. Each module can address these issues on its own, of course.


MicrosoftWindowsEventFullDeployment Using Wrong Input Parameter for ASimNetworkSessionMicrosoftWindowsEventFirewall Template

Describe the bug
What broke a few things:
5 days ago (Today is May 7th, 2022), several input parameters were renamed from Workspace Region to WorkspaceRegion
Commit: https://github.com/Azure/Azure-Sentinel/commit/0702fcb46686d9ce5a44741f0ebad30341a76fec
The ASimNetworkSessionMicrosoftWindowsEventFirewall Template Parameters Changed
This template had a parameter named Workspace Region. However, it is now called WorkspaceRegion.
https://github.com/Azure/Azure-Sentinel/blob/master/Parsers/ASimNetworkSession/ARM/ASimNetworkSessionMicrosoftWindowsEventFirewall/ASimNetworkSessionMicrosoftWindowsEventFirewall.json
Other Templates such as MicrosoftWindowsEventFullDeployment Affected
Other templates call / deploy the  ASimNetworkSessionMicrosoftWindowsEventFirewall template with the old parameter name Workspace Region.
https://github.com/Azure/Azure-Sentinel/blob/master/Parsers/ASim WindowsEvent/ARM/MicrosoftWindowsEventFullDeployment.json#L108
This Breaks Projects Using the MicrosoftWindowsEventFullDeployment  Template
https://github.com/OTRF/Microsoft-Sentinel2Go/issues/24
To Reproduce
Simply call this template and it should fail
https://github.com/Azure/Azure-Sentinel/blob/master/Parsers/ASim WindowsEvent/ARM/MicrosoftWindowsEventFullDeployment.json
Expected behavior
Templates to deploy ASIM parsers without any problem.
Additional context
Can we pick one parameter name for everything location related. Currently several ASIM parser ARM templates use parameter names such as:

Location
WorkspaceRegion
Workspace Region

This is why several other templates break the moment you change one name because you have to update other internal parent templates calling it.

Fixed in this PR https://github.com/Azure/Azure-Sentinel/commit/0eefa3ef465c9ead80d64cecc03397e950001b6b


Add mutator-rails to mutate code

Improve test suite.

Currently the wiki has instructions on running mutant, but the results are all in one log file. This would split them up by module.

adding mutator_rails is not easy as the dummy app is inside of the specs area. Should make this a low priority


fix: Fix range out of index error with a temporary workaround

Which issue does this PR close?

Closes #540.
Rationale for this change

What changes are included in this PR?

How are these changes tested?

I will add a custom Java importer implementation to use the fixed offset field from arrow-rs.

Looks like I need to copy more Arrow Java classes to make it work...

Looks like I need to copy more Arrow Java classes to make it work...

Instead of a real fix which requires adding offset support to Arrow Java, I added a hacky workaround in my arrow-rs repo, and added custom Java importer implementation.
I re-ran the query and verified this workaround fixes the range out of index error.

cc @andygrove This uses a temporary workaround to fix the issue. I'm not sure how long it will take to have the Arrow Java fix, so I think a workaround but not a real fix might be good for us to move forward.

Looks like I need to copy more Arrow Java classes to make it work...

Instead of a real fix which requires adding offset support to Arrow Java, I added a hacky workaround in my arrow-rs repo, and added custom Java importer implementation.
I re-ran the query and verified this workaround fixes the range out of index error.

I also ran the benchmarks and can confirm that this fixes the issue.

I proposed the workaround fix to arrow-rs at https://github.com/apache/arrow-rs/pull/5958.

@andygrove Could you verify again if the latest change still resolves the issue? I changed it to use the commit proposed for arrow-rs https://github.com/apache/arrow-rs/pull/5958 behind a feature flag.

Codecov Report
Attention: Patch coverage is 0% with 192 lines in your changes missing coverage. Please review.

Project coverage is 53.61%. Comparing base (2fc45a2) to head (b593d96).
Report is 14 commits behind head on main.

Files
Patch %
Lines

...g/apache/arrow/c/CometBufferImportTypeVisitor.java
0.00%
132 Missing :warning:

...in/java/org/apache/arrow/c/CometArrayImporter.java
0.00%
58 Missing :warning:

...rc/main/java/org/apache/arrow/c/ArrowImporter.java
0.00%
2 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##               main     #584       +/-   ##
=============================================
+ Coverage     34.13%   53.61%   +19.47%
+ Complexity      809      792       -17
=============================================
  Files           106      105        -1
  Lines         38586     9904    -28682
  Branches       8566     1867     -6699
=============================================
- Hits          13172     5310     -7862
+ Misses        22674     3642    -19032
+ Partials       2740      952     -1788

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

I changed the arrow-rs repo to use the change at https://github.com/apache/arrow-rs/pull/5964. See if everything in CI can pass.

@viirya I confirmed that this fixes the issue when I run the benchmarks locally.

@viirya I confirmed that this fixes the issue when I run the benchmarks locally.

Thank you, @andygrove. Then we can merge this and wait for new arrow-rs and DataFusion releases.

@andygrove This uses official arrow-rs repo now.

Thanks @andygrove


Issue with format Wide

Hello
I have two cases :
The first it is working well  with l'option wide, I have the aggregation by partition name

An second case it doesn't working with the same options , i don't have  aggregation by data source

Do you have any idea what the problem is?
regards

Hello @Davonex
We're using https://pkg.go.dev/github.com/grafana/grafana-plugin-sdk-go/data#LongToWide
Maybe An error is returned if any of the following are true: The input frame is not a long formatted time series frame. The input frame's Fields are of length 0. The time index is not sorted ascending by time. The time index has null values. is a reason ? like not sorted by time ? one of my team mate also reported maybe it expects one entry per time interval per label, a missing group by clause could then be a reason ?
You could try first to sort by time and then to group by time at the needed interval and let us know :)
Thank you!

Hello
Sorry for my late response
I tried by sorting the date and  indeed i get an aggregation
Thank your  for your advise .


Using sampler2D as global_uniform in a shader crashes engine on startup of fresh project

Godot version
4.0.beta1
System information
Mac OS Monterey 12.6 and Windows 10
Issue description
When opening a freshly cloned copy of a project with a shader using a global sampler2d uniform the engine crashes immediately without error messages
Steps to reproduce

Create a shader with a global uniform of type Sampler2D
Create a material for that shader
Save the material
Apply it to a mesh
Copy or clone the project
Open the project

The crash does not occur on the copy of the project you created the shader in
Minimal reproduction project
GodotBetaGlobalUniformCrashReproductionProject.zip
Alternatively clone this repo: https://github.com/Piet-G/BotanicaGame.git

After some more tests, it seems that it is still crashes when the sampler isn't even global. This seems like a major bug

Probably some importing bug - you can fix it by deleting the .godot folder before running the project.

It can't be that, the .godot folder isn't checked in in git and it breaks even when opening it from a fresh clone

Related to https://github.com/godotengine/godot/issues/62941.

I think it's fixed (running the attached project does not crash). Probably fixed by https://github.com/godotengine/godot/pull/69600.


Create repository configuring Workflows with 'Read and write'

The problem
Octoherd repositories are scaffolded with a release GitHub workflow (.github/workflows/release.yml), which uses semantic-release to manage the releases to NPM. This workflow requires the repository to be configured to grant workflows with 'Read and Write' permissions.
But by default, a repository is created with the configuration 'Read repository contents and packages permissions'. This makes new Octoherd scripts scaffolded not work out of the box when trying to do their first release.

Proposed solution
To update the repository creation logic and set this required configuration out of the box. GitHub offers a way to update this via API: https://docs.github.com/en/rest/actions/permissions?apiVersion=2022-11-28#set-default-workflow-permissions-for-a-repository
Questions
What would be the first choice if there is an existing parent configuration at org/user level?
a) To respect that parent configuration
b) To still apply the Read and Write for that octoherd repository, ignoring what's configured at org/user level.

I didn't know about that API, very cool!


Network Connection Drops

I've been running a new node on v1.7.1 for a few days now. My blockchain synced OK for the first few days, but after I installed apps yesterday my node has been seemingly randomly dropping the connection to my LAN (and WAN too, I think). This makes it so that I can't SSH into the node. I don't even see the node on my list of connected devices from my router.
In order for me to get my node working again, I must unplug the power cable and plug it back in to force a reboot (I feel terrible about this with SSD read/write operations).
I have tried looking through the raspiblitz debug logs and the syslog to try and find an answer as to why the connection drops, but so far have been unable to figure anything out.
Linked is my debug log from my currently working node: https://termbin.com/0mol
A couple of observations:

My blockchain is currently indexing after installing Mempool Explorer and BTC-RPC Explorer. This seems to lose progress after the network connection goes down. Said another way, I don't see much progress in the percentage indexed after I reboot the machine.
The rasberry pi is running really hot (~82C). I'm using the case that came with the Rasberry Pi 4 starter kit that I bought, but I bought a case that should cool the CPU better coming on Monday.

Please let me know if you have any other ideas on things to check. Thanks.

Does the screen say "waiting for dhcp". I have reported a similar issue under the following link https://github.com/rootzoll/raspiblitz/issues/2640#issue-1028652829.

The rasberry pi is running really hot (~82C).

Can you check what process is heatingh the CPU? Check command "htop" on command line.

Mostly it's just bitcoind that is using the most CPU. A few days ago I installed the rasberry pi in case that manages heat a lot better and that has drastically improved the performance of the node. It's hovering right around 50C now. Much better temps.
I almost want to certainly say this issue was caused by CPU overheating during the blockchain indexing after installing mempool and btc rpc explorer. It's probably safe to close this ticket.


Node 18.16.0

It's deprecated now.
This is what Vercel tells us for other projects:

Error: Node.js version 16.x has reached End-of-Life. Deployments created on or after 2023-08-15 will fail to build. Please set "engines": { "node": "18.x" } in your package.json file to use Node.js 18.

Looks like the tests didn't like the upgrade :(


Compatibility with the plugin MODE

Friend, hello. I suggest to make compatibility with the plugin MODE (2x2) https://dev-cs.ru/resources/87/ and new mode 2x2 https://dev-cs.ru/resources/630/

I think that having some amxx module for yapb management (like forcing bots to use one bombsite or forcing other things bots will do randomly) will be good idea, but developing it will be time consuming thing so no one will do it

There is one, but very limited and archived atm: https://github.com/yapb/amxx
As for 2x2 mode, well, it's hard to bot to know where virtual walls are set. I'll look at it at sometime in future.

Possible solution: when installing walls on the map (such as Mode2x2), it is necessary to implement support for loading other (new) navigation files for bots.
Add a cvar like prefix to custom.cfg, when loading\saving the graph, use it as <map_name>.
But this will not solve the problem of changing the online mode: as soon as there are more players, closed passages will open, but cvar will not change.


mako: Missing jq dependency for makoctl utility

Describe the bug
Some features of makoctl require jq to work.
Currently
Steps To Reproduce
Steps to reproduce the behavior:

...
...
...

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
output here

See #217756
Oops.


[draft] Add support for virtio

This PR enables realm to use virtio (e.g., file system access) by allowing shared pages to be accessed by the Host. With this PR, realm-linux will be able to boot to the shell with islet.
About fault redirection
linux-cca controls the accesses to the encrypted/decrypted page with the bit 33 in Realm's page table entry (the bit location can be configured by the user). The previous RSI_REALM_CONFIG returns an incorrect value which resulted in improper fault generation. For example, the access to a decrypted page (e.g., 0x184c00000) always triggered a fault for its encrypted page (e.g., 0x84c00000), while it should have been treated as the decrypted page. With the fix, the two different kind of accesses can be distinguished and handled properly.
Using level in page table traversing
In page table traversing API, (input) level and (output) level are added which is useful for the fine-grained traversing. (input) level indicates the intended (maximum) level to reach, while (output) level is the actual level reached (e.g., If the user passes 3 for (input) level and there are upto level 2 page tables for the given address, it would return 2 for the (output) level).
Additional Notes
Currently, islet's page table traversing (e.g., ipa_to_pa()) in default checks its validity with bit 0. However, this cannot be done in stage 2 page table, as bit [0-1] is used as description type, and valid page table entry has that bit on.
In this PR, ipa_to_pte() is added and this function does not conduct the bit 0 based validity check (here no_valid_check: true is passed in entry()) and returns the whole entry's value.
Also, page table entry modification in default attaches unintended flags which is not compatible with stage 2 table entry (e.g., RIPAS, HIPAS field). To avoid incorrect bits to be set, I've added is_raw parameter in PTE modification APIs, which when set as true, only sets the flags that the users have provided.
Remaining issue

TF-A failure case (will be resolved in another PR)

Not ready for review, but I want to say that this is a huge step forward! Great work in advance!

Not ready for review, but I want to say that this is a huge step forward! Great work in advance!

Thank you for the comment. Now, it gets ready!


does it secure beyond communication between application and LDAP provider?

One key question:
With Secure LDAP, will the application still be able to see the password user enters? My guess is yes, just like to confirm it. That means encrypting communication between application and LDAP provider. On the situation many applications implement different SSO with same ID provider, (secure) LDAP is still not secure enough because technically the password of a high level user might be intercepted by 3rd party application, unlike SAML/OAuth which only issues tokens.

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

ID: 41c4b68f-055e-3983-68dd-fc37c3f72da9
Version Independent ID: 0008043c-8703-85b4-ff09-3a087a65c29b
Content: Configure Secure LDAP (LDAPS) in Azure AD Domain Services
Content Source: articles/active-directory-domain-services/active-directory-ds-admin-guide-configure-secure-ldap.md
Service: active-directory
GitHub Login: @eringreenlee
Microsoft Alias: ergreenl

@ryangugcloudca
Thanks for your feedback! We will investigate and update as appropriate.

With LDAP you would enable password hash, so the application sees the hashed version of the password. The Active Directory domain service stores passwords in the form of a hash value representation, of the actual user password.
https://docs.microsoft.com/en-us/azure/active-directory/hybrid/how-to-connect-password-hash-synchronization
Your domain would be susceptible to brute force password attacks if you do not enable the proper NSGs. https://docs.microsoft.com/en-us/azure/active-directory-domain-services/active-directory-ds-admin-guide-configure-secure-ldap-enable-ldaps
CC: @eringreenlee

We will proceed to close this thread. Please let me know if you have any additional questions. Feel free to tag me in the comments if you would like to continue the conversation.

Thank you for the reply.
When users enter password into a LDAP application, the application is able to see the clear text password. This is  the reason I prefer SAML or OAuth.


Flask 1.0 backwards incompatible root url_prefix

Some of the recent PRs in blueprint.py seem to make Flask 1.0 not backwards compatible with 0.x versions.  When url_prefix='', url_prefix='/', or url_prefix is not specified, a ValueError('urls must start with a leading slash') is now raised.
Expected Behavior
I would have assumed, given the discussion in #2629, that '' and maybe / would be acceptable url_prefixes to indicate the blueprint is meant for the root.
An example of a broken library is flask-sitemap (https://github.com/inveniosoftware/flask-sitemap/blob/master/flask_sitemap/init.py#L120) which uses
            app.register_blueprint(
                self.blueprint,
                url_prefix=app.config.get('SITEMAP_BLUEPRINT_URL_PREFIX') # default is '/'
            )

Actual Behavior
In my testing, I continually get
    self.init_app(app)
  File "/home/albertyw/.virtualenvs/baseflask/local/lib/python2.7/site-packages/flask_sitemap/__init__.py", line 120, in init_app
    url_prefix='' # app.config.get('SITEMAP_BLUEPRINT_URL_PREFIX')
  File "/home/albertyw/.virtualenvs/baseflask/local/lib/python2.7/site-packages/flask/app.py", line 64, in wrapper_func
    return f(self, *args, **kwargs)
  File "/home/albertyw/.virtualenvs/baseflask/local/lib/python2.7/site-packages/flask/app.py", line 1113, in register_blueprint
    blueprint.register(self, options, first_registration)
  File "/home/albertyw/.virtualenvs/baseflask/local/lib/python2.7/site-packages/flask/blueprints.py", line 186, in register
    deferred(state)
  File "/home/albertyw/.virtualenvs/baseflask/local/lib/python2.7/site-packages/flask/blueprints.py", line 207, in <lambda>
    s.add_url_rule(rule, endpoint, view_func, **options))
  File "/home/albertyw/.virtualenvs/baseflask/local/lib/python2.7/site-packages/flask/blueprints.py", line 79, in add_url_rule
    view_func, defaults=defaults, **options)
  File "/home/albertyw/.virtualenvs/baseflask/local/lib/python2.7/site-packages/flask/app.py", line 64, in wrapper_func
    return f(self, *args, **kwargs)
  File "/home/albertyw/.virtualenvs/baseflask/local/lib/python2.7/site-packages/flask/app.py", line 1211, in add_url_rule
    rule = self.url_rule_class(rule, methods=methods, **options)
  File "/home/albertyw/.virtualenvs/baseflask/local/lib/python2.7/site-packages/werkzeug/routing.py", line 603, in __init__
    raise ValueError('urls must start with a leading slash')
ValueError: urls must start with a leading slash

unless I set url_prefix='//', in which case the url_prefix is the root.
Environment

Python version: python 2.7 and 3.6
Flask version: 1.0
Werkzeug version: 0.14.1

The real error here is that Flask-Sitemap doesn't use a leading slash for the rules, and relied on the prefix being prepended. Flask-Sitemap should be fixed to use an empty prefix and start the rules with leading slashes.


Guidelines checking

As a user, I want to have the guidelines of the fCM design support team to be checked. Starting with one examples guideline as a reference implementation. The exact guidelines are still to be decided and not part of this issue.

[ ] example guidelines is choosen
[ ] guidelines checking is implemented
[ ] guidelines violations are shown in the error list
[ ] pushed to develop

@ma-ro looks done to me with the "meaningful labels" guideline, right?


[Task]: Move GeoLocation APIs to worker

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
Currently, geolocation APIs are implemented in such a way that the worker sends a message to the main thread for access to any value, and later on, values are sent back to the worker.
It is now possible to implement geolocation API in the worker itself which would avoid the communication to the main thread.

@Rishabh-Rathod could you share some estimated performance gains or outcomes for this task?

This won't be possible as the WorkerNavigator class doesn't have access to geolocation APIs.


Normalize ModuleIdentifier

Context
This commit reworks the ComponentModuleIdentifier/ComponentModuleSelector/ModuleVersionSelector
classes to use ModuleIdentifier under the hood, instead of storing denormalized strings. This has
the advantage that we can reduce the use of the module identifier factory, which is called very
often during dependency resolution. Sharing instances reduces the need for conversions, and makes
comparisons faster.
The PR is huge because a lot of tests had to be updated to take normalization into account. I tried to make use of the factory methods as much as possible, which shows there's still room for improvement. However, it also highlights a number of inconsistencies in the way "nullability" was handled. Unfortunately, I couldn't streamline everything (like, making sure you never pass null) because some places in the code do expect null with a different semantics. It would be nice to change, but it's quite hard for backwards compatibility.

The result of these changes is particularly visible on the "large dependency graphs" : https://builds.gradle.org/viewLog.html?buildId=13627351&buildTypeId=Gradle_Check_PerformanceTestCoordinator&tab=report_project944_Performance

@adammurdoch I'm assigning you for review here because I'm not quite sure if there is an historical reason why the ModuleIdentifier wasn't used in a composition model, that is to say denormalized like it was before this PR.

I'm curious where and why we still use the factory now? Is it still pulling its weight?

It's still used in places where the cost is worth it: during metadata parsing, or in module selector notation parsing. Those are not called too often, and it's nice to have interning there, because it reduces the cost of equals during dependency resolution.

@adammurdoch I'm optimistically merging this, let me know if there's any issue with exposing more ModuleIdentifier.


Detectron2 external projects

Hey @jagin,
I would like to ask you how can I connect Detectron2 external projects(like this one: https://github.com/youngwanLEE/centermask2) to the pipeline?
Thanks!

@Auth0rM0rgan I was able to quickly look at that project and for me it looks like you should be able to use CenterMask models without any problem. Try to use --config-file option to load the model you need.

Hey @jagin,
I have tried to run the process_video.py with the absolute path to the centermask config file but I am getting this error
This is my command:
python process_video.py -i 0 -p -d  --config-file /home/XXX/centermask2/configs/centermask/centermask_V_39_eSE_FPN_ms_3x.yaml
This is the error I am getting:
Traceback (most recent call last):   File "process_video.py", line 155, in <module>     main(args)   File "process_video.py", line 91, in main     cpu=False if args.gpus > 0 else True)   File "/home/media4us/PycharmProjects/detectron2-pipeline/pipeline/utils/detectron.py", line 9, in setup_cfg     cfg.merge_from_file(config_file)   File "/home/media4us/PycharmProjects/detectron2/detectron2/config/config.py", line 47, in merge_from_file     self.merge_from_other_cfg(loaded_cfg)   File "/home/media4us/anaconda3/lib/python3.7/site-packages/fvcore/common/config.py", line 121, in merge_from_other_cfg     return super().merge_from_other_cfg(cfg_other)   File "/home/media4us/anaconda3/lib/python3.7/site-packages/yacs/config.py", line 217, in merge_from_other_cfg     _merge_a_into_b(cfg_other, self, self, [])   File "/home/media4us/anaconda3/lib/python3.7/site-packages/yacs/config.py", line 460, in _merge_a_into_b     _merge_a_into_b(v, b[k], root, key_list + [k])   File "/home/media4us/anaconda3/lib/python3.7/site-packages/yacs/config.py", line 473, in _merge_a_into_b     raise KeyError("Non-existent config key: {}".format(full_key)) KeyError: 'Non-existent config key: MODEL.VOVNET'
Thanks!

I'm not able to help you with that. First I would check if your model is working with the demo sources of centermask2 project. Maybe there is something missing in my pipeline considering centermask2.
Good luck!

Well, I was able to get rid off the KeyError: 'Non-existent config key: MODEL.VOVNET' by:

cloning vovnet-detectron2 into detectron2-pipeline project.
rename vovnet-detectron2 to vovnet_detectron2
Then edit detectron2-pipeline/pipeline/utils/detectron.py
Add and import statement above
from vovnet_detectron2.vovnet import add_vovnet_config
Then call add_vovnet_config() inside setup_cfg() as:

import torch
from detectron2.config import get_cfg
from vovnet_detectron2.vovnet import add_vovnet_config
def setup_cfg(config_file, weights_file=None, config_opts=[], confidence_threshold=None, cpu=False):
    # load config from file and command-line arguments
   cfg = get_cfg()
   add_vovnet_config(cfg)
   ...
   ...


pyLODE-ready RDF

updating the ontology metadata to be able to document it with pyLODE
NOTE: merge this only after merging PR #5

Thanks for merging. I was struggling to work out how to deal with the conflicts. Gotta learn sometime though!


Rotations and regions in plots

IceNet version: 0.2.3

Description
After fixing #149 it was apparent that the northern hemisphere region setting isn't intuitive. Southern hemisphere does allow zooming in but you can't move the region (plot and data extents calculate out differently)
process_regions will not be helping this, but basically the extent calculations need testing with respect to the data and cartopy. The rotation of the axis needs checking as well.
We may want to look at the latlon idea again for CLI - I don't know what happened to that issue!? Did that get raised @ots22 @rchan26?
What I Did
icenet_plot_forecast -l 1 -f png south results/predict/2023-03-04_south.nc 2023-03-04

# and then try some regions
# -r 0,0,432,432 works
# -r 0,0,400,400 shifts projection outside data extent
# -r 150,150,282,282 zooms in (equal differences from boundaries)
# -r 100,100,282,282 deviates both projection and data differently

Northern hemisphere is easier to see. Just plot the whole hemisphere then try and take the "top left" of quadrant (you'll get the "bottom left")

Ensured that the process_regions method is working adequately for different forms of data


[W socket.cpp:663] [c10d] The client socket has failed to connect to [DESKTOP-H2DRQRJ]:62468 (system error: 10049)

how to close distributed?
I only have 1 GPU, so I need to change the code, I change the parameter in xxx.yaml, GPUs [1] to GPUs[0].
But there may have some error: [W socket.cpp:663] [c10d] The client socket has failed to connect to [DESKTOP-H2DRQRJ]:62468 (system error: 10049).
I search it online, it seems like it usually come at distributed training with GPUs.
I only have 1 GPU. So how can I run the code??
Wait for your help!

by the way, my system is windows10, not linux

You should change the num_workers: 0 as well as gpus: [0] in XXX.yaml


Issue #1213 fix (RFC 9207)

The fix for issue #1213 is implemented:

added AuthConfig.authorizationResponseIssParameterSupported (defaults to true)
added OidcDiscoveryDoc.authorization_response_iss_parameter_supported (takes precedence over AuthConfig)
added iss param validation in OAuth Authorization Response for Authorization Code flow and Implicit flow

I am waiting for merged/accepted.

we are also waiting for a fix

Waiting for this fix

Also waiting for this fix

same

Hi @buchatsky,
your PR is popular, so I'd like to merge it.
Do you want to rebase it on current master, or should I do it myself?


Polish tremor-runtime API for better embeddability

Signed-off-by: Heinz N. Gies<EMAIL_ADDRESS>Pull request
Description
Publish nessessary API from tremor-runtime to allow embedding the runtime in other applications.
Also polish some of the public interface.
Checklist

[x] The RFC, if required, has been submitted and approved
[x] Any user-facing impact of the changes is reflected in docs.tremor.rs
[x] The code is tested
[x] Use of unsafe code is reasoned about in a comment
[x] Update CHANGELOG.md appropriately, recording any changes, bug fixes, or other observable changes in behavior
[x] The performance impact of the change is measured (see below)

Performance
-/-

:poland:

Ja clippy and coverage, fixing that right now :)


Export document incorrect

URL for relevant page?https://docs.influxdata.com/influxdb/v1.7/tools/influx_inspect/#export
What products and version are you using?influx_inspect.exe
Where did you look before opening the issue?documentation and other locations on the interwebs.
In the information, it says that the timestamp has to be in RFC3339, which links another area in influx documentation (https://docs.influxdata.com/influxdb/v1.7/query_language/data_exploration/#absolute-time). However, it actually requires a more formal RFC3339. The format that worked in the end is: 2019-01-25T00:00:00+00:00 without single quotes.

Thank you @justinbrock! Nice catch.
We added examples of valid RC3339 formats to the export document, and removed the link to #absolute time, as you noticed, that example reflects using single quotes when RFC3339 is used as part of a query. That said, we'd expect single quotes to be consumed by the shell (not cause issues).
We're curious if you'd share sample syntax of what didn't work (we'd love to help others know what to watch out for). Thanks again!

This is what eventually worked.
influx_inspect.exe export -out C:\export\export.txt -datadir F:\influxdb\data -waldir F:\influxdb\wal -database XXX -compress -start 2019-01-25T00:00:00+00:00 -end 2019-02-25T00:00:00+00:00
I tried multiple variations that were listed on that RFC page, but none of them worked. Not 100% sure why, but I know this worked.
I attempted with and without quotes this format 'YYYY-MM-DDTHH:MM:SS.nnnnnnnnnZ' I tried with the Z, without the Z, with the n's and without the n's.
I also tried this format 'YYYY-MM-DD HH:MM:SS.nnnnnnnnn'

Thank you so much @justinbrock!
For RC3339, Z (or +0.00) is required for no time zone offsets.
.nnnnnnnnn represents fractional seconds, which are optional for RC3339.
To be sure, did you try this format (and it didn’t work):
YYYY-MM-DDTHH:MM:SSZ

That information was not included in either the influx_inspect section or the section in the RFC help area.
I did try that format and it didn't work either.

@justinbrock yes sir, it was not included and confusing. Thanks to you, we've updated examples in our influx_inspect docs.
If you run into future issues using the suggested RC3339 formats in our docs, please send us your error. In your specific case, we'd expect 2019-02-25T00:00:00Z to work.


How to get path of selected file?

This library is awesome, it helps me a lot.
The thing is that I want a path of selected file/image/video whatever it is.
In my project, it is asking for path/location of a file. Can you please tell me how to get path?

Glad to hear you find this library useful!
The file path is not exposed through the current implementation, and I don't think we're going to change that. You can fork this library and modify the upload delegate method here return an NSURL instead of NSData.
If you look at these three lines where we call upload, it's always pulling the NSData from an NSURL, so it's easy to swap out that delegate call:
https://github.com/helpscout/HSAttachmentPicker/blob/master/HSAttachmentPicker/Classes/HSAttachmentPicker.m#L133
https://github.com/helpscout/HSAttachmentPicker/blob/master/HSAttachmentPicker/Classes/HSAttachmentPicker.m#L179
https://github.com/helpscout/HSAttachmentPicker/blob/master/HSAttachmentPicker/Classes/HSAttachmentPicker.m#L215


Datasheet for developers cable

Is there a datasheet for using the Sensel morph with the developers cable?
I'm particularly interested in expected power for each of the lines, i.e. is it expected that TX/RX are running at 5V, as USB requires a range between 4.75V and 5.25V, and of course if there are any hard timings for the sending and receiving data.

Yes - it is expected 5V. You can use a logic level converter like this for 3.3V systems
https://www.sparkfun.com/products/12009

Thanks, I've got this all working now with SM32, there was a small issue with the connection that was causing the line to drop below 3.3v. I've now resolved this.
Is there documentation for registers read/writes for how much data they return/expect?


Add new install instructions to README.md

This is not ready to be merged until brew cask merges our PR

Does this look any nicer? The html table formatting lets you set the widths of the columns

[Homebrew](http://brew.sh)
Manual
Rubygems

macOS
macOS
macOS or Linux with Ruby 2.0.0 or above

brew cask install fastlane
Download the zip file). Then double click on the install script (or run it in a terminal window).
sudo gem install fastlane -NV

Yes, it does! Thanks @asfalcone !

https://github.com/caskroom/homebrew-cask/pull/26868


Set a non-empty User-Agent

Some repositories have a policy that rejects requests with an empty User-Agent value.
This change sets the User-Agent to a valid non-empty value.

Would it be possible to get a review?


Created CRUD operation for News Backend

hey @cb7chaitanya have a review on this request .

@ShikharSinghKushwaha Add the req-res you get after testing the api endpoint so that we can verify that the endpoint is working

i didn't get what you are asking for @cb7chaitanya ,i mean on making any request the desired response is showing in postman ,what type of other req  res you want ,please specify

@ShikharSinghKushwaha you must have tested the endpoints using postman or a similar api testing client, right?
I just want the request body/params(If any) and the corresponding json response you get when testing an endpoint


[PTX-2232] Added logic to validate applications restored by backup driver

Jenkins job run where backup test passed:
https://jenkins.portworx.co/job/tp-backup-vinayak/107/console
Jenkins job run, where other tests passed:
https://jenkins.portworx.co/job/Torpedo/job/tp-dev-test/489/console


Downloaded YouTube video doesn't match advertised bitrate

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.05.08
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

$ youtube-dl -F https://youtu.be/A7p4pJ_mPe8
[youtube] A7p4pJ_mPe8: Downloading webpage
[info] Available formats for A7p4pJ_mPe8:
format code  extension  resolution note
249          webm       audio only tiny   57k , opus @ 50k (48000Hz), 5.63MiB
250          webm       audio only tiny   77k , opus @ 70k (48000Hz), 7.47MiB
140          m4a        audio only tiny  131k , m4a_dash container, mp4a.40.2@128k (44100Hz), 16.14MiB
251          webm       audio only tiny  162k , opus @160k (48000Hz), 15.25MiB
278          webm       256x144    144p   97k , webm container, vp9, 30fps, video only, 11.78MiB
160          mp4        256x144    144p  112k , avc1.4d400c, 30fps, video only, 10.85MiB
242          webm       426x240    240p  223k , vp9, 30fps, video only, 24.59MiB
133          mp4        426x240    240p  302k , avc1.4d4015, 30fps, video only, 24.44MiB
243          webm       640x360    360p  412k , vp9, 30fps, video only, 46.04MiB
134          mp4        640x360    360p  619k , avc1.4d401e, 30fps, video only, 50.45MiB
244          webm       854x480    480p  759k , vp9, 30fps, video only, 76.00MiB
135          mp4        854x480    480p  904k , avc1.4d401f, 30fps, video only, 78.75MiB
247          webm       1280x720   720p 1515k , vp9, 30fps, video only, 136.40MiB
136          mp4        1280x720   720p 1535k , avc1.4d401f, 30fps, video only, 113.41MiB
248          webm       1920x1080  1080p 2670k , vp9, 30fps, video only, 320.56MiB
137          mp4        1920x1080  1080p 4503k , avc1.640028, 30fps, video only, 400.26MiB
18           mp4        640x360    360p  726k , avc1.42001E, 30fps, mp4a.40.2@ 96k (44100Hz), 90.55MiB
22           mp4        1280x720   720p 1038k , avc1.64001F, 30fps, mp4a.40.2@192k (44100Hz) (best)

youtube-dl -v https://youtu.be/A7p4pJ_mPe8
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://youtu.be/A7p4pJ_mPe8']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.05.08
[debug] Python version 2.7.15+ (CPython) - Linux-4.4.0-18362-Microsoft-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 3.4.6, ffprobe 3.4.6, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] A7p4pJ_mPe8: Downloading webpage
[debug] Default format spec: bestvideo+bestaudio/best
WARNING: Requested formats are incompatible for merge and will be merged into mkv.
[debug] Invoking downloader on u'https://r4---sn-4gxx-hgnz.googlevideo.com/videoplayback?expire=1589724260&ei=BPDAXo6dDdfH1wbRt7yQBQ&ip=<IP_ADDRESS>&id=o-AJp54YGoR_Xjnx1i_DMa6Kou_QiIqdvpIoR7zYN0JSbK&itag=137&aitags=133%2C134%2C135%2C136%2C137%2C160%2C242%2C243%2C244%2C247%2C248%2C278&source=youtube&requiressl=yes&mh=7M&mm=31%2C29&mn=sn-4gxx-hgnz%2Csn-25ge7nsd&ms=au%2Crdu&mv=m&mvi=3&pl=14&initcwndbps=585000&vprv=1&mime=video%2Fmp4&gir=yes&clen=419700809&dur=1045.844&lmt=1586623707739236&mt=1589702615&fvip=4&keepalive=yes&fexp=23882514&c=WEB&txp=5535432&sparams=expire%2Cei%2Cip%2Cid%2Caitags%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&sig=AOq0QJ8wRAIgQIT8ZiSC1n_AljBuV2m_dwyoQcSBqx_aAQNfFbT95IACIBY8T0-W1g01YBmt_3kXmWPR0qnTOOnJbOZs2-xuNmh4&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRQIgOEO4FAFaoBRPNMtgHV0xM3BFT0oZQjWwUY-avtg-ddsCIQCuyIbP_FBu3PhfiMB6f96atRSZB2Fcg0JOpxAOwUhgyA%3D%3D&ratebypass=yes'
[download] Destination: ASMR From My Backyard 🌞 (Relaxing Personal Attention)-A7p4pJ_mPe8.f137.mp4
[download] 100% of 400.26MiB in 00:45
[debug] Invoking downloader on u'https://r4---sn-4gxx-hgnz.googlevideo.com/videoplayback?expire=1589724260&ei=BPDAXo6dDdfH1wbRt7yQBQ&ip=<IP_ADDRESS>&id=o-AJp54YGoR_Xjnx1i_DMa6Kou_QiIqdvpIoR7zYN0JSbK&itag=251&source=youtube&requiressl=yes&mh=7M&mm=31%2C29&mn=sn-4gxx-hgnz%2Csn-25ge7nsd&ms=au%2Crdu&mv=m&mvi=3&pl=14&initcwndbps=585000&vprv=1&mime=audio%2Fwebm&gir=yes&clen=15995764&dur=1045.881&lmt=1586560166892588&mt=1589702615&fvip=4&keepalive=yes&fexp=23882514&c=WEB&txp=5511222&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&sig=AOq0QJ8wRQIhAO31tS4Gp7tKG5AN_CBQQTRZJtThy623dzV2TDQ3U_z1AiBB3K4fOIt2Jo5oZzLpBK5fXhsx_XXsf2ENiH1XjaVlRA%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRQIgOEO4FAFaoBRPNMtgHV0xM3BFT0oZQjWwUY-avtg-ddsCIQCuyIbP_FBu3PhfiMB6f96atRSZB2Fcg0JOpxAOwUhgyA%3D%3D&ratebypass=yes'
[download] Destination: ASMR From My Backyard 🌞 (Relaxing Personal Attention)-A7p4pJ_mPe8.f251.webm
[download] 100% of 15.25MiB in 00:02
[ffmpeg] Merging formats into "ASMR From My Backyard 🌞 (Relaxing Personal Attention)-A7p4pJ_mPe8.mkv"
[debug] ffmpeg command line: ffmpeg -y -loglevel 'repeat+info' -i 'file:ASMR From My Backyard 🌞 (Relaxing Personal Attention)-A7p4pJ_mPe8.f137.mp4' -i 'file:ASMR From My Backyard 🌞 (Relaxing Personal Attention)-A7p4pJ_mPe8.f251.webm' -c copy -map '0:v:0' -map '1:a:0' 'file:ASMR From My Backyard 🌞 (Relaxing Personal Attention)-A7p4pJ_mPe8.temp.mkv'
Deleting original file ASMR From My Backyard 🌞 (Relaxing Personal Attention)-A7p4pJ_mPe8.f137.mp4 (pass -k to keep)
Deleting original file ASMR From My Backyard 🌞 (Relaxing Personal Attention)-A7p4pJ_mPe8.f251.webm (pass -k to keep)

ffprobe on video from a month ago
Input #0, matroska,webm, from 'ASMR From My Backyard (Relaxing Personal Attention).mkv':
  Metadata:
    COMPATIBLE_BRANDS: iso6avc1mp41
    MAJOR_BRAND     : dash
    MINOR_VERSION   : 0
    ENCODER         : Lavf57.83.100
  Duration: 00:17:25.86, start: -0.007000, bitrate: 4320 kb/s

ffprobe on video downloaded today
Input #0, matroska,webm, from 'ASMR From My Backyard 🌞 (Relaxing Personal Attention)-A7p4pJ_mPe8.mkv':
  Metadata:
    COMPATIBLE_BRANDS: iso6avc1mp41
    MAJOR_BRAND     : dash
    MINOR_VERSION   : 0
    ENCODER         : Lavf57.83.100
  Duration: 00:17:25.88, start: -0.007000, bitrate: 3331 kb/s

Description

Downloading the highest quality available results in a lower bitrate than the one given in the formats list. Expected bitrate is around 4500k, actual bitrate is 3300k. The same video downloaded a month ago has the correct bitrate.
I am aware of issue #15180, though it is closed and is 2 years old, and since I managed to get the correct bitrate video a month ago I figured it might be a different problem.
Downloading the webm version as proposed would not work as the highest bitrate is the f137 one. Is there any other workaround?

Blame Youtube for reporting incorrect bitrate.

This is not an incorrect bitrate, a month ago the download had the indicated bitrate - which I put in the issue.

@dstftw did you even read the end of the issue?


Add support for mocking referrer in Request

@jparise is there anyone at Pinterest who would like to review as well?

@bishwei would be a good person.

Thank you @zackargyle for approving. I really appreciate if we can merge this PR soon and publish a new version of he package.

@sreedhar7 I no longer have admin rights to merge, but I've notified @bishwei and he said he would try to get to it later today. I'll make sure it gets published to NPM shortly after.

@bishwei I really appreciate if you can take a look at this PR and merge it today.

@sreedhar7 working on it right now, sorry for the delay

@sreedhar7 Just as a note, the last release was a fairly significant backwards-incompatible change to make it possible for Google's Workbox project to use it better. If you're not on 2.0.0 already, you will probably want to take a look at the pull request (#109) from @kommander.

Thanks for the note @zackargyle. We started using the package recently and are already on 2.0.0. Please let us know when the new build with the fix has been published.

@sreedhar7 published 2.0.3 which includes your change.  Thanks again for the contribution!

Thank you @bishwei and @zackargyle for your quick responses.


Disable test suites that contain no tests

I haven't looked closely at this, but I'm hopeful that some compile time can be saved by not building empty test suites and doc test suites.
I think cargo has some ability to control which targets are built for testing. I hope it can control which are doc-tested.
So the task here is to find all of the empty test suites run by cargo test --all and disable them in Cargo.toml by setting either test = false or doctest = false. Then time cargo test --all --no-run before and after.

Do you mean to add #[cfg(test)] to those test functions ?

@jswh no, it's not that there are test cases that are missing the attribute, but that there are test suites with no test cases at all. And compiling them costs time. There is though a PR open for this.

I did some search but only found the cargo test --all --exclude <spec>  command that can exclude specific test suites.
The empty test suties could be found by the test output. For example，the test_storage test suite has no cases as the output says
Running /mnt/c/Codes/third/tikv/target/debug/deps/test_storage-436a8955ad0b3b77

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

The list was found with a little script as

fuzz
bfuzz_targets
fuzzer_afl
fuzzer_honggfuzz
fuzzer_libfuzzer
panic_hook
test_raftstore
test_storage

And when I try to run tests with time cargo test --all --no-run --exclude fuzz fuzzer_afl fuzzer_honggfuzz fuzzer_libfuzzer panic_hook, est_raftstore test_storage it show some error
error: Found argument 'fuzzer_honggfuzz' which wasn't expected, or isn't valid in this context

I am not sure if it is what the wanted and what to do next.
Newbee here.

I did another test, and found that disabling unused test suites did nothing for compile time. It makes me wonder if there's a bug in cargo, since I turned off 29 test suites.
Here's my raw notes:
# 6/29

counting "0 test" suites

cargo test --all -- --list 2>&1 | tee ../all-tests.txt

rg "\d tests, \d* benchmarks" ../all-tests.txt

rg "\d tests, \d* benchmarks" ../all-tests.txt | wc -l

42 test / bench suites

rg "^0 tests, 0 benchmarks" ../all-tests.txt| wc -l

29 suites with no tests or benches

70% of test suites are empty

posted time-commits.sh https://gist.github.com/brson/14642390bfc05edf1fa2212ae7d609b4

branch: https://github.com/brson/tikv/tree/notest

cargo test --all --no-run full before
    Finished dev [unoptimized] target(s) in 12m 04s
cargo test --all --no-run partial before
    Finished dev [unoptimized] target(s) in 1m 24s
cargo test --all --no-run full after
    Finished dev [unoptimized] target(s) in 12m 16s
cargo test --all --no-run partial after
    Finished dev [unoptimized] target(s) in 1m 24s

It does seem like disabling the tests prevented cargo from building the test suites - there are lots of binaries I don't see afterward. I'm super surprised it didn't reduce build time.


EAS submit Invalid Apple ID was specified

Build/Submit details page URL
https://expo.dev/accounts/lucietrottier/projects/cute-v3/builds/6b68e099-d3ff-4501-a1e9-7f1986756e72
Summary
when i build all working well, but after when i want to submit i got this error Invalid Apple ID was specified. It should be a valid email address. Example<EMAIL_ADDRESS>i don't know the reason why, 2 days before i had no error and all working well, i never had this error before
Managed or bare?
manged
Environment
expo-env-info 1.2.0 environment info:
System:
OS: macOS 14.3
Shell: 5.9 - /bin/zsh
Binaries:
Node: 18.19.0 - ~/.nvm/versions/node/v18.19.0/bin/node
Yarn: 1.22.21 - ~/.nvm/versions/node/v18.19.0/bin/yarn
npm: 10.2.3 - ~/.nvm/versions/node/v18.19.0/bin/npm
Managers:
CocoaPods: 1.15.0 - /opt/homebrew/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 23.2, iOS 17.2, macOS 14.2, tvOS 17.2, visionOS 1.0, watchOS 10.2
IDEs:
Xcode: 15.2/15C500b - /usr/bin/xcodebuild
npmPackages:
expo: ^49.0.7 => 49.0.22
expo-router: ^2.0.0 => 2.0.14
react: 18.2.0 => 18.2.0
react-dom: 18.2.0 => 18.2.0
react-native: 0.72.6 => 0.72.6
react-native-web: ~0.19.6 => 0.19.10
npmGlobalPackages:
eas-cli: 7.1.2
Expo Workflow: managed
✔ Check Expo config for common issues
✔ Check package.json for common issues
✔ Check dependencies for packages that should not be installed directly
✔ Check for common project setup issues
✔ Check npm/ yarn versions
✔ Check Expo config (app.json/ app.config.js) schema
✔ Check for legacy global CLI installed locally
✔ Check that native modules do not use incompatible support packages
✔ Check that packages match versions required by installed Expo SDK
✔ Check that native modules use compatible support package versions for installed Expo SDK
Error output
[Error [ValidationError]: Invalid Apple ID was specified. It should be a valid email address. Example<EMAIL_ADDRESS>{
_original: { language: 'en-US', appleId: 'com.11h11media.cute-app' },
details: [
{
message: 'Invalid Apple ID was specified. It should be a valid email address. Example<EMAIL_ADDRESS>path: [Array],
type: 'string.email',
context: [Object]
}
]
}
Reproducible demo or steps to reproduce from a blank project

eas.json

Hi,
The con.11h11media.cute-app doesn't seem like a valid Apple ID value. Apple ID is the email address that you use to log in to your Apple Developer account. It seems like this is your bundle identifier.
BTW I think that in your current submit setup you can drop the appleId altogether from your eas.json and it should still work fine.

thank you !! all is working now, i just removed like you said
thanks !

thank you !! all is working now, i just removed like you said
thanks !

hi @szdziedzic , Im having the same issue , should I just removed the apple ID value all together from my eas file ? Thanks

I'm seeing this after updating to v7.1.3 yesterday but for Apple Team ID. Never had an issue before.


support SafeAreaView bottom margin

Fix for issue: https://github.com/facebook/react-native/issues/18177
Modal component does not support SafeAreaView bottom margin

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

https://github.com/facebook/react-native/pull/23501#issuecomment-464498339

It doesn't seem like that is the right fix and it is a breaking change as well. Your PR also doesn't conform to the template, and there is no explanation about why this fixes the issue so I'm gonna close this PR for now. Feel free to make a similar modification in your project.

You're right, this is not the correct approach to solving the issue, I will try another way.


chore: Run CI on non-dev Python 3.12

Codecov Report

Merging #87 (94ca178) into main (69feff3) will not change coverage.
Report is 2 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #87   +/-   ##
=======================================
  Coverage   99.95%   99.95%
=======================================
  Files         868      868
  Lines       22742    22742
  Branches     1491     1491
=======================================
  Hits        22732    22732
  Misses          5        5
  Partials        5        5

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more