v11 --> v12 breaks edge position and placement when loading images

What platform were you using when you found the bug?
  System:
    OS: Linux 6.6 Manjaro Linux
  Binaries:
    Node: 18.19.0 - ~/.nvm/versions/node/v18.19.0/bin/node
    npm: 10.8.2 - ~/.nvm/versions/node/v18.19.0/bin/npm
    pnpm: 9.6.0 - ~/.nvm/versions/node/v18.19.0/bin/pnpm
  Browsers:
    Chromium: 127.0.6533.88
  npmPackages:
    @xyflow/react: ^12.0.4 => 12.0.4
    next: ^14.2.3 => 14.2.3
    react: ^18.3.1 => 18.3.1
    typescript: 5.5.4 => 5.5.4

Live code example
No response
Describe the Bug
I just updated my repo to the latest version of RF and I noticed that the behavior of the edge placement and updating has changed. Specifically when loading dynamic content such as videos and images, the edges used to update automatically. This also applied when dealing with dynamic CSS modifications (e.g changing the height of something in the node).
I deal with unknown content, so there's no way to prevent layout shifts. Also the scope and the number of images, make it unfeasible to constantly track image loading state and then call useUpdateNodeInternals.
V11
Position is correct, after content is loaded edges are updated.

V12
Position is incorrect, after content is loaded edges remain where they were mounted initially.

I'm wondering what has changed specifically that has caused this issue? Is this is design choice on your behalf and expected behavior?
Steps to reproduce the bug or issue
This can be tried using the custom node example with a large image
Expected behavior
I expect the edges to automatically shift to correct position.
Screenshots or Videos
No response
Additional context
No response

This definitely seems like a bug. We still very much support dynamically resized nodes.
Is there a chance you add the handles dynamically after the image has loaded?

@peterkogo I add the handles before the image is loaded. In this case the handles are always present.
This is the gist of the nodes on the right side, sorry if there are too many abstractions.
    <BaseNode
      {...nodeProps}
      className={cn('w-[15rem]', nodeData.inputs.image?.data && 'w-[30rem]')}
      pureNodeComponent={
        PureImageViewerNode as TypePureNodeComponent<TypeImageViewerNode>
      }
      hideMenuBar
    >

          {Object.values(nodeData?.outputs || {}).map((output) => (
            <SocketOutput output={output} key={output.id} />
          ))}
          {Object.values(nodeData?.inputs || {}).map((input) => (
            <SocketInput input={input} key={input.id} showLabel={false} />
          ))}

          {!nodeData.inputs.image?.data ||
          nodeData.inputs.image?.data === '' ? (

              <Icon
                icon="solar:gallery-bold"
                width="100%"
                className="w-8 text-neutral-400"
              />

          ) : (
            // eslint-disable-next-line @next/next/no-img-element

          )}

    </BaseNode>

Seems to be only happening with images. I replaced the image in one of the nodes with a square div. Seems to be working fine in that case
Before

After

Thanks for bringing this up. It should work automatically, since nodes change their dimensions. We will check it!

I think I'm running into a similar issue with dynamically sized nodes, probably due to the new node.measured stuff. I'll see if I can reproduce it.
The migration from v11 to v12 has turned out to be more challenging than I expected.

I can't reproduce this issue https://codesandbox.io/p/sandbox/sad-estrela-t22fx7?file=/src/nodes/index.ts - It would be super helpful if you could create an example for this.
@alexanderhorner That's sad to hear! If you find some time, you could open an issue or just write us a mail with your experiences.

I think I found the issue while creating the reproduction repo. The violating node attributes are width and height. If these are present dynamic positioning breaks. Rereading the docs I can see that Using width and height now means that the dimensions are not dynamic based on the content but fixed. smh

I was about to comment the same 😄
Good that you figured it out yourself though 👍

Nice, that you could solve it already. That's one of the biggest changes with React Flow 12.

Wondering if width and height should be ignore if measured is present? @moklick

measured is not something that is set by the user but whenever the resize observer detects a change and on first render.

My issue seems to arise from measured not being calculated correctly during the initial render. I rely heavily on custom edge logic that depends on width and height. It would be fine if it immediately re-rendered once the heights are known, but that doesn’t seem to be happening. Additionally, I'm facing issues when the size of a node changes due to content.
I'm not entirely sure what change is causing this. Another problem I'm encountering is an error in applyNodeChanges, also related to measured height/width (dimension change). It indicates that the object is read-only, but I haven't been able to reproduce this consistently either.

@moklick I still encounter strange issues with initialWidth/Height. This happens specifically when I throttle my connection using chrome's dev tools to Slow 3G, and the edges fail to be positioned properly. On speedy connections everything works as expected.
Also  opening this up again for @alexanderhorner.

@hsab could you update the repo you've posted yesterday? I can't reproduce the throttled network issue.
@alexanderhorner if you have a repo or codesandbox that would help!

I was working on a minimal reproduction, but unfortunately, I can't spend more time on this, at least for now.
My custom edge tries to find the closest edge and also uses node.measuredDimensions for that. However, when the dimensions change, the edge is not rerendered.
This could be fixed by providing a way to subscribe to these changes using the Zustand store.
I had a similar problem with the edge not re-rendering when the position of the actual handles changes. The fix was to subscribe to the internal handleBounds object and re-render the edge if something changes.
  const sourceHandleBounds = useStore(reactFlowStore, s => s.nodeLookup.get(source)?.internals.handleBounds)
  const targetHandleBounds = useStore(reactFlowStore, s => s.nodeLookup.get(target)?.internals.handleBounds)

Seems like a hacky work around.

The other problem is happening here:
onNodesChange: (changes) => {
  const nodes = get().nodes
  set({
    nodes: applyNodeChanges(changes, nodes) as TableNode[],
  })
}

index.mjs:582  Uncaught TypeError: Cannot assign to read only property 'width' of object '#<Object>'
    at applyChange (index.mjs:582:34)
    at applyChanges (index.mjs:557:13)
    at applyNodeChanges (index.mjs:617:12)
    at createReactFlowSlice.ts:48:16

Adding structuredClone seems to fix that issue.
onNodesChange: (changes) => {
  const nodes = structuredClone(get().nodes) // like the old JSON.parse(JSON.stringify(get().nodes)) hack
  set({
    nodes: applyNodeChanges(changes, nodes) as TableNode[],
  })
}

I tried reproducing this issue in CodeSandbox but wasn't able to. I would love to spend more time on this, but something else came up, so we've put the React Flow 12 update on hold for now. Hopefully, this information is still useful to you.

Fixed the edge calculations by also applying the same subscribe "hack"
  const targetDimensions = useStore(reactFlowStore, s => s.nodeLookup.get(source)?.measured)
  const sourceDimensions = useStore(reactFlowStore, s => s.nodeLookup.get(target)?.measured)

Now the custom edge rerenders when the measured dimensions change.

@moklick hmmm can't seem to reproduce the issue anymore. so that's good news! :) Will report back if it happens again. Cheers!

removing width and height fixed it for you @hsab ?

@moklick so at first I was using width and height incorrectly and the edges were not positioning properly. Removing those fixed the issue so I decided to explore initialWidth and initialHeight. With the initial values the edges were not positioning probably on slow networks (3G and Slow 3G). But since this morning there issue has been fixed and I really didn't change anything so I'm wondering what could have caused it.
I'm going to continue using the initial values because I save the store to an online database so the user can access it later on. And when I am saving the store I basically use the measured values for width and height to set the initial properties.
So far it seems to have been magically fixed. But since I'm using initialWH values I assume if it's an actual bug it's going to resurface again and if it does so I'll report back here.
I really appreciate your help though. You guys have amazing support and have built an incredible product! Kudos

Thanks for the update and your kind words! I will leave this open for a bit longer until we figured out what is happening with initialWidth/Height and wait for @alexanderhorner s feedback.
btw @hsab what are you building with React Flow?

@moklick bad news! the issue is still there. It is inconsistent, sometimes it works, sometimes it doesn't. Behaves unexpectedly on each visit, but more interestingly it behaves different in the same graph. So some edges are correct, some misaligned. See this photo. Green is good, red is bad

I can't seem to reproduce this out side of my app, need to investigate more.
ps. we're building a node-based creative suite for media processing. still in stealth but will definitely share once we're ready :)

We will investigate.

@hsab just spitballing here, are you by any chance using custom edges as well or just the built-in ones?

This seems like a very weird edge case, since initialWidth/-Height might be the cause of this. But this could also be some kind of race condition of the way you render your handles programmatically (even though you are using keys).
If you are interested, we can connect on our Discord to get to the bottom of this.

hi @peterkogo! I will reach out on discord for sure. appreciate you lending a hand. I agree that there's something really weird going on. To answer you earlier question, I do use custom edges but only as a placeholder for future development. This is how it's currently coded
import {
  BaseEdge,
  EdgeProps,
  Position,
  getSimpleBezierPath,
} from '@xyflow/react';
import React from 'react';

const CustomEdge = ({
  sourceX,
  sourceY,
  targetX,
  targetY,
  ...props
}: EdgeProps) => {
  const [edgePath] = getSimpleBezierPath({
    sourceX,
    sourceY,
    targetX,
    targetY,
    sourcePosition: Position.Right,
    targetPosition: Position.Left,
  });

  return <BaseEdge path={edgePath} {...props} />;
};

export default CustomEdge;

The other problem is happening here:
onNodesChange: (changes) => {
  const nodes = get().nodes
  set({
    nodes: applyNodeChanges(changes, nodes) as TableNode[],
  })
}

index.mjs:582  Uncaught TypeError: Cannot assign to read only property 'width' of object '#<Object>'
    at applyChange (index.mjs:582:34)
    at applyChanges (index.mjs:557:13)
    at applyNodeChanges (index.mjs:617:12)
    at createReactFlowSlice.ts:48:16

Adding structuredClone seems to fix that issue.
onNodesChange: (changes) => {
  const nodes = structuredClone(get().nodes) // like the old JSON.parse(JSON.stringify(get().nodes)) hack
  set({
    nodes: applyNodeChanges(changes, nodes) as TableNode[],
  })
}

I tried reproducing this issue in CodeSandbox but wasn't able to. I would love to spend more time on this, but something else came up, so we've put the React Flow 12 update on hold for now. Hopefully, this information is still useful to you.

So I figured out what the issue was. I'm using zustand together with immer. Turns out immer freezes objects deeply. But applyNodeChanges mutates the measured  object inside a node.
The other issue I had was also resolved by me subscribing to the changes manually by going into the reactflow store. Still feels a bit hacky and I wish there was a better solution for that.

@alexanderhorner for your immer issues, I think you can use current or the original methods from immer. Performance is much faster than structuredClone and actually even 2x faster than const { nodes } = state
additionally I highly recommend disabling autofreeze via setAutoFreeze(false)
onNodesChange: (changes: NodeChange[]) => {
  set((state) => {
    const { nodes } = current(state);
    state.nodes = applyNodeChanges(changes, nodes);
  });
}

Yes it was auto freeze, took a long time to figure out...
Why is const { nodes } = current(state) faster then const { nodes } = get() (get the method from the zustand store)?

@alexanderhorner I actually have no idea! I just noticed it as I was testing the performance of my onNodeChanges  method. But if I were to guess, I'd say by using current you're essentially bypassing all the proxy logic. Without currentc the function below was hitting 0.8ms while using current brought it down to 0.3ms.
To be clear, the performance gain can also be specific to my implementation. My nodes and edges are stored as key/value objects  instead of the default array[] format that is recommended by RF. So I have to do some logic here and there for updates/sets, but the key/val setup makes it much easier to access and modify individual nodes. outside of react components where I don't have access to hooks, without incurring a O(n) search penalty.
this is my actual function
onNodesChange: (changes: NodeChange[]) => {
  set((state) => {
    const { nodes } = current(state);

    const changedNodes = applyNodeChanges(
      changes,
      Object.values(nodes) as Node[],
    ) as AnyFuserNode[];

    for (const change of changes) {
      if (change.type === 'remove') {
        delete state.nodes[change.id];
      }

      if (
        change.type === 'select' &&
        !change.selected &&
        state.lastAddedNode === change.id
      ) {
        state.lastAddedNode = undefined;
      }
    }

    for (const node of changedNodes) {
      state.nodes[node.id] = castDraft(node);
    }
    state.lastUpdate = Date.now();
  });
},

I have the same error but can't precisely reproduce it. I don't use immer. I do programmatically change the size of nodes on hover. Disabling that doesn't solve it.

@ThimoDEV are you working with a custom onNodesChange implementation?

@ThimoDEV are you working with a custom onNodesChange implementation?

const [nodes, setNodes, onNodesChange] = useNodesState<Node>([]);   onNodesChange={(changes) => {           onNodesChange(changes);         }}
I use this, its the most basic I can have else nothing gets rendered. Can I let reactflow take over this part or something?

Should we keep this issue open? Is this still an issue @hsab and @alexanderhorner ?

still experiencing edge mis-placement when using initialWidth and initialHeight

@moklick I actually managed to fix the issue! There was in fact a race condition going on with one of my useEffects @peterkogo tysm for the tip! 🙏🏼

Great! I will close this for now. Don't hesitate to reopen this issue or to write us on DIscord  if the problem persists.


feat: support @httpApiKeyAuth trait

Issue #, if available: Fixes #453.
Description of changes: This adds codegen support for services and operations that use the @httpApiKeyAuth trait.
The generated client will have a new apiKey config attribute. This value will be injected into the header that's specified in the trait by new middleware that's included in the codegen client.
⚠ I've done my best to base this on existing patterns but I'm sure that there will be comments and things that need to change. Please let me know and I'll happily make adjustments.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

First round of fixes pushed, will rebase & squash them once I've got all the comments addressed.

@kstich @mtdowling @JordonPhillips what are the next steps I need to take for this PR? Is there any chance you'll have an opportunity to review soon? I don't mean to be pushy, just don't know what timeline to expect here as I haven't heard anything since Tuesday afternoon.
I left the changes from code review comments as separate commits so they would be easier to review, but I could rebase, squash back to a single commit, and mark ready for review. I think I've addressed all of Kevin's comments and the tests (unit tests as well as manual testing of a generated client) seem to indicate this does what I hoped it would.

Pushed a change (f0ee2e5) to delete a test; not 100% sure I understood the intended behaviour of the auth traits so backed off on this one as I discovered it was failing and it seemed like the expectation was incorrect. Would love to hear if the test was legitimately wrong or if there's a bug in the code.

Pushed changes to address most comments; still need to do the work to support an API key in a query parameter.

I think I've addressed all of the review comments so far. I've realized, though, that I'm missing the validator to check for conflicts. I probably won't get to that today, maybe tomorrow.

Silly me, I got confused, the validator is for a different PR. I think this is ready to be re-reviewed.

I think this is ready to move to non-draft, I've initiated the CI run.

Thanks @kstich !

Pushed mixup commits to address comments and fix missing @jest/types dependency so that the generated code can build & test.

Pushed a behavioural change in f2f40d3 so that the middleware doesn't throw an error when the API key is not available, because the middleware doesn't know if there is another auth mechanism available (for example if an operation can be authorized by an API key or a bearer token, it shouldn't fail because the API key isn't present).


s3 exception when no credentials set

It appears as though a cryptic error is thrown if no credentials are defined in an S3 block and no credntials are defined locally either:

Traceback (most recent call last):
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/aiohttp/connector.py", line 986, in _wrap_create_connection
    return await self._loop.create_connection(*args, **kwargs)  # type: ignore[return-value]  # noqa
  File "/usr/lib/python3.10/asyncio/base_events.py", line 1049, in create_connection
    sock = await self._connect_sock(
  File "/usr/lib/python3.10/asyncio/base_events.py", line 960, in _connect_sock
    await self.sock_connect(sock, address)
  File "/usr/lib/python3.10/asyncio/selector_events.py", line 499, in sock_connect
    return await fut
asyncio.exceptions.CancelledError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/aiohttp/client.py", line 535, in _request
    conn = await self._connector.connect(
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/aiohttp/connector.py", line 542, in connect
    proto = await self._create_connection(req, traces, timeout)
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/aiohttp/connector.py", line 907, in _create_connection
    _, proto = await self._create_direct_connection(req, traces, timeout)
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/aiohttp/connector.py", line 1175, in _create_direct_connection
    transp, proto = await self._wrap_create_connection(
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/aiohttp/connector.py", line 985, in _wrap_create_connection
    async with ceil_timeout(timeout.sock_connect):
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/async_timeout/__init__.py", line 129, in __aexit__
    self._do_exit(exc_type)
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/async_timeout/__init__.py", line 212, in _do_exit
    raise asyncio.TimeoutError
asyncio.exceptions.TimeoutError

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/aiobotocore/httpsession.py", line 178, in send
    response = await self._session.request(
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/aiohttp/client.py", line 539, in _request
    raise ServerTimeoutError(
aiohttp.client_exceptions.ServerTimeoutError: Connection timeout to host http://<IP_ADDRESS>/latest/api/token

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/prefect/cli/_utilities.py", line 41, in wrapper
    return fn(*args, **kwargs)
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/prefect/utilities/asyncutils.py", line 212, in wrapper
    return run_async_in_new_loop(async_fn, *args, **kwargs)
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/prefect/utilities/asyncutils.py", line 141, in run_async_in_new_loop
    return anyio.run(partial(__fn, *args, **kwargs))
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/anyio/_core/_eventloop.py", line 70, in run
    return asynclib.run(func, *args, **backend_options)
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 292, in run
    return native_run(wrapper(), debug=debug)
  File "/usr/lib/python3.10/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/usr/lib/python3.10/asyncio/base_events.py", line 646, in run_until_complete
    return future.result()
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 287, in wrapper
    return await func(*args)
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/prefect/cli/deployment.py", line 577, in build
    deployment = await Deployment.build_from_flow(
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/prefect/deployments.py", line 598, in build_from_flow
    await deployment.upload_to_storage()
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/prefect/deployments.py", line 465, in upload_to_storage
    file_count = await self.storage.put_directory(
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/prefect/filesystems.py", line 459, in put_directory
    return await self.filesystem.put_directory(
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/prefect/filesystems.py", line 342, in put_directory
    self.filesystem.put_file(f, fpath, overwrite=True)
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/fsspec/asyn.py", line 111, in wrapper
    return sync(self.loop, func, *args, **kwargs)
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/fsspec/asyn.py", line 96, in sync
    raise return_result
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/fsspec/asyn.py", line 53, in _runner
    result[0] = await coro
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/s3fs/core.py", line 1015, in _put_file
    await self._call_s3(
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/s3fs/core.py", line 325, in _call_s3
    await self.set_session()
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/s3fs/core.py", line 473, in set_session
    self._s3 = await s3creator.__aenter__()
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/aiobotocore/session.py", line 22, in __aenter__
    self._client = await self._coro
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/aiobotocore/session.py", line 102, in _create_client
    credentials = await self.get_credentials()
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/aiobotocore/session.py", line 133, in get_credentials
    self._credentials = await (self._components.get_component(
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/aiobotocore/credentials.py", line 814, in load_credentials
    creds = await provider.load()
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/aiobotocore/credentials.py", line 486, in load
    metadata = await fetcher.retrieve_iam_role_credentials()
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/aiobotocore/utils.py", line 175, in retrieve_iam_role_credentials
    token = await self._fetch_metadata_token()
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/aiobotocore/utils.py", line 88, in _fetch_metadata_token
    response = await session.send(request.prepare())
  File "/home/user/.cache/pypoetry/py3.10/lib/python3.10/site-packages/aiobotocore/httpsession.py", line 210, in send
    raise ConnectTimeoutError(endpoint_url=request.url, error=e)
botocore.exceptions.ConnectTimeoutError: Connect timeout on endpoint URL: "http://<IP_ADDRESS>/latest/api/token"
An exception occurred.

Thanks for reporting this!
I think this is related to the core Prefect's filesystem
https://github.com/PrefectHQ/prefect/blob/main/src/prefect/filesystems.py#L386

Please re-open if you still encounter this.


Remove '-dev' flag from 'npm install' command in the docs

Hi.
I noticed in the readme.md that the install command has the --save-dev option. The -dev option is only used for dependencies that are supposed to work only in the development environment, which is not the case of vuetable-2.

This little thing is a gem for my knowledge. Thanks a lot!

Thanks a lot @flowck. It is now updated in documentation site here


interop contracts for forked networks

When forking, the interop predeploy contracts do not exist. We'll need to manually apply them on startup so the contracts are available
Ideas:

Look into anvil_setCode or something at runtime to manually apply the bytecode. consider the constructor argument. If so, some storage slots to override if needed.
Look into genesis generation and pulling the bytecode from there
Set the dependency set information accordingly

cc @jakim929
Perhaps by default we dont apply these changes to forked networks. Instead we have sets of experimental flags that patch remote network state supersim fork --experiment.interop.


where are the gulpfile.js?

Good morning,
Where I find the gulpfile in the polymer-stater-kit ?
Thank!

This is the file you're looking for: https://github.com/PolymerElements/polymer-starter-kit/blob/master/gulpfile.js
For Polymer Starter Kit issues or questions, please note that https://github.com/PolymerElements/polymer-starter-kit/ is the repo you're after.


Docs.MS Release Updates for azure-abort-controller

Update docs metadata and targeting for release of azure-abort-controller

Docs Build status updates of commit 729d420:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-services/latest/abort-controller-readme.md
:warning:Warning
View (azure-node-latest)
Details

ci-configs/packages-latest.json
:white_check_mark:Succeeded

docs-ref-services/latest/abort-controller-readme.md

Line 2, Column 1: [Warning-title-missing] Missing required attribute: 'title'. Add a title string to show in search engine results.
Line 9, Column 16: [Warning-ms-prod-technology-invalid] Invalid value for 'ms.technology': 'azure' is not valid with 'ms.prod' value 'azure'.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


ci: add changelog link in release notification

:tada: This PR is included in version 0.2.1-alpha.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


add collapsible_headings extension

This extension is a newly-built version of the old, buggy, testing/hierarchical_collapse extension.
Collapsible headings, a la mathematica, are neat. I liked them, so I made them. This version also offers integration with the config page, optional toolbar button and keyboard shortcuts. Hopefully useful to others as well as just me...

Very nice. Can't try it out now, but this looks really useful.


LG C4

Hi I was just wondering if this works with the new lg c4 tv?

Yeah it should

It does

Closed / resolved


Dynamic Programming

Description
I noticed that DP section contains only two patterns of DP popular problems.
I know more patterns which I have trained on and solved a lot like :

LIS ( Longest Increasing Subsequence)
LCS (Longest Common Subsequence)
Coin exchange
Consecutive Ranges
Nested Ranges
Counting
Building Output

I can write clean standard code for all of them and showing time and space complexities with comments.
I will provide code with links to the standard problems of the patterns related to them.
please assign this issue to me.
Domain
Algorithm
Type of Contribution
Addition
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

You can only pick one problem at a time not many , create a issuse for one at a time and we'll be happy to accept that .

Kindly just work on Longest Increasing Subsequence, and make sure you come up with O(n^2) as well as o(n) time complexity , failing to come up with o(n), I shall reduce the question level to level1.
all the best, Go for it


Due to service inactivity, the client buffer exceeded maximum size. Resetting the buffer, Issue with cognitive-speech

Thanks for reaching out to us and reporting this issue. I am pasting the response from this issue thread: https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/1634 AS-IS.
Could you please follow the below sugesstion ?

the Speech SDK maintains an internal buffer of input audio (to cope with momentary connection issues, for example) and discards processed audio based on results from the service. If the service does not respond (fast enough), eventually you'll get this error as the buffer overflows.

Without any code or Speech SDK log, we can only comment in general:

Do not push audio faster than 1x real-time rate (like audio coming from a microphone). The service cannot necessarily keep up for long. If it's difficult to control this, consider using a pull stream instead.

Do not modify initialSilenceTimeout or endSilenceTimeout properties. Especially setting initialSilenceTimeout zero when input has long periods of silence is guaranteed to trigger this error. (ref. https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/1632#issuecomment-1233362574)

It is also possible that the service had genuine performance issues, but logs would be needed to find out more.

Originally posted by @navba-MSFT in https://github.com/Azure/azure-sdk-for-python/issues/28333#issuecomment-1383516363
I am using azure-cognitiveservices-speech==1.29.0 python3.10 SDK , initially it was working fine but now even with small files like 12MB size and length of 1 minute it stucks in this error
Note I also added pronunciation config to my speech_config but it should be deal
here am attaching my log file
my_logs_speechsdk.txt
I created a support ticket on the Azure portal, however no updates at all
Therefore I referenced this issue since it's fixed with clear details and solution

Thanks for the feedback, we’ll investigate asap.

Thanks for the feedback, we’ll investigate asap.

Thank you

An update to this was posted in https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/1989

Closing the issue and please use https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/1989 to track it.


Dart double and int troubles

If the json has numbers that can appear as both, for example, 1 or 1.5 the code will error trying to assign the wrong type to the int or double.

Use this template to treat with doubles in dart:
(json['value'] as num).toDouble()

I'm sorry, can you provide an example?

If you give an sample that has 1.5, but then actually try to convert data that has 1 it will error because int is not a double.
If you give an sample that has 1, but then actually try to convert data that has 1.5 it will error because double is not an int.
You can use the above solution by @brooth to prevent these errors.

@dvdsgl Please add the option for num. I want to use quicktype over json_serializable. But I get way too many errors when I don't know if the server will always return a int or a double. And It takes way too much time to go back and fix the data.

@dvdsgl Super needing to use num instead of int or double. https://github.com/flutterchina/json_model uses num.


#define ZXING_VERSION_MAJOR...

#define ZXING_VERSION_MAJOR  1
#define ZXING_VERSION_MINOR  0
#define ZXING_VERSION_PATCH  3
This would be extremely helpful. For more than obvious reasons I think. I can do it. I just want to make sure everyone is on board.
PS: Maybe in the "ZXConfig.h" file?

ZXConfig.h is good me.


Improper EXTI reset in HAL_GPIO_DeInit()

Dear @ALABSTM,
TL;DR
The HAL_GPIO_DeInit() contains a bug related to the order in which GPIO configuration and EXTI configuration is reset to its "reset defaults". The issue seems not to be present in other HAL drivers, such as for example in stm32f4xx_hal_driver. In those HAL drivers, disabling the EXTI interrupt comes before resetting the GPIO configuration as it should be.
Scenario
Let's assume you have an active low switch (switch with external pull-up) connected to a GPIO pin and one would like to trigger an EXTI interrupt on the falling edge when closing that switch. But, in the meantime (since number of GPIO pins is limited), that GPIO is shared with another hardware resource. In rest, the GPIO is configured for that latter/second resource. One would typically see the following scenario to switch to input with EXTI interrupt on falling edge:

Reconfigure the GPIO for EXTI with GPIO_MODE_IT_FALLING and all boiler plate code using GPIO_Init().
Wait for the EXTI interrupt to fire or timeout
Restore GPIO configuration to map on that second hardware resource with GPIO_Init().

Problem
In the scenario above, when that hardware resource is for example an output and the output isn't driven, the EXTI interrupt might fire since a '1'-to-'0' transition is observed on the EXTI line and that EXTI interrupt is still enabled.
Cause
The only way to clear EXTI configuration is by using HAL_GPIO_DeInit()
Solution
Use HAL_GPIO_DeInit() to clear EXTI configuration to reset defaults.
Follow-up problem 1
The solution for the problem causes another problem: in HAL_GPIO_DeInit() the I/O mode is reset to analog high impedance before the EXTI is disabled!
https://github.com/STMicroelectronics/stm32l0xx_hal_driver/blob/b6be1fcf863fe05224b0b858ee091f3376d80d0c/Src/stm32l0xx_hal_gpio.c#L311-L336
The GPIO analog high impedance configuration disables the internal TTL Schmitt trigger producing a '0' on its ouput (See Figure 26 on page 242 in RM0376). Again, a '1'-to-'0' transition is observed on the EXTI line and that EXTI interrupt is still enabled. The EXTI interrupt will fire again.
Cause
The code block responsible for resetting the I/O mode of the GPIO comes before the code block responsible for resetting the EXTI configuration.
Solution
Swap the two code blocks in HAL_GPIO_DeInit() around.
Follow-up problem 2
There is still another problem in the code block responsible for resetting the EXTI configuration itself: the line multiplexer is reset  before the EXTI is masked! This means that somehow the input data on GPIOA is '0', again a '1'-to-'0' transition is observed on the EXTI line and that EXTI interrupt is still enabled. The EXTI interrupt will fire again.
Cause
The code resetting the EXTI line multiplexer:
https://github.com/STMicroelectronics/stm32l0xx_hal_driver/blob/b6be1fcf863fe05224b0b858ee091f3376d80d0c/Src/stm32l0xx_hal_gpio.c#L332-L333
comes before the code resetting the EXTI interrupt masks:
https://github.com/STMicroelectronics/stm32l0xx_hal_driver/blob/b6be1fcf863fe05224b0b858ee091f3376d80d0c/Src/stm32l0xx_hal_gpio.c#L335-L341
Solution
Swap the two code blocks around.
What do you think? Could you please take a look at this?
Thanks,
Best regards,
Jelle De Vleeschouwer

Dear @jelledevleeschouwer,
First of all, allow me to thank you for this detailed report. The accuracy of your analysis is remarkable.
We are already aware of this issue and have fixed it internally. The fix will be available in a future release. Unfortunately we cannot share a date for the moment. We count on your patience and comprehension.
However, I would like to ask you about what you described in the "Follow-up problem 2" section. Could you systematically reproduce the issue while still having the the line multiplexer reset before the EXTI is masked?
Thank you in advance for your answer and thank you once more for this detailed report.
With regards,

ST Internal Reference: 77661

Hi @jelledevleeschouwer,
Any reply from your side?
With regards,

Hi @ALABSTM,
Yes, at the time I could reproduce the problem described in "Follow-up problem 2", but I don't have the time to try and reproduce the issue right now.
But the idea is simple, no? Imagine you have a GPIO (say, PB6) with external pull-up configured as digital input with EXTI interrupt on a falling edge. In rest, on the PB6 GPIO a high level '1' is present. But let's imagine you call HAL_GPIO_DeInit() now with a fix for the first issue as follows:
            /*------------------------- EXTI Mode Configuration --------------------*/
            /* Clear the External Interrupt or Event for the current IO */

            tmp = SYSCFG->EXTICR[position >> 2U];
            tmp &= (((uint32_t)0x0FU) << (4U * (position & 0x03U)));
            if(tmp == (GPIO_GET_INDEX(GPIOx) << (4U * (position & 0x03U))))
            {
                tmp = ((uint32_t)0x0FU) << (4U * (position & 0x03U));
                SYSCFG->EXTICR[position >> 2U] &= ~tmp;

                /* Clear EXTI line configuration */
                EXTI->IMR &= ~((uint32_t)iocurrent);
                EXTI->EMR &= ~((uint32_t)iocurrent);

                /* Clear Rising Falling edge configuration */
                EXTI->RTSR &= ~((uint32_t)iocurrent);
                EXTI->FTSR &= ~((uint32_t)iocurrent);
            }

            /*------------------------- GPIO Mode Configuration --------------------*/
            /* Configure IO Direction in Input Floting Mode */
            GPIOx->MODER |= (GPIO_MODER_MODE0 << (position * 2U));

            /* ... */

The code addresses SYSCFG_EXTICR2 in case of GPIO_PIN_6. tmp will be be 0x0100 when entering the if-statement. (GPIO_GET_INDEX(GPIOx) << (4U * (position & 0x03U))) will also yield 0x0100 which makes the code enter the if-statement. In the  if-statement EXTI6[3:0] are cleared in SYSCFG_EXTICR2, which makes the selects the PA6 GPIO to be selected as the source input for the EXIT6 external interrupt. At this point, the EXTI line is still not masked!
If at this point, the PA6 GPIO was now configured as Digital Input and a low value '0' is produced on that GPIO, a '1'-to-'0' (falling edge) will be detected by the EXTI "edge detect cricuitry" and thus will set the Pending Interrupt flag on line 6 (PIF6) in the EXTI_PR register.
This is the behavior that I observed. Could that be plausible, or am I missing something?
Thanks,
Best regards,
Jelle

Hi @jelledevleeschouwer,
I hope you are fine. I noticed you attached p-r #2 to this issue. The fix you proposed is already covered by the one published in the frame of v1.10.3. I assume the p-r covers both problems 1 and 2 you described above.
Please allow me then to close this issue. Thank you for your comprehension and thank you again for your detailed and well structured report.
With regards,


Refactor pool fetching to use N factories.

This PR refactors pool fetching to be on a Vec<dyn InternalPoolFetching>. This means that adding new pool factories just requires you to add a new InternalPoolFetching implementation to that list. Additionally, a Registry<Factory>: InternalPoolFetching type was  introduced to wrap a FactoryIndexing implementation into a InternalPoolFetching making the process of adding new factories even easier.
This PR has +500 lines, but there is a good chunk of that is just moved code. For example the logic in pool_cache module was split into pool_fetching::cache and pool_fetching::registry. I tried to mark the sections of moved code without changes to make reviewing easier.
One additional note - the BalancerPoolRegistry type is actually no longer needed. The only place it gets used is in a unit test that verifies that registering pools via events matches pools in the subgraph. I want to remove this file, but will do it in a separate PR to not make this one even bigger.
There is an additional follow up "TODO" which is to get rid of the FetchedBalancerPool, WeightedPool and StablePool types in favour of Vec<Pool>. This would also be a larger change that touches a much larger part of the codebase, so it was left for later.
Test Plan
There was very little new logic in this PR, most of it was moving and combining existing logic in a different way. There was some new tests added for the new stuff, otherwise we moved and adapted existing tests.
You can also run the solver with only Balancer pools as liquidity as a sanity check that we are indeed still fetching pools:
$ cargo run -p solver -- --baseline-sources BalancerV2 --solver-account 0xa6DDBD0dE6B310819b49f680F65871beE85f517e --transaction-strategy DryRun
...
2021-12-15T19:46:20.728Z DEBUG solver::liquidity_collector: got 35 AMMs
...

Codecov Report

Merging #1485 (0751d5a) into refactor-balancer-pool-fetching-3 (8378633) will decrease coverage by 0.08%.
 Consider uploading reports for the commit d4be564 to get more accurate results

@@                          Coverage Diff                          @@
##           refactor-balancer-pool-fetching-3    #1485      +/-   ##
=====================================================================
- Coverage                              66.05%   65.97%   -0.09%
=====================================================================
  Files                                    159      162       +3
  Lines                                  30567    30586      +19
=====================================================================
- Hits                                   20192    20179      -13
- Misses                                 10375    10407      +32


don't make /var/lib/docker/devicemapper/* MS_PRIVATE

I am unable to use the live-restore feature when I run the Docker daemon in its own mount namespace.
The reason is that /var/lib/docker/devicemapper/* is remounted as MS_PRIVATE as soon as the Docker daemon starts so anything mounted by the daemon under /var/lib/docker/devicemapper/* won't be propagated to the host.  When the Docker daemon restarts, obviously it is unable to find these mount points as it will be running in a different mount namespace.
Is there anything that can be done to not use MS_PRIVATE for /var/lib/docker/devicemapper/*?  What sort of issues are expected if we don't remount it to MS_PRIVATE?

I remember something about that being needed for bind mounts like /dev and such, @rhvgoyal could you take a look at this?

related to https://github.com/docker/docker/pull/22806 ?

I think there might be some unpleasant surprises if you remove it.
One thing is that its a safety net that docker mounts are not propagating into other mount namespaces. Otherwise removal of containers or cleaning of mount points will become even harder.
In the past people had raised some performance concerns also due to various mount propagation overhead. I don't have details though.
It also provides saftey net that nothing propagates into /var/lib/docker/devmapper/. Otherwise with -shared volumes feature it should be possible to pass in a directory in /var/lib/docker/ as source and then further mounts from containers will show up there. In the past I think @cpuguy83 did not want that kind of thing to happen.

@thaJeztah yes exactly!  I would still like though to be able to use a different namespace.  The current behaviour is to limit Docker to run in the host mount namespace.
@rhvgoyal the last issue seems to be related to the way the new mount /var/lib/docker/ is created?  If you don't want mount to get propagated there, it should not be set to MS_SHARED.

/var/lib/docker/ is not a mount point right? /var/lib/docker/devmapper/ is and its private by default to rule out the possibility of things propagating in or out by default.
If /var/lib/docker/devmapper is not a mount point, then default propagation properties will depend on properties of parent mount which will be system root (/). And in fedora and centos and other distributions it is shared by default.

Generally its more about mounts propagating to other namespaces making it difficult to clean up on exit.
Newer kernels handle this much better (3.16+?, I can't remember the version), where at least in Docker's namespace we can do a lazy unmount and clean up naturally, where as in older kernels the lazy unmount will still return EBUSY if it's still mounted in another namespace.

PR opened here; https://github.com/docker/docker/pull/31490

right, /var/lib/docker/devicemapper/is.  It is remounted to MS_PRIVATE in any case, even if it already existing, so there is no way to change this behaviour as it is now.
@cpuguy83 yes, that is fixed in linux 3.15+ which is quite old and it is probably going to be addressed in RHEL as well, so I bet the clean up on exit problem should not be a problem

lazy unmount does not fail if mount is busy in another namespace. But directory removal will fail if that directory is unmounted on in another mount namespace (with older kernels)

so beside not supporting old kernels (3.14 was released almost 3 years go) there should not be any blocker issue in dropping the MS_PRIVATE bits?  That looks like good news for running Docker in its own namespace and support live-restore.

I am not sure what problem you are trying to solve. So if problem is that you want to see mount points from previous run, you need to run docker daemon in host mount namespace. The very reason we try to run docker daemon in another namespace is that we dont want these mount points to become visible in other namespace.

BTW, why --live-restore needs to find old mount points. Why can't we save some metadata about the containers we leave running and try to find them upon reastart. If need be, create more mount points in your current mount namespace. That way we don't rely on finding mount points from previous run.

That is needed when you try to do "docker exec $CONTAINER ..." after you restart Docker.  Since the container was launched in the old mount namespace that is now not accessible, runc fails in this way:
[pid 17899] chdir("/run/docker/libcontainerd/23e8ed9bac60d52d9f5fecf694b15f3bbf6a0c84f18dc57bbc9005b61ba0914e") = 0
[pid 17904] chdir("/var/lib/docker/devicemapper/mnt/8e67a6553dbe35711a70d8d759f0a174445565b55b09d32eefc2ac8dc6c78019/rootfs") = -1 ENOENT (No such file or directory)

I can see that containerd:master is already managing exec directly from the shim process.  For v0.2.x, this quick hack patch fixes the issue for me.  Would something like this make sense?
diff --git a/runtime/container.go b/runtime/container.go
index 2e9e663..1c10f3b 100644
--- a/runtime/container.go
+++ b/runtime/container.go
@@ -445,10 +445,16 @@ func (c *container) Exec(ctx context.Context, pid string, pspec specs.ProcessSpe
                        c.RemoveProcess(pid)
                }
        }()
-       cmd := exec.Command(c.shim,
-               c.id, c.bundle, c.runtime,
+
+       ppid, err := readProcStatField(c.processes[InitProcessID].SystemPid(), 4)
+       if err != nil {
+               return nil, fmt.Errorf("could not check process ppid: %v", err)
+       }
+
+       cmd := exec.Command(
+               "nsenter", "-C", "-F", "-r/", fmt.Sprintf("--wd=%s", processRoot), "-m", "-u", "-i", "-n", "-p", "-t", ppid,
+               c.shim, c.id, c.bundle, c.runtime,
        )
-       cmd.Dir = processRoot
        cmd.SysProcAttr = &syscall.SysProcAttr{
                Setpgid: true,
        }

@crosbymichael


Fix use-after-free metadata corruption in C# when receiving response headers for streaming response calls

Fixes https://github.com/grpc/grpc/issues/25219 and https://github.com/grpc/grpc/issues/25004 (also see internal b/158826903)
I've been trying to debug this problem for long time, and I finally found the rootcause (with help of modifications that allowed me to reproduce more consistently and with more logging, see https://github.com/grpc/grpc/pull/23564).
The problem is that for calls with streaming response (server streaming and full duplex), gRPC C# registers for "recv_response_headers_on_client" event when the call starts and when the duration of the call is very short, sometimes the "recv_response_headers_on_client" can be delivered AFTER receiving all the responses and after delivering the final status of the call. If that happens, the call handle gets disposed (which destroys the underlying grpc_call native object), which can trigger unreferencing of the metadata that hasn't yet been delivered to the C# layer.
So in the "HandleReceivedResponseHeaders", when the initial metadata is being read from the native grpc_metadata_array object, it can be already unreffed by C core and due to the "use-after-free" it's basically changing under our hands as we read it. This was the reason why malformed metadata was being encountered in the InteropClientServerTest.CustomMetadata test.
This problem only affected calls with streaming response that 1. read the response headers  2. have a very short duration (so that the response headers that happens at the very beginning of the call can be delivered after all the responses and final status have been delivered - this won't happen for long lived calls), so hopefully the bug hasn't affected many users in practice.

@donnadionne  we should probably include this fix  in v1.41.x release once it gets merged.

Python bazel test failure is unrelated: https://source.cloud.google.com/results/invocations/9aba656b-1de8-491b-ba10-afd520cf8026/log


Enabling sasl after initial cluster is formed breaks Schema Registry

What happened?
Using the Helm chart, adding something like:
auth:
  sasl:
    enabled: true
    secretRef: "redpanda-superuser"
    users: []

And deploying the update will enable SASL and authorization for all listeners, but the Schema Registry and any internal Kafka client usage will not properly use authentication. This results in any client (Console, rpk, etc.) trying to use the Schema Registry to receive errors about it being unable to contact brokers:
Broker side:
WARN  2024-03-26 07:09:11,106 [shard 0:main] pandaproxy - service.cc:364 - mitigate_error: kafka::client::broker_error ({ node: -1 }, { error_code: broker_not_available [8] })

Client side:
{"error_code":50302,"message":"{ node: -1 }, { error_code: broker_not_available [8] }"}

What did you expect to happen?
Schema Registry to work 😆
How can we reproduce it (as minimally and precisely as possible)?. Please include values file.
Start with:

image:
  repository: docker.redpanda.com/redpandadata/redpanda
  tag: v23.3.9
  pullPolicy: Always

resources:
  cpu:
    cores: 1
    overprovisioned: true
  memory:
    container:
      min: 2Gi
      max: 2Gi

tls:
  enabled: true

statefulset:
  budget:
    maxUnavailable: 1
  initContainers:
    setDataDirOwnership:
      enabled: true

external:
  enabled: true
  service:
    enabled: false
  addresses:
  - redpanda-broker-0
  - redpanda-broker-1
  - redpanda-broker-2

Then update to:

image:
  repository: docker.redpanda.com/redpandadata/redpanda
  tag: v23.3.9
  pullPolicy: Always
resources:
cpu:
cores: 1
overprovisioned: true
memory:
container:
min: 2Gi
max: 2Gi
tls:
enabled: true
statefulset:
budget:
maxUnavailable: 1
initContainers:
setDataDirOwnership:
enabled: true
external:
enabled: true
service:
enabled: false
addresses:

redpanda-broker-0
redpanda-broker-1
redpanda-broker-2

auth:
sasl:
enabled: true
secretRef: "redpanda-superuser"
users: []

Anything else we need to know?
No response
Which are the affected charts?
Redpanda
Chart Version(s)
5.7.35
Cloud provider
n/a

And deploying the update will enable SASL and authorization for all listeners

Is this true? In the config I saw, enable_sasl and kafka_enable_authorization were both set to false.
The helm chart may have updated bootstrap.yaml and redpanda.yaml, but after a cluster is formed, cluster config from those two locations is ignored.

I think this issue is also related.

Good catch, this does seem like a broken workflow
Heads up @chrisseto .  These changes from the helm side to existing cluster configs must definitely set the cluster configs via RPK or Admin API and trigger a restart when necessary.  Node level configs could still go into redpanda.yaml and trigger a restart IIUC


Add platform hotspots metric back with better definition

          Removing this until the next iteration to have better definitions.

Originally posted by @adrielp in https://github.com/liatrio/openo11y.dev/pull/27#discussion_r1453864822

Closing as wontfix until it become prevalent. Can revisit in the future.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_f33ce138-04dc-4389-b744-73cda25fd1dd/pull/1 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target_f33ce138-04dc-4389-b744-73cda25fd1dd.git

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

File
Status
Preview URL
Details

:x:Error

Details

[Error-UserAccountBuildPermissionInsufficientError] Owner v-alji no longer has Read permission of the repository https://github.com/VSC-Service-Account/docs-archive-test-source. It might be due to recent GitHub changes on SSO. Please ask the owner v-alji to logon to Docs Portal https://ops.microsoft.com once again. If it is not the case, please ask anyone of Admin permission of the repo to assign a new owner via Docs Portal https://ops.microsoft.com

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

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

File
Status
Preview URL
Details

:x:Error

Details

[Error-UserAccountBuildPermissionInsufficientError] Owner v-alji no longer has Read permission of the repository https://github.com/VSC-Service-Account/docs-archive-test-source. It might be due to recent GitHub changes on SSO. Please ask the owner v-alji to logon to Docs Portal https://ops.microsoft.com once again. If it is not the case, please ask anyone of Admin permission of the repo to assign a new owner via Docs Portal https://ops.microsoft.com

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


LPA code refactoring + endpoint integration

Describe your changes

Integrating case officer review outcome POST with API endpoint
Refactoring to bring LPA code in line with new patterns and conventions introduced since work on this domain began (+ associated updates to templates

Issue ticket number and link
N/A
Type of change 🧩

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Other (please explain in the description section above)

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

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

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

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

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

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

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


[dev] transfer to TypeScript

Though it is easier to implement the audio playing service in Python, using Tkinter is too much trouble for the audio palyer, as I couldn't figure out why the GUI is sometimes not responding.

Issue Closed as problem is solved. The problem is on Python version, just change Python3.11 to Python3.9 or lower will be fine.
https://github.com/python/cpython/issues/110218


Interpreter for MaxOp and MinOp

fixes #185

I am confused by the error in  the build run. https://github.com/openxla/stablehlo/actions/runs/3132304609/jobs/5084509784
We do have a use of the function isSupportedIntegerType in Elements.cpp.

The only use of isSupportedIntegerType is in Element map(const Element &el, IntegerFn integerFn, FloatFn floatFn, ComplexFn complexFn) which appears to be unused. If the template is never expanded, it makes sense that the function is unused. All uses of map in that file take a LHS and RHS.

Also updated the spec draft to reflect out choice for max/min on complex types.


[SPARK-35159][SQL][DOCS] Extract hive format doc

What changes were proposed in this pull request?
Extract common doc about hive format for sql-ref-syntax-ddl-create-table-hiveformat.md and sql-ref-syntax-qry-select-transform.md to refer.
Why are the changes needed?
Improve doc
Does this PR introduce any user-facing change?
No
How was this patch tested?
Not need

Test build #137710 has started for PR 32264 at commit df246ea.

Test build #137711 has started for PR 32264 at commit d98f825.

I am not sure if we need to put this page in which menu page

Test build #137710 has finished for PR 32264 at commit df246ea.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
    SERDE serde_class [ WITH SERDEPROPERTIES (k1=v1, k2=v2, ... ) ]

Test build #137711 has finished for PR 32264 at commit d98f825.

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42239/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42239/

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

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

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42238/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42238/

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

not sure if we need to put this page in which menu page

We don't need to put it in the menu page.

Test build #137732 has started for PR 32264 at commit 203f544.

Test build #137732 has finished for PR 32264 at commit 203f544.

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

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42259/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42259/

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

@maropu do you want to take a look?

Test build #137797 has started for PR 32264 at commit 5fe64b5.

Test build #137797 has finished for PR 32264 at commit 5fe64b5.

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

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42326/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42326/

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

Could you add the screenshot of the new page in the PR description?

NOTE: I'm planning to backport this PR and https://github.com/apache/spark/pull/31010 into branch-3.1/3.0 because I think these document pages are useful for users.

Could you add the screenshot of the new page in the PR description?

DOne

Test build #137837 has started for PR 32264 at commit 9bfa1cf.

Test build #137837 has finished for PR 32264 at commit 9bfa1cf.

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

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42367/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42367/

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

thanks, merging to master!

thanks, merging to master!

@maropu shall we have a single backport PR or two?

They have different jira tickets, so I think its better to bckport them separately. Could you? @AngersZhuuuu

Anyway, late lgtm. Thank you, @AngersZhuuuu

They have different jira tickets, so I think its better to backport them separately. Could you? @AngersZhuuuu

ping

They have different jira tickets, so I think its better to backport them separately. Could you? @AngersZhuuuu

ping

Hmmm,  have conflict? need me to create backport PR?

yea, yes. I couldn't cherry-pick them into the previous branches.

yea, yes. I couldn't cherry-pick them into the previous branches.

Ok, ping you later when PR is ready


Add command teams set resolving #815

Add command teams set resolving #815

Coverage remained the same at 100.0% when pulling 32a0f8318d467df4b11166085d71bda7e5a333a7 on rabwill:issue-815 into e305fa793ff4ad96d6a66c5956073b76e6d9ab90 on pnp:dev.

Hey @waldekmastykarz / @VelinGeorgiev  this is ready for review , thanks :)

Awesome! Thank you @rabwill 👏

Hi @waldekmastykarz  I have made the necessary changes and pushed it as well, Could you please take a look? thank you.

Will do. Thank you! Have a great weekend!

Merged manually. Thank you! 👏


Disabled days

How can I disable specific days? For example the 1st, 23rd and 25th of a month?
Is there any way I can disable specific days using an date array?
Thank you

yes, check modifiers docs https://reactnicedates.hernansartorio.com/#api-reference
something like this will work
modifiers={{
  disabled: (date: Date) => [1, 23, 25].includes(date.getDate()),
}}


Only example that works is the carpole example.

It seems that the only example that works is the carpole example...could you please add some more explanations into the readme file and explain how each configuration works?

A separated example script file is created for usage along with the corresponding YAML files.


Add Gitlab CI/CD option

Done.

How do we use the GitLab CI/CD?
Also is there a way to replace all the GitHub links with GitLab links?

@austinmw Sorry, that is currently not something that is supported. This issue is from the very early stages of development, and I recall deciding against it because it became a bit complex to handle (and maintain) both Gitlab and GitHub pipelines. I don't think I ever implemented Gitlab. Sorry for the confusing comment above :) I am afraid that to use Gitlab CI you'd have to delete the .github directory and add a .gitlab-ci.yml yourself.


Move module Haskell.Ide.Engine.Plugin.Fuzzy

This is not a plugin, but supporting code.  I would like to see only plugins in the Haskell.Ide.Engine.Plugin directory.
I am not sure where it should go, perhaps it is time to make a Utils dir?
Suggestions welcome.

I would like to see the rest of Haskell.Ide.Engine.Plugin.HieExtras moved into a Utils folder or similar as well, since it's not a plugin either

I think it might also be worthwhile splitting up HieExtras into separate modules for completion/symbols/definitions as well, as some of these functions are getting quite beefy. Could they be moved to the LSP directory? I'm not sure if they're used by the JSON transport at all

This can be closed now, right?


Update io.js build targets for Travis CI config

This PR updates the io.js build targets for the Travis CI configuration: iojs-v1, iojs-v2, and iojs.
io.js follows semver which means that patch and minor versions are fully backwards compatible within one major version. I also added the latest iojs build target since io.js v3 is right around the corner as per nodejs/io.js#1807.

io.js is no longer considered in our travis config. I'm gonna close this PR.


Does cert-manager work with Kind: StepIssuer? Issue in getting certificate for ingress resources

Bugs should be filed for issues encountered whilst operating cert-manager.
You should first attempt to resolve your issues through the community support
channels, e.g. Slack, in order to rule out individual configuration errors.
Please provide as much detail as possible.

Describe the bug:
While requesting certificates for ingress resources I get an error in cert-manager stating necessary annotations are missing. I have annotations defined in the ingress definition
Logs
cert-manager/controller/ingress-shim "level"=0 "msg"="not syncing ingress resource as it does not contain a "certmanager.k8s.io/issuer" or "certmanager.k8s.io/cluster-issuer" annotation" "resource_kind"="Ingress" "resource_name"="kuard" "resource_namespace"="default"
Expected behaviour:
Annotations used:
   kubernetes.io/ingress.class:xxx
   certmanager.k8s.io/issuer-kind: StepIssuer
   certmanager.k8s.io/issuer-name: step-issuer

Steps to reproduce the bug:
Steps to reproduce the bug should be clear and easily reproducible to help people
gain an understanding of the problem.
Anything else we need to know?:

Certificate is generates using step-ca
Use step-issuer with cert-manager combination
https://github.com/smallstep/step-issuer

Environment details::

Kubernetes version (e.g. v1.10.2): v1.14.5
Cloud-provider/provisioner (e.g. GKE, kops AWS, etc): AKS
cert-manager version (e.g. v0.4.0): v0.10.0
Install method (e.g. helm or static manifests): helm

/kind bug

Thanks for opening this!
We need to add an extra annotation to set the group name field of the issuer. We also need to change a couple bits in the code to only support using the core types.
/assign

@JoshVanL Thanks a lot for looking into this! Looking forward to the fix.

@JoshVanL I guess this feature is currently experimental in cert-manager and needs to be enabled using the flag --feature-gates=CertificateRequestControllers=true for Step-issuer to work
Cert-manager v0.11.0 will probably include this feature enabled by default

Yes that's correct - CertificateRequests are becoming beta as of 0.11 and will be the only implementation of issuing certificates.
The ingress fix is still WIP. Hoping to get it into a review-able state today :crossed_fingers:

(also landing in 0.11)

@JoshVanL So am I good to use StepIssuer as a ClusterIssuer? Thanks for all the help on this

So you will need to install the StepIssuer deployment which you can get from here. You should be able to add the issuer name, kind, and group annotation to your ingress resources:
"cert-manager.io/issuer-name": "foo"
"cert-manager.io/issuer-kind": "StepIssuer"
"cert-manager.io/issuer-group": "certmanager.step.sm"

If I'm am reading correctly it looks like these issuers are namespaced so not cluster wide. Perhaps @maraino can provide some insight. Hope that helps in any case :)

@JoshVanL This requested fix went in for 0.10 version right and not in 0.11

does cert-manager have any preference for the scope of the external issuers?

Having both probably makes most sense. Indeed v0.11 comes with a lot of breaking API changes so things will need updating - external issuers will be supported by default though. Let us know if you need any help making those changes or implementing a cluster scoped issuer.

This requested fix went in for 0.10 version right and not in 0.11

The requested ingress changed is in 0.11. We should have a beta coming out tomorrow

@JoshVanL @maraino Deployed beta version of cert-manager today and getting this error
Error: invalid argument "CertificateRequestControllers=true" for "--feature-gates" flag: unrecognized feature gate: CertificateRequestControllers

Hi @devops2201
That flag no longer exists since it's the default and only implementation now :smile:
You're good to just remove it now


fixed test error

IBM Cloud toolchain: Delivery Pipeline deployed products to dev, including commit 741da67ebf9282369fa78a1b42e6fc4e922d8bf0

IBM Cloud toolchain: Delivery Pipeline deployed products to prod, including commit 741da67ebf9282369fa78a1b42e6fc4e922d8bf0


String.toComponent and Component.toString() methods?

This is what I use for minimessage + string to components:
    fun String.toComponent(): Component = MiniMessage.miniMessage().deserialize(this)

    fun Component.toString(): String = MiniMessage.miniMessage().serialize(this)

overriding toString doesnt seem like a good idea


Revamping CMakeList

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug, a feature request, or a significant problem with documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce:

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
Need a revamp of contrib/cmake/CMakeLists.txt to account for ComputeCPP
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.
No logs are needed as this is a feature request.
I am trying to rebuild the dev/amd_gpu version of TF and for that I would need to revamp the entire CMakeLists file(s). My first target would be to build a C++ example and then I will attempt the python version (which itself looks pretty messy).
In this regard, are there any special #define flags that need to be defined for ComputeCpp?

Hi @skn123,
thanks for your interest in contributing. As it happens, there's more than a couple of defines required for building TensorFlow with ComputeCpp! Forgive me if you've seen this, but because ComputeCpp interacts with OpenCL devices, a separate compiler is required to output instructions they can execute.  As it happens we maintain a FindComputeCpp.cmake module which does basically everything. It's recently been revamped to be much better-behaved and more in-line with modern CMake, this should be the last interface change for a while.
If you'd like to attempt a CMake build of TensorFlow + ComputeCpp, I would advise starting here. You can find some example usage of the module in the linked respository - the important function is "add_sycl_to_target", which will compile all the sources in a target with compute++ (our device compiler) and make sure that code is in the objects produced. I can understand that this might not be desireable - in which case you can use the function "__build_ir" directly, which might be more useful in this case. It operates on single source files, so you'd have a little more control.
If you have any other questions, please ask me, I've spent a bunch of time working on the module recently (however, I have no idea how what TensorFlow's CMake looks like :p).
In terms of special defines... I think the bazel configuration for SYCL should list them all. I can't remember offhand but there are a couple.


Location Expired

Hello Marius,
first of all thank you for your great project. I have a small issue (well not so small actually). Everything seems to work perfectly but when I open the share link no map appears. The message on the page is "Location expired. The shared location you tried to access was not found on the server. If this link worked before, the share might have expired." On the app everything seems to work perfectly. Maybe some directory permission problem?
Thank you in advance for your help.
Cheers
Max
ps: memcached info attached

Hi! Thanks for the feedback :-)
This typically happens when the memcached daemon itself isn't running - the PHP extension and memcached itself are two separate things that interact with each other, and both must be installed and running. Can you check if it is actually running? You can check using any of these:
user@host ~$ ps aux | grep memcached
memcach+     772  0.0  0.0 409200  5752 ?        Ssl  14:12   0:03 /usr/bin/memcached -m 64 -c 1024 -l <IP_ADDRESS>,::1 -o modern,drop_privileges

user@host ~$ sudo netstat -tulpnx | grep memcached
tcp        0      0 <IP_ADDRESS>:11211         <IP_ADDRESS>:*               LISTEN      772/memcached
tcp6       0      0 ::1:11211               :::*                    LISTEN      772/memcached

Thank you for your super-fast reply!
Here the result:
tcp        0      0 <IP_ADDRESS>:11211         <IP_ADDRESS>:*               LISTEN      22498/memcached

Thanks for the quick response :-)
Can you try editing your Hauk config file and setting memcached_host to <IP_ADDRESS> instead of localhost and see if that helps?

Nope :-( I got the same message. Maybe I should change something on ufw firewall? Or apache2 server?

Forwarding ports should not be necessary. I assume you're running apache2 and memcached on the same server?
You could try stopping memcached, then starting it manually with memcached -m 64 -c 1024 -l <IP_ADDRESS>,::1 -vv. That should give you detailed connection logs as output and show what's going on behind the scenes. I don't know exactly how helpful it will be, but if something happens, it'll at least show up as a connection.

Yes I did and then I tried to restart the app a couple of times and now it works!!!!
Actually I was starting to feel guilty, hope I didn't ruin your night ;-)
All the best

Ah, don't worry, at least it works now! I'm just glad to be here to try and help :-)
Hope you'll enjoy the app

FYI, one gets the same error message if the external .js scripts from that  CDN are blocked (eg. Via uMatrix)

FYI, one gets the same error message if the external .js scripts from that CDN are blocked (eg. Via uMatrix)

Interesting, didn't realize that. Should be fixed now with the resolution to #67.


Make conv2DBackpropInput public

Conv*D layers have corresponding public functions conv*D.
With them we can easily create customized convolutional layers.
On the other hand, TansposedConv*D is implemented with internal function conv2DBackpropInput.
It makes creating customized transposed convolutional layers hard.
I propose to make it public, or add new transposedConv2D function as alias.

Sorry for the late reply. I think renaming conv2DBackpropInput and making it a public API is reasonable!

func matmul has transposed: Bool arguments for transposing lhs or rhs:
/// Performs matrix multiplication with another tensor and produces the result.
@inlinable
@differentiable(where Scalar: TensorFlowFloatingPoint)
public func matmul<Scalar: Numeric>(
    _ lhs: Tensor<Scalar>,
    transposed transposeLhs: Bool = false,
    _ rhs: Tensor<Scalar>,
    transposed transposeRhs: Bool = false
) -> Tensor<Scalar> {
    precondition(lhs.rank >= 2, "Input tensors must have at least rank 2")
    precondition(rhs.rank >= 2, "Input tensors must have at least rank 2")
    if lhs.rank > 2 || rhs.rank > 2 {
        // TODO(TF-629): Conjugate to make compatible with the adjoint.
        return _Raw.batchMatMulV2(lhs, rhs, adjX: transposeLhs, adjY: transposeRhs)
    }
    return _Raw.matMul(lhs, rhs, transposeA: transposeLhs, transposeB: transposeRhs)
}

Adding a transposed: Bool = false argument (or arguments, for both input and filter?) to the existing function public func conv2D sounds ideal to me, if possible. @t-ae: would you be interested in looking into this?
Increasing the API surface area with public func transposedConv2D is less desirable.

I think public func transposedConv2D is worth adding.

Both TensorFlow and PyTorch have separate function for transposed conv2D.

TensorFlow:

https://www.tensorflow.org/api_docs/python/tf/nn/conv2d?version=stable
https://www.tensorflow.org/api_docs/python/tf/nn/conv2d_transpose?version=stable

PyTorch

https://pytorch.org/docs/stable/nn.functional.html#conv2d
https://pytorch.org/docs/stable/nn.functional.html#conv-transpose2d

Users may not expect conv2D has transposed option.

There's already TransposedConv2D layer.

If we add the argument transposed for conv2D, we should add same argument for Conv2D and deprecate TransposedConv2D to make interface consistent.

conv2DBackpropInput was renamed to transposedConv2D and made public in https://github.com/tensorflow/swift-apis/pull/672.
The transposedConv2D API is consistent with TensorFlow/PyTorch.


No way to enable FILESTREAM in a varbinary(max) column when using sql server

Problem
I wasn't able to find the sql server native attribute FILESTREAM when creating a varbinary(max) column. this is really necessary to me.
Suggested solution
Maybe add a native database type attribute that is db.VarBinaryWithFilestream that is equivalent to creating a column  as [Column_Name] varbinary FILESTREAM

Is this about https://learn.microsoft.com/en-us/sql/relational-databases/blob/filestream-sql-server?view=sql-server-ver16?
You can always modify the resulting SQL migration before applying it to the database. Does this cause any problems when later running migrate dev or other Prisma CLI commands?


Template for check status battery notebook

Hi, there is a Template for check status battery notebook? Regards

Hello,
You can try searching the directory listing located in readme.md to find all supported templates by models.

I have not found any template to check the status of the notebook battery, is there a guide to create templates? Regards

There are several ways to query the battery status. For instance using acpi, upower or just cat /sys/class/power_supply/BAT0/capacity.
Some of the tools provide additional information, but start with only capturing one thing - the battery level - and get that working. You can always extend the template later.

The battery query operation needs to run on the agent, and there are two ways to extend the agent configuration to run a custom battery level query command, either via user parameter, e.g. create a file /etc/zabbix/zabbix_agentd.d/user-parameter.conf and add
UserParameter=notebook.battery.level,cat /sys/class/power_supply/BAT0/capacity

(I am completely blank on naming policies regarding UserParameter keys, but I guess avoiding collisions with keys already defined in upstream Zabbix  is the safest choice. Alternatively system.hw.bat.level would be in line with the existing keys (although you then run a risk of collision if Zabbix should add that key in the future).)

The other alternative is to use system.run to execute the command. Then you need to add the following to zabbix_agentd.conf:
...
AllowKey=system.run[cat /sys/class/power_supply/BAT0/capacity]
...

DenyKey=system.run[*]
DenyKey=system.run

Of these two I would strongly suggest UserParameter since it hides the implementation detail from the Zabbix template and easily allows for different agents to run different commands to query the battery level.
As for guide on creating a template I do not know, but for this case I would make a copy of Applications/template_zabbix-apt since that is one of the smallest templates in this repository with just one item, i.e. ideal as a starting point and I think you just need to replace the text and check.
Notice that for the yaml file you need to change most of the uuids. I wrote a script to make these changes automatically, see this pull request.
I would be interested in using this battery, so I'll be happy to help testing.

I think this is the way but, I need make for windows

You can achieve this by using Windows agent and Powershell commands.
In your case: (Get-WmiObject win32_battery).estimatedChargeRemaining

You can achieve this by using Windows agent and Powershell commands. In your case: (Get-WmiObject win32_battery).estimatedChargeRemaining

Thanks  but how can agent take data and send to server? Regards

You sould save srcipt as script_name.ps1 somewhere on hard drive.
For example: c:\script\script_name.ps1
Then you should edit Zabbix agent config file, for example:
`LogFile=c:\zabbix_agentd.log
LogFileSize=10
DebugLevel=2
Server=<IP_ADDRESS>.
ListenPort=10050
Hostname=W7X86
EnableRemoteCommands=1
LogRemoteCommands=1
Timeout=30
UserParameter=somename[*],PowerShell.exe -nologo "C:\script\script_name.ps1" "somename"`

powercfg/batteryreport this command generate a complete report but output in html, can

You sould save srcipt as script_name.ps1 somewhere on hard drive.
For example: c:\script\script_name.ps1
Then you should edit Zabbix agent config file, for example:
LogFile=c:\zabbix_agentd.log
LogFileSize=10
DebugLevel=2
Server=<IP_ADDRESS>
ListenPort=10050
Hostname=W7X86
EnableRemoteCommands=1
LogRemoteCommands=1
Timeout=30
UserParameter=somename[*],PowerShell.exe -nologo "C:\script\script_name.ps1" "somename"

powercfg/batteryreport this command generate a complete report but output in html and need be administrator, how can send by agent?

powercfg /batteryreport /xml /output batteryreport.xml
$battery = [xml](Get-Content batteryreport.xml)
$health = [pscustomobject]@{
time = $battery.BatteryReport.ReportInformation.ScanTime;
maxpower = $battery.BatteryReport.Batteries.Battery.FullChargeCapacity;
design = $battery.BatteryReport.Batteries.Battery.DesignCapacity
}
$health | Format-Table
Write-Host "Max battery capacity is only $($health.maxpower / $health.design * 100)% of designed capacity"
I found this script very useful, now how can I get this data on the server? using a template

You sould save srcipt as script_name.ps1 somewhere on hard drive.
For example: c:\script\script_name.ps1
Then you should edit Zabbix agent config file, for example:
LogFile=c:\zabbix_agentd.log
LogFileSize=10
DebugLevel=2
Server=<IP_ADDRESS>
ListenPort=10050
Hostname=W7X86
EnableRemoteCommands=1
LogRemoteCommands=1
Timeout=30
UserParameter=somename[*],PowerShell.exe -nologo "C:\script\script_name.ps1" "somename"

powercfg /batteryreport /xml /output batteryreport.xml
$battery = [xml](Get-Content batteryreport.xml)
$health = [pscustomobject]@{
time = $battery.BatteryReport.ReportInformation.ScanTime;
maxpower = $battery.BatteryReport.Batteries.Battery.FullChargeCapacity;
design = $battery.BatteryReport.Batteries.Battery.DesignCapacity
}
$health | Format-Table
Write-Host "Max battery capacity is only $($health.maxpower / $health.design * 100)% of designed capacity"
I found this script very useful, now how can I get this data on the server? using a template. Regards


Init typeregistry

Closes issue #100.

Improved the absoluteUrl() function
Do not reload type indexes after patching
Refactor existing type registry code
Implement initTypeRegistry*() methods
When registering types, if registry resources don't exist, they're created.

Is this ready to review/merge?

Not yet

(Rebased this PR/branch on latest master).

Ok, registry handling code refactored, unit and integration tests added.
Ready for review & merge.


Is there an example of hook rust stdlib?

Such as hook Sleep.

It works, see https://github.com/loongs-zhang/open-coroutine-20240723/commit/caad9d9dae4201fd597ea3dfaabe76b87a52ca5f


Headphones update

Nice! Cant wait to see the finished textures! :)

this tidies up and removes all the colors that do not have a dye

Is there anything you would want me to do? I will probably create a concept art folder if that's okay with you.


--quiet silences "Using xyz" in brew bundle install

This is intended to resolve #1202. Only addresses the brew bundle install subcommand because I don't see any other subcommands where --quiet makes sense. I don't think it's desirable to suppress other non-error output (e.g. "Homebrew Bundle complete!..." and "Downloading https://formulae.brew.sh/api/cask.jws.json") unless we added a -qq option, which would require more far reaching changes to how arg parsing works (--quiet is parsed by code in homebrew/brew, not this repo).
Fixes #1202

Thanks so much for your first contribution!  You rock, @rrotter!


Using swashbuckle with generic base controller

I've a base controller that uses generic to implement some base operations.
The problem is that Swashbuckle enlist every model in whole system as model dependency of any controller, and that results to 1) incorrect results  2) freeze swagger ui due to huge dependency tree.
here is my base controller.
public abstract class BaseController<TModel, TViewModel> : ApiController
        where TModel : EntityBaseClass
        where TViewModel : EntityBaseClass
    {
        protected IBaseService<TModel> Service { get; set; }

        protected override void Dispose(bool disposing)
        {
            Service.Dispose();
            base.Dispose(disposing);
        }

        public DataSourceResult Get([System.Web.Http.ModelBinding.ModelBinder(typeof(WebApiDataSourceRequestModelBinder))]DataSourceRequest request)
        {
            return Service.GetAll().ToDataSourceResult(request).ToViewModel<TViewModel>();
        }

        public HttpResponseMessage Post(TModel model)
        {
            var result = Service.Create(model);
            if (result.Status == HttpStatusCode.InternalServerError)
            {
                return Request.CreateResponse(HttpStatusCode.InternalServerError, new DataSourceResult {Data = new[] {result.Extra}});
            }

            var response = Request.CreateResponse(HttpStatusCode.Created, new DataSourceResult { Data = new[] { model }, Total = 1 });

            return response;
        }
]
        public HttpResponseMessage Put(int id, TModel model)
        {
            if (id == model.ID)
            {
                try
                {
                    var result = Service.Edit(model.ToViewModel<TModel>());
                    if (result.Status == HttpStatusCode.InternalServerError)
                    {
                        return Request.CreateResponse(HttpStatusCode.InternalServerError, new DataSourceResult { Data = new[] { result.Extra } });
                    }
                }
                catch (DbUpdateConcurrencyException)
                {
                    return Request.CreateResponse(HttpStatusCode.NotFound);
                }

                return Request.CreateResponse(HttpStatusCode.OK);
            }
            else
            {
                var errors = ModelState.Values.SelectMany(v => v.Errors).Select(error => error.ErrorMessage);
                return Request.CreateResponse(HttpStatusCode.BadRequest, errors);
            }
        }

        public HttpResponseMessage Delete(int id)
        {
            try
            {
                var result =Service.Delete(id);
                if (result.Status == HttpStatusCode.InternalServerError)
                {
                    return Request.CreateResponse(HttpStatusCode.InternalServerError, new DataSourceResult { Data = new[] { result.Extra } });
                }
            }
            catch (DbUpdateConcurrencyException)
            {
                return Request.CreateResponse(HttpStatusCode.NotFound);
            }

            return Request.CreateResponse(HttpStatusCode.OK);
        }

    }

so, is it possible to make swashbuckle to work in this situation?
here is a swagger-sepc produced by swashbuckle  for one of my controllers: http://pastebin.com/CNDdbyu9

I don't believe this has anything to do with the use of a generic controller. I think you would observe the same behavior if you hand-coded each concrete controller without the use of a generic base class. My guess is that the specific model classes (i.e. the different classes you're passing for TModel) have an extremely complex object graph.
By design, Swashbuckle will recursively traverse ALL "serializeable" properties on a model class. This is because Swashbuckle describes the "actual" API contract you're exposing and not the "intended" API contract. In other words, data for all the models and fields described in the outputted Swagger "can" be passed through the JSON body and WebAPI "will" deserialize them. You may not use all those values in your implementation and therefore don't "intend" them to be passed. But none-the-less, if they're serializeable, they "can" be passed and this is what Swashbuckle documents. This is the underlying philosophy - code is truth, not specification.
So ideally, if you're using Swashbuckle you should use model classes that have been tailored so that they're public properties represent the desired API contract exactly.

sadly I'm not using Swashbuckle in my current projects mainly because I'm not developing in C# anymore.
But I appreciate your time and effort to solve this issue.
thanks


Release 0.4?

0.4 has sat on master since we stalled the release https://github.com/JuliaRegistries/General/pull/5939
There have been a number of beneficial changes since, so it would be good to put out a release.
Also, with #61 upcoming, it would be good to get a stable release before that merges
cc. @andrewadare @mgkuhn @samuelpowell

@ianshmean can you remind me why we stopped that registration?
@andrewadare any objections to releasing 0.4, I'm happy to do it.

I do not have any objections to releasing version 0.4. I also think we should get #61 merged and make a release from that. That PR has sat without approval for a long time, but it is ready to be merged.

The issue was this: https://github.com/JuliaIO/LibSerialPort.jl/pull/45#issuecomment-559243165
So we're ok with releasing 0.4 on the current master? And let's get #61 pushed out quickly after

#61 still needs to fix the default flow-control settings, such that the binary-transparency tests also pass on e.g. Ubuntu Linux (where XON/XOFF is active by default). But feel free to merge #61 into master and I'll then quickly follow up with PRs for that, and perhaps also for restoring the timeout functionality.

Okay, I'll do a release of 0.4, merge #61 and open an issue linking the comment above pending PR and then potentially 0.5

Just beat you to it @samuelpowell. I registered current master as 0.4 https://github.com/JuliaIO/LibSerialPort.jl/commit/801de822476adf44ecb9977f23d3f8bb66e620fe#commitcomment-41470735

Cross post - I also asked for registration, both requests link to the same PR on registry so it appears the outcome is okay. I will open issue about #61 now.

0.4 released, hopefully tagbot will tag shortly, will merge #61 to master


ECDS stats reuse stale listener stat prefix

Per https://github.com/envoyproxy/envoy/pull/19630, ECDS stat prefix is set from the first listener that creates a subscription. If that listener is deleted and the subscription is still active because other listeners use it, ECDS stat prefix continue referring to the deleted first listener.

no stalebot


Add an authorization technique to the docs using env vars

Google Auth supports environment variables as a means of authorizing service apps. Add this technique to the docs for the benefit of others. Issue #370

:+1: even this PR doc saved me SO much time.


Error in java start command and 500 error while retrieving payment intents

I want to have a working sample of a java server with react stripe.
I did:
stripe samples create react-elements-card-payment
# choose java server
mvn pacakge
SPARK_LOCAL_IP=<IP_ADDRESS> java -cp target/react-1.0.0-SNAPSHOT-jar-with-dependencies.jar com.stripe.sample.Server

Afterward, I start the client with
cd client
yarn
yarn start

When I reach the page on http://localhost:3000, I have the following error 500:
om.google.gson.JsonSyntaxException: java.lang.IllegalStateException: Expected a string but was BEGIN_ARRAY at line 1 column 26 path $.payment_method_types
	at com.google.gson.internal.bind.ReflectiveTypeAdapterFactory$Adapter.read(ReflectiveTypeAdapterFactory.java:200)
	at com.google.gson.Gson.fromJson(Gson.java:810)
	at com.google.gson.Gson.fromJson(Gson.java:775)
	at com.google.gson.Gson.fromJson(Gson.java:724)
	at com.google.gson.Gson.fromJson(Gson.java:696)
	at com.stripe.sample.Server.lambda$main$3(Server.java:88)

The client is sending a post request with an array of payment method, while the server expect a string,
I changed https://github.com/stripe-samples/react-elements-card-payment/blob/master/server/java/src/main/java/com/stripe/sample/Server.java#L43
With a List<String>, after that, the error is gone and I have a new error two lines after:
[qtp998684067-17] ERROR spark.http.matching.GeneralError -
com.stripe.exception.InvalidRequestException: No such PaymentMethod: card; code: resource_missing; request-id: req_Sj58q2xnBltBcq
	at com.stripe.net.LiveStripeResponseGetter.handleApiError(LiveStripeResponseGetter.java:742)
	at com.stripe.net.LiveStripeResponseGetter.staticRequest(LiveStripeResponseGetter.java:533)
	at com.stripe.net.LiveStripeResponseGetter.request(LiveStripeResponseGetter.java:81)

There's obviously an error, is it the client request or the java server?
How can I get the java example and the react work together?

Hi @kopax
Sorry about that. I've pushed a fix. We don't need to pass payment_methods_types as the default value is card needed for this example.
Please pull master, and you should hopefully be unblocked.
/k

@auchenberg-stripe you don't need to but you should have accepted my fix.
For the simple reason that the node version is merging those arguments and you just made those servers differents.


Changing the default storage manager to Shared Prefs instead Hive

The default storage manager is Shared Preferences.
With a hive implementation in example.
Related to #13

Hi @GabrielRozendo, can you please rebase?

Done @cristipufu