text:
Force XML v1.1 preable to avoid parse errors - WIP

Relates to #353. This is a work in progress patch. It currently introduces a dependency on stream flyer and does not have meaningful tests. It's been a while since I created it, so if anybody has a failing case to test against it would be helpful.

Hi @easel, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;

Ok, there's now a valid test case demonstrating the problem and the fix.

I am a little bit concerned about adding a new dependency for this. Are you refactoring this?

My thought was to attempt to factor out the dependency. The reason I introduced it in the first place was that I wasn't confident I knew all the possible permutations of ver= that might show up, but the VersionModifier from stream flyer didn't catch them all either so c'est la vie.
I think the thing to do is create a couple of obvious test cases such as a I have, create a naive implementation using StreamFilter that resolves them, and then add to the test cases if additional pre-amble incantations start showing up and are getting missed.
What do you think?

sounds good to me. Maybe @imario42 (as creator of the initial issue) also wants to provide some tests/data to be parsed?

@easel, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, AZPRBOT;

This works mostly but doesn't solve &#0 (null), which I've found sprinkled around. If I combine this code with a &#0 handler (example of one I tried was the RegexModifier from StreamFlyer), you consistently get a NPE in STAX, which some sources claim is a bug in STAX. Might be, tried both orders. Personally think it might be a buffer issue, but resorted for me to using RegexModifier to remove all XML 1.0 illegal chars (obviously not always a good idea)

That's interesting. When I was fighting this a lot I switched over to Woodstox, have you tried that? I don't think I have run into the null char now, but I have several layers of error recovery now so I might just not  be seeing it.

No, I haven't tried Woodstox. Might be relevant.

What is the status of this?

I tried it with Woodstox but ran into 0xb not being parsable. Below is the stack trace
com.ctc.wstx.exc.WstxParsingException: Illegal character entity: expansion character (code 0xb  at [row,col {unknown-source}]: [7,5]
at:  com.ctc.wstx.sr.StreamScanner.constructWfcException(StreamScanner.java:614)
at:  com.ctc.wstx.sr.StreamScanner.throwParseError(StreamScanner.java:487)
at:  com.ctc.wstx.sr.StreamScanner.reportIllegalChar(StreamScanner.java:2455)
at:  com.ctc.wstx.sr.StreamScanner.validateChar(StreamScanner.java:2402)
at:  com.ctc.wstx.sr.StreamScanner.resolveSimpleEntity(StreamScanner.java:1226)
at:  com.ctc.wstx.sr.BasicStreamReader.readTextSecondary(BasicStreamReader.java:4697)
at:  com.ctc.wstx.sr.BasicStreamReader.finishToken(BasicStreamReader.java:3722)
at:  com.ctc.wstx.sr.BasicStreamReader.next(BasicStreamReader.java:1082)
at:  org.codehaus.stax2.ri.Stax2EventReaderImpl.peek(Stax2EventReaderImpl.java:367)
at:  microsoft.exchange.webservices.data.core.EwsXmlReader.isEmptyElement(EwsXmlReader.java:1059)

@pathikrit right now status is waiting on a dependency free implementation of xml version mangling. There's also a bit of debate as to whether that fixes the extent of the issues people are seeing.

An alternative method (which should work in every case without introducing new dependencies) of dealing with this could be generating a dummy XML 1.1 document which includes a reference to an external entity (the XML from EWS that we actually want to parse). The XML spec says that if an XML 1.1 doc includes XML 1.0 entities, it still uses the XML 1.1 rules to process them.

Rebasing this fix it no longer seems to work 👎
Caused by: microsoft.exchange.webservices.data.core.exception.service.remote.ServiceRequestException: The request failed. ParseError at [row,col]:[1,20]
Message: XML version "1.1" is not supported, only XML 1.0 is supported.
   at microsoft.exchange.webservices.data.core.request.SimpleServiceRequestBase.internalExecute(SimpleServiceRequestBase.java:74)
   at microsoft.exchange.webservices.data.core.request.MultiResponseServiceRequest.execute(MultiResponseServiceRequest.java:158)
   at microsoft.exchange.webservices.data.core.ExchangeService.internalFindFolders(ExchangeService.java:377)
   at microsoft.exchange.webservices.data.core.ExchangeService.findFolders(ExchangeService.java:426)
   at microsoft.exchange.webservices.data.core.ExchangeService.findFolders(ExchangeService.java:461)
   at com.fsck.k9.mail.store.ews.EwsStore.getFolders(EwsStore.java:103)
   	... 11 more
Caused by: com.sun.xml.stream.XMLStreamException2: ParseError at [row,col]:[1,20]
Message: XML version "1.1" is not supported, only XML 1.0 is supported.
   at com.sun.xml.stream.XMLReaderImpl.setInputSource(XMLReaderImpl.java:211)
   at com.sun.xml.stream.XMLReaderImpl.<init>(XMLReaderImpl.java:192)
   at com.sun.xml.stream.ZephyrParserFactory.getXMLStreamReaderImpl(ZephyrParserFactory.java:282)
   at com.sun.xml.stream.ZephyrParserFactory.createXMLStreamReader(ZephyrParserFactory.java:147)
   at com.sun.xml.stream.ZephyrParserFactory.createXMLStreamReader(ZephyrParserFactory.java:142)
   at com.sun.xml.stream.ZephyrParserFactory.createXMLEventReader(ZephyrParserFactory.java:98)
   at microsoft.exchange.webservices.data.core.EwsXmlReader.initializeXmlReader(EwsXmlReader.java:121)
   at microsoft.exchange.webservices.data.core.EwsXmlReader.<init>(EwsXmlReader.java:92)
   at microsoft.exchange.webservices.data.core.EwsServiceXmlReader.<init>(EwsServiceXmlReader.java:59)
   at microsoft.exchange.webservices.data.core.request.ServiceRequestBase.readResponse(ServiceRequestBase.java:401)
   at microsoft.exchange.webservices.data.core.request.SimpleServiceRequestBase.internalExecute(SimpleServiceRequestBase.java:63)

authors: azurecla easel maheshsubedi mikebell90 pathikrit philipwhiuk qblix serious6

repo: OfficeDev/ews-java-api


text:
Ressource not found when setting up Author App in Teams

Following the guide and installing the app(s) went fine.
Adding the Cosumer part app even works fine in Teams.
However when trying to add the Authors App and trying to set ip up to a Team, where the UPN user allowed to author is owner of the Team fails with the following error:
The resource you are looking for has been removed, had its name changed, or is temporarily unavailable.
What could have gone wrong here?

Unfortunately we did not find a solution up to day. Did you manage to solve it in the meantime?

Just redeployed Version 4.1.0 from scratch. Now it's working. Unfortunately I cannot tell what I did differently to the last deployment except Naming the app not like the default in the first place.

source: gharchive/issue

license: MIT

license_type: permissive

license_source: github-api


text:
Support single tenant app and user assigned managed id for user and author bots

Azure Bot Service have been supporting sigle tenant app and user assigned managed id from Nov 2021.
https://docs.microsoft.com/en-us/azure/bot-service/bot-service-quickstart-registration?view=azure-bot-service-4.0&tabs=userassigned
Do you have any plan to support single-tenant app and user assienged managed id on Company Communicator?
I guess that multi tenant app is typical blocker to install production environment. Because some companies doesn't allow multitenant app on AAD.

Hi @runceel ,
Thanks for your suggestion. We will review the request internally and post that will add it to backlog items for future releases. As of now we don't have the support and also we won't be able to provide any ETA for this.

authors: runceel v-royavinash


text:
Outdated statement about shared runtime

This sentence,

Note that using a shared runtime means your add-in will use Internet Explorer 11, not Microsoft Edge.

is no longer always true. For Office on Windows, the shared runtime uses Microsoft Edge with WebView2 (Chromium-based) if the conditions for using it are met as explained in the article Browsers used by Office Add-ins. Else, it uses IE.

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

ID: 94ad7803-d4f7-a700-7c7f-204cd8f34e4a
Version Independent ID: 9c485bae-cab1-bdad-4c8e-baad2b12b22d
Content: Create custom functions in Excel - Office Add-ins
Content Source: docs/excel/custom-functions-overview.md
Product: excel
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

@Rick-Kirkham Thanks for raising this!
@alison-mk Can you take a look?
Thanks.

authors: ElizabethSamuel-MSFT Rick-Kirkham

repo: OfficeDev/office-js-docs-pr


text:
[All Hosts] (support) specify the window objects that are unsupported…

… in add-ins

Docs Build status updates of commit 40634af:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/reference/window-objects-unsupported-in-add-ins.md
:warning:Warning
View
Details

docs/toc.yml
:white_check_mark:Succeeded
View

docs/reference/window-objects-unsupported-in-add-ins.md

Line 10: [Warning-InvalidFileLink] Invalid file link:(~/concepts/browsers-used-by-office-web-add-ins).

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

Docs Build status updates of commit 97dcd3d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/window-objects-unsupported-in-add-ins.md
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 8fe9fe9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/window-objects-unsupported-in-add-ins.md
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 5e5beeb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/window-objects-unsupported-in-add-ins.md
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

source: gharchive/pull-request


text:
Validation - Skype Relative Links 4

Fixing about 43 docsAbsoluteLinks.
Also removing "?view=ucma-api" from the /dotnet/ links.

Docs Build status updates of commit 3a0e2bb:
:warning: Validation status: warnings

File
Status
Preview URL
Details

Skype/Trusted-Application-API/docs/RegistrationInAzureActiveDirectory.md
:warning:Warning

Details

Skype/Trusted-Application-API/docs/SfBRegistration.md
:warning:Warning

Details

Skype/Trusted-Application-API/samples/AnonMeetingJoinSamples/README.md
:warning:Warning

Details

Skype/Trusted-Application-API/samples/QuickStartSamples/README.md
:warning:Warning

Details

Skype/index.md
:warning:Warning

Details

Skype/UCMA/client-platforms.md
:bulb:Suggestion

Details

Skype/UCMA/communications-conversations-calls-conferences-and-back-to-back-user-agents.md
:bulb:Suggestion

Details

Skype/UCMA/conferencemanagement-quickstart.md
:bulb:Suggestion

Details

Skype/UCMA/connecting-to-audio-resources-in-ucma-5-0.md
:bulb:Suggestion

Details

Skype/UCMA/connection-manager.md
:bulb:Suggestion

Details

Skype/UCMA/contactgroupservices.md
:bulb:Suggestion

Details

Skype/UCMA/contacts-and-groups-management.md
:bulb:Suggestion

Details

Skype/UCMA/contacts-and-groups.md
:bulb:Suggestion

Details

Skype/Trusted-Application-API/docs/RegistrationInAzureActiveDirectory.md

Line 1, Column 1: [Warning: title-missing - See documentation] Missing required attribute: 'title'. Add a title string to show in search engine results.
Line 1, Column 1: [Warning: duplicate-h1s - See documentation] H1 'Registering your application in Azure AD' is duplicated with other articles: 'Trusted-Application-API/docs/RegistrationInAzureActiveDirectory.md, UCWA/RegisteringYourApplicationInAzureAD.md(2,1)'. First level headings must be unique within a docset.
Line 1, Column 1: [Warning: duplicate-titles - See documentation] Title 'Registering your application in Azure AD' is duplicated in: 'Trusted-Application-API/docs/RegistrationInAzureActiveDirectory.md, UCWA/RegisteringYourApplicationInAzureAD.md(2,1)'. Titles must be unique within a docset.
Line 14, Column 2: [Warning: duplicate-alt-text - See documentation] Alt text 'alt text' is duplicated. Within an article, alt text must be unique.
Line 18, Column 2: [Warning: duplicate-alt-text - See documentation] Alt text 'alt text' is duplicated. Within an article, alt text must be unique.
Line 22, Column 2: [Warning: duplicate-alt-text - See documentation] Alt text 'alt text' is duplicated. Within an article, alt text must be unique.
Line 28, Column 2: [Warning: duplicate-alt-text - See documentation] Alt text 'alt text' is duplicated. Within an article, alt text must be unique.
Line 32, Column 2: [Warning: duplicate-alt-text - See documentation] Alt text 'alt text' is duplicated. Within an article, alt text must be unique.
Line 36, Column 2: [Warning: duplicate-alt-text - See documentation] Alt text 'alt text' is duplicated. Within an article, alt text must be unique.
Line 42, Column 2: [Warning: duplicate-alt-text - See documentation] Alt text 'alt text' is duplicated. Within an article, alt text must be unique.
Line 1, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

Skype/Trusted-Application-API/docs/SfBRegistration.md

Line 1, Column 1: [Warning: title-missing - See documentation] Missing required attribute: 'title'. Add a title string to show in search engine results.
Line 1, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

Skype/Trusted-Application-API/samples/AnonMeetingJoinSamples/README.md

Line 1, Column 1: [Warning: title-missing - See documentation] Missing required attribute: 'title'. Add a title string to show in search engine results.
Line 1, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

Skype/Trusted-Application-API/samples/QuickStartSamples/README.md

Line 100, Column 1: [Warning: code-block-indented - See documentation] Indented code blocks aren't allowed. Use a Markdown code block surrounded by triple backticks (```).

Skype/index.md

Line 1, Column 1: [Warning: title-missing - See documentation] Missing required attribute: 'title'. Add a title string to show in search engine results.
Line 1, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

Skype/UCMA/client-platforms.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

Skype/UCMA/communications-conversations-calls-conferences-and-back-to-back-user-agents.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

Skype/UCMA/conferencemanagement-quickstart.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

Skype/UCMA/connecting-to-audio-resources-in-ucma-5-0.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.
Line 15, Column 95: [Suggestion: preserve-view-not-set - See documentation] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /dotnet/api/microsoft.rtc.collaboration.audiovideo.voicexml.browser?view=ucma-voice. Otherwise, remove the view parameter.

Skype/UCMA/connection-manager.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.
Line 73, Column 321: [Suggestion: preserve-view-not-set - See documentation] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /dotnet/api/system.net.ipendpoint?view=netframework-4.7.2. Otherwise, remove the view parameter.
Line 73, Column 462: [Suggestion: preserve-view-not-set - See documentation] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /dotnet/api/system.net.ipendpoint.port?view=netframework-4.7.2. Otherwise, remove the view parameter.
Line 73, Column 593: [Suggestion: preserve-view-not-set - See documentation] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /dotnet/api/system.net.ipendpoint.address?view=netframework-4.7.2. Otherwise, remove the view parameter.

Skype/UCMA/contactgroupservices.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

Skype/UCMA/contacts-and-groups-management.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

Skype/UCMA/contacts-and-groups.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

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

@ShawnVW1 hi Shawn, is this PR still happening or is it abandoned?

Actually, I'm not sure. While the infamous skype-docs was being sorted out, I had to switch to another project.
Meera, what should I do with this now? I'm not sure if I submitted it for peer review yet.

Shawn

From: Linda Lu Cannon @.>
Sent: Monday, April 18, 2022 1:14 PM
To: OfficeDev/skype-docs @.>
Cc: Shawn Wilson (AQUENT LLC) @.>; Mention @.>
Subject: Re: [OfficeDev/skype-docs] Validation - Skype Relative Links 4 (PR #705)
@ShawnVW1https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FShawnVW1&data=05|01|v-shawwilson%40microsoft.com|b1a612bdb08f4d8c7d7308da2177fe46|72f988bf86f141af91ab2d7cd011db47|1|0|637859096505719715|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=ef6vkMDxn5h1DFUyxvwmhpsf%2Fch1Ejhw21G26Sc%2BEhk%3D&reserved=0 hi Shawn, is this PR still happening or is it abandoned?

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FOfficeDev%2Fskype-docs%2Fpull%2F705%23issuecomment-1101724187&data=05|01|v-shawwilson%40microsoft.com|b1a612bdb08f4d8c7d7308da2177fe46|72f988bf86f141af91ab2d7cd011db47|1|0|637859096505719715|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=WTkwZgr9Uulumi92%2F6yOaZbASZ0T41jld6fe323Kzgs%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAYPBGZJIUWA4H74SU4KQS6DVFW7A7ANCNFSM5TEWZ6WA&data=05|01|v-shawwilson%40microsoft.com|b1a612bdb08f4d8c7d7308da2177fe46|72f988bf86f141af91ab2d7cd011db47|1|0|637859096505719715|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=BW%2B5a6MWBMkF%2BxcPF32e9H7eiJTucAiqVQIVEIcfHcE%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

@lindalu-MSFT
This PR is ready for a final reviewer.
The Content & Learning team is fixing build validation errors on docs.microsoft.com for the rest of H2. This will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. Please review within five business days and merge or comment in the PR with any changes you’d like to see. Thank you!

Docs Build status updates of commit 46b0ffc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Skype/Trusted-Application-API/docs/SfBRegistration.md
:bulb:Suggestion

Details

Skype/UCMA/client-platforms.md
:bulb:Suggestion

Details

Skype/UCMA/communications-conversations-calls-conferences-and-back-to-back-user-agents.md
:bulb:Suggestion

Details

Skype/UCMA/conferencemanagement-quickstart.md
:bulb:Suggestion

Details

Skype/UCMA/connecting-to-audio-resources-in-ucma-5-0.md
:bulb:Suggestion

Details

Skype/UCMA/connection-manager.md
:bulb:Suggestion

Details

Skype/UCMA/contactgroupservices.md
:bulb:Suggestion

Details

Skype/UCMA/contacts-and-groups-management.md
:bulb:Suggestion

Details

Skype/UCMA/contacts-and-groups.md
:bulb:Suggestion

Details

Skype/Trusted-Application-API/docs/RegistrationInAzureActiveDirectory.md
:white_check_mark:Succeeded

Skype/Trusted-Application-API/samples/AnonMeetingJoinSamples/README.md
:white_check_mark:Succeeded

Skype/index.md
:white_check_mark:Succeeded

Skype/Trusted-Application-API/docs/SfBRegistration.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

Skype/UCMA/client-platforms.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

Skype/UCMA/communications-conversations-calls-conferences-and-back-to-back-user-agents.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

Skype/UCMA/conferencemanagement-quickstart.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

Skype/UCMA/connecting-to-audio-resources-in-ucma-5-0.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.
Line 15, Column 95: [Suggestion: preserve-view-not-set - See documentation] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /dotnet/api/microsoft.rtc.collaboration.audiovideo.voicexml.browser?view=ucma-voice. Otherwise, remove the view parameter.

Skype/UCMA/connection-manager.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.
Line 73, Column 321: [Suggestion: preserve-view-not-set - See documentation] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /dotnet/api/system.net.ipendpoint?view=netframework-4.7.2. Otherwise, remove the view parameter.
Line 73, Column 462: [Suggestion: preserve-view-not-set - See documentation] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /dotnet/api/system.net.ipendpoint.port?view=netframework-4.7.2. Otherwise, remove the view parameter.
Line 73, Column 593: [Suggestion: preserve-view-not-set - See documentation] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /dotnet/api/system.net.ipendpoint.address?view=netframework-4.7.2. Otherwise, remove the view parameter.

Skype/UCMA/contactgroupservices.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

Skype/UCMA/contacts-and-groups-management.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

Skype/UCMA/contacts-and-groups.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

repo: OfficeDev/skype-docs


text:
File dialog to import/export off

should be better than recompiling every time :unknown:

This got done a while ago, closing this.

repo: OfficialURL/miratope-rs


text:
선거 사무소 위치 시각화하는데 큰 도움이 되었습니다.

이번 총선에도 선거구 지도 지난번보다 훨씬 더 좋아졌습니다.
다음번 선거에는 인구비례 선거지도도 많이 기대됩니다.
활용웹사이트; https://statkclee.quarto.pub/vdashboard/hexmap.html
활용사례:

안녕하세요.
오마이뉴스 이종호 기자입니다. 전에 한번 인사드린 적이 있습니다.^^
메일 주셔서 고맙습니다. 선생님 사이트 보면서 많이 배우겠습니다. 감사합니다.
오마이뉴스 이종호 드림.
2024년 3월 18일 (월) 오전 8:20, Kwangchun Lee @.***>님이 작성:

이번 총선에도 선거구 지도 지난번보다 훨씬 더 좋아졌습니다.
다음번 선거에는 인구비례 선거지도도 많이 기대됩니다.
활용웹사이트; https://statkclee.quarto.pub/vdashboard/hexmap.html
활용사례: image.png (view on web)
https://github.com/OhmyNews/2024_22_elec_map/assets/5325425/4017d257-7081-4ac2-9f85-72f31aa118cb
—
Reply to this email directly, view it on GitHub
https://github.com/OhmyNews/2024_22_elec_map/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC74I6VOLT4VXWL6D5QMI53YYYQLLAVCNFSM6AAAAABE2S4KRKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE4TAOJRHEZDCNI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


text:
Ошибка 'media' must be Media type

Не получается запостить комментарий, использую код
yourmedia = "B7Q80s0pOR7" media=Media(yourmedia) agent.add_comment(0, media, comment)
естественно, до выполнения третьей строки я авторизовался, могу получать имя пользователя
print("Успешная авторизация, " + agent.username + " или  " + agent.full_name)
но строка
agent.add_comment(0, media, comment)
выдает такую ошибку
Traceback (most recent call last):   File "prog.py", line 22, in <module>     agent.add_comment(0, media, comment) #пишем коммент   File "C:\Users\pal_p\AppData\Local\Programs\Python\Python38\lib\site-packages\instagram-2.1.0-py3.8.egg\instagram\exceptions.py", line 119, in wrapper     raise exception   File "C:\Users\pal_p\AppData\Local\Programs\Python\Python38\lib\site-packages\instagram-2.1.0-py3.8.egg\instagram\exceptions.py", line 114, in wrapper     return func(obj, *args, **kwargs)   File "C:\Users\pal_p\AppData\Local\Programs\Python\Python38\lib\site-packages\instagram-2.1.0-py3.8.egg\instagram\agents.py", line 1353, in add_comment     raise TypeError("'media' must be Media type") TypeError: 'media' must be Media type
Что я делаю не так?

разобрался, правильно писать agent.add_comment(media, comment) но теперь другая ошибка

repo: OlegYurchik/pyInstagram


text:
Empty prefix IMG: in the submenu titles

I found next error in the generation:
IMG: Science
IMG:/home/box/.cache/xdg-xmenu/icons/cantor.png Cantor (Frontend to Mathematical Software) cantor -qwindowicon cantor -qwindowtitle
R (environment for statistical computing) xterm -e R
IMG:/home/box/.cache/xdg-xmenu/icons/octave.png GNU Octave /usr/bin/octave --gui
IMG:/home/box/.cache/xdg-xmenu/icons/kalgebra.png KAlgebra (Graph Calculator) kalgebra
IMG:/home/box/.cache/xdg-xmenu/icons/kalgebra.png KAlgebra Mobile (Pocket Graph Calculator) kalgebramobile
IMG:/home/box/.cache/xdg-xmenu/icons/kalzium.png Kalzium (Periodic Table of Elements) kalzium -qwindowtitle
IMG:/home/box/.cache/xdg-xmenu/icons/marble.png KDE Marble (Virtual Globe) marble
IMG:/home/box/.cache/xdg-xmenu/icons/libreoffice-math.png LibreOffice Math (Formula Editor) libreoffice --math
IMG:/home/box/.cache/xdg-xmenu/icons/rocs.png Rocs (Rocs Graph Theory) rocs -qwindowtitle
Julia xterm -e julia
Singular xterm -e Singular
If the image doesn't exist in the app line, the prefix IMG is correctly removed, e.g. R and Singular lines.
But if the image missing for submenu names, there is still present IMG: prefix with empty icon which cause error message in the xmenu program:  xmenu: cannot load icon
The prefix IMG:  cannot be generated for submenu titles if there is no image for it, similar as app lines.

Thanks for pointing it out! The latest commit should fix it.

repo: OliverLew/xdg-xmenu


text:
windows problem

when i enter the username and the password it write:
"Traceback (most recent call last):
File "main.py", line 159 in
File "json_init__.pyc", line 339, in loads
File "json\decoder.pyc", line 364, in decode
File "json\decoder.pyc", line 382, in raw_decode
ValueError: No JSON object could be decoded

look the last update

repo: OlympicCode/vHackAPI-Python


text:
How to configure the startup parameters

Each reboot takes one week to rebuild the index. How to configure the startup parameters. The log is as follows:
2018-05-24 10:46:48 UpdateTip: new best=0000000000000000005a41f7e32daf3a38f2cd8a6b88b6763505928225c76475 height=505183 version=0x20000000 log2_work=87.919129 tx=294090628 date='2018-01-20 14:26:08'progress=0.972709
2018-05-24 10:46:56 UpdateTip: new best=00000000000000000045f51d2461567f3bfb594176d0bdad46a99143c05d1065 height=505184 version=0x20000000 log2_work=87.919176 tx=294092554 date='2018-01-20 14:36:12'progress=0.972711
2018-05-24 10:47:05 Pre-allocating up to position 0x400000 in rev01147.dat
2018-05-24 10:47:05 UpdateTip: new best=0000000000000000002a6fafb408ebc9043cd7085e06579cc3b323b086f24473 height=505185 version=0x20000000 log2_work=87.919223 tx=294094950 date='2018-01-20 15:05:30'progress=0.972716

remove -reindex

I'll try

Hey @zhou1124, looks like your node isn't completely synchronized or you are indeed using -reindex?

repo: OmniLayer/omnicore

license: mit

license_source: bigquery


text:
updated team

on popular demand :smile:

:)

authors: filipw nosami


text:
Trouble with editor.action.format

From @TJSoftware on January 29, 2016 17:35
Summary
I am unable to format code for xml and c#.  json and javascript do not have any issues.
What I have done
Originally I modified my editor.tabSize to be 2 (in workspace).  I removed this override and just left in the defaults.  I also overwrote the keybinding for editor.action.format to use ctrl+k ctrl+d.  I also removed those and left just the default in.  Working with test files of each type mentioned above I am able to format some file types, but not others.  This does not seem to be dependent on any of my settings, I have just added that information in just in case it is relevant.
C# example - what I expect
The following screenshot the brackets should line up in MyClass.
In this case while the tab size is set to 2 I would also expect that MyClass would be moved left to adjust to the correct tab size.
Officially the starting bracket would be on the same line as the class definition for my company policy, but I do not see that option available.  Bonus points if you can mention how to set this in VSCode. VS Enterprise was able to do this with just a checkbox in settings, I am ok with a bool in json, I just do not see the value anywhere.

XML example - what I expect
Here I expect that subLevel3 would be tabbed in to whatever the current tab setting is.

Tech Specs
Windows 10 Pro

Copied from original issue: Microsoft/vscode#2554

From @aeschli on February 1, 2016 11:3
VSCode has not built-in XML formatter. Do you use an extension?
If yes, can you file the issue against that extension?
We're using this issue for the C# issue

@jrieken I am not using an extension, I had found a StackOverflow post (I know not the best source) of what VSCode supports for formatting and I was going off that as I could not find that information from https://github.com/Microsoft/vscode or https://code.visualstudio.com/docs.  Thank you for looking into this.  Please let me know if you need anything else to help recreate the C# example

#24

authors: TJSoftware jrieken

repo: OmniSharp/omnisharp-vscode


text:
Error in logs

{
"Request_seq": 35,
"Command": "/v2/highlight",
"Running": true,
"Success": false,
"Message": ""System.ArgumentOutOfRangeException: Det angivne argument lå uden for det gyldige værdiområde.\r\nParameternavn: line\r\n   ved Microsoft.CodeAnalysis.Text.LinePosition..ctor(Int32 line, Int32 character)\r\n   ved OmniSharp.Roslyn.CSharp.Services.SemanticHighlight.SemanticHighlightService.d__1.MoveNext() i D:\\a\\1\\s\\src\\OmniSharp.Roslyn.CSharp\\Services\\SemanticHighlight\\SemanticHighlightService.cs:linje 36\r\n--- Afslutningen på staksporingen fra den tidligere placering, hvor undtagelsen blev udløst ---\r\n   ved System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   ved System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   ved OmniSharp.Endpoint.EndpointHandler2.<GetFirstNotEmptyResponseFromHandlers>d__19.MoveNext()\\r\\n--- Afslutningen på staksporingen fra den tidligere placering, hvor undtagelsen blev udløst ---\\r\\n   ved System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\\r\\n   ved System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\\r\\n   ved OmniSharp.Endpoint.EndpointHandler2.d__20.MoveNext() i D:\\a\\1\\s\\src\\OmniSharp.Host\\Endpoint\\EndpointHandler.cs:linje 230\r\n--- Afslutningen på staksporingen fra den tidligere placering, hvor undtagelsen blev udløst ---\r\n   ved System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   ved System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   ved OmniSharp.Endpoint.EndpointHandler`2.d__16.MoveNext() i D:\\a\\1\\s\\src\\OmniSharp.Host\\Endpoint\\EndpointHandler.cs:linje 131\r\n--- Afslutningen på staksporingen fra den tidligere placering, hvor undtagelsen blev udløst ---\r\n   ved System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\r\n   ved System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   ved OmniSharp.Stdio.Host.d__13.MoveNext() i D:\\a\\1\\s\\src\\OmniSharp.Stdio\\Host.cs:linje 215"",
"Body": null,
"Seq": 133,
"Type": "response"
}

duplicate of https://github.com/OmniSharp/omnisharp-vscode/issues/4255

authors: ffMathy filipw


text:
wrong url of Legacy C# Support extension in VSCode's introduction view with Mac OS X.

correct Legacy C# Support extension url is below.
https://marketplace.visualstudio.com/items?itemName=ms-vscode.omnisharp
but current target url is below.(goes to wrong empty page of marketplace.)
https://marketplace.visualstudio.com/items?itemName%3Dms-vscode.omnisharp

Oops, Close button has been pressed by my mistake.

Thanks @sassembla! Apologies for the error!

This should go away with this release. We're removing the need for the Legacy C# Support extension in 1.3

This has been removed from our readme.md. Once we ship the latest extension (1.3.0) to the marketplace, this will be reflected there as well.

authors: DustinCampbell sassembla


text:
CTCSS ≥ 210,7Hz does not TX/RX properly.

Hello, I noted that analog subtones between 210,7 Hz and 250,3Hz does not work. No proper TX nor RX. Checked against an SDR and other quality HTs (Yaesu, Icom, etc).
Tested with ENABLE_CTCSS_TAIL_PHASE_SHIFT := 0 and 1 (just in case), an got same result.
In theory, the calibration data of my Quansheng is untouched, so I don't know it is my unit failing or a firmware bug, since I don't have other Quansheng, but this behaviour doesn't occur with original untouched firmware.

oh, sorry about that, I'll look into it tomorrow, very late here now.

Found the problem
Thank you for letting me know.


text:
Fix ordering of initial schema to support migrations

Fix ordering of initial schema to support migration use cases.
In this case, some table create statements were referring to others that were not yet created.  This is a matter of ordering of statements so that dependencies are declared before they are used.
This appears to be caused by an upstream bug, perhaps in supabase, PostgreSQL, or its SQL dump tooling.  This PR should resolve the issue for our repository.
The error was observed was while running supabase start:

Error: Error starting database: ERROR:  relation "public.Attendee" does not exist

I had to move the Attendee create statement towards the top of the file, and then follow along with dependents of Attendee (Preso, profiles).  It appeared to possibly be in reverse-order.

I'm just gonna accept this and deal with the consequences 😅

Seems to have worked without issue, so I think we're good! 👍🏾

authors: iAmWillShepherd jmadler

repo: OneSignalDevelopers/ngosi


text:
Fixed Header with Column Filter Popup

The filter popup shows within the "scrollable" area of the header only.  It works fine if using a "non-scrollable" header.

screenshot?

authors: kenberkeley marcotana


text:
Write shortest possible code: How to join lists (python)

Please write shortest code example for this question:
How to join lists in python
How to do it:

Go to python codes
Create new file (named in underscore case, should contain key words from title) with md extension (markdown file).
Propose new file with following content (please use all three blocks if possible - title, code itself and explanations list):

# How to join lists

```python
code part1 part2 part3 ...
```

- part1 - explain code part 1
- part2 - explain code part 2
- ...

More advanced template for examples and linked solutions. More docs here.

@nonunicorn  Please assign this to me

repo: Onelinerhub/onelinerhub


text:
In case crowdsale owner key has lost during ICO period.

Plz check owner authority.
What's going to happen if owner key have stolen or lost? Is it serious or not?
What is alternative or solution to that matters?
@4000D @thomashin

I think it's not a question.
cannot call fianlize() or claimTokens(). If owner's private key is lost, all Ether is locked in vault forever because sale cannot be finalized. Also tokens is going to be obsolete.

Actually, zeppelin-solidity adds onlyOwner modifier to finalize() function.
https://github.com/OpenZeppelin/openzeppelin-solidity/blob/master/contracts/crowdsale/distribution/FinalizableCrowdsale.sol#L24

Multi-sig wallet is the exact solution in case of losing private key.

Closing key management is out of scope

repo: Onther-Tech/tokyo


text:
[csolution] Support multiple component instances

The component subnode instances is described in the specification but it is not yet implemented:
https://github.com/Open-CMSIS-Pack/cmsis-toolbox/blob/main/docs/YML-Input-Format.md#instances
There is also an open discussion in Open-CMSIS-Pack-Spec about this subject:
Handling of Multiple Component Instances #76

RTE library already handles multiple component instances and their config files.
What is not yet implemented:

describing number of instances in cproject.yml file and passing this information to RTE model when selecting components

Do we agree that a component with "MaxInstance>1" and no "attr=config" files in its description is still accepted ?
In such a case, you cannot really link easily the instance and the files associated to it.

@fred-r, sure you can use "MaxInstance>1" without config files as well.
RTE_Components.h is populated with zero-based component indexes, e.g:
/* Keil::File System&MDK-Plus:Drive:Memory Card@6.16.2 */
#define RTE_FileSystem_Drive_MC_0       /* File System Memory Card Drive 0 */
#define RTE_FileSystem_Drive_MC_1       /* File System Memory Card Drive 1 */

Considering the fact that the subnode instances is specified but neither allowed by the Input YAML schema files nor supported by csolution processing this becomes a defect rather than an enhancement.
Note that in the cprj format and the cbuildgen tools already support the attribute instances

We have closed the discussion about changes to the specification and agreed that we would use the existing scope and features as already specified.
We need to ensure that we reflect the specification of instances in the schema of the CMSIS solution project YAML format once the format has been agreed:
current:
components:
  - component: USB:Device
    instances: 2

proposed 2024-02-14 (https://github.com/Open-CMSIS-Pack/cmsis-toolbox/blob/main/docs/setup.md#instance-number)
adding optional instance count [=N] (default 1)
[Cvendor::] Cclass [&Cbundle] :Cgroup [:Csub] [&Cvariant] [=N] [@[~ | >=]Cversion]

components:
  - component: USB:Device=2

authors: brondani edriouk fred-r jkrech

repo: Open-CMSIS-Pack/devtools


text:
BUG? FileSystemLoader searchpath should be list

 pip show jinja2 |grep Version
Version: 2.10

 File "/home/ni_tools/mrQA/mrQA/mrQA/formatter.py", line 110, in render
    fs_loader = jinja2.FileSystemLoader(searchpath=self.template_folder)
  File "/usr/lib/python3/dist-packages/jinja2/loaders.py", line 163, in __init__
    self.searchpath = list(searchpath)
TypeError: 'PosixPath' object is not iterable

this could from using an outdated jinja2 library. 3.1 is current, I'm using 2.10

thanks Wiil. @sinhaharsh is looking into this and will comment soon.

Hi Will, did you try it with both versions of Jinja, or just your old version?

Hi Will, the codebase has undergone multiple major revisions since this issue, so closing it for now. Thanks for taking the time to help and contribute, appreciate it.

authors: WillForan raamana

repo: Open-Minds-Lab/mrQA


text:
recipes-openamp: get rpmsg apps from new repo

remove rpmsg-echo-test source files and modify relative recipe to sync code from openamp-system-reference repo
Signed-off-by: Tanmay Shah tanmay.shah@amd.com

@edmooring Thanks for approval. One minor change, we are moving LICENSE file to root directory in openamp-system-reference repo. So, I will have to change LICENSE file location in bitabake script.
I will send new patch soon.

Most of this should be factored out of the individual recipes and made an inc file at the rpmsg-examples level.
I don't want to update 3 different places when adjusting the commit id.
Most of the new recipe logic should be there: git url etc.  Even the license can be done this way as they all refer to the same file.
The other thing that is strange here is pointing the WORKDIR deep into the repo clone.
This makes the LICENSE file:// URL very strange also.
I will look for a better way to do this.
One obvious answer is build them all with one recipe and then split them into separate packages.  However that is a lot of change compared to what we have and may not scale as well.
@mhatle may have other suggestions

Most of this should be factored out of the individual recipes and made an inc file at the rpmsg-examples level. I don't want to update 3 different places when adjusting the commit id. Most of the new recipe logic should be there: git url etc. Even the license can be done this way as they all refer to the same file.

Ack.

The other thing that is strange here is pointing the WORKDIR deep into the repo clone. This makes the LICENSE file:// URL very strange also.
I will look for a better way to do this. One obvious answer is build them all with one recipe and then split them into separate packages. However that is a lot of change compared to what we have and may not scale as well.

Yes correct, we need better way to do this. I tried with making single bb file that compiles all the demos i.e. openamp-system-reference.bb, how ever it needs makefile in root dir. Which I believe wasn't good idea. I agreed with @arnopo and removed Makefile from root dir. So, I had to go all the way to example root dir where Makefile is available.
One solutiont could be, If we agree to keep Makefile in root dir, then we can call different targets from that make file from different recipes. Not sure if it's good idea though.
I definitely look forward to better and clean solution to this.

@mhatle may have other suggestions

LGTM

If we go the single makefile route, I would not delete the individual ones.  Just add a new one that calls each of the sub makes.
However I am OK to merge 3 recipies and look for a better way over time.
I would still like to cleanup the LICENCE URL.  Can ${S} be used in a file:// URL??

BTW: This will need to be rebased now that I have accepted the PRs from May

Also please  change LICENSE to match the new master.  "BSD" gives warnings.  You need to specify which BSD.

BTW: This will need to be rebased now that I have accepted the PRs from May

Sure I will do that.

You can use WORKDIR in the LIC_FILES_CHKSUM.
example:
LIC_FILES_CHKSUM = "file://${WORKDIR}/license.html;md5=5c94767cedb5d6987c902ac850ded2c6"
from:
https://docs.yoctoproject.org/current/dev-manual/common-tasks.html#specifying-the-lic-files-chksum-variable

This sounds good idea. Thanks.

If we go the single makefile route, I would not delete the individual ones. Just add a new one that calls each of the sub makes.

Actually I see that we may not need single make file, if we can accept that
S = "${WORKDIR}/git/examples/linux/rpmsg-echo-test"
With this Makefile in root dir is not needed for now. We can introduce common Makefile if needed in future.

Yes I am OK with that as long as we use ${WORKDIR} in the LIC_FILES_CHKSUM
WIth SRCURI and Licence info factored into an include common to all 3 examples.

Yes I am OK with that as long as we use ${WORKDIR} in the LIC_FILES_CHKSUM
WIth SRCURI and Licence info factored into an include common to all 3 examples.

One more thing. I did make common .inc file which will be included in all of the three .bb files.
However, for that to work, I may have to create single pull request and bring all the demo source files all togather in single commit. Otherwise it may break compilation.

This PR is already a single commit.  I am OK with it staying that way.

This PR is already a single commit. I am OK with it staying that way.

I am sorry I meant in https://github.com/OpenAMP/openamp-system-reference repository.
Three demos should be available in single commit.

This PR is already a single commit. I am OK with it staying that way.

I am sorry, I wasn't clear. I mean in openamp-system-reference repo: https://github.com/OpenAMP/openamp-system-reference
We need all demos in single commit.

I attempted to merge this request and got the following errors:
WARNING: rpmsg-proxy-app-1.0-r0 do_unpack: rpmsg-proxy-app: the directory ${WORKDIR}/git/examples/linux/rpmsg-proxy-app (/scratch2/fray/git/openamp/build-honister/tmp/work/core2-64-poky-linux/rpmsg-proxy-app/1.0-r0/git/examples/linux/rpmsg-proxy-app) pointed to by the S variable doesn't exist - please set S within the recipe to point to where the source has been unpacked to
WARNING: rpmsg-mat-mul-1.0-r0 do_unpack: rpmsg-mat-mul: the directory ${WORKDIR}/git/examples/linux/rpmsg-mat-mul (/scratch2/fray/git/openamp/build-honister/tmp/work/core2-64-poky-linux/rpmsg-mat-mul/1.0-r0/git/examples/linux/rpmsg-mat-mul) pointed to by the S variable doesn't exist - please set S within the recipe to point to where the source has been unpacked to
WARNING: rpmsg-echo-test-1.0-r0 do_unpack: rpmsg-echo-test: the directory ${WORKDIR}/git/examples/linux/rpmsg-echo-test (/scratch2/fray/git/openamp/build-honister/tmp/work/core2-64-poky-linux/rpmsg-echo-test/1.0-r0/git/examples/linux/rpmsg-echo-test) pointed to by the S variable doesn't exist - please set S within the recipe to point to where the source has been unpacked to
S is being set to:
S = "${WORKDIR}/git/examples/linux/rpmsg-echo-test"
The git checkout is empty, so your pull requests to git://github.com/OpenAMP/openamp-system-reference need to be approved before this can be merged.
Finally the SRCREV is set to 'AUTOREV' which will cause the system to pull data from GitHub on EVERY build.  Whatever the 1.0 release is, is what the SRCREV should point to.  Autorev will cause problems.

I have reworded this PR as new PR#28.
I belive I have addressed @mhatle's concerns above.
I am closing this one in favor of that one.

authors: TanmayShah-xilinx bentheredonethat mhatle wmamills

repo: OpenAMP/meta-openamp


text:
Remove unused logo file from "Hello OpenAssetIO" notebook

Closes #85. The logo file in the hello_openassetio resources directory was the deliberately misspelled logo from the querying_entity_versions notebook. Also, the logo file itself is never used within the notebook.
So remove the logo file, and use an obvious dummy path to the logo for exemplification purposes instead.

Path is relative errors on the new URL in tests. ... I wonder if we can be more descriptive about this, I got it a lot doing the BAL work and it's hard for the layperson to understand what about the path makes it relative, as it's normally to do with the leading slashes.

Path is relative errors on the new URL in tests. ... I wonder if we can be more descriptive about this, I got it a lot doing the BAL work and it's hard for the layperson to understand what about the path makes it relative, as it's often to do with the leading slashes.

It's tricky to find a URL/path that works cross-platform. file:///c:/path actually works on both platforms, but /c:/path doesn't "look right" on POSIX.  So now I've (force-) pushed an update that pins to PathType.kPOSIX.
There are a few places where we could be more specific about the reason for it being parsed as a relative path (see usages of kErrorRelativePath). Would involve updating every test in the massive test database, though, so it's not a 2 min change.

authors: elliotcmorris feltech

repo: OpenAssetIO/OpenAssetIO-MediaCreation


text:
remove warning by updating action versions

You are using "pypa/gh-action-pypi-publish@master".
The "master" branch of this project has been sunset and will not receive any updates, not even security bug fixes.
Please, make sure to use a supported version. If you want to pin to v1 major version, use "pypa/gh-action-pypi-publish@release/v1".
If you feel adventurous, you may opt to use use "pypa/gh-action-pypi-publish@unstable/v1" instead.
A more general recommendation is to pin to exact tags or commit shas.

Lgtm but i will let @ConorMacBride or @astrofrog  merge it from not-my-phone

I also updated the versions for other actions

Uh oh it broke

Can we run dependabot on this repo to keep these up to date?

Can we run dependabot on this repo to keep these up to date?

See https://github.com/OpenAstronomy/github-actions-workflows/pull/69 and https://github.com/OpenAstronomy/build-python-dist/pull/14

authors: Cadair ConorMacBride nabobalis

repo: OpenAstronomy/github-actions-workflows


text:
有兴趣可以加一下多模态群，大家一起交流多模态技术实战中遇到的问题

是否已有关于该错误的issue或讨论？ | Is there an existing issue / discussion for this?

[X] 我已经搜索过已有的issues和讨论 | I have searched the existing issues / discussions

该问题是否在FAQ中有解答？ | Is there an existing answer for this in FAQ?

[X] 我已经搜索过FAQ | I have searched FAQ

当前行为 | Current Behavior
如果二维码失效，可以加微信拉入群：yx116169

期望行为 | Expected Behavior
0
复现方法 | Steps To Reproduce
No response
运行环境 | Environment
0

备注 | Anything else?
0

Thank you

repo: OpenBMB/MiniCPM-V


text:
请问支持多言语吗？支持哪些语言？

非常感谢回复

支持中英文

authors: jctime namedzxy

repo: OpenBMB/MiniCPM


text:
Feature request  / 功能建议
对模型的性能很期待，期望能提供使用ollama部署的方法。

我们正在积极和llama.cpp 以及ollama合作，尽早支持ollama调用。感谢关注！
We are actively collaborating with llama.cpp and ollama to support ollama calls as soon as possible. Thank you for your attention!

We have supported llama.cpp, see this section. Ollama still have some issue.
我们已经支持了llama.cpp, 见这个章节. Ollama仍然有一些问题

Could you try this one? https://ollama.com/roger/minicpm

不正常输出

authors: JasonGuoo ShengdingHu yuanjie-ai


text:
Adding API doesn't seem to work

I added API exactly as per example, yet I have the response of model like this:
Thought:
Action: get_hello_world_for_hello_world
query to Customized-->hello_world-->get_hello_world_for_hello_world
Action Input: {}
Observation: {"error": "request invalid, data error. status_code=500", "response": ""}

It looks like it tries to get some info from remote, despite a completely locally defined API. How can I fix it?

Hi, can you provide the bash command you used? To avoid getting the api response from our toolbench server, we need to pass the --use_rapidapi_key  and --rapidapi_key ${rapidapi_key} parameters in the command, like this:
export PYTHONPATH=./
python toolbench/inference/qa_pipeline.py \
    --tool_root_dir data/toolenv/tools/ \
    --backbone_model toolllama \
    --model_path ToolBench/ToolLLaMA-7b \
    --max_observation_length 1024 \
    --observ_compress_method truncate \
    --method DFS_woFilter_w2 \
    --input_query_file /path/to/your/query/file \
    --output_answer_file /path/to/your/output/file \
    --rapidapi_key $RAPIDAPI_KEY \
    --use_rapidapi_key```

您好 ，如果用server启动的化支持自定义api吗？

Hi, can you provide the bash command you used? To avoid getting the api response from our toolbench server, we need to pass the --use_rapidapi_key and --rapidapi_key ${rapidapi_key} parameters in the command, like this:

Hello, thank you for your answer! I actually want a totally local example, as in your 'hello world' example. But as I dig through code, it looks like it will call for either one or another API server anyways. Can I at least use several?

Hi, sorry for the late reply. Before we have some bugs in api customization. Now we believe that the bugs have been fixed and it should work now. Please try again and feel free to open this issue again!

Hi @MartSlaaf did you manage to get a custom tool running, if so could you share your code? I'm quite confused how to use this repo as there's no example code or notebooks

Hi @mattdeeperinsights,
I just modified the rapidapi_wrapper class at inference/Downstream_tasks/rapidapi.py to be able to execute my custom functions. This is a somewhat dirty hack, but it works.
Personally, I've started to replace it with my custom environment altogether, but yet again, very specialized one, not a general purpose.

authors: MartSlaaf lyn233 mattdeeperinsights pooruss

repo: OpenBMB/ToolBench


text:
http://localhost:8082/status-all 这是个什么接口

http://localhost:8082/status-all  这是个什么接口？

这个接口用来查询模型推理或者应用空间的运行状态。如果本地没有启动模型推理或运行应用空间的需求，可以忽略。

authors: Rader liminngs

repo: OpenCSGs/CSGHub


text:
Slide-in Kloppen gegevens niet

This issue is imported from pivotal - Originaly created at Nov 15, 2017 by Thijs Kinkhorst
Zie bijlage voor tekst.
IdP support contact wordt weergeven.

(Thijs Kinkhorst - Nov 15, 2017)

repo: OpenConext/OpenConext-engineblock


text:
make text marking slightly smoother

When a typo is currently found, the styles that are applied to it feel quite sudden, perhaps they can be done slightly smoother (maybe through a quick transition or so).
Please note that this issue is only about making the feel of the styling a bit more natural/smoother, and no logical change should be made to the markings for now.

@AndiBraimllari are you referring to these boxes here on the side. If so would a simple fade-in effect be enough

I'm referring to the markings in the editor (left hand side), e.g.

After a request is made, the markings in the editor are shown right away, maybe there's a way of doing it a bit smoothly.

oh, i understand now. The only thing i can think of right now, is adding some animation to it like a color fill that pre-fills from left to right, or maybe a fade in.
https://github.com/OpenCovenant/quill/assets/57508905/6cc97b55-46d6-45f0-b52b-4768b751183b

Yup, I think something like that (perhaps the fade-in can be a bit faster) would be pretty cool.
What do you make of it, my reasoning is that the way the markings are applied to the text should feel natural (which is why I like some sort of brief animation to it).
If you'd like, you can add these changes in a PR and we can further discuss them there.

@AndiBraimllari Sounds good, i'll go ahead and start the process and make the changes for it.

@AndiBraimllari I agree, there’s nothing wrong with how it was before, but you’re right, adding some sort of secondary effect makes the entire experience feel different.

repo: OpenCovenant/quill


text:
Specialize cetl::type_id_value for built-in types and cv-qualified types

Enabling the CETL RTTI capability for built-in types will enable their use with cetl::any.
Pseudocode:
template <typename T>
constexpr type_id type_id_value = T::_get_type_id_();  // This is already defined in CETL.
template <> constexpr type_id type_id_value<bool>{{<UUID>}};
template <> constexpr type_id type_id_value<char>{{<UUID>}};
template <> constexpr type_id type_id_value<signed char>{{<UUID>}};
template <> constexpr type_id type_id_value<unsigned char>{{<UUID>}};
template <> constexpr type_id type_id_value<int>{{<UUID>}};
template <> constexpr type_id type_id_value<unsigned int>{{<UUID>}};
...

ALSO, we need specializations for cv-qualified types that implicitly remove the cv-qualifiers, similar to variant_alternative_t et al.

We'll also require specializations of is_instance_of and rtti_cast for the case where has_type_id is true while is_rtti_convertible is false.

repo: OpenCyphal/CETL


text:
It is no longer possible to change the data type of a publisher

It used to be possible but not anymore. Changing the data type when the publisher is already running is unnecessary, but it shall be possible to change it before the publisher is initialized.

It was never possible, just an illusion.

authors: pavel-kirienko silverv

repo: OpenCyphal/yukon


text:
Adicionar tutorial de Entrada e Saída de dados em prog1

É preciso fazer um tutorial ensinando o  básico de entrada e saída de dados, já que é um assunto ensinado em Programação 1.
O que deveria ser abordado:
raw_input(), input(), print() e etc...
Este tutorial deve ser feito em Python Notebooks, e colocado na pasta tutoriais/

@thayannevls , quem vai fzer esse? @LukeHxH ?

@jonathantvrs essa task tá livre, ninguém pegou até agora 😄

então coloca em andamento, vou fazer.

authors: jonathantvrs thayannevls

repo: OpenDevUFCG/Tamburetei


text:
fix up json parsing in MonologueAgent

This extracts out the cleanup logic so it's more consistent
Fixes https://github.com/OpenDevin/OpenDevin/issues/718

Thanks for doing this. There's another here: https://github.com/OpenDevin/OpenDevin/blob/911bbe6eca2eb2327bbeae6702ca5dddcdab375b/agenthub/monologue_agent/utils/monologue.py#L77
I think that one should(tm) be already useless

Thanks for doing this. There's another here:
https://github.com/OpenDevin/OpenDevin/blob/911bbe6eca2eb2327bbeae6702ca5dddcdab375b/agenthub/monologue_agent/utils/monologue.py#L77
I think that one should(tm) be already useless

I think it serves for different purposes from Robert's PR. stripping markdown and ensuring valid json. I don't know if I misunderstand sth.

authors: JustinLin610 enyst rbren

repo: OpenDevin/OpenDevin


text:
Repeatedly getting [ERROR] Evaluation failed. 'choices' on test server.

Hi,
I repeatedly get the error [ERROR] Evaluation failed. 'choices' on the test server, and many times, all 3 of my submissions get wasted due to this. Can this please be addressed?
Many thanks

Some of the openai key have no quota. We are replacing them soon.

Fixed. Please try to submit again.

authors: ChonghaoSima ayesha-ishaq

repo: OpenDriveLab/DriveLM


text:
编码器问题

实验中第一阶段用到的depth和pose 编码器是resnet 结构，第二阶段motionet 编码器也是resnet 结构，二者的设定有相关性吗？我理解第二阶段的视觉编码器应该是根据自己需求选用的吧？thanks

Yes, you can use other vision encoder structures in the second stage.

authors: Bobo-y WPH-commit

repo: OpenDriveLab/PPGeo


text:
[FS Overview] Check box in Factsheet overview - what for?

Description of the issue
The factsheet index page which displays all factsheets has a checkbox on the bottom right corner (depending on screen size). This checkbox cannot be clicked (Windows, Firefox) and it is not clear what functionality it should have.
I assume it might in the future help to choose several factsheets for a comparison.
Factsheet
Overview

Field
Name of the field, see Documentation
Ideas of solution
Delete checkbox as long as it does not come with functionality.

Yes, your assumption is correct. The goal for the checkboxes is the comparison, but the checkboxes are disabled because this functionality is still under development. I should remove the checkboxes in the next update...

This is outdated.

authors: adelmemariani han-f stap-m

repo: OpenEnergyPlatform/oekg


text:
update subclasses of has input / has output

closes #618

replace has numerical input by has information input, defined as p has information input c iff: p has input c, and c is an information content entity (analogous to the definition of has physical input).
add missing domains and ranges to the children of has input to further distinguish them
make has constraint a subproperty of has information input
do the same for has ... output

That's bad. I don't see an easy solution to this problem since has participant and its subclasses did always have the domain process in the definition, the only thing I changed was making it explicit in an axiom.
For now I removed the conflicting axioms so we can merge the rest of the PR and solve the inconsistency-problem in a new issue.

That's bad. I don't see an easy solution to this problem since has participant and its subclasses did always have the domain process in the definition, the only thing I changed was making it explicit in an axiom.
For now I removed the conflicting axioms so we can merge the rest of the PR and solve the inconsistency-problem in a new issue.

Agreed.

repo: OpenEnergyPlatform/ontology


text:
提示Error: Value must be an iterable.

我在轰炸时提示Error: Value must be an iterable.

https://github.com/OpenEthan/SMSBoom/issues/207

repo: OpenEthan/SMSBoom


text:
Binary not launching

Hi
I want to install OpenFAST v2.5.0 from binaries on Windows but the installer is not launching.
I downloaded OpenFAST v2.5.0 binaries zip. I experienced Issue #649, so I downloaded and installed the Intel® Fortran Compiler 19.1 for Windows. In this regard, I suggest updating the link in the documentation, as it currently directs to 2017 version.
However, if I now try to run the openfast_x64_double.exe (the same happens with openfast_x64.exe), even as administrator, it does not launch. For less than a second the mouse shows the system is busy but then nothing happens. Tried to restart the computer but didn't work.
I am using Windows 10 Home.
Thanks in advance

Hi @Seinacio,
When you say that you tried to run OpenFAST but "nothing happens," what do you mean? Are you running OpenFAST from a command prompt or by double clicking on it? See, e.g., https://github.com/OpenFAST/openfast/issues/463.
Best regards,

Hi @jjonkman, that solved my problem, thank you.

authors: Seinacio jjonkman

repo: OpenFAST/openfast


text:
[WIP] - Torsion Distribution plotting/analysis

This PR contains the Torsion Distribution plotting/analysis for for openFE trajectories.
Features:

read-out alchemical states from the trajectories
find all rotatable bonds
calculate trajectory torsions
plot torsion distribution

Todo:

[x] Initial running version for torsion Distribution plotting
[ ] Initial  Compare Torsion Distributions
[ ]  Test Code on more complex systems
[ ] Implement Tests
[ ] Refactor Code
[ ] Write Docs

More Ideas? :)
Example plots:

Hello @RiesBen! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file src/openfe_analysis/torsions_distributions.py:

Line 20:1: E302 expected 2 blank lines, found 1
Line 20:47: E225 missing whitespace around operator
Line 32:42: E225 missing whitespace around operator
Line 46:80: E501 line too long (87 > 79 characters)
Line 56:80: E501 line too long (92 > 79 characters)
Line 59:5: E731 do not assign a lambda expression, use a def
Line 59:80: E501 line too long (84 > 79 characters)
Line 67:80: E501 line too long (93 > 79 characters)
Line 80:80: E501 line too long (116 > 79 characters)
Line 81:80: E501 line too long (116 > 79 characters)
Line 86:1: W391 blank line at end of file

In the file src/openfe_analysis/utils/__init__.py:

Line 2:1: W391 blank line at end of file

In the file src/openfe_analysis/utils/lassoo_highlights.py:

Line 2:80: E501 line too long (101 > 79 characters)
Line 7:1: E265 block comment should start with '# '
Line 12:1: E402 module level import not at top of file
Line 13:1: E402 module level import not at top of file
Line 14:1: E402 module level import not at top of file
Line 15:1: E402 module level import not at top of file
Line 16:1: E402 module level import not at top of file
Line 17:1: E402 module level import not at top of file
Line 18:1: E402 module level import not at top of file
Line 19:1: E402 module level import not at top of file
Line 20:1: E402 module level import not at top of file
Line 24:80: E501 line too long (87 > 79 characters)
Line 35:80: E501 line too long (115 > 79 characters)
Line 67:80: E501 line too long (117 > 79 characters)
Line 99:80: E501 line too long (94 > 79 characters)
Line 105:13: E303 too many blank lines (2)
Line 108:80: E501 line too long (86 > 79 characters)
Line 123:80: E501 line too long (88 > 79 characters)
Line 129:80: E501 line too long (102 > 79 characters)
Line 139:80: E501 line too long (94 > 79 characters)
Line 141:80: E501 line too long (85 > 79 characters)
Line 143:80: E501 line too long (82 > 79 characters)
Line 148:80: E501 line too long (81 > 79 characters)
Line 151:80: E501 line too long (104 > 79 characters)
Line 153:80: E501 line too long (94 > 79 characters)
Line 161:80: E501 line too long (93 > 79 characters)
Line 167:80: E501 line too long (105 > 79 characters)
Line 170:80: E501 line too long (107 > 79 characters)
Line 176:80: E501 line too long (82 > 79 characters)
Line 198:80: E501 line too long (85 > 79 characters)
Line 221:80: E501 line too long (83 > 79 characters)
Line 257:80: E501 line too long (104 > 79 characters)
Line 258:80: E501 line too long (105 > 79 characters)
Line 259:80: E501 line too long (80 > 79 characters)
Line 260:80: E501 line too long (80 > 79 characters)
Line 265:80: E501 line too long (120 > 79 characters)
Line 275:80: E501 line too long (93 > 79 characters)
Line 281:80: E501 line too long (114 > 79 characters)
Line 287:80: E501 line too long (107 > 79 characters)
Line 293:80: E501 line too long (91 > 79 characters)
Line 333:54: W292 no newline at end of file

In the file src/openfe_analysis/visualization.py:

Line 3:13: E222 multiple spaces after operator
Line 4:19: E127 continuation line over-indented for visual indent
Line 21:80: E501 line too long (89 > 79 characters)
Line 50:80: E501 line too long (110 > 79 characters)
Line 50:110: E703 statement ends with a semicolon
Line 67:80: E501 line too long (94 > 79 characters)

Original Task description #11 :
Type
System sampling check
Question
Is phase space well sampled?
How
Minima sampled ~60%
Tool
MDA + rdkit
Note
Ideally what you should be looking at here is a combination of the solvent, vacuum, and complex differences and the variation in dihedral sampling across lambda states. I.e. is there a major change happening in my transformation that isn't being sampled across all states -- @IAlibay

Those plots look really nice :star_struck:

@RiesBen  Remove the hydrogens.

authors: RiesBen mikemhenry pep8speaks


text:
New datasets used for videochat2_mistral

Very happy to hear that you have updated the model with mistral LLM. Is there any place to find the newly added datasets? Thanks!

Good question! In the current version, only the smit is added. And we have updated the instruction data here.

Thanks for your reply! I noticed that some datasets like image caption -- coco has been reduced to a smaller size (100k). I'm curious about the reason and whether it is because the length of captions are relatively short? Thanks

OK, it's great to train faster with the same performance :)

repo: OpenGVLab/Ask-Anything


text:
OSD not working

openipc 2.2.11.04
.osd.enabled: true
.osd.font:
.osd.template: %F %T
.osd.corner:
.osd.posX: -16
.osd.posY: -16
.osd.privacyMasks:

wahtever settings I use, the OSD is never shown.

log says user.err majestic[922]: TTF font '/usr/share/fonts/truetype/UbuntuMono-Regular.ttf' load failed
so I downloaded a ttf file to /root, configured it in OSD and a works fine.
issue could be closed.
BTW: the autonight script doesn't seems to work right, when changing a setting and majestic restarts. it stays in day mode, but the cam is in a dark room.

For the problem with auto night, please open a new issue.


text:
BMP safety for big files and int32 overflow for offsets, and speedups

Fix places that could overflow int32.

Speed up read/write by about 40%, mostly by avoiding the pointless
zeroing out of allocated memory in cerain std::vector temporary buffers.

By the way, the background here is that after finding that bug in openexr deep files with large files, I wrote a test harness that creates, writes, reads (and checks that everything is still the same) a > 4GB file for each format we support. I'm saving the PR for that test until the end. But it identified problems in almost every file format, which I spent last weekend fixing one by one and now am doling out the PRs individually. For most of these formats, I wasn't the person who wrote them, and many of them are not the important formats we use daily in VFX. So quite often I was purposely just trying to make the minimally intrusive fix to make them demonstrably big-safe.

repo: OpenImageIO/oiio


text:
Dev Mode: Add integration tests

Add gradle equivalents of dev mode integration tests https://github.com/OpenLiberty/ci.maven/tree/master/liberty-maven-plugin/src/it/dev-it

WIP tests: https://github.com/ericglau/ci.gradle/tree/devBasicTest. Windows tests are failing, possibly due to #397 and #402


text:
🛑 LinkOS K8S Registry Mirror is down

In a0f7458, LinkOS K8S Registry Mirror (https://k8s.linkos.org) was down:

HTTP code: 0
Response time: 0 ms

Resolved: LinkOS K8S Registry Mirror is back up in d462f2d after 1 hour, 47 minutes.

repo: OpenLinkOS/uptime


text:
Is it a bug within xztl_media_check()?

`static int xztl_media_check (struct xztl_media *media)
{
struct xztl_mgeo *g;
/* Check function pointers */
if (!media->init_fn)
return XZTL_NOINIT;

if (!media->init_fn).  --------------------------------> Does it should be 'if(!media->exit_fn)'?
return XZTL_NOEXIT;

if (!media->submit_io)
return XZTL_MEDIA_NOIO;

if (!media->zone_fn)`

yes, correct, I'll submit a PR to fix it

it's fixed in v0.3 release

repo: OpenMPDK/xZTL


text:
FML: Mod could not be found OpenMods: [0.7,0.8)

I just updated my personal modpack and this thing popped up.
Error line from the console:
[19/9/2015 13:27:57 PM] [Client thread/ERROR] [FML]: The mod OpenPeripheral (OpenPeripheralAddons) requires mod versions [OpenMods@[0.7,0.8)] to be available
EDIT: I believe this is just a typo in the @Mod annotation

oops it should be in OpenMods/OpenPeripheral-Addons

repo: OpenMods/OpenModsLib


text:
[FSunstone] Groups Tab

Description
Implement Groups tab in FireEdge Sunstone.
Related to #5422

Progress Status

[ ] Code committed
[ ] Testing - QA
[ ] Documentation (Release notes - resolved issues, compatibility, known issues)

one-6.8 pending

repo: OpenNebula/one


text:
Worked on modernizing asynchronous python calls and resolving frontend errors

Summary

Switched over to async await instead of using
asynchronous generators.
Cached User Articles on Addition to allow frontend to reflect new
changes
Made nSubjects an optional parameter in the database
General Style updates (using static methods where appropriate) and removing unnecessary imports

Test Plan
Replicated errors received on frontend including
https://brainspell.herokuapp.com/json/v2/edit-global-article?github_token=b38a4968302c2948a9e5275c88de298c6107bb01&key=8a54a6d2574394f0a6d50b37d2a67f955196ae98&pmid=18367612&edit_contents={%22experiments%22:[{%22id%22:90000,%22caption%22:%22Areas%20showing%20greater%20activation%20for%20high-conflict%20trials%20than%20low-conflict%20trials,%20on%20trials%20where%20no%20response%20was%20required.%20The%20top%20half%20of%20the%20table%20describes%20activity%20as%20a%20result%20of%20the%20direct%20
and
https://brainspell.herokuapp.com/json/v2/edit-local-article?collection_name=naturalistic&github_token=b38a4968302c2948a9e5275c88de298c6107bb01&key=8a54a6d2574394f0a6d50b37d2a67f955196ae98&pmid=18367612&key_value_pairs={"90000":[]}&exclusion_reasons={"90000":"excluded"}

Verified these errors are now resolved.
Further tested the add-to-collections page to verify frontend clearly reflected newly added articles.
Also verified changes persist to DB and persist across refreshes of the page.

General Comments
Tornado recommends the use of native async over coroutines:
https://www.tornadoweb.org/en/stable/guide/coroutines.html

This looks good to me - good call on removing Python 3.4

authors: Sharabesh neelsomani

repo: OpenNeuroLab/brainspell-neo


text:
Pose Annotation not working in Single Camera without people tracking & Visualization Issues

We found out that Pose Annotation  does not work in Single Camera without the person tracking flag enabled.
So instead of :
roslaunch tracking single_camera_tracking_node.launch enable_pose:=true enable_people_tracking:=false enable_object:=false
we have to use:
roslaunch tracking single_camera_tracking_node.launch enable_pose:=true enable_people_tracking:=true enable_object:=false
I also had some Rviz visualization issues  in the Manual installation version of OPT while running Pose but I think that has to do with the options I was selecting and not the Annotation itself.

@SimeonOA it may be resolved - can you test?

Tested in 18.04 Docker using Kinect v2 and in 16.04 Docker using realsense.
Now works in both cases.
Will update wiki to reflect.

authors: SimeonOA jburkeucla


text:
Uncaught TypeError causing dozens of e2e test failures

When I open OpenRefine in the browser, there's an uncaught TypeError getting logged to the console. While it may seem innocuous, it's causing lots of e2e test to fail.
To Reproduce
Steps to reproduce the behavior:

Open OpenRefine, navigate to http://127.0.0.1:3333
Open Console in Developer Tools

or run refine e2e_tests in the terminal
Current Results

I just finished running the suite of e2e tests and found this issue causing more than 40 tests to fail. Some are triggered by cy.VisitOpenRefine().

Expected Behavior
I'd expect these tests to pass and for this error not to be logged to the concole.
Versions

Operating System: macOS 114.2.1 (23C71)
Browser Version: Chrome Version 124.0.6367.62 (Official Build) (arm64)
JRE or JDK Version: OpenJDK 64-Bit Server VM (build 19.0.2+7-44, mixed mode, sharing)
OpenRefine: Version 3.9-SNAPSHOT [a6eef0c]

Additional context
Seems like this issue stems from OpenRefineVersion.module_names being undefined at this line:
https://github.com/OpenRefine/OpenRefine/blob/c6f1b07b8bd7cbfbe923acfc962ae34196c2ade9/main/webapp/modules/core/scripts/index.js#L93

This is likely caused by switching between different branches which did or did not include the change to package classes as .jar files instead of bare .class files. Sadly running ./refine clean does not clean up any .class files left over (because the new configuration does not produce them there, so mvn doesn't see it as needing cleaning up).
Assuming you don't have any important untracked files in your clone, you could run git clean -fdx to remove them all, and run OpenRefine again, which should hopefully solve the problem.

You also may just need to refresh your server build. The errors are coming from something added relatively recently which requires support on the backend.
The end to end tests are run on every PR, so they are unlikely to be broken without anyone noticing.

Thanks @wetneb and @tfmorris. Running git clean -fdx fixed it. I thought I was going crazy. I kept pulling from the upstream master branch and running ./refine clean.

Another neat git command that I particularly use and can help (depending on if you are OK to delete untracked files) is git reset @{u} --hard which helps HEAD get back to a previous state quickly.  Besides --hard there's other modes, like default --mixed, depending on what you need.  But if you want to blow away changes and reset the branch to match upstream @{u} then it's a great way.

--hard
Resets the index and working tree. Any changes to tracked files in the working tree since  are discarded. Any untracked files or directories in the way of writing any tracked files are simply deleted.
https://git-scm.com/docs/git-reset#Documentation/git-reset.txt---hard

authors: steve-kasica tfmorris thadguidry wetneb

repo: OpenRefine/OpenRefine


text:
Updating codebase links

Changing obsolete links to the github wiki to the new docs.
I'm sure I haven't caught them all yet, and there is no rush on this just yet.

I should note that the three wiki pages here that are technically deprecated should be retained because older versions of OpenRefine (& older changlog pages) will continue to link to them. (History, Back Up Your Data, & the Documentation for Users homepage).
Those pages can be blanked + turned into manual redirects if you'd like, or they can stay the same with a warning+link at the top.

I should note that the three wiki pages here that are technically deprecated should be retained because older versions of OpenRefine (& older changlog pages) will continue to link to them. (History, Back Up Your Data, & the Documentation for Users homepage).
Those pages can be blanked + turned into manual redirects if you'd like, or they can stay the same with a warning+link at the top.


text:
Proper submodule commands need to be added for setup

The Setup development environment and deployment instructions don't include instructions for initializing and updating the submodules (kibana).
I believe all that needs to be run is -
git submodule update --init --recursive

Or you could instruct users to clone the repo using the command -
git clone --recursive

Done. The README.md has been updated.

authors: RyPeck rammie

repo: OpenSOC/opensoc-ui


text:
OSTPrime's wrap should not have an amount argument

When wrapping on OSTPrime, then the amount should be msg.value and not additionally provided as a function argument _amount.

This is already done in https://github.com/OpenSTFoundation/mosaic-contracts/pull/533

authors: deepesh-kn schemar

repo: OpenSTFoundation/mosaic-contracts


text:
Hash lock secret being emitted

https://github.com/OpenSTFoundation/openst-protocol/blob/334b3edffb8e43004935640661c2954ca20fe3fa/contracts/OpenSTValue.sol#L49:50
Please excuse me if I'm being a bit naive here :) The hashlocking used in Simple Token seems to share some principles outlined here for atomic cross-chain trading:
https://en.bitcoin.it/wiki/Atomic_cross-chain_trading
In this case, revealing the hashlock too soon would mean that the second party could run off with the money. Is there a similar attack vector here, as _unlockSecret is emitted by the ProcessedStake event?

Hi @addywaddy
The similarities are indeed there, in the sense that the protocol creates an atomic action on two independent systems; contrary though to the referred algorithm, there is only one party involved here:  a user wants to stake $OST on Ethereum mainnet in order to obtain the equivalent value in utility tokens on the auxiliary utility chain.  When calling ProcessStaking or ProcessMinting on either side (Ethereum mainnet or the utility chain) indeed the secret is revealed for all to know - but this is the intention to have the atomic action: confirming on one side equals confirming on the other side.
So as a user, until you call any Process_ function you want to protect your secret; revealing it once, it should be known to all, so we decided to put it in the event explicitly.  The platform APIs are built to abstract these actions for an (honest) actor

authors: addywaddy benjaminbollen

repo: OpenSTFoundation/openst-protocol


text:
Blog Post: Navigating Donations to OpenSats

WIP
See also:

https://github.com/OpenSats/website/pull/134#issuecomment-1829633160

I'll close this for now; maybe we revisit it at some point in the future.

repo: OpenSats/website


text:
rest cache

open for discussion. I make this pull request ready when this way of caching is reviewed.
When a retrieve-view from a ModelViewSet is called for the first time, the cache is created and returned on later request of the same view.
When a model is changed, the cache is also recreated. For that, all Serializers of the root_rest_element are used.
To know the list of Serializers for a root_rest_element, I tag the serializers with a decorator. An alternative would be to set the list of serializers manually in the model.

Possibly restricting the use of viewset and serializer features seam to be the only way to achive our caching goals. Therefor we need a detailed documentation. Maybe we also can do it less complex (not connecting cache keys to models and the like).

This has to be redone anyway...

authors: normanjaeckel ostcar

repo: OpenSlides/OpenSlides


text:
I2C Analog ADS1219 Addon Support

This code enables the I2C Analog ADS1219 add-on board support created by Fortinbra. The board uses i2c communication and talks to A0/A1/A2/A3 analog pins, which I currently map to left-stick X/Y, and right-stick X/Y respectively.
Future enhancements will include selection of A[x] pin maps to Analog controls and more refined options for the ADS1219 chip.

Tested and working well

authors: TheTrainGoes arntsonl


text:
191 user migration

This builds on #126.
Adds a migration command to migrate user accounts. The JSON is in the following format:
{
  "UID":{
    "uid":"UID",
    "name":"NAME",
    "mail":"NAME@example.com",
    "roles":{
      "2":"authenticated user",
      "4":"proposer"
    },
    "field_otf_real_name":[]
  },
  "UID1":{
    "uid":"UID1",
    "name":"NAME",
    "mail":"NAME@example.com",
    "roles":{
      "2":"authenticated user",
      "4":"administrator"
    },
    "field_otf_real_name": {
      "und":[{
        "value":"First Last",
        "format":null,
        "safe_value":"First Last"
      }]
    }
  }
}

Rebased and ready


text:
copy over PR feature-30-container-bugs, with app-3-0 base. + slot sharing must be explicit

Description:
Same as https://github.com/OpenTrons/opentrons/pull/258, but using app-3-0 as a base.
Plus:

Small documentation cleanup
#292

Lowercase Pipette axis arg by default
#315

Placing multiple containers in a slot must be explicitly written by user. This achieved through the kwarg share= on containers.load()

containers.load('96-flat', 'B2')
containers.load('trough-12row', 'B2', share=True)

Placing two containers of same type into the same slot requires at least one of them have a unique name:

containers.load('96-flat', 'B3')
containers.load('96-flat', 'B3', 'other-plate', share=True)

Breaking changes
New restrictions for shared slots (detailed above).
Deck.containers now returns a list, not a dict.
No more robot.containers(), instead use robot.get_containers()
Remove unused "private" method _get_all_containers(robot): from api/opentrons/util/state.py

Check List:

[ ] Are there breaking changes in the API and how are they being communicated?
[x] Update containers documentation (docs.opentrons.com) to reflect the change
[x] Who is reviewing your code?

LGTM 👍

@astaff @mcous
The make docs is passing now,
build is failing at The command "make -C app -j 2 build test && make -C app package" exited with 2. -- just like the base branch, app-3-0.
Good to merge into app-3-0 now?

app-3-0 should not be failing the build. Looks like #318 broke the build; my bad for not catching that in my review. @Kadee80 or @IanLondon can you fix the CSS lint errors introduced in app/ui/components/UploadPanel.css? I'm 👍 once that is done

@mcous on the lint error now

@mcous linting errors fixed, please review again

Codecov Report

:exclamation: No coverage uploaded for pull request base (app-3-0@3be8463). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff             @@
##             app-3-0     #324   +/-   ##
==========================================
  Coverage           ?   81.06%
==========================================
  Files              ?       62
  Lines              ?     4393
  Branches           ?        0
==========================================
  Hits               ?     3561
  Misses             ?      832
  Partials           ?        0

Impacted Files
Coverage Δ

api/opentrons/util/state.py
87.5% <ø> (ø)

api/opentrons/containers/placeable.py
96.95% <100%> (ø)

api/opentrons/instruments/pipette.py
98.11% <100%> (ø)

api/opentrons/robot/robot.py
87.23% <100%> (ø)

Continue to review full report at Codecov.

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

authors: IanLondon Kadee80 astaff codecov-io mcous

repo: OpenTrons/opentrons


text:
Allow copying and pasting curve data

Would it be possible to make it easy to enter curve data by simply copying and pasting data ranges?

I believe this is something could be done relatively easily - I've labeled and updated the title of this issue.

authors: camillenze lbutler

repo: OpenWaterAnalytics/epanet-gui


text:
[help] Strange entities created

I’m try to transition to this system and I have managed to configure the component on a RPi3 running HASSIO. I use Aeotec devices along with the Aeotec Gen5 gateway. It has found all my devices however my Aeotec Smart Switch plug sockets don’t seem quite right.
For example, a couple of the entities are named very strangely such as “sensor_gas_density" and “sensor_generic” and "sensor_water" instead of Voltage and Curren and "power_1," "power_2," and "power_8" for Wattage, seconds and kWh.
The Units of measurements are mostly correct but labelling is off. It also seems to think it has light function too (or a a light switch) as there is a switch to turn/off and and dim an RGB light.

Please check FAQ question 2

authors: daanvb robertsLando


text:
Remove Babel

We no longer use import :tada:
Fixes #1062

do we use jshint at all, actually?

LGTM after rebase @nventuro

that would be nice, since my not a Node.js expert and currently scratching my head trying to use the helper scripts for test but import and export don't work...

@sweatyc what do you mean? All helpers are imported using require now that this PR has been merged, a testing guide with information on how to use them, example, etc., is on the way (#1077).

authors: nventuro shrugs sweatyc

repo: OpenZeppelin/openzeppelin-solidity


text:
[BUG] Skills imported multiple times despite the match

Steps:
1 Imported whole profile (tree+jewels and items+skills)
1.1 Saved changes
2 On skills tab I have following skill chain

3 Import same profile once again (only items+skills), skill chain duplicated (same gems, same level and quality, same order, same item)

Additional info: character mommysPIZZAMAKER

The code that checks if a skill chain is already present in a build was being tripped up by a difference in capitalisation: the program's gem database had "Curse on Hit" whereas the gem's actual name is "Curse On Hit". I love bugs like this :). This is fixed in the next update. Thanks for the report!

repo: Openarl/PathOfBuilding


text:
Poison dps is not correctly calculated

With the 3.0 patch the way poison damage is scaled changed: many weapon nodes now add "increased damage with ailments" According to the Wiki  ailments include poison, bleeding, ignite, chill, freeze and shock, however selecting those nodes incorrectly does not actually show an increase in bleeding, poison or burning dps. I assume it will also not be factored into freeze/shock calculations, if there are any.
It would also be great to see the total "+damage with ailments" in the menu on the left.

I just tested with v1.4.59 and the Claw specific nodes near the shadow start at least work correctly, can you upload a pastebin for a build with non-working nodes?

I can't reproduce this; all of the Damage with Ailments nodes seem to be applying correctly when using the appropriate weapon type. Could you give me your build, and also point out some specific nodes that don't appear to be applying?

https://pastebin.com/vjcr1t2G is a build with a broken 100% to poison claw (for simplicity on comparing poison damage), the poison nodes near shadow are applying properly but the claw cluster just to the left of the poison cluster and the claw wheel are not applying to poison in anywhere near as meaningful of a way as the poison cluster.

The modifiers to Damage with Ailments from those nodes are applying correctly; this is from the Total Increased under Poison in the Calcs tab:

Then why does going from 0 to 100%+ increased damage not change the relationship between total damage per poison and average damage by more than 1%? Are those stats not included in the Poison DPS and Total Damage per Poison on the statbar on the left? The only nodes I've found that actually change those stats are the pure poison nodes.

Ah, never mind, for a bit there I forgot GGG changed the way ailments work in 3.0...

Fair enough then.
Incidentally, it is mechanically impossible for modifiers to be listed like that in the Calcs tab, but not apply to the numbers shown there or in the stats in the sidebar, as it all uses the same code to determine which modifiers apply.

This is the snapshot of the poison damage. Ailment increase is in the list, but the damage is not calculated correctly.
MH + Chaos shows as 4802.6 (looks correct) * 0.20 = 960 dps. This value is shown in the poison dps section. This for once completely ignores the 670% increase and additionally does not include the 80% chance to poison, so it should actually be 960 * 7.7 * 0.8 = 5913.6 dps. Same goes for the bleed dps calculation.
The total dps including poison then however shows a 10k dps increase for no apparent reason.

Actually, the 670% increase has already been factored into the calculation of the "4802.6" number. And the Poison DPS number is just the average DPS of each Poison stack; your Poison chance is factored in later, during the calculation of Total DPS inc. Poison, which is only shown in the sidebar.

With the base DPS of ~14k of mostly physical damage and a 20% * 80% poison dps of that dps it should be roughly around 2.2k dps without any modifiers, respectively around 17k dps with modifiers. The real number is of course lower, but 0.9k dps just cannot be correct given the numbers.

Doing the complete calculation:
MH + Chaos average hit: 2460
Including crit: 2619.9
At 4.12 attacks/s: 10793.988 dps
80% chance to poison for 20% of that damage: 1727,03808 poison dps
670% increased poison damage: 13298,193216 poison dps total.
There is either an error in my calculation or in the tool. ;)

Oh boy. What you're doing is fundamentally incorrect, due to the Damage over Time changes made in 3.0. Poison's base damage is no longer based on the actual hit damage, but rather is derived using an entirely separate calculation. That's why the program now displays "MH Source Chaos" (and should also display "MH Source Physical", but that's bugged ATM), which is the output of that calculation; and that calculation already includes all modifiers that affect poison damage.

Tough to find some exact information on that. Closest I could find was this from Mark GGG:

Poison deals 20% of the applicable (usually physical + chaos) base and added damage of the skill, per second

I can't however find any clarification what that added damage is.

"Added damage" means modifiers like "Adds X to Y Physical Damage to Attacks".

Got a very detailed official explanation: https://www.pathofexile.com/forum/view-thread/1930909/page/4#p14741874

I removed some items from the build to make the calculation easier:
Bow damage: 79 - 294 physical
Physical added: 48 - 75 (support gem and gloves)
Applicable modifiers: 1.33 (Belt) * 1.94 (Maim + Physical Projectile Support) * 0.69 (Chain support) * 0.614 (Barrage base damage modifier) = 1.093 total modifier
Applicable MH base damage is: (79 + 294 + 48 + 75) / 2 * 1.093 = 271,064
This means poison base damage is: 27.8 - 80.7 damage
Damage modifier: 7.5 (poison/ailment passive skills) * 1.49 (physical projectile support) = 11.175
Crit effect mod: 1+ 13% * 50% = 1.065
Avg poison dps per poison: (27.8 + 80.7) / 2 *11.175 * 1.065 = 645.65 dps (Calcs shows 547.1 here)
Poison damage total per hit: 645.65 dps * 2.8s = 1807.82 damage (Calc shows 1531.9 here based the the different value above)
Max poison stacks: 3.97 (attacks) * 2.8s = 11.1 (Calc shows 8.9 here)
Maximum sustained poison dps: 11.1 * 645.65 = 7177.05 (Calc shows 4870.8 here, not sure if this includes the 1785,1 bleed dps already, calculating total dps incl poison - total dps)
The numbers differ, but I cannot tell why. If my calculation is correct, poison dps displayed in the tool would be off significantly.

Could you give me the exact build? It'd be much easier for me to check the calculations if I had it.

bow_debug.zip

You don't need to upload the build .xml file to share builds; go to the Import/Export Build tab, then in the Build Sharing section you can generate a share code, which you can then turn into a pastebin.com link.
You've made two major errors in your calculations:

Damage modifiers (like PPAD and Chain) do not apply during the calculation of the base damage, and in fact Chain doesn't apply to Poison at all.
When modifying the base damage, you included modifiers for Chaos and Poison damage; but modifiers to Physical damage also apply to the Physical portion of the base damage (including the modifier from Maim).

So the correct calculations are:
Base weapon damage: (79 to 294) * 0.61 = 48.2 to 179.3
Base added damage: (48 to 75) * 0.61 = 29.3 to 45.8
Poison base damage: (48.2 + 179.3 + 29.3 + 45.8) / 2 * 0.2 = 30.2
Relevant modifiers:
650% increased Poison + Chaos Damage
121% increased Physical Damage
49% more Poison/Bleed Damage (from PPAD Support)
30% more Physical Damage (from Maim Support)
Crit effect modifier: 1.065
30.2 * (1 + 6.5 + 1.21) * 1.49 * 1.3 * 1.065 = 542.6 Poison DPS (off by ~1%, but that's rounding error)
Max stacks: 8.9 (you calculated that correctly)
542.6 * 8.9 = 4829.4

Thank you for the detailed explanation.
I am beginning to realize how difficult it is to get the correct formula with everything based on best guesses and occasional official explanations. Hopefully they fix the bug with the current display in the character sheet soon, so there is an officially correct base to check calculations.
It seems to me that I was wrong and the poison dps is correctly calculated (although displayed in a confusing way in the left summary window). Which also - sadly - means that poison dps is currently a worthless waste of skill points.

authors: Openarl Yajirushi twothe


text:
feat(js): update Formik to v2

overview
Serves as its own ticket
We use Formik in several projects and it had a major v1 -> v2 release in October. The breaking changes are minimal: https://jaredpalmer.com/formik/docs/migrating-v2
What do we get? Some nice stuff
For the migration to v2, I found 2 changes that affected us:

resetForm args changed. This PR changes resetForm(stuff) to resetForm({values: stuff})
Avoid render prop, this is deprecated. In this PR, I used a child function instead. (Unfortunately they still don't provide a hook interface, you need to use <Formik> or withFormik(). Maybe you could do <Formik><MyFormComponent /></Formik> and in MyFormComponent you could use useFormikContext hook? See docs)

changelog

Update Formik to latest v2.x.x
Migrate forms
Install flow-typed defs for Formik 2.x.x and use them in our Formik forms! This led to some fixes b/c before much of the forms were implicitly any-typed.

review requests

Code review
Please make sure the forms I touched still work correctly!

Run App

[ ] app/src/components/RobotSettings/SelectNetwork/ConnectForm.js

[ ]  app/src/components/AppSettings/AddManualIp/ManualIpForm.js

[ ] app/src/components/ConfigurePipette/ConfigForm.js

Labware Creator
It's just a big Formik form!

[ ] labware-library/src/labware-creator/index.js

Protocol Designer

[ ] protocol-designer/src/components/FilePage.js -- File Page's file fields
[ ] protocol-designer/src/components/LiquidPlacementForm/LiquidPlacementForm.js -- adding liquids to a labware
[ ] protocol-designer/src/components/LiquidsPage/LiquidEditForm.js -- creating/editing liquids in the Liquids tab

This is looking good to me so far, and would be quite helpful for the rewrite of ConnectForm we're trying to land as part of the Wi-Fi disconnect push. Would be great if we could validate and get this in asap

authors: IanLondon mcous

repo: Opentrons/opentrons