Added loadingInfo field to AsyncLoading

It's an optional object info, typically a String that describes the current loading state.
Could be useful when warming up capsules to show a text describing the loading state along side with a loading spinner.

I didn't add an extra generic type to the AsyncLoading class because it was giving some errors in other methods using switch pattern matching

Could you please share a use case for this? AsyncValues are typically just returned from the builtin side effects and none of those would support the extra loading info, so I’m not sure it’d make sense to add here.
if you need the loading info in your own app, maybe you can make an AsyncValueInfo that has an info or related object to store info alongside an AsyncValue?

I can't show you know a screen video of that, because I'm currently migrating my project from Riverpode to ReArch, but basically when my app starts it does the warmup. Now it has three steps ("Obtaining configuration directory", "Checking configuration directory", "Loading servers configurations"). I show a spinner and a text with the step info. So the way I would be able to show that with toWarmUpWidget would be a solution like the one I propose in this PR.
I know I can just show "Loading", but this way it fits better.
In a scenario where you know exactly T on AsyncValue<T> maybe is not strictly neccesary, but for toWarmUpWidget I think yes.
Also I think is a property that mustn't lack on AsyncLoading as error and stack trace on AsyncError

There is a doc error in my changes: [loadingBuilder] is returned when any of the current states are [AsyncLoading]. must be [loadingBuilder] is called to build the returned [Widget] when any of the current states are [AsyncLoading].

if you need the loading info in your own app, maybe you can make an AsyncValueInfo that has an info or related object to store info alongside an AsyncValue?

Maybe you're right.

Also I thought it could be helpful to have on loadingBuilder and in errorBuilder the full list of AsyncValues objects that match those states (loading and error).

Maybe we can use AsyncValueInfo? instead of Object?.

AsyncValues are typically just returned from the builtin side effects and none of those would support the extra loading info, so I’m not sure it’d make sense to add here.

I began to use AsyncValue manually (a lot).

I understand If you don't like these changes. I always can use manually my own AsyncValue implementation.

For your use case of warm up widgets/to show loading information, I have a better approach than modifying AsyncValue. I'll follow up in #51


ISSUE-415: Client side schema validation - Disable or Customize

UI changes for #415

@shahsank3t @satishd Kindly  review the changes.


Improve securebanking-openbanking-demo with a local environment profile (IG development mode)

Story
As a developer/customer I want to deploy the IG component in DEVELOPMENT mode so that I can access the ui and IG Studio to help build and deploy the IG configuration in the local environments.
Acceptance criteria

IG deployed in DEVELOPMENT mode.
Can access the IG UI and IG Studio.
The existing routes configured can see in the UI in deployed status.

Tasks

[x] Create a IG local configuration for obdemo-bank
[x] Prepare the IG local configuration to deploy it in DEVELOPMENT mode (UI and Studio)
[x] Create a docker stage for IG local
[x] Create a profile to deploy local environments with IG local
[x] Upgrade config.sh script to init IG local
[x] Create the kustomize configuration for IG local
[x] Upgrade skaffold descriptor with a new profile to deploy obdemo-bank with IG local for local environments
[ ] Update the readme files to introduce the new changes

Release Notes
Affected App: securebanking-openbanking-demo
Description: X


Enable per column validation

References issue:
https://github.com/wyuenho/backgrid/issues/101

can we set focus to current field if validation failed


Testing Select with Jest gives dom.getContentEl().scrollTo is not a function.

🐛 Bug report
When trying to run a test in Jest for the Select machine / component as it is in the docs, gives an error -
dom.getContentEl(...)?.scrollTo is not a function

💥 Steps to reproduce

Copy the code from the Select docs
Create a basic Jest test that tries to open the content popover
See error

💻 Link to reproduction
https://stackblitz.com/edit/vitejs-vite-kkkgte?file=src%2FApp.spec.js
or if the link doesn't work -
import * as select from '@zag-js/select';
import { normalizeProps, Portal, useMachine } from '@zag-js/react';
import { useId } from 'react';

const selectData = [
  { label: 'Nigeria', value: 'NG' },
  { label: 'Japan', value: 'JP' },
  { label: 'Korea', value: 'KO' },
  { label: 'Kenya', value: 'KE' },
  { label: 'United Kingdom', value: 'UK' },
  { label: 'Ghana', value: 'GH' },
  { label: 'Uganda', value: 'UG' },
];

export function Temp() {
  const collection = select.collection({
    items: selectData,
    itemToString: (item) => item.label,
    itemToValue: (item) => item.value,
  });

  const [state, send] = useMachine(
    select.machine({
      id: useId(),
      collection,
    })
  );

  const api = select.connect(state, send, normalizeProps);

  return (

        Label
        <button {...api.getTriggerProps()}>{api.valueAsString || 'Select option'}</button>

      <Portal>

            {selectData.map((item) => (

                {item.label}
                ✓

            ))}

      </Portal>

  );
}

describe('Temp tests', () => {
  it('should render', async () => {
    render(<Temp />);

    const root = screen.getByRole('combobox');

    await userEvent.click(root);

  });
});

🧐 Expected behavior
It shouldn't error
🧭 Possible Solution
🌍 System information

Software
Version(s)

Zag Version
0.61.1

Browser
Jest test

Operating System
Mac os

📝 Additional information
Stacktrace -
dom.getContentEl(...)?.scrollTo is not a function
TypeError: dom.getContentEl(...)?.scrollTo is not a function
at scrollContentToTop (node_modules/@zag-js/select/src/select.machine.ts:622:35)
at Machine.executeActions (node_modules/@zag-js/core/src/machine.ts:560:9)
at Machine.performExitEffects (node_modules/@zag-js/core/src/machine.ts:682:5)
at Machine.stop (node_modules/@zag-js/core/src/machine.ts:219:10)
at node_modules/@zag-js/react/dist/index.js:244:15
at safelyCallDestroy (node_modules/react-dom/cjs/react-dom.development.js:3918:844)
at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:4058:1065)
at commitDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:4045:544)
at recursivelyTraverseMutationEffects (node_modules/react-dom/cjs/react-dom.development.js:4077:129)
at commitMutationEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:4099:213)
at commitMutationEffects (node_modules/react-dom/cjs/react-dom.development.js:4075:147)
at commitRootImpl (node_modules/react-dom/cjs/react-dom.development.js:4583:21)
at commitRoot (node_modules/react-dom/cjs/react-dom.development.js:4545:209)
at performSyncWorkOnRoot (node_modules/react-dom/cjs/react-dom.development.js:4440:97)
at flushSyncCallbacks (node_modules/react-dom/cjs/react-dom.development.js:2155:108)
at flushSync (node_modules/react-dom/cjs/react-dom.development.js:4453:66)
at ReactDOMRoot.Object.<anonymous>.ReactDOMHydrationRoot.unmount.ReactDOMRoot.unmount [as unmount] (node_modules/react-dom/cjs/react-dom.development.js:4913:1902)
  at Object.unmount (node_modules/@testing-library/react/dist/pure.js:145:12)
  at node_modules/@testing-library/react/dist/pure.js:271:12
  at node_modules/@testing-library/react/dist/act-compat.js:88:24
  at act (node_modules/react/cjs/react.development.js:2194:20)
  at node_modules/@testing-library/react/dist/act-compat.js:87:25
  at node_modules/@testing-library/react/dist/pure.js:270:28
  at Array.forEach (<anonymous>)
    at cleanup (node_modules/@testing-library/react/dist/pure.js:266:22)
    at Object.<anonymous> (node_modules/@testing-library/react/dist/index.js:31:25)
      at Promise.then.completed (node_modules/jest-circus/build/utils.js:289:28)
      at new Promise (<anonymous>)
        at callAsyncCircusFn (node_modules/jest-circus/build/utils.js:222:10)
        at _callCircusHook (node_modules/jest-circus/build/run.js:213:40)
        at async _runTest (node_modules/jest-circus/build/run.js:186:5)
        at async _runTestsForDescribeBlock (node_modules/jest-circus/build/run.js:86:9)
        at async _runTestsForDescribeBlock (node_modules/jest-circus/build/run.js:81:9)
        at async run (node_modules/jest-circus/build/run.js:26:3)
        at async runAndTransformResultsToJestFormat (node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapterInit.js:120:21)
        at async jestAdapter (node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
        at async runTestInternal (node_modules/jest-runner/build/runTest.js:367:16)
        at async runTest (node_modules/jest-runner/build/runTest.js:444:34)

Please get ideas from Ark setup test file. We'll consider add it to the docs
https://github.com/chakra-ui/ark/blob/main/packages/react/src/setup-test.ts

thanks, I tried many different variations of whats in Ark, but I struggled to get it working, not sure if it is the difference between Jest and Vitest, or just something within my setup that im missing, nonetheless, I finally got it with -
Object.defineProperty(global.window.Element.prototype, 'scrollTo', { value: () => {} });

(didnt have to use jsdom in the end, not sure why, im not overly familiar with mocking browser behaviour like this)


#12224: Add optional output tensor support for lt_bw

Issue : #12224
Work Done : Overloaded Unary and Binary LT with optional output tensor and Q_ID support
CI : Passed

https://github.com/tenstorrent/tt-metal/actions/runs/10915611277 - passed


TypeError: null is not an object (evaluating 'Orientation.initialOrientation')

TypeError: TypeError: TypeError: TypeError: null is not an object (evaluating 'Orientation.initialOrientation')
This error is located at:
in ScarletScreen (at navigationStore.js:448)
in Wrapped (at SceneView.js:9)
in SceneView (at StackViewLayout.js:478)
in RCTView (at View.js:45)
in View (at StackViewLayout.js:477)
in RCTView (at View.js:45)
in View (at StackViewLayout.js:476)
in RCTView (at View.js:45)
in View (at createAnimatedComponent.js:151)
in AnimatedComponent (at screens.native.js:59)
in Screen (at StackViewCard.js:42)
in Card (at createPointerEventsContainer.js:26)
in Container (at StackViewLayout.js:507)
in RCTView (at View.js:45)
in View (at screens.native.js:83)
in ScreenContainer (at StackViewLayout.js:401)
in RCTView (at View.js:45)
in View (at StackViewLayout.js:400)
in StackViewLayout (at withOrientation.js:30)
in withOrientation (at StackView.js:49)
in RCTView (at View.js:45)
in View (at Transitioner.js:141)
in Transitioner (at StackView.js:19)
in StackView (at createNavigator.js:57)
in Navigator (at createKeyboardAwareNavigator.js:11)
in KeyboardAwareNavigator (at createNavigationContainer.js:376)
in NavigationContainer (at SceneView.js:9)
in SceneView (at createTabNavigator.js:39)
in RCTView (at View.js:45)
in View (at ResourceSavingScene.js:35)
in RCTView (at View.js:45)
in View (at ResourceSavingScene.js:24)
in ResourceSavingScene (at createBottomTabNavigator.js:108)
in RCTView (at View.js:45)
in View (at screens.native.js:83)
in ScreenContainer (at createBottomTabNavigator.js:98)
in RCTView (at View.js:45)
in View (at createBottomTabNavigator.js:97)
in TabNavigationView (at createTabNavigator.js:178)
in NavigationView (at createNavigator.js:57)
in Navigator (at createNavigationContainer.js:376)
in NavigationContainer (at SceneView.js:9)
in SceneView (at StackViewLayout.js:478)
in RCTView (at View.js:45)
in View (at StackViewLayout.js:477)
in RCTView (at View.js:45)
in View (at StackViewLayout.js:476)
in RCTView (at View.js:45)
in View (at createAnimatedComponent.js:151)
in AnimatedComponent (at screens.native.js:59)
in Screen (at StackViewCard.js:42)
in Card (at createPointerEventsContainer.js:26)
in Container (at StackViewLayout.js:507)
in RCTView (at View.js:45)
in View (at screens.native.js:83)
in ScreenContainer (at StackViewLayout.js:401)
in RCTView (at View.js:45)
in View (at StackViewLayout.js:400)
in StackViewLayout (at withOrientation.js:30)
in withOrientation (at StackView.js:49)
in RCTView (at View.js:45)
in View (at Transitioner.js:141)
in Transitioner (at StackView.js:19)
in StackView (at createNavigator.js:57)
in Navigator (at createKeyboardAwareNavigator.js:11)
in KeyboardAwareNavigator (at createNavigationContainer.js:376)
in NavigationContainer (at Router.js:97)
in App (at Router.js:122)
in Router (at App.js:61)
in App (at renderApplication.js:35)
in RCTView (at View.js:45)
in View (at AppContainer.js:98)
in RCTView (at View.js:45)
in View (at react-native-root-siblings/index.js:33)
in RootSiblingsWrapper (at AppContainer.js:112)
in RCTView (at View.js:45)
in View (at AppContainer.js:115)
in AppContainer (at renderApplication.js:34)
This error is located at:
in NavigationContainer (at SceneView.js:9)
in SceneView (at createTabNavigator.js:39)
in RCTView (at View.js:45)
in View (at ResourceSavingScene.js:35)
in RCTView (at View.js:45)
in View (at ResourceSavingScene.js:24)
in ResourceSavingScene (at createBottomTabNavigator.js:108)
in RCTView (at View.js:45)
in View (at screens.native.js:83)
in ScreenContainer (at createBottomTabNavigator.js:98)
in RCTView (at View.js:45)
in View (at createBottomTabNavigator.js:97)
in TabNavigationView (at createTabNavigator.js:178)
in NavigationView (at createNavigator.js:57)
in Navigator (at createNavigationContainer.js:376)
in NavigationContainer (at SceneView.js:9)
in SceneView (at StackViewLayout.js:478)
in RCTView (at View.js:45)
in View (at StackViewLayout.js:477)
in RCTView (at View.js:45)
in View (at StackViewLayout.js:476)
in RCTView (at View.js:45)
in View (at createAnimatedComponent.js:151)
in AnimatedComponent (at screens.native.js:59)
in Screen (at StackViewCard.js:42)
in Card (at createPointerEventsContainer.js:26)
in Container (at StackViewLayout.js:507)
in RCTView (at View.js:45)
in View (at screens.native.js:83)
in ScreenContainer (at StackViewLayout.js:401)
in RCTView (at View.js:45)
in View (at StackViewLayout.js:400)
in StackViewLayout (at withOrientation.js:30)
in withOrientation (at StackView.js:49)
in RCTView (at View.js:45)
in View (at Transitioner.js:141)
in Transitioner (at StackView.js:19)
in StackView (at createNavigator.js:57)
in Navigator (at createKeyboardAwareNavigator.js:11)
in KeyboardAwareNavigator (at createNavigationContainer.js:376)
in NavigationContainer (at Router.js:97)
in App (at Router.js:122)
in Router (at App.js:61)
in App (at renderApplication.js:35)
in RCTView (at View.js:45)
in View (at AppContainer.js:98)
in RCTView (at View.js:45)
in View (at react-native-root-siblings/index.js:33)
in RootSiblingsWrapper (at AppContainer.js:112)
in RCTView (at View.js:45)
in View (at AppContainer.js:115)
in AppContainer (at renderApplication.js:34)
This error is located at:
in NavigationContainer (at SceneView.js:9)
in SceneView (at StackViewLayout.js:478)
in RCTView (at View.js:45)
in View (at StackViewLayout.js:477)
in RCTView (at View.js:45)
in View (at StackViewLayout.js:476)
in RCTView (at View.js:45)
in View (at createAnimatedComponent.js:151)
in AnimatedComponent (at screens.native.js:59)
in Screen (at StackViewCard.js:42)
in Card (at createPointerEventsContainer.js:26)
in Container (at StackViewLayout.js:507)
in RCTView (at View.js:45)
in View (at screens.native.js:83)
in ScreenContainer (at StackViewLayout.js:401)
in RCTView (at View.js:45)
in View (at StackViewLayout.js:400)
in StackViewLayout (at withOrientation.js:30)
in withOrientation (at StackView.js:49)
in RCTView (at View.js:45)
in View (at Transitioner.js:141)
in Transitioner (at StackView.js:19)
in StackView (at createNavigator.js:57)
in Navigator (at createKeyboardAwareNavigator.js:11)
in KeyboardAwareNavigator (at createNavigationContainer.js:376)
in NavigationContainer (at Router.js:97)
in App (at Router.js:122)
in Router (at App.js:61)
in App (at renderApplication.js:35)
in RCTView (at View.js:45)
in View (at AppContainer.js:98)
in RCTView (at View.js:45)
in View (at react-native-root-siblings/index.js:33)
in RootSiblingsWrapper (at AppContainer.js:112)
in RCTView (at View.js:45)
in View (at AppContainer.js:115)
in AppContainer (at renderApplication.js:34)
This error is located at:
in NavigationContainer (at Router.js:97)
in App (at Router.js:122)
in Router (at App.js:61)
in App (at renderApplication.js:35)
in RCTView (at View.js:45)
in View (at AppContainer.js:98)
in RCTView (at View.js:45)
in View (at react-native-root-siblings/index.js:33)
in RootSiblingsWrapper (at AppContainer.js:112)
in RCTView (at View.js:45)
in View (at AppContainer.js:115)
in AppContainer (at renderApplication.js:34)
getInitialOrientation
D:\hms\react native test1\FluxTest\node_modules\react-native-orientation\index.js:98:23
componentWillMount
D:\hms\react native test1\FluxTest\Screen\ScarletScreen.js:49:18
callComponentWillMount
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\Renderer\oss\ReactNativeRenderer-dev.js:7430:4
mountClassInstance
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\Renderer\oss\ReactNativeRenderer-dev.js:7576:27
updateClassComponent
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\Renderer\oss\ReactNativeRenderer-dev.js:11485:6
performUnitOfWork
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\Renderer\oss\ReactNativeRenderer-dev.js:17276:21
workLoop
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\Renderer\oss\ReactNativeRenderer-dev.js:17316:41
renderRoot
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\Renderer\oss\ReactNativeRenderer-dev.js:17417:15
performWorkOnRoot
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\Renderer\oss\ReactNativeRenderer-dev.js:18423:17
performWork
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\Renderer\oss\ReactNativeRenderer-dev.js:18324:24
performSyncWork
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\Renderer\oss\ReactNativeRenderer-dev.js:18285:14
requestWork
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\Renderer\oss\ReactNativeRenderer-dev.js:18169:19
scheduleWork
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\Renderer\oss\ReactNativeRenderer-dev.js:17969:16
scheduleRootUpdate
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\Renderer\oss\ReactNativeRenderer-dev.js:18642:15
render
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\Renderer\oss\ReactNativeRenderer-dev.js:19512:20
renderApplication
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\ReactNative\renderApplication.js:61:34
run
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\ReactNative\AppRegistry.js:104:10
runApplication
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\ReactNative\AppRegistry.js:198:26
__callFunction
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\BatchedBridge\MessageQueue.js:366:47

D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\BatchedBridge\MessageQueue.js:106:26
__guard
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\BatchedBridge\MessageQueue.js:314:10
callFunctionReturnFlushedQueue
D:\hms\react native test1\FluxTest\node_modules\react-native\Libraries\BatchedBridge\MessageQueue.js:105:17
callFunctionReturnFlushedQueue
[native code]

@hms111111  Try do manual linking. Its works for me.

@hms111111, The npm version is not compatible with the latest version of React-Native, So remove your last installation of react-native-orientation by the following commands:
$ npx react-native link react-native-orientation
$ yarn remove react-native-orientation

And then directly install it from GitHub by using the following command:
react-native-orientation@git+https://github.com/yamill/react-native-orientation.git

Hello,
I'm trying to install from github because I have the same issue and it doesn't work (can't be installed). I'm new in React Native and I'm totally lost.
Could you help me please ?

@elo-tnt1130 add npm i in front


Containerd Integration

Docker Engine 1.11 will use containerd for container supervision. Because containerd ultimately relies on runC and the OCI specification for container execution, this will open the door for the Engine to be able to use any OCI compliant runtime.
This new architecture will also open the door for potentially making it possible to upgrade the daemon without shutting down all running containers in the future, and will bring a significant performance boost when handling a big amount of containers. Following the plumbing philosophy, a great aspect of this design is that Docker Engine execution layer will be entirely relying on well delimited tools that can be used independently, with the drawback that it won't ship as a single binary anymore.

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO


About Experiment's results

Excuse me, I follow your method and get results successfully，but I don't know how to use these results to get  this paper's evaluation indicators to know this model's power. So, will you update this github to show the model's result use?

Sorry,  I find them in results' logs and thank you!

Of course, because their results are in the logs and it's easy for you to find them in the latest log's end. Besides, the dataset's val or test just only one and the author made training and testing together in one code.

@xiaomengxin123 你好，请问实验结果，你得出来的和论文一样吗。我按照Readme文档将环境配置好后，代码没有进行更改，按照步骤训练isic2018数据集，可是得到的结果并不像论文里那样的结果，不知道哪里出差错了！


feat(#1690): Add ZodRequired

Closes #1690
This PR adds the ZodRequired type, which removes any undefined from both the schema's output and input.

~Note: I couldn't add .required() to the root ZodType class because it conflicts with the .required() method of ZodObject. Suggestions are welcome. I feel like it should live in the root ZodType class...~

@santosmarco-caribou looks good at a glance!
I haven't taken an in depth look - but I wonder if, for the sake of keeping .required() on the root ZodType, it would be possible to bring in the specific ZodObject funtionality at the top ZodType so that there's no confilct in the namespace? or via a _required() on the ZodObject with a instanceof check at the ZodType level? Just some ideas.

@santosmarco-caribou looks good at a glance!
I haven't taken an in depth look - but I wonder if, for the sake of keeping .required() on the root ZodType, it would be possible to bring in the specific ZodObject funtionality at the top ZodType so that there's no confilct in the namespace? or via a _required() on the ZodObject with a instanceof check at the ZodType level? Just some ideas.

I'm afraid it isn't possible to bring the ZodObject logic up to the root ZodType class since the return type depends on the schema, unknownKeys, and catchall, which ZodType does not have access to.

I came up with a solution using overloads. Don't know if that's the best way to handle this, but well, it works, passes all tests, and does not create any conflict.

Very neat! Good job @santosmarco . To be sure, it would be a good idea to add tests that flex the interaction of ZodObject with .require() specifically, I think we're missing those.

On further reflection, I think Zod should possibly consider generalizing this to ZodExclude. Though I'm still on the fence about whether this is worthwhile.
Note that the types in this PR are technically wrong. For instance, if .required() behaved as implemented in this PR, the following schema would have Output type string, when it should be string | undefined.
z.string().transform(val => Math.random() > 0.5 ? val : undefined).required()

There's some inherent ambiguity here, which is why I've been hesistant to merge this. The open question is this: does the "required" check happen before or after the innerType parsing? I think different people will have different expectations about how this should work, unfortunately.

I'm going to close this for now, but I'll keep this in mind over the development cycle for Zod 4. I need to do some restructuring and see if a more natural API emerges for making object schemas required (which is the fundamental issue solved by this PR). I also need to think more about how this will interact with Zod 4's new approach to exactOptionalPropertyTypes. I'll re-open if I decide to go ahead with ZodRequired. Thanks for the great work Marco 🙌


Fix an import deprecation warning

What does this PR do?
Fix an import deprecation warning (collections -> collections.abc or typing, for some classes). Similar to #2746.
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Update here too?
https://github.com/PyTorchLightning/pytorch-lightning/blob/d85de32dcfa86aa0da95ab466bf90383f0dcf5e4/pytorch_lightning/utilities/apply_func.py#L2-L4

Update here too?
https://github.com/PyTorchLightning/pytorch-lightning/blob/d85de32dcfa86aa0da95ab466bf90383f0dcf5e4/pytorch_lightning/utilities/apply_func.py#L2-L4

No. The reason is the following, and it's detailed in the PEP 585. collections.abc is used, among other things, to actually subclass and check for isinstance, as it's used in that import you mentioned. The ones in typing were introduced because there was no support for generic abstract collections. In Python 3.9 this is gonna be the case and it seems that those are gonna be deprecated from typing. My understanding is that at least in CPython both work with isinstance but typing doesn't have to because they are intended just to be used to declare types.
The deprecation I mention is related to the abstract classes formerly being found in collections and now they are in collections.abc. However, in the change in this PR they are used for types, that's why I changed it to typing in this one.

Oh! okay. Thanks, @bryant1410 :) didn't know that.


Docs or examples of InnoSimulator Python API usage?

How can we use InnoSimulator Python API?
I have not found examples or instructions for that.
We need to run short and simple scenarios: put the car in a defined position, set the movement of the NPCvehicle and check the results.

You can find it here with tests and docs here


What is the relationship between the delta lake table and the hive table?

What is the relationship between the delta lake table and the hive table?
Now that the data is in hive, if it exists on detlalake, how do you perform related queries with the previous data?

Please use delta forum for this and other questions about Delta Lake.

i'm sorry .I find I can not visit https://groups.google.com/g/delta-users .
Do you know if there are any related blogs that explain this issue？If you know, please send it to me, thank you


[FEA] Improve handling of shuffles during integration tests

Is your feature request related to a problem? Please describe.
While working on #2145 I noticed that the existing repartition test was not actually testing anything on the GPU.  The test was supposed to be testing that a partition and shuffle matched the CPU, but the plan optimized out the GPU shuffle since the other parts of the plan were not columnar.  The test should have failed since the plan was not columnar in any way, but it went ahead and compared CPU to CPU which isn't helpful.
Describe the solution you'd like
There should be a mechanism to avoid ignoring shuffles that are not on the GPU or force shuffles to be translated to the GPU during planning.

When doing the review of tests that have this problem, focus on tests that are shuffle related.

I was asked in standup how many tests are failing when we add back in the check for the exchange to be on the GPU for all of the tests.
It turns out that we have 8 scala tests that fail, and 209 python tests.  The 209 number is a bit misleading because we have parametrized most of the tests there for types so it is actually a much smaller number of specific tests that are failing in this way.
I'll take a crack at fixing fixing the tests, but I may have to take a closer look at the python tests because I would not expect them to have a shuffle on the CPU normally.


Fix node-specific shard access via HTTP

Overview
In #2336 we removed node-local HTTP access and redirected all traffic through /_node/{node-name}/.... This sneakily routes a new MochiWeb request through to couch_httpd under the covers.
Unfortunately, in the process we ended up URL-decoding the URL, meaning the passed-through URL couldn't include %2F for accessing individual shards.
This reintroduces URL encoding to fix the problem.
It also introduces support to query a specific view shard through /_node/{node-name}/{db-shard}/{ddoc}/. This allows inspection of individual view shard sizes.
Testing recommendations
dev/run -n 1 -a admin:password
curl -X PUT http://admin:password@localhost:15984/abc
curl -X PUT http://admin:password@localhost:15984/abc/_design%2ffoo -d '{"language":"javascript","views":{"all":{"map":"function(doc) { emit(doc._id, null);}"}}}'
curl -X PUT http://admin:password@localhost:15984/abc/def -d '{}'
curl -X PUT http://admin:password@localhost:15984/abc/qqq -d '{}'
curl http://admin:password@localhost:15984/_node/_local/_all_dbs
# substitute your timestamp code in the following lines...
curl http://admin:password@localhost:15984/_node/_local/shards%2f00000000-7fffffff%2fabc.1580273428
curl http://admin:password@localhost:15984/_node/_local/shards%2f00000000-7fffffff%2fabc.1580273428/def
curl http://admin:password@localhost:15984/_node/_local/shards%2f00000000-7fffffff%2fabc.1580273428/qqq

curl http://admin:password@localhost:15984/abc/_design/foo/_info
curl http://admin:password@localhost:15984/abc/_design/foo/_view/all
curl http://admin:password@localhost:15984/_node/_local/shards%2f00000000-7fffffff%2fabc.1580273428/_design/foo/_info
curl http://admin:password@localhost:15984/_node/_local/shards%2f80000000-ffffffff%2fabc.1580273428/_design/foo/_info
curl http://admin:password@localhost:15984/_node/_local/shards%2f00000000-7fffffff%2fabc.1580273428/_design/foo/_view/all
curl http://admin:password@localhost:15984/_node/_local/shards%2f80000000-ffffffff%2fabc.1580273428/_design/foo/_view/all

Example output for the last few lines (new per-view-shard endpoint):
$ curl http://admin:password@localhost:15984/_node/_local/shards%2f00000000-7fffffff%2fabc.1580515057/_design/foo/_info
{"name":"foo","view_index":{"signature":"de76da589940694f05807f0c4fb5d33a","language":"javascript","sizes":{"file":4190,"active":98,"external":24},"update_seq":1,"purge_seq":0,"update_options":[],"updater_running":false,"compact_running":false,"waiting_commit":false,"waiting_clients":0,"pending_updates":0}}
$ curl http://admin:password@localhost:15984/_node/_local/shards%2f80000000-ffffffff%2fabc.1580515057/_design/foo/_info
{"name":"foo","view_index":{"signature":"de76da589940694f05807f0c4fb5d33a","language":"javascript","sizes":{"file":8292,"active":98,"external":24},"update_seq":2,"purge_seq":0,"update_options":[],"updater_running":false,"compact_running":false,"waiting_commit":false,"waiting_clients":0,"pending_updates":0}}

$ curl http://admin:password@localhost:15984/_node/_local/shards%2f00000000-7fffffff%2fabc.1580515057/_design/foo/_view/all
{"total_rows":1,"offset":0,"rows":[
{"id":"def","key":"def","value":null}
]}
$ curl http://admin:password@localhost:15984/_node/_local/shards%2f80000000-ffffffff%2fabc.1580515057/_design/foo/_view/all
{"total_rows":1,"offset":0,"rows":[
{"id":"qqq","key":"qqq","value":null}
]}

Related Issues or Pull Requests
Closes #2500 fully.

Big thanks to @kocolosk who nudged me in the right direction on IRC for per-view-shard endpoint exposure. Like @nickva 's change, it turned out to be very little code to support this.


NameError: global name 'basestring' is not defined (python 3)

Hey there,
Nice work you have done, thanks!
I just came across an error while trying to download/upload files with my python3.3 install:

NameError: global name 'basestring' is not defined

After a quick web research, I found the solution mentioned here to work just fine.
I added the lines right after the import lines in client.py.

I also would really like to use you library with Python3. As @minami-o mentioned, adding
try:
    basestring
except NameError:
    basestring = str

after all the imports should be sufficient.

+1: same problem, same solutions

Same problem here. Could you possibly do a new release, please?

just add basestring = str


Support more or all properties in $header-styles

So far $header-styles only uses 'font-size', 'line-height', 'margin-top' and 'margin-bottom'.
It makes probably sense to support more or all properties to prevent duplicate declaration of header styles in more scss files which are not supported here
https://github.com/zurb/foundation-sites/blob/develop/scss/settings/_settings.scss#L140-L157
https://github.com/zurb/foundation-sites/blob/develop/scss/typography/_base.scss#L320-L360

See https://github.com/zurb/foundation-sites/issues/9483 and https://github.com/zurb/foundation-sites/pull/9500.
There was a lot of discussions about the subject, and I don't remember end up agreeing on something. The main problem is that $header-styles is a tool to help to quickly create header styles. With too many options or supported properties, it became easier to advise people to create their header styles manually.

Closing this as it is a duplicate of an existing ticket.
I think this is something we should definitely discuss for F7.

Right. I'll subscribe to the other two issues.


Bug 711932 fix broken addons

get_latest is now fixing or deleting a broken add-on

This solution looks like it will work.  Could this have been fixed with a SQL script that only updates the broken packages?

I'd actually want to concentrate ion finding the issue broken add-ons exist


Request to add a parameter to "User Parameters"

VSCode Version: 1.22.1

Steps to Reproduce:
To remove the item "Open editors" - i need to right-click on it and remove the checkmark.
Can this function be added as an option for "User Parameters"?
Thank you.

"explorer.openEditors.visible": 0, ?

So I did not see this setting. I apologize.


Remove FloatEqualHandler

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (1b5cac7) 100.00% compared to head (93ff420) 100.00%.
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #422   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           73        73
  Lines         2568      2572    +4
  Branches       415       416    +1
=========================================
+ Hits          2568      2572    +4

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


Is trying to access ~/.config/gitconfigs but in macos system the path is ~/.gitconfig

Hi,
Is trying to access
~/.config/gitconfigs and is panicing.
But in my macos system the right path is
~/.gitconfig
There is a way to configure it?
thank you

Hi ,
The git switcher uses ~/.config/gitconfigs path to store multiple git profiles.


added maintainers list

Signed-off-by: Ryan Marsh<EMAIL_ADDRESS>

Typo in the first line.

Merging. This is uncontroversial and is repo metadata, not a HIPE.


maintenance, etc

Hi,
I saw your "call for maintainers" here.
Our project may be interested in maintaining this ourselves, but I don't really understand what this module even does or why it's even a thing.
Excuse my ignorance, as I'm not very experienced w/ Webpack.

Hello 👋
This loader aims to take advantage of webpack dependency resolution and hot module replacement when running tests using mocha. Ideally, you want to re-run all affected tests every time when you change the code. Thanks to webpack, you can track all dependencies from the file you've just edited and trigger affected tests only.

@boneskull I'm closing this issue for now. If you're still interested, feel free to submit some PRs or contact us directly. Thank you for your support!


How to render objects metallic, preferably by editting XML files?

Hi,
I'm a student and I'm trying to use MuJoCo for rigid body dynamics simulation and computer vision techniques.
I'm looking for some help with how to render an object more metallic. Below is the figure of my teapot model whose body is glass and lid is stainless:

As you can see, the stainless lid appears more like plastic or ceramic. How can I make it appear more metallic?
Below is the XML:
<mujoco model="target">
  <compiler meshdir="assets" texturedir="assets" autolimits="true"/>
  <default>
    <default class="visual">
      <geom group="2" type="mesh" contype="0" conaffinity="0"/>
    </default>
    <default class="collision">
      <geom group="3" type="mesh"/>
    </default>
  </default>
  <asset>
    <material name="236,241,239" specular="1.0" shininess="1.0" rgba="0.925490 0.945098 0.937255 0.220000"/>
    <material name="190,188,186" specular="0.7372549999999999" shininess="1.0" rgba="0.745098 0.737255 0.729412 1.000000"/>
    <mesh file="object_0.obj"/>
    <mesh file="object_1.obj"/>
  </asset>
  <worldbody>
    <body name="object">
      <geom mesh="object_0" material="190,188,186" class="visual"/>
      <geom mesh="object_1" material="236,241,239" class="visual"/>
      <geom mesh="object_0" class="collision"/>
      <geom mesh="object_1" class="collision"/>
    </body>
  </worldbody>
</mujoco>

object_1 corresponds to the lid.
Besides, I have noticed that there is a description of the metallic parameter asset/material elements (link). But I need to find out which external renderer supports this parameter.
The best-case scenario for me is making the lid more metallic only by editing my XML, but information about renderers supporting metallic surface rendering is also helpful.
Thanks in advance!

Hello,
Meshes don't support reflectance which is key in achieving a metallic effect (among other things)
If I understand correctly, explicit compatibility with external renderers will receive further support and guidelines in the near future. Right now, if you are looking for Phyiscally Based Rendering of realistic materials on MuJoCo objects, some of the approaches include

Performing the rendering in Blender based un the MuJoCo scene (example: https://arxiv.org/pdf/2404.15194v1)
Using the Unity plugin of MuJoCo

Hi, @Balint-H,
Thanks for your response. I will look into the paper.
l'm just curious. In my understanding, mujoco uses opengl, which should have the capability to simulate reflectance. Why has the reflectance not been supported so far?


Branch command edit staff command

Command:
editstaff INDEX pn/Project_Name [sn/Staff_Name]...
Closes #95 , #85

Codecov Report
Base: 76.23% // Head: 77.24% // Increases project coverage by +1.01% :tada:

Coverage data is based on head (a3df6f8) compared to base (8779bdb).
Patch coverage: 94.44% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #130      +/-   ##
============================================
+ Coverage     76.23%   77.24%   +1.01%
- Complexity      730      759      +29
============================================
  Files           115      117       +2
  Lines          2142     2268     +126
  Branches        260      287      +27
============================================
+ Hits           1633     1752     +119
- Misses          439      440       +1
- Partials         70       76       +6

Impacted Files
Coverage Δ

...main/java/seedu/address/commons/core/Messages.java
0.00% <ø> (ø)

...java/seedu/address/logic/commands/EditCommand.java
90.90% <ø> (ø)

...u/address/logic/parser/EditStaffCommandParser.java
88.57% <88.57%> (ø)

...seedu/address/logic/commands/EditStaffCommand.java
96.59% <96.59%> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
93.54% <100.00%> (+0.21%)
:arrow_up:

...ava/seedu/address/model/staff/UniqueStaffList.java
88.37% <100.00%> (+0.56%)
:arrow_up:

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

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


Mass search/replace swaps the contents of different files

Type: Bug
About a month ago I noticed that the mass search/replace function swaps the content of different files. For example, I've only just replaced one line in about 100 files using the built-in search/replace feature and about 20-30% files have the content that belongs to another files that also were replaced.
VS Code version: Code 1.83.0 (Universal) (e7e037083ff4455cf320e344325dacb480062c3c, 2023-10-03T16:13:15.449Z)
OS version: Darwin x64 23.0.0
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz (12 x 2600)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
2, 2, 4

Memory (System)
16.00GB (0.08GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (10)

Extension
Author (truncated)
Version

markdown-mermaid
bie
1.20.0

vscode-eslint
dba
2.4.2

prettier-vscode
esb
10.1.0

relay
met
1.0.2

git-graph
mhu
1.30.0

python
ms-
2023.18.0

vscode-pylance
ms-
2023.10.10

vscode-fileutils
sle
3.10.3

vscode-styled-components
sty
1.7.8

vscode-sql-lit
the
0.7.0

(1 theme extensions excluded)

I'm seeing the same thing

I have hit this several times, rather reproducibly: I just did a find and replace, then reverted all the files it messed up. Did the fine and replace again.  This repeated until the last file was fixed.
I was using this workspace (and on the exact git revision linked: https://github.com/CraigMacomber/FluidFramework/blob/75cb7b8487b576d286e70001968a8daaabcedbb5/experimental/dds/tree2/.vscode/Tree.code-workspace )
The find and replace I did was all files, replace "toDocumentSchema" with "toSchema".
I'm on Linux (Ubuntu 23.04) and the files are on a non root ZFS file system.
There is always at least one file what is updated correctly, and when the bug occurs, some files which should have been updated got their content replaced with other files which were updated.
I then reverted my changes and ran the fine and replace again. This time I reloaded the VSCode window before doing the find and replace to see if the accumulated state was having some impact. This time the messed up files (which were the same files) got their content replaces with an absolute path to the location of one of the files in the workspace instead of the content of another file what that was being replaced. But when I reverted the changes and did it again, it they were replaced with contents of other files.
I'm suspecting that somehow some uninitialized buffers and getting written out and they often, but not always, contain data from earlier in the multi-file find and replace operation. For some reason it always seems to be the same files.
It might be relevant that my system has a lot of cores (16 cores, 32 threads), and is writing to nvme storage, so there is a lot of potential for parallelization here.

Using that exact workspace and revision and find and replace text, I was unable to reproduce the issue in a github CodeSpace. I don't really know what possible causes that eliminates, but does imply something about my setup might be the issue.
Now having disabled all my extensions on my local setup, it also has stopped happening. I'll try and narrow it down to a specific extension, and see if that one enables it to be reproduced in a codesapce.

I think it is the Prettier extension that is causing the issue.
Interestingly when doing the find and replace right after reloading the window, the file path that ends up replacing some of the file contents is the config file for Prettier.
The issue seems to be related to format on save (file mode) on multi file fine and replace.
Thus as a workaround, you can disable format on save when doing large multi file find and replace, then manually run your auto-formatter as a followup.
In a codespace, my repro process shows the error if you install the Prettier extension, and enable format on save.

/duplicate of https://github.com/microsoft/vscode/issues/192617, since that issue also discusses the prettier issue.

I would be curious about whether this is just due to the prettier extension, though.

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Fix memory ownership issue in the init function

So I've been debugging this problem for many days now. Over at my nrf-modem I made this same patch.
It turns out that the params given to the init function need to live forever. Seems like the params aren't copied over, but are still referenced later.
No idea why nobody caught this earlier.
But this fixes it.
If anybody reviews and merges I can publish the new release.
Note though that I have some difficulties to compile this crate now. I think the bindgen version for nrfxlib-sys is quite old now.

I think the init() function might be unsound - it needs a boolean guard to stop you calling it twice. Otherwise it will re-initialise a heap that might be in use, and over-write a configuration structure that the nrfxlib library might already be using.

I think the init() function might be unsound - it needs a boolean guard to stop you calling it twice. Otherwise it will re-initialise a heap that might be in use, and over-write a configuration structure that the nrfxlib library might already be using.

I guess you're right. Though calling it twice right now will error anyways. But it becomes unsound just before the error. This is an issue in my version as well.
But look, I'm not that interested in maintaining this crate. I'd rather put the effort in my own. I made this PR mostly for anyone using this because the bug was so nasty.


Updated public string wrapping

test error in unit tests

Hey @eugene0928, thank you for the PR. After consideration, we decided to do this task internally so we could make sure they were all done in a timely manner. As such, your PR is now outdated and I'm going to close it but please consider checking out the translations we currently have 🙂


xampp: fix config and add php setup

File xampp-control.ini is not in archive by default, so we create it.
Scoop links it as directory, not a file, if it does not exist, which leads to incorrect start of application.
Also add automatic PHP setup (does not change system files).

[x] I have read the Contributing Guide.

/vertify

/verify

/verify


[ready] refactor getitem round 2 :D

gonna give refactoring getitem another go before cleaning up/enabling test_indexing
once getitem is suuuuuper nice and readable, enabling and merging stuff from test_indexing should be easy

@chenyuxyz
ok I think I managed to make our __getitem__ much more readable and I think the logic is easier to reason about
with a +0 line diff :D
well, more readable according my me... who wrote it.... so take a look and lemme know if it improved lol
few things I did

turned all internal lists and tuples tensors
tracked the elements in indices using a Dict[type, dimension] and use that through all of getitem
raise most of the exceptions in # validation! raise Errors block cuz by then we have parsed enough to raise those errors
removed the annoying StopIteration error by replacing the dim injection and dim collapse calculation with straight forward pop and inserts
removed .contiguous() cuz for some reason CI passes (before it was using torch+cpu and was causing errors)
added some tests (not comprehensive but I want to start working on test_indexing so that I can get a good idea on what tests to add)

there are few more TODOs I haven't finished

for # TODO: move all slice(None) to the end and transpose non-None to the front
do we still need to do that?

for # TODO: figure out the exact acceptable types for indices, especially for internal list/tuple types
I think I'll figure that one out after finishing test_indexing

I'll correct the docs at the very end

I think the getitem refactor is good enough to merge for now.

i guess sum(1 for i in indices if i is None) is correct here if indices contains something that breaks with a eq check. sorry for the confusion


Possible mistake within CPrediction

I believe you forgot to add a reference (or another indirection) to this line
https://github.com/rollraw/qo0-base/blob/b3d9ec4124711b124da053f452b238368aa9c780/base/features/prediction.h#L36
I think it should be like this
CBaseEntity** pSetPredictionEntity = nullptr;
And then in the code
*pSetPredictionEntity = pSomePtr;
While adjusting the remaining code accordingly
Edit: I'm not tooo sure about this, so feedback is appreciated

both will have same pointing address, the only difference, your is better for readability


Reusable Package

Any chance of moving the patching logic to an importable package? I would like to reuse the patching logic in a function that I am working on, but cannot because it is part of the main package. I could work on a PR if this is doable.

I think it would be a good idea, as other folks can use the logic in their own functions if needed.

A few questions if we  did this:

Would it make sense for the logic to live in this repo? Probably, especially given the plan to eventually deprecate the version that is currently in c/c.
What would the expectation be around maintenance of the library - e.g. would we as the function-patch-and-transform be signing up for significantly more work by maintaining P&T as a library as well?

I am leaning into a separate repo due to the sync issues we are starting to see with upstream P&T and concerns about deprecation. It would be nice if upstream, the P&T function and any forks share common code. We cold ensure existing P&T users that the function behavior is equivalent.

https://github.com/crossplane-contrib/function-patch-and-transform/issues/54
https://github.com/crossplane/crossplane/issues/4746

I still feel pretty bullish on removing the native P&T code from Crossplane. So personally I would not factor in needing to support a library used by both Crossplane core and this function.
I could imagine emulating native P&T support by transparently using a function inside of Crossplane.

I'm wary of making a reusable patch and transform library. I'd like to hear more about how folks think it would be used.
My primary concerns are:

Maintaining a library is more involved than maintaining an "app" (in this case a function). If we expose code others can import, we have to start worrying about backward compatibility and breaking changes at the code level, not just the user-facing API level (i.e. function input schema).

Do we want to optimize for proliferation of P&T functionality? Of course today it can proliferate by either copying the code or forking this function. My hunch is that most of those forks would either be short-lived (e.g. to test out a new feature we may accept "upstream"), or fairly specialized (e.g. a fork to satisfy some private/unique requirement). Making P&T a library starts to feel like we expect there will be N function-patch-and-transform variants in use in the wild on an ongoing basis. Do we think that will be the case?

FWIW I don't think the fact that the code here duplicates the P&T code in c/c is a compelling reason to create a library. I'm also a little concerned about drift between the two implementations, but I hope to address that by freezing and one day removing the c/c P&T functionality.

@negz I would agree with you. I've since found alternative solutions to what I was trying to do that no longer requires P&T. I'd be fine with closing this if no one else has any issues.

After thinking about it for a while I still don't feel good about proceeding here.
My understanding is that right now only one known consumer that wants this change (https://github.com/MisterMX/crossplane-function-server). Given that, I don't want to change the contract of this function for now (i.e. for it to become a library).
If we find that there ends up with more than a small handful of forks of this function we can revisit.


drogon: 1.6.0 -> 1.7.0

Motivation for this change
new update released (https://github.com/an-tao/drogon/releases/tag/v1.7.0)
Things done

bump version
implement a workaround so I no longer need to disable parts of the test
disable examples because they're no longer used for tests anymore
swap from libuuid to libossp_uuid for darwin compatibility

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

1 package failed to build:

drogon

Result of nixpkgs-review pr 127436 run on x86_64-darwin 1
1 package failed to build:

@ethancedwards8 I don't have a Mac to test this on, any chance you could send a gist of the log?  In the mean time I'll mark as broken on darwin.

Result of nixpkgs-review pr 127436 run on x86_64-darwin 1
1 package failed to build:

@ethancedwards8 I don't have a Mac to test this on, any chance you could send a gist of the log? In the meantime I'll mark as broken on darwin.

Sure thing. https://termbin.com/7c6r

Result of nixpkgs-review pr 127436 run on x86_64-darwin 1
1 package failed to build:

@ethancedwards8 I don't have a Mac to test this on, any chance you could send a gist of the log? In the meantime I'll mark as broken on darwin.

Sure thing. https://termbin.com/7c6r

Think I found the culprit:
https://github.com/NixOS/nixpkgs/blob/bad3ccd099ebe9a8aa017bda8500ab02787d90aa/pkgs/os-specific/linux/util-linux/default.nix#L72
Looks to me like libuuid on nixpkgs is provided by util-linux and thus doesn't support Darwin.  Thanks for bringing this to my attention.  I think I can fix this by using libossp_uuid instead.  I'll try this out when I get the chance.

@ethancedwards8 could you give this another test when you get the opportunity?  I just swapped out libuuid, hopefully this can make drogon accessible to more people.

@ethancedwards8 could you give this another test when you get the opportunity? I just swapped out libuuid, hopefully this can make drogon accessible to more people.

Yup, got a notification, started compiling a few minutes ago.

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

1 package built:

drogon

Yay!

Thanks so much!


question about evaluation

Hi! Thanks for having this amazing project! Is it possible to open-source the evaluation code? I understand the code depends on ltu
I generated a json file result with gama_inf.py like:
 {
        "audio_id": "/mnt/NVME-VM/projects/LLaVa_Mic/GAMA/data/test/acl_sk_24/filtered_audios/Y0SSy52rc1BM.wav",
        "instruction": "Deduce the possible role of the man speaking softly in the midst of music and choir. Associate the auditory analysis with the provided visuals to create a comprehensive understanding of the scene.",
        "prediction": "The man's speech, amidst music and singing, could be an announcement or commentary, possibly guiding the audience through the event or providing information about the performance or venue.",
        "timestamp_events": "['(Choir-0.0-1.932)', '(Music-0.0-10.0)', '(Hubbub, speech noise, speech babble-0.0-10.0)', '(Choir-3.092-10.0)']",
        "ref": "The man's soft speech could be a personal conversation or commentary amidst the event. In the context of the visuals, he might be an attendee discussing or commenting on the ongoing performance."
    },

I am wondering how to evaluate these results.
I am still confused with all those metrics. I am a beginner in this field, and I don't quite understand the structure of ltu and what each code is evaluating. It would be really helpful even just explaining me some more details.
Thank you so much!

Hi @kayleeliyx ,
Looks like you are trying to evaluate responses on GAMA-IT. @sonalkum should be able to point you to the evaluation script which uses GPT to evaluate.

Hi! @sonalkum, I would be very grateful if you could help me with this! Thank you so much!

Hi @kayleeliyx, you can use this script to evaluate captions using GPT. Thanks!

Closing as completed. Let us know if you face any further issues!
Best,


planet, hm: use config.xdg.dataHome instead of hardcoding .local/share

Many of the home manager modules in planet hardcode .local/share instead of using config.xdg.dataHome.
The same is true for config.xdg.configHome and .config.
This issue is particularly common when setting up persistent files and directories.

Unfortunately, this won't work because config.xdg.dataHome and friends are absolute paths from the root, and not relative paths from the user's home directory; impermanence expects the paths to be relative to the user's home directory.
Because of this, the zoxide persistence configuration (added in 55ec503e7f602b5ec1d650ee94315929f13ba974) is currently broken.


ARROW-5485: [C++] Install libraries from googletest_ep into build output directory on non-Windows platforms.

Copy GTest libraries into the artifact folder
Use Rpath to install GTest libraries.
Turn tests back on for Gandiva crossbow.

Travis links : https://travis-ci.org/apache/arrow/builds/543159861
https://travis-ci.org/praveenbingo/arrow-build/builds/543160495

@wesm @pravindra This is ready now, can you please review..


Throw NullPointException when response code is 422

some restful api maybe return error http code like "401","403","422", but HttpURLConnection cannot Identify error http code except 404 Not Found,it makes feign.Response.<init> throw NullPointException  at checkNotNull(reason, "reason")
stack:
java.lang.NullPointerException: reason
        at feign.Util.checkNotNull(Util.java:104) ~[feign-core-8.12.1.jar:8.12.1]
        at feign.Response.<init>(Response.java:49) ~[feign-core-8.12.1.jar:8.12.1]
        at feign.Response.create(Response.java:60) ~[feign-core-8.12.1.jar:8.12.1]

this should be fixed in latest feign

I have the same error.

@xyalan please update to a more recent version of feign, as this was indeed addressed a while back https://github.com/Netflix/feign/blob/8.x/core/src/main/java/feign/Response.java#L49


Dependency Version Issues

See https://pastebin.com/zPCRVhKW (DGML)  My project  https://github.com/vbjay/TFS-Release-Notes is a powershell module that will generate release notes from workitems on VSTS.  The dependency version issues are causing problems.  Please fix dependency issues.
I have tried updating nuget packages to the latest and they still have the same issues.
Get-WorkItemChangesets : Could not load file or assembly 'System.Net.Http.Formatting, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' or one of its dependencies. The system cannot find the file
specified.
At E:\Apps\Git\FG TFS-Release-Notes\build\output\Change Request generator.ps1:51 char:1

Get-WorkItemChangesets -TFSCollection $tfs -WorkItemIDs $workitems -GetSubWorkIt ...

  + CategoryInfo          : NotSpecified: (:) [Get-WorkItemChangesets], FileNotFoundException
  + FullyQualifiedErrorId : System.IO.FileNotFoundException,TFS_Release_Notes.GetWorkItemChangesets

Get-WorkItemFiles : Could not load file or assembly 'System.Net.Http.Formatting, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' or one of its dependencies. The system cannot find the file
specified.
At E:\Apps\Git\FG TFS-Release-Notes\build\output\Change Request generator.ps1:54 char:1

Get-WorkItemFiles -TFSCollection $tfs -WorkItemIDs $workitems -GetSubWorkItems | ...

  + CategoryInfo          : NotSpecified: (:) [Get-WorkItemFiles], FileNotFoundException
  + FullyQualifiedErrorId : System.IO.FileNotFoundException,TFS_Release_Notes.GetWorkItemFiles

Get-WorkItemAttachments : Could not load file or assembly 'System.Net.Http.Formatting, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' or one of its dependencies. The system cannot find the file
specified.
At E:\Apps\Git\FG TFS-Release-Notes\build\output\Change Request generator.ps1:57 char:1

Get-WorkItemAttachments -TFSCollection $tfs -WorkItemIDs $workitems -GetSubWorkI ...

  + CategoryInfo          : NotSpecified: (:) [Get-WorkItemAttachments], FileNotFoundException
  + FullyQualifiedErrorId : System.IO.FileNotFoundException,TFS_Release_Notes.GetWorkItemAttachments

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

ID: 413163f0-28c3-a2a7-d626-fcb81bcb2257
Version Independent ID: 57c127da-2b1e-d29e-9abb-a3c258762ccf
Content: .NET Client Libraries for VSTS (and TFS)
Content Source: docs/integrate/concepts/dotnet-client-libraries.md
Product: devops
GitHub Login: @elbatk
Microsoft Alias: elbatk

*** Assembly Binder Log Entry  (6/4/2018 @ 11:33:20 PM) ***
The operation failed.
Bind result: hr = 0x80070002. The system cannot find the file specified.
Assembly manager loaded from:  C:\Windows\Microsoft.NET\Framework64\v4.0.30319\clr.dll
Running under executable  C:\Windows\System32\WindowsPowerShell\v1.0\powershell_ise.exe
--- A detailed error log follows.
=== Pre-bind state information ===
LOG: DisplayName = System.Net.Http.Formatting, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35
(Fully-specified)
LOG: Appbase = file:///C:/Windows/System32/WindowsPowerShell/v1.0/
LOG: Initial PrivatePath = NULL
LOG: Dynamic Base = NULL
LOG: Cache Base = NULL
LOG: AppName = powershell_ise.exe
Calling assembly : Microsoft.VisualStudio.Services.WebApi, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a.
LOG: This bind starts in LoadFrom load context.
WRN: Native image will not be probed in LoadFrom context. Native image will only be probed in default load context, like with Assembly.Load().
LOG: Using application configuration file: C:\Windows\System32\WindowsPowerShell\v1.0\powershell_ise.exe.Config
LOG: Using host configuration file:
LOG: Using machine configuration file from C:\Windows\Microsoft.NET\Framework64\v4.0.30319\config\machine.config.
LOG: Post-policy reference: System.Net.Http.Formatting, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35
LOG: GAC Lookup was unsuccessful.
LOG: Attempting download of new URL file:///C:/Windows/System32/WindowsPowerShell/v1.0/System.Net.Http.Formatting.DLL.
LOG: Attempting download of new URL file:///C:/Windows/System32/WindowsPowerShell/v1.0/System.Net.Http.Formatting/System.Net.Http.Formatting.DLL.
LOG: Attempting download of new URL file:///C:/Windows/System32/WindowsPowerShell/v1.0/System.Net.Http.Formatting.EXE.
LOG: Attempting download of new URL file:///C:/Windows/System32/WindowsPowerShell/v1.0/System.Net.Http.Formatting/System.Net.Http.Formatting.EXE.
LOG: Attempting download of new URL file:///D:/git/Release Notes Sample/build/output/TFS-Release-Notes/System.Net.Http.Formatting.DLL.
LOG: Assembly download was successful. Attempting setup of file: D:\git\Release Notes Sample\build\output\TFS-Release-Notes\System.Net.Http.Formatting.dll
LOG: Entering run-from-source setup phase.
LOG: Assembly Name is: System.Net.Http.Formatting, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35
WRN: Comparing the assembly name resulted in the mismatch: Build Number
ERR: The assembly reference did not match the assembly definition found.
ERR: Run-from-source setup phase failed with hr = 0x80131040.
LOG: Attempting download of new URL file:///D:/git/Release Notes Sample/build/output/TFS-Release-Notes/System.Net.Http.Formatting/System.Net.Http.Formatting.DLL.
LOG: Attempting download of new URL file:///D:/git/Release Notes Sample/build/output/TFS-Release-Notes/System.Net.Http.Formatting.EXE.
LOG: Attempting download of new URL file:///D:/git/Release Notes Sample/build/output/TFS-Release-Notes/System.Net.Http.Formatting/System.Net.Http.Formatting.EXE.
LOG: All probing URLs attempted and failed.
Call stack
Microsoft.TeamFoundation.Client.dll!Microsoft.TeamFoundation.Client.TfsConnection.CreateVssConnection(Microsoft.VisualStudio.Services.Common.VssHttpRequestSettings requestSettings)
Microsoft.TeamFoundation.Client.dll!Microsoft.TeamFoundation.Client.TfsConnection.ValidateAndPrepareGetClient<Microsoft.TeamFoundation.SourceControl.WebApi.GitHttpClient>(Microsoft.VisualStudio.Services.Common.VssHttpRequestSettings requestSettings)
Microsoft.TeamFoundation.Client.dll!Microsoft.TeamFoundation.Client.TfsConnection.GetClient<Microsoft.TeamFoundation.SourceControl.WebApi.GitHttpClient>()
Microsoft.TeamFoundation.Git.Client.dll!Microsoft.TeamFoundation.Git.Client.WebServiceLayer.WebServiceLayer(Microsoft.TeamFoundation.Client.TfsTeamProjectCollection projectCollection)
Microsoft.TeamFoundation.Git.Client.dll!Microsoft.TeamFoundation.Git.Client.GitRepositoryService.Initialize(Microsoft.TeamFoundation.Client.TfsTeamProjectCollection projectCollection)
Microsoft.TeamFoundation.Client.dll!Microsoft.TeamFoundation.Client.TfsTeamProjectCollection.InitializeTeamFoundationObject(string fullName, object instance)
Microsoft.TeamFoundation.Client.dll!Microsoft.TeamFoundation.Client.TfsConnection.CreateServiceInstance(System.Reflection.Assembly assembly, string fullName)
Microsoft.TeamFoundation.Client.dll!Microsoft.TeamFoundation.Client.TfsConnection.GetService(System.Type serviceType)
Microsoft.TeamFoundation.Client.dll!Microsoft.TeamFoundation.Client.TfsConnection.GetService<Microsoft.TeamFoundation.Git.Client.GitRepositoryService>()
TFS Release Notes.dll!TFS_Release_Notes.WorkItemCmdlet.BeginProcessing() Line 47
at D:\git\Release Notes Sample\TFS-Release-Notes\TFS Release Notes\Cmdlets\WorkItems\WorkItemCmdlet.vb(47)
Exception details
System.IO.FileNotFoundException
HResult=0x80070002
Message=Could not load file or assembly 'System.Net.Http.Formatting, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' or one of its dependencies. The system cannot find the file specified.
Source=Microsoft.VisualStudio.Services.WebApi
StackTrace:
at Microsoft.VisualStudio.Services.WebApi.VssConnection..ctor(Uri baseUrl, VssHttpMessageHandler innerHandler, IEnumerable`1 delegatingHandlers, Boolean allowUnattributedClients)
at Microsoft.TeamFoundation.Client.TfsConnection.CreateVssConnection(VssHttpRequestSettings requestSettings)
at Microsoft.TeamFoundation.Client.TfsConnection.ValidateAndPrepareGetClient[T](VssHttpRequestSettings requestSettings)
at Microsoft.TeamFoundation.Client.TfsConnection.GetClientT
at Microsoft.TeamFoundation.Git.Client.WebServiceLayer..ctor(TfsTeamProjectCollection projectCollection)
at Microsoft.TeamFoundation.Git.Client.GitRepositoryService.Initialize(TfsTeamProjectCollection projectCollection)
at Microsoft.TeamFoundation.Client.TfsTeamProjectCollection.InitializeTeamFoundationObject(String fullName, Object instance)
at Microsoft.TeamFoundation.Client.TfsConnection.CreateServiceInstance(Assembly assembly, String fullName)
at Microsoft.TeamFoundation.Client.TfsConnection.GetService(Type serviceType)
at Microsoft.TeamFoundation.Client.TfsConnection.GetServiceT
at TFS_Release_Notes.WorkItemCmdlet.BeginProcessing() in D:\git\Release Notes Sample\TFS-Release-Notes\TFS Release Notes\Cmdlets\WorkItems\WorkItemCmdlet.vb:line 47
at System.Management.Automation.Cmdlet.DoBeginProcessing()
at System.Management.Automation.CommandProcessorBase.DoBegin()
image.png
Redirects
Snippet

Powershell is not redirecting.
Sample project is found here.  https://github.com/vbjay/TFS-Release-Notes-Sample   Alter the
$tfs=get-tfs -ServerURL https://Your_VSTS_Here.visualstudio.com/DefaultCollection -VSTSToken $token
line  in the .\Change Request generator.ps1 file to point to your VSTS

I did fix the ad nuget mismatch by doing this.  https://github.com/vbjay/TFS-Release-Notes/commit/a7664ff55aebade33d9a5f7e2a6560a92f6533e5

This looks like a good question for the product team. Adding @alexmullans to help out!

I'll try an experiment of upgrading framework version.  I will let you know how it goes.

I upgraded all packages and reverted the ad package mismatch like I did above.  I still get the same issue.

FYI.  https://www.marss.co.ua/2017/05/reference-conflicts-analyzer-visual.html  Dependency Conflict Analyzer

I am going to try this  http://vunvulearadu.blogspot.com/2014/10/powershell-and-assembly-dll-how-to.html.

It does not work.  https://github.com/vbjay/TFS-Release-Notes/commit/9df2a26c62060df2469dc432534a1070e29effaf.

So frustrating. Trying to us the teamfoundation client from an existing WPF app. Where is the API documentation?!? all the links lead back to the sample page that requires the VSS nuget packages which don't work with .net standard.

No stale.


Eclipse WebService doubts

Hey, these may be noob questions but I have been trying to solve them for some time.

Once the setup is ready on eclipse, how to restart the web service once eclipse is closed and reopened?
There was a pom.xml but no Maven indications. I have set up eclipse project + configured with maven + added alloy42,jar to build path (for the services to have no import errors) but cannot figure out how to overcome the following errors on the WebContent part, any ideas on how to remove this errors (I assume this is needed to run the WebService)?:

I was able to run meteor, but without the webservice started and there was simply no error message when executing, is that "expected behaviour"?

Thank you!

Hi,

I think that you can start the server manually, through the eclipse interface.

I didn’t used maven throughout my setup process, however, did you followed these pre-instructions?

Yes, if there are no requests, the meteor is not giving feedback about the service status.

Concerning 3. I already identified source of the problem. See this issue: #15


Update gestures.md

Added extra information on the different touch stages. It was unclear that there were multiple click stages.

@OguzcanKarakoc if you sign the CLA, we can merge this contribution


quickstart flow

Motivation: allow a user to quickly play with their automation when doing a first tutorial. This will be available only in dev mode.

introduce webtools route, which supports both a terminal-in-browser tool and also messages.
to be used for quickstart tutorial and ability to "chat" with your automations.

Can you explain why do we need this tool? Look like the user can use Postman or similar tool.

added in description. talk to @haimzlato as well.

I'm not sure about the look and feel but the flow is OK. I would put the send at the top and the messages lower because the user will first send a request to ak and then see results not the other way around.

changed order

Can you explain why do we need this tool? Look like the user can use Postman or similar tool.
kinda late, but I would like to join Miki's question. Why do we want to implement this as a webtool instead of implementing this as a subcommand in ak for example? for me it feels much more naturally to have 2 shells, in one I could run ak up -t dev and in the second ak dev flow or similar.. doing this via the web feels for me less intuitive.
@itayd @haimzlato

add Experimental banner


only clean up git tests if file is left behind

Issue #, if available:
Description of changes:
if the git repository does not exist (failed to clone, already cleaned up by delete, etc) do not attempt to clean it up after running FluxGit tests.
Testing (if applicable):
ran this via the framework from my local and confirmed working
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/override eks-anywhere-attribution-files-presubmit
/override eks-anywhere-e2e-presubmit
/override eks-anywhere-presubmit


Add dev dependencies such as babel for backend

Any other useful dependencies that we may need

Will add as we go, basic ones are in


Early parser fails to report ambiguous trees

I am working on Python 3.7.3, Lark 0.7.1
Consider the following very simple example:
grammar = r"""
    start           : item_list
    item_list       : item_list? item
    item            : A | B
    A               : "a" | "ab"
    B               : "b"
"""
parser = Lark(grammar, parser="earley", ambiguity="explicit", debug=True)

result = parser.parse("ab")
print(result.pretty())

With the explicit option, I expected something like:
start
  __ambig
    item_list
      item_list
        item	a
      item	b
    item_list
      item ab

However, the output is:
start
  item_list
    item_list
      item	a
    item	b

Is this a bug? Or am I misunderstanding something?

What you're trying to do has to be enabled by the user, because it's potentially slow, and most users don't need it.
You can enable it using Lark(..., lexer='dynamic_complete')
Or in your case:
parser = Lark(grammar, ambiguity="explicit", lexer='dynamic_complete')

It behaves differently when I change the order of rules for A : "a" | "ab"to A : "ab" | "a". They mean the same thing semantically, don't they?
Here are the two sample outputs:
from lark import Lark
grammar = r"""
    start           : item_list
    item_list       : item_list? item
    item            : A | B
    A               : "ab" | "a"
    B               : "b"
"""
parser = Lark(grammar, ambiguity="explicit", lexer="dynamic_complete")
result = parser.parse("ab")
print(result.pretty())

output (this is what I expect):
start
  _ambig
    item_list
      item      ab
    item_list
      item_list
        item    a
      item      b

and the other rule:
from lark import Lark
grammar = r"""
    start           : item_list
    item_list       : item_list? item
    item            : A | B
    A               : "a" | "ab"
    B               : "b"
"""
parser = Lark(grammar, ambiguity="explicit", lexer="dynamic_complete")
result = parser.parse("ab")
print(result.pretty())

output (failing to catch ambiguity):
start
  item_list
    item_list
      item      a
    item        b

Using Python 3.7.3, Lark 0.7.1.

 I'll add the "bug" tag until proven otherwise.
@night199uk Do you have any insight to add?

P.S. It might be related to issue #319

will take a look tomorrow hopefully.

Yeah; this is odd, but I'm not sure if it's a bug . The problem in short (skipping some thing for brevity) is that the lexer is only returning one token for A and one token for B.
So there are two scenarios (depending on whether "ab" or "a" is listed first after A):
Scenario 1 (A: "a" | "b")
A matches "a" (lexer only returns a single token)
B matches "b"
Scenario 2 (A: "ab" | "b")
A matches "ab"
B matches "b"
In scenario #2, you can see an ambiguity with the 'dynamic_complete'; because dynamic_complete is a slightly misnomer. It doesn't actually tell the lexer to match all tokens, it tells the parser to dissect the returned token from the lexer and see if it could find a shorter match. So in this case, the lexer (which isn't a real lexer in Xearley but forget that a second), returns "ab" which is then turned into "a" and "ab" because "a" can also match the same pattern.
The question is whether this is a bug; and whether both variations of the lexer pattern:
A: "ab"
|  "a"
Should be added to the to_scan buffer for xearley.
I'll look more but @erezsh  what are your thoughts?

Sorry, I should have said. For xearley I think the best workaround is to ignore the lexer:
grammar = r"""
    start           : item_list
    item_list       : item_list? item
    item            : a | b
    a               : "ab" | "a"
    b               : "b"
"""

should produce the same output as:
grammar = r"""
    start           : item_list
    item_list       : item_list? item
    item            : a | b
    a               : "a" | "ab"
    b               : "b"
"""

Ahhh.
Yeah, I think the real point here is that  this:
A               : "ab" | "a"
essentially becomes a single terminal matcher that does this:
A: /ab|a/
or:
A: /a|ab/
This will only call the lexer once (as there is only 1 terminal - it's a terminal remember). The regex will return only one match, no matter what you do.
If you want two matches, you need to have two potential terminals (or two separate matches), where:
a: "a" | "ab"
equates to:
a: _ANON_1 | _ANON_2
_ANON_1: /a/
_ANON_2: /ab/
Thus allowing 2 matches to be returned as the lexer is called twice with two different pattern matches for two different terminals (ANON_1 and ANON_2). Hope that makes sense?

Yes, you're right. The regexp engine always returns one over the other, I might have missed it.
So there are two approaches to this:

Declare this the correct behavior. Require users to use rules if they want ambiguity resolution

Implement terminals as a rule anyway behind the scenes, for Earley. Then you get the desired behavior. So A: "a" | "ab" becomes !a: "a" | "ab"

Honestly, I'm leaning heavily towards the first approach. The second one takes more work, removes flexibility (what if someone wants to solve some ambiguity manually?), and only to fix something that users can easily fix themselves.
But I'm open to be persuaded otherwise. Until then, I consider this as not a bug.

@erezsh If you say this is a design by choice but not a bug, I can accept the way it is, as I understand there is a way to overcome this problem with minimal change of rules. Then it should better be documented somewhere, shouldn't it? It is against the natural expectation of most users, so they should know about it.

It's tied to how regular expressions work, and not Lark's implementation (except that it uses regexps), but I'm certainly open to improving the docs and making it clear.
Would you like to draft a paragraph explaining this to new users? I can then editorialize it and find the right place for it in the docs.

I'll give it a try in a couple of days.

Header
What happens when you use the OR operator to define terminals is not defined in the document. By implementation detail, it reduces to a single regex as @night199uk  explained, and I think that should be noted in the terminals section in the docs.

A : "ab" | "a"
essentially becomes a single terminal matcher that does this:
A: /ab|a/
or:
A: /a|ab/

Terminals with the OR operator
Two or more literals connected with the OR operator (|) is interpreted as a single regular expression. In the following example, the terminal A1 is equivalent to A2.
Example
A1 : "a" | "b"
A2: /a|b/

When one of the connected literals is a substring of another, it may lead to an unexpected behavior.
Example
start           : item_list
item_list       : item_list? item
item            : A | B
A               : "a" | "ab"
B               : "b"

For the string ab, this rule cannot catch the ambiguity but reports the following:
start
  item_list
    item_list
      item      a
    item        b

If you want two matches, you need to have two potential terminals:
Example
a: "a" | "ab"

equates to:
a: _ANON_1 | _ANON_2
_ANON_1: /a/
_ANON_2: /ab/

I'm not really sure, but do I have to ping you, @erezsh ? Just in case you missed it.

No no, sorry, I saw it, and it's in my task queue. It's a great start, Thanks!

Thanks! Just let me know if there is anything I can help with.

Updated. See here: https://lark-parser.readthedocs.io/en/latest/grammar/#terms


KAFKA-18025: Re-enable few tests in ShareConsumerTest.

What
After observing clean builds for a few tests over last 2 weeks, we are re-enabling some of these tests in the ShareConsumerTest suite.

Indeed, the report looks pretty good for this class https://ge.apache.org/scans/tests?search.relativeStartTime=P28D&search.rootProjectNames=kafka&search.tags=github,trunk&search.timeZoneId=America%2FNew_York&tests.container=kafka.test.api.ShareConsumerTest#
Looks like only two quarantined tests remain for this class:

testAcknowledgementCommitCallbackInvalidRecordStateException(String)[1]
testMultipleConsumersInGroupConcurrentConsumption(String)[1]


Make ArithmeticOperatorsTest dynamic

Test class gw.specification.expressions.arithmeticOperators.ArithmeticOperatorsTest requires itself to be compiled with JVM argument -DcheckedArithmetic=true in order to pass. But since maven compilation occurs in-process, this JVM argument must be set using the machine's JAVA_OPTS or MAVEN_OPTS variables when performing static compilation. This makes the test very brittle when run locally, and also means that the entire Gosu project is compiled using safe (and potentially slower) arithmetic operations - not desirable.
Some thoughts:

Convert ArithmeticOperatorsTest to be dynamic and executed inside an eval() block
gw.internal.gosu.ir.transform.AbstractElementTransformer caches the value of checkedArithmetic in a LazyVar, but provides no facility to reset/clear the value

Investigate exposing the mutators for checkedArithmetic as a command-line compiler argument

Fixed by 7016dd3523a3d44a68814fd73863adc9b24e789d; closing.


[turborepo] kitchen-sink example project does not work well with VS Code

What version of Turborepo are you using?
latest (1.10.16)
What package manager are you using / does the bug impact?
npm
What operating system are you using?
Mac
Describe the Bug
Every Typescript file opened in VS Code has the following error at its top:
Parsing error: Cannot read file '/users/tsopeh/...../delme/tsconfig.json'.
Expected Behavior
Since this is an example project, I expected that tooling "just works". Or at least some additional instructions on how to fix this.
PS: Webstorm IDE works without issues.
To Reproduce
I followed the instructions, and created an example project named delme. After that, I've ran the npm run dev command and everything has compiled and ran successfully. When I opened the project in VS Code, every typescript file had the following error at its top:
Parsing error: Cannot read file '/users/tsopeh/...../delme/tsconfig.json'.

Using

VS Code (latest, version 1.84.2)
Turborepo (latest, version 1.10.16)
Node v20.8.0
MacOS 14.1

Reproduction Repo
No response

We're aware of this! Mt current project is to breathe new life into our examples, perhaps with some re-architecting along the way.
Stay tuned!

Hey @tsopeh! I found a workaround. Create or find .vscode/settings.json and add the following:
{
    "eslint.workingDirectories": [{ "pattern": "apps/*/" }, { "pattern": "packages/*/" }]
}

@anthonyshew i know you've put some major effort into these, can you comment/close this one when you get a chance?

Yep! 🥳

How do we resolve this issue?  I am still seeing this when creating a new turbo repo project. @anthonyshew

I pop this in my .vscode/settings.json file
{
  "eslint.workingDirectories": [
    {
      "pattern": "apps/*/"
    },
    {
      "pattern": "packages/*/"
    }
  ]
}


Feature/terra3 105 more accounts for ecr repo access

Please review my PR.

This PR is included in version 1.21.0 :tada:


A ticket in progress needs an assignee

Yeah, I'm working on that. I've already assigned it to myself. This should close it

Great, thanks for the update @iheziqi


VPC delete action

I have a use-case where I would like to delete entire VPCs based on their creation date. It looks like there may already be the capability to apply generic filters to VPCs, but there doesn't appear to be a VPC deletion action (instead we can currently only delete security groups).

deleting a vpc, also means deleting all resources attached to it prior to invoking delete. are the vpcs your looking to delete already empty, or are you looking to delete every vpc attached resource (lambda, ec2, rds, redshift, efs, etc) before deleting the vpc?

at the moment if a vpc is non empty we would just error out in the action and stop that policy's execution.

I think that supporting the deletion of empty VPCs would address our use-case. We have an integration test suite which is somehow leaking empty VPCs. There are other layers of attempted cleanup, but I'd like to also be able to use Custodian as an additional safety net to cleanup those resources.


TENV_AUTO_INSTALL is not working

Describe the bug
I have TENV_AUTO_INSTALL set to true, but tenv refuses to acknowlege it
To Reproduce
➜  talos git:(main) echo $TENV_AUTO_INSTALL
true
➜  talos git:(main) tenv tf detect -n
Resolved version from .terraform-version : 1.9.0
Auto-install is disabled. To install Terraform version 1.9.0, you can set environment variable TENV_AUTO_INSTALL=true, or install it via any of the following command: 'tenv terraform install', 'tenv terraform install 1.9.0'
no compatible version found locally
Terraform 1.9.0 will be run from this directory.

Expected behavior
tfenv should install what's in .terraform-version
Screenshots
Environment (please complete the following information):

OS: NixOS 24.11 (Vicuna)
tofuenv version: v2.2.1

Additional context

Looks like a bug. Thank you for reporting, we'll fix it

Hi @myoung34, there is a -n flag (same as --no-install) in your command, this flag override the TENV_AUTO_INSTALL env var

@dvaumoron how do you think, do we still need this flag?

as now, auto install default to disabled, it's strange to use -n=false to activate it with the flag, we should probably add a simpler -i (still with priority to -n)


Minor issues in the prompts

Hi! I was quickly looking through the prompts and may have noticed a few minor issues in them:

The in-context example here and in other prompts has the wrong flights. The query asks for Roanoke to Illinois, but the travel plan uses Ithaca to Charlotte:

CostEnquiry is mis-spelled as CostEuquiry in a few places, like here. (The call here also has the incorrect flights)

Hi Brandon,
Thank you for your careful check. We have fixed these mirror errors.
If you have any further questions, please contact us.
Best,
Jian


[WIP] Proof of Concept

Awesome, I will try it and do the needful.
@Aagat Is it on purpose that the last commit (https://github.com/Aagat/new.hashbang.sh/commit/ad6499c9094550ee1b9903f0f736b0feaae7b3ed) is flagged [temp] ?

@KellerFuchs Yes, changes made in utils/clienthandler.js in that last commit is temporary so it could render user's details instead of welcome screen. Probably should've broken it down/not checked it.
I won't be able to push anything for a few days. Will finish things up and write tests if you don't find the time.


Academy links added contextually

Mattermost Academy links added conceptually to applicable documentation topics based on this spec
Repurposed the existing badge design code for these Academy links. Additional branding/styling can be applied to a specific academy-badge class if needed.
@matthewbirtch - I welcome your input on this docs PR. Is UX open to advising on look and feel for these Academy call outs?

Hey @cwarnermm  – just queuing this up - some Qs:

Where can I find the image used in the callout? The one with the laptop and clicky/cursor accent?
Is there a Figma for the callout? Just wanna make sure I got colours right.

TY!

@matthewbirtch - Can you help provide the image assets needed, please?

@emdecr @cwarnermm you can find the callout in this figma file
Or you can use this image directly:

@emdecr - Yes to all 4. And thank you!

@cwarnermm This should be good for a first round of reviews. I did a bulk update, so def flag if some badge instances look wonky on certain pages. Thanks!

Oh and one note: @matthewbirtch I added a placeholder hover style on the badges (just a bg colour change for now) – lemme know if you had specs in mind. TY!
Ex pg: http://mattermost-docs-preview-pulls.s3-website-us-east-1.amazonaws.com/6733/collaborate/collaborate-using-mattermost-for-microsoft-teams.html

These additions look FANTASTIC, @emdecr and @matthewbirtch!! Thank you both so much!
@emdecr - One Academy link is missing from this PR on the Search for messages page.
Also, I noticed that the perma links don't resolve yet to Academy; I imagine that's expected. :)

Oh and one note: @matthewbirtch I added a placeholder hover style on the badges (just a bg colour change for now) – lemme know if you had specs in mind. TY! Ex pg: http://mattermost-docs-preview-pulls.s3-website-us-east-1.amazonaws.com/6733/collaborate/collaborate-using-mattermost-for-microsoft-teams.html

Looks great @emdecr. I'm good with this hover state @emdecr. If we could also add a border-color: #DDDFE4 on hover, that would be good too.
I also just noticed that the layout of the callout breaks a bit between 768-960.

Looks great other than that!

@cwarnermm Thanks Carrie!

I don't see an .. include:: on that search page template. Where did we want it placed and which badge? academy-search.rst?
And yes, re: links – just added a task. Using this doc as reference – should be able to get to them all today.

@matthewbirtch Appreciate the flag + feedback! Should be fixed and added the border colour change 👍

@emdecr - Sounds like I missed adding that link, not you! :) Whoops! Yes, it would be the academy-search badge; the pl would be /pl/mattermost-academy-search-training. Placement would be directly following the intro text and before the first heading on that page. Really appreciate your help with this! Sorry for the miss!
And thanks for creating the pl links too! :)

@matthewbirtch Appreciate the flag + feedback! Should be fixed and added the border colour change 👍

Nailed it! Thanks

Sorry, one more really minor detail @emdecr: the border radius on the left side of the callout seems a little off. I think because there is the same border radius applied to both the outer container and the top-left and bottom-left corner of the image. We probably need to adjust the radius on the inner image to 5px instead of 6px to fix that.

@matthewbirtch No detail is too minor 😁  – def keep flagging anything; appreciate it 👍

@cwarnermm Circling-back to say the /pl/ links should be live, but just holler if one's missing. TY!


dele page

hello,
You can turn off this page, which will bring you bad luck
http://output.jsbin.com/fumugafuho
thanks

Not sure I follow, but I deleted the bin.


use @testing-library/react-native everywhere

closes #56

Pull Request Test Coverage Report for Build 417329174

6 of 6 (100.0%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+4.4%) to 91.176%

Totals

Change from base Build 417327363:
4.4%

Covered Lines:
84

Relevant Lines:
86

💛  - Coveralls


Augment example test with interpreter usage

Augmented the example test with interpreter usage demo.
nit: Fixed the dependency of MLIRParser on StablehloReferenceApi as API.cc uses parseSourceFile.

LGTM --

I'm wondering if we should have a simple example of add that takes input and one that uses constants?
I'd lean on more examples than in the future keep expanding our sole tests.
For this PR -- I think the contribution is good though. 👍

I'd lean on more examples than in the future keep expanding our sole tests.

Makes sense to me.
For this particualr PR: One of the motivation to create example is to demo feeding constant input to a stablehlo program using the APIs introduced in https://github.com/openxla/stablehlo/pull/2062. I agree that we should keep on iterate on this to make it simpler and intuitive to follow.


Snowfackery error when generating Lat/Long field: No encoder found for <class 'decimal.Decimal'>

Describe the bug
If the Snowfackery yaml files contains lat/long fields, the generate_and_load_from_yaml command fails with a  "No encoder found for <class 'decimal.Decimal'>" message.
Reproduction steps

Install CCI/Snowfackery
git clone https://github.com/SFDO-Tooling/Snowfakery.git

Authenticate and org
cci org connect  sdosb-aws   --sandbox

check the version version

$ cci version
CumulusCI version: 3.54.0 (/usr/local/Cellar/cumulusci/3.54.0/libexec/bin/cci)
Python version: 3.9.10<EMAIL_ADDRESS>You have the latest version of CumulusCI

run the Contact recipe that comes with the distribution:
cci task run generate_and_load_from_yaml -o generator_yaml examples/salesforce/Contact.recipe.yml -o num_records 3 -o num_records_tablename Contact --org sdosb-aws

Output:
[03/17/22 16:08:56] Org info updated, writing to keychain
                    Beginning task: GenerateAndLoadDataFromYaml
                    As user: XXXXXXXXX
                    In org: XXXXXXXXx

                    Beginning task: GenerateDataFromYaml

                    Generating batch 0 with 3

Error: Cannot write row : No encoder found for <class 'decimal.Decimal'> in SqlDbOutputStream for OtherLatitude, -59.3659685 in Contact
 near /Users/XXXXXXXX/snowfackery/snow_demo/Snowfakery/examples/salesforce/Contact.recipe.yml:30

Comment out all the Lat/Long fields in the yml file and try again -> it works

Your CumulusCI and Python versions
$ cci version
CumulusCI version: 3.54.0 (/usr/local/Cellar/cumulusci/3.54.0/libexec/bin/cci)
Python version: 3.9.10<EMAIL_ADDRESS>You have the latest version of CumulusCI

### Operating System

MacOS Big Sur 11.6.4

### Windows environment

_No response_

### CumulusCI installation method

Brew

### Error Gist

https://gist.github.com/vaumont/ed031a3ed123ceb2c89f600f0fce8d2b

### Additional information

_No response_

@vaumont  Thanks for the report.
As a workaround, please use code like this:
lat:
random_number:
min: -90
max: 90
long:
random_number:
min: -180
max: 180
This will only generate whole-number lats/longs, admittedly, but that is probably okay for most applications.

@vaumont:
Here is an easier workaround:
- object: Foo
  fields:
    lat: ${{fake.latitude | string}}
    long: ${{fake.longitude | string}}

Closing this bug as it is being tracked as https://github.com/SFDO-Tooling/Snowfakery/issues/630.


hls_playlist_length directive not honored ?!

I am trying to mimic the nDVR functionality of Wowza.by using the hls_playlist_length directive:
      application live {
            allow play all;
            live on;
            hls_playlist_length 4h;
            record_path /video_recordings;
            record_unique on;
            hls on;
            hls_nested on;
            hls_path /HLS/live;
            hls_continuous on;
            hls_cleanup off;
            hls_fragment 10s;

            hls_variant _source BANDWIDTH=4718592;
            hls_variant _high BANDWIDTH=1572864;
            hls_variant _low BANDWIDTH=819200;
          }

It looks that the specified directive is not honored in any way, because with 10s fragments a maximum of 12 fragments are kept in cache by the rtmp module.
Can anyone confirm or I am doing something wrong?
Thank you in advance!

The directive is only honoured globally :(