When Heading 1 is selected, all other heading levels are disabled

Describe the bug
If I style some text as Heading 1, I cannot switch it to any other heading level, because they are disabled in the MenuSelectHeading component.  I have to first switch it to Paragraph and then to the desired Heading 2-6.
This only happens with Heading 1, all other heading levels are fine.
To Reproduce
I'll try to replicate it in codesandbox when I have the time.
Steps to reproduce the behaviour:

write some text and style it as Heading 1
with that text selected or having the cursor in it, open the MenuSelectHeading dropdown

Expected behaviour
All heading levels in the dropdown should be enabled and selectable.
E.g. it should be possible to select the option Heading 2, which should switch the selected/focused text to the Heading 2 style.
Actual behaviour
All heading levels are disabled and cannot be selected. You have to first select Paragraph, and then the desired Heading 2-6.
Screenshots

System (please complete the following information):

mui-tiptap version: ^1.8.6
tiptap version: ^2.2.3
Browser: Edge
Node version: 20.11.0
OS: Pop!_OS 22.04
"react": "^18.2.0"
"typescript": "^4.5.4",

Additional context
I compared this behaviour with your Codesandbox example, and there it works fine - you can switch heading levels directly. I tried replicating the codesandbox code as closely as possible in my project - using the same versions of the tiptap and mui-tiptap packages, using the HeadingWithAnchor instead of the basic Heading extension, same order of the extensions. Nothing helped. The older package versions caused weird internal "cannot access property of undefined" errors.
Code excerpts:
Editor definition:
const editor = useEditor({
  editorProps: {
    attributes: {
      'data-ui-input': 'rich-text-editor-input-field',
    },
  },
  extensions: [
    BulletList,
    CodeBlock,
    Document,
    HardBreak,
    ListItem,
    OrderedList,
    Paragraph,
    CustomSubscript,
    CustomSuperscript,
    Text,
    Bold,
    Blockquote,
    Code,
    Italic,
    Underline,
    Strike,
    CustomLinkExtension.configure({
      autolink: true,
      linkOnPaste: true,
      openOnClick: false,
    }),
    LinkBubbleMenuHandler,
    Gapcursor,
    Heading,
    HorizontalRule,
    Dropcursor,
    TaskList,
    TaskItem.configure({
      nested: true,
    }),
    History,
    Placeholder.configure({
      emptyEditorClass: styles.isEditorEmpty,
      placeholder,
    }),
    CharacterCount.configure({
      limit: maxLength,
    }),
  ],
  content,
}, [placeholder, content])

Component:
<ThemeProvider theme={muiTheme}>
  <RichTextEditorProvider editor={editor}>
    <RichTextField
      className={styles.editor}
      RichTextContentProps={{
        className: clsx(styles.richTextField, richTextFieldClassName),
      }}
      controls={(
        <MenuControlsContainer>
          <MenuButtonUndo
            data-ui-element="rich-text-menu-item-undo"
            tooltipLabel={t('richTextEditor.undo')}
          />
          <MenuButtonRedo
            data-ui-element="rich-text-menu-item-redo"
            tooltipLabel={t('richTextEditor.redo')}
          />
          <MenuDivider />
          <MenuSelectHeading
            data-ui-element="rich-text-menu-item-select-heading"
            className={styles.menuSelectHeading}
            tooltipTitle={t('richTextEditor.selectHeadingTooltip')}
            labels={{
              paragraph: t('richTextEditor.paragraph'),
              heading1: t('richTextEditor.heading1'),
              heading2: t('richTextEditor.heading2'),
              heading3: t('richTextEditor.heading3'),
              heading4: t('richTextEditor.heading4'),
              heading5: t('richTextEditor.heading5'),
              heading6: t('richTextEditor.heading6'),
            }}
          />
          <MenuDivider />
          <MenuButtonBold
            data-ui-element="rich-text-menu-item-bold"
            tooltipLabel={t('richTextEditor.bold')}
          />
          <MenuButtonItalic
            data-ui-element="rich-text-menu-item-italic"
            tooltipLabel={t('richTextEditor.italic')}
          />
          <MenuButtonUnderline
            data-ui-element="rich-text-menu-item-underline"
            tooltipLabel={t('richTextEditor.underline')}
          />
          <MenuButtonStrikethrough
            data-ui-element="rich-text-menu-item-strikethrough"
            tooltipLabel={t('richTextEditor.strikethrough')}
          />
          <MenuButtonSubscript
            data-ui-element="rich-text-menu-item-subscript"
            tooltipLabel={t('richTextEditor.subscript')}
          />
          <MenuButtonSuperscript
            data-ui-element="rich-text-menu-item-superscript"
            tooltipLabel={t('richTextEditor.superscript')}
          />
          <MenuDivider />
          <MenuButtonEditLink
            data-ui-element="rich-text-menu-item-edit-link"
            tooltipLabel={t('richTextEditor.link')}
          />
          <MenuDivider />
          <MenuButtonOrderedList
            data-ui-element="rich-text-menu-item-ordered-list"
            tooltipLabel={t('richTextEditor.orderedList')}
          />
          <MenuButtonBulletedList
            data-ui-element="rich-text-menu-item-bullet-list"
            tooltipLabel={t('richTextEditor.bulletedList')}
          />
          <MenuButtonTaskList
            data-ui-element="rich-text-menu-item-task-list"
            tooltipLabel={t('richTextEditor.taskChecklist')}
          />
          <MenuDivider />
          <MenuButtonBlockquote
            data-ui-element="rich-text-menu-item-blockquote"
            tooltipLabel={t('richTextEditor.blockquote')}
          />
          <MenuDivider />
          <MenuButtonCode
            data-ui-element="rich-text-menu-item-code"
            tooltipLabel={t('richTextEditor.code')}
          />
          <MenuButtonCodeBlock
            data-ui-element="rich-text-menu-item-code-block"
            tooltipLabel={t('richTextEditor.codeBlock')}
          />
          <MenuDivider />
          <MenuButtonHorizontalRule
            data-ui-element="rich-text-menu-item-horizontal-rule"
            tooltipLabel={t('richTextEditor.insertHorizontalLine')}
          />
          <MenuButtonRemoveFormatting
            data-ui-element="rich-text-menu-item-remove-formatting"
            tooltipLabel={t('richTextEditor.removeInlineFormatting')}
          />
        </MenuControlsContainer>
      )}
    />
    <LinkBubbleMenu
      labels={{
        viewLinkEditButtonLabel: t('edit'),
        viewLinkRemoveButtonLabel: t('remove'),
        editLinkAddTitle: t('richTextEditor.addLink'),
        editLinkEditTitle: t('richTextEditor.editLink'),
        editLinkCancelButtonLabel: t('cancel'),
        editLinkTextInputLabel: t('text'),
        editLinkHrefInputLabel: t('richTextEditor.link'),
        editLinkSaveButtonLabel: t('save'),
      }}
    />
  </RichTextEditorProvider>
</ThemeProvider>

Hm, that's a strange bug! I was actually able to reproduce it with (1) your set of extensions and (2) the latest version of tiptap for all extensions. If either of those weren't done, it would work just fine (no bug). I haven't had a chance to look thoroughly into why that bug occurs if (1) and (2) are done, but I suspect it's an issue in Tiptap's can() function as described in this Tiptap issue (reportedly affecting Tiptap > 2.0.4): https://github.com/ueberdosis/tiptap/issues/4225#issuecomment-1682657548
since we call can().setHeading({ level: 1 }) in the MenuSelectHeading component to determine whether to allow the user to choose the heading option:
https://github.com/sjdemartini/mui-tiptap/blob/b5f6d0907fde75b494aed97836ef12a02206513f/src/controls/MenuSelectHeading.tsx#L187-L188
It's seemed to me that newer Tiptap releases have been less stable and reliable (with occasional somewhat dangerous lower-level changes/bugs unfortunately), so your best bet may be to switch back to 2.0.* for now sadly, if that's an option. I have been swamped so don't suspect I'll have time to investigate a workaround fix for this underlying Tiptap problem in the near future, but would welcome further insight from you and/or PRs/fixes if you know of one.

I'm not sure it's a bug on the TipTap side. See this codesandbox I managed to whip up (ignore the code quality, it was a quick job :sweat_smile: ):
https://codesandbox.io/p/devbox/zwn843
I used only tiptap ^2.2.3  so without mui-tiptap components. I modified a very simple toolbar from their example, and the editor?.can().setHeading(...) method seems to be working just fine. It does not disable the other heading buttons.

I think I got it! The problem is that the level is always set to 1 when determining the canSetHeading.
Once a text is set to Heading 1, it cannot be set to Heading 1 again (it can be "toggled" via toggleHeading(...) but not "set" via setHeading(...)).  Therefore editor?.can().setHeading({ level: 1 }) will return false.
If it's set to any other Heading 2-6, it can be set to Heading 1, therefore editor?.can().setHeading({ level: 1 }) will return true. It explains the behaviour I described in the repro steps.
So I think the canSetHeading needs to be rewritten into a method that takes the actual level. If you think I'm correct, I could take a look at it and submit a PR.

Thanks for investigating! Yes, I agree that that seems to be the cause of the issue on the newer version of Tiptap. It's a change in the longstanding behavior of can().setHeading(), and it seems Tiptap's release notes sadly didn't document this change. (The Heading extension file itself has not had any changes, so seems to be due to some lower level update.)
The code I linked to above from mui-tiptap is calling can().setHeading() only once as a performance optimization, rather than calling it for each heading level, since the "is user allowed to change to a heading" logic is the same for any given level. Despite Tiptap's new underlying behavior, we shouldn't disable the current heading's option in the dropdown options even if it's currently selected. For usability/accessibility/consistency with patterns seen across user interfaces, <select> elements don't mark the currently selected <option> as disabled (even if reselecting it would have no effect). i.e., if you are currently on some Heading 1 text, none the options in the dropdown should be disabled (like in the current CodeSandbox demo from the mui-tiptap README).
So perhaps the logic would need to be something like: canSetHeading = currentLevel === 1 || !!editor?.can().setHeading({ level: 1 }). Definitely open to reviewing a PR since I likely won't have time to work on this myself!

Fixed in https://github.com/sjdemartini/mui-tiptap/pull/222 and released in https://github.com/sjdemartini/mui-tiptap/releases/tag/v1.9.0


MRG: Minor fixes to examples

Closes #3720.

Current coverage is 86.37% (diff: 100%)

No coverage report found for master at f7ffe8a.

Powered by Codecov. Last update f7ffe8a...5938cf3

Ready for review/merge from my end

thx @Eric89GXL


WIP: Provide UI to manage kubernetes auditing

Required to take advantage of https://github.com/kubic-project/salt/pull/541
feature#auditing
fate#325337

There are some issues right now, @ereslibre is going to look a bit into them. This is not yet ready to be merged.

LGTM, thanks to @ereslibre  and @vitoravelino. I cannot approve the PR because I'm the owner :)


Reached heap limit when building since v4.2

Describe the bug
I noticed since upgrading to 4.2 that my builds are failing on CI (self hosted runner for GH Actions, 4GB RAM). I'm running vite build --mode testing --sourcemap false with a React project. Since I'm not building source maps, I think it's unusual to reach the heap limit so easily, although I'm no expert on this. The build transforms 13705 modules, most of which are from @mui/icons-material. Even when removing this dependency which lowers the module count to around 3000, I still reach the heap limit. On 4.1.4 I don't have any issues with the exact same codebase.
Has there been any change since 4.1 that could have impacted this? I did notice a bump from rollup 3.10 to 3.20, but can't tell from the changelog what the impact is.
I haven't found anything in this issue that helps me.
Update: Still happens on v4.3.1
<--- Last few GCs --->

[1723943:0x60973e0]    67449 ms: Mark-sweep (reduce) 1947.1 (1996.0) -> 1945.3 (1996.2) MB, 1608.4 / 0.0 ms  (average mu = 0.266, current mu = 0.028) allocation failure scavenge might not succeed
[1723943:0x60973e0]    69801 ms: Mark-sweep (reduce) 1946.3 (1996.2) -> 1946.2 (1997.0) MB, 2350.4 / 0.0 ms  (average mu = 0.131, current mu = 0.000) allocation failure scavenge might not succeed

[WebServer] vite v4.2.1 building for testing...
<--- JS stacktrace --->

FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory
[WebServer]  1: 0xb08f10 node::Abort()

<Removed some lines, can provide if necessary>

[WebServer] Aborted (core dumped)
[WebServer] error Command failed with exit code 134.
[WebServer] transforming...
[WebServer] ✓ 13705 modules transformed.

vite.config.ts:
{
    build: {
      sourcemap: 'hidden',
    },
    plugins: [
      react(),
      tsconfigPaths(),
      svgr(),
    ],
}

Reproduction
Not sure how to repro this without sharing my code
Steps to reproduce
vite build --mode testing --sourcemap false on the CI server.
System Info
System:
    OS: Linux 5.15 Ubuntu 22.04.2 LTS 22.04.2 LTS (Jammy Jellyfish)
    CPU: (2) x64 DO-Premium-AMD
    Memory: 2.69 GB / 3.83 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 16.19.1 - ~/runner-craft/_work/_tool/node/16.19.1/x64/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 8.19.3 - ~/runner-craft/_work/_tool/node/16.19.1/x64/bin/npm
  npmPackages:
    @vitejs/plugin-react: 4.0.0 => 4.0.0
    vite: 4.3.1 => 4.3.1

Used Package Manager
yarn
Logs

Click to expand!
[WebServer] 2023-04-21T07:27:34.746Z vite:config bundled config file loaded in 543.88ms
[WebServer] 2023-04-21T07:27:34.784Z vite:esbuild 29.76ms tsconfck init /home/github/my-path-here
[WebServer] 2023-04-21T07:27:34.800Z vite:config using resolved config: {
  build: {
    target: [ 'es2020', 'edge88', 'firefox78', 'chrome87', 'safari14' ],
    cssTarget: [ 'es2020', 'edge88', 'firefox78', 'chrome87', 'safari14' ],
    outDir: 'dist',
    assetsDir: 'assets',
    assetsInlineLimit: 4096,
    cssCodeSplit: true,
    sourcemap: 'false',
    rollupOptions: { onwarn: [Function: onwarn] },
    minify: 'esbuild',
    terserOptions: {},
    write: true,
    emptyOutDir: null,
    copyPublicDir: true,
    manifest: false,
    lib: false,
    ssr: false,
    ssrManifest: false,
    ssrEmitAssets: false,
    reportCompressedSize: true,
    chunkSizeWarningLimit: 500,
    watch: null,
    commonjsOptions: { include: [Array], extensions: [Array] },
    dynamicImportVarsOptions: { warnOnError: true, exclude: [Array] },
    modulePreload: { polyfill: true },
    cssMinify: true
  },
  plugins: [
    'vite:build-metadata',
    'vite:watch-package-data',
    'vite:pre-alias',
    'alias',
    'vite:react-babel',
    'vite:react-refresh',
    'vite-tsconfig-paths',
    'vite:modulepreload-polyfill',
    'vite:resolve',
    'vite:html-inline-proxy',
    'vite:css',
    'vite:esbuild',
    'vite:json',
    'vite:wasm-helper',
    'vite:worker',
    'vite:asset',
    'vite-plugin-svgr',
    'vite:wasm-fallback',
    'vite:define',
    'vite:css-post',
    'vite:build-html',
    'vite:worker-import-meta-url',
    'vite:asset-import-meta-url',
    'vite:force-systemjs-wrap-complete',
    'commonjs',
    'vite:data-uri',
    'vite:dynamic-import-vars',
    'vite:import-glob',
    'vite:build-import-analysis',
    'vite:esbuild-transpile',
    'vite:terser',
    'vite:reporter',
    'vite:load-fallback'
  ],
  server: {
    preTransformRequests: true,
    host: 'my-host-url',
    open: true,
    port: 8000,
    sourcemapIgnoreList: [Function: isInNodeModules],
    middlewareMode: false,
    fs: { strict: true, allow: [Array], deny: [Array] }
  },
  mode: 'testing',
  optimizeDeps: {
    disabled: 'build',
    force: undefined,
    include: [ 'react', 'react/jsx-dev-runtime' ],
    esbuildOptions: { preserveSymlinks: false }
  },
  esbuild: { jsxDev: false, jsx: 'automatic', jsxImportSource: undefined },
  resolve: {
    mainFields: [ 'module', 'jsnext:main', 'jsnext' ],
    browserField: true,
    conditions: [],
    extensions: [
      '.mjs',  '.js',
      '.mts',  '.ts',
      '.jsx',  '.tsx',
      '.json'
    ],
    dedupe: [ 'react', 'react-dom' ],
    preserveSymlinks: false,
    alias: [ [Object], [Object] ]
  },
  configFile: '/home/github/my-path-here/vite.config.ts',
  configFileDependencies: [
    '/home/github/my-path-here/vite.config.ts'
  ],
  inlineConfig: {
    root: undefined,
    base: undefined,
    mode: 'testing',
    configFile: undefined,
    logLevel: undefined,
    clearScreen: undefined,
    optimizeDeps: { force: undefined },
    build: { sourcemap: 'false' }
  },
  root: '/home/github/my-path-here',
  base: '/',
  rawBase: '/',
  publicDir: '/home/github/my-path-here/public',
  cacheDir: '/home/github/my-path-here/node_modules/.vite',
  command: 'build',
  ssr: {
    format: 'esm',
    target: 'node',
    optimizeDeps: { disabled: true, esbuildOptions: [Object] }
  },
  isWorker: false,
  mainConfig: null,
  isProduction: true,
  preview: {
    port: undefined,
    strictPort: undefined,
    host: 'my-host-url',
    https: undefined,
    open: true,
    proxy: undefined,
    cors: undefined,
    headers: undefined
  },
  envDir: '/home/github/my-path-here',
  env: {
    VITE_APP_API_URL: 'http://my-api-url/api/v1',
    BASE_URL: '/',
    MODE: 'testing',
    DEV: false,
    PROD: true
  },
  assetsInclude: [Function: assetsInclude],
  logger: {
    hasWarned: false,
    info: [Function: info],
    warn: [Function: warn],
    warnOnce: [Function: warnOnce],
    error: [Function: error],
    clearScreen: [Function: clearScreen],
    hasErrorLogged: [Function: hasErrorLogged]
  },
  packageCache: Map(1) {
    'fnpd_/home/github/my-path-here' => {
      dir: '/home/github/my-path-here',
      data: [Object],
      hasSideEffects: [Function: hasSideEffects],
      webResolvedImports: {},
      nodeResolvedImports: {},
      setResolvedCache: [Function: setResolvedCache],
      getResolvedCache: [Function: getResolvedCache]
    },
    set: [Function (anonymous)]
  },
  createResolver: [Function: createResolver],
  worker: {
    format: 'iife',
    plugins: [
      'vite:build-metadata',
      'vite:watch-package-data',
      'vite:pre-alias',
      'alias',
      'vite:modulepreload-polyfill',
      'vite:resolve',
      'vite:html-inline-proxy',
      'vite:css',
      'vite:esbuild',
      'vite:json',
      'vite:wasm-helper',
      'vite:worker',
      'vite:asset',
      'vite:wasm-fallback',
      'vite:define',
      'vite:css-post',
      'vite:build-html',
      'vite:worker-import-meta-url',
      'vite:asset-import-meta-url',
      'vite:force-systemjs-wrap-complete',
      'commonjs',
      'vite:data-uri',
      'vite:worker-post',
      'vite:dynamic-import-vars',
      'vite:import-glob',
      'vite:build-import-analysis',
      'vite:esbuild-transpile',
      'vite:terser',
      'vite:load-fallback'
    ],
    rollupOptions: {},
    getSortedPlugins: [Function: getSortedPlugins],
    getSortedPluginHooks: [Function: getSortedPluginHooks]
  },
  appType: 'spa',
  experimental: { importGlobRestoreExtension: false, hmrPartialAccept: false },
  getSortedPlugins: [Function: getSortedPlugins],
  getSortedPluginHooks: [Function: getSortedPluginHooks]
}
[WebServer] 2023-04-21T07:27:35.464Z vite:resolve 0.95ms src/styles/variables -> null
[WebServer] 2023-04-21T07:27:35.504Z vite:resolve 8.07ms src/styles/mixins -> null
[WebServer] 2023-04-21T07:27:37.579Z vite:resolve 3.33ms ../assets/fonts/Campton-Thin.woff -> /home/github/my-path-here/src/assets/fonts/Campton-Thin.woff
[WebServer] 2023-04-21T07:27:37.580Z vite:resolve 3.81ms ../assets/fonts/Campton-ThinItalic.woff -> /home/github/my-path-here/src/assets/fonts/Campton-ThinItalic.woff
[WebServer] 2023-04-21T07:27:37.580Z vite:resolve 3.97ms ../assets/fonts/Campton-ExtraLight.woff -> /home/github/my-path-here/src/assets/fonts/Campton-ExtraLight.woff
[WebServer] 2023-04-21T07:27:37.580Z vite:resolve 4.[11](https://github.com/myrepohere/actions/runs/4762425694/jobs/8464677922?pr=1066#step:7:12)ms ../assets/fonts/Campton-ExtraLightItalic.woff -> /home/github/my-path-here/src/assets/fonts/Campton-ExtraLightItalic.woff
[WebServer] 2023-04-21T07:27:37.580Z vite:resolve 4.36ms ../assets/fonts/Campton-Light.woff -> /home/github/my-path-here/src/assets/fonts/Campton-Light.woff
[WebServer] 2023-04-21T07:27:37.580Z vite:resolve 4.53ms ../assets/fonts/Campton-LightItalic.woff -> /home/github/my-path-here/src/assets/fonts/Campton-LightItalic.woff
[WebServer] 2023-04-21T07:27:37.581Z vite:resolve 4.65ms ../assets/fonts/Campton-Book.woff -> /home/github/my-path-here/src/assets/fonts/Campton-Book.woff
[WebServer] 2023-04-21T07:27:37.581Z vite:resolve 4.79ms ../assets/fonts/Campton-BookItalic.woff -> /home/github/my-path-here/src/assets/fonts/Campton-BookItalic.woff
[WebServer] 2023-04-21T07:27:37.581Z vite:resolve 4.91ms ../assets/fonts/Campton-Medium.woff -> /home/github/my-path-here/src/assets/fonts/Campton-Medium.woff
[WebServer] 2023-04-21T07:27:37.581Z vite:resolve 5.05ms ../assets/fonts/Campton-MediumItalic.woff -> /home/github/my-path-here/src/assets/fonts/Campton-MediumItalic.woff
[WebServer] 2023-04-21T07:27:37.581Z vite:resolve 5.19ms ../assets/fonts/Campton-SemiBold.woff -> /home/github/my-path-here/src/assets/fonts/Campton-SemiBold.woff
[WebServer] 2023-04-21T07:27:37.582Z vite:resolve 5.30ms ../assets/fonts/Campton-SemiBoldItalic.woff -> /home/github/my-path-here/src/assets/fonts/Campton-SemiBoldItalic.woff
[WebServer] 2023-04-21T07:27:37.582Z vite:resolve 5.27ms ../assets/fonts/Campton-Bold.woff -> /home/github/my-path-here/src/assets/fonts/Campton-Bold.woff
[WebServer] 2023-04-21T07:27:37.582Z vite:resolve 5.36ms ../assets/fonts/Campton-BoldItalic.woff -> /home/github/my-path-here/src/assets/fonts/Campton-BoldItalic.woff
[WebServer] 2023-04-21T07:27:37.582Z vite:resolve 5.47ms ../assets/fonts/Campton-ExtraBold.woff -> /home/github/my-path-here/src/assets/fonts/Campton-ExtraBold.woff
[WebServer] 2023-04-21T07:27:37.582Z vite:resolve 5.58ms ../assets/fonts/Campton-ExtraBoldItalic.woff -> /home/github/my-path-here/src/assets/fonts/Campton-ExtraBoldItalic.woff
[WebServer] 2023-04-21T07:27:37.582Z vite:resolve 5.68ms ../assets/fonts/Campton-Black.woff -> /home/github/my-path-here/src/assets/fonts/Campton-Black.woff
[WebServer] 2023-04-21T07:27:37.582Z vite:resolve 5.79ms ../assets/fonts/Campton-BlackItalic.woff -> /home/github/my-path-here/src/assets/fonts/Campton-BlackItalic.woff
[WebServer] 2023-04-21T07:27:37.772Z vite:resolve 0.30ms ../assets/fonts/Campton-Thin.otf -> /home/github/my-path-here/src/assets/fonts/Campton-Thin.otf
[WebServer] 2023-04-21T07:27:37.773Z vite:resolve 0.22ms ../assets/fonts/Campton-ThinItalic.otf -> /home/github/my-path-here/src/assets/fonts/Campton-ThinItalic.otf
[WebServer] 2023-04-21T07:27:37.774Z vite:resolve 0.18ms ../assets/fonts/Campton-ExtraLight.otf -> /home/github/my-path-here/src/assets/fonts/Campton-ExtraLight.otf
[WebServer] 2023-04-21T07:27:37.775Z vite:resolve 0.18ms ../assets/fonts/Campton-ExtraLightItalic.otf -> /home/github/my-path-here/src/assets/fonts/Campton-ExtraLightItalic.otf
[WebServer] 2023-04-21T07:27:37.775Z vite:resolve 0.18ms ../assets/fonts/Campton-Light.otf -> /home/github/my-path-here/src/assets/fonts/Campton-Light.otf
[WebServer] 2023-04-21T07:27:37.776Z vite:resolve 0.18ms ../assets/fonts/Campton-LightItalic.otf -> /home/github/my-path-here/src/assets/fonts/Campton-LightItalic.otf
[WebServer] 2023-04-21T07:27:37.776Z vite:resolve 0.30ms ../assets/fonts/Campton-Book.otf -> /home/github/my-path-here/src/assets/fonts/Campton-Book.otf
[WebServer] 2023-04-21T07:27:37.777Z vite:resolve 0.18ms ../assets/fonts/Campton-BookItalic.otf -> /home/github/my-path-here/src/assets/fonts/Campton-BookItalic.otf
[WebServer] 2023-04-21T07:27:37.778Z vite:resolve 0.18ms ../assets/fonts/Campton-Medium.otf -> /home/github/my-path-here/src/assets/fonts/Campton-Medium.otf
[WebServer] 2023-04-21T07:27:37.778Z vite:resolve 0.22ms ../assets/fonts/Campton-MediumItalic.otf -> /home/github/my-path-here/src/assets/fonts/Campton-MediumItalic.otf
[WebServer] 2023-04-21T07:27:37.779Z vite:resolve 0.21ms ../assets/fonts/Campton-SemiBold.otf -> /home/github/my-path-here/src/assets/fonts/Campton-SemiBold.otf
[WebServer] 2023-04-21T07:27:37.779Z vite:resolve 0.23ms ../assets/fonts/Campton-SemiBoldItalic.otf -> /home/github/my-path-here/src/assets/fonts/Campton-SemiBoldItalic.otf
[WebServer] 2023-04-21T07:27:37.780Z vite:resolve 0.18ms ../assets/fonts/Campton-Bold.otf -> /home/github/my-path-here/src/assets/fonts/Campton-Bold.otf
[WebServer] 2023-04-21T07:27:37.781Z vite:resolve 0.20ms ../assets/fonts/Campton-ExtraBold.otf -> /home/github/my-path-here/src/assets/fonts/Campton-ExtraBold.otf
[WebServer] 2023-04-21T07:27:37.781Z vite:resolve 0.18ms ../assets/fonts/Campton-BoldItalic.otf -> /home/github/my-path-here/src/assets/fonts/Campton-BoldItalic.otf
[WebServer] 2023-04-21T07:27:38.187Z vite:resolve 0.27ms ../assets/fonts/Campton-Black.otf -> /home/github/my-path-here/src/assets/fonts/Campton-Black.otf
[WebServer] 2023-04-21T07:27:38.188Z vite:resolve 0.35ms ../assets/fonts/Campton-ExtraBoldItalic.otf -> /home/github/my-path-here/src/assets/fonts/Campton-ExtraBoldItalic.otf
[WebServer] 2023-04-21T07:27:38.188Z vite:resolve 0.20ms ../assets/fonts/Campton-BlackItalic.otf -> /home/github/my-path-here/src/assets/fonts/Campton-BlackItalic.otf
[WebServer] 2023-04-21T07:27:42.104Z vite:resolve 0.80ms src/styles/_mixins.scss -> null
[WebServer] 2023-04-21T07:27:42.[16](https://github.com/myrepohere/actions/runs/4762425694/jobs/8464677922?pr=1066#step:7:17)6Z vite:resolve 0.[59](https://github.com/myrepohere/actions/runs/4762425694/jobs/8464677922?pr=1066#step:7:60)ms src/styles/_variables.scss -> null

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I'm getting the exact same issue with Nuxt 3. I have a project that before updating, it built without any issues but as soon as I updated it now I can't build it anymore on the server 🤷🏻♂ only on my local machine with 16GB of RAM (the server only has 1GB of free RAM but it had no issues building a few days ago)


FF patches

This PR includes fixes for bugs discovered on the consumer side while FF
testing, including:

Don't filter out PVs based on spec.storageClassName, as the index is
not recognized.
Move SCC checks under consumer mode.
Remove finalizer on SCC.

Signed-off-by: Pranshu Srivastava<EMAIL_ADDRESS>

/cc @nb-ohad @Madhu-1 @umangachapagain @iamniting @subhamkrai

/lgtm

/cherrypick release-4.11


Frontend with frameworks: Angular:Sharing data between parent and child components in Angular

Description 📜
:Sharing data between parent and child components in Angular
Fixes # <729>

Type of change 📝

[x] Documentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] Frontend Dev (React, Angular, Vue)

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

![Screenshot (104)](https://user-images.githubusercontent.com/74728397/133909499-03bbb822-7a0f-4662-818e-2e563bc1d011.png)

Please include some code examples

Ok I will add some examples of code

@RIYANSHGARG u explained it practically very well but it's of no use until u won't add code example so please add them as well :)


Embedded documents not persisted without "type" option in yaml mapping

Embedded documents are no more persisted (isEmbeddedDocument is false) without setting the type option to embeddedDocument:
    User:
      type: document
      embedOne:
        address:
          targetDocument: Address

    Address:
-----> type: embeddedDocument

AFAIK some time ago it was enough to specify the mapping on the parent class when using YAML; at least my app was working doing so and the docs show it that way too.
Was it an intentional change?

For one, it's intentional that you have to specify a type. Then, I'm also not sure about keeping YAML mappings around.


AWS S3 media store

Hello there,
I've been watching this repo for a while and you guys are doing a great job! :smile:
What you think about this idea:
I've followed this post (https://ashokfernandez.wordpress.com/2014/03/11/deploying-a-django-app-to-amazon-aws-with-nginx-gunicorn-git/) and now I'm able to create as many ec2 instances as I want with one single command and make my project use RDS and S3.
The problem is: this setup relies on djang-storages which is not supporting py3 yet. That's why I'm facing problems to upload files into S3.
My point is: do you consider using S3 in this setup? The post I mentioned is a great starting point. And, if so, I'd be very insterested in a solution that works in a py3 env.
Great project!

This should be addressed after we wrap up #186


Fix bug introduced in a1edfab

There was a bug introduced in a1edfab5d124aa70493dd988973f17bf4a9a4c30
https://github.com/rvaiya/keyd/blob/a1edfab5d124aa70493dd988973f17bf4a9a4c30/Makefile#L34
If $(DESTDIR)$(PREFIX)/lib/systemd exists, but there is no systemd/system directory, install will treat the command as "Install keyd.service into the file .../systemd/system" rather than "create a directory called system if it doesn't exist and install keyd.service within it". Adding the desired file name to the destination path fixes this.

Oops :P. Thanks for catching this. I've merged this, but taken the liberty of amending the commit message (which was a bit verbose for something like this).

Yeah all good


The CREDENTIAL in Example 1

The code provided won't give you the CREDENTIAL which is called in the TDE example

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

ID: 6e349050-7bbf-e076-2b4d-80be1b19611c
Version Independent ID: e19f0325-71bb-9cca-c6df-56675ebff3b0
Content: Integrate Key Vault with SQL Server on Windows VMs in Azure (Resource Manager) - Azure SQL Server VM
Content Source: articles/virtual-machines/windows/sql/virtual-machines-windows-ps-sql-keyvault.md
Service: virtual-machines-sql
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

Thanks for the feedback. Can you please elaborate on the updates you are suggesting to the doc for example 1?

Just above the TDE example there's code to create a credential called sysadmin_ekm_cred instead of Azure_EKM_cred. Just use that name and the sample code will all work.

I have created a PR with the update, changes will be live shortly.
Thanks for bringing this to our attention. We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.

Hi @mrdenny , thanks for improving our documentation! Just so  you know, it's also possible to make these changes  yourself, if you were so inclined :) doing so will improve our documentation, and it will add you as a contributor to the document.
In case you're interested, these are the steps to do so:

Navigate to the page of interest
Select the "Edit" button at the top right of the page
Select the "Pencil" icon on the right
Modify the text within the text box in markdown format - more info: https://github.com/adam-p/markdown-here/wiki/Markdown-Cheatsheet
Name your file change and provide a description, if necessary
Select "Propose file change"
Select "Create pull request" on the 'Comparing changes' page
Select "Create pull request" on the 'Open a pull request' page

You can find more information about that here: https://aka.ms/editsqldocs.
I hope that helps and hope you have a wonderful rest of your day!
Masha from the SQL Docs team

Thanks Masha for the PR guidance.
For now, I have created a PR here: https://github.com/MicrosoftDocs/azure-docs-pr/pull/107900
Changes will be live shortly.


Flag to disable sampling and annotate all functions

I'd like to be able to use this to annotate very short scripts!
For a minimal example:
def add(a, b):
    return a + b

print(add(3, 4))

RightTyper always samples the very first invocation of any function.

% cat test-issue21.py.typed

from typing import Any, Callable, Dict, FrozenSet, Generator, List, Never, Optional, Set, Tuple, TYPE_CHECKING, Union
def add(a: int, b: int) -> int:
    return a + b

print(add(3, 4))

This may have been caused by the same thing that caused #22, closing


Revert IO#write changes in 0.35.0 and let it return Nil

Reverts #9233 and #9363, yet keeps some cleanup, minor fixes and the return type of IO.copy as Int64.
You can read the motivations for reverting these changes at https://github.com/crystal-lang/crystal/pull/9233#issuecomment-642726015
Although is a breaking-change from 0.35.0, it's better to revert it now than make all the ecosystem adapt for the change in 0.35.0. That's why it's better to do it in 0.35.1.

I will throw some mentions and link to PRs that I am aware of that already migrate things to work with 0.35.0

https://github.com/schovi/baked_file_system/pull/36 @straight-shoota @schovi
https://github.com/busyloop/rucksack/pull/3 @jgillich @m-o-e
https://github.com/crystal-community/jwt/pull/33 @jamescook
https://github.com/stumpycr/stumpy_png/issues/28 @bararchy
https://github.com/crystal-lang/crystal-mysql/pull/90 @bcardiff  🤳
https://github.com/stakach/openssl_ext/pull/1 @stakach
https://github.com/spider-gazelle/bindata/pull/5 @stakach
https://github.com/amberframework/cli/pull/4 @drujensen

Appologies for the back and forth with this api changes


Use xarray Accessors for easy access to functionality

Because we use xarray.Dataset and xarray.DataArray as our data container it would be best and most convenient to make as much functionality as possible accessible via custom xarray Accessors so that we can e.g. plot the CML paths like this
ds_cmls.plg.plot_cmls()

where plg is our custom accessor.
If we go this step, we could also make some basics functions that work in the background more general regarding their input, e.g. allow plotting colored lines by passing the coordinates and values as numpy arrays and not rely on xarray.Dataset in these "low-level functions" because one can call the functions via the xarray Accessor. The only question would where to put the detailed doc string, in the low-level function or in the accessor method.

#29 implements the xarray Accessors for plotting the CML paths. IMO it works great.
Things we should do/discuss now:

Decide if we want to keep functions like plot_map.plot_lines(cmls, ...) which take a xarray.Dataset as input or if we consider these functions redundant because the exact same thing can be achieved with ds_cmls.plg.plot_cmls(...). This currently calls plot_map.plot_lines but it could also call the lower-level function plot_map.scatter_lines which does not require a xarray.Dataset as input and which can also serve as basis for plotting for SMLs.
Implement xarray Accessors for other functions, e.g. spatial distance, finding nearest neigbor, projecting coordinates.


Slurper as a Service?

Is it possible to package Mail Slurper to be used as a windows service so it starts on boot?

I use nssm.

On May 17, 2018, at 2:23 PM, Jonathon Wyza<EMAIL_ADDRESS>wrote:
Is it possible to package Mail Slurper to be used as a windows service so it starts on boot?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I too have used NSSM (https://nssm.cc). A great tool for installing regular console apps as a service in Windows.

I will close this unless there is anything else you need. Thanks!


Add general environment variables option to support PGADMIN_CONFIG_ overrides as per the container docs

What this PR does / why we need it:
Adds the ability to set arbitrary environment variables. Given the extent of options that could be overriden in config.py this seemed preferable to exposing each individually.
I was initially unable to deploy a default installation with a LoadBalancer against an AKS cluster. No user credentials worked when accessing via the LoadBalancer IP. There is a note in config.py as per disabling enhanced cookie protection within dynamic IP environments such as Kubernetes.
Which issue this PR fixes
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

Hello @patkearney,
Thanks for this improvement! It will be helpful.


Complete first version

all functionality completed for first version
test coverage goal of  99% still reached

Coverage increased (+0.12%) to 99.3% when pulling 3abb1a4a3a48ab070a4d5d0d7364ecc016fca2fe on complete-first-version into c37219341a36ce2bc426a9f8fc0840afed4d72f3 on master.


[Microsoft Documents Feedback]

Hello,
While I was doing troubleshooting for VM Backup Extension, I noticed that there was no example provided in the following document:

Set-AzVMBackupExtension

Can you please update this document by adding an example?
[Basil Kandah feedback]
Thank you very much!
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3ea9c977-831e-6c11-8a36-993c97ac198d
Version Independent ID: a1a4e76d-f614-b29b-8b74-283aa1315a9e
Content: Introducing the Azure Az PowerShell module
Content Source: docs-conceptual/azps-7.3.0/new-azureps-module-az.md
Service: azure-powershell
GitHub Login: @mikefrobbins
Microsoft Alias: mirobb

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

Thank you @MarileeTurscak-MSFT

@bakandah The cmdet reference for Azure PowerShell is autogenerated. I've logged an issue in the source code repo for Azure PowerShell to have the appropriate examples added.
See azure/azure-powershell#17504 for more details.


resume training will erase old results/game.csv & overwrite snapshots/game_$(EPOCH).pkl from 1

resume training will erase results/game.csv & overwrite snapshots/game_$(EPOCH).pkl from 1
I'd rather see append to game.csv, and snapshots file continuing from where it's left off.

Yes, this is unfortunate feature. I would add something like --start_epoch command line argument and resume.sh script to deduce that from snapshot file name. I'm hoping to fix it soon, but maybe not in this week.


Provide an EntityDecoder based on json4s' extract.

It's not as typeclassy as our other json integrations, but neither is json4s.  ¯_(ツ)_/¯

:+1:


Always disable confirmation when installing system dependencies

This fixes a bug similar to the one identified in #51, wherein curl ... | bash installation would fail due to stdin being EOF. In this case, apt-get was hanging because it needed some input to install the tzdata package. To address this, I've updated the system deps installation phase to just always proceed without confirmation. If a user wants to disable the installation of system deps, they already had to confirm as much during the options prompting phase.

@swift-server-bot test install please


chore: :zap: update version

:tada: This PR is included in version 0.9.0-alpha.1 :tada:
The release is available on:

npm package (@feature/232-sdk-deployment dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.9.0 :tada:
The release is available on:

npm package (@feature/232-sdk-deployment dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Basic Refactoring.

Basic refactoring and extraction of modules, improved execution control flow.

Unfortunately this PR came in a bit too late to the draw, as there's been massive additions to the script. I'll have to close this PR, but please feel free to resubmit these changes in another PR! Super thankful for your efforts.


Plopper runs into issues causing FML Spam (1.12.2)

Hi,
I'm having issues with the Plopper it seems to run into issues every so often with VanillaFix installed. It doesn't crash me to the main menu screen or crashes to desktop. Just causes spam and crash errors and these are two of them. It does this while I'm near them or when I come back from a long distance away and the chunk loads back up.
Minecraft: 1.12.2
Forge: <IP_ADDRESS>25
Theplopper: 1.12.x - v1.1
Crash log 1: https://gist.github.com/DatrixTHLK/1fe23ce3644a1c405f7f5cf8d0cddfdd
Crash log 2: https://gist.github.com/DatrixTHLK/b2d56c706f288be1e1c5405024ac855e

I have no idea what exactly causes this crash, but uploaded a fixed version nonetheless.

Thanks for the update and response. I appreciate it.


Upgrade component to legacy mode

Update this component as per the upgrade guide:

Update bower dependencies
Update DOM template
Update event functions
Minor fixes

I'll rebase everything later.

I think it can be closed due to the rebase, isn't it?

To clarify, this PR was for the legacy mode. What you call the rebase is for Polymer 2.x.

Whaaaaat....I lost my mind yesterday. Sorry about my nonsense comments. 😖


Kill won't work at all for windows, and is partially broken on linux boxes.

@blainesch true about windows. And do you have an idea how to fix it on linux? Would be cool if you can do a PR


Revert "Changed document field for international trade answer"

This reverts https://github.com/bitzesty/qae/commit/8fb1a336d87003b8cfb44a5236d43c459a7679f6

 [QAE18] Investigate pre-populated Application background when logged in as an Assessor


web demo not working

HI i have tried to use the web demo several times over the past few days and it does not work. it simply times out for me after a while. is it working for anyone else? meta plz fix i loved using it when it came out!

nvm it works in a different browser. if its not working for you try it in something other than chrome. edge worked for me


Error on running fetch-lines

Hello, I built on Ubuntu 16.04 with Rust 1.17.0, the build appeared successful and cargo test --verbose also seemed okay. However when I ran target/release/tflgtfs fetch-lines I recieved the following error:
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Syntax(invalid type: Map, 1, 1)', /checkout/src/libcore/result.rs:859

The backtrace is:
stack backtrace:
   0: std::sys::imp::backtrace::tracing::imp::unwind_backtrace
             at /checkout/src/libstd/sys/unix/backtrace/tracing/gcc_s.rs:49
   1: std::sys_common::backtrace::_print
             at /checkout/src/libstd/sys_common/backtrace.rs:71
   2: std::panicking::default_hook::{{closure}}
             at /checkout/src/libstd/sys_common/backtrace.rs:60
             at /checkout/src/libstd/panicking.rs:355
   3: std::panicking::default_hook
             at /checkout/src/libstd/panicking.rs:371
   4: std::panicking::rust_panic_with_hook
             at /checkout/src/libstd/panicking.rs:549
   5: std::panicking::begin_panic
             at /checkout/src/libstd/panicking.rs:511
   6: std::panicking::begin_panic_fmt
             at /checkout/src/libstd/panicking.rs:495
   7: rust_begin_unwind
             at /checkout/src/libstd/panicking.rs:471
   8: core::panicking::panic_fmt
             at /checkout/src/libcore/panicking.rs:69
   9: core::result::unwrap_failed
  10: tflgtfs::cmd::load_lines
  11: tflgtfs::main
  12: std::panicking::try::do_call
             at /checkout/src/libstd/panicking.rs:454
  13: __rust_maybe_catch_panic
             at /checkout/src/libpanic_unwind/lib.rs:98
  14: std::rt::lang_start
             at /checkout/src/libstd/panicking.rs:433
             at /checkout/src/libstd/panic.rs:361
             at /checkout/src/libstd/rt.rs:57
  15: __libc_start_main
  16: _start

And all that gets output is cache/_line-route.txt
{"$type":"Tfl.Api.Presentation.Entities.ApiError, Tfl.Api.Presentation.Entities","timestampUtc":"2017-05-04T12:32:25.3380909Z","exceptionType":"EntityNotFoundException","httpStatusCode":404,"httpStatus":"NotFound","relativeUri":"/Line/Route","message":"The following line id is not recognised: Route"}

I've tried to have a look to see anything obvious but I'm not familiar with the Rust language and what the backtrace is exactly saying, hopefully this makes sense. Let me know if there is anything I can do to help.
But yes this CLI seems really perfect for analysing TfL data!

Looks like an issue with an API that we rather hastily removed without proper deprecation - the /Line/Route API has been restored if you'd like to retry?

Great, I've rerun and it seems like its returning data now!
I'm getting some occasional missing field errors now as I watch the printouts:
Error decoding timetable: missing field "stations" at line 1 column 362
Error decoding timetable: missing field "lat" at line 1 column 15590
These are happening a few times, although when it finishes I'll see if this actually affects anything significantly.

Okay the process completed with a lot of these sorts of errors as well:
Error (line virgin-trains-east-coast): The stop you selected has now been removed from the route, and therefore we cannot show you a timetable. The route page will be updated shortly to reflect these changes.

could not find shape for 15 490013766F to 490013573W!!!

I'm going to put it through a gtfs network building tool and report back to see how it looks!

I encountered the same problem yesterday and still yet am not able to figure it out.
The message shows:
thread 'main' panicked at 'called Result::unwrap() on an Err value: Ssl(OpenSslErrors([UnknownError { library: "SSL routines", function: "SSL23_GET_SERVER_HELLO", reason: "tlsv1 alert protocol version" }]))', src/libcore/result.rs:906:4
note: Run with RUST_BACKTRACE=1 for a backtrace.
I am wondering how this could be fixed. I have both build few times and cargo clean before I build, still not working out.

@AnthonySu judging by the reference to SSL protocol versions, this is probably due to older TLS versions not being supported https://blog.tfl.gov.uk/2016/12/21/developers-using-tfls-open-data-feeds/
I'm not sure how this is controlled in Rust, but it could be as simple as upgrading your client-side libraries?

Hi
I installed the exporter successfully on Ubuntu 16.04 yesterday. However, I couldn't find an executable file. The command "./target/release/tflgtfs fetch-lines --format gtfs" could not been found when I used it. Do you have any ideas about it?

@zshuLiu did you build with rust first? cargo build --release ?

@bfrog Yes I did.  However I got a new problem. When I ran ./target/release/tflgtfs fetch-lines --format gtfs command, I received the following error


[2.x] Add missing field in InitializeTenancy middleware

onFail field was declared dynamically.
And one more question accordingly this middleware: if $onFail is null - from where $e will be taken for the fallback closure?

The exception is taken from the catch block and if no callback is provided, the default one just throws it.
https://github.com/cybercog/stancl-tenancy/blob/cb07fde42887498354bee7c2b52f641e15025e86/src/Middleware/InitializeTenancy.php#L34

Done. I don't like that it catches \Exception. If you are trying to catch specific exception from tenancy()->init() - then it will be better create custom exception class. If you want to catch any exceptions - then you'd better catch \Throwable interface.

Good point. Can you update it to catch only TenantCouldNotBeIdentifiedException?


Garment presentation page

https://drive.google.com/file/d/0BwyzL3qWnwU9YXl3VGZrYVg0X3M/view?usp=sharing

What is "platform ref"?

a generic code which would be automatically attributed (by the platform) to a new project (?)

a generic code which would be automatically attributed (by the platform) to a new project (?)

Ok, on our side, it will be the commit hash. That said, it's not clear it needs to be exposed explicitely. We'll see whether it's needed at some point.


Integration with Redux w/ HMR

Is integration with redux, in a sense of react-router-redux, on then table?
I'm currently using Navigator but the lack of HMR really slows down development.

Does this setup not work? I was hoping we would support it out of the box. But haven't given it a try to be quite honest. We need a simple Redux example in the repo. If you want to take a stab at it, please send a PR over. Otherwise, I will get to it soon.

Sorry for missing it. I will probably only try-out react-router-native when v4 is out, so I probably won't be able to help soon enough.
Thanks for this great work.


Potential feature request: delegating JsonConverter

This Stack Overflow post: http://stackoverflow.com/questions/42351272
led to this PR for Noda Time: https://github.com/nodatime/nodatime/pull/657
The basic problem is that JsonConverterAttribute specifies just the name of a type to use as the converter for a property. That makes it hard to use converter instances created elsewhere. The solution I've suggested for Noda Time is simply to have an abstract class which takes an existing converter in its constructor, and delegates to it.
This solution isn't specific to Noda Time at all, so if you see general benefit in it, it would be nice to see it in Json.NET instead of in Noda Time's code base. However, if this isn't a problem which has been noted before, I'm happy enough to put it in Noda Time instead.

It seems like a rather specific feature.

The basic problem is that JsonConverterAttribute specifies just the name of a type to use as the converter for a property.

JsonConvertAttribute also accepts ctor parameters which is what I imagined people would use for customizing converters via attributes - http://www.newtonsoft.com/json/help/html/P_Newtonsoft_Json_JsonConverterAttribute_ConverterParameters.htm

Right - that's probably fine for most cases, but doesn't work so well when you want to reuse existing converters, or when there are converters for so many different types.
If it's not being a problem for others, it's fine for us to just keep it in Noda Time - will close this issue.


Attempting to clone linkedin login site fails

Kingphisher rev 6bd8350
Ubuntu 16.04.2 LTS
Attempting to clone linkedin's login page fails with the following error:
WARNING  loaded empty on target resource: https://www.linkedin.com/analytics/noauthtracker?type=leo%2EpageTracking&pageType=full_page&pageKey=uas-consumer-login-internal_jsbeacon&trackingInfo=null
WARNING  requested the empty resource with python, but the length appears invalid
WARNING  loaded empty on target resource: https://www.linkedin.com/lite/ua/error?csrfToken=ajax%3A6281124414137269997&_1501023564061
WARNING  failed to request the empty resource with python
WARNING  loaded empty on target resource: https://www.linkedin.com/fizzy/admin?1501023564068
WARNING  requested the empty resource with python, but the length appears invalid
ERROR    error uid: 9281c231-2863-4ed5-8302-13f79f38ebde an unhandled exception was thrown
Traceback (most recent call last):
  File "/opt/king-phisher/king_phisher/client/web_cloner.py", line 282, in cb_get_data_finish
    self.copy_resource_data(resource, data)
  File "/opt/king-phisher/king_phisher/client/web_cloner.py", line 170, in copy_resource_data
    if ';' in mime_type:
TypeError: argument of type 'NoneType' is not iterable
WARNING  loaded empty on target resource: https://www.linkedin.com/lite/platformtelemetry
WARNING  requested the empty resource with python, but the length appears invalid
WARNING  loaded empty on target resource: https://www.linkedin.com/lite/rum-track?csrfToken=ajax%3A6281124414137269997
WARNING  requested the empty resource with python, but the length appears invalid
WARNING  loaded empty on target resource: https://www.linkedin.com/lite/platformtelemetry

@cclements Thank you for reporting this issue, the fix has been committed into the dev branch with commit 8ab3cd7


⚠ gorzdrav.org has degraded performance

In e59a5c6, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 2211 ms

Resolved: gorzdrav.org performance has improved in 8b2c671 after 5 minutes.


Provide a library/catalog of pre-configured SAML SP integrations

Rather than having the adopter manually configure SPs in CAS SAML2 mode, try to think about the following instead:

Collect recipes for SP integrations. Each SP integration may contain a specific service definition that outlines the entity id, metadata, attributes required for release, etc.
Automate the configuration of each SP integration in form of a auto-configuration module. Upon startup, the module will insert itself into the CAS runtime engine, making adjustments for the SP integration to work.

Concretely this might look like this:

Provide a cas-server-support-samlsps module as an example.
A @Configuration class that creates the relevant service definition, attributes, metadata entries, etc retrieved from CAS properties.
Obviously, relevant grouped properties for the integration to work such as:

cas.saml.sp[0].name=...
cas.saml.sp[0].description=...
cas.saml.sp[0].entityId=...
cas.saml.sp[0].metadataLocation=...

As we build up this library of SPs, the adopters will then simply just have to choose one from a menu and CAS will start to present a pre-built configuration for the most popular SPs, making the platform that much more attractive. A great starting example might be Office365.

This is done. We'll keep collecting more.


Any way to play test with headed (gui) browser?

For example i want to pass custom arguments to pytest, because i use some pytest plugins for reports and browsers

Hi Ivan, so there's a couple points to address here.
With regards to the browser, it will open in whatever your system default browser is set to. However, if you want to open in a different browser, you can pass --no-browse flag to pytest_commander and instead manually navigate to http://localhost:5000/ in your browser of choice after it has started. As a future enhancement I could potentially add the option to pass the name of a browser executable as a command-line parameter.
Regarding passing custom args to pytest, this is possibly a little more tricky since pytest_commander does not invoke the pytest CLI command but instead calls the pytest.main() library function. It should be possible to pass extra args to this method but there might be some limitations, I would have to look into it further.

Hey - if you update to the latest release you should see a new --browser option (or -b for short), this allows you to pass in the name of a browser executable to use in preference to your system's default browser. This can either be the full absolute path or just the command name if it can be found on your PATH.
With regards to your question about extra pytest plugins, you may customise the options passed to pytest by creating a pytest.ini file as described in the official docs: https://docs.pytest.org/en/latest/reference/customize.html#pytest-ini
Let me know if you have any further questions or issues, I'll close this for now.


136 - fix swagger 404

What
Fixes a 404 error when using swagger ui by upgrading to springdoc version 2.0 as spring boot 3.0 is in use and 1.6 does not work with spring boot 3. Also adjust the publish pattern to publish individual CVEs into kafka instead of a large doc to deal with CVE DB sizes exceeding kafka buffer sizes for messages.

Codecov Report
Base: 65.96% // Head: 66.20% // Increases project coverage by +0.24% :tada:

Coverage data is based on head (a7100b2) compared to base (11e8dde).
Patch coverage: 72.41% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #140      +/-   ##
============================================
+ Coverage     65.96%   66.20%   +0.24%
- Complexity       72       73       +1
============================================
  Files            28       28
  Lines           285      290       +5
  Branches         14       15       +1
============================================
+ Hits            188      192       +4
- Misses           89       90       +1
  Partials          8        8

Impacted Files
Coverage Δ

.../java/rocks/curium/mitresiphon/dao/CVEDAOImpl.java
3.33% <0.00%> (ø)

...a/rocks/curium/mitresiphon/tasks/NVDFetchTask.java
78.78% <60.00%> (-3.98%)
:arrow_down:

...a/rocks/curium/mitresiphon/actors/CveArchiver.java
72.50% <73.68%> (+0.40%)
:arrow_up:

...tresiphon/tasks/helpers/HttpModifiedCheckImpl.java
96.15% <100.00%> (+0.69%)
:arrow_up:

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.


BUG-99252 fix deployment fails on base image

@lacikaaa  do we need this for 2.4 ?

@doktoric I don't think so, because the change caused this error is from 2018. 03. 13.


ci: base extractor-tests

Description
Checks for if the extractor can be compiled in CI, setting the stage for #374

Whoops! Please ignore that commit, I accidently pushed to the wrong branch


Use SQLRewriteEngine and SQLRewriteDecorator decouple rewrite core and features

For #3139

Pull Request Test Coverage Report for Build 510

52 of 67 (77.61%)  changed or added relevant lines in 12 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 65.967%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-proxy/sharding-proxy-backend/src/main/java/org/apache/shardingsphere/shardingproxy/backend/communication/jdbc/wrapper/PreparedStatementExecutorWrapper.java
0
6
0.0%

sharding-proxy/sharding-proxy-backend/src/main/java/org/apache/shardingsphere/shardingproxy/backend/communication/jdbc/wrapper/StatementExecutorWrapper.java
0
9
0.0%

Totals

Change from base Build 435:
0.03%

Covered Lines:
10874

Relevant Lines:
16484

💛  - Coveralls


Change minibomb to be explosion resistant and start timer on damage

About the PR
Prevents C4+Minibomb combo from being instant and impossible to do anything about without excessive paranoia. Same idea as #32423, but done from the other end. Mutually exclusive with it, unless maintainers will want to go for maximum fun.
For even more fun, see #32428 that makes the container fly into a random direction before the second explosion.
Why / Balance
See #32423, but without the note about nukies. Instagib throwable shouldn't be near impossible to counterplay in an RP game full of distractions lasting 5+ seconds. This isn't a twitch shooter.
Technical details
Creates a new damage threshold behavior that works on explosives and causes them to start counting down. In combination with extreme explosive resistance, this allows for a bomb that's not deleted and is counting down, after being triggered by another explosive. Current version of this PR only does this to minibomb, though. So no funny business with C4, unless someone requests that. Also it will just disappear without exploding if damaged too much. Gotta be careful!
Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Changelog
:cl:

tweak: Minibomb is now explosion resistant and will start counting down if damaged by a C4

Gasp! A good PR!

i think #32423 solves the problem better

@Ilya246

i think #32423 solves the problem better

Biggest concern I have about that one is that there's just going to be another convenient instant explosive that might need to be patched up later. If you can think of any, will be very glad about it!

Biggest concern I have about that one is that there's just going to be another convenient instant explosive that might need to be patched up later. If you can think of any, will be very glad about it!

IEMs already exist, they have 20s fuse but they make no sound so they are perfect for detonating minibombs. Or chain chemical reactions, though after 50% dmg nerf latter won't gib anyone for 0TC anymore.

c4 uses blast caps irl to detonate, and theres nothing physically preventing you from rigging any high voltage source to them. this doesnt mean its good for gameplay though, once liltenhead made that video everyone copied a once niche tactic for untelegraphed instant gibs

I much prefer this option as opposed to the C4 timer on signal, as this allows for counterplay but still wounds, whereas C4 would still be able to be delivered without much counterplay, instead requiring the deliverer to just cook the C4 (which they can already do without a signaler, can they not?)

allows for counterplay but still wounds

Instant C4+timed minibomb centered at the same tile as the target person will, unless either the target or the container is moved, ensue gibbing. C4 crits people when exploded at the same tile and they got no armor.

forgot to mention that one, yeah the container moving should be merged too if this one is
it's just too funny to have bouncing bombs

Hello! We've had a discussion and vote among the maintainer team with the following result: Merge, but with changes.
The change the team would like to see is that the bomb still instantly detonates upon very high damage, rather than merely breaking into nothingness. If you get hit by, say, a China Lake point blank it might feel weird if the bomb either disappears or starts just beeping. The threshold should definitely be higher than just a single C4 though.
The other PRs related to this functionality (#32428 and #32423) have also been discussed and will be merged, pending testing and code review.

The change the team would like to see is that the bomb still instantly detonates upon very high damage, rather than merely breaking into nothingness. If you get hit by, say, a China Lake point blank it might feel weird if the bomb either disappears or starts just beeping. The threshold should definitely be higher than just a single C4 though.

Sure, can do that. Would appreciate any specific figure beyond "more than one C4" though. Currently I think it takes two to pass the destruction threshold.
Personally I think this sort of "instantly detonate on integrity failure" thing should cause a smaller explosion, rather than a full one. But that would require a bit more code changes I think. Should I try to go for that, or just implement instant detonation on failure for now and throw in the reduce damage on integrity failure later?

Sure, can do that. Would appreciate any specific figure beyond "more than one C4" though. Currently I think it takes two to pass the destruction threshold.
Personally I think this sort of "instantly detonate on integrity failure" thing should cause a smaller explosion, rather than a full one. But that would require a bit more code changes I think. Should I try to go for that, or just implement instant detonation on failure for now and throw in the reduce damage on integrity failure later?

Go for instant detonation on failure for now, it's gonna require a whole new maintainer discussion regardless. @VasilisThePikachu suggested something like 5 C4, so probably around that area. Try it out and see how it feels.

So, changed this to go boom at 45 damage. That's after 90% explosion damage reduction to the minibomb.
Making it start a countdown from explosion is pretty easy. Point blank chinalake does it just fine. So does point blank C4. Next tile C4 manages to not cook it off.
And it would have been more of a concern at 40 damage threshold cuz 1+3 and then modular grenade with 50+150 CLF3 would have made it go boom, but at 45 it juuuust barely not enough. And to have CLF3 cause bigger explosion, you need a bluespace beaker.

Making it start a countdown from explosion is pretty easy. Point blank chinalake does it just fine. So does point blank C4. Next tile C4 manages to not cook it off.  Damage being 50 blunt at start for gibbing and 100 split in other for total of 150. Though I do not remember if damage is after 50% chemical nerf or not, most likely not as it seems "a bit" high:

50(150 total) CLF damage = 3 fluorine, 1 chlorine 💥  1tile
75 = 6 + 2 💥  2tile
100 = 12 + 4
125 = 27 + 9 💥 2.5tile
150 = 51 + 17 💥 3 tile
175 = 84 + 28 💥  3.5 tile
200 = 210+70 💥 4 tile

And here are explosion radius increases without damage increase:

2.5 💥 7+ chlorine
3 💥 13+
3.5 💥 24+
4 💥 36+

I think I'm happy with threshold of 45 for now, after my own testing with 50/150 CLF3 mix, unless I hear about anything else ^^


Suggestion: c.RenderError(err) to also log the error

Hi all, first up: Revel is great!
I just spend a good chunk of time trying to figure out what was wrong in one of my deployed revel apps. I discovered that if your controller does return c.RenderError(err) it doesn't get logged.
The default 500.html contains 'This exception has been logged' - so i found this surprising that errors aren't logged by default.
Would you please consider adding a call to eg c.Log.Error(err.Error()) to the top of RenderError? I believe it'd follow the 'principle of least surprise' to do so :)
Have a great week all.

I think the status 500 probably is logged? But sure logging the error would make sense as well

You're right, the 500 is logged, but not the exception/error details (unless i'm mistaken) so i think this could be helpful.


update kubernetes app to utilize cadvisor snap plugin

lets just remove the "/intel/docker///*/stats/connection/tcp/" for now, and remove the related panels from the dashboards.  We dont want to keep running the docker collector as it does not perform well.  With the way the collector is written, even if we wanted it to just fetch the connection stats, the collector would still try and fetch all stats and then just return the connection stats.

Gotcha, I'll update that now

This should be good to go


Docerina support for record/object field-level documentation

Purpose

Docerina support for record/object field-level documentation

Ballerina:
# Person record
public type Person record {|
    # name of the person
    string name;
    # age of the person
    int age;
    # country code of the person
    CountryCode countryCode;
|};

API Docs:

If field description is given in the record/object Documentation and field-level both, priority is given to the record/object Documentation.
# Apartment record
#
# + number - number of the apartment
# + street - street of the apartment
# + countryCode - country code of the apartment
public type Appartment record {|
    # apartment no
    int number;
    # apartment street
    string street;
    # apartment country-code
    CountryCode countryCode;
|};

API Docs:

Fixes https://github.com/ballerina-platform/ballerina-lang/issues/22307
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #22310 into feature-apidocs will not change coverage by %.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           feature-apidocs   #22310   +/-   ##
================================================
  Coverage            14.59%   14.59%
================================================
  Files                   51       51
  Lines                 1411     1411
  Branches               219      219
================================================
  Hits                   206      206
  Misses                1189     1189
  Partials                16       16

Continue to review full report at Codecov.

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

Why was priority given to the field description at object/record def level? Shouldn't the priority be given for the field level doc? That's the one specified in the spec and the current way of documenting fields in the object/record doc is a convention we came up with.

Why was priority given to the field description at object/record def level? Shouldn't the priority be given for the field level doc? That's the one specified in the spec and the current way of documenting fields in the object/record doc is a convention we came up with.

Added a separate issue to track and resolve this
https://github.com/ballerina-platform/ballerina-lang/issues/22402


Indent using spaces function does not convert tabs to spaces in mixed indentation text content

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

VS Code Version: 1.89.1
OS Version: Linux

Steps to Reproduce:

Open a saved file that has mixed indentation whitespace content (tabs and spaces). ex:

	test with a tab
    test with a space

mixed_indentation_example_file.txt

Ctrl-Shift-P "Indent using spaces", default (4)

I expected this function to convert any tab characters present into the n number of spaces given in the functionality. This is useful for fixing content that has mixed use of whitespace indentation.
workaround: find and replace, regex mode, replace \t with four spaces

there is a separate convert indentation to spaces command

ok, I see that now. So does "Indent using spaces" only set the configuration of the indentation for any subsequent edits, the same as this bottom menu?

To me, calling it "Indent using spaces" indicates an action so I expected this to be the convert function.
I realise now this probably isn't going to be changed.

correct
I agree the naming could be a bit better
Indent with spaces
Spaces for indentation
Space indentation
idk

one could also imagine this function running the convert function and then setting subsequent edit indentation as it does.


fix(backport/1.0): make client shard aware when verifying entries on inactive shards

I realize it's not completely new, but I think it's a bit unfortunate that rekor clients like slsa-verifier need to have logic for sharding. It feels like clients are being exposed to too many implementation details and that's making things brittle.

I completely agree: sharding was meant to be internal client, but it wasn't: things are getting better (the return UUID sharding comparisons eventually can be removed) but seem like they won't be stable until after GA. It's been a pain to deal with this logic and debug.
I should be able to clean up this logic when GA does give us a stable client though. For e.g. we will no longer need to fetch the root hash and search shards for the correct one: a root hash will be returned with the entry.
I'm hesitant to make some of these changes though, because rekor keeps rolling back versions :|

I should be able to clean up this logic when GA does give us a stable client though. For e.g. we will no longer need to fetch the root hash and search shards for the correct one: a root hash will be returned with the entry.

It's a bit unclear from the outside what is GA and what isn't with regard to Rekor and that translates to slsa-github-generator since we rely on the public Rekor.

I'm hesitant to make some of these changes prematurely though, because rekor keeps rolling back versions :|

Completely understandable. We need to work with what we have currently.


Vertical display boards

Hi there! Sorry I'm very new in this, I just need to display the boards vertically not horizontally. Is there any way to change that? Thanks a lot!

I think you mean you need columns to be horizontal, isn't it? If was that, jKanban cannot display that way you need. All jKanban can do can be found at documentation and demo.

Ran across this, I know it's a while since asked...but thought I'd leave a help for others.
I think they actually did mean 'display the boards vertically' ...you are maybe selling the lib short saying it cannot.
To display the boards vertically, is pretty easy:

Just set a max height for the boards.
Set the container to allow the boards to wrap.
Set the container width to force wrap if needed or allow naturally.

Pretty easy.
They now display vertically on wrap, instead of continuing on horizontally.
To make the boards the same height while displaying vertically:

Set the container css
display: flex;
flex-wrap: wrap;

Flex will make all boards the same height in a row if using max-height when the boards expand with items, allowing the vertical display to be orderly.
Hope helps someone and the library.

Note: You can also display boards horizontally as normal and then display the items inside them vertically if you want. Libs should be thought of as a base to add to.

Will check to see if this is a bug in our lib or the jkanban lib or the dragula lib.

Thank you for your feedback. We hopely want you achieve your needs.

Ok. sooo tested. Someone may want to look at this.
Jkanban seems to turn off drag/dropping when a browser window is 700 pixels or smaller in width.
To replicate:

Open the jkanban demo page at: http://www.riccardotartaglia.it/jkanban/
Resize your browser window to 700px or less.
Refresh your browser...either a regular refresh or a shift-refresh
Try to drag any board or any item in a board.
Unable to drag.

We tested in Firefox and Google Chrome in latest versions. Question is why? And why 700px? Puzzling. =)
In our instrumented code it appears that The Dragula Drake disappears ...Uncaught TypeError: KanbanEL.drake.on is not a function
Thanks! And for the cool library. We'd like to try this on mobile, but the above may prevent that. And it's even a problem in a regular horizontal browser display if a user resizes window.
So we hope to be wrong. We'll see if we can find out why. But does 700px ring a bell? Special place to look?
Thanks again!


Extension causes collapsed code sections to expand on save

With formatOnSave set to true, any collapsed text gets expanded when the file is saved.

After Ctrl + S:

Related: https://stackoverflow.com/questions/47325602/how-to-keep-code-folding-on-save-in-visual-studio-code

+1


InfiniteLoader POC

TODO:

[x] Allow usage of infinite loader outside the tree
[x] Dump implementation example
[ ] Figure out row counts and how to make fast check if a certain row is loaded.

Coverage decreased (-2.5%) to 92.513% when pulling 8b230c8c05500ec43d7a0f5873d912d8fc091dcc on infinite-loader into 2671fbdb4a1d0b415160ba876ab307dce90d06b2 on master.


Temporary Disable button

add a disable button for users (people who hate torture). This button should simply remove all listeners and redirect rules happening in background.ts. Only thing that should probably stay active is probably the alarms (not sure though)

just call it "button for the weak"


response headers

how to get them?

Webview is an isolated process, the only way to get headers (and body) requests and responses is using proxy. You can set proxies and certificate. The proxy can run locally, so basically: Your app -> Webview -> Your App (Proxy) -> Webview -> Your App (Proxy) -> ...

I dont think thats true. at least on windows, it seems to be possible to get them:
https://learn.microsoft.com/microsoft-edge/webview2/reference/win32/icorewebview2httpresponseheaders


AttributeError: module 'torch' has no attribute '_six'

Describe the bug
I've recently started experiencing the following issue in the MNIST and the Pneumonia example: AttributeError: module 'torch' has no attribute '_six'.
To Reproduce
Steps to reproduce the behavior:

Submit either MNIST or PNEUMONIA example
Look at the silo's training logs.

Expected behavior
It should run seamlessly.
Screenshot

i have the same  issue

This looks like a problem related to a pytorch version? Does that show systematically in our CI/CD tests?

Ok I was able ot repro as well.
Looks like this import should not happen (depending on torch vs torchvision versions?)
There was a PR to fix this line:
https://github.com/pytorch/vision/pull/7243
Maybe we can just figure out the right combination of torch/torchvision versions? I'm curious why this did not fail our integration tests though.

probably safer by using

pytorch=1.12.1
torchvision=0.13.1

(if we don't need pytorch 1.13)

I updated both pipelines with the latest version of pytorch i.e. 2.0.0 and torchvision i.e. 0.15.1. This resolved the issue.


covr - Test Coverage for prolfqua

Is your feature request related to a problem? Please describe.
investigate on how to apply covr to prolfqua examples
Describe the solution you'd like
https://cran.r-project.org/package=covr

Would be great if we could compute the test coverage including the examples.
It should be possible to run test coverage including the examples but did not get it working in 2019.
See the issue in the covr package
https://github.com/r-lib/covr/issues/358

Working


fix(http.js): ignore static files

:tada: This PR is included in version 1.28.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Launcher Error Failed to construct 'IntersectionObserver': rootMargin must be specified in pixels or percent.

I cannot launch either octave or python kernels.  I get this error:
Launcher Error
Failed to construct 'IntersectionObserver': rootMargin must be specified in pixels or percent.

To reproduce it, just install JupyterLab_app on Ubuntu 21.04 and start it.  Then click the Python 3 launcher.
\
Expected behavior

Context
This is on Ubuntu 21.04.  I have an ordinary jupyter-lab that uses the browser installed too, and it works fine.  I used the .deb to install it just now.
\

Could you please go to "Settings" -> "Advanced Settings Editor" -> "Notebook" and take a screenshot of the settings in both panels, in particular observedBottomMargin and observedTopMargin?

Hi Michal,
I'd be glad to, but I can't find the menu, much less the Settings menu.  Here is what it looks like to me:

Any help?
Thanks,
Rob

That's weird. My guess is that you may have had a different version of JupyterLab installed and its settings are overwriting the jupyterlab-app settings; this would explain missing menu bar. To open Advanced Settings Editor directly you can use Ctrl + ,, but with the new bit of information about missing menu I am quite confident the issue is in the settings files being overwritten.
If you had installed an older version of JupyterLab previously, could you please run jupyter --path from the environment it was installed in and iterate through the config paths, checking if there is a lab/user-settings/@jupyterlab/ directory and trying to rename it temporarily and then restart/reinstall jupyterlab-app?
If you are using Ubuntu I would guess it might be in ~/.jupyter

CC @fcollonval and @echarles as FYI that maybe the transition from older versions could be disrupted by the recent changes in notebook settings and setting-based menus without defaults to fallback to (not requesting any action, just letting you know in case if you noticed this earlier or have ideas to solve this).

Yes, it is in ~/.jupyter as this shows:
frohro@frohro-A475:~$ jupyter --path
config:
/home/frohro/.jupyter
/usr/etc/jupyter
/usr/local/etc/jupyter
/etc/jupyter
data:
/home/frohro/.local/share/jupyter
/usr/local/share/jupyter
/usr/share/jupyter
runtime:
/home/frohro/.local/share/jupyter/runtime
I moved .jupyter to .jupyter_old and restarted JupyterLab_app, but no menu appeared.  Then I removed it including configuration files using Synaptic,  then I did $ sudo apt install ./JupyterLab-Setup-Debian.deb in the terminal.  I'm still seeing this?  Did I miss something?  Do I need to remove the stuff in /etc too?

Thanks,
Rob

I am able to start the settings with CTRL+, but I cannot find the Advanced Settings Editor.  There is a "Notebook" section, but it doesn't have the parameters I was looking for.
Rob

Thank you! This is exactly the Advanced Settings Editor (just labelled as Settings here). This is very useful, indeed there is no "Main Menu" tab, nor the observedBottomMargin/observedTopMargin; this is how it looks like for me:

Could it be that one of the other paths shown in your config section (after running jupyter --path) contains another copy of the lab/user-settings/@jupyterlab/?

Looking more at your screenshot, there is something else that is wrong. The CSS styles seem to correspond to older JupyterLab version (light blue instead of dark blur on selections) and the tooltip displayed after hovering on the Code Console visible on your screenshot shows that it uses 3.0.10 version - this is odd. I am out of ideas, maybe @mbektas could weigh in with some more knowledge.

Okay, one more: is your "ordinary" JupyterLab installation global (on system level rather than contained to a virtual environment)? Could you try uninstalling it and see if this gets fixed?

Uninstalling jupyterlab using pip uninstall jupyterlab seems to have worked!  I guess I have to decide which I'm going to use.  I'll leave it with the JupyterLab_app for a while.  I'm not using it that much this quarter, so it may not get tested fully, but at least now I have a menu.  :-)

Thanks!
Rob

I can also launch a kernel and do some python now!

Hooray!

Uninstalling jupyterlab using pip uninstall jupyterlab seems to have worked! I guess I have to decide which I'm going to use.

Possibly you could have both, but I would just ensure that the versions are the same.

I did a pip install --upgrade jupyterlab and now both versions seem to work.
Thanks!
Rob


Don't specify input file last, since it can cause unpredictable behaviours (overwriting input file)

In exporto0o.ts, the input file is specified last, after all the arguments:
https://github.com/mokeyish/obsidian-enhancing-export/blob/36acb485bd1f4d4a882ab042f28bf8af1cfff71c/src/exporto0o.ts#L178
If the user for some reason leaves a -o with no path specified at the end of the argument list, then Pandoc will overwrite the input file - likely with incompatible text formatting (Word code in an .md file). If the user does not have backups, this could potentially be very bad.
Therefore, I suggest that the input file should be specified first, before the -f argument. This is also the conventional argument order, as presented in the "Getting started" page of Pandoc: https://pandoc.org/getting-started.html#step-6-converting-a-file

To convert it to HTML, use this command:
pandoc test1.md -f markdown -t html -s -o test1.html

Sounds great, would you like to submit a PR to update it?


563 | Binary Tree Tilt | C++

Title - 563. Binary Tree Tilt
What will change - add a new file with a solution for 563. Binary Tree Tilt in C++
Type of Issue -

[x] Adding New Code

Programming Language

[x] C++

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

please assign me this issue

[x] Assigned to @khavinshankar


Remove **kwargs from biogtr.models.GlobalTrackingTransformer

Currently, we have **kwargs in the globaltrackingtransformer even though its mostly unused. We should see if we can remove it so that we don't have some conflicting args which are hard to debug down the line. So far the main issue I've seen is in the config yaml under the model section we have a ckpt_path which is passed to the GlobalTrackingTransformer and the **kwargs allows it to pass through but would cause errors otherwise. We can easily fix this by popping the key out in biogtr.Config.get_model andbiogtr.Config.get_runner()

Completed in #48


fix(env): dev environment

[fixup/dev-environment 08f3ecd] fix(env): dev environment

Codecov Report

Merging #414 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master     #414   +/-   ##
=========================================
  Coverage     93.72%   93.72%
  Complexity      264      264
=========================================
  Files            42       42
  Lines           781      781
=========================================
  Hits            732      732
  Misses           49       49

Impacted Files
Coverage Δ
Complexity Δ

src/Badge/Model/UseCase/CreateVersionBadge.php
100.00% <100.00%> (ø)
6.00 <0.00> (ø)

Continue to review full report at Codecov.

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


Need a proper 404 page

I just realised I'm still using the default Mojolicious 404 page.
Should probably check other error pages too (e.g. ensure nothing sensitive is leaked when an exception is thrown). Maybe make a test end-point that exists just to call 'die'.

Would be nice to go to something like a song page for U2's "I still haven't found what I'm looking for", but that is slightly complex - I'd have to put the song in the DB, have it hidden except when showing the error page.
Also, for other exceptions, maybe Buzzcock's Something's Gone Wrong Again.
For now, maybe just error pages with those songs embedded.

Fixed by PR #290
Basically, I added a "content' page that embeds the aforementioned songs.


Important params unused

Reviewing the code due to an error when executing the bog of "('Connection aborted.', ConnectionResetError (104, 'Connection reset by peer'))" and increasing the same error kept happening, I started to review the code and I see that the parameter frequency = config ['TRADE_OPTIONS'] ['RUN_EVERY']  is not used in any part of the code, that's why this error happens. you could review it?
I have also checked that the parameter coin_sl = order [coin] ['sl'] is not used either, but I'm not sure how important it is to close trades at a loss.

I also noticed that, any ideas  @CyberPunkMetalHead  ?


10-row-level-security.md

English:
enforce
Current Translation:
實施
Suggested Translation:
強制執行
Description:
Inconsistent translation

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Run bugfix tool and waiting for result.


Micro benchmarks addition and testing for FPGAs with DSP blocks

Motivate of the pull request

[ ] To address an existing issue. If so, please provide a link to the issue.
[X] Breaking new feature. If so, please decribe details in the description part.

Describe the technical details

What is currently done? (Provide issue link if applicable)

Previously, OpenFPGA's testing for heterogeneous blocks are limited in input size.
For example, for a 8-bit DSP resource available in FPGA, only 8-bit multipliers are used to test it.
However, the testing coverage is low because testing should cover those cases that <8-bit multipliers are mapped to the resources.

What does this pull request change?

This PR improves the following aspects:

[X] Added micro benchmarks for mac units whose input sizes range from 2 to 32.
[X] Deployed the micro benchmarks in testing an architecture which contains 8-bit DSP blocks
[X] Create a new architecture example which contains 16-bit DSP blocks
[X] Deployed the micro benchmarks in testing an architecture which contains 16-bit DSP blocks. The test case is designed to test that two 8-bit multipliers can be mapped to the multi-mode DSP block successfully.
[X] Added new syntax physical_mode_port_rotate_offset to support pin binding in multi-mode 16-bit DSP blocks
[X] Added HDL modeling for multi-mode 16-bit DSP blocks
[X] Updated documentation

Which part of the code base require a change
In general, modification on existing submodules are not acceptable. You should push changes to upstream.

[ ] VPR
[X] OpenFPGA libraries
[ ] FPGA-Verilog
[X] FPGA-Bitstream
[ ] FPGA-SDC
[ ] FPGA-SPICE
[X] Flow scripts
[X] Architecture library
[X] Cell library

Checklist of the pull request

[X] Require code changes.
[X] Require new tests to be added
[X] Require an update on documentation

Impact of the pull request

[ ] Require a change on Quality of Results (QoR)
[ ] Break back-compatibility. If so, please list who may be influenced.

Issue  #300 is found when creating the test cases and architecture files for the 16-bit multi-mode DSP block

Proof of verification of 12-bit mac unit mapped to FPGA:


Refactor: Do not import NumPy functions directly

We currently import several NumPy functions directly, e.g., here. Normally this isn't an issue, but min, max, mean, etc. override the standard definitions of these functions in Python.
We should refactor the code to import numpy as np then replace all of the NumPy function calls with np.<function name>.

This appears to be done


Rust with cargo doesn't seem to work

I haven't done extensive research on this, but :TestFile in a Rust project is doing something very strange:

it isn't picking up the failed test
it isn't running the tests in the file

I have the following:
mod a {
  #[cfg(test)]
  mod test {
    #[test]
    fn a_failing_test() {
      assert_eq!(false, true);
    }
  }
}

cargo test finds the error, as does :make test (which is configured via rust.vim).
:TestNearest displays the same behaviour as well, even if the point is inside the test, or on the fn a_failing_test itself.
The only thing that correctly picks up the failing test is :TestSuite.
I use dispatch.vim but I get this behaviour even with test#strategy="basic".
How can I go about troubleshooting this?
Thanks!

I have the following error:

I'm just starting to learn Rust and wanted to keep it TDD. But all test commands fail with the message Not a test file. How can I run the unit tests for my Rust source files?


Support for geojson? (feature request)

Since geojson is turning into a de-facto standard, it seems like it would be useful to have geojson support for adding points, similar to the standard leaflet library. It seems like it would be a fairly simple fix, and I would consider contributing if there is sufficient interest.

See #12


[qt5-imageformats] Disable parallel build

D:/installed/x86-windows/tools/qt5/debug/mkspecs/features/qt_plugin.prf:59: Cannot write file D:\buildtrees\qt5-imageformats\x86-windows-dbg\mkspecs\modules-inst\qt_plugin_qwbmp.pri: Cannot create parent directory
jom: D:\buildtrees\qt5-imageformats\x86-windows-dbg\src\plugins\imageformats\Makefile [sub-wbmp-make_first] Error 3
jom: D:\buildtrees\qt5-imageformats\x86-windows-dbg\src\plugins\Makefile [sub-imageformats-make_first] Error 2
jom: D:\buildtrees\qt5-imageformats\x86-windows-dbg\src\Makefile [sub-plugins-make_first] Error 2
jom: D:\buildtrees\qt5-imageformats\x86-windows-dbg\Makefile [sub-src-make_first] Error 2

Looks like the parallel build caused this problem. Disable it.

@Neumann-A I wish this PR can fix that issue.

I would probably try to switch out jom for nmake instead of completely removing parallel builds. Is imageformats small enough so that the build does not take considerably longer?

@Neumann-A Yes, it's very small, I tried to use nmake instead of jom in #8524, but it's not merged yet.


runh isn't able to run on Ubuntu 22.04

See #278

resolved in https://github.com/hermit-os/runh/pull/377


React 18 Streaming SSR support?

Any plans to support React 18 streaming SSR? Now it's a big blocker for many folks.
Here's the upgrade guide for library authors - https://github.com/reactwg/react-18/discussions/110

Dupe of https://github.com/styled-components/styled-components/issues/3658?


lint-staged -> nano-staged

Would you like to use the minimalist analog of nano-staged instead of lint-staged?
https://github.com/usmanyunusov/nano-staged

Sure, why not!
Can you send a PR? also to https://github.com/Aslemammad/tinypool

One of the main features of lint staged for us is clearing README before publishing
Does nano staged support it?

@sheremet-va
This makes clean-publish. Could I be wrong?

@sheremet-va This makes clean-publish. Could I be wrong?

I'm sorry! I was wrong. Yes, clean publish does that.

@usmanyunusov Feel free to send a PR, after this, could you also send one to tinypool?

https://github.com/Aslemammad/tinyspy/pull/12


[Bug]: An Internal IP exposing maybe a threat

Your current environment
While running my vllm code i checked and noticed a self calling interal ip request shown in netsat command i.e(netstat -ano).
HF_DATASETS_OFFLINE=1 TRANSFORMERS_OFFLINE=1 python -m vllm.entrypoints.openai.api_server --port=5000 --host=<IP_ADDRESS> --model "/home/abc/.cache/huggingface/hub/models--meta-llama--Llama-2-13b-chat-hf/snapshots/c2f3ec81aac798ae26dcc57799a994dfbf521496" --tokenizer=hf-internal-testing/llama-tokenizer --tensor-parallel-size=1 --seed 1234 --max-num-batched-tokens=4096
It shows 192.* series that i dont want. Please try to fix this and provide me a solution and I want my host address <IP_ADDRESS> should show as my foreign IP instead of my system ip <IP_ADDRESS>

while running program and then checking connection througfh netstat it shows localhost with some internal ip as well as 192.168..
🐛 Describe the bug
My vllm version is 0.2.2 and I  create the vllm conda environment and hit the below command:-
HF_DATASETS_OFFLINE=1 TRANSFORMERS_OFFLINE=1 python -m vllm.entrypoints.openai.api_server --port=5000 --host=<IP_ADDRESS> --model "/home/abcdt/.cache/huggingface/hub/models--meta-llama--Llama-2-13b-chat-hf/snapshots/c2f3ec81aac798ae26dcc57799a994dfbf521496" --tokenizer=hf-internal-testing/llama-tokenizer --tensor-parallel-size=1 --seed 1234 --max-num-batched-tokens=4096`

On the main branch and next version of vLLM, you can use the flag HOST_IP to override this.


Adding Support for Other Filing Types

Working on #87
So far:

Abstracted top section data class
Abstracted top section manager
Updated 10Q section list
Created 10K section list
Updated 10Q top section manager
Created 10K top section manager
Updated existing and dependencies, etc

To do:

Much more test coverage
Add tests for 10-K
Verify everything works
Double check that empty part names are not an issue

Completed:

Verified everything works
Double checked that empty part names are not an issue
Verified existing 10-Q implementation is effectively unchanged
Fixed formatting issues
updated 10-k parts to match sec standard: https://www.sec.gov/files/form10-k.pdf
Exposed Edgar10KParser to package
Updated part and item matching to support extended range found in files besides 10-Q

To-Do:

Add tests for 10-K

To-Do:

Add tests for 10-K

This kinda falls under #67 and #48 as both still apply, just need to expand the same to 10-Ks
The only thing I can think of from here is that Edgar10QParser and Edgar10KParser are currently duplicated code. assuming all files will follow the same steps, I could abstract it out in a similar to how I did TopSectionManager (but do we even want to do this?

Hi @akaashdash, thanks for the amazing work! 🚀⚡
I have a couple of questions:

In the latest commit, are the outputs of Edgar10KParser and Edgar10QParser essentially same?

Hi @akaashdash, thanks for the amazing work! 🚀⚡
I have a couple of questions:

In the latest commit, are the outputs of Edgar10KParser and Edgar10QParser essentially same?
 And is the unit tests accuracy of 10-Q filings same as it was before the new changes?

Edgar10KParser and Edgar10QParser are the same except one uses TopSectionManagerFor10K and the other uses TopSectionManagerFor10Q. At its core, both TopSectionManagers use the same code, just the list of headings is changed. Therefore, both will have the same output when ran on their respective filings. However, using a 10Q on the 10K parser or a 10K on a 10Q parser will still error out, but this is expected behavior as this is also how it was before my changes.

Hi @akaashdash, thanks for the amazing work! 🚀⚡
I have a couple of questions:

In the latest commit, are the outputs of Edgar10KParser and Edgar10QParser essentially same?
 And is the unit tests accuracy of 10-Q filings same as it was before the new changes?

I forgot to answer the second part sorry. I was testing my code manually, but unit tests for 10Ks need to be added. Not sure how to do it without breaking existing tests, some help on this would be appreciated.
All tests for 10Q have the exact same results as before, no changes on that front.

Hi @akaashdashm thanks for the answering the questions. I would strongly recommend writing tests for 10-K parser like we did for 10-Q parser as we need to cover extremely large number of filings / edge cases.

.. Not sure how to do it without breaking existing tests, some help on this would be appreciated.

Please go through Discussion 48 to understand how we have setup tests for 10-Q (or parsing test in general). Let me know if something doesn't make sense!

Completed:

Added exploratory tests for 10-K
Confirmed that all unit/regression/etc tests still perform the exact same