Updating dual editorStates not working.

My scenario: I have two draftjs editors. One is the core input, and the other is a helper input which wraps its input in an entity and adds it to the core input. The core input and the helper input work as intended when the helper only adds to the core input on blur. The problem arrises when I tried to get the helper input to update the core input live, (updating both editorStates on onChange of the helper input) I receive this error:
getUpdatedSelectionState.js:32 Uncaught TypeError: Cannot read property 'getIn' of undefined
  getUpdatedSelectionState @ getUpdatedSelectionState.js:32
  getDraftEditorSelectionWithNodes @ getDraftEditorSelectionWithNodes.js:35
  getDraftEditorSelection @ getDraftEditorSelection.js:33
  editOnSelect @ editOnSelect.js:26
  (anonymous function) @ DraftEditor.react.js:157
  ReactErrorUtils.invokeGuardedCallback @ ReactErrorUtils.js:70
  executeDispatch @ EventPluginUtils.js:87
  executeDispatchesInOrder @ EventPluginUtils.js:110
  executeDispatchesAndRelease @ EventPluginHub.js:42
  executeDispatchesAndReleaseTopLevel @ EventPluginHub.js:53
  forEachAccumulated @ forEachAccumulated.js:24
  processEventQueue @ EventPluginHub.js:215
  runEventQueueInBatch @ ReactEventEmitterMixin.js:18
  handleTopLevel @ ReactEventEmitterMixin.js:29
  handleTopLevelImpl @ ReactEventListener.js:73
  perform @ Transaction.js:136
  batchedUpdates @ ReactDefaultBatchingStrategy.js:63
  batchedUpdates @ ReactUpdates.js:98
  dispatchEvent @ ReactEventListener.js:150

I've boiled down my code as much as possible to find the perpetrator, and I've believe I've found the issue. Updating the editorState of the core Editor through the helper's editorState while the core Editor has readOnly={true} triggers the error. (And without readOnly={true}, the core editor constantly takes focus from the helper input.)
Here is the code to re-produce the error.
export default class DualInput extends Component {
  constructor(props) {
    super(props);
    this.setHelperState = this.setHelperState.bind(this);
    this.state = {
      readOnly: false,
      editorState: EditorState.createEmpty(),
      helperState: EditorState.createEmpty(),
    };
  }

  setHelperState(helperState) {
    this.setState({
      helperState,
      editorState: this.addCoreText(helperState.getCurrentContent().getPlainText('') || ''),
    });
  }

  addCoreText(text) {
    const editorState = this.state.editorState;
    const editorContent = Modifier.insertText(
      editorState.getCurrentContent(),
      editorState.getSelection(),
      text,
      null,
      null
    );

    return EditorState.push(editorState, editorContent, 'insert-characters');
  }

  render() {
    return (

 this.refs.core.focus()}>
          <Editor
            ref="core"
            editorState={this.state.editorState}
            readOnly={this.state.readOnly}
            onChange={(editorState) => this.setState({ editorState })}
          />

 this.refs.helper.focus()}>
          <Editor
            ref="helper"
            editorState={this.state.helperState}
            onChange={this.setHelperState}
          />

        <button onClick={() => this.setState({ readOnly: !this.state.readOnly })}>
          ReadOnly: {String(this.state.readOnly)}
        </button>

    );
  }
}

The above is a simplified version of my scenario, but it still illustrates the problem. Am I perhaps doing something wrong? Or is this a bug?
Thanks.

Thanks for the details! Admittedly, this is a little tricky for me to follow.
Based on the error, my hunch is that swapping in the new EditorState leads to some kind of mismatch between the DOM and the ContentState, but it's hard to say.
Any chance you could provide a live example for me to try this on and get a better sense?

Thanks for the reply!
Unfortunately I don't know too much about the DraftJS code base, so I won't be of too much help, but I wrote up this example quick this morning: http://schultzjust.in/DraftJs-DualEditor/ . Hopefully this helps illustrate the problem. All the important code is in the HTML file.

I am also seeing this issue. There is a timing window that exists from when a new editorState is passed into draftjs as a prop until that new state is rendered. If an onSelect event fires in this time, then the code will throw. getUpdatedSelectionState should detect this case and just return editorState.getSelection() in this scenario.

This is a really annoying bug. Spams console with errors, breaks focus. Any chance we could pull in the fix from textioHQ/draft-js?

I've just updated to DraftJS v0.10.0 and while I haven't been able to extensively test this issue, I think the problem has been fixed. Thanks @AnilRedshift!

@jussch I'm still seeing this

Reopening because folks are still encountering this issue.

Also encounter this under ssr

Currently It is happening in ssr. So I temporary fixed that by delegate rendering of Editor to client side.

I have same problem with rendering Editor with server-side. I am using next.js framework.

@Huvik I am using next.js too. I solved like this:
import {Editor} from 'draft-js';

class Page1 extends Component {
    constructor(props) {
        super(props);
        this.state = {
            editor: null
        };
   }
   componentDidMount() {
        this.setState({
            editor: withReduxForm(Editor),
        });
   }

  render() {
       return (

               {this.state.editor}

       )
  }
}
export default Page1;

I bind editor at client side using componentDidMount instead of ssr.

I got this when using draft inside block of other draft. How to fix this? Want to build table with draft in each cell

I would strongly recommend not attempting to implement a table in draft because it does not effectively deal with complex nested structures of that nature.

I'm also still seeing this

Just saw it now

I would strongly recommend not attempting to implement a table in draft because it does not effectively deal with complex nested structures of that nature.

I can confirm what @CameronAckermanSEL said - it's something we have not needed in the past at Facebook, and are just starting to look into. I don't expect we will focus on the editor-within-an-editor use case but I'd be happy to take PRs or examples of how to handle this use case better.
For implementing tables in Draft, there are folks working on this internally but it is still experimental. I expect it will take many months before we have anything ready to share in open source.

For anyone having trouble with issue, specifically editing a readOnly Editor and a live Editor at the same time, I found this trick solved it for me:
  componentWillReceiveProps(nextProps) {
    const { contentState } = nextProps;

    if (
      contentState &&
      this.props.contentState !== contentState &&
      this.state.editorState.getCurrentContent() !== contentState &&
      this.editorState.getCurrentContent() !== contentState
    ) {
      if (this.props.readOnly) {
        const decorator = new CompositeDecorator(mathDecorators);
        const editorState = EditorState.createWithContent(contentState, decorator);
        this.setState({ editorState });
        return;
      }

      this.setState({
        editorState: EditorState.push(
          this.state.editorState,
          contentState,
        ),
      });
    }
  }

Quick context here: I built a wrapper editor for DraftJS that edits the contentState rather than the editorState so that I'm not storing selection state in my Redux. However, to achieve this, the wrapper editor has to convert contentState to editorState to use DraftJS's editor.
So then, to explain my code, if the editor is readOnly, I avoid using EditorState.push(...) because it forces browser selection to happen. This causes a conflict if there is more than one editor editing the same contentState. Instead, I use EditorState.createWithContent(contentState, decorator) to generate my new editorState, which bypasses the conflicting selection modification.
Maybe this can help anyone running in to a similar issue.

I can confirm that I get this error when trying to render Editor server side. I followed @davidnguyen179  way:
class MyEditor extends React.Component {
  constructor(props) {
    super(props)

    this.state = {
      editorState: EditorState.createEmpty(),
      editor: null
    }

    this.onChange = editorState => this.setState({ editorState })
  }

  componentDidMount() {
    this.setState({ editor: Editor })
  }

  render() {
    const ClientEditor = this.state.editor

    return (

        {
          this.state.editor ?
            <ClientEditor
              editorState={this.state.editorState}
              onChange={this.onChange}
            /> :
            null
      }

    )
  }
}

export default MyEditor

This works fine even with SSR.
The thing is to render the Editor only when the component is mounted.

@jussch Could you please share the code of your wrapper editor? Really appreciate it! Have been struggling with this issue for a month now.

@flarnie Is there a ticket that we can follow to get update on table support in Draftjs? BTW, Slate JS can handle nested editor nicely. You guys may want to take a look at it. http://slatejs.org/

I don't know of an issue specifically for nested table support, go ahead and open one if you'd like. :)

https://github.com/facebook/draft-js/issues/600
https://github.com/facebook/draft-js/issues/220
https://github.com/facebook/draft-js/pull/388
There are more of these old issues and PRs that can be dug up from the backlog when searching for "Tables"
FWIW The author of #388 decided to move to Slate after the PR basically died.

Also, https://github.com/facebook/draft-js/pull/1084 has been opened for an eternity and resolves this problem. We had been using it for months before we migrated and it resolved this error.

@CameronAckermanSEL I know nested draft editor is tricky, but it looks like @bkniffler got it working and here is his demo site https://draft-wysiwyg.herokuapp.com/ I was trying to follow his thoughts with the latest draft, but got blocked by this issue due to editOnSelect got triggered for both editors even though one of them (the parent) is marked as readonly.

Hey @cupcoder, thanks for bringing up draft-wysiwyg. I've not been following the latest development of draft-js, but when I used it, nesting needed to be implemented in a very hacky way (focus management, events, drag&drop, etc don't work natively). There is no native tree-like data structure, but instead the document is flat.
I've found that using slate-js works much better in this specific case (nested editors with correct focus management, tree document structure). I've been able to use it to produce real nested, editable pages for a content-management system, with containers, accordions, etc.

@cupcoder I followed a similar approach and ran into the same problems that bkniffler listed above.
Here are some additional problems:

Keyboard navigation between the cells is made much more complicated by the nested editors in the cells.
The serialized document format explodes in size unless very precise serialization is done against the table cell contents (which are not just the cell but the entire document residing in the cell)
Because of the bug here, exactly one editor must be readonly=true in the cell because otherwise the editors will try to compete for focus and it's an overall nightmare.
The table performance quickly becomes abysmal as the number of cells increase.

Draft is simply not designed to support a table workflow, which is a shame because it's a very common structure in documentation.

+1 same issue

import React, { Component } from 'react'
import { actions } from 'mirrorx'

import Editor from 'draft-js-plugins-editor'
import createToolbarPlugin from 'draft-js-static-toolbar-plugin'
import 'draft-js-static-toolbar-plugin/lib/plugin.css'

import {
    EditorState,
    convertFromRaw,
    convertToRaw,
} from 'draft-js'

import styles from '../config/editor-styles'

const staticToolbarPlugin = createToolbarPlugin()
const { Toolbar } = staticToolbarPlugin
let plugins = [staticToolbarPlugin]

const handleChangeContent = (content) => {
    const contentState = content.getCurrentContent()
    const rawContent = convertToRaw(contentState)
    const currentContent = JSON.stringify(rawContent)

    actions.editor.setState(currentContent)
}

class MyEditor extends Component {
    constructor(props) {
        super(props)

        const { content, isReadOnly } = props

        if (isReadOnly) {
            plugins = []
        }

        const editorState = this._mountEditorState(content)

        this.state = {
            editorState,
        }

        this.onFocus = () => this.refs.editor.focus()

        this.onChange = (editorState) => {
            this.setState({ editorState })

            handleChangeContent(editorState)
        }
    }
    componentWillReceiveProps(nextProps) {
        const { content, isReadOnly } = nextProps
        if (content && isReadOnly) {
            const editorState = this._mountEditorState(content)

            this.setState({ editorState })
        }
    }
    _mountEditorState(content) {
        let editorState

        try {
            const rawContent = convertFromRaw(JSON.parse(content))
            editorState = EditorState.createWithContent(rawContent)
        } catch (err) {
            editorState = EditorState.createEmpty()
        }

        return editorState
    }
    render() {
        const { isReadOnly } = this.props
        const { editorState } = this.state

        return (

                { !editorState &&
Loading content...
 }

                    <Editor
                        readOnly={isReadOnly}
                        editorState={editorState}
                        onChange={this.onChange}
                        ref='editor'
                        spellCheck={true}
                        plugins={plugins}
                    />
                    { !isReadOnly && <Toolbar /> }

        )
    }
}

export default MyEditor

aha. I get rid of the error by using createFromBlockArray. Maybe you can try it.

And also I found that when I use onMouseDown to handle the sonEditor's focus event the error never appeared. It seems that the getIn function is fired after the focus Event and is bound with onClick. So we need to let the sonEditor focus before onClick event. I tried onMouseDown ant it worked.
Here is my code.
`
import React, { Component } from 'react'
import { Editor, EditorState, ContentState, Entity, convertFromRaw } from 'draft-js'
const emptyContentState = convertFromRaw({
entityMap: {},
blocks: [
{
text: '',
key: 'foo',
type: 'unstyled',
entityRanges: [],
},
],
})
export default class Caption extends Component {
constructor(props) {
super(props)
this.state = {
editMode: false,
editorState: EditorState.createWithContent(ContentState.createFromText(this._getValue()))
}
this.onChange = this._onChange.bind(this)
this._onClick = this._onClick.bind(this)
this._onBlur = this._onBlur.bind(this)
}
_onChange(editorState) {
	this.setState({
		editorState
	})
}

_onClick() {
	this.setState({
		editMode: true,
	}, () => {
		this.props.onStartEdit()
		this.caption.focus()
	})
}

_onBlur() {
	if (this.state.editMode) {
		this.setState({
			editMode: false
		}, () => {
			var entityKey = this.props.block.getEntityAt(0)
			this.props.contentState.mergeEntityData(entityKey, { caption: this.state.editorState.getCurrentContent().getPlainText() })
			this.props.onFinishEdit()
		})
	}
}

_getValue() {
	return this.props.contentState
		.getEntity(this.props.block.getEntityAt(0))
		.getData()['caption']
}

componentDidMount() {
	this._onClick()
}

render() {
	return (

			<Editor
				textAlignment="center"
				editorState={this.state.editorState}
				onChange={this.onChange}
				readOnly={!this.state.editMode}
				ref={(ref) => this.caption = ref}
				editorKey="caption"
			/>

	)
}

}
`

For those who use next-js, I've gotten around this issue by importing the Draft editor using next's dynamic import with ssr turned off:
const Editor = dynamic(import("draft-js").then(module => module.Editor), {
  ssr: false
});

This will obviously prevent server-side rendering 🙄

I would strongly recommend not attempting to implement a table in draft because it does not effectively deal with complex nested structures of that nature.

I can confirm what @CameronAckermanSEL said - it's something we have not needed in the past at Facebook, and are just starting to look into. I don't expect we will focus on the editor-within-an-editor use case but I'd be happy to take PRs or examples of how to handle this use case better.
For implementing tables in Draft, there are folks working on this internally but it is still experimental. I expect it will take many months before we have anything ready to share in open source.

so how do you quote a post created by a draftjs editor, if you cannot render an readonly editor within another editor ?


Update fishing quests

Last one:
{"api_no":292,"api_category":2,"api_type":1,"api_state":2,"api_title":"秋刀魚漁：もっと頑張ってもいいのよ！","api_detail":"かなり大変だけれど、秋刀魚漁、もっともっと頑張ってもいいのよ！","api_get_material":[800,300,300,300],"api_bonus_flag":1,"api_progress_flag":0,"api_invalid_flag":0}

https://github.com/gakada/KanColleViewer-Translations/blob/master/Quests.xml#L995
also, it seems like you don't need to collect 3 + 10 + 30, but just have 3, 10 and 30 of those things in the inventory for first, second and third quest respectively.

I already have 292 and yes, it's the same as all the older quests with inventory items.
Your clarification seems good, I'll look into it tomorrow.

Fixed in 54dddbece677d4573410793dd7070bb65394e912 (along with the addition of the new ranking reward).


How to use Developer Mode in Azure Application Insights for node.js

As I know, for c#, it can use this line of code: TelemetryConfiguration.Active.TelemetryChannel.DeveloperMode = true;
Then in node.js, how to use Developer Mode in Application Insights?
Thanks.

You can use setInternalLogging
const appInsights = require("applicationinsights");
appInsights.setup("<instrumentation_key>").setInternalLogging(true, true);
appInsights.start();

https://github.com/Microsoft/ApplicationInsights-node.js/blob/bf66555ad51f2e312866088cd1e006a73f04ee06/applicationinsights.ts#L243-L254


Patch fix urn clash

Rebased copy of https://github.com/pulumi/pulumi-digitalocean/pull/515.
I have checked with Platform, and they confirm that urn and id are reserved for their use.

Given the recent schema validation changes
a release would address a similar issue with get-schema, as in kubernetes:
https://github.com/pulumi/pulumi-kubernetes/issues/2683


Fantom header appears

In my RecyclerView I use 'notifyItemChanged(int position)' method to animate an image on an item when click on it. And when I click, above the item an excess header appears for a second and disappears.
This doesn't happen if the list has only one header or with the last header if they are many.
Step 1 (do nothing):

Step 2 (when I click on 'Attack 3'):

I'm having the same problem. It happens when I call notifyItemChanged(int position) during the animation.

same here

It has something to do with the default item animator and the fact that this lib doesn't take animations into account. I could not find the exact reason, but for me disabling the animations for notifyItemChanged worked.
In Kotlin this is:
(recyclerView.itemAnimator as SimpleItemAnimator).supportsChangeAnimations = false


Introspection Query does not work (exception)

Version and Environment Details
Environment:  Windows 10
IDE name and version: Intellij 2019.2
Plugin version:  2.2.0
Expected Behaviour
Schema should be created
Actual Behaviour
.graphqlconfig:

The introspection query returns the following result:
https://pastebin.com/0xY40rYp
And this is plugin exception:
https://pastebin.com/raw/MaZhted1
I'm not sure if I'm doing something wrong but GraphiQL works without any problems:

Hi Eugen.
Thanks for using the plugin.
The plugin uses graphql-java to build up a validated schema based on the introspection result. In this case the exception you attached indicates that the schema has a number of errors.
Specifically, there's a type called SchemaProxy which has a directives field with the output type __Directive. The exception states that The field type '__Directive' is not present when resolving type 'SchemaProxy, meaning that the schema is referring to an unknown type that isn't included in your schema.
The weird thing is that __Directive is part of the introspection types:  https://github.com/jimkyndemeyer/js-graphql-intellij-plugin/blob/master/resources/META-INF/graphql specification schema.graphql
I haven't encountered any other introspection results that tried to extend or make use of the introspection schema types before, so this is a first. Did you declare the SchemaProxy type yourself, or is it built-in to the GraphQL framework you're using?
As for a fix, the only way to get the schema to work with this plugin and graphql-java right now would be to not return the SchemaProxy type from your endpoint when it's introspected.
As for a more long term solution, I'm not sure whether introspection results are allowed to make use introspection types such as __Directive. This may require clarification in the GraphQL spec or a change in this plugin and/or graphql-java schema validation.
Best regards,
Jim.

I was using KGraphQL: https://kgraphql.in/
I decided to switch to graphql-java & https://github.com/ExpediaDotCom/graphql-kotlin which worked out in the end.


fix: display correct node version on download fail message

forgot to start docker, saw this message

Thanks!


Upgrade htmlparser2

The current version behaves unhelpfully when Parser.write(undefined) is called:
code
h = e => (...args) => console.log(e, { args });
p = new (require('htmlparser2').Parser)({
  onend:h('onend'),
  onopentag:h('onopentag'),
  ontext:h('ontext'),
  onclosetag:h('onclosetag'),
});
p.write(undefined);
p.end();

with current dependency (3.9.2)
ontext { args: [ 'undefined' ] }
onend { args: [] }

with latest version (9.1.0)
Uncaught TypeError: Cannot read properties of undefined (reading 'length')
    at Tokenizer.shouldContinue (htmlparser2-undefined/node_modules/htmlparser2/lib/Tokenizer.js:608:41)
    at Tokenizer.parse (htmlparser2-undefined/node_modules/htmlparser2/lib/Tokenizer.js:616:21)
    at Tokenizer.write (htmlparser2-undefined/node_modules/htmlparser2/lib/Tokenizer.js:150:14)
    at Parser.write (htmlparser2-undefined/node_modules/htmlparser2/lib/Parser.js:460:28)

I'm not opposed to updating htmlparser2. But I'm also having trouble thinking of how it could get passed undefined. Is there somewhere in Backend where that happens?

I doubt it happens deliberately.  It happened to me while modifying SQL generation for export streaming (unjoiner()?) and took a while to understand why things were failing and what exactly I'd broken.


License section needs updating

With recent changes to Azure MFA, this page needs to be updated to clarify whether the Office365 E1/E3/etc licenses now qualify for NPS MFA.

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

ID: a61a7e9a-44bf-7bdc-f810-87e2fd6792e1
Version Independent ID: 71d11105-b83e-952e-d107-1ff9a7c8d107
Content: Provide Azure MFA capabilities using NPS - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-mfa-nps-extension.md
Service: active-directory
Sub-service: authentication
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

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

Thanks for the feedback, @erichenson
This doc isn't specifically noting the different licensing tiers and SKUs, just that you need a valid license for Azure Multi-Factor Authentication, not per-user or per-device. We have an update lined up for the separate MFA licensing overview doc, but in general - the Office SKUs don't include Azure AD Premium licensing; Microsoft 365 or EMS start to bundle in Azure AD Premium P1 or P2 licenses.
#please-close


Extension causes high cpu load

Issue Type: Performance
Extension Name: python
Extension Version: 2019.2.5558
OS Version: Darwin x64 17.7.0
VSCode version: 1.32.3

:warning: Make sure to attach this file from your home-directory: /Users/leonardoalves/ms-python.python-unresponsive.cpuprofile.txt :warning:
Find more details here: https://github.com/Microsoft/vscode/wiki/Explain:-extension-causes-high-cpu-load

Please could you upload the cpuprofile.txt file.

Closing in favor of https://github.com/Microsoft/vscode-python/issues/4895


No retry logic on poll()/put() for both source/sink

Currently if Redis isn't available or there is an intermittent network issue both the Sink and Source connector throw errors such as
org.apache.kafka.connect.errors.ConnectException: Could not read messages from stream at com.redis.kafka.connect.source.StreamSourceRecordReader.poll(StreamSourceRecordReader.java:109) at com.redis.kafka.connect.source.RedisSourceTask.poll(RedisSourceTask.java:113) .... Caused by: io.lettuce.core.RedisCommandTimeoutException: Command timed out after 1 minute
Which completely stops the connectors and only a restart of the task will fix it.
For those more experienced with this connector, would setting redis.timeout in the connector config potentially fix this issue? Are there other settings I may have missed?
Does this connector need some retry logic wrapping the poll() and put() function calls (Source/Sink)?

Thanks for reporting this. I added some retry logic for both sink and source ee0ffd1


can't launch lens within ubuntu 18.04

Describe the bug
can't launch lens within ubuntu 18.04
To Reproduce
Steps to reproduce the behavior:

Go to '...'
have  install

snap list
Name          Version      Rev   Tracking       Publisher   Notes
core          16-<IP_ADDRESS>  9804  latest/stable  canonical✓  core
core18        20200724     x1    -              -           base
kontena-lens  3.5.3        x1    -              -           classic

Click on '....'
Scroll down to '....'
See error
not see any issue, but the UI can't see
cand  init 5
I can see the ico in the start menu.  but when click the lens ico,  it doesn't show the UI

Expected behavior
show the UI
and
Screenshots
If applicable, add screenshots to help explain your problem.

Environment (please complete the following information):

Lens Version:
OS: [e.g. OSX]
Installation method (e.g. snap or AppImage in Linux):

Logs:
When you run the application executable from command line you will see some logging output. Please paste them here:
Your logs go here...

Kubeconfig:
Quite often the problems are caused by malformed kubeconfig which the application tries to load. Please share your kubeconfig, remember to remove any secret and sensitive information.
your kubeconfig here

Additional context
Add any other context about the problem here.

I am unable to reproduce this, if this is still an issue. Please open a new issue.


Awesome Collaborators

The gitter chat to find people to collaborate with

There's nothing there...

Passion is there!


Issue with gstreamer plugin

Hello,
when i try to stream youtube links i get this error:
WARNING  Could not find a text/html decoder to handle media. WARNING  GStreamer warning: gst-stream-error-quark: No decoder available for type 'text/html'. (6) ERROR    GStreamer error: gst-core-error-quark: Your GStreamer installation is missing a plug-in. (12)
Shouldn't gstreamer plugins already be installed from mopidy's apt?
I run debian jessie and mopidy 2.0 on my machine

I have the same problem even though I installed the gstreamer0.10-plugins-bad package. Also running Raspbian jessie (lite) and mopdiy 2.0.
These are packages that are installed when installing 'gstreamer0.10-plugins-bad' on jessie:
The following NEW packages will be installed:
  freepats gstreamer0.10-plugins-bad gstreamer0.10-plugins-base libass5 libcdaudio1 libcroco3 libdc1394-22 libdca0
  libdirac-encoder0 libdirectfb-1.2-9 libenca0 libfaad2 libfftw3-double3 libflite1 libfribidi0 libgme0 libgsm1
  libgstreamer-plugins-bad0.10-0 libgstreamer-plugins-base0.10-0 libgstreamer0.10-0 libjpeg8 libkate1 libltdl7
  libmhash2 libmimic0 libmms0 libmodplug1 libmpcdec6 libofa0 libopenal-data libopenal1 libraptor2-0 librasqal3
  librdf0 librsvg2-2 librsvg2-common libschroedinger-1.0-0 libslv2-9 libsoundtouch0 libspandsp2 libvo-aacenc0
  libvo-amrwbenc0 libwildmidi-config libwildmidi1 libxslt1.1 libxvidcore4 libyajl2 libzbar0 libzvbi-common libzvbi0

Mopidy v2.0 uses gstreamer1.0 packages, not gtsreamer0.10.

Yeah just found that out and it works fine now. Got confused because the README says to use gstreamer0.10-plugins-bad.
Anyway, my issues have been fixed, hope it also works for @davoz96

Good spot, I've updated it in https://github.com/mopidy/mopidy-youtube/commit/d79bd5524a42210bc7acc9bf88275c9df62c73f7
If @davoz96 can confirm the solution we can close this.

@kingosticks I just installed the package according to d79bd55 but I still get the same error.
Any ideas?

After a lot of testing the issue is related to https://github.com/mopidy/mopidy/issues/1526
This issue can be closed.

I have the same issue however I don't think it is related to mopidy/mopidy#1526. I have tried setting the timeout and nothing changes.
These are the GStreamer related packages installed

root@raspberrypi1:~# dpkg --list | grep GStreamer
ii  gir1.2-gst-plugins-base-1.0           1.4.4-2                                   armhf        Description: GObject introspection data for the GStreamer Plugins Base library
ii  gir1.2-gstreamer-1.0                  1.4.4-2                                   armhf        Description: GObject introspection data for the GStreamer library
ii  gstreamer0.10-alsa:armhf              0.10.36-2                                 armhf        GStreamer plugin for ALSA
ii  gstreamer1.0-alsa:armhf               1.4.4-2                                   armhf        GStreamer plugin for ALSA
ii  gstreamer1.0-libav:armhf              1.4.4-2                                   armhf        libav plugin for GStreamer
ii  gstreamer1.0-omx                      <IP_ADDRESS>-0+rpi18rpi1g                      armhf        GStreamer OpenMAX plugins
ii  gstreamer1.0-plugins-bad:armhf        1.4.4-2.1rpi4rpi1g                        armhf        GStreamer plugins from the "bad" set
ii  gstreamer1.0-plugins-base:armhf       1.4.4-2                                   armhf        GStreamer plugins from the "base" set
ii  gstreamer1.0-plugins-good:armhf       1.4.4-2                                   armhf        GStreamer plugins from the "good" set
ii  gstreamer1.0-plugins-ugly:armhf       1.4.4-2                                   armhf        GStreamer plugins from the "ugly" set
ii  gstreamer1.0-pulseaudio:armhf         1.4.4-2                                   armhf        GStreamer plugin for PulseAudio
ii  gstreamer1.0-tools                    1.4.4-2                                   armhf        Tools for use with GStreamer
ii  gstreamer1.0-x:armhf                  1.4.4-2                                   armhf        GStreamer plugins for X11 and Pango
rc  libgstreamer-plugins-bad0.10-0:armhf  0.10.23-7.4                               armhf        GStreamer development files for libraries from the "bad" set
ii  libgstreamer-plugins-bad1.0-0:armhf   1.4.4-2.1rpi4rpi1g                        armhf        GStreamer development files for libraries from the "bad" set
ii  libgstreamer-plugins-base0.10-0:armhf 0.10.36-2                                 armhf        GStreamer libraries from the "base" set
ii  libgstreamer-plugins-base1.0-0:armhf  1.4.4-2                                   armhf        GStreamer libraries from the "base" set
ii  libgstreamer0.10-0:armhf              0.10.36-1.5                               armhf        Core GStreamer libraries and elements
ii  libgstreamer1.0-0:armhf               1.4.4-2                                   armhf        Core GStreamer libraries and elements
ii  python-gst-1.0                        1.2.1-1.1                                 armhf        GStreamer GObject Introspection overrides for Python

Could the issue be caused by "gstreamer0.10-alsa" still being installed? We use LXDE to run chromium in kiosk mode and it appears LXDE requires "gstreamer0.10-alsa"


Network requests tracking for 3rd parties

Is your feature request related to a problem? Please describe.
I would like Datadog to be able to automatically track the network requests for every third party libraries I have in my apps.
For some libraries, it's possible to give a custom Okhttp instance and add the required interceptors but not all libraries allow this.
This is particularly useful to get alerts when a 3rd party service is down.
Describe the solution you'd like
Ideally, something like what Firebase Performances propose:
https://firebase.google.com/docs/perf-mon/network-traces?platform=android&hl=en
Network requests are automatically tracked by just applying the plugin.
It also works on third-party libraries.

Hello @fleficher!
Indeed, we currently require manual setup for the particular OkHttpClient instance.
What Firebase Performance is doing is achieved by the bytecode injection done by the com.google.firebase:perf-plugin, which scans the code being compiled and modifies it to inject the necessary instrumentation.
We currently don't have such functionality, but we may look into it in the future.
Since it may be a big feature request for us, can I please ask you to also open a Feature Request by contacting our support and referencing this issue as well. This will allow us to have a better tracking of it.


Normative: Make ToDurationRecord same order as ToTemporalPartialDurationRecord

Currently, ToDurationRecord access the object with the following order:
"years", "months", "weeks", "days", "hours", "minutes", "seconds", "milliseconds", "microseconds", "nanoseconds"
This is different from ToTemporalPartialDurationRecord which follow the order of:
"days", "hours", "microseconds", "milliseconds", "minutes", "months", "nanoseconds", "seconds", "weeks", "years"
This PR change the accessing order of ToDurationRecord to be the same as ToTemporalPartialDurationRecord without impacting the math and order of other 4 AOs.
Please compare
https://tc39.es/proposal-temporal/#sec-temporal-duration-records
https://tc39.es/proposal-temporal/#sec-temporal-totemporalpartialdurationrecord
against
https://tc39.es/proposal-intl-duration-format/#table-duration-components
https://tc39.es/proposal-intl-duration-format/#sec-todurationrecord
Notice three other AOs in Intl.DurationFormat CANNOT change the order to keep the math
DurationRecordSign / PartitionDurationFormatPattern and  Intl.DurationFormat

ToDurationRecord should probably just be ToTemporalDurationRecord at some point, but this alignment is definitely an improvement.

When I first prototype, I call into my implementation of ToTemporalDurationRecord in my v8 code. But it actually are not compatable and break the test
https://github.com/tc39/test262/blob/main/test/intl402/DurationFormat/prototype/format/invalid-arguments-throws.js
somehow the test (based on https://tc39.es/proposal-intl-duration-format/#sec-todurationrecord ) will throw RangeError on those if I reuse the code of ToTemporalDurationRecord because they are not Object and after the ToString convert them to string and pass to the parser the do not meet the grammar. But the spec is mandating us to throw TypeError now, which is closer to ToTemporalPartialDurationRecord instead of ToTemporalDurationRecord. We should either change the spec to make it closer to ToTemporalDurationRecord now or decide we should make it closer to ToTemporalPartialDurationRecord instead.

Discussion: https://github.com/tc39/ecma402/blob/master/meetings/notes-2022-09-01.md#normative-make-todurationrecord-same-order-as-totemporalpartialdurationrecord-117
Conclusion: Discuss with Temporal first so we don't duplicate work.

@ptomato @gibson042  please help to resolve the need of this PR.

This looks in line with https://github.com/tc39/proposal-temporal/issues/2254 except that we want to enforce the alphabetical order by sorting the fields, rather than having it be implicit due to the table order. However, we have not yet made that change in Temporal so it seems fine to make this change now to bring the operation in line with Temporal, and later make a change to sort the fields.
Is there any other input you need from me?

Is there any other input you need from me?
@ptomato

yes, this PR is currently blocked by the comment in https://github.com/tc39/ecma402/blob/master/meetings/notes-2022-09-01.md#normative-make-todurationrecord-same-order-as-totemporalpartialdurationrecord-117
"SFC: If we convince Temporal to switch, we won’t need a second table. We should talk to the champions in order to see if we can make them change the order."
SFC's "to switch" means "to switch [to sort from the order in the Table https://tc39.es/proposal-temporal/#table-temporal-duration-record-fields   to the order in the Table  https://tc39.es/proposal-intl-duration-format/#table-duration-components "
My undrstanding is Temporal champions are not not willing to do such switch so I propose this PR. But SFC believe it is still possible someone (surely not me, but I have no idea who that will be) could still possible "convince Temporal to switch" threfore "won’t need a second table." So he want to "talk to the champions in order to see if we can make them change the order."

Sorry, I thought https://github.com/tc39/proposal-temporal/issues/2286#issuecomment-1248687027 and https://github.com/tc39/proposal-temporal/issues/2254#issuecomment-1248693340 already gave a clear answer on that. The answer is no, we are not going to read the properties in magnitude order for Duration. They should be read in alphabetical order. Shane was present at that discussion and agreed with the conclusion.
However, once we make the change to alphabetize the order explicitly instead of depending implicitly on the table order, we can order the rows of the table any way that is convenient for DurationFormat.

Superceded by #129


[WIP] Fix: Update default globbing pattern for collection matching

This update:

restricts the glob matching for molecule test
updates the default value for MOLECULE_GLOB="**/molecule/*/molecule.yml"

This allows:

users to call molecule from the collection folder and molecule will use the collection name in the ephemeral cache, .cache/molecule/COLLECTION_NAME.

WARNING:
If working within the ANSIBLE_COLLECTIONS_PATH directory, e.g. ~/.ansible/collections/ansible_collections/COLLECTION_NAME (default), using the new default setting MOLECULE_GLOB="**/molecule/*/molecule.yml" will globber clobber and delete the collection folder. This occurs in the presence of a galaxy.yml - as molecule calls ansible_compat.runtime(isolate=False) and will run ansible_compat.runtime.prepare_environment. However, if you delete the galaxy.yml OR chdir into COLLECTION_NAME/extensions and it will not delete the collection folder.
Currently (v24.2.1):
Calling molecule list will find all molecule.yml files within a collection folder that use a nested dir structure, i.e
COLLECTION_NAME/extensions/molecule/SCENARIO_NAME/molecule.yml. As it calls base.get_configs and sets the argument glob=str"**/molecule/*/molecule.yml".
This does not occur with molecule test, as it calls to base.execute_cmdline_scenarios and uses
MOLECULE_GLOB = os.environ.get("MOLECULE_GLOB", "molecule/*/molecule.yml").
Attempting to set an env var MOLECULE_GLOB="**/molecule/*/molecule.yml" will fail, as test attempts to insert the scenario name into every wild-card. But, this fuzzy matching acts as a safe guard against deleting the collection when working within the same ANSIBLE_COLLECTIONS_PATH directory:
$ MOLECULE_GLOB="**/molecule/*/molecule.yml" molecule test
CRITICAL 'defaultdefault/molecule/default/molecule.yml' glob failed.

So I was curious why molecule list is collection aware and all the other commands are not. The current work-around for making molecule collection aware is to use one of the following approaches:
$ MOLECULE_GLOB="./extensions/molecule/*/molecule.yml" molecule test
$ MOLECULE_GLOB="$(pwd)/extensions/molecule/*/molecule.yml" molecule test

This fix, partially addresses #4000 and #4117.

Is compatiable with PR #4160.

Yet, it easily exposes one of the current bugs within molecule (#4061) if end-users are working within the same directory as ANSIBLE_COLLECTIONS_PATH.


fix: native wallet label

Description

Notice

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

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

Risk

Testing

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #2497   👈

PR #2498

PR #2499

This comment was auto-generated by Graphite.

tested verified working


Implement diagnostic and code fix for SA1101 PrefixLocalCallsWithThis

Work In Progress (tests not implemented)
Fixes #35
The number of test cases is large to say the least. Basically I need to choose one item from each of the categories below and build a test for that set of conditions.

Element type

Type
Method
Property
Field
Event
Parameter
Local

Inheritance

New element
Override
Interface implementation
Explicit interface implementation
Hides static base element
Hides instance base element

Is Static

True
False

Expression type

Assignment to element
Read from (simple, e.g. var x = this.Property;)

For methods this is a reference to the method group

Read from (qualified, e.g. var x = this.Property.GetHashCode();)
nameof expression

Expression location

Attribute value
In static member
In instance member
Field initializer for static field
Field initializer for instance field
String interpolation
Documentation comment

Receiver

Current instance (or the containing type if Is Static is true)
Another instance

This gives a maximum of 4704 tests for complete coverage, although some combinations do not make since (e.g. you cannot assign to an element within a documentation comment or nameof expression).

I think the only reasonable way to do the testing for this is create a helper method that takes an enum value representing each of the above category and builds a test for that case.

nameof poses a remarkably bizarre set of challenges for this feature.

The nameof expression can be qualified with this when the target of the expression is an instance member, as long as you are in a context where this is available.
If the target of the expression is a method group, you can use this even if none of the methods in the group are instance methods.
The expression can always be qualified with the name of the containing class, regardless of whether or not the target is an instance member.

Since I really want to get this feature in a release, I'm implemented initial testing on a variety of edge cases in the above list.

I'm merging this one now that tests are in place so it gets included in the Alpha 2 release.


.Net: Improve DI support for Memory

The Memory feature of SK uses a MemoryBuilder in a similar style to the KernelBuilder, but unlike the KernelBuilder it doesn't internally use an IServiceCollection, meaning it's harder to integrate with dependency injection and by extension .NET Aspire.
It would be nice if the ISemanticTextMemory and IMemoryStore implementations all supported DI in a similar fashion to working with the IKernel, with a method like the AddSemanticKernel and AddAzureOpenAITextEmbedding APIs.

Hi @aaronpowell thanks for the feedback. We will be updating the memory connector packages and associated abstractions over the coming sprints. What you are requesting makes sense so we'll consider this as part of that work.

Closing as a dupe of #5278


Setting nested-array values in Container and Collection

(FYI, this is @designermonkey on another account, sorry)
Because of the way a project I am on is using environment config loading (.env), I have had to make minor modifications to the settings for Slim.
The test below highlights the change:
    /**
     * Test settings can be edited
     */
    public function testSettingsCanBeEdited()
    {
        $this->assertSame('1.1', $this->container->get('settings')['slim']['http_version']);

        $this->container->get('settings')['slim']['http_version'] = '1.2';
        $this->assertSame('1.2', $this->container->get('settings')['slim']['http_version']);
    }

As you can see, there is an extra level to isolate Slim settings from others within the app. Because of this change, the test fails with:

Indirect modification of overloaded element of Slim\Collection has no effect

This is due to the weird and wonderful way that Collection accesses it's values internally.
Many moons ago (at least two years if I remember right) we had this issue before, long before the introduction of the current container and collection structure, and I'm sure we solved it back then.
Can anyone think of how we could tackle this issue?
Irrespective of my alterations which have only highlighted the issue, we have prevented any way of having alterations made to nested settings without a horrible amount of extra code for the developer in getting from the container, getting from the collection, setting into the collection and then setting into the container.

For reference, the changes are here.
There are 2 failing tests due to this change, both attempting to modify the settings.

Of course, to fix the test I mentioned:
    /**
     * Test settings can be edited
     */
    public function testSettingsCanBeEdited()
    {
        $this->assertSame('1.1', $this->container->get('settings')['slim']['http_version']);

        $settings = $this->container->get('settings');

        $slim = $settings['slim'];
        $slim['http_version'] = '1.2';
        $settings['slim'] = $slim;

        $this->assertSame('1.2', $this->container->get('settings')['slim']['http_version']);
    }

But this goes against the simple approach requested by the test, I suppose.

@johnrobertporter You can pass in ENV settings into the container before you create slim. This allows you to override the settings.
https://github.com/slimphp/Slim/blob/3.x/Slim/Container.php#L95-L97

@geggleto this doesn't deal with the crux of the issue though.

Indirect modification of overloaded element of Slim\Collection has no effect

You can't modify nested values. This is something we need to fix, not make people code around.
I hope with the invent of Slim4, we find a better way of handling this.

John :D It's a pimple problem. In 4.x we aren't even shipping with a container!

I would be willing to entertain a new method on Slim's container that is a short-cut to replace a value though.

If it's been sidestepped in 4, no problem.


go-init permission denied when running with podman

Hi,
I'm having a problem building the go-init slave-base. The error is shown below;
podman build -t go-init-example .
STEP 1: FROM openshift/origin-release:golang-1.12 AS builder
STEP 2: COPY ./ /go/src/go-init
c9732b81e04f593170977d4930ca27fc709dc839db835517aab385fa87947a66
STEP 3: RUN go install go-init
can't load package: package go-init: open /go/src/go-init: permission denied
Error: error building at STEP "RUN go install go-init": error while running runtime: exit status 1
I have not edit the docker file, and I am running from master. ( I also tried the release-4.7 branch, same issue)
I'm running it on Rhel 7 with podman:
podman version
Version:            1.4.4
RemoteAPI Version:  1
Go Version:         go1.10.3
OS/Arch:            linux/amd64
Is there something I might be missing here? Thanks in advance.

Hi @AbdulR3hman ,
are you running the image form master?
if yes, I suspect that a recent security fix that we added may be not working with podman on rhel7.
Can you confirm ?


Start-PSBootstrap fails on PowerShell 5.1

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Start-PSBootstrap fails on PowerShell 5.1 due to use of unsupported type [System.Management.Automation.SemanticVersion] in build.psm1 therefore it is not possible to build latest versions of PowerShell 7 on PowerShell 5.1 anymore.
Expected behavior
*Start-PSBootstrap* should work on PowerShell 5.1

Actual behavior
*Start-PSBootstrap* throws error on PowerShell 5.1

Error details
Import-Module .\build.psm1
Start-PSBootstrap
Installing PowerShell build dependencies
VERBOSE: Calling Find-Dotnet from Start-PSBootstrap
VERBOSE: In Find-DotNet
ForEach-Object : Unable to find type [System.Management.Automation.SemanticVersion].
At D:\DEVELOPMENT\PowerShellCore\build.psm1:2106 char:81
+ ... *\.\d*)?' | ForEach-Object { [System.Management.Automation.SemanticVe ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (System.Manageme...SemanticVersion:TypeName) [ForEach-Object], RuntimeException
    + FullyQualifiedErrorId : TypeNotFound,Microsoft.PowerShell.Commands.ForEachObjectCommand

$error[0]|fl * -force

PSMessageDetails      :
Exception             : System.Management.Automation.RuntimeException: Unable to find type [System.Management.Automation.SemanticVersion].
                           at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext, Exception exception)
                           at System.Management.Automation.Interpreter.ActionCallInstruction`2.Run(InterpretedFrame frame)
                           at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
                           at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
                           at System.Management.Automation.Interpreter.Interpreter.Run(InterpretedFrame frame)
                           at System.Management.Automation.Interpreter.LightLambda.RunVoid1[T0](T0 arg0)
                           at System.Management.Automation.ScriptBlock.InvokeWithPipeImpl(ScriptBlockClauseToInvoke clauseToInvoke, Boolean createLocalScope, Dictionary`2 functionsToDefine, List`1
                        variablesToDefine, ErrorHandlingBehavior errorHandlingBehavior, Object dollarUnder, Object input, Object scriptThis, Pipe outputPipe, InvocationInfo invocationInfo, Object[] args)
                           at System.Management.Automation.ScriptBlock.<>c__DisplayClass57_0.<InvokeWithPipe>b__0()
                           at System.Management.Automation.Runspaces.RunspaceBase.RunActionIfNoRunningPipelinesWithThreadCheck(Action action)
                           at System.Management.Automation.ScriptBlock.InvokeWithPipe(Boolean useLocalScope, ErrorHandlingBehavior errorHandlingBehavior, Object dollarUnder, Object input, Object scriptThis,
                        Pipe outputPipe, InvocationInfo invocationInfo, Boolean propagateAllExceptionsToTop, List`1 variablesToDefine, Dictionary`2 functionsToDefine, Object[] args)
                           at System.Management.Automation.ScriptBlock.InvokeUsingCmdlet(Cmdlet contextCmdlet, Boolean useLocalScope, ErrorHandlingBehavior errorHandlingBehavior, Object dollarUnder, Object
                        input, Object scriptThis, Object[] args)
                           at Microsoft.PowerShell.Commands.ForEachObjectCommand.ProcessRecord()
                           at System.Management.Automation.CommandProcessor.ProcessRecord()
TargetObject          : System.Management.Automation.SemanticVersion
CategoryInfo          : InvalidOperation: (System.Manageme...SemanticVersion:TypeName) [], RuntimeException
FullyQualifiedErrorId : TypeNotFound
ErrorDetails          :
InvocationInfo        : System.Management.Automation.InvocationInfo
ScriptStackTrace      : at <ScriptBlock>, D:\DEVELOPMENT\PowerShellCore\build.psm1: line 2106
                        at <ScriptBlock>, D:\DEVELOPMENT\PowerShellCore\build.psm1: line 2106
                        at script:Start-NativeExecution, D:\DEVELOPMENT\PowerShellCore\tools\buildCommon\startNativeExecution.ps1: line 22
                        at Get-LatestInstalledSDK, D:\DEVELOPMENT\PowerShellCore\build.psm1: line 2105
                        at Find-Dotnet, D:\DEVELOPMENT\PowerShellCore\build.psm1: line 2259
                        at Start-PSBootstrap, D:\DEVELOPMENT\PowerShellCore\build.psm1: line 2054
                        at <ScriptBlock>, <No file>: line 1
PipelineIterationInfo : {}

Environment data
Name                           Value
----                           -----
PSVersion                      5.1.19041.1237
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.19041.1237
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Visuals

Personally, I think the minimum for development should be the last LTS.  5.1 is just not viable feature wise.  rewriting this functionality in 5.1 is a duplication of effort.    I think the other option is for start-psbootstrap to tell you, move to PowerShell 7 or bootstrap manually (possibly only that portion).  But I just wrote something that will be required and requires the same semanticVersion sorting which didn't exist in 5.1.
Marking for maintainer review.

It was too unexpected to find that the module did not work. I personally do not ready to switch completely to pwsh in my work (there are still many places where powershell has no alternative to pwsh).

adding @SteveL-MSFT

@PowerShell/powershell-maintainers have a consensus that PowerShell 7 is stable enough to be used for development of PowerShell. It would require a lot of effort to ensure Windows PowerShell 5.1 compatibility, which we think is unnecessary. We propose to add #requires -version 7 to build.psm1.


clang-format: disable bin packing

clang-format: disable bin packing
Bin packing often makes code hard to read. Disable it entirely.
Test plan:

CI

Codecov Report

Merging #402 (7902994) into main (c207972) will decrease coverage by 0.03%.
The diff coverage is 59.12%.

@@            Coverage Diff             @@
##             main     #402      +/-   ##
==========================================
- Coverage   68.96%   68.94%   -0.03%
==========================================
  Files         119      119
  Lines       11603    11639      +36
  Branches     1922     1922
==========================================
+ Hits         8002     8024      +22
- Misses       2623     2636      +13
- Partials      978      979       +1

Files
Coverage Δ

oi/CodeGen.cpp
70.94% <100.00%> (-0.16%)
:arrow_down:

oi/FuncGen.cpp
77.31% <100.00%> (-3.10%)
:arrow_down:

oi/Metrics.cpp
61.70% <100.00%> (+0.83%)
:arrow_up:

oi/OICompiler.cpp
81.46% <100.00%> (+0.13%)
:arrow_up:

oi/OILibraryImpl.cpp
61.70% <100.00%> (+0.27%)
:arrow_up:

oi/OIParser.h
88.88% <100.00%> (ø)

oi/PaddingHunter.cpp
86.11% <100.00%> (ø)

oi/TreeBuilder.cpp
78.65% <100.00%> (+0.08%)
:arrow_up:

oi/exporters/Json.cpp
85.33% <100.00%> (ø)

oi/exporters/test/TypeCheckingWalkerTest.cpp
95.65% <100.00%> (ø)

... and 28 more

:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


ADMX_Power/PW_PromptPasswordOnResume is incorrectly not applying due to a licensing rejection under Windows 11 Business

ADMX_Power/PW_PromptPasswordOnResume is incorrectly not applying due to a licensing rejection under Windows 11 Business:
MDM PolicyManager: Policy is rejected by licensing, Policy: (PW_PromptPasswordOnResume), Area: (ADMX_Power), Result:(0x82B00006) Unknown Win32 Error code: 0x82b00006.

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

ID: 776a0510-217f-8fc3-0a3d-53f47390acd8
Version Independent ID: cbcd7c1b-665a-df96-c191-9f63b27a9ec7
Content: Policy CSP - ADMX_Power - Windows Client Management
Content Source: windows/client-management/mdm/policy-csp-admx-power.md
Product: w10
Technology: windows
GitHub Login: @vinaypamnani-msft
Microsoft Alias: vinpa

Hey @pcamh, we're in the process of updating the CSP docs using a new format which will make this issue obsolete as the new format doesn't include Business applicability. As we continue, we'll work further on including the Business edition applicability appropriately for all CSPs, so I'm closing this issue for the time being.


add support for local origin errors in outlier detection

Envoy supports splitting of local origin/remote errors while deciding a host to be outlier.  https://www.envoyproxy.io/docs/envoy/latest/api-v3/config/cluster/v3/outlier_detection.proto#config-cluster-v3-outlierdetection.
This is useful in certain cases where we want to just rely on local failures to remove from load balancer pool.
The feature request is to add to Destination rule API.
split_external_local_origin_errors
consecutive_local_origin_failure
enforcing_consecutive_local_origin_failure

@nrjpoddar @howardjohn @hzxuzhonghu what do you think??

I looked at possible options of doing this via Envoy filter. But it is. becoming challenging to add to every client namespace especially when the services that need this behaviour are growing.
I think this would be useful API -  https://github.com/istio/api/pull/1989

This is useful in certain cases where we want to just rely on local failures to remove from load balancer pool.

Not sure i understand the case here, i think both side failures should be relied on by load balancer.

Not sure i understand the case here, i think both side failures should be relied on by load balancer.

This happens when server some times returns 5xx explicitly for few requests based on request but is generally up and processing requests. We do not want to eject that server because of that. We want to eject only if it is unreachable. This is to address the case explained here https://github.com/envoyproxy/envoy/issues/3643#issue-332919412

Seems reasonable to add though I wouldn't want to mimic Envoy APIs blindly.


Can't add author identifiers in the profile page

It's not clear how to add author identifiers.  Either they aren't saving correctly, or the values provided in testing are invalid but there's no validation/error messages to show why they are invalid.
005.zip

Closing as resolved


connection timeout by @Jupeter

New pull request to check travis build
See #104

Looks OK :+1:

What happened to this feature ?!

@bonswouar What do you mean?

@nunomaduro I don't see any timeout parameter on master branch, did I miss something ?


Improve history notice visibility on document page

Reduce the length of the copy and make the text larger.
Most users of these pages are specialists, rather than mainstream
users, and need less sign-positing that the content is from a
previous government. We think they'll be more interested in
the particular government.
This allows us to reduce the copy significantly, giving us space
to increase the font-size to match the page title. This should
be more impactful and more visible.
It also breaks the sentence on the government name, meaning the
second line will always lead with the XXXX to YYYY date part of
the government, which we think will be more visible and more
easily scanned, hopefully reducing banner-blindness and making
the history notice more visible overall.
Note: we're only breaking the government name on larger screen
sizes, to avoid orphan words and taking up too much space on
a smaller screen

:eyes:
Before
After

:iphone:

:computer:

![desktop-new] (https://cloud.githubusercontent.com/assets/63201/6853311/70fce11e-d3e2-11e4-9ccb-0680d1ea7b58.png)

:+1:

Love the line breaking.


Cannot inject sitekey or secret key from controller for multiple host

Hi,
In my project I having multiple sitekey/secretkey pair for multiple hosts and I am not able to configure the keys from my controller

hi,
This is not supported, but I am pretty sure you can add multiple host name to the same sitekey

Thanks for your reply Paul.
The problem for me is that an admin can create a new host and while creating a new host they wanted a new site key/secret key for security reasons.
I created my own custom method to validate the keys.
public async Task ValidateRecaptchaAsync(string secretKey)
{
string recaptchaResponse = this.Request.Form["g-recaptcha-response"];
        if (string.IsNullOrWhiteSpace(recaptchaResponse))
        {
            this.ModelState.AddModelError("captcha", "Please confirm that you are not a robot.");
            return false;
        }

        var httpClient = new HttpClient();
        var httpResponse = await httpClient.GetAsync($"{RecaptchaRequestUrl}?secret={secretKey}&response={recaptchaResponse}");
        if (httpResponse.StatusCode != HttpStatusCode.OK)
        {
            this.ModelState.AddModelError("captcha", "The reCAPTCHA is not valid.");
            return false;
        }

        var jsonResult = httpResponse.Content.ReadAsStringAsync().Result;
        dynamic jsonData = JObject.Parse(jsonResult);
        if (jsonData.success != "true")
        {
            this.ModelState.AddModelError("captcha", "The ReCaptcha is not valid.");
            return false;
        }

        return true;
    }


Implemented websocket server bind on unix-domain sockets

Implements UNIX-domain socket support as requested in #1662

Merged. Thanks a lot for submitting this! I'm sure it will be very useful going forward!


[Bug] [Relax]  Expect a not null value of IRModule

Actual behavior
Traceback (most recent call last):
  File "test.py", line 22, in <module>
    mod = tvm.relax.transform.LegalizeOps()(mod)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/software/tvm-lunder/python/tvm/ir/transform.py", line 238, in __call__
    return _ffi_transform_api.RunPass(self, mod)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/software/tvm-lunder/python/tvm/_ffi/_ctypes/packed_func.py", line 240, in __call__
    raise_last_ffi_error()
  File "/software/tvm-lunder/python/tvm/_ffi/base.py", line 481, in raise_last_ffi_error
    raise py_err
tvm._ffi.base.TVMError: Traceback (most recent call last):
  1: tvm::runtime::PackedFuncObj::Extractor<tvm::runtime::PackedFuncSubObj<tvm::runtime::TypedPackedFunc<tvm::IRModule (tvm::transform::Pass, tvm::IRModule)>::AssignTypedLambda<tvm::transform::{lambda(tvm::transform::Pass, tvm::IRModule)#7}>(tvm::transform::{lambda(tvm::transform::Pass, tvm::IRModule)#7}, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >)::{lambda(tvm::runtime::TVMArgs const&, tvm::runtime::TVMRetValue*)#1}> >::Call(tvm::runtime::PackedFuncObj const*, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, tvm::runtime::TVMRetValue)
  0: tvm::runtime::TVMMovableArgValueWithContext_::operator tvm::IRModule<tvm::IRModule>() const
  2: tvm::runtime::PackedFuncObj::Extractor<tvm::runtime::PackedFuncSubObj<tvm::runtime::TypedPackedFunc<tvm::IRModule (tvm::transform::Pass, tvm::IRModule)>::AssignTypedLambda<tvm::transform::{lambda(tvm::transform::Pass, tvm::IRModule)#7}>(tvm::transform::{lambda(tvm::transform::Pass, tvm::IRModule)#7}, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >)::{lambda(tvm::runtime::TVMArgs const&, tvm::runtime::TVMRetValue*)#1}> >::Call(tvm::runtime::PackedFuncObj const*, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, tvm::runtime::TVMRetValue)
  1: tvm::runtime::TVMMovableArgValueWithContext_::operator tvm::IRModule<tvm::IRModule>() const
  0: tvm::IRModule tvm::runtime::TVMPODValue_::AsObjectRef<tvm::IRModule>() const
  File "/software/tvm-lunder/include/tvm/runtime/packed_func.h", line 785
TVMError: In function transform.RunPass(0: transform.Pass, 1: IRModule) -> IRModule: error while converting argument 1: [21:44:09] /software/tvm-lunder/include/tvm/runtime/packed_func.h:2022: Check failed: (TObjectRef::_type_is_nullable) is false: Expect a not null value of IRModule

Steps to reproduce
import tvm
from tvm import relax

from tvm.script import ir as I
from tvm.script import tir as T
from tvm.script import relax as R

@I.ir_module
class Module:
    @R.function
    def main(v1_0: R.Tensor((1,), dtype="float16"), v6_0: R.Tensor((1, 40, 25, 1), dtype="float16")) -> R.Tensor((), dtype="float16"):
        with R.dataflow():
            lv: R.Tensor((), dtype="float16") = R.sum(v6_0, axis=None, keepdims=False)
            R.output(lv)
        return lv

mod = Module
mod = tvm.relax.transform.DeadCodeElimination()(mod)
mod = mod.show()
mod = tvm.relax.transform.LegalizeOps()(mod)

cc @Lunderberg @junrushao

mod = mod.show()

This line looks like a bug in the test case.  Displaying a module is not a transform, and so mod.show() returns None.  This then correctly errors out by detecting the nullptr in the IRModule.
(The error message may be improved by making IRModule be a non-nullable type, but a lot of the earlier portions of TVM rely on ObjectRef sub-types being nullable, so it would be a non-trivial change.  Would need to first find all instances IRModule that should be replaced by Optional<IRModule>.)

@Lunderberg Haha, I'm so sorry for misusing the API. Since I did not understand the error message, I submitted it directly.
Indeed,  improving the error message will be better. If you are willing to improve the error message, I'll leave this issue open. Otherwise, you can feel free to close this issue directly. Thanks!

Looking closer, the change that I was thinking of already occurs (here), and IRModule is non-nullable.  So, it already indicates (1) that the function signature is (0: transform.Pass, 1: IRModule), (2) that there is a problem with converting argument 1, and (3) that the problem is that it was Expect a not null value of IRModule.
So, the error message looks like it already gives all of the information it could.  Maybe it could be improved by returning a TypeError instead of a TVMError, but that would be about the extent of it.

@Lunderberg Thanks for your deep investigation. It's not very essential to change the error type from TVMError to TypeError. I closed it.


chore: Cleanup dependencies

What this PR does 📖

Use specific features from crates

Which issue(s) this PR fixes 🔨

Special notes for reviewers 🗒
Additional comments 🎤

In the end I couldn't find many candidates for feature disabling and also it turned out to be quite a boring task. 😆 I think this is good enough for now.


Set up GitHub issue templates as per the SSW Rule

https://www.ssw.com.au/rules/github-issue-templates/

Done by @sethdailyssw . Closing the issue.


TypeError: save_to_csv() got an unexpected keyword argument 'path'

Bug/Issue Description:
A clear and concise description of what the bug or issue is - in other words, what the unexpected behavior or output is.
Only when using the AdaptiveDetector do I receive a TypeError. Seemingly after the actual detection is complete. I tried to explicitly define a path for the stats_file_path but received the same error. Its unclear why this happens (I've tried looking in init.py but I'm unsure why the "path" argument is still unexpected.
Required Information:
Provide the following information to assist with reporting the bug:

Provide a full copy of the command line options you are using, for example:

scene_list = detect(video_file, AdaptiveDetector())
Traceback (most recent call last):
File "", line 1, in
File "E:\research\python_toolboxes\anaconda_3\envs\video_edits\lib\site-packages\scenedetect_init_.py", line 194, in detect
path=stats_file_path, base_timecode=video.base_timecode)
TypeError: save_to_csv() got an unexpected keyword argument 'path'
>>> scene_list = detect(video_file, AdaptiveDetector(), stats_file_path=base_path)
Traceback (most recent call last):
File "", line 1, in
File "E:\research\python_toolboxes\anaconda_3\envs\video_edits\lib\site-packages\scenedetect_init_.py", line 194, in detect
path=stats_file_path, base_timecode=video.base_timecode)
TypeError: save_to_csv() got an unexpected keyword argument 'path'

Add -v debug -l BUG_REPORT.txt to the beginning of the command, then re-run PySceneDetect and attach the generated BUG_REPORT.txt file.

Unfortunately I'm not sure how to generate this when running within python
Expected Behavior:
I expect the detection to run without this error and return the same list of scenes as with the other detector options.
Computing Environment:

OS: Windows
Python Version: 3.7.12
OpenCV Version: <IP_ADDRESS>

Duplicate of #299. Currently there is a PR to address this, #300 (hopefully merged soon). Will mark this as closed to consolidate discussion over there.

I can confirm the fix works for the issue I had.


Logout bug

I'm still running djangosaml2idp==0.6.3 and running into an issue w/ the logout view. It looks like the resp is already a string and not an object causing an exception to be thrown.
  try:
      # hinfo returns request or response, it depends by request arg
      hinfo = self.IDP.apply_binding(binding, resp.__str__(), resp.destination, relay_state, response=True)
  except Exception as excp:
      logger.error("ServiceError: %s", excp)
      return self.handle_error(request, exception=excp, status=400)

Specifically resp.destination is causing the issue.
ServiceError: 'str' object has no attribute 'destination'

resp is set prior via an IDP method.
resp = self.IDP.create_logout_response(req_info.message, [binding])

I was looking through the newest code and it appears this might have been fixed already. Not entirely sure.

Digging into this further it appears that because we have "sign_response": True, set in settings.py the IDP.create_logout_response is returning the fully rendered XML instead of the response object. Thus causing the exception to happen. We cannot currently move to the models and thus we're stuck on 0.6.3. I basically plan on patching the view and then injecting it into our urls.py. I'm not entirely sure how to go about making the changes correctly.

This bug for sure still exists in the most recent version. create_logout_response calls _status_response which returns either the class or the string representation of the class. The view goes forward assuming it got the object and not the string.

+1 with the same issue

@longshine I've been meaning to hop back on and get my PR fixed. What we did since we're stuck on 0.6.3 is just copy the view code over and implement the fix. Then interject into the urls.py to use our view before the packaged defined view.

I ran into the same problem with  djangosaml2idp==0.7.2 and pysaml2==7.0.1. I'm not sure what the proper fix is for this problem, but the following worked for me:
--- site-packages/djangosaml2idp/views.py.orig	2021-07-05 17:40:31.471614643 +0200
+++ site-packages/djangosaml2idp/views.py	2021-08-24 16:30:34.455854131 +0200
@@ -375,6 +375,7 @@
             return error_cbv.handle_error(request, exception=excp, status_code=400)

         resp = idp_server.create_logout_response(req_info.message, [binding])
+        rinfo = idp_server.response_args(req_info.message, [binding])

         '''
         # TODO: SOAP
@@ -390,13 +391,13 @@

         try:
             # hinfo returns request or response, it depends by request arg
-            hinfo = idp_server.apply_binding(binding, resp.__str__(), resp.destination, relay_state, response=True)
+            hinfo = idp_server.apply_binding(rinfo["binding"], resp, rinfo["destination"], relay_state, response=True)
         except Exception as excp:
             logger.error("ServiceError: %s", excp)
             return error_cbv.handle_error(request, exception=excp, status=400)

-        logger.debug("--- {} Response [\n{}] ---".format(self.__service_name, repr_saml(resp.__str__().encode())))
-        logger.debug("--- binding: {} destination:{} relay_state:{} ---".format(binding, resp.destination, relay_state))
+        logger.debug("--- {} Response [\n{}] ---".format(self.__service_name, repr_saml(resp.encode())))
+        logger.debug("--- binding: {} destination:{} relay_state:{} ---".format(rinfo["binding"], rinfo["destination"], relay_state))

         # TODO: double check username session and saml login request
         # logout user from IDP

This fix was inspired by code that I found in pysaml2: site-packages/saml2/client.py, line 678.

+1 same issue


fix: typos (#146)

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #148   👈

This comment was auto-generated by Graphite.


[5.5] Fix one test and disable two tests for Linux aarch64

@swift-ci test

@swift-ci test windows

This has not been approved by Swift Release manager, and it might be too late to pull into next release.


Fix LLaMA 2 prompt format.

I added these lines to generation.py to see the exact tokens being input into LLaMA:
             prompt_tokens.append(dialog_tokens)

+        for pt in prompt_tokens:
+            print("Prompt tokens:")
+            print(pt)
+            print("Decoded:")
+            print(self.tokenizer.decode(pt))
         generation_tokens, generation_logprobs = self.generate(

And for this message series:
messages = [
    {"role": "user", "content": "I am going to Paris, what should I see?"},
    {
        "role": "assistant",
        "content": """\
Paris, the capital of France, is known for its stunning architecture, art museums, historical landmarks, and romantic atmosphere. Here are some of the top attractions to see in Paris:

1. The Eiffel Tower: The iconic Eiffel Tower is one of the most recognizable landmarks in the world and offers breathtaking views of the city.
2. The Louvre Museum: The Louvre is one of the world's largest and most famous museums, housing an impressive collection of art and artifacts, including the Mona Lisa.
3. Notre-Dame Cathedral: This beautiful cathedral is one of the most famous landmarks in Paris and is known for its Gothic architecture and stunning stained glass windows.

These are just a few of the many attractions that Paris has to offer. With so much to see and do, it's no wonder that Paris is one of the most popular tourist destinations in the world.""",
    },
    {"role": "user", "content": "What is so great about #1?"},
]

Meta's code gives these tokens:
[1, 518, 25580, 29962, 306, 626, 2675, 304, 3681, 29892, 825, 881, 306, 1074, 29973, 518, 29914, 25580, 29962, 3681, 29892, 278, 7483, 310, 3444, 29892, 338, 2998, 363, 967, 380, 27389, 11258, 29892, 1616, 19133, 29879, 29892, 15839, 2982, 22848, 29892, 322, 6017, 7716, 25005, 29889, 2266, 526, 777, 310, 278, 2246, 19650, 1953, 304, 1074, 297, 3681, 29901, 13, 13, 29896, 29889, 450, 382, 2593, 295, 23615, 29901, 450, 9849, 293, 382, 2593, 295, 23615, 338, 697, 310, 278, 1556, 5936, 13902, 2982, 22848, 297, 278, 3186, 322, 16688, 2078, 271, 400, 5086, 8386, 310, 278, 4272, 29889, 13, 29906, 29889, 450, 4562, 12675, 6838, 29901, 450, 4562, 12675, 338, 697, 310, 278, 3186, 29915, 29879, 10150, 322, 1556, 13834, 19133, 29879, 29892, 27261, 385, 21210, 573, 4333, 310, 1616, 322, 24238, 29879, 29892, 3704, 278, 2598, 29874, 29420, 29889, 13, 29941, 29889, 24337, 29899, 29928, 420, 315, 21471, 29901, 910, 9560, 274, 21471, 338, 697, 310, 278, 1556, 13834, 2982, 22848, 297, 3681, 322, 338, 2998, 363, 967, 22883, 293, 11258, 322, 380, 27389, 380, 7114, 12917, 5417, 29889, 13, 13, 1349, 968, 526, 925, 263, 2846, 310, 278, 1784, 19650, 1953, 393, 3681, 756, 304, 5957, 29889, 2973, 577, 1568, 304, 1074, 322, 437, 29892, 372, 29915, 29879, 694, 4997, 393, 3681, 338, 697, 310, 278, 1556, 5972, 6282, 391, 15422, 800, 297, 278, 3186, 29889, 29871, 2, 1, 518, 25580, 29962, 1724, 338, 577, 2107, 1048, 396, 29896, 29973, 518, 29914, 25580, 29962]

Using this repository, I get a slightly different result:
chat_messages = [ChatMessage(content=m["content"], role=m["role"]) for m in messages]
prompt = build_llama2_prompt(chat_messages)
print(tokenizer.encode(prompt, add_special_tokens=False))

There's one missing token for 29871/▁. Adding a space before </s> fixes this.

Hey @float-trip,
Thank you for opening! Can you share what tokenizer is used? Is it the transformers tokenizer or the one from the official repo?


Error converting value "ITEM" to type 'GhostfolioSidekick.Ghostfolio.Contract.ActivityType'.

Hello, first of all great work with this sidekick - really helpful!
I'm having this error reported in the portainer log however cannot understand from where it comes from. Any idea how can I know which entry is causing this?

@pvieira84 , I have created a code change that should fix the issue, if you'd like you can test that version

thanks, this is what I'm getting now

Next unimplemented activity type 😢 , will give it a go

closed for now, let me know if there are issues

Seems to be working now, thanks


feat: add home navigation on the login/signup

issue: #71
Changes:

add website logo on the left top
add back to the home link with the appropriate design

Hi @piyush-gangrade
You’ve done an excellent job with this update! The logo placement and “back to home” link look great and add a nice touch to the auth pages. Thank you for your contribution!


MINOR: fix flaky TestUtils functions

TestUtils#produceMessages should always close the KafkaProducer, even
when there is an exception.  Otherwise, the test will leak threads when
there is an error.
TestUtils#createNewProducer should create a producer with a
requestTimeoutMs of 30 seconds by default, not around 10 seconds.
This should avoid tests that flake when the load on Jenkins climbs.
Fix two cases where a very short timeout of 2 seconds was getting set.

Merged to 1.0, 1.1 and trunk.


Add my name

On a side note, this project has several MB of files so it takes some time to be cloned... not ideal for first time contributors I guess.
Anyways, thanks for your work!

@LiteSoul, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?
We’ve added a tweet feature to our web app recently. Can you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


Advent of code 7 (3rd attempt)

--- Day 7: Bridge Repair ---
The Historians take you to a familiar rope bridge over a river in the middle of a jungle. The Chief isn't on this side of the bridge, though; maybe he's on the other side?
When you go to cross the bridge, you notice a group of engineers trying to repair it. (Apparently, it breaks pretty frequently.) You won't be able to cross until it's fixed.
You ask how long it'll take; the engineers tell you that it only needs final calibrations, but some young elephants were playing nearby and stole all the operators from their calibration equations! They could finish the calibrations if only someone could determine which test values could possibly be produced by placing any combination of operators into their calibration equations (your puzzle input).
For example:
190: 10 19
3267: 81 40 27
83: 17 5
156: 15 6
7290: 6 8 6 15
161011: 16 10 13
192: 17 8 14
21037: 9 7 18 13
292: 11 6 16 20
Each line represents a single equation. The test value appears before the colon on each line; it is your job to determine whether the remaining numbers can be combined with operators to produce the test value.
Operators are always evaluated left-to-right, not according to precedence rules. Furthermore, numbers in the equations cannot be rearranged. Glancing into the jungle, you can see elephants holding two different types of operators: add (+) and multiply (*).
Only three of the above equations can be made true by inserting operators:
190: 10 19 has only one position that accepts an operator: between 10 and 19. Choosing + would give 29, but choosing * would give the test value (10 * 19 = 190).
3267: 81 40 27 has two positions for operators. Of the four possible configurations of the operators, two cause the right side to match the test value: 81 + 40 * 27 and 81 * 40 + 27 both equal 3267 (when evaluated left-to-right)!
292: 11 6 16 20 can be solved in exactly one way: 11 + 6 * 16 + 20.
The engineers just need the total calibration result, which is the sum of the test values from just the equations that could possibly be true. In the above example, the sum of the test values for the three equations listed above is 3749.
Determine which equations could possibly be true. What is their total calibration result?
The data is located in aoc7.txt
git-bob try to implement this.

git-bob try this again as notebook.

--- Part Two ---
The engineers seem concerned; the total calibration result you gave them is nowhere close to being within safety tolerances. Just then, you spot your mistake: some well-hidden elephants are holding a third type of operator.
The concatenation operator (||) combines the digits from its left and right inputs into a single number. For example, 12 || 345 would become 12345. All operators are still evaluated left-to-right.
Now, apart from the three equations that could be made true using only addition and multiplication, the above example has three more equations that can be made true by inserting operators:
156: 15 6 can be made true through a single concatenation: 15 || 6 = 156.
7290: 6 8 6 15 can be made true using 6 * 8 || 6 * 15.
192: 17 8 14 can be made true using 17 || 8 + 14.
Adding up all six test values (the three that could be made before using only + and * plus the new three that can now be made by also using ||) produces the new total calibration result of 11387.
Using your new knowledge of elephant hiding spots, determine which equations could possibly be true. What is their total calibration result?
git-bob try to implement this.


UTM crashes while running VM in the background

Hello this is a follow up for the issue I mentioned before about UTM suddenly closing unexpectedly.
Im using version 3.0.4 guest is linux kernel5 and it has been created with utm v210 which was the most stable version I've tested so far despite of being a beta release.
Anyway the whole app closes when the guest VM is idle and I switch to mac desktop and when I wanna get back to UTM its already closed causing me to lose many relatively valuable data over the month.
last time I mentioned the issue you asked for logs, so here it is.
MBA 2020 M1 UTM V 304 linux kernel 5.15Created using UTMbotQuestioner: amirseyedian#0652Moderator: BluDood#0001

Logs here: https://cdn.discordapp.com/attachments/681548075969413191/943158552263032922/message.txt

log fileAttached FileAdded comment using UTMbotQuestioner: amirseyedian#0652Moderator: BluDood#0001

Since upgrading from 3.0.4 to 3.1.0, I've had the same experience, no matter which VM was running. Crash log attached
UTM-crash.log
Looking at the log, it appears a buffer was allocated too small a space, related to a copy-on-write in CoreImage, causing an attempt to reference an invalid address.  CGImageCreateCopy seems to be the culprit, or at least related.

Just did a bit more experimenting; it appears that even with "Do not save VM screenshot to disk" enabled, 3.1.0 is still saving VM screenshot to disk.  So this could definitely be part of the issue here; there's a bug somewhere in the screenshot code.

@adespoton I think you might have a different issue. However, "Do not save VM screenshot to disk" does not delete any existing screenshot. It just prevents new ones from being saved. Do you see new screenshots being saved?

I tried a few different combinations and screenshots, while a bit confusing, do seem to be working as designed.  With the box checked, quitting and re-launching UTM will display the saved shot on disk, which hasn’t changed.  However, it appears that the option only applies at quit — if I disable the option and run a VM, and then enable it prior to quitting UTM, no screenshot is saved to disk.

On Feb 19, 2022, at 6:33 PM, osy @.***> wrote:

@adespoton I think you might have a different issue. However, "Do not save VM screenshot to disk" does not delete any existing screenshot. It just prevents new ones from being saved. Do you see new screenshots being saved?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.